Documents over 2GB cause a database error and fail to upload

Issue Summary
When uploading a large document (2.41GB) the upload failed with an Integer out of range error.
Steps to Reproduce

Create a wagtail project
Attempt to upload a document where the file size is larger than the maximum int value in the database (about 2GB)

Any other relevant information. For example, why do you consider this a bug and what did you expect to happen instead?
I expected the document to upload normally as other documents do.

I have confirmed that this issue can be reproduced as described on a fresh Wagtail project: yes

Working on this
I have prepared a pull request to modify the AbstractDocument class in documents.models to use PositiveBigIntegerField for file_size instead of PositiveIntegerField. This will resolve the issue.

I just ran into this and had to override the document model and declare file_size as a BigIntegerField. Luckily we were already using custom document models, otherwise this would have required a data migration too. Dropping here in case it's helpful.
I do think you might as well make it a bigint so people don't hit this issue. Using django-s3direct with Wagtail works wonders, so it's easy to upload large video files these days.
from django.db import models

from wagtail.documents.models import AbstractDocument

class CustomDocument(AbstractDocument):
    file_size = models.BigIntegerField(null=True, editable=False)


Add TeXworks Recipe

Don't merge yet.
Has issues with localization and themes
It's a qt based ui for TeX. It works, but I have an issue with the theme and the locale
(I have an Italian keyboard layout, and it seems to pick that for the ui, even though the system language is english)
QGtkStyle was unable to detect the current GTK+ theme.
Qt: Cannot set locale modifiers:

Do I need some other libraries for that? (the locale is fixed if I set LC_ALL=$LANG before I run the AppImage, LANG being en_US.UTF-8)

Please try
sudo su
cat > /etc/Trolltech.conf <<EOF
[Qt]
style=adwaita
EOF

although this is not a solution I would like to know whether this changes anything.

Also, please bundle a LaTeX distribution. I have done this using texlive for TeXstudio with success.
See https://github.com/probonopd/AppImages/blob/master/recipes/texstudio/Recipe

I'm not sure that bundling a LaTeX distribution is a good idea. In the past I've had to install older versions of it, or keep more than one. Or even adding packages.
I'd prefer to keep the two separate.

A core principle of AppImage is that it must have no other dependencies than what can be assumed to be part of every target system. What you could do, though, is to have a wrapper script prefer a LaTeX installed on the base system and fall back to the minimal one inside the AppImage if none is found.

A LaTeX distribution is not strictly a dependency in this case, but I see what you mean. I'll have a look at that as soon as the other issues are fixed

The Trolltech.conf file worked if I used the one in ~/.config/.
I don't have that error any more, but the ui now looks like windows 98-era.

Strange. More investigation needed in this area. Now delete the Trolltech file in ~/.config/, does it still work? I haven't figured out what's going on here yet.

No, it goes back to the error if I remove that file

The system-wide file is /etc/xdg/Trolltech.conf.
I get the same result I got with the user one.

Please google around a bit. Maybe the theme can also be set using an environment variable?

As far as I can tell the environment variable is available only for qt5 (QT_STYLE_OVERRIDE=)

Some information is here https://wiki.archlinux.org/index.php/Uniform_look_for_Qt_and_GTK_applications

Yes, that's where I was looking earlier. I'll have another look around

The issue with the language seems to be caused by the fact that qt4 reads the locale from LC_NUMERIC (How you format your numbers).
In my case it's set to italian.
I assume the regular ubuntu install does something to fix this.

Thanks


Update ch01-00-simple-crud-getting-started.md

Missing features "attributes" for async-std on sample of manually edited Cargo.toml, it was correctly specified when using "cargo add". Thank you!

Hey @nicarran, thanks for spotting that! Actually, we're revamping the tutorials, so feel free to let us know your thoughts at:

https://github.com/SeaQL/sea-orm-tutorial/issues/8


Hyper-parameter Settings In Paper vs Code

Hi, I have a quick question regarding hyper-parameter settings mentioned in the paper vs. those set in train_model.sh
In the paper:
D = 200, H = 400, C = 5, L = 3, and Q = 2
In train_model.sh:
-miniBatchSize  64
-embeddingDim  64
-bowDim  200
-hiddenSize  64
-epochs  20
-learningRate 0.1
-validArticleDir  $OUT_DIR/valid.filter/article/
-validTitleDir  $OUT_DIR/valid.filter/title/
-window  $WINDOW
-printEvery   100
-encoderModel  "attenbow"
-attenPool  5
Are these two sets of hyper-parameters the same? I thought embeddingDim (decoder embedding dimension) has the same size as bowDim (encoder embedding dimension). Also, shouldn't hiddenSize be 400 (H=400)? I am also a little confused as to what C, L and Q map to in the code. Any clarification would be appreciated. Cheers.

thanks for your question! this does reveal some of the inconsistency between the codebase and the published article. you are right, some of these have different values than the published values 😕
bowDim (encoder size or encoder embedding dim) seems to be a hyperparameter separate from the hiddenSize: the encoder transforms the variable length article input into fixed dim context vector, H in the discussion of encoders in the paper. and in nnlm.lua it is considered as an additional constant P

D = embeddingDim
H = hiddenSize
Q = attenPool
C = window
L = 3, number of layers in convolutional encoder, is hardcoded in https://github.com/falcondai/NAMAS/blame/master/summary/encoder.lua#L79-L80 and it is not used in attention encoder

according to https://github.com/falcondai/NAMAS/blame/master/summary/nnlm.lua#L50 and https://github.com/falcondai/NAMAS/blame/master/summary/encoder.lua#L156

Right. So is it correct to say that these should be the hyper-parameter values in the code (to match the paper's):
-miniBatchSize 64
-embeddingDim 200
-bowDim 200
-hiddenSize 400
-epochs 15
-learningRate 0.05
-validArticleDir $OUT_DIR/valid.filter/article/
-validTitleDir $OUT_DIR/valid.filter/title/
-window 5
-printEvery 100
-encoderModel "attenbow"
-attenPool 2

These does look more faithful to the hyperparameters described in the paper. But both in terms of perplexity (27.74 vs 27.1 in the paper) and test examples generated, I found the default setting in code to be similar to the published figures. I would be interested to see the results using this particular setting.


Add JDBC for OAuth 2.0 Device Authorization Grant

https://github.com/spring-projects/spring-authorization-server/pull/1143

I have noticed that only memory based device authorization can be used now, and JDBC based device authorization is not currently implemented


core: correctly capitalize MAVLink

As requested in #136.

Coverage remained the same at 48.634% when pulling cf5e973eca64ebfa4eabacfa698cf05cc44edcf1 on mavlink-printfs into e12c969058b2925b13b2bd8da0f96fc5c9ce4892 on develop.

Coverage remained the same at 48.634% when pulling 3bb67f0a1207436f08d5abb47b8b6da11a52d14b on mavlink-printfs into 97d98b92a3f913308497f0e8ab4e9f61e742ddd5 on develop.


[JENKINS-59172] - Use GitHub as a source of the plugin's documentation on plugins.jenkins.io

There is nothing left on the Wiki Page except changelog, so we can safely replace it by GitHub page.
See https://groups.google.com/forum/#!topic/jenkinsci-dev/VSdfVMDIW-A

Please do not file origin branches.


[charts] Fix, double registration of charts API

CATEGORY
Choose one

[X] Bug Fix
[ ] Enhancement (new features, refinement)
[ ] Refactor
[ ] Add tests
[ ] Build / Development Environment
[ ] Documentation

SUMMARY
Was causing no impact but it's an error, Chart API is already being registered on app.py
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

REVIEWERS

Totally @villebro and @nytai


OpenCV error when using docker

Hi,
unfortunately, I get the following error when using your dockerfile to build a container and test the code:
AttributeError: module 'cv2.dnn' has no attribute 'DictValue'
I think it is an issue with the opencv version, since installing a newer version inside the container resolves it.

Hi,
I've encountered the same problem (opencv-python==<IP_ADDRESS>), can you let me know what version of opencv version you switched to in order to resolve it?

opencv-python==<IP_ADDRESS> seems to work. I will open a PR.

It appears updating open-cv to another version doesn't really work for me, as I just get the error when running:
Traceback (most recent call last):   File "anonymize.py", line 1, in <module>     import cv2   File "/opt/conda/lib/python3.8/site-packages/cv2/__init__.py", line 181, in <module>     bootstrap()   File "/opt/conda/lib/python3.8/site-packages/cv2/__init__.py", line 175, in bootstrap     if __load_extra_py_code_for_module("cv2", submodule, DEBUG):   File "/opt/conda/lib/python3.8/site-packages/cv2/__init__.py", line 28, in __load_extra_py_code_for_module     py_module = importlib.import_module(module_name)   File "/opt/conda/lib/python3.8/importlib/__init__.py", line 127, in import_module     return _bootstrap._gcd_import(name[level:], package, level)   File "/opt/conda/lib/python3.8/site-packages/cv2/gapi/__init__.py", line 301, in <module>     cv.gapi.wip.GStreamerPipeline = cv.gapi_wip_gst_GStreamerPipeline AttributeError: partially initialized module 'cv2' has no attribute 'gapi_wip_gst_GStreamerPipeline' (most likely due to a circular import)
Googling the errors appear to show that its caused by to multiple opencv installations, but adding the line:
RUN pip uninstall opencv-python opencv-contrib-python opencv-contrib-python-headless opencv-python-headless --yes
before installing dependencies in the docker file doesn't appear change anything.

You are right my workaround is to change the entry point of the container to start a bash terminal and then install pip install opencv-python==<IP_ADDRESS> in the container. Afterwards, running the anonymize.py script works. However, I also don't know why changing the opencv version in the Dockerfile throws the GStreamer error.

nothings worked, after tried those approaches

opencv-python==<IP_ADDRESS> seems to work. I will open a PR.

tks，my problem solved!

opencv-python==<IP_ADDRESS> does not work for me, however opencv-python-headless==<IP_ADDRESS> does! I also needed to add ftfy==6.1.3 as requirement into the Dockerfile as well.


Textboxes Cannot Be Typed In on Linux

I am trying to use Armory2D on Linux and I am unable to type into text boxes. Interestingly I can backspace if there is already text in the textbox, but I can't type any new characters into it. This might actually be an issue with Zui now that I think of it.

Same here actually. I am working on a big(ish) game at the moment and this feature will be crucial for the development. I hope this can be worked out soon! 😃 By the way, I'm having this issue on the 2.8 branch. Not sure if that makes a difference though.

Same problem here. Debian Buster/Sid.

I'm going to close this because because it is duplicated by armory3d/armory#888 and it is probably better to track it in that repo. Participants could :+1: that issue to show support.


Added Example section in README.md file

I have added an example explaining how to use this package. I was having difficulty and facing some errors in the following example which is used on the website.
<script setup>
import Datepicker from 'vue3-datepicker'
import { ref } from 'vue'
const picked = ref(new Date())
</script>

<template>
  <datepicker v-model="picked" />
</template>

So for the user guide, I have added an example that is working for me in Vue 3.

The snipped on the website should work as well, however a more classic example is a good idea! Thank you!


Authors app not able to search Teams groups or Channels while drafting the message

Authors app not able to search Teams groups or Channels while drafting the message and Getting error message - We could not find any matches.
Getting same message while searching any user.

@vikasssdn1 : Check if this helps: https://github.com/OfficeDev/microsoft-teams-apps-company-communicator/wiki/Troubleshooting#7-when-upgrading-company-commmunicator-from-a-version-prior-to-v4-please-ensure-that-the-users-app-is-installed-in-the-teams-to-send-messages-to-the-team-or-its-members
You would require to install user bot app to required teams channels. then respective teams channels will display in the dropdown(first two options)


logging: Allow request logging config to determine log level

Perhaps I am misunderstanding the original intentions, but the request logging configuration seems overly complicated.
I started working on documenting the config in https://github.com/thanos-io/thanos/pull/4934 and there are some comments there, but overall I find the config is unclear.
This PR changes the implementation slightly, since my take is that if I have gone to the trouble of configuring request logging, all I want to do is specify at what level I believe that should be output and have that determined by the  --log.level
This change allows the 'level' config in request logging 'options'
to determine the output level for those logs.
Prior to this, without setting --log.level flag on the component
to debug, no logs were emitted.
With this implementation, I think it would make sense to restrict the logging options to one of debug, info
If this approach seems reasonable to the maintainers we would need to tackle gRPC in similar fashion.

[ ] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes

Verification

closing based on the docs improvements in #4934
we can track improvements in #4961


Figure out algorithm lifecycles

Figure out state machines for the various cryptographic algorithms.
The intention is to enforce the transitions in due course.
e.g. for digests:
new --> init --> update +-> final +-> free
        ^        ^      |         |
        |        +------+         |
        +-------------------------+

You should be able to go from update to init as well.
And then there is the XOF which might allow repeating final (but only for XOF digests).

Agreed with both.  I was keeping it simple since it was only an illustration :)

