Failed to extract the cache, but taken as a successful cache hit anyway

First the cache was successfully restored here: https://github.com/stevencpp/cpp_modules/commit/bce1c922e0525b936336c67d6ae50e9aed114963/checks?check_suite_id=310043429#step:8:1
Then in the very next commit it failed to restore the cache from the same key to the same path: https://github.com/stevencpp/cpp_modules/commit/23a1cb1b99b080800c3d8f122085a2d5a2a5116a/checks?check_suite_id=311040306#step:8:1 .
There were no changes between the two commits that could explain the change, so maybe there were some changes to the action at that time ?
A few commits later it still failed to restore the cache but now all the checks were successful and yet it refused to save the cache, thinking that it was a cache hit
https://github.com/stevencpp/cpp_modules/commit/664390b2b1665f8a413fe3546db0fda3ea909390/checks?check_suite_id=311386473#step:27:1
So, it would be helpful to figure out what caused the extraction to fail in the first place, but in any case such failures should either fail the cache action or at least allow rebuilding and saving the cache afterwards.

Hey @stevencpp, we haven't released a new version of the action for ~9 days (https://github.com/actions/cache/releases)
I'm not sure what's going on as we should be correctly creating the /usr/local/share/vcpkg/installed directory before extracting the tar. (See https://github.com/actions/cache/blob/master/src/restore.ts#L89)
Is this a consistent failure?
Could you try enabling step debug logging? (https://help.github.com/en/actions/automating-your-workflow-with-github-actions/managing-a-workflow-run#enabling-step-debug-logging)
The cache-hit output should correctly be set to false, as it indicates if the cache failed to restore.  As caches are immutable, save will fail if the key already exists so it's correct to not save a new cache.

Yes it happens on every run. Here's the step debug output for the failed tar command https://github.com/stevencpp/cpp_modules/commit/93d0bbafce2b9208f314a04e4b08b3574403f9b1/checks?check_suite_id=311849601#step:8:61 .
By the way, the link in that article to https://help.github.com/en/articles/virtual-environments-for-github-actions#creating-and-using-secrets-encrypted-variables is outdated. That page has been moved to https://help.github.com/en/actions/automating-your-workflow-with-github-actions/creating-and-using-encrypted-secrets .

I added some code to the step immediately after the extraction to check whether the directory exists and it does. https://github.com/stevencpp/cpp_modules/runs/304845356#step:9:99 Is there any possibility that the effect of that mkdirP call is delayed until after the tar command ?

@stevencpp Great catch! Didn't realize mkdirP was an async function, and we do need to await it. Fixing at #100

Going to wait until Monday to bump the release v1 tag once we've tested everything in that release. I've updated the release branch which you can use in your workflow to test out the fix.
- uses: actions/cache@releases/v1

"Cache restored from key: Linux-vcpkg" .. seems to be working now. Thanks !

v1 has been updated with this fix, thanks for finding this!


[BUG] Warning during core tools build

While building core tools, there are some warnings related to scalastyle. We should remove them.
[INFO] Compiling 83 Scala sources to /path/classes ...
[WARNING] [Warn] /code_path/QualificationAppInfo.scala:19: Unused import
[WARNING] [Warn] /code_path/QualificationAppInfo.scala:314: patterns after a variable pattern cannot match (SLS 8.1.1)
[WARNING] [Warn] /code_path/QualificationAppInfo.scala:331: unreachable code due to variable pattern 'gpuCpuTransitions' on line 314

Fixed by https://github.com/NVIDIA/spark-rapids-tools/pull/701


Enable Quarkus CLI tests as upstream issue is now fixed

Summary
https://github.com/quarkusio/quarkus/issues/32219 is fixed.
Please check the relevant options

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Dependency update
[x] Refactoring
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] This change requires a documentation update
[ ] This change requires execution against OCP (use run tests phrase in comment)

Checklist:

[ ] Example scenarios has been updated / added
[ ] Methods and classes used in PR scenarios are meaningful
[x] Commits are well encapsulated and follow the best practices

Approved and green, merging.


build failing (pandoc / pypandoc related?)

https://travis-ci.org/Alephbet/gimel/jobs/136246047
not entirely sure why, because nothing related has changed. Running python setup.py check -m -r -s locally seems to work ok if pandoc is installed...
@Azd325 perhaps you can help?

Yep, this is pandoc related because the version of the travis ubuntu version is to old for the syntax.
I tried already on another branch to fix this with a new version of pandoc but to good result.
It give me some PATH issues. I tried also to install it over cabal from haskell.
Maybe we take the syntax check out of the tox check!

Yes, if it's so fragile, we might as well take it out. Or I'll have to live with an rst readme :)

For the moment it would be fine to take the check out on travis but keep it
for local testing.
Travis only test the environment what are defined in the travis.yml so it
would be fine if we split it in a  manifest and a Readme check? And we will
only check the manifest on travis...
gingerlime<EMAIL_ADDRESS>schrieb am Mi., 8. Juni 2016, 22:05:

Yes, if it's so fragile, we might as well take it out. Or I'll have to
live with an rst readme :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Alephbet/gimel/issues/11#issuecomment-224711122, or mute
the thread
https://github.com/notifications/unsubscribe/AAaCLemDmUmPWVsy0OYKj-IybFMXJOAcks5qJyCUgaJpZM4IxU9y
.

--

Tim


Fix active filter for urls with query string

I believe the latest changes (v. 3.2) broke some of the active menu items features.
As per this release the navigation menu active checker doesn't work well with submenu urls like:
'/b/method?status=0' and it doesn't highlight the menu path as before.
FYI  previous version (3.1.1) seems to be fine and  menu works as expected.

@mwoszczyna Can you send us your menu config?

Sure mate, I've sent it to<EMAIL_ADDRESS>e-maili.

The change in the function "checkPattern" break the activeCheker for submenu.

@chijinbo please open an issue with more details, like your menu config and the provided informations in the issue template.

@lacodimizer @chijinbo Note that is an old version of the source code, the current code is far away from that version. @chijinbo is there any reason for you to not use the latest version of the package?

Hi @lacodimizer @Shidersz
I just updated from V3.1.1 to the latest version, the change in the CheckPattern (fullUrl() to url()) broken the active check on the menu item with the query string.
The same issue like mwoszczyna mentioned in April, if the menu URL is "/b/method?status=0", then the activeChecker won't mark it as active.
My menu config is like:

[
'text'        => 'Test',
'url'         => '/tests',
'icon'        => 'fas fa-envelope',
'active' => ['/tests','/tests/'],
],
[
'text'        => 'Test Query',
'url'         => '/tests?includedTypes[]=Query',
'icon'        => 'fas fa-cogs',
'active' => ['/tests?includedTypes=Query'],
]

Cheers

@chijinbo Please open an issue explaining your problem with details. However, is there any reason you can't just do:
[
'text' => 'Test Query',
'url' => '/tests?includedTypes[]=Query',
'icon' => 'fas fa-cogs',
'active' => ['/tests'],
]

Why you need to match query parameters on the url?
Do you have multiple menu items pointing to the same base url, but with different query parameters?


Improved Performance

Improved application performance by reducing unnecessary memory allocation.
Removed classes AutoProp<T> and Property<T> to avoid unnecessary abstraction, and removed usage references to those classes.
Modified class NetUtil to store an underlying TextRc object instead of an array vector.
Modified class Socket to use addrinfo objects instead of addrinfoW objects.
Added TextRc::get_line method to improve performance.
Renamed conio header library to io.
Other small changes, including refactoring and reformatting.
Recompiled binaries.

Closing pull request - code needs to be edited.


Some Recent-Emoji doesn't show skinTone indicator

Couldn't be reproduced for a while, issue will be closed.


Fix: Dispose Matrix instances.

Reference Issue

What does this implement/fix? Explain your changes.

Dispose Matrix instances.
Any other comments?

This looks a bit dubious to me. Why is the matrix type disposable in the first place? Shouldn't it just be a Struct?

This looks a bit dubious to me. Why is the matrix type disposable in the first place? Shouldn't it just be a Struct?

Seems like it is using (allocating) Gdi objects internally:
https://github.com/dotnet/corefx/blob/e468cd3c45a1057cf649056ba61dacc99c7fa546/src/System.Drawing.Common/src/System/Drawing/Drawing2D/Matrix.cs#L16
https://github.com/dotnet/corefx/blob/e468cd3c45a1057cf649056ba61dacc99c7fa546/src/System.Drawing.Common/src/System/Drawing/Drawing2D/Matrix.cs#L71

@tebjan
Many Matrix instance were not disposed, but Matrix class implements IDispose.
@wieslawsoltes
Thanks for information.

@H1Gdev Looks like more objects are allocating native memory, maybe we should address all allocations same way as for matrixes?
https://github.com/dotnet/corefx/search?q=Gdip&unscoped_q=Gdip

@wieslawsoltes
#443, #469 ...
I will fix this issue in sequence.

@garyranson
This is only option right now...
Another issue is that SvgElement may implement IDisposable, so we will consider it again at that time.


qemu & goldfish update

Summary
qemu & goldfish update
Mainly:

add goldfish SMP support
add rpmsg syslog support for qemu armv7-a
fix smp boot not enter idle

Impact
qemu & goldfsih
Testing
CT

[Experimental Bot, please feedback here]
No, this PR does not meet the NuttX requirements.
Missing Information:

Summary:

Lacks a clear explanation of why the changes are necessary (fix, update, new feature).
Needs more detail on how the changes work. For example, how is goldfish SMP support added? How is rpmsg syslog support implemented?

Impact:

The impact section is too vague. Specify which architectures, boards, or drivers are affected by the changes to qemu & goldfish.
Address all other impact points (user impact, build impact, documentation, security, compatibility). State "N/A" if not applicable.

Testing:

"CT" is not sufficient.
Provide specific details about your build host (OS, CPU, compiler) and target (architecture, board, configuration).
Include relevant testing logs from before and after the change to demonstrate the issue and the fix.

Recommendations:

Expand the Summary:  Clearly articulate the problem being solved or the feature being added. Provide specific details about the changes made.
Complete the Impact Section: Address all impact points with specific information or "N/A" if not applicable.
Provide Detailed Testing Information: Include your build host and target details, and provide relevant logs from before and after the change.

By providing this missing information, you'll make it much easier for reviewers to understand and evaluate your PR.


Correct ontological term for Pseudonym

On the diagram from the wiki page for pseudonym module, pseudonym is annotated as rdc-meta, which is unresolvable, but means pseudonym. The shex file for this module updates to edam which resolves to identifier and not specifically to pseudonym.
PREFIX obo: <http://purl.obolibrary.org/obo/>
PREFIX dct: <http://purl.org/dc/terms/>
PREFIX edam: <http://edamontology.org/>
PREFIX xsd: <http://www.w3.org/2001/XMLSchema#>

:personShape IRI {
  a [obo:NCBITaxon_9606]
}

:identifierShape IRI {
  a [edam:data_0842];
  dct:identifier xsd:string;
  obo:IAO_0000219 @:personShape
}

thanks for reporting this issue has been fixed in the new model


Asp.net Problem installing

Problem encountered on https://dotnet.microsoft.com/en-us/learn/aspnet/hello-world-tutorial/create
Operating System: windows
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.
It keeps throwing this error when I'm installing the Asp.Net
Error: Invalid option(s):
-f net6.0
'net6.0' is not a valid value for -f. The possible values are:
netcoreapp3.1   - Target netcoreapp3.1
For more information, run 'dotnet new webapp --help'.

@mairaw

@JoeParsa what version of dotnet did you install? Did you make sure to open a new console after installing dotnet, to ensure the environment variables were properly set and read?


