Default scope IViewBufferScope causing issues with threaded job.

I've got a job which has a number of Tasks which generate emails using the example RenderViewToString.
The issue is when running under with concurrent Tasks I get random errors as below.
Index was outside the bounds of the array.|System.IndexOutOfRangeException: Index was outside the bounds of the array.
   at System.Collections.Generic.List`1.Add(T item)
   at Microsoft.AspNetCore.Mvc.ViewFeatures.Internal.MemoryPoolViewBufferScope.GetPage(Int32 pageSize)
   at Microsoft.AspNetCore.Mvc.ViewFeatures.Internal.ViewBuffer.GetCurrentPage()
   at Microsoft.AspNetCore.Mvc.ViewFeatures.Internal.ViewBuffer.AppendHtml(String encoded)
   at Microsoft.AspNetCore.Mvc.Razor.RazorPage.WriteLiteralTo(TextWriter writer, String value)
   at Microsoft.AspNetCore.Mvc.Razor.RazorPage.WriteLiteralTo(TextWriter writer, Object value)

Index was out of range. Must be non-negative and less than the size of the collection.
Parameter name: index|System.ArgumentOutOfRangeException: Index was out of range. Must be non-negative and less than the size of the collection.
Parameter name: index
   at System.ThrowHelper.ThrowArgumentOutOfRangeException(ExceptionArgument argument, ExceptionResource resource)
   at System.Collections.Generic.List`1.RemoveAt(Int32 index)
   at Microsoft.AspNetCore.Mvc.ViewFeatures.Internal.MemoryPoolViewBufferScope.GetPage(Int32 pageSize)

Object reference not set to an instance of an object.|System.NullReferenceException: Object reference not set to an instance of an object.
   at Microsoft.AspNetCore.Mvc.ViewFeatures.Internal.ViewBufferPage.Append(ViewBufferValue value)
   at Microsoft.AspNetCore.Mvc.ViewFeatures.Internal.ViewBuffer.AppendHtml(String encoded)
   at Microsoft.AspNetCore.Mvc.Razor.RazorPage.WriteLiteralTo(TextWriter writer, String value)
   at Microsoft.AspNetCore.Mvc.Razor.RazorPage.WriteLiteralTo(TextWriter writer, Object value)

