xcc: fix warning about unsupported compiler/linker flags on no standard defines/libraryes

XCC does not support the GCC flags to not include standard include files and libraries (e.g. -undef, -nodefaultlibs). So make these into parameters and XCC compiler configuration can override them.

@mped-oticon looks like the things you wanted to do in general for compiler/option abstraction

Codecov Report

Merging #10979 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #10979   +/-   ##
=======================================
  Coverage   53.26%   53.26%
=======================================
  Files         214      214
  Lines       25920    25920
  Branches     5715     5715
=======================================
  Hits        13806    13806
  Misses       9791     9791
  Partials     2323     2323

Continue to review full report at Codecov.

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

@mped-oticon looks like the things you wanted to do in general for compiler/option abstraction

This is definitely a step in the right direction, but what I will do will be higher level - to move as much logic out of CmakeLists.txt  as possible.

zephyr_ld_options (as opposed to zephyr_link_libararies) is supposed to test the flag and then only include it if it is supported by the toolchain.
So if everything is working as it should, there should be no warning and no need for this patch. I don't have xcc, could you post the warning you are getting or reproduce the warning with the SDK?

installing xcc is documented here: https://docs.zephyrproject.org/latest/boards/xtensa/intel_s1000_crb/doc/intel_s1000.html
You can download it and get a license directly.

XCC does emit warnings:
xt-xcc WARNING:  -nostartfiles is ignored
xt-xcc WARNING:  -nodefaultlibs is ignored

It seems like cmake does not pick up these warnings.


de

If this is a bug, please try reproducing using https://flow.org/try/.

Sorry, typing went bad here.


[Enhancement/Refactor]-  Issue 509/create new use message list hook

This PR:
Partially addresses #509. This PR aims to migrate over/refactor the existing workflow for messages loading and rendering from utils to React Query and lightened the burden on React Contact to just the update of the message state and loading statuses. Other functionality for Messages such as message building and message sending will be considered soon for refactoring in future PRs.
Had tried using just React Query for isLoading or isFetching, but the UI appears smoother with React Context and does not cause any noticeable re-renders of the message cards, so it remained with React Context. Existing changes does not break existing functionalities in live tests.
Unit tests that's related to existing changes has been updated. A new test file has been made, mainly for the new custom hook useMessageList.
Screenshots (if applicable):
https://github.com/codeforpdx/PASS/assets/14917816/5e0170ba-d72c-46e9-b1c2-d8c3fc79db0f
https://github.com/codeforpdx/PASS/assets/14917816/3e62a8f1-1085-4523-af1b-42abf4bab5bc
Future Steps/PRs Needed to Finish This Work (optional):
Future steps for this will include updates to other Message functionalities in utils similar to what is done here, but for message building and message sending.
Issues needing discussion/feedback (optional):
1.    Issue #509 will be on-going even after this is merged (see Future Steps above)

I'm a bit confused by this statement

Had tried using just React Query for isLoading or isFetching, but the UI appears smoother with React Context and does not cause any noticeable re-renders of the message cards, so it remained with React Context.

React Query is built on top of a react context, so this shouldn't make a difference. If there's UI smoothness issues, they probably come from how the UI itself is handling the different loading states.

Was a bit confused myself about this actually, when I first noticed this (see clip when I've used isFetching again), but I think I know what might be happening. Since clicking on unread message previews updates the read status for message Thing on Solid, I that's what's causing the loading screen to pop up.
https://github.com/codeforpdx/PASS/assets/14917816/a8c768f4-8992-4642-830e-38f04c2529f0

I notice you're hitting the network multiple times whenever you update a message. This may be why you were having weird behavior, and it shouldn't be necessary.

Think I've managed to fix this behavior by moving the condition for readStatus along with folderType === 'Inbox' in the update I'm coding up now.

The react query update mutations can handle all of that for you. You don't need to manually refetch in the components either. The purpose of these hooks is to hide the network from the components. Put all the networking logic in the hooks, and expose simple interfaces to the components.

Oh! That would be nice. I'll see if I could get mutations working for this. I simply wanted to update the readStatus for that one message. Like what's shown above in the clip, but without the loading screen appearing.

You may find it simpler not to handle both the inbox and the outbox in a single hook. Maybe try separating the inbox logic into its own useInbox hook, and the outbox into a useOutbox hook. I think you'll be able to get a clearer design that way.

I was planning on splitting up Inbox and Outbox into their own routes and pages in a later PR.

Hey @timbot1789, think I've managed to get the React Query mutation working! Also got the loadMessages being handled directly by isFetching now without triggering the loading animation unless it's manually refreshed via the refresh button or when loading into the Messages page for the first time.

I'm running this on my local and I'm noticing that when I click on messages from the top nav the Messages section loads with "No Messages Found" while the messages are being retrieved. I also have 20 pages of messages so that might be an edge case.

I'm running this on my local and I'm noticing that when I click on messages from the top nav the Messages section loads with "No Messages Found" while the messages are being retrieved. I also have 20 pages of messages so that might be an edge case.

Oh that's is strange. I'll take a look.

I'm running this on my local and I'm noticing that when I click on messages from the top nav the Messages section loads with "No Messages Found" while the messages are being retrieved. I also have 20 pages of messages so that might be an edge case.

Not really seeing it in my case (see clip), so not sure what might be going on.
https://github.com/codeforpdx/PASS/assets/14917816/ab84d8b1-7a45-48af-bef1-785a3cc7eb3f

I'm running this on my local and I'm noticing that when I click on messages from the top nav the Messages section loads with "No Messages Found" while the messages are being retrieved. I also have 20 pages of messages so that might be an edge case.

This sounds like a performance bottleneck that we're not set up to handle yet. We haven't implemented any kind of server-side pagination, so the network load when fetching multiple pages of records is quite hefty. I don't think it's an issue we can fix in this PR.

I'm running this on my local and I'm noticing that when I click on messages from the top nav the Messages section loads with "No Messages Found" while the messages are being retrieved. I also have 20 pages of messages so that might be an edge case.

Not really seeing it in my case (see clip), so not sure what might be going on.
Screen.Recording.2023-11-15.at.12.11.43.mov

I've pulled latest code and issue is resolved. Now showing "Loading Messages ..." 👍

I haven't tested this branch, but when creating a message - does the modal still behave like it did in very top video here?
The delay between clearing the form and closing the modal.
If so, is that necessary for some reason?

This looks good to me. Cleans up Messages. Separates concerns between UI and Solid data. Nice work 👍


Terminal command default decoration always renders one line too high after running ls

Testing #143433
Other commands don't have this issue--seems to only be ls so far.

Was ls the exact command you ran? I cannot repro this on mac/pwsh

Yes, ls on Windows/pwsh

Oh, this only happens if I run clear first. Probably a duplicate of https://github.com/microsoft/vscode/issues/143784

This is an issue with Windows only, very easy to reproduce with multi-line prompt:

It happens because conpty doesn't guarantee the cursor will be in the right position when outputting the command start sequence, so we need some extra smarts to try catch this case for windows ptys only.


UnixPb: Add fakeroot package to dockerfiles

[x] commit message has one of the standard prefixes
[ ] faq.md updated if appropriate
[ ] other documentation is changed or added (if applicable)
[ ] playbook changes run through VPC or QPC (if you have access)
[ ] for inventory.yml changes, bastillion/nagios/jenkins updated accordingly

ref https://github.com/adoptium/infrastructure/issues/2291
fakeroot is needed for some tests in jdk_tools_1 in the extended openjdk test suite

The fakeroot package installs fine in all of the affected docker containers


Alpha version - Edit: remove tags

fixes #85

Can consider changing all occurrences of tag to studyGroupTag or something like that. Also I think you should look through EditCommandParserTest to see if you need to make any changes. For example, parse_allFieldsSpecified_success() could probably also include the remove tag field. then for parse_resetStudyGroupTags_success() maybe you can check whether it is still applicable now that there is a remove tag feature.

Mostly everything looks good but can consider changing all occurrences of tag to studyGroupTag or something like that. Also I think you should look through EditCommandParserTest to see if you need to make any changes. For example, parse_allFieldsSpecified_success() could probably also include the remove tag field. then for parse_resetStudyGroupTags_success() maybe you can check whether it is still applicable now that there is a remove tag feature.

I think I kept it as tag to reduce the lengths of the names of the different associated fields and methods, because it sounded a bit complicated. But yes, I'll edit the test file, thanks!

Mostly everything looks good but can consider changing all occurrences of tag to studyGroupTag or something like that. Also I think you should look through EditCommandParserTest to see if you need to make any changes. For example, parse_allFieldsSpecified_success() could probably also include the remove tag field. then for parse_resetStudyGroupTags_success() maybe you can check whether it is still applicable now that there is a remove tag feature.

I think I kept it as tag to reduce the lengths of the names of the different associated fields and methods, because it sounded a bit complicated. But yes, I'll edit the test file, thanks!

Also, I think the parse_resetStudyGroupTags_success() is not applicable to the remove tags feature, and has been edited by those who worked on the add tag one already, since there's no functionality to reset the tags now.

LGTM!


can i create ionic (mobile view) within an already build in angular ( mean ) app &  without using full framework ?

Hello! Thanks for opening an issue with us! As this seems like more of a support question i will urge that you ask this question on our forum. Thanks for using Ionic!


Mulitline does not work well on windows command prompt

For every update, a new bar is written ie. the original bar is not updated.

can you provide a minimal example to reproduce the issue ?
which versions (node, cli-progress, windows) do you use ?

That was a really quick response!
I just solved it - I had to increase the screen buffer size of the command prompt.
PS. I am providing some more details on the issue now

the windows command prompt doesn't work as expected. please use the powershell.
since the issue is raised on an old windows version i'm unable to test it - cli-progress is not designed for windows even it may work
generally the window has to be bigger then the number of total progress bars - see #59

"cli-progress is not designed for windows even it may work" - it would be best mentioned on the README.md then

maybe...but this issue affects any UI component within the standard command prompts on windows

did you tried it with windows powershell on win10 ?

Same issue with powershell.

it works without any issues...the problem seems to be related to your environment (windows version, powershell version, environment config)

std windows command prompt ...

Thanks for your responses Andi. I've settled for how it is on Win8.
It would still help if README mentions that it works as expected on Win10 CMD version etc & above.
I dont have Win10, so cant verify at my end.

i've already added a "weak" notice about windows platforms: https://github.com/AndiDittrich/Node.CLI-Progress#compatibility
btw. cli-progress is also used in NodeMCU-Tool which doesn't encountered any issue with older windows versions yet - therefore you're observed behaviour is very strange


[Bug]: Unable to enable prefix caching for glm-4 model (not -hf versions)

Your current environment

The output of `python collect_env.py`
Your output of `python collect_env.py` here

Model Input Dumps
No response
🐛 Describe the bug
With glm-4 model, prefix caching is automatically disabled, because the engine is treating it as MLLM.
Maybe related to following codes:
https://github.com/vllm-project/vllm/blob/d427e5cfda8d2536b81e6021128e71b2dbc281aa/vllm/model_executor/models/chatglm.py#L758-L782
https://github.com/vllm-project/vllm/blob/d427e5cfda8d2536b81e6021128e71b2dbc281aa/vllm/engine/arg_utils.py#L1046-L1051
Unfortunately, the glm-4 and glm-4v models have the same model_type value, how can I override this behavior without changing the code?
Before submitting a new issue...

[X] Make sure you already searched for relevant issues, and asked the chatbot living at the bottom right corner of the documentation page, which can answer lots of frequently asked questions.

cc @jeejeelee

I will look into this issue tomorrow.


feat: update products section

link to docs added

https://user-images.githubusercontent.com/22061815/211779783-238734f0-efe3-4d8b-abb4-65c0f210fc5f.mov

updated products section as per the doc

removed the link from each card
added 2 buttons, primary and secondary with specified link redirection

https://user-images.githubusercontent.com/22061815/211779829-e219a845-6673-4a43-9231-08089aab03c0.mov
Doubts

The images are the same for the APY predictor and price oracle, where can I get them?
The alignment is changed (3rd card - price oracle is wrapped to next line because we have 2 vertical sections as for developers & for users

@mohandast52 please check all links to make sure they are correctly linked re external/not. Many internal links are opening in new tabs. I really don't want to ask for this again.


Update to AppVersion 0.31.5

What this PR does / why we need it:
Take latest version of flux and controllers for CVE fixes
Which issue this PR fixes
CVEs
Special notes for your reviewer:
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] helm-docs are updated
[x] Helm chart is tested
[x] Update artifacthub.io/changes in Chart.yaml
[x] Run make reviewable

Thanks for the work. But Out-of-date ;-)  Chart was updated to app 0.32.0


