Add Cowork to Made With Kontra?

I'm making Cowork, a gamified productivity software via Innkeeper Games. The "game" portion of it is made with Kontra, which has been perfect for this project! It's lightweight, accessible, and a breeze to build with.
Here is an image: https://i.snap.as/fH3xL7iw.png
I would love to add it here, but I understand if it's not a great fit.  It's just now starting to enter "alpha" testing, so it's not done, and the source code is private (at least for now).
Thanks for developing and maintaining this awesome software! You're enabling the creation of a bunch of cool things, too.

That's a good question. I'd say as long as it follows the [CODE_OF_CONDUCT.md guidelines I don't see any reason why it shouldn't be added.

Thank you. I am working on making some changes, including making the software open source. I will re-open this when Cowork is ready for an open-source release and submit a PR then.


Feature julian1906

Fehler in defaults.ts

There should be 3 =s. Don't see any problem that needs to be fixed here.


No float 16 support for NonMaxSuppressionV2

Hi ,
While running the object detector in keras and tensorflow in float 16 mode,
I get the following error -
TypeError: Input 'boxes' of 'NonMaxSuppressionV2' Op has type float16 that does not match expected type of float32.
Is there a way to add float 16 support for this operation here -
https://github.com/tensorflow/tensorflow/blob/9d2abd2ace95e6e352ba1292cc38c77b7bd1adc7/tensorflow/core/ops/image_ops.cc#L653
Thanks  :)
Have I written custom code
OS Platform and Distribution --- Ubuntu 16.04
TensorFlow installed from --- pip
TensorFlow version --- 1.8.0
Bazel version --- bazel release 0.13.1
CUDA/cuDNN version ---  9.2, V9.2.88
GPU model and memory - Titan X
Exact command to reproduce --- training keras retinanet but NonMaximumSupression doesnt support float 16.

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
TensorFlow version
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce

It has been 18 days with no activity and the awaiting response label was assigned. Is this still an issue?

It has been 33 days with no activity and the awaiting response label was assigned. Is this still an issue?

The feature does not make sense for the moment because the op is currently only available on CPU and float16 is not useful on CPU.

@ppwwyyxx Thanks for your comments. I agree that float16 will not make a difference for computing performance. The issue reported here is caused by that the current API does not support the float16 data type yet, but some use cases may need to handle float16 data. Considering this, do you think it will be useful for users? What do you think of this? @angersson @raghavgurbaxani


Documentation 3.1/aql tutorial

Still a lot of traffic on 3.1 docs, thus Jan2 wants it there too.
@dothebart Build fails because of