The MAC portion of this is discussed in #13210

Continuing the digest theme:

The primary path through is along the black edges going through the red labelled nodes.
The dark blue edges are less common, but still legal, paths.  I've used a dotted line to indicate a shortcut to free and a dashed for the rest.
The XOF only path is black and dashed.  I could use another colour here perhaps.
The light blue nodes represent the beginning and end of the lifecycle.

What are people's thoughts about this approach?  Suggestions etc welcome.

In the same vein, MAC and RAND.

IMO I would not add the dotted _free lines as they should be just implicit and obvious (or just add it as remark to the freed state).
Otherwise this is looking good in general sense. I am not sure though the EVP_MD_CTX diagram is correct though in regards what the current implementation is doing. In particular the EVP_MD_CTX_reset() puts the ctx into the new state and not initialised. Also you should be able to do Init from any state (except for freed of course).

The init from any state should IMO apply to EVP_MAC_CTX as well.

I'll fix the EVP_MD_CTX_reset() lines, rushing too quickly through the code without thinking through the implications.
Happy to drop the _free lines, that will simplify the diagrams a little -- maybe I could add a "any other state" box with the _free calls....

Getting rid of the _free lines simplifies things.

The tripath cipher.

I appreciate the pretty pictures but how does this help 3.0 get done?

I need to clarify my comment because it came across badly.  I think the pictures are cool.  They are clear and legible. I am just wondering how this affects 3.0?  That is an honest question.

Formal description of the state machine can be used to forbid the bad order of the operations

What @beldmit wrote.  This sprang from the bug in #13210 and the OTC wanted it looked into.

KDF

Updated digest that splits XOF and non-XOF better.

A PKEY overview.  The intention is to dig into one operation in more details.

I'm not sure we should allow the transition between Update and Init.

I'm not sure we should allow the transition between Update and Init.

Why not? It seems harmless. Basically you should be able to reinit your context at any point. And 1.1.1 certainly allows this.
Also you can run init from the init state as well. That is currently missing.

I'm not sure we should allow the transition between Update and Init.

Why not? It seems harmless. Basically you should be able to reinit your context at any point. And 1.1.1 certainly allows this.
Also you can run init from the init state as well. That is currently missing.

I don't see any scenario to allow it.

Why not? You can set up a context for an operation, init it and then at any point later by some external cause you can decide that the operation is not to be finished and it needs to be restarted again fresh. I do not see why you would have to start with a new context in such case when if you finalize it, you'd be allowed to reuse the context. It simply does not make sense to me to arbitrarily limit the possibilities to reinitialize the context.
Of course another option would be not allowing reinitialization at all or only via reset->init, but that would be a breaking change.

Personally I'm afraid that some implementations will not be accurate enough to cleanup in this case. Reset and init seems to be safer.

The only algorithm I showed init going back to itself was ciphers where this is used.  Digests support it too but it's kind of pointless.  I left this out because it (needlessly) complicates thing IMO.  I'm kind of ambivalent about permitting it or not: I doubt anyone would reinit while in the init state.  Going back to init from any other state is reasonable.
The free calls can be done from anywhere, this is also not pictured.  It greatly simplified things.
For PKEY operations, any of the inits can be called pretty much anytime.  I planned on including this in the single operation diagram, it was pointless in the combined one (as was the free call).

I do not really see why init from init complicates things when you still allow init from any other state.
As for the @beldmit worry about the cleanup - that means we should worry about cleanup when calling init from any other state except reset and that means allowing only reset from any state and that means breaking change.

It messed the pretty pictures up :)

You could do something like "any state except freed" and line with init from there.

And similarly for reset

Look ok to me Pauli.
ccm and gcm follow the same kind of pattern - the complexity is in the places WHERE params need to be setup for ivlen and tag depending on encryption or decryption

I had those bits in at one point.  For encryption it was okay, the tag is grabbed from the final state.  For decryption, the tag can be set in too many places for nice output.

I've created a spreadsheet with fuller state transitions.  The spreadsheet should be considered canonical with the diagrams begin illustrative only.
lifecycles.xlsx


feat: Inverses for TrivSqZeroExt

Defined inverses for TrivEqExtZero in a way that is consitent with DualNumbers.
Note that $(a + b\epsilon)^{-1} = \frac{1(a - b\epsilon)}{(a + b\epsilon)(a - b\epsilon)} = \frac{a - b\epsilon}{a^2}$
Which becomes $\frac{1}{a} - \frac{b}{a^2}\epsilon$.
We want to be able have left multiplicative inverses $x x^{-1} = 0$
So we write $\frac{b}{a^2} = a^{-1} \cdot b \cdot a^{-1}$
Also included a proof that $x \cdot x^{-1} = 1$ when $\text{fst } x \neq 0$

A few more handy lemmas would be:

(inl r)⁻¹= inl r⁻¹
(inr m)⁻¹= 0
eps ⁻¹= 0

I pushed some small cleanups and a few extra lemmas; this now looks ready to go, but probably should be signed off by another maintainer
maintainer merge


Error loading Settings

Hello
I see that when I launch Terminal app I get error
Failed to load settings
Line 114 column 4
missing "}"or object member name

Environment
Windows build number: [run `[Environment]::OSVersion` for powershell, or `ver` for cmd]
Windows Terminal version (if applicable):

Any other software?

Steps to reproduce

Expected behavior

Actual behavior

Okay, did you delete the } from the end of the file? Can you share your settings file? The error message suggests that there is something wrong on line 114.

@montyjain20 - what version of terminal are you using? I loaded the settings you sent to 1.4, 1.5 and latest development branch, and none of them returned an error (though a purple background in CMD was somewhat I was unprepared for).

1.5.3242.0 is the version i am using. even for non-preview released version i see same error. this started happening after latest update otherwise it was working fine. let me know anything else is needed

Well.. that's weird. It works for me on the same version. So I guess it is somehow related to the system.
Probably the team has better idea, but it can help to know:

What OS version are you running?
What languages are enabled?
Can you please attach the screenshot of the error

It could also help if you could help pinpointing the problem is by slicing the file content prior to line 114.
If possible, I would start with:

Removing the emojis from profiles (just wondering if there can be some weird unicode issue, though I see your file has BOM prepended)
If the issue still exists to remove profiles from the list one by one

Can you also generate a trace log of the crash following the steps in this comment? https://github.com/microsoft/terminal/issues/8358#issuecomment-731670909


[pre.14][solvers] SolverHandler IsHandTracked not working when set to Both Left+Right

Describe the bug
--When SolverHandler is set to Tracked Handedness Left+Right, HandConstraintPalm Up First Hand Detected and Last Hand lost Unity Event fields do not function.
--When there are two SolverHandler with Tracked Handedness set to Left and Right respectively, HandConstraintPalmUp  Last Hand Detected field does not consider the opposing hand.
To reproduce
--Add SolverHandler, HandConstraintPalmUp, HandBounds to a GameObject

Set a single instance of SolverHandler to Tracked Handedness Left+Right or Everything
Hook feedback to HandConstraintPalmUp First Handed Detected and Last Hand Detected Fields

OR

Add two instances the above component setup and set TrackedHandedness for one to Left and the other to Right
Hook feedback to HandConstraintPalm Up First Hand Detected and Last Hand Detected Fields for each solver

Expected behavior
--A single SolverHandler set to Left+Right or Everything should fire events correctly
--Last Hand Detected field on HandConstraintPalmUp should consider the opposing hand
Your setup (please complete the following information)

Unity Version 2021.3
MRTK Version 3.pre-14
MR OpenXR Plugin 1.7.0 (also tried 1.4.4, etc)

Target platform (please complete the following information)

Editor UWP w/Sim Hand
HL2

Additional context
In pre.12 this scenario also generated an Assertion Failed:
Microsoft.MixedReality.Toolkit.HandednessExtensions:ToXRNode (Microsoft.MixedReality.Toolkit.Handedness) (at Library/PackageCache/com.microsoft.mrtk.core@3.0.0-pre.12.18/Utilities/Extensions/HandednessExtensions.cs:93)

cc @Zee2  @RogPodge

^^Looks like this is where the issue stems from when set to Both

@srinjoym is this something needed for GA?

@srinjoym I chatted with @holomatt  about this.    There is an assumption that this issue stems from a limitation in XRI 2.2, and can be addressed by upgrading to XR1 2.3.    Can you help determine if there is a solution that doesn't require upgrading to XR1 2.2.  The issue for upgrading to XR1 2.3 is #11282

No repro after upgrading XRI to 2.3.  Please re-open if you encounter it again with XRI 2.3 installed.

Just tested on 3.0.0-pre.15 - this still reproduces. Setting SolverHandler to Both or Everything does not function. Still need to use two instances of the SolverHandler set to Left and Right respectively in order to implement hand menus for both hands.

