[FEATURE] Enhance ML Inference Search Request Processor to carry over the query metadata fields.

Is your feature request related to a problem?
Currently, when rewriting query type in ML Inference Search Request Processor, users can build up a new query using query_template parameters when configuring the processor. For example, rewriting neural search query into knn query.
However, the query meta fields in search request request body will be ignored.
For example, configuring a ML Inference Search Request Processor with a cohere embedding model, cohere.ai/v1/embed,  as below:
PUT /_search/pipeline/my_pipeline_neural_search
{
  "request_processors": [
    {
      "ml_inference": {
        "tag": "ml_inference",
        "description": "This processor is going to run ml inference during search request",
        "model_id": "K7WVcZEBXV92Z6odCZGJ",
        "query_template": """{
                              "query": {
                                "knn": {
                                  "review_embedding": {
                                    "vector": ${modelPredictionOutcome},
                                    "k": 5
                                  }
                                }
                              }
                            }""",
        "function_name": "REMOTE",
        "input_map": [
          {
            "texts": "query.neural.review_embedding.query_text"
          }
        ],
        "output_map": [
          {
            "modelPredictionOutcome": "embeddings[0]"
          }
        ],
        "ignore_missing": false,
        "ignore_failure": false
      }
    }
  ]
}

common use case to call query with query string:
GET /review_string_index/_search?search_pipeline=my_pipeline_neural_search
{
  "query": {
    "neural": {
      "review_embedding": {
        "query_text": "good review",
        "k": 5
      }
    }
  }
}

and it will rewrite to
GET /review_string_index/_search
{
  "query": {
    "knn": {
      "review_embedding": {
        "vector": "<model inference vector>",
        "k": 5
      }
    }
  }
}

However, if I add the meta datafield _source in search request body, for example,
GET /review_string_index/_search?search_pipeline=my_pipeline_neural_search
{
  "_source": {
    "excludes": [
      "review_embedding"
    ]
  },
  "query": {
    "neural": {
      "review_embedding": {
        "query_text": "good review",
        "model_id": "K7WVcZEBXV92Z6odCZGJ",
        "k": 5
      }
    }
  }
}

It will still rewrite the same knn query and the meta datafield _source will be ignored
GET /review_string_index/_search
{
  "query": {
    "knn": {
      "review_embedding": {
        "vector": "<model inference vector>",
        "k": 5
      }
    }
  }
}

What solution would you like?
Maybe try to add a parameter to opt in carry over query field other than query string, including _source, sort,  search_after, etc.
What alternatives have you considered?
A clear and concise description of any alternative solutions or features you've considered.
Do you have any additional context?
Add any other context or screenshots about the feature request here.

I think that would be a very useful feature and would love to see that implemented.
I'm currently trying to use the ml_inference processor in a hybrid search scenario which is giving me a hard time and I wonder if the implementation of this feature request would let me overcome the obstacles.
For context, I specify the hybrid search query together with the normalization-processor as part of a search_pipeline in the query_template of the ml_inference processor:
PUT _search/pipeline/ml_inference_pipeline
{
  "description": "search with predictions",
  "request_processors": [
    {
      "ml_inference": {
        "function_name": "remote",
        "full_response_path": True,
        "model_id": model_id,
        "model_input": """{ "parameters": {"input": "${input_map.features}"}}""",
        "query_template": """{
  "_source": {
    "excludes": [
      "title_embedding"
    ],
    "includes": "product_title"
  },
  "query": {
    "hybrid": {
      "queries": [
        {
          "multi_match": {
            "type": "best_fields",
            "fields": [
              "product_id^100",
              "product_bullet_point^3",
              "product_color^2",
              "product_brand^5",
              "product_description",
              "product_title^10"
            ],
            "operator": "and",
            "query": "iphone"
          }
        },
        {
          "neural": {
            "title_embedding": {
              "query_text": "iphone",
              "k": 50
            }
          }
        }
      ]
    }
  },
  "size": 1,
  "track_total_hits": true,
  "search_pipeline": {
    "phase_results_processors": [
      {
        "normalization-processor": {
          "normalization": {
            "technique": "l2"
          },
          "combination": {
            "technique": "arithmetic_mean",
            "parameters": {
              "weights": [
                ${keywordness},
                ${neuralness}
              ]
            }
          }
        }
      }
    ]
  }
}""",
        "input_map": [
          {
            "features": "query.term.features.value"
          }
        ],
        "output_map": [
          {
            "neuralness": "$.inference_results[0]output[0]dataAsMap.neuralness",
            "keywordness": "$.inference_results[0]output[0]dataAsMap.keywordness"
          }
        ],
        "ignore_missing": False,
        "ignore_failure": False
      }
    },
    {
      "neural_query_enricher": {
        "description": "one of many search pipelines for experimentation",
        "default_model_id": "i6jHTZMBflg_ePyfu9EK",
        "neural_field_default_id": {
            "title_embeddings": "i6jHTZMBflg_ePyfu9EK"
          }
      }
    }
  ]
}

Calling it returns a hitlist with scores that are not normalized.
Query:
POST ecommerce/_search?search_pipeline=ml_inference_pipeline

{
"query": {
  "term": {
    "features": {
      "value": "2, 0, 169, 1.1657, 8.58744, 0.67777"
      }
    }
  }
}

Response:
{
  "took": 74,
  "timed_out": false,
  "_shards": {
    "total": 1,
    "successful": 1,
    "skipped": 0,
    "failed": 0
  },
  "hits": {
    "total": {
      "value": 15129,
      "relation": "eq"
    },
    "max_score": 80.15181,
    "hits": [
      {
        "_index": "...",
        "_id": "...",
        "_score": -9549512000,
        "_source": {...}
      }
    ]
  }
}

Or am I missing something? When the hybrid search query is part of the query_template the normalization-processor appears to be ignored. My gut feeling is because we are defining a search_pipeline (with phase_results_processors) in a search_pipeline  (with the ml_inference processor).

@wrigleyDan this is a pretty complicated use case,
you create a search pipeline that has a ml inference request processor and a request neural_query_enricher, this should be fine.
But within the ml inference request processor, you also rewrite it with a inner search pipeline that has a normalization-processor which is a phase_results_processor. I have to debug a bit and see within ml inference request processor to confirm if the inner pipeline is created. I am not sure about this point.

@mingshl thanks for getting back to me!
FYI: I opened a feature request today and briefly discussed it in today's search community meeting: https://github.com/opensearch-project/OpenSearch/issues/16775#issuecomment-2518221994
Seems like there is path moving forward which is referenced in the comments.


need to send individual message to client

Please implement sending of individual messages to stomp client.
Great thanks for your work!

I figured out how to do it

@krtl Could you please share how you did it?

Unfortunately, I've done this by amending stompServer.js
2018-08-01 14:07 GMT+03:00 Bijay Dash<EMAIL_ADDRESS>
@krtl https://github.com/krtl Could you please share how you did it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/4ib3r/StompBrokerJS/issues/5#issuecomment-409538523,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ab6oZlzvq7vuxbnNu8ctxKGBrsgLfG_Iks5uMYvxgaJpZM4QwFZD
.

ok, I made two things:

add socket param to emit of 'send' event line 112 in original stompServer.js:
this.emit('send', {frame: {headers: frame.headers, body: bodyObj}, dest: args.dest, socket});

add following func to original stompServer.js

/** SendIndividual message to specific client

@param {socket} client
@param {object} headers Message headers
@param {string} body Message body */
this.sendIndividual = function (socket, topic, headers, body) {
var _headers = {};
if (headers) {
for (var key in headers) {
_headers[key] = headers[key];
}
}
var frame = {
body: body,
headers: _headers
};
var args = {
dest: topic,
frame: this.frameParser(frame)
};

var bodyObj = args.frame.body;
var frame = this.frameSerializer(args.frame);
var headers = { //default headers
  'message-id': stomp.genId("msg"),
  'content-type': 'text/plain'
};
if (frame.body !== undefined) {
  if (typeof frame.body !== 'string')
    throw "Message body is not string";
  frame.headers["content-length"] = frame.body.length;
}
if (frame.headers) {
  for (var key in frame.headers) {
    headers[key] = frame.headers[key];
  }
}

for (var i in this.subscribes) {
  var sub = this.subscribes[i];
  if (socket.sessionId !== sub.sessionId) {
    continue;
  }
  var match = this._checkSubMatchDest(sub, args);
  if (match) {
    args.frame.headers.subscription = sub.id;
    frame.command = "MESSAGE";
    if (socket !== undefined) {
      StompUtils.sendFrame(socket, frame);
    }
  }
}

};
Usage: In my project I needed only to reply to individuals:
stompServer.on('send', (ev) => {
//...
stompServer.sendIndividual(ev.socket, TOPIC_NAME, {}, REPLY_TEXT);
});
the same code for reply on subscribe:
stompServer.on('subscribe', (ev) => {
//...
stompServer.sendIndividual(ev.socket, TOPIC_NAME, {}, REPLY_TEXT);
});


Random seed value for Best ScanObjectNN Results

Hi Authors/Anyone,
Can you please share the seed value (from the log files) that gave 88.1% (87.7+0.4) accuracy on ScanObjectNN?
We know the seed value is randomly generated, but it is stored in the log file for that run.
It would greatly help in recreating results closer to the claimed ones.
Thank you so much for your attention and participation.
P.S: The best I have created is 87.6% accuracy (seed 7719).

Hi Authors/Anyone,
Can you please share the seed value (from the log files) that gave 88.1% (87.7+0.4) accuracy on ScanObjectNN? We know the seed value is randomly generated, but it is stored in the log file for that run.
It would greatly help in recreating results closer to the claimed ones.
Thank you so much for your attention and participation.
P.S: The best I have created is 87.6% accuracy (seed 7719).

As mentioned in the pre-trained model, the seed is 6166. You can find it in the pre-trained log file. https://drive.google.com/drive/folders/1A584C9x5uAqppbjNNiVqlA_7uOOOlEII

Thank you so much


Adding more control over settings

I've changed the constructor to pass a settings delegate in which includes the ability to control caching, addins and output files
example usage:
StyleCopAnalyse(settings =>
            settings
                .WithSolution(solutionFile)
                .WithSettings(settingsFile)
                .WithAddins(path1, path2)
                .UsingResultsCache(true)
                .LoadFromDefaultPath(false)
                .ToResultFile(resultFile)
        );

Now added additional html report generation
StyleCopAnalyse(settings => settings
  .ToHtmlReport(htmlFile)
);

Thanks for the PR, I'll review and approve soon!

Thanks :smile:


Plant Property And Equipment Value is Incorrect.

import yfinance as yf
stock = yf.Ticker(ticker)
dfBS = stock.balance_sheet
print(dfBS)

returns (among other things)Property Plant Equipment   4.533600e+10  3.737800e+10  4.130400e+10  3.378300e+10
The first column is incorrect and does not match Yahoo finance. However, the rest of the years are correct. I have found all the other data returned to be correct.

Not sure which ticker you are referring to, please provide an example.

Sorry, for the example above I used AAPL.

Fixed in yfinance 0.2


 Why does it happen and how can I make it exit without closing the terminal and hence killing the process?

Hi @CsatiZoltan, I quickly checked the script with brave browser and it works for me. I assume the error is coming from the dependency puppeteer-core. Could you please try to reinstall the latest version of puppeteer-core to see if it works?

Hi. I installed the dependencies as given in the README:
npm i puppeteer-core commander

which gives
npm WARN saveError ENOENT: no such file or directory, open '/home/zoltan/package.json'
npm WARN enoent ENOENT: no such file or directory, open '/home/zoltan/package.json'
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN zoltan No description
npm WARN zoltan No repository field.
npm WARN zoltan No README data
npm WARN zoltan No license field.

+<EMAIL_ADDRESS>+<EMAIL_ADDRESS>updated 2 packages and audited 95 packages in 0.84s
found 0 vulnerabilities

Version 8 is the latest.

Hi @CsatiZoltan. thanks for the info. And after that, run the script works or shows you the same error as well?

The same happens. As you can see in the video below, I get the translated text but it does not exit.
https://user-images.githubusercontent.com/12742890/109350182-5d7f5780-7877-11eb-9a06-63a558c22d73.mp4

@CsatiZoltan Ok, weird. I will take another look if it's something wrong in the script. By the way, you don't need to close the terminal, to kill the running process, ctrl+c can help.

Yes, that's true...

@CsatiZoltan ok, it seems the old nodejs version may cause this error. What's your node version? node -v. I think if you upgrade it to the latest v14 or v15 may help to solve the error.

I have v10.13.0. How can I update it safely (I don't want to mess up my installation)?

@CsatiZoltan You may try to use nvm https://github.com/nvm-sh/nvm, it can help you to switch between different nodejs version quickly.

Trying with version v15.10.0, I don't get the (node:7097) ExperimentalWarning: The fs.promises API is experimental warning any more but the process still doesn't finish by itself.

Hi @CsatiZoltan, It looks like we are having a steady progress :smile:
I do not see any reason why the browser is not closed. As you can see in the translate.js, there is no other code between showing translated result and closing browser. I need more info, if you can help to run this command env DEBUG="puppeteer:*" env DEBUG_COLORS=false ./translate.js -p /usr/bin/brave-browser -t de 'hold my beer' 2>&1 | grep -v '"Network' > debug.log. This line of command will generate debug logs into debug.log file.  If you could post the logs here, I will take a further look.

Hi @CsatiZoltan, I just updated translate.js to simplify some codes. It may solve your issue indirectly. Give it a try?

Hi. I executed
env DEBUG="puppeteer:*" env DEBUG_COLORS=false ./translate.js -p /usr/bin/brave-browser -t de 'hold my beer' 2>&1 | grep -v '"Network' > debug.log

with the new version of the translate.js script, but it didn't seem to solve the issue. Here is the log file.
debug.log

Hi @CsatiZoltan, thanks for testing. Ehm... the attached debug.log is truncated in the middle of running process.
Could you please run this line of command env DEBUG="puppeteer:*" env DEBUG_COLORS=false ./translate.js -p /usr/bin/brave-browser -t de 'hold my beer' 2>&1 | grep -v '"Network' | tee debug.log, it will print out the logs on the screen and meanwhile logging them into debug.log. Please leave the script  run for another 5 seconds after seeing the translation text Halte mein Bier. If it's not finished after 5 seconds, then kill the script. It would be appreciated if you could post result with debug.log. Thanks!

shameless workaround: replace puppeteer with jsdom @ https://github.com/milahu/deepl-client-jsdom

@milahu Use jsdom to replace puppeteer is a good idea in general. However, looking at he exec time ~5s, it feels there is still the potential to improve the speed :)