SyntaxError: C:\Daten\ArangoDB\arangodb\Documentation\Books\ppbooks\HTTP\book.json: Unexpected token @ in JSON at position 1094`

Probably caused by @GPRIORITY@, which does not get substituted I guess. Can you take a look?

WFM.


Remove IAllReadyDataAccess from ActivityApiController

Fixes #715
part of ongoing work for #567

ping @MisterJames @mheggeseth @dpaquette for review

Great PR @mgmccarthy, clean and focused. Thanks! :shipit:


Model value type

Implements parts of #12.
Compound objects (ModelNode) evaluate to true.
See test cases for behavior changes (count and a.* + b.*)

Works perfectly, thanks! 👍


Return generated keys in kundera mongoDb

How to get the inserted object primary key in kundera mongoDb
My code is below
@Entity
@Table(name = "category", schema = "aaa@mango")
public class Category implements Serializable{

	@Id
	@GeneratedValue(strategy = GenerationType.AUTO)
	private String category_id;

	@Column(name = "category_name")
	private String category_name;

	@Column(name = "created_date")
	private Timestamp created_date;
//setter and getters
}

Main class
EntityManagerFactory emf = Persistence
				.createEntityManagerFactory("aaa");
		EntityManager em = emf.createEntityManager();
		em.persist(category );

Help me

@sreekanth0041
Kundera keeps a reference to the object being persisted. So you can obtain the generated key simply by using the getter method of your field "category_id" after the object is persisted. HTH!
-Pragalbh

Yes its working.Thank you Pragalbh.

@sreekanth0041
Are we good to close on this ?
Chhavi

Yes chhavigangwal


Fix Result Time Property

https://github.com/kyverno/policy-reporter/issues/473

Pull Request Test Coverage Report for Build<PHONE_NUMBER>4
Details

3 of 5 (60.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-60.6%) to 22.232%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/violation/kubernetes/client.go
3
5
60.0%

Totals

Change from base Build<PHONE_NUMBER>:
-60.6%

Covered Lines:
1004

Relevant Lines:
4516

💛  - Coveralls


Uncaught Error: Invariant Violation: replaceState(...): Can only update a mounted or mounting component.

It appears Reflux.connect(...) doesn't check for this.isMounted() before calling this.setState(). This can cause the above error in the case where a parent and child component are listening to the same store, and the parent component includes/excludes the child based on that state:
var action =  Reflux.createAction({ asyncResult: true, children: ['started'] });

action.listen() {

    action.started();
    $.post(...)
        .done(action.completed)
        .fail(action.failed);
});

var store = Reflux.createStore({
    listenables: [action],

    state: {
        data: [],
        total: 0,
        searchInProgress: false,
    },

    getInitialState() {
        return this.state;
    },

    setState(...newState) {
        this.state = _.assign({}, this.state, ...newState);
        this.trigger(this.state);
    },

    searchStarted() {
        this.setState({ searchInProgress: true });
    },

    searchCompleted(response) {
        this.setState({ searchInProgress: false }, response);
    }
});

var Results = React.createClass({
    mixins: [Reflux.connect(store)],
    // ...
});

var Screen = React.createClass({
    mixins: [Reflux.connect(store)],

    render() {

        return <span>
            // ...

            { this.state.searchInProgress
                ? <Spinner />
                : <Results />}
        </span>
    }
});

This may be a duplicate of https://github.com/spoike/refluxjs/issues/246. I attempted to address this in https://github.com/spoike/refluxjs/pull/239 (but it's more of a hack than a true fix)

#239 was merged and released some time back.  We're now revisiting the issue in #393, as isMounted() is deprecated.


[RFE] Add a browser configuration option that allows to indicate if a url with the file:/// protocol has to be translated to a normal Windows path

Hello,
There are BrowserPicker use cases where you deal with urls with file:/// protocol.
In those cases it may sometimes be necessary for the "browser" (really any Windows program) to receive the url as a Windows path.
For this to be possible it would be necessary that BrowserPicker can be configured to indicate whether the url starting with file:/// should be converted to a path when passed as a parameter to the browser being selected.
I think it is important that it is configurable at the level of each browser because in each case it may or may not be necessary to transform that type of url.
For example, sometimes I use commands in Windows Terminal PowerShell tabs that generate as output urls starting with file:///.
Windows Terminal allows you to launch the default browser (which in this case would be BrowserPicker) by pressing Ctrl+Click on the url.
The desired goal is to invoke one of the existing programs that open the Windows context menu corresponding to the file they receive as a parameter.
Some examples of such programs are: AutohotkeyContextMenu.exe, Context.exe and WinContextMenu.exe.
Best regards,
Claudio Salvio
P.S.: thanks for making BrowserPicker available and for the effort invested in developing it.

Thanks for the suggestion, it has been implemented.


WIP: Sulu 2.4 Update , Trash Bin Feature

Waiting for this to be merged :D

@oprichnik Currently, the PR is still being worked on. Unfortunately, the implementation of the rubbish bin is more complex, as it requires a few modifications to the existing structure. However, I hope to be able to release the new version soon.


[top] No option in top_earlgrey.hjson to use .hjson that is not under hw/ip or hw/top_earlgrey

Specifically AST attr in top_earlgrey.hjson is defined as "reggen_only" so ast.hjson is taken from hw/top_earlgrey/ip/ast/data.
When using the 'real' (not open source) AST need to use a private ast.hjson that is located somewhere else (under Nuvoton's private repository) .

@sha-ron do you need for this for the full ral modeling at chip level?
We can certainly add a flag to look elsewhere is that is the cleanest solution.
@zi-v
@Jacob-Levy
Is there any way for you guys to expose a fully compatible (in terms of address map only) open source hjson?
You don't need to show any information on how any of it is used.  Ideally we would like the open source / closed source to at least match in terms of spacing / naming to test out our infrastructure.

@msfschaffner this is a bit different from the prim_csr thing, since ast isn't sitting behind another hjson or anything.

@tjaychen
yes need this for the ral modeling at chip level. A flag can be great,

is it at all possible to align the register names / address space between
open source / closed source? Because that would be the easiest way i think.
On Thu, Sep 16, 2021 at 6:39 AM sha-ron @.***> wrote:

@tjaychen https://github.com/tjaychen
yes need this for the ral modeling at chip level. A flag can be great,
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/issues/8207#issuecomment-920911568,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSXID5RBLCDZWBAOFETUCHXRVANCNFSM5EDD4KMQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Revealing the closed source names to the open source is problematic and using meaningless names in the closed makes it harder to debug and less readable.

Hi Michael,
I already updated the OS ast.hjson to match the NTIL ast.hjson
as far at the number of register and the adress space tot match the NTIL.
The name I choose for the OS are generic  and all are 32-bit R/W.
I think Sharon issue relate to the full NTIL registers DV test and it is not different that the synthesis of the full chip (secure code included).
Arnon had to develupe a flow to address the implementation and
I think  the NTIL full chip DV need to do the same from  the NTIL database
and not from the OS database.

so sounds like the conclusion is we have to have a way of picking alternative search paths, is that right?
does it make sense to say..in terms of search path priority..
alternative search path > top_earlgrey > ip?

I asked @arnonsha to look into implementing  similar solution of the implementation flow for the DV as far as the ip/ast.

so sounds like the conclusion is we have to have a way of picking alternative search paths, is that right?
does it make sense to say..in terms of search path priority..
alternative search path > top_earlgrey > ip?

As I see it yes but we will also hold an internal discussion on Sunday to make sure.
Can the alternative path be a parameter or something we configure in the hjson file?
The NTIL ast.hjson is located under the Nuvoton repository and not under top_earlgrey>ip.

i was originally thinking it would be a script input argument.. but you
have a good point.
How exactly would this be used? If it's in an hjson file, which hjson file
would our topgen read to know that there is another search path?
On Thu, Sep 23, 2021 at 9:27 AM sha-ron @.***> wrote:

so sounds like the conclusion is we have to have a way of picking
alternative search paths, is that right?
does it make sense to say..in terms of search path priority..
alternative search path > top_earlgrey > ip?
As I see it yes but we will also hold an internal discussion on Sunday to
make sure.
Can the alternative path be a parameter or something we configure in the
hjson file?
The NTIL ast.hjson is located under the Nuvoton repository and not under
top_earlgrey>ip.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/issues/8207#issuecomment-925969543,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSXUBBDOMPAJH5O6T2DUDNIPNANCNFSM5EDD4KMQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

In NTIL, we have two version of ast.hjson. One in the non-secure domain and the final one in the secure domain.
The implmentation flow runs in the secure domain. for DV we can still do most of the testing in the non-secure domain but we still need to run the DV in the secure domain to check that
the implementation data  is fully tested.

I thought we will use a new hjson that will import_cfgs the foundry chip level cfg hjson and this new hjson will include the path definition.

okay sounds good, if there is a new hjson that would make sense.
Sri what do you think? I think effectively we need to be able to pass a new
topgen argument based on hjson args.
On Thu, Sep 23, 2021 at 9:55 AM sha-ron @.***> wrote:

I thought we will use a new hjson that will import_cfgs the foundry chip
level cfg hjson and this new hjson will include the path definition.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/issues/8207#issuecomment-925989401,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSXPWZ76UQKXXOXO4LDUDNLYFANCNFSM5EDD4KMQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Talked about it internally and having a new argument will be great.

adding @imphil to see if he has any opinions on this, and whether it would work better as part of the ipgen scheme.
I'm not completely certain yet..but i feel like there might be something there.

@sha-ron I take it this is meant for top-level tests only, is that correct?
Please note that at the chip-level, SW like ROM code or drivers are relying on the open-source version of the AST register description (and all the generated constants).
We therefore need to make sure that the AST registers are correctly working using the generic open-source description only.
See this document here which describes this requirement in more detail.
So, no matter whether we add such a switch or not for debugging purposes or not, we still need to have tests in place that verify whether the AST registers work with the generic open-source AST description at the top-level. In order to avoid duplicate work it is probably better to just stick with the generic description, even though this may initially be harder to debug.

@msfschaffner
yes this is for top-level tests that use the proprietary AST.
Checking that the AST registers are working the open-source description is of course needed and we will have such tests.
But we also need to run top-level tests with the "real" AST registers (that some of them are proprietary/in the secure domain) and for that purpose we need the switch.


SAK-40264 - Cannot delete item in lessons

Seems like Lessons was already doing this
https://stackoverflow.com/a/17027553/3708872
Just not catching this new exception? Maybe this pattern will need to be used more often? I'll defer to the JPA experts. I tested and this is working though to allow for deletion again.

I'd like to assign bgarciaentornos since he's been fixing a few of these to look at but not yet in the team. Will after he accespts.


pod sandbox created without Uid cannot be deleted anymore through crictl

Description
The runtime service RunPodSandbox allows to create sandboxes that cannot be deleted using crictl by not specifying any sandbox uid.
Steps to reproduce the issue

RunPodSandbox(ctx, &v1.RunPodSandboxRequest{
		Config: &v1.PodSandboxConfig{
			Metadata: &v1.PodSandboxMetadata{
				Name:      testPod,
				Namespace: testNamespace,
			},
		},
	})

crictl rmp --all --force

Describe the results you received and expected
received
DEBU[0000] get runtime connection
getting sandbox status of pod "ec38fa6b815da5cd3ab2ac168dc8ec9133bced169d8a8790280ea7cd24ea0470": metadata.Name, metadata.Namespace or metadata.Uid is not in metadata "&PodSandboxMetadata{Name:wwcritestpod,Uid:,Namespace:wwcritest,Attempt:0,}"

expected
pod sandbox successfully got removed
What version of containerd are you using?
containerd containerd.io 1.6.21 3dce8eb055cbb6872793272b4f20ed16117344f8
Any other relevant information
crictl version
Version:  0.1.0
RuntimeName:  containerd
RuntimeVersion:  1.6.21
RuntimeApiVersion:  v1

Show configuration if it is related to CRI plugin.
#   Copyright 2018-2022 Docker Inc.

#   Licensed under the Apache License, Version 2.0 (the "License");
#   you may not use this file except in compliance with the License.
#   You may obtain a copy of the License at

#       http://www.apache.org/licenses/LICENSE-2.0

#   Unless required by applicable law or agreed to in writing, software
#   distributed under the License is distributed on an "AS IS" BASIS,
#   WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
#   See the License for the specific language governing permissions and
#   limitations under the License.

#disabled_plugins = ["cri"]

#root = "/var/lib/containerd"
#state = "/run/containerd"
#subreaper = true
#oom_score = 0

#[grpc]
#  address = "/run/containerd/containerd.sock"
#  uid = 0
#  gid = 0

#[debug]
#  address = "/run/containerd/debug.sock"
#  uid = 0
#  gid = 0
#  level = "info"

@thediveo Were you also creating the pods with crictl or were you using another client?

Not crictl but with a direct API client, https://github.com/thediveo/whalewatcher/blob/feature/cri-o/engineclient/cri/client.go that is a setting up the grpc connection and otherwise simply is the grpc CRI runtime v1 API client.
API call is here, https://github.com/thediveo/whalewatcher/blob/d439d28701afdaff39494e636f368a4218fe770e/engineclient/cri/cri_test.go#L57, originally without setting the UID in the create pod sandbox request struct.

crictl does not enforce the need for a UID either, as I was able to reproduce the error by using crictl to create a sandbox without specifying a UID.
It looks like the error message stems from here: https://github.com/kubernetes/kubernetes/blob/56f6030125d737dd031d77f45af6a413d1990862/pkg/kubelet/cri/remote/remote_runtime.go#L272C20-L272C20
It looks like the remoteRuntimeService implementation verifies the UID when removing a sandbox, but not when running a sandbox. I'm not sure if containerd is expected to enforce the existence of the sandbox UID value on creation or if that should be the responsibility of remoteRuntimeService or the client.

I used to use other configuration files, and I wanted to create a pod and container as follows.
But I did not fill in the uid:
"metadata": {
        "name": "test-sandbox",
        "namespace": "default",
},

I tried to remove it with the command crictl rmp -f -a and it failed.
After testing, you need to add a uid field to delete normally, for example as follows:
"metadata": {
        "name": "test-sandbox",
        "namespace": "default",
        "uid": "xxxxxxxxxxx"
},

Found this Python example here that also ignores the UUID and seems to get away with that: https://github.com/feiskyer/python-cri/blob/master/examples/busybox.py

There are more "plot holes", for instance, I don't have seen any definition whether the pod sandbox always is a container and thus reported as such with lifecycle events, or not. The lack of a common PID information is also unfortunate for tools that need to deal with containers on a lower level.

It looks like a fix for this has been implemented in cri-o and crictl (see the Kubernetes issue for discussion).


[Bug]: alias is not defined

What happened?
I caught this while looking through the code
Version check

[X] Yes I was.

Relevant log output
https://github.com/danielmiessler/fabric/blame/24e161686489935a68891c269912fd5d8581b035/installer/client/cli/utils.py#L396

Relevant screenshots (optional)
No response

I'm sorry, im not understanding. What error are you getting? what did you find in the code?

where is 'alias' defined in its use in the fstring here:
def find_line(self, name):
        for file in self.config_files:
            with open(file, "r") as f:
                lines = f.readlines()
            for line in lines:
                if line.strip("\n") == f"alias ${name}='{alias}'":
                    return line

find_line only takes two parameters: self and name?


New feature: dynamic code splitting support

I got a prototype working locally of using dynamic import on the server. Here's how it works:

Use Engine.EmbedHostObject to make "WebpackHelper" a globally available function
Use target: webworker in webpack.config. Since we have to roll our own file loading mechanism, the one provided by web workers was the easiest to work with.
Run a polyfill on the server before our scripts
When webpack encounters a dynamic import, it will generate a call to importScripts, which is expected to be synchronous. The polyfill will compile and execute the script, which will invoke the webpackChunk callback.
Tada! You should now be able to use a library like react-loadable to do component level code splitting.

I'm going to polish up this approach and do some testing with it before adding support to the library directly. We can release this in 4.1 if all goes well.
The JS server polyfill:
function importScripts(filename) {
	// This might break if the file is huge
	Function(WebpackHelper.ReadFile(filename))();
}

global.importScripts = importScripts;

// Allow local script debugging with Node
global.WebpackHelper = global.WebpackHelper || {
	ReadFile: function(filename) {
		var fs = require('fs');
		var vm = require('vm');
		var code = fs.readFileSync(filename, 'utf-8');
		vm.runInThisContext(code, filename);
	},
};

The webpack helper:
public sealed class WebpackHelper
{
	readonly IFileSystem _fileSystem;

	public WebpackHelper(IFileSystem fileSystem)
	{
		_fileSystem = fileSystem;
	}

	public string ReadFile(string path)
	{
		if (path == null)
		{
			throw new ArgumentNullException("path");
		}

		Console.WriteLine("Loading " + path);

		return _fileSystem.ReadAsString(path);
	}
}

There is a lot of cleanup to do, but I pushed up a working prototype here. Obviously it's ready for production, right?  🤔
https://github.com/dustinsoftware/React.NET/tree/react-loadable
I'll be pushing progress to that branch and do some testing with that approach.

I'll re-open this when I'm focused on finishing react-loadable support. It mostly works, but isn't ready to be merged in yet (needs more polish!)
If anyone has any interest, please comment on this issue :)
#655

@dustinsoftware
Have you worked on this any further?


Bulk Workflow Manager- workflowId is null

Running Bulk Workflow Manager on one particular server generates an error only on one workflow. You click start workflow, it starts, then errors immediately during the first job.
05.07.2016 11:48:48.090 *WARN* [pool-7-thread-2] com.adobe.acs.commons.workflow.bulk.impl.BulkWorkflowEngineImpl Could not find workflowId at [ Missing WorkflowId ] to purge.

05.07.2016 11:48:48.110 *ERROR* [pool-7-thread-2] com.adobe.acs.commons.workflow.bulk.impl.BulkWorkflowEngineImpl Error processing periodic execution: {}
java.lang.NullPointerException: Value should not be null (key = workflowId)
    at org.apache.sling.jcr.resource.internal.JcrModifiableValueMap.put(JcrModifiableValueMap.java:446)
    at org.apache.sling.jcr.resource.internal.JcrModifiableValueMap.put(JcrModifiableValueMap.java:47)
    at com.adobe.acs.commons.workflow.bulk.impl.BulkWorkflowEngineImpl.process(BulkWorkflowEngineImpl.java:473)
    at com.adobe.acs.commons.workflow.bulk.impl.BulkWorkflowEngineImpl.access$400(BulkWorkflowEngineImpl.java:71)
    at com.adobe.acs.commons.workflow.bulk.impl.BulkWorkflowEngineImpl$1.run(BulkWorkflowEngineImpl.java:257)
    at org.apache.sling.commons.scheduler.impl.QuartzJobExecutor.execute(QuartzJobExecutor.java:115)
    at org.quartz.core.JobRunShell.run(JobRunShell.java:202)
    at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
    at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
    at java.lang.Thread.run(Thread.java:745)

05.07.2016 11:48:48.113 *INFO* [pool-7-thread-2] com.adobe.acs.commons.workflow.bulk.impl.BulkWorkflowEngineImpl Bulk Workflow Manager stopped for [ /etc/acs-commons/bulk-workflow-manager/update_dam_7_3_911/jcr:content ]

Speaking to @davidjgonzalez in chat, he thinks it's a race condition and says he's working on a package of updates he'll include on this ticket. The only odd thing is that we haven't seen this on local or another server, nor even on another job on the same server. Race conditions can be tricky like that so it's possible, just seems odd to be so reliable in such a specific case.

@gharperhs2  these are AEM 6.1 compat builds that include a revamped impl of BWM.
Can you try them out and see if you see the same issue? In a pinch, you should be able to run the WF as Synthetic WF (now an option). This should be significantly more performant that normal non-transient WF -- naturally the WF's being run have to be compat w Synthetic to begin with.
acs-aem-commons-content-2.5.1-SNAPSHOT-min.zip
acs-aem-commons-content-2.5.1-SNAPSHOT.zip

We're doing an assessment on deploying the package, devops types are having a powwow to avoid a custom package if we can help it. It turns out the AEM instance we're looking at also won't manually kick off a single workflow instance for out-of-the-box processes, so we may need to resolve that first before we can come back to this. Right now it looks like we'll pursue that avenue before we have sufficient blame to attempt this package. I will update regardless of resolution.

@gharperhs2 thanks! yes - if you cant kick off a WF normally, then i expect you have some other more serious problems that could be contributing to the effects you're seeing in in BWM. If these other issues are at play, i doubt this package will resolve anything outside of using it to process w Synthetic WF - but you'd still have a broken instance, just one w processed assets so there's dubious value in that ;)

@davidjgonzalez Agree 100%. If it's okay I'll leave this ticket open until we can troubleshoot the single workflow issue w/ Adobe and guarantee there isn't a further issue here. If not can reopen/recreate later, no problem either way.

No problem - thanks!

@davidjgonzalez We have a meeting w/ Adobe this morning, so I should have an answer for you by EOD.

...and our Adobe contact fixed one part but broke another, pushing hard to get a final answer.

Looks like our issues were solved once Adobe fixed the server issues, so I'll have to close this. If there is a race condition we're not hitting it anymore and that's sufficient. Thanks.

Any chance you can give a general overview of what the problem stemmed from? Curious as someone else reporter similes at symptoms and I'm stumped as to what might be causing it outside some esoteric bug.

We were starting to do the csv-asset-import on a client server and kept having to toggle relevant workflows on and off so the client authors could keep working. A couple times we mixed up the process and csv-asset-import was overloading the server, so we had to nuke workflows to reset everything. The command we came up with while under fire deleted the root node for workflows when it should have removed just the instances under (/etc/workflow/instances). Restarting the server recreated the root node, but without the proper configs. Recreating a proper root node fixed our issue.

@gharperhs2 ah cool. BTW - ACS Commons has Workflow Remover which is a really handy tool for ad-hoc nuking of Workflow instances at scale. There are a number of parameters you can specify as well to only delete instances you (don't) care about.


Incorrect result of string concat() when used in SWITCH expression

There is a bug in the string concat() function that leads to incorrect result when
evaluation if(c0, concat(...), c1) or if(c0, f(c1), concat(...)).
This bug is a misuse of FlatVector<StringView>::getBufferWithSpace()
and causes the else-branch to overwrite the results from the then-branch in the if
expression.
When an expression if(c0, f(c1), g(c2)) is evaluated, f(c1) is first
evaluated on all rows where c0 is true and produces a partial result vector, then
g(c2) is evaluated on all the rest of rows and writes to the same result vector.
Therefore, the else-branch must be careful to not overwrite the existing content in
the result vector.
ConcatFunction::apply() calls flatResult->getBufferWithSpace(totalResultBytes)
to reserve enough space for the concatenation results. However, it doesn't skip
the existing content when writing to it, and doesn't update the buffer size after the
writing. These cause two issues:
When if(c0, concat(...), c1) is evaluated and c1's memory pool is
different from the pool of the EvalCtx, the else-branch copies content in c1 to the
partial result vector produced by concat(). But since concat() didn't update the
buffer size, the else-branch thinks the buffer is empty and hence overwrites the
content produced by the concat().
When if(c0, f(c1), concat(...)) is evaluated, concat() get a buffer via
getBufferWithSpace(). getBufferWithSpace() may return an existing
buffer of the result vector if it is large enough. However, concat() doesn't skip the
existing content when writing to the buffer, hence it overwrites the results produced
by the then-branch f(c1).
A unit test that reproduces this error:
TEST_F(StringFunctionsTest, concatInSwitchExpr) {
  auto data = makeRowVector(
      {makeFlatVector<bool>({true, false}),
       makeFlatVector<StringView>(
           {"This is a long sentence"_sv, "This is some other sentence"_sv})});

  auto result =
      evaluate("if(c0, concat(c1, '-zzz'), concat('aaa-', c1))", data);
  auto expected = makeFlatVector<StringView>(
      {"This is a long sentence-zzz"_sv, "aaa-This is some other sentence"_sv});
  test::assertEqualVectors(expected, result);
}

Closing this issue since the bug has been fixed.


Specify package Metadata using a pyproject.toml file

Closes #218 and closes #49
Updates

I updated all the packages to specify their metadata in a pyproject.toml which replaced all setup.py files.

Updated the package build backend to flit instead of using build as the package build back-end.

Updated the sitepdks package name from "sitepdks" to "hdl21-sitepdks"

The Pytest configuration was merged into the pyproject.toml from the pytest.ini.

Questions & Concerns

We should add the rest of the contributors to the authors list.
@ThomasPluck @kcaisley @uduse @growly @vighneshiyer @uduse @aviralpandey
Do you guys mind if I add your name and e-mail to the authors list?
If so please reply with the name and e-mail you would like included.  We can also just include your name if you'd prefer not to include your e-mail.  I added some of your names.

I dropped the twine dev dependency.  Flit also has a "flit publish" feature that can do the same thing as twine.

I couldn't share the version amongst all the packages to tie them to the vlsir version number.  Shouldn't these versions be updated independently?  We shouldn't publish a new version of a package unless something has changed.  We can just provide a major version range for the related packages.  I went ahead and updated the version range to peg them to a major revision of the related projects, e.g. vlsir and hdl21.  They're also in different states of maturity.  I would think that some of the less mature packages (e.g. the asap7 pdk)  should still be in a v0.x.x version number to reflect its immaturity.  It would be good to follow the semantic versioning recommendation to keep a package at "v0.y.z" until the api is stable.

Should we upgrade from black to ruff  in the dev depencies?  It's faster and has more features.

Testing
Tested both the pdk and main pyproject.toml by doing an install and editable install of each:

I still need to add a LICENSE file to the root of each package.  That will make sure it is included properly in the package and would be more explicit that all the packages use bsd3.

I still need to add a LICENSE file to the root of each package.  That will make sure it is included properly in the package and would be more explicit that all the packages fall under a bsd3 license.

Done

Just thought to write, thanks for this effort! Installation works for me from a poetry-managed environment with your commit with the new pyproject.toml.
I've just run this successfully based on this branch within pydantic v2 in Ubuntu py 3.10. Working on some examples so if I run into any issues will raise/fix as part of the pydantic v2 new release. Look forward to trying it out in the wild directly from pypi!
import hdl21 as h
import sky130_hdl21

@h.module
class SkyInv:
    """ An inverter, demonstrating using PDK modules """

    # Create some IO
    i, o, VDD, VSS = h.Ports(4)

    p = sky130_hdl21.Sky130MosParams(w=1,l=1)

    # And create some transistors!
    ps = sky130_hdl21.primitives.PMOS_1p8V_STD(p)(d=o, g=i, s=VDD, b=VDD)
    ns = sky130_hdl21.primitives.NMOS_1p8V_STD(p)(d=o, g=i, s=VSS, b=VSS)

CI fails, although (I think) only on a code-coverage upload step. (There's a lotta tests.)
Which (again I think) is because the source branch is on a fork?
We should test that, and probably make an issue for it.

OK never mind that codecov/ CI comment.
Seems it was a transient thing that resolved itself.
Both the re-run CI here and those on #221 (same code) both pass.


Adding playbook for a standalone PostgreSQL server.

This is a replacement for https://github.com/openmicroscopy/infrastructure/pull/302
Inventory and host vars already merged at https://github.com/openmicroscopy/management_tools/pull/540/files

Did openmicroscopy/infrastructure#302 (comment) get addressed?
No. That code was to make the install more robust in a situation I didn't find the source of - that of why some agent installs didn't "ship with" the plugin. If we want to remove those conditionals, in my view it only makes the deploy more brittle, as the source of the issues is as yet undiscovered. Also, given that code was already deployed to many other production systems, I just went with "what works". As you say, the whole check_mk plugin/agent install should be refactored out into a role.

Should the name be widened to include other PG boxes?
Potentially yes. I am spinning up a temporary one here using the same playbook, but didn't want to make it generic to support that, given it's one I'll be blowing away after use.

(additionally why is pg-int-testing in prod-hosts?)
It's what I think of a production for the team, but not production for external services.

I notice the postgresql LVM LV isn't created by this playbook, so we might want to hold ON_HOLD till I add that.

undoing --exclude label - since adding the postgresql LVM config for the VolumeGroup used here is beyond the scope of the original playbook. That piece was done manually.
Given each new DB server may have a differing disk layout (SSD/not, new VolumeGroup, etc) probably best leaving this as a manual bootstrapping step.

No. That code was to make the install more robust in a situation I didn't find the source of ...

Ok. Anything that needs capturing then?

Potentially yes. I am spinning up a temporary one here using the same playbook, but didn't want to make it generic to support that, given it's one I'll be blowing away after use.
...
It's what I think of a production for the team, but not production for external services

Guess my biggest concern is that this repo is "prod-playbooks", so the naming should mean something to someone coming in, e.g. databases.yml or pg-servers.yml?

Closing as per no activity. The need for a standalone PSQL playbook might come back and we can reuse this PR as a base.


CycloneDX License Encoding

What would you like to be added:
PR https://github.com/anchore/syft/pull/1743 changes how syft processes license encoding for the cycloneDX format.
If syft finds "other licenses", licenses that are not a valid SPDX Expression or in the SPDX license list for a given package, then it generates a list of cyclonedx.Licenses where any single valid spdx ID is it's own license, the "other licenses" are their own license, and any complex expression lifted from the source is slotted under the Name field of an individual cyclonedx license
If no other licenses are found syft merges single ID and expressions into a single SPDX expression under the expression cyclonedx field
This issue is to resolve the case where multiple complex SPDX expressions are lifted from a source along with "other licenses" - given the below test case - it's unclear if we want to drop the "other license" in favor of a merge OR parse the complex expressions into individual licenses.
{
	name: "multiple complex SPDX license expression and a non spdx id",
	input: pkg.Package{
		Licenses: []pkg.License{
			{
				SPDXExpression: "MIT AND GPL-3.0-only",
			},
			{
				SPDXExpression: "MIT AND GPL-3.0-only WITH Classpath-exception-2.0",
			},
			{
				Value: "FOOBAR",
			},
		},
	},
	expected: &cyclonedx.Licenses{
		{
			Expression: "(MIT AND GPL-3.0-only) AND (MIT AND GPL-3.0-only WITH Classpath-exception-2.0)",
		},
	},
},

Why is this needed:
Better license encoding from syft sbom --> CycloneDX format

it's unclear if we want to drop the "other license" (FOOBAR in this case)

I feel that we should never drop this kind of information. Assuming that's true, does that mean this is the answer?
(MIT AND GPL-3.0-only) AND (MIT AND GPL-3.0-only WITH Classpath-exception-2.0) AND FOOBAR


Hash with symbol keys using shorthand notation will be interpreted as NamedTuple

Discussion
I would like to discuss an edge case around the usage of Hash vs NamedTuple.
About NamedTuple:

A named tuple is a fixed-size, immutable, stack-allocated mapping of a fixed set of keys to values.
You can think of a NamedTuple as an immutable Hash whose keys (which are of type Symbol), and the types for each key, are known at compile time.

The issue is that initializing a hash with symbol keys and constant values it is then instantiated as a NamedTuple instead of a Hash. Eg.
just = {
  type: 4,
  laddr: 0,
  raddr: 0,
  state: 0
}

So when latter I want to update a value of the Hash with just[:type] = a_method(x) and error is thrown because NamedTuples are immutable.
Error: undefined method '[]=' for NamedTuple(type: Int32, laddr: Int32, raddr: Int32, state: Int32)

This comes from the confusion and the collapsing syntax between the two classes, it's not possible to determine if an object is a Hash or NamedTuple when the Hash keys are symbols and values are statically initialized.
I think NamedTuple should have been using a different syntax from the start to avoid this kind of conflicts. But now to avoid a breaking change an option could be provided to explicitly tell the Hash is a Hash and not a NamedTuple.
As a bad workaround one could initialize the Hash with Hash.new and then allocated some values. A cleaner workaround is to do a = { key: value }.to_h to ensure if it's detected as a NamedTuple it's then converted as a Hash. Another workaround a bit more annoying workaround is to use the :a => b syntax instead of a: b.
Here are test cases, however, I have no idea in mind for unpredictable type at compile type to test if the a: b syntax will lead to Hash or NamedTuple in such a case.
require "json"

a = { "test" => 5 }
puts typeof(a) # Hash because key is a string
b = { :test => 5 }
puts typeof(b) # Hash because it uses the :a => b syntax instead of a: b
c = { test: JSON.parse(File.read("/tmp/test.json"))["a"] }
puts typeof(c) # NamedTuple because dispite the type being impredicatble at compile time it cheats with a `JSON::Any` type.
d = { test: 5 }
puts typeof(d) # NamedTuple because the type is known at compile time

In the hypothesis that the classical symbol notation :a => b would be strictly reserved for Hash and the modern symbol notation a: b would be strictly reserved for NamedTuple and no plans are considered to change any actual behavior, then I suggest updating the following pages of the documentation to explicitly state it.

https://crystal-lang.org/reference/1.7/crystal_for_rubyists/index.html#types -> add a Hash section to tell that a: b will be seen as NamedTuple (Symbol shorthand notation for Hashes not supported in Crystal vs Ruby) + offer solutions
https://crystal-lang.org/reference/1.7/syntax_and_semantics/literals/hash.html -> same warning
https://crystal-lang.org/reference/1.7/syntax_and_semantics/literals/named_tuple.html -> same warning

Tell me if the assumption is correct so I can update the doc.

The issue is that initializing a hash with symbol keys and constant values it is then instantiated as a NamedTuple instead of a Hash.
[...]

I tried the following code, it works as intended:
thing = {
  :foo => "foo",
  :bar => "bar"
}
puts typeof(thing) # => Hash(Symbol, String)

thing[:baz] = "baz"
puts thing # => {:foo => "foo", :bar => "bar", :baz => "baz"}

Also note that the syntax in your example is named tuple syntax, not hash syntax which uses =>.

Here are test cases, however, I have no idea in mind for unpredictable type at compile type to test if the a: b syntax will lead to Hash or NamedTuple in such a case.

Example A and B are hashes because you're using =>, they're types are still known at compile time but it is not as strict as a named tuple, which C and D are because of :.
Having warnings in the documentation would be useful though, it's not exactly obvious that hashes use a specific syntax which can be confusing if you're coming from Ruby (or JavaScript, Python, etc).

Yes, as @devnote-dev pointed out, there is no "shorthand notation". Using a : denotes a named tuple while => denotes a hash. This is somewhat pointed out in the hash docs:
https://crystal-lang.org/reference/1.7/syntax_and_semantics/literals/hash.html

Hashes are typically created with a hash literal denoted by curly braces ({ }) enclosing a list of pairs using => as delimiter between key and value and separated by commas ,.

NamedTuple literal docs don't directly point out that you use :, but its part of the example.
I'm hesitant to think we need to document these "warnings" explicitly in the literal docs for each type. Having something more descriptive for named tuples like we have for hashes would probably be sufficient. Adding an explicit section in the Crystal for Rubyists would be a better alternative as this page fits this topic more.
Going to transfer this issue to the crystal-book repo.


Use the correct documentation comment style recognized by clang

See https://clang.llvm.org/docs/UsersManual.html#comment-parsing-options
This helps to have the doc comments carry over to the bindgen-generated Rust bindings. See https://docs.rs/bindgen/0.69.4/bindgen/struct.Builder.html#method.generate_comments

Lints will be fixed by #183

Merged #184, can you rebase?

Rebased.


Bump bk agent to v3.17.0

What this PR does / why we need it:
Update default app version to current bk version
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes # N/A
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] Chart Version bumped

👍


Unable to install plugins on Jenkins master

Cookbook version
Jenkins-2.4.1
Chef-client version
12.9.38
Platform Details
Ubuntu 14.04-LTS
Scenario:
I am trying to install plugins on Jenkins using Jenkins resource "jenkins_plugin"
Steps to Reproduce:
I am using .hpi method to install plugins.
Expected Result:
Installation of plugins ( I have mentioned plugin dependency url before installation of target plugin)
Actual Result:
``
Recipe: jenkins::nodejs_plugin  * jenkins_plugin[nodejs] action install
Recipe:
* remote_file[/var/chef/cache/nodejs-9db830c2e86512008cb5adfff203ec3c.plugin] action create[2016-04-21T14:35:24+00:00] ERROR: Timeout connecting to https://updates.jenkins-ci.org/download/plugins/nodejs/0.2.1/nodejs.hpi, retry 1/5
[2016-04-21T14:37:36+00:00] ERROR: Error connecting to https://updates.jenkins-ci.org/download/plugins/nodejs/0.2.1/nodejs.hpi, retry 2/5
[2016-04-21T14:39:49+00:00] ERROR: Error connecting to https://updates.jenkins-ci.org/download/plugins/nodejs/0.2.1/nodejs.hpi, retry 3/5
(up to date)
* remote_file[/var/chef/cache/jenkins-cli.jar] action create (up to date)
* remote_file[/var/chef/cache/update-center.json] action create (up to date)
* file[/var/chef/cache/extracted-update-center.json] action create (up to date)
================================================================================
Error executing action `install` on resource 'jenkins_plugin[nodejs]'
================================================================================

Mixlib::ShellOut::ShellCommandFailed
------------------------------------
Expected process to exit with [0], but received '255'
---- Begin output of "java" -jar "/var/chef/cache/jenkins-cli.jar" -s http://localhost:8080 install-plugin/var/chef/cache/nodejs-9db830c2e86512008cb5adfff203ec3c.plugin -name nodejs  ----
STDOUT:
STDERR: java.io.IOException: No X-Jenkins-CLI2-Port among [null, X-Required-Permission, X-Jenkins, X-You-Are-In-Group, X-Hudson, Content-Length, Expires, X-You-Are-Authenticated-As, X-Permission-Implied-By, Set-Cookie, Server, X-Content-Type-Options, Date, X-Jenkins-Session, Content-Type]
    at hudson.cli.CLI.getCliTcpPort(CLI.java:290)
    at hudson.cli.CLI.<init>(CLI.java:128)
    at hudson.cli.CLIConnectionFactory.connect(CLIConnectionFactory.java:72)
    at hudson.cli.CLI._main(CLI.java:479)
    at hudson.cli.CLI.main(CLI.java:390)
    Suppressed: java.io.IOException: Server returned HTTP response code: 403 for URL: http://localhost:8080/cli
            at sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:1628)
            at hudson.cli.FullDuplexHttpStream.<init>(FullDuplexHttpStream.java:78)
            at hudson.cli.CLI.connectViaHttp(CLI.java:158)
            at hudson.cli.CLI.<init>(CLI.java:132)
            ... 3 more
---- End output of "java" -jar "/var/chef/cache/jenkins-cli.jar" -s http://localhost:8080 install-plugin/var/chef/cache/nodejs-9db830c2e86512008cb5adfff203ec3c.plugin -name nodejs  ----
Ran "java" -jar "/var/chef/cache/jenkins-cli.jar" -s http://localhost:8080 install-plugin /var/chef/cache/nodejs-9db830c2e86512008cb5adfff203ec3c.plugin -name nodejs  returned 255

Cookbook Trace:
---------------
/var/chef/cache/cookbooks/jenkins/libraries/_executor.rb:84:in `execute!'
/var/chef/cache/cookbooks/jenkins/libraries/plugin.rb:316:in `install_plugin_from_url'
/var/chef/cache/cookbooks/jenkins/libraries/plugin.rb:116:in `block (2 levels) in <class:JenkinsPlugin>'
/var/chef/cache/cookbooks/jenkins/libraries/plugin.rb:153:in `block in <class:JenkinsPlugin>'

Resource Declaration:
---------------------
# In /var/chef/cache/cookbooks/jenkins/recipes/nodejs_plugin.rb

  3: jenkins_plugin 'nodejs' do
  4:   source 'https://updates.jenkins-ci.org/latest/nodejs.hpi'
  5:   action :install
  6:   notifies :restart, 'service[jenkins]', :immediately
  7: end

Compiled Resource:
------------------
# Declared in /var/chef/cache/cookbooks/jenkins/recipes/nodejs_plugin.rb:3:in `from_file'

