How to add multiple analyzer ,or run multiple analyzer docker container?

How to add multiple analyzer ,or run multiple analyzer docker container?

Engine scales out by running more analyzer containers. It depends on how you are running the system, but both Docker compose and Helm/K8s have ways of scaling a service to include more instances.
See: https://docs.docker.com/compose/reference/up/ for Docker Compose
or our helm chart's values file for examples: https://github.com/anchore/anchore-charts/blob/master/stable/anchore-engine/values.yaml#L273


Balance doesn't update after new contribution until refresh

Everything with the actual transaction was fine, but the budget balance did not reflect the contribution until I refreshed the page.
The path I followed was to make the contribution, then click the Collective name to go back to its page afterward from the success screen. I think people doing this are probably looking to see the impact of their contribution on the balance sometimes (I was, because I was making sure there was enough balance to pay an expense).
Can it automatically refresh the balance when going back to the Collective?
In this example, the $700 transaction is the one not showing in the total balance of $323.80.

This is still happening. I refreshed a few times but the balance is not updating.


Mithril Block clients update

On the page http://mithril.dev.ehealth.world/clients/id/block add current comment and if comment wasn't changed while block/unblock update with empty string (same logic as for users)

current comment is not shown. there is the same field for current comment and for editing

when the comment wasn't change next time will be shown the same comment, not updated to empty


Why I see that error?

💬 Questions and Help
When I try to export function with test, then I catch an error -
  ● Test suite failed to run

    Your test suite must contain at least one test.

Without function all is ok -

But how can I decompose large test in small tests and combine it?

By default, Jest finds every file in __tests__ folder or with suffix .spec.js/.test.js. Jest will try to run tests in these files. It expects at least one test result from each file.
In your case, test it("should render") is never run. Jest cannot see it because it's inside testTitle function, which is never invoked. Jest gets no results from this file so that's why it throws an error .
If you want to combine tests together from different files, you have to make sure that small test files are not run by Jest. You can check the documentation on how to do this.
Then import that function and invoke it.

largeTestFile.js

import { testTitle } from './testTitleFile'

testTitle()

But I would recommend that you don't do this. Create a file SearchTitle.spec.js and keep all the test related to SearchTitle there. You don't have to import this file anywhere. Jest will run it, because it matches the default pattern which I mentioned above.
If one of your test files is getting big, it's alright as long as tests inside are related. use describe blocks to separate tests, you can nest them too.

@grosto thanks for suggestion =)
In my case I solve problem with require(./path-to-test)
In general, this is only for the reporter.


fix: products sorting select styles in prod

CSS order can differ between development and production. This is what was happening with custom select styles. To fix this, I increased their specificity. I looked through all the pages and components - this fix is not needed anywhere else.

I'm still thinking how to do this differently.
Alternatives so far:

Import select.module.scss in the root route.
Explicitly "compose" with the base class name .select.productSortingSelect {}, like in Stylable.
Define two variants each having full styling: "regular" and "menu", so we won't have to add both to the same select. This doesn't fully solve the issue: e.g. if the variant sets the width: 100% by default, then a specific route will have problem overriding it.


patch: stack passing

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


IndexingService not indexing SearchParameters referring to arrays of type Resource

Chain search will not give results.
When searching for a Bundle that is of the type document and has a composition with that contains a string it will not give any items even though there are plenty of bundles that should meet the requirement.
Search:
<BaseUrl>/Bundle?type=document&composition.title:contains=<SubString>
<BaseUrl>/Bundle?composition.title:contains=<SubString>

@Crownie88 I've updated the title in accordance with the findings on this one

@kennethmyhra Perfect, thanks. I tried looking for these arrays since I thought this was the case but couldnt find the source of the search parameters.


Add kyonggi university

Please review, and if everything looks good, merge please
Thank you :)

@Happy-Lotus I am sorry, the domain kyonggi.ac.kr is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Fix documentation bugs

Description
I found some more docs that use the old non-vectorized SFML 2 APIs plus a few instances of incorrect (British) spellings.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall first build on docs_fixes at 55.726%

Totals

Change from base Build<PHONE_NUMBER>:
55.7%

Covered Lines:
11565

Relevant Lines:
19675

💛  - Coveralls


Bug:  Failed to compile CSS/SCSS When curly braces is present on css class as dynamic value for mixin

Terms

[X] I have read the guidelines for Contributing to Roots Projects
[X] This request is not a duplicate of an existing issue
[X] I have read the docs and followed them (if applicable)
[X] I have seached the Roots Discourse for answers and followed them (if applicable)
[X] This is not a personal support request that should be posted on the Roots Discourse community