im trying ^^
echo 'humans are the virus' | node deepl-cli.js en zh --time

time: 2.392 sec to require jsdom
time: 0.018 sec to require getPath
time sum: 2.425 sec to parse script

time: 0.007 sec to read stdin
time: 0.89 sec to load jsdom.fromURL
time: 1.275 sec to load document
time: 1.551 sec to get targetText
time sum: 3.727 sec to get targetText

time sum: 6.152 sec to parse script and get targetText

人类是病毒

require jsdom is faster with snapshots
require jsdom load jsdom.fromURL load document can be avoided by talking to the backend directly
... or by running the client as a daemon
get targetText is the bottleneck, mostly waiting for the deepl server
at least 0.3 sec of get targetText is rate limiting on the client side
-> debug mode says [Dict] Delaying dictionary lookup by 300
so, this short input could be translated in 1.2 sec
how fast does puppeteer load? ; )

Hey all, since there hasn’t been any information in the last three years, I’m closing this issue. If you encounter the same problem or have a different issue, feel free to leave a comment here or open a new one. Thanks.


Use only one time parameter when quering multiple pages

If a resource has multiple pages the script will use next value to retrieve next page.
Because all requests are done with datetime append to url (probably to avoid caching) next containes that link.
https://api.github.com/repos/user/repo/issues/events?per_page=100&1409248439577
Second page will be requested by appending a new datetime, so we'll have something like
https://api.github.com/repos/user/repo/issues/events?per_page=100&page=2&1409248439577=&1409257105342
Third page url will contain 3 datetime stamps and so on. At some point this will rise an error as browser (and server) has a limit for url size. This way it will cut some of the parameters or will simply throw an error.

Sounds like a plan. I'm not sure two regex replaces are the best or most performant option, though; also, it's a really old PR and I'd like some unit tests to ensure it doesn't break anything else. Mind doing so? Thanks!

Closed in error; reopening.


[GHSA-44vr-rwwj-p88h] Shescape vulnerable to insufficient escaping of whitespace

Updates

CWEs

Hi there @ericcornelissen! A community member has suggested an improvement to your security advisory. If approved, this change will affect the global advisory listed at github.com/advisories. It will not affect the version listed in your project repository.
This change will be reviewed by our highly-trained Security Curation Team. If you have thoughts or feedback, please share them in a comment here! If this PR has already been closed, you can start a new community contribution for this advisory


Fix link to quickstart docs in README.md

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Make tests pass when no proxy configured

If you do not have a proxy configured using System Properties then HttpClientExamples fails.

java.lang.IllegalArgumentException: 'http.proxyHost' and/or 'http.proxyPort' were not specified as system properties, use -D or set programatically


Add the psych engine logo to the github readme (and change the icon to the psych engine one)