jenkins_plugin("nodejs") do
  action [:install]
  retries 0
  retry_delay 2
  default_guard_interpreter :default
  declared_type :jenkins_plugin
  cookbook_name "jenkins"
  recipe_name "nodejs_plugin"
  source "https://updates.jenkins-ci.org/latest/nodejs.hpi"
end

Platform:
---------
x86_64-linux

Running handlers:
[2016-04-21T14:40:10+00:00] ERROR: Running exception handlers
Running handlers complete
[2016-04-21T14:40:10+00:00] ERROR: Exception handlers complete
Chef Client failed. 4 resources updated in 11 minutes 19 seconds
[2016-04-21T14:40:11+00:00] FATAL: Stacktrace dumped to /var/chef/cache/chef-stacktrace.out
[2016-04-21T14:40:11+00:00] FATAL: Please provide the contents of the stacktrace.out file if you file a bug report
[2016-04-21T14:40:11+00:00] ERROR: jenkins_plugin[nodejs](jenkins::nodejs_plugin line 3) had an error: Mixlib::ShellOut
::ShellCommandFailed: Expected process to exit with [0], but received '255'
---- Begin output of "java" -jar "/var/chef/cache/jenkins-cli.jar" -s http://localhost:8080 install-plugin /var/chef/cac
he/nodejs-9db830c2e86512008cb5adfff203ec3c.plugin -name nodejs  ----
STDOUT:
STDERR: java.io.IOException: No X-Jenkins-CLI2-Port among [null, X-Required-Permission, X-Jenkins, X-You-Are-In-Group, X
-Hudson, Content-Length, Expires, X-You-Are-Authenticated-As, X-Permission-Implied-By, Set-Cookie, Server, X-Content-Type-Options, Date, X-Jenkins-Session, Content-Type]
at hudson.cli.CLI.getCliTcpPort(CLI.java:290)
at hudson.cli.CLI.(CLI.java:128)
at hudson.cli.CLIConnectionFactory.connect(CLIConnectionFactory.java:72)
at hudson.cli.CLI._main(CLI.java:479)
at hudson.cli.CLI.main(CLI.java:390)
Suppressed: java.io.IOException: Server returned HTTP response code: 403 for URL: http://localhost:8080/cli
at sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:1628)
at hudson.cli.FullDuplexHttpStream.(FullDuplexHttpStream.java:78)
at hudson.cli.CLI.connectViaHttp(CLI.java:158)
at hudson.cli.CLI.(CLI.java:132)
... 3 more
---- End output of "java" -jar "/var/chef/cache/jenkins-cli.jar" -s http://localhost:8080 install-plugin /var/chef/cache
/nodejs-9db830c2e86512008cb5adfff203ec3c.plugin -name nodejs  ----
Ran "java" -jar "/var/chef/cache/jenkins-cli.jar" -s http://localhost:8080 install-plugin /var/chef/cache/nodejs-9db830c2e86512008cb5adfff203ec3c.plugin -name nodejs  returned 255
[2016-04-21T14:40:12+00:00] FATAL: Chef::Exceptions::ChildConvergeError: Chef run process exited unsuccessfully (exit code 1)
``