After more testing with @AMollis  - we found that this issue is an inspector bug specific to selecting the "Everything" value. The "Both" value works great, but once the user changes the Enum away from the default (Both) it latches to "Everything" when selecting more than one hand, resulting in non-operation.

Hopefully this is fixed now :)


bug: discover endpoint depends on search extension to be active

https://github.com/project-zot/zot/blob/1afc5c8c3fc8f4b4016bcda807ef7166f2c44ff9/pkg/extensions/extension_search.go#L103-L108
This prevents the discovery of other extensions if search is not enabled.

Not a bug! search is the only extension exposed via HTTP and without that exposing discover API is not useful.


mirror pylink-square 1.3.0

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe/meta.yaml:

ℹ No valid build backend found for Python recipe for package pylink-square using pip. Python recipes using pip need to explicitly specify a build backend in the host section. If your recipe has built with only pip in the host section in the past, you likely should add setuptools to the host section of your recipe.
ℹ PyPI default URL is now pypi.org, and not pypi.io. You may want to update the default source url.

This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/12374493105. Examine the logs at this URL for more detail.

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-webservice.
I tried to rerender for you, but it looks like there was nothing to do.
This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/12376415502. Examine the logs at this URL for more detail.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.


OpenAI integration for CLI

Current behaviour :
Some systems may not be able to load the model quickly or fail to load it at all. As they are hardware intensive, this might result in the answer of the query being delivered very late and can be inaccurate as well.
Solution :
We need to integrate OpenAI model using OpenAI API thus if system LLM model fails to load or answers are incorrect/inaccurate user should be able to retireve the answer using OpenAI.
Tasks to be done :

[ ] OpenAI integration to CLI mode
[ ] OpenAI integration to UI mode

Expected behaviour :
Once OpenAI is integrated, user should be able use OpenAI using python main.py --model_type --openai for CLI and select OpenAI model in ui mode.

@neokd we need to add openai model in load_llm.py files right?

Yes


Using $(ENV_VARIABLE) misbehaves with Email credential fields

Node-RED 0.16.2, MacOS Sierra
In Email Node's credential fields the contents of $(ENVIRONMENT_VARIABLE) are displayed instead of displaying the name of the variable.
I have successfully used environment variables with $(VARIABLE_NAME) with the Change Node where it works as expected. The problem seems to affect (at least) credential fields.
See the picture for details:

The Server address field works as expected and displays $(VARIABLE_NAME)
The UserId field misbehaves and displays the content of the environment variable after deploy.

However, the environment variable works. If I change the env variable's content and restart Node-RED, I see changes reflected in the UI. But, if I change the value in the UI, I lose the reference to the environment variable.
This makes it not apparent to the admin that the UserId value is actually read from the environment. If Admin changes the value in the UI, it will be difficult to find out why the Node configuration no longer works as expected.

As a side-effect, whenever a new credential is added in any node, the previously configured environment variables are replaced with their values in flows_cred.json.

Would love to see this fixed.
My workaround: Don't use credentials, instead use env variables (and config nodes)


Error of size

Where to set it within the code ?
assert inputs.size() == target.size(), 'predict {} & target {} shape do not match'.format(inputs.size(), target.size())
AssertionError: predict torch.Size([8, 1, 224, 224]) & target torch.Size([8, 2, 224, 224]) shape do not match

Can you explain the details of your training? It seems like there was a category setting error

My images are grayscale. I ran swin-unet on that and it worked so I am using same on this code

What dataset did you train on, ISIC2017?

yes it is binary. do I need to change low, middle or high thing?

Set the number of categories to 1 and use torch. nn BCEWithLogitsLoss should be used for loss, and your label should be single channel.

what to add in swin_transuper.py? my training data is according to binary but this code is for different channels

I don't know what the details of your code are, this error represents a different number of channels when calculating the loss. Please carefully examine the changes in the channel dimension of the code

May be the pre-trained are for RGB

how to modify pre-train


How to use the JFrog Bot in a GitLab project built with the gradle wrapper?

How can I configure jfrogbot so that it uses the Gradle wrapper specified by our project instead of reverting back to the inexistent gradle binary? See the sample error I am receiving in my gitlab-ci output:
16:58:07 [Info] Running Frogbot "create-fix-pull-requests" command
16:58:07 [Info] Auditing /builds/pare/my-sandbox
16:58:07 [Info] Detected: gradle.
16:58:07 [Info] Running gradle...
16:58:07 [Info] The build-info-extractor jar is not cached locally. Downloading it now...
 You can set the repository from which this jar is downloaded. Read more about it at https://www.jfrog.com/confluence/display/CLI/CLI+for+JFrog+Artifactory#CLIforJFrogArtifactory-DownloadingtheMavenandGradleExtractorJARs
16:58:07 [Info] Downloading build-info-extractor from https://releases.jfrog.io/artifactory/oss-release-local/org/jfrog/buildinfo/build-info-extractor-gradle/4.28.2/build-info-extractor-gradle-4.28.2-uber.jar
16:58:07 [Error] 'gradle' audit command failed:
exec: "gradle": executable file not found in $PATH
Cleaning up project directory and file based variables
00:01
ERROR: Job failed: exit code 1

Hi @lmcpare and thank you for using Frogbot. Gradle is currently not supported in Frogbot. We are planning to add it in the near future, and I will update this issue as soon as it is added.

Thank you for the feedback, @omerzi .
As a follow-up question, is there a way that I can configure Frogbot to not attempt a build but instead to specify an existing buildName and a buildNumber?

Hello @lmcpare, Frogbot is a Git bot that scans your pull requests and repositories for security vulnerabilities.
You can scan pull requests when they are opened, and Git repositories following new commits. Its action is similar to the JFrog CLI audit command.
Anyhow, the JFrog CLI build-publish command can accomplish what you are asking, as shown below: https://www.jfrog.com/confluence/display/CLI/CLI+for+JFrog+Xray#CLIforJFrogXray-ScanningPublishedBuilds.
Check out https://github.com/marketplace/actions/setup-jfrog-cli for the instructions on how to use JFrog-CLI as a GitHub action
For clarification, following my previous comment, Frogbot does support Gradle project scan, but not Gradle wrapper.

Hi @omerzi ,
Yes, we already use the jfrog-cli for several purposes, including Xray scans. We were hoping to replace that with FrogBot to accomplish the same and more, e.g. to block a GitLab MR when the MR brings new vulnerabilities.
Seems that our ambition level with FrogBot was a bit too high for the moment ;-)
We'll probably end up borrowing only the parts of it that communicates with GitLab. Since all our components are built with Gradle (gradlew), we will try to hook the MR blocking part from within Gradle as well.
Cheers!

Hey @lmcpare, Frogbot v2.4.1 has been released with the Gradle wrapper support. Let me know if it works for you, thanks.


[Feature Request]: Stargazers Timeline Animation

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
I want to add a stargazers timeline animation, which is generated using the remotion library. To create a a rendering component that will be used to render the main.mp4 file, for the stargazers timeline animation.
Use Case
To be able to showcase the performance & journey of the project.
Benefits
No response
Add ScreenShots
designdeck-234.webm
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC-EXT contributor
[X] I'm a HACKTOBERFEST contributor
[X] I have starred the repository

@GAVINESHWAR Can you assign me this issue for Level 3

@yashksaini-coder, please proceed


Next 11 breaks the plugin

Next 11 now has built in image loaders that get in the way of this plugin. You can do something like this:
      cfg.module.rules[2] = {
        oneOf: [
          {
            test: /\.svg$/,
            use: ['@svgr/webpack'],
          },
          cfg.module.rules[2],
        ],
      };

You can also yarn add next@canary as this is fixed in the v11.0.1 release.


DSPs prepared for loading persisted models

Few changes that will enable loading models from serialized sources:

added IR snapshot - a small struct that will allow to capture current IR configuration, and then recover IR basing on it
added new method that allows to create DSP basing on directly passed json config instead of file path.

This is a first part of the solution for issue sdatkinson/NeuralAmpModelerPlugin#163 second part is here: https://github.com/sdatkinson/NeuralAmpModelerPlugin/pull/206

Hey @sdatkinson did u have a time to take a look at this one? It is now polished, tested well 🤞 , and introduces much less 'breaking changes'. Thanks!

Thanks! I'll hopefully have time to look this weekend

@sdatkinson Yeah i think i can marry it without my changes. Dropped them, and changed naming. Now should be fine!


Bluetooth: Controller: Fixes for aux scanner

This fixes issues in aux scanner:

proper termination of HCI reports for failed periodic advertisement train scan
crash when scanning on aux context release

@andrzej-kaczmarek are you going to take care about this PR?

@cvinayak This PR seems to be not relevant anymore. Can we close it?


[Gluten-196]Support native columnar to row on clickhosue backend

What changes were proposed in this pull request?
Support native columnar to row on clickhosue backend
How was this patch tested?
manual tests
close #196

LGTM.

Verified on Jenkins.

Thanks, @rui-mo .


[Feature Request]: Question concerning  Mobile Backgrounding Question

Summary
Hello Mobile Devs:
on Mobile platforms do we have executable code that runs in separate process space for BackGrounding?
As analogous to Windows Service? As Analogous to Unix systemctl daemons? meaning they run each and every time the system is started.
If the answer is yes:

Are they allowed to send visual alerts?
can they take it one step farther and launch an app
Does Shiny do this.

Mark Wardell
API Changes
just a question
Intended Use Case
just a question
Code of Conduct

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

Please use the discussions forum in the future and follow the code of conduct within this project.
These questions are better answered by looking at the OS documentation than this project.  The answer isn't a straight forward yes/no.  Background jobs in Shiny are not running constantly like a daemon.  They react to system events.

Yes - you can use local or push notifications
If the user clicks on the notification
Shiny support several system events as well as having packages for both local & push notifications


KNL-1329 - Missing auto.ddl defaults for asn.receive.notifications

Looks correct to me +1

Thanks, I didn't update hsqldb because we want to remove that anyway. Can verify this tomorrow.


Fix: focus effect not valid for the whole row

Before

After

👍 Thanks for the patch. Looks good. Will be merging and updating it on Webstore soon.


Print config CLI command

Based on the feedback https://github.com/ethersphere/bee/issues/508, it would be helpful to have a new cli command that would print a bee configuration in yaml format to show available parameters.
This can be achieved with viper AllSettings() method on command.config and marshalling it with yaml encoder just as it is done in viper internally, pretty straightforward.

@metacertain , if you already have a PR for this, could you please link it?


.edit and .say: ability to use these commands without pinging users or roles mentioned in the content

