Lorenz issue with Param1 knob

Hello there and friendly greetings again!
First: sorry if I have not answered to my previous issue, I'll answer now if it is still useful.
It does not always draw out of bounds, only at certain parameters, like:
P1 = DEFAULT           T1 = DEFAULT
P2 = DEFAULT           T2 = 0
P3 = 0.132                T3 = 0.446
All the rest as DEFAULT
Another problem is that when all the parameter are DEFAULT, if I turn parameter P1 = 0 it seems that the whole module stop to work. I can twist any other knobs but I got no changes at all, both visually and audio. I have to CTRL+R the module to make it work again. I don't know if it is intended or if it is a bug.
If I turn
P1 = 1
P2 = 0.946
P3 = 0.095
All the T as DEFAULT, all the Filters as DEFAULT and all the Levels to the max (X = Y = Z = 10) it happens what you can see in the picture and consider that the cyan curve is static, not moving and there is no output sound I can hear.
All in all it seems (but I might be wrong, not knowing how the algorithm work) that the P1 knob/parameter work a little bit weird.

Hi Andrea,
Paul will come and tell you that he solved the out of screen rendering in the latest nightly, and I will tell you that the V1 version of the module reacted the same way , it's a very volatile module, it breaks up very easily, and Paul has not really changed any of the code.

yeah so the lorenz attractor (math) has plenty of parts of the space which are just collapsed to a single point not an orbit and that's what is happening here I presume. The parameters seem a bit mis-calibrated to me also but I'm hesitant to solve them because of other folks patches.
I suppose I could check a streaming version or some such thing and adjust them but that's more an "after 2.0.0" type thing.


IMPORTANT - Rendering Error in react-spline with Certain Scenes

Description: Since this morning, I, along with other users, have encountered a rendering issue when using react-spline with certain scenes, including the one linked below.
Url: https://app.spline.design/community/file/09a1967a-051f-4493-b214-9a87e7e53aec
Additionally, this issue occurs on the Spline.design website itself, where the scene fails to render correctly. This problem seems to affect only the Brave and Chrome browsers. The scene renders as expected in Safari.
LOG:
entry-server-routing.7a9d563f.js:3087 Uncaught (in promise) TypeError: Failed to execute 'uniform1fv' on 'WebGL2RenderingContext': The provided float value is non-finite.
    at _me.ime [as setValue] (entry-server-routing.7a9d563f.js:3087:12555)
    at Xb.upload (entry-server-routing.7a9d563f.js:3087:15800)
    at eo (entry-server-routing.7a9d563f.js:3145:78041)
    at JN.renderBufferDirect (entry-server-routing.7a9d563f.js:3145:66287)
    at ti (entry-server-routing.7a9d563f.js:3145:72211)
    at Ar (entry-server-routing.7a9d563f.js:3145:71747)
    at cr (entry-server-routing.7a9d563f.js:3145:70881)
    at JN.render (entry-server-routing.7a9d563f.js:3145:69380)
    at OpaquePass.render (pages_index.910bd624.js:2627:878)
    at MRTEffectComposer.render (pages_index.910bd624.js:2627:6334)
    at Post.render (pages_index.910bd624.js:2781:8104)
    at SplineRenderer.renderSplineScene (pages_index.910bd624.js:3187:7827)
    at rR.renderSingle (chunk-0428ada7.js:338:2143)
    at rR._renderFrames (chunk-0428ada7.js:338:8211)
    at rR.render (chunk-0428ada7.js:338:8060)
    at lQ.render (chunk-0428ada7.js:374:72221)
    at lQ.animate (chunk-0428ada7.js:374:67489)
    at lQ.start (chunk-0428ada7.js:374:71737)
    at lQ.setScene (chunk-0428ada7.js:374:72381)
    at Sj (chunk-0428ada7.js:374:75672)
    at chunk-0428ada7.js:882:62013

I upgrade @splinetool/runtime to 1.9.21 then it fixed


fix git-clang-format link

Update the link to git-clang-format (s/master/main).

CI fails due to #2474 but since this is not a functional change, I'm merging it.


complete JSON translations

JSON packet translator is incomplete:
https://github.com/gchux/cloud-run-tcpdump/blob/pcap-json/pcap-writer/pkg/transformer/json_translator.go

collaborators: @gmark3 , @itsmais

should be implemented in:
https://github.com/gchux/pcap-cli/blob/dev/v1.0.0/pkg/transformer/json_translator.go

done as per v1.0.0-RC55


remove group: prefix from snippets

it looks like the official schemas recommend to only use the user: prefix, but not the group: prefix. Maybe the snippets should conform to the schema?

Thanks for your feedback. Do you mean prefix for entity references? I can't find anything in the docs outlining this recommendation. In fact the Entity Reference documentation is using the group prefix in the example.


Deployment check can roll more pods than it thinks it has

time="2020-02-10T14:34:17Z" level=info msg="6 / 3 pods have been rolled."

should stop at:
time="2020-02-10T14:34:17Z" level=info msg="3 / 3 pods have been rolled."

Versions

Deployment Check 1.1.0

i think this is due to #303.  Because the eviction fails, two deployments spin up and fight over services (#302).  That results in the incorrect pod count here.


Bicep-generated files should include an autogenerated header

When a resource file is created in visual studio, a programming language-specific file is autogenerated for resource access.  At the top is a comment that looks something like this:
//------------------------------------------------------------------------------
//
//     This code was generated by a tool.
//     Runtime Version:4.0.30319.42000
//
//     Changes to this file may cause incorrect behavior and will be lost if
//     the code is regenerated.
//
//------------------------------------------------------------------------------
We should have something similar for bicep - a comment that says the file was autogenerated and the bicep version.
This discourages users from modifying the bicep file directly; it also (as @alex-frankel pointed out) allows the service to potentially collect telemetry on the # of deployments using compiled templates.

Discussed it at the team meeting today. The consensus appears to have template code generators use the top-level metadata property to store this information. This is the proposed schema:
"metadata": {
  "_generator": {
    "name": "<name of the code generator>",
    "version": "<version of the code generator>",
    "templateHash": "<template hash>"
  }
}

Considerations:

Discussed using a comment instead of a JSON property. We're not in favor of using meaningful comments due to their fragility and uneven support in JSON libraries across all the relevant platforms.
Template hash logic should reuse the existing template hash calculation logic that we already have in ARM telemetry and exposed in the API at https://github.com/Azure/azure-rest-api-specs/blob/8cef8014762a839e98f0aeaa57a0bbdb8982d3d4/specification/resources/resource-manager/Microsoft.Resources/stable/2020-10-01/resources.json#L4236
Template hash calculation should run on the entire content of the template except for the metadata._generator.templateHash property. This is technically a breaking change in ARM, but impact should be extremely low.
Also discussed adding a top-level multi-line comment with text similar to "This file is generated. Do not modify." This should be deferred until we fix bugs in line number handling in the runtime.

One thing that would be nice in the case of other generators is a way to add metadata around that generator's use, like a kind of user-agent string. That would help us understand where to make investments/who to optimise our generator for.
Whether that is just the _generator.name or some other field/object under _generator is something I'm certainly very flexible on.


Compatibility with azure orgs that do use the old url format is possible ?

Hi fellow comrades, hope this message finds you well...
Lately im asking myself if that is possible, I have been checking the code and id like to have that option. Is that a possibility?
urls like this: org.visualstudio.org

Hi @cdavidsc007 , in the latest versión you should be able to set the server property in the the step. Like this:
server: org.visualstudio.org
But you must check also API versión compatibility


<EMAIL_ADDRESS>Pact Binary Error when trying to run a test

I had an issue while trying to setup Pact provider when i try to run my tests here it follows
Software versions
Please provide at least OS and version of pact-js

OS: e.g. Mac OSX 10.11.5
Pact Node version:  10.14.0
Node Version: v16.11.1
Other Versions: Any other versions you think might be relevant (optional)

   "@pact-foundation/pact": "^9.16.5",
    "@pact-foundation/pact-mock-service": "^2.1.0",
    "@pact-foundation/pact-node": "^10.14.0",

Issue Checklist
Please confirm the following:

[x] I have upgraded to the latest
[ ] I have the read the FAQs in the Readme
[ ] I have triple checked, that there are no unhandled promises in my code
[ ] I have set my log level to debug and attached a log file showing the complete request/response cycle
[ ] For bonus points and virtual high fives, I have created a reproduceable git repository (see below) to illustrate the problem

