Are you planning to add TypeCompiler.Compile just like typebox, to achieve high performance?

Example: https://github.com/sinclairzx81/typebox?tab=readme-ov-file#typecompiler

I have similar ideas with a build time compiler in mind, but at the moment it is not feasible in terms of time. I can see that TypeBox and others are much faster, but Valibot is still not slow and probably not the bottleneck if your application has performance problems. It is also important to mention that the speed of most of these high performance schema libraries is not "free". There are drawbacks. For example, many of them only check the properties of an object you define and return the original object. This could be a security problem if you put this object in your database, as it could contain malicious data. When I have more time, I will probably write a blog post about the performance of schema libraries.

There are drawbacks. For example, many of them only check the properties of an object you define and return the original object. This could be a security problem if you put this object in your database, as it could contain malicious data. When I have more time, I will probably write a blog post about the performance of schema libraries.

FYI, there isn't a need to modify an object if the schematic outright prohibits additional data (additionalProperties: false). This is actually better for servers / databases as they can flat out reject malformed data (meaning a sender is mandated by the schematic to send correct data). In this regard, no provisions are made to transform a senders payload to match a target schematic. This is the principle applied in high performance validators.
Extra value processing (on top of Check), be it culling additional properties, value coercion, transforms or other kinds of processing is where bottlenecks are going to occur. TypeBox supports uncompiled versions of these, but the jury is still out on the most efficient implementation for each kind of operation. This said, Ajv still stands as one of the best integrated implementations for extra value processing. If you're looking to publish blog material on the topic of high performance validation + additional operations, Ajv would be a good thing to research.
Anyway, found this issue while browsing for TypeBox issues, and thought I'd leave a comment. Food for thought.
Cheers
S

Hey Haydn, I appreciate you taking the time to leave a comment.

FYI, there isn't a need to modify an object if the schematic outright prohibits additional data (additionalProperties: false)

I agree. Valibot currently follows a different philosophy and parses complex data structures that could include transformations (creating a copy) to not unintentional manipulate the input data. However, changing this behaviour and only "parsing" objects if necessary is not an option for Valibot in terms of bundle size as it would require additional code. In the long run, if we offer a compiler, we can change this depending on the schema provided and priorities for performance.

This is actually better for servers / databases as they can flat out reject malformed data (meaning a sender is mandated by the schematic to send correct data).

I didn't fully understand the advantage. Can you explain this in more detail?

Extra value processing (on top of Check), be it culling additional properties, value coercion, transforms or other kinds of processing is where bottlenecks are going to occur.

That's true. I see a lot of benefit in giving our users such powerful tools like transformations. That's why I prioritized functionality over maximum performance here.

If you're looking to publish blog material on the topic of high performance validation + additional operations, Ajv would be a good thing to research.

Thank you for the tip! I will keep that in mind.

