[BUG] model_config prefix is changed in payload for ML inference processors

What is the bug?
since this change in https://github.com/opensearch-project/ml-commons/commit/7cd52915d04d8ac7ddb6e37a74a256603587ce69 in  ModelExecutor, the model_config prefix is changed in payload.
before this commit,  when using the model_config parameters, the parameter are usually prefix with parameters, for example parameters.prompt.
after this commit, when using the model_config parameters, the parameter will be prefix with 'model_config', for example, model_config.prompt.
now that using ml inference processor with model_config, on both search request, response and ingest, for remote model, we need to set model_input to rewrite the prefix, or the model_config won't be recognized by the connector response body
The pros about the parameters prefix is that it can identify the same parameters name within input_map and model_config when compiling model_input. However, for remote models, the inference processor passed over config such as  model_config.prompt,  it won't be recognized in the response body, because in the connector, all parameters are prefix with parameter, for example parameters.prompt.
How can one reproduce the bug?
Steps to reproduce the behavior:
POST /_plugins/_ml/connectors/_create
{
  "name": "Amazon Bedrock Connector: Claude Instant V1",
  "version": "1",
  "description": "The connector to bedrock Claude model",
  "protocol": "aws_sigv4",
  "parameters": {
    "max_tokens_to_sample": "8000",
    "service_name": "bedrock",
    "temperature": "1.0E-4",
    "response_filter": "$.completion",
    "region": "us-west-2",
    "anthropic_version": "bedrock-2023-05-31",
    "inputs":"please summerize the documents"
  },
  "credential": {},
  "actions": [
    {
      "action_type": "PREDICT",
      "method": "POST",
      "url": "https://bedrock-runtime.us-west-2.amazonaws.com/model/anthropic.claude-instant-v1/invoke",
      "headers": {
        "x-amz-content-sha256": "required",
        "content-type": "application/json"
      },
      "request_body":  "{\"prompt\":\"${parameters.prompt}\",\"max_tokens_to_sample\":300,\"temperature\":0.5,\"top_k\":250,\"top_p\":1,\"stop_sequences\":[\"\\n\\nHuman:\"]}"
    }
  ]
}

PUT /review_string_index/_doc/1
{
  "review": "Dr. Eric is a fantastic doctor",
  "label":"5 stars"
}

PUT /review_string_index/_doc/2
{
  "review": "happy visit" ,
  "label":"5 stars"
}

PUT /review_string_index/_doc/3
{
  "review": "sad place" ,
  "label":"1 stars"
}

example 1, this is failing:
PUT /_search/pipeline/my_pipeline_request_review_llm
{
  "response_processors": [
    {
      "ml_inference": {
        "tag": "ml_inference",
        "description": "This processor is going to run llm",
        "model_id": "cf46K5EBoVpekzRp8x_3",
        "function_name": "REMOTE",
        "input_map": [
          {
            "context": "review"
          }
        ],
        "output_map": [
          {
            "llm_response": "response"
          }
        ],
        "model_config": {
          "prompt":"\n\nHuman: You are a professional data analysist. You will always answer question based on the given context first. If the answer is not directly shown in the context, you will analyze the data and find the answer. If you don't know the answer, just say I don't know. Context: ${input_map.context}. \n\n Human: please summarize the documents \n\n Assistant:"
        },
        "ignore_missing": false,
        "ignore_failure": false
      }
    }
  ]
}

example 2, this is success when we rewrite the model_config.prompt into parameters.prompt
PUT /_search/pipeline/my_pipeline_request_review_llm
{
  "response_processors": [
    {
      "ml_inference": {
        "tag": "ml_inference",
        "description": "This processor is going to run llm",
        "model_id": "cf46K5EBoVpekzRp8x_3",
        "model_input": "{ \"prompt\": \"${model_config.prompt}\"}",
        "function_name": "REMOTE",
        "input_map": [
          {
            "context": "review"
          }
        ],
        "output_map": [
          {
            "llm_response": "response"
          }
        ],
        "model_config": {
          "prompt":"\n\nHuman: You are a professional data analysist. You will always answer question based on the given context first. If the answer is not directly shown in the context, you will analyze the data and find the answer. If you don't know the answer, just say I don't know. Context: ${input_map.context}. \n\n Human: please summarize the documents \n\n Assistant:"
        },
        "ignore_missing": false,
        "ignore_failure": false
      }
    }
  ]
}

What is the expected behavior?
example 1 should be succeed.
What is your host/environment?

OS: mac
Version 2.15
Plugins

Do you have any screenshots?
If applicable, add screenshots to help explain your problem.
Do you have any additional context?
Add any other context about the problem.

maybe one way to fix this bug is to move the reconstructing payload within local model only @rbhavna please share your thoughts in this fix?
move
https://github.com/opensearch-project/ml-commons/blob/9663053d544b59ef238ae36bf49b774543552df5/plugin/src/main/java/org/opensearch/ml/processor/ModelExecutor.java#L84-L80

This issue is blocking using custom prompt in the ml inference response processor,
I have another idea to make this genAI use case to work,
when the model_config contains the the field from input_map and output_map, we should try to cast the datatype into escape string.
for example, ${input_map.context} in going to be in the substring of ${model_config.prompt}, we can try cast the ${input_map.context}   into escape string or it won't be a valid payload later.
@rbhavna can you try this case using the latest main branch and reproduce this issue?
PUT /_search/pipeline/my_pipeline_request_review_llm
{
  "response_processors": [
    {
      "ml_inference": {
        "tag": "ml_inference",
        "description": "This processor is going to run llm",
        "model_id": "cf46K5EBoVpekzRp8x_3",
        "model_input": "{ \"prompt\": \"${model_config.prompt}\"}",
        "function_name": "REMOTE",
        "input_map": [
          {
            "context": "review"
          }
        ],
        "output_map": [
          {
            "llm_response": "response"
          }
        ],
        "model_config": {
          "prompt":"\n\nHuman: You are a professional data analysist. You will always answer question based on the given context first. If the answer is not directly shown in the context, you will analyze the data and find the answer. If you don't know the answer, just say I don't know. Context: ${input_map.context}. \n\n Human: please summarize the documents \n\n Assistant:"
        },
        "ignore_missing": false,
        "ignore_failure": false
      }
    }
  ]
}


Adds EwDMET bath

Reimplements EwDMET bath for RHF. Use as bath_options=dict(bathtype='ewdmet', order=2, threshold=1e-8)

Codecov Report
Base: 67.32% // Head: 67.32% // Decreases project coverage by -0.00% :warning:

Coverage data is based on head (65e4f6b) compared to base (bf5da7e).
Patch coverage: 20.51% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #61      +/-   ##
==========================================
- Coverage   67.32%   67.32%   -0.01%
==========================================
  Files         108      108
  Lines       16406    16402       -4
  Branches     2386     2329      -57
==========================================
- Hits        11045    11042       -3
- Misses       4701     4704       +3
+ Partials      660      656       -4

Impacted Files
Coverage Δ

vayesta/core/qemb/qemb.py
67.75% <ø> (ø)

vayesta/core/qemb/fragment.py
75.60% <20.00%> (-1.00%)
:arrow_down:

vayesta/core/bath/ewdmet.py
26.53% <20.68%> (+4.11%)
:arrow_up:

vayesta/solver/solver.py
51.48% <0.00%> (-0.50%)
:arrow_down:

vayesta/ewf/ewf.py
64.25% <0.00%> (-0.24%)
:arrow_down:

vayesta/core/vlog.py
81.10% <0.00%> (-0.15%)
:arrow_down:

vayesta/mpi/rma.py
28.66% <0.00%> (ø)

vayesta/misc/cubefile.py
21.62% <0.00%> (ø)

vayesta/mpi/interface.py
40.00% <0.00%> (ø)

... and 2 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Display new section slug in ManualRelocator

I'm fairly confident that this message should print the old
(section.slug) and new (new_section_slug) slugs instead of the
printing the old slug twice.
Have run the script I can see that the section slug is indeed updated to
new_section_slug.
This appears to have been incorrect since the ManualRelocator class
was added in 43b669600d3ddc30fd9fccb7bdbe1e4b4ecf9887.

Thanks for the review, @floehopper. I've addressed your feedback, rebased on master and force pushed in preparation for merging.


Android banner ads, fix center align in fullscreen, landscape mode.

This fix, considers the device full width during full screen landscape mode, to correctly align the banner ad to the center.
Issue #270

Added missing imports.


[DISCUSS] e2e test cases

We have the following groups of e2e cases.

Annotations

Test wether the Annotations in ApisixRoute are normal.

Proxy

Test the proxy result to upstream.

Plugins

Test each plugin, just like APISIX.

TLS/SSL

Test TLS/SSL certificates.

ingress

Test the behavior of ingress controller, like bad CRDs.

Our current e2e test case has been basically covered, and I will close this issue. Feel free to reopen it, if you have any question.


Document new rest api - getLastMessageId

We introduced a new rest endpoint getLastMessageId at #3196. We should also update the website about this new api

we need to add the documentation to http://pulsar.apache.org/docs/en/admin-api-persistent-topics/

I will do that later.

refer https://github.com/apache/pulsar/pull/3243


Add PHP to jenkins

just a test

[run ci]

retest this please

retest this please

retest this please


Multibuild discussion

Ref #8134
Ref #8134
Ref matthew-brett/multibuild#71
Ref matthew-brett/multibuild#70
Ref matthew-brett/multibuild#58

