Fix for the issue #35

Using Hash#reject! instead of Hash#except!

Since Semantic Logger wants a hash, and we would probably want all parameters, since that is what it appears Rails is logging in #inspect, how about the following change?
        payload[:params] = payload[:params].to_unsafe_h.except(*INTERNAL_PARAMS)


Connect: Expose option to set protocol

Context
The method WithServerHostname will configure the protocol used to http if the value provided is localhost, however, it can be possible to try connector a compliant databricks service via a local proxy that does the TLS termination.
Diagram
service --> reverse proxy --> service
Outcomes
I am open to either outcome of:

New option that can parse a complete URL and apply the WithProtocol, WithHostname, WithPort, WithHTTPPath configuration values
New option that allows forcing http connection via a new option WithHTTPProtocol

Just to verify: you are not using a dsn to connect, you are using NewConnector().

Yes, sorry that is correct.

Version 1.4.0 has been released and allows for including http or https protocol when using WithServerHostname
ex. WithServerHostname("http:my.server.name")

Unless there are still problems with this we will close this issue.


Weighting in excess properties done by ThermoML mole fractions rather than simulation mole fractions

Describe the bug
What we seem to be observing is that when you compute an excess property, and you do the calculation (for enthalpy for example), then with $H_{mixing} = <H_{mixture}>- (MF_0<H_0>-MF_1<H_1>$), the $MF_0$ and $MF_1$ are taken from ThermoML mole fractions, but it is more accurate to take them from the actually simulated mole fractions.
In our case, the ThermoML mole fractions are 0.5098 and 0.4902, and the simulations ended up being done with 510 and 490 molecules, as that is what 509.8 and 490.2 get rounded to.
In this case, one sees that if the molar enthalpy of component 0 is -44.879 and of component 1 is 251.418, and you get a mixture enthalpy 98.855, then if you use the simulation MF, you get (0.510)*(-44.879) + (0.490)*251.418 - 98.855 = -1.45153, whereas if you use the ThermoML MF, you get (0.5098)*(-44.879) + (0.4902)*251.418 - 98.855 = -1.511, which a difference almost about 2x than the uncertainty; this -1.511 number is what Evaluator actually returned.  If the ThemoML MF had been, say 0.4904/0.5096, then you would have had a simulation of 1000 molecules still rounded to 490/510, and using the ThermoML MF would have been (0.5096)*(-44.879) + (0.4904)*251.418 - 98.855 = -1.57.
I would argue that the most important thing is to use the mole fraction consistent with the mole fraction of the mixture solution actually simulated, as you want the ideal solution composition consistent with the mixture composition.  The H_mix(x) is not going to change very much when changing by x by 0.002, but mismatches result in larger errors.  More specifically, because the ideal components of the enthalpy are ~1-2 order of magnitude larger than the nonideal components, mismatching the ideal and nonideal solutions will give errors 1-2 orders of magnitude large than the changes in the nonideal contribution itself.
To Reproduce
Any calculation should should work to show this error, but ones with smaller numbers of molecules so the rounding will give a larger difference.
Output
No error in output log per se. We figured this out by reproducing the calculation of enthalpy and figuring out mole fraction is being used.
Computing environment (please complete the following information):
I don't specifically recall the version of Evaluator - within the last 6 months.
Additional context

Definitely agree this is a problem and your proposed solution makes perfect sense, thanks so much for digging in and figuring out what was happening. (As a note to myself from quickly looking at the code, it looks like we need to construct a new Substance for the WeightByMoleFraction protocol that matches the simulation proportions -- I'll be able to spend more time on this when I'm back.) I don't think this part of the code has changed significantly over the past 6 months.
It sounds like you were working with a specific property -- would you be able to add it here? That would be a great test for fixing the issue.

@SimonBoothroyd is aware of this now as well and may be able to help.  He THOUGHT he had it set to use simulation mole fractions, but it didn't work as intended.  Barbara Morales in my group has the specific example we were using for molar enthalpy and she could add it - but I don't think she's added to the organization?  Let me see if I can do that.

Yeah we should be using the substance output when building the cords: https://github.com/openforcefield/openff-evaluator/blob/82ce709243435846532fddd14dcb76873eac52c7/openff/evaluator/protocols/coordinates.py#L86

I think
https://github.com/openforcefield/openff-evaluator/blob/82ce709243435846532fddd14dcb76873eac52c7/openff/evaluator/properties/properties.py#L173
should instance reference
mixture_protocols.build_coordinates.output_substance

@barmoral, looks like you are on now, you can add some files to reproduce things here.

Hello all! Here I attach a zip file that contains the inputs and outputs I used to run the replicate test. Some details about the test:

Molecule: OCCN(CCO)CCO (selected randomly)
Force field used: Sage 2.1
Water model: TIP3P-FB
System size: 1000 molecules
I focused on the mixture with mole fraction of 0.4902 for the solute and 0.5098 for the solvent/water. The id of this mixture assigned by evaluator is 6421.

As Michael mentioned, you will find the packmol output that states 0.49 and 0.51 were used to create the system. I have run replicates for the same test using system sizes of 1000 and 500 molecules, but at 2 ns. Let me know if these results might be useful as well, or if I can provide any other useful information.
1000_20ns.zip

Thanks @barmoral, that's super helpful. I'm setting up some tests for this now and it'd be super helpful if you could also provide the working data for these if you saved the files, especially those associated with the conditional_group steps.

Hi @lilyminium you can find those files in this github repository under the label test_1000_longer_r#. These are my full results for the runs, excluding trajectory .dcd and .nc files which are really heavy. Let  me know if you'd need some of those to send them separately.

Thanks @barmoral, very much appreciated. Unfortunately to step through the process and create integration tests it'd be most helpful to have the trajectory.dcd files for the conditional group output, to avoid re-running the simulations. Just one of them would be great -- I've been using the test_1000_longer_r1 files as a test case. We could possibly use Google Drive or similar to avoid GitHub's file size limits.


Feature - Obtain values from methods, extension methods and default interface implementations

This could be a weird and terrible thing to want, but I encountered a scenario today where it might have been neat to get values from:

Methods

-and/or-

Extension methods
Default method & property implementations from interfaces

Can you give more detail and example?

Since there is no further detail, I will close for now.
For custom mapping, you can take a look at https://github.com/MapsterMapper/Mapster/wiki/Custom-mapping.

Sorry just have missed the notification earlier.  I'm just on mobile, but you can imagine that it's just sitting values from similarly named methods and extension methods when mapping.

Mapster can map from method to property (map GetName() to Name) (https://github.com/MapsterMapper/Mapster/wiki/Data-types#mappable-objects).
For extension method & interface, I would like understand use cases, why we need those features.

Obviously for the same reason why one would map from a method, why would it be different?


Include active group of the logged user

Author Name: Daniel Molina (Daniel Molina)
Original Redmine Issue: 3791, https://dev.opennebula.org/issues/3791
Original Date: 2015-05-04

@Is it possible to configure Sunstone in order to besides the login user it also appears the group to which the user belongs?
This can be useful if an user belongs to different groups, so if the user changes between between groups, he/she always can see with what group he/she is operating by seeing the group used on the sunstone in the same way that it is showed the login user.@

Sunstone already shows the user group by clicking on the username, closing it.


Logentries log driver update

Proposed changes
Documentation updated to include line-only log option
Related issues (optional)
Described in moby/moby#31726 together with later changes from moby/moby#35626

It seems that the option shipped with 17.09, so let's merge this 🚀


Chips Icon Color

This is more a feature request. I'd like to be able to change icon color based on state on Chips card:
chips:

type: entity
entity: group.teste11
icon: mdi:lightbulb-group
tap_action:
action: toggle
icon_color: |
if (state === 'off') return 'rgb(0, 255, 0');

I think I can be a good idea to have the same behaviour on entity chip and entity card : only colored icon when entity is considered on (on, open, playing, etc...)
@pierrecle ?

Yes, absolutely.


Update imgui to 646df390032f64eb03f8b5e2e5cbaf0c8e3bb237

upgrade Dear ImGui to 646df390032f64eb03f8b5e2e5cbaf0c8e3bb237

wow, this new TextureID as uint64 is so awesome :tada:
We were able to get rid of strange uintptr->unsafe.Pointer conversions! (go nor C liked them)
Thank you @ocornut :heart:

Thanks! but you could have always #define ImTextureID to whichever type you want. I have only changed the default now!


Support "For you" feed in infinite-scroll

Resolve #5716
Test URLs
https://github.com/
Screenshot
https://user-images.githubusercontent.com/44045911/185581428-71a60c63-9803-4b7c-8b2b-acdc14311120.mov

It works. The two buttons are always in the DOM so listeners are only added once.


add option "displayFailedHookAsFailedTest" to not show passing hooks as tests and display failed hooks as failed tests in the correct teamcity format

add option "displayFailedHookAsFailedTest" to not show passing hooks as tests and display failed hooks as failed tests in the correct teamcity format
Proposed changes
add option "displayFailedHookAsFailedTest" to not show passing hooks as tests and display failed hooks as failed tests in the correct teamcity format.
Types of changes
What types of changes does your code introduce
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ x ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ x ] Lint and unit tests pass locally with my changes
[ x ] I have added tests that prove my fix is effective or that my feature works
[ x ] I have added necessary documentation (if appropriate)

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

