[RFC] Add support for multiple networks

podman run --network=net1,net2,foobar ...
@baude @mheon @rhatdan

I would prefer we change the CLI to use a StringSlice for --net, then you could specify
--net net1 --net net2 --net fedora ...

One thing I think we have to do is cache which networks were given to 'run' or 'create' so that we can intelligently tear them down later.  Otherwise you'd have to make sure you passed the same set of -net options to kill which could be error-prone.

@dcbw you have a git validation error in there ... if you can fix that, tests will run ... ping us for help on the validation error if this is new to  you.

@dcbw This needs a rebase.

@rhatdan yeah, I will.  I also tried to use StringSlice but that's going to be complicated because NetMode is tied in with the docker HostConfig and inspect stuff.

closing in favor of #1082 ... thanks for your contribution @dcbw


Recharging laser guns don't update ammo UI

Description

Advanced/Antique laser guns' ammo counters don't update when they recharge, just when you fire them or swap to their hand.
Reproduction

Use all the charge in an advanced laser gun, keep the hand selected. Laser icon updates, but ammo counter doesnt.
Screenshots

Additional context

Dupe https://github.com/space-wizards/space-station-14/issues/14127


Function: setting log retention causes rate exceeded with a lot of Functions

Problem
Log retentions are being configured via AWS SDK through custom resource. With a lot of Functions, AWS SDK calls are made concurrently and results in rate exceeded error.
Setting logRetentionRetryOptions to 100 retries will fix the rate exceeded error, but causes:
Received response status [FAILED] from custom resource. Message returned: A conflicting operation is currently in progress against this resource. Please try again. (RequestId: 27daaaf0-f4b5-472d-8cae-643d3664f740)

Solution 1
Creating the log group manually for Functions:
this.getAllFunctions().forEach(func => {
	new logs.LogGroup(this, `${func.node.id}-LogGroup`, {
		logGroupName: `/aws/lambda/${func.functionName}`,
		retention: logs.RetentionDays.THREE_MONTHS,
		removalPolicy: scope.defaultRemovalPolicy
	})
})

The issue with this approach is that if the log group already exists, deployment will fail.
Solution 2
Understand what is causing the A conflicting operation error, and either come up with a more sensible LogRetentionRetryOptions config; or rewrite the custom resource for setting log retention to gracefully handle the logic.

Requests:
https://serverless-stack.slack.com/archives/C01JG3B20RY/p1634839201037400
https://serverless-stack.slack.com/archives/C01JG3B20RY/p1635375910361800
Dan https://discord.com/channels/983865673656705025/996097936440180767

Hi @garretcharp, I tried the suggestion of adding a dependency from the function to the log and hit a circular dependency issue if you have stacks adding routes to stacks.  Thought I'd just mention it in case it helped with what you were doing!

Also, just doing solution 1 didn't work for me.  It got further on one deploy but sitll failed with the rate limit on one of the stacks.  A combination of setting the maxRetries on the defaultFunctionProps and solution 1 seemed to work though:
logRetentionRetryOptions: {         maxRetries: 100,       },

Now I've started hitting the duplicate name issue so the workaround worked for a while but has now stopped...  I'm looking into ways to check for the log existing beforehand.  I thought log.logFromGroupName might work but that seems to been a bust.

We are also running into this issue, sadly.

We are transition this repo to https://github.com/sst/v2 and are closing this issue for now. If it is still relevant please reopen it there - sorry for the trouble


Fix sideNav Variations ID

When you click on the item Variations of the sideNav of this page : http://materializecss.com/side-nav.html
You cannot go to the section mentionned
Tested on : Chrome 43.0.2357.124 m and Firefox 38.0.5

I uncapitalized the id instead in  8375acd93a5d62

Yes but it doesn't work currently. Why did you uncapatilize ?


Rename package

This way it should be possible to install it via npm and be consumable by tsc@2.
In package.json:
"devDependencies": {
   ...
    "@types/swagger-tools": "https://github.com/jzorn/typed-swagger-tools.git",
   ...
}

Thank you!


add localrc.test_target

This localrc.test_target file is used for creating a local,
simple swift endpoint for testing clients. the README.md was also
updated to show people how to use it.

Probably needs some more docs around auth credentials (or even some better keystone-style auth setup), but I'd like to get some feedback first on the idea.
(Also I think those things can come pretty easily in subsequent commits rather than needing to be in this PR.)


[zh] Delete Localization READMEs section in localized README

Description
ref. https://github.com/kubernetes/website/pull/47763#issuecomment-2400322169

Delete Localization READMEs section in localized README

Now, these links don't work well

/kind cleanup
Issue

Closes: #

/assign @tengqm
for approval

/lgtm
/approve


Add signature consideration to FunctionRefEvaluator

The FunctionRefEvaluator currently considers only the declared and runtime signatures for function resolution. Update the evaluator to take advantage of the signature element if it is present (representing the compile-time resolution of the function).
This is related to this translator issue: https://github.com/cqframework/clinical_quality_language/issues/1127
And this evaluator issue: https://github.com/cqframework/clinical-reasoning/issues/244

Done


Card stop working in 2v

The card stop displaying the list in version 2, sensors are there but nothing is displayed except the title of the card downgraded to 1.3.4 and the card is working again.
Home Assistant Version: 2021.12.10

@voggur Sorry, I should have mentioned in the v2.x.x release that the name of the sensor changed, so you will need to update your card to reference the new name.  It used to be sensor.steam_wishlist but it now includes your profile id, e.g. sensor.steam_wishlist_<your-id>.  If you click into the device for the steam wishlist integration from the integrations page and click on the Wishlist sensor you can find the new name.

I'll update the release notes and README to make note of this.

Thank you it works again :)


Change the input type to number for datepicker

I am using the following configuration to create a datepicker field in the UI:
<FormlyFieldConfig>{           key: field.elementid,           type: 'datepicker',           modelOptions: {             updateOn: 'blur',           },           expressionProperties: {             ...conditionalValidators,           },           ...asyncValidators,           templateOptions: {             ...fieldValidators,             label: elementLabel,             translate: true,             datepickerOptions: {               touchUi: true,             },           },         }
I would like to change the input type of the datepicker to "number" so that the appropriate keyboard is shown on mobile devices. Is it possible to do that without having to create a new custom component?
Thanks!!

I have created this pull request to support this behavior, it would be great if we could support this out of the box.
https://github.com/ngx-formly/ngx-formly/pull/3114

I have created this pull request to support this behavior, it would be great if we could support this out of the box.
https://github.com/ngx-formly/ngx-formly/pull/3114

In brief, the change will allow the user to use templateOptions.type to set the type to either 'text' or 'tel'. Type 'tel' would open the numeric keyboard on the mobile device, and will still support the formatted date.

In brief, the change will allow the user to use templateOptions.type to set the type to either 'text' or 'tel'. Type 'tel' would open the numeric keyboard on the mobile device, and will still support the formatted date.

@aitboudad : could you please take a look at this so that we can have a new build that supports this.

@juristr @mohammedzamakhan @kenisteward : Hello, would any one of you will be able to check the PR in case @aitboudad is not available?


Added power checks

Added power checks to:
Reagent Dispenser
Medical Scanner
Lathes
Research Console
Vending Machines
Added power device component to:
chem dispenser  prototype
medical scanner prototype
Fixes for #706

One thing I couldn't get was the queue to not open on the Lathe if it is un-powered.


buggy fixed_pos sampling

Hi and thanks for sharing your work
I saw in the other issues that someone pointed to the formatting issues with some parenthesis issues in the argument parser.
Besides that, here fixed is not defined https://github.com/evo-design/protein-dpo/blob/main/sample.py#L15
I would imagine you wanted to do something like
for fixed_idx in [int(pos) for pos in args.fixed_pos.split(",")]:

and specify --fixed_pos 1,2,3,4 (whatsoever, 1-indexed and comma separated for example)

Hi @adrienchaton, thanks for catching this! Just made the fix in the most recent commit. Let me know if it doesn't work.


Source code not found for webpack:///....

Firstly, thanks for this awesome plugin. It's been working great for us in production for some months!
I'm not sure if something on Sentry's end changed recently, but we're no longer seeing our code in Sentry from the uploaded sourcemaps.
Source code was not found for webpack:///../node_modules/angular/angular.js
Source code was not found for<EMAIL_ADDRESS>Source code was not found for webpack:///./apps/instances/modules/pixark/app.js

is what Sentry is reporting on any issues reported from our website, for example.
Our filenameTransform function is as follows:
filenameTransform: function(filename){
	return '~/assets/' + filename;
},

Again, this has been working just fine for months so I imagine something on Sentry's end changed? Would changing our filenameTransform function to be prefixed with webpack:// perhaps work?

Hey, sorry you never got a response on this. I've had to play around with the filenameTransform to get the sourcemaps linked up correctly; it depends on your webpack output path configuration. Also I'm not sure if those library files would have sourcemaps in the first place.


I found this error in package.json file if anyone knows plz help.

{
"resource": "/c:/Users/user/myApp/package.json",
"owner": "generated_diagnostic_collection_name#0",
"code": "undefined",
"severity": 4,
"message": "Problems loading reference 'https://schemastore.azurewebsites.net/schemas/json/package.json': Unable to load schema from 'https://schemastore.azurewebsites.net/schemas/json/package.json': Unable to connect to https://schemastore.azurewebsites.net/schemas/json/package.json. Error: getaddrinfo ENOTFOUND schemastore.azurewebsites.net schemastore.azurewebsites.net:443",
"startLineNumber": 1,
"startColumn": 1,
"endLineNumber": 1,
"endColumn": 2
}
The below is my package.json file ;
{
"name": "myApp",
"version": "0.0.1",
"author": "Ionic Framework",
"homepage": "http://ionicframework.com/",
"private": true,
"scripts": {
"clean": "ionic-app-scripts clean",
"build": "ionic-app-scripts build",
"lint": "ionic-app-scripts lint",
"ionic:build": "ionic-app-scripts build",
"ionic:serve": "ionic-app-scripts serve"
},
"dependencies": {
"@angular/animations": "5.2.9",
"@angular/common": "5.2.9",
"@angular/compiler": "5.2.9",
"@angular/compiler-cli": "5.2.9",
"@angular/core": "5.2.9",
"@angular/forms": "5.2.9",
"@angular/http": "5.2.9",
"@angular/platform-browser": "5.2.9",
"@angular/platform-browser-dynamic": "5.2.9",
"@ionic-native/core": "4.5.3",
"@ionic-native/splash-screen": "4.5.3",
"@ionic-native/status-bar": "4.5.3",
"@ionic/storage": "2.1.3",
"angularfire2": "^5.0.0-rc.3",
"cordova-ios": "4.5.4",
"cordova-plugin-device": "^2.0.2",
"cordova-plugin-ionic-keyboard": "^2.0.5",
"cordova-plugin-ionic-webview": "^1.2.1",
"cordova-plugin-splashscreen": "^5.0.2",
"cordova-plugin-whitelist": "^1.3.3",
"firebase": "^4.13.1",
"firebase-tools": "^3.18.4",
"ionic-angular": "3.9.2",
"ionicons": "3.0.0",
"promise-polyfill": "^7.1.2",
"rxjs": "5.5.7",
"sw-toolbox": "3.6.0",
"zone.js": "0.8.20"
},
"devDependencies": {
"@ionic/app-scripts": "3.1.8",
"typescript": "~2.6.2"
},
"description": "An Ionic project",
"cordova": {
"plugins": {
"cordova-plugin-whitelist": {},
"cordova-plugin-device": {},
"cordova-plugin-splashscreen": {},
"cordova-plugin-ionic-webview": {},
"cordova-plugin-ionic-keyboard": {}
},
"platforms": [
"ios"
]
}
}