Expected behaviour
It should create a server that runs the Pact provider and test all the cases
Actual behaviour
it gives me the error below:
[2021-11-04 15:02:50.945 +0000] INFO (97114 on MAC-CA-BELFOER2<EMAIL_ADDRESS>Creating Pact Server with options:
{"consumer":"customer-signup-gateway-consumer","cors":true,"dir":"/Users/belfoer1/Documents/oss-ui/packages/apps/signup/server/pact/pacts","host":"<IP_ADDRESS>","log":"/Users/belfoer1/Documents/oss-ui/packages/apps/signup/server/pact/logs","pactFileWriteMode":"update","port":3000,"provider":"customer-sign-up","spec":2,"ssl":false,"timeout":30000}
[2021-11-04 15:02:51.720 +0000] ERROR (97114 on MAC-CA-BELFOER2<EMAIL_ADDRESS>Pact Binary Error: /Users/belfoer1/Documents/oss-ui/node_modules/@pact-foundation/pact-node/standalone/darwin-1.88.77/pact/lib/ruby/lib/ruby/gems/2.2.0/gems/bundler-1.9.9/lib/bundler/shared_helpers.rb:78: warning: Insecure world writable dir /Users/belfoer1/Library/Android in PATH, mode 040777

[2021-11-04 15:02:52.066 +0000] ERROR (97114 on MAC-CA-BELFOER2<EMAIL_ADDRESS>Pact Binary Error: /Users/belfoer1/Documents/oss-ui/node_modules/@pact-foundation/pact-node/standalone/darwin-1.88.77/pact/lib/vendor/ruby/2.2.0/gems/pact-mock_service-3.9.0/lib/pact/mock_service/run.rb:75:in `initialize': Is a directory @ rb_sysopen - /Users/belfoer1/Documents/oss-ui/packages/apps/signup/server/pact/logs (Errno::EISDIR)
	from /Users/belfoer1/Documents/oss-ui/node_modules/@pact-foundation/pact-node/standalone/darwin-1.88.77/pact/lib/vendor/ruby/2.2.0/gems/pact-mock_service-3.9.0/lib/pact/mock_service/run.rb:75:in `open'
	from /Users/belfoer1/Documents/oss-ui/node_modules/@pact-foundation/pact-node/standalone/darwin-1.88.77/pact/lib/vendor/ruby/2.2.0/gems/pact-mock_service-3.9.0/lib/pact/mock_service/run.rb:75:in `open_log_file'
	from /Users/belfoer1/Documents/oss-ui/node_modules/@pact-foundation/pact-node/standalone/darwin-1.88.77/pact/lib/vendor/ruby/2.2.0/gems/pact-mock_service-3.9.0/lib/pact/mock_service/run.rb:69:in `service_options'
	from /Users/belfoer1/Documents/oss-ui/node_modules/@pact-foundation/pact-node/standalone/darwin-1.88.77/pact/lib/vendor/ruby/2.2.0/gems/pact-mock_service-3.9.0/lib/pact/mock_service/run.rb:37:in `mock_service'
	from /Users/belfoer1/Documents/oss-ui/node_modules/@pact-foundation/pact-node/standalone/darwin-1.88.77/pact/lib/vendor/ruby/2.2.0/gems/pact-mock_service-3.9.0/lib/pact/mock_service/run.rb:28:in `call'
	from /Users/belfoer1/Documents/oss-ui/node_modules/@pact-foundation/pact-node/standalone/darwin-1.88.77/pact/lib/vendor/ruby/2.2.0/gems/pact-mock_service-3.9.0/lib/pact/mock_service/run.rb:13:in `call'
	from /Users/belfoer1/Documents/oss-ui/node_modules/@pact-foundation/pact-node/standalone/darwin-1.88.77/pact/lib/vendor/ruby/2.2.0/gems/pact-mock_service-3.9.0/lib/pact/mock_service/cli.rb:32:in `service'
	from /Users/belfoer1/Documents/oss-ui/node_modules/@pact-foundation/pact-node/standalone/darwin-1.88.77/pact/lib/vendor/ruby/2.2.0/gems/thor-1.1.0/lib/thor/command.rb:27:in `run'
	from /Users/belfoer1/Documents/oss-ui/node_modules/@pact-foundation/pact-node/standalone/darwin-1.88.77/pact/lib/vendor/ruby/2.2.0/gems/thor-1.1.0/lib/thor/invocation.rb:127:in `invoke_command'
	from /Users/belfoer1/Documents/oss-ui/node_modules/@pact-foundation/pact-node/standalone/darwin-1.88.77/pact/lib/vendor/ruby/2.2.0/gems/thor-1.1.0/lib/thor.rb:392:in `dispatch'
	from /Users/belfoer1/Documents/oss-ui/node_modules/@pact-foundation/pact-node/standalone/darwin-1.88.77/pact/lib/vendor/ruby/2.2.0/gems/thor-1.1.0/lib/thor/base.rb:485:in `start'
	from /Users/belfoer1/Documents/oss-ui/node_modules/@pact-foundation/pact-node/standalone/darwin-1.88.77/pact/lib/app/pact-mock-service.rb:15:in `<main>'

[2021-11-04 15:02:52.080 +0000] WARN (97114 on MAC-CA-BELFOER2<EMAIL_ADDRESS>Pact exited with code 1.
[2021-11-04 15:02:52.080 +0000] INFO (97114 on MAC-CA-BELFOER2<EMAIL_ADDRESS>Removing Pact process with PID: 97235

Steps to reproduce
given code:
// eslint-disable-next-line import/no-extraneous-dependencies
import { Pact } from '@pact-foundation/pact';
import path from 'path';
import config from '../src/config';

const { PACT_CONSUMER, PACT_PROVIDER, PORT } = config();

global.provider = new Pact({
  cors: true,
  consumer: PACT_CONSUMER,
  provider: PACT_PROVIDER,
  pactfileWriteMode: 'update',
  port: parseInt(PORT, 10),
  log: path.resolve(process.cwd(), 'pact/logs'),
  dir: path.resolve(process.cwd(), 'pact/pacts'),
  logLevel: 'info',
  spec: 2,
});

/* eslint jest/expect-expect: "off" */
import request from 'supertest';
import { Matchers } from '@pact-foundation/pact';
import express from 'express';
import signupRoutes from './signup-routes';

const app = express();

app.use(express.json());
app.use(express.urlencoded({ extended: false }));
app.use(signupRoutes);

const newAccountMock = {
  accountUuid: 'mock-hash',
  realm: 'mock',
  userUuid: 'mock-hash',
};

const account = {
  emailAddress<EMAIL_ADDRESS>  password: 'myPWD@',
  firstName: 'jhon',
  lastName: 'PACT',
  customerName: 'company',
};

describe('pacts', () => {
  afterEach(() => global.provider.verify());
  describe('adding new customer', () => {
    it('should add a new customer', async () => {
      console.log('IT', global.provider.mockService);
      await global.provider.addInteraction({
        state: 'Add a new customer',
        uponReceiving: 'A request for API ',
        withRequest: {
          method: 'POST',
          path: '/signup',
          body: account,
        },
        willRespondWith: {
          status: 201,
          body: Matchers.eachLike(newAccountMock),
        },
      });
      await request(app)
        .post('/signup')
        .send(account)
        .set('Accept', 'application/json')
        .expect('Content-Type', /json/)
        .expect(201);
      console.log('am i here?????');
    });
  });
});

Duplicate of: https://github.com/pact-foundation/pact-js/issues/766


Update SDK to allow for updated grant flow

This updates adds to the exist SDK grant token functionality.

I'm good with this. We're gonna want to do a separate commit to master to bump the version. We can do that after we merge this in


Override call type only available for top-level call expressions

The "override call type" action seems to be unavailable for tail calls and calls nested in expressions (e.g. some_func() & 0xf).

Workaround: switch to MLIL and the Override Call Type option should be available there.

Fixed in 3.5.4328


fx/biquad: Add missing includes

Fixes the following build failure:
In file included from test.cpp:
q/q_lib/include/q/fx/biquad.hpp:72:24: error: unknown type name 'frequency'
         config_biquad(frequency f, std::uint32_t sps)
                       ^
q/q_lib/include/q/fx/biquad.hpp:73:20: error: no matching literal operator for call to 'operator""_pi' with argument of type 'unsigned long long' or 'const char *', and no matching literal operator template
          : omega(2_pi * as_double(f) / sps)

Thanks!
There are some conflicts, if change the base to develop. Can you see what's going on? PRs should be done on the develop branch, allowed to stabilize, then merged to develop.

Done


Feature/actions test

Made a mistake creating the PR. Closing.

Made a mistake creating the PR. Closing.


Implement debug suffix config loader

IMPORTANT: Where possible all PRs must be linked to a Github issue
Fixes #1269
Checklist

[ ] I have written Unit tests for any new feature(s) and edge cases for bug fixes
[ ] I have added any strings visible on UI components to the strings.xml file
[ ] I have updated the  CHANGELOG.md file for any notable changes to the codebase
[ ] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the project's style guide
[ ] I have built and run the fhircore app to verify my change fixes the issue and/or does not break the app

This is pending, there are a few bugs


Disable Ticket transactions and tests

Disable processing of ticket-related transactions -- attempting to use them will return the new temDISABLED error code.
Reviews: @JoelKatz, @vinniefalco

Obviously correct
:+1:

Merged as 0ccdea3cd86d41958f54aa6f4208ccc6b78f2094.


Refactor fuzziness interface on query builders

Signed-off-by: noCharger<EMAIL_ADDRESS>Description
Refactor fuzziness interface on query builders
https://github.com/opensearch-project/OpenSearch/issues/3749 is more of a matter of method signature consistency than of a bug. Here's the trade-off between taking Fuzziness class everywhere VS take Object everywhere (discussed with @msfroh offline):
Take Fuzziness class everywhere (QueryStringQueryBuilder way) - Recommanded

Pros:

Complie-time validation of an input value that must be of the Fuzziness typed object.
API is more self-documenting (since an API that takes Object could mean anything).

Cons:

Requires code change by REST high-level client users who were passing a string or number (ex: "Auto", 0, 1, 2)
REST high-level client users must create Fuzziness object by calling Fuzziness.build() or invoke static variables like Fuzziness.ZERO, which is on a class marked @opensearch.internal

Take Object everywhere (MultiMatchQueryBuilder way)

Pros:

Backward-compatible with existing behavior.(If someone is already passing a Fuzziness , it will still work based on Fuzziness.build() implementation)
Reduces need for users to use Fuzziness.build() directly, which is good since it’s marked @opensearch.internal.

Cons:

API is less self-documenting.
Makes it easier to pass invalid values that only show up at runtime (The querybuilders also marked @opensearch.internal so not recommanded to call it directly from users)
From @reta: using Object is very discouraging it reads literally "any reference" (in Java).

Issues Resolved
https://github.com/opensearch-project/OpenSearch/issues/3749
Check List

[x] New functionality includes testing.

[x] All tests pass

[x] Commits are signed per the DCO using --signoff
[x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@dblock @nknize any thoughts?

Hey @noCharger!  This looks like it needs a rebase because of changelog conflicts.  Can you do that so we can merge it in?  Thanks!


Possible Bug in Video Challenges

Issue Description

I'm having an issue with several of the Video Challenges in the Computer Basics section.  Lessons 1 through 8 have all logged properly (i.e. the links have faded out, they are check-marked, the brownie point was logged, etc.) However I've completed the remainder of the video lessons in the Computer Basics section a few times now (from "Computer Basics: Types of Computers" to "Computer Security"), and for some reason none of them have logged.
Sections "The DOM" and "Javascript Lingo" have both logged successfully, however, they were completed before I had finished the "Computer Basics" section.
I have tried both completing the challenges with playing the video and without playing the video, but get the same result.  Any challenge in the "Video Challenges" section now refuse to be logged as completed.
Browser Information

Browser Name, Version: Google Chrome 49.0.2623.112 (Official Build) (64-bit)
Operating System: Chrome OS
Mobile, Desktop, or Tablet: Lenovo Ideapad N20P 11.6 inch touchscreen Chromebook

Possible the same issue as this one: https://github.com/FreeCodeCamp/FreeCodeCamp/issues/8463

Closing in favor of #8455 (Oldest logged issue for this)


Update Bare development room link

The current link is to the old room which is abandoned because the admin account was lost. The new room is currently titled, "Bare Developement 🐻 (new)". @mafintosh is the admin of the new room.
I don't have the permissions for the permalink but here's a 2 week expiring link:
pear://keet/yrbogd9pbhubjow4md4atbb3tyswjjw776hn8bneh991k79zxz199ffwahgyfx3qkyk4pahkmnj8wc1wmmanumf7zikq8mhp4jh1honzouf4eo5g

@mafintosh would you mind create a non-expiry link for the new Bare Developement 🐻 room?

Got a more recent link but looks like it could still expire: pear://keet/yrbixd7rt6tcxuk44hcqq6mwxtgu157zc57jcwm1fy6tx5q5f8gsp8fwahgyfx3qkyk4pahkmnj8wc1wmmanumf7zikq8mhp4jh1honzotazwsug


Chandanagirish1998 madlibs

Values for the placeholders 6-10

There is a merge conflict. @chandanagirish1998 can you initiate pull from your end and resolve the merge conflict. After making the changes that resolve the conflict update the pr.

Conflicts are merged

Since atharva's branch has been merged into main, there is a conflict. Repull and make required changes


[CLI]: wandb.sagemaker_auth(path="source_dir") throws FileNotFoundError: [Errno 2] No such file or directory: 'source_dir/secrets.env'

Describe the bug

I am running the WandB and AWS Sagemaker integration example. I'm following the documentation from here https://github.com/wandb/examples/blob/master/examples/pytorch/pytorch-cifar10-sagemaker/train.ipynb

import os
import sys
import wandb
import sagemaker

wandb.login()
sagemaker_session = sagemaker.Session()

bucket = sagemaker_session.default_bucket()
prefix = 'sagemaker/pytorch-cifar10'

# If you are running this outside of a sagemaker notebook, you must set SAGEMAKER_ROLE
role = os.getenv('SAGEMAKER_ROLE') or sagemaker.get_execution_role()
wandb.sagemaker_auth(path="source")

FileNotFoundError                         Traceback (most recent call last)
<ipython-input-10-9438fd54dc0f> in <module>
      6 # If you are running this outside of a sagemaker notebook, you must set SAGEMAKER_ROLE
      7 role = os.getenv('SAGEMAKER_ROLE') or sagemaker.get_execution_role()
----> 8 wandb.sagemaker_auth(path="source_dir")
      9 inputs = "s3://{}/{}".format(bucket, prefix)

/opt/conda/lib/python3.8/site-packages/wandb/integration/sagemaker/auth.py in sagemaker_auth(overrides, path, api_key)
     25         )
     26     overrides[env.API_KEY] = api_key
---> 27     with open(os.path.join(path, "secrets.env"), "w") as file:
     28         for k, v in overrides.items():
     29             file.write(f"{k}={v}\n")

FileNotFoundError: [Errno 2] No such file or directory: 'source_dir/secrets.env'

Additional Files
No response
Environment
WandB version: 0.13.3
OS: AWS Sagemaker Notebook Instance; Linux 4.14.290-217.505.amzn2.x86_64: Image: Pytorch 1.10 and Python 3.8 CPU optimized
Python version: 3.8.10
Versions of relevant libraries:
Additional Context
No response

Hey @bandhakavi,
Looks like source_dir does not exist. Could you try running mkdir source_dir and try running this again?
Thanks,
Ramit

Hi @ramit-wandb, yes I tried that too and its the same error. However, adding os.environ["WANDB_API_KEY"] = YOUR_KEY_HERE before I call wandb.sagemaker_auth(path="source") created the secrets.env file. I was able to proceed from there. Here's the link to the document https://docs.wandb.ai/guides/track/launch


LibWeb: Incorrect referrer header

This is a copy of https://github.com/SerenityOS/serenity/issues/23255, the issue is still relevant. We still get a 403 in LB, but shouldn't.

Given this URL: https://fonts.wod-game.de/java_font_renderer/render?skin=skin-8&profil=font_menu-1-hovered&text=Ladybird 1
Opening this in a browser should show you an image with the text "Ladybird 1".
In Ladybird though we only get the "image did not load" box (because the remote has a referrer filter).
I debugged this on the server side and we can see a clear problem here:
Firefox:
95.90.204.x - - [19/Feb/2024:09:30:04 +0100] "GET /java_font_renderer/render?skin=skin-8&profil=font_menu-1-hovered&text=Ladybird%201 HTTP/1.1" 200 8741 "-" "Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:123.0) Gecko/20100101 Firefox/123.0"

Ladybird:
95.90.204.x - - [19/Feb/2024:09:31:22 +0100] "GET /java_font_renderer/render?skin=skin-8&profil=font_menu-1-hovered&text=Ladybird%201 HTTP/1.1" 200 8741 "-" "Mozilla/5.0 (Linux; x86_64) Ladybird/1.0"
95.90.204.x - - [19/Feb/2024:09:31:22 +0100] "GET /java_font_renderer/render?skin=skin-8&profil=font_menu-1-hovered&text=Ladybird%201 HTTP/1.1" 403 746 "https://fonts.wod-game.de/java_font_renderer/render?skin=skin-8&profil=font_menu-1-hovered&text=Ladybird%201" "Mozilla/5.0 (Linux; x86_64) Ladybird/1.0"
95.90.204.x - - [19/Feb/2024:09:31:22 +0100] "GET /favicon.ico HTTP/1.1" 404 547 "https://fonts.wod-game.de/java_font_renderer/render?skin=skin-8&profil=font_menu-1-hovered&text=Ladybird%201" "Mozilla/5.0 (Linux; x86_64) Ladybird/1.0"

These logs are produced when opening the site in a new tab. We should only see one request, or if we need two for whatever reason we should not add that referer there.
More context might be found in this and following messages: https://discord.com/channels/830522505605283862/830525031720943627/1209048373655773205

#360 (dupe) provides another way to replicate.

OP's link has rotted, and trying e.g. ladybird requestheaders.dev now, it seems the header isn't sent. Is this fixed?

That link should absolutely work. It is used many many times each day. Will check how LB behaves nowadays tomorrow.

You can also see the three requests with a python3 -m http.server (no logged referer though):
<IP_ADDRESS> - - [29/Oct/2024 13:28:48] "GET /classes.png HTTP/1.1" 200 -
<IP_ADDRESS> - - [29/Oct/2024 13:28:48] "GET /classes.png HTTP/1.1" 200 -
<IP_ADDRESS> - - [29/Oct/2024 13:28:48] code 404, message File not found
<IP_ADDRESS> - - [29/Oct/2024 13:28:48] "GET /favicon.ico HTTP/1.1" 404 -

Very importantly, this behaviour does not happen with text (or HTML) files. It only gappens with images!


Error with FontLoader on NNBD preview

While attempting to migrate our packages to NNBD (https://github.com/aboutyou/dart_packages/pull/165), I ran into an issue with a screenshot tests which uses custom fonts.
Steps to Reproduce
In a test file load a custom font, like:
import 'dart:io';
import 'dart:typed_data';

import 'package:flutter/cupertino.dart';
import 'package:flutter/material.dart';
import 'package:flutter/services.dart';
import 'package:flutter_test/flutter_test.dart';

// The tests are only run on macOS system, on other systems they will be skipped
// On other systems small differences can lead to failing goldens (e.g. text rendering)
Future<void> main() async {
  // Using Roboto here instead of Apple's San Francisco fonts, due to licensing issues
  //
  // Roboto is then remapped to the San Francisco's font name, so it gets picked up while running the tests
  // (`fontFamilyFallback` didn't work in tests and `Ahem` (block) font would be used instead)
  final fontData = File.fromUri(Uri.parse(
          '/Users/timm/Sources/dart_packages/packages/sign_in_with_apple/assets/fonts/Roboto.ttf'))
      .readAsBytes()
      .then((bytes) => ByteData.view(Uint8List.fromList(bytes).buffer));
  final fontLoader = FontLoader('.SF Pro Text')..addFont(fontData);
  await fontLoader.load();

  setUp(() {
    final binding = TestWidgetsFlutterBinding.ensureInitialized()
        // ignore: avoid_as
        as TestWidgetsFlutterBinding;

    binding.window.devicePixelRatioTestValue = 3;
    binding.window.physicalSizeTestValue = Size(300, 100) * 3;
  });

  testWidgets(
    'Test',
    (tester) async {},
  );
}

Run flutter test --enable-experiment=non-nullable .

Expected results: Font is loaded and available during tests
Actual results: Error in loadFontFromList: "type 'Null' is not a subtype of type 'String' of 'function result'"

Logs

00:10 +0 -1: loading /Users/timm/Sources/dart_packages/packages/sign_in_with_apple/test/x.dart [E]
  Failed to load "/Users/timm/Sources/dart_packages/packages/sign_in_with_apple/test/x.dart": type 'Null' is not a subtype of type 'String' of 'function result'
  dart:ui                                              loadFontFromList
  package:flutter/src/services/font_loader.dart 75:12  FontLoader.loadFont
  package:flutter/src/services/font_loader.dart 62:33  FontLoader.load.<fn>.<fn>

00:10 +0 -1: Some tests failed.
timm@Timms-MacBook-Pro sign_in_with_apple % ~/Downloads/flutter_null-safety/bin/flutter test --verbose --enable-experiment=non-nullable test/x.dart
[ +146 ms] executing: [/Users/timm/Downloads/flutter_null-safety/] git -c log.showSignature=false log -n 1 --pretty=format:%H
[  +85 ms] Exit code 0 from: git -c log.showSignature=false log -n 1 --pretty=format:%H
[        ] 2783f8e2e14efec8b7e08f668dde61c40d128c24
[        ] executing: [/Users/timm/Downloads/flutter_null-safety/] git tag --points-at HEAD
[  +21 ms] Exit code 0 from: git tag --points-at HEAD
[        ] 1.24.0-3.0.pre
[  +12 ms] executing: [/Users/timm/Downloads/flutter_null-safety/] git rev-parse --abbrev-ref --symbolic @{u}
[  +12 ms] Exit code 128 from: git rev-parse --abbrev-ref --symbolic @{u}
[        ] fatal: HEAD does not point to a branch
[  +23 ms] executing: [/Users/timm/Downloads/flutter_null-safety/] git rev-parse --abbrev-ref HEAD
[  +12 ms] Exit code 0 from: git rev-parse --abbrev-ref HEAD
[        ] HEAD
[  +44 ms] Artifact Instance of 'AndroidMavenArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterWebSdk' is not required, skipping update.
[   +3 ms] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[  +44 ms] Artifact Instance of 'MaterialFonts' is not required, skipping update.
[        ] Artifact Instance of 'GradleWrapper' is not required, skipping update.
[        ] Artifact Instance of 'AndroidMavenArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterWebSdk' is not required, skipping update.
[        ] Artifact Instance of 'FlutterSdk' is not required, skipping update.
[        ] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FontSubsetArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'PubDependencies' is not required, skipping update.
[ +102 ms] running test package with arguments: [-r, compact, --concurrency=6, --, /Users/timm/Sources/dart_packages/packages/sign_in_with_apple/test/x.dart]
00:00 +0: loading /Users/timm/Sources/dart_packages/packages/sign_in_with_apple/test/x.dart                                                                                [ +297 ms] test 0: starting test /Users/timm/Sources/dart_packages/packages/sign_in_with_apple/test/x.dart
[  +48 ms] test 0: starting shell process
[  +13 ms] Stopping scan for flutter_test_config.dart; found project root at /Users/timm/Sources/dart_packages/packages/sign_in_with_apple
[   +7 ms] Compiler will use the following file as its incremental dill file: /var/folders/qs/zb6n9xh17w55g_46b4tyblfc0000gn/T/flutter_tools.xe9Rhq/flutter_test_compiler.y9Hzbh/output.dill
[        ] Listening to compiler controller...
[   +9 ms] Compiling file:///var/folders/qs/zb6n9xh17w55g_46b4tyblfc0000gn/T/flutter_tools.xe9Rhq/flutter_test_listener.bwnIbD/listener.dart
[  +13 ms] /Users/timm/Downloads/flutter_null-safety/bin/cache/dart-sdk/bin/dart --disable-dart-dev /Users/timm/Downloads/flutter_null-safety/bin/cache/artifacts/engine/darwin-x64/frontend_server.dart.snapshot --sdk-root
/Users/timm/Downloads/flutter_null-safety/bin/cache/artifacts/engine/common/flutter_patched_sdk/ --incremental --target=flutter --debugger-module-names --experimental-emit-debug-metadata -Ddart.developer.causal_async_stacks=true --output-dill
/var/folders/qs/zb6n9xh17w55g_46b4tyblfc0000gn/T/flutter_tools.xe9Rhq/flutter_test_compiler.y9Hzbh/output.dill --packages /Users/timm/Sources/dart_packages/packages/sign_in_with_apple/.packages -Ddart.vm.profile=false -Ddart.vm.product=false --enable-asserts --track-widget-creation --initialize-from-dill
/Users/timm/Sources/dart_packages/packages/sign_in_with_apple/build/testfile.dill.track.dill --enable-experiment=non-nullable
[ +108 ms] <- compile file:///var/folders/qs/zb6n9xh17w55g_46b4tyblfc0000gn/T/flutter_tools.xe9Rhq/flutter_test_listener.bwnIbD/listener.dart
00:08 +0: loading /Users/timm/Sources/dart_packages/packages/sign_in_with_apple/test/x.dart                                                                                [+8510 ms] <- accept
[        ] <- reset
[        ] Compiling /var/folders/qs/zb6n9xh17w55g_46b4tyblfc0000gn/T/flutter_tools.xe9Rhq/flutter_test_listener.bwnIbD/listener.dart took 8631ms
[   +2 ms] /Users/timm/Downloads/flutter_null-safety/bin/cache/artifacts/engine/darwin-x64/flutter_tester --disable-observatory --enable-checked-mode --verify-entry-points --enable-software-rendering --skia-deterministic-rendering --enable-dart-profiling --non-interactive --use-test-fonts
--packages=/Users/timm/Sources/dart_packages/packages/sign_in_with_apple/.packages /var/folders/qs/zb6n9xh17w55g_46b4tyblfc0000gn/T/flutter_tools.xe9Rhq/flutter_test_listener.bwnIbD/listener.dart.dill
[   +2 ms] Using this directory for fonts configuration: /var/folders/qs/zb6n9xh17w55g_46b4tyblfc0000gn/T/flutter_tools.xe9Rhq/flutter_test_fonts.YC5MC9
[   +5 ms] test 0: awaiting initial result for pid 14774
00:09 +0: loading /Users/timm/Sources/dart_packages/packages/sign_in_with_apple/test/x.dart                                                                                [ +678 ms] test 0: process with pid 14774 connected to test harness
[   +1 ms] test 0: awaiting test result for pid 14774
00:09 +0 -1: loading /Users/timm/Sources/dart_packages/packages/sign_in_with_apple/test/x.dart [E]
  Failed to load "/Users/timm/Sources/dart_packages/packages/sign_in_with_apple/test/x.dart": type 'Null' is not a subtype of type 'String' of 'function result'
  dart:ui                                              loadFontFromList
  package:flutter/src/services/font_loader.dart 75:12  FontLoader.loadFont
  package:flutter/src/services/font_loader.dart 62:33  FontLoader.load.<fn>.<fn>

[ +132 ms] test 0: process with pid 14774 no longer needed by test harness
[        ] test 0: cleaning up...
[        ] test 0: ensuring end-of-process for shell
00:09 +0 -1: Some tests failed.
[  +13 ms] Deleting /var/folders/qs/zb6n9xh17w55g_46b4tyblfc0000gn/T/flutter_tools.xe9Rhq/flutter_test_compiler.y9Hzbh...
[   +3 ms] killing pid 14742
[   +4 ms] test 0: deleting temporary directory
[   +3 ms] test 0: shutting down test harness socket server
[   +3 ms] test 0: finished
[  +54 ms] Deleting /var/folders/qs/zb6n9xh17w55g_46b4tyblfc0000gn/T/flutter_tools.xe9Rhq/flutter_test_fonts.YC5MC9...
[   +6 ms] test package returned with exit code 1
[  +15 ms] "flutter test" took 10,095ms.
[   +7 ms]
           #0      throwToolExit (package:flutter_tools/src/base/common.dart:14:3)
           #1      TestCommand.runCommand (package:flutter_tools/src/commands/test.dart:308:7)
           <asynchronous suspension>
           #2      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:1033:18)
           <asynchronous suspension>
           #3      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:892:33)
           <asynchronous suspension>
           #4      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart)
           #5      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:29)
           #6      _rootRun (dart:async/zone.dart:1190:13)
           #7      _CustomZone.run (dart:async/zone.dart:1093:19)
           #8      _runZoned (dart:async/zone.dart:1630:10)
           #9      runZoned (dart:async/zone.dart:1550:10)
           #10     AppContext.run (package:flutter_tools/src/base/context.dart:149:18)
           #11     FlutterCommand.run (package:flutter_tools/src/runner/flutter_command.dart:881:20)
           #12     CommandRunner.runCommand (package:args/command_runner.dart:197:27)
           #13     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:335:21)
           #14     _rootRunUnary (dart:async/zone.dart:1198:47)
           #15     _CustomZone.runUnary (dart:async/zone.dart:1100:19)
           #16     _FutureListener.handleValue (dart:async/future_impl.dart:143:18)
           #17     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:696:45)
           #18     Future._propagateToListeners (dart:async/future_impl.dart:725:32)
           #19     Future._completeWithValue (dart:async/future_impl.dart:529:5)
           #20     Future._asyncCompleteWithValue.<anonymous closure> (dart:async/future_impl.dart:567:7)
           #21     _rootRun (dart:async/zone.dart:1190:13)
           #22     _CustomZone.run (dart:async/zone.dart:1093:19)
           #23     _CustomZone.runGuarded (dart:async/zone.dart:997:7)
           #24     _CustomZone.bindCallbackGuarded.<anonymous closure> (dart:async/zone.dart:1037:23)
           #25     _microtaskLoop (dart:async/schedule_microtask.dart:41:21)
           #26     _startMicrotaskLoop (dart:async/schedule_microtask.dart:50:5)
           #27     _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:118:13)
           #28     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:169:5)

[   +7 ms] ensureAnalyticsSent: 0ms
[   +1 ms] Running shutdown hooks
[        ] Shutdown hook priority 4
[        ] Shutdown hooks complete
[   +1 ms] exiting with code 1

No issues found! (ran in 15.6s; 15 public members lack documentation)

[✓] Flutter (Channel unknown, 1.24.0-3.0.pre, on Mac OS X 10.15.7 19H2 darwin-x64, locale en-DE)
    • Flutter version 1.24.0-3.0.pre at /Users/timm/Downloads/flutter_null-safety
    • Framework revision 2783f8e2e1 (2 weeks ago), 2020-10-22 09:36:06 -0700
    • Engine revision defa8be2b1
    • Dart version 2.11.0 (build 2.11.0-242.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/timm/Library/Android/sdk
    • Platform android-29, build-tools 29.0.2
    • ANDROID_HOME = /Users/timm/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.4.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.4.1, Build version 11E503a
    • CocoaPods version 1.9.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 3.5)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)

[✓] VS Code
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.16.0

[✓] Connected device (3 available)
    • macOS (desktop)  • macos      • darwin-x64     • Mac OS X 10.15.7 19H2 darwin-x64
    • Web Server (web) • web-server • web-javascript • Flutter Tools
    • Chrome (web)     • chrome     • web-javascript • Google Chrome 86.0.4240.183

• No issues found!

Fixed with https://github.com/flutter/engine/pull/22373

@tp the next dev release of flutter should have this fix https://github.com/flutter/engine/commit/7ab39127ee253f2c98013a7f2cf4344649dfaa56

Waiting for https://github.com/flutter/flutter/commit/78778d291bbdc538d6d4fb2e59fe1caa4112cb94 to be tagged...


Upgrade compiler: GCC-11 vs Clang-17

Sample perf for both on models_perf pipeline and compare resulting Throughput metric. Higher is better.
GCC-13 is not supported as default until Ubuntu 24.04. (Currently on Ubuntu 20.04 and 22.04)
The sheet below contains perf results averaged across 5 runs:
https://docs.google.com/spreadsheets/d/1kIPlO52QY0MgqkPJ0w9TZ_iWp0iB5VouRrGYW5tRDOM/edit#gid=0
CC: @TT-billteng
As observed from the sheet above, Clang17 outperforms GCC11 on every relevant metric. Most importantly there are noticeable gains for Throughput

https://github.com/tenstorrent/tt-metal/pull/8516 pushed


update pod watcher to handle all events occurring before the pod watcher is started

Changes
This the the PR I promised in https://github.com/shipwright-io/cli/pull/89#issuecomment-1015473951 around the outcome of the discussion @otaviof and I around the admittedly very small timing window around missing pod events when we instantiate the Follower after a BuildRun is started, which was desired in the case where we utilized the feature to auto generate the BuildRun name.
This conceptually is analogous to the "relist" concept you see in k8s controllers to address the possibility of missing watch events.
/kind cleanup
Submitter Checklist

[ /] Includes tests if functionality changed/was added
[n/a ] Includes docs if changes are user-facing
[ /] Set a kind label on this PR
[/ ] Release notes block has been filled in, or marked NONE

See the contributor guide
for details on coding conventions, github and prow interactions, and the code review process.
Release Notes
Add a form of relist on the pod watcher to help provide diagnostics for BuildRuns that fail very quickly

/approve

/assign @otaviof

the build controller rollout failed in the e2es:
# Waiting for Build Controller rollout...
Waiting for deployment "shipwright-build-controller" rollout to finish: 0 of 1 updated replicas are available...
error: timed out waiting for the condition
make: *** [Makefile:63: install-shipwright] Error 1

hopefully it is just a flake

ok e2e's all green now

/lgtm
Thanks, @gabemontero 👍

the build controller rollout failed in the e2es:
# Waiting for Build Controller rollout...
Waiting for deployment "shipwright-build-controller" rollout to finish: 0 of 1 updated replicas are available...
error: timed out waiting for the condition
make: *** [Makefile:63: install-shipwright] Error 1

hopefully it is just a flake

Most probably it's. We have a configurable timeout now, we may want to consider a higher one. But lets see if it goes away 🤞


feat: Implement Drawer Navigation

Replaces EventListActivity with EventListFragment
Adds MainAcivity which hosts all the fragments with Navigation Drawer Support

fixes #148

@iamareebjamal @rohanagarwal94 @Shailesh351 @arp95 Please review.

A lot of changes in MainActivity

Yeah, fragments handling logic need to be improved. Please do suggest. @iamareebjamal

@iamareebjamal waiting for your suggestions on MainActivity.

Yes @mejariamol Was busy with 3 PRs yesterday. Will surely take a look today

@mejariamol There is some binding error trying it locally. Can you please take a look?

There's a memory leak in AttendeeFragment
The method of updating attendee is convoluted again. Getting item position of the attendee and updating it is bad
Events -> Dashboard and other interactions are flat. Meaning that if you press back, it exits the app, whereas user expects that it will take me back to events page. I'll take a look at EventBrite app and suggest what's the better approach, but current one is confusing
Keep the navigation header light grey and empty for now as the current info isn't completely shown and the PR is already huge
Don't use dividers in Navigation Drawer. It's non standard and use icons and support selected menu item o that user is aware on which section he is

These are just some UI related points I recommend. As this is complete UI change, this may take long to get right.

Working on it.

[ ] There's a memory leak in AttendeeFragment
[ ] The method of updating attendee is convoluted again. Getting item position of the attendee and updating it is bad
[ ] Events -> Dashboard and other interactions are flat. Meaning that if you press back, it exits the app, whereas user expects that it will take me back to events page.
[ ] Keep the navigation header light grey and empty for now as the current info isn't completely shown and the PR is already huge
[ ] Don't use dividers in Navigation Drawer. It's non standard and use icons and support selected menu item o that user is aware on which section he is
[ ] Log Out should be in another section of navigation drawer

@iamareebjamal
I haven't used NavigationView (provided by support library) for side drawer, thinking that we can customize it as per our need. Is it fine or should we use support library's NavigationView? I am not sure if it enables adding menu items dynamically.

@iamareebjamal

Looks perfect

About the data. I think NavigationView supports it. I'll confirm that

I have used ListView for navigation drawer menus.

Ok, let's go with ListView only, but can we use DataBinding? I saw that you were previously manually loading stuff

@iamareebjamal please review

@iamareebjamal please review. have tried to keep things simpler.

@iamareebjamal have used databinding for view inflation in event list fragment. Please review. :)

OK, let's merge it and then solve the complexities after. But first, let's wait for the simpler PRs to get merged. Sounds good?

@mejariamol please resolve conflicts

@iamareebjamal please review.

@iamareebjamal please review :smile:

@niranjan94 please merge


Bugfix/light dark mode toggle

Bug
When setting an app to use dark mode, and then refreshing the page, the app reverted to light mode, and the localStorage value was overwritten to light.
Resolution
We made some changes recently so that controlling the theme of the ThemeProvider using the theme property took priority over reading the localStorage value. This is required to support testing of light/dark mode, and is also how storybook syncs the dark/light mode with MUI. However, the default value for the controlled theme was "light" so it now always took priority. I've removed the default so that the theme is now decided using the following steps:

Is the theme being controlled? If so, use this value, otherwise...
Is the theme stored in localStorage? If so, use this value, otherwise...
Default to light.

Changes

Resolved bug as above.
Migrated component and hook to typescript.
Corrected the way we override component styles in the MUI Theme. Typescript highlighted that we were using an outdated API. We now use component.componentName.styleOverrides, rather than overrides.componentName
Added unit tests to test all of the use cases for ThemeProvider.

Not included
In Remix apps, we see a flicker from light mode to dark mode when refreshing in dark mode. This is because these are server-side rendered, but we cannot know the users preference until the page loads in the browser. I am writing a story to address in YouTrack.
Testing notes

Check that everything looks correct in storybook.
Check that the tests pass.
Install this branch in an application, and test that you can toggle to dark-mode, and then refresh the page. You should still be in dark-mode. Previously you would be reset to light mode. Confirm in the browser window, and also by looking at local storage.

https://user-images.githubusercontent.com/27866636/234552507-39ca0e31-f2bb-406a-88b7-6ddd51c2d00d.mov

Release notes:

Resolved bug with ThemeProvider where any page refresh resulted in the theme reverting to "light" mode.
Migrated ThemeProvider and useTheme to typescript.

Author checklist before assigning a reviewer

[x] Reviewed my own code-diff.
[x] Assigned to me.
[x] Relevant tags added.
[x] Appropriate tests have been added.
[x] Lint and test workflows pass.

LGTM. You can merge when you are ready.


'Please use yarn to install dependencies.' when using yarn

Trying to install dependencies with yarn, which is installed globally via homebrew.
Yarn works perfectly for other projects.

VSCode Version: latest master
OS Version: Mac OS 10.15.3

Steps to Reproduce:

clone repo
run yarn

➜ node -v
v10.19.0

➜ yarn -v
1.22.4

➜ yarn
yarn install v1.22.4
$ node build/npm/preinstall.js
*** Please use yarn to install dependencies.

error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.

What's the version of npm?
Can you edit https://github.com/microsoft/vscode/blob/88b1952cc498807bf66850f9643565f0d0bfe7ce/build/npm/preinstall.js#L27
and add a console.log(process.env['npm_execpath'])?

➜ npm -v
6.13.4

Added the log statement, it gives
/Users/memark/.yarn/releases/yarn-1.22.4.js

Thanks for a quick fix!


Couple of small things..

Using the core name when creating admin link
adding simplejson dependency to README

Thanks for the input!


(DOC) more fixes in workflow/actions/workflow

A couple more fixes to workflow/actions/workflow (see #1563)

@hansva how to fix the bullets in You can do this by: to appear as list items? Maybe that's impossible in a table cell?

fixes list You can do this by: using syntax from https://github.com/apache/hop/edit/master/docs/hop-user-manual/modules/ROOT/pages/pipeline/transforms/jsoninput.adoc line 39

yep, you beat me to it was just fixing it. Don't ask my why but you need to add an 'a' in front of the cell start

for more info on this https://docs.asciidoctor.org/asciidoc/latest/tables/format-cell-content/#a-operator

My diff was being weird because it was merging your and my changes, I pushed the 'a' you can see it on line 36 of the jsoninput.adoc too.

Thanks @hansva I'll know for the future!
@gitmstoute can you weigh in on formatting of lists in tables?
In https://github.com/apache/hop/pull/1558/files you use + at the end of line?


DeepSpeed not using all available GPUs

Describe the bug
Hello,
I'm trying to train a 13b language model on 2 A100 80GB GPUs using ZeRO-2, but DeepSpeed doesn't seem to be using both GPUs for the training as CUDA immediately runs out of memory after loading the model and starting the training loop.
Expected behavior
160GB should be more than enough to train a 13b model especially in a ZeRO-2 setting
ds_report output
Here's the ds_report output:

Screenshots
Here's a screenshot of the deepspeed initial logs, which indicate that indeed two CUDA devices are set:

Here's a screenshot of the CUDA memory error which indicate that not even the first GPU is used to its full capacity

System info (please complete the following information):

GPU count and types: one machine with 2 A100s
Environment: Jupyter notebook

Launcher context
The experiment is launched using deepspeed (deepspeed finetune_model.py, as all other arguments including the config file are set by default in the training file)
Docker context
Are you using a specific docker image that you can share? No
Your help is most appreciated! x(

Closing, please reopen if needed.


Transfer FIO Domain (MAS-1640 / FIP-1 Part 1)

Add ability to transfer FIO Domain to new owner

Adding xferdomain action with associated exception handling
Adding /transfer_fio_domain end point
Adding new fee: transfer_fio_domain

Local Testing Complete:
./clio -u http://localhost:8889 push action -j fio.address xferdomain '{"fio_domain":"faucet","new_owner_fio_public_key":"FIO6HuPryLfqHnbNVJKb5vJaxi4DhPXLsjn5pYHgGKX99Bg541RDN","max_fee":"4000000000000","actor":"qhh25sqpktwh","tpid":""}' --permission qhh25sqpktwh@active

0x12f28e2106ce8fd8464885b80ea865e98b465149


[SPARK-8959] [SQL] [HOTFIX] Removes parquet-thrift and libthrift dependencies

These two dependencies were introduced in #7231 to help testing Parquet compatibility with parquet-thrift. However, they somehow crash the Scala compiler in Maven builds.
This PR fixes this issue by:

Removing these two dependencies, and
Instead of generating the testing Parquet file programmatically, checking in an actual testing Parquet file generated by parquet-thrift as a test resource.

This is just a quick fix to bring back Maven builds. Need to figure out the root case as binary Parquet files are harder to maintain.

Merged build triggered.

Merged build started.

Test build #36960 has started for   PR 7330 at commit 2492f0c.

Merged build triggered.

Merged build started.

Test build #36972 has started for   PR 7330 at commit cf69512.

Test build #36960 has finished for   PR 7330 at commit 2492f0c.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test build #36972 has finished for   PR 7330 at commit cf69512.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Merging to master.


fix: enums with paramType=body got parsed incorrectly

Describe the PR
Attempt to fix the issue #1608
Relation issue
https://github.com/swaggo/swag/issues/1608
Additional context
Adding a test case with the commentLine expression similar to the one suggested in the issue.

@hohobilly Thanks for your contribution!


Link to alternative in README

As this repo is still one of the first hits in Google, it would be helpful to redirect people to an active alternative.

Thank you for contributing to stackforge/python-cloudfoundryclient!
stackforge/python-cloudfoundryclient uses Gerrit for code review.
If you have never contributed to OpenStack before make sure you have read the
getting started documentation:
http://docs.openstack.org/infra/manual/developers.html#getting-started
Otherwise please visit
http://docs.openstack.org/infra/manual/developers.html#development-workflow
and follow the instructions there to upload your change to Gerrit.


Debugging flutter tests  fail if the descrription string has an interpolated value and a is composed of two concatenated strings

We have written the needed data into your clipboard because it was too large to send. Please paste.
Type: Bug
import 'package:test/test.dart';

void main() async {
  //
  int foo = 1;
  test("Start string $foo " "  end string", () async {
    expect(true, true);
  });
}

Click the debug button above the function and get the result:
Connecting to VM Service at http://<IP_ADDRESS>:64510/Y0ktArhJg9g=/ws
No tests match regular expression "^Start string .* " "  end string( \(variant: .*\))?$".
Exited (1)

VS Code version: Code 1.71.2 (Universal) (74b1f979648cc44d385a2286793c226e611f59e7, 2022-09-14T21:05:37.721Z)
OS version: Darwin arm64 21.6.0
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
Apple M1 Max (10 x 24)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
3, 3, 4

Memory (System)
32.00GB (2.25GB free)

Process Argv
--crash-reporter-id bb45615e-9720-4e28-b503-a3f3bbe08fcd

Screen Reader
no

VM
0%

Extensions (8)

Extension
Author (truncated)
Version

dart-code
Dar
3.50.0

flutter
Dar
3.50.0

dart-import
lua
0.3.1

cmake-tools
ms-
1.12.27

cpptools
ms-
1.12.4

cpptools-extension-pack
ms-
1.3.0

kb-macro
tsh
0.12.6

cmake
twx
0.0.17

(1 theme extensions excluded)

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.72.2. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Nope, still see this on
Version: 1.72.2 (Universal)
Commit: d045a5eda657f4d7b676dedbfa7aab8207f8a075
Date: 2022-10-12T22:16:30.254Z
Electron: 19.0.17
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 21.6.0
Sandboxed: No

Please file on the dart extension

Please file on the dart extension

Where?


Android

As far as I can tell, fully functioning as is.
TODO:

[x] Play with version settings
[x] App names and icons
[ ] Get a CI build process going

Will we have to make builds for updates will things always auto deploy like web?

Will we have to make builds for updates will things always auto deploy like web?

I think I can automate most of it

I mostly have auto builds going on.
I think we have to do something about the camera and clipboard. Those are the only two things that don't work on my phone.
https://capacitorjs.com/docs/apis/clipboard
https://capacitorjs.com/docs/apis/camera

I had to do a hacky script to get <head> tags showing up in the generated index.html file so that capacitor can inject their stuff into it. But that allowed me to get the clipboard plugin to work. Swapped out all clipboard checks with an if statement depending on if it's native or not.
The tricky one is going to be the camera. I think it'll have to be some default ugly camera popping up so we might lose our UI overlay over the camera, at least on native. And I don't know if we can stream frames over to our QR code scanner or not. This might be tough and might need @futurepaul's help.

The tools handle that.
-------- Original Message --------
On Jul 21, 2023, 11:24 AM, Paul Miller wrote:

@futurepaul commented on this pull request.

On android/app/src/main/res/mipmap-xxxhdpi/ic_launcher_foreground.png:
whoever thought it was smart for devs to have to make 5000 variants of the icon is no friend of mine
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Rebased. Doesn't work with the new send / receive screens. I'll have to troubleshoot it this weekend.

Seems to just be an emulator problem.


@Quoted annotation for string arguments in annotated methods

does it make sense to throw an exception here if both quoted and greedy are applied to one value?

probably yes


PyPI auto-publish has a warning about a deprecated GHA config

Sounds like we need to update something in github actions:

I think the message appears in the "build and push to docker hub" action https://github.com/jupyterhub/repo2docker/actions/runs/506449706. Can you confirm/deny that Chris?
If yes then I don't think it is related to the new PyPI action instead it is https://github.com/jupyterhub/repo2docker/blob/master/.github/workflows/docker.yml

I think the message appears in the "build and push to docker hub" action https://github.com/jupyterhub/repo2docker/actions/runs/506449706. Can you confirm/deny that Chris?
If yes then I don't think it is related to the new PyPI action instead it is https://github.com/jupyterhub/repo2docker/blob/master/.github/workflows/docker.yml

Oops yep you’re right, I just quickly made this issue so that I didn’t forget noticing it! Will update the title

Oops yep you’re right, I just quickly made this issue so that I didn’t forget noticing it! Will update the title


Question: Collaboration Continuing after today

What will we do to continue our communication after today? Is there someone who has a gitter room or something? Just curious, I want to continue to raise awareness even after today about this issue.

We're getting ready to kick off a new version of this widget to push calls to Congress! GitHub issues and pull requests will continue to be the best way to contribute to the development of this widget, but we'd also encourage you to sign up for Fight for the Future's email list at https://www.fightforthefuture.org/ ❤

You could also join our volunteer slack channel!

On Jul 12, 2017, at 5:51 PM, jdc20181<EMAIL_ADDRESS>wrote:
What will we do to continue our communication after today? Is there someone who has a gitter room or something? Just curious, I want to continue to raise awareness even after today about this issue.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/fightforthefuture/battleforthenet-widget/issues/120, or mute the thread https://github.com/notifications/unsubscribe-auth/ABI-SQynDYH660aDMpppzinNsvINqSyIks5sNT_vgaJpZM4OWQEd.

@textdog  Please provide the link Thanks!


Migration to Java

Rewrite the plugin only using plain old Java with minimal dependencies.

Or maybe use Kotlin by now?

Ditching Groovy for Kotlin is an interesting idea. I really love the expressiveness of Spock tests. But Spock is the only reason Groovy is in my stack everywhere.
My ultimate goal is to move the plugin to https://gradlex.org and we have the convention to have all plugins implemented in Java. That's why I want to port this one to Java as well.
The reasoning behind this is that we avoid issues with Groovy / Kotlin language version incompatibilities.

I really love the expressiveness of Spock tests. But Spock is the only reason Groovy is in my stack everywhere.

I guess you're familiar with Kotest? It has several similar test styles as well.

we have the convention to have all plugins implemented in Java. That's why I want to port this one to Java as well.

So just to double-check: Porting from Kotlin to Java? That sounds like a severe step backwards, IMO...

The reasoning behind this is that we avoid issues with Groovy / Kotlin language version incompatibilities.

I'd be interested in the details if you can share!

So just to double-check: Porting from Kotlin to Java? That sounds like a severe step backwards, IMO...

I don't think that Kotlin or Java is a step forward or backward in either direction. Language selection should depend on the requirements. For GradleX we decided to use Java to have the least friction for users.

I'd be interested in the details if you can share!

There's not a lot to share. Let's say you build your plugin against Kotlin 1.9.x and you use a language feature that's not available in 2.x anymore. Once Gradle moves the Kotlin language level to 2.x the plugin build on 1.9.x won't work anymore. Since Gradle is implemented in Java and Java is generally pretty backwards compatible, it's less likely to run in these kinds of issues. This is also the reason why we try to avoid to have any dependencies in GradleX plugins. This can very easily cause problems, see gradle/gradle#19457.

Once Gradle moves the Kotlin language level to 2.x the plugin build on 1.9.x won't work anymore.

Is that so? I mean, in the end it all gets compiled to JVM bytecode, and that's what Gradle sees when executing a plugin, or?

This is also the reason why we try to avoid to have any dependencies in GradleX plugins.

Reducing dependencies to a minimum, with a balance of reinventing the wheel, is certainly always a good idea.

I don't think that Kotlin or Java is a step forward or backward in either direction. Language selection should depend on the requirements. For GradleX we decided to use Java to have the least friction for users.

I've seen that discussion on Slack, but I want to share another perspective: With Gradle pushing Kotlin DSL, I doubt that migrating from Kotlin to Java would attract more contributors to this plugin. On the contrary, as Java feels so extremely verbose compared to Kotlin, I myself would not want to contribute to this project if it was written in Java.


[3pt] Login page and registration page

[ ]  Should be built with Devise.

[ ] You should follow the layout of the wireframes provided. You should personalize the rest of the design including colors, typographies, spacings, etc.


update PAPI and improve fellows-action

It fixes this issue, significantly simplifies the code-base, and improves its performance (by paralellizing the identity requests).
A few clarifications:

Starting the 2 clients in parallel is preferable than starting them sequentially, and there is no performance overhead for doing so.
I think that the issue that happened here, in fact has to do with this. However, that error shouldn't have happened... So, we are looking into that. We actually think that it's possible that the error being logged is actually masking an error with smoldot, but we haven't gotten to the bottom of that yet.
We have put a lot of effort to make sure that the consumers of PAPI can parallelize their storage requests. The library internally handles the complexities of ensuring that if all the slots guaranteed by the JSON-RPC spec are taken, that the requests are queued optimally... So, please, use that feature and avoid making sequential requests for things that could be parallelized 🙂.

Looks good to me.
I don't understand one thing though:
Before, we had two smoldot.addChain() invocations - one for polkadot and one for polkadot_collectives.
Now, there is only one - for polkadot.
What's up with that?

Well, as you can see, this is completely bogus. It's just adding a chain for the sake of it, which is not being used anywhere... 🤷

It's just adding a chain for the sake of it

Yeah but you still have to add it so it can sync, right?
I took a look and see that smoldot.addChain exists inside getSmProvider, so it makes sense to me now.


Use Streams' forEach() method while query-expression is desugarded

Sub-task under https://github.com/ballerina-platform/ballerina-lang/issues/21757

Used while statement to handle query expression desugaring as discussed in https://github.com/ballerina-platform/ballerina-lang/issues/21757


Fixed load media callback logic

Old logic triggered errors for no reason.

A little late to the issue here, but I don't see how this resolves anything. They fundamentally do the same thing.

Not sure this logic change is valid, if a chromecast attempts to send initialization more than once then it's a session error and we should terminate.  Can you explain the case where the media session is started twice?

As the reporter has not responded to the request for additional information this PR will be closed.  If further information can be provided at some point in the future, comment on this PR with the requested information and if appropriate the PR will be reopened.


amplify add notification causes exceptions

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

How did you install the Amplify CLI?
npm install -g @aws-amplify/cli
If applicable, what version of Node.js are you using?
v14.17.0
Amplify CLI Version
4.52.0
What operating system are you using?
MacOS Catalina v10.15.7
Amplify Categories
notifications
Amplify Commands
add
Describe the bug
Ran amplify add notification to add notification to my react native as per the documentation here https://docs.amplify.aws/lib/push-notifications/getting-started/q/platform/js
However, when the CLI prompts for APIKey, I provided my server key which I obtained from firebase cloud messaging console. and I received below exception
amplify add notification
Initializing new Amplify CLI version...
Done initializing new version.
Scanning for plugins...
Plugin scan successful
? Choose the push notification channel to enable. FCM
? ApiKey AAAA*******************************0KVx*********
✖ update channel error
FCM returned 401 Unauthorized
BadRequestException: FCM returned 401 Unauthorized
    at Object.extractError (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/protocol/json.js:52:27)
    at Request.extractError (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/protocol/rest_json.js:55:8)
    at Request.callListeners (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/sequential_executor.js:106:20)
    at Request.emit (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/sequential_executor.js:78:10)
    at Request.emit (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/request.js:688:14)
    at Request.transition (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/request.js:22:10)
    at AcceptorStateMachine.runTo (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/state_machine.js:14:12)
    at /usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/state_machine.js:26:10
    at Request.<anonymous> (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/request.js:38:9)
    at Request.<anonymous> (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/request.js:690:12)
    at Request.callListeners (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/sequential_executor.js:116:18)
    at Request.emit (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/sequential_executor.js:78:10)
    at Request.emit (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/request.js:688:14)
    at Request.transition (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/request.js:22:10)
    at AcceptorStateMachine.runTo (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/state_machine.js:14:12)
    at /usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/state_machine.js:26:10
    at Request.<anonymous> (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/request.js:38:9)
    at Request.<anonymous> (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/request.js:690:12)
    at Request.callListeners (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/sequential_executor.js:116:18)
    at callNextListener (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/sequential_executor.js:96:12)
    at IncomingMessage.onEnd (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/event_listeners.js:313:13)
    at IncomingMessage.emit (node:events:377:35)
    at IncomingMessage.emit (node:domain:470:12)
    at endReadableNT (node:internal/streams/readable:1312:12)
    at processTicksAndRejections (node:internal/process/task_queues:83:21)`

Expected behavior
Notification need to be configured with FCM without any errors
Reproduction steps

Run amplify add notification
Choose FCM for Choose the push notification channel to enable.
Provide a server key from firebase messaging console as APIKey
You'll see the error generated

GraphQL schema(s)

# Put schemas below this line

Log output

# Put your logs below this line

Additional information
No response

@rosmu this happens if you pass the wrong API key for FCM. The key that you should pass will be in Project Settings -> Cloud Messaging section on Firebase console. The API key is named as Server key

I copied the same server key but the line ending character messed it up. I copied the server key from firebase console into a text editor (vim) and copied back again to clipboard without the end line and pasted to Amplify CLI and this time it worked!.


Parsing of heredoc identifier broken?

This heredoc template expression breaks parsing:
<<__EOF__
aaa
__EOF__

It looks like because the identifier used here (__EOF__) starts with an underscore and that is deemed invalid.
I've tried to understand what characters are allowed in identifiers but going down the rabbit hole almost exploded my brain when trying to understand the unicode spec stuff.
HCL syntax spec says [1]:

The heredoc template expression type is introduced by either << or <<-, followed by an identifier.

and "identifier" is defined as [2]:

Identifiers name entities such as blocks, attributes and expression variables. Identifiers are interpreted as per UAX #31 Section 2.
Identifier = ID_Start (ID_Continue | '-')*;

Where the UAX link points to a spec where I completely lost comprehension.
So the question is, can the identifier start with an underscore? Terraform actually works just fine with it so I guess yes.
[1] https://github.com/hashicorp/hcl/blob/hcl2/hclsyntax/spec.md#template-expressions
[2] https://github.com/hashicorp/hcl/blob/hcl2/hclsyntax/spec.md#identifiers

Indeed, it turns out that, according to the Unicode spec, an underscore character _ (U+005F) is not an allowed ID_Start character. It is part of the Pc category, which is part of ID_Continue, but not ID_Start.
That said, it appears that an underscore is indeed parsed without complained, so I'm adding it as an exception and will create a ticket mentioning this on their repo.


Appearance Serializer for character does not include ancient options

Describe the bug
Appearance Serializer for own character and other players in scope does not include ancient items
To Reproduce
Steps to reproduce the behavior:

Login 'ancient'
Select any character
Check character appearance
// Notice that the armor does not have ancient set aura
Switch/reload character
// Notice that the armor now show ancient set aura

Expected behavior
Ancient set aura should be shown always if complete
Additional context

This also happen for new characters entering your scope of vicinity that has ancient sets equipped
From initial investigation, newPlayer.AppearanceData.EquippedItems does not have any ancient option included (data fetch issue?)

Thanks for pointing that out - I wasn't aware of this issue 🙈

Sure thing. I'm just familiarizing myself with the source code :D
Is it somewhere related to here?


Building test app fails on Windows MSVC 16 2019

Which Firebase Sample: Remote Config
Firebase C++ SDK version: 11.0.3
Additional SDKs you are using: Nothing
Platform you are using the SDK on: Windows
Platform you are targeting: Desktop

The issue happens every time I build project using Cmake
When I try to build test application using prebuilt binaries from here, there happen some weird compiler errors, listed here:
firebase_app.lib(d94e1bff3c0e8414a6679ea91c3e103e_firebase_app.dir_Debug_cleanup_notifier.obj) : error LNK2019: unresol ved external symbol __std_find_trivial_8 referenced in function "void * * __cdecl __std_find_trivial<void *,unsigned __ int64>(void * *,void * *,unsigned __int64)" (??$__std_find_trivial@PEAX_K@@YAPEAPEAXPEAPEAX0_K@Z) [C:\Users\Sasha\Docum ents\projects2021\github-garbage-folder\firebase-test-app\desktop_testapp.vcxproj] firebase_app.lib(d3d7d08a438878e74aeb2cbaaedfc967_flatbuffers.dir_Debug_idl_parser.obj) : error LNK2001: unresolved ext ernal symbol __std_find_trivial_8 [C:\Users\Sasha\Documents\projects2021\github-garbage-folder\firebase-test-app\deskto p_testapp.vcxproj] firebase_app.lib(d94e1bff3c0e8414a6679ea91c3e103e_firebase_app.dir_Debug_locale.obj) : error LNK2019: unresolved extern al symbol u_errorName referenced in function "class std::basic_string<char,struct std::char_traits<char>,class std::all ocator<char> > __cdecl firebase::internal::GetTimezone(void)" (?GetTimezone@internal@firebase@@YA?AV?$basic_string@DU?$ char_traits@D@std@@V?$allocator@D@2@@std@@XZ) [C:\Users\Sasha\Documents\projects2021\github-garbage-folder\firebase-tes t-app\desktop_testapp.vcxproj] firebase_app.lib(d94e1bff3c0e8414a6679ea91c3e103e_firebase_app.dir_Debug_locale.obj) : error LNK2019: unresolved extern al symbol ucal_getTimeZoneIDForWindowsID referenced in function "class std::basic_string<char,struct std::char_traits<c har>,class std::allocator<char> > __cdecl firebase::internal::GetTimezone(void)" (?GetTimezone@internal@firebase@@YA?AV ?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@XZ) [C:\Users\Sasha\Documents\projects2021\github-garbage- folder\firebase-test-app\desktop_testapp.vcxproj] firebase_app.lib(d94e1bff3c0e8414a6679ea91c3e103e_firebase_app.dir_Debug_heartbeat_storage_desktop.obj) : error LNK2019 : unresolved external symbol __std_find_trivial_1 referenced in function "char const * __cdecl __std_find_trivial<char const ,unsigned char>(char const *,char const *,unsigned char)" (??$__std_find_trivial@$$CBDE@@YAPEBDPEBD0E@Z) [C:\User s\Sasha\Documents\projects2021\github-garbage-folder\firebase-test-app\desktop_testapp.vcxproj] C:\Users\Sasha\Documents\projects2021\github-garbage-folder\firebase-test-app\Debug\desktop_testapp.exe : fatal error L NK1120: 4 unresolved externals [C:\Users\Sasha\Documents\projects2021\github-garbage-folder\firebase-test-app\desktop_t estapp.vcxproj]
Steps to reproduce:
cmake .
cmake --build .

Also I forgot to mention that I'm using MSVC 16 2019 compiler

The unresolved symbols appear to be normal c++ classes and not firebase-specific.
unresolved external symbol __std_find_trivial_8 ...
unresolved external symbol u_errorName referenced in function "class std::basic_string ...
unresolved external symbol ucal_getTimeZoneIDForWindowsID ...
unresolved external symbol__std_find_trivial_1 referenced in function "class std::basic_string ...

I am not entirely sure what would cause those failures but I suspect it is some incompatibility with the C++ settings the firebase binary was built with and the C++ compiler settings that you are using. Can you try updating your MSVC compiler and see if that has any impact on the error messages that you see? Also ensure that you are targeting C++ 14 or newer.


fix(pkg): minimum kver to build kmod for arm64 is 3.16.

What type of PR is this?
/kind cleanup
Any specific area of the project related to this PR?
/area pkg
What this PR does / why we need it:
See https://github.com/falcosecurity/libs/pull/843
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/cc @dwindsor :)

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: dwindsor, FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,dwindsor]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Opt-in bugsnag

Front-end atomic-data-browser uses bugsnag to gather errors. Helful for me to find issues. While this might be OK on atomicdata.dev, it's not OK on some local machine or a private atomic-server. Users should have an easy way to opt-out, or even better, make it opt-in.

Create a new flag. Makes the atomic-server CLI even more complicated. Don't like that.
Add some custom script using --script in production. Seems better.

https://github.com/joepio/atomic-data-browser/issues/133

now you need to add  ATOMIC_SCRIPT="window.bugsnagApiKey = mykey" in .env


Initial release

[x] Add initial release to Github
[x] Deploy release to PyPI
[ ] Deploy release to conda
[x] Document release instructions as part of #14

Conda-forge release pending via conda-forge/staged-recipes#24816


MITM crash when modifying request with 'chunked' content encoding

Problem Description
Traceback:
<IP_ADDRESS>:55064: mitmproxy has crashed!
<---snip--->
ValueError: Received both a Transfer-Encoding and a Content-Length header, refusing as recommended in RFC 7230 Section 3.3.3. See https://github.com/mitmproxy/mitmproxy/issues/4799 for detail

Steps to reproduce the behavior:

Run either mitmproxy or mitmweb configured to intercept requests from http://anglesharp.azurewebsites.net/Chunked (just a convenient site that should always return chunked responses)
Intercept a request from the above URL and modify the response in anyway, e.g. via

$ curl --output /dev/null --silent \
    --cacert ~/.mitmproxy/mitmproxy-ca-cert.pem \
    --proxy <IP_ADDRESS>:8080 \
    http://anglesharp.azurewebsites.net/Chunked

Resume sending the response

System Information
Paste the output of "mitmproxy --version" here.
$ mitmproxy --version
Mitmproxy: 8.0.0.dev (+114, commit 7e24e77)
Python:    3.8.6
OpenSSL:   OpenSSL 1.1.1l  24 Aug 2021
Platform:  Linux-5.10.0-0.bpo.8-amd64-x86_64-with-glibc2.28

More Details
Following up from https://github.com/mitmproxy/mitmproxy/pull/4811#issuecomment-925696622.
At a glance, the issue looks to be at https://github.com/mitmproxy/mitmproxy/blob/main/mitmproxy/http.py#L375 where the content-length header is unconditionally set whenever content is updated. Perhaps this just needs a simple if ce != "chunked" guard? With another glance at the spec, chunked encoding specifies the chunk size within the body of each chunk and I'm assuming once we have a response all these chunks have already been decoded, so we don't have to worry about updating this for a modified response

Thanks, this is an excellent catch. I think the guard you are proposing is one correct piece of the puzzle. The other part is that we should only validate inbound requests, it's fine if we send Content-Length + Transfer-Encoding upstream if a CL header is added by a user.

I think the guard you are proposing is one correct piece of the puzzle

:+1: https://github.com/mitmproxy/mitmproxy/pull/4827

Closing as this is fixed with the PR I linked above.


Add option to remove auto-spacing.

I'm enjoying the keyboard, but the auto-space on release means a lot of backspaces. I.E when typing auto-space; typing passwords with case changes and/or numbers and special characters; using anything where space is a default keybind (visidata in termux). Awesome work though.

Thanks :)
On Wed, Mar 31, 2021 at 9:32 AM tkjhamak @.***> wrote:

I'm enjoying the keyboard, but the auto-space on release means a lot of
backspaces. I.E when typing auto-space; typing passwords with case changes
and/or numbers and special characters; using anything where space is a
default keybind (visidata in termux). Awesome work though.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/flide/8VIM/issues/168, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABX5WR7LT7CHOZJTWUGRH3LTGKNGJANCNFSM42DTNYRQ
.

You can avoid the space by ending your movement outside the circle.
If such an option would be implemented, it might be useful to still add the space if ending outside the circle, i.e. invert the current behavior.

des

You can avoid the space by ending your movement outside the circle.
If such an option would be implemented, it might be useful to still add the space if ending outside the circle, i.e. invert the current behavior.

Think about it in terms of something like cursive writting, you don't need
to pick your pen up from paper for every character and once the word is
complelete the space is automatically inserted.
I understand this becomes an issues when entering passwords and id's that
don't make sense as a 'word', but the use case is infrequent in real life.
Maybe we can disable the autospacing for password fields, but I guess that
would be the extent of it.
Regards,
Ravi
On Wed, 31 Mar 2021, 19:16 tkjhamak, @.***> wrote:

That does work, thanks! I feel like the option would be a great
improvement though. It doesn't take long to press the center to add a space.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/flide/8VIM/issues/168#issuecomment-811081133, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABX5WR4PQUD5BY4OJPMJM4TTGMRULANCNFSM42DTNYRQ
.

