Services address is removed from /Common/Shared when deleting app in tenant

Environment

Application Services Version: 3.51
BIG-IP Version: BIG-IP v17.1.1 (Build 0.0.6)

Summary
Our config within a tenant/application references a service address (<IP_ADDRESS>) address in Common Shared (name: /Common/Shared/x_service-address_any). If you delete this application within a tenant, the service address in the common partition will also be deleted. This must not happen and disrupts our further process.
We use per app declaration
Steps To Reproduce
Steps to reproduce the behavior:

Submit the following  two declarations:
/Common/Shared

    "action": "deploy",
    "class": "AS3",
    "declaration": {
        "class": "ADC",
        "schemaVersion": "3.45.0",
        "Common": {
            "class": "Tenant",
            "Shared": {
                "x_service-address_any": {
                    "class": "Service_Address",
                    "virtualAddress": "<IP_ADDRESS>",
                    "icmpEcho": "disable",
                    "arpEnabled": false
                },
}
}

[adc-tenant](https://xxx/mgmt/shared/appsvcs/declare/adc-tenant/applications/)
{
  "app_<IP_ADDRESS>_24": {
    "class": "Application",
    "label": "<IP_ADDRESS>/24",
    "remark": "applicationx for <IP_ADDRESS>/24",
    "sf-ext_<IP_ADDRESS>_24": {
      "allowVlans": [
        {
          "bigip": "/adc-tenant/dmz_123"
        }
      ],
      "class": "Service_Forwarding",
      "forwardingType": "ip",
      "label": "<IP_ADDRESS>/24",
      "policyFirewallEnforced": {
        "use": "x_firewall-policy_<IP_ADDRESS>_24"
      },

      "remark": "service forwrding ext for <IP_ADDRESS>/24",
      "shareAddresses": true,
      "snat": "none",
      "translateServerAddress": false,
      "translateServerPort": false,
      "virtualAddresses": [
        [
          {
            "use": "/Common/Shared/x_service-address_any"
          },
          "<IP_ADDRESS>/24"
        ]
      ],
      "virtualPort": 0
    },
    "sf-int_<IP_ADDRESS>_24": {
      "class": "Service_Forwarding",
      "forwardingType": "ip",
      "label": "<IP_ADDRESS>/24",
      "policyFirewallEnforced": {
        "use": "x_firewall-policy_<IP_ADDRESS>_24"
      },
      "profileL4": {
        "use": "/Common/Shared/x_default_profiles_protocol_l4"
      },
      "rejectVlans": [
        {
          "bigip": "/adc-tenant/dmz_123"
        },
        {
          "bigip": "/Common/http-tunnel"
        },
        {
          "bigip": "/Common/socks-tunnel"
        }
      ],
      "remark": "service forwrding dint for <IP_ADDRESS>/24",
      "securityLogProfiles": [
        {
          "use": "/Common/Shared/x_default_profiles_security_log"
        }
      ],
      "shareAddresses": true,
      "snat": "none",
      "translateServerAddress": false,
      "translateServerPort": false,
      "virtualAddresses": [
        {
          "use": "/Common/Shared/x_service-address_any"
        }
      ],
      "virtualPort": 0
    },
    "x_firewall-policy_<IP_ADDRESS>_24": {
      "class": "Firewall_Policy",
      "label": "<IP_ADDRESS>/24",
      "remark": "firewall policy for <IP_ADDRESS>/24",
      "rules": [
        {
          "use": "/Common/Shared/x_firewall-rule-list_global"
        },
        {
          "use": "x_firewall-rule-list_<IP_ADDRESS>_24_product-related"
        },
        {
          "use": "x_firewall-rule-list_<IP_ADDRESS>_24_architecture-related"
        },
        {
          "use": "/Common/Shared/x_firewall-rule-list_drop"
        }
      ]
    },
    "x_firewall-rule-list_<IP_ADDRESS>_24_application-related": {
      "class": "Firewall_Rule_List",
      "label": "<IP_ADDRESS>/24",
      "remark": "firewall rule list aspplissssscation related for <IP_ADDRESS>/24",
      "rules": []
    },
    "x_firewall-rule-list_<IP_ADDRESS>_24_architecture-related": {
      "class": "Firewall_Rule_List",
      "label": "<IP_ADDRESS>/24",
      "remark": "firewall rule slist architecture related for <IP_ADDRESS>/24",
      "rules": []
    },
    "x_firewall-rule-list_<IP_ADDRESS>_24_product-related": {
      "class": "Firewall_Rule_List",
      "label": "<IP_ADDRESS>/24",
      "remark": "firewall rule lsisst product related for <IP_ADDRESS>/24",
      "rules": []
    }
  },
  "controls": {
    "class": "Controls",
    "logLevel": "debug",
    "trace": true
  },
  "id": "per-app-declaration",
  "schemaVersion": "3.51.0"
}

Delete with
https://xxxx/mgmt/shared/appsvcs/declare/adc-tenant/applications/app_<IP_ADDRESS>_24

Observe the following error response:

{
    "results": [
        {
            "code": 422,
            "message": "declaration failed",
            "response": "0107082a:3: All objects must be removed from a partition (adc-tenant) before the partition may be removed, type ID (18797)",
            "host": "localhost",
            "tenant": "adc-tenant",
            "runTime": 5720,
            "declarationId": "1719494046084"
        }
    ],

Expected Behavior

Correct response in the result because the app was successfully deleted
x_service-address_any should not be removed from the bigip in /Common/Shared

Actual Behavior
After the reproduction of the steps you will see, that the service any address is removed from /common/shared in the bigip gui.

Thank you for reaching out. We have created a internal product backlog AUTOTOOL-4433 for this item, in order for us to prioritize please reach out to us at<EMAIL_ADDRESS>


Provide more advanced image processing options (blur, crop…))

Thanks for this tool!
Is your feature request related to a problem? Please describe.
Sometimes I want to apply a specific transform to all images, like blur them, darken them, crop them… Other frameworks can do things like that, for instance sveltekit.
Describe the solution you'd like
Maybe provide an attribute with a list of transforms to apply and apply them sequentially using sharp, and for more advanced cases why not accepting an attribute that accept a function applying any operation on the image?
Describe alternatives you've considered
Sveltekit

This library is only intended to enable the features supported by next/image to be used at build time.
For example, placeholder and browser-specific extensions are supported.
If there are a lot of requests for image enhancements like the ones you mentioned, we will consider them, so if you agree, please 👍 your first post!


Show full date in title attribute

We format dates that we show in Frontend. @issa-tseng writes:

within a week it shows the day name, and beyond that it shows the exact date. we should make it so that when you hover over a date you always get the exact full date/time though.

I'm thinking we could create a date component that does this.

I think that behavior would be really great. I use that all the time in Github and it's exactly what I want. Is the discussion more about whether to build a component or about the behavior?

I think the only question I have is how to format the date in the title attribute. Should it just be how we format dates from a week or more ago? For example, you hover over a date that says Today 10:00 and see 2019/08/08 10:00.

yup, exactly. let's give them the seconds, too.

We've now implemented this component in #286. To close out this issue, I think we just need to go through existing components and use this new component wherever we show a date/time.


Issue & Bug with Ctrl Alt Tab

From what I am experiencing, CTRL+ALT+Tab shows all processes with a persistent window which allows you to search with the keyboard, right? However it isn't currently showing all of the processes. You can also navigate upwards but it doesn't show everything. It's kind of a buggy version of the normal alt-tab. You can discover hidden windows if you start typing titles that exist that aren't currently in the list.
BUG: Also using this shortcut, the window doesn't disappear after clicking other windows or lost focus.
Otherwise, I really enjoy this application and wish for it's success. It's a lot like Contexts for the mac.

Hi Jasong,
Please share some screenshots if possible.
BUG: Also using this shortcut, the window doesn't disappear after clicking other windows or lost focus.
It was available in older version but intentionally disabled in 2024.4, it's causing AltTab window to disappear frequently. I'll fix this in next version.
Thank you.


Missing the option of sending a message to a specific user from the aggregator

This version of the communications library is missing the option of sending a message to a specific user from the aggregator. We need this in order to implement algorithms under the POM3.

Solved.


Paired but no audio

My KCX_BT_EMITTER is a bit different from the one mentioned here (see photo). There is a jumper on the PCB, which is I left open and then it is in transmitter mode. I have audio signal on Audio_L and Audio_R with common ground on AGND. I do not use any other functions. Pairing with my Sony headphone and Samsung earbuds is going well, but unfortunately I don't hear any audio.
Does anyone have a suggestion of what might go wrong?

Sorry, this response is basically "I don't know".
My experience with up-to-date bluetooth earphones is that it would not connect. The version of the module I am using is for bluetooth 4.2, and the earphones were 5.something. Maybe the earphone chose not to accept the signal? I don't know. When I started looking for a bluetooth speaker, I realized that I should choose older speakers and maybe they would not all work. Eventually I may move to a version with a more up-to-date bluetooth, but so far this one does what I am trying to do and I have spent some time learning how to make it work reliably so I will probably use it for a few more years.

Thank you for your message. I know it is sometimes difficult to pair BT but the pairing works fine, just no sound. In the meantime I tested with Samsung Earbuds, a Sony Headphone and a set of very cheap earbuds from AliExpress. I can pair them all, pretty quickly, but no sound.
Maybe I have a faulty module, maybe there is another problem with my version. All I can do now is to be patient and wait for the old version to be delivered. Besides the radio works well through a pair of wired speakers ;-)
More info: https://github.com/WerkAG/ESP32Radio_PCB_v3.1/issues/3

LOL! You started my day with a smile...
I hope you discover the issue! If you do, please leave a note here.
Thanks!


TST: tolerance violations with SciPy 1.14.0rc1 on linux-{aarch64,ppc64le}

In https://github.com/conda-forge/scipy-feedstock/pull/277, I see new failures for aarch/ppc.
On linux-aarch64, there are some minor violations of relative tolerance:
=========================== short test summary info ============================
FAILED special/tests/test_hyp2f1.py::TestHyp2f1::test_region2[hyp2f1_test_case15] - AssertionError:
Not equal to tolerance rtol=1e-15, atol=0

Mismatched elements: 1 / 1 (100%)
Max absolute difference among violations: 7.42004997e-11
Max relative difference among violations: 1.24886275e-15
 ACTUAL: array(54749.216391-23078.144721j)
 DESIRED: array(54749.216391-23078.144721j)
FAILED special/tests/test_hyp2f1.py::TestHyp2f1::test_region5[hyp2f1_test_case44] - AssertionError:
Not equal to tolerance rtol=1.1e-15, atol=0

Mismatched elements: 1 / 1 (100%)
Max absolute difference among violations: 1.24126708e-15
Max relative difference among violations: 1.51901296e-15
 ACTUAL: array(-0.185152+0.795901j)
 DESIRED: array(-0.185152+0.795901j)
= 2 failed, 49224 passed, 2743 skipped, 159 xfailed, 14 xpassed, 106 warnings in 6336.31s (1:45:36) =

On linux-ppc64le, the failures are in completely different tests; while also related to tolerance violations, here it looks more like it's about values close to machine precision, and thus the absolute tolerance that would need to be loosened.
=========================== short test summary info ============================
FAILED optimize/tests/test_differentiate.py::TestDifferentiate::test_vectorization[shape1-6] - AssertionError:
Not equal to tolerance rtol=1e-07, atol=5e-15

Mismatched elements: 1 / 12 (8.33%)
Max absolute difference among violations: 6.02295991e-15
Max relative difference among violations: 2.00249219e-06
 ACTUAL: array([4.384204e-09, 4.384201e-09, 4.081055e-09, 3.501619e-09,
       2.696714e-09, 1.735705e-09, 6.991865e-10, 3.291873e-10,
       1.270939e-09, 2.059971e-09, 2.647979e-09, 3.007738e-09])
 DESIRED: array([4.384204e-09, 4.384203e-09, 4.081054e-09, 3.501620e-09,
       2.696713e-09, 1.735705e-09, 6.991875e-10, 3.291881e-10,
       1.270939e-09, 2.059969e-09, 2.647979e-09, 3.007732e-09])
FAILED optimize/tests/test_differentiate.py::TestDifferentiate::test_vectorization[shape2-6] - AssertionError:
Not equal to tolerance rtol=1e-07, atol=5e-15

Mismatched elements: 1 / 12 (8.33%)
Max absolute difference among violations: 6.02295991e-15
Max relative difference among violations: 2.00249219e-06
 ACTUAL: array([4.384204e-09, 4.384201e-09, 4.081055e-09, 3.501619e-09,
       2.696714e-09, 1.735705e-09, 6.991865e-10, 3.291873e-10,
       1.270939e-09, 2.059971e-09, 2.647979e-09, 3.007738e-09])
 DESIRED: array([4.384204e-09, 4.384203e-09, 4.081054e-09, 3.501620e-09,
       2.696713e-09, 1.735705e-09, 6.991875e-10, 3.291881e-10,
       1.270939e-09, 2.059969e-09, 2.647979e-09, 3.007732e-09])
FAILED optimize/tests/test_differentiate.py::TestDifferentiate::test_vectorization[shape3-6] - AssertionError:
Not equal to tolerance rtol=1e-07, atol=5e-15

Mismatched elements: 1 / 12 (8.33%)
Max absolute difference among violations: 6.02295991e-15
Max relative difference among violations: 2.00249219e-06
 ACTUAL: array([4.384204e-09, 4.384201e-09, 4.081055e-09, 3.501619e-09,
       2.696714e-09, 1.735705e-09, 6.991865e-10, 3.291873e-10,
       1.270939e-09, 2.059971e-09, 2.647979e-09, 3.007738e-09])
 DESIRED: array([4.384204e-09, 4.384203e-09, 4.081054e-09, 3.501620e-09,
       2.696713e-09, 1.735705e-09, 6.991875e-10, 3.291881e-10,
       1.270939e-09, 2.059969e-09, 2.647979e-09, 3.007732e-09])
= 3 failed, 49562 passed, 2741 skipped, 160 xfailed, 16 xpassed, 106 warnings in 5716.31s (1:35:16) =

In both cases it's possible that this is related to QEMU version and/or details of the underlying host agent. I can skip those tests if necessary, but it would be nicer to fix it properly.

The second failure is alread addressed in gh-20928 because I encountered the same problem with another array backend.

The test_differentiate failures are alread addressed in https://github.com/scipy/scipy/pull/20928 because I encountered the same problem with another array backend.

That said, it would of course be preferable not to backport an enhancemet PR to the release branch for the purposes of simple tol adjustments. Are you in favor of manual/temporary shims for that purpose on the release branch?

Oops didn't see this was about RC1 with the assumption that we need a backport. I can bump that tolerance in a separate PR.

Slightly offtopic but do we want to keep supporting PPC architecture? How relevant is that? Without continuous testing, such issues will occur in every release process. I understand we are widely used library but there has to be a tradeoff between required efforts and gains in my opinion.

ppc64le is definitely more niche. With these kinds of targets historical policy has been that we make necessary changes, with the expectation that the downstream consumers help in making the fixes. The expectations have to be correspondingly lower for performance/stability/speed of response.
IMO one advantage of keeping this kind of unofficial support is that it helps keep other targets in a better state.

How relevant is that? Without continuous testing, such issues will occur in every release process.

For better or worse, it's one of the main architectures in conda-forge, so support is important. That doesn't mean that SciPy here has to run CI for PPC on all PRs, but it should mean that we fix issues as they crop up. We've been doing this for a long time (even with PyPy, which used to break even more; now dropped on the feedstock), so I don't think we should change something.


KeyVault service is missing in 2017-03-09 profile

2017-03-09 profile does not have KeyVault service

@bganapa, ARM or data-plane?

Thanks Martin for looking in to this . We need both, we support the version  2016-10-01 for both.


Problems with calibration

I'd post this to the forum, but I'm not getting the verification emails when I sign up, and no one has responded.
I'm trying to calibrate my unisolder. When I go into the calibration menu, the R value is sitting at 35 with the 10 ohm resistor attached. I can turn the trimmer and have it go up to 40, or down and it goes to 30. If I disconnect the resistor, the value shoots up to 1100-ish. Reading the docs, it is supposed to be close to 1000 with the 10 ohm resistor? I tried searching through the forums, but I didn't find anything that solved my issue. I found some things that mentioned the current across Sense A and Vout1- should be around 380uA, mine measures 360uA (though I don't fully trust my multimeter). I double checked U11 was soldered on properly, as well as the PIC, but I haven't been able to figure it out. I even replaced Rc2 thinking it was a broken pot (original spun infinitely).

I also had problems getting the verification email from the forum. After several tries I succeded though.
I don't remember much of the calibration procedure but I'm pretty sure it wasn't straightforward for me either. The values weren't the ones expected. I think I had to check and redo some solder joints to solve the problem...

I figured the problem out, for some reason two of the pins on U17 were shorted somehow. I was able to calibrate, but now I have a new issue. When I plug the iron in it says "Heater Open". I double checked the wiring and I'm pretty sure the connector is wired up correctly. I'll do some more searching on the forum.

When it said "Heater Open", the iron was correctly identified. I found a short on C17, resoldered it, and now it says "no instrument". I went into the calibration screen and the IDs are set to 5 and 5. I then found shorts on R2 and R21, fixed those, and the same issue is present. Not sure what is going on, I'll keep looking.

Maybe this is of some help:
http://dangerousprototypes.com/forum/index.php?topic=7218.msg68141#msg68141

The recommendation is to reprogram, but it looks like the problem was resolved by replacing the PIC... I'll do some more looking on the forum, maybe I'll find something where I don't have to replace the entire PIC and that will work...

I figured it out, turns out I did wire it incorrectly. Had heater channel two + and - flipped. Guess that's what I get for soldering while tired. That explains why it detected the iron before, C17 was shorting channel 2 + and - together...


Performance Lag while building the Canonical Examples on Ubuntu 20.0.0 LTS Locally

Hi,
I was trying to build out the Canonical Demos/Code Samples supplied with the Project but unfortunately from the past few days, everytime I am trying to compile, it takes me even more than 2 hours to load all the dependencies and additionally even the build process does not complete successfully. My Machine goes into a Lag State and all the process that took me around 2-2.5 hours to setup goes into Vain.
I am skeptical whether is this Common for Ubuntu Users or this is something specific that I am only Facing
Any Help will be appreciated on this Part

Indeed, in our continuous builds, a fresh build + running all tests on Ubuntu 16.04.7 LTS takes almost 3 hours. I believe most of the runtime is due to the fact that the project compiles LLVM from scratch, which is a very large project. However, after the first successful build of LLVM on your machine, the compiled objects will be cached by bazel and not built again by default.
One thing you can try is compiling subsets of the project at a time (e.g., bazel build @yosys//:all, bazel build @abc//:all, bazel build @openfhe//:all, etc.). Future builds will use the cached results.
Bazel has a means to link with a pre-compiled libraries, but I have not tried it on this project. If it works, it would likely substantially reduce the build time. I understand this is frustrating, but we don't have any more specific guidance at this time.


https://fix-macosx.com/fix-macosx.py SSL Certificate expired.

I'm hoping that you may know how to contact them to inform them of the expired certificate.  I sent an email to the whois registrant<EMAIL_ADDRESS>

they are on https://github.com/fix-macosx/

Paging @landonf

Why not remove the link? The fix is trivial and safer than using a Python script from an unknown source.

That Web site's certificate has been renewed.


[v1.11] images: Bump Hubble CLI to v0.11.1

This bumps the Hubble CLI to the latest release, v0.11.1. By upgrading the Hubble CLI to the latest release, we ensure that the binary is built using the most recent version of Golang while preserving compatibility to the previous version using the new HUBBLE_COMPAT environment variable.

/test-backport-1.11

/ci-l4lb-1.11

will rebase after https://github.com/cilium/cilium/pull/23220 gets backported cc @ldelossa

/ci-awscni-1.11

/ci-aks-1.11

/ci-eks-1.11

/ci-gke-1.11

/ci-external-workloads-v1.11

/ci-multicluster-1.11

For some reason, the images cannot be fetched, even though the image build pipeline is green. Let's wait for #23220 and rebase afterwards to trigger the image build again.

/test-backport-1.11

/mlh new-flake Cilium-PR-K8s-1.21-kernel-4.9

ConformanceAKS fails a different way each time it runs - https://github.com/cilium/cilium/issues/20887, #23366, https://github.com/cilium/cilium/issues/23367


NOISSUE: ve 10node test

@dmitry-shidlovsky this is quite a large PR. What exactly this NOISSUE is needed for?

Branch and PR were deleted by approve of @dmitry-shidlovsky


Combine My Settings, Submit Feedback and Logout areas into single 'Account' settings

Description
LEARNER-2213
Combine My Settings, Submit Feedback and Logout areas into single 'Account' settings

Codecov Report

Merging #980 into master will increase coverage by 0.18%.
The diff coverage is 62.5%.

@@            Coverage Diff             @@
##           master     #980      +/-   ##
==========================================
+ Coverage   45.52%   45.71%   +0.18%
==========================================
  Files         405      407       +2
  Lines       26897    26998     +101
==========================================
+ Hits        12246    12341      +95
- Misses      14651    14657       +6

Impacted Files
Coverage Δ

Source/OEXRearTableViewController.swift
0% <0%> (ø)
:arrow_up:

Source/OEXRouter.m
53.27% <0%> (-1.35%)
:arrow_down:

Source/OEXRouter+Swift.swift
15.74% <0%> (-0.61%)
:arrow_down:

Source/OEXMySettingsViewController.m
22.5% <100%> (+1.98%)
:arrow_up:

Source/AccountViewController.swift
67.5% <67.5%> (ø)

Source/AccountViewCell.swift
86.66% <86.66%> (ø)

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 e269b56...d5c2d07. Read the comment docs.

@saeedbashir PR is ready for another pass. Thanks

@saeedbashir The PR is ready for another pass.

User profile is directly accessible from left drawer.

It would not be remove as per specs.

There should a title of User Profile screen

I have asked the question about this from Liz to show or not, currently we are not showing the title for profile.
Thanks

If it is easy to drop a screenshot here I could provide visual feedback early as well if that helps.

@marcotuts  I have added the screenshots, Please take a look into them, thanks.

@saeedbashir I have fixed comments. Thanks

@saeedbashir i am done with comment. Thanks


SineMovement 0.0.5

Version 0.0.5
Allow an object to move smoothly on the X and/or Y axis following a sine wave.
Example uses:

Floating objects, such as powerups or coins
Moveable platforms
Enemy movement patterns

Properties:

Center of movement, X position
Center of movement, Y position
Horizontal distance
Vertical distance
Horizontal speed
Vertical speed

Tips:

For circular or elliptical movement, the horizontal and vertical speed need to be the same
For horizontal movement only, set vertical distance to 0
For vertical movement only, set horizontal distance to 0
For figure-8 movement, set horizontal speed to 1/2 of the vertical speed

Example game
https://victrisgames.itch.io/extension-sinemovement-and-deptheffect
Extension file
SineMovementExtension_0.0.5.json.zip
Video
https://user-images.githubusercontent.com/8879811/118032950-726f7100-b325-11eb-8f78-de0856ce99a2.mp4

I've made small edits on description value like:

Otherwise, it's a great improvement, thanks for your time!
Approved and merged!

Thanks both 👍


Nothing coming back from asyncIterator

I'm just trying to test my KafkaPubSub and have a really simple setup:
Resolver
  Subscription: {
    messageCreated: {
      subscribe: () => pubsub.asyncIterator(MESSAGE_CREATED)
    }
  }

Publish
setInterval(() => {
  let r = pubsub.publish({
    channel: MESSAGE_CREATED,
    messageCreated: { id, content: new Date().toString() },
  });

  console.log(r, id)

  id++;
}, 5000);

The publishes are happening and I see no error with them, but even though my subscription is started up in the playground it just listens and nothing ever comes through. I'm not sure if maybe it is something with kafka itself? My configs are
export const KafkaPubSubConfig = {
    topic: <TOPIC>,
    host: <BROKER>,
    port: <PORT>,
    groupId: 'graphql-sub-test',
}

export const PublisherKafkaPubSubConfig = {
    ...KafkaPubSubConfig,
    globalConfig: {

        'security.protocol': 'sasl_ssl',
        'ssl.endpoint.identification.algorithm': 'https',
        'ssl.ca.location': <PATH_TO_CERT>,
        'enable.ssl.certificate.verification': false,
        'ssl.keystore.location': <PATH_TO_KEYSTORE>,
        'ssl.keystore.password': <PASSWORD>,
        'sasl.mechanism': 'GSSAPI',
        'sasl.kerberos.keytab': <PATH_TO_KEYTAB>,
        'sasl.kerberos.principal': <PRINCIPAL>,
        'sasl.kerberos.service.name': <NAME>,
        'group.id': 'librd-test',
    }
}

So I figured out that all it was that I needed to be updating the group.id in my tests. Once I started doing that I didn't have a problem.


added rule for NRC Q

Resolves issue: https://github.com/typpo/ad-detector/issues/34

Thanks! :+1:


License file missing

Hi, thank you for creating great new addition to Deep MVS world and providing pretrained weights.
I would like to test your network, but I need to know what a license is for using or modifying your code.
Thank you for answer!

Thanks. We add the license.


[Water Shader] - Fine-tune flow of water foam

Hello,
First of all, really neat asset!
I would really like to use the included water shader, but it's hard to make rivers, or other bodies of water, where the flow of the foam needs to be curved or fine-tuned.
I tried messing with the UVs to get the desired fine-tuned foam direction, but it's not that intuitive and it causes other problems.
A long shot, but could we use vertex colors instead, to determine the direction of the foam flow?
I know this might be a lot to task, but let me know what you think.

Thanks for the suggestion, we'll look what can be done.
When using vertex colors, how would you parametrize the flow direction? RGB as XYZ of the direction in object space?

Yeah! RGB to XYZ seems to be the most straightforward in paper at least. How would the values be mapped though? Considering we want the axis to go forward and backward. From 0 to 255 and from -1 to 1? Or something else?

[-1, 1] will have to be remapped to [0, 1].
Thank you for the suggestion, we will consider this for an update.


Login Response is not returning refresh-token in dj-rest-auth v3.0.0

I have been using dj-rest-auth for a while, i installed the version 3.0.0 in my current project and it is not working properly.
When i login a user the response return an empty string instead of a refresh.
My solution was downgrading to and old version that i know thar works well (2.2.4)
{
"access_token": "eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJ0b2tlbl90eXBlIjoiYWNjZXNzIiwiZXhwIjoxNjc3MTU5NzE4LCJpYXQiOjE2NzY1NTk3MTgsImp0aSI6Ijk2Y2M3NDZmM2ZmOTQ3NWRiYzBmMGVkYzFlYTYyYmY2IiwidXNlcl9pZCI6MX0.rgrJVCs0kgxgU7h6G6lsI-mSly6IZJULQEIh0vQ9lSA",
"refresh_token": "",
"user": {
"pk": 1,
"username": "admin",
"email": "EMAIL",
"first_name": "Pepe",
"last_name": "1234"
}
}
It is an example for the response of version 3.0.0, the request show it empty but the cookie for the refresh token seems to be updated.

My project config is :
REST_FRAMEWORK = {
'DEFAULT_AUTHENTICATION_CLASSES': [
'dj_rest_auth.jwt_auth.JWTCookieAuthentication',
],

'DEFAULT_PERMISSION_CLASSES': [
    'rest_framework.permissions.IsAuthenticated',
],

}
REST_AUTH = {
'USE_JWT': True,
'JWT_AUTH_COOKIE': 'my-app-auth',
'JWT_AUTH_REFRESH_COOKIE': 'my-refresh-token',
}
SIMPLE_JWT = {
'ACCESS_TOKEN_LIFETIME': timedelta(minutes=10000),
'REFRESH_TOKEN_LIFETIME': timedelta(days=10),
'ROTATE_REFRESH_TOKENS' : True,
}

If you add "JWT_AUTH_HTTPONLY": False, it will work.
REST_AUTH = {
    "USE_JWT": True,
    "JWT_AUTH_HTTPONLY": False,
    ...
}

We had the same issue with version 3.0. Setting JWT_AUTH_HTTPONLY to False did the trick.
This line is probably causing the issue, since it previously defaulted to False for JWT_AUTH_HTTPONLY and the refresh token is set right after.
https://github.com/iMerica/dj-rest-auth/compare/2.2.8...3.0.0#diff-6605a895156338ef26e802ff3d8fd57aa730b05e9340402bf69cd0a823e4b9dbL92

Sorry for late response. Actually, i already add a note about this behaviour in docs here https://dj-rest-auth.readthedocs.io/en/latest/configuration.html#jwt-auth-httponly.

Thank you guys for you support !


Don't know how to open this program and use it

Hello. I'm someone trying to use osukps. I have no idea which program to open. Please help me out here, because I don't know what to do to use this casually.

Anonymous

download it on here, https://github.com/yugecin/osukps/releases/download/180218/osukps.exe


Review OSF API

As a dev
I want to examine the API endpoints of OSF
so that I gain a general understanding of how to interact with it.
AC:

[x] Review selected API endpoint in the design document https://osf.io/anbhu/
[x]  Review official API documentation https://developer.osf.io/
[x] Get an understanding about Authentication to OSF
[x] Create example code for the Authentication mechanism
[ ] Create documentation in Gitbook - in progress

Use following criteria:

[x] Need way for person to use PresQT an authenticated way with OSF
[ ] Need ability to retrieve a single file
[ ] Need ability to retrieve all the files in a single item (probably as a zip file or a bagit file, but the format is not part of the requirement)
[x] Need ability to retrieve a list of items
[ ] Need ability to search (using the visibility of the authenticated user). Kinds of searches: boolean combinations of:

[ ] All items I can edit (even if via a group I belong to)
[ ] All items I deposited
[ ] All items having a given type
[ ] by DOI
[ ] by file hash
[ ] uploaded before/after a given date

[ ] Get information about an item: metadata, list of files, dates file was uploaded, revision history
[ ] Request a fixity check for an item
[ ] Deposit a single item
[ ] Add a file to existing item
[ ] Update data for a single item
[ ] Deposit a group of items

Ticket will be closed. API requirements are captured here now


feat(create)! subcommand: Very Good Flutter Project Usage

I want to work on this feature that supports other state management patterns like Riverpod and Provider.
This allows users to create projects with other preferred state management patterns but still supports Bloc by default.
Very Good Flutter Project Usage
very_good create
options:
--arch

Hi @Yczar 👋
Thanks for opening an issue!
Our templates are VGV-opinionated but feel free to create your own mason templates or use existing templates from the community to fit your needs!
Closing for now since this isn't within the scope of the Very Good CLI, hope that helps! 👍


Consume value from callback

What do you think about creating a new method TryWithData or changing the signature of the existing Try method like below
public static TryWithData<T = {}>(
    callback: (...args: unknown[]) => T,
    errorTransformer?: (e: unknown) => Error | null
  ): Result<T> {
    try {
      const data = callback();
      return new Result<T>(true).WithData(data);
    } catch (e) {
      const error =
        typeof errorTransformer === 'function' ? errorTransformer(e) : null;
      return Result.Fail<T>(error);
    }
  }

Note that in the snippet above, the callback returns T instead of any type. As a result, we can use the outcome as data

I think this makes a lot of sense. Modifying the existing Try method is probably the way to go, it was an oversight on my part. We should make the return type of callback() optional perhaps?

Let it be nullable. I'm fine with that. I've created PR #3

Fixed by #3


Unify transaction behavior with other drivers

There is now no need to explicit roll back a failed transaction
(run).
Commit and Rollback can only be called once.
Rollback does not return an error from a failed run anymore.

Companion PR: https://github.com/neo4j-drivers/testkit/pull/438


Add boolean validator rule to default rules

I think that is a nice addition as it is a standard validation rule to check if a checkbox is checked or not. This would then just validate whether a boolean is true(valid) or not||undefined(invalid).
A use case is a signup form where the user needs to aggree to the Terms&Condition.

for now i used a custom rule like this:
ValidationRules.customRule(
  'checked',
  (value, obj, bool) => {
    return value === null || value == bool;
  },
  "${$displayName} must be ${bool}.",
  (bool) => ({ bool })
);

........

.ensure('terms').satisfiesRule('checked', true)

Or you can simply say
.ensure(a => a.agree).satisfies(v => v === null || v == true).withMessage('You must agree to the terms')

Actually, unless I'm missing something...
.ensure(a => a.agree).required().withMessage('You must agree to the terms')

I was missing something. required() considers boolean false to be a value...
Might need to look at that too...

thx for that, i knew i oversaw something.

@eikaramba / @jsobell what do you think about:
.ensure(a => a.agree).equals(true).withMessage('You must agree to the terms')

Not sure how broadly applicable this will be but it seems like it might be a tiny bit more generic than a "checked" rule.

i think it is great :)

Sounds good. It will probably only ever be used for ===true.
Can you think of any other usage scenarios? My first thought was that you could allow an expression... which makes it identical to .satisfies.

lets sit on this one for a while and wait for some additional use-cases.

well every checkbox where you want it to be checked would be the use case. I guess that is quite a big one actually. but if satisfies() works, i guess it would be enough for now to update the docs and add the example.

This came up again today. We either need a .true() or a '.equals(). The only other case for .equalsI can think of is.equals("I AGREE")` on a text field, and even that's questionable.

in favor of a more generic solution i would prefer equals(). sure it seems unlikely that this has a real-world impact, but i guess one should better not limit the users expectations here. For example github is making you enter the exact repository name again in a textfield, before the delete button is activated.

ok, taking this off hold and adding it to the beta milestone.


New command: Remove administrative unit

Usage
m365 aad administrativeunit remove
Description
Removes an administrative unit
Options

Option
Description

-i, --id [id]
The ID of the administrative unit. Specify either id or displayName but not both.

-n, --displayName [displayName]
The display name of the administrative unit. Specify either id or displayName but not both.

-f, --force [force]
Don't prompt for confirmation.

Examples
Remove an administrative unit by its id
m365 aad administrativeunit remove --id 03c4c9dc-6f0c-4c4f-a4e6-0c9ed80f54c7

Remove an administrative unit by it displayName
m365 aad administrativeunit remove --displayName 'Marketing Division'

Default properties
No response
Additional Info
https://learn.microsoft.com/en-us/graph/api/administrativeunit-delete?view=graph-rest-1.0&tabs=http
It requires new scope AdministrativeUnit.ReadWrite.All.
I will work on this.

Looks good @MartinM85, I've assigned you..

A minor detail: -f, --force is a flag and doesn't expect a value so we should remove [force] from the spec and implementation

done!


Pin napalm-base driver

See details in the parent issue napalm-automation/napalm-base#107

Closing via https://github.com/napalm-automation/napalm-eos/pull/61


not able to run drawflow index.html with new code changes in drawflow.js and pointing script to .js file.

Hi ,
I am utilizing drawflow for my project purpose and I have followed the steps given in readme file.
It works well when script sources to drawflow.min.js from githlink .
Now I tried to add few buttons and its functionality and pointed the script to drawflow,js , it gives an error as below

Can you please help in this regard ?

Hi @sompoo
How are you importing drawflow?

hi  @jerosoler ,
I am importing drawflow at start of the drawflow.js thinking that when control will come on line 101 on index.html , it will have the Drawflow already imported and available.

Please suggest the correct way if this is wrong.

yes I installed drawflow package before importing and followed the steps as mentioned in readme.
If possible can we please connect over a call to quickly to check where this is going wrong?

How is the server running?
With what command?


refactor(csg.js): split out various pieces into seperate modules

This pr splits up the HUGE csg.js file into more manageable , specialised chunks

split out various 2d & 3d shapes
split out maths (vectors, matrices etc)
split out utilities
make code base more manageable and offer better overview for future improvements

This might take a while because of the huge amount of interdependencies, but it is feasible :) (I have done it before in Coffeescript

It seems to me that some "classes" are redundant. For example CSG.Line2D and CAG.Path. Good luck.

Wow! That's a huge improvement in readability and understandability.
Look fine.

thanks @z3dev !

Not perfect, but it all works ok, I plan a second one to improve things, but one step at a time, merging!


Refactor collaborations into its own table

This PR introduces a new Collaboration struct which is a direct relation between a repository and a non-organization user. Introducing this table will make it much easier to refactor the access table.

Superseded by #899


Links with svgs = Hard refresh

I was wondering why clicking some links was doing an hard refresh instead of loading the page via barba.js. Here is the link :

<svg class="icon--logo " role="presentation"><use xmlns:xlink="http://www.w3.org/1999/xlink" xlink:href="/images/sprites.svg#logo"></use></svg> Logo name

The linkClicked event is not triggered.
If I replace this code by a regular a:href element with some text I dont have any issues anymore.

Wow it seems this point has been fixed in latest release!! Sorry for the noise!! Great job on this library.

Actually I just checked and this is still a bug.
The problem is that, clicking on the <use> tag, the xlink:href is detect as the final link.
https://github.com/luruke/barba.js/blob/master/src/Pjax/Pjax.js#L193
But then the link is ignored because
https://github.com/luruke/barba.js/blob/master/src/Pjax/Pjax.js#L266
I'll think about a solution!

I should maybe discard the link on all the use tag... hope it make sense

Yes, I just double check by upgrading to version 1.0.0 and this is still an issue.
Maybe the script should find the closest parent that is a .

A non-JS workaround could be to set  pointer-events: none on the <svg> element.

Has anyone got this to work using @andershopkins technique or something else? I'm using v1.0.0 and I've tried using pointer-events: none on the relevant parts of the navigation svg but I can't capture the click.
I'm thinking the solution might be to use non  elements and just recreate the "clicking the link" event manually but that seems a shame when I suspect Barba does what I want and I'm just missing something obvious (to you!). Any ideas?

Just realized issue #174 is due to this bug. What's really strange is that on my test project it only happens intermittently:
http://twelvenyc.grossdevelopment.net/

This article actually explains why it could happen intermittently — it depends on whether the click event target is the wrapping anchor tag, svg, or use element:
https://css-tricks.com/slightly-careful-sub-elements-clickable-things/


Add a name to the V3 file-based SDS tls secret response

This applies only to the V3 protocol.
This impacts the tls_certificate_sds_secret.json config file, not the
response sent over the wire to xDS clients.
Marin3r generates this file and adds it to a k8s secret which is
mounted into Envoy pods. Envoy uses this file to retrieve the
bootstrap certs that allow it to talk to Marin3r over TLS. If there's
no "name" field then Envoy v0.17.0 can't process the cert, although
older Envoy versions can.
[2021-02-18 23:20:51.419][1][critical][main] [source/server/server.cc:109] error initializing configuration '/etc/envoy/bootstrap/config.json': Proto constraint validation failed (UpstreamTlsContextValidationError.CommonTlsContext: ["embe
dded message failed validation"] | caused by CommonTlsContextValidationError.TlsCertificateSdsSecretConfigs[i]: ["embedded message failed validation"] | caused by SdsSecretConfigValidationError.Name: ["value length must be at least " '\x0
1' " runes"]): common_tls_context {
tls_certificate_sds_secret_configs {
sds_config {
path: "/etc/envoy/bootstrap/tls_certificate_sds_secret.json"
}
}
}
[2021-02-18 23:20:51.419][1][info][main] [source/server/server.cc:782] exiting
Proto constraint validation failed (UpstreamTlsContextValidationError.CommonTlsContext: ["embedded message failed validation"] | caused by CommonTlsContextValidationError.TlsCertificateSdsSecretConfigs[i]: ["embedded message failed valida
tion"] | caused by SdsSecretConfigValidationError.Name: ["value length must be at least " '\x01' " runes"]): common_tls_context {
tls_certificate_sds_secret_configs {
sds_config {
path: "/etc/envoy/bootstrap/tls_certificate_sds_secret.json"
}
}
}
Thank you @roivaz for pointing me to this fix!

This is a trivial change but will hopefully save somebody some time when they decide to upgrade to Envoy v0.17.x.

Closes #74

/ok-to-test

/lgtm

Tanks @acnodal-tc for the contribution! :)
One of the test fails after the change because it does not expect a name property to be set:
--- FAIL: TestConfig_GenerateSdsResources (0.00s)
    --- FAIL: TestConfig_GenerateSdsResources/Returns_a_string_with_the_envoy_booststrap_static_configuration (0.00s)
        bootstrap_test.go:79: Config.GeneratSds() = map[tls_certificate_sds_secret.json:{"resources":[{"@type":"type.googleapis.com/envoy.extensions.transport_sockets.tls.v3.Secret","name":"xds_client_certificate","tls_certificate":{"certificate_chain":{"filename":"/tls.crt"},"private_key":{"filename":"/tls.key"}}}]}], want map[tls_certificate_sds_secret.json:{"resources":[{"@type":"type.googleapis.com/envoy.extensions.transport_sockets.tls.v3.Secret","tls_certificate":{"certificate_chain":{"filename":"/tls.crt"},"private_key":{"filename":"/tls.key"}}}]}]

Could you also update the test expected result with the name property so it passes correctly? It would be these lines:
https://github.com/acnodal/marin3r/blob/6d5e58480e68c562890daec77ffc791f082c8f43/pkg/envoy/bootstrap/v3/bootstrap_test.go#L65-L67. You run the unit tests locally using make unit-test.

Sorry about that, and thank you for the pointer to the fix!

/ok-to-test

@acnodal-tc the unit tests are now passing, but the e2e tests have failed for the v3 API using envoy 1.16. I've looked into it and it seems that the name specification is also required here:
https://github.com/3scale-ops/marin3r/blob/8325a106a8b8d5dbb884637376018cd16f718d30/pkg/envoy/bootstrap/v3/bootstrap.go#L41-L46
It should be
{
	Name: "xds_client_certificate",
	SdsConfig: &envoy_config_core_v3.ConfigSource{
		ConfigSourceSpecifier: &envoy_config_core_v3.ConfigSource_Path{
			Path: c.Options.SdsConfigSourcePath,
		},
	},
},

The appropriate unit test would need to be modified also here to add the name:
https://github.com/3scale-ops/marin3r/blob/8325a106a8b8d5dbb884637376018cd16f718d30/pkg/envoy/bootstrap/v3/bootstrap_test.go#L29


Feature/auth service tests

I've created 3 scenarios:

Single account registration
Registration of two accounts one after another. Each has different email.
Registration of two accounts one after another. Both have the same email.

NowakSharedRentApplicationTests can be removed.


Concourse extremely slow

Hi guys, we've been using concourse-up for more than a year and all good, but since the upgrade to 4.x everything got painfully slow.
I've opened this one https://github.com/concourse/concourse/issues/2604 but so far it only got ignored.
Additionally, we are unable (and I am not sure why) to ssh in the boxes, either downloading the private/pub keys from the json in s3 or with the command line setting up the env variables.
Can you help?
Thanks

Hi @aterreno - sorry we missed your original issue. Let's try to work this one out. Could you join us on Slack to help diagnose the issue? https://join.slack.com/t/concourse-up/shared_invite/enQtNDMzNjY1MjczNDU3LTA1NzIxYTZkYjFkMjA2ODBmY2E2OTM3OGE3YTc2OGViNTMxYTY4MjYwNGNjOTAxNDNiOGE5NzhmMTQ2NWVhNzQ

Adding context and infos as we progress this on Slack.

Cpu at 40+% on idle (no jobs running)

Recreating the worker didn't change the situation, cpu is at min (not even avg) at 40+%
concourse.worker.0-20180927-104150-734269313.zip
Attached logs of worker when idle.

Closing this one up for now, as performances improved substantially by removing the npm-repo-cache resource from all my pipelines (20~)


There is not link to here on the Modrinth page

There is no source or Issues button to get to this github page on modrinth.

I fixed it.


Flow is confused when a method is overloaded with a different signature

Consider the following scenario:
class Base {
  change(value: string):void {
  }
}

class A extends Base {
  change(value: string, option: boolean): void {
  }
}

It shouldn't be an issue but Flow complains with:
undefined (too few arguments, expected default/rest parameters). This type is incompatible with
9:   change(value: string, option: boolean): void {

In order for class A to be a subtype of class B,  any overridden methods must also also be a corresponding subtype. Fn1 is a subtype of Fn2, if the input of Fn1 is a supertype of the input of Fn2 (contravariant) and the output of Fn1 is is a subtype of the output of Fn2 (covariant).
https://github.com/facebook/flow/pull/3620
Given this type hierarchy...
?string :> string :> 'foo'  ...and... ?number :> number :> 0
...this is valid:
class Base {
  change(value: string): ?number {
  }
}

class A extends Base {
  change(value: ?string): number {
    return 0
  }
}

No, this blog post helped me to understand that this is actually because of the way javascript natively overrides (or actually doesn't) methods.
The method parameters are natively an Array, and Flow obviously sees identical functions and expects their parameters / arrays to match.
This is an edge case, but I think this is a real issue since it seems to be valid to override a method with a different signature, even though it's probably not really a good programming practice.

valid to override a method with a different signature
It's only valid to override a method with a subtype of the method of the parent.

Outside of mutating the the class, can you construct an example where this is not safe?

It's only valid to override a method with a subtype of the method of the parent.
Do you mean my initial example, flow aside, is not ES2015 valid ? It compiles and works as expected.

Correct. Your initial example is unsafe:
class Animal {
  change(value: string):void {
  }
}

class Dog extends Animal {
  change(value: string, option: boolean): void {
    // option is undefined!
  }
}

function needsAnimal(value: Animal): void {
  value.change('')
}

const mayDog = new Dog()

needsAnimal(mayDog)

But this is safe:
class Animal {
  change(value: string, option: boolean):void {
  }
}

class Dog extends Animal {
  change(value: string): void {
    // all good!
  }
}

function needsAnimal(value: Animal): void {
  value.change('', true)
}

const myDog = new Dog()

needsAnimal(myDog)

Unsafe, but valid 😉
And actually, in my project, change() was actually private, therefore perfectly valid to use !
Again, I think the Flow error could be more helpful, but I'm closing the issue anyway, cf #4590 !!


x/pkgsite: landing page for docs provides no structural information at a glance.

This is a UX issue.
When reading docs on a package there are two important pieces of information a programmer is after: an overview (AKA the package comment) and a list of functions, types and their constructors.
When visiting the old godoc this is how it is presented:

As you can see there is the package name, the overview and at a glance you get all you need to construct and use types in the package.
When visiting the new pkg.go.dev this is what the user is presented with:

to get the same kind of information of the old doc the user has to click 9 times to expand all folded menus, and it uses way more space.
For a small package like io I had to flip my screen in vertical mode to take the screenshot below.

Please also note that the only way to know the parameters of NewSectionReader is to hover it with the mouse as the left pane is not resizable.
It would be nice to have an overview of the package structure, types and functions at a glance, without having to dig into folds. This view:

provides no information as it is going to be identical for every package I visit.
Note that this, at least for me, is a big enough reason to stop using pkg.go.dev since it does not provide the information I am always after, but this is a personal opinion and I don't know if others might feel the same.

Possibly a dup of #40612?

@empijei I have a CL to expand all items by default. While I understand this doesn’t cover all of your concerns, would you consider that mitigation sufficient for this bug? We can track further improvements on #40612.
Let me know what you think.

This is indeed a dup of #40612, sorry for not finding it.
My bug also adds that the space taken by the fully expanded list is way more than it used to. I'll close this and add the comment on the other bug. Thanks.


[WIP] [1.22] cgroup v2

Link to the enhancement: https://github.com/kubernetes/enhancements/issues/2254
Signed-off-by: Giuseppe Scrivano<EMAIL_ADDRESS>

thank you for opening this!
/assign @chrisnegus
/sig node

Hi @giuseppe :wave: I'm the 1.22 release docs shadow assigned to track this PR. Please take a look at Documenting for a release - PR Ready for Review to get your PR ready for review before Tuesday 20th of July, 11:59 PM PDT. Thank you!

@chrisnegus Giuseppe is out on vacation. I will look at this early next week and open a new PR if needed.

@chrisnegus @mrunalp @saschagrunert I had a look at the documentation, I am not sure there is anything we need to add for cgroup v2 as there are no user visible changes

I think we could add a paragraph about cgroups v1 vs v2 to https://kubernetes.io/docs/setup/production-environment/container-runtimes/ mentioning:

Outlining the benefits of using v2 instead of v1
How to configure a node using cgroups v2
That container runtimes (as well as OCI runtimes) have to support this as well
That nothing should change on the user-experience when switching to v2

Hi @giuseppe 👋 I'm the 1.22 release docs shadow assigned to track this PR. Please take a look at Documenting for a release - PR Ready for Review to get your PR ready for review before Tuesday 20th of July, 11:59 PM PDT. Thank you!

Hey folks, friendly reminder that today is the deadline for PRs ready for review for docs, please make sue that this PR is ready to review by the end of the day, thank you!

/remove-language fr
/remove-language id
/remove-language ja
/remove-language ko
/remove-language pt
/remove-language ru
/remove-language zh

@tengqm thanks for the review.  Addressed the comments and pushed a new version

/lgtm
/approve


[BUG] sensorextraction/ros-to-parquet module failing at run time due to requests lib not installed

Describe the bug
requests lib is missing from requirements.txt in ros-to-parquet module at autonomous-driving-data-framework/modules/sensor-extraction/ros-to-parquet/src/main.py. This is causing the batch job to fail at runtime with the following error:
To Reproduce
Steps to reproduce the behavior:

deploy ros-image-demo
run airflow job
sensor-extraction.parquet-extraction-batch-job fails
view aws batch job logs

@timestamp
@message

2022-11-04 18:18:21.649
Traceback (most recent call last):

2022-11-04 18:18:21.649
File "main.py", line 24, in

2022-11-04 18:18:21.649
import requests

2022-11-04 18:18:21.649
ModuleNotFoundError: No module named 'requests'

Expected behavior

Module does not error and successfully converts ros topics to parquet.

PR is merged


Annotate Cloud Bigtable implementation details as Internal

Annotate Cloud Bigtable implementation details as Internal

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

@pabloem @mutianf

@pabloem

LGTM!


ensure initial route change is registered

This fixes an issue that the breadcrumb is not displayed right after page load or page refresh.

Cheers @mmmichl! This is now in https://github.com/emilol/angular2-crumbs/releases/tag/v1.2.0


docs: add guide for jwt and acl plugin use

What this PR does / why we need it:
Adds a guide for configuring the JWT and ACL plugin via kubernetes charts.   Demonstrates the use of kubernetes secrets to store RSA keys and ACL groups
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

This is awesome. We read through some of the code just the other day to reverse engineer the process of setting up an ACL, and it was on our TODO list to contribute docs for it, but you beat us to it with a PR that covers even more. 👍

great!

Let's make sure the guide works as is. Users should be able to copy paste the commands and make it work. I don't see that being possible with the current guide.
Let's also add the steps to test the setup to prove that it works. See other guides on how it is done there.

 All committers have signed the CLA.

@lostrouter A friendly ping to see if there is any progress here. This is really close and a very important doc that we would like to add.

@hbagdi i will set aside some time this week to to make the requested changes.

@hbagdi changes have been made.

@lostrouter  Please fill in the contributor form to claim your Kong swag: https://github.com/Kong/kong/blob/master/CONTRIBUTING.md#contributor-t-shirt


Return value and typing documentation discrepancies

While working on adding PySCHISM support, I noticed that in both PySCHISM and ADCIRCPy some of the "model forcing" types are not actually derived from the base Forcing (in case of ADCIRCPy) or ModelForcing (in case of PySCHISM) types. One example of this is Tides type in both.
In spite of this, the return values are assumed to be of Forcing type for ADCIRCPy configuration classes:
https://github.com/noaa-ocs-modeling/CoupledModelDriver/blob/a97e5ec9ee44ed30cf8cb313b8418f4807f1ce3e/coupledmodeldriver/configure/forcings/base.py#L148-L149
How should this be addressed? This issue becomes more crucial when adding other models (e.g. PySCHISM in my case #136).
This issue discusses possible solutions for this issue

The basic issue is that ForcingJSON has to_adcircpy which should ideally return a single type - a superclass of all forcing types - is returning the superclass of only "some" of the forcing types; e.g. Tides is a Boundary Condition not a subclass of Forcing. While it can be debated that tidal boundary conditions are not forces physically or mathematically, still it would be good to have a single abstraction returned from the this method. Let's just call every constrain on the model a "force".
The problem now becomes how to provide that abstraction. Should there be ADCIRCPyForcingAdaptor or PySCHISMForcingAdaptor class? @zacharyburnettNOAA how would you see the Enum work here?
An alternative approach is to have a ADCIRCPyGenerator or PySCHISMGenerator objects that get these JSON configuration objects as input and generate solver specific outputs, instead of the *JSON objects all having to_<model> methods. This might require extensive refactoring.

that might be a more extensible and modular approach, instead of having to incorporate a to_<model> method for each model

Adding the "generator" classes would change the API a little bit which also requires a change in tests. Is that OK to proceed with it? I can create a new branch and try it and then switch back to pyschism effort

Actually I guess it might be better if instead of a single generator class, we have a generator for each forcing inside the model (e.g. coupledmodeldriver/generate/adcirc/forcings/*). Does that make sense?

If we use generators, we need to find a good way to take care of from_adcircpy and from_user_input methods too

sure thing! I think that would work with only a few changes

@zacharyburnettNOAA I'm still a little bit stuck here. The ideal solution for me looks like this: configure modules are base, then generate modules rely on the configure, and lastly client relies on both. Also I'd like configure not to have solver specific (SCHISM or ADCIRC) code. But I don't see how it's possible.
I can move the *.to_adcirc() and *.adcirc_forcing stuff under generate/adcirc; but then for the from_adcirc functionality I either need to add/keep adcircpy dependency in configure or instead move from_* to generate/adcirc as well; then I'll have circular dependency between configure and generate/adcirc. Alternatively the *JSON classes could be modified such that there's no from_adcircpy or to_adcircpy or adcircpy_forcing method or property, but rather the object needs to be passed to another object or function defined in generate/adcirc or generate/schism to go back and forth between JSON and Configuration realization of it.
What do you think?
For now instead of this refactoring, I'll focus on adding SCHISM. But let's continue discussing the approach to refactoring to make it ocean model agnostic!


Push to live

:x: Validation status: errors

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/NuGet/docs.microsoft.com-nuget, working directory: W:\x4yo-s, target branch: live, target branch commit: 65725d432e536454b5447b984d8959eeb08f89fd, source branch: master, source branch commit: .

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.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/Create-Packages/Publish-a-package.md
:white_check_mark:Succeeded

docs/Tools/cli-ref-install.md
:white_check_mark:Succeeded

docs/Tools/cli-ref-push.md
:white_check_mark:Succeeded

docs/Tools/cli-ref-restore.md
:white_check_mark:Succeeded

docs/Tools/cli-ref-update.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.


SyntaxError: Unexpected token :

Describe the bug

If default is () => ({ a: 1, b: 2 }). We got SyntaxError: Unexpected token :
If default is default: () => ({}) or default: () => ({ a: 1 }). We got -.
If default is default(){return{}}. We got default() {\n  return {};\n}.

To Reproduce
{
  props: {
    name: {
      default: () => ({ a: 1, b: 2 }),
    }
  },
}

Expected behavior

No error.
Add ` before and after the function code to avoid \n?

Screenshots

With Vuepress

Which version do you use:
@vuese/parser: 2.1.3

@babel/generator 7.0 has a bug:
const source = '() => ({ a: 1, b: 2 })'
const code = generate(source)  // () => { a: 1, b: 2 }  Lost Parens

It seems that @babel/generator 7.4 solves this problem, so I tried to upgrade the version of @babel/* related dependencies, but I have encountered some problems: there is no corresponding update in @types/*, which leads to more problems and needs a lot of Modifications, I think it's a more correct choice to wait until @types/* is updated.
Now my solution is to avoid interruptions caused by program errors, which means that the resolution of the default values may not be correct, so we recommend making things perfect by adding leading comments. E.g:
{
  props: {
    name: {
      // `{ a: 1, b: 2 }`
      default: () => ({ a: 1, b: 2 }),
    }
  },
}

BTW: Use<EMAIL_ADDRESS>


rework cli to accept circuit params

Rework cli to use commands and accept the required circuit params
Work split out from #686

closing in favor of #702


LineItemAttributes Struct is breaking on parsing custom fields

When parsing the custom fields the line items struct breaks searching the new structure if the value of $lineItem->getPayload()['customFields'] is null.
https://github.com/mollie/Shopware6/blob/master/src/Struct/LineItem/LineItemAttributes.php#L197-L203
# load the custom fields
$customFields = $lineItem->getPayload()['customFields'];

# ---------------------------------------------------------------------------
# search in new structure

$fullKey = 'mollie_payments_product_' . $keyName;
$foundValue = (array_key_exists($fullKey, $customFields)) ? (string)$customFields[$fullKey] : '';

array_key_exists can not work with null values. $customFields should be checked before.

Hi @michnhokn
oh wow, that use case has slipped thru
thx so much, we'll add a fix and a unit test :)

Hi
a PR is already existing (https://github.com/mollie/Shopware6/pull/381)
thank you for telling us about this one

released in 2.3.1

@boxblinkracer
This is still an issue on version 2.5.0 of Mollie plugin, Shopware version <IP_ADDRESS>
[2022-09-07T09:30:04.652639+00:00] request.CRITICAL: Uncaught PHP Exception TypeError: "array_key_exists(): Argument #2 ($array) must be of type array, null given" at /vendor/store.shopware.com/molliepayments/src/Struct/OrderLineItemEntity/OrderLineItemEntityAttributes.php line 139 {"exception":"[object] (TypeError(code: 0): array_key_exists(): Argument #2 ($array) must be of type array, null given at /vendor/store.shopware.com/molliepayments/src/Struct/OrderLineItemEntity/OrderLineItemEntityAttributes.php:139)"} []

Hi @wexollm
a damn, because it has been fixed in the LineItemEntity attribute and not OrderLineItemEntityAttribute
cart vs. order....
I'll get it fixed, sorrry

Hi @wexollm
heres the fix
https://github.com/mollie/Shopware6/commit/fd1d4e972c1564f9e39df4bba3f66bcd9b811482
comes next week :)

@boxblinkracer
great :)


Audio support for Goose

Add audio media support for goose for such things like speech-text prompting

Rudimentary support for the meantime https://github.com/block/goose/issues/114


Metaphor Key - for cloud deploy - no .toml and no .env

I am trying to deploy to Azure App Serivices. I have ARM templates that work fine. The one issue I am having is that i need to set API keys as if they are stored as environment variables.
For OpenAI -
 os.environ["OPENAI_API_KEY"] = st.secrets.openai_key
from utils.py makes sense
but am not seeing anything that straightforward for metaphor
Am I missing something?

with some sage guidance from @logan-markewich -i have tbis figured out - will get pr together today

PR Submitted - https://github.com/run-llama/rags/pull/54


Add clone deep

Fixes issue #897

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Васильев Анатолий Андреевич01 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Change the callback in DLExt::ForceIml to take an integer argument

This is in order to have a consistent API for the Sampler classes in PySAGES.

Fixed in #8.


react scripts now logs long useless logs, add option to remove them

Is your proposal related to a problem?

After upgrading to react scripts 5, npm start now has these ugly logs each time

assets by status 3.38 KiB [cached] 1 asset
assets by chunk 8.92 MiB (name: main)
asset static/js/bundle.js 8.91 MiB [emitted] (name: main) 1 related asset
asset main.61ec10f63a1c61f3f0e3.hot-update.js 10.2 KiB [emitted] [immutable] [hmr] (name: main) 1 related asset
assets by path *.json 431 bytes
asset asset-manifest.json 403 bytes [emitted]
asset main.61ec10f63a1c61f3f0e3.hot-update.json 28 bytes [emitted] [immutable] [hmr]
asset index.html 1.69 KiB [emitted]
Entrypoint main 8.92 MiB (8.64 MiB) = static/js/bundle.js 8.91 MiB main.61ec10f63a1c61f3f0e3.hot-update.js 10.2 KiB 3 auxiliary assets
cached modules 8.43 MiB [cached] 1092 modules
runtime modules 28.3 KiB 14 modules

they weren't in the previous version and i don't care about webpack or babel,
also when an error pops i get this

ERROR in ./src/Components/HomePage/HomePage.js 7:0-51
Module not found: Error: Can't resolve './HomeasdsadPage.module.scss' in 'C:\Users...................-frontend\src\Components\HomePage'
resolve './HomeasdsadPage.module.scss' in 'C:\Users............-frontend\src\Components\HomePage'
using description file: C:\Users.................-frontend\package.json (relative path: ./src/Components/HomePage)
Field 'browser' doesn't contain a valid alias configuration
using description file: C:\Users........-frontend\package.json (relative path: ./src/Components/HomePage/HomeasdsadPage.module.scss)
no extension
Field 'browser' doesn't contain a valid alias configuration
C:\Users........-frontend\src\Components\HomePage\HomeasdsadPage.module.scss doesn't exist
.web.mjs
Field 'browser' doesn't contain a valid alias configuration
C:\Users........-frontend\src\Components\HomePage\HomeasdsadPage.module.scss.web.mjs doesn't exist
.mjs
Field 'browser' doesn't contain a valid alias configuration
C:\Users..........-frontend\src\Components\HomePage\HomeasdsadPage.module.scss.mjs doesn't exist
.web.js
Field 'browser' doesn't contain a valid alias configuration
C:\Users............-frontend\src\Components\HomePage\HomeasdsadPage.module.scss.web.js doesn't exist
.js
Field 'browser' doesn't contain a valid alias configuration
C:\Users......-frontend\src\Components\HomePage\HomeasdsadPage.module.scss.js doesn't exist
.web.ts
Field 'browser' doesn't contain a valid alias configuration
C:\Users........-frontend\src\Components\HomePage\HomeasdsadPage.module.scss.web.ts doesn't exist
.ts
Field 'browser' doesn't contain a valid alias configuration
C:\Users..........-frontend\src\Components\HomePage\HomeasdsadPage.module.scss.ts doesn't exist
.web.tsx
Field 'browser' doesn't contain a valid alias configuration
C:\Users.......-frontend\src\Components\HomePage\HomeasdsadPage.module.scss.web.tsx doesn't exist
.tsx
Field 'browser' doesn't contain a valid alias configuration
C:\Users.................-frontend\src\Components\HomePage\HomeasdsadPage.module.scss.tsx doesn't exist
.json
Field 'browser' doesn't contain a valid alias configuration
C:\Users.............-frontend\src\Components\HomePage\HomeasdsadPage.module.scss.json doesn't exist      .web.jsx
Field 'browser' doesn't contain a valid alias configuration
C:\Users.........-frontend\src\Components\HomePage\HomeasdsadPage.module.scss.web.jsx doesn't exist
.jsx
Field 'browser' doesn't contain a valid alias configuration
C:\Users.............-frontend\src\Components\HomePage\HomeasdsadPage.module.scss.jsx doesn't exist
as directory
C:\Users..........-frontend\src\Components\HomePage\HomeasdsadPage.module.scss doesn't exist
@ ./src/App.js 12:0-58 24:21-29
@ ./src/index.tsx 10:0-24 27:37-40

when i used to get only the first two lines and that was all i needed
why was it even changed??
Describe the solution you'd like

either don't make them the default, orr add an option to hide them with a paremeter to npm start because they are useless to our team for everyday use
Describe alternatives you've considered

(Write your answer here.)
Additional context

(Write your answer here.)

already being discussed here - https://github.com/facebook/create-react-app/issues/11871


[BUG] SLCs don't work (either not reaching chain or are not being attested)

some chains for which we have no data yet (BNB, OP, Polygon, Blast, Base)
some chains for which SLCs reach the chain, but we are seeing an attestation failure in the logs
(Arb)

{"level":"warn","module":"server","tx_hash":"0x85670a939cb9ee32430b41f533bf8ad5492b6c73fd46cee886e6ca8cb6bc4332","valset_id":1352,"time":"2024-07-15T17:19:15Z","message":"SubmitLogicCall VerifyAgainstTX failed"}

ETH
Jul 15 18:20:20 mainnet-validator palomad[3224627]: {"level":"warn","module":"server","tx_hash":"0x6fc82c217aff38732d8c068b2d1a1e1c413b76c82356361f00b8da46ba636b1c","valset_id":1365,"time":"2024-07-15T18:20:20Z","message":"SubmitLogicCall VerifyAgainstTX failed"}

some chains for which SLCs don't reach chain since valset id is reported by pigeon as 0 instead of the actual valset id. This also means that subsequent valset updates don't go through (Gnosis)

{"level":"warn","module":"server","tx_hash":"0x9ebd627c2b37523c9615325638734eeb8178ba8b4b6c210e0a7ca66bcbe50f0b","valset_id":0,"updated_valset_id":1352,"time":"2024-07-15T17:05:14Z","message":"UpdateValset VerifyAgainstTX failed"}

Note:
Gnosis bot script (offramp-eure-bot on bots-server) was stopped, once restarted it will add more logic calls to the queue

Note that #2 is not actually a blocker for the bots to be working at the moment

From the latest information this most likely all comes down to an attestation issue. Unclear yet why attestation sometimes fails

The issue happens when:

Some pigeons sign the message in the queue and a pigeon picks it up
While this pigeon is relaying the message, some more pigeons sign it
When we try to attest it, the SignData in paloma doesn't match the SignData of the original message, since more pigeons signed it since!

@maharifu will you please confirm that this issue covers the failure of Gnosis relays to complete as we saw here: https://gnosisscan.io/address/0x5d5dab5c8f2b182ca96b3201d50e3b610c3250f1#internaltx
as per @verabehr
The last valset is not yet attested, so the next one can't go through. From the root cause analysis here, it seems that this doesn't always happen, so we got lucky the last two times and probably will again.

for completeness: I was referring to the root cause analysis in the comment of this ticket https://github.com/VolumeFi/paloma/issues/1875#issuecomment-2231232481

@taariq Yes, it should be the same issue. UpdateValset messages are not successfully attested, so we use the wrong valset on SLCs and they don't go through. The PR applies for both SLC and UpdateValset messages.


reuse runtimeAPIClient.buffer

Issue #, if available:
runtimeAPIClient.buffer is defined on here https://github.com/aws/aws-lambda-go/blob/55dc88be4cdfaaf4bb4d9f65debd8d4310e0a83c/lambda/runtime_api_client.go#L32
but it is never used.
Description of changes:
runtimeAPIClient.next() is never called concurrently, so we can reuse runtimeAPIClient.buffer.
It avoids extra re-allocation.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #325 into master will decrease coverage by 1.82%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #325      +/-   ##
==========================================
- Coverage   74.46%   72.63%   -1.83%
==========================================
  Files          18       18
  Lines         834      720     -114
==========================================
- Hits          621      523      -98
+ Misses        150      134      -16
  Partials       63       63

Impacted Files
Coverage Δ

lambda/runtime_api_client.go
75.00% <100.00%> (-2.78%)
:arrow_down:

lambda/rpc.go
55.55% <0.00%> (-8.09%)
:arrow_down:

events/epoch_time.go
77.77% <0.00%> (-4.05%)
:arrow_down:

events/cloudwatch_logs.go
60.00% <0.00%> (-3.64%)
:arrow_down:

lambda/invoke_loop.go
72.00% <0.00%> (-2.55%)
:arrow_down:

events/attributevalue.go
80.32% <0.00%> (-2.44%)
:arrow_down:

cmd/build-lambda-zip/main.go
33.82% <0.00%> (-2.29%)
:arrow_down:

lambda/panic.go
87.87% <0.00%> (-2.13%)
:arrow_down:

events/duration.go
42.85% <0.00%> (-1.59%)
:arrow_down:

... and 9 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 55dc88b...8d9388c. Read the comment docs.

Nice catch, thanks! This was the original intent, but I think I missed it during an internal refactor :)


Fix build with --enable-xlib=false

Since Jan 2022 Swiftshader no longer supports xlib (it suggests using xcb instead)

I believe our examples still use xlib to display a window on Linux.

They should still work, but prior to this slang just didn't compile with enable-xlib=false


Esri-leaflet sample code link broken

Use the code in http://esri.github.io/esri-leaflet/examples/renderers-plugin.html as a jumping off point.

@jgravois

not sure what you're talking about.  that link loads fine for me.

Done.  https://github.com/Esri/geodev-hackerlabs/commit/603d320029134d49ef71011dc13ab3e0df9938d4


Make modulesPath work again in NixOS modules

I think the recent modules change broke modulesPath. A remedy I've seen applied in a couple of places is to use <nixpkgs> paths, but those don't necessarily behave the same way in all contexts, so it would be nice to get the original functionality back. I still see evidence of modulesPath in the current code, so it seems like there might be some intent to support it, but the current implementation is somehow broken.
cc @nbp @shlevy

@edolstra , this was why I restored it.  Because we have no good substitute for it at the moment, what do you think?

If you can bring it back in the 15.06 timeframe, that'd simplify some of the EC2 work I want to get into it, but otherwise I can probably work around it.

#8188 is a more concrete version of proposal, so I'm closing this in favor of that.


chore(github): run update-book-cli in the lint workflow

implement and close #8333

This doesn't do anything right now, it should commit and push to the current PR?

This doesn't do anything right now, it should commit and push to the current PR?

yeah, or check that there is no diff like the fmt --check workflow

Ok lets do that, git diff with some flags should do it

Now it's working, after running the make update-book-cli, if the git workspace has any diff, the git diff will print the diff into the screen, eg:
https://github.com/paradigmxyz/reth/actions/runs/9207887154/job/25328904977?pr=8335
Run git diff --exit-code
diff --git a/book/cli/reth/db/checksum.md b/book/cli/reth/db/checksum.md
index a8147b0..66f0a86 100644
--- a/book/cli/reth/db/checksum.md
+++ b/book/cli/reth/db/checksum.md
@@ -22,6 +22,[9](https://github.com/paradigmxyz/reth/actions/runs/9207887154/job/25328904977?pr=8335#step:6:10) @@ Options:

           [default: default]

+      --start-key <START_KEY>
+          The start of the range to checksum
+
       --chain <CHAIN_OR_PATH>
           The chain this node is running.
           Possible values are either a built-in chain or the path to a chain specification file.
@@ -31,6 +34,[12](https://github.com/paradigmxyz/reth/actions/runs/9207887154/job/25328904977?pr=8335#step:6:13) @@ Options:

@DaniPopes @Rjected This PR is ready, please take a look, thanks.


Provide a command to reset the completed steps of a walkthrough

Currently, VSCode allows users to reset all walkthroughs using the command workbench.action.resetGettingStartedProgress. However, there is no means to execute a command from an extension to reset a specific walkthrough.
Feature Request:
Please provide a command that allows resetting the completed steps of a specific walkthrough. This would enable extensions to programmatically reset the progress of a particular walkthrough, enhancing the user experience by providing more granular control over walkthrough progress.
Proposed Implementation:
Introduce a new command, such as workbench.action.resetSpecificWalkthroughProgress, which can be executed with the necessary parameters to reset a specific walkthrough.
Example Usage:
vscode.commands.executeCommand(
  "workbench.action.resetSpecificWalkthroughProgress",
  "extensionPublisher.extensionName.#WalkThroughId"
);

Use Case:
This feature would be particularly useful for extension developers who wish to provide users with the ability to restart specific walkthroughs without affecting the progress of other walkthroughs. It would also be beneficial in scenarios where a walkthrough needs to be revisited or reset due to updates or changes in the extension.
Thank you for considering this feature request.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_smiling_face: This feature request received a sufficient number of community upvotes and we moved it to our backlog. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Default File Icon Association Only Works for "file"

[x] I'm sure this issue is not a duplicate.

The vsicons.associations.fileDefault.file setting only works with the "icon": "file". I've tried several different icons, with a variety of different formats ("svg", "FileFormat.svg", etc.) and everything other than file does not work. I've attached two screenshots demonstrating the issue.

I apologize if this is a duplicate question or is covered in the wiki. I've been searching through the repository and have not yet found a solution. I'm not sure if the issue is a bug, or is just a syntax error.
Any help is greatly appreciated.
Thank you in advance. 👍

I looked into adding extends functionality to the default files and folders but unfortunately, this is more complex than it looks and I'm afraid that it may destabilize the extension.
Still, though the way to achieve what the OP wants is to create the vsicons-custom-icons folder as described in Custom Icons and copy inside it the preferred icon and remain it accordingly (i.e.default_<value_of_icon_property>.<icon_file_extension>).


Fix docs.rs build error

https://docs.rs/crate/wxrust-base/0.0.1-alpha
Consult their doc:

https://docs.rs/about/builds

Installed packages to build (WIP)

git
cargo (rustup snap package didn't work with 20.04)

These packages seems already installed...

libwxbase3.0-dev
libwxgtk3.0-gtk3-dev

kenz@ubuntu2004:~/src/wxRust2$ wx-config

Command 'wx-config' not found, but can be installed with:

sudo apt install libwxbase3.0-dev      # version 3.0.4+dfsg-15build1, or
sudo apt install libwxgtk3.0-gtk3-dev  # version 3.0.4+dfsg-15build1

apt install part of dockerfile of crates-build-env
# Install the packages contained in `packages.txt`
COPY packages.txt /opt/crates-build-env/packages.txt
RUN apt-get update && \
    cat /opt/crates-build-env/packages.txt | DEBIAN_FRONTEND=noninteractive xargs apt-get install -y && \
    rm -rf /var/lib/apt/lists/*

Installed packages to build (WIP)

git
cargo
libwxgtk3.0-gtk3-dev

error occurred: Failed to find tool. Is `c++` installed?

These packages seem already installed...

https://github.com/rust-lang/crates-build-env/blob/master/linux/packages.txt#L902
didn't include -dev package. I'll send PR to add this.

I sent PR: https://github.com/rust-lang/crates-build-env/pull/108

Fixed!


Fix image links and some grammatical corrections

Thank you for making these changes @jcer 👍

Could you please add a link to the translation in https://github.com/Roshanjossey/first-contributions/blob/master/Translations.md and Line no. 14 in Readme?
Also can we reach out to you in the future for any reviews on changes to this file?

Absolutely

Hi, your always welcome to reach out for a review. Kind regards, Johan

Awesome @jcer 👍 🖖


Branch tracking detection fails for different system language

svn2git parses the git error messages to detect whether branch tracking is enabled, see https://github.com/nirvdrum/svn2git/blob/master/lib/svn2git/migration.rb#L360. This fails if the messages are localized. A workaround is to use LANG=C before calling the script. This was already mentioned in #132

I've logged this issue in my new implementation. Hopefully I can fix it other than depending on a workaround.

OK I've resolved this issue within a new release in svn2git.net. In case anyone wants it, just download the latest release from https://github.com/mazong1123/svn2gitnet/releases


"Surround with" in TS/JS snippets

Fixes #138564.

Thanks! Seems like a great addition. Let's continue discussing how to make these more discoverable too


add access logs for passthrough listener

We would like to have logs for requests going through the passthrough listener/cluster. This will help us transition onto the mesh and identify traffic that is not yet fully onboarded.
Also ref: https://discuss.istio.io/t/telemetry-metrics-logging-on-passthroughcluster/2005
NOTE: I still need to look at the tests here.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: dwradcliffe
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: ostromart
If they are not already assigned, you can assign the PR to them by writing /assign @ostromart in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/pkg/networking/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@dwradcliffe: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
104d442ea5891eab7dae7ba8ffacf1d781a8608c
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: dwradcliffe, rshriram
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/pkg/networking/OWNERS [rshriram]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@dwradcliffe: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
104d442ea5891eab7dae7ba8ffacf1d781a8608c
link
/test istio-integ-k8s-tests

prow/istio-unit-tests.sh
104d442ea5891eab7dae7ba8ffacf1d781a8608c
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

/retest

@dwradcliffe: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
104d442ea5891eab7dae7ba8ffacf1d781a8608c
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Do we need some sort of integration / e2e test that validates we are seeing a desired behavior here?  I don't see any tests in this PR.


Plural expression for ie

Adding support for the Interlingue plural expression.

Please follow best git commit message practices — “Interlingue plural is not a descriptive commit message". Please don’t add duplicate commits that don’t even do what their subject says.
Finally, please also observe that the file you modified says it was generated by scripts/extract-plural-forms.py — in fact, it says so on a line you modified. You have to make changes to that script, not the generated file, for them to have a lasting effect.

In the future please amend your PRs (both git and GitHub has ample documentation) instead of making a gigantic mess by closing PRs with no explanation and replacing them with the same thing anew, thus loosing any history.
Replaced by #356, apparently.


Change/pack the .search css selector

The class is very common and can be used outside the component.

Not sure if I understand your question/request. Could you elaborate a bit more?

There is a .search css style which is global and once the component is used
in an application it can affect a component with the same class (the name
search is very common). My suggestion is to rename it or to use more
specific selector.
On Thu, Nov 21, 2019 at 12:53 PM Huang-Ming Chang<EMAIL_ADDRESS>wrote:

Not sure if I understand your question/request. Could you elaborate a bit
more?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/iannbing/react-simple-tree-menu/issues/113?email_source=notifications&email_token=AA5OUKZYIF7QN2YRPXCK3ZTQUZSA5A5CNFSM4JPTFW4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEZZ2PI#issuecomment-557030717,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA5OUK5FAE2573O4FGMSXHTQUZSA5ANCNFSM4JPTFW4A
.

Ah, you're right. Should've added a prefix for the css classes. Thanks!
I'm not sure when I'll have time to fix this. Feel free to send your PR 😃.


Add useEndpointAttemptedMessages hook for react-svix

Feature Request
Motivation
Display a list of attempted messages for selected endpoint

Proposal
Implement useEndpointAttemptedMessages hook that should call a List of Attempted Messages under the hood with filters.
Caveats
The name of this new hook (useEndpointAttemptedMessages) will be confusing with the existing one useEndpointMessageAttempts. It would be great to make these names more obvious based on what they return. For example, we can name it useEndpointDestinationMessages
Alternatives
Call GET /api/v1/app/:app_id/endpoint/:endpoint_id/msg directly without react-svix

Hey! Sorry for the super late response, we missed this one. We added a [useAttemptedMessages](https://www.npmjs.com/package/svix-react#useattemptedmessages) hook in the latest version of svix-react.


The simplest of examples, as usual.

What if I need to use a connection string from the config file? like, you know,
public ApplicationDbContext CreateDbContext(string[] args)
{
var builder = new DbContextOptionsBuilder<ApplicationDbContext>();
builder.UseSqlServer( --> Configuration.GetConnectionString("DefaultConnection") <-- );

return new ApplicationDbContext(builder.Options);

}

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

ID: 2fa64390-1232-8dd3-f170-7d5c68e29ad9
Version Independent ID: 245bbba8-3102-4a66-15eb-650e05eb9e62
Content: Design-time DbContext Creation - EF Core
Content Source: entity-framework/core/miscellaneous/cli/dbcontext-creation.md
Product: entity-framework
GitHub Login: @bricelam
Microsoft Alias: bricelam

Hey @bdaniel7!
You could add the code for acquiring a connection string inside the CreateDbContext method.
Have a look into issue https://github.com/aspnet/EntityFrameworkCore/issues/8332 where others were facing the same problem and provided their solution.
Cheers,
René

Helpful, but why not update the docs HERE with a real world example.

I use json.net and File.ReadAllText to load config in appsettings.json


Babel-loader error when using custom location for pages directory

I'm having a small issue with using a custom location for the pages directory. It appears as though the babel-loader is no longer able to parse the components referenced in each page.
Any thoughts? Or am I going to have to implement my own babel-loader in the next.config.js?
https://github.com/stephencorwin/nextjs-hello-world
├── /app
|  ├── /components
|  |  ├── /Footer
|  |  |  ├── Footer.js
|  |  |  ├── index.js
|  |  |  └── styles.js
|  |  ├── /Greeting
|  |  |  ├── Greeting.js
|  |  |  ├── index.js
|  |  |  └── styles.js
|  |  └── /Header
|  |     ├── Header.js
|  |     ├── index.js
|  |     └── styles.js
|  └── /templates
|     └── /Home
|        ├── Home.js
|        ├── index.js
|        └── styles.js
└── /server
   ├── index.js
   ├── /middleware
   |  └── index.js
   └── /pages
      ├── index.js
      └── _document.js

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior
Components imported from the /apps/components/ folder should be parsed using the Next.js implementation of the babel-loader.
Current Behavior
error  in ./app/components/Header/Header.js

Module parse failed: D:\stephencorwin\projects\nextjs-hello-world\node_modules\react-hot-loader\webpack.js!D:\stephencorwin\projects\nextjs-hello-world\app\components\Header\Header.js Unexpected token (1:34)

You may need an appropriate loader to handle this file type.
| export default ({ className }) =>
Header
;

Steps to Reproduce (for bugs)

Clone Repository
npm -i
npm run dev

Context

Ideally, I would like to have the pages directory be segregated from the other components.
It currently works if I colocated the pages directory next to the templates and components that are also being loaded. View the Pull Request below for details.
https://github.com/stephencorwin/nextjs-hello-world/pull/1
├── /app
|  ├── /components
|  |  ├── /Footer
|  |  |  ├── Footer.js
|  |  |  ├── index.js
|  |  |  └── styles.js
|  |  ├── /Greeting
|  |  |  ├── Greeting.js
|  |  |  ├── index.js
|  |  |  └── styles.js
|  |  └── /Header
|  |     ├── Header.js
|  |     ├── index.js
|  |     └── styles.js
|  ├── /pages
|  |  ├── index.js
|  |  └── _document.js
|  └── /templates
|     └── /Home
|        ├── Home.js
|        ├── index.js
|        └── styles.js
└── /server
   ├── index.js
   └── /middleware
      └── index.js

Your Environment

Tech
Version

next
4.2.3

node
8.7.0

OS
Windows 10

Ultimately, it seems as though this is a limitation of how the babel loader is set up relative to the pages directory. For example, I cannot have the tree:
├── /app
|  ├── /components
|  └── /pages
└── /lib

and expect to be able to parse anything from /lib.
It would instead HAVE to be:
├── /app
|  ├── /components
|  └── /pages
|  └── /lib


M2M edge insertion logic ignores upsert conflict resolution options

Hello! First of all, thanks for your work on this package.
I have the following schema:
// user.go

func (User) Fields() []ent.Field {
	return []ent.Field{
		field.String("name").Unique(),
	}
}

func (User) Edges() []ent.Edge {
	return []ent.Edge{
		edge.To("books", Book.Type),
	}
}

// book.go

func (Book) Fields() []ent.Field {
	return []ent.Field{
		field.String("author"),
		field.String("title"),
	}
}

func (Book) Edges() []ent.Edge {
	return []ent.Edge{
		edge.From("user", User.Type).Ref("books"),
	}
}

func (Book) Indexes() []ent.Index {
	return []ent.Index{
		index.Fields("author", "title").Unique(),
	}
}

And the following code:
// main.go

func main() {
	ctx := context.Background()

	client, err := ent.Open("postgres", "...")
	if err != nil {
		panic(err)
	}

	if err := client.Schema.Create(ctx); err != nil {
		panic(err)
	}

	userID, err := client.User.Create().
		SetName("John Doe").
		OnConflictColumns(user.FieldName).Ignore().
		ID(ctx)
	if err != nil {
		panic(err)
	}

	err = client.Book.Create().
		AddUserIDs(userID).
		SetAuthor("Arthur Conan Doyle").
		SetTitle("The Valley of Fear").
		OnConflictColumns(book.FieldAuthor, book.FieldTitle).Ignore().
		Exec(ctx)
	if err != nil {
		panic(err)
	}
}

The first execution succeeds, and creates the following entries:
postgres=# select * from users;
 id |   name
----+----------
  1 | John Doe
(1 row)

postgres=# select * from books;
 id |       author       |       title
----+--------------------+--------------------
  1 | Arthur Conan Doyle | The Valley of Fear
(1 row)

However, the second execution fails with ent.ConstraintError error (add m2m edge for table user_books: pq: duplicate key value violates unique constraint "user_books_pkey"), despite the fact that upsert was requested via OnConflictColumns().
Looking deeper, it seems that M2M edge insertion logic completely ignores the existence of upsert:
https://github.com/ent/ent/blob/51b63c17c0185ff9f75c502e030ed38f3c025c10/dialect/sql/sqlgraph/graph.go#L1111-L1144
It would be nice if M2M edge conflicts would resolve automatically depending on the upsert options requested, or via a separate option.

It would be nice if M2M edge conflicts would resolve automatically depending on the upsert options requested, or via a separate option.

Found https://github.com/ent/ent/issues/537#issuecomment-641894671 that proposes to add Add<E>IfNotExist() method to solve this:

Maybe, we can add something like Add<E>IfNotExist() for edges, but I need to think about it - I'll create a separate issue for this if I'll think it's necessary.

@edigaryev agree with you. I think Upsert philosophy covered your mention. Hope that someone will add that functionality

@a8m maybe some recommendation?

Closed with https://github.com/ent/ent/pull/3108. Please, update to master to get this functionality. Thanks 🙏🏻


Support <<~ HereDocs

Description of issue
Syntax highlighting doesn't understand the <<~HEREDOC; notation using the tilde.  This feature was added in Perl 5.26.
Screenshot of the issue.

Sample code:
print <<"ENDHEREDOC";
This is highlighted correctly. It's treated as a literal as expected.
ENDHEREDOC

print <<~"ENDHEREDOC";
    Note the tilde added above.
    This isn't shown correctly. The tick appears to start a literal string.
    ENDHEREDOC

Link to vimrc
n/a, it's empty
Output of vim --version
VIM - Vi IMproved 8.1 (2018 May 18, compiled Mar  4 2019 16:22:18)
Included patches: 1-994
Modified by<EMAIL_ADDRESS>Compiled by<EMAIL_ADDRESS>Huge version without GUI.   Which version of vim-perl are you using?  This was implemented in #255, but maybe we missed a corner case.

This is on a current Fedora 29 machine.
$ rpm -qf /usr/share/vim/vim81/syntax/perl.vim
vim-common-8.1.994-1.fc29.x86_64

The header of perl.vim looks like so:
 " Vim syntax file
" Language:      Perl 5
" Maintainer:    vim-perl<EMAIL_ADDRESS>" Homepage:      http://github.com/vim-perl/vim-perl/tree/master
" Bugs/requests: http://github.com/vim-perl/vim-perl/issues
" Last Change:   2017-09-12
" Contributors:  Andy Lester<EMAIL_ADDRESS>"                Hinrik Örn Sigurðsson<EMAIL_ADDRESS>"                Lukas Mai <l.mai.web.de>
"                Nick Hibma<EMAIL_ADDRESS>"                Sonia Heimann<EMAIL_ADDRESS>"                Rob Hoelz<EMAIL_ADDRESS>"                and many others.
"
" Please download the most recent version first, before mailing
" any comments.
"

Clearly I didn't read that last line before submitting this. Also my issue-search foo is lacking.
Disregard.

No problem.  Glad you have the feature now.


[init][draft] trino lance connector boiler plate code

first version runnable closing and move to upstream


exfiltrate-sensitive-data: add new ruleid

Closes #263.

Looks good, any chance you can rebase and we can merge? :)

Looks good, any chance you can rebase and we can merge? :)

Done


Fixed link

Fixed the link pointing the "Capacity management and sizing overview for SharePoint Server 2013" document: the actual one was pointing to the SharePoint 2010 version of the document.

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.