Posible duplicate of #48298
Are you using a proxy?


Theme check: command doesn't inspect any file after CLI upgrade

Describe the bug
Running the command line 'shopify theme check' is not inspecting files, returning a Success message , 0 files inspected with no offenses.
Source
It happens on all Theme projects, even Dawn. It also happens to my other colleagues on their computers.
Expected behaviour
The command should output a large list of errors and suggestions.
Actual behaviour
The command is not inspecting files.
The command returns:
Success
Theme Check Summary.
0 files inspected with no offenses found.
Debugging information

OS Windows
OS Version 11
Shopify CLI v3.56.3

Additional context
The command was working fine until I upgraded shopify CLI from v3.48.x to v3.52.x. It is broken since then.
See an example with a fresh start of a Dawn theme:

How about shopify theme check --path .? Same thing or no?
Same thing, and it appears in the .theme-check.yml file. I also tried with ., and ./ but there is no difference
Also, is there a .git in that folder?
Yes there is, it was created by the shopify theme init command. I tried removing the folders .git, .github, .vscode and running theme check again, but still get the same result.
Don't you need to dir into the theme before running the command?
Sorry, I forgot the cd into dir on the example. But anyway, I did it now and still get the same result.
New info from my colleagues, it seems that the issue happens only on Windows. Theme check still works on their Mac computers.


[SPARK-6910] [SQL] Support for pushing predicates down to metastore for partition pruning

This PR supersedes my old one #6921. Since my patch has changed quite a bit, I am opening a new PR to make it easier to review.
The changes include-

Implement toMetastoreFilter() function in HiveShim that takes Seq[Expression] and converts them into a filter string for Hive metastore.
This functions matches all the AttributeReference + BinaryComparisonOp + Integral/StringType patterns in Seq[Expression] and fold them into a string.
Change hiveQlPartitions field in MetastoreRelation to getHiveQlPartitions() function that takes a filter string parameter.
Call getHiveQlPartitions() in HiveTableScan with a filter string.

But there are some cases in which predicate pushdown is disabled-

Case
Predicate pushdown

Hive integral and string types
Yes

Hive varchar type
No

Hive 0.13 and newer
Yes

Hive 0.12 and older
No

convertMetastoreParquet=false
Yes

convertMetastoreParquet=true
No

In case of convertMetastoreParquet=true, predicates are not pushed down because this conversion happens in an Analyzer rule (HiveMetastoreCatalog.ParquetConversions). At this point, HiveTableScan hasn't run, so predicates are not available. But reading the source code, I think it is intentional to convert the entire Hive table w/ all the partitions into ParquetRelation because then ParquetRelation can be cached and reused for any query against that table. Please correct me if I am wrong.
cc @marmbrus

Merged build triggered.

Merged build started.

Test build #36516 has started for   PR 7216 at commit dce7abf.

Test build #36516 has finished for   PR 7216 at commit dce7abf.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

This is looking very close!  Thanks again for doing such a thorough evaluation of the differing things Hive supports here. :)

Merged build triggered.

Merged build started.

Test build #37074 has started for   PR 7216 at commit f134e91.

Merged build triggered.

Merged build started.

Test build #37074 has finished for   PR 7216 at commit f134e91.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test build #37081 has started for   PR 7216 at commit 28c8843.

Test build #37081 has finished for   PR 7216 at commit 28c8843.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

@marmbrus Thank you for reviewing my patch. I think I addressed all your comments. Can you please take a look again?
Btw, the last jenkins failures seems unrelated to my PR. In fact, jenkins build had succeeded first and then failed in the 2nd run triggered by my minor commit that fixed imports order.

Merged build triggered.

Merged build started.

Test build #37092 has started for   PR 7216 at commit 9f5feec.