I suppose my use case isn't the norm, so that makes sense.  I mostly wanted a reliable keyboard for TUI's since small touch screen keyboards are horrendous in Termux and this one seems precise.
I'll just keep switching keyboards for now. I appreciate the quick response and your time. Best of luck on all your projects.


input-buffer still in use?

In the reference config "input-buffer" is still listed, but I don't see this in the codebase.  Is this variable still in use?
If so, can I track it's utilization with one of the metrics?  What does it do?
https://github.com/lomik/go-carbon/blob/master/conf-examples/carbon.conf#L20

Sample in conf-examples is not actual. Please, see configuration section in README.md https://github.com/lomik/go-carbon#configuration
In next version sample config and readme.md will be unified. You can see actual sample config for  master here https://github.com/lomik/go-carbon/blob/new-deploy/deploy/etc/go-carbon.conf

okay cool. thanks for the info!  we still have this value in our config from old versions -- we'll likely remove it with our next major update.


[ENHANCEMENT]

What is your issue or question?
fade in effects of elements to the page on scrolling
i would like to work on this , please assign me this under gssoc
please give gssoc and appropriate label to this
thankyou
Checklist

[X] I have checked the existing issues.
[X] I have read the Contribution Guidelines.
[X] I have read the Code Of Conduct.
[X] The changes don't break the code.

