Provide Authentication sample for ASP.NET Core

It would be extremely helpful if the team could provide an authentication sample using ASP.NET Core. I'm not talking CoreClr support here, and an example targeting .NET 4.6 is sufficient.
The closest example of this is the old ASP.NET Core sample in the repo around Microsoft Graph, however this example suffers from:

Being outdated and not targeting a stable version of ASP.NET Core

Intermingling an example of authenticating against the Microsoft Graph, which makes it difficult to figure out which portion of the Authentication example provided applies to Bot authentication versus Microsoft Graph authentication.
What should be covered in the sample/samples:

[ ] How to authenticate your bot using current asp.net Security middle-ware i.e. (OpenIdConnect and/or JwtBearer)
[ ] How to hand-roll an ActionFilter which accomplishes what the BotAuthentication attribute accomplishes using just HttpClient and REST calls (this would essentially provide a C# implementation of this: https://docs.botframework.com/en-us/restapi/authentication/#navtitle)

I have tried both methods myself and for reasons which are unclear to me, I did not have success, but I am by no means an expert on all the technologies involved in making the proper security handshake.
This would also provide users with guidance/an example of how to create a bot messaging endpoint in an existing ASP.NET Core project.

So I spent some time and figured out everything I need to do to get this working.
I've posted a solution here: https://github.com/DamianReeves/EchoBotForCore. There were some non-intuitive steps involved so I hope this helps someone else.

@DamianReeves based on your work and code from the Connector I created a version of the BotAuthenticationAttribute that works with System.IdentityModel.Tokens.Jwt version 5.0.0, so that I can combine it in a DotNetCore project that also uses Microsoft.IdentityModel.Protocols.OpenIdConnect.
See: https://gist.github.com/Maarten88/84d06d858429cdc0110cfb471808f068
In my project I use a private build of BotBuilder and Connector that uses project.json instead of packages.config, this attribute enables me to remove all files and references that use WebApi/Katana from the BotBuilder and Connector projects.

Adding @msft-shahins for .Net Core.

We prereleased Microsoft.Bot.Connector nuget 3.5.1-alpha which adds support for asp.net core to connector SDK. The bot authentication is implemented as a middleware: https://github.com/Microsoft/BotBuilder/blob/master/CSharp/Library/Microsoft.Bot.Connector.AspNetCore/BotAuthentication.cs#L73 and here is an example asp.net core bot: https://github.com/Microsoft/BotBuilder/tree/master/CSharp/Samples/Microsoft.Bot.Sample.AspNetCore.Echo

When I compile the sample it always fail to load the dependency :
"Microsoft.Bot.Connector.AspNetCore": "1.0.0-*", which mentioned in the project.json
https://raw.githubusercontent.com/Microsoft/BotBuilder/master/CSharp/Samples/Microsoft.Bot.Sample.AspNetCore.Echo/project.json
Because of this, not able to call the middleware at startup.js for the bot authentication. Using VS 2015, asp.net core template for .net framework with version 4.5. Any comments ?

@sowsan The problem is that the Microsoft.Bot.Connector.AspNetCore library targets netstandard1.6 which is not supported by .NET 4.5 as you can see in the documentation.
@msft-shahins What is the reason why you can't target a lower netstandard version? Because 1.6 is not supported by any version of the full .NET Framework right now.

@sowsan the sample is using the connector library as a project dependency rather than a nuget dependency. You can use the prereleased nuget instead by just referencing it in your project.json. Below is a sample project.json that consumes the connector library as nuget:
{
  "dependencies": {
    "Microsoft.NETCore.App": {
      "version": "1.0.1",
      "type": "platform"
    },
    "Microsoft.AspNetCore.Mvc": "1.1.0",
    "Microsoft.AspNetCore.Routing": "1.1.0",
    "Microsoft.AspNetCore.Server.IISIntegration": "1.1.0",
    "Microsoft.AspNetCore.Server.Kestrel": "1.1.0",
    "Microsoft.Extensions.Configuration.EnvironmentVariables": "1.1.0",
    "Microsoft.Extensions.Configuration.FileExtensions": "1.1.0",
    "Microsoft.Extensions.Configuration.Json": "1.1.0",
    "Microsoft.Extensions.Configuration.CommandLine": "1.1.0",
    "Microsoft.Extensions.Logging": "1.1.0",
    "Microsoft.Extensions.Logging.Console": "1.1.0",
    "Microsoft.Extensions.Logging.Debug": "1.1.0",
    "Microsoft.Extensions.Options.ConfigurationExtensions": "1.1.0",
    "Microsoft.Bot.Connector": {
      "include": "all",
      "version": "<IP_ADDRESS>-alpha"
    },
    "Microsoft.AspNetCore.StaticFiles": "1.0.0"
  },

  "tools": {
    "Microsoft.AspNetCore.Server.IISIntegration.Tools": "1.1.0-preview4-final"
  },

  "frameworks": {
    "netcoreapp1.0": {
      "imports": [
        "dotnet5.6",
        "portable-net45+win8"
      ]
    }
  },

  "buildOptions": {
    "emitEntryPoint": true,
    "preserveCompilationContext": true
  },

  "runtimeOptions": {
    "configProperties": {
      "System.GC.Server": true
    }
  },

  "publishOptions": {
    "include": [
      "wwwroot",
      "**/*.cshtml",
      "appsettings.json",
      "web.config"
    ]
  },

  "scripts": {
    "postpublish": [ "dotnet publish-iis --publish-folder %publish:OutputPath% --framework %publish:FullTargetFramework%" ]
  },

  "tooling": {
    "defaultNamespace": "testbot"
  },
}

@berhir the asp.net core version of the connector library is dependent on "Microsoft.AspNetCore.Mvc.Core": "1.0.1" which requires net.standard 1.6. The library still have support for full .net framework. If you look at the nuget package it has assemblies for both net.standard 1.6 and .net 4.5.

@msft-shahins I checked again but I can't find a way to get the asp.net core version running on .net 4.5. The assembly that targets .net 4.5 is for Web Api and not for asp.net core.
From what I see you support asp.net core only on .net core. But we use asp.net core with .net 4.5. I guess you will have to create a second nuget package for the asp.net core version. The existing nuget package is for .net 4.5 and the legacy web api. The second nuget package should contain assemblies for asp.net core targeting both .net 4.5 and .net core.

because of some of our dependencies we are targeting net standard 1.6: https://github.com/Microsoft/BotBuilder/blob/master/CSharp/Library/Microsoft.Bot.Connector.AspNetCore/project.json#L18
and I believe that currently it is not possible to run it on top of net framework: https://docs.microsoft.com/en-us/dotnet/articles/standard/library
We plan to revisit our .net core support for connector once net standard 2.0 is released.

Is there any hope of .net core support now that net standard 2.0 is out?
Thank you.

#572  is where that conversation is taking place @EPinci feel free to join the conversation on that issue.


Making this compatible with Github Actions + some custom actions

Description
Hi @simonbrowndotje!
First off, I have to say – I'm genuinely impressed by your work! The C4 Architecture Model, the DSL, Structurizr... you've created some amazing tools that have been incredibly beneficial in my professional environment. I just wanted to start by saying a huge thanks for your commitment to developing solutions that truly makes life easier for us humans.
I've been actively promoting the use of the C4 model and Structurizr within my company. I've encouraged our engineering team to embrace the C4 model for documenting our software, coupled with the documentation-as-code approach (v2.0). While implementing Structurizr, however, I noticed a disconnect in integration that was causing some friction in our developer experience (DX). So, I dedicated the last day to bridging this gap, and I'm excited to share my insights with you and introduce a toolkit I developed to smooth out this process.
Observation
Our team has been using Structurizr, but we've run into a snag. Both our Product Managers and Engineers have found the diagrams visualisations challenging to generate, leading to documentation that quickly becomes outdated. This issue has led the team to seek workarounds, including reverting to static diagram tools, straying away from the documentation-as-code principle.
Solution
To tackle this issue head-on, I created a series of Github Actions. These actions automatically generate rendered images and attach them as comments in pull requests. The images get committed to the repository, allowing any accompanying documentation to reference these files easily. Developers can also manually compare diagrams, and I'm planning to enhance these actions to offer 'before/after' comparisons.
I did encounter some challenges with the structurizr/cli Docker image you provide. The Dockerfile's configuration seemed incompatible with Github Actions and lacked some tools needed for exporting images with PlantUML. Specifically, I noticed:

The WORKDIR setting gets overridden during the Github setup, rendering the CLI unusable in Github Actions.
The ENTRYPOINT setting, as a relative reference, also conflicts with Github Actions' restrictions.
The Docker image includes transient data that isn't necessary for the final published image - data storage and transfer have a cost.

To turn these issues into a solution, I've published a new Docker image with the necessary adjustments. This image functions just like yours but includes PlantUML, is compatible with Github Actions, and is smaller in size. Could you take a look at the changes I've made and let me know if they align with your vision? Your input could help us collaborate to refine your existing Dockerfile, making these improvements available to a wider audience. Integrating these changes could render my Docker image unnecessary and bring enhanced resilience to everyone needing this integration (at the moment, I manually update the image when you publish a new release for the CLI). What are your thoughts on this?
The approach would be:

Introduce a new workflow in this repository, to be triggered on new release published.
Modify the Dockerfile to build with the said release file.
Introduce a new workflow in this repository, to publish the newly build Docker image.

Resources:

Demo repository and Meaningful demo PR
Action to generate images with code and demo
Action to comment on PRs with code and demo
My Docker image and code

Looking forward to your thoughts!
Steps to reproduce

Create a Github workflow file referencing the Docker image structurirz/cli:latest, followed by a bash script to execute the export command.
-> structurizr.sh not found

Screenshot
No response
Code sample
No response
Configuration
No response
Severity
Major
Priority
I'm willing to fix this myself and raise a PR (please confirm approach first)
More information
❤

Thanks for the kind words! Adding PlantUML to the Docker image defined in this repo will likely result in future issues being opened along the lines of, "we don't use PlantUML, can you add Mermaid support too?" ... and I'd rather avoid that.
So, from my perspective, my preference is to keep the CLI Docker image as simple as possible (I will modify theENTRYPOINT though), and allow people to build their own Docker images if needed ... that's the beauty of open source after all. I've added a link to your repo in the docs though ... thank you! 👍


Describe the rolling distribution in the Bloom tutorial for ROS 2.

I tried to strike a balance between providing context for the information about the Rolling distribution and keeping the tutorial focused on the process of creating a release and not deviating too far into explaining the ROS distribution concept.
I'd greatly appreciate feedback from @tfoote and @maryaB-osr on suggestions for improvement or good resources to link from here.
Depends on #777 and #779 for linked content.

In discussion with @tfoote and @maryaB-osr we've actually opted to drop all references to distribution (rolling or otherwise) in this tutorial. The description of the Rolling distribution in #779 mentions that releasing packages into the Rolling distribution follows the same procedures as any other ROS 2 distribution and links to this tutorial.


Schedule Now

Currently adding a job like :
    schedule.every(5).minutes.do(my_job)

will run the job after 5 min from now. add a func which lets the user to specify if the job should be run at the time of adding it. Example :
    schedule.every(5).minutes.do(my_job).now(True)

Does schedule library support this?

Thanks for creating this issue! My thinking here is that we don't need to introduce a new schedule feature for that – we can just use a Python function call:
schedule.every(5).minutes.do(my_job)
my_job()

Or alternatively, if we need to use this pattern several times we can write a simple helper function:
def schedule_and_run_now(jobfunc):
    schedule.every(5).minutes.do(jobfunc)
    jobfunc()

schedule_and_run_now(my_job1)
schedule_and_run_now(my_job2)
schedule_and_run_now(my_job3)

actually I was looking through some examples and I saw .run() method.
schedule.every(5).minutes.do(jobfunc).run()
This seems to be working.


fluidd: 1.20.1 -> 1.21.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/cadriel/fluidd/releases.
meta.description for fluidd is: "Klipper web interface"
meta.homepage for fluidd is: "https://docs.fluidd.xyz"
meta.changelog for fluidd is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.21.0 with grep in /nix/store/15jicvjkcv9vf5p0qh2wv1fvbyi8r4n4-fluidd-1.21.0
found 1.21.0 in filename of file in /nix/store/15jicvjkcv9vf5p0qh2wv1fvbyi8r4n4-fluidd-1.21.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
fluidd

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/15jicvjkcv9vf5p0qh2wv1fvbyi8r4n4-fluidd-1.21.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A fluidd https://github.com/r-ryantm/nixpkgs/archive/22b4e589332a492f8d4bee799f2e0a5b727042f4.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/15jicvjkcv9vf5p0qh2wv1fvbyi8r4n4-fluidd-1.21.0
ls -la /nix/store/15jicvjkcv9vf5p0qh2wv1fvbyi8r4n4-fluidd-1.21.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

fluidd

Maintainer pings
cc @zhaofengli for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/cadriel/fluidd/releases.
meta.description for fluidd is: "Klipper web interface"
meta.homepage for fluidd is: "https://docs.fluidd.xyz"
meta.changelog for fluidd is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.21.1 with grep in /nix/store/l6bqs48id5bpna79mh7s4q5aq0azp9nh-fluidd-1.21.1
found 1.21.1 in filename of file in /nix/store/l6bqs48id5bpna79mh7s4q5aq0azp9nh-fluidd-1.21.1

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
fluidd

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/l6bqs48id5bpna79mh7s4q5aq0azp9nh-fluidd-1.21.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A fluidd https://github.com/r-ryantm/nixpkgs/archive/f0927996e7a6f954ce1488ac543df1839764685a.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/l6bqs48id5bpna79mh7s4q5aq0azp9nh-fluidd-1.21.1
ls -la /nix/store/l6bqs48id5bpna79mh7s4q5aq0azp9nh-fluidd-1.21.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

fluidd

Maintainer pings
cc @zhaofengli for testing.


No source distribution provided on PyPI

@rpkilby If a user wants to cache dependencies for offline install and use source downloads, this fails for djangorestframework-guardian
$ pip download --no-binary=:all: djangorestframework-guardian
Collecting djangorestframework-guardian
  ERROR: Could not find a version that satisfies the requirement djangorestframework-guardian (from versions: none)
ERROR: No matching distribution found for djangorestframework-guardian

You can see that only the .whl is uploaded here: https://pypi.org/project/djangorestframework-guardian/0.3.0/#files
During normal installs, pip will default to using .whl, so this will not slow down the normal use case, but will enable users to pull down source distributions if they wish.
Could .tar.gz source archive also be added for 0.3.0 (and future versions)?
I think the command you can use to generate it is:
$ python setup.py sdist --formats=gztar

Can do - I just need to make sure the sdist is setup correctly (tests/readme included in the sdist, but not installed with the package).

@rpkilby Pinging about this issue to see if it can be resolved. Thanks.

Hi @johnthagen. Finally got around to this. Basically, I wanted to ensure that the sdist included the test files, but that they weren't installed to the environment when installing the package.
sdist for v0.3.0 is up on pypi.


how to set only image in listView

i cann't make only imageview in listView ..please help me...

@narawatls It's not a project issue. Please close it


Restricting draggable position.

I want to drag a div in another div. I mean it should be draggable in side that div only.
How can I achieve it?
Thanks in advance.

$('#draggable3').draggable({
containment: "parent"
});
worked

That's a question for draggable, not rotatable.

On Jul 5, 2016, at 03:16, HarshaVardhanPVK<EMAIL_ADDRESS>wrote:
I want to drag a div in another div. I mean it should be draggable in side that div only.
How can I achieve it?
Thanks in advance.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


chore: fixed broken URLs

The following PR fixes some broken URLs in the GeometryUtils README file. Now the links are pointing out to the correct URL.

Thank you for correcting these!


cotrol stacked borrows consistency check with its own feature flag

Fixes https://github.com/rust-lang/miri/issues/3431

@bors r+

:pushpin: Commit 4a055befbcbd460610944c53b61f1f5af56e358b has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit 4a055befbcbd460610944c53b61f1f5af56e358b with merge<PHONE_NUMBER>2358d05139c18e8c78852893c81075...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing<PHONE_NUMBER>2358d05139c18e8c78852893c81075 to master...


Ultimate Direction

Obviously the Cayley project doesn't have the resources and 'star' power that Neo4J has, atleast at the moment.
I was wondering what the ultimate direction was for Cayley?
Is it meant to be a powerful substitute to Neo4J for super serious projects and operate in a production capacity?
Will it be built to scale and operate in large scale projects - not necessarily to facebook's level - but something slightly less (i.e. the kind of stuff Neo4J is being used for)?
I'm just asking because I wondering whether I should invest learning Neo4J or Cayley.
I think a lot of people new to graph databases would also like to know.

pjebs, love the question!  But I think it fits better on our discussion forum: https://discourse.cayley.io/ under Technical Discussions and maybe an entry should go on the FAQ Thread.
If you want to have a longer conversation we should have it over there, but briefly I will answer your top level questions.  Cayley intends to be a performant, best in class graph database layer supporting all scales (using correct backends).  Cayley is already used in business critical places.  As for the bolder long-term vision, I will leave those pronouncements to @barakmich.

@robertmeta Is there any information on what the different advantages/disadvantages are of the different backends? particularly speed.
Also are you familiar with the way Neo4J works?
If not, there is a small image near the top of this page which shows conceptually the difference in the way data can be handled: http://blog.thefrontiergroup.com.au/2014/10/look-cayley/
My question is, is their an artificial way to store and extract data similar to the way Neo4J does it. Obviously Cayley doesn't work like this as per image on Right Hand side. But would storing data as a json string work? Would we lose ability to query if we even attempt to store data like Neo4J?

Lets keep the conversation on discourse.


The Spam Reporting example uses Office.onReady, which will not work on Classic Outlook

URL of sample
https://github.com/OfficeDev/Office-Add-in-samples/tree/main/Samples/outlook-spam-reporting
Describe the bug
This sample wraps the Office.actions.associate(...) call, whichis only needed on classic Outlook for windows, in an Office.onReady(...) call. That onReady call will never be invoked on classic Outlook for this style of add-in, per MS's documentation: https://learn.microsoft.com/en-us/office/dev/add-ins/outlook/spam-reporting

As you develop a spam-reporting add-in that will run in Outlook on Windows, keep the following in mind.

Imports aren't currently supported in the JavaScript file that contains the code to handle the spam-reporting event.
Code included in the Office.onReady() and Office.initialize functions won't run. You must add any add-in startup logic, such as checking the user's Outlook version, to your event handlers instead.

To Reproduce
Steps to reproduce the behavior:

Install the sample, and try to use it to report a message on classic Outlook for Windows.
The add-in gets stuck showing "Pending..." after you try to submit your report.

Expected behavior
The report doesn't get stuck, and runs to completion correctly.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment

Platform [PC desktop, Mac, iOS, Office Online]: PC Desktop
Host [Excel, Word, PowerPoint, etc.]: Outlook
Office version number: latest
Operating System: Windows 11
Browser (if using Office Online): N/A

Additional context
N/A

Thanks for reporting this @mikelly-psat.
@samantharamon, could you please investigate?

Hi @mikelly-psat,
Apologies for the delayed response. I'm currently updating the sample to resolve this issue. I'm creating a separate JS file containing code that applies to classic Outlook on Windows. I should have a fix that you can test out next week.

@mikelly-psat, changes were made in PR #808. If you continue to experience issues, please reopen this issue or create a new one. Thanks.


Value box on horizontal linear gauge

Hi,
It would be nice to have the value box on horizontal linear gauges, too.

Vertical linear gauge has a value box, horisontal have not.

My fault! It would be nice to have a box on horizontal linear gauges, too.

I agree, would love to have the value box on the horizontal gauges as well.  A top/bottom/left/right position, padding, or offset for it would be great as well to adjust for collisions.


Multiple typos and formatting fixes

rest of it looks good!
I'll merge it with wording fix

actually, I just ran it locally, docusaurus and jsx does not support  tags. I just replaced them with newlines and its good.


Design Panel: Fix transparent color edit

Summary
Fix transparent color edit under design panel.
To-do
N/A
User-facing changes
The color editor should persist when user change the opacity to 0%
Testing Instructions
The color editor should persist when user change the opacity to 0%

Fixes #5369

The color editor should persist when user change the opacity to 0%

Could you perhaps add before/after screenshots to the PR description?

Screenshots:
Before the fix:

After the fix:


Remove dependency on k8s.io code

Firstly, this project is really interesting, thank you, and I believe that statedb will be extremely useful as the core data store for many applications outside Cilium and Kubernetes.
However, the current dependency graph is huge and includes k8s.io/api, k8s.io/apimachinery, and k8s.io/client-go. This makes the project significantly less appealing as few non-Kubernetes projects want to pull in large sections of the huge Kubernetes codebase as a dependency, even if most of it will be eventually pruned by the linker.
This is a draft PR, and probably should not be merged, to indicate the effort required to make this code more generic:

The Kubernetes reflector is simply removed, as this is application-specific code and not relevant to statedb's core functionality. It can be moved to the application using statedb.
The single type (a generic set) from k8s.io/apimachinery/pkg/util/sets is vendored, removing the dependency on k8s.io/apimachinery.

This is a draft PR designed to encourage discussion, not to be merged.

Thanks! Agree on removing the k8s specific code so let's go for that. I'll move it into cilium/cilium or into a statedb utils repo.
I would not vendor sets.Set since we can instead just use map[T]bool or map[T]struct{} internally where needed. Also CollectSet now returns the wrong type and it's used in cilium/cilium in places that expect the sets.Set type. I think we can just drop CollectSet for now.
Could you refactor the remaining sets.Set uses to just use map[T]bool? It's mostly just test+example and NewTable so should be straightforward.

Could you refactor the remaining sets.Set uses to just use map[T]bool? It's mostly just test+example and NewTable so should be straightforward.

Done.

I couldn't resist adding an extra commit to use map[T]struct{} instead of map[T]bool for internal set types. As you know, this reduces memory usage at the cost of slightly different ergonomics. However, memory usage is unlikely to be a significant factor here (all sets are currently temporary, either within a function or within a test).


Migrate from istio self-managed to Managed ASM

Remove dependency on fixed istio releases in favor of Managed ASM release channel.
Some things to keep in mind:

support for EnvoyFilters to preserve OPA addon support.

This will reduce the manual dependencies and deployment complexity.

Note to self: https://cloud.google.com/service-mesh/docs/managed/service-mesh


Update tests and CI

This also contains some formatting changes and updates the typedoc documentation.

For reference, I had to add my IEX cloud credentials to the project in CircleCI for this to pass.

@schardtbc Just checking in - I know this is a rather large PR most of it is merely formatting. One thing this does is get our tests passing again so we can rely on the CI pipeline.


init: 19=operation not supported by device

Hi Marco,
I'm on mac, I managed to download and compile the mchfuse binary, but when i try to connect from the command line:
sudo ./mchfuse -u<EMAIL_ADDRESS>-p "mypassword" "My Own Cloud Home Duo" /Users/myuser/tmpmnt/ -d
I get
19=operation not supported by device
User/pwd should be correct, because at first I put the wrong one and it was correctly telling me which one was available ... any hint?! I tried also without sudo, same outcome ...
Thanks and best,
-g

Hi Giampaolo,
it looks like mchfuse is not able to speak with the osxfuse extension.
What version of OSX you have? What version of osxfuse?
I've just realized that I do not mention in the readme that osxfuse is required to run mchfuse on OSX. You can install it using homebrew with the following command:
brew cask install osxfuse

Pfff ... I wrote quite and extensive email but Mail crashed and no draft, grrr :(
Let's do it shorter and online :)

Yes, highlight the need of osxfuse for Mac users, despite the hint in the name "mchfuse"
Indeed, upgrading to 10.15.5 (I was on the publica beta) but mainly reinstalling osxfuse did the trick. I had probably quite an old version of osxfuse lingering around, I trigged brew to reinstall it, and eventually worked ...

Thanks, IMHO you we can close it ...
Best,
-g


Worker node's resources don't show on Webportal

OpenPAI Environment:

OpenPAI version: 0.14.0
OS (e.g. from /etc/os-release): Ubuntu 18.04

The main problem:
I've faced a problem second time with worker nodes when I can't see their resources on the dashboard (Grafana) on webportal and on Virtual Clusters page (I see IP of worker node, but all resources show me as 0). However, I can see any information on "Hardware" page and I see all resources there. Also, I see it in "total nodes" not in "worker nodes" (Grafana). In addition, everything is in a good condition in k8s.
They are together had the same bug after the situation when their memory was over and they didn't have swap. After this docker restarted and containers for services were in "created" status, not in "UP".
What I tried to do:

I tried to remove and add the wrong machine in the cluster.
I tried to delete all dedicated clusters where this machine was.
I tried to restart daemon and docker service after removing this machine from the cluster.
I deleted all data in /datastorage directory.
I added swap for this wrong machine and did anything which I noticed before.
I tried to fix crashed etcd node after when I added it in the cluster using python paictl.py machine etcd-fix -p /path/to/config -l machine-list.yaml and had some wrong output:

I solved this problem for the first time only after deleting all machines and services. However, I can't do it now, because I want to save all information from "Jobs" and I don't want to clean configuration.

Hi @andrewsen13  - thanks for following, using and providing feedback to OpenPAI. Would you mind share us more info about your usage for our reference (only)?

Is the customer for your organization or your personal usage? if it is your org, would you mind share the Organization Name? We will only use it to label and cluster issues.
We had already released a pure K8S version OpenPAI 1.x, do you have any plan to switch to this new architecture?
Would you mind share your contact email for further feature discussion or OpenPAI usage feedback questionaries?

Thanks

@scarlett2018,

It's for my personal usage.
Yes, I know about it. However, I saw that some users have problems during the deployment process. That's why I want no use 0.14.0 version for this period of time. I'm planning to upgrade PAI version later.
Sure. You can check it in my bio.

Do you have a solution of my problem or any ideas how to solve it?

I've solved this problem.
Free space of ssd was so low - around of 1Gb that's why logs couldn't write there.
By the way, can I clean some data in /datastorage/hdfs? I see that this folder can be over that 1Gb.

Hi @andrewsen13 , /datastorage will store the job history and the node/job metrics. For the metrics, you can change this line.
https://github.com/microsoft/pai/blob/4ec7a1daafedcdf87ebb0fdfbf187633648fd917/src/prometheus/deploy/prometheus-deployment.yaml.template#L68
Set the retention to a small time. Then, redeploy the promethues by:
paictl service stop -n prometheus
paictl service start -n prometheus

Old data will be remove automatically.
For job history data, there is a similar config. @abuccts  Can you help to show the steps to change the  retention time for job history? And any suggestions for this issue?


Text Summarizer

Description
Make an application for text summarizer that takes input as a paragraph and returns the summary of the text.
It would be great if you can make a simple GUI for this.
Technologies and Languages

Python
Natural Language Processing

you are free to use any other language and technology as well.

Hello can you please assign it to me @bislara

Sure. Go ahead

Hey @Anustup900   any update? Do let me know if you need any help

Sure. Just explain too how you are going to solve this.

@bislara I'll be using the NLTK library and the weighted frequency method for summarization. I'll also explain the approach in the dosctrings.

Cool. Go ahead. Looking forward to see the project


audio not looping

I want to loop the whole video if the user wants it.For testing i enabled it by default
<link href="https://cdnjs.cloudflare.com/ajax/libs/video.js/7.4.1/alt/video-js-cdn.min.css" rel="stylesheet">
    <script src="https://cdnjs.cloudflare.com/ajax/libs/video.js/7.4.1/video.min.js"></script>
	<script src="https://unpkg.com/videojs-abloop/dist/videojs-abloop.min.js"></script>
	<audio id="playerPFLHdOzobjFj" class="player-width player-height video-js vjs-default-skin vjs-big-play-centered" controls  preload="none"  poster="https://devzone2.filterbypass.me/s.php?k=57lt69gvoGRjIkf2fcojXCXRTrYSc4x4jN%2FESX8%2Bhgsyelg4S2op7fYnSmdHLob1&b=4">
<source src="REMOVED" type="audio/mp4">
</audio>
	<script>
			var audio = videojs("playerPFLHdOzobjFj",{
				plugins: {
					abLoopPlugin: {
						start: 0,
						end: false,
						enabled: true,
						createButtons: true
					}
				}
			});
		</script>

However the video doesn't loop at all for me although the toggle shows loop on .I tested with chrome latest version.
Thanks for the help and the plugin

I just tested with the below and it seems to loop ok on Chrome 72.
Does the video (audio) play at all? Is there any output in the browser console to suggest what is wrong?
<link href="https://cdnjs.cloudflare.com/ajax/libs/video.js/7.4.1/alt/video-js-cdn.min.css" rel="stylesheet">
<script src="https://cdnjs.cloudflare.com/ajax/libs/video.js/7.4.1/video.min.js"></script>
<script src="https://unpkg.com/videojs-abloop/dist/videojs-abloop.min.js"></script>
<audio id="playerPFLHdOzobjFj" class="player-width player-height video-js vjs-default-skin vjs-big-play-centered" controls  preload="none"  poster="https://devzone2.filterbypass.me/s.php?k=57lt69gvoGRjIkf2fcojXCXRTrYSc4x4jN%2FESX8%2Bhgsyelg4S2op7fYnSmdHLob1&b=4">
<source src="http://techslides.com/demos/samples/sample.mp4" type="audio/mp4">
</audio>
	<script>
			var audio = videojs("playerPFLHdOzobjFj",{
				plugins: {
					abLoopPlugin: {
						start: 0,
						end: false,
						enabled: true,
						createButtons: false
					}
				}
			});
		</script>

Unable to reproduce. If it's still occurring please reopen with more details of the issue, including any errors from browser console.


Add inverse function for affine matrix3x2

Description
Adds inverse(boost::gil::matrix3x2<T> m) that returns the inverse of the given affine matrix to the numeric extension. This is useful to implement an undo of an affine transformation or to compute the coordinate of points in the original coordinate system gievn the coordinate in transformed one.
Use BOOST_TEST_WITH and defines with_tolerance rather BOOST_GIL_TEST_IS_CLOSE as discussed in boostorg/core#38
Tasklist

[x] Add test case(s)
[ ] Ensure all CI builds pass
[ ] Review and approve

No output has been received in the last 10m0s, this potentially indicates a stalled build or something wrong with the build itself.

@mloskot Before re-running all the pipelines because one single little tiny travis stalled job, is there a way, as maintainers, to trigger a re-build of a specific job? Just to save the polar bears (and more) from climate change...

@sdebionne Good thinking! Yes, there is a way and, I think, as a committer you are authorized to do it:

Go to https://travis-ci.org/github/boostorg/gil
Sign in using GitHub
Hit the gray circle next to the job to re-run it

Unfortunately I don't have the gray circle next to the job. Maybe some configuration on the travis side is required?

That is another reason I'd like to switch everything over to GitHub Actions, to streamline the CI chore as accessible across our team.

I don't much about GitHub Actions but if it makes the CI more accessible then 👍

@mloskot My bad, login out and back in fixed the permissions! Thanks for the tip.

Sorry for ridiculous delay, but I completely missed this PR

Sorry for ridiculous delay, but I completely missed this PR


add quick start to readme

I was struggling to get webpage with the styling up and running, so after a hint from @rduivenvoorde, this one works nicely on my PC

Thanks Peter!


\b used instead of \n in host.cpp

In samples/core/hosting/host.cpp, \b is used instead of \n around line 168.
	else
	{
		printf("Runtime started\b");
	}

Because of this error, this and next output are displayed into one line unintentionally.

Thanks for creating the issue and making the fix @tandasat. For small issues like this where you intend to make the fix, you can skip the issue creation step and just submit the fix in a PR!


Analyse CI uses duplicate change log extraction method

Analyze CI has an added step to verify change log and this uses a duplicated code to extract release notes and version from history.md file. This should uses a common code instead of maintaining two different copy of extract method in two repos.
Reference: scripts/devops_tasks/find_change_log.ps1

@chidozieononiwu has another PR to use new change log check. Closing this duplicate issue


Update storage.js

defaultCorestore didn't have an "else" case, and it' also assume unix directory seperators "/".
this patch would fix those problems

Thanks @heapwolf


Omniscient reader viewpoint

Source information
Morenovel
Steps to reproduce
1No text
Expected behavior
Hope it's fixed soon
Actual behavior
No text appears
LNReader version
1.1.15
Android version
Android 11
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 1.1.15.
[X] I have updated all installed extensions.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

The chapters are loading fine for me. Could be a temporary issue or the site was down.
Reopen if the issue still persists.


VSC freezes when using URL's in config file

Looks like this project has inherited bug from the upstream:
https://github.com/woodjme/vscode-junos-syntax/issues/1
i.e in case of any valid URL, VSCode crashes.
Do you  have any ideas how to fix/prevent it?

As the workaround, I've removed whole http section from ~\.vscode\extensions\jamiewoodio.junos-1.3.0\syntaxes\Junos.tmLanguage : lines from 215 to 222:
		<dict>
			<key>comment</key>
			<string>URL Addresses, either matching character URL patterns, or http(s):// followed by an IPv4 address. </string>
			<key>match</key>
			<string>\b((https?:)|(scp:)|((s?|t?)ftp:))//((([\da-zA-Z\.-]+)\.([A-Za-z\.]{2,6}))|(\b((([0-9]|[1-9][0-9]|1[0-9]{2}|2[0-4][0-9]|25[0-5])\.){3}([0-9]|[1-9][0-9]|1[0-9]{2}|2[0-4][0-9]|25[0-5]))\b))(([/\w\.&?=+-]*)*)/?</string>
			<key>name</key>
			<string>keyword.operator.junos</string>
		</dict>

And after VSCode restart I was able to open Junos config file containing URL without application crash.
I am not completely sure, what exactly is wrong with that section, but maybe regexp is incorrect? https://regex101.com/ showing me "pattern error" for PCRE(PHP) and JavaScript, but it's ok for Python&Golang. Which type of regular expressions is eligible to use in VSCode extensions?

Thanks for your crash report,
Yes, the regexp might be incorrect. Each / should be \/.
Could you help to try correct one and see if VSCode still crashes?
- \b((https?:)|(scp:)|((s?|t?)ftp:))//((([\da-zA-Z\.-]+)\.([A-Za-z\.]{2,6}))|(\b((([0-9]|[1-9][0-9]|1[0-9]{2}|2[0-4][0-9]|25[0-5])\.){3}([0-9]|[1-9][0-9]|1[0-9]{2}|2[0-4][0-9]|25[0-5]))\b))(([/\w\.&?=+-]*)*)/?
+ \b((https?:)|(scp:)|((s?|t?)ftp:))\/\/((([\da-zA-Z\.-]+)\.([A-Za-z\.]{2,6}))|(\b((([0-9]|[1-9][0-9]|1[0-9]{2}|2[0-4][0-9]|25[0-5])\.){3}([0-9]|[1-9][0-9]|1[0-9]    {2}|2[0-4][0-9]|25[0-5]))\b))(([\/\w\.&?=+-]*)*)\/?

You can test here: https://regex101.com/r/FyLGcF/1
It doesn't freeze even with incorrect one in my environment actually, so I have no idea about the root cause so far.

@codeout , I've tried with adjusted regexp, but it did not helped.
The behaviour exactly the same, in case of any URL appearing in the file, VSCode is getting stuck completely:

Thanks. Then we need to patch as you pointed out until the upstream is fixed.

Just published 0.1.4 with your workaround. Luckily, we don't see any significant change here.
( ae1 below looks storage a bit but acceptable, I think )
0.1.3

0.1.4

@codeout , i've updated  extension , all fine. Thanks a lot!


Handle mixed indentation

Previously it was assumed that a file had consistent indentation, this fixes that incorrect assumption.
Fixes #63.

Just some minor nits, after that LGTM

Merge at will


Setting WebServicesUrl in swagger config

This is required so that the api docs contain correct basepath.
Before:

After:

Fixes https://github.com/GoogleCloudPlatform/kubernetes/issues/3411

Nice. LGTM.


Access sites using hosts IP

Environment:

MacOS High Sierra
devilbox master-branch

It would be great if there is any way I can access my sites with hosts IP address since I'm using devilbox as my debug env to host my REST API and access it with my phone. It's hard to change /etc/hosts on my android phone without having root access.
Thank you.

@stepgr sorry for the long waiting time. Please do send me those rules and I try to prepare something for you.

https://github.com/cytopia/devilbox/issues/248
I am already working on this problem. Please find all relevant information in the newly created issue: https://github.com/cytopia/devilbox/issues/248
To reach a broader audience I will close this ticket and link it to the new one (which will be done with multiple related issues as well).


Docs updates for "br" usability improvements

https://github.com/apache/brooklyn-client/pull/29

Thanks, merging.


Implement Trained Eye

Resolves #33.
Most of the file changes are due to the signature of ActionMask::from_level changing to accommodate Trained Eye.
The level attribute in Recipe is possibly redundant, since I think it can be computed from recipe_level. I'm not sure whether this is actually true or not, so right now it just reads the level from the RecipeLevelTable.
The two tests in game_data/tests/test_get_game_settings.rs cover the case where Trained Eye is not applicable due to being an expert craft, and the case where Trained Eye is applicable. The Ponzecake test should cover the case where Trained Eye is not applicable due to insufficient level difference.
Issues:

Currently I don't know how to prioritize Trained Eye over Reflect. I suspect in most cases both other openers can be ignored entirely, unless the recipe requires a ridiculous amount of progress or the user has insufficent CP to finish the craft.
Trained Eye interacts awkwardly with the overflow quality display. It will show e.g. 10000 / 10000 (55535 overflow). This occurs whenever Trained Eye is used. I am unsure as to exactly why this happens.

Thank you for your contribution! This looks great!
About the issues you listed:

You don't need to hard-code prioritizing Trained Eye over Reflect (or any other quality action). The solver will figure it out on its own (states that don't use Trained Eye should be Pareto-dominated by states that use Trained Eye).
This is my fault.
The quality value in states is actually stored as missing quality. Back then. the intention was to use u16::saturating_sub to guarantee that the quality never goes over max quality. I wanted to avoid using quality = std::cmp::min(max_quality, quality)
I should have reverted this when implementing the quality overflow view. Right now, it works by using a very ugly hack: for the simulation view, settings.max_quality is set to u16::MAX.

You're welcome to change state.missing_quality to state.quality. Or, if you don\t want to do it., create another PR for the develop branch and I will take it from there :)


[Observables] Implement Observable Merging

Summary
Human Summary
This PR allows you to merge N observables into a single observable via the Merge function with takes a slice of observables and a bool as to whether the merged observable should fail fast or not. This means should the merged observable fail (call UnsubsribeAll the first time an observable its merging errors or to go on notifying using the others.
AI Summary

Summary generated by Reviewpad on 11 Jan 24 12:50 UTC
This pull request includes changes across multiple files. Here is a summary of the overall changes made:

File observable.md in the pkg/observable package:

Added a table of contents.
Updated the structure of the file.
Added a section on merging observables.

File observer.go:

Fixed a typo in the comment of the unsubscribe() function.

File interface.go in the pkg/observable package:

Added a new import statement for context.
Replaced the single-line import block for context with a multi-line import block.

File observer.go:

Renamed the import statement errorsmod to skderrors.
Added a new import block for the skderrors package.
Modified the variable ErrObserverClosed to use the imported skderrors package.
Added a new variable ErrMergeObservableMultipleFailModes.

These changes aim to improve the functionality, structure, and error handling in the codebase. Please review and provide any feedback or concerns you may have.

Issue

Fixes #280

Type of change
Select one or more:

[x] New feature, functionality or library
[ ] Bug fix
[ ] Code health or cleanup
[ ] Documentation
[ ] Other (specify)

Testing

[d] Run all unit tests: make go_develop_and_test
[ ] Run E2E tests locally: make test_e2e
[ ] Run E2E tests on DevNet: Add the devnet-test-e2e label to the PR. This is VERY expensive, only do it after all the reviews are complete.

Sanity Checklist

[ ] I have tested my changes using the available tooling
[ ] I have performed a self-review of my own code
[ ] I have commented my code, updated documentation and left TODOs throughout the codebase

Not a priority in the near future so closing out.


[bug]: FeeCap for Anchor Channels overshoots when (initial) Commitment Tx has no Remote Output

Background
I am just documenting this behaviour here because it caught my eye, I am not really sure whether we wanna fix this edge-case because its maybe not worth the hustle.
Basically when opening a Anchor Channel with the setting the max-commit-fee-rate-anchors to cap the feerate of the commitment transaction, the FeeRate for the Commitment Transaction (no push amt set) will be higher than than this fee-limit because lnd estimates the initial Commitment Tx based on (2 Anchors + 1 To_Local + 1 To_Remote) but the commitment will only have 1 Anchor + 1 To_Local in case this channel did never send an amount successfully on this channel and resolving the channel onchain.
Test/Reproduce:

Open a channel without a pushamount
With lncli listchannels you will see the Commitment Weight, Fee, and FeeRate
When calculating Weight * FeeRate you will get a weigh lower Commitment Fee than the Commitment Tx actually has.

            "commit_fee": "3140",
            "commit_weight": "772",
            "fee_per_kw": "2500",

So here the feelimit was 2500 sat/kw but the commitment tx has a feerate of 3140/772*1000 => 4000 sat/kw
Now when a state update successfully sent funds to remote the output of listchannels fits again (minus some estimation inaccuracies when calculating the weight) 2810/1116 *1000 => ~2500 sat/kw
            "commit_fee": "2810",
            "commit_weight": "1116",
            "fee_per_kw": "2500",

Setup
lnd: current master branch
For more details see here the initial weight estimate:
https://github.com/lightningnetwork/lnd/blob/master/lnwallet/reservation.go#L224

Just to add some context the spec specifies that a simplified formula for expected weight is used with local and remote amt and both anchors

Very interesting that the Spec actually recommends it that way. Not sure what your thought about aborting the opening is, but the above explained case basically happens most of the times, because the major part of channels are single funded. So essentially lets just maybe document it here and remove the bug label, because as you referenced correctly the spec recommends it even in that way.
I guess the listchannels output is a bit confusing in this case thats why I recognized this behaviour in the first place. But I don't have a good solution how to signal this edge case to the user either. Anyways just a nit but I think worth documenting.

Closing as it ss not a bug, but we have the behaviour documented now.


Use Python Version: Optionally pip install dependencies (as with Conda)

After a fair amount of foot-shooting where I would set up a multi-phase Python build and forget to install dependencies for each phase, I think that Use Python Version should optionally take a list of packages or a requirements.txt and Pip install them from the public PyPI.  More specialized functionality (like installing from a specific package index) would have to be done separately, but this would cover the common case.
The most painful case of this is when you have a phase that switches the Python version more than once (e.g. a "Linting" phase where you lint Python 2 then lint Python 3).  You would want to update dependencies after switching the version.
Switching the Python version and installing dependencies are closely coupled operations, as seen in Conda and Pipenv.  This would also make the Use Python and Conda tasks more similar to each other, which I think is a good thing.  It would be convenient and simple if they could be drop-in replacements for one another.

Considering the direction we've been going in with deprecating the Conda Environment task and the new use syntax, this improvement doesn't make as much sense anymore.


mu: 1.4.3 -> 1.4.5

Motivation for this change
A new version of mu got released a few days ago.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
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 nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build mu

@GrahamcOfBorg build mu


implement auxotrophy-based curation

As detailed in this paper, various curations are proposed to improve Yeast-GEMs performance to predict auxotrophy. The curations are very nicely included detailed here, so can easily be implemented in the yeast-GEM repo.

Not all curations are exactly implemented as proposed in the paper, as commented in the curation script.


Update stable version in repo from 0.2.4 to 0.2.5

Think about update stable version to 0.2.5
Version: stable=0.2.4 (76 days old) => latest=0.2.5 (16 days old)
Installs: stable=23 (88.46%), latest=3 (11.54%), total=26
Click to use developer portal
Click to edit
Do not close this issue manually as a new issue will be created if condition for update still exists.
Please drop a comment if any reason exists which blocks updating to version 0.2.5 at this time.
Note: This is an automatically generated message and not personally authored by bluefox!
@mcm1957 for evidence

This issue seems to be outdated.
This issue suggests to update the stable version of this adapter to 0.2.5 but the current stable version is already 0.2.5.
So this issue should be closed.
@mcm1957 for evidence


Fix pagination error on 1 page per element

Fixes a bug on paginator, on iterating on single page table paginators.

Codecov Report

Merging #56 into master will decrease coverage by 0.15%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #56      +/-   ##
==========================================
- Coverage   86.61%   86.46%   -0.16%
==========================================
  Files          16       16
  Lines         979      968      -11
==========================================
- Hits          848      837      -11
  Misses        131      131

Impacted Files
Coverage Δ

src/widgetastic_patternfly4/pagination.py
95.49% <ø> (-0.41%)
:arrow_down:

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 3cc3117...170d0e6. Read the comment docs.


Add FAQ about what ECSY is and what it is not

done https://github.com/MozillaReality/ecsy/pull/228


Fix "Unable to chat"

It is wrong that one can chat with a paper in the summary

AI chat tab should show this, too.
Currently empty:

https://github.com/JabRef/jabref/pull/11666

:laughing:


Base ?col page directs ?colnames

For some reason,
https://www.rdocumentation.org/packages/base/topics/col
incorrectly redirects to
https://www.rdocumentation.org/packages/base/versions/3.5.1/topics/row%2Bcolnames
The correct redirect target exists
https://www.rdocumentation.org/packages/base/versions/3.5.1/topics/col

This seems to redirect correctly now. Closing this issue :)


Add documentation for asdf-dojo

Add documentation on how to install and manage dojo versions using asdf-dojo.

@glihm let me know if I can take care of this one

yes i think we can have a small section about installing thru asdf in the quick start section.

@tarrencev @kariy created a PR to add dojo as part of asdf registry https://github.com/asdf-vm/asdf-plugins/pull/1003


conmon-rs: 0.6.3 -> 0.6.5

Description of changes
Update to the latest release.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

PTAL @NixOS/podman

This should also fix an issue with the most recent rust version.
Refers to #332957


[Runtime Bug]: crash when alt tabbing from the game - Call of duty 2

Describe the bug
I was taking a screenshot for mark on discord and alt tabbed from the game, i switched back from discord and the game crashed
How do you reproduce the bug?
alt tab from cod2 and switch back. it's not 100% but it's quick to reproduce and will happen after 2 minutes of alt tabbing
What is the expected behavior?
The runtime should...not crash when alt tabbing
Version
662
Logs
NvRemixBridge.log
d3d9.log
CoD2MP_s - playershadow_d3d9.log
Crash dumps
NvRemixBridge.exe_20241008_191735.dmp
Media

REMIX-3611 for tracking


[Documentation] Need more guidance on using environment variables in Dart

String.fromEnvironment()
fromEnvironment() functions were added recently. I want more guidance on what are the ways of setting these environment variables.
One way that I know of is --dart-define.
Is there a way to set these variables in something like a .env file equivalent of how things are in JS?

https://pub.dev/packages/flutter_dotenv

/cc @sfshaza2

These are not in any way related to system environment variables

/cc @lrhn

Since we have not gotten anywhere so far, I looked up the language specification v2.2.
Here is the only reference to fromEnvironment I found:

The compilation environment is provided by the platform. It maps some string keys to string values and can be accessed programmatically using the const String.fromEnvironment constructor. Tools may choose to only make some parts of the compilation environment available for choosing configuration URIs.

Can we now have an explanation of what compilation environment means here, and what can a developer do to affect this environment?
/cc @mit-mit

The compiler environment is a mapping from some string keys to string values which is available to the Dart compiler during compilation. Some mappings are provided by the compiler itself, those starting with dart.. The only ones that are consistently defined are dart.library.X being true when X is a library name where the platform name dart:X is available to the current compilation. Those environment mappings are used for conditional imports.
You can define other entries if a compiler provides a way to do so. The current compilers allow a -Dfoo=bar flag to add entries to the compiler environment, but you should check your tool documentation. There is no formal requirement to have such a feature, but the current Dart compilers (and analyzer) agree on -D. Some of the tools might recognize, or provide, other mappings as well, but that's up to the individual tool.


not available

Issue Id: ffc5c5e6-9404-acca-2e75-4068d1fda0bcVersions - 1.25.1Stack Error: not available/vs/workbench/services/preferences/common/preferencesModels.ts#518:97 (update)/vs/workbench/parts/extensions/electron-browser/extensionsActions.ts#691:74/vs/base/browser/ui/list/listWidget.ts#890:30 (domNode)/vs/base/browser/ui/list/listWidget.ts#292:12 (_styleElement)/vs/base/browser/ui/breadcrumbs/breadcrumbsWidget.ts#100:7 (scrollLeft)/vs/base/browser/ui/scrollbar/scrollableElement.ts#506:18 (_mouseIsOver)/vs/base/browser/ui/scrollbar/scrollableElement.ts#443:7 (this)/vs/base/browser/ui/list/listView.ts#168:42 (onPointer)/vs/base/browser/ui/list/listWidget.ts#509:9 (event)/vs/base/common/event.ts#277:16 (open)/vs/base/browser/ui/list/listWidget.ts#504:14 (Object)/vs/base/browser/ui/list/listWidget.ts#866:1/vs/workbench/electron-browser/shell.ts#139:1 (query)/vs/platform/extensionManagement/node/extensionGalleryService.ts#397:12/vs/base/common/winjs.base.js#10:0    at Object.s.__generator.a.label [as next] (ou

Here is another similar callstack
Error: not available/vs/workbench/parts/extensions/node/extensionsWorkbenchService.ts#215:49 (getManifest)/vs/workbench/parts/extensions/electron-browser/extensionsList.ts#191:12 (renderElement)/vs/base/browser/ui/list/listPaging.ts#43:24 (renderElement)/vs/base/browser/ui/list/listWidget.ts#816:12 (renderElement)/vs/base/browser/ui/list/listView.ts#301:11 (insertItemInDOM)/vs/base/browser/ui/list/listView.ts#195:9 (_splice)/vs/base/browser/ui/list/listView.ts#137:15 (splice)/vs/base/browser/ui/list/listWidget.ts#36:14 (splice)/vs/base/browser/ui/list/listWidget.ts#946:56 (fn)/vs/base/common/event.ts#403:2 (bufferEvents)/vs/base/browser/ui/list/listWidget.ts#946:21 (splice)/vs/base/browser/ui/list/listPaging.ts#126:12 (model)/vs/workbench/parts/extensions/electron-browser/extensionsViews.ts#550:21 (setModel)/vs/workbench/parts/extensions/electron-browser/extensionsViews.ts#112:7/vs/base/common/winjs.base.js#10:0    at Object.s.__generator.a.label [as next] (ou

Hit this wile testing. Reliable repo for me:

Open extensions view and search Forcecode

Looks like the ForceCode extension was not packaged using our vsce tool and thus the missing manifest.
We introduced the checking of manifest in extensionsList.ts in the last milestone, and so when searching for ForceCode, the error is fired.
Fixed it by not bubbling up the error when trying to fetch the manifest from the extensions view.
This error can still occur (with a different call stack of course) when the user clicks on such extension. This is because the extension editor tries to fetch the manifest as well.
Assigning to @sandy081 for further calls on how we want to deal with extensions that do not have a manifest.

To Verify

Open extensions view and search Forcecode
Make sure no errors are seen in the console


Add "no proof check" option for VC decoding

As a result, we should bypass failing tests from VC Test Suite, see https://github.com/w3c/vc-test-suite/issues/101 .

I am about to start working on this issue.

@kdimak I am considering if we should put this option behind a build tag testsuite, and then build the generator with this build tag.


expose prefix in dispatch error no-match and input-exhausted cases

It would be helpful to have the cmds https://github.com/babashka/cli/blob/05868bfb0b613d745b62abcd2340d0cf2017ea7b/src/babashka/cli.cljc#L638 also exposed in the error data for the :no-match and :input-exhausted cases, because it's helpful to know how far dispatch traversed the command tree until it encountered the error to give better error messages.
Specifically, if I have
(def cmd-table
  [{:cmds ["foo" "bar"] :fn identity}
   {:cmds ["foo" "baz"] :fn identity}])
(dispatch cmd-table ["foo" "quux"] {:error-fn identity})
;; => {:type :org.babashka/cli, :wrong-input "quux", :all-commands ("bar" "baz"), :cause :no-match, :opts {}}

if I had the prefix "foo", I could genenrate an error message that lists all valid commands with prefix "foo" (i.e. "foo bar" "foo baz").

Feel free to submit a PR, I prefer this to be called :cmds since that's what it's also called in the dispatch table?

It's called :cmds in the dispatch table, but is (and was) called :dispatch in the args to the fn called by dispatch. I don't have strong opinions though.

Nowhere in a public API is the name :dispatch exposed right? (I mean on master, not in a previous WIP PR)

It is (and was) exposed on master as part of the dispatch fn, first introduced here https://github.com/babashka/cli/commit/72edb2896c5ae2daacd9825b72e352081bc5a7fb#diff-17c5192974650f8980e4dbdaa029b04d0ee52e3f3d77afc46a1944d79448aa0bR214

Wait, looks https://github.com/babashka/cli/commit/c870e2402437e1a9ac5dc6c4f9758b91a220800e removed it again.
But https://github.com/babashka/cli/commit/712fee82e0c64a0bf32b63b5d669647edcf17f20#diff-17c5192974650f8980e4dbdaa029b04d0ee52e3f3d77afc46a1944d79448aa0bR634 reintroduced it.
The docs listed it the entire time https://github.com/babashka/cli/blob/05868bfb0b613d745b62abcd2340d0cf2017ea7b/API.md?plain=1#L77, which is what I was looking at when implementing the dispatch-tree changes.


Ambiguous parse: []*bool

This could be interpreted either as the multiplication of an empty array with bool, or as the type "slice of pointers to bool."
A workaround is to parenthesize *bool as in [](*bool), or explicitly name the type.
But long-term we will need a new syntax for at least one of:

empty arrays
slice types
pointer types

Oops, wrong repository.


left_sidebar: Introduce multi-line topics.

This PR presents rewritten grid rows for .topic-box and opens up topics for multiline presentation. Controls as well as the resolved checkmark are held to the first line of multiline topics. Topics that would ordinarily be a single line are undisturbed. All others are clamped to two lines with an ellipsis.
CZO discussion
Screenshots and screen captures:

Before
After

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

Updated the chat.zulip.org deployment.

Updated the chat.zulip.org deployment.


[IMPROVEMENT] Support 6.1 input.

Currently, 6.1 inputs are disallowed. Is it possible to allow 6.1 and either upmix to 7.1 (by using Cb as Lb and Rb) or downmix to 5.1 (by mixing Cb into Ls and Rs)? Thanks.

I don't plan to add support for any input channel layout that is not supported by DEE. If I were to support 6.1 then I would need to support other weird layouts aswell.

Fair enough. Thanks for the quick response.

It would/could be difficult to detect the actual layout for non-standard layouts. And sometimes these inputs have empty channels aswell that would need to be detected aswell. And there are a lot more subjective decisions on what the default behaviour should be when someone encodes a file like this.


[Snyk] Upgrade @apollo/client from 3.7.15 to 3.8.6

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade @apollo/client from 3.7.15 to 3.8.6.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 36 versions ahead of your current version.
The recommended version was released a month ago, on 2023-10-16.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

kultus-admin-ui branch is deployed to platta: https://kultus-admin-ui-pr312.dev.hel.ninja :rocket::rocket::rocket:


[ZEPPELIN-3306] Web e2e test fails

What is this PR for?
This is one of the flaky tests that fails with "should have a welcome message" because Z-Server did not start by the time test started executing.
What type of PR is it?
[Bug Fix]
What is the Jira issue?

ZEPPELIN-3306

How should this be tested?

CI should pass

Questions:

Does the licenses files need update? N/A
Is there breaking changes for older versions? N/A
Does this needs documentation? N/A

@zjffdu
Can you help review this? Looks like this was actually because of the latest version of chrome (65.0.3325.146) and had to update webdriver-manager's chrome version.

Thanks @prabhjyotsingh , LGTM, the selenium test still fails due to corrupted scala jars, could you clean travis cache and rebuild it again to make sure all the test pass ?

CI green https://travis-ci.org/prabhjyotsingh/zeppelin/builds/351259448.

Awesome, thanks @prabhjyotsingh

Will merge this soon.

@felixcheung it doesn't happen always but at times before could Zeppelin-server start test case starts executing, and by making spawn="false" maven waits for the command (./bin/zeppelin-daemon.sh start) to complete, and CI=true make zeppelin-daemon to wait for localhost:8080 to be active.


[BrokenLinksH2]

Docs Build status updates of commit 168a60b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

1-0-1108-44/icorewebview2.md
:white_check_mark:Succeeded
View (webview2-1.0.1108.44)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Global effort to fix broken links
@peiche-jessica
The Content & Learning team is fixing broken links on docs.microsoft.com for the rest of H2. This effort will eliminate potential accessibility, security, and usability issues. This PR includes only link fixes and does not change other content. Please review within five business days and merge, or comment in the PR with any changes you'd like to see. Thanks!


fix(TDI-40195): Case sensitivity SnowflakeInput

What is the current behavior? (You can also link to an open issue here)
Snowflake Input with condition mode crashes with exception if columns/table names are in lower case
What is the new behavior?
Option 'Convert columns and table to uppercase' is added to advanced settings, true by default.
Please check if the PR fulfills these requirements

[x] The commit message follows Talend standard
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features) ?
[ ] The code coverage on new code >75%
[ ] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[ ] Other... Please describe:

Does this PR introduce a breaking change?

[ ] Yes
[x] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:

Pull request test result: success :star:
Sonar results

Pull request test result: success :star:
Sonar results


CP: define _countof  [vcruntime.h]

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

@jianchun I just saw it. closing. Any idea on https://github.com/Microsoft/ChakraCore/issues/168 it's blocking.


Tabs and newlines are not included when pasting text from Debug Console into Word

Version: 1.30.0-insider (user setup)
Commit: a152a5c7818a13666a5879b4450eb26a4d45fbde
Date: 2018-12-07T10:34:36.706Z
Electron: 2.0.12
Chrome: 61.0.3163.100
Node.js: 8.9.3
V8: 6.1.534.41
OS: Windows_NT x64 10.0.15063
Steps to Reproduce:

Output to the Debug Console and include tabs (\t) or newlines (\n)
Select the output and copy it
Paste it into Microsoft Word. Tabs and newlines (besides the new line caused by outputting a new message) are not included.

Example:

Pasting this into Word 2016 yields:

Pasting tabs and new linesworks fine for me when I paste it into vscode.
For empty lines this currently does not work, however we can not address it since the debug console is implemented using a tree.
For pasting into word issues please file an issue / report against microsoft word.
Closing as will not fix


FLUME-3327 add useKafkaHeader option to convert flume headers as kafk…