Description
Currently, we have a mixin function that generates content based on how the user uses it.
@mixin headings($from: 1, $to: 6) {   @for $i from $from through $to {     h#{ $i } {       @content;     }   } }
If you notice the h#{ $i }, the curly braces return the error below

Below are the dependencies that we have
  "devDependencies": {     "@roots/bud": "^6.9.0",     "@roots/bud-eslint": "^6.9.0",     "@roots/bud-prettier": "^6.9.0",     "@roots/bud-sass": "^6.9.0",     "@roots/bud-stylelint": "^6.9.0",     "@roots/eslint-config": "^6.9.0",     "@roots/bud-entrypoints": "6.9.0",     "@roots/bud-postcss": "6.9.0",     "@roots/sage": "6.9.0"   },
Steps To Reproduce

Install Sage Theme
Install depencies below:
  "devDependencies": {  "@roots/bud": "^6.9.0",  "@roots/bud-eslint": "^6.9.0",  "@roots/bud-prettier": "^6.9.0",  "@roots/bud-sass": "^6.9.0",  "@roots/bud-stylelint": "^6.9.0",  "@roots/eslint-config": "^6.9.0",  "@roots/bud-entrypoints": "6.9.0",  "@roots/bud-postcss": "6.9.0",  "@roots/sage": "6.9.0" },
Add a mixin that contains curly braces on css Class.
ex:  @mixin headings($from: 1, $to: 6) { @for $i from $from through $to {  h#{ $i } {    @content;  } } }
Yarn Build

Expected Behavior
Since this is a SCSS function, this should be working perfectly

Actual Behavior

but the issue render this error
Relevant Log Output
No response
Versions
Sage 10

Let's use your existing support topic on Roots Discourse for the back-and-forth on this — Sage doesn't include Sass out of the box, so this is either an issue with our docs or upstream in bud


Google controlled history not reduced to 10 days

Where to find the issue

https://www.coronawarn.app/en/faq/results/#keys_matches "What do the exposure check logs show?"

Describe the issue
Under the Android operating system, Exposure Notification Random IDs are stored and managed by Google. They are kept for 14 days and purged by a daily job run from Google Play services. The length of time these IDs are kept by Google is not under the control of the Corona-Warn-App (CWA) and does not change depending on the version of CWA in use.

https://www.coronawarn.app/en/faq/results/#keys_matches "What do the exposure check logs show?" has been changed to read:
"Exposure checks
This is where the checks of up to 15 days are logged (today plus the last 14 (10 days since version 2.20)).
This is incorrect as far as Android is concerned. Google clearly states in their UI, that Random IDs are deleted after 14 days (not 10 days).
Suggested change
Please check if the text is also incorrect for iOS, in which case the suggestion is:

remove the added text "(10 days since version 2.20)" from https://www.coronawarn.app/en/faq/results/#keys_matches "What do the exposure check logs show?"

remove the added text "(10 Tage ab Version 2.20)" from https://www.coronawarn.app/de/faq/results/#keys_matches "Was bedeutet die Überprüfung auf mögliche Begegnungen?"

PR #2680 submitted to resolve this issue for Android and for iOS.


Cannot get link with video fullhd

Video fullhd return 403 when result link access. SignatureDecoder not work as expected
ex: https://www.youtube.com/watch?v=NUsoVlDFqZg

I am reading function decode in SignatureDecoder class. Can you explain about this function?

Actually This project is originally forked from https://github.com/Athlon1600/youtube-downloader
I have only done the frontend UI changes and more easily accessible API of the download links. It would be better to post this issue in his repo. https://github.com/Athlon1600/youtube-downloader/issues
Thanks


tile - create correctly the next_tile_fifo_arb_id acording to the current tile! (not the next tile)

Use it in the mini_core_tile
The current override in the router is always looking at the next_tile_id to correctly define whech fifo_arb needs to allocated.
In the transaction from mini_core->router we need to figure out who is the correct fifo_arb to allocated according to the current tile_id (not next tile_id)

we can set this aside currently. Amichai, you said you will handle it @amichai-bd

is this done @yeonatanPerelman ?


Add user photo, not working yet

This is very much not working yet. I can't figure out at all how to actually handle receiving the file from Ember. We expect a base64 encoded image right now. I cannot seem to make this work. It looks like Plug automatically parses for files and handles those directly.
Possible that we need to rewrite the Ember side to handle the image uploads as form data.

I got this to a point where, when running a test, arc tries to connect and then times out. If I export environment variables we use in code-corps-api, then I get
test/models/user_test.exs:101
     ** (EXIT from #PID<0.497.0>) an exception was raised:
         ** (FunctionClauseError) no function clause matching in String.contains?/2
             (elixir) lib/string.ex:1712: String.contains?({:system, "S3_BUCKET"}, ".")
             (ex_aws) lib/ex_aws/s3/request.ex:40: ExAws.S3.Request.host_and_bucket/2
             (ex_aws) lib/ex_aws/s3/request.ex:26: ExAws.S3.Request.url/3
             (ex_aws) lib/ex_aws/s3/request.ex:12: ExAws.S3.Request.request/5
             lib/arc/storage/s3.ex:14: Arc.Storage.S3.put/3
             (elixir) lib/task/supervised.ex:94: Task.Supervised.do_apply/2
             (elixir) lib/task/supervised.ex:45: Task.Supervised.reply/5
             (stdlib) proc_lib.erl:240: :proc_lib.init_p_do_apply/3

I guess the bucket URL is wrong, or the instructions for arc are bad. If I replace the bucket config line with a direct value instead of a Key, I get a different error
test/models/user_test.exs:101
     ** (MatchError) no match of right hand side value: {:error, [{:http_error, 403, "<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n<Error>InvalidAccessKeyId
....

At this point, my guess is, it should work, and what's missing is proper configuration on my part.

I can't quite figure out why this build is failing even after adding in the env vars to CI.

Looks good other than the leftover inspects. :+1:


Move System Configs tool to separate repo

An idea worth a discussion
Move the System Configs portion to a separate repo. Potential advantages:

Cleaner split between qcr tool (entry point tool to multiple things) and system configs tool
Could make system configs tool generic, usable by anyone (currently restricted to QCR due to token embedded in scripts)
Allow documentation about the system_configs tool to be alongside it without creating confusion between the qcr tool and system configs

I see the system configs tool as a standalone thing that could be beneficial to the wider community. But to do that, it needs to become a standalone thing.

@btalb @suddrey-qut @RSLamont I know you guys are no longer with QCR or on leave, so feel free to ignore. However, as primary developers, would love to get your thoughts on this

For memory, we did the first step (splitting the scripts from the actual per-robot/machine config repos).
A next logical step would be to split it to a standalone tool without the QCR specifics. Got to consider engineering burden though. The goal of this repo was to bring together "odds & ends" scripts into one place for it all to be maintained. As things spread out further, there may be more maintenance burden / plumbing together / documenting "how it all fits together".
If a motivation is use outside QCR, just remember the feature set here is relatively basic. There are plenty of tools already around in this space that offer more features but come with all the costs / challenges associated with 3rd party dependencies.

The first part we have done. The systems config tool is a separate repo, and it creates a separate repo for each system/machine. This discussion is more about simply moving scripts/systems_config to its own repo, and then doing something to how the services tool operates.
I thought the aim of this repo was to act as the entry point to various QCR tools. So QCR users only have one command to remember and can quickly see the various tools without having to search through repos or documentation. Obviously may have to go to a repo or documentation to see the purpose and how to use each tool.
Agree that there are other tools out there that might be a suitable replacement for the systems config tool. However, a small feature set is not necessarily a bad thing. Typically means easy for a newbie to understand possibilities and get started. Additionally, removing the encrypted qcrbot token and making it more generic, means students/staff could use it to track their own machines without having to have the config stored in the qcr-configs or qcr organisation.

Moved to https://github.com/qcr/system-configs


ci: remove publish-website action

Closes #192

Codecov Report
Base: 98.06% // Head: 98.06% // No change to project coverage :thumbsup:

Coverage data is based on head (7b0f465) compared to base (d519734).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #193   +/-   ##
=======================================
  Coverage   98.06%   98.06%
=======================================
  Files          20       20
  Lines         207      207
  Branches       43       43
=======================================
  Hits          203      203
  Misses          4        4

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.


data division only works if len(trajectory)%n_parts==0

sites = SitesData(
    structure=s,
    trajectory=trajectory,
    floating_specie='Li',n_parts=10
)

Executing the code above throws an error.
This is resolved by instead running:
sites = SitesData(
    structure=s,
    trajectory=trajectory,
    floating_specie='Li',n_parts=6
)

Note that len(trajectory) == 12378. The error is resolved because 12378%6 == 0.
12378%10 != 0 that's why the first code snippet throws an error
This took a while to figure out though and is not robust. This error should be anticipated. Often the MD step number is not divisible by 10
One idea to solve this is clipping some of the final frames from the trajectory so that len(trajectory)%10==0. That would at most mean the loss of 0.18 ps of simulation, that is fine considering ~100 ps are run typically. However, when such clipping occurs a warning message should be displayed saying something like"In the course of the data analysis your simulation time was shortened by 0.xx ps"

Another option is to have uneven parts, e.g. for 38 with n_parts=4: 10, 10, 9, 9


Lab 2 Markdown Support

I want to add markdown support to your project I will add the two features of Headings and Italics. To do this I will create two new functions that mirror ones you have already made but add markdown support so that when They need to be used instead of text documents they will

Implemented with commit c17aa1db371d8f15d85fec7ae86b0743df0464d7

Issue reopened to add comments to the code and improve the document.

Feature implemented issue closed.


401 - authentication error

Installations worked like a charm. But the plugin fails to retrieve tado information. Seems to be an authentication error. Checked my credentials twice - they are correct. Might be a problem with special characters: I use a "&"-sign in my tado-password. Had similar issues with other plugins.
[2018-3-26 12:41:23] [Thermostat] Getting HomeID...
[2018-3-26 12:41:23] [Thermostat] HomeID Error: Error: Failed to load data, status code: 401

btw: Where can i find HomeID manually?

Set homeID manually will not fix the problem because i.e. requesting the state will also need credentials
I will test transforming the credentials and try to fix that in the next version

v3.1.7 is up
Should be Fixed, can you try it pls

Perfect. Authentication succeeded. But probably found another issue. Will open a different issue for it.

Ok, im closing this issue then


Socket extension legacy methods

Addressing https://github.com/SidharthNabar/corefx/commit/673e137418234e5279f693937ce3066204cc810f#diff-a92dcc50fa7799a1b77b747a99c6222b
"Trying out the pattern to separate out some legacy methods of Socket as
extension methods. Generating PR to get feedback on coding pattern and
naming."

@CIPop @pgavlin @SidharthNabar @davidsh - PTAL. If the pattern looks good we'll do the same for the other legacy methods in sockets.

Only one nit comment regarding changing files more than they need to.
Otherwise, this looks good. Please continue towards a full implementation of the legacy contract and proper pruning of the Sockets <IP_ADDRESS> contract.
Current PR should not be merged as is.

Closing to prevent accidental merge.


Allow exchange package to re-use backend modules

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.5 out of 6 committers have signed the CLA.:white_check_mark: renesy:white_check_mark: Kuzirashi:white_check_mark: kosecki123:white_check_mark: JosephBagaric:white_check_mark: chonla:x: renovate-botYou have signed the CLA already but the status is still pending? Let us recheck it.


Implement atan2

This addresses #6095.
Since atan2 is not implemented in Eigen, this is loosely based on the rint implementation. Suggestions are welcome.

That will require and API review.

I've addressed your code comments (although I won't get a chance to test the GPU code myself until tomorrow).

Jenkins, test this please.

Could we add a test that checks random values, and also all special values listed here?
https://docs.scipy.org/doc/numpy/reference/generated/numpy.arctan2.html
Thanks

Done. Gradients for random arguments are rather flaky, unless I clip the inputs to be well away from zero. Perhaps this is because the gradients blow up near the origin. Curiously, I have to clip them more severely for float64 inputs, due to the stricter tolerance on the check.

Jenkins, test this please.

Apparently math.inf doesn't exist in python 2. I've fixed that, but it doesn't explain the python 3 failure. How can I see the detailed log of the failure? I can't find it on the CI web interface.

FWIW, running bazel test --config=opt //tensorflow/python/kernel_tests:cwise_ops_test passes now (with my previous commit) for both python 2 and 3 on my local machine. Perhaps you could test it again?

You'll need to add an entry here to extend the public API.
Jenkins, test this please.

Done.

Jenkins, test this please.

@martinwicke this is an API change, WDYT?

This is good from an API perspective after reverting to the (y, x) signature.

Done.

Jenkins, test this please.

Jenkins, test this please.

Fetch errors.
Jenkins, test this please.


add some filters by year, country, etc..

Now in UI https://greybax.github.com/IT-films we have only 3 categories:

movies
documentary
tv-series

Also would be nice to have some specific filters for each category such as year and country. It could be done because we already have this data for each movie.

Hi @greybax can you share some description about the issue.

@mShreyasKumar updated description to this issue


David/feature/ping2.0

I refactored the ping.js and user.js code to v14. Simple changes.

just start over man


Adding Indonesia Language

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR type

[ ] Bugfix
[x] Feature
[ ] Refactoring

What's new?

[ ] PR closes #<issue_number>
[ ] <brief_description_of_what_I've_added>

Before submitting Pull Request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality
[x] I've posted a screenshot of demonstrated change in this PR

Optional

[x] I've tested it at </rmrk/collection/26902bc2f7c20c546a-1FVG7>
[x] I've tested PR on mobile and everything seems works
[x] I found edge cases
[x] I've written some unit tests 🧪

Had issue bounty label?

[x] Fill up your KSM address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot

[x] My fix has changed something on UI; a screenshot is best to understand changes for others.

add payout address and let me know when it's ready to merge. Best if someone from Indonesia could review this :)

all goes well
https://beta.kodadot.xyz/transfer/?target=GF4v68qJmh47eShYNpPs5JNfn5bxG5Kz1U2Y4xC2xAqXqmq

tambahkan alamat pembayaran dan beri tahu saya jika sudah siap untuk digabungkan. Lebih baik jika seseorang dari Indonesia dapat mengulas ini :)

https://beta.kodadot.xyz/transfer/?target=GF4v68qJmh47eShYNpPs5JNfn5bxG5Kz1U2Y4xC2xAqXqmq
do you have any suggestion someone to review this sir ❓

you need to put address into PR template at payout :)

Anda harus memasukkan alamat ke dalam template PR saat pembayaran :)

already edited~ :heavy_dollar_sign:

pay 100 usd

😍 Perfect, I’ve sent the payout
💵 $100 @ 138.53 USD/KSM ~ 0.722 $KSM
🧗 GF4v68qJmh47eShYNpPs5JNfn5bxG5Kz1U2Y4xC2xAqXqmq
🔗 0x9d2eeb9bd4f32b95bdefb9bff7f84025eb46473f3844c6107cb02613d85570a5
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues

@magrisya if you have twitter https://twitter.com/KodaDot/status/1498972701016395778 ;)

hello, Indonesian here. I think You did a really good job with the word choice mate! But a really tiny improvement though, I think 'mengubah' should be 'ubah', judging from the word used in our english site :)
Thank You!


Where do the version numbers come from or get synced from?

When sub-packages reference another one, I'm currently using * but dunno if that's kept in sync somehow?
(My project currently has no tags)

Use version ranges (such as *, ^2.0.0, 3.x, ~1.2.3) just as you would with external dependencies, but consider that:

Those ranges will be honored by npm, yarn, etc. when your users install your published sub-packages.

They will not be honored by oao in your monorepo installation, since the internal dependencies will be yarn linked rather than installed.


rails g foundation:install command doesn't produce full list of components in foundation_and_overrides file

So as in subject....
In foundation_and_overrides.scss after installing foundation in rails there should be all components uncommented.
But I found one missing @include foundation-menu-icon;
This took me couple hours to find out why the hamburger menu doesn't show up.
I didn't suspect this place to be affected as the comment says:
// We include everything by default.
Would you please check the generator or whatever to produce the real full list of components in the final settings file.
Thank you in advance!

+1
@include foundation-flex-classes; is also missing

👍 Please fix the .scss file with the proper includes and the proper order.


Error occurred updating bot with new SL/TP values: Other error occurred: record_invalid Invalid parameters {'trailing_enabled': ['is not included in the list']}.

Describe the bug
I'm getting the following error from time to time (not always) with the trailingstoploss_tp.py script:
"3CQSBOT_MULTI_BANSHEE": BUSD_ATOM/1331030122 profit (3.02%) above activation (2.5%). StopLoss activated on 2.52%.
TakeProfit increased from 5.0% to 5.26%
Error occurred updating bot with new SL/TP values: Other error occurred: record_invalid Invalid parameters {'trailing_enabled': ['is not included in the list']}.
To Reproduce
not sure, it seems random... most of the time it works fine, sometimes I get this error instead and no SL is set
Environment:

Python version: python3 -V:
Python 3.8.10
Operating System: uname -a
Linux ubuntu-3commas-bots 5.4.0-97-generic #110-Ubuntu SMP Thu Jan 13 18:22:13 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
Version of software [e.g. 0.6.1]:
0.9.0

Additional context
Add any other context about the problem here.

Can you see with which update you get this, like Stoploss change or TP change?

Yes, TP change.
The weird thing is I ran the script for several days with only one bot, no problems.
Yesterday I added a second bot (by adding a second bot ID to the array), and with the first attempt to update a deal from the second bot it happened.
Also one update did work fine on that same bot for some reason a little later.
I then created a complete new section in the ini file for that second bot, and it still went wrong 2 more times.
I now have the script running with only the second bot (removed the first one) and it's been running fine since then.
So either that was a complete coincidence and 3commas was having problems only while I had 2 bots in the config, or it only happens when there are more than one bot configured...
Sorry I can't be more help, I've found no good way to consistently reproduce it.

I could not reproduce it either, I will close for now, please (re)open if needed!


Create, upgrade and manage Player Fortress

Name
Location on Map (See it on the Map page?)
Workers
Upgrades
Combat Scores
Siege Mode?

A proof of concept was made in #273 but with the Lands Update will need to be prioritized over this

No time


Allow searching by value against the RID

The CTS currently runs a battery of tests for findEntitiesByPropertyValue based on an exact match of the identifier field of the new OpenMetadataRoot entity type, which at the moment are all failing.
This would be a search by RID in IGC (something we should be able to implement for most scenarios)...
Example output:
findEntitiesByPropertyValue found 0/1 expected results using parameters: {asOfTime=null, entityTypeGUID=5a2f38dc-d69d-4a6f-ad26-ac86f118fa35, fromEntityElement=0, limitResultsByClassification=null, limitResultsByStatus=null, pageSize=5, searchCriteria=\\Qb1c497ce.8a5be154.001mts4th.n7gtdul.bkhqd3.ulsfcvqfk8dhq09ajld3a\\E, sequencingOrder=null, sequencingProperty=null}"

Fix is probably simply to add the _id field as one of the string-based properties to include in the search when doing findEntitiesByPropertyValue (?)

So we can implement this for an exact match on ID, but for any other kind of match (contains, starts with, ends with) this will not be possible -- IGC does not support searching based on partial IDs.
Wondering if this is even a use case that we should expect to work in general (suspect most repositories will have trouble searching for a primary key based on a partial match) (?)

Discussed on call on 19/11 and determined that these properties on OpenMetadataRoot were a copy / paste error, and will be removed -- so no need to cover these searches or this issue anymore.


Area Scans change URL

Source information
Area Scans 1.4.31
Source language
Arabic
Source new URL
https://ar.areamanga.xyz/
Other details
No response
Acknowledgements

[X] I have updated all installed extensions.
[X] I have opened WebView and checked that the source URL has not changed yet.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I will fill out all of the requested information in this form.

@a7mad115 old url still works? close this issue please

If you want a new site added you should use the 'source request' issue template.


Create LICENSE

The posthog.com contents is under the following license:
# For content in the /contents/ folder

Copyright (c) 2020

Permission is hereby granted, free of charge, to any person obtaining a copy
of this software and associated documentation files (the "Software"), to deal
in the Software without restriction, including without limitation the rights
to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
copies of the Software, and to permit persons to whom the Software is
furnished to do so, subject to the following conditions:

The above copyright notice and this permission notice shall be included in all
copies or substantial portions of the Software.

 IN NO EVENT SHALL THE
AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
SOFTWARE.

Should we consider some form of Creative Commons license?

Happy to go with MIT.
Fun fact: The reason posthog.com isn't MIT, is that we had people (specifically, other open core companies, with similar investors and potentially similar customers)  fork our repo to create their own startup (fair play though!), and an important part of our web design is to differentiate us.

Done, given we're on a bias for action streak here... ;)

The reason posthog.com isn't MIT, is that we had people (specifically, other open core companies, with similar investors and potentially similar customers) fork our repo to create their own startup (fair play though!)

That's great! How about CC BY 4.0 to give the freedom of use but there's also some attribution.


how to use with multiple pages?

I would like to use the template for multi page project.
I am unclear as to how can I have multiple pages.
looking at the demo, I can see each page redefined the head with all the scripts. same goes for the menu.
do I need to use some kind of a generator? or copy paste manually? I am unclear as to how this template supports multiple pages and would appreciate it if you elaborated on the topic.

You would need to use a JavaScript front end framework.


Adjust Markdown output, so it works well with Slate

lord/slate is one of the more popular static site generators for Markdown that is optimized for API documentation. I have adjusted the Markdown output in following ways:

use code formatting instead of bold for most keywords and primitive types
remove block quote output, because slate reserves this for examples
move all headlines one or two levels up, as slate generates a TOC from H1 and H2
corrected line breaks after headlines and lists
removed lots of horizontal rules, as they can be styled better with CSS
changed spacing around punctuation from French style (spaces before and after) to English style (only space after)
added ability to use .html in links and .html.md for generated output (I'm planning to make this configurable)
include short description of the class in package listing.

Please let me know when you reach some stability point, so i can push a new version into Maven central :)

Will do!

Quick heads-up: I won't get anything done this week, so if that's enough stability for you, feel free to push.


[RZ_A1H]Fix some bugs about InterruptIn, SPI, I2C and modify some settings.

Hi,
We fixed some bugs and modify some settings as below.
InterruptIn

Add terminal setting of IRQ4 and IRQ6 that leaked.
When set the interrupt function by rise()/fall(), the interrupt disable state will be released by disable_irq().
Interrupt will be continued to occur when execute disable_irq() after rise(NULL)/fall(NULL) set.
Fix the setting timing of PMC register.

SPI
A noise will occur when execute frequency() after set the "mode" to 3 by format().
I2C
Implement a stop condition transmission setting function of I2C.
User can specify the sending of STOP condition by this implement.
scatter file
Update scatter file to increase usable RAM area.
Remove a usable RAM area limit of RW and ZI area.
RTOS
Increase STACK size of thread.
device.h
Take measures about optimization problems of web compiler.
Regards,
Hamanaka

Hi,
I added a comments, why we implement the code.
Please check.
Regards,
Hamanaka

Thanks, I'll merge this.
Why did you place that pragma to the device header file? Did you encounter problem with inlining in multiple files?

Hi,
It does not occur in any other files.
But, we do not want to worry about this problem in any other files.
So, we decided to add this code in device header file.
Regards,
Hamanaka

The device header file is included in many other files, in every target file I would say, therefore it affects almost the entire build :-)


TIMEOUT error

Describe The Bug:
Since 2-3 weeks I have a timeout issue.
I use this plugin since 2/3 years without any issue since.
I can ping the ip address, so it is live.
I can fetch the rtsp stream on VLC
I turn off two factor auth
I try with without username
I try the passwords all the possible way.
I try change version of the plugin
I try putting same password for account and stream
same for username

To Reproduce:
Try to setup the plugin

Expected behavior:
should connected and fetch the device

Logs:
Error during setup of camera Camera salon connect ETIMEDOUT xxx.xxx.xxx.xxx:2020

Plugin Config:
{
    "cameras": [
        {
            "name": "Camera salon",
            "ipAddress": "xxx.xxx.xxx.xxx",
            "username": "",
            "password": "xxxx",
            "streamUser": "xxx",
            "streamPassword": "xxxxxxxxxxx"
        }
    ],
    "_bridge": {
        "username": "xx:xx:xx:xx:xx",
        "port": xxxxx
    },
    "platform": "tapo-camera"
}

Environment:

Plugin Version: v2.3.4
Homebridge Version: v20.11.0
Node.js Version: latest
NPM Version: latest
Operating System: MACOS

Is there any working on this plugin anymore?


etcdmain: fix incomplete proxy config file

etcd might generate incomplete proxy config file after a power failure.
It is because we use ioutil.WriteFile. And iotuile.WriteFile does
not call Sync before closing the file.
Fix https://github.com/coreos/etcd/issues/3861
/cc @heyitsanthony @gyuho

LGTM!

lgtm


feat: add duration to affected releases on sync

The purpose of this feature is to show the duration it took to update or delete each release when a sync, destroy, or apply command is called. Below are some examples:
Old display of affected releases:
UPDATED RELEASES:
NAME                CHART           VERSION
release-one         charts/test       0.1.0
release-two         charts/test       0.1.0

DELETED RELEASES:
NAME
release-three
release-four

FAILED RELEASES:
NAME
release-five
release-six

New display of affected releases:
UPDATED RELEASES:
NAME              CHART           VERSION     DURATION
release-one       charts/test     0.1.0             8s
release-two       charts/test     0.1.0            13s

DELETED RELEASES:
NAME              DURATION
release-three           0s
release-four            0s

FAILED RELEASES:
NAME              CHART           VERSION
release-five      charts/test       0.1.0
release-six       charts/test       0.1.0

/cc @mumoshu I think this is a good idea. I saw the code. The logic is straightforward. and almost all changed files are test files.

/cc @xiaomudk waiting for your review.

I'll review this asap! Thanks a lot for your efforts!

@AlexAxenti please fix DCO~
DCO

@felipecrs need your help.

@felipecrs need your help.

Yeah.. there is not much I can do now unfortunately.
But thanks a lot anyway! Let's wait for @AlexAxenti.

@xiaomudk @yxxhero Hey! I think @AlexAxenti has already signed-off every commit he added and it would be https://github.com/helmfile/helmfile/pull/773/commits/8cae3e5dcf1ec8d576912f526f59958d439ef647 which made DCO check unhappy!
Also, https://github.com/helmfile/helmfile/pull/773/commits/8cae3e5dcf1ec8d576912f526f59958d439ef647 broken tests and other references to former "Duration" which is now "duration" so we'd better check this branch out locally and amend the commit to include other fixes, along with a sign-off.

@AlexAxenti and I can also fix this tomorrow, if you prefer. No worries from my side.

Just fixed the errors caused by the incorrect Duration being referenced as well as signed-off the commits, hopefully should be good now.


how to add rain function

Hello, if I want to add a rain function, how do I do it?

I know this is old, but would advise against a rain or soak function without the implementation of a database.


Create rabiya.in

i am trying to upload  my file but its not getting uploaded

i am trying to upload my file but its not getting uploaded

It looks like you're trying to upload an empty file?


Add a #pragma unroll to block_radix_rank.

Speeds up clang sorts with 32-bit keys by a factor of 1.5-2x.  No change to 64-bit keys, and no change with nvcc.

Thanks, this LGTM. I'll merge it during my next round of integrations.
Also -- if fast 4-byte radix sort performance is important to you, keep an eye on #26. Andy's new implementation is significantly faster than the current version.

cc @Artem-B

keep an eye on #26.

Ooh, thank you!

Testing under internal CL 29046322.


Trouble highlighting first character in line

Issue Type: Bug
When trying to drag-select from the beginning of a line, clicking when the cursor goes from a pointer to a text cursor and dragging does not select anything. In order to start a selection, the cursor must be at least a few pixels into the first character of the line. This can be quite difficult at times. Sometimes I cannot select the first character at all and have tro make the selection right-to-left. Other times it is doable but difficult. No matter what though, a selection cannot begin to the left of the first character (though most editors allow this). And usually not anywhere within the first few pixels of that character.
VS Code version: Code 1.26.1 (493869ee8e8a846b0855873886fc79d480d342de, 2018-08-16T18:38:57.434Z)
OS version: Windows_NT x64 10.0.16299

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6600U CPU @ 2.60GHz (4 x 2808)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
7.41GB (1.47GB free)

Process Argv
C:\Program Files\Microsoft VS Code\Code.exe

Screen Reader
no

VM
0%

Extensions (22)

Extension
Author (truncated)
Version

code-gnu-global
aus
0.2.2

unique-lines
bib
1.0.0

vscode-astyle
chi
0.9.0

bracket-pair-colorizer
Coe
1.0.59

toggle-excluded-files
eam
1.6.0

base-converter
hoo
0.0.1

auto-comment-blocks
kev
1.0.1

vscode-smart-column-indenter
lmc
0.0.13

cppcheck
mat
0.2.2

vscode-devskim
MS-
0.2.2

python
ms-
2018.7.1

cpptools
ms-
0.18.1

sublime-keybindings
ms-
4.0.0

highlight-dodgy-characters
nac
1.0.0

preview-control-characters
nes
0.0.1

vscode-systemd
Neu
0.1.0

material-icon-theme
PKi
3.5.3

psi-header
psi
1.5.0

reloaded-cpp
rel
0.1.6

vscode-coverage-gutters
rya
2.1.1

perforce
sle
3.1.0

rewrap
stk
1.9.1

(1 theme extensions excluded)

What's most bizzare is that the cursor changes from normal select  to text select  a few pixels before clicking has any effect! I have to move the cursor even more for clicking to actually select that position.


Save and Load MABs

WIP

Codecov Report

Merging #185 (d82ddb8) into main (147df93) will decrease coverage by 0.38%.
Report is 5 commits behind head on main.
The diff coverage is 36.36%.

@@            Coverage Diff             @@
##             main     #185      +/-   ##
==========================================
- Coverage   74.10%   73.73%   -0.38%
==========================================
  Files         129      129
  Lines        7678     7706      +28
==========================================
- Hits         5690     5682       -8
- Misses       1988     2024      +36

Files Changed
Coverage Δ

...olem/core/optimisers/genetic/operators/mutation.py
89.21% <ø> (-1.79%)
:arrow_down:

...m/core/optimisers/adaptive/mab_agents/mab_agent.py
36.84% <30.00%> (-3.70%)
:arrow_down:

golem/core/optimisers/optimization_parameters.py
97.22% <100.00%> (+0.16%)
:arrow_up:

... and 12 files with indirect coverage changes


Updated custom aggregate function example and default template of class Aggregate

I'm not sure that the line # template = '...' # write it if you need to change the template is correct

I do not understand why the check failed.


[HOLD for payment 2024-10-22] [$250] Web/Desktop - Chat - Focus is lost after deleting a message from edit mode

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.7-4
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4725448
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause - Internal Team
Action Performed:

Start ND on Desktop App or go to staging.new.expensify.com
Login with any account
Open any chat and send a message
Press Arrow Up key on keyboard
Select All - Ctrl / Cmd + A
Press 'Delete' key
Press 'Enter' key
Confirm message deletion with 'Enter' key

Expected Result:
Message deleted. Focus is on composer field
Actual Result:
Focus lost after deleting message in edit mode
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/user-attachments/assets/67231f2f-46a8-4f10-93cf-7389de640c18
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01fe43add27dc504ef
Upwork Job ID:<PHONE_NUMBER>269229690
Last Price Increase: 2024-09-18
Automatic offers:

brunovjk | Reviewer | 104052891

Issue OwnerCurrent Issue Owner: @brunovjk

We think that this bug might be related to #vip-vsp

This is a pretty edge case bug - don't think it needs to be a blocker

There can be something like handle by state while deleting due to re-render focus can change.
or there can be something handle through searchparams...

Hi All, this one was reproducible on staging and prod, but I was not able to reproduce it on dev main branch. Could this already be resolved?

You're right, I just tested it and at the last main (v9.0.23-0) it seems to no longer be reproducible:
https://github.com/user-attachments/assets/904b7bff-560d-410b-b5c7-db57b6b9261c

Issue is still reproducible
https://github.com/user-attachments/assets/4fe03dee-f213-40b2-8d6a-eb43f07a0a1d

Bumped into Slack to search for proposals.

Strange, because in staging I can reproduce it, but not in the last main (v9.0.25-14). I'll keep an eye out for proposals and test again in a week.

I can still reproduce it in staging. Bumped into Slack for proposals.

Proposal
Please re-state the problem that we are trying to solve in this issue.
When a user is in edit mode and deletes a message, the composer field loses focus.
What is the root cause of that problem?
The application is not properly handling the focus state of the composer field after a message is deleted.
What changes do you think we should make in order to solve the problem?

Observe changes to the lastReportAction?.message in the ReportActionCompose component.
When lastReportAction?.message updates, indicating a message has been deleted, call a focus() function to re-focus the composer input field.

Here:
https://github.com/Expensify/App/blob/aaa60b9103af83aaa58222e3f36e8cbfbd57e5c3/src/pages/home/report/ReportActionCompose/ReportActionCompose.tsx#L124
we should add this code:
useEffect(() => {
    focus();
  }, [lastReportAction?.message]);

https://github.com/user-attachments/assets/6daa78ef-bbc5-45c0-8416-24a4aae0a9a6

Reviewing proposal.

Thank you @brunovjk.
I believe this is a new feature, not a regression.
Looking at the code, we manually focus the input in a few specific cases. For example, when a user tries to submit an empty message while editing, a delete confirmation modal appears. If the user cancels, the input is refocused because it lost focus when the modal was shown.
You can see that here: https://github.com/Expensify/App/blob/89b9226fd6fa0884b8a9c9761759a787e820fcf8/src/pages/home/report/ReportActionItemMessageEdit.tsx#L302-L303
However, when a message is actually deleted, we don’t refocus the input, which is why the behavior feels different. We can’t simply add the refocusing logic when the user confirms the deletion, because two different components are involved: one for writing a comment and another for editing a comment.
Since we don’t have direct access to the comment input from within the edit comment file, we need to handle refocusing by tracking when the comment value updates after a deletion. That can be done in this file: https://github.com/Expensify/App/blob/aaa60b9103af83aaa58222e3f36e8cbfbd57e5c3/src/pages/home/report/ReportActionCompose/ReportActionCompose.tsx#L124
There’s another place where we manually refocus the input when hiding the emoji picker modal: https://github.com/Expensify/App/blob/89b9226fd6fa0884b8a9c9761759a787e820fcf8/src/pages/home/report/ReportActionCompose/ReportActionCompose.tsx#L560-L567
I suggest we follow a similar approach to the one I outlined in my proposal. Alternatively, we could create a new key in Onyx, like commentDelete, and update it when the user confirms deletion. We could then track that value in useEffect and use it to refocus the input.

Proposal
Please re-state the problem that we are trying to solve in this issue.
Web/Desktop - Chat - Focus is lost after deleting a message from edit mode
What is the root cause of that problem?
The focus loss happens because the setUpComposeManagerFocus function does not account for the scenario where the Composer loses focus after returning from the delete message modal. The focus on the Composer is not restored when it becomes visible again after the modal is closed.
What changes do you think we should make in order to solve the problem?
To solve the problem, we should modify the setUpComposeManagerFocus function to check whether the Composer is visible and, if so, restore focus appropriately. The updated version should handle the modal close event effectively and refocus on the Composer as expected.
Following changes ensure that when the modal is closed, the Composer field regains focus and provides a better user experience.

1.webm

Thanks for the proposal @raza-ak, I believe your RCA is not very correct, I tested your proposed solution, and it is triggering focus in the main composer unnecessarily:
https://github.com/user-attachments/assets/4312ae4f-99b3-4025-9915-f53baf34b09b

Thanks for clarifying @Shahidullah-Muffakir, although it makes sense to me, I'm still not 100% sure, could you put everything together in your proposal? That way I can ping an internal eng to take a look. Thanks.

@brunovjk, I’ve updated my original proposal. I combined my earlier explanation with it. Let me know if you need anything else. Thanks!

@bernhardoj sorry to bother you, but can you please take a quick look at this issue here, do you think it might be related to your PR here somehow? Thanks a lot.

I don't think it's related to that PR since that PR fix an auto-focus issue on mount.

@brunovjk I've updated my proposal and fixed all focus issues. Please review it again.

Thank you very much for confirming @bernhardoj.

Thanks for the update and your time @raza-ak, but I believe we still don't have the correct cause of the issue.
@Shahidullah-Muffakir  I've been investigating and I believe that your RCA is also not adequate, sorry about that. When delete or edit a comment it seems to me that we have already taken care to update the composer focus after:
https://github.com/Expensify/App/blob/e9642dc7829f912bbb20be6c0d89b3e6529df7b6/src/pages/home/report/ReportActionItemMessageEdit.tsx#L273-L278
When we edit the focus it is updated successfully (call deleteDraft):
https://github.com/Expensify/App/blob/e9642dc7829f912bbb20be6c0d89b3e6529df7b6/src/pages/home/report/ReportActionItemMessageEdit.tsx#L308-L309
However, by leaving the comment empty, we should also update the focus (call deleteDraft):
https://github.com/Expensify/App/blob/e9642dc7829f912bbb20be6c0d89b3e6529df7b6/src/pages/home/report/ReportActionItemMessageEdit.tsx#L303-L305
However deleteDraft is not being called in delete case
https://github.com/user-attachments/assets/9ede165a-c6d1-4850-9d6e-96528ee112c2
I haven't figured out why yet, @raza-ak and @Shahidullah-Muffakir Thoughts?

@brunovjk Did you watch my updated video in the proposal and did you test my latest code changes locally? I've fixed that issue and i believe i've shared the correct cause of issue with you.

@raza-ak Yes, I have reviewed it, and although the changes seem to solve the problem I want to be sure of the root cause. I understand, I will take a closer look at your proposal and let you know if I find anything different.

@brunovjk I've identified the root cause of the bug.  The issue was that the onConfirm callback (deleteDraft) which should be called when a user confirms the deletion of a message in the showDeleteModal, was not being invoked as expected.
The problem stemmed from a typo in the file:
https://github.com/Expensify/App/blob/b76d91ca3b2f973da1bd0a76871d3b6f9ef588bf/src/pages/home/report/ContextMenu/PopoverReportActionContextMenu.tsx#L33
The variable onConfirmDeleteModal was incorrectly spelled as onComfirmDeleteModal in three instances. After correcting these typos, the functionality is now working as intended, and focus is properly regained after the deletion action.
I've tested the solution, and it resolved the issue. Thank you.

@brunovjk  Thanks for telling me about the deleteDraft function not working. At first, I tried to fix it but couldn't, so I stopped trying. But when you mentioned it again, it made me look harder. Because of that, I found out what was really wrong

Hi @Shahidullah-Muffakir, Thank you for identifying the typo in onComfirmDeleteModal. I corrected it across the file, but after testing, it doesn't seem to affect the functionality or resolve the issue. Upon further review, I couldn't find any references where this ref is actually used. Were you able to resolve it just by fixing the typo?

@brunovjk  I am sorry I forgot to mention one more small issue int the line
https://github.com/Expensify/App/blob/b76d91ca3b2f973da1bd0a76871d3b6f9ef588bf/src/pages/home/report/ContextMenu/PopoverReportActionContextMenu.tsx#L267
it should be
       callbackWhenDeleteModalHide.current =  runAndResetCallback(onConfirmDeleteModal.current);
You can test it with these changes , Thank you.

@brunovjk,  In the original code we are wrapping the execution of runAndResetCallback in an anonymous function. This means the assignment to callbackWhenDeleteModalHide.current is now a function that, when executed, will:
Call runAndResetCallback(onConfirmDeleteModal.current).
Assign the result of runAndResetCallback to onConfirmDeleteModal.current.
However, the result of runAndResetCallback(onConfirmDeleteModal.current) is a noop function (() => {}), and we are assigning this noop function back to onConfirmDeleteModal.current. As a result:
onConfirmDeleteModal.current will get replaced with () => {}
in my suggested changes, we are directly invoking runAndResetCallback and assigning its result to callbackWhenDeleteModalHide.current. This ensures that the callback runs immediately, and the reset behavior works as intended.
Thank you.

Hi @Shahidullah-Muffakir, Thank you for following up and for your persistence in addressing the issue. I tested the latest change you suggested, and it seems like this indeed resolves the problem. The root cause appears to be how the callback was handled in callbackWhenDeleteModalHide, specifically assigning a reset function incorrectly. Could you please update your proposal with these changes? Once that's done, I'll submit it for internal review and assign you if they approve. Thanks again for your efforts!

@brunovjk , Thank you for your feedback! I’ll update the proposal with the suggested changes. Regarding the typos in "onConfirmation," do you want to keep them as they are,  as they  are irrelevant now, or should I correct those as well, ? Thank you.

@brunovjk , Thank you for your feedback! I’ll update the proposal with the suggested changes. Regarding the typos in "onConfirmation," do you want to keep them as they are, as they are irrelevant now, or should I correct those as well, ? Thank you.

I believe we can fix them yes :D

@brunovjk proposal https://github.com/Expensify/App/issues/45472#issuecomment-2336825317 updated, Thank you.

I think we can go with @Shahidullah-Muffakir's proposal now.
🎀👀🎀 C+ reviewed

🎀👀🎀

I will submit the PR within 24hrs.

PR under review.

BugZero Checklist:

[x]  [@brunovjk] The PR that introduced the bug has been identified. Link to the PR: I didn't find any change/PR that could have caused it, from the file history the lines that were changed in this PR were like that since the file was created.
[x]  [@brunovjk] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: The change updated the code by directly assigning the result of runAndResetCallback() to callbackWhenDeleteModalHide.current, eliminating unnecessary indirection and making the behavior more predictable.
[x]  [@brunovjk] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:
[x]  [@brunovjk] Determine if we should create a regression test for this bug. Yes
[x]  [@brunovjk] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Regression Test Proposal

Start the ND Desktop App or go to staging.new.expensify.com and log in with any account.
Open any chat and send a message.
Press the Arrow Up key, then use Ctrl + A (or Cmd + A) to select all text, followed by the 'Delete' key.
Press 'Enter' to confirm the deletion.
Ensure that the focus remains on the composer field after deletion.

Do we agree 👍 or 👎?
cc: @johnmlee101

BugZero Checklist:

[x]  [@brunovjk] The PR that introduced the bug has been identified. Link to the PR: I couldn't find any changes or pull requests that would have caused this issue. According to the file history, those lines have remained unchanged since the file was created.
[x]  [@brunovjk] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: The change updated the code by directly assigning the result of runAndResetCallback() to callbackWhenDeleteModalHide.current, eliminating unnecessary indirection and making the behavior more predictable.
[x]  [@brunovjk] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:
[x]  [@brunovjk] Determine if we should create a regression test for this bug. Yes
[x]  [@brunovjk] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Regression Test Proposal

Start the ND Desktop App or go to staging.new.expensify.com and log in with any account.
Open any chat and send a message.
Press the Arrow Up key, then use Ctrl + A (or Cmd + A) to select all text, followed by the 'Delete' key.
Press 'Enter' to confirm the deletion.
Ensure that the focus remains on the composer field after deletion.

Do we agree 👍 or 👎?
cc: @youssef-lr, @johnmlee101

Regression test here: https://github.com/Expensify/Expensify/issues/438069

All set to issue payment. We need to pay:

@Shahidullah-Muffakir - $250 for PR vio Upwork.
@brunovjk - $250 for C+ visa Upwork.

@brunovjk $250 sent and contract ended!

@Shahidullah-Muffakir you were never hired, so created a new upwork job and send you an offer for $250

@joekaufmanexpensify, Thanks, I accepted it. Just to mention that I've already been assigned to a few issues and have received payment through Upwork (Same Upwork account).

TY! The issue might've been that this issue was several months old.

@Shahidullah-Muffakir $250 sent and contract ended!

New upwork job closed.

All set. Thanks everyone!


Define DBROOT after DEVICE is set

DBROOT depends on the DEVICE variable, so we need to be past any code
that set DEVICE before settting DBROOT.
This cause nmigen fail when using the symbiflow toolchain. See:
https://github.com/SymbiFlow/symbiflow-examples/issues/123
This regression was caused in b35514859f7f02c289c2f6bc058fd3ca1c00143b

@acomodi Please review again :)

@acomodi Please review again :)