Add screen shots please

Sample visualisation:
https://github.com/Muskansahuincredible/StudyNotion-An-Online-Education-Platform/assets/152693538/438b84da-9bca-4841-8e28-787a93a3d83d
Or a sliding effect can also be used:


Version number

On the version "znc-1.6.0-alpha1" it displays instead "znc-1.5.0-alpha1".
You notice this when executing: znc or znc -v.

Want to back this issue? Place a bounty on it! We accept bounties via Bountysource.

http://wiki.znc.in/User:Psychon/HowToRelease says to update the version number only for RC...

I am just seeing ZNC - 1.5.0-git-485-230a39f - http://znc.in, was it changed in some newer commit or is that alpha somewhere outside of git?

It's in topic of #znc, and on website http://znc.in/

Expired issue?

I agree that "znc-1.5.0-alpha1" is misleading. I was the alpha release of 1.6.0.


moving datadir when copy-back fails when symlink

Hi @ShahriyarR,
I'll try to make a PR for this later when I have some time. As you might have noticed I'm starting to use your tool in production so I'm currently doing a bit of bughunting by usage.
My mysql datadir in production are symlinks. E.g. /var/lib/mysql points to a mountpoint. When using autoxtrabackup --prepare the copy-back step literally moves this symlink to the tmp_dir.
Example:
root@db4:/mnt/backup# ls -ltr
total 8288
drwx------ 2 root root   16384 Dec 13  2017 lost+found
drwxr-xr-x 2 root root    4096 May  9 10:01 et_events
-rwxr-xr-x 1 root root      98 Jun 18 11:49 backup_start.sh
drwxr-xr-x 4 root root    4096 Jun 20 11:33 backups
drwxr-xr-x 3 root root    4096 Jun 26 14:00 archive
lrwxrwxrwx 1 root root      17 Jun 27 15:23 tmp -> /mnt/flash/mysql/
-rw-r--r-- 1 root root 8449837 Jun 27 15:23 autoxtrabackup.log