Sorry about this guys.

/cc @matthew-brett

We should just pin the multibuild version to a known-good commit hash.

In my understanding the reasons for using multibuild are:

it can install different python.org versions
multibuild is used to build lots of scipy wheels for OSX.

Just throwing it out here - the first issue could be tackled by using miniconda in the travisCI for OSX.

There's actually no need to use multibuild in this case because brew can install Python. I'll reopen that PR.

That's #8134. Merge if you agree with the concept and it works. Otherwise, just close it.

Well, it did pass the tests: https://travis-ci.org/scipy/scipy/jobs/299298821

There's actually no need to use multibuild in this case because brew can install Python. I'll reopen that PR.

I don't think you can specify which version of Python to install with brew, you can with multibuild and conda.
pv's suggestion to use a specific commit seems reasonable.

I don't think you can specify which version of Python to install with brew, you can with multibuild and conda.

Correct, but we're not testing that.

Correct, but we're not testing that.

At the moment.

I don't think we would test that in the future either, esp. considering the Travis CI mac situation.

This is where the multibuild aspect has it's advantages. We're dogfooding, the CI tests are using the same kinds of processes that we use to build the scipy wheels.

That's actually a valid concern. I'm not sure this would have been caught otherwise.

And on that point, pinning to a specific commit maybe isn't a good idea; numpy causes test failures sometimes.

Possibly setting the branch to master would help though. That's one option.

That's certainly the safer option.

@matthew-brett Any idea what caused the failures?


macos universal build failed

universal build (CMAKE_OSX_ARCHITECTURES=x86_64;arm64) failed due to :
if(CMAKE_OSX_ARCHITECTURES)
    # If multiple architectures are requested (universal build), pick only the first
    list(GET CMAKE_OSX_ARCHITECTURES 0 ARCH)
elseif(MSVC)
...

We don't support compiling for multiple targets, so this was explicitly added to disallow it.

See #1085 and #1016.


Bugfix the render instance without workfile being saved

Changelog Description
Resolve #3
Testing Notes

Launch Blender
Create Render Instance
It should be saving the new scene for creating render instance if there is no workfile saved.

