Update ApplicationLinks to use be buttons instead of NavLinks.

Summary
Convert tabs to use buttons instead of NavLinks and remove list style, as list functionality is rolled up into the side navigation menu.
Thanks for contributing to Terra.
@cerner/terra

Looks like the readme needs updated to remove the list example

Out of curiosity, will this change allow us to catch and 'correct' anchor link routing within a page? This issue is logged in terra-dev-site and I wonder if we would run into similar issues with this component as well.

Looks like the travis-ci build is glitched, might need a reset for this PR.

@JakeLaCombe I think what has been happening here (and in other PRs) is deleting the entire reference directory for a clean wdio run causes the timestamps to update on the images and check in as a different image.

Functional verification complete.  Notes a visual inconsistency with selection style on IE10 but after discussing with @dkasper-was-taken this is already being addressed in #77 .  No further issues noted.


yt-dlp runs in background and can't be ctrl+c to stop

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting a bug unrelated to a specific site
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Provide a description that is worded well enough to be understood
yt-dlp runs in background and can't be ctrl+c to stop
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
-

You will need to provide a bit more info. What operating system are you on? How did you start yt-dlp? What do you mean by running in the background?
In most terminals, Ctrl + C should cancel the current process. If not, you can always kill the yt-dlp process.

i would guess the url had an & which was interpreted by the shell
in bash you can run fg to bring it into the foreground so you can stop it
in future 'quote' the url

in future 'quote' the url

This one:

I've checked that all URLs and arguments with special characters are properly quoted or escaped


Use pattern matching instead of cond..do

Also, by using strings instead of atoms, there are less guards against mistakes. See #1174


git submodule update --init --recursive 失败

googletest当不下来啊
fatal: unable to access 'https://chromium.googlesource.com/external/googletest/': Failed to connect to chromium.googlesource.com port 443: Operation timed out fatal: clone of 'https://chromium.googlesource.com/external/googletest' into submodule path '/Users/rookie/Desktop/mj_server/3rd/sdsl-lite/external/googletest' failed Failed to clone 'external/googletest'. Retry scheduled Cloning into '/Users/rookie/Desktop/mj_server/3rd/sdsl-lite/external/googletest'... fatal: unable to access 'https://chromium.googlesource.com/external/googletest/': Failed to connect to chromium.googlesource.com port 443: Operation timed out fatal: clone of 'https://chromium.googlesource.com/external/googletest' into submodule path '/Users/rookie/Desktop/mj_server/3rd/sdsl-lite/external/googletest' failed Failed to clone 'external/googletest' a second time, aborting Failed to recurse into submodule path '3rd/sdsl-lite'

你这是要翻墙啊

@yuanfengyun 翻墙也当不下来😂😂😂😂😂

https://chromium.googlesource.com/external/googletest/ 因为这个链接已经失效了，sdsl-lite已经更新成了gitHub 上的链接https://github.com/google/googletest.git


drawCategoryMask is not a function

Have I written custom code (as opposed to using a stock example script provided in MediaPipe)
Yes
OS Platform and Distribution
Linux Mint
Mobile device if the issue happens on mobile device
No response
Browser and version if the issue happens on browser
No response
Programming Language and version
JavaScript
MediaPipe version
0.10.8
Bazel version
No response
Solution
ImageSegmentation
Android Studio, NDK, SDK versions (if issue is related to building in Android environment)
No response
Xcode & Tulsi version (if issue is related to building for iOS)
No response
Describe the actual behavior
When try to use drawCategoryMask get an error drawingUtils.drawCategoryMask is not a function
Describe the expected behaviour
It shall be possible to use drawCategoryMask function
Standalone code/steps you may have used to try to get what you need
1. Install latest package of "@mediapipe/tasks-vision": "0.10.8",
2. Perform segmentation on an image and try to draw the mask

    const result = segmenter.segmentForVideo(video, performance.now());
    if (result && result.categoryMask) {
      const confidenceMask = result.categoryMask;
      const ctx = canvas.getContext("2d") as CanvasRenderingContext2D;
      const drawingUtils = new DrawingUtils(ctx);
      drawingUtils.drawCategoryMask(confidenceMask, COLOR_MAP);
    }

Start the application and get a runtime error