expression: add builtin sha1

What have you changed? (mandatory)
add sha1 to function in builtin_encryption.rs
What are the type of the changes? (mandatory)

improvement (non-breaking change which is an improvement to an existing feature)

How has this PR been tested? (mandatory)
pass make expression
Does this PR affect documentation (docs/docs-cn) update? (mandatory)
no
Does this PR affect tidb-ansible update? (mandatory)
no

Thanks! Please also add tests about null case. Rest LGTM

@breeswish I had added the null case in test_sha1, just like test_md5 above. i just implement like test_md5 do.

LGTM. Please address comments.

@overvenus I have updated the api from result_str -> result

Hm test failed 🙁

@hicqu result return the real memory view of the sha1, but we need hex encoded result, so that result_str is the best option. If we use result, we still need to hex encode it manually

Equivalent code in Python

@overvenus @hicqu just as @TennyZhuang say, we think result_str is the best option, I had revert the code back to this implement.

LGTM.


No input sanitization for emoteStreakEndingText

Reflected XSS: JavaScript can be injected in the URL to show NSFW stuff. Very low impact as it would require the streamer to click the link, but given it is your website they might feel inclined to do so?
https://github.com/jahaanjain/Emote-Overlay/blob/a8b273f928969d10e7cb25fc97ce5bc8c8028b9f/main.js#L12
https://github.com/jahaanjain/Emote-Overlay/blob/a8b273f928969d10e7cb25fc97ce5bc8c8028b9f/main.js#L210

Thanks for opening the ticket! I've resolved this in the latest commit.


Documation All Code

The codebase needs comprehensive documentation to improve understanding and maintainability. This includes adding comments, explanations, and any necessary documentation to all sections of the code. The goal is to make the code more accessible for future reference, debugging, and collaboration.
Tasks:

[ ] Add comments to explain each function and major code block.
[ ] Provide inline documentation for complex algorithms or logic.
[ ] Ensure consistency in coding style and documentation across the entire codebase.
[ ] Update existing documentation or add new documentation where needed.
[ ] Verify that documentation aligns with the latest code changes.

Deadline: [Specify a reasonable deadline for completing the documentation tasks.]

❤


Hello from valtio-yjs

Hi, just wanted to say hello.
I'm trying a project valtio-yjs, but it's still very far from MVP. There might be something I could learn from your project.
I'm also interested in @YousefED 's work. (My hope was/is I could just use it without me creating a new one.)
https://github.com/YousefED/reactive-crdt/issues/5

Let me know if you've got any questions! I am ashamed to say I have a few things to learn from this code as well, as all I really know about how it works is the bare essentials I learned from YJSON and the fact that the tests pass. 😅
I'm going to close this issue, but we can continue the conversation via email or twitter.


Better (I hope) smoothing of rms spectra; better handling of invalid …

…files

Hello @matteobachetti! Thanks for submitting the PR.

In the file srttools/imager.py, following are the PEP8 issues :

Line 144:80: E501 line too long (93 > 79 characters)
Line 504:17: E125 continuation line with same indent as next logical line
Line 771:5: E303 too many blank lines (2)
Line 1415:1: E302 expected 2 blank lines, found 1


Figure Counter ignored in the rendered figure when using counter(figure).update()

Description
When resetting the figure counter using counter(figure).update(somevalue), the rendered figure does not reflect the modified counter value.
How to reproduce
Here is a MWE:
#counter(figure).display()
#figure(
	rect[Hello World],
	caption: [This is a rectangle]
)
#counter(figure).display()
#counter(figure).update(4)
#counter(figure).display()
#context counter(figure).update(3)
#context counter(figure).display()
#figure(
	rect[Hello World 2],
	caption: [This is a rectangle (again)]
)

which results in:

Installation Details
Operating System: Arch Linux
Installation Method: package manager (pacman)
Typst Version: typst 0.11.1 (50115102)

Ahh that makes sense, thanks.


add docs around scim provisioning user conflicts

Problem description
if users already have account when someone is scim provisioning them they will not be provisioned. lets document the situations in which this occurs, and what the person provisioning users should do to fix it. if its something the user can resolve themselves lets also document that.
potential fix
was mentioned that we should consider an faq section for this content
links to docs for context (contains explanation of the different scenarios):
https://docs.google.com/document/d/16_mODtVfuGgKFM7vs9ZdMdBt8GbmXTGaArcVzFuoC8Q/edit
https://docs.google.com/document/d/18mJDn1u2c8kY7F6DHROvSEu4KS4o1EhQyeRJgeXVsPg/edit

@infin8x - @praneetloke and I met to do a last walkthrough to verify some SCIM provisioning bits when it came to the user conflicts. I had noticed some unexpected behaviors in my dev environment yesterday which is why I wanted to verify. We unfortunately wrongly assumed that the email was updated when re-syncing with your third party provider, e.g. GitHub.  This was actually a pre-existing behavior in the console at an earlier point in time that was then removed at a later date,  https://github.com/pulumi/pulumi-service/pull/2929. This is also how we asked Telus to resolve the conflict issue they were having and we assumed all is well and worked for them as we had not heard anything back. This functionality was removed to get around the unique constraint we have on emails (i.e. if an email is refreshed to match the third party provider and the updated email happens to already exist in our system there would be a conflict and it wouldn't be possible) similar to what we are trying to get around from the SCIM provisioning aspect.
Anyways, in light of this, we discussed just updating the doc for now to tell the user if there is a conflict they can optionally delete their account in order to release the email and then their organization will be able to provision the user account for them via SCIM. They can also optionally create a new pulumi account with their personal email address and transfer the stacks out to the new account if they would like to retain their existing stacks. We feel this is probably the most straight-forward process we can give them at this point in time. There are other potential workarounds, but would be a bit complicated to have the user go through them on their own without much guidance. If they would like other options we can leave a note to reach out to customer support. Then we can walk them through the process as we have been doing thus far. Given the small number of users that would likely be affected we think this is a reasonable approach for now just to get this out the door and if it ends up becoming a huge issue we can always look into prioritizing https://github.com/pulumi/pulumi-service/issues/6159. Sorry about all the confusion, let me know if this sounds reasonable and I'll update the doc with those steps.

im tots cool with moving forward and getting this out quickly, but i worry so much about telling users to delete their account as a solution. its really an awful experience. so if there's any way we can prioritize the follow-up work sometime soon id super advocate for that.

Thanks, Sean, for digging into this. I totally forgot that we removed updating emails via sync when we introduced the email uniqueness in the system. 😞

but i worry so much about telling users to delete their account as a solution. its really an awful experience

Yeah agreed but if you look at the alternatives, it is not any worse unfortunately. The workarounds may involve more blocks depending on the user's specific situation. The second workaround of "connect your identity and then try reprovisioning" itself has limitations too between Okta and Azure AD. For example, in Okta you can edit failed provisioning tasks which you may need to do if, say, your GH identity used a different username than what we derive from your email in SAML/SCIM case. In Azure AD, you can't edit failed provisioning tasks. There is just no facility for that. You can edit a user profile but that's not helpful in this situation. And we don't yet know what GSuite offers. So there isn't a good solution that works across IdPs (not to mention we don't know what users will run into most of the time.)
Note that implementing https://github.com/pulumi/pulumi-service/issues/6159 may solve OAuth identity users being able to change their emails within Pulumi and unlocks the workaround of "change your email in Pulumi and then retry SCIM provisioning" (which still isn't perfect as I mentioned the case where your GH username is different from your SAML one and therefore could lead to undesirable effects.) But if the user doesn't want to do that, then the alternatives aren't good.

+1, thanks for digging into this. I had a look at the latest docs - they look good to me. "contact support" is a good escape valve at our current size.
anything else to do here or should we close this?

Thanks, I think we are good for now! I went ahead and just closed this out.


Can't run the example

For example this
import Prelude

import Effect (Effect)
import React as R
import React.DOM as R
import React.DOM.Props as RP
import ReactDOM as RDOM
import Thermite (defaultPerformAction, defaultRender)
import Thermite as T
import Thermite.DOM (defaultMain)

type Model = { counter :: Int }

initialState :: Model
initialState = { counter: 0 }

spec ::forall props event. T.Spec Model props event
spec = T.Spec {performAction: defaultPerformAction, render: defaultRender}

main :: Effect Unit
main = defaultMain spec (const initialState) "MyComponent" {}