I am not sure if it is the best solution of versioning up the current workfile (so that's why it is at a draft) so any better tweaks are welcome.


310 bug cannot unload model on nitro 022

What this PR will fix:

Fix the wrong way to include clip (in previous version need manually now it's properly specified in cmakelist of llava)
Fix the CV inside new llama cpp update, we can revisit this later on for a bit of extra performance, right now it's better just do busy waiting using while loop


AUTHORS: Adding Brian Redbeard to AUTHORS

Contributor checklist

[X] I have read the CONTRIBUTING.rst
[X] I have tested my contribution on these devices:

N/A

[X] My contribution is ready to be merged as is

Description
As per CONTRIBUTING.rst including a commit for Brian 'redbeard' Harrington to AUTHORS

Throwing up my hands.  The script exits zero on my machine:
[~/Projects/python-escpos]$ git checkout origin/pr/353
Note: checking out 'origin/pr/353'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by performing another checkout.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -b with the checkout command again. Example:

  git checkout -b <new-branch-name>

HEAD is now at fb82e5d AUTHORS: Adding Brian Redbeard to AUTHORS
[~/Projects/python-escpos]$ ./doc/generate_authors.sh --check
\nAuthorsfile in version control:\n
Ahmed Tahri
akeonly
Alex Debiasio
Asuki Kono
belono
Brian 'Redbeard' Harrington
Christoph Heuel
Cody (Quantified Code Bot)
csoft2k
Curtis // mashedkeyboard
Davis Goglin
Dean Rispin
Dmytro Katyukha
Gerard Marull-Paretas
Hark
Joel Lehtonen
Justin Vieira
kennedy
Kristi
ldos
Lucy Linder
Manuel F Martinez
Michael Billington
Michael Elsdörfer
mrwunderbar666
Nathan Bookham
Omer Akram
Patrick Kanzler
primax79
Qian Linfeng
Ramon Poca
reck31
Renato Lorenzi
Romain Porte
Sam Cheng
Sergio Pulgarin
Stephan Sokolow
Thijs Triemstra
Thomas van den Berg
ysuolmai
\nNew authorsfile:\n
Ahmed Tahri
akeonly
Alex Debiasio
Asuki Kono
belono
Brian 'Redbeard' Harrington
Christoph Heuel
Cody (Quantified Code Bot)
csoft2k
Curtis // mashedkeyboard
Davis Goglin
Dean Rispin
Dmytro Katyukha
Gerard Marull-Paretas
Hark
Joel Lehtonen
Justin Vieira
kennedy
Kristi
ldos
Lucy Linder
Manuel F Martinez
Michael Billington
Michael Elsdörfer
mrwunderbar666
Nathan Bookham
Omer Akram
Patrick Kanzler
primax79
Qian Linfeng
Ramon Poca
reck31
Renato Lorenzi
Romain Porte
Sam Cheng
Sergio Pulgarin
Stephan Sokolow
Thijs Triemstra
Thomas van den Berg
ysuolmai
\nUsing diff on files...\n
[~/Projects/python-escpos]$ echo $?
0

Hi @brianredbeard,
first of all thank you for your contribution! :-)
I am a bit confused now, which PR is the right one, because this one does not contain code. How about you create or reopen the right one and then we will see into the authors-file-issue. (The current solution is not optimal, but in case nothing else works, I can push the right file into your branch)


More Salamander CVar Changes

Allows command staff to early launch the shuttle, and also enables biomass hard mode.
This should be 75% less controversial than the last PR, hopefully

i approve this if someone else does ill merge


Orange eco-conception guidelines review

eco-conception guidelines review
General description
The Orange Eco-Conception team started to write some guideline. They are at a very first draft stage and are still being updated.
The objective is to perform an assessment of what exists today and identify:

what could be clarified
what could be updated
what is missing
what we should do in the current ODS guidelines to be more compatible with those guidelines

Detailed information
Contact: Maria-Jose PRESSO & Pierre RUST
Documentation: https://recommendations-preview.innov.intraorange/software-ecodesign-mobile-applications/
Implementation Tasks

[x] review the existing guideline
[x] provide feedback on the assessment
[ ] list further actions in new git issues

Analysis:
Network communications
MOBILE-RADIO-01: Don't make periodic polling : out of scope for design system
MOBILE-RADIO-02: Don't use persistent connection : out of scope for design system
CPU : use batching task each time you need a specific processing like processing data or photo : out of scope for design system
Background state : optimize background state to limit resources usage (CPU, network, memory) : out of scope for design system
Low power mode : support low power mode when activated by the user, example : remove animations : can be illustrated in the demo app
Support dark mode : already done in the demo app
Measure energy impact :  out of scope for design system (not relevant for demo app)
Use of assets - 1 : optimize assets, prefer vector format (PDF, SVG), when it’s possible: already applied in the demo app
Use of assets - 2: re-use SFSymbols when it’s possible: can be illustrated in the demo app
Use of animations - 1 : avoid complex animation, limit the number of animation and prefer small animation of icons:  can be illustrated in the demo app
Use of animations - 2 : prefer optimized framework for animation like ‘Lottie’ or ‘Core Animation’ (native):  can be illustrated in the demo app
Webviews - 1 : use optimized content for your web views - you can check it by using tools like PageSpeed Insight: out of scope for design system (not relevant for demo app)
Webviews - 2 : stop using UIWebView and move to WKWebView or SFSafariViewController:  already done in the demo app (module « About » )
Use of analytics : When needed, choose an analytics tool with minor impact on resources (size of library, number of request calls, memory usage, …) and minimize the number of analytics in your app : out of scope for design system (not relevant for demo app)

In addition , more information about the size of some applications:
Size of the APP (TestFlight):

ODS Qualif : 20,5 Mo (v0.5.0)
ODS Prod : 20,3 Mo (v0.4.1)
Orange Telephone v4.0.0 : 40,6 Mo

On Apple Store:

Orange et Moi v8.9.0 : 98,2 Mo
Orange Pro v7.10.0 : 106,4 Mo
TV d’Orange v9.7.2 : 113,5 Mo
Ma Livebox v8.14.3 : 99,5 Mo
Mail Orange v3.32.2 : 63,5 Mo
Orange Telephone v3.6.0 : 50,3 Mo
Orange Bank v3.9.3 : 91,8 Mo
Orange Business Lounge v6.10 : 38,3 Mo
Orange Money Afrique : v5.07.04 : 17,6 Mo
Apple Mail: ???
Apple calendrier : ???
Apple Telephone : ???

It seems that Apple doesn't want to share about apple app's version and size ....

Great  feedback, I can see 4 actions we need to list in our backlog to see how to be dealt with


[bug] unable to type numbers in the fields on the swap page

On the swap page, you're able to type text, but no numbers.
I've rolled back commit a5b3c4f9c66b771dad74ec8145bc864609626ab8 on teia.rocks and that fixes it.
I'm not smart enough to debug this further.

On the objkt quantity and price fields, only letters are accepted.

Thanks for the report!!
Fixed in #129 / #130

Doesn't work on https://teia-ui-main.onrender.com (and was also broken on my mirror, I put back the old fix from #132 )


@conda-forge-admin, please rerender¶

Additional comment:
No response

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I rerendered the recipe in conda-forge/sunode-feedstock#9.


magic-wormhole: fix test for twisted 24.10.0

Description of changes
nixpkgs-review result
dependency of tahoe-lafs is already broken on master
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:x: 4 packages failed to build:

tahoe-lafs
tahoe-lafs.dist
tahoe-lafs.doc
tahoe-lafs.info

:white_check_mark: 6 packages built:

gnome-keysign
gnome-keysign.dist
magic-wormhole (python312Packages.magic-wormhole)
magic-wormhole.dist (python312Packages.magic-wormhole.dist)
python313Packages.magic-wormhole
python313Packages.magic-wormhole.dist

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Thanks!


List App Service SKU

Users don't want to know the SKU of the Azure App service plan.
Expose a more readable format to users to choose the most suitable consumption plan.
Pricing information would be a plus.

Pricing: https://prices.azure.com/api/retail/prices?$filter=serviceName eq 'Azure App Service' and contains(productName, 'Linux')

serviceName eq 'Azure App Service' and armRegionName eq 'northeurope'


x/build/cmd/release: use darwin-amd64-11_0 releaselet starting with Go 1.17 Beta 1

This is consistent with our approach of using latest major macOS version builder for creating macOS releases. Also see https://github.com/golang/go/issues/40561#issuecomment-840725710.
CC @golang/release.

@heschi agreed to take this on. Thanks!


fix(nuxt): simplify plugin InjectionType template conditional

🔗 Linked issue
Resolves #29021
📚 Description
This PR simplifies the generated InjectionType types used for plugins by removing the type IsAny and instead relying on directly inferring the default export of plugins. The reasoning behind this is that IsAny was not a consistent check and could be inlined.

Thank you! ❤


Make resource names unique for supporting charm resources.

We can't deploy the bundle with resources because there's no way to differentiate between e.g.,
https://github.com/juju-solutions/kubernetes/blob/master-node-split/cluster/juju/layers/kubernetes-master/metadata.yaml#L35
and
https://github.com/juju-solutions/kubernetes/blob/master-node-split/cluster/juju/layers/kubernetes-worker/metadata.yaml#L23

[ ] Make these unique so that there isn't a conflict.
[ ] Check for other conflicts.

@wwwtyro I don't know that bundles support resources at this time. When they do I would expect them to use a key for the charm name and the keys would be subordinate to the charm name which would sufficiently differentiate the resources.
I could envision many charms with the same resource name, but actually needing different resources.  An nginx charm may use the "nginx" resource, and a third party isv charm that uses an nginx layer but has a completely different name could also use an "nginx" resource.
We should get the core team to weigh in on resources in bundles to make sure my assumptions are correct. In any case I don't expect bundles to support resources any time soon.

@mbruzek Thanks, I'll close this for now, then. :+1:


offline infer different from your demo.

The output of the model is very short, even i set "answer more than xxx words" in the prompt doesn't help. here is my test code, any help please!
from PIL import Image
import requests
import torch
from transformers import AutoModelForCausalLM, AutoTokenizer
from accelerate import init_empty_weights, infer_auto_device_map, load_checkpoint_and_dispatch

tokenizer = AutoTokenizer.from_pretrained("BAAI/Emu2-Chat") # "BAAI/Emu2-Chat"

with init_empty_weights():
     model = AutoModelForCausalLM.from_pretrained(
        "BAAI/Emu2-Chat", # "BAAI/Emu2-Chat"
        torch_dtype=torch.bfloat16,
        low_cpu_mem_usage=True,
        trust_remote_code=True,
        cache_dir="./ckpt")

device_map = infer_auto_device_map(model, max_memory={0:'50GiB',1:'50GiB',}, no_split_module_classes=['Block','LlamaDecoderLayer'])
# input and output logits should be on same device
device_map["model.decoder.lm.lm_head"] = 0

model = load_checkpoint_and_dispatch(
    model,
    './ckpt/models--BAAI--Emu2-Chat/snapshots/20ea30b04f8fee599cf97535e655c200df728501',
    device_map=device_map).eval()

frms = [
  "img1.jpg",
  "img2.jpg",
  "img3.jpg",
  "img4.jpg",
  "img5.jpg",
  "img6.jpg",
  "img7.jpg",
  "img8.jpg",
]

video = [Image.open(frm).convert('RGB') for frm in frms]

import time

start_time = time.time()
query = "{}Describe the video in details.".format("[<VID_PLH>]"*len(frms))

inputs = model.build_input_ids(
    text=[query],
    tokenizer=tokenizer,
    video=video

)

with torch.no_grad():
     outputs = model.generate(
        input_ids=inputs["input_ids"],
        attention_mask=inputs["attention_mask"],
        video=inputs["video"].to(torch.bfloat16),
        max_new_tokens=2048,
        length_penalty=-1)

output_text = tokenizer.batch_decode(outputs, skip_special_tokens=True)

print("infer cost", time.time()-start_time)

output_text

after test on online demo, answer length are the same. close

Chat Template
To use the Emu2-Chat, the inputs to the model should follow the chat template "<SYSTEM MESSAGE> [USER]: <YOUR PROMPT> [ASSISTANT]:". The system message and more detailed inputs construction codes can be found at here
Video Format
For video understanding tasks, the video frames part in you prompt should be organized as "[VIDEO][<VID_PLH>][<VID_PLH>][<VID_PLH>]...". There should be a special token [VIDEO] before the first frame.
More Detailed Output
You may adjust your prompt to force the model to output more results. For example, "What happened in the video? Describe as many elements as possible.", etc.

Chat Template
To use the Emu2-Chat, the inputs to the model should follow the chat template " [USER]:  [ASSISTANT]:". The system message and more detailed inputs construction codes can be found at here
Video Format
For video understanding tasks, the video frames part in you prompt should be organized as "[VIDEO][<VID_PLH>][<VID_PLH>][<VID_PLH>]...". There should be a special token [VIDEO] before the first frame.
More Detailed Output
You may adjust your prompt to force the model to output more results. For example, "What happened in the video? Describe as many elements as possible.", etc.

Thanks!


Documentation in Chinese

Add Chinese documentation to better serve Chinese-speaking users

Finished in #61, #67, #68 and #74. The Chinese documentation is available at https://graphscope.io/docs/zh/.

Finished in #61, #67, #68 and #74. The Chinese documentation is available at https://graphscope.io/docs/zh/.


Activity doesnt shows transfer nor sales yet lot of holders

https://canary.kodadot.xyz/ksm/collection/9ecdab6f55355b191a-GAMMA_BUBL/activity?transfer=true&sale=true
https://canary.kodadot.xyz/ksm/collection/9ecdab6f55355b191a-GG_XPS/activity?sale=true&transfer=true

maybe @daiagi would know?

I'll check

🤔

Hello I just release marck v2
I made a ton of fixes for last days so hope it would be done
◷ Compressing the squid to marck-v2.tar.gz ... 152 file(s) ✔
🦑 Releasing the squid
◷ Uploading marck-v2.tar.gz... ✔
◷ Preparing the squid... ✔
Downloading squid archive from https://storage.googleapis.com/subsquid-rest-api-tmp-upload/tmp/8917c7ec-37a4-4b9d-9b46-c5d10355b21c.tar.gz...
Extracting archive...
Extracted 153 files from archive, hash 63a2eac066f957ed2553528d75b3729a2705a652
◷ You may now detach from the build process by pressing Ctrl + C. The Squid deployment will continue uninterrupted.
◷ The new squid will be available as soon as the deployment is complete.
◷ Building the squid... ⡿

Possible fixes:

kodadot/rubick#231
kodadot/rubick#233
kodadot/rubick#234
kodadot/rubick#237

Collection stats are showing now

As I mentioned in - https://github.com/kodadot/rubick/issues/235
There was a bug on indexer I updated new version and it seems to be working correctly


Support for having file attributes in the State object

Gobblin now preserves the owner, group, permissions and replication factor for files. The state object now supports the following configuration keys which may be used to capture these attributes and pass them on to the output file.

HDFS_FILE_OWNER
HDFS_FILE_GROUP
HDFS_FILE_REPLICATION_FACTOR
HDFS_FILE_PERMISSIONS

I believe we also discussed replacing AvroHdfsDataWriter.CodecType with CodecFactory.fromString(String s) since it will provide a safer option for setting the codec. Do you want to do that in this pull request, or a different one?

I was planning to do the refactor as a different pull request.

LGTM other than the three comments above.

LGTM.

LGTM. In the next pull request, can we also add a unit test for AvroHdfsDataWriter to ensure that it is preserving all the properties properly.

sure. I can do that.
For the current changes though, the test will have to check the results of a bunch of set commands.


Add noisy simulator backend to qsharp

This PR does two things:

[x] Adds a noisy simulator backend to the qsharp repo.
[x] Exposes a thin PyO3 interface around the simulator.

I just noticed the top-level folder name has a dash in (noisy-simulator), whereas our others (language_server, resource_estimator) use underscores. Maybe change this for consistency also @orpuente-MS .

I just noticed the top-level folder name has a dash in (noisy-simulator), whereas our others (language_server, resource_estimator) use underscores. Maybe change this for consistency also @orpuente-MS .

Thanks, I changed the folder name and also the name of the crate in the Cargo.toml to use underscores.


Update LICENSE:"

Adhere
Hey there! So you want to contribute to a Firebase SDK?
Before you file this pull request, please read these guidelines:
Discussion

Read the contribution guidelines (CONTRIBUTING.md).
If this has been discussed in an issue, make sure to link to the issue here.
If not, go file an issue about this before creating a pull request to discuss.

Testing

Make sure all existing tests in the repository pass after your change.
If you fixed a bug or added a feature, add a new test to cover your code.

API Changes

At this time we cannot accept changes that affect the public API.  If you'd like to help
us make Firebase APIs better, please propose your change in a feature request so that we
can discuss it together.

No license changes are needed at this time.


cargo llvm-cov run changes pwd

When calling cargo llvm-cov run from a sub directory of the current workspace, pwd is changed from this directory to the workspace root before the runnable command is executed.
This makes it extremely hard to use llvm-cov run for testing when tests are structured into multiple folders.

Thanks. I think this was originally intended to make --remap-path-prefix or something work, but we don't use it anymore and I don't think the current dir change is needed any longer.
Filed #161 to fix.

Oh wow, thank you so much for the quick help! You're amazing :D

Published 0.4.0 which includes the fix for this issue.


Change iOS default quality to Medium

This should fix #6 , Medium quality seems like the most sane default to me.
Currently opening this as a draft, because I haven't had a chance to test this yet.

Haha, I made exactly the same change as you did locally before seeing this PR. I already tested it and this will be part of the new update.
Unfortunately I should have done it before, because it will break code and I will have to bump version 2.0.0 because it use SEMVER.

Thank you for that. I will close to bump the new version here.


internal error: binding null type in envBindTypeRef

Opened by henke
In Fsi:
[Loading X:\logary\src\Logary\WinPerfCounters.fs]
error FS0192: internal error: binding null type in envBindTypeRef: Generic IKEv1, AuthIP, and IKEv2
In C:\Program Files (x86)\Microsoft SDKs\F#\3.0\Framework\v4.0\Fsi.exe, Windows 8.1.
On this line of code:
Steps to repro:

start fsi
#load "WinPerfCounters.fs"

comments
henke wrote Aug 11, 2014 at 9:36 AM [x]
Also, IntelliSense doesn't show the symbols even after changing the commas to underscores.

Mini-repro:
run fsi with:
module ``Generic IKEv1, AuthIP, and IKEv2`` =
  [<Literal>]
  let Category = "Generic IKEv1, AuthIP, and IKEv2";;

Just testing a fix for this.


provide examples

Could you add support for bidirectional parameter as it's not implemented in the current xLSTM model? Thanks a lot.
It is mentioned in the todo list of this repository.

Sure, soon!


mock rest api locally

Is your feature request related to a problem? Please describe.
I want to be able to run both the frontend and backend locally, control with environment variables when to produce/consume mock data and have the urls map to the right environment. I don't use GraphQL!
Describe the solution you'd like
amplify serve and mock -> frontend on localhost:3000 with env variables of mock && backend on localhost:3001 with env variables of mock
Describe alternatives you've considered
writing my own bash script passing environment variables in to every backend function

@alexladerman thanks for the feature request, would you elaborate in details on your requirements, how would you imagine the developer workflow?

+1 for local mocking of API Gateway. I'm surprised this isn't supported yet by amplify mock. It would be nice to be able to run api mock and aws-exports.js which is consumed by our web app would automatically point to the localhost REST API rather than the cloud one.
I'm currently able to run the express server locally via amplify function invoke, but automatically changing aws-exports.js and hot reloading the express server on code change does not happen through this method.

Add me to this list. Let's pass this bill to congress!

This would be really useful!

At the moment i'm using:
amplify function invoke restapiName
this works pretty well.

+1 on this one. Would be great to quickly run amplify mock api when you have a basic rest api with express. I don't have a GraphQL backend so mock doesn't work.

+1

we can no longer use amplify function invoke to do this, this feature would be really useful for those of us who aren't using GraphQL!

+1 need this functionality

+1 or any alternative?

Hey guys, an alternative is to cd into the src folder and run the command “node app.js” and using dotenv for local environments, this will host your function on localhost port 3000 and u can test with postman. You can also use nodemon for hot reload.

+1

+1

+1

hello, can we please add the old amplify function invoke flow back? this is definitely making definitely significantly more tedious as the mock functionality replaced something that was working with something that is broken...

I used @AbirAbbas method and it does work. (thank you) It would be VERY NICE if I could use amplify mock api <NAME>, have aws-exports.js generate my local environment for my clientside app, and fully test.  I am also thinking this would allow for easier testing in a pipeline.

+1

+1

+1

+1

+1

+1

+1 We are also waiting for this

+1

in addition to @AbirAbbas workaround, for ease of development you can add a script to your main package.json file that runs the command "node app.js".
package.json:
"scripts": { "server_dev": "node amplify/backend/function/<function_name>/src/app.js" }
and then just run npm run server_dev

This works only if you use a pure node project. If you use layers for example, it's not work.

+1

+1

+1

Is this why - when I submit a query to the mock api - I get:
{ errorMessage: 'Must provide Source. Received: undefined' }

?

+1, since we are able to use sam local start-api to test the REST APIs locally, I think it is reasonable to assume that Amplify is able to provide something similar :)