Test build #37092 has finished for   PR 7216 at commit 9f5feec.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Isn't this a test issue? I keep getting the following pyspark error. Looks like it is because of a minor difference in two datetimes 532502 != 532501. The test was added on Jul 9th by commit c9e2ef52bb54f35a904427389dc492d61f29b018.
FAIL: test_time_with_timezone (__main__.SQLTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/jenkins/workspace/SparkPullRequestBuilder/python/pyspark/sql/tests.py", line 718, in test_time_with_timezone
    self.assertEqual(now, now1)
AssertionError: datetime.datetime(2015, 7, 11, 12, 23, 36, 532502) != datetime.datetime(2015, 7, 11, 12, 23, 36, 532501)

Merged build triggered.

Merged build started.

Test build #37100 has started for   PR 7216 at commit 254d62a.

Test build #37100 has finished for   PR 7216 at commit 254d62a.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test build #1054 has started for   PR 7216 at commit 254d62a.

Test build #1054 has finished for   PR 7216 at commit 254d62a.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build triggered.

Merged build started.

Test build #37133 has started for   PR 7216 at commit aa1490f.

Test build #37133 has finished for   PR 7216 at commit aa1490f.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class Least(children: Expression*) extends Expression
case class Greatest(children: Expression*) extends Expression

Merged build finished. Test PASSed.

Thanks!  I'm going to merge this to master.

@piaozhexiu can you close this pr since github is not closing it automatically? Thanks!

Sure. Thank you for merging it!


Add option to disable blurring of image

Hi luhuiguo,
I added an option to disable blurring.
Could you please merge this and publish it to npm as a new version?
Thanks!
Greetings from Germany,
Malte

Hi luhuiguo,
friendly reminder....
can you please merge my pull request and publish the new version on npm?
Greetings,
Malte


Dates are sometimes wrong on hour total

Today, the date is 11-31 on the hours report when it should be 10-31 probably. Not sure why, might be something weird with end of month days.

I can't reproduce this

@Zjjc123 it only happens on the last day of the month.

Is this an issue with the admin page? Or regular user's tables?


TLS client-certificates performance

Are the client certs faster than regular TLS handshake event without session resumption? Or is it just like a token, passed after TLS handshake and regular DH occurs? In that case, NOT using client certificates could be more performant albeit a non standard approach so this needs evaluation.

Moved to: https://github.com/neptulon/neptulon/issues/36


sd_setImageWithURL in 3.7.6  is different from 3.7.5

I updated my pod to version 3.7.6 and found that the image is twice bigger than before. Is this a bug or using the different strategy to deal with retina screen?

this was reverted
more info here


Tooltip

Given
An event based gateway
When
I use the context menu to append an intermediate catching event and hover over the event symbols
Then
I get the same tooltip for all events:  Append IntermediateCatchEvent
Expected Behavior
The tooltip contains the exact event type (Signal, Message, etc.)

This needs a fix in bpmn-js, our underlying modeling toolkit.

Pending bpmn-js release.

Closed via https://github.com/camunda/camunda-modeler/commit/00f3ec8e7d3e5d74b098e716abc3de83ba50e149.


keypress event issue, is it necessary to optimize.

Hi~ I am not a developer and do not understand internal code,

💥 Current bugged behavior:

keypress ꘌ keydown / keyup

❥ Expected/fixed behavior:

keypress ≠ keydown / keyup

Add a e.g. 30ms delay to keydown to judge keypress event ?
see more at:     https://github.com/Mottie/GitHub-userscripts/issues/136

Thanks for the issue @dnknn.
I believe what you're asking for us to do is measure a full keypress, which includes the keyup event, such that holding down a button doesn't cause keycombos with duplicate keys (e.g. t t) to fire. If my understanding of this issue is wrong please let me know.
While this is a reasonable position, the additional complexity required to implement this outweighs the benefits for us. We currently don't have any keycombos which require duplicate keys, so adding this code would not improve use cases for us. Because of this I'm going to mark this as wontfix. We can continue the discussion further if you'd like, but I'm going to close the issue for now.

First of all thank you very much for your reply!
Okay, understand, I thought it could be easily fixed before I put it forward, this is not a big problem.


Decoding always empty

"laravel/lumen-framework": "5.8.*"
"vinkla/hashids": "6.0.0"
I made some modifications on hashids.php config file like this:
'main' => [
'salt' => 'F28F86043514AEB26943F1A67F21B94251201F322199855F1DF1B982E3CF7C92',
'length' => '6',
'alphabet' => 'abcdefghijklmnopqrstuvwxyz1234567890'
],
When i tried to decode Hashids::decode('34mm93'), it doesn't work. I have no response and the Lumen stops with "Illegal instruction: 4".
There's something wrong?

Don't know how to solve your problem. Illegal instruction is probably not an issue with this library.


Bug running task in a folder with parentheses

When a folder name has parentheses, the full path from source is copied to dest folder. Example:
Considering this folder structure:
/optimize(images)/
    /compressed/
    /img/2015/10/image.png
    Gruntfile.js
    package.json

And this task:
imagemin: {
    png: {
        options: {optimizationLevel: 7},
        files: [{
            expand: true,
            cwd: 'img',
            src: ['**/*.png'],
            dest: 'compressed',
            ext: '.png'
        }]
    }}

This is the result:
/optimize(images)/
    /compressed/2015/10/optimize(images)/img/2015/10/image.png
    /img/2015/10/image.png
    Gruntfile.js
    package.json

The final image is in fact optimized, the problem is just with the result folder path.
BTW, this problem is happening with me because my files are in a Dropbox folder named “Dropbox (Personal)”. I've tried creating a symbolic link with a different name, but the result was the same.

+1

+1

This looks to be a bug with imagemin. I've filed a bug there: https://github.com/imagemin/imagemin/issues/156

This bug might be fixed in version 5 but this package installs version 4.

In<EMAIL_ADDRESS>we're just using vinyl-fs to read and write the files, so it might have something to do with it.

Hi @all, i had the same problem, an created an work around. Maybe it helps someone.
here we run imagemin-cli (or imagemin-newer) to imagemin-temporary-directory, copy everything with grunt-copy (in my case for crossOS reasons) into the origin-image-source, and remove imagemin-temporary-directory.
update package.json
...
  "devDependencies": {
    "grunt-contrib-copy": "^0.6.0",
    "grunt-shell": "^1.3.0",
    "imagemin": "latest",
    "imagemin-newer": "^1.0.1",
  },
  "scripts": {
    "imagemin_script_1": "imagemin-newer <source-dir-1> <destination-dir-temp1> -d".
    "imagemin_script_2": "imagemin-newer <source-dir-2> <destination-dir-temp2> -d"
  },
...

update gruntfile.js
...
    shell: {
      options: {
        stderr: false
      },
      imagemin: {
        command: [
          'npm run imagemin_script_1'
          'npm run imagemin_script_2'
        ].join('&&')
      },
      removeImageminTmp: {
        command:  [
          'rm -rf <source-dir-1>'
          'rm -rf <source-dir-2>'
        ].join('&&')
      }
    },
    copy: {
      imageminCssImages: {
        files: [{
          expand: true,
          flatten: true,
          src: ''source-dir-temp1/**/*.{png,gif,jpg,svg,ico}',
          dest: 'build/assets/img'
        }]
      },
      imageminHtmlImages: {
        files: [{
          expand: true,
          flatten: true,
          src: ''source-dir-temp2/**/*.{png,gif,jpg,svg,ico}',
          dest: 'build/img'
        }]
      }
    },
...

...
  grunt.loadNpmTasks('grunt-shell');
  grunt.loadNpmTasks('grunt-contrib-copy');

  grunt.registerTask('test', [
    'shell:imagemin',
    'copy',
    'shell:removeImageminTmp',
  ]);
...

Thanks for sharing, I was becoming insane not understanding why sub-directories named per parent directories where being created...

Can you try out 2.0.0? If the issue still persists I'll reopen.


[DOCS] Adding a "Table on content" in the readme to make it easier to navigate

what's wrong in the documentation?
We must add a "Table of content" in the readme of the project to make it easier to navigate and more professional.
An example would be ->

Add screenshots
added above
Code of Conduct

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

@JasonDsouza212 Please assign me this issue under GSSoC'23


IResultOperatorHandler uses an unusual pattern in D.I.

Consider a service for which there there is one contract in core with an extended contract in relational or a specific provider. For example, IDatabaseCreator and IRelationalDatabaseCreator. When IRelationalDatabaseCreator is registered in D.I. it is also registered as IDatabaseCreator. This means that core services that don't know about anything relational still get the same service instance as relational services that need the extended contract.
IResultOperatorHandler doesn't fit this pattern because IRelationalResultOperatorHandler, which derives from IResultOperatorHandler, also itself depends on the base contract of IResultOperatorHandler. (In other words, it both derives from and composes over the same base type.) These therefore cannot be the same service and core code that depends on the base contract will get a different service than relational code that depends on the extended contract. This is presumably by design, but opening this so it can be looked at because it is the only example of a service that does this and could be confusing if it is not understood.

@anpete Putting this on the backlog, but if you think what we have is fine, then feel free to close this.


Tech preview

adds in faq and tech preview pages
updates for-filers pages with new links
updates pdfs to be the Filing Instruction Guide (FIG)

@wpears - can you take a look?

Looks good overall. Can you add a line-height: 1.375em to p and li elements in the for-filers, faq, and tech-preview pages?
As an aside, navigation doesn't really function since links generally point to pretty urls like <root>/for-filers and the connect server doesn't auto-route this to <base dir>/for-filers.html. This could be easily done with middleware, but I was wondering if there is a preexisting solution that I missed? Not germane to this PR, but thought I'd mention it ^^;

Should be good to go.
Yeah, the nav has been that way, just don't think anyone ever got around to doing anything about it. 😉

@awolfe76 Merge away


The multiclass_nms should return the global indice.

Motivation
This PR improves the codes for multi_class bbox NMS by returning the global indice.
Currently multiclass_nms returns the local indices after batched_nms:
https://github.com/open-mmlab/mmdetection/blob/b9dedfdc8b660804bb31ae031a251e86a4905ade/mmdet/core/post_processing/bbox_nms.py#L85-L92
The returned indices is useless outside the function unless it's a global indice w.r.t the inputs.
Modification
The modification is quite slight: change https://github.com/open-mmlab/mmdetection/blob/b9dedfdc8b660804bb31ae031a251e86a4905ade/mmdet/core/post_processing/bbox_nms.py#L92
to return dets, labels[keep], inds[keep]

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

Originally I fixed it in pr #4912.However #4912 is blocked, I think we can merge this pr first

@jshilong Thx for the quick response. The only concern is that there may be other codes just used the 'local indices' but I think this case is very rare.

Ping @ZwwWayne

@jshilong Thx for the quick response. The only concern is that there may be other codes just used the 'local indices' but I think this case is very rare.

@ZwwWayne We should notify this modification to MMtracking and MMAction and MMpose


feat: linux/arm64 multi-platform support

Closes #431
This PR introduces beta support for linux/arm64 images, mainly to enable M1 users to run Cypress docker images locally.
The only known limitation is that none of Chrome, Firefox, or Edge officially support linux-arm64 yet - so this means none of the browsers or included images will have browsers on linux/arm64. Users will have to use --browser electron or obtain their own arm64 browser binaries.
As arm64 binaries become available, they will be added to these images. See https://github.com/cypress-io/cypress-docker-images/issues/695 for details.
For now, users will see this warning on arm64:

This PR also:

Moves our AWS ECR image generation to Circle, dropping the need for AWS BuildSpec
Switches to docker buildx in CI
Adds snapshot tests for "generate-config" (1418 lines of this diff are snapshots)

@admah we still need to make a decision on how far back we want to go with updating older images to have linux/arm64 builds - right now, I'm thinking we'll just do the latest included image, and the browsers and base images upon which that relies.
Pre-merge:

[ ] Remove Circle environment variables, add Docker Hub and AWS ECR creds to Circle Context

@flotwig will this also close out #694? Did you remove all references to cly?

The only known limitation is that none of Chrome, Firefox, or Edge officially support linux-arm64 yet - so this means none of the browsers or included images will have browsers on linux/arm64. Users will have to use --browser electron or obtain their own arm64 browser binaries.

I may not be following the flow of how this will be used, if this is a known limitation, why would we produce a browsers image for linux/arm64? Would it be better to only provide a base image for this so prevent any confusion?

The only known limitation is that none of Chrome, Firefox, or Edge officially support linux-arm64 yet - so this means none of the browsers or included images will have browsers on linux/arm64. Users will have to use --browser electron or obtain their own arm64 browser binaries.

I may not be following the flow of how this will be used, if this is a known limitation, why would we produce a browsers image for linux/arm64? Would it be better to only provide a base image for this so prevent any confusion?

@emilyrohrbough Yeah, this incongruity b/t arm64 and amd64 is going to be a source of confusion no matter how we do it. We ended up going this direction because it's the easiest to explain, and the easiest for us to fix transparently for users when browser binaries are available (without them having to update their documentation or command used)
Full discussion and the other alternatives explored are here: https://cypressio.slack.com/archives/C033S1JBPV0/p1656603284262969?thread_ts=1656531594.354589&cid=C033S1JBPV0

will this also close out #694? Did you remove all references to cly?

@admah I just removed the final mention :+1: This PR can replace #694 now


Stock Price Scraping Support [GSSoC'23]  #221

for issue #221
Description
This pull request adds the StockPrice class and finance module, which enables scraping stock price for a given stock name and its index(optional). The class provides methods to retrieve latest price of stock and retrieve historical stock prices in given dates range.
Currently the class supports retrieving data for NSE stocks only(can be extended easily).
Changes Made

Added StockPrice class with methods for retrieving latest stock price and price of stock for a given range.
Added NSE class which handles all data requests made for nse stocks

Documentation -- In Progress
Known Issues

None

Reviewer Notes

Please review the implementation and provide feedback on the code structure, logic, and adherence to best practices.
The module has been designed in such a way that it can be extended to include BSE, NASDAQ and Crypto just by creating the classes. Further work will be done on other stock indexes based on feedback and suggestions for this code.

Would these changes be okay? I just committed them.


add v2f service account and attach IAM for bucket

I think this is p straightforward! I have hard coded the bucket to write to, I put it in the command-center's service accounts file because we're not spinning up a processing project for this, and I believe the objectCreator role is correct; those are the things I'd love to get double checked if anyone sees anything awry with 'em.

Oh and the purpose of this service account is for an argo workflow to have write permissions to the v2f bucket in question.


Piccoma FR

Can it working with the Piccoma fr website : https://piccoma.com/fr?
thanks

only supported for --chapter-url option.
Example:  piccoma.exe --chapter-url https://fr.piccoma.com/fr/viewer/47/1423

French version seems to have changed a lot since before.
Currently it is difficult to download and may not be supported.


Improve defaulting

This makes enables us to deploy a working telemetry with default values with minimal CR:
telemetry:
  enabled: true
  autoscaling:
    enabled: true
  ceilometer:
    enabled: true
  metricStorage:
    enabled: true

Depends-On: https://github.com/openstack-k8s-operators/heat-operator/pull/344

recheck

/retest

recheck

recheck

recheck

What is missing for us to actually enable ceilometer, metricStorage, and autoscaling by default?

I don't think there are plans to have autoscaling enabled by default. With metricStorage the issue is the COO dependency. We want to tell the users to always enable it, it should be enabled in all the VAs, but if we enable it here, the CI in all of the other operators will fail, because metricStorage will get deployed, but it'll never get to a ready state without COO.

recheck

recheck

What is missing for us to actually enable ceilometer, metricStorage, and autoscaling by default?

I don't think there are plans to have autoscaling enabled by default. With metricStorage the issue is the COO dependency. We want to tell the users to always enable it, it should be enabled in all the VAs, but if we enable it here, the CI in all of the other operators will fail, because metricStorage will get deployed, but it'll never get to a ready state without COO.

Thanks for this, Jaromir. I hadn't fully understood where the "real" defaults would be set (Verified Architecture manifests), vs the defaults that need to be in use for CI. This helps!
Ultimately, I hope that we find a supportable way of installing COO (and other operator bundles we depend on) the way we do with RPMs.

recheck

recheck

Hmm, looks like the failures are actual real failures because of the PR. I'll need to look into it.

/hold

This tries to default memcached, which is getting removed here: https://github.com/openstack-k8s-operators/telemetry-operator/pull/365 Once 365 gets merged, I'll get back to this, rebase it and remove the memcached defaulting.

This tries to default memcached, which is getting removed here: #365 Once 365 gets merged, I'll get back to this, rebase it and remove the memcached defaulting.

This has been resolved and other PRs are passing now

/unhold

/lgtm


Deployed Django keeps asking for reload each 30 seconds

This problem appeared when I deployed this forked project on a server and connected website. When I launch a project on a localhost, the problem disappears. Nothing has changed in the architecture. Does anybody know what to do stop this notification?

Stopped TinyMCE in order to see the difference. The problem still persists.

It turned out after inspecting Chrome Developer Tools that django_browser_reload, so commenting out in settings.py / MIDDLEWARE the following row:
"django_browser_reload.middleware.BrowserReloadMiddleware"
Will stop reload.


Add links to the User Guide

It will be helpful to add links for every paragraph of the Overview in the User guide doc file.
https://github.com/vmware/harbor/blob/master/docs/user_guide.md

@sigsbee
Would you please help to have a look if we can add links in the user guide for each paragraphs. Thanks.


[Bug] Not using GPT-4

Describe the bug
I'm using API, it's succdesfully showing models and using 3.5 Turbo most of the time.
When I select GPT-4 above the comment box it does not use the GPT-4 model.
I confirmed this, after suspecting, by specifically asking:
what gpt model are u?
I am based on OpenAI's GPT-3 model. As of my last training data in September 2021, GPT-4 or any subsequent models have not been released by OpenAI.
To Reproduce
As above
Screenshots

Desktop (please complete the following information):

OS: MacOS Ventura 13.6.1
CGPTN v2.9.9

Common sense: When you ask GPT about its version, it doesn't know who it is.

How is that "common sense"? Maybe you mean "common knowledge", and given I didn't, well 🤷♂.
So, it's still showing

Shouldn't it be at least Jan 2022, if not later - for GPT4?
See: https://www.reddit.com/r/OpenAI/comments/17h5hbc/chatgpt_is_now_updated_to_january_2022/

@m0rg5
api and https://chat.openai.com/ are different
do test here :  https://platform.openai.com/playground

@m0rg5 api and https://chat.openai.com/ are different do test here : https://platform.openai.com/playground

So all API clients cutoff is Sept 2021 - is yr point?
At https://chat.openai.com/ 👍

Still asking how to confirm/verify in CGPTN app use of whatever model.

So all API clients cutoff is Sept 2021 - is yr point?
yes
also read here about privacy for api https://openai.com/enterprise-privacy

Still asking how to confirm/verify in CGPTN app that we are using whatever model.

Bump?
How can we confirm/test this is working correctly?


It should be possible to pass a project name on the command line

As e.g.
./bin/SHADERed my_proj.sprj

@ctrlcctrlv sure! It should now work.
It was kind of implemented before but SHADERed only checked for the provided file in the path where the executable was located - silly me.
I will also try to make some improvements by adding the "-render" flag (and similar flags).


feat: leapfrogai api registry1 flavor

Description
Add a flavor override to the LeapfrogAI API Zarf package that uses an IronBank hardened Python 3.11.x image.
Describe alternatives you've considered
N/A
Additional context
Uses Python 3.11 (upstream) (possible IronBank)

https://repo1.dso.mil/dsop/opensource/defenseunicorns/leapfrogai/api/-/merge_requests/26

Next steps beyond Registry1 IronBank hardening process:

Create a pipeline and nightly/weekly cronjob for Registry1 E2E tests
Setup Makefiles, Dockerfiles, Zarf packages, and UDS Bundles to accept a resgitry1 flavor argument


Minor method renaming that propagated to a lot of places.

We're going to hold off merging this until after the next minor java version release.

@leerho @jmalkin the next minor java version release (3.3.0) is busy propagating to the release mirrors and this should be safe to merge.


Support custom .rclone.conf file location

I too am having trouble with RcloneBrowser (version 1.2) seeing my rclone.conf on Windows (Windows 8.1 x64). I've tried putting the conf file in C:\Users\<username>\.config\rclone and in C:\Users\<username> but neither seems to work. I've to Rclone 1.38 (and, if this is relevant, the rclone.conf is copied from a working Linux installation of Rclone).

Do you want custom location or default one?
For default one just execute rclone config file. It will show location where rclone expects config file to be. It supports multiple locations by defaault, but this command will show just the first default location.

Thanks, but `rclone config file' shows
C:\Users\<username>\.config\rclone\rclone.conf
which is where my conf file is already and yet . . no dice.

Have you set custom rclone.conf location in File->Preferences? Remove it, if you have.

I had set that. I've removed it. No change. By the way, I wasn't sure whether that dialogue wanted the full path, including the leafname, or just the folder. I still have a custom location set for rclone itself.

That field needs full path to .conf file itself, not just the folder. If the field is empty (make sure its empty and not set to space or smth) then RcloneBrowser will use exactly same path has "rclone config file" command shows.
You can check registry setting rconeConf in Computer\HKEY_CURRENT_USER\Software\rclone-browser\rclone-browser\Settings. If it is set to anything, just delete it.

The registry key was blank. I deleted it entirely. No change. I provided full paths (including leafnames) to the Preferences window, for rclone.conf and for rclone.conf. No change. I clicked the 'config' button. Nothing happened.
I am on the verge of giving up.


Issues with the Actions on Google Sample. Throwing an error for response received by Actions on Google Simulator

So I tried the sample, but I'm getting an error at other Intent Handler for Actions on Google. The error that I'm getting on the Actions on Google Simulator reads:

MalformedResponse
expected_inputs[0].input_prompt.rich_initial_prompt: 'item[1]' must not be empty.

Here's a screenshot for the same

What sample are you using?

@matthewayne , I'm using this sample.

This was fixed in v0.4.1

I'm using v0.4.1 and still seeing the issue identified on this thread.  ???


Automated Package Upgrade

:tada: This PR is included in version 2.2.398 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Doc Upate: Copy Edits, etc.

Docs Build status updates of commit 489c390:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/point-and-commit.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Added support for KNM for Convolution using Winograd

Modified convolution using winograd to support KNM architecture

Codecov Report

Merging #139 into master will decrease coverage by -0.08%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #139      +/-   ##
==========================================
- Coverage   12.52%   12.45%   -0.08%
==========================================
  Files         157      157
  Lines       23250    23322      +72
  Branches     4424     4439      +15
==========================================
- Hits         2912     2904       -8
- Misses      20028    20107      +79
- Partials      310      311       +1

Impacted Files
Coverage Δ

...rc/libxsmm_generator_convolution_winograd_driver.c
0% <ø> (ø)
:white_check_mark:

src/generator_convolution_winograd.c
0% <ø> (ø)
:white_check_mark:

...rc/generator_convolution_winograd_forward_avx512.c
0% <ø> (ø)
:white_check_mark:

src/libxsmm_dnn_handle.c
0% <ø> (ø)
:white_check_mark:

src/generator_gemm_common.c
46.92% <ø> (-0.58%)
:x:

src/generator_gemm_sse3_avx_avx2_avx512.c
60.51% <ø> (-0.52%)
:x:

src/generator_x86_instructions.c
16.37% <ø> (-0.25%)
:x:

src/libxsmm_fsspmdm.c
0% <ø> (ø)
:white_check_mark:

src/libxsmm_malloc.c
51.76% <ø> (ø)
:white_check_mark:

src/libxsmm_main.c
48.15% <ø> (+0.14%)
:white_check_mark:

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 7e72501...e1b7a16. Read the comment docs.

Thanks! Merged!


"How to Contribute?" is not documented and has Lorem ipsum as information

Is there an existing issue for this?

[X] I have searched the existing issues

What's missing?
This page available on the website does not contain content for "How to Contribute?" instead only Lorem ipsum is written.
It should be changed to Contributing guidelines.
I am interested to work on this issue. Here is the page that is to be edited.

Hi that page is just default for authors to add text into and not actually a page on how to contribute.


More Settings

Description
Just more settings people can mess with. (Fixed the super long commits history)
Motivation and Context
Wanted to add more settings.
How has this been tested?
Works on my pc
Screenshots (if appropriate):

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Still missing what I said on the other PR, pass the arguments via CLI not editing the jsons. Also, just the significant options

Ok, i tried making it so its sent by CLI but it just doesnt work idk why but i cant figure it out maybe im just not smart enough or maybe i just dont see it. Some help would be greatly appreciated. Also whats wrong with just editing the json? Would it decrease performance or would it just make the code to complicated/bloated.


Publish some reproducible benchmark

We know, that it works. And it works just fine.
But the intuition says that it must be slow.
But how slow? In the age of SSDs and gigs of memory.
We need some numbers!
Ideally, compare to RabbitMQ (with persistent messages) or Kafka (How to deploy this beast? :D )

I will try to create some. Asic comparison to have at least something to share.

@kedlas I tried, but the Publish was very slow. Regardless of driver, or pgq or plain sql in Go. I suspect something in Go takes the time. I'm not sure if I tried the jack/pgx directly 🤔 but the same query to the same server from cli was much faster.


Full vectorized image

Signed-off-by: Maxime Soulé<EMAIL_ADDRESS>

Pull Request Test Coverage Report for Build 225

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

Totals

Change from base Build 223:
0.0%

Covered Lines:
4497

Relevant Lines:
4511

💛  - Coveralls


WIP: Prototype string() function.

@ozevren: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth.sh
87661b2b56c193742b1cad12ab2382398457bbd7
link
/test e2e-mixer-no_auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth.sh
87661b2b56c193742b1cad12ab2382398457bbd7
link
/test e2e-mixer-no_auth

prow/e2e-simpleTests.sh
87661b2b56c193742b1cad12ab2382398457bbd7
link
/test e2e-simpleTests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth.sh
87661b2b56c193742b1cad12ab2382398457bbd7
link
/test e2e-mixer-no_auth

prow/e2e-simpleTests.sh
87661b2b56c193742b1cad12ab2382398457bbd7
link
/test e2e-simpleTests

prow/istio-pilot-multicluster-e2e.sh
889faf7b9820d4f5876c3f2235c45556720f591d
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth.sh
87661b2b56c193742b1cad12ab2382398457bbd7
link
/test e2e-mixer-no_auth

prow/istio-pilot-multicluster-e2e.sh
889faf7b9820d4f5876c3f2235c45556720f591d
link
/test istio-pilot-multicluster-e2e

prow/e2e-simpleTests.sh
889faf7b9820d4f5876c3f2235c45556720f591d
link
/test e2e-simpleTests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Bug with copied images.

this is what I get when I copy an image:

interesting though - When I click on image "copy image", it works fine. but when I take a snapshot and save into the clipboard (cmd+ctrl+shift+3), then I get this unknown format thingie.
Originally posted by @alongouldman in https://github.com/mattDavo/Yippy/issues/1#issuecomment-590790176

This will be fixed in the next release.

Resolved in 2.2.0


New submissions that are not upvoted at all are shown at the very bottom of the main page.

New submissions that are not upvoted at all are shown at the very bottom of the main page.
I think this is how the popularity is calculated.
Simple fix: auto uproot your own posts, comments.

Good call, votes added automatically after posting now.
I'll leave this open as I intend to improve the popularity calculations soon, was looking at these:
http://amix.dk/blog/post/19588  for topics
http://www.redditblog.com/2009/10/reddits-new-comment-sorting-system.html for comments

Good, I read both.
I think there should be a way to temporarily block a certain topic. As in:
"Block anything related to:Dogs____"
in this case you would now see mostly just cats if this was in a realm like /r/aww

I seem to be able to edit your posts...

What do you mean by edit my posts?
For blocking topics, I think that can be part of the keyword search (not available yet).  For example, searching for "cats" would only find cats, but searching for "-cats" would exclude all cats.

I like it!
Also, "no nsfw" and "no violence" toggles would be useful too.
As well as a permanent block option, like searching for -cats but always activated.
After 2 or 3 searches of -cats, an option to perma-block cats would pop up (not as a pop-up, but in the same window)


Time consuming hyperparameter tuning

Should explore the possibility of using pre-trained models, or leveraging transfer learning. Pre-trained models have already been trained on large datasets, so we can use them as a starting point and fine-tune them for our specific task. This approach can save considerable training time compared to training from scratch.


Filename limitation

telegram has length limit on filename which is 60, for example upload a text file with more than 65 characters on the filename, telegram will trim the last 5 characters

Good one! I'll add it shortly


Form elements are too close to each other

We need more margin between Form elements.

problem is resolved in next branch


tag does not load messages in Vue Component (project with Vite)

Reporting a bug?
We are migrating a large multi-language project from Vue 2 on webpack to Vue 3 on Vite and we are encountering a problem when loading the translations for each of the components using the  tag in it.
These are the node dependencies for the new project:
"dependencies": {
    "@primevue/themes": "^4.2.4",
    "@vue-a11y/skip-to": "^3.0.3",
    "axios": "^1.7.8",
    "chance": "^1.1.12",
    "jwt-decode": "^4.0.0",
    "pinia": "^2.2.6",
    "pinia-plugin-persistedstate": "^4.1.3",
    "primeflex": "^3.3.1",
    "primeicons": "^7.0.0",
    "primevue": "^4.2.4",
    "vue": "^3.5.12",
    "vue-cal": "^4.10.0",
    "vue-i18n": "^10.0.5",
    "vue-router": "^4.4.5",
    "vue3-touch-events": "^4.2.0",
    "vuedraggable": "^2.24.3"
  },
  "devDependencies": {
    "@intlify/unplugin-vue-i18n": "^6.0.0",
    "@testing-library/vue": "^8.1.0",
    "@tsconfig/node22": "^22.0.0",
    "@types/jsdom": "^21.1.7",
    "@types/node": "^22.10.1",
    "@vitejs/plugin-basic-ssl": "^1.2.0",
    "@vitejs/plugin-vue": "^5.1.4",
    "@vitest/eslint-plugin": "1.1.7",
    "@vue/eslint-config-prettier": "^10.1.0",
    "@vue/eslint-config-typescript": "^14.1.3",
    "@vue/test-utils": "^2.4.6",
    "@vue/tsconfig": "^0.5.1",
    "axe-core": "^4.10.2",
    "cypress": "^13.15.1",
    "eslint": "^9.14.0",
    "eslint-plugin-cypress": "^4.1.0",
    "eslint-plugin-vue": "^9.30.0",
    "jsdom": "^25.0.1",
    "npm-run-all2": "^7.0.1",
    "prettier": "^3.3.3",
    "sass-embedded": "^1.81.0",
    "start-server-and-test": "^2.0.8",
    "typescript": "~5.6.3",
    "vite": "^5.4.10",
    "vite-plugin-vue-devtools": "^7.5.4",
    "vitest": "^2.1.4",
    "vue-tsc": "^2.1.10"

Y un ejemplo de componente migrado:
<template>

{{ $t('forbidden_title') }}

          {{ $t('forbidden_message', { genderText: userTextGender }) }}

          <fwjsTimer :timeLimit="redirectTimeout" />

</template>

<i18n src="../locales/auth.i18n.json" />

Expected behavior
The message with name ‘forbidden_title’ should display the string ‘You do not have permission’ in the template but we got [intlify] Not found key 'forbidden_title' in 'es' locale messages.
The same happens if we embed the translation strings inside the i18n tag.
Reproduction
The project in its current initial state is in:
git clone git@github.com:SedueRey/portal-ui-vue3-ongoing.git
npm install
npm run dev

You can take a view in home page src/App.vue because the warn also happens with safariButton message, you don't need to configure the whole project
Issue Package
unplugin-vue-i18n
System Info
System:
    OS: Windows 11 10.0.22631
    CPU: (16) x64 13th Gen Intel(R) Core(TM) i7-13620H
    Memory: 47.23 GB / 63.74 GB
  Binaries:
    Node: 20.12.1 - C:\Program Files\nodejs\node.EXE
    npm: 10.8.3 - C:\Program Files\nodejs\npm.CMD
  npmPackages:
    vite: ^5.4.10 => 5.4.11
    vue: ^3.5.12 => 3.5.13
    vue-i18n: ^10.0.5 => 10.0.5

Screenshot
No response
Additional context
No response
Validations

[X] Read the Contributing Guidelines.
[X] Read the README
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion.

Thank you for your reporting!
I've checked your reproduction repo.
I've noticed that your <i18n src="../locales/auth.i18n.json" /> doesn't specify locale attr.
You need it on i18n custom block.

Good morning
I have updated the main branch in the repository with what was commented in the previous comment (my apologies, I thought that as it had both translations it wasn't necessary) but it still can't find the translation key.
I upload, also, two screenshots


chore(backend): template enum values in to command help

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#436
#435  👈
#434
#433
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @NathanFlurry and the rest of your teammates on  Graphite


#1. Install all the dependencies in an easy way

Use Poetry as a dependency manager / virtual environment. This makes sure the user has all the dependencies with a single install command. Also deinstalling gets easier.

Todo:

Change version number to code
Edit author to Google
Add readme
Remove fire for now

Currently fails on libffi:

I locally fix this with the following line, which is macOS-specific:
export PKG_CONFIG_PATH="$(brew --prefix libffi)/lib/pkgconfig/"

Why use poetry instead of regular pip?

Poetry gives some nice handles to manage dependancies.
Example in pip: If you need dependancy A, you also need to install B. If you remove A again, B will still be in your in your project.
But Rod also asked to switch over to pip, so I'll get on to it 🚀


Fix RaycastOcclusionCull World3D scenario memory leak

Fixes https://github.com/godotengine/godot/issues/81476

Related issue https://github.com/godotengine/godot/issues/71884, I don't think this PR fixes that one but in theory its possible due to the changes in thread and other resource lifetime management in this PR. I tried to check the tagged Liblast commit, but it's too old to work with 4.2 without a lot of modifications, including some GDScript changes.
The problem is that RaycastOcclusionCull tracks it's scenarios (what World3D basically does behind the scenes) in a pretty unusual matter. When creating and deleting them using add_scenario() and remove_scenario() it tries to do so lazily. The problem is that when a scenario is deleted, it just marks it as removed but doesn't actually delete it yet. This becomes problem because the only place it actually does the deletion is in RaycastOcclusionCull::buffer_update(), but this is only called if the scenario is active.
The result is that if a World3D / scenario (or a node like Viewport using them) is removed, in many cases the scenario culling data never actually gets freed because it no longer gets updated, so it can't check the removed flag. In the issue MRP RaycastOcclusionCull::scenarios HashMap keeps growing forever as the scenarios are never released as the Viewports are no longer in the scene tree and thus are not updated and cleaned up.
I fixed this by making the add_scenario() and remove_scenario() work in a more traditional way, which makes the lifetime management much easier. I also fixed another leak as scenario commit_thread Thread objects were not always freed properly.
However, as the existing solution was a pretty non-standard compared to other RID allocate / delete API's, there might be a reason why it is done in such a strange way. I tested this PR with a lot of projects including the occlusion culling demo and I could not find any issues, but it's always possible it's done that way because of some obscure corner-case. Or maybe it's just a workaround for some old issue that has already been fixed as I don't see how the existing RID check in add_scenario() can trigger in any normal use, it didn't happen even a single time during my testing.

Thanks!


Dnn Manifest XSD Schema would be nice

Description of problem
Currently the dnn manifest specs are:

Only documented in a handful of outdated blog posts
Described (also outdated) in this old documentation page https://www.dnnsoftware.com/docs/developers/extensions/dnn-manifest-schema.html
The new documentation project is not yet online and a quick search for "manifest" returned nothing

Description of solution
I think understanding the manifest is the first step to understand what makes some piece of code become a Dnn extension.
I am working on creating and XSD file (XML Schema Definition). If we include this file in this repository and have both the EVS and the Dnn installer validate manifests against it, it would force us to maintain the file (one source of truth), xsd also allows self-documentation to quickly explain what is what. Referencing this xsd from a manifest should provide intellisence (code completion) in Visual Studio and many other popular editors. This could also be used as the source of truth by other tools like module templates and generators.
Description of alternatives considered
Watch the bachelor with a few bottles of whisky.
Additional context
I am logging here the assumptions I am making right now from the various information I can find, please correct me where I am wrong. When I have something roughly usable, I will submit a pull request where we can do the final fine-tuning.
Assumptions / Questions about extension types

Connector: I did not find any documentation on this and no matching installer for it here https://github.com/dnnsoftware/Dnn.Platform/tree/development/DNN Platform/Library/Services/Installer/Installers from what I can see it is because there is nothing special about that package type and it just uses the Generic Component types.
DashboardControl, this was removed with Dnn 9, should we still keep it in the xsd because the installer would not crash if it is there? or have multiple versions of the xsd or just keep one version of the xsd but not put this pakage type in?
PersonaBar: I have not found much documentation about the manifest but we have plenty of examples available, I should be fine documenting that one.

Versioning
If there are plans on changing stuff regarding the installer or regarding the supported extension types, should we have a dnn-manifest-v9.xsd and then a dnn-manifest-v10.xsd and so on so we can have different rules? Or XSD 1.1 allows having in a single file different rules depending on a parent parameter (in our case it could be the minimum target version), but I think most Microsoft tools only support XSD 1.0
Any suggestions are welcome.

I, for one, would not recommend the alternative solution suggested.  🤢

Also, while it would indeed be "nice" to have schema's for prior releases, it may indeed be good idea to only focus on 9.xx and newer to have at least one more reason to encourage upgrades.  I wouldn't recommend backwards compatibility for things like this in most cases.  Specifically, Oliver cited security as a concern, and I agree if not for any other reason.

The idea of having them for previous versions is that I would like to have EVS use that as a first step and warn about stuff that does not match the XSD, I guess there are probably still a lot of module vendors that have modules still Dnn 8 compatible being maintained... Maybe not going back too far but I would include Dnn8. Plus I think there would be only 2 small differences between 8 and 9 extensions support (persona bar and dashboard) so it would not be a huge undertaking to make the two.

There would only be additions between 8 & 9, so the DNN 9 XSD wouldn't reject a valid DNN 8 XML file.  The only concern would be a module that targets DNN 8 but uses a feature introduced later.

If there is a small difference between a DNN8 and DNN9 manifest, it would be easy to create a DNN8 XSD as well. That would help to validate DNN8 targeted extensions.

I have tried to accomplish this but because the manifest is extensible and because the extension type is a property instead of a node type, it does not look like we can rely on xsd unless we have multiple and then that would limit is usefulness to only packages with a singly type of of extension. So for the time being I think this should be abandoned and I am closing the issue.


typo in step2 of "Mount the Azure file share on-demand with mount"

"smbPath=$(echo $httpEndpoint | cut -c7-$(expr length $httpEndpoint))$fileShare" should be "smbPath=$(echo $httpEndpoint | cut -c7-$(expr length $httpEndpoint))$fileShareName" instead

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 7af47c05-12ce-1bc0-7086-4a2556f51e13
Version Independent ID: 8ec694f1-3001-d97a-a6d2-fcde97b00be0
Content: Use Azure Files with Linux
Content Source: articles/storage/files/storage-how-to-use-files-linux.md
Service: storage
Sub-service: files
GitHub Login: @roygara
Microsoft Alias: rogarana

@ccontino84 Thank you for your contribution,
We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


[FEATURE] - Save graph filters

I would really love this! I see it's on your possible future features list. Do you have any plans to do it? Thanks :) Laura

Saving them as URIs, which could be linked to from within notes, would be very useful. Even better if they could be embedded in notes.

Hi, yeah, I have plans!  Rest assured this plugin isn't abandoned, I'm just currently busy finishing my university studied.

Great to hear - thanks! Good luck with uni first though :)

Awesome! Love your owl portrait

Sounds great. I'd love it if we could have different sets of filters and switch fast between them.


concourse 7.3.1 upgrade failure

Hi Team,
IaaS:  vSphere
Mode: Cluster
Current Versions:
bpm/1.1.9
concourse/7.2.0
postgres/43
Stemcell/xenial_621.123
Upgrading To:
bpm/1.1.1
concourse/7.3.1
Stemcell/Bionic_1.1
Failed compiling bpm-runc package.  Install and partial task debug log are attached
DEBUG.txt
install.log

The bpm-release repo is a better place to ask why their package fails to compile: https://github.com/cloudfoundry/bpm-release
We did test deploying with all the versions you're upgrading to and it worked: https://ci.concourse-ci.org/teams/main/pipelines/concourse/jobs/bosh-topgun-core/builds/692

@taylorsilva : BPM Team released 1.1.12 that fixes this issue. Do we have any ETA on including this in concourse manifest?
https://github.com/cloudfoundry/bpm-release/releases/tag/v1.1.12
https://github.com/cloudfoundry/bpm-release/issues/149

It gets automatically bumped whenever we release a new version of Concourse. You can of course override this yourself anytime locally. Not sure how your deployment works but this should be an option. Is there any reason you can't do that?

Thats fine. Don't want to manually create and edit a local manifest file.


Prune all EventStream events

BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1348625 (1 of 6)
I am going through our components to make sure they are pruning tables correctly.
Before:
It is currently pruning system events that have type="EventStream" class.
This is how the original code was written, but when a new top level class was introduced, it probably was overlooked.
After:
Now, it prunes all EventStream events.

So, from what I can tell we were purging EmsEvent records properly, just not all EventStream records?  That is, technically, before the rename and after the rename everything was the same and working correctly.  So, this is something new in that system events are now purged after 6 months.  @gmcculloug Is this ok?

This is not a performance thing, so removing the label...It's technically a new feature (purging of system events).

EventStream records contain provider events (EmsEvent records) and our events for policy and request.
Before table rename, only EmsEvent records are saved into DB.
After table rename, these policy events and request events are saved in DB as well.

fixed failing test.
@Fryguy In the BZ, Joe sold it to me as performance because we will be trimming down these tables.
@lfu Is this something else we want to do here?

@lfu Oh I see...we never held them before, so this is a needed enhancement anyway.

LGTM @kbrock .  Will merge when green.  Let's clean up the two minor things in follow up PRs.

@kbrock Nice job! 👍

Euwe Backport details:
$ git log
commit f915117f045f9e2308428cb1b03d921bae5a6b7b
Author: Jason Frey<EMAIL_ADDRESS>Date:   Tue Oct 11 18:19:43 2016 -0400

    Merge pull request #11813 from kbrock/prune_system_events

    Prune all EventStream events
    (cherry picked from commit a9497c3d792341e1c81113a7a071e634a92bdc03)

    https://bugzilla.redhat.com/show_bug.cgi?id=1348625

@simaishi please backport to darga per https://bugzilla.redhat.com/show_bug.cgi?id=1412448

Darga backport details:
$ git log -1
commit 71935b1eb82ebf7559affd376c603c599305f10a
Author: Jason Frey<EMAIL_ADDRESS>Date:   Tue Oct 11 18:19:43 2016 -0400

    Merge pull request #11813 from kbrock/prune_system_events

    Prune all EventStream events
    (cherry picked from commit a9497c3d792341e1c81113a7a071e634a92bdc03)

     https://bugzilla.redhat.com/show_bug.cgi?id=1412448


pythonPackages.clickhouse-driver: drop nose dependency

Description of changes
This packages don't use nose

#326513

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

It looks like the ClickHouse package works now; we might be able to run these tests?

Still missing clickhouse as the comment says

What do you mean? We have a clickhouse package that seems to be building successfully on the unstable channel these days. I assume at the time of the comment it wasn’t working.

What do you mean? We have a clickhouse package that seems to be building successfully on the unstable channel these days. I assume at the time of the comment it wasn’t working.

Oh yeah, well added it, not much of a progression:
================== 30 passed, 1 skipped, 418 errors in 5.12s ===================

What should I do?

Seems like this needs a running clickhouse which is pretty out of scope to add in this PR.


Fix encoding Falcon secret key basis polynomials

Describe your changes
Puts the field elements in the correct range before encoding them.
Checklist before requesting a review

Repo forked and branch created from next according to naming convention.
Commit messages and codestyle follow conventions.
Relevant issues are linked in the PR description.
Tests added for new functionality.
Documentation/comments updated according to changes.

Indeed, that's the aim of this PR.


chore(master): release 1.2.48

:robot: I have created a release beep boop
1.2.48 (2023-09-01)
Miscellaneous

deps: update dependency wayofdev/cs-fixer-config to v1.2.50 (ee018eb)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wayofdev/laravel-symfony-serializer/releases/tag/v1.2.48 :sunflower:


Feature/robust ast parsing

flows summarize

Hello, I am a code review bot on flows.network.
It could take a few minutes for me to analyze this PR. Relax, grab a cup of coffee and check back later. Thanks!


resolves #2205 - middleware instantiation error in 1.10

I tested this locally and error went away and local server started.

Fixed via a1786228dd2013a6b56e4270c787c4ea860d36b8


Additional Components Needed

Currently we have bar charts.  Working on Line Graphs.  Would like to add in other visualizations (pie, scatter, stacked bar, radar, doughnut, etc).

Pie charts added as of current version


Fix the volume value

<input type={'range'} min={'0'} max={'100'} onChange={ this.setVolume } /> is ok.
But I'd like the volume at 100 at start. But when setting the value parameter:
<input type={'range'} min={'0'} max={'100'} value={'100'} onChange={ this.setVolume } />, then, even if the change event is triggered while changing volume, the selector stay at the same place.
I think it is related to Electron and not React, but I'm not sure.

Fixed in full-refactoring branch.


MINOR: Improve LogCleaner buffer size change logging

Right now it looks like the buffer size only grows if you are watching the cleaner logs. It would be nice if we also logged when the buffer size was restored to avoid confusion when the log just keeps saying the buffer has grown repeatedly to and from the same values.

@cotedm, ping in case you forgot about this. :)

@ijuma thanks, I had forgotten :) Updated to move inside the if instead of logging every time we call restoreBuffers()

@ijuma this one look ok now? should I retarget to 0.11?


DataOne Links should go somewhere

When searching DataOne many links are exclusively DOI's.  When you click on them they ask you to find a program to open them.  If they could be converted to a website URL so that functionality is consistent it would be great.

yesssssss :+1: :+1:
I don't think this happens very often, for the record. @amb8805, correct me if I'm wrong.

It happens very often from the queries i've ran.

really? yikes yea we should be able to construct a DOI url ourselves.

Also i have found some results are also xml files


refactor(rust): slim down the NodeManagerWorker for node / tcp

This is my try at solving https://github.com/build-trust/ockam/issues/6708

@rbran we're almost there. You need to rename your commits to avoid capital letters, then we should be good

@rbran we're almost there. You need to rename your commits to avoid capital letters, then we should be good

Should I squash the commits?

@rbran can you please open a separate PR to accept the CLA: https://github.com/build-trust/ockam/blob/develop/.github/CONTRIBUTORS.csv? Thank you.

@rbran sorry I missed your CLA PR! Let's merge and re-run


Add generic regs.gov link option to RegComment

Stakeholders would like the link in this sentence to go directly to the comment form on Regs.gov for the specific document:

If you would like to include an attachment with your response, please submit your comment at Regulations.gov.

We can't do this ALL the time, because in situations like the Payback Playbook, we may need to publish our commenting form before the document is fully populated with content on Regulatons.gov, and we don't want people going to an empty page. Thus, the need for this new option in the Wagtail module.
Additions

RegComment organism: New option to use a generic link for commenting at Regulations.gov or going directly to the specified document's comment form.

Review

@kurtw
@rosskarchner
@kave
@richaagarwal
@sebworks
@anselmbradford
@jimmynotjim
@KimberlyMunoz

Todos

This option (and the statement as a whole) can be removed when/if we add the ability to submit attachments via our form.

Checklist

[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the front end playbook
[ ] Passes all existing automated tests
[ ] New functions include new tests
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)

Looks good to me 👍

👍


I had tried what u had said but i am facing an error by following these link  https://m.youtube.com/watch?v=UVOV2hTkkvE

sry bro...........i had got the calendar UI.

Do you still have this issue?

Solved offline.


return images (PNG) not only JSON

We can solve this problem by making the  R script return a base64 encoded image which will be passed to the lambda handler. In order for the API to be able to respond to a GET request for an image,  binary payloads should be activated at the integration response level of the API.
E.g
aws apigateway update-integration-response \   --profile davemaple \   --rest-api-id [api_id] \   --resource-id [resource_id] \   --http-method GET \   --status-code 200 \   --patch-operations '[{"op" : "replace", "path" : "/contentHandling", "value" : "CONVERT_TO_BINARY"}]'

The main problem still remains the fact that a plot generated by R should be saved in the file system and then encoded as base64 (using base64enc package). When R generates a plot, it automatically looks for a file called  Rplots.pdf in order to write it, and this file cannot be found when the script is run in a Lambda environment.
A possible solution would be to deactivate this kind of behavior of the R plotting functions.


Is the cluster configuration documentation correct?

Our cluster was created by a partner, but I don't see the "Read/Write" or "Edit" options in the following steps:

In the top right corner, select Read/Write.
Select Edit and update the fabricSettings JSON element and add a new element:

If this documentation doesn't apply to my situation, it would be nice to have a note as to why - and how to determine that.
Also, in case this is relevant, I tried using "Custom fabric settings" setting to add the option we need ("PruneContainerImages") but it's not in the "Hosting" section "Parameter" drop-down.
It would be great to have a pointer to info on where and how to change these other options.
Cheers,
Nick.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 8aac72f9-bdf3-c656-ae5c-1475c413de13
Version Independent ID: dd937997-96cb-16c3-8cdf-6d6977ad4b92
Content: Upgrade the configuration of an Azure Service Fabric cluster - Azure Service Fabric
Content Source: articles/service-fabric/service-fabric-cluster-config-upgrade-azure.md
Service: service-fabric
GitHub Login: @dkkapur
Microsoft Alias: dekapur

@sicklittlemonkey
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.

@dkkapur
Can you please check and add your comments on this doc update request as applicable.

Sorry, the original issue isn't valid - it's my mistake. I didn't know about ARM templates and didn't notice the URL was resources (not portal) dot azure dot com - this is all new to me.
So the documentation is correct, sorry for the confusion.
I still wish that the "Custom fabric settings" had the option I want to change, but I will try to use the resource manager.

Thanks for following up. I'll close the issue now #please-close


Renaming of OptionException should be mentioned in changelog

It was described in README and was clearly part of API so it was kind of surprising seeing such API breakage in "minor release".

Ah I missed that one. I'll add it to the readme for the version where it was changed.


help

[PermissionsEx] Error occurred with PermissionsEx!

Obviously, you need to give us a LOT more information than that if you want help.

What's the error?
What version of Minecraft are you running?
What version of PEX are you using?

No reply in 9 days.  Issue must be resolved.


Nuget Package when viewed within Visual Studio does not have a License field

Compare:

With:

oh, And can it also have Themes in the Tags as well ;-)

Will fix that on the next NuGet release. I still need to find a VS extension to manage these, as I've been relying on NuGet Package Explorer.

Please also include the *.xml files in the nuget package

Would that be the theme files from the extended toolkit?

No, They are created during compile to aid intelli-sense for the API's etc.
They have the same name as the dll, but with the xml extension

Ahh, ok will upload them in a .1 release.

Done, will update other packages tomorrow.


Add USPS FIM E mark

The USPS created another FIM mark, named FIM E, with the binary code 101000101.
I think I've got the encoding right - I've tested it by comparing it side-by-side (but vertically, so misaligned bars would be very noticeable) against the FIM-D (which has every bar the FIM-E does and then some) and it looks visually correct, but it's possible I've made a mistake in encoding it.
Here's an example output:

.

Looks good. Thanks!


[type: refactor]Replace with computeIfAbsent in ExistedValidator.

Make sure that:

[ ] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[ ] Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #4297 (88a1b66) into master (ecc9937) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #4297      +/-   ##
============================================
- Coverage     68.29%   68.29%   -0.01%
- Complexity     7354     7356       +2
============================================
  Files          1003     1003
  Lines         28378    28373       -5
  Branches       2545     2544       -1
============================================
- Hits          19382    19378       -4
+ Misses         7509     7507       -2
- Partials       1487     1488       +1

Impacted Files
Coverage Δ

...u/admin/validation/validator/ExistedValidator.java
73.33% <100.00%> (-6.67%)
:arrow_down:

...che/shenyu/sync/data/http/HttpSyncDataService.java
85.71% <0.00%> (-4.09%)
:arrow_down:

...henyu/plugin/grpc/resolver/ShenyuNameResolver.java
56.38% <0.00%> (+1.06%)
:arrow_up:

...ruptor/RegisterClientServerDisruptorPublisher.java
64.70% <0.00%> (+11.76%)
:arrow_up:

...controller/ShenyuClientHttpRegistryController.java
100.00% <0.00%> (+22.22%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Fix a broken link in misc section

The link https://crowell.github.io/blog/2014/11/24/hosting-a-local-kernel-ctf-challenge/ seems to
be broken.
The owner of the blog had removed his blog and put all the files into another repository, which has
the same content.
The link to that has been added in this patch.

Thanks!


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 56 ms

Resolved: ClamAV performance has improved in 7fa331f.


EXEC without MULTI error

I am a newbie at Redis and HiRedis, but I know my way around C coding.
I have a C application that makes use of the HiRedis API. It does seem to work as I expected, but I have come across some issues that I just can't sort out.
I am using HiRedis 0.13.3, with Redis 4.0.9, from a C application under Linux. In this application I have a function F, invoked in a POSIX thread, that, in essence, consists of a loop where a number of invocations of redisCommand() are made. The command arguments to redisCommand() in this loop are the following:
   MULTI
   SMOVE
   HSET
   EXEC

This set of commands is executed 100 times in the loop.
This does what I expect, in that the SMOVE and HSET operations do what they are supposed to do - I have verified this using the redis-cli tool. I have, however, encountered the following issues:

When invoking the SMOVE or HSET commands from redisCommand(), the value in the 'type' field of the returned redisReply pointer can be either REDIS_REPLY_INTEGER or REDIS_REPLY_STATUS. Why? When I do the sequence above by hand, at the redis-cli tool, MULTI returns OK (obviously REDIS_REPLY_STATUS), SMOVE and HSET return QUEUED (which is REDIS_REPLY_STATUS as well), and EXEC returns an array with two integers, which I interpret to be the return values of the SMOVE and HSET commands, after they were queued.

The function F gets invoked a number of times without any problem (other than the issue above.) However, all of a sudden the redisCommand() invocation for 'EXEC' returns with REDIS_REPLY_ERROR in the 'type' field, and the following diagnostic in the 'str' field:
ERR EXEC without MULTI

The thing is, I know for a fact that MULTI has been successfully invoked, as described in the sequence above - I have lots of tracing code that tells me that redisCommand(con, "MULTI") returns REDIS_REPLY_STATUS.
Why would the redisCommand(con, "EXEC") invocation all of sudden think that there is no matching 'MULTI' when, in fact, there is one?
A few comments about the application:

It consists of a main thread M that carries out other HiRedis operations with a HiRedis context C.
M launches the thread that runs F, using a separate HiRedis context C'. C and C' are two different HiRedis contexts.
Ultimately, M will launch several threads where F will run concurrently, each instance with its own redisContext. For simplicity in this report, M is launching a single F thread - I have tested with M launching several F threads, and the issue is (unsurprisingly) still there.

Hi @ijklsd,
It's hard to say exactly what's happening with your application without a small program we can use to replicate the problem.
I think you've correctly diagnosed that you're getting out of sync somehow, and that the integer replies you're seeing are either from HSET or SMOVE when your application is expecting a reply from MULTI.
If you're truly working with a "share nothing" setup, then this should work in theory.  The symptoms you're describing (weird hard to reproduce failures and being out of sync) sure sounds threading related though. 😀
I wrote a little program that does what you're describing (several worker threads executing MULTI...SMOVE...HSET...EXEC in a loop, pulling jobs pushed to a list from the main thread, but I am not seeing the behavior you describe.
You could try using valgrind --tool=helgrind and see if anything immediately jumps out at you.

Were you able to solve your problem?
I'm going to close the issue because I don't think there is a bug here, but if you can provide an example program that triggers the problem I'm happy to investigate further.


Fix harbor api webpage access issue

Issue #, if available:
Can't access harbor api webpage
Description of changes:
Fix harbor portal image.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve


Update Varnish 6.0 (LTS) and 6.2 versions

This updates to the 6.0.4 and 6.2.1 versions of Varnish, which address the VSV00003 DoS attack vector (CVE-2019-15892).
More info here: https://varnish-cache.org/security/VSV00003.html

lgtm

I've updated this PR to the 6.0.5 and 6.2.2 versions of Varnish, which addresses the VSV00004 workspace information leak.  More info here: https://varnish-cache.org/security/VSV00004.html
I've switched to the official Docker image for Varnish but I feel like there's still some value in merging this to update the versions, so long as this remains available on Docker Hub.

In the process of updating the official Varnish Docker images (a different repo with the same name) and doing some cleanup after the merge, I accidentally deleted my remote branch for this repo which contained these updates.  I've restored the branch, so I'm reopening this.

I updated this PR to use versions 6.0.6 (LTS) and 6.2.3 to address VSV00005.  I imagine this repo isn't being maintained anymore but I figured I may as well update this anyway.

ping @teohhanhui ?

Version 6.0.7 (LTS) has been released and the download URL now contains a unique string, so this repository's update.sh file doesn't generate the correct URLs anymore. update.sh creates https://varnish-cache.org/_downloads/varnish-6.0.7.tgz (404), whereas the URL from the first-party website that works is https://varnish-cache.org/_downloads/41841608341add28256b374dc367af04/varnish-6.0.7.tgz.
Seeing as this repository hasn't been updated to include the 6.3.x, 6.4.x, or 6.5.x releases and this PR for important security fixes to the 6.0.x and 6.2 series has been open and unmerged for over a year, I'm going to go ahead and close this. Bringing this repository up to date would require more work than what's in this PR and users are better off simply using the official varnish images at this point.


Characters is Deprecated

In Swift 4 / Xcode 9.1+, String's characters property is now deprecated and produces a compiler warning. The receiver should be used instead of accessing the characters property (they're the same thing).

closing as dupe of https://github.com/adamwaite/Validator/pull/85


should '_LocalProxy__local' actually be '__local' ?

https://github.com/pallets/werkzeug/blob/d388a734aa4785b617fba5ab6c345841ef64c7d2/werkzeug/local.py#L293
I was recently curious about how flask.request work, then I found https://stackoverflow.com/questions/20036520/what-is-the-purpose-of-flasks-context-stacks
discussed the local.py file, and  ?

Calling __setattr__ does not trigger name mangling so you need to do i yourself here.
Also, please use IRC or Stack Overflow for such questions; they do not belong here.


Adding Metrics

Begining the process of adding metrics. Currently functions for OfficeController. will expand to more with annotations, time permitting.

Looks good to me. I could never figure out a great use case for the health check registry because everything was either all-up or all-down but maybe this time it will make more sense.

Yeah, mostly it's there because the metrics servlet we want breaks without it, but perhaps it will become useful over time.


isMaster fix

What this PR does / why we need it:

Correcting the function to check if the pod has a master role or not in case there is only 1 worker in the PyTorch job.

Which issue(s) this PR fixes (optional, in Fixes #<issue number>, #<issue number>, ... format, will close the issue(s) when PR gets merged):
Fixes #1961
Checklist:

[ ] Docs included if any changes are user facing

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

5 of 5 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 42.896%

Totals

Change from base Build<PHONE_NUMBER>:
0.01%

Covered Lines:
3756

Relevant Lines:
8756

💛  - Coveralls

Related: #1961

/lgtm
/approve

Could we make sure that the PR titles are descriptive before merging next time?


btn-group adds borders to children of children

Current Behavior
When using the .btn-group like so:

    <button>

Text

				<svg class="w-6 h-6">(some Icon)<svg>
    </button>
    <button>
			  =
Text

				<svg class="w-6 h-6">(some Icon)<svg>
    </button>

The buttons have a divider, but the text and icon inside the button also have a divider which is not wanted/expected behavior.
Illustration from @endigo9740:
https://cdn.discordapp.com/attachments/1151114412896309330/1151254611227852860/Screenshot_2023-09-12_at_3.34.37_PM.png
Expected Behavior
Only direct buttons and anchors have a border, the rest is ignored
Steps To Reproduce

Using the btn-group class
Apply it to a div
Add a button
Add a p to the button
Add a svg to the button
Observe small divider is inside the button

Link to Reproduction / Stackblitz
https://stackblitz.com/edit/skeletonlabs-repl-cabu4o?file=src%2Froutes%2F%2Bpage.svelte
More Information
This was discovered and discussed here: https://discord.com/channels/1003691521280856084/1151114412896309330/1151114412896309330
A possible fix would be to use the > css selector to only select children 1 level deep into the dom from the element where the btn-group is applied. We could also only allow button and anchor elements to be targeted with these styles so folks can even have other elements in the root level without being affected.

I'd suggest adding a new class, for example group-item and targeting the class instead to get rid of any conflicts.
On the down side, this would be a breaking change 😕

@Mahmoud-zino the change proposed would be very simple and shouldn't rock the boat at all
.btn-group *+* { ... }

Now becomes:
.btn-group > *+* { ... }

Simple and easy, and ensures it both works as originally intended, and meets users expectations.


[question] Enforce internal/parent import order with alias

Hello!
I can't find a way to enforce import order to internal / parent using aliases. Maybe it is just not possible.
Example:
--- src
     |--- folderA
           |--- fileA1.ts
           |--- fileA2.ts
     |--- folderB
           |--- folderC
                 |--- fileC1.ts
           |--- fileB1.ts
           |--- fileB2.ts

Set an alias in my tsconfig.json
{
  "compilerOptions": {
    ...
    "baseUrl": "./",
    "paths": {
      "@src/*": ["src/*"],
    }
  }
}

In fileC1.ts
import { foo } from '@src/folderA/fileA1'
import { bar } from '@src/folderB/fileB1'

I would really love to be able to enforce the import from B (parent) to happen after import A (internal) but I'm not sure it is possible since I believe that alias would "hide" it.
Thanks in advance!

I'm unclear on what you're asking. Wouldn't folderA and folderB both be internal?

Maybe I'm just totally wrong but based on my understanding being in fileC1.ts, file fileB1 is a parent and fileA1.ts is internal. So given a configuration like
'import/order': [
      2,
      {
        groups: ['builtin', 'external', 'internal', 'parent', 'sibling', 'index'],
        'newlines-between': 'never'
      },
    ],

I would like to enforce
import { bar } from '@src/folderB/fileB1' after import { foo } from '@src/folderA/fileA1'
With the above given config and (I believe) since I'm using alias the linter doesn't care about the order so:
import { bar } from '@src/folderB/fileB1'
import { foo } from '@src/folderA/fileA1'

is still valid.

oh right, you're in fileC at that point. Yes, folderB would be a parent and folderA would be internal.
So what you're saying is, you WANT them to be sorted so that folderA comes first (because it's internal), but because it's using a TS alias, it's recognizing it as a parent?
Can you share your eslint config? Hopefully you're using the TS eslint resolver.

You got it! (btw I don't really know if it's folderA recognize as parent or is it viceversa - so it's folderB recognize as internal, what I know is that switching their order would not trigger any error)
eslintrc.js
module.exports = {
  parser: '@typescript-eslint/parser',
  parserOptions: {
    project: 'tsconfig.json',
    sourceType: 'module',
    tsconfigRootDir: __dirname,
  },
  plugins: ['@typescript-eslint/eslint-plugin', 'unused-imports'],
  extends: [
    'plugin:@typescript-eslint/recommended',
    'plugin:import/recommended',
    'plugin:import/typescript',
    'plugin:prettier/recommended',
    "plugin:@dword-design/import-alias/recommended",
  ],
  root: true,
  env: {
    node: true,
    jest: true,
  },
  ignorePatterns: ['.eslintrc.js'],
  rules: {
    '@typescript-eslint/explicit-function-return-type': 1,
    '@typescript-eslint/explicit-module-boundary-types': 0,
    '@typescript-eslint/no-explicit-any': 0,
    '@typescript-eslint/no-inferrable-types': 0,
    '@typescript-eslint/no-shadow': 1,
    '@typescript-eslint/no-unused-vars': [1, { ignoreRestSiblings: true }],
    'import/namespace': 0,
    'import/no-extraneous-dependencies': [2, { devDependencies: ['**/test/**'] }],
    'import/order': [
      2,
      {
        groups: ['builtin', 'external', 'internal', 'parent', 'sibling', 'index'],
        'newlines-between': 'never'
      },
    ],
    'lines-between-class-members': [2, 'always', { exceptAfterSingleLine: true }],
    'unused-imports/no-unused-imports': 2,
  },
  overrides: [
    {
      files: ['*.decorator.ts'],
      rules: {
        '@typescript-eslint/explicit-function-return-type': 0,
      },
    },
  ],
  settings: {
    'import/parsers': {
      '@typescript-eslint/parser': ['.ts', '.tsx'],
    },
    'import/resolver': {
      typescript: {
        alwaysTryTypes: true,
      },
    },
  },
};

cc @JounQin if the TS resolver is reporting @src/folderB/fileB1 as a parent instead of internal, there's not much the plugin can do, I think.

cc @JounQin if the TS resolver is reporting @src/folderB/fileB1 as a parent instead of internal, there's not much the plugin can do, I think.

@ljharb Sorry I didn't notice the mention previously, but I'm not quite sure to understand why it's related to the TS resolver, does it resolve a incorrect absolute path?

I'm not certain, but it seems like it's resolving it as a relative path (a parent) rather than a bare specifier (in node_modules).

@ljharb

I'm not certain, but it seems like it's resolving it as a relative path (a parent) rather than a bare specifier (in node_modules).

I'm still confused, it's using TS alias, and @src is not a bare specifier (in node_modules).
Besides, @andreafspeziale can you please provide a minimal reproduction?

Right, but a TS alias should still report as if it's npm linked in, i assume?

@andreafspeziale Can you try DEBUG=eslint-import-resolver-typescript first to see what absolute paths are resolved?

Right, but a TS alias should still report as if it's npm linked in, i assume?

@ljharb I don't think that's possible, ts resolver return absolute paths, it's not npm linked, is it?

@JounQin Sure I'll try to ship a sample ASAP

@JounQin @ljharb As promised.
https://github.com/andreafspeziale/eslint-issue-sample
So we can also be sure that the issue is not me messing-up something or just not understanding that it's simply not possible since aliases are "flattening" everything ^^


Add more explanation about what it means to be a distinguished contributor

This adds a bit more text to the listing of distinguished contributors quoting what the recognition means from the governance docs.

CC @afshin

Thanks for the review, Darian!