Error:Cannot read properties of undefined (reading 'LogicalName')

Unfortunately I get the following error while doing the quick-start :
Fetching EDMX metadata
√ Select entities to include [type to filter list, space to select] · AttributeMetadata, EntityMetadata, OptionSetMetadata, account
√ Select actions to include [type to filter list, space to select] · No items were selected
√ Select functions to include [type to filter list, space to select] · No items were selected
No items added to configuration
√ Would you like to generate the types now? (y/N) · true
Fetching EDMX metadata
Fetching CDS metadata for AttributeMetadata

Error:Cannot read properties of undefined (reading 'LogicalName')
Stack:TypeError: Cannot read properties of undefined (reading 'LogicalName')
    at SchemaGenerator.<anonymous> (C:\Users\x\AppData\Local\npm-cache\_npx\25e184db857706b9\node_modules\dataverse-gen\lib\SchemaGenerator.js:98:37)
    at Generator.next (<anonymous>)
    at fulfilled (C:\Users\x\AppData\Local\npm-cache\_npx\25e184db857706b9\node_modules\dataverse-gen\lib\SchemaGenerator.js:5:58)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
{}

What I am doing wrong?
I tried to select different options but the info No items added to configuration does still appear.  I also tried to select only the account. This message did not display if I select a action too:
Fetching EDMX metadata
√ Select entities to include [type to filter list, space to select] · account
√ Select actions to include [type to filter list, space to select] · AddToQueue
√ Select functions to include [type to filter list, space to select] · No items were selected

Configuration updated:
0 entities(s) added
1 actions(s) added
0 functions(s) added
√ Would you like to generate the types now? (y/N) · true
Fetching EDMX metadata
Fetching CDS metadata for AttributeMetadata

Error:Cannot read properties of undefined (reading 'LogicalName')
Stack:TypeError: Cannot read properties of undefined (reading 'LogicalName')
    at SchemaGenerator.<anonymous> (C:\Users\x\AppData\Local\npm-cache\_npx\25e184db857706b9\node_modules\dataverse-gen\lib\SchemaGenerator.js:98:37)
    at Generator.next (<anonymous>)
    at fulfilled (C:\Users\x\AppData\Local\npm-cache\_npx\25e184db857706b9\node_modules\dataverse-gen\lib\SchemaGenerator.js:5:58)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
{}

But the generation does still fail.
Something else: After the installation of the required packages I got several vulnerabilities:
# npm audit report

electron  *
Severity: high
Depends on vulnerable versions of @electron/get
Renderers can obtain access to random bluetooth device without permission in Electron - https://github.com/advisories/GHSA-3p22-ghq8-v749
Compromised child renderer processes could obtain IPC access without nodeIntegrationInSubFrames being enabled - https://github.com/advisories/GHSA-mq8j-3h7h-p8g7
AutoUpdater module fails to validate certain nested components of the bundle - https://github.com/advisories/GHSA-77xc-hjv8-ww97
Electron's sandboxed renderers can obtain thumbnails of arbitrary files through the nativeImage API - https://github.com/advisories/GHSA-mpjm-v997-c4h4
IPC messages delivered to the wrong frame in Electron - https://github.com/advisories/GHSA-hvf8-h2qh-37m9
Google Chrome base/memory/weak_ptr.h WeakPtr Class Template Use-after-free DoS Weakness - https://github.com/chromium/chromium/commit/0b308a0e37b9d14a335c3b487511b7117c98d74b,https://bugs.chromium.org/p/chromium/issues/detail?id=817982,https://github.com/electron/electron/commit/3ad830c8044d3b194bfffcbf9d58c79d5b2b7e2a,https://github.com/electron/electron/pull/26438,https://github.com/electron/electron/releases/tag/v10.1.6
Google Chrome ANGLE Unspecified Out-of-bounds Read Memory Disclosure - https://github.com/electron/electron/releases/tag/v16.1.1,https://github.com/electron/electron/releases/tag/v15.4.2,https://github.com/electron/electron/releases/tag/v14.2.8,https://security-tracker.debian.org/tracker/CVE-2022-0792,https://www.debian.org/security/2022/dsa-5089,https://forums.opensuse.org/showthread.php/567514-openSUSE-SU-2022-0075-1-important-Security-update-for-chromium,https://www.forbes.com/sites/gordonkelly/2022/03/02/google-chrome-warning-upgrade-hack-security-new-chrome-update/,https://docs.microsoft.com/en-us/DeployEdge/microsoft-edge-relnotes-security#march-3-2022,https://bugs.chromium.org/p/chromium/issues/detail?id=1285885,http://cve.mitre.org/cgi-bin/cvename.cgi?name=2022-0792,https://chromereleases.googleblog.com/2022/03/stable-channel-update-for-desktop.html
Google Chrome MediaStream Unspecified Use-after-free Issue - https://github.com/electron/electron/releases/tag/v16.1.1,https://github.com/electron/electron/releases/tag/v15.4.2,https://github.com/electron/electron/releases/tag/v14.2.8,https://security-tracker.debian.org/tracker/CVE-2022-0798,https://www.debian.org/security/2022/dsa-5089,https://forums.opensuse.org/showthread.php/567514-openSUSE-SU-2022-0075-1-important-Security-update-for-chromium,https://docs.microsoft.com/en-us/DeployEdge/microsoft-edge-relnotes-security#march-3-2022,https://bugs.chromium.org/p/chromium/issues/detail?id=1283402,http://cve.mitre.org/cgi-bin/cvename.cgi?name=2022-0798,https://chromereleases.googleblog.com/2022/03/stable-channel-update-for-desktop.html
Google Chrome Unspecified High Severity Issue (1305655) - https://github.com/electron/electron/releases/tag/v14.2.9,https://github.com/electron/electron/releases/tag/v16.2.0,https://github.com/electron/electron/releases/tag/v15.5.0,https://bugs.chromium.org/p/chromium/issues/detail?id=1296876,https://bugs.chromium.org/p/chromium/issues/detail?id=1305655,https://chromereleases.googleblog.com/2022/03/stable-channel-update-for-desktop_15.html
Google Chrome ANGLE Unspecified Use-after-free Arbitrary Code Execution (2022-0978) - https://github.com/electron/electron/releases/tag/v16.2.0,https://kb.igel.com/securitysafety/en/isn-2022-07-chromium-browser-vulnerabilities-57327412.html,https://www.debian.org/security/2022/dsa-5104,https://docs.microsoft.com/en-us/deployedge/microsoft-edge-relnotes-security#march-17-2022,https://www.forbes.com/sites/gordonkelly/2022/03/16/google-issues-warning-for-millions-of-chrome-users/,https://bugs.chromium.org/p/chromium/issues/detail?id=1299264,http://cve.mitre.org/cgi-bin/cvename.cgi?name=2022-0978,https://chromereleases.googleblog.com/2022/03/stable-channel-update-for-desktop_15.html
Google Chrome GPU Unspecified Heap Buffer Overflow - https://github.com/electron/electron/releases/tag/v14.2.9,https://github.com/electron/electron/releases/tag/v16.2.0,https://github.com/electron/electron/releases/tag/v16.1.1,https://github.com/electron/electron/releases/tag/v15.4.2,https://kb.igel.com/securitysafety/en/isn-2022-07-chromium-browser-vulnerabilities-57327412.html,https://bugzilla.suse.com/show_bug.cgi?id=1197163,https://forums.opensuse.org/showthread.php/567998-openSUSE-SU-2022-0085-1-important-Security-update-for-chromium,https://www.debian.org/security/2022/dsa-5104,https://docs.microsoft.com/en-us/deployedge/microsoft-edge-relnotes-security#march-17-2022,https://www.forbes.com/sites/gordonkelly/2022/03/16/google-issues-warning-for-millions-of-chrome-users/,https://bugs.chromium.org/p/chromium/issues/detail?id=1296866,http://cve.mitre.org/cgi-bin/cvename.cgi?name=2022-0976,https://chromereleases.googleblog.com/2022/03/stable-channel-update-for-desktop_15.html
Side-channel information leakage in autofill in Google Chrome prior to 89.0.4389.72 allowed a remote attacker to obtain potentially sensitive information from process memory via a crafted HTML page. - https://crbug.com/1182767,https://chromereleases.googleblog.com/2021/03/stable-channel-update-for-desktop.html
fix available via `npm audit fix --force`
Will install<EMAIL_ADDRESS>which is a breaking change
node_modules/electron
  dataverse-auth  <=1.0.9
  Depends on vulnerable versions of electron
  node_modules/dataverse-auth

got  <11.8.5
Severity: moderate
Got allows a redirect to a UNIX socket - https://github.com/advisories/GHSA-pfrx-2q88-qq97
fix available via `npm audit fix --force`
Will install<EMAIL_ADDRESS>which is a breaking change
node_modules/got
  @electron/get  <=1.14.1
  Depends on vulnerable versions of got
  node_modules/@electron/get

4 vulnerabilities (1 low, 2 moderate, 1 high)

I installed NodeJS and did the quick-start.

