Support running notebooks that rely on .env via CLI

The content of the configuration file .env is handled when starting the container using docker-compose. However the content of .env is never used when running the container using docker run, for example when rendering notebooks using docker run.
The current use case is to generate the notebook synapse.Rmd that requires access to the Synapse Config file generated at start based on SYNAPSE_TOKEN. As a result, the following command fails:
$ docker run --rm \
>     -v $(pwd)/notebooks:/data \
>     -e RENDER_INPUT="/data/examples/synapse.Rmd" \
>     tschaffter/rstudio \
>     render
Quitting from lines 41-50 (synapse.Rmd)
Error in py_call_impl(callable, dots$args, dots$keywords) :
  SynapseNoCredentialsError: No credentials provided.

Detailed traceback:
  File "/opt/miniconda/envs/sage-bionetworks/lib/python3.9/site-packages/synapseclient/client.py", line 420, in login
    raise SynapseNoCredentialsError("No credentials provided.")
Calls: <Anonymous> ... withVisible -> eval -> eval -> <Anonymous> -> py_call_impl
Execution halted

The image rocker/rstudio uses s6 init system. In this context, the CMD of the official image is /init.
$ docker inspect tschaffter/rstudio | grep -C 2 "Cmd"
            "StdinOnce": false,
            "Env": null,
            "Cmd": null,
            "Image": "",
            "Volumes": null,
--
                "miniconda_bin_dir=/opt/miniconda/bin"
            ],
            "Cmd": [
                "/init"
            ],


How to run my own image with motion_blur.py?

Thanks for your amazing work, my current work need to use your motion_blur.py, it can work when i use CarDataset,but how can i run my own image for test?

Maybe i need to run DeblurGAN to get shape image firstly?

Thank you for your interest in our work!
Just load the image, convert it to Tensor, and put it into the network. Specifically,

Make a copy of motion_blur.py

Remove / Ignore the code related to using the dataloader.

Load your image by i) or ii) written below:

i)  blur = cv2.imread( pth_to_image ) , then , blur = cv2.cvtColor(blur, cv2.COLOR_BGR2RGB)
ii) blur = Image.open( pth_to_image )

Convert the image into Tensor using transforms.ToTensor()

Put that into the network and get the result. Please see [ line: 74 - 80 ] in motion_blur.py for reference.

Ok, thank you!


Prototype recipe of ROAST and LEAST

Summary
Implementation of AST and SSAST w/ RoFormerEncoderLayer and LexTransformerEncoderLayer

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 84.14%. Comparing base (4fdab14) to head (72ecd49).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #117      +/-   ##
==========================================
+ Coverage   83.87%   84.14%   +0.27%
==========================================
  Files         121      121
  Lines       12574    12574
==========================================
+ Hits        10546    10580      +34
+ Misses       2028     1994      -34

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


voyageai[major]: upgrade pydantic

integration tests pass: https://github.com/langchain-ai/langchain/actions/runs/10710381065


Metaculus community forecasts are flipped

Steps to reproduce:

visit https://metaforecast.org/?forecastingPlatforms=metaculus
See difference between metaforecast and metaculus

Ohhh. Unfortunate refactoring, thanks for catching this! Will fix soon.
(It's not just metaculus, all probabilities are replaced with max(p, 100-p)...)


MultipartReader [ERROR] http: multipart handled by ParseMultipartForm

I got this error after using submit api to upload one image

This library is only tested against v7.x and below, and I will no longer maintain it.


[Table] selectedRows are not reflected in onRowSelection

set selectable, multiSelectable and canSelectAll to true

click select all

onRowSelection is not fired

choose one row and click on its checkbox

the checkbox is deselected, but the row is still in highlight, and onRowSelection is fired for this row

Also:

set selectable, multiSelectable and canSelectAll to true

select one row, then shift select more rows below this row

onRowSelection gives an index and an Object

then press a row between the shift-selected rows, the row cannot be deselected and the results from onRowSelection are wrong

@jackytck Thanks for the feedback. I'm currently working #1088 and have addressed the select all issues. I am also working on fixing the SHIFT clicking and onRowSelection event firing. Right now, when all rows have been selected I'm considering returning having onRowSelection fire with true or all. I do not want to attempt to populate a full list of the row indexes. For other scenarios, the onRowSelection will return a list of all of the selected row indexes instead of objects which represent ranges.

I think that is better that if all row as selected the onRowSelection return the full list of element of the list, if i am working with a dinamic table and not sure of what element are in i will prefer to have a list of all elements.
PD: sorry for my english

@valinol The new version of the table will only return index values as it does not know what the content is (and shouldn't care). You should have the model somewhere in your app. If you're not sure how this might work, consider looking at flux/reflux. There are lot of flux like patterns out there.

@jackytck we had a few PRs addressing this / similar issues with the Table component. Can you test this again and let us know?

Closing this for lack of response from the OP. If the issue is still not addressed feel free to leave a comment below and we will open the issue up for you.

@tintin1343 This issue is still persisting in v0.15.0 can you reopen this issue?


Added unittest bootstrap, and config test.

I've added a bootstrap which allows for loading from the test namespace.
Additionally I've added a test for the Config class, which isn't complete of the die statements that are in the class under test. This should be replaced by Exceptions, which allows for catching and exception handeling. Or not and just let the exception bubble up towards the user.

Klaus can we merge this pull request. I am willing to add additional test, however this PR needs to be done first.

Bump!

@garygreen could look into this PR as well ? I would love to add more testing but would need this PR merged first.

@DannyvdSluijs thanks for this. The only thing I would say is I'm not too keen on adding another dependency just for mocking files (vfsStream).
Any file/dir checks abstract into a function on the Config class so they can be mocked in phpunit if required (use mocks instead of vfsStream).

That would remove the need for vfsStream and simplify the tests.

@garygreen Seem's legit, I can take on these changes. You are correct that adding the vfsStream dependency is because the Config class is calling file systems methods.
How do you / @klaussilveira  envision this ? I would opt for an interface called IConfigReader with a single public method read(file  : string) : string[], this would allow for config to be stored in any format being read by a concrete reader class, the initial one being an IniReader.
interface IConfigReader
{
    public function read($file);
}

class IniConfigReader implements IConfigReader
{
    public function read($file)
    {
        ....
    }
}

I'm guessing that the vfsStream dependancy can be dropped as well if we can create some files specific for unit testing. This way we can postpone this rather large change and write this down as nice to have for the rewrite ? Any suggestions ?

Why should we reinvent the wheel of configuration management? Projects like Configula and Symfony Config already do this pretty well.
However, why should an application like GitList offer different configuration strategies? It's useless. It adds unnecessary complexity, increases the maintainability cost and offer virtually no advantages. How many times do you change the format in which your application configuration is stored? Do we really want to add this kind of complexity to the codebase just because someone might want to transform their GitList config.ini to config.xml, because he likes the syntax better?


Remove the VOLUME directive from Dockerfile

Currently the Dockerfile includes the directive:
VOLUME ["/config"]

This is un-necessary since anyone following the documentation will manually specify a -v option to docker run, and the directive means that if you run the container without an appropriate -v then it will silently create a persistent volume in the background which the user must know to manually delete.  It is not possible to start a completely ephemeral container with no persistent storage.
Removing the VOLUME directive would not have any negative effects on current users who are following the documentation, but it would open up use cases like a throwaway web browser to explore potentially dangerous links, where you don't want to persist the config directory.

The VOLUME directive is also very useful for another reason: it is used by NAS OSes like unRAID, Synolog, QNAP or by container management softwares like Portainer to propose users semi-automatic configuration of the container.

then it will silently create a persistent volume in the background which the user must know to manually delete

I don't see this behaviour.  If I run docker volume ls before and after temporarily creating and running a container (docker run --rm jlesage/firefox), I don't see additional volumes.

Doing docker run --rm ... is roughly equivalent to docker run ... followed by docker rm -v <container>, which does indeed remove anonymous volumes, but a docker run (without the --rm) followed later by a docker rm will leave the volume dangling, as will docker-compose up followed by down if the compose file does not explicitly mount the volume.

Yes you are correct.   And the docker-compose down operation also supports the -v option.
So by using the correct commands, someone can create a throwaway browser without creating orphan volumes.


[build] VSCode instead of Npp?

For a few years now I've been using Visual Studio Code (NOT VISUAL STUDIO - they are different). VSCode is an open source, cross platform, general purpose editor built on Electron framework.
For c++ i found  VSCode to be great because it has code completion, jump to definition, build/test tasks, debugger, code minimap (like sublime).
Not only can VSCode do what Npp does, but it is also cross platform. So you can essentially distribute the same editor for every system.
I'm not a MS fan, but I have come to love using this tool. The fact that it is open source makes it safe choice without having to worry about MS gimmicks.
Take a look, let me know what you guys think.
https://github.com/Microsoft/vscode

Yes, I know VSCode, it's amazing, it would be great to configure raylib for VSCode, or at least provide required configuration tasks to integrate with raylib and MinGW, actually, it's on the roadmap...
The point is that it requires some work and Notepad++ is already preconfigured for raylib (compiling scripts, intellisense help, program configuration...).
Undoubtely is an editor to consider for the future. Any help on that task is welcome! :)

Just a reference in case someone wants to compile C from VSCode: http://stackoverflow.com/questions/39998471/how-to-build-and-run-c-code-in-visual-studio-code

Hey there! Not to sound like a total noob, but I use VScode for c and compile using the built in terminal. Couldn't I just place the files of raylib in my folder and compile my code just fine?

Hi @Gamerfiend, which C compiler are you using with VSCode?
raylib provides several Makefiles to compile sources and also examples, I think they could be just called from terminal but it would be great to provide pre-configured VSCode tasks.
It shouldn't be difficult to implement it, actually, one user is already working on that..

@raysan5  How is that coming along? I currently have mingw installed, which from command line I just type "gcc test1.c" to compile something, then "a.exe" to run it :)

