Web - Triple clicking last message select’s random text from entire page

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:

Launch the app -> Login in account
Navigate to any conversation
Triple click last message
Try to copy from (cmd + c)
Paste on composer
Notice pasted text comes along with additional unselected text

Expected Result:
Only selected text should be pasted
Actual Result:
Additional unselected text pasted
Workaround:
unknown
Platforms:

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

[ ] MacOS / Chrome / Safari
[ ] Mac / Desktop

Version Number: 1.2.76-3
Reproducible in staging?: y
Reproducible in production?: n
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:
https://user-images.githubusercontent.com/43996225/220964006-3859341c-ab86-4a40-9eb7-36e8b4bd83a6.mov
https://user-images.githubusercontent.com/43996225/220964101-66d2c6f4-87db-4471-90f5-97f120778149.mp4
Expensify/Expensify Issue URL:
Issue reported by: @dhairyasenjaliya
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1677122727940939
View all open jobs on GitHub

Triggered auto assignment to @abekkala (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

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

Here's a revert PR which fixes this https://github.com/Expensify/App/pull/15417

Thanks, merged!

@abekkala I'm reopening this so that we can pay @dhairyasenjaliya his reporting bonus.

I've sent the $250 Reporting Bonus contract to @dhairyasenjaliya

Paid! @dhairyasenjaliya
Thanks!

Reviewing label has been removed, please complete the "BugZero Checklist".

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

[ ] [@puneetlath] The PR that introduced the bug has been identified. Link to the PR:
[ ] [@puneetlath] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:
[ ] [@puneetlath] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:
[ ] [@puneetlath] Propose regression test steps to ensure the same bug will not reach production again.
[ ] [@puneetlath / @abekkala] Link the GH issue for creating/updating the regression test once above steps have been agreed upon:

@neil-marcellini am I also eligible for compensation for helping finding the root cause quickly? Maybe same amount as reporting bonus? Here's context.

I certainly appreciate that you found the root cause quickly, but I don't think we pay bonuses for that.


Backpacked breaks replaymod recordings taken in singleplayer

Mod Name
Replaymod & Backpacked (Forge)
Mod Homepage
https://www.replaymod.com & https://www.curseforge.com/minecraft/mc-mods/backpacked
Minecraft version
1.20.1
Describe the bug
"Joining world..." overlay stuck on replay when backpacked is installed
Steps to reproduce

Install Replaymod and Backpacked (Forge).
Record a replay.
Enter the replay.

Logs
https://gist.github.com/itsactuallyezra/02e04d159df82292108027e54a2c84e1
Additional context
Installed mods

connector-1.0.0-beta.44+1.20.1.jar
fabric-api-0.92.2+1.11.8+1.20.1.jar
backpacked-forge-1.20.1-2.2.5.jar
framework-forge-1.20.1-0.7.6.jar (backpacked dependency)
replaymod-1.20.1-2.6.13.jar

When the Backpacked mod is installed, singleplayer records are broken. The "Joining world..." overlay never goes away, preventing you from interacting with anything on the screen. You cannot even exit the replay, you have to force quit Minecraft. You don't have to equip anything from the Backpacked mod, it simply has to exist in the modpack. If the recording was taken from a multiplayer server, there are no issues. This bug only exists in replays taken in singleplayer.
This bug doesn't exist in a purely fabric environment (Replaymod x Backpacked (Fabric)). I've also tried the Fabric version of Backpacked with Sinytra, however it crashes the game.
https://github.com/user-attachments/assets/e0d7240a-2427-4293-9fb8-5dfa090720ad

Duplicate of #1244, seems like this got opened twice accidentally


Create a docs site

Even though this library isn't done, writing docs can be very informative. It will help us figure out where we have a coherent story and where we don't. We can also put in placeholders for things we know we need to implement, to sketch out the API.

I got us started with a GitBook! https://apollostack.gitbooks.io/apollo-client/content/
Didn't write anything useful yet, and there appears to be some weird comment-related error. Will figure out more soon.

Cool, it works now! Let's get some docs going!

I got some basic docs on the page, going to close this in favor of more specific issues.


FBE: Kotlin wait for fix with UInt

https://youtrack.jetbrains.com/issue/KT-28921

Fixed in Kotlin 1.3.21


firefox: Apply a IndexDB Patch (backport)

backport of https://github.com/NixOS/nixpkgs/pull/77126
I tried the fix from https://bugzilla.mozilla.org/show_bug.cgi?id=1601707
but it didn‘t solve my problem https://github.com/vector-im/riot-web/issues/11711
Therefore I instead applied
https://salsa.debian.org/mozilla-team/firefox/commit/e888ce1578d68116949782035f8149cdbc5e832c
which is supposedly a patch for the same problem.
I admit I don‘t know why this helps. I just tried it out because I
heard that the bug does not exist under debian.
cc @andir

IIUC this bug is fixed in 72.0, which was released today? In that case, it makes sense to just bump firefox and not apply this patch.

I have now used a patch provided by mozilla instead of the one from debian. This is apparently applied in firefox 74, so this is just a backport of an upstream fix.

Fixes #76899

Superseeded by #77292.


Add mutating versions of functions that reuse external storage for sigma points and weights

Hey! Thanks for this amazing package. Would it be possible to add mutating version of all functions that would support and reuse external storage for sigma points and weights instead of allocating a new one each time, e.g:
n = 100

sigmap  = Vector{Float64}(undef, n)
weights = Vector{Float64}(undef, n)

alpha = 1.0

gausslaguerre!(sigmap, weights, n, alpha)

Usually it is a good practice in Julia to have such functions, e.g. we have map!, filter! etc.
The benefit of this would be improved performance in cases where you need continuously recompute sigma points and weights for different alpha parameter for example (for gausslaguerre). Non-mutating versions would be simply:
# Float64 here is just an example, maybe support type for storage as well?
gausslaguerre(n, alpha) = gausslaguerre!(Vector{Float64}(undef, n), Vector{Float64}(undef, n), n, alpha)

We would be happy to accept a PR for this.

👍 I'll try to find time to implement this feature and return to you if I need help.

@dlfivefifty I started to implement this feature for gausslaguerre and noticed that some of your tests are incorrect. E.g. this one: https://github.com/JuliaApproximation/FastGaussQuadrature.jl/blob/7b256f3b10f66e42caea5f38c4bd8593a18c96b8/test/test_gausslaguerre.jl#L82
It should be
@test abs(x_gw[37] - 98.388267163326702) < tol

Notice different x and x_gw. Do you want me to open another issue or it can be a part of my PR for in-place mutating functions. I'm not sure though how long it would take me to go through your codebase and add mutating versions for all of your functions.
P.S. fixed tests do pass

Update Gauss-Laguerre looks some-what like this:
https://github.com/bvdmitri/FastGaussQuadrature.jl/commit/3146d9d6858a18e3011342a0a9a15d6f667d3e04
I also added more place for external storage re-usage and as a result we save some allocations:
function test()
     dummy = 0.0
     for n in 1:1000
         x, w = gausslaguerre(n)
         dummy += sum(w)
     end
     return dummy
 end

# master
julia> @btime test()
  762.632 ms (54755 allocations: 12.85 MiB)
999.9999999999991
# after update
julia> @btime test()
  744.996 ms (53507 allocations: 12.60 MiB)

It is also interesting to see how much do we safe with in-place versions:
function test_reuse(x, w)
    dummy = 0.0
    for n in 1:1000
        x, w = FastGaussQuadrature.gausslaguerre!(x, w, n)
        dummy += sum(view(w, 1:n))
    end
    return dummy
end

julia> @btime test_reuse(s[1], s[2]) setup=(s=(Vector{Float64}(undef, 1000), Vector{Float64}(undef, 1000)))
  743.897 ms (51507 allocations: 4.69 MiB) # third of the original allocations
999.9999999999991

I'm not sure where do this remaining allocations come from. Probably there are some other places that I overlooked. It looks that saved extra allocations do not contribute much in the overall performance, but it may affect performance in larger programs with a lot of GC pressure I suppose.

The code is mostly a Matlab port so there’s a lot of room food improvement


Clean up Data backup command

The restore procedure was incorrect. Simplifying the command to work with the restore.

Proposed changes
The restore procedure was incorrect. Simplifying the command to work with the restore.
Unreleased project version (optional)

Related issues (optional)

Deploy preview for docsdocker ready!
Built with commit ff38e269b2267adeeba8aacd57370139f6f6bb25
https://deploy-preview-7436--docsdocker.netlify.com

Approving with @bermudezmt


Bundling and superclassing

Two disadvantages of type-level-list-based effect systems are that they do not support

Bundling, as in type MonadStateWriter s m = (MonadState s m, MonadWriter s m)
Superclassing, as in class MonadGet r m => MonadReader r m

Some possibilities of solving this:
-- A pseudo effect representing a bundle of several effects
data Bundle (es :: [Effect]) :: Effect

-- This will stuck if the user uses a polymorphic effect type
-- Most of the times they don't, but it is possible
type family e :> es :: Constraint where
  Bundle es :> xs = es :>> xs
  e :> xs = (Elem e xs, Super e xs)

-- A possibly better way is to just modify the (:>) typeclass
class e :> es where
instance {-# INCOHERENT #-} xs :>> es => Bundle xs :> es
instance Elem e es => e :> es

class Super e es => Elem e es

-- Users can define custom instances of this typeclass
-- to override the default behavior (no superclasses)
class Super e xs
instance {-# OVERLAPPABLE #-} Super e xs

Some initial tests show that while superclassing is easy to add as a nonbreaking change, bundling is largely breaking, bringing worse ergonomics.

Closed; wontfix


tye run runs the incorrect build version.

Description
I attempted to use tye run against an existing sample solution of mine which I recently upgraded to .NET 5 from 3.1. The solution contains a front end application (Razor Pages) which calls a backend API service, similar to the sample.
tye run attempts to run the incorrect version.
To Reproduce
I updated the front end to use the service discovery configuration.
When I execute tye run, the API project starts successfully. The web front end fails repeatedly (see output below).
The build produces output in the bin\Debug\netcoreapp5.0
The run step, however, seems to pick up an older build from the bin\Debug\netcoreapp3.0 folder, which exists from historical builds.
dotnet "E:\Projects\Pluralsight\Worker Services Course\Tye Playground\src\TennisBookings.Web\bin\Debug\netcoreapp3.0\TennisBookings.Web.dll"
Full Output
dotnet build "E:\Projects\Pluralsight\Worker Services Course\Tye Playground\src\TennisBookings.Web\TennisBookings.Web.csproj" /nologo
Restore completed in 70.83 ms for E:\Projects\Pluralsight\Worker Services Course\Tye Playground\src\TennisBookings.Web\TennisBookings.Web.csproj.
You are using a preview version of .NET Core. See: https://aka.ms/dotnet-core-preview
TennisBookings.Web -> E:\Projects\Pluralsight\Worker Services Course\Tye Playground\src\TennisBookings.Web\bin\Debug\netcoreapp5.0\TennisBookings.Web.dll
TennisBookings.Web -> E:\Projects\Pluralsight\Worker Services Course\Tye Playground\src\TennisBookings.Web\bin\Debug\netcoreapp5.0\TennisBookings.Web.Views.dll
Build succeeded.
0 Warning(s)
0 Error(s)
Time Elapsed 00:00:01.55
[tennisbookings.web_3504bce3-8]:dotnet "E:\Projects\Pluralsight\Worker Services Course\Tye Playground\src\TennisBookings.Web\bin\Debug\netcoreapp3.0\TennisBookings.Web.dll"
[tennisbookings.web_3504bce3-8]: Could not execute because the specified command or file was not found.
[tennisbookings.web_3504bce3-8]: Possible reasons for this include:
[tennisbookings.web_3504bce3-8]: * You misspelled a built-in dotnet command.
[tennisbookings.web_3504bce3-8]: * You intended to execute a .NET Core program, but dotnet-E:\Projects\Pluralsight\Worker Services Course\Tye Playground\src\TennisBookings.Web\bin\Debug\netcoreapp3.0\TennisBookings.Web.dll does not exist.
[tennisbookings.web_3504bce3-8]: * You intended to run a global tool, but a dotnet-prefixed executable with this name could not be found on the PATH.

Workaround
If I manually delete all bin folders and run tye run it works as expected.
The API worked without this but did NOT contain a netcoreapp3.0 folder in bin\Debug. It does contain netcoreapp2.2 and netcoreapp3.1 which seem to be ignored.
Further technical details
tye --version = 0.1.0-alpha.20168.8+f45b9444d4894009bde48e2a3411a52dd09497b1

Platform = Windows

This is fixed in the latest builds https://github.com/dotnet/tye/issues/184. Not multi-target projects are not supported.

Also if we want to, we could support multi-targeting by adding a property. It's the kind of thing we should make the user's problem instead of guessing.

Also if we want to, we could support multi-targeting by adding a property. It's the kind of thing we should make the user's problem instead of guessing.

Right, there should be a  --framework switch as well as a --configuration switch for https://github.com/dotnet/tye/issues/90

Closing this as fixed. I'll open issues for the other features


re-style the search page

Currently has the default item (just title). Use old JSX files.

Fixed by @zzacharo


Add ruby 3.0 support

Related github issue for this PR
Resolves #001
Checklist

[ ] I have performed a self-review of my own code
[ ] I added comments, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation README.md
[ ] I added tests that prove my fix is effective or that my feature works
[ ] bundle exec rake passes locally
[ ] rubocop gives no error locally
[ ] My PR title includes "WIP" or is draft PR if work is in progress

Description

Type of change

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not
work as expected)
Documentation update

Which tests are written for this PR? And what are the expected results for these tests?

Codecov Report

Merging #118 (4828269) into master (828dff9) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #118   +/-   ##
=======================================
  Coverage   99.69%   99.69%
=======================================
  Files          14       14
  Lines         651      651
=======================================
  Hits          649      649
  Misses          2        2

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


Add support for autodetecting URLs and making hyperlinks

This pull request is the initial implementation of hyperlink auto
detection
Overall design:

Upon startup, TerminalCore gives the TextBuffer some patterns it
should know about
Whenever something in the viewport changes (i.e. text
output/scrolling), TerminalControl tells TerminalCore (through a
throttled function for performance) to retrieve the visible pattern
locations from the TextBuffer
When the renderer encounters a region that is associated with a
pattern, it paints that region differently

References #5001
Closes #574

Regarding accessibility. I haven't had a chance to test this out yet. Especially since it's in draft. But here's some resources and an idea of the work we'll probably have to do (probably in a separate PR):

UWP ITextChildProvider API
ITextChildProvider Guide
Hyperlink Control Type

In UiaTextRange, we'll need to update...

GetChildren(): hyperlink ranges count as a child
GetEnclosingElement(): if we're on a hyperlink, return the parent text range. Otherwise, normal behavior.
FindAttribute() and GetAttribute(): hyperlinks need to have the hyperlink attribute (exposed via GetAttribute()). And FindAttribute() needs to be implemented to find if one exists in the text range.

Another thought. Hyperlink support is just for Windows Terminal, not ConHost, right? That'll complicate things a bit with our accessibility model because UiaTextRange is currently shared. I see the implementation for autodetection is in TextBuffer (which makes this easier), but if ConHost won't expose it, we need to figure out a way to not expose hyperlinks that don't exist in ConHost. Of course, if this functionality is shared, just completely ignore this paragraph haha.

Hyperlink support is just for Windows Terminal, not ConHost, right?

If this is the plan, might this be reconsidered? While Windows Terminal is very feature-filled, in my experience conhost has much wider usage.

@codeofdusk, @carlos-zamora Hyperlink support is Terminal-FIRST, not Terminal-ONLY. Sorry for the confusion.
Since we can move so much faster in Terminal it lets us figure out the interaction model a lot sooner than if we throw something together and hope that people jump on the Windows Insider builds and test our thing out.
Many, many changes that make it into Terminal will eventually flow back. This is especially true for anything that deals with accessibility, standards compliance and text rendering.
We'd like to figure out the interaction model for formatted text and browseable regions in Terminal before we bring anything like this back into conhost, absolutely.

Now I don't want to go too crazypants here, but I'm thinking about future extensions to some degree.
I'm curious if you considered a design where the text buffer figures out the patterns on its own instead of being pumped to do that action by a thread from way outside itself, in this case from up in the Terminal* and friends.
I'm not saying this one is bad or wrong, but an alternate vision I have is that you would register an object with the TextBuffer that contained both a pattern, region of interest, and a callback method/event inside of it. The TextBuffer would just... of its own accord as things are written in on the other methods (and maybe with its own internal, private thread or threadpool worker) grind through the region with the pattern and make callbacks as necessary.
Almost like you have an ITextScanner interface with an ITextMatchFound callback interface... which might lend itself toward easy adaptation into a COM/WinRT-based plug-in model for arbitrary buffer scanners in the future.

Now I don't want to go too crazypants here, but I'm thinking about future extensions to some degree.
I'm curious if you considered a design where the text buffer figures out the patterns on its own instead of being pumped to do that action by a thread from way outside itself, in this case from up in the Terminal* and friends.
I'm not saying this one is bad or wrong, but an alternate vision I have is that you would register an object with the TextBuffer that contained a pattern, region of interest, and a callback method/event inside of it. The TextBuffer would just... of its own accord as things are written in on the other methods (and maybe with its own internal, private thread or threadpool worker) grind through the region with the pattern and make callbacks as necessary.
Almost like you have an ITextScanner interface with an ITextMatchFound callback interface... which might lend itself toward easy adaptation into a COM/WinRT-based plug-in model for arbitrary buffer scanners in the future.

As Dustin and I are musing about this... your ITextScanner could just also contain ITextEffect on it as an optional in lieu of a callback that is just handed straight into the renderer and now you've also managed to design rendering effect extensions at the same time...

Now I don't want to go too crazypants here, but I'm thinking about future extensions to some degree.
I'm curious if you considered a design where the text buffer figures out the patterns on its own instead of being pumped to do that action by a thread from way outside itself, in this case from up in the Terminal* and friends.
I'm not saying this one is bad or wrong, but an alternate vision I have is that you would register an object with the TextBuffer that contained a pattern, region of interest, and a callback method/event inside of it. The TextBuffer would just... of its own accord as things are written in on the other methods (and maybe with its own internal, private thread or threadpool worker) grind through the region with the pattern and make callbacks as necessary.
Almost like you have an ITextScanner interface with an ITextMatchFound callback interface... which might lend itself toward easy adaptation into a COM/WinRT-based plug-in model for arbitrary buffer scanners in the future.

As Dustin and I are musing about this... your ITextScanner could just also contain ITextEffect on it as an optional in lieu of a callback that is just handed straight into the renderer and now you've also managed to design rendering effect extensions at the same time...

OK we all discussed this with Teams. I'm willing to concede this point because Hyperlinks are so interesting here that they push the boundaries of this sort of unified model. And where the threads live is a bit contentious. And Dustin keeps coming up with more ideas why, while this model sort of works, it has sharp edges. So I'm fine with proceeding per what this draft has as a design.

Oh maybe the regex is a little too forgiving...

Oh maybe the regex is a little too forgiving...

Oh yeah it definitely is. Waiting on some licensing stuff to get sorted out before @miniksa is able to provide a better one - till then I figured might as well have this be ready for review since the regex change is just a 1 line change anyway

Alrighty then, my block was over the OSS thing, so I'm gonna remove it since that's resolved. I'll hold off approving till the regex is a little more robust though 😉

this feels so right
it's in selfhost for the team now

When we resize the buffer, it looks like we drop the patterns. Another thing to copy in the Reflow method!

oh no
I'm sure that it's not because of this branch, but if I ctrl+click the link that was auto-created, it does open in the browser, but the terminal hangs afterwards:

Bad news! A hyperlink that starts at 0,0 always appears hovered.

I've got an interesting use case that triggers some minor badness!
powershell
"http://howett.net`n"*10;sleep 1;"`e[10A" + ("x x x x x x x xxxxxx`n"*10)

the lines clear up when they're forced to re-render, which suggests that we're missing an invalidation

(This is adapted from a real-world use case, which is me using IRC in a text-mode IRC client)

@DHowett both issues fixed now!

SO! When I get a link at the top of the screen using the mouse wheel, and then use the scroll bar... something odd happens.

In general, scrollbar interactions seem to cause the URLs to break or get otherwise messed up.

SO! When I get a link at the top of the screen using the mouse wheel, and then use the scroll bar... something odd happens.

Yeah we were only clearing/updating the pattern tree on mouse scrolling, changed it to update on either mouse or scroll bar scrolling

I edited your PR body down to be a nice commit body :)

I guess there is something wrong with handling wide characters.

Scott's wish finally became true: https://youtu.be/veqs2WVou9M?t=797

I love this feature!
I have a question, would it be possible to have some "visual feedback", that we're hovering on a link element? Like with websites, as in a:hover. It could for example underline the link element, give it a background or anything if is possible to make it customizable.

I guess that would be another issue

@xeho91 today it underlines on hover. Is that not working for you?

@xeho91 today it underlines on hover. Is that not working for you?

It does. However, in the example when I am using bat to preview my *.md files. It has a syntax which already recognizes and underlines the links in the output. There's a quick preview below.

When I am hovering with the cursor, there's no "visual feedback", except this small dialogue box "Ctrl + Click to follow link.".
What I am asking for to give us ability to define the style effect in settings for hovering on links. Like make the background white, and black text. If that's possible, no pressure, I still love this feature. ❤

@xeho91 I've tossed this request into #5001, since this isn't actually the first time this has come up 😉 Thanks!

I'd like to use this ability to open an html file stored on a remote computer using the UNC path. I haven't been able to get it to open. Any ideas?
Example: \COMPUTER\UNC\Path\To\File.html


Hubbard model nits

This was bothering me when mypy complained that $t$ or $u$ (not $\mu$ as $u$ was called as a variable) were floats (which they are in general). Also uses $u$ as in the paper which for some reason doesn't use the capital $U$ hubbard U.

Ok I ended up fixing the hubbard model notebook too to use the autogen feature but appended the resource estimation to the end.

fixes #929


Wrong xref for petunidin and petunidin chloride

Hi
CHEBI:75318 (petunidin) has a xref to KEGG:C08727 that is Petunidin chloride. I think it should not be. No Cl in CHEBI:75318.
KEGG:C08727 is already a xref to CHEBI:75035 that is really petunidin chloride.
CHEBI:75035 also has a xref to HMDB:HMDB0003173, but I guess, according to its structural information, it should be linked to CHEBI:75318. No Cl in structural information of HMDB:HMDB0003173.
I guess the Wikipedia xref should also only go to CHEBI:75318

Now fixed. Thanks


Fix #1042, remove use of osapi-os-loader.h

Describe the contribution
Remove use of osapi-os-loader.h from ES UT.  This is the only reference to OSAL header other than the standard osapi.h and common_types.h (which are staying).
Fixes #1042
Testing performed
Build and run unit tests.
Expected behavior changes
None.  This header was redundant since it was already including osapi.h which has everything.
System(s) tested on
Ubuntu 20.04
Contributor Info - All information REQUIRED for consideration of pull request
Joseph Hickey, Vantage Systems, Inc.

CCB 2020-12-09 APPROVED

CCB 2020-12-09 APPROVED

Added dependency label since nasa/osal#675 breaks without this when OMIT_DEPRECATED=true


Add meta passing in header

@maralla

Ehh I reverted this PR since it merged directly to the master branch..
Please fire another one to develop branch.


Edit page link fallback not working

What version of starlight are you using?
0.11.0
What version of astro are you using?
3.0.6
What package manager are you using?
pnpm
What operating system are you using?
Windows
What browser are you using?
Brave
Describe the Bug
When clicking "Edit page" at the bottom of the page (when enabled) but the page is not available in the current language (the fallback page is currently shown instead), then the link does not use a link to GitHub to the fallback page but to a page that does not exists, therefore leading to this page:

I at least encountered this problem, when English is not the default local...
Link to Minimal Reproducible Example
https://starlight.astro.build/de/guides/overriding-components/
Participation

[ ] I am willing to submit a pull request for this issue.

Ah, good point! I wonder what the correct behaviour is.

Link to edit the default locale file being used for fallback content
Link to create a new file for the current language
Hide the edit link on pages using fallback content

Link to create a new file for the current language

This is something I did in a project by overriding the <EditLink/> component when isFallback is true and I changed the sentence to "Translate this page" and the link uses the filename query parameter to specify which file to create (the value one is not really usable to pre-fill the file content).
I personally like that behavior a lot.

Yes, I think I agree that would also be the ideal behaviour although it would require some more config for a create-page base URL, and a new string in our translation dictionaries.
Given the increasing config complexity, it may also be nice to provide some presets for common hosts to avoid everyone having to figure out the base URLs when it can be done based on repo + branch etc.

I personally only thought about behaviour 1 (Link to edit the default locale file being used for fallback content)...
Therefore, I would say, this should be the default behaviour, but maybe the behaviour could also be chosen by the developer in astro.config.mjx...
Maybe there could even be multiple buttons on the bottom: "Edit this page", "Create page in this language"...

Maybe it would be smart to split this into two steps:

Fix the current behaviour to point to the existing file

Treat support for a "Translate this page" link as a feature request


Drop support for JDK 14.

Coming from https://github.com/opensearch-project/opensearch-plugins/issues/132, remove support for JDK 14.
In OpenSearch 2.0 we are building the complete distribution and every component with JDK 11, and testing and bundling JDK 17. Support for JDK 14, 15 and other non-LTS versions can be dropped. Remove building and testing with these JDKs from CI.
See https://opensearch.org/blog/technical-post/2022/03/opensearch-java-runtime/ for more information.

closed by https://github.com/opensearch-project/observability/pull/607/


Add basic test for the plugin

Fixes #2

Would you be so nice @andrewvc to review this? Thanks a lot.

LGTM (but it needs to be rebased it looks like)


Drat: Add test for Decrement, Increment and View

Codecov Report

Merging #257 (700ce4e) into master (2df0af1) will increase coverage by 4.54%.
The diff coverage is 80.00%.

@@             Coverage Diff              @@
##             master     #257      +/-   ##
============================================
+ Coverage     68.82%   73.37%   +4.54%
- Complexity      571      600      +29
============================================
  Files           112      112
  Lines          1957     1957
  Branches        207      212       +5
============================================
+ Hits           1347     1436      +89
+ Misses          535      440      -95
- Partials         75       81       +6

Impacted Files
Coverage Δ

src/main/java/seedu/foodrem/model/item/Item.java
81.81% <ø> (-2.80%)
:arrow_down:

.../java/seedu/foodrem/model/util/SampleDataUtil.java
0.00% <ø> (ø)

...odrem/logic/commands/itemcommands/ViewCommand.java
92.30% <50.00%> (+83.21%)
:arrow_up:

.../logic/commands/itemcommands/DecrementCommand.java
90.62% <83.33%> (+87.05%)
:arrow_up:

.../logic/commands/itemcommands/IncrementCommand.java
90.62% <83.33%> (+87.05%)
:arrow_up:

...gic/parser/itemcommandparser/NewCommandParser.java
100.00% <100.00%> (ø)

...del/item/itemvalidators/ItemQuantityValidator.java
93.33% <0.00%> (+6.66%)
:arrow_up:

...in/java/seedu/foodrem/model/item/ItemQuantity.java
85.71% <0.00%> (+14.28%)
:arrow_up:

... and 4 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


cmdstan 2.17.0 unable to compile the program after 'make build' in windows 10

Summary:
I had some issues but finally were able to build the cmdstan-2.17.0 on windows 10. Now I am trying to run the first example, bernoulli but without any success
Description:
After a while I was able to get:
--- CmdStan v2.17.0 built ---
But when I am trying to run the bernoulli example I got the following message:
C:\cmdstan-2.17.0>make examples/bernoulli/bernoulli.exe
uname: not found
uname: not found
make: Nothing to be done for `examples/bernoulli/bernoulli.exe'.
Reproducible Steps:
I modified the extension of the file 'bernoulli.stan' on the examples folder to 'bernoulli.exe' according to the guide on page 10.
As I mentioned before I am trying to run the bernoulli example but I got:
make: Nothing to be done for `examples/bernoulli/bernoulli.exe'.
Current Output:
make: Nothing to be done for `examples/bernoulli/bernoulli.exe'.
Expected Output:
I am supposed to generate a C++ code bernoulli.hpp and the compiled executable.
Additional Information:
Provide any additional information here.
Current Version:
v2.17.0

@cmelog, I now understand your confusion.

change the name of the file from bernoulli.exe to bernoulli.stan
then type make examples/bernoulli/bernouli.exe

Please let me know if that works.

Hi, I changed the file to bernoulli.stan and got:
C:\cmdstan-2.17.0>make examples/bernoulli/bernouli.exe
uname: not found
uname: not found
make: *** No rule to make target `examples/bernoulli/bernouli.exe'.  Stop.
I am now running Stan from Mathlab and is working, which was my original goal. I am not sure if I should also be able to run on cmdstan

Hello all,
I'm facing the same issue No rule to make target `C:/Users/EVerma/cmdstan/examples/bernoulli/bernoulli.exe'
How did you resolve this?
Thanks

Hi this is what I did:

Installed the newest version of Rtools.
On the path I included: C:\Rtools\mingw_64\bin to select the adequate version of the Rtools.
I created a local file (no extension)  in the C:\cmdstan-2.17.1\make with the following lines:
CC=g++

I hope this helps.

What is meant by local file (no extension)?
Thanks in advance

Just a file inside the make directory called “local”: should be “make/local” without an extension. Hopefully that adds a little clarification.
If you wanted to generate the file, you could type:
echo CC=g++ > make/local
And that’ll create the file appropriately.

On Mar 8, 2018, at 4:17 AM, Ekansh Verma<EMAIL_ADDRESS>wrote:
What is meant by local file (no extension)?
Thanks in advance
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.

Hi,
I have exactly the same problem. Adding the local file did not resolve the problem.
FYI I am working on Windows 7. I attached an snapshot from cmd

What's the problem? It says CmdStan 2.17.1 has been built. The uname lines
are just extraneous output. (I haven't been able to suppress it
successfully)
On Tue, May 8, 2018 at 9:58 AM, Majeed7<EMAIL_ADDRESS>wrote:

Hi,
I have exactly the same problem. Adding the local file did not resolve the
problem.
FYI I am working on Windows 7. I attached an snapshot from cmd
[image: cmdstan]
https://user-images.githubusercontent.com/13550419/39761541-abc3f21a-52d8-11e8-96ec-1ee9ef27ca6a.jpg
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/cmdstan/issues/588#issuecomment-387411811,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAZ_F8p6FGDjJ2XgwH9EQYTgBnsJNtoDks5twaSggaJpZM4Q7I8K
.


Update to v0.1.2 - Add ValueHolder

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/52151
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.2 -m "<description of version>" 67638dec523b5d0bb724415b43509738149fce70
git push origin v0.1.2


Add support for camel-hazelcast

We should probably not do this, because WildFly comes natively with Infinispan, which is covered by #76


safely render html descriptions

Sometimes descriptions contain simple HTML such as links (see example below). We can actually the description string as an html element. Typically this can be a security vulnerability if handled incorrectly.
Read https://reactjs.org/docs/dom-elements.html#dangerouslysetinnerhtml for more on this. Also read https://medium.com/better-programming/what-is-dangerouslysetinnerhtml-6d6a98cbc187 to see how to implement this

/sound/464889 is an example sound that would have noticeable changes in its description field.


Feature Request: add images based on the last update

I have consistently issues updating from the last image file to the latest update. I have to retry 3-4 times, it usually locks up at the end of the process, when you have to press a button to restart. Restarting manually corrupts the system and it doesn't boot anymore.
Can you add an updated image ready to flash?

Which unit are you having this issue with?

RG351V

It happened with the penultimate update as well

Sounds like a problematic sd card or you're battery may have been low on charge as I can't replicate this issue on my end.

Ok, I'll close it... I'll try again with different cards


feat: add more bios & photos from Adela's mail, requested layout changes

Closes #18
@adeladelrio Please let me know if this is the desired layout for the steering committee page.

In the advisory board, Wil van der Aalst is missing (see https://bpm-conference.org/organization/).
I would also remove the "Assistants to the steering committeee" section. Just the steering committee and advisory board


Database for event management

Should contain an event keyword and a timespan in which the event took place.

GUI

[ ] Add event to database

date-picker tkinter

[x] Add multiple events to database
[x] Clear database
[x] Remove event from database
[ ] Modify images with event X?
[x] Dump events to json file
[x] Show all events

Funktionality

[x] Add event to database
[x] Add multiple events to database
[x] Clear database
[x] Remove event from database
[ ] Modify images with event X?
[x] Dump events to json file
[x] Get a event using a date
[x] Get all saved events

Dump events to json file - commit: 8c4dc9b629d20dd3b1d550bbac339eb5ccf033ae

Add event to database - commit: 0cc581d7fbef9a349b813112bb7511fa229dfeee

Outsourced task to Issue #15.
Everything else is done - closed.


Fix logout not working

From Django 5, logging out requires a CSRF token and so the logout option needs to be enclosed within a form and include the {% csrf_token %}. The current structure just did not allow for this to be implemented easily, so I've re-write entirely that part of the menu, giving also a bit more of flexibility.
Fixes #184

@dalonsoa Just merged the latest changes from main and the same test fail is back!

Ok, it now works. I'm going to merge before it fails again :P


Set up a github page to host a demo version of the site

TODO

This is done on #17 .  Both are done in one swoop.


Consider replacing kitsu.js with kitsu

kitsu.js is deprecated in favour of the more feature complete kitsu package that is fully JSON-API compliant.
Search is implemented with:
import Kitsu from 'kitsu'

const api = new Kitsu()

api.fetch('anime', {
  filter: {
    text: 'Cowboy Bebop'
  }
})

Sure, I don't have time to edit anime. I'll give the task to the developers, thanks!


[Feature]: Consider extending add_bounds() to override existing bounds?

Is your feature request related to a problem?
Per @lee1043 in our 3/15/23 meeting, bounds in the dataset might not always be correct and sometimes users want to override the bounds.
For example, monthly time bounds should be at the beginning and end of the month. If we have a time coordinate (2000, 1, 1):
# Time coordinate with incorrect time bounds
(2000, 1 , 1)  -> [(1999, 12, 16), (2000, 1, 16)]

# Time coordinate with correct time bounds
(2000, 1 , 1)  -> [(2000, 1, 1), (2000, 1, 31)]

Some datasets might use the incorrect time bounds which can throw off analysis results.
Describe the solution you'd like
Possible Solutions:

The user drops existing bounds manually and adds xCDAT bounds

ds = xcdat.open_mfdataset("path")

# User needs to know the name of the time bounds to drop. If they are looping over many datasets, this might be tricky because the name can vary. Users can probably write a function with `cf_xarray` to do this.
ds = ds.drop_vars("time_bnds")

# Option 1
ds = ds.bounds.add_bounds(axis="T")
# Option 2
ds = ds.bounds.add_missing_bounds(axis="T")

# With both options, the frequency for the time bounds is inferred (e.g., set monthly time bounds if xCDAT infers monthly time coordinates) pending PR #418 and #434

Extend add_bounds() with an override boolean attribute.

If override=False -> raises KeyError if bounds exist
If override=True -> override existing bounds for coordinates related to an axis

Describe alternatives you've considered
No response
Additional context
Related to #418 and #434

Just opening this issue to have it in our backlog for further consideration

@tomvothecoder As now we can do like this (no specification of axis): ds = ds.bounds.add_missing_bounds()
How about extending add_bounds() with an override boolean attribute with below options:

default: override=False -> add missed bounds, but without raising key error if bounds exist
If override=True -> override existing bounds for coordinates related to an axis

Just weighing in that I usually lean toward convenience, but in this case I'm not so sure. It could lead users to systematically ignore the bounds that the dataset creator took the time to add. Could this be justified with some analysis (e.g., when we drop bounds, xcdat generally replaces the bounds with the correct values AND in some cases it actually leads to better bounds)?

For users who analyze individual datasets at a time, override=True might be a good option if they have the knowledge that the bounds in their datasets are not as "good" as the ones produced by xCDAT.
On the other hand, for package developers, override=True might cause issues because xCDAT will always override existing bounds for whatever dataset(s) passed regardless if the existing bounds are already "correct". Some package devs might use override=True without understanding the consequences of this assumption in their APIs. I think this is related to what you're saying by "it could lead users to systematically ignore the bounds that the dataset creator took the time to add."

@pochedls @lee1043 do you guys typically use bounds from the dataset as the default option in most cases (if they exist) or mainly use xCDAT's add bounds/add missing bounds APIs.
My thought was that the dataset bounds should be the higher priority for most cases.

My regular code flow in the PMP is like below. As I am dealing with multiple models knowing some models may have their bounds info missed, I use add_missing_bounds for all models to fill the missed ones if there is any.
for model in models:
  # open
  ds = xc.open_dataset(model_files)
  ds = ds.bounds.add_missing_bounds()
  # Then analyze
  ....

@lee1043 – this will add the missing bounds, but this ticket is advocating for dropping the dataset bounds and then having xcdat trying to infer them.

Oh,,,, @pochedls thanks for clarifying, I must have confused it with the add_missing_bounds! Sorry for the confusion. What I suggested here is about the add_missing_bounds, not add_bounds.

Oh,,,, @pochedls thanks for clarifying, I must have confused it with the add_missing_bounds! Sorry for the confusion. What I suggested here is about the add_missing_bounds, not add_bounds.

I think the same scenarios and questions from the comments above will apply to add_missing_bounds() too.
My understanding is that any existing bounds take priority over xCDAT generated bounds. If the existing bounds are incorrect for some datasets, it would be a data quality issue that should probably be addressed explicitly by the user rather than systematically dropping and replacing bounds using override=True (especially to many datasets in a for loop).
If package devs use add_missing_bounds(override=True) in their APIs, users might not be aware of this implicit behavior going on behind the scenes which might be a bit dangerous.

@tomvothecoder agreed, thank you for clarifying!

@tomvothecoder I am okay to close the issue, thank you for the discussion.

I know this is closed, but if there were obvious cases where we should drop a dataset's bounds and re-generate them it would be useful to document them (in case we wanted to consider an override feature in the future).

@pochedls Sure, we can re-open this issue to gather ideas. If we don't have any substantial reason to have this feature, then we can re-close it.

I know this is closed, but if there were obvious cases where we should drop a dataset's bounds and re-generate them it would be useful to document them (in case we wanted to consider an override feature in the future).

I think I have had such cases that I wanted to drop and regenerate bounds systematically for collective operation across different models because some models' bounds information was not correctly generated. @tomvothecoder I like the idea of drop_bounds() and as @pochedls said we should document that in the API doc for the drop_bounds(). How about that?

I was suggesting documenting the examples where this is an issue (before beginning work on this). I think I've seen this in FGOALS, but I think it is pretty rare. If you are routinely dropping bounds and re-generating them, it would be helpful to demonstrate that you fix more problems than you introduce. If you loop over 50 models, do you reproduce the correct bounds for the vast majority of models that are fine (while fixing the rare model that has bad bounds)?

Below is one example case from CMIP5, ACCESS 1-0 model, whose time bounds are not aligned with time step.
input_file = "ts_Amon_ACCESS1-0_historical_r1i1p1_185001-200512.nc"
ds = xc.open_dataset(input_file)
ds["time"]
ds["time_bnds"]

In such case, maybe something like drop_time_bounds() and add_time_bounds() might be helpful if exist.
If I loop over 50 models, I might reproduce the correct bounds for the vast majority of models that are fine while fixing the rare model that has bad bounds. But I feel that would be more efficient than identifying the bad ones each time that happens, which I think is not rare.
My workaround considering such cases is like this:
def reset_time_bounds(ds, freq="month"):
    time_bnds_key = get_time_bounds_key(ds)
    return ds.drop_vars([time_bnds_key]).bounds.add_time_bounds("freq", freq=freq)

Note: get_time_bounds_key()

Just following up after some time of inactivity in this ticket.

If I loop over 50 models, I might reproduce the correct bounds for the vast majority of models that are fine while fixing the rare model that has bad bounds. But I feel that would be more efficient than identifying the bad ones each time that happens, which I think is not rare.

@pochedls @lee1043 Is there a simple, programmatic way of identifying which models have bad bounds and only dropping and regenerating bounds for those models? It seems more computationally efficient and "correct" compared to dropping bounds for all models regardless if the bounds are correct or not (even if only a few are bad), then regenerating them.

I'm not sure...I think suspicious bounds could be identified, but in the example @lee1043 gives above – I'm pretty sure they are wrong, but I don't know they are wrong. I don't really care if people systematically remove/regenerate the bounds as long as it is an option that is off by default (i.e., requires the user to do something to produce this behavior).

Just following up after some time of inactivity in this ticket.

If I loop over 50 models, I might reproduce the correct bounds for the vast majority of models that are fine while fixing the rare model that has bad bounds. But I feel that would be more efficient than identifying the bad ones each time that happens, which I think is not rare.

@pochedls @lee1043 Is there a simple, programmatic way of identifying which models have bad bounds and only dropping and regenerating bounds for those models? It seems more computationally efficient and "correct" compared to dropping bounds for all models regardless if the bounds are correct or not (even if only a few are bad), then regenerating them.

@tomvothecoder I don't know any programmatic way of identifying the issue other than looking at time bounds of each model individually.

If both of you think this type API would be generally useful to the broader user community, I'm happy to add it to the backlog as a lower priority enhancement.

@tomvothecoder thank you for giving a thought. That would be definitely useful for PMP, although I don't have a good idea how much broader users would need (or even aware of the issue -- which in this case, having that capability would be helpful).

This seems like a pretty risk-free feature to me. We could add a regenerate flag if we wanted to regenerate the bounds using xcdat. This also seems like a perfect dev day task. Maybe we should label this as "good first issue" or something?

This seems like a pretty risk-free feature to me. We could add a regenerate flag if we wanted to regenerate the bounds using xcdat. This also seems like a perfect dev day task. Maybe we should label this as "good first issue" or something?

The logical flow of drop_bounds() with a regenerate type flag makes more sense to me than add_bounds() with an override flag.
I updated the title of this issue and tagged it with "good first issue" .

I agree, like the regenerate idea.


Generic verision match lookup workflow

Often, lookup vars that match the full version is not feasible. Instead, a generic version lookup logic can be introduced:

first lookup vars from the full version match (as it is done today) - e.g. 23.3.3
if not found, lookup vars from the major version - 23.3
if not found, lookup vars from the year release - 23
if not found, lookup vars from the default location - e.g. default/vars.yml

yes, i looked at that. As it turns out, it's not so easy to do that in Ansible (checking the presence of a file). You need a bunch of tasks to accomplish that per role.


Minor grid changes

Changed

moved the aggregate applications menu under applications
Changed the grid's default filter to be today

👍


Wheel support for linux aarch64[arm64]

Summary
Installing newrelic on aarch64 via pip using command "pip3 install newrelic" tries to build wheel from source code
Problem description
newrelic doesn't have wheel for aarch64 on PyPI repository. So, while installing newrelic via pip on aarch64, pip builds wheel for same resulting in it takes more time to install newrelic. Making wheel available for aarch64 will benefit aarch64 users by minimizing newrelic installation time.
Expected Output
Pip should be able to download newrelic wheel from PyPI repository rather than building it from source code.
@newrelic-team, please let me know if I can help you building wheel/uploading to PyPI repository. I am curious to make newrelic wheel available for aarch64. It will be a great opportunity for me to work with you.

A few questions regarding this!
The wheels are compiling native wrapt extensions and certain internal c extensions.

Wrapt doesn't appear to test on arm (see travis. Additionally, we don't test our native c extensions with arm. Do you think we'll encounter problems if we compile and ship arm binaries?
We use github actions to deploy our wheels. Does github actions provide an arm runner to build the binaries?
Our github flows use the setup-python action which downloads binaries from here. I don't see aarch64 binaries there. Do you know if there are plans to support aarch64 there?

@a-feld thanks for your quick response. Please find my answers inline-

Wrapt doesn't appear to test on arm (see travis). Additionally, we don't test our native c extensions with arm. Do you think we'll encounter problems if we compile and ship arm binaries?

I ran Wrapt travis-ci job on arm64 for python3.7 and it ran successful. I think we wouldn't encounter any problems

We use github actions to deploy our wheels. Does github actions provide an arm runner to build the binaries?

We can use qemu to run arm64 builds on x86_64 machine

Our github flows use the setup-python action which downloads binaries from here. I don't see aarch64 binaries there. Do you know if there are plans to support aarch64 there?

I will work on making setup-python binaries available for arm64.


[stable/kube2iam] Add liveness probe

Hey - I recently had a problem where kube2iam pods went unresponsive and ec2 metadata requests were left hanging - restarting the failed pods resolved the issue.
I'd like to get Kubernetes to do that for me so have added a liveness check to use the /healthz resource, introduced in https://github.com/jtblin/kube2iam/issues/26
Thanks for taking a look!

/lgtm


Make it works with UUID

As discussed on a precedent PR about identifier parsing, admin-on-rest doesn't support UUID as entities identifier.

What is the issue with UUID identifiers? I am trying to use UUIDs and I haven't found any problem yet other than the prop type warning on Datagrid.

It'll not work with a ReferenceManyField because of a parseInt in its rendering.

Fixed by #67


Added Food Expiration Dates

I added my app, built with Jetpack Compose, to the list.

I added my app, built with Jetpack Compose, to the list.

Thanks for your contribution. 🦾


kinetics-600

Can you please add support for kinetics-600? The resources/classes and categories need to be updated. I was wondering if you had a script that automatically generated these files.

@asimkadav Yes, I will look into it. Thanks for the suggestion.
Best,
Ondrej


ssm/send-command

Purpose is to have an action which results in a call to ssm:SendCommand with the ability to pass DocumentName so scripts can be executed on these ec2 instances based on filters on the ec2 resource.
Potential usage example: in reaction to mode type: ec2-instance-state where events == running, the SendCommand is invoked with DocumentName=
"Name": "AmazonInspector-ManageAWSAgent" to install Inspector and/or
"Name": "AWS-RunInspecChecks" to run checks

another interesting usage for memory acquisition on compromised server via sssm command https://github.com/ThreatResponse/ssm-acquire


Renaming rhapsody -> mmvec

Here, we are renaming all instances of rhapsody to mmvec to reduce confusion (for those wondering, see this twitter post).
We are going to push for Version 1 with the Tensorflow dependencies due to the tight deadline for the manuscript.  See checklist here

Tests are passing, and the tutorials are returning sane results.  Merging.


PhpUnit test with DunglasAngularCsrfBundle

I'm trying to set up DunglasAngularCsrfBundle with phpunit api test, and when user login token not created, if user login by http_auth token created but generated token in test not equal
http://stackoverflow.com/questions/34989606/how-to-test-symfony2-api-with-dunglasangularcsrfbundle

Solved self::$client->request('GET', '/') set token in cookie


Automator: update proxy@master in istio/istio@master

Generated by Automator - 2021-05-25T21:16:49+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-helm-tests_istio
59a4df675e55e70b43adab45517e2004578bb3a0
link
/test integ-helm-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-helm-tests_istio
59a4df675e55e70b43adab45517e2004578bb3a0
link
/test integ-helm-tests_istio

integ-security-k8s-tests_istio
59a4df675e55e70b43adab45517e2004578bb3a0
link
/test integ-security-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-multicluster-tests_istio
d64e6acdc08e8cf4b308ba295e06b324d3a3b98c
link
/test integ-pilot-multicluster-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-security-multicluster-tests_istio
d64e6acdc08e8cf4b308ba295e06b324d3a3b98c
link
/test integ-security-multicluster-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-security-multicluster-tests_istio
d64e6acdc08e8cf4b308ba295e06b324d3a3b98c
link
/test integ-security-multicluster-tests_istio

integ-pilot-multicluster-tests_istio
d64e6acdc08e8cf4b308ba295e06b324d3a3b98c
link
/test integ-pilot-multicluster-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-multicluster-tests_istio
d64e6acdc08e8cf4b308ba295e06b324d3a3b98c
link
/test integ-pilot-multicluster-tests_istio

integ-security-multicluster-tests_istio
c4d764114bbaa805566b7316cc382bdef46ca487
link
/test integ-security-multicluster-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Ability to create tags while writing/editing posts

Currently the only way to create new tags is to go to the tags section and create them there.
It would be a big time saver to simply be able to type them into the tags section of the Settings > Metadata panel when writing or editing a post and have them saved that way. Similar to as WordPress...but better...because it's not WordPress :)

See https://github.com/Postleaf/postleaf/issues/40#issuecomment-298473436

Ok. Well if it's a case where they need extra config why not have an "Add Tag" button on the Metadata panel which opens a modal where the user can properly configure the new Tag.
Or remove the ability to type at all from the metadata tags section and replace it with a select box and some more info about what tags are in Postleaf.
At the moment it's misleading being able to type only to realize they don't save after you save the post.

I have a feeling this will come up more often than I expected. I've reopened #40 for voting, so let's continue the discussion there.


Build EOS failed on MAC

Scanning dependencies of target eosio-applesedemo
[ 87%] Building CXX object plugins/http_plugin/CMakeFiles/http_plugin.dir/http_plugin.cpp.o
[ 87%] Building CXX object programs/eosio-applesedemo/CMakeFiles/eosio-applesedemo.dir/main.cpp.o
/Users/ws/Documents/workspace/git/eos/programs/eosio-applesedemo/main.cpp:37:13: warning: unused variable 'ret' [-Wunused-variable]
OSStatus ret = SecItemCopyMatching(keyAttrDic, (CFTypeRef*)&attributes);
^
1 warning generated.
[ 87%] Building CXX object programs/eosio-applesedemo/CMakeFiles/eosio-applesedemo.dir/r1_signature_compactor.cpp.o
[ 87%] Linking CXX executable eosio-applesedemo
ld: framework not found security
clang-4.0: error: linker command failed with exit code 1 (use -v to see invocation)
make[2]: *** [programs/eosio-applesedemo/eosio-applesedemo] Error 1
make[1]: *** [programs/eosio-applesedemo/CMakeFiles/eosio-applesedemo.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
[ 87%] Linking CXX static library libhttp_plugin.a
[ 87%] Built target http_plugin
make: *** [all] Error 2
>>>>>>>>>>>>>>>>>>>> MAKE building EOSIO has exited with the above error.

On branch master

Maybe it's because my disk is case-sensitive.
Change "path-to-eos/programs/eosio-applesedemo/CMakeLists.txt"
"-framework security -framework corefoundation"
to
"-framework Security -framework CoreFoundation"
will resolve this problem.

@ericiles note this issue, perhaps we should configure our CI system for OSX/Darwin to use a case-sensitive file system so these issues are spotted automatically.


Change directory stracture.

Change

src to Source.

Create

Create Test directory.
Create System directory under Source directory.
Create Mvvm directory under Source directory.

Move each project to new directories.


Resolve the issue #1511

Pull Request Type
What kind of change does this PR introduce?
[X] Solved an issue/task
[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:
What is the current behavior?
Currently read_csv.c file under tasks/C/medium directory requires code to read contents in the CSV file.
Issue Number: https://github.com/fork-commit-merge/fork-commit-merge/issues/1511
What is the new behavior?
The code which is added will read the CSV file and displays the contents in a human readable format.

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out our forkcommitmerge.io website repository. The repository contain real issues that you can help to resolve or create some of your own if you see any bugs or want to add new features to the site, etc. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


feat(pr-merge): create PR comment with workflow execution results after merge

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


Windows 10: Menubar items blurry

VSCode Version: 0.10.11
OS Version: Windows 10 64bit latest(all updates installed)

Steps to Reproduce:

Just open the editor and the menubar texts are blurry.

The items each menu triggers are ok.
Could this be because of the engine(Chromium) that Visual Studio Code shares with Atom editor(Atom is havig the same issue)?!

bpasero, I think it's not fixable in VSCode. Those who are interested into the details, may take a look at the discussion for Atom:
https://github.com/atom/atom/issues/11442
IMHO, it's a pity simple things like this typography issue are plaguing these wonderful new tools.
We came to an agreement to close the Atom's issue for now - do you think we should do it for VSCode as well?!

I am closing this because I believe this is fixed, otherwise please report back. If you can reproduce I would appreciate if you could try to reproduce with our insiders build:
You can give our preview releases a try from: http://code.visualstudio.com/Download#insiders

@kostadinnm since you voted down, you still see this? I am not being notified if you react like that, you gotta talk.

Of course I am, Benjamin. I guess, it should be visible elsewhere utilizing DirectWrite's sub-pixel positioning. Had you even tried follow the conversation with damieng on the Atom thread?!
The thing is it's just not clear who's acting up. Clear my mind - was VSCode Chromium-based?! I need to find some other programs that utilize DirectWrite's sub-pixel positioning and compare the fonts' rendering. Still, since I'd written it wouldn't be fixable from VSCode, I must have read sth about the real blur-maker.
Anyways, telling us you believe it's fixed - I'm not ok. It's just that - I expressed a feeling. In the Atom thread we made some conversation. Here, I pointed towards the constructive discussion and we are getting a "robotish" bloaty answer. Simply closing the issue would've been better.

@kostadinnm I apologise, I went over a lot of issues where I was asking the people to give our insider release a try and somehow I missed to ask this here. We updated VSCode 1.6.0 to a much newer Electron version which bundles Chrome 52. Can you give that one a try to see if the issue is fixed or not?
You can give our preview releases a try from: http://code.visualstudio.com/Download#insiders
Also please keep in mind that you seem to be the only one having this issue on Windows 10 as I have not heard back from anyone else and we have many 100k users on Windows 10 I believe.

No worries.
Here's what I get from the VSCode insiders 1.6.0(are you able to spot the difference between the two 'i'-symbols):

It's not a surprise to me, as with the latest chrome browser has the same output in its settings.
Cheers,
Kostadin

Looks like I am getting subpixel anti aliasing on my Windows 10. You say you get grayscale anti aliasing?

Actually you get subpixel anti aliasing but your result looks different compared to mine (which may not be surprising depending on the monitor hardware).

Mhm, I hadn't known that matters as well. I have four monitors I could test with. Two of which(some old Belinea 1945SW and one LG Monitor TV M2352) are nearby. It must be monitor-hardware inability on those. The other two are Asus ProArt PA246Q and BenQ BL3200PT.
I'll try my best to get them tested as well. I'm strictly using digital connections(DVI, HDMI) and RDP-ing between the machines(font-smoothing thicked), but I doubt the latter matters. Plus, VSCode is pretty lightweight and I may install it even on my weakest machine.
Cheers,
Kostadin

Wait, Benjamin, it's not my monitor hardware. Your screen shot has at least three different 'i'-symbols as well. It's a whole other issue described in this article:
The price of smooth looks: Anti-aliased fonts hurt eyes and damage eyesight
So, it's a generic OS-issue(to me). In fact, I haven't tested myself like it's described in the article(how long a person is able to read without feeling tired eyes), but the fact that I am able to detect it in my full hd screen makes me crazy.
And, just to make it all clear, since chromium is a web rendering engine sticking to the trends(allowing fancy fonts) in conjunction with the latest OS which assumes people are all using 4k monitors leads to that cramped small fonts feel.

@kostadinnm but you are saying you only have this issue with the menu bar items not inside VS Code right?

No, it's not in the menu only. Look at those 'i'-symbols:

In Atom and Google Chrome settings it's the same. I am getting really distracted by this.
Even the github-form that I am writing this comment from is using the same method(although a different browser) - what a downfall :(
Here's a petition in that regards:
Bring sharp fonts back in Google Chrome (Disable DirectWrite)
But, it's not only Google involved, the OS-development is also there - I found complains about not being able to disable ClearType on modern Windows and its equivalent on OS X.

Ok, I am taking my words back about the github-form and partially on the issue:
-I wasn't aware that font anti-aliasing is not working that well on small fonts. No anti-aliasing isn't an option as well - I tried it and its worse.
-The github-form is at least consistent across the symbols-rendering. Some seem blurry on my .285 pixel pitch monitor(the lowest dpi one I have, but shouldn't be any different on the highest, which has pixel pitch of 0.27), others are crisp. For example, the 'i' and the 'l' -symbols are pretty readable, while the 'm' has some little distortion in the middle and the 'u' is hard to get on the right. There are combinations of hw/sw anti-aliasing, font-size, symbol overlapping, etc that designers are setting as preferred for the browser to render. So, I guess for this form they did their best.
To recap - this shouldn't be misclasified as a general fault on any OS or browser engine. It's just the way anti-aliasing works.
Offtopic: I now have doubts on the petition(s) I linked. There's another one from the same person, which is more general. I don't really understand what's that wrong with DirectWrite/Direct2D vs GDI. To me non-antialiased small fonts are worse than anti-aliased. If it's only for the overlapping - I think I can bear it. If there's a discussion, it should be on "designing desktop views with dpi-aware font size". This means maintaining readability on the older monitors and smoothen text even further for the new 4K-screens. As simple as that :)
I've written this on the active thread, but should apply to Atom-users as well.
So, Benjamin, one more time - shall we close(not mark 'fixed') this one now!? Disclaimer: feel free to initiate technical improvement had you got to think it's a matter crippling VSCode-usability To me, it's still not quite ok, but, as now I know what's under the hood, I consider it normal and would just honor any improvement had anyone decides to dig in.
Cheers,
Kostadin

The reason you get three different looking i's is because it is not only using sub-pixel rendering but sub-pixel positioning, probably using DWRITE_RENDERING_MODE_CLEARTYPE_NATURAL
DWRITE_RENDERING_MODE_CLEARTYPE_GDI_NATURAL would probably be a better choice for the menu bar.

Todays VS Code insider build comes with Electron 1.6.x, would be interesting to hear if this update solves this issue for anyone: http://code.visualstudio.com/Download#insiders

No, same thing(in actual size the 'i' symbols in the menu are different):

Best regards,
Kostadin

Still happening?

I uninstalled the preview, but with 1.18 it still is:

Anyways, thanks for still being concerned :)

Has anyone bumped into this lately - it continues to tease me(comparing it to a python-biased editor)?!
Here's a link on the matter I'd been keeping in my outbox-drafts, but never shared, neither here, nor at the atom-issue: Subpixel accurate GPU rendering of text using a glyph atlas. As most of the monitors nowadays are pretty wide, I guess removing that blurriness at the expense of widening the text could be at least configurable. It still may not help, but at least the issue isn't slaughtered by a lock-bot :)
Cheers,
Kostadin

Here's a screenshot of some options available on another editor:

Zoom to notice how the right one(subpixel) is so much even in regards to the font rendering. It still needs adjustment(contrasting colors, font size), but the letters and spaces between them are rock consistent.

We will change to custom menus on Windows. For now you can set "window.titleBarStyle": "custom".

That allows me to change the background through titleBar.activeBackground. What you imply is that soon those fonts should be adjustable and the user should be able to pick whatever(preferably anti-aliased) fonts they like?!
But that's not the point. It's the anti-aliasing method that's missing(even in the editor window where the code lives and I'm already tired of making screenshots). Otherwise there wouldn't have been such stackoverflow questions:
Blurry/flickering text rendering in Electron
So again, should you close this vital eye-healthiness issue or help present it to someone who could give some more  (isn't it ironic)?!


improve user experience of plugin development

...so developers don't need to run some ugly python code to test their plugins; something like dock run-plugin [--path ...] {"argument1": "value"} could work.
Also, allow plugin loading from arbitrary paths during build.

resolved:
https://github.com/DBuildService/dock/blob/master/README_PLUGIN_DEV.md
https://github.com/DBuildService/dock/commit/1b520b3f4f9681b6d196589124044f0a5c165e97


Update CircleCI config to use new GH org.

Codecov Report
Base: 46.85% // Head: 46.83% // Decreases project coverage by -0.02% :warning:

Coverage data is based on head (b55fc9a) compared to base (6342307).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2028      +/-   ##
==========================================
- Coverage   46.85%   46.83%   -0.03%
==========================================
  Files         206      206
  Lines       27625    27625
==========================================
- Hits        12945    12939       -6
- Misses      13788    13791       +3
- Partials      892      895       +3

Flag
Coverage Δ

airflow-operator
89.13% <ø> (ø)

armada-server
45.72% <ø> (-0.03%)
:arrow_down:

python-client
93.83% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

client/python/armada_client/event.py
85.71% <ø> (ø)

cmd/armadactl/cmd/analyze.go
0.00% <ø> (ø)

cmd/armadactl/cmd/cancel.go
0.00% <ø> (ø)

cmd/armadactl/cmd/commands.go
0.00% <ø> (ø)

cmd/armadactl/cmd/kube.go
0.00% <ø> (ø)

cmd/armadactl/cmd/params.go
0.00% <ø> (ø)

cmd/armadactl/cmd/queue.go
61.37% <ø> (ø)

cmd/armadactl/cmd/reprioritize.go
0.00% <ø> (ø)

cmd/armadactl/cmd/resources.go
0.00% <ø> (ø)

cmd/armadactl/cmd/root.go
0.00% <ø> (ø)

... and 106 more

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

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


TASK: Remove arguments from VH render methods

This removes the arguments from render() in Fluid ViewHelpers, and
registers them using registerArgument instead.
Arguments to render() were deprecated as of Flow 4.0 and support
is removed with Flow 6.0.

^ Always be careful with those return types ;)


Add testing and CI

The controller should have a minimum amount of tests to ensure that the core functionality works as expected and easily extendable when bugs arise so we avoid regressions.

Closing as a duplicate of https://github.com/kubewarden/kubewarden-controller/issues/62


Make an api call to updateUser when a user submits the quiz

Summary
Make a function that has an api call to the server to update the user's skin type and concerns when the form is submitted. This issue depends on #54 so you would want to do that first
Objectives

Grab the uid from firebase
Make a call to the function getUserByUID to get the user object and pass it to updateUser()
These functions should go through the route localhost:5001/api/user/:userId

Acceptance Criteria

User is successfully changed
You can check this in mongodb compass

Due Date

Sprint 3

Additional Notes / References

You could look into changing the updateUser() function to take in a uid instead of a user object

completed in #78


Mark phone numbers as landlines when Twilio exceptions tell us they are landlines

Background
Right now, when we send patient links via SMS, Twilio will occasionally fail to send and respond with an exception stating that the number is a landline and doesn't support SMS.
Action requested
When that occurs in the app, we should mark the phone number as a landline in the DB rather than a mobile number
Acceptance Criteria

Add logic that handles this specific exception by updating the relevant phone number as a landline and saving to the DB.

Additional context
In response to https://prime-cdc.pagerduty.com/incidents/Q3I8FWZ7BIJ6L4?utm_source=slack&utm_campaign=channel and other similar ones

Potentially a backend starter bug

This could be a good starting point: https://github.com/CDCgov/prime-simplereport/blob/fff57ccf152c9c77d7b0394a1b09afb752c3a863/backend/src/main/java/gov/cdc/usds/simplereport/service/sms/SmsService.java#L28-L28

@johanna-skylight and I went through this morning and confirmed that Twilio is appropriately sending these errors to our backend, and that we're correctly transforming the affected numbers into Landline types! (There was some concern that Twilio wasn't sending the errors back to the app, as their logs are incomplete in prod, but everything seems to be working)


Help

From VS Feedback:
how to ado .net entity model(i.e when i am creating a entity frame work it does not show the file ado.net entity model while adding new item)

EF Team Triage: Closing this issue as the requested additional details have not been provided and we have been unable to reproduce it.
BTW this is a canned response and may have info or details that do not directly apply to this particular issue. While we'd like to spend the time to uniquely address every incoming issue, we get a lot traffic on the EF projects and that is not practical. To ensure we maximize the time we have to work on fixing bugs, implementing new features, etc. we use canned responses for common triage decisions.


update OWNERS file

trying to onboard this repo to prow but it's complaining about @lennysgarage ID.

/lgtm


Now compatible with windows and python 3.5

Compatibility with Python 3.5 on windows.

Change xrange to range in data_generator.py
Change iteritem to item in train_frcnn.py and test_frcnn.py
Use floor division line 345 in data_generator.py
Change import "file" to from . import "file"
Rectified to drawing of the rectangle inside test_frcnn.py
change data.next() to next(data) everywhere
Change the open mode of the file from text to binary ('w' and 'r' to 'wb' and 'rb')

This look great - thanks for the PR!
I have a bunch of changes I'm about to push, I'll be ready to merge this in a day or two.

thank you so much for helping me save time but I still found this "Exception: 'float' object cannot be interpreted as an integer" , do you know how to solve it?

Indeed I found why.
I will change it this evening.
Regards,

Corrected.

Thanks , it works

Great work! Works on Mac and Python 3.5.3

Merged with a few extra fixes - thank you @Ahleroy


only sudo works for rocm-commands in rocm-docker

On bare metal all works ok with the latest ROCm 1.7.1, but within docker we now have something unexpected: rocm only works when using sudo. So with normal rights using a clean docker image:
docker run -it --device="/dev/kfd" --device="/dev/dri/renderD128" --device="/dev/dri/renderD129" rocm/rocm-terminal
(...)
rocm-user@2888cb18b1a3:~$ rocminfo
hsa api call failure at line 900, file: /rocmdata/jedwards/git/compute/rocrinfo/rocminfo.cc. Call returned 4104

Using sudo we get perfect response for clinfo and rocminfo:
rocm-user@2888cb18b1a3:~$ sudo /opt/rocm/bin/rocminfo
=====================
HSA System Attributes
=====================
Runtime Version:         1.1
(...)

What could be the cause?

Hi @VincentSC , thanks for the report.
The issue is actually caused by a defect of docker itself, please refer to this docker issue
The fix in docker has been merged to their master branch.
I've tried this docker nightly build and it works fine for rocm/rocm-terminal docker image with rocm-user.

I use Docker version 18.02.0-ce, build fc4de44 and this still happens to me. I use rocm 1.7.1 with ubuntu 16.04 host with AMD Vega 64. docker pull rocm/rocm-terminal:1.7.1 then docker run -it --device=/dev/kfd --device=/dev/dri --group-add video rocm/rocm-terminal:1.7.1 and I can't execute ./vector_copy from /opt/rocm/hsa/sample/ without sudo. Same for rocminfo.
I have no problem with rocm/hiptensorflow:rocm1.7

@Sumenia, as mentioned in the above message, the issue is from a docker defect in 18.02.0 build.
Please manually install the following docker version and try again:
https://download.docker.com/linux/ubuntu/dists/xenial/pool/nightly/amd64/docker-ce_18.04.0~ce~dev~git20180315.170650.0.8fabfd2-0~ubuntu_amd64.deb

Oh! My bad @sunway513 ,  I did see your message, did check my version (17.~ something), did read docker documentation in order to get the latest build from apt-get, but didn’t check the version I received from it. I saw “18.~something” and though it was ok. I should have doubled checked, sorry about that and thank your for you answer 👍

Thanks @Sumenia for confirming :-)

In order to use GPU without sudo in rocm-docker, the regular user inside the container must have the same uid/gid as a host system user permitted for GPU access. Here is an edited repository which fixes the problem: https://github.com/dmikushin/ROCm-docker You can also pull it from DockerHub: https://hub.docker.com/repository/docker/marcusmae/rocm-docker


Discover tab list add 'new' tab

Expected Behavior
Describe the expected behavior here.
Specifications
As a user, I would like to action so that reason.
Features:

describe feature details here.

Development Tasks:

[ ] Task 1
[ ] Task 2

Dependencies
List any dependencies that are required for this feature by providing links to the issues or repositories.
References
List any references that are related to this feature request.

Resolved in #676


Fix generation of shortnames

Fixes generation of shortNames and adds both rg and rgs as short name.

:+1:

:+1:


Does this work?

Yeah. Should you change any instance of a stat to _baseStat?
For example, change _health to _baseHealth?
That way we can add modifiers later and we can know what exactly we're working with later.


update description and demo for back to top

Updates to the last paragraph of back to top component description, thank you @zakiya
Change the demo/preview section to only be plain sample markup
Add paragraph in the demo section saying the component is in the bottom right

The new description looks great. OK to merge on my end!


Stuck on Creating Ramdisk

./downr1n.sh 11.3
[*] Command ran: ./downr1n.sh 11.3
downr1n | Version 3.0
Created by edwin, thanks palera1, and all people creator of path file boot
[] Waiting for devices
[] Detected recovery mode device
[] Waiting for devices
[] Detected recovery mode device
[] Getting device info...
Detected cpid, your cpid is 0x8010
Detected model, your model is j71bap
Detected deviceid, your deviceid is iPad7,5
[] To get into DFU mode, you will be guided through 2 steps:
[] Press any key when ready for DFU mode
Get ready (0)
Release power button, but keep holding home button (9)
Release power button, but keep holding home button (0)
[-] Device did not enter DFU mode, try again
[] Waiting for devices
[] Detected recovery mode device
[] To get into DFU mode, you will be guided through 2 steps:
[] Press any key when ready for DFU mode
Get ready (3)
Hold home + power button (4)
Release power button, but keep holding home button (10)
[] Device entered DFU!
ls: ipsw/.ipsw: No such file or directory
YOU DON'T HAVE AN IPSW SO WE ARE GONNA DOWNLOAD IT, THE IPSW WILL BE for iPad7,5 AND the version 11.3, DO YOU WANT TO CHANGE THE VERSION (YES) OR (NO)
NO
You answered NO. so using the 11.3.
[] Downloading ipsw, it may take few minutes.
############################################################################################## 100.0%
[] Argument detected we are gonna use the ipsw specified
[] Creating ramdisk
I also tried 14.3

I'm stupid I just closed this for no reason.

./downr1n.sh 11.3 [*] Command ran: ./downr1n.sh 11.3 downr1n | Version 3.0 Created by edwin, thanks palera1, and all people creator of path file boot
[] Waiting for devices [] Detected recovery mode device [] Waiting for devices [] Detected recovery mode device [] Getting device info... Detected cpid, your cpid is 0x8010 Detected model, your model is j71bap Detected deviceid, your deviceid is iPad7,5 [] To get into DFU mode, you will be guided through 2 steps: [] Press any key when ready for DFU mode Get ready (0) Release power button, but keep holding home button (9) Release power button, but keep holding home button (0) [-] Device did not enter DFU mode, try again [] Waiting for devices [] Detected recovery mode device [] To get into DFU mode, you will be guided through 2 steps: [] Press any key when ready for DFU mode Get ready (3) Hold home + power button (4) Release power button, but keep holding home button (10) [] Device entered DFU! ls: ipsw/.ipsw: No such file or directory YOU DON'T HAVE AN IPSW SO WE ARE GONNA DOWNLOAD IT, THE IPSW WILL BE for iPad7,5 AND the version 11.3, DO YOU WANT TO CHANGE THE VERSION (YES) OR (NO) NO You answered NO. so using the 11.3. [] Downloading ipsw, it may take few minutes. ############################################################################################## 100.0% [] Argument detected we are gonna use the ipsw specified [] Creating ramdisk
I also tried 14.3

not iOS 11 supported ofc, is your cpu an amd or intel ?

Apple m1

m1 are not good with faster soo you will have to try manually putting your device in dfu mode and forcing reboot it and try again gaster until you get it success the exploit, this is the command for faster ./binaries/$(uname)/gaster pwn

Thanks


Maven profile configuration problem

Describe the bug
Maven profile configuration problem.
<profile>
      <id>main</id>
      <activation>
        <activeByDefault>true</activeByDefault>
        // Why add jdk activation mode in main profile?
        // If so. When I want to use the core-modules profile to package,
        // some modules that I don't want to activate will also be introduced
        <jdk>[8,)</jdk>
      </activation>
      <modules>
     ...
      </modules>
</profile>

To Reproduce
Steps to reproduce the behavior:

Master branch
Use command 'mvn -Pcore-modules -DskipTests clean package'

Expected behavior
Than some modules like test or docker modules will be packaged, this is not what I want.
Screenshots
No.
Desktop (please complete the following information):

OS: macos

Additional context
Add any other context about the problem here.

The issue had no activity for 30 days, mark with Stale label.

Closed as stale. Please create a new issue if it's still relevant to the maintained versions.
Now, you can ask questions in https://github.com/apache/pulsar/discussions/categories/q-a instead, or send feature request which you don't have idea to implement in https://github.com/apache/pulsar/discussions/categories/general, since we almost use issues to track actionable tasks.


uploading file to s3 using aws-sdk-cpp-transfer do nothing

I try to upload a file to s3 with transfer, but nothing happens and trace log is empty.
Any Idea ?
`
Aws::Utils::Logging::InitializeAWSLogging(Aws::MakeSharedAws::Utils::Logging::DefaultLogSystem("Internal_Logger", Aws::Utils::Logging::LogLevel::Trace, "aws_sdk_"));
ClientConfiguration config;
config.scheme = Scheme::HTTPS;
config.connectTimeoutMs = 30000;
config.requestTimeoutMs = 30000;

TransferClientConfiguration transferConfig;
transferConfig.m_uploadBufferCount = 20;

std::shared_ptr<S3Client> m_s3Client = Aws::MakeShared<S3Client>("Upload", config);
std::shared_ptr<TransferClient> m_transferClient = Aws::MakeShared<TransferClient>("Upload", m_s3Client, transferConfig);

std::shared_ptr<UploadFileRequest> requestPtr = m_transferClient->UploadFile("myFile.zip", "MyLiTTleBucket", "myFile.zip", "application/zip", false, false);

Aws::Utils::Logging::ShutdownAWSLogging();`

`
[TRACE] 2016-02-03 16:31:13 FileSystemUtils [139892860249920] Checking HOME for the home directory.
[DEBUG] 2016-02-03 16:31:13 FileSystemUtils [139892860249920] Environment value for variable HOME is /home/chemidy
[DEBUG] 2016-02-03 16:31:13 FileSystemUtils [139892860249920] Home directory is missing the final / appending one to normalize
[DEBUG] 2016-02-03 16:31:13 FileSystemUtils [139892860249920] Final Home Directory is /home/chemidy/
[INFO] 2016-02-03 16:31:13 ProfileConfigFileAWSCredentialsProvider [139892860249920] Setting provider to read credentials from /home/chemidy//.aws/credentials, for use with profile default.
[INFO] 2016-02-03 16:31:13 InstanceProfileCredentialsProvider [139892860249920] Creating Instance with default EC2MetadataClient and refresh rate 900000.
[INFO] 2016-02-03 16:31:13 EC2MetadataClient [139892860249920] Creating HttpClient with max connections 2 and scheme http
[INFO] 2016-02-03 16:31:13 CurlHandleContainer [139892860249920] Intializing CurlHandleContainer with size 2
[INFO] 2016-02-03 16:31:13 CurlHandleContainer [139892860249920] Initializing Curl library
[INFO] 2016-02-03 16:31:13 CurlHandleContainer [139892860249920] Intializing CurlHandleContainer with size 25
`

I think this might be because the UploadFile call will trigger the upload to happen asynchronously. So you'll need to wait for the upload to finish before your program quits.
You could do something like this to wait for the request to finish (either successfully or with an error).
while (!requestPtr->IsDone()) {
    std::this_thread::sleep_for(std::chrono::seconds(1));
}

You can also look at WaitForUploadAndUpdate in aws-cpp-sdk-transfer-tests/TransferTests.cpp for more ways to check on the status of the upload.

Emulating synchronous behavior is probably not what you want in a real environment (unless the code above is your production code and isn't doing anything more complex).
The problem is that this pointer being returned by UploadFile is getting cleaned up when you exit the stackframe, which is calling the destructor of UploadFileRequest. You need to take a handle to that pointer outside of the scope of this method and then use the callbacks to handle the upload finishing.
If all you need to do is what you posted above, then kahkeng's advice is best.

closing. Please repoen if issue persists.


Cross platform build script.

Issue #226
Implement cross platform build script.

cross building is currently not supported, the recommendation is to build from the target machine. https://todo.sr.ht/~eliasnaur/gio/165#event-53403


Linux az interactive mode is not working anymore

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az interactive
Errors:
cannot import name 'ENV_ADDITIONAL_USER_AGENT'
Traceback (most recent call last):
python3.6/site-packages/knack/cli.py, ln 215, in invoke
    cmd_result = self.invocation.execute(args)
cli/core/commands/__init__.py, ln 625, in execute
    raise ex
cli/core/commands/__init__.py, ln 689, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
cli/core/commands/__init__.py, ln 682, in _run_job
    six.reraise(*sys.exc_info())
...
cli/command_modules/interactive/custom.py, ln 41, in start_shell
    azext_interactive.start_shell(cmd, style=style)
cliextensions/interactive/azext_interactive/__init__.py, ln 10, in start_shell
    from .azclishell.app import AzInteractiveShell
interactive/azext_interactive/azclishell/app.py, ln 31, in <module>
    from azure.cli.core.commands.client_factory import ENV_ADDITIONAL_USER_AGENT
ImportError: cannot import name 'ENV_ADDITIONAL_USER_AGENT'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az interactive --debug

Expected Behavior
Environment Summary
Linux-5.4.0-050400rc2-generic-x86_64-with-debian-buster-sid
Python 3.6.5
Installer: DEB

azure-cli 2.3.1

Extensions:
interactive 0.4.3

Additional Context

Ok, I found this to be solution:
https://github.com/Azure/azure-cli/issues/12837#issuecomment-607610509
But fix should be in .deb as well.

interactive

Hi @ZEROF, interactive is not bundled in the deb. It is always installed dynamically as an extension.


Version requirement change

@decentralgabe generalizing to "unique string" still work? i'm guessing recommendation to monotonic integer can be moved to specific profiles or impl guide, etc


[$500] Web - Send money - Save and continue button infinitely loading after entering additional details

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

Version Number: 1.3.81-6
Reproducible in staging?: Y
Reproducible in production?: No. The "Send money" option is not present in production
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
Expensify/Expensify Issue URL:
Issue reported by: Applause-Internal Team
Slack conversation: @
Action Performed:

Navigate to OldDot, log in, and add a personal bank account (Fidelity)
Go to ND and log in to the same account
Navigate to any conversation
Click on "+" button > Send money
Enter the amount and select "Next"
On the final review screen, select "Pay with Expensify"
On the "Additional details, fill out the fields
Click on the "Save&Continue" button

Expected Result:
User is able to proceed to the Onfido privacy page
Actual Result:
The "Save&Continue" button is infinitely loading, user is unable to proceed
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari
https://github.com/Expensify/App/assets/115492554/7e7b3f25-75e1-45a1-a976-94164ea686d3

MacOS: Desktop

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01b05455d41ac4f2b6
Upwork Job ID:<PHONE_NUMBER>765879296
Last Price Increase: 2023-10-11

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

cc @techievivek since this seems related to your recent Send Money PR

I'm headed out for the next few hours, I'll help tomorrow if needed.

I don't think this is an issue with NewDot. On staging, the UpdatePersonalDetailsForWallet request is returning Please review your account number and routing number one more time before continuing. Reimbursements are not reversible.. That seems like an issue with the bank account we're using for test, so not an issue. That being said we should be displaying an error to the user, which is the real issue here.
Gonna remove the blocker label and keep this open to address the error message issue.

PR here

@luacmartins will https://github.com/Expensify/Web-Expensify/pull/39205 fix this issue? Or are you working on some other PR? Either way, it sounds entirely backend, so we can remove @eVoloshchak's assignment, right?

Yes, it should have been fixed in that PR. We can retest this now that the PR made it to production.

@izarutskaya could you please retest this when you haev a chance and let us know if it's fixed or still reproduceable in the latest build? Thanks!

Bump @izarutskaya, could you please retest this?

Closing this issue since it doesn't seem to be reproducible anymore. Please reopen if needed.


.Net: Python: improve chat_system_prompt default and messages behaviour for OpenAIChatRequestSettings

Describe the bug
A clear and concise description of what the bug is.
There is a default system message in the OpenAIChatRequestSettings class that get's loaded into the ChatPromptTemplate when instantiating with OpenAI or AzureChatRequestSettings, this is not controlable or transparent.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Platform

OS: [e.g. Windows, Mac]
IDE: [e.g. Visual Studio, VS Code]
Language: [e.g. C#, Python]
Source: [e.g. NuGet package version 0.1.0, pip package version 0.1.0, main branch of repository]

Additional context
Add any other context about the problem here.

Any plans on this? I think this is quite urgent and basic function for chat based QA scenario. Currently, it is a bit messy like how we override and not appending the default system message located in OpenAIChatRequestSettings class.
And the ChatPromptTemplate.restore() is buggy to add system message as mentioned in the #4705

@HuskyDanny just created a PR for this, have a look and let me know if you think this will solve your issues as well!

Thanks! This resolves and now it is much more consistent to use restore and init