Not configurable header name problem

I need to give X-UserEmail as a header key, but I can't, I couldn't configure the name of the request headers it takes whatever I give for the variable name as the key, please add configuration for it.
For example:
@KompendiumParam(ParamType.HEADER) val email: String
If we would have param like "name" I think it would be great because we can't create X-UserEmail as a variable name.

This is a good point! I wonder if it makes more sense to have dedicated annotations for each parameter type (query, path, header cookie) rather than a one size fits all annotation in V2 of this library.
For V1 I think we can do this in a non breaking way by doing what you said and having an annotation parameter that defaults to an empty string.

Actually... I believe this will work for ya
data class HeaderNameTest(
  @KompendiumParam(type = ParamType.HEADER) val `X-UserEmail`: String
)

Kotlin is pretty flexible for naming conventions if you use backticks around the field name.
Can you confirm if this works for what you need?

It worked, thank you very much @unredundant you saved the day!


Integration with cyclepy

I made a quick python port of cycle.js here at cyclepy. I did this mostly just as an exercise. When I saw you project I wondered if it would be possible to use it to somehow to update the ui.
Do you think this would be possible?

I am not familiar with these projects, so its hard for me to say. What would you need from Flexx to do what you want?

To put is as simple as possible, Cycle uses observables (rx) with cyclical
dependencies (drivers). So a driver listens to an observable and emits an
observable. This is extremely useful with a UI.
So an integration would be in the form of a driver, which would take an
observable that emits a tree/object that describes what the current view
should look like. I guess that's what I would need flexx to do. This driver
would also send back a way to listen to events (clicks, text input, etc...)
In which the developer would be able to tell flexx how to update the ui.
Let me know if you need me to try and clarify anything. Thanks for your
reply!
On Wed, Sep 2, 2015, 7:59 AM Almar Klein<EMAIL_ADDRESS>wrote:

I am not familiar with these projects, so its hard for me to say. What
would you need from Flexx to do what you want?
—
Reply to this email directly or view it on GitHub
https://github.com/zoofIO/flexx/issues/22#issuecomment-137047484.

As I understand it, you're looking for a way to connect Flexx's events/signals to Cycle, as well as (part of) the rendering. I would recommend having a look at the implementation of some of the widgets, and see how you can create a custom widget that suits your needs.
I suspect you'd need to load some JS to get things working. Loading custom JS is something on the todo-list, but you can probably hack your way out of this by including a <scrip> DOM element from JS.

Okay, I'll definitely look into this at some point. Thanks for your suggestions!

You can now load additional JS and CSS, which should make it easier to do what you want. See the docs and this example


Updated help reference

I was reviewing an issue - to me, it looks like the VS Code help reference was a bit out of place. Having it up front right under the prerequisites might be a better spot.

@daveabrock 	thanks!


Badge: Add default badge for the User

Starting a remote transaction from a user with several tags and a default one is not choosing the default.

@WillyLellouch: you confirm it works as intended?

@jerome-benoit Tested with several chargers and several users with different types of badges.
The default badge is always selected.
However the default badge is selected even if deactivated, is this normal ?

@jerome-benoit Tested with several chargers and several users with different types of badges.
The default badge is always selected.
However the default badge is selected even if deactivated, is this normal ?

@jerome-benoit Tested with several chargers and several users with different types of badges.
The default badge is always selected.
However the default badge is selected even if deactivated, is this normal ?

I don't think so. I think a warning should be display that the default badge is deactivated and the user should not be able to start a transaction.

@jerome-benoit Tested with several chargers and several users with different types of badges.
The default badge is always selected.
However the default badge is selected even if deactivated, is this normal ?

I don't think so. I think a warning should be display that the default badge is deactivated and the user should not be able to start a transaction.

I don't think so. I think a warning should be display that the default badge is deactivated and the user should not be able to start a transaction.

Is this going to constitute a new issue or should it be included in this one ?

I don't think so. I think a warning should be display that the default badge is deactivated and the user should not be able to start a transaction.

Is this going to constitute a new issue or should it be included in this one ?

Ping for my earlier comment ?

I don't think so. I think a warning should be display that the default badge is deactivated and the user should not be able to start a transaction.

Is this going to constitute a new issue or should it be included in this one ?

I have created a separated issue.

Then this one is done. Thanks !
I'll go ahead and close this issue.


C85 component fell off PCB

As we were disassembling to access raspberry pi and add an arm, the C85 component fell off the PCB. Would you recommend a repair or replace? Is the PCB safe to power on without the use of the port 2?

Hello Collin
Can you tell me what revision of the PCB you have? It should be written in an area on the underside of the PCB

Closing due to inactivity.


Ive created a vscode plugin to help in your leptos rust projects

I notice there is no leptos formatter on vscode
so I created one
https://github.com/codeitlikemiley/leptos-fmt
https://marketplace.visualstudio.com/items?itemName=masterustacean.leptos-fmt
demo:
https://github.com/user-attachments/assets/a0d4615d-7ee8-4311-85c2-fa8481c1334a
I would like to also share my other vscode plugin
https://github.com/codeitlikemiley/cargo-runner
https://marketplace.visualstudio.com/items?itemName=masterustacean.cargo-runner
Which you can use as well to quickly run , build ,test/doctest, benchmark , debug rust code...
by pressing CMD (MAC)or CTRL(Windows/Linux) + R
In cases you are using leptos and you wanna override the run or test command
you can do so with a Makefile ,
also you can override the parameters for any context mentioned
e.g. adding ENV var to prepend the cargo command or adding more parameters like --release
by pressing by pressing CMD (MAC)or CTRL(Windows/Linux) + SHIFT + R
Hope you guys can share it on the documentation so others can use it as it can save them time
Peace!

Awesome! Thanks a lot for this. You are most welcome to put a PR to our Leptos DX guide and in the Awesome Leptos list.
I'll be closing this issue for now to clean up our issues list, since there's no action item on this for now, but I would love to see a PR on the above two repos.
Once again, thanks a ton!


TextInput as PasswordInput

I want to create a TextInput as a PasswordInput so when i write a password into this box the characters returns the sign ***** back instead showing the password. Sadly i cannot find anything in the documentation. Is there a workaround for that?

There's currently no built-in option for that, but it sounds like a legit request, so I added it.
If you can't wait for the next NuGet version you can try to apply the patch yourself (https://github.com/RonenNess/GeonBit.UI/commit/ea43274d8f452e0e461a37620f1515574f96b6d9#diff-94c848706442535bb0751b56ff6a7b0e) or build the latest version from master (should be stable).
I think I'll release a mini-version pretty soon though.
Cheers!


How to prevent auto duplication of list items in infinite scroll mode

When I set my list to infinite scroll mode using mLayoutManager.setInfinite(true);, the library seems to auto duplicate items to fill the screen if there are only a few items (like two or three) in the list. How can I prevent this from happening. What should I do if I want to see only the items I have set in the list even if there is only 1 or 2 items, while still being able to scroll them infinitely?

When enable setInfinite, It should be able to scroll in both direction. So I don't think it could work when size is smaller than 3. All I can do is offer an api which sets the max visible item count.

fix by commit: d5ea4946eaac1ef3de847bf656812400ae00de95
I added a method called setMaxVisibleItemCount(int count), You can set the size as max visible item count, I think this can solve your problem. And this feature will be included in next release.

@leochuan  thanks that's a cool feature.


Handle optional predefined AnyVals in Marshallable

Fixes an issue where the marshallable macro emits code that fails to
compile when an optional property's underlying value is an AnyVal with
no valueGetter.
fixes #257

great stuff, thank you!


encode/decode configs for the video codec

Hi, thank you for the impressive work!
TBH, I nearly have no idea of the video codec. After investigating a little bit of the codebase, I couldn't find the encoding/decoding configs for the video. So I want to know how to determine the intra_period and p_period? (which is required by CodingStructure) And do I need to change other parameters if I use the provided config files (e.g. enc/medium.cfg, dec/mop.cfg) for the video codec? Thank you!

Hi, thanks for your interest in our work!
For the current version (3.2), we've temporarily disabled video coding as it is undergoing an important reworking which will be released in the following months. If you want to use Cool-chic for video coding as in the paper Cool-chic video: Learned video coding with 800 parameters, Leguay et al., you have to revert to the previous Cool-chic version using git clone --branch v3.1 https://github.com/Orange-OpenSource/Cool-Chic.git.
⚠ Everything below is valid only for Cool-chic 3.1 ⚠
Coding structure
Regarding the parameterization of the coding structure, it works as follows.
Cool-chic always encodes a Group Of Pictures (GOP), consisting in 1 intra frame followed by 0 to 255 inter frame(s).
Intra period
A GOP starts with an intra frame and followed by an arbitrary number of inter (P or B) frames. The number of frames in the GOP and the number of frames encoded by Cool-chic is thus: intra period + 1:
# This codes 9 frames from the video
# number_of_coded_frames = number_of_frames_in_gop = intra_period + 1
(venv) python3 src/encode.py --intra_period=8 --input=xxx.yuv
P-period
Both GOPs below have the same --intra_period=8 but have a different --p_period
# A low-delay P configuration
# I0 ---> P1 ---> P2 ---> P3 ---> P4 ---> P5 ---> P6 ---> P7 ---> P8
python3 src/encode.py --intra_period=8 --p_period=1 --input=xxx.yuv
# A hierarchichal Random Access configuration
# I0 -----------------------------------------------------> P8
# \-------------------------> B4 <-------------------------/
#  \----------> B2 <---------/ \----------> B6 <----------/
#   \--> B1 <--/ \--> B3 <--/   \--> B5 <--/  \--> B7 <--/
python3 src/encode.py --intra_period=8 --p_period=8 --input=xxx.yuv
Note the introduction of the --p_period argument in the command line. This sets the distance between the I0 frame and the successive P frames. Reducing the temporal distance alleviates the motion compensation process, which can be useful. It can also enables more complex GOP structure such as:
# There is no more prediction from I0 to P8. Instead the GOP in split in
# half so that there is no inter frame with reference further than --p_period
# I0 -----------------------> P4 ------------------------> P8
#  \----------> B2 <---------/ \----------> B6 <----------/
#   \--> B1 <--/ \--> B3 <--/   \--> B5 <--/  \--> B7 <--/
python3 src/encode.py --intra_period=8 --p_period=4 --input=xxx.yuv
Configuration files
There is no config file in cool-chic 3.1. However, the script samples/video/encode_video.sh gives some relevant parameters to encode video. You might want to set --n_itr to 10000 to have better compression performance. Change the --intra_period and  --p_period values according to your needs.
Hope this helps! 🙂

