Update Serverless URL to final destination

This is the final planned URL.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
6284

Relevant Lines:
7526

💛  - Coveralls


Redux: Use const for Action Types - More Comments Needed

Describe your problem and how to reproduce it:
Lines 32 and 38 of this challenge need the comments  // Change this line.
Add a Link to the page with the problem:
https://www.freecodecamp.org/learn/front-end-libraries/redux/use-const-for-action-types
See also
https://forum.freecodecamp.org/t/redux-use-const-for-action-types-error/444251
Tell us about your browser and operating system:
N/A
If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box):

At this point in the curriculum, I would prefer not to use // Change this lane as it feels like it makes the challenge too easy.  How about the following:
// Change code below this line
const loginUser = () => {
  return {
    type: 'LOGIN'
  }
};

const logoutUser = () => {
  return {
    type: 'LOGOUT'
  }
};
// Change code above this line

Good point. That seems reasonable to me. So perhaps
// Change code below this line

// Change code above this line

const defaultState = {
  authenticated: false
};

const authReducer = (state = defaultState, action) => {
// Change code below this line
  switch (action.type) {
    case 'LOGIN':
      return {
        authenticated: true
      }
    case 'LOGOUT':
      return {
        authenticated: false
      }
    default:
      return state;
  }
// Change code above this line
};

const store = Redux.createStore(authReducer);
// Change code below this line
const loginUser = () => {
  return {
    type: 'LOGIN'
  }
};

const logoutUser = () => {
  return {
    type: 'LOGOUT'
  }
};
// Change code above this line

@jeremylt I like it.  Let's see what others think and then we can open it up for first timers.

If we are adding additional comments, I'm in favour of the example Jeremy provided.
I'd argue, though, that at this point we've effectively eliminated the usefulness of the comments (there are a total of 5 lines of code not within "change this" comments) and would vote for removing them entirely.

I thought the comments were used by fcc as scope to check the solutions. If not needed, I'd like more subtle comments like “fix the following function”

That's not one of the three comments as I understand them

// Change code below this line
// Change code above this line
// Change this line

https://github.com/freeCodeCamp/freeCodeCamp/blob/main/curriculum/dictionaries/english/comments.json
This is our list of approved comments.

Huh, that's a bigger list than I thought. I think stripping them all together should be fine though. The instructions are essentially 'create two constants and use them', which shouldn't require too much of a safety net.

@RandellDawson are you okay with removing the comments entirely? If so, this could be opened up to first timers.

@nhcarrigan From just this challenge?

Yes.,

We are opening this issue up to first timers.
Please review our contributing documentation if you are interested in working on this issue.
For clarity's sake: The consensus is to remove the helper comments from this challenge's seed entirely.

Hey, can I take this up? I'm a newbie, so can someone walk me through it?

Hey, can I take this up? I'm a newbie, so can someone walk me through it?

@supzi-del Make sure to look at the link provided by @nhcarrigan above to get started.

This issue is closed right?

Not yet.


Redis reconnect problem

In our web application,when there is a problem with the redis cluster.The configcenter will automatically issue new cluster connections,and  old connections will be cleared.In this case, redis needs to reconnect.But there is no way to do this now, I have to restart the application.
Therefore, I modified the following code  to solve this problem
_redisServerConnection.ConnectionFailed += (_, e) =>
{
        // We use the subscription connection type
         // Ignore messages from the interactive connection (avoids duplicates)
         if (e.ConnectionType == ConnectionType.Interactive)
           {
                   return;
            }
        RedisLog.ConnectionFailed(_logger, e.Exception);
       // When ConnectionFailed ,try reconnect
       _redisServerConnection = await _options.ConnectAsync(writer);
       _bus = _redisServerConnection.GetSubscriber();
};

Any Suggestions for this situation?

Redis Cluster isn't officially supported in SignalR.
Redis will automatically try to reconnect, see the docs on reconnect (https://stackexchange.github.io/StackExchange.Redis/Configuration#reconnectretrypolicy). It looks like by default it waits 5 seconds between retries.

@BrennanConroy It will try,but the redis host is invalid.Log below:

StackExchange.Redis.RedisConnectionException: No connection is available to service this operation: PUBLISH
The connection needs to start with a new host.That means ConnectAsync need to execute again.

  internal async Task<IConnectionMultiplexer> ConnectAsync(TextWriter log)
  {
            // Factory is publically settable. Assigning to a local variable before null check for thread safety.
            var factory = ConnectionFactory;
            if (factory == null)
            {
                // REVIEW: Should we do this?
                if (Configuration.EndPoints.Count == 0)
                {
                    Configuration.EndPoints.Add(IPAddress.Loopback, 0);
                    Configuration.SetDefaultPorts();
                }

                return await ConnectionMultiplexer.ConnectAsync(Configuration, log);
            }

            return await factory(log);
 }
The latest connection address is obtained by redisName.
Redis config

AddRedis(option =>
{
string redisName = "signalR";
option.ConnectionFactory = async writer =>
{
var logger = new LoggerHelper();
var connection = await RedisConnectionHelper.GetRedisConnection(redisName, writer);
connection.ConnectionFailed += (_, e) =>
{
logger.Info("StartUp", "SignalR", "Redis", $"Connection Redis failed，ex:" + e.Exception.ToString());
};
                 if (!connection.IsConnected)
                 {
                     logger.Info("StartUp", "SignalR", "Redis", $"Connection did not connect");
                 }
                 else
                 {
                     logger.Info("StartUp", "SignalR", "Redis", $"Redis Connection connected,redisName:{redisName},connectStr:" + connection.Configuration);
                 }
                 return connection;
             };
         })


Data Listing controls are not working in IE 11

When viewing the web-codebook renderer (version f5b2a4c125445ddd6832a46688d7291c61104179 and Webcharts Version v1.9.0-RC3) in IE 11, the Controls section does not seem to work. I can click on different options within the various controls, but none of them operate to change the records listed in the table. When the renderer loads, all of the options are selected by default, but when try to click on an option (or ctrl + click) multiple options, it doesn't make the clicked options selected and the others in the list unselected, but it leaves the options I didn't click on highlighted in blue and highlights the options I did click on in gray.

Fixed in latest version


Back?

Hey @bot-hubaaa,
I'm on vacation until I finish writing this on vacation auto-response app :-)
This comment was automatically generated by Hubaaa's GitHub Vacation Responder app.

still?

Hey @bot-hubaaa,
vcvcvcv
This comment was automatically generated by Hubaaa's GitHub Vacation Responder app.


Missing assets, source files, and xcodeproj

Thank you for taking the time to prepare this release! I'm very happy to see this project open sourced.
Goal: I'm a huge Alienblue fan and want to compile and hack up my own build.
Problem: The current source tree is missing assets, source code files, and an xcodeproj file which together prevent this project from compiling/working.
Solution: The source files and various assets would need to be added for the project to be in a compilable/usable state. (An xcodeproj would be useful too, but that can also be reconstructed without the original).
Do you anticipate being able to release more parts of the source code / assets?
Examples of classes that are referenced in various source files but do not have implementations in this source tree:

JMYoutube
JMGallery

Thank you!
Ethan

@mattrobmattrob Is the current source the most complete src that is able to be made public?

@mattrobmattrob Is the current source the most complete src that is able to be made public?

Yes.

Is that a hard stop or could I get somewhere by bugging higher ups about it?
Thanks for preparing the current release btw. Steve said you (and some others) spent a bit of time getting this ready. The effort is very much appreciated


Metal files silently compiled

Description
SwiftPM 5.3 added support to process .metal files using the resource functionality.
It seems that somewhere between SwiftPM 5.3 and 5.7 all .metal files in a package are compiled into a default.metallib file
I cannot find any documentation on this behaviour. Is it expected? What swiftpm version was this feature added?
Expected behavior
No response
Actual behavior
No response
Steps to reproduce
No response
Swift Package Manager version/commit hash
No response
Swift & OS version (output of swift --version && uname -a)
No response

Resources of known types are automatically handled, as per the resources proposal.
Could you elaborate more on what the issue is? Also it sounds like you're saying the behavior changed between versions, what changed exactly?

Ah it was my understanding that the only way to compile metal was via resource.process. From the sounds of it seems SPM has always automagically compiled .metal files then?
TBH it's kind of unclear from proposal that what the expectations of metal files are. Maybe some clarity on what files are considered known and are handled automatically?
Is it possible to opt out of automatic handling? (so we could process metal files ourselves with a plugin?)

You should be able to use excludes.
I'm not 100% sure, but it might also mean the Target APIs wouldn't include these files and you would need to look for them manually on the file system in your plugin implementation.

BTW, the proposal was intentionally not descriptive about specific file types since we didn't want to (and couldn't) define concrete Xcode behavior in there. The documentation on resources in Xcode is a little bit more clear, but it unfortunately also doesn't really have an exhaustive list of file types which are handled automatically.

Yeah fair enough. Maybe then make .process() fail if it tries to process file that should be automatically handled? Can probably close out this ticket but yeah it is a bit confusing.


Add UserAPI and option to show profile images in nav

Added models and endpoint for new user account API
Show profile images in the nav if USER_PROFILES_ENABLED flag is true

https://openedx.atlassian.net/browse/MA-1287

cc @mikekatz for profile config flag

@1zaman Please review?

:+1:

Additional change: I just widened the navigation drawer. I don't think that will cause issues though. I extracted it's container to a layout file that's now shared everywhere.


Doc Gen Tool Generates Unrestricted Items

This fixes #1998. This also updates the table of contents to put Unstable namespaces at the end.

@ScottCarda-MS  what about @Config(Base) / Config(Adaptive) items? Are we intentionally excluding them now?
(Acknowledging that exposing all configurations sounds difficult, given the current design of conditional compilation. So a compromise may be needed here).

@ScottCarda-MS what about @Config(Base) / Config(Adaptive) items? Are we intentionally excluding them now?
(Acknowledging that exposing all configurations sounds difficult, given the current design of conditional compilation. So a compromise may be needed here).

This should include all items, including Base and Adaptive. I added a test for finding the Adaptive Std.Intrinsic.AND operation. I couldn't find any Base items in our library to test against.

@ScottCarda-MS
But AND has a @Config(not Adaptive) implementation as well, are you sure that's not the one you're getting?
@Config(not Adaptive)
operation AND(control1 : Qubit, control2 : Qubit, target : Qubit) : Unit is Adj {
    PhaseCCX(control1, control2, target);
}

Would doc generation work for items that have a Adaptive/Base implementation and nothing else?

@microsoft-github-policy-service agree

@microsoft-github-policy-service rerun

Would doc generation work for items that have a Adaptive/Base implementation and nothing else?

