fix stew build with local dependencies

There's a breaking change upstream, it's documented in the code. This creates a new code path that aims at reusing the old code path 😎

link to the actual failure: https://github.com/coveo/stew/actions/runs/9954505753/job/27500276639#step:3:278


commander delete GPS receiver checks and minor cleanup

https://github.com/PX4/Firmware/pull/8914/commits/b6ee38ebe50b2bb8d1d7b36fe91e7bde457b982f -
Due to the gpssim driver and a rare race condition the SITL tests are occasionally getting stuck in gps failure. The commander GPS receiver checks are superseded by the estimator checks and probation periods for local and global position validity. This seems like a good excuse to delete the old code.
The other commits are minor commander cleanup I came across in the process.

CI passing.


Improve camel-quarkus-velocity test coverage

Add tests for:

Custom Velocity context
Dynamic templates

I remember scenarios where velocity is using a lot of reflection. Then, I wonder if it would would be interesting to add more coverage based on camel tests above ?
https://github.com/apache/camel/blob/main/components/camel-velocity/src/test/java/org/apache/camel/component/velocity/VelocityMethodInvokationTest.java
https://github.com/apache/camel/blob/main/components/camel-velocity/src/test/java/org/apache/camel/component/velocity/VelocityBodyAsDomainObjectTest.java

@aldettinger I can try that. I made the tests according to the documentation.
Just for your information, there is a lot of coverage already in the repository.
For example https://github.com/apache/camel-quarkus/blob/main/integration-tests/velocity/src/test/java/org/apache/camel/quarkus/component/velocity/it/VelocityTest.java#L79 and https://github.com/apache/camel-quarkus/blob/main/integration-tests/velocity/src/test/java/org/apache/camel/quarkus/component/velocity/it/VelocityTest.java#L129
(which at first glance should cover both tests you mentioned)
This PR only adds 2 missing configuration options which were not covered so far.

@JiriOndrusek Good point. I've double-checked and actually the camel-quarkus testBodyAsDomainObject is already covering both. The involved classes just need to be marked as @RegisterForReflection.
I wonder if a cq-velocity specific doc statement would be needed to document that ?

There is no mention of registering for reflection in the doc - https://github.com/apache/camel-quarkus/blob/main/extensions/velocity/runtime/src/main/doc/configuration.adoc
I'll add a chapter about it and ammend it to this PR.


Errors after adding lsps

I tried adding custom lsp servers in lsp.fnl (I just added them to the array after installing server itself)
For example in Svelte files I get
Error executing vim.schedule lua callback: ...ache/nvim/hotpot/Users/icy/.config/nvim/fnl/pack/lsp.lua:33: module 'telescope.builtin' not found:
        no field package.preload['telescope.builtin']
        no file './telescope/builtin.lua'
        no file '/usr/local/Cellar/luajit-openresty/2.1-20220411/share/luajit-2.1.0-beta3/telescope/builtin.lua'
        no file '/usr/local/share/lua/5.1/telescope/builtin.lua'
        no file '/usr/local/share/lua/5.1/telescope/builtin/init.lua'
        no file '/usr/local/Cellar/luajit-openresty/2.1-20220411/share/lua/5.1/telescope/builtin.lua'
        no file '/usr/local/Cellar/luajit-openresty/2.1-20220411/share/lua/5.1/telescope/builtin/init.lua'
        no file '/Users/icy/.cache/nvim/packer_hererocks/2.1.0-beta3/share/lua/5.1/telescope/builtin.lua'
        no file '/Users/icy/.cache/nvim/packer_hererocks/2.1.0-beta3/share/lua/5.1/telescope/builtin/init.lua'
        no file '/Users/icy/.cache/nvim/packer_hererocks/2.1.0-beta3/lib/luarocks/rocks-5.1/telescope/builtin.lua'
        no file '/Users/icy/.cache/nvim/packer_hererocks/2.1.0-beta3/lib/luarocks/rocks-5.1/telescope/builtin/init.lua'
        no file './telescope/builtin.so'
        no file '/usr/local/lib/lua/5.1/telescope/builtin.so'
        no file '/usr/local/Cellar/luajit-openresty/2.1-20220411/lib/lua/5.1/telescope/builtin.so'
        no file '/usr/local/lib/lua/5.1/loadall.so'
        no file '/Users/icy/.cache/nvim/packer_hererocks/2.1.0-beta3/lib/lua/5.1/telescope/builtin.so'
        no file './telescope.so'
        no file '/usr/local/lib/lua/5.1/telescope.so'
        no file '/usr/local/Cellar/luajit-openresty/2.1-20220411/lib/lua/5.1/telescope.so'
        no file '/usr/local/lib/lua/5.1/loadall.so'
        no file '/Users/icy/.cache/nvim/packer_hererocks/2.1.0-beta3/lib/lua/5.1/telescope.so'
stack traceback:
        [C]: in function 'require'
        ...ache/nvim/hotpot/Users/icy/.config/nvim/fnl/pack/lsp.lua:33: in function '_on_attach'
        ...pack/packer/opt/nvim-lspconfig/lua/lspconfig/configs.lua:280: in function '_setup_buffer'
        ...pack/packer/opt/nvim-lspconfig/lua/lspconfig/configs.lua:183: in function ''
        vim/_editor.lua: in function <vim/_editor.lua:0>

Although  lsp and cmp works perfectly afterwards
I also get error [lspconfig] Cannot access configuration for rust-analyzer. Ensure this server is listed in `server_configurations.md` or added as a custom server.  when editing any .rs file, I have rust-analazyer installed through Mason and brew
Am I missing something?

Happens to me from time to time as well, and I'm not sure why. I reckon it's probably due to the weird way 'rust-tools.nvim' is loading the lsp_server, though I'm not sure how to fix it. I'll take a look again soon

Ok then, I will be waiting

Happens to me from time to time as well, and I'm not sure why. I reckon it's probably due to the weird way 'rust-tools.nvim' is loading the lsp_server, though I'm not sure how to fix it. I'll take a look again soon

This is possibly because the way you lazyload the plugins, sometimes Telescopes is loaded after LSP hence it's module was not found.

@babygau rust-tools.nvim and/or lsp shouldn't be dependent on Telescope, I'm not really sure whats happening here.

@babygau rust-tools.nvim and/or lsp shouldn't be dependent on Telescope, I'm not really sure whats happening here.

But this is what I see in your lsp config
https://github.com/shaunsingh/nyoom.nvim/blob/main/fnl/pack/lsp.fnl#L38-L42
Btw, just my guess 😀

Oops, forgot about that piece . Thanks ill move that mapping over to be loaded on telescopes load :)

Fixed in a seperate branch that contains some work I'm doing on modularizing the config a bit more. I'll back port the changes soon
https://github.com/shaunsingh/nyoom.nvim/blob/ae23a9884f9d030aa66f667e17f6dd08a20e8b1e/fnl/modules/tools/lsp/config.fnl#L52-L58

Fixed in a seperate branch that contains some work I'm doing on modularizing the config a bit more. I'll back port the changes soon
https://github.com/shaunsingh/nyoom.nvim/blob/ae23a9884f9d030aa66f667e17f6dd08a20e8b1e/fnl/modules/tools/lsp/config.fnl#L52-L58

Does it fix Rust-analyzer problem?

Oh, ok, maybe a MacOs problem, I will have test on Arch, huge thanks anyway! Love your work

I think it should be fixed now, on main


filetype: mplstyle files are not recognized

Problem: mplstyle files are not recognized.
Solution: Add a pattern for mplstyle files

thanks!


Need to add some note to indicate that Basic Auth is still required if the end users use security & compliance cmdlet since connect-ippssession is required for them

Need to add some note to indicate that Basic Auth is still required if the end users use security & compliance cmdlet since connect-ippssession is required for them. customers are confused that since v2.0.6 preview doesn't need basic auth, they though connect-ippssession doesn't need as well.

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

ID: de2836a0-0af3-c147-2d4a-78220ac4e0f1
Version Independent ID: 2cde9735-6281-bb9f-bf07-b30b2f6c7f37
Content: About the Exchange Online PowerShell V2 module
Content Source: exchange/docs-conceptual/exchange-online-powershell-v2.md
Service: exchange-powershell
GitHub Login: @chrisda
Microsoft Alias: chrisda

@frankchen76, updates per referenced PR. Updated topics should publish by ~ 4:00 PM PDT M 7/25.
Thanks for the feedback.
Closing


Move Integration Extensions How-tos to Refguide

Hi @quinntracy ,
Overall it looks good! Some suggestions for improvements:

Some images appear very big, could they be resized?
Could you add the Read More section to the newly moved docs?
Perhaps the External database connector documentation could use a bit more explanation how three documents relate to each other? We have /refguide/external-database-connection/, /refguide/use-the-external-database-connector/, /appstore/modules/external-database-connector/.

@MariaShaposhnikova Which Read More section are you referring to?


Add further autoload file to fix examples

Without this addition the examples failed with "Uncaught Error: Class 'Amp\Parallel\Example\BlockingTask' not found" because this class are not loaded.

Coverage remained the same at 65.57% when pulling 6f16d0c8c42f586f5cbac81486022769edd49117 on tommy-muehle:master into 6b2f5cd3fa086dc9c3c39eabe488b0dbfa50f7f6 on amphp:master.

This has no business in the generic autoload file. (It would cause to load BlockingTask class for all code using this library too.) It may make sense to add a new PHP file requiring both the actual autoloader and the BlockingTask class inside the examples folder.

Ah, ok. I will change it.
Bob Weinand<EMAIL_ADDRESS>schrieb am Mo. 26. Juni 2017 um
16:08:

This has no business in the generic autoload file. (It would cause to load
BlockingTask class for all code using this library too.) It may make sense
to add a new PHP file requiring both the actual autoloader and the
BlockingTask class inside the examples folder.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/amphp/parallel/pull/17#issuecomment-311069941, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABSgoPdigkJbCXIbQT_JudDUU8fNlIkWks5sH7tfgaJpZM4OFQW_
.


Upgrade terraform version to 1.0.0

Terraform is version 1.0.0! Upgrade qhub to depend on this version

This should be a good beginner issue. I believe it would only involve changing: https://github.com/Quansight/qhub/blob/main/qhub/constants.py. Let's leave this one for someone new on the team.

Success Criteria

QHub now uses terraform 1.0 for all QHub runs.

@danlester @tylerpotts @costrouc  As discussed i have performed below test and analysis. Please correct me if i have gone wrong somewhere.
Test1:
I did the deployment using the terraform v0.14.9.
Did redeployment using  terraform v1.0.0 to check if the existing qhub deployment can be updated. test was passed and redeployment went fine.
Test2:
Did qhub deployment using terraform v0.14.9.
and qhub destroy using terraform v1.0.0
test went fine.
Test3:
did normal deployment using terraform v1.0.0 and deployment went fine.
I do not see any issue with upgrading the terraform version to v1.0.0 .
Above test run were conducted locally, AWS, and GCP.

I have even tried the Latest terraform version v1.0.1 for local, AWS and GCP deployments and everything went fine.
If agreed we can upgrade the terraform version to v1.0.1 instead of v1.0.0

