TaskbarX incorrectly centering or completely hiding all icons.

This issue happens very often. Usually occurs when opening new applications and it needs to auto-center. Once you open the new application is will either not center and just default to original Windows location, or it will make all icons and opened programs completely disappear like it's been shoved off to the side.
Here is my system specifications, TaskbarX information, and other information that may be helpful:
Version For TaskbarX: <IP_ADDRESS>
Windows Edition: Home
Windows Info: Windows 10 v2004
RAM: 4GB
Processor: AMD Ryzen 3 3300U with Radeon Vega Mobile Gfx, 2100 Mhz, 4 Core(s), 4 Logical Processor(s)
Build Version: 10.0.19041 Build 19041
Type: x64-Based
This has is a re-occuring problem that I run into when opening any application. Restarting TaskbarX does nothing. Restarting Windows results in the same auto-centering issue. Here is a few information on my TaskbarX configuration settings:
Primary Taskbar Offset: 0px
Secondary Taskbar Offset: 0px
Skip Resolution: (0px, 0px, 0px)
Compensate Tray Clock: Disabled
Don't Center Taskbar: Disabled
Revert To Zero Beyond Tray: Enabled
Pause Loop When Fullscreen: Enabled
Update Taskbar Toolbars: Enabled
Refresh Rate: 400ms (Default?)
Rest of the settings are default and haven't been touched in the "TaskbarX Configurator" program. Uninstalling and Re-Installing TaskbarX still results in the same problem after a few restarts. Here is a few more information for the guidelines you provided:
Store or Regular Version: Regular
.NET Framework 4.8 Installed?: Yes.
Any Anti-Virus Software: Yes. Windows Defender, Malwarebytes. None which flag TaskbarX.
Third Party Tweakers?: No.
Reproduce?: Yes. Enable my settings and try opening some programs. Centering works but after a few opening programs it will start glitching out and failing to center properly.
Error Message?: None
The priority for this issue is low. Most users may not run into this problem. It might be harder to reproduce for some users.

@noahhx1 for better performance TaskbarX calculates the width of the taskbar by looking for the first icon's left position and last icon's right position in the taskbar.
This can be tricky if the loopers refresh rate is set to fast. So let's try it the old way i did with FalconX. In <IP_ADDRESS> pre-release it goes through every icon inside the taskbar and add up the total width. This way taskbarX can just request the width of an icon instead of it's position which is more stable while the taskbar is moving or doing other stuff.
https://github.com/ChrisAnd1998/TaskbarX/releases/tag/<IP_ADDRESS>

