Potential context setup enhancement for TaskMockRunner

I have been working on creating a pipeline task for terraform. While doing so I was trying to figure a good way to make certain parts of my test reusable from test to test. Some examples of what I was trying to share are the answers and input setup. For most tests I need a which answer for terraform and I want a checkPath of true. Additionally, certain exec answers are needed on almost all tests. For example terraform version as this is run to verify terraform is installed before each command is run.
So what i came up with is still setting up mocha spec's as done in the documented examples
describe('terraform init', function(){
    it('should succeed', function(){
        let taskRunnerPath = require.resolve('./init-success');
        let testRunner = new MockTestRunner(taskRunnerPath);
        testRunner.run();
        assert.equal(testRunner.succeeded, true, 'should have succeeded');
        assert.equal(testRunner.errorIssues.length, 0, "should have no errors");
    });
});

But the module that contains the setup and execution of TaskMockRunner looks like this..
let scenario = new TaskScenario()
    .givenInput(new TerraformCommandAndWorkingDirectory("init"))

    .givenAnswer(new TerraformExists())
    .andAnswer((answers) => new TerraformCommandIsSuccessful(answers))
    .andAnswer((answers) => new TerraformCommandWithVarsFileAsWorkingDirFails(answers))
    .run();

The classes being instantiated are decorators the author of the task can create to define a step that is used in multiple scenarios.
Is this something that you feel would be useful as a contribution to this library? If so i can submit a PR to have this framework added to the testing framework.
More examples here...
https://github.com/charleszipp/azure-pipelines-tasks-terraform/tree/master/TerraformCLI/src/tests

I'm not sure I fully understand what you're doing here. Is it basically a way to set a bunch of inputs in the same way? Couldn't you just do something like:
input_file:
export inputs = {key1: val1, key2: val2...}

TaskMockRunner file:
const inputs = require('input_file').inputs;
for (key in inputs) {
   taskRunner.setInput(key, inputs[key]);
}

Maybe I'm missing what's going on here, but this seems overdone.

@damccorm
If you look at where my tests started and where they ended up after this, you will notice the amount of repetition in not only the boilerplate code but also, references to input names, file paths, and modules across the tests. I was looking for an example of a way of not doing this and I was not able to find one. If you look through the azure-pipelines-tasks github repo, there isnt any consistency there either about how they promote some reuse across different tests. Some do it using environment vars (dotnetcore), some just repeat most of the boilerplate (npm).
This is even more challenging in that the pattern for testing is completely different than the BDD style testing that frameworks such as Mocha and Jasmine promote is not usable in these tasks. I cant organize my tests into describes such that the same preconditions are reused by nested describe and it functions.
I have posted examples of what my scenario files looked like before implementing this. The amount of repetition, especially in the boilerplate setup and key string references has been dramatically reduced. Additionally, the step class describe the intent for which they should be used.
Feel free to close the issue down if you still feel this is not useful. I can make this available in a separate repo & npm package
init-success
import ma = require('azure-pipelines-task-lib/mock-answer');
import { TaskMockRunner } from 'azure-pipelines-task-lib/mock-run';
import path from 'path';

var taskPath = require.resolve('./../index');
let taskRunner: TaskMockRunner = new TaskMockRunner(taskPath);
let workingDirectory = path.resolve("./../TerraformTemplates/sample");
taskRunner.setInput("command", "init");

// azure pipeline will set this to have the value of the working directory even if no value was provided
taskRunner.setInput("varsFile", workingDirectory);
taskRunner.setInput("workingDirectory", workingDirectory);

let a: ma.TaskLibAnswers = {
    which : {
        "terraform" : "terraform"
    },
    checkPath : {
        "terraform" : true
    },
    exec : {
        "terraform version" : <ma.TaskLibAnswerExecResult>{
            code : 0,
            stdout : "version successful"
        },
        "terraform init" : <ma.TaskLibAnswerExecResult>{
            code : 0,
            stdout : "init successful"
        }
    }
};
a.exec = a.exec || {};
a.exec[`terraform init -var-file=${workingDirectory}`] = <ma.TaskLibAnswerExecResult>{
    code : 1,
    stdout : "init failed. working dir provided to -var-file"
}
a.exec[`terraform init --var-file=${workingDirectory}`] = <ma.TaskLibAnswerExecResult>{
    code : 1,
    stdout : "init failed. working dir provided to --var-file"
}
taskRunner.setAnswers(a);
taskRunner.run();

validate-success
import ma = require('azure-pipelines-task-lib/mock-answer');
import { TaskMockRunner } from 'azure-pipelines-task-lib/mock-run';
import path from 'path';

var taskPath = require.resolve('./../index');
let taskRunner: TaskMockRunner = new TaskMockRunner(taskPath);
let workingDirectory = path.resolve("./../TerraformTemplates/sample");

taskRunner.setInput("command", "validate");
taskRunner.setInput("varsFile", workingDirectory);
taskRunner.setInput("workingDirectory", workingDirectory);

let a: ma.TaskLibAnswers = {
    which : {
        "terraform" : "terraform"
    },
    checkPath : {
        "terraform" : true
    },
    exec : {
        "terraform version" : <ma.TaskLibAnswerExecResult>{
            code : 0,
            stdout : "version successful"
        },
        "terraform validate" : <ma.TaskLibAnswerExecResult>{
            code : 0,
            stdout : "validate successful"
        }
    }
};
a.exec = a.exec || {};
a.exec[`terraform validate -var-file=${workingDirectory}`] = <ma.TaskLibAnswerExecResult>{
    code : 1,
    stdout : "validate failed. working directory provided to -var-file arg"
}
taskRunner.setAnswers(a);
taskRunner.run();

I guess I just don't fully understand why you can't do something similar to what I suggested in my previous comment, e.g.
shared.ts
export sharedInputs = {"command": "validate", "varsFile": workingDirectory};
export sharedAnswers = {which: ....}

init-success.ts
import ma = require('azure-pipelines-task-lib/mock-answer');
import { TaskMockRunner } from 'azure-pipelines-task-lib/mock-run';
import shared = require('./shared');

var taskPath = require.resolve('./../index');
let taskRunner: TaskMockRunner = new TaskMockRunner(taskPath);

const sharedInputs = shared.sharedInputs;
for (i in sharedInputs) {
   taskRunner.setInput(i, sharedInputs[i]);
}
let a = shared.sharedAnswers;
a.exec['terraform validate'] = {…}
a.exec[`terraform init -var-file=${workingDirectory}`] = {…}
a.exec[`terraform init --var-file=${workingDirectory}`] = {…}
taskRunner.setAnswers(a);
taskRunner.run();

validate-success.ts
import ma = require('azure-pipelines-task-lib/mock-answer');
import { TaskMockRunner } from 'azure-pipelines-task-lib/mock-run';
import shared = require('./shared');

var taskPath = require.resolve('./../index');
let taskRunner: TaskMockRunner = new TaskMockRunner(taskPath);

const sharedInputs = shared.sharedInputs;
for (i in sharedInputs) {
   taskRunner.setInput(i, sharedInputs[i]);
}
taskRunner.setInput("workingDirectory", workingDirectory);
let a = shared.sharedAnswers;
a.exec['terraform init'] = {…}
a.exec[`terraform validate -var-file=${workingDirectory}`] = {…}
taskRunner.setAnswers(a);
taskRunner.run();

If you wanted, you could also do this with environment variables as you mentioned to give more of a traditional mocha before(), test(), after() sequence. This isn't really any more code than implementing your suggestion, yours just places a little bit of the complexity in any-steps.ts. Overall, I just feel like this is a little overengineered/introduces an unnecessary additional layer of complexity/size.

Closing this as this probably isn't the best forum to describe the problem. I'll keep isolated to my own project for now and maybe work with CSE-EEST team to see if they have seen similar issues to what I'm trying to describe.


Optimize WorldFileData

I see you have your own form of RegionFileCache in WorldBorder
I suggest you apply the following improvement to the code to improve IO performance reading Region Files :)
https://github.com/PaperMC/Paper/blob/b62dfa0bf95ac27ba0fbb3fae18c064e4bb61d50/Spigot-Server-Patches/0086-Reduce-IO-ops-opening-a-new-region-file.patch
This will improve performance.