PS: I did even try to use TERRAFORM_VERSION >= "1.0.0"  to accommodate future version upgrade. But the "qhub/provider/terraform.py" code doesnot support ">" or in other words, it expects "="

Thank you.

@tarundmsharma That's fantastic, thanks for running the tests.
I had thought @utabe said there was a problem upgrading like that, but maybe he just meant he hadn't been able to try it.
Agree 1.0.1 is fine, or if easier we can just accept Abraham's existing PR.

I have merged the PR upgrading to 1.0.0 anyway for now.


Reimplement markdown to asciidoc conversion

We currently use a fork of a github repository which relies upon pegdown, which no more seems to be maintained.
So reimplement that in a more future-proof way.

This is required for #90


Incorrect parsing of MAKE environment variable

I have set
MAKE='make -j16'

on my development box, this is the recommended way to enable makefile parallelism by default. Now node-gyp dies with
gyp ERR! build error
gyp ERR! stack Error: not found: make -j16
gyp ERR! stack     at F (/home/vbraun/.hashdist/bld/node/64ucaro72c57/lib/node_modules/npm/node_modules/which/which.js:40:28)
gyp ERR! stack     at E (/home/vbraun/.hashdist/bld/node/64ucaro72c57/lib/node_modules/npm/node_modules/which/which.js:43:29)
gyp ERR! stack     at /home/vbraun/.hashdist/bld/node/64ucaro72c57/lib/node_modules/npm/node_modules/which/which.js:54:16
gyp ERR! stack     at Object.oncomplete (fs.js:108:15)
gyp ERR! System Linux 4.1.7-200.fc22.x86_64
gyp ERR! command "node" "/home/vbraun/.hashdist/bld/node/64ucaro72c57/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /home/vbraun/Talque/tools/talque/lib/node_modules/protractor/node_modules/selenium-webdriver/node_modules/ws/node_modules/bufferutil
gyp ERR! node -v v0.10.36
gyp ERR! node-gyp -v v1.0.3
gyp ERR! not ok

It is not valid to assume that MAKE is set to the name of an executable alone. I also don't see why node-gyp has to run which, it should just respect what I set for MAKE.

Any particular reason you don't set MAKEFLAGS="-j16"?  MAKE="make -j16" will break more than just gyp / node-gyp.

What @bnoordhuis said. You're looking for MAKEFLAGS=, not MAKE=.

Setting MAKE is the recommended way; MAKEFLAGS is implementation-dependent and meant for the communication between make instances. Autotools works just fine with MAKE='make -jN'.

http://stackoverflow.com/questions/19611004/recursive-make-correct-way-to-insert-makeflags/19622824#19622824
https://www.gnu.org/software/autoconf/manual/autoconf-2.68/html_node/The-Make-Macro-MAKEFLAGS.html#The-Make-Macro-MAKEFLAGS

Please read the pages you linked to carefully: it's about reading or changing MAKEFLAGS inside a Makefile, which indeed you should not normally do.  It's perfectly fine for passing arguments to make from outside.

The problem is that MAKEFLAGS is implementation depedent, it has nothing to do with when you set it. Yet posix requires make to look at it and pass it down to sub-makes.
Say you have gnu make and foo make installed. You can't export MAKEFLAGS=--foo in your .bashrc, this will break gnu make. Just like using a gnu extensions like -jN would break non-gnu make.
Sure you can in addition set MAKE to the make that accepts the given MAKEFLAGS but thats not exactly convenient. Its much better to just set MAKE to begin with.

Both GNU make and BSD make support -j just fine but okay, if you're worried about breaking pmake, use GNUMAKEFLAGS.  GYP doesn't support non-GNU make anyway.
In case it wasn't clear, the fact that you can't put it in MAKE is a GYP limitation.  You'll have to live with it unless you want to put in the effort of getting a patch accepted upstream.
I'll close the issue because there isn't much else to discuss but at least there's a bit of background and solutions now for people coming here from Google.


Modernize ELT, kill bower and enable yarn

Adressing #430.
:skull_and_crossbones:  bower.json

[x] 533f6df: Update ember-cli to 2.13, reconfigure to use npm & ember-source instead of bower
[x] f964cb0: Faker / Pretender shouldn't be needed in bower.json as those are brought in by ember-cli-mirage https://github.com/offirgolan/ember-light-table/blob/master/bower.json
[x] 4761317: font-awesome should be a devDependency when we move it to the package.json
[x] a9154f2: Kill bower.json

Also

[x] 533f6df: Enable yarn
[x] 84abf5b: Remove jquery@2 from bower.json
[x] 275b2a9: Upgrade demo to use<EMAIL_ADDRESS>[x] 533f6df7: Switch from jshint to eslint
[x] 73490a9: document font-awesome usage, so it's no surprise that the ascending and descending icons don't work when users drag and drop our code snippets, if they don't have font-awesome, and haven't copied our setup.
[x] fd32329c: replace ember-suave with the spiritual successor eslint-plugin-ember-suave

[x] 3d47c1c: configure sensibly and fix new linter errors

[x] 47d98d9: circumvent https://github.com/babel/babel/issues/5862
[x] 6d38976: Update all other dependencies

Likely out of scope

Properly set up targets or includePolyfill, so that #427 gets fixed. Was already broken prior to this PR.

Thanks for reviewing! :smiley:


Add option to disable stunt jumps while recording a run

Fairly self-explanatory
Toggle should disable stunt jumps (camera change, slowdown, sound effects) - or maybe even just set this behavior as the standard without a toggle
Of course there is a way to get around stunt jumps in singleplayer (by aiming) but it'd be cool to have this happen automatically

Doesn't seem like there's an easy fix, and I'm not gonna let it monitor and cancel the stunt jumps, or make it disable any stunt jumps.
Use a separate mod for that, if any even exists.


in realsense viewer

Before opening a new issue, we wanted to provide you with some useful suggestions (Click "Preview" above for a better view):

Consider checking out SDK examples.
Have you looked in our documentations?
Is you question a frequently asked one?
Try searching our GitHub Issues (open and closed) for a similar issue.

All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)

Required Info

Camera Model
{  D400 }

Firmware Version
(Open RealSense Viewer --> Click info)

Operating System & Version
{Linux (Ubuntu 14/16/17)  }

Kernel Version (Linux Only)
(e.g. 4.14.13)

Platform
PC

SDK Version
{ legacy / 2.. }

Language
{python }

Segment
{Robot/Smartphone/VR/AR/others }

Issue Description
The displayed device error is particularly large, the actual distance is 2.4m, and the measured distance is 3m. The closer the error is, the closer the error is 0.
<Describe your issue / question / feature request / etc..>

May you need to recalibrate the camera? I never needed to be done before, but your experienced error is indeed large. Check #2329 problem.

 @g2-bernotas  Does the depth map calibration separately calibrate the left and right grayscale images?

@Can these calibration APIs calibrate the d435i, why do I show no device connection?

Please double check version of the tool. D435i support was added in version 2.6.8.
https://downloadcenter.intel.com/download/28517/Intel-RealSense-D400-Series-Calibration-Tools-and-API


How can i change the BirdsCount variable value ?

In the Main.cs script there is a line:
public float BirdsCount = 10;
I tried to change it to 10 to 100 to 1000 to 1 but it didn't change the number of birds when running the game.
I want to be able to change the number of birds in two cases one when the game is running and then in the Inspector on the Main.cs script attched to the Main Camera if i type change the value to 1 there will be 1 bird if i change to 60 there will be 60 birds.
The second case is when the game not running when i'm in the editor and change the value in the Inspector and then running the game.
What i did so far and i guess i'm wrong is added a public variable to the top of the Main.cs:
public int _birdscount;
Then in the Update function i added this:
`bool isepressed = false;
void Update()
{
if (Input.GetKeyDown ("space"))
OnSettingsClick ();
	if (Input.GetKeyDown("e"))
	{
		isepressed = true;
		BoidSettingsEx eee;
		eee = settings.boidSettings[settings.instancePointNum];
		eee.BirdsCount = _birdscount;
		SaveSettings `();
	}

//We call it here to be sure that click on button desn't lead to camera target changing
if( !guiTools.MouseOverGUI && Input.GetMouseButtonDown(0) )
  cameraControl.CheckForNewTarget( Input.mousePosition );

