Compute auto-correlation only on relevant part.

Since x_lp is already downsampled, we only need the first half.
See: https://github.com/xiph/rnnoise/blob/90ec41ef659fd82cfec2103e9bb7fc235e9ea66c/src/pitch.c#L184
Not a huge performance improvement since this is not a in a performance critical part. But still, the length of x_lp is 1728.

Did you check that this has an effect? Of course I no longer remember what's going on, but the docs for process say that input has size PITCH_BUF_SIZE, and the docs for PitchFinder say that pitch_buf has size PITCH_BUF_SIZE/2, and those are the two arguments that get sent to pitch_downsample...

Oh true, I looked at the wrong length. x_lp is at the correct length.


Fix subtests

Previously:
A test with subtests would cause cricket to crash
Now:
Cricket will not crash!
All errors will be output
Status will be the "worst" status found
(Room for improvement: capture all subtest statuses!)

This looks great, given that you've done some refactoring and pulled some of the code into standalone functions, could we get some more tests to cover the new changes?
Thanks!

Also I'm not sure what's going on with Travis, but I pulled the branch locally and everything seems to run fine!


allow interfaces on functional layers

interface parameters at the ends of functional layers are not properly supported.
Polymer brush and freeform volume fraction profiles merge layers at the ends so that at least for the interface with the layer below.  The interface with the layer above could be supported in the same way, but the parameter is currently ignored.
The full solution may be to microslab the surrounding layers and apply a gaussian convolution or erf mixing over some range, but the precise algorithm to use is not yet defined.

This might be resolved with the current blend algorithms for the profile. It would be easy to check by comparing the reflectivity from a pair of slabs to reflectivity with one of the slabs replaced by a constant function.


Doesn't work on Python 3.11

Documentation
Hello, thanks for releasing the source code. I had a quick try and I found the repo doesn't build on Python 3.11, the build error is from pyFirmata, see https://stackoverflow.com/a/74586109.
I tried on Python 3.10 and had no issues. Maybe it's worth mentioning it in https://github.com/epfl-dlab/aiflows/blob/main/README.md#installation. Thanks!

Hi Jiahao,
Thanks for noticing the problem and identifying its source! We'll look into and find a solution that works for both Python 3.10 and 3.11, so I will keep the issue open until this is taken care of.

Hi @jiahao42 ,
I've fixed this and aiflows should work on python 3.11.
It's worth noting that installing aiflows on Python 3.12 is not recommended. Some flows from the FlowVerse, such as the VectorStoreFlowModule used in AutoGPTFlow, may have dependencies that are incompatible with Python 3.12. For instance, the dependency to chromadb within VectorStoreFlowModule appears to be unattainable in Python 3.12 due to its pulsar-client dependency.

Thank you so much for fixing it so quickly, appreciate it!


Calling Position::set on existing object leads to wrong results

As far as i can say Position::set assumes the object is blank and does not clear the existing values in it, which leads to wrong move generation if the object is reused:
  int main() {
   initialise_all_databases();
   zobrist::initialise_zobrist_keys();

   Position p;
   Position::set("8/2p5/3p4/KP5r/1R3p1k/8/4P1P1/8 w - - ", p);
   auto n = perft<WHITE>(p, 4);
   std::cout<<n<<std::endl;
   Position::set("rnbqkbnr/pppppppp/8/8/8/8/PPPPPPPP/RNBQKBNR w KQkq - 0 1", p);
   Position::set("8/2p5/3p4/KP5r/1R3p1k/8/4P1P1/8 w - - ", p); //same as first position
   n = perft<WHITE>(p, 4); //same as the first perft call
   std::cout << n << std::endl; //this gives wrong perft count

   return 0;
  }

@Dark42ed nice temporary fix, but I'll be reworking Position::set soon, it's the most poorly implemented function in the whole library


Add trust-dns-proto as a dependency

Hi, why is trust-dns-proto in the libs/ directory instead of using it as a dependency in the Cargo.toml file.
https://crates.io/crates/trust-dns-proto

I know you closed this, but I'll answer anyway. Because it's been modified!
The problem is that not all the generator types construct a valid Message, some of them, like the randompkt type, just make a Vec<u8>. The trust-dns-proto library has no way to send data that's not a Message, which makes sense for their API. You most likely don't want to send an invalid message. But if I want to copy that functionality from flamethrower, that's what I need... therefore the lib got copied & modified.
Maybe the randompkt generator is not useful, and maybe it will be removed and I will just pull in trust-dns as a dependency, I'm not sure yet.

thanks for the answer, i was just curious (thought i asked the wrong question)


Question | How does remote-ui ultimately differentiate from Module Federation?

Consideration | Couldn't figure out a better way to ask this question than opening an issue

Hey @dhyegocalota, sorry I missed this issue being opened. This is a very interesting question — I'm curious what made you connect these two technologies together?
I think the main similarity is that both in theory let you split up a larger application into multiple smaller, independently-deployable applications. In module federation, this is done by making shared asset dependencies be loadable asynchronously from each of the smaller applications. In remote-ui, this is done by forcing each of the smaller applications directly call into the host to access any shared logic.
The main difference is that module federation lets you patch together a bunch of these "micro frontends" together on the same HTML page. In remote-ui, the intention is for you to load each of the small applications in a sandboxed environment controlled by the main HTML page, like by using a web worker or iframe. The good part of remote-ui’s approach is that you can fully control what the micro frontends can do — what components they can render, where they are rendered on the page, and what APIs they can access. However, this is most useful for potentially untrusted code; if you trust all the code that’s running, and don’t care about potential conflicting uses of global variables, running several applications on the main page using a technique like module federation is totally fine.

That's a cool use case, thanks for sharing! remote-ui (or, as we just recently updated it to, Remote DOM) is definitely capable of sandboxing untrusted code in this way — that's how we use it at Shopify, where we sandbox third party code from our checkout and admin “host” applications. remote-ui also definitely allows use of React, and the newly-released version uses the DOM API, which makes it possible to use standard React DOM, in addition to libraries like Preact, Svelte, etc.
The main concern you'll likely have is that Remote UI/DOM expect you to define the list of allowed components; third parties can’t just write an application using raw HTML, as they might expect if they were running on the main page. This is a pretty big restriction, but it's one of the main design goals of the library. If that's not what you want, you'll likely prefer using standard iframes instead, which offer the same kind of sandboxing as Remote UI/DOM, but allows developers to use all the power of the browser in their sandbox.


Adjust output filename for temurin to still be called hotspot

I hadn't compensated for the output filename in https://github.com/adoptium/ci-jenkins-pipelines/pull/226 - it would be undesirable to change it to temurin at the moment since it would be an externally visible change. This should switch it back to having hotspot in the filename
Signed-off-by: Stewart X Addison<EMAIL_ADDRESS>

NOTE: Fell foul of the macos installer code as it failed to have a suitable definition if neither hotspot or openj9 was found in the tarball filename


Player speed bugged after using /flyspeed

Walk speed appears to be perma slowed after messing with /flyspeed
To Reproduce
Steps to reproduce the behavior:

type /flyspeed 10
walk around
you're slow nerd

Expected behavior
It shouldn't affect movespeed of walking but it does.
Username: ColdPiplup (happening to many Omen ranks right now though)
All pixelmon servers
Need logs? I don't think so but LMK if you do.

Fixed


Parallel tests

There should be a set of parallel test to see how it works when multiples conexions are writing in the logs.
This set of tests will not be part of the already existent group of test.
The existent suite is unit tests. However, this proposed test is part of the integration tests.
It seems that it does not have any problems because everything is done in stored procedure.
However the test should include other type of loggers, like DB2logger, and I do not think that will work because it uses file handles.

There is an example that runs the demobank in parallel


Fixed typo in toctree link which forced missed documentation.

Hi @NickleDave! Was getting familiar with the package and noticed that the Reference section of the docs was missing due to a typo in the toctree (the space before config created a faulty filepath). Hope this is helpful. Also added extra headers to remove a sphinx warning.
Documentation after the fix (local build):

Hi @Luke-Poeppel
Thanks so much for taking the time to fix this.
I did check out the PR locally and can confirm it fixes that link.
Could I ask you to just slightly amend the commit message to something like this?
    DOC: fix typo in doc/reference/reference.rst

    - remove extra space so link to config.rst works
    - also fix header length in config.rst to silence sphinx warning

I would do it myself but I want to make sure you get credit for your contribution--not sure if that will be the case if I just push my own git commit --amend.
I'm glad to hear you are getting familiar with the package. I know the docs really need attention (as you might guess from all the issues open about them) -- @yardencsGitHub and I are currently focused on getting a revision of a paper back in, but fleshing out the documentation will be very near the top of my to-do list as soon as we get that done.
In the meantime it might help you to look at some more recent config files in this repo:
https://github.com/yardencsGitHub/tweetynet
I don't have much bandwidth with my real job and this paper revision on my plate but if you have specific questions, please feel free to raise a issue or drop me an email.
A bit off-topic but I did check out what you're working on and thought you might be interested in this Timing Variablity Model if you're not familiar with it: https://www.utsa.edu/troyerlab/software.html
Anyway: I'll be happy to merge this in if you don't mind just making that little revision to the commit message. Not to be super fussy about someone freely giving me their time but I do try to follow a consistent commit message format--guess I also need to add something to the docs about that too :smirk:  Again I really appreciate it

Thanks for taking a look! I changed the commit message to your suggested formatting and also opened an issue surrounding contribution guidelines (see here).
No worries about the documentation as I'm mostly sifting through the source for now. I'd like to use vak in my senior thesis at NYU this coming year, so I'm looking forward to getting to know it better. Feel free to label any of the issues as "Good first issues" if some easier contributions would ever be helpful. I'll also check out the configs at tweetynet.
Good luck with the revision! :smile:
P.S. Thanks for the rec on the Timing Variability Model! Will take a look this week.

:pray: :pray: :pray:
thank you so much for making that change to the commit message
yes you are totally right it would be good to have contributing guidelines
No one should have to sift through source code in a perfect world.
It would definitely be super helpful if you can go through and raise issue on things that are not clear in the documentation, once I get it updated.
Awesome to hear you plan on using vak in your thesis. Not sure if that model is at all relevant for what you're working on but just thought you might be interested. Would love to chat at some point to hear more
LGTM! Merging in


Carrier Shipment Method Management

What is the motivation for adding/enhancing this feature?

Figma link: https://www.figma.com/file/bVPRRw282CqGKMdbz7dciH/HC-Ionic-design-system?type=design&node-id=34101-119419&mode=design&t=yjAE31llpwfSiHhN-11
Doc link: https://github.com/hotwax/oms-documentation/blob/user-guides-pub/user-manuals/fulfillment/carrier-and-shipment-methods.md

Dependant on following changes in OMS -

https://git.hotwax.co/commerce/oms/-/merge_requests/4753 - Merged on May 8, 2024
https://git.hotwax.co/commerce/oms/-/merge_requests/4742 - Merged on May 7, 2024