What feature do you want to get added on the base engine?
Title (I couldn't figure out how to shorten it so yeah).
Kind of surprised that no one's done this yet.
I'm talking about the one made by riveren used in the two psych engine ost videos by iflicky - https://www.youtube.com/watch?v=mX9sgiSUf5g and https://www.youtube.com/watch?v=a7ksO5xVJU8
And while you're at it, you might want to consider changing the icon over the psych engine one:

(If you're wondering, I found it in the mind games megafolder. It was a bit out of proportion, but I fixed it)
To test your sight, and reliability, please select the option of what should NOT be requested.
Stage Editor, 6K+ support, and winning icons.

but why

why?
theres no point

Well, none of the other issue templates fit this suggestion any better, nor is there a miscellaneous/other template.
Anyway, I don't really know how to explain it. It's like if a company employee created a cool, awesome logo, but the company decided not to use it and stick with no logo (and just plain text) for their branding and marketing because???

Ah well, issue's closed. At least I tried though.


Update dependencies and use node 8

It's been a while since the dependencies were updated so I just updated it all of them and run the test locally. If needed I can put the test to run in the GitHub actions

@kevva can you please review this please?

@kevva can you please review it?


Reduce flakiness of end-to-end tests in CI

Use --runInBand when running integration tests in the CI (source).
Use jest.retryTimes(3) to retry failing integration tests 3 times (source, thanks for the suggestion @adamrusted).

This required upgrading to Jest v27 (c79f1c0b99a7f86958e20fe93bbd756fcfe435a1), as it introduced jest-circus as the default test runner.

Use Jest's fake timers to prevent open handles when test suites finish (source, found using --detectOpenHandles).

Miscellaneous changes

Rename "integration" tests to "end-to-end" tests.
Remove unused variable in search.test.js.

I'm merging this in with just the one approval now in an effort to prevent flaky tests from affecting contributions from outside of the simple-icons organization. If anyone still has suggestions, feel free to share them here or open a separate issue/Pull Request for it :slightly_smiling_face:


MH_LoadEngine: Failed to locate VideoMode_SearchBase

Used the one-click installer for Sven Coop and launched from the shortcut, but get this error:

Using the latest build

using the july 29 release fixed it for me.
haven't tested it but later releases probably only work on 5.26 version builds, but im probably wrong.

Fixed in https://github.com/hzqst/MetaHookSv/releases/tag/v20241007a

Thank you, the fix works


Updated documentation to reflect the method for ip address

Changes documentation of ipAddress() to getIPAddress()

Good catch. Thanks!


feat: support custom filters

Closes #9

Codecov Report

Merging #10 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #10   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           2      2
  Lines         152    151    -1
  Branches       11     12    +1
=====================================
- Hits          152    151    -1

Impacted Files
Coverage Δ

src/index.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


[DNM]region bounds as compaction guards

used to build binary for testing.

/release

/release

/release

/test


The IDF1 value is very low

Hello authors, I have tested using your code on the S02 dataset of AIC22, the detector used is yolov7-e6, I successfully got the mtmc_resnet50_ibn_a_gap.txt file, but when I evaluated the trace results using the eval.py file in the AIC22 eval folder, I got Very low IDF1 values where the IDP values are reasonable but the IDR values are very unreasonable, I am confused about this, I would like to ask you how you evaluated the final trace resolution, looking forward to your reply!

Hello.
We guess that the problem is maybe because of the box expansion at the last part of our run_mtmc.py.

We added this code because the GT boxes in AIC19 is annotated lot bigger than the objects meanwhile the detector regresses tight bounding boxes.

We are not sure since we do not have the AIC22 dataset, but it is possible that in AIC22 the GT boxes are refined tightly.
So the predicted boxes do not match the GTs, which is why the performance is as it is.
We hope our answer helps you to solve the problems.
Thank you for your attention.

Thank you very much for your reply, based on your reply I commented out the code you mentioned and re-ran run_mtmc.py but unfortunately I got a lower IDF1 score.

Considering that I was using the AIC22 Track1 dataset, the dataset may have changed, so I went to the AICity Challenge website to download the AIC19 dataset, but unfortunately there is no longer an option for datasets prior to 2021 in the dataset request form on the AICity Challenge website.

I would be grateful if you could provide me with the ground_truth_validation.txt file of AIC19 or the download link of AIC19 to help me validate the model.
Also I would like to ask if your naming convention for video frame images is the same as in my images.

Hi, if you only made predictions for S02 scene, the other scene information in ground_truth_validation is not necessary for evaluating the prediction accuracy.


Add Waterfall Network Proposal

Proposal for Waterfall Network to become a Hyperledger project with incubation status.

Going to close this PR for now since we have a lab. When you have taken care of the licensing issues and are ready to request incubation again, please either re-open this PR or a new PR.


Adding /accounts listing endpoint.

Signed-off-by: mugdhagoel<EMAIL_ADDRESS>

These queries took about 400 ms on a non-full set of data for accounts.


Connecting to mLab DB failes

I can connect to local database just fine, but connecting to a DB hosted in mLab results in Auth error.
I am using the same DB in other projects and they are working just fine.

because mongodb running V3.0+  uses  ScramSHA1 as auth provider instead of mongocr.
There is an PR on this
https://github.com/gordonmleigh/promised-mongo/pull/36
alternatively, you can npm install using the following github repo, which is just a fork of this repo with that PR merged in
https://github.com/mindeavor/promised-mongo

Will try.
But, the links in your answeres are going nowhere when clicked, I had to copy paste :1234: .

fixed thanks for letting me know


CONTRIBUTING.rst mentions outdated "nox -s docs" command

CONTRIBUTING.rst instructs contributors to build the docs with nox -s docs, but this session was removed in #1271.
It is unclear if and how the docs should be regenerated.

Is this referring to the API documentation currently found in docs/epy?

No, this is about the CONTRIBUTING.rst file at the root of the project tree.
https://github.com/googleapis/google-api-python-client/blob/33762a6405456ca7eae3b5edfbb130eacdda7912/CONTRIBUTING.rst?plain=1#L178


[SPARK-16381][SQL][SparkR] Update SQL examples and programming guide for R language binding

https://issues.apache.org/jira/browse/SPARK-16381
What changes were proposed in this pull request?
Update SQL examples and programming guide for R language binding.
Here I just follow example https://github.com/apache/spark/compare/master...liancheng:example-snippet-extraction, created a separate R file to store all the example code.
How was this patch tested?
Manual test on my local machine.
Screenshot as below:

Test build #61881 has started for PR 14082 at commit e7def7c.

Test build #61881 has finished for PR 14082 at commit e7def7c.

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

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61881/
Test PASSed.

@shivaram @mengxr It would be nice if any of you can help review this one, thanks!

I'll take a look at this today. Also cc @felixcheung

looks good, only this minor comment: https://github.com/apache/spark/pull/14082#discussion_r70001842
and some improvement suggestions to example code.

thanks a lot @felixcheung , working on it

Test build #61946 has started for PR 14082 at commit 1af09f3.

Test build #61946 has finished for PR 14082 at commit 1af09f3.

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

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61946/
Test PASSed.

Test build #61970 has started for PR 14082 at commit 828b2cf.

Test build #61971 has started for PR 14082 at commit 7dca42d.

Test build #61970 has finished for PR 14082 at commit 828b2cf.

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

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61970/
Test PASSed.

Merged build finished. Test PASSed.

Test build #61971 has finished for PR 14082 at commit 7dca42d.

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

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61971/
Test PASSed.

Test build #61993 has started for PR 14082 at commit cd184b3.

Test build #61995 has started for PR 14082 at commit d5b0b7f.

Test build #61993 has finished for PR 14082 at commit cd184b3.

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

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61993/
Test PASSed.

Test build #61995 has finished for PR 14082 at commit d5b0b7f.

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

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61995/
Test PASSed.

Thanks @keypointt -- Changes look good to me. @felixcheung any other comments ?

looks good except for one comment. thanks for putting this together!

Test build #62004 has started for PR 14082 at commit a1eca2b.

LGTM

I just did a quick scan, the no-space style like  session(appName='MyApp', sparkConfig=list(spark.executor.memory="1g")) is only found in example/ folder (in above commit), and all r files are following space style under R/pkg/

Test build #62005 has started for PR 14082 at commit 7195750.

Thanks @keypointt -- LGTM. Will merge this once Jenkins passes

Thank you for reviewing :)

Test build #62005 has finished for PR 14082 at commit 7195750.

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

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62005/
Test PASSed.

Test build #62004 has finished for PR 14082 at commit a1eca2b.

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

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62004/
Test PASSed.

Since both @shivaram and @felixcheung signed this off, I'm merging this to master and branch-2.0.
Thanks @keypointt for working on this and @shivaram and @felixcheung for the review!


Update documentation

As part of the introduction of the new builder API (see #471), the README documentation, samples etc. need to be updated to use the new API.

More diagrams
I'd like to see some diagrams that explain how the various options present in the bus configuration map to different architectures. Ideally these might be described using a few different 'patterns'.
ie:
Configuration Pattern Pub/Sub

publisher configuration

                config.Publications(x =>
                {
                    x.WithTopic<X>();
                });

subscriber1 configuration

                config.Subscriptions(x =>
                {
                   x.ForTopic<X>();
                });

subscriber2 configuration

                config.Subscriptions(x =>
                {
                   x.ForTopic<X>();
                });

Resulting architecture

Producing the diagrams

PlantUML sequence diagrams seem to lend themselves to this kind of documentation.
They are plain text and the markup is easily learnt just through experimentation
They are linkable
There is a website that renders the below to an image: https://www.planttext.com/
There is a visual studio code plugin that can render them using a plantuml server
There is a docker image for the server

 docker run -d -p 8080:8080 plantuml/plantuml-server:jetty

PlantUML used to produce the above
@startuml component

title PubSub using JustSaying and AWS SNS/SQS
hide footbox
skinparam BackgroundColor #EEEBDC
skinparam BoxPadding 20
skinparam ParticipantPadding 20

box "Publishing\nApplication" #lightgrey
    participant Publisher order 1
end box
box "AWS" #orange
    participant "AWS SNS Topic" as SNS_Publisher order 2
    participant "AWS SQS Queue #1" as SQS_Subscriber1 order 3
    participant "AWS SQS Queue #2" as SQS_Subscriber2 order 4
end box
box "Subscribing\nApplication #1" #lightgrey
    participant "Subscriber" as Subscriber_1 order 5
end box
box "Subscribing\nApplication #2" #lightgrey
    participant "Subscriber" as Subscriber_2 order 6
end box

== Publishing ==
group publishing
    Publisher-[#Green]>SNS_Publisher: Publish message to SNS Topic
    SNS_Publisher-[#gray]>SQS_Subscriber1: Deliver message to SQS Queue
    SNS_Publisher-[#gray]>SQS_Subscriber2: Deliver message to SQS Queue
    return ok
end

== Polling ==
group subscriber polling
    SQS_Subscriber1<[#Green]-Subscriber_1: List messages
    return Message returned
end
group subscriber polling
    SQS_Subscriber2<[#Green]-Subscriber_2: List messages
    return Message returned
end

== Acknowledging ==
group subscriber acknowledging
    SQS_Subscriber1<-[#Green]-Subscriber_1 !! : Delete message
end
group subscriber acknowledging
    SQS_Subscriber2<-[#Green]-Subscriber_2 !! : Delete message
end
@enduml


add source locations to errors

It would be nice to annotate assignments to ec with source code locations. This can be done with a special macro.

This was implemented (across multiple PRs).


Always use current year for docs footer

Big fail this PR xD. Neither linter was happy and code was not working hahahaha


Use new official images

First of all thanks for sharing this setup! It's lean and uses sane defaults, perfect for prototyping. :+1:

The images from the Docker Hub are now deprecated in favour of new images maintained by Elastic.
Notable changes:

These images include X-Pack by default, therefore there is no need to install it explicitly
The Logstash configuration is split between config/ (holds the agent configuration) and pipeline/ (holds the input/output plugins configuration)

I would gladly backport this PR to the other branches as well but I would like to know your opinion first regarding the way to proceed in order to disable authentication. Would you rather:

enable anonymous auth?
disable it using configuration flags?
uninstall X-Pack in all images?
something else?

Hi @antoineco ! Thanks for the words and for the work !
I'd accept a PR that disable X-Pack using configuration flags for the master branch, I think that should be good enough and would keep the setup simple.
I did a quick review of your PR, nice work, it looks even simpler now :)

Alright, thanks for your answer, let me send a separate PR.


Avoid blurry images and be retina ready

Reference to issue #3381. It's described here that you can resize gravatar images. Since 320px is the double of 160px, the gravatar should also be retina ready.

Thanks @jonathan-s! :-)


Not picking up XState machine on reload

Thanks for this great tool!
I'm developing a React app using webpack-dev-server. The first time I start the server, Ninja works as expected. However, if I reload the page, I get the 'No state machines were found on the current page' screen, and I need to restart the dev server to get Ninja working again.
Redux dev tools pick up the transitions regardless.

Thanks for the report. Can you make a repro in Codesandbox please?

Not sure how I would get webpack-dev-server and xstate-ninja running in a Codesandbox.

Please update your web extension to v1.7.1.


ability to load RDS files

Example RDS file: https://github.com/DARPA-ASKEM/program-milestones/blob/main/Forecasting Challenges/Data/T2-cum-deaths-predictions.RDS

The HMI side of this task is done, however, it requires a chunk of work to be done on the beaker side to arbitrarily support switching file types on the fly. Pausing this for now.


Uploading ipa to TestFlight with --api_key results in error

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
I'm trying to deploy an ipa to testflight with the pilot command. But it results in an error.
Command executed
 fastlane pilot upload  --ipa \"path_to_ipa\"  -g "group_name" --team_name "team_name" --changelog "features and fixes" --api_key "{\"key_id\": \"key_id\", \"issuer_id\": \"my_issuer_id\", \"key_filepath\": \"path_to_p8\"}"
Complete output when running fastlane, including the stack trace and command used
Traceback (most recent call last):
	16: from /usr/local/bin/fastlane:23:in `<main>'
	15: from /usr/local/bin/fastlane:23:in `load'
	14: from /Library/Ruby/Gems/2.6.0/gems/fastlane-2.165.0/bin/fastlane:23:in `<top (required)>'
	13: from /Library/Ruby/Gems/2.6.0/gems/fastlane-2.165.0/fastlane/lib/fastlane/cli_tools_distributor.rb:111:in `take_off'
	12: from /Library/Ruby/Gems/2.6.0/gems/fastlane-2.165.0/pilot/lib/pilot/commands_generator.rb:18:in `start'
	11: from /Library/Ruby/Gems/2.6.0/gems/fastlane-2.165.0/pilot/lib/pilot/commands_generator.rb:166:in `run'
	10: from /Library/Ruby/Gems/2.6.0/gems/commander-fastlane-4.4.6/lib/commander/delegates.rb:15:in `run!'
	 9: from /Library/Ruby/Gems/2.6.0/gems/fastlane-2.165.0/fastlane_core/lib/fastlane_core/ui/fastlane_runner.rb:76:in `run!'
	 8: from /Library/Ruby/Gems/2.6.0/gems/commander-fastlane-4.4.6/lib/commander/runner.rb:476:in `run_active_command'
	 7: from /Library/Ruby/Gems/2.6.0/gems/commander-fastlane-4.4.6/lib/commander/command.rb:153:in `run'
	 6: from /Library/Ruby/Gems/2.6.0/gems/commander-fastlane-4.4.6/lib/commander/command.rb:178:in `call'
	 5: from /Library/Ruby/Gems/2.6.0/gems/fastlane-2.165.0/pilot/lib/pilot/commands_generator.rb:66:in `block (2 levels) in run'
	 4: from /Library/Ruby/Gems/2.6.0/gems/fastlane-2.165.0/pilot/lib/pilot/build_manager.rb:17:in `upload'
	 3: from /Library/Ruby/Gems/2.6.0/gems/fastlane-2.165.0/pilot/lib/pilot/manager.rb:16:in `start'
	 2: from /Library/Ruby/Gems/2.6.0/gems/fastlane-2.165.0/pilot/lib/pilot/manager.rb:20:in `login'
	 1: from /Library/Ruby/Gems/2.6.0/gems/fastlane-2.165.0/pilot/lib/pilot/manager.rb:33:in `api_token'
/Library/Ruby/Gems/2.6.0/gems/fastlane-2.165.0/spaceship/lib/spaceship/connect_api/token.rb:43:in `create': wrong number of arguments (given 1, expected 0) (ArgumentError)
	16: from /usr/local/bin/fastlane:23:in `<main>'
	15: from /usr/local/bin/fastlane:23:in `load'
	14: from /Library/Ruby/Gems/2.6.0/gems/fastlane-2.165.0/bin/fastlane:23:in `<top (required)>'
	13: from /Library/Ruby/Gems/2.6.0/gems/fastlane-2.165.0/fastlane/lib/fastlane/cli_tools_distributor.rb:111:in `take_off'
	12: from /Library/Ruby/Gems/2.6.0/gems/fastlane-2.165.0/pilot/lib/pilot/commands_generator.rb:18:in `start'
	11: from /Library/Ruby/Gems/2.6.0/gems/fastlane-2.165.0/pilot/lib/pilot/commands_generator.rb:166:in `run'
	10: from /Library/Ruby/Gems/2.6.0/gems/commander-fastlane-4.4.6/lib/commander/delegates.rb:15:in `run!'
	 9: from /Library/Ruby/Gems/2.6.0/gems/fastlane-2.165.0/fastlane_core/lib/fastlane_core/ui/fastlane_runner.rb:76:in `run!'
	 8: from /Library/Ruby/Gems/2.6.0/gems/commander-fastlane-4.4.6/lib/commander/runner.rb:476:in `run_active_command'
	 7: from /Library/Ruby/Gems/2.6.0/gems/commander-fastlane-4.4.6/lib/commander/command.rb:153:in `run'
	 6: from /Library/Ruby/Gems/2.6.0/gems/commander-fastlane-4.4.6/lib/commander/command.rb:178:in `call'
	 5: from /Library/Ruby/Gems/2.6.0/gems/fastlane-2.165.0/pilot/lib/pilot/commands_generator.rb:66:in `block (2 levels) in run'
	 4: from /Library/Ruby/Gems/2.6.0/gems/fastlane-2.165.0/pilot/lib/pilot/build_manager.rb:17:in `upload'
	 3: from /Library/Ruby/Gems/2.6.0/gems/fastlane-2.165.0/pilot/lib/pilot/manager.rb:16:in `start'
	 2: from /Library/Ruby/Gems/2.6.0/gems/fastlane-2.165.0/pilot/lib/pilot/manager.rb:20:in `login'
	 1: from /Library/Ruby/Gems/2.6.0/gems/fastlane-2.165.0/pilot/lib/pilot/manager.rb:33:in `api_token'
/Library/Ruby/Gems/2.6.0/gems/fastlane-2.165.0/spaceship/lib/spaceship/connect_api/token.rb:43:in `create': [!] wrong number of arguments (given 1, expected 0) (ArgumentError)

 [REPLACE THIS WITH YOUR INFORMATION]

Environment

 ### Stack

Key
Value

OS
10.15.6

Ruby
2.6.3

Bundler?
false

Git
git version 2.24.3 (Apple Git-128)

Installation Source
/usr/local/bin/fastlane

Host
Mac OS X 10.15.6 (19G2021)

Ruby Lib Dir
/System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib

OpenSSL Version
LibreSSL 2.8.3

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
12.0.1

System Locale

Error

No Locale with UTF8 found 🚫

fastlane files:
No Fastfile found
No Appfile found
fastlane gems

Gem
Version
Update-Status

fastlane
2.165.0
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

did_you_mean
1.3.0

slack-notifier
2.3.2

rouge
2.0.7

xcpretty
0.3.0

terminal-notifier
2.0.0

unicode-display_width
1.7.0

terminal-table
1.8.0

plist
3.5.0

CFPropertyList
2.3.6

public_suffix
4.0.6

addressable
2.7.0

multipart-post
2.0.0

word_wrap
1.0.0

tty-screen
0.8.1

tty-cursor
0.7.1

tty-spinner
0.9.3

babosa
1.0.3

colored
1.2

highline
1.7.10

commander-fastlane
4.4.6

excon
0.76.0

faraday
1.0.1

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20190701

http-cookie
1.0.3

faraday-cookie_jar
0.0.7

faraday_middleware
1.0.0

fastimage
2.2.0

gh_inspector
1.1.3

json
2.1.0

mini_magick
4.10.1

rubyzip
2.3.0

security
0.1.3

xcpretty-travis-formatter
1.0.0

dotenv
2.7.6

naturally
2.2.0

simctl
1.6.8

jwt
2.2.2

uber
0.1.0

declarative
0.0.20

declarative-option
0.1.0

representable
3.0.4

retriable
3.1.2

mini_mime
1.0.2

multi_json
1.15.0

signet
0.14.0

memoist
0.16.2

os
1.1.1

googleauth
0.13.1

httpclient
2.8.3

google-api-client
0.38.0

google-cloud-env
1.3.3

google-cloud-errors
1.0.1

google-cloud-core
1.5.0

rake
13.0.1

digest-crc
0.6.1

google-cloud-storage
1.28.0

emoji_regex
3.0.0

jmespath
1.4.0

aws-partitions
1.364.0

aws-eventstream
1.1.0

aws-sigv4
1.2.2

aws-sdk-core
3.105.0

aws-sdk-kms
1.37.0

aws-sdk-s3
1.79.1

nanaimo
0.2.6

colored2
3.1.2

claide
1.0.3

atomos
0.1.3

xcodeproj
1.16.0

bundler
2.1.4

forwardable
1.2.0

logger
1.3.0

stringio
0.0.2

ipaddr
1.2.2

openssl
2.1.2

ostruct
0.1.0

strscan
1.0.0

date
2.0.0

fileutils
1.1.0

etc
1.0.1

io-console
0.4.7

zlib
1.0.0

libxml-ruby
3.1.0

rexml
3.1.9

psych
3.1.0

mutex_m
0.1.0

webrick
1.4.2

generated on: 2020-10-30

You can try to use --api_key_path argument as a workaround

You can try to use --api_key_path argument as a workaround

That means I have to put that json into a .json file right? Not just a path to the .p8?

You can try to use --api_key_path argument as a workaround

Your suggestion worked. It's maybe even a more clean solution to what I was trying. Still weird that it didn't work though.

IIRC we don't officially support Hash arguments via CLI. Any Hash arguments have to be passed via ruby or env vars. That may be the root cause of this bug 😬
Link to 'a' docs page that shows this statement: https://docs.fastlane.tools/actions/clean_build_artifacts/#cli
Sorry for the inconvenience.

Since it looks like this issue has been resolved, I'm closing this issue 😊 thanks @mrchnk for the solution! 🚀


Tell users to enable reaction updates

Closes #990.

Approved in https://github.com/grammyjs/website/issues/990#issuecomment-1938738998, @roj1512 please nitpick

/cc @dzikrl

whoop whoop


New package: sharkdp.pastel version 0.9.0

Pull request has been created with Komac v1.4.0 :rocket:
Microsoft Reviewers: Open in CodeFlow

This PR is blocked until support for dependencies is implemented in:

https://github.com/microsoft/winget-cli/issues/163

This is already implemented and released to stable in 1.6.

@wingetbot waivers Add Validation-Executable-Error


fix($componentController): expressions passed to $componentController '&'-bindings are now parsed

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Bug fix
What is the current behavior? (You can also link to an open issue here)
Behavior is explained in issue #14426.
What is the new behavior (if this is a feature change)?
The new behavior is the ability to pass expressions to $componentController &-bindings. Example:
  it('should call the onDelete binding when a hero is deleted', function() {
    scope.deleteFn = jasmine.createSpy('deleteSpy');
    component = $componentController('heroDetail',
      {$scope: scope},
      {hero: hero, onDelete: 'deleteFn(hero)'}
    );

    component.onDelete({hero: component.hero});
    expect(spy('deleteSpy')).toHaveBeenCalledWith(component.hero);
  });

The expression is parsed in a way similar to the parsing of a binding during the compilation of a component element.
Does this PR introduce a breaking change?
No.
Please check if the PR fulfills these requirements

[x] The commit message follows our guidelines: https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#commit-message-format
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

I signed it!

It's not properly a bugFix, its a new feature. And I do not see the point, it makes testing more costly. It is the same for the case that you have exposed:
  it('should call the onDelete binding when a hero is deleted', function() {
    component = $componentController('heroDetail',
      {$scope: scope},
      {hero: hero, onDelete:  jasmine.createSpy('deleteSpy')}
    );

    component.onDelete({hero: component.hero});
    expect(spy('deleteSpy')).toHaveBeenCalledWith(component.hero);
  });

In addition your solution is not accurate, it does not takes into account what happens about other cases:
bindToController: {
  fireSelect: '@onSelect',
},

But it should be interesting to add the testing that I have wrote here.

It's either a bugfix in the angular code or a bugfix for the docs. The case you mention above (the one from the docs) does not work.
  it('should call the onDelete binding when a hero is deleted', function() {
    component = $componentController('heroDetail',
      {$scope: scope},
      {hero: hero, onDelete:  jasmine.createSpy('deleteSpy')}
    );

    component.onDelete({hero: component.hero});

    // This expectation will fail, as `deleteSpy` is called with the `$parse` locals context
    expect(spy('deleteSpy')).toHaveBeenCalledWith(component.hero);

    // This expectation will succeed, but is not representative for the way your bound function
    // will be called in the real world
    expect(spy('deleteSpy')).toHaveBeenCalledWith({ hero: component.hero });
  });

My fix (or feature, depending on how you look at it) does not necessarily make testing more costly. Only expression bindings are parsed. You can still pass a function reference into the binding directly and skip the parsing.
My fix does fall short when it comes to named bindings, thanks for noticing! I will update the PR to add this.

I fixed the named binding issue, it now works fine. I also added a test case to prove it.

Indeed docs are wrong:
  it('should call the onDelete binding when a hero is deleted', function() {
    component = $componentController('heroDetail',
      {$scope: scope},
      {hero: hero, onDelete: jasmine.createSpy('deleteSpy')}
    );

    component.onDelete({hero: component.hero});
    expect(spy('deleteSpy')).toHaveBeenCalledWith({hero: component.hero});
  });

IMHO I would appreciate a PR to fix docs (it is a pitty that there are not unit tests for the docs). But add a test first to mocksSpec.js, please.
About not to change $componentController is that semantics can be a little weird for <  and  = if we try to make them consistent. Now they are expected to take values as is, but if & takes a string and parses it from scope, ¿what to expect from < and = in such cases? Example:
    component = $componentController('colorShowroom',
      { $scope: scope},
      { color: 'red' } // bindings: { color: '<' }
    );

So in this case, what is red? ¿the string value expected for color or the value of red inside the $scope?
If it is not changed, it might be confusing, some bindings getting parsed and others not.
And even worst, $componentController is analogous to $controller, both have the same interface and both create an instance of a $controller, why one version behaves different from the other?
If you want to test the component (not the directive), you might do: $compile('<color-showroom color="foo"></color-showroom>)($rootScope)`, and no need for compute bindings or other stuff.

I think the current behavior is better. The docs should be fixed though.
The bindings argument or $componentController, should be passed the function that gets assigned on the controller. All the parsing (and other safeguards we have in place) that happens as part of parsing a directive's bindings are out of scope here.
When testing the controller, you should be only concerned with the part of the API that interacts with it, in this case, your controller only knows it gets an onDelete function and calls it with {hero: ...}.
That is what you need to test.
Thx for working on this, but I'm going to close it, since the current behavior is the intended one.
It's a good spot, though, and you are more than welcome to submit a new request with the docs fix.
:+1:


What version is in Swift 2.3?

I am having trouble finding what Version supports swift 2.3?
 Could you include this in the docs for future reference?
Thanks!

Hi, @uknowmeright, I am sorry, there is no Swift 2.3 version of the library, unfortunately.

Closing the issue. Feel free to reopen if needed.


Add ability to override/disable response file use in ArgsResolver

This change deprecates ArgsResolver API that takes a boolean forceResponseFiles in favor of a new API that takes an ResponseFileHandling enum. This enum allows the previous behaviors of .forced and .heuristic (true and false, respectively of forceResponseFiles), and adds an option to override disable use of response files completely (.disabled).
This is useful in the example of dependency scanning when it is done via a C interface to a compiler library: even for exceedingly-long command-line argument lists, using response files is not necessary since we just pass in an address of an already-allocated memory buffer containing the argument list, directly.
Resolves rdar://99296444

@swift-ci test


How to move any window to bottom layer?

I am trying to get something like: https://imgur.com/a/8Mq4A
Actually, I want Cava to run on any window transparent and move it to the bottom layer so it won't affect other windows.
Thanks.

you can't

So, there is no chance to get what I want?

it should be the client's job tbh - https://github.com/kovidgoyal/kitty/pull/2590
you can write your own I guess

I'm sorry but, I can't.

https://github.com/hyprwm/hyprland-plugins/tree/main/hyprwinwrap should handle this
@vaxerski

https://github.com/hyprwm/hyprland-plugins/tree/main/hyprwinwrap should handle this @vaxerski

Thanks @MightyPlaza  will try and let you know the review.


Repository concurrency

Current implementation can only support one repository concurrently but the code is mostly network bound and not CPU or IO.
Main obstacle is globals in the API interfaces. We either need to return a new instance if each (a little inefficient but not a big deal if we're only processing a few concurrently) or push all state back into repo config object.
Also need to make sure we don't leak token to any logs.

Waiting on #633


Fixed blood detection and added interpretation result, but haven't sh…

Fixed blood detection and added interpretation result, but haven't showed results on UI yet

@hungngo97 If you can update the UI on this branch. Then I can merge it to the master.

#54 Fixed


Connection by ssh

Dear Sir,
Can you give me a way to remote Mikrotik router by SSH ?
Thank you

I recommend you check out and ask your question here: http://forum.mikrotik.com/

Thank you, but I mean to connect by your API.

I'm not sure what you are asking. If are asking how to use the API to setup SSH, I don't know how. You need to find the RouterOS command line commands and translate them to API syntax. If you're asking to use the API over SSH, it doesn't work that way, it uses the RouterOS API protocol.
The README should be enough to get you going.


Do you need username/ user-password if you are only searching  by tags?

I looked over the README and examples contained within, but it seems like if you are only scraping tags, you don't need to enter username/user-password?
However, if I 'm attempting to scrap tags only, I'm being prompted for an Instagram username password.
I don't have an account, so I'm wondering if I will have to create one.
Here is my sample command:
instagram-scraper salmon --tag -d= path/to/my/directory -t=image -m 500
This was working fine before the update.

This commit
https://github.com/rarcega/instagram-scraper/commit/a96e72b49233ea010b7637784beba1d1391f61bf
made them mandatory.


Implementation logos are not displayed correctly on mobile

In #21 I've introduced a list of current and upcoming implementations next to Rudr.
Unfortunately it looks like I broke the section as logos are not displayed correctly.

I took the logos out for the narrow view for now.


[Purview] Purview GA Azure CLI request

Onboard Azure CLI for Microsoft.Purview Resource Provider to preform CRUD operations on resources.https://github.com/Azure/azure-cli/issues/19186
closed https://github.com/Azure/azure-cli/issues/19186
This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

closed https://github.com/Azure/azure-cli/issues/19186

closed [https://github.com/Azure/azure-cli/issues/19186]

Purview


Deepface.Represent

When I put the argument force_detection = false what the function will return?

Follow the read me, this is described in there.


[BUG] : nav elements gets distorted on squeezing the screen size

on a smaller screen size the nav elements moves towards bottom rather staying at the center.

Please add the GSSoC label and assign me this issue, a simple change in CSS in media queries section of nav will sort this out. Thank You!

Go for it @Sudhanshu2306


{AKS} Preview SDK based on API version 2024-01-01

Related command
Description

{AKS} Preview SDK based on API version 2024-01-01
Swagger PR #27676

one feature web app routing introduced

Python SDK release request #4966

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

AKS


Prevented back arrow into splash

Moved the BackHandler event listener into componentDidMount and added a catch specific to the home screen that causes the app to exit.

#87


data: add Profile

Hey there! Exciting news! I've created an open source project called "devFind" where you can add your profile and others can search by skills. Join in and contribute to make it better!
You can start with adding your profile. Check the Readme file if you are a beginner.

ok, I will add my Profile and another too, Assign me this under hacktoberfest.

please add me to this Project @shyamtawli

@Utkarsh750 @bhalerao-2002 You can start by adding your profile.

I'll also add my profile

please add me to this project


first attempt to add InsertText intent

updated:

alexa.py (main logic)
./speech_assets/SampleUtterances.xx.txt (Intent)
./speech_assets/IntentSchema.json (Intent)
utterances.xx.txt (Intent)
templates.xx.yaml (answer card)

still don't know if these modifications are enough to generate the InsertText intent from 'lexigram deploy kanzi' command

Looks good, but there is a conflict on one of your files.

Looks good, but there is a conflict on one of your files.

I was behind your latest merged PR cause you accepted it before this. Should do fine now, thank you!


Wrong type_name in EmptySubscription

https://github.com/async-graphql/async-graphql/blob/9086ca7ef0bf89723bbfe6d054e7f086b0b1cbf0/src/types/empty_subscription.rs#L16
Got a panic because I used the EmptyMutation and EmptySubscription at the same time, is it intentional that EmptySubscription has "EmptyMutation" as type_name?

Could you make a PR, please? The panic message as well as the minimal reproducible example would be helpful too.


[$250] QBD - Admin is not selected as default Preferred exporter after connecting to QBD

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.58-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: N/A
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/5184323
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause - Internal Team
Action Performed:
Precondition: Have a workspace connected to QBD

Navigate to Workspace > Accounting
Click on Export
Click Preferred exporter

Expected Result:
Admin is selected as default Preferred exporter after connecting to QBD
Actual Result:
Admin(owner) is not selected as default Preferred exporter after connecting to QBD
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Standalone
[ ] Android: HybridApp
[ ] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/user-attachments/assets/1084f55d-46de-4aa8-a17e-68c5c4d687c7
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021854584654634877527
Upwork Job ID:<PHONE_NUMBER>634877527
Last Price Increase: 2024-11-07

Issue OwnerCurrent Issue Owner: @hoangzinh

@hoangzinh I don't think there is a problem at all. And I do agree that if there ever was, we should instead fix it in the backend and not the frontend.
QBD integration backend logic hasn't changed regarding the preferred exporter and it works fine on OldDot.
cc @francoisl for your input.

Not overdue, see my comment above.

@hoangzinh thanks for testing it out in OldDot. I had trouble connecting to QBD this morning (Web connector screen stuck).
Let's do @ZhenjaHorbach's solution then?
It wouldn't be eligible for payment though, as @ZhenjaHorbach you were the one in charge of handling the feature in https://github.com/Expensify/App/pull/50297 and it could be considered as a regression from OldDot.


68 add file importexport for workspaces

I think the buttons need to be reworked. Maybe make them single column stack. Or perhaps a single combo box for load and delete, styled kinda like the layout combo box above. Delete should prompt for confirmation in that case.


Add colours to loan and bike lists

The list of loans in the UI is now yellow, and the list of bikes (accessible via listbikes) is blue.
This has been done to differentiate the list of loans from the list of bikes, so the user can more easily notice which list they are looking at.

Pull Request Test Coverage Report for Build 547

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 91.713%

Totals

Change from base Build 526:
0.0%

Covered Lines:
1981

Relevant Lines:
2160

💛  - Coveralls


BLE-MESH Vendor Callbacks (IDFGH-4565)

Is your feature request related to a problem? Please describe.
Hello,
I am currently developing an application based on Bluetooth Mesh and i have a few recommendations i would like you to hear to simplify development and programming with regards to Vendor Models.
Describe the solution you'd like
Currently, SIG models init and deinit their correspondent model and user_data  by means of stack initialized callbacks.
typedef struct {
    /** Callback used during model initialization. Initialized by the stack. */
    esp_ble_mesh_cb_t init_cb;

    /** Callback used during model deinitialization. Initialized by the stack. */
    esp_ble_mesh_cb_t deinit_cb;
} esp_ble_mesh_model_cbs_t;

However, Vendor Models are not allowed (or do not use) to access this callbacks. Therefore, Vendor Models user_data must be deinit manually which may lead to Memory leaks. I'd propose to create some vendor model only API to register these callbacks into the Stack. This way, the deinit of the Bluetooth Mesh Stack will also help freeing user allocated memory.
In addition, i would also like to have separated messages callbacks for different vendor models. Currently, all Vendor Messages arrive to the same callback where messages should be filtered and linked to their correspondent response function. I'd propose an API call similar to the one below. This way, code understanding gets reduced.
//Propose API function call
esp_ble_mesh_register_custom_model_callback(esp_ble_mesh_model_t *vendor_model, void (*msg_cb_vendor_function)(esp_ble_mesh *param, ...));

Additional context
If there already exist features similar to the ones i explained, i would like to have the latest patch you have. It would be really interesting for my project.
Thanks!

Thanks for raising this feature request.

Thanks for raising this feature request.

Hi @MrCurio
Thanks for this.
Could this be achieved by initializing the model->cb when defining a vendor model? Then no need to add a new API for this.
Currently ESP_BLE_MESH_VENDOR_MODEL is used to define a vendor model, maybe we could add a new macro, e.g. ESP_BLE_MESH_VENDOR_MODEL_CB for defining a vendor model with the corresponding callbacks registered. And users could choose to use either of them to define a vendor model based on their requirements.
Thanks.

Hi @MrCurio
Thanks for this.
Could this be achieved by initializing the model->cb when defining a vendor model? Then no need to add a new API for this.
Currently ESP_BLE_MESH_VENDOR_MODEL is used to define a vendor model, maybe we could add a new macro, e.g. ESP_BLE_MESH_VENDOR_MODEL_CB for defining a vendor model with the corresponding callbacks registered. And users could choose to use either of them to define a vendor model based on their requirements.
Thanks.

Hi @Campou
That would be great. I wish i could help developing those features, bit I'm afraid i don't have the time.
If anyone plans to develop it, please let me know with a comment here.
Thanks!

Hi @Campou
That would be great. I wish i could help developing those features, bit I'm afraid i don't have the time.
If anyone plans to develop it, please let me know with a comment here.
Thanks!

Hi @MrCurio
That's fine, thanks for the idea!
I will mark this as a feature in the next release version, maybe for idf 4.4, and I will try to implement this before the release.
Thanks.

Hi @MrCurio
That's fine, thanks for the idea!
I will mark this as a feature in the next release version, maybe for idf 4.4, and I will try to implement this before the release.
Thanks.