(see also https://github.com/TrashMonks/Ereshkigal/issues/25)
When running the edit or say commands with a text argument that mentions users or roles, posting the command message will ping those roles (followed by ereshkigal immediately deleting the command message). When ereshkigal herself posts the subsequent message containing that text argument in the case of say, it will also ping the mentioned users or roles.
To update the server-info channel, we need to post a message that mentions almost every role in the server. It would be best if there were a way to use say and edit on this message that did not result in pinging thousands of users, for server management reasons.

resolved by f959f93646bba0356f456dcd617696c224d62e82


Slower performance of OpenBLAS for inverse operation on ARMv7

Hi All,
I have done some benchmarking for multiplication and inverse operations for ARMv7. I am using the armadillo c++ library which itself uses BLAS and LAPACK. The performance of OpenBLAS for multiplication is way better than the normal BLAS. Howevever, for inverse, armadillo makes use of LAPACK routines which in turn use BLAS implementations to compute the inverse using LU factorization. Using OpenBLAS leads to a slower performance compared to the normal unoptimized BLAS. At the first sight, this does not make sense because the LAPACK routine make use of BLAS for solving the LU factorization problem, or is it some extra overhead (vectorization) being incurred due to the particular nature of LU factorization. or something else is at play here?
I am using buildroot to cross compile the OpenBLAS for ARM A53.

Slow performance of (banded) LU factorization was reported in #901 but by all accounts appeared to be specific to Haswell systems running OSX. Could you run your test code on some other platform to see if it is generic or limited to the ARMv7 target, or could you provide a (small,self-contained) testcase ?

Thanks for the comment @martin-frbg . Well, i indeed ran my benchmarking code for Intel Haswel running CentOS, and experienced similar behaviour. However, for a higher matrix size, e.g., 1000, OpenBLAS was faster. This led to me conclude, in my first post, that some overhead of vectorization could be at play here. However, i could not replicate the same scenario on ARM; increasing the matrix size took longer for OpenBLAS.

Does it work faster with single thread than many?

Did you build OpenBLAS with the included LAPACK or without (i.e. make NO_LAPACK=1) ? I found out the hard way today that the LAPACK compile step within OpenBLAS does not inherit the default "-O2" optimization level of its BLAS counterpart. So if one not does not set the FFLAGS environment variable or edit Makefile.system accordingly, one may end up with a set of LAPACK modules that perform noticably poorer than a regular "Release mode" build of the exact same sources from netlib...

@brada4 I only used a single thread. And it is the cgetrf_ cgerti_ routines of LAPACK that perform the inverse using LU factorization and slower.

@martin-frbg I indeed use NO_LAPACK=1 because my toolchain does not have a cross fortran compiler and build a CLAPACK package separately linking it with OpenBLAS manually.

Below is the test code. Unfortunately it is not self contained. To run it, one must have the armadillo library installed (which is fairly straightforward).
main.cpp
`#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#include <time.h>
#include <float.h>
#include <assert.h>
#include
#define ARMA_DONT_USE_WRAPPER
#include
/* Basic data types */
typedef signed char             tInt8;
typedef unsigned char           tUint8;
typedef short                   tInt16;
typedef unsigned short          tUint16;
typedef int                     tInt32;
typedef unsigned int            tUint32;
typedef long long               tInt64;
typedef unsigned long long      tUint64;
typedef unsigned char           tBoolean;
typedef signed int              tError;
typedef float                   tFloat;
typedef float _Complex          tCFloat;
typedef double                  tDouble;
static std::complex* Amatrixmem = NULL;
static std::complex* Bmatrixmem = NULL;
static std::complex* Cmatrixmem = NULL;
static std::complex* Dmatrixmem = NULL;
static std::complex* Ematrixmem = NULL;
using namespace arma;
//extern "C" void openblas_set_num_threads(int num_threads);
void init_matrixinverse(tUint32 N, tUint32 K){
int i = 0;
Dmatrixmem = new std::complex[NNK];
Ematrixmem = new std::complex[NNK];
for (i=0; i<K; i++) {
	cx_fmat D(Dmatrixmem+i*N*N, N, N, false, true);
	D.randu();

	cx_fmat E(Ematrixmem+i*N*N, N, N, false, true);
	E.fill(0);
}

}
void init_matrixmultiply(tUint32 N, tUint32 K){
int i = 0;
Amatrixmem = new std::complex[NNK];
Bmatrixmem = new std::complex[NNK];
Cmatrixmem = new std::complex[NNK];
for (i=0; i<K; i++) {
	cx_fmat A(Amatrixmem+i*N*N, N, N, false, true);
	A.randu();

	cx_fmat B(Bmatrixmem+i*N*N, N, N, false, true);
	B.randu();

	cx_fmat C(Cmatrixmem+i*N*N, N, N, false, true);
	C.fill(0);
}

}
void destroy() {
if(Amatrixmem != NULL) {
delete Amatrixmem;
}
if(Bmatrixmem != NULL) {
	delete Bmatrixmem;
}

if(Cmatrixmem != NULL) {
	delete Cmatrixmem;
}

if(Dmatrixmem != NULL) {
	delete Dmatrixmem;
}

if(Ematrixmem != NULL) {
	delete Ematrixmem;
}

}
void exec_matrixmultiply(tUint32 N, tUint32 K) {
int i = 0;
for (i=0; i<K; i++) {
cx_fmat A(Amatrixmem+iNN, N, N, false, true);
cx_fmat B(Bmatrixmem+iNN, N, N, false, true);
cx_fmat C(Cmatrixmem+iNN, N, N, false, true);
C = A * B;
}
}
void exec_matrixinverse(tUint32 N, tUint32 K) {
int i = 0;
for (i=0; i<K; i++) {
cx_fmat D(Dmatrixmem+iNN, N, N, false, true);
cx_fmat E(Ematrixmem+iNN, N, N, false, true);
E = D.i();
}
}
void display_matrixmultiply(tUint32 N, tUint32 K) {
int i = 0;
for (i=0; i<K; i++) {
cx_fmat A(Amatrixmem+iNN, N, N, false, true);
cx_fmat B(Bmatrixmem+iNN, N, N, false, true);
cx_fmat C(Cmatrixmem+iNN, N, N, false, true);
cout << C << endl << " = " << endl << A << endl << " * " << endl << B << endl;
}
}
void display_matrixinverse(tUint32 N, tUint32 K) {
int i = 0;
for (i=0; i<K; i++) {
cx_fmat D(Dmatrixmem+iNN, N, N, false, true);
cx_fmat E(Ematrixmem+iNN, N, N, false, true);
cout << E << endl << " = " << endl << D << endl << "^-1 " << endl ;
}
}
int main(int argc, char *argv[]) {
int i;
// command line parameters
tUint32 N,K = 0;
bool display = false;
bool matrixmult = false;
bool matrixinv = false;
int nthreads = 0;
clock_t start;
tFloat elapsed_time = 0;

// get command line parameters
if( argc == 1 )
{
	printf("Usage:\n\n");
	printf("--lines : specify the matrix row size\n");
	printf("--tones : specify the number of tones\n");
	printf("--display\n");
	printf("--inv\n");
	printf("--mul\n");
	printf("--nthreads\n");
	return 0;
}

for( i = 1; i < argc; i++ )
{

	if( !strcmp( argv[i], "--lines" ) )
	{
		N = atoi(argv[++i]);
	}
	else if( !strcmp( argv[i], "--tones" ) )
	{
		K = atoi(argv[++i]);
	}
	else if( !strcmp( argv[i], "--display" ) )
	{
		display = true;
	}
	else if( !strcmp( argv[i], "--mul" ) )
	{
		matrixmult = true;
	}
	else if( !strcmp( argv[i], "--inv" ) )
	{
		matrixinv = true;
	}
	else if ( !strcmp( argv[i], "--nthreads" ) )
	{
		nthreads = atoi(argv[++i]);
	}
}

if (nthreads != 0) {
//	openblas_set_num_threads(nthreads);
}

// check parameter consistency
if(N == 0){
	printf("No size specified! Use --size <number> to define the matrix size.\n");
	return 0;
}
if(K == 0){
	printf("No number of tones specified! Use --tones <number> to define the number of tones.\n");
	return 0;
}
if(!matrixinv && !matrixmult) {
	printf("no mult or inv specified!\n");
	return 0;
}

// initialize data structures

if(matrixmult) {
	init_matrixmultiply(N, K);

	start = clock();
	exec_matrixmultiply(N,K);
	elapsed_time = ((float)(clock()-start))/CLOCKS_PER_SEC;
	printf("Matrix multiply for %u lines and %u tones took %f seconds\n\n\n", N, K, elapsed_time);

	if(display){
		display_matrixmultiply(N,K);
	}
}

if(matrixinv) {
	init_matrixinverse(N,K);

	start = clock();
	exec_matrixinverse(N,K);
	elapsed_time = ((float)(clock()-start))/CLOCKS_PER_SEC;
	printf("Matrix inverse for %u lines and %u tones took %f seconds\n\n\n", N, K, elapsed_time);

	if(display){
		display_matrixinverse(N,K);
	}

}

// free memory
destroy();

return 0;

}`
Makefile
`CPPFLAGS += -DNO_PROTOTYPES=1 -DDEBUG=0 -O3 -mhard-float
LDLIBS += -llapack
LDLIBS += -lopenblas
LDLIBS += -lm
all: main
main: main.cpp
$(CXX) $(CFLAGS) $(CPPFLAGS) $(LDFLAGS) -o $@ $^ $(LDLIBS)
clean:
rm -f *.o main
.PHONY: all clean
`
Usage:
./main --lines 1000 --tones 1 --inv

Unfortunately i do not have a self contained test case. One has to have armadillo installed (which is fairly straightforward) to run it. Would that be possible for you @martin-frbg ?

OpenBLAS didn't use ARM Neon (SIMD) instruction for ARMv7. Therefore, the performance is not good.

@xianyi the performance is not good when computing the inverse. It is definitely good while doing multiplication. Are there any plans for supporting ARM Neon instructions in the future?

@xianyi Does OpenBLAS with ARMv8 target use ARM Neon instructions?

According to #854 the ARMv8 target optimization uses Advanced SIMD (NEON) instructions.

@martin-frbg I used the lapack that comes with OpenBLAS, and indeed, as you mentioned, it is worse than the standalone lapack. I tried passing -O2 opimization flags in the Makefile.system but it does not seem to work. Also i see that OPT variable in the make.inc correctly holds the -O2 optimization flag. How were you able to do it?

Just clone a fresh copy of develop branch.

That problem should be solved in the "develop" tree now, basically you need to add the "-O2" option on
the line that starts with "override FFLAGS" (around line 1000 in Makefile.system) so that it is seen by the
fortran compiler (and not only by the c compiler used when building blas)

@martin-frbg
https://github.com/xianyi/OpenBLAS/issues/1045#issuecomment-271972504 gave me an at least 30% speed up with  LAPACKE_dgels on atom cpu  with a 3x3 matrix. Big leap for me. :)
Thanks.

inversion/decomposition is at complexity above O(dimension^3) while multiplication is below that.
All the locality optimisations will only give linear speedup. Unless you make algorithmic breakthrough there is no chance it gets simpler.

@brada4
I believe you, but then it means that the multiplication within LAPACKE_dgels was so much slower before and got so much faster now?

That complexity comment was probably adressed to sohailshuaib who started this topic, but I guess it is largely irrelevant as the issue was not that the computation appears to be slower in general but slower when done with OpenBLAS.

@martin-frbg @brada4 @xianyi there is some further update regarding the topic. I have done some profiling of using lapack with OpenBLAS vs netlib-blas on an Intel Haswell architecture. At first sight, an apparent difference is the implementation level 2 routine (cgeru) with level 1 routine (caxpy) which could probably be the cause of the bad performance or could there be something else at play here? I have tried perusing the code to look for a way not to have this transformation between level 1 and 2, but it seems impossible. I am attaching below the profile with the call graph for both OpenBLAS and netlib-blas. I did not use the lapack that comes along with OpenBLAS. Any thoughts or ideas are welcome.
 (Does performance improve when you limit the number of threads created to 2 or even just 1 by setting the environment variable OPENBLAS_NUM_THREADS accordingly ?)

Well, i am using a single thread only, to keep a comparison with netlib-blas valid. The size of matrix i am trying to invert is 48, and i typically perform this action a 1024 times. Are there any possibilities to backport a particular routine to its native implementation?
On Tuesday, 17 January 2017, 20:11, Martin Kroeker<EMAIL_ADDRESS>wrote:

I am not sure if I can provide much insight here, but how exactly are you testing - what argument sizes, how many threads ? (Does performance improve when you limit the number of threads created to 2 or even just 1 by setting the environment variable OPENBLAS_NUM_THREADS accordingly ?)—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.

I guess it would be possible to link the netlib cgeru (or equivalent c code) into your application so that the OpenBLAS implementation is not called, or alternatively to add netlib's straightforward algorithm to the function(s) under the CNAME alias in ger_thread.c so that it gets used in place of the more elaborate code whenever lda is small. (I seem to remember something similar being discussed in earlier issues where the OpenBLAS implementation is overkill for tiny matrix sizes)

Thanks @martin-frbg for the suggestions. However, being a newbie to openblas, i am finding it difficult to wade through the apparently complex implementation of openblas. I have located the aliases you mentioned and put a few printfs around in the code to have a feeling of the code flow. It compiles successfully, but does not print anything. I have tried, but could not figure out if printfs are being re routed or suppressed altogether by openblas? Is there some other documentation i could consult to have an understanding of the code structure/design and code flow of openblas?

@sohailshuaib Are you building for ARMv8 ? The topic says ARMv7 but in the comment you mentioned A53. This is contradicting.
If you are indeed building for ARMv8, can you please try building with make option "TARGET=CORTEXA57"

@ashwinyes it is possible to build for ARMv7 on a A53 device. The discussion has evolved to be rather independent of ARMvx and relates to lower performance of openblas when used for certain operations (e.g. matrix inverse)

@martin-frbg i can corroborate your thoughts on openblas being an overkill for tiny matrices. I was able to confirm this by profiling for larger (on the order of 1000) sized matrices.

@sohailshuaib Thanks for clarifying.
The reason I asked is because I saw the following question in one of the comments.

Does OpenBLAS with ARMv8 target use ARM Neon instructions?

To clarify further, "TARGET=ARMV8" uses default C implementations for majority of the APIs.
However "TARGET=CORTEXA57" uses the assembly implementations which has the ARMv8 SIMD instructions.
Anyways, my suggestion doesn't look relevant to this discussion. Hence please ignore it.

@ashwinyes thanks for the input. Just for the record, I, indeed, tried your suggestion, but to no effect.

@sohailshuaib the only documentation I am aware of is https://github.com/xianyi/OpenBLAS/wiki/Developer-manual which at least provides a quick overview of the code layout. And as far as I know, fprintf(stderr,...) should work everywhere (though with all the CNAME substitution it is sometimes helpful to consult the local Makefile to find which file gets compiled into which function under the influence of which defines or un-defines).

Thanks @martin-frbg , the developer manual had good hints.

@sohailshuaib One more suggestion if you have not already tried it.
The profiling dat  that you shared in file libOpenblasLapackWithLibraryDebugEnabled.txt shows that blas_thread_server is taking up the majority of time.
You can try by completely disable threading in OpenBLAS by using make option "USE_THREAD=0".

trying to replace a level 2 implementation of openblas with its netlib-blas (using clapack) counterpart has led me to a confusion with regards to the implementation of complex data in openblas routines. For example CGERU (zger.c) expects FLOAT which in itself is mapped to the primitive float instead of a complex datatype as in netlib-blas. Is it the case or am i missing something?

Without looking at the code I would suspect that the array is treated as paired real and imaginary parts internally ?

i could observe some improvements after using netlib-blas implementation of some routines e.g cgeru etc. In the meanwhile i also tried the suggestion of @ashwinyes which turned out to be the real deal breaker. Using this option, i can report that openblas performed better than netlib-blas for matrix inverse. I need to see in detail why this could have been a bottleneck earlier, and also test it on ARM. Currently i tested this on Intel Haswell.

Good to know that this worked. Using threads for small problem sizes is obviously an overhead.
However, setting OPENBLAS_NUM_THREADS=1 with multithreaded OpenBLAS library also should have seen similar performance improvemnet

Shouldn't CGERU (zger.c) use a single thread for small problem sizes automatically, thanks to jeromerobert's work on #731 ? Perhaps there is another similar bottleneck elsewhere.

Without code sample I cannot guess which of dozen LU factorisation functions to approach....
@ashwinyes can you run perf record yourprogram ; perf report -U > profile.txt and attach file?

So the fix was to not to build openblas in a multithreaded configuration or enforce to use a single thread at runtime by setting OPENBLAS_NUM_THREADS=1. Essentially, creating and managing threads for small problems introduces additional overheads which far outweighs the gains of openblas. With this lesson, the issue can be closed now. Thanks to everyone who shared their suggestions and helping remarks :).


