fpga compiler command failed with exit code 1

Summary
fpga compiler command failed with exit code 1
Steps to reproduce
By following the tutorial, I can't compile program for FPGA.
Observed behavior
u50623@s001-n001:~/download/oneAPI-samples/DirectProgramming/DPC++/DenseLinearAlgebra/simple-add$ make hw -f Makefile.fpga
dpcpp -O2 -g -std=c++17 -fintelfpga a_buffers.o -o simple-add-buffers.fpga -Xshardware
aoc: Compiling for FPGA. This process may take several hours to complete.  Prior to performing this compile, be sure to check the reports to ensure the design will meet your performance targets.  If the reports indicate performance targets are not being met, code edits may be required.  Please refer to the oneAPI FPGA Optimization Guide for information on performance tuning applications for FPGAs.
Error (23035): Tcl error:
Error (23031): Evaluation of Tcl script build/entry.tcl unsuccessful
Error: Quartus Prime Shell was unsuccessful. 2 errors, 0 warnings
For more detail, full Quartus compile output can be found in files quartuserr.tmp and quartus_sh_compile.log.
Error: Compiler Error, not able to generate hardware
dpcpp: error: fpga compiler command failed with exit code 1 (use -v to see invocation)
Makefile.fpga:33: recipe for target 'simple-add-buffers.fpga' failed
make: *** [simple-add-buffers.fpga] Error 1
What should I do?

Alby is a member of our FPGA customer support organization - she can help you or assign someone to do so.

Hi there, it looks like you tried to execute the fpga compile on a DevCloud node that is not enabled to do so (s001-n001). Any FPGA compile jobs should be launched on nodes with the "fpga_compile" tag to ensure that you have the resources to complete it successfully. Any nodes not part of the "fpga_compile" group won't have the correct resources installed for an FPGA hardware compile.
You can see the DevCloud Getting Started page to see an example of compiling for FPGA with vector add: https://devcloud.intel.com/oneapi/get-started/base-toolkit/#fpga-vector-add-sample-walkthrough


Build for Apple M1 - Darwin ARM

Is your feature request related to a problem? Please describe.
Please could you provide an Apple M1 build in dasel's releases. I could not see any darwin_arm build.
Describe the solution you'd like
Build for Apple M1 - MacOS on ARM (Darwin_ARM)

Thanks to @reynn for his contribution here. New releases will include a darwin/arm64 release executable and a -alpine docker image.


xst not changing background

Hi there, experimenting with pywal and xst fork, working well in general, but when executing wal --theme random all the colors changing except background color. When opening new xst instance it shows proper background. Any suggestions how to fix this?

Sounds like a problem with xst and not pywal. The README says that the fork is in progress, and there's a list of things that aren't done yet.


How to use python models

I am looking all night long how to get a tensorflow python trained model inside tensorflowsharp. I know about freezing the model and importing it.  I dont know what I am doing wrong, I think it might be:
1 wrong encoding
2 not really a frozen model
3 an unsupported model (I am trying a simple soccer predicter using the DNNclassifier as an example)
I hope someone can help me how to solve this. I attached both python, my c# and current model and the exported model I made. I am willing to write an article/summary about it this workflow because I find tensorflow lacking in some parts of its documentation.
python
#starting the model
model = tf.estimator.DNNClassifier(
    feature_columns=feature_columns,
    hidden_units=[10,10,10],
    optimizer=tf.train.FtrlOptimizer(learning_rate=0.01,l1_regularization_strength=0.001),
    config = tf.estimator.RunConfig(model_dir="MODELDIR")
)

#doing some training
for i in range(2):
    model.train(input_fn=trainSet,max_steps=500000)

    accuracy_score = model.evaluate(
        input_fn=testSet,
        steps=500000
    )["accuracy"]
    print('Accuracy: {0:f}'.format(accuracy_score))

#generating export columns
feature_spec = tf.feature_column.make_parse_example_spec(feature_columns)
export_input_fn = tf.estimator.export.build_parsing_serving_input_receiver_fn(feature_spec)

#exporting the application
model.export_savedmodel(r'EXPORTDIR',export_input_fn )

c#
using (var graph = new TFGraph())
        {

            var modelFile = File.ReadAllBytes(@"modelDir");

            //here it starts throwing errors about invalid graphdef

            graph.Import(modelFile);

            var session = new TFSession(graph);
            var runner = session.GetRunner();
            runner.AddInput(graph["Input/IsTraining"][0], false);

            runner.AddInput(graph["Input/Values"][0], new TensorFlow.TFTensor(TFDataType.Float, new long[] { 1, 44, 60 }, 1 * 44 * 60 * 4));
            runner.Fetch(graph["Output/Predictions"][0]);

            var output = runner.Run();

            // Fetch the results from output:
            TFTensor result = output[0];
        }

'
saved_model.zip

Can you attach the actual exception?

model.export_savedmodel(r'EXPORTDIR',export_input_fn )
...
var modelFile = File.ReadAllBytes(@"modelDir");
graph.Import(modelFile);

SavedModel fromat is not just single model-file, it's directory with certain structure, look:
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/python/saved_model/README.md
Try to use TFSession.FromSavedModel to load model in SavedModel format.

Added some information to the documentation

@klaas3 @alex-zu  can TFSession.FromSavedModel solve your problem? I tried to use this, but I can not graph["inputs"] is null

@yiylin Try to use saved_model_cli to discover your model signature:

$saved_model_cli show --dir
/tmp/saved_model_dir --tag_set serve --signature_def serving_default
The given SavedModel SignatureDef contains the following input(s):
inputs['x'] tensor_info:
dtype: DT_FLOAT
shape: (-1, 1)
name: x:0
The given SavedModel SignatureDef contains the following output(s):
outputs['y'] tensor_info:
dtype: DT_FLOAT
shape: (-1, 1)
name: y:0
Method name is: tensorflow/serving/predict

here "name: x" - x is name of input," name: y" - y is name of output.
In TFSharp use like:
var input = graph["x"][0];
..
var runner = session.GetRunner().Fetch("y", 0);

@alex-zu thanks. Your reply help me.

hi @alex-zu and @yiylin . Do you have a example how to use the TFSession.FromSavedModel I have a hard time to fill the metaGraphDef. How to declare it correctly?

@klaas3 Hello, metaGraphDef is an output parameter and optional in native, but for some reason @migueldeicaza check it for null. Just pass the dummy TFBuffer.
//instance fields
private TFGraph _tfGraph;
private TFSession _tfSession;

...

//load SavedModel
_tfGraph = new TFGraph();

using (var tmpSess = new TFSession(_tfGraph))
using (var tfSessionOptions = new TFSessionOptions())
using (var metaGraphUnused = new TFBuffer())
{
	//for some reason FromSavedModel is not static
	_tfSession = tmpSess.FromSavedModel(tfSessionOptions, null, exportDir, new[] { "serve" }, _tfGraph, metaGraphUnused);
}

model.export_savedmodel(r'EXPORTDIR',export_input_fn )
...
var modelFile = File.ReadAllBytes(@"modelDir");
graph.Import(modelFile);

SavedModel fromat is not just single model-file, it's directory with certain structure, look:
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/python/saved_model/README.md
Try to use TFSession.FromSavedModel to load model in SavedModel format.

So I wanted to ask about the current accuracy of the statement that the saved model is not just a single file.  There is an example that does this in TensorFlowSharp using the MNist dataset.  Python code generates a model and saves the model to a "pb" file.  The "pb" file is then loaded in a C# application using TensorFlowSharp:
`byte[] buffer = System.IO.File.ReadAllBytes(modelfile);
int countOfFailedClassifications = 0;
using (var graph = new TensorFlow.TFGraph())
{
graph.Import(buffer);
using (var session = new TensorFlow.TFSession(graph))
{
foreach (string file in filesTestingRandomized)
{
Stopwatch sw = new Stopwatch();
sw.Start();
var runner = session.GetRunner();
var tensor = Utils.ImageToTensorGrayScale(file);
runner.AddInput(graph["conv1_input"][0], tensor);
runner.Fetch(graph["activation_4/Softmax"][0]);
                    var output = runner.Run();
                    var vecResults = output[0].GetValue();
                    float[,] results = (float[,])vecResults;
                    sw.Stop();
                    ///
                    /// Evaluate the results
                    ///
                    int[] quantized = Utils.Quantized(results);
                    ///
                    /// Use the parent folder name to deterimine the expected digit
                    ///
                    string parentFolder = System.IO.Directory.GetParent(file).Name;
                    int iParentFolder = int.Parse(parentFolder);
                    int[] expected = Utils.GetQuantizedExpectedVector(iParentFolder);
                    bool success = quantized.SequenceEqual(expected);
                    if (!success) countOfFailedClassifications++;
                    string message = $"Directory={parentFolder}    File={System.IO.Path.GetFileName(file)}    Bit1={results[0, 0]} Bit2={results[0, 1]}   Elapsed={sw.ElapsedMilliseconds} ms, Success={success}";
                    Console.WriteLine(message);
                }
            }
        }`


More efficient incremental updates

Describe the problem you are trying to solve
Toolchain updates are currently very inefficient, particularly with documentation, most of the new content is identical to the previous content; this wastes internet bandwidth for low bandwidth regions, and we currently churn on disk - removing the unaltered content and then replacing it, which has knock on effects on search indices, for a net waste of CPU and battery
Describe the solution you'd like
Installing an update that is 99% the same as the previous version would only take 1% of the resources - a small download and a small number of writes to disk.
Notes
This was discussed in the rustup-wg meeting 23/4/19; it isn't currently resourced but it was considered an interesting idea.
I think we have broadly two basic performance sensitive use cases:

First install: whether interactive or automated for a CI job.
This needs to be decently fast, so we cannot ignore its performance.
No atomicity is needed: rust isn't working before the install begins, and until it completes thats understood.
Daily/adhoc maintenance: almost always human initiated I suspect, but perhaps not?
This needs to also be decently fast, but ideally it would be faster than the initial install (currently it is slower because we have to remove the old content)
As rust was working before it was run, minimising the time period during which a given toolchain doesn't work is desirable

I'm not sure if the rollback/transactional aspect really is important to preserve as-is; I rather suspect that a recoverable model would be better - more flexible for this sort of optimisation, and a better fit for reality (because reality doesn't guarantee processes can complete).
A rustup modified to work like this would look something like the following:

an additional server side step taking the various components and putting them into some distribution framework (e.g. rsync or bittorrent or <...>). This may require a server side component to run, or we may be able to use dumb servers with precalculated information - this is a framework specific consequence
rustup initial installation of a toolchain can proceed as it does today unaltered
rustup updates to a toolchain would use the distribution framework in situ, rather than staging a new entire component version, deleting the old one, and moving the staged one into position.
if an in-situ update fails or is interrupted, running it again would converge it on the desired content, though this might be less efficient than the clean case from known-state a to known-state a'.

There are some possible complications - e.g. do we have components nested within others, which many distribution systems are unlikely to like? - but these should be able to be worked through with some care.
If someone wants to test to see what the potential benefits might be, just measure updating .rustup/toolchains from another machine using some delta system (rsync/dropbox/bittorrent/etc) ; key examples would be nightly and stable.

This does sound great to have. I believe our servers at the moment are very dumb, and I think we probably need to keep them that way, but if we can pre-calculate the information we need, that would be fine.

I don't have the time to take on building this at this point, but I do have the time to help someone interested work through technology choices, interactions and implications on different platforms, and likely constraints/desired capabilities for the components we install to make this really fly.

I would like to take this. @rbtcollins What do I need to do?


chore(package): update kronos-interceptor to version 2.4.14

https://greenkeeper.io/

Codecov Report

Merging #47 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #47   +/-   ##
=======================================
  Coverage   83.33%   83.33%
=======================================
  Files           1        1
  Lines          12       12
=======================================
  Hits           10       10
  Misses          2        2

Continue to review full report at Codecov.

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


Hard delete doesn't propagate to downstreams beyond first layer

Hard deleting a node invalidates only the first layer of downstream nodes, but it doesn't propagate down to all subsequent layers of downstreams. For example, when a source node is hard deleted, it will invalidate a transform that depends on it, but the metrics that depend on the transform remain valid. They should all be invalid.

Thanks @shangyian, good catch! It looks like it's just relying on get_downstream_nodes. I wonder if that function isn't including the metrics in your example.


dircolors: align TERM matching behavior with that of GNU dircolors

Fixes #3761 by accepting both ^ and ! for negation.

Are we good to merge this?

it think so, thanks for the ping


docs: typo lable to label

Description

This pull request addresses the issue described here: #<?>

In this pull request, I have …

For features/enhancements:

[x] I have added/updated the relevant references in the documentation files.

For API changes:

[x] I have made the necessary changes in the types index file.

Thanks!


Add Kalispell, MT event location for GDCR2018

I followed the steps on the register event page found here: https://www.coderetreat.org/register-event/

Welcome 🎉


Board Review: Azure Maps C# SDK

Thank you for submitting this review request. Thorough review of your client library ensures that your APIs are consistent with the guidelines and the consumers of your client library have a consistently good experience when using Azure.
The Architecture Board reviews Track 2 libraries only. If your library does not meet this requirement, please reach out to Architecture Board before creating the issue.
Please reference our review process guidelines to understand what is being asked for in the issue template.
To ensure consistency, all Tier-1 languages (C#, TypeScript, Java, Python) will generally be reviewed together.  In expansive libraries, we will pair dynamic languages (Python, TypeScript) together, and strongly typed languages (C#, Java) together in separate meetings.
For Tier-2 languages (C, C++, Go, Android, iOS), the review will be on an as-needed basis.
Before submitting, ensure you adjust the title of the issue appropriately.
Note that the required material must be included before a meeting can be scheduled.
Contacts and Timeline

Responsible service team: Azure Maps
Main contacts: charlie.chen@
Expected code complete date: July/2022
Expected release date:  Aug./2022

About the Service

Link to documentation introducing/describing the service: https://docs.microsoft.com/en-us/azure/azure-maps/
Link to the service REST APIs:  https://github.com/Azure/azure-rest-api-specs/tree/main/specification/maps/data-plane
Link to GitHub issue for previous review sessions, if applicable: https://github.com/Azure/azure-sdk/issues/4186

About the client library

Name of the client library: Azure Maps SDK - Search, Route, Render, GeoLocation modules.
Languages for this review: C#

Artifacts required (per language)
Please read through “API Review” section here to understand how these artifacts are generated. It is critical that these artifacts are present and are in the right format. If not, the language architects cannot review them with the SDK Team’s API review tool.
.NET
Search - https://apiview.dev/Assemblies/Review/0c8fa90052e24a5fb7f7e6b453ea6b10
Route - https://apiview.dev/Assemblies/Review/90d040066ce04f5ba45b7894aa148ad3
Render - https://apiview.dev/Assemblies/Review/e2c2b4239085400bbec0e35d10752270
Hero scenarios:
Search - Search an address
Route - https://github.com/dubiety/azure-sdk-for-net/tree/feature/maps-route/sdk/maps/Azure.Maps.Route#route-directions
Render - https://github.com/dubiety/azure-sdk-for-net/tree/feature/maps-render/sdk/maps/Azure.Maps.Render#render-images
For all other languages, send a request to the Architecture Board to discuss the best format on individual basis.
Thank you!

Scheduled for 8/9 4p-6p PDT

@dubiety, when you say that expected release date is 8/22, do you mean preview or GA?

Recording (MS INTERNAL ONLY)

We mean preview here.


Update Readme.md

What this PR does / why we need it
Which issue(s) this PR fixes

Fixes #
Special notes for your reviewer
Does this PR introduce a user-facing change?

None

/kind documentation
/lgtm
/approve


Extend Pytorch

Release Pytorch extension for working with Fiboa + AI extension, and ideally fully working with ASU dataset, with examples.

I think this is done.


Using test utilities to produce a fake media source which emulates buffering behaviour

I'm trying to write some unit tests and using the existing tests as a guide (like this one: https://github.com/androidx/media/blob/d833d59124d795afc146322fe488b2c0d4b9af6a/libraries/exoplayer/src/test/java/androidx/media3/exoplayer/analytics/PlaybackStatsListenerTest.java)
I'm trying to test the behaviour of a custom Player.Listener that should react only once for each media item that starts playing. Using FakeMediaSource I can emulate most of the behaviours I'm interested in covering. However one behaviour I'd like to cover is content which buffers part way through.
If I set a real media item on the player from a remote mp3 url and observe the events, I get a tonne of onPlaybackStateChanged and onIsPlayingChanged events as it swaps between buffering and ready.
I'd like to have a test that covers this behaviour but if possible I'd like to do it using local resources rather than my current method of hitting a real mp3 on a remote server.
Is there a way to emulate this using FakeMediaSource or one of its subclasses? Or would I need to put a local mp3 file in my test resources and load that?

I think this can be controlled by the FakeMediaPeriod.TrackDataFactory that is passed to the FakeMediaSource constructor. It produces the samples put into the queues for playback. By default it's a single sample following the the end-of-stream signal. See here. If you leave out the end-of-stream signal you create partially buffered sources that don't progress beyond a certain point.
If you need to let them load more data at a later point in the test, you'd need to go one level down and inject your own custom FakeSampleStream (by overriding createMediaPeriod and createSampleStream, see this example). FakeSampleStream has an append method to add more samples to the list as needed.

Hmm, it seems that creating a custom FakeMediaSource which overrides the sample stream to omit the end of stream signal, or to introduce more sample items doesn't seem to have an effect in the test. It doesn't appear to trigger the buffering behaviour that is expected.
In this setup the runUntilPlaybackState(player, Player.STATE_BUFFERING) call times out.
val mediaItem = MediaItem.Builder().setMediaId("TEST_ID").build()
val fakeMediaSource = object : FakeMediaSource(timelineForMediaItem(mediaItem, 10.seconds)) {
    override fun createMediaPeriod(
        id: MediaSource.MediaPeriodId,
        trackGroupArray: TrackGroupArray,
        allocator: Allocator,
        mediaSourceEventDispatcher: MediaSourceEventListener.EventDispatcher,
        drmSessionManager: DrmSessionManager,
        drmEventDispatcher: DrmSessionEventListener.EventDispatcher,
        transferListener: TransferListener?
    ): MediaPeriod = object : FakeMediaPeriod(
        trackGroupArray,
        allocator,
        { _, _ -> ImmutableList.of() },
        mediaSourceEventDispatcher,
        drmSessionManager,
        drmEventDispatcher,
        false
    ) {
        override fun createSampleStream(
            allocator: Allocator,
            mediaSourceEventDispatcher: MediaSourceEventListener.EventDispatcher?,
            drmSessionManager: DrmSessionManager,
            drmEventDispatcher: DrmSessionEventListener.EventDispatcher,
            initialFormat: Format,
            fakeSampleStreamItems: MutableList<FakeSampleStream.FakeSampleStreamItem>
        ): FakeSampleStream = FakeSampleStream(
            allocator,
            mediaSourceEventDispatcher,
            drmSessionManager,
            drmEventDispatcher,
            initialFormat,
            listOf(
                FakeSampleStreamItem.oneByteSample(5.seconds.inWholeMicroseconds),
                FakeSampleStreamItem.oneByteSample(5.seconds.inWholeMicroseconds)
            )
        )
    }
}
player.setMediaSource(fakeMediaSource)
player.prepare()
player.play()

TestPlayerRunHelper.playUntilPosition(player, 0, 1.seconds.inWholeMilliseconds)
TestPlayerRunHelper.runUntilPlaybackState(player, Player.STATE_BUFFERING)

It's not running because

The source doesn't publish any tracks, so the sample stream logic isn't even called. You can fix this by adding a Format to the FakeMediaSource constructor, e.g. FakeMediaSource(timelineForMediaItem(mediaItem, 10.seconds), ExoPlayerTestRunner.VIDEO_FORMAT)
The first sample at least needs to be a keyframe to start playback, which can be set with FakeSampleStreamItem.oneByteSample(5.seconds.inWholeMicroseconds, C.BUFFER_FLAG_KEY_FRAME)

If I make both these changes, the test runs through as expected.

Thanks, I'll give that a try

Hmmm, I'm still not seeing the expected behaviour with those changes:
private class FakeBufferingMediaSource(
    mediaItem: MediaItem
) : FakeMediaSource(timelineForMediaItem(mediaItem, 10.seconds), ExoPlayerTestRunner.VIDEO_FORMAT) {

    override fun createMediaPeriod(
        id: MediaSource.MediaPeriodId,
        trackGroupArray: TrackGroupArray,
        allocator: Allocator,
        mediaSourceEventDispatcher: MediaSourceEventListener.EventDispatcher,
        drmSessionManager: DrmSessionManager,
        drmEventDispatcher: DrmSessionEventListener.EventDispatcher,
        transferListener: TransferListener?
    ): MediaPeriod = object : FakeMediaPeriod(
        trackGroupArray,
        allocator,
        { _, _ -> ImmutableList.of() },
        mediaSourceEventDispatcher,
        drmSessionManager,
        drmEventDispatcher,
        false
    ) {
        override fun createSampleStream(
            allocator: Allocator,
            mediaSourceEventDispatcher: MediaSourceEventListener.EventDispatcher?,
            drmSessionManager: DrmSessionManager,
            drmEventDispatcher: DrmSessionEventListener.EventDispatcher,
            initialFormat: Format,
            fakeSampleStreamItems: MutableList<FakeSampleStreamItem>
        ): FakeSampleStream = FakeSampleStream(
            allocator,
            mediaSourceEventDispatcher,
            drmSessionManager,
            drmEventDispatcher,
            initialFormat,
            listOf(
                FakeSampleStreamItem.oneByteSample(5.seconds.inWholeMicroseconds, C.BUFFER_FLAG_KEY_FRAME),
                FakeSampleStreamItem.oneByteSample(5.seconds.inWholeMicroseconds),
                FakeSampleStreamItem.END_OF_STREAM_ITEM
            )
        )
    }

}

Logs during test output (I have a listener set on the player which prints the different events)

The logs I expect to see with content that buffers (from a real remote mp3 media source)

With the changes suggested above, removing the END_OF_STREAM_ITEM from the list causes the test to timeout (waiting for STATE_ENDED when it will never come), so clearly the sample list is having some effect here.

That sounds like working as intended I think. If you add the END_OF_STREAM_ITEM sample, the player will just play these sample and then go the ended state. If you omit END_OF_STREAM_ITEM., then the player will stay in a buffering state and can't make further progress (and in particular never reaching STATE_ENDED).
As per my original comment, you can change the sample stream later with the append method. So you can have a test setup like
// Create player and set up FakeMediaSource without END_OF_STREAM_ITEM
player.prepare();
player.play();
runUntilPlaybackState(player, Player.STATE_BUFFERING);

// Make assertions about your app here that depend on this buffering.

fakeSampleStream.append(END_OF_STREAM_ITEM); // Allow the stream to end.
runUntilPlaybackState(player, Player.STATE_ENDED);

Even using append I'm a bit lost here.
I've updated my FakeBufferingMediaSource to allow for appending sample items (including calling writeData on the sample stream which seems to be needed for things given to the append function to be written).
internal class FakeBufferingMediaSource(
    timeline: Timeline
) : FakeMediaSource(timeline, ExoPlayerTestRunner.VIDEO_FORMAT) {

    private lateinit var sampleStream: FakeSampleStream

    fun appendNextSample(item: FakeSampleStreamItem) {
        sampleStream.append(listOf(item))
        sampleStream.writeData(0)
    }

    override fun createMediaPeriod(
        id: MediaSource.MediaPeriodId,
        trackGroupArray: TrackGroupArray,
        allocator: Allocator,
        mediaSourceEventDispatcher: MediaSourceEventListener.EventDispatcher,
        drmSessionManager: DrmSessionManager,
        drmEventDispatcher: DrmSessionEventListener.EventDispatcher,
        transferListener: TransferListener?
    ): MediaPeriod = object : FakeMediaPeriod(
        trackGroupArray,
        allocator,
        { _, _ -> ImmutableList.of() },
        mediaSourceEventDispatcher,
        drmSessionManager,
        drmEventDispatcher,
        false
    ) {
        override fun createSampleStream(
            allocator: Allocator,
            mediaSourceEventDispatcher: MediaSourceEventListener.EventDispatcher?,
            drmSessionManager: DrmSessionManager,
            drmEventDispatcher: DrmSessionEventListener.EventDispatcher,
            initialFormat: Format,
            fakeSampleStreamItems: MutableList<FakeSampleStreamItem>
        ): FakeSampleStream = FakeSampleStream(
            allocator,
            mediaSourceEventDispatcher,
            drmSessionManager,
            drmEventDispatcher,
            initialFormat,
            listOf()
        ).also { sampleStream = it }
    }

}

In my test though, nothing at all happens until an END_OF_STREAM_ITEM is appended.
fun `play event triggered once for content that buffers midway through`() {
    val mediaItem = MediaItem.Builder().setMediaId("TEST_ID").build()
    val source = FakeBufferingMediaSource(
        timeline = timelineForMediaItem(mediaItem, 10.seconds)
    )
    player.setMediaSource(source)
    player.prepare()
    player.play()
    TestPlayerRunHelper.runUntilPendingCommandsAreFullyHandled(player)

    source.appendNextSample(oneByteSample(2.seconds.inWholeMicroseconds, C.BUFFER_FLAG_KEY_FRAME))

//   TestPlayerRunHelper.runUntilPlaybackState(player, Player.STATE_BUFFERING)
//   source.appendNextSample(oneByteSample(2.seconds.inWholeMicroseconds, C.BUFFER_FLAG_KEY_FRAME))

//    TestPlayerRunHelper.runUntilPlaybackState(player, Player.STATE_BUFFERING)
//    source.appendNextSample(END_OF_STREAM_ITEM)

    TestPlayerRunHelper.runUntilPlaybackState(player, Player.STATE_ENDED)

    playEvents.map { it.mediaItem } shouldContainExactly listOf(mediaItem)
}

With the END_OF_STREAM_ITEM line commented out, nothing even begins to play, it times out waiting for a STATE_ENDED event that never comes but beyond that there isn't even a STATE_READY event that occurs.

If I uncomment just that line appending the END_OF_STREAM_ITEM suddenly the READY state occurs:

The other commented lines appending new samples have no effect

Sorry, I didn't get back to this conversation. I haven't tried it yet, but I think your test may be blocked on the (Default)LoadControl requiring a minimum amount of buffered data to become 'ready'.

I tried playing around a little with the LoadControl on the fake player, setting different parameters on it and specifying larger and larger blocks of sample data in the sample stream. Doesn't seem to have had an effect :(

I'm afraid we can't debug your setup in detail. Simulating a "buffering" source is quite hard because of the many moving parts that need to stay in the right position, so I could mark it as a testing utils enhancement to provide something like that out of the box (it will be low-priority though).
As an alternative, if you just need a buffering Player (not ExoPlayer) for your test, you could use a SimpleBasePlayer to modify the current state in any way you like.


Bump "Tested up to" for WooCommerce 4.1

Now that WooCommerce 4.1 is available, ensure we're compatible (which, based on our CI/CD pipeline, we are).

Closed with #29


Added createOptions scriptLocation

Adding a create option where the script, which contains REACT_ENGINE, can be added to the head of the html.  Ideally I would like an option to not automatically render the script tag automatically plus a function to generate the script which allows me to put it where I want in the html component manually, but this works for now.
--- Usage ---
const engine = reactEngine.server.create({
routes: require('./routes.jsx'),
routesFilePath: './routes.jsx',
scriptLocation: 'head',
page404: require('./not-found.jsx'))
})

