Update synopsys references to blackduck

This PR updates the following for sierra changes:

Update package names
Update email recipients
Update License
Remove EULA
Update dependency group names

Looks like source code headers still contain "Copyright (c) 2024 Synopsys, Inc.".

Looks like source code headers still contain "Copyright (c) 2024 Synopsys, Inc.".

As soon as this PR gets merged, https://github.com/blackducksoftware/integration-resources/pull/2, we can update all the headers at once by building the project, so to avoid manual work I will push one commit to update copyright, import statements and new version of library at the end before releasing.

@dterrysynopsys Should not regenerated code automatically apply rebranded names for packages? Would manual rename affect future generation of code?

@dterrysynopsys Should not regenerated code automatically apply rebranded names for packages? Would manual rename affect future generation of code?

I think we would need to change the generator, run a generation, and see what is left. I know we have some manual tweaks so changing some files by hand is likely necessary but yes, I agree we need to change the generation or we'll likely stomp on these changes. I'd start by looking at https://github.com/blackducksoftware/blackduck-common-apigen/blob/master/src/main/java/com/synopsys/integration/create/apigen/generation/finder/ClassNameManager.java#L20


Stabilize os daemons test

Increase delay timeout to fix this failure:
**error:{assertNotEqual,[{module,couchdb_os_daemons_tests},
                 {line,207},
                 {expression,"D # daemon . kill"},
                 {value,undefined}]}

So far noticed this failure once out of more than 100 runs.

Well, it didn't help:
    couchdb_os_daemons_tests:100: should_check_daemon...*failed*
in function couchdb_os_daemons_tests:'-check_daemon/3-fun-2-'/2 (test/couchdb_os_daemons_tests.erl, line 207)
in call from couchdb_os_daemons_tests:check_daemon/3 (test/couchdb_os_daemons_tests.erl, line 207)
**error:{assertNotEqual_failed,[{module,couchdb_os_daemons_tests},
                        {line,207},
                        {expression,"D # daemon . kill"},
                        {value,undefined}]}

Since we cannot subscribe to any event here, I suggest to make retry loop with the checks like what we have for compaction tests.

+1


Cannot generate grapqhl schema for Up Bank API

Describe the bug
Not able to generate Graphql Schema with openapi-to-graphql cli using Up Bank's API OAS definition
To Reproduce
Steps to reproduce the behavior:

install the openapi-to-graphql-cli
try generating schema by running:
openapi-to-graphql https://github.com/up-banking/api/blob/master/v1/openapi.json

This also occurs if the schema is downloaded as a .json file and run withthe command openapi-to-graphql .\api.json
Expected behavior
The graphql server starts with generated Up Bank API schema
Actual result
Getting the following error:
openapi-to-graphql https://raw.githubusercontent.com/up-banking/api/master/v1/openapi.json
OpenAPI-to-GraphQL creation event error: undefined

@Kamholtz This is a very strange error. Thank you for reporting it and also including a link to the original OAS. I would like to take a look at it some time but I am very busy with other work, just to let you know my situation.

@Kamholtz this maybe the cause:
I pop the swagger into https://editor.swagger.io/ and I am getting back:

Semantic error at paths./transactions/{transactionId}/relationships/tags.delete.requestBody
DELETE operations cannot have a requestBody.
Jump to line 2072

Interesting. I tried removing the delete from the openapi.json i.e. removing the following
      "delete": {
        "tags": [
          "Tags"
        ],
        "summary": "Remove tags from transaction",
        "description": "Disassociates one or more tags from a specific transaction. Tags that are\nnot associated are silently ignored. An HTTP `204` is returned on\nsuccess. The associated tags, along with this request URL, are also\nexposed via the `tags` relationship on the transaction resource returned\nfrom `/transactions/{id}`.\n",
        "requestBody": {
          "content": {
            "application/json": {
              "schema": {
                "$ref": "#/components/schemas/UpdateTransactionTagsRequest"
              }
            }
          }
        },
        "parameters": [
          {
            "name": "transactionId",
            "in": "path",
            "schema": {
              "type": "string"
            },
            "required": true,
            "example": "d8c190d6-be35-4d64-b2c4-e1aa0c09a0e0",
            "description": "The unique identifier for the transaction.\n"
          }
        ],
        "responses": {
          "204": {
            "description": "Successful Response",
            "x-up:example": {
              "type": "exampleRequestResponse",
              "request": {
                "headers": {
                  "Authorization": "Bearer up:demo:cktrwA7Wmm5eSlBA",
                  "Content-Type": "application/json"
                },
                "pathParameters": {
                  "transactionId": "734affdd-fc33-4dec-b4d1-ce1b2a2fd883"
                },
                "queryParameters": {
                },
                "payload": {
                  "data": [
                    {
                      "type": "tags",
                      "id": "Holiday"
                    },
                    {
                      "type": "tags",
                      "id": "Queensland"
                    }
                  ]
                }
              },
              "response": null
            }
          }
        }
      }

to produce the attached file, but that caused a similar error message:

openapi-to-graphql .\server\openapi-no-delete-request-body.json
OpenAPI-to-GraphQL creation event error: undefined

openapi-no-delete-request-body.zip

Your OAS is failing validation because it has a templated callback path "{webhookURL}" but doesn't declare webhookURL as a path parameter.   The error message should have explained this but isn't being displayed correctly.


[SPARK-8894][SPARKR][DOC] Example code errors in SparkR documentation.

Merged build triggered.

Merged build started.

Test build #36794 has started for   PR 7287 at commit da63898.

Test build #36794 has finished for   PR 7287 at commit da63898.

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

Merged build finished. Test PASSed.

LGTM. Thanks @sun-rui for the fixes.


diffusion: include gaussian refinement in warning

Why this PR?
If we mention that centroid refinement is an option, we should also mention gaussian refinement. This omission was noticed by Aafke.

I merged this into the release PR.


Update RP2040-hal example urls

Hello,
Here a comment fix :
https://github.com/rp-rs/rp-hal/tree/main/rp2040-hal/examples/ is outdated
replaced by:
https://github.com/rp-rs/rp-hal/tree/main/rp2040-hal-examples/
🚀

Thanks!


Address reset password functionality not working properly. Update tests.

Fixes

Addresses test case discussed in https://github.com/Police-Data-Accessibility-Project/data-sources-app/issues/162

Description

Addresses test case of checking to make sure that a user cannot use their own reset password token to reset another user's password (and thereby be able to log in as them).
Through the above test case, discovered a vulnerability that enabled the above scenario to occur, and resolved it.
In so doing, discovered a bug in reset password (and the associated tests) where the reset token, rather than the new password the user provided, was being made into the new password hash, This would have prevented anyone from resetting their password.
The relevant integration was failing to check that a user could log in with the new password -- it merely checked that the new password hash and the old password hash were different. That test has been updated accordingly.

Testing

Run all tests in tests directory to confirm functionality.

Performance

Impact marginal

Docs

Identified error in docs for /reset-password, where new_password input was not mentioned. Addressed.

Breaking Changes

None.

@josh-chamberlain I addressed this bug, but it did provoke two observations:

Prior to the full release of v2, we should do a final manual pass through every single endpoint to make sure they behave as expected, following the instructions provided in the API docs, particularly those that affect the behavior of other endpoints (like /reset-password and /login). In some cases, as with this one, simply testing that it returns a success response may not fully indicate its health.
Similarly, I should make a passthrough of my integration tests to ensure that my integration tests, wherever possible, are interacting with the database through endpoints rather than directly inserting, updating, or pulling from the database at the test level. Our endpoints are robust enough to enable this for more integration tests. This will make the tests a bit more cumbersome (absent some clever refactoring, which I love doing), but it does mean those tests more closely approximate user interactions with the web app. In this case, my original test was checking that the two password hashes in the database were different, but that's not what users care about; they care about whether they can log in with one password but not the other.

@maxachis agreed that tests should use the API when possible, because that's closer to the actual user/client behavior. Nice!


⚠ Samsung has degraded performance

In 21e6907, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2274 ms

Resolved: Samsung performance has improved in 437a0e2.


fix(Data Mapper): able to move nodes again after rf upgrade and only layout nodes if no metadata is found on load

Requirement Checklist

[x] The commit message follows our guidelines
[ ] Tests for the changes have been added (for bug fixes or features)
[ ] Docs have been added or updated (for bug fixes or features)

Type of Change

[x] Bug fix
[ ] Feature
[ ] Other

Current Behavior

New Behavior

Impact of Change

[ ] This is a breaking change.

Screenshots or Videos (if applicable)


dr-train not working with stable diffusion XL

The dr-train.py file is setup to use the train.vendor.huggingface.diffusers.train_text_to_image main(), which is not compatible with SDXL. For example, the line:
  model_pred = unet(noisy_latents, timesteps, encoder_hidden_states).sample
in train_text_to_image fails with an error:
unet_2d_condition.py", line 876, in forward
    if "text_embeds" not in added_cond_kwargs:
       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: argument of type 'NoneType' is not iterable

I was using this command to run dr-train:
dr-train \
  --pretrained-model-name-or-path "${BASE_MODEL}" \
  --dataset-name "${DATASET}" \
  --output-dir "${BASE_PATH}/model/${MODEL_NAME}" \
  --cache-dir "${BASE_PATH}/cache/model/${MODEL_NAME}" \
  --resolution "${MODEL_IMAGE_RESOLUTION}" \
  --reshuffle-tags \
  --tag-separator ' ' \
  --center-crop \
  --random-flip \
  --train-batch-size "${BATCH_SIZE}" \
  --learning-rate 4e-6 \
  --use_ema \
  --max-grad-norm 1.0 \
  --checkpointing-steps 5000 \
  --lr-scheduler constant \
  --lr-warmup-steps 0 \
  --mixed-precision "${PRECISION}" \
  --resume-from-checkpoint "latest" \
  --dataloader-num-workers $(nproc)

Non-elegant solution (what worked for me) was a modified dr-train.py specifically for SDXL:
import sys
from train.vendor.huggingface.diffusers.train_text_to_image_sdxl import main as vendor_main
from train.vendor.huggingface.diffusers.train_text_to_image_sdxl import parse_args

def main():
    args = parse_args(sys.argv[1:])
    vendor_main(args)

if __name__ == "__main__":
    main()

With the above, I was able to successfully start training by passing the exact same command/arguments listed above.

@RevelBerry There is a separate script for XL, dr-train-xl which does work. Fair point that this should probably be transparent for the user.

Fixed in README


rustfmt fail (>Format Document) with async syntax (wrong edition)

Typing edition 2018 syntax like:
async fn test() {
    0;
}

Makes the formatting to fail with:
error[E0670]: `async fn` is not permitted in the 2015 edition
  --> <stdin>:25:1
   |