Any change you just installed jenkins 2.0?

Jenkins 2.0 is being installed by default. And it has auth by default.
Problem is - following attribute is not working, while installing from package (not present in apt repo).
node['jenkins']['master']['version'] = "***"

Yes its installing version 2.0 by default. How can we revert it back to previous version to make Jenkins resources work.

put the actual version in the attribute.
Also, duplicates #466

@daften what do U mean "put"?
Obviously I just put there *** for example, as neither one of versions U put there - will work.
Apt repo has no candidates for any 1.*** version.

I fixed this by doing this in a recipe (Ubuntu Linux).
node.set[:jenkins][:master][:repository] = 'http://pkg.jenkins-ci.org/debian-stable'
node.set[:jenkins][:master][:repository_key] = 'http://pkg.jenkins-ci.org/debian-stable/jenkins-ci.org.key'

This will fetch jenkins package from debian-stable repository that contains Jenkins LTS releases. LTS releases are in 1.6.x branch at least for some time.

I can't even install jenkins now.
``
================================================================================
Error executing action create on resource 'remote_file[/var/chef/cache/jenkins-ci.org.key]'
================================================================================
  Errno::ECONNREFUSED
  -------------------
  Connection refused - Connection refused connecting to http://pkg.jenkins-ci.org/debian/jenkins-ci.org.key, giving