guiTools.ManualUpdate();

}`
And in the SaveSettings i did:
void SaveSettings() 	{ 		if (Application.isEditor) { 			using (var str = new FileStream (SettingsFilePath, FileMode.Create)) 				serializer.Serialize (str, settings); 		} 		if (isepressed == true) { 			using (var str = new FileStream (SettingsFilePath, FileMode.Create)) 				serializer.Serialize (str, settings); 			isepressed = false; 		} 	}
So when i click the e button it should change the value and update the Settings.xml file on the hard disk.
So it does changing the value in the Settings.xml file in the first part:
6
But in the other two places in the file where there is the line:
10 The value not change.
Thank you for the help

Sorry for delayed response.
You can change the Number of Birds from Settings. You don't need to modify the code:

Don't forget to press "Restart" after that

Great Thank you.


Feature request: an op that returns timestamp

This is useful for stats tracking in distributed TensorFlow (ie, measuring TF communication latency between workers and plotting it in TensorBoard). Perhaps it could be called CurrentTimestamp, tf.current_timestamp()
#include "tensorflow/core/platform/env.h"
static EnvTime* env_time = tensorflow::EnvTime::Default();
uint64 now_micros = env_time->NowMicros();

If you would like to plot time stamp on tensorboard, you can use include it in tf.summary
For example,
summary_writer = tf.summary.FileWriter()
summary = tf.Summary()
summary.value.add(tag='xxx', simple_vale=yyy)
summary_writer.add_summary(summary, global_step)

How would you plot communication latency between workers using tf.summary?
You need something like CurrentTimestamp op to execute on one worker, and then compare it against timestamp on another worker to get the latency


Improve and update README

Markdown
Basically the README.md file is improved to markdown standards.
Issues
Resolves #103
Fixes #104
Note
I used Git! for the GitHub link just like the other app below.
I made the headers uppercase so it matches the first column (APP), they should be the same style.

@martincarrera this is taking long, pleasereview and merge.

Hello @AmirSavand, please note that Martin is not full time working on this open source API. Feel free to fork this repository and use it on your side.

Thanks @martincarrera


T CMB from CLASS

https://github.com/LSSTDESC/CCL/blob/6a288ec8748f19b121e26391e6bafcbceefee9a9/src/ccl_core.c#L209
Should we use 2.725 or 2.7255? Noticed this now

I think that's a C-level remnant that doesn't really do anything:
>>> import pyccl as ccl
>>> cosmo = ccl.CosmologyVanillaLCDM()
>>> cosmo['T_CMB']
2.7255


Hide docker desktop menu icon option for MacOS

Tell us about your request
Hide docker desktop menu icon option in preferences for MacOS.
Which service(s) is this request for?
Docker desktop for MacOS
Tell us about the problem you're trying to solve. What are you trying to do, and why is it hard?
Docker desktop  icon over-populating the menu bar whilst there are versions of macbooks which have a notch bar.
On the other another person using one's laptop may accidentally and/or due to unawareness stop the docker engine which is not good.
Are you currently working around the issue?
There's almost no way to maintain this right now.

Facing the same problem. Bumping this up. It's frustrating to not be able to remove the icon.

This is a dupe of #281.


SKLearn error

[Enter feedback here]
At one point the progress halts while running through the notebook wind-turbine-scikit.ipynb.
Did anyone rewrite this code to supported Python code?
This code in a demo does not run well:
WARNING:azureml.train.sklearn:'SKLearn' estimator is deprecated
from azureml.train.sklearn import SKLearn
estimator = SKLearn(source_directory='./Scripts',
compute_target=compute_target,
entry_script='train.py',
inputs=[tabular.as_named_input('training')],
pip_packages=['azureml-dataprep[fuse,pandas]','joblib==0.14.1','azureml-interpret','azureml-contrib-interpret','matplotlib','scikit-learn==0.22.1','seaborn'])
run = experiment.submit(estimator)
run

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

ID: c8d834cf-b3ff-801d-1804-0d22d6c63089
Version Independent ID: df52a587-695a-19a7-70d1-91a431e329b0
Content: Deploying Azure SQL Edge on turbines in a Contoso wind farm
Content Source: articles/azure-sql-edge/tutorial-renewable-energy-demo.md
Service: sql-edge
GitHub Login: @kendalvandyke
Microsoft Alias: kendalv

Hi, @bragemo-coder Thanks for reaching out, We will Investigate and update you shortly.

I'm assigning this to the author for the feedback review.

#assign:rwestMSFT

@bragemo-coder I've taken over this area of the documentation, and was curious if you wouldn't mind revisiting the issue to let me know if it is still a problem. Thanks!

Since I haven't heard from you for a while, I'm marking this as closed. Please feel free to leave a comment if your circumstances change.

#please-close


bug(popover): overflowing window borders

Popovers are overflowing over window borders.
http://plnkr.co/edit/q2Zj1F
Solution might be to add "Auto" positioning option.

:+1:

+1

+1

Still need this! Were I JS ninja sufficient I would whip it up. Alas :(

A workaround is to add another directive to the element and bind a mouseenter event or whatever is being used to fire the tooltip and adjust the position:
angular.module('app')
    .directive('positionTooltip', ['$timeout', function ($timeout) {
        return {
            restrict: 'A',
            scope: false,
            link: function (scope, element, attrs) {
                var el,
                    arrow,
                    offset;
                element.bind('mouseenter', function () {
                    $timeout(function () {
                        el = element.next();
                        if (el[0].offsetLeft < 0) {
                            offset = el[0].offsetLeft;
                            el.css('left', 0);
                            arrow = el[0].querySelector('.tooltip-arrow');
                            arrow = angular.element(arrow);
                            arrow.css('margin-left', (offset + -5) + 'px');
                        }
                    })
                });
            }
        }
    }]);

Does https://github.com/twbs/bootstrap/pull/12328 end up in here somehow?

+1

+1

I have a couple of ideas I've been working on for this issue I'd like to share.  I need to pull them together in a plunk, then will post them shortly.

@thegreatmichael thanks for mentioning that issue, we can use that as a reference implementation.

+1

+1

It's already clear that this is a wanted feature.
Please stop all the +1 responses
We'd like to keep this open for on-topic discussion instead of having to receive notifications for all +1 responses.

Just thought I'd post my work-around if it's of use to anyone.  This enables a position to be set just before the popover becomes visible.  I had to tweak the CSS to initially hide the popover until positioning was complete as there were times when the popover would show and then jump into the correct position.

is this solved?

I was having this issue with the uib-popover-template directive. It took me a while to figure out how to hack this dang thing (I'm new to coding so it may be simple and I'm an idiot [2 $timeouts were required])
This hack is for uibPopoverTemplatePopup but you could just find out what directive ui-angular is using for the tooltip or w/e by looking at the code.
if you don't use popover-append-to-body="true" the popover will be inline (the next element after your button). If you allow the popover to insert directly after the button the offset left may not work (because it offsets to the nearest parent with position relative (I should probably check that).
Lastly, I only tested the code if you use popover-placement="top". It will probably work if you use popover-placement="bottom" but I didn't test it. It won't work for left or right (because I'm using margin-left)
<!-- In the Template -->
<a href popover-placement="top" uib-popover-template='"popovers/ProgramProgressionHelp.html"'  popover-append-to-body="true">Button</a>

// Directive code
app.directive('uibPopoverTemplatePopup', ['$timeout', function($timeout) {
  return {
    restrict: 'A',
    link: function (scope, element, attrs) {
      $timeout(function() {
        $timeout(function() {
          var popover = element;
          var offset = popover[0].offsetLeft;

          if (offset < 0) {
            var arrow = popover[0].querySelector('.arrow');
            angular.element(popover).css('margin-left', (offset*-1) + 'px');
            arrow_margin_left = angular.element(arrow).css('margin-left');
            angular.element(arrow).css('margin-left', (offset + parseInt(arrow_margin_left)) + 'px');
          }
        });
      });
    }
  }
}]);

Sorry I can't do a better fix. I looked at the source and your using fancy providers, I don't know how to use those things :)

@gruberjl, we have smart/fixed positioning on our radar.

Closing in favor of #4762.


update Manifests

The manifests should have been re-resolved after #677, since that requires DataInterpolations 4.4, but the Manifests still used 4.0.
This just updates all packages in them also switching to a newer PackageCompiler version for instance.
This should fix TeamCity builds.

This also checks in the core/Manifest.toml, to avoid breakage like this in the future. With that we essentially lock all our depencencies, in pixi.lock and the several Manifest.toml files.


Fails to connect plex to my account

I have a new server at OVH, with Debian/jessie installed on it.  First thing I do is install docker.io then timhaak/docker-plex.  I start plex successfully with the start command below.  I use putty to connect an SSH tunnel to my server.  Then I browse to the plex installation at http://localhost:8888/web/
I accept the EULA(?) and go to Settings to try to connect the server to my plex account.  I successfully sign in with my credentials, but it fails to connect the server to my account.  The error message is below.
========= command used to start the plex docker install
sudo docker run -d
-h <IP_ADDRESS>
-v /data-store:/config
-v /:/data-store/downloads/media:/data
-p 32400:32400
timhaak/plex
========= Error message in Plex
Connecting your server to Plex allows you to stream media outside your home network.
Signed in as<EMAIL_ADDRESS>SIGN OUT
Signed in to Plex
You have successfully signed your server in to Plex, but we were unable to reach it from outside your network.
If you would like to enable this now, you will need to configure your router or firewall to allow Plex to contact your server. Please refer to our Connecting a Server article for more information.

Did you try ticking "Manually map my port" option? I find this usually fixes this issue for me.

Well that fixed it right up.  Thank you!

If someone else has a similar problem they should also confirm that port 32400  is open on there firewall and mapped to the server.

@timhaak For some reason, even though the port is always open (i.e I can access 32400 through the IP to get to the webui) until i ticket "Manually map the port" and put in the exact same port (32400) it won't work! I've had this every single time I've installed Plex, on at least 6 different networks and different servers, where ports are always open. So I've always figured there's a bug or something going on there.

I agree with @bhavicp on this.  I've installed Plex manually several times before on Debian without this happening.  This is my first time using Docker though.

It could be something with the fact that the IP that the docker container has and that plex see's is different to the external IP of the server which is mapped to the docker plex container.


Fail to deploy the Java project to a basic Java Function App with an error

OS: Win10
Build Version: 20220407.4
VS Code Version-insiders: 1.67.0-insider (user setup)
Function Core Tools: v4
Repro Steps:

Create a Java project.
Create a basic Java Function App.
Deploy the Java project to the Java Function App.
Check whether succeeds to deploy the Java project to a basic Java Function App.

Expect:
Succeed to deploy the Java project to a basic Java Function App.
Actual:
Fail to deploy the Java project to a basic Java Function App with an error.

Output logs:

More Info:

This issue also reproduces for a Linux Java Function App.
This issue doesn't reproduce for other languages.
[Unstable] [Codespace] Fail to deploy the Java project to a basic Java Function App with a different error.

Same issue here.

@MateoSaez sorry you ran into this bug, the good news is we are working on a fix for this right now, and we have a development build available if you'd like to try and verify the fix works for us.

Steps to install extension development build:

If needed, update VS Code to 1.66.0 or greater.
Download the extension .VSIX from our Azure DevOps Pipeline - Direct download link | View artifacts
From the command palette (F1 or Command/Ctrl + Shift + P), run the "Extensions: Install from VSIX..." command.
Select the downloaded .VSIX file, then reload VS Code as prompted.
Navigate to the extensions view and verify the version is 1.6.2-alpha

Once installed, attempt to deploy your app as normal. Please let us know if deployment succeeded/failed.

Follow along with #2844 for more updates and discussion, we're directing all of these zip deploy issues over there.

Closing as a duplicate of #2844

Fix has been released in version v1.6.2 of the extension.


first round of updates from DM-20238

I think we should incorporate this first round of updates into the 'official' version of DMTN-102, even though ticket DM-20238 (and this branch) will remain open for further updates.
Requesting @ebellm review because he reported DM-20238.

Other comments:

should the document have a glossary?
should we say anything about whether the "alert archive" is available for bulk download (via the same mechanism as the rest of the bulk download capability)?

I'm also attempting to add a glossary, but am not sure exactly what to say about bulk downloads from the alert archive -- they'd probably have some cap like other bulk downloads?

I've asked @timj to review this PR because I need help making the glossary work.
I followed the example set by DMTN-109 on how to incorporate a glossary and list of acronyms, and used generateAcronyms.py to create aglossary.tex and acroynms.tex, but I can't figure out why the PDF doesn't show them. Plus, based on the contents of aglossary.tex it seems to maybe be a bit too many terms for such a short little DMTN, but I can fix that after.

@gcomoretto and @timj helped me get the glossary working (problems in the Makefile), and i updated the change record version description. i still didn't add anything specific about how people might download the alert archive, figured that could wait?
about to merge.


class-generator: fix when kind do not have properties

Do not fail when the resource does not have properties in the schema.
Do not overwrite existing files when the schema is generated with older cluster versions.

Summary by CodeRabbit

New Features

Enhanced version management for schema generation, including tracking of the last generated cluster version.
Expanded list of supported Kubernetes and OpenShift resource types, including new entries for various components and services.
Introduced new JSON schema definitions for multiple resources within the Tekton and OpenShift Pipelines frameworks.
New configuration file .coderabbit.yaml to manage project settings, review processes, and tool integrations.

Bug Fixes

Improved handling of schema properties to prevent errors when expected structures are missing.

Chores

Updated .gitignore to include new entries for ignored files and directories, streamlining version control.
Added a new version identifier for the cluster component.

Report bugs in Issues
The following are automatically added:

Add reviewers from OWNER file (in the root of the repository) under reviewers section.
Set PR size label.
New issue is created for the PR. (Closed when PR is merged/closed)
Run pre-commit if .pre-commit-config.yaml exists in the repo.

Available user actions:

To mark PR as WIP comment /wip to the PR, To remove it from the PR comment /wip cancel to the PR.
To block merging of PR comment /hold, To un-block merging of PR comment /hold cancel.
To mark PR as verified comment /verified to the PR, to un-verify comment /verified cancel to the PR.
verified label removed on each new commit push.
To cherry pick a merged PR comment /cherry-pick <target branch to cherry-pick to> in the PR.

Multiple target branches can be cherry-picked, separated by spaces. (/cherry-pick branch1 branch2)
Cherry-pick will be started when PR is merged

To build and push container image command /build-and-push-container in the PR (tag will be the PR number).

You can add extra args to the Podman build command

Example: /build-and-push-container --build-arg OPENSHIFT_PYTHON_WRAPPER_COMMIT=<commit_hash>

To add a label by comment use /<label name>, to remove, use /<label name> cancel
To assign reviewers based on OWNERS file use /assign-reviewers
To check if PR can be merged use /check-can-merge

Supported /retest check runs

/retest tox: Retest tox
/retest python-module-install: Retest python-module-install
/retest all: Retest all

Supported labels

hold
verified
wip
lgtm

/verified

/verified cancel

/verified

@coderabbitai configuration

/verified

/verified

/verified

/verified

/verified

/verified

/verified

/verified

/verified


Jkirkwood/model loading poc1

Add some testing for loading models during a request event and serve that model.

Also, I'd like to suggest for us to setup the following branches in this repo:

upstream - tracks the https://github.com/tensorflow/serving verbatim. We should be able to get changes from TensorFlow without worrying about conflicts etc.
master - integrates between our fork and upstream
zd-load-predict - the feature branch where the load + predict endpoint will be created.

For this PR, we can still merge into poc1, but once merged, we can create zd-load-predict off the poc1's HEAD.
The next PR you have https://github.com/zendesk/serving/pull/2 should be merged against zd-load-predict.

There's a number of feedback I've given - let's discuss and see what you think.

Closing this in lieu #2


Table styles leaking / overriding global styles

The table theme is creating a global style tag which is interfering with table styles elsewhere

The solution should be to wrap the table with the current theme rather than calling createGlobalStyle


[ Request ] Update img to 0.5.6

With current version (0.5.1), login to private registry fail with dns queries Oo upgrading to 0.5.6 fixes this issue.

I've pushed a new image as concourse/builder-task (not concourse/builder!) with img 0.5.6. :+1:

Cool! Thanks!

@vito Actually sorry to bother again... Are you building img yourself ? I can successfully login with the release bin but not with your image.
concourse/builder-task:

release bin:

I'm inheriting from their image, not building it myself

Oh well Oo I'll just install it over in the image then xD Thanks anyway!

fwiw this should be up-to-date now:
~/workspace/builder-task master*
❯ docker run concourse/builder-task version
img:
 version     : v0.5.6
 git hash    : 013bb211
 go version  : go1.11.7
 go compiler : gc
 platform    : linux/amd64


Style to Attributes beaks files (with keeping style-element)

Input

Output

Version:
https://jakearchibald.github.io/svgomg/
Copyright
source: https://commons.wikimedia.org/wiki/File:Equator_and_Prime_Meridian.svg
Author: https://commons.wikimedia.org/wiki/User:NuclearVacuum
license: https://creativecommons.org/licenses/by-sa/3.0

Closing as duplicate of #1365


Deployments will log errors but not abort when deleting routes (issue #3258)

Currently the deleteDeployment API will return with an error if there was a problem deleting service or routes in a deployment config.  This PR changes that to simply log the error and continue.
After this PR is merged, deleting a deployment can return successfully (and delete the deployment config) even if there was an issue deleting a route and/or a service within the deployment.
After this PR, all other errors (such as invalid space name or error deleting the deployment config itself) will still return errors.  More logging has been added, even in these cases.
resolves https://github.com/openshiftio/openshift.io/issues/3288

Ike Plugins (test-keeper)
Thank you @stooke for this contribution!

It appears that no tests have been added or updated in this PR.
Automated tests give us confidence in shipping reliable software. Please add some as part of this change.
If you are an admin or the reviewer of this PR and you are sure that no test is needed then you can use the command /ok-without-tests as a comment to make the status green.
Your plugin configuration is stored in the file.

Codecov Report

Merging #2098 into master will decrease coverage by 0.09%.
The diff coverage is 33.33%.

@@           Coverage Diff            @@
##           master   #2098     +/-   ##
========================================
- Coverage   67.49%   67.4%   -0.1%
========================================
  Files         158     158
  Lines       14683   14702     +19
========================================
- Hits         9911    9910      -1
- Misses       3816    3836     +20
  Partials      956     956

Impacted Files
Coverage Δ

kubernetes/deployments_kubeclient.go
65.77% <33.33%> (-0.49%)
:arrow_down:

remoteworkitem/jira.go
75% <0%> (-25%)
:arrow_down:

remoteworkitem/scheduler.go
53.65% <0%> (-7.32%)
:arrow_down:

Continue to review full report at Codecov.

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

@stooke Your image is available in the registry. Run docker pull registry.devshift.net/fabric8-services/fabric8-wit:SNAPSHOT-PR-2098 to get it.

I am going to disable the test check for now, and add them later.
/ok-without-tests


RenderTexture ping-pong

When accumulate any graphics over time while swapping double buffer of RenderTexture, the output image becomes yellow incrementally. This issue occurs when using WebGLRenderer.
http://codepen.io/ichitaro/pen/gifkw

Fixed in v3. We now upload a 32uint as the color so no more color degradation. Noice! :+1:


Using OpenRewrite to fix issues detected by scores

It would be great is issues detected by the project could be fixed with a click.
We could use https://docs.openrewrite.org/recipes/jenkins and probably contribute new recipes to that project.
We could display a link on plugins.jenkins.io/whatever/healthscore on a detailed line to trigger the execution of the mvn rewrite:run on a container or similar.

Being able to run command on the clone of a plugin is not a simple issue.
The idea to start a k8s pod in order to be able to run a command on a Git clone is not a simple thing to do.
With #452, we can create guide on jenkins.io (with redirect if needed) to provide the steps to solve a problem.
I'm closing this because of all that.


Update storage folder permissions

The current storage folder setup doe snot work on Ubuntu as it does not have the nobody group. This should be update to support all distro (https://github.com/sassoftware/viya4-deployment/blob/main/playbooks/playbook.yaml#L39)

https://github.com/sassoftware/viya4-deployment/pull/62


Fixes - Child theme from site of network install not exported ?

When the plugin executes the exporting process, is_child_theme() returns false even the site is using a child theme.
So, replacing that function with get_template_directory() !== get_stylesheet_directory() solve the issue.

@sultann Thanks for the PR!


Use Training Strategy Pattern in Trainer

The current implementation of bayesflow.trainers.Trainer implements several training strategies in one large class. This can be problematic for both maintainers and users, since

Implementing a new strategy from outside bayesflow requires inheriting from the Trainer class
Modifying small parts of a strategy outside of its input parameters can be difficult or impossible
Maintaining a mental overview over the Trainer functionality is impeded.

We can remedy these issues by implementing the trainer strategies explicitly as their own concept, and following composition over inheritance in the Trainer class.

That's a very important one! Especially since we want to add some new strategies in the process.

Superceded by the removal of the trainer object


Installations can delete /tmp/sensu_server_loaded_files, causing handlers to silently fail

Handlers require /tmp/sensu_server_loaded_files in order to operate, but many installations prune files from /tmp.  The Filesystem Hierarchy Standard states: "Programs must not assume that any files or directories in /tmp are preserved between invocations of the program", and also "data stored in /tmp may be deleted in a site-specific manner".
The default setting of tmpwatch (as distributed for Ubuntu Trusty) is to delete files when they are thirty days (measured via mtime).  Yelp configures this threshold lower, to ten days. When the file is eventually removed, all Sensu handlers begin to fail. The only way to detect this failure is to monitor the Sensu server error log, using some monitoring framework other than Sensu.  The error message appears as "No such file or directory - /tmp/sensu_server_loaded_files (Errno::ENOENT)"
I can think of three ways to remedy this issue:

recreate (or simply touch) the file each time a handler is spawned
pass the same data via a pipe.  The file descriptor number for the spawned handler to read from would be passed as /dev/fd/ in an environment variable or command-line argument.
require the system administrator to configure a directory for Sensu to write temporary state into, which will not be automatically cleaned. (an extra step; rather annoying)
a variant of 3, continue to use TMPDIR for temporary state, and document that Sensu requires a TMPDIR which is automatically cleaned (easy to overlook).

4 - I think you mean not automatically cleaned?

Yes. I've edited the description, thanks.

Just merged https://github.com/sensu/sensu-settings/pull/20 - Sensu 0.22 will include this environment variable, configurable through /etc/default/sensu.

#20 doesn't seem to be taking affect,
cat /etc/default/sensu
EMBEDDED_RUBY=true
SENSU_LOADED_TEMPFILE_DIR=/var/run/sensu

Yet the files still get created under /tmp and not under /var/run/sensu

@Gillingham try export TMPDIR=/var/run/sensu in /etc/default/sensu.

@portertech that seems to have resolved it, TMPDIR should probably be in the sensu docs somewhere, doing a google for sensu TMPDIR doesn't result in any really relevant hits outside of #20

FWIW, just
TMPDIR=/var/run/sensu in /etc/default/sensu does the trick
thanks!


Upgrade NLTK version to circumvent unsafe pickling in v3.8.1

Description
#1096 pinned nltk==3.8.1 for the JAX ci since nltk>=3.8.2 introduced a breaking change. However, this is because the punkt package consists of unsafe pickles, so it is deprecated and not used in NLTK 3.8.2 to circumvent the underlying security vulnerability. Further reading:

https://github.com/nltk/nltk/issues/3293#issuecomment-2283807174
https://github.com/nltk/nltk/issues/3266#issuecomment-2284001819
https://www.vicarius.io/vsociety/posts/rce-in-python-nltk-cve-2024-39705-39706

Type of change

[ ] Documentation change (change only to the documentation, either a fix or a new content)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Infra/Build change
[ ] Code refractor

Changes

Pin minimum version of nltk to 3.8.2 for jax tests
Switch from punkt to punkt_tab package for nltk.

Checklist:

[x] I have read and followed the contributing guidelines
[x] The functionality is complete
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

/te-ci jax


How to connect to Redhshift Cluster which is using SSH-Tunnel.

I am not able to connect to the Redshift cluster which is using the SSH-Tunnel, I  have the Integration Runtime gateway installed on the virtual machine and I can connect to the Redshift cluster from the virtual machine using Datagrip tool, but cant connect the ADF V2 to the Redshift

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

ID: 72915b57-8b87-a071-6f77-9690d3380c31
Version Independent ID: b7cf7a74-225f-765a-b912-ea4c517e6bcb
Content: Copy data from Amazon Redshift using Azure Data Factory
Content Source: articles/data-factory/connector-amazon-redshift.md
Service: data-factory
GitHub Login: @linda33wj
Microsoft Alias: jingwang

Hi alberto I have raised a support ticket to Microsoft, can I share the support request ticket no with you so that you can assign it to correct persons?
Get Outlook for Androidhttps://aka.ms/ghei36

From: Alberto Vega<EMAIL_ADDRESS>Sent: Tuesday, October 16, 2018 10:55:47 PM
To: MicrosoftDocs/azure-docs
Cc: Anmol Ganju; Mention
Subject: Re: [MicrosoftDocs/azure-docs] How to connect to Redhshift Cluster which is using SSH-Tunnel. (#16954)
Hi @Anmolgan81https://github.com/Anmolgan81 This will require some deeper analysis. Can you please file a support request @ https://aka.ms/azsupt? If you do not have access to a support plan, please reach out @ AZCommunity<EMAIL_ADDRESS>with a link to this Doc/Issue as well as your subscription ID and we can help get the support ticket opened for this issue.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/16954#issuecomment-430324573, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AhHKpwdXHpoyz5a6RnjEcYBQZsI5n9cHks5ulhabgaJpZM4XefRB.
This message contains confidential information and is intended only for the individual named. If you are not the named addressee you should not disseminate, distribute or copy this e-mail. Please notify the sender immediately by e-mail if you have received this e-mail by mistake and delete this e-mail from your system. E-mail transmission cannot be guaranteed to be secure or error-free as information could be intercepted, corrupted, lost, destroyed, arrive late or incomplete, or contain viruses. The sender therefore does not accept liability for any errors or omissions in the contents of this message, which arise as a result of e-mail transmission. If verification is required please request a hard-copy version. IFI Techsolutions Pvt. Ltd. : www.ifi.tech

@Anmolgan81 support have their own way to channel tickets to the right engineers. Were you able to resolve this issue?

So ADF v2 does not support ssh tunneling, the pg team just confirmed that.
Get Outlook for Androidhttps://aka.ms/ghei36

From: Alberto Vega<EMAIL_ADDRESS>Sent: Tuesday, October 23, 2018 5:05:10 AM
To: MicrosoftDocs/azure-docs
Cc: Anmol Ganju; Mention
Subject: Re: [MicrosoftDocs/azure-docs] How to connect to Redhshift Cluster which is using SSH-Tunnel. (#16954)
@Anmolgan81https://github.com/Anmolgan81 support have their own way to channel tickets to the right engineers. Were you able to resolve this issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/16954#issuecomment-432028574, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AhHKp9pfDQmfNL4P6Uo68JN8fqzJwQtiks5unlYugaJpZM4XefRB.
This message contains confidential information and is intended only for the individual named. If you are not the named addressee you should not disseminate, distribute or copy this e-mail. Please notify the sender immediately by e-mail if you have received this e-mail by mistake and delete this e-mail from your system. E-mail transmission cannot be guaranteed to be secure or error-free as information could be intercepted, corrupted, lost, destroyed, arrive late or incomplete, or contain viruses. The sender therefore does not accept liability for any errors or omissions in the contents of this message, which arise as a result of e-mail transmission. If verification is required please request a hard-copy version. IFI Techsolutions Pvt. Ltd. : www.ifi.tech

We will now proceed to close this thread.


Update content/es/docs/concepts/workloads/pods/podspreset.md

Spanish translation of content/es/docs/concepts/workloads/pods/podspreset.md

/check-cla

I signed it

/hold

/check-cla

/retest

Hello @leeyspaul, thank you very much for the PR!
/check-cla

Hey Rael! I tried the CLA command, and the bots being a bit unresponsive. Not sure if the command is broken and needs fixing or if there's something else going on in the background.

/hold cancel

Hi Rael, the content seemed to already have been translated, I just re-organized the content and changed some syntactical items. Please let me know if there is something I am missing!

The CLA issues are usually related to a difference between the email used for GitHub and the mail used for signing the CLA, are you sure that both are the same?

I signed it

I made sure both locally and on the website the email was the same. Either way, I went ahead and filed a ticket, we'll see what they say. Thanks Rael

I signed it

I signed it

🎉 🎉 🎉
The CLA looks good now!

/lgtm
/approve
Thanks for the PR @leeyspaul and welcome to kubernetes!


fix(deps): update icr.io/ext/sysdig/agent:13.0.0 docker digest to c8e0288

This PR contains the following updates:

Package
Update
Change

icr.io/ext/sysdig/agent
digest
a321dd7 -> c8e0288

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline


auto-update: snakemake-workflows.yaml: python=3.6 -> 3.8.0; packaging=19.1 -> 19.2; conda=4.7.11 -> 4.8.0

This is an automated pull-request... :robot:

Snakemake does not support python3.8


Inline elements should be nestable inline

IMHO b > i > "foo" should result in
foo

rather than

foo

In other words, the same way that the i can detect that its contents are inline, that should be recursively detectable by the b as well.

The two outputs you showed are equivalent, the hbml parser does not prettyify the html it outputs.
Aydenh.ca on the other hand has a primitive html prettifier that runs on the output.

Fair enough, but I do think it could be worth explicitly declaring the strategy for printing the output. In other words, IMHO there should be a canonical output, following clear and consistent rules.
For example, it could be a pure AST mode, with all tags fully expanded, without optimizations like the "foo" being surrounded by the  in the same line; or the strategy could be of a pragmatic format with  heuristics like always collapsing chains of nested elements with a single child all the way to the leaf, etc. Currently it seems to be whatever the implementation produces, which I don't think is ideal.


Make it possible to disable SSE

Description
Make it possible to disable SSE
See https://github.com/owncloud/ocis-charts/issues/491
Motivation and Context

How Has This Been Tested?
locally on minikube
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 change)
[ ] Technical debt
[ ] Tests only (no source changes)

Checklist:

<!-- or create documentation PR in https://github.com/owncloud/docs-ocis/tree/master/modules/ROOT/pages/deployment/container>
- [ ] Code changes
- [ ] Unit tests added
- [ ] Acceptance tests added
- [ ] Documentation generated (`make docs`) and committed
- [ ] Documentation ticket raised:
- [ ] Documentation PR created:

@d7oc I added conditional running of the SSE service. Please have a look.


[runtime/tokio] Benchmark Performance of TCP on WAN

According to some Solana operators, Celestia researchers, and a bit of my own experience, sending even a modest amount of data over a WAN via TCP can take much longer than the expected link latency (due to congestion control/head-of-line blocking/etc.):

Turbine currently utilizes UDP for block propagation, providing immense latency benefits. According to a validator operator, transmitting 6 MB + erasure coding worth of data from us-east-1 to eu-north-1 using UDP takes 100ms while TCP takes 900ms.

We should create a harness that allows us to test the performance of any runtime networking implementation and validate these claims. It would be interesting to see if the use of different frame sizes or enabling BBR affects this performance at all (as the objects being sent to TCP to transfer over the stream can be much smaller than the actual object)...

Its also interesting to point out that we multiplex streams over the connections provided by runtime in p2p but don't use this info to optimize anything on the connection layer. If we employed something like QUIC, we could avoid blocking on different channels (if it is actually an issue we run into).
QUIC isn't some magic bullet and under a number of conditions behaves poorly: https://dl.acm.org/doi/10.1145/3589334.3645323.

Here is a great video on BBR: https://www.youtube.com/watch?v=VIX45zMMZG8
(we may get 80/20 by just employing this)

This harness would also be useful when evaluating if we should employ http/2 streams for multiplexing rather than handling it in p2p. I suppose we could compare what we have, streams via QUIC, streams via http/2.


permision error 13

Issue Type: Feature Request
whenever i wnat to read write a file it shows permission denied
error 13
please help me with this
VS Code version: Code 1.67.2 (c3511e6c69bb39013c4a4b7b9566ec1ca73fc4d5, 2022-05-17T18:15:52.058Z)
OS version: Windows_NT x64 10.0.22000
Restricted Mode: No

/gifPlease

Thanks for reporting this issue! Unfortunately, it's hard for us to understand what issue you're seeing. Please help us out by providing a screen recording showing exactly what isn't working as expected. While we can work with most standard formats, .gif files are preferred as they are displayed inline on GitHub. You may find https://gifcap.dev helpful as a browser-based gif recording tool.
If the issue depends on keyboard input, you can help us by enabling screencast mode for the recording (Developer: Toggle Screencast Mode in the command palette).
Happy coding!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


feature/coveralls

[x] add coveralls support
[ ] add coverage badge - readme

Changes Unknown when pulling 6dff50d5dd6cf379ab0769a77176229ffc4f9aba on feature/coveralls into ** on develop**.

Changes Unknown when pulling 6bd6d6b8350eedd5f2d4e1bbe09218be4871cc9d on feature/coveralls into ** on develop**.

Changes Unknown when pulling d4200918d95c89ef7774ce3cd40c3d782598826d on feature/coveralls into ** on develop**.


Move default Vault Location to \ProgramData

Currently the default Vault is located in %appdata%\letsencrypt-win-simple which is a user specific location which can cause problems with scheduled tasks as a user profile is required to gain access to this folder.
Seems like it would be a better idea to use a central location like \ProgramData to hold the default vault so that renewals and command line operations can be run from any account.

I think the reason it was under the user's app data folder is because each registration to the ACME server requires an email address. With it under the user's folder each user could get notifications on their certificates expiry. If it was under a central location, then only 1 email address would get alerts for all certificate expiry notifications. There are probably reasons why people want both options so it could go either way.
Right now you can specify a different location for the certificates it generates, but that could be reworked to allow the entire config folder to be moved. I feel that this should be an option or everyone currently using the application might have to re-register and that will mess up renewals.

But you can create multiple registrations in a single vault I think so the folder should have no impact on the email address. You would need an option to specify the email address as part of the command line or config options though to trigger that.
Agree though - if the path is configurable this isn't a big issue but I don't see that option currently. It looks like the docs are out of date for command line options (--manualhost --webroot options aren't there for example).

You can only create 1 registration per Base URI that you use right now. There would have to be more changes to support having multiple registration files.
As I had stated the path is configurable for the certificates that are normally stored in that location and with a bit of tweaking that could be reused to move all of the config .
Those had been added a long time ago and I guess nobody ever added them to the Wiki page. However, you can see examples of them here.
I'm adding a few more arguments now, so i'll update the arguments Wiki page after I get them added.

Just to give my $0.02, in the ACMESharp library, with the 0.8 release, I moved to a "Profile" mechanism, that allows you to create multiple Vaults (one with each Profile).  Along with that is the default Profile which is implicitly stored in the \ProgramData folder if you run with Admin privileges, and in the user's %LOCALAPPDATA% if you don't.
All of the PowerShell cmdlets take an optional Profile name to override which Vault they operate on, which by default goes to the default Profile.  I think this has turned out to be a good compromise for flexibility with overly complicating things.

I also will appreciate a way to have a "per server" way to store data (for example a flag so save them under the executable folder). I'd like to install the program on our servers, schedule renewals and so on, and allow any server administrators to manage it. A "per user" scenario is a big problem and trouble in a company environment.
thanks

If anyone wants to see this enhancement, add your +1 (via add your reaction) to the first comment of this issue.

Using the user's ApplicationData folder can also be a security issue: it's part of the roaming profile, so if the user is a domain user their registration key will end up getting synced to the DC and every other domain-joined machine they log into. If it's a dedicated production domain that might actually be wanted, but if the domain is shared with workstations it could result in super-easy key compromise if any single machine they've logged into is compromised with a RAT trojan.
We should at least have the option to keep the config directory local.

Add my vote. To me it makes no sense at all to store the config data under C:\Users<account>. An IIS web site is not per user. We have multiple admins on our server. If adminA logs in and configures letsencrypt, adminB will login and won't be able to run letsencrypt.exe. If adminA leaves, his account is deleted from the server, and everything breaks.
The correct location for config data is C:\ProgramData.
Similarly, the scheduled task should not run as the user who happened to run letsencrypt.exe first. It should run as SYSTEM by default.

to add to what @richard-browne says. Every server I've configured has been done by using "psexec -i -s" to make LE-win-simple run as system.

I've created a fork with the minimal changes required to change the vault location (which works in our corporate environment).
No attempt to address security or email issues raised.
https://github.com/murray-grant-fe/letsencrypt-win-simple

This should be fixed in v2 where the configpath is configurable on the command line - https://github.com/Lone-Coder/letsencrypt-win-simple/pull/394

#483


Missing output file(s) *assemblies_mqc.csv expected by process `NFCORE_METATDENOVO:METATDENOVO:TRANSRATE