I rebased the patch.

I rebased the patch.

@acomodi and @litghost What to do about the "Architecture Definitions (Presubmit)" failure?

@acomodi and @litghost What to do about the "Architecture Definitions (Presubmit)" failure?


Logo proposal

Hello, @farmerbb i am a graphics designer. I will like to know if you want me to create a new logo for your project. I will be creating the logo for free.

Thank you very much!  I would love for you to contribute a new logo to the project :)

I will work on it as soon as possible

Hello @farmerbb this is what i came up with after making several sketch. You can make suggestion, so that can edit any part you want to remove before i proceed to make them into different icon sizes.

Thank you so much!  I like 4, 5, and 6 more than I like 1, 2, and 3.
I do have a couple of suggestions:

Can I see a version with #A3C638 as the background color?  (this is the app's primary color)
Can I see a version without the phone in the foreground, just the computer monitor?

Thank you so much! I like 4, 5, and 6 more than I like 1, 2, and 3.
I do have a couple of suggestions:

Can I see a version with #A3C638 as the background color? (this is the app's primary color)
Can I see a version without the phone in the foreground, just the computer monitor?
Also, a version with the phone a little bit smaller?

I really, really appreciate it!

This is a preview of your request @farmerbb
I applied a slight dark color to the screen to make it more visible, for those with color blindness.