Hi @PlanBernhard - I think the issue is that AttributeMetadata, EntityMetadata, OptionSetMetadata are not actually entities in the same way as account.
Once you have run dataverse-gen and added an entity, they will be in the dataverse-gen.json file:
{
  "entities": [
    "AttributeMetadata",
    "EntityMetadata",
    "OptionSetMetadata",
    "account"
  ],
...

You will need to manually remove those and leave just entity, or delete the .dataverse-gen.json file and re-run dataverse-gen init.
I'll address this in version 2 which is currently in beta. The other packages that have vulnerabilities will also be addressed in v2.
Thanks for helping with dataverse-ify!

Hi @scottdurow ,  😄
as you recommended I deleted the .dataverse-gen.json file and started npx dataverse-gen init again. This time I only selected account and it worked!
Thanks a lot for your help! 👍


bpf/nodeport: split ipv4/6 nodeport nat call into egress/ingress

Having the process running on two different tail calls will decrease
the verifier complexity as well as improving maintainability of the
code.
Signed-off-by: Sahid Orentino Ferdjaoui<EMAIL_ADDRESS>

/test

/test-1.16-4.9

Converting to draft until the comments have been addressed + IPv6 part to avoid pinging folks to do a review.

Morning Martynas, it's all addressed and with IPv6 support. I just has pushed that an hour ago :-)

it's all addressed and with IPv6 support. I just has pushed that an hour ago

Oops, sorry, I didn't refresh the page.

it's all addressed and with IPv6 support. I just has pushed that an hour ago

Oops, sorry, I didn't refresh the page.

No worries at all I should have certainly not rush at making the PR back to "ready for review", thanks for your activity on it.

/test

/test

/test

/test-1.24-net-next

/test-1.24-net-next

We can ignore the test-gke failures. Once @pchaigno has reviewed the PR, we can merge it.

Thank you for the heads-up, once it's merged I will provide the change to split snat_v4/v6_process.

@sahid noticed too late, but how does this guarantee atomicity upon up/downgrade of the datapath? Doesn't this race where pkts could for short period of time take wrong tail call?

Hello @daniel, I did not have think about any of the points that you have mentioned.
Should not we had to add new index for the new tail calls introduced and have considered a function to do the redirection to the new ones for the “old” tail call?

Does it release progs for newly added tail call numbers upon downgrade?
Certainly not, could you share with me pointers to understand what we usually do in such circumstance ?

Thanks a lot for your comments

but how does this guarantee atomicity upon up/downgrade of the datapath

I think it should be not a problem, as the downgraded datapath should switch atomically. I.e., the  downgraded progs should use different prog map. cc @ti-mo who should know better.

but how does this guarantee atomicity upon up/downgrade of the datapath

I think it should be not a problem, as the downgraded datapath should switch atomically. I.e., the downgraded progs should use different prog map. cc @ti-mo who should know better.

Ack, seems not an issue given tail call map size changes for up/downgrade and progs are maps are repopulated. Sorry for the false alarm, but just wanted to double check.

We should also have coverage for that upgrade path (assuming said tests haven't been quarantined).

We should also have coverage for that upgrade path (assuming said tests haven't been quarantined).

I doubt that we have - the migrate-svc is testing only the E/W path, but not the N/S which this PR has changed.

From the loader's perspective, as well as in many other parts of the agent, there's no such thing as an upgrade. The loader only sees a map with a different maxentries compared to the already-pinned tail call map for the endpoint, so it knows it needs to recreate the map. When the agent starts up (or when a contributor changes some BPF .c and triggers an endpoint regenerate, same thing), there are 2 scenarios:

The tail call map's properties (type, k, v, maxentries, flags) are different, so the map needs to be recreated. In this case: build ELF, load ELF from disk, see map properties have changed, move old map, create new map, pin new map, load all progs in the ELF (including entrypoint) into the kernel, put all prog fds into new tail call map (one by one..), atomically replace bpf entrypoint on the qdisc/xdp.
Map properties are the same (not grown or shrunk), so the same pinned map is used. Build ELF, load ELF, open pinned map, load all progs into the kernel, put all prog fds into the pinned map one by one (which is still actively used by an existing qdisc/xdp), only then replace the entrypoint.

Now I think about it, re-using a pinned tail call map causes an inconsistent view of the world while the new progs are being inserted into the existing map. If we move some logic from e.g. tail call 1 to tail call 11, packets are still handled while we're repopulating the tail call map. This could cause packets to be accepted or dropped erroneously.
I propose we remove the map migration concept entirely. Not only because it complicates the loader process, but also because the gains are negligible. The modulo between the two scenarios is the bpffs dir rename (and removal afterwards) and creation of the new tail call map, which consumes only a small amount of memory.
Will create an issue and pick this up as part of the ebpf-go refactoring tasks.

We should also have coverage for that upgrade path (assuming said tests haven't been quarantined).

I doubt that we have - the migrate-svc is testing only the E/W path, but not the N/S which this PR has changed.

:grimacing: Sounds like something we should fix.

grimacing Sounds like something we should fix.

https://github.com/cilium/cilium/issues/13530


preview and slides must looks the same

Preview and slide have to be the same, not always the case, especially when changing plot, closing spectacle presentation, reopening. Changes are reflected on the slides, but not in the preview.

I can confirm that slide preview do not stay up-to-date with the actual slide contents.


Snapshot: Simulators Boot and Shutdown quickly, Test Marked as Success with no errors but no screenshots taken

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
The issue occurs when running the snapshot within via a Fast lane file. I have multiple configurations within a test plan that determine which state the app is launched in. However, when the screenshots command is run the simulators do no boot successfully unless the simulator has just been ran on Xcode. It almost appears to be a caching issue.
When running the simulator alongside snapshot, I can see the simulators starting but not completing passed the load screen and thus, the screenshots are not taken. Can someone help please?
The UI tests work fine within Xcode and through running the UI test command on the terminal.
Complete output when running fastlane, including the stack trace and command used

Cannot do so due to position. No error messages are received. Red x's are marked against the simulators. Test is marked as success but no tests are run.

This looked to be an issue with the application. Closing this Issue. Thanks.


Remove inplace op in mistral patcher

What does this PR do?

OpenVINO exporter fails with this op even though ONNX exporter is okay with it.
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

Who can review?

Thanks, can you open a PR in Transformers as well?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


04/07/2018 17:50 : Create Pull Request for branch prTesting_2018040717 -> master

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Properly check that modin/pandas/test/dataframe/test_map_metadata.py defaults to pandas

We need to ensure that the tests we expect to default to pandas actually default to pandas.

Cross-reference with https://github.com/modin-project/modin/issues/3955


Using reCAPTCHA v3 in the frontend

Hello, while this is not specific to the PHP client library, but I hope to get some answers about the use of the new reCAPTCHA v3.
Basically I have a contact form in PrestaShop, and I want to be able to add reCAPTCHA v3 so that there is no extra interaction for users while making sure I don't get spammed from bots. From the docs page, the idea being brought forward is to add the script tag with the link toapi.js with the render GET parameter ?render=reCAPTCHA_site_key. Doing so you will not need to run grecaptcha.render() manually, which is an important point (more on this later).
Now, I have to add grecaptcha.execute() in the callback of grecaptcha.ready(). Here I can define an action, and finally in the then() promise fulfillment I get a token. I take this token and update the value attribute of a hidden input tag in the form, named g-recaptcha-response. This way it gets sent to the server alongside with the regular form fields and I can make the server verification in ContactController::postProcess() method.
Since I want users to fill the form with several fields and a message, they might take even minutes to submit the form. I'm also using jQuery to run on the form submission, doing some validation checks and in case calling event.preventDefault() and showing an alert if some required fields are empty or invalid.
In my tests I found out that tokens expire rather quickly. Since we don't manually run grecaptcha.render() as mentioned above, we cannot specify an expired-callback to run when the token expires. At this point I resorted to add a setInterval with a 60000 delay (1 minute) to run again grecaptcha.execute().
Is this the right way to use the frontend library? Is it correct to run multiple checks once per minute? Will this affect users' score?
$(document).ready(function () {
  function reCAPTCHA_execute () {
    // grecaptcha instantiated by external script from Google
    // reCAPTCHA_site_key comes from backend
    grecaptcha.execute(reCAPTCHA_site_key, { action: 'contactForm' }).then(function (token) {
      $('[name="g-recaptcha-response"]').val(token);
    }, function (reason) {
      console.log(reason);
    });
  }

  if (typeof grecaptcha !== 'undefined' && typeof reCAPTCHA_site_key !== 'undefined') {
    grecaptcha.ready(reCAPTCHA_execute);
    setInterval(reCAPTCHA_execute, 60000);
  }
});

Initially I thought of running grecaptcha.execute() on form submission, in the same callback function that does data validation, but since grecaptcha.execute() is asynchronous, I cannot stop the form submission (event.preventDefault()) and the resume in the promise fulfillment.
  $('.contact-form-box').on('submit', function (e) {
    // Check if email is empty
    if ($('#email').val() === '') {
      e.preventDefault();
      window.alert(contact_emptyEmail);
      return;
    }

    // Check if email is not valid
    if (!validate_isEmail($('#email').val())) {
      e.preventDefault();
      window.alert(contact_invalidEmail);
      return;
    }

    // etc…

    // here I cannot do grecaptcha.execute() because it's asynchronous,
    // and stopping the form submission cannot be reverted.
  });

I haven't used the V3 API yet, but this flow works on the V2 invisible version.

var frm = document.forms.item(0);

frm.addEventListener('submit', function (e) {
    e.preventDefault();
    grecaptcha.execute();
});

function formSubmit(response) {
    frm.submit();
}

So your code should probably be something like this
$('.contact-form-box').on('submit', function (e) {
    var frm = this;
    e.preventDefault();

    // Check if email is empty
    if ($('#email').val() === '') {
        window.alert(contact_emptyEmail);
        return;
    }

    // Check if email is not valid
    if (!validate_isEmail($('#email').val())) {
        window.alert(contact_invalidEmail);
        return;
    }

    // etc…

    grecaptcha.execute(reCAPTCHA_site_key, { action: 'contactForm' }).then(function (token) {
      $('[name="g-recaptcha-response"]').val(token);
      frm.submit();
    }, function (reason) {
      console.log(reason);
    });
});```

Hmm, when you do frm.submit(); wouldn't that be blocked by e.preventDefault(); since the the whole $('.contact-form-box').on('submit', …) would be called again?
Does the library remove any event listeners on the form? If that was the case, does it do it when the execute() is called or after the first call of the listener? In the former case it would be okay since I can first do all the checks and validations in my form event listener and only call execute() when the form is okay and ready to be submitted.  Originally I was testing a similar setup but it seemed not to work that way though…

https://developer.mozilla.org/en-US/docs/Web/API/HTMLFormElement/submit

When invoking this method directly, no submit event is raised (in particular, the form's onsubmit event handler is not run),  and constraint validation is not triggered either.

Wow, thanks! You never stop learning!

I've been trying to figure this out, and while this looks promising, doesn't the (token) always get returned no matter what? I think you have to take that token and submit it to the google verify API server side in order to get the actual valid or unvalid response. Otherwise this method would submit the form no matter what.
Can anyone confirm this?

That’s correct, in my case the PHP server verifies the token with Google to
get the score etc…
David Gasperoni

Sorry to hijack, but we've implemented the recaptchaV3 action on form submit and are noticing a lengthy delay (> 1000 milliseconds) after users click the submit button.
      grecaptcha.ready(function () {
        const account_id = 'XXXXXXXXXX';
        $('#my-form').submit(async function(event) {
          event.preventDefault();
          const $this = $(this);
          await grecaptcha.execute(account_id, {action: 'form_submission'})
            .then(function(token) {
              $('#token-field').val(token);
          });
          $this.unbind('submit').submit();
        });
      });

Is this expected?  Otherwise, the only other thing I can see is refreshing the token every 120 seconds on the client system to work around the "lag on submit" we're seeing.  Would these sorts of token refreshes have negative impacts?
Thanks for your consideration.


Spring AOP Advices applied again on every Rebind

I have a problem where my Advisor gets applied multiple times to the same Bean. The setup is following:
    @Bean(name = "systemAviationCoreRestService")
    @ConfigurationProperties(prefix = "aviationcore.rest")
    public CxfJaxRsClientFactoryBean<AviationCoreRestService> systemAviationCoreRestService() {
        return new CxfJaxRsClientFactoryBean<>(AviationCoreRestService.class);
    }

With a PostProcessor:
public class JaxRsBeanPostProcessor extends AbstractAdvisingBeanPostProcessor {
// .. snip ..
    @Override
    protected void evaluateProxyInterfaces(Class<?> beanClass, ProxyFactory proxyFactory) {
        boolean hasAtLeastOneInterfaceToProxy = false;
        for (Class<?> ifc : ClassUtils.getAllInterfacesForClass(beanClass, getProxyClassLoader())) {
            if (hasAnnotationAnywhere(ifc, javax.ws.rs.Path.class)
                    && hasAnnotationAnywhere(ifc, ExceptionMapping.class)) {
                proxyFactory.addInterface(ifc);
                hasAtLeastOneInterfaceToProxy = true;
            }
        }

        if (!hasAtLeastOneInterfaceToProxy) {
            throw new RuntimeException(
                "expected at least one JaxRs interface to proxy in bean class \"" + beanClass + "\""
            );
        }

        proxyFactory.setProxyTargetClass(false);
    }

The AviationCoreRestService matches all constraints (isEligible = true). Now the problem is that when starting the context my systemAviationCoreRestService Bean gets the advice applied and is proxied using a JDKProxy not a cglib proxy (proxyFactory.setProxyTargetClass(false);).
So after the first round everything works as expected, then spring cloud comes and refreshes the context, which calls org.springframework.cloud.context.properties.ConfigurationPropertiesRebinder#rebind(java.lang.String) with my systemAviationCoreRestService bean.
At this point this is called:
				this.applicationContext.getAutowireCapableBeanFactory().destroyBean(bean);
				this.applicationContext.getAutowireCapableBeanFactory()
						.initializeBean(bean, name);

InitializeBean re-applies the PostProcess from above to the already proxied bean because of
(org.springframework.aop.framework.AbstractAdvisingBeanPostProcessor#postProcessAfterInitialization)
		if (bean instanceof Advised) {
			Advised advised = (Advised) bean;
			if (!advised.isFrozen() && isEligible(AopUtils.getTargetClass(bean))) {
				// Add our local Advisor to the existing proxy's Advisor chain...
				if (this.beforeExistingAdvisors) {
					advised.addAdvisor(0, this.advisor);
				}
				else {
					advised.addAdvisor(this.advisor);
				}
				return bean;
			}
		}

I suspect the issue is here: https://github.com/spring-cloud/spring-cloud-commons/blob/master/spring-cloud-context/src/main/java/org/springframework/cloud/context/properties/ConfigurationPropertiesRebinder.java#L96 - The check should be AopUtils.isAopProxy not AopUtils.isCglibProxy.

I opened a PR #289 with a proposed fix, please review carefully maybe there is a better solution or the solution may not be complete.


C# refactor menu only shows "Surround with" options

Type: Bug
I am coding in C# for a Unity project.
I want extract some code as a method from a C# class.
Here's the example code :
public class test{
	void ToExtract()
    {
        bool test1 = false;
    }
}

I want to select the line inside my function and extract as a method :
private void NewMethod()
{
     bool test1 = false;
}

When I press ctrl+shift+r or ctrl+. the only options shown are Surround With : #if , #region, snippets and others.

VS Code version: Code 1.72.0 (64bbfbf67ada9953918d72e1df2f4d8e537d340e, 2022-10-04T23:20:39.912Z)
OS version: Windows_NT x64 10.0.19043
Modes:
Sandboxed: No
Extensions (4)

Extension
Author (truncated)
Version

doxdocgen
csc
1.4.0

csharpier-vscode
csh
1.3.5

unity-code-snippets
kle
1.3.0

csharp
ms-
1.25.0

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263cf:30335440
pythondataviewer:30285071
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593cf:30376535
pythonvs932:30410667
cppdebug:30492333
vsclangdf:30486550
c4g48928:30535728
dsvsc012cf:30540253
azure-dev_surveyone:30548225
i497e931:30553904
vsccc:30566497
pyindex848:30577860
40g7c324:30573242

/causedByExtension

This issue is caused by an extension, please file it with the repository (or contact) the extension has linked in its overview in VS Code or the marketplace for VS Code. See also our issue reporting guidelines. If you don't know which extension is causing the problem, you can run Help: Start extension bisect from the command palette (F1) to help identify the problem extension.
Happy Coding!


Feature request : prune subcommand

I started to work on a pacman hook and his prune script.
https://github.com/eoli3n/zectl-pacman-hook
I get into some little problems which could be fixed by some zectl query arguments.
Would you check TOFIX lines and tell me what's possible or not ? (It's not usable as is, just a POC to show the idea).
https://github.com/eoli3n/zectl-pacman-hook/blob/master/zectl-prune.sh#L4
Maybe you should consider managing prune subcommand in zectl ?
Something like borg prune would be great : https://github.com/eoli3n/dotfiles/blob/wayland/roles/borgbackup/files/backup#L34
See you.

I'll definitely think about it, I don't want to add too much extra code that isn't core to the tool, so I'm hesitant to add extra commands, but I will consider it. I do think it would be possible to get the functionality you looking for with your script.
Reviewing your TOFIXes I have some questions/feedback.

L4

TOFIX: 'zectl' needs an 'unset' subcommand

Are you just looking to "erase" a property?
be_active_ds="$(zpool get bootfs -H -o value)"
be_root="${be_active_ds%/*}"
zfs inherit org.zectl:yourprop "${be_root}"

L24

TOFIX: 'zectl get -H nonexistantkey' should print an error and exit 1

Can you work with just having the property empty -? I'm trying to follow zfs get as closely as possible, and that is the behaviour it uses for a non-existent user property.
L47

TOFIX: 'zectl list' needs a positionnal arg to filter/query 'ACTIVE|BOOTED|NONE' envs

I would rather not add a argument just to remove non N/R/RN environments.
L52

TOFIX: 'zectl list' needs a positionnal arg to order by date

This, I would like to just do by default. If you'd like to open an issue for a feature request to remind me, I will put it on my list. I'm not promising any time frame though. We also take PRs :relaxed:

Are you just looking to "erase" a property?
Thanks it worked.

❯ zfs get all zroot/ROOT | grep zectl
zroot/ROOT  org.zectl:pacmanhook             yes                              local
zroot/ROOT  org.zectl:bootloader             systemdboot                      local
zroot/ROOT  org.zectl:org.zectl.pacmanhook   yes                              local
zroot/ROOT  org.zectl:pacmanhook-prunecount  5                                local

~
❯ sudo zfs inherit -r org.zectl:org.zectl.pacmanhook zroot/ROOT

~
❯ zfs get all zroot/ROOT | grep zectl
zroot/ROOT  org.zectl:pacmanhook             yes                              local
zroot/ROOT  org.zectl:bootloader             systemdboot                      local
zroot/ROOT  org.zectl:pacmanhook-prunecount  5

Can you work with just having the property empty -?

I do with this -> https://github.com/eoli3n/zectl-pacman-hook/blob/e654483d2d461c0407990ff7b15607cc08bb5746/zectl-prune.sh#L31

I'm trying to follow zfs get as closely as possible

I agree with this.

I would rather not add a argument just to remove non N/R/RN environments.

So let's keep piping into grep -Ev '\ N\ |\ NR\ |\ R\ ' : https://github.com/eoli3n/zectl-pacman-hook/blob/e654483d2d461c0407990ff7b15607cc08bb5746/zectl-prune.sh#L48

We also take PRs

I would like to, but i'm not a C dev :)
Let's rename that issue. Thanks for your answer !

About the prune subcommand, i think that i would be a useful feature.
Creating snapshots is a thing, but it's more powerful when automated, and it's harder to manage snapshots rotation well without a prune function.
Here's a good base of what it could be : https://borgbackup.readthedocs.io/en/stable/usage/prune.html

AUR hook package in online ;) https://aur.archlinux.org/packages/zectl-pacman-hook/


Support notification in archci.yml

Support email like how drone archived.
image: golang:1.4
env:
  - GOROOT=/usr/local/go
  - PATH=$PATH:$GOROOT/bin:$GOPATH/bin
script:
  - go test success_test.go
notify:
  email:
    recipients:
      -<EMAIL_ADDRESS>  webhook:
    urls:
      - https://webhooks.gitter.im/e/$$GITTER_KEY
    on_started: false
    on_success: true
    on_failure: true

Now the .archci.yml file looks like this.
image: golang:1.4

env:
  - GOPATH=/root/go
  - PATH=$PATH:$GOPATH/bin

script:
  - go get -v
  - go test success_test.go

email:
  success:
    -<EMAIL_ADDRESS>  failure:
    -<EMAIL_ADDRESS>
webhook:
  success:
    - http://<IP_ADDRESS>:10010/v1/hook/gitlab/push
  failure:
    - http://<IP_ADDRESS>:10010

And it's implemented by https://github.com/ArchCI/simple-worker/commit/8118de01d1a07b1bca1781c6efb628dca0c3e0d9

The webhook works like a charm and we will implement to send email later.


Extra rows for first PR after release

The first two rows show commit tags, then indicates that the "before" is a versioned commit (this is the first PR after tagging a new release). This is taking up valuable vertical space in the table, and it might be extract these rows into a single text line above or below the table, e.g.
Comparing commit [abe797f] (<v0.1.2>) and [0304e9]. Click [here] to view all benchmarks.

| Before | After | Ratio | Method   |
| ------ | ----- | ----- | -------- |
|    14s |   10s |   400 | nonsense |

This additional row would presumably also show up for commits to main and nightly runs after tagged releases.

Need to test with the latest version of ASV, and see if this is still an issue or not.

I investigated this behavior and with the previous versions of asv (< 0.6) we would indeed get an extra row specifying the version tags. On the latest versions, however, they seem to appear next to the "Before" and "After" column headers, the same way branch names usually appear when opening a PR against main.
I agree that we should extract these values, especially because branch names can get quite large. I just opened a PR to extract them and add them to a one-liner before the table.
Input table
All benchmarks:

Change
Before [a4e3082d] <0.0.1~59>
After [6f9dea7d] <new-branch>
Ratio
Benchmark (Parameter)

724±600ms
1.83±0.4s
~2.53
benchmarks.TimeSuite.time_range

671±300ms
1.55±0.7s
~2.30
benchmarks.TimeSuite.time_keys

1.25±0.3s
2.35±1s
~1.88
benchmarks.TimeSuite.time_xrange

1.26±0.4s
1.15±0.8s
0.91
benchmarks.TimeSuite.time_iterkeys

-
5.98k
3.1k
0.52
benchmarks.MemSuite.mem_list

Formatted table
Comparing <0.0.1~59> and <new-branch>

Before [a4e3082d]
After [6f9dea7d]
Ratio
Benchmark (Parameter)

724±600ms
1.83±0.4s
~2.53
benchmarks.TimeSuite.time_range

671±300ms
1.55±0.7s
~2.30
benchmarks.TimeSuite.time_keys

1.25±0.3s
2.35±1s
~1.88
benchmarks.TimeSuite.time_xrange

1.26±0.4s
1.15±0.8s
0.91
benchmarks.TimeSuite.time_iterkeys

5.98k
3.1k
0.52
benchmarks.MemSuite.mem_list


ASB-184: acceptance tests added for search functionality on project d…

…ocuments page

:tada: This PR is included in version 1.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[Proposal] Use more modern tools

Hi @pcottle,
I really like learnGitBranching, it looks exactly like what I had mind for a side project of mine. :slightly_smiling_face:
Instead of reinventing the wheel or even forking your project, I'd like to contribute.
Would you accept a pull request to replace some of the tools by more modern ones?
Looking at the codebase I suggest:

Bumping the dependencies.
Adding prettier.
Replacing JSHint with ESLint
Maybe use a more modern flavor of JavaScript (e.g. let instead of var).
Replacing grunt with webpack.
Adding a "proper" watch mode.

I think this would make this project more attractive for contributors.
This would also be a good way for me to become familiar with the codebase as
I plan to help you close some issues afterward, if you are OK with that.
What do you think about this?

That'd be absolutely awesome @dorian-marchal ! I agree on making it more attractive to new contributors -- its quite clearly old (in terms of JS community years) when it comes to the tools and choices made.
I push production on a separate branch in case but like to keep master healthy, so that's usually my only advice. thankfully we have the jenkins integration though so that should keep us honest.
Look forward to any/all improvements! (Also a huge fan of prettier at work)

(closing out old issues, but thankfully a lot of this did get improved!)

(closing out old issues, but thankfully a lot of this did get improved!)


Refine Requirement: Chatbot to provide access to search engine

Requirement needs to be analyzed for the state it is in in the current application (it may appear fully complete), which should then drive questions to the customer about what they truly want. Document findings as a comment here. Questions can be asked to the team or put into the "Customer Questions" document in the General/Files area.
#34 will help with this if completed, alternatively you can run the app on your own.
Consider highlighting potential redundancies/distinctions between #21, #14, #20, #15, #16
If there is already a good baseline for this in place within the application, consider benefits of potentially placing this requirement as a sub-requirement of each individual feature to be split amongst both teams instead of Team B taking this task across the board in order to reduce cross-team dependencies.

From the customer meeting:
o	Is the chatbot intended to be able to access all current and planned application functions? (changing user views, scanning, listing/adding/removing notifications, search, performing link actions within messages)
Yes
o	Is this intended to perform any functions external to the app (ex. Open browser with search query or search a user’s device storage beyond what is being stored as a result of the OCR function)?
Only in-app


Cloves are overpowering

What the title says. When I take a sip my tongue goes numb, the cloves are way too overpowering

Odd, considering there are just two cloves in the entire mixture. I'll experiment with just adding one and see if it improves the recipe


Do not merge/buildkit testing

👮🏻👮🏻👮🏻 !!!!  DO NOT FORGET !!!! 👮🏻👮🏻👮🏻
PR Checklist:

[ ] Have you run the unit, integration and smoke tests as described here?
[ ] If you added public APIs, did you write the JavaDocs?
[ ] If the changes are of interest to application developers, have you added them to the changelog (/docs/source/changelog.rst), and potentially the release notes (/docs/source/release-notes.rst)?
[ ] If you are contributing for the first time, please read the contributor agreement now and add a comment to this pull request stating that your PR is in accordance with the Developer's Certificate of Origin.

Thanks for your code, it's appreciated! :)

@E1P3 can also mark as a draft in this case via GH UI -(I've done this)


Links do not work in app

Links appear as links (underlined and in different color) however when you press on it nothing happens. Ironically, when you share the event as a text message, the link does work
But there's weird formatting

UPDATE: on the app end the problem was there was a space in front of the https BUT we still need to fix the formatting of the sharing of the event through text messages. Is it possible to change the hyperlink into a link when we share events?

Yes, when we share them, we can probably do something similar to the reformatting that we do when displaying it. Simon did this in the RichDescription widget.


[shell] Add 'server' command to shell app.

Problem
The shell app has limited functionality. It doesn't provide access to core device functionality such as pairing or IM/ZCL command handling.
Change overview
Adds the server command to provide the ability to server start or server stop the equivalent of the all-clusters-app peripheral. Once the server is started, one can pair the device with chip-tool and send commands to it.
One can examine what sessions are active with the server sessions command. The ZCL structure can be inspected with the server endpoints and server clusters commands.
This work is an incremental step towards instrumenting early multi-device topology testing, such as using a chip-tool or chip-repl controller to manage two server nodes where each server is on a different port. This model could provide a flexible basis for scripted functional tests where each node is chip-shell running in a process to serve as a simulated server node, and a collection of such nodes can form a topology for more advanced N-node tests.
The work is also a step toward making the chip-shell capable of being either a client or a server (or potentially both with enough rework of the core).
Testing
Local testing on linux and darwin.
CI for other embedded builds.

This has been ready-to-go.  Hopefully we can muster enough reviewers to approve and merge it.


Use maven local repo for dependencies

I have those artifacts installed in my local maven repository but sbt is trying to download them.

Since splice engine uses maven for the build, the plugin should use it too (and cut the time needed for dependency resolution / download).

Added the Resolver.mavenLocal resolver in https://github.com/jaceklaskowski/splice-machine-spark-connector/commit/e8b1b0422c1f5320b823c2862bbc275ecdae1f81.


List of string_scan.cpp bugs.

Unfortunately, string_scan.cpp, unlike a lot other programs - comes shipped with a lot of bugs. Here are a list of currently-found bugs. This issue will be closed once all bugs are addressed.
If you found a new bug, please open a new issue here and I will look into it.
Issues that are currently found:

The if loop seems very problematic (huge bug). The code snippet seems unable to accurately scan all words.

Last recurring word will not be counted inside the loop.

I will look into it. Thanks, @Jeffrey10th for your hard work.


Error in catalog

Expected Behavior
I am expecting to be able to see the catalog without any issues.
Actual Behavior
I am seeing the error in catalog. for example if I open /catalog/default/resource/live-db-slave I am seeing this error.
Steps to Reproduce
Here I would like to share my settings where you can see what I am configuring and how. I am honestly wondered that its able to scrape but still saying that there are some problems..... :(

here is a config what I am using: txt is just for a github adding.
app-config.production.yaml.txt
use this config
Seeing this error.

Context
Its affecting me the way that I am not able to use the backstage or configure the components etc...
If you want any screenshots or anything else please let me know I can explain anything and send you everything :peanuts: :smiley_cat:
Your Environment

Browser Information: Chrome, Firefox

OS: Ubuntu

Server side: Docker container with connection to DB

Gitlab: 14.10 selfhosted

Output of yarn backstage-cli info:
yarn run v1.22.19
error Command "backstage-cli" not found.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

can be closed issue solved


Ability to teleport a window on screen

Description of the new feature / enhancement
A way to tell that you want this app, or the active app, to move its window on screen
Scenario when this would be used?
Every time you change your settings in a multiscreen environment, somehow, all your windows are not on your screen anymore (including the window to change the screen layout),
Some can be moved with the shift right-click on the taskbar icon, But it's meh, the problem is some applications, don't support that at all. Just try to move the window on the screen! ;p
Supporting information
No response

Thank you for the feedback!
Your request mainly concerns the Windows behavior to persist windows' positions in a multi-screen environment and PowerToys cannot do much about it. Please report your idea to the Windows Feedback hub app


fix: upgrade TypeScript and async

The previous PR to upgrade axios and async merged but the release ultimately failed because the updates introduced TypeScript code that was not supported by the version of TypeScript in the project. So version 2.6.2 "exists" but was not published.
This PR upgrades TypeScript to version 4.7.4 which can deal with the new syntax that caused the issue in the previous PR. As a result, ts-node and typedoc also had to be updated to be inline with the updated version of TypeScript. Finally, the async package also gets updated to address a vulnerability.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Spec: promote "Attribute Limits" up one heading level

Context: in v1.10.0 of the spec, subsections were created under the level-3 heading Attribute Limits. Prior to that these were pseudo-sections introduced by a heading formatted as a bolded regular paragraph.
Consider promoting the section Attribute Limits (and its subsections) up one heading level, in https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/common/common.md, so that it appears in the auto-generated TOC.

Should that file be named README.md?

Followup to #1273

PR submitted:

#3861


Does not read subtitles.

WebTorrent version:  0.3.3 (0-a5366c3)
Node.js version: 5.0.0
Browser name/version (if using WebTorrent in the browser):

Unfortunately I've just come to report a bug, not to fix it as I do not know how. Anyways, I noticed that subtitles are not recognized

This will be fixed in the next version of the app. Will be v0.4.0. Released in the next couple of days.

Awesome! Thanks for the reply man, and great work so far. Its a great project!


Added custom_cypher_query_corrector

custom_cypher_query_corrector is a custom function provided by user to correct the generated query. It is observed that the generated query is in right format but not as per the question asked. custom_cypher_query_corrector will enable end user to customize the validation of cypher as per business need.
Thank you for contributing to LangChain!

[ ] PR title: "package: description"

Where "package" is whichever of langchain, community, core, experimental, etc. is being modified. Use "docs: ..."  for template changes, "infra: ..." for CI changes.
Example: "community: add foobar LLM"

[ ] PR message: Delete this entire checklist and replace with

Description: a description of the change
Issue: the issue # it fixes, if applicable
Dependencies: any dependencies required for this change
Twitter handle: if your PR gets announced, and you'd like a mention, we'll gladly shout you out!

[ ] Add tests and docs: If you're adding a new integration, please include

a test for the integration, preferably unit tests that do not rely on network access,
an example notebook showing its use. It lives in docs/docs/integrations directory.

[ ] Lint and test: Run make format, make lint and make test from the root of the package(s) you've modified. See contribution guidelines for more: https://python.langchain.com/docs/contributing/

Additional guidelines:

Make sure optional dependencies are imported within a function.
Please do not add dependencies to pyproject.toml files (even optional ones) unless they are required for unit tests.
Most PRs should not touch more than one package.
Changes should be backwards compatible.
If you are adding something to community, do not re-import it in langchain.

If no one reviews your PR within a few days, please @-mention one of baskaryan, efriis, eyurtsev, hwchase17.

@baskaryan Please check the PR

@physaikat there are some lint errors


iOS: No OpKernel was registered to support Op 'Conv2d' with these attrs

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
YES

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Mac OS 10.12.6 (16G29)

TensorFlow installed from (source or binary):
I only build the iOS library from source
when I trained the model, I install it use pip

TensorFlow version (use command below):
1.7.0

Python version:

Bazel version (if compiling from source):

GCC/Compiler version (if compiling from source):

CUDA/cuDNN version:
Not use GPU

GPU model and memory:

Exact command to reproduce:
build_all_ios.sh -g mymodel.pb

Describe the problem
I use command list below to generate the ops_to_register.h file. and build the iOS library use "build_all_ios.sh -g mymodel.pb"
$ bazel build tensorflow/python/tools/print_selective_registration_header
$ bazel-bin/tensorflow/python/tools/print_selective_registration_header \
  --graphs=graph.pb > tensorflow/core/framework/ops_to_register.h

After that , I config my Xcode and run the model on an iphone, but it issues error:
Could not create TensorFlow Graph: Invalid argument: No OpKernel was registered to support Op 'Conv2D' with these attrs.  Registered devices: [CPU], Registered kernels:

 [[Node: init_conv/Conv2D = Conv2D[T=DT_FLOAT, data_format="NHWC", dilations=[1, 1, 1, 1], padding="VALID", strides=[1, 4, 4, 1], use_cudnn_on_gpu=false](Pad, init_conv/weights/read)]]

but the conv*.cc is in the tf_op_files.txt,  Is there some error on my usage?

Nagging Assignee @jart: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

This question is better asked on StackOverflow since it is not a bug or feature request. There is also a larger community that reads questions there.
If you think we've misinterpreted a bug, please comment again with a clear explanation, as well as all of the information requested in the issue template. Thanks!

@asdemon have you solved this problem? i am facing the same problem when try to use Conv2d in iOS

@jart @asdemon @chanwkaa I checked my ops_to_register.h which was generated by
bazel-bin/tensorflow/python/tools/print_selective_registration_header \
--graphs=my_frozen_graph_def.pb > tensorflow/core/framework/ops_to_register.h

and confirmed ios build with -O3 -DANDROID_TYPES=__ANDROID_TYPES_FULL__ -DSELECTIVE_REGISTRATION -DSUPPORT_SELECTIVE_REGISTRATION
however, there are still errors exactly the same as you:
Invalid argument: No OpKernel was registered to support Op 'Conv2D' with these attrs.  Registered devices: [CPU], Registered kernels:
  <no registered kernels>

	 [[{{node FeatureExtractor/MobilenetV1/MobilenetV1/Conv2d_0/BatchNorm/batchnorm/mul_1}} = Conv2D[T=DT_FLOAT, data_format="NHWC", dilations=[1, 1, 1, 1], padding="SAME", strides=[1, 2, 2, 1], use_cudnn_on_gpu=true](Preprocessor/sub, FeatureExtractor/MobilenetV1/Conv2d_0/weights/read/_24__cf__27)]]

the source of ops_to_register.h contains:
    constexpr const char* kNecessaryOpKernelClasses[] = {
"BinaryOp< CPUDevice, functor::add<float>>",
"BinaryOp< CPUDevice, functor::add<int32>>",
"AssertOp",
"BiasOp<CPUDevice, float>",
"CpuCastOp",
"ConcatV2Op<CPUDevice, ::tensorflow::int32>",
"ConcatV2Op<CPUDevice, float>",
"ConstantOp",
"Conv2DOp<CPUDevice, float>",
"DepthwiseConv2dNativeOp<CPUDevice, float>",
"EnterOp",
"BinaryOp< CPUDevice, functor::equal_to<int32>>",
"ExitOp",
"UnaryOp< CPUDevice, functor::exp<float>>",
"ExpandDimsOp<int32>",
"FillOp<CPUDevice, ::tensorflow::int32, int32>",
"FillOp<CPUDevice, float, int32>",
"GatherOp<CPUDevice, float, int32>",
"BinaryOp< CPUDevice, functor::greater<float>>",
"BinaryOp< CPUDevice, functor::greater<int32>>",
"IdentityOp",
"BinaryOp< CPUDevice, functor::less<int32>>",
"LoopCondOp",
"BinaryOp< CPUDevice, functor::maximum<float>>",
"MergeOp",
"BinaryOp< CPUDevice, functor::minimum<float>>",
"BinaryOp< CPUDevice, functor::minimum<int32>>",
"BinaryOp< CPUDevice, functor::mul<float>>",
"BinaryOp< CPUDevice, functor::mul<int32>>",
"NextIterationOp",
"NoOp",
"NonMaxSuppressionV2Op<CPUDevice>",
"PackOp<CPUDevice, ::tensorflow::int32>",
"PackOp<CPUDevice, float>",
"PlaceholderOp",
"RangeOp<::tensorflow::int32>",
"BinaryOp< CPUDevice, functor::div<float>>",
"Relu6Op<CPUDevice, float>",
"ReshapeOp",
"ResizeBilinearOp<CPUDevice, float>",
"ShapeOp<int32>",
"UnaryOp< CPUDevice, functor::sigmoid<float>>",
"SizeOp<int32>",
"SliceOp<CPUDevice, float>",
"SplitOpCPU<float>",
"SqueezeOp",
"StridedSliceOp<CPUDevice, ::tensorflow::int32>",
"BinaryOp< CPUDevice, functor::sub<float>>",
"BinaryOp< CPUDevice, functor::sub<int32>>",
"SwitchOp",
"TensorArrayPackOrGatherOp<CPUDevice, ::tensorflow::int32, false >",
"TensorArrayPackOrGatherOp<CPUDevice, float, false >",
"TensorArrayReadOp<CPUDevice, ::tensorflow::int32>",
"TensorArrayReadOp<CPUDevice, float>",
"TensorArrayUnpackOrScatterOp<CPUDevice, ::tensorflow::int32, false >",
"TensorArrayUnpackOrScatterOp<CPUDevice, float, false >",
"TensorArraySizeOp",
"TensorArrayOp",
"TensorArrayWriteOp<CPUDevice, ::tensorflow::int32>",
"TensorArrayWriteOp<CPUDevice, float>",
"TileOp<CPUDevice, int32>",
"TopK<CPUDevice, float>",
"TransposeCpuOp",
"UnpackOp<CPUDevice, float>",
"WhereCPUOp<bool>",
"ZerosLikeOp< CPUDevice, float>",
"RecvOp",
"SendOp",
};

as you can see, "Conv2DOp<CPUDevice, float>", has been registered but the session still cannot run because of it. So generally, I don't think you can get an answer from stack overflow, however, it is apparently a bug.

bazel:  0.16.0
TensorFlow: r1.10
OS: macOS 10.13.6
Xcode: Version 9.4.1 (9F2000)
python: 3.6
target: iOS simulator 10.3, x86_64

If that's still an issue you can take a look at #9073


Report back guest OS Info obtained from guest agent

Is this a BUG REPORT or FEATURE REQUEST?:
/kind enhancement
What happened:
When a Guest VM is deployed, the VMI status does not report back the Guest OS Information(OS Name,  Kernel version etc).That information would be valuable to have.
What you expected to happen:
Expose Guest OS Info  in VMI Status similar to how Interface IP addresses are reported back.
How to reproduce it (as minimally and precisely as possible):
Deploy a VM and check VMI status
Anything else we need to know?:
If the experts agree this would be useful to the community, I could work on a PR to make this happen.
Environment:

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

Hi @pbudds thanks for submitting this issue. I have a great news. This feature is planned for in one of the next releases.

Thank you @petrkotas. Would you know if anybody started to work on it already? If not, I would be more than happy to pass my changes on to the community for further review.

Hi @pbudds yes, actually it is me who is working on this.

Would be cool to see your changes, Prashanth

@fabiand - Sure. Will send those out in a PR.
@petrkotas - Do not mean to disrupt your plans, but just check out the changes, at the bare minimum, the tests should be useful to you.


manage DNS with dnsmasq alone

Previously:

NetworkManager would manage DNS via dnsmasq
if interfaces were brought up with cloud-init (as on GCE) NetworkManager would not start the dnsmasq instance. Additionally, if dnsmasq died NetworkManager would not necessarily restart it.
NetworkManager would overwrite /etc/resolv.conf

Now:

dnsmasq is managed by systemd, which will keep it alive
NetworkManager's DNS settings are disabled
the default DNS servers are used for domains other than consul_dns_domain, since we can specify separate search settings.

I'd appreciate if someone could test this on OpenStack and another Vagrant machine. I've already tested that the /etc/resolv.conf will survive across changing the hostname via nmcli gen hostname {new name} and rebooting, two instances in which NM would previously overwrite our custom config.

I'm merging this into feature/terraform right away, since the NM configuration is blocking further work there.

This is awesome. I'll test later today.

I'm testing this now.

Testing in Texas:

dnsmasque runs on boot.
Consul service can be found at 3 separate IPs.
server=/consul/<IP_ADDRESS>#8600 appears in /etc/dnsmasq.d/10-consul.
After changing the hostname and changing it back, all of the above still works.
After reboot the above all still works.


Fix/text field custom validation

fixes #421
This does fix the issue, I am just not sure if this is how you want it to be fixed :blush:

Codecov Report

Merging #430 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #430      +/-   ##
==========================================
+ Coverage   96.09%   96.11%   +0.01%
==========================================
  Files          53       53
  Lines        1819     1827       +8
  Branches      214      217       +3
==========================================
+ Hits         1748     1756       +8
  Misses         71       71

Impacted Files
Coverage Δ

packages/text-field/Input.js
95.78% <100%> (+0.38%)
:arrow_up:

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 a468f02...2586c48. Read the comment docs.

@moog16 I have updated the helper-text README.md to show custom validation. Not quite sure what to add in the main README.md. Should I add a reference to the helper-text one? Or create a duplicate example of the custom validation?

@hvolschenk please also sign the PR

I signed it

Thanks! I changed the base to rc7.0. And will wait for #436 to pass, then merge it!


Blank Screen

Good day
I had to set this in html

if not the screen is backgroud colour

Does #100 help?

Ive tried

 => this still works for me.
Thanks

style="position:static" works  for me
thanks


Enforce the order of CMS creating ctrl calls

On encryption

Allocate the necessary EVP_PKEY_CTX
Set passed parameters, if any
Set information about the cipher used, if necessary
Call the ASN1 envelope ctrl

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

@slontis, @levitte - could you take a look, please?

It's currently a semi-hack solution. To finalize it, we need to get rid of CMS_KEY_PARAM processing on encryption.

Travis failures are relevant.

Travis builds has succeeded all but 2 hanged ones.

I've replaced EVP_PKEY_CTX_ctrl to EVP_PKEY_CTX_set_params but had to change the returning code of EVP_PKEY_CTX_set_params to -2 in case we don't have any appropriate settings. I'm not sure it's a good decision.
Any guidelines will be appropriated. @slontis, could you please look again?

It's also worth mentioning that EVP_PKEY_CTX_set_params is incompatible with ENGINE interfaces, and I'm not sure that all the engines will be converted to providers very fast...

I think we need smth like EVP_PKEY_CTX_param_is_settable(EVP_PKEY_CTX *, const char *).

Also not sure whether I should deal with OPENSSL_strdup and OPENSSL_free on error or just remove the const modifier from the result of EVP_CIPHER_name explicitly.

Also not sure whether I should deal with OPENSSL_strdup and OPENSSL_free on error or just remove the const modifier from the result of EVP_CIPHER_name explicitly.

Depends on how it gets used.. It looks like it is not stored - it is just used to resolve a cipher and then thrown away - so the cast should be ok.. Maybe a comment saying this and a cast is ok.

CI is happy now.

@slontis, could you please look at it now?

Travis is happy.

@slontis, could you please re-review and approve if there are no problems with this patch?

ping @openssl/otc

Ok, what's the state of this PR now?

It is mostly superseded by #10904 so if you could look at it, it will be great.

As #10904 is merged, this PR may be closed.


Uncaught TypeError: Cannot set properties of undefined (setting '1')

Describe the bug
Such as title;
When I invoke a long task in eachbatch, at the same time, when rebalance is completed, the this.resolvedOffsets will reinitialize;
To Reproduce

prerequisite

two partitions
eachBatch task longer eg: sleep(10e3);

step

start one consumer
start two consumer
wait 10s
start 3rd consumer

Expected behavior

automatically ignore overdue offset
isSate should include checking for offset exists

Observed behavior

{
    "level": "DEBUG",
    "timestamp": "xxxxxxxxxxxxxxxxxx",
    "logger": "kafkajs",
    "message": "[Runner] Error while fetching data, trying again...",
    "groupId": "<groupId>",
    "memberId": "<memebrId>",
    "error": "Cannot set properties of undefined (setting '1')",
    "stack": "TypeError: Cannot set properties of undefined (setting '1')\n    at OffsetManager.resolveOffset (<myPath>/node_modules/kafkajs/src/consumer/offsetManager/index.js:109:43)\n    at ConsumerGroup.resolveOffset (<myPath>/node_modules/kafkajs/src/consumer/consumerGroup.js:386:24)\n    at resolveOffset (<myPath>/node_modules/kafkajs/src/consumer/runner.js:263:30)\n    at Runner.eachBatch (<myPath>/first.js:33:13)\n    at Runner.processEachBatch (<myPath>/node_modules/kafkajs/src/consumer/runner.js:242:18)\n    at onBatch (<myPath>/node_modules/kafkajs/src/consumer/runner.js:409:20)\n    at <myPath>/node_modules/kafkajs/src/consumer/runner.js:423:15\n    at retry (<myPath>/node_modules/kafkajs/src/retry/index.js:43:5)\n    at <myPath>/node_modules/kafkajs/src/retry/index.js:75:5\n    at new Promise (<anonymous>)",
    "retryCount": 0,
    "retryTime": 1183
}

Environment:
{
    "node": "16.11.1",
    "kafajs"<EMAIL_ADDRESS>    "Kafka": "2.7.0",
    "os": "Darwin Kernel Version 21.3.0"
}

Additional context

Possible solutions
When the number of consumers is greater than the number of partitions, this error may occur when one consumer cannot allocate consumption partitions；So can this be solved?  source code

const decodedAssignment =
  decodedMemberAssignment == null
    ? topics.reduce((p, topic) => ({ ...p, [topic]: [] }), { __proto__: null })
    : decodedMemberAssignment.assignment;

I saw similar error
TypeError: Cannot set property '0' of undefined
    at OffsetManager.resolveOffset (/<redacted>/node_modules/kafkajs/src/consumer/offsetManager/index.js:108:44)
    at ConsumerGroup.resolveOffset (/<redacted>/node_modules/kafkajs/src/consumer/consumerGroup.js:358:24)
    at resolveOffset (/<redacted>/node_modules/kafkajs/src/consumer/runner.js:207:30)
    at Runner.eachBatch (/<redacted>/src/functions/event-loader.js:1:15258)
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (internal/process/task_queues.js:97:5)
    at async Runner.processEachBatch (/<redacted>/node_modules/kafkajs/src/consumer/runner.js:186:7)
    at async onBatch (/<redacted>/node_modules/kafkajs/src/consumer/runner.js:328:9)
    at async /<redacted>/node_modules/kafkajs/src/consumer/runner.js:376:15


Display name update bug

Current state:
at the moment, we have 2 display names for contacts: there is one displayName in conversation.metadata and one displayName in contacts
The problem is that these 2 infos can get out of sync: for example, when the user updates the display name on the UI, the display name in conversation.metadata gets updated but not the display name in contacts.
We can fix this by updating both informations when the user updates the display name on the UI.

This is on purpose since different sources may contribute different source names.

We can fix this by updating both informations when the user updates the display name on the UI.
This is when the user updates the display name on the conversation UI, correct?

@chrismatix yes


ignition-msgs5: Add support for cross-compilation

What does your PR fix?
This PR enables compilation of the ignition-msgs5 port on triplet that require compilation, i.e. among official supported triplets arm64-windows and arm-uwp .

Which triplets are supported/not supported? Have you updated the CI baseline?
This PR add support for compiling ignition-msgs5 for arm64-windows and arm-uwp, but I did not updated the CI baseline as the port will fail if  ignition-msgs5:x86-windows is not installed, and I do not know how this is supposed to be handled at the CI baseline level.

Does your PR follow the maintainer guide?
Yes.

This PR also required some changes in the protobuf ( https://github.com/microsoft/vcpkg/commit/fd8143bb1753b7c14473618f2f084dbdd151d450 ) and ignition-modularscripts (https://github.com/microsoft/vcpkg/pull/11538/commits/5405edad4035b7735b93c7402882cf92f905d1b3) ports, feel free to check the commit messages for more details on those changes.

I will test the features later. Once finished, I will update the results.

The feature has passed with the following triplets:

x86-windows
x64-windows
x64-windows-static

@traversaro I just merged a protobuf PR, could you resolve the conflicts?

@traversaro I just merged a protobuf PR, could you resolve the conflicts?

Done.

x86_windows and x64_osx faiures seems to be unrelated, while the arm64-windows fails with the following message:
CMake Error at ports/ignition-msgs5/portfile.cmake:29 (message):
  Cross-targetting ignition-msgs5 requires the x86-windows ignition-msgs5 to
  be available.  Please install ignition-msgs5:x86-windows first.
Call Stack (most recent call first):
  scripts/ports.cmake:76 (include)

That from https://github.com/microsoft/vcpkg/pull/11538#discussion_r430206281 it should be something that instead should be installed in the CI machines.

@traversaro
Could you please resolve the conflicts?

@traversaro
Could you please resolve the conflicts?

Done.

The arm64-windows failure is due to:

CMake Error at ports/ignition-msgs5/portfile.cmake:29 (message):
Cross-targetting ignition-msgs5 requires the x86-windows ignition-msgs5 to
be available.  Please install ignition-msgs5:x86-windows first.
Call Stack (most recent call first):
scripts/ports.cmake:76 (include)

Based on https://github.com/microsoft/vcpkg/pull/11538#discussion_r430206281 , this should be something installed in the CI machines.

@traversaro
The regression on x64-osx should be fixed in this PR #11616.
CMake Error in src/CMakeLists.txt:
  Imported target "TINYXML2::TINYXML2" includes non-existent path

    "/Volumes/data/work/1/s/packages/tinyxml2_x64-osx/debug/include"

  in its INTERFACE_INCLUDE_DIRECTORIES.  Possible reasons include:

  * The path was deleted, renamed, or moved to another location.

  * An install or uninstall procedure did not complete successfully.

  * The installation package was faulty and references files it does not
  provide.

Could you please merge the latest commit on master to this PR?

Could you please merge the latest commit on master to this PR?

Done.

Seems it doesn't work. I noticed that this was the same reason with #11616. Could you look into this again?

Seems it doesn't work. I noticed that this was the same reason with #11616. Could you look into this again?

I remember that at some point https://github.com/microsoft/vcpkg/pull/11616 contained the fixes for https://github.com/microsoft/vcpkg/issues/10039, but I can't find any reference to those in the final merged version (for example there  is no vcpkg_fixup_pkgconfig call). I am not sure what happened, possibly @orudge has some insights.

I remember that at some point #11616 contained the fixes for #10039, but I can't find any reference to those in the final merged version (for example there is no vcpkg_fixup_pkgconfig call). I am not sure what happened, possibly @orudge has some insights.

Good question, I'm not too sure. My local branch shows the commit pushed to orudge/vcpkg but somehow it's gone missing!
I've made a new PR (https://github.com/microsoft/vcpkg/pull/11966) with the fix.

The PR has been rebased on the top of the latest master, in particular fixing the conflicts with https://github.com/microsoft/vcpkg/pull/11275 .

The arm64-windows job is failing with error:
CMake Error at ports/ignition-msgs5/portfile.cmake:29 (message):
  Cross-targetting ignition-msgs5 requires the x86-windows ignition-msgs5 to
  be available.  Please install ignition-msgs5:x86-windows first.
Call Stack (most recent call first):
  scripts/ports.cmake:76 (include)

So it seems that ignition-msgs5 is not installed for x86-windows as it was discussed in https://github.com/microsoft/vcpkg/pull/11538#discussion_r430206281 .

/azp run

@traversaro
Could you please resolve the conflicts?

@traversaro
Could you please resolve the conflicts?

Hi @NancyLi1013,  I should have fixed all conflicts.

CI is failing with messages like:
Convert-Path : Cannot find path 'C:\agent\_work\1\a\xml-results\x64-uwp.xml' because it does not exist.
At C:\agent\_work\1\s\scripts\azure-pipelines\analyze-test-results.ps1:424 char:42

at the first glance they do not seems to be related to the PR.

/azp run

It seems the command /azp run doesn't work.
@traversaro
Could you please merge master to retrigger this PR?

It seems the command /azp run doesn't work.
@traversaro
Could you please merge master to retrigger this PR?

I rebased the PR on the top of latest master.

Sorry @JackBoosY, I had missed you request change. However, I solved the conflicts w.r.t. to the latest master.

Sorry for late, I lost track with this PR.
These changes caused protobuf build failure on uwp triplet, could you fix it?
     6>D:\buildtrees\protobuf\src\dfb7589f92-61fbc4fa5f.clean\src\google\protobuf\compiler\subprocess.cc(104,46): error C2065: 'HANDLE_FLAG_INHERIT': undeclared identifier [D:\buildtrees\protobuf\arm-uwp-dbg\libprotoc.vcxproj]
     6>D:\buildtrees\protobuf\src\dfb7589f92-61fbc4fa5f.clean\src\google\protobuf\compiler\subprocess.cc(105,29): error C2065: 'HANDLE_FLAG_INHERIT': undeclared identifier [D:\buildtrees\protobuf\arm-uwp-dbg\libprotoc.vcxproj]
     6>D:\buildtrees\protobuf\src\dfb7589f92-61fbc4fa5f.clean\src\google\protobuf\compiler\subprocess.cc(109,48): error C2065: 'HANDLE_FLAG_INHERIT': undeclared identifier [D:\buildtrees\protobuf\arm-uwp-dbg\libprotoc.vcxproj]
     6>D:\buildtrees\protobuf\src\dfb7589f92-61fbc4fa5f.clean\src\google\protobuf\compiler\subprocess.cc(110,29): error C2065: 'HANDLE_FLAG_INHERIT': undeclared identifier [D:\buildtrees\protobuf\arm-uwp-dbg\libprotoc.vcxproj]
     6>D:\buildtrees\protobuf\src\dfb7589f92-61fbc4fa5f.clean\src\google\protobuf\compiler\subprocess.cc(119,26): error C2065: 'STARTF_USESTDHANDLES': undeclared identifier [D:\buildtrees\protobuf\arm-uwp-dbg\libprotoc.vcxproj]

@traversaro
Could you please also resolve the conflicts?

@traversaro
Could you please also resolve the conflicts?

Hi @NancyLi1013 , the conflicts have been solved. @JackBoosY let's see if the uwp failure are still there, probably there is some upstream protobuf regression regarding compilation of the libprotoc library on uwp.

There are several failures, but they do not seems to be related to this PR.

/azp run

The failures in CI are consistent with a malformed CONTROL file somewhere and they aren't happening outside of this PR. Just in case I pushed a merge with master, we'll see what comes back.

/azp run

@BillyONeal Any progress?

@BillyONeal Any progress?

The failure on this are due to a regression in protobuf, that does not support anymore in the latest version compilation in uwp .
As they are already some active PR in the direction of getting protobuf to correctly cross compile (see https://github.com/microsoft/vcpkg/pull/14165), let's try to see if those PR before starting to work again on this one.

Related PR: https://github.com/microsoft/vcpkg/pull/15424 .

Related PR: https://github.com/microsoft/vcpkg/pull/15424 .


Give die a status code when the env file is not valid

The purpose of this PR is to address the rather rare case when the .env file is invalid. This could pop up when one of the lines has a key but no value, and it's followed by a comment. An example line might look like:
CUSTOM_VALUE=                # This is a value to be used for tests

Having a value like this causes the die method to be called and thus kills the app, but since previously there was a string in the place of a status code, it would exit with a 0 status code despite the process actually being killed.
For some CI platforms any application that closes with a 0 status code is considered to be successful. Which means our tests pass even though our tests never actually run.
This PR aims to address this false positive by forcing the die method to have a status code that isn't text, thus making the program exit with a non-zero status.
Personally I feel like there should also be a similar echo and die in the InvalidPathException that is just discarded, but that may not be needed since it was previously ignored.

Why does Laravel just die? Seems like there should be a more elegant solution here.
But I definitely agree it should exit with a non-zero return code. Otherwise build/deploy scripts assume all tests pass.

I don't think we need to just start dieing on a patch release.

@taylorotwell I'm not sure if you noticed, but that is the current behavior. For an InvalidFileException, it just dies (with a successful return code). However, this fixes the return code, so CI builds don't just pass without running tests.
As for the InvalidPathException I agree that's a breaking change and should be removed from this PR. @austinkregel can you please remove that and submit a new PR, or maybe this one can be re-opened?

@taylorotwell the code already dies if the .env is is invalid, this just changes the status code.
The added part for an invalid exception path should be removed since that branch doesn't die currently.


Slovak android app inaccessible abroad

When I try to download https://play.google.com/store/apps/details?id=sk.nczi.covid19
from outside Slovakia (Switzerland), I get an error "this item is not available in your country".
I think this will cause a lot of problems in practice, as

many people coming back to Slovakia will have Google Play country setting set to different country (and it is very non-trivial to change) so they will not be able to download app at the border
people will not be able to download the app in advance.

This app is intended to be used in Slovakia only as Slovak Ministry of health is coordinating tests for possibly infected people. As there is no position tracking it would be very difficult to identify if the person is from Slovakia.
There will soon be app for home quarantine, also for people returning back to Slovakia, this app will be available worldwide.

Unfortunately it's a genuine issue, the number of people affected by this is underestimated, there are enough legitimate Android users in Slovakia without a Slovak Google account, while the negative effects of having this app available globally are overestimated. If anyone has malicious intentions they can still sideload the APK from elsewhere while legimate and less-skilled users are locked out. Considering the dependence of this app on the network effect, I would really welcome lifting the geoblock.


Add this Asriel face

LOL WHERE DOES THIS EVEN COME FROM

Added where it comes from as alt tag on the image above

oops I mean Asgore


Slayer count off by 1 when logging out/in

Based on Discord report slayer task count is off by 1 sometimes when you logout and relog after a while mid slayer task. The issue occurs using regular killing method (so no cannon/barrage/bursting). User reported that it occured while killing Black Dragons with rcb.

It was caused by the user using a genie lamp on slayer

I have had this happen to me multiple time killing iron dragons too. Slayer plugin will still show what you have to kill but display 0 instead of 1. Which is confusing because then I think my task is done and leave to find out I still need to kill 1.
I have never used a genie lamp on slayer.

Do we have a better way of determining when the player kills a monster? The slayer xp drop doesn't seem reliable enough.

Occurred for me today at Iron Dragon. 1 over.

One thing I want to note that on OSBuddy (about 3 months ago, no idea if it still happens), if you kill multiple monsters on the same game tick, the slayer counter only decreases by 1. Maybe it is not possible to detect how many monsters were killed on the same game tick?

Gamer1120, due to how RS works, some events are bundled together and it is difficult to attribute what in game actions lead to the results.
You can see this happen if you try to clean like 10 grimy herbs as fast as possible. Only maybe 3 to 7 "actions" may come through.
The same concept may apply if you are using the dwarf cannon and two monsters die in the same tick. The reward is granted in one step instead of two.

I guess duplicate of #1046 or so.


Add handling client with "zero length" client-id

In this case the server MUST check that clean session is true otherwise refuses connection.
If it can accept connection, a unique client-id must be generated and assigned to the endpoint.

Closing on behalf of c64a5417f364b8b66c57d4b0807830bcab8daeb6


new vehicle

I am trying to add a new vehicle and new can card. I have followed the relevant how do docs to do so. Do I have to modify anything else in Apollo to support these changes. Where do I change the CAN signals for the new vehicle?
I am using Apollo 3.5 on 16.04

https://github.com/ApolloAuto/apollo/blob/master/modules/canbus/conf/canbus_conf.pb.txt


Configure provider-id for the machines/nodes

What this PR does / why we need it:
machine-controller will inject provider-id for the machines that are created against cloud providers that don't have in-tree or external CCM support. The generated providerID will be configured on the machine and node respectively.
Which issue(s) this PR fixes:

Fixes #1552
What type of PR is this?

/kind feature
Special notes for your reviewer:
Does this PR introduce a user-facing change? Then add your Release Note here:

Machine-controller will inject provider-id for the machines that are created against cloud providers that don't have in-tree or external CCM support

Documentation:

NONE

pull-machine-controller-e2e-vmware-cloud-director will fail and requires https://github.com/kubermatic/machine-controller/issues/1619 to fix it.
pull-machine-controller-e2e-kubevirt seems to be flaking on different tests.

/test pull-machine-controller-e2e-kubevirt

/retest

/cherry-pick release/v1.57

@ahmedwaleedmalik: #1723 failed to apply on top of branch "release/v1.57":
Applying: Configure provider-id for the machines/nodes
Using index info to reconstruct a base tree...
M	pkg/cloudprovider/provider/digitalocean/provider.go
M	pkg/cloudprovider/provider/equinixmetal/provider.go
M	pkg/cloudprovider/provider/hetzner/provider.go
M	pkg/cloudprovider/provider/kubevirt/provider.go
M	pkg/cloudprovider/provider/linode/provider.go
M	pkg/cloudprovider/provider/nutanix/provider.go
M	pkg/cloudprovider/provider/opennebula/provider.go
M	pkg/cloudprovider/provider/openstack/provider.go
M	pkg/cloudprovider/provider/vmwareclouddirector/provider.go
M	pkg/cloudprovider/provider/vsphere/provider.go
M	pkg/cloudprovider/provider/vultr/provider.go
M	pkg/controller/machine/controller.go
Falling back to patching base and 3-way merge...
Auto-merging pkg/controller/machine/controller.go
Auto-merging pkg/cloudprovider/provider/vultr/provider.go
CONFLICT (content): Merge conflict in pkg/cloudprovider/provider/vultr/provider.go
Auto-merging pkg/cloudprovider/provider/vsphere/provider.go
Auto-merging pkg/cloudprovider/provider/vmwareclouddirector/provider.go
Auto-merging pkg/cloudprovider/provider/openstack/provider.go
Auto-merging pkg/cloudprovider/provider/opennebula/provider.go
Auto-merging pkg/cloudprovider/provider/nutanix/provider.go
Auto-merging pkg/cloudprovider/provider/linode/provider.go
Auto-merging pkg/cloudprovider/provider/kubevirt/provider.go
Auto-merging pkg/cloudprovider/provider/hetzner/provider.go
Auto-merging pkg/cloudprovider/provider/equinixmetal/provider.go
Auto-merging pkg/cloudprovider/provider/digitalocean/provider.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Configure provider-id for the machines/nodes
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

Will do a manual backport


Add support for unverifiable breakpoints

Hello,
I am developing an extension to support SIMD-lane specific breakpoints and create them using vscode.SourceBreakpoint() adding a specific condition and then processing them inside the extension. Because of this, all created breakpoints do not pass validation and are displayed in gray.
Could you add support for breakpoints that should not be processed by the DAP and not pass checks, but at the same time appear active?
In my understanding, this is not a difficult task and you only need to add some flag and check it, for example something like
SourceBreakpoint(location: vscode.Location, enabled?: boolean | undefined, condition?: string | undefined, hitCondition?: string | undefined, logMessage?: string | undefined, unverifiable?: boolean | undefined)
what would it mean:
unverifiable?:
true: require verification
false: does not require verification
undefined: behavior by default
Thank you in advance for the answer, I am interested in your point of view on this question. Is it also possible that such an opportunity already exists?

Hello @roblourens , of course:
At the moment, there is no way in VS Code to create completely custom breakpoints, so I use a breakpoint with a condition, and then intercept them and pass them to the debugger.
Of course, such breakpoints are not valid and are displayed something like this:

I would like to be able to disable the check of breakpoints that makes them "gray".

Hello @roblourens ,
If you need additional information, I am ready to provide it.

So your extension is separate from the actual debugger? That seems suspicious to me. You are basically implementing a feature that the debugger doesn't actually support? It seems safer/more correct for the debugger itself to implement this feature so that we can communicate normally over DAP and that everyone is on the same page.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Corrected a typo in the url and improved resources section.

Fixing "Link no longer working" #50
Updated resource section to codelab article

@googlebot I signed it!


Configuring tauri v2 for android development

Hello, I'm trying to create devenv.nix for tauri development but I'm stuck on installing android studio.
I'm trying to follow tauri tutorial and this? but I'm failing miserably.
This is the current state of the config but it's not even remotely close to what I'm looking for devenv.nix.txt
The goal is to run npm run tauri android dev and compile the project.

You could try asking around on our Discord, though I'm not sure there are too many people running this type of setup.
Unless there's a specific devenv issue here, I'm not sure we'll be able to help you out.

I'm closing the issue now, i found a solution and it's posted on my github in tauri-mobile-nixos-template


Use 'bindings' insteadof 'node-pre-gyp' in prod build

'node-pre-gyp' has a lot of dependencies which undesired in production usage
'bindings' is clear and simpler

node-pre-gyp is required for downloading pre built binaries as well as for compiling native module for platforms for which there are no precompiled versions available


Framing issue

Hi,
When I ran easyflytracker on a single 6-well plate video, the framing worked perfectly fine. However when I ran a four 6-well plates video, the software change frame size of the video and cropped out the other three well plate. I think it is something to do with roi (region of interest)/undistortion().

Sorry, I don't quite understand what you mean. Could you please send me the video and configuration file to test it?

test2.mp4
https://drive.google.com/file/d/1ric0--C3Op0RiLoqzBeB5HUxISwt1NH5/view?usp=drive_web
Here it is. The error message I have also copied them here.
Thanking you,
Tashi (Wartburg College, U.S.)
P.S- Is it possible to set up zoom call with you to get help with the
software. I will appreciate it a lot.
On Tue, 25 Oct 2022 at 20:48, Qj Z @.***> wrote:

Sorry, I don't quite understand what you mean. Could you please send me
the video and configuration file to test it?
—
Reply to this email directly, view it on GitHub
https://github.com/azzhu/EasyFlyTracker/issues/3#issuecomment-1291377089,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATRTG7SSNW2PX5GGHLWRJGDWFCEX7ANCNFSM6AAAAAARMO2STA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi, bro, this is the result of the test data I ran. You can download it to see how it differs from your configuration.
http://<IP_ADDRESS>:3557/sharing/QKv5XEFPr
Good luck!

I don't know why but I couldn't open the link. Could you maybe share a google drive link instead?

Could you please change another search engine? It will be worked sometimes.

I don't know why but I couldn't open the link. Could you maybe share a google drive link instead?


Documentation errors during install

Running devtools::install() with the Master branch gives this output. Can this be addressed?
Enter one or more numbers, or an empty line to skip updates:
✔  checking for file ‘/Users/rmorin/git/GAMBLR/DESCRIPTION’ ...
─  preparing ‘GAMBLR’: (470ms)
✔  checking DESCRIPTION meta-information ...
Warning: bad markup (extra space?) at chromosome_arms_grch37.Rd:16:2
Warning: /private/var/folders/kk/mjt9hn3n7nn25nfvhv6mkjm40000gn/T/Rtmpk0nEGf/Rbuildab634845df0c/GAMBLR/man/chromosome_arms_grch37.Rd:19: unexpected section header '\usage'
Warning: /private/var/folders/kk/mjt9hn3n7nn25nfvhv6mkjm40000gn/T/Rtmpk0nEGf/Rbuildab634845df0c/GAMBLR/man/chromosome_arms_grch37.Rd:22: unexpected section header '\description'
Warning: /private/var/folders/kk/mjt9hn3n7nn25nfvhv6mkjm40000gn/T/Rtmpk0nEGf/Rbuildab634845df0c/GAMBLR/man/chromosome_arms_grch37.Rd:25: unexpected section header '\keyword'
Warning: /private/var/folders/kk/mjt9hn3n7nn25nfvhv6mkjm40000gn/T/Rtmpk0nEGf/Rbuildab634845df0c/GAMBLR/man/chromosome_arms_grch37.Rd:26: unexpected END_OF_INPUT '
'
Warning: bad markup (extra space?) at chromosome_arms_hg38.Rd:16:2
Warning: /private/var/folders/kk/mjt9hn3n7nn25nfvhv6mkjm40000gn/T/Rtmpk0nEGf/Rbuildab634845df0c/GAMBLR/man/chromosome_arms_hg38.Rd:19: unexpected section header '\usage'
Warning: /private/var/folders/kk/mjt9hn3n7nn25nfvhv6mkjm40000gn/T/Rtmpk0nEGf/Rbuildab634845df0c/GAMBLR/man/chromosome_arms_hg38.Rd:22: unexpected section header '\description'
Warning: /private/var/folders/kk/mjt9hn3n7nn25nfvhv6mkjm40000gn/T/Rtmpk0nEGf/Rbuildab634845df0c/GAMBLR/man/chromosome_arms_hg38.Rd:25: unexpected section header '\keyword'
Warning: /private/var/folders/kk/mjt9hn3n7nn25nfvhv6mkjm40000gn/T/Rtmpk0nEGf/Rbuildab634845df0c/GAMBLR/man/chromosome_arms_hg38.Rd:26: unexpected END_OF_INPUT '

I picked up this issue too. This is already fixed in PR #177 That PR is ready for review

The fix is now merged to master


maximumCpuCount not explained

The detailed documentation is not talking anything about maximumCpuCount attribute.  Also, if it is boolean, the name ending with "count" is misleading.  Should it be useMaximumCpuCount instead which makes it sound like boolean?

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

ID: 22342679-18af-7c79-f902-fe9c0bb55379
Version Independent ID: a8e7c201-6593-458b-5a81-b54e93fde4b5
Content: MSBuild build and release task - Azure Pipelines
Content Source: docs/pipelines/tasks/build/msbuild.md
Product: devops
GitHub Login: @andyjlewis
Microsoft Alias: alewis

I believe this might be relevant: https://docs.microsoft.com/visualstudio/msbuild/building-multiple-projects-in-parallel-with-msbuild. Adding @vijayma to advise