Description of the bug
What is "*assemblies_mqc.csv" and how can I create one? I have run my assembly on rnaspades (spades/3.15.5+dfsg-2) outside of the pipeline (with all samples together) and then I have copied the assembled files to pipeline results/spades folder and provided assembly to the pipeline. However, no file named "assemblies_mqc.csv" was created by spades... This files is not described in spades documentation, so where is it coming from?
Command used and terminal output
nextflow run nf-core/metatdenovo -r dev -resume '[intergalactic_mccarthy]' -profile oist --save_trimmed true --assembly /flash/MillerU/Vibrio_first_paper_data/results/spades/transcripts.fasta --skip_eggnog --eukulele_db gtdb --outdir /flash/MillerU/Vibrio_first_paper_data/results/ --input /flash/MillerU/Vibrio_first_paper_data/samples_for_first_vibrio_paper.csv
ERROR ~ Error executing process > 'NFCORE_METATDENOVO:METATDENOVO:TRANSRATE (/flash/MillerU/Vibrio_first_paper_data/results/spades/transcripts.fasta)'
Caused by:
Missing output file(s) *assemblies_mqc.csv expected by process NFCORE_METATDENOVO:METATDENOVO:TRANSRATE (/flash/MillerU/Vibrio_first_paper_data/results/spades/transcripts.fasta)
Command executed:
transrate
--threads 2
--assembly transcripts.fasta
--output /flash/MillerU/Vibrio_first_paper_data/results/spades/transcripts.fasta_transrate \
mv /flash/MillerU/Vibrio_first_paper_data/results/spades/transcripts.fasta_transrate/assemblies.csv /flash/MillerU/Vibrio_first_paper_data/results/spades/transcripts.fasta_assemblies_mqc.csv
cat <<-END_VERSIONS > versions.yml
"NFCORE_METATDENOVO:METATDENOVO:TRANSRATE":
transrate: $(transrate --version)
END_VERSIONS
Command exit status:
0
Command output:
[ INFO] 2024-10-29 14:33:49 : Loading assembly: transcripts.fasta
[ INFO] 2024-10-29 14:33:56 : Analysing assembly: transcripts.fasta
[ INFO] 2024-10-29 14:33:56 : Results will be saved in /flash/MillerU/Vibrio_first_paper_data/results/spades/transcripts.fasta_transrate/transcripts
[ INFO] 2024-10-29 14:33:56 : Calculating contig metrics...
[ INFO] 2024-10-29 14:34:06 : Contig metrics:
[ INFO] 2024-10-29 14:34:06 : -----------------------------------
[ INFO] 2024-10-29 14:34:06 : n seqs                       235994
[ INFO] 2024-10-29 14:34:06 : smallest                         73
[ INFO] 2024-10-29 14:34:06 : largest                       10055
[ INFO] 2024-10-29 14:34:06 : n bases                    96555122
[ INFO] 2024-10-29 14:34:06 : mean len                     407.53
[ INFO] 2024-10-29 14:34:06 : n under 200                    1939
[ INFO] 2024-10-29 14:34:06 : n over 1k                     11489
[ INFO] 2024-10-29 14:34:06 : n over 10k                        3
[ INFO] 2024-10-29 14:34:06 : n with orf                     4775
[ INFO] 2024-10-29 14:34:06 : mean orf percent              51.89
[ INFO] 2024-10-29 14:34:06 : n90                             246
[ INFO] 2024-10-29 14:34:06 : n70                             298
[ INFO] 2024-10-29 14:34:06 : n50                             388
[ INFO] 2024-10-29 14:34:06 : n30                             706
[ INFO] 2024-10-29 14:34:06 : n10                            1741
[ INFO] 2024-10-29 14:34:06 : gc                             0.41
[ INFO] 2024-10-29 14:34:06 : bases n                      454609
[ INFO] 2024-10-29 14:34:06 : proportion n                    0.0
[ INFO] 2024-10-29 14:34:06 : Contig metrics done in 10 seconds
[ INFO] 2024-10-29 14:34:06 : No reads provided, skipping read diagnostics
[ INFO] 2024-10-29 14:34:06 : No reference provided, skipping comparative diagnostics
[ INFO] 2024-10-29 14:34:06 : Writing contig metrics for each contig to /flash/MillerU/Vibrio_first_paper_data/results/spades/transcripts.fasta_transrate/transcripts/contigs.csv
[ INFO] 2024-10-29 14:34:11 : Writing analysis results to assemblies.csv
Command error:
[DEPRECATION] The trollop gem has been renamed to optimist and will no longer be supported. Please switch to optimist as soon as possible.
[DEPRECATION] The trollop gem has been renamed to optimist and will no longer be supported. Please switch to optimist as soon as possible.
Work dir:
/flash/MillerU/Vibrio_first_paper_data/work/5e/77811d1cc91b56aa4f7f586fee340d
Container:
/flash/MillerU/Vibrio_first_paper_data/work/singularity/depot.galaxyproject.org-singularity-transrate-1.0.3--hec16e2b_4.img
Tip: you can try to figure out what's wrong by changing to the process work dir and showing the script file named .command.sh
-- Check '.nextflow.log' file for details
ERROR ~ Pipeline failed. Please refer to troubleshooting docs: https://nf-co.re/docs/usage/troubleshooting
Relevant files
No response
System information
N E X T F L O W  ~  version 24.10.0
HPC
slurm
Singularity
CentOS Linux
metatdenovo dev