Hmm, looks reasonable and straightforward enough. Not gonna lie though, that header.clear() threw me for a loop until I looked into what that clear() actually does (as opposed to what you'd expect) and how asIntBuffer() initializes based on the current position.
I'll probably test and implement this in a little while when I have the time for it.

After some brief testing, doing a timing profile of the specific section of code with 10 runs on the same initial world and Fill task, 5 runs before and 5 after making the suggested change, it indicated a bit over 5x faster execution of that algorithm in my test environment. Nice.
Went ahead with it.
https://github.com/Brettflan/WorldBorder/commit/af4727d1b62de224e9c9476e0006df6b662b7fe5


Stock App malfunctioning

The app in stock-app is featured in the Shiny for Python gallery. Its thumbnail looks like this (and I can confirm that the app originally worked).

When you click through to launch the app, the app looks like this

I'm not sure what's changed exactly, likely a change in the plotting package dependency. But the solution is almost certainly to call float(alpha) on whatever value is being used for alpha there.


No handling of empty uptime strings results in errors

https://github.com/postmodern/ruby-nmap/blob/43e851324f9f725ad6e51b88a1e80a5f4c5ae98e/lib/nmap/host.rb#L266-L276
The above method checks if uptime exists within an nmap report and then parses the lastboot attribute as a timestamp. This fails if lastboot is the empty string and no checking is done for this.
Nmap 7.92, when generating simple ping reports, contains the uptime tag without any of the attributes being filled. This ruby-nmap therefore fails for any ping only reports.
I have implemented a simple patch on my local branch by substituting:
@uptime ||= if (uptime = @node.at_xpath('uptime'))

with
@uptime ||= if ((uptime = @node.at_xpath('uptime')) && !(uptime['lastboot'].empty?))

If uptime is completely missing, the second part of the condition which I added is not evaluated, which avoids any [] referencing on an non-existent object. This handles the cases where uptime and associated strings are set to actual values, set to empty strings or when the uptime tag is completely missing from the report.
I have not submitted a PR as I don't usually use Ruby and am not sure of the most idiomatic way of doing this.
If someone can prettify my patch and get it implemented, that would be appreciated!

Can you share the nmap command-line option flags your using to generate this XML output? I downloaded and built locally nmap-7.92 and cannot reproduce this. It seems to me that if lastboot is empty, it should just be omitted from the XML output all together, because the nmap DTD says lastboot is IMPLIED (aka optional). This may be a XML bug in nmap-7.92.

I have regenerated some XML files at my end to replicate my own results. The command run was nmap -sn <LOCAL_CIDR_RANGE>
I agree that the result of this when running nmap-7.92 is that the uptime node (with associated lastboot attribute) is ommitted. It appears that the sample XML I was working with was exported from zenmap. Running the same command in zenmap and then saving the report to xml introduces this error.
I understand if this is out of scope of this project, but would appreciate the patch nonetheless to ensure that a zenmap report can also be properly parsed.

I would also suggest submitting a bug report to zenmap so this can be fixed at the source. They should filter out empty values from their attributes.

Closing this as it was an issue with zenmap.


How to make an array of object?

                    let data = array.into_iter().map(
                        |item| {
                            object! {
                                "ex_id" => 1,
                                "ex_order_id" => &item[""],
                                "ex_execution_id" => &item[""],
                                "side"=> &item[""],
                                "price"=> &item[""],
                                "size"=> &item[""],
                                "t_execution"=> &item[""],
                                "fee"=> &item[""],
                                "fee_currency"=> &item[""],
                                "commission"=> &item[""],
                                "commission_currency"=> &item[""]
                            }
                        }
                    ).collect();

I tried to make this...
expected struct json::object::Object, found enum json::JsonValue

The object! macro always produces a JsonValue::Object variant. You can unwrap it with if let JsonValue::Object(object) = value { ... }.
I haven't been doing much here for a while, this might be a thing I could change before going 1.0.

by getting value, I failed to make
                    let data = array![];
                    array.iter().map(
                        |item| {
                            let value = object! {
                                "ex_id" => 1,
                                "ex_order_id" => &item["child_order_id"],
                                "ex_execution_id" => &item["child_order_acceptance_id"],
                                "side"=> &item["side"], //BUY / SELL
                                "price"=> &item["price"],
                                "size"=> &item["size"],
                                "t_execution"=> &item[""],
//                                "fee"=> 0,
//                                "fee_currency"=> ,
                                "commission"=> &item["commission"],
                                "commission_currency"=> "JPY".to_string()
                            };
                            if let json::JsonValue::Object(object) = value {
                                data.push(value);
                            }

                        }
                    );

because of the trait std::convert::From<&json::JsonValue>is not implemented forjson::JsonValue

If you are pushing onto array![], you can do either:
if let json::JsonValue::Object(object) = value {
    data.push(object); // not `value`, otherwise you can't take ownership of `value` line above.
}

Or just:
data.push(value); // This is fine because `push` expects `Into<JsonValue>` anyway.

Other than that, you are not consuming the iterator. You should use .for_each or a regular for item in array.iter() { loop instead. Hope this helps!

i tried both of
if let json::JsonValue::Object(object) = value {
    data.push(object); // not `value`, otherwise you can't take ownership of `value` line above.
}

or
data.push(value);

the same error of the trait std::convert::From<&json::JsonValue>is not implemented forjson::JsonValue  exists.

What line is  that error on? It seems you are borrowing the values here:
"ex_order_id" => &item["child_order_id"],
"ex_execution_id" => &item["child_order_acceptance_id"],
...

You can try getting an owned iterator (.into_iter()) or mutable one (.iter_mut()) on array, then do one of the following:

"ex_order_id" => item.remove("child_order_id") - removes the JsonValue from item
"ex_order_id" => item["child_order_id"].take() - same as above, but leaves JsonValue::Null in its place, results are roughly the same.
"ex_order_d" => item["child_order_od"].clone() - leaves item intact, but performs an extra allocation if the field contains an array, object or a string.

39 |                               let value = object! {
   |  _________________________________________^
40 | |                                 "ex_id" => 1,
41 | |                                 "ex_order_id" => &item["child_order_id"],
42 | |                                 "ex_execution_id" => &item["child_order_acceptance_id"],
...  |
50 | |                                 "commission_currency"=> "JPY".to_string()
51 | |                             };
   | |_____________________________^ the trait `std::convert::From<&json::JsonValue>` is not implemented for `json::JsonValue`

do you have anyway to do messaging? I can update the result here.

If you put a working example of putting multiple objects into array here or in tests, I will be able to figure that out as well.

@abbychau you can find me on Ratel Gitter. It seems to me the issue is what I described above.


Replace 'clone_from_slice' with 'copy_from_slice'

Currently, shared_memory::safe_copy uses clone_from_slice to
perform memory copy. clone_from_slice iterates over each element of
slice to invoke 'clone_from'. According to docs of clone_from_slice,
it can be more performant to use copy_from_slice instead when copy target implements Copy.
Since we're copying a u8 slice, I thought replacing clone_from_slice with copy_from_slice would be a helpful change regarding performance.
Thank you for reviewing this PR :)

bors d+
copy_from_slice does seem to be the right tool here.

bors r+


[docs] Demo VirtualElementPopover crashes

[ ] I have searched for similar issues in this repository and believe that this is not a duplicate.

Steps to reproduce

Visit https://mui.com/material-ui/react-popover/
Scroll down to virtual element.
Highlight text
demo VirtualElementPopover crashes

Your environment

Tech
Version

Version
v5.14.20

Netlify deploy
https://656eed661c7afa000852ba6a--material-ui-docs.netlify.app

Browser
Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/17.1.2 Safari/605.1.15

https://github.com/mui/material-ui/assets/30326191/5f859cb7-4f9a-4bb8-8f5e-263e1c017164

https://github.com/mui/material-ui/assets/30326191/7d6d57c3-22f7-424c-be92-b4957098888d

Hey y'all, I have fixed this issue in this PR: 🙌🏼
https://github.com/mui/material-ui/pull/40239

Unfortunately I am still seeing the same error. is it just my device?

With the deployment URL on this pull request, I am not able to recreate this issue on Safari anymore. Can you double check the PR? Thank you!
https://github.com/mui/material-ui/pull/40239
https://github.com/mui/material-ui/assets/44305048/ab897e9f-a5ad-430d-96a8-46cc9690b272

My mistake, yes it is fixed in the deployment URL. Thank you!


Allow buttons with text+icon or icon+text

Only using block on mat-icon-button means you can have buttons that are not icon only (icon+text or text+icon)

@googlebot I signed it!

@friedcell thanks for taking the time to send a PR. Unfortunately I don't think we can accept this change for two reasons:

Changing the display: block element here would be a pretty breaking change, since any custom application CSS that's setting e.g. width or height would stop working as expected
We're trying to avoid making substantive changes to components that have an equivalent MDC Web-based component in @angular/material-experimental, which will replace the current component in the near future


LightStep http2

Fixing https://github.com/datawire/ambassador/issues/796
I could not yet test the entire request flow with LightStep since I don't have a valid API key.
@danielbryantuk If you want to try it out, you can test this build using docker image: agervais/ambassador:lightstep-http2-162f648-dirty
The generated Envoy config looks like:
Zipkin
{
    "lb_type": "round_robin",
    "name": "cluster_ext_tracing",
    "type": "strict_dns",
    "urls": [
        "tcp://zipkin:9411"
    ]
}

LightStep
{
    "features": "http2",
    "lb_type": "round_robin",
    "name": "cluster_ext_tracing",
    "type": "strict_dns",
    "urls": [
        "tcp://collector-http.lightstep.com:8081"
    ]
}

and Envoy is starting/reloading successfully now, no longer complaining about cluster_ext_tracing collector cluster must support http2 for gRPC calls

Hey Alex,
Many thanks for this. The config generated looks good, and I'm getting no initialisation errors now. But, I'm still not able to upload tracing data to LightStep. I did an "network down" type error message at one point (in relation to the tracing cluster), but I can't repeat this. Unfortunately this could be any number of reasons, and so more testing from my side is needed.
Unfortunately I won't be able to look at this properly now until Monday, as I'm working on another project today and Friday, but I'll swing back around then. I might try and get some of the LightStep folk involved then too. It would be nice if we could turn on more detailed tracing debugging within Envoy itself.
Best wishes,
Daniel

At this point I'm assuming my LightStep endpoint collector-http.lightstep.com:443 is invalid:
[69][debug][http] source/common/http/async_client_impl.cc:92] async http request response headers (end_stream=true):
':status', '200'
'content-type', 'application/grpc'
'grpc-status', '14'
'grpc-message', 'upstream connect error or disconnect/reset before headers'

