Sent UST from BSC to TERRA using Terra Bridge and still have not received it

Transaction Hash: 0x0cb510813e45d2f3a401515b0fc2e9b0bcb1316d02c1857018f8ec15c6e9fae1

I sent UST (BEP20 Wormhole) through the Terra Bridge (terra.bridge.money) from BSC to Terra over 6 hours ago and still have not received it. Can someone please look into this and help? Thank you.

Hello @DEGV35   , A support ticket has been opened for you with Support ticket number TB7833HH. You should talk to a Terra money Dev on the [Live support page]. So that your transaction and issues can be looked into. Thank you. [CLICK HERE]

Transaction Hash: 0x0cb510813e45d2f3a401515b0fc2e9b0bcb1316d02c1857018f8ec15c6e9fae1
I sent UST (BEP20 Wormhole) through the Terra Bridge (terra.bridge.money) from BSC to Terra over 6 hours ago and still have not received it. Can someone please look into this and help? Thank you.

Hello, A support ticket has been opened for you with Support ticket number BN36YSH. You should talk to a Terra money Dev on the LIVE SUPPORT PAGE So that your issue can be looked into. Thank you. CLICK HERE

Hello @DEGV35,
Please write to support : Terra ecosystems support  for support requests and swift assistance.

Transaction Hash: 0x0cb510813e45d2f3a401515b0fc2e9b0bcb1316d02c1857018f8ec15c6e9fae1
I sent UST (BEP20 Wormhole) through the Terra Bridge (terra.bridge.money) from BSC to Terra over 6 hours ago and still have not received it. Can someone please look into this and help? Thank you.

Kindly reach out to Terra Ecosystems Support Team with the ticket id LUN7478 using the livechat page, the best team of dev will assist you on the issue immediately.


Support ES Modules, and by extension, Deno and web browsers

Node.js has ES Modules feature turned on by default, albeit still experimental. Once Node.js can use ES module syntax to load js-yaml, https://pika.dev can convert js-yaml to something web browsers and Deno can consume (alternatively, snowpack can be use to create web modules).
This change would require rewriting almost all code to modern import syntax, I am willing to work on a pull request.

Hi! JS syntax change is not a big deal, and i have no principal objections about es6. The most boring thing is to change development process. If anyone can help, that would be very useful. After change to es6, this topics should be covered:

Backward compatibility. Still should support commonjs & old node.js.
Drop ./dist. Need better way to automate commonjs browser builds. May be some CDN can do that?
How to publish both versions with minimal efforts?

Should we provide consistent cjs version on github or in npm only?

Simple development. Preferably, without additional monitoring tools and compilers.

If you know "best practices" about topics above (and anything else i could miss) - let me know.

Backward compatibility. Still should support commonjs & old node.js.

Babel can compile them

Drop ./dist. Need better way to automate commonjs browser builds. May be some CDN can do that?

You may not need to commit ./dist to git, but you should still publish it to npm, there're countless npm-based cdn (unpkg, pika.dev, jspm.io to name a few). Babel can take care of this too.

How to publish both versions with minimal efforts?

Should we provide consistent cjs version on github or in npm only?

Write code in ES Module syntax, use Babel to transpile source code to CommonJS, publish both of them. In package.json, set "main" to CommonJS file and "module" to ES Module file.

Simple development. Preferably, without additional monitoring tools and compilers.

We would need at least Babel.

Concept "babel will help" is too generic. I mean a more concrete recipes - content of package.json and additional scripts if required. Ideally - examples of well-known project, useable as template.

I mean a more concrete recipes - content of package.json and additional scripts if required

I can create a simple repo for you.

Ideally - examples of well-known project

Take a look at Preact, it supports CommonJS, ES Modules, and UMD.
There's also iter-tools, it's simpler, it supports CommonJS, ES Modules, and TypeScript.

useable as template.

I can convert this very repo for you (after I have converted the code to using ES Module syntax).

Update: I have created a simple repo with full explanation. I have also published a package from that repo. It is supported by Pika's CDN so you can try import it from the browser using ES module syntax.

May be i'm too old, i don't understand idea to transpile everything.

Why should we transpile es6 code instead of use native one?
Who cares es5 build will be in sync on npm publish?

Why should we transpile es6 code instead of use native one?

Because, ES6 is the standard and will probably be succeded by ES7. Having code in newer standards enables newer contributors to add more value to project. Not many who have learnt JavaScript after classes came to be can work with prototypes.

Who cares es5 build will be in sync on npm publish?

If we are testing new code, might as well keep the old one in sync with any added features.

Because, ES6 is the standard and will probably be succeded by ES7. Having code in newer standards enables newer contributors to add more value to project. Not many who have learnt JavaScript after classes came to be can work with prototypes.

I prefer KISS principle. This project is stable and has nothing to extend with. I don't like to overcomplicate things for uncertain features in future (that will never happens in real world).
If es6 code can't be used native, it's more simple to stay on es5 and organize simple wrapper.

If we are testing new code, might as well keep the old one in sync with any added features.

Please, understand me right. Changing syntax to es6 is trivial. The real problem is change of process. And all questions like this need to have concrete answers before start.

I prefer KISS principle.

KISS principle was never possible in JS world, it's not possible then, it's not possible now.
Before ESM, people have to use various inconsistent ways to bundle dependencies (CommonJS, UMD, etc), you may think that just adding a <script> tag is simple, but Node.js has no <script> tag, so you would have to create a CommonJS version.
Now with ESM, things can be simpler, you can drop legacy support entirely (seriously, semver allows you to do that without fear of breaking things for other people) and people that care can still support old browser if they want to (by using Babel). But if you don't want to increase major semver, then you would have to transpile yourself, making development more complex. As for me, I am fine either way.

I don't like to overcomplicate things for uncertain features in future (that will never happens in real world).

ES Module is a very certain and stable past for at least 2 years now. It has a standard. All major browsers support it (except IE). It will not stop working.

If es6 code can't be used native, it's more simple to stay on es5 and organize simple wrapper.

I can run ECMAScript 2020 (and by extension, es6, which can be considered old at this point) in the Chrome browser, right now, without wrapper.
Not to mention, ESM allows developer to use tree-shaking to reduce bundle size.

You tell well known things, but i still don't understand why es6 should not be used natively in this project (with es5 transpiled by babel).

but i still don't understand why es6 should not be used natively in this project

You mean for the tests? The tests are already in old syntax, I don't want to touch it if it for now. But if you want to change it to es6 too, it's fine.

You mean for the tests?

I mean for everywhere where import directive needed.

The tests are already in old syntax, I don't want to touch it if it for now.

 IMO it's more rationale to plan new approach and to do all at once.

You mean for the tests?

I mean for everywhere where import directive needed.

Node.js support ES Modules now. But I'm not sure if dependencies and test framework this repo uses support ES Modules out-of-the-box.

 IMO it's more rationale to plan new approach and to do all at once.

So you don't want to minimize code change? Fine, but we may need to add transpilation plugins for reason above.

But I'm not sure if dependencies and test framework this repo uses support ES Modules out-of-the-box.

I'm 100% sure, after node.js officially enabled modules all popular dev tool will support those very soon. If there is lack of support now, the most obvious solution is wait for 1-2 months, doing nothing :)

So you don't want to minimize code change? Fine, but we may need to add transpilation plugins for reason above.

No, i have no goal to minimize code change. I have goal to get new comfortable development process (with import support and fallback to old way). And IMO using es6 native (with transpilers for es5 only) will be more comfortable than transpiling everything.
Under es6 i mean version with import support, and anything supported before. For example i don't mean classes must be used, but those can be used if needed - because if js engine supports import, it supports classes too.

hello. i can't use this library in my modern es-module-only browser apps
does anybody know of an alternative which distributes es modules?
authors, would you be willing to accept a pull request which

made no breaking changes
minimized changes to this repository
maintained dist/
added dist-esm/

:wave: thanks

hacky es module build now available!

js-yaml-chase-esm/dist/js-yaml.esm.js

if you need a working esm build of js-yaml, feel free to use this
published under npm module js-yaml-chase-esm
it's just a simple hack using browserify
i probably should have minified it.. i'll try to remember to update this comment if i ever get around to that..

greetings! for my own purposes, i've created a quick esm fork. here are the changes and my explanation

modified package.json — rename package to js-yaml-chase-esm
modified dist/

add js-yaml.esm.js — es module exports whole jsYaml module as an object
add js-yaml.esm.d.ts — typescript declaration, but only for jsYaml.safeLoad

add esm/

after.js — one-liner exports jsYaml object
before.js — one-liner initializes jsYaml variable
entrypoint.js — one-liner browserify entry imports and sets jsYaml variable
build — bash script to generate dist/js-yaml.esm.js
index.html — quick html console.log test to check the es module works

if you clone the fork, you can run the build with npm i -s && ./esm/build
i do not think this should be merged into js-yaml! it's too silly and hacky. instead, my recommendation would be a typescript rewrite. perhaps an ambitious junior-level developer looking to contribute on open source projects would be up for the task
    :wave: chase

Fixed by https://github.com/nodeca/js-yaml/commit/1b5fea8526fea734041334db3687ae5a919a0dfc


[feat] API: Reverse Replication Create workflow

Design Doc
The reverse replication create workflow does the following:

Create a gcs bucket if needed
Create a change stream if required
Create a metadatadb for reverse replication if required.
Launch Reader dataflow job
Launch writer dataflow job

The entry point of the API is via create.go, where each of these steps is denoted via an Activity.  Each of these activities implement the interface in IActivity.go.
Other major changes include:

Introduction of accessors (like dataflow accessor, spanner accessor, storage accessor) and singleton clients. All functions that access a client should from now on go through here.
Introduction of DAO which is used to interact with the SMT metadata tables.
Zap logger for the reverse replication workflow.

TL;DR - Please break up this PR.
Please break up this PR into separate logical stacked PRs.  I will suggest breaking this up into atleast 4+ PRs, something like this -

One Accessor implementation
Rest of the accessor implementation
Activity interface and one implementation
Rest of the activities
Workflow
Changes to web layer (I see some changes made in web.go)

This will also help you write code iteratively and test as you go, as you write tests for each PR.
A good rule of thumb for PR size is less than 1000 lines or 10 files - whichever comes first.
https://thestartupfactory.tech/pull-request-best-practices
Stacked PRs blogs -

https://blog.logrocket.com/using-stacked-pull-requests-in-github/
https://timothya.com/blog/git-stack/


Start-PSPester should default to running CI and Feature tests

