Deploy with existing app setting overwrites value in portal with blank

azure-functions-maven-plugin - 1.3.1
        <configuration>           <resourceGroup>${functionResourceGroup}</resourceGroup>           <appName>${functionAppName}</appName>           <region>${functionAppRegion}</region>           <appSettings>             <property>               <name>FUNCTIONS_EXTENSION_VERSION</name>               <value>beta</value>             </property>             <property>               <name>MSDEPLOY_RENAME_LOCKED_FILES</name>               <value>1</value>             </property>             <property>               <name>TRANSACTIONS_EVENT_HUB_CONNECTION_STRING</name>               <value>${TRANSACTIONS_EVENT_HUB_CONNECTION_STRING}</value>             </property>           </appSettings>         </configuration>         <executions>           <execution>             <id>package-functions</id>             <goals>               <goal>package</goal>             </goals>           </execution>         </executions>
@FunctionName("myFunction") public Object run(@EventHubTrigger(name = "myEvent", eventHubName = "myEventHub", connection = "TRANSACTIONS_EVENT_HUB_CONNECTION_STRING", cardinality= Cardinality.ONE, dataType="binary")
Expected behavior
App setting TRANSACTIONS_EVENT_HUB_CONNECTION_STRING is set in the portal, value of EventHub Connection on function should remain the name of the environment variable
Actual behavior
The EventHub Connection is blank, the value of TRANSACTIONS_EVENT_HUB_CONNECTION_STRING in App Settings is also blank after a deploy
Steps to reproduce the problem
mvn clean package
mvn azure-functions:deploy

@josh-roberts-phision Thanks for your report and sorry for the delay. I tried with function plugin 1.3.2 and it works, I could see TRANSACTIONS_EVENT_HUB_CONNECTION_STRING  after deployed with value. Is the property set properly? Or cloud you please share the pom file?


libc test builds two different versions of libc

cd /Development/source/libc/libc-test
cargo test

In output:
Compiling libc v0.2.142
...
Compiling libc v0.2.142 (/Development/source/rust/libc)

At least on Haiku, this causes a linking error as the current libc on crates.io hasn't been updated with a fix to a linking issue, which has been fixed in my local repo (and has landed here, a new release hasn't been cut yet).

0.2.143 will fix this.

Ah, I have more changes to make to the libc crate, but need to be able to test properly. Is there no way for cargo test to use the local libc instead of from crates.io?

I'm not sure if I understand your argument correctly, but wouldn't just cloning the repo and running the test on it be sufficient?

in libc/libc-test/Cargo.toml:
[dependencies.libc]
path = ".."
version = "0.2.142"
default-features = false

But during build, I'm assuming some dependency is pulling in libc, and cargo downloads a copy from crates.io, instead of using libc in the parent directory. Hence why when running cargo test, I get a link failure due to not using the local libc.
Is the libc-test sub-directory not meant to be used directly for cargo test?

You have to add them under [patch.crates-io]. See https://doc.rust-lang.org/cargo/reference/overriding-dependencies.html if you're new to Cargo, it's quite helpful :)

I tried that in the sub directory, but it seemed to have no effect. Is that maybe because it's part of a workspace?

I don't have your workspace and cannot provide further directions, unfortunately :( I'd suggest reading the above docs more and investigating further (also, I think it's not a libc issue anymore but Cargo's usage).


Genycloud optimize and teardown

Description:
Work on #2429 continues here. This is the 2nd iteration, following #2446.
Still a WIP with many things to be done, yet already good enough for evaluating on CI.

Closing in favor of smaller PR's, which I will open soon


Registering data from user causing exception.

There might be an issue when registering the user that causes the exception, particularly when this happens inside AA.
Maybe outside too?  This would require further research.

Tried to reproduce this and seems like working to me.
Closing for now.


Chat - When user send message with triple markdown symbol "fd_fd " mark down applied

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to staging.new.expensify.com
Go to any chat
Enter:

Expected Result:
It must displays the 3 characters and no markdown must be applied like:

Actual Result:
When user send message with triple markdown symbol "fd_fd " mark down is applied
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[x] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/944be97b-98e3-4573-93d5-acba97a73e41
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:

Proposal
Please re-state the problem that we are trying to solve in this issue.
When user send message with triple markdown symbol _fd_fd_  mark down applied
What is the root cause of that problem?
Before save commit, we parse text on getParsedComment of ReportUtils.js.
https://github.com/Expensify/App/blob/2d2008aae95588443adb0814b12151023ddb9792/src/libs/ReportUtils.js#L1296-L1299
During parser.replace(), text is replaced by italic rule.
_a_a_ => a_a

What changes do you think we should make in order to solve the problem?
we need to update regex of italic
What alternative solutions did you explore? (Optional)

This doesn't seem like a problem worth solving


TH support for data families

This is my attempt at adding data family derivation support for Generics.Deriving.TH.
Since it is impractical to pass an entire data instance's type (e.g., Family Int a) to Template Haskell, this works by passing in the name of a data family instance constructor to derive-, which infers the instance it corresponds to:
data family DF a b
data instance DF Int b = DF1 Int
                       | DF2 Int Int