[69][warning][tracing] source/extensions/tracers/lightstep/lightstep_tracer_impl.cc:31] Failed to send report: Network is down

GRPC Status 14 being: "The service is currently unavailable."

Looking back at the example config I found, the only thing missing was the ssl_context
        "ssl_context": {
          "ca_cert_file": "/etc/ssl/certs/ca-certificates.crt",
          "verify_subject_alt_name": ["collector-grpc.lightstep.com"]
        },

I done some very basic experimentation with the example below, which works as-is, but as soon as I remove the ssl_context I get nothing:
https://github.com/danielbryantuk/envoy-lightstep-example
Can we easily add the ssl_context into the generation cluster? We would need to parameterise the verify_subject_alt_name but the ``ca_cert_file``` should be good?

Tested with build agervais/ambassador:lightstep-http2-d013fb1-dirty.
The ca_cert_file exists in the docker image and the verify_subject_alt_name is inferred from the existing driver configuration.
Resulting Envoy config:
{
  "name": "cluster_ext_tracing",
  "connect_timeout_ms": 3000,
  "type": "strict_dns",
  "lb_type": "round_robin","features": "http2",
  "hosts": [
    {
      "url": "tcp://collector-http.lightstep.com:443"
    }
  ],
  "ssl_context": {
    "ca_cert_file": "/etc/ssl/certs/ca-certificates.crt",
    "verify_subject_alt_name": ["collector-http.lightstep.com"]
  }
}

Awesome work @alexgervais ! This works as expected for me, and so I'll leave @kflynn to let us know when this can be merged


Microsoft Graph v1.0: teams message reply list

The teams message reply list command uses a beta endpoint that is now in v1.0.
https://github.com/pnp/cli-microsoft365/blob/266fd8d68e2a71a9c3d8d58d5c133916193f99b9/src/m365/teams/commands/message/message-reply-list.ts#L35
Get chatMessage in a channel or chat: https://docs.microsoft.com/en-us/graph/api/chatmessage-get?view=graph-rest-1.0&tabs=http#example-3-get-reply-to-a-message-in-a-channel

Hi @garrytrinder, Please assign this to me.


Render jinja2 automatically if template tags appear in SQL

The jinja arg to result() was clunky UI, and led to unpleasant errors if there's jinja in the SQL but the arg is set to False. Instead, this revision looks for a simple regex {%.*%} which should detect template tags in the SQL code (and seems unlikely to appear in normal non-jinja-fied SQL).

Also some improvement of comments. help(db) looks pretty good now.


is it active project?

Hi, this is great app, but is it in active development?
and in demo, are you mining bitcoin?is there any information about this somewhere in you project for visitor/user?
I found http://www.monkeyminer.net:8080/lib/cryptonight.wasm from your demo.

yes but to small, i am forget to remove, thanks for your infomation, now i remove them


add url-choice / flag to create in app.py

How does this affect the build command and other? Do we have to pick this from the pyproject.toml or similar in other places now?

https://github.com/search?q=repo%3Astatisticsnorway%2Fssb-project-cli STAT_TEMPLATE_REPO_URL&type=code


Investigate initial wizard delay on radio button selection

The initial radio button pick takes a while, I suspect because the discoclient does a HTTP request in the event queue.

I still see a small delay there.


Starter template for javascript

Hey,
I have added a starter template for javascript language. Also, this is my first contribution to codecrafters, let me know if anything else needs to be done for contributing to this repo.
Thanks

Hey @rasoolkhan06 - thanks so much for this! I think you caught us in the middle of when we were expanding JS support 🙂 This got merged and is live now: https://github.com/codecrafters-io/build-your-own-http-server/pull/10/files.


Updates to FTA importer

Updates the FTA spreadsheet importer to handle more exotic quotas and also to correctly understand dates across the Brexit boundary.

Handle origin certificates in origin quotas.
Actually output multiple countries into one envelope.
Fix date handling in the FTA importer
Handle quota suspensions in the FTA importer.
Handle excluded quota origins in the FTA importer.
Handle quota associations in the FTA importer.
Import universal style quotas into the FTA importer.
Don't output licensed quotas from the FTA importer.
Untie date boundary handling from parsing of rates

Closing – we need a proper decision about what to do with this stuff.

Closing – we need a proper decision about what to do with this stuff.


chore: release 2.3.11

:robot: I have created a release *beep* *boop*
2.3.11 (2021-08-31)
Bug Fixes

Cleanup of Docker image building (8f36616)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/ptarmiganlabs/butler-cw/releases/tag/v2.3.11 :sunflower:


Fit watermark into code org (fixes #13069)

Ben, could you take a quick look? One issue is that the contribution is instantiated before the editor part's container element is created, so I wait for partService.joinCreation(), but that's already a little late.

I'm actually not sure if partService.joinCreation() is too late, maybe it is still in time?

Coverage decreased (-0.0004%) to 61.218% when pulling 09bb4f6a1c783bd9416c356cc07a8c91dad2766e on chrmarti/watermarkcodeorg into 9fcf0f62495e51d78fb4596c3e4ca812293a4955 on master.

@chrmarti change looks good to me, feel free to merge. joinCreation is the right thing to do to ensure the workbench is there 👍

Rebased locally, thanks.


`SubjectNerd.PsdImporter.PsdParser.PsdDocument.Disposed': event must be of a delegate type

Hey there, when I am importing this in unity 2018.2. This error is coming and I am not able to use it. Am I the only one facing this issue.?

I'm having this issue as well. Anyone have any ideas?


[Documentation][IDE] Add a reference to the docs for the intellij-move IDE

There's a project called intellij-move which is Move Language Plugin for Jetbrains IDEs. It supports all of them, including the free PyCharm Community. IDE should suggest installing it when any .move file is opened, or just download it from the Marketplace in Settings -> Plugins section.
We started it in the middle of 2020, and now it's in quite good shape.
Supported features are:

Parsing of both Move language and MSL
Name resolution in Move and MSL
Types and type inference in Move
Autoformatter
Support for Move.toml, move-cli commands, local deps, git deps (partially)

It would be great to have some official recognition of the project, maybe add a link in the docs.
I know Diem started their own language server at https://github.com/diem/move/tree/main/language/move-analyzer, based on rust-analyzer libraries, but it has only some initial implementation. At Pontem, we tried this approach too (code at move-language-server) but it was scrapped when we realized that Diem has started move-analyzer. It has a little bit of parser implemented, I can try to finish the parser later and merge it upstream if needed.

That sounds excellent, thanks for the info @mkurnikov!
@vgao1996 @modocache: I don't have the most context on the status of move-analyzer, but wanted to tag you all to see if bringing in the parser from Pontem's vscode plugin could help.


error

I want phar2zip
Warning: ZipArchive::close(): Can't open file: Permission denied in /var/www/pmt.mcpe.fun/pages/phar2zip/convert.php on line 82
Warning: chmod(): No such file or directory in /var/www/pmt.mcpe.fun/pages/phar2zip/convert.php on line 83

Should be fixed in a few seconds.

ok


Add parameter hints

Parameter hints are displayed in ResultDisplay when a command word is typed
Resolve #67

Feature would have to update new commands in the future


Fix mappings

The current implementation was not working, this refactors the key
mappings according to best practices.

Any chance we can get this merged?

Could you please describe in more detail what issue this fixes, so I know how to test this PR? You say the keymappings were not working, but they seem to have been working for everyone else up until now. What specifically wasn't working for you, which version of Vim / NeoVim, which platform, is there a way to reproduce your issues easily?
Thank you for your hard work.

@ajhager absolutely. Sorry for not providing more details upfront.
So, I'm running Neovim, version 0.1.4. The suggested keymappings was not working at all for me. The defined autocommands were indeed executed, but had no effect.
First of all, I moved out the mapping definitions to plugin/elm.vim. This have no effect for the problem I was having but it's standard practice to define <Plug> mappings in plugin/x.vim so that the mappings are globally available for power users and other VimScript developers.
Secondly, I removed the autocommands and redefined the mappings as buffer local commands.This is also standard practice. See "FileType Events" in http://learnvimscriptthehardway.stevelosh.com/chapters/12.html.
These mappings can not have been working for anyone else since they got moved from the README into the ftplugin/elm.vim file. The ftplugin/elm.vim file is executed as you open a new file and thus the defined autocommands are not triggered. You would have to reopen another file for this to take effect.
If anything is unclear, don't hesitate to ask. I'm not experienced with VimScript at all so feel free to question any change I made. 😄

The mapping work fine for me before this patch, but with it they stop working completely. I will investigate your changes more, and check out the link you provided. Hopefully we can find a solution that is correct on both of our computers.

@ajhager make sure that you've set maplocalleader to the same key as your mapleader.


Words are truncated in the column

As you can see the email column data has been truncated, it works in Chrome, but not works in Microsoft Edge and Internet Explorer, any solution to help on this? Appreciate if any guy would help on it.

A simple solution would be to set the width of the column. Checkout this.
It would be easy to understand the specific scenario if you post a code snippet as well.

Another solution could be to use the column.cssClass configuration option in order to customize the table cell. You can see an example at http://gijgo.com/Grid/Configuration/column.cssClass