Gives me something like:
No type class instance was found for\n\n    Prim.Row.Union t0\n                   t4\n                   ()\n\n  The instance head contains unknown type variables. Consider adding a type annotation.\n\nwhile applying a function defaultMain\n  of type ReactPropFields t0 t1 => Spec (Record t2)\n                                     { children :: Children\n                                     | t0\n                                     }\n                                     t3\n                                   -> ({ children :: Children\n                                       | t0\n                                       }\n                                       -> Record t2\n                                      )\n                                      -> String -> ... -> ...\n  to argument spec\nwhile inferring the type of defaultMain spec\nin value declaration main\n\nwhere t3 is an unknown type\n      t1 is an unknown type\n      t0 is an unknown type\n      t2 is an unknown type\n      t4 is an unknown type\n"

And something like this happens when I try any example from readme

Thanks @DoctorRyner, sorry for neglecting this - I'll get a new version re-issued hopefully by tomorrow; things are just a bit busy for me at the moment. Sorry again!

Ok. So, I'm sorry but is there any progress?

Duplicate of #104
Sorry for the delay; the README has been updated. The issue is due to how purescript-react solves a typeclass relating it's props argument when using createElement. The README in the github repo is updated, but the pursuit documentation will have to wait until another version is iterated. I hope this helps!

Good. And by the way, is there a way to pass something like initialActions? I'm trying to create a counter that increments itself every second, of course, you should be able to increment / decrement it with buttons as well


Extract genomes and info from ENA

Currently there are two major databases that phage and plasmid info in this project is coming from: NCBI and ENA. Fetching data from NCBI is mostly taken care of (see issue #2 and acc2gb.py), but we also need to extract genomes and info from ENA, analogous to what's done in #2.
The accession numbers are things like AL591985.1 and correspond to a result page like this.
I haven't tried in detail to find out what to do, in case anyone out there knows of a BioPython module, say, that can do this. This python API tutorial might be relevant.

Turns out you can just use the ENA accession numbers to fetch straight from NCBI, so this issue is unneeded.


The new Endpoint routing behaves in unexpected and undocumented ways

Describe the bug
I'm migrating a website originally written in asp.net to asp.net core.
In routing, I had minor problems until 2.1, but in 2.2 lots of things are broken.
I cannot reproduce my original problem, but I've found other problems along the path to diagnose it.
The original problem is that I get:
AmbiguousMatchException: The request matched multiple endpoints. Matches:

MySite.Web.Controllers.HomeController.PermanentNotFound (MySite.Web)
MySite.Web.Controllers.HomeController.Dashboard (MySite.Web)
MySite.Web.Controllers.CompanyController.Index (MySite.Web)
MySite.Web.Controllers.HomeController.Index (MySite.Web)

The 2.1 docs on routing say that:

When a request arrives, the routes in the collection are processed in the order that they exist in the collection

Maybe implying that once a route is matched, no further attempt is made.
2.2 docs lack such explanation. Btw, whatever I did to try and reproduce the problem, I failed, even by:
        routes.MapRoute(
                    name: "default",
                    template: "{controller}/{action}/{id?}",
                    defaults: new { controller = "Home", action = "Index" });

        routes.MapRoute( //intentionally creating a second, similar route, trying to cause AmbiguousMatchException
            name: null,
            template: "{controller}/{action}/{id?}",
            defaults: new { controller = "Print", action = "Index" });

Reading the Routing source code, I could not understand the situation when the first match is chosen vs when such AmbiguousMatchException can occur. The compexity is in DefaultEndpointSelector.ProcessFinalCandidates:
        private static void ProcessFinalCandidates(
            HttpContext httpContext,
            EndpointSelectorContext context,
            CandidateSet candidateSet)
        {
            Endpoint endpoint = null;
            RouteValueDictionary values = null;
            int? foundScore = null;
            for (var i = 0; i < candidateSet.Count; i++)
            {
                if (!candidateSet.IsValidCandidate(i))
                {
                    continue;
                }

                ref var state = ref candidateSet[i];
                if (foundScore == null)
                {
                    // This is the first match we've seen - speculatively assign it.
                    endpoint = state.Endpoint;
                    values = state.Values;
                    foundScore = state.Score;
                }
                else if (foundScore < state.Score)
                {
                    // This candidate is lower priority than the one we've seen
                    // so far, we can stop.
                    //
                    // Don't worry about the 'null < state.Score' case, it returns false.
                    break;
                }
                else if (foundScore == state.Score)
                {
                    // This is the second match we've found of the same score, so there
                    // must be an ambiguity.
                    //
                    // Don't worry about the 'null == state.Score' case, it returns false.

                    ReportAmbiguity(candidateSet);

                    // Unreachable, ReportAmbiguity always throws.
                    throw new NotSupportedException();
                }
            }

            if (endpoint != null)
            {
                context.Endpoint = endpoint;
                context.RouteValues = values;
            }
        }

The problems with this code:

As soon as it finds two endpoints with the same score, it throws, but reporting on all endpoints in the set. Not all of them might have the same Score
The Score value in the comments is supposed to only show the order in a CandidateSet, but sometimes it's default (0) and only in one code path it has non-zero value. All in all, the above code looks like a very complex way of selecting the first Valid candidate.

Now about constraints, consider this constraint:
public class MyConstraint : IRouteConstraint
  {
    public bool Match(HttpContext httpContext, IRouter route, string routeKey, RouteValueDictionary values, RouteDirection routeDirection)
    {
      return false;
    }
  }

And these routes:
routes.MapRoute(
                    name: "default",
                    template: "{controller}/{action}/{id?}",
                    defaults: new { controller = "Home", action = "Index" },
                    constraints: new { domain = new MyConstraint() });

If using CompatibilityVersion.Version_2_1, the MyConstraint.Match method will be hit and the route will match.
With CompatibilityVersion.Version_2_2, the MyConstraint.Match will not be  hit and the route matches.
With CompatibilityVersion.Version_2_2, and changing constraints: new { action = new MyConstraint() }) the MyConstraint.Match will be hit but although it returns false, the HomeController.Index endpoint will be selected. With CompatibilityVersion.Version_2_1, none of the routes will match.
Here are my suggestions:

Improving the documentation by explicitly stating the expected behavior when multiple endpoints match (even more explicitly than the 2.1 docs)
Fixing the code to avoid this strange behavior, or if it is by design, emphasizing it.
Fixing endpoint routing constraints to act like the previous version, or documenting the difference.

Thank you very much

I ran into the same issue with code that was working under 2.1.and had to set the compatibility to 2.1 in the Startup.cs file
//.SetCompatibilityVersion(CompatibilityVersion.Version_2_2)
.SetCompatibilityVersion(Microsoft.AspNetCore.Mvc.CompatibilityVersion.Version_2_1)
There is one additional issue with simple two types of inputs with one parameter and one optional not shown. If 'rname' is assigned a value such as 'abcdefg' it should load the variable rname to be 'abcdefg'.
It does return that value but it also has prepended the text of the route so the entire return variable value for 'rname' is now returned as 'rname=abcdefg' and not 'abcdefg'.
I will have to parse the return value and remove the 'rname=' part of the string.
Complex Controller HttpGet's that are similar work fine so it maybe related to 2.2 and the single value with the parameter name included in the route. The first GET call option lacking the parameter name returns the correct value of 'abcdefg'. The 2nd GET format of including the parameter name was to follow a more 'RPC' style RESTful call for some customers who need that type of call format.
[HttpGet("v1/routename2routeidlist/{rname:required}"),
HttpGet("v1/routename2routeidlist/rname={rname:required}")]

I have the same problem, IRouteConstraint doesn't work when compatibility version is 2.2

Apologies for the delay in following up on this issue. This was a bug in the original release of 2.2 and has been patched: https://github.com/aspnet/AspNetCore/issues/4591

I had this problem in Core 3.0.  I finally found the solution was to put a route attribute on the action - e.g. [Route("NodeInfo")]. That fixed it


Not working as of today.

Thought it was the always allow thing but that's not even an option I can find then I read it's not needed anymore.

Hello there, please re-open this issue if your problem hasn't been fixed after updating.
Cheers


ngc / aot compiler errors not being reported

Short description of the problem:
ngc / aot compiler errors not being reported.
What behavior are you expecting?
I expect to see ngc / aot compiler errors.
Steps to reproduce:

Start a new blank project.
Modify home.html to bind to something that doesn't exist like {{ blah.should.fail }}
npm run build --aot

Which @ionic/app-scripts version are you using?
global packages:

    @ionic/cli-utils : 1.3.0
    Ionic CLI        : 3.3.0

local packages:

    @ionic/app-scripts              : 1.3.7
    @ionic/cli-plugin-ionic-angular : 1.3.0
    Ionic Framework                 : ionic-angular 3.3.0