$(deriveAll0And1 'DF1)
-- Alternatively, one could pass 'DF2 instead

Unfortunately, this means that you can't derive Generic(1) for a data instance with no constructors at the moment (although I could probably address this in a different pull request).
Another wart to be aware of is that there is a bug present in GHC 7.8 which can cause the rightmost type variables in a data family instance to be eta-reduced away when looking up data family info via reify, which makes it impossible to know for sure what the names of those type variables are. I added a comment at the top of Generics.Deriving.TH explaining to users how they should work around this.
There's quite a bit of code in this pull request, so please let me know if there's anything that looks odd. I added a test to Examples.hs demonstrating its use, and I also tested this against my text-show test suite, which uncovered some bugs related to PolyKinds (that I corrected with b9c88b82e61383130c4f885493177c2d5cd0cac5).
(P.S. I generalized the typeVariables function into buildTypeInstance, which clashed with the implementation of simplInstance. I'm not exactly sure what simplInstance does, so make sure I didn't accidentally break it.)

This looks very good, thanks!


Create Mesh Export Service Integration Test

Create an integration test which calls the MES using a test user and determines if the tile version of the requested model is supported by the frontend under which the test is running. The test will:

request a tileset from the MES
Check the tile version of the provided tileset
Check if the frontend under which the test is running will support rendering the tileset

While this test in and of itself is not the most useful or robust test, it serves two major purposes. Once the integration testing framework is set up, it will be significantly easier to expand upon the test that is here and create more robust tests. Additionally, the test can serve as an example to other teams of a way tile version compatibility can be checked in their own use cases.

Where will this test be run? As part of the iTwin.js Integration pipeline, or an external repo?

Where will this test be run? As part of the iTwin.js Integration pipeline, or an external repo?

Currently undecided.  Most likely not iTwin.js integration pipeline.

Does it really make sense to have this in this repo then?

That's part of the discussion we're still having. Based on conversations with @arvindsvenkat, we will most likely want it to be an automated test, but we aren't set on where exactly, yet. Open to any suggestions.

Where will this test be run? As part of the iTwin.js Integration pipeline, or an external repo?

Currently undecided. Most likely not iTwin.js integration pipeline.

Does it really make sense to have this in this repo then?

@aruniverse, This test is intended to prevent the error that was encountered when the export tile version changed and wasn't supported by the iTwin viewer. I would assume it's directly correlated to upgrades in the iTwin.js version and the Mesh Export Service exports.
In my opinion, it should be run in the iTwin.js Integration test pipeline and also in the Mesh Export Service release pipeline (possibly scheduled in QA).
cc: @andremig-bentley  @markschlosseratbentley , @danieliborra

When exactly will this test actually run?
rush test:integration expects a script named test:integration, which your package.json lacks.
The iTwin.js Integration - GitHub job runs a hard-coded set of integration tests, which doesn't include this one.

When exactly will this test actually run? rush test:integration expects a script named test:integration, which your package.json lacks. The iTwin.js Integration - GitHub job runs a hard-coded set of integration tests, which doesn't include this one.

After further discussions, we have decided to add this test to the iTwin.js integration pipeline. The command to run the test has been changed to test:integration:tileversion to follow the general integration test naming conventions in core, but we don't want it to run automatically with rush test:integration, so that command does not exist in this package. As of now, this PR will only include the addition of the test, and the pipeline integration will be handled in a later PR.
Additionally, based on discussions with @arvindsvenkat and @markschlosseratbentley, the test has been changed to begin by kicking off an export, then polling for the status of that export, and once it is complete, going forward with the tile version testing on that new export, ensuring we get the latest tile version information from the MES.

where is this actually going to run? i dont see any pipeline changes. you can grep for this within the repo and look at the other tests as an example

I had the same question. @andremig-bentley later stated above:

After further discussions, we have decided to add this test to the iTwin.js integration pipeline. [...snip...]
The command to run the test has been changed to test:integration:tileversion to follow the general integration test naming conventions in core, but we don't want it to run automatically with rush test:integration, so that command does not exist in this package. As of now, this PR will only include the addition of the test, and the pipeline integration will be handled in a later PR.

I'm not sure where the further discussions occurred or whether you were involved in them. I was not.

where is this actually going to run? i dont see any pipeline changes. you can grep for this within the repo and look at the other tests as an example

We had decided to have this PR include only the addition of the test, with pipeline changes coming later as time allowed. If you think we should roll it all into this PR, that can be done.

@pmconne @aruniverse
To clear up any confusion, since the goal of this test is to validate the interoperability between the frontend here and the MES backend, we have decided that this test would be useful to run in two locations: As part of the Mesh Export Service Release Pipeline, and pertinent to this PR, in itwinjs-core. We think it is useful to run it here to validate that any frontend changes do not unintentionally break this connection.
Initially, we had chosen to only include it in the iTwinjs Integration - Github job as a matter of simplicity. However after discussing more with @markschlosseratbentley, we have decided that we will add it as well to rush test:integration as it would be just as useful there.
Additionally, we had also considered having this PR only include the actual addition of the test and not the changes to the pipeline etc. needed to add the test to the iTwinjs Integration - Github job and rush test:integration due to prioritization of other work. Our thought being we could finish this PR, have the test in place, and work on actually connecting it to the correct testing infrastructure as time allowed. However, it seems that it would make more sense to have this PR include all necessary changes for the test to run, so those changes are now WIP and will be included in this PR.
Due to this change in plan and the PR now being considered WIP, I am converting it back to draft and will re-request reviews when it is ready. Let me know if there are any other questions I can answer in the mean time.

/azp run iTwin.js

/azp run iTwin.js Docs - YAML

/azp run iTwin.js Integration - GitHub


KeyVault support for a Nuget Credential provider (.NET Core)

This allows to configure the provider to use Azure KeyVault to store PAT tokens for Nuget feeds.
For cloud deployments with multiple on demand VMs, all that needs to be set is a KeyVault URL in the config file.
The configuration supports authentication to KeyVault using MSI (Managed Server Identity) or via a certificate installed in the user or machine store. In the second case, authentication is to the AzureID application identified by cliendId, and that application should have access to KeyVault.

Is there any interest to getting this PR? We've been using this functionality for 2 years, seems stable

.vsts-hosted.yml

Closing. There doesn't appear to be broad support for taking this change at this time. Thank you for your contribution.


Officially support Python 3.9, 3.10 on PyPI

PyPi only lists support of s3transfer to Python <= 3.8. I would like to see s3transfer officially supporting newer versions - Python 3.9, 3.10. And also, development status on PyPI is 3 - Alpha - can you increase it to 5 - Production/Stable maybe?
From my testing, s3transfer==0.5.0 works with Python 3.9, 3.10.

Hi @uri-rodberg, we have added updated version support to the setup.py for s3transfer. There just hasn't been a feature update to warrant a new upload to PyPI since then. The next release will update the public metadata.
In regards to Alpha vs Stable, s3transfer is still considered an implementation detail of Botocore. It's not intended or supported as a stand-alone product which is the reason for the current classification. If the library ever reaches a 1.0 release, it should have the development status updated.

Thank you.

@uri-rodberg @nateprewitt The s3transfer package may run on newer Pythons, but threading changed in Python 3.8, so if it is being used in a threaded application, something is broken at 0.4.0. (It bit me!) This ticket advises rolling the product back to 0.3.x (rolling back Boto3 to an older version effectively does just that) when those issues pop up.


can you have this code docker-image?

Hi, @traveller59 , I will have two questions:
1、this  torch2trt project could transfer the second model to tensorrt(especially pointpillars) successfully ?
2、if so, can you have this project docker images ?
Thanks.

This project can't convert pointpillars directly, we need to modify:

change RPN inputs and outputs from dict to tuple (torch.jit can't handle dict for now)
modify PFN:

class PFNLayerTensorRT(nn.Module):
    def __init__(self,
                 in_channels,
                 out_channels,
                 use_norm=True,
                 last_layer=False):
        super().__init__()
        self.name = 'PFNLayerTensorRT'
        self.last_vfe = last_layer
        assert self.last_vfe is True, "tensor rt don't support this."
        if not self.last_vfe:
            out_channels = out_channels // 2
        self.units = out_channels

        if use_norm:
            BatchNorm2d = change_default_args(
                eps=1e-3, momentum=0.01)(nn.BatchNorm2d)
            Conv2d = change_default_args(bias=False)(nn.Conv2d)
        else:
            BatchNorm2d = Empty
            Conv2d = change_default_args(bias=True)(nn.Conv2d)

        self.linear = Conv2d(in_channels, self.units, 1)
        self.norm = BatchNorm2d(self.units)

    def forward(self, inputs):
        """inputs: [1, num_features, max_num_points, max_points_per_voxel]
        [1, 64, 12000, 100]
        """
        x = self.linear(inputs)
        x = self.norm(x)
        x = F.relu(x)
        # x: [N, C, numPoints, numPointPerVoxel]
        # x = x.view(1, -1, 60)
        # max operation very slow in tensorrt consider using maxpool
        x_max = torch.max(x, dim=3, keepdim=True)[0]
        # x_max = F.max_pool2d(x, [1, x.shape[-1]], [1, x.shape[-1]])
        if self.last_vfe:
            return x_max
        else:
            # may need to use conv to implement repeat in tensorrt.
            x_repeat = x_max.repeat(1, inputs.shape[1], 1)
            x_concatenated = torch.cat([x, x_repeat], dim=2)
            return x_concatenated

@register_vfe
class PillarFeatureNetTensorRT(nn.Module):
    def __init__(self,
                 num_input_features=4,
                 use_norm=True,
                 num_filters=(64, ),
                 with_distance=False,
                 voxel_size=(0.2, 0.2, 4),
                 pc_range=(0, -40, -3, 70.4, 40, 1)):
        super().__init__()
        self.name = 'PillarFeatureNetTensorRT'
        assert len(num_filters) > 0
        num_input_features += 5
        if with_distance:
            num_input_features += 1
        self._with_distance = with_distance
        assert with_distance is False
        # Create PillarFeatureNetOld layers
        num_filters = [num_input_features] + list(num_filters)
        pfn_layers = []
        assert len(num_filters) == 2, "tensorrt don't support repeat"
        for i in range(len(num_filters) - 1):
            in_filters = num_filters[i]
            out_filters = num_filters[i + 1]
            if i < len(num_filters) - 2:
                last_layer = False
            else:
                last_layer = True
            pfn_layers.append(
                PFNLayerTensorRT(
                    in_filters, out_filters, use_norm, last_layer=last_layer))
        self.pfn_layers = nn.ModuleList(pfn_layers)

        # Need pillar (voxel) size and x/y offset in order to calculate pillar offset
        self.vx = voxel_size[0]
        self.vy = voxel_size[1]
        self.x_offset = self.vx / 2 + pc_range[0]
        self.y_offset = self.vy / 2 + pc_range[1]

    def forward(self, features, num_voxels, coors, voxel_point_mask):
        """TensorRT PFE must use specified inputs. All tensorrt inputs must be float for now.
        features: [1, num_point_features, max_num_points, max_points_per_voxel]
        num_voxels: [1, 1, max_num_points, 1]
        coors: [1, 4, max_num_points, 1]
        voxel_point_mask: [1, 1, max_num_points, max_points_per_voxel]
        """
        # features = features.permute(2, 0, 1).contiguous().unsqueeze(0)
        # num_voxels = num_voxels.type_as(features).view(1, 1, -1, 1)
        # coors = coors.type_as(features).t().contiguous().view(1, 4, -1, 1)
        # voxel_point_mask = voxel_point_mask.squeeze().unsqueeze(0).unsqueeze(0)

        device = features.device

        dtype = features.dtype
        # Find distance of x, y, and z from cluster center
        points_mean = features[:, :3].sum(
            dim=3, keepdim=True) / num_voxels.view(1, 1, -1, 1)
        f_cluster = features[:, :3] - points_mean

        # Find distance of x, y, and z from pillar center
        f_center_x = features[:, 0:1] - (coors[:, 3:4] * self.vx + self.x_offset)
        f_center_y = features[:, 1:2] - (coors[:, 2:3] * self.vy + self.y_offset)
        # print(f_cluster.squeeze().permute(1, 2, 0).mean())
        # Combine together feature decorations
        features_ls = [features, f_cluster, f_center_x, f_center_y]
        features = torch.cat(features_ls, dim=1)

        # The feature decorations were calculated without regard to whether pillar was empty. Need to ensure that
        # empty pillars remain set to zeros.
        features *= voxel_point_mask
        # Forward pass through PFNLayers
        for pfn in self.pfn_layers:
            features = pfn(features)
        # features shape: [1, 64, 12000, 1]
        return features # .t()

write a custom tensorrt plugin for PointPillarsScatter.

I can't provide docker image because the upload speed is very slow and unstable in my network environment.

@traveller59 thanks for your reply, I will try it.
best wishes.

@traveller59 Thanks for the PFN implementation. I've made the program running, but when I extracted the PFN, it looks like below:
middle_class_name PointPillarsScatter
num_trainable parameters: 69
PillarFeatureNet(
(pfn_layers): ModuleList(
(0): PFNLayer(
(conv1): Conv2d(9, 64, kernel_size=(1, 1), stride=(1, 1))
(norm): DefaultArgLayer(64, eps=0.001, momentum=0.01, affine=True, track_running_stats=True)
(conv2): Conv2d(64, 64, kernel_size=(1, 34), stride=(1, 1), dilation=(1, 3))
)
)
)
Which seems like before, still an empty Model.
Could you tell me where I goes wrong?
Many thanks


BarChart component

Implement the BarChart component, following the model of LineChart and similar props (yScale, xScale). Note that the chart below also includes a line chart, ignore that for the time being.
Figma

https://github.com/covid-projections/act-now-packages/pull/65


cigien: Watch filltofull.com

cigien requests the watch of the watch_keyword filltofull\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

filltofull\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

filltofull\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Mast in Charcoal HQ

Approved by Mast in Charcoal HQ


specify optional table property for use with lines

Description
Allows the user to specify a table for iptables_rule when using the lines property.  For example:
iptables_rule 'http_8080' do
  lines '-A PREROUTING -i eth0 -p tcp --dport 80 -j REDIRECT --to-port 8080'
  table :nat
end

Issues Resolved
https://github.com/chef-cookbooks/iptables/issues/75
Check List

[x] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[ ] New functionality includes testing.
[x] New functionality has been documented in the README if applicable
[x] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

Released in 4.1. Thanks


Update Person Class

closes #39

Closing PR as you opened another one (#46) on the same issue. Will follow that instead.


GitHub: support email when the email isn't public

Per @simonbyrne's comment https://github.com/coreos/dex/pull/972#issuecomment-320718897
https://developer.github.com/v3/users/emails/

Note that the downside is the plural: many users (including myself) have more than 1 email associated with their account.


Add Bedrock Support

What feature do you want to see added?
Now that GeyserMC supports the conversion of Resource packs, this request is to start the process of adding support for Bedrock players to join and play on the server.
This may not actually work, but this issue is to track any problems and progress

[ ] ItemsAdder starts generating font-images from unicode E000 which conflicts with the unicode icons used in the Bedrock version. Will need to update ALL the font-images manually to use a higher number as ItemsAdder does not support changing the starting unicode number. This can be done by adding the YAML item symbol: "\uE200".


Can I continue using web design standard package(JS&CSS) file after our team was separated from IBM then join "Kyndryl"?"

Summary
Relevant information
we are using IBM standard file(JS&CSS) in our project(v8 - v18) , and our team will be separated from IBM then join "Kyndryl",may I know how to deal with the IBM starndard file URL in our project ?Can we continue using IBM standard file(JS&CSS) after our team was migrated to "Kyndryl"?"
file & URL as below
https://w3.ibm.com/ui/v8/css/v4-screen.css
www.ibm.com/common/v14/screen.css
www.ibm.com/common/v16/css/screen-uas.css
www.ibm.com/i/v14/rules/grey_rule.gif
www.ibm.com/i/v16/t/ibm-logo.gif
www.ibm.com/common/js/ibmcommon.js
Name and organization
Boston
<EMAIL_ADDRESS>

Hi @bostonlf, There are a few conversations on the #carbon-for-ibm-dot com slack channel regarding your question here after this issue was opened. Do you still have questions about using the web design standard package?

@bostonlf I would recommend not using anything that is going to be sunset.  I'll close this issue.  Please use Slack #carbon-for-ibm-dotcom for further communications.


fix: resolved token matching order

Summary

Fixed the token matching order for FLOAT_NUMBER and NUMBER in the lexer.
Added test cases to ensure FLOAT_NUMBER is correctly prioritized over NUMBER.

Issue Reference
Fixes #225
Changes Made

Modified the order of token definitions in TOKENS to prioritize FLOAT_NUMBER.
Updated test_lexer.py to include edge cases for floating-point numbers.

Verification Steps

Ran all test cases, including edge cases for number tokenization.
Verified that FLOAT_NUMBER is matched before NUMBER in all scenarios.

Checklist

[x] Code changes reviewed and tested.
[x] All existing tests pass.
[x] Added new test cases for the fix.

Could you please resolve the merge conflicts


Use either pytest or unittest, not both

Currently everything in algorithm/ uses pytest for testing while everything in fqe_ops/ uses unittest. Is there a reason for using both? AFAIK it's standard to only use one.

@rmlarose @ncrubin just a heads-up: On the basis of the above conversation, we are migrating all of the tests to pytest. This is also useful in testing both python and c code path in the C-accelerated FQE. The migration will be included in the delivery of the C-accelerated FQE scheduled at the end of this month.


Support negative latitudes to PV panel orientation

Fixes #103 .
Untested yet. Await response by @WeselyDAN .

I tested it and compared it to renewable ninjas. Everything thing works as expected

I tested it and compared it to renewable ninjas. Everything thing works as expected


Unable to do patch request from curl

Unable to do patch request from curl. It works fine with UI only troubling during Curl or postman request.
@baijum , can you please look into it.

PATCH should work if you pass all necessary arguments.  Here is an example, using curl.
If the developer mode is enabled, you can generate a set of tokens like this:
curl http://localhost:8080/api/login/generate

set the token to an environment variable for convenience, say TKN.
Here is an example PATCH call.
First call the URL and set the version to an environment variable for convenience, say VRSN
curl http://localhost:8080/api/workitems/2578

Now you can run the curl like this:
curl http://localhost:8080/api/workitems/2578 -X PATCH  -H "Authorization: Bearer $TKN" -d '{"data":{"attributes":{"system.description":"sdfa", "version": $VRSN}, "id": "2578", "type":"workitems"}}'

In the above example, we are trying to update the description.
If you want to run it again, get the VRSN value through GET.

@baijum , Thanks that helps !!


feat: silent mode

Useful for mass scanning. Resolves issue #8

Sorry, I had a pull request on hold that change the whole structure and my fiver was so high the whole afternoon that I couldn't merge it. Now your changes cannot easily be applied anymore.

I can merge the changes and will recommit. I only have to change a couple spots anyways


Error When loading the application

Hi Team,
Iam getting the below error when loading the angular application.

The error it seems in system.js.config file. Iam following angular.io with latest 2 release.Please assist.
Below is the system.js.config file
/**

System configuration for Angular samples

Adjust as necessary for your application needs.
*/
(function (global) {
System.config({
paths: {
// paths serve as alias
'npm:': 'node_modules/'
},
// map tells the System loader where to look for things
map: {
// our app is within the app folder
app: 'app',
// angular bundles
'@angular/core': <EMAIL_ADDRESS>'@angular/common': <EMAIL_ADDRESS>'@angular/compiler': <EMAIL_ADDRESS>'@angular/platform-browser': <EMAIL_ADDRESS>'@angular/platform-browser-dynamic': <EMAIL_ADDRESS>'@angular/http': <EMAIL_ADDRESS>'@angular/router': <EMAIL_ADDRESS>'@angular/forms': <EMAIL_ADDRESS>'@angular/upgrade': <EMAIL_ADDRESS>
// other libraries
'rxjs':                      'npm:rxjs',
'angular-in-memory-web-api': 'npm:angular-in-memory-web-api',

},
// packages tells the System loader how to load when no filename and/or no extension
packages: {
app: {
main: './main.js',
defaultExtension: 'js'
},
rxjs: {
defaultExtension: 'js'
},
'angular-in-memory-web-api': {
main: './index.js',
defaultExtension: 'js'
}
}
});
})(this);

Are you doing more than simply running the QS app? The QS app doesn't use the router.
In my experience the @angular/router package is in node_modules.
Ah ... I suspect that you are unaware that ES import names are case-sensitive. The package is @angular/router, not @angular/Router.
This is an ES5 thing, not an Angular thing.

I just downloaded toh-6 , angular quick start .
Then i ran "npm install" and then npm start.
http://localhost:3000 seems to be working fine on chrome and firefox.
Its not loading on safari.
Please fix this issue.


log4net in MAUI application

Hi all,
I'm migrating from xamarin form to MAUI (ios and android) and one library has log4net as dependency.
I have the exception below.
How can I solve it?
thanks a lot
[DOTNET] log4net:ERROR Exception while reading ConfigurationSettings. Check your .config file is well formed XML.
[0:] log4net:ERROR Exception while reading ConfigurationSettings. Check your .config file is well formed XML.
[DOTNET] System.Configuration.ConfigurationErrorsException: Configuration system failed to initialize
[DOTNET]  ---> System.PlatformNotSupportedException: Operation is not supported on this platform.
[DOTNET]    at System.Configuration.ClientConfigPaths..ctor(String exePath, Boolean includeUserConfig)
[DOTNET]    at System.Configuration.ClientConfigPaths.GetPaths(String exePath, Boolean includeUserConfig)
[DOTNET]    at System.Configuration.ClientConfigurationHost.get_ConfigPaths()
[DOTNET]    at System.Configuration.ClientConfigurationHost.GetStreamName(String configPath)
[DOTNET]    at System.Configuration.ClientConfigurationHost.get_IsAppConfigHttp()
[DOTNET]    at System.Configuration.Internal.DelegatingConfigHost.get_IsAppConfigHttp()
[DOTNET]    at System.Configuration.ClientConfigurationSystem..ctor()
[DOTNET]    at System.Configuration.ConfigurationManager.EnsureConfigurationSystem()
[DOTNET]    --- End of inner exception stack trace ---
[DOTNET]    at System.Configuration.ConfigurationManager.PrepareConfigSystem()
[DOTNET]    at System.Configuration.ConfigurationManager.GetSection(String sectionName)
[DOTNET]    at System.Configuration.ConfigurationManager.get_AppSettings()
[DOTNET]    at log4net.Util.SystemInfo.GetAppSetting(String key)
[0:] System.Configuration.ConfigurationErrorsException: Configuration system failed to initialize
---> System.PlatformNotSupportedException: Operation is not supported on this platform.
at System.Configuration.ClientConfigPaths..ctor(String exePath, Boolean includeUserConfig)
at System.Configuration.ClientConfigPaths.GetPaths(String exePath, Boolean includeUserConfig)
at System.Configuration.ClientConfigurationHost.get_ConfigPaths()
at System.Configuration.ClientConfigurationHost.GetStreamName(String configPath)
at System.Configuration.ClientConfigurationHost.get_IsAppConfigHttp()
at System.Configuration.Internal.DelegatingConfigHost.get_IsAppConfigHttp()
at System.Configuration.ClientConfigurationSystem..ctor()
at System.Configuration.ConfigurationManager.EnsureConfigurationSystem()
--- End of inner exception stack trace ---
at System.Configuration.ConfigurationManager.PrepareConfigSystem()
at System.Configuration.ConfigurationManager.GetSection(String sectionName)
at System.Configuration.ConfigurationManager.get_AppSettings()
at log4net.Util.SystemInfo.GetAppSetting(String key)

@TheSundayDev do you have a config file?
If yes, can you post the content?
If not, can you create one?
Do you have a reference on System.Configuration?

Thanks for your quick reply!
Reference to System.Configuration.ConfigurationManager added.
I created a app.config file in project root and mark as Embedded resource.
For sure I'm doing something wrong.
Content is
<?xml version="1.0" encoding="utf-8" ?> <configSections> 	<section name="log4net" type="log4net.Config.Log4NetConfigurationSectionHandler, log4net" /> </configSections> <log4net> 	<appender name="ConsoleAppender" type="log4net.Appender.ConsoleAppender" > 		<layout type="log4net.Layout.PatternLayout"> 			<conversionPattern value="%date| %level | %message%newline" /> 		</layout> 	</appender> 	<root> 		<level value="INFO" /> 		<appender-ref ref="ConsoleAppender" /> 	</root> </log4net>
thanks

@TheSundayDev embedded ressource is wrong. The config file must be copied as is to the target directory (the default build action for config files)
Is the exception still the same?
Can you post your project file?

@TheSundayDev is this issue still relevant?
Otherwise I will close this next week.

@TheSundayDev feel free to reopen when you have more details.


chore(main): release 0.12.0

:robot: I have created a release beep boop
0.12.0 (2024-06-24)
✨ New Features

added inkwells to checkbox and radio button (a8b31d3)
create slider component (#53) (6394ef6)
Created dropdown list item (#101) (868b26c)
Created list item variants (#98) (a8b31d3)
Created ZetaAnimationDuration tokens (868b26c)
created ZetaList to add borders to list items (a8b31d3)

🪲 Bug Fixes

minor bugs on ZetaChatItem (#105) (c37e51d)
removed hover color on disabled radio buttons (a8b31d3)
UX-1078: fixed button group disabled color. (88d82d8)
UX-1080: Add snackbar action bold text (88d82d8)
UX-1081: Change snackbar action color (88d82d8)
UX-977: change indicator border color (88d82d8)

📈 Documentation

Added guides for golden and unit testing (4b41f73)
Fix documentation macros (c37e51d)

⛓ Dependencies

Update zeta-icon library (#107) (ccaf8a9)

🧪 Tests

100% test coverage for theme, utils, Zeta and ZetaProvider (4b41f73)
Added ZetaTooltip test (4b41f73)
Organise tests folder, and increase code coverage and add goldens for existing tests (0e2d8be)

🧹 Miscellaneous Chores

organise existing tests (#108) (0e2d8be)
rename debounce file (c37e51d)
Update widgetbook to pull readme from github (c37e51d)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.12.0
:sunflower:


frr-k8s: apply affinity/resources/classes as the speaker

Aligning with what we already do for the speaker's nodeSelector/tolerations

Is this a BUG FIX or a FEATURE ?:
/kind bug
What this PR does / why we need it:
Special notes for your reviewer:
Release note:

Apply affinity/resources/classes to the frr-k8s daemon according to the SpeakerConfig, as we already follow the speaker for nodeSelector and tolerations.

LGTM


To read .time and .data logs

What do you guys use to read logs properly? How can I combine both of this to check on logs properly?

Normally a cat on the .data should be enouth, or a less -R
You can also replay all logs with scriptreplay


plot_compare_evokeds creates wrong plot (should be average, is GFP?)

I'm currently plotting the ERPs for some conditions using plot_compare_evokeds , however I stumbled across some inconsistent behaviour:
If given ERPs with several channels, the function seems to plot only the first channel of the original data, not the average of all supplied ERP channels or choices. Here is some sample code
Describe the bug
In this example, I simulate 13 participants with 128 trials, where for each trial channels 0-5 have a positive waveform and 5-64 have a negative waveform. This means the grand average is looking like this:

If I now create ERPs on selected channels I would expect the evoked plot to be either positive or negative in shape. Selecting occipital electrodes should lead to a plot that shows the average of the selected electrodes for the epochs. However, this does not seem the case.
Actual results
Now when I use plot_compare_evokeds and limiting the data to occipital channel (i.e. negative waveforms) using
mne.viz.plot_compare_evokeds({'condition1': erps}, picks=occipitals, show_sensors=True)

the resulting plot looks like this:

The plot indicates that occipital eletrodes have been selected and are displayed. However, what is displayed are the GFPs?
When narrowing down the electrode selection to a single electrode (e.g. O2), the waveform get's displayed correctly. However, as soon as several electrodes are selected, the first channel of the raw is displayed.
mne.viz.plot_compare_evokeds({'condition1': erps}, picks='O2', show_sensors=True)

Here is some code to reproduce
import mne
import numpy as np
import matplotlib.pyplot as plt

# simulate EEG data
n_subj = 13
n_channels = 64
n_trials = 128
sampling_freq = 100  # in Hertz
montage = mne.channels.make_standard_montage('biosemi64')

# two different waveforms: positive and negative wave
times = np.linspace(0, 1, sampling_freq, endpoint=False)
wave1 = np.sin(np.pi * times)
wave2 = np.sin(np.pi * times)*-1

all_epochs = []
for subj in  range(n_subj):
    trials = []

    # construct n_trials
    for _ in range(n_trials):
        # channels 0 to 5 have a positive waveform ('Fp1', 'AF7','AF3', 'F1')
        # channels 5 to 64 have a negative waveform
        trial = np.array([(wave1 if x<5 else wave2)*(x/100+1) for x in range(n_channels)])
        trials.append(trial)

    data = np.array(trials)
    info = mne.create_info(montage.ch_names, sfreq=sampling_freq, ch_types='eeg')
    epochs = mne.EpochsArray(data, info)
    epochs.set_montage(montage)

    # add to a list with epochs for this simulated participant
    all_epochs.append(epochs)

#%% plotting
# form ERPs for all epochs
occipitals = [ch for ch in montage.ch_names if 'O' in ch]

erps = [epoch.average(occipitals) for epoch in all_epochs]

# for sanity, plot the ERP individual average ERPs
ax = plt.gca()
for erp in erps:
    # they are all negative
    erp.plot(axes=ax)

# expectation: this plots a average ERP of the selected electrode, i.e. a negative waveform
# however, it plots a positive waveform, namely the waveform of the first channel
mne.viz.plot_compare_evokeds({'condition1': erps}, picks=occipitals, show_sensors=True)

# even when manually choosing "picks=occipitals", the other channels are plotted
mne.viz.plot_compare_evokeds({'condition1': erps}, picks=occipitals, show_sensors=True)

# only when a single channel is plotted, the waveform is correctly displayed
mne.viz.plot_compare_evokeds({'condition1': erps}, picks='O2', show_sensors=True)

Okay, I think you can disre


Transparent faking

Hi,
I could make it work with fake RPi.GPIO and smbus in my application.
But I can't change all the imports (from from RPi.GPIO import GPIO to from fake_rpi import GPIO) in my project and all dependencies.
So I found a solution inspired by this stackoverflow answer something that replace the modules by this fake one:
import sys
import fake_rpi

sys.modules['RPi'] = fake_rpi
sys.modules['smbus'] = fake_rpi

# Then everywhere...
import RPi.GPIO as GPIO
import smbus

Then, I don't need to change my project and dependencies imports, I just have to replace modules once in my main file.
It could work if inside the folder fake_rpi, there were another modules that reflect the RPi, smbus... modules
Do you think this way of using fake_rpi is relevent ?

I illustrated in that PR https://github.com/MomsFriendlyRobotCompany/fake_rpi/pull/2 what I mean by a working example with the fake smbus.
I wanted to do the same with RPi.GPIO, but the import is different and I couldn't understand how to reflect the import style.

interesting ... I will take a closer look later at how you did this.

Hi, I also could reproduce RPi.GPIO import, so I made it work also for GPIO in the same PR.
I updated the example: https://github.com/MomsFriendlyRobotCompany/fake_rpi/pull/2/files#diff-3b5e31446f546cea5285cfd44d32d046R6

accepted ... thanks!

Thank you. I'll see if I can also update Readme


Notifications privacy issue

It looks like privacy isn't checked before sending notifications.
How to reproduce:
user A is subscribed to user B, user B makes a post with "me only" privacy and post comment there, user A receives PUSH notification about this action and upon clicking on the notification "Access denied. This is private content." appears. Actually notification shouldn't be sent if user has no rights to view it.
Should be done:

Reorganize Alert registration in BxNtfsResponse::response method. Email and Push notifications shouldn't be processed immediately when Add Alert was received. New cron task should be added. Last processed notification should be saved.
When notification is preparing to send, the list of recipients should be collected first and then a notification should be check in accordance with each recipient. In this case Privacy can be checked correctly.
Before sending a Email\Push notification it should be checked in bx_notifications_events2users table. If user already viewed the notification on Site, then Email/Push notifications aren't needed.

Note. During the next update the newly added 'bx_notifications_processed_event' setting should be initialized with the last Event ID to avoid repeated notifications for all events in the 'bx_notifications_events' table.


Check if witnesses are known

This PR is an attempt to fix the error reported on issue https://github.com/WebOfTrust/keria/issues/54 by checking if the url is in the hab or if the obbi was already resolved for new witnesses introduced in a icp or a rot event.
@pfeairheller , if you thinks that this is a correct approach, I will add and fix some tests.

replaced by PR #92


v14.3.0 optimise Player Entity

[Current coverage][cc-pull] is 99.20%

Merging [#304][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master       #304   diff @@
==========================================
  Files            24         24
  Lines           374        374
  Methods          77         77
  Messages         77        154    +77
  Branches          0          0
==========================================
  Hits            371        371
  Misses            3          3
  Partials          0          0

Powered by Codecov. Last updated by b8cba51
[cc-base-branch]: https://codecov.io/gh/eugene-matvejev/battleship-game-api/branch/master?src=pr
[cc-pull]: https://codecov.io/gh/eugene-matvejev/battleship-game-api/pull/304?src=pr

codecov != failed tests


Cope with .globl lines ending with comments

When compiling with clang-5.0 -S, my assembly outputs look like this:
	.globl	kernel                # -- Begin function kernel

This confused c2goasm. I fixed it in this PR by using a regular expression to extract the symbol.

Done.


Annoying Casting Issue with unknown reason

For an unknown reason mapping of lrmiData results in two different data structures. We either get:
"creator": [
    {
      "@id": "https://orcid.org/0000-XXXX-XXXX-XXXX",
      "academicDegree": "Prof.",
      "affiliation": {
        "@id": "https://ror.org/024z2rq82",
        "prefLabel": "Heinrich-Heine-Universität Düsseldorf",
        "type": "Organization"
      },
      "prefLabel": "Tycho Typo"
    },

Affiliation in this case is of type Json Object
Or we get:
  "creator": [
    {
      "@id": "https://api.hoerkaen.hbz-nrw.de/adhoc/uri/RmlvbmEgRmllc2Vsc2Nod2VpZg==",
      "academicDegree": "https://d-nb.info/standards/elementset/gnd#academicDegree/Prof.",
      "affiliation": [
        {
          "@id": "https://ror.org/027b9qx26",
          "prefLabel": "Hochschule Niederrhein",
          "rdftype": [
            {
              "@id": "http://schema.org/Organization",
              "prefLabel": "http://schema.org/Organization"
            }
          ]
        }
      ],
      "prefLabel": "Fiona Fieselschweif"
    },

Where Affiliation is of type Json Array.  I can find a reason for this behavior.

[ ] From https://dini-ag-kim.github.io/amb/draft/schemas/creator.json the affiliation should be a Json Object. Is that correct @TobiasNx ?
[ ] My assumption is that the reason for the difference in the Object Type mapping may lies within etikett data. Is that correct @jschnasse  ?

, nor can say which is the correct type we have to implement.

It should be an object, see:
https://dini-ag-kim.github.io/amb/draft/schemas/affiliation.json

btw. it shouldnt be academicDegree but honorificPrefix (or honoricPrefix which is the faulty info from the upload formular)

Evtl. hängen die Unterschiee auch damit zusammen, ob das Mapping erstmalig stattfindet (Senden übe das List-Formular inkl. Mapping) oder durch erneutes Editieren zustande kommt (ORCA-Formular).

honoricPrefix heißt es in LRMI, academicDegree in RDF. Das Beispiel ist RDF (...aber darüber bin ich auch erst gestolpert).

deswegen heißt es auch prefLabel (RDF) und nicht name (LRMI).

btw. it shouldnt be academicDegree but honorificPrefix (or honoricPrefix which is the faulty info from the upload formular)

My examples are from lobid data not from lrmiData. Therefore academicDegree is correct ;-)

btw. I now prevent code from runtime errors by cast HashSet into ArrayList if requiered. BUT this means I also loose all information of sequence that is normally part of ArrayList. Therefore I'm on the way to provide a manual  circumvent for this situation

Inspecting the code and check for the issue I learned it's not a bug. It's a missconception in the way we use JSONObjects. Therefore I close this issue.


Decisions on LinkML and documentation 2022-09-20

After much discussion, we will:
Keep the name, title, and ID fields.
linkml name = structured comment name
linkml title = item
linkml ID = term IRI (but can display in the form of a CURIE like MIXS:0001234

Changes to documentation display pages that were dicussed:
[] on xx, checklists before packages
TODO: finish this

names, titles and IDs

The LinkML derived from schemsheets that use the MIxS terms verbatim as names fail when used in various data-conversion, schema-conversion and data-validation steps. I have retained the MIxS names as annotations but  have applied systematic names for all terms/slots (snake_case) and all checklist/package classes (PascalCase)

checklists/packages

Checklists should be listed before packages on the index page
names and IDs of combination classes should follow the pattern {checklist_portion)_{package_portion}


Error when accessing Dark Fog Settings on New Game creation

I hope this error message is helpful for you to help debug! Thank you for your efforts on updating this mod! I cannot play DSP without your mod anymore lol :)

Ah, I forgot to mention that I am using your mod version 2.12.0


Create mdl-form-field helper component

With both checkboxes and radios we've seen cases where we need to accurately align a label with its corresponding input. The mdl-checkbox-wrapper css class does this within checkbox, and the mdl-form-field css class within the MDL radio demo (TK) does this as well. Because this is such a common case, and the UI appears in many places in the spec, we should refactor it out into its own component.

Closed by #4815


Where is the google analytics code?

https://github.com/google/google-api-dotnet-client is the core library, the .dlls for each of the different APIs are built automatically from the discovery API so there isn't a project for them on GitHub.
The package source code is available at:
https://developers.google.com/resources/api-libraries/download/analytics/v3/csharp


🐛 BUG: [b] open a browser hotkey takes you to wrong host in remote mode + Access

Which Cloudflare product(s) does this pertain to?
Workers/Other
What version of Wrangler are you using?
2.8.1
What operating system are you using?
macOS
Describe the Bug
When running wrangler dev in remote mode for a Worker which runs behind Access, once the dev session is ready, pressing b takes you to subdomain.example.com:8787 rather than localhost:8787 or whatever host we're using these days (<IP_ADDRESS>/<IP_ADDRESS>/etc.)

@GregBrimble It looks like this was a bug introduced in https://github.com/cloudflare/wrangler2/pull/1883, which applies for all remote mode dev, rather than just for workers behind Access. I'm assigning to you since you worked on that PR and probably have more context for what's going on there

Transferring over from #2691 - myself & @threepointone have run into this bug today.

I assume it's broken for everyone with routes btw, I wasn't even using it as a custom domain, just a plain route. This sounds like it should be higher priority.

Can confirm - I was originally under the impression this was for custom domains only but having tested with a route, that is also being opened by the open a browser hotkey.

This is also an issue when using --port 0 - with no routes or custom domains - and that will open localhost:0 in the browser.


FIPS: document support and limitations

What this PR does / why we need it:
This PR documents support for FIPS nodes.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

/test pull-kubevirt-e2e-k8s-1.25-fips-sig-compute

/test pull-kubevirt-e2e-k8s-1.25-fips-sig-compute

/test pull-kubevirt-e2e-k8s-1.25-fips-sig-compute


DeprecationWarnings-Langflow-v1.0.15

Bug Description
Warnings messages after installing Langflow, whether you do it through pip or from the main source.
Starting Langflow v1.0.15...
/repo/langflow/.venv/lib/python3.12/site-packages/pydantic/_internal/_config.py:291: PydanticDeprecatedSince20: Support for class-based config is deprecated, use ConfigDict instead. Deprecated in Pydantic V2.0 to be removed in V3.0. See Pydantic V2 Migration Guide at https://errors.pydantic.dev/2.8/migration/
warnings.warn(DEPRECATION_MESSAGE, DeprecationWarning)
repo/langflow/.venv/lib/python3.12/site-packages/litellm/utils.py:113: DeprecationWarning: open_text is deprecated. Use files() instead. Refer to https://importlib-resources.readthedocs.io/en/latest/using.html#migrating-from-legacy for migration advice.
with resources.open_text("litellm.llms.tokenizers", "anthropic_tokenizer.json") as f:
Reproduction
Install Langflow via pip or main and you'll see the warnings.
Expected behavior
It shouldn't have any warnings expected.
Who can help?
No response
Operating System
Mac Sonoma Version 14.6.1
Langflow Version
1.0.15
Python Version
3.12
Screenshot
No response
Flow File
No response

#3416


Bug Report: Launcher Glitch When Exiting WebUI on Cardputer

First of all, let me say what an amazing project the Cardputer and this Launcher is! I've just started tinkering with it, and I'm excited about building from source and installing new things based on the ESP32 platform.
However, I found a small but impactful bug in the Launcher that might affect other users trying out different projects. Here’s what I encountered:
Issue Description:
When I flash the M5Launcher and then install projects like 'Marauder for Cardputer', 'Palnagotchi', or 'MicroHydra for Cardputer' (as far as I tested, this might be happening with all other projects you can install on the Cardputer), and then get back into the M5Launcher with an image flashed pressing 'ok' when the letters show, I experience the following issue:
Upon starting the Launcher's WebUI and then exiting it by pressing 'ok', the Launcher glitches for a few seconds.
After the glitch, it unexpectedly starts the previously flashed project (e.g., Marauder) instead of staying inside the M5Launcher.

Expected Behavior:
I believe it should stay inside the M5Launcher after exiting the WebUI, effectively performing a 'RESET' to the main screen of the Launcher, ensuring a smooth return to the Launcher interface. This would allow users to either enter another .bin file or manually reset to load the previously flashed image if they want to.
This issue can be quite disruptive, especially for users frequently switching between different projects. Resolving it would significantly enhance the robustness of the Launcher and improve the overall user experience.
Thank you for your hard dedication to this project. I'm looking forward to future updates or any improvements in the building-from-source process!
Best regards
PS: Just checked with M5StickC PLUS2 and this issue also happens on that device

It's really unexpected..
I tried here and I see that this is a problem with watchdogs being triggered.. I'll let it deactivated in the next update
Thank you for reporting!!

I changed the way  the list of files is sent to the browser, fortunetly now we will be able to heve more files on screen


RNC-2 && RNC-7:: feat: setup axios && fetch API from heroku

Fetch API contract from
https://contact.herokuapp.com/contact

add axios: 1.6.8
add react-native-toast-message: 2.2.0
add axiosInstance
add error msg when timeout
fetch temp to index.tsx

This PR includes RNC-2: as a FE developer, I want to setup axios & RNC-7: as a FE developer, I want to fetch API from heroku
Note:
Reminder to make different branch for setup and implementation


My Bins not loading

File dropdown My Bins link not working.
cmd + o not working.
Chrome 51.0.2704.103
Firefox Dev Edition 49.0

Fixed by the man himself. Cheers!


fix: show 'no preview available' for files correctly

The games code removed a line that correctly showed a message saying "No preview available" for certain files.

Manually tested with @tzarebczan
@skhameneh Let me know if there are issues with this logic, but it seems to work.


*_visible method doesn't work as documented

I'm following these instructions: https://github.com/natritmeyer/site_prism#methods-supporting-capybara-options .
According to this, I should be able to wait for a text to appear,
@create_job_page.wait_until_items_modal_visible text: 'Job name'
but it throws this error:
NoMethodError:
       undefined method `zero?' for {:text=>"Job name"}:Hash
     # /Users/rra/.asdf/installs/ruby/2.3.4/lib/ruby/2.3.0/timeout.rb:74:in `timeout'

So, I changed the method call, passing the timeout as the first argument.
@create_job_page.wait_until_items_modal_visible(3, text: 'Job name')
and this generates this warning as SitePrism is passing down an array to Capybara:
Unused parameters passed to Capybara::Queries::SelectorQuery : [{:text=>"Job Name"}]
Looks like the documentation is outdated, correct?
And even passing the timeout as a parameter the method doesn't work. Is this a bug or there is another way to do that?

Can you provide us an SSCCE? http://www.sscce.org
Is the name of your element defined as items_modal - And how is it defined? section / element e.t.c.
Do you know if the item "is" visible / invisible at the point you run the method?
In the docs we don't specify that you can pass a timeout and a query parameter into the wait_until method, But I imagine it would work in the same way.
I know we have a couple of PR's to standardise this area (And add more feature tests, as this is an area that is a bit light).
But yeh some food for thought for now...

please check out #246


Fix btcrpcclient to correctly disable raw transaction lock time.

btcrpcclient currently doesn't build.

Hi @c00w -
I believe #67 is the correct fix for this.

As @dajohi said, #67 is the correct fix.  There was a new parameter added to the searchrawtransactions command which the caller can specify.
Please try again now that #67 has been merged.

I'm going to close this since it was fixed by #67.


Lavalink isnt working

Lavalink node: 'narco.buses.rocks', has error: 'Unexpected op "ready" with data: [object Object]'.
Lavalink isnt working am did all like in the setup guide

Yes it is an error, but could you try playing some music. As I have noticed with the newer lavalink server it does display this error but still continues to play and use effects

now i get this error
Cannot read properties of undefined (reading 'conf')
at SlashCreate.run (/home/container/src/events/custom/slashCreate.js:35:28)
at Egglord. (/home/container/src/bot.js:71:38)
at Egglord.emit (node:events:513:28)
at InteractionCreate.run (/home/container/src/events/bot/interactionCreate.js:34:43)
at Egglord. (/home/container/src/bot.js:71:38)
at Egglord.emit (node:events:513:28)
at InteractionCreateAction.handle (/home/container/node_modules/discord.js/src/client/actions/InteractionCreate.js:97:12)
at module.exports [as INTERACTION_CREATE] (/home/container/node_modules/discord.js/src/client/websocket/handlers/INTERACTION_CREATE.js:4:36)
at WebSocketManager.handlePacket (/home/container/node_modules/discord.js/src/client/websocket/WebSocketManager.js:352:31)
at WebSocketShard.onPacket (/home/container/node_modules/discord.js/src/client/websocket/WebSocketShard.js:494:22)
and i turned on PRESENCE INTENT, SERVER MEMBERS INTENT and MESSAGE CONTENT INTENT

What command did u run, cuz I don't see how that's possible?

i tried the meme command but it doenst show in the slash command menu
because the bot didnt created them

yes as you need to go on the server that u assigned as the SupportServer in the config and run /reload misc feature:interactions

ahhh okay good to know thanks


Mono: Force preemptive thread suspend mode as a temporary workaround

See #29812
This is only meant as a temporary workaround until we find an actual fix.

Thanks!

Cherry-picked for 3.1.2.


DOC: update link to glasser supplementary info; convert to footbib

fixes a 404 link, and converts the docstring to use footcite while we're at it.

CircleCI is broken, @GuillaumeFavelier can you look? It has to do with pyvistaqt. Locally when I switched to using pyvista master and pyvistaqt there were some tests that were not happy...
In the meantime, let's pin pyvista<0.25 (this PR is fine) to avoid this problem @drammock

hmm... a bunch of failures in test_brain and tst_3d in the azure 3.7 conda build, see here.
On the other hand, the docstring changes look good

I see that pyvista has been pinned to 0.24 in requiremenrs.txt but not in environment.yml. That could explain the failures on conda


Error loading library: indy_vdr specified version because it occurs when installing as pip3 install indy-vdr.

An error occurs while installing indy_vdr when running a demo.
Verified normal operation if the version is specified.
This request specifies the version of indy_vdr

@mingi3442 — could you please apply DCO (DCO - Developer Certificate of Origin - https://github.com/apps/dco) as per the “Details” on the failed check?  Sorry for the hassle, but that is required.

I'm sorry. I'll apply DCO again and commit to request it


hash of chunk for integrity check on server

It would be handy if chunks were sent with an integrity hash that could be compared server side for data integrity

I found https://github.com/23/resumable.js/issues/135
I haven't put together the workaround listed in the ticket but it seems like the method presented doesn't block when the chunk hash isn't computed does it?
Is there an interface available to block the upload until the hash is computed for a particular chunk?

I also came across https://github.com/flowjs/flow.js from reading the comments in #135 and I guess that is why they added the preprocess callback function.


Add --mini flag option to countries

desired command: openaq countries list --mini

What would the "mini" output look like as compared to the full output?

I wouldn't it for countries get as well like command openaq countries get 13 --mini but that is up for debate

right. the current columns are:
 COUNTRIES_ID | ISO_CODE | NAME   | DATETIME_FIRST       | DATETIME_LAST        | PARAMETERS | LOCATIONS_COUNT | MEASUREMENTS_COUNT | PROVIDERS_COUNT

what are the columns you are proposing to be included in the --mini version?

ah, perhaps COUNTRIES_ID ISO_CODE NAME & PARAMETERS


refactor(vacuum): improve vacuum commands support

[x] use ha data helpers
[x] convert boolean options to command list for editor
[ ] update documentation

@acesyde I simplified the commands usage in the editors. I grouped them in a single control for better maintenance (like you did before 😅)
Yaml config
type: custom:mushroom-vacuum-card
entity: vacuum.0_ground_floor
commands:
  - start_pause
  - stop
  - return_home
  - locate

Editor

Nice idea 👍


Adds support for importing individual models

Codecov Report

Merging #1893 (b7bd10e) into master (bb5073f) will decrease coverage by 2.08%.
The diff coverage is 66.71%.

@@             Coverage Diff              @@
##             master    #1893      +/-   ##
============================================
- Coverage     72.08%   70.00%   -2.09%
- Complexity     5126     5585     +459
============================================
  Files           473      530      +57
  Lines         21970    24629    +2659
  Branches       2351     2672     +321
============================================
+ Hits          15838    17241    +1403
- Misses         4925     6077    +1152
- Partials       1207     1311     +104

Impacted Files
Coverage Δ

api/src/main/java/ai/djl/modality/cv/Image.java
69.23% <ø> (-4.11%)
:arrow_down:

...rc/main/java/ai/djl/modality/cv/MultiBoxPrior.java
76.00% <ø> (ø)

...rc/main/java/ai/djl/modality/cv/output/Joints.java
71.42% <ø> (ø)

.../main/java/ai/djl/modality/cv/output/Landmark.java
100.00% <ø> (ø)

...main/java/ai/djl/modality/cv/output/Rectangle.java
72.41% <0.00%> (ø)

...i/djl/modality/cv/translator/BigGANTranslator.java
21.42% <0.00%> (-5.24%)
:arrow_down:

...odality/cv/translator/BigGANTranslatorFactory.java
33.33% <0.00%> (+8.33%)
:arrow_up:

...nslator/InstanceSegmentationTranslatorFactory.java
14.28% <0.00%> (-3.90%)
:arrow_down:

.../cv/translator/SemanticSegmentationTranslator.java
0.00% <0.00%> (ø)

.../cv/translator/StyleTransferTranslatorFactory.java
40.00% <ø> (ø)

... and 428 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Added warning message while using the commands with ADO server

Please make sure the code is following contribution guidelines in CONTRIBUTING.md

[ ] : This PR has a corresponding issue open in the Repository.
[ ] : Approach is signed off on the issue.

Fixes: #1151
To show the warning message code resolves the organization URL.
the resolved URL is then compared with the know patterns.
https://dev.azure.com/*
*.visualstudio.com
If URL does not matches with know patterns the warning message is shown

will be good to have a better PR title and some more details around the logic in description

will be good to add test cases

can you share the experience how does this looks to the customer both

against hosted
against onprem

we need review from @aaronhallberg on the messaging

I will suggest to not say that the support is in preview as we don't plan to make this support GA

Working on test cases and changing the message

Done with test cases and message changes
Message:
Support for Azure DevOps Server, is under development
Test cases
    def test_check_organization_with_dev_url(self):
        showWarning = check_organization(self._TEST_DEVOPS_ORGANIZATION)
        self.assertEqual(False, showWarning)

    def test_check_organization_with_visual_studio_url(self):
        showWarning = check_organization(self._TEST_VISUAL_STUDIO_ORGANIZATION)
        self.assertEqual(False, showWarning)

    def test_check_organization_with_ado_server_url(self):
        showWarning = check_organization(self._TEST_ADO_SERVER_ORGANIZATION)
        self.assertEqual(True, showWarning)


docs: remove misplaced sentence from Quick Installation guide

Hi there 👋 , while following the Cilium installation guide I noticed that in the Install Cilium step, there seems to be a sentence that belongs to the previous step. This PR removes that sentence to avoid confusion. No need to move the sentence to the previous step as it is already clear.
Fixes: #15970
Signed-off-by: Lorenzo Fundaró<EMAIL_ADDRESS>

Good catch, thank you for your contribution!

happy to help ! 😄


Visible Rule doesn't work with forward slash

Please check our current Issues to see if someone already reported this https://github.com/Microsoft/azure-pipelines-task-lib/issues
Environment
azure-pipelines-task-lib version: latest
Issue Description
I have an input whose value is "test/value"
Eg:
{
"name":  "abc",
"type":  "radio",
"label":  "Abc",
"defaultValue":  "test1/value",
"required":  true,
"options":  {
"test1/value":  "Test 1",
"test2/value":  "Test 2"
}
},
"name":  "Abc2",
"type":  "pickList",
"label":  "Abc2",
"defaultValue":  "test1/value",
"required":  true,
"visibleRule":  "abc = test1/value",
"options":  {
...
}
},
Abc2 should be visible anytime abc = test1/value. But this is not the case. Kindly advise.

Hey @MaryanneNjeri, let me close this ticket since the issue is not related to pipeline task SDK. Feel free to request a reopening if you have any further questions.


Don't work after refresh

I haven't been able to reproduce, as seen in screencast:

If you know the exact steps to reproduce, please reopen issue!

Ok, I'ill try it.

Is quite simple to reproduce it.

Go to http://battery.moogs.io/demo/
Close
Reopen from history

You will see that isn't working after refresh, is the same issue I have but with other perspective.
I'm currently using 49.0.2623.112 m.

If you get the same issue after reproduce these steps, please reopen issue (as reporter I can't do it).

@miguelmota.


Cannot read properties of null (reading 'assignedSlot')

Describe the bug
MainLayout.tsx:32 TypeError: Cannot read properties of null (reading 'assignedSlot')
at getParentNode (getParentNode.js?v=2be83490:12:13)
at getScrollParent (getScrollParent.js?v=2be83490:15:26)
at getScrollParent (getScrollParent.js?v=2be83490:15:10)
at listScrollParents (listScrollParents.js?v=2be83490:19:22)
at Object.setOptions (createPopper.js?v=2be83490:70:19)
at createPopper (createPopper.js?v=2be83490:215:14)
at Object.fn (PopperUnstyled.jsx:133:24)
at runComputation (dev.js:695:22)
at updateComputation (dev.js:678:3)
at runTop (dev.js:783:7)
Additional context
It happens randomly.

This cannot be debugged without a reproduction and looks like it could be a MUI bug.
We are closing issues for Popper to focus on tracking Floating UI, but PRs are still being accepted.


How to avoid deserialization of untrusted data attacks with Hyperion serializer?

Discussed in https://github.com/akkadotnet/akka.net/discussions/5329

Originally posted by jonnydee October 19, 2021
We are currently using Akka 1.4 with Hyperion serialization configured for all user messages sent between Akka systems. We want to make sure Hyperion cannot be tricked into deserialization of untrusted data (see CWE-502 for details about that matter).
I know about Hyperion's akka.actor.serialization-settings.hyperion.disallow-unsafe-type configuration option which can be used to enable a built-in blacklist for dangerous types. This approach doesn't suffice in our case, however, because we need to trust that the built-in blacklist currently is complete and also will remain complete in the future. Note that being forced to upgrade Akka to some newer release just because new/missing types might have been added to the blacklist is not an option for us. Being able to configure types on the blacklist ourselves would surely be an improvement but the general problem of an outdated blacklist would still remain. It would just be a shift in who is responsible / trusted to have the most recent and complete list of types on the blacklist.
We would like to have an option for a whitelist approach where we list all types that are allowed to be deserialized explicitly. I looked into Hyperion's "Pre-Configured Types" mechanism which involves an IKnownTypesProvider. I hoped it would allow us to configure known types explicitly while at the same time ruling out unknown ones. Please correct if I am wrong, but it seems like this is not the case. Instead, known types would be replaced by some id (which is good) but "unknown" types would still be serialized with type information (which is bad). Furthermore, Hyperion's current implementation for mapping ids to known types seems to depend on the order of the types returned by IKnownTypesProvider.GetKnownTypes() method. (The id corresponds to the type's position in the returned IEnumerable<Type>.) This makes an approach where known message types are collected by some automatism (e.g. a reflection-based approach) difficult, because all involved Actor Systems would have to see exactly the same list (size/items/order) of known types even though some of them would never see all messages in that "overall" list.
We know that using some schema-based serialization technology (e.g. Google's Protocol Buffers) doesn't suffer from such attack vectors but, unfortunately, we cannot use it in our product. Also securing communication between Actor Systems using a VPN is not an option for us.
So the questions are:

Is there some other way to defend against untrusted data attacks when using Hyperion?
Do you have any idea on how to implement a whiltelisting machanism for Hyperion in an easy and future-proof way?

We need to find a way to configure black and white listing of types for the Hyperion serializer.

[x] Add Whitelist/blacklist implementation
[ ] Improve KnownValueProvider to not depend on array index for its manifest id

Hyperion security update
General improvements
In SerializerOptions class:

IEnumerable<Type> knownType .ctor parameter should be changed to IDictionary<ushort, Type> knownTypes
Type[] KnownTypes should be changed to Dictionary<ushort, Type> KnownTypes
Changes would be made so that it is backward compatible with the old .ctor

In Serializer class:

ValueSerializer[] _knownValueSerializers should be changed to Dictionary<ushort, ValueSerializer> _knownValueSerializerMap

In SerializerSession:

_nextTypeId should be initialized with serializer.Options.KnownTypes.Max(kvp => kvp.Key) instead of array length
_trackedTypes should be changed to Dictionary<ushort, Type>

In DeserializerSession:

_offset should be initialized with serializer.Options.KnownTypes.Max(kvp => kvp.Key) instead of array length
_identifierToType should also be changed to `Dictionary<ushort, Type>

User known types ID will be limited from 0 to 32768 (half a ushort) because we need to make sure that there are enough leeway for the dynamic IDs, this will be checked during serializer initialization
Reasoning:
As of now value serializer manifest index is based on an array index of the knownTypes type array passed in the option, this is bad because manifest can only be added, not removed.
Whitelist/blacklist implementation
Base interface design
Use callback method:
public interface ITypeFilter
{
    bool IsAllowed(string typeName);
}

Strict list type using a builder pattern
var myFilter = TypeFilterBuilder.Create()
   .Include<TMessage1>()
   .Include<TMessage2>()
   .Include<TMessage3>()
   .Build();

Build pattern should return an implementation of the callback interface.

Default implementation
public class TypeFilter: ITypeFilter
{
    public string FullName { get; }
    public string AssemblyName { get; }
}

It would use a simple lookup internally for performance reasons
Should split and store the type full name and assembly name in string
Assembly name should not contain versioning and public key token information
To check for equality, the input is split on the first , char

index 0 should be compared to the type full name
index 1 should be compared to the assembly name
index 2, if exists, has to match "%core%"

Changes would be made so that it is backward compatible with the old .ctor

Need to ensure wire compatibility too. We should probably save some output from previous versions of the serializer into binary files and incorporate those directly into the repository for forwards-compatibility. For backwards compat we'll need a test harness that can be programmed to run using older versions of Hyperion to read output created from a newer version.
Maybe we can just create a set of Docker images to do this?

public interface ITypeFilter
{
    bool IsAllowed(string typeName);
}

I like the idea that this is the top-level interface and only a single one of these should be passed into the options / settings class. If this method returns false then we log a warning / error and drop the message.
Users could, in theory, compose multiple of these ITypeFilters together on their own and just use boolean logic to wire it up for scenarios such as having per-library filters and only one of them needs to return true in order for the message to be processed.
I like the idea of having a default implementation of the ITypeFilter that simply contains an array of strings containing "simple" type names (identical to how they're exposed in Hyperion's wire format, otherwise it wouldn't work) and a builder that allows the user to produce those strings by supplying the set of types. Users could extend those builders to include assembly-scanning, "find all types that inherit this interface"-type functionality but that's not really our job. Provide something that is simple and secure.
I think the design so far here looks good - the wire compat is going to be the biggest concern...

Wire format compatibility would be a bit hard to do since the manifest id would be different if the user uses a different indexing scheme compared to the old implementation, and leaving it to the user to make their indexing scheme compatible with the old implementation is a bit of a tall order...

The thing that will break compatibility is the manifest id changes, and its a bit hard to separate the implementation so that we can maintain both the old and new implementation working at the same time, since it is tightly woven into the code. We defer the manifest id changes and just do the security update to maintain compatibility for now.

The thing that will break compatibility is the manifest id changes, and its a bit hard to separate the implementation so that we can maintain both the old and new implementation working at the same time, since it is tightly woven into the code. We defer the manifest id changes and just do the security update to maintain compatibility for now.

Separating the concerns is a good idea. We should probably resume our years'-old project to formalize the wire format.

Reopening this because it is only half solved

Ah, is this because of the known types providers?

Yes, I'll open a new issue instead


marmoset missing from NCBITaxon slim

please add NCBITaxon:9483

It looks like you use your own taxslim. You are welcome to use the 'central' taxslim. The advantage here is that it is regenerated with each NCBITaxon release.
Simply add your ID here
https://github.com/obophenotype/ncbitaxon/blob/master/subsets/taxon-subset-ids.txt
Click edit, add ID, make a pull request
More on NCBITaxon
http://obofoundry.org/ontology/ncbitaxon.html
Slims:
https://github.com/obophenotype/ncbitaxon/blob/master/subsets/README.md

Yep, had looked into this. I'll submit a pull request it a few days. Thanks!


[Bug]: PDF Reverse Words Copy

Minimal, reproducible example
// define the options per output
let pdf_publish_options = {}
// for pdf
pdf_publish_options = {
  path: outputFile,
  format: 'A4',
  landscape: false,
  preferCSSPageSize: true,
  ignoreInvalidPageRanges: true,
  printBackground: true,
  displayHeaderFooter: false
}

// standard log info: script.js:path/input.html|path.output.ext|a4|landscape
console.log(path.basename(args[1]) + ' (' + outputType + '):' + inputFile + '|' + outputFile);

// prepare launch args
puppeteer_args = ['--disable-web-security']
// check special config when running as root
if (process.getuid() == 0) {
  puppeteer_args.push('--no-sandbox')
}
// convert html into pdf
const browser = await Puppeteer.launch({ headless: 'new', args: puppeteer_args }).catch(e => console.error(e));
const page = await browser.newPage().catch(e => console.error(e));

page.on('console', msg => {
  for (let i = 0; i < msg.args().length; ++i)
    console.error(`${i}: ${msg.args()[i]}`);
});

// use the content received by eleventy
await page.setContent(elevResult[0].content, { waitUntil: 'networkidle2' });
// save as pdf
await page.pdf(pdf_publish_options).catch(e => console.error(e));

// stop the headless session
await browser.close().catch(e => console.error(e));

})(); // directly call the async block

Background
I use puppeteer to render html content into PDF.
Expectation
To get a PDF where I can copy the content from to clipboard.
Reality
The Content is in "Reverse Order" when copied and the marker is (half) on the lower half of the text.

The copied text is:

If I use puppeteer release v23.0.0 up to v23.7.1 everything works as expected.
I use custom fonts (Noto Sans) but I embedd the ttf fonts data into the CSS like:
@font-face {
  /* latin */
  font-family: 'Noto Sans';
  font-style: normal;
  font-weight: 400;
  font-stretch: 0% 200%;
  font-display: swap;
  src: url(data:application/x-font-woff2;charset=utf-8;base64,AAEAA ....);
}

I tested all releases up to release v23.11.1 but non newer than v23.7.1 does work.
It does not work on MacOs either.
Puppeteer configuration file (if used)
No response
Puppeteer version
23.8.0
Node version
20.11.0
Package manager
pnpm
Package manager version
9.15.1
Operating system
Linux

looks like #2206

Does it happen if you print using the built-in print dialog? Please include the test page that reproduces the issue including the font files (I think it should happen with any way you print using Puppeteer) into the report and file an issue to https://crbug.com/new as Puppeteer only saves the PDF produced by the browser for your. Issue https://github.com/puppeteer/puppeteer/issues/2206 indicates that this could be an issue with web fonts so perhaps using a different (newer) font file could fix it?

Hi Alex @OrKoN
you find a testing repository at:
https://github.com/TomFreudenberg/puppeteer-test
I added a readme and described the tests.
The test HTML and PDF results are already included in the repository.
PDF for "copy" failed on:

puppeteer(v23.11.1)/chrome/linux-131.0.6778.69/chrome-linux64
puppeteer(v23.8.0)/chrome/linux-131.0.6778.204/chrome-linux64

PDF for "copy" works on:

puppeteer(v23.7.1)/chrome/linux-130.0.6723.116/chrome-linux64
puppeteer(v23.11.1 OSX)/chrome/mac_arm-131.0.6778.204

Maybe it is not a puppeteer but a chromium-headless problem?
Hopefully the information will help you to indicated the issue.
If I can help in any way, please let me know.
Cheers
Tom

Hi Alex @OrKoN
to day I tested to use npm release v23.11.1 from puppeteer but changed the linked chromium release to chrome/linux-130.0.6723.116
This works as well, so the issue is "back" with chromium chrome/linux-131.0.6778.???
Cheers
Tom

I debugged a little bit deeper by using
npx @puppeteer/browsers install<EMAIL_ADDRESS>
and
npx @puppeteer/browsers install<EMAIL_ADDRESS>
and between this two chrome release the issue occurs.
That's the diff: https://chromium.googlesource.com/chromium/src/+log/131.0.6765.0..131.0.6766.0/?n=10000
You also see a big increase of the resulting PDF size. While the "working" ones are around ~55KB the later and not workings are about ~115KB.

Should I profile this issue now on chromium site?
Cheers
Tom

Hi @mathiasbynens
After reading https://developer.chrome.com/blog/chrome-for-testing maybe you may jump in and guide for the correct follow up?
Thx
Tom

I opened an issue at: https://issues.chromium.org/issues/386463464


(FACT-828) puppetversion: massive speed boost

New pull request as of https://github.com/isaacs/github/issues/18, retargeted agains 2.x as requested.
Cheers,
Thomas

CLA signed by all contributors.

Simple change, makes things better. Thanks for your contribution!


init_thread_pool on a Yew App built with Trunk doesn't work

Hello,
I'm trying to build a App with Yew that uses Rayon and I'm using Trunk to build it. Both with and withou no-bundler fails to execute the parallel task.
Without no-bundler
When the app is built without the no-bundler feature the workerHelper.js throws on the const pkg = await import("../../.."); because it receives a response of MIME text/html as the requested resource is the index.html of the App.
Failed to load module script: Expected a JavaScript module script but the server responded with a MIME type of "text/html". Strict MIME type checking is enforced for module scripts per HTML spec.
Uncaught (in promise) TypeError: Failed to fetch dynamically imported module: http://<IP_ADDRESS>:8000/

If i change the import on the workerHelper.js to be const pkg = await import("../../../yewapp.js");, the error changes to be the same as when with the no-bundler features.
With no-bundler
When the app is build with the no-bundler feature the App throws on the initialization of the Yew component with:
Uncaught (in promise) Error: Can't find the global Window

What can I try to see if it causes change?

I'm starting to process what is happening, the issue is that an Yew app works by modifying the HTML of the page, and since the all workers load the entire Yew app, every worker tries to run the code that accesses the page Window, but don't have access to it.
It would need to break the Yew app into the rendering part and the parallel part and have the worker only load the parallel part.


Setup the production environment to take file uploads

In #20, I setup the app to handle file uploads on the disk for development and test environments. In order to push to a live system, we'll need to configure a remote service to accept uploads. I don't know what you want to do with this @joemasilotti, but I wanted to open this issue to field what you're thinking.

Thanks for logging the issue! I can take care of this one once I set up our production environment.

Done!


How to use ask!

Hi Folks,
Sorry for the noob question here. Can I have one example of how to use ask! to implement a smart contract in AssemblyScript on Substrate please?
Thank you in advance!
Cheers
Rajesh

Hi, we are writing the docs

@yjhmelody
Is the doc ready yet?

@masafumimori  It will be release this weekend.

@koushiro
Fantastic!
Do you know SwankyCLI will also support ask?
I tried to run swanky init test and chose "Ask!" for the framework and it threw an error saying Sorry, ask is not supported yet

Do you know SwankyCLI will also support ask? I tried to run swanky init test and chose "Ask!" for the framework and it threw an error saying Sorry, ask is not supported yet

No, I don't know.

@koushiro
Hey! Where can I find the doc?
It should be available now according to what you mentioned?

@masafumimori It will be released this weekend.

@masafumimori sorry about that.
We found that Ask! is not compatible with the latest metadata of pallet-contracts last week, so the documentation isn't all done yet.
I'll let you know once it's done, it should be ready this week 🤣


fix #12: Importing apispec only if OpenAPIResponse is used

I wanted to fix the tests but I think it's not a good solution because you install it into poetry.lock.

@PeRDy I have just "black" it, normally should be ok.

Updated


ENH: signal: added irrnotch and iirpeak functions.

Hello, my name is Antonio and I am a Brazilian electrical engineer currently pursuing my master degree. This would be my first contribution to scipy.
I have implemented and included two new functions to scipy: iirnotch and iirpeak which are
corespondent to the matlab functions with the same names (iirnotch and iirpeak).
The notch filter can be used to remove a frequency component from signal and can be useful, for instance, if you want to remove power-line interference from a signal. The peak filter is dual to the notch filter, attenuating every frequency component except the specified one.
Thanks for taking the time to consider my pull request.

If it to go to "Matlab-style IIR filter design" then function signatures are good. Otherwise, they can be tweaked somehow.
Someone who is more closely related to scipy.signal should take a look.

Thanks for looking into this, looks like a good start.
First, you need a rebase to latest master. (Don't merge -- use rebase.)
Can you add to the docs some comments on the relationship between this and band-stop IIR filter design? Also maybe add band-stop version to your example code.
We have started having output=... arguments, e.g. 'sos' or 'zpk' to our filter design functions, especially since high-order filters are often unstable with numerator/denominator coeffecient output. However, it looks like this only ever creates a second-order filter, is that correct? If so, could a high-order band-stop filter do better?

I would like to highlight some points about this methods:

The iirnotch computes the coefficients of a band-stop filter with a narrow band and the iirpeak of a band-pass filter with narrow band.
They always produce second order filters and therefore are less likely to suffer from numerical instability (I didn't include an output=... argument for this reason, but I can include it if you guys find that it would be better to have it).
If the application require a single frequency to be removed from the signal the notch filter have advantages in relation to a high-order band-stop filter because the notch filter is a low order filter
that manages to completely eliminate a frequency component from the signal (it includes a zero on the correspondent frequency that manages to do it) with a very narrow bandwidth.
On the other hand, if the application requires to reject a range of frequencies a band-stop filter would be preferable. A similar argument can be used to compare the peak filter with the band-pass filter.

My final suggestions:

Use the log-scale for frequencies in the plots.
Use degrees instead of radians for the phase.

I believe both are conventional choices for such plots.
Other than that, I think it's in a good shape. @Eric89GXL please make a final review and decision.

For consistency's sake, instead of ab, I think it would be better to use rp/rs as is used in functions like signal.ellip.
Instead of testing equivalence with hardcoded values from MATLAB, I'd prefer if the filters were tested more directly. I.e. verify via freqz (or even filtering) that the filter requirements are met.

Instead of testing equivalence with hardcoded values from MATLAB, I'd prefer if the filters were tested more directly. I.e. verify via freqz (or even filtering) that the filter requirements are met.

I would say replace instead of with in addition to, since it seems like both types of tests have value.

I like the example output now:
https://521-1460385-gh.circle-artifacts.com/0//home/ubuntu/scipy/doc/build/html-scipyorg/generated/scipy.signal.iirnotch.html#scipy.signal.iirnotch
But maybe it's worth showing a few different Q values?

Ok so I am almost finishing the changes on the code and I would like to point out what suggestion I have fulfilled:

I have used degrees instead of radians.
I have changed ab to rp.
I have added one test for each function. These test evaluate if the frequency response comply with the specifications.
I have include the page numbers.

I have not implement some of the suggestions and I would like to justify why:

I have  not used log-scale for the frequencies. The reason is the following: while for analog filters the use of log-scale is ubiquitous, when work with digital filters that have a frequency response that goes from 0 to pi it is more commom to use linear scale (e.g. all the figures from [1] uses a linear scale when plotting normalised frequencies). Since we are working with a digital filters but choose to represent the frequencies using Hz we are kind of in a grey area and I think both would work. But in the case of notch/peak filters there are no great need to work with both small and large scales (that is what the log-scale does), the main interest is around the frequency f0, and the log-scale will only distort that region.
I have not showed different values of Q in the example. The reason is the following: I think the example plots would be more informative if I have used more than one value of Q and illustrated the effect of it,  nevertheless I think it would make the example more complicated. The way it is I think the user can copy and paste the example, adjust its parameters and would be able to use the filter out-of-the-box. If I started to add adicional analysis I think it would become less straight forward to do so.

If the aforementioned comments did not manage to convince you guys I will implemented the suggestions when asked.
[1] Oppenheim, Alan V., and Ronald W. Schafer. Discrete-time signal processing. Pearson Higher Education, 2010.

Well explained about a log-scale for frequencies, let's leave it in a linear scale.
As for the phase plot for iirnotch: wouldn't it be better to plot it from 0 to -360 so it looks continuous?

An idea about Q-factor: if it such a convenient parameter it can be allowed as an alternative to bw as (for example) iirpeak(w0, bw=None, ab=3, Q=None) with an explanation that either bw or Q must be set. Is it known how to compute bw from Q with ab != 3?
Alternatively, bw can be replaced by Q completely, like iirpeak(w0, Q=10, ab=3).
But I'm not sure that it will be better than the current version, just an idea.

About using Q as an alternative parameter: the problem is that I don't know how to compute Q for ab != 3. And I am afraid it may make things more confusing since each parameter would impose some restrictions on the others...
About the angle of iirnotch the discontinuity at w0 is of 180 and reflects that the module goes from positive to negative at that frequency. The discontinuity could go from -90 to +90 or could go from -90 to -270, but it would be there nevertheless. I am already using unwrap so I don't know how could I restrain the fase from 0 to -360 without manually subtracting 360... What do you think?

About Q-factor. I also suspected that it can bring unnecessary complications.
About plots. Sorry, I didn't realize that it was 180 degrees jump (only briefly looked at plots with radian scale). Sure nothing can be done with it.

This isn't a hard objection, but  here are my two cents about Q vs bw:
I use these kinds of filters often, and prefer using Q, since it's invariant in a geometric way. In effect, there is some degeneracy in bw and rp, i.e. in any notch produced in this way, there is always a band where the attenuation is -3dB, so one could argue for a simplified call signature of just iirnotch(w0, Q). (This is what I use in some personal functions)

Hey @e-q thank you for your comment. Your argument that use Q is better than using bw is
convincing and I agree that there are advantages in using iirnotch(w0, Q) as a signature.
On the other hand, using iirnotch(w0, bw) as a signature the user can easily specify the Q factor
using the formula bw = w0/Q, clearly stated at the documentation, and have the extra liberty to set the parameter rp, calling the function as iirnotch(w0, bw, rp).

I would prefer iirpeak(w0, Q) version, unless it is explained what is the advantage of using rp and why it is helpful for this kind of filter (otherwise it looks like a parameter no one going to use).

My sense is that Q is more natural as well

Ok! I will change it.

About the implementation, I would like your guys suggestions about something:
So far I have put the functions on Matlab-style IIR filter design but with the signature
change I don't think it fits there any longer. So I see two options:

Delete it from this place and change it to the filter design section.
Maintain functions on Matlab-style IIR filter design section with a signature
identical to the matlab one and add functions notchfilter(w0, Q) and peakfilter(w0, Q)
under the filter design section.
What do you guys think? I like the second option better, but I not sure about it.

I think it still fits within the MATLAB-style section even if the function signature is not identical. If it can perform the same functionality with a similar signature it seems okay to put it there.

I changed the signature to irrnotch(w0, Q) and irrpeak(w0, Q), I was reluctant at first but I think it  looks nice!

This is looking good. Unfortunately, it looks like there are conflicts with the master branch. If you are able, you should try to rebase this PR off of the current state of the master branch.

I rebase my branch. I think it is ok now.

Other than my last gripe, LGTM +1 for merge.
@e-q @nmayorov are you happy now, too?

+1 from me too.

Looks good! Could probably use a squash before merging.

Ok @e-q I squash all commits into one.

Great, unless anyone else wants to have a look, I'll merge in a day or two.

Hey guys, I rebase the branch to solve the conflicts that have shown up recently.

In it goes! Thanks @antonior92


Option to bind ctrl-0 to last tab

Hi, could you add that option to match Firefox normal use case?
I upgrade  to Ubuntu 18.04 and found that these shortcuts do not work in Firefox for Gnome.
However, your Addon pretty much solved it.
Problem is that I'm used to ctrl-0 going to last tab.
Thank you!

Hey, do you know what keybinding is used for this on a Mac? When this extension used Ctrl-0, people were reporting issues because it is used to reset zoom on OS X, IIRC.

The actual binding for the last tab is Ctrl-9. Even in Linux, Alt-9 takes you to the last tab, so this addon is inconsistent with that. Would be nice to adapt it.


pre-processor support for sizeof(type)

As far as i recall, you can write
#define SIZE_OF_INT sizeof(int)

in C.
However, when I write
DEF USE_LONG = 1
IF USE_LONG == 1:
    ctypedef long Int
ELSE:
    ctypedef int Int
DEF SIZE_INT = sizeof(Int)

in Cython, the compiler gives the error: Invalid compile-time expression for the last line above. But the result of sizeof(int) can be known at compile-time.

This can't be done, because the generated c file may be compiled on platforms with different integer sizes. (There are two compile phases, pyx -> c and c -> so/dll and this error refers to the former whereas sizeof(int) is only known at the latter.)


Fix error spam when adding tabs to TabBar without deselect

Ensures current has a valid value after resizing if deselection is not allowed.
The revert marker being visible for the deselect disabled case is a bit annoying and confusing but is a bit more complex to solve, and unsure how it would affect other things with the default values etc., but worth looking into in the future as well.
MRP:
TabMRP.zip
Note how it will print:
ERROR: Index p_idx = -1 is out of bounds (tabs.size() = 2).
   at: TabBar::ensure_tab_visible (scene/gui/tab_bar.cpp:1555)

When opening the file, and also when resizing the tab bar
This ensures the current value is updated to the correct range in this case
Edit: Confirmed that this works on top of:

https://github.com/godotengine/godot/pull/88477

Works good for changing the tab amount.
The issue can still happen when toggling the tab's disabled property.
Set them all to disabled, set the current to -1, then enable one of the tabs, and the error will occur.
To fix this, a similar change should be made in TabBar::set_tab_disabled().
And it can't be set in the inspector, but we might want to do the same in TabBar::set_tab_hidden() since _can_deselect() also checks that.

Will fix that later today, thank you!

Removed the check for _can_deselect from the ensure_tab_visible, this solves the above errors as well, it doesn't make sense to have that check there as ensuring the visibility of the deselected tab doesn't mean anything regardless of the deselection state

Thanks!

Thank you!


FAQ needs revision: Can I assign Pod subnets from a different VNet altogether

Can I assign Pod subnets from a different VNet altogether?
The pod subnet should be from the same VNet as the cluster.
There seems to be gray area in this answer. Is it feasible or not? Caveats? Performance trade-offs?

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

ID: 37ba0343-7e43-6e2e-e564-5d3a0f523a75
Version Independent ID: 0ba581a1-3e4a-27b4-c667-4a9b969a3ade
Content: Configure Azure CNI networking in Azure Kubernetes Service (AKS) - Azure Kubernetes Service
Content Source: articles/aks/configure-azure-cni.md
Service: container-service
GitHub Login: @zr-msft
Microsoft Alias: zarhoads

@MarcosMMartinez Thank you for your question!  We are reviewing the documentation and will get back to you shortly.

Thank you for the feedback.
We've updated the article to address your feedback and the changes should be available soon.


[json-rpc] Fix Verifying API

This contains fixes so that the verifying API has a uniform API.
Now the get_transaction_with_proofs function will behave like the get_events_with_proofs and has an optional known_version parameter that allows to ask for proofs against a specific version.
It also adds back a test that shouldn't be flaky but that seem to randomly fail on master.
Related PRs
#7062
#6539
#6914
#6399

This PR may have modified JSON-RPC server code.
Breaking changes policy:

Chaning JSON-RPC method signatures or removing/modifying fields
in the response in /V1 will not be accepted.
Adding fields is ok.
Adding a new JSON-RPC method is ok.

Please ensure you have also done the following to update the docs:

Update json-rpc/API-CHANGELOG.md.
Add/update type/method documentation under json-rpc/docs.

This PR may have modified JSON-RPC server code.
Breaking changes policy:

Chaning JSON-RPC method signatures or removing/modifying fields
in the response in /V1 will not be accepted.
Adding fields is ok.
Adding a new JSON-RPC method is ok.

Please ensure you have also done the following to update the docs:

Update json-rpc/API-CHANGELOG.md.
Add/update type/method documentation under json-rpc/docs.

could you add an item to API-CHANGELOG for the change of api params?

could you add an item to API-CHANGELOG for the change of api params?

/land

/land