up
  Resource Declaration:
  ---------------------
  # In /opt/chef/embedded/lib/ruby/gems/2.1.0/gems/chef-2.9.38/lib/chef/provider/apt_repository.rb

  170:         declare_resource(type, cached_keyfile) do
  171:           source new_resource.key
  172:           mode "0644"
  173:           sensitive new_resource.sensitive
  174:           action :create
  175:         end
  176:

  Compiled Resource:
  ------------------
  # Declared in /opt/chef/embedded/lib/ruby/gems/2.1.0/gems/chef-12.9.38/lib/chef/provider/apt_repository.rb:170:in`install_key_from_uri'

  remote_file("/var/chef/cache/jenkins-ci.org.key") do
    provider Chef::Provider::RemoteFile
    action [:create]
    retries 0
    retry_delay 2
    default_guard_interpreter :default
    source ["http://pkg.jenkins-ci.org/debian/jenkins-ci.org.key"]
    use_etag true
    use_last_modified true
    declared_type :remote_file
    cookbook_name "jenkins"
    mode "0644"
    atomic_update true
    path "/var/chef/cache/jenkins-ci.org.key"
  end

  Platform:
  ---------
  x86_64-linux

================================================================================
Error executing action `add` on resource 'apt_repository[jenkins]'
================================================================================

Errno::ECONNREFUSED
-------------------
remote_file[/var/chef/cache/jenkins-ci.org.key] (/opt/chef/embedded/lib/ruby/gems/2.1.0/gems/chef-12.9.38/lib/chef/p

rovider/apt_repository.rb line 170) had an error: Errno::ECONNREFUSED: Connection refused - Connection refused connecting to http://pkg.jenkins-ci.org/debian/jenkins-ci.org.key, giving up
Resource Declaration:
---------------------
# In /var/chef/cache/cookbooks/jenkins/recipes/_master_package.rb

``

@IevgenKabanets  I mean, put this in the attributes of the cookbook; role; environment; ... (whatever suits your way or working)
override['jenkins']['master']['version'] = '1.658-1.1'

The solution from @jsyrjala should indeed work for a limited time, see also https://github.com/chef-cookbooks/jenkins/blob/master/attributes/master.rb#L195
I've opened a PR that should enable jenkins2 support and would like feedback there. There are still some manual steps needed, and the checks are too limited, but I got it too work now at least.

@daften yes, I know how to set attribs for chef, but obviously it didn't work, because of the apt repo.
Will check the PR, thnx.

@IevgenKabanets  Just use my fix, without changing the apt-repo, and that should work. Or check if the version number is correct, the version number I gave is for centOS ;)

@daften
It won't work, I'm afraid. Here's why:
# apt-cache policy jenkins
jenkins:
  Installed: 1.651.1
  Candidate: 2.1
  Version table:
     2.1 0
        500 http://pkg.jenkins-ci.org/debian/ binary/ Packages
 *** 1.651.1 0
        100 /var/lib/dpkg/status

Even with override['jenkins']['master']['version'] = '1.658'. If apt-repo offers 2.1 (while 1.658 is there too).
Using fix from @jsyrjala - will install 1.651.1 (which is not latest of 1.*** versions).

That is because you're using the stable repo i'm guessing. That doesn't have 1.658. Don't just blindly try to combine solutions ;)

@daften
I'm using 'latest' repo. This one - http://pkg.jenkins-ci.org/debian/
With override attribute (same goes for 'normal' attribute).
Repo has 1.658, right?
Apt begs to differ, though.

That might be a problem with the debian/ubunte repo, i'm not sure. i'd advice you to dig deeper, because on centos, this works perfectly atm.

Yes, exactly my point... Combining solutions is not the case here.
Anyway, looking forward your PR merged.

This wrapper worked for me today on Ubuntu 14.04 (via Vagrant):
node.set[:jenkins][:master][:repository] = 'http://pkg.jenkins-ci.org/debian-stable'
node.set[:jenkins][:master][:repository_key] = 'http://pkg.jenkins-ci.org/debian-stable/jenkins-ci.org.key'
node.override['jenkins']['master']['version'] = '1.651.*'
include_recipe "jenkins::master"

My experience was that without specifying debian-stable repo, apt-policy shows only 2.8 available and the attempted 1.651.2 override fails.
STDERR: E: Version '1.651.2' for 'jenkins' was not found
apt-cache policy jenkins
jenkins:
  Installed: (none)
  Candidate: 2.8
  Version table:
     2.8 0
        500 http://pkg.jenkins-ci.org/debian/ binary/ Packages

With only setting debian-stable, the receipe defaults to the latest available as indicated by @IevgenKabanets.  However, with debian-stable, apt-policy showed 1.651.2 (today) also available:
apt-cache policy jenkins
jenkins:
  Installed: 2.7
  Candidate: 2.7
  Version table:
 *** 2.7 0
        100 /var/lib/dpkg/status
     1.651.2 0
        500 http://pkg.jenkins-ci.org/debian-stable/ binary/ Packages