with raylib is a bit more complicate, it requires linking with several libraries and it could also require compiling raylib library itself. Current required line for code compilation using MinGW:
gcc -o $(NAME_PART).exe $(FILE_NAME) -s $(RAYLIB_DIR)\raylib\raylib_icon -Iexternal -lraylib -lopengl32 -lgdi32 -std=c99

But paths to raylib dir and MinGW tools must be defined.

Currently I have the PATH variable in my environment variables defined to mingw bin folder, which works perfectly. For raylib, should i create a environment variable defined as RAYLIB_DIR pointing to the raylib directory? or in your code should I replace $(VARIABLE) with my own words, such as gcc -o mygame.exe /gameout -s C:\raylib\raylib\raylib_icon, like that?

@Gamerfiend those $(VARIABLE) come from Notepad++, you should replace them by your own and make sure that paths for libraries are set in the Environment Variables (by default raylib installation is prepared to work as standalone, it doesn't touch the Environment Variables, so paths are required in Notepad++ script).
Example:
gcc -o mygame.exe mygame.c -s C:\raylib\raylib_icon -IC:\raylib\raylib\src\external -lraylib -lopengl32 -lgdi32 -std=c99


e2e: find a way to use main version of elemental-operator

For CI it's useful to test the current main version of elemental-operator rather than on old released version.

As the Dev code of elemental-operator is always built in OBS on each merge, it's easier to use this OBS Dev version by default.
Done in this commit.


Support UTF-8 label matchers: Update the docs on how to use UTF-8 in label matchers and parse mode feature flags

This pull request updates the docs to:

Explain that there is a new parser for label matchers, the transition period, and how Alertmanager reverts to the classic parser.
How to use the feature flags to switch between fallback, UTF-8, and classic parse modes.
The difference between UTF-8 matchers and classic matchers.
Improved the documentation on composing matchers to make more complex expressions.

I rebased main.


Better documentation for built-in formatters

We don't really document them anywhere.
related: #1376

Would the idea here to be to add some details to each formatter's TypeScript files similar to IRuleMetadata which then can be compiled into the website docs? My guess would be IFormatterMetadata with fields along the lines of:

name
description
descriptionDetails
example output
whether the formatter output is for consuption by humans or other tools - ?
notable tools which accept the format (helpful for people googling) - ?

Not tagged as accepting PRs but I'd like to give it a go if possible.

@bencoveney that sounds great, go for it


optimize: add ForceSetTag method for RemoteInfo, which is used to reset Tag in special scenario

What type of PR is this?
optimize
Check the PR title.

[x] This PR title match the format: <type>(optional scope): <description>
[x] The description of this PR title is user-oriented and clear enough for others to understand.

(Optional) Translate the PR title into Chinese.
optimize: RemoteInfo 新增 ForceSetTag 方法，用于特殊场景 reset Tag
(Optional) More detailed description for this PR(en: English/zh: Chinese).
en:
zh(optional):
Which issue(s) this PR fixes:
none

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


Drop Book Page Numbers from Spells

Overview
This is coming off of #154. I'm going to drop the page numbers because technically all of this data is from the SRD which doesn't necessarilly match everything in the PHB.

Oh! @ogregoire, if Discord is a thing you use, we'd love to have you on it. Here's an invite link! We've been recently talking about ways to allow other developers to make packages of content for things like Homebrew and easily get loaded into the DB for local development. I'd love to get your opinions on the matter.

Content would probably only live in forks but still it would be nice to make it easier for others.


Sends on Fader/Fader Flip Action and Feedback

Sends on Fader action and feedback (As per issue #39)

Tested on X32 and its happy but feel free to test for yourself too :)


chore(deps): update terraform github.com/terraform-ibm-modules/terraform-ibm-landing-zone to v6.4.0

This PR contains the following updates:

Package
Type
Update
Change

github.com/terraform-ibm-modules/terraform-ibm-landing-zone
module
minor
v6.3.0 -> v6.4.0

Release Notes

terraform-ibm-modules/terraform-ibm-landing-zone (github.com/terraform-ibm-modules/terraform-ibm-landing-zone)
v6.4.0
Compare Source
Features

update OCP default version to to 4.16 (#922) (a93f177)

v6.3.1
Compare Source
Bug Fixes

a fix was added to the VSI extension variation so that VSI only get created in the subnets that are meant for VSIs. If upgrading from a previous version, you may see VSI's being destroyed in some subnets where they were not supposed to be provisioned. (#661) (4b3d100)

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

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

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

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

GitHub release
v1.2.6

Your semantic-release bot :package::rocket:


Add support for signed URLs in GCP

Both the AWS and Azure interfaces for blobs, IAwsS3BlobStorage and IAzureBlobStorage have a definition for pre-signed/SAS URLs (GetUploadUrlAsync for AWS and GetBlobSasAsync for AWS).
Can we please:

Have similar support for signed-urls in GCP blobs (https://cloud.google.com/storage/docs/access-control/signed-urls)
Standardize pre-signed URLs so we can have a similar API for the three common cloud providers (AWS, Azure, and GCP)

Can you suggest which APIs need to change/new APIs needed so I can understand the interface you are looking at?

AWS's blob interface has definitions for pre-signed URLs for upload and download.
https://github.com/robinrodricks/FluentStorage/blob/develop/FluentStorage.AWS/Blobs/IAwsS3BlobStorage.cs
Google Cloud also supports pre-signed URLs for upload and download in a similar manner of returning a signed URL:
https://cloud.google.com/storage/docs/access-control/signing-urls-with-helpers#storage-signed-url-object-csharp
Aside for the mime type in the AWS method declaration, the same method declaration can be used for GCP.
The implementation can either sign locally if a service account key is available. If a signing key is not available, the implementation can call the signBlob API to sign the URL.
https://cloud.google.com/storage/docs/access-control/signed-urls#signing-iam


Record response for a specific site

Would love to know if there a way to only record response for a specific URL.

Currently not, but feel free to create a pull request :-)


Handlebars has no method 'compile'

I'm trying to get a basic example of Handlebars templating to work, but I get the following error:
Object # has no method 'compile'
I'm including the handlebars.runtime-1.0.0.beta.6.js file correctly. This is the code where I get the error:
function bindProducts() {
$.ajax({
type: "POST",
url: "handlers/productsearchhandler.ashx",
data: "brands=11233,11232",
error: function () {
alert("error");
},
success: function (data) {
var source = $("#product-template").html();
var template = Handlebars.compile(source); //<-- this is the where the error appears
$("#products-container").html(template(data));
}
});
}
The template is loaded correctly. It looks like this:

Have I missed something fundamental about how this works?

Update: It works when I use the Handlebars version used in http://tryhandlebarsjs.com/

I think you might have downloaded one of the handlebars js versions that only included the runtime. That version does not include the compilation engine and is normally used with precompiled templates. The one I use on tryhandlebarsjs.com is the full runtime (which is actually mislabelled on the github download page).
handlebars-1.0.0.beta.6.js — Handlebars 1.0.0.beta.5 - several IE fixes

@Radiv3 I suspect @jblotus is right. If it turns out that this does not solve your problem, let me know and I'll reopen.


NO CA FOUND

I am getting this error:

I added bin files to true adress which in below.

What is my fault? Help please

I used these ofsets
client_cert	0x40	5	0x2a000	8K
client_key	0x40	6	0x2c000	8K
client_ca	        0x40	7	0x2e000	8K
and my at command set is
AT+CWMODE=1
AT+CIPMUX=0
AT+CWJAP="Test","helloworld"
AT+CIPSNTPCFG=1,1,"us.pool.ntp.org"
AT+CIPSNTPTIME?
AT+CIPSSLCCONF=3,0,0
and when I send
AT+CIPSTART="SSL","a2u7s0al75g2nd-ats.iot.us-east-2.amazonaws.com",8443
It returns NO CA FOUND

Please do not change the partition name. And your amazon_ca looks 14KB, but the size of partition client_ca is 8K

What should I do for that size? What do you mean dont change the partition name?

download.txt
I am using these adresses

I increased ofset but still same error and I changed my bin file's name.

still it is not working.

Please help

@xcguang @ustccw ?

UART1 and UART0 log

Please provide your at_customize.csv

I dont use your esp-idf library. I am connecting esp32 with stm and there is uart wire between esp32 and stm. What is relevant at_customize.csv ? Do you mean at_customize.bin?

I fixed it. CA certificate should download with click save as. After that move my certificate file had 1k size. .docx is making file's size too high. Thank you.


medium.com connect failed

use chrome visit medium and get ERR_CONNECTION_RESET

it may have some dns problem...

Actually, it's dns cache pollution which cause you visit facebook. So, change the dns server to <IP_ADDRESS> (google) .


Bug: Stretch Marker Mouse Context and Strech Marker Display with 5.04 pre 1 Update

I'm afraid that with the last pre-release (5.04 pre1), the great updates on stretch markers broke the Get Stretch Marker Under Mouse Context... =/
It does work horizontally (where the markers are) but not vertically (we have to put the mouse where their original vertical position was, even if they are above or below).

That sucks...more reverse engineering of pixels -_- Not really sure when I will have the time for that...let pre cycle first finish to see where it ends up

Hey guys, any news on this?

@Jeff0S , I've spoken with Breeder and he is no longer able to work on this. With your knowledge of and access to the inside of REAPER, are you able to get this working, please? It's a very useful part of SWS and it's killing me that scripts crucial to my workflow no longer work because of this.

Honestly, I'd much prefer to go back to the old stretch marker system. I really hate the cluttered views and who ever needs rate transitions anyway? I can't imagine anything they're useful for and I've been editing audio for 20 years.
The handles are also impossible to grab when there's a volume envelope over the item set to 0dB. You actually have to hide the envelope first.
Plus, you have to reach for different places when adjusting them. That SUCKS for workflow.
I'm sure there's a better place for this conversation, I'm just not sure where it is. Please advise.

Hello? Is anyone here anymore?

Bump, not it that it helps.

Hey now, no need to be cranky.  I'll take a look next time I build.

Sorry, it's just been a year and a half since the thread was opened and I didn't think anyone was actively maintaining SWS anymore.

@swstim
Thanks ! Javolatt made some suggestions about how to solve it, maybe it can help.

FYI the bump does help - the list of bugs here is prohibitively long and it's good to know what people are hitting their heads against.  I need to get an update out soon as it's been WAY too long.

@swstim Indeed, reports and FR list is long, but some issues could be closed because they are already solves like this one : https://github.com/reaper-oss/sws/issues/350
Reviewing all 11 pages of FR to check if it need to be closed, add BUG or FR labels, or tells that it can be scripted and doesn't need to be included right in SWS can be a bit boring, for sure. But maybe some people can help ?

@swstim Thanks Tim, that's super-exciting to hear you're willing to put some time into this.
This sounds like a great solution, if it will work: "Remove the y-check entirely and just see if the mouse is within STRETCH_M_HIT_POINT of a stretch marker horizontally."

@X-Raym, great idea to do a cull. Normally I would love to help with this, but I'm neck-deep in building the studio complex in London at the moment.

OK - I've removed the Y hit logic per Lokasenna's suggestion here and daxliniere's agreement above.  I will leave this open to maybe someday update the logic to account for actual stretch marker logic or other issues that crop up because of this change.

Thank you SO MUCH Tim! I'll test this tomorrow and let you know how it goes.
(Perfect timing as I have a drum editing session tomorrow.)

Cool ! My Reset stretch marker under mouse may be usable again ! :P

OMFG!! Raymond's script Snap stretch marker under mouse to closest grid line works again!!!!
I'm so happy!!

@daxliniere At last 🎉 👍
@swstim Thanks 📦
Closed because fix here : https://github.com/reaper-oss/sws/commit/dbae5b93b58595a0c87796aaffe52691f03c5551


Convert to public CI

See if we can use free GitHub runners for CPU jobs. This has to be after release.

For now I just disabled the slow tests, will follow up in #9


Cowboy 1.0.4 escaping plus sign in path

When I pass in a + into the path of a URL (e.g. somePath/my+var), it is being translated to a space once I receive it in the handler as well as in any path constraints (my+var is becoming "my var").  Debugging shows that it is being received properly by Cowboy, but I haven't been able to figure out where it is being escaped.  Plus signs are supposed to be escaped in the query string, but not in the path.  Any ideas where this might be happening?
Thanks!

Here: https://github.com/ninenines/cowboy/blob/1.0.x/src/cowboy_router.erl#L325
This has been fixed in master: https://github.com/ninenines/cowboy/blob/master/src/cowboy_router.erl#L321
I'm mostly working on master now but let's leave the ticket open in case I do another point release. Thanks!

Awesome, thank you, Loïc!

This will be fixed in 1.1 released later today. Thanks!


test: use wait-on for CI runs

Adds 'wait-on' to CI builds to be more proper.
Current CI builds rely on cypress waits in the cy.visit() command. It should rely not start tests unless the bot and ui are started successfully.

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

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Added so the embeddings are actually dropped

At the moment the TextEmbeddingReturner just returns a mask where it given the % chance using cond_drop_prob, is setting some mask values to false.
This works great if the model you are working with respects the mask, however x-transformers attention does not do this since it takes the context and uses the raw context and feeds it to the k and v linear layers.
https://github.com/lucidrains/x-transformers/blob/0c6266ee44ea99a4449cd9201ba55924a6a7eae7/x_transformers/x_transformers.py#L944
kv_input = default(context, x)

q_input = x
k_input = kv_input
v_input = kv_input
r_input = x

q = self.to_q(q_input)
k = self.to_k(k_input)
v = self.to_v(v_input) if exists(self.to_v) else k
r = self.to_r(r_input) if exists(self.to_r) else None

oh, the text mask is actually sent here and then applied here

yea, there is some wasteful projections, even for the tokens that are masked out, but that's how most transformers are trained these days

oh, the text mask is actually sent here and then applied here

Correct, but the mask isn't applied everywhere :/

yea, there is some wasteful projections, even for the tokens that are masked out, but that's how most transformers are trained these days

I've tried using the cond_drop_prob but I had very little success until I did the changes in this commit.
I think it's due to these wasteful projections since that will leak information that it shouldn't have seem and poisoning the output.
Image if you have a full transformer that takes tokens and have the mask to hide some of the tokens and the decoder should reconstruct the full sequence. If some of the unmasked data would leak into the model then the training poisoned.

the mask should be applied or that would be a pretty big bug in x transformers
let me check tonight, out with 🐕

oh shoot, maybe the condition dropping isn't being applied to cached text embeddings 🤦♂ will fix if not!

the mask should be applied or that would be a pretty big bug in x transformers
let me check tonight, out with 🐕

It's applied later on in he code but the k and v value at the start is not.
If you call it with the context and context_mask, the kv_input will be default context, then the next lines the kv_input  value is pass to k and v linear layers.
This means that they get the full context, I don't see the context in previous steps that the mask get applied.
https://github.com/lucidrains/x-transformers/blob/0c6266ee44ea99a4449cd9201ba55924a6a7eae7/x_transformers/x_transformers.py#L944
    def forward(
        self,
        x,
        context = None,
        mask = None,
        context_mask = None,
        attn_mask = None,
        rel_pos = None,
        rotary_pos_emb = None,
        prev_attn = None,
        mem = None,
        mem_mask = None,
        return_intermediates = False,
        cache: Intermediates | None = None,
    ):
        b, n, h, kv_h, head_scale, num_mem_kv, device, has_context = x.shape[0], x.shape[1], self.heads, self.kv_heads, self.head_scale, self.num_mem_kv, x.device, exists(context)

        kv_input = default(context, x)

        q_input = x
        k_input = kv_input
        v_input = kv_input
        r_input = x

        if exists(mem):
            k_input, mem_packed_shape = pack([mem, k_input], 'b * d')
            v_input, _ = pack([mem, v_input], 'b * d')

        q = self.to_q(q_input)
        k = self.to_k(k_input)
        v = self.to_v(v_input) if exists(self.to_v) else k

    r = self.to_r(r_input) if exists(self.to_r) else None

later masking is fine, gradients cannot propagate to the point you are concerned about

https://github.com/lucidrains/meshgpt-pytorch/blob/main/meshgpt_pytorch/meshgpt_pytorch.py#L1362 the issue is because cached text embeds are passed through here and never cond dropped, will fix! thank you Marcus 🙏

later masking is fine, gradients cannot propagate to the point you are concerned about

I'm not quite sure if I'm misunderstanding but doesn't it matter that k and v which have gotten the full context?
k = self.to_k(k_input)
v = self.to_v(v_input)

https://github.com/lucidrains/meshgpt-pytorch/blob/main/meshgpt_pytorch/meshgpt_pytorch.py#L1362 the issue is because cached text embeds are passed through here and never cond dropped, will fix! thank you Marcus 🙏

I don't think that's my problem since when cond_drop_prob is None the conditioners it will use the self.cond_drop_prob which is  0.0 for me.
However it might be good to change that just in case.

it doesn't matter if they are masked out later

it doesn't matter if they are masked out later

You are probably correct, sorry I thought that the linear layer pays attention to the other dims in the length.
Little off topic, is there any mode or what not that can input (b, length, dim) and output another length, e.g. (b, 400,dim) to (b, 900, dim) ?
I know it works with autoregression but I'm not sure if there is any solution that can output in another length in a single pass.

@MarcusLoppe just doubled checked the code and couldn't find any errors
i think it should be working

you can pad a dimension, in your example, it would be F.pad(tensor, (0, 0, 0, 900 - 400)) if you wished it for the right hand side

you can pad a dimension, in your example, it would be F.pad(tensor, (0, 0, 0, 900 - 400)) if you wished it for the right hand side

Hey, i was just thinking if it was possible to modify the autoencoder to encode point clouds and then decode into a 3D mesh.
But since point clouds and triangle count isn't the same I wanted to know if it's possible to modify the output length.
There is quite a lot of point cloud autoencoders so I was thinking of using the encoder part and then modify the autoencoder so it inputs the embeddings of cloud point and decodes into 3D mesh.
Another question, I'm having issues finding actual information if it's possible for diffusion models to handle different length outputs?
Current 3D diffusion models are pretty bad since they seems to output a near fixed triangle count and this messes up simple object since they use way to many triangles and overpopulate it.


PMID:25356590

[ ] New term : normal cenH-derived RNA level
A cell phenotype observed in the vegetative growth phase of the life cycle in which the amount of RNA transcribed from cenH measured in a cell is normal. cenH is a region located between mat2-P and mat3-M that is homologous to centromeric repeats, and that produces non-coding RNAs and small interfering RNAs.

[x] New term : decreased subtelomeric heterochromatin RNA level
A cell phenotype in which the amount of RNA derived from subtelomeric heterochromatin regions is lower than normal. Total RNA or a specific RNA, such as the mRNA transcribed from a gene of interest, may be affected.

[x] New term : decreased LTR-derived RNA level
A cell phenotype observed in the vegetative growth phase of the life cycle in which the amount of any RNA transcribed from long terminal repeat elements (LTRs) measured in a cell is lower than normal.

[x] New term : decreased protein localization to chromatin at protein coding gene
A cell phenotype observed in the vegetative growth phase of the life cycle in which the localization of a protein to chromatin at one or more protein-coding genes is decreased.

[x] New term : normal protein localization to chromatin at protein coding gene
A cell phenotype observed in the vegetative growth phase of the life cycle in which the localization of a protein to chromatin at one or more protein-coding genes is normal (i.e. indistinguishable from wild type).

[ ] New term : normal protein localization to pericentric heterochromatin during vegetative growth
A cell phenotype observed during the vegetative growth phase of the life cycle in which the localization of a protein to chromatin at the pericentric regions is normal (i.e. indistinguishable from wild type).

[x] New term : decreased cenH-derived RNA level
A cell phenotype observed in the vegetative growth phase of the life cycle in which the amount of RNA transcribed from cenH measured in a cell is lower than normal. cenH is a region located between mat2-P and mat3-M that is homologous to centromeric repeats, and that produces non-coding RNAs and small interfering RNAs.

[x] New term : normal LTR-derived RNA level
A cell phenotype observed in the vegetative growth phase of the life cycle in which the amount of any RNA transcribed from long terminal repeat elements (LTRs) measured in a cell is normal (i.e. indistinguishable from wild type).

[x] New term : normal subtelomeric heterochromatin RNA level
A cell phenotype in which the amount of RNA derived from subtelomeric heterochromatin regions is normal (i.e. indistinguishable from wild type). Total RNA or a specific RNA, such as the mRNA transcribed from a gene of interest, may be affected.

[x] New term : increased histone H3-K4 acetylation at protein coding gene during vegetative growth
A cellular process phenotype observed in the vegetative growth phase of the life cycle in which acetylation of lysine at position 4 of histone H3 occurs to a greater extent than normal in regions containing protein-coding genes.

[x] New term : decreased histone H3-K4 acetylation at protein coding gene during vegetative growth
A cellular process phenotype observed in the vegetative growth phase of the life cycle in which acetylation of lysine at position 4 of histone H3 occurs to a lower extent than normal in regions containing protein-coding genes.

[x] New term : increased histone H3-K4 acetylation at LTR during vegetative growth
A cellular process phenotype observed in the vegetative growth phase of the life cycle in which acetylation of lysine at position 4 of histone H3 occurs to a greater extent than normal at long terminal repeat (LTR) regions.

[x] New term : decreased histone H3-K4 acetylation at LTR during vegetative growth
A cellular process phenotype observed in the vegetative growth phase of the life cycle in which acetylation of lysine at position 4 of histone H3 occurs to a lower extent than normal at long terminal repeat (LTR) regions.

[x] normal telomere repeat region derived RNA level

@kimrutherford  should a configuration issue go on Canto tracker or config tracker?

should a configuration issue go on Canto tracker or config tracker?

We have been putting them on the config tracker but it doesn't matter too much.  I'm subscribed to all the trackers (except for curation) so I'll see issues wherever they go.

Yes, that's working, thanks !
Going back to the session to change the annotation, I noticed that the same extension type needs to be added for the "decreased" version of this term, so I will add it to the file as well.


Decode expired token

Hi, I have a question regarding decoding expired tokens. Is it possible to decode an expired token?
This is the token
Bearer eyJ0eXAiOiJKV1QiLCJhbGciOiJIbWFjU0hBMjU2In0.eyJleHAiOjE0NTAyNzUyNzIsInVzZXIiOnsiaWRVc2VyIjo4Nzc3MywiaWRDb25jZXB0VXNlciI6NTgxMSwiZmlyc3ROYW1lIjoiRnJ1bnphIiwibGFzdE5hbWUiOiJMZWFmIiwiZW1haWwiOiJmcnVuemFAYXNzaXN0LnJvIn19.4um3Q2IuyXWMaohwUF5_fAiVdrIlen68Oqs2CLLLVlY

I can decode the token using http://jwt.io I am able to see the payload, without specifying any secret key.
{
  "exp":<PHONE_NUMBER>,
  "user": {
    "idUser": 87773,
    "firstName": "Frunza",
    "lastName": "Leaf",
    "email"<EMAIL_ADDRESS>  }
}

Thanks,
Marius

Indeed, there is no way to do that currently, all decoding methods also validate the token since you should trash all invalid tokens anyway. Is there a good reason to read such tokens? I don't really like adding methods that people shouldn't use.

@pauldijou Thanks for the quick response!
I have some desktop apps (Desktop-App) that need to login using my application (Auth-App), but those desktop apps don't call directly to Auth App, they go thought a middleware-application (Middle-App - does a lot of things). So when a user submits his credentials to the Middle-App, the Middle-Apps calls Auth-App and gets an JWT token that has 5 min exiration time (also puts some data in payload, and adds user data in Redis Cache, so we can identify the user later, the maxAge of user session in Auth-App is 30 days, after this date the user has to relog), this token is sent back to the client (Desktop-App) through Middle-App as I said earlier.
Now the user can make calls to to Middle-App (authorized calls) for the next 5 minutes, after those 5 minutes the token expires, and it needs to be refreshed. So the Middle-App needs to call the Auth-App with an expired token.
I have to decode the expired token, get some date from the payload and match the redis data, if the date matches I refresh user's token else I redirect the user to login page.
If you have a better idea please advise me!

I understand. Sounds legit and I don't have any better solution.
It's quite easy to implement but I'm not sure how to impact the current API : adding an optional parameter to disable verification or adding new methods. There are already quite a lot of methods so maybe the 1st solution. Any thoughts?
PS: just wandering, if someone stole a token (a 5min one), what prevent him from refreshing it after expiration date? Do you have bonus security in the middle app?

I have not yet thought of this, but I have something in mind. I will share with you as soon as I will implement it.
Regarding decoding expired tokens I followed some of library's code and managed to decode the token, by writing a few methods.

Thank you!


Remove unnecessary dependencies from requirements.txt

If we need any of these dependencies again, we can add them back in. @wimo7083 if you see anything that we're obviously going to need, point it out and I'll add it back in.

I'll go ahead and merge this. If I find any issues I'll try and fix later. Thanks!


bump versions to remove vendored copy of githubrelease

Description
This started from trying to get it to build with poetry and it erroring because the vendored githubrelease version was considered improper by poetry. So I saw the new version had been released and the vendored copy could be removed. This then led to conflicts on the backoff package and the need to upgrade the opentelemetry packages.
I don't know how this project intends to version with respect to upstream so I'm not sure this change is proper, but opening for comment to find out :)
Type of change
Please delete options that are not relevant.

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

How Has This Been Tested?

[X] Tested manually
[ ] Added automated tests

Checklist:

[ ] Changelogs have been updated
[ ] Unit tests have been added/updated
[ ] Documentation has been updated

Closing as not relevant


Drag and drop is not working when implementing check box with list

(https://github.com/naveenkumarvk/DragAndDrop)
Refer the URL, I have the sample code which we are trying to implement list with checkbox . it is better if u please help me in this ,
Issue is when implementing the drag and drop with checkbox ,we cant able to enable or disable checkbox.

HI There,
This worked for me on your example:
<Sortable
   tag="ul"
   className="block-list"
   options={{delay: true}}
   onChange={console.log}
>
	{simpleList}
</Sortable>

Sortable assumes the checkbox is draggable and doesn't allow click events to trigger the checkbox change. The delay means the user has to drag before it works. You'd find that it'd work fine on mobile but not on desktop.

If you use key and assign value as Index of map loop. Do not use the index as a key for your list items.


Read in IMAS data

There are 2 options for reading IMAS data:

IMAS Python HLI (legacy)
IMASPy, newer, pure Python interface which is a bit higher level

There are 2 backends:

MDS+ (proprietary, slow, large files)
HDF5 (newer, faster, smaller files, available soon)

Prefer IMASPy + HDF5 where possible.

I think we now got some grip on loading data. I made a new issue to support HDF5 when available: #76


ERRO[0002] Failed to get controller., OriginalError: %!w(*errors.errorString=&{EOF})

hi @mostafa  i am trying to get username password for saslConfig from keyvault, i have added the saslConfig inside a method and called it from the setup method but it gives below error
ERRO[0002] Failed to get controller., OriginalError: %!w(*errors.errorString=&{EOF})  error="Failed to get controller., OriginalError: %!w(*errors.errorString=&{EOF})"
ERRO[0002] GoError: Failed to get controller., OriginalError: %!w(*errors.errorString=&{EOF})
import { LoadJKS, TLS_1_2 } from "k6/x/kafka";
import {
    Writer,
    Reader,
    Connection,
    SchemaRegistry,
    SCHEMA_TYPE_JSON,
    SASL_SCRAM_SHA512,
    TOPIC_NAME_STRATEGY, KEY, VALUE
} from "k6/x/kafka";
import VaultManager from "./VaultManager.js";

let username;
let password;
let saslConfig={};
let offset = 0;
// partition and groupId are mutually exclusive
let partition = 0;
let numPartitions = 1;
let replicationFactor = 1;

export const options = {
    stages: [
      { target: 1, duration: '30s' },
      { target: 3, duration: '30s' },
      { target: 1, duration: '30s' }
    ],
  };

// Write your Vault access code in setup() - Called only Once
export function setup() {
    //This would be taken from environment in the future
    const acccesTokenToiPACE = 'Bearer aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa';
    const teamName = 'team';
    let vault = new VaultManager(acccesTokenToiPACE, teamName, 'DEV');
    console.log('Empty MAP = ' + vault.getJSONFromSecretMap());
    username = vault.getSecretFromVault("AZURE-USERNAME");
    console.log('username = ' + username);
    password = vault.getSecretFromVault("AZURE-PASWORD");
      console.log('password = ' + password)
    setSslConfig(username, password);
    return "abc";
  }

 function setSslConfig(username, password){
   saslConfig = {
          username: username,
          password: password,
          algorithm: SASL_SCRAM_SHA512
     };
   }

const jks = LoadJKS({
  path: "/truststore.jks",
  password: "password",
  clientCertAlias: "kafka",
  clientKeyAlias: "kafka",
  clientKeyPassword: "password",
  serverCaAlias: "root",
});

const tlsConfig = {
  enableTls: true,
  insecureSkipTlsVerify: true,
  minVersion: TLS_1_2,
  clientCertPem: jks["clientCertsPem"][0], // The first certificate in the chain
  clientKeyPem: jks["clientKeyPem"],
  serverCaPem: jks["serverCaPem"],
};

  const brokers = ["emp-digital.net:443"];
  const topic = "topic.internal.any.v1";

  const schemaRegistry = new SchemaRegistry();

  const writer = new Writer({
      brokers: brokers,
      topic: topic,
      sasl: saslConfig,
      partition: partition,
      tls: tlsConfig,
  });
  const reader = new Reader({
      brokers: brokers,
      topic: topic,
      partition: partition,
      offset: offset,
      sasl: saslConfig,
      tls: tlsConfig,
  });
  const connection = new Connection({
      address: brokers[0],
      sasl: saslConfig,
      tls: tlsConfig,
  });
//

export default function (data) {
 console.log('You get access to the data variable which contains your secrets in default = ' + data);
  // const obj = JSON.parse(data);

        let messages = [
            {
                key: schemaRegistry.serialize({
                    data: {
                        correlationId: "test-id-abc-" + 1,
                    },
                    schemaType: SCHEMA_TYPE_JSON,
                }),
                value: schemaRegistry.serialize({
                    data: {
                        templateId: 5,
                        testId: 4,
                        status: "deactive",
                        message: "Hi, i am testing",
                    },
                    schemaType: SCHEMA_TYPE_JSON,
                }),
            },
        ];
        writer.produce({ messages: messages });

    // console.log("---------_>>>>>",messages.length)
    let mesg = reader.consume({ limit: 10 });

        console.log("correlation=====>>>", schemaRegistry
        .deserialize({ data: mesg[0].key, schemaType: SCHEMA_TYPE_JSON })
        .correlationId);

        console.log("schema------>>>>>", schemaRegistry.deserialize({
        data: messages[0].value,
        schemaType: SCHEMA_TYPE_JSON,
        }));

}

export function teardown(data) {
    writer.close();
    reader.close();
    connection.close();
}

Hey @arpitaso,
Please use code highlighting while posting your code here. Set connectLogger to true, so you can see the internal logs of the Writer and Reader, and we can have a detailed log of what happens.

@arpitaso
I suppose this is resolved. Feel free to reopen it if the issue persists.

I'm encountering an identical error message. I'm utilizing a port forwarding mechanism to establish a connection with the Kafka broker located within a Kubernetes pod. In my specific scenario, I've introduced a consumer that reads from the same partition.

@nirajkmr Can you elaborate with a working script that has your data removed?

I received the same error because I didn't have TLS configuration set up. Just an FYI.

Hey @mkosta,
SASL authentication is already supported: https://github.com/mostafa/xk6-kafka/blob/main/scripts/test_sasl_auth.js.

thank you, Mostafa. i have seen these files. our kafka server requires jaas config.
when i test connection with performance tester.sh from kafka i can only send messages with saasl.jaas.config property provided. same with offset explorer. it requires jaas config property.
in k6 i can create topic with my saslconfig, but when i need to produce messages i get eof error. i tried everything it seems. we use scram sha 512. on my local env same code sends messages just fine. but i have no sasl.
i will look at setting sasl on my local kafka and see when it will start failing))
i even copied Writer from your examples, very simple, yet on Sasl scram sha 512 it throws eof errors :(

@mkosta Use connectLogger (bool) on the Writer to see what error you're getting from Kafka. Also, SASL config must be accompanied with TLS config to work properly.

i tried logger, but output was not showing much. just error and code lines. i can attach output tomorrow. and i added tls conf after reading this thread, just set it to false.

that's what i am getting
ERRO[0019] Failed to get controller., OriginalError: EOF  error="Failed to get controller., OriginalError: EOF"
ERRO[0019] GoError: Failed to get controller., OriginalError: EOF
	at file:///home/kosta/Public/qa/k6/altenar/aip.tests.load/helpers/kafka.js:23:19(86)
	at go.k6.io/k6/js.(*requireImpl).require-fm (native)

const writer = new Writer({
    brokers: ["cut:9092"],
    topic: "kosta-settlement",
    autoCreateTopic: true,
    sasl: saslConfig,
    connectLogger: true,
    tls: tlsConfig

 const saslConfig = {
    username: "cut",
    password: "cut",
    // Possible values for the algorithm is:
    // NONE (default)
    // SASL_PLAIN
    // SASL_SCRAM_SHA256
    // SASL_SCRAM_SHA512
    // SASL_SSL (must enable TLS)
    // SASL_AWS_IAM (configurable via env or AWS IAM config files - no username/password needed)
    algorithm: SASL_SCRAM_SHA512,
  };

const tlsConfig = {
    enableTls: false,
    insecureSkipTlsVerify: false,
    minVersion: TLS_1_2,
   }

ok, i figured out eof issue) there was import of one of our helpers and it was actually failing somewehere down that file. so auth works without jaas. now onto context deadline issue)

@mkosta Also, SASL/SCRAM won't work without TLS and you have TLS disabled: enableTls: false.

thanks for your help and extension! it actually worked this way. as i mentioned the issue was in import. other class was lazy initializing? new connect that was not used and was not given sasl config but was failing main test. hence connectLogger was not giving more info)