System:

    Node       : v7.8.0
    OS         : Windows 10
    Xcode      : not installed
    ios-deploy : not installed
    ios-sim    : not installed```

This issue is almost 7 months old.  Is this being looked at at all?

I see commit d7c7136 by @danbucholtz has the comment

fix(aot): remove template validation until we can properly handle the error message format

this comment makes it seem like it could be related, except that these issues are much older than the commit. @danbucholtz, can you give any insight into what's going on here?


chore: release version 1.8.0

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.02%) to 92.682%

Files with Coverage Reduction
New Missed Lines
%

box_sdk_gen/internal/base_object.py
2
85.95%

Totals

Change from base Build<PHONE_NUMBER>1:
-0.02%

Covered Lines:
11018

Relevant Lines:
11888

💛  - Coveralls


Question with LogViewer and .csv files

I accidentally opened the .csv files with microsoft excel. And now the LogViewer stop to record any of those logs now. I tried to download a KCV but it didn't work. What should I do to fix that?

You mean while they're opened in Excel or even after you closed them?
If they're not locked by Excel, then Logger will add new entries (you can check KanColleViewer*/Logs/*.csv with any text editor) and LogViewer will display things properly (unless Excel somehow messed up the format, check the files, or move them out of Logs directory).

I'd believe that Excel somehow messed up the whole setting for .csv files. I tried to move them out of the Logs folder, and it won't even create new .csv files any more. And I'm not quite sure how to check if they are locked by Excel.

Should't be locked if you can move/delete them.
So, no BuildItemLog.csv or BuildShipLog.csv when you build item or ship? Even if you delete/backup the whole thing and download and extract KCV into some other folder?

Nope, it didn't work. But the strange thing is that it still won't fix even after I uninstalled Excel from my computer.

Well, maybe it's not Excel then, the Logger and LogViewer both are quite simple, they just write and read files, I don't know what can possibly go wrong here.
Maybe see https://github.com/KCV-Localisation/KanColleViewer/blob/master/FAQ.md, if permissions are right, you can also try to delete %APPDATA%\grabacr.net\ folder, start KCV from its directory and check the settings, including Logging checkbox in Settings -> Other (it's disabled by default).

Yeah, deleting that folder is right! No wonder why it always remembers my proxy server. Thank you! :)


Steamroller issue.

Challenge Name

https://www.freecodecamp.com/challenges/steamroller
Issue Description

The tab freezes and I can't even reset the code on opening the problem again, it seems there is auto-compilation or something going on. Even after waiting for 10 minutes, no response and I have to kill the page.
I somehow bypassed it, by logging in with Firefox and typing out a new code there, not that the earlier code wasn't there by default, and it was just a template when I logged in with Firefox, so there was no freeze.
Browser Information

Google Chrome Version 56.0.2924.87
Windows 7 Professional 64-bit
Desktop
Your Code

> var final=[];
function steamrollArray(arr){
	for(i=0;i<arr.length;i++){
		if(Array.isArray(arr[i])){
			steamrollArray(arr[i]);
		}else{
			final.push(arr[i]);
		}
	}
	return final;
}

steamrollArray([1,[2],[3,[[4]]]]);

Note that, when I ran the sam above code on other online IDE it was smooth and gave an output-
http://prntscr.com/eqn3x9
Screenshot

http://prntscr.com/eqn2fz
This is my first post here, and I am just an amateur.

Thank you for reporting your issue.
Firstly all your links are broken and do not work for me :/
But I can see from your code that you are using a global variable. Programs, that have multiple tests, cannot have global variables. Since the tests test only the function, not the whole code. The final variable won't be empty on the second test, it will have array of the first test inside of it.
So your code is wrong and try to redo it so you are using variable inside a function
Instead of asking for help here, please click the "Help" button on the challenge on freeCodeCamp, which will take you to the help chatroom for that specific challenge. You can also view our full list of official chatrooms.
If you think I'm wrong in closing this issue, please reopen it and add further clarification.
Thank you, and happy coding.


removed some unecessary code and comments in app.js

removed some unecessary code and comments from app.js to clean up code.

force merged.


Remove unnecessary static of not_matched

static here is unnecessary and duplicates the data in every translation unit.

Any chance this can be merged?


Mapping 9009 with "Not found"

I would suggest possibly mapping the "9009" exit code with "9009 (Not found)".

9009 is "'%1' is not recognized as an internal or external command, operable program or batch file."
Part of the intent of flexprompt is that people can write their own modules.
I don't think I want to get into the business of maintaining a list of text abbreviations for various error codes.
Further, a program can return 9009 as its exit code, and that can mean something completely different.
Exit code numbers are contextual.

ok, actually I am not aware of a standard list of error codes on Windows; perhaps the only two ones are 0 and 9009. Thanks, I would close the issue then.

ok, actually I am not aware of a standard list of error codes on Windows; perhaps the only two ones are 0 and 9009. Thanks, I would close the issue then.

Yeah, there are hundreds (probably thousands) of error codes predefined by Windows.  But different APIs return error codes from different predefined sets -- i.e. the same number can mean different things depending on what it came from.
And additionally, each program can define its own return codes.  So given just a number, there's no way to really know what it means.  The HRESULT error code system defines some high order bits that can help to include more context in the error code, but it still only partially improves the situation, i.e. context still matters.
For 9009 specifically:

If the 9009 comes directly from CMD itself, then it means "'%1' is not recognized as an internal or external command, operable program or batch file."
If the 9009 comes from an OS API, then it probably means "DNS server not authoritative for zone." depending on what API was called.
If the 9009 comes from another program, it can mean something entirely different.  E.g. robocopy builds its exit code by OR-ing together bits that mean different things.  The value can't get as high as 9009, but it's a real world example illustrating that exit codes don't necessarily even correspond to error codes.

And there's no way to know where the 9009 came from.

Yes, right, the label related to a 9009 exit code might vary depending on the context. Thanks again for the clarifications.


unwrap panic when resizing an egui window over the top of a graph

unwrap occurs here in graph_view.rs:
if resp.drag_started() {
    if let Some(idx) = self.g.node_by_screen_pos(meta, resp.hover_pos().unwrap()) {
        self.set_drag_start(idx);
    }
}

I can't reproduce it. I resized it in configurable example and everything was fine. Could you please specify the conditions under which panic happens?

Isn't #175 fixes it?


doc: Update quick start with local chart option

Description
What this PR does / why we need it:

breakup quick start steps for clarify
Add more details about dynamic plugin
Add a link notary configuration for cert Store

Type of change
Please delete options that are not relevant.
Doc update
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration
Manually tested local Quick start steps

The ratify dev build PR has been merged and we are all set to generate and publish dev build images. Here's how to use it going forward:

Since the latest dev builds may not match the charts in the previous release, the local chart installation option must be used as described in this PR. (NOTE: if there is a breaking change to the chart, a new dev build should be manually triggered after PR is merged to avoid confusion)
The ratify image and ratify-crds image for dev builds will exist as separate packages on Github here and here.
the repository crdRepository and tag fields must be updated in the helm chart. Please set the tag to be latest tag found at the corresponding -dev suffixed package. An example install command:

helm install ratify \
    ./charts/ratify --atomic \
    --namespace gatekeeper-system \
    --set image.repository=ghcr.io/deislabs/ratify-dev
    --set image.crdRepository=ghcr.io/deislabs/ratify-crds-dev
    --set image.tag=dev.20230307
    --set-file notaryCert=./test/testdata/notary.crt

NOTE: the tag field is the only value that will change when updating to newer dev build images


Modify CREATE / SET/REMOVE / DELETE / MERGE / VLE MATCH

What are the details of this?

please assign

ready for assignment

Hi, I am the new Operations Manager Steve from Vancouver Team.


Hindi Branch automatically redirects to url/hi/hi

Automatic redirection to hi/hi
Steps to reproduce it

Open the hindi branch of the project.
Try to open home page.

Screenshots of the issue

I would like to work on this issue (if needed)

@limzykenneth Kindly let me know if yes :)

This has been fixed on the master branch. Can you rebase or merge the latest master branch and let me know if the problem persist. Thanks.

@limzykenneth I wanted to work on Hindi Internationalization improvements. So, should I continue those changes in master branch?

This should be addressed in #614. You can continue on the hindi branch.


Adds an abstract date selection model

This PR is copied from this commit: f5e4fe31e5740aa03a6a21bfb7dabc6b2c70e41f

Closing, this work is superseded by the work @crisbeto is doing on the date-range branch


Strange bin/obj folders created

I'm not sure if this is an Ionide thing, but lately I've noticed that in my code folder (in which all my repositories live), bin and obj folders keep getting created. I've tracked it down to something to do with Code - I delete the folders, but when I open a new Code, they shortly reappear.
The bin folder contains an empty debug\netcoreapp2.0 folder.
The obj folder contains two files in debug\netcoreapp2.0:
Repos.AssemblyInfo.fs which contains the following:
// <auto-generated>
//     Generated by the FSharp WriteCodeFragment class.
// </auto-generated>
namespace FSharp

open System
open System.Reflection

[<assembly: System.Reflection.AssemblyCompanyAttribute("Repos")>]
[<assembly: System.Reflection.AssemblyConfigurationAttribute("Debug")>]
[<assembly: System.Reflection.AssemblyFileVersionAttribute("<IP_ADDRESS>")>]
[<assembly: System.Reflection.AssemblyInformationalVersionAttribute("1.0.0")>]
[<assembly: System.Reflection.AssemblyProductAttribute("Repos")>]
[<assembly: System.Reflection.AssemblyTitleAttribute("Repos")>]
[<assembly: System.Reflection.AssemblyVersionAttribute("<IP_ADDRESS>")>]
do()

and Repos.AssemblyInfo.cache which just contains a GUID.
Any ideas?

I don't think it has anything to do with Ionide.... maybe it's something with out auto restoring process? Lot of things in Ionide assume that you open root of your repo/solution, not folder that contains lot of subfolders with different repos/solutions.

That's what's strange - I am opening the folder of the repo, and not the parent folder.


Ubuntu 16.04下成功，CentOS 7.4下失败了

感谢大佬的脚本，现在我可以在Ubuntu下使用k8s了！
问题：Ubuntu 16.04下成功，CentOS 7.4下失败了
具体错误信息如下：
Complete!
Created symlink from /etc/systemd/system/multi-user.target.wants/kubelet.service to /etc/systemd/system/kubelet.service.
[123] ## turn off swap for k8s doesn't support it ##
[123] ## configure kubelet for downloading pause image ##
[123] ## and restart kubelet ##
[123] ## we are ready to go; kubeadm init... ##
FirewallD is not running
[init] Using Kubernetes version: v1.10.5
[init] Using Authorization modes: [Node RBAC]
[preflight] Running pre-flight checks.
        [WARNING SystemVerification]: docker version is greater than the most recently validated version. Docker version: 18.06.0-ce. Max validated version: 17.03
        [WARNING FileExisting-crictl]: crictl not found in system path
Suggestion: go get github.com/kubernetes-incubator/cri-tools/cmd/crictl
[preflight] Some fatal errors occurred:
        [ERROR Hostname]: hostname "vm_0_15_centos" a DNS-1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*')
[preflight] If you know what you are doing, you can make a check non-fatal with `--ignore-preflight-errors=...`
[124] ## copy config files ##
cp: cannot stat ‘/etc/kubernetes/admin.conf’: No such file or directory
[124] ## init calico network ##
The connection to the server localhost:8080 was refused - did you specify the right host or port?
The connection to the server localhost:8080 was refused - did you specify the right host or port?

修改hostname后OK了:sweat_smile:


Update run_server.py. metrics_server_config is not supported in JetStream[8128c8a] yet

8128c8a

@FanhaiLu1 Jetstream v0.2.2 was just released, possible to bump from 8128c8a59f859e7691726180b7953e92c6ad4b2b -> v0.2.2?

Closed. Thank you everyone!


Create formula to sort PRs

Since the criteria for sorting the lists of PRs and users are increasing in number, a formula could be introduced to define a metric using those criteria, with different weights, to sort more effectively.

With the addition of #26 , this becomes more necessary, since it is not clear if lines of code or number of PRs constitute more workload, and to what degree.

@all-contributors please add @sntokos for ideas


Allow launching Linux client script from outside its directory

This change updates the Linux client launch script to allow launching it without already being in its directory, to allow launching it via a symlink in the system's path to more easily allow it to be packaged for Linux distributions.
This page gives a good overview of different ways to do this across different operating systems and shells with different levels of compatibility. Since this script looks to be Linux specific, I went with the most compatible Linux-specific option.
Examples:
~> ln -s ~/VPin-Studio/VPin-Studio-linux_x64.sh ~/bin/vpin-studio

Without this change:
~> vpin-studio
tar: zulu11.72.19-ca-fx-jre11.0.23-linux_x64.tar.gz: Cannot open: No such file or directory
tar: Error is not recoverable: exiting now
/home/dmccombs/bin/vpin-studio: line 9: ./zulu11.72.19-ca-fx-jre11.0.23-linux_x64/bin/java: No such file or directory

With this change (launches successfully):
~> vpin-studio
12-06 06:56:23.966 [Studio Resource Updater] INFO  d.m.v.u.StudioUpdatePreProcessing - Finished resource updates check.
12-06 06:56:24.089 [JavaFX Application Thread] ERROR d.m.v.r.c.VPinStudioClient - Get version failed for http://localhost:8089/
12-06 06:56:24.289 [JavaFX Application Thread] ERROR d.m.v.r.c.VPinStudioClient - Get version failed for http://localhost:8089/
12-06 06:56:24.317 [pool-2-thread-1] ERROR d.m.v.r.c.VPinStudioClient - Get version failed for http://localhost:8089/
12-06 06:56:24.318 [Thread-4] INFO  d.m.v.u.l.LauncherController - Checking connection to <IP_ADDRESS>
12-06 06:56:24.359 [pool-3-thread-1] INFO  o.s.w.c.RestTemplate - HTTP GET http://<IP_ADDRESS>:8089/api/v1/preferences/avatar (27ms)
12-06 06:56:24.362 [pool-3-thread-1] INFO  o.s.w.c.RestTemplate - HTTP GET http://<IP_ADDRESS>:8089/api/v1/preferences/systemName (3ms)

Looks good to me, thank you for your contribution!


Convert pkg/controller/gcp/database to new Controller structure/test paradigm

Overview
This issue is a subset of #269

@ichekrygin Is it the GCP package we want to convert, or the Azure one? I thought GCP was the reference "good" implementation?

@negz this issue is just for the database package of GCP.  The other packages in GCP are OK but the database one needs conversion/updating.
Most (all) of the Azure packages also need updating.
The key to look for is if the controller has member fields that are functions and can be set directly for unit testing purposes, e.g., https://github.com/crossplaneio/crossplane/blob/release-0.1/pkg/controller/gcp/compute/gke.go#L75-L78
If a controller has that, then it is following the agreed upon paradigm.

I believe the Resource Groups for Azure will serve as a good example of what converting over the other Azure resources will look like. They are modeled after the GKE controller. @negz feel free to take a look here until merged: https://github.com/HashedDan/crossplane/blob/azure_resource_groups/pkg/controller/azure/provider/resourcegroup.go

@ichekrygin Do you think it's worth keeping this issue open? I believe at this stage many of our controllers need refactoring, so I'm not sure it's worth having an issue for this specific one.

@negz is there a parent issue for that encompassing refactor? This issue could become a bullet of the parent and we could create more vite-size issues (per controller needing refactoring).  These may be good first issues once the pattern is established.

@negz I let you decide. I don't mind either way.

it looks like #269 is the umbrella issue and some related tasks to accomplish that overall vision are being linked there.  I think we should be tracking all of this work at the task level not at the code area level.  So I think we can close this one that is specific to one part of the code area.


test gradient report relative error

It'd be nice to see the relative as well as the absolute error.  In looking over some widely scaled problems, it's hard to see where there might be na error.  Output now looks like:
...
param idx           value           model     finite diff           error
...
        65        -1.30092     5.49828e+07     5.49828e+07       0.0253369
        66         1.59305        -38942.9        -38948.4         5.50558
        67        -1.55866        -18093.3        -18093.3      0.00672078
        68        -1.51286     7.86573e+07     7.86573e+07          -2.525
        69         1.76965         6024.47         6024.45       0.0116251
        70        0.610684    -1.20547e+08    -1.20547e+08        -1.35109

I think it makes sense to define relative error this way:
(a - b) / ((abs(a) + abs(b) / 2)

We may already have a function lying around to do this.

[ ] add relative error
[ ] modify manual examples

I don't think this one's going anywhere due to the vagaries of finite diffs.  If people need relative errors, they can eyeball them easily enough.  In the end, we want to produce all of this in a more structured output than just printing.


[Task]: Add a request timeout for test data generator call

Description:
In the current implementation when a call is made to the test data generation backend, we get time out after 30s. But that time is too long and it will be bad for user experience. So need to have a time from the client side (vsCode). waiting time should be 15 seconds.
Describe your task(s)
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

This issue is fixed in the PR https://github.com/wso2-enterprise/ballerina-plugin-vscode/pull/793


Ensure authority and/or host fields are not empty

We could have an authority or host that consists of an empty string

I am unsure how to correctly to create a HeaderField with an empty value for the tests. If that is even possible.  If it isn't then maybe this PR is redundant and I should only update the compliance?

Hi, thanks for the PR!
You are using the try_from method in the test to create the header. This method is also used wen receiving a request.
try_from parses the header field to a typed header field from the http crate which allows no empty values.
I would leave the check and the test for the host field where it is.
For the authority field, i would copy the compliance comment here and assert the result of try_from instead of into_request_parts

@Ruben2424 as it turns out, once I got the tests working I discovered that the initial code already errors out when either field is empty. The host field errors out when it is built into a Uri since the Uri Builder rejects empty Uris. At least we have more tests and documentation now


Fix issue with invoking next middleware.

If websocket is handled, we don't want any further middleware. Only non-websocket requests should be passed to next middleware.

The same check is at the start of the Invoke method:
		if (!context.WebSockets.IsWebSocketRequest)
		{
			return;
		}

Hi!
Thank you very much for the contribution you provided so far!
Sorry for delaying this, I have been focusing on different projects lately, but I would love to get back working on this.
In order to do this, I would love to enable you people to contribute directly to the projects. Please let me know if you want to be granted write access to branches of this repo.
Thanks,
Radu M

@radu-matei Hi, yep. Thank you.

Ok.
Since there will be no support for .NET Core before 2.1 for SignalR (will document this soon), I believe there is a strong point in continuing support for this library.
I added people who responded as contributors, guidelines coming soon :)
Thanks!


Race condition leading to unpopulated items returned by promise

We have a query (that we run repetitively) that resolves an array of items each of which also has recursively resolving parents. We encountered a race conditions where sometimes some of the items in the array have parents that are not fully resolved. These parents have been constructed but the data for them has not been loaded.
Looking at the code, it seems most likely that the race condition happens because:

Items are added to PotionBase.cache as soon as the constructor is initialized
The resolution of children uses PotionBase.fetch()
PotionBase.fetch() returns items from the cache, regardless of whether their promise has been resolved or not.

Assuming that circular references don't happen, the problem could be solved by placing a promise for the item (rather than the item itself) onto the cache.

@rolandjitsu I considered that as well. It might solve the specific issue we're experiencing, but it would still leave a race condition because there could be a pending request for a list of items.

Note that the fix for this is not yet published on NPM due to the issue with cross references.


Adopt black for code formatting

I'm -> <- this close to accepting to relinquish control of the code formatting to an automated tool.

Done in 1928ad71c0d97be6f7aeac84fc4c6605f3a7bb32.


Meilisync Not Resyncing After Deleting Documents

Hello!
I'm having trouble with Meilisync after deleting all documents in Meilisearch. It seems like Meilisync isn't picking up the changes and resyncing the data.
Is there a way to manually trigger a resync in Meilisync after deleting documents in Meilisearch, or is there a specific configuration needed to handle this scenario?

Hi @n-jaisabai, I think you are misunderstanding how Meilisync works.
Meilisync listens for changes in your database, not in Meilisearch, so the sync would be triggered if you delete something in your database, not in Meilisearch.

Thank @matinone, for your response.
I understand that Meilisync doesn't listen for changes in Meilisearch. I just want to resync the database from the beginning.
I stopped and restarted Meilisync, expecting it to resync the data, but it doesn't seem to be working.
Is there any way to resync from the beginning?
Database: PostgreSQL 14


UPDATE: Add submit to Button, ButtonLink

Added submit prop to Button and ButtonLink

@wEprous Can we cover more button types ? button submit submit ?

Did you mean reset? We talk about it with @darkwindcz and we don't to provide this behavior.


[stable/aws-es-proxy] HTTP request signing reverse proxy for Amazon Elasticsearch service

Manually verified to work by installing it with:
$ helm upgrade esproxy ./ --install \
  --set config.awsSecretAccessKey=MYSECRETKEY \
  --set config.awsAccessKeyID=MYKEYID \
  --set config.endpoint=https://your-aws-es-domain \
  --set config.verbose=true \
  --set podAnnotations.foo=bar

From another ubuntu pod, fetching elasticsearch indices on AWS Elasticsearch Service cluster:
root@xenial-1527645050:/# curl esproxy-aws-es-proxy:9200/_cat/indices
yellow open myindex ogrlJsTIQrSJoPvnNMhxxw 5 1 271 0 256.5kb 256.5kb
green  open .kibana    NETaiYeLT-iSu87VSucE9w 1 0   3 1  15.6kb  15.6kb

And then reading logs:
$ stern es-proxy
esproxy-aws-es-proxy-86c8cdc58b-ld6bv aws-es-proxy 2018/05/30 02:14:06 Listening on <IP_ADDRESS>:9200...
esproxy-aws-es-proxy-86c8cdc58b-ld6bv aws-es-proxy 2018/05/30 02:14:09 Generated fresh AWS Credentials object
esproxy-aws-es-proxy-86c8cdc58b-ld6bv aws-es-proxy 2018/05/30 02:14:10  -> GET; <IP_ADDRESS>:54772; /_cluster/health; ; 200; 0.328s
*snip*
esproxy-aws-es-proxy-86c8cdc58b-ld6bv aws-es-proxy 2018/05/30 02:33:36  -> GET; <IP_ADDRESS>:46446; /_cat/indices; ; 200; 0.030s

Templates and values.yaml is mostly similar to stable/oauth2-proxy's.
The main difference is config: in values.yaml, which directly reflects aws-es-proxy's own configuration file.

/assign @foxish

@foxish Would you mind taking a look at this, or perhaps assign another reviewer if you have any trouble? 🙏 Thanks.

Could anyone review this?

@mumoshu this is a new chart, the title should say that.

please rebase against master and sign the DCO
The chart version should be at least 1.0.0 since you are pointing that to master
awsKey and secret are required. Add the require in the template to make sure the user is setting that

@cpanato  I don't think aws key and secret should be required, kube2iam can provide aws credentials in aws k8s clusters.

@mumoshu I think this chart would be very useful too. Hope this chart merged soon

This should be good to go now.

/ok-to-test


runtime: Reducing theoretical heap fragmentation

It is possible that after one or more GC cycles a span may contain only a single allocated object. After the program makes a phase change that size object may become unpopular and there is no need to allocate in that span again. This would be detectable after several GCs cycles as a span that has not been allocated in. It would be really nice if we could move that object and use that span for a different size class. That approach is often touted as an advantage of a moving collector. It is important to point out that this is merely a theoretical problem and currently there is no indication of it in the wild in an important application. Someday if it becomes a problem this issue offers a possible solution.
The perhaps novel idea is that we morph the span into a span with a larger size class according to the needs of the new phase. The single allocated object remains where it is but the garbage collector treats it as a larger sized object based on the span's new class size. There will be (new_size - old_size) fragmentation for that single object but that seems a small price to pay. The rest of the span would be used to allocate objects of the new popular size. Not only would this work for a single object but for any sparsely populated span where sparsely is determined by some TBD heuristic.
What if the original object straddled two of the new size class' objects? This could be easily avoided by selecting some other size class. But assuming we didn't want this restriction. If there is space both before and after the object in the new size class simply put a pointer in those free slots to the other part of the object and adjust the GC pointer map. The result would be that the GC would never free the object until both halves were no longer live.

What if the original object straddled two of the new size class' objects? This could be easily avoided by selecting some other size class. But assuming we didn't want this restriction.

An alternate approach would be to introduce a new special that links the slots together. Sweeping already has to scan all of the specials, so it would be easy and inexpensive for it to also handle these specials.
This would also make it possible to transition a span from a larger size class to a smaller size class.


Internal parser: Episode titles that start with numbers are parsed as a pack

Expected behaviour:
It parses the title as an episode and not a pack.
Actual behaviour:
It parses the episode as a pack eg. Some.Show.S01E01.90.days is parsed as S01E01-90.
Steps to reproduce:

Step 1: Run the config.

Config:
tasks:
  test_parser:
    mock:
      - {title: 'Some.Show.2017.S01E<IP_ADDRESS>0p.H.264-NOGRP'}
    series:
      - Some Show (2017)

Log:
Away at the moment, but to paraphrase: It is being parsed as a pack and rejected for being too large eg. S01E01-11. https://github.com/Flexget/Flexget/blob/develop/flexget/utils/titles/series.py#L329-L330

Any update on this? Still not working, entries undecided.
Config:
tasks:
  test_parser:
    mock:
      - {title: 'Gotham.S05E06.13.Stitches.1080p.NF.WEBRip.DDP5.1.x264-CasStudio[rartv]'}
    series:
      - Gotham

Log:
http://dpaste.com/14P15W1

Anything?


The "riscv64-linux-x86_64-20200702.tar.gz" package doesn't exsit.

I followed the "Build boot.ext4 (kernel + opensbi + dtb) & rootfs" steps and met the following errors:
>>> toolchain-external-custom  Downloading
--2023-07-17 00:54:53--  https://occ-oss-prod.oss-cn-hangzhou.aliyuncs.com/resource/1356021/1594381796470/riscv64-linux-x86_64-20200702.tar.gz
Connecting to <IP_ADDRESS>:7890... connected.
Proxy request sent, awaiting response... 404 Not Found
2023-07-17 00:54:53 ERROR 404: Not Found.

--2023-07-17 00:54:53--  http://sources.buildroot.net/toolchain-external-custom/riscv64-linux-x86_64-20200702.tar.gz
Connecting to <IP_ADDRESS>:7890... connected.
Proxy request sent, awaiting response... 404 Not Found
2023-07-17 00:54:54 ERROR 404: Not Found.

--2023-07-17 00:54:54--  http://sources.buildroot.net/riscv64-linux-x86_64-20200702.tar.gz
Connecting to <IP_ADDRESS>:7890... connected.
Proxy request sent, awaiting response... 404 Not Found
2023-07-17 00:54:54 ERROR 404: Not Found.

make[2]: *** [/home/[myusername]/buildroot/thead_9xxf_enhanced_5.10_glibc_br_defconfig/build/toolchain-external-custom/.stamp_downloaded] Error 1
package/pkg-generic.mk:144: recipe for target '/home/[myusername]/buildroot/thead_9xxf_enhanced_5.10_glibc_br_defconfig/build/toolchain-external-custom/.stamp_downloaded' failed
make[1]: *** [_all] Error 2
Makefile:23: recipe for target '_all' failed
make[1]: Leaving directory '/home/[myusername]/buildroot/thead_9xxf_enhanced_5.10_glibc_br_defconfig'
make: *** [all] Error 2
Makefile:63: recipe for target 'all' failed

I tried to use explorer to search this package in those source urls, but didn't find it, too. How can I find a substitude toolchain?
thanks in advance.

Hey @zhuotianshu,
I experienced the same problem, maybe I can share a few tips to help you.
You may go into configs_enhanced/thead_9xxf_enhanced_5.10_glibc_br_defconfig and replace the following line with a new source (this one worked for me):
BR2_TOOLCHAIN_EXTERNAL_URL="https://datashare.ed.ac.uk/bitstream/handle/10283/4835/Xuantie-riscv64-linux-x86_64-20210618.tar.gz"
I had a similar issue with the zero_stage_boot package, whose link was broken and unfortunately I had to remove in order to continue the build.
Also, note that another user left some useful tips on their gitlab page: https://gitlab.com/c-sky/buildroot/-/issues/46


Amplify UI gen2 will not offer to create service role with monorepo option

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[X] I have searched for duplicate or closed issues.
[X] I have removed any sensitive information from my code snippets and submission.

Amplify Hosting feature
Monorepo
Is your feature request related to a problem? Please describe:
The UI has a bug. This is NOT a feature request.
If user selects monorepo while connecting repository, the UI skips creating service role. Which leads to deployment failures.
Second related problem, there is no option to set role in the UI (the respective edit button simply doesn't show anything role related in the UI).
Workaround is to create and set role manually
aws amplify update-app --app-id xxx --iam-service-role-arn arn:aws:iam::xxx:role/AmplifyDeployments --profile app-prod
Resources:
  AmplifyDeploymentRole:
    Type: AWS::IAM::Role
    Properties:
      RoleName: AmplifyDeployments
      AssumeRolePolicyDocument:
        Version: "2012-10-17"
        Statement:
          - Effect: Allow
            Principal:
              Service: amplify.amazonaws.com
            Action: sts:AssumeRole
      ManagedPolicyArns:
        - arn:aws:iam::aws:policy/service-role/AmplifyBackendDeployFullAccess
        - arn:aws:iam::aws:policy/AmazonSSMFullAccess <-- this may be optional depends how your pipeline deployed stuff

Overall, not a good experience since it was hard to figure out why deployments were failing. They were using some default role from amplify-owned account.
Describe how you'd like this feature to work
Document when and why this role should be created, what permissions it needs, what options users have to create it etc.

Hi @OperationalFallacy 👋 , When testing with an Nx monorepo workspace containing a React app and an Amplify Gen 2 backend, I could not reproduce the outlined behavior. Instead, I was able to create a new service role during the app creation flow. Please refer to the video below:
https://github.com/aws-amplify/amplify-backend/assets/38470430/5107d25f-15fc-47fe-baf1-192c506254ad
To investigate further, it would be helpful if you could provide the following:

Detailed reproduction steps, including the commands and configurations you used in your monorepo
Your project structure
Any additional context or relevant information about your setup.

Please feel free to share any other relevant screenshots or information that could aid our investigation. Thanks!

@Jay2113 thank you for checking this
Probably monorepo option is unrelated
I don't see service role option with any selection

@OperationalFallacy can you share the package.json file which is present at the root of your project/repository?

@Jay2113 is it ok if I give you access to the repo to check layout? Its in early stages of development, there is not much code yet

Hi @OperationalFallacy 👋 , we have recently released improvements to the app create workflow. Can you confirm if you are still experiencing this issue? If yes, can you please provide us access to the repository so we could investigate into it further.

Closing this thread since we are no longer able to reproduce the outlined issue. If you are still experiencing it, feel free to reopen the issue and we can investigate into it further.


Factor quantum_info-dependent code out of opflow.AbelianGrouper

The code in abelian_grouper.py does not depend in an essential way on any part of opflow. It could be factored out and moved to quantum_info, or to some organized code whose only qiskit dependency is quantum_info.  The opflow class could then call this factored-out code.

Let me link this since it was about the grouping code Qiskit/retworkx#347 in that the groups being returned is not deterministic and under Aer, which uses the seed with RNG once per job, a change in ordering of groups, and hence the circuits that were executed to measure their expectations, led to different unit test outcomes.

While we working on the retworkx issue couldn't you just do sorted() on the output dict? The issue is the insertion order is unordered (because the underlying result object is a HashMap that doesn't define an order) so iteration order is different but the output should be deterministic, it's just a different insertion order.

@irajput will be working on this.

Thanks. Nice proposal. I think it is better after the internal data of SparsePauliOp is replaced by PauliList.

FYI This is the issue Matthew mentioned https://github.com/Qiskit/retworkx/issues/347

I think it is better after the internal data of SparsePauliOp is replaced by PauliList

Yes. The first step would be to implement grouping for PauliList in quantum_info. This is independent of opflow.

Since OpFlow is deprecated, reorganizing grouping of Paulis is better addressed elsewhere.
Note however that a part of this work that will continue to be useful was done in #6690


User Avatar does not fall back on default when offline, and instead a blank space is shown.

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:

Force close the app
Turn on airplane mode
Open the app

Expected Result:
App should load on offline mode and user avatar should be displayed
Actual Result:
App is not loading the user avatar on offline mode. A blank space is displayed.
Workaround:
Going back online fixes the issue.
Platform:

Where is this issue occurring?

iOS
Android
Desktop

Version Number: 1.1.8-0
Reproducible in staging?: Yes
Reproducible in production?: Yes
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

When testing this, I noticed on my device that the avatars from the LHN conversation list were also not loading

Expensify/Expensify Issue URL:
Issue reported by: @roryabraham
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1633893109079200
View all open jobs on GitHub
view this job on upwork

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

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

@Luke9389 do you have an opinion on the pending proposal?

@developvsn if you haven't applied to the upwork job, can you do so? If you have applied, can you let me know who you are on UW so I can hire you? you can also message me through the job posting on UW and let me know your gh username if you'd prefer.

Hello @Luke9389 , we started the PR (branch fix-default-avatar) !
Hello @jboniface, we'll message you on upwork , we haven't applied yet, thanks !

@developvsn we noticed the PR was submitted by @RGutu. Our process for managing and paying contributors is setup so that the same account that is hired in the Github issue is the same who submits the PR. Can you use the same accounts in the future?

@mallenexpensify okay, we just wanted to submit it from our organisation account, and it didn't work, so in the future we'll use same accounts , understood.

Thanks @developvsn , it'll make our lives much easier.
For this issue, can you apply to the Upwork job here https://www.upwork.com/jobs/~01382d9aca711ad10a then confirm in a comment here once you have.  Please include your first name and initial of last name so we can ensure we hire the correct person.
I'm also going to confirm in the PR that payment will be going to the user in Upwork to ensure wires don't get crossed.
Thanks!

@mallenexpensify
We applied to the Upwork job. The name on upwork is: Alexandru. S.

We are placing all non-critical issues on hold while we're on a company offsite.  The hold is expected to be lifted on 11/19 (but could go longer).  For any PRs that are submitted during the hold, we will add a $250 bonus payment.

temporary hold, not overdue

I hired Alexandru @developvsn

📣 @developvsn You have been assigned to this job by @jboniface!
Please apply to this job in Upwork and let us know when we can expect a PR to be ready for review 🧑💻
Keep in mind: Code of Conduct | Contributing 📖

@Luke9389 Hi ! Can you please review or give us an update regarding Pull Request

@mallenexpensify okay, we just wanted to submit it from our organisation account, and it didn't work, so in the future we'll use same accounts , understood.

@developvsn as per your comment here, I have tested this on Staging iOS app and the issue still prevails. Therefore, we should still go ahead with trying to solve this or find out why is it it should work on production but not on staging (I only have staging on my phone right now, so I can't test production behaviour).

@mountiny @jboniface  @Luke9389  We downloaded production app for 3 different IOS devices and 1 android, and we can't reproduce this bag.
IOS
https://user-images.githubusercontent.com/71173662/142903856-b9380962-20ed-46e0-97a4-65e4118a7cba.MP4
Android
https://user-images.githubusercontent.com/71173662/142903920-75ea71b4-2b7e-4530-8463-7eb59f205f6f.mp4
We also tried to do it in github source code and it's the same result:
https://user-images.githubusercontent.com/71173662/142904731-cb82d08f-1f7b-446b-b3f3-46cb0cac9d39.mp4

This is my experience on Staging. I guess we need to test more. Why would it occur only on staging but prod and dev would be fine, interesting 😅
https://user-images.githubusercontent.com/36083550/142930385-7ca83875-5a1c-476f-96c9-4823b42eafa8.MOV

For now, I'm going to put this issue on HOLD and then have this retested after the next deploy.
Frankly, if this issue fixed itself, that'd be ideal, because the changes in the PR don't feel optimal to me. More discussion on that if need be.
Rest assured @developvsn, we'll pay you for your time regardless of whether your PR is merged.

@isagoico Would you be willing to have this tested again? It looks like we had a successful deploy today, so we should have a clear idea of whether this is still occurring.

Of course! happy to try and repro again :)
I was unable to reproduce this on Android on build 1.1.16-1. I currently don't have my iOS on me today so I'll check this tomorrow too (since it was the device I was able to reproduce initially).

Of course! happy to try and repro again :)
I was unable to reproduce this on Android on build 1.1.16-1. I currently don't have my iOS on me today so I'll check this tomorrow too (since it was the device I was able to reproduce initially).

Have you had a chance to look into it?

I was unable to reproduce the Profile image issue on the main user but I was able to reproduce the missing profile picture in LHN conversation. Please check the video and image below
https://user-images.githubusercontent.com/44479856/143454448-c2daf154-5ccd-4550-b915-a43a968e0e4b.MP4

@isagoico It might be that the main user profile picture in your case is the default one, or at least I am thinking it might be a different scenario since originally Rory and then when I tried it, we both had a profile pic uploaded.

Then what's the plan? Do we implement changes or keep it like this?

I just tested, it looks like MY avatar is showing but not all the rest.  FWIW...

It looks like this is still being discussed?

It looks like this is still being discussed?

sort of. Waiting on Luke's comment here
It seems like it need more testing — looks like some devices loading avatars in offline and some not.

@jboniface, can we pay @developvsn for their time?
I'd like to close this issue and open a new one with a more clear description of the problem and solution.

@jboniface, can we pay @developvsn for their time?
I'd like to close this issue and open a new one with a more clear (and up-to-date) description of the problem and solution.

Any updates? @jboniface

@developvsn sorry, i missed that message. i just paid the contract. closing.


TLS support for subscribing

Hi,
The broker that we are using needs TLS ca_certs, cert, keyfile. I dont see any option in Connect, subscribe/subscribe and validate keywords that lets me specify these anywhere (unlike Publish Single).
I can also see that  https://pypi.org/project/paho-mqtt/#single shows 2 functions for subscribe simple and callback, but https://github.com/randomsync/robotframework-mqttlibrary/blob/master/src/MQTTLibrary/MQTTKeywords.py doesnt
Is it possible to add this option,  because using just publish is of no use if we are not able to subscribe and retrieve what we we have published? Thanks

Well you can modify the existing connect function inside the library.
Let me know if you need it, i can revert back with the code !

Thank you, Unfortunately, I don't have enough support from the dev team @ my workplace. Is it possible for you to help me with this? Thanks!

I also have the same problem, I need to pass 3 certs file inorder to connect to MQTT broker.
Can you give me the code for that?

I was wondering how I missed adding the simple() keyword, but then realized that the Subscribe module was added after majority of the code for this library was written and I haven't had time to add any new features since then.
I can add it, however pull requests are welcome.


Resolving extract folder name conflicts

Type of Change

[x] Bug fix (non-breaking change, fixes #5447 )
[ ] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[x] Pester test is included
[ ] If new file reference added for test, has is been added to github.com/sqlcollaborative/appveyor-lab ?
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Purpose
Fix the issue
Approach
Adding random guids to the folder name every time
Commands to test

Screenshots

Learning

fabulous, thank you @nvarscar !


Update background.js

Just a minor fix to return error if the sender ID has not been recognized.

It is not responded deliberately so that a malware can't scan the senderID.

That makes sense.
On the other hand, there's no way to handle the error if that's not a malware scanning but another extension trying to send the message.
Perhaps we could consider it an option to choose whether such response should be sent or not, what do you think?

After all, I thought that you had to fork this repository and run it locally to develop the extension integration. Could you actually develop it with the extension you installed from the store?

The top priority was to stick to the original extension from the store, the thing is I got into trouble handling this error so decided to fork and test the easiest solution - this PR is the result of that and only that. Although it's not critical to have it handled at all, I wanted to make it less troublesome for anyone using it (and providing an error that the extension IDs do not match may be quite important imo, my bad I didn't consider malware scanning).
I truly hope to finish what I started and release it to the public soon.

Please have a look at https://github.com/netguru/sync-aws-extend-switch-roles - any feedback is more than welcome!


Add valid until for account update

Explain your changes:
This PR adds valid_until precondition to zkapp account-updates. This precondition would be compared against the global_slot that the block was producted. Further more, this PR changes the staged_ledger.create_diff and staged_ledger.apply_diff functions that makes it accept the current global_slot as one of the arguments. This global_slot would be used to compute vrf and check against the valid_until precondition. This PR also adds global_slot to pending_coinbase.
Explain how you tested your changes:
*
Checklist:

[ ] Modified the current draft of release notes with details on what is completed or incomplete within this project
[ ] Document code purpose, how to use it

Mention expected invariants, implicit constraints

[ ] Tests were added for the new behavior

Document test purpose, significance of failures
Test names should reflect their purpose

[ ] All tests pass (CI will check this if you didn't)
[ ] Serialized types are in stable-versioned modules
[ ] Does this close issues? List them

Closes #0000

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me


Forked block data is missing block_datetime column

Workflow updated to prevent this going forward. Backfilled the missing column in fc294910717e433e4e810a06589f8fff9f2c7d20.


[master] Bump Mesos to nightly 1.8.x 2f8c3d1

High-level description
This is a routine bump to the latest Mesos and mesos-modules.
Related JIRA Issues
Checklist for all PRs

[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Changelog:
https://github.com/apache/mesos/compare/2c7cee0fded247da6f8f66e75fc759f0b11be42f...2f8c3d17691df767bb0fac1274077bbc8b6da0f0

[ ] Test Results: [link to CI job test results for component]

[ ] Code Coverage (if available): [link to code coverage report]

@mesosphere-mergebot bump-ee


Made the color of the scroll marker the same as the editor marker

ok ...

Feel free to update. I've moved master to main and it's now in JS rather than CoffeeScript.


this removes func format from everything

format is dead now, so we can remove all the flags and docs about it here.
technically, we should preserve this in old functions format to migrate,
however since there's no field to migrate to, it doesn't seem worth the
cognitive baggage?
there's a test that all our runtimes work off our generated func.yaml and
boilerplate, so that should prove that this change is ok.

thank you


Set a GPT label for EFI partition

This allow to easly update the EFI loader and sync with a standard FreeBSD install.

We probably should use the same name as the installer and iirc we don't use "efiboot0" there.

I believe we're standardizing on efiboot0 everywhere.
IIRC old versions of bsdinstall used efiboot, but when multi-esp support was added, it was added as efiboot%d starting at 0.

I believe we're standardizing on efiboot0 everywhere. IIRC old versions of bsdinstall used efiboot, but when multi-esp support was added, it was added as efiboot%d starting at 0.

Yes indeed.


Fix underlay BGP and description for overlay_controller uplinks

Change Summary

Fix underlay BGP from upstream device to overlay_controller
Types of changes

[x] 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 change)
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes)
[ ] Documentation content changes
[ ] Other (please describe):

Related Issue(s)

Component(s) name
eos_l3ls_evpn
Proposed changes

How to test

Tested with my test repo.
After adding this PR but not changing anything else we now get:
   neighbor 172.32.255.x peer group IPv4-UNDERLAY-PEERS
   neighbor 172.32.255.x remote-as 65100
   neighbor 172.32.255.x description DC1-RSx

on the relevant leafs.
Checklist:

[X] My code has been rebased from devel before I start
[X] I have read the CONTRIBUTING document.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have updated molecule CI testing accordingly
[ ] All new and existing tests passed (pre-commit, make linting and make sanity-lint).

LGTM!

LGTM!


Implements API for connecting to TCP servers in workers

Depends on https://github.com/capnproto/capnproto/pull/1567.
This PR implements a new connect API which establishes a connection with a TCP server, then exposes that connection through two streams. In workerd this connection can be established directly or via a HTTP proxy. Examples of both are demonstrated using a new sample which implements a Gopher client.
Test Plan
Testing using the sample provided:
$ bazel run //src/workerd/server:workerd -- serve $PWD/deps/workerd/samples/tcp/config.capnp --watch --verbose

Testing using custom code:
$ bazel run @workerd//src/workerd/server:workerd -- serve $PWD/my-config.capnp --watch --verbose
$ tinyproxy -d -c tinyproxy.conf
$ nc -l 6666

Verified that streams get closed properly when nc is terminated.
Verified that sockets can be closed.
Verified that exceptions can be caught.
Verified that all of the above works using a proxy.

CI is failing because the types generator is failing with:
AssertionError [ERR_ASSERTION]: Unknown structure type: workerd::api::SocketOptions

@mrbbot is this an issue with the script or do I need to add info about this type somewhere?

@mrbbot Ahh! Thanks for the pointer. :)

Last push was a rebase.

Last push was a rebase.


lodash is >90% of this library size

This library is good and easy to use but it put all lodash into my bundle
<EMAIL_ADDRESS>

Pull request welcome :)

okay will do it on this week I think

you can use size-limit to avoid of this problem


why flash can't accelerate on A40 machine?

Hi  i tested flash-attn operation on an A40 machine and it showed no improvement on trainning speed , Moreover , I printed out the time cost of self-attention calculation part between two models . One model used normal attention , the other used flash-attn. Aart from that ,these two models are consistent.  So can flash-atten only accelerate trainning speed on A100, but A40 can't?  by the way,flash-atten did reduce the cuda memory usage!
paste the env and other configurations:
work: speech recognition
env: cuda12.1   torch2.1.2   flash-atten 2.5.2
models:  11*conformers
operation used: flash_attn_qkvpacked_func

Please benchmark just the attention operation


bump to 3.9, drop 3.6

Codecov Report

Merging #270 (cd0a948) into main (64192e2) will decrease coverage by 0.68%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #270      +/-   ##
==========================================
- Coverage   69.78%   69.10%   -0.69%
==========================================
  Files         119      119
  Lines        6348     6208     -140
  Branches      508      508
==========================================
- Hits         4430     4290     -140
  Misses       1794     1794
  Partials      124      124

Impacted Files
Coverage Δ

pyEX/refdata/symbols/options.py
31.57% <0.00%> (-6.52%)
:arrow_down:

pyEX/refdata/symbols/futures.py
46.66% <0.00%> (-6.28%)
:arrow_down:

pyEX/refdata/symbols/crypto.py
53.84% <0.00%> (-6.16%)
:arrow_down:

pyEX/premium/stocktwits/__init__.py
26.66% <0.00%> (-4.59%)
:arrow_down:

pyEX/metadata/__init__.py
35.71% <0.00%> (-4.29%)
:arrow_down:

pyEX/refdata/ric.py
71.42% <0.00%> (-3.58%)
:arrow_down:

pyEX/refdata/isin.py
71.42% <0.00%> (-3.58%)
:arrow_down:

pyEX/account/__init__.py
61.29% <0.00%> (-3.42%)
:arrow_down:

pyEX/refdata/search.py
66.66% <0.00%> (-3.34%)
:arrow_down:

pyEX/fx/fx.py
62.79% <0.00%> (-2.43%)
:arrow_down:

... and 34 more

Continue to review full report at Codecov.

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


DFP contains/include operator (~) doesn't work?  (Possible DFP issue?)

First off, a huge thanks for your work on this.  It's made working with DFP enjoyable, which is no small feat!
It's possible this is 100% a DFP issue, but thought i'd ask here in case anyone else had run into it.
I'm trying to target a line item using the UrlPath custom variable with the 'include/contains' (~) operator.  My UrlPath is /2015/07/politics-election/foo/ and my criteria is set to 'UrlPath is ~politics-election'.  I've confirmed that the UrlPath variable is being set by jquery.dfp.js by looking at the delivery diagnostics.
No matter what I try DFP wont target the campaign.  Fiddling with delivery diagnostics, it seems to me that the ~ operator doesn't work.  DFP will only target the exact phrase (in my case 'politics-election'), not a segment of the UrlPath.
Has anyone else run into this or been able to get contains/include to work?

Hah!  Just received this in an e-mail from DoubleClick.  Definitely nothing to do with dfp.jquery:
"Dear Customer,
We are currently experiencing an issue that is causing DoubleClick for Publishers Query Tool “contains” filters to be case sensitive. We are working to restore full functionality of the affected filters.
At this time we do not have an estimate for when this issue will be resolved. We will send an update by 5PM ET tomorrow, Thursday, July 16th.
Please note that ad serving is not affected.
We thank you for your patience and apologize for any inconvenience this may cause.
Regards,
The DoubleClick for Publishers Team"

Yeah I just got that email as well! Sorry for the slow reply... but I guess this is the cause :smile:

No worries at all!  Thanks so much for your work on this, it's much appreciated.

Hmm, just got another e-mail from them about this, and i'm worried that they're only fixing this in the query tool.  I'm going to go try my hand on the Google Product forums...
"Dear Customer,
We are continuing to work to address the current case sensitivity of the DoubleClick for Publishers Query Tool “contains” filters.
We expect to resolve the issue by Tuesday, July 21st. Please note that this time frame is an estimate and may change.
Please note that ad serving is not affected.
We thank you for your patience and apologize for any inconvenience this may cause.
Regards,
The DoubleClick for Publishers Team
"

Just got the email saying this is resolved... is this issue fixed now?

Apologies for not responding to this sooner.
It doesn't look like Google/DFP actually fixed this.  If I set the targeting criteria to ~/test-page and the url is http://www.url.com/test-page the targeting works.
If I set the criteria to ~page and visit http://www.url.com/test-page the targeting does not work.
This is 100% a DFP issue as far as I can tell.  You can go ahead and close if you want.
Maybe I'm just reading their documentation wrong?
https://support.google.com/dfp_premium/answer/177381
"Include" match
Use a tilde (~) at the beginning of a value to include values that have the entire word in them, in any location. For example, ~travel would match values like adventure travel and travel insurance, but not travelers or travelers insurance.

Hi, i read the docs as the word "travel" is matched, not the string "travel". As stated, "travel" is not matched in "travelers" or "travelers insurance". It seems to be implemented as (pseudo code):
function tildeMatch(haystack, needle){
if(needle.substring(0,1)=='~')needle=needle.substring(1);
var words = haystack.split(' ');
for(i=0;i<words.length;i++){
if(words[i]==needle) return true;
}
return false;
}

On 26/08/2015, at 21.31, creinhardt<EMAIL_ADDRESS>wrote:
Apologies for not responding to this sooner.
It doesn't look like Google/DFP actually fixed this. If I set the targeting criteria to ~/test-page and the url is http://www.url.com/test-page the targeting works.
If I set the criteria to ~page and visit http://www.url.com/test-page the targeting does not work.
This is 100% a DFP issue as far as I can tell. You can go ahead and close if you want.
Maybe I'm just reading their documentation wrong?
https://support.google.com/dfp_premium/answer/177381
"Include" match
Use a tilde (~) at the beginning of a value to include values that have the entire word in them, in any location. For example, ~travel would match values like adventure travel and travel insurance, but not travelers or travelers insurance.
—
Reply to this email directly or view it on GitHub.

Yeah its looks like an their implementation only matches whole words... I think @soreng is right... probably not a lot we can do here... you could probably come up with your own custom function to take the URL and extract every possible combination of words you might want to match... it would be kind of similar to the way I used to have to things before the contains and starts with operators became available...
https://github.com/coop182/jquery.dfp.js/blob/dbb03002e9e6b2d1150848e6a91b00c97406285a/jquery.dfp.js#L319
I will close this for now... but feel free to reopen it if you think there is anything that can be done to improve things from this end.


InvokeSync and InvokeAsync instead of Invoke

Currently, we have a single system call for invoking other functions: Invoke, which is synchronous and returns a response.
More useful (and simpler to implement?) would be an asynchronous version that that doesn't include responses. For example, in a chain for functions, we don't want the second to last function to wait for the last function, the third to last to wait for the second to last (and implicitly the last), etc... Instead, we want to invoke the next function in the chain, asynchronously, and return.
A synchronous version may still be useful, so I propose:

InvokeAsync which does not wait
InvokeSync which does and returns a respoinse

Both take the exact same arguments
Alternatively, we could just add a sync field to Invoke and, if it is not present (or if its false), we don't wait for a response.

Never mind, misread the code..


Cereal JSON up to 5x slower than RapidJson

I found that the performance difference between the Binary archives and the Json archives were quite big, so I made a quick benchmark(scroll down for the source code) to compare the Json archive vs rapidjson. I found that rapidjson is faster, of course, but the serialization of json through cereal is up to 5x slower, which sounds a bit much to me.
Is there any known explanation for this?

This will need to be investigated - it is expected that we should be a little slower since we have some overhead that raw use of RapidJSON wouldn't, but 5x sounds like quite a big difference.

Just saw this. I think the reason for the speed difference may be that in the rapidjson version you are serializing to a rapidjson::StringBuffer and deserializing from a char *, while the cereal version uses streams, i.e. serializes and deserializes directly to/from a std::stringstream, which according to the rapidjson documentation is considerably slower.
Also, your serialization/deserialization task is rather trivial, so any overhead introduced by cereal may have a relatively large impact.


CreateSharedMediaReport.java

Translated Example from Python

I suspect that this script also will run into the 429 errors that the python version has, so backoff logic is needed here.


PLAT-1536 Page Down and Page Up arrow is not disabling

Enyo-DCO-1.1-Signed-off-by: Art Dahm<EMAIL_ADDRESS>

Closing PR as jira has been closed.


AE is going to have a new status of Warning

This has a small chance of happening for PPM transfers.
We don't want to re-sent these, but have accounting go in and manually fix them in oracle.
We need to figure out a status for these, and probably a way to move them to completed.
Maybe just leave them in processing, and have some kind of info thing saying they are warning?

done w/ log