Hi @farmerbb which of it is your preferred choice, so that i can proceed to make other sizes.

It is not too much, based on your suggestion. I have not create the icon yet. I was only showing you the background. Material design drop shadow will be done when i create the icon, i will get that done now.

@farmerbb This is a launcher icon preview based on material design
xxxhdpi preview

xxhdpi preview

If you like it i will proceed to make a pull request, why i also submit a link to editable files.

Looks fantastic! Thank you so much!

Am glad you like it i will go and make a pull request. This is a link to editable filehere, i use android studio to create drop down shadow for material design.

@farmerbb i have create a pull request i have replaced old icon with new icon

Thank you very, very much!
I went ahead and merged your pull request, but I am going to make a few small tweaks myself using the PDF you linked (mainly to support adaptive icons on Android 8.0+)
I'll make sure to attribute you with "App logo by mayowadavid" on the about screen :)

Hello @farmerbb there is no need to make any tweak i just include a file with an adaptive icon.

@farmerbb in case you need any other thing about the logo am always  available.


[Feature] Use Math-LLaVa data

Motivation
Use data from Math-LLaVa paper for better document handling
https://github.com/HZQ950419/Math-LLaVA
https://arxiv.org/abs/2406.17294
Related resources
No response
Additional context
No response

Thank you for your advice!