Thank you for prompt response. Ignoring the error with config file helped and now pipeline is running without problem.


Implement tasks

Tasks
Tasks provide an easy way of integrating looping timer logic into modules. They can be specified to run every x milliseconds, with an initial start offset of y milliseconds, and can be told to run z times.
Usage
The @Module.task() decorator can be used to add tasks to modules. It takes a  ModuleTaskOptions as arguments, which contain the parameters talked about above.
class TestModule extends Module {
    @Module.task({ runFor: 3, runEvery: 3e3, offset: 3e3 });
    async task() {
        console.log('Meep!');
    }
}

ModuleTask

options {ModuleTaskOptions} - the task's options.
name {string} - the name of the task, used for identifying and dynamically starting/stopping the task.
runCount {number} - how many times the task has run.

ModuleTaskOptions
The options for the task.

runEvery {number} - runs the task with the specified interval, with times in milliseconds. Default: 0
runFor {number} - runs the task for a fixed number of loops. Default: 0
offset {number} - the delay after which a task will run, with times in milliseconds. Default: 0

To-do List

[ ] Fix this not binding correrctly during handler exectution.
[ ] Generalize logic and move it into Module.ts
[ ] Add logic for easier dynamic modification of the task's options.

 Tasks


Add quay support

Signed-off-by: Ashmita Bohara<EMAIL_ADDRESS>I am still trying to figure out ways to test it properly before merging. Looking for feedback to validate the approach.
Which problem is this PR solving?
Solves https://github.com/jaegertracing/jaeger/issues/2498
Short description of the changes