We've seen a few cases where an engineer runs start-pspester and it passes but it breaks our nightly runs as the Feature tests weren't run.  We should change the default of start-pspester to run both CI and Feature tests and update our CI scripts to explicitly pass -Tag CI.

fixed by #4273


Did not found any category with collection name: Summer-BBQ/Hidden-Category

I installed Shopware and Nextjs commerce projects locally but when I'm visiting the home page I get this error in logs
[useListing][search] Did not found any category with collection name: Summer-BBQ/Hidden-Category
[useListing][search] Did not found any category with collection name: Summer-BBQ/Hidden-Carousel-Category

I added categories Hidden-Category and Hidden-Carousel-Category but it still isn't working

BTW the Nextjs app is running in the broswer:

Currently, it is hardcoded in:
components/carousel.tsx
and
components/grid/three-items.tsx
You can change it to your needs:
  const collectionName = isSeoUrls()
    ? 'Summer-BBQ/Hidden-Carousel-Category'
    : 'ff7bf3c59f1342a685844fbf8fdf9dc8';
  const { products } = await getCollectionProducts({
    collection: collectionName
  });

It would be best if you matched the seoURL or the category UUID depending on whether you use seoURLs.
Check the correct seoURL in the default Storefront and copy it.


https://rdp.codeplex.com/workitem/2

Parser.cs
Line 145
if ( token.Key == "emdash")
{
CheckBuffer();
myStr.Append('¡ª'); // <_ not a valid Character
return true;
}
assumed the emdash character changed to
myStr.Appsed('\u2014');
This compiled but got two more of these

Recompile << now have two errors in
RTFNodGroup.cs
Line 176
Line 184

if( node.Keyword == "emdash")
{
AddString( myStr , buffer );
// notice!! This code may cause compiler error in OS which not support chinese character.
// you can change to myStr.Append('-');
myStr.Append( '¡ª'); // <- Line 176
continue ;
}
if( node.Keyword == "" )
{
AddString( myStr , buffer );
// notice!! This code may cause compiler error in OS which not support chinese character.
// you can change to myStr.Append('-');
myStr.Append( '¨C' ); // <- Line 184
continue ;
}
}

Fixed by changing encoding to UTF-8
https://github.com/NikolayIT/RtfDomParser/commit/e9bf42b0023641e74bb9b3365a10e8007a0e81b2


Need to define both M and Measure in QuantumSimulatorBase

Describe the bug
There is two separate methods for measurement in QuantumSimulatorBase.cs that seem to be redundant? M is just a special case of Measure and needing to have two separate methods to fully implement the IQuantumProcessor interface seems like it could lead to problems. Not obviously actively causing problems at the moment, I just noticed this while working on writing a new simulator for stabilizer subtheories:  https://github.com/crazy4pi314/chp-sim.
Expected behavior
I would expect that the interface only require me to implement the Measure method and then make M a shortcut/special case of Measure internally.

Hi @crazy4pi314! I believe @bettinaheim is following up with you separately, but just to summarize our thoughts:
We agree that needing to implement both Measure and M should be unnecessary, although it's not necessarily the case that M should be implemented in terms of Measure. Measure can also be decomposed into M and other gates, and which one should have the explicit implementation could depend on the processor architecture.
It would be nice to have some way to provide default implementations of both Measure (in terms of M), and M (in terms of Measure), and require processors to implement one (but not necessarily both). The current system may not be flexible enough for this - @bettinaheim would know more.

Note quite there yet, but we are moving towards facilitating target packages specifically for the quantum instruction set used by a particular target: https://github.com/microsoft/qsharp-runtime/pull/421


Fixed broken Azure & AWS documentation links.

High-level description
Fixing the Azure documentation link as the documentation site has been reorganized recently.

@mesosphere-mergebot bump-ee

@mesosphere-mergebot label ready for review

Re-running teamcity/dcos/test/azure/arm as there were a bunch of errors for test_networking.py & the build hung.

@mesosphere-mergebot override-status teamcity/dcos/test/azure/arm https://jira.mesosphere.com/browse/DCOS_OSS-2115

@mesosphere-mergebot override-status teamcity/dcos/test/azure/arm https://jira.mesosphere.com/browse/DCOS_OSS-2115

@mesosphere-mergebot merge-it


"nyc report --reporter=lcov" doesn't do anything

Please use the template provided below, when reporting bugs:
Expected Behavior
Something output or something happen
Observed Behavior
Nothing happens (0 exit code)
Bonus Points! Code (or Repository) that Reproduces Issue
Forensic Information
Operating System: macOS Sierra
Environment Information: I ran nyc npm test, my test script is NODE_ENV=test babel-tape-runner test/**/*.test.js. The output from that command was:
(after test results)
--------------------------------|----------|----------|----------|----------|----------------|
File                            |  % Stmts | % Branch |  % Funcs |  % Lines |Uncovered Lines |
--------------------------------|----------|----------|----------|----------|----------------|
All files                       |    59.47 |    41.38 |    49.17 |     63.1 |                |
 src                            |    89.58 |       50 |    92.86 |    92.68 |                |
  index.js                      |    90.63 |       60 |    91.67 |    96.15 |             13 |
  load-conf.js                  |     87.5 |     37.5 |      100 |    86.67 |        186,189 |
 src/file-stores                |    42.86 |        0 |      100 |    42.86 |                |
  index.js                      |    42.86 |        0 |      100 |    42.86 |      8,9,10,12 |
 src/file-stores/instagram      |    13.48 |        0 |    11.11 |    13.89 |                |
  index.js                      |    13.48 |        0 |    11.11 |    13.89 |... 220,221,240 |
 src/migrations                 |      100 |      100 |      100 |      100 |                |
<PHONE_NUMBER>5417_file.js        |      100 |      100 |      100 |      100 |                |
 src/models                     |     87.1 |       50 |       75 |     93.1 |                |
  base-model.js                 |       80 |       50 |    71.43 |    88.89 |          13,26 |
  index.js                      |      100 |      100 |      100 |      100 |                |
 src/models/file                |    98.71 |    85.71 |      100 |    99.28 |                |
  compute-hashes.js             |      100 |      100 |      100 |      100 |                |
  compute-picture-dimensions.js |      100 |      100 |      100 |      100 |                |
  index.js                      |    98.55 |       85 |      100 |    99.21 |             33 |
 src/resources                  |    60.47 |    45.45 |    66.67 |    60.71 |                |
  external.js                   |    22.62 |        0 |       15 |    21.92 |... 126,128,129 |
  files.js                      |    69.51 |    53.13 |      100 |    68.49 |... 5,96,97,105 |
  healthz.js                    |      100 |      100 |      100 |      100 |                |
  index.js                      |      100 |      100 |      100 |      100 |                |
  tus.js                        |    92.11 |    83.33 |      100 |    90.91 |       48,49,50 |
  upload.js                     |    82.93 |    66.67 |    83.33 |    86.49 | 12,54,59,60,72 |
 src/resources/transform        |    54.42 |    37.26 |       60 |    54.43 |                |
  index.js                      |    89.47 |    66.67 |      100 |    87.88 |    11,12,55,60 |
  peg.parser.js                 |    52.22 |    35.86 |    54.55 |    52.51 |... 3,1034,1037 |
 src/routes                     |      100 |       75 |      100 |      100 |                |
  index.js                      |      100 |       75 |      100 |      100 |                |
 src/services                   |    89.69 |     61.9 |    90.91 |    90.24 |                |
  hashids.js                    |      100 |      100 |      100 |      100 |                |
  index.js                      |      100 |       75 |      100 |      100 |                |
  store.js                      |    85.51 |    60.53 |    86.67 |    86.21 |... 7,85,95,133 |
 src/transform-tasks            |    74.84 |       50 |       80 |    76.52 |                |
  cache.js                      |    94.44 |      100 |       80 |    92.86 |             28 |
  index.js                      |    81.03 |    69.57 |    83.33 |    84.69 |... 193,205,206 |
  urlscreenshot.js              |    23.81 |        0 |        0 |       25 |... 49,51,53,54 |
 src/transform-tasks/gm         |    35.31 |    27.27 |     7.33 |    49.58 |                |
  collage.js                    |    27.69 |        0 |    15.79 |    29.41 |... 85,88,90,93 |
  index.js                      |     35.4 |    33.33 |     4.69 |    53.76 |... 436,438,439 |
  stream-cb.js                  |    66.67 |       50 |    66.67 |       75 |       14,15,16 |
--------------------------------|----------|----------|----------|----------|----------------|

Everything seems fine so far. I'm not using babel-plugin-istanbul (what does it do exactly? I'm using a bunch of babel plugins but nyc didn't seem to have trouble with my code...)
I then ran nyc report --reporter=lcov but it doesn't output anything.

run the following script:

sh -c 'node --version; npm --version; npm ls' > output.txt

share a gist with the contents of output.txt.

v6.6.0
3.10.3
...

Oh, I just noticed that created a coverage/ directory.


chore: updating code owners to team

I don't have strong feelings either way, but I think I recall @skjindal93 had moved some repos away from team ownership, so might want to discuss that first.

Team base approach makes it easy to add/remove members, isn't it? Or that's not the case?

I don't have strong feelings either way, but I think I recall @skjindal93 had moved some repos away from team ownership, so might want to discuss that first.

Team base approach makes it easy to add/remove members, isn't it? Or that's not the case?

Just that team based approach needs admin to add new code owners to the team
It becomes easy otherwise to update code owners, if they are just listed out in the CODEOWNERS file. Anyone can raise a PR to add them

Just that team based approach needs admin to add new code owners to the team

Not a global or repo admin though, a team maintainer. So if that's the approach we want, we could choose to make all (or just some) members of each team as maintainers.

Just that team based approach needs admin to add new code owners to the team

Not a global or repo admin though, a team maintainer. So if that's the approach we want, we could choose to make all (or just some) members of each team as maintainers.

I am fine with either approach, as long there is an easy way to add more code owners, and seems like there is


Pd-Skeleteon-Loader

Low Prio
We have a skeleton loader in our own library but it would make sense to move it over to your library

Two props (width and height)
CSS
@keyframes cp-skeleton-animation {
  from {
    transform: translateX(-100%);
  }
  to {
    transform: translateX(100%);
  }
}

.cp-skeleton-animation {
  animation: cp-skeleton-animation 1.2s infinite;
  height: 100%;
  left: 0;
  position: absolute;
  right: 0;
  top: 0;
  transform: translateX(-100%);
  z-index: 1;
  background: linear-gradient(
                  90deg,
                  rgba($white, 0),
                  rgba($white, 0.4),
                  rgba($white, 0)
  );
}