disable VSO rollout restart

Description
Disable VSO rollout restart
Changes proposed in this pull request:

Currently VaultStatic secret controller has websocket connection problem -> triggered unexpected rollout restart
Rollout restart disabled

Related issue(s)

Resolves #6094: https://github.tools.sap/kyma/backlog/issues/6094

/kind bug


Mk/public

I went looking for docs on the public/ folder. Turns out they are on the old site here: https://ember-cli.com/extending/#addon-project-structure, but not here.
Got a little carried away with editing and made some other changes in another commit too. Happy to do a couple round of edits if needed, but if they're too controversial, I can just remove the commit also :) (Or someone can merge and update them to better language also!)

It would be helpful for reviewers if you could split out the edits into a separate PR!

@locks no problem, sorry about that. Reduced this PR to just the additional docs about the public directory.

@locks I changed the example to a concrete example and I think it removes the need to explain anything about npm restricted scopes, etc. Also removes the need for the additional note! Thanks for the suggestion, I think this is simpler?


Concurrent checkouts lead to incorrect 'num_allocated' stocks

When two customers check out concurrently for the same product, only one of the checkouts is reflected in the num_allocated field.
Steps to reproduce:

Start sandbox, create two customers, create/pick test product.
Both customers add the product to their baskets and perform a checkout concurrently, making sure both customers finish checkout (click the Place Order button) as close to the same time as possible.