Great, thank you for the quick reply!
I was actually stuck at the encoding step for several hours before I realized that I needed to change the output channel of layers_synthesis in cfg/dec/mop.cfg from 3 to X. It seems to be running now, but does this mean it won't work, even though it's running, since you mentioned that the video codec only functions correctly in version 3.1?
By the way, I now understand all the concepts. Thanks again for your help!

In the current version (3.2), you'll only be able to encode video i.e. overfit one Cool-chic for each frame of the video. That will gives you an estimate of the rate-distortion performance of the system in workdir/results_best.tsv and the learned model in workdir/video_encoder.pt.  What's not going to work on this version is using the learned video_encoder.pt to write an actual bitstream and decode this bitstream with a standalone Cool-chic decoder.
If you just want to assess the codec performance, you can do it with the current version (3.2). If you need to produce a bitstream and send it to a different device for decoding, you have to revert to the previous one (3.1).
PS: good catch changing the configuration files. I'll modify them so that no one will get stuck here in the future. 😉

Just to update you, I've successfully implemented the video codec using Cool-Chic version 3.1. However, I have one question: it seems that, at the moment, we can only encode and decode the first GOP. From my understanding, a video typically contains multiple GOPs. Is it possible to encode and decode the entire video rather than just the first GOP? Please let me know if I've missed something. Thank you!

Great to hear that you've managed to make it work using Cool-chic 3.1!
Cool-chic is indeed limited to coding the first GOP of a video. To code several GOPs you could split the video file into smaller files with one file per gop e.g. video.yuv would be splitted in gop1.yuv and gop2.yuv. Then encode separately gop1.yuv and gop2.yuv. A bit rudimentary, but it works!


Preserves file attributes during copy step

Summary - This fixes #217 by doing a direct replacement using path.copy, which preserves attributes. If the file already exists and cannot be overwritten, then tsrc will fail similarly to copying a file that doesn't exist.

[x] You follow indications from the code manifesto
[x] All existing linters pass
[x] All existing tests run
[x] The new feature comes with appropriate tests

Updated the test performing a copy operation to copy a file with the "executable" mask on, checked against os.X_OK

[x] update the changelog (in docs/changelog.md)

Oh, and I forgot to say thanks for your contribution, and that you can add your name to the list in the THANKS file if you wish :)

Will make all these changes today. FWIW, your CONTRIBUTING.md is awesome for non-python folks. Poetry literally eliminated all the agony I was expecting in getting a virtual-env set up on OSX.

FWIW, your CONTRIBUTING.md is awesome for non-python folks.

Thank you - really appreciated.

Poetry literally eliminated all the agony I was expecting in getting a virtual-env set up on OSX.

I'll let the author know :)
Thanks for your contribution!


The point of these endpoints is mainly for testing. Right now I need to automate the delete of data to help me test cleanly

if it's only needed for testing, it's better to just write it in the testing code...

fair point. i'll do that.


Numerical failure in i8 matmul: m=n=32 k=64

This PR adds this failing test, and labels it as an expected numerical failure. By adding it like this we might be more likely to track and fix it. Issue created: https://github.com/nod-ai/iree-amd-aie/issues/273

This passes for me locally. i.e. the numerical test fails for me locally. So there's something between CI and my local setup.


Fix parseToken array popping

Description
Fixed an issue in the flat_object where array field names were incorrectly removed after processing the first object, leading to inaccurate key-paths in flattened JSON data.
Related Issues
https://github.com/opensearch-project/OpenSearch/issues/13351
Check List

[x] New functionality includes testing.

[x] All tests pass

[ ] ~New functionality has been documented.~

[ ] ~New functionality has javadoc added~

[x] Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)
[x] Commits are signed per the DCO using --signoff
[x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)
[x] Public documentation issue/PR created

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@msfroh
I tried to fix the issue where array fields disappear in flat_object Is this content correct?
https://github.com/opensearch-project/OpenSearch/pull/13259#issuecomment-2070714539

@naomichi-y -- are you still working on this PR?
@OVI3D0 -- are you able to help get it across the line? You can clone from https://github.com/naomichi-y/OpenSearch/tree/feature/fix-parseToken-array-popping and try to do the refactoring that @reta suggested.

@naomichi-y , @OVI3D0 -- I hope you don't mind, but I pushed a refactoring commit for parseToken that (in my opinion) is a lot easier to understand.
@reta -- I think my refactoring should address your concerns.

@reta -- I think my refactoring should address your concerns.

Totally, thanks a lot @msfroh !

The build failures seem to be related to this. I'll try running the yamlRestTest target to see what's breaking.

Oof... that was complicated. Even debugging the yamlRestTests, the cause of failure wasn't obvious.
It turns out that it was all because I was stepping past the final END_OBJECT token, which DocumentParser doesn't like.

@msfroh
Sorry for the late response. Thank you for your help!

Hi - not sure if this is the place to ask, but can't find anything in the release docs.
When can this be expected to be tagged in a release? We're running into this on AWS OpenSearch

Hi - not sure if this is the place to ask, but can't find anything in the release docs.
When can this be expected to be tagged in a release? We're running into this on AWS OpenSearch

This was backported to 2.x in https://github.com/opensearch-project/OpenSearch/pull/15235, so it will be in the next open source 2.17 release (https://opensearch.org/releases.html).
For Amazon OpenSearch this is really a question for Amazon support. The service deploys every other release, see https://docs.aws.amazon.com/opensearch-service/latest/developerguide/release-notes.html and you can see the past dates to get an idea of when you can expect something (2.9: October 2, 2023, 2.11: November 17, 2023, 2.13: May 21, 2024, 2.15 would be next).


Support Moving Up / Down Multiple Items At Once

E.g. when all this is selected and I press control+down:

Thanks, this is a great suggestion!

I've added this feature for most move statements in v1.147.0. When there is a selection that spans several elements (can be partial) and move gets started, it moves multiple code elements.


Fix exporting sawyer as json

Why
When exporting variable, it exports #<Sawyer::Resource:0x00000001201eb678> string as sidekiq argument.
I want to get the parameter what the sidekiq used as hash or JSON.
How
Exporting hash variable to make hash deeply.
The same code is existed in BugsnagErrorEventDownloader::Commands::GenerateCsvMap, but it is not needed to modify. (This may make too much parameters.)

@okumud
Thank you for sending in your wonderful pull request!
I did not notice that Sawyer::Resource was not showing up correctly.
I will merge this in!

The same code is existed in BugsnagErrorEventDownloader::Commands::GenerateCsvMap, but it is not needed to modify. (This may make too much parameters.)

Yes, it is true.
But I'll give it some thought.


Prefilling to and from fields

Not sure if using the state is the best here. Maybe props would be better?

Great, let's merge this now and look for cleanup options later.


Improve comparison of git and release archive source trees