All docker images are now built with two tags one for dockerhub and another for quay.io.
All docker images are now uploaded to both dockerhub and quay.io.
All github actions now try to login to quay.io registry.

Thanks @Ashmita152 .
To my taste, there's too much repetition. What if we encapsulate the docker vs quay only within the scripts/travis/upload-to-docker.sh script? I don't think anything above it needs to know about where the images are being uploaded. And ideally we would even move the logging in steps into the same script, as there's no reason to do it before the actual upload decision is made. I assume we can pass the login/password via env and it won't be leaked to the logs (we should verify that).

Hi @yurishkuro
Thank you for the feedback. I see your point. Let me think about it.

Hi @yurishkuro
I have updated the PR with your suggestion. I hope this time, I took the expected approach.
Two things we need to do before merging:

create jaegertracingbot user in quay.io and allow it to be able to upload images to jaegertracing quay org.
get its token and set QUAY_TOKEN envvar in github.

Regards.

did you try pushing the login steps into scripts/travis/upload-to-docker.sh? Before GHA we used to do login manually, and if it is done in that script, we do not need to keep adding the login steps in all the GHA workflows.

@Ashmita152 see my test PR #2797. This is the GH run: https://github.com/jaegertracing/jaeger/runs/1846319012?check_suite_focus=true
The env var is never printed in the logs when logging into docker. We can remove all the login steps from the workflows.

Sure Yuri, sounds good. I will update it.

Hi @yurishkuro
I just realised that the DOCKERHUB_TOKEN is exposed in the above GitHub Action run you shared.

I agree with you that we need to cleanup this login. I am trying to find a way to move dockerhub login and quay login as a separate GitHub Action if possible.

Oh I misunderstood your point. You are right, it will never show up in logs.

Hi Yuri,
I decided to keep the dockerhub+quay login part as a separate github action mainly because of two reasons:

This way those tokens are only visible to this github action script, which I felt more safer since it won't change often.
Our upload-to-docker.sh isn't very neat so I didn't want to add login part there.

Let me know in case you feel otherwise.

A couple of comments:

the username should be externalized, as Quay supports robot accounts tied to the organization, which aren't real accounts. The username would then be something like jaegertracing+github_workflows
you can assume that all repositories are prefixed, like: docker.io, quay.io, ...

I just added the following as "repository secrets"

The DOCKERHUB_TOKEN and JAEGERTRACINGBOT_GITHUB_TOKEN were already set as organization secrets.

Thank you for the feedbacks. Sure Yuri, I will test this PR from my fork properly both from merge to master + release perspective and share with the links.

Nice!

I think this is almost ready to merge (just one tiny fix), but it would be nice to see a test run logs. Of course, we could merge and see if the publishing succeeds and fix forward.

Right, I am still testing with my fork. I will share the result once I am done testing.

Hi @yurishkuro
I can confirm that this PR works as expected. I tested by merging my branch in my fork.
I wasn't able to make the quay's robot user work for pushing to quay image and keep getting this error while pushing from robot account.
unauthorized: access to the requested resource is not authorized