Report: All the capital cities in a region organised by largest to smallest

A report containing the following columns

Name.
Country.
Population.

provide a Report on All the capital cities in a region organised by largest to smallest


Fix: az vm boot-diagnostics get-boot-log key1 -> keys[0] (#1544)

Hi @ashikv, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (ashikv). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

The fix is incomplete, sorry


Limit ammount of Sub-resources showed in context menu

Godot 3 5f805de6cbfe8a3afc8c5453317fc843b7228c73
Issue description:
Godot has this new cool feature - it shows sub-resources of node when you right click it in inspector.
Problem starts when you have a lot of Sub-resources...

Steps to reproduce:

create node with big amount of sub-resources. Easiest way: create theme using #12718 and assign it to node
right click on node

It would be nice to move this to the bottom, below the other options.


Mocaptools: add metahuman biped support

Updated the tool to support metahuman. It remains backwards compatible with normal biped. It uses the default naming.

should have made a pull request to the master?


new API rgb2gray

closes #xxxx

[ ] tests added
[ ] all tests pass
[ ] documentation
[ ] AUTHORS entry

Thanks for these changes!
Can you please move both nodes (and their tests) to [test_]image.py? Also, each Node needs the following decorators:
@ReturnOutputs
@Output("extracted_image")

And __init__(...) needs to call super().__init__() as the first thing it does.

I guess testimage branch is not merged to master or I am having a problem with branch merge because test_image.py is not appearing on my files when I checkout to this branch(newpipelinecolor). Should I copy this code to testimage branch?

Still no commits. Did you push newpipelinecolor or some other branch?

I am still unable to find test_image.py

Yes, please create a new one, as discussed.

I have shifted code to [test_]image and have deleted the files as discussed, However annotation.py and base.py are not deleted yet. I will delete them once I shift annotation.py to new pipeline. Also I was confused about object_scale.py whether to delete it or not. Can you please confirm?

Please also resolve this error:
_____________________ ERROR collecting tests/test_color.py _____________________

ImportError while importing test module '/home/travis/build/morphocut/morphocut/tests/test_color.py'.

Hint: make sure your test modules/packages have valid Python names.

Traceback:

tests/test_color.py:2: in <module>

    from morphocut.pipeline import Gray2RGB, RGB2Gray

morphocut/pipeline/__init__.py:6: in <module>

    from morphocut.pipeline.pipeline import Pipeline, MultiThreadPipeline

E   ModuleNotFoundError: No module named 'morphocut.pipeline.pipeline'

You can see the build status at the bottom of this page when you click on "Details" next to "continuous-integration/travis-ci/pr". If the build failed, please review the errors and try to fix them.

You need to remove test_color.py:
ImportError while importing test module '/home/travis/build/morphocut/morphocut/tests/test_color.py'.

Hint: make sure your test modules/packages have valid Python names.

Traceback:

tests/test_color.py:2: in <module>

    from morphocut.pipeline import Gray2RGB, RGB2Gray

Before you submit the final version, can you please always run the complete test suite? We otherwise lose valuable time...

You need to remove test_color.py:
ImportError while importing test module '/home/travis/build/morphocut/morphocut/tests/test_color.py'.

Hint: make sure your test modules/packages have valid Python names.

Traceback:

tests/test_color.py:2: in <module>

    from morphocut.pipeline import Gray2RGB, RGB2Gray

Before you submit the final version, can you please always run the complete test suite? We otherwise lose valuable time...

Acknowledged.


Fix nosimage.textproto rpcs examples

Forgot to update the example in the last PR.

Pull Request Functional Test Report for #2428 / 533987529b770dc562cb90ba21f7db69da1bb346
No tests identified for validation.
Help

@dplore PTAL


cabana: syc the last message between message list and history logs

issue: the last message in historyLogs occasionally out of sync with the last message in messageList.
fixed:  fetch data with lastMessage's timestamp instead of can->currentSec.
before:

after:

Is it possible to make the values sync/update when you seek while paused?

Is it possible to make the values sync/update when you seek while paused?

I'll do it in another pr

added a function updateLastMsgsTo to update the last messages after seeking. all states are set to the correct time. the messageList, BinaryView, HistoryLog and signaleView are also updated to the correct last message.
updateLastMsgsTo runs very fast. it takes about 50ms for 11 minutes of data on an old laptop.


Display questions for tasks

... so that people see which question triggered the task.

This was added in RDMO 1.4.

This was added in RDMO 1.4.


New Steam VR icons for ALVR

ALVR already uses the Quest icons from the SteamVR installation. The icons specified in driver.vrresources are unused.


Generate csv file with relevant attributes

Joining the 2 CSV files from Mapi to produce a complementary attribute file to the geojson/topojson

This is currently performed using Pandas as developed in #1


Update postgresql to 42.3.5

Updates org.postgresql:postgresql from 42.3.4 to 42.3.5.
Changelog
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.postgresql", artifactId = "postgresql" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequest = { frequency = "@monthly" },
  dependency = { groupId = "org.postgresql", artifactId = "postgresql" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
86 unchanged lines in 8 files lost coverage.
Overall coverage decreased (-23.2%) to 52.044%

Files with Coverage Reduction
New Missed Lines
%

core/src/main/scala/io/aiven/guardian/kafka/Errors.scala
1
0%

cli-backup/src/main/scala/io/aiven/guardian/kafka/backup/App.scala
2
33.33%

core-cli/src/main/scala/io/aiven/guardian/cli/MainUtils.scala
4
0%

core-restore/src/main/scala/io/aiven/guardian/kafka/restore/KafkaProducer.scala
7
18.18%

restore-s3/src/main/scala/io/aiven/guardian/kafka/restore/s3/RestoreClient.scala
8
0%

core-backup/src/main/scala/io/aiven/guardian/kafka/backup/KafkaClient.scala
12
46.15%

core-backup/src/main/scala/io/aiven/guardian/kafka/backup/BackupClientInterface.scala
14
75.86%

backup-s3/src/main/scala/io/aiven/guardian/kafka/backup/s3/BackupClient.scala
38
0%

Totals

Change from base Build<PHONE_NUMBER>:
-23.2%

Covered Lines:
191

Relevant Lines:
367

💛  - Coveralls

Superseded by #236.


Token-swap ts, flow bindings and build using rollup.

Added token-swap ts and flow bindings and build pure JS versions using rollup, to match the token program

This looks great -- thanks for the work!

Do you mind if I merge?

Go for it!


Update install.md: dep installation via apt

Installation via go did not work as described, with apt it's easier.

The command as is, is actually OS agnostic. What issues did you have with the prior instructions?

Figured it out I think. Because of earlier GOPATH issues I had installed dep via apt and also ran it. Later even with the apt version removed the go version was not found as bash had cached ('hashed') dep's location. ´hash -r´ fixed things.


Revamp replacements

This revamps replacements to make them more consistent.
Changes:

All strings can have placeholders.
Crate name placeholder for strings is available as {{crate_name}}.
Breaking Change: Instead of subdir, tag prefix uses the crate name from Cargo.toml. In practice the crate name should usually be the same as the subdir anyway.
Breaking Change: {{version}} in pro_release_commit_msg now refers to the
version that was released. To refer to the version that will be in git after
cargo-make runs, use {{next_version}}.

Let me know if this should be split out or if that last change is undesired. It felt inconsistent that {{version}} changed depending on what config key you put it in, and the way it was prevented you from using the just released version in pro_release_commit_msg.

I think now that we are doing this the whole mental model can be simplified.  For example, why do we need tag-prefix when we could just specify tag like myprefix-{{version}}. I think the prefix thing was likely there because you didn't do replacements on everything?

Thanks! The idea sounds pretty good to me!
Yes, by default the tag was just version numbers. To add some customization we supported tag prefix for some users who want release-v1.1 in case they have other tags. And using these global variables we can more consistent and flexible.
I will walk through the details in next few days.

ping


dbasize should support -1 (Unlimited)

Is this:
Just place "x" to indicate selection (e.g. "[x]")

[x] feature
[ ] bug:

System Details
in Database country, -1 size means unlimited. Dbasize should be able to do that:

Accept -1 as value
Present "Unlimited" as string interpretation of value

Solved in:
https://github.com/sqlcollaborative/dbatools/pull/1132


merge from arlac77/template-github,arlac77/template-kronos-module

package.json

chore(package): add ^3.7.1 remove ^3.7.0 (devDependencies.ava)

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Can't run static code analysis dur to import conflict

Description of your problem
What steps will reproduce the problem?

Create a package with a module named io
Import something (e.g. StringIO) from standard library's io in cutom module io
Try to run static code analysis on custom module io
Or try to run static code analysis on any other module located in custom io's directory

What is the expected output? What do you see instead?
Instead of running the static code analysis, I get an error related to an import conflict between standard lib io and my custom io module.

Fatal Python error: Py_Initialize: can't initialize sys standard streams
Traceback (most recent call last):
File "/my_package/io.py", line 35, in
ImportError: cannot import name 'StringIO'

Please provide any additional information below
This error raises even on a module located in /my_package/ that does not import the conflicting module or even refer to it.
The same thing happens when calling Python or IPython from this directory, but everything is fine using my_package as a namespace.
It suggests that static code analysis starts a Python kernel right inside the parent directory of the inspected module.
Versions and main components

Spyder Version: 3.1.2
Python Version: 3.6.0 64bits
Qt Version: 5.6.2
PyQt Version: 5 5.6
Operating system: Linux

Dependencies
Please go to the menu entry Help > Optional Dependencies (or
Help > Dependencies), press the button Copy to clipboard
and paste the contents below:
jedi >=0.8.1     :  0.9.0 (OK)
matplotlib >=1.0 :  2.0.0 (OK)
nbconvert >=4.0  :  4.2.0 (OK)
numpy >=1.7      :  1.11.3 (OK)
pandas >=0.13.1  :  0.19.2 (OK)
pep8 >=0.6       :  1.7.0 (OK)
psutil >=0.3     :  5.0.1 (OK)
pyflakes >=0.6.0 :  1.5.0 (OK)
pygments >=2.0   :  2.1.3 (OK)
pylint >=0.25    :  1.6.4 (OK)
qtconsole >=4.2.0:  4.2.1 (OK)
rope >=0.9.4     :  0.9.4-1 (OK)
sphinx >=0.6.6   :  1.5.1 (OK)
sympy >=0.7.3    :  1.0 (OK)

This clearly shows you that shouldn't use standard library names in your packages.
I'm closing this because we're not going to invest time in fixing something that is a bad practice.

It is acknowledged in PEP328 that shadowing standard library modules can happened, and will likely happened as the standard library expends.
Beside, it is why namespaces exist, so that you can use meaningful module names and prevent collision. Doesn't starting Python from within a package goes against the design of that package as a namespace?
So even if using this name probably wasn't the best idea I had (I will look to renaming), I think this issue is still valid.

You're more than welcome to send us a pull request to fix this. But we have to prioritize things and right now don't have time to focus on something like this, sorry.

Just for info, I have been investigating a bit, and the issue does not exist at all with my Windows Anaconda install.
Calling Python or IPython from a directory were a module shadows a name from the standard lib works, although you cannot import the module itself but only the one from stdlib, as expected.
As this is likely the origin of the bug under Linux, does anyone have an idea on how to explain this OS-specific behaviour?


Release 0.4.2

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

@k82cn Is there any reason why we do not just tag master branch to make a release?

Is there any reason why we do not just tag master branch to make a release?

We're going to maintainer latest 2 branch, e.g. release-0.4 & release-0.5; we'll not cherry-pick feature but will cherry-pick bugs.


fish-fillets-ng: install desktop file

Motivation for this change
Thank you, @raskin, for your work maintaining this game in nixpkgs!
This pull requests adds the desktop file (lifted from Debian) so that the game is easily reachable from graphical menus.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

I'm closing as op is not responsive.

Sorry for being not responsive! I will implement the changes suggested by the review in the next couple of days and then reopen this pull request. :-)

@doronbehar I finally got around to tending to this pull request and believe that it can now be merged. Could you have a new look?

@doronbehar I finally got around to tending to this pull request and believe that it can now be merged. Could you have a new look?

@doronbehar I finally got around to tending to this pull request and believe that it can now be merged. Could you have a new look?

@doronbehar I finally got around to tending to this pull request and believe that it can now be merged. Could you have a new look?

Thank you @SuperSandro2000 for your review, I believe I fixed all issues you mentioned. :-)