Would you mind updating the readme Server options spec section?  Also could we use scriptLocation instead of scriptLoc?

@vuhwang I will get that changed to scriptLocation as well as update the README.  Thanks for taking a look.

@vuhwang ok the scriptLocation changes have been made

thanks @JustinMGaiam. v3.3.0 has been published with this change.


Please add vc5 to emerging applications list

Project is eBPF/XDP based, open source (GPL2), uses a founder-leader model, under active development and has been used in production for over a year.
Many thanks.

Thanks for the addition, just one small nit

Thanks for the addition, just one small nit

Well, you did go to Oxford after all!

Exactly ;) thanks for the addition


Add names, locations and github usernames of all pod members

Please, go through these steps before you submit a PR.

[X] My Pod Leader knows I'm working on this Pull Request
[X] I've explained what the Pull Request is adding.
[X] I've explained why this is important.

Fixes #13

Hi @akshgpt7, can you please review this PR?

Also, I don't see where you've uploaded the images and they're not showing up in the site either. can you let me know if you committed them?

Also, I don't see where you've uploaded the images and they're not showing up in the site either. can you let me know if you committed them?

Actually, me and @ohsweetmani divided our tasks so we can work on them asynchronously... so I was working on updating the names and locations of the fellows and Priyanka will be updating the images of the fellow

Please review this pull request,  @akshgpt7

Hi @akshgpt7, I and @ohsweetmani have resolved the merge conflicts. Please check


Update javadoc

Ensure all public API javadoc is correct and accounts for changes in terminology like Datastore → DocumentStore

As of now the only outstanding javadoc should be index/query, which can be done once this is addressed

index/query javadoc was in #471 so closing


MCS-393 Removed unused AI2-THOR prefabs and materials from the Resources folder

To optimize runtime performance. We can always copy the AI2-THOR files back into Resources from the Physics, Prefabs, or QuickMaterials folders.

Did something get missed for deletion in unity/Assets/Resources/MCS/AI2-THOR/Objects/Physics/SimObjsPhysics/Kitchen Objects/Bowl/SourcePrefabs.meta?

@rartiss55 Yeah I wasn't completely sure what that file was being used for, but I think you're right, I don't think we actually need it.


Skip first state change of EXCHANGE Partition.

It only generates warnings/errors and the first state change does not affect TiFlash, it only sets an internal field ExchangePartitionInfo
It is only the second stage transition that is interesting for TiFlash
Should be combined with https://github.com/pingcap/tidb/issues/46125
What problem does this PR solve?
Issue Number: close #xxx
Problem Summary:
What is changed and how it works?
Check List
Tests

[ ] Unit test
[x] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

https://github.com/pingcap/tiflash/blob/c7bce89b46fdee97dc55ea916d49a2a519d1569e/tests/fullstack-test2/ddl/alter_exchange_partition.test#L15-L16
alter_exchange_partition.test also need to be enabled in this pr

/check-issue-triage-complete

Please format your code with
python3 format-diff.py --diff_from `git merge-base xxx/master HEAD`

/run-all-tests

/run-all-tests

/cherry-pick release-6.5

/cherry-pick release-7.1


[Landing Page UX] Simplify search / filter UI experience

Problem
Currently, the search and filter UI display too many elements (6 buttons and 3 search fields), making it confusing for users to navigate through the various posts on the feed:

Solution
This ticket introduces a simplified search and filter UI of one search box and three dropdown filter buttons.

Scope
The scope of this ticket is to replace the existing 9 fields with the 4 new ones. There will be separate tickets to address the create experience, boards, and sorting functionality.
User Story
As a user, I want to have a simplified search and filter UI experience so that I can easily find posts.
Acceptance Criteria

A user sees the updated search/filter UI elements (Search box, Sort dropdown, Author dropdown, and Label dropdown) per the design
The search box has a minor UI change per the design with no functionality change
The sort label will have a dropdown with three options:

Latest – This is the default selection. It loads the most recent top-level posts and replaces the existing home button
Hottest – This shows the most popular posts and replaces the existing hottest button
All replies – This shows all the replies (both parent or child level) in most recent order and replaces the existing recent button

The author and label sort dropdowns reveal a search box with the list of labels or authors.

Attachments
Figma design:
https://www.figma.com/file/nTfDxzBc1RbjMxUA0bqKNF/Search-%2F-Filter-UI?node-id=0%3A1&t=7dCdqS2Y1QqAqnpd-1

@ailisp Can you please size this and let me know if you have any questions? I assume this might be a good first issue ticket? If so, can you please add that label? Thank you.

The author and Label dropdown is a dropdown with a search box. It resembles github's issue filter by author / label and needs to implemented as a nearsocial widget. We cannot reuse TypeAhead component given required functionality is not exported by nearsocial viewer. I expect this part would takes some time so I make the estimation to 3 days.

@ori-near Would love to take up this ticket

Hi @ben-jamming-reilly – just wanted to check in on this ticket and see if you need anything from @ailisp or @frol?

@ori-near It seems that it was addressed in #122. Can you check DevHub and submit a new issue if there is anything that is left to be implemented?


Update generated names for Extensions

This looks like the right behavior to me for extension naming; I'll give @tbkka a chance to look at it as well.

Assuming no other comments, I'll squash/commit this tomorrow.


isVisible() is not a function

Version
1.0.0-beta.11
Reproduction link
https://codesandbox.io/s/mo2r2y6j09
Steps to reproduce
Use the function isVisible() according to the documentation.
What is expected?
The test utilities check whether an element in the wrapper is visible.
What is actually happening?
A TypeError occurs:
TypeError: el.find(...).visible is not a function

This is related to #327, the method indeed exists but is called visible() and not isVisible().

#425 will resolve this.

This is fixed in beta.12

There is no type for isVisible

Are you able to make a PR?

Hey, guys! What is the status of this issue? Is it released? I have the same issue

I still have the problem. The last version.

@TheDeveloperTom  Are you using @vue/test-utils?

@eddyerburgh using what?

@eddyerburgh thank you

     expect(wrapper.find('.panel').isVisible()).to.be(false);// isVisible
Object is not a constructor (evaluating 'expect(wrapper.find('.panel').isVisible()).to.be(false)')


Dragonrage: Watch sexouaq.com

Dragonrage requests the watch of the watch_keyword sexouaq\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

sexouaq\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Nisse Engström in Charcoal HQ