I got an email saying this (https://teamcity.jetbrains.com/viewLog.html?buildId=3703204&buildTypeId=TeamCityThirdPartyPlugins_MochaTeamcityReporter_Build) failed but I'm not sure what that is for.

Thanks @fernyb for this.
Additional test failures would be expected coming from the validate reporter if you have added more tests that expose failures to be reported in TeamCity.
I will have a good play with these changes and ill update the expected failed test count in on the validate reporter configuration to match the new expected count assuming they are all expected failures.

Hey @fernyb, have thought of  some potentially breaking scenario for these changes, open to any suggestions.
Now that all test results are getting proceed at the end what if a test caused the mocha process to crash or timeout, no test results will get propagated right? Under the previous implementation at least previous tests would be propagated to Teamcity messages as they came through.
Received timestamps on Teamcity are not going to be accurate in relation to the service message, does this matter or should every message now have a timestamp property?
I would prefer to keep the approach of logging messages as soon as events come through, can you think of a way to implement the displayFailedHookAsFailedTest functionality without processing all messages at the end? Potentially a way to mitigate lost messages would be to process them in chunks per each hook?
Also if anyone else if requiring this change please comment so we can establish potential usage of it.

@fernyb tries to solve really annoying issue:
We do not really want to see hooks in TC tests section if they are executed successfully.
But we do want to see these hooks in TC tests section if they failed.
Some time ago I fixed an issue https://github.com/travisjeffery/mocha-teamcity-reporter/issues/35 by adding testStarted to every hook because without it TeamCity was not able to show failed hook as a failed test (with recordHookFailures=true). Actually I tried to fix this issue by simple code change.
It was not the best fix, because it caused an issue with showing all hooks as tests (when recordHookFailures=true). So I added skipHooksWithTag in scope of https://github.com/travisjeffery/mocha-teamcity-reporter/issues/56.
I believe this feature is needed. But I am not sure what is the best way to implement it. I have a feeling that it should be a part of recordHookFailures. And behavior with logging hooks should be reworked. Probably ignoreHookWithName should be removed at all.
And depending on your decision, I would suggest that you should amend the following description (if new flag is going to be added):

Here we should add a notice that all hooks are treated as tests if flag is enabled.

Thanks for the Info @DJ-Glock
An alternative right now could be to publish this PR as a pre-release version if there is some demand for this.
I still think a alternative way needs to be thought of to eliminate the risk of loosing log messages and timestamp inconstancies or as you suggested the new functionality here is controlled all by a feature flag and the old way of logging messages is retained when the feature is not enabled.

Hi. I am very interested in trying out this PR. We sometimes run into errors in hooks, and to have them show up in TeamCity would speed up troubleshooting.


Create an SAP Fiori App Using SAP Business Application Studio

Tutorial URL: https://developers.sap.com/tutorials/appstudio-fioriapps-create.html
Please specify the step you are referring to

Great tutorial, but not enough context along the way.
What is /BusinessPartnerSet?  Is that a predefined bapi provider?
The question about other templates where the answer YEOMAN Templates?
I didn't see that information along the tutorial.
Thanks@!

Identical to https://github.com/SAPDocuments/Tutorials/issues/5956


Is Ramda ever planning on being as safe as lodash?

For example
R.filter(R.whereEq({ foo: 'bar' }))(baz)
breaks when baz = undefined and baz = [..., undefined], is there any way to compose around this?

Check out R.defaultTo

The two initial founders of Ramda, @buzzdecafe and I, started with a strong sense that we were not interested in hand-holding.  We didn't want to type-check your arguments for you or worry overmuch about what would happen if you supplied bad ones.  Garbage in, garbage out.  The third person to join our development, @davidchambers, was more interested in this, and in avoiding partial function, and eventually he started an entirely different library where these concerns are central.  Sanctuary has become a very interesting alternative to Ramda, with a reasonable overlap, but also some very different concerns.  It is likely much safer than lodash.
But now, as to that safety:
const gt5 = x > x > 5

_.filter([5, 6, 7], gt5) //=> [6, 7]
_.filter({a: 5, b: 6, c: 7}, gt5) //=> [6, 7]
_.filter(undefined, gt5) //=> []

R.filter(gt5, [5, 6, 7]) //=> [6, 7]
R.filter(gt5, {a: 5, b: 6, c: 7} //=> {b: 6, c: 7}
R.filter(gt5, undefined) //~> error

So lodash has the ability to generate a reasonable default value if your filterable object is missing.  Ramda, which arguably has a more powerful filter function, cannot do so.  What would it be?  An empty object?  An empty array?  An empty member of your custom Filterable type?
There is certainly a case to be made for more useful error messages.  While I've softened on this point a bit, I haven't changed much, and I don't think @buzzdecafe's position has altered even as far as mine.  Writing and maintaining that sort of code does not interest me much.

@CrossEye is this a typo?
const gt5 = x => x > 5

@alanwei0:  Yes it was, thank you.  Fixed.

@CrossEye How did you fixed? They are the same.

@atilkan:
I corrected a typo.  I originally wrote: const gt5 = x > x > 5.  I edited my answer to say instead const gt5 = x => x > 5. (replaced the first > with =>.)
You can edit your comments using the dropdown menu in the top-right of the comment header (...).  You can see the edits to a comment in a dropdown (edited) in the comment header.

Ohh, thank you for the detailed information. I would never be able to find that dropdown despite the fact i am 10+ years member.


Make target for building docs with sphinx

We should have a make target for building the docs using sphinx. It should be used in the github action for building the website.

This could be done in conjunction with https://github.com/CDCgov/PyRenew/issues/231

This could be done in conjunction with #231

Added Dylan on here given this statement.

@damonbayer can you clarify whether you want this in the top-level project directory or in the docs subdirectory? Sphinx has its own standard Makefile for the docs subdirectory. We could just reference that from the top-level, depending on how Harmful we Consider Recursive Make.

@dylanhmorris I don't have any opinion on the location. I just want a one line command that lets you see the rendered docs locally. Apologies if that already exists, and I missed it.

I think for now let's go with make html within docs, which as of 3a3a566 will render the tutorials to markdown (provided you've deleted the placeholders and/or modified the .qmd source) and then render the website as html locally via sphinx-build. We could tweak it a bit more to handle the placeholders better, but I'd rather just do #459

Addressed by #457


Expose keywordTypes

We need it for developing our plugin which needs to override your parser.readWord. Thanks!

Sure, that should be okay.

@marijnh When can we have this released?

@marijnh Please release it soon since our package depends on this.

4.0.5 is out


Cannot add API Key in n8n

Hi!
I'm trying to add my current token in n8n self-hosted, but I cannot add the token through the Workflow or Credentials sections.
I installed it manually and through the GUI. The Vikunja option appears in the Workflow, but the token cannot be added.
I tried restarting Docker and using versions 1.41.0 and 1.42.0, but the issue persists.

Does it show up if you search for "Vikinja" in the cl"add new credential Dialoge"?

My bad, wrote the comment while on the go :)
Not sure why that's the case. Since it works fine in general, I guess this is a problem with n8n. Can you verify the Vikunja nodes package was installed correctly?

I am facing the same problem after installing version 1.41.0. I am not able to add Vikunja API key exactly as shown in the image from @Niskeletor

Yes, I have made sure it will be installed, through GUI and . It is strange, because if nobody has reported any issue I will try to check my n8n instance.

I've just published a new version with updated dependencies. Does it work if you upgrade to 0.1.3?

Wow, now it appears!
I just tested it and it works correctly.
I have also done a couple of Get tests and it shows me the information.
I am very grateful, I must admit that I love Vikunja's project, I hope I can contribute to the development with you in the near future.

I guess I need to get into the habit of publishing these more often 🤓


Refactor value converters and implement ExtractValues for Row

Pull Request Description
This PR proposes a fix for #72 , which includes the following changes:

The logic of boolConvert, dateTimeConvert, etc are decoupled from Row.ToStruct. Now, each value type has its own Convert method, which transforms its value into a reflect.Value. All Convert methods are taken as is from the logic used by Row.ToStruct. Moreover, each value type is placed in its own go source file because the value.go file has too many lines.
A new method called ExtractValues is added to struct Row. This method implements the intended functionality of Columns, which allows users to unpack the row's values into interfaces, enabling convenient and simplified access to these values.
Fix the docs of Row.Columns.

ExtractValues can be used like this:
var userId int64
var userName string
row.ExtractValues(&userId, &userName)

Future Release Comment
Features:

Row.ExtractValues is added to easily unpack a row's values into a set of interfaces.

It seems like that in the refactoring we lost this logic before each conversion (for each type it's slightly different of course):
val, ok := k.(value.Bool)
	if !ok {
		return fmt.Errorf("Column %s is type %s was trying to store a KustoValue type of %T", col.Name, col.Type, k)
	}

Is there a reason for removing this?

It seems like that in the refactoring we lost this logic before each conversion (for each type it's slightly different of course):
val, ok := k.(value.Bool)
	if !ok {
		return fmt.Errorf("Column %s is type %s was trying to store a KustoValue type of %T", col.Name, col.Type, k)
	}

Is there a reason for removing this?
Are there more changes to note?

To my best understanding, such a type conversion is not needed if we have a polymorphism method Convert on the interface value.Kusto. In the prior implementation, each convert method accepts different types of value.Kusto, so we'll have to manually do type casting before passing the values into these methods. Now, we can call the convert method on value.Kusto directly, and the implementation that will be executed depends on which type the value is. This also implicitly impose a constraint on each type inherited from value.Kusto where the type must have a Convert method to do such conversion.
This code snippet may help you understand the idea: https://go.dev/play/p/j4rZWDJrdCu

Got it.
First of all, can you merge from main? there had been some changes and I want to resolve the conflicts early.
For now, can you expand the test you wrote to cover all types, just to make sure we didn't miss anything.
I also think we should check ExtractValue in etoe tests, I might add it myself to this PR.


Submission template needs to allow NR in Variant Count column

Curators need to be able to enter NR in the Variant Count column.

At the moment, an error is received:

as first step investigate this and estimate amount of work/implications

as first step investigate this and estimate amount of work/implications

seems like NR cannot be entered in the curation interface either, so the issue here is to allow curators to not enter SNPn, whilst forcing users to enter it, so could be something in the format -1 as proposed in the sprint meeting. Seems like NR is displayed in the user interface, if nothing is entered in the curation interface (needs verifying)

seems like NR cannot be entered in the curation interface either, so the issue here is to allow curators to not enter SNPn, whilst forcing users to enter it, so could be something in the format -1 as proposed in the sprint meeting. Seems like NR is displayed in the user interface, if nothing is entered in the curation interface (needs verifying)

I can't enable this change from the template schema alone unless you want the field to be optional for all. With curator status, if I create a submission from an "open for submission" type, the template that gets used is "METADATA", which is the template that is used for non-curators.
To understand this we need to understand the call between the backend and template-service - this might be something @sajo-ebi can help investigate

Tested -1 in variant count column for 601d5e01b13a9000016e3d15 / GCST90013480 and it validated, and nothing was imported in the variant count field.

Since we have a workaround, I think we can close this ticket - if you agree @jdhayhurst

We're going to run another test in sandbox to see how the data is materialised in the DB.

note - blocked by problems in sandbox

@earlEBI and @ljwh2 I've tested in sandbox and can confirm that a variant count of -1 is imported to curation db as Null.


Update descriptor-manifest-json-74038a5.md

Hi,
minor change to make the example manifest.json a valid JSON.
I came back to this page a few times to quickly grab the example and test a few things. Might as well make it a proper/valid JSON example to use copy/paste on.
BR.

Hi @wridgeu, thanks for your contribution! We'll look into it and get back to you.

I have updated the documentation for 1.129.

Thank you for your valuable content contribution, @wridgeu! So that we can recognize your contribution in the SAP Community, please check your SAP Community user ID (this is a number) in your personal settings page and share it with us in a reply to this comment. Make sure you just include the number in the reply.
Your user ID is displayed as follows:
Change display name for User ID N
where N is your user ID. For example, 53 is the user ID of the user 'qmacro'.
Please note that we are currently refactoring our profile and badge system on the SAP Community, and will start assigning badges again when that's complete.

11821


Orchard.Layouts css class should be prefixed by .layout-designer

In order not to mess up with clent theme classes with same names, think that Orchard.Layouts css class should be prefixed by .layout-designer.
For example, in default-grid.css instead of ".row" shoud use ".layout-designer .row" otherwise in a bootastrap theme,  it overrides bootstrap row class.
cc:/ @sfmskywalker @DanielStolt

Well, you should override parts.layout in order to suppress the inclusion of that script file anyway, if you're using Bootstrap's grid system.

Ideally there should be a site setting "Use Bootstrap 3" as this is the pretty much the first thing everyone is going to have to mod in order to make the feature useful :)

Ideally, backend styling should not mess up frontend theme by any means. Maybe, not requesting default-grid.css as client is the solution.
@bleroy this should be used out of the box, although your solution is accurate for the time being.
@rtpHarry there are many ui frameworks that use .row class, not only bootstrap 3 :-)

