Remote queue manager should deal with blips in upstream latency

Currently the queue is per shard and only buffers a 100k samples; if you have a remote target that is usually very fast, but periodically slow, we will have one shard that can easily overwhelm its queue.
We could consider having a globally fixed length queue that gets partitioned by the number of shards.   Or we could make resharding happen when the queue is overwhelmed.  Or we could just accept dropping samples in this case.

More notes on what I was seeing.  The time to call my remote write was typically 80ms (0.999), 20ms (0.99), and 5ms (0.9).  Occasionally I would see blips where my 0.999 quantile was over 1s.
Since I've switched to using Undertow as my embedded server, I'm seeing far fewer blips to 1s.  I've also added more scrape targets and I would say that Prometheus is now actively adjusting shard counts.  I'm seeing no dropped records.  If this is due to the fewer blips, the change to Undertow, or the more active sharding in Prometheus, I have no idea.
I'll report more some next week after this has had a chance to soak in for a while.

So things are generally running better now.  I have 500,000 timeseries and I see about 300 remote write requests per second (almost 2 million samples per minute).  I still see occasional blips but they are occurring much less frequently.  My shards seem to bounce around between 4-6.
However, I've also noticed that when I do a configuration reload (POST to /-/reload) that my shards gets reset to 1.  Does Prometheus not save this information between runs?  When it gets reset to 1 I go through the process of losing more write requests until it works its shards back up again.

Yeah, config reloads are still fairly intrusive for the remote write path,
as that all gets completely flushed/reloaded on config reload. See also
this comment:
https://github.com/prometheus/prometheus/blob/master/storage/remote/write.go#L36-L37
On Thu, May 4, 2017 at 4:54 PM, Daniel Shiplett<EMAIL_ADDRESS>wrote:

So things are generally running better now. I have 500,000 timeseries and
I see about 300 remote write requests per second (almost 2 million samples
per minute). I still see occasional blips but they are occurring much less
frequently. My shards seem to bounce around between 4-6.
However, I've also noticed that when I do a configuration reload (POST to
/-/reload) that my shards gets reset to 1. Does Prometheus not save this
information between runs? When it gets reset to 1 I go through the process
of losing more write requests until it works its shards back up again.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/prometheus/issues/2648#issuecomment-299209558,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAg1mCVxhRlSEXDLWmlt9zXEE8zSpuxNks5r2eaQgaJpZM4NEf6-
.

We should indeed optimise this bit, it shouldn't be too hard.
On Thu, May 4, 2017 at 6:28 PM Julius Volz<EMAIL_ADDRESS>wrote:

Yeah, config reloads are still fairly intrusive for the remote write path,
as that all gets completely flushed/reloaded on config reload. See also
this comment:
https://github.com/prometheus/prometheus/blob/master/storage/remote/write.go#L36-L37
On Thu, May 4, 2017 at 4:54 PM, Daniel Shiplett<EMAIL_ADDRESS>wrote:

So things are generally running better now. I have 500,000 timeseries and
I see about 300 remote write requests per second (almost 2 million
samples
per minute). I still see occasional blips but they are occurring much
less
frequently. My shards seem to bounce around between 4-6.
However, I've also noticed that when I do a configuration reload (POST to
/-/reload) that my shards gets reset to 1. Does Prometheus not save this
information between runs? When it gets reset to 1 I go through the
process
of losing more write requests until it works its shards back up again.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<
https://github.com/prometheus/prometheus/issues/2648#issuecomment-299209558
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AAg1mCVxhRlSEXDLWmlt9zXEE8zSpuxNks5r2eaQgaJpZM4NEf6-
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/prometheus/issues/2648#issuecomment-299253898,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAbGhYsObO7A3wrq-JqB5NVkqPAC-6Fqks5r2grAgaJpZM4NEf6-
.

It looks like the queue configuration was addressed in https://sourcegraph.com/github.com/prometheus/prometheus/-/commit/454b6611458d01398678baca372a57973df4215f and then documented in https://github.com/prometheus/prometheus/pull/4126

I'd say we can close this: the new WAL based remote_write code in 2.8 includes a much more proactive resharding code that responds to upstream latency variations very quickly.


Drop CJS support

Currently, we are compiling Hydrogen to both CJS and ESM. It would be much simpler and faster to use only ESM.
In tsconfig.cjs.json we explicitly include src/framework to be compiled to CJS. However, the whole project ends up being compiled to CJS because in src/framework we are importing types from other folders (and TSC follows imports). This has two issues:

It's slower: we compile files to CJS that we are not using later.
It can break: targeting CJS in files that are supposed to be ESM can crash the TSC process. For example, it breaks when encountering import.meta (cc @cartogram )

We need CJS at the moment because Vite compiles "import" statements in vite.config.js to "require" in projects (user apps) that don't have "type": "module" in package.json. Therefore, everything that is imported in vite.config.js (plugins) needs to be in CJS.
If we want to use ESM only, we would need to add "type": "module" to both Hydrogen package and Hydrogen apps. In this scenario, Vite would not use "require" in vite.config.js. However, this has two problems:

ESM complains about missing file extensions. This can be worked around by using the flag --experimental-specifier-resolution=node when running Vite.
ESBuild crashes due to this issue. No workaround so far.

ESBuild crashes due to https://github.com/evanw/esbuild/issues/1921. No workaround so far.

It looks like there's a PR pending to fix it: https://github.com/evanw/esbuild/pull/2067

Vite appears to have just merged a workaround for the ESBuild issue above.

I was looking at going through the codebase and adding all the exact file extensions to address issue 1 above, but I think I'm running into an issue here:
Take this export from src/index.ts for example:
export {useShop} from './foundation/useShop';

If we want to add the complete file extension so that it's an actual file extension:
// TS error: An import path cannot end with a '.ts' extension. Consider importing './foundation/useShop/index.js' instead. ts(2691)
export {useShop} from './foundation/useShop/index.ts';