There are few more issues which I have found  -

If 'only methods for this carrier' toggle is set to ON, then it does not change on creation of new carrier or switching between carriers. Ideally it should get set back to OFF in both the situations.
When changes are applied for the carrier code or delivery days, then the methods name are getting disappeared from the UI.(Check in product store section)
Last line appearing in the popover of method card can be removed.
Method or carrier name can be edited and saved with no change. Same for carrier code and delivery days. User is able to type and save the carrier/delivery days field using special characters/characters/space.
Sequencing of methods is not working. Sequencing modal should not be open in case one selected method. Also modal save button should appear disabled on open because of no change. Also on trying to setup the same sequence as of earlier, save button should appear disabled.

Considering the above raised issues, reopening this ticket for now.

Method or carrier name can be saved with blank space.
On trying to save the delivery days/carrier code/gateway with no value, there should not be any toast message generated.
If negative values are entered in delivery days/carrier code and user tries to save it, then in this case the toast message should be modified.
When user tries to create a method whose ID already exists, then in this case toast message should be modified.
Popover should get closed on selecting any option from it.
'Only methods for this carrier' toggle should be OFF when user finishes the carrier setup.
Long method name is overlapping with the delivery days chip.
Scroll position is not retaining its top position in case user opens a carrier > navigates to some other section of App > Go back to carrier shipment method section.

#1 Fixed
#2 Fxied
#3 Fixed
#4 Fixed
#5 Fixed
#6 Fixed
#7, #8 Create a sperate ticket with low priority.

Verified all the fixes and are working fine in release v2.4.3. Hence ,closing this ticket.

https://github.com/hotwax/fulfillment/pull/567
Verified the above pull request changes in release v2.5.0 and found that they are working fine.

Verified the https://github.com/hotwax/fulfillment/pull/568 pull request changes and are working fine in release v2.5.0.


Story Editor: Carousel Page Preview Aria Labels

Summary
Small one. aria labels were already getting created here, just weren't applied properly to the non collapsed page previews. This just fixes that real quick.
User-facing changes
Page aria labels should now be on non-collapsed page preview thumbs in story editor carousel & grid view (they were already being applied to the collapsed ones, so didn't need to change anything)
Testing Instructions
Open up a story in the story editor. Inspect the page preview buttons on the uncollapsed carousel & the grid view. See that they have aria-labels designating them as Page <index>.

Fixes #4244

Verified in QA


How to update tags values in ?

Hi, how do I update the  with the current tag values in the container?
or I have to write it manually?

Use data-initial-value='[{"text": "Algeria", "value" : "Algeria"}, {"text": "Angola", "value" : "Angola"}]'

i have same problem to add tags selected in input from DB so user can modify selection.
is there any example to do this ?


SSE: Last Event Not Consumed

I am experiencing an issue with Gatling during Server-Sent Events (SSE) testing when using a proxy based on Spring Cloud Gateway. The issue is that Gatling retrieves all SSE events except the last one, according to the log records and tests. This behavior can be observed in the provided example which demonstrates the interaction between the proxy and Gatling.
Tests:

Test with consuming last event. Wiremock produces file
Test without consuming the last event. Wiremock produces file

I see that the proxy consumes the full data and puts it on the network. (according the log)
Expected Result:

Confirmation that Gatling reads everything from the network socket and the issue lies within my example.
Alternatively, clarification if this is unexpected behavior from Gatling's side and whether it requires a specific event to stop processing.

Link to Example app: https://github.com/Asinrus/gatling-sse-issue
There you can find the instruction on how to reproduce this behaviour and run the environment.
Additional info:
Laptop: Macbook Pro M1
OS: 15.1 (24B83)

It looks to me that your stream is malformed.
As per the SSE specification:

event         = *( comment / field ) end-of-line

Your last event is missing the end-of-line.


Help with LoopBack

Hi, I'm trying to get this to work with LoopBack which is based on Express.
Building a LoopBack app with webpack is not straightforward, but I've created a fork of the basic example repo to demonstrate it: loopback-webpack-example.
I'm now trying to add HMR to this simple example and I've been working from your example but I'm a little lost. I create the express app with:
var loopback = require('loopback');
var app = loopback();

and create the webpack compiler as per your example, attaching both middlewares. The problem is that I now need to bootstrap loopback and attach to this app instance. This is done within the bundled code. In my example it's done here.
As you can see, the app setup and server start are normally done within the bundled code so I guess I need to move this outside the bundle for HMR? Is there a way I can access the bundled boot-instructions.json module from outside so I can then execute it?

Are you trying to make the server-side code hot reload, or the client?
This module only deals with serving hot-reloaded client bundles to the browser from express.

Server-side. Ok that explains it! :) Any idea how I could approach server-side hot-reloading?

https://github.com/glenjamin/ultimate-hot-reloading-example is the best example I have for this, although it doesn't use webpack on the server component.
I don't really know much about loopback, but the general idea is to pick a seam somewhere in the system between stuff that can change, and stuff that can't. And then bundle only the things which can change, and plug them into the unchanging bit.

Great, I'll check that out. Thanks a lot.


EL-560 Move pod dashboard from environments repo

Also, the filename in grafana dashbaords needs to be unique, or they start 404ing
Link to the Jira ticket
Checklist
Before you ask people to review this PR:
Tests and rubocop should be passing
Github should not be reporting conflicts; you should have recently run git rebase main.
There should be no unnecessary whitespace changes. These make diffs harder to read and conflicts more likely.
The PR description should say what you changed and why, with a link to the JIRA story.
You should have looked at the diff against main and ensured that nothing unexpected is included in your changes.
You should have checked that the commit messages say why the change was made.

Seeking everyone's forgiveness for this is a v naughty merge without approval. Hopefully harmless, this just being a dashboard not the app itself. Everyone else is off, and this could do with merging to bring the repo in line with the k8s state.


German localization broken

Category

[ ] Enhancement
[x] Bug
[ ] Question

Version
Please specify what version of the library you are using: [ 2.1.0        ]
Expected / Desired Behavior / Question
When testing a webpart that leverages PropertyFieldSwatchColorPicker the property is not showing with "gulp serve --locale=de-de"
Observed Behavior
It seems that line 1 of<EMAIL_ADDRESS>:

declare var define: any;
is causing the issue. Propably it shouldn't be there at all...

Steps to Reproduce
see above

Same for the dutch nl-nl localization file. The declare var define... first code line is causing an error. By removing that line in the nl-nl.js file, the problem was solved. Looked at some other localization files, but not all are affected: no.js is fine, but nb-no.js or fr-fr.js are broken. Fixed it for now by removing the first line.

Thank you @otze for reporting that!
And thank you @mvdungen for checking other localizations.
The fix has been done and will be included in the next release.
In a meanwhile you can test the functionality in beta version.


NOTs included in annotated gene products

When I search on a term (e.g DNA recombination), and download results list, it includes
SPCC736.09c
SPAC30.03c
SPAC15E1.07c
SPCC24B10.14c
These are only annotated to "recombination" wth a NOT qualifier.  It is not obvious from the filter options or the results column, that NOT annotations are included.
I think NOT should be filtered out by default, with the option to include them if required?

Could you qualify the page you're working on and the exact process you're using to produce the download? Most term pages offer the "Qualifier" filter that would take care of this.

http://amigo2.berkeleybop.org/amigo/search/bioentity?q=*:*&fq=regulates_closure:"GO:0006310"&sfq=document_category:"bioentity"
Gene products. There is no "qualifier filter" on this page because I am looking at gene products no annotations. However, if the only annotation is a NOT, they are not  actually annotated to the term at all....
At PomBase we display these on the gene pages
http://www.pombase.org/spombe/result/SPCC736.09c
but we exclude from counts of gene products annotated to the term
I think that is what a biologist would expect. Its confusing to retrieve the list of genes annotated to a term, only to find that many of the annotations are negative. Especially as you are not downloading the qualifier field using the download default  (I used the default which is Acc and Label). I only figured it out when I looked at the intersect   to figure out the differences.

I think we could use @cmungall here. An issue here is that there is no single GAF line that corresponds to a line in the table that you see in the search (or are able to download), but rather it is a weak sum of the information collected about a bioentity. In this view, there is no "NOT" to speak of since there is no particular annotation under discussion.

@ValWood is correct. This is a bug. They simply should not be in the bioentity closure. They make sense in the association closure since the qualifier is a property of the association

@cmungall the "showtopper" tag implies pulling the plug until solved, but there is no clear action here that isn't upstream. Should we pull the backup amigo at this point?

I think we can get a golr load fix out soon?

Okay to verify, the idea is that when the owltools fix is in place (provided by @hdietze?), we'll bake that into the amigo branches and set the load to go, accepting a five hour outage, during which time we'll redirect to a message stating that we're having an emergency rebuild?

Pushed a fix in owltools, waiting for integration build to verify tests

Let me know when it lands. To verify, there should be nothing new in the owltools code that would prevent the amigo release branch (of owltools) from merging in master?

Integration build passed.
I checked all the changes since the fork for amigo-2.3.x in owltools and they appear to be not problematic.

@cmungall, since there will be at least a five hour outage, I wanted to verify that I should go ahead. I think the easiest route would be to leave the AmiGO live, but add a very LARGE and RED warning message that it is being reloaded.

As well, it might make sense to make a more public announcement as well.

@hdietze, after a reload, the following still appears in the closure:
PomBase	SPCC736.09c	tfx1	not	GO:0000724	PMID:18062930	TAS		P	TRAX		protein	NCBITaxon:4896	20080829	PomBase

Okay, that needs further investigation. I'm not sure that we use/display the proper closures here or need some other mechanism.

@kltm do you mean GO:0000724 appears in the closure for the bioentity document for SPCC736.09c?
I'm trying this on toaster:
/solr/select?defType=edismax&q=bioentity:"PomBase:SPCC736.09c" AND document_category:"bioentity"&wt=json&indent=on
but I don't see it (which indicates the fix worked)

Things seem to work as expected on the user views

I have not tested this thoroughly but based on what I see so far I'm confident about closing this ticket, thanks everyone!

Okay, I'll take that. I was going off of Val's first comment--I read it as annotation download from term page.


Implement Group Messaging

This is from https://satellite-im.atlassian.net/browse/WARP-121

This is an ongoing effort that will span a few weeks . . . can we break it up into smaller chucks?

Yea this would get split up into 2. One would be smaller group messaging (eg been 3 or more people) while the other would be a bit larger (eg community-style group). The former would be implemented first to lay out the ground work then would extend out to permissions, group and user management (which would be a refactor into the group interface), etc.


block_given? always returns false when it calls in block

Fix #2695
I made sure #2665, #2678 are not destroyed.
block_given? always returns false when it calls in block.
block cannot recieve block.
Could you tell me a sample code that block_given? return true in block.

Could you tell me a sample code that block_given? return true in block.