We have reproduced this behavior on a single machine, by performing the above steps with one customer in one browser window and another customer in a second browser window.
Expected Result:
The num_allocated for the product is 2.
Actual Result:
The num_allocated for the product is 1.

Thanks for the report.
I'll try to re-create with a test.

There is another issue from last year discussing this #1811

Closing as a duplicate of #1811.


Question: can flask create 'access.log', and 'error.log'

I am hoping to create two types of logs:

access.log: lists all requests made on the flask application
error.log: lists all warnings, and errors made within the flask application

Currently, I have defined app.py, which contains logic recycled from a gist comment:
import logging
from logging.handlers import RotatingFileHandler
from interface import app
app.run(host='<IP_ADDRESS>')

LOG_FILENAME = '/vagrant/log/access.log'

formatter = logging.Formatter(
    "[%(asctime)s] {%(pathname)s:%(lineno)d} %(levelname)s - %(message)s")
handler = RotatingFileHandler(LOG_FILENAME, maxBytes=10000000, backupCount=5)
handler.setLevel(logging.DEBUG)
handler.setFormatter(formatter)
app.logger.addHandler(handler)

log = logging.getLogger('werkzeug')
log.setLevel(logging.DEBUG)
log.addHandler(handler)

When I adjust my application, by making an unnecessary typo in the logic; or conversely, if I complete a successful post request on my application, nothing gets logged into /vagrant/log/access.log.  However, the /vagrant/log/access.log file is created, and empty.  Is the above logic equivalent to the error.log that I had intended?  If so, how can I correct this, while also creating a functional /vagrant/log/access.log?