And overriding to exactly 1.651.2 worked, as did specifying 1.651.* .
I view this as a workaround pending the 2.0 merge (https://github.com/chef-cookbooks/jenkins/pull/471).

Closing this out as #557 has been merged along with a few other related fixes.


Check key/secret size before casting as a fixed-length array

Current behavior
At vault_kms.rs, the data size is not being checked before casting it as a fixed-length array, which would cause an unexpected panic in case of mismatch.
Desired behavior
We need to check the data has the correct size before creating the fixed-length array so we can return a proper error. Here is an example of how we should do the check.
The functions we need to review are (every branch where use array_ref):

compute_public_key_from_secret
sign_with_secret
compute_key_id

Hey i would like to take this task

Fixed with #5765


Contributing

Can you add some docs about contributing ? Looking to help out this is a dope project.

@Josiassejod1 I have a PR up for this, would love some feedback on the docs.

perfect thanks @devlyn37


Placeholder - Include linked ILR data in F&E tool

As well as NPD data, people can apply for data extracts of 'linked' ILR (i.e. further education) and HESA (higher education) data.
Further work is need to investigate what the exact requirements are.

HESA and SWF data are not part of the NPD so are out of scope for inclusion in F&E.
Applications for HESA and SWF data will remain outside of the NPD data application process.
ILR data will be integrated into the NPD Data Tables, so needs to be added to F&E.
The title of this ticket has therefore been changed to  remove HESA and SWF.


Merge coverage result from multiple test runs

In PR #332, I separate tests into 2 runs: 1 for unit tests, and 1 for integration tests. The second test run currently overwrites the coverage report from the first test run. This PR merges the 2 coverage results instead.

@hasyimibhar  workflow file is malformed


sqlite fts5

Sqlite fts5 is faster than tantivy. Can we add this feature.

Can I clarify what feature exactly?
The FST5 module for SQLite is big and covers a lot of things (most of which tantivy already supports), or is this a request to add SQLite to the benchmark game?

I want to know why fts5 is faster than tantivy and where tantivy needs to be improved.

Can you share more information, where fts5 is faster?
Which queries on what dataset


Compile errors upgrading AWS to 1.10.5 and above - ambiguous reference in dynamodb

I'm trying to incorporate the new CodeCommit API into the project but when I upgraded to the latest version of the AWS SDK I hit this issue (see compile errors at bottom of post).
I traced back to version <IP_ADDRESS> which is the latest version that still compiles ok. I had a look at the diff of these two versions but can't figure out what the breaking change is:
https://github.com/aws/aws-sdk-java/compare/<IP_ADDRESS>...1.10.5
The only way I know how to fix this is to not use the 'wrapAsyncMethod' on these methods and just hand code the conversion of the java Future to a Scala one (not ideal).
This will need to be addressed so that we can move forward with the latest AWS developments and services.
Thanks.
[error] /home/tim/v4/aws-wrap/src/main/scala/dynamodb/dynamodb.scala:45: ambiguous reference to overloaded definition, [error] both method batchGetItemAsync in class AmazonDynamoDBAsyncClient of type (x$1: java.util.Map[String,com.amazonaws.services.dynamodbv2.model.KeysAndAttributes], x$2: com.amazonaws.handlers.AsyncHandler[com.amazonaws.services.dynamodbv2.model.BatchGetItemRequest,com.amazonaws.services.dynamodbv2.model.BatchGetItemResult])java.util.concurrent.Future[com.amazonaws.services.dynamodbv2.model.BatchGetItemResult] [error] and  method batchGetItemAsync in class AmazonDynamoDBAsyncClient of type (x$1: com.amazonaws.services.dynamodbv2.model.BatchGetItemRequest, x$2: com.amazonaws.handlers.AsyncHandler[com.amazonaws.services.dynamodbv2.model.BatchGetItemRequest,com.amazonaws.services.dynamodbv2.model.BatchGetItemResult])java.util.concurrent.Future[com.amazonaws.services.dynamodbv2.model.BatchGetItemResult] [error] match expected type (?, com.amazonaws.handlers.AsyncHandler[?,?]) => java.util.concurrent.Future[?] [error]     wrapAsyncMethod(client.batchGetItemAsync, batchGetItemRequest) [error]                            ^ [error] /home/tim/v4/aws-wrap/src/main/scala/dynamodb/dynamodb.scala:64: ambiguous reference to overloaded definition, [error] both method batchWriteItemAsync in class AmazonDynamoDBAsyncClient of type (x$1: java.util.Map[String,java.util.List[com.amazonaws.services.dynamodbv2.model.WriteRequest]], x$2: com.amazonaws.handlers.AsyncHandler[com.amazonaws.services.dynamodbv2.model.BatchWriteItemRequest,com.amazonaws.services.dynamodbv2.model.BatchWriteItemResult])java.util.concurrent.Future[com.amazonaws.services.dynamodbv2.model.BatchWriteItemResult] [error] and  method batchWriteItemAsync in class AmazonDynamoDBAsyncClient of type (x$1: com.amazonaws.services.dynamodbv2.model.BatchWriteItemRequest, x$2: com.amazonaws.handlers.AsyncHandler[com.amazonaws.services.dynamodbv2.model.BatchWriteItemRequest,com.amazonaws.services.dynamodbv2.model.BatchWriteItemResult])java.util.concurrent.Future[com.amazonaws.services.dynamodbv2.model.BatchWriteItemResult] [error] match expected type (?, com.amazonaws.handlers.AsyncHandler[?,?]) => java.util.concurrent.Future[?] [error]     wrapAsyncMethod(client.batchWriteItemAsync, batchWriteItemRequest) [error]                            ^ [error] /home/tim/v4/aws-wrap/src/main/scala/dynamodb/dynamodb.scala:127: ambiguous reference to overloaded definition, [error] both method deleteTableAsync in class AmazonDynamoDBAsyncClient of type (x$1: String, x$2: com.amazonaws.handlers.AsyncHandler[com.amazonaws.services.dynamodbv2.model.DeleteTableRequest,com.amazonaws.services.dynamodbv2.model.DeleteTableResult])java.util.concurrent.Future[com.amazonaws.services.dynamodbv2.model.DeleteTableResult] [error] and  method deleteTableAsync in class AmazonDynamoDBAsyncClient of type (x$1: com.amazonaws.services.dynamodbv2.model.DeleteTableRequest, x$2: com.amazonaws.handlers.AsyncHandler[com.amazonaws.services.dynamodbv2.model.DeleteTableRequest,com.amazonaws.services.dynamodbv2.model.DeleteTableResult])java.util.concurrent.Future[com.amazonaws.services.dynamodbv2.model.DeleteTableResult] [error] match expected type (?, com.amazonaws.handlers.AsyncHandler[?,?]) => java.util.concurrent.Future[?] [error]     wrapAsyncMethod(client.deleteTableAsync, deleteTableRequest) [error]                            ^ [error] /home/tim/v4/aws-wrap/src/main/scala/dynamodb/dynamodb.scala:143: ambiguous reference to overloaded definition, [error] both method describeTableAsync in class AmazonDynamoDBAsyncClient of type (x$1: String, x$2: com.amazonaws.handlers.AsyncHandler[com.amazonaws.services.dynamodbv2.model.DescribeTableRequest,com.amazonaws.services.dynamodbv2.model.DescribeTableResult])java.util.concurrent.Future[com.amazonaws.services.dynamodbv2.model.DescribeTableResult] [error] and  method describeTableAsync in class AmazonDynamoDBAsyncClient of type (x$1: com.amazonaws.services.dynamodbv2.model.DescribeTableRequest, x$2: com.amazonaws.handlers.AsyncHandler[com.amazonaws.services.dynamodbv2.model.DescribeTableRequest,com.amazonaws.services.dynamodbv2.model.DescribeTableResult])java.util.concurrent.Future[com.amazonaws.services.dynamodbv2.model.DescribeTableResult] [error] match expected type (?, com.amazonaws.handlers.AsyncHandler[?,?]) => java.util.concurrent.Future[?] [error]     wrapAsyncMethod(client.describeTableAsync, describeTableRequest) [error]                            ^ [error] /home/tim/v4/aws-wrap/src/main/scala/dynamodb/dynamodb.scala:191: ambiguous reference to overloaded definition, [error] both method listTablesAsync in class AmazonDynamoDBAsyncClient of type (x$1: Integer, x$2: com.amazonaws.handlers.AsyncHandler[com.amazonaws.services.dynamodbv2.model.ListTablesRequest,com.amazonaws.services.dynamodbv2.model.ListTablesResult])java.util.concurrent.Future[com.amazonaws.services.dynamodbv2.model.ListTablesResult] [error] and  method listTablesAsync in class AmazonDynamoDBAsyncClient of type (x$1: String, x$2: com.amazonaws.handlers.AsyncHandler[com.amazonaws.services.dynamodbv2.model.ListTablesRequest,com.amazonaws.services.dynamodbv2.model.ListTablesResult])java.util.concurrent.Future[com.amazonaws.services.dynamodbv2.model.ListTablesResult] [error] match expected type (?, com.amazonaws.handlers.AsyncHandler[?,?]) => java.util.concurrent.Future[?] [error]     wrapAsyncMethod(client.listTablesAsync, listTablesRequest)

It you take a look at the history for AmazonDynamoDBAsync
You can see that it changed in 1.10.5 to add a bunch of overloads.

batchGetItemsAsync as of 1.9.33
batchGetItemsAsync as of 1.10.5

Hence the issue with the ambiguous overload.
I’ll take a look to see how this ambiguity can be resolved.

@glidester fixed in PR #38


pilot error: Could not set team ID

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
I had a simple script to upload builds to TestFlight, it looked like this:
export FASTLANE_APPLE_APPLICATION_SPECIFIC_PASSWORD="some_password"
export DELIVER_ITMSTRANSPORTER_ADDITIONAL_UPLOAD_PARAMETERS="-t DAV"

bundle exec fastlane pilot upload -u<EMAIL_ADDRESS>-i $1 -q "ABCDE" --dev_portal_team_id "12345" --skip_waiting_for_build_processing true

it was working fine. Now when I run it, I get an error:
[!] The request could not be completed because:
	Could not set team ID to 'ABCDE', only found the following available teams:

- 12345 (SomeCompany)
- 123456 (SomeOtherCompany)

That seems odd, but ok, I changed ABCDE in the script to 12345, still the same error. I went to Appfile and changed team_id there to 12345, no more ABCDE in any configs, nor scripts, still the same error.
It seems it gets this id somewhere from the Apple's servers, and this might be related to recent developer id changes.
Command executed
bundle exec fastlane pilot upload
Complete output when running fastlane, including the stack trace and command used

 [✔] 🚀
[13:41:56]: Login to App Store Connect<EMAIL_ADDRESS>Looking for related GitHub issues on fastlane/fastlane...
➡  Execution fails on fastlane-0.1.14
https://github.com/fastlane/fastlane/issues/79 [closed] 5 💬
22 Sep 2016
➡  Documentation - ipa vs. ipa (shenzhen)?
https://github.com/fastlane/fastlane/issues/72 [closed] 2 💬
22 Sep 2016
➡  Extension Support
https://github.com/fastlane/fastlane/issues/76 [closed] 6 💬
11 Dec 2016
and 14336 more at: https://github.com/fastlane/fastlane/search?q=edited✓
🔗  You can ⌘ + double-click on links to open them directly in your browser.
[!] The request could not be completed because:
Could not set team ID to 'ABCDE', only found the following available teams:

12345 (SomeCompany)
123456 (SomeOtherCompany)

Environment

 ✅ fastlane environment ✅
Stack

Key
Value

OS
10.14.3

Ruby
2.3.7

Bundler?
true

Git
git version 2.17.2 (Apple Git-113)

Installation Source
/usr/local/bin/fastlane

Host
Mac OS X 10.14.3 (18D109)

Ruby Lib Dir
/System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/lib

OpenSSL Version
LibreSSL 2.6.4

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
10.1

System Locale

Variable
Value

LANG
en_US.UTF-8
✅

LC_ALL
en_US.UTF-8
✅

LANGUAGE

fastlane files:
`./fastlane/Fastfile`
# This file contains the fastlane.tools configuration
# You can find the documentation at https://docs.fastlane.tools
#
# For a list of all available actions, check out
#
#     https://docs.fastlane.tools/actions
#
# For a list of all available plugins, check out
#
#     https://docs.fastlane.tools/plugins/available-plugins
#

# Uncomment the line if you want fastlane to automatically update itself
# update_fastlane
#ENV['FASTLANE_DISABLE_COLORS'] = 1

default_platform(:ios)

platform :ios do
  desc "Push a new beta build to TestFlight"
  lane :beta_sa do
    upload_to_testflight(ipa: ENV["BUILT_IPA_PATH"])
  end

  desc "Download dSYMs from AppStore Connect and upload to Crashlytics"
  lane :refresh_dsyms_sa do
    download_dsyms(version: "latest")   # Download dSYM files from iTC
    upload_symbols_to_crashlytics      # Upload them to Crashlytics
    clean_build_artifacts               # Delete the local dSYM files
  end
  lane :upload_dsyms_sa do
    download_dsyms(version: ENV["DSYM_VERSION"], build_number: ENV["DSYM_BUILD_NUMBER"])   # Download dSYM files from iTC
    upload_symbols_to_crashlytics      # Upload them to Crashlytics
    clean_build_artifacts               # Delete the local dSYM files
  end

  lane :upload_hockey do
    hockey(api_token: ENV["HOCKEY_TOKEN"],
      public_identifier: ENV["HOCKEY_APP_ID"],
      notify: "0",
      ipa: ENV["BUILT_IPA_PATH"],
      dsym: ENV["BUILT_DSYM_PATH"],
      notes: ENV["BUILD_NOTES"],
      notes_type: "0",
      bypass_cdn: false)
  end
end

platform :android do
  lane :upload_hockey do
    hockey(api_token: ENV["HOCKEY_TOKEN"],
      public_identifier: ENV["HOCKEY_APP_ID"],
      notify: "0",
      apk: ENV["BUILT_IPA_PATH"],
      notes: ENV["BUILD_NOTES"],
      notes_type: "0",
      bypass_cdn: false)
  end
end

`./fastlane/Appfile`
app_identifier("xxxx") # The bundle identifier of your app
<EMAIL_ADDRESS># Your Apple email address

itc_team_id("12345") # App Store Connect Team ID
team_id("12345") # Developer Portal Team ID

# For more information about the Appfile, see:
#     https://docs.fastlane.tools/advanced/#appfile

fastlane gems

Gem
Version
Update-Status

fastlane
2.117.1
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

did_you_mean
1.0.0

bundler
1.17.1

io-console
0.4.5

CFPropertyList
3.0.0

concurrent-ruby
1.1.3

i18n
0.9.5

minitest
5.11.3

thread_safe
0.3.6

tzinfo
1.2.5

activesupport
4.2.11

public_suffix
2.0.5

addressable
2.6.0

atomos
0.1.3

babosa
1.0.2

claide
1.0.2

fuzzy_match
2.0.4

nap
1.1.0

cocoapods-core
1.5.3

cocoapods-deintegrate
1.0.2

cocoapods-downloader
1.2.2

cocoapods-plugins
1.0.0

cocoapods-search
1.0.0

cocoapods-stats
1.0.0

netrc
0.11.0

cocoapods-trunk
1.3.1

cocoapods-try
1.1.0

colored2
3.1.2

escape
0.0.4

fourflusher
2.0.1

gh_inspector
1.1.3

molinillo
0.6.6

ruby-macho
1.3.1

nanaimo
0.2.6

xcodeproj
1.8.1

cocoapods
1.5.3

colored
1.2

highline
1.7.10

commander-fastlane
4.4.6

declarative
0.0.10

declarative-option
0.1.0

digest-crc
0.4.1

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20180417

dotenv
2.7.1

emoji_regex
1.0.1

excon
0.62.0

multipart-post
2.0.0

faraday
0.15.4

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

faraday_middleware
0.13.1

fastimage
2.1.5

jwt
2.1.0

memoist
0.16.0

multi_json
1.13.1

os
1.0.0

signet
0.11.0

googleauth
0.6.7

httpclient
2.8.3

mime-types-data
3.2018.0812

mime-types
3.2.2

uber
0.1.0

representable
3.0.4

retriable
3.1.2

google-api-client
0.23.9

google-cloud-env
1.0.5

google-cloud-core
1.3.0

google-cloud-storage
1.16.0

json
2.2.0

mini_magick
4.5.1

multi_xml
0.6.0

plist
3.5.0

rubyzip
1.2.2

security
0.1.3

naturally
2.2.0

simctl
1.6.5

slack-notifier
2.3.2

terminal-notifier
1.8.0

unicode-display_width
1.5.0

terminal-table
1.8.0

tty-screen
0.6.5

tty-cursor
0.6.1

tty-spinner
0.9.0

word_wrap
1.0.0

rouge
2.0.7

xcpretty
0.3.0

xcpretty-travis-formatter
1.0.0

generated on: 2019-03-14

@KleMiX Heyyyy! This is totally related to recent developer changes. The team ids that were used for developer portal are no more and the ones that were originally on App Store Connect (iTunesConnect) are now also used in the developer portal. I would try either using "12345" for both or use -team_name "SomeCompany"
I personally use the team name 😊

Hi @joshdholtz, thanks for the info! Though adding -team_name gave me an error:
OptionParser::AmbiguousOption: ambiguous option: -team_name

But I tried to remove -q parameter at all, and it worked! So it is enough to pass dev_portal_team_id for uploading a build.


Use /tmp/perf-[pid].map

perf uses data from /tmp/perf-[pid].map files to symbolize stack traces -- this is required for JIT-ed languages to update the mappings dynamically. There are tools for such languages for generating these files, e.g. jcmd <java-pid> Compiler.perfmap or perf-map-agent for Java. It would be great if blazesym used these perf map files when symbolizing addresses for a process.

I was just looking at what bcc supports in terms of symbolization, and it seems it supports perf maps as well. E.g., https://github.com/iovisor/bcc/blob/1a784cfe1a2936f2c5cf0a8ee045dcdfd841ba5f/src/cc/bcc_perf_map.c

This seems to be the "documentation": https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/tools/perf/Documentation/jit-interface.txt

@TimPushkin this support is available now. Feel free to give it a try.


websocket

use private channel subscribe back {"Data":null,"Raw":null,"Error":{"Code":null,"Message":"Authentication failed: No credentials provided for private endpoint","Data":null},"Success":false}

Did you provide your API credentials?

yes，I wrote it like this：
var ws = new OKXWebSocketApiClient();
ws.SetApiCredentials("XXX", "XXX", "XXX");
var response=await ws.OrderBookTrading.Trade.SubscribeToOrderUpdatesAsync((data) =>
{
if (data != null)
{
Console.WriteLine($"{DateTime.Now.ToString("HH:mm:ss")}:{JsonConvert.SerializeObject(data)}");
// ... Your logic here
}
}, OkxInstrumentType.Any, "", "");
Console.WriteLine($"{DateTime.Now.ToString("HH:mm:ss")}:{JsonConvert.SerializeObject(response)}");
response is {"Data":null,"Raw":null,"Error":{"Code":null,"Message":"Authentication failed: No credentials provided for private endpoint","Data":null},"Success":false}

same result.  How to fix it?

The problem is completely different. I tried many times, there is no problem with login. But problem is related with your request. Even API docs says some parameters is optional, websocket returns message below
{ "event":"error", "msg":"channel:orders,instType:ANY,instFamily:,instId: doesn't exist", "code":"60018" }
I've fixed this situation, it shows correct error response now. I'll release new version today.
I wont close this issue and wait for your response.
Thanks for feedback.


Should the transaction return a callback?

It might be more beneficial since transaction is needed mostly in async callbacks. #109 I need many more examples to decide what to do with it.


update Rails/Ruby support

Hey @joecorcoran,
I think we should take a look at updating our Ruby and Rails support moving forward. I'd be down to update Appraisals/Travis once we agree on versions. My suggestion would be:
Ruby 2.2+
Rails 5.0+
What do you think?
thanks

I don't really have much of a handle on Rails version uptake these days, so if you think that sounds reasonable, go right ahead. 👍


Remove Peaks2Maps from NiMARE

Closes None, but finishes what was proposed in #615 and started in #616.

Changes proposed in this pull request:

Remove the Peaks2MapsKernel and associated functions, documentation, and examples.

Now that 0.0.12 has been released, I can finally merge this! 🎉


Remove does not check depencies!

Remove really needs to have a way to know of and check dependencies before getting rid of something!
Related issue: I need to be able to test things on a dev version easily, instead of polluting what people might be installing!

Fixed in the next commit!


B 18331 add ppm closeout office to move details

[B-18331] populate data in Closeout office field under Move Details
Summary
The QAE and TOO users should be able to see the Closeout office under the Move Details section.
Verification Steps for the Author
These are to be checked by the author.

[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] Have the Agility acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.

[ ] Has the branch been pulled in and checked out?
[ ] Have the BL acceptance criteria been met for this change?
[ ] Was the CircleCI build successful?
[ ] Has the code been reviewed from a standards and best practices point of view?

How to test

Login as a QAE role
Access a PPM move
Verify that the Closeout Office field appears under the Destination ZIP field and that it's populated with data if the Closeout Office exists.  This should be available for both Approved Shipments and Requested Shipments.
Login as a TOO role and repeat the steps above.

Frontend

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, Edge).
[ ] There are no new console errors in the browser devtools.
[ ] There are no new console errors in the test output.
[ ] If this PR adds a new component to Storybook, it ensures the component is fully responsive, OR if it is intentionally not, a wrapping div using the officeApp class or custom min-width styling is used to hide any states the would not be visible to the user.
[ ] This change meets the standards for Section 508 compliance.