I'm not able to tell at this point as there is no such content in the existing library code. Trying to update the libraries to include such code is running into issues. I'm going to save this for a future PR if/when we add such content to the libraries.
Since the existing test doesn't determine what I thought it did (as you pointed out, AND has both an Adaptive and not Adaptive definition), I'm removing the test.

Closing in favor of #2012


swallowing gulp errors

I am using concurrently and one of the tasks it runs is gulp, which in turn runs a sass and js linter (Sasslint and ESLint respectively). If a lint error is reported by either of those, concurrently swallows it and we're not getting that output in the terminal that we'd normally see if we ran Gulp directly.
I tried adding --raw but that did not work. What am I missing here? Appreciate any help you can offer.

Sure, give me a few days, I'm currently out of town for work. Please don't close the issue in the meantime, I will definitely do that as this is holding us up from using concurrently effectively so i'd really love to get it solved.

@gustavohenke So I finally got some time to put together a small reproducible project which immediately made me realize that the error was not in concurrently at all and I was able to get everything working as expected. Apologies for the noise but that was a helpful exercise to finally help me track this down. Many thanks.

No worries, glad you found the root of your problem!


Add support for RTSP Mp4a-Latm

Added Mp4a-latm RTP Packet reader and added support for Mp4a-latm
playback through RTSP.
Change-Id: Ia590393f53ca880af926907843f6bea9ff0f4b35

This CL has merged. The Pull request may be marked as completed.


Disable Issues

You should disable Issues for this repository so you're not bombarded with troll messages in your issues.

Looking at when the last troll issues were I think they have passed. My “issue” is more of a suggestion or to help someone who also is having issues in VScode.

I'd hope that if you were using this, you wouldn't need to know how to install a Python virtual environment.
Doug likely won't give users direct support for either of his repositories nor will he be making any further changes (but I am not Doug, I do not speak for him. It is his choice)
It's an "as-it-is" situation; if it works, it works.
As such, there's no reason to need the Issues tab (right now).


I tried to use your preprocessing methods and pretrained model but it didn't work. May I check with you?

Hello,
I tried to use your preprocessing methods and pretrained model but it didn't work on my dataset. May I check with you about 3 questions?
(1) I used DCMTK with: dcmj2pnm +on2 --min-max-window --set-window -600 1500 pathToDCM pathToPNG16
Did I use DCMTK with the same command line as yours?
(2) I studied augmentations.py and followed its methods to change group of PNG16 into tensor
([mean std]=[128.1722, 87.1849] for normalization), (TorchIO to do interpolation, but I changed the voxel to 1.5 * 1.5 * 1.5 mm), (The tensor [min, max] was about [-1.4701, 1.4547]) (The tensor input into the model had shape (C T H W))