…a ones transparently in KafkaSource,
KafkaChannel, and KafkaSink

+1 for this feature ^^

Is there a chance for it to get in if all the comments are addressed?

Is there a chance for it to get in if all the comments are addressed?

I don't have the time and resources to do a proper review here, but when the comments are addressed and @tmgstevens approves the change, then I don't see why not.

Yes, I'm happy to push this once the comments have been resolved.

This has conflicts with the patch I just applied for FLUME-3435 which adds support for the timestamp and headers to KafkaSink and KafkaSource. I did not add them to KafkaChannel as I wasn't sure if the semantics should be the same.


Fix MongoStorageAdapter.findOneAndUpdate to return Parse Object (#3053)

This PR resolves the general issue that the response to PUT request to update an object returns the result formatted as a Mongo DB object rather than a parse object. This results for example as dates objects being serialized as strings instead of objects with appropriate keys e.g.{"__type":"Date","iso":"2016-11-14T07:37:49.838Z"}.
This also generalizes to responses with objects nested in arrays such as recorded in #3053.

FYI I've tried to reproduce the failing test locally but am unable to. Any tips to reproduce / understand this? Or is this an unreliable test in general?

Looks like it's green now.


math: Nextafter, Nextafter32 clarify behavior with -0, +0

What version of Go are you using (go version)?

$ go version
go version go1.15.4 linux/amd64

Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/tmp/freedesktopCache/go-build"
GOENV="/home/nsajko/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/nsajko/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/nsajko"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/lib/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/lib/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build930141060=/tmp/go-build -gno-record-gcc-switches"

What did you do?
Some explanations are in the comments: https://play.golang.org/p/vaQBl7C6FNm
What did you expect to see?
Nextafter(x, y) should return the next representable
float64 value after x towards y, because none of the
three special cases apply here, because negative and
positive zeros and 1 and -1 are neither the same value
nor NaN.

-4.941e-324 -0 0 0
-0 -0 0 4.941e-324

What did you see instead?
...

-4.941e-324 -0 -0 4.941e-324
-4.941e-324 0 0 4.941e-324

Other notes
Relevant observation: negative and positive floating point zeros are distinct values in Go, but compare equal with the == operator. This is OK, but could be considered as the cause of the bug in the implementation.
Inconsistency with other finite float64 arguments
The inconsistency with the handling of zero arguments causes unexpected behavior when looping using the Nextafter functions.
This is not an actual issue for me, but it seems like it might be for someone who uses Nextafter: assuming a nonzero negative number x is chosen, start iterating on it like this: x = math.Nextafter32(x, 0), with the aim of breaking the loop at zero (a similar situation is observed when starting from a positive value, instead of a negative one). The loop, unexpectedly, will never exit in some cases: https://play.golang.org/p/eWs0pg8SXri
Inconsistency with C
Even though at first I thought that this is a relatively straightforward issue (because of the "next representable float64 value" explicit wording), this actually seems to be a tricky issue if compared with C, and it's even possible you may want to consider this a documentation bug instead of a behavioral bug, along with a couple other options.
The C nextafter functions were supposedly the inspiration for the Go version so it might be relevant to compare to them, also, it might be desirable to be compatible with them if you're appropriating their names already.
Both C17 and the current C draft have this to say about nextafter:
Description
The nextafter functions determine the next representable value, in the type of the function, after x
in the direction of y, where x and y are first converted to the type of the function.244) The nextafter
functions return y if x equals y. A range error may occur if the magnitude of x is the largest finite
value representable in the type and the result is infinite or not representable in the type.