.cp-skeleton-loader {
  display: flex;
  height: v-bind(height);
  width: v-bind(width);
  position: relative;
  overflow: hidden;

  background-color: $gray-3;
  border-radius: 4px;
}

HTML

It is just some fancy css with a pretty simple HTML Template :)

i see u use v-bind here for the width and height. how should i implement this?

with a property like
or should it set it to 100% so you can control it width the parent elements?

Ready to test in v3.0.8


Clear media position if video has played to the end.

Closes #4174
PR Checklist

Please check all that apply to this PR using "x":

[x] I have checked that this PR is not a duplicate of an existing PR (open, closed or merged)
[x] I have checked that this PR does not introduce a breaking change
[ ] This PR introduces breaking changes and I have provided a detailed explanation below

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting)
[ ] Refactoring (no functional changes)
[ ] Documentation changes
[ ] Other - Please describe:

Fixes
Issue Number: #4174
What is the current behavior?
When a video plays to the end, the media position is set to the end of the video.
What is the new behavior?
When a video plays to the end, the media position is cleared.
Other information
I could have just set the media position to 0 (using a lot less code), but I thought that completely clearing the relevant entries from the redux store would be better to prevent an over accumulation of media positions.

Thanks @jeffslofish ! 🙂


dotnet graphql update throws an Unhandled exception error

Is there an existing issue for this?

[X] I have searched the existing issues

Product
Strawberry Shake
Describe the bug
When I try to use strawberryshake.tools command dotnet graphql update, I get an Unhandled exception error. The init and download command works fine.
schema.graphql
schema {
  query: Query
}

type Query {
  test: String!
}

// other code removed for brevity

TestQuery.graphql
query Test{
  test
}

The query just returns a simple string: public string Test => "this is a test";
Steps to reproduce

In the console navigate to the project where StrawberryShake is implemented
run dotnet graphql update