Hi @fabian-hiller
Thanks for the reply (I had been searching for reported TypeBox issues following a recent update, and happened across this issue, thought I'd leave a comment :)). I can try answer the above question, but usually prefer to go into detail on such things as it helps to explain why an approach might be preferred within a broader context, ill try my best below.
Validate, Don't Parse

This is actually better for servers / databases as they can flat out reject malformed data (meaning a sender is mandated by the schematic to send correct data).

I didn't fully understand the advantage. Can you explain this in more detail?

So, what I mean here is rather than having the validator try to accommodate for data that is mismatched to the target schematic (i.e. Transform, Truncate, Coerce, Initialize (defaults), etc), the validators role is reduced to simply checking if the data is mismatched. This is going to be faster (as noted above), but it also pushes back to a Sender to adhere to the contract (Type) expected by a Receiver.
Here is a more concrete example.
class Publisher<Type extends TSchema> {
   constructor(private readonly type: Type) { ... }

   public Publish(value: Static<Type>) {
      // There is no potential for a value to be transformed before
      // Send. The caller of Publish must pass a correct value. The
      // call to Check (non-mutable) is an assurance to the Publisher
      // that the value is correct with no additional processing.

      if(Value.Check(this.type, value)) return Send(value)

      // Fail
      throw Error('invalid value')
   }
}

The responsibility of transforming the value now rests with the Sender
const publisher = new Publisher(Type.String())

publisher.Publish('hello')                // ok

publisher.Publish(12345)                  // error

publisher.Publish((12345).toString())     // explicit coercion | transformation

The principle here is that by having a validator only validate (not parse), it forces a clear distinction between the roles of the Sender, Receiver and Validator. The implementation effectively removes all guess work as to whether the value will be transformed in-fight by the validator (a good thing if the Receiver happens to be a Database) ... and the Receiver is always going to be fast (another good thing)
Of course, a possible downside one might consider with this approach is that Senders are now required to perform explicit transformations "before" sending values (i.e. (12345).toString() as above). However a Sender is still free to implement transforms in other ways unknown to the Receiver (where we move the performance cost of transformation to the Sender side)
// publisher.Publish( (12345).toString() )

publisher.Publish( Value.Convert(Type.String(), 12345) ) // Make the Sender Pay the Cost

This is the principle taken by TypeBox where any operation that may implicate performance is made opt in and explicit. By breaking down operations to their constituents (rather than embedding operations like transform within a validator) it becomes very clear what component of a system is performing what operation, and generally results in less guess work overall.
It should always be clear where transformations are taking place (imo)

So in saying all this, TypeBox does actually have a Parse function ...
const T = Value.Parse(Type.String(), 'hello')

.. And bi-directional codecs.
const T = Type.Transform(Type.String({ format: 'date-time' }))
   .Decode(value => new Date(value)) // Safe because validation checks Iso format
   .Encode(value => value.toISOString()) // Safe because typescript asserts as Date

It's just that in most implementations I work on, I've tended to avoid using these functions due to the overhead and complexity they can introduce (although Transform/Decode does see some usage in Server work). TypeBox got these functions (albeit somewhat reluctantly) due to "Parse, don't Validate" causing some expectations for library authors to adopt these principles, but I've rarely needed to use them in practice.
Hope this brings some insight!
Cheers
S

Thank you, @fabian-hiller and @sinclairzx81! It's such a pleasure that the authors of the two libraries I’m interested in have responded to my question.
After exploring the approaches of several libraries:
On the backend, I’ve decided to use Typebox for validations requiring strict data type correctness to achieve maximum performance. For more complex validation logic, I’ll implement custom checks.
As for Valibot, I’ll use it for certain asynchronous validations and some straightforward business logic integrations.
On the frontend, I plan to use Valibot exclusively to take advantage of its quick customization options and its small footprint.
Once again thank you both 👍

Thank you @sinclairzx81 for sharing your philosophy. It totally makes sense now. I really enjoy such conversations! I will keep your words in mind for future research and decisions.


SecChrootDir in ModSecurity 3?

Hi,
I'm trying to migrate to ModSecurity 3, docs are not ready at this moment, so how can i use SecChrootDir?
Thank you

@xocuer
SecChrootDir is not supported on libModSecurity yet.
libModSecurity 3 docs are being worked on at this moment, for now you can use the v2 docs at: https://github.com/SpiderLabs/ModSecurity/wiki/Reference-Manual-(v2.x).
If you are curious about a feature being supported on libModSecurity, look for the "Supported on libModSecurity" tag on the reference manual.

Thanks for the reply @victorhora
Do you know if in the future SecChrootDir will be supported?

Hi @xocuer
Actually I believe SecChrootDir should be deprecated for quite some time as Chroodir is available in Apache Core since 2.2.10 so there should be no need to have the feature implemented in libModSecurity.


BUG: Update pandas-gbq to latest version of google-cloud-bigquery

There was a breaking change in 0.32.0 which changed the way
configuration for the query job gets loaded. Also, it added the
'description' field to the schema resource, so this change updates the
schema comparison logic to account for that.
Note: I've created this PR based on the code currently in master in the google-cloud-bigquery repo. The tests pass locally, but they won't pass anywhere else until google-cloud-bigquery 0.32.0 is released. I've added the DO NOT MERGE label to this PR for now.
Since pandas-gbq will be broken when google-cloud-bigquery does its next release, I think it would make sense to review this before than so it can be merged when everything is ready. Once it goes in, I can create a pandas-gbq release to minimize the breakage time.

I think we should either make this backwards compatible or check the version on import and raise. Otherwise people are going to get very obscure error messages if their imports aren't correct (which is very possible with pinned installs)

Agreed. I had a hard time figuring out how to make this one backwards compatible since old versions would accept the config but then the API call to BigQuery fails later on.
I've updated the version number in _check_google_client_version(), so folks should get an understandable error message.

Though IIUC it's only backward incompatible for those passing in config dicts, so a very small share of users. Not worth a big lift, then

I've updated the version number in _check_google_client_version(), so folks should get an understandable error message.

Ah great! (some crossed wires with my last msg)

We could update the final build to also use master from google.cloud.bigquery as well as pandas - would be generally helpful as well as for this

@maxim-lian Good idea. I've made the "MASTER" build use the version of google-cloud-python hosted on GitHub. https://github.com/pydata/pandas-gbq/pull/152/commits/5f427a02c675c4a010761e306e2becbc7ae3385c
I'll keep an eye on the MASTER build on Travis to see if it works. https://travis-ci.org/tswast/pandas-gbq/builds/357586225

Finally got a green build on MASTER. https://travis-ci.org/tswast/pandas-gbq/jobs/357606308 Got failures on the other builds as expected, since google-cloud-bigquery hasn't released 0.32.0 yet.


Help Please

I understand that this is CLI so it needs some basic knowledge, but I'm hoping you'll breakdown for me a bit more. I have the file for timeular-reader-0.1.1 but I don't understand how to start using it. I know I need to open something in terminal but nothing seems to be working. My plan is to use Clockify. I have everything I need from Clockify but I'm not sure how to start using the CLI to connect them.

That's what I was doing wrong. I pulled the source code and not the actual release from Assets. Sorry. I was really tired.


feat: rename Listbox to Listboxkit and adjust its modules

Codecov Report

Merging #16 (2322327) into master (68d9cbd) will decrease coverage by 14.16%.
The diff coverage is 84.09%.

@@             Coverage Diff             @@
##           master      #16       +/-   ##
===========================================
- Coverage   92.36%   78.20%   -14.17%
===========================================
  Files           6        7        +1
  Lines         249      289       +40
  Branches       33       34        +1
===========================================
- Hits          230      226        -4
- Misses         18       61       +43
- Partials        1        2        +1

Impacted Files
Coverage Δ

src/ListboxKit__DropdownListbox.bs.js
0.00% <ø> (ø)

src/Listboxkit.bs.js
0.00% <0.00%> (ø)

src/Listboxkit__Listbox.bs.js
100.00% <ø> (ø)

src/Listboxkit__Dropdown.bs.js
88.09% <88.09%> (ø)

Codecov Report

Merging #16 (2322327) into master (68d9cbd) will decrease coverage by 14.16%.
The diff coverage is 84.09%.

@@             Coverage Diff             @@
##           master      #16       +/-   ##
===========================================
- Coverage   92.36%   78.20%   -14.17%
===========================================
  Files           6        7        +1
  Lines         249      289       +40
  Branches       33       34        +1
===========================================
- Hits          230      226        -4
- Misses         18       61       +43
- Partials        1        2        +1

Impacted Files
Coverage Δ

src/ListboxKit__DropdownListbox.bs.js
0.00% <ø> (ø)

src/Listboxkit.bs.js
0.00% <0.00%> (ø)

src/Listboxkit__Listbox.bs.js
100.00% <ø> (ø)

src/Listboxkit__Dropdown.bs.js
88.09% <88.09%> (ø)


L2: handle commit tx in mempool if node dies

If the node dies or is shut down, when it is restarted it starts producing blocks and tries to send the commitment transaction to the L1. If there is a pending transaction in the mempool, the transaction will fail.
We should handle this edge case.

Fixed in #1234


[ncp] Transforming spinel frames between AP and NCP

NCP Spinel Transformer allows to transform spinel frames sent between
Application Processor (AP) and Network Co-Processor (NCP).
Spinel frames can be transformed by using transformer library
(or libraries) implementing spinel_transformer.hpp. Libraries
can be specified using option --with-ncp-spinel-transformer-libs.
Addidionaly, transformer's outbound buffer can be changed if needed,
using option --with-ncp-spinel-transformer-outbound-buffer-size.

I signed it!

Hi @kbogucki , thanks for contributing to OT.
Can you please help provide some background/info about this change? What would be an example of a spinel_tranform.hpp implementation? Where/how do you see it being used? Thanks.

@abtink - this is for the security reasons. We are using this change to encrypt data transfer between NCP and AP, basically to avoid clear-text transmission over UART on the device. Our case is that the border router device (with some linux board and NCP, connected via UART) cannot be reliably protected from physical tampering by means of hardening the case (or it would be impractical).

Related PR in wpantund repo: https://github.com/openthread/wpantund/pull/258

Spinel Transformer example implementation (XOR):
https://gist.github.com/kbogucki/d16d2336bdb00efd196e37fe30b7e597

Thanks guys.


FIX: state_space_cython misc

add missing colons and fix indentation

This fixes #696

I have fixed the failed test case regarding the new version of matplotlib. @ajgpitch Could you update the branch? It seems that your changes fail on Python 2.7. Could you have a look at it? Thanks.

This PR should be closed, since it has been merged as part of PR #719.

Closing due to https://github.com/SheffieldML/GPy/pull/697#issuecomment-626277808
Thank you @jbect and @ajgpitch .


When initialization fails component's code should be taken from cache

When the client side core initializes it registers the host with the registry and saves the generated hostId. Component requests are queued until the request succeeds, but they are not released from the queue if the registration fails. We should try to take the code from local cache if it's available and release the queued requests anyways.

solved in #67


Changed deterministical generation of profile pic from user Id to user name

Changed deterministical generation of profile pic from user Id to user name


Verible README.html links to tools are broken...

The verible-verilog-ls link on https://chipsalliance.github.io/verible/README.html#language-server goes to https://chipsalliance.github.io/verible/verilog/tools/ls which is a 404.
Other similar links are also broken.
I think that either some extra README.md files need to be copied in the website build or the links changed?

@hzeller -- PTAL.


Disable CSP

Firstly, great project! Much thanks to the contributors for keeping it up.
Is there any way I could disable CSP for a while without breaking anything?
Silly question, but couldn't find anything in the issues or readme.

Just remove the security express middleware, r cherrypick from the middleware itself.


Bootstrap Migration for PriorityBillsCard

This issue is meant to track the remaining work needed to close out the migration from css modules to react-bootstrap/styled components/typescript. This issue covers the PriorityBillsCard component.
Success Criteria:

[ ] Refactor components to integrate Bootstrap and styled components while removing CSS modules.
[ ] When called for, find or create reusable (sub)components to reduce repetition and ensure uniformity.
[ ] Create or revise the Storybook story for these components, and any new sub-components.
[ ] Convert from js to tsx
[ ] Delete the HotBillsCard component. This seems to be an older version of the PriortiyBillsCard component that is no longer in use.

This card just covers this work for the PriorityBillsCard component.

UPDATE:
This is not currently used, and we're not sure when we'll have the resources to implement it. We should deprioritize this at a minimum, and probably just remove this and the Priority Bills Dashboard altogether until we're ready to commit.

@Mephistic can you clarify the implications of "remove" ? I assume we will still be able to retrieve the code when we do have the resources to tackle "priorities"? I believe we discussed creating a way to track what we've "removed"

We'll always be able to walk back through the git history to see the code.
It seemed like this component was abandoned over a year ago - if we're still planning to move forward with this, the update work involved in this card is very minimal and we can just do it.


DNM: testing apple/swift-installer-scripts#95

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

Please test with following PRs:
apple/swift-installer-scripts#95
@swift-ci please build toolchain windows platform

Please test with following PRs:
https://github.com/apple/swift-installer-scripts/pull/97
@swift-ci please build toolchain windows platform

Please test with following PRs:
https://github.com/apple/swift-installer-scripts/pull/97
@swift-ci please build toolchain windows platform

Please test with following PRs:
https://github.com/apple/swift-installer-scripts/pull/103
@swift-ci please build toolchain windows platform

Please test with following PRs:
https://github.com/apple/swift-installer-scripts/pull/103
@swift-ci please build toolchain windows platform

Please test with following PRs:
https://github.com/apple/swift-installer-scripts/pull/103
@swift-ci please build toolchain windows platform

Please test with following PRs:
https://github.com/apple/swift-installer-scripts/pull/103
@swift-ci please build toolchain windows platform

Please test with following PRs:
https://github.com/apple/swift-installer-scripts/pull/103
@swift-ci please build toolchain windows platform

Please test with following PRs:
https://github.com/apple/swift-installer-scripts/pull/103
@swift-ci please build toolchain windows platform

Please test with following PRs:
https://github.com/apple/swift-installer-scripts/pull/104
@swift-ci please build toolchain windows platform

Please test with following PRs:
https://github.com/apple/swift-installer-scripts/pull/105
@swift-ci please build toolchain windows platform


Test refactoring

I've splitted the big test.js into a multiple files, and added code coverage reports with coveralls.

Changes Unknown when pulling 274069c0587bafd7dd236bb68519c9249c331415 on test-refactoring into ** on master**.

Changes Unknown when pulling a9788613749973273aef92b9014309aa1eb911b0 on test-refactoring into ** on master**.

Changes Unknown when pulling 33a706d0b2256a66271a21fa0d61eee8e5142a5e on test-refactoring into ** on master**.

Changes Unknown when pulling a6ea26e989313b1d06cfd28fd8d54a1646b5f770 on test-refactoring into ** on master**.

once pickie thing that was originally my fault, then good for merge

Changes Unknown when pulling 6bc1af7788a3a3fff71a478a56ad90792981e6d9 on test-refactoring into ** on master**.


chore: add missing option cookie to URL screenshot

Hi @cherfia
I was trying to use the URL screenshot feature via cookies but, although the documentation says that I can use the cookie option, the screenshot.capture method doesn't seem to recognize it.

So, I checked by testing it on my gotenberg instance and it worked. I decided to add the missing option to the documentation. Please, let me know if I can help with anything else. Thanks!

Out of this PR, it looks like the Screenshot API also accepts the header and footer options. But, there is no such option in gotenberg for Screenshots FYI

@abdullah Thank you very much for your contribution 👏  I'll be releasing a new version that includes your changes. Also, I really appreciate the hint 💡
It's possible I misunderstood this part of the documentation:

You can capture screenshots using the following three routes, which function similarly to their PDF equivalents.

I'll double-check everything, clean up the code, and include it in the same release.

@cherfia 🎉
If you need a live instance, please email to<EMAIL_ADDRESS>can provide mine ♥


Update all.json

Site
Urlscan

dapptools.mobi
https://urlscan.io/result/7f5ac9c8-16f8-4ad6-977d-8aadae07a1ec/dom/

mintconnectpad.com
https://urlscan.io/result/6914ef6c-93b2-46a5-88a1-2cd9eff64aea/dom/

pro-walletconnect.com
https://urlscan.io/result/56fb0a71-d2d1-4c35-991c-8ab889dc487e/

walletconnectslive.org
https://urlscan.io/result/fc6f0eab-42dc-4af6-821e-7916b2ab5704/dom/

Force-merging. GH actions is up again, but seems it didn't trigger above.


Add filament grouping

Is your feature request related to a problem? Please describe.
It's currently difficult to manage multiple spools of the same material.
Describe the solution you'd like
It would be useful for the webUI to show filament grouped by type with a total material available stat. e.g. if I had 3 spools of Prusament Galaxy Black, it would be useful to have a group called "Prusament Galaxy Black" and it to note "3kg available"
Under this I would be able to view the individual spools available.
Describe alternatives you've considered
Additional context

Thats what the "Filament" is for.

Note there's a little bug with this filter: https://github.com/Donkie/Spoolman/issues/276 - but doesn't matter if you're only using a single material type.


Unable to create new container using New-AzStorageContainer

I am using below pattern to create a container by passing variables using powershell, however there seems to be some issue with the context being created, which is not allowing me to create the container and is showing 404(the specified resource does not exist).
$storage= New-AzStorageAccount -ResourceGroupName $RESOURCE_GROUP_NAME -Location $LOCATION 	-Name $TF_STATE_STORAGE_ACCOUNT_NAME
-SkuName Standard_LRS
$ctx=$storage.Context
New-AzStorageContainer -Name $TF_STATE_CONTAINER_NAME -Context $ctx

Note:(When i retried using the same command multiple times ,it showed me the same error and after around 10 minutes, my container got created, so maybe i am missing something regarding the context being created ??)
I have posted(link below as well) a question over the different context's  being showed as referencing the LazyStorage Constructor of the namespace Microsoft.WindowsAzure.Commands.Common.Storage, however the cmdlet New-AzStorageContainer accepts type "Microsoft.Azure.Commands.Common.Authentication.Abstractions.IStorageContext" as interface which should be fetching the AzureStorageContext Class not LazyAzureStorageContext Class.
https://docs.microsoft.com/en-us/powershell/module/az.storage/new-azstoragecontainer?view=azps-5.5.0
Question posted link :
https://docs.microsoft.com/en-us/answers/questions/289096/new-azstoragecontainer-command-not-accepting-valid.html?childToView=290003#comment-290003
Note: i have tried putting in Wait-Events as well between all the commands in the ps script, however of no use.

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

ID: b04a28c3-81d7-123d-5b16-6642636c07a8
Version Independent ID: 2179498b-00e8-1c9d-bd11-4a903ae3ad52
Content: Quickstart - Create a blob with PowerShell - Azure Storage
Content Source: articles/storage/blobs/storage-quickstart-blobs-powershell.md
Service: storage
Sub-service: blobs
GitHub Login: @tamram
Microsoft Alias: tamram

@chetansir Thank you for your query and posting the same on QnA . We will investigate and update the thread on QnA as it is a troubleshooting query.

@chetansir Thank you for your query and posting the same on QnA . We will investigate and update the thread on QnA as it is a troubleshooting query. We will close this query here and will post the answer after our testing directly on QnA question. Thank you for your patience with us.

Hi Team,
Any updates on this. I can't see any response on the QnA as well.


Can't generate dbt docs

Describe the bug
We use the dbt-databricks adapter and can successfully read our models from dbt and also write them. However, when we call dbt docs generate we get the following error: Expected only one database in get_catalog, found.
To be on the safe side, I took another look at the source.yml. Unfortunately, only the schema is set there and I don't know why dbt docs generate shouldn't work.
Steps To Reproduce
Try to generate the dbt docs for a databricks destination / source
Expected behavior
Docs should be generated
Screenshots and log output

System information
The output of dbt --version:
╰─$ dbt --version
installed version: 1.0.1
   latest version: 1.0.1

Up to date!

Plugins:
  - databricks: 1.0.0
  - spark: 1.0.0

The operating system you're using:
The output of python --version:
╰─$ python --version
Python 3.8.12

@mcfuhrt Could you also try with dbt-databricks 1.0.1?
@jtcohen6 Could you help to take a look at this?

@ueshin I installed dbt-databricks 1.01. but the result of dbt docs generateis the same.
╰─$ dbt --version
installed version: 1.0.1
   latest version: 1.0.1

Up to date!

Plugins:
  - databricks: 1.0.1
  - spark: 1.0.0

06:32:18  Found 84 models, 0 tests, 0 snapshots, 0 analyses, 215 macros, 0 operations, 0 seed files, 55 sources, 0 exposures, 0 metrics
06:32:18
06:32:19  Concurrency: 1 threads (target='prep')
06:32:19
06:32:19  Done.
06:32:19  Building catalog
06:32:20  Encountered an error:
Compilation Error
  Expected only one database in get_catalog, found [<InformationSchema INFORMATION_SCHEMA>, <InformationSchema INFORMATION_SCHEMA>]

@ueshin dbt-spark is raising this explicit error in the get_catalog method here, because it:

Operates on one schema at a time (today by running show table extended in <schema_name> like '*'
Stores all those schemas as belonging to the same "null" database (since dbt is organized to think in three hierarchy levels, with database at the top)
Won't know what to do if passed more than one database, since even the first is really just an internal placeholder

The history here is all around supporting full dbt functionality on Spark, which doesn't have a distinction between database + schema as separate hierarchy layers: https://github.com/dbt-labs/dbt-spark/pull/92
The method that passes schemas into get_catalog, called _get_catalog_schemas, is not reimplemented in dbt-spark, so we're using the version defined in dbt-core. The idea is to walk through the enabled relational nodes in the manifest, and to return a SchemaSearchMap() that includes all databases relevant for catalog generation, mapped to the schemas they contain.
In Spark/Databricks, this SchemaSearchMap should look like:
{<InformationSchema INFORMATION_SCHEMA>: {'schema_one', 'schema_two'}}

Note that the database name is missing (specifically, it's set to None). On adapters that have separate notions of schema/database, even ones that support only one active database, we'd see:
{<InformationSchema "database_name".INFORMATION_SCHEMA>: {'schema_one', 'schema_two'}}

On a database that supports multiple databases, and a different hierarchy between them, we should see this object return:
{<InformationSchema "database_one".INFORMATION_SCHEMA>: {'schema_one', 'schema_two}, <InformationSchema "database_two".INFORMATION_SCHEMA>: {'schema_three'}}

Yet, somehow, @mcfuhrt is seeing two different databases returned in the SchemaSearchMap, both with missing (None) database names. I've tried a few different things locally to reproduce this exact error (source definitions, model configurations, profiles.yml configurations). The only way I've managed to trip this error, without first tripping a different error, is by setting database for a source, using the same value as its schema. E.g.:
sources:
  - name: my_src
    schema: my_src
    database: my_src
    tables:
      - name: tbl_one

Then I see:
08:46:55  Encountered an error:
Compilation Error
  Expected only one database in get_catalog, found [<InformationSchema INFORMATION_SCHEMA>, <InformationSchema my_src.INFORMATION_SCHEMA>]

Crucially, however, I see the database name (my_src) included in the second SchemaSearchMap entry, which gives me a hint as to where the error is originating.

@jtcohen6, is there anything I can do to support here or do you need more information (e.g. configuration, source.yml, etc.) from me?
I am not sure how to proceed to solve the problem.

@mcfuhrt I think we'll need some more configuration details to get to the bottom of this one. Are you able to reproduce the error in a simpler project, with the same basic config as your real project? If so, would you be up to share it here?
If not, and you're willing to share your real project (including profiles.yml) via DM in dbt Slack, that also works

I have built a very small example that can be reproduced quickly. Unfortunately, I also get the error when generating the doc files.
Setup:
Single node cluster hosted on Azure (West-Europe).

Notebook for reading in the sample data and saving it as a delta table.

schema = 'default'
table = 'diamonds'
spark.sql(f"CREATE DATABASE IF NOT EXISTS {schema};")

diamonds = spark.read.format('csv').options(header='true', inferSchema='true').load('/databricks-datasets/Rdatasets/data-001/csv/ggplot2/diamonds.csv')

(
    diamonds
    .write
    .format("delta")
    .option("overwriteSchema", "true")
    .mode("overwrite")
    .saveAsTable(f"{schema}.{table}")
)

Configuration of the dbt project
name: 'analytics'
version: '1.0.0'
config-version: 2

vars:
  'dbt_date:time_zone': 'Europe/Berlin'

profile: 'databricks'

model-paths: ["models"]
analysis-paths: ["analysis"]
test-paths: ["tests"]
seed-paths: ["data"]
macro-paths: ["macros"]
snapshot-paths: ["snapshots"]

target-path: "target"
clean-targets:
    - "target"
    - "dbt_modules"

models:
  analytics:

Configuration sources.yml
version: 2

sources:
  - name: default
    schema: default
    loader: Direct
    loaded_at_field: none

    quoting:
      database: true
      schema: false
      identifier: false

    freshness:
      warn_after: {'count': 8, 'period': hour }
      error_after: {'count': 24, 'period': hour}

    tables:
      - name: diamonds

Configuration of the Databricks profile

databricks:
  outputs:
    prep:
      host: #HOST#
      http_path: #HTTP_PATH#
      schema: default
      threads: 1
      token: #TOKEN#
      type: databricks
  target: prep

dbt, plugins and python version

╰─$ dbt --version
installed version: 1.0.3
   latest version: 1.0.3

Up to date!

Plugins:
  - databricks: 1.0.1 - Up to date!
  - spark: 1.0.0 - Up to date!

╰─$ python --version
Python 3.8.12

dbt run result

╰─$ dbt run                                                                                130 ↵
13:56:37  Running with dbt=1.0.3
13:56:37  Unable to do partial parsing because a project config has changed
13:56:37  Found 1 model, 0 tests, 0 snapshots, 0 analyses, 212 macros, 0 operations, 0 seed files, 1 source, 0 exposures, 0 metrics
13:56:37
13:56:40  Concurrency: 1 threads (target='prep')
13:56:40
13:56:40  1 of 1 START view model default.default_diamonds_source......................... [RUN]
13:56:43  1 of 1 OK created view model default.default_diamonds_source.................... [OK in 2.31s]
13:56:43
13:56:43  Finished running 1 view model in 5.70s.
13:56:43
13:56:43  Completed successfully
13:56:43
13:56:43  Done. PASS=1 WARN=0 ERROR=0 SKIP=0 TOTAL=1

dbt docs generate result
╰─$ dbt docs generate
13:57:27  Running with dbt=1.0.3
13:57:27  Found 1 model, 0 tests, 0 snapshots, 0 analyses, 212 macros, 0 operations, 0 seed files, 1 source, 0 exposures, 0 metrics
13:57:27
13:57:29  Concurrency: 1 threads (target='prep')
13:57:29
13:57:29  Done.
13:57:29  Building catalog
13:57:29  Encountered an error:
Compilation Error
  Expected only one database in get_catalog, found [<InformationSchema INFORMATION_SCHEMA>, <InformationSchema INFORMATION_SCHEMA>]

Thanks for this @mcfuhrt! I was able to reproduce the error, using the example you've provided.
It looks like this line in your source definition is to blame:
    quoting:
      database: false

If I switch that to database: true, or remove it entirely, docs generate works fine.
Alternatively, if I add those lines to dbt_project.yml so that the project-wide quoting config matches (docs), it also works.
There are two ways forward I can see:

Fix the logic within SchemaSearchMap (called by _get_catalog_schemas), which currently views a quoted "None" database and an unquoted "None" database as distinct entries in the set of databases. This makes sense in the general case—they have different quote_policy—but it doesn't make sense when the value is None.
Defining this config doesn't (shouldn't) have any actual effect. Given that we don't allow database to be defined elsewhere in dbt-spark (+ dbt-databricks, at least until https://github.com/dbt-labs/dbt-spark/issues/281), it really feels like our answer here should be to raise an explicit error any time the quoting config is defined at the database level.

Thank you @jtcohen6  for your detailed explanation of how the error came about and how you deal with it!
I also think that it should be enough at this point to throw the corresponding exception and then refer to the database quoting in the dpt_project.yml.  I tested your fix yesterday and as expected, dbt docs generate now runs successfully.

@jtcohen6  is there a possibility to set the database name in the generated docs? I want to connect our dbt docs and our databricks tables within amundsen and it seems like this won't work while the database names are inconsistent or null.

@mcfuhrt For adapters with "two-tiered" namespacing, the top-level tier (database) will always be null. This should be totally consistent, minus the issue you uncovered around quoting. I don't imagine this would prevent you from being able to integrate with Amundsen, so long as you're matching on the schema + identifier for each object?
Looking ahead:

Definitely aware of the need for cosmetic improvement in the docs site: https://github.com/dbt-labs/dbt-docs/issues/104
We're also aware that Spark is looking to adopt a three-tiered namespace, with the top-level tier called "catalog," at which point the behavior in dbt-spark + dbt-databricks will be more consistent with other adapter plugins

@jtcohen6 I'm facing the same error with database: false when using dbt with mysql. Is there a way around this ?
dbt: 1.0.9
dbt-mysql: 1.0.0


[bug] Bitrate selection menu pops up without any keyboard shortcut

[bug] Inspite of set_max_bitrate set to true the bitrate selection menu keeps popping up on any new instance of netflix.
Also, it is a bit unclear what the override and reset buttons functions are in the bitrate selection menu. A small note in the readme would be useful.
Browser: Firefox 124.0.1

setting set_max_bitrate to true is what makes it pop up, set it to false if you want to invoke it manually

(the menu is supposed to disappear once the max bitrate has been selected, but it's buggy)

is it possible to automatically pick the highest bitrate rather than picking from the pop-up every episode?

That's what set_max_bitrate is supposed to do, that's the option

I found the reason, at least on my system.
set_max_bitrate() returns false when one of the selects is disabled. But the audio select is disabled for me. Thus I changed that abort:
https://github.com/DavidBuchanan314/Turbo-Recadmiumator/blob/50dff66f61a08b95c3206e2766ad4ed8e9bfe706/src/cadmium-playercore-shim.js#L129
to:
if (select.disabled){
	if (list == AUDIO_SELECT) {
		continue
	} else {
		return false;
	}
}

That wasn't enough though, because this check also failed:
https://github.com/DavidBuchanan314/Turbo-Recadmiumator/blob/50dff66f61a08b95c3206e2766ad4ed8e9bfe706/src/cadmium-playercore-shim.js#L139
I'm not sure what the intention was here? So I removed that if.
Now it works for me.
PS: Even with set_max_bitrate set to false the player patching works in enabling 1080p playback on Linux, if available. Not sure why, just by adding the profile maybe? But I also stumbled over a movie where all of this had no effect, I guess where netflix really does not deliver 1080p playback. Or maybe there are additional playback profiles though ought to be enabled there?

PS: Even with set_max_bitrate set to false the player patching works in enabling 1080p playback on Linux, if available. Not sure why, just by adding the profile maybe? But I also stumbled over a movie where all of this had no effect, I guess where netflix really does not deliver 1080p playback. Or maybe there are additional playback profiles though ought to be enabled there?

I don't know for sure, but I believe certain content is configured to require "VMP" (Verified Media Path), which sadly is not available on the Linux platform.

That might be it.
Well, I hope the described code changes help with the max bitrate issue at least :)


[Refactoring] Client & Server base components

We want to create client & server base components to provide similar benefits of API Handler.
https://github.com/bcgov/platform-services-registry/blob/main/core/api-handler.ts

[x] 1. Role and permissions checking with redirection to a specified URL as a fallback.
[x] 2. Validation of path parameters.
[x] 3. Validation of search parameters.
[x] 4. Provision/provide of parsed parameter data and session information.
[ ] 5. Mitigation of confusion between Next.js client and server components.('use client')

https://nextjs.org/docs/app/building-your-application/rendering/client-components


Updated master branch doesn't install "vector_math" Dart package by default

Hi!👋
So, i've created week ago my app with Flutter being on fresh master branch for this whole time, and it worked as expected.
Couple of days ago (27-28/04/2020) i've update master with latest changes, but cleared pub-cache folder before, so just to have new updated cache of Dart packages.
After that, when trying to start my App on iOS Emulator, realized, that it doesn't work, console throws bunch of exceptions, blaming that there are no vector_math package in cache installed.
With flutter pub cache add vector_math problem was solved, but it manual installation, is there are particular reason why this package was removed from initial, fresh flutter installation?🤔
And this behavior only with master and dev branches, on stable and beta this package still exists and everything working as expected

Hi @ZaJarv
it may have something to do with how the pub_cache folder was cleared
did you run flutter pub cache repair or what did you delete it manually
could you please provide the steps to reproduce this behaviour?
thank you

Hey @iapicca 👋
It was classical rm -rf inside pub-cache directory.
I looked into my project, and there is dependency for vector_math: vector_math:file:///Users/zajarv/.dev/pub-cache/hosted/pub.dartlang.org/vector_math-2.0.8/lib/ in .packages file
When i created this project (couple of weeks ago) with help of Android Studio, Flutter was on master branch, and since then i updated Flutter with flutter upgrade multiple times and from time to time deleting content of pub-cache manually before upgrading flutter, so flutter always fills pub-cache directory with Dart packages.
But couple of days ago when i've done completely the same procedure, emptying pub-cache with rm -rf and then flutter upgrade my app could not start because there was no vector_math installed now😨
Repeating this procedure but being in stable or beta installs all required packages including vector_math. master and dev doesn't install this package.
I can install it manually of course, but still ineteresting why it's removed from default installation procedure and seems that list of packages differs between branches🧐

After that, when trying to start my App on iOS Emulator, realized, that it doesn't work, console throws bunch of exceptions, blaming that there are no vector_math package in cache installed.

@ZaJarv few questions:

Does your own code reference vector_math? or it is some other code?
If your code references vector_math - do you have vector_math dependency declared in your pubspec.yaml?
Why are you purging pub cache? You should not need to do that ever.

@mraleph

Seems not, since it very basic App with mostly layout widgets imported from 'package:flutter/material.dart';

code not references vector_math so not references in pubspec.yaml too

No answer to this😄But i don't think that it relevant, sine Flutter when upgrades or after switching branches and with flutter upgrade anyway downloads dart packages

Update.
With fresh installation of Flutter everything works as expected🤔
Steps:

performed fresh installation of Flutter
used branch - master
checked pub-cache directory, vector_math package was not found
opened Android Studio and created new Flutter project, checked dependencies in it, found vector_math as dependecy.
checked pub-cache directory and found vector_math package with different timestamp than other packages, so it's possible that it was installed by Android Studio🤔

But not sure then, why earlier everything was okay when i changed branches to stable and beta🤷♀

Yeah, solved, this, and a few other packages installed by Android Studio when new project is created, just checked, sorry for disturbance guys🤦♀


Investigate https://www.sassymamasg.com/the-ultimate-guide-to-local-breakfast-in-singapore/

Tested and works now!


Feature Request - allow pass env-vars to the SUT

I would like to be able to instruct nyc to pass environment variables to it's sub-processes so I would not need one more wrap in form of more cli context libraries such as

https://www.npmjs.com/package/env-pass
https://www.npmjs.com/package/cross-env

Hi @osher sorry for the late reply - while I appreciate you filing this issue, I think it is outside the scope of nyc, unless, you're willing to see it through with a PR and appropriate tests :) or if there's strong enough of a response from the community. Since this has been stale for a while I am going to close it for now.


Encoding issues

At least one query 香港 returns an invalid XML:
https://backend.deviantart.com/rss.xml?type=deviation&q=香港&0
Another one with 東京 says "Error generating RSS." in the description :
https://backend.deviantart.com/rss.xml?type=deviation&q=東京&0
It seems there are problem with encoding.

Closing this ticket due to its age. If this is still an issue, please feel free to reopen. Thanks!


SyntaxError: invalid syntax in AREPL

Hi @Xirider, I just installed your extension, but when I run it, I seem to be getting an error from within the internals of the extension. I am getting the following error:
File "xirider.livecode-1.3.10/node_modules/arepl-backend/python/arepl_python_evaluator.py", line 43
    userError: str,
             ^
SyntaxError: invalid syntax

I also get the following error on top of that:
internal livecode error: Error stack: Error [ERR_STREAM_DESTROYED]: Cannot call write after a stream was destroyed
	at doWrite (_stream_writable.js:414:19)
	at writeOrBuffer (_stream_writable.js:402:5)
	at Socket.Writable.write (_stream_writable.js:302:11)
	at PythonShell.send (/home/anon/.vscode/extensions/xirider.livecode-1.3.10/node_modules/python-shell/index.js:285:20)
	at PythonEvaluator.execCode (/home/anon/.vscode/extensions/xirider.livecode-1.3.10/node_modules/arepl-backend/index.js:62:22)
	at ToAREPLLogic.onUserInput (/home/anon/.vscode/extensions/xirider.livecode-1.3.10/out/src/toAREPLLogic.js:79:34)
	at PreviewManager.onAnyDocChange (/home/anon/.vscode/extensions/xirider.livecode-1.3.10/out/src/PreviewManager.js:337:54)
	at listOnTimeout (internal/timers.js:533:17)
	at processTimers (internal/timers.js:475:7)

I'm not sure what's going on here, but I thought I'd point it out.
My Specs

Component
Version

OS
Ubuntu 20.04

Architecture
x86_64

VSCode
1.46.1

LiveCode
1.3.10

Python
3.8.3

Note: In the first error, I removed the first part of the path to make it shorter. It's actually /home/applemonkey496/.vscode/extensions/...

Hi @applemonkey496 ,
thanks for the report! This can happen if LiveCode can't find your python path:
Please try to set livecode.pythonPath in the settings to your python 3 installation.
I hope that helps.

It works now, thanks!

@applemonkey496
sir ，may i ask how did u do that ,i can't find any to change ,oh ,maybe  because i used no good ,can u describe that how to do ,what kind code id need
by the way ,i am mac m1,always told me verson 2.7
thanks
your hangzhou friend


List songs in a playlist in the playlist selection activity

Should look like the mockup.

Currently rethinking UI, and this will likely not be applicable.

I've simplified the project plans to exclude a visible tablet, since it was redundant to the web interface. As such, I won't be implementing this.


Ignores -Wattributes for GCC 7 to mask compiler bug.

See https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89325
This bug produces massive amounts of spurious warnings and is present
in the version of g++ currently available in RHEL devtoolset-7.

Travis-ci pipeline looks like it failed because the build system couldn't find a public key needed by clang
W: GPG error: http://apt.llvm.org/precise llvm-toolchain-precise-3.5 InRelease: The following signatures couldn't be verified because the public key is not available: NO_PUBKEY 15CF4D18AF4F7421
Appveyor failed for something similar.:
make: fork: Resource temporarily unavailable

Thanks! I'm happy to merge this as is, but I have a question. Why is the upper bound 7.4? Looking at the bug you linked, it appears to still be unresolved.

I was looking around for similar complaints and found that a couple users
of Eigen3 couldn't reproduce the issue on gcc 7.4 on Ubuntu.  I did not
test this myself, but will investigate further later tonight.
On Wed, Dec 4, 2019, 11:35 Anton Bachin<EMAIL_ADDRESS>wrote:

Thanks! I'm happy to merge this as is, but I have a question. Why is the
upper bound 7.4? Looking at the bug you linked, it appears to still be
unresolved.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aantron/better-enums/pull/79?email_source=notifications&email_token=AGISHAAGUJGSLV2OSAMFRDLQW7L3HA5CNFSM4JSIDPB2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEF5UBLA#issuecomment-561725612,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGISHACDYHKRS3RGSTZVPMLQW7L3HANCNFSM4JSIDPBQ
.

Thanks for the info. If it ends up being too much trouble, we can merge this as is. We can always widen the constraint later, or adjust it otherwise.
I don't have time to fix the CI at the moment, but the Linux CI setup seems to have broken entirely due to clang repos :/

So this GCC bug still exists in devtoolset-8, I made the version where this warning is ignored open-ended since it appears to not be fixed at this time.

Thank you!


fix: linking with from prop and a param containing slash

A failing test case for #1762 that sets up a page on /$postId with a <Link from="/$postId" to="/$postId">Link to self with from prop set</Link>. When this page is visited and the $postId params contains a slash, the router crashes with Invariant failed: Could not find match for from: /$postId

closed in favor of https://github.com/TanStack/router/pull/1799


feature-request: Ask a second relay for the payload, if the winner relay did not provide payload in time

Previously, there has been several times when a mev relay who won the bid but did not provide the payload in time (either connection issue or a service issue), resulting in missed proposals and loss of money.
In future, is it possible that we could ask a second relay for the payload as a backup simultaneously? Or ask in sequence when the first relay fails to respond and then ask a second best bid?

Examples of the missed proposals due to not getting payload from best bidder in time:
...
time="2022-11-02T..." level=info msg="best bid" blockHash=0x2433c38afa8ae9770a7aaf07176b135b270b6a5d6af3747fc06d52d05b9f4dad blockNumber=xxx method=getHeader module=service parentHash=xxx pubkey=xxx<EMAIL_ADDRESS>slot=xxx txRoot=0xf41a6dbbc4e5a49105148422c206975f60580221582e5f785472049132527308 value=97027507961221801 version=1.3.2
time="2022-11-02T..." level=info msg="http: GET /eth/v1/builder/header/xxx/xxx/xxx 200" duration=0.649754 method=GET module=service path=/eth/v1/builder/header/xxx/xxx/xxx status=200
time="2022-11-02T..." level=error msg="error making request to relay" blockHash=0x2433c38afa8ae9770a7aaf07176b135b270b6a5d6af3747fc06d52d05b9f4dad error="Post "https://relay.edennetwork.io/eth/v1/builder/blinded_blocks\": context deadline exceeded (Client.Timeout exceeded while awaiting headers)" method=getPayload module=service parentHash=xxx slot=xxx url="https://relay.edennetwork.io/eth/v1/builder/blinded_blocks" version=1.3.2
time="2022-11-02T..." level=error msg="no payload received from relay!" blockHash=0x2433c38afa8ae9770a7aaf07176b135b270b6a5d6af3747fc06d52d05b9f4dad method=getPayload module=service parentHash=xxx relays="https://0xb3ee7afcf27f1f1259ac1787876318c6584ee353097a50ed84f51a1f21a323b3736f271a895c7ce918c038e426
...
time="2022-10-17T..." level=info msg="best bid" blockHash=0x6fab46a948a6cd834aa7eaa5dc244cb3102c3ab5e3ca0b6058839c1fda53ca8c blockNumber=xxx method=getHeader module=service parentHash=xxx pubkey=xxx<EMAIL_ADDRESS>slot=xxx txRoot=0x758a300d635146604d935fb5db3dc7f1a56120632f70760f466523753e90df14 value=74472431003129321 version=1.3.2
time="2022-10-17T..." level=info msg="http: GET /eth/v1/builder/header/xxx/xxx/0xxxx 200" duration=0.963004 method=GET module=service path=/eth/v1/builder/header/xxx/xxx/0xxxx status=200
time="2022-10-17T..." level=debug msg=getPayload method=getPayload module=service version=1.3.2
time="2022-10-17T..." level=debug msg="calling getPayload" blockHash=0x6fab46a948a6cd834aa7eaa5dc244cb3102c3ab5e3ca0b6058839c1fda53ca8c method=getPayload module=service parentHash=xxx slot=xxx url="https://relay.edennetwork.io/eth/v1/builder/blinded_blocks" version=1.3.2
time="2022-10-17T..." level=error msg="error making request to relay" blockHash=0x6fab46a948a6cd834aa7eaa5dc244cb3102c3ab5e3ca0b6058839c1fda53ca8c error="HTTP error response: 400 / {"code":400,"message":"read tcp <IP_ADDRESS>:9062-\u003e<IP_ADDRESS>:53390: i/o timeout"}\n" method=getPayload module=service parentHash=xxx slot=xxx url="https://relay.edennetwork.io/eth/v1/builder/blinded_blocks" version=1.3.2
time="2022-10-17T..." level=error msg="no payload received from relay!" blockHash=0x6fab46a948a6cd834aa7eaa5dc244cb3102c3ab5e3ca0b6058839c1fda53ca8c method=getPayload module=service parentHash=xxx<EMAIL_ADDRESS>slot=xxx version=1.3.2
Note: The relays are working as intended 99% of the time, but things like the above just happens.

Hey @biteric2000 that's a great suggestion, but unfortunately it's not possible. At this particular moment, you have submitted a SignedBlindedBeaconBlock to the relay and would be at risk of being slashed if you proposed a different block as a fallback. The relay could gossip the block you sent it and you would be slashed for making two proposals.
According to the builder-specs:

This means if a validator publishes a signature for a BlindedBeaconBlock (via a dissemination of a SignedBlindedBeaconBlock) then the validator MUST not use the local build process as a fallback, even in the event of some failure with the external builder network.

In your examples, both of the missing payloads came from the Eden relay. I suggest reaching out to them.

Thank you @jtraglia for your helpful comment and explanation.
The above logs are just examples. Actually we just missed one proposal with relaywooor yesterday, with a reward tip of 0.08 ETH lost (excluding execution reward). Previously we also tried to contact the relay service providers through channels like discord, but with little feedback / solution.
Anyway, looking forward to any possible means towards solving the similar problems in future!


Add embeddings for bottle aliases

Dont think this is ultimately going to be that useful. It could be a lazy way out for a moderation queue, but going to keep this as a draft until I validate more rule-based approaches.

Refs #55

Deciding this is likely still going to be useful for the manual matching - basically we can build a queue of actions needed, use the embeddings to find the 'most likely match', and make it much faster for a human to review the queue. We'll then add a new BottleAlias for every match, and slowly widdle down issues. Its not the most efficient way, but it will certainly be a correct way.
Likely will continue then with improving BottleFinder to have additional rule-based approaches similar to the matches examples included.


We.Retail pages do not load correctly inside the Target Visual Experience Composer (VEC)

When loading a we.retail webpage inside Adobe Target Visual Experience Composer (VEC), the page does not render correctly as shown below:

The page footer is located right below the header, instead of the bottom of the page.
This is because the we.retail expects the body element to have relative position, however inside the VEC , Adobe Target overrides the body position and makes it static:

I was able to solve this on my local AEM instance by removing the bottom value from .footer .we-Footer
(https://github.com/Adobe-Marketing-Cloud/aem-sample-we-retail/blob/master/ui.apps/src/main/content/jcr_root/apps/weretail/components/structure/footer/clientlibs/less/footer.less)
I don't think the bottom value is necessary as the footer will flow to the bottom without it. Please check. This fix is required for demoing We.Retail content inside Adobe Target.
cc @keepthebyte , @martinbuergi , @gabrielwalt @gknobloch @bpauli

I agree, we could have the footer positioned statically.

(CQ-4263821)

This issue has been fixed and will be available with the 4.0.0 release of We.Retail


fix: Change updateURL

Update updateURL.
Fromdark-whatsapp to onyx.
Fix #68

It's by design. Announced in v1.6.3 release notes.


[Feature Request]: Add Clustering for non-linear DML models

Describe the feature you want to propose or implement
Extend the NonLinearScoreMixin to handle data with clustering.
It should be sufficient to extend the estimation of the coefficient/parameter as the variance estimation is identical.
Propose a possible solution or implementation
For dml1, just use the current implementation without clustering (should be identical).
For dml2, adjust the scaling of the score analogously to LinearScoreMixin.
Did you consider alternatives to the proposed solution. If yes, please describe
No response
Comments, context or references
No response

It seems possible to combine the methods _est_coef and _est_coef_cluster_data, since the estimation should be identical except for a scaling factor. This would require to move the distinction between 'dml1' and 'dml2' from _est_causal_pars to the mixins.
As a result _est_causal_pars and _est_coef_cluster_data could be removed.


cygwin_nt-6.1-wow64-x86/bin/arm-linux-androideabi-gcc no such a file or directory

trying to install ffmpeg on Cygwin on Windows 7, 64bit.
in /android-ndk-r9c-windows-x86_64/android-ndk-r9c/toolchains/arm-linux-androideabi-4.8/prebuilt/
I have windows-x86_64 directory in which the compiler does exist.
but for some reason that I don't understand, the configure looks for cygwin_nt-6.1-wow64-x86/ which doesn't exist.
Does any body know how can I overcome this problem? either get the folder, or make it so that it looks for right directory?

There is a waste of time trying to compile ffmpeg on windows, so please use linux or mac


New version

Hi, the last release was 3 years ago. Please push a new version onto pypi.

@svenstaro, this repo is for pre redis 3.0 . are you aware of that ?

Oh, no I'm not. I thought it was a way to backport that functionality to redis 2.x :/

Yep this is for redis 2.x


Run tests on travis

Do you mind reopening it? This should be fixed

In order to correctly download GTest, you need to tell cmake how to download it.
You can find how to do this in xeus-python:

You will need this file:
https://github.com/jupyter-xeus/xeus-python/blob/master/test/downloadGTest.cmake.in
And you will need to configure it:
https://github.com/jupyter-xeus/xeus-python/blob/master/test/CMakeLists.txt#L48

Great! It's green. Would you mind rebasing and squashing your PR?

Great! It's green. Would you mind rebasing and squashing your PR?

Sure!


Question: site support

[x] I've verified and I assure that I'm running youtube-dl 2016.04.13

[x] At least skimmed through README and most notably FAQ and BUGS sections

[x] Searched the bugtracker for similar issues including closed ones

[ ] Bug report (encountered problems with youtube-dl)

[ ] Site support request (request for adding support for a new site)

[ ] Feature request (request for a new functionality)

[x] Question

[ ] Other

Regarding site support, how do I know if a site infringes on copyright laws?

https://github.com/rg3/youtube-dl#can-you-add-support-for-this-anime-video-site-or-site-which-shows-current-movies-for-free


Output breaking changes in a copy/paste format for --err-ignore option

Is your feature request related to a problem? Please describe.
When breaking changes are detected with this tool, it's not obvious what format should be used in conjunction with --err-ignore. The docs do describe the format, but it'd be nice to not have to copy and paste parts of the output to form the right structure.
Describe the solution you'd like
The output for oasdiff breaking should include a copy/pastable version of breaking change errors that can be dropped into a file that's passed to --err-ignore.
Example:
1 breaking changes: 1 error, 0 warning

error [response-required-property-removed] at revision.json
	in API GET /api/route
		removed the required property 'required' from the response with the '200' status

To ignore: GET /api/route removed the required property 'required' from the response with the '200' status

The message output could use some work... but just trying to illustrate the point.
Describe alternatives you've considered
Maybe another output format? e.g. via --format
Additional context
N/A

You can easily do this by piping the output into a file:
oasdiff breaking data/openapi-test1.yaml data/openapi-test3.yaml > /tmp/err-ignore.txt
oasdiff breaking data/openapi-test1.yaml data/openapi-test3.yaml --err-ignore /tmp/err-ignore.txt

Please let me know if this solution is sufficient.

Oh interesting... how does that work? It looks like the output format is different when redirected to a file somehow?

Yes, the code checks if the output is piped or redirected and, if so, uses a simplified output format.
BTW, you can achieve the same result without a file too:
oasdiff breaking data/openapi-test1.yaml data/openapi-test3.yaml | cat


"Missing sitekey" error with Angular 11.2.14

Hi,
when using the web-component as described in the documentation, I get the following error in the console:
[hCaptcha] Missing sitekey - https://hcaptcha.com/docs/configuration#jsapi
How I included it:
<h-captcha
            [attr.site-key]="siteKey"
            (verified)="verified($event)"
            *ngIf="showHCaptcha"
        ></h-captcha>

If I am using it like this it works:
<h-captcha
            site-key="<my-site-key>"
            (verified)="verified($event)"
            *ngIf="showHCaptcha"
        ></h-captcha>

Is there a way to fix this? Thanks.

It seems like there is a delay, when binding the value. By wrapping a timeout around the contents of the connectedCallback()-function the bug is solved:
connectedCallback() {
    setTimeout(() => {
        this.config = {
            sitekey: this.getAttribute('sitekey') || this.getAttribute('site-key'),
            theme: this.getAttribute('theme') || this.getAttribute('dark') !== null ? 'dark' : undefined,
            size: this.getAttribute('size'),
            tabindex: this.getAttribute('tabindex'),
            language: this.getAttribute('language'),
            reCaptchaCompat: this.getAttribute('reCaptchaCompat') === "false",
            challengeContainer: this.getAttribute('challengeContainer'),
            rqdata: this.getAttribute('rqdata'),
            sentry: this.getAttribute('sentry') === "true",
            apiEndpoint: this.getAttribute('apiendpoint') || 'https://hcaptcha.com/1/api.js',
            endpoint: this.getAttribute('endpoint'),
            reportapi: this.getAttribute('reportapi'),
            assethost: this.getAttribute('assethost'),
            imghost: this.getAttribute('imghost'),
        };
        loadApiEndpointIfNotAlready(this.config).then(this._apiLoaded.bind(this)).catch(this.onError.bind(this));
    }, 0);
}

The issue seems to be with angular which would render the component with no attributes at first.
This should be fixed in the latest 1.0.0-alpha.3. Please let me know if you still have the issue with the latest version.


Fix data race between Array.chop and Array.undefined's use of realloc

Closes #4179

Hi @SeanTAllen,
The changelog - fixed label was added to this pull request; all PRs with a changelog label need to have release notes included as part of the PR. If you haven't added release notes already, please do.
Release notes are added by creating a uniquely named file in the .release-notes directory. We suggest you call the file 4223.md to match the number of this pull request.
The basic format of the release notes (using markdown) should be:
## Title

End user description of changes, why it's important,
problems it solves etc.

If a breaking change, make sure to include 1 or more
examples what code would look like prior to this change
and how to update it to work after this change.

Thanks.

The implementation looks good :+1:

@jemc im worried about the test because it is possible that this works and that test will fail. it would be very rare but its possible. i dont want to write such a test. ive been trying to think of a test that cant randomly happen and because it is using undefined, i think that might be "unpossible".


fixed editing comment timestamp

What does this pull request aim to achieve? 💯
correctly display timestamp on edited comments
closes #1645

[x] I have checked that no one else is working on similar changes.
[x] I have read the latest README and understand the development workflow.
[x] The name of this branch is: feature/<branch_name>, or hotfix/<branch_name> if this is a hotfix, where <branch_name> briefly describes the issue(s) resolved and is prefixed with the issue number(s) (e.g. 1127-1130-update-git-branching-model). The branch is created in a cloned version of this repo, not a forked repo.
[x] This branch is rebased/merged with the latest staging (or the latest production if it is a hotfix).
[x] There are no merge conflicts.
[x] There are no console warnings and errors.
[x] On the right hand side of this PR, I have linked the appropriate issue(s) under "Issues" and added a project progress under "Projects".
[x] The title describes the issue(s) being addressed using format: <issue_numbers> - <brief_issue_description> (e.g. #1127/#1130 - Update Git Branching Model)

The edited comment for a post display both 'created and edited' time for the comment. This looks complicated to the end user. Can we display created time along with comment as 'edited'? Please refer below screenshot.

@rabbi08


Feature/offline mode

This is the first commit of the offline mode branch. It should contain the basic possibility to read emails offline. This closes issue #383.

Pull Request Test Coverage Report for Build 849251817

7 of 17 (41.18%)  changed or added relevant lines in 8 files are covered.
37 unchanged lines in 6 files lost coverage.
Overall coverage decreased (-2.3%) to 70.124%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/IMAPClient-Core.package/ICEndPoint.class/instance/connectAndLoginWith..st
0
1
0.0%

packages/IMAPClient-Core.package/ICFolder.class/instance/startUpdateProcess..st
0
1
0.0%

packages/IMAPClient-Core.package/ICFolder.class/instance/retrieveFlagsFrom.to..st
0
4
0.0%

packages/IMAPClient-Core.package/ICFolder.class/instance/update.st
0
4
0.0%

Files with Coverage Reduction
New Missed Lines
%

packages/IMAPClient-Core.package/ICEmail.class/instance/deleted.st
4
0%

packages/IMAPClient-Core.package/ICEmail.class/instance/deleted..st
4
0%

packages/IMAPClient-Core.package/ICEmail.class/instance/flagged.st
4
0%

packages/IMAPClient-Core.package/ICEmail.class/instance/seen.st
4
0%

packages/IMAPClient-Core.package/ICEndPoint.class/instance/retrieveFlagsFrom.to.in..st
8
0%

packages/IMAPClient-Core.package/ICEndPoint.class/instance/parseFetchedFlagsAndPopulate..st
13
0%

Totals

Change from base Build 199376006:
-2.3%

Covered Lines:
906

Relevant Lines:
1292

💛  - Coveralls

Is this WIP (Work in progress)? Two of the tests seem to fail at the moment.

Also, please never create a pull request into master. During the project, only the develop branch should be used!

Are there still failing tests?

No, they are all green now. :)


feat(publisher): Check if app port is open before publishing it. Fixes 3115

@glogiotatidis why use ExecStartPost like in deis-builder?.
With that it will appear as start-post in fleetctl list-units and not as started.
https://github.com/deis/deis/blob/master/deisctl/units/deis-builder.service#L12

@aledbf I'm not aware of this fleet functionality. I'd prefer thought that this goes to publisher so that Deis gets to have the final call and it's fleetd independent.

https://get.docker.io is currently down, so this may fail until we can merge #3162 and rebase it off that.

Rebased

Having publisher check that the app port is open before publishing it seems like a good idea. Code LGTM.

ooh, tests and everything! LGTM!

nice work, @glogiotatidis

Great!


Pictures please...

Im very stupid, and therefore cannot read the menu. Need pictures so i know what i will eat

I suggest https://pixabay.com/, free to use api if you create an account


Create custom size PDF

I suggest the following patch to enable custom PDF size.
The idea is that the format parameter can take an object with width and height properties.
Example:
            var pdf = new jsPDF({
                orientation: widthMm > heightMm ? "L" : "P",
                unit: "mm",
                format: {
                    width: widthMm,
                    height: heightMm
                }
            });

In that example, the format would be widthMm x heightMm. The units of the format are the PDF units. Then, my patch creates a new pageFormat with key "custom" and size translated to standard pt units.
Please find the patch below, it only affects a few lines of the original code:
162a163,181
> 		// JGUIX
> 		switch (unit) {
> 			case 'pt':  k = 1;          break;
> 			case 'mm':  k = 72 / 25.4;  break;
> 			case 'cm':  k = 72 / 2.54;  break;
> 			case 'in':  k = 72;         break;
> 			case 'px':  k = 96 / 72;    break;
> 			case 'pc':  k = 12;         break;
> 			case 'em':  k = 12;         break;
> 			case 'ex':  k = 6;          break;
> 			default:
> 				throw ('Invalid unit: ' + unit);
> 		}
> 		if (typeof format === 'object') {
> 			// Add pageformat in pt units
> 			pageFormats['custom'] = [format.width * k, format.height * k];
> 			format = 'custom';
> 		}
> 		// JGUIX
201d219
<
869,870c887,888
<
< 		switch (unit) {
---
> 		// JGUIX
> 		/*switch (unit) {
881c899,900
< 		}
---
> 		}*/
> 		// JGUIX

+1

+1

+1

+1

Try this:
var pdf = new jsPDF('p', 'in', [5,2]);

This will create a 5in x 2in PDF.

@Flamenco Thank you much! Couldn't find that in docs.

You can even set different sizes for every page you add:
var doc = new jsPDF('p', 'mm', [68,20]);
doc.addPage(100, 200);

Solved by PR #1590
Merged on 11 January 2018
Will be part of next version.

Thank you guys!

That would be so nice... Any update, when the next version will be available?

Does this still work ?


Erro na hora de criar pipeline via arquivo.json

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az synapse pipeline create
Errors:
The command failed with an unexpected error. Here is the traceback:
'properties'
Traceback (most recent call last):
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\knack/cli.py", line 231, in invoke
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 658, in execute
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 721, in _run_jobs_serially
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 692, in _run_job
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 328, in __call__
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/command_operation.py", line 121, in handler
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/command_modules/synapse/manual/operations/artifacts.py", line 82, in create_or_update_pipeline
KeyError: 'properties'

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

Put any pre-requisite steps here...
az synapse pipeline create --workspace-name {} --name {} --file {}

Expected Behavior
Environment Summary
Windows-10-10.0.22000-SP0
Python 3.8.9
Installer: MSI

azure-cli 2.34.1 *

Additional Context

route to CXP team

@MatheusBelo Thanks for reaching out to us and reporting this issue. We are looking into this issue and we will provide an update.

@MatheusBelo I wanted to do quick follow-up to check if you had a chance to look at my above comment. Please provide us the answer to the above questions. This will help us to assist you better. Awaiting your reply.

@MatheusBelo The action is currently pending on you to get back to us on the above queries. Please feel free to reopen this thread once you get a chance to look into this. We would be happy to help again.


ORM: Make connections optional

Is there currently a way to make a DB connection optional at sails lift time? Understandably the idea is to catch broken DB connections when the ORM hook loads, but we have a need where we want the application to load and report that the database connection is broken. Under the current scheme, the ORM hook blocks the boostrap process until the hook timeout fires, bringing the server down again.
A better example may be -- we're looking for a system where the application can come online and report periodically to a monitoring service the health of the instance, and the broken DB connection would be one such message we would send to the service.
Assuming that this is not presently supported, could this be supported in some form in the future? If I could listen for the ORM timeout event, I could send off any notifications that the DB is down, but that would be a one-shot deal. The server would be shut down after that regardless. Is there such an event for hook:orm:timeout or similar?

i'm also pretty new to sails, but what I'm planning to do on AWS is rely on AWS RDS monitoring as well as EC2 monitoring. Having the monitors be independent sounds a bit safer to me than relying on your application tier reporting db health. So for your example, I would see ec2 (sails) is down and look at AWS reporting to see if something is whacky with an RDS instance.  not an answer to your question by any means - just a different thought.


Proposed NickelMenu logo/icon

I'm a graphic designer and thought I'd design a logo for NickelMenu.  If you'd like to use it, I give it free and open source.  If you do, I can send (or attach) the original .eps or .svg file.

This is code from my Kobo, I hope it works as I no longer have a Kobo to test on.
experimental:menu_main_15505_label:NickelMenu
experimental:menu_main_15505_icon:/mnt/onboard/.adds/NickelMenuIcon.png


Skips symlink creation without asking me

Previously, when I ran script/bootstrap, I skipped all the symlink creations. Now, for some reason, when I run the script, it automatically skips all of them without asking me.
I want have the symlinks created automatically.
Thank you.

Sorry. False alarm. I did a Backup all.


Add get_adb_connection to get ADB connection info

been checking your documentation for Add get_adb_connection to get ADB connection info
sample command for it on how to use it
secondly the clicks by coordinates dont work well, because of the pop up ads from memuplay all the time

There is a search feature on the documentation website. Here's the link to the page you're looking for:
https://pymemuc.martinmiglio.dev/en/latest/pymemuc.html#pymemuc.pymemuc.PyMemuc.get_adb_connection

AttributeError: 'PyMemuc' object has no attribute 'get_adb_connection'

probably need to update the pymemuc module with pip install --upgrade pymemuc


Adding in some functionality to use different clouds

This support is needed for the air gapped environments.  There are three ways to add a new cloud environment.  These were all taken from examples in the v1 sdk.

The SKD will look for a default configuration file and try to find
cloud environments in there.
If you set an environment variable called ARM_METADATA_URL, it will
look there for cloud configurations.  If you do not set this, it will
use a default URL in the _azure_environments.py file to find them.
The SDK exposes two new functions, add_cloud which will add the new
configuration to the configuration file mentioned in number 1, and
update_cloud which will update the added configuration.

Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

Typo in item 1--SKD not SDK

@microsoft-github-policy-service agree company="Microsoft"


"Using build tools as Conan packages" example is misleading

Good evening
In this tutorial
Using build tools as Conan packages
it looks like required build tools will be installed on its own and only conan is required.
However, you need to install your own system wide cmake before you can execute
conan install . --output-folder=build --build=missing

otherwise it can compile zlib in your tutorial, complaing about that cmake is missing.
It is especially confusing because it states that
cmake/3.27.7: Appending PATH environment variable: /Users/user/.conan2/p/cmake253f38c8fbec3/p/CMake.app/Contents/bin

which looks like PATH env variable is update and will use cmake from conanfile.txt.
Couldn't figure out for some time, how it is supposed to work due to this line. Thought that PATH isn't updated for some reason.
In reality it leads to empty code which is just a stub for the future
https://github.com/conan-io/conan/blob/5955fdb87ee2f5e37aa3f8aea6e9c5ff62c494a7/conans/model/build_info.py#L46
And then you can build your own code with cmake from conanfile.txt.
But in order to do that you need to have system cmake in the first place.
If this is intended behaviour (albeit from user perspective it would be cool if conan just use cmake from conanfile.txt to build other dependencies) then please remove Appending PATH environment variable line. It is confusing as it makes user think that $PATH is updated with path to cmake, when in practice it doesn't

Hi @kuddai
Thanks for your feedback.
You are right about this. This tutorial assumed that there are pre-compiled binaries for the dependencies.
But if this is not the case, then zlib needs to be built from source, but cmake/xxx is a tool_requires only for the user project not for dependencies, and if you don't have cmake in the system for it, it will fail.
I agree we need to clarify this, adding a clear note that this is needed.
For your info, it is possible to easily use tool_requires for dependencies too, it is enough to add it to your profile like:
... # you can include(default) or define your own setttings
[tool_requires]
cmake/version

And then pass it as conan install ... -pr=myprofile. This will use the conan tool-require for building zlib too
Let's improve the explanations in the docs about this, thanks again!

Thank you for quick response!
I have modified profile with tool_requires as you mentioned and it worked!
The moment with "Appending PATH environment" was the thing which truly confused me. Given that it is quite common in for e.g. in python package managers to make mistake in PATH setup somewhere.
Spent ~2-3 hours pretty printing conan source code (great source code btw) trying to figure our why PATH isn't modified. And it is really hard to find the right place if it isn't modified :)

You can find that message not in the Conan source code, but in the cmake ConanCenter recipe, see: https://github.com/conan-io/conan-center-index/blob/ecd1df7d4659d5a8e3ac0ff0aa52dfbae30f89e9/recipes/cmake/binary/conanfile.py#L65.
This is mostly legacy from Conan 1.X, and it will be removed from the recipe when (it will take some time, not soon) Conan 2.0-only recipes are in ConanCenter.  Furthermore in Conan 2.0 it has absolutely no effect, and PATH is not modified until the consumer activates its conanbuild virtualenvironment (in self.run() and other commands)

Hi @kuddai,
Thanks a lot for reporting. I'm transferring this to the docs repo and adding a note there to clarify.


Revert "buildPython*: wrap setuptools in hook for catching conflicts"

This reverts commit 13bb0f49f7afe8296452d70e7782ac1d67b064da.
The outfall of this change is much bigger than expected and will be fixed in #191333
Description of changes

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

OK, I suppose we're going forward.  It's not ideal, but adding setuptools into individual packages is fairly easy at least, so I expect that will progress faster than Hydra.


fix(cli): correct package import

Changes

Checklist

[x] the list of features is updated (if it's a feature and it's relevant)
[x] I followed contributing guidelines

🤔 do you have the same output building CLI locally?
https://github.com/vuestorefront/shopware-pwa/pull/1391/checks?check_run_id=2093617614#step:7:53


Incorrectly duplicates errors to console

http://hastebin.com/ijoxajajol.avrasm

http://hastebin.com/oturugayec.avrasm

This also caused a lot of lag on my server.

Are the players still able to join?
Because I had this issue before and it made it impossible for any players to join.

I'll look into modifying our error handler again to see if we can optimise it so that we don't get this spam. We just don't want to lose proper errors.

Should be fixed in next version, I've commited the fix.


FCL SDK: Android

Outblock would like to build this.

For visibility, the issue created by Outblock: https://github.com/onflow/developer-grants/issues/21


Make Chart's IsRoot available to templates

What this PR does / why we need it:
This PR closes #9991 as it adds the information if the currently processed (templated) Chart is the root or not. This allows to have different behavior in case a Chart is installed on its own or included as part of another Chart.
Templates can check this flag like so:
IsRoot: {{ .Chart.IsRoot }}

Special notes for your reviewer:
This is my first PR to Helm so please let me know if I'm missing something
If applicable:

[ ] this PR contains documentation
[ ] this PR contains unit tests
[ ] this PR has been tested for backwards compatibility

Hi @technosophos ,
I've added a test case to cover both cases - IsRoot false and true.
Let me know what you think.


Import Bot?

So, we have our internal monorepo syncing out to our different OSS projects -- yay! It's working great.
Now, the question of course is how do we accept PR's? It seems to be that most, if not all, of the fbshipit tooling could be used to build functionality like "@facebook-github-bot import".
I'm curious about two things:

is the fbshipit code used internally to build the import/shipit bots?

Do you have any thoughts on how we might handle the case where we've moved files from an internal __github__ folder into the root directory of the external repo during the outward sync, but on the inward sync those files need to be reconciled back to their proper place internally? An example of this in fb OSS seems like it would be the Relay repo. There's a moveDirectories filter that does self::RELAY_ROOT.'/__github__/' => ''. What way have you guys found to reconcile that when doing an import and generating a diff for Phabricator?

Curious to hear your thoughts. I want to build this! Manually building patches from PR's is for the birds :-)
Thanks!
/cc @fredemmott

At the moment, it's an entirely separate system, though we'd like to change this.

Some projects explicitly list the files, though the most common approach is probably manually applying these patches.

I too am exploring how we can use fbshipit to sync internal projects from our private monorepo to public open source repos. (It seems a bit odd to have such a one-way tool atm but maybe I'm missing the point!)
I was wondering if, since fbshipit seems to have first-class support for writing to git & github, both of my repos live within github (they do - one private, the other public) I could theoretically have two shipit scripts which independently sync each way?
Would fbshipit be able to detect the, potentially interleaving changes? We could even avoid that problem by always running both scripts together (in some "sync" event) so on each side you collect changes since last sync and then apply diff in each direction.
@fredemmott @skevy

ShipIt does not support two-way sync, and it's not something we're particularly likely to work on; a single source of truth simplifies a huge number of problems.
That said, if you make it possible to change the tracking line added to commit messages and use a different one for each direction, skip if one's already present for the direction you're already going in, it will basically work - but it would probably be extremely time-consuming to debug and fix if the sync breaks in one direction only at some point and you keep pushing out patches in the other.
That said, it would be reasonable to use this approach if pull requests are merged to the internal repo instead of trying to have a true two-way-sync, i.e:

Pull requests could either be directly pushed by shipit to your internal repo (assuming it's Git or you send a PR for #1 :p):

have new shipit script with github as source, internal repo as dest
run shipit with --source-branch=pull/PR_NUMBER/head (https://help.github.com/articles/checking-out-pull-requests-locally/) and either:

do the differing identifier lines mentioned above
run with --skip-push, and with some other script, export the patch, amend it to remove the identifier, and land to your internal repo as normally

you'll want your filter function, an additional phase, or a separate script to add 'fixes #nnnn' to the commit message so that github closes the PR when it lands

I'm actively working on this internally as we speak.  Expect the bulk of the code to land today or tomorrow.

Commits that have already landed:

https://github.com/facebook/fbshipit/commit/0b11437cb5c85368feac836bef009ded90ae38b8
https://github.com/facebook/fbshipit/commit/c91a1d827229a52fe743452d6bbca703a41acfd4
https://github.com/facebook/fbshipit/commit/8930077fd4f256ba598f254bf16dbdf4dabe8e83


There is no Intellisense whatsoever in C#

How can you possibly comment that this is a C# editor if there's no intellisense whatsover?
http://d.pr/i/12NZq+

That is not planned behavior. Could you post the sublime text logs, as well as some information on what system you are running?

Steps to reproduce:

install sublime
install this package
open a single .cs and check if there's intellisense.

Hmm, could you post the sublime text logs, as well some system information?

As if this would make any difference what version of mac os I am running

It might, the plugin has platform specific binaries it calls. Also could you get me your version of mono?

Did you even try simulating the problem? Literally all you have to do is right click a .cs file, open with Sublime and you'll see that there's no intellisense.

Yes, I have. And guess what, it works. That's why I need some basic information such as logs, and system information. I wan't to fix it, but without that information I can't actually do anything.

Im pretty sure it doesn't work, you probably have the .cs file opened as part of a project. I'm done with this thread, bye.


SearchContextMissing In ES

Please use the following questions as a guideline to help me answer
your issue/question without further inquiry. Thank you.
Which version of Elastic are you using?
[ ] elastic.v7 (for Elasticsearch 7.x)
[X] elastic.v6 (for Elasticsearch 6.x)
[ ] elastic.v5 (for Elasticsearch 5.x)
[ ] elastic.v3 (for Elasticsearch 2.x)
[ ] elastic.v2 (for Elasticsearch 1.x)
Please describe the expected behavior
I was previously using Elastic 6.2 version (single machine) with the GO elastic module of elastic 5.X
And everything was working fine till then.
As I have upgraded the Elastic search version to 6.7 (now 4 data node and 1 master node every on a different machine in a cluster) with same elastic 5.X. I am getting following errors with elastic search and not able to serve the Result using ES query
My code works fine for 2-3 hours first and then everything collapse.
ES error
[1344693] Failed to execute query phase
org.elasticsearch.transport.RemoteTransportException: [elasticsearch-6-7-XXX-XX-XX-XX][172.XX.XX.XX:9300][indices:data/read/search[phase/query/scroll]]
Caused by: org.elasticsearch.search.SearchContextMissingException: No search context found for id [1344693]
        at org.elasticsearch.search.SearchService.getExecutor(SearchService.java:496) ~[elasticsearch-6.7.2.jar:6.7.2]
        at org.elasticsearch.search.SearchService.runAsync(SearchService.java:373) ~[elasticsearch-6.7.2.jar:6.7.2]
        at org.elasticsearch.search.SearchService.executeQueryPhase(SearchService.java:435) ~[elasticsearch-6.7.2.jar:6.7.2]
        at org.elasticsearch.action.search.SearchTransportService$8.messageReceived(SearchTransportService.java:376) ~[elasticsearch-6.7.2.jar:6.7.2]
        at org.elasticsearch.action.search.SearchTransportService$8.messageReceived(SearchTransportService.java:373) ~[elasticsearch-6.7.2.jar:6.7.2]
      at org.elasticsearch.transport.RequestHandlerRegistry.processMessageReceived(RequestHandlerRegistry.java:66) ~[elasticsearch-6.7.2.ja
r:6.7.2]
        at org.elasticsearch.transport.TcpTransport$RequestHandler.doRun(TcpTransport.java:1087) ~[elasticsearch-6.7.2.jar:6.7.2]
        at org.elasticsearch.common.util.concurrent.AbstractRunnable.run(AbstractRunnable.java:37) ~[elasticsearch-6.7.2.jar:6.7.2]
        at org.elasticsearch.common.util.concurrent.EsExecutors$DirectExecutorService.execute(EsExecutors.java:192) ~[elasticsearch-6.7.2.jar
:6.7.2]
        at org.elasticsearch.transport.TcpTransport.handleRequest(TcpTransport.java:1046) [elasticsearch-6.7.2.jar:6.7.2]
        at org.elasticsearch.transport.TcpTransport.messageReceived(TcpTransport.java:932) [elasticsearch-6.7.2.jar:6.7.2]
        at org.elasticsearch.transport.TcpTransport.inboundMessage(TcpTransport.java:763) [elasticsearch-6.7.2.jar:6.7.2]
        at org.elasticsearch.transport.netty4.Netty4MessageChannelHandler.channelRead(Netty4MessageChannelHandler.java:53) [transport-netty4-
client-6.7.2.jar:6.7.2]

This clearly says that there is a problem with the ES scroll.
So I thought maybe there is a version compatible issue with the current ES 6.7 and ES go module 5.X
So I have updated the ES go module to 6.X.
Now I have ES version of 6.7 with ES Go lib version to 6.X.
My code and query related is

q := elastic.NewTermQuery("ABC", 1)
	bq = elastic.NewBoolQuery().Filter(q)
	bq = bq.MustNot(elastic.NewTermQuery("wrong", "XXX"))
        bq = bq.Must(elastic.NewTermQuery("ID", 12132))

query = bq

hits := make(chan json.RawMessage)
g, ctx := errgroup.WithContext(context.Background())
g.Go(func() error {
	defer close(hits)
	scrollService := elastic.Scroll(Index).Type(Type).Query(query).Size(250)
	for {
		results, err := scrollService.Do(context.TODO())
		if err == io.EOF {
			log.Println("In for EOF")
			return nil // all results retrieved
		}
		if err != nil {
		 // This error occurs after some time and logs the error
		 //[Elastic Error: elastic: Error 404 (Not Found): all shards failed //[type=search_phase_execution_exception]
			log.Println("[Elastic Error:", err)
			continue
		}

		// Send the hits to the hits channel
		for _, hit := range results.Hits.Hits {
			select {
			case hits <- *hit.Source:
			case <-ctx.Done():
				return ctx.Err()
			}
		}
	}
})

for i := 0; i < 100; i++ {
	g.Go(func() error {
		for hit := range hits {
			err = json.Unmarshal(hit, &XXX)
			if err != nil {
                log.println("xxx)
				continue
			}

			My processing is here !!!

			select {
			default:
			case <-ctx.Done():
				return ctx.Err()
			}
		}
		return nil
	})
}

// Check whether any goroutines failed.
if err := g.Wait(); err != nil {
	log.Printf("Go Routine Err:%s", err.Error())
}

### Any steps to reproduce the behavior?
The code works fine for 2-3 hours and after that, it throws error a lot

You typically get this error when the token that gets passed for the scroll operation times out. Can you check and see if you can set a longer timeout.

how do i set a a longer timeout ?

See https://github.com/olivere/elastic/blob/release-branch.v7/scroll.go#L101

See https://github.com/olivere/elastic/blob/release-branch.v7/scroll.go#L101
@olivere
Added KeepAlive to 10m
but still not able to remove the error

Everything I read about SearchContextMissingException on the internet says that you’re trying to access a „cursor“ that has expired or doesn’t exist any more in the ES cluster. I don’t know why that happens though. It may of course be a bug in the library, but it’s been in use for years, so I guess that should’ve been uncovered in the recent years. is there any chance you can reproduce the problem with a dataset? Do you delay work while scrolling sometimes?

yes, there is a delay but that's too in microseconds.

But the question is why this is a problem when I switch to the ES cluster having 4 data nodes.
while this code works fine in a single elastic machine with 1 node having the same documents
Is there any problem with scroll in version 6.7
bcoz everything works fine in version 6.3

I don’t know if any issues. I think Scroll API didn’t change significantly since the 3.x to 5.x update.


Missing file

In the Pytorch version, requirement.txt is missing. It only exists in Tensorflow version.

Hi @kernelizd
Sorry I had forgotten. This has been added now.


Convert bool values from a .ini file

When I use a .ini file for configuration, some variables are passed to bravado-core in the form of str and always treated as True.
Since all variables except formats are boolean, they should be converted using asbool(): https://bravado-core.readthedocs.io/en/stable/config.html
The same thing happens for enable_api_doc_views in __init__.py, and thus I fixed.

Coverage decreased (-99.6%) to 0.0% when pulling cbe5f57b59547179710c6b54a0e6104c773bf944 on hiromu:convert-bool-values-in-ini-file into 2c8d37f1727b61d2be4218cbe52e13198bbc5d59 on striglia:master.

I understand your concerns on the forward compatibility.
Are you considering to implement an API within bravado-core? Otherwise, I'll update this pull request so as to use a hard-coded allowlist to determine whether to apply asbool.

how were you specifying config values that you ran into this issue?

In my implementation, I'm specifying bravado_core.internally_dereference_refs = false in environment.ini and just calling config = Configurator(settings=settings); config.include('pyramid_swagger') in main(global_config, **settings) of Pyramid.


Add collaborators to this repo

Comment below to be added as a collaborator
We will all be using this repository today. Even though it's public right now, you will not be able to make changes until you're given the correct permissions. We've automated this process via the GitHub API. Once you comment, we will add you as a collaborator.
You'll start to receive a lot of emails. (❗) You can slow these down immediately by clicking the Unwatch button at the top of the page and selecting either the Participating and @mentions or Ignore option.

Hello!

Hello!

Hello!

Hello!

Hello!

Hello from student!

Hi

Hello from Shruti !

Hello!


State that Face is required if Position is set.

Resolves issue  #515.

Tweaked this myself but thank you for the PR!


Migrate to new reporter.

Short description of what this resolves:
Added new reporter integration
Changes proposed in this pull request:

Added new reporter integration
Updated mvn dependencies to use latest stable versions.

Note: this build is depends on cube SNAPSHOT dependency.
Fixes: #25

[ ] Resolve merge conflicts
[ ] Update Cube

How about the versions in the main pom - shouldn't be the version updates part of another commit/PR? Or do we somehow depend on the updates in this PR?

@MatousJobanek We depends on new version of assertJ & Cube for this PR.

OK, cool. Then LGTM :-)
Thanks @dipak-pawar


not working on Esp 12F module

I have tried on NODECU 12E, its work like charm, but when i tried it on esp 12f , only getting a garbage values on serial out at 115200 BAUD. Tried downlaod esp_init_data_default bin file at address 0x7fc000 and 0xfc000. but did not worked.
Please help.

On a 4MB module the init_data has to go to 0x3fc000.

Tried with DIO mode and its works like charm for both esp12f and webmos D1.
Thanks a lot.


Allow users to have complex logic in templates

Fixes #56, #64

Looks great! You've definitely overhauled my little contribution huh? 😂😅
Just a thought, for completeness, there should probably be a power (a**b) and modulo (a mod b) operator in the maths helper, then pretty much every kind of maths you might want to do is covered!

note to self - can probably add https://github.com/joplin/plugin-templates/pull/69 too.

Thanks for implementing this! Any documentation on how to use it?

Hi @dniku, this feature will be released with the next version of the plugin (sometime around end of November).

Sorry to add noise, but are there any plans for a new release?

Hi @q-wertz, have been super busy lately, will try to get this out this month.

No problem 😊
Thanks
And happy new year 🚀

Sorry for annoying. Any news on a new release? :see_no_evil:

I'm sorry we lost attention, GSoC keeps us somehow busy.
I mentioned it again towards those who has the permission to release the new version (I'm currently don't have)

Thanks a lot 🙏😌
Have fun at GSoC :blush:

👻😿

Any news regarding the update? I would be very interested :)


⚠ Bofrost.at has degraded performance

In b8cda48, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1368 ms

Resolved: Bofrost.at performance has improved in 95dc3f2 after 5 minutes.