Returns
The nextafter functions return the next representable value in the specified format after x in the
direction of y.

Notice the return y if x equals y explicit requirement about zeros - this is where C differs from Go currently.
But, except for that, the C implementations are actually like Go: they don't follow the spec when x is negative zero and y is a positive nonzero value, or when x is positive zero and y is a negative nonzero value. (In that case the other zero is "skipped over".)
C program for comparison (give it "0" on its stdin):
#include <math.h>
#include <stdio.h>

int
main(void) {
        int n;
        scanf("%d", &n);
        double x = n;
        double negX = -x;
        printf("%.4g %.4g %.4g %.4g\n%.4g %.4g %.4g %.4g\n",
            nextafter(negX, (double)-1), negX, nextafter(negX, x), nextafter(negX, (double)1),
            nextafter(x, (double)-1), nextafter(x, negX), x, nextafter(x, (double)1));

        return 0;
}

Outputs:
-4.941e-324 -0 0 4.941e-324
-4.941e-324 -0 0 4.941e-324

This all depends on the phrase

the next representable float64 value

I would interpret the next representable value after -0 to be 4.941e-324, not 0. 0 is not "after" -0 in my mind, as 0 == -0.
I can certainly see how that might be confusing, though.
We could add additional special cases to the docs, if you think that would help. Or was there something else you had in mind?