Feature Request - Android - Add a notification to update the app - Reported by: @Santhosh-Sellavel

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

Action Performed:

Open the app in a Android device
Wait for an update to be available

Expected Result:
There should be a notification to notify the user that there's an update.
Actual Result:
There's no notification to let the user know there's an update available.
Workaround:
Manually check if there's an update available
Platform:

Where is this issue occurring?

Android

Version Number: 1.1.14-0
Reproducible in staging?: Yes
Reproducible in production?: Yes
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Example provided by @Santhosh-Sellavel :

Expensify/Expensify Issue URL:
Issue reported by: @Santhosh-Sellavel
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1635278281139300
View all open jobs on GitHub

Triggered auto assignment to @conorpendergrast (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

My immediate sense is that there's relatively limited value here because both iOS and Android auto-update apps; I'll see if there's any other perspectives via Slack!

Yes, we should do it for both android & iOS.

@Santhosh-Sellavel Thanks! What's helping you see that it's important and valuable to add this? Do your app not auto-update?

Do your app not auto-update?
No, it doesn’t auto-update for android (auto-update enabled over wifi). Testflight builds auto-update regularly.

What's helping you see that it's important and valuable to add this?
There are some cases auto-update settings can be disabled or not work in the play store.
There might be some incremental optional or mandatory updates. There are chances people might miss important bug fixes.
In-app update notification helps users to be aware of available updates, get updated regularly.

Triggered auto assignment to @francoisl (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

Sounds good - discussed in Slack too, and seems like a safe "just in case" feature to add

Triggered auto assignment to @NicMendonca (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

Posted job to Upwork: https://www.upwork.com/jobs/~01bebfff671c09d7ed

Triggered auto assignment to @AndrewGable (Exported), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

@Santhosh-Sellavel - Is this only Google Play beta problem? Feels like an edge case personally to me.

Is this only Google Play beta problem?

No @AndrewGable, not just a Google Play beta-only problem.
IMO wouldn’t call it an edge case. As I said earlier

There are some cases auto-update settings can be disabled or not work in the play store.
There might be some incremental optional or mandatory updates. There are chances people might miss important bug fixes.
In-app update notification helps users to be aware of available updates, get updated regularly.

But should not force users to update unless it's a mandatory update.

Ok what platforms have this issue and for beta or production only? I understand can turn off auto updates or they can't work, but I don't think we should have "forced updates".

But I don't think we should have "forced updates".

Yes, we shouldn’t have forced updates. Unless there is a vulnerability in the app, that's when forced update will be used!
I experienced the problem in android. We should do the similar in iOS too.

I don't feel the necessity of this feature. If a user blocks the auto-update for some reason. It will definitely make him angry to see update notifications.
Other way, if the app auto-updates for normal users then showing a notification for the same would disturb some users(like me).
If someone blocks a notification then it's his duty to work with bugs. I think the best thing a user does to fix bugs is to update the app.
Lastly, why would someone block the app update? If he chose to update only on wifi then he is on limited internet and would definitely not download the update.

I agree @parasharrajat, in addition managing what releases are "required" updates and not seem like a headache. Thank you for the proposal but I don't think we will implement this at this time.


Document more effectively that async work must be canceled in dispose

...or, failing that, that you must check mounted before calling setState. But that's considered a hack.

Closing as this has now been documented in all of the relevant api docs and the exception.


Device Utilities Idea

Idea for an Extension that interacts with devices connected to the user's computer

cool

Wow! Someone with more extension knowledge than me (lol) should make this


Allow own validation of usernames

with this option it is able to admins to set their own regex to validate the username.
Usecases could be:

blacklisting certian words like admin or support (scammers)
establishing naming standards for private homeservers like @mike.dep24 or @bjoirn.dep12

It could be that changes to config.py have to be made; as far as i read evrything from the config file gets dumped into config.config

Possibly needed features:

check if supplied regex is still matched by original regex so that no invalid usernames can be generated if regex is used wrongly

right yeah that is how this works, so the option needs to be added to the sample config and to make the tests pass here:
https://github.com/ZerataX/matrix-registration/blob/master/tests/test_registration.py#L65
would be nice to have another test that more specifically tests the usecase here described.

testing if one regex is a superset of another doesnt exactly seem trivial to me. almost makes me think it makes more sense to just give a list of strings disallowed at the begining of a username or a seperate regex that if matched aren't allowed?

Yea after a bit of thinking and googeling its prob much easier to make 2 checks - one with the custom and one with the default pattern afterwards

Should be ready for review now
2 Things:
What is this for? https://github.com/xelantro/matrix-registration/blob/master/matrix_registration/api.py#L41

would be nice to have another test that more specifically tests the usecase here described.
And what do you mean by that

ehm you can run the tests by executing
python setup.py test
you'll probably need to copy that entire function and use a different config file with that one. or expand the parameters to also take a config ^^

test_register_13_admin (tests.test_registration.ApiTest) ... ok test_register_14_matrixadmin123 (tests.test_registration.ApiTest) ... ok [...] Ran 54 tests in 2.484s OK
Worked

not yet sure if i like that it both time checks the domain part, but maybe that is nicer for the error message anyways.
also not sure if it's trivial to add a 2nd pattern to the html
https://github.com/ZerataX/matrix-registration/blob/master/matrix_registration/templates/register.html#L52
but it would be nice ^^

Checking the domain part both times is kind of required to make sure that in only applies the username regex to the username and dont freaks out when enounterning a domain with 'bad' words like @guenther:synapseadmin.org or sth.
putting 2 regexes into a pattern field is kind of supported see here and here
Will commit that and then go to sleep as well

Although d9af5e5 will need a test.
Also I wasn't sure if it's username.validation_regex or username['validation_regex']

@ZerataX if you want to test it out https://regex101.com/r/pz3WWQ/1
no admin and no support would be (?!.*?(?:admin|support) instead of .*123.*
Note that this regex will also filter the domain name here

In rare caes that the regex does not allow the domain, it can still be left out in the website
If you want me to not let this happen in the api I could revert 5b4b3db and/or have it check the pure regex provided by the config and check the matrix one afterwards

changing config structure significantly as disscussed in dm

Yas it seems to work \°^°/

Ups forgot to update the to new config style template

Since #70 came up multiple times by now  I propose using server_location: 'http://localhost:8008' as default since most users use it on the same machine anyway
Also referring #61

okay so this seems to work well, but it's not exactly good for packaging. the schema should just be located like e.g. the worldlist, otherwise you have to do all these checks for where it might be and sometimes it just seems to be nowhere.
idk datafiles just seem to end up where ever it's weird.
i only do this for the config file cos i want it in the working directory or somewhere we a user can easily access it.
honestly i should probably just put the config also directly in matrix_registration and then just copy it to working directory if it's not found there or not if --config-path is set. idk.
(though i think looking in .config/matrix-registration and /etc/matrix-registration is also expected behavior from some users(? not sure)
so yeah just copy the schema to matrix_registration and access it like any other file. that way it should be way more stable.
I've also been playing around with https://selenium-python.readthedocs.io/getting-started.html
but haven't yet done much. getting proper tests for the html/js seems like a good idea, but that doesn't have to be part of this pr.

I must be missing something, but running 1.0.0.dev7 and setting the following I was able to register a username with the word "support"
username:
  validation_regex: [] #list of regexes that the selected username must match.        Example: '[a-zA-Z]\.[a-zA-Z]'
  invalidation_regex: '(admin|support)' #list of regexes that the selected username must NOT match.  Example: '(admin|support)'

It is a list, so you are supposed to go like
username:
  validation_regex: []
  invalidation_regex: ['(admin|support)']

or
username:
  validation_regex: []
  invalidation_regex:
     - '(admin|support)'

I really need to make that wiki article

@seth586 i think that functionality isnt actually in that release. ehm it's a little confusing, but it should be in
v0.9.2.dev2
stable 1.0.0 was postponed a little, so these tags are actually older


Disabled ACK, Empty Information Icon.

Describe the bug
Can't click/change/see ACK in Zabbix problems. Also if I click to information icon, next to problem, receiving empty query.
Expected behavior
After clicking ACK expecting that I can write information about problem or change severity.

After clicking information icon

Software versions

Grafana
Zabbix
Grafana-Zabbix Plugin

v8.3.4
4.2
4.2.4

Duplicates #1357


Font Size

when opening under Ubuntu 18.04 the font is too big, cuts off the bottom 1/4 of the characters
happy to provide images or build versions

A screenshot would be handy.

This problem persists in ubuntu derivatives.  I think it might be specific to OpenJDK.  I was running Oracle java until a recent reinstall, and I don't remember having this font problem.
java -version
openjdk version "<IP_ADDRESS>" 2020-11-04
OpenJDK Runtime Environment (build <IP_ADDRESS>+1-Ubuntu-0ubuntu1.20.04)
OpenJDK 64-Bit Server VM (build <IP_ADDRESS>+1-Ubuntu-0ubuntu1.20.04, mixed mode, sharing)
I tried a number of different fonts, and found a couple that work pretty well.
"Coolvetica regular" looks really good, but loses the super cool LCD look of the current digital-7 font.  The license is free for both personal and commercial use.
https://www.1001fonts.com/coolvetica-font.html
"balls on the rampage" almost looks like an LCD display, but the license is for personal use only.
https://www.1001fonts.com/balls-on-the-rampage-font.html


Inform the user that they need to enable HTTP mode

On most recent versions, the codecs default to HTTPS only. This diverges all the HTTP traffic to HTTPS and makes the module fail, since we are not accepting self-signed certs.
The easiest solution is to tell the user to enable HTTP+HTTPS under Network Services.

Alternatively, we could change the rest request to accept insecure connections.

I think the way going forward is https://github.com/bitfocus/companion-module-cisco-webex-websocket, which uses either SSH or Websockets (with the added benefit of providing feedbacks and using a standard Cisco library).


[KSMv2] Make KSMv2 a long-running CoreCheck

What does this PR do?

Make KSMv2 a long-running check (i.e the Interval() method always returns 0)
Make the check collection interval configurable via collection_interval
Implement the Stop() method to shutdown the KSM informers when the check is unscheduled: The Stop() method cancels the context used to build the KSM store and sends a signal to the main Run() method to stop collecting metrics
Make AddWorkStats a public function
Small refactoring of the collector/runner package

Motivation

KSMv2 should be a long-running check because the KSM informers keep running asynchronously and keep watching resources and generating metrics between the check runs, it's required to shutdown the informers gracefully when unscheduling the check
By making KSMv2 a long-running check, we guarantee that the Stop() is called when the check is unscheduled, this is critical - the KSM informers must stop when the check is unscheduled. As opposed to regular checks where the Stop method is only called if the check is being unscheduled during a not-finished check run execution.
The refactoring of the collector/runner package is not necessary, but it would make more sense to have all the functions handling the checkStats variable together in a single file (check_stats.go), since it already exposes functions like GetCheckStats and RemoveCheckStats to other packages. The real change in this package is making AddWorkStats accessible by other packages (needed by the long-running check to update the stats in this case)
Without calling AddWorkStats, the long-running check will never appear in the agent status output

Additional Notes
Understanding long-running checks and how they're handled by the Scheduler/Collector would help reviewing this PR, here is a TL;DR:

The collector will call the long-running check Run method only once, the long running check has to implement a Stop() method which will be called by the collector when unscheduling the check.
In contrast, the regular checks are expected to be stateless, the collector calls their Run method regularly (every 15s), they can implement a Stop() method but it’s called only if the check is running while unscheduling it, i.e the Stop() method will only be called if Unschedule() is called while Run() didn’t finish yet which is very very unlikely.
The Interval() method must return 0 to tell the Scheduler when it's a long-running check.

Describe your test plan

Schedule/Unschedule multiple KSM check v2 dynamically without restarting the Agent, the KSM informers in question must stop when a check is unscheduled
The check checks must appear on the status page
The check must honour the collection_interval  config, fallback to 15s when not defined

closing this in favor of https://github.com/DataDog/datadog-agent/pull/6636 and https://github.com/DataDog/datadog-agent/pull/6652


Rhythm section of chrome music lab isn't working on chromebooks.

I don't write codes or do any programming. I want to use the rhythm section of chrome music lab and it's not working on my Chromebook - meaning it more likely won't on my students' either.
If this is not a suitable issue for this forum, I digress.

Thanks for the issue, but this is related to https://github.com/googlecreativelab/chrome-music-lab, not this repo. If you're looking for a fix, i suggest you file it there.


Add DNS servers to vm definition

What this PR does / why we need it:
Add dns server configuration to vm definition
What type of PR is this?

/kind feature
Special notes for your reviewer:
Does this PR introduce a user-facing change? Then add your Release Note here:

Anexia Provider: Enable up to 4 custom dns entries

Documentation:

NONE

merged upstream as https://github.com/kubermatic/machine-controller/pull/1637


#41_Breadcrumbs

Breacrumb component
ps: This component should be used with the NavLink. But this one has a large padding (1.6rem), which makes the display of the breadcrumb a bit wide in my opinion. It should be possible to set the padding of the NavLink (in absolute value, or at least by pre-defined values (ex: small/normal/large))
ps2: I've also fixed a typing error in readme.md ;-)

Breacrumb component
ps: This component should be used with the NavLink. But this one has a large padding (1.6rem), which makes the display of the breadcrumb a bit wide in my opinion. It should be possible to set the padding of the NavLink (in absolute value, or at least by pre-defined values (ex: small/normal/large))
ps2: I've also fixed a typing error in readme.md ;-)

Hi @Shinji22 feel free to create a PR that allows to update the padding of the NavLink. Maybe we can use some attribute like paddingSize: 'none' | 'small' | 'medium'  | 'large'

nice catch on the README


Dark/light mode profile icons and message image icons

Hello, Is there a way of setting a dark and light mode image icon for chat profiles? Also, is it possible to use image icons on chat message avatar icons? The only way I can think of changing these is through custom css/js, if so, what is the best way to check if the UI is using dark/light theme?

Hello, this enhancement is not in the roadmap, but it should not be hard to implement ; if you have the time, do not hesitate to make a PR to propose this feature in chainlit


default portal coin-recieved

just experience kania sa lan base using v2.1 walang tunog pag nag hulog ng barya. using captive portal and pisowifi app
pero pag google chrome meron.
before kasi meron lahat sa lumang version v1

@chealski captive portal depends on phone if its supports sound or not, v1 actually does not support any captive portal you need to do login in chrome thats why walang issue before


Making embedding from prompt results in different results

First of all thanks for making such a great extension!
Second: when I try just to make an embedding from a prompt and negative prompt the results look different. It’s a pretty long regular prompt separated by commas as is the usual way.

This extension may not be compatible with latest version of WebUI. I have to look into that.
Confirm that you don't use any special syntax, for example attention parenthesis, square brackets, words AND or BREAK – because they cannot make any sense inside and embedding (because are applied at later step) and thus cannot be parsed except literally as characters.

Well, I cannot reproduce. Here are my steps:

commit 22bcc7be428c94e9408f589966c2040187245d81 (pretty recent)
EmbeddingMerge works fine as it is (surprisingly for me that nothing broke it!)
I generate "a photo of a cat, beautiful, high res" with negative "blurry, cropped" at default parameters, fix the seed.
I go into EM and put "a photo of a cat, beautiful, high res", saving as "em_my_cat" for example.
I put "blurry, cropped" saving as "em_my_blur".
I return to txt2img and use em_my_cat as prompt and em_my_blur as negative.
Compere the results!

a photo of a cat, beautiful, high res
Negative prompt: blurry, cropped
Steps: 20, Sampler: Euler a, CFG scale: 7, Seed: 42, Size: 512x512, Model hash: 58c5c27858, Model: suzumehachi_V10

em_my_cat
Negative prompt: em_my_blur
Steps: 20, Sampler: Euler a, CFG scale: 7, Seed: 42, Size: 512x512, Model hash: 58c5c27858, Model: suzumehachi_V10

They aren't "similar", they are exactly the same!
What is your case, step-by-step?

Well, I cannot reproduce. Here are my steps:

commit 22bcc7be428c94e9408f589966c2040187245d81 (pretty recent)
EmbeddingMerge works fine as it is (surprisingly for me that nothing broke it!)
I generate "a photo of a cat, beautiful, high res" with negative "blurry, cropped" at default parameters, fix the seed.
I go into EM and put "a photo of a cat, beautiful, high res", saving as "em_my_cat" for example.
I put "blurry, cropped" saving as "em_my_blur".
I return to txt2img and use em_my_cat as prompt and em_my_blur as negative.
Compare the results!

a photo of a cat, beautiful, high res
Negative prompt: blurry, cropped
Steps: 20, Sampler: Euler a, CFG scale: 7, Seed: 42, Size: 512x512, Model hash: 58c5c27858, Model: suzumehachi_V10

em_my_cat
Negative prompt: em_my_blur
Steps: 20, Sampler: Euler a, CFG scale: 7, Seed: 42, Size: 512x512, Model hash: 58c5c27858, Model: suzumehachi_V10

They aren't "similar", they are exactly the same! What is your case, step-by-step?

Thanks for running that test. I have a very long prompt for both regular and negative prompt. The regular one can’t be compressed into one embedding because it’s too long apparently. So I had to take away all commas to make that one.
Regular:
beautiful award-winning portrait photo, pale washed out 90s style, clear eyes, Holga camera style, retro, dreamy, nostalgic, grainy, blurry, soft focus, vignetting, light leaks, distorted, imperfect, moody, artistic, painterly, ethereal, whimsical, plastic lens, low fidelity, medium format, square format, film photography, toy camera, manual focus, fixed aperture, bulb mode, multiple exposure, zone focusing, plastic body, limited control, unpredictable results, experimental, lo-fi
Negative:
ugly cartoon drawing, blurry, blurry, blurry, blurry, hands, hands, hands, hands, double heads, deformities
In front of the regular prompt I use “a woman” or “a man” etc.
Question: can I split the regular prompt into several parts and make several embeddings that I then merge together and so get the same result as just using the prompt?

I have a very long prompt for both regular and negative prompt.

WebUI implicitly splits large prompts at last comma before 75 tokens, and then combines all parts with invisible BREAK statement. To have more control on this splitting, you may as well use explicit BREAK before you reach 75 tokens.

So I had to take away all commas to make that one.

Why do you need to create embedding of the entire prompt in the first place?

split the regular prompt into several parts and make several embeddings that I then merge together

Merge or concatenate? To merge things, you either may use BREAKkeyword, or EmbeddingMerge inline syntax<'first prompt'+'second prompt'>`

get the same result as just using the prompt

I don't understand the reasons behind it, but yes, this is technically possible but makes little sense.
When you have very large prompt which ends on "… vvv www, xxx yyy, zzz" – you ought to delete everything from the ending up to the point until the token counter would show 75/75.
Then you need to cut this at the last comma. For example, let's assume that 75/75 is showed when your string is "… vvv www, xxx". Then, cut at the comma and get 73/75 as "… vvv www". This will be your first embedding, save it.
Start from the rest of your prompt ("xxx yyy, zzz") and if it is also larger than 75 – do this again; otherwise, this will be your last embedding.
Then you should be able to use your new embeddings in one string, separated with spaces (not commas anymore). This is because if you split at wrong position in original prompt – then implicit BREAK would not be able to split your prompt inside the embedding!
For example, let's pretend that limit is 5 and not 75. If you have
one two, four five, seven eight
(where 3 and 6 are commas)
It would be split by WebUI to
one two BREAK four five, seven eight
(not sure whether it deletes the comma when inserting break, or leaves it at the end of left part)
If you add something short the left side, sometimes it will not shift the break position, for example
zero one two BREAK four five, seven eight
If you try to create embedding from the whole prompt – it will fail. It also fails at the second comma.
But it will succeed exactly in the middle, like one two, four five
Then your second embedding will be seven eight
At the end you will have:
[one two, four five], [seven eight]
WebUI has to put BREAK just after your large embedding, and also before it if you add anything to the left. And you will end up with
zero BREAK [one two, four five] BREAK , [seven eight]
– Which is not what you was getting without embeddings.
Thus, if you use large prompts – you need to split them manually in the same position where they are cut internally.
I can recommend abandoning large prompts with implicit breaking, and using explicit BREAK each time you hit 75 limit. This way you can reserve some space (for example, breaking at 60 or 50) so your added words won't shift the slice and thus not drastically modify the whole composition.
Personally, I use EmbeddingMerge to somewhat shorten my large prompt, for example typing <'white'+'short'> dress instead of white short dress or white dress, short dress (sometimes it works as I want, sometimes it's not…)
But I never convert to embeddings whole prompt because it doesn't give any benefits while hiding things from generation info! (I mean, you won't be able to recover what words you had in your embedding, since it will be seen as external, not as created by EM on the fly as in <''>-syntax).


Create Gitea hook service

This proposed Gitea hook is based on the code of the GitHub's hook.
Fixes #178

I'll have time to look at this on Sunday, thanks for the contribution!

I like having the separate service for Gitea, since it may diverge more in the future, but I think we can reduce quite a bit of the duplication unless I'm missing something. Like the form could be a subclass of the Github form, right? Most of the differences are in the payload.

Thanks for your reply.
The events lists are different (some names are changed, there's at least one additional event and some github events do not exists in gitea). That list could possibly be somehow a parameter of ConfigForm, I guess, but there's also one config option that does not make sense for Gitea. So 2 config 'widgets' are not shared.
There are some 'minor' changes in the payload.
So, yes, most of the code is duplicated. Same goes with gitlab.py, actually, that's why I did not try to factorize the code.
The difference of code between those 3 hooks is sometimes quite "subtle", so factorizing in a base class is perhaps not that easy.

Works for me, thanks again for the contribution. It might take a day or two to deploy, our CI provider doesn't do open source anymore and the deploy tools I used for this project a decade ago don't work anymore 😆 I'll move over to GitHub actions.

Thanks for the merge, and thanks for your work !

Hello, @TkTech. Any ETA ?

The host for n.tkte.ch is switching to hetzner, just twiddling my thumb waiting for their support to get it running.

@sdegrande progressing in #207

Deployed, and future deployments are automatic.

Great ! Thanks a lot.


Allow overriding the default http client

I think it would be highly beneficial for webview to allow the user to specify a http client to use. The main reason for this is to be able to intercept http requests and return custom http, this would be useful when writing desktop applications with a custom gui without exposing a port.
Although I'm not sure this is possible, if it is that would be amazing.

Did you check web browser engines such as WebKit and Edge to see if it is possible e.g. through the dev tools? The natural solution in my opinion is to use an HTTP(S) proxy but from what I understand it undesirable for you.


Fix git ownership issues on leader config volume

Deploys a ConfigMap resource that is mounted to /.gitconfig which defines the safe path:
[safe]
directory = /opt/cribl/config-volume

this is to resolve the error:

Error: fatal: detected dubious ownership in repository at '/opt/cribl/config-volume'

Resolves #59 #163 #164 #165

Also threw in the ability to deploy extra manifests through the chart. This functionality is copied from the edge chart.

Verified this is working in the customer's environment.


Ollama Multithreaded UI integration, issue with code testing below python 3.10

Need to verify code test #3 doesn't cause breaks due to dependency on python  <3.8

@eleijonmarck  - Merge confirmed

Python versions lower than 3.10 are not supported.
I think the Ollama stuff you built in https://github.com/gpt-engineer-org/gpt-engineer/pull/1098 makes more sense as a seperate repo.
Closing.


Fixed "Show the app in the terminal" URL (zh-cn)

The app is listening on port 8080 so that must be included otherwise port 80 (default) is used and the curl request will fail.
Issue: 41259

/hold
For en PR #41929 to get merged first.

/hold
For en PR #41929 to get merged first.


an idea

hi there,
:>

1

there is a case:
i d use combined keys <ctrl+shift+[1~9]> to add [1~9] item(s) to my current table, the number of new item is depended on the number key i pressed but its not so friendly for me to write like
hotkeys('ctrl+shift+1,ctrl+shift+2, ctrl+shift+3, ....', callback());

i` want to know if wildcards are supported, such as regular matching or variables,
for example,
hotkeys('ctrl+shift+[:mykey]', (e, h) => {
  console.log(h.params.mykey);  // log to see
})

2

and i`m curious about when i use
hotkeys('*', (e, h) => {
  console.log(h.key)
})

it will log '*' no matter what button I press
(personally think it will show which key i pressed, however
thanks

This is a bad idea and will increase the size of the lib.
@lopo12123

哦 原作者没加这功能你抄不出来啊 hhh

@lopo12123 是的，我等原作者实现了这个功能，我才能实现。


build(deps): @sentry/cli from 2.31.0 to 2.31.2

This PR resolves []
Description

PR Checklist

[X] I have tested my changes on iOS and Android.
[X] I hid my changes behind a feature flag, or they don't need one.
[X] I have included screenshots or videos, or I have not changed the UI.
[X] I have added tests, or my changes don't require any.
[X] I added an app state migration, or my changes do not require one.
[X] I have documented any follow-up work that this PR will require, or it does not require any.
[X] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

update @sentry/cli from 2.31.0 to 2.31.2

Need help with something? Have a look at our docs, or get in touch with us.

This PR contains the following changes:

Dev changes (update @sentry/cli from 2.31.0 to 2.31.2)

  Generated by :no_entry_sign: dangerJS against 29fc4248e8ab0a0318a74c415283903ca60f1320


How to process transformations on a container

For the needs of startinblox, we need to process some transformations on a container to sort, filter, group... resources.
To do this, we recreate an array of resources from the AsyncIterator returned by query-ldflex to perform the transformations.
We have two main concerns with this approach:

it's complicated to use native javascript functions on the arrays like sort, filter because of the asynchronous proxies
it's probably not a good practice to recreate an object from an already existing one.

Is there a smarter way to handle this?

You will probably like https://www.npmjs.com/package/iter-tools 🙂

Thanks, it sounds interesting! I'll take a look ;)

Feel free to reopen if you don't find what you need!

I added iter-tools to startinblox and it fits almost all my needs. The only problem I face is that I can't sort a list of resource, because of the asynchronicity of the proxies (see https://github.com/iter-tools/iter-tools/issues/286)
Should the sort feature be added to query-ldflex as a parameter? I guess it make sense to sort resources before creating an iterator?

Sorting may indeed be valuable, as this could be pushed down to the SPARQL level by LDflex.

Good idea; tracking sorting in https://github.com/RubenVerborgh/LDflex/issues/41/.
Note however that, in several case, sorting makes operations blocking, i.e., you receive all of them at the end instead of incrementally. So this is why we, by default, prefer to leave it to the consumer to perform sorting. For instance, if you are feeding a UI, you could take unsorted results and then insert them in the right place as they arrive. But that all depends on the use case.
As iter-tools also says:

For example: sorting, shuffling and in general all operations that rely on having the full array at your disposal. In that case the way to go is to convert the iterable in an array and use that.

So you could do https://www.npmjs.com/package/iter-tools#to-array and take it from there for now?

Yes the solution would be to re-create an array with the sorting key and then sort the resources. This was our last choice but I think this is the only one possible for now

Sorting landed via https://github.com/RubenVerborgh/LDflex/pull/43.


feat: add headers to connect

Status
READY
Description
provide headers-setting
Type of Change

[x] ✨ New feature (non-breaking change which adds functionality)
[ ] 🛠 Bug fix (non-breaking change which fixes an issue)
[ ] ❌ Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🧹 Code refactor
[ ] ✅ Build configuration change
[ ] 📝 Documentation
[ ] 🗑 Chore

I can't seem to get this to work. I'm using v0.1.1 with headers support, but even though I'm passing in headers, it does not include them in the request.
Do you perhaps have a working sample I can look at?


rust: integration with an existing colours crate

on option as suggested on discord: https://crates.io/crates/css-colors
it would be nice to either use these colour types directly, or provide a feature flag gated integration with the crate. that way we won't need to duplicate all its functionality for our own custom colour type.

And in case we do proceeds with using css-colors types directly, it would be wise to open an issue in css-colors about integrating with palette another rust colors crate, which has a lot more functionality, but lacks the option of formatting directly as css values, which imo feels like a pretty solid feature for this crate.
Palette is used in a lot of larger Rust projects, so it would make sense to be able to convert between our color types, whether that will be our own, or css-colors, and palette's


Bluetooth: Mesh: LPN: Clear sent_req on failure

When trying to establish friendship the Friend must respond to the
initial Friend Poll with a Friend Update. If this initial Friend Update
response is not received the Friendship establishment process must start
again.
When starting a second Friendship establishment processes the sent_req
field of the lpn struct was left set to TRANS_CTL_OP_FRIEND_POLL.
This prevented the initial Friend Poll being sent out on the second
attempt. Since the Friend Poll was not sent, no timeout is set and
nothing happens ever again. No more Friendship Requests are sent.
This commit clears sent_req back to zero when no Friend Update
response has been received after the initial Friend Poll.
Fixes #16678
Signed-off-by: Rich Barlow<EMAIL_ADDRESS>

@MariuszSkamra @tedd-an would it be possible to run all LPN PTS test cases with this PR, to see if anything breaks?

@richardbarlow I'm assuming you've tested that this fixes your issue equally well (or better) than the hack you had with toggling LPN through some callback?

@jhedberg ah yes apologies, I meant to say what testing I'd done, but forgot in my haste to get lunch.
I first of all made the problem reproducible by adding a return 0 to the start of bt_mesh_friend_poll in friend.c so that the friend node would never respond to a Friend Poll. I flashed this hobbled firmware onto my friend node and then ran the LPN node firmware on a separate board to verify that it did indeed reproduce the problem.
I then applied the fix to the LPN node firmware and observed that the problem was no longer present (it continually kept making Friend Requests).

@jhedberg I will test this PR tomorrow


Switch to another storage account

Could you please advise how can I switch Linux Diagnostic extension to another storage account with PS ?
Supplying account name and credentials to Public and Private configs and then Set-AzureRmVMExtension with them seems not to be working - account remains the same. Though configuration obtained with Get-AzureRmVMExtension after change contains new storage account values.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 13249027-1524-dd0c-9d83-15a9733a3d19
Version Independent ID: bf8a2d1a-8d9b-228a-6919-16127f315d40
Content: Azure Compute - Linux Diagnostic Extension
Content Source: articles/virtual-machines/extensions/diagnostics-linux.md
Service: virtual-machines-linux
GitHub Login: @abhijeetgaiha
Microsoft Alias: agaiha

Also, if you try to change storage account in Diagnostics Settings on Azure portal, this error appears:
Failed to update diagnostics settings for . Changing property 'publisher' is not allowed.

Thanks for the question! We are investigating and will update you shortly.

I also get the "Failed to update diagnostics settings for . Changing property 'publisher' is not allowed" when trying to change the storage account or set "Basic metrics" in the portal.  Looking at the VM configuration via the Azure CLI az vm command show's that the diagnostic extension is configured to collect various metric's but  "Basic metrics" is not checked in the portal.

@abhijeetahuja Can you please chime in and share your insights on what is the best way to switch Linux Diagnostic extension to another storage account.

Which version of LAD is this - 2.3 or 3.0?

Adding 3.0 after removing 2.3, VM was restarted between removing 2.3 and
adding 3.0
On Tue, 21 Aug 2018, 20:06 Abhijeet Gaiha<EMAIL_ADDRESS>wrote:

Which version of LAD is this - 2.3 or 3.0?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/13442#issuecomment-414787464,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYsuPATrJrxiaMm0ff3vYW_FI941LIlHks5uTFovgaJpZM4V-CpG
.

I would advise deleting the extension completely and then retrying the install script provided in the docs with the new storage acct.

My 2 cents. I have LAD 3.0 and only want to redirect it to new storage account. I know that reinstalling LAD will achieve this goal but I want to redirect LAD, not to reinstall it.

@Drony77 The only way to update the LAD config is by installing the extension again. You don't need to uninstall it but you do need to run the install command again. That will send the updated config to the already installed extension on the VM.

@OwainWin How are you selecting the metrics? The docs have a default metric configuration that is used by the install script. You can modify that for your purposes.

@Drony77 The portal team should be able to help out with the portal issues. They will also update the config on the VM through a similar mechanism as running the install command a second time.

@abhijeetgaiha Is it by design ?
What  PS cmdlet Set-AzureRmVMExtension is used for then ? As I stated in first message, I can see updated config in LAD itself, but it's not updated on portal and it does not use new storage account. Do you confirm it is a bug ?

@abhijeetgaiha I need to change storage accounts for tens of virtual machines, so I need script to do it and I can't contact portal or other support to help me out.

@Drony77 If the portal doesn't reflect the updated config, then it's a bug in the portal.
Is the data flowing to the updated storage account? You can check if the tables are created and the metric entries are being added to the table to verify that.

@Drony77 A script that runs the PS command for every machine would work in your situation.

@abhijeetgaiha No, data is flowing to the old storage account

In the metrics blade of the portal by clicking the basic check box.  As I
said viewing the VM's JSON configuration via the CLI shows that there are
metrics set but they can't be changed in the portal.
On 22 Aug 2018 10:44, "Abhijeet Gaiha"<EMAIL_ADDRESS>wrote:
@OwainWin https://github.com/OwainWin How are you selecting the metrics?
The docs have a default metric configuration that is used by the install
script. You can modify that for your purposes.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/13442#issuecomment-414975733,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYsuPFgJX7dVxznQhnjj3_mTaEWB241Qks5uTSfwgaJpZM4V-CpG
.

@Drony77 @OwainWin Can you email us at<EMAIL_ADDRESS>and provide us with your Subscription ID and link to this thread? We can enable a one time free support request for you.

@Karishma-Tiwari-MSFT We have Premier Support agreement and I can raise support request on my own if you recommend it

Ok, I'm currently on holiday so I'll do this when lm back in the office in
about a week and a half.
On Wed, 22 Aug 2018, 21:51 Karishma Tiwari - MSFT<EMAIL_ADDRESS>wrote:

@Drony77 https://github.com/Drony77 @OwainWin
https://github.com/OwainWin Can you email us at
<EMAIL_ADDRESS>and provide us with your Subscription ID and
link to this thread? We can enable a one time free support request for you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/13442#issuecomment-415175309,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYsuPCNggP3VYRPotFMAyduGcFwyJug7ks5uTcREgaJpZM4V-CpG
.

@OwainWin Could you provide all the files under /var/log/azure? That will help us investigate further.

@Drony77 Could you provide all the files under /var/log/azure? That will help us investigate further.

@Drony77 @OwainWin As discussed above, please go ahead and open a support request. We will now close this issue. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.
@abhijeetgaiha Thanks for looking into it. Appreciate your help.

Looks like this is as design.
Portal UX does not support LAD 3.0 yet. https://github.com/Azure/azure-linux-extensions/tree/master/Diagnostic

@alexMS Thanks for the update ! Then it leaves only first question open: how can one switch storage account for LAD 3.0 with PS ?
In the same article below:
LAD 3.0 is installed and configured via Azure CLI, Azure PowerShell cmdlets, or Azure Resource Manager templates.


[$1000] Only single line of text is visible in message composer in expanded mode.

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

Action Performed:

Go to any chat,
Type at least three lines of text,
Expand composer,
Scroll back and forth.

Expected Result:
Composer box should have multiple lines of text
Actual Result:
Only a single line of text is visible
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

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

[x] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.2.20-3
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/60e3ccc5-4ce2-4b1a-bfb7-de0f5326e955
https://github.com/Expensify/App/assets/43996225/9faf2bbf-afc7-468a-ae04-a76750c20c76
Expensify/Expensify Issue URL:
Issue reported by: @nishancx
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1685293401457519
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0126bc26134a0dfb53
Upwork Job ID:<PHONE_NUMBER>656848896
Last Price Increase: 2023-06-01

repro'd added external label to start reviewing proposals

Proposal
Please re-state the problem that we are trying to solve in this issue.
Single line of text is visible in message composer expanded mode.
What is the root cause of that problem?
The maximumNumberOfLines props passed to RNTextInput [ here] currently sets to undefined on expanded mode. This causes the issue of single line expanded mode. (https://github.com/Expensify/App/blob/main/src/components/Composer/index.android.js)
maximumNumberOfLines={!this.props.isComposerFullSize ? this.props.maxLines : undefined}
What changes do you think we should make in order to solve the problem?
Setting maximumNumberOfLines to a very large number of lines will solve this problem. We can implement this -
maximumNumberOfLines={!this.props.isComposerFullSize ? this.props.maxLines : 10000}
What alternative solutions did you explore? (Optional)
Setting the maximumNumberOfLines from the maxLines props can also be explored in case of expanded mode.

thanks for the clarification! ill move to weekly while we wait for the other fix to be live.

i will attempt to repro now that the other PR is merged.

i believe this is fixed / i can no longer reproduce this on my end so i am closing this one out!


update to centos 7 based ambari 2.2.1 docker images

@akanto

Jenkins build failed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1948/

Jenkins build failed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1949/

Jenkins build failed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1950/

Jenkins build failed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1951/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1952/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1954/


Add dependency for ansible.posix.synchronize

ansible.posix.synchronize is a wrapper around rsync.1 This module is used in the resolveconf role.
As rsync is now a required dependency for a role, make sure it is actually available on the target system.
Good alternative would be to just trash the file on the remote system, because systemd-resolved is used nevertheless.

The rsync package is only necessary on the Ansible controller and not on all systems, right? We use the packages role to manage the packages on the nodes themselves.
rsync is installed in the osism-ansible container image (which is used to execute the resolvconf role, among other things). I thought that was sufficient?
https://github.com/osism/container-image-osism-ansible/blob/main/Containerfile#L55

The rsync package is only necessary on the Ansible controller and not on all systems, right? We use the packages role to manage the packages on the nodes themselves.

The role resolveconf actually delegates to each node. Therefor, rsync is necessary on each server, as ansible.posix.synchronize is executed there.

rsync is installed in the osism-ansible container image (which is used to execute the resolvconf role, among other things). I thought that was sufficient?

Nope. I don't even think this rsync is used at all.
Another approach would be to replace the whole ansible.posix.synchronize call with a combination of ansible.builtin.copy and ansible.builtin.file → state: absent. This would get rid of the only appearance of ansible.posix.synchronize and with that we could drop rsync at all.

Ah I overlooked the delegate_to. I would prefer to get rid of the ansible.posix.synchronize task. I think the approach you proposed works.

I will create a new PR for this.


Critical Display issue

A user with a fresh install sends USDT to his account on a HUAWEI p30pro, but the wallet displays the '.' separator instead of the ',' and reads 1.5$ instead of 1,500$. Works properly as a watch wallet on my end (as seen on the screenshot), so I cannot replicate it. Etherscan shows the correct 1500+.

The 'insufficient USDT' message itself might also come from the fact he has no ETH to perform the transaction, but the display issue still stands.

Wallet address: 0xf0f096F2Da9F7768B6F968fba970124fe18a89eE

Got it, yes this should be standardised

Got it, yes this should be standardised

Created #2358. Sorry missed it earlier. Should be relevant.

Closing this issue as solved by #2358


[WIP] Implement configRetriever and ipv6routerenderer enhancements

Coverage increased (+0.1%) to 76.124% when pulling 60246c14361d448e87554cfdd8c3f32e501ab267 on lukasmacko:dev into 029d096b67a8af2c62a88f19ae604a973d8c2148 on contiv:dev.


Added TextStyle color for ClickableText

Hi!
Regarding to #837
My solution would be (in SearchScreen.kt) to  add in the TextStyle type the next line:
color = MaterialTheme.colorScheme.secondary

ClickableText would stay like this:
ClickableText(
   text = tryAnotherSearchString,
   style = MaterialTheme.typography.bodyLarge.merge(
       TextStyle(
           color = MaterialTheme.colorScheme.secondary,
           textAlign = TextAlign.Center,
       ),
   ),
       modifier = Modifier
       . padding (start = 36.dp,
   end = 36.dp, bottom = 24.dp,
)
       .clickable {},
)

What do you think?

Does work?
color = MaterialTheme.colorScheme.secondary

what abou the dark theme? Is it working fine on dark theme?

Yes, it works in both themes, you can see it in the images in the previous message. @ayaankhan98


Correct ~/.vim/bundlesVimRc.custom filename

Additionally, you may want to remove the documentation inside vimrc.custom.before that says you can put NeoBundle statements in there. I ran into errors trying to do that.

Thanks!


Update setuptools.txt

This took me forever to figure out, and it seems to be working. I don't see anywhere in the documentation that references local packages that aren't on git or pypi.

Summary of changes

Closes
Pull Request Checklist

[ ] Changes have tests
[ ] News fragment added in changelog.d. See documentation for details

I haven't yet had any luck making this work. With setup.py install:
draft $ cat > setup.py
import setuptools
setuptools.setup(
 name='foo',
 version='1.0',
 install_requires=['zipp'],
 dependency_links=[
  'zipp @ file:///Users/jaraco/m/zipp',
 ],
)
draft $ python -m venv env
draft $ env/bin/python setup.py install
running install
running bdist_egg
running egg_info
creating foo.egg-info
writing foo.egg-info/PKG-INFO
writing dependency_links to foo.egg-info/dependency_links.txt
writing requirements to foo.egg-info/requires.txt
writing top-level names to foo.egg-info/top_level.txt
writing manifest file 'foo.egg-info/SOURCES.txt'
reading manifest file 'foo.egg-info/SOURCES.txt'
writing manifest file 'foo.egg-info/SOURCES.txt'
installing library code to build/bdist.macosx-10.9-x86_64/egg
running install_lib
creating build
creating build/bdist.macosx-10.9-x86_64
creating build/bdist.macosx-10.9-x86_64/egg
creating build/bdist.macosx-10.9-x86_64/egg/EGG-INFO
copying foo.egg-info/PKG-INFO -> build/bdist.macosx-10.9-x86_64/egg/EGG-INFO
copying foo.egg-info/SOURCES.txt -> build/bdist.macosx-10.9-x86_64/egg/EGG-INFO
copying foo.egg-info/dependency_links.txt -> build/bdist.macosx-10.9-x86_64/egg/EGG-INFO
copying foo.egg-info/requires.txt -> build/bdist.macosx-10.9-x86_64/egg/EGG-INFO
copying foo.egg-info/top_level.txt -> build/bdist.macosx-10.9-x86_64/egg/EGG-INFO
creating dist
creating 'dist/foo-1.0-py3.8.egg' and adding 'build/bdist.macosx-10.9-x86_64/egg' to it
removing 'build/bdist.macosx-10.9-x86_64/egg' (and everything under it)
Processing foo-1.0-py3.8.egg
Copying foo-1.0-py3.8.egg to /Users/jaraco/draft/env/lib/python3.8/site-packages
warning: install_lib: 'build/lib' does not exist -- no Python modules to install

zip_safe flag not set; analyzing archive contents...
Not found: zipp @ file:///Users/jaraco/m/zipp
Adding foo 1.0 to easy-install.pth file

Installed /Users/jaraco/draft/env/lib/python3.8/site-packages/foo-1.0-py3.8.egg
Processing dependencies for foo==1.0
Searching for zipp
Reading https://pypi.org/simple/zipp/
Downloading https://files.pythonhosted.org/packages/be/69/4ac28bf238f287f1677f41392e24d2c4ffafcf11648c23824f5f62ef6ccb/zipp-2.1.0-py3-none-any.whl#sha256=ccc94ed0909b58ffe34430ea5451f07bc0c76467d7081619a454bf5c98b89e28
Best match: zipp 2.1.0
Processing zipp-2.1.0-py3-none-any.whl
Installing zipp-2.1.0-py3-none-any.whl to /Users/jaraco/draft/env/lib/python3.8/site-packages
writing requirements to /Users/jaraco/draft/env/lib/python3.8/site-packages/zipp-2.1.0-py3.8.egg/EGG-INFO/requires.txt
Adding zipp 2.1.0 to easy-install.pth file

Installed /Users/jaraco/draft/env/lib/python3.8/site-packages/zipp-2.1.0-py3.8.egg
Finished processing dependencies for foo==1.0

With pip install:
draft $ rm -r env; python -m venv env; env/bin/pip install -v .
Created temporary directory: /private/var/folders/c6/v7hnmq453xb6p2dbz1gqc6rr0000gn/T/pip-ephem-wheel-cache-dvbud3e3
Created temporary directory: /private/var/folders/c6/v7hnmq453xb6p2dbz1gqc6rr0000gn/T/pip-req-tracker-yusw2kug
Created requirements tracker '/private/var/folders/c6/v7hnmq453xb6p2dbz1gqc6rr0000gn/T/pip-req-tracker-yusw2kug'
Created temporary directory: /private/var/folders/c6/v7hnmq453xb6p2dbz1gqc6rr0000gn/T/pip-install-od68kdj6
Processing /Users/jaraco/draft
  Created temporary directory: /private/var/folders/c6/v7hnmq453xb6p2dbz1gqc6rr0000gn/T/pip-req-build-zo5rbfpg
  Added file:///Users/jaraco/draft to build tracker '/private/var/folders/c6/v7hnmq453xb6p2dbz1gqc6rr0000gn/T/pip-req-tracker-yusw2kug'
    Running setup.py (path:/private/var/folders/c6/v7hnmq453xb6p2dbz1gqc6rr0000gn/T/pip-req-build-zo5rbfpg/setup.py) egg_info for package from file:///Users/jaraco/draft
    Running command python setup.py egg_info
    running egg_info
    creating pip-egg-info/foo.egg-info
    writing pip-egg-info/foo.egg-info/PKG-INFO
    writing dependency_links to pip-egg-info/foo.egg-info/dependency_links.txt
    writing requirements to pip-egg-info/foo.egg-info/requires.txt
    writing top-level names to pip-egg-info/foo.egg-info/top_level.txt
    writing manifest file 'pip-egg-info/foo.egg-info/SOURCES.txt'
    reading manifest file 'pip-egg-info/foo.egg-info/SOURCES.txt'
    writing manifest file 'pip-egg-info/foo.egg-info/SOURCES.txt'
  Source in /private/var/folders/c6/v7hnmq453xb6p2dbz1gqc6rr0000gn/T/pip-req-build-zo5rbfpg has version 1.0, which satisfies requirement foo==1.0 from file:///Users/jaraco/draft
  Removed foo==1.0 from file:///Users/jaraco/draft from build tracker '/private/var/folders/c6/v7hnmq453xb6p2dbz1gqc6rr0000gn/T/pip-req-tracker-yusw2kug'
Collecting zipp (from foo==1.0)
  1 location(s) to search for versions of zipp:
  * https://pypi.org/simple/zipp/
  Getting page https://pypi.org/simple/zipp/
  Found index url https://pypi.org/simple
  Looking up "https://pypi.org/simple/zipp/" in the cache
  Request header has "max_age" as 0, cache bypassed
  Starting new HTTPS connection (1): pypi.org:443
  https://pypi.org:443 "GET /simple/zipp/ HTTP/1.1" 304 0
  Analyzing links from page https://pypi.org/simple/zipp/
    Found link https://files.pythonhosted.org/packages/53/87/3763ae5d8eedf3189f43d810880474afbed8e3133edb274564f1213a75ca/zipp-0.1.0-py2.py3-none-any.whl#sha256=d4f0e226d38a00cddd05f84850971a5e8a7dfdab8e8827ca45642e4078f01461 (from https://pypi.org/simple/zipp/) (requires-python:>=2.7), version: 0.1.0
    Found link https://files.pythonhosted.org/packages/36/69/e69f37b9381538360254bff365ade32f54c829041a14f8150992f5dba3d0/zipp-0.1.0.tar.gz#sha256=849d59947815d7f86686eab57283224aa549b27b4deb8ccb61b80014b9de574c (from https://pypi.org/simple/zipp/) (requires-python:>=2.7), version: 0.1.0
    Found link https://files.pythonhosted.org/packages/32/2b/06b52565ab91fa6daa3ce9098e5ee47d62193f077f6bdbb21b452f43bdbf/zipp-0.2.0-py2.py3-none-any.whl#sha256=17f0de3aea3b7d9fffb89822a8e4e92cb2f929e0bfd8a33ff08f6afbe18d457a (from https://pypi.org/simple/zipp/) (requires-python:>=2.7), version: 0.2.0
    Found link https://files.pythonhosted.org/packages/de/04/884bb7742f68215e6a7727ec256d5092068e4fa6bffab5d37ca79bb5c630/zipp-0.2.0.tar.gz#sha256=68e3b1dc4c36b36614def9a7f4fbce806bb8b1f21a77ad62c596b5533c503d22 (from https://pypi.org/simple/zipp/) (requires-python:>=2.7), version: 0.2.0
    Found link https://files.pythonhosted.org/packages/ba/5c/42991c949734405fee0b92c2de07f9eab5aae45c5c665c03ccfdd4a0d9d5/zipp-0.2.1-py2.py3-none-any.whl#sha256=dd757322c77af7be1cef4872b58c02295556f11b9a7dbb8021b5fb9e344f28a5 (from https://pypi.org/simple/zipp/) (requires-python:>=2.7), version: 0.2.1
    Found link https://files.pythonhosted.org/packages/a3/18/996efec45f7dfc7c10b7ea46f10c7e5872b67885a1e8e6eb5ddb7656c533/zipp-0.2.1.tar.gz#sha256=f06a23a7fc6d1020329f887031b76f03fd9dc0f51d02870cfc03d4cd97abbf9f (from https://pypi.org/simple/zipp/) (requires-python:>=2.7), version: 0.2.1
    Found link https://files.pythonhosted.org/packages/ff/d7/d2a8a4d4297c72b44a18b4cfccb546b8f80c0dca6794de3631f76b33ee6b/zipp-0.3.0-py2.py3-none-any.whl#sha256=1a0015bd9ff522c02629f8f1ad4395606ff0f5d86599dc6267a754763a8e6be7 (from https://pypi.org/simple/zipp/) (requires-python:>=2.7), version: 0.3.0
    Found link https://files.pythonhosted.org/packages/98/2d/5d399b6cd9c51a592df2ff9f523721512f48ccf3f4b0972be9b90c6577fa/zipp-0.3.0.tar.gz#sha256=b6d5f3e3ba5d38eb7f6f682dd697802f4ff44a394151817701b410e1b959f6c6 (from https://pypi.org/simple/zipp/) (requires-python:>=2.7), version: 0.3.0
    Found link https://files.pythonhosted.org/packages/f7/de/718b936768de59dd7b9b29e5872378b3e6b747f726c9f9f962cbfc5c35e2/zipp-0.3.1-py2.py3-none-any.whl#sha256=7ba6ae2d86276d925e30c3ecf848de7a18a6cfdf08fd0dc477d7c41a56cd13ef (from https://pypi.org/simple/zipp/) (requires-python:>=2.7), version: 0.3.1
    Found link https://files.pythonhosted.org/packages/50/e9/6a2d6160b1a5c6fe8e5d70dacdd1310d68822f5110d62f5f1d1ff1897dbb/zipp-0.3.1.tar.gz#sha256=30aeba39f049e37be6fee6ed6af9f8816a9078eac47c80a197fe64684ae04a79 (from https://pypi.org/simple/zipp/) (requires-python:>=2.7), version: 0.3.1
    Found link https://files.pythonhosted.org/packages/23/b6/2d5424acc0f4e58dbed6ff1b29e16b9767d7cfbde5373750e25ea731086f/zipp-0.3.2-py2.py3-none-any.whl#sha256=d24400b1649ccb43ee82f5fabe37546d6e0268f0beca7ed8e33987e4bbc39224 (from https://pypi.org/simple/zipp/) (requires-python:>=2.7), version: 0.3.2
    Found link https://files.pythonhosted.org/packages/dc/ea/0ee7490966af70ccfcc4172def5741a2d66f7400cedbf7ef44cba401c4a7/zipp-0.3.2.tar.gz#sha256=118501fe80ac3dbef3cfc67c0398e7e1a2eac29bb0d2122de4b2cc0dfd8dca0e (from https://pypi.org/simple/zipp/) (requires-python:>=2.7), version: 0.3.2
    Found link https://files.pythonhosted.org/packages/00/42/10a1f8d29eaf9c7f2b555a5d1d602b9261f2a0cbec08f8f23d9056f5915d/zipp-0.3.3-py2.py3-none-any.whl#sha256=682b3e1c62b7026afe24eadf6be579fb45fec54c07ea218bded8092af07a68c4 (from https://pypi.org/simple/zipp/) (requires-python:>=2.7), version: 0.3.3
    Found link https://files.pythonhosted.org/packages/0f/f4/930f91b0527d9701623d4895a978ec0abd6b8904ef272a2701190fdbf9f8/zipp-0.3.3.tar.gz#sha256=55ca87266c38af6658b84db8cfb7343cdb0bf275f93c7afaea0d8e7a209c7478 (from https://pypi.org/simple/zipp/) (requires-python:>=2.7), version: 0.3.3
    Found link https://files.pythonhosted.org/packages/93/bc/6633bc2b7bb9ded59358636966914e23f7531b2c770aee3286fd7de9bfb8/zipp-0.4.0-py2.py3-none-any.whl#sha256=139391b239594fd8b91d856bc530fbd2df0892b17dd8d98a91f018715954185f (from https://pypi.org/simple/zipp/) (requires-python:>=2.7), version: 0.4.0
    Found link https://files.pythonhosted.org/packages/b7/bc/2086e90e0ca8438414acd6e35736439ac14608361ed556b2dcda7793a8d3/zipp-0.4.0.tar.gz#sha256=8047e4575ce8d700370a3301bbfc972896a5845eb62dd535da395b86be95dfad (from https://pypi.org/simple/zipp/) (requires-python:>=2.7), version: 0.4.0
    Found link https://files.pythonhosted.org/packages/a4/cd/80a10d136899d5128ad79685ade788e5d8b48ccd4c666370d52b2a9331ab/zipp-0.5.0-py2.py3-none-any.whl#sha256=46dfd547d9ccbf8bdc26ecea52818046bb28509f12bb6a0de1cd66ab06e9a9be (from https://pypi.org/simple/zipp/) (requires-python:>=2.7), version: 0.5.0
    Found link https://files.pythonhosted.org/packages/44/65/799bbac4c284c93ce9cbe67956a3625a4e1941d580832656bea202554117/zipp-0.5.0.tar.gz#sha256=d7ac25f895fb65bff937b381353c14eb1fa23d35f40abd72a5342cd57eb57fd1 (from https://pypi.org/simple/zipp/) (requires-python:>=2.7), version: 0.5.0
    Found link https://files.pythonhosted.org/packages/a0/0f/9bf71d438d2e9d5fd0e4569ea4d1a2b6f5a524c234c6d221b494298bb4d1/zipp-0.5.1-py2.py3-none-any.whl#sha256=8c1019c6aad13642199fbe458275ad6a84907634cc9f0989877ccc4a2840139d (from https://pypi.org/simple/zipp/) (requires-python:>=2.7), version: 0.5.1
    Found link https://files.pythonhosted.org/packages/f9/c4/15a1260171956ed4f8190962b1771c7dbca4a39360c15f9c2b77e667a489/zipp-0.5.1.tar.gz#sha256=ca943a7e809cc12257001ccfb99e3563da9af99d52f261725e96dfe0f9275bc3 (from https://pypi.org/simple/zipp/) (requires-python:>=2.7), version: 0.5.1
    Found link https://files.pythonhosted.org/packages/da/bd/1a5fdf15aa44231fd09f63ecf175b60f057ae37ec65b343bb009364923f3/zipp-0.5.2-py2.py3-none-any.whl#sha256=8a5712cfd3bb4248015eb3b0b3c54a5f6ee3f2425963ef2a0125b8bc40aafaec (from https://pypi.org/simple/zipp/) (requires-python:>=2.7), version: 0.5.2
    Found link https://files.pythonhosted.org/packages/66/ae/1d6693cde3b3e3c14e95cf3408f24d0e869ead42a79993b611d8817d929a/zipp-0.5.2.tar.gz#sha256=4970c3758f4e89a7857a973b1e2a5d75bcdc47794442f2e2dd4fe8e0466e809a (from https://pypi.org/simple/zipp/) (requires-python:>=2.7), version: 0.5.2
    Found link https://files.pythonhosted.org/packages/74/3d/1ee25a26411ba0401b43c6376d2316a71addcc72ef8690b101b4ea56d76a/zipp-0.6.0-py2.py3-none-any.whl#sha256=f06903e9f1f43b12d371004b4ac7b06ab39a44adc747266928ae6debfa7b3335 (from https://pypi.org/simple/zipp/) (requires-python:>=2.7), version: 0.6.0
    Found link https://files.pythonhosted.org/packages/57/dd/585d728479d97d25aeeb9aa470d36a4ad8d0ba5610f84e14770128ce6ff7/zipp-0.6.0.tar.gz#sha256=3718b1cbcd963c7d4c5511a8240812904164b7f381b647143a89d3b98f9bcd8e (from https://pypi.org/simple/zipp/) (requires-python:>=2.7), version: 0.6.0
    Found link https://files.pythonhosted.org/packages/f4/50/cc72c5bcd48f6e98219fc4a88a5227e9e28b81637a99c49feba1d51f4d50/zipp-1.0.0-py2.py3-none-any.whl#sha256=8dda78f06bd1674bd8720df8a50bb47b6e1233c503a4eed8e7810686bde37656 (from https://pypi.org/simple/zipp/) (requires-python:>=2.7), version: 1.0.0
    Found link https://files.pythonhosted.org/packages/d4/cd/ef86396dce8910413b6ca1ef31ec09367c47e15fc1a12def2cc8ae134dea/zipp-1.0.0.tar.gz#sha256=d38fbe01bbf7a3593a32bc35a9c4453c32bc42b98c377f9bff7e9f8da157786c (from https://pypi.org/simple/zipp/) (requires-python:>=2.7), version: 1.0.0
    Found link https://files.pythonhosted.org/packages/21/21/ff0bfa221d5f5c5dc495977219c4de8f44ba8203f6724fb648113f9902cb/zipp-1.1.0-py2.py3-none-any.whl#sha256=15428d652e993b6ce86694c3cccf0d71aa7afdc6ef1807fa25a920e9444e0281 (from https://pypi.org/simple/zipp/) (requires-python:>=2.7), version: 1.1.0
    Found link https://files.pythonhosted.org/packages/c2/a1/53403f6a433049cc0d30b1304cc33cc49157c5918b87ae2d027d354e872a/zipp-1.1.0.tar.gz#sha256=d9d2efe11d3a3fb9184da550d35bd1319dc8e30a63255927c82bb42fca1f4f7c (from https://pypi.org/simple/zipp/) (requires-python:>=2.7), version: 1.1.0
    Found link https://files.pythonhosted.org/packages/45/d8/f1e05f399573d2b4723dae1c1f1487dd2e7871c79ed32da741f49faaa226/zipp-2.0.0-py3-none-any.whl#sha256=57147f6b0403b59f33fd357f169f860e031303415aeb7d04ede4839d23905ab8 (from https://pypi.org/simple/zipp/) (requires-python:>=3.6), version: 2.0.0
    Found link https://files.pythonhosted.org/packages/ee/f7/6c3ca4764a8025da9422825c2bc367fdc8dd115ebb52e4ef057f234938c4/zipp-2.0.0.tar.gz#sha256=7ae5ccaca427bafa9760ac3cd8f8c244bfc259794b5b6bb9db4dda2241575d09 (from https://pypi.org/simple/zipp/) (requires-python:>=3.6), version: 2.0.0
    Found link https://files.pythonhosted.org/packages/e1/0f/c25ab0632254f276599b664c3b9973f537a04910130321e0fb936806ff40/zipp-2.0.1-py3-none-any.whl#sha256=e013e7800f60ec4dde789ebf4e9f7a54236e4bbf5df2a1a4e20ce9e1d9609d67 (from https://pypi.org/simple/zipp/) (requires-python:>=3.6), version: 2.0.1
    Found link https://files.pythonhosted.org/packages/69/d2/daa94177e934b0b1b0780f5c856c41c248a88b2f43c046ae23afb0116ec0/zipp-2.0.1.tar.gz#sha256=b338014b9bc7102ca69e0fb96ed07215a8954d2989bc5d83658494ab2ba634af (from https://pypi.org/simple/zipp/) (requires-python:>=3.6), version: 2.0.1
    Found link https://files.pythonhosted.org/packages/be/69/4ac28bf238f287f1677f41392e24d2c4ffafcf11648c23824f5f62ef6ccb/zipp-2.1.0-py3-none-any.whl#sha256=ccc94ed0909b58ffe34430ea5451f07bc0c76467d7081619a454bf5c98b89e28 (from https://pypi.org/simple/zipp/) (requires-python:>=3.6), version: 2.1.0
    Found link https://files.pythonhosted.org/packages/11/b5/89f3ab6d45b2709863761bab58c574b2344ef215749abb5407818c21c9ca/zipp-2.1.0.tar.gz#sha256=feae2f18633c32fc71f2de629bfb3bd3c9325cd4419642b1f1da42ee488d9b98 (from https://pypi.org/simple/zipp/) (requires-python:>=3.6), version: 2.1.0
  Given no hashes to check 34 links for project 'zipp': discarding no candidates
  Using version 2.1.0 (newest of versions: 0.1.0, 0.2.0, 0.2.1, 0.3.0, 0.3.1, 0.3.2, 0.3.3, 0.4.0, 0.5.0, 0.5.1, 0.5.2, 0.6.0, 1.0.0, 1.1.0, 2.0.0, 2.0.1, 2.1.0)
  Created temporary directory: /private/var/folders/c6/v7hnmq453xb6p2dbz1gqc6rr0000gn/T/pip-unpack-0nan5l_b
  Looking up "https://files.pythonhosted.org/packages/be/69/4ac28bf238f287f1677f41392e24d2c4ffafcf11648c23824f5f62ef6ccb/zipp-2.1.0-py3-none-any.whl" in the cache
  Current age based on date: 734056
  Ignoring unknown cache-control directive: immutable
  Freshness lifetime from max-age: 365000000
  The response is "fresh", returning cached response
  365000000 > 734056
  Using cached https://files.pythonhosted.org/packages/be/69/4ac28bf238f287f1677f41392e24d2c4ffafcf11648c23824f5f62ef6ccb/zipp-2.1.0-py3-none-any.whl
  Downloading from URL https://files.pythonhosted.org/packages/be/69/4ac28bf238f287f1677f41392e24d2c4ffafcf11648c23824f5f62ef6ccb/zipp-2.1.0-py3-none-any.whl#sha256=ccc94ed0909b58ffe34430ea5451f07bc0c76467d7081619a454bf5c98b89e28 (from https://pypi.org/simple/zipp/) (requires-python:>=3.6)
  Added zipp from https://files.pythonhosted.org/packages/be/69/4ac28bf238f287f1677f41392e24d2c4ffafcf11648c23824f5f62ef6ccb/zipp-2.1.0-py3-none-any.whl#sha256=ccc94ed0909b58ffe34430ea5451f07bc0c76467d7081619a454bf5c98b89e28 (from foo==1.0) to build tracker '/private/var/folders/c6/v7hnmq453xb6p2dbz1gqc6rr0000gn/T/pip-req-tracker-yusw2kug'
  Removed zipp from https://files.pythonhosted.org/packages/be/69/4ac28bf238f287f1677f41392e24d2c4ffafcf11648c23824f5f62ef6ccb/zipp-2.1.0-py3-none-any.whl#sha256=ccc94ed0909b58ffe34430ea5451f07bc0c76467d7081619a454bf5c98b89e28 (from foo==1.0) from build tracker '/private/var/folders/c6/v7hnmq453xb6p2dbz1gqc6rr0000gn/T/pip-req-tracker-yusw2kug'
Installing collected packages: zipp, foo

  Created temporary directory: /private/var/folders/c6/v7hnmq453xb6p2dbz1gqc6rr0000gn/T/pip-record-lgf6mvxd
    Running command /Users/jaraco/draft/env/bin/python -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/private/var/folders/c6/v7hnmq453xb6p2dbz1gqc6rr0000gn/T/pip-req-build-zo5rbfpg/setup.py'"'"'; __file__='"'"'/private/var/folders/c6/v7hnmq453xb6p2dbz1gqc6rr0000gn/T/pip-req-build-zo5rbfpg/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' install --record /private/var/folders/c6/v7hnmq453xb6p2dbz1gqc6rr0000gn/T/pip-record-lgf6mvxd/install-record.txt --single-version-externally-managed --compile --install-headers /Users/jaraco/draft/env/include/site/python3.8/foo
    running install
    running build
    running install_egg_info
    running egg_info
    writing foo.egg-info/PKG-INFO
    writing dependency_links to foo.egg-info/dependency_links.txt
    writing requirements to foo.egg-info/requires.txt
    writing top-level names to foo.egg-info/top_level.txt
    reading manifest file 'foo.egg-info/SOURCES.txt'
    writing manifest file 'foo.egg-info/SOURCES.txt'
    Copying foo.egg-info to /Users/jaraco/draft/env/lib/python3.8/site-packages/foo-1.0-py3.8.egg-info
    running install_scripts
    writing list of installed files to '/private/var/folders/c6/v7hnmq453xb6p2dbz1gqc6rr0000gn/T/pip-record-lgf6mvxd/install-record.txt'
  Running setup.py install for foo ... done
  Removing source in /private/var/folders/c6/v7hnmq453xb6p2dbz1gqc6rr0000gn/T/pip-req-build-zo5rbfpg
Successfully installed foo-1.0 zipp-2.1.0
Cleaning up...
Removed build tracker '/private/var/folders/c6/v7hnmq453xb6p2dbz1gqc6rr0000gn/T/pip-req-tracker-yusw2kug'
WARNING: You are using pip version 19.2.3, however version 20.0.2 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.

No matter what I do, the package seems to be resolved from PyPI. What else do I need to do?

Feel free to re-open if you can help me understand why the suggested change is beneficial.


How can we get the discussion of merge request using gitlab4j-api

Use the NotesApi.
GitLabApi gitLabApi = new GitLabApi("http://your.gitlab.server.com", "YOUR_PRIVATE_TOKEN");
List<Note> mergeRequestComments =
    gitLabApi.getNotesApi().getMergeRequestNotes(projectId, mergeRequestIid);

Make sure to use the merge request IID.

Thanks sir, I was using 4.6.0 version so not able to find

@ricky001
I should have let you know that this functionality was just made available in 4.8.3.


Assignment 4

Hi Lina, I don't see any changes to the lab 4 files. Could you double check to make sure you have committed your changes?

Hi Danil,
Thanks for your email. I am sorry I didn't see this email last night. Can I check my computer tonight cause I am in the internship now.
There must be something wrong when I committed the changes, I did the the assignment 4 actually.
Hope it would not interfere with my overall grade and thanks for your reminder and understanding.
Best,
Lina

Hi Danil,
Thanks for your email. I am sorry I didn't see this email last night. Can I
check my computer tonight cause I am in the internship now.
There must be something wrong when I committed the changes, I did the the
assignment 4 actually.
Hope it would not interfere with my overall grade and thanks for your
reminder and understanding.
Best,
Lina
On Thu, Oct 20, 2016 at 4:54 PM, Danil Nagy<EMAIL_ADDRESS>wrote:

Hi Lina, I don't see any changes to the lab 4 files. Could you double
check to make sure you have committed your changes?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dmc-2016/dmc/pull/87#issuecomment-255225376, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AVEOr4cEGchmqqHRaXoJQJMgukci1gehks5q19UMgaJpZM4KVJLF
.

--
Lina Yin
Master of Science in Urban Planning
Degree Candidate, 2017
Graduate School of Architecture, Planning & Preservation || Columbia
116th Street and Broadway
New York, NY 10027
917-536-8780
<EMAIL_ADDRESS>


Make data set/append helpers

just the next candle or adding another 20 candles onto the end, same thing just appending to the graph), avoiding the need for a recompile each time I want to add a data point.

Big subject, imo. Exposing append & prepend methods would be super useful !
https://github.com/C451/trading-vue-js/blob/8df612c90dac0bde521e08a17afd03f87c13c819/src/components/Chart.vue#L223
Partial rendering of the candles & indicators etc...
Originally posted by @Tucsky in https://github.com/C451/trading-vue-js/issues/7#issuecomment-485255757

An extended sample that uses ccxt as a candle source would be a nifty real-life async demonstration of this:
https://github.com/ccxt/ccxt#javascript-for-use-with-the-script-tag
Each range-stretch could trigger a REST API request to fill that missing gap.

Yep, I got an idea. I'm not sure about including a third party API, but a simple callback will certainly do the job.

If someone is still looking for a solution i got it working by adding an @mouseup event on a div that wraps the vue component like so.
<div @mouseup="onChartMouseRelease" class="chartDiv">
<trading-vue
        :data="chartContainer"
        :width="this.width"
        :height="this.height"
        ref="tradingVue">
</trading-vue>
</div>

the event fires off this method
onChartMouseRelease(event) {
      console.log("mouse released.");
      this.rangeCheck();
    },

which then checks if more data needs to be fetched
rangeCheck: function() {
      var range = this.$refs.tradingVue.getRange();
      if (range[0] < this.chartContainer.chart.data[0][0]) {
        console.log("get more data..");
        this.fetchMoreData();
      }
    }

and finally get the data somehow and unshift it to the chart data array
fetchMoreData: function() {
      var startTs = this.chartContainer.chart.data[0][0];
      var tf = this.getCurrentTfInt();
      axios
        .get(
          BASE_URL +
            "/Home/GetPreviousMarketData?timeStamp=" +
            startTs +
            "&timeFrame=" +
            tf +
            "&count=100"
        )
        .then(res => {
          console.log(JSON.stringify(res.data.data));
          this.chartContainer.chart.data.unshift(...res.data.data);
        })
        .catch(err => console.log(err));
    }

Nice! Hopefully, DataCube will make this process easier. Currently WIP.

Hi @C451
When you fix this issue?
Thanks

DataCube

How to use DataCube?

See the progress: https://github.com/C451/trading-vue-js/blob/master/src/helpers/datacube.js
API is undocumented for now.

Hi @C451
when do you update document for new api?

Timing is the hardest part. Now the data helper is unstable/experimental, the API may change. When you see at least half of the checkmarks filled, you can expect some docs: https://github.com/C451/trading-vue-js/projects/1

Hi @C451
I like this feature.
How to use this library on NuxtJS?

Hi @thenewspacesky1990, unfortunately I haven't tried it myself yet. Are there any requirements for that?

Really good. I'm waiting for this function
When will you finish?
Thanks

@learntolearn99 I think you can

https://github.com/C451/trading-vue-js/blob/master/test/tests/DataHelper.vue
An example of connection to an exchange. (Unstable, there may be errors)


🌎: Update Italian Translation

Summary
translation update + new transaltions
Change Type

[x] Translation update

Checklist

[x] My code adheres to this project's style guidelines
[x] I have performed a self-review of my own code
[x] I have commented in any complex areas of my code
[x] I have made pertinent documentation changes
[x] My changes do not introduce new warnings
[x] I have written tests demonstrating that my changes are effective or that my feature works
[x] Local unit tests pass with my changes
[x] Any changes dependent on mine have been merged and published in downstream modules.

thank you!


chore: fix cargo bench -- --save-baseline

Inspired by https://github.com/influxdata/influxdb_iox/pull/5400.

Thank you!


chore: adding startup delay to handle topology changes gracefully by configuring startup delay

won't this cause rebalance in consumer group?
cc: @laxmanchekka

won't this cause rebalance in consumer group? cc: @laxmanchekka

I have tested it on dev sandbox. Seems to work fine for trace enricher.


KC-1745:  Upgrade jettty, Jersey, commons-codec.

This is to sync CVE fixes from https://github.com/confluentinc/kafka/pull/565.
This will be merged to 5.5.x ce-kafka.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Task :support-metrics-client:allDepInsight
commons-codec:commons-codec:1.11
variant "runtime" [
org.gradle.status          = release (not requested)
org.gradle.usage           = java-runtime (not requested)
org.gradle.libraryelements = jar (not requested)
org.gradle.category        = library (not requested)
]

commons-codec:commons-codec:1.11
--- org.apache.httpcomponents:httpclient:4.5.13
+--- runtime
+--- project :support-metrics-common
|    --- runtime
--- org.apache.httpcomponents:httpmime:4.5.13
+--- runtime
--- project :support-metrics-common (*)
(*) - dependencies omitted (listed previously)
A web-based, searchable dependency report is available by adding the --scan option.

Task :support-metrics-common:allDepInsight
commons-codec:commons-codec:1.11
variant "runtime" [
org.gradle.status          = release (not requested)
org.gradle.usage           = java-runtime (not requested)
org.gradle.libraryelements = jar (not requested)
org.gradle.category        = library (not requested)
]

commons-codec:commons-codec:1.11
--- org.apache.httpcomponents:httpclient:4.5.13
+--- runtime
--- org.apache.httpcomponents:httpmime:4.5.13
--- runtime
(*) - dependencies omitted (listed previously)


pagination-plugin and fetch options on count fix

Related Issues: #1913

Introduction
Make the pagination plugin fully compatible with others bookshelf plugins
Motivation
In my project, I use the plugin bookshelf-paranoia to use the soft delete feature. I have a problem with pagination.
If I want to show soft deleted items using pagination, the fetchPaginate function return correctly the models but don't return correct count.
After resarch, it appears the count part don't use passed fetch options except the transacting parameter.
Proposed solution
I propose to use others fetch options on count part omit all default fetch params (withRelated, ...) to avoid problems

I hope I make correct tests, I try if the count lost standard options and keep custom options

Nice work! I'll review this properly soon.

Great !! Have you a release date for this PR integration in bookshelf ? I realy need this in my current development

I will most likely make it to the next version (0.14.0) which will be released soon.


Confidence Interval calculation

Reading through the plot code, I noticed that you used the x_pts_to_graph as the p argument of the binomial distribution in
https://github.com/numeristical/introspective/blob/2aedbb433f15d845a71e5f0e87207ec1e58494e4/ml_insights/modeling_utils.py#L642
Shouldn't the estimated probability (fraction of positives) be used instead, y_pts_to_graph?
I guess in this case the choice is irrelevant given that both have the same values, correct?

Hi Lourenco,
Thanks for your comment.  The error bars are drawn from a "hypothesis test" point of view.  That is, assuming the null hypothesis of perfect calibration, what is the range of values we would expect to see in the empirical probability.  So, assuming that the predicted probabilities (x) are well-calibrated, we ask what range of values would we expect to see for y (the observed fraction of 1s in the bin).  Under that assumption, Y should be (1/n) * Binomial(n, x), where n is the number of points in the bin. This way, one can view each bin as a hypothesis test, where a point outside the error bars represents a rejection of the null for that bin.
There is another (minor) issue, in that we are ignoring that the probabilities of each trial vary slightly across the bin.  We treat each trial as having the same probability x, whereas in reality, they have slightly different probabilities which average out to x.  The consequence is that the error bars we plot are actually slightly wider than they would be if we simulated the "coin flips" to have the slightly different probabilities.
Does this make sense?  It's possible I'm missing something.

Hi Brian.
Under your assumptions, your solution seems correct to me. Thank you for clearing this out.
Then, I guess I am working with other assumptions, hence the proposed changes. I have the same "hypothesis test" but with the assumption that each bin has probability p=fraction_of_positives. And then compute the 95% confidence interval for that hypothesis.
You can see the resulting error bars in the following plot.

Yes, it is reasonable to look at it from that point of view as well.  Essentially, you are creating a confidence interval around the "true probability" in the bin.
In terms of the code, I think the best idea is to have the option to do it either way.  Perhaps we add an argument called ci_ref (confidence interval reference) with options axis and point to choose whether you want the confidence interval around the existing way (axis) or your proposed way (point).  Does that sound reasonable?
Would you be interested in working on this? :)

Yes, sure I can contribute on this :)

Great!  I assigned it to you. :)


Impossible to seek within a live item when using Chromecast

Version
Media3 1.1.0
More version details
N/A
Devices that reproduce the issue

Samsung Galaxy A32 running Android 13

However, should be reproduced on any other device.
Devices that do not reproduce the issue
N/A
Reproducible in the demo app?
Yes
Reproduction steps

Add the attached HLS livestream to cast demo media sources
Start playing it using local ExoPlayer
Start casting the media

Expected result
The TimeBar in the UI has a draggable playhead that allows to seek backwards (as when playing this content locally without casting)
Actual result
The TimeBar has no draggable playhead, so it's impossible to seek backwards
Media
https://cph-msl.akamaized.net/hls/live/2000341/test/master.m3u8
Bug Report

[ ] You will email the zip file produced by adb bugreport to<EMAIL_ADDRESS>after filing this issue.

bumping in case this got lost

This is not supported with CastPlayer.  We are aware this is a missing feature. There is a pull request #475 to map the cast live window to the timeline.
This is currently not on the roadmap I'm afraid. We keep this issue open and update when there is news.


Add 'px' for svg.style('height')

Otherwise the height property won't work in Firefox and the plot will be cropped.

Ah, nice. I think this closes #88.

Thank you!


feat: adding the option to disable ttl once it was enable before

what
The change aims to be able to disable the TTL once a dynamodb table was enabled before.
why

With the current dynamic block on the TTL option inside the aws_dynamodb_table resource if you change the var.ttl_enabled = false there are no changes on the infrastructure and the TTL is not disabled:
No changes. Your infrastructure matches the configuration.
With this change you can disable the TTL on a dynamodb table that had it enabled before following this process:

Change the var.ttl_enabled = false Do the plan and apply
Change later the var.ttl_attribute="" or leave it empty to consolidate the state. If you don't do this second change your plan is going to say something like this and if you try to apply you will have an error...

 ~ resource "aws_dynamodb_table" "default" {
        id                          = "test"
        name                        = "test"
        tags                        = {
            ...
        }
        # (9 unchanged attributes hidden)

      ~ ttl {
          + attribute_name = "ttl"
            # (1 unchanged attribute hidden)
        }

        # (3 unchanged blocks hidden)
    }

Plan: 0 to add, 1 to change, 0 to destroy.

Do you want to perform these actions in workspace "test"?
  Terraform will perform the actions described above.
  Only 'yes' will be accepted to approve.

  Enter a value: yes

module.dynamodb["test"].module.dynamodb_table.aws_dynamodb_table.default[0]: Modifying... [id=test]
╷
│ Error: updating Amazon DynamoDB Table (test): updating Time To Live: ValidationException: TimeToLive is already disabled
│ 	status code: 400, request id: XXXXXXXXXXXXXXXXXXXXXX
│
│   with module.dynamodb["test"].module.dynamodb_table.aws_dynamodb_table.default[0],
│   on .terraform/modules/dynamodb.dynamodb_table/main.tf line 46, in resource "aws_dynamodb_table" "default":
│   46: resource "aws_dynamodb_table" "default" {
│

references

This is the origin of the issue https://github.com/hashicorp/terraform-provider-aws/issues/10304

Hi @joe-niland @Gowiem, this PR is open for some months ago, Do you have any comments?
thanks in advance

Hi @joe-niland @Gowiem, any additional requirements to merge this PR?
Many thanks!

@alfredo-gil can you update the readme (see the failing check)

@alfredo-gil and also please update your branch from master


Optimization summary incorrect

Optimization summary statements show average derivative time instead of average forward time and vice versa:
Example:
Optimization terminated successfully    (Exit mode 0)
Final objective value                : 6.598148e-11
Final optimality                     : 1.319630e-10
Final feasibility                    : 4.840572e-14
Number of major iterations           : 5
Number of function evaluations       : 5
Number of derivative evaluations     : 5
Average Derivative evaluation time   : 0.001741 s per evaluation
Average Function evaluation time     : 0.060738 s per evaluation
Total Function evaluation time       : 0.008703 s [  0.08%]
Total Derivative evaluation time     : 0.303692 s [  2.76%]
Optimizer time                       : 10.675094 s [ 96.94%]
Processing time                      : 0.024357 s [  0.22%]
Visualization time                   : 0.000000 s [  0.00%]
Total optimization time              : 11.011846 s [100.00%]

Thanks for raising the issue. The issue was fixed in the GitHub repo. However, it won't reflect on PyPI until the next patch release.

The issue is corrected in the latest release on PyPI.