update for discount manager enhancement

Coverage increased (+0.001%) to 34.34% when pulling cf81b5794f6ae700299be91a8c4ef5a928e44928 on AchmadSetiaji:dev into 7db52a1a36f53845390a2985c746a03beceab622 on danliris:dev.


Windows Terminal crashes when changing the system dark/light mode

Windows Terminal version
1.17.1023
Windows build number
10.0.19044.2364
Other Software
No response
Steps to reproduce

Start Windows Terminal
Open the Settings UI
Navigate to the Appearance tab
Set the Theme to "Use Windows theme" and Save
Leave the Settings UI open
Open the Windows settings app
Navigate to the Personalization > Colors section
Switch the color scheme from Dark to Light (or vice versa)

Expected Behavior
Windows Terminal should update to match the system color scheme.
Actual Behavior
Windows Terminal crashes.
When running from within the VS debugger, I get an exception like this:

Unhandled exception at 0x00007FFBBDCEFE32 (KernelBase.dll) in WindowsTerminal.exe: 0xC000027B: An application-internal exception has occurred (parameters: 0x0000024A0FCC6090, 0x000000000000000C).

And a stack trace like this:
>	KernelBase.dll!RaiseFailFastException()	Unknown
 	combase.dll!RoFailFastWithErrorContextInternal2(HRESULT hrError, unsigned long cStowedExceptions, _STOWED_EXCEPTION_INFORMATION_V2 * * aStowedExceptionPointers) Line 1455	C++
 	Windows.UI.Xaml.dll!DirectUI::ErrorHelper::ProcessUnhandledError(DirectUI::ErrorInfo & errorInfo, unsigned int fSkipFailFastIfNoErrorContext, unsigned int * pfHandled) Line 616	C++
 	Windows.UI.Xaml.dll!DirectUI::ErrorHelper::ReportUnhandledError(HRESULT hrError) Line 502	C++
 	Windows.UI.Xaml.dll!CXcpDispatcher::Tick() Line 1455	C++
 	Windows.UI.Xaml.dll!CXcpDispatcher::OnReentrancyProtectedWindowMessage(HWND__ * msg, unsigned int lParam, unsigned __int64) Line 1041	C++
 	[Inline Frame] Windows.UI.Xaml.dll!CXcpDispatcher::ProcessMessage(HWND__ *) Line 890	C++
 	Windows.UI.Xaml.dll!CXcpDispatcher::WindowProc(HWND__ * hwnd, unsigned int msg, unsigned __int64 wParam, __int64 lParam) Line 839	C++
 	Windows.UI.Xaml.dll!CDeferredInvoke::DispatchQueuedMessage(bool * dispatchedWork, bool * hasMoreWork) Line 301	C++
 	[Inline Frame] Windows.UI.Xaml.dll!CXcpDispatcher::MessageTimerCallback() Line 1534	C++
 	Windows.UI.Xaml.dll!CXcpDispatcher::MessageTimerCallbackStatic(void * myUserData) Line 1526	C++
 	CoreMessaging.dll!Microsoft__CoreUI__Dispatch__TimeoutHandler$CallbackThunk(class System::Delegate *)	Unknown
 	CoreMessaging.dll!Microsoft::CoreUI::Dispatch::TimeoutManager::Callback_OnDispatch()	Unknown
 	CoreMessaging.dll!Microsoft::CoreUI::Dispatch::EventLoop::Callback_RunCoreLoop(struct Microsoft::CoreUI::Dispatch::RunMode,bool,bool &)	Unknown
 	CoreMessaging.dll!Microsoft::CoreUI::Dispatch::UserAdapter::OnUserDispatch()	Unknown
 	CoreMessaging.dll!Microsoft::CoreUI::Dispatch::UserAdapter_DoWork()	Unknown
 	CoreMessaging.dll!Microsoft::CoreUI::Dispatch::UserAdapter_WindowProc(struct HWND__ *,unsigned int,unsigned __int64,__int64)	Unknown
 	user32.dll!UserCallWinProcCheckWow()	Unknown
 	user32.dll!DispatchClientMessage()	Unknown
 	user32.dll!__fnDWORD()	Unknown
 	ntdll.dll!KiUserCallbackDispatcherContinue()	Unknown
 	win32u.dll!NtUserGetMessage()	Unknown
 	user32.dll!GetMessageW()	Unknown
 	WindowsTerminal.exe!wWinMain(HINSTANCE__ * __formal, HINSTANCE__ * __formal, wchar_t * __formal, int __formal) Line 149	C++
 	[External Code]

Hmmmmmmmmm I can't seem to repro this on main, nor can I on 1.18.833 (which predated the MUX 2.8 changes). Maybe this was a win10-specific bug?

Woah okay yea, on win10, it did ~blast off into space~ crash. That's wild!
Other observations:

~If I don't have the SUI open, we don't crash, but we also don't update the titlebar color. And things that seemingly should update the RequestedTheme (opening tabs, opening About) don't seem to. That's weird.~
If the SUI is open, even on the main page, then the terminal crashes.
Interesting, switching the OS theme another time did cause the Terminal to hang. Loading dump now...

The fact that this is something that can be fixed by an OS update is... troubling...

BOY I don't know exactly how or why, but this is fixed in 1.18, even on windows 10. Just installing 1.18.833 on 19044 works like a charm. Probably due to restructuring how we determine if there's a quake window. I know that sounds insane.

Can confirm that it's working for me now in the dev build. Thanks.


DEBUG timeout

Signed-off-by: Matthew B White<EMAIL_ADDRESS>

Would you mind updating the commit message/Pr title to be a bit more explanatory too please?


hack/publish-ocp.sh fix cut syntax

The current syntax doesn't work, at least on coreutils 8.32 that's
shipped with Fedora 35

/lgtm
/approve
/retest-required

/retest-required

/retest-required

/retest-required


Clone

Method (literally or otherwise) to clone a moment recur object.

You could use the .save() function, which generates an options object that can be fed back into a new recurrence.

The project was EonJS https://bitbucket.org/gso/eonjs/src
Not sure off the top of my head why this arose, possibly some sort of edit 'undo' feature.  If I wake the project up again the problem looks solved.
Cheers,


Pretty: Expr.Do

My idea is that render an Expr.Do as Single Line or Multi-Line (see below)
Single Line (all bindings in one line):
do { x <- xs, y <- ys, return (x + y) }

Multi-Line (one binding each line):
do {
  x <- xs,
  y <- ys,
  return (x + y)
}

I dislike (simply Doc.sep when lineRemaining is insufficient):
do { x <- xs, y <- ys,
  return (x + y)
}

Can we advise the PrettyPrinter to flat something even though the page width is enough?

Where the test should be? I placed them in ParseTest, but its name is not good (We have to prevent the doExpr from desugaring)

Ιs there DistillerTest or PrettyTest?

@HoshinoTented I merged your (another) PR and there are now conflicts. Can you resolve them?

I merged your PR into another branch

@HoshinoTented You can make it public


Suddenly: Error while installing/updating Homebridge

Hi,
suddenly installing/upgrading homebridge is giving me the following error/output:
npm install -g --unsafe-perm homebridge
/usr/local/bin/homebridge -> /usr/local/lib/node_modules/homebridge/bin/homebridge

<EMAIL_ADDRESS>install /usr/local/lib/node_modules/homebridge/node_modules/curve25519-n
node-gyp rebuild

make: Entering directory '/usr/local/lib/node_modules/homebridge/node_modules/curve25519-n/build'
CXX(target) Release/obj.target/curve/node_curve.o
as: symbol lookup error: /usr/lib/libbfd-2.25-system.so: undefined symbol: bfd_plug)n_arch
curve.target.mk:93: recipe for target 'Release/obj.target/curve/node_curve.o' failed
make: *** [Release/obj.target/curve/node_curve.o] Error 1
make: Leaving directory '/usr/local/lib/node_modules/homebridge/node_modules/curve25519-n/build'
gyp ERR! build error
gyp ERR! stack Error: make failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/usr/local/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:258:23)
gyp ERR! stack     at ChildProcess.emit (events.js:182:13)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:225:12)
gyp ERR! System Linux 4.9.35-v7+
gyp ERR! command "/usr/local/bin/node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /usr/local/lib/node_modules/homebridge/node_modules/curve25519-n
gyp ERR! node -v v10.0.0
gyp ERR! node-gyp -v v3.6.2
gyp ERR! not ok
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: node-gyp rebuild
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR! A complete log of this run can be found in:
npm ERR!     /root/.npm/_logs/2018-05-11T10_57_42_574Z-debug.log
I'm running on rasping jessie und my homebridge was running fine until current update. How can I get it back working? I think the error is related to build essentials and/or one of its dependencies. Am I right? Or do I misinterpret something?
Any directions what I can do would be highly appreciated.
Thank you!

After updating node (to another major version), uninstall homebridge and re-install it:
$ sudo npm -g un homebridge
$ sudo npm -g i homebridge --unsafe-perm

Hi @ebaauw,
thanks for your hints. Updating node and reinstalling results in the same error. Sth. breaks during update in that way that getting back to previous version doesn't work too. :-(

You need to uninstall homebridge before re-installing it.  Somehow node-gyp rebuild fails on simply updating homebridge (installing homebridge over an existing installation).

I DID the uninstall. :-(

Bummer.  I'm afraid I don't have any other hints.

Nevertheless thanks for your help. Currently I'm restoring my Image from 2018-05-01 to get online again. I gave it a second chance but ran in the same error. So currently upgrading to latest homebridge is not possible to me. :'(

Thanks! That worked for me.
I played with gulp yesterday, and today found out that homebridge stopped working.
Reinstalling showed the same error.
But performing uninstall THEN install fixed it.
Thx

Nothing helped than reinstalling from scratch. Strange issue, but now I‘m running again with latest homebridge version.

Not to necro-post, but in case it helps others: i had the same issue as dehsgr and it turned out i was missing the —unssafe-perm switch. i.e. after updating node i had to run
sudo npm uninstall -g homebridge
sudo npm install -g —unsafe-perm homebridge


Deprecated links

Removing deprecated Taito refrences. The following files still contain some Taito stuff. Leave a comment if there is a good reason not to remove it

[ ] apps/trinity.md

[ ] apps/blast.md

[ ] apps/force.md

[ ] apps/grid_blast.md

[ ] cloud/fgci/fgci-using-arcrunner-to-run-large-job-sets.md

[ ] cloud/pouta/pouta-videos.md

[ ] data/kaivos/kaivos_batch_job.md

[ ] data/Allas/index.md

[ ] data/Allas/using_allas/a_commands.md

[ ] data/Allas/hpc-archive.md

[ ] data/Allas/migration_tutorial.md

[ ] support/tutorials/index.md

[ ] support/tutorials/conda.md

[ ] support/tutorials/nomachine-usage.md

[ ] support/tutorials/puhti_quick.md

[ ] support/tutorials/mahti_quick.md

[ ] support/faq/how-to-move-data-between-puhti-and-allas.md

[ ] support/faq/how-fast-is-the-pouta-network.md

keep Maestro Taito reference. I'll remove it once I get to update the video (which talks about Taito)


rm -rf node_modules

wait this isnt terminal

what am i doing

:wq

You’re using the wrong editor!

i drank the wrong amount of alcohol


Make Layout a class that can be passed to the logger.

Allows layouts to hold state.
Replace weird swap-based appender logic with reusable mixin.

Switched back to the previous way of doing the appender. I thought about it wrong. Static is fine.


Conditional log output?

Hi, I have a rule in Snakemake that doesn't have any wildcards. Other rules have a wildcards.tissue_name wildcard. When I use the below config.yaml configuration to submit jobs to slurm, I get an error that says the following
RuleException in rule get_screen_genomes in file /lustre/work/helikarlab/joshl/snakemake/Snakefile, line 346:
AttributeError: 'Wildcards' object has no attribute 'tissue_name', when formatting the following:
mkdir -p logs/{rule}/{wildcards.tissue_name} && sbatch --job-name=smk-{rule}-{wildcards} --account=helikarlab --cpus-per-task={threads} --output=logs/{rule}/{wildcards.tissue_name}/{rule}-{wildcards}.out --mem={resources.mem_mb} --time={resources.runtime} --parsable

Is there any way to set a "conditional" --output in the sbatch section? I see the conditional and dynamic resources examples, but it wasn't for the sbatch section
Thanks for any help!
This is my config.yaml
cluster:
  mkdir -p logs/{rule}/{wildcards.tissue_name} &&
  sbatch
    --job-name=smk-{rule}-{wildcards}
    --account=helikarlab
    --cpus-per-task={threads}
    --output=logs/{rule}/{wildcards.tissue_name}/{rule}-{wildcards}.out
    --mem={resources.mem_mb}
    --time={resources.runtime}
    --parsable

cluster-cancel: scancel
cluster-cancel-nargs: 50
restart-times: 0
max-jobs-per-second: 10
max-status-checks-per-second: 1
local-cores: 1
latency-wait: 60
jobs: 100
printshellcmds: True
scheduler: greedy
use-conda: True
conda-frontend: mamba

Sorry for a late response, this is the earliest I've been able to work on this project. Your suggestion works great!
I only have 1 rule that does not have a "tissue_name" wildcard, so I've set the default resources as such
default-resources:
  - tissue_name={wildcards.tissue_name}

And then in the rule that doesn't have a tissue name, I've set the resources as such
resources:
	tissue_name=""

Thank you for your help!
For anyone referencing this in the future, this is my final configuration file
cluster:
  mkdir -p logs/{rule}/{resources.tissue_name} &&
  sbatch
    --job-name=smk-{rule}-{wildcards}
    --account=helikarlab
    --cpus-per-task={threads}
    --output=logs/{rule}/{resources.tissue_name}/{rule}-{wildcards}.out
    --mem={resources.mem_mb}
    --time={resources.runtime}
    --parsable

default-resources:
  - tissue_name={wildcards.tissue_name}

cluster-cancel: scancel
cluster-cancel-nargs: 50
restart-times: 0
max-jobs-per-second: 10
max-status-checks-per-second: 1
local-cores: 1
latency-wait: 60
jobs: 100

printshellcmds: True
scheduler: greedy

use-conda: True
conda-frontend: mamba

Sorry for a late response, this is the earliest I've been able to work on this project. Your suggestion works great!

@JoshLoecker I'm glad you found a solution! And thank you very much for sharing here for others to learn

I only have 1 rule that does not have a "tissue_name" wildcard, so I've set the default resources as such
default-resources:
  - tissue_name={wildcards.tissue_name}

Ah, very cool approach. I don't believe I've ever tried to reference a specific wildcard value directly in the YAML, but in retrospect I guess it makes sense that this works


cmd/asm: integer comparison is always false

cmd/asm/internal/asm/parse.go:817: comparison of identifier shift < 0 is always false
shift is a uint64, and can never be negative.  The check and error is unreachable code.
func (p *Parser) term() uint64 {
        value := p.factor()
        for {

...
                case lex.RSH:
                        p.next()
                        shift := p.term()
                        if shift < 0 {
                                p.errorf("negative right shift %d", shift)
                        }

The fix should have been to change shift < 0 to int64(shift) < 0.

Replied to CL.

I believe this is finally fixed. If not, please don't make any more changes before Go 1.5 is released.

It is fixed.


Question: Howto add custom command line options on v2.0.0

Hello,
I just updated to plugin version 2.0.0 while using Ansible <IP_ADDRESS> on Ubuntu 16.04.1.
In v1.4.0 there was an option to add custom command line options to an Playbook executed in a workflow step. This seems to be gone.
Is there a replacement or may it come back?
Thank you for your time and work
Jard

Hi @jardleex ,
Is there a specific option that you would like to add ? I believe most of the options provided in ansible are covered.
The whole idea behind adding all the option fields is to not have to add the options manually, and make some options like passwords and public key benefit from the functionalities offered by rundeck (like secure options and storage path ...) so adding the the old Extra Options fields will just make it more confusing, since the managed options can interfere with the ones defined manually.
Thanks,

Hello @yassineazzouz,
thanks for your fast reply.
I'd like to use --tags for example. Also playing around with --limit to find out how to couple with our existing roles and site.yml. A -vvv might be also nice sometimes.
Thank you.

So for --limit : there are a couple of ways you can do this:
1- In your Job target hosts : the nodes you select as your job target hosts, will be used to limit the execution.
2- In your project configuration : you can use project.ansible-limit in your project configuration file to limit   the jobs execution (at the project level)
Concerning -vvvv you can run the jobs on Debug output, or you can also use project.ansible-debug to have debug output enabled at the project level
--tags is not included for now, but can easily be added :)

--limit - SOLVED
Agreed. I played around with dispatch and was able to fulfill my needs. The other options at project or job level are good to know but dispatch is sufficient for now.
-vvv- WISHED
I tried project.ansible-debug but aint got the known Ansible verbose output in /var/log/rundeck/service.log.
May it possible to add another option at a workflog/job level to optionally enable the debug output? Maybe a selector from 0-4 or similar.
--tags- WISHED
Having this would be awesome. Our site.yml will grow in the next time and I'd like to be able to run only parts from it. On ~600 targets, this might save time.

Thos two would be great to:
--check - WISHED
--diff - WISHED
When running playbooks via the Rundeck sheduler they would enable the user to see possible changes if the comitter ain't run the playbook after his change.
Some may also use --list-(hosts|tags|tasks) for debugging purposes as well.
As you see a custom command line options field wouldn't be that bad ;-).

Hi @jardleex ,
Concerning the other options I will check a way to have a custom ansible options field, I agree that it is easier to just have a "custom attributes" field than having every single cmd option in one separate option, although I still think that it could be a bit confusing to have some options managed automatically by the plugin and some others set in a separate field, plus typically rundeck is used for jobs that run in a production like environment, more for automating operational processes, not really for for debugging scripts.
Concerning the debug output please use the job Log Level (screenshot below)

Thanks,

Hi @yassineazzouz,
I can understand the conflict I brought you in. Each dedicated option would need to check if it was also given in the custom options field. Or maybe Ansible takes care of this itself? E.g. having multiple --tags or --check may simply get merged by Ansible.
In version 1.4 I used dispatch and --limit at the same time to find out how the Plugin + Rundeck + Ansible work together. I'm pretty sure that my custom option has overwritten my dispatch selection (which is I think also only uses --limit).
You're right that the usage of Rundeck+Ansible is more of a daily use then for debugging purposes. Just wanted to mention it so in certain cases users may want to do debugging here. But thats not top prio.
Having the --tags would be much greater :)
Regarding the debug output:
You're right, now I got my verbose Ansible output in Rundeck. My assumption was that setting just project.ansible-debug would be sufficient was wrong. Thank's for the hint.
Best regards
Jard

Hi @jardleex ,
Check release 2.0.1, should include what you are looking for.
Thanks,

Works like a charm.
Thank you!


Determine if a line is a FoldPoint, and FoldState

Will this be a feature that will be added in the future?

I committed 21754eae31afe1fe90deac0ae9a1b91d8e56936f which should have what you need. There is a lot in the commit to sift through, but the properties you're probably looking for are Line.FoldLevel and Line.FoldLevelFlags. The first controls the nesting level of the line and the second whether the line is a fold point.

What value for FoldLevel determines if it is folded, or unfolded? Is it unfolded if it is 1024?

The FoldLevel sets the nesting of the line and any subsequent lines so that they are considered grouped (or children). The first line in that sequence must have the FoldLevelFlags.Header flag set. It doesn't get set for you; you must set it. My guess is that you have not set that flag on the fold point line.
P.S. This is all only necessary if you are creating your own folding / lexer logic. If you are looking to get folding working for one of the included lexers, see the Automatic Code Folding wiki article.


Entity tag for other pages

Some Entity tag functionality is strictly linked to main page (open in detail)....

@adammertel if we don't need to use it in other way, we could just disable dbl click there..?


Milestone1:TODO List

TODO List: List Structure Module
In this Milestone, I:
✅ Created the module to display the to do lists
✅ Added styling to the to do list app

Kindly assist me with any idea of fixing lighthouse error se,I have tried all methods suggested by my peers but still not working.


Fields don't get disposed properly when outside directories

To reproduce:

make a component that can be mounted/unmounted
add a tweak in the component
if the tweak is in a directory, it will be disposed when unmounting, otherwise it will be re-created

Should be fixed in #26


⚠ Spryr has degraded performance

In b5fa4e3, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 3398 ms

Resolved: Spryr performance has improved in fc257d4 after 7 minutes.


[BUG][Functional]  Introduce some cleanup and optimize no of iam roles

ASEA should embrace reusability of iam roles and should cleanup roles required for accelerator run.
Required Basic Info

Accelerator Version:  1.26-a, main
Install Type: First clean then pointed ot main branch
Upgrade from version: 1.26-a to main

Describe the bug
In my operation account, I endup with over 90 Admin access roles like these
arn:aws:iam::xx:role/cdk-nsl-cfn-exec-role-xx-ap-northeast-1.
This happened as a side effect of  https://github.com/aws-samples/aws-secure-environment-accelerator/issues/660
I recommend if we are going to create multiple roles with have same access, then why not reuse the existing PipelineRole with added trust permissions. Infact all the ASEA provisioned accounts have more than 35-40 iam roles.
Failure Info
No failures, expected
Required files
https://gist.github.com/rverma-nsl/b3e390d4430972eff955c11e9bfd4e10
Steps To Reproduce
Expected behavior
Limited reusable roles. Even better if we can have more human friendly names sk
Screenshots

Hi rverma,
NOTE: main branch is always a work in progress.  We do not support customer installs off the main branch, and encourage customers to stick with official releases.  Just a friendly warning to proceed at your own risk :-).  Strongly encourage you using the tested and supported code releases :-).
I believe you are reporting a couple of items on this ticket:

AWS best practice actually mandates that we use tightly scoped and unique roles for every purpose, hence the multitude of roles which exist in each AWS account.  This is actually a good thing, as it means the scope of any potential vulnerability would be limited and prevent lateral movement;
The deployed roles are used by both the Accelerator state machine on each execution, by enabled AWS services in the account(s) including config rules and SSM remediation documents, we do not believe it would make sense to cleanup the roles;
I would agree with you, the quantity of cdk roles is not ideal, but also not a problem.  The new AWS CDK installer deploys 4 roles per region and their is nothing we can easily do about this.  What we did do when we moved from the legacy installer (in v1.2.6-a) to the new installer in the main branch was centralize all the CDK buckets and roles in a single central account.  Had we not done this you would have had those 68 roles in every account, instead they only exist in the central-operation-services account.  The role names are also largely controlled by CDK;
I accepted your other bug report and we will research the failure to cleanup 'anything' when we stop 'managing' a region.  In the meantime, you can manual remove any stacks in reverse order, from the regions you wanted un-managed regions (Phase5, then Phase 4, ...) and this should cleanup the extra roles for the regions for which you do not want to secure

Hopefully this helps clarify!

Hi Brian,
Just a note on this, even if you look at control tower implementation, they also have a single role for managing control tower operations. The whole process can be justified if you consider ASEA as a single entity.
Infact if we are using tightly scoped role the whole proposal got invalidated. It actually come from the observation that we are having multitude of roles having AdministratorAccess, I would rather have 1 or 2 Roles having admin Access, few with poweruseraccess and unlimited with scoped access. I am afraid of if an auditor deploy something like https://github.com/salesforce/policy_sentry and identified that we have so many over-provisioned roles.

Hi,
Unfortunately, your suggestion violates our security practices (and is therefore not something we will consider).  It is important to note that the roles you keep referencing are only usable by the CloudFormation Service, all the other tightly scoped roles are used by other services like custom resource Lambda's - considerably different use cases.
As previously stated, I cannot change the way those CDK* roles work/are created/deployed as this is the way the CDK was designed and functions.  If you have an enhancement suggestion for the CDK team, I'm going to suggest you pursue your enhancement requests directly with them.  If they decide to change the behavior, please let us know so we can incorporate when available.
Sorry, we couldn't be of more help.


Create marketplace entry for tWAS base single server

Hello @git4rk Could you pls assign it to @edburns? Thanks.

Created

Offer Alias
Offer ID

2022-01-06-twas-single-server-base-image
2022-01-06-twas-base-single-server

@git4rk  Please close this issue.

Hello @edburns , I'd like to keep this issue open until the offer you created can be referenced in the tWAS single server offer as a workable image. What do you think?

The urn of the vm image is: ibm-alliance-usa-ny-armonk-hq-ibmstorage-6201192:ibm-svpc-azure-843-base-vm:svpc-8430-211209-2206:843.0.2112092206, closing the ticket now.


deleteFork cheatcode

Component
Forge
Describe the feature you would like
I'm currently working on a project where the integration test suite involves running multiple tests, each having its own forks pre-defined. It would be great to have a cheatcode to delete forks so we can call after each test execution.
Additional context
No response

could you provide some sample code for this?

We've developed a governance framework to create and simulate governance proposes. As part of our integration testing through CI, we have a test that verifies if any new proposals have been added in the current branch. If a new proposal exists, the proposal integration test is executed, which generates the proposal calldata and runs tests against mainnet fork. Because proposals can be cross-chain (this is achieved by relaying the calldata to the other chain via Wormhole VAA), we need to switch forks and test against all chains that the proposal interacts with.
function setUp() public {
        addresses = new Addresses();
        vm.makePersistent(address(addresses));

        // Get proposals added or changed in the current branch
        string[] memory inputs = new string[](1);
        inputs[0] = "./branch-proposals.sh";

        string memory output = string(vm.ffi(inputs));

        // Convert output to array of lines
        string[] memory lines = output.split("\n");

        if (lines.length > 0) {
            proposals = new Proposal[](lines.length);

            for (uint i = 0; i < lines.length; i++) {
                address proposal = deployCode(lines[i]);
                proposals[i] = Proposal(proposal);
                vm.makePersistent(proposal);
            }
        }
    }

    function testBranchProposals() public {
    ///As we iterate through proposals, it is crucial to delete forks after each execution since proposals can have one or more
   ///forks. Cross-chain proposals will have at least two forks, while chain-specific proposals won't switch forks. However,
   ///executing a cross-chain proposal first and then a chain-specific one causes Foundry to revert without a specific
   /// reason.
        for (uint i = 0; i < proposals.length; i++) {
            proposals[i].run();
        }
    }