In /etc/bck.conf :
tmpdir=/mnt/backup/tmp

For me it would also make sense to not-move the datadir before copying back. My datadir is rather large and can better be seen as lost. Would you like me to implement an '--do-not-move-datadir' flag or so? Or a setting in bck.conf ?
Cheers,
Jasper
edit: other part of the symlink problem I just found out: copy-back creates a new dir in /var/lib/mysql. So the restore get written to my / drive. Which isn't a good idea in my case. probably a rm -rf /var/lib/mysql/* would be better instead of removing and recreating this dir.

edit2: the second problem is 'solved' by changing the datadir (from /var/lib/mysql to my /mnt/whatever/mysql ) in the bck.conf.
But the move to tmp when copying-back should still be optional (enabled by default). Do you agree?

@JasperAlgra are you still interested on this? :)

@ShahriyarR yes I am.. was a bit busy with work and now gone for holidays until end of aug. When I've got some spare time in sept I'll have a go at it!

@JasperAlgra Great news.
Have a nice holiday :+1:


Update RELEASE.rst pip usage for cache avoidance

This is more aggressively cached by pip as of the new warehouse backend for PyPI.

What do you mean by "aggressively" here?

What do you mean by "aggressively" here?

as in, the new cache settings used upstream will prevent pip install upgrade from immediately seeing a newly published version (for up to an ~hour) unless the cache is explicitly ignored. previously, this was instantaneous. hence, the caching policy is more "aggressively" utilizing the cache.