Please take this to StackOverflow.


Revert always_use_private_address

@adamjt @nahiluhmot @bfulton
This was just broken.  We yanked v1.4.0 which contained this code, but forgot to revert the code. Will yank v1.5.0 and push v1.5.1.
More yanking for aerosol

:+1:


Configuration Only Sees System Environment Variables When Deployed to Windows Server 2012 R2

Hi All,
Just as a quick preface, I did do my best to search through open and closed issues and did a fair bit of googling before submitting this. My apologies in advance if this is something that I should have answered on my own...
I'm working through deploying my application to some hosted infrastructure. First time doing this. Followed all the instructions here: .
Did a test deploy with a brand new ASP.NET 5 (Core) app straight from the template and ran without issue.
Now it was time to get my app running, and I do have a number of configuration values that needed to be set as environment variables. I first tested locally, setting ASPNET_ENV to Staging in my local environment. I added the environment variables to the "User variables for " section in the Environment Variables dialog under advanced system properties. Everything AOK.
After deploying my application to a Windows Server 2012 R2 VM I found that the configuration values were all coming back blank. I have logging in place to show the identity of the process. I definitely put the environment variables in for that user. If I make the environment variables "System variables" then in fact my application does see the values.
Is there something weird about the way environment variables present themselves to the code that on a server wouldn't allow user env vars to be accessible? I was googling around to see if some group policy setting would affect this somehow.
Anyways, I'd prefer to use user environment variables to further obfuscate where sensitive info is stored. I know it's not going to prevent people from getting access to secrets, but it does lock things down another level (if the secrets are in system variables then all someone has to do is type SET at a command prompt to see them, but if they're in user variables of a special user account then you'd have to be signed in as that special user).
Any ideas?
Thanks!
Brian Eriksen