i have the same problem :((((((((((((((((((

Windows Pro 10 20H2 19042.867
Version For TaskbarX: <IP_ADDRESS>
I have the same problem

I have the same problem.
TaskbarX <IP_ADDRESS>.
Windows 10 Pro (OS build 21354.1)

To be able to focus on major/current bugs I had to close all issues. The issues section has gotten very messy to a point that I couldn't follow it anymore. Most issues are duplicates, very old or are already fixed/solved. Feel free to reopen this issue if you still have this issue on the latest version of TaskbarX to get my attention. All feature requests have been put in a list here: 💡| Feature requests #870
I hope you understand ❤


it can not successed run code in debug mode in windows, it  successed  in release mode

process didn't exit successfully: target\debug\tomotroid.exe (exit code: 0xc00000fd, STATUS_STACK_OVERFLOW)

Hi Thanks for reporting this. I actually was aware of this, but never documented it (as I should have). Unfortunately it is an upstream issue, and I haven't spent a lot of time on it. Based on Slint issue #3375, it's possible it can be resolved by increasing the Windows stack limit in the MSVC linker. I hadn't bothered to try changing the stack limit yet, but I just did and it appears to work.
So my quick test was I created a .cargo/config.toml, and added the below code increasing the Windows stack from 1mb to 2mb:
[target.x86_64-pc-windows-msvc] rustflags = [     "-C", "link-arg=/STACK:2000000" ]
and sure enough it compiled and ran.
Possibly Relevant Slint Issues / Discussions:

https://github.com/slint-ui/slint/issues/133
https://github.com/slint-ui/slint/issues/3375
https://github.com/slint-ui/slint/issues/4586
https://github.com/slint-ui/slint/discussions/5058.

This has been resolved with the addition of the config.toml increasing the stack size on windows (commit 7cc5e7a)


Roll Flutter Engine from e916d5bfddba to 6bdb41641ef8 (1 revision)

https://github.com/flutter/engine/compare/e916d5bfddba...6bdb41641ef8
2023-11-20<EMAIL_ADDRESS>Roll Skia from 343b249b2cac to 5010d0578f4d (1 revision) (flutter/engine#48215)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jonahwilliams@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Auditability / Cheating

Just a small thought after seeing this project for the first time: The name of this crate implies that the embedded information allows ensuring that no vulnerable dependencies were used. However, if I'm not mistaken, it would be very simple to overwrite this section of the binary, and simply lie about the actual dependencies used, right? (For example, to change versions, or to hide dependencies with "unsuitable licenses".) So while this section makes it easier to find vulnerabilities in your own binaries, it does not make your binary that much more auditable towards third parties that don't trust you.
Maybe a FAQ section in the README ("Can I be sure that the dependency information embedded in the binary is correct?") would make sense. Or maybe "cargo auditable" isn't the best name? (But you've probably thought more about this than I have. Related to #1.)

That's true, but if an attacker can modify your binaries, you have way bigger problems than potentially vulnerable dependencies. Your binary is probably stuffed with malware!
Perhaps I should add an entry to the FAQ?

Hm, that (an attacker modifying my binary) wasn't the case I meant... Instead, I meant a company saying "here are our binaries, you can audit them", who have tampered with their own binaries to make their binaries look safe (or GPL free) while they aren't.

Oh! A company can give you anything if your builds aren't reproducible, the binary is just a black box then.
This again is something that applies not just to audit data but also to the rest of the binary - the much more interesting use is injecting malware!
Reproducible builds are the solution to that, and the format is explicitly designed to avoid disrupting them, but cargo auditable doesn't provide that itself.

Yep, that's my point 🙂
The thing is, if you get the project source, and run cargo audit on it, you can be sure that vulnerable dependencies in it will get caught. If you get a binary, you can't be (unless it's reproducible). That's the point I'm trying to make, the FAQs should probably point it out (in case people assume that "cargo auditable" makes a binary auditable).

I'm adding a related clarification to the README in #109

Was the clarification in https://github.com/rust-secure-code/cargo-auditable/pull/109 sufficient? Can I close this, or is there something else to be done here?

I think so, yes! Thank you.


Supplying altitude in Location causes ColumnValidationException or does not arrive at all

Describe the bug
Supplying altitude in Location causes ColumnValidationException. Example from Crash:
cr> select * from  "mttest31"."etthing";
+-----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+-------------+
| __original_ngsi_entity__                                                                                | entie_index |
+-----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+-------------+
| {"data": {"id": "test", "location": {"type": "geo:json", "value": {"coordinates": [51.716785, 8.752131, 11.0], "type": "Point"}}, "location_centroid": {"type": "geo:point", "value": "8.752131611, 51.716783624"}, "time_index": "2020-11-ption[ColumnValidationException: Validation failed for location: Cannot cast {\"coordinates\"=[51.716783624, 8.752131611, 11], \"type\"=\\'Point\\'} to type geo_shape]')", "failedBatchID": "a54a4481c4844272931574c7f13100e0"}       | test1190952 |

from QL logs:
crate.client.exceptions.ProgrammingError: SQLActionException[ColumnValidationException: Validation failed for location: Cannot cast {"coordinates"=[51.716783624, 8.752131611, 23], "type"='Point'} to type geo_shape]

To Reproduce
Steps to reproduce the behavior:

create a sub from Orion to QL
Send data to Orion. Payload example:

{
	"id": "test",
	"location": {
		"type": "geo:json",
		"value": {
			"coordinates": [51.71678362411, 8.7521316110816, 23],
			"type": "Point"
		}
	}
}

Expected behavior
Elevation is inserted or Documentation is updated stating that Elevation cannot be used.
Environment (please complete the following information):
Data comes from Orion (version 2.4.2.) to QuantumLeap  & Crate (3.3.5) via subscription.
Additional context
Tested with 2 versions: 0.7.5 where data does not arrive to Crate DB and with smartsdk/quantumleap:redis-cache it arrives partially, as shown above. https://tools.ietf.org/html/rfc7946 states that elevation MAY be included.

@ilarimikkonen cratedb does not support 3d coordinates, as the error in the log states:
crate.client.exceptions.ProgrammingError: SQLActionException[ColumnValidationException: Validation failed for location: Cannot cast {"coordinates"=[51.716783624, 8.752131611, 23], "type"='Point'} to type geo_shape]

Expected behavior
... or Documentation is updated stating that Elevation cannot be used.

Yep, that's a sore point. Generally speaking, I feel we haven't done a good job about spec'ing what's an "acceptable" input, e.g. in your case, like you rightly point out, there should be a some kind of description of what types the notify endpoint can digest. To be fair tough, I think this is part of a broader problem with NGSI v2 which defines a generic data model of sorts (entity/attrs) that in practice isn't really useful for interoperability since at the end of the day JSON provides already a generic data model and communicating parties will still have to agree on the actual format of the data to exchange as well as on its meaning.

Documentation actually provides this information in relation to CrateDB:
https://quantumleap.readthedocs.io/en/latest/admin/crate/

CrateDB is a simple to use database backend for many applications. Nowadays a huge percentage of data is geo-tagged already. CrateDB can be used to store and query geographical information of many kinds using the geo_point and geo_shape types. With these it is possible to store geographical locations,ways, shapes, areas and other entities.These can be queried for distance, containment,intersection and so on. Currently, CrateDB supports 2D coordinates but it does not supports 3D coordinate

@ilarimikkonen given that the crate limitation is actually documented, anything else you expect from us?

nope, this is perfect. Thanks!

[Crate] Documentation actually provides this information in relation to CrateDB:

The point I was trying to make is that we don't but also have no easy way of doing it :-)
Our spec basically says anything with an entity ID and type can be shovelled down the notify endpoint which isn't true as @ilarimikkonen has learnt the hard way :-) Now you could say, well you can pass in whatever you like as long as the record r resulting from QL transforming the input JSON e into tabular format can be inserted into the table T QL generated for the specified entity type. Which is what you suggest in referring @ilarimikkonen to the Crate docs? We could update the spec along these lines, but I don't think it'd help since we'd have to clarify how r and T depend on  e. In turn this isn't an easy feat to accomplish because NGSI v2 says nothing about what kind of types (format + meaning) clients can exchange...


Footer test

What does this PR do?

sends routeType props to footer and added unit testing for footer

has one mistake = when navigating tabs with the footer the navbar doesnt update its activeTab (PROPOSAL TO CORRECT THIS  ON LAST COMMIT -> THIS IMPLY CHANGES IN NAVBAR (and navbar.test xd) )

Related issues
#31
Merge Checklist:

[ ] The branch is up-to-date (i.e. rebased) with the base branch
[ ] No code style issues (eslint and flow)

For this one:

Remove that line, and leave the function simply as a normal function.
For this one:

Add typing annotation, you can check Contact or any other updated component to check how it works.

Thank you again Juarez for this. On my side it's a LGTM once the todo is added, but it depends on @RicardoChapaRomero

The links get on top of the logo on medium view, please fix the responsiveness for this case.

The links get on top of the logo on medium view, please fix the responsiveness for this case.

The links get on top of the logo on medium view, please fix the responsiveness for this case.

i didn´t get to see this case on my local machine, however i did see some overlapping so i changed things a bit, please tell me if everything is working correctly


[matio] fix inter feature dependencies

Fixes matio[core,hdf5]:
CMake Error at cmake/src.cmake:80 (target_link_libraries):
  Target "matio" links to:

    MATIO::ZLIB

  but the target was not found.

and
Fixes matio[core,mat73]:
CMake Error at cmake/thirdParties.cmake:68 (message):
  MAT73 requires HDF5
Call Stack (most recent call first):
  CMakeLists.txt:29 (include)

Features matio[core,hdf5] and matio[core,mat73] passed with following triplets:
x86-windows
x64-windows
x64-windows-static


Update Stormpot dependency from 2.4 to 2.4.2

The new version has some bug fixes, and reduced CPU usage from the background thread.

Thanks!


(Kind of) Windows Port

I (kind of) wrote a windows port that uses AutoHotkey (windows only program) to have a similar effect.

Thanks! Feel free to add it to the community directory.


Error loading temp table data within a sql transaction

Hi again,
Sorry, I wish that I didn't have to create these issues and that everything worked fine.
What I am trying to do? Within a sql transaction:

Get rows to process <- this has to happen within the transaction
Process rows and gather data in memory
Save data in temp tables
Call stored procedure to merge the data from the temp tables into the real tables

The problem that I encountered is when I save the data to the temp tables:
System.InvalidOperationException: ExecuteNonQuery requires the command to have a transaction when the connection assigned to the command is in a pending local transaction.  The Transaction property of the command has not been initialized.
at Microsoft.Data.SqlClient.SqlCommand.ValidateCommand(Boolean isAsync, String method)
 at Microsoft.Data.SqlClient.SqlCommand.InternalExecuteNonQuery(TaskCompletionSource`1 completion, Boolean sendToPipe, Int32 timeout, Boolean& usedCache, Boolean asyncWrite, Boolean inRetry, String methodName)
 at Microsoft.Data.SqlClient.SqlCommand.ExecuteNonQuery()
 at Facil.Runtime.CSharp.GeneratedCodeUtils.LoadTempTables(SqlConnection conn, IEnumerable`1 tempTableData) in C:\projects\facil\src\Facil.Runtime.CSharp\GeneratedCodeUtils.cs:line 48
 at Facil.Runtime.CSharp.GeneratedCodeUtils.ExecuteQueryEager[T](SqlConnection existingConn, String connStr, Action`1 configureNewConn, Action`1 configureCmd, Action`1 initOrdinals, Func`2 getItem, IEnumerable`1 tempTableData) in C:\projects\fac
il\src\Facil.Runtime.CSharp\GeneratedCodeUtils.cs:line 150

Basically this code ends up  being called and it blows up when it tries to execute cmd.ExecuteNonQuery()
        private static void LoadTempTables(SqlConnection conn, IEnumerable<TempTableData> tempTableData)
        {
            foreach (var data in tempTableData)
            {
                using var cmd = conn.CreateCommand();
                // Note: If there is ever a need for letting users configure the command,
                // do not use the configureCmd parameter passed to methods on this class,
                // which also sets any parameters.
                cmd.CommandText = data.Definition;
                cmd.ExecuteNonQuery();

                using var bulkCopy = new SqlBulkCopy(conn) { DestinationTableName = data.DestinationTableName };
                data.ConfigureBulkCopy(bulkCopy);
                var reader = new TempTableLoader(data.NumFields, data.Data);
                bulkCopy.WriteToServer(reader);
            }
        }

Any idea on how to workaround this? I cannot move the processing of the data and the saving of the processed data into the temp tables outside the transaction.
Thanks

Ok, I ended up writing my own bulk copy code based on the Facil code. BulkCopyTempDataLoader is based on the TempDataLoader. I would have used it by it is marked as internal.
module ProcessingUtils

open System.Data
open Microsoft.Data.SqlClient
open FSharp.Data
open Facil.Runtime.CSharp

(*

        private static void LoadTempTables(SqlConnection conn, IEnumerable<TempTableData> tempTableData)
        {
            foreach (var data in tempTableData)
            {
                using var cmd = conn.CreateCommand();
                // Note: If there is ever a need for letting users configure the command,
                // do not use the configureCmd parameter passed to methods on this class,
                // which also sets any parameters.
                cmd.CommandText = data.Definition;
                cmd.ExecuteNonQuery();

                using var bulkCopy = new SqlBulkCopy(conn) { DestinationTableName = data.DestinationTableName };
                data.ConfigureBulkCopy(bulkCopy);
                var reader = new TempTableLoader(data.NumFields, data.Data);
                bulkCopy.WriteToServer(reader);
            }
        }
*)

type BulkCopyTempDataLoader(fieldCount: int, data: seq<obj array>) =

      let enumerator = data.GetEnumerator()

      interface IDataReader with
        member x.Read() = enumerator.MoveNext()
        member x.FieldCount with get() = fieldCount
        member x.GetValue i = enumerator.Current.[i]

        member x.Close() = failwith "not implemented"
        member x.Dispose() = ()
        member x.GetBoolean i = failwith "not implemented"
        member x.GetByte i = failwith "not implemented"
        member x.GetBytes(i, fieldOffset, buffer, bufferoffset, length) = failwith "not implemented"
        member x.GetChar i = failwith "not implemented"
        member x.GetChars(i, fieldOffset, buffer, bufferoffset, length) = failwith "not implemented"
        member x.GetData i = failwith "not implemented"
        member x.GetDataTypeName i = failwith "not implemented"
        member x.GetDateTime i = failwith "not implemented"
        member x.GetDecimal i = failwith "not implemented"
        member x.GetDouble i = failwith "not implemented"
        member x.GetFieldType i = failwith "not implemented"
        member x.GetFloat i = failwith "not implemented"
        member x.GetGuid i = failwith "not implemented"
        member x.GetInt16 i = failwith "not implemented"
        member x.GetInt32 i = failwith "not implemented"
        member x.GetInt64 i = failwith "not implemented"
        member x.GetName i = failwith "not implemented"
        member x.GetOrdinal name = failwith "not implemented"
        member x.GetSchemaTable() = failwith "not implemented"
        member x.GetString i = failwith "not implemented"

        member x.GetValues values = failwith "not implemented"
        member x.IsDBNull i = failwith "not implemented"
        member x.NextResult() = failwith "not implemented"
        member x.Depth with get() = 0
        member x.IsClosed with get() = failwith "not implemented"
        member x.RecordsAffected with get() = failwith "not implemented"
        member x.Item
            with get (name: string) : obj = failwith "not implemented"
        member x.Item
            with get (i: int) : obj = failwith "not implemented"

let bulkCopyData (cn: SqlConnection) (tran: SqlTransaction) (configCmd: SqlCommand -> unit) (tempTableData: #seq<TempTableData>) =
  tempTableData
    |> Seq.iter (fun data ->

      use cmd = cn.CreateCommand()
      configCmd cmd

      cmd.CommandText <- data.Definition
      cmd.ExecuteNonQuery() |> ignore

      use bulkCopy = new SqlBulkCopy (cn, SqlBulkCopyOptions.Default, tran)
      bulkCopy.DestinationTableName <- data.DestinationTableName
      bulkCopy.BatchSize <- 10000

      use reader = new BulkCopyTempDataLoader(data.NumFields, data.Data)
      bulkCopy.WriteToServer(reader)

    )

Any other ideas? If not, you can close this. Also if you want to use the code above, you can go ahead.
Thanks

Sorry, I wish that I didn't have to create these issues and that everything worked fine.

No worries, just means that either there are bugs, or you're using Facil in ways I have not intended and which may or may not fall within the project's scope. Let's try to figure that out.

What I am trying to do? Within a sql transaction:

Get rows to process <- this has to happen within the transaction
Process rows and gather data in memory
Save data in temp tables
Call stored procedure to merge the data from the temp tables into the real tables

Can't all of this be done in a single SQL script (which, as the last step, invokes the desired procedure)?
If not, I need a minimal repro so that I see exactly what you are trying to do. I realize the DB part can be hard in a minimal repro, but if you at least can post a project containing the generates code, and your code that invokes it, that's very helpful. Even more so if you have a separate script to generate the tables, so I can test myself. Keep in mind I'm asking for a minimal repro, so remove anything that's not strictly required to reproduce.

I need a minimal repro so that I see exactly what you are trying to do.

https://github.com/costa100/TestFacilTransaction
I created a test case that reproduces the error that I experience in my real project. I hope this helps. I don't think the code should fail if I execute the operations inside a transaction.

It would also be nice if Facil can support out-of-the-box inserting the data in the temp table and selecting the data from the temp table as two individual operations instead of supporting only insertion of data and selection as a single operation.

I added one more test case related to the previous comment.

Thank you again for making this changes.
One more question - is there any chance that you might make the bulk copy operation into a standalone action that won't invoke the retrieval of the data from the temp table?
Thanks!

Could you explain a use-case for that, that is hard/cumbersome/impossible with the current functionality?

In my use case, I validate and extract data (coming from an external system) from a single table. The table has varchar(max) fields with json arrays which I have to save in different tables.  I save the data in temp tables, but I don't need to select back from those tables. Then I call a stored procedure that moves the data from the temp tables into the final tables. The stored procedure does a bunch of merges and additional processing.
I got the whole thing working as you can see in this project: https://github.com/costa100/TestFacilTransaction/tree/master/TestTempTable.
I thought it would be nice if your library offered the functionality out-of-the-box.

I think I finally understand. It seems to me that this can be solved if you can specify temp tables for stored procedures just like you can with scripts. That should be fairly simple. I'll start working on it.
Just for the record, here are some possible workarounds:

Insert into the tables using a dummy script with just e.g. SELECT 1 (no need to select from the temp tables, or even reference the temp table in the script; Facil just uses the temp table information in the YAML file). After calling this script, call the stored procedure using Facil.
Have a script that you configure with temp tables as above, and which (in the SQL script) invokes the stored procedure. You may lose some parameter type inference, though.

v2.3.0 should be out shortly with support for temp tables in procedures, just like in scripts.

Hi, thank you 🙏, I just rewrote the code to use temp tables with the stored procedure within a transaction and it works!

Happy to hear it!


Improve table and column descriptions

@AayushAnand007 and @Alifath57  please look at the table and columns descriptions in SchemaSpy or in Metabase and list in this ticket what descriptions need to be added/edited.


ARROW-2707: [C++] Add Table::Slice

Allow zero copy slices on Tables

Looks spurious: https://travis-ci.org/apache/arrow/jobs/529571148#L1183

@xhochy rebased but still red

Failures are from https://github.com/pypa/setuptools_scm/issues/333 still, it appears.

thanks @bkietz!


Chroniques Galactiques (new french sheet) + Chroniques Oubliees v1.4

New character sheet  for "Chroniques Galactiques" (french scifi RPG).
And adjustments for Chroniques Oubliées v1.4

Does this new sheet fall under a specific rpg system, for grouping purposes? (Like all the Dungeons & Dragons games are grouped together) I usually look this up myself, but since this is a French game and I don't speak French I am not able.

Yes in fact it's the same base rules as Chroniques Oubliées (already a sheet for that on Roll20 that I've done.

What is the name of the system? So I can group them together.

"Chroniques Oubliées" in fact.
The first sheet (named Chroniques Oubliées) was for "Chroniques Oubliées Fantasy" (if you can rename it). I should have named it this way at first.

Yup, I can do that. Thanks for the info, and for the quick response.

You're welcome and thanks for paying attention to this kind of details!


update Construct

SteamDB app page links to a few games using this

Antenna

Brief explanation of the change
added construct2_license.txt to the rule, I don't know if this file is worth adding but I assume some other games have it too

After running changed rules in this pull request (this bot only tests rules.ini changes):
Changed technologies: Engine.Construct
357 apps still match.
These apps now match: (1) 443580
Please confirm that all these changed apps are correct.

alright, that's not worth adding I assume


Animated Images Support

Currently, animated images (WebP, GIF) are turned into static images.
Based on https://sharp.pixelplumbing.com/api-constructor, it looks like animated images support could be enabled by simply adding { animated: true } to https://github.com/Niels-IO/next-image-export-optimizer/blob/ddb110c2b0715996fd67a6ac7d731927cf4e7a20/src/optimizeImages.js#L417
It would be awesome if this functionality could be exposed through an environment variable.
Note that next/image claims to bypass animated images.

Decision could also be made based on the source image extension (as read in L331) , to set {animated: true } for webP, GIF only

Thanks for the example of using unoptimized.
In my own experiments, adding animated to your code seems to work perfectly, albeit with very slow performance:
https://github.com/dli7319/next-image-export-optimizer/commit/8741983c1345a1000a70fc5e12a155a12b699916
The 67.4 MB GIF gets converted properly into smaller animated WebP images

E.g. 720p_gif-opt-750.WEBP
Nevertheless, it would also be nice if unoptimized could be automatically applied for animated images like in next.js in case someone doesn't want animated images to be optimized:
https://github.com/vercel/next.js/blob/1c5983fe95b79cc9ea1e52d19e69adf7827eda5a/packages/next/src/server/image-optimizer.ts#L589

Interesting that it works for you. When I only change this line in the code
const transformer = sharp(imageBuffer, { animated: true });
the generated image is not animated anymore. Did you do something else that I am missing?
Regarding your second point: The unfortunate thing is that the isAnimated package is a server package and on the client side where I would need to set the unoptimized prop, we cannot run such a check.

I put all the changes I made in that commit and just ran optimize_gif.sh so I have no idea why it wouldn't work for you as well.
Sharp is installed from npm ci so it should match the version in your package-lock.json.
FWIW, I'm testing on Ubuntu 22.04.

I just tested using your static noise gif and animated png (https://github.com/dli7319/next-image-export-optimizer/commit/39af8a663e0b14960296c07c2cbdfd15691c4c13).
The noise GIF works: https://raw.githubusercontent.com/dli7319/next-image-export-optimizer/39af8a663e0b14960296c07c2cbdfd15691c4c13/public/images/nextImageExportOptimizer/402107790_STATIC_NOISE-opt-256.WEBP
The animated PNG becomes static: https://raw.githubusercontent.com/dli7319/next-image-export-optimizer/39af8a663e0b14960296c07c2cbdfd15691c4c13/public/images/nextImageExportOptimizer/animated-opt-256.WEBP

Thanks, I just came to the same conclusion. I was irritated as the webp version did not animate in the file preview, but they do in the browser.
Also, the WEBP animated image works. The issue with the apng seems to be described here: https://github.com/lovell/sharp/issues/2375


What is the size of the raw data set?

I just finished downloading the nyc data set using ./download_raw_data.sh . It's about 55GB. I wonder whether it's the correct size.

@amosbird that sounds too small. In my experience it's something more in the neighborhood of 200GB. Can you check the list of files that you downloaded and compare to the list here: https://github.com/toddwschneider/nyc-taxi-data/blob/master/raw_data_urls.txt
There should be 161 files total

Ok, thanks for the info. Yeah, I saw a bunch of incomplete data files.

Given file has been moved to https://github.com/toddwschneider/nyc-taxi-data/blob/master/setup_files/raw_data_urls.txt


OSX build out isn't the same

So previously I could do:
$ electron-packager ./ 'Lowbrow' --platform=darwin --arch=x64 --out=releases/build/osx/x64

And it would create releases/build/osx/x64/Lowbrow.app. Now it creates:
releases
└── build
    └── osx
        └── x64
            └── Lowbrow-darwin-x64
                ├── LICENSE
                ├── Lowbrow.app
                └── version

Oddly this is also not in line with the linux build:
lowbrow
├── LICENSE
├── Lowbrow
├── content_shell.pak
├── icudtl.dat
├── libffmpegsumo.so
├── libgcrypt.so.11
├── libnode.so
├── libnotify.so.4
├── locales
├── natives_blob.bin
├── resources
├── snapshot_blob.bin
└── version

I mean the linux one makes sense, since there's no "self contained linux package format". What happened to the previous way? In addition, why the name convention?

Also I just notice that the linux build folder and mac build folder are now different.

Is the behavior consistent with the outcome of #38?

I don't understand @malept.

--out and its default behavior changed when #38 was closed (particularly with regards to Linux). Is what the discussion in that issue, plus what Max did in 86edff6 consistent with the the output directory behavior that you're seeing?

This change has to do with --all (and support for building multiple platforms/archs from one command) being added.  For consistency's sake, all builds now result in a top-level directory whose name includes the platform and arch.
This does result in extra nesting of the Mac app, which I realize could be considered a bit awkward, but the same process is now being followed for all platforms, and results in consistent naming of output folders when building multiple platforms at once (e.g. myapp-darwin-x64, myapp-linux-x64, myapp-windows-x64).


Some plugins does not work after latest update

I'm using erela.js-deezer and erela.js-filters and does not work after latest update.
C:\Users\emirk\Projects\kabalsbot\bot\node_modules\erela.js\dist\structures\Manager.js:102
                    throw new RangeError(`Plugin at index ${index} does not extend Plugin.`);
                          ^
RangeError: Plugin at index 2 does not extend Plugin.
    at new Manager (C:\Users\emirk\Projects\kabalsbot\bot\node_modules\erela.js\dist\structures\Manager.js:102:27)
    at exports.default (C:\Users\emirk\Projects\kabalsbot\bot\src\utils\MusicOptions.ts:12:19)
    at new CustomClient (C:\Users\emirk\Projects\kabalsbot\bot\src\modules\CustomClient.ts:60:30)
    at Object.<anonymous> (C:\Users\emirk\Projects\kabalsbot\bot\src\bot.ts:5:16)
    at Module._compile (node:internal/modules/cjs/loader:1126:14)
    at Module.m._compile (C:\Users\emirk\Projects\kabalsbot\bot\node_modules\ts-node\src\index.ts:1618:23)
    at Module._extensions..js (node:internal/modules/cjs/loader:1180:10)
    at Object.require.extensions.<computed> [as .ts] (C:\Users\emirk\Projects\kabalsbot\bot\node_modules\ts-node\src\index.ts:1621:12)
    at Module.load (node:internal/modules/cjs/loader:1004:32)
    at Function.Module._load (node:internal/modules/cjs/loader:839:12)

Solution for now:
I fixed this problem with downgrading erela.js #2e2e65cd9a
yarn add MenuDocs/erela.js#2e2e65cd9a or npm install MenuDocs/erela.js#2e2e65cd9a

i do not use erela but this issue is not an erela issue. the problem is that erela.js-deezer and erela.js-filters haven't been updated in months and therefore uses outdated versions of erela

i do not use erela but this issue is not an erela issue. the problem is that erela.js-deezer and erela.js-filters haven't been updated in months and therefore uses outdated versions of erela. whether my statement is true or not, i personally recommend u ask the plugin authors to update their packages.

Thank you for your response


node-typescript-app skaffold and helm config

This currently does not work, but I created a issue on the skaffold repository:
https://github.com/GoogleContainerTools/skaffold/issues/961

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

I opened a separate pull request(#24) for the yarn cache so we can fasttrack that.
I have also been able to get helm and skaffold to work together. It's not great because the default hem configuration separates out the image name and tag, and skaffold relies on finding the full image-name:tag in the values so that it can identify the tag to associate with the image name. Probably won't be able to get around that till reusing tagPolicy variables is implemented. There is someone having a similar issue here.

This PR looks good. I don't mind the workaround for now.

I think this should be good to go for now :shipit:


Added 2 new ECSQLOPTIONS

@RohitPtnkr1996 I re-implemented this feature with a different approach in https://github.com/iTwin/imodel-native/pull/446


Update @fastly/js-compute to 0.3.0

Bumps @fastly/js-compute to the lastest version, 0.3.0: https://www.npmjs.com/package/@fastly/js-compute
Removed --skip-pkg argument from the npm build script

thank you for this.


Component overview and required updates

Summary | Résumé
Move the "Components tags" section below the list of components
Move the sentence "We're sharing..." below the "Browse components" section
Add Github and Figma links
Change badge text
Add description for required components to each component page with a required component

Closing this PR as this work was done in another PR.


Flower basic auth

Can one of the admins verify this patch?

Codecov Report

Merging #113 into master will not impact coverage.

@@          Coverage Diff           @@
##           master    #113   +/-   ##
======================================
  Coverage    93.8%   93.8%
======================================
  Files          28      28
  Lines        1017    1017
  Branches       46      46
======================================
  Hits          954     954
  Misses         55      55
  Partials        8       8

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 20d9090...d328f11. Read the comment docs.


How can I use the second DFI-Adress in my wallet to send DFI

Actual behavior: Can't send DFI from the second address in the wallet
Expected behavior: Can send DFI from the second address in the wallet
How to reproduce it (as minimally and precisely as possible):
When receiving DFI, you can easily select your address:

When sending DFI, however, you cannot select a sending address and the first address is selected by default.

I haven't found a function anywhere to change the default address. Please help! Thank you very much.
What are your environment parameters?

DeFiCh/app Version:
Operating System (OS):
– RAM (GB):
– CPU (Intel Core i5/i7/etc.):
Current free storage capacity:

Anything else we need to know?

OK, I saw it was related to problem #913 that I couldn't send anything specifically from my second address. Now I have noticed that the credit from several addresses is combined when sending. But as an improvement for the app, it would make sense to be able to choose exactly which account you want to send DFI from. For example, if you want to separate several accounts for family members or private and business accounts. Thank you.

Hi @SKapi1985 , your feedback has been conveyed to the team, however I do not have a timeline on whether and when this would be implemented. Just wanted to update you before this thread becomes stale. Rest assured that it'll be considered by the team. There are plans for multi addresses — like in the comment you mention. If you want a quicker access to this feature, I would suggest DeFiChain Light Wallet. You can download from the app store, Create Wallet then transfer you funds to continue where you left off.
Do note that the Desktop app and Light Wallet are built differently, hence you won't be able to Restore via Wallet Backup on the mobile wallet. Thank you for your patience.


unable to enable basic auth

> winrm set winrm/config/client/auth @{Basic="true"}
Error: Invalid use of command line. Type "winrm -?" for help.

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

ID: 65e527db-a4c2-782e-5a5f-ad8b3159a977
Version Independent ID: f2de1272-5e37-c59f-fcb9-34fa66b0d268
Content: Connect to Exchange Online PowerShell using multi-factor authentication
Content Source: exchange/docs-conceptual/exchange-online/connect-to-exchange-online-powershell/mfa-connect-to-exchange-online-powershell.md
Service: exchange-online
GitHub Login: @chrisda
Microsoft Alias: chrisda

A more 'PowerShell-friendly' approach is Set-Item WSMan:\localhost\Client\Auth\Basic -Value 'True', this requires administrative permissions


[Office365]  - Error logging into other accounts "a transport layer error occurred. Got a protocol error response:"

I am trying to use NotRuler to determine which of our mailboxes, if any, were compromised. I am our Office 365 administrator so I therefore have full access to all mailboxes in our tenant. I am able to run NotRuler with the - - self argument with no issues.
When I attempt to run NotRuler on mailboxes other than my own, I get the error "a transport layer error occurred. Got a protocol error response:". No error response number is returned.
I am running NotRule on a Windows 10 machine from an elevated command prompt. Please help as a security firm has confirmed that we were attacked by Ruler.

Hi! Thank you for the report.
I'll need to get setup on Office 365 again to try and debug. In the meantime, could you try the following two options:
not-ruler --rpc
and
not-ruler --debug
The first option will try switch to the older RPC transport, which might work. I have a feeling Office365 might be using cookie values for auth instead of only checking the MAPI data. The --debug might shed some light on that
I'll try get a test account setup asap to verify.

Thanks for your help staaldraad.

I've figured out where the issue is, now need to figure out a way around it without breaking things :)

This version will definitely try login to the second mailbox, however with my test accounts it is failing with RPC errors. Specifically this: https://www.codetwo.com/kb/mapi-e-logon-failed-0x80040111/ - which I don't think is the real issue but haven't been able to get around it.
Please see if this works for you:
notruler-win64.zip

Thanks staalraad. I tried the new package and it is giving a more descriptive error: "
ERROR: HH:MM:SS notruler.go:566 Looks like<EMAIL_ADDRESS>failed: mapi: Invalid login. Admin privileges requested but user is not an admin".
Do I have the arguments in the correct place? The --username argument is the admin user correct?

.

It is the same as what I'm running into. It seems like there is an issue in Office365 with the UseAdminPrivilege flag needed for this. The full error happening in the back-end is similar to this:
IMicrosoft.Exchange.Rpc.ServerUnavailableException: Connection must be re-established ---> Microsoft.Exchange.RpcClientAccess.ServerUnavailableException: Connection must be re-established ---> Microsoft.Exchange.RpcClientAccess.SessionDeadException: The
primary owner logon has failed. Dropping a connection. ---> Microsoft.Exchange.Data.Storage.ConnectionFailedTransientException: Cannot open mailbox /o=ExchangeLabs/ou=Exchange Administrative Group (FYDIBOHF23SPDLT)/cn=Recipients/cn=202a71c9a32d412cbbc6c9
a593. ---> Microsoft.Mapi.MapiExceptionLogonFailed: MapiExceptionLogonFailed: Unable to open message store. (hr=0x80040111, ec=-2147221231)

I'm trying to find a way around this. In theory it should work exactly the same as on-premise, but it seems like there might be some context switching issues that get introduced in multi-tenant systems like Office365.

Do you think the Powershell script would help us detect Ruler canaries? Remember, we are 100% Office 365.

If you are using Federated Auth, then I believe you may be able to detect the login events.
From Office365 side, I'm not sure what the logging capabilities offer, I have not had a chance to dig into it in detail. If you can see how often a user logs into a mailbox, Ruler activity should cause a small spike in login activity - as it does a "connect" - "make change" - "disconnect". Where-as normal Outlook behaviour is to keep a session alive.

I have a feeling this might work; It does seem to be a permission issue so this makes sense: https://social.technet.microsoft.com/Forums/ie/en-US/6b0d9dbe-8aa4-4cc7-82ae-79cb3cb00f99/majority-of-mailboxes-fail-to-migrate-to-exchange-online-cutover-migration-from-exchange-2010?forum=onlineservicesmigrationandcoexistence
Basically need to give the user extended rights to "receive-as":
Add-ADPermission -Identity "Mailbox Database<PHONE_NUMBER>" -User "iif.admin" -ExtendedRights Receive-As

Unfortunately in Office 365, I don't think we have access to the Mailbox Database. This is a big limitation. My company is 100% Office 365. No hybrid environment.

Hello staaldraad, please let me know if you can think of anything else for this issue. Thanks for your help.

Unfortunately I haven't had a chance to find a solution yet. It is on the stack :+1:

Thanks staaldraad.


Implement and test 'sixes'

Review 'sixes'

looks slow to me
git needs more sauce
return np.sum(dice[dice == 2])


Add direction to trains.json

It would be good to have a direction/course field (in degrees) in trains.json because it looks weird if a train icon is facing the wrong way on the map :)

I'll try and add that over the weekend.

Thanks! Trains will appear on https://map.events.ccc.de once they are running (I'm filtering out the demo train on the live map).


add(cvt): impl cvt for paddle

给paddle复现CvT论文。

非常棒！请问精度是否和官方对齐？我这边readme 没有看见相关的信息

我正在改这部分，目前基本对齐，还需要进一步改代码风格和对齐更多模型@defensetongxue

Now the cvt_13 ported weights obtains correct performance on Imagenet (81.60%).
There are still several issues should be finished before merging:

The implementations in cvt.py could be simplified, please refer to our ViT implementation (here)
Please convert more ported weights form the original repo and align the accuracies.
If possible, please also make the training settings same as the original paper.

ok,i'll simplify the model soon

I have  simplifed the code style and test the model cvt_13_384x384-1k(0.8275) and cvt_21_224x224-1k(0.8252).please check the code style if it can meet the requests ,thanks @xperzy

I have tested the performances and now the results are same as official repo.
So I merge this PR.


Process list doesn't show up in Windows 1903

Process list doesn't show up in Windows 1903. I've set the VM to run as root. Followed the other instructions in the thread and haven't had any luck.
xxx:~/Code/vmread/build$ sudo ./example
Using Mode: MODE_EXTERNAL
PML4:	1aa000	| KernelEntry:	fffff80242aca900
Kernel Base:	fffff80242000000 (2000000)
PsInitialSystemProcess:	fffff8024256f3a0 (256f3a0)
System (PID 4):	ffffd586c827d080 (17c27d080)
NT Version:	1000
Process List:
xxx:~/Code/vmread$ ./inject
[New LWP 3999]
[New LWP 4004]
[New LWP 4006]
[New LWP 4007]
[New LWP 4008]
[New LWP 4010]
[New LWP 4011]
[New LWP 4027]
[New LWP 4028]
[New LWP 4030]
[New LWP 4031]
[New LWP 4111]
[New LWP 4112]
[New LWP 4113]
[New LWP 4114]
[New LWP 4115]
[New LWP 4116]
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib/x86_64-linux-gnu/libthread_db.so.1".
0x00007fcccd118cf6 in __GI_ppoll (fds=0x55c62f550e00, nfds=76, timeout=, sigmask=0x0) at ../sysdeps/unix/sysv/linux/ppoll.c:39
39	../sysdeps/unix/sysv/linux/ppoll.c: No such file or directory.
[Thread 0x7fcb827fc700 (LWP 4115) exited]
[Thread 0x7fcb81ffb700 (LWP 4116) exited]
[Thread 0x7fcb82ffd700 (LWP 4114) exited]
[Thread 0x7fcb837fe700 (LWP 4113) exited]
[Thread 0x7fcb83fff700 (LWP 4112) exited]
[Thread 0x7fcb97cfc700 (LWP 4111) exited]
[Thread 0x7fcb95af5700 (LWP 4031) exited]
[Thread 0x7fcb963f7700 (LWP 4030) exited]
[Thread 0x7fcb96bf8700 (LWP 4028) exited]
[Thread 0x7fcb973f9700 (LWP 4027) exited]
[Thread 0x7fccbd7ff700 (LWP 4011) exited]
[Thread 0x7fccbe3ff700 (LWP 4010) exited]
[Thread 0x7fccbeffc700 (LWP 4008) exited]
[Thread 0x7fccbf7fd700 (LWP 4007) exited]
[Thread 0x7fccbfffe700 (LWP 4006) exited]
[Thread 0x7fccc07ff700 (LWP 4004) exited]
[Thread 0x7fccc4805700 (LWP 3999) exited]
[Inferior 1 (process 3986) exited with code 01]
The program being debugged exited while in a function called from GDB.
Evaluation of the expression containing the function
(__dlopen) will be abandoned.
$1 = "Injection failed!"
You can't do that without a process to debug.
The program is not being run.
I've checked /tmp/ permissions, library permissions as well.

Tried upgrading CEMU to a higher version. Inject still crashes.

This is a valid issue and should not have been closed. Version 1903 must have had its internal structures updated, thus an offset update is required. It has to be backwards compatible with the previous versions, thus version checking has to be implemented. That is available through ntBuild variable in WinCtx. However, it appears that this variable is never assigned a value. So, a function has to be implemented for that as well.

I have proposed a change in my pull request here https://github.com/Heep042/vmread/pull/6
I have updated the offsets for Windows 10 1903 and added manual version selection via user input from the terminal.

Thanks I appreciate it. As a workaround I'm just using 1803 for now.

Should be fixed in https://github.com/Heep042/vmread/commit/2cb22dfbd3f842f4f939d6c67ebb1760bd139550


Reduce use of with Interfaces

By subtyping Unsigned_32 the user do not need to with Interfaces when using To_Pixel.

Thanks Luke 💪


Split the builtin application distribution to 'gs-apps' package

What do these changes do?

[x] Add prebuilt frames/apps with graph has 'string' OID
[x] Split the builtin application distribution to 'gs-apps' package
[x] Precompiled  some common apps used in NetworkX
[ ] Support to compile app under GCC 4.8+ (New issue opened #1428)

Related issue number

Fixes #1283
Fixes #1292

Codecov Report

Merging #1429 (85ccd99) into main (47b0d42) will decrease coverage by 37.64%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main    #1429       +/-   ##
===========================================
- Coverage   77.85%   40.21%   -37.65%
===========================================
  Files          88       88
  Lines        9674     9674
===========================================
- Hits         7532     3890     -3642
- Misses       2142     5784     +3642

Impacted Files
Coverage Δ

python/graphscope/tests/unittest/test_app.py
0.00% <0.00%> (-100.00%)
:arrow_down:

python/graphscope/tests/unittest/test_lazy.py
0.00% <0.00%> (-100.00%)
:arrow_down:

python/graphscope/tests/unittest/test_context.py
0.00% <0.00%> (-100.00%)
:arrow_down:

python/graphscope/tests/unittest/test_session.py
0.00% <0.00%> (-100.00%)
:arrow_down:

python/graphscope/tests/unittest/test_download.py
0.00% <0.00%> (-100.00%)
:arrow_down:

python/graphscope/tests/unittest/test_java_app.py
0.00% <0.00%> (-100.00%)
:arrow_down:

...ython/graphscope/tests/unittest/test_cython_ast.py
0.00% <0.00%> (-100.00%)
:arrow_down:

...ython/graphscope/tests/unittest/test_serailaize.py
0.00% <0.00%> (-100.00%)
:arrow_down:

python/graphscope/analytical/udf/patch.py
3.47% <0.00%> (-96.53%)
:arrow_down:

...thon/graphscope/tests/unittest/test_scalability.py
0.00% <0.00%> (-96.16%)
:arrow_down:

... and 47 more

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 47b0d42...85ccd99. Read the comment docs.


Type inference fails when inferring type parameter of type argument

If a type parameter has a type parameter itself (e.g., <T extends Iterable<U>, U>), the inner type parameter is not inferred (U).
dev.66. I've attached a repro. I expected the output of this program to be List<String> String, but it is List<String> dynamic.
void main() {
  final object = getFirstObject(["string"]);
}

U getFirstObject<T extends Iterable<U>, U>(T iterable) {
  print("$T $U");
  return iterable.first;
}

The type inference solution of T=List<String> and U = dynamic is a correct inference for the program at hand. It's not necessarily the most precise solution, but it satisfies all the constraints that your program sets.
If you change final object = ... to final String object = ..., you will see that U is now inferred as String. That's because the only place U occurs in the signature of getFirstObject is in the return type, so in order to constraint the inferred type for U, you need to put a constraint on the return type.
I'm not sure this is actually an error, just not the result you expected. I'll leave it to the inference experts to say what this particular case should give. @leafpetersen ?


Add cask URL location to audit problems.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

Makes GitHub annotations use the correct line instead of all being at the top.

Thanks again @reitermarkus!


Add run build option to cog.yaml

The existing pre_install was intentionally left undocumented
because the name was not ideal. (What is it "pre"? What's
"install"?)
"run" seems apt, because it maps directly to the Dockerfile
instruction and it's "running" these things, not "installing" them.
Known issues:

This does not work with multiple lines.
There should be some method of getting files in that works
efficiently even when building base images. Maybe a run command
could say what files it needs copied?

I will create separate issues for these when this is merged.
Closes #159

Makes sense. In case we need to mention it anywhere, perhaps to "copy" code is better nomenclature, following the same logic in this change.

Known issues:
https://github.com/replicate/cog/issues/222
https://github.com/replicate/cog/issues/223


Is it possible to use face-api?

Question
Hey!!
I am also trying to use face-api with this library and this one:
https://github.com/mrousavy/react-native-blob-jsi-helper
In order to transform resized images buffers in blobs so In can send them to the face detection api from face api, is that possible?
What I tried

VisionCamera Version
3.9.1
Additional information

[ ] I am using Expo
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar questions in the issues page as well as in the discussions page and found none.

I realized that I do not need firebase cause MLKit is free.
But my other question is remaining, is it possible to use face-api?

Yes you can build a native Frame Processor plugin to use any native Obj-C/Swift/Java/Kotlin library.


Update clang and clang-format on CI?

It looks like the clang-format version used in CI is not very up-to-date and sometimes there are discrepancies between that version and more recent ones that we may use in development setups. The clang compiler version seems to be relatively old as well (6.0), which can be a problem when we have to deal with warnings/errors that may not be reproducible with more recent versions of the compiler. It would be great if we could update both tools in CI to a more recent (13?) version.

These are pretty separate decisions. With clang-format, we basically just have to pick one and pin it. The choice is pretty arbitrary, but if we want to do a later one, that's fine with me. With clang, we deliberately build with the oldest version of clang that we say we support. If we want to drop support for clang 6 explicitly, then we can have that conversation. It's clang-6 because that's the highest available for Ubuntu LTS release 18.04. Something I would like to do is have the latest version of clang in some builder as well so we get all the latest features covered

Keeping some instances of clang 6 and moving some of them to more recent versions sound great to me!

We've upgrade to clang-9 as our lowest supported version. I've filed https://github.com/google/iree/issues/9468 for also building with later clang versions on the CI. The clang-format version used is clang-format-9 as well: https://github.com/google/iree/blob/6a4c9df93f/.github/workflows/lint.yml#L97
Would you like to propose that we update it to a later version? No matter what version we use, there will be slight difference from whatever version someone has locally, unfortunately

Sure, I'm alright with that

Yeah, that sounds good to me. Version 12 is only one year behind and I would expect clang format to be relatively stable. Let's go with that!


Add UI mappers

WHAT
Add mappers to map models from the data layer into models from the UI layer.
WHY
Reduce amount of code in the view model that would have to map those models across layers.
Make it easier to unit test.
HOW

Add dependency on media-data and media3-common to media-ui;
Add PlayerUiStateMapper;
Add MediaItemUiModelMapper;
Add TrackPositionUiModelMapper;
Add unit tests;

Checklist :clipboard:

[x] Add explicit visibility modifier and explicit return types for public declarations
[x] Run spotless check
[x] Run tests
[x] Update metalava's signature text files

cc: @dpreussler


Basic docs

A few urgent doc requests. The urgency is because we would like new people to join the project, and it doesn't inspire a lot of confidence if these basic things don't exist.

[x] Expand README.md. It currently looks like a not fully baked project because it doesn't have a lot of content. But we actually have a lot of material in readthedocs which should be replicated here. At the minimum, we should say what the project is, what capabilities it plans to have, install instructions, contributor list.

[x] We need a prominent link back from readthedocs to the github page, probably at the top of all the pages, or at least the main page.

[x] Right now, the docs link on the main github page is not really prominent because because it's in the top right corner. This should also be linked at the top of the README as a badge.

Fixed by 27017018c82a011d33394010a3c35c6ca86a11c6, 36fd8549120cbbf4dfc6941523e0ce1d733d53c0, and b0acbd2482de5d646c100cb8ac808fea8dae13c3.


Add basic unit test to check conversion process

This uses the lstm functional unit test data.
It then creates an input file and compares to the downloaded one.
It then creates the network file using the new input file and compares
to the old one.
In each case a diff is performed on new an old. If it's empty the test
is successful.

Hmm, so this seems like a nitpick but I'm not super happy to add a 450 KB data file to the main repository. I know it's not huge, but the entire git repository is 504 KB at this point, i.e. it's doubling the size of the repository in one commit.

Just to elaborate a bit: would it be possible to check basic_lstm_network_output.json into the same repo where we put the rest of the data and then squash these commits? In general I think we should avoid using the data/ directory.

Yup, I agree with that and new this comment was coming :). I'll change it later today.


refactor(about): extract presentation page into its own comp

Extracts presentation page in its own component to make room for next sections

Current dependencies on/for this PR:

main

PR #49   👈

This comment was auto-generated by Graphite.


can't get this to work with oauth

I have successfully authenticated to BB with oauth.  I have 4 hashes:

consumer secret
consumer key
oauth token
oauth token secret

You  use the vocabulary:
config.oauth_key    = 'key'
config.oauth_secret = 'secret'
are these the consumer key and consumer secret?  If so, how do you use the access token?  I've tried every combo, I get a 401 every time.
Thanks for any help,
kevin

Examining your code, I believe you have a typo in your Readme, didn't have rights to issue a PR, but please change
bucket = Tinybucket.new do |config|
config.oauth_key    = 'key'
config.oauth_secret = 'secret'
end
to
bucket = Tinybucket.new do |config|
config.oauth_token    = 'key'           #<------------ change this line please
config.oauth_secret = 'secret'
end

Thanks for your report.
It is exactly a typo and I'll fix it.


aframe deploy command for publishing an A-Frame scene

to IPFS (distributed, permanent CDN)

\o/  this is so great!

thanks. merging 🛩


Change test script to interactive mode

At the moment, tests inside Guppy are running once and the user need to re-run manually. As @joshwcomeau mentioned in a comment that's a temporary hack.
It was a good idea from Josh to do this later as this is a pretty large task. But maybe two people could work on this.
One for implementing the UI with jest-stare and the other for parsing/handling the interactive commands. But it's also possible to tackle this with-out splitting - just do it like this: First UI then command handling or start with a minimal temporary UI and work on the business logic first then improve UI.
Is your feature request related to a problem? Please describe.
UX could be better if it would be possible to use interactive mode.
Describe the solution you'd like
By removing line 188 to 190 in task.saga. We're running in interactive mode and the output in a terminal will look like in the following screenshot (ran in a real terminal as the Guppy output seems to be filtered):

Todos
UI

[ ] Remove terminal from test preview as we're rendering everything to the UI
[ ] The status should change if the interactive test is passing to green & on fail to red.
[ ] Extract every info from the terminal output from top down to string Watch usage. Maybe we could also change to a json reporter so it's easier to get the output - not sure if this is working with interactive mode.
[ ] Render the test result - I think we could use jest-stare to render it.
[ ] Show progress bar & expected test time (optional, less important)
[ ] Create a UI element for each command:

[ ] Run all tests a key
[ ] Run only failed tests f key
[ ] Filtering p & t key with a input field
[ ] Stop tests q key
[ ] Re-run test enter key

[ ] Use the same keys as keyboard shortcuts so it's possible to trigger them with keyboard.

Command handling

[ ] Create a runTest generator in task.saga that's listening for a Redux action that will pass the  command to the test child process. So you have to use take(stdioChannel) and takeEvery(TRIGGER_TEST_COMMAND, handleTestCommand) inside a while(true) loop.
[ ] handleTestCommand will use sendCommandToProcess to send the command passed with the action TRIGGER_TEST_COMMAND to the test child process.

Edge cases

What happens if a snapshot needs to be updated? Parse the info from the output and display a button & also show a list of snapshot files that are not matching so they can be opened in editor. Jest-stare is already doing a nice diff display - maybe we could display this if we're detecting that a snapshot failed.

https://github.com/Raathigesh/majestic looks like a cool Jest front-end that fits better with Guppy's UI out of the box, perhaps another option to consider.

Good idea. Maybe we could use it as starting point, inspiration for the UI or directly integrate it. I like the navigation through the test files but I have some points I don't like:

No progressbar during test run - just indefinite spinners at each test suite (jest console output got a time & progress bar indication)
Takes pretty long to load (not measured but I think around 5-10 sec. for Guppy tests)

Guppy tests in Majestic

How would we add the testing pane? I think it would be good if we could have a second project page so we're getting more space for the UI. I don't like to add this into the modal or would this fit into the modal?
If we would use Majestic would we create a fork (so we can customize the UI & add features) and launch it with npx https://github.com/forked-majestic? Or should we add it as submodule and run it with-out npx (probably a better dev. experience)? Once the server is started, we could show the Majestic server output in an iframe.


Update pt-BR.json

Add line 22.

Will ping you when I am done with the next release, as I have some more options added.


init new components

Description
This is the initialization of our four new components.
References

NA

Blocked by

NA

N/A


Fix grammar in the readme

Simply updates some of the grammar in the README. If I missed anything suggest below. I'm not a English major.
I mainly focused on the first section of information since I don't feel like I have authority to change the authors technical comments.

Thanks.


WirePlumber integration doesn't handle PipeWire/WirePlumber restart

When I restart PipeWire (journalctl --user -u pipewire.service) the WirePlumber integration stops working without logging anything.
It also says segmentation fault on ^C:
^C[2024-02-03 10:11:42.019] [info] Quitting.
^C[1]    10856 segmentation fault (core dumped)

Here's my waybar config:
{
    "layer": "top",
    "position": "top",
    "height": 24,

    "modules-left": ["hyprland/workspaces"],
    "modules-center": ["hyprland/window"],
    "modules-right": ["network", "memory", "wireplumber", "battery", "clock", "tray"],
    "clock": {
        "format": "{:%a %b %d %T}",
        "interval": 1,
        "tooltip": false
    },
    "memory": {
        "interval": 30,
        "format": "RAM: {used:0.2f} / {total:0.2f} GiB",
        "tooltip": false
    },
    "wireplumber": {
        "format": "VOL: {volume}% {icon}",
        "format-bluetooth": "{volume}% {icon}",
        "scroll-step": 2,
        "on-click": "pavucontrol",
        "tooltip": false
    },
    "network": {
        "interface": "wlp1s0",
        "format": "D: {bandwidthDownBytes}, U: {bandwidthUpBytes}",
        "interval": 1
    },
    "battery": {
        "format": "BAT: {capacity}%",
    },
    "tray": {
        "icon-size": 16
    }
}

% waybar -v
Waybar v0.9.24

Is this also possibly related to my issue which is that the wireplumber module sometimes doesn't work when I first login to hyprland? I have to restart waybar for the module to show properly (otherwise it's stuck on 0%)

I have the exact same problem on described by @Fxzzi

I have the same issue. I autologin to arch and start Hyprland on a launch script within getty so I'm assuming the quick start isn't giving enough time for wireplumber to start.

+1

If you change the pipewire default device after starting waybar it fixes the issue. I just put pactl set-default-sink <default-device> to execute after waybar automatically and the issue is gone. To get the device name you can use the command pactl list

An alternative solution I currently use is simply to start waybar slightly delayed after Hyprland starts with:
exec-once = sleep 1 && waybar

If home-manager, programs.waybar.systemd.enable should be disabled with this to prevent conflicts.


WIP: Core Wallet UI Refresh

Variety of enhancements and additions to the core wallet interface for ODIN.

Everything starting to look great so far, fixed some build issues after bringing in enhancements from Phore.


HTML watcher only works once

First off - great work on the generator!
I have been running into an issue where the html change watcher only works once. E.g. I run 'gulp serve', make a change to an html file and the first time the browser refreshes successfully, but when I change the same html file again, the change is not detected. I can change a different html file which also triggers a browser refresh once. If I change a .ts file, the change is always detected. I have run the generator a few times and the issue is always reproducible.
Following is my setup and the configuration options I have used:
OS: OSX 10.11.5
Node version: 4.2.4
NPM version: 3.9.5
Framework: Angular 1
Modules manager: Webpack with NPM
CSS preprocessor: SASS
JS preprocessor: TypeScript
Sample app: Techs
Any help would be much appreciated!

Hi,
Can you replace "watch" function in gulpfile.js to
function watch(done) {
  gulp.watch(conf.path.src('app/**/*.html'), cb => {
    browserSync.reload();
    cb();
  });
  done();
}

Perfect! Thanks so much for the quick response :)

Should be fixed in the next release


As a developer I want to know which JS Frameworks should be included

AC:
research into different frameworks; React, Node, Angular, Vue, etc.
DOD:
enough knowledge about the various frameworks to make a decision on which to be included initially

This one confuses me a little, react is a front end framework, snyk is a container vulnerability tool and NPM is a package manager for node module dependencies. If you want to bootstrap a project with react for developers then there are tools that do that already, you could probably incorporate those as direct dependencies of your project.
Or do you mean decide on the actual stack of your project? Maybe that is what it is and the word "included" made me think of the other issues where you talk about things being included in the project.

was confused myself...corrected my mistake there.
When I say "include", what I mean is to be a part of the overall project as an available option to the user.

OK then I'm more confused now :joy:
Maybe take a look at something like: https://www.freecodecamp.org/news/how-and-why-to-write-great-user-stories-f5a110668246/


[QUESTION] Suggestions for rendering Popup from Semantic UI React

Problem Description

Consider I wish to color a cell as red/green/yellow depending on the threat level. On plain React Semantic UI table it will be done as shown:

<Popup
       trigger={
            <Table.Cell style={highlight_threat}>
                {utils.getThreatIcon(i.threat)}
            </Table.Cell>
      }
       inverted={true}
       position="left center"
       size="mini"
>
     <ThreatPopup threat={i.threat}/>
</Popup>

Semantic UI React binding requires to pass the rendering cell as a trigger

Now in the case of React Table we have handle to render the Cell and override, so I used the following:

const columns = [
            {
                header: '',
                accessor: 'threat',
                render: props => {
                    const highlight_threat = utils.getThreatStyle(props.value);
                    return (
                        <Popup
                            {/* This needs to contain the Table Cell or TdComponent if I am right */}
                            trigger={
                                <span style={highlight_threat}>
                                    {utils.getThreatIcon(props.value)}
                                </span>
                            }
                            inverted={true}
                            position="left center"
                            size="mini"
                            style={highlight_threat}
                        >
                            <ThreatPopup threat={props.value}/>
                        </Popup>
                    )
                }
            },
]

Now this one obviously will not work, since I am passing the span and not the cell itself, that colors the checkmark icon but not the actual cel..

Now how do I do this? Is there a way I can pass the "this" component somehow. I tried that but the this is only bound to my parent component.

I am trying to play with the source code, any help is greatly appreciated.
VERSION<EMAIL_ADDRESS>

I'm not sure what Popup is, but this seems like it should work based on your description.  Anything returned in render (in versions less than 6) will render inside of the table cell (you do not need to create the table cell yourself.

Thanks for your response @tannerlinsley
Sorry to continue the conversation in a closed issue.

I'm not sure what Popup is

<Popup /> is an UI component in Sementic UI and I am using the React binding for it. Essentially it expects to send the actual component within its prop. Now, I need to send the actual cell as a component into the Popup.
Avoiding any specific discussion about Popup, and to make it more generic:
Here is the doubt:
I can understand that we can override render method or Cell in the latest version to override the Cell render, but it doesn't let me to push a background color for that cell itself right?
Since a render, is only providing me access for the  within the cell, I am not able to set a background color for that cell.
Hope I am clear now. So the question is, is there a way to reference the Cell itself within the render/Cell function?
Thanks again.

I think what may solve your problem is using the getProps callback in
your column definition.  In that callback, you are provided all available
information about the cell, row, data, and column, and you can return an
object of props to be placed on the cell container. In your case you
could easily do: getProps: () => ({ style: { backgroundColor: ‘red’ } })
On Thu, May 25, 2017 at 11:29 AM Vaithiyanathan Sundaram <
<EMAIL_ADDRESS>wrote:

Thanks for your response @tannerlinsley https://github.com/tannerlinsley
Sorry to continue the conversation in a closed issue.
I'm not sure what Popup is
 is an UI component in Sementic UI https://semantic-ui.com/
and I am using the React binding
https://github.com/Semantic-Org/Semantic-UI-React for it. Essentially
it expects to send the actual component within its prop. Now, I need to
send the actual cell as a component into the Popup.
Avoiding any specific discussion about Popup, and to make it more generic:
Here is the doubt:
I can understand that we can override render method or Cell in the latest
version to override the Cell render, but it doesn't let me to push a
background color for that cell itself right?
Since a render, is only providing me access for the
within the cell, I am not able to set a background color for that cell.
Hope I am clear now. So the question is, is there a way to reference the
Cell itself within the render/Cell function?
Thanks again.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tannerlinsley/react-table/issues/287#issuecomment-304071175,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFUmCTIxtP-ePz0lI-QcnG3o0kDt0QFVks5r9bqUgaJpZM4NhG7F
.

This is excellent.
Thanks a lot for your quick response, let me try it out and keep you posted.
On Thu, May 25, 2017 at 11:41 AM, Tanner Linsley<EMAIL_ADDRESS>wrote:

I think what may solve your problem is using the getProps callback in
your column definition. In that callback, you are provided all available
information about the cell, row, data, and column, and you can return an
object of props to be placed on the cell container. In your case you
could easily do: getProps: () => ({ style: { backgroundColor: ‘red’ } })
On Thu, May 25, 2017 at 11:29 AM Vaithiyanathan Sundaram <
<EMAIL_ADDRESS>wrote:

Thanks for your response @tannerlinsley <https://github.com/
tannerlinsley>
Sorry to continue the conversation in a closed issue.
I'm not sure what Popup is
 is an UI component in Sementic UI https://semantic-ui.com/
and I am using the React binding
https://github.com/Semantic-Org/Semantic-UI-React for it. Essentially
it expects to send the actual component within its prop. Now, I need to
send the actual cell as a component into the Popup.
Avoiding any specific discussion about Popup, and to make it more
generic:
Here is the doubt:
I can understand that we can override render method or Cell in the latest
version to override the Cell render, but it doesn't let me to push a
background color for that cell itself right?
Since a render, is only providing me access for the
within the cell, I am not able to set a background color for that cell.
Hope I am clear now. So the question is, is there a way to reference the
Cell itself within the render/Cell function?
Thanks again.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<https://github.com/tannerlinsley/react-table/issues/287#issuecomment-
304071175>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AFUmCTIxtP-ePz0lI-
QcnG3o0kDt0QFVks5r9bqUgaJpZM4NhG7F>

.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tannerlinsley/react-table/issues/287#issuecomment-304089943,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQSdKQgVpEO4fK1-qXcItSmYjNcsBR0sks5r9ctZgaJpZM4NhG7F
.

@tannerlinsley  Thanks for your suggestion, using TdProps helped.
Thanks again


Appstudio update test-component-pac-eqdftc

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-eqdftc-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
5 seconds
init

✅ Succeeded
20 seconds
clone-repository

✅ Succeeded
2 minutes
build-container

✅ Succeeded
2 minutes
sast-snyk-check

✅ Succeeded
5 seconds
deprecated-base-image-check

✅ Succeeded
22 seconds
sbom-json-check

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
26 seconds
clair-scan

✅ Succeeded
5 seconds
show-summary

✅ Succeeded
6 seconds
show-sbom


OAuth2 Error: invalid_request

When I run gcalcli after deleting .gcalcli_oauth, my browser opens to a Google error page with the following message:
"400. That's an error.
Error: invalid_request
Invalid response_type: code\"
If I manually edit the url in the browser by removing all four instances of "\" and refresh, everything works fine. Any ideas why the original url is incorrect? I am working with gcalcli 3.3.2, google-api-python-client 1.4.2, and oauth2client 1.4.12.

I don't believe this has ever been seen before.  What operating system, version of python, etc?
Do you see the same issue if you install the latest test release (pip install gcalcli==4.0.0a1) and delete the $HOME/.gcalcli_oauth file?

I just upgraded to gcalcli 3.4.0, and the issue no longer occurs after deleting the .gcalcli_oauth file. I would consider the issue closed.


Uniqueness support in STS

Hi,
we would like to use STS as a global queue for JMeter threads, unfortunately, STS does not allow to do that because it does not maintain uniqueness of data being inserted.

Ok, thanks


Convert to char array?

I am trying to do the Diffie Hellman key exchange here. It require a very very large prime number, so I end up using this library. But then the key must be in char[] type.
So is there any proper way to convert?

Yes, the toString function does that. For example:
// function to display a big number and free it afterwards
void printBignum (BigNumber & n)
{
  char * s = n.toString ();
  Serial.println (s);
  free (s);
}  // end of printBignum


How to get it running?

Hi Thomas,
I stumbled upon your project because I wanted to make some automatization via pimatic (https://pimatic.org/)
Installed pyhton and the executed the
pip install raumfeld
But then...yeah what then?
How do I proceed? Do I have to execute the setup.py? If yes, I cannot get it running...
Could you maybe give me a helping hand?
Thanks in advance.
quintz

Hey Quintz, you might want to check out PyRaumfeld it supports almost all control functions of the Raumfeld system. It comes with a small WebServer that allows for easy automation via http requests. Just download it and start RaumfeldControl.py like this
python2.7 /path/to/RaumfeldControl.py .
Sorry for hijacking this - python-raumfeld is a great library! - but I think it's not what quintz is looking for.
Best.

Even though it's an other lib, I still won't get yours running.
Everytime I execute it I get
Traceback (most recent call last):
  File "/home/pi/raumfeld/PyRaumfeld-master/RaumfeldControl.py", line 12, in <module>
    from bottle import route, run
ImportError: No module named bottle

Thoughts? : /

pip install bottle
should do the trick for you. if you run in other problems you should open an issue over there


Generate JSON Web Token (JWT) on Console

Issue in ./help/landing/api-authentication.md
The guide indicates to use the Developer Console to generate the JSON Web Token (JWT). This is somewhat ok for testing and evaluation. But it is NOT ok for regular use. For regular use the JWT must be generated automatically. How this is done is well documented across the web and particularly here for Adobe: https://www.adobe.io/developer-console/docs/guides/authentication/JWT/
I propose you add a notice box to the guid indicating that this description is for tutorial, testing, and evaluation only and that JWT should actually be generated programmatically according to https://www.adobe.io/developer-console/docs/guides/authentication/JWT/
/cc @cxs504, @theManikJindal

Thank you for submitting this issue. We will review the documentation and make the necessary updates as quickly as possible. Tracking internally in https://jira.corp.adobe.com/browse/PLAT-114254.

The documentation has now been updated.
We have added a note to the appropriate section in the authentication tutorial: https://experienceleague.adobe.com/docs/experience-platform/landing/platform-apis/api-authentication.html#jwt


lr_mul, wd_mul params probelms in optimizer.py

Hi, functions set_lr_mult and set_wd_mult in optimizer.py are designed to set multi learning rates and multi wd rates. However, i haven't found any place call these functions except in function __init__ with input {}. In other words, i don't know how to set multi learning rate with current code.
So I change the __init__ function to invoke  functions set_lr_mult and set_wd_mult and it works. Here is the changed code:
    def __init__(self, rescale_grad=1., param_idx2name=None, wd=0.,
                 clip_gradient=None, learning_rate=0.01, lr_mult=None, wd_mult=None,
                 lr_scheduler=None, sym=None, begin_num_update=0):
        self.lr = learning_rate
        self.wd = wd
        if lr_mult is None: lr_mult = {}
        if wd_mult is None: wd_mult = {}
        self.set_lr_mult(lr_mult)
        self.set_wd_mult(wd_mult)

I don't know whether it is bug.
If no, please show me some hits or example code to set multi_rate.
Thank you!

You should specify lr_mult in corresponding symbol's attribute.
set_lr_mult and set_wd_mult are invoked in __init__ to construct a dictionary named self.lr_mult which is used to determine specified learning rate of params.

@sodabeta7
Thank  you. I find the documents about attr of  symbols' attribute.


Extend GzipReader specs to cover returned Enumerators.

These additional specs aim to cover the recent fixes from https://github.com/jruby/jruby/issues/4443. This PR might be good enough for the time being.
However, seems to me that the specs for GzipReader could benefit from reusing most of the tests from core/io/shared/each.rb, but unfortunately I couldn't figure out how to implement the reuse (I'm kind of new to the Rspec/Mspec language).

Thanks, this looks great.
It seems Zlib::GzipReader#each is a bit different, for instance it does not accept a separator argument like IO#each does. I think the current specs are good, it seems rather involved to share specs between these two.
Thank you for your contribution!


chore: replace eth2_hashing with ethereum_hashing

This is a necessary step to update ring, which is necessary to build in risc-v.
Part of #1444.
What was wrong?
trin fails to build in risc-v
How was it fixed?
It is still not fixed. This is just one step forward.
To-Do

[ ] Clean up commit history and use conventional commits.

Thanks to @haurog and @michaelsproul for fixing the ring dependency in ethereum_hashing:
https://github.com/sigp/ethereum_hashing/pull/8

@come-maiz CI is going to fail because how how you named your commit, could you amend the commits name following these guidelines https://www.conventionalcommits.org/en/v1.0.0/

'chore: replace eth2_hashing with ethereum_hashing'
^ something like this would be good

Yes, done.

Yes, done ^_^

Yes, done ^_^

Thank you for your first contribution


Revert "Make fetchable_fields more like other field overrides"

This reverts my earlier accepted PR #534. :disappointed:
I now realize that a class-level implementation of fetchable_fields loses access to the requested @model object, making checks like "is the requested model owned by the current user" impossible.
Example from my UserResource:
def fetchable_fields
  filtered_fields = [:password]

  current_user = context[:current_user]
  unless current_user && (current_user.admin? || current_user == @model)
    filtered_fields << :email
  end

  super - filtered_fields
end

Just ran into this same issue :+1:

Thanks for fixing this @reidab . Wish I had caught it earlier.

@lgebhardt If this solution has been accepted, shouldn't we also change updatable_fields and creatable_fields to keep consistency and a possibility to use @model also in there?
Just asking...

@Exelord Yes, in the ideal world that would be the case. However where the methods are used we don't have a resource instance instantiated. It might be possible, but it would be a significant refactor. I'll look into it in the refactor I'm working on for multiple operation support since that's changing a lot of the request parsing logic.


Strings which contain NUL '\0' are truncated on read

I'm using EF Core with both SQL Server and SQLite.
I can insert a string which contains a NUL character "foo\0bar". It seems to be written correctly.
I can read it back using a regular sql connection. But if I use EF to read it back, I get only "foo".

This is by design for Microsoft.Data.Sqlite. We use NUL to determine the end of the string.
The SQLite documentation also states:

The result of expressions involving strings with embedded NULs is undefined.

Note, in 2.0 Microsoft.Data.Sqlite is building on top of SQLitePCL.raw which does the same thing.

OK by design then. Thanks for clarification.

@denisz1 @T18970237136 It looks like this is "fixed" in version 3.0.0

Hi @bricelam,
in which area was this fixed? When I use Microsoft.Data.Sqlite 3.1.4 and test with a raw INSERT and SELECT using an SqliteCommand, strings still seem to be cut off after a NUL (\0):
    using var memoryDb = new SqliteConnection(new SqliteConnectionStringBuilder {
        DataSource = ":memory:"
    }.ToString());
    memoryDb.Open();

    const string checkValue = "a\0b\0c";

    using var cmd = memoryDb.CreateCommand();
    cmd.CommandText = "CREATE TABLE T1(a TEXT); INSERT INTO T1 (a) VALUES (@P1);";
    cmd.Parameters.AddWithValue("@P1", checkValue);
    cmd.ExecuteNonQuery();
    cmd.Parameters.Clear();

    cmd.CommandText = "SELECT a FROM T1;";
    string strValue = (string)cmd.ExecuteScalar();

    Console.WriteLine("String is equal: " + (strValue == checkValue)); // prints "False"

@T18970237136 Can you submit a new issue? Comments on closed issues tend to get lost or forgotten.


[SMALLFIX] Avoid powermock and whitebox in the block store test

We're still using powermock to mock the BlockStoreContext singleton, but all other usages of powermock and whitebox are rendered unnecessary.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/10925/
Test PASSed.

@calvinjia Thanks for the review! I've addressed all comments, PTAL when you have a chance

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/10956/
Test PASSed.

LGTM


Enabled support for deterministic project generation

Fixes #125

Coverage increased (+0.002%) to 87.983% when pulling bfc6d5bce1eeb511f629f5b8a6c95d68dc6a97f2 on feature-deterministic-uuids into 2f3a7b801e1ee7eb0c7fec9767020c89c5fdbf43 on develop.

Getting this released whenever you have time would make debugging issues easier, as the different struct option effects can be viewed in a project diff

Hi @yonaskolb, you can make use of this right away without waiting for us to release if you checkout the code and install it from that:
git clone git@github.com:workshop/struct.git
cd struct
git checkout feature-deterministic-uuids
gem install bundler # no need to run this if you already can run 'bundle' in Terminal, may need sudo
gem install rake # no need to run this if you can already run 'rake' in Terminal, may need sudo
bundle install # may need sudo
rake install # may need sudo

Thanks, that will help!
I actually tried to do this in a post generate script but it doesn't seem to work. Would you expect this to work?
run do |spec, xcodeproj|
    xcodeproj.predictabilize_uuids
end

I would yes, there may be nuances in the ordering between the hooks and xcodeproj processing that cause it to randomise again though. I'm not sure if once you do predictabilize_uuids you're unable to touch the project again without it re-randomising.
It also may be worth you adding some puts statements to see if your hook is actually running.

Looks as though hook wasn't actually running due to an problem I'll open an issue about. Once it's running though predictabilize_uuids in post-generate doesn't help however. Oh well

Yeah @yonaskolb I'd try using this branch then, I've confirmed the UUIDs are stable after adding the command in the right place

I tested this branch. It does in fact create an identical .xcodeproj file if the contents of the spec are EXACTLY the same.
However if anything changes in the spec, all the id's are created from scratch and rearranged, meaning the use case of seeing diffs in the xcodeproj file doesn't work.
An ideal solution would be to generate the id's sequentially like the Swift Package Manager does (OBJ_1, OBJ_2), or based on a common random seed and using the hashes of object names. That sort of functionality may be locked behind cocoapods/xcodeproj though

That's interesting, because the documentation indicates that when using this method the IDs are based on MD5 hashes of each object, which is exactly what you're proposing.

From looking into the diffs, it seems this does partially work, however the ordering of the objects within the pbxproj file seems to affect its MD5 hash for some reason, which is why the IDs still change. It seems like the object order is non-deterministic too, so even if the IDs are stable, your diff still won't be clean.
It does have an impact though, the diff looks much nicer than before where every ID is included, so I'll still merge this.

 Looks like the the output of the xcodeproj

PBXFileReference -- /mainGroup/children/children:children:|,|,|,|,displayName:iOS,isa:PBXGroup,name:iOS,sourceTree:<group>,,children:|,|,displayName:tvOS,isa:PBXGroup,name:tvOS,sourceTree:<group>

...hundreds more lines...

Interesting, we're not logging anything. If you scroll to the very top, is there a warning abut duplicate UUIDs?

Nope no warning. It actually begins like this:
Processing project 'project'...
[Xcodeproj] Generated duplicate UUIDs:

PBXFileReference -- /mainGroup/children/children:children:...
... hundreds of lines...

Saved 'PATH_TO_PROJECT/project.xcodeproj'

Yeah I thought so - the Generated duplicate UUIDs: bit is what I was referring to. For now it may be safer to remove this functionality until it can be investigted further.


Ooo case insensitive

Thanks Deanna!  I hand merged both of your fixes into my latest PR (since I had a bunch of changes to the same files):
https://github.com/delaurentis/planner/pull/21


feat: [SSCA-847]: SBOM generation and Ingestion for Snyk and Trivy

Harness Developer Pull Request
Pages Modified:

What's supported by Harness SSCA
Ingest SBOM from Blackduck into SSCA
Ingest SBOM from Aqua Trivy into SSCA
Ingest SBOM from Snyk into SSCA

What Type of PR is This?

[ ] Issue
[x] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

SSCA-842
SSCA-847

House Keeping
Some items to keep track of. Screenshots of changes are optional but would help the maintainers review quicker.

[x] Tested Locally
[x] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6567bce976ceac23c8beb33a--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6568ef67bb9092040504d6c7--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6568ef9fc2dad606392da8ba--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6568f026c5b850008d48ffbb--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6568f0cdb6210d07b3a6531e--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6568f0e56969be06d31c0331--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://656a0650824ffc0bef24d447--harness-developer.netlify.app

@aimurphy Thanks, I think I have made all the changes to reflect the feedback.

@williamwissemann The heading levels on the Snyk topic still need to be incremented. Headings must start with level 2 (##). I also suggested moving some text around for flow. Please see the comments here: https://github.com/harness/developer-hub/pull/4366/files#diff-b4497922206925d6218a7e5464f9f121d6abc43823b4a84bc593d2e2416af593

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://656a26fe8dc25f24b53b6d92--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://656a31c88dc25f2c9c3b71cc--harness-developer.netlify.app

@aimurphy Okay, that should be all the changes. It doesn't look like I have permission to merge.


Make an argument in findOne / findAll with the option not to save locally

In some cases, saving locally must be implemented manually. For example, the local object has changes and the remote one has changes and I have to do the merge by my own rules.

figured out how to implement this using the onSuccess override


How to handle deleted job

When the job is in deleted state, the raw data, stats are still available.
At the moment, the library assumes deleted jobs a bad so one will see an error such as '2/2/1' does not have any items or 2/2/1 has 'deleted' state.
It's possibly to run rules against such jobs, but does it make any sense?

I don't think we need to run Arche through the deleted jobs. If the necessary job was deleted then we have to find an other one to work with

Deleted job is a bad job.


Block copy does not work

Hello,
I am currently using version 1.60 installed with cargo (installation on Archlinux with AUR results in an error when running thoth), but I am unable to copy a block or selection, either in main mode, edit mode or in CLI.
Any copy action from thoth just empties the system's clipboard.
I currently manage like this:
thoth view block_name | xsel --clipboard
but it's not very practical.
Any idea where the problem might be coming from? Let me know if you need more information.
I really like the way thoth works and would like to be able to use it in its entirety.
Thanks

OS: Archlinux
Terminal: wezterm 20240203-110809-5046fc22

Hi there! Sorry that you're running into this issue :/ . Just so that I have a better understanding on what is going on, are you using X11 or Wayland as your window system? Or do you not have the display set?

Thanks I appreciate it! Ok cool, I'm investigating. Seems there might be a bug with copypasta but i'm double checking

^ I have a PR for this out. I was able to get it working on my machine, although i will say I run Manjaro rather than Arch Linux directly. Let me know if this fixes the issue or not since I can't fully test the solution via Arch

I just made a release with the clipboard fix. When you get the chance @alban-b could you let me know if it works on your end?

Just installed via Cargo and working perfectly, thank you!

Excellent glad to hear! Thanks so much for reporting the bug :)

I've been wanting to learn rust for years, and I finally started this week after digging through the sources of your project during the bug report (any good resources for an experimented developer eager to learn is welcome :grin:).
Hope to contribute one day!

Awesome! I'm still very new to Rust as well as this is my first major Rust project haha. I can describe the way that I've been going about learning:

Went through the Rust Book (I'm sure you've heard of it). Apparently there is an interactive version of the book that came out: https://rust-book.cs.brown.edu/experiment-intro.html
Went through Rustlings: https://github.com/rust-lang/rustlings?tab=readme-ov-file
After this I've just been building small project, asking questions to more experienced people, and learning from the compiler haha  as I find that I learn the best from experience
Now I'm starting to work on some bigger projects and trying to contribute to open source projects here and there

I hope that helps!