Every time Docker Restarts I loose all settings

Checklist

Have you pulled and found the error with jc21/nginx-proxy-manager:latest docker image?

Yes

Are you sure you're not using someone else's docker image?

Yes

Have you searched for similar issues (both open and closed)?

Yes

Describe the bug
When docker restarts the image I have to reset admin password again and add all my port forwards
Nginx Proxy Manager Version
v2.10.3
To Reproduce
Steps to reproduce the behavior:

Start Docker container
Go To http://localhost:81/
Change email and password
Restart computer
Start Docker container and all has to re-done

Expected behavior
I shouldn't have to keep adding proxies and changing admin setting
Operating System
Window 11 WSL Ubuntu 22.04

I don't know how proficient you are with docker but are you mouting the data folder to your host system? Because if not, the configs will not persist.

Hello! I am not proficient with Docker (at all), so I have used the suggested basic docker-compose.yml file. Using that, restarting Docker means that I lose all my settings. What should I do differently to preserve my settings? Would a change to the suggested docker-compose.yml file be needed to achieve that? In that case, would it be possible to change the suggested file so that even users who are not proficient with Docker can maintain functionality?
Thank you!

Here is my docker-compose.yml file
version: "3"
services:
  app:
    image: 'jc21/nginx-proxy-manager:latest'
    restart: always
    ports:
      # Public HTTP Port:
      - '80:80'
      # Public HTTPS Port:
      - '443:443'
      # Admin Web Port:
      - '81:81'
    environment:
      # Mysql/Maria connection parameters:
      DB_MYSQL_HOST: "db"
      DB_MYSQL_PORT: 3306
      DB_MYSQL_USER: "npm"
      DB_MYSQL_PASSWORD: "npm"
      DB_MYSQL_NAME: "npm"
      # Uncomment this if IPv6 is not enabled on your host
      DISABLE_IPV6: 'true'
    volumes:
      - ./data:/data
      - ./letsencrypt:/etc/letsencrypt
    depends_on:
      - db

  db:
    image: 'jc21/mariadb-aria:latest'
    restart: always
    environment:
      MYSQL_ROOT_PASSWORD: 'npm'
      MYSQL_DATABASE: 'npm'
      MYSQL_USER: 'npm'
      MYSQL_PASSWORD: 'npm'
    volumes:
      - ./mysql:/var/lib/mysql