Upgrade Ubuntu OS versions for GitHub Actions

Starting on 8/8/2022, the Ubuntu 18.04 image was deprecated. It will be completely deprecated and unsupported on 04/01/2022.
We need to change our Ubuntu versions to 20.04 and 22.04. Making this change also means we need to update the README regarding the minimum supported GCC/Clang compilers.

I can work on this. From what I understand the workflow  build settings have to be updated in order to remove Ubuntu 18.04 along with updating the README file.

@ShivamNegi Sure thing!
And yes, the workflow build settings have to be updated in order to remove Ubuntu 18.04. Additionally however, Ubuntu 22.04 should be added as a new build OS.


chore: add protocol requirement

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Tests are broken

Simplest possible pull request:
https://github.com/ethersphere/bee/pull/395
But automatic tests don't pass, so something is broken.
Run failed for master (81dc9b0)
Repository: ethersphere/bee
Workflow: Coverage
Duration: 1 minute and 48.0 seconds
Finished: 2020-06-29 18:35:13 UTC

View results

Jobs:
coverage failed (1 annotation)
—
You are receiving this because this workflow ran on your branch.
Manage your GitHub Actions notifications here.

Links to problem:
https://github.com/ethersphere/bee/runs/819592642
https://github.com/ethersphere/bee/actions/runs/151814431

Hi, thanks for reporting this. Actually the tests are not broken, its just that our static analysis tool does not work when PRing from private repos.


Manage caching for messages