Backend

[ ] Code follows the guidelines for Logging.
[ ] The requirements listed in Querying the Database Safely have been satisfied.

Warnings

:warning:
This PR does not include changes to unit tests, even though it affects app code.

  Generated by :no_entry_sign: dangerJS against 3ae207cd389c6ca49a4df9484a90b45284cb012d


invalid/incorrect swaps identified in transaction containing both swap and liquidity supply

Currently block 11608984 causes mev-inspect-py to hang at the arbitrage calculation stage. The cause seems to be transaction 0xd0bc1728db3e6cbe2769c2f534a5f7a9a5a46b13428c1a3f39c18dfce0154ec1, which contains both swap and liquidity supply actions for several tokens.
This causes a problem for mev-inspect-py, which assembles "swap" objects by identifying the transfers into/out of the liquidity pools, but when both swap and liquidity supply occur within the same transaction, it appears that invalid "swaps" are being identified where the sending and receiving token are the same. The erroneous swap objects then cause the arbitrage routing algorithm to get stuck.

Looking into this a bit deeper, it appears that this transaction is most likely an exploit of the "Emojis Farm" project. This project, launched in December 2020 and apparently inactive from around March 2021, appears to have used UniswapV2 as part of its internal tokenomics, but likely did so in an insecure way. Transaction 0xd0bc1728db3e6cbe2769c2f534a5f7a9a5a46b13428c1a3f39c18dfce0154ec1 seems somehow to be calling the UniswapV2 swap function without actually supplying tokens for the swap.
Because mev-inpect-py cannot find the inward token transfer for the swap, it instead uses an earlier transfer into the UniswapV2 pool, but as it happens that, transfer is for the same asset as the outward transfer. As a result mev-inspect-py assembles an invalid swap object in which the inward and outward transfers are for the same asset, and this invalid swap then causes the arbitrage routing algorithm to fail to terminate.


Investigate failing test update job

Modify evm-semantics/kevm script to add new commands which only just generate kore files for the tests.
Or figure out if the issue comes from adding that


[Bug]: Testigma Test cases page not Loading in Brave Browser

Is there an existing issue for this?

[X] I have searched the existing issues