No, I'm pretty sure Bootstrap have filed a trademark claim for .row... :)

FYI you can totally de-active the Layouts grid by overriding the CSS file (default-grid.css?) in your theme/dependent module even with an empty file.

Would it be an idea to not include the default-grid.css from the module's view, but from TheThemeMachine? In other words, using the default-grid would become opt-in, instead of opt-out.

@rtpHarry lol
@piedone sure
@sipke sounds good to me

I keep thinking about the user that isn't a developer. If you install the theme via the gallery then should Orchard let you toggle the grid include via the admin panel?
I think @sfmskywalker was right that it's the theme author's responsibility to support modifications to the Layouts module in their theme (#5788).
If we make the grid opt-in via code only then it breaks compatibility in every theme that you can currently install via the gallery.

Until we decide to declare that Bootstrap is our default choice for front-end themes, we can't bake the Bootstrap grid in. And if we decide that BS is the default, then the default theme that Orchard ships with must be a Bootstrap theme. I'm all for it, but it's a bigger decision than this bug encompasses.

I was thinking about a solution that modules expose only "client" stylesheets and hide "admin" ones...

You mean there's a problem that modules have admin stylesheets?

Yes, I think there should be a way to distinct them.

And, there are unnecessary calls to "admin" stylesheets client-side.

I'm still not sure I understand what problem you're creating a solution for.
But if you're seeing calls to admin specific stylesheets then that's most likely a bug that is probably caused by some view including the wrong style.
Could you please tell me what admin stylesheet is being loaded from the front-end?

For example, I have a site with bootstrap theme. When I create a page with layout, default-grid.css row style overrides bootstrap one. @bleroy and @piedone solutions work but in fact I do not need to load default-grid.css...

Ah. But default-grid.css is intended to be loaded on the front-end so that the layout looks good in themes that don't use a grid system, such as TheThemeMachine. If your theme does use a grid system, then the way to handle that is by creating shape overrides, as you would do with any other shape template that needs to be tuned for your theme.

That's clear. If it is intended behavior, it is ok. But still, sounds strange to me. It was just a point,  feel free to close issue. Cheers.

I get that it would sound strange if you consider "default-grid.css" to be an admin stylesheet, but it really isn;t intended just for the admin. Like Bootstrap, which could be used for both back end as well as front end, "default-grid.css" is a general-purpose, lightweight grid system. It's useful for the back end as well as the front end, as it doesn't require a dependency on heavier-weight grid systems.

I think the best solution is to have the default-grid.css inside TheThemeMachine and inside the module so that the admin can require it. This way a child theme can override the content, and a new theme doesn't have to inject it.

Looks like a good idea, but may break existing themes that rely on it.

Yes, it's potentially breaking, this is why I assigned to dev. We'll have to document it in the release notes.

Actually moving it to 1.10.


Typescript data/ and config/formats

Also removes Battle Factory methods accidentally re-added in d0a4a689a721ceb602a3d812a332d3080b5e881e.
Sorry for the large changelog but it was easiest to do everything together. I had to split Format.desc into desc: string and threads: string[] to be able to reuse other interfaces.

I massively approve this!

This is definitely worth merging now and figuring out minor problems later, though.


Updated Texture.glsl To properly work with Textures and SPIR-V

Textures where showing up incorrectly
Textures are having weird white stripes in rendering
PR impact:

Impact
Issue/PR

Issues this solves
None

Other PRs this solves
None

Proposed fix
Update Texture.glsl to handle flat TexIndex

This is literally the same code as the pull request i made 10 days ago.
Duplicate of #464

Oops, didn't catch that one!


Bump octoxlabs 0.4.4 to 0.5.0 in /docker/octoxlabs

Status
Ready
Related Content Pull Request
This will be added after this image deployed to demisto docker hub organization.
Description
Update octoxlabs package to 0.5.0
Also details: https://github.com/octoxlabs/py-octoxlabs

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hi @ogulcanhero, thank you for you contribution!
We had some technical issues with Github that have affected our ability to properly review the PR.
I'll review your PR shortly. Thank you for your patience.

Hi @samuelFain,
Thank you for your review.
Yes, that's right, I ran the pipenv update command and wanted the lock file to change because I want the octoxlabs module that I will use in the "content" to be in the 0.5.0 version, so I think just updating pipenv will be enough.
Have a nice day.

@ogulcanhero Thank you for your clarification.
Due to a technical issue we had with Github, can you please re-sign the Contributor License Agreement? I believe It is currently blocking the PR from running all its tests (Github actions). Also please merge the the changes from the base branch if needed.
Thank you!

Hi @samuelFain,
I re-signed the CLA. License fyi.
Thank you!

Hi @samuelFain,
Auto merge is not possible due to verify. How can we solve this?
Thank You!

Hi @samuelFain, Auto merge is not possible due to verify. How can we solve this? Thank You!

Hi @ogulcanhero
I'm working on it, apologies on the delay.

@ogulcanhero We are currently handling a technical issue which prevents the ci/gitlab/gitlab.xdr.pan.local Github action to run on contribution PRs in this repository.
To avoid delaying the merging of changes in this PR, I have manually opened an internal PR https://github.com/demisto/dockerfiles/pull/34709 with your changes and merged it.
Please do not close this PR, as we will use it to verify our fix for this issue. However, you can consider your contribution as merged at this time.
Thank you for your understanding.

Thank you @samuelFain, I'm sorry to bother you, good job.


Add .NET 5 nightly test

Just want to note, I did notice that all the inputs are failing (probably due to your change in the create experience) but I know that you know that.  But didn't want you to think that I was being totally negligent in reviewing 🤣


Add CSS containment features

The situation seems to be this:

Size, layout and paint containment shipped together in all browsers
Style containment came later in Firefox. It didn't work with quotes in Chrome initially, and still doesn't in Safari.
Inline-size containment is a later addition in all browsers

https://caniuse.com/css-containment seems to cover initial support for the contain property, which means size/layout/paint containment.
The way I've split it in this PR is like the above bullet points. Another way of splitting it would be to have the 5 types of containment as individual features, and later do a composite feature for them.
I need some expert feedback to decide on how to best represent this to web developers. @vmpstr how do you think this is best structured so that web developer can understand it?

FWIW, I think style containment was marked as at-risk for a while, which is why it came later.
I have a slight preference for splitting all of the features separately. But if that's too verbose, then the structure you have seems fine: we have a "base initial set", which is size layout and paint, and then we have individual things that were added later

I have a slight preference for splitting all of the features separately.

I think we should go with that. We could later consider a composite feature for the whole contain CSS property if it stops getting new functionality.

@foolip This needs a merge/rebase to continue, I think.


[DEN-231] Create ARCHITECTURE.md

@stevenpaz I like what you have here, thank you! The content is good. We should try to maintain similarities between the SDK and terraform repo with that we cover in the readme vs architecture documents. Feel free to discuss with the team as well.


Profile request

I'd like to have my profile on ghuser.io, thanks!

here you go :) https://ghuser.io/jackw01


Images?

Hi, thank you for extension. It works perfectly except for images.
Is there any way to add images via URL or UUID?
It always fails  with "Network error", no pdf created. I can add images as base64 string, but i didn't come up with solution to encode them on the fly in flows yet.
And I'm wondering if there's an easier way with a URL or  UUID?
I'm running Directus 11.2.2 in docker.
Extension version 1.2.0 from marketplace

What version of the extension do you have?

Version 1.2.0 from marketplace.
I made a simple flow for tests with manual launch. I checked with links in the content [] and in images {}. Both uuid and direct links to <IP_ADDRESS>:8055(and external link)/assets/uuid and just images from the Internet with a direct link.
It freezes for a minute and shows a pop-up with network error.

So with the template
{
    "content": [
        {
            "image": "testimage"
        },
        "TEST"
    ],
    "images": {
        "testimage": "http://<IP_ADDRESS>:8055/assets/5978bd41-03bf-46bf-966a-533b42dff008"
    }
}

in console
You have triggered an unhandledRejection, you may have forgotten to catch a Promi
se rejection:
ReferenceError: XMLHttpRequest is not defined
    at file:///directus/extensions/.registry/d0e045f1-8679-4d27-b6e3-1961781fec0f