Background
Since a webhook was implemented in the Transaction Service for message-related actions (https://github.com/safe-global/safe-transaction-service/pull/1629), now it is possible for the CGW to manage a Message cache. See https://github.com/safe-global/safe-transaction-service/issues/1240 to gather more context on the specification.
Problem

Until now, messages were not being cached in the CGW, due to the lack of a proper webhook.

Solution

Store the messages for a given Safe address in the cache.
Implement a handler that will clear the related cache fields when the associated webhook is received.

A threadpost in slack was created to test the merging of this ticket about messages. Slack Thread
I tested comparing current prod: https://app.safe.global/ (v1.17.1)
and staging: https://safe-wallet-web.staging.5afe.dev/ (v1.17.1)
I tested by using a dummy app we have to create messages on demand.
https://github.com/5afe/eip-1271-dapp
The dapp allows for off-chain tx, so everything I did was off-chain siging
Tested in this safe:
https://safe-wallet-web.staging.5afe.dev/transactions/messages?safe=gor:0x0EDc6697427E9e9d62A1eE1A2951AfA7bE9e75B7
And this one in prod:
https://app.safe.global/transactions/messages?safe=gor:0x5C82F5eA987F115c1AcAD627aDe6fF7396BD79dB
I tested by opening side by side 2 browsers Chrome and FF, each with a different owner of a safe:
In chrome I had the app connected and used to see the message proposal and do the first signing.
Firefox was to see the "Messages" tab being updated
On staging:
1- I created a message in Chrome that require 1 signature and Signing it
Expected:
For the message to show up in the firefox browser fully signed
Obtained:
The message never showed up
I tried several times but the fully signed message never showed up.
2 - I created a message in Chrome that requires 2 signatures from owners. Signed with the first one
Expected:
The message popUp in firefox to be signed
Result:
The message never showed up
So in chrome, I just switch to the other owner, fully signed the tx. NOW in Firefox, in the message tab, the fully signed message plus all the previous messages that were created in point 1 show up in the tab
In prod:
Everything I was expecting happened:
The message, when only 1 owner was required to sign in chrome, showed up right away in the message tab in the firefox browser
For the 2 owner message, after signging it with the 1st owner in chrome, it showed up right away in the Firefox message tab so the 2nd owner can sign.


Not working in Expo 18

Used to work with Expo 17. Tried a boilerplate app, either I get a blank page or a "Could not connect to server"

Hello @ASAPHAANING.
As for now we do not support expo because of some native libraries like realm.io. We will probably  implement it in near future.

Hi, @malashkevich !
Can you add support of another data provider for now? Need kittenTricks for Expo too....
Thanks!

Most people that use React Native use Expo in some way or form, so hoping this will be implemented 🙏

Hello @malashkevich. Seems to work fine in expo here: https://expo.io/@community/kitten-tricks
Curious...what steps did you need to take to make it work? Are you using something other than realm there?

Hi, elgiovanno! In this expo example not enabled realm. It’s not supported in expo. I think it’s just commented lines, and using only json files for listing items in feed/dialogs list/ etc. Because cannot write PM and view profile or blog post from list except one. So I’m do the same - just disable realm.io using in source code, make read data from json files that used for “bootstrap” database. Also successfully added auth via fetch and real-time messaging via socket.io. If you need some help with this - feel free to ask me.

@vladimir-tsapkov Awesome! I'll have a try at your work-around and see what I find. Any chance you have a repo that can be forked? PS sorry for late reponse, I live in puerto rico and right now it's not so easy to communicate after Hurricane Maria.


Migrate Operations missing

In official api document, there are DomainMigrate operations.
However, in this repository they are missing.  There are only some basic operations such as DomainMigratePrepare3, DomainMigratePerform3...
Is there anyone can help me finish the migrate operations ?

I think DomainMigratePerform3 is the remote version of virDomainMigrate3 :thinking:. I'll ask around to see if we can get you a concrete answer here.


Control, styles and color

Wow @ricardo-valero these are some nice changes. Thank you for doing this.
I've left some comments for your consideration, nothing blocking I think. Let me know what you think I'd be happy to merge as is and do anything in a follow up if that is easier?
This is a nice to have, but I think it'd be great to add a doc comment on the exposed types and functions. The package is starting to get much larger and this would be super helpful for users. Also you can add code snippets in doc comments which could be super helpful too.
Also, if you're interested (though definitely out of scope for this PR) -- we have another idea I haven't got around to implementing that would be a massive performance enhancement for drawing the screen. Essentially, change the draw functionality to draw to a virtual "screen" and then diff it and only output the draw commands to the terminal buffer to draw the changes.


Lukebroganws rename

@lukebroganws I've migrated the changes in this PR to a new PR (PR #3) in the new mend-tookit org.
Except for a handful of obvious typos, I copied all the changes as-is, so the new PR still needs to be reviewed independently (I've already reviewed most of it and added a few comments that need to be resolved).
Either way, please halt any further work on this PR (in whitesource-ft) and continue in the new one.
I'm closing this PR.


New package: APattern v1.0.1

Registering package: APattern
Repository: https://github.com/RFspin/APattern.jl
Created by: @bosakRFSpin
Version: v1.0.1
Commit: 476c097ab984d79a5e3f045ce64c68d5a18eaf7a
Reviewed by: @bosakRFSpin
Reference: https://github.com/RFspin/APattern.jl/issues/18#issuecomment-1691746618
Description: Antenna data import and processing.

Shouldn’t this be called AntennnaPattern?

I was trying to keep it short. We can rename it no problem.On 24. 8. 2023, at 17:00, Michael Goerz @.***> wrote:
Shouldn’t this be called AntennnaPattern?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

Shouldn’t this be called AntennnaPattern?

I have renamed the package. Should I start a new register request?

Yes, as you did…
This PR will automatically close after 30 days

Yes, as you did…
This PR will automatically close after 30 days

Amazing, is the package publishable? Do I need to worry about anything? When will I see if the package was published or not?

Yes, you’re good. https://github.com/JuliaRegistries/General/pull/90269 will be automatically merged in three days


Add items to workbench by default

Hi!
I want to add some catalogue items to the workbench when the user loads the page.
I found a setting for init.json called "workbench" (which takes an array of strings, presumably catalogue item-names), but I can't get it to work..
Any help on this?
Thanks in advance

@nf-s ?

Workbench are array of item ids, every catalog item can have it, if not specified terria will autogenerate id but I would avoid using them


Layer hover config in style block

The hover{} configuration should be moved into the style block and support multiple entries in a similar fashion to the labels config. It should be possible to enable and disable hover through the style panel and switch between different hover entries.

https://github.com/GEOLYTIX/xyz/wiki/Workspace-Configuration#hover


How does it work?

I would like to know how does this work?
Moreover without our Heroku Emails how can it switch apps with just the API Keys?
It would help if these things are mentioned in the Readme.
Btw Love your App :)

It transfers your app to second account on 15th day and to main acc on 1st day of a month.

@sayeed205 I know that
But how with just the API Key?
Every other applications asks for both the API Key and the Associated Heroku Email.

@sayeed205 I know that But how with just the API Key? Every other applications asks for both the API Key and the Associated Heroku Email.

You are providing API key means you are providing everything related to your account.
Check this line here getting info about account and here getting the email.
I hope you understood.

sayeed missemily is known for copying repos and labeling as her own.

sayeed missemily is known for copying repos and labeling as her own.

I know very well maybe check this original maybe and this copy maybe


SDK download fails

The SDK version and download links are obsolete. The code works on Linux with SDK 1.36   downloaded independently, unpacked and the library files copied to a "lib" directory in the src tree. It interacts with a new ASI 6200 camera with include("LibASICamera.jl")  in REPL.  However, I was not able to get an automated download to work, and the package, as is, will not install from git.
Nevertheless, this is a great resource to build science imaging code for the ZWO cameras.

Thanks for the report! Seems like they changed the way they distribute the SDK. It now comes in one file for all operating systems. Currently I don't have time to fix it, but if you want to, you can go ahead and fix the build script. Shouldn't be complicated, just rewrite the download and extraction code.

Thanks for the response, and for developing the wrapper library.  I'll see
if I can write a script to download the SDK as it is currently offered.  I
built a local version of your code and installed the library by hand.  It
works fine that way, incorporated into a program for astronomical image
acquisition with FITS file output.  It may be a generally useful
addition too.  Let me see where this leads, and then I can offer the
changes that work here.
On Wed, Sep 25, 2024 at 11:36 AM Max Freudenberg @.***>
wrote:

Thanks for the report! Seems like they changed the way they distribute the
SDK. It now comes in one file for all operating systems. Currently I don't
have time to fix it, but if you want to, you can go ahead and fix the build
script. Shouldn't be complicated, just rewrite the download and extraction
code.
—
Reply to this email directly, view it on GitHub
https://github.com/AlfTetzlaff/LibASICamera.jl/issues/3#issuecomment-2374439065,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALM7ETP7NEM7M3VKGJ2PJ2DZYLKBFAVCNFSM6AAAAABORVZW3GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNZUGQZTSMBWGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
John Kielkopf
Louisville, Kentucky
Office Tel:<PHONE_NUMBER>
Zoom: https://zoom.us/j/5116430632


Add ability to disable URL with lazy loading

Ionic version:  (check one with "x")
[x] 3.x
I'm submitting a ...  (check one with "x")
[x] bug report
[x] feature request
Current behavior:
Currently it's not possible to disable the URL when using lazy loading. As deeplinking does not work well with complex navigation structures (yes I already tested v3.5.0) we need to be able to at least hide the URL.
Expected behavior:
For example with a config property which disables/hides the URL.
In my opinion this is really important because if someone wants to publish their app as PWA but the deeplinker does not work well with their nav-structure the user-experience will be even worse as with deeplinks.

Hello, thanks for using Ionic! Would you mind opening an issue with us about the deeplinking issues your seeing? Also, thanks for the feature request. I'm going to move this issue over to our internal list of feature requests for evaluation. We are continually prioritizing all requests that we receive with outstanding issues. We are extremely grateful for your feedback, but it may not always be our next priority. I'll copy the issue back to this repository when we have begun implementation. Thanks!

This issue was moved to ionic-team/ionic-feature-requests#263


docs: fix Configuration, JestPlatform and JestObjectAPI docs for 26

Summary
Just noticed we miss modern fake timers in 26 docs, so fixing. Adjusted other docs as well. Don't think changelog is necessary :D
Test plan
CI

doc deploy failed, same error as on CI for this


Uml 914 production

Purpose
Enable lpa-codes on production
Fixes UML-914
Approach
Set the flag use_legacy_codes_service to false
Learning
Checklist

[ ] I have performed a self-review of my own code
[ ] I have added relevant logging with appropriate levels to my code
[ ] I have updated documentation (Confluence/GitHub wiki/tech debt doc) where relevant
[ ] I have added tests to prove my work
[ ] The product team have tested these changes

Coverage remained the same at 81.604% when pulling 4a7f5e55094ea841fa02bfc90496702de19d73de on UML-914-production into 2a1442b790eaed373f99e5ad68790d6cc1dd58bb on master.


Support nested values for display_field in DynamicModelChoiceField

As ...
P.D. - Plugin Developer
I want ...
Currently, when the UI presents the end-user with a list of dynamically generated choices (for example list of available vlans on the interface), the list is rendered using the display attribute of the results returned by the backend API.  The request to the API is made by the machinery included in select2 library and processed on the Nautobot side to formulate how the list of choices looks like. By default, it uses the display value of each record in the response.
This behavior is not always desirable and in some cases, does not provide enough information to the end-user. One particular use case we are currently facing involves selecting a VLAN while editing the Interface, where the list of available choices includes VLAN 200 from VlanGroup "A" and VLAN 200  from VlanGroup "B". With the default behavior, the list of rendered choices just lists "VLAN 200" twice with no way of clarifying which is which.
As a plugin developer, I was able to override a dcim:interface_edit view to replace a model form with a custom form. This custom form uses an adjusted DynamicModelChoiceField with a custom display_field. This is what is possible with the current version of Nautobot at the moment.
Unfortunately, in its current form display_field allows accessing only root-level entries in the API's response. In other words, one can access id, name, and display attributes, but cannot access vlan_group.name.
So that ...
Plugin developers should be able to specify custom display fields, such as computed_fields.description_with_vg or vlan_group.name instead of just top-level fields.
I know this is done when...
I would like to propose a small change (PR incoming) in processResult() that would allow accessing arbitrary fields in the API response. During our testing, we  used computedFields.description_with_vg successfully after modifying's nautobot code and adding the below code to our plugin:
# view_overrides/views.py
from nautobot.core.forms.fields import DynamicModelChoiceField
from nautobot.dcim.forms import InterfaceForm
from nautobot.dcim.views import InterfaceEditView
from nautobot.ipam.models import VLAN

class MyInterfaceForm(InterfaceForm):
    untagged_vlan = DynamicModelChoiceField(
        queryset=VLAN.objects.all(),
        required=False,
        label="Modified Untagged VLAN",
        display_field="computed_fields.description_with_vg",
        query_params={
            "available_on_device": "$device",
            "include": "computed_fields"
        },
    )

class InterfaceEditViewOverride(InterfaceEditView):
    model_form = MyInterfaceForm

override_views = {
    "dcim:interface_edit": InterfaceEditViewOverride.as_view(),
}

I would consider this issue "done" when the above code renders a list of choices displaying a value of computed_fields.description_with_vg returned by the API.
Optional - Feature groups this request pertains to.

[ ] Automation
[ ] Circuits
[ ] DCIM
[ ] IPAM
[ ] Misc (including Data Sources)
[ ] Organization
[X] Plugins (and other Extensibility)
[ ] Security (Secrets, etc)
[ ] Image Management
[ ] UI/UX
[ ] Documentation
[ ] Other (not directly a platform feature)

Database Changes
No changes are required.
External Dependencies
No external dependencies.

Relates to #2668 but more narrowly scoped.


Add tests for insert_or_update

Weird, I don't even know how I managed that @josevalim - sorry. I continue to surprise myself. :)

:heart: :green_heart: :blue_heart: :yellow_heart: :purple_heart:


scopes

I'm trying to fill in the step where the user is asked to confirm scopes, which is currently being skipped.
I'll send a PR when I get it working :)

Trying to wrap my head around this (learning as I go along here)

authorize redirects to login if the user is not logged in
login redirects (back) to authorize if login is successful
authorize redirects to scopes if the request is not authorized
this depends on the function getIsAuthorizationApprovedFromSession which is yet to be implemented

I guess it's safe to store the accept yes/no decision in the cookie


Proposal to fix RHEL-07-040600 For Red Hat Enterprise Linux operating systems using DNS resolution, at least two name servers must be configured.

I think we can make this fixable. Here's what I propose

After confirming that DNS name resolution is being used, we default to Google's DNS nameservers but of course allow for end users to provide a list variable of DNS servers to use instead
In the event of local host auth, we can ensure that resolv.conf is empty

Check Text
Determine whether the system is using local or DNS name resolution with the following command:
# grep hosts /etc/nsswitch.conf
If the DNS entry is missing from the host's line in the /etc/nsswitch.conf file, the /etc/resolv.conf file must be empty.
Verify the "/etc/resolv.conf" file is empty with the following command:
# ls -al /etc/resolv.conf -rw-r--r--  1 root root        0 Aug 19 08:31 resolv.conf
If local host authentication is being used and the /etc/resolv.conf file is not empty, this is a finding.  If the DNS entry is found on the host's line of the /etc/nsswitch.conf file, verify the operating system is configured to use two or more name servers for DNS resolution.
Determine the name servers used by the system with the following command:
 # grep nameserver /etc/resolv.conf nameserver
If less than two lines are returned that are not commented out, this is a finding.

This seems like the correct course of action to me. The only issue I ran into with the DNS thing was that it required 2 nameservers which breaks in AWS environments. Also I'm not sure what this means for folks who are getting their DNS from DHCP.  Setting the immutable flag as detailed at the bottom of that Fix text seems like overkill and would definitely break dynamic environments.

I would want to check if there's an existing DNS server and keep that rather than point to Google. Also, if the check doesn't fail for not having immutable, I would skip that step, as that should be a deliberate administrator action and not touched by automation IMHO.

Yeah I wouldnt want to clobber something that exists. The tricky part becomes what do we do if it's only one server listed? Or it's dynamic? Maybe we do a custom error message?

The check doesn't explicitly fail for not having immutable but it does have a note about a having a documented exception:

If the "/etc/resolv.conf" file must be mutable, the required configuration must be documented with the Information System Security Officer (ISSO) and the file must be verified by the system file integrity tool.

It becomes tricky to leave exiting settings since most systems are going to have "something" in their when this role is run. They will need it to having a functioning network connection.
Maybe this becomes one of those rules that people need to disable if they don't want it to clobber their existing settings with whatever is set in the nameserver vars?

If we do end up setting the immutable flag we should definitely not put any workarounds to unset it if the role would otherwise change the file. In that case, we might set ignore errors yes on the task. If it the administrator set a file as immutable we should absolutely respect that setting.

Yeah...I'm wondering how Ansible will fail if the file is immutable. It might be nice to have a custom failed_when instead of just ignoring errors. That or check for the immutable flag first and skip the task if it is immutable.

Ansible will fail with permission denied

I think that in the default mode, we should just warn if there are not two DNS servers configured.  Otherwise, I re-iterate my recommendation of using rhel-system-roles/network to do the actual config as that role is fully supported by Red Hat.  There are other common cases to consider, such as having a local dnsmasq configured in the /etc/resolv.conf (only one <IP_ADDRESS> exists, but I suppose we could add ::1 as well) and the dnsmasq forwards lookups as necessary to the actual DNSN server(s)

I agree with just making this a warning. There are many instances where having two DNS servers is not relevant.

V2R7 adds "If the file is mutable and has not been documented with the Information System Security Officer (ISSO), this is a finding." meaning that folks need to document if they want mutable /etc/resolv.conf

Hello,
I wanted to reach out and let you know that this issue is being closed. We have re-worked the role and want to start with a fresh issues list with this latest version. There was a post in the Ansible-Lockdown google group (https://groups.google.com/g/ansible-lockdown) with the details of the changes that are coming. Please checkout the thread titled RHEL 7 CIS and STIG Changes for all of the details, I also have the message pasted at below.
Please as you use the latest version and open issue tickets as you find them, it is the best way for us to improve the role for everyone. Thank you for being part of the community and providing awareness of problems or advice on improvement. Reporting is a huge part of improving this project.

Hello,
Thank you to everyone in the Ansible-Lockdown community who has contributed to RHEL7 STIG/CIS. Our team at MindPoint Group has been working with the entirety of the Ansible-Lockdown project, and we have some significant updates for both RHEL 7 STIG and CIS. With these updates, some larger changes have been made. I have these changes/updates outlined below.
Testing:

CI/CD - We have implemented some automated testing pipelines to test pull requests into the devel and main branches. With the current workflow, the community will PR into the devel branch (never the main branch) for review by the administrators. When your PR is created, the first check will remain the DCO check. The second check is a functional testing pipeline that will automatically perform a functional test of the branch the PR is initiated from. Once both tests pass, someone from the Administrator Team will review the changes and merge them into the devel branch. From there, an additional review is completed before the devel branch is merged into the main branch. Only the Admin Team will perform PRs/merges into the main branch. There is also an automated pipeline for PRs from devel to main. Please do not edit the .github/workflows files since those are used as part of the pipeline.
Compliance Checking – MindPoint Group has been working to create our own compliance audit scan tool. The tool uses a goss framework executable to run custom checks that we have created. The goal is to provide a more thorough check for control compliance and decrease the number of false positives/negatives. For example, it will check the configuration file related to the control and as well as checking if that configuration is active. With a smarter scan, we can hopefully identify attempts to trick scanners as well (for example stacking a parameter in a config file where the first instance is enabled and second disabled. Most audit tools search for the first instance but the application might look for the last instance of the parameter, thus making the scanning tool think it's enabled). In testing, we have found that our audit scan runs significantly faster than other audit tools, reducing audit times. Our audit tool and profiles will have their own repositories in the Ansible-Lockdown org, but within the remediation role there will be an integrated way to incorporate the audit. Keep an eye out for the audit tools as they are released. We plan on developing a goss audit profile for each current remediation role. Going forward, we plan to release a remediation role and goss audit tool profile simultaneously.
Role Updates:
RHEL 7 STIG/CIS – We have re-written much of the RHEL 7 STIG and CIS roles to increase clarity and readability and address some functionality items. We performed these updates while creating our goss testing framework for each of these roles. We plan on pushing our update to the devel and main branches. We will move the current devel and master branches to a devel_stable_ and master_stable_ branch in the respective repositories. Accordingly, community members who rely on the current version can still use that version going forward; this process will not remove what is currently there. The latest versions of the roles have also been updated to comply with the latest benchmarks.
Role Architecture – All roles will change with regard to the structure in the tasks folder. Taking CIS as an example, there will be a folder per section and yaml files for each sub-section. For example control 1.2.1 in CIS will be located in RHEL7-CIS/tasks/section_1/cis_1.2.x.yml. The cis_1.2.x.yml file will contain all controls related to section 1.2.x. This will hopefully make updates to roles a bit easier with less risk. This matches the architecture of our audit tool, creating consistency across remediation and audit platforms. The end goal is to repeat this architecture (the best we can) on STIG roles, but we are starting with CIS.
Existing PRs and Issues – With all of these changes comes the task of cleaning up existing PR’s and issues. Our plan is to close all of the existing PR’s and issues because of the re-work. Our team is growing and should be able to stay on top of the new issues and PRs as they come in.
Again, I would like to thank the community for your involvement in this project. The input and work from the community has contributed significantly to the success of this project. Please keep an eye out for these changes, which will be rolling out in the coming weeks.


new card variation with title on top of image (cards overflow on tablet issue)

Taskman comment: https://taskman.eionet.europa.eu/issues/155228#note-37

@eea-jenkins build storybook


Nuget

Added nuget info to project.json
Fixed issues with running dotnet pack to build nuget-module

@avbel merggge conflict :(


Whether it is a bug ?

When I use Uber's goleak to run the test function TestWriteAndGetOnCache() in https://github.com/allegro/bigcache/blob/master/bigcache_test.go, I got the following output information. I was wondering if it is a bug ?

You can reproduce it like this:

Nice catch. Probably we should call defer cache.Close() after cache is created to prevent such leak.
https://github.com/allegro/bigcache/blob/982ec3b09cacb9936b88dcfdaed329870065cf23/bigcache.go#L122-L125


iter_sample does not work

Using pm.iter_sample in examples/run.py raises the following error:
Traceback (most recent call last):
  File "/home/bicyclus/examples/run.py", line 316, in <module>
    main()
  File "/home/bicyclus/examples/run.py", line 312, in main
    sample(args, pymc_model, initvals)
  File "/home/bicyclus/examples/run.py", line 290, in sample
    for trace in sampler:
  File "/home/anaconda3/envs/pymc-env/lib/python3.10/site-packages/pymc/sampling.py", line 981, in iter_sample
    for i, (strace, _) in enumerate(sampling):
  File "/home/anaconda3/envs/pymc-env/lib/python3.10/site-packages/pymc/sampling.py", line 1062, in _iter_sample
    point, stats = step.step(point)
  File "/home/anaconda3/envs/pymc-env/lib/python3.10/site-packages/pymc/step_methods/arraystep.py", line 155, in step
    apoint = DictToArrayBijection.map({v.name: point[v.name] for v in self.vars})
  File "/home/anaconda3/envs/pymc-env/lib/python3.10/site-packages/pymc/step_methods/arraystep.py", line 155, in <dictcomp>
    apoint = DictToArrayBijection.map({v.name: point[v.name] for v in self.vars})
KeyError: 'feed_assay_interval__'

I am currently investigating the error. Both pm.sample and pm.iter_sample eventually call pymc/sampling.py:1025 (function _iter_sample) to perform the sampling. I suspect that pm.sample does something to the initial values that transforms them into the correct format, compare the following pdb output:

Using pm.sample:(Pdb) w
  /home/bicyclus/toymodel/toymodel.py(452)<module>()
-> main()
  /home/bicyclus/toymodel/toymodel.py(447)main()
-> arviz_summary, trace, emu, solver_str = run_simulations(model, **kwargs)
  /home/bicyclus/toymodel/toymodel.py(291)run_simulations()
-> trace = pm.sample(
  /home/anaconda3/envs/pymc-env/lib/python3.10/site-packages/pymc/sampling.py(655)sample()
-> mtrace = _sample_many(**sample_args)
  /home/anaconda3/envs/pymc-env/lib/python3.10/site-packages/pymc/sampling.py(769)_sample_many()
-> trace = _sample(
  /home/anaconda3/envs/pymc-env/lib/python3.10/site-packages/pymc/sampling.py(915)_sample()
-> for it, (strace, diverging) in enumerate(sampling):
  /home/anaconda3/envs/pymc-env/lib/python3.10/site-packages/fastprogress/fastprogress.py(41)__iter__()
-> for i,o in enumerate(self.gen):
> /home/anaconda3/envs/pymc-env/lib/python3.10/site-packages/pymc/sampling.py(1025)_iter_sample()
-> model = modelcontext(model)
(Pdb) start
{'large_norm_interval__': array(1.44301298), 'small_unif_interval__': array(-0.17056391)}

Using pm.iter_sample:(Pdb) w
  /home/bicyclus/toymodel/toymodel.py(452)<module>()
-> main()
  /home/bicyclus/toymodel/toymodel.py(447)main()
-> arviz_summary, trace, emu, solver_str = run_simulations(model, **kwargs)
  /home/bicyclus/toymodel/toymodel.py(283)run_simulations()
-> for trace in sampler:
  /home/anaconda3/envs/pymc-env/lib/python3.10/site-packages/pymc/sampling.py(981)iter_sample()
-> for i, (strace, _) in enumerate(sampling):
> /home/anaconda3/envs/pymc-env/lib/python3.10/site-packages/pymc/sampling.py(1025)_iter_sample()
-> model = modelcontext(model)
(Pdb) start
{'large_norm': 7559.40626711426, 'small_unif': 0.2070833560593321}

In pymc/sample.py:554:
    if initial_points is None:
        # Time to draw/evaluate numeric start points for each chain.
        ipfns = make_initial_point_fns_per_chain(
            model=model,
            overrides=initvals,
            jitter_rvs=filter_rvs_to_jitter(step),
            chains=chains,
        )
        initial_points = [ipfn(seed) for ipfn, seed in zip(ipfns, random_seed_list)]

This is where the large_norm_interval__ initial values are created, and this is missing in iter_sample. I have to consult with PyMC people to find out if this is a PyMC bug or if it is the desired behaviour.

I always used pm.iter_sample to do stuff after samples got accepted (e.g., save Cyclus output to file). This is not possible anymore because of the bug. However, the (PyMC-preferred) way to do it is to use pm.sample's callback kwarg: https://www.pymc.io/projects/examples/en/latest/howto/sampling_callback.html

I will close this issue for now and continue discussion in #15 .


docs(lenis/react): fix wrong function name within the Framer Motion integration example

A small one

thank you


Support for package elm-lang/lazy at /try

People keep running into problems with recursive JSON decoding, and reporting those as issues on GitHub, e.g., https://github.com/elm-lang/core/issues/624.
To point out solutions to them, it would be really helpful if those solutions were runnable on http://elm-lang.org/try, but that requires package elm-lang/lazy to be available there.

Don't need the lazy library for this AFIAK. Just use andThen with a thunk.

Right, thanks for pointing that out. I gave a new answer over there that does not use elm-lang/lazy.


Add OpenTelemetry plugin for gateway 3.0

Summary
Add OpenTelemetry plugin for gateway 3.0.

Pushed a commit that fixes Plugin Hub rendering for maps in the examples.
Old:

New (tested working):


[upstream] [kiali] add 1.3.1

Adds Kiali Operator v1.3.1

I can't tell why this failed. No obvious errors in travis logs. I see no reason why this would fail.

@jmazzitelli I tried it locally and operator was installed successfully, just scorecard has been frozen


[EmbeddingAPI-Stability] Fix MultiXWalkViewWebsite & MultiXWalkViewVi…

…deo bugs
-Fix MultiXWalkViewWebsite & MultiXWalkViewVideo bugs
Impacted tests(approved): new 0, update 4, delete 0
Unit test platform: [Android]
Unit test result summary: pass 4, fail 0, block 0
BUG=https://crosswalk-project.org/jira/browse/XWALK-6111
BUG=https://crosswalk-project.org/jira/browse/XWALK-6114

@Shao-Feng please help to review.


Nightly check: download and use latest snapshot

Issue summary
Currently, the nightly check uses a fixed snapshot from February due to Forest limitation (it doesn't work yet for NV15). When this limitation is lifted, it should use the last snapshot.
Task summary
Download and use the last snapshot in the nightly check.

[ ] Add a cronjob to download the latest snapshot. Ensure that previous snapshots are deleted to not run out of disk space.
[ ] Modify the nightly check to use the latest snapshot. (may not be needed, it already takes the last modified .car file from $HOME/nightly_check/snapshots.

Acceptance Criteria

[ ] Nightly check is using the latest snapshot without issues.

This appears to be done (modulo from the dont-run-out-of-space part, it has been moved to a separate issue).


Error "Get-TfsWorkItem : Value cannot be null. Parameter name: values" when specifying -Fields '*'

    I'm curious about Get-TfsWorkItem and the -Fields property. The documentation says to specify a '*' to return all fields. Any ideas?

Get-TfsWorkItem -AreaPath $areaPath -WorkItemType "User Story" -Fields "*"
Get-TfsWorkItem : Value cannot be null.
Parameter name: values
At line:1 char:1

Get-TfsWorkItem -AreaPath $areaPath -WorkItemType "User Story" -F ...

  + CategoryInfo          : NotSpecified: (:) [Get-TfsWorkItem], ArgumentNullException
  + FullyQualifiedErrorId : ArgumentNullException,TfsCmdlets.Cmdlets.WorkItem.GetWorkItem

Originally posted by @sbrassard in https://github.com/igoravl/TfsCmdlets/discussions/183#discussioncomment-5102336

I have the same issue, seems that wildcard is not work correctly or i m using it in a wrong way.
I obtain alwasy: Get-TfsWorkItem: Value cannot be null. (Parameter 'values')

I found a workaround, is not a beauty solution but works.

Fixed in #214. Release available at https://github.com/igoravl/TfsCmdlets/releases/tag/v2.6.1%2B3491.1


Explicit ROS Message Support

Feature Request
One liner: Provide support for using ROS message types as inputs to the ManagedFilter class
FormaK aims to combine symbolic modeling for fast, efficient system modelling with code generation to create performant code that is easy to use.
The Five Key Elements the library provides to achieve this user experience are:

Python Interface to define models
Python implementation of the model and supporting tooling
Integration to scikit-learn to leverage the model selection and parameter tuning functions
C++ and Python to C++ interoperability for performance
C++ interfaces to support a variety of model uses

This design focuses on the 2nd and 5th elements for Python and C++ support for different use cases. Many robotic systems run in the ROS environment, so supporting ROS messages directly would allow for easier interoperability with existing systems. There may also be performance benefits (e.g. reduced memory usage) to directly supporting the message types instead of having each user re-implement an interoperability layer between ROS messages and the C++ (or Python) structures used by the FormaK runtime.
Design Decisions

Does supporting ROS require a dependency on ROS? In Python, duck typing can be used. In C++, templates could be used to provide support. If ROS isn't depend-ed upon, how will I test the interoperability?
ROS1 vs ROS2 vs supporting both. ROS: Getting Started indicates there is LTS for both.
How much effort is it worth investing in ROS-bazel configuration?

References

Potential Avenue for ROS2 support: https://github.com/mvukov/rules_ros2
Potential Avenue for ROS1 support: https://github.com/nicolov/ros-bazel

How would you recommend to handle different rates and possibly lack of exact synchronisation for the models which formak produces?
I'm envisioning using it to replace things like robot localisation. I presume having a motion model specific to your vehicle will be much better than using one very general kalman filter?

How would you recommend to handle different rates and possibly lack of exact synchronisation for the models which formak produces?

Are you referring to synchronizing inputs, or the outputs of the estimator?
For the input side, I'm working on a design for a more automated system for performing synchronization as #24 to improve the "FormaK Runtime".
In the current version of the library, there is manual work required to perform synchronization. The outline of the algorithm I'd recommend is:

Maintain a clock reference for publishing time
Queue inputs as they arrive (e.g. as ROS messages from subscribers), sorted by time
At the desired output rate, take the front of the message queue that are at or before the desired reference time
Pass that list/vector as input to the FormaK ManagedFilter .tick method (managed.tick(reference_time, list_of_messages)) and the ManagedFilter will process the messages according to their time stamp using the process model to shift in time and align with the stamped time
The output from the tick function will be the estimate of the filter for all of the messages up to the reference time

Example usage of the ManagedFilter class
Note: Right now, there's a conversion step from the ROS message to a C++ structure that I'm skipping over. This design #29 would update the library to generalize the current logic to interoperate with ROS types and reduce the effort required to perform these conversions.

I'm envisioning using it to replace things like robot localisation. I presume having a motion model specific to your vehicle will be much better than using one very general kalman filter?

In general, a more specific model will provide more "information" by constraining the estimator. For example, integrating IMU motion could be used in many vehicles/applications, but the expected motion of say a rocket launch vs a person walking would be different, so you could likely get a better estimate for each use case by using a more specialized model.
The caveat that I'd offer is that sometimes adding many extra parameters for a more complicated model can increase its sensitivity to noise, so there can be diminishing returns to making a more complicated model of the system.
One of the areas that I'd like to work on but haven't come to a solid design for is automating the model selection process to allow for choosing between, say, a general model or a specific model or even simplifying a complicated model based on data from the system.


Cleanup code

The code base still contains a lot of old code that is now unused, and should be removed. Probably there could also be some refactorings that simplify the remaining code then.

@PhilippWendler I'd deem the code quality in a pretty good state right now. I've removed nearly all of the unused code in previous commits (e.g., see commit bd08d71ee7b86b492faedfe22d78fcd59599b69a and aa70eadfe0ef00976d64ac73d4540385a1c69371), with some deliberate exceptions. The excessive use of error-values (int) as returning values seems a bit odd to me, though it is well and consistently coded, and working as intended (i.e., nothing that needs to be refactored imo).
The only thing left to do would be the use of a code formatter for a consistent code base. Do you agree with me here? I would use the linux kernel code style format (see https://www.kernel.org/doc/html/v4.10/    process/coding-style.html), because afaik that's the format that the code was previously based on (partially, at least).

I don't like the kernel coding style very much (indentation is 8 spaces). If the code is only a little bit inconsistent, we could keep it and just improve new and changed code. But if you want, you can of course also reformat it. I just would prefer to keep indentation at 2 spaces.

The code was formatted in #17 . For reference, I've used clang with a .clang-format file that contained the following specifications:
BasedOnStyle: LLVM
IndentWidth: 2
UseTab: Never
BreakBeforeBraces: Attach
AllowShortIfStatementsOnASingleLine: false
AllowShortLoopsOnASingleLine: false
AllowShortFunctionsOnASingleLine: false
IndentCaseLabels: false
ColumnLimit: 100


[SPARK-38496][PYTHON][TESTS] Add unit tests for map functions

What changes were proposed in this pull request?
Adding missing Unit tests
Why are the changes needed?
Increase test coverage for PySpark  API
Does this PR introduce any user-facing change?
No
How was this patch tested?
python/run-tests

@HyukjinKwon @itholic please review

Can one of the admins verify this patch?

Merged to master.


Update Agent Based Modeling page

cc: @Datseris

The badges look really good @hyrodium, thanks a lot for making it happen!


Add virtualFocus click handling to calendar component

I've wired up the Calendar component with the same onClick handler that is placed on the DateInput component. The DateInput's onBlur is triggered when any part of the calendar is clicked. Adding this click handler to the root of the Calendar component prevents hideCalendar from being called when navigating between months or when clicking on inconsequential portions of the calendar.
Resolves #158

Thanks, looks good. Please make sure the PR is up to date with the latest master so we are able to merge this PR.

@martijnrusschen should be good to go. :+1:


Move era.test.ts under unit-tests so that it is actually executed

Description of proposed changes

Practical tips for PR review & merge:

[ ] All GitHub Actions report PASS
[ ] Newly added code/functions have unit tests

[ ] Coverage tools report all newly added lines as covered
[ ] The positive scenario is exercised
[ ] Negative scenarios are exercised, e.g. assert on all possible errors
[ ] Assert on events triggered if applicable
[ ] Assert on changes made to storage if applicable

[ ] Modified behavior/functions - try to make sure above test items are covered
[ ] Integration tests are added if applicable/needed

Codecov Report

Merging #1320 (3195f25) into dev (70a0bc7) will decrease coverage by 0.50%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev    #1320      +/-   ##
==========================================
- Coverage   69.66%   69.17%   -0.50%
==========================================
  Files         105      105
  Lines       12220    12220
  Branches      121      121
==========================================
- Hits         8513     8453      -60
- Misses       3698     3767      +69
+ Partials        9        0       -9

see 10 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[nrf noup] boards: don't set BUILD_WITH_TFM default for 91ns

SPM is the default for NCS. Revert this change to ensure
that SPM is set for NCS.
Ref: NCSDK-9282
Signed-off-by: Håkon Øye Amundsen<EMAIL_ADDRESS>

cc @ioannisg


Checking for the existence of a symlink is fixed

Bugfix

How to reproduce the issue?

How to reproduce the issue?
The error happens sometimes. There are no clear steps for the issue
Functions is_file returns false if symlink is exists. it`s no correct
I so think. You can check code

is_file() should resolve symlinks well. What are your OS and file system?

PHP 7.1.10-1+ubuntu16.04.1+deb.sury.org+1 (cli) (built: Sep 29 2017 17:04:25) ( NTS )
php -r 'var_dump(is_file("/var/www/persdev/site"));'
bool(false)
php -r 'var_dump(is_link("/var/www/persdev/site"));'
bool(true)
php -r 'var_dump(file_exists("/var/www/persdev/site"));'
bool(true)

Seems the change doesn't pass tests: https://travis-ci.org/yiisoft/yii2/jobs/290372337

@raoptimus What returns php -r 'var_dump(is_dir("/var/www/persdev/site"));'?

This is probably duplicate of #6226.
@raoptimus Could you try master branch and check if problem still occurs?

@raoptimus What returns php -r 'var_dump(is_dir("/var/www/persdev/site"));'?
bool(true)
@raoptimus Could you try master branch and check if problem still occurs?
i`ll answer later

bool(true)

The this fix is incorrect - you can't use publishFile() for directories.


lottie: Slot overriding enhancement

These lists show the lack of slot overriding coverage, which needs improvement

[x] File > Image
[ ] Transform → Position
[ ] Transform → Scale
[ ] Transform → Radius
[ ] Transform → Opacity
[ ] Modifier > Rounded Corner
[ ] Expressions

@hermet do you have a plan for gradient override support ?

@theashraf we already support Gradient filings.

see: https://lottie.github.io/lottie-spec/latest/specs/helpers/#slottable-property


1.8 regression: pyvisa.errors.VisaIOError: VI_ERROR_TMO

Hello,
I have the following code:
import visa
rm = visa.ResourceManager('/usr/local/vxipnp/linux/lib64/libvisa.so')
inst = rm.open_resource('TCPIP::<IP_ADDRESS>::5025::SOCKET')
print(inst.query("*IDN?"))
inst.close()

I am using Linux.
The above code works fine with the 1.7 release, however with 1.8 I'm getting the following error:
File "~/src/visaTest.py", line 4, in <module>
  print(inst.query("*IDN?"))
File "/usr/lib/python2.7/site-packages/pyvisa/resources/messagebased.py", line 407, in query
  return self.read()
File "/usr/lib/python2.7/site-packages/pyvisa/resources/messagebased.py", line 332, in read
  message = self.read_raw().decode(enco)
File "/usr/lib/python2.7/site-packages/pyvisa/resources/messagebased.py", line 306, in read_raw
  chunk, status = self.visalib.read(self.session, size)
File "/usr/lib/python2.7/site-packages/pyvisa/ctwrapper/functions.py", line 1582, in read
  ret = library.viRead(session, buffer, count, byref(return_count))
File "/usr/lib/python2.7/site-packages/pyvisa/ctwrapper/highlevel.py", line 188, in _return_handler
  raise errors.VisaIOError(ret_value)

pyvisa.errors.VisaIOError: VI_ERROR_TMO (-1073807339): Timeout expired before operation completed.

I get the same result with both python2 and 3.

I cannot remember any change in that area. Can you do a bisect to find out
the guilty commit?
El 4:42 Jue Sep 3, 2015, Grey Christoforo<EMAIL_ADDRESS>escribió:

Hello,
I have the following code:
import visa
rm = visa.ResourceManager('/usr/local/vxipnp/linux/lib64/libvisa.so')
inst = rm.open_resource('TCPIP::<IP_ADDRESS>::5025::SOCKET')
print(inst.query("*IDN?"))
inst.close()
I am using Linux.
The above code works fine with the 1.7 release, however with 1.8 I'm
getting the following error:
File "~/src/visaTest.py", line 4, in
print(inst.query("*IDN?"))
File "/usr/lib/python2.7/site-packages/pyvisa/resources/messagebased.py", line 407, in query
return self.read()
File "/usr/lib/python2.7/site-packages/pyvisa/resources/messagebased.py", line 332, in read
message = self.read_raw().decode(enco)
File "/usr/lib/python2.7/site-packages/pyvisa/resources/messagebased.py", line 306, in read_raw
chunk, status = self.visalib.read(self.session, size)
File "/usr/lib/python2.7/site-packages/pyvisa/ctwrapper/functions.py", line 1582, in read
ret = library.viRead(session, buffer, count, byref(return_count))
File "/usr/lib/python2.7/site-packages/pyvisa/ctwrapper/highlevel.py", line 188, in _return_handler
raise errors.VisaIOError(ret_value)
pyvisa.errors.VisaIOError: VI_ERROR_TMO (-1073807339): Timeout expired before operation completed.
I get the same result with both python2 and 3.
—
Reply to this email directly or view it on GitHub
https://github.com/hgrecco/pyvisa/issues/180.

Any news on this?

I haven't been able to spend the time to track down the offending commit. Can you at least reproduce the error?

I see the same error

I am getting the same error message while trying to connect to my instrument.
Saw another thread with the same error (Support for TCPIP SOCKET #38, tested in June 2015). Perhaps it will narrow the timeframe of commits to search through.

Closing given that the issue is old, feel free to re-open if there is more to discuss.


TEP-0090: Matrix - Implement isSuccessful for Runs

Changes

TEP-0090: Matrix proposed executing a PipelineTask in parallel TaskRuns and Runs with substitutions from combinations of Parameters in a Matrix.
In this change, we implement the isSuccessful member function of ResolvedPipelineRunTask for matrixed Runs.
If the ResolvedPipelineRunTask is matrixed, it is successful only if all of its Runs completed successfully.
/kind feature
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[n/a] Docs included if any changes are user facing
[x] Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Release notes block below has been filled in (if there are no user facing changes, use release note "NONE")

Release Notes

Matrixed `PipelineTasks` with `Custom Tasks` are successful when all `Runs` have completed successfully.

Related PR: https://github.com/tektoncd/pipeline/pull/4980

/approve

/test pull-tekton-pipeline-integration-tests

Tagging a PR as a feature without release notes is little misleading, please add a note if possible 🙏
thank you @jerop 👎
/lgtm

@pritidesai had not included release notes because it's an implementation detail of a larger feature - added a note describing what this PR is doing though - or maybe what I should do is remove the "feature" label? 🤔

@pritidesai had not included release notes because it's an implementation detail of a larger feature - added a note describing what this PR is doing though - or maybe what I should do is remove the "feature" label? 🤔

yeah that would be helpful. I generally like to collect notes for the feature PRs while creating a GitHub release page and making sure they are highlighted.


Skip some app that don't want to be upgraded

Description of the new feature / enhancement
Hi, the idea is simple, some cracked software (not proud of that, but they're expensive) won't work once it's upgraded. Or some work requires a specific version of the software. It would be nice to allow users to choose some app that won't check the available upgrade
Proposed technical implementation details
No response

App Pinning is what you are looking for and it already has a issue tracking it

Duplicate of https://github.com/microsoft/winget-cli/issues/476

Duplicate of #476