I see these options:

document the special cases around zero and keep the behavior the same
have a special case for x == y (return y), like in C standard and implementations, otherwise keep behavior the same. I don't know where does this matter, but it might make sense to be compatible with C.
switch to the way I, at least, interpreted the "next representable float64 value" phrase. I.e., 0 and -0 are not the same value.

I vote for 1.  I guess you could argue that this fits under the "bugs" exception, but it is not clear to me that it is even a bug. I think we'd need a stronger argument to make a backwards-incompatible change.

/cc @griesemer @rsc


Make runner internal

Currently all tool runner are public. IMHO they should always be called through aliases and since they are sealed it doesn't bring any benefit to have them public.

@pascalberger I don't agree. Sometimes (like in a Cake.Frosting application) you want to instantiate the tool yourself with a different set of dependencies (I've done this myself in some projects).

That was exactly the use cake I was thinking about too.
On May 7, 2017 4:49 PM, "Patrik"<EMAIL_ADDRESS>wrote:

@pascalberger https://github.com/pascalberger I don't agree. Sometimes
(like in a Cake.Frosting application) you want to instantiate the tool
yourself with a different set of dependencies (I've done this myself in
some projects).
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cake-build/cake/pull/1600#issuecomment-299733658, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABfthWcvvTXRPG4H0wWqvdhSZJNxkq0Vks5r3i48gaJpZM4NTMK6
.