The subplots in this image are some slices from the interpolated tensor (plot by plt.imshow(1SliceOfTensor, cmap='gray'))
Did I changed the PNG16 into tensor in the correct way?
Do you think it's a bad idea to change its voxel to 1.5 * 1.5 * 1.5 mm (you use 1.4 * 1.4 * 2.5 mm)?
(3) I load your pretrained model's encoder back into standard r3d_18 and replace its last fc layer so that it can train on my 5 classes dataset.
resnet3d = torchvision.models.video.r3d_18(pretrained=True)
path = "/path/to/65fd1f04cb4c5847d86a9ed8ba31ac1aepoch=10.ckpt"
checkpoint = torch.load(path, map_location="cpu")
(the layers' names in your pretrained model are different from the standard r3d_18  so that I change them back)
state_dict = {("layer"+k[20:]): v for k, v in checkpoint["state_dict"].items()}
state_dict["stem.0.weight"] = state_dict.pop("layer0.0.weight")
state_dict["stem.1.weight"] = state_dict.pop("layer0.1.weight")
state_dict["stem.1.bias"] = state_dict.pop("layer0.1.bias")
state_dict["stem.1.running_mean"] = state_dict.pop("layer0.1.running_mean")
state_dict["stem.1.running_var"] = state_dict.pop("layer0.1.running_var")
state_dict["stem.1.num_batches_tracked"] = state_dict.pop("layer0.1.num_batches_tracked")
model_dict_copy = resnet3d.state_dict()
pretrained_dict = {k: v for k, v in state_dict.items() if k in model_dict_copy}
model_dict_copy.update(pretrained_dict)
resnet3d.load_state_dict(model_dict_copy)
Your pretrained model has other layers after the encoder but I am not sure whether I should use them.
Do you think I use your pretrained model correctly?
I freezed the encoder and trained only the fc layer because my dataset is small. But the training accuracy stay very low (40% for 5 classes classification).
Do you think I made any mistakes?
Thank you very much.

Hi,
Thanks for reaching out!

I think that's the same as the DCMTK conversion command we use, which is:

dcmj2pnm +on2 +Ww -600 1500  dicom_path  image_path

You've got the right tensor dimensions, so that should be good. In terms of the voxel shape, I might keep the original parameters if you intend to use the pretrained weights, but definitely feel free to experiment as they can be context-dependent.

In terms of the model, it may be more straightforward to initialize the SybilNet class (see for instance the load_model function in model.py).  The additional layers perform pooling on the output activations from the 3D ResNet, and learn an attention. I would use the last hidden layer from that (see pool_output["hidden"] in SybilNet.

Then your model could look something like:
class NewModel(nn.Module):
    def __init__(self, checkpoint_path):
        super(NewModel, self).__init__()

        checkpoint = torch.load(checkpoint_path, map_location="cpu")
        args = checkpoint["args"]
        model = SybilNet(args)

        # Remove model from param names
        state_dict = {k[6:]: v for k, v in checkpoint["state_dict"].items()}
        model.load_state_dict(state_dict)  # type: ignore

        self.sybil_model = model
        # some model architecture you wish to train
        self.classifier = nn.Linear(512, 5)

    def forward(self, x):
        output = self.sybil_model(x)
        hidden = output["hidden"]
        y_hat = self.classifier(hidden)
        return y_hat


0.14.0-beta.x versions are published to npm as latest

🐞 Bug report
Command (mark with an x)

- [ ] new
- [ ] build
- [ ] serve
- [ ] test
- [ ] e2e
- [ ] generate
- [ ] add
- [ ] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Is this a regression?
I don't remember it happening with the 0.13, but I might be wrong
Description
The beta versions shouldn't be tagged as "latest", as one does not want to install them by default when calling npm install --save @angular-devkit/build-angular
🌍 Your Environment

$ npm --version
6.9.0

Apparently the issue is due to the proxying in Verdaccio. No idea why. I'm closing for now.


⚠ API (search) has degraded performance

In e79f32c, API (search) (https://api.dev.package-search.services.jetbrains.com/search-packages) experienced degraded performance:

HTTP code: 200
Response time: 7333 ms

Resolved: API (search) performance has improved in d889369 after 6 minutes.


Function: Fix 3851: replace hardcoded runtime for Function with props provided runtime

refer issue 3851
I've gone for the naively simple option here.

We are transition this repo to https://github.com/sst/v2 and are closing this issue for now. If it is still relevant please reopen it there - sorry for the trouble


docs: updated Medium link in footer

Old link: https://medium.com/@_carbondesign
New link: https://medium.com/carbondesign

did travis not do his 1 job?

(source seems correct but netlify build doesn't seem to reflect it.)

Fixed in #1653!


New 2.16.x release?

Working on a PR to dependabot to add support for composer v2, and one of the things that is blocking that is a new v2.16.x release on this package. Reason for that is the use of composer/semver v3 where the latest v2.16.x release on this package only supports v1.4. Already found out that the dev branch for v2.16.x on this package has support for composer/semver. A new release would be greatly appreciated.

Please have a look @ https://github.com/FriendsOfPHP/PHP-CS-Fixer/issues/5130#issuecomment-708976913
Next Monday is another alignment between members and maintainer and if there is any new I'm happy to share :)

Done, and looking forward to that. Also, make sure you and the rest of the team take care of your lives before OSS projects :).

releases have been done :)

Thanks! There is one more dependency to update:
 - php-cs-fixer/diff (v1.3.1) latest is v2.0.2
composer why php-cs-fixer/diff

>friendsofphp/php-cs-fixer  v2.16.7  requires  php-cs-fixer/diff (^1.3)

@lptn , does it block you with anything?
PHP CS Fixer uses diff 1.3 (v1.3.1 is ready for PHP 8) or 2.0 (v2.0.2 is ready for PHP 8)
update to v2 is bringing different behaviour thus behaviour of PHP CS Fixer itself would be different, causing a BC break, which we applied on v3@dev

@keradus

does it block you with anything?

No, sorry if I confused you. Thanks for the information 👍


CSS in markdown is purged

I double the report cause I don't understant whoes the fault is.
Version:
@nuxt/content: v1.41.0
nuxt: v2.15.3
@nuxtjs/tailwindcss: v4.2.1
Reproduction Link
https://github.com/StefanoTesla/cssproblem
Steps to reproduce
generate the repository, and run npn start, div style is broken (the class are purged olso if in safelist)
What is Expected?
If you made a npm run dev style is ok
What is actually happening?
generate the repository, and run npn start, div style is broken
Strange fact

Copy the div  insiede the two markdown filse and paste in a new, or existing, nuxt page .vue
generate the project
npm start
style is ok

any suggestions?

You may want to take a look at https://tailwindcss.nuxtjs.org/options#config


Specific package task definition causes nondeterministic behavior

What version of Turborepo are you using?
1.1.6, 1.1.9
What package manager are you using / does the bug impact?
Yarn v1
What operating system are you using?
Mac
Describe the Bug
turbo run determines which tasks to run in a nondeterministic manner.
We have two packages a and b, and we have below turbo.json:
{
  "pipeline": {
    "generate": {},
    "build": {
      "dependsOn": [
        "generate"
      ]
    },
    "b#build": {}
  }
}

With this setting, turbo run build sometimes only run build script of a and sometimes runs both generate and build scripts of a.
We see two different results randomly:
yarn run v1.22.17
$ turbo run build
• Packages in scope: a, b
• Running build in 2 packages
a:build: cache hit, replaying output 967be7f13e8415ed
a:build: warning package.json: No license field
a:build: $ mkdir -p dist && echo built > ./dist/a.txt

 Tasks:    1 successful, 1 total
Cached:    1 cached, 1 total
  Time:    36ms >>> FULL TURBO

✨  Done in 0.15s.

yarn run v1.22.17
$ turbo run build
• Packages in scope: a, b
• Running build in 2 packages
a:generate: cache hit, replaying output 83eccbfec7d66441
a:generate: warning package.json: No license field
a:generate: $ mkdir -p generated && echo generated > ./generated/a.txt
a:build: cache hit, replaying output 967be7f13e8415ed
a:build: warning package.json: No license field
a:build: $ mkdir -p dist && echo built > ./dist/a.txt

 Tasks:    2 successful, 2 total
Cached:    2 cached, 2 total
  Time:    29ms >>> FULL TURBO

✨  Done in 0.16s.

Expected Behavior
Both generate and build scripts of a should be always run.
To Reproduce
Reproduction repo: https://github.com/uhyo/turborepo-nondeterministic
Clone the repo and run yarn install, and then run yarn repro many times.

This fixed in canary with the new —filter and graph changes.

Wow, good to hear that. Thank you!

@jaredpalmer Sorry but I still observe the issue after updating turbo to 1.2.0-canary.0. I hope you could recheck that. 🥺

Will investigate and get back to you--
Jared Palmer @.** https://twitter.com/jaredpalmer)
▲ Engineer at Vercel https://vercel.com/

I can confirm that the issue is fixed now. Thank you!


create an Execute template function

For each command that do something in the blockchain(transfer, sell_asset, vote_for_witness, etc) the OnOK method of the command class is called: https://github.com/bitshares/gwallet/blob/master/panels/transfer.cpp#L32
This do some field conversion, validate some data, check if the cli is checked, create the command without broadcasting, present the confirmation dialog and finally executes and get a signed_transaction as response.
In all commands, the same is done. An Execute() function with the number of arguments needed can be created i think in commands.cpp to centralize this code and optimize it in one place.
The tricky part i think is that you need to send wallet method name as argument and call the function, i think it can be done with something like this:
https://stackoverflow.com/a/7558083

No template function was needed, just a normal one. Function is created here https://github.com/bitshares/gwallet/commit/3aa698284b98fad5648658d340a037005b854350 and then applied to all operations at https://github.com/bitshares/gwallet/commit/fff49e35a9b0a8b91f49ddcfdc0c11ef9591d0af

sellasset was missing, added at https://github.com/bitshares/gwallet/commit/71d0b32610a17882aea0ed36d68d30b4ae716afc


yaml-cpp is unconditionally linked and included

The current CMakeLists.txt (here) does not do any detection of the required yaml-cpp dependency, but unconditionally links against it.
This will fail on systems without yaml-cpp, or with yaml-cpp outside the standard include and link paths and should be fixed.

Could this be a solution?
find_package(PkgConfig)
if(PKG_CONFIG_FOUND)
pkg_check_modules(NEW_YAMLCPP yaml-cpp)
endif()
if(NEW_YAMLCPP_FOUND)
target_link_libraries(${PROJECT_NAME} yaml-cpp)
endif()

It is part of a solution, yes. I'd do it slightly differently (finding yaml-cpp itself does not necessarily mean that it is the new version). If you can give me some time, I'll propose something in a PR.


[Show Graph] command 'yarnspinner.show-graph' not found

What is the current behavior?
My vscode notify “command 'yarnspinner.show-graph' not found”  when I click “Show Grash”.

Please provide the steps to reproduce, and if possible a minimal demo of the problem:

open yarn file

Wow!
My first ever Yarn script in Unity!

-> Gosh!
-> Incredible!
-> I'm amazed!

Anyway, time to get writing!
===

Click “Show Graph” and then the error happened

What is the expected behavior?
I want the graph to be displayed when I click "Show Graph"

Please tell us about your environment:

Extension Version:  v2.2.128

Other information
I find vscode is downling  .net, does it matter?

I've had this issue recently after fresh install of the extension. Turns out I was just on a bad connection and .NET download was failing. When I allowed it to download, Yarn extension started working normally, including graph view.


Update Azure Cosmos DB emulator documentation with Docker Compose guidance

This article needs to be updated to show how to use Docker Compose with the emulator and import the SSL certificate with the correct hostname.
When this article was originally published, it was unclear how to do this. Fortunately, some GitHub users have figured out how to accomplish this:

https://github.com/Azure/azure-cosmos-db-emulator-docker/issues/76#issuecomment-2166553017
https://github.com/Azure/azure-cosmos-db-emulator-docker/issues/76#issuecomment-1973885019

Special thanks to:

@davilink
@hughesjs

This guidance is very necessary so we can enable scenarios like devcontainers, GitHub Codespaces, and GitHub Actions service containers.
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 856db70a-8c23-1ba7-a58d-c0f05e0006a5
Version Independent ID: a77a1937-5016-99e5-c928-26c55ceeb94f
Content: Use the emulator for development and CI - Azure Cosmos DB
Content Source: articles/cosmos-db/how-to-develop-emulator.md
Service: cosmos-db
GitHub Login: @seesharprun
Microsoft Alias: sidandrews

#assign:seesharprun

@seesharprun this issue stayed behind in the old repo. FYI


Proposal: Improve documentation for ISemanticZoomInformation and existing implementations

Currently, the documentation for this interface (both for UWP and the WinAppSDK) just has basic descriptions of the properties and when some methods are invoked. It'd be awesome to have documented examples, specially on how to integrate with the already existing implementations - those being ListViewBase and Hub.
I think this checklist would cover all bases:

[x] Basic descriptions of the interface's properties
[x] Basic descriptions of the interface's methods
[ ] Guidance for a basic implementation
[ ] Guidance for the use of SemanticZoomLocation, specially for the methods that use it in the interface
[ ] Guidance for integration with existing (and maybe custom?) implementations
[ ] Documented examples of the above - I would suggest having something like this:

[ ] A custom implementation that does more than displaying the headers already on the list - I think an example with FlipView would work very well here
[ ] A more advanced scenario with a custom list control - having it revolve around ItemsRepeater would be pretty cool

It is a lot, but I think it's worth it. Semantic zoom is a very interesting concept, and being able to expand on it would be easier with better documentation.

I have found a few SO questions where they have an implementation, but they don't seem to handle all cases or just implement the interface without further enhancements:
https://stackoverflow.com/questions/14345110/semantic-zoom-for-custom-view-hierarchy-instead-of-plain-grid-or-listview
https://stackoverflow.com/questions/13825577/semanticzoom-without-gridview
Anything I can do? Really looks like I'll have to implement my own controls from scratch.

WinUI 3 is now open source, so I no longer need this.


[ISSUE #4781] Add config file for controller run as Namesrv plugin

Make sure set the target branch to develop
What is the purpose of the change
close #4781
Brief changelog

Add config file for controller run as Namesrv plugin
add fast-plugin.sh for quick start controller as Namesrv plugin

Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Codecov Report

Merging #4787 (4951924) into develop (859bc15) will decrease coverage by 0.14%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop    #4787      +/-   ##
=============================================
- Coverage      45.14%   44.99%   -0.15%
+ Complexity      7634     7595      -39
=============================================
  Files            976      976
  Lines          67814    67814
  Branches        8964     8964
=============================================
- Hits           30612    30511     -101
- Misses         33437    33522      +85
- Partials        3765     3781      +16

Impacted Files
Coverage Δ

...rg/apache/rocketmq/common/stats/StatsSnapshot.java
84.61% <0.00%> (-15.39%)
:arrow_down:

...apache/rocketmq/broker/longpolling/PopRequest.java
31.03% <0.00%> (-13.80%)
:arrow_down:

...ache/rocketmq/common/stats/MomentStatsItemSet.java
39.13% <0.00%> (-13.05%)
:arrow_down:

.../apache/rocketmq/common/stats/MomentStatsItem.java
38.09% <0.00%> (-9.53%)
:arrow_down:

...q/namesrv/routeinfo/BrokerHousekeepingService.java
72.72% <0.00%> (-9.10%)
:arrow_down:

...va/org/apache/rocketmq/common/stats/StatsItem.java
50.00% <0.00%> (-6.67%)
:arrow_down:

...a/org/apache/rocketmq/logging/inner/SysLogger.java
28.57% <0.00%> (-5.72%)
:arrow_down:

...etmq/namesrv/routeinfo/BatchUnRegisterService.java
94.73% <0.00%> (-5.27%)
:arrow_down:

...org/apache/rocketmq/common/stats/StatsItemSet.java
50.74% <0.00%> (-4.48%)
:arrow_down:

...java/org/apache/rocketmq/logging/inner/Logger.java
51.67% <0.00%> (-4.31%)
:arrow_down:

... and 20 more

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

Coverage decreased (-0.009%) to 49.291% when pulling 49519240dc74bdb2ab3a7202ebd196169cdb8ab8 on mxsm:rocketmq-4781 into 859bc15a6822893635daf147f001cc2ff7d3ca49 on apache:develop.

Coverage decreased (-0.009%) to 49.291% when pulling 49519240dc74bdb2ab3a7202ebd196169cdb8ab8 on mxsm:rocketmq-4781 into 859bc15a6822893635daf147f001cc2ff7d3ca49 on apache:develop.


quick fix for the broken used name

Quick fix #3666 .
Simply get ride of the \n, this solution works currently, but seems very tricky
I wonder whether it should be merged.
If this pr is merged, I will send pr to zinc

Could you send the fix to Zinc first, plz?


gcli: set meta.platforms

Description of changes
Otherwise, no rebuild on darwin for ofborg / Hydra.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 286810 run on aarch64-darwin 1

1 package built:

gcli


Missing docs on VariableName

Describe the bug
The documentation site of the existing checks, is missing the VariableName check.
Source
https://shopify.dev/docs/storefronts/themes/tools/theme-check/checks#liquid-file-checks
Expected behaviour
Expect to see a VariableName check reference, with it's own page, to describe the possible filters.
Actual behaviour
Just missing from the docs site.

Hi @tmmgrafikr!
Thanks for your patience. The documentation has been added. You can view it at https://shopify.dev/docs/storefronts/themes/tools/theme-check/checks/variable-name


erorr attrs!=21.1,>=17 on centos8

[root@pg1 ~]# cat /etc/os-release
NAME="CentOS Stream"
VERSION="8"
ID="centos"
ID_LIKE="rhel fedora"
VERSION_ID="8"
PLATFORM_ID="platform:el8"
PRETTY_NAME="CentOS Stream 8"
ANSI_COLOR="0;31"
CPE_NAME="cpe:/o:centos:centos:8"
HOME_URL="https://centos.org/"
BUG_REPORT_URL="https://bugzilla.redhat.com/"
REDHAT_SUPPORT_PRODUCT="Red Hat Enterprise Linux 8"
REDHAT_SUPPORT_PRODUCT_VERSION="CentOS Stream"
dnf install python3 python3-attrs python3-humanize python3-psutil python3-psycopg2
rpm -ivh https://rpmfind.net/linux/epel/8/Everything/x86_64/Packages/p/python3-wcwidth-0.2.5-3.el8.noarch.rpm
rpm -ivh https://rpmfind.net/linux/epel/8/Everything/x86_64/Packages/p/python3-blessed-1.19.1-1.el8.noarch.rpm
rpm -ivh https://download.postgresql.org/pub/repos/yum/common/redhat/rhel-8.5-x86_64/pg_activity-2.3.1-1.rhel8.noarch.rpm
[root@pg1 ~]# pg_activity
Traceback (most recent call last):
File "/usr/bin/pg_activity", line 6, in
from pkg_resources import load_entry_point
File "/usr/lib/python3.6/site-packages/pkg_resources/init.py", line 3095, in
@_call_aside
File "/usr/lib/python3.6/site-packages/pkg_resources/init.py", line 3079, in _call_aside
f(*args, **kwargs)
File "/usr/lib/python3.6/site-packages/pkg_resources/init.py", line 3108, in _initialize_master_working_set
working_set = WorkingSet._build_master()
File "/usr/lib/python3.6/site-packages/pkg_resources/init.py", line 570, in _build_master
ws.require(requires)
File "/usr/lib/python3.6/site-packages/pkg_resources/init.py", line 888, in require
needed = self.resolve(parse_requirements(requirements))
File "/usr/lib/python3.6/site-packages/pkg_resources/init.py", line 774, in resolve
raise DistributionNotFound(req, requirers)
pkg_resources.DistributionNotFound: The 'attrs!=21.1,>=17' distribution was not found and is required by pg-activity
[root@pg1 ~]# rpm -qa | grep attrs
python3-attrs-22.1.0-1.rhel8.noarch

That's a packaging problem from PGDG repository, please report this there if the issue is not known already.
In the meantime, I'd recommend installing from pip.


PP-5669 Add types and fixtures for ledger report contracts

Add types and fixtures that use these for the following report contracts
which will be produced by ledger endpoints:

Get payments by state report. This will return the count of payments
with each state for the parameters specified.
Get payments report. This will return some high level aggregate
statistics for payments. Initially this will be the total payment
count and the gross amount.

Closes https://github.com/alphagov/pay-toolbox/projects/1#card-27176481


Specify absolute path to audio file

When using an absolute path that doesn't point directly to a sound file, SoundJS tries to load "/undefined" relative to the HTML page my javascript is running on.
Example:
createjs.Sound.registerSound('http://api.voicerss.org/?key=e9a546d445234733b3113b8fb54ba6cd&src=+Hello+Brad.&hl=en-us&c=OGG');
Internally, _parsePath returns false
Should it not just return something like {src:"ORIGINAL_URL_HERE"} for absolute paths that don't contain a file name?

currently soundjs needs a filetype to properly load audio, which is gets from filepath/filename.filetype for example "mypath/sounds.mp3".
We're considering ways to support the type of request your currently using in future releases, but currently this will not work.
Hope that helps.


[Teradata] impossible to select schema for SQL Editor

System information:

Operating system (distribution) and version Mac
DBeaver version 7.2.5 EA

Connection specification:

Database name and version Teradata

Steps to reproduce, if exist:
Open SQL Editor for Teradata and try to set schema in upper toolbar
Expected: the selected schema is displayed in upper toolbar
Actual result: N/A is displayed in the toolbar after schema selection
Note: the same if open SQL Editor by buttons or by reading data in SQL console

Select default schema is now available.
And the default scheme at the beginning of work is also selected.
Also solved the problem of missing quotes for query, which selects the default scheme in Teradata and Snowflake.
Can be checked on the Teradata.All and Snowflake.ILIKE

verified


Bad indentation, found 2 spaces, expected 4

In the following 2 files, under the following 2 if conditions, Only 2 spaces of indentation is found. If I'm not wrong it should be 4 spaces of indentation right?
https://github.com/chroma-core/chroma/blob/133a70df1d08b63110db9017067086feac022121/chromadb/proto/coordinator_pb2.py#L23-L62
https://github.com/chroma-core/chroma/blob/133a70df1d08b63110db9017067086feac022121/chromadb/proto/chroma_pb2.py#L21-L73

Hey @Sai-Suraj-27 , thank you for pointing this out!
These files are autogenerated by our protobuf pipeline -- instead of fixing the indentation, do you think you could exclude them from our precommit checks? Another option would be to configure the proto pipeline to properly indent these but I'd prefer we just ignore their format rather than try to format them correctly.

Sweet! Thank you for reporting this. I'm going to close this issue since we don't care about the formatting of this file but keep the issues coming!


Temporarily skip tests that exercise the --experimental-faster-testing feature

See https://github.com/flutter/flutter/issues/160685.

subsumed by https://github.com/flutter/flutter/pull/160694


Update Swift tools version to 5.9

Bug/issue #, if applicable: rdar://116871051
Summary
This updates SwiftDocC to Swift 5.9 and

uses shorthand syntax for if let (SE-0345)
modernizes generics syntax (SE-0328, SE-0341, SE-0361)
removes a no longer needed pack-port of pointer API (SE-0370)
replaces FileManagerProtocol SPI with package access level (SE-0386)

There are lots more that we can do now, but I feel that those are better addressed in separate PRs. I opted for these changes because they each make the code more readable and have small (but very frequent) and understandable diffs.
Dependencies
None
Testing
Nothing in particular. There should be
Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

~[] Added tests~
[x] Ran the ./bin/test script and it succeeded
~[ ] Updated documentation if necessary~

I made each change as its own commit, to make it easier to review. If you prefer I can break it apart into separate PRs but I didn't feel that that would make it easier to review.

@swift-ci please test

Hm.. the Linux CI seems to be pinned to SWIFT_VERSION=5.8.1 which causes it to fail.

This is blocked on the CI (rdar://125077916)

@swift-ci please test

The team talked about this today and decided to revert the package access level commit and add it back in a follow up PR to unblock this.


BpmMultiplier is inverted in MathHelper

return Clamp((float)-timingPoint.BeatLength, 10, 1000) / 100f;
e.g. when beat length is -1000, bpm multiplier should be 0.1 instead of 10.

Have you actually faced any issues because of this? Fixing this would break endtime calculation. I understand that this implementation is unintuitive as hell and may be incorrect, but that's how it's been implemented in osu! for ages.

Not really an issue I guess, but like you said, it's very confusing when doing things involving beat length and bpm multiplier.
Sorry that I forgot to mention it originally but I meant to also adjust endtime calc for this. The way I did it was by removing the bpm multiplier from BeatLengthAt and adding a new method to beatmap to get the final slider velocity (100 * SliderMultiplier * BpmMultiplier).
int duration = (int)(pixelLength / beatmap.SliderVelocityAt(startTme) * repeats * beatmap.BeatLengthAt(startTime));
Don't know if that's a good way to do it but it made a lot more sense to me. And imo this would be helpful for anyone else that comes across this.

Thanks a lot, I'll take a look at it a bit later and will come back to you!


Class view not well structured

Hi all,
I personally have the feeling that the class view in Visual Studio is not well used by the Python tools. There is no structuring of the nodes into packages. Instead it is simply structured by the Python file names. It would be more helpful, if the behaviour would be similar to that for C++ regarding namespaces for example.
So I would except such a structure for example:

PackageA

ClassA
Class B

PackageB

ClassC
ClassD

Instead Python tools will show:

file1.py (PackageA)

ClassA
file2.py (PackageA)

ClassB

file3.py (PackageB)

ClassC
ClassD

In the curent implementation there is no big difference between the project manager and the class view. I would be very happy to see further development on this issue, because the class view is  a very powerful tool of Visual Studio.
Best regards,
Ralf

This would be a nice enhancement, to use the module name (or package name when the file is __init__.py) instead of the file name.

I'm sorry, can anyone redirect me to some place where it would be apropriate for me to ask and find out why Class View does not show anything at all in my case? (VS2017, PTVS, Python file with classes - all checked) It never worked for me.

@UniBreakfast Class View (and Object Browser) is disabled for Python code right now - it was causing more problems than it was worth. We'd like to re-enable it, but that will likely wait until after we have done some serious improvements on our code analysis in general (which will make Class View better).


Support CORS in Cesium's development server

At least when it is running locally.
After
app.use(compression());

add
app.use(function(req, res, next) {
  res.header("Access-Control-Allow-Origin", "*");
  res.header("Access-Control-Allow-Headers", "Origin, X-Requested-With, Content-Type, Accept");
  next();
});

The similar server.js in the 3d-tiles-samples repo already has this.

@pjcozzi I would like to take on this, if is still relevant.

Thanks @bampakoa, that would be great!  You can find information for contributing a pull request here: https://github.com/AnalyticalGraphicsInc/cesium/blob/master/CONTRIBUTING.md#opening-a-pull-request

Go for it, @bampakoa!

I have made some tests and it seems that requests from different domains are handled correctly from the express server, with the current configuration. Could you please give some more info about the issue?

@bampakoa I think all we want to do is add this code:
https://github.com/AnalyticalGraphicsInc/3d-tiles-samples/blob/master/server.js#L50-L56
to the copy of server.js in master.
https://github.com/AnalyticalGraphicsInc/cesium/blob/master/server.js#L49-L50

@hpinkos I agree but..is it needed? I mean, even without this code, CORS requests seem to be handled correctly. Sorry about the trouble, but I really need to understand before I act :smile:

I just tried to use an asset cross origin and it didn't work.

In one console, node server.js --port 8080
In a second cosole, node server.js --port 8082
Navigate to http://localhost:8082/Apps/Sandcastle/index.html?src=Hello%20World.html&label=Showcases
Paste and run this code sample:

var viewer = new Cesium.Viewer('cesiumContainer', {
    baseLayerPicker: false,
    imageryProvider: Cesium.createTileMapServiceImageryProvider({
        url : require.toUrl('http://localhost:8080/Source/Assets/Textures/NaturalEarthII')
    })
});

You will see CORS errors in the console because the server at localhost:8080 does not allow cross origin access to the server at localhost:8082 that is trying to load the tiles from the first server
Adding the access control headers should fix this

@hpinkos You were right! I wasn't testing it properly. Thanks for the info.


histogram tests: data

[ ] Start by creating another method that will check that the bin height for each bin is correct. and then write tests for that too.

INstructor helper content - let's think about this . more

For histogram data tests. What if we have a wrapper that accepts a mpl plot. and returns the data for a histogram (i could see us doing this for other plot types as well). This might entail pulling out some of the code from the assert statements into helper functions that grab the data from the plot.
we can make the function "smart" to test for how many columns are in the data.

also addressed in #143


Add files via upload

This file creates a predictive model for running duration using distance, elevation gain and pace as input

It doesn't look like you've changed any files, please retry with a more clear commit message


The add new task , sort the tasks modal should be responsive when viewing the Site in landscape mode .

You must have observed the modal is not exactly responsive when viewing the site in landscape mode  . You need to fix it .

I want to work on this issue under JWOC

@Shruti025  I have assigned the issue to you, Best Wishes, for any problem reach out to me on discord.

Sure, thanks!


Add inter-op BBE's for array, vararg, fields

Thanks @MaryamZi did the fixes, please re-revew.

@MaryamZi Thank you very much for the fixes.


Spatial Fragment Shader code causes "Program Compilation Failed" error

Godot version:
latest master branch
OS/device including version:
Ubuntu Linux 16.04 64 bit
Issue description:
I've tried to port this shader (https://www.youtube.com/watch?v=RLAG4RbT-5U) to GD 3.0 and I get some errors in console while no line  is marked in the editor. This makes it impossible for me to code a 3d spatial fragment shader.
Steps to reproduce:
Code:

shader_type spatial;
void fragment() {
	sampler2D grass;
	sampler2D rock;
	sampler2D dirt;
	sampler2D splatmap;
	vec3 result;
	vec3 grasscol;
	vec3 rockcol;
	vec3 dirtcol;
	float grassval = texture(splatmap,UV).g;
	float rockval = texture(splatmap,UV).r;
	float dirtval = texture(splatmap,UV).b;
	grasscol = texture(grass,UV).rgb * grassval;
	rockcol = texture(rock,UV).rgb * rockval;
	dirtcol = texture(dirt,UV).rgb * dirtval;
	result = grasscol+rockcol+dirtcol;
}

Error Output:
drivers/gles3/shader_gles3.cpp:167 - SceneShaderGLES3: Fragment Program Compilation Failed: 0:1163(2): error: opaque variables must be declared uniform 0:1164(2): error: opaque variables must be declared uniform 0:1165(2): error: opaque variables must be declared uniform 0:1166(2): error: opaque variables must be declared uniform 0:1171(27): warning:m_splatmap' used uninitialized
0:1172(26): warning: m_splatmap' used uninitialized 0:1173(26): warning:m_splatmap' used uninitialized
0:1174(22): warning: m_grass' used uninitialized 0:1175(21): warning:m_rock' used uninitialized
0:1176(21): warning: m_dirt' used uninitialized drivers/gles3/shader_gles3.cpp:440 - SceneShaderGLES3: Fragment Program Compilation Failed: 0:1163(2): error: opaque variables must be declared uniform 0:1164(2): error: opaque variables must be declared uniform 0:1165(2): error: opaque variables must be declared uniform 0:1166(2): error: opaque variables must be declared uniform 0:1171(27): warning:m_splatmap' used uninitialized
0:1172(26): warning: m_splatmap' used uninitialized 0:1173(26): warning:m_splatmap' used uninitialized
0:1174(22): warning: m_grass' used uninitialized 0:1175(21): warning:m_rock' used uninitialized
0:1176(21): warning: m_dirt' used uninitialized drivers/gles3/shader_gles3.cpp:448 - Method/Function Failed, returning: __null drivers/gles3/shader_gles3.cpp:122 - Condition ' !version ' is true. returned: false ./drivers/gles3/shader_gles3.h:376 - Condition ' !version ' is true. returned: -1 ./drivers/gles3/shader_gles3.h:376 - Condition ' !version ' is true. returned: -1 ./drivers/gles3/shader_gles3.h:376 - Condition ' !version ' is true. returned: -1 ./drivers/gles3/shader_gles3.h:376 - Condition ' !version ' is true. returned: -1 ./drivers/gles3/shader_gles3.h:376 - Condition ' !version ' is true. returned: -1

Minimal reproduction project:
splatmap.zip

[ ] I searched the existing GitHub issues for potential duplicates.
Similar issue: https://github.com/godotengine/godot/issues/13022

Background information:
shader_type spatial;
void fragment() {
	uniform sampler2D grass;
	uniform sampler2D rock;
	uniform sampler2D dirt;
	uniform sampler2D splatmap;
	vec3 result;
	vec3 grasscol;
	vec3 rockcol;
	vec3 dirtcol;
	float grassval = texture(splatmap,UV).g;
	float rockval = texture(splatmap,UV).r;
	float dirtval = texture(splatmap,UV).b;
	grasscol = texture(grass,UV).rgb * grassval;
	rockcol = texture(rock,UV).rgb * rockval;
	dirtcol = texture(dirt,UV).rgb * dirtval;
	result = grasscol+rockcol+dirtcol;
}

Error Output:
:4 - Expected expression, found: OUT drivers/gles3/rasterizer_storage_gles3.cpp:1671 - Condition ' err != OK ' is true.

The first line 'uniform sampler2D grass;' is marked and I got the same error for every hint (vec4, int, float, sampler2D). That's why I've removed "uniform".

Move your uniforms out of the fragment() method.  The documentation indicates that these are global to the shader, so I would define them as such.

Thank you for your advice and work, thought that it was some engine bug. I close this issue now as the error was triggered by an invalid shader script.

This is a bug, I'm surprised the compiler allowed it


Deprecate lnpbp_bech32 in favor of baid58

It become apparent that the use of Bech32m encoding was a mistake. You can read more about this in here:
https://github.com/UBIDECO/rust-baid58/blob/0d9f2b2af90212212b0e9ca5723788bf07b5607b/README.md?plain=1#L72-L94
Additionally to what is said there, Bech32(m) simply doesn't work for anything longer than 90 encoded characters: it is a clearly stated in the BIP-173:

A Bech32[2] string is at most 90 characters long

The reason for the length limit is the fact that the error correction can't handle longer strings; and 6 character checksum appended to the end of the Bech32(m) string is just a waste of space in this case - plus "safety" guarantees of the Bech32 are broken.
Instead, recommend Base58 encoding using additional optional checksums and human-readable identifiers: Baid58.

ACK, it looks good, I'd prefer this over Base64.


[GAE-Java] Add necessary configurations in pom.xml to publish GRAPE jdk to maven central repo

Add some configuration and plugins in pom.xml, so we can publish to maven central repo.
https://issues.sonatype.org/browse/OSSRH-85660
Fixes #2165

Codecov Report

Merging #2166 (f7ba685) into main (321d938) will decrease coverage by 7.12%.
The diff coverage is n/a.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2166      +/-   ##
==========================================
- Coverage   72.93%   65.81%   -7.13%
==========================================
  Files          89       89
  Lines        9789     9789
==========================================
- Hits         7140     6443     -697
- Misses       2649     3346     +697

Impacted Files
Coverage Δ

...aphscope/tests/kubernetes/test_resource_builder.py
0.00% <0.00%> (-100.00%)
:arrow_down:

...on/graphscope/tests/kubernetes/test_demo_script.py
0.00% <0.00%> (-91.75%)
:arrow_down:

python/graphscope/deploy/kubernetes/utils.py
11.90% <0.00%> (-58.58%)
:arrow_down:

python/graphscope/deploy/kubernetes/cluster.py
23.01% <0.00%> (-51.70%)
:arrow_down:

python/graphscope/tests/unittest/test_java_app.py
53.01% <0.00%> (-46.99%)
:arrow_down:

...thon/graphscope/tests/kubernetes/test_with_mars.py
0.00% <0.00%> (-44.83%)
:arrow_down:

python/graphscope/analytical/app/java_app.py
24.36% <0.00%> (-27.92%)
:arrow_down:

...n/graphscope/deploy/kubernetes/resource_builder.py
30.52% <0.00%> (-26.32%)
:arrow_down:

python/graphscope/interactive/query.py
71.95% <0.00%> (-17.08%)
:arrow_down:

python/graphscope/client/rpc.py
74.04% <0.00%> (-12.22%)
:arrow_down:

... and 10 more

Continue to review full report at Codecov.

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


Release 7.1

Summary
Merging weblate translations into release
Ticket Link
Related Pull Requests
https://github.com/mattermost/mattermost-webapp/pull/10652
Release Note
NONE

Hello @ctlaltdieliet,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

E2E tests not automatically triggered, because PR has no approval yet. Please ask a developer to review and then try again to attach the QA label.

@amyblais @Willyfrog
Cherry-pick the weblate translations


feat(object_recognition_utils): check polygon area on iou calculation

Description
Fix a issue reported by @mebasoglu
ISSUE https://github.com/autowarefoundation/autoware.universe/issues/6617
This PR is alternative of PR https://github.com/autowarefoundation/autoware.universe/pull/6618.
Tests performed

A normal recompute simulation under TIER IV data is performed.
The issue related data should be confirmed

Effects on system behavior
Prevent divide by zero on IoU calculation even abnormal objects size are come.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

Dear @mebasoglu,
This is a replaced fix for the issue you reported.
Can you test it with the data you have?

Hello @technolojin , I have tested with our data and confirmed that it works 👍. Thank you.

@scepter914 Can I request to review this PR?


question: why pass by reference in sc_str_substring((char** str, uint32_t start, uint32_t end)?

I'd say I am still new to learning to code in C. This library has been really helpful for me to see how C code is written in the real world. So, thank you!
I was wondering why pass the str by reference in some cases sc_str_destroy() and sc_str_trim(), because I think it's possible to work with pointers without it. For example:
void alter(char* str) {
  str[0] = 'f';
  free(str);
}

int main() {
  char* str = malloc(6);
  strcpy(str, "hello");
  alter(str);
}

Hi @midnqp, glad this repo is being helpful to you.
Just to be clear, you wonder why it passes double pointer e.g. char** instead of char* right? Reason is that some functions might need to change the value of the pointer.
For example, here: see the link
This function makes another allocation and updates the value of the pointer. To do that, you need to get char**.
So, you need to get "pointer to pointer" if you need allocate new memory in your function. In other cases, as in your example above, if you just want to change its content (its characters) it is not necessary.
One exceptional case is sc_str_destroy(). You may use char* as it just calls free() for the pointer.  But just to make it safe to call twice, this function sets pointer to NULL. See: link
So, user does not have to remember if it called sc_str_destroy() before, it will be safe to call it multiple times.
e.g:
char *c = sc_str_create("hello");
sc_str_destroy(&c); // This call will set c = NULL
...
sc_str_destroy(&c); // Safe to call it twice.

// Otherwise you would need to write
if (c != NULL)  {
    sc_str_destroy(c);
    c = NULL;
}
// So, it saves you adding these NULL if checks in your application.

Let me know if this is clear? Cheers!

Wow, this is really simple and clear as a crystal 🔮 Thanks for your time in explaining this to me! I actually did notice you mention that everytime we use set, create, append a new pointer is created. Perfect!
However, we could extend the previous pointer's memory. But.. oh, realloc() still malloc()s under the hood. So we don't have a choice other than to create a pointer everytime.
Wow, this library is well-built.

Be careful with realloc, it is not guaranteed to return the same value even when you call it to shrink the memory. I've seen that is happening, I had to debug a painful issue :)
So, you don't have any other way, you have to get char** if you want to (potentially) update the pointer.

One last thing: kindly elaborate a tiny bit on this please? "it is not guaranteed to return the same value" - I understand realloc() returns a new pointer, but not the same value?

I understand realloc() returns a new pointer, but not the same value?

My point is realloc() may return same value on some calls and it may return another value on other calls. Because allocator (malloc(), realloc() implementation) is free to return whatever it wants.
void *x = malloc(10);

// y points to same memory with x
void *y = x;

// Extend memory allocation. After realloc(), will y be equal to x?
y =  realloc(y, 20);

// Assume that x points to a memory which has already 20 bytes empty space,
// then allocator can mark your allocation as 20 bytes and it can return the same pointer. (y == x);
//
// If there is not enough space (assume x points to 10 bytes empty space and there are used bytes
// by someone else just after that) , allocator has to return another pointer to you (y != x)

So, my point is we cannot say realloc() will return same or different value. It totally depends on allocator implementation.
Your interpretation is correct, if we assume realloc() will return new pointer all the time, we won't make mistakes but as you can see above, in reality, that might not be true :)
Let me know if it is clear.

@midnqp please feel free to open issue if you have other questions! Cheers!


Allow tagging of release and hotfix branches to also trigger deployments

This will allow for pre-release packages to be published to NuGet and Chocolatey, etc.

@pascalberger if/when you have a need, can you take this for a spin?  You should now be able to publish to NuGet/Chocolatey etc, when you have a tagged build on a release/hotfix branch.

👍 Will test it with the next beta release of Cake.Prca

@pascalberger said...
Will test it with the next beta release of Cake.Prca

Nice one!


🧤 Gloves: provide an easier alternative to writing harnesses

As discussed here https://github.com/jscutlery/test-utils/issues/4 and here https://github.com/angular/components/issues/20871, component harnesses are easy to use but hard to implement.
It would be nice to provide another approach with the following constraints:

[ ] It should be easier than writing a harness,
[ ] It should be easier than hacking the DOM,
[ ] It should be framework agnostic,
[ ] 🤔 Should it be able to interact with the state?

I understand that the goal is:

to provide a way of using existing harnesses with Cypress

One of the key features of harnesses is the test environment abstraction and harness reuse through environments (TestBed, Protractor etc...) but if I am using TestBed and Cypress and if I can't reuse my harnesses with Cypress then it somewhat defeats the purpose of harnesses.

This means that what this library can control is:

What is in the test spec

Loading a harness
custom commands to make more readable tests and enable them to have proper types.

base abstractions for new test harnesses, e.g. default implementation of static with and shared functions like selecting elements

In regards to the below constraints and the goal:

It should be easier than writing a harness - we need to write harnesses in a cdk way and the only thing we can do to make it easier is abstractions and shared functions
It should be easier than hacking the DOM - to allow harnesses to be used outside of cypress they need to exclusively use the TestElement, i.e. the harness cannot use cy internally.
It should be framework agnostic - the harness class should be framework agnostic, but the instantiation of it should be framework specific, i.e. getHarness in this library is specifically getting a cypress version of the harness. Maybe there could also be a lighter weight instantiation for when you don't need the TestBed, e.g. in storybook and E2E tests.
Should it be able to interact with the state? What do you mean by state. In general, I think the state should only be changed by interacting with the UI as that is what you are testing.

Can you define what you think the library should have in terms of the:

base ComponentTestHarness class for abstractions and shared functions
custom commands to make more readable tests and enable them to have proper types
harness instantiation

I understand that the goal is:

to provide a way of using existing harnesses with Cypress

One of the key features of harnesses is the test environment abstraction and harness reuse through environments (TestBed, Protractor etc...) but if I am using TestBed and Cypress and if I can't reuse my harnesses with Cypress then it somewhat defeats the purpose of harnesses.

This means that what this library can control is:

What is in the test spec

Loading a harness
custom commands to make more readable tests and enable them to have proper types.

base abstractions for new test harnesses, e.g. default implementation of static with and shared functions like selecting elements

In regards to the below constraints and the goal:

It should be easier than writing a harness - we need to write harnesses in a cdk way and the only thing we can do to make it easier is abstractions and shared functions
It should be easier than hacking the DOM - to allow harnesses to be used outside of cypress they need to exclusively use the TestElement, i.e. the harness cannot use cy internally.
It should be framework agnostic - the harness class should be framework agnostic, but the instantiation of it should be framework specific, i.e. getHarness in this library is specifically getting a cypress version of the harness. Maybe there could also be a lighter weight instantiation for when you don't need the TestBed, e.g. in storybook and E2E tests.
Should it be able to interact with the state? What do you mean by state. In general, I think the state should only be changed by interacting with the UI as that is what you are testing.

Can you define what you think the library should have in terms of the:

base ComponentTestHarness class for abstractions and shared functions
custom commands to make more readable tests and enable them to have proper types
harness instantiation


Fix error finding the port

@jcabrerizo  Looks good, let's just wait for the build too pass and merge it.


Adding a button to change themes

There is no light mod toggle button in the nav

Yeah but the theme is changed a lot. As the theme on the main hacktoberfest site is dark. I think this should be dark too

Hi @max-programming , then you can close this issue, Right?


Add the ability to specify restart policies in services

There is the ability to specify restart policies in containers (#896), but there is no ability to specify them for services.
Having had a superficial look at the pull request that closed #896, it seems that this is implemented, but simply does not appear in the GUI for creating services (although it appears in the GUI for containers).

#896 was related to templates.
Indeed, there is no support for yet for restart policies in services. Will track it as an enhancement.

I think the restart policy is not implemented for services cause swarm orchestrator do the job. You just have to define the replicas and the constraints, and swarm take care of that.
You can try to run a nginx service with one replica, then  docker rm -f {container}. Swarm will rerun the container to satisfy the one replica schedule rule

Not correct...
Cli -> Docker service create —restart-condition any/none/on-failure
Rgds,
Neil Cresswell
On 10/10/2017, at 9:24 AM, Thomas Krzero <notifications<EMAIL_ADDRESS>wrote:
I think the restart policy is not implemented for services cause swarm orchestrator do the job. You just have to define the replicas and the constraints, and swarm take care of that.
You can try to run a nginx service with one replica, then docker rm -f {container}. Swarm will rerun the container to satisfy the one replica schedule rule
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_portainer_portainer_issues_1178-23issuecomment-2D335275266&d=DwMFaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=Ph4Kscmu8KLf2zx5TBMr90tfzOXkqY1kCbjWHKnfen4&s=A5QNodrvPHaQRh4OkBbPhPnAYbYlJNDW68y2spYh0MU&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AWGrlT3gsJVFQ-5FoQ5HEuO4eGZvJl-2D68Aks5sqoDkgaJpZM4PM7IS&d=DwMFaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=Ph4Kscmu8KLf2zx5TBMr90tfzOXkqY1kCbjWHKnfen4&s=0XSqP_oJCygDj5z0VGsi1MmeevK5bol4uwtOQvloxXQ&e=.

Oh yeah, default values are so good to me !
From the docs :
--restart-condition : Restart when condition is met (“none”|”on-failure”|”any”) (default “any”)
--restart-delay : Delay between restart attempts (ns|us|ms|s|m|h) (default 5s)
--restart-max-attempts : Maximum number of restarts before giving up
--restart-window : Window used to evaluate the restart policy (ns|us|ms|s|m|h)

Yeah, I have an uncommon use case in which I don't want my containers to restart. It's possible to change the restart condition after creating the service, but that's not very user friendly.


window_equalize function not setting equal observation dates

Using this code I get an ValueError: object1 and object2 do not have the same observation dates
balance_sheet = fp.series('WALCL')
rrp = fp.series('RRPONTTLD')
tga = fp.series('WTREGEN')
balance_sheet, rrp, tga = fp.window_equalize([balance_sheet, rrp, tga])

balance_sheet.minus(rrp)

Hi wetcircle,
The issue is that WALCL and RRPONTTLD have different frequencies. The first
is Weekly, as of Wednesday ('W-WED' in Pandas). The second is Daily ('B' in
Pandas). To combine them in a mathematical expression, you have to resample
one of them; most likely convert RRPONTTLD to W-WED frequency.
fredpy doesn't handle the rich set of frequencies that Pandas allows for.
So I would use fredpy to import the data and then just work with the data
using Pandas methods. Here's an example:
import fredpy as fp
import pandas as pd
Import data from FRED and store only data (not series metadata)
balance_sheet = fp.series('WALCL').data
rrp = fp.series('RRPONTTLD').data
tga = fp.series('WTREGEN').data
Reample rrp to Weekly, Wednesday by averaging
rrp = rrp.resample('W-WED').mean()
Concatenate data into a DataFrame
df = pd.concat([balance_sheet, rrp, tga],axis=1).dropna()
df.columns = ['balance_sheet','rrp','tga']
Difference
diff = df['balance_sheet'] - df['tga']
print(diff)
Hope this helps.
Brian
On Tue, Aug 23, 2022 at 7:50 PM wetcircle @.***> wrote:

Using this code I get an ValueError: object1 and object2 do not have the
same observation dates
`balance_sheet = fp.series('WALCL')
rrp = fp.series('RRPONTTLD')
tga = fp.series('WTREGEN')
balance_sheet, rrp, tga = fp.window_equalize([balance_sheet, rrp, tga])
balance_sheet.minus(rrp)`
—
Reply to this email directly, view it on GitHub
https://github.com/letsgoexploring/fredpy/issues/3, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB2ZTBES7VZKLQYUO7273L3V2WEXHANCNFSM57NUBXYQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Yes that helps a lot. Thanks for the reply!


Upgrade pygments version to ^2.4.1

Changelog / Overview

Upgrade pygments version to ^2.4.1

Motivation

Prevents installation errors
Explanation for Changes

jupyterlab-pygments 0.1.2 has requirement pygments >= 2.4.1
nbconvert 6.0.7 has requirement pygments >= 2.4.1

Testing Status

Further Comments

Checklist

[x] I have read the Contributing Guidelines
[x] I have written a descriptive PR title (see top of PR template for examples)
[x] I have written a changelog entry for the PR or deem it unnecessary
[x] My new functions/classes either have a docstring or are private
[x] My new functions/classes have tests added and (optional) examples in the docs
[x] My new documentation builds, looks correctly formatted, and adds no additional build warnings

Reviewer Checklist

[ ] The PR title is descriptive enough
[ ] The PR is labeled correctly
[ ] The changelog entry is completed if necessary
[ ] Newly added functions/classes either have a docstring or are private
[ ] Newly added functions/classes have tests added and (optional) examples in the docs
[ ] Newly added documentation builds, looks correctly formatted, and adds no additional build warnings

irrc, poetry should be handling it right?

irrc, poetry should be handling it right?

When I used it, it wasn't handling it properly and kept pygments on 2.3.1

When I used it, it wasn't handling it properly and kept pygments on 2.3.1

I tried in a fresh env, Poetry installed 2.8.1. Manim doesn't need a specific version of pygments (prove me wrong!), so I don't think we should specify a version.

When I used it, it wasn't handling it properly and kept pygments on 2.3.1

I tried in a fresh env, Poetry installed 2.8.1. Manim doesn't need a specific version of pygments (prove me wrong!), so I don't think we should specify a version.

It seems like a one time error as I'm unable to reproduce it anymore. It may have been caused by installing packages while in poetry shell. If it does happen again I'll post a screenshot of it.


Provide an option to use run image with preinstalled JRE/JVM

Can you provide an option to use run image with preinstalled Java?
This will allow to use security team pre-approved base images with Java.
Now we cannot actually make use of buildpacks.

@vkochnev Thanks for your request.  Unfortunately, supporting pre-installed JVMs isn't in the spirit of the Cloud Native Buildpacks project and therefore we won't be doing this.
To give you a little context, Stacks (the base image analog for Cloud Native Buildpacks) should only provide ABI-compatible software.  The JVM has never been (and admittedly never claimed to be) ABI compatible between releases.  Therefore it is unsafe for us to rebase an application's image that has a JVM in it without retesting the entire thing from scratch.
We prove Cloud Native Buildpacks for a number of popular JVM distributions

AdoptOpenJDK
Amazon Corretto
Azul Zulu
BellSoft Liberica
Eclipse OpenJ9
GraalVM
SapMachine

and if there is another OpenJDK-based freely distributable distribution that you'd like us to support, please feel free to open an issue about it.  If you can't use one of the seven current distributions, you'd be better off creating your own buildpack to provide the enterprise sanctioned binaries you can use.  libjvm does the heavy lifting and If you look at [BellSoft Liberica] for example, you'll find that the entire buildpack code is contained in a single Golang file and a list of dependencies.

@nebhale I understand the rationale and appreciate your explanation. Creation of a custom buildpack definitely doesn't look too difficult and we will check if it is worth investing in this. Or maybe we'll use another approach. Thanks!

@vkochnev Please keep in touch and let us know either how it goes or how we can help you.  As you can see, it’s not a technical issue (we’d have already done it if it was!), but a philosophical one.  We work with many enterprises that have these kinds of requirements and I’m interested in hearing what ended up being acceptable for yours.


Support for ints in hex/binary format?

Hello,
Love this parser. Do you have plans to support hex/binary inputs for integers? I can write it if you think it's a good idea. Let me know.

This parser implements TOML specification. As long as TOML spec supports hex/binary inputs, I will have a plan. But currently there is no such spec?

See also https://github.com/toml-lang/toml/issues/409

Oh haha. I didn't know toml specified that. Well, thanks for your work on this.
Get Outlook for iOShttps://aka.ms/o0ukef

From: mayah<EMAIL_ADDRESS>Sent: Wednesday, June 7, 2017 8:38:48 AM
To: mayah/tinytoml
Cc: Eric Hargitt; Author
Subject: Re: [mayah/tinytoml] Support for ints in hex/binary format? (#24)
This parser implements TOML specification. As long as TOML spec supports hex/binary inputs, I will have a plan. But currently there is no such spec?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/mayah/tinytoml/issues/24#issuecomment-306781855, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGYy3rsKLQG_3QaeMkUuIF7h3MKJrrMbks5sBpnYgaJpZM4NymvR.

Closing


Fix pose of plane visual with non-default normal vector

Trying to verify my comment led me to this bug. It looks like the rotation due to a plane's normal vector is accounted for during the creation of the visual but not in pose update. The creation is in: https://github.com/ignitionrobotics/ign-gazebo/blob/ddd7fcda563b1f8c47c92610181a2d5f797c80bc/src/rendering/SceneManager.cc#L261-L267  and the pose update  is done in: https://github.com/ignitionrobotics/ign-gazebo/blob/ddd7fcda563b1f8c47c92610181a2d5f797c80bc/src/rendering/RenderUtil.cc#L860 https://github.com/ignitionrobotics/ign-gazebo/blob/ddd7fcda563b1f8c47c92610181a2d5f797c80bc/src/rendering/RenderUtil.cc#L425
To test this, run the shapes.sdf example and set the plane visual's normal to 1 0 0.

diff --git a/examples/worlds/shapes.sdf b/examples/worlds/shapes.sdf
index 5dec565b..2136df07 100644
--- a/examples/worlds/shapes.sdf
+++ b/examples/worlds/shapes.sdf
@@ -35,3 +35,3 @@ Try moving a model:
             <plane>
-              <normal>0 0 1</normal>
+              <normal>1 0 0</normal>
             </plane>
@@ -42,3 +42,3 @@ Try moving a model:
             <plane>
-              <normal>0 0 1</normal>
+              <normal>1 0 0</normal>
               <size>100 100</size>

The plane should intersect with the shapes like so

Without this patch, the plane is not affected

I believe this fixes #26.

looks good to me. I think the remove of user data should be fine as well since we don't expect the geometry pose to change at run time. Approving once builds turn green

looks good to me. I think the remove of user data should be fine as well since we don't expect the geometry pose to change at run time. Approving once builds turn green


allow setting and closing connection on the fly

https://github.com/brycedarling/redux-autobahn-js/issues/2
https://github.com/brycedarling/redux-autobahn-js/issues/7
Perhaps not the best approach. it manages 3 internal variables _connection _session _dispatch
user can set connection and close connection (on the case of logout) via middleware.setConnection and middleware.closeConnection.

also you don't want to pass Connection object when middleware is created.
E.g.,

  const middlewares = [
    sagaMiddleware,
    routerMiddleware(history),
    AutobahnMiddleware,
  ];

I found no build script. let me know some details for contribution then I'll update this PR to meet your requirements.
Cheers

Looks good enough to me! I'd probably want to wrap some tests around the new functionality, but I just don't have time right now for it, so I really appreciate you taking your time to add this stuff as it'll help some other people out. Thanks!


Revise the API

Right now, a sample sketch looks like that:
#include <Homie.h>

const int PIN_RELAY = D1;

void inputHandler(String node, String property, String message) {
  if (node != "light" || property != "on") {
    return;
  }

  if (message == "true") {
    digitalWrite(PIN_RELAY, HIGH);
    Homie.sendProperty("light", "on", "true"); // update property, act as a feedback
  } else if (message == "false") {
    digitalWrite(PIN_RELAY, LOW);
    Homie.sendProperty("light", "on", "false");
  }
}

void setup() {
  pinMode(PIN_RELAY, OUTPUT);
  digitalWrite(PIN_RELAY, LOW);
  Homie.setFirmware("awesome-relay" ,"1.0.0");
  Homie.addNode("light", "light");
  Homie.addSubscription("light", "on");
  Homie.setInputHandler(inputHandler);
  Homie.setup();
}

void loop() {
  Homie.loop();
}

I think addNode, addSubscription and setInputHandler are kind of messy. I would see something like that instead:
#include <Homie.h>

const int PIN_RELAY = D1;

HomieNode light("light", "light");

void lightOnHandler(String message) {
  if (message == "true") {
    digitalWrite(PIN_RELAY, HIGH);
    Homie.sendProperty(light, "on", "true");
  } else if (message == "false") {
    digitalWrite(PIN_RELAY, LOW);
    Homie.sendProperty(light, "on", "false");
  }
}

void setup() {
  pinMode(PIN_RELAY, OUTPUT);
  digitalWrite(PIN_RELAY, LOW);
  Homie.setFirmware("awesome-relay" ,"1.0.0");
  light.subscribe("on", lightOnHandler);
  Homie.registerNode(light);
  Homie.setup();
}

void loop() {
  Homie.loop();
}

There are several benefits:

Feels cleaner
You only have to define your node ID on the HomieNode object
We can add three types of input handlers:

As shown above, directly on a property of a node (void lightOnHandler(String message))
On a node, with a third parameter to HomieNode (void lightHandler(String property, String message))
Like we already have for now, a global input handler (void globalHandler(String node, String property, String message))

What do you think? @enavarro222 @furyfire

:+1: looks nice. It should avoid -always the same- if/elseif in global handler.
Maybe handler could return a boolean that indicate if more specific ones should be call or not. For example on a msg:

global handler is called,
if returns false, then node level handler is called
if returns false, property handler is called.
I'm not sure about the order (global to property level, or reverse...). At least if you want to have a logging on each msg, better to have global first.

Other good think I see is to create generic HomieNode by inheritance. For instance one can have a RelayNode, that you can use like that:
#include <Homie.h>
#include <HomieRelayNode.h>

HomieRelayNode light("light", "light", D1);

void setup() {
  Homie.setFirmware("awesome-relay" ,"1.0.0");
  light.setup(false); // set the initial state
  Homie.registerNode(light); // do the subscribe and callback binding
  Homie.setup();
}

void loop() {
  Homie.loop();
}

Your propagation idea is a great one! Will definitely implement it. Yes, global to property is the best option.
The inheritance idea is complicated. The simplest case implies a relay, as you show. The goal is simple: set a relay HIGH or LOW. The fact is, some relays are HIGH-active, others are LOW-active, so there is already a complexity. Homie for ESP8266 goal is to throw the network complexity away, not hardware ones -  at least not for now -. ;)
BTW, to implement SSL and user/pass auth, the JSON config structure has to be changed. What do you think of:
{
  "name": "kitchen light",
  "wifi": {
    "ssid": "Network_1",
    "password": "I'm a Wi-Fi password!"
  },
  "mqtt": {
    "host": "<IP_ADDRESS>",
    "port": 35589,
    "username": "user",
    "password": "pass",
    "ssl": true,
    "fingerprint": "CF 05 98 89 CA FF 8E D8 5E 5C E0 C2 E4 F7 E6 C3 C7 50 DD 5C"
  },
  "ota": {
    "host": "<IP_ADDRESS>",
    "port": 35590,
    "username": "user",
    "password": "pass",
    "path": "/ota",
    "ssl": true,
    "fingerprint": "CF 05 98 89 CA FF 8E D8 5E 5C E0 C2 E4 F7 E6 C3 C7 50 DD 5C"
  }
}

Yep, maybe inheritance is out of the scope on Homie (to keep it generic and lightweight), however it could be done in third part lib. For instance I got few devices with similar hardware (a buzzer, RGB lamps, ...). I already have some class that handle that hardware, for now the (always the same) binding with Hommie/MQTT is made by hand in the main file... It could be factorize in the hardware class (that inherit HomieNode).
New config json looks ok. It will also permit to have a ota server different of the mqtt one, that's nice.

I implemented a proof of concept where homie would get the clock from and ntp server.
This allows me to use the https://github.com/PaulStoffregen/Time and the https://github.com/PaulStoffregen/TimeAlarm modules which simplifies the main loop alot.
The timemodule let's you schedule events periodically or at fixed times.

And I just posted on the wrong issue. Disregard that last comment

Time was something I thinked about, thanks for that. Wrong issue as you said, but just be sure nothing fails if there is no internet connection, I want Homie-ESP8266 to be cloud-independant.

The time.h lib simply starts the clock from UNIX epoch. Every schedule you start will simply run relative to that.

Both the config API and the new API with node and property levels handlers have been implemented. Examples have been updated to reflect the changed.
The next big step is the documentation.

The API in the README is not up-to-date about the handlers, but it works the way it is described in the first post, and as @enavarro222 said:

global handler is called,

if returns false, then node level handler is called
if returns false, property handler is called

This is probably not that clear, but it will be when the docs will be finished. :)

Docs are in place, I can close this.

Nice ! good job ;)


What is scalar_on_right?

I find the name and documentation for this argument not very informative. It seems calling _MinusScalar(sym, scalar=2, scalar_on_right=True) means 2 - sym? Then shouldn't the argument be scalar_on_left?

I think so. Default will be left.

I think we need to fix this, rename scalar_on_right->scalar_on_left, and fix this case https://github.com/dmlc/mxnet/blob/master/src/operator/elementwise_binary_scalar_op-inl.h#L139
@yajiedesign

cc @yajiedesign who contributed the elementwise binary op. It is partly due to my uncareful code review.
I think two things need to be fixed

rename scalar_on_right to scalar_on_left, which is what it really means on code.
Check https://github.com/dmlc/mxnet/blob/master/src/operator/elementwise_binary_scalar_op-inl.h#L139 , which actually is logic for right operand, and change it

https://github.com/dmlc/mxnet/pull/653 change ok


Support @access and @usage

Fix https://github.com/Azure/autorest.csharp/issues/3519.
We are introducing the decorator @access and @usage.
This PR does not handle orphan scenario because we haven't uptake the getAllModels.
Also include https://github.com/Azure/typespec-azure/pull/3347

/azp run


requirements problem

git and zipfile
pip doesn't find anything and halts installation
when done manually, but looking closer at the console when UI loads, it says they're installed... not sure what's going on, but I'm n00b level
opencv-python overwrites an existing install by
WAS Node Suite: OpenCV Python FFMPEG support is enabled

what's your enviroment?
There is some problem with open-python and was? because i don't want to use an older version if is not strictly necessary.

yes, it seems Was uses an older version.  The other two aren't causing problems, I was just curious why pip doesn't recognize them since everything in "requirements.txt"  usually returns something

but there is some error in was? or even the newer version is working?
The only reason for the error in pip i think is that is running under anothe installation of python that doesn't have those modules installed. Do you have a specific error?

I'm not sure if WAS breaks if the newest version is installed... I was doing so many permutations, I forgot

but I'm sure other nodes keep upgrading numpy and that breaks compatibility with a module called "numba" which then breaks WAS

very BUGest install

but I'm sure other nodes keep upgrading numpy and that breaks compatibility with a module called "numba" which then breaks WAS

yes, I've edited my .bat so that after the numpy upgrade due to the installation of llama-cpp-python, it restores it to a version compatible with numba


Remove console log line (#42) and tableToolbar (#43)

This request is for issue 42 and 43
Issue 42: remove unwanted console log
Issue 43: property tableToolbar
What is also in here is a new option to be able to insert html (OInsertHtml). But you might not want this. I don't know how to do a pull request of a specific commit. Next time I'll make more branches (one per issue)

:tada: This PR is included in version 1.8.0 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.8.0 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Add character: Lars

Hero Description
Lars is a aquatic character, he is agile and tricky.
Concept Art

Inspiration
Lars is a character inspired by, big daddy from Bioshock, Nautilus (League of Legends), and Murquinho from HOTS.
Moves
Idle

[STICK] Walk
Lars runs with the anchor in his shoulder.

[LEFT] Look! overhead!
Lars uses his anchor by throwing it up towards the enemy, the anchor falls on the enemy, then lars pulls back the anchor next to the affected enemies.
[MIDDLE] Heavy Load
Lars uses his anchor to hit the ground creating an impact of water on the ground that stuns the enemies and creates 7 rocket projectiles that apply a slight slow on the hit enemies, the projectiles would be arranged around Lars in a half circumference.
[RIGHT] Ocean protection
Lars heals, gains armor (damage reduction) and immunity to group control for a few seconds

[Q] Pescado Fillet
Lars throws a giant fish at the opponent, applying damage to it.
Obs: The fishs stays for a while, as a obstacle.
[SPACE] Mossy Zone
Lars invokes algae in a large circle, these algae will apply a relevant slow to enemies that remain in the area, after a period of time, enemies that remain in the algae, receive a "root" effect.

[E] TAUNT
In reference to nautilus from League of Legends, Lars begins to swin in air.

Tasks

[x] Idle
[x] Walk
[x] [LEFT] Look! Overhead! (GIF REFERENCE) *personal reference
[ ] [LEFT] Look! Overhead!
[x] [MIDDLE] Heavy Load (GIF REFERENCE) *personal reference
[ ] [MIDDLE] Heavy Load
[ ] [MIDDLE] Heavy Load _ projectile
[ ] [RIGHT] Ocean Protection_ cast animation
[ ] [RIGHT] Ocean Protection_ fishes
[x] [Q] Pescado Fillet  (GIF REFERENCE) *personal reference
[ ] [Q] Pescado Fillet
[x] [SPACE] Mossy Zone _ cast animation (GIF REFERENCE) *personal reference
[ ] [SPACE] Mossy Zone _ cast animation
[x] [SPACE] Mossy Zone

Approved for work!

wip_01
resume:

Idle
Walkcycle
[LEFT] Look! Overhead! (GIF REFERENCE) *personal reference
[MIDDLE] Heavy Load (GIF REFERENCE) *personal reference
[RIGHT] Ocean Protection_ fishes
[Q] Pescado Fillet (GIF REFERENCE) *personal reference
[SPACE] Mossy Zone

animations updated in the main comment.

wip_02
hours worked: 27 hours and 25 minutes
resume:

[LEFT] Look! overhead! (wip)
[MIDDLE] Heavy Load (wip)

animations updated in the main comment.

wip_03
hours worked: 32 hours and 15 minutes
resume:

[LEFT] Look! overhead!
[MIDDLE] Heavy Load_ castanimation
[MIDDLE] Heavy Load_ impact
[MIDDLE] Heavy Load_ projectile
[TAUNT]

animations updated in the main comment.

wip_04
hours worked: 38 hours
resume:

[RIGHT] Ocean Protection_ cast
[SPACE] Mossy Zone_ cast
[TAUNT]

animations updated in the main comment.

wip_05
hours worked: 43 hours and 50 minutes
resume:

[RIGHT] Ocean Protection_ cast
[SPACE] Mossy Zone_ cast

animations updated in the main comment.


Ingore files under benchmark folder

Do not consider files under the benchmark folder as benchmarks.
This PR will fix the Prow job errors after .ko.yaml was added to the benchmarks folder in serving, see https://prow.knative.dev/view/gcs/knative-prow/logs/ci-knative-serving-recreate-clusters/1275330518623195136
/cc @vagababov

Did it try to create a benchmark with .ko.data name?

Did it try to create a benchmark with .ko.data name?

No, it only tried to create a cluster with the serving--.ko.yaml name.


#399 Fix codacy report upload problems

Thanks for your pull request.  Please review the following guidelines.

[x] Title includes issue id.
[x] Description of the change added.
[x] Commits are squashed.
[ ] Tests added.
[ ] Documentation added/updated.
[x] Also please review CONTRIBUTING.md.

LGTM.

https://github.com/paypal/squbs/pull/402 is taking care of this


add status page

Add a status page where all certificates, fingerprints, renewal and expiry dates are listed. Available in html and json.

Implemented in v1.99.9.


Enchancement enchance materialized view where clause match

Proposed changes
enchance materialized view where clause match
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

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

run buildall

run feut

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 35.56 seconds
stream load tsv:          444 seconds loaded<PHONE_NUMBER>9 Bytes, about 160 MB/s
stream load json:         23 seconds loaded<PHONE_NUMBER> Bytes, about 97 MB/s
stream load orc:          71 seconds loaded<PHONE_NUMBER> Bytes, about 14 MB/s
stream load parquet:          30 seconds loaded 861443392 Bytes, about 27 MB/s
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230328105552_clickbench_pr_121799.html

run buildall


NAR hash mismatch in input

here is the full log
🤞 Activating configuration non-hermetically on bee via ssh
+ /nix/store/85fcpkyv8qg4kpnv0p2nql2k7qp6q4ab-openssh-9.0p1/bin/ssh -t<EMAIL_ADDRESS>'sudo nixos-rebuild switch --flake /nix/store/ln57g0jd3pqmdwfkb8vmca17hcfm4awv-source#bee'
[sudo] password for yt:
error: NAR hash mismatch in input 'github:nixos/nixpkgs/72d1b0d0fac131df1ea254b65413c85609bdd2ee' (/nix/store/3jvygw6vzsazkfhxcd66w9gcg7y91c3k-source), expected 'sha256-dLqquYHvj6/EYPljULXJVUKcizB/8cbgdIL7aEXTDZY=', got 'sha256-g5Wo75ddDQmWnL70rJCMm+JJlvHbzPFUePUpuMNn5qk='
(use '--show-trace' to show detailed location information)

just started getting this today, no idea why.
I'm executing from a darwin machine to build a nix machine.
I'm not using the hermetic feature. I'm building on "remote" of course.
Not sure where this is coming from.

Is your full source code and flake that you're trying to work with available on Github anywhere?

yes it is. I'm trying to build bee https://github.com/happysalada/dotfiles/blob/master/flake.nix#L68
by running nix run .#apps.nixinate.bee
you might just need to change the ip and ssh user and you should be fine.
(l think all files are checked by git).

I thought originally that it was because the nix version was different on both machines and so those were computing the NAR differently. I verified since then that the nix version is the same on both machine.
I'm wondering if it would be a darwin vs nixos thing.
It feels like it has nothing to do with nixinate.


Deploy on NAS

Not sure if this possible deploy this on my synology NAS server via docker? Instead of using AWS.

I don't have it so I'm not sure either. Try it out and find out? 😅