Then I used the main quay user account.
I found one bug and fixed it here: https://github.com/jaegertracing/jaeger/pull/2783/commits/3b2ea2e2a742501d0ea1624533661d4bfcc91ff6
Otherwise for quay.io,
docker push ${registry}/${image}

This will fail because image will be missing because we was never labelling the latest tag for quay.
All the GH actions completed successfully: https://github.com/Ashmita152/jaeger/runs/1873169008
All Quay Images: https://quay.io/organization/ashmita1 (quay by default creates private images so you won't be able to see many images)
All Dockerhub Images: https://hub.docker.com/u/ashmita1
Regards.

Given the positive test evidence, I think it's time to merge, and see if it works with real credentials / repos.

@jpkrohling any thoughts?
printenv QUAY_TOKEN | docker login quay.io --username jaegertracingbot+github_workflows --password-stdin
Error response from daemon: Get https://quay.io/v2/: unauthorized: Could not find robot with specified username

Oops, it should be jaegertracing+github_workflows (no bot) I updated the env var QUAY_USERNAME with the correct username.


[MB-9111] too can submit and auto-approve SIT extension for customer

Description
Making a draft PR for now - there is something wrong with my service, as the test fails. It doesn't seem to actually create the new SIT extension, or maybe properly associate it with the shipment.
I also need to integrate the work @NamibiaTorres is doing on their branch for Prime API SIT Extension creation.
Reviewer Notes
Is there anything you would like reviewers to give additional scrutiny?
Setup
Add any steps or code to run in this section to help others prepare to run your code:
echo "Code goes here"

References

Jira story for this change

Messages

:book:
:link: MB-9111

  Generated by :no_entry_sign: dangerJS against 0171e87a6cf16d127b24d52baf6e069057547ec7


Error: missing 'path'