I'll share my workaround for this issue, it will allow local testing for the REST API, I've not looked into a local GraphQL API.
I just run yarn start for the frontend and yarn mock for the backend. You can also pass in events, see below.
package.json (at project root level to save changing to the function directory each time)
"scripts": { "start": "cross-env PORT=8000 react-app-rewired start", "mock": "nodemon amplify/backend/function/[YOUR API NAME]/src/index.js", "mock:event": "nodemon amplify/backend/function/[YOUR API NAME]/src/index.js ./events/event.json", }
Backend index.js
`
const handler = async (event, context) => {
const awsServerlessExpress = require('aws-serverless-express')
const app = require('./app')
const server = awsServerlessExpress.createServer(app)
console.log(EVENT: ${JSON.stringify(event, null, 2)})
// Express app
return awsServerlessExpress.proxy(server, event, context, 'PROMISE').promise
}
;(async function () {
/**

Support running function locally

Run yarn mock from project root
Requires .env in project root with:

ENV=

REGION=

AWS_ACCESS_KEY_ID=

AWS_SECRET_ACCESS_KEY=

MOCK=true
*/
if (
process.env.npm_package_scripts_mock &&
process.env.npm_package_scripts_mock.includes('nodemon')
) {
process.env.MOCK = true
require('dotenv').config()

// getting the event from passed args
const event = process.argv[2] ? require(process.argv[2]) : ''
if (!event)
  console.log(
    `No event - run 'yarn mock:event' etc...`
  )
const res = await handler(event, {
  done: () => {}, // err, success
})
if (res) console.log({ statusCode: res.statusCode, body: res.body })

}
})()
module.exports = {
handler,
}
`
React frontend
`
import Amplify from 'aws-amplify'
import React from 'react'
import ReactDOM from 'react-dom'
import App from './app'
import awsconfig from './aws-exports'
if (process.env.NODE_ENV === 'development') {
awsconfig.aws_cloud_logic_custom = awsconfig.aws_cloud_logic_custom.map(
(config) => {
if (config.name === '[YOUR API NAME]') {
config.endpoint = http://${window.location.hostname}:3000
}
return config
}
)
}
Amplify.configure(awsconfig)
ReactDOM.render(, document.getElementById('root'))
`
Note, I've taken this code from my app and stripped it down, might need adjusting to work.

+1

+1
please add it!

+1

We are currently evaluating migrating away from AWS Amplify for exactly these kind of reasons. Not being able to test locally is an essential feature the lack of which drastically slows down development cycles.

I'm also having issues running locally and it's being really a pain. I'm reading the docs: https://docs.amplify.aws/cli/usage/mock
From my understanding, this only mocks the API and not the functions. Could u please recommend me clear instructions about how can I work locally?

Our team will start migrating away from 'amplify' and back to containers.
Issues like this make it impossible to develop and iterate fast!
I wouldn't say it was a waste of time though to try amplify. There are a lot of neat features, but it definitely needs more engineering force behind its development.

The method of @AbirAbbas is the simplest but you can also run a local Docker container.

Create the Dockerfile

# ./amplify/backend/api/<name>/src/Dockerfile

FROM public.ecr.aws/bitnami/node:14.15.1-debian-10-r8

ENV PORT=3000
EXPOSE 3000

WORKDIR /usr/src/app

COPY package*.json ./
RUN npm install
COPY . .

CMD [ "node", "index.js" ]

Build image and launch container

docker build -t node-app:1.0 .
docker run -p 3000:3000 -d node-app:1.0

Test POST method

curl -H "Content-Type: application/json" -d '{"key": "value"}' http://localhost:3000

+1

+1

Still no update on this topic?

+1

in case the maintainers need more +1's, here's another one, +1


Can't sign in to Google Account when using Advanced Protection Program

I am running the Android version 1.1.2, from the Play Store, on Pixel 3 with Android 10.
I have the Google Advanced Protection Program enabled with my Google Account.
When trying to sign into the account, in order to extract my Location History data, the browser window shows a message which says "This browser does not support security keys."
If I tap on "Try another way", I have only the "Security Key" option available (since I have the APP enabled). So currently I can't seem to use this new feature.

Thanks for your feedback! It helps us improve the app.
I admit that we haven't tested the app with Google Advanced Protection Program enabled, so we didn't encounter this issue. Your feedback will help us investigate it and try to find a solution.
Note that for the login to your Google account, the app uses the native WebView, so it seems like an inherent limitation of Android WebView with Google's own Advanced Protection Program, not unique to our app.
However, we'll try to further investigate it and look for a solution or at least a workaround.