Are you running under IIS? IIS runs under it's own user and your "User variables" will not be accessible to your application.

Which user would IIS be running under? So the environment variables are read by IIS, and then transferred to the identity of the app pool?

In summary, IIS 7 and above offer the following:
The IUSR built-in account replaces the IUSR_MachineName account.
The IIS_IUSRS built-in group replaces the IIS_WPG group.

http://www.iis.net/learn/get-started/planning-for-security/understanding-built-in-user-and-group-accounts-in-iis


[SPARK-44017][PS][CONNECT][TESTS] Reorganize ps.Groupby unit tests

What changes were proposed in this pull request?
Reorganize ps.Groupby related unit tests, break them into several topics
Why are the changes needed?
to make them easy to maintain
Does this PR introduce any user-facing change?
No, test-only
How was this patch tested?
updated CI

cc @HyukjinKwon @itholic  all updated tests passed

Merged to master.


Add skip text for cancelling TapTargetSequence

@xiphirx I have had a merge request for TapTargetSequence. It will be better to have an option for cancelling it.

@prscX let's waiting for a new property https://github.com/prscX/react-native-app-tour

Thanks for taking the time to work on this. Unfortunately this does not conform to the material design guidelines, so I will not be merging it in. Sorry!


Support for Unity3D

Hi,
I currently use Unity which only supports .NET Standard 2.0 Dlls, is making the library compatible with  .NET Standard 2.0 on the plans?
Thank you!

Hey,
I have not personally tested this, but have you tried changing the Unity scripting runtime to 4.x?
I'll get back to this tomorrow if that doesn't solve it.

@Nominom thanks for your fast answer!
Yes, i also tried this, and I always end up with the following error:
Error: Could not load signature of BCnEncoder.Shared.Bc6Block:StoreIndices due to: Could not resolve type with token 0100001b (from typeref, class/assembly System.Span1, netstandard, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51) assembly:netstandard, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51 type:System.Span1 member:(null) signature:<none>
I feel like the System.Span  is not available in stable versions of Unity as mentioned here :   Unity Forum. It should be available in Unity 2022 but we're far from an stable release of this from unity.

Quick update, I could import the library opening and building the net4.5 branch of the repo. Still using .net standard 2.0 but it works this time! Thank you :)

Yeah, there's a Net45 version available. It just hasn't been updated for a while and is missing some features. If it works enough for you then you can use that. Let me know if you run into a problem with it.

@Nominom re-opening the issue because I don't manage to make the library work correctly on Unity using KTX+BC1.
Even though after a bit of battle the dlls load into Unity, I can't load the texture in Unity.
Here's an example code:
    public Texture2D loadedTexture;
    public string textureInput = @"C:\Users\Pierre\Desktop\example.png";
    public string textureOutput = @"C:\Users\Pierre\Desktop\example.ktx";

    void SaveTexture()
    {
        using SixLabors.ImageSharp.Image<SixLabors.ImageSharp.PixelFormats.Rgba32> image = SixLabors.ImageSharp.Image.Load<SixLabors.ImageSharp.PixelFormats.Rgba32>(textureInput);
        byte[] rgbaBytes;
        if (image.TryGetSinglePixelSpan(out var pixelSpan))
        {
            rgbaBytes = MemoryMarshal.AsBytes(pixelSpan).ToArray();
            BcEncoder encoder = new BcEncoder();

            encoder.OutputOptions.quality = CompressionQuality.Fast;
            encoder.OutputOptions.format = CompressionFormat.BC1;
            encoder.OutputOptions.fileFormat = OutputFileFormat.Ktx;

            using (FileStream fs = File.OpenWrite(textureOutput))
            {
                encoder.Encode(rgbaBytes, image.Width, image.Height, fs);
            }
        }

    }

    void LoadTexture()
    {
        BcDecoder decoder = new BcDecoder();
        using (FileStream fs = File.OpenRead(textureOutput))
        {
            var decompressedTexture = decoder.Decode(fs);
            loadedTexture = new Texture2D(decompressedTexture.Width, decompressedTexture.Height, TextureFormat.DXT1, false);
            loadedTexture.LoadRawTextureData(decompressedTexture.data);
            loadedTexture.Apply();
        }

    }

The texture loads using LoadRawTextureData (no crash or exception, so the arrays must be right-sized) but doesn't look like anything in Unity. I also had to use TryGetSinglePixelSpan to get the bytes array to create the .ktx file, maybe I'm doing something wrong on this side.
Any clue?
Thank you!

If I'm reading this correctly, looks like you're telling Unity that the input data would be in Dxt1 format, but the BcDecoder.Decode method will turn the given file back to Rgba format.
If you want the raw Dxt encoded data from the Ktx file, you can use KtxFile.Load to first load the file, then get the Dxt1 data from ktxfile.MipMaps[0].Faces[0].data
If you're looking to do this at run-time, there should be a EncodeToRawBytes method in the Encoder class to just get the Dxt1 data without saving it to a file first.

@Nominom thank you that's awesome, I managed to make it work and loaded a KTX DXT1 texture directly into Unity! One question though: is there some way to add some loss to the compression? For my textures, I'm currently using the crunch algorithm (.CRN) which compresses a 2K png texture into a 500kb file using DXT1 with a bit of loss. The same texture gives me a 2MB .KTX and I wish I could reduce this a bit. I tried the different settings (CompressionQuality) but none seem to have an noticeable effect.
Thanks again and here's some sample code for those coming after me.
    void SaveTexture()
    {
        using (SixLabors.ImageSharp.Image<SixLabors.ImageSharp.PixelFormats.Rgba32> image = SixLabors.ImageSharp.Image.Load<SixLabors.ImageSharp.PixelFormats.Rgba32>(textureInput))
        {
            if (image.TryGetSinglePixelSpan(out var pixelSpan))
            {
                byte[] rgbaBytes = MemoryMarshal.AsBytes(pixelSpan).ToArray();
                BcEncoder encoder = new BcEncoder();

                encoder.OutputOptions.quality = CompressionQuality.Fast;
                encoder.OutputOptions.format = CompressionFormat.BC1;
                encoder.OutputOptions.fileFormat = OutputFileFormat.Ktx;
                encoder.OutputOptions.ddsBc1WriteAlphaFlag = false;
                using (FileStream fs = File.OpenWrite(textureOutput))
                {
                    KtxFile file = encoder.EncodeToKtx(rgbaBytes, image.Width, image.Height);
                    file.Write(fs);
                }
            }
        }
    }

    void LoadTexture()
    {
        using (FileStream fs = File.OpenRead(textureOutput))
        {
            KtxFile file = KtxFile.Load(fs);
            KtxMipmap mipmap = file.MipMaps[0];
            byte[] rawData = mipmap.Faces[0].Data;
            loadedTexture = new Texture2D((int) mipmap.Width, (int)mipmap.Height, TextureFormat.DXT1, false);
            loadedTexture.LoadRawTextureData(rawData);

            // Uploads texture from CPU to GPU
            loadedTexture.Apply();
        }
    }

Hmm,
My first thought about the extra size was the MipMaps, but if you're using the Net45 version it does not support them. DXT1 is a fixed-size algorithm so the quality of the encoding does not affect the file-size either. It could be a bug in the Net45 version, or if you got the real version working, turn off the MipMap generation from encoder OutputOptions.
I can investigate it better in the weekend when I have a bit more free time.