Thank you @SuperSandro2000 for your review, I believe I fixed all issues you mentioned. :-)

Fixed according to diff. :-)

Fixed according to diff. :-)

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 86912 run on x86_64-linux 1

1 package built:

fish-fillets-ng

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 86912 run on x86_64-linux 1

1 package built:

fish-fillets-ng

Awesome, thank you both, @SuperSandro2000 and @7c6f434c for reviewing and merging!

Awesome, thank you both, @SuperSandro2000 and @7c6f434c for reviewing and merging!


Message Attachments

This is working but needs design help.

Fixes #5

Soundcloud looks weird but w/e

Nice! :sparkles:


BLE Service security

The ble service can be connected to by anyone within ble reach. The service is currently announcing itself indefinitely and basically anyone could connect and control the gateway using the API.
I would say that this is maybe not super good.
There are some options.
Limit the announcement of the gatt service - gpio button could make this pretty smooth. Maybe also do this via the API? The downside is that you must activate the announcement before you connect.
Limit what can be done using the ble service. Atm you can do anything the API allows - maybe this is not good?
Use some form of authentication i.e. pin code (or maybe the walled public key can be used in some way, e.g sending a signed message from the ble client to the service (that already has been initated with the public key)
Simple auth via some pin/code

announce time is now limited to 3 minutes for the Solaris batch


[android] Create a service to listen to the screen

We have switched from having the PwoDiscoveryService be a long running
service, but SCREEN_ON and SCREEN_OFF don't actually trigger when set
in the manifest.  Therefore, let's create a second service that will
monitor these things.  Since the code is isolated, it will be easy to
remove if/when we come up with a better solution

LGTM


Add environment variables to .NET Core task

Required Information
Question, Bug, or Feature?
Type: Feature
Enter Task Name: .NET Core
list here (V# not needed):
https://github.com/microsoft/azure-pipelines-tasks/tree/master/Tasks/DotNetCoreCLIV2
Issue Description
Currently if one needs to supply environment variable to the .NET Core task they need to use a script task. It is very common that the dotnet test command needs environment variables to better configure the test. The current way to provide environment variable is not good enough, as it capitalizes every variable, and that makes it impossible to use that as configuration that is not uppercase for a .NET Core service (using .NET Core configuration).
The way to fix this is to simply allow for environment variables to be specified, as it is currently possible on the script task.
If you need more information please let me know.

You can create a powershell inline script and define the variable as:
$env:variable = "value1"

@internetgdl Not on the azure-pipelines.yaml DotNetCoreCLIV2 task, you can't. This variable on a previous script would not survive for the next task.

I stumbled over the same problem. Finally I found out that environment variables can be assigned to any task. See the pseudo schema for the tasks here: https://docs.microsoft.com/en-us/azure/devops/pipelines/yaml-schema?view=azure-devops&tabs=schema#task
@giggio , this means simply add env: with the desired vars on the  same indent as the task.

Is this new? Ok, this solves my problem.

Not sure, when this came in, I am pretty new to this ;-). Happy to hear that it solves your problem :-).