[KYUUBI #2463] Redact kyuubi.ha.zookeeper.auth.digest in Spark engine

Why are the changes needed?
close #2463
spark.kyuubi.ha.zookeeper.auth.digest configuration item will see account and password in Spark UI.
use spark.redaction.regex.
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[x] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

Fix

Codecov Report

Merging #2464 (d394ff9) into master (3ecdd42) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##             master    #2464   +/-   ##
=========================================
  Coverage     63.18%   63.18%
  Complexity       69       69
=========================================
  Files           366      366
  Lines         17442    17446    +4
  Branches       2342     2342
=========================================
+ Hits          11020    11023    +3
+ Misses         5396     5395    -1
- Partials       1026     1028    +2

Impacted Files
Coverage Δ

...rg/apache/kyuubi/engine/spark/SparkSQLEngine.scala
77.84% <100.00%> (+0.51%)
:arrow_up:

...ache/kyuubi/operation/KyuubiOperationManager.scala
94.44% <0.00%> (-1.86%)
:arrow_down:

.../org/apache/kyuubi/operation/KyuubiOperation.scala
66.66% <0.00%> (-1.34%)
:arrow_down:

...in/scala/org/apache/kyuubi/config/KyuubiConf.scala
96.44% <0.00%> (-0.12%)
:arrow_down:

...rg/apache/kyuubi/engine/trino/TrinoStatement.scala
71.26% <0.00%> (+2.29%)
:arrow_up:

Continue to review full report at Codecov.

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

thanks, merging to master


fix: double quoted string in function

Closes #196
This PR addresses an issue with the SQL function that accepts double-quoted strings. The function works correctly when the string is an existing column but fails otherwise.
Examples:

Supported: SELECT length("username") FROM users;
Not supported: SELECT length("foobar") FROM users;

@JeanArhancet I am closing this pull request because I believe the current behavior is correct. Double quotations in SQL are there to specify identifiers (table names, column names, etc.) in case sensitive manner. Single quotations are used for string literals.


BI Book Power BI Connector Instructions

Added instructions for how to use BI Book Power BI Connector.

@microsoft-github-policy-service agree company="Renance - Automated Financial Services Oy"

@winbergo - Thanks for your pull request. A member of the content team will review it.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


The program can't run in "on created sprite of kind" s help

Describe the bug
The program can't run in "on created sprite of kind" s help
Steps to reproduce the behavior
1.Navigate to https://arcade.makecode.com/beta#
2.Drag on created sprite of kind in Sprites
3.Right-Click----->help
4.Run the program in the help doc
Actual behavior
The program can't run in "on created sprite of kind" s help

Additional context
1.OS: Windows(rs6)
2.arcade version: 1.3.16
3.Microsoft MakeCode version: 6.6.16

This still reproes.

This still reproes.


SSS innocent/traitor examine is broken

It shows innocent on traitors sometimes.

Closing in favor of #2902, this was supposed to be fixed by PJB


Comparison between Caffe and ArmnnCaffeParser processing time

I'm trying to compare the processing time running a convolutional network with Caffe C++ library and ArmnnCaffeParser built on ARM ComputeLibrary. These are details:
target architecture: Freescale IMX6Q
input image: 256x256 pixel and 3 channels
architecture:
conv2d 3x3 - 32 filters + RELU
conv2d 3x3 - 32 filters + RELU
max pool 2x2
conv2d 3x3 - 64 filters + RELU
conv2d 3x3 - 64 filters + RELU
max pool 2x2
conv2d 3x3 - 128 filters + RELU
max pool 2x2
conv2d 3x3 - 256 filters + RELU
max pool 2x2
conv2d 3x3 - 256 filters + RELU
max pool 2x2
fully connected layer with 48 output
relu
fully connected layer with 3 output
softmax
processing time:

Caffe c++ library (without ComputeLibrary optimization) : 4.5 seconds
ArmnnCaffeParser (with ComputeLibrary optimization): 2.2 seconds

I'm a little bit disappointed since my expectations were to boost the processing time more than a factor 2
Do you think that I'm doing something wrong or is this improvement factor comparable with your benchmarks?
Thanks

Hi @liviolima80 we're seeing about 5 times faster than upstream Caffe across various tests on Cortex-A72 and Cortex-A54 using the NEON backend. I'm afraid we don't have benchmarking running on Cortex-A9, but realistically a smaller speedup would be expected.

Do you have public benchmark with the comparison to official Caffe library?

Hi @liviolima80 multi-threading is high up on our roadmap. In the meantime, you can try calling the methods of arm_compute::Scheduler yourself. The documentation for that is here: https://arm-software.github.io/ComputeLibrary/v18.03/classarm__compute_1_1_scheduler.xhtml

Please have a look to this [question] (https://github.com/ARM-software/ComputeLibrary/issues/387#issuecomment-373679698)

Hi @AnthonyARM,
thank you for the link. I tried to change the number of threads in the scheduler setting but it seems that the better performance are obtained with the default config that is equal to 4 threads, that corresponds to the number of cores in the Cortex A9 architecture I'm using.
I'm looking forward to see if next releases of armnn will have some further improvement


Add new committers

There are 9 more current committers that are missing from that list. Shall I add those too? I lack their organizations, though.

There are 9 more current committers that are missing from that list. Shall I add those too? I lack their organizations, though.

Yes, you can.

There are 9 more current committers that are missing from that list. Shall I add those too? I lack their organizations, though.

Yes, you can. I can tell you their organization.

Great, you can suggest them in the changes tab.

Shall we sort the names alphabetically?

Shall we sort the names alphabetically?

It's ok.

Jerry Shao's organization should be Datastrato instead of DataStrato. He Qi's organization should be Datastrato.


Switch to use watch resource from SDK

Fixes

Custom function for watching resources are no longer needed
Description

Import usek8sWatch* from @openshift/dynamic-plugin-sdk-utils which uses websockets for communication (polling is no longer needed).

Type of change

[x] Refactoring (no functional changes, no api changes)

@rohitkrai03 looks like the problem was on API side of things and they were not sending correct messages. I tried that now by using the UI in two tabs. Added application in one tab and it automatically refreshed the table. Can you give it another try please?

@rohitkrai03 looks like the problem was on API side of things and they were not sending correct messages. I tried that now by using the UI in two tabs. Added application in one tab and it automatically refreshed the table. Can you give it another try please?

@karelhala Tried it again and it seems the data is getting updated automatically now but sometimes the npm run start:prod:beta fails automatically with the below error -

This time it failed when I deleted an application resource.

@rohitkrai03 are you using Mac? Looks like Macs have a problem when accessing WS, it automatically remebers to accept self-signed certs and that throws this error. I can't experience this on Linux (could be older version of chrome as well), but this command in chrome fixes it https://stackoverflow.com/a/31900210

@karelhala With the chrome flag enabled its working fine now. I've approved the PR, will add lgtm once you fix the merge conflict.

/lgtm


"module load python" fails on accessdev

resolution_fixed   |    by<EMAIL_ADDRESS>
$ module load python
conda(3):ERROR:105: Unable to locate a modulefile for 'conda/analysis2'
conda/analysis2 module does not exist. Presumably this should be conda/analysis27

Issue migrated from trac:334 at 2024-01-31 18:31:09 +1100

<EMAIL_ADDRESS>changed status from new to closed

<EMAIL_ADDRESS>set resolution to fixed

<EMAIL_ADDRESS>commented

Fixed
Scott


fix/security_issue_yaml_pkg

The testify package was the only one using the yaml package.
Thus, I've upgraded it with
go get -u github.com/stretchr/testify

and cleaned it up with
go mod tidy

This should resolve our security warning regarding the yaml package.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 87.319%

Files with Coverage Reduction
New Missed Lines
%

ws/websocket.go
2
90.14%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
6273

Relevant Lines:
7184

💛  - Coveralls


fix nix flake

Switch to fenix in order to meet the requirements of Rust 1.70+ for typst.
Built on x86_64-linux.
See: https://github.com/typst/typst/pull/1450

Thank you!

@Wybxc I'm getting the following error when attempting to upgrade to this commit or any afterwards using my flake:
error: attribute 'workspace' missing

       at /nix/store/9g1j75arlf9gzn2nc4rcdb6drjnpajpl-source/flake.nix:52:18:

           51|         pname = "typst-lsp";
           52|         inherit ((importTOML ./Cargo.toml).workspace.package) version;
             |                  ^
           53|

I'm not overriding any of the inputs in my flake, so this is quite surprising. I'm running NixOS.
Do you have any ideas what could be going on, if this is building for you?

@Wybxc I'm getting the following error when attempting to upgrade to this commit or any afterwards using my flake:
error: attribute 'workspace' missing

       at /nix/store/9g1j75arlf9gzn2nc4rcdb6drjnpajpl-source/flake.nix:52:18:

           51|         pname = "typst-lsp";
           52|         inherit ((importTOML ./Cargo.toml).workspace.package) version;
             |                  ^
           53|

I'm not overriding any of the inputs in my flake, so this is quite surprising. I'm running NixOS.
Do you have any ideas what could be going on, if this is building for you?

It was my mistake to copy the configuration from the typst repository without verifying it. I will make amends by submitting a new PR to address this issue.


shortex Repository

Hi Dr. Trapp,
I noticed that you created this repository recently. You might also be interested in some of the commands at https://github.com/trevorcampbell/shortex!
Best,
Nikola

Nice, thanks! I wasn't aware Trevor did the same. I am actually working with him atm. Small world. I will have a look, thanks!


Add CHANGELOG entry for dependabot PRs

Currently our checks complain about the dependabot PRs since they dont add a CHANGELOG entry.
Maybe someone can look into https://github.com/dangoslen/dependabot-changelog-helper
and see if it can be used to turn the PR-checks green.

can take a look but do we really need this?
I'm fine with the one failing test and we can merge these PR's if all other tests are green.
I don't need every version update in the changelog, this will only blew up this file with unneeded things. For me it's o.k. to mention only significant updates as e.g. electron and have an entry dependencies updated for the normal updates (what we are doing already).
And I think we don't need this for the whole dev dependencies.

as an alternative approach we could add the label Skip Changelog to the dependabot PR's to avoid the failed tests.

as an alternative approach we could add the label Skip Changelog to the dependabot PR's to avoid the failed tests.

I am totally fine with this approach

still not working, the Skip Changelog label is missing so check fails.

maybe the PR needs to find its way to master first?

I you want to, you can cherrypick it. I myself wouldnt mind  aiting for the next release branch.


Add Operating Systems Course?

Here are a couple of free courses that seems solid:

Coursera: https://www.udacity.com/course/introduction-to-operating-systems--ud923
saylor.org: https://learn.saylor.org/course/CS401

Okay but ftu forum is mostly for udemy courses.  you need some operating
system You can check Unix or Linux courses there other many more courses
you can also check some courses on freecoursesite. If u want some
university courses then goto classcentral u will find many coursera courses.
Respect from,
Omar.
On Fri, Jan 10, 2020, 11:21 AM Namito Yokota<EMAIL_ADDRESS>wrote:

Closed #35 https://github.com/ForrestKnight/open-source-cs/issues/35.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ForrestKnight/open-source-cs/issues/35?email_source=notifications&email_token=AM2KUCX7N3JATDCN7T22P5LQ5AAWFA5CNFSM4J7G4NMKYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOV4KBILY#event-2937328687,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AM2KUCWV4Q2SGRO76L62UCDQ5AAWFANCNFSM4J7G4NMA
.


Assertion: key should be down before it is removed from keystate

When I pressed the prtScn button on MAL while I had the slider focused. Not sure what that is about. Perhaps it is a button that isn't recognized by out current keystate tracker?

Encountered this again today, discussing wiht @zepumph, we would like to try to make the KeyStateTracker  have the most accurate representation of the keyboard by adding or deleting the modifier keys from the state if they are present in keydown and keyup, so in both keydownUpdate and keyupUpdate, we do something like
      if ( domEvent.shiftKey && !this.shiftKeyDown ) {
        this.keyState[ KeyboardUtil.KEY_SHIFT ] = {
          keyDown: true,
          keyCode: domEvent.keyCode,
          timeDown: 0 // in ms
        };
      }
      if ( domEvent.altKey && !this.altKeyDown ) {
        this.keyState[ KeyboardUtil.KEY_ALT ] = {
          keyDown: true,
          keyCode: domEvent.keyCode,
          timeDown: 0 // in ms
        };
      }
      if ( domEvent.ctrlKey && !this.ctrlKeyDown ) {
        this.keyState[ KeyboardUtil.KEY_CTRL ] = {
          keyDown: true,
          keyCode: domEvent.keyCode,
          timeDown: 0 // in ms
        };
      }
      if ( !domEvent.shiftKey && this.shiftKeyDown ) {
        delete this.keyState[ KeyboardUtil.KEY_SHIFT ];
      }
      if ( !domEvent.altKey && this.altKeyDown ) {
        delete this.keyState[ KeyboardUtil.KEY_ALT ];
      }
      if ( !domEvent.ctrlKey && this.ctrlKeyDown ) {
        delete this.keyState[ KeyboardUtil.KEY_CTRL ];
      }

This came up again for @phet-steele in the debug version of ohms-law from https://github.com/phetsims/QA/issues/289.  The case we encountered was caused by the number pad with Num Lock enabled. When Num Lock is enabled, you can hit this assertion by pressing and holding shift, then pressing and holding a number on the keyboard, then releasing shift, then releasing the number ~0.5s later.
I have also encountered this when pressing the "Windows" key, then pressing an arrow key. My guess is that in all of these combinations, we are running into cases like "PrtScn", where the browser doesn't receive all events due to special modifier keys.

@phet-steele said that the assertion messages he was getting were actually on keydown update

shift key inconsistency between event and keystate.
ctrl key inconsistency between event and keystate.

But it seems likely to be related to this issue because of

where the browser doesn't receive all events due to special modifier keys.

I tried https://github.com/phetsims/scenery/issues/918#issuecomment-460417397 but I am still seeing

Assertion failed: key should be down before it is removed from keystate

When I press shit + 5 the release in order with num lock on, or when using arrow keys while pressing my windows key to move apps around the screen.

@zepumph in https://github.com/phetsims/scenery/issues/918#issuecomment-452134152 you suggested a more flexible KeyStateTracker, and I think you are totally right now. There are too many cases where this is happening to guarantee that we have them all, even just on Windows!
What do you think about just removing this assertion?

      // first tab (or shift tab) into the document will not register a keydown event
      // many browsers don't have keydown events for print screen, see https://github.com/phetsims/scenery/issues/918
      if ( assert && !_.includes( UP_WHITELIST, keyCode ) ) {
        assert( this.isKeyDown( keyCode ), 'key should be down before it is removed from keystate' );
      }

Pushed in the above commit, aqua is passing and the cases where these assertions were showing up are no longer happening with modifier keys, Prt Scn, and others (..mostly because the assertion was just removed but...still yay). But I still committed https://github.com/phetsims/scenery/issues/918#issuecomment-460417397.
@zepumph can you please review at your convenience?

My changes caused unit tests to fail that assumed that shift key should not be down if shift is comes through as a modifier key. But our decision in https://github.com/phetsims/scenery/issues/918#issuecomment-460417397 made it so that the KeyStateTracker would indicate that shift key is down in this case. So I just changed the tests. @zepumph as part of review can you make sure that this makes sense?

That is great thanks, closing


Not all runtime headers installed

It's not clear to me whether the problem is here or in nnvm. nnvm references a runtime header directly, however, it's not installed when INSTALL_DEV is enabled.
It seems either NNVM should not be using a private header otherwise it should be installed by tvm.

Thanks for pointing this out. It should be able to be solved by https://github.com/dmlc/nnvm/pull/163


Uninstall doesn't quit the app

Application version: 3.0.0
OS: Windows 10

Steps to Reproduce:

Install Station.
Run Station.
Uninstall Station via "Add or remove programs"
Station doesn't close itself or offer to close itself.

Expected behavior

The Station uninstaller asks to close the program and offers an option to terminate its process forcibly before uninstalling.

I'm closing the ticket to keep backlog clean.
But PR is welcomed


How to get downloadable file in my script so that I can send it to mail or any other sharing platform?

I need to attach pdf or image to the mail so how to do it?

I have the same need.


update action

chore: fix typo in readme (#186)
chore(ci): update github cache action

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 89.177%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
330

Relevant Lines:
366

💛  - Coveralls


⚠ ClamAV has degraded performance

In b8ce066, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 90 ms

Resolved: ClamAV performance has improved in 353e2a9.


Improved logic when displaying number of observations

Improved logic when displaying number of observations.

Just make sure you fix whatever has caused the build to fail.


Enchancement for put object part tests.

As described here https://github.com/minio/minio/issues/4400 , check for last modified stamp in putobject part test.

@hackintoshrao I used the following program to reproduce the putObjectPart issue.
package main

import (
	"log"
	"strings"
	"time"

	minio "github.com/minio/minio-go"
)

func main() {
	clnt, err := minio.NewCore("localhost:9000", "minio", "minio123", false)
	if err != nil {
		log.Fatalln(err)
	}

	bucket := "krisis"
	object := "multipart-obj"
	uploadID, err := clnt.NewMultipartUpload(bucket, object, nil)
	if err != nil {
		log.Fatalln(err)
	}
	part, err := clnt.PutObjectPart(bucket, object, uploadID, 1, 5, strings.NewReader("aaaaa"), nil, nil)
	if err != nil {
		log.Fatalln(err)
	}

	zeroTime := time.Time{}
	if part.LastModified == zeroTime {
		log.Fatalln("ObjectPart last modified time is not filled")
	}
}

closing this for now as we don't have enough cycles to do this now


document folder option for etcd backups

LGTM

@moelsayed is this available as of today or only in RKE v0.3.0/Rancher v2.3?

@deniseschannon it's available today with RKE 0.2.x and Rancher 2.2.x. Not sure how long its been there exactly, but at least since 2.2.7.


Same storage already exists

Hello,
Thanks for your work in building this tool. We've been developing and testing heavily based of cvat. Most recently we kept running into the error "Same storage already exists" with status code 400, when attaching a new cloud storage with AWS S3.
We've verified that there's no duplicate (as specified in the manifest.jsonl file) in our existing cloud storages, even across different user accounts. One thing we did differently is that instead of putting the images directly under the bucket root, we put it under a prefix as "s3://<bucket_name>/some_prefix/<image_name>.jpg", and modified the manifest.jsonl file to reflect that.
A few questions:

Does CVAT has an upper limit on the number of cloud storages we could attach?
(This is a hack) The error went away, when we tried to modify an existing cloud storage under a different prefix, and replace with the new manifest.jsonl. Is this expected?

Thanks in advance for your help.

@Marishka17 , could you please help to investigate?

@jamesdi1993, Thanks, I'll investigate this case. But maybe one clarifying question:
Are you sure that you don't have another bucket with the same provider/bucket/credentials set?

Hi @Marishka17 just wanted to follow up on this thread, could you share if there's any info / update on this issue?

@jamesdi1993, Hello, As I remember, the current solution was implemented so that users can create only one cloud storage with unique  provider/bucket/credentials set.  This approach was implemented due to unique bucket name requirements on the provider's side (e.g you cannot create a bucket with some name if such bucket already exists). But if you have several subsets on the bucket you can create several manifests and attach them when creating cloud storage in CVAT.
If it was available in your case to create several cloud storages in CVAT with unique provider/bucket/credentials set, please, provide more details.

@Marishka17 Thanks for the reply. I followed your suggestion of using attaching single storage, and import the subsets of data by attaching multiple manifests under the same bucket. I think this would work for us in the short / medium term.
I'm able to attach multiple storages on the same provider & bucket & credentials, as provided by the example above. The confusing part is that it works for 2-3 storages and gave error messages then after trying attach more storages. Perhaps it's worth either fix the logic to enable it as a feature, or disabling it altogether.

I am using CVAT v2.1.0 and we have the same bucket organization of @jamesdi1993, i.e. one bucket and several subfolders:

bucket_name/image_folder_1
bucket_name/image_folder_2
...
I've successfully created a and I would like to create a cloud storage inside an organization  We created a manifest.jsonl for image_folder_1


ENTESB-13183 - use Teiid language server for ddl files

requires org.teiid:teiid-language-server to be available on Maven repository, see https://issues.redhat.com/browse/ENTESB-13172

I guess you are trying wrap the JAR into a node process here?

The jar is launched in its own process. It communicates through standard I/O with VS Code Language client.
The VS Code Language client is handling the lifecycle of the Language Server. We are only providing how to launch it: https://github.com/jboss-fuse/vscode-datavirt/pull/94/files#diff-6ed723866f12e4828a41c1157411db1dR31-R32 and for which language it needs to be used https://github.com/jboss-fuse/vscode-datavirt/pull/94/files#diff-6ed723866f12e4828a41c1157411db1dR38


Add a button to pre-generate the character skills

All characters have a baseline of skills.
Often, people's character sheets drift from this, and that can break
things like defense calculations.
This also helps DM's create new characters and monsters.
Also added a confirmation popover

I wonder if the simple fix is to only show the button when there are no skills.

That sounds like a great solution.
On Wed, Feb 21, 2024 at 8:29 PM Jaz @.***> wrote:

I wonder if the simple fix is to only show the button when there are no
skills.
—
Reply to this email directly, view it on GitHub
https://github.com/epic-power-rpg/roll20/pull/100#issuecomment-1958679439,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACTV55T7JD2X3A2VWDM4SPLYU3CQZAVCNFSM6AAAAABDOUWTESVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJYGY3TSNBTHE
.
You are receiving this because your review was requested.Message ID:
@.***>

And it would be easily extendable to generate different kinds of base
characters. It would be fine to take lots of screen real estate, since it
would only show up when making a new character.
On Wed, Feb 21, 2024 at 8:50 PM John Lamping @.***>
wrote:

That sounds like a great solution.
On Wed, Feb 21, 2024 at 8:29 PM Jaz @.***> wrote:

I wonder if the simple fix is to only show the button when there are no
skills.
—
Reply to this email directly, view it on GitHub
https://github.com/epic-power-rpg/roll20/pull/100#issuecomment-1958679439,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACTV55T7JD2X3A2VWDM4SPLYU3CQZAVCNFSM6AAAAABDOUWTESVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJYGY3TSNBTHE
.
You are receiving this because your review was requested.Message ID:
@.***>

I couldn't figure out how to hide the button if there are already skills in the repeating list.
I did get the confirmation popover in place.


Add minLeadingWidth to SwitchListTile

Add the minLeadingWidth field like the ListTile have to SwitchListTile too pls.

Closing this as duplicate of https://github.com/flutter/flutter/issues/83898


include applicant's additional information in the details page

on the detail page (which is available in the lots-admin section);

We should also display additional information (any improvements that they made to the yard, other properties that they own, and many other fields) that the applicant included in their application.
Right now, the additional information is only available by accessing the CSV dump.

this is what the detail page shows right now:


feat: added explain new tab toast. (#135)

Reviewers
Functional:
@tihuan

Changes

Added hook to check for explainNewTab query string param and pop work-in-progress toast if present.

Hurray! Thanks so much for the PR, @MillenniumFalconMechanic 🎉 !
Can you help check with @brianraymor to see if this needs to go into the hotfix branch please?
if so, we probably just need to update this branch to be based on the hotfix branch and merge against hotfix, in case it pulls in commits from main we don't want 🙏
Thank you!!


fix: popover arrow looking cut

Now Popover arrow looks right even when using align start or end
Fix #83

this was automatically fixed when update radix to v1 on PR #105


⚠ Turnos 24x7 has degraded performance

In 91b29be, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 814 ms

Resolved: Turnos 24x7 performance has improved in 78faf7f.


libcouchbase_make_doc_request coredumps

I just pulled the lastest from the source(Feb. 24, 2012)
It looks like the lastest code make the libcouchbase_make_doc_request segv.
This was OK for my previous pull about two week ago.
#0  0x00007ffff7635a75 in raise () from /lib/libc.so.6
#1  0x00007ffff76395c0 in abort () from /lib/libc.so.6
#2  0x00007ffff766f4fb in ?? () from /lib/libc.so.6
#3  0x00007ffff76795b6 in ?? () from /lib/libc.so.6
#4  0x00007ffff7680312 in realloc () from /lib/libc.so.6
#5  0x00007ffff7995821 in min_heap_reserve (ev=0x605798, tv=0x605880, tv_is_absolute=) at minheap-internal.h:125
#6  event_add_internal (ev=0x605798, tv=0x605880, tv_is_absolute=) at event.c:2013
#7  0x00007ffff799591a in event_add (ev=0x605798, tv=0x605880) at event.c:1926
#8  0x00007ffff799fea3 in be_socket_enable (bufev=0x605700, event=) at bufferevent_sock.c:552
#9  0x00007ffff799edce in bufferevent_enable (bufev=0x605700, event=4) at bufferevent.c:422
#10 0x00007ffff79aef38 in evhttp_connection_connect (evcon=0x608380) at http.c:2196
#11 0x00007ffff79b021a in evhttp_make_request (evcon=0x608380, req=0x605a70, type=, uri=) at http.c:2251
#12 0x00007ffff7bd86a2 in libcouchbase_make_doc_request () from /home/varmour/varmour/common/obj/tlib/lib/libcouchbase.so.1
#13 0x0000000000400d26 in main (argc=1, argv=0x7fffffffe668) at couchview-mine.c:126

I use a c program based on the libevent2 only, verified that the memory issues was caused by the libcouchbase. It has no problem to talked to the couchbase server correctly either chunked or not. Another suggestion for the api is that normally, for the libevent2, the error callback need to stop the event loop. But in the libcouch base, this is not possible because the io definition is hidden as internal. It it better for the libcouchbase to stay same as libevent2, but hide all the smart vbucket logic internally

Hi @czhan this stuff is in progress now. But thanks for report, I'll update the ticket when it will be finished


About Batch_Size.

According to your supplementary file, the model is trained for 150,000 iterations with a batch size of 24 on four 32G V100 GPUs.
Do you mean the IMG_PER_BATCH is 24 or IMG_PER_BATCH is 6, and the total BS is 24?
Thanks.

Hi,
Please note that we have further fine-tuned some hyperparameters for better performance. The current checkpoint was trained using the default configuration files mentioned in the README. Thank you for your reminder, and we have added this information to the documentation.
For customized training, the value of SOLVER.IMS_PER_BATCH should be set as the total batch size across all GPUs (e.g., 48 for 6 samples * 8 GPUs).


Improve the error msg when missing args

🔖 Changes

Improve the error message when missing arguments

🖼 Screenshots
BEFORE

AFTER

I would rather create a --help option, and trigger that option in case no arguments are found.


make projected solar zenith a convenience function

Is your feature request related to a problem? Please describe.

A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

As mentioned in https://github.com/pvlib/pvlib-python/pull/1725#discussion_r1190000002 the projected solar zenith (PSZ) is used more than once, and may benefit from having a standalone function for reuse to remove redundant code.
Describe the solution you'd like

A clear and concise description of what you want to happen.

a new function perhaps in tools or tracking like this:
sp = pvl.solarposition.get_solarposition(times, latitude, longitude, ...)
proj_zenith = np.degrees(np.arctan2(
    np.sin(np.radians(sp.azimuth))*np.sin(np.radians(sp.apparent_zenith)),
    np.cos(np.radians(sp.apparent_zenith))))

Describe alternatives you've considered

A clear and concise description of any alternative solutions or features you've considered.

For linear shade calculation in #1725 the user is required to calculate PSZ externally.
Additional context

Add any other context or screenshots about the feature request here.

one version of the PSZ calculation is currently in tracking.py based on the solar vector as seen in the orientation of the tracker axes, that's why axis tilt appears in this "transformed" solar vector:
xp = x*cos_axis_azimuth - y*sin_axis_azimuth
zp = (x*sin_axis_tilt*sin_axis_azimuth
          + y*sin_axis_tilt*cos_axis_azimuth
          + z*cos_axis_tilt)

https://github.com/pvlib/pvlib-python/blob/275e6718caf7486cb5b7dcf29acd59499ad51f7f/pvlib/tracking.py#L432

I'm definitely in favor of having a function to calculate this quantity.  It's a step towards https://github.com/pvlib/pvlib-python/discussions/1581#discussioncomment-4060499.

This angle is also referred to as the profile angle sometimes, correct?

This angle is also referred to as the profile angle sometimes, correct?

I'd like to know that too, in order to contribute to this issue and move forward #1725

IDK the correct name for this. If anyone has a stronger opinion please weigh in, otherwise @echedey-ls  just go with “profile angle” as @adriesse suggests. As long as it’s well defined, hopefully that should be enough. Do not let perfect be the enemy of good.

I don't get how axis_tilt and axis_azimuth should be implemented here. @kandersolar can you point me to any equation that takes that into account?

Where should it live? pvlib.tracking and pvlib.shading seem appropriate to me. If this function is intended to be user-facing (which I think it should be), I'd be -1 to putting it in pvlib.tools.

I'm also -1 to a user facing function in pvlib.tools. pvlib.irradiance is a bit of a junk drawer, but is it more appropriate than the relatively specific pvlib.tracking and pvlib.shading? At the risk of scope creep and unwise API deprecations, geometry.py came to mind for this and the AOI functions in irradiance.

I'm in favour of a geometry.py or something along angles.py or angular.py for this function, aoi and aoi_projection. I didn't expect the latter two to be in irradiance.py

For the curious:
https://www.google.com/search?q="profile+angle"+sun

For me, aoi.py > geometry.py. Into shading.py seems like a last resort. My thought is that a user might guess a function like this would be in aoi.py and wouldn't know what to expect of geometry.py.

In case there aren't enough options yet, maybe spatial.py?  I would personally go the path of least resistance and put it in irradiance.py until the next big reorg.


Incorrect output height and width on retina

Currently provided example has incorrect output dimensions on retina display:

I was able to fix it by replacing window.size(); with window.draw_size(); on line 33, but I didn't test it on other platforms, so I'm not sure it's a correct solution.
Btw, maybe the example should be in a separate repository? I had to create my own project and copy contents of the example file to run it. Or maybe there's a simple way to run it, that I missed?
Anyway, thank you for a great library!

Similar issue: https://github.com/PistonDevelopers/piston_window/issues/22

Closed by #14


0.12 migration guide doesn't mention the rename of file_watcher

How can Bevy's documentation be improved?
https://bevyengine.org/learn/migration-guides/0.11-0.12/
Asset Hot Reloading
Feature filesystem_watcher has been renamed to file_watcher.  In addition, you no longer need to manually configure the ChangeWatcher in the AssetPlugin as it is now configured automatically when the feature is enabled.

Could you reopen this on the bevy-website repo? :) Great issue, and I want to make sure it doesn't get lost.

Could you reopen this on the bevy-website repo? :) Great issue, and I want to make sure it doesn't get lost.

Ahh, just saw this, looks like you already took care of it though.  Thanks! <3


Add Page Titles to Video Routes Using Helmet

Summary
Using react-helmet, add page titles and descriptions to each new route created in #10 and the mock API responses added in #11 .
Variables in Titles and Descriptions
Some of the page titles and descriptions will require you to pull variables from the mock api responses. I've provided the related Bible.com URLs for comparison sake. But, instead of hard-coding exactly what you see on Bible.com, you'll need to use variables in your titles and descriptions. For example, ${video.title} - From ${publisher.name}.

[ ] Install react-helmet using npm, making sure it gets added as a dependency in package.json
[ ] For path /videos, title and description should match: https://www.bible.com/videos
[ ] For path /videos/:id/series, title and description should match: https://www.bible.com/videos/https://www.bible.com/videos/266/series
[ ] For path /videos/:slug, title and description should match: https://www.bible.com/videos/269-why-have-you-forsaken-me-from-lightworkers-media

Yes. Precisely! Thanks Gale.

Is this what you were thinking of? https://github.com/WagicalGale/youversion-web-open-ideas/commit/81a51809363673fcd459fa734b52e429649f2f49


Re-enable nightly Key Vault tests

Is this going to blow up our test costs again because of HSM?  If so, can we run this less often?  Say weekly maybe?

To be clear, these were disabled because of service performance issues. If cost is a concern, we should probably open a new issue to conditionalize our test matrixes. Hypothetically, with the changes @benbp made for matrix generation we probably could run them on weeklies.
On a related note, .NET's "weeklies" are still running both Saturday and Sunday. Shouldn't that just run once? /cc @weshaggard

This is done, though we are experiencing a new problem with certificate LROs. Discussing offline with the service team.


feat: improve changelog defaults

I've instead started to split this PR up into smaller incremental changes as it makes testing & releasing simpler.
This branch hasn't been touched in a while and unfortunately does not have the latest code either if you are here looking to copy it.


nixos/ncdns: listen on IPv6 by default

Description of changes
One small step towards deprecating IPv4
Things done

[x] Tested nixosTests.ncdns
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[x] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg test ncdns


needle === haystack ?

I don't get this part:
if (nlen === hlen) {
    return needle === haystack;
  }

this means:
"apple" === "appel"  false
Shouldn't this example return true?
Why is nlen === hlen important ?

it's an optimization shortcut

I think I have to admit this was the best explanation of confusing part of the code I have ever seen in my entire life!

I've decided to rewrite it as I consider code of this lib hard to understand and also I needed it to return informations about which parts were fuzzy and which were input
Maybe it'll be useful for someone (it's in typescript)
type MatchRoleType = 'input' | 'fuzzy' | 'suggestion';

interface FuzzyMatchData {
  content: string;
  type: MatchRoleType;
}

export interface FuzzyMatchOptions {
  truncateTooLongInput?: boolean;
}

export function fuzzyMatch(
  fuzzyInput: string,
  fullString: string,
  { truncateTooLongInput }: FuzzyMatchOptions = {},
): FuzzyMatchData[] | false {
  // input is longer than fullString and truncating is disabled
  if (fuzzyInput.length > fullString.length && !truncateTooLongInput) {
    return false;
  }

  // truncate if fuzzyinput is longer than fullstring
  if (fuzzyInput.length > fullString.length && truncateTooLongInput) {
    fuzzyInput = fuzzyInput.substr(0, fullString.length);
  }

  // both fuzzy and full string are equal - they match without being fuzzy
  if (fuzzyInput === fullString) {
    return [{ content: fuzzyInput, type: 'input' }];
  }

  const fuzzyMatchData: FuzzyMatchData[] = [];

  const leftFuzzyLetters = fuzzyInput.split('');

  let fuzzyLettersBuffer: string[] = [];
  let matchingLettersBuffer: string[] = [];

  function clearFuzzyBuffer() {
    if (fuzzyLettersBuffer.length > 0) {
      fuzzyMatchData.push({
        content: fuzzyLettersBuffer.join(''),
        type: 'fuzzy',
      });
      fuzzyLettersBuffer = [];
    }
  }

  function clearMatchingBuffer() {
    if (matchingLettersBuffer.length > 0) {
      fuzzyMatchData.push({
        content: matchingLettersBuffer.join(''),
        type: 'input',
      });
      matchingLettersBuffer = [];
    }
  }

  for (let fullStringLetter of fullString) {
    const currentFuzzyLetter = leftFuzzyLetters[0];

    console.log({ currentFuzzyLetter, fullStringLetter });

    // no more input
    if (!currentFuzzyLetter) {
      // clear buffers
      console.log('no more fuzzy letters');
      break;
    }

    // another fuzzy letter to add
    if (fullStringLetter !== currentFuzzyLetter) {
      // make sure to clean matching letters buffer if we have some
      clearMatchingBuffer();
      fuzzyLettersBuffer.push(fullStringLetter);

      continue;
    }

    // match!
    leftFuzzyLetters.shift();
    // clear fuzzy letters buffer
    clearFuzzyBuffer();

    matchingLettersBuffer.push(fullStringLetter);

    if (!leftFuzzyLetters.length) {
      clearMatchingBuffer();
    }
  }

  if (leftFuzzyLetters.length > 0) {
    return false;
  }

  const matchedPart = fuzzyMatchData.map((match) => match.content).join('');

  const suggestionPart = fullString.replace(matchedPart, '');

  if (suggestionPart) {
    fuzzyMatchData.push({ content: suggestionPart, type: 'suggestion' });
  }

  return fuzzyMatchData;
}


Risks and Rabbit Holes Pitch Section

Task

Delve into the risks and rabbit holes associated with each ideated concept and specific mitigation strategies

Deliverables

Risks and Rabbit Holes section of pitch document

Detailed risks and rabbit holes while focusing on strategies to avoid and mitigate them.


Unable to deserialize recursively

Hi there! I believe this could be similar to this open issue: https://github.com/aetherknight/recursive-open-struct/issues/69 but I figure I'd make one for our use case.
It seems like we are unable to
irb(main):001:0> require 'recursive-open-struct'
=> true
irb(main):002:0> Marshal.load(Marshal.dump(RecursiveOpenStruct.new(red: [RecursiveOpenStruct.new]))).red
/Users/maple.ong/.rbenv/versions/3.1.2/lib/ruby/gems/3.1.0/gems/recursive-open-struct-1.1.3/lib/recursive_open_struct.rb:80:in `[]': undefined method `[]' for nil:NilClass (NoMethodError)

    elsif v.is_a?(Array) and @options[:recurse_over_arrays]
                                     ^^^^^^^^^^^^^^^^^^^^^^
	from /Users/maple.ong/.rbenv/versions/3.1.2/lib/ruby/gems/3.1.0/gems/recursive-open-struct-1.1.3/lib/recursive_open_struct.rb:142:in `block (2 levels) in new_ostruct_member'
	from (irb):2:in `<main>'
	from /Users/maple.ong/.rbenv/versions/3.1.2/lib/ruby/gems/3.1.0/gems/irb-1.4.3/exe/irb:11:in `<top (required)>'
	from /Users/maple.ong/.rbenv/versions/3.1.2/bin/irb:25:in `load'
	from /Users/maple.ong/.rbenv/versions/3.1.2/bin/irb:25:in `<main>'

We are trying to upgrade our application to Ruby 3 but we are relying on this behaviour to work. Since it is broken on the latest version we cannot upgrade.

We are seeing the same problem while attempting to upgrade one of our services to Ruby 3 as well. (Thanks for posting this - great timing!)

It looks like marshal_load is passed the options like recurse_over_arrays but they are just passed in at the top level and it will fallback to using OpenStruct's marshal_load implementation which will assign them in as keys in the struct rather than interpreting them as options.
Something like this may help:
class RecursiveOpenStruct
  # …

  def marshal_load(attributes)
    @options ||= {}
    @sub_elements ||= {}

    self.class.default_options.keys.each do |option|
      @options[option] = attributes.delete(option) if attributes.key?(option)
    end

    @deep_dup = DeepDup.new(@options)

    super
  end
end

Note: I have never written one of these marshal_load methods before so I'm not confident that this follows all the best practices. I'm not sure if it's common practice to re-do a lot of the same work that initialize typically does. This would also mean it wouldn't be possible to create a RecursiveOpenStruct that has any of the three option names as a key in the struct. Perhaps if we defined our own marshal_dump method then we could support all key names properly.


Possibility to add a custom HTTP header ?

Hi,
I wanted to know if it's possible to add a custom HTTP header for response-header -- I mean, to the server (I'm using Ajax to get data from C++ API).
My only solution here, is to make Silicon running behind a reverse proxy (nginx, where I can put the custom header).
Thank you.

Yes this is possible with the mhd backend:
[] (mhd_response* r)
{
req->set_header("header_key", "value");
});

Can I have an example more specific please ?
i.e:
GET / _login * get_parameters(_id = int())  = [] (auto param) {
    return param.id;
}

Where do I place the mhd_response ?
Thank you.

Just add it to the parameter:  [] (auto param, mhd_response* r) {

Thank you, it's actually working great 👍


viyacaddy02: reports import not ignoring files with wrong extensions

tables import ignores files not supported, but reports imports does not. It should restrict the types to json and xml - the server may still reject these due to bad content but there is no way to fix that.

reports import now ignores files that do not have the proper extensions(json or xml)

Noticed an extra in masSetup - this was removed.

a test version of restaflib.masAddModels was added.


Issue 123

Issue: 123
Resolved merge conflicts
added controller to update collaborators

i removed unwanted checks you can review it now


WIP: Changing help texts

This PR build further upon #48, hence the WIP and I'll fix this once that PR has been accepted or denied.
When calling rbw-rofi with an --action argument, the help message and alt-codes in rofi (--mesg) is unchanged. I think it would be neat to make these adaptive, such that the default action is not an alt-code.
In the case that the help message is not rendered, having it adaptive would be confusing (imo) and thus the list is then a fixed order: both, username, password, menu.
PS: this was my main motivation for PR #48.

When I was writing the description of this PR I realized that not having the default action as an alt-code might be undesired by some users, hence I just added a commit that adds the default action at the end of the action list, but does not render this last action in the help message.

Hi @tosti007, I'd like to prepare a new release. If you think this this PR is still useful, can you either rebase it or drop me a note so that I can have look? Thank you 🙂

Hey @fdw, I think it's best to release it without this PR. I do think it's usefull, but I am not sure if I have time to fix it before the release.

Hey @tosti007, how's it looking? 😉

Haha sorry for the large delay, I saw the large update to 1.0.0 and found I didn't have the time at that moment to rebase. I have just fixed it.
If you think this shouldn't be added, I'll probably extract some small changes from this PR to a new PR (such as the show_help arg), so don't worry about those.

Because we now support custom keybindings with correct, dynamic help texts, this PR isn't necessary anymore 🙂


GN-4583: Remove node properties on backspace

Overview
Performs correct action of removing node properties/backlink on backspace press when selection is present.
Also remove mentions of "alternative backspace", as it won't be "alternative" anymore 🤔
connected issues and PRs:
GN-4583
Setup

Checkout

How to test/reproduce

Use RdfaTest
Select an RDFA node (e.g. Cmd + Click it)
Press backspace
Observe correct properties/backlinks were deleted
Observe selection was deleted/modified correctly
Select multiple RDFA nodes with text in between
Press backspace
Observe correct properties/backlinks were deleted
Observe selection was deleted/modified correctly

Challenges/uncertainties

As we have discussed - this is not the best way to remove properties/backlinks on RDFA nodes being deleted, as this does not react on delete, selecting and typing in. To fix the above I am currently working on using appendTransaction with modification of changedDescendants
I have tried to implement a visible selector for the user, so he can press it directly to select the node, but I've decided against it, as it was not working properly. It was depending on handleDOMEvents and using posAtDOM detect what node to select, but in the end it was not the best implementation, as posAtDOM was not alway selecting what I hoped it would, even with the bias param adjustments. Better way here would be to implement an EmberView for RDFA nodes, then selecting a node will be a breeze. Users still can use "Cmd + Click".

isolating: true,
selectable: true,

is a free way to select node on backspace, cannot not like it :)

Checks PR readiness

[X] UI: works on smaller screen sizes
[X] UI: feedback for any loading/error states
[X] Check cancel/go-back flows
[X] Check database state correct when deleting/updating (especially regarding relationships)
[X] changelog
[X] npm lint - target branch is failing
[X] no new deprecations

@dkozickis confirmed that my comment is unrelated to your work


Again allowed 0 in the Nr_SNPs .janno column

This should solve https://github.com/poseidon-framework/poseidon-schema/issues/85

Hmm, this is OK, but different from what I thought we discussed in https://github.com/poseidon-framework/poseidon-schema/issues/85. I thought we wanted to still issue a warning if the NrSNPs is 0. Now we only get an error if it's negative. That's quite different. Is that on purpose?

Puh - I didn't consider the warning important, honestly. The problem is that the packages with Nr_SNPs < 1 don't get read right now.
Warnings require separate code, because make only lives in MonadFail. I can add it :man_shrugging:.


Add Python 2.7 support

Adding support for Python 2 with minimum impact. This entails:

Adding an explicit namespace for opencivicdata (to continue combining this package and python-opencivicdata-django under one namespace).
Adding Python 2 unicode compatibility to csv reading (using backports.csv).
Enabling tests for Python 2.7.

I also added a requirements-test.txt because I was hitting a version conflict with py.test in Travis (example here). Not totally sure why this fix works, but it has the advantages of being consist with how python-opencivicdata-django is set up.
This change is required in order to add Python 2 support for python-opencivicdata-django (see opencivicdata/python-opencivicdata-django/76).

Coverage increased (+0.2%) to 85.714% when pulling d773cdaa6e1c47fa3530e23e8bdb946b49260008 on california-civic-data-coalition:master into c1f7458f08db4d8d449ad1ba7314ec98c61c2f57 on opencivicdata:master.

@jamesturk Got the conditional install/import of backports.csv working. I followed Hynek Schlawack's advice pretty much to the letter, which meant adding some crufty stuff to setup.py.
One thing to note, though: He claims that that the simple use of the environment markers in the install_requires arg to setup() will silently fail for setuptools 21 or greater, but I'm not seeing that. I'm running version 34 locally, and the use of the environment marker in requirements_test.txt worked. Also seems to be working on travis.
TBH, this is much deeper into the weeds of setuptools than I have ever ventured, so I'm willing to trust an expert and believe that what I've just pushed will work for the widest range of users.

the conditional stuff looks alright to me, thanks!
I suppose the namespace stuff is alright to be in both places, @fgregg did you have a better solution for dedupe or was this what you did?

@jamesturk this is pretty much what I did.


Custom certificate is not applied to the WebSocket Connection

I confirm (by marking "x" in the [ ] below: [x]):

[x] This is not a troubleshooting question. Troubleshooting questions go here: https://docs.mattermost.com/install/troubleshooting.html.
[x] This doesn't reproduce on web browsers (such as in Chrome). If it does, issue reports go to the Mattermost Server repository.
[x] I have read contributing guidelines.

Summary
Custom certificate is not applied to the WebSocket Connection
Environment

Operating System: Ubuntu 22.04
Mattermost Desktop App version: 5.2.0
Mattermost Server version: irrelevant

Steps to reproduce

add a custom certificate to /usr/local/share/ca-certificates/...
run update-ca-certificates
start Mattermost and set the server address to an HTTPS URL
accept the "untrusted certificate" in the popup

Expected behavior

Mattermost functioning as usual

Observed behavior

Mattermost starts and messages can be sent, but incoming messages are not shown right away
after a while, a red banner pops up with the message "server unreachable, check your WebSocket port"
the .config/Mattermost/certificate.json file contains the custom certificate, but only for the https://... address
the log contains the message [warn]  Ignoring certificate for unmatched origin wss://..., will not trust

Possible fixes
Manually copying the https:// section for wss:// as follows seems to fix the problem:
{
  "https://...": {
    "data": "-----BEGIN CERTIFICATE-----...-----END CERTIFICATE-----\n",
    "issuerName": "...",
    "dontTrust": false
  },
  "wss://...": {
    "data": "-----BEGIN CERTIFICATE-----...-----END CERTIFICATE-----\n",
    "issuerName": "...",
    "dontTrust": false
  }
}

Thanks for the report, created a ticket here: https://mattermost.atlassian.net/browse/MM-48407

Hi, sorry to jump in but in case this helps get it fixed I thought I should leave a comment.
I believe I have encountered this exact same issue. Older Mattermost clients would work as they added both the https://... and the wss://... entry to the certificate.json file when a user selects to trust an unknown certificate in the popup. Newer clients however appear to only add the https://... entry to certificate.json which results in the red banner appearing due to the WebSocket connection being blocked.
The workaround suggested above to manually add the wss://... entry to certificate.json has worked every time in my testing so far. Noting that the client needs to be properly closed and reloaded for the changes to take effect.
Hope this helps.

I tried the suggested work around but it didn't work for me.  It seems to re-write that file every time the client starts.  I'm using 5.2.2 on Linux.


SYSENG-1357: generic client default options

Description

This PR adds functionality to define default requests options via WithRequestOptions when creating a new generic API client.
Checklist

[ ] added release notes to Unreleased section in CHANGELOG.md, if user facing change

References

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

All relevant integration tests went through, I'm ok with merging without all of them being green


Add Blur option

+1

I made a pull request concerning this feature, don't hesitate to tell me if you see any bugs or potential bugs related to this feature.

@GawenBerger Thanks!