Relevant log output
Unhandled exception. Newtonsoft.Json.JsonReaderException: Unexpected character encountered while parsing value: ?. Path '', line 0, position 0.
   at Newtonsoft.Json.JsonTextReader.ParseValue()
   at Newtonsoft.Json.JsonTextReader.Read()
   at Newtonsoft.Json.JsonReader.ReadAndMoveToContent()
   at Newtonsoft.Json.JsonReader.ReadForType(JsonContract contract, Boolean hasConverter)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.Deserialize(JsonReader reader, Type objectType, Boolean checkAdditionalContent)
   at Newtonsoft.Json.JsonSerializer.DeserializeInternal(JsonReader reader, Type objectType)
   at Newtonsoft.Json.JsonSerializer.Deserialize(JsonReader reader, Type objectType)
   at Newtonsoft.Json.JsonConvert.DeserializeObject(String value, Type type, JsonSerializerSettings settings)
   at Newtonsoft.Json.JsonConvert.DeserializeObject[T](String value, JsonSerializerSettings settings)
   at StrawberryShake.Tools.Configuration.GraphQLConfig.FromJson(String json) in /home/runner/work/graphql-platform/graphql-platform/src/StrawberryShake/Tooling/src/Configuration/GraphQLConfig.cs:line 46
   at StrawberryShake.Tools.UpdateCommandHandler.UpdateSingleSchemaAsync(UpdateComman   at StrawberryShake.Tools.UpdateCommandHandler.UpdateSingleSchemaAsync(UpdateComman   at StrawberryShake.Tools.UpdateCommandHandler.UpdateSingleSchemaAsync(UpdateCommandContext context, String clientDirectory, CancellationToken cancellationToken) in /home/runner/work/graphql-platform/graphql-platform/src/StrawberryShake/Tooling/src/dotnet-graphql/UpdateCommandHandler.cs:line 83
   at StrawberryShake.Tools.UpdateCommandHandler.ExecuteAsync(UpdateCommandArguments arguments, CancellationToken cancellationToken) in /home/runner/work/graphql-platform/graphql-platform/src/StrawberryShake/Tooling/src/dotnet-graphql/UpdateCommandHandler.cs:line 46
   at McMaster.Extensions.CommandLineUtils.CommandLineApplication.ExecuteAsync(String[] args, CancellationToken cancellationToken)
   at StrawberryShake.Tools.Program.<Main>(String[] args)

Additional Context?
As a workaround I am using init each time to update the schema.graphql file but that changes the .graphqlrc.json each time
Version
13.7.0

Do you have a reproduction repository?

Might be caused by a BOM. Try saving the schema files as UTF-8 without BOM.

This is a reproduction repo I made while trying to learn strawberry shake. All the files were generated with the init command I am not supposed to change their encoding.

@michaelstaib @tobias-tengler
Could this be resolved by #4925? I'm very surprised that this hasn't come up before (for Strawberry Shake).

I can confirm that removing the bom on the generated .graphqlrc.json resolved the issue for me. Adding the bom back in reproduced the error.

Ran into this just now as well. If I didn't find this github issue I would have been completely stuck. Probably good to hotfix quickly.

Can confirm the problem and the solution by @Bahri-Haythem

#6751 has been merged, so in the next release, the files will be written without a BOM.

Fixed in 13.8.0-preview.1 and 14.0.0-p.16.

I am still having this issue in version 13.9.9

@RichardWhitfield Please create a minimal reproduction of the issue.


Beginner's Guide: Meaning of the Labels

I'd like to see the webapp as a low-entrance project making it possible for many many alumnis of different levels of experience to contribute.
I'm currently learning a lot to get up to speed, and thought a legend explaining the labels could be something useful.
So here's what I currently think I've figured out (please correct and complete!):

discussion: A topic that is (hopefully) worth thinking, talking, and deciding about.
question: Something you were wondering about. This could be a question about a specific feature you think would be cool for the app, or also a more general question (like the what's all this for?)
help wanted: This issues needs personnel! Take a look, and if you like what you see it would be great if you could hop on the train : )
in progress: This one is not quite ready to be worked on. Something's still pending (e.g. a "good-to-go" response from the official Udacity side)
ready: Here's an issue that you can start working on right now. Start digging freely!
enhancement: Working on this one will make the whole project become a better version of itself. Could be new features or improvements of already existing ones.
bug: Something's going wrong here. This needs a fix in the code.
invalid: The issue raised is not actually an issue. (explain why!)
wontfix: This is a hearty acknowledgement that something is not working, with the clarification that it's not worth the effort to fix it or that it would stray the project elsewhere (explain why!)
duplicate: This issue/question already came up and is discussed elsewhere (include a link to find that "elsewhere")

Labels can be generated in the Labels tab. And here is how someone else is using them.

Awesome!! Thanks so much for taking the time to do this!

I tried finding the location to add a label but for some reason I couldn't add a label

@bryanj4: You should be able to add Labels here. Not sure about permissions for this, but I was able to add discussion.
Under the Issues tab there is Labels button that then features a green New Label button.

@martin-martin as a member apparently I had less permissions than a non-member when I had no repository access. Issue was fixed and we'll be attaching proper labels for now on.

Hei @bryanj4 - I'd like to keep this open, since I believe a few people who are new and not so experienced could maybe benefit from reading about the labels (that was my initial intention when writing this post).
There might be a better place for it maybe - such as the wiki - and eventually we can move it there. But for now it seems to make sense to me to sit around here right next to where one will actually see the labels.

Moved this information into a wiki page: https://github.com/udacityalumni/alumni-client/wiki/Github


Reduce allocations in reading

This is a follow up of #193.
My understanding is that this crate is optimized for reading to (the owning enum) Value, that encapsulates all types, like e.g. serde-json does.
This design makes it expensive to deserialize it to other formats. Specifically, the main challenge is that given a rows of strings, each row will have to allocate a new String because Value owns String, even when the values were already allocated into a Vec<u8> during buffering + decompression (See Reader and Block). This happens to all non-copy types (String, Bytes, Fixed and Enum).
Formats such as arrow have their own way of storing strings. Thus, currently, using Value::String(String) adds a large overhead (I can quantify them if we need to) to interoperate. Specifically, the data path from a Read of a uncompressed file with 1 column of strings to Value is:
Read - [read] -> Vec<u8> - [decompress] -> Vec<u8> - [decode] -> String - [move] -> Value

This yields an extra allocation per row per column of non-copy types (String, Bytes, Fixed and Enum).
Usually, it is more efficient to read it to a Vec<u8> and produce non-owned values from it, in this case this would be
Read - [read] -> Vec<u8> - [decompress] -> Vec<u8> - [decode] -> &'a str -> Value<'a>

(there are other tricks to avoid re-allocating a Vec<u8> per Block + decompression which can be discussed in separate).
One way to address this is to declare a "non-owned" Value (e.g. ValueRef::String(&'a str)), and have the decoder to be of the form decode<'a>(&'a[u8]) -> Result<(ValueRef<'a>, usize)> where usize is the number of read bytes.
Since each row block has a declared number of bytes upfront and we read them to a buf: Vec<u8> as part of the buffering in Reader + decompression, we can leverage the existence of this buf to decode them to non-owning ValueRef, thereby avoiding the allocation per row per column.
We can offer impl<'a> From<ValueRef<'a>> for Value that calls to_string() (and the like to other types such as Bytes), if the user wishes to remove the lifetime at the cost of an allocation.
This is a pretty large, though, because we would need to also change the signatures of zag_i64 and the like to read from &[u8] instead of R: Read, since they must return how many bytes were consumed to "advance pointer of &[u8]".
I recently did this type of exercise as part of a complete re-write of the parquet crate parquet2, where this exact same problem emerges: a column is composed by pages (avro "blocks"), that are decompressed and decoded individually; it is advantageous to read each page to Vec<u8> before decompressing and decoding them, but we should not perform further allocations when not needed, specially for strings, bytes, etc.
The result of this re-write was a 5-10x speedup, which is why I am cautiously confident that this effort may be worth here too.

There's the same kind of suboptimal writing in the serde module, e.g. it uses visit_str instead of visit_borrowed_str.


SNAP App is way behind!

Hi there,
the Desktop Client announced version 3.10.5 but I'm in Ubuntu 20.04 so it is installed via 'snap':
# snap info rocketchat-desktop
name:      rocketchat-desktop
summary:   Official Desktop Client for Rocket.Chat
publisher: Rocket.Chat (rocketchat✓)
store-url: https://snapcraft.io/rocketchat-desktop
contact:   https://open.rocket.chat/channel/ubuntu-snap
license:   unset
description: |
  Desktop application for Rocket.Chat using Electron.
commands:
  - rocketchat-desktop
snap-id:      G4dFdVrFArll5teSvqpgWfRXIMGjME0l
tracking:     latest/stable
refresh-date: 2020-10-02
channels:
  latest/stable:    3.0.1          2020-10-02 (196) 69MB -
  latest/candidate: 2.7.0          2017-05-17   (4) 88MB -
  latest/beta:      3.0.0-develop  2020-09-30 (191) 69MB -
  latest/edge:      3.2.0-snapshot 2021-01-28 (322) 67MB -
installed:          3.0.1                     (196) 69MB -

Please update your snap App regulary!
Kind Regards.

Indeed, we are also getting security risk warnings on our RocketChar server and the latest is 3.9.3:
Security Alert
Critical security hotfix for Live Chat available in our latest releases 3.10.5, 3.9.7, 3.8.8. Please upgrade as soon as possible.

These are the latest available snaps:
tracking:     3.x/stable
refresh-date: 41 days ago, at 09:31 CET
channels:
  3.x/stable:       3.9.3  2020-12-20 (1449) 273MB -
  3.x/candidate:    3.6.2  2020-09-21 (1442) 284MB -
  3.x/beta:         ^
  3.x/edge:         ^
  latest/stable:    2.4.14 2020-12-23 (1453) 267MB -
  latest/candidate: 2.4.14 2020-12-23 (1453) 267MB -
  latest/beta:      2.1.1  2019-10-21 (1414) 251MB -
  latest/edge:      2.4.11 2020-02-27 (1427) 267MB -
  2.x/stable:       2.4.14 2020-12-23 (1453) 267MB -
  2.x/candidate:    2.4.14 2020-12-23 (1453) 267MB -
  2.x/beta:         2.4.14 2020-12-23 (1453) 267MB -
  2.x/edge:         ^
installed:          3.9.3             (1449) 273MB -

Indeed, we are also getting security risk warnings on our RocketChar server and the latest is 3.9.3:
Security Alert
Critical security hotfix for Live Chat available in our latest releases 3.10.5, 3.9.7, 3.8.8. Please upgrade as soon as possible.

These are the latest available snaps:
tracking:     3.x/stable
refresh-date: 41 days ago, at 09:31 CET
channels:
  3.x/stable:       3.9.3  2020-12-20 (1449) 273MB -
  3.x/candidate:    3.6.2  2020-09-21 (1442) 284MB -
  3.x/beta:         ^
  3.x/edge:         ^
  latest/stable:    2.4.14 2020-12-23 (1453) 267MB -
  latest/candidate: 2.4.14 2020-12-23 (1453) 267MB -
  latest/beta:      2.1.1  2019-10-21 (1414) 251MB -
  latest/edge:      2.4.11 2020-02-27 (1427) 267MB -
  2.x/stable:       2.4.14 2020-12-23 (1453) 267MB -
  2.x/candidate:    2.4.14 2020-12-23 (1453) 267MB -
  2.x/beta:         2.4.14 2020-12-23 (1453) 267MB -
  2.x/edge:         ^
installed:          3.9.3             (1449) 273MB -

Any news on this?

Please test if the latest release did solve this. If not please reopen the issue or create a new one and tell us. I will be closing this one for now.
Thank you!


How to compose a container from a Image and layer in Docker 1.10

Two questions. Thanks in advance.
1: In V 1.10, how to get new layer ID after creating a new container from a image? In previous version, the layer ID is the same as container ID which is returned from RestAPI "POST /containers/create"? Looks like the new layer ID is saved in the last item in manifest.json, but this assumption is not safe.
2: How to restore a container from an Image and above layer.tar? RestAPI "PUT /containers/(id or name)/archive" may only handles a normal tar file instead of lay.tar, and it can't handle the deleted files in the new container.

A layer ID is created when a container is committed. At that point the changes to the container are snapshotted, put into a new layer, and its layer ID is calculated from the content. This is the ID you will find in the manifest.
Individual layers are not imported from an image to create a container or new image. If you want to create a new image with a single layer you can use docker import. If you want to "restore" a container from a tar file, you will need to create a new container from the targeted image and unpack your tar file in the image (see https://docs.docker.com/engine/reference/api/docker_remote_api_v1.23/#extract-an-archive-of-files-or-folders-to-a-directory-in-a-container). Images are normally "restored" with all of its layers as layer.tar entries in a image tar file created from docker save.

I would encourage questions like these to be asked at https://forums.docker.com/ in the future. They can be better directed as this question is not really related to the distribution project, but more relevant to the main docker project.

Thanks. But I can't use PUT archive API to extract a tar ball to container root directory /. In curl request message, the extract destination path was always extended by tar ball name, e.g. /layer.tar. Then the server side can't complaints that "lstat /var/lib/docker/devicemap/mnt/containerID/rootfs/layer.tar: no such file or directory". By the way, I can use PUT to extract it to non root directory, e.g. /tmp/.
Why the destination path in PUT command was always extended by tar ball name? Thanks.
Output of docker version:
Client:
Version: 1.10.3
API version: 1.22
Go version: go1.5.3
Git commit: 20f81dd
Built: Thu Mar 10 15:39:25 2016
OS/Arch: linux/amd64
Server:
Version: 1.10.3
API version: 1.22
Go version: go1.5.3
Git commit: 20f81dd
Built: Thu Mar 10 15:39:25 2016
OS/Arch: linux/amd64
Steps to reproduce the issue:
1.curl -s -XPUT -T layer.tar <IP_ADDRESS>:42431/containers/04c9ac705ca0/archive?path=/
//Only root dir / is specified in above path field in PUT command, but the path was changed to path=/layer.tar
About to connect() to <IP_ADDRESS> port 42431 (#0)
Trying <IP_ADDRESS>...
Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 42431 (#0) > PUT /containers/04c9ac705ca0/archive?path=/layer.tar HTTP/1.1 > User-Agent: curl/7.29.0 > Host: <IP_ADDRESS>:42431 > Accept: / > Content-Length: 62464 > Expect: 100-continue > < HTTP/1.1 404 Not Found < Content-Type: text/plain; charset=utf-8 < Server: Docker/1.10.3 (linux) < X-Content-Type-Options: nosniff < Date: Fri, 13 May 2016 21:19:13 GMT < Content-Length: 148 < Connection: close 2. 3.

Describe the results you received:
In the server side, it can't find rootfs/layer.tar.
DEBU[175614] Calling PUT /containers/04c9ac705ca0/archive
DEBU[175614] PUT /containers/04c9ac705ca0/archive?path=/layer.tar
DEBU[175614] container mounted via layerStore: /var/lib/docker/devicemapper/mnt/375d0bc5401eedfc9a3dce6c8ded065820fadcac57f5f39333336f05a719bdd7/rootfs
DEBU[175614] devmapper: UnmountDevice(hash=375d0bc5401eedfc9a3dce6c8ded065820fadcac57f5f39333336f05a719bdd7)
DEBU[175614] devmapper: UnmountDevice(hash=375d0bc5401eedfc9a3dce6c8ded065820fadcac57f5f39333336f05a719bdd7) END
ERRO[175614] Handler for PUT /containers/04c9ac705ca0/archive returned error: lstat /var/lib/docker/devicemapper/mnt/375d0bc5401eedfc9a3dce6c8ded065820fadcac57f5f39333336f05a719bdd7/rootfs/layer.tar: no such file or directory

I can extract files to root directory with path=%2F "curl -s -XPUT -T layer.tar <IP_ADDRESS>:42431/containers/04c9ac705ca0/archive?path=%2F -v". But path=/ still doesn't work.


Network usage is prohibited for themes

It appears that your theme uses network connections to load assets (e.g. fonts, icons, or images). This is prohibited by the official Obsidian developer policies because themes should function completely locally.
You can bundle an asset for local use by using data URLs. See this guide.
Please let us know if you have any questions. Any themes that use network connections will be removed from the official directory in the first week of January 2024.

The Obsidian team.

It appears that your theme uses network connections to load assets (e.g. fonts, icons, or images). This is prohibited by the official [Obsidian developer policies]
Please let us know if you have any questions. Any themes that use network connections will be removed from the official directory in the first week of January 2024.

This should be now a non-issue with the release of v8.1.9 of my theme. I have removed all url links to external websites. If i happen to have missed any, which i don't think I have let me know so i can patch it out.

Closing this has there have been no responses from you (@joethei), so after update v8.1.9 which removed the link to the resources in question, I assume that you or other Obsidian staffers would find that this theme now conforms correctly to the new policy.


Fix cloning of Icons with width/height

Fixes the issue mentioned in https://github.com/openlayers/openlayers/pull/14364#issuecomment-1402894519
Please review.

Thanks for the review.


docs: Add python3 to ./tools/provision command

Updated the description in README.md in step 5 and step 6 to make it more user-friendly.

@aero31aero, Can you please tell me why the Travis checks are failing?

You can check the details here. I'm not sure what's causing the failures, but its the same for both your PRs, apparently.
Also, change privision to provision in your commit message.

@showell, can you please tell me that why these tests are failing?

I have a hunch that you might have branched off of master when master was failing. I'm not on a laptop right now so I cannot confirm, but can you try rebasing your branch with the latest master? Oh, and also tell if the tests are running fine locally.
This cannot be from your changes, so I believe it should not be a blocker for your PR.

All the tests are running fine locally. My patch-1 branch is just one commit ahead of the latest master branch.

@Shivamgera Can you fix the commits to not have a merge command?

@showell, Done.


Separate pages for previous Hacktoberfest

Using the commit history add pages for previous Hacktoberfest from the data which is there in the repo.

Can you assign it to me because i have comeup with a solution.

@swapnilsparsh why is this repo marked excluded in Hacktober 2024

@swapnilsparsh why is this repo marked excluded in Hacktober 2024

Don't know


Fix listener method type checking

Fix #782

Add checking to listener method parameter to allow using subtype of original method's type only when the parameter is subtype of view.

Add testcase in OnTextChangedTest.java

Looks like com.android.tools.lint:lint-tests:25.1.2 can't be found in https://plugins.gradle.org/m2/ if add https://dl.bintray.com/android/android-tools to repositories, it builds with no problem.

Add mavenCentral(). It shouldn't have been removed in the first place.

Thanks!


Fix bug that was ending phase incorrectly

Checklist

[x] Use a separate branch in your local repo (not master).
[x] Test coverage is 100% (or you have a story for why it's ok).

Test
When a move is made it checks if the turn should end and if the phase should end.
In the situation of phase A ending the turn and the phase, it should start phase B end check phase B endPhaseIf.
Expected:
phase A - call onPhaseEnd 1 time
phase b - don't call onPhaseEnd

I signed it!

Coverage remained the same at 100.0% when pulling afd4cdaa014c139237326f7e8293ff7f94a6c41d on janosimas:endPhaseOnMove into bfafa94ef684556abf66a8e23d7300473e71e679 on google:master.


LIBS-392 - reduce io and startup-time

Pull Request Test Coverage Report for Build 242

1 of 11 (9.09%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-3.8%) to 14.583%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/main/java/com/avides/springboot/testcontainer/mysql/EmbeddedMysqlContainerAutoConfiguration.java
0
10
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/main/java/com/avides/springboot/testcontainer/mysql/EmbeddedMysqlContainerAutoConfiguration.java
1
0.0%

Totals

Change from base Build 97:
-3.8%

Covered Lines:
7

Relevant Lines:
48

💛  - Coveralls


Use Jekyll's link tag

WIP, depends on #3045

This seems a bit too cumbersome as it requires absolute URLs:

The path to the post, page, or collection is defined as the path relative to the root directory (where your config file is) to the file, not the path from your existing page to the other page.

https://jekyllrb.com/docs/liquid/tags/#link

So this would need to change a thousand plus links and it would also necessitate (smaller) changes to the archival script...


NumberInput: Input invalid value issue

I'm using NumberInput widget, and my usage scenario is that there are one NumberInput and one Button, when the NumberInput is with valid value, the Button is enabled, when NumberInput is with invalid value, the Button is disabled.
While I find when I try to input a value that lower than minimum value or clear the input area to be empty, it automatically resets to the minimum value, BUT the value I can track is already invalid, then the Button is disabled, however，the value show in NumberInput widget is the minimum value which is a valid one that means Button shall be enabled.
So my question is that how I detect the value to make my condition to judge button with disabled correctly?
Right now, I'm using value === ' ' or value > {<max_value>} or value < {<min_value>}
Take clear the input area as an example, when I clear the input area, the value I can detect is ' ', so button disabled is true, button is shown in disabled status, BUT, the value shown in NumberInput widget is automatically changed to minimum value(it is 1 in my test scenario) , which is a valid value and the button shall be enabled.

The code:
import React from 'react';
import { Button, NumberInput } from 'carbon-components-react';
export default class AlertConfigBK extends React.Component {
constructor(props) {
super(props);
this.state = {
numberInputValue: 0,
disabled: false,
};
}
onChange = value => {
value = value.imaginaryTarget.value;
let disabled = !!(value === '' || value > 100 || value < 1);

this.setState({ numberInputValue: value, disabled });

};
render() {
let { numberInputValue, disabled } = this.state;
return (

      <NumberInput
        id="number_input"
        value={numberInputValue}
        max={100}
        min={1}
        step={1}
        onChange={this.onChange}
      />
      <Button kind="primary" disabled={disabled}>
        test
      </Button>

);

}
}

Detailed description

Describe in detail the issue you're having.

Is this a feature request (new component, new icon), a bug, or a general issue?

Is this issue related to a specific component?

What did you expect to happen? What happened instead? What would you like to see changed?

What browser are you working in?

What version of the Carbon Design System are you using?

What offering/product do you work on? Any pressing ship or release dates we should be aware of?

Steps to reproduce the issue

Step one
Step two
Step three
etc.

Additional information

Screenshots or code
Notes

Hi :wave: thank you for bringing up this topic! As you may have imagined, our <NumberInput> auto-caps the given value. You can cap the value yourself to see if the number input is in an invalid state, like:
const newValue = Math.min(Math.max(1, Number(value.imaginaryTarget.value)), 100);
const disabled = isNaN(newValue);
this.setState({ numberInputValue: newValue, disabled });

If you desire <NumberInput> not to auto-cap the given value, you can create a new issue for that. Hope this helps!


[Bug]:  Make a page for the Shop clothing

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
I want to make a shop clothing page and attach to link by which it will open another page and feel free to explore the cloth shopping.
Add Screenshots / Video Clip

make a shopping  page for the cloth shopping to enhance the shopping style.
What browsers are you seeing the problem on?
No response
Agreements

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC contributor
[X] I'm a Hacktoberfest contributor
[X] I want to work on this issue [Optional]
[X] I'm willing to provide further clarification or assistance if needed.

please add the best label for this bug @Tejashri-Taral

Proceed!

please add level 2 or 3 label for this work @hars-21

Level Labels will be added based on your work. I'll add the labels once you make the PR


Singular / plural and other inflections?

Is Globalize.localize already able to handle inflections or how would one devise such an extension?
//Pseudo example defining 3 forms of test

// Map of messages used by .localize()
    messages: {
       test : {
          _0 : "Zero tests",
          _1 : "%1s test",
          _1+ : "%1s tests"
       }
    }

No, this is (so far) out of scope. I'll leave the issue open to reflect that we may want to consider it as a feature in the future.


Fails to fix require_trailing_commas rule in assert invocations

    assert(conclusion != null ||
        (completedAt == null && status != CheckRunStatus.completed));

Code like this causes the require_trailing_commas lint, but there is no corresponding fix!

https://dart-review.googlesource.com/c/sdk/+/237694

Bless you, sir 🙏


Availability of theoretical chemical kinetics methods in the literature

The following summarizes (and adds to) Table 1 of Dzib (2018). In particular, I want to organize what is available in overreact concerning what is available in other codes.
All of the following lists are organized from most commonly available to least commonly available in other codes:
Kinetic theory

[x] Transition State Theory
[ ] Collins-Kimbal Theory
[ ] Marcus Theory
[ ] Rise-Ramsperger-Kassel-Marcus
[ ] Variational Transition State Theory
[ ] Master Equation
[ ] Instanton Theory

Molecularity

[x] Unimolecular
[x] Bimolecular
[x] Termolecular (but untested)

Phase

[x] Gas
[x] Solution
[ ] Gas-solid
[ ] Solid

Tunneling effects

[x] Eckart
[x] Wigner
[ ] Small Curvature Tunneling
[ ] Zero Curvature Tunneling
[ ] Large Curvature Tunneling

User interfaces

[x] Command-line interface
[ ] Graphical user interface

Other characteristics

[x] quasi rigid-rotor harmonic-oscillator (QRRHO)
[ ] Hindered rotors (probably not required since we have QRRHO, but happy to be proved wrong)

@Leticia-maria I have a feeling there are things on this list that might interest you (spoiler: Marcus theory and a GUI 🐱).


Unset the body background color

Applying a dark background color to the body tag causes some browsers (Chrome, Firefox, Safari) to paint the scroll thumb in a light color in order to provide suitable contrast, but since we’re setting the background color of page content back to white, the scroll thumb gets lost, which makes it difficult to visualize the size of a page and one’s scroll position within it.
Before & after:

LGTM, good find! 👍


Factor out commonality in Databricks non-UC and UC tests into shared scripts

Yup, DRY.

Looks better as no DRY. Did not refactor test_copy_model_version.py as the arguments to copy_model_version.copy differ substantially.


nixos/nginx: use list for logrotate files attribute

Description of changes

I'm using a flake to configure multiple nginx vhosts on one system. Every vhost has its own logfile written to /var/log/nginx/example.com/dev.example.com/access.log, /var/log/nginx/example.com/www.example.com/access.log, etc. So it's two additional subdirectories and the default logrotate rule /var/log/nginx/*.log doesn't apply.
I now need to do
main.nix
{
  services.logrotate.settings.nginx.files = [ "/var/log/nginx/*.log" ];
}

vhost-1.nix
{
  services.logrotate.settings.nginx.files = [ "/var/log/nginx/vhost-1.com/*/*.log" ];
}

vhost-2.nix
{
  services.logrotate.settings.nginx.files = [ "/var/log/nginx/vhost-2.com/*/*.log" ];
}

With the change proposed by this PR the change inside main.nix isn't required anymore.
I'm not 100% sure if this change is backward compatible if someone else or another module made changes to their services.logrotate.settings.nginx.files. But probably it shouldn't break stuff? If there are concerns we maybe want to do this only starting with NixOS 24.11?
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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/)
24.11 Release Notes (or backporting 23.11 and 24.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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@RossComputerGuy I think it would be good if we could we get this into 24.11, what do you think?

Please rebase and not merge


Apply Workaround for SDK e2e tests to avoid SSL error

Issue link

https://issues.redhat.com/browse/RHOAIENG-14785
Removed TokenAuthentication to avoid SSL error for 2.14 downstream testing
What changes have been made

Verification steps

Checks

[x] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[x] Manual tests
[ ] Testing is not required for this change

@Srihari1192 So if I am understanding correctly we'll be using the Kubernetes config file for authentication purposes?

@Bobbins228 Yes oc login with kubeconfig file tests working fine #1948


BrowserSync v2.12.4 breaks default file watching

As stated in the title, BS 2.12.4 breaks default file watching behavior. It changes how the files input array is processed. I've opened an issue on the BS repo, but it may be worth considering updating the default files array, depending upon how they choose to handle the issue.
Bug repro steps

Make sure your package is using BrowserSync v2.12.4
Use the default files option (files: ['./**/*.{html,js,css}'], I believe)
Run lite-server
Modify a file that should trigger a reload
No reload

Environment

lite-server version: 2.2.0
nodejs version: 5.3.0
npm version: 3.8.1
OS type/version: OSX

This is resolved in v2.12.5, sorry for any inconvenience caused :)


[Feature Request] Add keybinding labels to Cut / Copy / Paste in right click meny

Context

[X] This issue is not a bug report. (please use a different template for reporting a bug)
[X] This issue is not a duplicate of an existing issue. (please use the search to find existing issues)

Description
Almost all of the menus in the right click context menu have keybinding labels on the right side of the menu item. Even though Cut / Copy / Paste do have keybindings the labels are not displayed.

VS Code does show the labels and I think it makes for a better user experience to do so.

Any updates on it? Is it accepted as improvement?

Dang. This seems to have been fixed in the latest couple of versions (0.46.0 and 0.47.0), however, these versions also broke platform detection (#4378), so I can't upgrade.
Here's my context menu on 0.45.0:

And here it is on 0.46.0:

So nice! Except for the fact that this is on a Mac 😭

This is a pity haha. Having the same problem here as @codebykat

Tragically, in 0.48.0 the platform detection is fixed... but there are no labels on the context menu anymore, so I guess the labels may have been a side effect of it incorrectly thinking I was on a Linux machine.

In further investigating this issue, I came across this comment over in the vscode repo:

When VS Code runs in Safari (or any other browser), we leave default copy/cut/paste keybindings blank, majorly because we want to avoid security warning or inconsistent experience.

So now I'm wondering if this is intended behavior, though I'm also seeing missing labels in Electron, not just in the browser. At the moment I'm hacking the right-click menu to remove the default Cut/Copy/Paste and adding my own that show keybindings, which is obviously not ideal for several reasons.


Performance: iPhone: Lesson 1: first-time page load is slow, resulting in activity "ghosts"

Tackling this from both server and client sides.
Server side

Set up CDN for better performance and reliability when serving the media.
Turn on diagnostics log to be used for troubleshooting potential server side issues

Client side
3. Give user an ability to retry (in progress)
4. Implement local cache (in progress)
If 1st & 2nd address the major issues we saw, we can then work on 3rd & 4th as next set of priorities.

Update as of 2/21.
Server side

Set up CDN for better performance and reliability when serving the media (Done)
Turn on diagnostics log to be used for troubleshooting potential server side issues (Done)

Client side
3. Give user an ability to retry (Done)
4. Implement local cache (in progress)
There have been additional loggings and error handling added as part of 3rd item.
Assuming all three changes made so far would address/mitigate the original issue, we will track 4th item above as separate issue in lower priority.

As of now, as mentioned above, all three things are done, and a separate issue for 4th item about cache has been created. Since the majority of the intended work is done, closing this issue. Any new problem around this, should be tracked as new issue.


Auto-Disconnecting Issue

Please tick all applicable boxes.

[X] I am using Python 3.5.3 or higher (run python --version on the command line)
[X] I have followed the official guides to install the bot for my system
[X] I have updated my dependencies to the latest version using the appropriate update script

Which version are you using?

[X] The latest master version (release-260419)
[ ] The latest review version

What type of issue are you creating?

[X] Bug
[ ] Feature request
[ ] Question

Description of issue
I summon the bot and attempt to play a youtube playlist. It then disconnects after putting in the play command and it produces this wall of text.
Steps to reproduce

Summon the Bot
Use !play to play something

Log file
Please attach your MusicBot log file (located at logs/musicbot.log) to this issue. You can do so by dragging and dropping the file here. If you do not include your log file, you WILL be asked to provide one.
musicbot.log

Try doing git checkout review then python -m pip install -U -r requirements.txt and go to the Discord Developer Portal and enable intents

Try doing git checkout review then python -m pip install -U -r requirements.txt and go to the Discord Developer Portal and enable intents


Event Grid trigger function: internal server error when subscribing to a topic

I deployed a new .NET 5 function app using the newly released 1.0.0 functions worker. It has two functions, one with an Event Grid trigger and one with a HTTP trigger. I want to consume events from an Event Grid topic.
I can successfully subscribe the HTTP function to my topic with az eventgrid event-subscription create --event-type webhook .... (The function includes an implementation of the Event Grid handshake, of course.)
However, if I try to subscribe the Event Grid function with az eventgrid event-subscription create --event-type azurefunction ..., I instead get an error like this at the Azure console
Deployment failed. Correlation ID: 9c0e3d2f-9591-46f0-9581-adf3f00877d9. The operation failed due to an internal server error. The initial state of the impacted resources (if any) are restored. Please try again in few minutes. If error still persists, report 1b81e514-92f6-4b5e-a36c-ab5fbb44a861:3/11/2021 4:25:26 PM (UTC) to our forums for assistance or raise a support ticket .

There are no invocations logged of the function and no messages in its event stream so it looks like it doesn't get that far. However, I do notice that in the Functions section of my Function App in the Azure Portal, whilst my functions are shown in the table, the Trigger column is blank (unlike in some Function Apps using .NET Core 3 in-process functions, where the Trigger column is correctly populated.)
I tried to follow the sample closely and both triggers work fine in local debugging.

This might be somewhat unrelated to the worker, but it would be good to have access to the logs for closer investigation. Would you be able to share the app name?
For the portal issue, this is one of the known problems we're tracking with UX and a deployment is currently rolling out to address that. We hope to have that addressed soon.
@mhoeger and @ankitkumarr ; I know you've spent time going through the EG flows, so I'm tagging you here just in case I missed something obvious above.

@fabiocav Thanks for the quick reply. The app is "softools-staging-workflow-service".

@haeberle The az command I described above started working for me as of a few weeks ago, so something has been fixed somewhere. Try that 😄
At the time when that command started working, I tried Azure Portal as you describe and that still didn't work for me. I automated my deployment with the CLI, so I haven't needed to try Azure Portal again.

Thanks a lot, works well with your workaround!

-opened my IoTHub in the azure portal

want to add a new Event Subscription
select Azure Function as end point
Select my function as function app
no function is selectable

This still seems to be an issue in the Azure portal, in my case I'm trying to bind an event from Azure Media Services to my isolated process function and the function trigger that I'm expecting to see for EventGrid does not show up:


Allow initial install of Konveyor to be seeded with a demo app so a new user can quickly run several workflows

For background I was reading this link and it made me think of how we could 'shorten time to value' for a new user who is installing Konveyor by seeding enough demo configuration that they can install and then run a few given scenarios to get a hands on feel of capabilities.
What if we defaulted the behavior of the operator to:

Seed an application in Inventory that highlights analysis capabilities we want to show.  Assume this is configurable and can be disabled via a parameter.

In addition, I think it would also be worth considering if we want to default to disabling 'Auth' and make this an opt-in.

I think this is a great idea.
It would also be good for all the stuff we care about to use this example as a integration test.

+1. I have always appreciated a few projects where they had a well written demo + documentation around that demo . We can also may be add a little design diagram with the app & how we can use Konveyor to modernize if possible. This was especially helpful when I had to try out multiple tools to solve an issue in a limited time and make a decision on which one to pick.

I've received feedback about users finding it very hard to figure out where to begin and how things work in an empty Konveyor instance with no data on it. The actual quote was that "it took too much time and effort to get to a point where value could be demonstrated". @dymurray  @jwmatthews @savitharaghunathan I believe we should pick this up for 0.4 alongside some documentation that explains what the sample data stands for. My only gripe about the original proposal would be that I don't think we should default to seeding the instance with the sample data, as I believe it should be the other way around.
I think the tackle-testapp application and the petclinic example would be great for sample data, although I guess we would need to figure out if there are any legal implications to that, both upstream and other potential downstream distributions. I imagine the best approach would be to leave this as a configurable parameter or dataset, so vendors can add whatever they want in their downstream distributions.


Junierch/mi tde

This PR fixes #
Create TDE wizard experience for the user to configure certificate migration through ADS.

Add TDE migration step

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.002%) to 42.071%

Totals

Change from base Build<PHONE_NUMBER>:
0.002%

Covered Lines:
28595

Relevant Lines:
63135

💛  - Coveralls

Please resolve the conflicts, and also whenever you update the STS version please include a link to the diff between the current version and the one you're updating to - e.g. https://github.com/microsoft/sqltoolsservice/compare/<IP_ADDRESS>...<IP_ADDRESS> - and then tag anyone who's changes are in the list of commits.
(note in this case it's only your change so it should be fine, but you should still always include that for anyone reviewing the changes)

@junierch, synced with @AkshayMata offline.
action items for this PR:

vbump extension version to 1.2.3

1.2.1 contains DB + VM UI improvements, and login migrations ON. This is in insiders already, and should not go to worldwide.
1.2.2 contains DB + VM UI improvements, and login migrations OFF. This needs to go to worldwide. (and there's no point publishing this to insiders)
1.2.3 will contain DB + VM UI improvements, and login migrations ON (see point 3 below), and TDE ON.

vbump STS version in mssql/config.json to <IP_ADDRESS>

notice that in the release/1.41 branch (the version that will make it in to the final ADS 1.41 product), we are using <IP_ADDRESS>. However, note that in main, this is still <IP_ADDRESS>. Technically that means prod ADS 1.41 is actually ahead of insiders right now in terms of backend. This probably explains why you were seeing that your new changes were not being consumed in insiders, which is maybe why you tried to vbump the ADS version to 1.42 instead of 1.41. Once this change is made, that will no longer be needed and the latest NuGet will be consumed in insiders properly

re-enable login migration button (basically, revert https://github.com/microsoft/azuredatastudio/pull/21717)


Stop using soon-to-be-deprecated Spring Security API

Spring Security 6.4's Kotlin DSL deprecates the authorizeRequests block and
suggests using authorizeHttpRequests instead. Make that change so that the
upgrade will succeed rather than failing to compile with a deprecation warning.
The upgrade requires a workaround for a missing feature in the Kotlin DSL; when
they've addressed that, we can remove the workaround.

#2634  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


SKIPPING SONG - FAILED TO QUERY METADATA

Hi!
I tried downloading a song with zotify and got this error (see below) and I'm not really sure what to do with it.
SKIPPING SONG - FAILED TO QUERY METADATA
Track_ID: 3yfqSUWxFvZELEM4PmlwIR
replace() argument 2 must be str, not None
Traceback (most recent call last):
File "C:\Users\alexa\scoop\apps\python\current\Lib\site-packages\zotify\track.py", line 175, in download_track
output_template = output_template.replace("{ext}", ext)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: replace() argument 2 must be str, not None
in the config I changed the output to: "OUTPUT": "{artist} - {song_name}.{ext}", this was shown as an example, but since I changed this I am not able to download any songs no more.
I'm new to github, so I'm not really sure how to fix this

Hi there,
This is likely happening because you have input an incorrect file type into the download format field in your config
It should look like this:
"DOWNLOAD_FORMAT": "ogg",
hope this helps :)


Clean up some old code supporting Python 2

Should fix tests against Django master branch.

Codecov Report

Merging #219 into master will decrease coverage by 0.07%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #219      +/-   ##
==========================================
- Coverage   73.92%   73.85%   -0.08%
==========================================
  Files          32       32
  Lines         349      348       -1
==========================================
- Hits          258      257       -1
  Misses         91       91

Impacted Files
Coverage Δ

health_check/exceptions.py
100% <100%> (ø)
:arrow_up:

health_check/storage/backends.py
90.24% <100%> (-0.24%)
:arrow_down:

health_check/backends.py
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

master still failing to build but I believe it should succeed after merging #215 that updates the Travis build environment to Ubuntu Xenial with a newer SQLite.


Dependency upgrade and more settings for the theme

I needed a more recent version of font awesome to include (among other things) a mastodon icon.
I also prefer to list the posts in the blog with the newest on top (like in the RSS feed) or have the fade-in effect only on the main page

Hi Kevin,
It would be of great help to me if you separate your work into PRs that are centered on one change. I would like to merge your FontAwesome upgrade (Mastodon! Yay!), but your other changes will take more time to review and integrate.
Please submit these changes as separate PRs so they can be more gracefully merged.

I'm more used to Gerrit type workflows hence the state of this PR. Sorry about that, will resubmit shortly.


epilogs for VST straight line reco

Hi @bonventre,
You have proposed changes to files in these packages:

JobConfig

which require these tests: build.
@Mu2e/fnalbuild-users, @Mu2e/write have access to CI actions on main.
:hourglass: The following tests have been triggered for cb4bc7d20fc6602bbcf15b07603e221156770065: build  (Build queue is empty)
About FNALbuild. Code review on Mu2e/Offline.

:sunny: The build tests passed at cb4bc7d20fc6602bbcf15b07603e221156770065.

Test
Result
Details

test with
:white_check_mark:
Command did not list any other PRs to include

merge
:white_check_mark:
Merged cb4bc7d20fc6602bbcf15b07603e221156770065 at fa249cf002f28aafcb5c607c630198a5aff5d106

build (prof)
:white_check_mark:
Log file. Build time: 12 min 16 sec

ceSimReco
:white_check_mark:
Log file.

g4test_03MT
:white_check_mark:
Log file.

transportOnly
:white_check_mark:
Log file.

POT
:white_check_mark:
Log file.

g4study
:white_check_mark:
Log file.

cosmicSimReco
:white_check_mark:
Log file.

cosmicOffSpill
:white_check_mark:
Log file.

ceSteps
:white_check_mark:
Log file.

ceDigi
:white_check_mark:
Log file.

muDauSteps
:white_check_mark:
Log file.

ceMix
:white_check_mark:
Log file.

rootOverlaps
:white_check_mark:
Log file.

g4surfaceCheck
:white_check_mark:
Log file.

FIXME, TODO
:white_check_mark:
TODO (0) FIXME (0) in 0 files

clang-tidy
:white_check_mark:
0 errors 0 warnings

N.B. These results were obtained from a build of this Pull Request at cb4bc7d20fc6602bbcf15b07603e221156770065 after being merged into the base branch at fa249cf002f28aafcb5c607c630198a5aff5d106.
For more information, please check the job page here.
Build artifacts are deleted after 5 days. If this is not desired, select Keep this build forever on the job page.


Feature/improvment build package

I add some changes to make more comfortable the 'tools/build_package.sh' script output.

The set -x option is disabled except if the -v flag is added as an argument.
The publisher script output has been redirected to a log file in /tmp so that the standard output does not bother the terminal in which it is working

Thanks Kaiwalya, sorry for spelling


Extract method and variable not working on Untitled notebooks in WSL

 Environment data

Language Server version: 2024.1.102
OS and version: linux x64
Python version (and distribution if applicable, e.g. Anaconda): 3.12, Anaconda
python.analysis.indexing: true
python.analysis.typeCheckingMode: basic

 Code Snippet

An untitled notebook with the following sample:
x = 10
y = 20
print(x+y)

 Repro Steps

Create a new untitled notebook with the code sample above
Select all the code in the cell and trigger the "extract method" code action

 Expected behavior
It should extract the method in that same cell like it does when using the latest stable version of the 3 extensions:

 Actual behavior
It extracts the method in a separate untitled Python file

 Logs

pylancelogsnotebookswsljan3024.txt

Also reproducing it on Windows:

Interestingly enough, if I enable sync server it fails silently:

2024-01-30 13:54:50.948 [info] [Trace - 1:54:50 PM] Received response 'workspace/executeCommand - (50)' in 17ms. Request failed: Request workspace/executeCommand failed with message: applyEditSync failed with error 0 (-32603).
2024-01-30 13:54:50.949 [info] [Error - 1:54:50 PM] Request workspace/executeCommand failed.
2024-01-30 13:54:50.949 [info]   Message: Request workspace/executeCommand failed with message: applyEditSync failed with error 0
Code: -32603

sync server failure should be this bug:
https://github.com/microsoft/pylance-release/issues/5404

Oh I think I know the problem. The .py in the notebook name. It's not being removed from the cell uri.
untitled://Untitled-1.ipynb.py?jupyter-notebook#W0sdW50aXRsZWQ=
It's likely the mapped cell uri is not being used in the extract method.

This issue has been fixed in prerelease version 2024.1.103, which we've just released. You can find the changelog here: CHANGELOG.md


Where is the option 'Export HTML Documentation' in insomnia?

I have been installed the plugin in insomnia designer, but i didn't find this option to generate the docs

I have been installed the plugin in insomnia designer, but i didn't find this option to generate the docs

Click on the workspace name at the top left corner in Insomnia app, you should see it in the plugins menu group.

I have been installed the plugin in insomnia designer, but i didn't find this option to generate the docs

Click on the workspace name at the top left corner in Insomnia app, you should see it in the plugins menu group.

This option doesn't exists in isnomnia designer, i think that this plugin was designed to default insomnia app, but i can install this plugin in plugins section of insomnia desginer.

This option doesn't exists in isnomnia designer, i think that this plugin was designed to default insomnia app, but i can install this plugin in plugins section of insomnia desginer.

This option doesn't exists in isnomnia designer, i think that this plugin was designed to default insomnia app, but i can install this plugin in plugins section of insomnia desginer.

Oh, yes it will only work in Insomnia Core, I honestly skipped the "designer" word in your comment.

This option doesn't exists in isnomnia designer, i think that this plugin was designed to default insomnia app, but i can install this plugin in plugins section of insomnia desginer.

Oh, yes it will only work in Insomnia Core, I honestly skipped the "designer" word in your comment.


Announce ongoing events on a channel

API Request from Wiki? Manually doing some tricky things? IDK. Help is wanted.

Found Module! https://www.npmjs.com/package/node-schedule
Now just the will to schedule things.. @rey2952 Pls

http://www.animalcrossingcommunity.com/event_calendar.asp?CurrentMonth=1%2F1%2F2017&ACGameID=3
EVENTS CALENDAR IS HERE! NOW WE NEED HELP TO ARRANGE DATES AND TIME!

Ready for merging.

Done.


Improved Support for Affordable Aliexpress Thermostats (e.g., TV02-Zigbee) [part one / basic support]

Hi jmcollin78,
I hope this message finds you well. I've been actively using your project and have noticed some areas where I could contribute to enhancing the support for affordable Aliexpress thermostats, specifically the TV02-Zigbee model.
Before discovering your project, I was using better_thermostat, which generally worked well for me. However, I encountered occasional outliers where the temperature offset would get stuck at a large value (e.g., -5), causing the thermostats to attempt to heat up to 26°C. With my modifications in this project here, I've managed to achieve reliable functionality with the TV02-Zigbee devices. Although the implementation is not perfect yet, I've made substantial progress.
My plan moving forward is to continue refining the support for these thermostats. I'm also looking into leveraging native features provided by the thermostats in Home Assistant, such as native frost protection, etc. To achieve this, I'm considering creating a 'FeatureController' class for the UnderlayingClimates. This class would automatically recognize various features and pass them as child setups for the UnderlayingClimate.
Additionally, I aim to introduce a local calibration algorithm similar to that in better_thermostat. This algorithm continually adjusts the calibration to ensure that the integrated temperature sensor aligns with the external one.
I believe these enhancements will contribute to a more robust and versatile integration of affordable Aliexpress thermostats into your project. I'm eager to collaborate and receive feedback on these proposed changes.
Thank you for considering my pull request, and I look forward to your thoughts on these improvements.
Improvements for the future:

[ ] Cleaner implementation for the heatstop switch
[ ] Offset-Based Temperatur Controlling for the Thermostats which supports local_callibration

Here some results of the regulation with thermostats own regulation with this changes:

A small problem is that the heat state is not always reported correctly - the state is heat when the set_point > current_temperature, but these thermostats seems to heat a little bit when they are already on the temperature to hold the temperature, i am unsure how to improve that. The TV02-Zigbee doesn't report any status for heating.

@j0nxs, as for the local calibration you're suggesting, this is partly available in the "self regulation" mode, specifically in the "slow" setting, which will adjust the target temperature based on the average difference between target temperature and measured temperature over the past 24 hours. I do think this feature is too hidden and should also be named "calibration offset" instead of self-regulation, but as there's a lot of optimisations that can be done here it might be better to first think about what we need and want and how it should be configurable by the users. For example I believe there are a couple of different time frames that all are relevant and should be available – currently you have to choose between slow and fast:

long term offset between target temp and measured temp due to calibration offsets of the TRV
multiple hour effects of overheating due to solar radiation, cooking,…
external weather: does someone want the internal target temperature lower or higher when it's very, very cold outside?
quick regulation: I cannot say anything about that, but I believe it's for electrical heating and buildings that do not store heat well.

Also I think these features might need different algorithms. The long term offset might be a candidate for EMA (exponential moving average), the overheating due to external effects might be handleable with the temperature slope updates that @jmcollin78 is considering for next weekend (especially the "slow" curve that looks at the changes over the past 60 minutes), the quick regulation might need a PI-algorithm as currently used.
If you have any proposal for parts of these concepts, please post them. I find it best to first draft a concept together, otherwise the code and configuration will get very messy over time due to all these features.

Hello @j0nxs,
Thank you so much for this PR.
For regulation, I'm totally aligned with @maia. A new feature called self-regulation have been introduced in 3.8.0 release and is now almost mature. A new release with important improvements have been release yesterday.
I am confident that with this and @maia improvements we will have something which is working fine without having to do more than this.
You can have  at a look at the release 4.1.0 releases yesterday and try the 'Strong regulation'. I'm confident that will solve your issue. Else of course we can help to tune the parameters.
Concerning the 'FeatureController' class for the UnderlayingClimates, I will have a look as this could be an interesting way to add features with clear separation of concern in the code.

Hello @j0nxs ,
Do you have tried the regulation mode ? It is now fully operational. It would be great if you could confirm it is working on your environement.
If yes I will close this PR because it is no more needed. Thanks anyway for this PR.

I just want to add, that the self regulation does not replace the local calibration. the PI algorithm cant work correctly if the TRV thinks it has a different temperature than what Versatile thermostat thinks the current temperature is. I have had this exact problem with VT, the VT regulation creates a +2degree offset because its way to cold, but the TRV thinks its already 2degree warmer so it still doesnt heat. I solved it by creating my own local calibration automation, which greatly improved the heating accuracy.

I just want to add, that the self regulation does not replace the local calibration. the PI algorithm cant work correctly if the TRV thinks it has a different temperature than what Versatile thermostat thinks the current temperature is. I have had this exact problem with VT, the VT regulation creates a +2degree offset because its way to cold, but the TRV thinks its already 2degree warmer so it still doesnt heat. I solved it by creating my own local calibration automation, which greatly improved the heating accuracy.

I've also had this issue and the current logic does not allow to implement a (long term) calibration offset. I started a discussion in #208 and would love to see your comments in that discussion, ideally also the automations you created, are you willing to share it? Thanks!

the PI algorithm cant work correctly if the TRV thinks it has a different temperature than what Versatile thermostat thinks the current temperature is

On the contrary I believe this will work with a Strong regulation. At each cycle, the error factor is incremented and so contribute to a self calculated offset. So if 2° is not enough, try with the strong regulation and you will up to 5° of regulation. This error is now persistent (thanks to @maia idea) so you have a dynamically adaptable offset with this solution.
Can you please give it a try ? You have to wait let's say 12h to have a correct initialisation.


[gdal] Update to 3.7.3

Removed patches which are part of upstream now.

3.8.0 is out now... maybe skip straight there?

3.8.0 is out now... maybe skip straight there?

No.


Raised an error when executing any ever-command

When executing any ever-command, such as ever sync, ever search, there is an error like below raised by VSCode, both in my PC and Mac:
Http Error: 411- Length Required, Check your ever config please.

Even after revoke a new token from yinxiang.com, there isn't a workaround solution.

After reinstall evermonkey in my win10 PC's VSCode, it recovered.
But in my Mac, even to reinstall this extension didn't work. It worked days ago and there is no change on VSCode.

This error thrown by evernote sdk. I have been using evermonkey on my Mac these days, it works well till now. I will keep an eye on this and try to find it out.
You can reinstall vscode to have a try.


Adding close preview feature

- Summary
Added the basic functionality (via a button) to hide/show the preview panel in the Entry Editor.  It mirrors the request publish functionality. This was created to address issue #113. (Note that it deals with the second level of integration @calavera mentioned in his most recent comment on the issue.)

- Test plan
Tested code using the local "example" page. Added button(s) which call a function which toggles the state (added to code as 'previewOpen'). Added a showOnlyEditor, which displays the editor fullscreen without the preview. Depending on the status of this.state.previewOpen (boolean) the main render function displays either the original JSX or calls the showOnlyEditor. Attached gif below demonstrates its functionality:

(Note: Different styling maybe needed but css is not my strong-suit, and I was having some issues using the linked styles file. Would appreciate further direction or having someone with artistic ability step in on this part.)

- Description for the changelog
Adds a Button allowing the preview pane to be toggled opened & closed.

- A picture of a cute animal (not mandatory but encouraged)
Same dog as last PR, different headgear.

@americool awesome contribution! 😎 We'll get this reviewed very soon.

@americool we should be able to wrap the existing PreviewPane in a ternary, use a single button for toggling, and handle visibility and button label text based on state. Any reason not to go that route?

I definitely wanted to conditionally render the preview pane originally, but I'm a little unclear on how to do it with the  component that's wrapping both the control pane and the preview pane. If we don't remove that component as well we simply end up rendering a blank pane next to the editor rather than have it go full screen. Any thoughts?

@erquhart I tried this locally with suggestion and confirmed that only hides the <PreviewPane /> and does not expand the editor. This is because the use of fixed width. I chatted with @americool and saw that conditionally rendering the defaultSize of the SplitPane, actually breaks dragging as well.

What are your thoughts on moving the showOnlyEditor and the content of the EntryEditor Return to their own component and render conditionally.
return this.state.previewOpen ? <ShowSplitPaneEditor {...this.props} /> : <ShowOnlyEditor {...this.props} />

If the above is not ideal, the other solution could be to rethink the CSS, maybe with flexbox (removing the fixed widths) which will adjust the width automatically.

@bdougie agreed, rendering conditionally makes sense, and I don't even think separate components are necessary, the swappable template portions can just be saved to variables right in the render function.
@americool sound good?

So I'd just finished writing a version with two components before reading this. Feel free to look into that code. I'll try to attempt a conditional render with save variables now. May have some questions along the way. Thanks for the feedback!

A cheap improvement that should probably be present at merge time is persisting the preview visibility state. We can follow the method used for persisting the rich text editor visual mode: https://github.com/netlify/netlify-cms/blob/a4d7622adead7e259caccdac5c97499d977de070/src/components/Widgets/MarkdownControl.js#L23

I may not understand how to appropriately save the swappable template portions into variables, because as I start to code it, looking at two separate consts that hold all the relative jsx, which are both already very long, will also each have to have all the destructed props passed in, along with the styles, classnames, and functions. It seems really long and ugly so I figure I must be approaching this wrong, because as it stands the separate files at least seem to look a lot more elegant. Happy to do it either way, but I want to make sure what I'm doing makes sense before I go down a foolish path.

@americool like this:
  render() {
    const {
        collection,
        entry,
        fields,
        fieldsMetaData,
        fieldsErrors,
        getAsset,
        onChange,
        onValidate,
        onAddAsset,
        onRemoveAsset,
        onCancelEdit,
    } = this.props;

    const controlClassName = `${ styles.controlPane } ${ this.state.showEventBlocker && styles.blocker }`;
    const previewClassName = `${ styles.previewPane } ${ this.state.showEventBlocker && styles.blocker }`;
    const editor = (

        <ControlPane
          collection={collection}
          entry={entry}
          fields={fields}
          fieldsMetaData={fieldsMetaData}
          fieldsErrors={fieldsErrors}
          getAsset={getAsset}
          onChange={onChange}
          onValidate={onValidate}
          onAddAsset={onAddAsset}
          onRemoveAsset={onRemoveAsset}
          ref={c => this.controlPaneRef = c} // eslint-disable-line
        />

    );
    const editorWithPreview = (
      <ScrollSync>

          <SplitPane
            defaultSize="50%"
            onDragStarted={this.handleSplitPaneDragStart}
            onDragFinished={this.handleSplitPaneDragFinished}
          >
            <ScrollSyncPane>
              {editor}
            </ScrollSyncPane>

              <PreviewPane
                collection={collection}
                entry={entry}
                fields={fields}
                fieldsMetaData={fieldsMetaData}
                getAsset={getAsset}
              />

          </SplitPane>

      </ScrollSync>
    );
    return (

        {this.state.showPreview ? editor : editorWithPreview}

          <Toolbar
            isPersisting={entry.get('isPersisting')}
            onPersist={this.handleOnPersist}
            onCancelEdit={onCancelEdit}
          />

    );
  }

Thanks, sorry about the confusion. I see the mistake I was making now, I wasn't creating the constants correctly which messed up my scope so I couldn't access all the other data easily.  (it also didn't occur to me to toss the editor component inside the editorwithpreview) Thanks for the help! I'll have something up soon!

I think at this point it's ready for further review.

Thanks for the feedback. The most recent commit should take care of those issues.


Setup analyzer with latest Flutter SDK

Also, clone only what's needed and target the correct commit

CC @isoos


Add popen3 and zone resource record iterator

This PR adds an implementation of popen3 to execute a process and attach stdin, stdout and stderr. It also extends the suite of unit tests to verify the call works as expected. For easy debugging of unit tests I also extended the cutest executable to accept a -r parameter to make it possible to run only tests that match the regular expression passed with the -r parameter.
Please consider pulling these changes.

I added a slightly rewritten version of the CreDNS zone resource record iterator and a test that demonstrates it works properly.

Added a unit test to verify SIGCHLD signals are properly captured if handled through libevent as udp/tcp handlers moved to libevent (as opposed to netio) since CreDNS. I actually just used this to see if it works as I expected, but since we have a libevent compatible implementation in mini_event.c, I figured I might as well leave the test in.


clarify supported parameters better in api server

There's a comment for "ignored or currently unsupported" parameters but I'm pretty sure some of them are supported (for instance, I think logprobs is supported?).
Also when I generate an openapi client off the openapi.json there are errors because of these.
So just delete the things that are unsupported or clarify that they're actually supported / provide more info on the existing support.

Just leaving some notes here for parameters i see in app.py
class Settings:

model - obviously supported, accepted (but ignored) by the API, action: do nothing
n_ctx, - obviously supported but not accepted by the API, action: do nothing
n_batch, - obviously supported but not accepted by the API, action: do nothing
n_threads, - obviously supported but not accepted by the API, action: do nothing
f16_kv, - obviously supported but not accepted by the API, action: do nothingg
f16_kv, - obviously supported but not accepted by the API, action: do nothingg

f16_kv, - obviously supported but not accepted by the API, action: do nothingg


Removed unnecessary conditional on resource for AADOAuth2AutoConfigur…

Pull request to remove the ConditionalOnResource for the aad.enable.config
I hit an issue earlier on today where by this file wasn't found on the classpath, when I created it in my own project it was suddenly found.
Given the other conditional checks for the likes of the tenant-id, I'm not 100% sure if this check adds any extra value.
Happy to discuss and would be really interested to know what if any extra benefit this additional check gives.
Thanks,
Chris

Can one of the admins verify this patch?

You could find the file here https://github.com/Azure/azure-sdk-for-java/tree/master/sdk/spring/azure-spring-boot-starter-active-directory/src/main/resources and when the aad starter is added to your project the file will be on your classpath.

Thanks @saragluna,
That doesn't really address my issue though.
What purpose does this check hold? is it purely to check that we have the ad starter on the classpath?
Given that both of project have the same dependencies:
https://github.com/Azure/azure-sdk-for-java/blob/master/sdk/spring/azure-spring-boot-starter-active-directory/pom.xml
https://github.com/Azure/azure-sdk-for-java/blob/master/sdk/spring/azure-spring-boot/pom.xml
and there are no code changes in the azure-spring-boot-starter-active-directory it seems defunct to bother checking for that file at all, particularly when as i've proven its just an extra blank file in peoples projects that doesn't really need to be there

The azure-spring-boot contains all sorts of auto-configuration code for components like aad, cosmos, or service bus etc. All dependencies defined here are optional which means when only such jar appears on the classpath will the auto configuration be activated. And the aad starter is the Spring Boot starter for aad integration, only containing a pom file which includes aad related dependencies.

Fair Enough,
It seems a bit odd to me to look for a resource inclusion rather than having a property but I get your point,
Thanks,
Chris