What is the current behavior?
HI Testigma Team,
Just came across your awesome open source project and wanted to get involved in your open Soruce contributions Either via code
or helping you guys test if anything comes in to my notice
Since being fan of brave browser and since it's built on chromium as same as edge
What came in to my notice is that some part of your webpage is being blocked either it maybe because of whitelabelling ar maybe due to some 3rd party blocker which happened same with facebook. even that one was reported by me and you can get a reference what might be the issue (https://github.com/brave/brave-browser/issues/15629)
I am attaching a small video for reference in comparisson between brave and Edge Browser
Please do take a look and forgive me in advance if it's mistake from my side.
Brave Browser Result
https://user-images.githubusercontent.com/40737659/188262496-1d8b9d93-8ad1-49c6-a34b-7a6d92b64f01.mp4
Edge Browser Result
https://user-images.githubusercontent.com/40737659/188262549-4c518ea7-2b81-437f-90e6-e68a50a8780e.mp4
What is the expected behavior?
Expected Result
Name and Input url are hidden / blocked by the browser as compared to edge it's visible on Edge
Solution
Please create a issue on Brave to Either whitelabel your product or get assistanc from them
Steps To Reproduce
Download Brave Browser
Click on create test case
Inputs are hidden / blocked by brave browser
Version
Testsigma Cloud

HI, Can anyone acknowledge this? Will close if this is stale/ no actions if required.

@howdyAnkit, Thanks for brining it to our notice.  We will support the Brave browser in the coming release.

@PratheepV can i get assigned for this ? i'll take a look maybe unless someone's not working on it.

@howdyAnkit Glad to know that you are interested to contribute. Please go ahead raise PR, for the same.

@howdyAnkit Can you please update to the latest Testsigma version(2.7) and check ? Let us know if you are still experiencing this issue


Fix link/unlink, use Proxy to UDP and Image sinks, finally remove detrimental h264-profile workaround

Fixes #160
Helps #179

@joaoantoniocardoso let me know once it's ready for review

oops, wrong diff went into wrong commits.

@patrickelectric Okay, finally it's ready for review, sorry for the confusion

merge at will.

I'm making small changes regarding the debug messages, as suggested :)

Once this is merged, I would like to have a release of MCM and with that Blueos beta 18

Yeah, but only after the next PR.


Screen resolution changes randomly

Hello,
I don't know if I'm in the right place and sorry if I'm not.
I have an issue with my selenium-grid-extras. I'm playing with 2 grids and 17 nodes.
For the last 2 weeks, randomly, some application tests have been failing due to the screen resolution. The fact is that when I watch videos of 2 executions (one KO and the other OK). I can see that the screen resolution has changed.  It's good to know that every time I reboot my node (VM win10), I have a powershell script to set the resolution to 1920x1080. When I connect to the node with the default port 3000, I can see the wrong resolution. So I connect directly to the node via remoteControl. I find the right resolution 1920x1080. After that I refresh the URL of the node and the resolution is now good and my test passed again. This problem starts again on another node sometimes after 5min or 1-2h. I have investigated the issue without success.
All my nodes have the same conf and I only use the IE browser.
Below are the different versions I use:

SeleniumGridExtras-2.0.4
web driver: 3.141.59
iedriver_3.150.1_Win32bit

Thanks in advance for your help

Below is a screenshot of the same node before and after I connected to it :


Make few things public in pinot-java-client

Description

Making some classes and class methods for pinot-java-client public to be accessible for extending some end-to-end feature testing. E.g. we need create test scenarios to mock Pinot response with custom ExecutionStats and ResultTableResultSet. By making these public, we can also extend AbstractResultSet and make our custom response implementation for testing.
Upgrade Notes
Does this PR prevent a zero down-time upgrade? (Assume upgrade order: Controller, Broker, Server, Minion)

[ ] Yes (Please label as backward-incompat, and complete the section below on Release Notes)

Does this PR fix a zero-downtime upgrade introduced earlier?

[ ] Yes (Please label this as backward-incompat, and complete the section below on Release Notes)

Does this PR otherwise need attention when creating release notes? Things to consider:

New configuration options
Deprecation of configurations
Signature changes to public methods/interfaces
New plugins added or old plugins removed

[ ] Yes (Please label this PR as release-notes and complete the section on Release Notes)

Release Notes

Documentation

Codecov Report

Merging #7186 (e0f1458) into master (e0163f4) will decrease coverage by 8.30%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #7186      +/-   ##
============================================
- Coverage     73.54%   65.24%   -8.31%
  Complexity       92       92
============================================
  Files          1506     1506
  Lines         73808    73808
  Branches      10650    10650
============================================
- Hits          54285    48155    -6130
- Misses        15984    22250    +6266
+ Partials       3539     3403     -136

Flag
Coverage Δ

integration
?

unittests
65.24% <100.00%> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...n/java/org/apache/pinot/client/ExecutionStats.java
68.88% <ø> (ø)

.../org/apache/pinot/client/ResultTableResultSet.java
88.00% <ø> (ø)

...ava/org/apache/pinot/client/AbstractResultSet.java
66.66% <100.00%> (-6.67%)
:arrow_down:

...a/org/apache/pinot/common/metrics/MinionMeter.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../apache/pinot/common/metrics/BrokerQueryPhase.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../apache/pinot/common/metrics/MinionQueryPhase.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...pache/pinot/common/utils/grpc/GrpcQueryClient.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...pinot/minion/exception/TaskCancelledException.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...t/core/startree/plan/StarTreeDocIdSetPlanNode.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ot/common/restlet/resources/TableMetadataInfo.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 356 more

Continue to review full report at Codecov.

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

I think it'll be nice to document what these features are (that require these to be public). These classes will now be part of the public API and will have to be supported for posterity.

I think it'll be nice to document what these features are (that require these to be public). These classes will now be part of the public API and will have to be supported for posterity.

@suddendust thank you for pointing out, I have updated the description. Hope this make sense!


fixed utils.urlPatternToRegex

utils test was failing at line 25.

Changed expected result by changing
all ‘\’ to ‘\’.
Changed utils.urlPatternToRegex removed all but two
changes to pattern. and updated ‘.’ to ‘\.' .

All tests are now passing.
Might want to look into handling of reggex

@justincy Failing test in node v4+ #18 ... this is how i fixed the issue. i had to change the test. i think it has to do with a change in es5 or es6

This is no longer an issue in v1 though I can't remember how it was fixed.


Roll is always highest value

Maybe I am the luckiest person, but every roll I have completed in the last hour has been the max value.
Here is a snippet recently as example

I am dumb, forgot to put the d in front.


SQLite.swift turned many frameworks red

Hello
I installed SQLite.swift from cocoapods and it has turned the products under my project and SQLite.framework and Pods.framework red and it giving me 10 errors. I added the bridging file and added
#import <sqlite3.h>
#import "SQLite-Bridging.h"
now it gives me only 7 errors.
Undefined symbols for architecture x86_64:
"_ACAccountTypeIdentifierTwitter", referenced from:
-[PF_Twitter getLocalTwitterAccountAsync] in Parse(PF_Twitter.o)
"OBJC_CLASS$_ACAccountStore", referenced from:
objc-class-ref in Parse(PF_Twitter.o)
"OBJC_CLASS$_FBSDKAccessToken", referenced from:
objc-class-ref in ParseFacebookUtilsV4(PFFacebookUtils.o)
objc-class-ref in ParseFacebookUtilsV4(PFFacebookAuthenticationProvider.o)
"OBJC_CLASS$_FBSDKApplicationDelegate", referenced from:
objc-class-ref in ParseFacebookUtilsV4(PFFacebookUtils.o)
"OBJC_CLASS$_FBSDKLoginManager", referenced from:
objc-class-ref in ParseFacebookUtilsV4(PFFacebookAuthenticationProvider.o)
"OBJC_CLASS$_FBSDKSettings", referenced from:
objc-class-ref in ParseFacebookUtilsV4(PFFacebookAuthenticationProvider.o)
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

@sehgalsaheel Did you read through the Contributing Guidelines? Can you provide a Podfile/project with a reproducible case?

Ya I checked the Contributing Guidelines and no mistake there.
i boiled it down to 3 errors.
Podfile
platform :ios, '8.3'
use_frameworks!
pod 'SQLite.swift', git: 'https://github.com/stephencelis/SQLite.swift.git'

@sehgalsaheel So you cleaned your build, etc.? The Podfile contents you pasted above works fine in a new project as it only references SQLite.swift. Your error above seems to at the very least also include FBSDK and PFFacebookUtils.

I am new to iOS programming so this is my first time using cocoapods and I wanted to use this!
I did added those and the error count went up to 17!

is it ok for pods.framework to be red in the target->build phases-> link binary with libraries?

I solved it thanks...

@sehgalsaheel Can you comment on your solution?


Change check wallet from TRM to Chainalysis sanctions addresses

Switch to using the Chainalysis smart contract to check for sanctioned addresses instead of querying TRM.

I originally made this use chainId, and checked the contract on the chain the user was on, but then decided it's best to just always use the Ethereum contract because:

The sanctions list should always be the same on every network.
It will work better with caching (each address is cached instead of network + address combo)
This way we don't have to change the frontend.

@garethfuller It looks like this smart contract does contain addresses sanctioned by other places too. It mentions it on here: https://www.chainalysis.com/free-cryptocurrency-sanctions-screening-tools/#sanctions-list
I downloaded the UK list from https://www.gov.uk/government/publications/the-uk-sanctions-list and it only had one Ethereum address in it: 0x7FF9cFad3877F21d41Da833E2F775dB0569eE3D9 I ran that against the API and it came back as true, it is sanctioned.


Duplicate machines listed at locations in Portland (iPad)

Weird one. See screenshot. I ONLY see this in Portland. iPad only. Still there after I restart app.

If I ADD a machine to a location it only shows up once. No dupe, even after restarting app. Dunno what to make of this. Weird data glitch? Not showing up on android or iPhone, though.

Can you uninstall and reinstall? I'm not able to replicate this on my end. If you see it again after uninstall/reinstalling, try to note the steps you took to make it happen.

Resolved after uninstall and reinstall.


HashMap replaced with LookupMap

We should use near_sdk::collections instead of std::collections in our examples.

@volovyk-s looks like commit message needs to follow https://github.com/near/create-near-app#about-commit-messages to unbreak CI


Typo for label when is iOS adUnit.

Hi guys.
I was implementing the ads for my app in flutter following the codelab and found this Issue.

I mark the mistake but to explain.
When is implementing the iOS adUnit, the placeholder for it say: YOUR_ANDROID_INTERSTITIAL_AD_UNIT_ID but has to say: YOUR_IOS_INTERSTITIAL_AD_UNIT_ID instead.
Since when we create the adUnit is true you can use the same, but for tutorials intend I think is better to be more clear.
Thanks for your hard work.

Thanks for filing the issue! I'll fix the content, and will let you know once it's complete.

Just rolled out the change:) https://codelabs.developers.google.com/codelabs/admob-ads-in-flutter#4


Added gcc12 (from 11)

Upgrading the latest gcc from 11 to 12 and using C++20 for gcc_12 test.

Related to https://github.com/grpc/grpc/pull/30339


[TACHYON-768] Add util method for creating the storage directory path in tachyon.util.io.FileUtils.java

https://tachyon.atlassian.net/browse/TACHYON-768

Can any admin tells me why my local maven test all passed succussfully but here not? Thanks!

Jenkins, test this please.

@fanshiqing , Jenkins went wrong sometimes :)

Thanks for the PR, @fanshiqing ! I added a few questions/comments.

@gpang Thanks for your comments, i made some explanation above :)

LGTM

+1 LGTM
Please resolve the conflicts, though

Conflicts fixed, thanks for reviewing!

@apc999 @gpang Can one of the admins help merging this PR? Or I have to fix conflicts now and then as the master branch grows. Thanks a lot :)


feat: aio app pack command

(this is based on the WIP PoC for the aio app package command)

depends on https://github.com/adobe/aio-cli-plugin-api-mesh/pull/61

How Has This Been Tested?

npm test

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

I'm on Windows, and it was all passing -- had to reclone the repo and, npm i and npm test again to see the Windows errors (related to line endings).

Looks good, I left a couple nits. We may need to move around the pre-pack/post-pack hook timing.

@purplecabbage Is this a change we need in this PR? What changes are needed?

Also, should we skip cleaning up the output dir? While we are developing/integrating it might be good to have this easily inspectable. We do a similar thing with zipped actions where we do not discard the files so developers can easily see what is going in the zip.

We don't clean it up after the command is done - we only clean up at the start of the command

I'll merge this first thing tomorrow and do more testing, which will include #662


[Tracking]: Error Code Paths

What subject does this issue tracking?
[Subject]
While the team is focusing on the good code paths, the error code paths still require a lot of work for improving on-call experience.
These include, for example:

Better error diagnostics
Report the "how" and "why" when something breaks
Report when HMR is broken, it is currently silent

### Tasks
- [ ] #3674 Unclear error message

We have a similar tracking issue: https://github.com/web-infra-dev/rspack/issues/3459

We have a similar tracking issue: #3459

@Boshen Can we merge these issues together?

Merged.


Add /blog suffix to blog base URL in RSS generator

Currently, the RSS feed is generated with all links broken because of missing /blog suffix in base variable.
I have marked all broken links:
<?xml version="1.0" encoding="UTF-8" ?>
<rss version="2.0" xmlns:atom="http://www.w3.org/2005/Atom">
  <channel>
    <title>JSON Schema Blog RSS Feed</title>
    <link>https://json-schema.org/rss.xml</link> <!-- 👈 -->
    <atom:link rel="self" href="https://json-schema.org/rss.xml" type="application/rss+xml"/> <!-- 👈 -->
    <description>JSON Schema Blog</description>
    <language>en-gb</language>
    <copyright>Made with :love: by the JSON Schema team.</copyright>
    <pubDate>Thu, 01 Sep 2022 09:34:32 GMT</pubDate>
    <generator>next.js</generator>
    <item>
      <title>Using Dynamic References to Support Generic Types</title>
      <description>A step in the right direction for modelling data with JSON Schema</description>
      <link>https://json-schema.org/posts/dynamicref-and-generics?utm_source=rss</link> <!-- 👈 -->
      <category>blog</category>
      <guid isPermaLink="true">https://json-schema.org/posts/dynamicref-and-generics?utm_source=rss</guid> <!-- 👈 -->
      <pubDate>Wed, 31 Aug 2022 00:00:00 GMT</pubDate>
      <enclosure url="https://json-schema.org/img/posts/2022/dynamicref-and-generics/cover.webp" length="15026" type="image/webp"/> <!-- 👈 -->
    </item>
    <!-- ... -->
  </channel>
</rss>

Yikes, that's quite broken. If I still used an RSS reader, I might have picked that up. Thank you @flaksp !


GPU Particles lacks info for the Velocity Pivot value

Godot version
v4.2.beta4
System information
Windows 10.0.19045 - Vulkan (Forward+, Mobile) - GLES3 (Compatibility) - dedicated NVIDIA GeForce GTX 1650 (NVIDIA; <IP_ADDRESS>92) - Intel(R) Core(TM) i7-2600K CPU @ 3.40GHz (8 Threads)
Issue description
Velocity Pivot is used to control the pivot point of Orbit and Radial animated velocities, but the description is currently empty.

Steps to reproduce
You can use the MRP to check the value in action.
Minimal reproduction project
GPUParticle2D Orbit Pivot.zip (from #84220)

The missing description ought to be added here:
https://github.com/godotengine/godot/blob/6afd320984cf14198368cc6c53752813a02169e3/doc/classes/ParticleProcessMaterial.xml#L386-L387