Looking at the issue, I found that making IViewBufferScope  Transient, it stops the errors.
Although this appears to fix the issue, is it the underlying cause or am I possibly missing something?
Is making it Transient the correct solution, should it be Transient by default or should I create new scopes per thread? Although the third option isn't easy as I'm using the default DI and there is this issue (https://github.com/aspnet/DependencyInjection/issues/246).
Note: I've tested this against 1.1.0-alpha1-21752

Could you elaborate on what you're trying to run concurrently?

Hi, corrected my initial description., I'm actually kicking off multiple tasks(not threads) and awaiting them with a Task.WhenAll().
Each task will gather the data required for external sources and then generate the email, rendering it as HTML. Only once the email has been generated and queued for sending is that task complete.

It would've be useful to see a code snippet of what you're doing. That said, the code sample in Entropy isn't designed to be run concurrently. You might be able to add it as a transient service and create a scope for each Task that gets executed. But I'm not sure if that necessarily works.
// ConfigureServices
services.AddTransient<RazorViewToStringRenderer>();

// Run
var serviceScopeFactory = serviceProvider.GetRequiredService<IServiceScopeFactory>();
emailsToProcess.Select(t => Task.Run(() =>
{
  using (var scope = serviceScopeFactory.CreateScope())
  {
    var renderer = provider.GetRequiredService<RazorViewToStringRenderer>();
    ....
  }
});

IViewBufferScope needs to be scoped, it's to used to coordinate use of memory across views and partials as part of the same 'request'. I know in your case there isn't a request, but the right approach is something like what @pranavkm is suggesting - you'll need to await all of the tasks.

Should we update the sample in Entropy? Or is this really just an exercise left to the reader?

Leave it as an exercise? Doing all the extra stuff probably detracts from the the primary purpose of the sample.

Thanks @pranavkm. I guess the perfect example would have been what you provided without the fix (As below for reference).
public Test(RazorViewToStringRenderer renderer)
{
	_renderer = renderer
}
...
public void DoWork()
{
	...
	emailsToProcess.Select(t => Task.Run(() =>
	{
	   _renderer.RenderViewToString("EmailTemplate",model);
	  ...
	});
	...
}

I've modified my code as recommended with Transient and scoping and it worked.
It wouldn't hurt to have a comment in the example at least. Also given the solution is to at least have it scoped correctly. The example has it as a singleton which could be misleading.

If you're up for it, feel free to send a PR to Entropy. You could fix the lifetime of the the service and maybe add a separate method that shows how it could be used to process more than one thing.

Closing this since it's not specifically and Mvc issue. Feel free to start a discussion in https://github.com/aspnet/Entropy/issues

Just a note that this same problem also caused me issues (and probably for the same reasons). I'm not using the Entropy library, though am doing something similar with a custom Razor host that relies on many of the existing services. I'm seeing intermittent exceptions from the MemoryPoolViewBufferScope class:
System.ArgumentOutOfRangeException: Index was out of range. Must be non-negative and less than the size of the collection.
Parameter name: index
   at System.ThrowHelper.ThrowArgumentOutOfRangeException(ExceptionArgument argument, ExceptionResource resource)
   at System.Collections.Generic.List`1.RemoveAt(Int32 index)
   at Microsoft.AspNetCore.Mvc.ViewFeatures.Internal.MemoryPoolViewBufferScope.GetPage(Int32 pageSize)
   at Microsoft.AspNetCore.Mvc.ViewFeatures.Internal.ViewBuffer.GetCurrentPage()
   at Microsoft.AspNetCore.Mvc.ViewFeatures.Internal.ViewBuffer.AppendHtml(String encoded)
   at Microsoft.AspNetCore.Mvc.ViewFeatures.Internal.ViewBufferTextWriter.Write(String value)
   at Microsoft.AspNetCore.Mvc.Razor.RazorPage.WriteLiteralTo(TextWriter writer, String value)
...

In my case, I am scoping the service provider, but the scope encompases many parallel Razor compilations. I could narrow the scope to wrap each individual view compilation, but I'd hate to do that because I'm relying on the scope being broader for other parts of my app.
Is this the only service that gets used during Razor compilation and/or rendering that isn't thread-safe? If so, it does strike me as more of an "MVC issue" since it prevents writing any Razor host that does parallel processing (not just the simple one in Entropy).

In my case, I am scoping the service provider, but the scope encompases many parallel Razor compilations.

The error you're seeing here shouldn't happen as part of compilation. It's a result of trying to render one or more views concurrently without the right kind of scoping. Do you still see this error if you create a scope per view execution?

would a PR to MemoryPoolViewBufferScope to make it thread-safe be considered?
There's probably other areas of the code in this area that aren't thread safe, so I'm not sure fixing the MemoryPoolViewBufferScope alone would suffice.

The error you're seeing here shouldn't happen as part of compilation

It's definitely happening during rendering, I just combine compilation and rendering for each view into a single task and then run those in parallel across all the views that need to be rendered.

Do you still see this error if you create a scope per view execution?

Nope - limiting the scope appears to resolve the issue. I'm not thrilled about that since I want a broader scope that covers this whole process for my other components.

There's probably other areas of the code in this area that aren't thread safe

This is the gotcha, I think. I was able to get past this particular problem with the code below, but I'm worried there might be other similar concurrency issues I just haven't hit yet. The alternative is to just render each of the views in sequence, but that'll take much longer for a very large set. I'll fall back to that if I have to though. So far, so good.
Here's how I ended up getting around this in case someone else stumbles on this issue in the future:
serviceCollection..AddScoped<IViewBufferScope, SafeMemoryPoolViewBufferScope>();
// ...
internal class SafeMemoryPoolViewBufferScope : IViewBufferScope, IDisposable
{
    private readonly object _lock = new object();
    private readonly MemoryPoolViewBufferScope _pool;

    public SafeMemoryPoolViewBufferScope(ArrayPool<ViewBufferValue> viewBufferPool, ArrayPool<char> charPool)
    {
        _pool = new MemoryPoolViewBufferScope(viewBufferPool, charPool);
    }

    public ViewBufferValue[] GetPage(int pageSize)
    {
        lock (_lock)
        {
            return _pool.GetPage(pageSize);
        }
    }

    public void ReturnSegment(ViewBufferValue[] segment)
    {
        lock (_lock)
        {
            _pool.ReturnSegment(segment);
        }
    }

    public PagedBufferedTextWriter CreateWriter(TextWriter writer) => _pool.CreateWriter(writer);

    public void Dispose() => _pool.Dispose();
}

The IViewBufferScope is tied to the unit of work - it's designed to manage all of the buffers that are needed by a single run of the page. Making it thread-safe breaks that contract.
Consider simplifying to something like: https://github.com/aspnet/Mvc/blob/a78f77afde003c4a3fcf5dd7b6dc13dd9c85f825/test/Microsoft.AspNetCore.Mvc.ViewFeatures.Test/Internal/TestViewBufferScope.cs
You don't need to mess with any of the custom stuff we're doing here, you just need new and let the GC handle it.
Despite all that, I want to be absolutely crystal clear that....
Anything in MVC's services that has a scoped lifetime or uses the ViewContext is not thread safe. The fact that what you're doing today works is not a guarantee that it will be supported in the future. A component that is thread safe but scoped today could become non-thread-safe in the future.
The only supported use of these services is within a 'scoped' container that has the proper lifetime. For views this means creating the scoped container before running the view, running it with a single thread, and then disposing the container.
====================
@pranavkm should we update the sample to use scopes properly?

Filed https://github.com/aspnet/Entropy/issues/167 to track the Entropy work.

Anything in MVC's services that has a scoped lifetime or uses the ViewContext is not thread safe

Good to know - thanks for the unambiguous clarification. I'll plan accordingly and remove anything that assumes thread safety.


Update .m3u with when synced command is used

Requested Feature
Update.m3u files when spotdl sync <temp.spotdl> command is used. Their is need to currently include the the playlist link in the sync command in order to get the current playlist without it the playlist.m3u is not included in the updated file. The new cleaner method of only syncing the .spotdl file is better but needs to also pull the .m3u file also
Possible implementation
have no clue

fixed on dev branch


init_comm: should we reply all commands in queue when got INIT_FINISHPOD messsage?

here: https://github.com/hyperhq/runv/blob/master/hypervisor/init_comm.go#L263
Image such a case:

Client initiate a Kill command to the last container, which will lead to the container exit and pod finish.

Before the second command was replied by hyperstart, the init_comm got INIT_FINISHPOD

Then the second kill  command will never got replied.

yes, we should

I think it was fixed in this case. And https://github.com/hyperhq/runv/pull/447 ensure all the commands will be replied in all cases.


Clangd fails to produce "documentation" on Class-only comment documentation

If the documentation for a class method is only defined inside the method's declaration in the header file and not in the cpp file, the "documentation" element is NOT produced. This bug was found when trying to use You Complete Me with vim and finding missing documentation in the preview window.
Here is a working example: https://gist.github.com/Desperationis/ee5f1dc1039968a7fd28499c17f1d83c
Broken example: https://gist.github.com/Desperationis/a421cde1e6f0e36128ef524678586b98
Log:
// Working
{
  "detail": "void",
  "documentation": {
    "kind": "plaintext",
    "value": "@brief This is a comment."
  },
  "filterText": "ah",
  "insertText": "ah",
  "insertTextFormat": 1,
  "kind": 2,
  "label": "ah()",
  "score": 1.33012056350708,
  "sortText": "4055be9cah",
  "textEdit": {
    "newText": "ah",
    "range": {
      "end": {
        "character": 7,
        "line": 9
      },
      "start": {
        "character": 7,
        "line": 9
      }
    }
  }
}

// Not working
{
  "detail": "void",
  "filterText": "ah",
  "insertText": "ah",
  "insertTextFormat": 1,
  "kind": 2,
  "label": "ah()",
  "score": 0.9184742569923401,
  "sortText": "4094dedfah",
  "textEdit": {
    "newText": "ah",
    "range": {
      "end": {
        "character": 10,
        "line": 6
      },
      "start": {
        "character": 10,
        "line": 6
      }
    }
  }
}

As you can see, the "documentation" element is not found whatsoever in the broken example, leading to editors not being able to sense class-only header documentation.
System information
Editor/LSP plugin: vim / YCM
Operating system: Ubuntu 20.04 Focal Fossa

#564 looks like a similar issue for fields, quite possibly they have the same underlying cause


why logits of model outputs is None

🐛 Describe the bug
logits are not calculated here:
https://github.com/linkedin/Liger-Kernel/blob/6ab3b9febc29f5045e6d2e27ba6bacaa4f041d91/src/liger_kernel/transformers/model/qwen2.py#L99-L110
so is it set to None for some reasons? or just a bug?
thanks?
Reproduce
No response
Versions
Environment Report:
-------------------
Operating System: Linux-3.10.0-1160.el7.x86_64-x86_64-with-glibc2.35
Python version: 3.10.12
PyTorch version: 2.4.0a0+f70bd71a48.nv24.06
CUDA version: 12.5
Triton version: 3.0.0
Transformers version: 4.45.0

https://github.com/linkedin/Liger-Kernel/blob/6ab3b9febc29f5045e6d2e27ba6bacaa4f041d91/src/liger_kernel/transformers/monkey_patch.py#L502-L510
logits are not materialized while training with fused_linear_cross_entropy to reduce peak memory usage
https://github.com/linkedin/Liger-Kernel/blob/6ab3b9febc29f5045e6d2e27ba6bacaa4f041d91/src/liger_kernel/ops/fused_linear_cross_entropy.py#L55-L61
they are computed chunk by chunk as intermediate values only in above function

thanks for your explain!


Setting a Badge on Local Cluster Fails Attempting to Downgrade Cluster?

Setup

Rancher version:
2.8.1
Rancher UI Extensions:
None
Browser type & version:
Firefox 121

Describe the bug
After running into https://github.com/rancher/dashboard/issues/9840#issuecomment-1915518215 I added badges for all the clusters, not just those that clashed. Trying to set a badge for the local cluster (which unlike the others is a manually installed RKE2 which is running Rancher itself) failed. Now if the failure had just been generic I wouldn't be filing the bug. The failure message appeared to say that setting the badge had failed to downgrade the cluster version! That would surely have been an epic failure. Luckily that downgrade was prevented by some catch mechanism, but surely that shouldn't happen at all when setting a badge right?
To Reproduce
Navigate to local cluster, attempt to set a badge for it.
Result
Failure as seen below
Expected Result
A badge set or even fail without attempting to alter the cluster beyond just the badge
Screenshots

Additional context

on a local cluster running rke2 v1.27.8+rke2r1, I was not able to reproduce this issue when setting the local cluster's badge which is running rancher v2.8.1:

I'm on firefox 121 as well. I don't know how that error would have shown up but it is interesting. Do you have any information about what you were doing before setting the badges? i.e. had you attempted an upgrade?

Earlier in the day I had been updating one of the other clusters, yes, from 1.26.something to 1.27.something, but not this cluster and nothing to do with 1.24

Cluster badge information is saved as annotations on the target cluster's /v3/cluster resource. It could be that the kube version within that resource is out of step with the actual cluster's resource. What's the kube version of the cluster, and what does the value in v3/clusters state (visible via the brower's dev tool network requests tab)?

apiVersion: management.cattle.io/v3
kind: Cluster
metadata:
  annotations:
    management.cattle.io/current-cluster-controllers-version: 1.27.9+rke2r1
    ...
  name: local
...

vs
$ rke2 --version
rke2 version v1.27.9+rke2r1 (378bd59c4f0f97094c23c350d668f37f33aba406)
go version go1.20.12 X:boringcrypto

looks the same to me

Would you be able to provide the response to the v3/cluster network request the browser makes? there's additional information rancher adds and it could be somewhere in that chain

More specifically:
spec:
  rke2Config:
    kubernetesVersion: v1.24.12+rke2r1

I wonder how that happened. I guess that could be the original version that was running when I installed rancher. Both the cluster's rke2 and rancher have been updated since then.

Interesting, the annotation is correct but spec is not. I think this one might need a separate issue over at the rancher/rancher repo

I agree @richard-cox , are you able to migrate the ticket or should I open a new one there and link this one?

I guess there might still be an issue here as well though

I see several similar reports there already:

https://github.com/rancher/rancher/issues/43331
https://github.com/rancher/rancher/issues/43172
https://github.com/rancher/rancher/issues/42313

https://github.com/rancher/rancher/issues/43331looks like the one!


Call "sub action" from Fastfile

How can I call a "sub action" from my Fastfile?
I want to call  fastlane pilot distribute but from within my Fastfile. I managed to import the class using require "pilot" and then run Pilot::BuildManager.new... but then I had to define all my options again. I would like to have my already predefined options sourced from Fastlane.

Let us know if that works fr yu!

@ohayon Then I'm only calling the main fastlane pilot task (fastlane pilot), I want to call the sub task: distribute.

Ah, got it. So you can you use the Fastfile that I mentioned above and make the pilot call look like this: pilot(distribute_external: true)

@ohayon Ok, but how do I run for example pilot builds?

That can be done by using another option that you can pass to pilot. Have a look at https://docs.fastlane.tools/actions/pilot/#pilot to see all the different options you have. Let us know if you have any more questions! 🚀

How? I have looked at the docs and none of the parameters are build?

Ah, my apologies, for something like that, the best thing to do would be to call into the library directly. If you have a look at the code for that action here https://github.com/fastlane/fastlane/blob/master/pilot/lib/pilot/commands_generator.rb#L76 you can see how it works to implement some similar code in your Fastfile.

I managed to import the class using require "pilot" and then run Pilot::BuildManager.new... but then I had to define all my options again. I would like to have my already predefined options sourced from Fastlane

@vikeri @ohayon I am also looking for a solution for a similar problem. In my case I am trying to list all the builds uploaded using fastlane pilot builds. What's the elegant way to recreate existing options from fastlane core?

Same question here. From command line I can call "fastlane pilot distribute -g xxx", but in Fastfile it's required to provide the path of ipa file while I'm trying to distribute a previously uploaded build.

use case: you don't want to give your CI bot write permissions to your repo so when you deploy a new version, you want to check fastlane pilot builds first to check existing builds to +1 the build number to avoid version collisions when pushing to TestFlight.


Skeleton recipe

Modified and added files into /model that should serve as a base for our TP.

Codecov Report

Merging #12 (32ea4d7) into master (52b1800) will decrease coverage by 9.28%.
The diff coverage is 1.08%.

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


build error

point-cloud-annotation-tool/visualizer.h:41:39: error: cannot declare field ‘Visualizer::colorHandler’ to be of abstract type ‘PointCloudColorHandlerLUTpcl::PointXYZI’ PointCloudColorHandlerLUT colorHandler; ^~~~~~~~~~~~

Hello I got the same error, could u tell me how to do with it?

in ubuntu18.04 it is built and works.


add territory boundaries to Northern Canada

@e-schott: Northern Canada is a combined region. There are also cities that are combined (e.g. Moncton - St. John.) and use a combined shape. How do you represent that the values across these regions are an average and they don't just happen to have the same value individually?

Should be closed with 34a3c08


Shell auto-completion

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
Add shell auto-completion using clap_complete

The code completions are available here: https://github.com/convco/convco/releases/tag/v0.5.1
Depending on how you install convco code completions will be available e.g. brew install convco will also install the completions

I've installed it via cargo install, so I think it doesn't provide completions. Wouldn't it be possible to add a completion command to the CLI that would allow specifying a shell I want completions installed for?

It could be added under a feature completions. and be implemented using the subcommand completions with--shell as option if it could not be autodetected by parsing the SHELL env variable.


Add version tracking system, add qog_fetcher and unwpp_fetcher, add user data editing

Thanks! Will start testing and deployment of this now.


Mdns support

clsx is actually really useful:
import { clsx } from 'clsx'
import { twMerge } from 'tailwind-merge'
import type { ClassValue } from 'clsx'

export function cn(...inputs: ClassValue[]) {
    return twMerge(clsx(inputs))
}

I use this on my projects :)


No ksvc is created in source to sink example

https://redhat-developer-demos.github.io/knative-tutorial/knative-tutorial-eventing/eventing-src-to-sink.html#eventing-sink-service
It says "Run the following commands to create the service:" but the command later creates a source, not a service. Actually, in that page, no command creates a service.
kubectl apply -n knativetutorial -f eventing-hello-sink.yaml should be the command to create the ksvc.

cc @kameshsampath


Order by on Immediate Descendants (tested case)

This:
$immediateDescendants = static::where('parent_id', $root->id)->orderBy('order')->get();
And
$immediateDescendants = $root->immediateDescendants()->orderBy('order')->get();
Return the same results but the second case does not obey the 'order'. Is this the intended behaviour?

Maybe put this on the readme.md as it's quite a usefull method :)


Support of hstore -> linked map

Background
Currently Postgres' HStore is translated into Scala as Map.
Feature
Figure out a way how to make it translated into LinkedMap, because the order of the keys matter.

This approach might not gonna work in all the circumstances, Hstore does not guarantee the order of the records to be based on insertion-order (especially UPDATE operations might cause problems with the original order).
This means that the usage of LinkedMap would not help since the underlying DB type doesn't guarantee this.
We'll have to represent the partition-related column in a different way. A new set of tickets will be created, thus closing this one.


Server could not find CRD

Following the quick start, I get:
kubectl logs trireme-csr-controller-5zssq -n kube-system
INFO	start watching Certificates objects
INFO	Everything started. Waiting for Stop signal
ERROR: logging before flag.Parse: E0222 22:43:19.574805       1 reflector.go:205] github.com/aporeto-inc/trireme-csr/controller/controller.go:65: Failed to list *v1alpha2.Certificate: the server could not find the requested resource (get certificates.certmanager.k8s.io)
...

Kubernetes v1.9.3

https://github.com/aporeto-inc/trireme-csr/issues/27

@andrewrynhard , your problem is most likely this file https://github.com/aporeto-inc/trireme-kubernetes/blob/master/deployment/trireme/certificate-crd.yaml ... please change the version in the CRD from v1alpha1 to v1alpha2. We recently bumped up the API version, and I must have forgotten about this file

it's actually going to be fixed once https://github.com/aporeto-inc/trireme-kubernetes/pull/147 gets merged.

@mheese that got the controller to run, thanks! Now I am seeing this:
FATAL	error loading Certificates for PKI Trireme	{"error": "Error Sending and waiting couldn't create CSR Kube object: Certificate.certmanager.k8s.io \"ip-10-35-122-114\" is invalid: metadata.namespace: Required value"}

Think I might need:
scope: Cluster

in the CRD.

@andrewrynhard yes, exactly ... as in the file in the PR. Sorry about that.


loading-graphql-files with absolute path doens't work.

https://create-react-app.dev/docs/loading-graphql-files/
based on the docs again how can you load graphql files absolute path...
all of the imports work fine with the absolute path but not loading graphql file..
any ideas?

so nobody came across this issue before?


Failed to import but successfully used

When importing the icon an error occurs as follows, but the icon can still be used.

@PremaNandika thanks for reporting it. Will have a look
Which version are you using?

@PremaNandika fixed with the latestv6.0.3
Thanks


Fix matchers for Vitest v0.31+

Changes
Vitest v0.31 and up removed the global Vi assertion namespace, so the matchers don’t work in TypeScript anymore. This updates this library to work with Vitest v0.31 and greater by following their advice from the changelog

The change should be minor:
- declare namespace Vi {
+ declare module 'vitest' {
  interface Assertion<T = any> extends CustomMatchers<T> {}
  interface AsymmetricMatchersContaining extends CustomMatchers {}
}

Testing
Testing these changes locally, my tests are working again:
Before

After

This is a breaking change and will require a few other modifications as well as a new release. Mainly we need to bump the Vitest peerDependency.

@chaance can we get this merged please?

@chaance please can you get this merged and cut a release so we can benefit from the fixes? If you're struggling to find the time to maintain this module I'd happily help maintain/take ownership if it helps?

Not blocking at all, but I’ve also opened up #6 to update this library’s internal dependencies (no breaking changes for consumers) and make this library easier to maintain. No code changes at all; just maintenance things.

Sorry it took me a bit, but I just published v1.0.0-pre.0 with this fix and a couple of others to bring us up-to-date with jest-axe. Gonna take it for a test run over the next few days but feel free to open new issues if you run into anything!


Bug? Calling JumpTo scene with the current scene does nothing.

I was expecting a scene change event, and possible a scene enter event if we do that: #519

As with the other issue, I'm closing this with no action taken:

On reflection, there is enough machinery in place that we don't really need this. Scene changes are triggered by user action, and there is nothing stopping a user from sending a custom event instead.

An additional thing the user might want in this case is a to reset the sceneTime. However, they could always track the time themselves, or simply change scene's twice to achieve the same effect (as a workaround).


Guess the celebrity using two pictures

PR Description 📜
Created a JavaScript game where players have to guess the correct celebrity based on the given images.

Two images containing hints will be displayed on the screen.
Players will also see four random options containing names of celebrities.
Click on the option that you think matches the correct celebrity name.
If your guess is correct, you will earn 5 points and proceed to the next level.
If your guess is wrong, you can try again until you complete all the levels.
Your final score will be displayed at the end of the game.

Fixes #2568

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[x] I have added working video of the game in README.md (optional)
[x] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @amelia2802 ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Hey @amelia2802 add your folder link and game name in the main README.md

Hi @Deepanshu0703, Thank you for the insight but I believe I've added my game folder to the main README table, still the change request coming there. It would be great if you could review the PR and point me out what am I, lacking!

Thank you  @amelia2802 , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


[W6.4b][W11-B2]Zhan Yu

Completed LO-Non Functional Requirements. Added some more stuff.

@ZY-Ang
Good job on this one!
Some comments added. Please ack & close the PR after reading comments.


Add Linux containers support with all_container playbook

Which issue(s) this PR fixes:

Fixes #30
Additional comments:

This PR adds a specific playbook for Linux container hosts. This can be useful for teams running TDP on LXD containers.
To make clear that you license your contribution under
the Apache License Version 2.0, January 2004 and that you give permission to TOSIT,
you have to acknowledge this by using the following check-box.

[x] I hereby declare this contribution to be licenced under the Apache License Version 2.0, January 2004

[x] I hereby agree to grant TOSIT a copyright license to use my contributions.

I wonder if this the good way to do it. I think a facts can have the information if we are in a container and disable the chrony playbook or tasks.

facts

We might look at the fact: ansible_virtualization_type, I don't know if it helps (on Openstack, it returns openstack...)

facts

We might look at the fact: ansible_virtualization_type, I don't know if it helps (
On my openstack vm it returns:
ansible_virtualization_type: openstack
ansible_virtualization_role: guest

On my host it returns:
ansible_virtualization_type: kvm
ansible_virtualization_role: host

On a LXD container it returns:
ansible_virtualization_type: lxc
ansible_virtualization_role: guest

Found an other variable ansible_virtualization_tech_guest:
ok: [edge-01] =>
  ansible_facts.virtualization_tech_guest:
  - lxc
  - container

facts

We might look at the fact: ansible_virtualization_type, I don't know if it helps (
On my openstack vm it returns:
ansible_virtualization_type: openstack
ansible_virtualization_role: guest

On my host it returns:
ansible_virtualization_type: kvm
ansible_virtualization_role: host

On a LXD container it returns:
ansible_virtualization_type: lxc
ansible_virtualization_role: guest

Found another variable ansible_virtualization_tech_guest which might be more generic. Not sure that its available on all Ansible versions:
ok: [edge-01] =>
  ansible_facts.virtualization_tech_guest:
  - lxc
  - container

virtualization_tech_guest returns [] on my host (which is normal since it's an Host machine
On openstack the fact is missing

ansible_virtualization_type

Ok let's stick with ansible_virtualization_type then.

Forced push to only call the chrony role when host is not a Linux container.


Format yield* the same as function*

Fixes #920 to have yield* formatting follow function* formatting

I'm not sure how to have the checks complete properly for this - if there's a way to do that, please let me know!

Hello, I'm sorry for the delay.  I've had no time to review changes or to fix the issue with the CI tools. Please be patient.

Awesome, just wanted to make sure I wasn't missing anything. If I can do anything in the mean time, please let me know.

Would also be happy if this was fixed. Recently started using redux-saga for async processing, and it is heavily dependant on generator functions.

@jgeurts - Thanks, I took your change and made some tweaks and added python implementation.

Great, thanks!


Command Line Interface

[ ] Should be able to install chackages
[ ] Interact with text editor
[ ] Resizable

Build from scratch or use library?

Using built in console. Moved to Icebox.


Background CPU usage when opening a notebook with folded markdown cells

Open a notebook, fold some markdown headers that include markdown cells
Close and reopen the notebook so the folded cells are persisted as folded
See the renderer process using CPU

I took a profile and saw that we keep trying to prerender a markdown cell, but then stop because it is folded:
https://github.com/microsoft/vscode/blob/97dddde1f742d43184bd8988958a5d9618dbc5f5/src/vs/workbench/contrib/notebook/browser/notebookEditorWidget.ts#L2628
so the code here:
https://github.com/microsoft/vscode/blob/97dddde1f742d43184bd8988958a5d9618dbc5f5/src/vs/workbench/contrib/notebook/browser/notebookEditorWidget.ts#L1198
should not retry the same cell over and over
Not any recent regression as far as I can tell

@roblourens I guess we kept checking hidden ranges and then checking if a cell is visible. Good catch.


Recommendations for net.ipv4.tcp_retries2

Hello.
The QualityCheck v2 is looking for "net.ipv4.tcp_retries2 = 3".
According to the SUSE document here:  https://www.suse.com/support/kb/doc/?id=000019722
(created 21-Oct-2021).
A value of less than 8 will go against the recommendations of RFC 1122.
That same document shows a default value for "net.ipv4.tcp_retries1 = 3" (retries1), is there some confusion between retries1 and retries2 settings ?   or are we saying that they both need to be "3", against what the SUSE KB is recommending?
Thanks

yes, this seems to be a new note, I will discuss it internally.
the note doesn't prohibit to decrease the value and it mentions that decreasing the value might solve an issue.

updated according to SUSE notes


host tf2bd-util.pazer.us is down

Describe the bug

Bot Detector (v<IP_ADDRESS>6 Nightly) Hangs on update check indefinitley, refuses to qui on exit, or alt-f4. Must kill process.

To Reproduce

Steps to reproduce the behavior:

Launch Bot Detector.exe (I did it from taksbar)
See error

Expected behavior

Update check finishes and goes to launch TF2 screen.

Logs

Application hangs before logs are generated.

Screenshots

Desktop (please complete the following information):

OS: Windows 10 21H2 19044.1645
Version of bot detector v<IP_ADDRESS>6 Nightly

Additional context

Can't ever laucnh app normally. Works if automatic update check disabled.

Same, disabled updates check

This is due to tf2bd-util.pazer.us currently being down. There is ongoing work to migrate to a new host. Disable checking for updates and if you want the steam API integration you have to use your own key and switch to direct access. See https://github.com/PazerOP/tf2_bot_detector/wiki/Integrations:-Steam-API for information on how to do that

Hey is it still down? I just recently found this github and am really new to it. Would have asked how to install it

It's still down, but it can be fixed by changing "Notify about new stable releases" to "Disable automatic update checks"

I just learned about this software and came across the same issue. Is this still being maintained?

hasn't had an update in a while but it still works fine

Sadly it seems there's new bots that don't get detected, so you have to alt+tab and mark them as a cheater on the program. Thankfully though that process only takes a few seconds. Eventually though I did manage to build up a list so when they'd re-join, this program would instantly start a vote kick against them.

I've got a list that works with most of the new bots on my profile

I've got a list that works with most of the new bots on my profile

Is this compatible with the compiled windows version? or do I need to compile it? I've got a large playerlist.biglist.json also, can I merge your changes with that somehow? Thanks :3

I've got a list that works with most of the new bots on my profile

Is this compatible with the compiled windows version? or do I need to compile it? I've got a large playerlist.biglist.json also, can I merge your changes with that somehow? Thanks :3

It's a seperate file you put alongside the json files, they're named rules.qfoxb.json and playerlist.qfoxb.json

Thank you <3. It works great!


NIFI-6028: Updates to ignore changes between local version of a flow …

…and a remote version of a flow if the difference is the addition of a new property that has the default value
Thank you for submitting a contribution to Apache NiFi.
Please provide a short description of the PR here:
Description of PR
Enables X functionality; fixes bug NIFI-YYYY.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[ ] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ ] Has your PR been rebased against the latest commit within the target branch (typically master)?

[ ] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

Will review...

Outside of the minor comment above, I am a +1 for this PR. However, we cannot merge it in as it requires a new version of NiFi Registry. Once the Registry is released, we should be good to merge this in.


Trouble building extension

I've attempted to clone, build, and link webln:
git clone git@github.com:/wbobeirne/webln
cd webln
yarn
yarn link

and then clone and build joule-extension:
git clone git@github.com:/wbobeirne/joule-extension
cd joule-extension
yarn link webln
yarn
yarn run build-dev
yarn run dev

node: v10.13.0
npm: v6.4.1
yarn: v1.12.1
This does not appear to build a manifest file in ./dist, so I'm unable to install the unpacked extension.
Did I skip a step?

That's embarrassing, you're 100% correct. I had hard-coded a couple of the files to dist, but they were obviously .gitignored. I'll get webpack to build them correctly at some point, but in the mean time, I'll go ahead and push up the missing files.

Pushed with 727f5174ddb0ae9550616247873bf5955ee4d3c9, thanks again for letting me know.


Variable item expiration

Hi again
What's the best way to implement specific cache expiration rules for a single query?  We have one query that needs to be refreshed much more frequently than the other 99% of queries.  It would be nice if there were a simple override to Cacheable() that just let us pass TotalSeconds to cache.
thanks!
jasen

Added it via #https://github.com/VahidN/EFSecondLevelCache.Core/commit/97eb7dac2c05a5586b6c5ba669bd99d8589f0239


Added tiscamera yocto recipe, 4.x onward (kirkstone)

I added the recipe that I used in yocto to build tiscamera software. This recipe works with yocto version 4.x onward and for tiscamera version 1.0.0.
It can be copied inside a custom layer by the end user, or it can be a starting point for creating an official tiscamera meta layer

Thank you very much!


[ISSUE] Table doesn't exist error when running session data cleanup script.

Describe the issue:
When running the session data clean up task [1] following error was encountered.
ERROR 1146 (42S02): Table 'testdewni1.IDN_AUTH_USER_SESSION_MAPPING' doesn't exist
This error is due to the script modification from PR [2]. Ideally, this script should function successfully for all IS versions.
Environment information:

Product Version: IS 5.2.0
Database: MySQL 5.7

[1] https://github.com/wso2/carbon-identity-framework/blob/master/features/identity-core/org.wso2.carbon.identity.core.server.feature/resources/dbscripts/stored-procedures/mysql/sessiondata-cleanup/mysql-session-data-cleanup.sql
[2] https://github.com/wso2/carbon-identity-framework/pull/2792

Same error occurs when trying to run this against an APIM-2.1.0 during a migration. This needs to be fixed.
Conditional check for the existence of the table would resolve this IMO.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Question about perf considerations in tick()

Hi,
I was just reading through the code and noticed the tick() method:
https://github.com/martinohanlon/CanaryRaspberryJuice/blob/master/src/com/stuffaboutcode/canaryraspberryjuice/RemoteSession.java#L109
	/** called from the server main thread */
	public void tick() {

		if (origin == null) this.origin = plugin.getSpawnLocation();
		int processedCount = 0;
		String message;
		while ((message = inQueue.poll()) != null) {
			handleLine(message);
			processedCount++;
			if (processedCount >= maxCommandsPerTick) {
				plugin.getLogman().warn("Over " + maxCommandsPerTick +
					" commands were queued - deferring " + inQueue.size() + " to next tick");
				break;
			}
		}

		if (!running && inQueue.size() <= 0) {
			pendingRemoval = true;
		}
	}

Would you mind explaining some of the findings that led to this implementation? In particular, what are your observations about the behavior of the server at various maxCommandsPerTick thresholds?
Regards,
Steve

Hi Steve,
A lot of the implementation decisions where made when the original bukkit version of RaspberryJuice was created, but I give you my insight.
The tick method is called for each session each time the minecraft server 'ticks' see CanaryRaspberryJuiceListener.onTick(). This allows the server to 'choose' when commands are processed, the upshot being that if the server is busy and not ticking, raspberry juice doesnt take up additional resources.
The maxCommandsPerTick is a threshold which stops Raspberry Juice slowing the server tick, by limiting the number of commands it will process per tick. The value of 9000 was picked through a bit of trial an error as being enough but not too many!  One of the changes I need to make at some point is make this value configurable.
I hope that helps.
Mart


fix: issue with some acl configurations

writeAuthenticationMiddleware is not acl related?

Oops. Working on too manny things at once. That should have been in the put branch. Will fix...

screwed this one up


feat: upgrade deps

Description
Upgrading deps while making sure there are no dep conflicts
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Internal change (a change which is not visible to the package consumers)
[ ] Documentation change or requires a documentation update

How Has This Been Tested?

[ ] Tested manually
[ ] Added automated tests

Checklist:

[ ] Unit tests have been added/updated
[ ] Documentation has been updated
[ ] Change file has been generated (npm run change:new)
[ ] Delete this branch (after the PR is merged)

Codecov Report

Merging #481 (702d9c1) into main (fc7b002) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #481   +/-   ##
=======================================
  Coverage   77.87%   77.87%
=======================================
  Files          26       26
  Lines         764      764
  Branches      170      170
=======================================
  Hits          595      595
  Misses        169      169

Impacted Files
Coverage Δ

src/tracing/SplunkBatchSpanProcessor.ts
100.00% <100.00%> (ø)

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 fc7b002...702d9c1. Read the comment docs.


[Impeller] Android The screen breaks when the screen is rotated.

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

Repo.  https://github.com/BBUBBA/rotatetest
Run.
Click - rotate1, rotate2, rotate3

Expected results
The Android screen rotates horizontally or vertically.
Actual results
The Android screen is divided in half.
When the home screen is moved and re-executed, the screen is output normally.
(Android Emulator, Galaxy S22, and Galaxy S23 are the same symptoms.)
Code sample
Code sample
[Paste your code here]

void _rotate1() {
    SystemChrome.setPreferredOrientations([
      DeviceOrientation.landscapeLeft,
      DeviceOrientation.landscapeRight,
    ]);
    setState(() {
      _rotate = 1;
    });
  }

  void _rotate2() {
    SystemChrome.setPreferredOrientations([
      DeviceOrientation.portraitUp,
      DeviceOrientation.portraitDown,
    ]);
    setState(() {
      _rotate = 2;
    });
  }

  void _rotate3() {
    SystemChrome.setPreferredOrientations([
      DeviceOrientation.portraitUp,
      DeviceOrientation.portraitDown,
      DeviceOrientation.landscapeRight,
      DeviceOrientation.landscapeLeft,
    ]);
    setState(() {
      _rotate = 3;
    });
  }

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[Paste your output here]

[√] Flutter (Channel stable, 3.16.0, on Microsoft Windows [Version 10.0.19045.3570], locale ko-KR)
    • Flutter version 3.16.0 on channel stable at C:\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision db7ef5bf9f (6 days ago), 2023-11-15 11:25:44 -0800
    • Engine revision 74d16627b9
    • Dart version 3.2.0
    • DevTools version 2.28.2

    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop Windows apps (Visual Studio Professional 2022 17.7.7)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Professional
    • Visual Studio Professional 2022 version 17.7.34302.85
    • Windows 10 SDK version 10.0.19041.0

[√] Android Studio (version 2022.3)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-10027231)

[√] VS Code (version 1.84.2)
    • VS Code at C:\Users\csh34\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.68.0

[√] Connected device (4 available)
    • sdk gphone64 x86 64 (mobile) • emulator-5554 • android-x64    • Android 12 (API 32) (emulator)
    • Windows (desktop)            • windows       • windows-x64    • Microsoft Windows [Version 10.0.19045.3570]
    • Chrome (web)                 • chrome        • web-javascript • Google Chrome 119.0.6045.160
    • Edge (web)                   • edge          • web-javascript • Microsoft Edge 119.0.2151.72

[√] Network resources
    • All expected network resources are available.

• No issues found!

Thanks for filing this.
I see this seems to reproduce on the latest stable channel but not on the latest master channel.
Are you able to check this as well on your end.
stable, master flutter doctor -v

[!] Flutter (Channel stable, 3.16.0, on macOS 14.0 23A344 darwin-arm64, locale
    en-NG)
    • Flutter version 3.16.0 on channel stable at
      /Users/damilolaalimi/sdks/flutter
    ! Warning: `dart` on your path resolves to
      /opt/homebrew/Cellar/dart/3.1.5/libexec/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/damilolaalimi/sdks/flutter.
      Consider adding /Users/damilolaalimi/sdks/flutter/bin to the front of your
      path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision db7ef5bf9f (5 days ago), 2023-11-15 11:25:44 -0800
    • Engine revision 74d16627b9
    • Dart version 3.2.0
    • DevTools version 2.28.2
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/damilolaalimi/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/damilolaalimi/Library/Android/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15A507
    • CocoaPods version 1.12.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      17.0.6+0-17.0.6b802.4-9586694)

[!] Android Studio (version unknown)
    • Android Studio at /Users/damilolaalimi/Downloads/Android Studio
      Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to determine Android Studio version.
    • Java version OpenJDK Runtime Environment (build
      17.0.7+0-17.0.7b1000.6-10550314)

[✓] VS Code (version 1.84.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.50.0

[✓] VS Code (version 1.83.1)
    • VS Code at /Users/damilolaalimi/Downloads/Visual Studio Code.app/Contents
    • Flutter extension version 3.50.0

[✓] Connected device (5 available)
    • sdk gphone64 arm64 (mobile) • emulator-5554                        •
      android-arm64  • Android 14 (API 34) (emulator)
    • Damilola’s iPhone (mobile)  • 00008110-001964480AE1801E            • ios
      • iOS 17.1.1 21B91
    • iPhone 15 (mobile)          • F17D2919-6D61-4295-8408-1719692FE958 • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-17-0 (simulator)
    • macOS (desktop)             • macos                                •
      darwin-arm64   • macOS 14.0 23A344 darwin-arm64
    • Chrome (web)                • chrome                               •
      web-javascript • Google Chrome 119.0.6045.159

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 2 categories.

[!] Flutter (Channel master, 3.17.0-12.0.pre.39, on macOS 14.0 23A344 darwin-arm64, locale en-NG)
    • Flutter version 3.17.0-12.0.pre.39 on channel master at /Users/damilolaalimi/fvm/versions/master
    ! Warning: `dart` on your path resolves to /opt/homebrew/Cellar/dart/3.1.5/libexec/bin/dart, which is not inside your current Flutter SDK checkout at /Users/damilolaalimi/fvm/versions/master. Consider adding /Users/damilolaalimi/fvm/versions/master/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision b6ef78e955 (2 hours ago), 2023-11-20 23:37:25 -0500
    • Engine revision 70b1c73412
    • Dart version 3.3.0 (build 3.3.0-152.0.dev)
    • DevTools version 2.30.0-dev.4
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/damilolaalimi/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/damilolaalimi/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15A507
    • CocoaPods version 1.12.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[!] Android Studio (version unknown)
    • Android Studio at /Users/damilolaalimi/Downloads/Android Studio Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to determine Android Studio version.
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)

[✓] VS Code (version 1.84.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.50.0

[✓] VS Code (version 1.83.1)
    • VS Code at /Users/damilolaalimi/Downloads/Visual Studio Code.app/Contents
    • Flutter extension version 3.50.0

[✓] Connected device (5 available)
    • sdk gphone64 arm64 (mobile) • emulator-5554                        • android-arm64  • Android 14 (API 34) (emulator)
    • Damilola’s iPhone (mobile)  • 00008110-001964480AE1801E            • ios            • iOS 17.1.1 21B91
    • iPhone 15 (mobile)          • F17D2919-6D61-4295-8408-1719692FE958 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-17-0 (simulator)
    • macOS (desktop)             • macos                                • darwin-arm64   • macOS 14.0 23A344 darwin-arm64
    • Chrome (web)                • chrome                               • web-javascript • Google Chrome 119.0.6045.159

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 2 categories.

What API level is the Android emulator configured with?

From investigation, the problem is that on certain rotations we re-render too "early" and as a result the surface properties have not been updated yet. This causes us to still present with the outdated rotation.
The solution appears to require that we specifically listen to native window resize events and propagate a new frame when that fires. See also: https://github.com/google/vulkan-pre-rotation-demo/blob/b3f836dee9d9925820e7631dc9f4b3fe77cfa56b/app/src/main/cpp/main.cpp#L72C31-L72C52

Note that fixing this may actually allow us to go back to suboptimalKHR instead of polling every frame too.


Using from_netcdf getting a TypeError

Hi all,
I'm fairly new to this, so it could be just user error. I'm using model output from a mom01v5 model, and trying to use the FieldSet.from_netcdf method to create a fieldset, but I'm getting the error
TypeError: '>' not supported between instances of 'datetime.timedelta' and 'int'
My code:
filenames = "/g/data3/hh5/tmp/cosima/mom01v5/KDS75/output*/ocean__*_*.nc"
variables = {'U': 'u',              'V': 'v'}
dimensions = {'lat': 'geolat_c',               'lon': 'geolon_c',               'time': 'time',               'depth': 'st_ocean'}
fieldset = FieldSet.from_netcdf(filenames, variables, dimensions)
The issue appears to be in parcels/field.py line 310
assert(np.all((time[1:]-time[:-1]) > 0))
when called from parcels/fieldset.py line 201-203
fields[var] = Field.from_netcdf(paths, var, dims, inds, grid=grid, mesh=mesh, allow_time_extrapolation=allow_time_extrapolation, time_periodic=time_periodic, full_load=full_load, **kwargs)
Any help would be great! Thanks.

Hi Michael,
Thanks for reporting this Issue; it appears to be a bug yes. For some reason, your time is a datetime, rather than a float, as Field.from_netcdf is always supposed to return a float. Not really sure how that could happen..
Would you be able to share one of your output files with me? Then it will be easier for me to debug and provide a fix

Hi Erik, thanks for getting back so quickly. Unfortunately the files are rather large (~5.6GB each). Are there any diagnostics I could run which would help?

Can't you share one file over e.g. wetransfer with me? I'm at<EMAIL_ADDRESS>That really is easier (and faster!) than trying to debug over this Issue tracker

Whatever you can send is useful. But make sure it has the correct
dimensions (including time) as the problem seems to be in reading that
dimension
On Mon, 27 Aug 2018 at 14:23 Michael Denes<EMAIL_ADDRESS>wrote:

Unfortunately the files are sitting on a remote server (NCI supercomputer
in Australia), and I'm not sure the best way to get them to you. I can send
you a netcdf file from the same model run/experiment with scalar outputs
(no velocities, just total ocean salt/total ocean melt etc.), would that be
useful?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/OceanParcels/parcels/issues/425#issuecomment-416209096,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANpuNqkLK7l3JB9LOt0DO55jofv9ez2kks5uU-TbgaJpZM4WNouR
.

Hi Michael,
The good news is that I downloaded your files and could reproduce the bug. I also know now that the issue is with the time:calendar_type = "NOLEAP" attribute in your netcdf file.
I'll dig a little deeper in how we could support these NOLEAP calendars in Parcels. The issue is that they don't get converted to numpy.datetime64 objects, as is expected by Parcels. More soon!

Thanks Erik, I really appreciate this! I've not had much experience with ocean model output or netcdf files before, so thank you!

I've created a new branch at https://github.com/OceanParcels/parcels/tree/cftime_noleap_calendar that is a quick and dirty implementation of cftme._cftimeDatetimeNoLeap calendars. Could you check if you can run Parcels using this branch? Perhaps you may need to pip install cftime too

Hi Erik, I believe it's a related issue, so I won't open a new one.
When I create a particle set
pset = ParticleSet.from_line(fieldset=fieldset, size=10, pclass=JITParticle, start=(31, -31), finish=(38, -38))
and try to plot the particles with the U field
pset.show(field=fieldset.U)
I get the type error:
TypeError: cannot compare cftime.DatetimeNoLeap(96, 1, 3, 12, 0, 0, 0, 2, 3) and 0.0
The issue looks to come down to line 965 in parcels/field.py
ti = (time_data <= g.time[tindex]).argmin() - 1
Similarly, if I try to advect those particles with
pset.execute(AdvectionRK4, runtime=timedelta(days=20), dt=timedelta(days=1))
I get the same error
TypeError: cannot compare cftime.DatetimeNoLeap(96, 1, 3, 12, 0, 0, 0, 2, 3) and 0.0
Same line (965 in parcels/field.py) throws this error.

Hi Erik, thanks for this, I've rerun with your suggestion for the ocean only area. I could get the particles to show using pset.show(), and advect using pset.execute.
However, I couldn't get pset.show(field=fieldset.U) working for example, but I think that's down to user error! I'll likely read through the code itself to understand what's going on and how I can use the show() function. Thank you again!

Hi Erik, when using the output_file argument in pset.execute, it throws the same type error
TypeError: cannot compare cftime.DatetimeNoLeap(96, 1, 3, 12, 0, 0, 0, 2, 3) and 0
Issue on line 68 of parcels/particlefile.py
My code for reference:
output_file = pset.ParticleFile(name="test_single_particle", outputdt=timedelta(hours=12)) pset.execute(AdvectionRK4, runtime=timedelta(days=20), dt=timedelta(days=1), output_file=output_file)

I've just updated the https://github.com/OceanParcels/parcels/tree/cftime_noleap_calendar branch to also be compatible with NOLEAP calendars in ParticleFile writing. Let me know if this works

Hi Erik, apologies for the delay. It looks to have worked, I just need to think about the land set to NaN problem now! I'll play around with some more functions to see if the issue pops up elsewhere, but thank you for all your help so far!

Hi Erik,
One thing I have noticed is that once a particle goes out of bounds, there is a type error:
TypeError: ufunc add cannot use operands with types dtype('O') and dtype('<m8[s]')
caused on line 33 of parcels/kernels/error.py
time = fieldset.time_origin + np.timedelta64(int(time), 's')
Would this be caused by the time type as well?

Hi Michael, I was wondering if you've had a try of the new code in PR #456. Does it work for you?  If so, we can merge the PR and close this Issue

Fixed in #456


Update: Add IPFS downloads and uploads considerations

@jcramer Here is the PR as requested.  Would love to hear your guys' feedback and suggestions.
Changes:

Added details for operation of uploading/download of content stored in IPFS
Documented file_uri_utf8option for IPFS
Added optional file_uri_metadata_utf8 to describe the URI external storage type (micro format within
Attribution from MatterBCH changed, as that is Matter's reddit username that I used. Reference

Really curious how we can improve this last option file_uri_metadata_utf8 as this not my favorite part, but I think it is valuable for posterity for the files stored and the protocol itself. It is optional, and clients can in theory reconstruct this with tuning the options until they arrive at it. This is why we recommend to not change any of the ipfs settings when creating the hash.
Also here are some comments and questions after reviewing the specification that may be helpful.

Additional Notes/Questions:

There is a funding TX required in a chain of utxos. If the user calculates the fee incorrectly, during network congestion (yes, this should "never" happen), then they may not be able to complete uploading all their chunks in a timely fashion. This can be mitigated later on with a mining node upgrade, see next few comments.

In order to support larger files, then the max chain of unconfirmed transactions must be increased, or the user must be willing to wait for more block confirmations in turn and therefore the upload may take hours, or even days (depending on file size).  Related to errortoo-long-mempool-chain

Nonetheless, this is a very elegant and predictable solution to optimizing for storage space since no chunk counter and/or reference txid needs to be be embedded (and therefore no space of the OP_RETURN is lost for the data chunks).  It will be easier to optimize the max unconfirmed chained transactions than it is to potentially lose 34 Bytes+ for every OP_RETURN referencing/chunk counters.

What is the difference between <size_bytes_int*>  and  <chunk_X_data_bytes*>?   Answer: size_bytes_int is the total size of the file, whereas chunk_X_data_bytes is the bytes of the chunk contained in the same transaction as the metadata. An observation here is that the size_bytes_int is technically redundant, but best practice should be to provide it here too, since there is a field for it.

How would data retrieval work if there are many chunk transactions? Is there a way something like bitdb.network could return all of the chunks with a single query?

Perhaps this could use more clarification:

If the file includes 1 or more data chunks AND the chunk data is not included in this metadata transaction then first input to this transaction (i.e., vin: 0) shall be used to locate the first data chunk.

The vout1 of the metadata transaction points to the first chunk, correct?

Note: file_extension_utf8 should be adequate to determine file type. It is preferable to use a file extension over a full MIME type to conserve space.

@jcramer Here is the addition of the 'IPFS URI storage device' as per the latest updates.
The key change here is the addition of an optional file_uri_metadata_utf8 that can provide additional information about the storage parameters at the URI.
This is needed for IPFS, since hashes are dependant upon more than just the sha256 of the file content.  Perhaps in the future, there will be other storage devices which will have similar characteristics which may necessitate metadata for the purposes of being able to reconstruct the hash.
What do you think?

How would data retrieval work if there are many chunk transactions? Is there a way something like bitdb.network could return all of the chunks with a single query?

Bitdb could be used possibly, but the design intent was for each raw transaction to be downloaded in full and BFP data parsed.

The vout1 of the metadata transaction points to the first chunk, correct?

Actually, it is the other way around.  Note that the Metadata Transaction is the final transaction in the series of data chunk uploads.  vin:0 of the Metadata Txn points to the previous chunk transaction.

The key change here is the addition of an optional file_uri_metadata_utf8 that can provide additional information about the storage parameters at the URI.
This is needed for IPFS, since hashes are dependant upon more than just the sha256 of the file content. Perhaps in the future, there will be other storage devices which will have similar characteristics which may necessitate metadata for the purposes of being able to reconstruct the hash.
What do you think?

The intent of the field titled file_uri_utf8 was added specifically for external file uri.  Are you saying that file_uri_metadata_utf8 is information that is required in additional to the files uri?
Let me know and I can merge this.
Also, based on your additions here can I list you in the authors section?

@a-aros great.  Please update your PR with rewritten option 2 for including the query parameters in the file_uri_utf8 field.   Also please move your name into the Authors section separated using a comma.  I will await your update. Thank you

@jcramer Changes updated for your review. (ps I've moved my name from 'acknowledgements' to 'authors' as per your request...let me know if this is incorrect) Thanks


SharpHook will lock the thread in single threaded application

Hi, I'm making an app that uses SharpHook to hook all kb/mouse inputs but have a peculiar issue at the moment.
My app is single threaded with custom synccontext so all async operations will guarantee to return to mainthread. When I instantiate SharpHook it will just lock my app entirely which I suspected due to SH's asynchronous background worker that runs in the background.
This issue won't happen if I spawn a dedicated thread just for SH to use, but this is not a solution due to how my app is made specifically to be single threaded.
Any idea how can i solve this? or at least a way to queue SH operations to my dedicated messageloop instead?
Thanks in advance..

wtf, why I opened the issue here ?


workspace_tools: Use python2 in shebang

The scripts are not python3 compatible so we should explicitly call
python2.  Some distributions use python3 as the default python interpreter (ArchLinux for example) so executing the scripts directly fails on those platforms.

Hi, can you please sign : http://developer.mbed.org/contributor_agreement/ ?

Done: You accepted the Contributor Agreement on Thu 23 Apr 2015.


How to set the perl path for ROUGE-1.5.5.pl

Hi, @pltrdy Thx for this nice rouge package～
But I run files2rouge got an error like:
Can't locate LWP/UserAgent.pm in @INC (@INC contains: /users/XXX/.files2rouge /usr/local/lib64/perl5 /usr/local/share/perl5 /usr/lib64/perl5/vendor_perl /usr/share/perl5/vendor_perl /usr/lib64/perl5 /usr/share/perl5 .)
/users/XXX/.files2rouge is my files2rouge setup path during the installation.
In fact, I install LWP/UserAgent.pm  in my perl5 path, but I am not a root user so I can't install any perl package in /usr path.
Therefore, I want to know whether there is a way to set the perl path for rouge-1.5.5.pl?

Fixed. I can directly set the path in the first line of the install ROUGE-1.5.5.pl.
BTW, I found a similar problem like #43 . I follow the Readme of BART for reproducing CNN/DM results.
I directly use the public checkpoint for CNN/DM, so maybe there is no problem in training/inference.
I use the Corenlp to tokenized the reference and prediction file.
I use the pyrouge or files2rouge to evaluate.
The pyrouge results are below:
`1 ROUGE-1 Average_R: 0.51297 (95%-conf.int. 0.51018 - 0.51576)
1 ROUGE-1 Average_P: 0.40694 (95%-conf.int. 0.40444 - 0.40952)
1 ROUGE-1 Average_F: 0.44288 (95%-conf.int. 0.44063 - 0.44507)
1 ROUGE-2 Average_R: 0.24449 (95%-conf.int. 0.24151 - 0.24725)
1 ROUGE-2 Average_P: 0.19458 (95%-conf.int. 0.19220 - 0.19700)
1 ROUGE-2 Average_F: 0.21127 (95%-conf.int. 0.20876 - 0.21368)
1 ROUGE-L Average_R: 0.35718 (95%-conf.int. 0.35430 - 0.35986)
1 ROUGE-L Average_P: 0.28160 (95%-conf.int. 0.27928 - 0.28400)
1 ROUGE-L Average_F: 0.30727 (95%-conf.int. 0.30490 - 0.30957)`
This R-L result is very lower the expected 40.90.
The files2rouge results are below:
1 ROUGE-1 Average_R: 0.40390 (95%-conf.int. 0.40134 - 0.40648)
1 ROUGE-1 Average_P: 0.50956 (95%-conf.int. 0.50689 - 0.51227)
1 ROUGE-1 Average_F: 0.43971 (95%-conf.int. 0.43750 - 0.44215)
1 ROUGE-2 Average_R: 0.19393 (95%-conf.int. 0.19147 - 0.19646)
1 ROUGE-2 Average_P: 0.24389 (95%-conf.int. 0.24102 - 0.24680)
1 ROUGE-2 Average_F: 0.21063 (95%-conf.int. 0.20813 - 0.21326)
1 ROUGE-L Average_R: 0.37440 (95%-conf.int. 0.37190 - 0.37693)
1 ROUGE-L Average_P: 0.47246 (95%-conf.int. 0.46979 - 0.47529)
1 ROUGE-L Average_F: 0.40764 (95%-conf.int. 0.40535 - 0.41010)
We can observe the difference on R-L. Could you give some advice for this situation?

I don't know how pyrouge works but if I remember right we calculate sentence level ROUGE-L (check original article for reference). Maybe pyrouge does not split sentence correctly, thus giving different results.

Thanks a lot.


doesnt work in 1.20.6

I will give $ to have this updated to 1.20.6, if you are up to it. please.
Thank you so very much!

updated, thank you!


Release dates parsed incorrectly

All the TmdbReleaseDate obtained by calling movies.getDetails on a Tmdb3, with AppendResponse.RELEASES_DATES, contain a  null releaseDate.
I suspect it's because of a bug in LocalDateTimeSerializer: the class tries to parse the String as an instant, which it is, but then ignores the result, see this line.
An example of String being parsed to null is 1993-07-08T00:00:00.000Z: toInstant() works, while toLocalDateTime() errors out with:
java.time.format.DateTimeParseException: Text '1993-07-08T00:00:00.000Z' could not be parsed, unparsed text found at index 23

This happened in all movies I've tried, e.g. 526896

Thanks. I fixed the issue!

Thanks!


Argument $number of unitless($number) must be a number

Trying to use the modular scale function in a heading map like so:
$h1-map: (
	null: (32px, 1.2),
	large: (ms(2), 1.2)
);

But I get this error:
Message:
    node_modules/typi/scss/ty/_functions.scss
Error: argument `$number` of `unitless($number)` must be a number

       Backtrace:
       	node_modules/typi/scss/ty/_functions.scss:30, in function `unitless`
       	node_modules/typi/scss/ty/_functions.scss:30, in function `_ty-calc-font-size`
       	node_modules/typi/scss/ty/_functions.scss:15, in function `_ty-font-size`
       	node_modules/typi/scss/ty/_mixins.scss:65, in mixin `-ty-write-props`
       	node_modules/typi/scss/ty/_mixins.scss:36, in mixin `-ty-breakpoint`
       	node_modules/typi/scss/typi/_typi.scss:29, in mixin `typi`
       	src/scss/base/_typography.scss:8
        on line 30 of node_modules/typi/scss/ty/_functions.scss
>>   @if unitless($font-size) {
   ------^

Did you have modular scale installed?
Sent from my iPhone

On 17 Feb 2017, at 11:47 AM, Matthew Kosloski<EMAIL_ADDRESS>wrote:
Trying to use the modular scale function in a heading map like so:
$h1-map: (
null: (32px, 1.2),
large: (ms(2), 1.2)
);
But I get this error:
Message:
node_modules/typi/scss/ty/_functions.scss
Error: argument $number of unitless($number) must be a number
   Backtrace:
   	node_modules/typi/scss/ty/_functions.scss:30, in function `unitless`
   	node_modules/typi/scss/ty/_functions.scss:30, in function `_ty-calc-font-size`
   	node_modules/typi/scss/ty/_functions.scss:15, in function `_ty-font-size`
   	node_modules/typi/scss/ty/_mixins.scss:65, in mixin `-ty-write-props`
   	node_modules/typi/scss/ty/_mixins.scss:36, in mixin `-ty-breakpoint`
   	node_modules/typi/scss/typi/_typi.scss:29, in mixin `typi`
   	src/scss/base/_typography.scss:8
    on line 30 of node_modules/typi/scss/ty/_functions.scss

@if unitless($font-size) {
------^
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Yeah, and I'm running Typi 2.3.0.  I tried it with a custom function I made and got the same error.  Here is my repo file.  The strange thing is when I used this with Webpack, I didn't get the error.

Uhhh. Not too sure what's happening ><. Can you try again with V3 (Docs just updated. There's a breaking change to the typi map).

Closing until further comments.

I am  running in the same problem using modular-scale inside $typi. I tried different versions of modular-scale and typi. Maybe someone run into the same trouble?
Example:
$typi: (
base: (
null: (15px, 1.4),
small: ms(2)
)
);
Error: $number: "ms(2)" is not a number for 'unit'


pythonPackages.python-openid: init at 2.2.5

Motivation for this change
Package python-openid.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

The first seems to have a patch available that we could fetch (and its only DoS anyways). The second seems to be resolvable by adding a pycurl dependency.

Yes, these two might be fixable. But it's generally a bad idea to use security-related software that hasn't been updated in 4 years. There are probably many more undiscovered issues.

But it's generally a bad idea to use security-related software that hasn't been updated in 4 years

Thats true, but imo not in the control of a package manager. I am packaging this because sagenb (jupyter notebook) depends on flask-openid which depends on this. In the case of a jupyter notebook, the bad security might be acceptable. But in any case, wheather or not to use this is the decision of the upstream developers (sagenb in this case).

What does @FRidh think?

The last commit is 4 years old, and the last release 8 years. That's not going in.
@timokau not everything one needs, needs to be upstreamed to Nixpkgs. In this case you can keep the expressions for yourself or elsewhere.

 Does that mean I can't upstream sage at all or can I just include python-openid as a private dependency that is not exposed?

not everything one needs, needs to be upstreamed to Nixpkgs.

Of course not, but its usually better for everybody when it is.

Investigate whether this dependency is really needed. openid is just an authentication method so it's unlikely it is a mandatory dependency.

Of course not, but its usually better for everybody when it is.

Indeed, usually. Large pieces of software with many dependencies (sage and gitlab but also say chromium) require quite the dedication to make and keep it working. I would love to see these in Nixpkgs (chromium is, gitlab also but is causing trouble), but they require dedicated maintainers, and typically more than one as well. If there is a working implementation, stable over some time, then it can absolutely go in. Until then I suggest to keep it in an overlay and find collaborators for it.

I would love to see these in Nixpkgs (chromium is, gitlab also but is causing trouble), but they require dedicated maintainers, and typically more than one as well.
If there is a working implementation, stable over some time, then it can absolutely go in. Until then I suggest to keep it in an overlay and find collaborators for it.

Since I resurrected the sage build (current nixpkgs sage rebuilds all its dependencies in isolation), some people already helped me debug it. Among them (non-comprehensive) @7c6f434c, @dtzWill, @vcunat.
Other people reported issues for it:

https://github.com/NixOS/nixpkgs/issues/36235
https://github.com/NixOS/nixpkgs/issues/31714
https://github.com/NixOS/nixpkgs/pull/34724
https://github.com/NixOS/nixpkgs/pull/33673#issuecomment-377469632

While that doesnt proof that theres sufficient interest to sustain it, I think its worth to give it a shot. I don't think maintaining an overlay would work as well.

@FRidh that claim about «more than one» is an idealistic dream which doesn't correspond to what actually happens with TeXLive, LibreOffice etc. Using it as a gatekeeping criteria seems a bit too much to me.
@timokau has already managed to create a working Sage package, but Sage bundles everything and that creates quite a bit of pain so the goal is to make it behave by using system-wide packages for dependencies. «Keep it local» in this case will end up meaning a sage/default.nix-local package definition.
Also, I think that judging whether the software — that still works — is useful by whether it keeps changing all the time is just making the bit rot problem worse.
I do agree with your policy of marking things as broken if problems arise and are not fixed quickly by somebody, but in case of a Sage dependency it is quite likely that once there are actual problems there will be either a patch or a Sage version not needing the package (in this case dropping sounds completely natural)

that claim about «more than one» is an idealistic dream which doesn't correspond to what actually happens with TeXLive, LibreOffice etc. Using it as a gatekeeping criteria seems a bit too much to me.

I don't know if you can compare them. With the information that's been given to me in these PR's we deal here with apparently quite some outdated and unmaintained dependencies.
I'm not against having sage or sagenb in. Before going on, which one are we now really talking about Since I've seen PR's related to sagenb, and then the discussion shifted to sage.

has already managed to create a working Sage package

Great!

I do agree with your policy of marking things as broken if problems arise and are not fixed quickly by somebody, but in case of a Sage dependency it is quite likely that once there are actual problems there will be either a patch or a Sage version not needing the package (in this case dropping sounds completely natural)

See the linked thread https://github.com/sagemath/sagenb/issues/440. They're not moving that fast.

I hope «more than one» was about the Sage undertaking as a whole. This PR is just about one of the thousand paper cuts about building Sage (even if the build works somehow).
Sage is not moving fast about doing something about unmaintained but still working dependencies, and I support lack of hurry in this respect. I mean, there should be some incentive to use code that has a chance to work for more than three months after release. I hope that if a dependency becomes actually hard to build after some transitive dependency update, Sage will have either a patch or a way to do without this dependency faster than if the dependency works and there is no reason to expect it to break.


Peer Review CSV spec test (+ upgrades)

NOTE: This also contains the pull request from here: https://github.com/MarkUsProject/Markus/pull/2649
The only file that needs to be focused on is peer_reviews_controller_spec.rb
Overall I think it's a bit messy, but I figured I could get some comments on it and see what is the best way to clean this up (if possible)

@WChrisK the test is a little on the long side, but I do like that it's checking that the upload/download formats are inverses of each other. I'll merge this in after you've addressed my comment.


Updated RDS Doc Migration entrypoint to use local snapshot

Description

Updated the RfsMigrateDocuments.java entrypoint to accept a local snapshot
Added a new, local test that runs the full RfsMigrateDocuments.java against local target/source in Docker containers and a local snapshot.  The test asserts that the process exits with the expected status code.

Issues Resolved

https://opensearch.atlassian.net/browse/MIGRATIONS-1807
https://opensearch.atlassian.net/browse/MIGRATIONS-1809

Testing
Ran the new tests using gradle DocumentsFromSnapshotMigration:slowTest
Check List

[X] New functionality includes testing

[X] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented
[X] Commits are signed per the DCO using --signoff

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

In the PR description, say "main", or even "program" rather than entrypoint.  I was wondering what you were doing with docker when I read the description.

Minor comments. Please file a new task to add a test to make sure that process death for expiration works as expected.

Done - https://opensearch.atlassian.net/browse/MIGRATIONS-1823


IZPACK-1340: Use Assume.assumeTrue() to skip test execution on Window…

…s without administrative rights.

Running as Administrator:
Results :

Tests run: 14, Failures: 0, Errors: 0, Skipped: 0

Running not as admin:
Results :

Tests run: 14, Failures: 0, Errors: 0, Skipped: 4

Thanks for contributing.


fix non-Go code set up signal handler without SA_ONSTACK flag

I found a workaround for issue: https://github.com/dveselov/go-libreofficekit/issues/13
By processing the signal on the C side, and adding a corresponding flag to the signal that comes from the cgo side.

Repo seems unmaintained, but I appreciate that someone cares. I copied the files locally. Do I have to call init() as it a private function?

You don't need to call "init" yourself, as soon as you import the package, the function will be called.
func init() {
   C.libre_init()
}

You just use the package as in the examples.


wow!!! beauty ui ,  can use with nuxt3?  😻

Thanks. I will try Nuxt 3 today to see if I can make it work.

@chanthavong Use this setup for Nuxt 3


Couldn't interpret '/Users/Paul/Desktop/newDirectory' as a hosted git url

Description
Hello,
I am new to brunch, and I've just installed it.
I am trying to execute :
brunch new brunch/with-es6 -s /Users/Paul/Desktop/newDirectory
But I get this error :
Couldn't interpret '/Users/Paul/Desktop/newDirectory' as a hosted git url
Environment

Brunch: 2.5.0
Node: 5.9.0
NPM: 3.7.3
Operating system: Mac OSX, El Capitan (10.11.3)

Any idea on what's causing the issue?

Solved it :
I was typing in a wrong format, it should be :
brunch new /Users/Paul/Desktop/Neww -s brunch/with-es6
Hope this helps!


Adobe overriding quicklook

Similar to the Pixelmator issue described in the readme, Adobe is now overriding the wonderful svg preview with its own icon.  How would I go about finding a similar fix?  Is there a way to give qlImageSize a higher priority?

Just do the same thing than for Pixelmator I guess.


Image Viewer distorts imported image after saving.

When importing a png into an IMZ, the image viewer properly shows the imported image. After saving, the image still appears as it should. When reopening the IMZ or patching the image into the game, however, the image is photo negative and has substantial quality loss. It also drastically increases in file size.
After importing:

After saving (and reopening the IMZ):

Size difference:

While not exactly a duplicate of #223, the issues are both relevant to each other. This should be fixed quickly if possible. For now however, the solution is to generate IMGZ files with the command line tool OpenKh.Command.ImgTool instead. I have used it reliably with zero issues for a while now.

A workaround I've found is to select "Convert to 8-bpp image" before saving. I think the issue may be the rgba8888 format the png is imported as

Thanks for using and reporting!
Currently the file size increase problem is brought by technical difficulty.
GDI+ component loads 4bpp/8bpp png image as 32bpp (Rgba8888).
8bpp → 32bpp : file size x4 times.

A workaround I've found is to select "Convert to 8-bpp image" before saving. I think the issue may be the rgba8888 format the png is imported as

Yes, this workaround is valid way for now (although this behavior is undocumented).
I will check channel swapping problem of Blue and Red, as you have reported, and it is bug.

I will check channel swapping problem of Blue and Red, as you have reported, and it is bug.

Sorry I misunderstood. This channel swapping issue seems to be fixed by merging #254

@DiamondHuntet can you check if the issue has been fixed in the latest release? If yes, I plan to close this issue. If not, we still need to look at it.

Color swap fixed as of #254, swizzling/quality loss fixed as of #329. I'll go ahead and close this and will document the 8bpp -> 32bpp conversion workaround in our tool documentation so any future users do not experience that issue as well.


Addon fails to store persistent data in host's filesystem

zigbee2mqtt add-on version (if edge, please report commit hash): 1.17 but applies to all
Operating environment (HassOS, Virtual Machine, Device/platform running Home Assistant):

docker supervised install on Pi3 running raspbian
Description of problem:
Persistent data (e.g. database.db) is stored in /usr/share/zigbee2mqtt but this folder is not mapped from a host folder. Most other addons store persistent data in a folder that is accessible outside of the container and is thus accessible even if the container is not running.

You can switch to https://github.com/zigbee2mqtt/hassio-zigbee2mqtt that uses /config by default however you can switch where z2m stores the config from the add-on UI. it's up to you to decide where you want to store data about it.

Thanks, @ciotlosm. I'll switch over to that.


[WFLY-5814] Upgrade Weld to 2.3.2.Final

@mkouba can you submit a PR for jbossas/jboss-eap7 with a JBEAP7 issue? thanks


Resolved save Post Bug

Related Issue
Issue is that the saved movies deleted itself while going to different routes, also they are very slow while fetching the saved movies.

Closes: #294
Description of Changes
I have firstly fixed the itself deleting bug on moving to different routes and made it fast to fetch save posts by storing them in local cache.
For example:

Added fast fetching of saved movies
Updated SavedMovies and favourite to fix the bug

Checklist:

[x] My code adheres to the established style guidelines of this project.
[x] I have conducted a self-review of my code.
[x] I have included comments in areas that may be difficult to understand.
[x] I have made corresponding updates to the project documentation.
[x] My changes have not introduced any new warnings.

https://github.com/sourabhsikarwar/Scene-Movie-Platform/assets/114489502/26149277-ef17-4613-9351-bfc44feec6d4

Recently, the page has been updated with some features and UI changes. Kindly go through that and also resolve the merge conflicts @Deepanshu0703

Recently, the page has been updated with some features and UI changes. Kindly go through that and also resolve the merge conflicts @Deepanshu0703

Resolved

please review the resolved code @sourabhsikarwar

Please check the updated code @sourabhsikarwar

@sourabhsikarwar


feat(api): add proof-of-inclusion feature

Linked Issues

Closes #849

Notes to Reviewer

As a reviewer, please pay particular attention to the following areas when reviewing this PR and tick the above boxes after you have completed the steps.
API Changes

[ ] Test the API endpoints which were added/changed.
[ ] Review the API documentation changes and confirm that it matches the actual functionality.

Test cases

[x] Review and run tests that were added/changed.
[x] Suggest places that may benefit from test cases.

INX Changes

[x] Run chronicle using an INX connection.

Database Changes

[ ] Review database queries for correctness/conciseness.
[ ] Ensure queries are supported by indexes if needed.
[x] Check for breaking changes in the data model and matching (conventional) commit message prefix.

Additionally

[ ] Run the inx-poi Hornet plugin and compare the generated proofs of inclusion. It should be possible to validate a PoI with Chronicle that was generated by inx-poi and vice versa.

Two differences I noticed when comparing inx-poi and our impl:

We include metadata and stateMetadata that does not exist in their response when it is 0x.
We hash the value and they do not (it is just the requested block ID).

I think the 1st is fine, but we probably should change the 2nd.


Inconsistent support of Waila and TOP

The Booster supports Waila but not TOP. Also, the following blocks support TOP but not Waila:

[x] Ender Monitor
[ ] Endergenic Generator
[ ] Timer
[ ] Power Relay
[x] Locator
[x] Scanner
[ ] Sequencer

The endergenic generator, timer, power relay, and sequencer don't currently sync the necessary data to  the client for Waila to display.

Make sure this syncing only happens as needed (i.e. when WAILA asks for it)


wasm-bindgen download URL is not always valid

🐛 Bug description
When Cargo.toml specifices a range like wasm-bindgen = "0.2", wasm-pack's attempt to fetch a prebuilt binary goes thusly:
Sep 23 09:37:48.487 WARN could not download pre-built `wasm-bindgen`: received a bad HTTP status code (404) when requesting https://github.com/rustwasm/wasm-bindgen/releases/download/0.2/wasm-bindgen-0.2-x86_64-apple-darwin.tar.gz. Falling back to `cargo install`.

wasm-bindgen, however, does releases of concrete semver versions, e.g., x.y.z. If both 0.2s are substituted with 0.2.22 (for example), it doesn't 404.
🤔 Expected Behavior
I think it'd be nice if wasm-pack, when fetching wasm-bindgen, had the following behavior: If the current project's wasm-bindgen requirement is...

specific, it attempts to fetch a prebuilt binary for that release. (This should be working today.)
a range constraint, it attempts to fetch the latest release that satisfies that constraint.

The latter may not be worth the effort (probably querying the GitHub API), so maybe "it attempts to fetch the first release satisfying that constraint", i.e., appending ".0"s as necessary until it's of the form x.y.z.
👟 Steps to reproduce
Implied above.
🌍 Your environment
Include the relevant details of your environment.
wasm-pack version: 7df088caada068f4e7f93ff37da3284ef082e95c
rustc version: rustc 1.29.0 (aa3ca1994 2018-09-11)

Might be moot under https://github.com/rustwasm/wasm-pack/pull/302, where the notion of 'desired version of wasm-bindgen' is always a concrete version.

hey @spencerwilson ! sorry you ran into this- the reason we've been waiting on release is because we need #302 to land. i'm pretty sure that #302 should fix this- and it will be similar to what you specced out, except instead of grabbing latest, it will grab whatever version of wasm-bindgen you have in Cargo.lock. wasm-bindgen requires that its version match, so snagging latest would sadly not work. thanks for filing a really detailed report- hopefully all of this will happen tomorrow and the new release should fix all your woes :)


FR: Migrate Admin SDK to google-auth-library-java

Currently the Admin SDK depends on the google-api-client library for all credential processing. But GCP at large is moving towards the new https://github.com/google/google-auth-library-java library. This library has better, cleaner APIs for parsing credential files, and integrates well with other gcloud SDKs. We should migrate the Admin SDK to use this new library.
This dependency is still at v0.6 and the API is subject to change. It might be a good idea to wait until they release v1.0.

I've done some prototyping related to this in the hkj-auth-creds branch. It seems we can get rid of a significant amount of code by doing this migration. I'm also trying to get https://github.com/google/google-auth-library-java/issues/117 as part of this, which will make things even easier for us.

This was released today: https://firebase.google.com/support/release-notes/admin/java#5.4.0


Failed to Connect to Sql Server with MultiSubnetFailover .netCore 2.0

I'm running a .NET Core App 2.0 in a container against a SqlServer 12 database with MultiSubnetFailover.
My connection string looks like this...
Data Source=..***.com;Initial Catalog=DATABASENAME;User ID=USER_ID;Password="USER_PASSWORD";Connect Timeout=60;MultiSubnetFailover=True
When i try to connect, it fails with the following error
Message="A network-related or instance-specific error occurred while establishing a connection to SQL Server. The server was not found or was not accessible. Verify that the instance name is correct and that SQL Server is configured to allow remote connections.
Same issue is documented here for 1.1
https://github.com/dotnet/corefx/issues/14978

As recently announced in the .NET Blog, focus on new SqlClient features an improvements is moving to the new Microsoft.Data.SqlClient package. For this reason, we are moving this issue to the new repo at https://github.com/dotnet/SqlClient. We will still use https://github.com/dotnet/corefx to track issues on other providers like System.Data.Odbc and System.Data.OleDB, and general ADO.NET and .NET data access issues.

Hi @ezeasharma @epoyraz
Please confirm if issue exists with Microsoft.Data.SqlClient (latest version)

confirmed this is fixed. thanks!


Eliminado código de error 2104 duplicado

Pull Request Test Coverage Report for Build 8

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 100.0%

Totals

Change from base Build 7:
0.0%

Covered Lines:
18

Relevant Lines:
18

💛  - Coveralls

Gracias!


Dojo Toolkit Usage

Hi,
I use dojo in my project. But admin panel is not working properly.
Is there a discrepancy between admin panel and dojo?
Thanks

Unfortunately we didn't test it with Dojo. Gentelella it's using jQuery only.


findstr is obsolete

following warning when running tf() :
warning: findstr is obsolete; use strfind instead

following warning when running tf() :
warning: findstr is obsolete; use strfind instead

Can you show me an example?

Line 136 of tf.m
if(length(findstr(char(polyString(1,i)), '-')) > 0)

Line 136 of tf.m
if(length(findstr(char(polyString(1,i)), '-')) > 0)

So you cannot do this?
>> G = tf([2 3], [3 5 2], 4)
G =

  scalar structure containing the fields:

    num =

       2   3

    den =

       3   5   2

    delay =  4
    tfnum =  2s + 3
    tfdash = --------------e^(-4s)
    tfden =  3s^2 + 5s + 2
    type = TF
    sampleTime = 0

>>

I am not disputing this. The issue I am reporting is that Matlab function findstr() is now obsolete:

I am not disputing this. The issue I am reporting is that Matlab function findstr() is now obsolete:

Thank you. I have fixed the working now by replacing the command. I did also update the "updatematavecontrol" command.


AssignLibraryPlugin: improve error message for invalid library name

What kind of change does this PR introduce?
improvement
Fixes #11632

Did you add tests for your changes?
yes

Does this PR introduce a breaking change?
No

What needs to be documented once your changes are merged?
Nothing

/cc @sokra

Thanks

Thanks


feat(Computable): define P, NP, and NP-complete

Revised to focus on languages that are binary strings, given the issue of non-acceptable encodings of unspecified Types

please merge master and mark as awaiting-review if it's ready for review

Working on it