@Nominom Unfortunately, I didn't manage to make the new version working with Unity because of still the same error, would be very appreciated to have the latest version compatible with Net4.x or Net Standard 2.0 as Unity only supports those. In any case, thank you very much for your help :)
TypeLoadException: Could not resolve type with token 0100001b (from typeref, class/assembly System.Span`1, netstandard, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51)
BCnEncoder.Shared.ImageFiles.KtxFile.Load (System.IO.Stream s) (at <44e4fb09b69c444a9c0353d346f2f4fb>:0)

I just did a quick calculation and the file size seems to be actually correct. One pixel takes up half a byte of storage with DXT1, so 2048x2048x0.5 should be about 2MB. Are you sure the crunch compression isn't doing anything else to the texture, like resizing, zipping or using a different algorithm?
You could also experiment with .NET's DeflateStream or GZipStream to try and bring the file-size down.

@Nominom After a bit of research, as mentioned in the Binomial doc, the CRN is an already highly compressed format.
Adding a strong GZip compression on top of my 2K KTX files brings the files down to 250/400kb, which is around the same thing as crunch. I'm closing the ticket as you answered all of the questions I had, and I thank you once again for your support !

I'm glad if I could help.
Adding full support for Standard 2.0 is currently quite of a hassle, so I'll just hope Unity hops on board the new .NET 5+ train soon. I might have to do it if enough people have a need for it though.
If you run into any more problems, don't hesitate to ask!


Put number of past participants on the web

Past Aalto workshop had 35 participants.

i'm thinking how to do this. On the workshops page, in parenthesis after each completed workshop? Or somewhere else? I don't see an immediately obvious place to put it

Somewhere on the overview page a sum of workshops and sum of participants.

Then in the sources we should keep track up to which workshop we have summed up so that we can update it in future.

which of these is better? one is more cluttered but shows more interesting scientific disciplines. difficult tradeoff...

I think they are not too cluttered - I would only shorten the long "engineering" line. Thank you for preparing these!

maybe it's even better to merge all the engineering disciplines, i.e. Mechanical, Electrical, Electronic, Information, Civil, Chemical, Materials Engineering, Other Engineering and Technologies, and in the same vein merge all medicine and health sciences. Like here:

this is solved in 1e0c71ab0ee9c466188f3de3055785596a3a6259
closing


SSO token access to /corp/StarbaseList.xml.asp

according to this  corporationStructuresRead doesn't map to 524288, which according to this  is needed to access /corp/StarbaseList.xml.asp.  It certainly seems like it should.
Technically, this isn't a documentation bug, it's a mapping bug, but I'm not sure where else to open this.  Feel free to redirect to the right place to get this fixed.

Can you check the actual mapping the XML API does when presented with that scope? I recall I added this mask to the scope mapping after the initial release of the SSO integration -- it's possible the documentation hasn't been updated yet.

Testing indicates it's not working.  A SSO with corporationStructuresRead  I'm using to read citadel data is generating the following when I attempt to access  /corp/StarbaseList.xml.aspx:
Error code 221: Illegal page request! Please verify the access granted by the key you are using!

What's the access mask presented to you by the /account/APIKeyInfo.xml.aspx endpoint? If you haven't already, you do need to specify the token's access type for corporation endpoints, that is accessType=corporation
https://eveonline-third-party-documentation.readthedocs.io/en/latest/xmlapi/authentication.html#sso-access-tokens

Never mind!  I was granting esi-corporations.read_structures, but not corporationStructuresRead.  Forgive my badness and thanks for pointing me in the correct direction.

Technically, I guess this is now a documentation bug though, since it doesn't mention 524288 in the mapping, which does seem to be working, so I'll re-open in the hopes of a documentation fix.

Documentation relating to this has been merged.


fix: Update windows-rs to 0.44

The immediate motivation is a CI failure in the Bevy AccessKit PR. While Bevy is indirectly depending on windows-rs 0.43, I was told we should go ahead and update to 0.44.

To me their job will still find two versions of windows-rs unless they update to 0.44 as well. Crossing fingers.


Move the stocks example to dev

Fixes https://github.com/flutter/flutter/issues/22718
The stocks example is no longer a good customer example as it's unmaintained and has a pretty uninspirational UI. We can't delete it as we use it for benchmark testing, so moving it to the dev/ folder.

Closing, now tracking in https://github.com/flutter/flutter/issues/22718

Closing, now tracking in #22718


Feature request: print read ids of reads supporting variant

Is it possible to dump the read ids corresponding to those reads which supported a variant from nanopolish variants? I would like to use this for haplotype phasing.

One possible workflow is to call variants with nanopolish variants, then calculate read support using phase-reads. That is a bit roundabout though.
If I add this to nanopolish variants what output format would you want? I could put it in the VCF INFO tags but the lines would be very long.

I did play around with phase-reads but even then this only calls SNPs, where preferably I would have everything.
I agree it's probably a bit long to include in the VCF file, even if it's optional. Would it make more sense to optionally spit out another file in a similar form to what call-methylation does?
chromosome      pos     read_name       log_lik_ratio   log_lik_ref      log_lik_alt    REF .    ALT


Add Dockerfile for tuf server

Currently the server is built using ko. For RHTAP, we need this to be built with a Dockerfile.

/assign

The Dockerfile should be named uniquely at the root. E.g. Dockerfile.tuf-server.

Sounds good, I will try it like that.


Aligned RGB and thermal pairs

Hi, authors!
It seems that the RGB and thermal images are not aligned after running ns-process-data. I am wondering how I could obtain the aligned RGB and thermal pairs so that I could do some experiments on this dataset.
Many thanks!

Hi, thanks for the interest in our work! I am not sure if this addresses your question, but please note it is expected for the RGB and thermal images after being extracted from the FLIR images to appear visually slightly non-aligned (e.g. not from exactly the same perspective) due to the different camera parameters of the RGB and thermal cameras. This should not be an issue in practice due to the calibration process which computes the camera parameters of the RGB and thermal cameras, accounting for this discrepancy. Please let me know if I've misunderstood your issue and can answer your question differently!

It answers my question. Thank you!


iOS 10.2 [NSConcreteMutableData appendBytes:length:]: unable to allocate memory for length

I have similar issue

Same here

same here

Hard to say what’s going on from that screenshot; there isn’t any AFNetworking code there. Is this reproducible? If so, please re-open with more information.


Convert the felts' internal representation to Montgomery form.

Felts currently use a normal (binary) internal representation. Switching to Montgomery form should allow significant performance improvements on non-trivial operations like multiplication.

Hey @azteca1998, can I try working on this?


Create package setup

Prepare setup.py for the language server packaging and some minor updates on documentation.

:balloon:


Feature Request: ability to output into parsable format?

Hi, I played with swift-ast, but I don't see any way to somehow parse AST output. Is there an option to dump e.g. json of AST? Thanks

Maybe someone can leverage https://github.com/yanagiba/swift-transform and work on conversations to perhaps XML and/or json?

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates.
Please make sure to update to the latest version and check if that solves the issue. Let us know if that works for you by adding a comment.

This issue will be auto-closed because there hasn't been any activity for a few months. Feel free to open a new one if you still experience this problem.


Use the project Identity Path to identify project dependencies

The identity path is the "true" identifier of a project within the build tree. Use this identifier when referencing other projects during publishing.

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@jvandort Can you check that this doesn't have an impact on GE?

How do you propose doing so?

@big-guy I assume you're specifically mentioning this due to @UsedByScanPlugin?

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

Pre-tested commit build failed.

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

I've triggered a build for you. Click here to see all failures if there's any.

Pre-tested commit build failed. Click here to see all build failures.

@bot-gradle test apt

I've triggered the following builds for you. Click here to see all build failures.

AllPerformanceTestsReadyForRelease build

@bot-gradle test and merge

I've triggered a build for you. Click here to see all failures if there's any.


Update sccache bucket

This PR updates the sccache configuration settings to use a new bucket, rapids-sccache-east.
Unlike the previous rapids-sccache bucket, rapids-sccache-east resides in the same AWS region as the rest of our CI infrastructure (us-east-2).
This should result in faster, more reliable sccache connections and will also help keep our data transfer costs down.
Important Note: The changes from this sccache PR are required to use a bucket in us-east-2. These changes were incorporated in sccache v0.3.2, so you'll need that version or later (preferably the latest, v0.3.3). Please ensure that your CI images satisfy this constraint before merging this PR.
Additionally, this PR updates the SCCACHE_REGION to the region of the bucket. This value is not meant to be a variable.

https://github.com/nv-morpheus/MRC/pull/290

/merge


Add service account authentication in bigframes read_gbq method like in pandas-gbq package

Very simple in pandas-gbq:
from google.oauth2 import service_account
import pandas_gbq

credentials = service_account.Credentials.from_service_account_file('
    path/to/key.json'
)
df = pandas_gbq.read_gbq(query, project_id="YOUR-PROJECT-ID", credentials=credentials)

Thanks for the feature request!
This is currently possible using the bigframes.options.bigquery.credentials property:
from google.oauth2 import service_account
import bigframes.pandas as bpd

credentials = service_account.Credentials.from_service_account_file('
    path/to/key.json'
)
bpd.options.bigquery.project = "YOUR-PROJECT-ID"
bpd.options.bigquery.credentials = credentials
df = bpd.read_gbq(query)

That said, I think it makes sense to try and stay compatible with pandas-gbq usage. We already do something similar to this for autodetecting the location when read_gbq is the first call in a new BigQuery DataFrames session.

@tswast When I run this in VSCode Jupyter notebook it throws the following error:
---> [64]  if self._local.bigquery_options is not None:
[65]      # The only way we can get here is if someone called
[66]     # _init_bigquery_thread_local.
[67]      return self._local.bigquery_options
[69]   return self._bigquery_options
AttributeError: '_thread._local' object has no attribute 'bigquery_options'
Any idea what causes this?


jQuery reverted, cookie concent, reCAPTCHA & passmeter

Reverted back to jQuery 2.2.4, fixes #635 .
@sahat I added reCAPTCHA, cookie concent and passmeter. Let me know if you don't agree having these features in the boilerplate.

@elarb I appreciate the PR. The jQuery revert is no longer required with the latest Bootstrap 3.5.7 version. As for other additions, I am going to have to politely decline them. Great for some use cases, but not extremely important to have in this boilerplate.


Reshape error (num_input_elements != num_output_elements) when using LSTM layer on ESP32

I exported a TFLite model containing a 40-cell LSTM layer:
lstm_100 (LSTM)              (None, 40)                8160
dense_1300 (Dense)           (None, 11)                451
In TFLite Micro on the ESP32, I get the following errors during AllocateTensors();:
lib/tflite-lib/tensorflow/lite/micro/kernels/reshape.cc:76 num_input_elements != num_output_elements (560 != 40)
lib/tflite-lib/tensorflow/lite/micro/kernels/reshape.cc:76 num_input_elements != num_output_elements (560 != 40)
lib/tflite-lib/tensorflow/lite/micro/kernels/reshape.cc:86 ReshapeOutput(context, node) != kTfLiteOk (1 != 0)
Node RESHAPE (number 7f) failed to prepare with status 1
I am using this tflite-micro version (master branch)

Have you solved this problem ?

Unfortunately I could not get it to work.


Fix: Event property name

Fixes Android build failing because of #35.

Good catch, @dafrie . I'll ask my teammates to merge as I'm not in charge of the React Native project. @yuhao900914 Could you merge?

Hi @dafrie,
Just release the new version v2.0.6 to npm, please check it out.
Thanks for the contribution.


Wildcard in index name

dump v5.0.1 , elastic 5.0.2
For massive operations, you have to support multi index syntax. There's just no other way. curator does it already.
./bin/elasticdump --bulk --limit 5000  --type data --input 'http://xx:9200/smtp-*-2016-*' --output http://xx:9200/

The goal of elasticdump is to create backup files per index.  You can use MultiElasticdump which is included in this package to do more than one index in parallel.
The case you describe would be: ./bin/multielasticdump --limit 5000 --type data --input 'http://xx:9200' --match 'smtp-*-2016-*' --output 'http://xx:9200'

Yeah, this use case should really be documented. Like: If you need multiindex syntax, use this binary/version.

Also, it doesn't work:
$ ./bin/multielasticdump  --limit 500  --type data --input http://xx:9200 --match 'smtp-transaction-2016-10-*' --output http://xx:9200/ | tee -a log.txt
Wed, 07 Dec 2016 12:54:54 GMT | We are performing : dump
Wed, 07 Dec 2016 12:54:54 GMT | options: {"debug":true,"parallel":4,"match":"smtp-transaction-2016-10-*","input":"http://xx:9200","output":"http://xx:9200/","scrollTime":"10m","limit":500,"offset":0,"direction":"dump"}
/home/kopf/elasticsearch-dump/bin/multielasticdump:76
  if (!fs.existsSync(options.output)) { throw new Error('--output does not exist') }
                                        ^

Error: --output does not exist
    at Object.<anonymous> (/home/kopf/elasticsearch-dump/bin/multielasticdump:76:47)
    at Module._compile (module.js:570:32)
    at Object.Module._extensions..js (module.js:579:10)
    at Module.load (module.js:487:32)

If you want to add more to the readme about this use case, please send in a pull request!
Can you un-obfuscate your output URL?    if (!fs.existsSync(options.output)) { throw new Error('--output does not exist') } indicates that elasticdump thinks your output is a file descriptor not another URL.  There may be something wrong here if that is the case https://github.com/taskrabbit/elasticsearch-dump/blob/master/lib/is-url.js

Oh! It looks like multielasticdump only supports dumping to files.  You will need to dump locally to files and then to your new cluster with this tool.  My mistake!

Here's a quick python script that does what you're trying to do - copy indices mapping & data with a wildcard.
import subprocess

host_input='https://input-host.com:9200'
host_output='https://output-host.com:9200'
es_index='skynet-*'

# Get list of indices
output = subprocess.check_output("curl -s -XGET "+host_input+"/_cat/indices/"+es_index+" | awk '{print $3}'", shell=True)

# Loop through list of indices
for index in output.splitlines():
	print "STARTING MAPPING: "+index
	subprocess.call("docker run --rm -i taskrabbit/elasticsearch-dump \
		--input="+host_input+"/"+index+" \
		--output="+host_output+"/"+index+" \
		--type=mapping", shell=True)

	print "STARTING DATA: "+index
	subprocess.call("docker run --rm -i taskrabbit/elasticsearch-dump \
		--input="+host_input+"/"+index+" \
		--output="+host_output+"/"+index+" \
		--type=data", shell=True)


Field long_text AskAwesomely::Field::LongText does not exist

AskAwesomely::FieldTypeError (Field long_text <AskAwesomely::Field::LongText> does not exist, please use one of: short_text, long_text, multiple_choice, picture_choice, statement, dropdown, yes_no, number, rating, opinion_scale, email, website, legal):
The offending code:

  field :long_text do
    say "xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx"
    tags :field_name1
    ref :field_name1
  end

I think this is because I implemented say as a method only for statement fields (which makes a NoMethodError). The new release (0.2.8) changes this so you can use say or ask in any field.

Thanks @leemachin


feat: add response and json response body processor

Closes #480

I still have to implement forceResponseBody

At some point we need to do the JSON as lazy load maybe to avoid parsing the entire JSON if not using all fields.

Let's first merge https://github.com/corazawaf/coraza/pull/679


[OpenID Connect (OIDC)] fix prompt=login login api query parameter

Context: Authentication with OpenID Connect (OIDC) identity layer
I need renew parameter in the login url, so I use prompt query parameter with 'login' as value.
prompt parameter is used to rewrite login url but is never removed from original url. It throws infinite loop on login.
In this PR I remove prompt parameter before redirect url rewrite (containing renew parameter). The parameter is already used by the pre-handler to know if the parameter renew is added or not to redirect url.
This PR fixes infinite login loop.

Who can review this path?

Thank you very much for the patch. I'll try to set aside some time next week to review this. Look pretty legit to me so far.

While I think this does the job, I dont think the file you are modifying here is the right place; prompt parameter is a concept in OIDC and the OAuth module shouldn't try to handle it. A better place for you to remove the prompt would likely be in OidcCallbackAuthorizeViewResolver#resolve(). Something like:
       if (prompt.contains(OidcConstants.PROMPT_LOGIN)) {
            val newUrl = ... // remove the prompt
            return new ModelAndView(new RedirectView(newUrl));
        }

OidcCallbackAuthorizeViewResolver

Right :) I'll fix it

@mmoayyed  When I read my code, I'm thinking about the parameter position. I remove "&prompt=login", but if the parameter is in the first position, it's "?prompt=login". My question is, have I to take care of the order or the propt parameter never appears at the first position.

Order doesn't and shouldn't matter. As long as you remove it, it should be fine.
You can do something like this:
public static String removeOidcPromptFromAuthorizationRequest(final String url, final String prompt) {
        val uriBuilder = new URIBuilder(url);
        val newParams = uriBuilder.getQueryParams()
            .stream()
            .filter(p -> !OidcConstants.PROMPT.equals(p.getName()) || !p.getValue().equalsIgnoreCase(prompt))
            .collect(Collectors.toList());
        if (!newParams.isEmpty()) {
            return uriBuilder.removeQuery()
                .addParameters(newParams)
                .build()
                .toASCIIString();
        }
        return url;
    }

...and then:
       if (prompt.contains(OidcConstants.PROMPT_LOGIN)) {
            LOGGER.trace("Removing login prompt from URL [{}]", url);
            val newUrl = OidcAuthorizationRequestSupport.removeOidcPromptFromAuthorizationRequest(url, OidcConstants.PROMPT_LOGIN);
            LOGGER.trace("Redirecting to URL [{}]", newUrl);
            return new ModelAndView(new RedirectView(newUrl));
        }

Order doesn't and shouldn't matter. As long as you remove it, it should be fine.
You can do something like this:
public static String removeOidcPromptFromAuthorizationRequest(final String url, final String prompt) {
        val uriBuilder = new URIBuilder(url);
        val newParams = uriBuilder.getQueryParams()
            .stream()
            .filter(p -> !OidcConstants.PROMPT.equals(p.getName()) || !p.getValue().equalsIgnoreCase(prompt))
            .collect(Collectors.toList());
        if (!newParams.isEmpty()) {
            return uriBuilder.removeQuery()
                .addParameters(newParams)
                .build()
                .toASCIIString();
        }
        return url;
    }

...and then:
       if (prompt.contains(OidcConstants.PROMPT_LOGIN)) {
            LOGGER.trace("Removing login prompt from URL [{}]", url);
            val newUrl = OidcAuthorizationRequestSupport.removeOidcPromptFromAuthorizationRequest(url, OidcConstants.PROMPT_LOGIN);
            LOGGER.trace("Redirecting to URL [{}]", newUrl);
            return new ModelAndView(new RedirectView(newUrl));
        }

Nice! I'll do that and probably add some tests.


Step group template

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Preview environment: https://ci.pr.harness.io/pr-625

Preview environment: https://ci.pr.harness.io/pr-625

Preview environment: https://ci.pr.harness.io/pr-625

Preview environment: https://ci.pr.harness.io/pr-625

Preview environment: https://ci.pr.harness.io/pr-625


Analysis tab - View PCA section

Just as in Studio, this section should include the following:

A slider (labeled "Standard deviation") with 11 ticks. It starts in the middle (at 0 standard deviations away from the mean). The render window will show one renderer with the mean shape. As the user moves the slider (to the left for n standard deviations below the mean and to the right for n standard deviations above the mean), the shape in the single renderer will instead show the mesh corresponding to that standard deviation.

The mesh is found as the file value on the current pca_ value object for the current mode. The current pca_value object will have a value equaling the number of standard deviations from the mean.

A toggle (labeled "Animate"). When enabled, the standard deviation slider will slide back and forth on its own.
A selector (labeled "Mode"), which determines the current mode.
A display of the current lambda value. The lambda value is also found on the current pca_value object.
A display of the current eigenvalue, explained_variance, and cumulative_explained_variance values. These are found on the current mode.

We also want this section to include the toggle for "show difference to mean". It enables a visualization mode, tracked in #192.


Fixing internal calls of get_sample_cn_segments

This PR resolves errors related to internal calls of get_sample_cn_segments, calling this function with now deprecated parameters. Affected functions are assign_cn_to_ssm and collate_pga.

Before merging this PR, could you fix the commit title, please?
Change get_sample_cn_states to get_sample_cn_segments.


Delete "basic access control" and "secure access control" pages.

These pages were there before Istio RBAC was introduced. We should
remove them now to avoid confusion.

Please add aliases in some other .md file in order to redirect any bookmarks to these deprecated files.

@geeknoid I did a search in the repo. I did not find other pages referencing these two pages.

I meant bookmarks that people may have to those pages. We want backwards compatibility for people's bookmarks. See https://preliminary.istio.io/about/contribute/writing-a-new-topic/#renaming-or-moving-pages
Basically, if someone had bookmarks to those pages, where should they land instead? Go to where they should land and add "aliases" in the front matter of the page.

@geeknoid I added aliases for the deleted pages. Thanks.


Modify the request body in sequential call with dynamic value

I am implementing a sequential call which include two API.They are API1 and API2.API 1 is Get and API2 is post call .
The initial request will route to API 1, After it is successfully executed, The request will route to API2.I want to set the request body of API2 with the response of API1..For that , i am doing following configuration .
Here xyz is the key which i am setting in API1 response .
"github.com/devopsfaith/krakend-martian":
{
"body.Modifier": {
"scope": ["request","response"],
"body": "{resp0_xyz}"
}
}
}
Can you please advise how to achieve this .

This is already answered in https://github.com/devopsfaith/krakend-ce/issues/422. Closing the issue