change line 97  in server.py from:
return send_from_directory(directory=app.config['UPLOAD_FOLDER'], filename= platforms[__dev]['file'],
to:
return send_from_directory(app.config['UPLOAD_FOLDER'], platforms[__dev]['file'],
python/flask has changed the format.

Works for me.  Flask V2.00


Create Wiki page for devonfw cobigen ide setup for intellij

As a CobiGen developer, I want to use the intellij IDE so that I don't have to rely completely on the Eclipse IDE.
We need to create an easy intellij configuration guide based on the latest tech talk by @Lur1an

Hello, I am interested to work on this issue.
I am new in open source but for sure I can do this.
please assign it to me than I should start working on this.

@professorabhay This ticket is quite peculiar as it depends on a tech talk I gave in our intern CobiGen team. But feel free to look at other issues if you want to contribute!


[Tests] Improve createami test to validate and use the created AMI

Description of changes
By enabling DisableValidateAndTest we're going to execute all the Kitchen tests with tag:install_ prefix as part of the Image Builder validation phase.
We extended the test to use the generated AMI for a cluster and run a job on it.
We also extended the dimensions to run it on every supported OS and removed scheduler dimension.
Tests

Executed the test for all the OSes.

test-suites:
  createami:
    test_createami.py::test_build_image:
      dimensions:
        - regions: ["eu-west-3"]
          instances: {{ common.INSTANCES_DEFAULT_X86 }}
          oss: {{ common.OSS_COMMERCIAL_X86 }}
          schedulers: ["slurm"]

New pr: https://github.com/aws/aws-parallelcluster/pull/6168


Modularity on Swarm example

Current Swarm example requires users to modify manifests to choose which services deployed to swarm cluster. The ease the operation, the examples should adhere to Compose and/or Kubernetes examples that are modular.

Support for Swarm has been deprecated.


Add failing regression test for #100

This commit adds a failing regression test which demonstrates the issue reported in #100.  This test asserts that the size of the compressed SnappyOutputStream output should only depend on the size and contents of the input and not on how many separate write() calls are used to pass that input to the output stream.

ok. let's work on fixing this issue based on this test.

started development at https://github.com/xerial/snappy-java/tree/fix-output-size branch

Addressed in #102


minify stopped working after upgrading to latest version

Hello,
I've upgraded an Ionic app to the latest version of libs, going from:
"@angular/common": "2.4.8",
"ionic-angular": "2.3.0",
"@ionic/app-scripts": "1.1.4",
To (tried different options below):
"@angular/common": "4.0.2",
"ionic-angular": "3.0.1" / "nightly",
"@ionic/app-scripts": "1.3.4" / "1.3.2",
Everything working great, except I'm getting the following error when trying to minify:
[]  ionic-app-scripts 1.3.4
[]  minify started ...
[]  cleancss started ...
[]  uglifyjs started ...
[]  ionic-app-script task: "minify"
[]  Error: Cannot read property 'filter' of undefined in undefined at line undefined, col undefined, pos
undefined
Error: Cannot read property 'filter' of undefined in undefined at line undefined, col undefined, pos undefined
at BuildError.Error (native)
at new BuildError (C:\appdir\node_modules@ionic\app-scripts\dist\util\errors.js:16:28)
at taskReject (C:\appdir\node_modules@ionic\app-scripts\dist\worker-process.js:36:22)
at C:\appdir\node_modules@ionic\app-scripts\dist\worker-process.js:14:13
at process._tickCallback (internal/process/next_tick.js:103:7)
npm ERR! Windows_NT 10.0.14393
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js" "run" "buildminify"
npm ERR! node v6.9.5
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE
I'm on Win10, latest Ionic version

I've solved it for now by downgrading to  ionic-app-scripts 1.3.1

@xims IMO downgrading und closing the issue for sure is not the solution and also will not help the team to figure out the bug. I think you should reopen the bug the have the attention from the team to resolve the bug dor all us.
Thx

Same bug here!

it's not more a solution
➜  ionic2 git:(dev) ✗ npm install<EMAIL_ADDRESS>xxx@ /opt/git/ionic2
└── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>

Yeap, the problem only appears on V1.3.2.
It seems that it's this added line (36 of uglifyjs.ts as of V1.3.2)
const jsFilePaths = context.bundledFilePaths.filter(bundledFilePath => bundledFilePath.endsWith('.js'));
I do not have the time now for investigating this issue, since I'm not acquainted with this project's source code.
Will use V1.3.1 for now (and no, --prod is not working for me).

@akaztp I can't use 1.3.1 it not in npm any more

that's strange, i've just installed it normally moments ago.
npm install<EMAIL_ADDRESS>

I've managed to make "--prod" work in V1.3.1.
But after V1.3.1 the uglyfyjs also breaks on a "--prod" build...

I'm seeing the same thing as @akaztp with v1.3.7.

I just realized that in my case, uglify was choking on es6, which is what I had my TypeScript compiling to. When I switched to es5, --prod worked fine. Hopefully this will help someone else.

I've solved it for my project, use another command.
ionic-app-scripts build --minifyCss --minifyJs


Resizeconstraint

Add Flatten op
Add constraints which (hopefully) allow for mutation of graphs with Flatten and Reshape without messing up the sizes
Add Softmax op

Codecov Report

Merging #11 into master will decrease coverage by 6.94%.
The diff coverage is 58.59%.

@@            Coverage Diff             @@
##           master      #11      +/-   ##
==========================================
- Coverage    92.3%   85.35%   -6.95%
==========================================
  Files          10       11       +1
  Lines         481      601     +120
==========================================
+ Hits          444      513      +69
- Misses         37       88      +51

Impacted Files
Coverage Δ

src/ONNXmutable.jl
100% <ø> (ø)
:arrow_up:

src/deserialize/ops.jl
97.24% <100%> (+0.18%)
:arrow_up:

src/serialize/serialize.jl
97.53% <100%> (+0.12%)
:arrow_up:

src/deserialize/constraints.jl
27.77% <27.77%> (ø)

src/deserialize/deserialize.jl
95.16% <97.14%> (+2.05%)
:arrow_up:

src/shapes.jl
82.69% <0%> (+3.84%)
:arrow_up:

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 78c7b47...6529cbd. Read the comment docs.


opencl-clhpp: 2.0.15 -> 2024.05.08

Description of changes
Rebased https://github.com/NixOS/nixpkgs/pull/277072 and applied a patch to arrayfire, which fails to build with new opencl-clhpp.

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

[x] 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.

@ofborg build arrayfire

I guess arrayfire isn't built due to freeimage being marked insecure, so ofborg won't build it.

Result of nixpkgs-review pr 335392 run on x86_64-linux 1

2 packages marked as broken and skipped:

arrayfire
forge

7 packages built:

clfft
clpeak
mandelbulber
opencl-clhpp
sanjuuni
virtualgl
virtualglLib

Result of nixpkgs-review pr 335392 run on x86_64-darwin 1

1 package built:

opencl-clhpp

Closes #276448

This should be ok to merge, right? LGTM, but I was waiting for another opinion.

I don't see why not. Arrayfire is already insecure, but it does build with the patch applied. Fixing it should be easier in the future.


SQLAlchemy error when adding users

When running the dev server at git tag '14a' (or later) I encounter the following error when adding a new User into the db:
sqlalchemy.exc.StatementError: utcnow() takes no arguments (1 given) (original cause: TypeError: utcnow() takes no arguments (1 given))
I'm using Python 3.4.  According to http://stackoverflow.com/questions/22481905/sqlalchemy-python-3-default-column-value this is a known issue.
What version of Python should I be using for the flasky app?
Many thanks in advance.

The issue with default values was fixed long ago. Are you using the latest versions of Python 3.4 and SQLAlchemy?

That was indeed the problem.  I had installed SQLAlchemy 0.8.4 specified in the requirements.txt with my Python 3.4.2 installation.  Upgrading SQLAlchemy to 0.9.8 solved the problem.
Thanks Miguel.


Grammar improvements in power-bi-visual-add-3d-column-layer.md

Per prior arrangement, if this PR needs approval, or if the commits need to be pulled into the private repo, only @eriklindeman should be notified. Thanks.

#sign-off


Export VAceEditorInstance

Export the VAceEditorInstance type from the package. Now you can have type checking when using TypeScript:
import type { VAceEditorInstance } from 'vue3-ace-editor';

const editor = ref<VAceEditorInstance>();

You should use InstanceType<typeof VAceEditor>

It doesn't work.


Adding light theme

Sorry, but I'm not interested in maintaining any additional themes. Feel free to maintain this within your own fork though. Thanks anyway, but closing this.

No problem. Thanks for sharing.


upgrade database after dde:system:update

Problem
We should check if a new version requires dde to upgrade the database (for example after the mariadb image has been updated).
Otherwise it will crash like this:
[ERROR] InnoDB: Plugin initialization aborted with error Generic error
[Note] InnoDB: Starting shutdown...
[ERROR] Plugin 'InnoDB' init function returned error.
[ERROR] Plugin 'InnoDB' registration as a STORAGE ENGINE failed.
[Note] Plugin 'FEEDBACK' is disabled.
[ERROR] Unknown/unsupported storage engine: InnoDB
[ERROR] Aborting
[Note] [Entrypoint]: Entrypoint script for MariaDB Server 1:10.11.4+maria~ubu2204 started.
[Note] [Entrypoint]: Switching to dedicated user 'mysql'
[Note] [Entrypoint]: Entrypoint script for MariaDB Server 1:10.11.4+maria~ubu2204 started.
[Note] [Entrypoint]: MariaDB upgrade (mariadb-upgrade) required, but skipped due to $MARIADB_AUTO_UPGRADE setting
[Warning] Setting lower_case_table_names=2 because file system for /var/lib/mysql/ is case insensitive
[Note] Starting MariaDB 10.11.4-MariaDB-1:10.11.4+maria~ubu2204 source revision 4e2b93dffef2414a11ca5edc8d215f57ee5010e5 as process 1
[Note] InnoDB: Compressed tables use zlib 1.2.11
[Note] InnoDB: Number of transaction pools: 1
[Note] InnoDB: Using ARMv8 crc32 + pmull instructions
[Note] mariadbd: O_TMPFILE is not supported on /tmp (disabling future attempts)
[Note] InnoDB: Using liburing
[Note] InnoDB: Initializing buffer pool, total size = 128.000MiB, chunk size = 2.000MiB
[Note] InnoDB: Completed initialization of buffer pool
[Note] InnoDB: Buffered log writes (block size=512 bytes)
[ERROR] InnoDB: Upgrade after a crash is not supported. The redo log was created with MariaDB 10.5.21. You must start up and shut down MariaDB 10.7 or earlier.
[ERROR] InnoDB: Plugin initialization aborted with error Generic error
[Note] InnoDB: Starting shutdown...
[ERROR] Plugin 'InnoDB' init function returned error.
[ERROR] Plugin 'InnoDB' registration as a STORAGE ENGINE failed.
[Note] Plugin 'FEEDBACK' is disabled.
[ERROR] Unknown/unsupported storage engine: InnoDB
[ERROR] Aborting

Current workaraound/fix:
docker exec -it dde_mariadb mysql_upgrade -uroot -proot

This is currently resolved with the fixed Version: https://github.com/whatwedo/dde/blob/b02f5af79319f67eba4fbb5323cb6c0ad8859c39/docker-compose.yml#L35
Will become relevant when we bump the version


Use tagged logger already configured on Rails

Fixes #488

fix: call tagged from an existing tagged logger
fix: correctly uninitialize logger between runs

Thanks @kriansa, I'll get a release out soon.

this change caused:
.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/semantic_logger-4.15.0/lib/semantic_logger/base.rb:190:in `block in tagged': no block given (yield) (LocalJumpError)
	from /Users/bradleyrosintoski/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/semantic_logger-4.15.0/lib/semantic_logger/semantic_logger.rb:319:in `fast_tag'
	from /Users/bradleyrosintoski/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/semantic_logger-4.15.0/lib/semantic_logger/base.rb:197:in `tagged'
	from /Users/bradleyrosintoski/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/dotenv-3.0.3/lib/dotenv/rails.rb:93:in `block in <class:Rails>'

Fixes #488

fix: call tagged from an existing tagged logger
fix: correctly uninitialize logger between runs


guard $event.target, $event.currentTarget and infer currentTarget type

Codecov Report

Merging #278 (2a925c3) into main (909d207) will increase coverage by 0.03%.
The diff coverage is 84.61%.

@@            Coverage Diff             @@
##             main     #278      +/-   ##
==========================================
+ Coverage   41.12%   41.15%   +0.03%
==========================================
  Files         129      129
  Lines       19871    19878       +7
  Branches     1163     1164       +1
==========================================
+ Hits         8172     8181       +9
+ Misses      11699    11697       -2

Impacted Files
Coverage Δ

packages/compiler-tsx/src/generate.ts
88.59% <84.61%> (+0.05%)
:arrow_up:

packages/shared/src/rpc/handlers.ts
97.93% <0.00%> (+0.51%)
:arrow_up:

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 909d207...2a925c3. Read the comment docs.


Calibre ebook Manager install failure when PowerToys running.

PowerToys version: 0.21.1

PowerToy Utility:

Running PowerToys as Admin: YES

Windows build number: [run "winver"] 19041.508

Calibre ebook manager install file: calibre-4.23.0.msi

When Power Toys running (as Admin), latest version of Calibre ebook manager fails to install.
Gets part-way through the install process then rolls back.
Close PowerToys and repeat install = successful install.

@syclik50
is this the same as https://github.com/microsoft/PowerToys/issues/6429?

Apart from Windows and Calibre version differences, aappears to be same issue as #6429.
Thanks for the pick-up.


Gate Http Request Chain Body Syntax

Note
Issues in this repo are for tracking bugs, feature requests and questions for the tasks in this repo
For a list:
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks
If you have an issue or request for the Azure Pipelines service, use developer community instead:
https://developercommunity.visualstudio.com/spaces/21/index.html )
Required Information
Entering this information will route you directly to the right team and expedite traction.
Question, Bug, or Feature?
Type: Question
Enter Task Name: All Gates
list here (V# not needed):
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks
Environment

Server - Azure Pipelines or TFS on-premises?

If using TFS on-premises, provide the version:

If using Azure Pipelines, provide the account name, team project name, build definition name/build number:
life-pacificlife, Databases, definition id 104

Agent - Hosted or Private:

If using Hosted agent, provide agent queue name:

If using private agent, provide the OS of the machine running the agent and the agent version:

Issue Description
[Include task name(s), screenshots and any other relevant details]
There is a syntax available in the gate but I am unable to find the documentation for it.
For example the following in in the body of a call. It does work for formating the time but I am unable to identify the method/function to get the current time.
{{#if scheduleendtime}},"u_end_date": "{{#if scheduleendtime}}{{toDateTimeFormat '$(scheduleendtime)' 'yyyy-MM-dd HH:mm:ss UTC'}}{{/if}}"{{/if}}
Task logs
[Enable debug logging and please provide the zip file containing all the logs for a speedy resolution]
Troubleshooting
Checkout how to troubleshoot failures and collect debug logs: https://docs.microsoft.com/en-us/vsts/build-release/actions/troubleshooting
Error logs
[Insert error from the logs here for a quick overview]

@Dejulia489 Apologies for the delay in response. There is no available method to get the current UTC time. The snippet posted above takes the time value in an input field from the user. Optionally, you can use the stage level pre-defined variable "$(Release.Deployment.StartTime)".

No worries, thanks for getting back to me.  I noticed there are methods to format time 'toDateTimeFormat'. What other methods are available, is it documented?
Thanks

There is no documentation available. If there is a specific functionality you looking for, let me know and i'll see if there is anything that can help.

Documentation would be nice. Closing this issue.


Adding a reference for a traefik example

As suggested here a proposal for a reference to my traefik example repo.

Thanks for contributing, and I will merge now and refactor the reference in README later.


Great idea, understanding is a vector inverted index,right?

Hi @lonngxiang, thank you for your interest.
You can think of DensePhrases as a vector inverted index since it has an inverted file system (IVF) as its component.


i18n xhr problem loading local file

phonon.js line 904
 if (xhr.status === 200) {
should be
if (xhr.status === 200 || !xhr.status && xhr.responseText.length) {
see https://github.com/quark-dev/Phonon-Framework/issues/41

Hi @PaulRieger,
Thank you for reporting this.
I added the same patch as #41.
It will be available asap as a release.


LegionProf issue

I got this error message with the attached prof log
Reading log file prof_0.gz...
parsing prof_0.gz
Matched 45761 objects
Generating interactive visualization files in directory legion_prof
Traceback (most recent call last):
  File "/home/kihiro/Softwares/legion/tools/legion_prof.py", line 4036, in <module>
    main()
  File "/home/kihiro/Softwares/legion/tools/legion_prof.py", line 4030, in main
    file_names, show_channels, show_instances, force)
  File "/home/kihiro/Softwares/legion/tools/legion_prof.py", line 3860, in emit_interactive_visualization
    mem_level = mem.emit_tsv(mem_tsv_file, 0)
  File "/home/kihiro/Softwares/legion/tools/legion_prof.py", line 846, in emit_tsv
    point.thing.level, None)
  File "/home/kihiro/Softwares/legion/tools/legion_prof.py", line 1853, in emit_tsv
    inst_name = repr(self)
  File "/home/kihiro/Softwares/legion/tools/legion_prof.py", line 1981, in __repr__
    if (pos+1)%4 == 0 and pos != dim_order:
NameError: name 'dim_order' is not defined

I am on commit 22954a1842f8334013743172637d32d232547786 on control_replication.
prof_0.gz

You can safely comment out these two lines for now and nothing bad will happen:
https://gitlab.com/StanfordLegion/legion/-/blob/master/tools/legion_prof.py#L1981-1982
@seemamirch will need to say if that is the correct long term fix.

pushed a fix to master
https://gitlab.com/StanfordLegion/legion/-/commit/7961c04906048570b3cd50ba2a407cf2ebc6fbf8

fixed, thanks

fixed, thanks


UnableToWriteFile on Laravel 9

Laravel Version: 9.11
PHP Version: 8.0.2
Package & Version:
flysystem-aws-s3-v3: 3.0

Hello there,
I was upgrade the package, but I can't upload any file to my bucket even the system didn't hit my IAM.
I try to change the throw to enable in config/filesystems.php, The error returns "Unable to write file at location: xxx.jpg"
What's wrong with that? Can anyone help me?

Does it still fail when using 9.17 (latest version) ?

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.

If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Hi @henzeb, I was upgrading the version to 9.17, but it's same.

I am having this issue with 9.52.7
logging tells me the access key (which is blank when one AWS service is talking to another AWS service) is the source of the error:
<Error>InvalidAccessKeyId<Message>The AWS Access Key Id you provided (truncated...)


ENFORCE_NUMERICAL_STABILITY

We have NaN enforcer working in single thread, and it becomes serious bottleneck in current architecture. So we might use some perf improvements there.
https://github.com/deeplearning4j/deeplearning4j/issues/1701

We MAY want to just disable this?

Why? Let's just make it simd-able like any other op out there?

I mean it's just bad practice to use in general.
On Fri, Jun 17, 2016 at 9:21 AM, raver119<EMAIL_ADDRESS>wrote:

Why? Let's just make it simd-able like any other op out there?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/deeplearning4j/nd4j/issues/1027#issuecomment-226814451,
or mute the thread
https://github.com/notifications/unsubscribe/ABF18memndQVqCJZYoPaphDTXwM7jSivks5qMsl0gaJpZM4I4g4u
.

Implemented as native here: https://github.com/deeplearning4j/nd4j/pull/1025
However, removal of replaceNans call after each op is still a concern, and to be decided


Update deploystudio_helper.php

This fixes an issue caused when a DeployStudio workflow was deleted.
Uncaught Exception: SQLSTATE[HY000]: General error: 1366 Incorrect string value: '\xF3\xB3\x06\x00' for column 'dstudio_last_workflow' at row 1

Please revert the changes in site_helper.php

Please revert the changes in site_helper.php

To be more clear: I get merge errors with your PR. Please fork the latest wip branch and submit a PR with only changes to the ds helper file.

fixed in #596