Uncaught runtime errors:
×
ERROR
drawingUtils.drawCategoryMask is not a function
TypeError: drawingUtils.drawCategoryMask is not a function
    at HTMLVideoElement.predict (http://localhost:3000/main.35358a831bb5ee7a25ca.hot-update.js:246:20)

Note: The typings are there for the drawCategoryMask, but it seems, that the function is not in the production bundle.

### Other info / Complete Logs

_No response_

@schmidt-sebastian thank you!


3.1.6 Error

Hi there,
After cloning this theme I get ...
Please report a bug if this causes problems. Configuration file: /Users/svenpersonal/myAlgolia/_config.yml   Dependency Error: Yikes! It looks like you don't have algoliasearch-jekyll or one of its dependencies installed. In order to use Jekyll as currently configured, you'll need to install this gem. The full error message from Ruby is: 'cannot load such file -- algoliasearch-jekyll' If you run into trouble, you can find helpful resources at http://jekyllrb.com/help/!  jekyll 3.1.6 | Error:  algoliasearch-jekyll
but I've checked that it is in the gemfile and config file as per the plugin docs and installed dependencies.  Cannot get it listed as a subcommand though?
Any ideas?!

Hey @svenhutchinson, thanks for the report. Did you manage to fix your issue?

Hello!
Kind of.  It seems to be linked to the fact that the theme incorporates Travis?
When I run  serve jekyll  it fails with that  error above but if I run bundle exe serve jekyll it works.
I think I've narrowed the reasoning down to Travis and the travis file?
I'm a little new to this so I'm not 100% what I am talking about if you have an explanation!
As a follow up I managed to get it all up and running on github pages now.
Algolia update indexes it through travis when I bundle exec algolia push to gh-pages  and it working great with the instant search 👍
I love that instant result page so much!
It only works if I prepend these things with bundle exec though.  Otherwise it fails.
I do have one issue though that you maybe able to help me out with.
When using ALGOLIA_API_KEY='your_write_api_key' bundle exec jekyll algolia push  again it has to have that bundle exec to work...  I get this error..
jekyll 2.5.3 | Error:  Cannot PUT to https://application_id.algolia.net/1/indexes/jekyll_NAME_tmp/settings:
<title>400 Bad Request</title>

400 Bad Request

nginx

(ID's removed)
There is an issue with jekyll_NAME_tmp/settings regardless of the index name used.
It does actually get indexed going through travis but I always get that error on the command line. I would love to eradicate and solve that issue as it seems to be the only remaining problem.
Any help/guidance would be much appreciated.
Sven.

So the bundle exec part is to be expected. This has nothing to do with this theme or with Jekyll, but with the way Ruby packages its dependencies. There are basically two ways to have dependencies in Ruby: global and local.
Global dependencies are installed on your machine, so if you install Jekyll with gem install jekyll you will have the jekyll command globally available.
Local dependencies are constrained to a specific directory and subdirectories through the use of a Gemfile file. In this file we list all the needed dependencies and their version number. If I add jekyll to my Gemfile, and do a bundle install, I'll then be able to use this local version of Jekyll through bundle exec jekyll.
The local dependencies are really useful when you are dealing with several projects on the same machines and they all need different versions of the same libraries. This is what I use for this theme.
There is a tool, called rvm that allows you to easily install different versions of Ruby, and switch between them. And it also comes with a bash hook that auto-detect when you are in a directory with a Gemfile and automatically appends the bundle exec before the needed command. That's what I use, but I should update the readme to always suggest to use bundle exec.
So, to sum up, whenever you see a repository with a Gemfile, you should do a bundle install to install all the dependencies on your machines, then call any tool referenced in the Gemfile with bundle exec. Hope this clarifies a bit how things work :)

Now regarding your other bundle exec jekyll algolia push issue, this should work. I'll have another look and see what is going on.

I just pulled the repo, updated the algoliasearch-jekyll dependency to 0.7 and did a bundle exec jekyll algolia push. It worked.
Could you try changing the version in your Gemfile and try again? If this still does not work could you send an email to<EMAIL_ADDRESS>mentionning your APPID so I can have a closer look?

Thanks! that cleared a bunch of things up with the detailed reply.
As a extra note I had to include the ALGOLIA_API_KEY in its own file.
When I just use the command with my Algolia  ALGOLIA_API_KEY='your_write_api_key' bundle exec jekyll algolia push
I get an error...
Algolia Error: No API key defined    You have two ways to configure your API key:      - The ALGOLIA_API_KEY environment variable      - A file named _algolia_api_key in your source folder
not sure if I am missing anything or if that helps at all.

I'll try that now and get back to you asap.  Thank you very much for your help.

Perfect!  That fixed it!  Must have been the 0.4 version in the Gemfile causing the issue.. cheers man

Great! I've upated the repo to include the 0.7 now.
Thanks for the report.


Replicator migrate fails on existing database

What is the bug?
Run into a number of issues while running the farcaster replicator migrate command on an existing database.

kysely_migration and kysely_migration_lock tables do not get created so migration fails. It's not clear whether this is because the database has existing tables in it or more specifically because there are kysely migration tables in a different schema in the database for another application
After creating these tables manually the migration fails because we have a table called reactions in another schema
Connected with this i could not see a way to put replicator tables in their own schema so had to move other tables out of the public schema to keep things seperated

How can it be reproduced? (optional)
Run migrate on a db with existing tables. We're running ponder (ponder.sh) which also has kysely migrations in the ponder schema in our db.

We recommend you use the Shuttle project instead of the old replicator. There were many issues with the replicator that won't be fixed.


ncurses throws expcetion with 0.13.15 (Also not colour in terminal)

(See the guidelines for contributing, linked above)
steps
Run sbt test with ncurses-6.0+20170902-3 and sbt 0.13.15
problem
I am using sbt 0.13.15 as our application has dependencies set for this, and I had to update ncurses to the latest version due to another OS issue. When doing this my terminal throws an exception before running sbt test java.lang.NumberFormatException: For input string: "0x100" However test does run successfully. Further issue is there is no longer any colour code in the command, no green/red for tests.
expectation
Sbt to run without error and colour green or red for each test
notes
sbt version: Installed 1.1.2-1. Application is locked to 0.13.15
ncurses version ncurses-6.0+20170902-3

Downgraded ncurses has worked (OS issue currently not occuring)

Turns out the OS issue I have is with libtinfo which is resolved by using the latest version of ncurses which means my issue still occurs

This is https://github.com/sbt/sbt/issues/3240, which was fixed in sbt 0.13.16.
What's the reason you can't upgrade to 0.13.16?

@dwijnand It appears moving to 0.13.16 seems to be fine for sbt. I tried the latest sbt, but I think that was too much of a jump for the project.


Fix(i18nService) : Minor fr translation fixes

Hey there,
here are some better translated i18n for french language
Regards, Olivier

Very good, thanks.


AI not reporting ILogger logs when run outside of Visual Studio

I'm not sure if this a bug, but for sure a weird behavior.

Given this config file:

  "Logging": {
    "LogLevel": {
      "Default": "Trace"
    }
  },
"ApplicationInsights": {
    "InstrumentationKey": Key"
  }

AI Initializing step
calling services.AddApplicationInsightsTelemetry(configuration); in ConfigureServices

Repro Steps

Create  logger from the loggerFactory _logger = loggerFactory.CreateLogger("category here");
log something _logger.LogInformation("this should appear in AI");
publish the app using dotnet publish
run the app using dotnet or directly from the exe . The log is not shown in AI dashboard

Actual Behavior
The log is not shown in AI dashboard
Expected Behavior
The log should be shown in AI dashboard
This is not happening when I run the app from Visual studio.
If I wanted to get it working outside of VS I need to add to the configure method in startup:
loggerFactory.AddApplicationInsights(serviceProvider, LogLevel.Trace);
How is this possible? And why it is working from VS and not from outside?
Version Info
SDK Version :2.2.0-beta1
.NET Version :	 netcoreapp2.0
How Application was onboarded with SDK(VisualStudio/StatusMonitor/Azure Extension) :
OS :
Hosting Info (IIS/Azure WebApps/ etc) : Kestrel

OK so I checked the source code of AI and I found the potential issue:
if (options.Value.EnableDebugLogger && string.IsNullOrEmpty(options.Value.InstrumentationKey))
            {
                // Do not use extension method here or it will disable debug logger we currently adding
                var enableDebugLogger = true;
                loggerFactory.AddApplicationInsights(serviceProvider, (s, level) => enableDebugLogger && Debugger.IsAttached, () => enableDebugLogger = false);
            }

Why you guys are calling loggerFactory.AddApplicationInsights only when there is a debugger attached?
Also, it seems like the EnableDebugLogger is always true (from the default options).
So, you are enabling AI's logging only when there is no instrumentation key? How is that helpful?
After this discovery, I don't understand how this could have happened based on the Repro Steps I described above.
This is so confusing..
My take away on this is you should not call AddApplicationInsights based on the debugger. It should be based on the config or enabled in the application code. It should be consistent.

@dhayhak Inorder for applicationinsights to collect traces logged via ILogger, you need to enable/configure your application manually. (While running from Visual Studio, it enables log capture automatically).
https://github.com/Microsoft/ApplicationInsights-aspnetcore/wiki/Logging
I have put together some basic documentation on how to do this.
Can you follow the documentation and let us know if there is something missing still?

@dhayhak , did you have change to follow the docs @cijothomas provided?

Yes, it worked. It is clear now that when running from Visual Studio, AI collects ILogger logs automatically by default. Thanks.


Task 105 - Story Completion

Please review.

Thanks!
I have a bit issue with the meaning of "complete"/"incomplete" in this context. What does it mean for a story to be "incomplete"? Take these four sentences, for example:
Sentence1: It was  my final performance in marching band. Sentence2: I was playing the snare drum in the band. Sentence3: We played Thriller and Radar Love. Sentence4: The performance was flawless.

Sounds like a "complete" to me, even without a 5th sentence. All that to say that, I would avoid using the phrasing of "complete"/"incomplete".

Thanks!
I have a bit issue with the meaning of "complete"/"incomplete" in this context. What does it mean for a story to be "incomplete"? Take these four sentences, for example:
Sentence1: It was  my final performance in marching band. Sentence2: I was playing the snare drum in the band. Sentence3: We played Thriller and Radar Love. Sentence4: The performance was flawless.

Sounds like a "complete" to me, even without a 5th sentence. All that to say that, I would avoid using the phrasing of "complete"/"incomplete".

I corrected the definition, please review again.

Please review again.

Thanks! Please pull and merge with the main branch. That should trigger the automatic tests.

To resolve a conflict and some other errors, I am closing this pull request and creating a new one with the same task.


Update CSSessionSubsystem.cpp

Create and Update delegates were reversed.

Thanks for the PR. I haven't checked this repo in a longer while, so sorry for the delay.


Bulk rename: Allow VSCode as EDITOR

Is your feature request related to a problem? Please describe
The default EDITOR nano is weak in functionality, but when I switch to VSCode, the bulk file is always blank.
Will you be willing to contribute this feature?

[ ] Yes, I will try to implement it

Describe the solution you'd like
Make VSCode open bulk file correctly.
Describe alternatives you've considered
No response
Additional context
No response

This is because the code command exits immediately after execution, without waiting for the user to actually finish editing the file, thus not blocking the current terminal. This makes Yazi think that the user has completed editing - because code indeed exits.
I'm not a VSCode user, is there any way to make code block the terminal until the user finishes editing and exits VSCode?

It seems --wait can deal with it. I'll give it a try back home, thanks for troubleshooting.

Yea you're right, just tested it works for me:
{ exec = 'code -w "$@"', block = true },

https://github.com/sxyazi/yazi/assets/17523360/73211c88-b89f-4edf-a7f6-cb43a7df20df


Add support for custom generate upload file name method

This enables more control over the file name generation. Inspired by the issues from https://github.com/Lith/strapi-provider-upload-google-cloud-storage/issues/82, but it does not solve them. One can use dynamic paths base on properties of the file though.
Examples, test and documentation is included.
:+1: for creating this plugin.

It has been merged. I've got two more changes in the pipeline. Once this once is merged I'll create a new PR, in order to avoid new merge conflicts.


allow to use json as replace for signed content

https://github.com/edenlabllc/ehealth.api/issues/1032

Changes Unknown when pulling c9e7c345fe926b3c049a40eeafdcc1792cb2efa3 on signed_content#1032 into ** on master**.


support passing filter as test binary arguments

Adding partial support for specifying test binary arguments like cargo. It can
greatly simplify the transitioning from cargo to nextest.
Signed-off-by: tabokie<EMAIL_ADDRESS>

@sunshowers Another feature request while migrating from cargo. PTAL~

Thanks for the contribution?
Hmm, I'm not totally convinced by the direction of this diff. Could you elaborate on the goal and intended audience for this a bit more? Is it to be able to pass in arguments like --include-ignored after a --? If so, I'm hesitant to introduce this sort of hidden alternative method to control the ignored status. Instead, I'd rather look through the passed in filters, warn on such arguments (and also --exact and --skip once filter expression support is stable) and direct users to the recommended ways instead.

Yes, I understand your concern. Personally I think of it as a first step to support test binary arguments (instead of a temporary workaround for certain filters). Ultimately we might want to push down unsupported test binary arguments to internal calls.
Aside from that, currently my main use case is really to keep the test command consistent with cargo workflow. E.g. cargo $CARGO_TEST_COMMAND --workspace -- --ignored can work for both CARGO_TEST_COMMAND="test" and CARGO_TEST_COMMAND="nextest run"

it would be OK to support an extremely limited form of this. --ignored and --include-ignored, test binary literals, and nothing else.

In this case, I'll use clap to implement the parsing logic instead.

That needless_bool lint suggestion is unhelpful and should be disabled for this block.

Looks fantastic now, thanks! Going to land this once CI passes -- hopefully get a release out in a few days as well.


Add box_intact_types

#78
from box import Box

class MyList(list):
    def sq(self):
        return [x*x for x in self]

b = Box(a = MyList([1,2,4]), box_intact_types = (MyList, ))
b.a.sq()
# [1,4,16]

Hello @pwwang! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file box.py:

Line 467:13: E113 unexpected indentation

This will also take some work to add the same features to BoxList, add testing and docs, but I can get around to that in the coming days. Will merge tomorrow unless told otherwise.

Coverage remained the same at 99.078% when pulling 2f00456d864777db34a0b957a734224ce8bb4a95 on pwwang:master into 86a11e162e63690a014235d07de04807dfa05b6c on cdgriffith:master.

Coverage remained the same at 99.078% when pulling 2f00456d864777db34a0b957a734224ce8bb4a95 on pwwang:master into 86a11e162e63690a014235d07de04807dfa05b6c on cdgriffith:master.

Coverage remained the same at 99.078% when pulling 2f00456d864777db34a0b957a734224ce8bb4a95 on pwwang:master into 86a11e162e63690a014235d07de04807dfa05b6c on cdgriffith:master.

Coverage remained the same at 99.078% when pulling 2f00456d864777db34a0b957a734224ce8bb4a95 on pwwang:master into 86a11e162e63690a014235d07de04807dfa05b6c on cdgriffith:master.

Coverage remained the same at 99.078% when pulling 2f00456d864777db34a0b957a734224ce8bb4a95 on pwwang:master into 86a11e162e63690a014235d07de04807dfa05b6c on cdgriffith:master.

This will also take some work to add the same features to BoxList, add testing and docs, but I can get around to that in the coming days. Will merge tomorrow unless told otherwise.

Make sense, I also initiated some tests for it. You may more rigorous ones.
Thanks.


Support dapr multirun from CLI.

Signed-off-by: Artur Souza<EMAIL_ADDRESS>It works but I still get an error if I try to update CLI. There is a difference in version being used by runtime and CLI for k8s.io/client-go. We might need a new RC for CLI.
$ go get -u<EMAIL_ADDRESS>go: downloading golang.org/x/net v0.6.0
github.com/dapr/cli imports
        github.com/dapr/cli/cmd imports
        github.com/dapr/cli/pkg/kubernetes imports
        k8s.io/client-go/plugin/pkg/client/auth/openstack: cannot find module providing package k8s.io/client-go/plugin/pkg/client/auth/openstack

I verified it worked on standalone mode.


Update twitter username

I've changed twitter username to be more consistent, so little PR to update it :)

👍


Process data model

Create model classes for all entities as shown in the ERD diagram.
Set up associations between models.
Create custom methods, for:

Users - A method that returns the 3 most recent posts for a given user.
Posts - A method that updates the posts counter for a user and a method that returns the 5 most recent comments for a given post.
Comments - A method that updates the comments counter for a post.
Likes - A method that updates the likes counter for a post.

Thanks for the suggestions.


Support multiple slack accounts

This doesn't yet support a different profile photo for each slack account.


Unexpected vertical line in line chart

Hi there. I tried to look up this problem from web but didn't find related topics. Can someone give me some quick tips on how to remove the unwanted vertical line? I will try to reproduce it on jsfiddle if it's needed.

Are you using a plugin to draw any extra lines? Can you post a fiddle for which this reproduces?

Thanks for the reply. I'm not using any plugins, I will try to reproduce the case in JSfiddle now.

Fiddle Link Updated
https://jsfiddle.net/afreet220/oqod0fyo/2/

Ok, I think this is a duplicate of #2994

Thank you very much! I'm sorry to ask since I'm not that familiar with the workflow: so when will that fix be live on the CDN?  If I want to incorporate the fix now I have to download the js file and fix it locally?

It will be live when we release next (probably in the next week or so).
If you incorporate the fix now, you will have to clone the repository, make the fix and do gulp build on the command line to generate a newly built file which you can use

@etimberg Thanks a lot for the info, I will look into that and best luck in future!


Delegate

It would be great if you could add a delegate to know when the navigation bar is expanded or not.

That's a good suggestion - thanks.

Thanks for the suggestion @taglia3 - this has now been added. It'll be in the next version (to be released soon).


Clarify RAR task's SearchPaths values

The existing documentation on SearchPaths was reasonably accurate but confusing and not well-formatted. Reworded in an attempt to be clearer, and added code literals for the {} marker values treated specially by the task.
@kempb suggestions on wording welcome!

@rainersigwald,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@kempb please review

Reviewed, looks good.


Feature_Request:Adding a logo for the website and make navbar concise.

Title and Issue number

Title : Feature_Request:Adding a logo for the website and make navbar concise.
Issue No. : #403
Code Stack : HTML, CSS
Close #403

Video/Screenshots (mandatory)
Attached the working video of my deployed project:
https://github.com/apu52/Travel_Website/assets/78197261/3d7a5a17-5178-4d83-a081-27af5c7121c0
Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have gone through the  contributing.md file before contributing

Additional context (Mandatory )
Are you contributing under any Open-source programme?

@aatmajajoshi are you still working on this issue or I will close this PR as well as the issue??

@apu52 can u please check now ?

Closing this PR as this issue has already been exhausted


Prohibit module initialization

This fixes #679. The solution is more complicated than just forbidden the method. I choose to let it work more like Ruby:

Class < Module and Module < Object
Module class doesn't support #new (but support .new)
Class class support #new

Before today our inheritance chain for Class class is Class < Object, so I changed it as described above.
@hachi8833 can you help me compare the Goby's behavior with Ruby again?

@hachi8833 can you review and approve it if looks ok?


Added basic libdatabroker support

Added basic libdatabroker support so that it can appear on titan panel or similar.
Let me know if you want me to change/tweak any of it.

Thank you very much for your contribution. I will include this for the next update along with a few other changes and fixes.
A very simple tweak I'll add is to make a right click hide the options window if it is already shown.


Per-suite retry logic (and timeout)

🚀 Feature Proposal
A per-suite retry logic (and an associated per-suite timeout enforcement) at the runner level to allow automatic retry of parts of a concurrent (>1) run of saucectl.
Motivation
User Experience enhancement moving logic implementation of retries from users to tool configuration.
Details
In some cases, with either existing or in-development test suites, users may face test flakiness due to infrastructure inconsistencies (read: hardware devices and cosmic rays - no this isn't a joke). In such an event, specifically when using saucectl with concurrency: >1, facing test failures could lead to a re-run of test suites that may have been successful, wasting resource allocation that could have been used for other test runs.
Thus, the request would be in these parts:

Feature 1: --suiteTimeout enforced by saucectl as a per-suite timeout limit, instead of a full saucectl timeout limit, working in tandem with
Feature 2: --suiteAutoRetry [true, false] and --suiteMaxRetries X implementing saucectl-driven test suite retry logic, simplifying test retries and limiting retries to failed suites instead of users having to write parsing logic to deduce - or just re-run the saucectl invoke outright.

Per-suite timeout covered by #384

Closing, as retries have also been implemented since.


Good stuff

https://github.com/jantimon/favicons-webpack-plugin

Added, thanx
https://github.com/theaidem/stapp/commit/20c1c4f555cad7cfd3acdb786691270e7a076ddd


Adding Table in an Empty Modal

Adding Table in an Empty Modal
When I try to add a table in an empty modal, it appends to the parent node not to itself as it shown at the image below.

Can you reproduce the bug here: https://alex-d.github.io/Trumbowyg/demos/#plugins-table ?

I couldn't reproduce it because the table is not empty when mounted.

Can you detail steps to reproduce your issue please?

When I try to add a table,
If the editor div is empty (if the first input will be a table), the table is being appended to the parent node, not inside to the editor div as it shown in the image.
But if any other input inserted already in the editor div ( as there is already a text inserted https://alex-d.github.io/Trumbowyg/demos/#plugins-table ) everything works fine and the table is being appended to the editor div.

If I clean the editor, it does works too, so it's really strange :/

It is for real, I couldn't even come up with an idea of what can cause that.

Can you make an isolated reproduction from here: https://jsfiddle.net/AlexandreDemode/p7hb3x6u/

I can't, that's a huge project written in Ember and it's gonna take a huge time to adapt the code in jsfiddle which I don't have.

OK, So I can't help you :/
Good luck!


Fullscreen

Hi love the app!
So when I put HS in fullscreen then the trackers dissaper!
When I go out again, then they come back.
Can You make it so it stays there when in fullscreen?

I already looked if it's possible with osx fullscreen, but it looks it's very difficult... but i'll check :)

If you could make it happen in fullscreen I would love you FOREVER!

I would also vote +1 for this. If real full screen is not possible, a reasonable compromise would be to draw  100% opaque windows on top and bottom of the screen to cover the extra elements (basically take the size of the HS window and draw black windows on the rest of the screen area). Won't be the same effect, but it will do ok.

not actually possible, closing

I still not possible?

I somehow made this work with my old League of legends jungle timer app. I will look into it.


Batch example fails to compile

The folowing code snippet offered as an example of using batch gives the following error in the IDE:
Two positional arguments expected but one found.
From looking at the code, insertAll is expecting a TableInfo and a list of Companions.  If this is correct the example should be updated accordingly.

Yes, the first argument should be a table info (todos in this case). Thank you for the report, I've fixed the problem and the website will rebuild shortly.


build my docker image ,but run error

Provide Your Network Information

Where is the location of your server? (i.e. City, or In/Out China)
china
Which cloud platform(AliYun/Qcloud/DigitalOcean/etc) are you using?
no just my macbook

Run npm run doctor or wechaty run doctor(for docker user), paste output here
<EMAIL_ADDRESS>check-node-version /wechaty
check-node-version --node ">= 6.9"
node: 7.9.0
npm: 4.4.4
yarn: 0.22.0
Wechaty Doctor

Wechaty version: 0.7.71
Linux x64 version 4.9.12-moby memory 711/1999 MB
Docker: true
Node version: v7.9.0
Tcp IPC TEST: PASS
Chromedriver: ChromeDriver 2.25 (5a911c1ac99b36efadff2041357740080cc27220)`

this is my dockerfile
`FROM zixia/wechaty
WORKDIR /bot
COPY ./dist/ .
COPY package.json .
RUN npm install
ENTRYPOINT [ ]
CMD [ "node","index.js" ]`
Expected behavior
can execute normally
Actual behavior
has errors as follows
01:47:06 INFO Wechaty v0.7.71 initializing...
01:47:13 WARN PuppetWebBrowserDriver valid() driver.getSession() timeout(halt?)
01:47:13 WARN PuppetWebBrowserDriver valid() driver.getSession() exception: valid() driver.getSession() timeout(halt?)
01:47:13 WARN PuppetWebBrowserDriver initChromeDriver() got invalid driver
01:47:13 WARN PuppetWebBrowserDriver initChromeDriver() with retry: 1
01:47:20 WARN PuppetWebBrowserDriver valid() driver.getSession() timeout(halt?)
01:47:20 WARN PuppetWebBrowserDriver valid() driver.getSession() exception: valid() driver.getSession() timeout(halt?)
01:47:20 WARN PuppetWebBrowserDriver initChromeDriver() got invalid driver
01:47:20 WARN PuppetWebBrowserDriver initChromeDriver() with retry: 2
01:47:27 WARN PuppetWebBrowserDriver valid() driver.getSession() timeout(halt?)
01:47:27 WARN PuppetWebBrowserDriver valid() driver.getSession() exception: valid() driver.getSession() timeout(halt?)
01:47:27 WARN PuppetWebBrowserDriver initChromeDriver() got invalid driver
01:47:27 WARN PuppetWebBrowserDriver initChromeDriver() with retry: 3
01:47:34 WARN PuppetWebBrowserDriver valid() driver.getSession() timeout(halt?)
01:47:34 WARN PuppetWebBrowserDriver valid() driver.getSession() exception: valid() driver.getSession() timeout(halt?)
01:47:34 WARN PuppetWebBrowserDriver initChromeDriver() got invalid driver
01:47:34 WARN PuppetWebBrowserDriver initChromeDriver() not valid after retry: 4 times: Error: initChromeDriver() got invalid driver
at BrowserDriver. (/wechaty/dist/src/puppet-web/browser-driver.js:117:35)
at Generator.next ()
at fulfilled (/wechaty/dist/src/puppet-web/browser-driver.js:4:58)
at process._tickCallback (internal/process/next_tick.js:109:7)
01:47:34 ERR PuppetWebBrowser init() exception: initChromeDriver() got invalid driver
01:48:06 WARN PuppetWebBrowserDriver valid() getSession() catch() rejected: unknown error: Chrome failed to start: exited abnormally
(Driver info: chromedriver=2.25 (5a911c1ac99b36efadff2041357740080cc27220),platform=Linux 4.9.12-moby x86_64)
(node:1) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): WebDriverError: unknown error: Chrome failed to start: exited abnormally
(Driver info: chromedriver=2.25 (5a911c1ac99b36efadff2041357740080cc27220),platform=Linux 4.9.12-moby x86_64)
(node:1) DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
01:48:13 WARN PuppetWebBrowserDriver valid() getSession() catch() rejected: unknown error: Chrome failed to start: exited abnormally

Duplicated with #147
Please wait for us to release an onbuild version of docker image, because that would be easy to use.

so  now i cannot build a docker image to run my bot ? or has other method?

yes you can.
I'll write about this when I have time.

Could you give me some notice ,maybe i can try haha

@ditunes You can have a try by
FROM zixia/wechaty
WORKDIR /bot
COPY ./dist/ .
COPY package.json .
RUN npm install

- ENTRYPOINT [ ]
- CMD [ "node","index.js" ]
+ CMD [ "npm", "start" ]

Remember to make sure you have a start script defined in your package.json file.

See: https://github.com/Chatie/wechaty/issues/147#issuecomment-296448684

Hi did the new onbuild image solve your issue?
I'll close this one, if you still have the problem related to the docker, please reply under #147 instead.


Add thread pool design document (phase 1)

This is a successor to #18864 which includes only the parts which are for current implementation in the OpenSSL codebase and for which there appears to be broad consensus. My intention is that we should be able to merge this soon, as the applicable implementing code has already been approved.
The remaining unmerged parts of #18864 remain a proposal available for comment. Therefore, I've converted that to a draft and ask for reviews of this PR instead.

Updated.

24 hours has passed since 'approval: done' was set, but this PR has failing CI tests.  Once the tests pass it will get moved to 'approval: ready to merge' automatically, alternatively please review and set the label manually.

I do not think the CIs are relevant.

Merged to master branch. Thank you.


Research assertion library for Puppeteer

Research how to set up Chai (assertion library) with Puppeteer (make sure we can have tests run and print sensible console messages) and Docassemble (make sure DA doesn't eat/corrupt files).

Using Chai now. See tests/sample-chai-assertions.js.


https://www.github.com/Microsoft/WPF-Samples/tree/master/Windows%2FMessageBox%2FMessageBox.csproj

Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Ssssss


Update documentation for "Mirroring the NVD from NIST"

The four properties mentioned in Item #2 of https://jeremylong.github.io/DependencyCheck/data/mirrornvd.html does not seem to be correct, at least for CLI module.
I ended up using the following to override the default URLs to get my copy of Dependency Check working with local copies of the NVD files.

cve.url-1.2.modified
cve.url-2.0.modified
cve.url-1.2.base
cve.url-2.0.base

For example

cve.url-1.2.modified=file:///myNVDcopy/mvdcve-Modified.xml.gz
cve.url-2.0.modified=file:///myNVDcopy/mvdcve-2.0-Modified.xml.gz
cve.url-1.2.base=file:///myNVDcopy/mvdcve-%d.xml.gz
cve.url-2.0.base=file:///myNVDcopy/mvdcve-2.0-%d.xml.gz

This is a hidden but very useful feature. Perhaps makes sense to make it more visible inside the individual module documentation (.cli doc has no mention of it at all I believe)

Thanks for reporting this. The CLI is the only version that does not expose these configuration options - I'll add these options to the next release.

The arguments have been added to the CLI; documentation still needs to be updated.

I succeeded in using this today. Here's the script I'm using to point have testing runs point to the local files I downloaded using nist-data-mirror:
#!/bin/sh
REPO=/home/dale/git/DependencyCheck/
CLI_SCRIPT=$REPO/dependency-check-cli/target/release/bin/dependency-check.sh
ARGS="--app Test --out /tmp --log /tmp/dependency-check-log.txt --cveUrl20Base file:///home/dale/NVD/nvdcve-2.0-%d.xml.gz --cveUrl12Base file:///home/dale/NVD/nvdcve-%d.xml.gz --cveUrl20Modified file:///home/dale/NVD/nvdcve-2.0-Modified.xml.gz --cveUrl12Modified file:///home/dale/NVD/nvdcve-Modified.xml.gz"
$CLI_SCRIPT $ARGS $@

The documentation has been updated and the patch for this issue will be included in the next release.


Fix test_rom boolean check for Version Number

https://github.com/bespoke-silicon-group/bsg_f1/blob/99d7e801bd67ee7f15c735fd7d85c450e271a810/cl_manycore/regression/library/test_rom.c#L45
Getting a PhD doesn't guarantee you can code...

Fixed in #228


[WIP] Decimal signature

Decimal Arithmetic operations need to support function signatures of the format:
 DECIMAL(r_precision,r_scale) ADD (DECIMAL(a_precision, a_scale), DECIMAL(b_precision, b_scale))

SignatureBinder doesn’t support binding variables to literals. Only binds Type variables to Types.
Introduced longVariablesBindings_ member in SignatureBinder to map variables like a_precision, a_scale etc, to their long literals in the incoming function call.

SignatureBinder cannot resolve return types that have variable parameters. For decimaltype, the return type must be inferred from the precision and scales of the inputs. To support this, introduced a function pointer member to the FunctionSignature. The SignatureBinder will invoke this function pointer at the time of resolving the return type.

Testing:

Compilation succeeds.
Test crashes because the DECIMAL operation is not implemented yet.

Is this PR still needed or can we close it?


Add cloning via PVC annotation

This PR enables provisioning new volumes as a clone of an existing PVC.  To
request a clone, the user should add a clone request annotation to the PVC:
kind: PersistentVolumeClaim
apiVersion: v1
metadata:
  name: clone-claim
  annotations:
    k8s.io/CloneRequest: default/source-pvc

The provisioner will locate the indicated PVC and determine the underlying
associated cinder volume ID.  The provisioner will use this ID as the
source-vol-id parameter when provisioning the new volume resulting in a clone
at the storage level.
Cloning will only work if the cinder backend supports "smart-cloning".  This
means that the clone will complete without the need for cinder to perform a
bit-for-bit copy using the host.  Because of this, the cinder provisioner will
not attempt to clone unless the associated storage class contains the
parameter: "smartclone" to indicate that volumes provisioned from this storage
class can be cloned correctly.
If the provisioner was able to clone the volume it will apply the
'k8s.io/CloneOf' annotation to the PVC.

/cc @j-griffith

/cc @hogepodge
Can you mark this one ok-to-test?

/ok-to-test

if the cinder backend supports "smart-cloning" - how I can see does our cinder backend support this? Is this using basically volume clone command?

@zetaab The short answer is "depends on the configuration".  I define "smart-cloning" as the ability to provision a new volume clone utilizing exclusively the storage backend.  When a source volume id is passed into the create verb cinder will request a clone from the backend storage if supported.  Otherwise, cinder will perform a host-assisted clone.  Host assisted clone is also needed when cloning across volume types.  Cinder host-assisted clone will obviously not work well in a kubernetes context so we want to avoid this.  This is why this patch requires the smartclone: true parameter to be added to the storage class being serviced by the provisioner.

/approve
/lgtm
@aglitke please add some notes from here to the docs/


Implement automatic cloud proxying

cc @sreya92
cc @code-asher
Please review carefully. waitForPath in particular is very subtle!

Only the first commit is relevant, rest are minor CI fixes.


Getting selected item(s) in "checkboxes"

Hi, I would not make a trivial question, but how can I get the "selected items" of the checkboxes in a function of my controller?
This is my use case: I have some checkboxes, I want to call a function every time an item is selected. inside this function I must know what item(s) is selected because for each item I want to perform different actions (like adding some extra field in my form schema and definition).
This is my approach:
Schema:
{
            type : "object",
            title : "Reporting",
            properties : {
            ......
                needs_procedures : {
                    type : "array",
                    items : {
                        type : "string",
                        enum : [ "1", "2", "3" ]
                    }
                }

            }
}

FormDef:
{
    type : "section",
    htmlClass : "col-xs-6",
    items : [ {
        key : "needs_procedures",
        title : "Procedures",
        type : "checkboxes",
            onChange : "itemSelected(form.key, model.needs_procedures)",
        titleMap : [ {
                value : "1",
                name : "pneumological examination"
                }, {
                value : "2",
                name : "cardiologic examination"
                    }, {
                value : "3",
                name : "spirometry"
                }, {
                value : "4",
                name : "electrocardiogram"
                } ],
                } ]
            }

"needs_procedures" field is inside a section, that is inside a section, that is inside a fieldset, that is inside a tab structure.
In my controller
$scope.itemSelected = function(key, value) {
        console.log(key);
        console.log(value);
    };

The First time that an element is checked the function itemSelected() is called, but in the second parameter I have a empty array.
The second time I have an array with one element: the previous...
the third time I have an array with two elements: the two previous...
and so on...
Then it is as if the method is invoked immediately before the model is updated.

@fabiomcj can you make a plunker demo or gist for us?

We are probably going to add arrayIndeces soon so you can traverse your model more easily. But a plunker/gist would help.

Please re-open if you can provide more information to help us to help you :)


The commands are not editor commands

If I do any of the commands from this plugin it gives:
E492: Not an editor command
I am using neovim on arch linux
arduino ide is installed via pacman
I can do :help arduino

I just needed to open a .ino file

I'm having the same issue here to, running Neovim v0.9.4, I can't seem to use these commands when in an .ino file and or .cpp or .cc file, but when I'm either in my dashboard/empty buffer window I can use them. I'm using Lazy.vim as my package manager, would that be an issue, I saw #59 and the fact it was closed with a fix, is there something else going on here at play?

I am using alpha.nvim as my dashboard here, which might be affecting it. I don't know. I looked through :help arduino and didn't see anything at all that I could do.

When running inside of a .ino file,

I for some reason have autocomplete for these command when I have no open buffer.
Sorry for commenting on a closed issue, I saw this was very similar to mine and thought it would be better to ask it here.

Looking through the code of this plugin, it looks like there is no support with .cc files, which that not working its fair, though I don't know whats causing the issue between .ino files and .cpp files.

I have the very same problem that you have @MilkJug1. I am using LazyVim. I can use the commands only when no file is open. Some condition is reversed, maybe?

I have the very same problem that you have @MilkJug1. I am using LazyVim. I can use the commands only when no file is open. Some condition is reversed, maybe?

It does sound like it, though it doesn't make a lot of sense on why that would be. Maybe something to do with the way lazy.nvim handles plugin loading? I might poke around the source code and see if it's just some check not working properly or something.

It would be nice if you could have a look… I am still very new to Lua and
NeoVim.
Alex Escalante
Web & Mobile Development For Hire
http://audelabs.com
On Wed, Jan 24, 2024 at 7:18 PM MilkJug @.***> wrote:

I have the very same problem that you have @MilkJug1
https://github.com/MilkJug1. I am using LazyVim. I can use the commands
only when no file is open. Some condition is reversed, maybe?
It does sound like it, though it doesn't make a lot of sense on why that
would be. Maybe something to do with the way lazy.nvim handles plugin
loading? I might poke around the source code and see if it's just some
check not working properly or something.
—
Reply to this email directly, view it on GitHub
https://github.com/stevearc/vim-arduino/issues/26#issuecomment-1909188067,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFAIRSHINDW3SDS6J3NLE3YQGXFXAVCNFSM4NOYL24KU5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TCOJQHEYTQOBQGY3Q
.
You are receiving this because you commented.Message ID:
@.***>

It would be nice if you could have a look… I am still very new to Lua and NeoVim. Alex Escalante Web & Mobile Development For Hire http://audelabs.com
…
On Wed, Jan 24, 2024 at 7:18 PM MilkJug @.> wrote: I have the very same problem that you have @MilkJug1 https://github.com/MilkJug1. I am using LazyVim. I can use the commands only when no file is open. Some condition is reversed, maybe? It does sound like it, though it doesn't make a lot of sense on why that would be. Maybe something to do with the way lazy.nvim handles plugin loading? I might poke around the source code and see if it's just some check not working properly or something. — Reply to this email directly, view it on GitHub <#26 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAFAIRSHINDW3SDS6J3NLE3YQGXFXAVCNFSM4NOYL24KU5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TCOJQHEYTQOBQGY3Q . You are receiving this because you commented.Message ID: @.>

Okay, after a while and remembering to fix it. I think I did something to get it working. All I had to do was add the ft = "arduino" as an option for Lazy when installing it in my plugins.lua
What I mean:
	{
		'stevearc/vim-arduino',
		lazy = false,
		ft = "arduino",
	},

This seemingly fixed it, well I think at least. I do have this setup in my lsp.lua config which uses mason, heres what that looks like, since that could also be what fixed it.
require('mason').setup()
require('mason-lspconfig').setup_handlers({
		 --- Omitting the other parts of my lsp config, since thats not needed, just the arduino part.

	["arduino-language-server"] = function()
		require('lspconfig').arduino_language_server.setup {
			on_attach = on_attach,
			capabilities = capabilities,
			cmd = {
			'arduino-language-server'
		}
	}
end
})

Try this, it might fix it.  @AlexEscalante


please provide how to make jitsi installer step by step with  proper document?

I tried two or three ways but i am not able to make installer of jitsi so please provide proper way to make installer of jitsi.....

Search the mailing list archives, this has been discussed at length in the past. And stop opening new issues as this is neither a support forum nor your problem something that belongs here.


Documentation needs to be updated

Some of the code samples in the ReadMe don't work in xCode

Hi @thomasbaldwin, please put yourself in our shoes. Your comment is incredibly unhelpful.

Which code samples are incorrect?
Which version of the README are you looking at?
Which version of Xcode are you using?

Most likely the code samples work just fine if you are using the version of Xcode the README specifies.

Hi @cnoon, sorry about not being specific. I was trying to use this code in xCode 7, but it throws errors:
        Alamofire.request(.GET, "http://httpbin.org/get", parameters: ["foo": "bar"])
            .responseJSON { response in
                print(response.request)  // original URL request
                print(response.response) // URL response
                print(response.data)     // server data
                print(response.result)   // result of response serialization

                if let JSON = response.result.value {
                    print("JSON: \(JSON)")
                }
        }

I ended up getting it to work by doing something like this: http://stackoverflow.com/questions/32506703/alamofire-not-working-properly-tuple-issue-different-number-of-elements

@thomasbaldwin you are using an old version of alamofire. Update your alamofire and the examples in the documentation should work fine.

I've installed using cocoa pods and am using version 2.0.2
platform :ios, '8.0'
use_frameworks!

target 'apollo' do
    pod 'PureLayout', '~> 3.0.1'
    pod 'SwiftyJSON', '~> 2.3.0'
    pod 'KMPlaceholderTextView', '~> 1.0.0'
    pod 'Alamofire', '~> 2.0.2'
end

Yep I upgraded to version 2.0.2 from 2.0 and it did the trick. Thanks @cristeahub !

@cristeahub I'm using cocoapods and using version 2.0.2 and still getting the error. What version do you suggest I use?
platform :ios, '8.0'
use_frameworks!

target 'apollo' do
    pod 'PureLayout', '~> 3.0.1'
    pod 'SwiftyJSON', '~> 2.3.0'
    pod 'KMPlaceholderTextView', '~> 1.0.0'
    pod 'Alamofire', '~> 2.0.2'
end

@thomasbaldwin If you want the documentation for 2.0.2, view the 2.0.2 tag on GitHub. Current master is for 3.0.0.

Yep that's it. I just blindly installed it by copying the pod from the master readme to my podfile which was set for version 2.0. Thanks for the help @jshier :+1:

The readme still has reference to the install being a 2.0.x install along with the 3.0.0 examples.  FYI

Good call @seigel! Fixed in afc8cdb3.


Get updated pictures

Some of the pictures should probably get updated at some point so it doesnt seem outdated?

Can you make a list of which ones?

Yep!

New Photos:

Visit Page

Kelly & Denise LifeGroup

Ben and Kristen LifeGroup

This is obsolete because the site has been over to SquareSpace.


Module 3 exercises [Onneetse Moleta]

mod 3 EX 1 land condition in Panda.pdf

Well done, Onneetse. You clearly grasped the material outlined in Module 3 and successfully applied it to these exercises. Thank you for your participation!


Update README file  with shorter content with  links to other files

I.e. the procedure for uploading models can moved to a separat file : UploadModels.md
A link to this file is created in README.md file

[ ] Cut upload part to separate md file and link from readme


fix(android): check disableFocus when state is ready

Update the changelog
fix(android): check disableFocus when state is ready
Describe the changes
check disableFocus prop when state is ready (currently only works on playback toggle)

I don't understand why you do this change ...
Audio focus and keep screen on are 2 differents features, controlled independently.
No need of this change in my point of view

@freeboub
You're right, I was just trying to sync the behavior with this code.
I think it would be better to remove the if (!disableFocus) from the existing code, what do you think?

@YangJonghun I agree, removing the other reference makes more sense !

@freeboub I fixed it. thanks!


Refactor tests to remove repetition.

This is a superficial refactor of the tests to be a bit more streamlined
by removing obvious repetition from the test classes. Most of this is
done by simply adding a Flask app instance with an API in the setUp
method of the TestCase classes. The tests in terms of what they do have
been unchanged.
This provides slightly simpler code to read - and provides new tests
with pre setup app and api instances if needed.
Also a minor refactor of import orderings in tests to match PEP8.
This was done as a colleague of mine had been building unit tests based off of the ones in flask-restful, leading to a conversation about reducing repetition in tests and making them more readable.
I note there was also a PR to move to pytest (https://github.com/flask-restful/flask-restful/pull/704) which was rejected. Considering nose has been deprecated for years and that pytest generally results in smaller and simpler tests, is there a reason it was rejected?
I'd be happy to redo this PR with pytest as an example.

Looks like the coverage decreased because the total volume of the test code decreased :-D

Thank you :)
Test code is not included in coverage results, so somehow, this change did decrease coverage. I am not sure why. Or maybe coveralls is lying 😆I don't know which yet.

I note there was also a PR to move to pytest which was rejected.

Yeah it changed so much stuff and there was still active development at the time and the merge conflicts weren't worth it. Personally I like pytest a lot, but I don't have

Damn I was hoping for an easy explanation 🤔.
Well if you're happy with pytest and development has slowed down I can get a pytest PR out and move to functional tests instead of class based.
Maybe it'll sort out the coverage issues.

@joshfriend see this PR: https://github.com/flask-restful/flask-restful/pull/834


Introduce version lifecycle management into the operator

Traditionally, a Kubernetes operator uses an independent lifecycle from the software it manages. This creates scope of impact for upgrades so that upgrading the operator does not adversely impact the software instances, and a single software instance can be upgraded or downgraded without impacting the other software instances.
The Kabanero operator does not adhere to this principal at the moment. Upgrading the operator may cause software instances to be upgraded because we only have a single static orchestration built into the operator.
This epic proposes respecting the version identifier provided by the user in the Kabanero resource kind, introducing per version orchestrations, and consolidating the management of versions into human editable configuration files.

This was resolved by PR #184


add core_apps.code_result.models.CodeEecutionResult

Add CodeExecutionResult to store the execution result of the submitted code to the RCEE engine

The model will store the final result of the execution of the user code in the Remote Code Execution Engine Service.


TypeScript: Jimp types are not corresponding to Jimp implementation

Expected Behavior
When importing Jimp as described in the doc for a TypeScript project:
import Jimp from 'jimp'

I should be able to use Jimp directly as is:
Jimp
  .read(fileData)
  .then((image: any) => {
    ...
  })
  .catch((err: any) => {
    ...
  })

Current Behavior

The compile step is OK, but there is  runtime error saying:
TypeError: Cannot read property 'read' of undefined

If I use the require-like import syntax:
import * as Jimp from 'jimp'

and use Jimp as below:
Jimp
  .read(fileData)
  .then((image: any) => {
    ...
  })
  .catch((err: any) => {
    ...
  })

I then have a compile error saying:
Property 'read' does not exist on type 'typeof import("/path/to/node_modules/jimp/types/ts3.1/index")'

If I disable manually type checking for Jimp:
// @ts-ignore
Jimp
  .read(fileData)
  .then((image: any) => {
    ...
  })
  .catch((err: any) => {
    ...
  })

There is no runtime error and Jimp works as expected.
Failure Information (for bugs)

Context

Jimp Version: 0.8.4
Operating System: macOS 10.15 Catalina
Node version: 10.16.3
NPM Version: 6.11.3
TypeScript Version: 3.6.3

Workaround
To use Jimp with types information, I have to import the types separately and bind them to the classic require syntax:
import Jimp from 'jimp'
// tslint:disable-next-line: no-var-requires
const jimp: Jimp = require('jimp')

And then, use "jimp" instead of "Jimp":
jimp
  .read(fileData)
  .then((image: any) => {
    ...
  })
  .catch((err: any) => {
    ...
  })

@crutchcorn

I sincerely think we should revert the export type to only export using namespace ala 0.6.4 typings. This doesn't mean we'd revert the 3.1 typings. I don't know enough about tsconfig options to provide an alternative.

I have no clue what's happening with @noomly example. I'd have to look further

Same for me.
Types are completely fucked.
Documentation is completely fucked.
Examples from npm doc give problems with strict TS config

With version 0.8.4, @RomainFallet workaround at the top works for me.
With version 0.8.5, the workaround no longer works. Errors given that jimp.read and jimp.AUTO are static members of type Jimp.

Never mind. doesn't work with 0.8.4 either. Does anyone know any kind of workaround to get jimp to work in a typescript project now? I downgraded all the way to 0.5.3 still gives typescript errors.
Even if I lose ability for types...I just need to be able to build the project and run and client is not happy now. Will downgrading to a certain typescript version get this to work? Or what changed that caused the break? thank you

Got this to work again by downgrading...
typescript down to 3.4.5
jimp down to 0.6.4
using import * as Jimp from 'jimp'; to import

I'm taking a look at fixing some of the type issues right now

@noomly it seems the bug you were experiencing was unrelated to the issue described in the OP. However, it's the first PR I've made for ;)
If you can test against 0.8.6-canary.810.481.0 it should include that fix
For everyone else, I have a solution to the overaching issues many users are facing since 0.6.4 in terms of typing problems. I'll have a PR open soon

I will also note to @WeekendMan:
I've been needing a break from working for OSS for a while because I tend to overwork myself. The only times I usually get to work on things is actually shown in your handle: The weekend, man.
Be patient and know that most of us working on OSS do so as a hobbyist project. Because it's open source, you're always able to contribute back upstream if you feel a fix is needed immediately

I had the strange issue that doing the simple import Jimp from 'jimp' was working until I ran my unit tests. I'm think this is due to webpack being able to avoid this problem somehow. However, I would run my unit tests (which are not webpack'd) and I'd get this problem which led me here.
After adding the const jimp: Jimp = require('jimp') hack, everything seemed to be working with my unit tests. However, this then broke the webpack side of things. So I had to create a super hack in the form of const jimp: Jimp = (Jimp || require('jimp')) and it seems to be working in both cases. Yuck!

If someone reverts Corbin changes (i think that’s what was decided on) I
will merge it. I don’t use this project in my work at all so any help
maintaining it is very appreciated
On Wed, Nov 20, 2019 at 12:51 PM Tim Schmidt<EMAIL_ADDRESS>wrote:

I had the strange issue that doing the simple import Jimp from 'jimp' was
working until I ran my unit tests. I'm think this is due to webpack being
able to avoid this problem somehow. However, I would run my unit tests
(which are not webpack'd) and I'd get this problem which led me here.
After adding the const jimp: Jimp = require('jimp') hack, everything
seemed to be working with my unit tests. However, this then broke the
webpack side of things. So I had to create a super hack in the form of const
jimp: Jimp = (Jimp || require('jimp')) and it seems to be working in both
cases. Yuck!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/oliver-moran/jimp/issues/803?email_source=notifications&email_token=AAJDEBCWTUMDDOAAMETAV3TQUWPM3A5CNFSM4JAGSKA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEU2TYI#issuecomment-556378593,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJDEBA7IGBDQS5B3UWKKZDQUWPM3ANCNFSM4JAGSKAQ
.

@hipstersmoothie I don't think it needs to revert my changes, I think we need to modify the exports or improve the docs with TS usage. The fact that there's more than one working project with #815 and that #816 was quickly and able to be tested to work with one of the reported issues makes me believe that the changes I'd made as-of-now are no worse than the 0.6.5 release with the issues in them and significantly improved due to testing and more accurate overall tests

I've done the closest thing to "reverting" the changes from 0.6.5 that are causing many of the problems that people are reporting (and why so many are saying that 0.6.4 were the last working version for them):
https://github.com/oliver-moran/jimp/pull/820
(I would ask folks to test the version of jimp associated with this PR but just my luck the release step failed)
We also need to bring in the PR:
https://github.com/oliver-moran/jimp/pull/815
In order to fix the runtime errors that were caused unrelated to the TS definition changes in 0.8
So when testing the version above, if you notice runtime errors, that is fixed in a different PR
Lastly, @noomly's issues with the 0.8.5 defintion files (that were a regression from 0.8 from other versions) have been fixed in a PR for a while that just needs to be merged:
https://github.com/oliver-moran/jimp/pull/810

Are y'all able to test against 0.9.2 stable release. There was some work done to improve TS stability and should fix everyone's issues with typings (if you were one of the ones that had to revert to 0.6.4, this includes you). Please test against and confirm/deny crosses fingers

🤚 @crutchcorn I'm really good at writing typings, are you open to a large PoC PR on this?

Hi @forivall 👋 Small world!
What'd the PoC PR be in regards to? I've done a fair amount of work in order to modularize the typings for plugins and it seems to be in a fairly stable place now especially with the type tests that were written

Speaking of @hipstersmoothie, think we can close this issue out? I didn't realize it was still open and according to many users, it seems we're finally at a stable place

Closed! Good job by the way. Now that the storm has settled it seems like all is good.

I was originally thinking of something along the lines of how express does plugins, but then, by playing around in my node modules, I found a satisfactory solution: https://github.com/oliver-moran/jimp/compare/master...forivall:fix/split-ctor-instance-types

wait, nvm, i've still got more tweaks to do.

Oh gotcha I see what you're up to! Love what you've been able to do thus far. If we're able to get some stuff into our type tests, I'd appreciate that as well! I think once you have more tweaks (as you'd mentioned, I didn't have any notes at a first glance), I'll be able to review the PR today and we can probably get it moved forward.
Thanks so much for jumping in and helping! :D

:rocket: Issue was released in v0.10.0 :rocket:

:rocket: Issue was released in v0.10.0 :rocket:

Still having this issue, you could use the require statement to fix it but it's bad practice, no other library has this issue

"no other library has this problem" lol ok. PRs welcome, I'll review them myself :)

                   🕯
          🕯              🕯
    🕯                         🕯

🕯         PRs welcome    🕯
    🕯                               🕯
          🕯              🕯
                    🕯

For the record, I've just encountered this behaviour when updating jimp version - for me solution was to replace
import Jimp from "jimp";
with
import { Jimp } from "jimp";
So it's not the way documentation suggests.
Hope it helps someone :)

Good catch. just pushed an update to the docs


Support for Additional Social Media Platforms

Enhance the Social Media Downloader by expanding its capabilities to include support for multiple popular social media platforms, allowing users to download videos, images, or other media with ease.

Proposed Platforms and Functionalities

Twitter

Input: Valid Twitter video URL.
Processing: Use yt_dlp or a suitable scraping tool to extract and download video content.
Output: Save the video in the preferred format and directory.

Reddit

Input: Reddit post URL containing videos or images (e.g., from subreddits like r/videos).
Processing: Extract the media content using praw or a scraping library.
Output: Download the video or images and save them in the desired directory.

LinkedIn

Input: LinkedIn post URL containing media content.
Processing: Use BeautifulSoup or API-based methods to locate and extract video or image URLs.
Output: Download the media to the specified folder.

Pinterest

Input: Pinterest pin URL with downloadable media.
Processing: Use a scraping library to parse the pin and extract media links.
Output: Download the image or video to the media directory.

Snapchat (Public Stories/Spotlight)

Input: Snapchat story or Spotlight video URL.
Processing: Use specialized tools or APIs to retrieve the media files.
Output: Save the downloaded media in the defined folder.

Tumblr

Input: Tumblr post URL with video or images.
Processing: Leverage Tumblr API or scraping methods to extract media links.
Output: Download the media to the specified directory.

Steps to Implement

Research available APIs or libraries (yt_dlp, praw, etc.) for the above platforms.
Create individual downloader functions for each platform, integrating them into the main script.
Add error handling and notifications for unsupported or inaccessible URLs.
Extend the CLI interface with new menu options for each platform.
Update the download log system to include platform-specific data for better tracking.

Benefits

Provides a comprehensive, one-stop solution for media downloading.
Reduces the need for separate tools for each platform.
Increases the app's utility and appeal for a wider audience.
Streamlines user experience across multiple social media platforms.

Thank you, @tatianakuznetsova2, for your detailed suggestion and enthusiasm for enhancing the Social Media Downloader! 🙌
Your proposal to expand the downloader's capabilities to include platforms like Twitter, Reddit, LinkedIn, and more is exciting. Here's my current stance and thoughts on this:
Current Focus
At present, my goal is to ensure stability, performance, and bug fixes for the platforms currently supported (YouTube, Instagram, Facebook, TikTok). While I don't have immediate plans to implement additional platforms, I agree that these ideas would significantly improve the tool's utility.
Future Possibilities
I am open to revisiting these features in future updates. Platforms like Twitter and Reddit are particularly interesting additions that I may prioritize when I have the bandwidth to work on them.
Fork & Contribute
Since this project is open-source, I encourage you (or anyone interested) to fork the repository and work on implementing these features. I’d be happy to review and accept a pull request if the implementation aligns with the project's structure and coding standards.
Suggested Workflow for Contributions

Fork the Repo: Clone the repository and work on the proposed features.
Implement and Test: Ensure new platform integrations are robust, with proper error handling.
Pull Request: Submit a detailed pull request describing the changes, new features, and testing performed.
Collaboration: I’ll work with contributors to refine and integrate the changes.

Long-Term Vision
Your feedback has inspired me to think beyond the current roadmap, and I'll keep this in mind for future updates. Please feel free to continue sharing your suggestions or ideas!
Thank you again for your support and contributions to making Social Media Downloader better for everyone. 🚀
Warm regards,
Nayan Das
GitHub: nayandas69
Email<EMAIL_ADDRESS>

Thank you, @tatianakuznetsova2, for your detailed suggestion and enthusiasm for enhancing the Social Media Downloader! 🙌
Your proposal to expand the downloader's capabilities to include platforms like Twitter, Reddit, LinkedIn, and more is exciting. Here's my current stance and thoughts on this:
Current Focus
At present, my goal is to ensure stability, performance, and bug fixes for the platforms currently supported (YouTube, Instagram, Facebook, TikTok). While I don't have immediate plans to implement additional platforms, I agree that these ideas would significantly improve the tool's utility.
Future Possibilities
I am open to revisiting these features in future updates. Platforms like Twitter and Reddit are particularly interesting additions that I may prioritize when I have the bandwidth to work on them.
Fork & Contribute
Since this project is open-source, I encourage you (or anyone interested) to fork the repository and work on implementing these features. I’d be happy to review and accept a pull request if the implementation aligns with the project's structure and coding standards.
Suggested Workflow for Contributions

Fork the Repo: Clone the repository and work on the proposed features.
Implement and Test: Ensure new platform integrations are robust, with proper error handling.
Pull Request: Submit a detailed pull request describing the changes, new features, and testing performed.
Collaboration: I’ll work with contributors to refine and integrate the changes.

Long-Term Vision
Your feedback has inspired me to think beyond the current roadmap, and I'll keep this in mind for future updates. Please feel free to continue sharing your suggestions or ideas!
Thank you again for your support and for sharing such a thoughtful suggestion to make Social Media Downloader better for everyone. 🚀
Warm regards,
Nayan Das
GitHub: nayandas69
Email<EMAIL_ADDRESS>


Use PropTypes package

Getting this:
Warning: Accessing PropTypes via the main React package is deprecated. Use the prop-types package from npm instead.

This appears to be fixed on master, but not in the latest version (2.0.12).

Hi @timscott, thx for your feedback.
You're right, I forgot to release it.
The 2.0.15 version, is now warning-free.


Remote access to phy and our data with docker and Jupyter

[ ] Create a docker image based on a Jupyter image with phy and vispy also installed (starting point here)
[ ] In the image, mount a special shared folder in read-only mode (shared between the container and the host). This folder contains a list of notebooks, and symlinks to some data. There might be some tweaks to do in phy to make it work (e.g. the kwik file is currently opened in a mode -- that might fail on a read-only volume).
[ ] Set up a public-facing server exposing the password-protected notebook server (running in the docker image), such that http://.../notebooks/mynotebook.ipynb opens directly the requested notebook
[ ] [Upload the image on dockerhub (create a kwikteam org account if possible?)]

Now, to share a notebook, you can do this:

Copy the notebook and data (or use symlinks) to the shared volume
Send the notebook URL to someone
Now, someone else can go to the URL and see and run the notebook

This is only for private sharing. For public sharing (lower priority) we can use tmpnb; in this case, a new temporary docker image with a fixed set of predefined notebooks is run for every new user.

@nippoo what's the status of this? we can close, right?

Ah. The thought was to integrate this with Thunder / CodeNeuro right?

i think so... closing YAGNI for now?

yeahhhh.


github actions

@antoninbas  Any way to stop github from checking travis anymore? Any settings that needs to be changed ?


[Blog] Assigning Node Metadata to pods

https://deploy-preview-1161--kyverno.netlify.app/blog/2024/02/19/assigning-node-metadata-to-pods/
Related issue
Proposed Changes
Adds a new blog entitled "Assigning Node Metadata to pods"
Checklist

[x] I have read the contributing guidelines.
[x] I have inspected the website preview for accuracy.
[x] I have signed off my issue.

Sorry for the noise, folks. We need to figure out how to allow requesting review from the maintainers group without having to approve every message.

No other reviews provided. Merging.

/cherry-pick release-1-11-0


Identifiers validation failed for dataset Colección de Reptiles del Instituto de Investigación de Recursos Biológicos Alexander von Humboldt (IAvH-R)

Identifier validation failed for the dataset Colección de Reptiles del Instituto de Investigación de Recursos Biológicos Alexander von Humboldt (IAvH-R):

Crawler attempt: 355
Publishing organization: Instituto de Investigación de Recursos Biológicos Alexander von Humboldt
Publishing country: COLOMBIA
Installation: IPT IAvH
Number of occurrences indexed for this dataset at the time of the issue: 8786
Cause: GBIF ID problems exceed 50% threshold: 100% duplicates; 9383 total records; 9383 absent records
Pipelines execution steps: http://api.gbif.org/v1/pipelines/history/execution/3526312/step

New IDs sample:

Old IDs sample:

IAvH:COLECCIONES:REPTILES:IAvH-R-9270
IAvH:COLECCIONES:REPTILES:IAvH-R-9056
IAvH:COLECCIONES:REPTILES:IAvH-R-9059
IAvH:COLECCIONES:REPTILES:IAvH-R-9060
IAvH:COLECCIONES:REPTILES:IAvH-R-9061
IAvH:COLECCIONES:REPTILES:IAvH-R-9062
IAvH:COLECCIONES:REPTILES:IAvH-R-9063
IAvH:COLECCIONES:REPTILES:IAvH-R-9064
IAvH:COLECCIONES:REPTILES:IAvH-R-9090
IAvH:COLECCIONES:REPTILES:IAvH-R-9057

Publisher email
Hello,
I am contacting you from the GBIF Secretariat about a dataset published by the Colección de Reptiles del Instituto de Investigación de Recursos Biológicos Alexander von Humboldt (IAvH-R) : https://doi.org/10.15472/bg0gnd.
We noticed that the occurrenceIDs were changed. We have temporarily paused the ingestions of this dataset.
As you might already know, when an occurrence record has a new occurrenceID for a given dataset, our system considers it to be a new occurrence.
This means that it will be given a new gbifid and a new occurrence URL (like this one: https://www.gbif.org/occurrence/1252968762) and the old gbifid and URL will be deprecated.
In this case, this means that the occurrence URLs would be deprecated when ingesting the newest versions of these datasets.
We would like to check with you if those changes were intentional. Do you know if this is the case?
Please let us know, thanks! We are happy to resume the dataset ingestion.
Note that some users rely on those occurrence URLs and gbifids (like https://bionomia.net for example). In an attempt to improve
the stability of the occurrence URLs and gbifids, we have implemented a warning system to detect these type of changes in datasets (see this news item).
If the data publisher can provide us with a list of old and new occurrenceIDs per record, we can avoid the identifier and URL changes. Could that be an option?
Please let us know if you have any question. Thanks!
All the best,

You can skip/fix identifier validation using the registry UI.

@EstebanMH-SiB

Identifier validation still  failing for the dataset Colección de Reptiles del Instituto de Investigación de Recursos Biológicos Alexander von Humboldt (IAvH-R):

Crawler attempt: 380
Publishing organization: Instituto de Investigación de Recursos Biológicos Alexander von Humboldt
Publishing country: COLOMBIA
Installation: IPT IAvH
Number of occurrences indexed for this dataset at the time of the issue: 8786
Cause: GBIF ID problems exceed 50% threshold: 100% duplicates; 9383 total records; 9383 absent records
Pipelines execution steps: http://api.gbif.org/v1/pipelines/history/execution/3997546/step

New IDs sample:

Old IDs sample:

IAvH:COLECCIONES:REPTILES:IAvH-R-9270
IAvH:COLECCIONES:REPTILES:IAvH-R-9056
IAvH:COLECCIONES:REPTILES:IAvH-R-9059
IAvH:COLECCIONES:REPTILES:IAvH-R-9060
IAvH:COLECCIONES:REPTILES:IAvH-R-9061
IAvH:COLECCIONES:REPTILES:IAvH-R-9062
IAvH:COLECCIONES:REPTILES:IAvH-R-9063
IAvH:COLECCIONES:REPTILES:IAvH-R-9064
IAvH:COLECCIONES:REPTILES:IAvH-R-9090
IAvH:COLECCIONES:REPTILES:IAvH-R-9057

Publisher email
Hello,
I am contacting you from the GBIF Secretariat about a dataset published by the Colección de Reptiles del Instituto de Investigación de Recursos Biológicos Alexander von Humboldt (IAvH-R) : https://doi.org/10.15472/bg0gnd.
We noticed that the occurrenceIDs were changed. We have temporarily paused the ingestions of this dataset.
As you might already know, when an occurrence record has a new occurrenceID for a given dataset, our system considers it to be a new occurrence.
This means that it will be given a new gbifid and a new occurrence URL (like this one: https://www.gbif.org/occurrence/1252968762) and the old gbifid and URL will be deprecated.
In this case, this means that the occurrence URLs would be deprecated when ingesting the newest versions of these datasets.
We would like to check with you if those changes were intentional. Do you know if this is the case?
Please let us know, thanks! We are happy to resume the dataset ingestion.
Note that some users rely on those occurrence URLs and gbifids (like https://bionomia.net for example). In an attempt to improve
the stability of the occurrence URLs and gbifids, we have implemented a warning system to detect these type of changes in datasets (see this news item).
If the data publisher can provide us with a list of old and new occurrenceIDs per record, we can avoid the identifier and URL changes. Could that be an option?
Please let us know if you have any question. Thanks!
All the best,

You can skip/fix identifier validation using the registry UI.

Identifier validation still  failing for the dataset Colección de Reptiles del Instituto de Investigación de Recursos Biológicos Alexander von Humboldt (IAvH-R):

Crawler attempt: 383
Publishing organization: Instituto de Investigación de Recursos Biológicos Alexander von Humboldt
Publishing country: COLOMBIA
Installation: IPT IAvH
Number of occurrences indexed for this dataset at the time of the issue: 8786
Cause: GBIF ID problems exceed 50% threshold: 100% duplicates; 9383 total records; 9383 absent records
Pipelines execution steps: http://api.gbif.org/v1/pipelines/history/execution/4052613/step

New IDs sample:

Old IDs sample:

IAvH:COLECCIONES:REPTILES:IAvH-R-9270
IAvH:COLECCIONES:REPTILES:IAvH-R-9056
IAvH:COLECCIONES:REPTILES:IAvH-R-9059
IAvH:COLECCIONES:REPTILES:IAvH-R-9060
IAvH:COLECCIONES:REPTILES:IAvH-R-9061
IAvH:COLECCIONES:REPTILES:IAvH-R-9062
IAvH:COLECCIONES:REPTILES:IAvH-R-9063
IAvH:COLECCIONES:REPTILES:IAvH-R-9064
IAvH:COLECCIONES:REPTILES:IAvH-R-9090
IAvH:COLECCIONES:REPTILES:IAvH-R-9057

Publisher email
Hello,
I am contacting you from the GBIF Secretariat about a dataset published by the Colección de Reptiles del Instituto de Investigación de Recursos Biológicos Alexander von Humboldt (IAvH-R) : https://doi.org/10.15472/bg0gnd.
We noticed that the occurrenceIDs were changed. We have temporarily paused the ingestions of this dataset.
As you might already know, when an occurrence record has a new occurrenceID for a given dataset, our system considers it to be a new occurrence.
This means that it will be given a new gbifid and a new occurrence URL (like this one: https://www.gbif.org/occurrence/1252968762) and the old gbifid and URL will be deprecated.
In this case, this means that the occurrence URLs would be deprecated when ingesting the newest versions of these datasets.
We would like to check with you if those changes were intentional. Do you know if this is the case?
Please let us know, thanks! We are happy to resume the dataset ingestion.
Note that some users rely on those occurrence URLs and gbifids (like https://bionomia.net for example). In an attempt to improve
the stability of the occurrence URLs and gbifids, we have implemented a warning system to detect these type of changes in datasets (see this news item).
If the data publisher can provide us with a list of old and new occurrenceIDs per record, we can avoid the identifier and URL changes. Could that be an option?
Please let us know if you have any question. Thanks!
All the best,

You can skip/fix identifier validation using the registry UI.


Kruskal's Algorithm

Overview

Write code for Kruskal's Algorithm in C++.
Add that code into a directory named "Graphs", if the directory doesn't exist, create one and add the code file.

Please take care of these things while creating the pull request.

The code file was added to the correct directory.
Code quality looks good.
The programming language used is the same as mentioned in the issue.

Please assign me this issue.

Can I work on this issue??

Hey @muskaanrajput14,
This issue has already been assigned.


[feature request] can provide a parameter to save the historical naming replacement rules

can provide a parameter to save the historical naming replacement rules
and can give this rule a name.
which I find very useful. :)
Thank you to developed this tools, I love it. 🥰

Thanks @bindiry, I'm glad you find this tool useful.
I am not sure if I understood your request correctly. If you want to repeat previous command or handle your history, it may be better to use just other tools in your shell. For example:

Creating aliases
Wrapping rnr in a custom script
Using utilities that handle arbitrary command history (e.g. zsh-history-substring-search)

Feel free to open it again if these suggestions do not solve your problem.


Now Playing Panel Not Loading

After some serious using the party website, the only major problem is the Now Playing panel vanishing. We think we have it narrowed down as to the possible cause(s). These do not cause the Now Playing panel to disappear every time,

When a song is upvoted It will cause that or the next song to clear the Now Playing panel. Refreshing will bring it back, but that takes about a minute since the chat page is huge. (sorry, we talk too much..lol) There are situations where some listeners are seeing the Now Playing panel while others do not.

Sometimes when numerous songs are being added, some will sporadically turn off Now Playing.

After the song that is not being displayed is completed the next song will start normally.
Hope this helps the ones working on this portion of the program. Let me know if I need to keep an eye out for any specifics to look for.
Thanks,
Alan (FreeForm Radio)

One other thing that I've noticed is when logged in on two computers, the Now Playing panel does not go away. You just have to click the resume button.
Any guesses on when someone will have a chance to work on this? :-)


[BEAM-9980] use constant for job_Inventory.groovy and do not use hardcoded python versions

use python versions constant for job_Inventory.groovy.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @tvalentyn


Another list index out of range

In the following JSON code a closing curly bracket is missing (and the referenced parameter does not exist also).
"param2" : "${IAMNOTEXISTING",
This causes the following error message:
'list index out of range'
Why this? A list is not involved.
Expected is an error message telling about a missing closing bracket.

Not related to missing brackets only.
Same error message also caused by empty expression: "param2" : "${}"
Without quotes around the expression "param2" : ${} the error message changes to
Expecting value:
This is misleading. It should be possible to detect empty brackets.
I would expect an error message like:
Empty dollar operator expression (${}) detected in line ...

Hello Holger,
This issue was fixed in stabi branch (namsonx/task/stabi_branch)
Current error message is:
Exception: Invalid nested parameter format in line: "param2" : "${IAMNOTEXISTING",

Thank you,
Son

Hi Son,
retest successful.
But please remove the term "nested" from error message:
Exception: Invalid nested parameter format in line: "param2" : "${IAMNOTEXISTING",
Because there is no nested parameter.
My understanding is, that this
"${IAM${NO${TEX}IST}ING"
would be a nesting of parameters.

Hello Holger,
Thank you for the hint, I will update.
Thank you,
Son

Hi @namsonx ,
what is the status of this ticket?
Thank you,
Thomas

Hello Thomas and Holger,
I created new commit 6cabd822 on stabi branch to update error message log, I also modified the log template of testcases ID: JPP_0350 - JPP_0354
Thank you,
Son

already fixed, Holger gives feedback.

Retest successful. Issue can be closed.


Not downloading

Ok so it got to 47 percent and would'nt go any farther, hm, i also have 2 Mbps of internet speed sooo?

The snaps are hosted on the Fastly CDN, so your best bet is to go to http://fastly-debug.com and seek their customer service for help.


Interprocess communication - the right way

Hi!
I use telegram-bot-ruby in an application that consists of two main parts - telegram API workers and a rails app. They interact via RabbitMQ. There is no information in documentation about sending Bot::Client received messages via rabbit, so in my proof-of-concept I use standard ruby object marshalling. It seems to be a great overhead, and I'm pretty sure that I can use some internal method for extracting raw message data and creating message objects manually from that raw data on the other side . So the question is, what solution for sending messages between processes would you recommend?
Thank you!

@ivanovaleksey @atipugin help please :)

Hi @govorov, sorry for being silent.

I'm pretty sure that I can use some internal method for extracting raw message data and creating message objects manually from that raw data on the other side.

You are right about that. If you have raw response from Telegram API you can use the same logic as Client#fetch_updates does. Create Update object and extract message from the update.
For more information check Client#fetch_updates source code.
That being said I don't know details about your application. Are you going to send via RabbitMQ raw message (say it would JSON) from Telegram workers to Rails app? And then create a message object on the Rails side and work with it?
Just FYI: there can be another option.
For example, on Telegram worker's side you can extract from the message useful information and send it via RabbitMQ. This way you can simplify logic on Rails-app side (but this would be present in worker), reduce traffic (if useful payload is small part of the whole message), and, for instance, push plain messages to one queue but inline queries to another one.
I am not sure about what solution would be the best and can't really recommend you to use (or not to use) any of this.


Change database on the fly

Hi everbody,
first of all thank you for this great piece of software.
I wonder if it is possible to change the database on the fly so HTTP requests go to the appropriate db in backend.
imagine a web application which is multi tenant. I create a database for every tenant. now if somebody hits the url http://tenantsubdomain.awesome-app.com/rpc/login I want to switch to the right database named tenantsubdomain in this case.
any ideas or other recommendations?

other options that come to my mind would be:

create one schema per tenant rather than database
use one schema for all tenants (with proper ACL and RLS probably easily doable)

you can do that like this:
each customer has a database
to each database, you connect a dedicated postgrest instance (id does not use a lot of memory),
then, in front of all of them, you have one instance of openresty. there you have a location where you define a proxy_pass to an upstream config.
now here comes the interesting part. inside the upstream definition, since it's openresty, you can have a custom lua script that decides the IP/port or the upsteam postgrest instance where to send the request.
this is how you do it.

here is how an upstream definition looks like
https://github.com/subzerocloud/postgrest-starter-kit/blob/master/openresty/nginx/conf/includes/http/postgrest_upstream.conf
(please close)

@ruslantalpa,
I think I will stick to one schema per tenant since it makes it easier to create new tenants via trigger functions rather than manually have to create a database, migrate or fill it and then define an upstream in openresty. of course this only works if I really cannot get access to header.host of the HTTP within my functions (I assume I can).
so the concept would look like this:

tenant checks for available subdomain in SaaS application (which will be my tenant name)
if postgrest function returns true (in "available") then tenant can sign up to get his subdomain
this request will go to a function which creates a schema, a role, and all necessary grants for that role and schema.
finally give new role grants to authenticator

with this I end up having one schema per tenant. and web users sharing one tenant role in claim. (still able to identify single user by claim "email" for example).
does this sound sane to you guys?

it's sane but you are missing a crucial step. postgrest is tied to a particualr schema so usually you would have to boot up a new postgrest instance for each customer .... but ... due to a "hack" i would say ... you might be in luck
if you specify to postgrest the schema as an empty string, then whenever it will generate a quuery, it will not fully qualify it, giving you the option to use search path.
so, do all the things that you said but when you start postgrest, give the schema as an empty string, and for each customer/dbuser that you create, make sure you set for him the apropriate search path so that the queries he generates are routed to his custom schema.
i  think you rusecase is doable

alright. thank you very much for your help. :+1:

another example that comes to my mind is:

start postgrest with default schema rather than empty string
use SET search_path to "schema of tenant" to update underlying table in appropriate table of customer (schema)
and use only the views defined in this default schema as postgrest documentation explains.

with this I don't have to SET a schema within my "PRE_REQUEST" procedure. I just SET a schema in every view of my public exposed schema.

This will not work, views can not dinamically change the source table.

yes you are right. I think then it is not possible what I want to achieve.
because setting the schema beforehands (before the request comes in) with a pre_request, sets the schema to $tenantSubDomain. and now I would have to create views in every schema of a tenant. basically the same sql schemas I am using in the boilerplate postgrest starterkit, I would have to execute them after every signup for every schema.
but here is the problem: the views AND data would be in one schema. so there would be no isolation of data. right?

so the only option is to really start a new postgrest instance for every tenant. same database, but different schema.

which still wouldn't solve the isolation problem. I would need two schemas for each tenant. one publicly exposed schema and one for his tables (hidden tables I reach via views in the public schema of the tenant)
so the more easier way would be to start a new postgrest instance with a different database. this results in one database per tenant.

more thoughts on this:
one database per tenant sounds like a lot of maintenance issues to me.
imagine I want to update my SaaS application with an additional view in the public schema. I would have to update every postgrest instance. (sure I could use a central repo for all my sql files, so I don't have to deal with N tenant files.)
beside that I would have to setup each subdomain in the reverse proxy by hand (or automated, but still something would have to be done on the infrastructure).
what do you guys think of all this? what would be the best practice here for me to accomplish my goal?

why do you need to isolate customer data, each with his own schema in the first place?

Because I dont feel good putting all tenants into same tables. Also to be able to scale up/out in tge future.

feeling good is not a technical reason
you don't have either an app or users right now to think about scaling
don't complicate things and use normal tables, views, rls.
the only real reason to use schema separation is when the data model for each customer will be different (different columns/tables), almost every other reason is a mistake.

Ok I will start with all tenants in one db. Still I will make use of the subdomain somehow by asserting the json web token claim eg "tenant" equals the X-Shbdomain header set by openresty. And several other securities. I think RLS is enough to realize tenant data isolation.

Logical or physical data isolation for different customers is an almost always present security requirement for enterprise applications and with postgresql, using different schemas to host data for different tenants is the most common way of complying with the requirement. Supporting multiple schemas that clients can switch between per request would open this up to a whole new suite of applications.


#59 Add commands to clear and assign global roles to a user

#59 Add commands to clear and assign global roles to a user

We could also make use of new commands in cy.createUser. Could be done however in a new PR.


[tevclient] Add tevclient port

[x] Changes comply with the maintainer guide.
[x] The name of the port matches an existing name for this component on https://repology.org/ if possible, and/or is strongly associated with that component on search engines.
[x] Optional dependencies are resolved in exactly one way. For example, if the component is built with CMake, all find_package calls are REQUIRED, are satisfied by vcpkg.json's declared dependencies, or disabled with CMAKE_DISABLE_FIND_PACKAGE_Xxx.
[x] The versioning scheme in vcpkg.json matches what upstream says.
[x] The license declaration in vcpkg.json matches what upstream says.
[x] The installed as the "copyright" file matches what upstream says.
[x] The source code of the component installed comes from an authoritative source.
[x] The generated "usage text" is accurate. See adding-usage for context.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is in the new port's versions file.
[x] Only one version is added to each modified port's versions file.

@microsoft-github-policy-service agree

Regarding the CMakeLists.txt file of this library, I have a few questions:

add_library(tevclient STATIC)

Does this library not support dynamic builds? If it does not, please add "supports": "static" in vcpkg.json.

target_link_libraries(tevclient PRIVATE wsock32 ws2_32)

Does this library support UWP? If it does not, please add "supports": "!uwp" in vcpkg.json.

add_library(tevclient STATIC)

Does this library not support dynamic builds? If it does not, please add "supports": "static" in vcpkg.json.

Even if it does not, DO NOT add "supports": "static" in vcpkg.json....
Use vcpkg_check_linkage(ONLY_STATIC_LIBRARY) at the top of the portfile.


add asset header font size options

Asset header is now different from the main dataset accordion title

@Thuhaa Looks fine. Please fix a thing I commented regarding to storybook setting before merging.

done