25 | async fn test() {
   | ^^^^^

In handlers.rs:547, rustfmt is started without an edition provided.
A workaround is to set the edition in rustfmt config file.
But I think rust-analyser should know the current edition, and provide it when running rustfmt

We run rustfmt as an external process, and I feel this is out of scope for the current implementation. Once we use rustfmt as a library (which won't be any time soon), we should make sure to pass the proper edition. For the time being, it seems like it might be a good idea for rustfmt to fetch edition from Cargo.toml? Otherwise, every project would have to use a dummy rustfmt.toml which sets an edition, which seems not really necessary from the user's POW.

OTOH, I'll accept a PR which papers over this specific issue :)

I've a local patch that add --edition 2018 to the command would you accept that?

I've a local patch that add --edition 2018 to the command would you accept that?

Just adding --edition 2018 would be wrong, b/c edition could be 2015. We need to check to which crate the current file belongs, and only pass this option if it matches Cargo.toml


Popper with transformed element

Hi @FezVrasta
I'm working to remove Tether dependency from Bootstrap (https://github.com/twbs/bootstrap/commits/v4-experimental-tooltip), and we have a unit test which check if the tooltip for a transformed element is correctly placed.
It seems it's not the case
CodePen demo
http://codepen.io/Johann-S/pen/JNPBrY
What is the expected behavior?
The 1 should be center

Hi, unfortunately transformed elements support isn't supported quite well at the moment. Right now I'm working on some internal refactoring to fix higher properties bugs, but I'm open to PRs to improve support for this use case.

To add some context, we already have a failing test (disabled for now) to test the functionality (in at least one specific context).

I'll follow the development of Popper to know when this issue will be fixed 👍

May I know why do you need to support this case? Is it that common?

It's a historical test introduced in v3.3.0 and if we choose Popper instead of Tether we have to ensure that everything will work the same way than before.
See the test : https://github.com/twbs/bootstrap/blob/v4-dev/js/tests/unit/tooltip.js#L734-L769

I see, I was just curious to understand why is it so important to be inside the test suite. Maybe it's something that the Bootstrap's CSS itself makes use of or else. 🤔

No it's not in the Bootstrap's CSS itself, but it something folks use.
I've finished my work to integrate Popper to Bootstrap and everything work fine, but not this test 😢

Right now I'm stuck on this problem:
https://github.com/FezVrasta/popper.js/pull/201#issuecomment-293559345
Any help to proceed in this direction will speed up the work needed to start working on #202.

Your issue has been fixed, if the CI pass I expect to release a pre release, let you try it and then release a new version with the fix.

Nice ! Thank you @FezVrasta
In my next PR for Bootstrap, Popper will be used for Tooltip, Popover and Dropdown

Ok great!
Please install the latest pre release with popper.js@next and let me know if it works for you. I'll wait your reply to publish it as stable.

I updated my CodePen (http://codepen.io/Johann-S/pen/JNPBrY) but it seems better but not centered totally

That's something I noticed happens only in CodePen, I'm not sure what happens there 🤔

Same not on CodePen :

Ops, yes, looks like I fucked up the test. Sorry.

So, I'm having some problems computing the scrollbar presence in transformed elements... If you have ideas please write them down here. The interested function is getBoundingClientRect.

I made a CodePen with an exemple of Tether on a transformed element : http://codepen.io/Johann-S/pen/GmgyYG
Maybe it can help you

Thank you, unluckily it doesn't help because their logic is very basic and doesn't take in account a lot of variables. Btw I'm on it, expect a fix in few minutes.

Please try<EMAIL_ADDRESS>again, on CodePen there's a small offset, but on my tests not (this time for real 🙂)
If you understand why CodePen behaves differently I can fix it.

Update, figured out the problem. My tests live in a wrapping div, without it I get the same bug.
I'm working on it.

Seems better 👍 but a bit too much at the left (http://codepen.io/Johann-S/pen/JNPBrY)

So, this is my test:
https://codepen.io/FezVrasta/pen/wdByGL?editors=0010
I don't get why here everything works fine, and in your codepen it doesn't..

Fixed: https://codepen.io/FezVrasta/pen/oWgEpP

You can use 1.5.1

Thank you @FezVrasta 👍 My test on Bootstrap still failed but what I see is much much better thanks to you

Make sure it addresses decimal pixels, my tests use a custom toBeApprox assertion to make sure they don't fail if the position is just few decimals off.

Thank you I'll check that 👍
Do you know when a minified version of 1.5.1 will be available ?

It's already available:
<EMAIL_ADDRESS><EMAIL_ADDRESS>


Problem with Signing in and Creating Accounts

Hi,
I have been trying to install your dashboard using npm does not work. I get stuck at doing npm knex migrate:latest.
So instead I have installed yarn was able to use that successfully. But now I have it running and it render the pages all okay. But when it comes to functionality it is not working. I just installed so not really sure really sure where to dig in first to fix. Or if it is because I am using yarn.
Here is the error I am getting when I preform password change, register user, login, etc...
Note* In my postgresdb it does show that the users that I created were created.
TypeError: Cannot set property 'messages' of undefined
    at createUser (/home/rtarson/www/dashboard/public_html/features/register/commands/create-user.js:18:24)

Hi! I'm actually having an issue logging in. The database is seeded and I checked it so I know the users are there, but I'm not able to log in. I've removed the auth from the routes for the moment so I can do some production testing, but any help would be amazing

I purchased the PRO version and came up against similar issue when trying to login
TypeError: Cannot set property 'messages' of undefined
But my error message points to the file: verify-request-body.js

Today i have same problem, but fix as follow =)

If the session stored on redis, but u not configure the session storage. system can't save the session.
simple, now work perfect!!

thanks @MwSpaceLLC this worked for me!

@MwSpaceLLC i tried the above code that you wrote in app.js. it is giving the following error.
Error: secret option required for sessions
at session (/Users/sanketpatel/WebstormProjects/argon-dashboard-nodejs/node_modules/express-session/index.js:200:12)
at Layer.handle [as handle_request] (/Users/sanketpatel/WebstormProjects/argon-dashboard-nodejs/node_modules/express/lib/router/layer.js:95:5)
at trim_prefix (/Users/sanketpatel/WebstormProjects/argon-dashboard-nodejs/node_modules/express/lib/router/index.js:317:13)
at /Users/sanketpatel/WebstormProjects/argon-dashboard-nodejs/node_modules/express/lib/router/index.js:284:7
at Function.process_params (/Users/sanketpatel/WebstormProjects/argon-dashboard-nodejs/node_modules/express/lib/router/index.js:335:12)
at next (/Users/sanketpatel/WebstormProjects/argon-dashboard-nodejs/node_modules/express/lib/router/index.js:275:10)
at SendStream.error (/Users/sanketpatel/WebstormProjects/argon-dashboard-nodejs/node_modules/serve-static/index.js:121:7)
at SendStream.emit (events.js:310:20)
at SendStream.error (/Users/sanketpatel/WebstormProjects/argon-dashboard-nodejs/node_modules/send/index.js:270:17)
at SendStream.onStatError (/Users/sanketpatel/WebstormProjects/argon-dashboard-nodejs/node_modules/send/index.js:421:12)
at next (/Users/sanketpatel/WebstormProjects/argon-dashboard-nodejs/node_modules/send/index.js:764:28)
at /Users/sanketpatel/WebstormProjects/argon-dashboard-nodejs/node_modules/send/index.js:772:23
at FSReqCallback.oncomplete (fs.js:167:21)

Solved the error. I have to remove the RedisStore configuration from app.js.

Anyfix for the problem without removing RadisStore configuraitons?

Here's my app.js file, and my RedisStore function works fine. Make sure you install the npm package as well.
app.js.zip


Make initial slashes optional

Can make initial slashes in route table optional, plus could allow them to be added in index pages route setup.

Decided against doing this in route table, can now use absolute paths in tab page setup


"CREATE OR REPLACE TABLE" failed with SQLite3 table if given table doesn't exist

What happens?
CREATE OR REPLACE is a statement to create a table, or replace table if it already exists with single statement.
It worked as expected with in-memory temporary table, however, it didn't work as expected with SQLite Extension (cf. "To Reproduce" section follows).
% duckdb -c "CREATE OR REPLACE TEMPORARY TABLE xs AS SELECT unnest(['foo', 'bar', 'baz']) AS x;" -c "SELECT * FROM xs;"

┌─────────┐
│    x    │
│ varchar │
├─────────┤
│ foo     │
│ bar     │
│ baz     │
└─────────┘

To Reproduce
Tried with DuckDB CLI.
% rm -f ./tmp.sqlite
% duckdb -c "ATTACH 'tmp.sqlite' AS tmp (TYPE SQLITE);" -c "USE tmp;" -c "CREATE OR REPLACE TABLE xs AS SELECT unnest(['foo', 'bar', 'baz']) AS x;" -c "SELECT * FROM xs;"

INTERNAL Error: Failed to drop entry "xs" - could not find entry
This error signals an assertion failure within DuckDB. This usually occurs due to unexpected conditions or errors in the program's logic.
For more information, see https://duckdb.org/docs/dev/internal_errors
Catalog Error: Table with name xs does not exist!
Did you mean "system.information_schema.tables"?
LINE 1: SELECT * FROM xs;
                      ^

OS:
macOS 14.5 (23F79)
DuckDB Version:
v1.0.0 1f98600c2c
DuckDB Client:
DuckDB CLI
Full Name:
Yuu Yamashita
Affiliation:
Treasure Data, Inc.
What is the latest build you tested with? If possible, we recommend testing with the latest nightly build.
I have tested with a stable release
Did you include all relevant data sets for reproducing the issue?
Yes
Did you include all code required to reproduce the issue?

[X] Yes, I have

Did you include all relevant configuration (e.g., CPU architecture, Python version, Linux distribution) to reproduce the issue?

[X] Yes, I have

Thanks for filing this, transferred to here because its related to the sqlite scanner.


[Feature request] Toast position on Toast Options config

Currently, I can only configure position and order-reserve in Toaster. Hope you can do it <3

This is possible with react-hot-toast 2.0, which I just released 🎉


Gage spatial reference locations.

Probably best to honor initial source spatial location until we have more experience with location quality.
Need to output a table of all gage spatial locations for info.geoconnex.us to host. Attributes should be focused on SELFIE info properties. In the first iteration, linked-feature properties will be included. A subjectOf link to the NWIS landing page should be included.
Should do a proximity test to determine if there are overlapping gage ids that are actually the same location. Consider avoiding problem gages rather than reviewing before closing.

Realizing I don't need to do a proximity test until I have another source of sites. It's not up to the reference list to deduplicate. If, in the future, we want to start to declare sameAs and / or remove un-needed sites from this, we can.


cannot add op after concat

hi
when I convert ONNX model to CoreML model, I met a problem that I cannot add any op after concat. When I try to add op such as conv or max pool, it would have the error:TypeError: Error while converting op of type: MaxPool. Error message: Input shape not available
this is my error info:

@beibei1998 which ONNX version are you using? ONNX shape inference had few bugs which does not compute input shape which is required for above ops.
If ONNX 1.5.0 also does not work for you. I would recommend to build it from source from tot and try.
I recently faced similar issue, tot ONNX worked for me

I used the latest version of onnx which is 1.5.0. but it seems not work. and my onnx－coreml is also 3.0beta.

@beibei1998 is it possible to share ONNX model? also, please try with latest ONNX built from source. As I mentioned it worked for me in past.

thx. but i just do the convert work. what i want to ask now is that i have checked the output of the concat and the input of conv and they matches, both of them are （1,1,3,512,512）why the input is not
available?

i will try to build source thx very much

I have rebuilt the onnx and onnx-coreml source, it seems not work. The error still exists.

@beibei1998 is it possible for you to share ONNX model? Looks like something is off with onnx shape inference

I will share the model later. may I ask now why it is incorrect when conv is after concat while other convs have no error

there is a possibility that ONNX graph is not able to compute shape for concat op (possible bug) and hence, shape is not provided to conv layer

this is my model code, the error exists between the concat and conv
simple_seg.txt

can this bug fix？

convert script:
import torch
import torch.onnx as torch_onnx
from torch.autograd import Variable
from networks.versa_v2 import versa_v2, SegMattingNet
model = versa_v2()
model.load_state_dict(torch.load(path_model_path))
input_shape = (3, 512, 512)
input_names = ["input_array"]
output_names = ["output_array"]
dummy_input = Variable(torch.randn(1, *input_shape))
output = torch_onnx.export(model,
dummy_input,
model_onnx_path,
verbose=False,
input_names=input_names,
output_names=output_names)

may I ask that if this is the bug of onnx-coreml or my script/model?

thank you very much!
upcoming coremltools beta release?do you mean 1.0b2?

ok i get
thank u  very much
wait for the latest version!

@beibei1998 could you please install onnx latest version from source? Read here for instructions https://github.com/onnx/onnx#source
Shape inference is fixed in changes later than 1.5.0 release and you should be able to convert with it.

thank u
I have reinstalled ONNX from source today, but it seems no difference
if I need to update others?

@beibei1998 could you please try with latest beta? SoftMax and Rank information change is available in 1.0b3

have created PyTorch issue for the same https://github.com/pytorch/pytorch/issues/27376

@beibei1998 thanks for providing more information
This is blocked on due to onnx/onnx#1420
We have a solution via coremltools and it will be available in upcoming coremltools beta release.
vesa_simple is able to convert with the expected rank change.
There are few issues i would like to point to you

PyTorch is export ONNX op v9 and we not support v10
Shape inference issue is hitting for conv and batchnorm and hence blocked on above change

I am using following conversion script to force upsample scalar factor=2 (which is known) and using slice operator for v9
import onnx
from onnx_coreml import convert
from coremltools.proto import NeuralNetwork_pb2

INT_MAX = 2**30
def _convert_upsample(builder, node, graph, err):
    params = NeuralNetwork_pb2.CustomLayerParams()
    params.className = node.op_type
    params.description = "Custom layer that corresponds to the ONNX op {}".format(node.op_type)

    if 'scales' in node.attrs:
        scales = node.attrs['scales']
    elif len(node.input_tensors):
        scales = node.input_tensors[node.inputs[1]]
    else:
        # HACK: Manual scales
        scales = [1, 1, 2 * graph.shape_dict[node.inputs[0]][2],  2 * graph.shape_dict[node.inputs[0]][3]]

    scale_h = scales[2]
    scale_w = scales[3]
    input_shape = graph.shape_dict[node.inputs[0]]
    target_height = int(input_shape[-2] * scale_h)
    target_width = int(input_shape[-1] * scale_w)

    builder.add_resize_bilinear(
        name=node.name,
        input_name=node.inputs[0],
        output_name=node.outputs[0],
        target_height=target_height,
        target_width=target_width,
        mode='UPSAMPLE_MODE'
    )

def _convert_slice_v9(builder, node, graph, err):
    '''
    convert to CoreML Slice Static Layer:
    https://github.com/apple/coremltools/blob/655b3be5cc0d42c3c4fa49f0f0e4a93a26b3e492/mlmodel/format/NeuralNetwork.proto#L5082
    '''
    data_shape = graph.shape_dict[node.inputs[0]]
    len_of_data = len(data_shape)
    begin_masks = [True] * len_of_data
    end_masks = [True] * len_of_data

    default_axes = list(range(len_of_data))
    default_steps = [1] * len_of_data

    ip_starts = node.attrs.get('starts')
    ip_ends = node.attrs.get('ends')
    axes = node.attrs.get('axes', default_axes)
    steps = node.attrs.get('steps', default_steps)

    starts = [0] * len_of_data
    ends = [0] * len_of_data

    for i in range(len(axes)):
        current_axes = axes[i]
        starts[current_axes] = ip_starts[i]
        ends[current_axes] = ip_ends[i]
        if ends[current_axes] != INT_MAX or ends[current_axes] < data_shape[current_axes]:
            end_masks[current_axes] = False

        if starts[current_axes] != 0:
            begin_masks[current_axes] = False

    builder.add_slice_static(
        name=node.name,
        input_name=node.inputs[0],
        output_name=node.outputs[0],
        begin_ids=starts,
        end_ids=ends,
        strides=steps,
        begin_masks=begin_masks,
        end_masks=end_masks
    )

## Load ONNX Model
model = onnx.load('/Volumes/Common/Work/issues/github/concat-shape/simple_seg.onnx')

## Convert ONNX Model into CoreML MLModel
c_model = convert(model, disable_coreml_rank5_mapping=True, custom_conversion_functions={'Upsample':_convert_upsample, "Slice": _convert_slice_v9})

i got the following error when i run the project in xcode:
coremlc: Error: compiler error:  Espresso exception: "Invalid blob shape": generic_elementwise_kernel: cannot broadcast [48, 64, 32, 1, 1] and [96, 128, 32, 1, 1]
this is from upsampling, the second list is the scales that passed to 'builder.add_resize_bilinear'.
the two lists are in opposite order. looks they should be like: [1, 1, 32, 64, 48] and [1, 1, 32, 128, 96].
Any idea about this?

I tried to use the conversion script to solve the upsample scale issue, but I receive an error saying that disable_coreml_rank5_mapping is not a recognized argument for convert. Do I need to use a beta version? I am using onnx-coreml version 1.2 and coremltools version 3.3

Hello.
I have tried to use the above mentioned script, but it seems graph.shape_dict does not contain needed indices:
Exception has occurred: KeyError
'356'
  File "/Users/dmitryrybakov/Documents/hand-tracking/hand-pose-estimation/3D-Hand-Pose-Estimation/convert2CoreML.py", line 28, in _convert_upsample
    scales = [1, 1, 2 * graph.shape_dict[node.inputs[0]][2],  2 * graph.shape_dict[node.inputs[0]][3]]

I am using CoreML tools ver. 3.3:
Requirement already up-to-date: coremltools in /Users/dmitryrybakov/miniconda3/envs/pt/lib/python3.5/site-packages (3.3)

disable_coreml_rank5_mapping

@colinheye you should use minimum_ios_deployment_target option instead of disable_coreml_rank5_mapping
ref: https://github.com/onnx/onnx-coreml#parameters

@rdv0011 above example is for reference purpose. If you know the input shape for Upsample, you can provide that in custom conversion function.
We don't know input shape because, onnx shape inference is not able to infer shape beyond cat or similar layers. But in practice, you should know shape and can provide it.
I would recommend to provide this hard shape

@bhushan23
Thanks for giving an instruction how to fix it it worked for me. I have another question regarding a Tile block. It seems it expects an input to be presented in the input_tensor which is not the case for me. Do you know what to pass a values for an reps argument in the function builder.add_tile()?

disable_coreml_rank5_mapping

@colinheye you should use minimum_ios_deployment_target option instead of disable_coreml_rank5_mapping
ref: https://github.com/onnx/onnx-coreml#parameters

Thank you, this appears to fix that issue, but now I receive a KeyError
File "onnx_coreml3.py", line 39, in _convert_slice_v9
    data_shape = graph.shape_dict[node.inputs[0]]
KeyError: '88'

@bhushan23
Thanks for giving an instruction how to fix it it worked for me. I have another question regarding a Tile block. It seems it expects an input to be presented in the input_tensor which is not the case for me. Do you know what to pass a values for an reps argument in the function builder.add_tile()?

You should the number of times to replicate compile time and hence it should be part of input tensors.
Ref:
Here is proto documentation: https://github.com/apple/coremltools/blob/master/mlmodel/format/NeuralNetwork.proto#L5418
Here is builder API:  https://github.com/apple/coremltools/blob/master/coremltools/models/neural_network/builder.py#L4298

disable_coreml_rank5_mapping

@colinheye you should use minimum_ios_deployment_target option instead of disable_coreml_rank5_mapping
ref: https://github.com/onnx/onnx-coreml#parameters

Thank you @bhushan23 , this appears to fix that issue, but now I receive a KeyError. Any ideas?
File "onnx_coreml3.py", line 39, in _convert_slice_v9
    data_shape = graph.shape_dict[node.inputs[0]]
KeyError: '88'

@colinheye please refer to this comment https://github.com/onnx/onnx-coreml/issues/453#issuecomment-615485972
I posted this as an example and you can manually fix the input shape if you know it (highly recommend to encode shape you are expecting)

Closing out this issue as does have work around with custom conversion functions and will be unblocked partially after upgrading to new onnx version (hopefully with new shape inference fixes)
Please feel free to re-open or comment if you need more help on this.


Add postgresql tests

Depends on https://github.com/SUSE/BCI-dockerfile-generator/pull/278 being deployed. Before that the postgres tests will fail.

Shouldn't we try to use bind mount/volume with postgresql containers? Other than that, LGTM

Shouldn't we try to use bind mount/volume with postgresql containers? Other than that, LGTM

That is already happening automagically. The postgres container has a VOLUME /var/lib/postgresql/data directive, so a volume will be created by podman/docker automatically.

Neat! Do you think it would be worth adding more coverage like more crud operations with users, permissions, (besides tables and data)?

No, I think it would be far more important to test that all the environment variables used in the docker-entrypoint do "the right thing^TM"

I have added some additional tests, please take a look. They will be failing currently though, because the necessary fixes are not yet deployed.

@dcermak needs rebase on top of the php tests

@dirkmueller could you help me review the postgres scenario test matrix? There's still plenty of failures but I lost track whether these are supported or unsupported scenarios…


Kubevirt Build Issue

What happened:
Got the following errors by running make
ERROR: /root/go/src/kubevirt.io/kubevirt/vendor/k8s.io/client-go/informers/networking/v1beta1/BUILD.bazel:3:11: GoCompilePkg vendor/k8s.io/client-go/informers/networking/v1beta1/go_default_library.a failed: (Exit 1): builder failed: error executing command bazel-out/k8-opt-exec-2B5CBBC6/bin/external/go_sdk/builder_reset/builder compilepkg -sdk external/go_sdk -installsuffix linux_amd64 -tags selinux,selinux -src ... (remaining 61 arguments skipped)

Use --sandbox_debug to see verbose messages from the sandbox and retain the sandbox build root for debugging
compilepkg: error running subcommand PATH=/usr/bin:/bin \
GOEXPERIMENT=nocoverageredesign \
CGO_ENABLED=1 \
GOARCH=amd64 \
GOPATH= \
GOOS=linux \
GOROOT=bazel-out/k8-fastbuild-ST-b33d65c724e6/bin/external/io_bazel_rules_go/stdlib_ \
TMPDIR=/tmp \
GOTOOLCHAIN=local \
GOROOT_FINAL=GOROOT \
external/go_sdk/pkg/tool/linux_amd64/compile -p kubevirt.io/kubevirt/vendor/k8s.io/client-go/informers/networking/v1beta1 -importcfg /root/.cache/bazel/_bazel_root/6f347497f91c9a385dcd9294645b76e0/sandbox/processwrapper-sandbox/38/execroot/kubevirt/bazel-out/k8-fastbuild/bin/vendor/k8s.io/client-go/informers/networking/v1beta1/importcfg4105555531 -pack -trimpath=/root/.cache/bazel/_bazel_root/6f347497f91c9a385dcd9294645b76e0/sandbox/processwrapper-sandbox/38/execroot/kubevirt -o /root/.cache/bazel/_bazel_root/6f347497f91c9a385dcd9294645b76e0/sandbox/processwrapper-sandbox/38/execroot/kubevirt/bazel-out/k8-fastbuild/bin/vendor/k8s.io/client-go/informers/networking/v1beta1/go_default_library.x -linkobj /root/.cache/bazel/_bazel_root/6f347497f91c9a385dcd9294645b76e0/sandbox/processwrapper-sandbox/38/execroot/kubevirt/bazel-out/k8-fastbuild/bin/vendor/k8s.io/client-go/informers/networking/v1beta1/go_default_library.a -- /root/.cache/bazel/_bazel_root/6f347497f91c9a385dcd9294645b76e0/sandbox/processwrapper-sandbox/38/execroot/kubevirt/vendor/k8s.io/client-go/informers/networking/v1beta1/ingress.go /root/.cache/bazel/_bazel_root/6f347497f91c9a385dcd9294645b76e0/sandbox/processwrapper-sandbox/38/execroot/kubevirt/vendor/k8s.io/client-go/informers/networking/v1beta1/ingressclass.go /root/.cache/bazel/_bazel_root/6f347497f91c9a385dcd9294645b76e0/sandbox/processwrapper-sandbox/38/execroot/kubevirt/vendor/k8s.io/client-go/informers/networking/v1beta1/interface.go: fork/exec external/go_sdk/pkg/tool/linux_amd64/compile: resource temporarily unavailable
ERROR: /root/go/src/kubevirt.io/kubevirt/vendor/k8s.io/client-go/informers/flowcontrol/v1/BUILD.bazel:3:11: GoCompilePkg vendor/k8s.io/client-go/informers/flowcontrol/v1/go_default_library.a failed: (Exit -1): builder failed: error executing command bazel-out/k8-opt-exec-2B5CBBC6/bin/external/go_sdk/builder_reset/builder compilepkg -sdk external/go_sdk -installsuffix linux_amd64 -tags selinux,selinux -src ... (remaining 61 arguments skipped)

Use --sandbox_debug to see verbose messages from the sandbox and retain the sandbox build root for debugging
Action failed to execute: java.io.IOException: Cannot run program "/root/.cache/bazel/_bazel_root/install/0a416bf0d912af171c75b5970fc86ced/process-wrapper" (in directory "/root/.cache/bazel/_bazel_root/6f347497f91c9a385dcd9294645b76e0/sandbox/processwrapper-sandbox/42/execroot/kubevirt"): error=11, Resource temporarily unavailable
ERROR: /root/go/src/kubevirt.io/kubevirt/vendor/k8s.io/client-go/informers/BUILD.bazel:3:11 GoCompilePkg vendor/k8s.io/client-go/informers/go_default_library.a failed: (Exit 1): builder failed: error executing command bazel-out/k8-opt-exec-2B5CBBC6/bin/external/go_sdk/builder_reset/builder compilepkg -sdk external/go_sdk -installsuffix linux_amd64 -tags selinux,selinux -src ... (remaining 61 arguments skipped)

Use --sandbox_debug to see verbose messages from the sandbox and retain the sandbox build root for debugging
INFO: Elapsed time: 5.875s, Critical Path: 2.04s
INFO: 64 processes: 64 internal.
FAILED: Build did NOT complete successfully
make: *** [Makefile:18: bazel-build] Error 1

What you expected to happen:
N/A
How to reproduce it (as minimally and precisely as possible):
make
Additional context:
N/A
Environment:

KubeVirt version (use virtctl version): v1.3.0
Kubernetes version (use kubectl version): N/A
VM or VMI specifications: N/A
Cloud provider or hardware configuration: N/A
OS (e.g. from /etc/os-release): N/A
Kernel (e.g. uname -a): N/A
Install tools: N/A
Others: N/A

Are you using some custom configuration? Have you tried make clean && make?

/triage need-information

/triage needs-information

Are you using some custom configuration? Have you tried make clean && make?

No custom configuration. Yes I tried make clean && make. I also tried make -j 1 but still got the same error.

Problem is resolved. The machine I'm using has both docker and podman installed. As long as I force the building process to use docker the building process runs fine without errors.


For loop optimization

Optimizes the for loop.
Any part of the loop that is in the second part for(first;second;third) will be checked once per loop.  It gets costly to access an object property every run through unless necessary.

I compared optimized and unoptimized loops in Firefox 42.0, and there seems to be no performance difference:
https://jsperf.com/for-vs-length

http://mrale.ph/blog/2014/12/24/array-length-caching.html

In utter ignorance of any microbenchmark, I tend to use the 'new' .map, forEach, filter etc methods. Most times I ask myself: is there really need to declare and maintain a new counter variable (or even more) in order to write this? As a bonus: map, filter etc are really a bonus on readability.

Sorry this is a year old and I should've closed it I just forgot about it/let it go.
I've since repented and understand that microbenchmarks with any good JIT will not increase performance by an substantial degree, etc.


New features for Factorio 0.15 version

Would-be-nice list:

Max number of shots for each cannon
Disallow "All" GUI when lots of cannons in orbit
Migrate config.lua to in-game mod config
Disable freeplay "Launched without satellite" warning
Massively reduce "Ion Cannon Ready" spam
Add achievements
Track entities killed by blasts

Additionally: Upgraded levels of auto-targeter for use with Bob's mods.

Got most of these in -- no cheevs or limited-shots-per-cannon. Also, new auto-targeting methodology obviates the need for any dedicated stations.


touched is not correctly set on ion-item nor ion-input after blurring

Ionic version:
[ ] 1.x
[x] 2.x
[x] 3.x
I'm submitting a ...
[x] bug report
[ ] feature request
[ ] support request
Current behavior:
At the moment, when you set focus on an ion-input and remove focus, ng-touched is not set. It appears to only be set when you type however, this is the same desired behaviour for setting ng-dirty.

ng-touched should be set as soon as the user blurs an ion-input.
ng-dirty should be set when the user updates the control.

There is no way to set ng-touched alone, since typing triggers ng-dirty alongside ng-touched.
Screenshot illustrating said issue:

If I change this form to use input only controls, not ion-input controls, the correct behaviour is adhered to. Which makes me think this is something ion-input related.

The underlying FormControl also reflects the incorrect behaviour, and shows touched = false after blurring.

Expected behavior:
When you blur focus, for ng-touched to be set and for to be removed, and the FormControl to be marked as touched, as per documentation: https://angular.io/api/forms/AbstractControl#touched
Steps to reproduce:

Clone the following repo: https://github.com/kelvindart/Ionic-ng-touched-issue git clone https://github.com/kelvindart/Ionic-ng-touched-issue && cd Ionic-ng-touched-issue
npm i to install missing node modules.
ionic serve to deploy to a local server and run the example.

Related code:
I have created a repo to illustrate this issue: https://github.com/kelvindart/Ionic-ng-touched-issue. Please clone and ionic serve to see what I mean.
Other information:
N/A
Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
cli packages: (/usr/local/lib/node_modules)

    @ionic/cli-utils  : 1.9.1
    ionic (Ionic CLI) : 3.9.1

global packages:

    Cordova CLI : 6.5.0

local packages:

    @ionic/app-scripts : 2.1.4
    Cordova Platforms  : ios 4.3.1
    Ionic Framework    : ionic-angular 3.6.1

System:

    Android SDK Tools : 26.0.2
    ios-deploy        : 1.9.1
    ios-sim           : 6.0.0
    Node              : v8.2.1
    npm               : 5.4.1
    OS                : macOS Sierra
    Xcode             : Xcode 9.0 Build version 9A235

Closed - resolved in #12812.


Remove Kotlin and Gradle dependencies from jnigen / publish 0.11.0

Closes #1135.
Everything seems to still work without these dependencies and we don't have to keep updating them every time flutter updates them.

rubberstamp drive by 🚗

🚗 💨💨💨  🧍♂ <(thaaanks!)


Integrate with UseForwardedHeaders in the proxy

ASP.NET Core has a UseForwardedHeaders middleware that processes incoming x-forwarded-* headers and applies them to the request. Here are things it does:

Updates HttpContext.Connection.RemoteIp/Port, Request.Scheme/Host, etc.
Trims consumed values from the x-forwarded-* headers
Adds x-original-* headers with the consumed values

That middleware design assumes the app is the endpoint, it doesn't work well if the request is going to be forwarded since it's modified the request headers and fields. Today's proxy x-forwarded transforms would technically work since they'd take the current RemoteIp/Port/Scheme/Host fields from the requests and append them to the headers, but intermediate values from those headers would be lost.
UseForwardedHeaders probably needs some (opt-in?) design updates:

Still update HttpContext.Connection.RemoteIp/Port, Request.Scheme/Host, etc.
Store the original values in a new feature
Don't modify the request headers
Don't add the x-original request header, store this data in a feature instead.

The proxy's x-forwarded transform logic would need to check for the new feature with the original values and use those rather than the current values.

Triage: Either ASP.NET makes the middleware more proxy friendly, or fork it.
For v1 guidance might be sufficient

Related: #1331

This seems to be more of an issue with the middleware, rather than YARP. I'm wondering if this should be tracked with an issue in ASP.NET repo for .NET 7 rather than YARP.

@adityamandaleeka we will need ASP.NET 7.0 middleware work to light this up in YARP.

Thanks @karelz. @Tratcher is there an issue filed in aspnetcore for this already?

I've opened https://github.com/dotnet/aspnetcore/issues/39423. We'll want to react to those changes in YARP when they are available.


admin is lack of Shiro core and failed to run after pull

Describe the bug

Environment

Soul version(s): [e.g. 2.2.1]

Steps to reproduce
Expected behavior
admin is lack of Shiro core and failed to run after pull
Debug logs

Click to expand
replace this line with your debug logs

Please show your exception stack.


Code folding support

I was trying to implement collapsible code (or ideally folded with the first couple lines visible) in my site, but the general methods for workarounds for github don't seem to translate to Jekyll.
### R script header

  Summary

    ```r
    #!/usr/bin/env Rscript
    ## ---------------------------
    ```

Unfortunately, Jekyll uses a different parser for "GitHub Flavored Markdown" than GitHub. What works on GitHub doesn't always work with Jekyll 🙄 (and the other way round, which is why the GitHub preview doesn't render the call-outs in the theme home page).
The following markup works with Jekyll:
### R script header

  Summary
```r
#!/usr/bin/env Rscript
## ---------------------------
```


Problem adding git repo to dependencies

Hey, i've been trying to add this repo as a dependency to a project using yarn using the following command:
yarn add https://github.com/DylanVann/react-native-fast-image.git

However it fails with
git submodule update --init --recursive
fatal: Not a git repository (or any of the parent directories): .git
error Command failed with exit code 128.

Is there anyway to fix this? We are already using version 4.0.14 but want to upgrade to the latest master build as it has features that we really require for our app.

Solved.
Worked when i used:
yarn add DylanVann/react-native-fast-image.git

@Mikunj Thanks a lot!


Don't run app bundling in forked repos

Description

I'm getting notifications about app bundling workflows running on my fork of napari. I think this is happening because I/we forgot to uncomment line 17 of make_bundle.yml, which is what restricts the app bundling to only run on the main repo. The cron scheduling is triggering the workflow to run on all forks right now.
Type of change

[x] Maintenance/CI

References

https://github.community/t/do-not-run-cron-workflows-in-forks/17636/2
How has this been tested?

I don't think there's an easy way to test these kind of CI changes.
Final checklist:

[x] My PR is the minimum possible work for the desired functionality
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works

@GenevieveBuckley it looks like this PR has the full test suite running on it too (i.e. 18 checks including all python versions for all operating systems). Any idea why that might be? Doesn't seem connected to this PR

@GenevieveBuckley it looks like this PR has the full test suite running on it too (i.e. 18 checks including all python versions for all operating systems). Any idea why that might be? Doesn't seem connected to this PR

Ah looks like because this is coming from a branch on the main repo, not from your fork. Not sure if that matters, I was just wondering what was going on!!

@GenevieveBuckley it looks like this PR has the full test suite running on it too (i.e. 18 checks including all python versions for all operating systems). Any idea why that might be? Doesn't seem connected to this PR

Ah looks like because this is coming from a branch on the main repo, not from your fork. Not sure if that matters, I was just wondering what was going on!!

Oops, sorry about that. I will try to be more careful from now on.
(Also, isn't is a holiday in the US right now?


Hotswap signal

Changelog
Prep work for actual hot swap which requires switching to nginx
Issues fixed or closed

Closes #15
Fixes #19

Questions (open the PR then click the check boxes)
Did you update the documentation related to your changes?

[x] Yes
[ ] My changes were not already documented

Did you run make before committing code and opening this PR?

[x] Yes
[ ] I didn't change any code

Did you run porter create-stack and porter sync-stack to verify provisioning
works?

[x] Yes
[ ] N/A

@davepersing updated

👍


Add CondChange

aims to add an easy to to see if an expression accepts a changemode. currently untested

let me know if I'm over simplifying this but couldn't we just add ChangeMode as a enum in skript-reflect and do something like
%objects% supports %changemode% changer

if slot supports delete changer
if player's tool supports set changer

might be better to suffix modes with changer by default since idk if it'll conflict with related effects

getAcceptedChangeModes

getAcceptedChangeModes is intended for documentation purposes, not real logic


[R&D task] Deploy the server and make a CI/CD pipeline

The server should be ready to deploy by now, right?
It's time to make it happen and let it run on some cloud.
The important part is to have a CI/CD for it, so that master is always deployed as soon as it's ready.
Steps:

Find a suitable host that's always free and offers GitHub integration.
Comment here a link and explain why it's good (pros/cons etc...)
I'll create an account.
We'll establish a pipeline.

The sooner the better, if possible then this weekend.

How about creating a Github Actions pipeline that deploys the site as to github-pages?

Is it relevant for back-end?

Yeah I didn't think about that fact that it's for the backend and not the front


Add (FE)DIACOV to DA Tools

Add DIACOV to AccordDaTools.
Source code in Harmonie:src/utilities/bcov_lam but we can create scripts to produce data and plot data.
Added here @magnuslindskog and so it won't be forgotten!

@uandrae : as discussed, let's try to use GitHub issues to discuss - you can also gently nudge Magnus in this direction!

that will not be an easy task for Ulf... have a nice weekend ..

that will not be an easy task for Ulf... have a nice weekend ..

Welcome @magnuslindskog! I am all set to nudge from Ireland too!

Hej @magnuslindskog ,
Testing the version in pre-CY46h1 (same as develop) with fort.4 containing (for a 100x100xL65 domain):
&NAMDIACOV
NFLEV=65,
NSMAX=49,
NDGL=100,
PPDELTAX=15000.,
CJBTYPE='STABAL96',
/

I get an ABORT as follows:
ewhelan@realin23:gcc-8.4.1:run_diacov> ./DIACOV
 NFLEV           65
 NSMAX           49
 NDGL           100
 PPDELTAX    15000.000000000000
 CJBTYPE STABAL96
 Subroutine DIAGCOV started, Jb conf=STABAL96
 Reading covariance files, set=STABAL96
    searching parameter            4  in GSA file stabal.cv                                                                       ...
    clid =  ALADIN98
    clcom =   Balanced statistcs for a LAM, after L. Berre 1998
     ID=ALADIN98
     comment= Balanced statistcs for a LAM, after L. Berre 1998
               85    20110801           0           5
     scanning set no.           1
               1         168           0           1           0
               1          13          50           0           0           0
     scanning set no.           2
             300          45           1           2           1
              65          65           4           4           1           1
    ...found in set no.           2 . Reading the data.
 ABOR1 CALLED
 ERROR - BAD MATRIX NUMBER
 ABORT!    1 ERROR - BAD MATRIX NUMBER

Not convinced of some of the checks in src/utilities/bcov_lam/others/rdgsacov.F90
Let me know how you are getting on.

I did this like 15 years ago when Gergely thought me .. I think on issue
here is that you need to have teh cvt file that we usually through away,
not the cv file. I have not tried yet.

Commenting out the "BAD MATRIX" check on L60 helps. I get a selection of *.y and *.xy output. Until next week ...

From my old script below.
it will generate a lot of. .x.xy. files
#!/bin/ksh
namelist variable CJBTYPE :
NONSEP93 for diagnosing stabal.cvt (covs of full (=total) variables)
STABAL96 for diagnosing stabal.cv  (covs of control variables)
echo ".... catting the namelist ...."
cat > nam_stat << EOF
65
323
648
2500
NONSEP93
EOF

cat nam_stat
ls -lrt
./diacov.x

Hi,
I have renewed the Gergo's package for computation of B matrix (cy43t2_bf10). There is also new visualization which is mixture of perl, ksh and gnuplot.
I hope that it will help you with computation of B matrix on cycle 43.
Cheers, Tonda.
i did not manage to upload the tar  file to this cjhat but i do have it, magnus

Hej @magnuslindskog ,
Glad you had some success (your results attached). I did too - with different Jb files. I think my problem may have been an issue with files produced by jbconv. With "uncooked" Jb files it works fine for me too. I will work on a script to call DIACOV.
Eoin

/home/sm_mlind/hm_home/cy43mc_modarl/src/utilities/bcov_lam/programs
I see
-rw-r--r-- 1 sm_mlind sm_foup  856 Sep 28 12:51 diacov.F90
-rw-r--r-- 1 sm_mlind sm_foup 1066 Sep 28 12:51 festat.F90
But not yet nmcstat.F90, Is it in develop so that I can fetch?
Thanks,
Magnus

https://doi.org/10.1175/1520-0493(2000)128<0644:EOSAMF>2.0.CO;2
plot like in figs 4,5, and 9 are very informative

/home/sm_mlind/hm_home/cy43mc_modarl/src/utilities/bcov_lam/programs I see -rw-r--r-- 1 sm_mlind sm_foup 856 Sep 28 12:51 diacov.F90 -rw-r--r-- 1 sm_mlind sm_foup 1066 Sep 28 12:51 festat.F90 But not yet nmcstat.F90, Is it in develop so that I can fetch? Thanks, Magnus

NMCSTAT is called from festat - I should have said festat can produce the "cov" files. Add FESTAT not NMCSTAT to your defaults.mk

check if i did a pull correctly

check if i did a pull correctly

Hej @magnuslindskog, I can see you have created a fork.
The next steps are to

Create a feature branch ()
Commit you script to this branch
Create a "pull request"

Or if you think you have done this in a clone on your PC then you need to push these changes with git push origin feature/branch_name
Happy to give a quick tutorial if you have time today.

Hej @ewhelan,
Please suggest the best way where I can find the program 'diacov.F90'.
Actually, I need to plot all the statistics from only three available files stat.bal , stat.cv and stat.cvt file.
And for this I need to run 'rundiacov' , but now exe file is available.

Hi @swapanmallicksmhi  You need to use a DIADOV exe compiled by Harmonie or similar.


fix: Fix table selections when user selection in 'multi' selection mode when user selects and paginates afterwords

Description
This solves an issue when the user has a pagination with checkboxes in the table and tries to select an item and then go the next page, in this case the checkbox in the table's header will not be selected and this is not desired.
How has this been tested?
To test the problem you can go to light/table/sticky-header-scrollable-container page and try with/without this change.
[How did you test to verify your changes?]
[How can reviewers test these changes efficiently?]
[Check for unexpected visual regressions, see CONTRIBUTING.md for details.]
Documentation changes
[Do the changes include any API documentation changes?]

[ ] Yes, this change contains documentation changes.
[ ] No.

Related Links
[Attach any related links/pull request for this change]

Review checklist
The following items are to be evaluated by the author(s) and the reviewer(s).
Correctness

[ ] Changes are backward-compatible if not indicated, see CONTRIBUTING.md.
[ ] Changes do not include unsupported browser features, see CONTRIBUTING.md.
[ ] Changes were manually tested for accessibility, see accessibility guidelines.

Security

[ ] If the code handles URLs: all URLs are validated through the checkSafeUrl function.

Testing

[ ] Changes are covered with new/existing unit tests?
[ ] Changes are covered with new/existing integration tests?

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

How about adding a unit test for the new behavior?


Dynamic Label Steps

Hello,
When we add data to an barchart the data is always double. My problem is that the legend on the left side shows also the values in double but i want these values as integer. how can i assume this?
As second how to begin the left legend from 0?

When i use setResponsive(false); then it shows the values on the left legend only as integer but it gets little. by setResponsive(true); its the full size of XS_5 / LG_5

@cRUSHER-HLG do you have a screenshot to share? And some pieces of code about chart config options?

Okey works now. The configs are the default once.
The step that solved this problem for me:
Double.parseDouble(NumberFormat.getFormat("0").format(buffer[i]));

FINE! But let me ask you: when you wrote "legend" you meant "legend" or "axes"?
Because on Axes you have ticks config and callbacks to do that


Add tooltip to the explore card types

Description

Closes #863
Add a tooltip as described in the issue, and updated the CRD as well. At the same time, updated part of the code to make the badge show the correct color that they should be.

How Has This Been Tested?

Go to the explore page and hover over the type badges.
Merge criteria:

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work

found one issue, hold for a moment.

@andrewballantyne Can you review it again? I made a small mistake just now but now it's fixed.

@andrewballantyne Can you review it again? I made a small mistake just now but now it's fixed.

Good catch -- I missed that.


C2000Ware projects generation error

For types like dsp, the generated projects contains more than one source files in different directories, and they shares the same file names, which caused the confusion of project building and file change after generation

Fixed in commit 31cc82eb1ef0b7a76b3632413d0e6f9cf111bc87


Support --version and version log printout in all Meridio binaries

This is invaluable when trouble-shooting during development (to verify that you run what you think) and in customer systems (nobody knows what they run, least of all themselves).
The version should be set to the output of;
Meridio > git describe --dirty --tags
v0.3.0-5-gc9e270d-dirty

Set a version variable (string) at build-time with;
-X main.version=$(git describe --dirty --tags)"

And a --help/-h option is always a good thing in any program/script,


Select by association does not take current version into account

Describe the bug
the tool fails to select any feature that is associated with a feature that exists only in the current version
To Reproduce
Steps to reproduce the behavior:

create a feature in a named version in ArcGIS Pro
associate this feature with another that exists in Default and in the named version (in my case, a containment asso)
validate topology and save (at this point, the associationstatus of the new feature is set to Container and the other one is set to Content)
execute the "select by association" tool -> nothing is returned

Expected behavior
The associated feature should be selected
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows 10
pro : 3.3.2
UDMS :  3.1.2 and 3.3.2
AGE : 11.3

actually, I think the code should work without changes.  How are you filling out the tool?
We get the version info from the Utility Network layer used in the input.  Are you picking the layer in the GP tool or browsing to the path of the UN in the service?

Ok, tracked this down to a core bug, that ChangeVersion is not working, looks like it is slated for 3.3.3 patch.

BUG-000156453

Hello Mike,
I also took a look at the code and saw that the version name was taken from the UN layer (in my cas it is the TOC layer selected in the drop down list on the toolbox, and not the complete path to the UN).
Note that we are using the tool in a custom .pyt toolbox that we developped and that will call the UDMS toolbox and pass the TOC UN's name to the 'select by association' input parameter. It works as expected in Default.
Thanks for your work on this. When you say it will be adressed in a 3.3.3 release, are you talknig about a UDMS version or an ArcGIS Pro patch ?

ArcGIS Pro patch 3.3.3.  I tested with 3.4 and it is fixed in there.

Hope it will be released soon !

I did not find a date, but I would imagine it would be within a month.  I pushed a change that has logic to ensure the version change and raise an error when it does not.


Assign extension validation files to the GDExtension and .NET teams in CODEOWNERS

As discussed in Rocket Chat.
I originally also wanted to assign the files that end with _extension to the GDExtension team, but I think that would override their current owner unless we explicitly add both which means we can't use a simple regex like *_extension.* but instead would have to add one for each area (e.g.: /modules/openxr/**/*_extension.*, /servers/text/**/*_extension.*).

Thanks!


After packaging, connect can not get dispatch, no problem in the test environment

What happens?
After packaging, connect can not get dispatch, no problem in the test environment.After packaging, call this.props.dipatch APP to crash.
Mini Showcase Repository(REQUIRED)
no
How To Reproduce
Steps to reproduce the behavior:
1.After packaging, connect can not get dispatch, no problem in the test environment
2.After packaging, call this.props.dipatch APP to crash. delete this.props.dipatch APP do not flash back and<EMAIL_ADDRESS>too.
Expected behavior

APP do not flash back

Context

Dva Version: "dva": "^2.4.1",
"dva-loading": "^3.0.20",
"dva-no-router": "^1.2.1",
"react-native": "^0.62.2",

Node Version: 12.12.0

Platform: Android

Does not support yield

If you don't use yield, it won't flash back after packaged into apk. Does it not support Generator?(如果不使用yield, Android打包后就不会闪退)

Promise cannot be used even after it is packaged into an apk,
Debug mode can get data. Why?

I found the reason because of "console.assert" reason, because the fetch request triggered the console. assert, to fix it, add
if (!__DEV__) {
   global.console = {
     assert: () => {}
   }
}

Or comment the console.assert of node-modules => event-target-shim => event-target-shim.js.


UI update

@shubhamk008 Could you provide the rgb data for the colors?

I have given edit access @.*** might be able to click and see the colors now. Let me know how it goes.
Regards,
Shubham Kumar
On Wednesday, 1 February, 2023 at 03:31:11 pm GMT-6, HanBoyou<EMAIL_ADDRESS>wrote:

@shubhamk008 Could you provide the rgb data for the colors?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Hi @HanBoyou, how are we doing with the redesign?

You could checkout the branch itself. There has been some progress and currently the problem's at the splitter button.


Add purge to ProducerInterface

Initial draft for resolving #14.
Depends on / based off #15.
Adds the purge method to ProductInterface.

looks good otherwise (not tested).


Readme fix

The URLs for the twitter examples in the README no longer work, so this change updates them.  I've verified the new example for the CLI personally, but haven't for twitter sign in (the changes do match Twitter's documentation, though).

Fixed in 82dd2cdd4954cd7b8983d5d64c0dfd9072bf4650


[improvement](jdbc catalog) when lower_case_table_names of jdbc catalog properties is set to true, use the real table name to query the jdbc data source

Proposed changes
Issue Number: close #xxx

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

run buildall

run buildall

run buildall

run buildall

run buildall

run buildall


Badge for Build Description

We use the set build description to list the version of the component we just built.  We can utilize that and make a badge so the repository can show the exact snapshot version of what's the latest available.

do you want this in before we deploy ?


BUG : CHATBOT

Describe the bug

Hello All,
There is a Chatbot in the webpage & the chatbot is not working upto the mark.
Expected behavior

I just want the chatbot functionable . Make the chatbot functionable even can make it more functionable by adding more elements
Screenshots (Mandatory)

Additional Context (Mandatory )
Those whoever comment first with JWOC mentioned in it  I will assign it to HER/HIM
Are you contributing under any open-source program ?

JWOC'24

i would like to work under jwoc @apu52 kindly assign

@Anushkabh You are assigned. You have 2-3 days to submit your PR.
Happy coding!!!!!!!!!

Hey guys :)
I'm the maker of Conferbot, I see that you have run out of your chat quota for the month.
Since you guys are students, I'll be happy to increase your chat quota to 1500 chats/month for free for 2 months.
A little tip to help you manage better: Try turning off the Bot Auto Open feature. You'll find it under Customizations -> Advanced. This should help prevent using up your chat quota too quickly.
If you'd like to take up this offer, just let me know.
Good luck!

@dPacc Thank you for your concern & the tips  . As now for somedays this repo is under a open source programme JWOC , & this issue is already assigned so , @Anushkabh I think you can take the tips mentioned above

@apu52 yes I have taken into account the tips mentioned .. working on that will push it by eod

@apu52 kindly review the pr #231  and merge if satisfied.


feature: setAdVolume

Added the option to control ad volume while the ad is playing

Could you give an example usage for this? This new method just updates the IMA ad volume but doesn't touch the UI, so calling this method would cause the UI to fall out of sync with the actual volume, no?

Closing for lack of response, but if you'd still like for this to be included feel free to re-open :)


Updated embedded images with URI to raw images

Seems like there are some conflicts with the branch.

Conflicts resolved


Display dates in local format

Thanks for this app.  Would it be possible to display dates for articles in the local format or at least ISO standard.  For example British day/month/year rather than US month/day/year.  Using the phones setting would be perfect.

Hello, thanks for the feedback. Changing to ISO is quite easy. I'll check what it would imply to use the phone settings.
For reference:
https://api.flutter.dev/flutter/intl/DateFormat-class.html
https://stackoverflow.com/questions/50923906/how-to-get-timezone-language-and-county-id-in-flutter-by-the-location-of-device

Sounds good. Did you decide whether to include this?

I just deployed a new version including this change :-)


Add support for WebSockets

It's my understanding that WebSockets are initiated by sending the correct request headers to the server at which time the server responds with headers that indicate that the WebSocket can be opened and the connection remains open. This so that HTTP port 80 can also be used for WebSockets.
Seems to me that support for this should be added to Iron. What say ye?

There's already a hyper-based websocket implementation (https://github.com/cyderize/rust-websocket) so one approach would be to merge this code into iron. Unfortunately, as is, it needs to run on a separate port using a separate hyper instance.

Related: https://github.com/hyperium/hyper/issues/521

FYI: That related issue has now been closed.
Also related: https://github.com/cyderize/rust-websocket/blob/master/examples/hyper.rs

Is anyone actively working on that?

The rust-websocket seems to have stopped development. Is there anyone looking into it again?

There's always https://github.com/housleyjk/ws-rs
But there are recent commits to rust-websocket also: https://github.com/cyderize/rust-websocket/commits/master

I was hoping for a direct Integration or a way to integrate one directly so it would use only one port

Any plans with this?

another year has passed, is this still on the roadmap?

Personally I think it's pretty difficult to fit websocket into Iron's request-middleware-response model. I haven't seen elegant abstraction in other languages for this.


Handle operations with ByRef-like types gracefully in PowerShell

PR Summary
Fix #7301
Handle operations with ByRef-like types gracefully in PowerShell.
ByRef-like types are supposed to be used on stack only, so we need to fail gracefully when accessing properties, calling methods, or creating objects related to ByRef-like types.
Found one issue when working on this fix, tracked by #7534
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Change is not breaking
[ ] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

User-facing changes

[ ] Not Applicable
OR
[x] User-facing Documentation needed

[x] Issue filed - Issue link:

Testing - New and feature

[ ] Not Applicable or can only be tested interactively
OR
[x] Make sure you've added a new test if existing tests do not effectively test the code changed

[x] Add [feature] if the change is significant or affects feature tests

@iSazonov Sorry for the late response (I'm currently on vacation :)).
Yes, I thought about implicit cast to/from Span/ReadOnlySpan. The implicit cast doesn't help regular conversion operation in PowerShell for the same root reason -- the target ByRef-like type cannot be boxed.
But when calling methods via Expression with jitting, Expression.Convert can be used to implicit cast an argument to the Span types before passing to the call (see an example below).
var arg = @"e:\abc\def";
var method = typeof(Path).GetMethod(nameof(Path.IsPathRooted), new Type[] { typeof(ReadOnlySpan<char>) });

var body = Expression.Call(method, Expression.Convert(Expression.Constant(arg), typeof(ReadOnlySpan<char>)));
var func = Expression.Lambda<Func<bool>>(body, null).Compile();
var rest = func();
Console.WriteLine(rest);

> True

PowerShell evaluates with interpretation by default, which bacially translating Expression tree to pre-defined C# code, so the Expression.Convert might not work like when Expression tree is jitted (haven't looked into it, will investigate that). If it works with interpreter too (or we can update the interpreter to make it work), we will need to update the method resolution method. We currently use the same "figuring-out-conversion" method for regular conversion in powershell as well as when resolving the best matching method. The implicit cast for ByRef-like target types should continue to be "no-conversion" for regular conversion operation, but acceptable for method resolution.
Anyway, this should be done in a separate PR. I will open an issue to track it.

@daxian-dbw Thanks for depth comment! I think support dynamically converting Span based methods is important because main direction in .Net Core is to implement methods with Span parameters and other methods only wrap them. Later we can get APIs without the wrapped methods in CoreFX or third party libraries and PowerShell will not be able tocall them. So my fisrt think was that we could generate the wrap methods in runtime.

@lzybkr and @iSazonov Your comments have been addressed. Please take another look when you have time. Thanks!

Will it resolve:
 [System.Text.Encoding]::GetEncoding(0)
format-default : Cannot create boxed ByRef-like values.
+ CategoryInfo          : NotSpecified: (:) [format-default], InvalidProgramException
+ FullyQualifiedErrorId : System.InvalidProgramException,Microsoft.PowerShell.Commands.FormatDefaultCommand

I get this on 6.1.0 RC1.

Here is what it looks with changes from this PR:
PS:1> [System.Text.Encoding]::GetEncoding(0)

Preamble          :
BodyName          :
EncodingName      : Western European (Windows)
HeaderName        :
WebName           : windows-1252
WindowsCodePage   :
IsBrowserDisplay  :
IsBrowserSave     :
IsMailNewsDisplay :
IsMailNewsSave    :
IsSingleByte      : True
EncoderFallback   : System.Text.InternalEncoderBestFitFallback
DecoderFallback   : System.Text.InternalDecoderBestFitFallback
IsReadOnly        : True
CodePage          : 1252

Instance property access doesn't throw in powershell, even in strict mode. So accessing Preamble in the formatting code won't break it like it did previously.

BTW, the AppVeyor CI PR failed with the following error
TEST FAILURES
Description: Enable-PSSession Cmdlet creates a default PSSession configuration untied to a specific PowerShell version.
Name:        Validate Enable-PSSession Cmdlet.Enable-PSSession Cmdlet creates a default PSSession configuration untied to a specific PowerShell version.
message:
Expected a value, but got $null or empty.
stack-trace:
at <ScriptBlock>, C:\projects\powershell\test\powershell\Modules\Microsoft.PowerShell.Core\PSSessionConfiguration.Tests.ps1: line 855
855:             $matchedEndpoint | Should -Not -BeNullOrEmpty
1 tests in test/powershell failed

The same failure is happening in the daily build, so it's not caused by chnages in this PR.


Pod Predicate NodeAffinity failed error seen on default-http-backend pod

What kind of request is this (question/bug/enhancement/feature request): bug
Steps to reproduce (least amount of steps as possible):

Deploy an azure v1.19.3 cluster in 2.5.1
Run pre upgrade checks
Upgrade rancher to 2.5.2-rc5
Run post upgrade checks.
Noticed Pod Predicate NodeAffinity failed error seen on default-http-backend pod

Other details that may be helpful:
This is seen on one of the clusters in the setups
Other clusters in the setup:
AKS - 1.17
EKS - 1.17
Windows 1.16
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): 2.5.2-rc5
Installation option (single install/HA):

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): Azure node driver

Kubernetes version (use kubectl version):

1.19.3

SURE-3728

Does not reproduce all the time. Potential upstream issue: https://github.com/rancher/rke2/issues/127#issuecomment-699254134. Moving out of 2.5.2.

See this issue on a HA local cluster on 2.5-head - commit id: 9ee39b9 on coredns pod when the nodes used are flatlinux stable-2605.6.0 instances with t3a.medium.
The cluster.yml file has these network settings

network:
  plugin: calico
  options:
    calico_flex_volume_plugin_dir: /opt/kubernetes/kubelet-plugins/volume/exec/nodeagent~uds
    flannel_backend_type: vxlan

services:
  kube-controller:
    extra_args:
      flex-volume-plugin-dir: /opt/kubernetes/kubelet-plugins/volume/exec/

This seems to be fixed in Kubernetes 1.21, relevant PR https://github.com/kubernetes/kubernetes/pull/99336. QA will test on that version and confirm if it's still an issue.

Setup For Reproduction

rancher Version: v2.6.2
Ha Deploy

Downstream Clusters

azure node drivers
rke1
k8s versions: v1.18.20-rancher1-3, v1.19-16-rancher1-3, v1.20.15-rancher1-1, v1.21.9-rancher1-1.
Nginx Default Backend within advanced options

Steps For Reproduction Attempts

Deploy clusters using the Azure node driver, with the above versions of k8s and Nginx Default Backend enabled.
Upgrade rancher version from v2.6.2 to 2.6.3

Results
There were no node affinity issues.
Setup for Validation

rancher Version: v2.6.3 upgrade to 2.6-head(
Ha Deploy

Downstream Clusters

azure node drivers
rke1
k8s versions:  v1.19-16-rancher1-3, v1.20.15-rancher1-1, v1.21.9-rancher1-1.
Nginx Default Backend within advanced options

Steps For Validation Attempts

Deploy clusters using the Azure node driver, with the above versions of k8s and Nginx Default Backend enabled.
Upgrade rancher version from v2.6.3 to 2.6-head(e55a04c)

Results
The clusters came up fine with no errors relating to Node Affinity


Fix DNS and/or proxy

Issue 1
AJAX Calls don't work in the current DNS settings.
Example: http://materialapex.com/pls/apex/f?p=12192:200:::NO
So going directly to https://apex.oracle.com/pls/apex/f?p=12192:200 works better for now.
Issue 2
I would like these URLs

http://materialapex.com
http://www.materialapex.com
https://materialapex.com
https://www.materialapex.com

to ALWAYS redirect to:

https://www.materialapex.com

cc @fuzziebrain

I'll be working on the CORS issue here: https://github.com/fuzziebrain/competition-odtug/issues/2


Dynamic light blockiness (tile edges visible)

Using the tiled renderer, the area covered by a dynamic light can have visible, blocky edges. The issue can be observed by firing a single pulse rifle shot. Here's a screenshot by illwieckz:
The tiled lighting works in two phases:

In the lighttile shader, compute the set of lights which is believed to affect each tile. The assumption is that tiles sufficiently far away should be close enough to completely dark not to matter.
When rendering a surface, apply each light in the list for the relevant tile.

It seems that the assumption in (1) is wrong and some of the tiles considered to be not affected actually would have had a significant amount of light. Two obvious approaches to fix this:
(a) Change the calculation of which tiles are affected to rule in more tiles
(b) Change the falloff curve of brightness vs. distance so that the brightness is actually close enough to 0 at the edges

Would be really amazing to fix this!


[feature request] Extend option for ExperienceReplay

I am using the parts of the repository such as the ER for my own algorithms and research. A pattern that is often encountered is adding experience (SARS'D) from batched environments and in consequence need to be separated and stored in the ER, thus adding an option to extend (like in python's lists) would be quite helpful.
Something along these lines:
buffer = ExperienceBuffer(2**20)
state = env.reset() #obs now has NxM(xZ) shape
action = agent(state)
nstate, reward, done, _ = env.step(action)
buffer.extend(state, action, reward, nstate, done)

I would go about implementing this by considering the shape of done to infer the number of individual observations, then either reusing the add method or adding the code to avoid the extra calls.

Just to recap I understood correctly: Here state, action etc are lists of states/actions/etc, and with buffer.extend we add all of these experiences to the buffer with one call, rather than looping over and doing buffer.add?
Sounds like an useful function, and given that Python lists already use this term extend for same purpose it should be clean and easy to understand too :). If this is describes the feature you wanted to add, feel free to start working on a PR!

Yes this is what I would like to do. It saves a few lines of code that probably exist on everybody's projects and saves a quite a few functioncalls.


failed: Error during WebSocket handshake: Unexpected response code: 403

If my project link is http://<IP_ADDRESS>/notification or http://localhost/notification then how can I use websocket? It return some errors into browser console those are
WebSocket connection to 'ws://<IP_ADDRESS>/notification:6001/app/my_app_key?protocol=7&client=js&version=5.0.1&flash=false' failed: Error during WebSocket handshake: Unexpected response code: 403
WebSocket connection to 'wss://<IP_ADDRESS>/notification:443/app/my_app_key?protocol=7&client=js&version=5.0.1&flash=false' failed: Error in connection establishment: net::ERR_CERT_AUTHORITY_INVALID
I want to run the project on xampp not by php artisan serve command then how can I solve it.

@uncannyMBM i think it should be  ws://<IP_ADDRESS>:6001/notification

@uncannyMBM Bump

I'm having the same issue on Ubuntu 20.04, PHP 7.4, Laravel 8.7.1, Websockets 1.8.
I haven't tried pushing anything to it yet as I can't even get it connect.
This is running through Laravel Forge where I opened port 6500 for any connection, I started a daemon on the server to run websockets with --port=6500
I copied the SSL paths directly from the Nginx serve block, so those are correct.
I even added wss to the dashboard template to make sure it could be accessed.
I do have some allowedOrigins ... is it possible I'm getting the 403 from that?
'allowed_origins' => [
    '*.example.org',
    'local.test'
],

Is this bad formatting for these?
I can't imagine what else can be stopping this from running.
Here is my Nginx location block:
location /app {
    proxy_pass             http://<IP_ADDRESS>:6500;
    proxy_read_timeout     60;
    proxy_connect_timeout  60;
    proxy_redirect         off;

    # Allow the use of websockets
    proxy_http_version 1.1;
    proxy_set_header Upgrade $http_upgrade;
    proxy_set_header Connection 'upgrade';
    proxy_set_header Host $host;
    proxy_cache_bypass $http_upgrade;
}

I also tried setting the proxypass to https but got the same results.
If someone can please help.
Thanks

That was indeed the issue ...
I removed the wildcard and listed all the domains including the socket server subdomain and now she fires like a champ.
Hopefully this will help someone.

PHP 7.4 won't be a problem in this: https://github.com/beyondcode/laravel-websockets/issues/510


WSL2 and SQL Server

Describe the solution you'd like
Any idea when Windows Subsystem for Linux for Windows 10 will support SQL Server as an installation target?

This is a question for reddit, twitter, etc


gptp release

Now that the gptp is separated out of the OpenAvnu code, can we have a separate release for the gptp starting with the versioning v1.0

Sure. I assume we/you would create release notes as well? I'm open to a pull request with a release tag and some sort of history document. Please add other comments.

@andrew-elder I can create the required notes and history document. Let me know, if you want me to follow some reference format. But for release tag as per my knowledge, we cannot raise a PR for tags.

Release notes.txt
@andrew-elder Let me know if the attached release notes looks ok to you.

history.txt
I have generated a history of the PR's for gptp from earlier project.
This can be included in the release notes or as separate history document.

Thanks. Should the file types be .md instead of .txt? They are markdown formatted, right?
Also, are you planning to put a version number in the Release notes.txt file?

@andrew-elder
Forgot to mention, yes it should be .md but github does not support attaching .md files in the comment, hence had renamed to .txt
For version can we start with v1.0? I have added the same in the attachment.
Release notes.txt

@andrew-elder Can you please make a first official release? Let me know if you need any other details.

Please do a pull request with history.md and release_notes.md in them. That way we can tag the notes files as well.

@andrew-elder Now that the documents are merged, can you please make the first official release?

Done. I tagged v1.0.0.


asDecimal() return wrong results

Different results return from intl and  number_format()
What steps will reproduce the problem?
Yii::$app->formatter->asDecimal('18446744073709551615');
What is the expected result?
When i have used php function number_format() that time return below output. so i think is a expected or not ?.  not confirm
Results : 18,446,744,073,709,551,616
What do you get instead?
// convert using intl
Results :  18,446,744,073,709,600,000
Additional info

Q
A

Yii version
2.0.12

PHP version
7.1

Operating system
Ubuntu 16.04

Verified

@SilverFire @samdark would not be asDecimal completely rewritten without using native functions like number_format() and intl extension?
Test
$value = '123456789012345678901234567890';
$this->assertSame('123,456,789,012,345,678,901,234,567,890', $this->formatter->asDecimal($value, 0));

Output
Failed asserting that two strings are identical.
Expected :123,456,789,012,345,678,901,234,567,890
Actual   :123,456,789,012,345,677,877,719,597,056

PHP internal casting big integers to float leads to badly mistaken result.

Probably a good idea.


Make BusMem query params work like Circle ones

What's this PR do?

Makes ?level=<level-name> work on the BusMem page, just like it does on Circle
Consolidates the two FormBuckets.vue components into one

Why are we doing this? How does it help us?
I'm trying to get this app's front end as clean as possible before it gets a major workout during UMP development. That means consolidating duplicate components and standardizing as much as possible.
This will also allow our revenue people to send potential customers the BusMem page with a pre-selected level. I also think it makes sense to just use a single level= parameter as opposed to separate ones for installment period, amount, etc., given that subsidiary info can be derived from a particular level constant.
Cheers to Val for getting this page off to a wonderful start!
How should this be manually tested?

If you're not testing on the Heroku branch, do a yarn then a make restart
Visit /business?level=<bigTexYearly|bigTexMonthly|loneStarYearly|loneStarMonthly|hatsOffYearly|hatsOffMonthly> and ensure the proper bucket radio button is selected on page load. Make a few test donations and ensure the info comes thru properly in #stripe.
Make a few test donations on /circle just for good measure. Ensure the info comes thru properly in #stripe.

How should this change be communicated to end users?
I'll let April know and then she can decide.
Are there any smells or added technical debt to note?
No.
What are the relevant tickets?
https://3.basecamp.com/3098728/buckets/736178/todos/1493933269
Have you done the following, if applicable:

[ ] Added automated tests? (someday)
[ ] Tested manually on mobile? (NA)
[ ] Checked for performance implications? (NA)
[ ] Checked for security implications? (we're now only processing a single query param, and only processing it if it contains a valid level value, so I feel good about things)
[ ] Updated the documentation/wiki? (NA)

I've been thinking about something similar. Let me know if it should go into this PR or not. Now that we have different endpoints for the different types (Biz, Circle, etc) the only parameter that needs to be passed to the backend is the installment period. Installments and open-ended status don't need to be sent to the backend at all. Hopefully, that will simplify parts of the frontend. What do you think?

@x110dc I think it's a great idea. I'd prefer it not go in this PR just for scope/number of files changed purposes, but I'll be happy to make the needed front-end changes to accommodate.

@x110dc I'd like to get this PR merged before we do this. I requested a review; please look at your convenience -- no big rush.


Support headless mode

I haven't tested this but it shouldn't make any difference. It is a prerequisite for running it as a daemon.

Is the idea to run the headless as the background process?
I'm thinking what you might have in mind because I assume selenium needs the browser in the foreground for this to work, correct?

Nope. The browser doesn't need be in foreground because it is not the mouse or keyboard doing the work, Chrome driver is.


Compile failed, "module Mongo.Pool is not loaded and could not be found"

I tried installing mongodb_ecto, here is the relevant mix.exs:
# Configuration for the OTP application
#
# Type "mix help compile.app" for more information
def application do
  [applications: [:logger, :mongodb_ecto, :ecto],
   mod: {MyApp, []}]
end

# Dependencies can be Hex packages:
#
#   {:mydep, "~> 0.3.0"}
#
# Or git/path repositories:
#
#   {:mydep, git: "https://github.com/elixir-lang/mydep.git", tag: "0.1.0"}
#
# Type "mix help deps" for more examples and options
defp deps do
  [{:mongodb_ecto, "~> 0.1"}]
end

Then I ran mix deps.get and all was fine.
Running mix compile yields this error
== Compilation error on file lib/mongo_ecto/admin_pool.ex ==
** (CompileError) lib/mongo_ecto/admin_pool.ex:4: module Mongo.Pool is not loaded and could not be found
    (elixir) expanding macro: Kernel.use/2
    lib/mongo_ecto/admin_pool.ex:4: Mongo.Ecto.AdminPool (module)
    (elixir) lib/kernel/parallel_compiler.ex:116: anonymous fn/4 in Kernel.ParallelCompiler.spawn_compilers/1

could not compile dependency :mongodb_ecto, "mix compile" failed. You can recompile this dependency with "mix deps.compile mongodb_ecto", update it with "mix deps.update mongodb_ecto" or clean it with "mix deps.clean mongodb_ecto"

I googled, but nothing showed up. Since ecto is compiling fine, I suppose there is a problem with mongodb_ecto. Here is the content of mix.lock to see the versions:
%{"connection": {:hex, :connection, "1.0.4", "a1cae72211f0eef17705aaededacac3eb30e6625b04a6117c1b2db6ace7d5976", [:mix], []},
  "db_connection": {:hex, :db_connection, "1.1.0", "b2b88db6d7d12f99997b584d09fad98e560b817a20dab6a526830e339f54cdb3", [:mix], [{:connection, "~> 1.0.2", [hex: :connection, optional: false]}, {:poolboy, "~> 1.5", [hex: :poolboy, optional: true]}, {:sbroker, "~> 1.0", [hex: :sbroker, optional: true]}]},
  "decimal": {:hex, :decimal, "1.3.1", "157b3cedb2bfcb5359372a7766dd7a41091ad34578296e951f58a946fcab49c6", [:mix], []},
  "ecto": {:hex, :ecto, "1.0.7", "d8deca5d5a03e7c9ed4821e6f1f85dad736aca960785f154e3a8c1bee1aeaa13", [:mix], [{:decimal, "~> 1.0", [hex: :decimal, optional: false]}, {:mariaex, "~> 0.4.1", [hex: :mariaex, optional: true]}, {:poison, "~> 1.0", [hex: :poison, optional: true]}, {:poolboy, "~> 1.4", [hex: :poolboy, optional: false]}, {:postgrex, "~> 0.9.1", [hex: :postgrex, optional: true]}, {:sbroker, "~> 0.7", [hex: :sbroker, optional: true]}]},
  "mongodb": {:hex, :mongodb, "0.2.0", "95f85074534bb0a65c4983995903a458e36d0b096f62c1bbdd56bc196f48cf80", [:mix], [{:connection, "~> 1.0", [hex: :connection, optional: false]}, {:db_connection, "~> 1.0", [hex: :db_connection, optional: false]}]},
  "mongodb_ecto": {:hex, :mongodb_ecto, "0.1.4", "d8eebaed05851802b0e7cd53dcfbb4b62a5819e8acd7c584b8eaa3c8ead43c2c", [:mix], [{:ecto, "~> 1.0.0", [hex: :ecto, optional: false]}, {:mongodb, "~> 0.1", [hex: :mongodb, optional: false]}]},
  "poolboy": {:hex, :poolboy, "1.5.1", "6b46163901cfd0a1b43d692657ed9d7e599853b3b21b95ae5ae0a777cf9b6ca8", [:rebar], []}}

I see nothing wrong in there.
And here is elixir version:
Erlang/OTP 19 [erts-8.0.3] [source] [64-bit] [smp:4:4] [async-threads:10] [hipe] [kernel-poll:false]

Elixir 1.3.2

Since I found nothing by googling, I assume I am doing something wrong, though it seems not. I would add bug label if it shows up to be a problem with mongodb_ecto. I already googled to find what is Mongo.Pool but again nothing showed up. I would be more than happy to fix this problem, if I had a clue on the cause.

I think I have found the problem. This line in mix.lock
  "mongodb": {:hex, :mongodb, "0.2.0", "95f85074534bb0a65c4983995903a458e36d0b096f62c1bbdd56bc196f48cf80", [:mix], [{:connection, "~> 1.0", [hex: :connection, optional: false]}, {:db_connection, "~> 1.0", [hex: :db_connection, optional: false]}]},

caused the problem. Adding {:mongodb, "~> 0.1.1"} to mix.exs fixed the problem.
I think the dependency of mongodb_ecto should be changed to ~> 0.1.0, since it won't work with 0.2 branch of mongodb.

Seems to be fixed in master, commit 6d47b6ee3c7fb6a705a298eb286de97f647cc970, though no notification here. I was just going to create PR that I saw it was fixed.
Thanks anyway, I'm closing the issue.

I've just released 0.1.5, which has a stricter dependency on mongodb, and should fix this issue.