So we can't end the file extension with .ts.
Ok, so what if we end it in .js, which appears to be the recommended TS way? (There are a TON of issues in the TS github repo about this, but here's a recent example of it: https://github.com/microsoft/TypeScript/issues/49083 )
// "./foundation/useShop/index.js" is not found. eslint[node/no-missing-import](https://github.com/mysticatea/eslint-plugin-node/blob/v11.1.0/docs/rules/no-missing-import.md)
export {useShop} from './foundation/useShop/index.js';

ESLint isn't happy about the fact that you're trying to import a non-existent file, which is quite reasonable to be honest.
I explored this because I wanted to see if we could address issue 1 without using an experimental flag, but the solution seems to be either use the flag, or disable ESLint's error checking for this type of error so that TS can have its way.
🤷

@frehner Thanks for trying that! Yeah, I think we will need to disable this error in ESLint. I don't like using .js when we have TS but I'm aware that the TS team recommends it so... 🤷
The experimental flag will be removed at some point so we should not rely on it...

Do not rely on this flag. We plan to remove it once the Loaders API has advanced to the point that equivalent functionality can be achieved via custom loaders.

@frehner Thanks for trying that! Yeah, I think we will need to disable this error in ESLint. I don't like using .js when we have TS but I'm aware that the TS team recommends it so... 🤷
The experimental flag will be removed at some point so we should not rely on it...

Do not rely on this flag. We plan to remove it once the Loaders API has advanced to the point that equivalent functionality can be achieved via custom loaders.

I actually don't mind using .js for export index files. Makes it easy to spot exports vs actual components/utils..

I'm not a fan of the file extension .js when the dev is working with .ts files. Seems to me like a possible cause of confusion.

Agreed, I don't think anyone likes it. But it must be done that way, sadly.


Applying autofixes uses wrong line endings / should honour .editorconfig

.editorconfig:
[*.cs]
end_of_line = lf

Example.cs:
namespace Bug
{
    public static class Example
    {
        public static int MissingNewLine(bool condition)
        {
            if (condition)
            {
                return 42;
            } // SA1513: Closing brace should be followed by a blank line
            return 0;
        }
    }
}

Applying the automatic fix for SA1513 (and probably a bunch of others, but this is the one where I tracked down the problem) inserts a CRLF (carriage return + line feed) combination, but the file uses LF only. In Visual Studio, you can observe the state of line endings for the current file in the bottom right corner of the file-editor pane. It should initially say LF, when the automatic fix is applied it will switch to MIXED instead:

The next time the file is opened (but not immediately when the problem is introduced!), Visual Studio will complain about mixed line endings, and suggest to 'normalize' the line endings (using the wrong default option, of-course):

The root-cause is probably because the fixer appears to use a hard-coded CRLF (SyntaxFactory.CarriageReturnLineFeed) rather than the currently configured line-end character.

There are about 40 places in the code fixers that uses SyntaxFactory.CarriageReturnLineFeed directly. Do you see any problem with replacing them with code that creates trivia from GetOption(FormattingOptions.NewLine, LanguageNames.CSharp), @sharwell?

For SA1513, I'd rather update it to match the newline already at the end of the line where the fix is being applied.


sql_exporter as linux service

Hi, is there a possibility to run this as a background service on Ubuntu 20.04?

Just add a sql-exporter.service file in /etc/systemd/system/
Here is a basic example
[Unit]
Description=SQL Exporter

[Service]
Restart=always
Type=simple
# The working directory contains your configuration files
WorkingDirectory=/full/path/to/working/directory/
ExecStart=/full/path/to/sql_exporter
User=the_running_user
Group=the_running_group

[Install]
WantedBy=multi-user.target

Start and enable it with systemctl enable --now sql-exporter.service


Update ListView.tsx _fieldRender - link validation

Q
A

Bug fix?
[ x]

New feature?
[ x]

New sample?
[ ]

Related issues?
fixes #1171

What's in this Pull Request?
Address this issue:   Empty link Url creates a clickable link
https://github.com/pnp/sp-dev-fx-controls-react/issues/1171
Add basic logic to test if Url property is a valid link structure.
I only tested the raw js in the compiled ListView.js file in my own project.
I reverse engineered the tsx which was pretty basic.
The validation logic could be prettier regex but this long-hand syntax does work and is easier to read.
But if someone could improve it further and validate there are no issues with the update, it would be great!
    // Check if the URL property is specified
    if (field.linkPropertyName) {
      return (item: any, index?: number, column?: IColumn) => { // eslint-disable-line @typescript-eslint/no-explicit-any

        // Testing to see if Url value is valid... has a value, is a string, and either starts with http or /sites/
        const isValid = item[field.linkPropertyName] && typeof item[field.linkPropertyName] === "string" &&
            ( item[field.linkPropertyName].indexOf("/sites/") === 0 || item[field.linkPropertyName].indexOf("http") === 0 ) ? true : false;

        // Return element as a link if the Url passes simple validation.  Else just return the displayed value as normal span
        return isValid === true ? {item[column.fieldName]} :
          {item[column.fieldName]};
      };

Hi @mikezimm , just checking if you need any help with the PR?

@mikezimm just looking into this in more detail and I believe the control already allows you to apply your validation without requiring changes.
This is the current code to render fields:

If a render property is provided, the component will not return a hyperlink, but instead, use the render function specified for that field. In your case, all you should be required to do is to recreate the same logic on the render method.
If you take as an example the render method highlighted below that is available on the ControlsTest file of this repository, you can just change it to include the validation that you require and return either a hyperlink or text.

Is it OK to close this PR as no longer required, or am I missing something?

Closing as no response for a long period. Can reopen if required


remove _extensible

I could not find any sense in _extensible field.
It adds a lot of problems with constructing structs and I don't see any way where it can be usefull.
cc @rust-embedded/tools
cc @japaric can you explain why you add this? Is this still needed?

The extensible field is used for future proofing. Its a "hack" until rust-lang/rust#44109 is implemented.

Closed in favour of #93.


Came across this article

https://gbhackers.com/cyber-threat-intelligence-tools/
It looks like they copied this list, thought it was worth mentioning here.

Thank you, @cloudtracer! I did find a reference to my personal LinkedIn and a link to this GitHub repository on that page, so I'm ok with it 🙂


Updated Openssl Doccument

Updated Openssl Doccument

Approved but please check documentt build, you can locally use yarn start to debug this


curl: disable SSLv3 only for known good libcurl versions

Disable SSLv3 by enforcing TLSv1.x for curl >= 7.34.0 and < 7.39.0.
Versions prior to 7.34 and at least when compiled against openssl
interpret this parameter as "limit to TLSv1.0" which fails for sites
which enforce TLS 1.1+.
Starting with curl 7.39.0 SSLv3 is disabled by default.
Set the user-agent using the curl options instead of creating a raw
header.
The curl version is calculated by splitting the version string at the dots, convert each number to hex and the concatenated hex number to decimal:
Version: 07.19.7
hex: 07 13 07
dez: 463623

:+1:

Using the string version number and the function version_compare() is probably easier to read. Hopefully, there is your detailed comment to describe the change.


Limit the number of pending send bytes in client

A limit can now be placed on the amount of pending data to be transmitted that is buffered in bytes. If there is no room the send call will return false to denote the caller needs to try sending it again later. The limit can be specified at a global level and also overridden on a per-client basis.

Not sure why semver is failing even though there are only additions. Chetan is there any hard restriction on the classes that new methods cannot be added.

Addition does not fail semver. I'll take a look.

Any updates guys @243826 or @vrozov

Pramod, can you check why it says that the server is broken? Also
separately can you look at how you can avoid volatile for keeping track of
bytes?
On Thu, Nov 5, 2015 at 10:24 AM, Pramod Immaneni<EMAIL_ADDRESS>wrote:

Any updates guys @243826 https://github.com/243826 or @vrozov
https://github.com/vrozov
—
Reply to this email directly or view it on GitHub
https://github.com/DataTorrent/Netlet/pull/36#issuecomment-154144465.

Sounds good

Chetan the build was failing because of this flag in pom.xml true
Made changes to avoid using volatile for normal counters and update counter visibility once in a while using a configurable timeout. Check it out.


Validate CSV

As a CSV editor
I want to run a validation against CSVlint
So I can improve the quality of my data

CSVlint API is validated here:
https://github.com/theodi/csvlint/wiki/Webservice
Suggest we create a Data URI from the file contents and create a validation that way. I think that's how file uploads work

Sub-tasks:

Send file for validation
Wait for validation
Handle validation results
Display in pane

Also:

Implement fail condition in file stream


Ability to preserve Status subresource on restored resources

Describe the solution you'd like
[A clear and concise description of what you want to happen.]
Some applications depend upon the Status subresource in Kubernetes for managing state. An example of this is the Kubernetes Service Catalog, which stores the state of a given Service Instance in the Status subresource. For OpenShift, a good example of this are ImageStreams which stores the dockerImageReference associated with that ImageStream in the status field.
While sometimes it is desired to wipe away the status before restoring a resource, we do not want to do this for every single resource. I recommend moving the function resetMetadataAndStatus (https://github.com/heptio/velero/blob/0fd7872ef48ce617e561e6e45f8ccb0f11637f58/pkg/restore/restore.go#L1079) inside of a plugin so that the user may optionally write a restore plugin for specific resources that preserves the status.
Optionally, this could be handled via a whitelist/blacklist on the restore's Custom Resource.

FYI, most (if not all) built-in resources don't allow you to restore status - when you create a new resource and it has status, the server-side code tends to wipe it out. For CRD instances, you can restore status as long as the CRD does not have the /status subresource enabled. If it does, you won't be able to restore status.

@ncdc Thank you for that information, I wasn't aware of that. Our main use case is interested in restoring provisioned service instances from the Kubernetes Service Catalog, so I will have to dig more into those CRDs to determine if we would be able to restore these.

After thinking this through more, it would suffice if the restore plugin simply had the ability to access the data on the /status subresource to perform some custom restoration logic. Can the restore code reset the status field after the restore plugins run?

That's coming in #1123 #1256

Excellent, thank you! I think that this issue can be closed then since the above will satisfy our concerns.


Controllers Structure Redesign

Let's redesign controllers structure, so they ain't duplicate themselves for day/week functionality.

Controllers re-structed.
Now day/week controllers moved to united file, that routed them to different handlers.


Input PURL does not match output PURL

Using the Package API to check for PURLs that are present and might have vulnerabilities, I discovered that for some PURLs like pkg:deb/, the actual correct name requires the debian namespace, which is missing.

@nikkuAg I think it is... @johnmhoran @TG1999 any input? Anyone working on this already?

@nikkuAg you are welcome to work on this.

Could you please explain this issue?

VulnerableCode already uses/provides the package namespace correctly so there is nothing to fix here. The work is needed in SCTK - https://github.com/nexB/scancode-toolkit/issues/3443


Payload not add to response

I replace
https://github.com/dialogflow/dialogflow-fulfillment-nodejs/blob/master/samples/actions-on-google/functions/index.js#L74-L87
With
  function other (agent) {
    const facebookPayload = {
      attachment: {
        type: 'template',
        payload: {
          template_type: 'generic',
          elements: [
            {
              title: 'Title: this is a title',
              image_url: `https://assistant.google.com/static/images/molecule/\
    Molecule-Formation-stop.png`,
              subtitle: 'This is a subtitle',
              default_action: {
                type: 'web_url',
                url: 'https://assistant.google.com/'
              },
              buttons: [
                {
                  type: 'web_url',
                  url: 'https://assistant.google.com/',
                  title: 'This is a button'
                }
              ]
            }
          ]
        }
      }
    }

    agent.add(new Payload(agent.FACEBOOK, facebookPayload))
  }

I expect response to have payload, but I got this response instead
{
  "fulfillmentMessages": [],
  "outputContexts": []
}

BTW It work with Card example but not payload, any hint?

Oh, I just realize that I need to use Facebook to get the payload, tested on DialogFlow won't show any.

@katopz I have the same issue. You mean just because it's FACEBOOK payload it won't show on DIAGNOSTIC INFO json? I also tried using postman but still blank fulfillmentMessages. Where can I check it without going to facebook?

The request needs to come from the platform (FB, Slack, etc) in order for the payload to appear


Context passing around in StatelessWidget

Why do we have to pass the context around in a StatelessWidget?
Improvement Question: Wouldn't it be better to allow also in a StatlessWidget the context everywhere? Or do I miss some point?
Bad example!  Context needs to be passed around in a StatelessWidget.
class ContextPassingAroundNeeded extends StatelessWidget {
  @override
  Widget build(BuildContext context) => method1(context);

  Widget method1(BuildContext context) {
    context.inheritFromWidgetOfExactType(Type); // whatever

    return method2(context);
  }

  Widget method2(BuildContext context) {
    context.inheritFromWidgetOfExactType(Object); // whatever
    return Container();
  }
}

In a StatefulWidget it is also possible without passing the context around, because the context variable is accessable everywhere in the state. It's nice however why isn't it also possible in a StatelessWidget?
class ContextPassingAroundNotNeeded extends StatefulWidget {
  @override
  _ContextPassingAroundNotNeededState createState() => _ContextPassingAroundNotNeededState();
}

class _ContextPassingAroundNotNeededState extends State<ContextPassingAroundNotNeeded> {
  @override
  Widget build(BuildContext context) => method1();

  Widget method1() {
    context.inheritFromWidgetOfExactType(Type); // whatever

    return method2();
  }

  Widget method2() {
    context.inheritFromWidgetOfExactType(Object); // whatever
    return Container();
  }
}

A stateless image is immutable, therefore no way to store the context anywhere when it becomes available.

I guess you would point out a stateless widget is immutable instead of an stateless image


Makefile add cover

Add cover block for tests and some changes in makefile

Codecov Report

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

@@           Coverage Diff           @@
##           master      #28   +/-   ##
=======================================
  Coverage   74.13%   74.13%
=======================================
  Files           4        4
  Lines          58       58
=======================================
  Hits           43       43
  Misses          8        8
  Partials        7        7

Continue to review full report at Codecov.

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


Trying to download a reddit page, all I get is a text file

Ripme version: 1.7.35
Java version:  9.0.1
Operating system:  Mac OS 10.13.5 Beta
Exact URL you were trying to rip when the problem occurred: https://www.reddit.com/r/funsized/
Please include any additional information about how to reproduce the problem:

Expected Behavior
Detail the expected behavior here.
I expected it to download the pictures as normal.
Actual Behavior
It downloaded a text file.
Detail the actual (incorrect) behavior here. You can post log snippets or attach log files to your issue report.

It seems you have the option "Save URLs only" checked. Go to the configuration tab and uncheck it

Duh, okay.  Thanks.


Chrome effectively freezes when closing its dev tools and Vue Devtools is installed

Vue devtools version
6.5.0
Link to minimal reproduction
https://sfc.vuejs.org/
Steps to reproduce & screenshots

Open the SFC Playground
Press F12 to open Chrome Devtools
Inside the Chrome Devtools, open the Vue panel
Make sure that Vue is detected
Press F12 again to close Chrome Devtools

What is expected?

I can continue to use Chrome like usual

What is actually happening?

CPU and Memory Usage increases immensely after closing the Chrome Devtools
As a result Chrome becomes unresponsive

System Info
Ubuntu Linux 22.04
Kernel 6.2.0-32-generic
Chrome 117.0.5938.88 (Official Build) (64-Bit)
Vue Devtools 6.5.0

Any additional comments?
This problem came up today while developing on my company's Vue powered web apps. One colleague can reproduce this issue while another one can't. I could not find this issue reported yet, hence me creating it now.
Does anyone else have this issue?
For now I need to uninstall Vue Devtools from Chrome and use Firefox instead to debug my Vue apps, because over there it works as expected.

I have the same issue, too.
And it works fine in Firefox.

Confirmed on both MacOS and Windows 11 as well

I'm having the exact same issue. Doesn't matter if closing the devtools in Edge or Chrome, both show the exact same behaviour as described by the reporter. Any help on this?

Soo because this topic still hasn't been touched by the developers yet I have found a rather primitive and temporary solution. That is to refresh the page each time the devtools are being closed. I found out that refreshing the page "resets" the memory leak. In order to automate this reload event I made a simple chrome extension (currently in review) that does this for me and only on specific sites (e.g. localhost).
refresher.zip

Seeing same problem in Brave on my Mac:
MacOS version: 14.1.1 (23B81)
Brave version: 1.60.118 Chromium: 119.0.6045.163 (Official Build) (arm64)
Vue devtools version: 6.5.1

any solutions yet ?

Same, vue devtools are unusable for me as of a few weeks ago. Switching routes takes 3 seconds of freeze

Experiencing the same with devtools 6.5.1 and latest chrome Version 120.0.6099.216 (Official Build) (x86_64)
Using firefox on the other hand works perfectly fine for me with 6.5.1 extension and firefox at 121.0.1
I am running the latest MacOS too

What I observe that it'll only freezes browser only if we close inspect tool window. If I not closing an inspect then vue tool works fines but after closing browser inspect it'll freezes so keep open your inspect tool

Affirmative.
In my case everything works fine until the hot reload is triggered. As soon as I change something in my code, it updates and I reload the page, the browser freezes intentionally.
I noticed in the htop output that one of the cores is 100% loaded at that point
Developers, please give feedback on this problem! Using chrome for vue development has become extremely difficult (if I need vue inspector, I have to open Firefox...).
Fedora 38
Kernel 6.5.5-200.fc38.x86_64
Version 121.0.6167.85 (Official Build) (64-bit)
Vue Devtools 6.5.1
Gnome 44.8 with Wayland

Can not use it... if you open it, it crashes chrome...

Do vue developers never use it? Why can such a serious problem exist for so long?

any solutions yet ?

I created this basic chrome extension tool that gives a temporary solution:
https://chromewebstore.google.com/detail/devtools-close-refresher/nfcfibfdokdooefjclehlhgojpmchlhp?hl=en-GB&authuser=0&pli=1

I'm seeing a similar thing using Chrome inspector and Vue dev tools the memory usage would increase really quickly, it's unusable for me at the moment. Chrome task manager reported the tab was using up to 8gb and then tab would crash.
Refreshing the tab didn't work very well for me either.
I've disabled Vue dev tools extension and memory usage is normal.
Ubuntu Linux 22.04
Chrome 121.0.6167.184
Vue Devtools 6.6.1

Project is using Vue3 with Vite 5.1.1

Maybe this will help someone.
Recently devtools has been updated to 6.6.1, maybe the problem has been fixed. Unfortunately I didn't have enough time to verify it
If you are still getting the problem you can try the following:

Download the 6.4.5 release (https://github.com/vuejs/devtools/releases/tag/v6.4.5)
Build and install the extension manually

I have this bug clearly disappeared in the 6.4.5 release.
P.S. I would like to say thanks to the developers for not abandoning this project

Maybe this will help someone. Recently devtools has been updated to 6.6.1, maybe the problem has been fixed. Unfortunately I didn't have enough time to verify it If you are still getting the problem you can try the following:

Download the 6.4.5 release (https://github.com/vuejs/devtools/releases/tag/v6.4.5)
Build and install the extension manually

I have this bug clearly disappeared in the 6.4.5 release.
P.S. I would like to say thanks to the developers for not abandoning this project

I downloaded this version but when i want to add in load unpacked extension got this error :
Failed to load extension
File
~/Downloads/devtools-6.4.5
Error
Manifest file is missing or unreadable
Could not load manifest.

I failed to get older versions working, one work-around which is to use the stand-alone app.
That hasn't shown any problems yet - seems speedy.

I don't know if it's when it closes or not, but I have the same problem, Chrome constantly freezes on Mac. This makes for a horrible experience, I have to force quit all of Chrome every 5 minutes

6.6.1 has still same memory leak issue. Edge freezes and memory starts to increase by Gigabytes

the same, Chrome, Windows

Chrome keeps on freezing after opening Vue Devtools for some time. I have to check Vuex store values like this for now:
Array.from(document.querySelectorAll('*')).find(e => e.__vue__).__vue__.$store.state

Also experiencing something similar. Immediately after opening vue devtools, the memory usage would start quickly going up to 20+ gb until chrome would just crash. I only got a 20-30 second window to use the devtools.
What seems to have helped is disabling React Devtools, Angular Devtools and other extensions.

v6.6.1 reproduces

Thanks a lot my friend, after 6 month finally my vue devtools is working.
@lxmfly123

Also experiencing something similar. Immediately after opening vue devtools, the memory usage would start quickly going up to 20+ gb until chrome would just crash. I only got a 20-30 second window to use the devtools.
What seems to have helped is disabling React Devtools, Angular Devtools and other extensions.

I disabled Angular devtools and everything gets ok.

I can confirm that this also solves the problem for me.
So it seems to be an interaction problem between Vue DevTools and Angular DevTools?

I don't have Angular devtools installed and Vue devtools still makes Chrome devtools crash. It happens a lot with 6.6.* version, less times with version 6.5.1 (I tried downgrading to that version).

@oskarnrk  Try to use this extension it's much stable than newer one but sometime it also lags but not that frequent
https://chromewebstore.google.com/detail/vuejs-devtools/iaajmlceplecbljialhhkmedjlpdblhp

Is there any update?
This issue is open for nearly a year and the bug still exists :/
As a Workaround I use the dev tools via Firefox. Here the vue dev tools are working.

makes it a pain to use Vue

I can confirm that deactivating the angular extension worked for me.


dotnet msbuild hanging

From @fuocor on April 24, 2018 18:12
building using the cli is hanging at completion on Windows Server.
This is the abbreviated output (full output is attached).
build.zip
 1:2>Done building target "IncrementalClean" in project "Ossiaco.Common.csproj".
     Target "PostBuildEvent" skipped, due to false condition; ('$(PostBuildEvent)' != '' and ('$(RunPostBuildEvent)' != 'OnOutputUpdated' or '$(_AssemblyTimestampBeforeCompile)' != '$(_AssemblyTimestampAfterCompile)')) was evaluated as ('' != '' and ('' != 'OnOutputUpdated' or '' != '')).
     Target "GenerateBuildDependencyFile" skipped. Previously built successfully.
     Target "GenerateBuildRuntimeConfigurationFiles" skipped, due to false condition; ( '$(GenerateRuntimeConfigurationFiles)' == 'true') was evaluated as ( '' == 'true').
 1:2>Target "CoreBuild" in file "C:\Program Files\dotnet\sdk\2.1.300-preview3-008427\Microsoft.Common.CurrentVersion.targets" from project "C:\ossiaco\dotnet\src\Ossiaco.Common\Ossiaco.Common.csproj" (target "Build" depends on it):
 1:2>Done building target "CoreBuild" in project "Ossiaco.Common.csproj".
 1:2>Target "AfterBuild" in file "C:\Program Files\dotnet\sdk\2.1.300-preview3-008427\Microsoft.Common.CurrentVersion.targets" from project "C:\ossiaco\dotnet\src\Ossiaco.Common\Ossiaco.Common.csproj" (target "Build" depends on it):
 1:2>Done building target "AfterBuild" in project "Ossiaco.Common.csproj".
     Target "_CheckForInvalidConfigurationAndPlatform" skipped. Previously built successfully.
     Target "_CheckForInvalidConfigurationAndPlatform" skipped. Previously built successfully.
 1:2>Target "Build" in file "C:\Program Files\dotnet\sdk\2.1.300-preview3-008427\Microsoft.Common.CurrentVersion.targets" from project "C:\ossiaco\dotnet\src\Ossiaco.Common\Ossiaco.Common.csproj" (entry point):
 1:2>Done building target "Build" in project "Ossiaco.Common.csproj".
     Target "_PackAsBuildAfterTarget" skipped, due to false condition; ('$(GeneratePackageOnBuild)' == 'true' AND '$(IsInnerBuild)' != 'true') was evaluated as ('false' == 'true' AND 'true' != 'true').
 1:2>Done Building Project "C:\ossiaco\dotnet\src\Ossiaco.Common\Ossiaco.Common.csproj" (Build target(s)).

It requires a Ctrl+C to break out of the build. Here is the dotnet --info
.NET Core SDK (reflecting any global.json):
 Version:   2.1.300-preview3-008427
 Commit:    423010bf0d

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.14393
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\Program Files\dotnet\sdk\2.1.300-preview3-008427\

Host (useful for support):
  Version: 2.1.0-preview3-26411-06
  Commit:  8faa8fcfcf

.NET Core SDKs installed:
  2.1.300-preview3-008427 [C:\Program Files\dotnet\sdk]
  2.2.100-preview1-008633 [C:\Program Files\dotnet\sdk]

.NET Core runtimes installed:
  Microsoft.AspNetCore.All 2.1.0-preview2-30431 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.1.0-preview2-30475 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.App 2.1.0-preview2-30431 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.1.0-preview2-30475 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 2.0.5 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.1.0-preview2-26314-02 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.1.0-preview3-26411-06 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]

To install additional .NET Core runtimes or SDKs:
  https://aka.ms/dotnet-download

Copied from original issue: dotnet/templating#1519

This log is odd in a few ways.

SDK 2.1.300-preview3-008427 is not a version I'd expect; maybe it's an old CLI master-branch build?
MSBuild <IP_ADDRESS>28 is fairly old now, predating some node-reuse changes.
The log indicates that the project is multitargeted to netstandard2.0;netcoreapp2.1, but only the netstandard2.0 version completes. That explains the hang--MSBuild is waiting on the netcoreapp2.1 build.

I suspect a failure to launch or connect to a worker node
@fuocor can you try a few things?

Does this repro with the preview2 SDK?
If so, can you use Process Monitor to take a trace of process start and process exit events during the build?
Can you also set two debugging flags for MSBuild and report the traces?

set MSBUILDDEBUGCOMM=1
mkdir c:\msbuildcommlogs
set MSBUILDDEBUGPATH=c:\msbuildcommlogs

dotnet build

You can use a different folder, of course--and please zip up everything from that folder and attach it.

Thanks for the quick response...

Yes it builds with the preview 2 sdk

here is the zip containing the process & build logs.
I did notice while monitoring the processes, that the build tries restarting msbuild on a regular interval
MSBuild_CommTrace_PID_7608.zip

the problem appears to be related to node... I am building on a windows machine using nvs to target chackracore and node.
Rerunning nvs and selecting another version appears to have resolved the issue and reselecting chackracore works as well.
rich

Thanks for checking it out. This is an instance of #3142, which was fixed for preview 2 and up. The symptoms match. It is indeed trying to launch MSBuild every 30 seconds--it'll give up eventually after trying 10 times.
Please let us know if you see this on a preview2+ build and we can reopen and investigate further.

Duplicate of #3142


Tests failing when MeasureAction codes with > 3 digits were generated

Various test fixes, not quite sure how I triggered all these bugs ... but this was blocking me from running tests.
MeasureActionFactory code too long:
Running tests can run the MeasureActionFactory > 1000 times, even though there are only a few MeasureActions in any one test, this caused the sequence to hit 1000 and an illegal code with 4 digits was generated, this is fixed by wrapping round every 1000 iteations.
Upgrade pytest and friends.
Pytest and pytest-* dependencies are all upgraded.
Fix bug in manage.py
An earlier fix I made, required using DJANGO_SETTINGS_MODULE, but that actually broke tests, this undoes that by making manage.py aware of ENV=dev
Fix logging bug in util/common/xml
Drive-by fix for a logging issue here, where transactions were meant to be logged, but weren't

Codecov Report

Merging #557 (d9c6253) into master (7bcd016) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #557   +/-   ##
=======================================
  Coverage   92.67%   92.67%
=======================================
  Files         319      319
  Lines       19676    19676
  Branches     1573     1573
=======================================
  Hits        18234    18234
  Misses       1145     1145
  Partials      297      297

Impacted Files
Coverage Δ

common/tests/factories.py
99.50% <100.00%> (ø)

common/xml/util.py
91.04% <100.00%> (ø)

Continue to review full report at Codecov.

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


Eq 523 wcag error

What is the context of this PR?
Error messaging to improve accessibility
Header to include hero banner on the landing page
amends to fixed.css to include new theme and improve ie9 and earlier
How to review
Check Error messaging
• Launch basetheme questionnaire
• Go to the 'Retail turnover' data picker
• Submit the form without filling in the fields
• Use JAWS, NVDA or Voicover to check links to errors
Check banner
• check that the .bar--hero class only appears on the landing page
Check fixed.css is used
• check ie9 uses css/basetheme/fixed.css

[ ] Do all commits have sensible commit messages?
[ ] Is all new code unit tested?
[ ] Are there integration tests if needed?
[ ] Is there sufficient logging?
[ ] Is there documentation or is the code self-describing?

Who worked on the PR
@bennpearson

Current coverage is 77.82% (diff: 100%)

No coverage report found for master at 92ebdd0.

Powered by Codecov. Last update 92ebdd0...6d211ab


Prompt tuning: Allow to pass additional args to AutoTokenizer.from_pretrained

Fixes #1032
Description
Currently, when using prompt tuning with TEXT, we call AutoTokenizer.from_pretrained with only the model id. However, it may be necessary to pass additional arguments, e.g. trust_remote_code=True. This fix allows to pass more arguments by setting the argument tokenizer_kwargs in the PromptTuningConfig.
I also added a check that when tokenizer_kwargs is set, the TEXT option is actually being used.
Moreover, I noticed that we have no tests for prompt tuning with TEXT, so I added those tests for decoder models.
Additional changes
There was a bug in PromptEmbedding where the device of the init_token_ids was not set, which resulted in errors when using CUDA.
Finally, I removed an unused constant CONFIG_CLASSES from a test.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Link to Roger's sample test case for SIMSWAP does not exist

https://github.com/questsin/UBC-Rogers-CAMARA-Hackathon/blob/main/tests/test-api-sim-swap.html
Does not exist. It is referenced in the README.

The sandbox site has a version that works


Revert "Added AWS V4 auth for S3 compatibility. List buckets changed to match the backend change in response"

Reverts sodafoundation/dashboard#298
Wrong merge

LGTM


Protect against error in Manager shutdownTabletServer when tserver not found

Closes #2747

Is ManagerApiIT.java passing?

ManagerApiIT passes for me locally.


UsageStats permission requested after app crashes

When starting MADCAP for the first time, the user is requested to grant permission to access the UsageStats (normal behavior).
When the app crashes (not sure why that happens), the app automatically restarts and the user is taken to the page to grant permission to access the UsageStats. MADCAP shows as already having permission. The user shouldn't be taken to the UsageStats screen if MADCAP already has permission.

The permission screen is there for debugging purposes.

Then this issue can be a reminder to remove it from production.


Creating login page

I can create an login page for the website using HTML and CSS.

Thank You for the interest but I think a login page is not necessary for this type of website.


UI Improvements

In call/SMS log, the item height should expand vertically if the rule name is long.
Regex entry fields should have word-wrap and should expand vertically as you enter text. Scrolling left and right is difficult.

Like this:

Yeah, exactly.

done in this action https://github.com/aj3423/SpamBlocker/actions/runs/8951796439

Thanks


add bell plus & minus icon

bulb-plus
blub-minus

Can try closing the gap more for the alarm-minus icon.

done


Usage of SVGO breaks SVG output in my docs.

🐛 Bug Report
Upgrading from alpha.66 to alpha.69 does break my SVG icons (as mentioned in https://github.com/facebook/docusaurus/pull/3691#issuecomment-734921324).
With alpha.66 the SVG output that got produced was:
<svg xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" width="24" height="24" viewBox="0 0 24 24">...</svg>

However, with the version alpha.69 the output is:
<svg xmlns="http://www.w3.org/2000/svg" width="24" height="24">...</svg>

The SVG icons are used "inline" (through a custom JSX component).
The code inside the mdx file for the above (extracted from the real file and added line breaks to make it readable inside this issue block):
import IconInText from '../../src/components/icon-in-text/IconInText';
import MenuIcon from './assets/icons/dots-vertical.svg';

## Overview

The overview page shows all students saved in the system and their current schein status.
As an adminstrator you can edit or delete students by clicking on the menu button <IconInText icon={MenuIcon} /> and selecting the desired option.
You can access the [_Student info page_](./student_info) by clicking on the "Information" button.
Furthermore, you can search for a specific student by using the search bar at the top.
For more information see the [_Student Management page_](./student_management).

Have you read the Contributing Guidelines on issues?
Yes 😀
To Reproduce

Update to alpha.69.
Start dev-server and open the documentation on localhost.
Visit a page with one of the icons.
See that the icons are miss aligned.

Expected behavior

The new version should show the icons in the same way as the old one did.
Actual Behavior

Icons / generated SVG are broken. More precisely: While the container (circle) is scaled properly to match the text that SVG icons itself are not affected by the scaling due to the missing viewBox attribute (even though the SVG files them self contained this attribute).

alpha66
alpha69

Your Environment

Docusaurus version used: 2.0.0-alpha.69
Environment name and version: Chrome (87.0.4280.88), Node.js (v14.13.1)
Operating system and version (desktop or mobile): Win10, desktop

Reproducible Demo
The actual code lives in a monorepo inside the docs folder: https://github.com/Dudrie/Tutor-Management-System/tree/update-docusaurus/docs
The selected branch of the link is the one if the alpha.69 version used. If necessary I can extract the docs from the monorepo and move it to a different repository it that is easier for you.

Thanks
I don't see this file in your site:
import MenuIcon from './assets/icons/dots-vertical.svg';

Can you give me the link, so that I can see the real/exact SVG content? (probably useless but still better to have a 100% reprocase, just in case)

Related to https://github.com/facebook/docusaurus/pull/3691

As far as I understand:

We can pass Webpack loader options as querystring: https://blog.johnnyreilly.com/2017/01/webpack-configuring-loader-with-query.html
The options are the ones of SVGR: https://react-svgr.com/docs/webpack/

What I suggest:

Extract this SVGR loader from core and create a SVGR plugin
Let the user pass his own config as an object
Apply existing querystring config by default for retrocompatibility
Add this svgr plugin to the themes by default, with a svgr attribute for configuring it

This would allow you to opt-out of SVGO using {svgr: {svgo: false}} on the classic theme config
    {
      test: /\.svg$/,
      use: '@svgr/webpack?-prettier,+svgo,+titleProp,+ref![path]',
    },

Thanks
I don't see this file in your site:
import MenuIcon from './assets/icons/dots-vertical.svg';

Can you give me the link, so that I can see the real/exact SVG content? (probably useless but still better to have a 100% reprocase, just in case)

Link to the file:
https://github.com/Dudrie/Tutor-Management-System/blob/update-docusaurus/docs/docs/handbook/assets/icons/dots-vertical.svg


Implement DWDs using ACSets, v2

All checks are now passing but I plan to do some more cleanup.
I also need to add more tests: I want the DWD core to have basically 100% coverage, but new branches were introduced by the four cases in ACSet (passing, incoming, outgoing, internal) and not all of them are covered in all methods. Thus, I am not confident that this code works.

All checks are now passing but I plan to do some more cleanup.
I also need to add more tests: I want the DWD core to have basically 100% coverage, but new branches were introduced by the four cases in ACSet (passing, incoming, outgoing, internal) and not all of them are covered in all methods. Thus, I am not confident that this code works.


Fix #854: Dependency TerragruntOptions was not updated in cycle check

This fixes the bug identified in #854. Specifically, the cycle logic was not properly updating TerragruntOptions.TerragruntConfigPath each time it recursed through the dependencies. This caused the subsequent parsing of include blocks to fail because it was appending the path to the wrong reference point.

Merging and releasing. Thanks for the review!


slsc cheat sheet

slsc is a spreadsheet calculator like sc
slsc [filename]
/ Menu
cursor to move, return to select
OR select by first letter
? Help (for further help see section help below)
First letters of a topic+Enter are enough
Enter to go back
mark region ends with CTRL+G (if you use dvtm, you may have to escape it with CTRL+G, CTRL+G)
Start a text/Enter in a field:
< or > to align a text field to left or right
= to enter a number or a formula
TAB to edit the text of a cell
Backspace (<-) do delete the cell content
The config file is ~/.slscrc

Fixed. Check: curl cheat.sh/slsc


JacksonValueMapper doesn't support Immutables

Immutables support Jackson serialization and deserialization using annotations. However, the JacksonValueMapper limits creating InputFields to those that can be directly deserialized. This forces the creation of POJOs for inputs, while Immutables can be used for outputs.
The JacksonValueMapper should support the annotated fields within a deserializable class.

I'll see what can be done. Since it seems to rely on regular Jackson features, it can be probably be easily supported.
But, in general, it's not reasonable to expect every library under the sun to work out of the box. This is why key interfaces like InputFieldBuilder  are made customizable. You could provide your own Immutables-aware implementation.

Input field mapping rules for abstract types have been greatly relaxed, so Immutables should now work just fine.
See #293 for details.


issue 685 - Allow "Stay logged in" text to select checkbox

This even simplifies the tap handler
Fixes #685

[ ]   General Comment
<img border=0 src='https://avatars.githubusercontent.com/u/554999?v=3' height=16 width=16'> I can't tell why this is failing in Travis CI    :(

This pull request is now managed with CodeReviewHub. Refresh.

I can't tell why this is failing in Travis CI    :(

Darn - my comments when granting the PR didn't seem to get through.
First, I think I tracked down and resolved the Travis problem - see separate PR #691.
Second, and just incidentally, I think I proposed this change in #229, when the setting was still called "Remember me".


OSDOCS-5240 #57427: review rendered page with important fixes on the structure

Review of https://github.com/openshift/openshift-docs/pull/57427:

Reordering sections
suggestions to review the file names
fixes identified from the preview page https://57427--docspreview.netlify.app/openshift-enterprise/latest/installing/installing_aws/installing-aws-localzone.html
adding module machine-edge-pool-review-nodes.adoc with a quick overview of the nodes after installation (helping user to understand what is delivered by the feature)
adding a helper page to deploy user workload on the "Next Steps"

Notes:

The fixes on CloudFormation and helpers present on 4.12 are super important to be backported, let's track it when we are ok with the final version of OSDOCS-5240

@mtulio Is there currently a ticket for backporting some content to 4.12?

@mtulio Is there currently a ticket for backporting some content to 4.12?

No, AFAIK all have been closed. Let's focus on 4.13 review, than we will know what need to be adjusted on 4.12

@mtulio Left a few more comments.

@mtulio Left a few more comments.

that's fine for me to move on to the main stream

thanks a lot! =)

@mtulio Re-request review/DM/ping whenever you're okay with me merging this.

@mtulio Re-request review/DM/ping whenever you're okay with me merging this.

@maxwelldb we can merge this one.
/lgtm


⚠ Asset Delivery has degraded performance

In 332e1cb, Asset Delivery (https://assetdelivery.roblox.com) experienced degraded performance:

HTTP code: 200
Response time: 1568 ms

Resolved: Asset Delivery performance has improved in 27adc7a after .


Cruise Ship Detection and Face Detection

Pull Request for DL-Simplified 💡
Issue Title :

Info about the related issue (Aim of the project) :
Name:
GitHub ID:
Email ID:
Idenitfy yourself: (Mention in which program you are contributing in. Eg. For a JWOC 2022 participant it's, JWOC Participant)

Closes: #issue number that will be closed through this PR
Describe the add-ons or changes you've made 📃
Give a clear description of what have you added or modifications made
Type of change ☑
What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested? ⚙
Describe how it has been tested
Describe how have you verified the changes made
Checklist: ☑

[ ] My code follows the guidelines of this project.
[ ] I have performed a self-review of my own code.
[ ] I have commented my code, particularly wherever it was hard to understand.
[ ] I have made corresponding changes to the documentation.
[ ] My changes generate no new warnings.
[ ] I have added things that prove my fix is effective or that my feature works.
[ ] Any dependent changes have been merged and published in downstream modules.

Create an issue first and then start working on it. This is not the way you should contribute in any open source projects.


Choice to ignore error when resolving errors

We should give the user an option to ignore/skip an error. On the API side, if the user does not provide a value for selectedOption, we should ignore that error (no fix). If a value is provided, then apply it.
In the demo application, we should give the user an additional option "none" (as a first option; we should still bring the default fix as selected though). If none is selected by the user, we should ignore the error.

@ugurdogrusoz @hasanbalci now, none option is added to demo and is shown as at the top. If there is no selectedOption, error is ignored.

I checked the sample app; the option "None" is there. But there are some other problems:

[ ] 1. On Windows, the combo box width is not enough to hold some sample names. Let's slightly increase the width of this combo box:

[ ] 2. When we cannot solve an error, we should not list any resolution alternatives (that only contain "None" as an option). I think this results in some other inconsistencies.

[ ] 3. When multiple processes are listed as resolution alterative, we just list "process, process, process, ..." which is ambiguous. For processes, let's use "process with input(s)/output(s) xyx" where "xyz" is a list of all input and output EPN labels. If "xyz" is empty, we should just display "process" as before.

[ ] 4. Similar to processes, with logic gates, let's use "{and,or,not} with input(s)/output(s) xyz", where "xyz" is a list of input and/or output EPN labels. Notice there that some of the inputs/outputs might be non-EPNs (other logic gates or processes); in that case, they are not listed. If "xyz" is empty, we should just display "{and,or,not}" as before.

[ ] 5. When you resolve errors and re-list remaining errors we get "Resolution Alternative" (singular) rather than "Resolution Alternatives". This is also worrying since it points to not re-using associated code.

@ugurdogrusoz now changes done for all items listed above.


changing ticketit's source files

Hi, I was trying to combine ticketit's logic into my project. There are some parts that I would like to unify for my project and ticketit, like nav.blade.php and errors.blade.php files. Some redundancy will be removed too.
The question is if I changed the source code of ticketit in my local project, will I got a problem when ticketit has a new version and I run "composer update"? Will I lose all my changes? Thanks.

Dont change the source code of the project. If you don't like one specific blade file, you can override it. For example create file resources/views/vendor/kordy/ticketit/shared/nav.blade.php to override this project's nav.blade.php file

However if you think, you can improve the quality of the code, feel free to make pull requests for the 1.0 branch


update security email link

As far as I know<EMAIL_ADDRESS>still works as per Reporting a Security Issue, but we should probably update the mailing list now that Deis has been acquired by Microsoft.

Should this be a MS email address or something for the community?

closed via #4124


Update web-sites-configure.md

To access Application Settings,

In the Azure Portal, open the blade for the web app.
Click Application Settings under Settings tab.

@rmcmurray

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@AshokKumar-P : Thanks for your contribution to the Azure documentation! The author, @rmcmurray, has been notified to review your proposed change.

This file is deleted. #please-close


Add missing rfc cipher names.

Using the "-stdname" flag for the "ciphers" command showed quite a few "UNKNOWN".
A little digging showed that there were ciphers missing from the list.
I used this mapping to populate the list:
https://testssl.sh/openssl-rfc.mappping.html
There seems to be a disagreement over the first 30 or so ciphers.
Additionally I left ciphers in that the mapping did not have.
I use the "ciphers" command to get a list of ciphers available to use with "s_client".
For report generation I would like the RFC name to be displayed, but openssl had missing names.
For reference I am using this fork:
https://github.com/drwetter/openssl-pm-snapshot
This fork is tailored to enabling weak ciphers for security testing.
It is likely that the ciphers that I saw missing were ciphers not enabled normally.
Either way its just a suggestion to maintain a full list of rfc cipher names, feel free to close if you think it is dumb.

Just the "top-level" comment is fine, thanks.

Latest commit has requested changes, I've gone ahead and tracked down sources for ciphers not listed on IETF's "tls-parameters" page.
It may be better to only keep cipher names that are explicitly listed on that page, but I'd leave that to your judgement.

Just add a comment with your “research” ☺  Thanks!

I've gone ahead squash my commits and push'd with a comment stating cipher "TLS_KRB5_WITH_DES_CBC_SHA" used to be "SSL_FORTEZZA_KEA_WITH_RC4_128_SHA" with a few links to details their definition and being overridden.

I have not signed the CLA, this seemed like a "trivial" contribution to me but on your request I can do so. Also amended the commit with my personal email address instead of my work email address (as requested in the policy).
For your question, 0x1E was "SSL_FORTEZZA_KEA_WITH_RC4_128_SHA" but is now "TLS_KRB5_WITH_DES_CBC_SHA" (single DES), and 0x1F is "TLS_KRB5_WITH_3DES_EDE_CBC_SHA" (triple DES).

not trivial, you did some real work tracking things down.  please sign the cla.  thanks.

I am awaiting approval from the company I work for to sign the CLA, just to make sure everything is kosher.
Sorry for the delay.

I've sent this to my legal department several times but I am just a peon. We have "free" time to work on projects that we want to, and I spent that time doing this pull. To make sure everything is kosher I want to make sure the company is ok with me submitting this as myself or for whatever legal technicality exists. Just thought I'd post an update that I am still waiting and I apologize for the wait.

Please take a look, I think #3859 does the job needed now. Closing.  Re-open if there's more.


feat: adding support for docker capabilities

Hello, I would like to request new feature to add option to pass docker runtime capabilities.
As example if I would like to use strace in docker container, container need to be started with
docker run --cap-add=SYS_PTRACE "$@"

when adding option to Kamal i get an error that this is not supported:
deploy.yml
cap_add:
  - SYS_PTRACE

ERROR:
  ERROR (Kamal::ConfigurationError): unknown key: cap_add

More information available here: https://docs.docker.com/engine/reference/run/#runtime-privilege-and-linux-capabilities

have you tried using the options field to pass the custom configuration
https://kamal-deploy.org/docs/configuration/roles/#custom-role-configuration

@acidtib thanks a lot!
yes option does work, it just was not obvious that option is used for additional docker parameters.
what is the logic behind splitting docker commands? like [cmd, logger, image] are available in similar syntax like docker-compose but for, [user, cap_add] it's need to go under options ?

FYI
servers:
  web:
    hosts:
      - example.com
    options:
      "cap-add":
        - SYS_ADMIN


Consider Online Storage and Syncing

It might be worth swapping out Dexie.js for PouchDB, since Pouch is designed around syncability with an online store in the form of CouchDB.
There is also a full-text-search plugin for PouchDB: https://github.com/pouchdb-community/pouchdb-quick-search
To implement the same in Dexie... well, I'm still figuring that out. You'd use Dexie.Syncable which is... possibly more configurable, but ultimately more painful to setup than it sounds Couch/Pouch is.
A third option is to use a traditional setup... create an API attached to a SQL database (Postgres/MySQL) and communicate synchronisation in the background... Very traditional approach, robust, future-proof. Unsure if it's the right call for this project.
When it does come to putting features online, however, I would still like to enable like... "Deploy to Heroku", enabling users to instantaneously get both a server and the SPA setup in a flash  that they can immediately start using.
At the time of writing, the application is a simple notes application that works for a single environment storing offline - no sync even between offline environments

What I'm thinking is... CouchDB, though a fantastic package, is probably too restrictive in authentication terms.
So I'm thinking of sticking with Dexie.js and implementing a backend (probably REST API on Rails with Postgres DB) that is communicated with via the iSyncProtocol: https://dexie.org/docs/Syncable/Dexie.Syncable.ISyncProtocol
I know nothing about that setup, but I think it may be the way to go...
Alternatively... consider whether Rails is the right choice. Maybe given that we're already working with JavaScript, a Node.js server (perhaps Express) would be more appropriate and allow more contributions from a community...

What I'm thinking is... CouchDB, though a fantastic package, is probably too restrictive in authentication terms.
So I'm thinking of sticking with Dexie.js and implementing a backend (probably REST API on Rails with Postgres DB) that is communicated with via the iSyncProtocol: https://dexie.org/docs/Syncable/Dexie.Syncable.ISyncProtocol
I know nothing about that setup, but I think it may be the way to go...
Alternatively... consider whether Rails is the right choice. Maybe given that we're already working with JavaScript, a Node.js server (perhaps Express) would be more appropriate and allow more contributions from a community...


alignment: Implement x16/x32 functions

Without these, all execute memory transactions will be converted into regular read transactions.

Published 0.1.2 to crates.io, which includes this fix


Fix the docs-dev script

Webpack was choking on a require(../package.json) statement in react-docgen/node_modules/doctrine/lib/utility.js.
This PR adds a json-loader to stop the choking. It’s like a maneuver.

@ryan-roemer Re:dividing dependencies consistently, my understanding is limited to building/generating static docs from this archetype.
So I am not comfortable dividing them without knowing how it would affect a victory component. Could that be a new issue and tabled for another PR?

@paulathevalley -- On re-inspection, json-loader does look like it's in the correct place. You correctly have the module pattern for the dev dep. But you need to revert the one for babel-loader to the old form of require.resolve("babel-loader"). No re-review needed and good diagnosis / work!


Add key to error message of value's runtype.

This PR adds additional documentation to the type checking errors in Record and Optional types. If one of the key checks fails, the Record will add In key ${key}: to the beginning of the string, indicating which of the keys has the problem.

Coverage decreased (-0.7%) to 99.327% when pulling de63dd5d73500e75733d70887448a9e02408e574 on DanielBrookRoberge:field-name-in-error-messages into 61b0efc0703a405002dfaaf99c4dabba419647a3 on pelotom:master.

Coverage remained the same at 100.0% when pulling 092db0ae86e42b67279010dba0b7795756fe243b on DanielBrookRoberge:field-name-in-error-messages into 61b0efc0703a405002dfaaf99c4dabba419647a3 on pelotom:master.

Nice, thanks!


Group - ‘Next’ button size is not consistent with ‘Start group’ button size

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v1.4.63-0
Reproducible in staging?: Y
Reproducible in production?: N
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:

Click on ‘FAB’ > Start chat
Select multiple users > Click on ‘Next’ button

Expected Result:
‘Next’ button size should be consistent with ‘Start group’ button size
Actual Result:
‘Next’ button size is not consistent with ‘Start group’ button size
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[ ] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/115492554/0548ab9a-d224-40ec-bebd-202d9baa0368

View all open jobs on GitHub

@strepanier03 I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors.

We think this issue might be related to the #vip-vsb.

Production
https://github.com/Expensify/App/assets/115492554/8d869ab0-1366-44a3-852d-b507fd9d2f03

Proposal
Please re-state the problem that we are trying to solve in this issue.
Group - ‘Next’ button size is not consistent with ‘Start group’ button size
What is the root cause of that problem?
This happens as the Start Chat page uses a custom button component, which is normal sized:
https://github.com/Expensify/App/blob/9b839f4f18d724dba1ed17ed5967f7e033781428/src/pages/NewChatPage.tsx#L266-L271
Wheres on New Chat Confirm Page we use the SelectionList to render the footer, which by default creates a large button:
https://github.com/Expensify/App/blob/9b839f4f18d724dba1ed17ed5967f7e033781428/src/components/SelectionList/BaseSelectionList.tsx#L622-L630
What changes do you think we should make in order to solve the problem?
We can:

Increase the size of the Next button by:

passing large as prop to the button component in NewChatPage.tsx here
https://github.com/Expensify/App/blob/9b839f4f18d724dba1ed17ed5967f7e033781428/src/pages/NewChatPage.tsx#L266-L271

Decrease the size of the Start Group button by:

Passing a custom footer button
Adding a prop to SelectionList to control button size with a prop

I don't think this needs to be a deploy blocker, what do you think @puneetlath.

Looks like there's a pretty noticeable pixel difference.

I agree. I'm gonna add this to the polish pile for Group Chats. @puneetlath I can take this off your hands if you'd like.

@strepanier03 Just a heads up that someone ended up fixing this issue in another PR so we can follow the process here.

Thank you, taking a look now.

All done, paid and contract closed for @neonbhai. Thanks for pinging me!


material-design-icons: 6.6.96 -> 7.0.96

Description of changes
Bump version of material-design-icons.
https://github.com/Templarian/MaterialDesign-Webfont

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Build fails with:
   > tar: /build/file: Cannot open: No such file or directory
       > tar: Error is not recoverable: exiting now

Package was broken by #173430, along with many other font packages (see #193478). I've fixed the build for now by following #175381 as an example, but I'm not sure if its approach will be good for the long-term. Seems like a much bigger issue that needs to be addressed, what with the large number of font packages that are broken.


⚠ Secret Website 1 has degraded performance

In 899df82, Secret Website 1 ($SECRET_WEBSITE_1) experienced degraded performance:

HTTP code: 200
Response time: 2475 ms

Resolved: Secret Website 1 performance has improved in 5132192.


Add closable badge variant

Oh good idea indeed!
I've refactored it a bit so that the closable badges consist of 2 badges (a bit like split buttons) where both sides can have separate hover effects/actions.
The idea for the repo module for example would be that the left part of the badge could lead to the filtered view of the repos that have the tag enabled and the right part would remove the tag), let me know if this makes sense/works for you!

Cursor pointer has been added ✔


sum method for arrays

Related to #63: if we define our own sum method, we can use better word length scaling for the arrays.
a.sum() or a.sum(dim) behaving like numpy
a.convolve(b) (possibly with additional mode argument, see https://numpy.org/doc/stable/reference/generated/numpy.convolve.html )

Superseded by #303


Caching extremely slow with big directories, almost defeats purpose of caching

/usr/bin/docker exec  b9aecdd1312d7ac605e6f897371c1032c55ff0ac43610845cae92e02d750800e sh -c "cat /etc/*release | grep ^ID"
The cache action detected a local S3 bucket cache. Using it. // We use runs-on's fork but that's unrelated to the slowness
/usr/bin/tar --posix -cf cache.tgz --exclude cache.tgz -P -C /__w/api/api --files-from manifest.txt -z
Cache Size: ~2118 MB<PHONE_NUMBER> B)
Uploading cache from /__w/_temp/3787d239-de30-4a12-b9e3-26f64466b95a/cache.tgz to org-runs-on-s3bucketcache-y51bkein77av/cache/org/api/71ee539a69b4092a757e60e07a31650a220dbc09d2d0[28](https://github.com/org/api/actions/runs/11682390370/job/32529413116#step:12:29)76da35435f74632221/api-debug-build-Linux-51d25692387488814d8005a20d74d91f144127ef
Uploaded part 1/67.
Uploaded part 2/67.
Uploaded part 3/67.
Uploaded part 4/67.
Uploaded part 5/67.
Uploaded part 6/67.
Uploaded part 7/67.
Uploaded part 8/67.
Uploaded part 9/67.
Uploaded part 10/67.
Uploaded part 11/67.
Uploaded part 12/67.
Uploaded part 13/67.
Uploaded part 14/67.
Uploaded part 15/67.
Uploaded part 16/67.
Uploaded part 17/67.
Uploaded part 18/67.
Uploaded part 19/67.
Uploaded part 20/67.
Uploaded part 21/67.
Uploaded part 22/67.
Uploaded part 23/67.
Uploaded part 24/67.
Uploaded part 25/67.
Uploaded part 26/67.
Uploaded part 27/67.
Uploaded part 28/67.
Uploaded part [29](https://github.com/org/api/actions/runs/11682390370/job/32529413116#step:12:30)/67.
Uploaded part 31/67.
Uploaded part 32/67.
Uploaded part 33/67.
Uploaded part 34/67.
Uploaded part 35/67.
Uploaded part 36/67.
Uploaded part 38/67.
Uploaded part 37/67.
Uploaded part 39/67.
Uploaded part [30](https://github.com/org/api/actions/runs/11682390370/job/32529413116#step:12:31)/67.
Uploaded part 40/67.
Uploaded part 41/67.
Uploaded part 42/67.
Uploaded part 43/67.
Uploaded part 45/67.
Uploaded part 44/67.
Uploaded part 46/67.
Uploaded part 47/67.
Uploaded part 48/67.
Uploaded part 49/67.
Uploaded part 51/67.
Uploaded part 50/67.
Uploaded part 52/67.
Uploaded part 53/67.
Uploaded part 54/67.
Uploaded part 57/67.
Uploaded part 56/67.
Uploaded part 58/67.
Uploaded part 59/67.
Uploaded part 60/67.
Uploaded part 61/67.
Uploaded part 62/67.
Uploaded part 63/67.
Uploaded part 64/67.
Uploaded part 65/67.
Uploaded part 66/67.
Uploaded part 67/67.
Uploaded part 55/67.
Cache saved successfully.
Cache saved with key: api-debug-build-Linux-51d25692[38](https://github.com/org/api/actions/runs/11682390370/job/32529413116#step:12:39)7488814d8005a20d74d91f144127ef

I notice Uploaded part and what happens after it is not the slow part.
It apparently just takes too much time to create the upload parts.

Did I forget to mention how long it actually took?
Depending on the file size it can take more than 5.5 mins, for a 2.8GB file.
Usually it takes around 3 mins which is still pretty high.


validate clip_norm argument in clip_by_norm API

The API clip_by_norm have argument clip_norm which accepts  0-D (scalar) Tensor > 0 . But if we pass -ve value for this argument then its not raising intended error and converting the input tensor into Negative which IMO is wrong. Hence I am adding validation code for -ve values to raise value error.
Attaching gist for the same with solution for reference.
Fixex #61158

Here are the internal errors, @SuryanarayanaY can you please verify ? Thank you!
Traceback (most recent call last):
File "/tensorflow/python/ops/math_ops.py", line 1486, in binary_op_wrapper
out = r_op(x)
File "/tensorflow/python/util/traceback_utils.py", line 141, in error_handler
return fn(*args, **kwargs)
File "/tensorflow/python/ops/math_ops.py", line 1509, in r_binary_op_wrapper
y, x = maybe_promote_tensors(y, x, force_same_dtype=True)
File "/tensorflow/python/ops/math_ops.py", line 1441, in maybe_promote_tensors
ops.convert_to_tensor(tensor, dtype, name="x"))
File "/tensorflow/python/profiler/trace.py", line 183, in wrapped
return func(*args, **kwargs)
File "/tensorflow/python/framework/ops.py", line 701, in convert_to_tensor
return tensor_conversion_registry.convert(
File "/tensorflow/python/framework/tensor_conversion_registry.py", line 209, in convert
return overload(dtype, name)  #  pylint: disable=not-callable
File "/tensorflow/python/framework/tensor.py", line 762, in tf_tensor
raise ValueError(
ValueError: x: Tensor conversion requested dtype float32 for Tensor with dtype float64: <tf.Tensor 'embedding_lookup:0' shape=() dtype=float64>
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/tensorflow/python/framework/test_util.py", line 1634, in decorated
return f(self, *args, **kwargs)
File "/tensorflow/python/kernel_tests/nn_ops/embedding_ops_test.py", line 584, in testHigherRankMaxNorm
simple = embedding_ops.embedding_lookup(params, ids, max_norm=1.0)
File "/tensorflow/python/util/traceback_utils.py", line 141, in error_handler
return fn(*args, **kwargs)
File "/tensorflow/python/util/dispatch.py", line 1260, in op_dispatch_handler
return dispatch_target(*args, **kwargs)
File "/tensorflow/python/ops/embedding_ops.py", line 326, in embedding_lookup
return _embedding_lookup_and_transform(
File "/tensorflow/python/ops/embedding_ops.py", line 144, in _embedding_lookup_and_transform
result = _clip(
File "/tensorflow/python/ops/embedding_ops.py", line 73, in _clip
return clip_ops.clip_by_norm(
File "/tensorflow/python/util/traceback_utils.py", line 141, in error_handler
return fn(*args, **kwargs)
File "/tensorflow/python/util/dispatch.py", line 1260, in op_dispatch_handler
return dispatch_target(*args, **kwargs)
File "/tensorflow/python/ops/clip_ops.py", line 223, in clip_by_norm
intermediate = values * clip_norm
File "/tensorflow/python/util/traceback_utils.py", line 141, in error_handler
return fn(*args, **kwargs)
File "/tensorflow/python/ops/math_ops.py", line 1491, in binary_op_wrapper
raise e
File "/tensorflow/python/ops/math_ops.py", line 1475, in binary_op_wrapper
return func(x, y, name=name)
File "/tensorflow/python/ops/math_ops.py", line 1856, in _mul_dispatch
return multiply(x, y, name=name)
File "/tensorflow/python/util/traceback_utils.py", line 141, in error_handler
return fn(*args, **kwargs)
File "/tensorflow/python/util/dispatch.py", line 1260, in op_dispatch_handler
return dispatch_target(*args, **kwargs)
File "/tensorflow/python/ops/math_ops.py", line 529, in multiply
return gen_math_ops.mul(x, y, name)
File "/tensorflow/python/ops/gen_math_ops.py", line 6590, in mul
_, _, _op, _outputs = _op_def_library._apply_op_helper(
File "/tensorflow/python/framework/op_def_library.py", line 778, in _apply_op_helper
_ExtractInputsAndAttrs(op_type_name, op_def, allowed_list_attr_map,
File "/tensorflow/python/framework/op_def_library.py", line 589, in _ExtractInputsAndAttrs
raise TypeError(
TypeError: Input 'y' of 'Mul' Op has type float32 that does not match type float64 of argument 'x'.

Hi @SuryanarayanaY Any update on this PR? Please. Thank you!

I have gone through the error stack and root cause is as below.

File "/tensorflow/python/kernel_tests/nn_ops/embedding_ops_test.py", line 584, in testHigherRankMaxNorm
simple = embedding_ops.embedding_lookup(params, ids, max_norm=1.0)

In the above test the params is float64 and max_norm=1.0 which are then passed to clip_by_norm() as t and clip_norm respectively.

File "/tensorflow/python/ops/clip_ops.py", line 223, in clip_by_norm
intermediate = values * clip_norm

Then in the above line of code t is converted to values  as float64 tensor and clip_norm=1.0
values = ops.convert_to_tensor(
        t.values if isinstance(t, indexed_slices.IndexedSlices) else t,
        name="t")

With the proposed amendment clip_norm = math_ops.maximum(clip_norm, 0), the clip_norm becoming float32 tensor and this is causing the error and test fail. Without modification it remains as 1.0 and hence values * clip_norm works fine here.
One observation is the args t and clip_norm of API tf.clip_by_norm both the args should be of same dtype else exception will be raised which is also not documented.
My proposed solution is:
clip_norm = tf.cast(math_ops.maximum(clip_norm, 0),dtype = values.dtype)

which can validate -ve inputs, resolves the issue mentioned above regarding dtype of t and clip_norm and it also eliminates testfail.
I have added a gist for above exercise for reference.
@cantonios , Needs your review and comments for proceeding.

Hi @cantonios, Can you please review this PR ? Thank you!

Hi @cantonios, Can you please review this PR ? Thank you!

Hi @SuryanarayanaY Can you please check @cantonios's comments and keep us posted ? Thank you!


Feature request: smaller space for mono icons

When using the monospaced nerd fonts, the whitespace between icons and names is very wide. An option to reduce this space would be nice.

Same issue using jetbrains nerd font

We should have a fix for this in https://github.com/Peltoche/lsd/pull/469 which should land pretty soon. Closing this for now, keeping #363 as the tracking issue for this.

We should have a fix for this in https://github.com/Peltoche/lsd/pull/469 which should land pretty soon. Closing this for now, keeping #363 as the tracking issue for this.

Thanks, @meain and @OrangeFran!

Thanks, @meain and @OrangeFran!


Bedits

try adjustments to Boris advance step following Tom's Python implementation more closely.

substantial re-build of code to account for larmor radiation. Problem was: too much kinetic energy loss - unexplained. Tracked to wrong continuous process in physics list. Inelastic scattering model does not apply to thin gas as ours; switched off. Bogus energy losses disappeared. Now perfect energy conservation; also wrong. Larmor radiation losses not accounted for. Change of all Magnetic fields to EM-fields in G4 framework with permitted energy loss. Not due to electric field as intended by G4 but our magnetic field motion. Needed class re-structuring of all magnetic field classes and equation of motion dependence and Boris solver. Latter now follows Tom's textbook implementation to the letter. Larmor radiation now accounted for, checked with step kinetic energy output to file in addition to intended antenna signal output (larger file). Last issue was G4 requirement of step time accounting done by developer for energy 'non-conserving' transport hence small addition to Boris solver again.
Currently, all compiles and works fine. Checking with step prints has been removed, added analysis functions for kinetic energy checks on output file.


Is there a way to add an edit form to modify records?

See title.

Figured it out!

Hey @pboling,
at the moment TinyAdmin is mainly proposed for read-only operations.
I would like to propose features it in the future for CRUD operations but I can't provide a road map.
Feel free to share some code if you like, it could help some other user.

I'll try to find time to do that.


WIP Stop route relation

I also think it's worth adding tests for this

Good for me, should we merge?


Issue importing a server previously hosted with essential

I had a friend who hosted a small modded server over essential, which quickly became more popular than his PC or wallet could handle, so I took on the task of setting up a dedicated server. I had MSCS already installed and decided to use that, but whenever I tried to import the files he sent me it would always give me the "directory containing a minecraft  world was not provided" error, and I've been tearing my hair out trying to fix it. Any help would be appreciated, thanks.
(also please ignore the poor crop on the image)

I believe you need to go up a directory (/home/****/pokemon)

I've tried that to no difference, but thanks

Could you post a directory listing of /home/****/pokemon?
Also, I wonder if path expansion doesn't work on that command, have you tried:
mscs import /home/****/pokemon pokemon 25565

Sorry I didn't respond sooner, the contents of /pokemon are just the zip file of the server my friend sent me and the folder /Pokemon, which contains what was listed in the first image. Also I tried that command to nothing different

If there is only the zip file, you are missing a few files. You only copied over the world data directory.
The /home/****/pokemon directory should look like this:
crash-reports/
libraries/
logs/
Pokemon/
banned-ips.json
banned-players.json
crash-reports/
logs/
ops.json
server.properties
usercache.json
whitelist.json

Once you get these files in place, try to import again.
After that, you might want to create a mscs.properties file for this server
https://minecraftservercontrol.github.io/docs/mscs/adjusting-world-server-properties#individual-world-properties
Depending on the server type, you might need something like this:
mscs-client-version=1.16.4
mscs-server-version=1.16.4
mscs-server-jar=forge-1.16.4-35.1.7/forge-1.16.4-35.1.7.jar
mscs-server-url=

If you run into issues again just let me know!

Sorry I didn't respond sooner, I managed to get a few more files from my friend and with a lot of trial and error and only one casualty (lost one mod to an actually unique glitch) I managed to get it working, and it runs way better than when it was on an old laptop. Thank you so much for the help!

Awesome! I'm happy that it works and glad I could help!


Use HashSet for duplication detection

Description
The duplication detection is now amortized O(n) instead of O(n^2).
Does your commit message include the wording below to reference a specific issue in this repo?

No issue opened for this trivial change.

Does your pull request have unit tests?

This is a performance fix and thus cannot be unit tested.

I'll agree that using a HashSet is better since we do not need the random index access of the List, so element lookup would be faster, though I do not expect any noticeable speed up in class generation.

run ci


selected function not returning values

C3 version:0.6.0
D3 version:5.6.0
Browser: Firefox
OS: Ubuntu 16.04

I upgraded to a newer C3/D3 combo and I keep running into this problem. I've been using the selected() function before and it worked fine. When I do that after the update I get a 'map is not a function' error. Here's a fiddle: https://jsfiddle.net/z8bxskqy/5/
Here when you select something in the chart and press the button you're supposed to get the selected values but instead it throws an error.

Seems to be related to/same as: https://github.com/c3js/c3/issues/2477

Closing as #2477 was fixed. If issue is still occurring on latest version please reopen


Bug: live query does not return a json patch

Describe the bug
I'm trying the live query with diff params set to true.
It works but the patch I'm receiving is not a JSON patch as stated in the doc (which would be useful).
I get patch like this {op: "change", path: "/value", value: "@@ -1,3 +1,3 @@↵-301↵+439↵"}]
I could write a custom patch / diff method but JSON patch is probably better (sorry If I'm missing the point)
Steps to reproduce
Very simple. Just using js sdk with something like await db.live( , cb, true);
Expected behaviour
Receive a correct patch (op: "replace" vs "change")
SurrealDB version
1.0
SurrealDB.js version
0.11
Contact Details
<EMAIL_ADDRESS>Is there an existing issue for this?

[X] I have searched the existing issues

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hey @blobinabottle, the patch specification in SurrealDB slightly differs from JSON Patch if I'm not mistaken. I'll get this checked out, make sure the docs are correct and then come back to this issue. Thanks!


input:after doesn't work on Firefox

This is great and I really loved the radio and checkbox buttons, but they don't work on firefox. Apparently input:before or input:after are not valid css, because input element does not have content:
https://stackoverflow.com/a/4574946
Is it possible to move the buttons to the label?

:tada: This issue has been resolved in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Suggest or add new things in the readme.md files

hi! can you assign this issue to me ? @tomriddle77

Hey i think i can add amazing of value ,can you please assign this to  me ?
Have a nice day

please tag your PRs here so that I can merge it


Add drop database helper to database manager script

Also name object pointer self instead of cls on adjacent object methods

Will add a test


No show posters being displayed

Add image caching from another third party website, I'm tending towards fanart.tv.

https://github.com/flownzu/TraktTV-UpdateClient/commit/a316a2c24c8baa06f34780d3dac1921f29fa6b40 Added automatic image caching that should solve this issue.


Can't revoke certificates from zerossl-bot

It looks like zerossl-bot uses ACME directly so when I try to revoke certificates from the zerossl.com dashboard it's unable to and directs me to these instructions: https://help.zerossl.com/hc/en-us/articles/900005244486-Revoking-Certificates-Issued-via-ACME
I can't use the original ACME account as I can see in the zerossl-bot logs that it uses a new account variously which I assume is generated from my zerossl-api-key.
I tried the other method using the private key but the zerossl.com dashboard gives me two .crt files: ca_bundle.crt and certificate.crt and I no longer have the files zerossl-bot issued
Neither of these are in the .pem format required by cerbot revoke - I tried converting them with openssl but it doesn't look like either of those files is the private key as certbot revoke tells me the converted pem files do not match.
If there's a way to revoke the certificates from zerossl-bot please let me know.

@Blackbam
I've noticed that if I used the --eab-kid and --eab-hmac-key params (along with zerossl-api-key) on zerossl-bot it seems to pass them to certbot.
Can you let me know if this will be consistent? Rather than using the random ACME account?
If that's the case then I can use the ACME account provided on the zerossl dashboard so I can revoke the certificates from zerossl-bot in the future.
Also wouldn't it make sense for zerossl-bot to use that ACME account from my zerossl account?
The lack of revokation ability could be a security issue in the case of key leakage.

Seems I was wrong in my previous comment, if I pass my own EAB flags, they are duplicated and passed again by zerossl-bot.
I've settled on using certbot with the --server flag to zerossl server.

@Blackbam
Yes, it would probably useful to add to the README.


Fix typo in app label deprecation notice

What does this PR do?
Fixes a typo in app label deprecation notice.
Should this change be mentioned in the release notes?

[x] CHANGELOG.md has been updated (if it exists)

Small typo fix, most of folks on a vacation, will just merge.


Finnish language assets

It seems Finnish is not very myspell or aspell friendly:
https://wiki.ubuntu.com/ConsolidateSpellingLibs
Bug: https://phabricator.wikimedia.org/T158587

Phew :D

Codecov Report

Merging #301 into master will decrease coverage by -0.13%.
The diff coverage is 63.41%.

@@            Coverage Diff             @@
##           master     #301      +/-   ##
==========================================
- Coverage   85.26%   85.14%   -0.13%
==========================================
  Files         225      227       +2
  Lines        7264     7305      +41
==========================================
+ Hits         6194     6220      +26
- Misses       1070     1085      +15

Impacted Files
Coverage Δ

revscoring/languages/finnish.py
0% <0%> (ø)

revscoring/languages/tests/test_finnish.py
100% <100%> (ø)

Continue to review full report at Codecov.

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


Basic skeletons for desired OpenMM high throughput workflow

This PR implements mostly empty skeletons of what could be a working HTMD implementation.

Looks great. I'm going to step through it and ask questions to make sure I understand.


Replace Archive module source code by download from the gallery

I pointed it to www.powershellgallery.com and not the myget at the moment.
I think it's reasonable at the moment.
If we decide to consume dev build of Archive module in the future, we can change the SourceLocation easily.

Fixing helpuri failure https://github.com/PowerShell/Microsoft.PowerShell.Archive/pull/25
@vors you should compare the test code between the copy we have in PS/PS and what is in the Archive repo and merge them

The old failures seems to relate to the older (non-compatible with Core edition) version of Acrhive module in the gallery.
If they persist, we would need to initiate a new release for the archive before the switch.

@SteveL-MSFT tests yes, we should remove them before merging.
Meanwhile, they revile that we have old, Unix-incompatible release in the gallery.
This is blocked on https://github.com/PowerShell/Microsoft.PowerShell.Archive/issues/26

@vors I'll have someone look into getting those issues fixed

Can you please resolve the conflicts?

CC @anmenaga

@SteveL-MSFT @anmenaga - What is the current status of this? Is it still blocked?

The dependent work item (PowerShell/Microsoft.PowerShell.Archive#26) is almost done. I expect that it will be checked in next week.

@anmenaga ANy progress with the checkin?

Ready for a review!
@mirichmo @anmenaga


Update error

vscode-inno-updater.log
Just tried updating today and got this error...

I get the same issue trying to install the automatic update with my current version as 1.23.1.  I got this with the previous automatic update as well.  I am on Windows 10, Version 10.0.16299 Build 16299.

/duplicate #47841


Add email sign up screen.

Add email sign up.

Just noticed I don't have the prettier plugin installed in VSCode.


chore(ci): add tree-sitter nightly test

This change is

Done, you can see here the action with all the files parsed


Does UI.prompt support timeout?

Hi team,
is possible to provide a default answer (after certain timeout interval) for UI.prompt() call?
Thanks
🚫 fastlane environment 🚫
Stack

Key
Value

OS
10.15.5

Ruby
2.6.3

Bundler?
false

Git
git version 2.24.3 (Apple Git-128)

Installation Source
/usr/local/bin/fastlane

Host
Mac OS X 10.15.5 (19F101)

Ruby Lib Dir
/System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib

OpenSSL Version
LibreSSL 2.8.3

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
11.5

System Locale

Error

No Locale with UTF8 found 🚫

fastlane files:
No Fastfile found
No Appfile found
fastlane gems

Gem
Version
Update-Status

fastlane
2.150.1
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

did_you_mean
1.3.0

slack-notifier
2.3.2

rouge
2.0.7

xcpretty
0.3.0

terminal-notifier
2.0.0

terminal-table
1.8.0

plist
3.5.0

multipart-post
2.0.0

word_wrap
1.0.0

tty-cursor
0.7.1

tty-spinner
0.9.3

babosa
1.0.3

colored
1.2

highline
1.7.10

commander-fastlane
4.4.6

unf
0.1.4

domain_name
0.5.20190701

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

fastimage
2.1.7

gh_inspector
1.1.3

mini_magick
4.10.1

multi_xml
0.6.0

security
0.1.3

xcpretty-travis-formatter
1.0.0

dotenv
2.7.5

naturally
2.2.0

simctl
1.6.8

jwt
2.1.0

uber
0.1.0

declarative
0.0.10

declarative-option
0.1.0

representable
3.0.4

retriable
3.1.2

mini_mime
1.0.2

httpclient
2.8.3

google-api-client
0.38.0

emoji_regex
1.0.1

nanaimo
0.2.6

colored2
3.1.2

claide
1.0.3

CFPropertyList
3.0.2

atomos
0.1.3

xcodeproj
1.16.0

unicode-display_width
1.7.0

public_suffix
4.0.5

addressable
2.7.0

tty-screen
0.8.0

excon
0.75.0

faraday
1.0.1

unf_ext
<IP_ADDRESS>

faraday_middleware
1.0.0

json
2.3.0

rubyzip
2.2.0

bundler
2.1.4

multi_json
1.14.1

signet
0.14.0

os
1.1.0

memoist
0.16.2

googleauth
0.13.0

digest-crc
0.5.1

google-cloud-errors
1.0.1

google-cloud-env
1.3.2

google-cloud-core
1.5.0

google-cloud-storage
1.26.2

aws-eventstream
1.1.0

aws-sigv4
1.2.0

aws-partitions
1.332.0

jmespath
1.4.0

aws-sdk-core
3.100.0

aws-sdk-kms
1.34.1

aws-sdk-s3
1.69.0

forwardable
1.2.0

logger
1.3.0

stringio
0.0.2

ipaddr
1.2.2

openssl
2.1.2

ostruct
0.1.0

strscan
1.0.0

date
2.0.0

fileutils
1.1.0

etc
1.0.1

io-console
0.4.7

zlib
1.0.0

libxml-ruby
3.1.0

rexml
3.1.9

psych
3.1.0

mutex_m
0.1.0

webrick
1.4.2

generated on: 2020-07-03

Not at the moment but I'm interested in your use case 😇 How would you plan on using this?

Hi Josh,
I'm making my fastlane script able to pick one of my targets. Before doing the dirty work I would to ask the user the option to choose a branch where a particular language pack is located; user may type features/translations_for_amazing_feature or master or anything else.
This is a special option and usually the master branch is enough for common needs so I would to ask for it and at the same time use the timeout to auto-select the master option.
Side note: playing with prompts package for node I also seen it supports an autocomplete menu where you could set a list of choices (I could get the list of branches using our gitlab apis). It could be great but frankly I didn't understand how it works (are they ansi escape codes for terminals like colors?)


Version 1.0.1

Improvements

Documentation fixes

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
120

Relevant Lines:
120

💛  - Coveralls


Kubevirt was installed according to the documentation, but CRD was not installed

What happened:
https://kubevirt.io/quickstart_minikube/
export VERSION=$(curl -s https://api.github.com/repos/kubevirt/kubevirt/releases | grep tag_name | grep -v -- '-rc' | sort -r | head -1 | awk -F': ' '{print $2}' | sed 's/,//' | xargs)
echo $VERSION
kubectl create -f https://github.com/kubevirt/kubevirt/releases/download/${VERSION}/kubevirt-operator.yaml
kubectl create -f https://github.com/kubevirt/kubevirt/releases/download/${VERSION}/kubevirt-cr.yaml
[root@master kubevirt]# kubectl get all -n kubevirt
NAME                                READY   STATUS    RESTARTS   AGE
pod/virt-operator-fdc587684-2sr5n   1/1     Running   0          129m
pod/virt-operator-fdc587684-lvrsz   1/1     Running   0          129m
NAME                            READY   UP-TO-DATE   AVAILABLE   AGE
deployment.apps/virt-operator   2/2     2            2           129m
NAME                                      DESIRED   CURRENT   READY   AGE
replicaset.apps/virt-operator-fdc587684   2         2         2       129m
NAME                            AGE    PHASE
kubevirt.kubevirt.io/kubevirt   127m
[root@master kubevirt]# kubectl api-resources |grep kubevirt
kubevirts                         kv,kvs            kubevirt.io/v1                         true         KubeVirt
How to reproduce it (as minimally and precisely as possible):
Steps to reproduce the behavior.
Additional context:
Add any other context about the problem here.
Environment:

KubeVirt version (use virtctl version): V0.55.0
Kubernetes version (use kubectl version): v1.20.7
VM or VMI specifications: N/A
Cloud provider or hardware configuration: N/A
OS (e.g. from /etc/os-release): N/A
Kernel (e.g. uname -a): N/A
Install tools: N/A
Others: N/A

How is the CRD yaml file generated? For example, virtualmachineinstance

Sorry, it's not a bug
I have successfully installed


[ALLUXIO-2948] Remove docker support in vagrant deploy module

https://alluxio.atlassian.net/browse/ALLUXIO-2948

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/17899/
Test PASSed.

Merged build finished. Test PASSed.


Fix markdown links

This fixes markdown links in CHANGELOG.md file.
Thanks!

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: mominul sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.