@patriksvensson OK, makes sense


esm-style export don't get lazy loaded - Same as https://github.com/swc-project/swc/issues/4799

Describe the bug
I am having the exact same issue described in https://github.com/swc-project/swc/issues/4799
The exact same code still output the same result and the expected behavior listed in the issue is never reached.
Looking at the documentation for lazy loading (https://swc.rs/docs/configuration/modules) I am not 100% sure what is the expected behavior ? Can we force lazy loading for this code in some way if this is part of an exception ?
Input code
export {
    createP
} from './St';

Config
{
    "module": {
      // You can specify "commonjs", "es6", "amd", "umd"
      "type": "commonjs",
      "strict": false,
      "strictMode": true,
      "lazy": true,
      "noInterop": false,
      "ignoreDynamic": false
    }
}

Playground link (or link to the minimal reproduction)
https://play.swc.rs/?version=1.3.100&code=H4sIAAAAAAAAA0utKMgvKlGo5lIAguSi1MSS1ACuWoW0ovxcBXU9%2FeASdWsAVGIICyMAAAA%3D&config=H4sIAAAAAAAAA1WPSw7DMAhE9z6FxbrbbnqHHsJySeTIPwGRakW5e%2B3UTpsdzAw82JTWsLCFh95qWZtsiJHOvipcoph3VUBKRrbkssBtuMLNmoxnPKT964AYmlHaFPK9x8GnxDjiXQsuuqn8A20KmZD5GmxRE2ePV5zqSAjptR5mf6Qd2%2FB1W0hxYfglB%2FHcDo6fY1xoRbV%2FALu7P%2BkZAQAA
SWC Info output
Operating System:
    Platform: darwin
    Arch: x64
    Machine Type: x86_64
    Version: Darwin Kernel Version 23.2.0: Wed Nov 15 21:54:10 PST 2023; root:xnu-10002.61.3~2/RELEASE_X86_64
    CPU: (12 cores)
        Models: Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz

Binaries:
    Node: 20.11.0
    npm: 10.2.4
    Yarn: N/A
    pnpm: N/A

Relevant Packages:
    @swc/core: 1.4.2
    @swc/helpers: N/A
    @swc/types: 0.1.5

SWC Config:
    output: N/A
    .swcrc path: N/A

Next.js info:
    output: N/A

Expected behavior
Object.defineProperty(exports, "createP", ({
enumerable: true,
get: function get() {
return _St().createP;
}
}));
function _St() {
var data = webpack_require("../path/to/St");
_St = function _St() {
return data;
};
return data;
}
Actual behavior
Object.defineProperty(exports, "createP", ({
enumerable: true,
get: function get() {
return _st.createP;
}
}));
var st = webpack_require ("./path/to/St");
Version
1.4.2
Additional context
No response

https://github.com/swc-project/swc/issues/4864#issuecomment-1172855122

Also, see https://github.com/swc-project/swc/pull/5103/files#diff-fb5c3473351c5bd1ebf2ead2ddee9b8dabe8cd230a0905d3df32f14d01835c98


Connection Refused

I couldn't get this to work, so I tried running the Command Line Application from the project. When I do that, and then try ab I get the following:
b -n 12800 -c 128 http://localhost:8080/
This is ApacheBench, Version 2.3 <$Revision: 1663405 $>
Copyright 1996 Adam Twiss, Zeus Technology Ltd, http://www.zeustech.net/
Licensed to The Apache Software Foundation, http://www.apache.org/

Benchmarking localhost (be patient)
apr_socket_recv: Connection refused (61)

When I run with curl http://localhost:8080 it just hangs.

Hm, I'm having the same issue with the Taylor Webserver. Maybe this is because I'm on Yosemite on this system. Other web servers (Python, Ruby, etc) work fine though. I also tried various ports.

I had problems running the command line in debug mode. Changing it to release does the trick. I shared the command line application scheme with the release configuration and pushed to master. See if it works.

Can I close this issue?

Sorry for the delay! I was busy working on a different project. Yeah that works.
Thanks!


Oh hai template snippets

#fix#
CHANGELOG
Added ember-code-snippets to allow demo documentation to reference the code as the sample text.  Cleaned up the button demo as an example.

I'm going to ask @EWhite613 to make the highlighter a bit prettier when he frees up - should be able to match most of what we have in the atom/vsc/brackets highlighters

Approved


Update-Office365Anywhere error handling improvements

Hi,
I've received similar feedback from two customers regarding the script Update-Office365Anywhere around how errors are handled/returned:

Is it possible to add actual return codes to the script if it fails? At the moment it always returns 0.
Can we add better explanations why a failure occurs? For example, when you call with the script with these parameters -EnableUpdateAnywhere $false -UpdateToVersion 16.0.6741.2071 and this particular version is not available in the primary update source, it should output/log that it "didn't find the version in the update source"
Thanks!

@pretzer we are working on a more advanced solution to improve error handling for all of the scripts. Identifying milestones like you provided in your example is very helpful for us.

Logging and error logging added to scripts


Upload DBs to zenodo

Don't distribute the DBs with the repo, but upload a snapshot to zenodo.
Then, provide a shell script within /db (?) to streamline their download.

Alternative to zenodo: https://osf.io/ ?

Decouple the db folder from Nextflow's $baseDir, because as is, updating CAMITAX is not possible without dropping the local copy first and then re-downloading both, pipeline and databases.
The Makefile solution is a bit clumsy because the Nextflow project's $baseDir is usually not visible to the user, therefore either use storeDir in camitax.nf (i.e. download the databases for each genome project), or provide an additional init.nf to download the files to a user-specified target folder which the user can can then specify as a db folder for any run, manually.

https://doi.org/10.5281/zenodo.1250044


add image to preview after uploading

add uploaded image to main page avoiding manually refreshing

added at commit 844ecb6975c3f57cb6cc542b664b9d436949d2d1


WIP FGA columns filtering

This is not the final PR. The work isn't finished yet (some tests to be added) but I think to the filtering is applied where it should be.
This is just to get your feedback, on the design or on things I may have forgotten (this is my first time with arlas-server :) ).
You don't need to read it carrefully, the final merge will be splitted into multiple PRs to be easier to read.