If you didn't mount the docker data folder to a host folder you will loose all your settings! Depending on the system you use you can mount a folder like this: (note: the example is based on Synology docker as I'm using that)
    volumes:
      - /volume1/docker/npm/config/config.json:/app/config/production.json
      - /volume1/docker/npm/data:/data
      - /volume1/docker/npm/letsencrypt:/etc/letsencrypt
      - /etc/localtime:/etc/localtime:ro
      - /etc/TZ:/etc/timezone:ro

Something like this!

I figured out the issue:
replace DRIVE with drive letter and USER with USERNAME
    volumes:
      - /mnt/(DRIVE)/Users/(USERS)/AppData/Local/Docker/wsl/data:/data

On Windows I used this to get it to work
version: '3.8'
services:
  app:
    image: 'jc21/nginx-proxy-manager:latest'
    restart: unless-stopped
    ports:
      - '80:80'
      - '81:81'
      - '443:443'
    volumes:
      - proxy-manager:/data
      - proxy-manager:/etc/letsencrypt
volumes:
  proxy-manager:
    driver: local

On Windows I used this to get it to work
version: '3.8'
services:
  app:
    image: 'jc21/nginx-proxy-manager:latest'
    restart: unless-stopped
    ports:
      - '80:80'
      - '81:81'
      - '443:443'
    volumes:
      - proxy-manager:/data
      - proxy-manager:/etc/letsencrypt
volumes:
  proxy-manager:
    driver: local

Even though this is older, I had the same problem
volumes:
  proxy-manager:
    driver: local

solved my problem as well. Thank you


log event

Type
enhancement

Description

Enhanced debug logging in the GitHub App by including the event parameter. This improvement aids in better understanding the context of logs by specifying the type of event that triggered the log entry.

Changes walkthrough
Relevant filesEnhancement

github_app.pyEnhance Logging with Event Parameter in GitHub App

pr_agent/servers/github_app.py
Enhanced logging by including the event parameter in debug logs for PR comments, new PRs, and PR synchronize actions.

+3/-3

✨ PR-Agent usage:
Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

/analyze

/help

/help

/help

/help

/help

/help


how can I debug the request?

1, I run yarn dev.
2, I refreshed my browser.
3, the browser still in loading state...
4, In terminal, I got nothing.
still says:
Compiled successfully!

Ready on http://localhost:3000
Note that pages will be compiled when you first load them.

here is the question.
how can I debug my code？
dose Next.js have a debug mode?
thanks.

Hi @robert7git! These general support requests belong on Spectrum.
Please create a new thread there asking for help, thank you!


chore: MLKit Page renamed

Hi everyone,
The smooth_app package is totally independent from the barcode scanner algorithm (MLKit / ZXing), that's why calling the page MLKitScannerPage is now irrelevant.

Codecov Report

Merging #3364 (37ca569) into develop (2621b3a) will not change coverage.
The diff coverage is 50.00%.

@@           Coverage Diff            @@
##           develop    #3364   +/-   ##
========================================
  Coverage    10.47%   10.47%
========================================
  Files          255      255
  Lines        12421    12421
========================================
  Hits          1301     1301
  Misses       11120    11120

Impacted Files
Coverage Δ

...b/pages/preferences/user_preferences_dev_mode.dart
0.00% <ø> (ø)

packages/smooth_app/lib/pages/scan/scan_page.dart
4.90% <ø> (ø)

...es/smooth_app/lib/pages/scan/camera_scan_page.dart
0.43% <50.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Make streaming and batch abstract classes

kdq-tree has a lot of weirdness due to handling both streaming and batch, e.g. in the validation: right now, the dimensions of new numpy arrays aren't checked. We could probably add checks for the length of self.input_cols vs the shape of a numpy array. This still doesn't seem ideal, but it's definitely break-able as it is.
Rather than continuing to fool with the complications introduced by the detector taking both streaming and batch input, we should probably have parallel classes that have dependency on the same partitioner/similar. This seems like a better structure going forward, as we add more detectors that can do both. The further we get into implementing validation and similar, the more troublesome these problems are.

A little tangent regarding method resolution order (MRO) with the multiple inheritance scheme. The following is how I've designed the new scheme.
class A():
    def __init__(self, a):
        self.a = a

class B():
    def __init__(self, b):
        self.b = b

class C(A, B):
    def __init__(self, a, b, c):
        A.__init__(self, a)
        B.__init__(self, b)
        self.c = c

if __name__ == '__main__':
    o = C(a=1, b=2, c=3)

I've based this off of Nathaniel Jones' answer, in the "Source code accessible: correct use of 'old style'" portion, within this StackOverflow question.
However it may still be determined that our system is or is not 'designed for multiple inheritance', and that some other structure e.g. mixin classes or the like may be more applicable.