Proposal script run function:
    function run() public virtual {
        // We start the executing with the primary fork, the internal functions
        // called here can switch to other forks if needed.
        vm.selectFork(primaryForkId());

        address deployerAddress = vm.addr(PRIVATE_KEY);

        console.log("deployerAddress: ", deployerAddress);

        vm.startBroadcast(PRIVATE_KEY);
        if (DO_DEPLOY) deploy(addresses, deployerAddress);
        if (DO_AFTER_DEPLOY) afterDeploy(addresses, deployerAddress);
        if (DO_AFTER_DEPLOY_SETUP) afterDeploySetup(addresses);
        vm.stopBroadcast();

        if (DO_TEARDOWN) teardown(addresses, deployerAddress);
        if (DO_BUILD) build(addresses);
        if (DO_RUN) run(addresses, deployerAddress);
        if (DO_VALIDATE) {
            validate(addresses, deployerAddress);
            console.log("Validation completed for proposal ", this.name());
        }
        if (DO_PRINT) {
            printCalldata(addresses);
            printProposalActionSteps();
            printAddressesChanges();
        }

        // Thats the feature we are looking for, we want to be able to drop all/selected forks
        vm.deleteForks();
    }

can't you just use && forge cache clean to remove everything after script is executed?
https://book.getfoundry.sh/reference/forge/forge-cache

Why do you need to delete the forks during execution of the forge script/test though? Is it a memory consumption issue?

@anajuliabit per comment above, is this due to a memory issue? An alternative would be to use vm.ffi() cheatcode
https://book.getfoundry.sh/forge/differential-ffi-testing#primer-the-ffi-cheatcode and just remove the forks, would this work for you?


MySQL Connection Pool

@jacobzelek Thanks for the Pull Request!
What do you think about re-using this connection pool library: https://github.com/ysbaddaden/pool ?
Additionally, it would be nice to have the public API for the connection pool functionality in the README.

@waterlink I think it would be nice to have a connection pool in this library. It eliminates the dependency.

I have seen in the source code of the library, that it is using an IO Pipe
Do you know what is the difference between that implementation and this one?
Small modular dependencies are actually a good thing in my honest opinion, especially when we have a good package manager.

I actually didn't look at the 2 yet, but after a quick review this is what I think is different:
PR:

uses Mutex and should be thread safe
it blocks the fiber when trying to checkout a connection and will do so until it can

Pool lib:

uses Fiber.current, so is safe for fibers but not thread safe yet
it uses the IO.pipe as a timeout mechanism, it will sleep the fiber until a connection becomes available or throws a timeout error

I think the blocking mutex in PR implementation is a deal breaker. So in this case I would not use it and prefer the Pool lib

Small modular dependencies are actually a good thing in my honest opinion, especially when we have a good package manager.

They sure can be. It's not really an issue at the moment but, if for instance one was to use a redis client and a mysql client and they both use a different connection pool library, you run it to problems if the pool libs both use the same names.
Another option might be to vendor the connection pool library and namespace it within the mysql lib.
Yet another option might be to wait for the DB class in the STDLIB, that might come with a connection pool.

@waterlink I'll try the pool lib and let you know what I find. I agree that Mutex isn't favorable.

Well I think the mutex will be required when threading is enabled, but waiting for the lock to free should be done in a non blocking way so you can throw timeouts like the pool lib does :)

Good point! I'll play around with that

@jacobzelek Hi! What is the current status on this? Did you try to play around with that?

@waterlink Not yet. Day job has had me busy for the last couple of weeks. When thing slow down I'll have a chance to try some more things out

Great! Thank you!
On Wed, Jun 15, 2016 at 11:44 AM Jacob Zelek<EMAIL_ADDRESS>wrote:

@waterlink https://github.com/waterlink Not yet. Day job has had me
busy for the last couple of weeks. When thing slow down I'll have a chance
to try some more things out
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/waterlink/crystal-mysql/pull/42#issuecomment-226139259,
or mute the thread
https://github.com/notifications/unsubscribe/AAlLFnt00uKkCCnlvqts_9BeW1PB4Cvgks5qL8l8gaJpZM4Hw2B7
.


Fix shrink_in_place

The current shrink_in_place implementation is incorrect. It returns Ok if the usable_size returned by xmallocx is >= new_size.
However, xmallocx returns the usable_size of the original allocation, which is >= new_size, if:

the allocation cannot be shrunk in place, e.g., because  new_size lands in a different size-class -  we must return error here since new_size cannot be used to deallocate the allocation but Alloc::shrink_in_place users are allowed to do that on success
new_size is in the same size-class as the original allocation - in which the allocation was shrunk appropriately by zero, and new_size can be used to deallocate the allocation since it lands in the same size-class

cc #64

+cc @interwq for the xmallocx question.

The xmallocx description looks right to me. Any questions or comments on the behavior / performance there? The only thought I have right now is, we can probably do a much better job at shrinking large allocations (across size classes, but stays large), e.g. shrink from 128k to 64k; does it seem like a common workload for you @gnzlbg?. Small size classes I don't think much can be done really.

@interwq since you asked: the new logic being proposed here returns Ok(()) if usable_size is any value less than layout.size().
The assumption that I believe the client is going to make is that if they get Ok(()) back, they can subsequently dealloc the resized block  passing its size as new_size
So, my question: is there any potential scenario where:

client requests new_size
xallocx returns usable_size, where new_size < usable_size && usable_size < layout.size(), and
it is not sound to use new_size as the associated size of the block when calling dealloc, and one must instead use usable_size to get correct deallocation behavior?

@pnkfelix : in the scenario you had, if new_size is not in the same size class as usable_size, then it won't be correct to use it for deallocation. The diff looks right to me, as it queries nallocx before returning ok.

I don’t think it calls nallocx for the Ok(()) on line 204
@gnzlbg it sounds to me like you might need to fix that?

(Maybe the easiest fix is to check usable_size == new_size there instead of usable_size < old_layout.size())

@interwq Given:

a pointer ptr to an allocation of size old_size, with an usable size of old_usable_size,
a request to shrink the allocation in place to new_size such that new_size < old_size && new_size_usable_size < old_usable_size,

we call some_usable_size = xmallocx(ptr, new_size, 0, flags).
Iff some_usable_size < old_usable_size, is some_usable_size == new_usable_size ?

Iff some_usable_size < old_size:

is some_usable_size < nallocx(old_size) ? I think so, this means the allocation was shrunk.

Yes.

is some_usable_size == nallocx(new_size) ? That is, if xmallocx returns a usable size that is smaller than that of the old allocation, can we interpret that as a success in shrinking the allocation to the size class of new_size such that the allocation can be deallocated via sized-deallocation by passing new_size?

Yes it will shrink to new_size, or not shrink at all. The manual doesn't really "enforce" this, but the current implementation works this way and I don't see us changing this. You can see the implementation detail here: https://github.com/jemalloc/jemalloc/blob/5e23f96dd4e4ff2847a85d44a01b66e4ed2da21f/src/large.c#L96
where we split the existing extent into two; to cut off the excess pages.

@interwq thank you for chiming in. @pnkfelix I think we can then land this as is ?

I certainly won’t block it landing. @alexcrichton is the one with review privileges here. 🙂


Compatibility for RN version < 0.40

fixes Duplicate RCTMethodInfo error
reference : https://github.com/yonahforst/react-native-permissions/issues/137

I guess this can be closed, as it is done in https://github.com/zhangtaii/react-native-google-place-picker/pull/31 and already on master.

thank you. i am closing it.


4K content

First of all -
I LOVE your work with this, I was only curious if you were planning on updating to support 4K content now they all the screensavers have been updated with the release of the 4K Apple TV?

This is already discussed heavily in #378

Closing as duplicate of #378 .


Build fix to allow use COPY --link

Fixes for https://github.com/jim60105/docker-whisperX/issues/25

Thank you for your PR, this is a good point.
My client is up to date so I didn't notice this matter.😅


Use StartupWMClass

I don't know if you can set that using NodeJS , but the desktop application should have the same name as the StartupWMClass.

The popover of the application name shows Chromuim-browser instead of Skype (due to the missing StartupWMClass)

Dude, it seems to be an issue of your desktop environment. In my DE it pops up the current URL location title.
Example:
Before you sign in:
"Sign in your Skype account"
After you did it, it pops up:
"Skype"

Yeah, i just found the issue on Node.js repo and it's only happens on Gnome (or if you use a dock as plank on other DE)

What DE are you using Kuch?

Gnome on Arch

Odd; I remember getting the same problem as bil, but on a Fedora derivative. Also with GNOME.

Try using ALT+TAB on  Gnome and you will see it :) (or on other DE too)

Would if I could; unfortunately (Since I dual-boot), an update to Winderp screwed over my Linux partition. So ATM I'm stuck on Winderp.

Now we have a file in /opt/skype_unofficial_client/  called "skype.html".
Put a Skype test in there, and see if it fix the problem.

Sadly, it's still the same thing :(

:(
Hope node.js people fix it soon. Anything reopen the issue.


Image_captioning: How to save the model?

Hello
I get this error when I try to save the model:
cannot be saved because the input shapes have not been set. Usually, input shapes are automatically determined from calling .fit() or .predict(). To manually set the shapes, call model.build(input_shape).
please help me if you know how to resolve this Error

Did you get any solution to this issue?

I think this is also referenced in this issue https://github.com/keras-team/keras-io/issues/1364

Hi, In order to save the model with custom object using Keras saving format .keras you need to override the get_config() method, for detailed steps follow the guide here https://keras.io/guides/serialization_and_saving/#custom-objects


Load API generated by Cuberite's scripts

Cuberite can currently generate a list of API functions that are automatically exported by ToLua++, via its src/Bindings/GenerateBindings.lua script. The ApiLoader should load the API from that format, as well as have a way to specify more API description files to read (for the manual bindings).

All done.


[EPIC] Media Playing

Acceptance Criteria

[ ] Should have a universal audio player with basic playlist support
[ ] should have a standard video player, that pauses audio when playing

@burkeshartsis when you get a chance, can you add issues to this (and maybe tweak the acceptance criteria) to encompass what needs to be done still with audio / video stuff?
(This is just encompassing the players and the persistent player at the bottom of the screen - the music browser in content is a separate epic #66)

@conrad-vanl @burkeshartsis What are we planning on doing with these [EPIC] github issues?

They're just buckets to connect other issues to. They should be checked off and closed. They were used to initially organize and estimate work from our initial engagement brief.
I can work on that.

@conrad-vanl That's great!


feat(rust): adapt and re enable complex bats tests

Tests involving credentials and project enrollment where disabled,  here we adapt them to take into account the changes introduced,  and re-enabled them.
Added the case of  enrollment-token member enrollment.

@mergifyio rebase