Update Black + formatting

Closes #4816

Codecov Report

Merging #4926 (6630424) into master (07913cf) will decrease coverage by 0.01%.
The diff coverage is 98.87%.

@@            Coverage Diff             @@
##           master    #4926      +/-   ##
==========================================
- Coverage   95.53%   95.52%   -0.02%
==========================================
  Files         618      618
  Lines       65484    65478       -6
==========================================
- Hits        62562    62546      -16
- Misses       2922     2932      +10

Flag
Coverage Δ

servertests
36.10% <25.84%> (-0.01%)
:arrow_down:

unittests
95.46% <98.87%> (-0.02%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

moto/acm/responses.py
83.33% <ø> (ø)

moto/apigateway/exceptions.py
99.30% <ø> (ø)

moto/apigatewayv2/responses.py
100.00% <ø> (ø)

moto/athena/models.py
92.72% <ø> (ø)

moto/cloudformation/responses.py
96.59% <ø> (ø)

moto/cloudfront/responses.py
100.00% <ø> (ø)

moto/cognitoidentity/responses.py
100.00% <ø> (ø)

moto/cognitoidp/models.py
94.82% <ø> (ø)

moto/cognitoidp/responses.py
98.62% <ø> (ø)

moto/core/utils.py
93.75% <ø> (ø)

... and 111 more

Continue to review full report at Codecov.

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


Allow apply two sides at time

Hi coul be great add two sides at time

Yeah, have the effect on X and Y axis for a double logarithmic scale.
I'd rather keep the code simple, until the math is settled.
Current workaround

apply this extension
rotate 90°
apply this extension again,
rotate back.


[JENKINS-31111] made DownstreamTriggerParameterContext extensible

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests


Command to install dependencies

I would like to use LuaRocks to install the dependencies listed in a rockspec without starting a full build/install. This is handy in build scripts that must integrate LuaRocks with other build tools.
For example, I need to:

Install project dependencies.
Build and run tests using other tools (not LuaRocks).
Only then use LuaRocks to create a rock.

I want to avoid repeating my dependencies both in the rockspec and in the build script (where I'd have to write many "sudo luarocks install x" lines).
In Ruby I may have a "Gemfile" within a project and run the command bundle to install dependencies and prepare the environment to build the project. Is there anything similar in luarocks?
Thanks

A new flag (--only-deps ?) should be added in order to do that. It should be forwarded to the build command as well and it should stop after fulfilling the dependencies.
@hishamhm is that correct?

Also, you may give Moonbox a go.

Yes, this could be possibly added as a flag to build without adding a lot of code.

Fixed by #349


gcc's optimise for size broken

I've tried to build a pico project with optimise-for-size.
One particular function has an integer division in it.
gcc generated a call to divmod_s32s32.
So far so good.
divmod_s32s32 clobbers r3, as is allowed. https://github.com/raspberrypi/pico-sdk/blob/master/src/rp2_common/pico_divider/divider.S#L107
but gcc assumes r3 is preserved, at least at that particular call.
Does divmod_s32s32 need some more annotation re calling convention?
Only happens with optimise-for-size. With optimise-for-speed it uses a different register and r3's value doesn't matter.
gcc version is 9.2.1, pretty old... i think i followed some how-to for installing this when i first started playing with the Pico... edit: it's the version available in wsl's distro, afaict.

Weird, can you attach a repro, and/or say whether it is fixed in later GCC

will try to compile a recent gcc from source. i have no idea how to work linux package management to get a recent binary into wsl...

You can download recent arm-none-eabi-gcc versions from https://developer.arm.com/downloads/-/arm-gnu-toolchain-downloads. Just unpack them and put the executable directory on your path.

ah thanks for the link! that makes my life much easier...
12.2.Rel1 from arm's website does not show this problem. I've tried a few different combinations of global (cmdline flags) and local (function attribute) optimisation flags and the disassembler output looks fine. And the impacted function produces the correct result.
12.2 is a lot more bloaty though: bin size is 400k vs 340k for 9.2.


moment calendar month

I think it would be a great addition to add nextMonth and lastMonth to the Locale#calendar.
Extending it like this:
moment.locale('en', {
    calendar : {
        lastDay : '[Yesterday at] LT',
        sameDay : '[Today at] LT',
        nextDay : '[Tomorrow at] LT',
        lastWeek : '[last] dddd [at] LT',
        nextWeek : 'dddd [at] LT',
        lastMonth: 'foo',
        nextMonth: 'bar',
        sameElse : 'L'
    }
});

Closing in favor of the issue being tracked by #2996


px-vis-scale: allow it to read dev set extents

improve scale so it will use dev set extents instead of calculating them.
Also allow support for some extents being set.
Example:
data goes from 10 to 100, but we want the chart to start at 0. So it only needs to find a max, not a min.

Whoa, completed long ago with the addition of completeSeriesConfig


Paramount Plus blocked on adware+malware

Recently, the following was added to the adware+malware blocklist:
# [cbsaavideo.com]
<IP_ADDRESS> vod-gcs-cedexis.cbsaavideo.com

This effectively blocks all video playback on Paramount Plus.
I use this list in Adguard to try to block ads and malware, not entire video streaming services.

Thanks for this @BlohoJo.
This come to us via the Adaway list, so it originates upstream.  @jawz101 what are your thoughts? Thank you as always.

I removed it the next day

Thanks @jawz101.
@BlohoJo this will go live in the next release, due shortly.
Closing.


Delete worker jobs which are done

Fixed in https://github.com/kimmobrunfeldt/pandoc-api/commit/6bbca9945fb76e98e6ad11cafbfc57c45a49dceb


Add microsecond start span timestamp and fix usage of nanos across JVMs

Also fix dtrace->wingtips naming in Tracer log messages

LGTM


Do we have ACLs, to restrict admin operations on topics?

+1 interested in this

+1
If Kafka Manager can support ACLs ( add /update/delete ) then it would be just awesome
we don't then have to rely on CLI for ACLs , we can use Kafka Manager for every admin operation .

Suppose I grant principal "user_X" to view only topics Y and Z (through ACL from CLI), and suppose I launch cmak by passing the jaas file for SASL authentication -Djava.security.auth.login.config=/path/to/jaas/user_X.conf. Then cmak should only display the topics Y and Z. What happens is that cmak shows all the topics. However, if I launch the the kafka-topic list (with user_X SASL credentials) command from CLI, it returns only the topics Y and Z.
Any idea on how to avoid this with cmak?


Problem registering escaped characters and non escaped characters.

Two instances of this have occurred. Once, it was % becoming ". Another time it was " " becoming "" "".

Please provide short examples that demonstrates the error, this makes it easier to debug.

I think the issue is this
V1:
MsgBox `% becoming `"

V2 (Converted):
MsgBox("`% becoming ``"")

V2 (Expected):
MsgBox("`% becoming `"")

Committing fix soon

Should be fixed now in https://github.com/mmikeww/AHK-v2-script-converter/commit/d5b0b30e47fb78d94efb79a3d407d1d020a7e9d4, if not feel free to reopen


DOC: create a primer for the JSON file format

[x] I have reviewed the documentation on main here.

Location of Documentation
This could be a part of parse_json.rst
Documentation Problem
Currently there is only docstring documentation. This can be useful when calling functions and using the code but gives no insight into how to build the JSON file format expected by pype_schema.parse_json. A user has to go through sample.json and intuit the structure themselves, which is not good for adoption.
Suggested Fix
Add a section that goes through each field in the JSON and explains valid data format. Something like this: https://github.com/we3lab/flows-prep/blob/main/README.rst

The first draft of this was completed in #80