Is there a way to

I stumbled over the same problem. Finally I found out that environment variables can be assigned to any task. See the pseudo schema for the tasks here: https://docs.microsoft.com/en-us/azure/devops/pipelines/yaml-schema?view=azure-devops&tabs=schema#task
@giggio , this means simply add env: with the desired vars on the same indent as the task.

Is there a way to do this for classic pipelines?

Could you share the exact Yaml syntax?
Doesn't like mine...

task: DotNetCoreCLI@2
enabled: true
displayName: 'Run GremlinApp'
inputs:
command: 'run'
projects: './GremlinApp/GremlinApp.csproj'
workingDirectory: './GremlinApp'
env: AUTHKEY KEY

This works:
      - task: DotNetCoreCLI@2
        inputs:
          command: test
          projects: "tests/**/*Tests.csproj"
        env:
          foo: 1
          bar: 2

I stumbled over the same problem. Finally I found out that environment variables can be assigned to any task. See the pseudo schema for the tasks here: https://docs.microsoft.com/en-us/azure/devops/pipelines/yaml-schema?view=azure-devops&tabs=schema#task
@giggio , this means simply add env: with the desired vars on the same indent as the task.

Is there a way to do this for classic pipelines?

I know this is closed and it's been a year, but it took me a while to track this down so sharing to possibly help someone else. For classic pipelines you can add a Powershell task with an inline script to set your environment variable just before your dotnet test task. Script should be in the following format:
Write-Host ("##vso[task.setvariable variable=EnvironmentVariableName;]$(ValueVariableName)")

EnvironmentVariableName should be the name of the environment variable you are setting. In my case, ValueVariableName is the name of a value stored in Azure Key Vault, but you can use any variable or value you want.


Tests should fail if client cannot connect

Since we have merged #767, if a client instance in a test can't connect to the cluster for some reason, the test hangs indefinitely. This makes the development hard and causes long-running CI jobs (e.g. this happened when I was working on windows GA builds.).
Tests should simply fail if the client couldn't connect for some time.

Proposal: mWe can use the ClientTestSupportBase::get_config in all tests where we instantiate client_config and build on top of it. We can set the cluster connect timeout to a lower value suitable for the tests in this factory class.


selected GPIO not possible

I'm running this library on a Raspberry Compute Module 3, but I can't get it to work.
Particularly, I edited strandtest.py, changing the following:
LED_COUNT = 12
LED_PIN = 45
LED_CHANNEL = 1

Plus I changed pin 45 mode to PWM1, by issuing gpio mode 45 ALT0.
But when I run python3 strandtest.py I get the following:
Traceback (most recent call last):
...
... rpi_ws281x.py", line 130, in begin
raise RuntimeError('ws2811_init failed with code {0} ({1})'.format(resp, str_resp))
RuntimeError: ws2811_init failed with code -11 (Selected GPIO not possible)

However, it looks like the setup is fine. Am I missing something?

I declared three strip objects  and  the same error
`
LED_1_COUNT = 512        # Number of LED pixels.
LED_1_PIN = 18          # GPIO pin connected to the pixels (must support PWM! GPIO 13 and 18 on RPi 3).
LED_1_FREQ_HZ = 800000  # LED signal frequency in hertz (usually 800khz)
LED_1_DMA = 10          # DMA channel to use for generating signal (Between 1 and 14)
LED_1_BRIGHTNESS = 255  # Set to 0 for darkest and 255 for brightest
LED_1_INVERT = False    # True to invert the signal (when using NPN transistor level shift)
LED_1_CHANNEL = 0       # 0 or 1
LED_1_STRIP = ws.WS2811_STRIP_RGB
LED_2_COUNT = 512        # Number of LED pixels.
LED_2_PIN = 13          # GPIO pin connected to the pixels (must support PWM! GPIO 13 or 18 on RPi 3).
LED_2_FREQ_HZ = 800000  # LED signal frequency in hertz (usually 800khz)
LED_2_DMA = 10          # DMA channel to use for generating signal (Between 1 and 14)
LED_2_BRIGHTNESS = 255  # Set to 0 for darkest and 255 for brightest
LED_2_INVERT = False    # True to invert the signal (when using NPN transistor level shift)
LED_2_CHANNEL = 1       # 0 or 1
LED_2_STRIP = ws.WS2811_STRIP_RGB
LED_3_COUNT = 512        # Number of LED pixels.
LED_3_PIN = 21          # GPIO pin connected to the pixels (must support PWM! GPIO 13 or 18 on RPi 3).
LED_3_FREQ_HZ = 800000  # LED signal frequency in hertz (usually 800khz)
LED_3_DMA = 10          # DMA channel to use for generating signal (Between 1 and 14)
LED_3_BRIGHTNESS = 255  # Set to 0 for darkest and 255 for brightest
LED_3_INVERT = False    # True to invert the signal (when using NPN transistor level shift)
LED_3_CHANNEL = 1       # 0 or 1
LED_3_STRIP = ws.WS2811_STRIP_RGB
`
and I get the following:
Traceback (most recent call last):   File "start.py", line 578, in <module>     strip3.begin()   File "/usr/local/lib/python3.7/dist-packages/rpi_ws281x/rpi_ws281x.py", line 130, in begin     raise RuntimeError('ws2811_init failed with code {0} ({1})'.format(resp, str_resp)) RuntimeError: ws2811_init failed with code -11 (Selected GPIO not possible)