/dist/api.js?t=1732828511187:32:68865
    at new Promise (<anonymous>)
    at file:///directus/extensions/.registry/d0e045f1-8679-4d27-b6e3-1961781fec0f
/dist/api.js?t=1732828511187:32:68832
    at file:///directus/extensions/.registry/d0e045f1-8679-4d27-b6e3-1961781fec0f
/dist/api.js?t=1732828511187:32:69429
    at new Promise (<anonymous>)
    at s.resolve (file:///directus/extensions/.registry/d0e045f1-8679-4d27-b6e3-1
961781fec0f/dist/api.js?t=1732828511187:32:68681)
    at r._createDoc (file:///directus/extensions/.registry/d0e045f1-8679-4d27-b6e
3-1961781fec0f/dist/api.js?t=1732828511187:32:71568)
    at r.getBuffer (file:///directus/extensions/.registry/d0e045f1-8679-4d27-b6e3
-1961781fec0f/dist/api.js?t=1732828511187:32:73703)
    at file:///directus/extensions/.registry/d0e045f1-8679-4d27-b6e3-1961781fec0f
/dist/api.js?t=1732828511187:32:1674655
    at new Promise (<anonymous>)    ```

ok, I have uninstalled the extension and recreated the container.  I installed the extension manually by uploading the folder from here to the extensions folder. And I finally saw the fields for the images as seen in the source code.
This is strange.
As I see the links in the body of the content do not work.
I can dynamically upload images by keys in the endpoint settings. But empty links cause an error and interrupt the creation of the document. I suppose I need to check the number of images in flow and replace the missing images with one pixel or something
Is there any direct way to use url in the content[]?
I'm sorry if my questions are dumb. I'm certified Copy&paste coder 🤣.

Don’t worry @P43YM. What exactly is your need? That way, I can help by either creating a patch to fix the issue, a specific tutorial for your case, or adding a new feature.
Kind regards!

Hi @somaromero . Thank you.
I have a collection that contains 1-10 images randomly.
I'm trying to figure out how to create a single template and generate a pdf with all the images.
If I can't use the url in the content, then I have to create the maximum number of images in the library but use the key instead of the URL (images with a null url work until I put a link to them in the content). And maybe generate an array like { "image": "image1"}, into the content depending on the number of uuids that I received earlier. pdfmake does not generate a document with images without a link.
Or you can make your fetch functions check if the links are null and serve blank base64 string to cheat pdfmake. this way we can at least make template with all images and dynamically supply links to each item in images{}

As you mentioned, you can generate a script node before the PDF generator node that creates a random array with the image keys.
That's right, I can upload an empty image instead of the missing ones, at the stage of collecting uuids. and add a link to the item in the array to the library. I will share my solution here, for those who come after me. I think this will be the easiest solution. I'm already just confused with all these attempts. And the lack of a quick flow test feature is already driving me crazy. ,
Thank you @somaromero again.

Don’t worry, it happens to everyone. Often, all it takes is explaining your problem to someone else to realize how you can fix it yourself.😂😂 We’re always here and ready to help!

It works!
So after feeding through another flow list of relational ids i have in response array with uuids like this
[
  {
    "result": "5978bd41-03bf-46bf-966a-533b42dff008"
  },
  {
    "result": "cb97510e-78fb-46a3-9d26-5721f42e6ce4"
  },
  {
    "result": "de053dfb-d71d-467e-b604-0ab3a635671d"
  },
  {
    "result": "242220f0-690a-4adb-9948-b152e0c8e7f0"
  },
  {
    "result": "40ab7ecf-005b-49bd-be8d-b17673504f03"
  },
  {
    "result": "82048699-666f-4565-96e4-d90fa39acc33"
  }
]

I downloaded blank.jpg from wikipedia, lol.  jpg with 1 to 1 pixel
here is my script that feeds uuid of my blank to array if there is less then 10 items
module.exports = async function(data) {
    const arraydata = data.another_flow; //response from another flow
    const requiredLength = 10;
    const specificUUID = "18be9f16-d47f-4e56-b881-014d99efe08e"; // UUID of blank.jpg

    while (arraydata.length < requiredLength) {
        arraydata.push({ result: specificUUID });
    }

    return { result: arraydata };
};

I added 10 images to the library with a key instead of a url like{{exec_wpk1h.result[0].result}}
And my template for testing
{
    "content": [
        {
            "image": "image1"
        },
        {
            "image": "image2"
        },
        {
            "image": "image3"
        },
        {
            "image": "image4"
        },
        {
            "image": "image5"
        },
        {
            "image": "image6"
        },
        {
            "image": "image7"
        },
        {
            "image": "image8"
        },
        {
            "image": "image9"
        },
        {
            "image": "image10"
        }
    ],
    "styles": {
        "header": {
            "fontSize": 14,
            "bold": "true"
        },
        "subheader": {
            "fontSize": 13,
            "bold": "true"
        },
        "quote": {
            "italics": "true"
        },
        "small": {
            "fontSize": 8
        }
    }
}

Works like a charm

Correction: it's better to use blank.jpg with 550x1px dimensions. That way i can use "width": 550 to fit the page and blank will not be scaled into a huge square.
Here's my blank


Support CSDGM Data Quality

The CSDGM standard data quality reports need to be supported in mdJSON and in output to CSDGM XML. In the case of the CSDGM reader, data quality is currently lost on translation.
An analysis has been done to map CSDGM data quality to ISO data quality:
Attribute Accuracy Report (attaccr)

nonQuantitativeAttributeCorrectness > descriptiveResult = Attribute Accuracy Report (attraccr)
quantitativeAttributeAccuracy > quantitativeResult > value = Attribute Accuracy Value (attraccv)
quantitativeAttributeAccuracy > evaluationMethod > evaluationMethodDescription = Attribute Accuracy

Quantitative Attribute Accuracy Report Assessment (qattracc)

quantitativeAttributeAccuracy > quantitativeResult > value = Attribute Accuracy Value (attraccv)
quantitativeAttributeAccuracy > evaluationMethod > evaluationMethodDescription = Attribute Accuracy Explanation (attracce)

Logical Consistency Report (logic)

conceptualConsistency > descriptiveResult = Logical Consistency Report (logic)

Completeness Report (complete)

completenessOmission > descriptiveResult = Completeness Report (complete)

Horizontal Positional Accuracy Report (horizpar)

absoluteExternalPositionalAccuracy > descriptiveResult = Horizontal Positional Accuracy Report (horizpar)

Horizontal Positional Accuracy Assessment (qhorizpa)

absoluteExternalPositionalAccuracy > quantitativeResult > value = Horizontal Positional Accuracy Value (horizpav)
absoluteExternalPositionalAccuracy > evaluationMethod > evaluationMethodDescription = Horizontal Positional Accuracy Explanation (horizpae)

Vertical Positional Accuracy Report (vertaccr)

absoluteExternalPositionalAccuracy > descriptiveResult = Vertical Positional Accuracy Report (vertaccr)

Vertical Positional Accuracy Assessment (qvertpa)

absoluteExternalPositionalAccuracy > quantitativeResult > value = Vertical Positional Accuracy Value (vertaccv)
absoluteExternalPositionalAccuracy > evaluationMethod > evaluationMethodDescription = Vertical Positional Accuracy Explanation (vertacce)

@dwalt Here is the final mapping:
2.1 attracc
2.1.1 attraccr -> descriptive result statement (descriptive result name = 'Attribute Accuracy Report', report type DQ_NonQuantitativeAttributeCorrectness)
2.1.2 qattracc (report type DQ_QuantitativeAttributeAccuracy)
<IP_ADDRESS> attraccv -> quantitative result value (quantitative result name = 'Attribute Accuracy Value')
<IP_ADDRESS> attracce -> evaluation method description
2.2 logic -> descriptive result statement (descriptive result name = 'Logical Consistency Report', report type DQ_ConceptualConsistency)
2.3 complete -> descriptive result statement (descriptive result name = 'Completeness Report', report type DQ_CompletenessOmission)
2.4 posacc
2.4.1 horizpa (report type DQ_AbsoluteExternalPositionalAccuracy)
<IP_ADDRESS> horizpar -> descriptive result statement (descriptive result name = 'Horizontal Positional Accuracy Report')
<IP_ADDRESS> qhorizpa
<IP_ADDRESS>.1 horizpav -> quantitative result value (quantitative result name = 'Horizontal Positional Accuracy Value')
<IP_ADDRESS>.2 horizpae -> descriptive result statement (descriptive result name = 'Horizontal Positional Accuracy Explanation')
2.4.2 vertacc (report type DQ_AbsoluteExternalPositionalAccuracy)
<IP_ADDRESS> vertaccr -> descriptive result statement (descriptive result name = 'Vertical Positional Accuracy Report')
<IP_ADDRESS> qvertpa
<IP_ADDRESS>.1 vertaccv -> quantitative result value (quantitative result name = 'Vertical Positional Accuracy Value')
<IP_ADDRESS>.2 vertacce -> descriptive result statement (descriptive result name = 'Vertical Positional Accuracy Explanation')
There are 2 that do not follow the specified mapping: <IP_ADDRESS>.2 horizpae & <IP_ADDRESS>.2 vertacce
Instead of using the evaluation method I decided to go with descriptive result so I could add a name that allows me to distinguish the difference between horizontal and vertical. I also added names where I could to state what the result was associated with.


Use oov_token for low frequency tokens in Tokenizer

If both num_words and oov_token are specified in Tokenizer init, then low frequency tokens are going to be skipped. Isn't it better to replace them with oov_token instead?
https://github.com/keras-team/keras/blob/c476792e366626bec3b5a44b642c315239311516/keras/preprocessing/text.py#L275

Was thinking exactly the same. There should at least be an argument for enabling this in texts_to_sequences


How can I get modal open for search in this theme.

I have hosted my sphinx_rtd_theme on the Azure app service.
I am attaching the below picture for reference.

Hi @rahulsrivastava1
I think you are asking for support. But your question should also be directed more broadly at documentation hosting. sphinx-rtd-theme is a theme for the Sphinx documentation tool, not a piece of software that can manage login flows.
The modal search you are looking at is made for Read the Docs. The whole search backend is hosted on Read the Docs. So there is unfortunately not a quick way to port it to another platform.


Open Wayback fails to restart due to com.sleepycat.je.EnvironmentFailureException

After reboot Open Wayback crashes with the following error
org.apache.catalina.core.StandardContext filterStart
SEVERE: Exception starting filter [RequestFilter]
org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'localbdbresourceindex' defined in URL [file:/local/apache-tomcat-9.0.30/webapps/ROOT/WEB-INF/BDBCollection.xml]: Cannot create inner bean 'org.archive.wayback.resourceindex.bdb.BDBIndex#1b49a173' of type [org.archive.wayback.resourceindex.bdb.BDBIndex] while setting bean property 'source'; nested exception is org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'org.archive.wayback.resourceindex.bdb.BDBIndex#1b49a173' defined in URL [file:/local/apache-tomcat-9.0.30/webapps/ROOT/WEB-INF/BDBCollection.xml]: Invocation of init method failed; nested exception is com.sleepycat.je.EnvironmentFailureException: (JE 4.1.6) Environment must be closed, caused by: com.sleepycat.je.EnvironmentFailureException: Environment invalid because of previous exception: (JE 4.1.6) /xxxx/xxxx/index fetchTarget of 0x11e8/0x12424aa parent IN=22387642 IN class=com.sleepycat.je.tree.IN lastFullVersion=0x1221/0x1f4263f parent.getDirty()=false state=0 LOG_FILE_NOT_FOUND: Log file missing, log is likely invalid. Environment is invalid and must be closed.
After deleting the index, Open Wayback restarts without any problems.
Open Wayback version 2.4.0
Apache Tomcat 9.0.3
Java 11.0.6
Ubuntu
Could this be due to a bug in Berkeley DB JE 4.1.6? I've come across a similar error in the changelogs. Are there any plans of upgrading to a newer version of Berkeley DB JE?
Thank you in advance for any help you can provide.

Hi @schmika, unfortunately I can't provide much assistance with your issue, as I was unable to reproduce your error using the default OpenWayback configuration (except a different wayback.basedir that would persist after a reboot) with this environment:
OpenWayback 2.4.0
Apache Tomcat 9.0.33
Java 11.0.6
Ubuntu 18.04.4
Regarding upgrading Berkeley DB JE: at this point we are only providing minimal maintenance to OpenWayback (due to its age, its not keeping pace with website trends, and lack of developer resources) and encouraging people to use pywb. However we are still happy to consider pull requests submitted to upgrade/improve OpenWayback.

Hi @ldko, thank you for looking into it and getting back to me. I'll see whether I can figure out the issue and take a look at pywb.


skip rewrite-icons for certain conditions

The current behavior for icons is to rewrite any text that matches /:(#?[a-z_-]+[a-z\d]*):/gi with a span
Problem: this replaces strings that intentionally use colons inside them, like URNs
There are a couple cases I think this could be avoided without being too much of an impact:

inside code elements

I would argue it's unlikely authors intend to insert icons inside code blocks, at least when being used for code
this would match github/slack functionality, eg. :+1: :+1:

inside links/hrefs

some links contain URNs, eg. sharepoint
unlikely icons are intentionally inserted into href patterns, like maybe/(?!https?:\/\/[^\s]*):(#?[a-z_-]+[a-z\d]*):/gi

this would technically be a breaking change, but not sure how often each case might be relied on right now
wdyt @tripodsan @trieloff @dominique-pfister ?

I think this is ok. @davidnuescheler should decide :-)

What about not processing unless they are terminated by spaces?
/\s:(#?[a-z_-]+[a-z\d]*):\s/gi

@bstopp I think that is contrary to what people are used to in places like slack or:rocket:github, which is nice to stick close to for icons imo
for URNs I think case 2 could just be adjusted to include urn: as an ignored scheme along with https?:


Metrics: default to normalizing by max observation?

For normalized metrics (specifically NMAE, NMBE, NMRSE), does it make sense to support both a) user-supplied normalizing factors (current approach) and b) a default method for computing the normalizing factor (e.g. max observation value)? From a coding standpoint, this would change, e.g., normalized_mean_absolute(obs, fx, norm) to normalized_mean_absolute(obs, fx, norm=None) (i.e. change norm to a kwarg and add an if/else statement for switching between normalizing by a user-supplied value or a default).
If we agree to also supporting a default method for normalization, should we stick with using the max observation or something else (e.g. mean)? From my experience, normalizing by the max observation is more common, but I'm open to discussions.

The data model for PV plant specifications requires dc_capacity and ac_capacity. I'd use those numbers. Normalization for irradiance is most likely to be done with time dependent clear sky index, but we could consider a fixed number like 1000 W/m^2 too.
Using that kind of metadata would mean the metrics functions would remain the same but the calling function would need to get and provide data from elsewhere.
I'm hesitant to rely on the max observation value given the likely poor quality of some incoming data.

That makes. In that case, I agree with keeping the metric functions the same and then having the calling function pull the necessary data from elsewhere. Therefore, I'll go ahead and close this issue.


Footers and Sections and Sticky Footers as a feature?

This is a library I cannot live without but the above mentioned is a feature I sorely miss and have no idea how to implement with the existing sections and headers, it would be great if footers can behave just like headers but in the reverse direction, capability to add footers to sections, expandable footer(items being inserted upwards), sticky footers and I daresay capability to make a header or footer item behave sticky towards to the top and bottom.
Is there anything that I can do to guide myself to extend on this amazing library and add these kind of features?

@droidluv, as written in #717 it is enough interesting, but exceeds my effort for this project.
Moreover, the solution applied for sticky headers is just enough for simple use cases: a real layout manager is THE solution.


./install.sh doesn't work..

my problem is.. I deleted by mistake parts of the code... I tried to delete everything an tried to install it again.. but.. all I get is the following message:
./install.sh
Stopping moonraker-telegram-bot instance ...
Failed to stop moonraker-telegram-bot.service: Unit moonraker-telegram-bot.service not loaded.
I have no idea what to do..  I hope you can help me solving the problem because I love your bot
btw.. the Logfile has no entries..

so it looks like with the new release it will work 😊 fine as I have a second Printer I will see when the update is available. Thank you very much !!


dataset pagination only allows to show first 10K records

Describe the bug
I uploaded 21K records to a dataset and it only show 10K within the UI. Also, it doesn't allow me to paginate past the first 10K records without adding queries.
To Reproduce
Steps to reproduce the behavior:

Upload more than 10K records
Go to the dataset in the UI
Try to go past 10K record in pagination

Expected behavior
I would expect the record to be available or at least get a reference on why not all records are shown.
Screenshots

Environment (please complete the following information):

OS [e.g. iOS]: iOS
Browser [e.g. chrome, safari]: Firefeox
Argilla Version [e.g. 1.0.0]: 1.1.1
ElasticSearch Version [e.g. 7.10.2]:7.10.2
Docker Image (optional) [e.g. argilla:v1.0.0]: 1.1.1

Additional context
N.A.

@leiyre @Amelie-V  we need to take into account this use case after removing the Records count in the filters header.

Given that we can get the total number of records via Elastic, is seeing only 1000 pages a UI memory issue or something? If so, wouldn´t we be able to lazy-load only a part of the pages? The X surrounding pages for each one of the pages that we can directly click on?
In the example above, we can directly click 1,2,3 and 1000, with X=3 we could load 1,2,3,4,5,6, 997,998,999,1000.


[boost-regex] Build error

Host Environment

OS: linux
Compiler: gcc

To Reproduce
Steps to reproduce the behavior:
./vcpkg install boost
building in debug, dynamic mode
Failure logs
CMake Error at installed/x64-linux/share/boost-build/boost-modular-build.cmake:193 (message):
-Cut and paste the appropriate build messages from the console output.
-Please attach any additional failure logs mentioned in the console output.
-- Extracting source /home/allen/vcpkg/downloads/boostorg-regex-boost-1.78.0.tar.gz
-- Using source at /home/allen/vcpkg/buildtrees/boost-regex/src/ost-1.78.0-c212e9f671.clean
-- Including /home/allen/vcpkg/ports/boost-regex/b2-options.cmake
-- Found external ninja('1.10.2').
-- Getting CMake variables for x64-linux-debug-mixed-dbg
-- Configuring x64-linux-debug-mixed-dbg
-- Building x64-linux-debug-mixed-dbg
CMake Error at installed/x64-linux/share/boost-build/boost-modular-build.cmake:193 (message):
  No libraries were produced.  This indicates a failure while building the
  boost library.
Call Stack (most recent call first):
  ports/boost-regex/portfile.cmake:15 (boost_modular_build)
  scripts/ports.cmake:145 (include)

I think this was bad on my part. closing.


Create migration guide to 1.0

We shall provide a guide to point a path of migration of an existing project using a 0.x version fo flame to make it use the upcoming 1.x versions.
This shall make the way to this milestone alongside the preview(RC) versions.

Are we still planning on doing this, or is it too difficult at this point?

Are we still planning on doing this, or is it too difficult at this point?

I'm voting for too difficult, and not worth it since there are so few people that use 0.26.x now.

 If done, a migration guide should cover the migration from some RCs that is not doable. Shall we cancel this?


daemon: Turn on policy debug logging if Cilium is started with --debug

The purpose of the separate --debug-verbose=policy setting is to be
able to turn on policy map debug logging without turning debug on in
general. In the reverse case, when debug is enabled, we generally also
want to get the policy map debugs, though.
Signed-off-by: Jarno Rajahalme<EMAIL_ADDRESS>

test-me-please

Known flake (#12511) on netnext, no other CI issues.


Stop storing defaults

Leverage https://github.com/datavis-tech/reactive-property/issues/4 to eliminate publicPropertyDefaults internally.

Part of reactive-model 1.0 https://github.com/datavis-tech/reactive-model/issues/31


Add search result list widget

I want to respond to Interaction calls using a widet, which displays the first 7 search results and enables the user to choose one.

Closed in favor of #6


Get-SSRSObject cmdlet

Someone on the Slack community channel saw this script I shared and asked if I could create a feature request and share the script in case you'd be able to use it for a future cmdlet.
The idea of a Get-SSRSObject cmdlet seems like it could be really nice. With parameters to choose the object type, and name glob filters, etc.
Summary of new feature

A cmdlet that has the ability to pull all of or a specific SSRS object directly from the ReportServer database.

Proposed technical details (if applicable)
I recently needed to figure this out using powershell, so this is the script I came up with. It is currently only configured to export report RDL files, but it could be adapted to other objects as well.
$query = "SELECT c.[Path], c.Content FROM dbo.[Catalog] c WHERE c.[Type] = 2"
$data = Invoke-DbaQuery -SqlInstance SERVERNAME -Database ReportServer -Query $query
$data | % {
    $reportName = ".$($_.Path).rdl";
    $reportDir = $reportName | Split-Path;
    $null = New-Item -Path $reportDir -ItemType Directory -Force;
    [System.Text.Encoding]::UTF8.GetString($_.Content) | Set-Content -Path $reportName -NoNewline
}

Latest version of dbatools as of writing

1.0.5

We generally point folks to the module written for SSRS and Power BI that the Microsoft team for that product maintains:
https://github.com/microsoft/ReportingServicesTools
Is this covering something that is not available on that module?

Yes, the reportingservicestools module covers downloading items from a report server.
For a particular item: Out-RsCatalogItem or Out-RsRestCatalogItem
For all items in a folder: Out-RsFolderContent or Out-RsRestFolderContent
The two REST commands will also download items that are not reports, datasources or datasets (eg PowerBI reports or Excel files)
The module is very actively maintained so definitely post there if you see something that is missing.

I do not see enough need to try and cover functionality in dbatools that the SSRS module should be (or already is) more than capable of handling.
Closing.


[OAP-1543][oap-data-source][arrow] File descriptor leak in v1

#1543

Hi @xwu99 Travis CI on mlib seems failing on master, could you please take a look?

Hi @xwu99 Travis CI on mlib seems failing on master, could you please take a look?

The reason it failed is oneAPI released a new beta version and changed the default path, I will submit PR to fix this.

Let's not to treat this CI failure as blocker as relevant jobs were passed. Merging

@zhztheplayer #1551 temp removed mllib CI.


dApp Tweaks

Change Copy of Seller Center to Seller Hub

Move the Redemption Point above 'Additional information'
Change Copy:
OLD: The website from which buyers can redeem the rNFT. By default the redemption point will be the Boson dApp.
NEW: The default redemption point is the Boson app. You can add a specific website from which buyers can redeem the rNFT, for example, your customised Web3 storefront.

In the footer, change the second 'Sell' that goes to the Seller landing page to read 'Templates & Guides'

fix image that appears in the banner as it is not what you see in the image previewer

banner in storefront should fill up horizontal space (change to object-fit: contain?)

done


Building v0.2.20 fails on non-glibc systems

Building OpenBLAS 0.2.20 fails on non-glibc systems, due to unsafe use of the __GLIBC_PREREQ preprocessor macro. Will there be a patch release soon or should downstream maintainers patch this?

It is #1247 that reverts offending change

Master branch has the problematic change already reverted in anticipation of a quick interim release,
current develop branch has a fixed version of the intended functionality.

I think it is more to @xianyi if he intends 0.2.21 tomorrow....

Inadvertently closed with my PR for develop - reopening until 0.2.21 is out.

Guess we can skip 0.2.21 now as it would no longer be a "quick fix" after six months have passed, but hopefully 0.3.0 can be released "very soon now".


Use goimports instead of gofmt

This updates script/fmt to use goimports instead of the built-in gofmt.  The main benefit here is that it organizes the import statements with stdlib packages above the 3rd party libs.  The main change is in script/fmt itself:
+hash goimports 2>/dev/null || {
+  echo "Installing goimports"
+  echo "go get golang.org/x/tools/cmd/goimports"
+  go get golang.org/x/tools/cmd/goimports
+}
+
 # don't run gofmt in these directories
 ignored=(/bin/ /docs/ /log/ /man/ /tmp/ .vendor)
 for i in */ ; do
   if [[ ! ${ignored[*]} =~ "/$i" ]]; then
-    gofmt -w -l "$@" "${i%?}"
+    goimports -w -l "$@" "${i%?}"
   fi
 done

Is this valid, or should this fall back to gofmt?  gofmt respects the import style that goimports forces, so we won't have to worry about competing code formatting tools.

:heart:
:+1: for falling back


minor Navbar enhancements

I have just added title (appearance of the description on hover) to the Navbar items, so it is easy to recognize what it is for. Also Opened the Docs in a new window so that the user doesn't lose control from the application.

Awesome! thanks Arshul.


Nonworking languages

Please Describe the issue or question and share your OS and Python version.

OS: macOS
OS Version: 10.11.6
Python Version: 3.6.1
The language command line flag is not working for Pashto. The input python3 nettacker.py -L ps results in the following error:
Traceback (most recent call last):
  File "nettacker.py", line 13, in <module>
    load()  # load and parse the ARGV
  File "/Applications/OWASP-Nettacker-master/core/parse.py", line 34, in load
    module_names, graph_names)
  File "/Applications/OWASP-Nettacker-master/core/args_loader.py", line 117, in load_all_args
    help=messages(language, "select_profile").format(_all_profiles))
ValueError: Single '}' encountered in format string

Hi @cwcromwell thanks for reporting,
@pradeepjairamani please take a look and send the patch. let me know if you have any questions.
Regards.

@Ali-Razmjoo I will have a look into it

Hi, should I close this one?


Incorrect behavior of OdfElement.getTextContent()

While investigating #220, I came across OdfTableCell.getOdfElement().getTextContent(), which is defined by org.w3c.dom.Node.getTextContent() and implemented in OdfElement. My interpretation of the Node.getTextContent() documentation is that it should recursively retrieve the text contents of all child nodes and concatenate them together. However, the OdfElement implementation only looks at the children, rather than all descendants.
I think it is also worth noting that OdfWhitespaceParser.getText(Node) appears to do what OdfElement.getTextContent() is supposed to do. Or at least something very similar.

i don't know the ODFDOM code well, but as a general rule about ODF: inside of a paragraph (text:p/text:h) and its child elements, there is character content which is part of the paragraph text (most elements), and there is character content which is part of some nested object and not part of the paragraph text.
there are a bunch of elements for drawing shapes, draw:frame etc. and also office:annotation, that should comprise most of the latter case - these should probably not be included in any API that promises some sort of text content of "the paragraph" or "the table cell".
the criteria for what is considered paragraph text are in section "6.1.2 White Space Characters" of ODF 1.3 part 3, and i think the text of a table cell should simply be the concatenation of its paragraphs.

You are absolutely right, Todd.
OdfElement has the base functionality to concatenate the text content:
https://github.com/tdf/odftoolkit/blob/master/odfdom/src/main/java/org/odftoolkit/odfdom/pkg/OdfElement.java#L2633
but the every text node containing element like OdfTextSpan should override this method and define its specific behavior.
By this method implementation, the specific behavior - Michael has pointed out it his previous comment - would be reflected.
I am uncertain atm, if we have a way in the ODF Specification to differentiate this behavior, if not it would be worth to add it.
Taking this over issue over!
Will aim to generate this as part of the upcoming release candidate.
Thanks for pointing out!

The ODF specification explains in the bullet list of 6.1.1 all elements containing text that should not be considered, see
https://docs.oasis-open.org/office/OpenDocument/v1.3/os/part3-schema/OpenDocument-v1.3-os-part3-schema.html#a_6_1_Basic_Text_Content
The funcationality is currently spread and needs to harmonized, e.g. this whitespace collapsing function is implemented in the characters(..) method of ChangesFileSaxHandler
https://github.com/tdf/odftoolkit/blob/master/odfdom/src/main/java/org/odftoolkit/odfdom/changes/ChangesFileSaxHandler.java#L2252
#222 and this #229 are quite related and will likely be fixed in one patch.


Allow evaluating fitness not in parallel

This PR should allow evaluating fitness, not in parallel:
...
population.evolve(&mut environment, &p, /* evaluate in parallel */ true);
...

Thanks!
I don't have travis running in this fork; if you run cargo test you will see that there are some compilation errors in tests.

I already tried to run examples, it works fine. I think it will be good to create two separate envrinments for evaluating. Current Envrionment requires Sync implementation and this doesn't allow using non-sync fields in implementation. Example:
struct MyEnv {
    window: piston_window::Window // does not implements Sync and Send
}

impl Environment for MyEnv {...} // error

Hm, I'm not sure how we can achieve that. Because whether stuff is Sync is evaluated at compile time while we select at runtime whether or not parallelism.
I think we should indeed make it possible to have shared state in the env such as a window or accumulation of data or whatever. What about wrapping the window in a mutex or rwlock?
Btw, does cargo test work for you? You should make sure that it compiles and passes.

@Ploppz So I fixed tests, everything works now.
➜  rustneat git:(master) ✗ cargo test
   Compiling rustneat v0.3.0 (/home/aprokurov/Projects/Rust/rustneat)
    Finished dev [unoptimized + debuginfo] target(s) in 2.11s
     Running target/debug/deps/rustneat-ed65baaa88918f43

running 20 tests
test nn::ctrnn::tests::neural_network_activation_stability ... ok
test nn::tests::genomes_with_same_genes_with_little_differences_on_weight_should_be_in_same_specie ... ok
test nn::tests::mutation_add_connection ... ok
test nn::tests::mutate_add_neuron_should_not_change_output ... ok
test nn::tests::mutation_add_neuron ... ok
test nn::tests::should_allow_multiple_output ... ok
test nn::tests::genomes_with_big_weight_difference_should_be_in_other_specie ... ok
test nn::tests::mutation_connection_weight ... ok
test nn::tests::should_not_raise_exception_if_less_neurons_than_required ... ok
test nn::tests::activate_organims_sensor_without_enough_neurons_should_ignore_it ... ok
test nn::tests::should_work_with_cyclic_networks ... ok
test nn::tests::should_propagate_signal_over_hidden_layers ... ok
test nn::tests::try_to_inject_a_unconnected_neuron_gene_should_panic ... ok
test nn::tests::two_genomes_with_big_difference_should_be_in_different_species ... ok
test nn::tests::two_genomes_with_little_differences_should_be_in_same_specie ... ok
test nn::tests::should_be_able_to_get_correct_matrix_representation_of_connections ... ok
test specie::tests::specie_should_return_correct_average_fitness ... ok
test nn::tests::should_propagate_signal_without_hidden_layers ... ok
test population::tests::population_should_be_able_to_speciate_genomes ... ok
test population::tests::after_population_evolve_population_should_be_the_same ... ok

test result: ok. 20 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out

     Running target/debug/deps/hyper_opt-813d1064b1db0adb

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out

     Running target/debug/deps/xor_performance-956904be4fc2071f

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out

     Running target/debug/deps/rustneat_tests-4bd81080989dc05f

running 5 tests
test test::population_can_evolve ... ok
test test::population_can_be_tested_on_environment ... ok
test test::can_generate_a_population ... ok
git add srtest test::can_solve_xor ... ok
test test::xor_can_only_improve ... ok

test result: ok. 5 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out

   Doc-tests rustneat

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out

To allow non-sync environments it's possible to define two different environments:
trait Environment {...}
trait ParEnvironment: Sync {...}

Perhaps yeah. I would call it SeqEnvironment and Environment, the latter being the parallel one because I think that should be default. Then I suppose we will need
pub fn evolve(&mut self, env: &mut Environment<G>, p: &NeatParams)

and
pub fn evolve_seq(&mut self, env: &mut SeqEnvironment<G>, p: &NeatParams)

What do you think?
Should I wait to merge or merge now?

I think it's possible to do something like this:
enum EnvKind<'a> {
   Seq(&'a mut SeqEnvironment),
   Env(&'a mut Environment)
}

impl Into<EnvKind> for &'_ mut SeqEnvironment {...}
impl Into<EnvKind> for &'a mut Environment {...}

pub fn evolve(&mut self,env: impl Into<EnvKind<'_>>,p: &NeatParams)

And should I wait to merge or merge now?
I think it will be better to merge this pr,but that's your choice :)


update palllet weights hbs template

There are hbs template in maintain subdirectory with outdated header information (years, license?! etc.. )

The year is updated in .38 upgrade.


Make reference to spreading in arguments of grid

This PR makes a reference on "spreading" in the description of the childen argument of a grid. Note that in the original PR, I inserted a direct/manual/hard link to https://typst.app/docs/reference/foundations/arguments/#spreading. I am not sure about how to use internal links (like [grid documentation]($grid) from https://github.com/typst/typst/blob/main/crates/typst-library/src/layout/table.rs), but that should be done before this is merged.
This PR spawned from this issue

The fact that you can spread here is implied by the fact that you can spread in any function call. Instead, I believe this would be better suited as a usage example (as it's a common way to use grid) rather than a simple statement.

I agree, it would be better in an example. As for the current grid-example in the grid documentation, I find it to be unnecessarily complicated due to the local definition of cell. This is because it could make it appear to less experienced users as if one has to define this cell-function to use grid.
So perhaps it would make sense to change the example completely. I suggest creating the following two grids in the example:
#grid(
  columns: (40mm, 100pt, 1fr),
  gutter: 3mm,
  [First content],
  [Second content],
  [Third content],
  [Fourth content],
  [Fifth content],
  [Sixth content],
)

#let my_array_int = range(6)
#let my_array_str = my_array.map(str)
#grid(columns: 2,
	..my_array_str
	)

That cell function is in there because the output looks very bland otherwise. I understand how it can be confusing though.

That cell function is in there because the output looks very bland otherwise. I understand how it can be confusing though.

Right, and I absolutely see its value. Perhaps it would also be sufficient to add some text along the lines of
"""
To show the size of the grid-cells, we define a helper-function cell for this example. For typical use, replace cell(height: 100%)[My content] with [My content].
"""
Should I add that in this PR?

I'm not sure people will read it. I'll try to come up with a better example.

That sounds great. Please consider adding two examples, one with spreading of an array of content. I have to say I find the examples I created above simple and illustrative, so please copy from them whatever you find useful.

How about this?
#set text(10pt, style: "italic")
#set rect(
  inset: 8pt,
  fill: rgb("e4e5ea"),
  width: 100%,
)

#grid(
  columns: (60pt, 1fr, 2fr),
  rows: (auto, 60pt),
  gutter: 3pt,
  rect[Fixed width, auto height],
  rect[1/3 of the remains],
  rect[2/3 of the remains],
  rect(height: 100%)[Fixed height],
  image("tiger.jpg", height: 100%),
  image("tiger.jpg", height: 100%),
)

And for spreading maybe an additional example, something like your proposal. I would inline the range and map and add some gutter, then it'll look better:
#grid(
  columns: 5,
  gutter: 5pt,
  ..range(25).map(str)
)

That loos good to me. Is it possible to add a comment line in the setup?
#set text(10pt, style: "italic")
// We use `rect` to visualize the cells in the grid
#set rect(
  inset: 8pt,
  fill: rgb("e4e5ea"),
  width: 100%,
)

So removing the #set text, and adding a code-comment about why we do #set rect, sounds very nice.

Yeah. Do you want to make these changes in this PR?

I will give it a shot. Let me know if I mess something up - I am not sure how to make 2 examples, and I hope that the output is automatically placed next to the code.

I have no idea how to look into that. Help would be appreciated.


node: Distinct start node log to distinguish node type info from rest of logs

Looks like this now:
2022-04-27T15:34:21.209+0100	INFO	node	node/node.go:104

/_____/  /_____/  /_____/  /_____/  /_____/

Started celestia DA node
node type: 	light
network: 	devnet-2

/_____/  /_____/  /_____/  /_____/  /_____/

The p2p host is listening on:
*  /ip4/<IP_ADDRESS>/tcp/2121/p2p/12D3KooWQAFwc4p5hQUpzU6UgLDEcMD5JVGpMrqWQ8SfUtrZ1G4v
*  /ip6/::1/tcp/2121/p2p/12D3KooWQAFwc4p5hQUpzU6UgLDEcMD5JVGpMrqWQ8SfUtrZ1G4v

Codecov Report

Merging #659 (981943d) into main (804c1d9) will increase coverage by 0.28%.
The diff coverage is 94.20%.

@@            Coverage Diff             @@
##             main     #659      +/-   ##
==========================================
+ Coverage   61.57%   61.85%   +0.28%
==========================================
  Files          95       97       +2
  Lines        5020     5055      +35
==========================================
+ Hits         3091     3127      +36
  Misses       1603     1603
+ Partials      326      325       -1

Impacted Files
Coverage Δ

ipld/retriever.go
89.55% <89.55%> (ø)

service/share/full_availability.go
93.75% <91.66%> (-6.25%)
:arrow_down:

ipld/read.go
77.94% <100.00%> (-7.25%)
:arrow_down:

ipld/retriever_quadrant.go
100.00% <100.00%> (ø)

node/node.go
62.50% <100.00%> (+1.63%)
:arrow_up:

service/share/light_availability.go
91.89% <100.00%> (+0.22%)
:arrow_up:

service/share/share.go
80.26% <100.00%> (+0.26%)
:arrow_up:

Continue to review full report at Codecov.

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


Consider integrating with fastdom somehow

The fastdom project does a very similar read/write distinction, maybe there's an opportunity to have a fastdom debug mode, where nope.js is used to throw when dumb things are happening.
@wilsonpage, you might find this interesting.

nope-js looks like an interesting project, how is it supposed to be used?

The main premise is that this library emphasizes bad ideas by throwing exceptions. It's sort of like a debug mode for your web app.
Currently, I have two ideas for deployment:

As an extension (on/off button), dynamically injected into the page -- to help with small-scale deployments or checking existing sites for badness.

As a development-time check that you only don't use in production.

Who is responsible for setting the lifeCycleMode?

You still have to do that yourself, sadly. In your case, you'll flip it just before running the batch of "reads"

Ah I get ya! So using nope-js would throw if user tried to do a 'read' in a 'write' callback?

Yes. For example, currently, if you access element.offsetLeft in "write", nope will throw.

I am currently compiling a list of layout triggers (DOM props/methods that will cause synchronous layout/reflow) for Blink here: https://github.com/dglazkov/nope-js/blob/master/nope.js#L12


Added AutoGluon Logo to README.md

Issue #, if available:
Description of changes:
Added AutoGluon Logo to README.md
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Job PR-369-1 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-369/1/index.html

Should the logo be smaller? It looks a bit large to me at the moment, I'd prefer if its width were more similar to the width of the text below: "AutoML Toolkit for Deep Learning"

I updated to make it smaller, how does it look now?

Job PR-369-2 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-369/2/index.html

Job PR-369-3 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-369/3/index.html


WIP: feat(invoice): add basis

Hi everyone!
I want to introduce a new payment feature that implements this api: https://core.telegram.org/bots/api#payments
Recently, in my project, it became necessary to implement payments through a bot.
I'm using your library with great pleasure, so I decided to create my first PR and start development from this endpoint
This part of the code implements a minimum of functionality of sendInvoice endpoint, so i need some feedback from you about code style and classes usability:

Using of Arrfacade
Classes naming
Dto structure (about missing __constructdeclaration)
Using Dto as parameter in TelegramChat model
etc.

Basically, I'll be glad to receive any comments from you about code-style that you use for

Hi, thanks for your contribution, will check ASAP

Hi @pepperfm we have a payment feature under testing, so I will take your PR into account in order to improve our code, thanks! we'll be out with it in short time, thanks!

Hi @fabio-ivona,
I wanted to follow up on the status of the payment implementation
Has there been any progress?
When will it be released?
I need this and if you want I can help in implementing it

@ah-rahimi I have a working beta, are you available for testing it?

@fabio-ivona No problem. I can test it in the coming days.

in this case, you can use this to test it: composer require defstudio/telegraph:dev-payments
let me know if you feel something to report


amplify push does not work when existing GraphQL API is added

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
20.9.0
Amplify CLI Version
12.10.0
What operating system are you using?
Mac
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
I have added an existing GraphQL API with amplify add codegen --apiId xxxxxxxx. No other "manual" changes were made.
Describe the bug
I want to use Amplify in my JS web app to utilize authentication with Cognito and query data from DynamoDB tables via a GraphQL API (AppSync).
amplify status returns the following:

As you can see, I have already added the authentication with amplify add auth.
I have an existing GraphQL API in AppSync and I want to add it to my Amplify configuration. I use amplify add codegen --apiId xxxxxxxx to generate the necessary code and the API gets added to my local backend as follows.

So far so good... but when I run amplify push to also transfer this configuration to my remote backend in Amplify, basically nothing happens. Except that the operation changes from No Change to Create:

Then I tried to force push this with amplify push --force to see what happens, but it didn't go any further than this:

After 30 minutes I stopped this with CMD + C, but I found my GraphQL API in Amplify Studio:

Even though the GraphQL API seems deployed I cannot access it within my backend.
Expected behavior
Whenever I add a GraphQL API to my project with amplify codegen add ... it also shows up when I run amplify status. So when I push these changes I expect it to be correctly pushed to the amplify backend.
Reproduction steps

Add Authentication - amplify add auth
Add existing GraphQL API - amplify add codegen --apiId xxxxxxxx
Push local backend - amplify push

Project Identifier
Project Identifier: 79f15d992b5c3e5a87865affa94191ba
Log output

# Put your logs below this line

Additional information
No response
Before submitting, please confirm:

[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

Hey @lukasulbing, Thanks for raising this. I'm unable to reproduce the problem; it appears to be related to a potential network issue. Could you confirm if your app and API are in the same region?

Hey @AnilMaktala, thanks for your reply. Yes, my Amplify project as well as my AppSync API are in the same region (eu-west-1). The following is my amplifyconfiguration.json:
{
  "aws_project_region": "eu-west-1",
  "aws_cognito_region": "eu-west-1",
  "aws_user_pools_id": "eu-west-1_xxxxxx",
  "aws_user_pools_web_client_id": "xxxxxxxxx",
  "oauth": {
    "domain": "xxxxxxx.auth.eu-west-1.amazoncognito.com"
  },
  "aws_cognito_username_attributes": [
    "EMAIL"
  ],
  "aws_cognito_social_providers": [],
  "aws_cognito_signup_attributes": [
    "EMAIL"
  ],
  "aws_cognito_mfa_configuration": "ON",
  "aws_cognito_mfa_types": [
    "TOTP"
  ],
  "aws_cognito_password_protection_settings": {
    "passwordPolicyMinLength": 8,
    "passwordPolicyCharacters": [
      "REQUIRES_LOWERCASE",
      "REQUIRES_UPPERCASE",
      "REQUIRES_NUMBERS",
      "REQUIRES_SYMBOLS"
    ]
  },
  "aws_cognito_verification_mechanisms": [
    "EMAIL"
  ],
  "aws_appsync_graphqlEndpoint": "https:/xxxxxxxxx.appsync-api.eu-west-1.amazonaws.com/graphql",
  "aws_appsync_region": "eu-west-1",
  "aws_appsync_authenticationType": "API_KEY",
  "aws_appsync_apiKey": "da2-xxxxxxxx"
}

Whenever I try to push this configuration it says No AppSync API configured. Please add an API at the end like in the screenshot above.

By the way, I've found a similar issue here, but it has been closed without providing a solution:
https://github.com/aws-amplify/amplify-cli/issues/8394

I also was not able to reproduce the issue. Did you try removing the API like suggested in issue you linked?

Yes, I removed the API with amplify api remove and linked it again with amplify add codegen --apiId xxxxxx but this didn't help.
One of the staff members over on the Amplify discord server suggested to add the output of amplify push --debug to this issue.

I see some error messages on here, could this lead to a problem?

Also, I am willing to set up a completely fresh amplify app to see if it's related to my local machine. In that case, it would be kind of you to tell me the commands you when you tried to reproduce the issue.

Hey @lukasulbing, Thank you for providing the additional information. I've performed the following steps to replicate the problem:

amplify init
amplify add auth
amplify add codegen --apiId xxxxxxxx
amplify push

Please let me know if I am missing anything in my approach here.

Alright, I have created a new Amplify app and js project to test this. I've even left out adding the authentication.
I've run the same commands you provided (except add auth) and the only thing I did additionally was to enable Amplify Studio and pull this backend into the project.
Then I added the GraphQL API from AppSync and pushed everything. The result was the same, nothing happened in Amplify Studio under the GraphQL tab...
This basically summarizes the steps I've taken to reproduce the issue:

create new Next.js project
install dependencies
amplify init
enable Amplify Studio
amplify pull --appId xxxxxx --envName dev
amplify add codegen --apiId xxxxxxxx
amplify push

I can't tell if it's related to my machine unfortunately. Do you have any more ideas what I could do to find the root cause?

Hi, I've also tested this on a computer of my colleague (also Mac). I followed the exact same steps as mentioned above and it resulted in the same problem. After amplify push it says No AppSync API configured. Please add an API and no API is showing in Amplify Studio.

@AnilMaktala @dpilch I think i have a repro for this issue, was testing multiple things.

on the AWS AppSync console, created a new API in eu-west-1.
New API -> API type(GraphQL APIs, Design from scratch)
Specify API details -> added some name(testApi)
Specify GraphQL resources -> Create type backed by a DynamoDB table now.
Model name(test) -> created some fields -> Resolver runtime(default: AppSync JavaScript (APPSYNC_JS)) -> Configure model table(Todo) -> create API
amplify init (didnt create a frontend app in my test and forget my profile was for us-east-1)

✖ Getting API details
AppSync API was not found in region us-east-1
? Do you want to choose a different region Yes
? Choose AWS Region EU (Ireland)
⠏ Getting API detailsSuccessfully added API tesimp to your Amplify project
✔ Getting API details
? Choose the code generation language target javascript
? Enter the file name pattern of graphql queries, mutations and subscriptions src/graphql/**/*.js
? Do you want to generate/update all possible GraphQL operations - queries, mutations and subscriptions Yes
? Enter maximum statement depth [increase from default if your schema is deeply nested] 2
✔ Downloaded the schema
✔ Generated GraphQL operations successfully and saved at src/graphql

amplify add codegen --apiId <apiId>
amplify push
amplify status shows the API is in create

    Current Environment: dev

┌──────────┬───────────────┬───────────┬─────────────────┐
│ Category │ Resource name │ Operation │ Provider plugin │
├──────────┼───────────────┼───────────┼─────────────────┤
│ Api      │ tesimp        │ Create    │                 │
└──────────┴───────────────┴───────────┴─────────────────┘

@lukasulbing is this config of the AppSync API similar to the reproduction steps above?

@ykethan Yes, this what I've done. There are some in GraphQL API linking, but I don't think they can have an impact.
I'll let you know about them just in case:
The code generation language target in my project is typescript and I've changed the file name pattern to app/graphql/**/*ts. Also, I also see the option to change the location of API.ts which I've updated to app/API.ts.
Other than that, this is exactly what I'm seeing and when I try to push it, nothing changes in Amplify Studio under the GraphQL API tab.

Hey @lukasulbing, Can you verify the region from which you're attempting to import? Additionally, consider adding the -region in the import command as suggested here to check if it resolves your issue..

Hey @AnilMaktala, I have been using the command from the Webinterface of AppSync npx @aws-amplify/cli codegen add --apiId xxxxxxxxxx --region eu-west-1 to import the GraphQL API.


Propose Block for London Mainnet Activation

This pull request proposes block 12965000 for the mainnet activation of the London network upgrade. The ETA for this block should be between 13:00 UTC and 17:00 UTC on August 4, 2021.
We had agreed on ACD116 to decide the block async and try and get client releases done for ACD117 (July 9). If we can roughly hit that timeline, then August 4th should provide enough time to advertise the client releases that are mainnet compatible, and allow infrastructure providers & node operators to upgrade.

Yes 🔥❤

Where we go!

yes

❤🔥

🚀🚀🚀

lets go

Cautious optimism.

Yeeey

LET'S GO 🚀🚀🚀

Fortuna Favet Fortibus

:clap: :clap: :clap:

<3

Takeoff imminent 🚀🚀🚀

Yeet!

🚀🚀🚀

🎉

🎊   🚀  🎉   🚀

Good news!

🚀🚀🚀🚀🚀🚀

Can't wait!

This is the way

Follow the Vitalik Buterin's lead. Waifu soon.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@05d6bf4). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #223   +/-   ##
=========================================
  Coverage          ?   94.07%
=========================================
  Files             ?       13
  Lines             ?      827
  Branches          ?        0
=========================================
  Hits              ?      778
  Misses            ?       49
  Partials          ?        0

Flag
Coverage Δ

unittests
94.07% <0.00%> (?)

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

Continue to review full report at Codecov.

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

 ETH is turning to "rich getting richer" scam thanks to greedy Vitalik.

nice,looking forward to the day

Mining pools will reject this crap.
https://stopeip1559.org

good boy


Disable console output

Hi there, is is possible to disable the console output when running the detection?
...
Optimizing fused batch norm node name: "FeatureExtractor/MobilenetV1/Conv2d_13_pointwise_2_Conv2d_5_3x3_s2_128/BatchNorm/FusedBatchNorm"
op: "FusedBatchNorm"
input: "FeatureExtractor/MobilenetV1/Conv2d_13_pointwise_2_Conv2d_5_3x3_s2_128/convolution"
input: "FeatureExtractor/MobilenetV1/Conv2d_13_pointwise_2_Conv2d_5_3x3_s2_128/BatchNorm/gamma"
input: "FeatureExtractor/MobilenetV1/Conv2d_13_pointwise_2_Conv2d_5_3x3_s2_128/BatchNorm/beta"
input: "FeatureExtractor/MobilenetV1/Conv2d_13_pointwise_2_Conv2d_5_3x3_s2_128/BatchNorm/moving_mean"
input: "FeatureExtractor/MobilenetV1/Conv2d_13_pointwise_2_Conv2d_5_3x3_s2_128/BatchNorm/moving_variance"
device: "/job:localhost/replica:0/task:0/device:CPU:0"
attr {
  key: "T"
  value {
    type: DT_FLOAT
  }
}
attr {
  key: "data_format"
  value {
    s: "NHWC"
  }
}
attr {
  key: "epsilon"
  value {
    f: 0.001
  }
}
attr {
  key: "is_training"
  value {
    b: false
  }
}
...

I was able to disable tensorflow logging messages, however I'm unable to disable the above output.
It doesn't work even if I disable entire stdout and stderr.
Any suggestions?

I think this is a problem of tensorflow version, if the log is really bothering you, try to use a old version of tensorflow.