Yes, I assumed this would not be a top-priority bug, and as you said - it is probably related to the WebView itself. But I just wanted to get the issue posted. Maybe somebody will know of some workaround or solution for this.


Not working

I have installed the mod into my modpack, but when I opened it it did not work at all. no particles were showing up! Anyone know how to fix?

i've been having the same problems, i'm using both 1.19.2 visuality, cloth config, and fabric api

Make sure you particles enabled


Add GraphML support to TangeloHub

This issue pertains to adding the graph type and a graphML parser to romanesco so graph-based analyses can be added to TangeloHub.

can use the python wrapped interfaces to igraph for format conversion


Recent page changes appear to have hidden the references to IE

It appears this page format was updated recently (today?), which removed significant, descriptive text.
One such area was the specific notations regarding the need to add certain URL's into the IE Trusted Sites settings for our user's workstation browsers.
Getting requests for supportive documentation and would greatly appreciate such information being made available again

@mzappy can you clarify which article(s) don't have the descriptive text you're looking for?

https://docs.microsoft.com/en-us/office365/enterprise/urls-and-ip-address-ranges?redirectSourcePath=%252fen-us%252farticle%252fOffice-365-URLs-and-IP-address-ranges-8548a211-3fe7-47cb-abb1-355ea5aa88a2
Thank you, Denise for your time and assist with this request.
Mike Zappacosta
Microsoft Cloud Platform Engineer
Exelon IT Cloud & Infrastructure
From: Denise Vangel-MSFT<EMAIL_ADDRESS>Sent: Friday, August 24, 2018 1:13 PM
To: MicrosoftDocs/OfficeDocs-Enterprise<EMAIL_ADDRESS>Cc: Zappacosta, Michael A:(PHI<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: [EXTERNAL] Re: [MicrosoftDocs/OfficeDocs-Enterprise] Recent page changes appear to have hidden the references to IE (#121)
@mzappyhttps://github.com/mzappy can you clarify which article(s) don't have the descriptive text you're looking for?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/OfficeDocs-Enterprise/issues/121#issuecomment-415823060, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ARASBbTjl3JHtLqhzN5osu4AKXMUYjUPks5uUDQngaJpZM4WIHYf.
This Email message and any attachment may contain information that is proprietary, legally privileged, confidential and/or subject to copyright belonging to Exelon Corporation or its affiliates ("Exelon"). This Email is intended solely for the use of the person(s) to which it is addressed. If you are not an intended recipient, or the employee or agent responsible for delivery of this Email to the intended recipient(s), you are hereby notified that any dissemination, distribution or copying of this Email is strictly prohibited. If you have received this message in error, please immediately notify the sender and permanently delete this Email and any copies. Exelon policies expressly prohibit employees from making defamatory or offensive statements and infringing any copyright or any other legal right by Email communication. Exelon will not accept any liability in respect of such communications. -EXCIP

@mzappy You can find this information in a new article here: https://docs.microsoft.com/office365/enterprise/additional-office365-ip-addresses-and-urls


DOC: Fix API doc on RTD

To build API doc, sklearn, scipy and matplotlib deps should be optional. Not sure there is a better way...
Build result (non-public):

http://pandas-ml.readthedocs.io/en/mock/generated/pandas_ml.skaccessors.html

Coverage decreased (-0.3%) to 94.535% when pulling 30f84fef266dd5ba9cb0c108da344d5a876807f4 on mock into 86583d0f242eebe227d95d224029882ab8d3d453 on master.


Log matched requests in a browser's console

Those requests which have matched the request handlers defined in the composeMocks() call should be logged into a browser's console.
Motivation

Easier and faster for the developer to see if something has been mocked

@kettanaito I think it would be nice to have a configuration to turn it on. The mock's print message would be mixed with the application's print message, and while we can do some printing styles, it would be better for the developer to choose whether to turn it on or not. What do you think?

@hehehai, a good call. It would be good accept such logging preference via an argument. I wonder, where should it be provided?
I though of providing it to the start() function, but it accepts two optional arguments that control Service Worker registration, and may still be useful. Providing them both in order to set the third argument with the logging level feels redundant.

Since Mock Service Worker is recommended for development only, I see little worry over seeing too many messages. At the least you can always filter the console messages based on type and RegExp to find what you were looking for.
On the other hand, logging matched requests would improve the developer's experience dramatically when working with the library.


Update tutorial-use-gitops-flux2.md

Since GitOps has gone GA. We no longer require customers to enable the AKS-ExtensionManager feature to enable GitOps

Learn Build status updates of commit 0dc2f6d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/azure-arc/kubernetes/tutorial-use-gitops-flux2.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Thanks for the update @kelvien-microsoft !
#sign-off

@JnHs/@mayurigupta13, I am not authorized to merge this pull request and it seems like it needs a review from @JnHs as well before we can merge this PR.

#sign-off


HBASE-26069 Remove HStore.compactRecentForTestingAssumingDefaultPolic…

…y and DefaultCompactor.compactForTesting

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 35s
Docker mode activated.


Roll Flutter Engine from 7d70dc81cfa5 to 28e4a21a0165 (6 revisions)

https://github.com/flutter/engine/compare/7d70dc81cfa5...28e4a21a0165
2023-05-03<EMAIL_ADDRESS>Reland: "Determine lifecycle by looking at window focus also" (#41094) (flutter/engine#41702)
2023-05-03<EMAIL_ADDRESS>Roll Skia from 887e58fff02f to 93c869da03f5 (12 revisions) (flutter/engine#41705)
2023-05-03<EMAIL_ADDRESS>[web] Improve null safety for color->css (flutter/engine#41699)
2023-05-03<EMAIL_ADDRESS>[web] Remove unused function parameter (flutter/engine#41700)
2023-05-03<EMAIL_ADDRESS>Migrate Android Engine Debug to Engine V2. (flutter/engine#41687)
2023-05-03<EMAIL_ADDRESS>Migrate SkSurface::getBackend* methods to static functions (flutter/engine#41693)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jimgraham@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


[windows][web] Horizontal and Vertical scrolling did not happen when scrolling through the external mouse

Steps to Reproduce

Run the below simple sample.
Resize the window to make the view as scrollable.
Do horizontal and vertical scrolling by mouse click and drag.

class NestedScrollDemo extends StatefulWidget {
  NestedScrollDemo({Key? key}) : super(key: key);

  @override
  _NestedScrollDemoState createState() => _NestedScrollDemoState();
}

class _NestedScrollDemoState extends State<NestedScrollDemo> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('Nested scroll demo'),
      ),
      body: SingleChildScrollView(
        scrollDirection: Axis.horizontal,
        child: SingleChildScrollView(
          scrollDirection: Axis.vertical,
          child: Column(
            children: List.generate(30, (index) => _buildRow(index)),
          ),
        ),
      ),
    );
  }
}

Widget _buildRow(int rowIndex) {
  return Row(
    children: List.generate(
      6,
      (colIndex) => Container(
        alignment: Alignment.center,
        decoration: BoxDecoration(
            border:
                Border.all(color: Colors.grey.withOpacity(0.45), width: 1.0)),
        child: Text('Column $rowIndex'),
        width: 100,
        height: 50,
      ),
    ),
  );
}

Expected results: Scrolling should happen.
Actual results: Scrolling did not happen.

Flutter doctor

[√] Flutter (Channel stable, 2.5.0, on Microsoft Windows [Version 10.0.18363.1440], locale en-US)
[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    X Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/windows#android-setup for more details.
[√] Chrome - develop for the web
[√] Visual Studio - develop for Windows (Visual Studio Professional 2019 16.4.5)
[√] Android Studio (version 3.3)
[√] VS Code, 64-bit edition (version 1.58.0)
[√] Connected device (2 available)

! Doctor found issues in 1 category.
PS D:\FLUT-5268-Linter-warnings\flutter-datagrid\flutter_datagrid> flutter doctor -v
[√] Flutter (Channel stable, 2.5.0, on Microsoft Windows [Version 10.0.18363.1440], locale en-US)
    • Flutter version 2.5.0 at C:\src\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 4cc385b4b8 (8 days ago), 2021-09-07 23:01:49 -0700
    • Engine revision f0826da7ef
    • Dart version 2.14.0

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at C:\Users\AshokKuvaraja\AppData\Local\Android\sdk
    X cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    X Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/windows#android-setup for more details.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Professional 2019 16.4.5)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional
    • Visual Studio Professional 2019 version 16.4.29806.167
    • Windows 10 SDK version 10.0.18362.0

[√] Android Studio (version 3.3)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 34.0.1
    • Dart plugin version 182.5215
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)

[√] VS Code, 64-bit edition (version 1.58.0)
    • VS Code at C:\Program Files\Microsoft VS Code
    • Flutter extension version 3.25.0

[√] Connected device (2 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.18363.1440]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 93.0.4577.63

@ashok-kuvaraja
This seems to be a known issue. See this issue comment for your reference.
Also this PR. The change doesn't seem to be yet rolled into stable. You may check on latest master and see if it works.

Hi @darshankawar,
I checked the issue in the current master channel. Seems the issue is not resolved in that too. I could able to reproduce the issue in web and windows platforms in the master channel too.

 flutter doctor -v
[√] Flutter (Channel master, 2.6.0-6.0.pre.127, on Microsoft Windows [Version 10.0.18363.1440], locale en-US)
    • Flutter version 2.6.0-6.0.pre.127 at C:\src\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ec3bd9edc4 (36 minutes ago), 2021-09-16 20:02:02 -0700
    • Engine revision 72f08e4da3
    • Dart version 2.15.0 (build 2.15.0-116.0.dev)

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at C:\Users\AshokKuvaraja\AppData\Local\Android\sdk
    X cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    X Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/windows#android-setup for more details.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Professional 2019 16.4.5)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional
    • Visual Studio Professional 2019 version 16.4.29806.167
    • Windows 10 SDK version 10.0.18362.0

[√] Android Studio (version 3.3)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 34.0.1
    • Dart plugin version 182.5215
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)
    • Flutter extension version 3.25.0

[√] Connected device (2 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.18363.1440]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 93.0.4577.82

Regards,
Ashok K

@ashok-kuvaraja
Thanks for verifying. I think I missed to link this breaking change earlier. Also see this issue comment for your further reference.

Hi @darshankawar,
Thanks for sharing the information. It's working fine now.
Regards,
Ashok K


Native hook issues with SM-A526B

App Information*

Device Information*
Samsung Galaxy A52 5G (SM-A526B)
OS:   Android OS 13
Log*

Thanks for contributing!
Your issue looks like this other Samsung-specific issue: #146. Please try the debug-version linked here: https://github.com/nfcgate/nfcgate/issues/146#issuecomment-1944757296. We will soon incorporate this into a future release.

Now it works perfectly

Now it works perfectly

Does the cloning work correctly though? For my device even after Native hook is enabled, the NFC chip does not accept the NFCID and presents a random one instead (#153).
I wonder if it is a problem with my device only or with all Samsung devices.

@Lassie111 It looks like the behavior you are describing has nothing to do with Samsung devices and everything to do with the type of NFC Chip. Your NXP chip has the same issues as many OnePlus models with the same chip (i.e. it only uses random NFCIDs and does not accept a manually set one). @cydef1 has a Samsung NFC chip, where that issue has never been reported so far.


User-reported unhelpful error message

Program: https://code.world/#PMKAwGHravlK5Rwgqs33D_Q
Message:
program.hs:1:1: error:
    The variable program is not defined in your code

Student used main instead of program.  We can do a better job of this by detecting it in pre-parse.


SqlRender

SqlRender - This is an R package for rendering parameterized SQL, and translating it to different SQL dialects. SqlRender can also be used as a stand-alone Java library and a command-line executable.
https://github.com/ohdsi/SqlRender
Error
Installing package into ‘C:/Users/rsrini86/Documents/renv/library/R-4.2/x86_64-w64-mingw32’
(as ‘lib’ is unspecified)
Error: evaluation nested too deeply: infinite recursion / options(expressions=)?
Execution halted
Resolution Tip
https://stackoverflow.com/questions/49368446/error-evaluation-nested-too-deeply-infinite-recursion-optionsexpressions


Modify fire weather index calcs to reproduce R library functionality

Description
The official Canadian Forest Service R package for Canadian Forest Fire Danger Rating System (includes FWI) has functionality that differs form our current FWI implementation based on GFWED.
R package : https://cran.r-project.org/web/packages/cffdrs/index.html
Docs pdf : https://cran.r-project.org/web/packages/cffdrs/cffdrs.pdf
In particular cffdrs FWI allows for:

[ ]  different temperature thresholds for season startup / shutdown
[ ]  for 'overwintering' of the drought code

The first case is likely relatively simple to fix
The second could be more work we could look into https://cfs.nrcan.gc.ca/pubwarehouse/pdfs/29152.pdf or contact Alan Cantin (contact for R Package)

I'd add:

[ ] use dask now that multiple outputs are supported

More info on overwintering DC
https://essd.copernicus.org/articles/12/1823/2020/

And if not too difficult:

[ ] Pass "starting_date" as an array, in order to prescribe different season start ups for each grid cells. Requires to rethink the dc0/dcm0/ffmc0 machinery.


Missing multiple features on clean install

I'm using the recommended configuration, but how do I enable multiline suggestions? I've already configured it with the tabnine hub
Also, the dictionary isn't working,

Here's my configuration:
https://pastebin.com/raw/uzvGBsxY

Multiline suggestions are not consistently working. Its not a problem with this plugin but with tab nine itself. You can look at previous issues where its discussed.
As for the "dictionary is not working" I have no idea what you mean.

As of the last few days, can confirm that the multi-line completion issue is resolved. Ensure you get the latest version of the engine as shown on the hub and are triggering cmp on a new line. I think the default is Ctrl-space, else check your mappings.
Thank you to all those involved.

Closing this as stale.


Colab demo requires multiple upgrades

Devolearn requires torch==1.7, but I think it would also work with the latest versions of torch. Using the latets version of torch would make the installation much faster on google colab notebooks because the latest version of torch is already installed there.
The URL for the weights to the embryo segmentation model is most likely broken, throws an HTTP error.
arg mode = 'cpu' should be replaced with device = 'cpu'
optionally, we can also let the user decide the GPU ID like device = 'cuda:0'
move the run tests cell to the bottom

Note that this is a beginner friendly issue, all the changes are to be done on this notebook. Feel free to reply to this if you're willing to take this up.

Hi, I've started to take a look on the issues, to help out :)
Regarding 1 - Where is the colab notebook?
a - I tried to go into Examples notebook - but it took me to nbviewer.org notebook
b - I'm seeing in requirements.txt that torch>=1.7 which is fine?
Continuing to check the rest

@alon1samuel  https://colab.research.google.com/github/DevoLearn/data-science-demos/blob/master/devolearn_docs/devolearn_quickstart.ipynb this is link for the notebook and ,I was also going through the issue and found out main error was coming at "embryo_segmentor "actually ,I found out there was no file named embryo segmentor in the whole devolearn repo , I was still search for this thing, if u found out any leads ..pls let me knew

Hi @sushmanthreddy and @alon1samuel 👋
The colab notebook's source code can be found here. Feel free to make a PR there with the fixed notebook.

Thanks guys! I'll take a look @sushmanthreddy @Mayukhdeb

I think, I have found the root of the problem. While running the "tests" cell. It throws an import error saying it cannot import name 'cell_membrane_segmentor' from 'devolearn' (/usr/local/lib/python3.7/dist-packages/devolearn/__init__.py)
This error can be resolved by releasing a new package on pypi by the admins

So if you install the package from github it fixes the problem?
I mean if you do  pip install git+https://github.com/DevoLearn/devolearn  ... ?

Yeah after trying the command the issue seems to resolved the only thing that remains is that embryo_segmentor is no longer a function that is used in the newer versions of devolearn and it's name has been changed to something else.
Can the admins update the same over here so it can be changed.

Thanks, I'll check it later. Hopefully create a PR soon

after using

So if you install the package from github it fixes the problem?
I mean if you do pip install git+https://github.com/DevoLearn/devolearn ... ?

no it doesn't resolve the issue actually the embryo_segmentor has been changed to cell_membrane_segmentor..when we try to import cell_membrane_segmentor it was throwing out an error at header files and while installing also it was showing an error..these were the errors ,which I have encountered..

Hi @sushmanthreddy ,
If you delete the runtime and re-run the notebook the issue should be fixed. I did the same it worked for me.

Hi @sushmanthreddy ,
If you delete the runtime and re-run the notebook the issue should be fixed. I did the same it worked for me.

thanks for helping out

I've left a PR for the changes on devolearn side.
Changes are still needed in the data-science notebook

fixed issues
1 ." The URL for the weights to the embryo segmentation model is most likely broken, throws an HTTP error"  , I have tried to fix this issue .
2 . arg mode = 'cpu' should be replaced with device = 'cpu' , this has been fixed..
Issues need  to  be fixed :
1 .optionally, we can also let the user decide the GPU ID like device = 'cuda:0'
2 .move the run tests cell to the bottom..

Thank you! Very nice work

@alon1samuel
use this code in run test cell...the path is inappropriate which u were using that is the reason it isn't working
import devolearn
os.path.dirname(devolearn.__file__) + "/" + "tests" + "/" + "test.py"
!python /content/devolearn/devolearn/test

I have closed my pr because it has some incomplete work..and above code snippet works fine just path need to updated correctly

And I was encountering another error at installation place..at first code cell.I don't knew what it is! but it seems to look like this...if u find out same issue pls try too resolve it

Great! The tests run smoothly, I'll check on my notebook as well.
About your errors it's really weird about these dependencies. For example: I haven't seen tensorflow being needed for the installation. I'll slack you, maybe we can fix this in a huddle.

I've fixed some other stuff in the notebook.
Now the tests are failing since 1 file is not showing up. Even after installing devolearn from git by
'pip install git+https://github.com/DevoLearn/devolearn.git@master`
I'm still getting a message - successfully installed devolearn-0.3.0 (which is weird)
And the file that is missing in the tests is "devolearn/tests/sample_data/images/nucleus_seg_sample.png":

Anyone has an idea why id does that?

I've ignored this in the end. Very strange anyway.

I've created a PR with recent changes to notebook on repo
I think the issue can be closed @Mayukhdeb as @Anantk2908 did most of the fixes there in the last MR


I think you dont need this file in project

https://github.com/Gayaneh91/djangoProject/blob/272909faf53b2fd963bc72a7f93c4ce2289a6266/Desktop/task/cinema/films/film_scrap.py#L1

Yes, I created this file to do scraping and be sure that it works, and forgot removing from project. Already moved to another directory.


If slidesPewView is half the total of slides, navigation breaks

This is a (multiple allowed):

[x] bug

[ ] enhancement

[ ] feature-discussion (RFC)

Swiper Version: 6.4.5

Platform/Target and Browser Versions: Mac, Chrome 87.0.4280.141

Live Link or JSFiddle/Codepen or website with isssue: https://stackblitz.com/edit/swiper-demo-31-thumbs-gallery-with-loop-mode-f1q6fz?file=index.html

What you did
Create a slider that has N slides and should show N/2 slides as looped thumbs.
Expected Behavior
The navigation works as expected, I can see the active slide when I click on the navigation arrows
Actual Behavior
If the number of slidesPerView is exactly half of the number of total slides, then the thumbs slider jumps to a slide out of view. If its 1 more or 1 less the behaviour is normal
For example, 10 slides, slidesPerView is 5, when navigating and reaching the fifth slide, the next one active will be out of view.

Note: If I remove the loop from the thumbs, the slider behaves half as expected, you get the loop but you don't get a smooth loop, it navigates back to the first slide

Swiper v9 comes with fully reworked and now different loop functionality. If you have similar issues in Swiper 9, open a new issue with a CodeSandbox showing the issue.


[PROJECT] Marketing pages - E2C

Overview
We are putting up a page to explain all the exit to community idea, a.k.a. e2c
Details to be updated

Plan for this week / W2S2Q4 - Week 48

[ ] Fill in issue details
[ ] Wireframes 2nd stage (How to get to e2c)

Plan for this week / W2S3Q4 - Week 49

[ ] Fill in issue details
[ ] Wireframes 2nd stage (How to get to e2c)


added json-to-csv.py

Description
Please include a summary of the change and which issue is fixed. List any dependencies that are required for this change.
Fixes #1672
Type of change

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md
[x] My documentation follows Template for README.md
[x] I have added the project meta data in the PR template.
[x] I have created the requirements.txt file if needed.

Project Metadata
No requirements
Category:

[x] Networking
[x] OS_Utilities
[x] Automation
[x] Others

Title: json-to-csv.py
Folder: JSON-to-CSV
Requirements: none
Script:json-to-csv.py
Contributor: sagnik-p
Description: python script that converts json file to csv file

Done!
Please check
Thanks for the opportunity


haskellPackages.ekg-core: fix build by relaxing bounds

Motivation for this change
This fixes the build for ekg-core. There's an upstream fix but we are just waiting for a hackage release.
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

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

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

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

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

[ ] Fits CONTRIBUTING.md.

@ofborg build haskellPackages.ekg-core

Cherry picked as a0ed90b345508f7ee7b0a8ed51ceb80183bc3def with an assert added, so we are reminded to clean this up later. Thanks!


Rewrite to Application Load Balancer doesn't work (I suspect its because of TLS1.3)

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

App Id
dqbbd1fflds4t
Region
us-east-1
Amplify Hosting feature
Redirects
Describe the bug
Rewrite entry on redirects doesn't work when pointing to an AWS Application Load Balancer.
Expected behavior
Should work.
Reproduction steps
Just hit: https://localizalia.dqbbd1fflds4t.amplifyapp.com/api/session
It's random, sometimes it works, but usually it gets stuck at the second time you try.
Build Settings
rewrite entry not working:

{
        "source": "/api/<*>",
        "target": "https://xxxxx-my-alb/api/<*>",
        "status": "200",
        "condition": null
}

Additional information
I suspect this has to do with tls1.3 because I see it in access logs and it is not supported by ALB.
Here's the log with two requests. One times out after 300 seconds and we get a response. The other returns an error because I closed the browser. The request should return instantly.

time-to-first-byte
x-edge-detailed-result-type

306.154
Error

0.000
ClientCommError

30_3_2022-access-logs (5).csv

Can anyone help please? Should I redirect somewhere else?
This looks like a serious issue.
Regards.

Hi @jcardus 👋🏽 thanks for opening this issue, we are taking a look. Will update once we have more information.

Hello @hloriana, do you have any update? Regards.

Hello @hloriana, is there any workaround?

Any news on this?

It's been 5 months... Nothing?

Hi @jcardus sincerest apologies for the delay. I was able to reproduce this behavior and this is a bug with our rewrites functionality. We added this to our product backlog for prioritization but I do not have a date for when this will be fixed. Currently, the workaround is to leverage redirects instead of rewrites.

Hello @hloriana, still nothing?

Hello @hloriana, any updates on this? It's been almost one year...

Hello @hloriana, it's been one year. Obviously redirect is not an option, there are lot's of problems with that, cross site cookies for example.

Hello @hloriii , it's been one year. Obviously redirect is not an option, there are lot's of problems with that, cross site cookies for example.

@jcardus is the issue still occurring?
curl https://localizalia.dqbbd1fflds4t.amplifyapp.com/api/session is correctly returning a response from the target URL

Like I said, the error was random, trying the url once wouldn't mean it's ok.
But I made some quick tests and it seems ok now, I'll try more times.

Looks like its fixed

@saadataz do you confirm the bug is fixed?


Incompatible with python 3

Running python3 partyparrot.py:
Traceback (most recent call last):
  File "partyparrot.py", line 69, in <module>
    force=args.force
  File "partyparrot.py", line 51, in convert_str_to_emoji
    output_string += arr_to_str(c, emoji_iterator.next(), space)
AttributeError: 'itertools.cycle' object has no attribute 'next'

http://www.diveintopython3.net/porting-code-to-python-3-with-2to3.html#next

Because I believe the party parrot is important in everyone's daily life, I have fixed this egregious issue as soon as possible. Please enjoy the party parrot at your earliest convenience.


1010: Invalid Transaction: Transaction has a bad signature

I get this error when trying to sign the transaction:
1010: Invalid Transaction: Transaction has a bad signature
Using Karura ledger app Version 1.2083.0

:link: zboto Link

Hi! Could we have the transaction information? Pallet, method, arguments, etc.
Thanks!

Hi! Could we have the transaction information? Pallet, method, arguments, etc. Thanks!

Thank you for fast reply. I'm trying to transfer KAR from https://apps.karura.network/ signing it with Karura ledger app Version 1.2083.0.
Info attached

Thanks! It would be helpful if I get the first 4 chars of the address (your one, not the receiver) and the mode that you selected on Crowdloan mode of the app (3rd screen).

Thanks! It would be helpful if I get the first 4 chars of the address (your one, not the receiver) and the mode that you selected on Crowdloan mode of the app (3rd screen).

Address starts with: sXyB5iew
I tried in legacy mode (screenshots), but also get same error when using "Crowdloan".

I'll test and come back to you as soon as possible👌🏻

Just noticed a new version was released. Does this fix problem explained here?

Hi @DonCrypto78!
I was investigating a bit into that error and I wasn't able to reproduce. I think you are trying to sign a transaction with a Ledger account not matching the one that is installed in the device. Let's check that.
Try to go to https://polkadot.js.org/apps > Accounts > Click the three dots in the account that you are trying to use > Show address on hardware device. Then check if it matches with the one that is written in polkadot-js.

Thank you, and you are correct. The Polkadot.js.org site shows address starting with "15Eh79" while the apps.karura.network shows address starting with "sXyB5i".
Do you have any suggestions on how to solve this?

In that case there's not an actual solution for the moment. Karura is a parachain of Kusama, so the derivation paths available to use are the one particular of Karura, and the Kusama one.

How did you add that account to Polkadot-js extension, using which particular app?

The address 15Eh79 was created in Ledger Live then attached to polkadot(js) extension

.

So, you are using one account derived with the Polkadot path. Could you give me an email? Let's try to solve that, but actually is a big problem. If we manage to solve it, please move all your funds to an account generated by Karura app. You should always use for each chain the appropriate app.

Thank you. My email is<EMAIL_ADDRESS>

Nice, sent.

Hello. I have the same problem. At the beginning of the year, I created polkadot.js.org account for Ledger on the Kusama network. Then, in the application, Karura sent tokens to Ledger. And now I can't sign the transaction(1010: Invalid Transaction: Transaction has a bad signature). Help solve the problem please.

Could you try enabling the expert mode and using Crowdloan address in Account menu?

Could you try enabling the expert mode and using Crowdloan address in Account menu?

You are my savior friend!♥ Thank you so much. Let me send you a KSM as a token of gratitude.
Can I leave this feature enabled (Crowdloan)?

That's our job! I'm glad I could help☺
And, yes! If you don't change anything, that feature is going to be enabled forever!

That's our job! I'm glad I could help☺ And, yes! If you don't change anything, that feature is going to be enabled forever!

Thank you sir! I've been waiting for this for almost a year.🙂


Add SCSS file to npm package

Is your feature request related to a problem? Please describe.
No. It's for anyone who wants to use SASS in their project.
Describe the solution you'd like
Simply add scss file to npm package.
Describe alternatives you've considered
None.
Additional context
Also for anyone who use LESS you can convert the file to less.
Thank you.

Styles are already included in the main.js file.
Therefore, I believe that there is no reason to publish thescss file in the npm package.

Hımm different perspective. Thanks.


[textanalytics] Moves textanalytics track2 and renames namespace/package

Moves Text Analytics Track 2 to sdk/textanalytics
Renames package/namespace to azure-ai-textanalytics
Rename changes from this PR #9094

Can one of the admins verify this patch?

This PR just moves Track 2 to its new directory. I already reverted the PR that overwrote track 1 with track 2 in the cognitiveservices directory.


⚠ Secret Website 3 has degraded performance

In 4c4031f, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 2991 ms

Resolved: Secret Website 3 performance has improved in 4989fef.


nz-radio-group nzDisabled=false attribute overrides child nz-radio nzDisabled=true

Reproduction link
https://stackblitz.com/edit/angular-p9vfqg
Steps to reproduce
Add nzDisabled="false" to <nz-radio-group> and nzDisabled="true to child <label nz-radio...>. The parent make all child options enabled, even if specific options are supposed to be disabled.
What is expected?
Child nz-radio disabled state should be independent, from global disabled state on <nz-radio-group>
What is actually happening?
<nz-radio-group> should not override nz-radio when value disabled is false.

Environment
Info

ng-zorro-antd
8.0.2

Browser
Chrome 75

it is by design, the nzDisabled value in the <nz-radio-group> will rewrite all its children's nzDisabled
I think you should not add it to the <nz-radio-group>

For me it makes sense that it sets all children as disabled, but not enable them if they are disabled. My logic when doing forms is that there is some logic that would disable an option, but I also have the need to disable all options, for that I use the global control disabled state, so I can clearly separate the concerns and come back to the previous state when I re-enable globally.

I got your point, but it will be very strange for users to understand why it has a different rewrite strategy when the nzDisabled value changed.

But then if you look at other components like nz-select, this is what happens.
https://stackblitz.com/edit/angular-5sg2jb
nz-select has nzDisabled=false, but that change the child nz-option with nzDisabled=true

yes, because it won't change all nz-opiton's nzDisabled to true as well if nz-select has nzDisabled=true

Regarding the nz-select, in terms of UX, the user will still be kept from selecting those options. In terms of behaviour, it is similar.
About the [nzDisabled] for nz-radio-group, the thing about it is that it doesn't seem to work when you dynamically add the attribute when you need it. [attr.nDisabled]="condition"

yes, because nzDisabled is an @Input not a @Atrribute


Whether the algorithm will be open source

Hi, thanks for this innovative work!
Would you please make it clear to the community whether you will release your algorithms?
Thanks for your precious time!

Yeah, I am eager to know


Respect xunit analyzers

we have close to 200 asserts that trigger warning xUnit2000: The literal or constant value xxx should be the first argument in the call to 'Assert.Equal(expected, actual)'

Yeah. My next task is getting rid of all the warnings we have in units. We missed that on reviews...


Update to flash-attn2

Update Comments
Modified scgpt/model/model.py & scgpt/model/multiomic_model.py for flash-attn2. Updated README.md with flash-attn2 installation and notes.
Updated Dependencies
flash-attn 1.x  -->  flash-att 2.x
Is Code Tested?
Yes. The updated code was tested on A100 GPU (Linux), and it works fine doing fine-tuning integration with the latest Flash Attention 2.
Note:
In the current version of code, scGPT uses FlashMHA. And the updated flash-attn2 uses a module named mpa.py. The details of newer MHA implementation can be found HERE starting at line 354 flash_attn/modules/mha.py

In ur latest changes, u have set the use_flash_attn=False  in model.py.  I have also modified the code to it. It seems like the model actually didn't call the flash-attn but call the standard mha. And the metric result of my "tutorial_annotation" is terrible. But if I don't set use_flash_attn to False as you did, the program would get some error about padding_mask. Do u have the same problem?

Update Comments
Modified scgpt/model/model.py & scgpt/model/multiomic_model.py for flash-attn2. Updated README.md with flash-attn2 installation and notes.
Updated Dependencies
flash-attn 1.x --> flash-att 2.x
Is Code Tested?
Yes. The updated code was tested on A100 GPU (Linux), and it works fine doing fine-tuning integration with the latest Flash Attention 2.
Note: In the current version of code, scGPT uses FlashMHA. And the updated flash-attn2 uses a module named mpa.py. The details of newer MHA implementation can be found HERE starting at line 354 flash_attn/modules/mha.py

@Ding3LI
Hello, thanks for your suggestions. I have also updated to flash-attention 2.4 following your suggestions, but the performance in the zero-shot /Tutorial_ZeroShot_Integration_Continual_Pretraining.ipynb is very poor. Have you encountered the same issue? The first image shows the performance using flash-attention 2.4, and the second image shows the performance without using flash-attention


Python program does not run on latest visual code. It does nothing when run button hit.

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.77.3
OS Version: Windows_NT x64 10.0.22621

Steps to Reproduce:

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!

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


failed to validate the PCClient event log in VMware

I've created the following script for testing proposes and I'm running it on a VMware VM with a virtualized TPM with RHEL8 as the OS. I got the following error when running server.VerifyAttestation:
failed to validate the PCClient event log:
failed to replay event log: event log failed to verify: the following registers failed to replay: [0]
Is this expected to happen? I'm thinking about testing this on my real machine with the actual TPM but I afraid to break something.
The code is pretty much the same as the verify_test.go one but using VMwares virtualized TPM.
`package main
import (
"crypto"
"fmt"
"github.com/google/go-tpm-tools/client"
"github.com/google/go-tpm-tools/server"
"github.com/google/go-tpm/tpm2"

)
func main() {
tpm, err := tpm2.OpenTPM()
if err != nil {
	fmt.Println("[ERROR] opening tpm failed: ", err)
	return
}

AttestationKeyRSA, err := client.AttestationKeyRSA(tpm)
defer AttestationKeyRSA.Close()

if err != nil {
	fmt.Println("[ERROR] RSA AK Creation failed: ", err)
	return
}
fmt.Println("[INFO] AK Created: ", AttestationKeyRSA)

nonce, err := tpm2.GetRandom(tpm, 15)

if err != nil {
	fmt.Println("[ERROR] nonce creation failed: ", err)
	return
}
fmt.Println("[INFO] nonce Created: ", nonce)

attestation, err := AttestationKeyRSA.Attest(client.AttestOpts{Nonce: nonce})

if err != nil {
	fmt.Println("[ERROR] Attestation failed: ", err)
	return
}

MachineState, err := server.VerifyAttestation(attestation, server.VerifyOpts{
	Nonce:      nonce,
	TrustedAKs: []crypto.PublicKey{AttestationKeyRSA.PublicKey()},
})

if err != nil {
	// handle error
	fmt.Println("[ERROR] VerifyAttestation failed: ", err)
	return
}

fmt.Println("INFO: Grub State", MachineState)

}`

This is not unexpected. See https://github.com/google/go-tpm-tools/issues/141 for another example.
Basically, this error means the event log located at /sys/kernel/security/tpm0/binary_bios_measurements failed to replay against PCR0 (BIOS measurements, including CRTM). It could be a firmware bug as firmware has been known to log a different measurement in the event log than what they measure into the TPM.
Feel free to dump that here and I can take a look, or you can also examine the event log yourself by running tpm2_eventlog /sys/kernel/security/tpm0/binary_bios_measurements. (https://github.com/tpm2-software/tpm2-tools/blob/master/man/tpm2_eventlog.1.md)

okay, no worries! I can check it. One question though, how risky is it to run this types of routines agains the real TPM? I don't need to do more than a credential activation to verify that the AK comes from a real TPM as shown on the readme of go attestation, and after that do boot measurements. I think it's risk free right? Because I tried this same program on ubuntu 22.04 and it didn't work also. It might be VMware the one causing issues.

I think the only mutating command you're running above is in creating the AttestationKey:
AttestationKeyRSA, err := client.AttestationKeyRSA(tpm). This is cached, however, and should not create a new one if you call it again (provided the template is the same)
Everything else reads from the TPM or the firmware event log and does not modify the TPM.


Issue with AdyenCardComponent with stored payment method on IOS

Hello Adyen
I am feeding the following paymentMethod to adyenCardComponent to trigger the payment with stored payment method behavior
paymentMethod = {
        'brand': cardTypeToString[widget.selectedCreditCard!.cardType]!['brand'],
        'holderName': widget.selectedCreditCard!.holderName,
        'id': widget.selectedCreditCard!.cardId,
        'lastFour': widget.selectedCreditCard!.number,
        'name': cardTypeToString[widget.selectedCreditCard!.cardType]!['name'],
        'supportedRecurringProcessingModels': ["Ecommerce", "ContAuth"],
        'type': 'scheme',
      };

I am receiving the expected result on Android but not on IOS (blank screen)
I think it should have a consistent behavior on both platforms
how can I trigger the payment with existing card (storedPaymentMethod) on IOS as well?

Hi @Shahin-Sefidgaran,
Thanks for sharing your paymentMethod object. As assumed, stored cards are supported on iOS as well. While Android displays the card component, iOS renders a prompt for entering the CVC. The screenshot shows the example app with a stored card.
By comparing your paymentMethod object with the example, I identified that these three fields are missing:
'expiryMonth' : 'MM',
'expiryYear': 'YY',
'supportedShopperInteractions': ["Ecommerce", "ContAuth"]
The value for supportedRecurringProcessingModels should be as set of  ["CardOnFile",  "Subscription", "UnscheduledCardOnFile"]. You can find more information in our docs.
Do you have the data for these fields and could you try adding them to the paymentMethod object as well?

Hi @Shahin-Sefidgaran,
I would like to follow up on your question? Have you been able to try out adding the fields?