Something like this? (It doesn't return true anymore after applying your patch.)
def foo; define_singleton_method(:bar) { block_given? }; end; foo {}; bar

@cremno Thank you very much.
I close this PR.


Entity Framework - Entity with List

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Hi Community
I receive following errors if I have a List Object in my Entity that I added as QueryType
but if I add GraphQlIgnore that works
[GraphQLIgnore]
public List<Analyt.Analyt> Analyten {get;set;}

Unable to infer or resolve a schema type from the type reference Input: IDbContextTransaction. (HotChocolate.Types.InputObjectType<Microsoft.EntityFrameworkCore.Infrastructure.DatabaseFacade>)
Unable to infer or resolve a schema type from the type reference Input: IModel.
Unable to infer or resolve a schema type from the type reference Input: DbContextId.

I have following Nugets:
	  <PackageReference Include="HotChocolate.AspNetCore" Version="12.7.0" />
	  <PackageReference Include="HotChocolate.AspNetCore.Authorization" Version="12.7.0" />
	  <PackageReference Include="HotChocolate.Data.EntityFramework" Version="12.7.0" />
	  <PackageReference Include="HotChocolate.Types.OffsetPagination" Version="12.7.0" />

Can you help me?
Best regards from Zurich
Steps to reproduce

[GraphQLIgnore]

public List<Analyt.Analyt> Analyten {get;set;}

 public List<Analyt.Analyt> Analyten {get;set;}

Relevant log output
HotChocolate.SchemaException: For more details look at the `Errors` property.

1. Unable to infer or resolve a schema type from the type reference `Input: IDbContextTransaction`. (HotChocolate.Types.InputObjectType<Microsoft.EntityFrameworkCore.Infrastructure.DatabaseFacade>)
2. Unable to infer or resolve a schema type from the type reference `Input: IModel`.
3. Unable to infer or resolve a schema type from the type reference `Input: DbContextId`.

   at HotChocolate.Configuration.TypeInitializer.DiscoverTypes()
   at HotChocolate.Configuration.TypeInitializer.Initialize()
   at HotChocolate.SchemaBuilder.Setup.InitializeTypes(SchemaBuilder builder, IDescriptorContext context, IReadOnlyList`1 types, LazySchema lazySchema)
   at HotChocolate.SchemaBuilder.Setup.Create(SchemaBuilder builder, LazySchema lazySchema, IDescriptorContext context)
   at HotChocolate.SchemaBuilder.Create(IDescriptorContext context)
   at HotChocolate.SchemaBuilder.HotChocolate.ISchemaBuilder.Create(IDescriptorContext context)
   at HotChocolate.Execution.RequestExecutorResolver.CreateSchemaAsync(NameString schemaName, RequestExecutorSetup options, RequestExecutorOptions executorOptions, IServiceProvider serviceProvider, TypeModuleChangeMonitor typeModuleChangeMonitor, CancellationToken cancellationToken)
   at HotChocolate.Execution.RequestExecutorResolver.CreateSchemaServicesAsync(NameString schemaName, RequestExecutorSetup options, CancellationToken cancellationToken)
   at HotChocolate.Execution.RequestExecutorResolver.GetRequestExecutorNoLockAsync(NameString schemaName, CancellationToken cancellationToken)
   at HotChocolate.Execution.RequestExecutorResolver.GetRequestExecutorAsync(NameString schemaName, CancellationToken cancellationToken)
   at HotChocolate.Execution.RequestExecutorProxy.GetRequestExecutorAsync(CancellationToken cancellationToken)
   at HotChocolate.AspNetCore.HttpPostMiddlewareBase.HandleRequestAsync(HttpContext context, AllowedContentType contentType)
   at HotChocolate.AspNetCore.HttpPostMiddlewareBase.InvokeAsync(HttpContext context)
   at Microsoft.AspNetCore.Builder.EndpointRouteBuilderExtensions.<>c__DisplayClass13_0.<<UseCancellation>b__1>d.MoveNext()
--- End of stack trace from previous location ---
   at Microsoft.AspNetCore.Routing.EndpointMiddleware.<Invoke>g__AwaitRequestTask|6_0(Endpoint endpoint, Task requestTask, ILogger logger)
   at Microsoft.AspNetCore.Authorization.AuthorizationMiddleware.Invoke(HttpContext context)
   at Microsoft.AspNetCore.Diagnostics.DeveloperExceptionPageMiddleware.Invoke(HttpContext context)
Microsoft.AspNetCore.Hosting.Diagnostics: Information: Request finished HTTP/2 POST https://localhost:44326/graphql application/json 266 - 500 - text/plain;+charset=utf-8 1431.1810ms

Additional Context?

Product
Hot Chocolate
Version
12.7.0

It seems strange that the InputObjectType would have the DatabaseFacade as the generic type parameter. That should typically be the model class. Its difficult to tell from the information provided where that fault may come from.

Thank you for your answer.
If you want, I can create a test Project
Best regards

It looks like you forgot to add the [ServiceAttribute] which will lead to the schema engine inferring an input from your dbcontext.

Please provide us with a repro or some more code otherwise it looks like what I have stated above.

@michaelstaib  Thank you for your Suggestion. After your comment, I searched more and find the solution.
IAdded this and now works. I dont know why ;) I hope this will also help to other
.RegisterDbContext()

@M2xAy did you get this information from the docs? If so, could you let me know why its still unclear and what's confusing to you?

Hi tobias
i first tested graphql with another simple project, everything worked fine, then i installed and configured the same setup in another big complex project and i had the error with the lists in dbentity. so i am confused because RegisterDbContext was not in the first project.
After Michael's comment, I googled it and found the documentation you linked. I tested it and it works.
Why I am confused: because without RegisterDbContext it worked in the first simple project.
With kind regards
Murat


Fixed missing sequence port in VisualScript seed()

Bug fix if there is no output port a sequence port is needed to run the VS_Node
Image Godot 3.3.0 stable

@DavidCambre I think you have made this PR for 3.x, but you're trying to get it merged in master. You've already made #48964 for the master branch. Can you rebase this PR to 3.x?

@theoway No need to do it - This could be simply cherry-picked to 3.x.

@theoway No need to do it - This could be simply cherry-picked to 3.x.

Right, but again, now there are two PRs fixing the same thing. I think one of 'em should be closed...

@theoway No need to do it - This could be simply cherry-picked to 3.x.

Right, but again, now there are two PRs fixing the same thing. I think one of 'em should be closed...

@theoway I opend a 3.x first that is tested and has no build problems.
Then was advised to open a master witch I am not able to test (VS in master is not in a good state) and has build issues.
So until further notice,... there are 2.

@DavidCambre I test this PR's master version, and it works fine. I've mentioned it there. I think that PR can be merged. If that gets merged, this PR can be closed.

Right, it's identical to https://github.com/godotengine/godot/pull/48964 (merged), so I close it.


Fix Threefry RNG

Fixes #745

@Ulfgard Please review

I'll focus on fixing Travis CI builds tomorrow.

Coverage increased (+0.03%) to 83.766% when pulling 77b4a4910b80f9d665def4d9c8e494fd67a27458 on jszuppe:fix_threefry into d36ef02b7860b9bcf5ea4911b77f1aa77b5f4059 on boostorg:develop.

lloks good. ship it :)


Update "TI Map" analytics rules for performance

Required items, please complete
Change(s):

Changed query to put any applicable WHERE filters before any SUMMARIZE calls for performance.
Added ThreatIntelligenceTaxii data connector on missing analytics.

Reason for Change(s):

Many of these queries would fail on real-world datasets due to summarization of unnecessary data.
ThreatIntelligenceTaxii is a valid provider of ThreatIntelligenceIndicator table data and should be included

Version Updated:

Yes

Testing Completed:

Yes, as much as possible (don't have all data sources to test appropriately for some rules)

Checked that the validations are passing and have addressed any issues that are present:

Yes

To whoever ends up picking this up, the KQL validation failures is due to Line 397 of  https://github.com/Azure/Azure-Sentinel/blob/master/.script/tests/KqlvalidationsTests/KqlValidationTests.cs
The regex being used is overly-specific and not conducive for the performance improvements being made here.

Hi @kfriede, thanks for raising this PR. We will review this PR by 13 Oct 2023.

This traces back to commit 16564be from @aprakash13

Hi @kfriede, for KQL validation, we will see what can be done in this case.
We would need you to repackage this solution using this tool https://github.com/Azure/Azure-Sentinel/tree/master/Tools/Create-Azure-Sentinel-Solution/V3
Thanks

Hi @kfriede, please provide update on above comment.

Hi @kfriede, please provide some update on this PR

Hi @v-rbajaj, apologies I was out last week. I will look at repackaging the solution for these edits

Hi @v-rbajaj, apologies I was out last week. I will look at repackaging the solution for these edits
Edit: Solution has been repackaged for 3.0.2

Thanks @kfriede, will review the PR by 27 Oct, 2023.

Hi @kfriede, thanks. We are currently looking into the KQL validations and once that is fixed, we will revisit this PR by 2 Nov 2023.

Hi @kfriede, we are still investigating on KQL validations, will get back to by 03 Nov 2023.

Hi @v-rbajaj, it's been over 1 week since your last update. Is this still under review?

Hi @kfriede,
Apologies for the inconvenience, but due to unavailability, the review has been delayed. We assure you that you will receive an update on this by the end of the day tomorrow. Thank you.

Hi @kfriede, we are working on the KQL changes and we will soon provide an update.

@v-rbajaj after refactoring, files now give: Kqlvalidations.Tests.KqlValidationTests.Validate_DetectionQueries_SkippedTemplatesDoNotHaveValidKql(fileName: "IPEntity_DuoSecurity.yaml", encodedFilePath: "L2hvbWUvdnN0cy93b3JrLzEvcy9Tb2x1dGlvbnMvVGhyZWF0IE"...) in testing

Hi @kfriede, sorry for the above commits, was trying to fix few errors from KQL validations. We are checking on it, will get back to you by 29 Nov 2023.

@v-rbajaj I’m good with #9369, and we can close this (#9181) when #9369 is merged
--
Kevin Friedemann
On Nov 28, 2023 at 5:01 AM -0800, v-rbajaj @.***>, wrote:

Hi @kfriede, can we merge #9369 as the changes there look similar and all the KQL validations have been fixed by @blauwers.
If there are any changes you want to suggest on #9369 or want to contribute your changes, can you please comment on that PR?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Thanks @kfriede, I have changed the PR status from open to draft and will close later once #9369 is merged.

Closing this PR as #9369 is merged.


[MUGEN][Bugfix] Transformer residual layer

Stack from ghstack:

#307
-> #305
#304
#306
#303

Summary:

Fixed a bug that overrode the input to the transformer layer before the latter was added to the attention output
Updated unit tests

Test Plan:
CI

Codecov Report

:exclamation: No coverage uploaded for pull request base (gh/langong347/44/base@7b1a2f5). Click here to learn what that means.
The diff coverage is n/a.

@@                   Coverage Diff                    @@
##             gh/langong347/44/base     #305   +/-   ##
========================================================
  Coverage                         ?   93.63%
========================================================
  Files                            ?       54
  Lines                            ?     3236
  Branches                         ?        0
========================================================
  Hits                             ?     3030
  Misses                           ?      206
  Partials                         ?        0

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

@langong347 has imported this pull request.  If you are a Meta employee, you can view this diff on Phabricator.

@langong347 has imported this pull request.  If you are a Meta employee, you can view this diff on Phabricator.


Fix some version numbers

Fixed 3 version numbers

Great - thanks! Maybe we should make the sanity checker enforce the x.yz version format?


Saga ID can now be generated when the saga first starts

If a saga implements gbus.SagaIDGenerator it will be able to generate it's own ID instead of the auto generated one by grabbbit

Coverage decreased (-0.2%) to 76.054% when pulling 17739c32e2ab348e9c00efbac5ef0997688a27c8 on feat-set-saga-id-on-reciver into e7e27a51cf24765fad39746a116317267461a540 on v1.x.

@rhinof since you closed and abandoned the approach in #211 should we close this PR as well?

@vladshub it's up to you....however this PR will run into the same problem as #211 and I currently do not see a clear way to solve it....but maybe you can :)


Add dns test that simulates searchpath expansion.

In user clusters, the AAAA records and searchpath expanded queries
contribute to much of DNS latency. This change adds an all-queries.txt
file which includes external and in-cluster service names along with
the searchpath expanded queries. The default config also runs the test
for 10 minutes instead of 1 minute.
Updated README on the steps to run comparison tests between NodeLocal
DNSCache and ClusterDNS.
Modified run binary to drop "--dns-server" parameter when specifying
"--cluster-dns" which automatically looks up kube-dns service IP.

/assign @bowei

/lgtm
/approve


fix/Timer Trigger Typo

Fix typo in the JavaScript code example

@PCOffline : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


package name wrong for generated validate code

I am using below command to generate validate.pb.go files
protoc --proto_path=api/proto/v1 --proto_path=third_party --go_out=pkg/api/v1 --validate_out=lang=go:pkg/api/v1 endpoint.proto
The code is generated successfully but the package name is coming as v_1 in the generated code under pkg/api/v1

I wouldn't say it's mandatory (and the naming there might be a bug in how we split names in the PG* library), but it's definitely strongly recommended by the protoc-gen-go plugin to signal what the resulting package name ought to be.


HN firebase deploy: include a message w/ the commit SHA?

https://github.com/dart-lang/angular/blob/dd1fc24b3c20c47d2c2e1faa2272f367bfbfc3db/tool/deploy.sh#L14
See https://docs.travis-ci.com/user/environment-variables/#Default-Environment-Variables
We can use TRAVIS_COMMIT env variable
firebase deploy --non-interactive -m "For commit $TRAVIS_COMMIT" or something...

I don't think we can... use the environment variable in the site, can we?

It's just the message showing up on the firebase admin page – so we can see which commit is currently deployed – or if it's just me messing around, etc...

If you don't mess around we don't need to check what the commit is :)

Imagine the message at the bottom was "Auto-deployed from Travis at commit XYZ123" – that's all I want...

Looks like its done at HEAD! @kevmoo


DataGrid inside a ScrollViewer can not scroll normally.

Describe the bug
I have a DataGrid within a ScrollViewer. The ScrollViewer will be also scrolled when I try to scroll the DataGrid. I have tried the ScrollViewer.IsScrollChainingEnabled and set the event args's Handled as true, but no effect.
    <ScrollViewer>
        <Border Height="1000"
                Margin="40"
                BorderBrush="Red"
                BorderThickness="1">
            <DataGrid Height="500" ScrollViewer.IsScrollChainingEnabled="False"
                      VerticalAlignment="Top"
                      ItemsSource="{dataGrid:Enum Key}">
                <DataGrid.Columns>
                    <DataGridTextColumn Binding="{Binding Path=.}" Header="Key" />
                </DataGrid.Columns>
            </DataGrid>
        </Border>
    </ScrollViewer>

To Reproduce
Steps to reproduce the behavior:

Put the mouse onto the DataGrid.
Scroll.
Then the DataGrid is scrolled and the ScrollViewer is also scrolled.

Expected behavior
The outer ScrollViwer should not be scrolled if the DataGrid can handle that event.
Screenshots
Desktop (please complete the following information):

OS: win11
Version 11.0.0-11.0.2.

Additional context
A sample for that here. Avalonia.DataGrid.zip

I fixed the problem with a temporary solution. After checking the source code of ScrollBar, I found that the ScrollBar will find it's owner ScrollViewer upwards along the visual tree. But there is no ScrollViewer in DataGrid's ControlTemplate. So the ScrollViewer outer of the DataGrid will become the ScrollBar's _owner.
So I break it down.
public class DataGridScrollBar : ScrollBar
{
    protected override Type StyleKeyOverride { get; } = typeof(ScrollBar);

    protected override void OnAttachedToVisualTree(VisualTreeAttachmentEventArgs e)
    {
        base.OnAttachedToVisualTree(e);

        this.TrySetField<ScrollViewer>(typeof(ScrollBar), "_owner", null);
    }
}

// ScrollBar.cs in 11.0.2 -------------------
ScrollBar
{
    ......
    /// <summary>
    /// Locates the first <see cref="T:Avalonia.Controls.ScrollViewer" /> ancestor and binds to its properties. Properties which have been set through other means are not bound.
    /// </summary>
    /// <remarks>
    /// This method is automatically called when the control is attached to a visual tree.
    /// </remarks>
    internal void AttachToScrollViewer()
    {
      ScrollViewer owner = this.FindAncestorOfType<ScrollViewer>();
      if (owner == null)
      {
        this._owner = (ScrollViewer) null;
        this._ownerSubscriptions?.Dispose();
        this._ownerSubscriptions = (CompositeDisposable) null;
      }
      else
      {
        if (owner == this._owner)
          return;
        this._ownerSubscriptions?.Dispose();
        AttachedProperty<ScrollBarVisibility> visibilitySource = this.Orientation == Orientation.Horizontal ? ScrollViewer.HorizontalScrollBarVisibilityProperty : ScrollViewer.VerticalScrollBarVisibilityProperty;
        IDisposable[] array = ((IEnumerable<IDisposable>) new IDisposable[7]
        {
          IfUnset<StyledProperty<double>>(RangeBase.MaximumProperty, (Func<StyledProperty<double>, IDisposable>) (p => this.Bind<double>(p, owner.GetObservable<Vector, double>((AvaloniaProperty<Vector>) ScrollViewer.ScrollBarMaximumProperty, new Func<Vector, double>(this.ExtractOrdinate)), BindingPriority.Template))),
          IfUnset<StyledProperty<double>>(RangeBase.ValueProperty, (Func<StyledProperty<double>, IDisposable>) (p => this.Bind<double>(p, owner.GetObservable<Vector, double>((AvaloniaProperty<Vector>) ScrollViewer.OffsetProperty, new Func<Vector, double>(this.ExtractOrdinate)), BindingPriority.Template))),
          IfUnset<StyledProperty<double>>(ScrollBar.ViewportSizeProperty, (Func<StyledProperty<double>, IDisposable>) (p => this.Bind<double>(p, owner.GetObservable<Size, double>((AvaloniaProperty<Size>) ScrollViewer.ViewportProperty, new Func<Size, double>(this.ExtractOrdinate)), BindingPriority.Template))),
          IfUnset<StyledProperty<ScrollBarVisibility>>(ScrollBar.VisibilityProperty, (Func<StyledProperty<ScrollBarVisibility>, IDisposable>) (p => this.Bind<ScrollBarVisibility>(p, owner.GetObservable<ScrollBarVisibility>((AvaloniaProperty<ScrollBarVisibility>) visibilitySource), BindingPriority.Template))),
          IfUnset<StyledProperty<bool>>(ScrollBar.AllowAutoHideProperty, (Func<StyledProperty<bool>, IDisposable>) (p => this.Bind<bool>(p, owner.GetObservable<bool>((AvaloniaProperty<bool>) ScrollViewer.AllowAutoHideProperty), BindingPriority.Template))),
          IfUnset<StyledProperty<double>>(RangeBase.LargeChangeProperty, (Func<StyledProperty<double>, IDisposable>) (p => this.Bind<double>(p, owner.GetObservable<Size>((AvaloniaProperty<Size>) ScrollViewer.LargeChangeProperty).Select<Size, double>(new Func<Size, double>(this.ExtractOrdinate)), BindingPriority.Template))),
          IfUnset<StyledProperty<double>>(RangeBase.SmallChangeProperty, (Func<StyledProperty<double>, IDisposable>) (p => this.Bind<double>(p, owner.GetObservable<Size>((AvaloniaProperty<Size>) ScrollViewer.SmallChangeProperty).Select<Size, double>(new Func<Size, double>(this.ExtractOrdinate)), BindingPriority.Template)))
        }).Where<IDisposable>((Func<IDisposable, bool>) (d => d != null)).Cast<IDisposable>().ToArray<IDisposable>();
        this._owner = owner;
        this._ownerSubscriptions = new CompositeDisposable(array);
      }

      IDisposable? IfUnset<T>(T property, Func<T, IDisposable> func) where T : notnull, AvaloniaProperty => !this.IsSet((AvaloniaProperty) property) ? func(property) : (IDisposable) null;
    }
    ......
}

This sounds like it might be the same problem I'm running into with Avalonia 11. Anytime I put a DataGrid in the ScrollViewer, I can't seem to get a separate horizontal ScrollBar for the DataGrid added. And adding a new DataGrid to a ScrollViewer also resets the ScrollViewer X Offset back to 0. (no other control does that when added to a ScrollViewer)
I've gotten most of the other issues fixed with upgrading to Avalonia 11, but this one will be a blocker without a fix or workaround so any fix would be appreciated. :)
I also had created a repo for the offset reset, and stumbled on this. And after reading this one I found I had an even bigger problem with the missing horizontal scrollbars that I hadn't noticed before.
https://github.com/garyhertel/AvaloniaScrollViewerDataGridOffset

@garyhertel it's not wise to do this. DataGrid should have a fixed height, either by specifying a height or having it placed in a Grid or similar layout control that gives fixed heights.

@liwuqingxin if you have an idea how to fix this issue (looks like you have), a PR is welcome :-)

Thanks for the quick fix! This looks like it fixed my problem :)


Revise the reference hint text

Trello card URL:
https://trello.com/c/npmtxxuy/803-reference-numbers-for-job-alerts-should-not-be-machine-generated-but-a-mandatory-human-reference-so-that-the-alert-reference-is
Changes in this PR:
New reference text based on feedback from Fiona.

 Reference numbers for job alerts should not be machine generated but a mandatory human reference so that the alert reference is clear and understandable


perf(plugins/keycloak-access-token): set split array limit

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
176

Relevant Lines:
176

💛  - Coveralls


Automatic PU deletion once its campaign is closed

Impact of the new feature
Disk space of CMS sites
Is your feature request related to a problem? Please describe.
When a campaign is not used anymore, it is archived - i.e. PnR removes the campaign from the campaign configuration file and updates the CouchDB. In such cases, the PU of the campaign keeps occupying space on disk, although it is not used anymore. The deletion of these PUs require manual intervention, which might be time consuming.
Describe the solution you'd like
Once a campaign is archived, its PU/PUs should be deleted from disk automatically only if they are not used by any active campaign.
Describe alternatives you've considered
In order to add an additional layer of check, we might want to develop the automation such that it asks for an approval.
Additional context
@nsmith- @amaltaro @todor-ivanov FYI

Thanks for creating this issue, Hasan. The more I think about Pileup management within WMCore, the more I'm convinced that we need one specific microservice/thread to deal with them in an unified way. Sort of what I started discussing here:
https://github.com/dmwm/WMCore/issues/9779

I believe this issue has actually been addressed with the recent developments in MSPileup, especially tracked in this GH issue: https://github.com/dmwm/WMCore/issues/11426
@haozturk I am closing this issue out. But please reopen it in case I am missing anything. Thanks


Revisit blog title capitalization

We need to change capitalization of some blog titles for consistency

I changed a bunch of them for consistency this morning.  Take a look at the preview in the PR for Meshan's post.

@daniel-coleman-1 I somehow missed these two that you can fix later.


exclude _attachments from encryption

allows to workaround #13
fixes #17 (outdated browserify version was the cause)

hm so currently this is a bit of a foot gun potentially leaving you exposed, we have a couple other options

throw an informative error if somebody tries to upload an attachment
delete the attachments by default
encrypt them somehow

now 3 would be preferred but are we able to actually do that without also monkey patching [get|set]Attachment

An option to leave attachments unencrypted would be great. Some Privacy and Security Audits are ok with unencrypted binaries (e.g. images) for some reason. But of course just encrypting them is even better. Don't know about the performance implications.

Encrypting large binary objects is probably going to be a huge performance drain, especially in browsers. Also there's the awkwardness of translating between Blobs and Browserified Buffers.
I think just adding a note to the README that this module doesn't encrypt attachments should be fine. If somebody wants encrypted attachments, they can clone it.

👍

Added a notet to readme.md that attachments are not encrypted. Can this be merged now?

I'm on the fence about this, but I think I'd like you to have to pass an option to get it to leave attachments untouched.  Encryption isn't very helpful if it silently doesn't encrypt your stuff.
I think the default behavior should be to remove attachments and log a warning, but you can pass an option to get it to ignore them.  I have a pull that is waiting on nolanlawson/transform-pouch#19 which simplifies the creation process which should make passing params easier.

"remove attachments" in the sense of "ignore attachments" / not encrypt them? Happy to help review https://github.com/nolanlawson/transform-pouch/pull/19 to, just won’t get to it today

closing in favor of #29


Modify to use templates

This is a partial modification to move inline HTML statements into templates.

Currently only one template is used, but it significantly cleans up the perl6 code for pod2html
The templating module is Template::Mustache. Although Mojo was my first choice, it is not currently passing all tests, whilst Template::Mustache is passing all tests.
A template allows a user to provide custom HTML.
A 'partal' modification because various other routines generate little bits of HTML. Some, perhaps do not need to be changed because they just wrap text into obvious tags.
The toc subroutine output should also be moved to a template. TODO.
Added tests for templates. Since the lack of templates generates output on stderr, I have included Test::Output to capture and verify the error message in the test code.

finanalyst requested your review on this pull request.

I left a couple of comments, but I think someone from the docs team could be a better judge of this work. I see @JJ opened https://github.com/perl6/Pod-To-HTML/issues/35 , perhaps he could review this PR as well.

Can you please take a look at these changes? We can then merge it.


Connection Fails in GitHub Actions

I have noticed this a couple of times. Installing the Trilogy Gem locally and connecting to MySQL (8.3.0) works without issues. However, when used in GitHub Actions, it generates this error:
ActiveRecord::ConnectionNotEstablished: trilogy_auth_recv: TRILOGY_UNEXPECTED_PACKET (ActiveRecord::ConnectionNotEstablished)
A little further down the stack, this:
Caused by:
Trilogy::QueryError: trilogy_auth_recv: TRILOGY_UNEXPECTED_PACKET (Trilogy::QueryError)

My GitHub actions are using the latest MySQL. I am happy to provide more information if needed. Hopefully, I just needed to include a setting or config option.
Rails 7.1
Ruby: 3.2.2
MySQL (8.3.0)

I think you are running into Trilogy's current non-support with caching_sha2_password. You'll need to put MySQL into mysql_native_password mode.
Here's many examples you might look at for doing so:
https://github.com/search?type=code&q=path%3A.github%2Fworkflows%2F*+"--default-authentication-plugin%3Dmysql_native_password"

I got hung up on this for hours. I think I had an issue with loading the wrong MySQL config a second time and stomping the guidance above.
Anyway, there's my ci.yml for a pretty standard Rails app using Trilogy, if that helps:
https://gist.github.com/kmcphillips/55043742fcf6e882a7e2eb90611a4910

I'm going to close this because we already have an open issue for caching_sha2_password.


ci: update issue templates

Fixes #N/A
Description
Updates the issue templates to use the kind/* labels and add the needs-triage label by default.
How was this change tested?
N/A
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
7960

Relevant Lines:
10012

💛  - Coveralls


When some incomplete data arrives to mountain chart we get error in getFrame on load

Reproduced currently on develop (because it was switched to the new data)
The data is likely to blame, but we should handle it nicely, not break

@sergey-filipenko told me how to fix it by doing some awesome detective work. resolved now!


mangapanda

What steps will reproduce the problem?
What is the expected result?
What do you get instead?
Additional info

Q
A

Yii version
2.0.?

PHP version

Operating system

sorry my mistake how can i delete this

There is no way to delete an issue. We can only close it and make it funny


Problem with newline \n at end of string

Hello,
I will read a string as the following:
${stag}    yash.Execute Command   ${id}   show evcs evc inband-evc stag
After this,  ${stag} is:
stag : 16
As I am only interested in the number ( range from 0 -4095), I do the following:
${stag_result} =    Fetch From Right    ${stag}    :${SPACE}
stag_result=16
This is at first sight what I want, but unfortunately there is (invisible) a newline at the end of the number '16'.
And therefore the following command does not work:
${integer} = Convert To Integer ${stag_result}
I have not been able to remove the newline so far. the following is unfortunately without effect:
${stag_result}= Remove String ${stag_result} \n
Does anyone have an idea ?
thanks in advance

I tested that
Convert To Integer    10\n

works fine so newline at the end doesn't seem to matter. Are you certain you have a newline there and not, for example, the null byte? What exact error do you get from conversion?

Ok, thanks. I will check this.

I have figured it out now, this is how it works.

Therefore one does not recognize the problem immediately
${stag} is:   'stag : 16\r\t'

${result} =    Fetch From Right    ${stag}    :${SPACE}
log   result1=${result}    console=y

${result} =    Fetch From Left    ${result}    \r\n
log   result2${result}    console=y

${integer}=  Convert To Integer  ${result}\r\n
log   integer=${integer}    console=y

It cannot be just due to \r\n because int('16\r\n') works just fine. You showed error

ValueError: invalid literal for int() with base 10: '1256

which contains an extra ' so most likely the value is something like '1256\r\n.
You can use formatter=repr with the Log keyword to see special characters like \r and \n better.
It seems to me there's no problem on RF side so I close this issue.

Thanks

When  ${stag}  =  '1256\r\n'
I am not able to remove the quotation marks  '
${result} = Remove String ${stag}   '
has no effect.


docs: notes on various forms of k8s upgrade disruptions

Extracted from https://github.com/2i2c-org/infrastructure/pull/3464

Thank you for reviewing @sgibson91!
I've now installed a english grammar plugin to vscode to help me not emit as many of these mistakes going forward - I've concluded that the extension didn't have as sharp eye as you but at least caught a few of the mistakes :)

English grammar rules are so weird and difficult, I'd be amazed if they could ever be truly automated 😅 But at least we don't have too many cases!


Support signed binary packages

This is a work in progress.  There is an issue with emerge commands failing unless PORTAGE_SSH_OPTS= is set in the environment.  The was this is set up, it only works with the SDK scripts since the FETCHCOMMAND_GS and RESUMECOMMAND_GS variables are not in the user's login environment.

It may be worth dropping the final commit (for downloading+verifying), and just taking the commits for signing packages to allow builds to start uploading signatures.

The download/verify part has been dropped for now.


opentelemetry-collector: add NixOS module

Description of changes
Also: like we set it in opentelemetry-collector-contrib, we should set the mainProgram for opentelemetry-collector. This makes the module work for both packages.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Thank you for preparing this module @bouk - I took the liberty of writing a nixosTest for this in my fork, I branched off your branch here to add the test. If you'd like to just merge/cherry pick into your branch to update this PR, I think that'll adequately cover the module :+1:

Thank you @domenkozar


Better token expiry messaging

When the client's token expires in the VIP-CLI tool we have some suboptimal messaging. Here's an example.
> vip wp @example-develop.go-vip.co
App example-develop does not exist

> vip wp @123
Error: Resource not found

> vipbox example-develop.go-vip.co
Invalid or expired token. Please login with `vipgo login`

A few things here:

Can we make a more friendly message than Error: Resource not found, e.g. jump straight to the token expiry message?
We shouldn't be referencing vipgo login, that's not the command name
There is no vip login command, I think we should be referencing simply vip as the command to run?

Props @justnorris

Probably the used version is old. The new version gives this:
Unauthorized: You are unauthorized to perform this request, please logout with vip logout then try again.

It was the newest version at the time ;)

Just spotted the vipbox command on the last command 🤦♂ Closing as wontfix (?) for now.


UI expands if data is more on the screen

Navigate to JWT Vulnerability UI screen and fetch the token for level 2 or level 3 and you will see that the X-Axis becomes expanded. we need to wrap the text such that the UI doesn't become distorted.

Hey @preetkaran20
I am interested in working on this issue. Can you please assign it to me?


Duplicate classes from different packageDirectories getting overwritten without a warning

Summary
We extensively use packageDirectories to separate concerns and split package into smaller, more understandable pieces.
However, I've recently discovered that duplicate file names (tests in particular) would get overwritten (I think by the order specified in packageDirectories) without producing error or even warning.
Steps To Reproduce:
Consider this sfdx-project.json:
{
  "packageDirectories": [
    {
      "path": "myns",
      "default": true
    },
    {
      "path": "test_files"
    }
  ],
  "namespace": "MyNS",
  "sourceApiVersion": "45.0"
}

sfdx force:apex:class:create -t ApexUnitTest -n TestA -d myns/main/default/classes/
sfdx force:apex:class:create -t ApexUnitTest -n TestA -d test_files/main/default/classes/
Rename one of the methods from to myUnitTest1
sfdx force:source:push
sfdx force:apex:test:run -w 2 -t TestA.myUnitTest1,TestA.myUnitTest

Expected result
Error when pushing.
Actual result
Push succeeds, step 5 produces:
TEST NAME          OUTCOME  MESSAGE                                                                          RUNTIME (MS)
─────────────────  ───────  ───────────────────────────────────────────────────────────────────────────────  ────────────
TestA.myUnitTest   Skip     Method myUnitTest does not exist in the the apex class with Id :01pS0000005lLQp
TestA.myUnitTest1  Pass                                                                                22

Additional information
Latest catalina and sfdx

@sjurgis We are currently working on improvements to the CLI that will improve the experience with having multiple package directories in one project. This work is being tracked internally and is currently in progress. We expect to have something released in spring if not sooner. I am going to close this issue, thank you for reaching out

@clairebianchi unsure why you are closing the issue if it's not resolved...

This is quite serious defect too...

I agree with OP. This is a VERY serious problem with the CLI! 🚨
Whenever you retrieve metadata from an org, the CLI saves the downloaded source in {defaultPath}/main/default . And if you happen to have a path anything different than that, there's a considerable chance that you might save duplicate metadata in your source.
Basically the only thing that would warn you about something wrong in the deployment caused by this would be a compile error, which might not happen often. 🤷♂

We dealt with this by not using force-app/main/default for anything, all files are stored in different subdirectories. We have a Git Hook and CI job that ensure that directory does not exist. It was a significant cause of problems for us but since adding those rules/checks we've been much happier.
Rather than Apex we saw more issues with part of the object definition being pulled down here and preventing other directories from contributing to the object definition which would break things as it wouldn't compile the correct Custom Object or suppress changes, depending on the type of org we were working with.

@aheber Yeah, we'll probably end up checking and failing the CI job if Salesforce CLI's default dir is present in the source code...
But still, this would not solve the issue. the problem still persists if one has a source structure with different dirs. And then create class A in ./src/main/classes/domain/DomainA/ClassA.cls and ./src/main/classes/domain/DomainB/ClassA.cls, and one of them will be silently overwritten during a deployment... 😞

FYI, this (Multiple Package Directories) issue should now be fixed.
See here: https://developer.salesforce.com/docs/atlas.en-us.sfdx_dev.meta/sfdx_dev/sfdx_dev_ws_mpd.htm
Also here: https://github.com/forcedotcom/cli/issues/379
Please update to the latest CLI and let us know if you're still running into any issues with this.
Thanks!

@rvwatch I had sfdx-cli/7.65.4 darwin-x64, updated 3 days ago. Updated to the lastest (as of now, version 7.65.6) and this still persists. Important to notice I'm not using multiple packages directories. The use case I tested is grouping classes in different directories for a single-packaged project, which is different from OP's use case. The problem he faced though is the same.
I made a simple test to show what I mean. I have my source like so:

And @ sfdx-project.json:
{
  "packageDirectories": [
    {
      "path": "src",
      "default": true
    }
  ],
  "namespace": "",
  "sfdcLoginUrl": "https://login.salesforce.com",
  "sourceApiVersion": "48.0"
}

When running sfdx force:source:deploy -c -u dev56 -w 10 -p ./src (deploying to a sandbox. no scratch environments involved):
Job ID | [REDACTED]
SOURCE PROGRESS | ████████████████████████████████████████ | 1/1 Components
Successfully validated the deployment

Although classes are duplicated, no warnings nor errors are shown. It also shows only a single metadata was deployed. The command finished succesfully... Not sure how it chooses which of the two classes is deployed. And if the classes have the same name, but different implementations, one of them is deployed, and the other is ignored 🤔

Not sure if under the hood the CLI converts the source code to the mdapi format before deploying, but I found out that if I run sfdx force:source:convert -p ./src -d ./mdapi-format in this project, no errors nor warnings are thrown regarding the duplicate metadata, and I end up with 1 class as well:

Yeah I did notice there's a warning now when using multiple packageDirectories, but @MuriloKakazu use case should be solved as well.
cc @rvwatch @clairebianchi


[WIP] Updated Checker Operation to handle concurrency

Summarization of Commits:

Tried to implement a model where single MetaStore is used by different concurrent operations.
Implemented initialization of a single Kopia Server which returns fingerprint to allow different Kopia Clients to connect to it.
Used Override Username and Hostname to differentiate the Kopia Clients.
Replaced a single Checker with multiple Checkers (one for each client), as it might help in initializing a new restore directory for each client.
Similarly did this for snapshotter, and FileWriter, as it might ease the process.
Included initialization of all of them.

Signed-off-by: Rahul M Chheda<EMAIL_ADDRESS>

Have not made the changes for InitFileSystem, please take a look, if the approach looks fine we can proceed.

In this PR, I have only implemented the mutliple client model forInitS3WithServer, and not for any other method. Had to comment out a lot of code to make it working without changing them all to that model.


Adding a little bit of clarity to installation

Adding a little bit of clarity to installation for someone completely new to unity packages

I have nothing against this, but the PR should target canary instead.


Update dialog styling and scrollable containers.

Issue Description
Remove inner dialog-body div, wrapper div, and inner content styling.
Consume terra-scroll.
Issue Type

[ ] New Feature
[x] Enhancement
[x] Bug
[ ] Other

Expected Behavior
The terra-dialog should provide a consistent content area for scrolling and expand to fill functionality in order to allow consumers to have access to scrollable values programmatically.

Closing this in lieu of #1741


The length computation of the last line in a multi-line is wrong

https://github.com/masaccio/compact-json/blob/234356f5eb7aa335e1193c49cbad72589d8145a5/src/compact_json/formatter.py#L431
the current code results in a "tail" at the last line as the length of the last element is set to 0:

I think this line should be changed to
item_length + len(self.padded_comma_str)

which take into account the comma length no matter whether there will be a comma after the item. This could prevent such tail from happening when the max length is exactly 1 character less than the length of k elements with trailing comma. For example (I edited the bugged line to item_length + (len(self.padded_comma_str) if not_last_item else 0) before running the following code):

Many thanks for the patch. Latest PyPI has the change.


Change ambient sound while playing time

Hi Afonso.
I try to change the ambient sound while playing time, but this is not working?
What do you think about this feature? Its possible?
Thanks and nice job

HI man,
This is a nice feature :)
I need fix this :(
rsr


Implement one database transaction per HTTP API call

There are cases (like in Create/Edit Article commands that multiple calls to SaveChanges are made inside the command). If the second SaveChanges call fails for some reason, the data saved by the first SaveChanges call does not get rolled back, leading to database inconsistencies.
It would be nice to implement a transaction per HTTP API call (as a filter added to the ASP NET core pipeline).

I was thinking to implement this functionality as described here.

I've done something similar before but without EF Core.  It's a good idea.

Closed.


No WALs can be directly redone in page server for pg_bench simple_update?

Steps to reproduce
run pg_bench simple update
Expected result
Some WALs can be redone within page server so as to improve performance
Actual result

this is computed using pageserver_wal_redo_seconds (which is updated for all redos) and pageserver_wal_redo_records_histogram (which is updated only for Postgres redos). It seems that no batches are applied within page server. This might not be a bug but I feel it is strange that we cannot redo anything in page server under this workload as it requires IPC.
Environment
Logs, links

@skyzh is this issue still relevant? If not, please close. If it is, can you try to ELI5 what the problem is, I didn't understand it?


Re-factor code python execution framework

Ensure we have a generic way of executing python code, modules with support for env variables
Improvements to gulp to speed up compilation and improved dev experience
Added dependency injection
Modifications to unit tests framework (running pytests, nose, etc in extension) to make use of the new python execution engine along with DI
Code refactoring

Move types into types.ts file
Ensure all promises have exception handlers

Use latest version of typescript

Is this all the work instead of just the tests?

Yes this is, but scooped just around the feature support of running pytest, nose and unit test in the extension,
Other PRs will follow with changes around formation, linting and intellisense and musc (sort imports, etc).

@brettcannon fixed.


Fork Baseline React SSR project out into its own repository for continued development and use

For this project off into its own baseline project, complete with its own project name and documentation, for future use on any React SSR projects.
Acceptance Criteria:

New project exists for the React SSR baseline

Closing in favor of #32


fix(oauth-dashboard): force to delete OAuthClient CR as well when delete secrete for dashboard

do force OAuthClient deletion along with secret deletion
update owner to rhods-operator
update APIVersion for the patch, this is to fix error when we only want to patch OAuthClient from secrectgenerator_contorller:
ERROR	Reconciler error	{"controller": "secret-generator-controller", "controllerGroup": "", "controllerKind": "Secret", "Secret": {"name":"dashboard-oauth-client","namespace":"odh"}, "namespace": "odh", "name": "dashboard-oauth-client", "reconcileID": "067a9ca5-89c8-4353-b0c5-ef241b8946ab", "error": "Incorrect version specified in apply patch. Specified patch version: v1, expected: oauth.openshift.io/v1"}
rename function name
add more comments and debug info message
fix: https://github.com/opendatahub-io/opendatahub-operator/issues/712

new: without force deletion oauthclient
live build quay.io/wenzhou/rhods-operator-catalog:v2.4.122-1
old:
live buid to be tested quay.io/wenzhou/opendatahub-operator-catalog:v2.4.122-0

close this one in favor of PR 136 to rhods-2.5


aysnc auth middleware keeps flipping out upon logout

I'm trying the concept of auth middleware that first goes after the store, and then an endpoint to verify/refresh the logged in user:
So my middleware/auth.js is as so:
export default async function (ctx) {
  if (ctx.store.state.user) {
    return true
  } else {
    try {
      const result = await ctx.app.$axios.get('/me?json=true')
      if (result.data.data) {
        ctx.store.commit('SET_USER', result.data.data)
        return true
      }
    } catch (e) {
      ctx.redirect('/')
      return true
    }
  }
}

My problem is when I Logout and then redirect the user to the front page via the way I found to do it in issue via this.$router.replace({path: '/'}) I get this endless loop:

here is my logout method:
    logout () {
      this.$axios
        .get('/logout')
        .then((response) => {
          this.$store.commit('SET_USER', null)
          this.$refs.toast.add({type: 'info', title: 'Authentication', body: 'Logout Successful'})
          this.$router.replace({path: '/'})
          // this.$router.go('/')
        })
    },

P.S. any tips on how I can handle things smoother please let me know!

Hello,
Where do you import your middleware ? In your page, layout ?
I think you use your middleware in all pages, don't use your middleware in your / route, and use it only for your protected page :)
You can see it here : https://nuxtjs.org/examples/auth-routes#__nuxt
Have a good day ;)


Calendar shows wrong week numbers

Hi. When I open the calendar, it shows wrong week numbers for 2016. Monday january 4th is the start of week 1 (at least it is so in Norway).
I notice that with the common unix tool cal, run with cal -w, I get the same behaviour as with this plugin. However, if I state that the week starts on Monday, i.e. cal -wm, then the unix tool correctly shows that the week number starts with 1 on January 4th.

Okay. I'll fix this. I didn't know the correct definition of week number...
https://en.wikipedia.org/wiki/ISO_week_date
https://en.wikipedia.org/wiki/Week#Week_numbering
Due to the above links, the week 1 is assign to the week containing 1/4 on calendars beginning with Monday, and to the week containing 1/1 on calendars beginning with Sunday. The current implementation seem to follow the latter rule.

Fortunately the week numbers in the last year are correct because 2015/1/4 is Saturday.

Thanks for the links, interesting to read about the correct definitions of the first week number.

Thanks! :D

:)


Read connectionstring and endpoint from config

This PR will allow for setting these by appsettings.json or environment variables. It will also correct a typo in the settings MongStore. Changed compare branch and base branch from #160.

@kennethmyhra refreshed version of #160, towards develop branch. Please note that I tried to add a singleton with configuration in 2ff58d8 as well as a frontpage showing the endpoint. Just revert if not needed!

Thanks @losolio, merged and moved forward to stu3/develop and r4/develop, did some minor refactoring in a4b237fdd


fix(WKWebView): StatusBar is gone after fullscreen (iOS 12)

Summary
This fixes #62 for iOS 12. It listens for a specific notification that is only fired when entering / exiting fullscreen. I tested on simulator and device with iOS 12.2. This PR also attempts to preserve the state of the status bars when the web view was initialized.

@ericlewis where did _MRMediaRemotePlayerSupportedCommandsDidChangeNotification come from? I can't find any documentation about that.
Trying to figure out why the statusbar's dissapear in some use cases, explained here: https://github.com/react-native-community/react-native-webview/issues/735

@ericlewis, can you comment on my question above?
Because that event also fires even when there's no fullscreen activity as explained here: https://github.com/react-native-community/react-native-webview/issues/735
There's no documentation to be found about that _MRMediaRemotePlayerSupportedCommandsDidChangeNotification


Enabling CoreML Support produces link error with v1.3.0

Whisper.cpp v1.3.0 has optional support for CoreML acceleration, which speeds up inference significantly (about 4x for me on a MacBook Air M1). To test this out, I added -DWHISPER_COREML=1 to the cmake command in build.rs, but got an error in the linking step:
 = note: Undefined symbols for architecture arm64:
            "_whisper_coreml_encode", referenced from:
                whisper_encode_internal(whisper_context&, whisper_state&, int, int) in libwhisper_rs_sys-c9402f202599effc.rlib(whisper.cpp.o)
            "_whisper_coreml_free", referenced from:
                _whisper_free_state in libwhisper_rs_sys-c9402f202599effc.rlib(whisper.cpp.o)
            "_whisper_coreml_init", referenced from:
                _whisper_init_state in libwhisper_rs_sys-c9402f202599effc.rlib(whisper.cpp.o)
          ld: symbol(s) not found for architecture arm64
          clang: error: linker command failed with exit code 1 (use -v to see invocation)

My guess is that this an upstream issue where the symbols aren't getting exported publicly when building the library, but my C++ isn't good enough to figure out a solution. Hoping that someone can verify that there isn't anything else we should be doing to enable this.

I have no way to check this, so your best chance at getting this fixed is going to be hoping someone else can reproduce it. Sorry about that.

May be closed by #35


clarify @svgr/webpack behavior with file-loader

when using @svgr/webpack together with file-loader, it looks like svgo optimizations are only applied to the React component, but not to the emitted svg file.
not sure if I am doing something wrong, or if this is intended behavior?
small repro adapted from the example/webpack project in this repo.

Hello @stefanprobst, yeah SVGR is intended to transform SVG into React Component, used with webpack it does not touch the emitted SVG file. I thought it was clear enough but I will precise it in documentation.


Add bower support

Hello,
is there any reason to not have bower support (bower.json) like the main project (dustjs-linkedin)?
Regards,
Paulo

+1

Closed by #116


Cannot create connection for Node after tests finished successfully

Hello All!
I'm getting a strange INFO message from reactive mongo when the tests are finished
the following INFO messages occur:
Cannot create connection for Node(<IP_ADDRESS>:27017,Primary,Vector(Connection([id: 0xa86b495b, L:<IP_ADDRESS>/<IP_ADDRESS>:52666],Connecting,Set(),None)),Set(),None,ProtocolMetadata(2.6, 4.2),PingInfo(182239255, 0, -1, None),false)
Full Log Message:
Testing started at 17:51 ...
... TESTS ...
2020-06-20 17:51:44,342 INFO   [laTest-run] - .r.DefaultReactiveMongoApi(64) - ReactiveMongoApi stopping...
{"@timestamp":"2020-06-20T17:51:44.342+03:00","@version":1,"message":"ReactiveMongoApi stopping...","logger_name":"play.modules.reactivemongo.DefaultReactiveMongoApi","thread_name":"ScalaTest-run","level":"INFO","level_value":20000,"application.home":"/Users/Zvi/Desktop/Git (Company)/ServiceName/."}
2020-06-20 17:51:44,364 INFO   [spatcher-2] - .r.DefaultReactiveMongoApi(67) - ReactiveMongoApi connections are stopped
{"@timestamp":"2020-06-20T17:51:44.364+03:00","@version":1,"message":"ReactiveMongoApi connections are stopped","logger_name":"play.modules.reactivemongo.DefaultReactiveMongoApi","thread_name":"application-akka.actor.default-dispatcher-2","level":"INFO","level_value":20000,"application.home":"/Users/Zvi/Desktop/Git (Company)/GuardManager/."}
2020-06-20 17:51:44,364 INFO   [spatcher-2] - r.c.a.MongoDBSystem(44)        - [Supervisor-1/Connection-1] Stopping the MongoDBSystem
{"@timestamp":"2020-06-20T17:51:44.364+03:00","@version":1,"message":"[Supervisor-1/Connection-1] Stopping the MongoDBSystem","logger_name":"reactivemongo.core.actors.MongoDBSystem","thread_name":"reactivemongo-akka.actor.default-dispatcher-2","level":"INFO","level_value":20000,"application.home":"/Users/Zvi/Desktop/Git (Company)/ServiceName/."}
2020-06-20 17:51:44,365 INFO   [spatcher-2] - r.api.Driver(44)               - [Supervisor-1] Closing instance of ReactiveMongo driver
{"@timestamp":"2020-06-20T17:51:44.365+03:00","@version":1,"message":"[Supervisor-1] Closing instance of ReactiveMongo driver","logger_name":"reactivemongo.api.Driver","thread_name":"application-akka.actor.default-dispatcher-2","level":"INFO","level_value":20000,"application.home":"/Users/Zvi/Desktop/Git (Company)/ServiceName/."}
[INFO] [06/20/2020 17:51:44.367] [reactivemongo-akka.actor.default-dispatcher-4] [akka://reactivemongo/user/Connection-1] Message [scala.Tuple2] from Actor[akka://reactivemongo/user/Connection-1#2057401464] to Actor[akka://reactivemongo/user/Connection-1#2057401464] was not delivered. [1] dead letters encountered. If this is not an expected behavior, then [Actor[akka://reactivemongo/user/Connection-1#2057401464]] may have terminated unexpectedly, This logging can be turned off or adjusted with configuration settings 'akka.log-dead-letters' and 'akka.log-dead-letters-during-shutdown'.
[INFO] [06/20/2020 17:51:44.368] [reactivemongo-akka.actor.default-dispatcher-4] [akka://reactivemongo/user/Connection-1] Message [scala.Tuple2] from Actor[akka://reactivemongo/user/Connection-1#2057401464] to Actor[akka://reactivemongo/user/Connection-1#2057401464] was not delivered. [2] dead letters encountered. If this is not an expected behavior, then [Actor[akka://reactivemongo/user/Connection-1#2057401464]] may have terminated unexpectedly, This logging can be turned off or adjusted with configuration settings 'akka.log-dead-letters' and 'akka.log-dead-letters-during-shutdown'.
2020-06-20 17:51:44,375 INFO   [spatcher-5] - r.api.Driver(44)               - [Supervisor-1] Stopping the monitor...
{"@timestamp":"2020-06-20T17:51:44.375+03:00","@version":1,"message":"[Supervisor-1] Stopping the monitor...","logger_name":"reactivemongo.api.Driver","thread_name":"reactivemongo-akka.actor.default-dispatcher-5","level":"INFO","level_value":20000,"application.home":"/Users/Zvi/Desktop/Git (Company)/ServiceName/."}
2020-06-20 17:51:44,406 INFO   [spatcher-6] - r.c.n.ChannelFactory(44)       - [Supervisor-1/Connection-1] Cannot create channel to '<IP_ADDRESS>:27017' from inactive factory
{"@timestamp":"2020-06-20T17:51:44.406+03:00","@version":1,"message":"[Supervisor-1/Connection-1] Cannot create channel to '<IP_ADDRESS>:27017' from inactive factory","logger_name":"reactivemongo.core.nodeset.ChannelFactory","thread_name":"reactivemongo-akka.actor.default-dispatcher-6","level":"INFO","level_value":20000,"application.home":"/Users/Zvi/Desktop/Git (Company)/ServiceName/."}
{"@timestamp":"2020-06-20T17:51:44.407+03:00","@version":1,"message":"[Supervisor-1/Connection-1] Cannot create connection for Node(<IP_ADDRESS>:27017,Primary,Vector(Connection([id: 0xa86b495b, L:<IP_ADDRESS>/<IP_ADDRESS>:52666],Connecting,Set(),None)),Set(),None,ProtocolMetadata(2.6, 4.2),PingInfo(182239255, 0, -1, None),false)","logger_name":"reactivemongo.core.actors.MongoDBSystem","thread_name":"reactivemongo-akka.actor.default-dispatcher-6","level":"INFO","level_value":20000,"application.home":"/Users/Zvi/Desktop/Git (Company)/ServiceName/."}
2020-06-20 17:51:44,407 INFO   [spatcher-6] - r.c.a.MongoDBSystem(44)        - [Supervisor-1/Connection-1] **Cannot create connection for Node(<IP_ADDRESS>:27017,Primary,Vector(Connection([id: 0xa86b495b, L:<IP_ADDRESS>/<IP_ADDRESS>:52666],Connecting,Set(),None)),Set(),None,ProtocolMetadata(2.6, 4.2),PingInfo(182239255, 0, -1, None),false)**
Process finished with exit code 0

Dependencies:
import sbt._

object Dependencies {
  lazy val scalaTest = Seq(
    "org.scalatest" %% "scalatest" % "3.0.5" % Test,
    "org.scalatestplus.play" %% "scalatestplus-play" % "3.1.2" % Test,
    "org.scalacheck" %% "scalacheck" % "1.14.0" % Test,
    "org.pegdown" % "pegdown" % "1.6.0" % Test,
    "de.leanovate.play-mockws" %% "play-mockws" % "2.6.5" % Test,
    "org.elasticmq" %% "elasticmq-rest-sqs" % "0.13.8" % Test,
    "org.mockito" % "mockito-core" % "2.13.0" % Test
  )

  lazy val kamon = Seq(
    "io.kamon" %% "kamon-system-metrics" % "0.6.7",
    "io.kamon" %% "kamon-scala" % "0.6.7",
    "io.kamon" %% "kamon-play-2.6" % "0.6.8",
    "io.kamon" %% "kamon-akka-2.5" % "0.6.8",
    "io.kamon" %% "kamon-datadog" % "0.6.8"
  ).map(_.exclude("org.asynchttpclient", "async-http-client"))

  val logbackVersion = "1.2.3"
  val logging = Seq(
    "com.typesafe.scala-logging" %% "scala-logging" % "3.7.2",
    "net.logstash.logback" % "logstash-logback-encoder" % "4.11",
    "ch.qos.logback" % "logback-core" % logbackVersion,
    "ch.qos.logback" % "logback-classic" % logbackVersion,
    "ch.qos.logback" % "logback-access" % logbackVersion,
    "ch.qos.logback.contrib" % "logback-jackson" % "0.1.5",
    "ch.qos.logback.contrib" % "logback-json-classic" % "0.1.5"
  )

  val reactivemongo = Seq(
    "org.reactivemongo" %% "reactivemongo-akkastream" % "0.18.8",
    "org.reactivemongo" %% "play2-reactivemongo" % "0.18.8-play26"

  )
  lazy val akka = Seq(
    "com.typesafe.akka" %% "akka-testkit" % "2.5.8" % Test withSources(),
    "com.typesafe.akka" %% "akka-stream-testkit" % "2.5.8" % Test withSources(),
    "com.lightbend.akka" %% "akka-stream-alpakka-sqs" % "0.15"
  )

  lazy val enumratum = Seq(
    "com.beachape" %% "enumeratum" % "1.5.12",
    "com.beachape" %% "enumeratum-play-json" % "1.5.12"
  )

  lazy val playJson = Seq("ai.x" %% "play-json-extensions" % "0.14.0")
  lazy val googleBindings = Seq("com.google.inject.extensions" % "guice-multibindings" % "4.1.0")
}

The Test:
Note: There nothing in the test, the injection of the dao making the INFO message
class PrioritiesDaoSpec extends FreeSpec
  with MockitoSugar
  with Matchers
  with GuiceOneAppPerSuite
  with ScalaFutures
  with BeforeAndAfterEach
  with Eventually
  with GeneratorDrivenPropertyChecks
  with Injecting
  with BeforeAndAfterAll
  with IntegrationPatience with LazyLogging {

  override implicit val generatorDrivenConfig: PropertyCheckConfiguration =
    PropertyCheckConfiguration(minSuccessful = 1, sizeRange = 1)

  override val fakeApplication: Application = GuiceApplicationBuilder()
    .disable[com.vatbox.common.ApplicationStarterModule]
    .build()

  private val collection = dbConfig.prioritiesCollection
  private lazy val dbConfig = inject[DBConfig]
  private lazy val dao = inject[PrioritiesDao] // Throws Error

  private implicit val instantFormat: Format[Instant] = CommonSerilizers.InstantFormatBson

  private def cleanCollection() = collection.remove(Json.obj()).futureValue

  override def beforeEach() {
    cleanCollection()
  }
  override def afterEach(): Unit = {
    cleanCollection()
  }

  "getting priorities, there's no priorities" - {
          "be indicated appropriately" in {
            // Nothing in the test - still getting INFO Message
          }
  }
}

PrioritiesDao:
@Singleton
class PrioritiesDao @Inject()(dbConfig: DBConfig, implicit val exec: ExecutionContext) extends BaseDao
...

When DBConfig injects ReactiveMongo
Anyone know why I'm still getting the INFO message?
Thanks!

Hi,
This tracker is not the place to ask question/for help, please use the Mailinglist.


PHP variable & data type edits

cleaned up and merged a number of pages relating to PHP variables and variable data types

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #19324

@chiffonsigh, thanks for taking the time to submit this PR.
I'm all for cleaning up and consolidating things, but I'd just like to get a second opinion before we merge your changes. Just want to make sure that we're not going to lose any valuable info or examples here. @raisedadead, what do you think? Also, is it alright that these changes are coming from the master branch?

@chiffonsigh, thanks for taking the time to submit this PR.
I'm all for cleaning up and consolidating things, but I'd just like to get a second opinion before we merge your changes. Just want to make sure that we're not going to lose any valuable info or examples here. @raisedadead, what do you think? Also, is it alright that these changes are coming from the master branch?

@chiffonsigh, thanks for taking the time to submit this PR.
I'm all for cleaning up and consolidating things, but I'd just like to get a second opinion before we merge your changes. Just want to make sure that we're not going to lose any valuable info or examples here. @raisedadead, what do you think? Also, is it alright that these changes are coming from the master branch?

@scissorsneedfoodtoo we can hardly do anything about contributors not making branches. Branching is clearly mentioned in the Contributing docs.
They would not be able to sync up the repo in a clean way, and most likely end up with deleting an re-forking. So my advise in such scenario will be to simply accept their current PR and leave it be.

@raisedadead, thank you for taking a look and explaining that. Will keep that in mind when reviewing PRs in the future.

Congratulations on your first contribution to freeCodeCamp!
Thank you for helping us improve this section of the guide, @chiffonsigh. We're all really looking forward to your next contribution :+1: :+1:


Automate multisig functionality

https://github.com/input-output-hk/cardano-node/pull/1788
https://github.com/input-output-hk/cardano-node/blob/a3bd79e41363133e098367526ba9c68333ead18c/doc/reference/multisig.md

Manual tests executed on PR1788
Waiting for PR to be merged in order to execute the final testing + automation.

https://github.com/input-output-hk/cardano-node-tests/pull/144
https://github.com/input-output-hk/cardano-node-tests/pull/139

https://github.com/input-output-hk/cardano-node-tests/pull/149


Formatting issue (might be VS specific, not sure where else to mention it)

I know that at some point in the past I didn't have this problem TS, but over the last several months it's been doing some funky stuff when auto-formatting.
http://imgur.com/yVORrUA
It used to move my {}'s over to line up with their ".then", but no longer :(
Any ideas?

this looks like a dupe of #1888 that was already fixed.


making relative links not relative - support, blog, site

Hello @ksigler7,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


How to render icon inside search selection dropdown?

I am using the following for a search selection field as you suggested doing in issue 17, and I am setting this inside widgets={} in forms.py:
"my_field": forms.TextInput(attrs={"_style": "search", "_override": "Select"})
The dropdown icon is rendered on the right side of the field instead of the search icon.  If I do:
"cover_stock": forms.TextInput(attrs={"_style": "search", "_override": "Select", "_icon": "search"})
then nothing changes in the rendering - the dropdown icon is still rendered inside the <input>, not the search icon.  Fortunately, everything else looks right in the rendering and the search field behaves properly, but how can I replace the dropdown icon with the search icon?

Good idea.
Add {"_dropdown_icon": "search"} to your attributes.


For completion, default to links/aliases, make leading #/^ complete for anchors/blocks

🚀 The feature, motivation and pitch
Since v3.7.0, the addition of support for anchor and block search has resulted in the default behavior of the completion to change.
Previously typing [[ would (after a configurable amount of characters) initiate search among the note names and their aliases, much like how Obsidian works. The recent addition of the support for anchors and blocks changes two things:

The completions no longer resemble the default behavior of Obsidian, due to Obsidian requiring a prefix after [[ for initiating a completion different than note names and aliases ([[# for headings in the current note, [[## in all notes; [[^ for blocks in this note, and [[^^ in all notes).
The amount of completions to process in the vault increases, meaning that the time before the completion menu pops up has increased notably.

While having the ability to get completion for anchors and blocks is useful, to both be more in line with the working of Obsidian, and to avoid the performance penalty of always completing for everything (notes, aliases, anchors, and blocks), would it be a good idea to introduce a similar kind of prefix detection (i.e. looking for something like [[#) to the completion triggering?
Alternatives
An alternative would be making it possible to disable the anchor and block completion, to go back to the previous way things worked. The README made me think that this should be possible by changing the wiki_link_func config opt, but (unless I've missed something) this does not seem to do the trick.
Additional context
No response

Hey @linduxed, I like the idea of only giving header/anchor link completion when the current prompt ends with # or ^. That seems perfectly reasonable to me and I think would regain the previous performance. I didn't know about Obsidian's [[## / [[^^ global search prompt, but that seems like a good feature to add here as well.

I like the idea of only giving header/anchor link completion when the current prompt ends with # or ^.

@linduxed I just opened a PR for this. Can you test this out?

Hello @epwalsh! Thank you for the very quick response and code!
I checked out the PR locally and tested. Observations:

The PR does indeed make it so that [[ only searches for note names.
[[ does not show aliases in the completion. Not sure if this is expected or not.
The searching for headers/anchors is either confusing to me as to how it should be used, or something is not working:
a. Writing [[# does not seem to give any results.
b. Writing [[## seems to give a short list of inaccurate results: I have a Purchases.md note with a ## Books heading, and this heading does not get found, while 3 other results that are totally unrelated get returned. The same 3 results seem to be returned for some other queries, so the text after [[## does not seem to matter. For a brief moment, however, there seems to be a flash of a long list of completion results that cover the screen, and maybe those results are good, but the flash is too quickly gone to tell.

Oh, wait, my bad:

when the current prompt ends with # or ^.

I misunderstood. That seems to work. In that case, I guess my only relevant observation is the part about aliases not being shown.

Further observations upon closer inspection:

Now that I understand how to invoke the header functionality, the completions seem to contain false positives. Example: I have a note that's called Purchases.md with a header called ## Books, and a note called Purchase of apartment.md that does not have a header containing the word "books". Nonetheless, Purchase of apartment#books is one of the completions for [[purchase#books.
Aliases are indeed there in the completion for [[, but they get listed in the completion as actual notes, and get inserted as actual note titles. Example: let's presume I have a note called Books.md with the alias To read. To read will be listed as a completion, and if one selects that, at least what one would expect that what would be inserted as text in the document would be [[Books|To read]], but instead [[To read]] is inserted.

Question: how is one to reference headers in the same document with this PR, excluding writing out the name of the current note and then the header (i.e. in a note called Foo.md, writing [[Foo#?

Would you want me to move some of the content in my previous messages to the PR thread, to have it all in context there?


Remove extraneous CORS method

Jira ticket URL:
https://dfedigital.atlassian.net/browse/TEVA-789
Changes in this PR:
Removes a method which introduced complexity and hard-to-solve bugs, and which is not necessary for the functionality to work.
This breaks one test, which should be reimplemented. Set to pending and added TODO for now.
After:
From correct origin (should return results)
Pre-prod

From external origin (developer.mozilla.org: should reject requests)
Pre-prod

Requests to https://localhost:3000 are accepted regardless of origin.
Local

I'm confused.  Wasn't the whole reason we proxied this api endpoint was so we could control access? If we aren't going to do that, then surely we don't need to add the overhead the proxy brings?