I solved the problem by changing LED3`s channel to 0

But GPIO 45 is PWM1, that means channel 1 no?

maybe the channel  is not correspond with PWM


Dev

:tada: This issue has been resolved in version 2.6.2-rc.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 2.6.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


has_object_permission doesn't get called

In my viewset I have the following:
permission_classes = [     Or(And(IsReadOnlyRequest, IsAuthenticated),          And(IsPutRequest, IsObjectUser)) ]
Where IsObjectUser has:
def has_object_permission(self, request, view, obj):         return obj.user == request.user
The has_object_permission method is never invoked when wrapped inside the conditional statement. However, If I use the IsObjectUser as a plain permission class in permission_classes list, it works fine. This isn't suitable for me because I need them for each method, not for the whole viewset, always.
If this is bug, I would love to see it fixed. Any suggestions or workarounds are appreciated.

You will find that the outer Or is evaluating to True and short circuiting (not evaluating the second argument) because And(IsReadOnlyRequest, IsAuthenticated).has_object_permission returns True.
This is because IsReadOnlyRequest and IsAuthenticated both inherit from BasePermission which just returns True for has_object_permission. This is a gotcha when combining permissions with an or, which doesn't normally happen with the default DRF behavior.
To get your desired behavior, you will need to subclass IsReadOnlyRequest and IsAuthenticated so that their has_object_permission returns a meaningful result. I normally just do this:
class IsAuthenticated(permissions.IsAuthenticated):
    def has_object_permission(self, request, view, obj):
        return self.has_permission(request, view)


 Sorry but this looks like a problem with Obspy not Spyder, so please report it with them.

Nope,its actually working in other places like jupyter notebook ,pyzo,etc.
I already reported there and they strongly suggested it is something with
the spyder.
On Fri, Dec 18, 2020, 8:42 PM Carlos Cordoba<EMAIL_ADDRESS>wrote:

Closed #14455 https://github.com/spyder-ide/spyder/issues/14455.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/spyder-ide/spyder/issues/14455#event-4130204382, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AFL65WOUOUP3VUWKDWSOQLDSVNWMXANCNFSM4VA4VSHA
.

Ok, but we don't have access to the files you're reading, so it's not possible for us to reproduce this problem.

No problem ,You could try the following code.
from obspy import read
from obspy.clients.fdsn import Client
from obspy import UTCDateTime
import matplotlib.pyplot as plt
client = Client("IRIS")
net = "IU"  # network of the station
sta = "TATO"  # station code
loc = "00"  # to specify the instrument at the station
chan = "BHZ"
eventTime = UTCDateTime("2020-10-19T20:54:39")
starttime = eventTime - 60  # 1 minute before the event
endtime = eventTime + 15 * 60  # 15 minutes after the event
st = client.get_waveforms(net, sta, loc, chan, starttime, endtime)
st.plot()
[image: Mailtrack]
https://mailtrack.io?utm_source=gmail&utm_medium=signature&utm_campaign=signaturevirality5&
Sender
notified by
Mailtrack
https://mailtrack.io?utm_source=gmail&utm_medium=signature&utm_campaign=signaturevirality5&
19/12/20,
01:12:24 pm
On Sat, 19 Dec 2020 at 05:51, Carlos Cordoba<EMAIL_ADDRESS>wrote:

Ok, but we don't have access to the files you're reading, so it's not
possible for us to reproduce this problem.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/spyder-ide/spyder/issues/14455#issuecomment-748385590,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFL65WPUG7BY4SCBG6K6UMLSVPWZFANCNFSM4VA4VSHA
.

--
~Best Regards
Stay Healthy, Stay Safe
Vivek G Babu
Mob : +919446270718

Closing because this is quite old now. Hopefully you managed to solve this problem.


[regression-test][conf] add master_sync_policy = WRITE_NO_SYNC replic…

…a_sync_policy = WRITE_NO_SYNC
There is no power off scene in regression-test, so add these two configure has no side-effect.
Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


can't work fine with TERM=screen-256color

If I set TERM=screen-256color in .zshrc, when I print cmd, it will display error, use TERM=term-256color work find.

os:ubuntu 16.04
gnome terminal

It looks like this is an issue with the title function: https://github.com/robbyrussell/oh-my-zsh/blob/482587ccc16da0a399f3f91ef4d29def8d22a441/lib/termsupport.zsh#L24-L26
It's weird though, the sequence printed should be invisible and only be recognized by the terminal emulator. This must be a bug in gnome-terminal. This is expected behavior: if you're not running screen or tmux that sequence is not swallowed by the underlying terminal, so it makes sense that it shows up in your prompt.


[ecos] Update ecos library version to 2.0.8

Describe the pull request
Update ecos port version from 2.0.7 to 2.0.8

What does your PR fix?

Fixes ecos library version.

Which triplets are supported/not supported? Have you updated the CI baseline?

No, all supported.

Does your PR follow the maintainer guide?

Yes, I tried my best.

Yes

LGTM, thanks for the PR!


Stabilize ticks with constant sleep and amortize cost of out of order merges

Coverage increased (+0.1%) to 77.929% when pulling 87bdd4c4bc4ef35cea3290d13b06baccaa812e1c on r/tick-stabilization into 59f549a03d16b8d6c34a8fbd6b7f875844157497 on master.

Coverage increased (+0.1%) to 77.929% when pulling 87bdd4c4bc4ef35cea3290d13b06baccaa812e1c on r/tick-stabilization into 59f549a03d16b8d6c34a8fbd6b7f875844157497 on master.

Coverage increased (+0.004%) to 77.837% when pulling 32787bfc3344b3afbcfaefd368079068ee5e6cdd on r/tick-stabilization into 59f549a03d16b8d6c34a8fbd6b7f875844157497 on master.

Coverage increased (+0.004%) to 77.837% when pulling 32787bfc3344b3afbcfaefd368079068ee5e6cdd on r/tick-stabilization into 59f549a03d16b8d6c34a8fbd6b7f875844157497 on master.

Coverage increased (+0.001%) to 77.834% when pulling 9716c6fc3979998a8f5146e37e4a1d84b8dab29c on r/tick-stabilization into 59f549a03d16b8d6c34a8fbd6b7f875844157497 on master.

Coverage decreased (-0.05%) to 77.778% when pulling 9716c6fc3979998a8f5146e37e4a1d84b8dab29c on r/tick-stabilization into 59f549a03d16b8d6c34a8fbd6b7f875844157497 on master.

Coverage decreased (-0.06%) to 77.77% when pulling c1977329b662eb1d21463dd2e37f634b52a64397 on r/tick-stabilization into 59f549a03d16b8d6c34a8fbd6b7f875844157497 on master.

Coverage decreased (-0.06%) to 77.77% when pulling c1977329b662eb1d21463dd2e37f634b52a64397 on r/tick-stabilization into 59f549a03d16b8d6c34a8fbd6b7f875844157497 on master.

Coverage decreased (-0.06%) to 77.775% when pulling 13262728ebad5a5650faab1787fd2a9a21fdc0bb on r/tick-stabilization into 59f549a03d16b8d6c34a8fbd6b7f875844157497 on master.

Coverage decreased (-0.05%) to 77.782% when pulling 0cb7d7b876ed242458879f48ed1e96961f62aa61 on r/tick-stabilization into 59f549a03d16b8d6c34a8fbd6b7f875844157497 on master.

Coverage decreased (-0.05%) to 77.782% when pulling 0cb7d7b876ed242458879f48ed1e96961f62aa61 on r/tick-stabilization into 59f549a03d16b8d6c34a8fbd6b7f875844157497 on master.

Coverage decreased (-0.004%) to 77.829% when pulling fee2a3f3845ea5ccfa5341d2be374a5eec3db477 on r/tick-stabilization into 59f549a03d16b8d6c34a8fbd6b7f875844157497 on master.

Coverage decreased (-0.2%) to 77.674% when pulling eb84380a3eb617e062cf8a157a16552b6d1434ae on r/tick-stabilization into dbfa4e03bb1293f1cfb597775ed71c9c3296672b on master.


added total of factors

Prints out total number of factors

Hello @akshitagupta15june , this algorithm prints the total number of factors of a number; can you please approve this to count towards hactoberfest?


Hardcoding AriaNG Settings for Multiple RPC Servers

I’ve set up multiple servers with Aria2, and all of them have RPC enabled. I’m connecting to these servers using Cloudflare under subdomains, with connection URLs like:
http://us.domain.com:6800/jsonrpc
http://eu.domain.com:6800/jsonrpc

Additionally, I’ve set up a separate web server running the AriaNg-1.3.8-AllInOne interface.
However, I’m facing a few challenges:
The number of servers that the graphical interface needs to connect to is quite large, and their configurations are different.
I want to find a way so that users don’t have to manually configure these settings, even on their first visit. I would like the interface to automatically display all RPC connections as soon as the user accesses it.

How can I pre-configure multiple RPC settings in AriaNg-1.3.8-AllInOne so that they are saved by default and ready to use without the user needing to make any changes?

If you want to set up one rpc setting, you can use AriaNg command api (http://ariang.mayswind.net/command-api.html) to configure rpc settings by a url. You cannot pre-configure AriaNg settings, because all of AriaNg settings are stored in  you browser, but you can export & import settings.

Hello,
I hope you're doing well.
I’m developing a platform called DloadBox, which integrates a web server, aria2c, a Linux-based RPC service, and a Telegram bot connected to this service.
After evaluating various UI options for Aria2, I found that AriaNg All-In-One perfectly suits my needs.
For the installation process of my platform, I need to use your index.html file.
I plan to use it without making any modifications, but to simplify the installation process, I’d like to include this file directly in my repository so that users won’t need to download it from a separate repository.
I wanted to ask for your permission to include the index.html file in my repository.
If you’re okay with this, would it suffice if I credit your work and include a link to your repository in my project’s README file?
If you’re not comfortable with this, I’d appreciate it if you could suggest an alternative approach.
Thank you for your time and consideration.
Best regards

Hello,
I hope you're doing well. I’m developing a platform called DloadBox, which integrates a web server, aria2c, a Linux-based RPC service, and a Telegram bot connected to this service.
After evaluating various UI options for Aria2, I found that AriaNg All-In-One perfectly suits my needs.
For the installation process of my platform, I need to use your index.html file. I plan to use it without making any modifications, but to simplify the installation process, I’d like to include this file directly in my repository so that users won’t need to download it from a separate repository.
I wanted to ask for your permission to include the index.html file in my repository. If you’re okay with this, would it suffice if I credit your work and include a link to your repository in my project’s README file?
If you’re not comfortable with this, I’d appreciate it if you could suggest an alternative approach.
Thank you for your time and consideration.
Best regards

AriaNg is based on the MIT license. As long as you observe the license agreement, you can do anything without my confirmation.