(Watch out, this is on top of #2 to avoid future rebase pain, you probably only want to look at the last commit when reviewing)
The previous way allowed differences in the other direction from what we like, for example having mvnw in the source archive but NOT in the git checkout would've been fine. After staring at the output of diff --recursive for a while and contemplating how to parse it well, I decided the proper way is to move the whole logic to Python. Luckily the standard library has a module for comparing directories, so this PR takes that, and uses it to ensure we see only differences we like.
To test, I used python main.py brave-karaf 0.1.2 --gpg-key BB67A050 --git-hash 3cf4ac6577eb0d4775d20f24814e7a0852fa1635. I've removed .gitignore from allowed_left_only and Jenkinsfile from expected_top_level_git_only, which yielded a failure on the check_git_revision check with the below output:
[FAIL] git revision
Expected exactly the below files to be only in the git checkout, but not in the source release:
.git .gitignore mvnw.cmd mvnw .mvn
but instead found these:
.git .gitignore .mvn Jenkinsfile mvnw mvnw.cmd

/tmp/tmpyxm75p14/git/incubator-zipkin-brave-karaf.git/itests/.gitignore is only in the git checkout

See above for a full output of diff.

After adding them back in, the check passes:
[PASS] git revision

Meh, and this is still not quite good enough: that blacklist needs to be applied across all the source tree, not just the top level.

There, I'm feeling good about this one.


Transformalize Task Scheduling

It would be handy to have the ability to schedule Transformalize Task with Orchard Core's background task feature.
Resources

https://github.com/Lombiq/Orchard-Training-Demo-Module/blob/dev/Services/DemoBackgroundTask.cs
https://github.com/atifaziz/NCrontab
https://crontab.guru
https://www.youtube.com/watch?v=Rx11bdawew0

Notes

Granularity is 1 minute.  Schedule is hard-coded on task (via attribute).

Update to the latest version, then:

In Admin > Configuration > Features,  enable Timer Workflow Activities.
Create a new Workflow Type
Add a Timer Event and set it as the Startup Task.
Add a Transformalize Task.
Connect the Done output on the Timer Event to the Transformalize Task.

You provide the Transformalize Task with the alias of the one you want to run.  It will run it when the timer event tells it to.  Here's a nice video demonstrating workflow in Orchard Core.


ReferenceHandler.Preserve does not work well with JsonConverters

When I use a simple JsonConverter, with Reference handler equal to Preserve to serialize cyclic objects, the $id property added to each object is not unique, in this example "$id":1 exists two times :
System.Text.Json : 5.0.0
using System;
using System.Diagnostics;
using System.Text.Json;
using System.Text.Json.Serialization;

namespace ConsoleApp2
{
    public class ActionResultDto<T>
    {
        public T Result { get; set; }
    }
    public class Data
    {
    }
    class Program
    {
        static void Main(string[] args)
        {
            var data = new ActionResultDto<object>
            {
                Result = new Data()
            };
            var options = new JsonSerializerOptions()
            {
                ReferenceHandler = ReferenceHandler.Preserve
            };
            options.Converters.Add(new EmptyJsonConverter());
            var str = JsonSerializer.Serialize(data, options);
            Debug.Assert(str.Split("\"$id\":\"1\"").Length == 2); // we must have only one occurence of "$id":"1"
        }
    }
    internal class EmptyJsonConverter
       : JsonConverter<object>
    {
        public override object Read(ref Utf8JsonReader reader,Type typeToConvert,JsonSerializerOptions options)
        {
            throw new InvalidOperationException("Should not get here.");
        }

        public override void Write(Utf8JsonWriter writer,object objectToWrite,JsonSerializerOptions options)
        {
            JsonSerializer.Serialize(writer, objectToWrite, options);
        }
    }
}

// Output =>    {"$id":"1","Result":{"$id":"1"}}

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

ID: 5de37d79-751f-e372-e866-efde28f4908c
Version Independent ID: 678c4f23-764a-7ccd-0d6d-159a30957477
Content: How to write custom converters for JSON serialization - .NET
Content Source: docs/standard/serialization/system-text-json-converters-how-to.md
Product: dotnet-fundamentals
GitHub Login: @tdykstra
Microsoft Alias: tdykstra

@tdykstra FYI I added doc-enhancement as the categorization label. Please correct it if I'm mistaken.

Thanks, that's the right label.

consider adding a sample for

preserving references + custom converters
preserving references across multiple (de)serialization calls

@layomia or @jozkee can you provide an example suitable for docs for each of these scenarios? Something like the one for non-custom-converter preserve references would be more helpful than a converter for Object.
I don't see how to adapt this approach to types other than Object because, as @layomia noted, we advise not passing in options when calling Deserialize/Serialize in converter code, and it's not clear why doing that in this example doesn't cause a StackOverflow exception.
Also, when I run the example code it fails at options.ReferenceHandler = null; because you can't change an options instance after it's been used with a call to Deserialize/Serialize.

preserving references + custom converters

Assuming that you wrote a custom converter for Company and you don't want to manually serialize the Supervisor, which is an Employee. you want to delegate that to the Serializer and you also want to preserve the references that you have already saved.
https://gist.github.com/Jozkee/88ac585c1912f6a3c6816254cecc54b5

preserving references across multiple (de)serialization calls

Assuming that you have a list of Employees and you have to serialize each one individually and you also want to take advantage of the references saved in the ReferenceHandler's resolver.
https://gist.github.com/Jozkee/0b50c40b58fa93c006a3ba322b4e1165

Also, when I run the example code it fails at options.ReferenceHandler = null; because you can't change an options instance after it's been used with a call to Deserialize/Serialize.

@tdykstra  I fixed that in above examples by adding a Reset method that creates a new ReferenceResolver instead of mutating the JsonSerializerOptions.

we advise not passing in options when calling Deserialize/Serialize in converter code, and it's not clear why doing that in this example doesn't cause a StackOverflow exception.

I think that's only problematic when you funnel the call to JsonSerializer.Serialize and T is the same type as your value; or in the case of Deserialize, T is the same type as typeToConvert.

preserving references + custom converters

Assuming that you wrote a custom converter for Company and you don't want to manually serialize the Supervisor, which is an Employee. you want to delegate that to the Serializer and you also want to preserve the references that you have already saved.
https://gist.github.com/Jozkee/88ac585c1912f6a3c6816254cecc54b5

preserving references across multiple (de)serialization calls

Assuming that you have a list of Employees and you have to serialize each one individually and you also want to take advantage of the references saved in the ReferenceHandler's resolver.
https://gist.github.com/Jozkee/0b50c40b58fa93c006a3ba322b4e1165

Also, when I run the example code it fails at options.ReferenceHandler = null; because you can't change an options instance after it's been used with a call to Deserialize/Serialize.

@tdykstra  I fixed that in above examples by adding a Reset method that creates a new ReferenceResolver instead of mutating the JsonSerializerOptions.

we advise not passing in options when calling Deserialize/Serialize in converter code, and it's not clear why doing that in this example doesn't cause a StackOverflow exception.

I think that's only problematic when you funnel the call to JsonSerializer.Serialize and T is the same type as your value; or in the case of Deserialize, T is the same type as typeToConvert.

Thanks for the code and explanations, @Jozkee ! These are added to the docs in https://github.com/dotnet/docs/pull/22322

Thanks for the code and explanations, @Jozkee ! These are added to the docs in https://github.com/dotnet/docs/pull/22322

I think that's only problematic when you funnel the call to JsonSerializer.Serialize and T is the same type as your value; or in the case of Deserialize, T is the same type as typeToConvert.

@jozkee @tdykstra - as a sidebar to the main discussion of this issue; given the provided example:
internal class EmptyJsonConverter : JsonConverter<object>
{
    public override object Read(ref Utf8JsonReader reader,Type typeToConvert,JsonSerializerOptions options)
    {
        throw new InvalidOperationException("Should not get here.");
    }

    public override void Write(Utf8JsonWriter writer,object objectToWrite,JsonSerializerOptions options)
    {
        JsonSerializer.Serialize(writer, objectToWrite, options);
    }
}

It just occurred to me that the serializer calls object.GetType() when typeof(object) is passed at the root level when serializing. So, the object is processed as the run-time type of the instance, which is certainly different from typeof(object). This is why we don't have a stack overflow caused by the JsonSerializer.Serialize(writer, objectToWrite, options) line: the "Ts" are different.

I think that's only problematic when you funnel the call to JsonSerializer.Serialize and T is the same type as your value; or in the case of Deserialize, T is the same type as typeToConvert.

@jozkee @tdykstra - as a sidebar to the main discussion of this issue; given the provided example:
internal class EmptyJsonConverter : JsonConverter<object>
{
    public override object Read(ref Utf8JsonReader reader,Type typeToConvert,JsonSerializerOptions options)
    {
        throw new InvalidOperationException("Should not get here.");
    }

    public override void Write(Utf8JsonWriter writer,object objectToWrite,JsonSerializerOptions options)
    {
        JsonSerializer.Serialize(writer, objectToWrite, options);
    }
}

It just occurred to me that the serializer calls object.GetType() when typeof(object) is passed at the root level when serializing. So, the object is processed as the run-time type of the instance, which is certainly different from typeof(object). This is why we don't have a stack overflow caused by the JsonSerializer.Serialize(writer, objectToWrite, options) line: the "Ts" are different.

@Jozkee , I have the same problem but now with IgnoreCycles: I need to add $type property for an array that will contain objects of different types ex : ( [A,B,C] of IEnumerable< A> => [A,{$type:"B",$value:B},{$type:"C",$value:C}] )
My custom JsonConverter will have a JsonSerializer.Serialize in the Write method in case when the type of the object == T in IEnumerable< T> so when there is no need to add $type :
        private abstract class AbstractEnumerableJsonConverter<TObj, TCollection> :
            JsonConverter<TCollection> where TCollection: IEnumerable<TObj>
        {

            public override TCollection? Read(ref Utf8JsonReader reader, Type typeToConvert, JsonSerializerOptions options)
            {
                var list = new List<TObj>();
                while (reader.Read())
                {
                    if (reader.TokenType == JsonTokenType.EndArray)
                    {
                        break;
                    }
                    if (reader.TokenType == JsonTokenType.StartObject)
                    {
                        var document = JsonDocument.ParseValue(ref reader);
                        if (document.RootElement.TryGetProperty("$type", out var typeName))
                            list.Add((TObj)JsonSerializer.Deserialize(document.RootElement.GetProperty("$value").GetRawText(), Type.GetType(typeName.GetString()), options));
                        else
                            list.Add((TObj)JsonSerializer.Deserialize(document.RootElement.ToString(), typeof(TObj), options));
                    }
                }
                var ret = (IList)Activator.CreateInstance(typeToConvert, list.Count);
                if (ret.Count == 0)
                    for (var i = 0; i < list.Count; ++i)
                        ret.Add(list[i]);
                else
                    for (var i = 0; i < ret.Count; ++i)
                        ret[i] = list[i];

                return (TCollection)ret;
            }

            public override void Write(Utf8JsonWriter writer, TCollection value, JsonSerializerOptions options)
            {
                writer.WriteStartArray();
                foreach (var obj in value)
                {
                    if( obj.GetType() != typeof(TObj))
                    {
                        writer.WriteStartObject();
                        writer.WriteString("$type", obj.GetType().FullName);
                        writer.WritePropertyName("$value");
                        JsonSerializer.Serialize(writer, obj, options);   //<---- The state is lost so we can no more check cycles**
                        writer.WriteEndObject();
                    }
                    else
                    JsonSerializer.Serialize(writer, obj, options);       //<---- The state is lost so we can no more check cycles**
                }
                writer.WriteEndArray();
            }
        }

I have tried to do something like https://github.com/dotnet/docs/issues/21777#issuecomment-736751404 , but it seems that IgnoreCycles relies on a lot of things that are internal and resides in the Core code ( like PushReferenceForCycleDetection that maybe would help if it's protected and not internal https://github.com/dotnet/runtime/blob/01b7e73cd378145264a7cb7a09365b41ed42b240/src/libraries/System.Text.Json/src/System/Text/Json/Serialization/IgnoreReferenceResolver.cs#L23 ).
Is there any way I can get around these cycles?


Scope ordering problems in multi-threading applications

Hi,
I'm not sure if multi-threading is in the scope of the project but as there is a lot of focus put on synchronization by adding a mutex I thought it might be.
In some scenarios, the scope order might be incorrect.
Since every thread has their own current scope in a thread-local variable (ThreadState.scope), situations might occur where a variables scope id is less than it's dependencies.
Since the variables/operations are reordered according to their scope in a kernel, this might result in them reading values that are uninitialized.
This effectively destroys the topological order of the variables.
The following is a somewhat contrived example, but one could imagine recording a loop in a different thread, which increments the global scope.
TEST_BOTH(11_threading) {
    jit_set_log_level_stderr(LogLevel::Trace);

    UInt32 x = opaque<UInt32>(1, 2);

    UInt32 y;
    auto yp = &y;
    printf("\n");
    printf("Scope of thread 1: %d\n", jit_scope(Backend));

    std::thread t([=](){
        jit_new_scope(Backend);
        printf("Scope of thread 2: %d\n", jit_scope(Backend));
        *yp = opaque<UInt32>(1, 2);
    });

    t.join();

    printf("Scope of thread 1: %d\n", jit_scope(Backend));

    UInt32 z = x + y;
    jit_var_schedule(z.index());
    jit_eval();

    printf("%s\n", z.str());
    jit_assert(strcmp(z.str(), "[2, 2]")== 0);
}

Here, the scope is explicitly incremented in a different thread and the variable y will "inherit" the current scope in t.
Therefore the scope of variable x will be less than the one of variable y.
The current scope of thread 1 stays the same and therefore the scope of variable z will be the same as x, which is lower than that of y.
When compiling the kernel, Dr.Jit will reorder operations according to their scope with the assumption that this will not break the order from the topology sort performed previously.
Therefore the body of the kernel in CUDA looks something like this (I added comments):
body: // sm_86
    // x is loaded
    ld.param.u64 %rd0, [params+8];
    mad.wide.u32 %rd0, %r0, 4, %rd0;
    ld.global.cs.u32 %r4, [%rd0];

    // x and (not yet loaded) y are added to produce z
    add.u32 %r5, %r4, %r6;

    // z is stored
    ld.param.u64 %rd0, [params+16];
    mad.wide.u32 %rd0, %r0, 4, %rd0;
    st.global.cs.u32 [%rd0], %r5;

    // y is loaded after z has already been stored
    ld.param.u64 %rd0, [params+24];
    mad.wide.u32 %rd0, %r0, 4, %rd0;
    ld.global.cs.u32 %r6, [%rd0];

    add.u32 %r0, %r0, %r1;
    setp.ge.u32 %p0, %r0, %r2;
    @!%p0 bra body;

done:

Hi @DoeringChristian
I think we are aware of this behavior. The solution has been to build a new scope after joining your threads.
I believe we only use this scoping feature in Mitsuba's parallel scene loader. If you take a look at this snippet: https://github.com/mitsuba-renderer/mitsuba3/blob/8bab68b6cec812d27b5e60474a5457c7b6b5f7eb/src/core/xml.cpp#L1172-L1181
We create a new  scope counter for the main thread after the root node has been constructed. This is semantically equivalent to creating a new scope after you've joined all your threads.
I'm not sure if this is a satisfying answer ?

Thanks for the quick response!
Do you think it would make sense to resolve the scope automatically in such cases or would the overhead be to big?
As I only wanted to bring this to your attention, but you are already aware of it I think the issue is more or less resolved.

Sorry I lost track of this.

Do you think it would make sense to resolve the scope automatically in such cases or would the overhead be to big?

Yeah, I think this would be too much of an overhead. We should provide better utilities to handle this for users - like a context manager of some sort.


Typo in JavaDoc: "of for"

Found in ServiceScan and CloudScan

Upon service scanning, if there is a unique bean of for service type, you can inject it
using the following code (shows Redis, but the same scheme works for all services):

Fixed via 93631a26676c3afab09d2eac20b342bebb513363


BREAKING Drop targeting/retrieving apps by name in all models

Resolves: #1030
Change-type: major
See: https://www.flowdock.com/app/rulemotion/i-cli/threads/uFbbCpQLJqxAguhU7PB9FW0Znnl
Signed-off-by: Thodoris Greasidis<EMAIL_ADDRESS>
Contributor checklist

[ ] Includes tests
[ ] Includes typings
[ ] Includes updated documentation
[ ] Includes updated build output

I don't think it depends on #1086, since the other PR is just about the getAll variants.


Revert "Replace newlib with picolibc"

The switch caused a regression in semihosting support which will take
time to fix. The patch will be relanded once the fix is in place.
This reverts commit 1e08e801d5c5020e6a2b8b17b070e53379d3250d.

LGTM.


Init command

Please add a klepto init command so we can start quickly.

@boekkooi-fresh I'll work on that


Update accessing_the_api.md

This is also an experiment to see if I can generate a commit on my fork and a PR on github all using the web interface.


[FEATURE] Password Recovery Flow

Figma
/v1/api/auth routes:

/resetPassword: POST with body {email: string, phoneNumber: string }. Support email before phone numbers. For security, returns 200 regardless of existing account.

Note: flask_user, the library we use to manage users, automatically installs routes related to password recovery and email verification. Rather than implement our own routes, we could leverage these. These are the /user routes listed by flask routes
Frontend Work

[ ] Build forgot password form
[ ] Connect form to /resetPassword endpoint
[ ] Test password recovery flow

Backend Work

[ ] Use flask_user to send and handle recovery emails
[ ] Configure credentials for sending emails
[x] Create /resetPassword route

Looks like a lot of the /user routes provided by flask-user are all packaged with the views provided. Looking into seeing if we can change the email URL to direct to the frontend and use as much of the flask user routes as possible.


Error VAR_IN_OUT 'MQTT_INFO' must be assigned in call of 'HANDLE_MQTT'

I'm looking at the GreatExampleOfAdvantages project
In the MQTT_STUFF (FB) is this piece of code.
Client(
MQTT_IN_OUT:= GVL_MQTT.MQTT_IN_OUT,
ENABLE:= TRUE,
URL:= 'test.mosquitto.org:1883',
TIMEOUT:= T#10S);
When I build the project I get the following error.
C0039:  VAR_IN_OUT 'MQTT_INFO' must be assigned in call of 'HANDLE_MQTT'
I think the MQTT_INFO must be set in the function call?

hmmm,
Looks like I didn't update the example to the new lib version, I'll check this…
BR Stefan

Hi,
I Updateted the projekt, Looks like I didn't made the upload. And test.mosquitto.org is acually dead....
Let me know if its looks fine for you...
BR Stefan


Connected to my AP but no connection !

I have a big problem in Archlinux.
when I run create_ap, it works fine for about 10 seconds and after that it kinda stop.
my phone is connected to the AP but no connection!
I"ve noticed some things, when I run iwconfig here's the outputs:
`enp0s25 no wireless extensions.

ap1 IEEE 802.11 ESSID:off/any
Mode:Managed Access Point: Not-Associated Tx-Power=15 dBm
Retry short limit:7 RTS thr:off Fragment thr:off
Power Management:off

wlp0s26u1u3 IEEE 802.11 ESSID:"DFDFDDFD"
Mode:Managed Frequency:2.412 GHz Access Point: 77:8B:S7:B8:9E:99
Bit Rate=28.9 Mb/s Tx-Power=20 dBm
Retry short long limit:2 RTS thr:off Fragment thr:off
Power Management:off
Link Quality=47/70 Signal level=-63 dBm
Rx invalid nwid:0 Rx invalid crypt:0 Rx invalid frag:0
Tx excessive retries:8 Invalid misc:699 Missed beacon:0

ap0 IEEE 802.11 ESSID:off/any
Mode:Managed Access Point: Not-Associated Tx-Power=15 dBm
Retry short limit:7 RTS thr:off Fragment thr:off
Power Management:off

lo no wireless extensions.

wlo1 IEEE 802.11 ESSID:off/any
Mode:Managed Access Point: Not-Associated Tx-Power=15 dBm
Retry short limit:7 RTS thr:off Fragment thr:off
Power Management:off
`

here's what im doing to run create_ap :
create_ap wlo1 wlp0s26u1u3 XXXX pass --hidden --daemon
HELP please

same here bro


add credhub_cli uaa client

This adds the credhub_cli client to uaa. This is needed when uaa is configured for external ldap auth.

This seems like it would be good as another ops file. So people can opt into it and not create extraneous users/clients they don't use.

@vChrisR Hi! Just wanted to know if you've had the time to consider the suggestions that @jtarchie and @taylorsilva has mentioned? Thanks!


As a user, I can filter skills based on max level, members only, and other attributes

User should be able to interact with easy-to-understand UI elements
Should work in conjunction with ordering and searching

Story points estimation: 5 (hours)

Possibly save max level for searching?

Backend supports this

Took about 5 hours for the front end


node-red-contrib-alexa-home-skill

Hi All
Is there an updated node-red-contrib-alexa-home-skill it dont seem to work, this node node-red-contrib-alexa-home conflicts with it.
Any help with this, Thanks

No

An update won't fix anything, I can't change the name of the nodes that clashes as that will break existing users.
So it's one of the other

Hi
Ok i'll try and sort something out.
Thanks
Regards
Tony
On Thu, 3 Dec 2020 at 15:27, Ben Hardill<EMAIL_ADDRESS>wrote:

An update won't fix anything, I can't change the name of the nodes that
clashes as that will break existing users.
So it's one of the other
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hardillb/node-red-alexa-home-skill-web/issues/110#issuecomment-738078341,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIFE6GEQBVGY3UEV4R333HTSS6U6JANCNFSM4UMD66GQ
.

--


[JENKINS-46945] Run duration set before RunListener.fireCompleted

Doing this will put the calculation of duration at the same location
where this is handled in core Jenkins' Run class:
https://github.com/jenkinsci/jenkins/blob/master/core/src/main/java/hudson/model/Run.java#L1758
Details here: https://issues.jenkins-ci.org/browse/JENKINS-46945

Any thoughts on this @jglick or @abayer? Sorry to ping you on this.

@jglick I added a real-world scenario and can put together a test for it. Is there an easy way to inject a RunListener into Jenkins to validate this? I think I have some ideas but there might be an easier way. Thanks!

Hey @jglick, good point on the test. I've added a test in 5a468ee and updated the JIRA a bit. Hopefully the JIRA is a bit more specific and the test helps. The test breaks if the change in WorkflowRun is reverted.

Sorry to bug you again on this @jglick. Does this seem reasonable? This PR gives us better metrics and a consistent interface to get build time metrics.
Could someone kick off the build again, it seems like there were some environmental errors at the time of the build that seem completely unrelated to my changes.

Any additional thoughts on this @jglick? I had updated the ticket and added tests.

Sorry to ping you on this.

BTW use the “request a review” gesture when possible. Otherwise this message goes into the 6164 and counting unread messages from GitHub.

Will do. Sorry about that.

Thank you for the contribution -- I'm going ahead and merging this now.


problem of wandb

hi, I'm having a 'wandb' issue when running a 'run.sh' file，I found a way not to use wandb in your other answers, and I wonder if not using wandb will have an impact on the experiment，Looking forward to hearing from you.
requests.exceptions.HTTPError: 404 Client Error: Not Found for url: https://api.wandb.ai/graphql
wandb.errors.CommError: Permission denied, ask the project owner to grant you access
wandb: ERROR Internal wandb error: file data was not synced
Problem at: /home/workspace/prob/main_open_world.py 165 main
wandb.errors.MailboxError: transport failed
wandb: While tearing down the service manager. The following error has occured: [Errno 32] Broken pipe

Hi @lshssel,
Not using wandb should not affect the performance of the model. However, it does make it more difficult to debug if that is needed (for example if you are training with a different system and different hyperparameters that may be needed).
Best,
Orr

ok, thank you very much!


Expression to aggregation expression translators review

Somehow github didn't use the base commit I thought I had selected.


:robot: Add invokedosfuscation.vm

Automated PR from https://github.com/mandiant/VM-Packages/issues/489 :rocket:

@Menn1s, @day1player, @skollr34p3r please review


[WIP] FAQ - update 'encounter_19_calc' entry for #971

Changelog:

Move last paragraph's text sections to a better place
"only" removed in english entry
remove mention of 10 minutes signal attenuation time window and just keep 5 minutes

The description about the risk calculation regarding dB levels does not match the documentation on https://github.com/corona-warn-app/cwa-documentation/blob/master/images/risk_calculation/risk_calculation_enf_v2_overview.pdf.

There is a lot of overlap between
http://localhost:8000/en/faq/#encounter_count_19 and
http://localhost:8000/en/faq/#encounter_19_calc
Please consider combining the two FAQ articles, so that the risk calculation is only described in one place.

There is no explanation of what dB levels mean. See https://github.com/corona-warn-app/cwa-website/issues/712 and the explanation published to https://github.com/corona-warn-app/cwa-documentation/issues/484#issuecomment-781382074.

@MikeMcC399 I'll let this PR stay up until the PR for encounter_count_19 is up too for direct comparison.

To avoid hitting CWA users with the confusing naming issues, I would suggest calling it "Exposure Notification Framework" in any place where Apple and Google are being referred to.

Noted. I've added it to the general issues/remarks in #971

Closed in favour of https://github.com/corona-warn-app/cwa-website/pull/1222


Remove consistently failing integration sync test

Description
Removes an end-to-end test that we no longer rely on. This test fails consistently which habituates us to failing master builds, which can hide underlying real issues.
Tested
These are tests
Other changes
None

So I actually know why this test is failing (the alfrajoresstaging environment was deployed with a different epoch size than exists in the .env.alfajorestaging file at master) at the moment and am working on a fix. If I do fix it, would it be worth reinstating or has it been historically unreliable?

I've created https://github.com/celo-org/celo-monorepo/pull/1504 to address the standing issue with this test if we want to keep it.

We decided to go with #1504 to fix the test rather than nixing it.


Terraform AZ Redis

Need to terraform AZ Redis. Created one manually for immediate development that can be deleted once we have terraform scripts -> current Resource: tempedehl7.
Suggested Configuration:
[ ] DEV -> Name: edehl7dev; Type: Basic C0 (250Mb, no replication) ($16/month)
[ ] TST ->  Name: edehl7tst; Type: Standard C1 (1Gb + Replication) ($100/month)
[ ] STG -> Name: edehl7stg; Type: Standard C1
[ ] PRD -> Name: edehl7; Type: Standard C1 (if we want Redis persistence, it will require Premium layer - P1 $202.21/month with options of reserved instances.)

Once Terraform is created and run against DEV, create a ticket so developers can switch configuration to the new Redis service.

Created #148 for tracking all the other environments. Moving to completed.


'includes' method is missing from Iterable prototype

According to docs includes method should be available on Iterable and other classes extending it.
http://facebook.github.io/immutable-js/docs/#/Iterable/includes
However, its alias contains is working.

The documentation represents the latest version of the library. includes was added in v3.7.1 in response to changes made in the ES6 spec around the same time. Please ensure you're using the latest version of the library.

Could you please update the library included inside the doc website?
(it's useful to easily test there but the version is 3.6.4)


provisioin SQL database for Warm Storage

connects to #59

Hi @hveiras, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, AZPRBOT;

Is there particular reason that you removed some validations? We need to keep those validations in my mind.

@hanzzhang I removed validations for all the parameters that are directly passed to Azure Cmdlets. From my point of view performing those in the scripts does not add much value because they are already performed server side. Having those validations there makes the readability of parameters difficult. I've been checking some examples of azure scripts and typically that kind of validations are not performed.
E.g: https://gallery.technet.microsoft.com/scriptcenter/Create-a-HDInsight-cluster-4edce5d7

@hanzzhang @francischeung Added validations back.

Shouldn't the Provision-SQLDatabase script create the resource group for the SQL Server if it doesn't exist? Currently I have to create the resource group manually before running Provision-All otherwise the New-AzureSqlServer command fails with a "resource group not found" exception.

@JohnPWSharp yes, seems to be a bug in the scripts. Let me check it.

@JohnPWSharp should be fixed now.


WSL --mount will not mount a microSD card

Version
22000.527
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
Ubuntu 20.04
Other Software
No response
Repro Steps
Surface Go 1 (Yes I'm aware it doesn't officially support Windows 11)
Insert an unpartitioned microSD card
Run GET-CimInstance -query "SELECT * from Win32_DiskDrive"
wsl --mount \.\PHYSICALDRIVE1
Screenshot
https://i.imgur.com/TqfUBoZ.png
Expected Behavior
SD card gets mounted into WSL2
Actual Behavior
The system cannot find the drive specified
Diagnostic Logs
No response

wsl --mount feature does not support removable media, use USP/IP instead.

If you are looking for guidance on how to connect a USB device (flash drive, SD card reader, etc), see Connect USB devices.
https://docs.microsoft.com/en-us/windows/wsl/wsl2-mount-disk
https://github.com/MicrosoftDocs/WSL/commit/ea7499d84f3493590300495f003f84ffc36ffff0

I did try that before leaving this report. That does not work either. The SD card just doesn't show up in the list. And not sure if it matters, but it isn't a USB SD card reader, but a PCIe one
https://i.imgur.com/XuYNZJk.png

I have the same issue on my laptop. SD card reader is not in the USB device list, and I can't mount it.

This is due to the fact, that there are simply no USB storage drivers in the WSL Kernel.
You can use usbipd for connectiong any USB devices to WSL and they will also show up under lsusb, however due to the missing drivers in the kernel, no devices will be listed under lsblk.
I'm now trying to compile my own kernel according to the usbipd docs, but it's probably faster to just flash ubuntu over your windows machine at this point.
I have really no idea why those drivers are not simply compiled into the default kernel.
See also many other issues like https://github.com/microsoft/WSL/issues/8408

I am still running into this issue for the Broadcom SD Card reader built into my old Acer laptop.
wsl --mount \.\PHYSICALDRIVE2 reports no drive found.
device appears as SCSI Disk Drive in Device Manager
usbipd wsl list does not list the SD Card.

I am still running into this issue for the Broadcom SD Card reader built into my old Acer laptop.
wsl --mount \.\PHYSICALDRIVE2 reports no drive found. device appears as SCSI Disk Drive in Device Manager usbipd wsl list does not list the SD Card.

i'm in the same boat now, did you ever find a fix?

Same issue. I need the feature to access an SD card for a Raspberry Pi and modify some files, but it's not working :(

Still no support for USB Storage in 2024 it seems.

The only working solution is to recompile the WSL kernel. It's less painful than you think. The full instructions are here: https://www.tomshardware.com/how-to/access-linux-ext4-partitions-in-windows
In addition to the instructions above, in order to read Steam Deck SD Cards, you must also enable casefolding support which requires UTF-8 character handling Under "File Systems -> UTF-9 normalization and casefolding support".
I hope Microsoft adds USB storage support to newer kernels.

This may be the same issue, I've got more detail in my issue: https://github.com/microsoft/WSL/issues/12153


get the remaing search engines to index blog.cyberoblivion.com

use bing webmaster tools
https://www.bing.com/webmasters

the site was crawled but many pages remain unindexed probably should look into what's taking so long


EWFmount Error

Hi All,
I am struggling to mount E01 images on the sift workstation using ewfmount. I get the below error:

I am so confused and not sure whats causing this issue. I using vmware player to run the environment.
Any assistance will be highly appreciated.
Thanks.

Your command line looks correct. Try renaming your E01 to remove the space.  Also, confirm that /mnt/ewf_mount exists.  It should, but check it anyway.
You can also verify the E01 with ewfinfo and ewfverify.
ewfinfo filename.E01
ewfverify filename.E01

I have tried with other E01s that dont have spaces and get the same error. I will perform the top two instructions and show output. Thanks for the speedy response.

Hi @mark-hallman please see screenshot below:

The said images mount perfectly on my WSL2 environment.
Thanks

@princedavinci When running the ewf commands in WSL and in the VM, are you using the same image (E01) to test both environments? If so, can you try the following on the image from within the SIFT Workstation:
xxd 002-MEM-DTP.E01 | head
mmls 002-MEM-DTP.E01
ls -lah 002-MEM-DTP.E01

One difference could be how you're accessing the E01 from within the SIFT Workstation. If you have a Shared Folder mapped into the VM, then permissions to access the file might not be propagating through the Shared Folder. The same could be said if the file is accessed through a network share.
You could also try checking the MD5 of the file (not the E01 content hash, but simply the file) from within the SIFT Workstation and within the WSL environment to see if you have the same hash with both, and if you're able to access the E01 properly through both environments. This will help narrow down whether this is an ewftools issue or not.
Cheers

@digitalsleuth You make a lot of sense as I dragged files across onto the VM while on WSL I used the shared folder. Let try the above and will revert back.
Thanks

Hi @princedavinci , did you ever get this resolved?

Hi I eventually left it after realising the environment I was working on had too man security issues which in turn affected my access to the EO1 files. Sent from my iPhoneOn 28 Jul 2023, at 20:39, Digital Sleuth @.***> wrote:
Hi @princedavinci , did you ever get this resolved?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

Try using a more recent version of ewfmount

Hi Joachim
will boot up the machine and try updating.
Thanks.
On Sat, Feb 10, 2024 at 11:52 AM Joachim Metz @.***>
wrote:

Try using a more recent version of ewfmount
—
Reply to this email directly, view it on GitHub
https://github.com/teamdfir/sift/issues/601#issuecomment-1936957148, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABKBEW3PRD2IEVL25F3P2MLYS47PBAVCNFSM6AAAAAAZPF2FR6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMZWHE2TOMJUHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

will boot up the machine and try updating.

Does SIFT provide a more recent version?

I'll be looking into this tomorrow to see where we source and what's available.

Sounds good, thanks!
On Mon, Feb 12, 2024 at 6:44 AM Erik Kristensen @.***>
wrote:

I'll be looking into this tomorrow to see where we source and what's
available.
—
Reply to this email directly, view it on GitHub
https://github.com/teamdfir/sift/issues/601#issuecomment-1938059980, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABKBEW4J3HJ2VWONQM4QZKDYTGM4JAVCNFSM6AAAAAAZPF2FR6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMZYGA2TSOJYGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

ewfmount comes from the GIFT.
22.04 - ewfmount 20140816 which is the latest from the PPA repository
20.04 - ewfmount 20140812 which is the latest from the PPA repository

as you can see in the screenshot the reporter runs 20140807

20.04 - ewfmount 20140812 which is the latest from the PPA repository

we don't support 20.04 any more, please upgrade to 22.04 and expect us to upgrade to 24.04 once available relatively quickly

@joachimmetz understood you don't, but the packages are still available (please do not remove them, if you are I need a heads up so I can clone them)
22.04 is the new base image and we'll support 24.04 as soon as we can but you are 1 of hundreds we have to have support for to effectively make a new base image. Plus it takes time to swap out the major OS in classes.
@princedavinci you should be able to run apt-get upgrade or apt-get update && apt-get install libewf-tools to get the latest available ewfmount version.

22.04 is the new base image and we'll support 24.04 as soon as we can but you are 1 of hundreds we have to have support for to effectively make a new base image.

Unclear what you mean with "supporting" me in this context? I don't need SIFT to support me, nor have I ever received support from SIFT or SANS for that matter.
This is why for Plaso we strongly recommend using Docker


Adjust line width in RST docs

Line width is all over the place.

@delucchi-cmu if you have a minute, coul you provide an example of where the line widths are no good, and perhaps an example of where they are so that whoever picks this up knows what it's suppose to look like?

When I copied over README content, I didn't insert line breaks in the rst source files. At least in VSCode, it isn't wrapped reasonably when I have an editor open, and looks silly.
https://raw.githubusercontent.com/lincc-frameworks/python-project-template/main/docs/index.rst
For an example of run-on source lines.
https://raw.githubusercontent.com/LSSTDESC/RAIL/main/docs/source/citing.rst
For an example of wrapped source lines.
Ideally, this could be managed by an autoformatter, but I don't know of a good one for RST.


Add ForeFire

Insert URLs to the project here:
https://github.com/forefireAPI/firefront

[x] The projects is active, documented, open source licensed, shows usage from external parties and is directly targeting environmental sustainability. Find more details in the Contribution Guide.

All issues labeled as 'Good First Issue' of the project listed on OpenSustain.tech will be visible on ClimateTriage.com. This is a great way to welcome new community members to your project.

@antonio-leblanc @filippi @forefire


Create a custom runtime image with jlink

The benefits of creating a custom runtime image are:

[x] Reduction of the number of requirements
[x] Minimizing the size of the Docker image
[x] Less friction with new users

The following steps can be used to create a custom run-time image:

Get the jmods list for the server's dependency with $JAVA_HOME/bin/jdeps --list-deps libs/app.deps.jar

Run jlink agains the module list:

$JAVA_HOME/bin/jlink -c --no-man-pages --no-header-files -G \
--module-path $JAVA_HOME/jmods/ \
--add-modules \
jdk.scripting.nashorn,\
jdk.scripting.nashorn.shell,\
java.base,\
java.compiler,\
java.instrument,\
java.logging,\
java.management,\
java.naming,\
java.scripting,\
java.security.jgss,\
java.xml,\
java.xml.bind,\
java.sql,\
java.rmi,\
jdk.sctp \
--output custom-jvm

Additional testing must be done to create platform-specific images. Also, the base image for "sipio-docker" does not support JDK9 yet, which makes this task slightly difficult.

Task completed!
Binaries for macOS, Linux and Windows are now available for the 1.0.0-M5 release. This binaries ship with a custom runtime image that has only the necessary jmods.

This is not working in JDK 11

This run works in JDK 12 but causes issues with sslv3
$JAVA_HOME/bin/jlink -c --no-man-pages --no-header-files -G
--module-path $JAVA_HOME/jmods/
--add-modules
jdk.scripting.nashorn,
jdk.scripting.nashorn.shell,
java.base,
java.compiler,
java.instrument,
java.logging,
java.management,
java.naming,
java.scripting,
java.security.jgss,
java.xml,
java.sql,
java.rmi,
java.desktop,
jdk.sctp,
jdk.unsupported
--output jre

Updated all jre assets and build scripts

Note on targeting one os from another:
https://stackoverflow.com/questions/47593409/create-java-runtime-image-on-one-platform-for-another-using-jlink/47594270#47594270


[BD-46] feat: create chip carousel

Description
Create chip carousel component
Deploy Preview
https://deploy-preview-2378--paragon-openedx.netlify.app
Merge Checklist

[x] If your update includes visual changes, have they been reviewed by a designer? Send them a link to the Netlify deploy preview, if applicable.
[x] Does your change adhere to the documented style conventions?
[x] Do any prop types have missing descriptions in the Props API tables in the documentation site (check deploy preview)?
[x] Were your changes tested using all available themes (see theme switcher in the header of the deploy preview, under the "Settings" icon)?
[x] Were your changes tested in the example app?
[x] Is there adequate test coverage for your changes?
[x] Consider whether this change needs to reviewed/QA'ed for accessibility (a11y). If so, please add wittjeff and adamstankiewicz as reviewers on this PR.

Post-merge Checklist

[x] Verify your changes were released to NPM at the expected version.
[x] If you'd like, share your contribution in #show-and-tell.
[x] 🎉 🙌 Celebrate! Thanks for your contribution.

Thanks for the pull request, @monteri!
When this pull request is ready, tag your edX technical lead.

Yeah, I'd say the nesting isn't a hard requirement given we have the convention of BEM class names (mostly) prefixed with pgn__{componentname}. Unlikely for collisions to happen, though still possible.

Updated to use nesting in this case. Also noticed that when we changed componentName.scss to index.scss we forgot to update component-generator. Made quick fix as part of this PR

Updated to use nesting in this case. Also noticed that when we changed componentName.scss to index.scss we forgot to update component-generator. Made a quick fix as part of this PR

Good catch!

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

:tada: This PR is included in version 20.46.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Strange behavior with code blocks using form helpers and Slim

Hi!
I'm playing with this project for use in production but I encountered a possible bug related with passing a Slim/HTML block to Rails helper.
# app/components/test_component.rb

class TestComponent < ActionView::Component::Base
  def initialize(form:)
    @form = form
  end

  attr_reader :form
end

/ app/components/test_component.html.slim

div
  = form.text_field :name
  = form.label :name do
    div.label__text
      | Nombre

/ app/views/tests/show.html.slim

= form_with model: @test do |form|
  /! Test Component Start
  = render TestComponent.new(form: form)
  /! Test Component End

Expected output:

  <input type="text" name="test[name]" id="test_name">

Nombre

Current output:

  <input type="text" name="test[name]" id="test_name">

Nombre

      <input type="text" name="test[name]" id="test_name">

Nombre

In normal Rails the block of code passed to the helper works perfectly. Any ideas? Thanks!

This looks really similar to the issue I ran into that caused me to open https://github.com/github/actionview-component/pull/240.

Just ran into this and for a simple case temporarily solved it by doing
<%#
  <% label_contents = '
    Cambiar imagen
    Abrir
  ' %>

  <%= @form.label(@attached_method, raw(label_contents), class: "form-file-label") %>

It seems that this error is happening with link_to code blocks and tag.<tagname> blocks too.
Any reason why this is happening?


[FIX] Device errors in tests

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

10 of 10 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.09%) to 88.363%

Totals

Change from base Build<PHONE_NUMBER>:
0.09%

Covered Lines:
1177

Relevant Lines:
1332

💛  - Coveralls


[ODB] 503 serviceNotAvailable when get siteUrl

Category

[x] Question
[ ] Documentation issue
[ ] Bug

Hi all,
I change my OAuth flow to request GRAPH DRIVE API in order to get the onedrive endpoint.
https://graph.microsoft.com/v1.0/me/drive/sharepointIds/siteUrl
However, this API failed with following error:
http code 503
{
  "error": {
     "code": "serviceNotAvailable",
     "message": "Service unavailable",
     "innerError": {
     "request-id": "b3d019b0-f262-44f1-929f-2843e2bc57f9",
     "date": "2019-04-16T11:48:44"
    }
  }
}

But this API works fine with user info part.
https://graph.microsoft.com/v1.0/me
Clue 1: This API have little chance to be successful, most time is broken.
Clue 2: Some of my client feedback they enabled the MFA (Multi-Factor Authentication), it might be related.
Is possible that we can fix in client part of it would be a server-side problem?
Thank you.

The 503 response suggests that the request is being throttled. Are you seeing any other requests to the tenant in question getting 503s as well, or just this siteUrl request through Graph?

Hi @ificator ,
thank you for reply,
In my testing, only drive related API received this response, the rest API work fine(User or mail)
And it is not the only case in this tenant, we encountered this with many accounts.
In my experience, the kind of error would recovered maybe after weeks later.

Investigation indicates that the 503 is returned as expected for the state of the service.  If the 503 is persisting, please open a support ticket with https://www.office.com.
In general you may want to take a look at https://aka.ms/scanguidance and ensure that your app is following our best practices for working with high volume apps.

Hi @JeremyKelley ,
Thanks for reply,
But the support on https://www.office.com. do not accept API things about graph,
I have ask for help and then rejected.
That why I am here.
Do you have any other channel to report API things in specific account?
thanks.

As part of a repository clean up effort we are closing older issues. If this issue remains, please: open a new issue, reference this issue, and provide any additional details that may help in resolution. Thank you for your understanding as we work to improve our responsiveness.


Deprecate direct imports of enhancers

For now we have an ability to import every enhancer directly like
import withProps from 'recompose/withProps', there are 2 reasons to deprecate such behaviour:

Now we are supporting es2015 modules, so direct import is not the best way to reduce size
Two different imports can be a problem, as an example current problem

import { setObservableConfig } from 'recompose';
import createEventHandler from 'recompose/createEventHandler';
import rxjsObservableConfig from 'recompose/rxjsObservableConfig';
setObservableConfig(rxjsObservableConfig);
const { handler, stream } = createEventHandler();
// OOPS stream is not rxjs stream because `createEventHandler` imported directly and use other instance of _config object

We can somehow add a deprecation warning at a build step for every enhancer, (not stream configs), or just remove them and release a new major version.

@istarkov I guess this case is for major. It's not kind of API.

Hey, guys!
Here's a  jscodeshift mod to automatically convert imports in your project as proposed by @istarkov.
`cd /path/to/your/project`

# Install jscodeshift (if you have not already)
yarn add jscodeshift

# Checkout the mod
git clone git@gist.github.com:0c38799b83d75e2aef529a3560eb65d7.git codemods

# Run migration, replace ./src with your project files root
jscodeshift -t ./codemods/convert-recompose-import.js ./src

# Remove the mod file
rm -rf ./codemods

We should add a warning in next patch version (might be v0.23.5) first.

What's the suggested way to reduce size? Does this assume tree shaking is really working reliably in apps, including ones using Typescript? Without it working reliably, this would be a huge performance regression - direct imports are still the only way I know to reliably avoid bundling unused code.
Based on these open PRs in Webpack (and my own experience of having no luck getting it working with components or HOCs), tree shaking has a long way to go:
https://github.com/webpack/webpack/issues/1750
https://github.com/webpack/webpack/issues/2867

We have tests for them. All is good. Your components may not treeshake if you use Babel display name plugin because it's expressions in module scope.

That tree shaking test suite is fantastic, and I'm absolutely going to lift that idea for another project. Thanks for commenting!
There's still some overhead (603 bytes gzipped or so), but I see there are outstanding issues for that. Rollup really does a very nice job of getting around the Uglify side effect complaints.


chore(deps): reduce strictness of dependencies

What kind of change does this PR introduce?

Not 100% what to classify this as, but I guess it falls under a feature (non-breaking change).
What is the current behavior?

As per #40, currently projects using this library have to use a very specific version of dependencies which this libraray prescribes. Instead of pinning the versions, we should be allowing all versions of our dependencies which don't break for this project.
What is the new behavior?
With the change here, the project has converted from using == to >= for dependency versions. This still keeps some restrictions on anyone who uses this library to make sure the code functions properly, but also more flexibility in dpendency versions which are acceptable for this project.
Does this PR introduce a breaking change?

No changes, anyone who uses this version of the libraray once it gets released won't have to pin their dependencies with previously limited versions.
Other information
I did find one bug when testing requests==1.0.0 which caused the tests to break. Specifically, in past versions of the requests library asserting equality on HTTPBasicAuth failed because they were instantiated as two distinct instances of hte class. It seems that this was fixed later on so as the equality check would pass, albeit I'm not sure when this fix was made.
Side note, if anyone is interested in how I automaated the process of checking which versions of the project worked / didn't work, let me know and I can put those steps in the comments on this PR.

@DaleSeo thanks for the review. For posterity's sake, I went with using the compatible release syntax to restrict it to the current major version of aiohttp and requests. The websockets library I kept at >=5.0 since 5.0 release was more recent (2018) and it seems it frequently releases major version changes. It may be worth switching this to pin it to a major version later, which should probably be done with the addition of an alert system for major version changes of dependencies.


Platform-specific limitations?

I'm in the middle of writing a project in HaxeFlixel. I just discovered that their audio.pitch doesn't work on Flash and HTML5.
How does Snow (and conversely, luxe, which uses the audio classes directly) handle audio? Are there any features that don't work on Flash or HTML5, or any other platforms?
Also, I didn't see any mention in any of the documentation. Are there any features or functionality that don't span all platforms?

snow doesn't support flash.
All endpoints in the snow api's that aren't on the module level have parity across targets. That means native (android, ios, linux, mac, windows) and web (webgl) have parity for all the audio endpoints. The module distinction means that modules (like OpenAL, or maybe native file access) can still provide access to features that break compatibility, but the distinction is clear that the user must be wary.
So, there are features that don't span for the sake of access, but the snow api itself on the top level is.
A simple audio test can be played with here.
As far as details goes, on native (currently) OpenAL is used on all targets by default, and on web the Web Audio API is used for all but streaming audio, in which case an <audio> tag is used due to the fact you can't stream anything yet on Web Audio without loading the entire file first (which defeats the point of streaming in most cases).
Does that help?

Yes, that helps a lot, thanks.


Revert "adding COC doc for all knative repos (#112)"

This reverts commit f723a9004cbd85cb4585ba303004faae28c11104.
Per the discussion here, the COC doc in community is not necessarily suitable for all repos, so this PR takes it back out of the automation.
cc @markusthoemmes  @evankanderson

/lgtm
/approve

/lgtm
/approve


Feature: VSCode extension for typia autocomplete / lint

Feature Request
Thanks for your suggestion. Feel free to just state your idea. Writing the issue, it would better to filling the below items:

A description of the problem you're trying to solve.
Mistakes can easily occur while creating comment based validations
An overview of the suggested solution.
Autocomplete / lint of typia tags and values
Examples of how the suggestion would work in various places.
@format uui (would error or shown tab to complete to uuid)

What is the purpose? Detailed description please.

Linter: a static code analysis tool used to flag programming errors, bugs, stylistic errors and suspicious constructs; Reasons why? Comments can include any text, including invalid content; Without a linter, it would be hard to identify and correct mistakes;
Autocomplete: a software function that gives users the option of completing words or forms by a shorthand method on the basis of what has been typed before; Reasons why? Reduce errors and increase development speed;
Visual Studio Code extension: VSCode has 81% market coverage;

https://typia.io/docs/validators/tags/
Use type tags of v5 instead, it is type safe as you wish.


No version of GetRawBBCorners that works with BoundsControl is public

Describe the bug
GetRawBBCorners from the BoundingBoxHelper is obsolete, since it uses the deprecated BoundingBoxes. The hint says "Use GetRawBBCorners and pass in TargetBounds", but GetRawBBCorners has no overloading that accepts a BoxCollider. However, there exists GetRawBoundsCorners, which accepts a BoxCollider, but it can't be used outside of the assembly since it is internal.
To reproduce
Steps to reproduce the behavior:

Create a BoundingBoxHelper object
Get a BoundsControl from the scene
Try to pass the TargetBounds of the BoundsControl to the GetRawBBCorners method of the helper

Expected behavior
Some version of GetRawBBCorners should be usable with the new BoundsControl.
Your setup

Unity Version 2019.4.5f1
MRTK Version v2.7.2

Target platform

HoloLens 2
OpenVR

Hello @SebastianMeinberger thank you for submitting this BoundsControl bug! We will investigate this and get back to you with our findings. Could you please add some additional details to your repro steps to help us investigate this bug faster? Thanks!


Dive hangs just after opening the main view

What happened:
I wanted to analyze a container with dive, but it hangs just after showing the main view. It's not responsive. Even Ctrl+C does not terminate the program.
What you expected to happen:
Use dive as it is intended to. Operate menu ect..
How to reproduce it (as minimally and precisely as possible):
I have no idea, why, It just hangs.
Anything else we need to know?:
It doesn't happen for other images.
Environment:

OS version: Linux  6.6.16-2-MANJARO #1 SMP PREEMPT_DYNAMIC Sat Feb 10 09:40:02 UTC 2024 x86_64 GNU/Linux
Docker version: 25.0.2, build 29cf629222

Is it an image you could share?

Same here. Fedora 39 / WSL2 / Podman. Even if downgraded to 0.11. The latter release did work for me on the same config, so it might not be something directly related to dive? 🤔

Same here.
Ubuntu 20.04.6 LTS
dive 0.12.0

Same exact issue for me, downloaded via brew.
Tried:

Reinstalling
Different terminals

No luck.
(Long time fan of the project, if you need any more info lmk ❤)
dive: 0.12.0
colima: 0.6.9
OS: macOS 14.5 23F79 arm64
Host: Mac15,7
Kernel: 23.5.0
Shell: zsh 5.9
CPU: Apple M3 Pro
Terminal: alacritty 0.13.2

Version 12.0 also has similar problem on Ubuntu 22.04 on custom docker image.

I have the same problem. Sadly i can't share the image because it includes intellectual property of my employer.
OS: Ubuntu 22.04
Kernel: 6.8.0-41-generic
Terminal: kitty 0.32.2
Shell: zsh 5.9
CPU: AMD Ryzen 7 7840U
dive: 0.12.0
docker: 27.1.2

The image is based on node-alpine and is used for a nuxt instance.

I was having the same issue.  Installed on Mac via brew install dive.  However, I had a very large image (5GB) and I just needed to wait longer.  After a couple minutes it finally displayed.


DD log retention is limited to 2w

mesh-perf action sends 6 log lines to DD once a day, later we build DD charts based on these logs. Unfortunately, log retention is limited to 2w without possibility to make an exception. We need to find a way to see the history of perf testing.

We could try push metrics but we need to be careful about cardinality


Add filetype to Science Museum script

Fixes

Related to #536 by @obulat
Description

This PR adds filetype and filesize to the list of data collected from the Science Museum API.
Testing Instructions

The tests for the script should pass.
Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no visible errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

And that works great! But I think the documentation should still read with keys of "measurement"."dimensions", "units", and "value". (since the top level key is still measurements rather than dimensions).

Ah, I didn't even see the docstring! 🫢

I am going to draft this for now so that the reviewers are not bothered, and if the new Provider Data Ingester is merged today, I'll move the work from here to #576.


[SPARK-7656] [SQL] use CatalystConf in FunctionRegistry

follow up for #5806

Merged build triggered.

Merged build started.

Test build #32757 has started for   PR 6164 at commit 15e6697.

Test build #32757 has finished for   PR 6164 at commit 15e6697.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class SimpleFunctionRegistry(val conf: CatalystConf) extends FunctionRegistry

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/SparkPullRequestBuilder/32757/
Test PASSed.

/cc @marmbrus

Thanks, merged to master and 1.4.


fix: add dev image with commit tag in GHA

fix: https://github.com/envoyproxy/gateway/pull/313#discussion_r961019619
cc @arkodg
before: https://github.com/envoyproxy/gateway/pull/313
Signed-off-by: Xunzhuo<EMAIL_ADDRESS>

Codecov Report

Merging #333 (b3b21c2) into main (c5138a3) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #333   +/-   ##
=======================================
  Coverage   58.26%   58.26%
=======================================
  Files          31       31
  Lines        2842     2842
=======================================
  Hits         1656     1656
  Misses       1087     1087
  Partials       99       99

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


UI issues on FreeBSD with st and Ncurses 5.9.20140222

Newsboat version:
newsboat 2.21.0
System: FreeBSD 12.2-RELEASE-p2 (amd64)
Compiler: g++ 4.2.1 Compatible FreeBSD Clang 8.0.1 (tags/RELEASE_801/final 366581)
ncurses: ncurses 5.9.20140222 (compiled with 5.9)
libcurl: libcurl/7.74.0 OpenSSL/1.1.1d zlib/1.2.11 nghttp2/1.42.0 (compiled with 7.74.0)
SQLite: 3.34.0 (compiled with 3.34.0)
libxml2: compiled with 2.9.10
Issue
On startup, newsboat randomly hides (i.e makes the whole line black) feeds and it only shows them if you hover over them. It might also apply wrong colors to a feed. If the feed happens to be unread, it highlights it even if you're not hovering over it, so you end up with several highlighted feeds at the same time. The highlighting goes away if you hover over it again.
Colors also get messed up when reading an article too.
Config file (only the following lines are part of the issue):
highlight all "---.*---" yellow
highlight feedlist ".*(0/0))" black
highlight article "(^Feed:.*|^Title:.*|^Author:.*)" cyan default bold
highlight article "(^Link:.*|^Date:.*)" default default
highlight article "https?://[^ ]+" green default
highlight article "^(Title):.*$" blue default
highlight article "\\[[0-9][0-9]*\\]" magenta default bold
highlight article "\\[image\\ [0-9]+\\]" green default bold
highlight article "\\[embedded flash: [0-9][0-9]*\\]" green default bold
highlight article ":.*\\(link\\)$" cyan default
highlight article ":.*\\(image\\)$" blue default
highlight article ":.*\\(embedded flash\\)$" magenta default

Without these lines in the config, some of the issues still persist.
Steps to reproduce the issue:

Add the lines I've included above in the config file.
Open newsboat.
Make sure there are unread feeds.

Startup (notice the yellow feeds as well)

Hovering over the "blacked out" feeds

Unread feeds

Hovering over the unread feeds (if you hover back, they'll get highlighted again)

Thanks! Does pressing ^L make the issue go away?

Thanks! Does pressing ^L make the issue go away?

Also, what version of STFL is Newsboat built against?

Also, what version of STFL is Newsboat built against?

Thanks! Does pressing ^L make the issue go away?

Quite the opposite, it created even more issues. :-)
^L either does nothing or it has very weird behaviour. After scrolling a bit back and forth and then pressing ^L, the highlighting of all the feeds got reversed. Then I kept scrolling down and every time I pressed ^L, all the feeds above the selected one got  either black or yellow.

Thanks! Does pressing ^L make the issue go away?

Quite the opposite, it created even more issues. :-)
^L either does nothing or it has very weird behaviour. After scrolling a bit back and forth and then pressing ^L, the highlighting of all the feeds got reversed. Then I kept scrolling down and every time I pressed ^L, all the feeds above the selected one got  either black or yellow.

Also, what version of STFL is Newsboat built against?

$ pkg info | grep stfl

stfl-0.24_1                    Structured Terminal Forms Language/Library

Also, what version of STFL is Newsboat built against?

$ pkg info | grep stfl

stfl-0.24_1                    Structured Terminal Forms Language/Library

Okay. I don't see a way for Newsboat to mess this up: ^L gets propagated straight to STFL and then Ncurses. You already have the latest version of the former; is there any chance you can build Newsboat against a newer version of Ncurses?
Another angle of attack would be the terminal.

Are you running Newsboat in a TTY, or is it a terminal emulator?
What does echo $TERM print out?
Can you please run in e.g. xterm and see if the problem persists there?

Okay. I don't see a way for Newsboat to mess this up: ^L gets propagated straight to STFL and then Ncurses. You already have the latest version of the former; is there any chance you can build Newsboat against a newer version of Ncurses?
Another angle of attack would be the terminal.

Are you running Newsboat in a TTY, or is it a terminal emulator?
What does echo $TERM print out?
Can you please run in e.g. xterm and see if the problem persists there?

Is there any chance you can build Newsboat against a newer version of Ncurses?

I might try it later but I'll post the results here.

Are you running Newsboat in a TTY, or is it a terminal emulator?

It's a terminal emulator.

What does echo $TERM print out?

st-256color

Can you please run in e.g. xterm and see if the problem persists there?

xterm seems to be rendering colors properly, so it might not be newsboat related.

Is there any chance you can build Newsboat against a newer version of Ncurses?

I might try it later but I'll post the results here.

Are you running Newsboat in a TTY, or is it a terminal emulator?

It's a terminal emulator.

What does echo $TERM print out?

st-256color

Can you please run in e.g. xterm and see if the problem persists there?

xterm seems to be rendering colors properly, so it might not be newsboat related.

Got it! FWIW, on Debian testing with st 0.8.4 and STFL 0.22, the rendering looks correct to me — no gaps like in your screenshots.
newsboat r2.22.1-130-gc7cf-dirty
System: Linux 5.10.0-1-amd64 (x86_64)
Compiler: g++ 10.2.1 20201224
ncurses: ncurses 6.2.20201114 (compiled with 6.2)
libcurl: libcurl/7.74.0 OpenSSL/1.1.1i zlib/1.2.11 brotli/1.0.9 libidn2/2.3.0 libpsl/0.21.0 (+libidn2/2.3.0) libssh2/1.9.0 nghttp2/1.42.0 librtmp/2.3 (compiled with 7.74.0)
SQLite: 3.34.0 (compiled with 3.34.0)
libxml2: compiled with 2.9.10

(The build is "dirty" due to unrelated changes to the docs, no code changes were made relative to c7cf0a3beadca44af19b168382f72f9a74fd9831)
It might also be a problem with st itself, so if you aren't running the latest version (or at least 0.8.4 that I did), it might be worth updating that one too. Looking forward to those results!

Got it! FWIW, on Debian testing with st 0.8.4 and STFL 0.22, the rendering looks correct to me — no gaps like in your screenshots.
newsboat r2.22.1-130-gc7cf-dirty
System: Linux 5.10.0-1-amd64 (x86_64)
Compiler: g++ 10.2.1 20201224
ncurses: ncurses 6.2.20201114 (compiled with 6.2)
libcurl: libcurl/7.74.0 OpenSSL/1.1.1i zlib/1.2.11 brotli/1.0.9 libidn2/2.3.0 libpsl/0.21.0 (+libidn2/2.3.0) libssh2/1.9.0 nghttp2/1.42.0 librtmp/2.3 (compiled with 7.74.0)
SQLite: 3.34.0 (compiled with 3.34.0)
libxml2: compiled with 2.9.10

(The build is "dirty" due to unrelated changes to the docs, no code changes were made relative to c7cf0a3beadca44af19b168382f72f9a74fd9831)
It might also be a problem with st itself, so if you aren't running the latest version (or at least 0.8.4 that I did), it might be worth updating that one too. Looking forward to those results!

Compiling from /usr/ports/www/newsboat with the most recent version of ncurses fixed the issue. Thanks a lot!

Compiling from /usr/ports/www/newsboat with the most recent version of ncurses fixed the issue. Thanks a lot!

Ah, awesome! Let's close this then.

Ah, awesome! Let's close this then.


pass View[String] into FileUtility#writeSql instead of String

Summary by CodeRabbit

Refactor

Enhanced the file writing process for better reliability and logging.

Bug Fixes

Improved error handling during file operations.

Codecov Report
Attention: 16 lines in your changes are missing coverage. Please review.

Comparison is base (677208f) 71.50% compared to head (ca568d4) 70.74%.

Files
Patch %
Lines

src/main/scala/utils/FileUtility.scala
0.00%
12 Missing :warning:

src/main/scala/Main.scala
0.00%
4 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #92      +/-   ##
==========================================
- Coverage   71.50%   70.74%   -0.77%
==========================================
  Files          20       20
  Lines         372      376       +4
  Branches       47       44       -3
==========================================
  Hits          266      266
- Misses        106      110       +4

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


Enable hostPort feature in CFCR

Enables hostPort access

The hostPort setting applies to the Kubernetes containers. The container port will be exposed to the external network at :, where the hostIP is the IP address of the Kubernetes node where the container is running and the hostPort is the port requested by the user.

In order to support the hostport we need

to have the latest CNI binaries
to have a chained cni config

This PR requires that we upload a new blob
bosh add-blob ~/Downloads/cni-plugins-amd64-v0.7.1.tgz cni/cni-plugins-amd64-v0.7.1.tgz
bosh upload-blobs

What this PR does / why we need it:
This PR enables in CFCR the ability of user to apply specs that have hostPort, use now can apply the following
apiVersion: v1
kind: Pod
metadata:
  name: nginx-hostport
spec:
  containers:
  - image: nginx:1.13-alpine
    imagePullPolicy: Never
    name: nginx
    ports:
    - containerPort: 80
      hostPort: 40801
  restartPolicy: Always

How can this PR be verified?
Either by inspecting the iptables rules of the worker where the pod lives
(iptables -nvL -t nat |grep 40801)
or simply by curl <worker-ip>:40801
Is there any change in kubo-deployment?
No
Is there any change in kubo-ci?
There should be a test since kubo-ci has the tests
Does this affect upgrade, or is there any migration required?
It should not affect  the upgrade process, nevertheless it would be clean if it comes with a recreation of the stemcell (e.g. to clean some old iptables rules)

@karampok why would the user use hostPort instead of nodePort?

@alex-slynko I am not sure of the exact use case but it has to do that some apps require to have low port on the host for initial syn reasons. HostPort seems to be supported in gke, azure but not in CFCR.
In my opinion, the code included is useful because it updates the cni plugins (which we have not updated long time) and move the flannel configuration to the conflist format which allows the use of the portmap cni plugin.


Rename InstructionScheduleMap to CircuitInstructionMap

Summary
Closes #3592. This PR renames InstructionScheduleMap to CircuitInstructionMap and deprecates InstructionScheduleMap.
Details and comments

Since both "circuit" and "instruction" are used in the circuit model, CircuitInstructionMap can be confusing as it is not explicit that you are mapping to a pulse schedule. InstructionScheduleMap makes more sense to me.

I'm not sure I agree. This attribute is already in the pulse module and therefore outputting schedules within this context is expected. The unspecified context is what it is mapping which in this case are circuit instructions and not pulse instructions which also exist. I think the best name would be CircuitInstructionToScheduleMap but that is pretty verbose.

@taalexander ping me whenever it's ready for review :)

can we do InstructionScheduleMap and I'll open a new PR with the attribute updated/deprecated only? let's get this over with


ReadMe file for generating API documentation

Coverage remained the same at 97.248% when pulling dee17361c9855a159ba37db96c220be39189998c on arafay/APIDoc into bec9d3b4837be2b74900cef127e63cc116f9f342 on master.


FA Image not mirrored correctly.

Example
I wonder if it's the fault of the source image, since Reddit also seems to struggle with the image preview, and my Reddit client (Alien Blue) did the same.

Looks like a duplicate of #18.


Authentication through GitHub Integrations

Thanks for the fantastic library!
I'm curious to know if there's any plan to support authentication through GitHub Integrations? Currently, it seems that there is support for authentication through HTTP/HTTPS, SSH w/Agent and GitHub Two Factor Auth.
See here for more details.
Thanks!

:wave:
You should be able to get a token for an integration that uses NodeGit and then provide that token as the password for git actions performed with the library.
https://developer.github.com/early-access/integrations/integrations-vs-oauth-applications/#git-access

@johnhaley81 thanks for the fast reply!
I presume the username will just be my GitHub account username?

I think so. At least it's a good first start :)

@johnhaley81
As per https://developer.github.com/early-access/integrations/integrations-vs-oauth-applications/#git-access

The token is used as the HTTP password.

Is the token in this context being referred to as the JSON Web Token or the Installation access token

I have to admit, I've haven't used this form of auth yet. You're going to have to experiment a bit.
I would use the token like we do here: https://github.com/nodegit/nodegit/blob/master/examples/cloneFromGithubWith2Factor.js

Okay, I'll keep pokin' around. Thanks!