feat: add an output for the tag suffix

This allows us to avoid using the DOCKER_REPO secret in the output.
Supersedes #58

Does this mean we should deprecate docker-tag since it will always be masked?

We don't need to. The masking is only important if you're trying to send the value between jobs; it can still be shared between steps of a single job.


log to custom installation location of openlog.nsf

cannot log to different installation than default data/openlog.nsf ?

Not at the moment. In the latest release of the plugin version (4.01), logging to OpenLog is disabled. I'm working on a release in which that function is enabled again, including being able to set the log database using the xsp.properties.


Any way to force a cetain browser engine version like 27.1.1 or 26.8.0 ?

whenever an update happens it screws up the engine version and downloads the latest one but my Indian IP is throttled by your Server whenever i download i get super low speeds and have to use vpn to fix it , when i try multi threaded applications with same engine directory this causes a huge issue any way to set the engine  version ?

Entire Engine is screwed by the latest update, it auto updates itself and fucks up everything and there's no rolling back, how do i force it to use only version 26.8.0 which is my local latest version that works

@singhaditya4 Please stop spamming the same type of messages in several tickets. You cannot specify a specific version of the engine to work with - at the next update, I will additionally indicate the transition to a new one, so you can stay on the old one simply without updating the versions.
As for the error from the screenshot, the problem has already been described many times, also please refer to:
https://github.com/CheshireCaat/puppeteer-with-fingerprints/issues/61
https://github.com/CheshireCaat/puppeteer-with-fingerprints/issues/71
https://github.com/CheshireCaat/puppeteer-with-fingerprints/issues/6


Installation requirements (GCC, zlib and libpng) only for development?

Robotgo version (or commit ref): irrelevant
Go version: 1.8
Gcc version: ?
Operating system and bit: Mac, Windows 64, Linux
Can you reproduce the bug at Examples:

[ ] Yes (provide example code)
[ ] No
[X] Not relevant

Description
Are the installation requirements (GCC, zlib and libpng) only for development or are they required to be  pre-installed by end user? In other words, once I build a Go executable for (Windows, Linux and Mac) , is that all the user needs or do they need to pre-install the dependencies? It might be worthwhile to update the home page documentation for newbies, to avoid misunderstandings.

Thank you for your suggestion, I will think about it or add it in future.

What about my question though? Are the installation requirements (GCC, zlib and libpng) only for development or are they required to be pre-installed by end user?

Because it is dynamic link, the need for users to install in advance, perhaps remove it will be better or improved. If you do not need bitmap, you can switch to the robot branch

But in Windows is statically linked, you do not need pre-installed. Just on the mac, you need it.

Thanks. If it's dynamically linked on Mac, is it possible to include the libraries in the executable path for mac without installation? I am trying to make it easy to install for the user (simple installation).

The loading path is  "/usr/local/opt/libpng/lib/...", let me think how to make it better. Now, I want to fix findbitmap, maybe in the near future will change it.


Resize Columns doesn't work from v7.0.0-beta.20

Using v7?
Thanks for using the beta version of React Table v7! We're very excited about it.
Describe the bug
From version v7.0.0-beta.20, useResizeColumns hook doesn't work anymore.
To Reproduce
Steps to reproduce the behavior:

Go to https://codesandbox.io/s/github/tannerlinsley/react-table/tree/master/examples/column-resizing
See that resizing columns feature doesn't work
Change in package.json version to "react-table": "7.0.0-beta.19",
Resizing works as expected

Expected behavior
Resize of columns doesn't work
Codesandbox!
Use a new react-table codesandbox to reproduce the issue.
SANDBOX
Desktop (please complete the following information):

OS: [MacOS]
Browser [chrome]
Version [78]

@enco3ap you should update to 7.0.0-beta.23 it already fixed there

Yep, it's fixed in versions after beta.23. Make sure codesandbox isn't caching the old version.


Discard a SecurityError raised when navigating on a local file, making navigation work again.

Navigation doesn't work when the slides are opened "locally", as opposed to "served by a web server".
This issue has been observed in Chrome (Windows and MacOSX).
The reason is that window.history.replaceState() raises a SecurityError. Catching it fixes the navigation.

+1


Update typo in git config command

The doc reads "To sign all commits by default in local repository on your computer, run git config --global ...". So propose this change to correct the typo to --local

Why:
Closes #15287

What's being changed:

Check off the following:

[x] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

@sshesh Thanks so much for opening a PR! I'll get this triaged for review :zap:

👋🏻 @sshesh, the beginning of that sentence is:

To sign all commits by default in any local repository on your computer

The --global flag does just that, so we'll leave this as is.
Let me know if you have any questions!


Allow Symfony 3 to be installed

Todo

[x] Squash commits
[x] Make tests pass
[x] Revert commenting other tests

Tests all still pass :tada:

Thanks


Use internal nuget feed for develop branch

Use internal nuget feed for develop branch
Also updated FrameworkUDK to use the internal version.
WIP or Coming later:
Use version.details.xml to bring in the FrameworkUDK and flow the versions down to the dependencies so we don't have version numbers repeated in different projects.

/azp run

/azp run

Moving forward, we should figure out a better way to update the well-known package dependencies.

Definitely. This is my next change.


Meetings SidePanel Build Pipeline Fixes

No Need TO Merge, This PR Is raised For Build Pipeline Fixes but not needed now.


Issue/5399 add icon

fix gravitee-io/issues#5399

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


Error While running py.test

ImportError while importing test module '/root/Streaming/src/salesforce-streaming-client/tests/test_it.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
Streaming/src/salesforce-streaming-client/tests/test_it.py:33: in
from salesforce_streaming_client import SalesforceStreamingClient
E   ImportError: No module named salesforce_streaming_client
can you please tell me, where is problem ?

The error message says that salesforce_streaming_client cannot be found (it's not in your PYTHONPATH). This is probably happening because you cloned the repository (as opposed to pip install) and did not add the salesforce-streaming-client/src directory to the path.
I would recommend using virtualenvwrapper that has the very convenient add2virtualenv command. Then you can go to the salesforce-streaming-client/src directory, run add2virtualenv . which will add the src directory to the Python path. Then try running the tests again.


[BUG] Missing indexed attribute in the color object.

🐛 Bug Report
There is an indexed attribute in the color object. When you set a cell's background color with color in the default color palette, Excel will probably add an indexed attribute to the fill.color object, instead of an argb attribute. It seems that this situation is quite common, for colors in the default color palette are used oftenly.
I did a little bit research on this attribute, and found a document that explained what these indexed's value represent:
https://openpyxl.readthedocs.io/en/stable/styles.html. Just open the webpage and Ctrl+F search for indexed, and you will get the result.
I think it is better to have an converting function to convert the indexed and theme attribute to an arbg color code, for when handling .xlsx document, the color code is what I really want to get.

Incase someone needs the colors mapped out:
const indexedColors = [
  '000000',
  'FFFFFF',
  'FF0000',
  '00FF00',
  '0000FF',
  'FFFF00',
  'FF00FF',
  '00FFFF',
  '000000',
  'FFFFFF',
  'FF0000',
  '00FF00',
  '0000FF',
  'FFFF00',
  'FF00FF',
  '00FFFF',
  '800000',
  '008000',
  '000080',
  '808000',
  '800080',
  '008080',
  'C0C0C0',
  '808080',
  '9999FF',
  '993366',
  'FFFFCC',
  'CCFFFF',
  '660066',
  'FF8080',
  '0066CC',
  'CCCCFF',
  '000080',
  'FF00FF',
  'FFFF00',
  '00FFFF',
  '800080',
  '800000',
  '008080',
  '0000FF',
  '00CCFF',
  'CCFFFF',
  'CCFFCC',
  'FFFF99',
  '99CCFF',
  'FF99CC',
  'CC99FF',
  'FFCC99',
  '3366FF',
  '33CCCC',
  '99CC00',
  'FFCC00',
  'FF9900',
  'FF6600',
  '666699',
  '969696',
  '003366',
  '339966',
  '003300',
  '333300',
  '993300',
  '993366',
  '333399',
  '333333',
];

For tinting, you can probably refer to this stackoverflow answer:
https://stackoverflow.com/questions/5560248/programmatically-lighten-or-darken-a-hex-color-or-rgb-and-blend-colors


application packages treated as lib by pub

This issue was originally filed by<EMAIL_ADDRESS>
It seems that application packages(AP) are treated as lib packages by pub.
http://stackoverflow.com/questions/17553846/if-and-how-to-publish-application-packages-to-pub
1.AP needs a lib folder in order to be published.
2.The installing tab on the pub package site should be adaptable by the author. So instead of adding the AP to the dependencies and importing it, we should see eg. download the package to your drive and run ###.dart.
currently I use the readme which might confuse people since they might think they have to do both installations.
additionally it would be awesome
3.Since the AP is not listed inside a pubspec file, there is no update functionality for the package (except if the user works with the bleeding edge git repo or would it work with path too?)
Thank you

This issue has been moved to dart-lang/pub#594.


Add performance test page

I want to understand if there are any performance tweaks we can do to
make react-waypoint use fewer browser resources. Before I do that, it
helps having a page rendered with many waypoints, to dive into things
like fps, repaints, etc using the Chrome dev tools.
See #177 for more context.

This looks good to merge to me.


Update examples

It would be nice to update examples with the latest libraries, like Retrofit 2.0, nucleos 2.0.5...

Hi, updated examples are in the nucleus project folder: https://github.com/konmik/nucleus/tree/master/nucleus-example


Evolution request - device auto decomissining

On a router the devaddr resources are costly (about $1 per device, corresponding to 100.000DCs) considering with can have a overasignement of 100 devices per DevAddr.
The problem is that many devices are not removed from the console as it is a non cost for users. This pollute the XOR filters with non active devices. and force to increase the DevAddr pool for a reduced number of active devices. The economical impact is large.
The evolution request is to automatically decommission (remove the device from the XOR filter with a button to re-insert it later) after a certain time of inactivity (no join request, no uplink), this duration would be in a setting the router owner can select. I think the right unit is in days and the idea is to have a default inactivity period of 60 days.

Until HIP 59 this was too costly to do. We have now plans to rewrite the XOR Filter handler so that it can support removal.
I will link this ticket as soon as I have more details.


only restart the pods when namespaceMembers are changed

Currently restarting the pods after creation or deletion of the role and rolebinding.


Rename /kms path to something else

From https://github.com/trustbloc/edge-service/pull/126#discussion_r395327464

@troyronda @rolsonquadras the bike shed is open...


training efficiency

hello,thx for your work.i have a problem when run TPT48 task.It takes me 20 seconds to run a epoch and more than 18s are wasted in generate_beta(reconstruct_u_graph),Is that a normal speed? Is there a way to accelerate it?

Hi HaoWenJohn,
Thanks for using our code! This is normal because "generate_beta(reconstruct_u_graph)" will calculate the pairwise Wasserstein distance (W distance) between different domains. The time complexity would be O(N^2 * T(W distance)), where N is the number of domains. We suggest the following ways to accelerate the training:

Lower the usage frequency of "generate_beta". Currently, we generate the beta once per epoch. You can modify the code to generate it once per 2 epochs or once per 3 epochs.
Find ways to accelerate the W distance calculation. Our implementation uses the "sinkhorn loss" from " GeomLoss" library. We are working towards a more time-efficient solution, and you are welcome to try your hand at it as well.
Create your own "generate_beta" function. Essentially, "generate_beta" is an aggregation function that maps local domain indices "u" to global domain indices "beta". While we've attempted to use "average" (the most basic aggregation method) to obtain global domain indices, it does not produce the desired domain indices. You are free to design a more efficient "generate_beta" function to bypass the time-consuming W distance calculation.

Don't hesitate to reach out if you encounter any issues or have additonal questions!

Thank you for your reply and look forward to your future work.


rav4 hybrid 2019 port

this time i actually updated the readme. waiting on cabana logs

it could have been 1 commit if i were more careful... fixed those typos now

@wocsor do you have a route you can share? I've looked at dongle 76280b332ccbc474 and I've found a larger fingerprint

https://my.comma.ai/cabana/?route=0fb8905d1fa7dbdb|2019-07-14--17-26-57&max=27&url=https%3A%2F%2Fchffrprivate-vzn.azureedge.net%2Fchffrprivate3%2Fv2%2F0fb8905d1fa7dbdb%2F187b38b0e5e38c1bdc4cf42c073a773f_2019-07-14--17-26-57
sorry about all the little errors btw

no worries. Unfortunately this fingerprint conflicts with some of the the Camry Hybrid. It will need to wait until VIN query, which is coming soon.

ETA on that VIN query @rbiasini ???  :-)

funny story on that, over the past week a friend and I did a lot of under the hood work on openpilot and found that there is a working VIN query in car helpers, but if a valid VIN is written to params boardd will change the safety to nooutput. after commenting that out it then only writes the first message per bus in sendcan. we haven't gone any further but we suspect another blocker in boardd is the issue

:crying:

Assisted sideplugmike on Discord with getting this working on 2019 Rav4 Hybrid XSE.
Dongle ID 689f254b15632b32
Stock Drive: https://my.comma.ai/cabana/?route=689f254b15632b32|2019-08-20--09-38-35&max=11&url=https%3A%2F%2Fchffrprivate-vzn.azureedge.net%2Fchffrprivate3%2Fv2%2F689f254b15632b32%2Fa22611d366b256d72767078c4bb8a25a_2019-08-20--09-38-35
Camry Hybrid fingerprints had to be commented out to resolve conflict.
2019 Rav4 Hybrid XSE Fingerprint: 36: 8, 37: 8, 166: 8, 170: 8, 180: 8, 295: 8, 296: 8, 401: 8, 426: 6, 452: 8, 466: 8, 467: 8, 550: 8, 552: 4, 560: 7, 562: 6, 581: 5, 608: 8, 610: 8, 643: 7, 658: 8, 713: 8, 728: 8, 740: 5, 742: 8, 743: 8, 761: 8, 764: 8, 765: 8, 800: 8, 810: 2, 812: 8, 814: 8, 818: 8, 824: 8, 829: 2, 830: 7, 835: 8, 836: 8, 863: 8, 865: 8, 869: 7, 870: 7, 871: 2, 877: 8, 881: 8, 882: 8, 885: 8, 889: 8, 896: 8, 898: 8, 900: 6, 902: 6, 905: 8, 918: 8, 921: 8, 933: 8, 934: 8, 935: 8, 944: 8, 945: 8, 950: 8, 951: 8, 953: 8, 955: 8, 956: 8, 971: 7, 975: 5, 987: 8, 993: 8, 998: 5, 999: 7, 1000: 8, 1001: 8, 1002: 8, 1014: 8, 1017: 8, 1020: 8, 1041: 8, 1042: 8, 1044: 8, 1056: 8, 1057: 8, 1059: 1, 1063: 8, 1071: 8, 1076: 8, 1077: 8, 1082: 8, 1084: 8, 1085: 8, 1086: 8, 1114: 8, 1132: 8, 1161: 8, 1162: 8, 1163: 8, 1164: 8, 1165: 8, 1166: 8, 1167: 8, 1172: 8, 1228: 8, 1235: 8, 1237: 8, 1264: 8, 1279: 8, 1541: 8, 1552: 8, 1553: 8, 1556: 8, 1557: 8, 1568: 8, 1570: 8, 1571: 8, 1572: 8, 1592: 8, 1594: 8, 1595: 8, 1649: 8, 1745: 8, 1775: 8, 1779: 8, 1786: 8, 1787: 8, 1788: 8, 1789: 8, 1990: 8, 1998: 8

Could fingerprint conflict ports be upstreamed without the fingerprint for now so forks don't have to be maintained with each release?

@illumiN8i , yes, I'm fine merging in with no fingeprint. Btw, Rerunning the fingerprint conflict checker script now (selfdrive/test/test_fingerprints.py) does not show conflicts anymore. Not sure why it showed red flags before. Are you saying that the CarmyH fingerprint had to be commented out though to have OP working on sideplugmike's car?

@rbiasini Correct, sideplugmike's car didn't work until I commented out the CamryH fingerprints. @wocsor Can you rebase and comment out the RAV4H_TSS2 fingerprint?

@illumiN8i has another port which does not have conflicts.


[core]Rename method to prohibit jadx errors

link: https://drive.google.com/file/d/1EYP3KNNmSez3SNjFjX7Qe4kdl8kCmi4V/view?usp=sharing
As in the picture, jadx wrongly changes the method name to showVersionInfo, how to prohibit it from renaming incorrectly?

This is a feature to remove synthetic bridge methods by "merging" with wrapped method.
This can be disabled by Inline methods option (if checkbox missing, please use latest unstable build :sweat_smile:)
I also commit an additional rename reason: /* merged with bridge method [inline-methods] */
And if debug code comments are enabled, merged methods signatures will be shown:
    /* JADX DEBUG: Method merged with bridge method: showVersionInfo(Lcom/nomagic/b/ag;)Z */
    /* JADX DEBUG: Method merged with bridge method: showVersionInfo(Lcom/nomagic/rcpf/product/lic/ProductVersion;)Z */

Hope this makes it easier to understand why method was renamed and what option control it.
Related issues #1109 and #1170.


Render issue markdown correctly

Currently issue markdown isn't rendered correctly. For example, ### Header is shown instead of the styled header.

(sorry for low-quality screenshot, still working out some iOS 11 issues)

Implemented in #326

Fixed in #326 !


Support set of IDs in role_assignments

It would be nice if one could give the principals to give access for a given role as a set instead of a single ID.
role_assignments = {
+    admins = {
+      role_definition_id_or_name = "Key Vault Administrator"
+      principal_id               = data.azuread_groups.admin_groups.object_ids[0]
+    }
+  }

instead:
role_assignments = {
+    admins = {
+      role_definition_id_or_name = "Key Vault Administrator"
+      principal_ids               = data.azuread_groups.admin_groups.object_ids
+    }
+  }

single principal is just a special case (a set with one entry) - makes it easier to assign more

Hi,
Thanks for the suggestion.
I agree that this would be more efficient.
I think we will come across an issue whereby we have to construct the keys of the for_each loop.
We would need to combine the original map key with the principal_id.
In the case that the principal id was a string this would be ok. However, we often create the principal (e.g. Entra ID group) in the same apply and, as such, the principal id would not be known until after the apply. This would cause an error with the for_each loop.
It's very annoying but I think we must continue with the current approach.
Thoughts?

ok - just one of terraforms wonders - good to see good tf experience driving decisions here instead :-)


Control an accessory by URL

I'm new to the Homebridge solution, and am trying to find a way to control a number of devices from my Loxone kit.
Within Homebridge and the Config UI X I can see and control all of my devices, including the Loxone ones which pulled across fine, but I can't find a way of turning on or off a TV for example which can be done from within the Accessories tab fine.
Is there either a plugin that would allow me to do this, or a way of creating a URL that would carry out a specific function from the accessories tab ?
Thanks,

The Homebridge service allows you to do this directly.
Connect to the Homebridge port/ip via http, eg:
GET http://homebridge.local:54321/accessories

GET http://homebridge.local:54321/characteristics?aid=1&iid=2

PUT http://homebridge.local:54321/characteristics
Authorization: 453-23-23
{
  "characteristics": [
     {
       "aid": 1,
       "iid": 2,
       "value": 23
     }
  ]
}

See https://github.com/homebridge/homebridge/issues/2512#issuecomment-619303434


Error when trying to re-build examples

To make the error, follow the steps described here. Then you should get a message like:
no package named `rtest` found (required by `rtest-examples`)
location searched: file:///home/imperio/rust/cargo-issue/example
version required: = 0.9.11

Don't hesitate to ask if you have any question !

Closing as a dupe of #1145, as discussed there. Thanks for filing nonetheless!


subscriptions and stages need to seperated in to two kinds

Summary
when creating a stage a subscription is part of it, but in fact, a subscription is a different identity that needs to be its own kind.

so the stage can register to a subscription, making the flow much more visible and manageable.
Motivation
make the source of the repo/chart/image a subscription that stages can then be registered to, and then stages can also register to other upstream stages.
Proposal
freight then should be a stage registered to a subscription, a stage without a subscription, will be able only to consume freights, not generate them, this will assist with more agility in designing the pipeline into different scenarios.
stage should look like:
apiVersion: kargo.akuity.io/v1alpha1
kind: Stage
metadata:
  name: test
  namespace: kargo-demo
spec:
  upstreamStages:
    - name: <<name of the stage>>
  registerTo:
    subscriptions:
      - name: <<name of the subscription>>
  promotionMechanisms:
    argoCDAppUpdates:
    - appName: kargo-demo-test
      sourceUpdates:
      - repoURL: https://github.com/<username>/kargo-demo-gitops.git
        helm:
          images:
          - image: nginx
            key: image.name
            value: Image # Update Application to reference the new image version

Subscription should look like this:
apiVersion: kargo.akuity.io/v1alpha1
kind:  Subscriptions
metadata:
  name:   subscriptions
  namespace: kargo-demo
spec:
  repos:  #atleast one type of repo is required for a subscription
      images:
      - repoURL: nginx
        semverConstraint: ^1.24.0
      charts:
      - registryURL: https://charts.bitnami.com/bitnami
        name: nginx
        semverConstraint: ^14.1.0
      git:
      - repoURL: https://github.com/<username>/kargo-demo-gitops.git
        branch: helm # Watch this branch for new commits

This is sort of an incidental duplicate of #631, which proposes a Freight CRD and, perhaps more importantly, decoupling Freight production from the "zero stages." (This is an unofficial shorthand we use to denote stages on the far left of a pipeline that subscribe directly to repositories.)
Somewhere in the thread, this is said:

We would need a way for users to define something that produces Freight (e.g. FreightFactory)
Stages would no longer have syntax to subscribe to image/git/helm explicitly but rather a selector of freight. Though, I still like the syntax of subscribing to upstream Stages.

Note the name "FreightFactory" will not necessarily stick, but it got the point across. Whatever we call it, I believe this is exactly what you're requesting here, so I am going to close this as a duplicate of #631 -- only because that's the issue we're already working from. Work on this is already underway.
I invite you to please re-open this issue if you disagree that the proposal here and #631 are less similar than I understand them to be.

@krancour , yes I understand and agree with most if not all what is written in #631 , but I just what to make a not that I think that is missing there.
From what I understand/feel freight should be a bundle of sources(images/helm/git) and a combination of those should create a freight.
So a stage that is subscribe to sources will start a freight chain and upstream stages will will be able to promote/convey to it.
So there will be 3 types of kinds??

Source/subscription,  a single image helm and git kind
FreightGenerator/freightliners combination of source/subscription.
Stages, consumer of freights or other stages.

Hope this make some sense

So there will be 3 types of kinds??

Source/subscription, a single image helm, and git kind
FreightGenerator/Freightliner combination of source/subscription.
Stages, consumers of freights or other stages.

No. 1 doesn't seem necessary at this time. Decoupling the production of Freight from the "zero stages" is already a humongous pivot. I'd like to get that done before introducing more CRDs into the equation.
Can you explain why you think no. 1 and no. 2 need to be separate things?

@krancour ,from my view point.
We are talking about kargos (cargos) and freight and Freightliner.
So delivery hub for cargos with freights.
What goes in the freight can be made from different sources, and if I have a source that can be found in 2 freight, there no reason to query/ create this resource twice.
Look at my pictures it really explain the idea.

@tal-hason I don't want to be dismissive of ideas, but I want to propose to you that this isn't really necessary.
If you could imagine the repo(s) subscription section that is currently part of a Stage spec being lifted out of there and put in it's own CRD, that is what #631 is going for. You've left with something that is decoupled from the "zero stage" and it can assemble/create Freight from multiple sources.
It is the idea that each source needs to be a resource that I am not convinced of the necessity of.
Compare the following bits of equivalent pseudo-YAML:
Source:
  Name: my-image-repo
  URL: my-image-url
---
Source:
  Name: your-image-repo
  URL: your-image-url
---
FreightFactory:
  Name: our-freight-factory
  Sources:
  - my-image-repo
  - your-image-repo

vs
FreightFactory:
  Name: our-freight-factory
  Sources:
  - my-image-url
  - your-image-url

Either would produce the same Freight, but one is clearly more complicated than the other. I am struggling to understand what value that extra bit of complexity is supposed to be adding here.
fwiw, nothing would prevent it from being visualized exactly as you depicted.

I'll re-open this until we understand if there really is something here that's different from #631.

@krancour , if we are just moving the zero-stage to a freight-factory, there isn't much change except that now stages will be registered to freight id's instead of a stage that holds that freight-id.
This will still requires duplicating repos if there is a need to build several deployments stages from a combination of sources.
Think of sources as your storage dispenser, you have couple of those, now you want to create different delivery packages for different stages, i.e.
Developer a, is working on part 1 but needs to get all other parts of the system.
Developer b is working on part 2 but needs to get all other parts of the system.
Development env. Needs to accumulate both part 1 and 2. And from there continue the uat and prod.
I'm my idea you only have 2 sources and 3 frieght factorys
Vs.
The freight-factory only you will have 3 freight-factory's with 4 source's
So the controller will have more reconciliation loops/sampling for those repos. Twice as much.
And with more images or repos it can be multiplied many more.

there isn't much change except that now stages will be registered to freight id's instead of a stage that holds that freight-id.

This isn't true. It's a huge change.
Before #631 is done, Freight isn't a CRD. It's something that exists only as a status entry in each Stage. Eventually, every stage will forget old pieces of Freight and then they're effectively gone forever. Worse, the ability to query, filter, sort Freight, etc. requires jumping through lots and lots of hoops. It's terribly inefficient at present.
#631 will formalize each piece of Freight as a resource -- a more tangible thing that can easily be queried, filtered, sorted, and even garbage collected according to user-defined policies -- so that old Freight that is, for some reason, especially important, can be retained for longer periods.
If Freight becomes a CRD, something like the "FreightFactory" has to produce it.
If each piece of Freight then references the "Factory" that created it, every piece of Freight then has an implicit "type." The notion of a Freight type could prevent unsupported scenarios. The other day, you tried to merge multiple parallel pipelines and that's something we don't support (yet). If your final Stage had expressed that it expected Freight of a certain type, which was different from the types of Freight your upstream Stages had been expecting, the invalidity of what you were attempting would have been immediately clear.
So Freight and "FreightFactory" CRDs buy us a lot.
The thing I am still not convinced of is that we need resources to represent individual repositories.

I don't want the rules for what makes its way in the door to be defined at the "Repository" level. Imagine that you have two different pipelines that need the same image, but one wants to move every new major release of the image through the pipeline and the other wants to move every new patch release through. What I really don't want to do is have to define two separate "Repo" resources where one uses one set of subscription rules and the other uses another. I want the "Factory" that assembles Freight from upstream sources to decide on these rules.

Taking the above into consideration, if the subscription "rules" are defined at the "Factory" level, then a hypothetical "Repo" resource is almost nothing more than a simple reference to some remote repo. If that's all it is and not anything more, easier for "Factories" to reference repos by URL than by a reference to another resource. i.e. I don't want a new resource type where a string would do the trick.

Last, I am averse to additional complexities until they prove themselves absolutely necessary. The idea of making Kargo too complicated to understand is a genuine concern.

@krancour , thanks, I get your point, and I get, that the freight will hold the sources and their needed version/tag to create the unique freight ID, and I can agree with that and also live with that.
Just want to highlight #879 which regards using patterns and placeholders for image details.
so how this will be incorporated with a FreightFactory? if at all
thanks again for the detailed answer :smiley:

I think this doesn't factor into #879
#879 proposes patterns and placeholders within certain promotion mechanisms. Promotion mechanisms are mostly a mapping from an artifact to an action.  #879 doesn't change that. "When you see an image that matches this pattern, here's what I want you to do with it." I think that still works fine after these changes.
What I will concede to is #631 (or #631 + the extra CRD you proposed; with or without doesn't matter here) combined with the pattern/matching support creates a strange possibility that a "FreightFactory" could begin subscribing to an additional artifact that it didn't subscribe to before and this might be matched by a regex in a stage and now the Stage is doing a new promotion step that it wasn't before and perhaps didn't intend. That could be avoided if we didn't do #631, but #631 is a priority for too many other reasons.
Priority-wise #631 comes first. I am already working on it. We can re-evaluate after how that affects the pattern/placeholder proposal and how to mitigate it. It's also possible that we decide the issue is too esoteric to deserve any special consideration. Since "FreightFactories" would be scoped to the same project/namespace as the Stages that consume them, I am perhaps overstating the likelihood that someone changes a Factory without anticipating its downstream repercussions.
At any rate, we're starting to go off on a tangent that we can worry about later when it's time to deal with #879.
For now, I am satisfied that the original issue in this thread is indeed a duplicate of #631 and am going to close this, but we'll be open-minded about re-opening this if we discover after #631 that something still seems to be missing.


Design: Rethink dept header design layout (mobile)

see screenshot

I propose that we left-align the menu buttons. If we do this, then we should also switch the order so that the "Menu" button is closest to the edge of screen.


The v1.6.1 candidate failed to install on LXD ... then it worked!

I was testing the latest 1.6.1 candidate today on LXD via conjure-up. The conjure-up screen indicated the "Application did not start successfully" in red which means the charms did not install correctly.
I checked the juju status and everything looks fine, so I went off investigating which charm failed (possible bug here that conjure-up does not tell us which charm failed). I found an ERROR in the kubernetes-worker charm log file. http://paste.ubuntu.com/24330227/
It looks to me like the snap failed to download (unexpected EOF), but when Juju retried it it worked. This confuses me.
2017-04-06 21:28:25 INFO install core 76.86 MB / 78.35 MB   98.10% 4.26 MB/s 0
2017-04-06 21:28:25 INFO install core 77.80 MB / 78.35 MB   99.30% 4.27 MB/s 0
2017-04-06 21:28:25 INFO install core 78.31 MB / 78.35 MB   99.96% 4.25 MB/s 0
2017-04-06 21:28:25 INFO install error: cannot perform the following tasks:
2017-04-06 21:28:25 INFO install - Download snap "core" (1577) from channel "stable" (unexpected EOF)
2017-04-06 21:28:25 INFO install
2017-04-06 21:28:25 INFO install Traceback (most recent call last):
2017-04-06 21:28:25 INFO install   File "/var/lib/juju/agents/unit-kubernetes-worker-0/charm/hooks/install", line 19, in <module>
2017-04-06 21:28:25 INFO install     main()
2017-04-06 21:28:25 INFO install   File "/usr/local/lib/python3.5/dist-packages/charms/reactive/__init__.py", line 78, in main
2017-04-06 21:28:25 INFO install     bus.dispatch()
2017-04-06 21:28:25 INFO install   File "/usr/local/lib/python3.5/dist-packages/charms/reactive/bus.py", line 434, in dispatch
2017-04-06 21:28:25 INFO install     _invoke(other_handlers)
2017-04-06 21:28:25 INFO install   File "/usr/local/lib/python3.5/dist-packages/charms/reactive/bus.py", line 417, in _invoke
2017-04-06 21:28:25 INFO install     handler.invoke()
2017-04-06 21:28:25 INFO install   File "/usr/local/lib/python3.5/dist-packages/charms/reactive/bus.py", line 291, in invoke
2017-04-06 21:28:25 INFO install     self._action(*args)
2017-04-06 21:28:25 INFO install   File "/var/lib/juju/agents/unit-kubernetes-worker-0/charm/reactive/kubernetes_worker.py", line 137, in install_snaps
2017-04-06 21:28:25 INFO install     snap.install('kubectl', channel=channel, classic=True)
2017-04-06 21:28:25 INFO install   File "lib/charms/layer/snap.py", line 44, in install
2017-04-06 21:28:25 INFO install     _install_store(snapname, **kw)
2017-04-06 21:28:25 INFO install   File "lib/charms/layer/snap.py", line 142, in _install_store
2017-04-06 21:28:25 INFO install     stderr=subprocess.STDOUT)
2017-04-06 21:28:25 INFO install   File "/usr/lib/python3.5/subprocess.py", line 626, in check_output
2017-04-06 21:28:25 INFO install     **kwargs).stdout
2017-04-06 21:28:25 INFO install   File "/usr/lib/python3.5/subprocess.py", line 708, in run
2017-04-06 21:28:25 INFO install     output=stdout, stderr=stderr)
2017-04-06 21:28:25 INFO install subprocess.CalledProcessError: Command '['snap', 'install', '--classic', 'kubectl']' returned non-zero exit status 1
2017-04-06 21:28:25 ERROR juju.worker.uniter.operation runhook.go:107 hook "install" failed: exit status 1
2017-04-06 21:28:25 DEBUG juju.worker.uniter.operation executor.go:84 lock released

We should take a look at fixing this, if possible.

iunexpected EOF seems to me the network connection dropped. juju
should auto retry the hook but we should have the snap layer retry
snap install failures regardless.
On Thu, Apr 6, 2017 at 5:58 PM Matt Bruzek<EMAIL_ADDRESS>wrote:

I was testing the latest 1.6.1 candidate today on LXD via conjure-up. The
conjure-up screen indicated the "Application did not start successfully" in
red which means the charms did not install correctly.
I checked the juju status and everything looks fine, so I went off
investigating which charm failed (possible bug here that conjure-up does
not tell us which charm failed). I found an ERROR in the kubernetes-worker
charm log file. http://paste.ubuntu.com/24330227/
It looks to me like the snap failed to install, but when Juju retried it
it worked?
2017-04-06 21:28:25 INFO install core 76.86 MB / 78.35 MB   98.10% 4.26 MB/s 0
2017-04-06 21:28:25 INFO install core 77.80 MB / 78.35 MB   99.30% 4.27 MB/s 0
2017-04-06 21:28:25 INFO install core 78.31 MB / 78.35 MB   99.96% 4.25 MB/s 0
2017-04-06 21:28:25 INFO install �[Kerror: cannot perform the following tasks:
2017-04-06 21:28:25 INFO install - Download snap "core" (1577) from channel "stable" (unexpected EOF)
2017-04-06 21:28:25 INFO install
2017-04-06 21:28:25 INFO install Traceback (most recent call last):
2017-04-06 21:28:25 INFO install   File "/var/lib/juju/agents/unit-kubernetes-worker-0/charm/hooks/install", line 19, in
2017-04-06 21:28:25 INFO install     main()
2017-04-06 21:28:25 INFO install   File "/usr/local/lib/python3.5/dist-packages/charms/reactive/init.py", line 78, in main
2017-04-06 21:28:25 INFO install     bus.dispatch()
2017-04-06 21:28:25 INFO install   File "/usr/local/lib/python3.5/dist-packages/charms/reactive/bus.py", line 434, in dispatch
2017-04-06 21:28:25 INFO install     _invoke(other_handlers)
2017-04-06 21:28:25 INFO install   File "/usr/local/lib/python3.5/dist-packages/charms/reactive/bus.py", line 417, in _invoke
2017-04-06 21:28:25 INFO install     handler.invoke()
2017-04-06 21:28:25 INFO install   File "/usr/local/lib/python3.5/dist-packages/charms/reactive/bus.py", line 291, in invoke
2017-04-06 21:28:25 INFO install     self._action(*args)
2017-04-06 21:28:25 INFO install   File "/var/lib/juju/agents/unit-kubernetes-worker-0/charm/reactive/kubernetes_worker.py", line 137, in install_snaps
2017-04-06 21:28:25 INFO install     snap.install('kubectl', channel=channel, classic=True)
2017-04-06 21:28:25 INFO install   File "lib/charms/layer/snap.py", line 44, in install
2017-04-06 21:28:25 INFO install     _install_store(snapname, **kw)
2017-04-06 21:28:25 INFO install   File "lib/charms/layer/snap.py", line 142, in _install_store
2017-04-06 21:28:25 INFO install     stderr=subprocess.STDOUT)
2017-04-06 21:28:25 INFO install   File "/usr/lib/python3.5/subprocess.py", line 626, in check_output
2017-04-06 21:28:25 INFO install     **kwargs).stdout
2017-04-06 21:28:25 INFO install   File "/usr/lib/python3.5/subprocess.py", line 708, in run
2017-04-06 21:28:25 INFO install     output=stdout, stderr=stderr)
2017-04-06 21:28:25 INFO install subprocess.CalledProcessError: Command '['snap', 'install', '--classic', 'kubectl']' returned non-zero exit status 1
2017-04-06 21:28:25 ERROR juju.worker.uniter.operation runhook.go:107 hook "install" failed: exit status 1
2017-04-06 21:28:25 DEBUG juju.worker.uniter.operation executor.go:84 lock released
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/juju-solutions/bundle-canonical-kubernetes/issues/250,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAET1enoKXCNNe4gyqxaBnwqskfsDjZPks5rtWAOgaJpZM4M2ODR
.

The spell uses juju-wait to watch for the deployment to settle.  The current behavior of juju-wait is to fail immediately if it sees a hook error, which is not entirely unreasonable, but doesn't account for Juju's automatic hook retry.  I think that juju-wait should be modified to give an errored application a bit of time to allow for the auto-retry to kick in.
I also think that either snapd or the snap layer should handle this more gracefully, though, so that the hook doesn't error in the first place.

While working on a fix it was pointed out to me that is is really a snapd issue, and it should retry if it gets an error. Fixing this error at a snapd level makes the most sense for me.
I found a snapd bug about EOF that might be related, and added the log information I got here. https://bugs.launchpad.net/snapd/+bug/1677557

My proposed fix landed in layer-snap here: https://github.com/stub42/layer-snap/pull/8
We should rebuild the kubernetes charms and verify snap installs continue to work.


fix: eip6963 connection issues

Breaking Changes
N/A
Changes

feat:
fix: issue where auto connects would trigger pop ups from locked wallets
chore:

Associated Issues
https://github.com/WalletConnect/web3modal/pull/1698

@arein left it written down here. Feel free to modify/move/update https://github.com/WalletConnect/web3modal/issues/1704


Out-of-memory error due to excessive allocation in Lists with big size.

When trying to compute proofs, for the beacon state validators array, it brakes with the following error:
memory allocation of<PHONE_NUMBER>8800 bytes failed
Aborted (core dumped)

I assume this is cause due to the following line allocating the full node_count space. But, this can cause problems with bigger size lists especially if they are not full. For example if we try to proof that a validator in the BeaconState is active or exists, on computing that proof it will brake due to allocating too many slots on the buffer for empty nodes as it will try to allocate  slots for the full list up to VALIDATOR_REGISTRY_LIMIT.
You can reproduce it with the following code:
    const VALIDATOR_REGISTRY_LIMIT: usize =<PHONE_NUMBER>776;

    let vec = vec![100u64, 1000u64];

    let h = List::<u64, VALIDATOR_REGISTRY_LIMIT>::try_from(vec).unwrap();
    let p = h.prove(&[1.into()]);

hi @noahfigueras thanks for this!
I think your assessment is correct and basically I used a more naive version of Merkle tree computation for proving as for the initial implementation I focused on getting the APIs right and everything else around the proof generation/verification
Instead, the line you linked should use this implementation which is memory-efficient and I think will resolve this problem
I haven't had time to update the implementation but PRs are very welcome!

I think a nice way forward would be to adjust Tree so that it can handle "virtual" nodes (subtrees that are instances of a "zero" tree) and then refactor merkleize_chunks_with_virtual_padding so that it uses this updated Tree (so just returning the root node in this function), and then during proving the Tree can just be walked (via generalized indices) to compute the necessary internal nodes in a memory-efficient way

ah, this is referenced in #141 btw

hi @ralexstokes! I think that's good approach, I'll try to get this working. Is it ok moving merkleize_chunks_with_virtual_padding inside Tree?

Hey, will the PR be merged? I had a similar error.


export helpers and expose crudSwagger decorator

export helper files in index.ts, so we can create a custom decorator to add Swagger props to custom routes.
also, this package can expose CrudSwagger() decorator to add similar swagger props to that used by @Crud()

[ ]  replace export * from './crud/crud-routes.factory'; with export * from './crud';

[ ] expose CrudSwagger() decorator

usage:
@Crud(...)
export class MyController implements implements CrudController<MyEntity>{

@Get("me")
@CrudSwagger()
customRoute(){
 return this.service.getUserInfo()
}

}

@its-dibo  I don't quite get why do you need both. The idea of exposing the internals of crud is to be able to implement the CrudSwagger() decorator outside of the package, right ?
Anyway, I don't have the time now to implement this decorator, but would definitely help if someone takes on it.

ok, as a first step we'll just export the helpers so others can implement their own decorators.
a next step is to provide a ready-made functionality that can be used directly.


Adds B3SinglePropagation for the "b3" header

This will be used for JMS and other propagation formats such as w3c
tracestate entry. It is notably more efficient than multi-header.
Example header: b3: 4bf92f3577b34da6a3ce929d0e0e4736-00f067aa0ba902b7-1
To aid in migration, this teaches the normal B3 to attempt to extract
single header variants as well.
See https://github.com/openzipkin/b3-propagation/issues/21

cc @openzipkin/instrumentation-owners in case you'd like to see an example

review after commit welcome. merging to progress some messaging things which hinge on this

This is a follow-up change which gets somewhat obsessive about parsing tests in order to expose functions public cc @devinsba https://github.com/openzipkin/brave/pull/769


Support While Node

@tkonolige

Thanks @Hzfengsy!

Hi @tkonolige, can you help to release it?
I'd like to add while node support in TVMScript, so it would be great if we can make it support by a release version of synr.

@Hzfengsy I've published the 0.4 release that includes this PR. Note that it also adds local function support, decorator support and global and non local support.


Warn when trying to uv run a package without build configuration

This enhances uv run logic in order to detect and warn if it is trying to run a packaged script without any build-system configuration in the project manifest.
Ref: https://github.com/astral-sh/uv/issues/6998#issuecomment-2329291764
Closes: https://github.com/astral-sh/uv/issues/7034

@lucab thanks for the self-review comments!

From an out-of-band chat with @zanieb:

the condition detection can be simplified as a has_scripts() && !is_package() heuristic.
uv sync is a better place for the warning for now, as uv run could be too annoying in the general (non-script) case.
this could be improved by parsing the scripts tables and key-match; that allows key-matching in uv run and adding more details in uv sync. Tracked at https://github.com/astral-sh/uv/issues/7428.


low default fetchMaxWaitMs?

Curious why the default fetchMaxWaitMs is so low at 100ms? This kind of defeats the purpose of long polling.
Is there some known issue with higher values?

cc @hyperlink

it appears it is 500, but I'm shocked by that too....
Not sure why this wouldn't be 5+ minutes.


fix one typo and one markdown formatting

@user670 Thank you for your contribution. Would you take a moment to sign the Contributor License Agreement (CLA)? After the CLA is signed, someone can review your pull request. Thanks!
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@microsoft-github-policy-service agree
Sorry was waiting for end of my shift to sign this on my own (without involving my company) but then forgot about it

@User670 Looks good, thanks!


refactor(client): remove "description" from Commit._repr_attrs

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 83.434%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4991

Relevant Lines:
5982

💛  - Coveralls


BatchData should support components in dict or nested lists

In Keras we often organize components of features/targets into dicts, which is practical when the model has many options that affect number of components and their shape.
Most Tensorpack DataFlows have no problem with that, except for BatchData which only expects flat list of components (not even nested list).
A workaround is to prepare flat list, apply BatchData and then reorganize components into the desired structure. However it's not quite nice. This should be responsibility of BatchData.
I'd be happy to implement this.

Upvote!

Now most DataFlow assumes list. I feel some inconvenience with it as well. Allowing dict will be great!

And I prepared a DataFlow that compares two others like zip, checks
recursively equality of their nested components and reports differences.
Useful regression testing to quickly see we broke anything. I'll try to
clean it up and make a PR. I wish I had it earlier today. :)
Dne čt, 17. 5. 2018 19:00 uživatel Yuxin Wu<EMAIL_ADDRESS>napsal:

Now most DataFlow assumes list. I feel some inconvenience with it as well.
Allowing dict will be great!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ppwwyyxx/tensorpack/issues/768#issuecomment-389937808,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAbOrIjRJQuURF3kNNILiWqPqgCo3eWWks5tzazFgaJpZM4UC5k7
.

So far I think it can be done in the code using:
    def _aggregate_batch(data_holder, use_list=False):
        if isinstance(data_holder[0], map):
            iterat_indices = data_holder[0].keys()
        else:
            size = len(data_holder[0])
            iterat_indices = range(size)
        result = []
        for k in iterat_indices:

...or even better having a separate function, that would give us iterator of indices to iterate over (indices for list, keys for map, anything for anything else ...)


spark: 1.6.0 -> 2.1.0

Tested in linux with:
nix-env -f . -i spark-1.6.0
nix-env -f . -i spark
cc @thoughtpolice @offlinehacker
Motivation for this change
Have nixpkgs provide a newer version of Spark.
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[X] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

Thank you


LoadingPlaceholder no longer works

Hi,
I use the Droid part of your library, and as of 2.1.1 the LoadingPlaceholder is no longer functioning.
It worked as of 2.1.0.
Steps:

Use the LoadUrl method, to load an asset from the internet
Use the LoadingPlaceholder referencing a drawable using ImageSource.CompiledResource
wait for the remote asset to load.

Expected:
While loading the image from the internet, I would expect to see the placeholder asset
Actual:
The LoadingPlaceholder asset is only used to transition to the downloaded image upon loading of the asset from the network.
Notes:
I have tested using the latest version of the library (2.1.6) and this issue still exists.
I regressed the versioning to find the last point this actually worked which might help you track down the cause of the problem.

Are you sure? Please clone repo, run sample Xamarin.Forms app and see that's working fine.

Hi all,
I have the same issue as well , but I'm using Forms. placeholders are not working and I'm on the latest update .

You have to provide more info, do you use stable Android channel? Xamarin.Android Version: <IP_ADDRESS>

Hi,
I have created a test project that shows the error
https://github.com/roberthewitt/FFImageExample
Please also find attached my Xamarin -> About -> Details output.
I use Genymotion, using a Nexus 5, on 6.0.0 , api23.
This bug also happens on any device that I deploy it to so, in my eyes, is unrelated to the emulator.
Thanks,
Rob
xamarin_about.txt

@daniel-luberda  did you need any more details or is what I provided ok?
Cheers,
Rob

Hi. I'll look into this today.

Any updates on the status of a fix?

It's fixed now, sorry for a delay.

Thanks for the fix,
Is this being rolled into a public release?


Expose decoder to user

Exposing the decoder improves code quality and increases the amount of code that can be shared between the platforms for decoder implementation. Also this should give users an easy way to create and plug in custom decoder logic.
This is a part of #8

This is on hold, its probably impossible to provide a generic decoder interface that is valid for all platforms :(

This wont happen, closing PR keeping branch.


Coverage decreased (-97.7%) to 0.0%

VSCode Version: 1.25
OS Version:

It seems there is some issue with code coverage for this repo, as coverage decreased (-97.7%) to 0.0%. If this is intended behavior, then please remove the coverage badge from readme.
Steps to Reproduce:

Please visit: https://coveralls.io/builds/18036959

Does this issue occur when all extensions are disabled?: Yes/No

I have deleted the Coveralls build, since we don't really use it.
Created https://github.com/Microsoft/vscode/issues/55647


Propagate Peer 'error' event up from Pool

It seems this is useful to be able to catch when using Pool rather than Peer directly.

Coverage remained the same at 91.92% when pulling 146c9630aba0daa5ecb37037373f5d758e710362 on throughnothing:propagate-error-from-pool into bb1ecb3fbc6ba672c6bfb2bb39c609e9a04b88ec on bitpay:master.

LGTM


Hide Flash Button

I'm using card.io on an android tablet (Nexus 7 2013) and it does not have a camera flash. However, there is a flash button displayed on the camera view. Is there a way to hide it?

Closing this due to age and the hope that this bug is fixed in newer versions of the Nexus Android.


New generic set_services is now in place with the legacy interface being

just a deprecated stub.

Description

Implemented a new generic setservices that is designed to wrap user setservices.  This is as opposed to the legacy generic setservices that is called from within user setservices.  That legacy procedure is now an empty (deprecated) subroutine.
Related Issue

Motivation and Context
Small step to supporting NUOPC.
How Has This Been Tested?

Tested with C12 on laptop with ifort.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Trivial change (affects only documentation or cleanup)

Checklist:

[X] I have tested this change with a run of GEOSgcm (if non-trivial)
[X] I have added one of the required labels (0 diff, 0 diff trivial, 0 diff structural, non 0-diff)
[X] I have updated the CHANGELOG.md accordingly following the style of Keep a Changelog

I think we are hitting a build issue in the GEOSldas where they need an update (https://github.com/GEOS-ESM/GEOSldas/pull/515) to build GEOSldas. A PR was accepted in GEOSgcm_GridComp and they couldn't update the LDAS until that PR was merged.
Now that it is, they have a new PR to let it all work again.

I think we are hitting a build issue in the GEOSldas where they need an update (GEOS-ESM/GEOSldas#515) to build GEOSldas. A PR was accepted in GEOSgcm_GridComp and they couldn't update the LDAS until that PR was merged.
Now that it is, they have a new PR to let it all work again.

@mathomp4:  As far as I can tell, the "build issue in GEOSldas" was because I renamed a constant in the GCM GC that is also used in GEOSldas.  Most times when there are land changes in GCM GC (typically by us), we need a matching GEOSldas PR to make it all work.  I don't think this has to do with generic or legacy MAPL interfaces.  Maybe I'm missing something...

I think we are hitting a build issue in the GEOSldas where they need an update (GEOS-ESM/GEOSldas#515) to build GEOSldas. A PR was accepted in GEOSgcm_GridComp and they couldn't update the LDAS until that PR was merged.
Now that it is, they have a new PR to let it all work again.

@mathomp4: As far as I can tell, the "build issue in GEOSldas" was because I renamed a constant in the GCM GC that is also used in GEOSldas. Most times when there are land changes in GCM GC (typically by us), we need a matching GEOSldas PR to make it all work. I don't think this has to do with generic or legacy MAPL interfaces. Maybe I'm missing something...
Update: Actually, it wasn't the renaming of a constant in this case, but rather the intentional change in the interface of subroutine catch_calc_soil_moist() in the GCM GC. But same message - this was caused (intentional) land changes.

@gmao-rreichle Yes. I just wanted to let @tclune know that the issue wasn't in his PR. It's just part of all the changes GEOS has been having today due to fun repo updates. The GEOSgcm was also broken in CI for a while today until Scott and I could get things synced up. (And, weirdly, the Chem repo didn't break but that was due to a bug in my CI. Fun day for CI!)

Re-submitted the CI. Let's do this! 🚀

Grah! What is happening with the adas test?

This is zero-diff! But blocking until @tclune says it's good to go (so I don't accidentally merge in)


Fix send invoice logic and linked unit tests

I want to merge this change because it fix send invoice logic.

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-3993 Missing channel_slug in send_invoice


[ISSUE #1146]Update Github Actions CI

Which Issue(s) This PR Fixes(Closes)

Fixes #1146
Brief Description

How Did You Test This Change?

🚀Thanks for your contribution🎉. CodeRabbit(AI) will review your code first🔥


[Bug]: Firmware updates not working

Checklist

[X]  I am not using Home Assistant. Or: a developer has told me to come here.
[X] I have checked the troubleshooting section and my problem is not described there.
[X] I have read the changelog and my problem is not mentioned there.

Deploy method
Docker
Zwavejs2Mqtt version
6.12.0
ZwaveJS version
9.3.0
Describe the bug
When attempting to update a firmware, it lets me select the device, and the firmware file and then does nothing. I also see a user disconnect/reconnect in the live debug logs.
To Reproduce

Open z2m web interface
Select device, click advanced, Firmware update - begin
Use default of target 0
Select firmware file
Dialog closes, nothing happens, and I see this in the z2m debug screen

This is all that is logged in the z2m log (set to debug), and nothing at all is logged in the zwavejs log file:
2022-06-15 17:15:42.990 DEBUG SOCKET: User disconnected TRWpuXXUGzn_8xrDAAAZ
2022-06-15 17:15:44.314 DEBUG SOCKET: New connection kgRV8vkesEqQ4GO2AAAd
2022-06-15 17:15:44.325 DEBUG SOCKET: Event INITED emitted to kgRV8vkesEqQ4GO2AAAd
Expected behavior
Firmware update should begin
Additional context
No response

@Botched1 do you press enter or click on the button to start the update?

As soon as I click "open" on the file dialog where I select the firmware file, it immediately goes back to the advanced dialog. This is immediately after clicking "open" on the file selection dialog:

And of course the next time I tried it - it worked. Confusing, as I tried maybe 10 times before reporting (including after reboots, and restarts of the software)...
I would say ignore this for now and I'll see if I can reproduce it... If not, I'll just close this issue.

I can't reproduce this, closing issue.

I don't have a way to reliably reproduce it either, but I've noticed the same issue as @Botched1: Very often the FW update does not "take". Also, there is no UI feedback for sleeping nodes (the circle only starts once the sleeping node has been woken up).


Revert "Add deprecation notices for old k8s"

Reverts cert-manager/website#396

/override dco

/lgtm


Created duplicated label and cannot delete it

My actions before raising this issue

[x] Read/searched the docs
[x] Searched past issues

During configuring my first project, I have managed to create two labels with the same name

After I've discovered it, I've tried to remove one of "door" labels, however, if I press on a cross sign or remove respective JSON text in "Raw" editor, I see pop-up with Error 400

Expected Behaviour

I should not be allowed to create duplicated labels
I should be allowed to remove duplicated label, if they managed to appear.

Current Behaviour
I cannot edit project labels and remove duplicates
Your Environment

Git hash commit (git log -1):
 # git log -1
 commit 7f4b1853aba06584afb7698bd07358cb2f4ae7ca (HEAD -> develop, origin/develop, origin/HEAD)
 Author: Andrey Zhavoronkov<EMAIL_ADDRESS> Date:   Thu Jul 1 17:29:05 2021 +0300

 Fixed cvat_ui image build in case of OOM error (#3379)

 * fixed cvat_ui image build in case of OOM error

 * updated changelog

Edited nginx configuration template according to #1231:
diff --git a/cvat_proxy/conf.d/cvat.conf.template b/cvat_proxy/conf.d/cvat.conf.template
index 0c7ab685..bb496e87 100644
--- a/cvat_proxy/conf.d/cvat.conf.template
+++ b/cvat_proxy/conf.d/cvat.conf.template
@@ -1,9 +1,3 @@
-server {
-    listen       80;
-    server_name  _ default;
-    return       404;
-}
-

Docker version docker version
Docker 20.10.7

Are you using Docker Swarm or Kubernetes?
No

Operating System and version (e.g. Linux, Windows, MacOS):
Ubuntu 20.04 with all recent updates;

Next steps
You may join our Gitter channel for community support.

Hi @wl2776, thx for the report. Could you please provide the exact steps to create two labels with the same name? I cannot reproduce it.

Hi @wl2776, thx for the report. Could you please provide the exact steps to create two labels with the same name? I cannot reproduce it.

It was not easy, but I've done it again :)

Open two tabs with CVAT UI in the browser. I use Opera, so I right-clicked on the tab then clicked on "Duplicate Tab" menu item
Add labels in those tabs.

I've managed to resolve this issue by renaming one of labels.

Hi @wl2776, thx for the report. Could you please provide the exact steps to create two labels with the same name? I cannot reproduce it.

I've done it accidentally. Reproduction was not easy, but I've done it again :)

Open two tabs with CVAT UI in the browser.  I use Opera, so I right-clicked on the tab then clicked on "Duplicate Tab" menu item.
Add labels in those tabs.

This can be done in Chrome too.
And I've managed to resolve initial issue by renaming one of labels.
Also I had the issue i moved the task to new project , but wondering whether i can do it programmatically for many task


ARROW-4490: [Rust] Add explicit SIMD vectorization for boolean ops in "array_ops"

This PR adds explicit SIMD for boolean ops, and, or and not.
I moved array_ops into the new compute module.  From the outside this module serves the same purpose as the previous array_ops module (all kernels will be accessible from this namespace) and the remaining array_ops implementations are exposed via the compute module currently.  As I add explicit SIMD for more kernels they will migrate from array_ops into their own modules under compute.  I am keeping sub-modules under compute (as apposed to compute.rs) as SIMD can get rather verbose and it seems that compute may expand in the future.
I have included benchmarks where I re-create the old default implementations for others to take a look at the speed improvement.  It's not clear whether we need the non-SIMD versions in the benchmarks long term but I left them in for now to make the non-SIMD/SIMD comparison.
There are likely more optimizations possible (processing the values and null bit buffers in a single loop for instance) but I wanted to get the cleanest impl first and add further optimizations later if needed.

cc @andygrove @sunchao @nevi-me @siddharthteotia


[Skeleton] Move from lab to core

Breaking changes

[Skeleton] Move the component from the lab to the core. This component will become stable.
-import Skeleton from '@material-ui/lab/Skeleton';
+import Skeleton from '@material-ui/core/Skeleton';

Skeleton and Alert was moved from lab to core in 5.0.0-alpha.11
But after update i see, that Alert and Skeleton  are  enable  from /core/ and  also from /lab/
node_modules/@material-ui/lab/Skeleton
node_modules/@material-ui/core/Skeleton

@Krasnopir If you import from the lab, you will get a deprecation warning.


POD mismatch between landing page and WestDAAT main page

I'm curious why the landing page has a lot more PODs showing up that dont show up in the landing page
Landing pages
https://westdaat.westernstateswater.org/details/right/UTwr_WR291014
Homepage view
https://westdaat.westernstateswater.org/?state=N4Ig7gTiBcoA4HsA2BPJBLAdgUxgbQF0AaETJAE3QDF0kAXbCAEQEM6WZQk306BXcrmiY%2BSJCSQJMAc14ChIsSUoRsAYzropAZxgBmZWxYAFBFjq7oAdgC%2BNkgFsWcTiABeCBA4Ay2AG7YSDAAjMEAdAAMwQAsAKzBegAcEQBsEbHRwVYSPPyCMJlh0ZlWeikATHqxVhF6AJwGIJIycvnQALSh5WHBiQl1sX0RqWV1dkA

Wait, if I search for owner name="Usa Fish Wildlife Service", the Bear River Bird Refuge does not show up, why?
https://westdaat.westernstateswater.org/?state=N4IgtghgDiBcoC8D2SwBkCmA3DAbOAjAAwB0A7ACwDMATEWQGwCcNArCwwBw0A0IuEAC4BLQQFcAJhjgUCJCjUpNORKqwZlKZPriQA7AOajJ02AFoCBGiQJkmrKhQqsiyl2QC%2BfAO4AnOKAARhh6GABmwgDGwhC4AKoAzhgJcADaALp8SN6hvgDCAgkJwhGRQsL6KbAZfLG6ZSL6API5GP6wIIkQAAQAYsIJABbdAOrCuBK4JRjdAMptWFHSPkJts0hivpEYACoAnlDJaZkgCYKrVTUgvsI4vgBCEMV6AHIQYEfVJ8J6kbgmAFEAB4YMBQQRwMKxJJ8KBIXB7KahY58PQTYT9XCCNoAESEEDgejEuFwHg8QA
Landing page with this owner
https://westdaat.westernstateswater.org/details/right/UTwr_WR291014

So sorry; this issue actually is because of Mapbox layers. Zoom in more and the PODs show up.


alpha / beta support?

I'm trying to test out 2.0alpha; I'm currently blocked with no 2.0.0 alpha release for postgresql-provider.  Any idea when it will be added?
Thanks.

Actually, I think the issue is with -driver, not -provider.

@treaves the issue will be with both given the Provider will need to point to Vapor 2 to work. I think that it shouldn't be much of an issue and none of the provider stuff has changed (currently) in Vapor 2

The first release with Vapor 2 beta support is here: https://github.com/vapor-community/postgresql-provider/releases/tag/2.0.0-beta.1
Please submit any bugs or regressions. A lot changed. :-)


[iOS][cocoapods] Fixes import error in "RCTNativeAnimatedNodesManager.h"

Fixes import error in "RCTNativeAnimatedNodesManager.h" mentioned at #13198.
This same fix was included in #13217 but it was not merged.
Unfortunately #13785 didn't fix this problem.

The import in the header needs to be namespaced, this is not a correct solution.

@javache when using nameapace it's impossible to compile the code without a script that changes the import.
I know the non namespaced imports are deprecated but this is a long time blocker for anyone who uses cocoapods and wants to upgrade

This definitely worked on 0.44.0 after #13785
There was no need for namespace change.

@vjeranc Could you please tell us in what device configuration (OSX/Xcode version) are you able to build it?
Also, we're using cocoapods 1.2.1.

I just doubled check, it's not working:
react native - 0.44.1
xcode - 8.3.2
cocoapods -  1.2.1
We are using swift pods in the project so the podfile contains use_frameworks! declaration

Looks like we have to add this header to "Copy header files" build step inside for RCTAnimation project.

if this will fix it that would be awesome.. :)

RCTValueAnimatedNode.h is already under Copy Files build phase in RCTAnimation project

¯_(ツ)_/¯

Any conclusion to this PR? can't get my project to build without hacks 😕

Please merge. Thanks!

@javache
What about
instead of
import "RCTValueAnimatedNode.h"
use
import  <React/RCTValueAnimatedNode.h>
it works (but tested only with the lib at cocoa pods)

Okay, does anyone have idea which exact problem we are trying to solve?
Why exactly is the issue #13198 happening?

I must admit to having no idea why the previous fix does not work. We've been building 0.45.1 and newer updates without any issues.
There might be some conflicts in project.pbxproj and the modified version after applying *.podspec definitions. Our project.pbxproj was heavily cleaned (removed references to all unneeded projects like tests and similar). Although I do remember trying out the fix on a clean 0.44 project and it worked.

@rodriguesbreno Changed import "RCTValueAnimatedNode.h" to import <React/RCTValueAnimatedNode.h> and it works now 👍   Updated the PR, thank you!

This should probably be solved using this approach: https://github.com/facebook/react-native/issues/13198#issuecomment-371749528

@shergin The root issue is that when using cocoapods the React Native modules/frameworks are all built as one and thus can't reference/import from each other using the namespaced global/system imports (Header Search Paths), as they're not separate pods but just sub-specs of one pod. Instead, in this case, the files need to be imported locally from the User Header Search Paths. https://stackoverflow.com/a/1044399/1925631 Importing locally breaks the non-cocoapods builds, thus requiring the if statement to handle both cases. https://clang.llvm.org/docs/LanguageExtensions.html#include-file-checking-macros
#if __has_include(<React/RCTValueAnimatedNode.h>)
#import <React/RCTValueAnimatedNode.h>
#else
#import "RCTValueAnimatedNode.h"
#endif


Could not load type "Nest.IBulkResponse" with 7.0.0-beta1 Nest client and Automapper in .Net Core 2.2

NEST/Elasticsearch.Net version: 7.0.0-beta1
Elasticsearch version: 7.1.1
When I use the latest NEST client in combination with Automapper in .Net Core 2.2 I get

Could not load type "Nest.IBulkResponse" from assembly "Nest, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=96c599bbe3e70f5d";

I don't get this error when I'm using 7.0.0-alpha1 (unfortunately tokenizer in custom analyzers seems to have issues with the alpha1)

I suspect one of your projects depends on 7.0.0 release. Can you verify all your projects are on 7.0.0 and not 7.0.0-beta1

Thank you, mate! A third party library was referencing the 6.8 version. Sorry for the false alarm, at least now its documented and if someone else has similar problems they might find your solution. :) Now, I'm using the 7.0 stable Version in my project and updated the third party library - everything works fine!
Cheers!


Videos Don't Load Ad-Hoc on Big Sur

General troubleshooting tips
Before logging an issue please check that:

[x] You have the latest version installed (There may be a beta version that fixes your issue), see here for the latest releases and bug fixes: https://github.com/JohnCoates/Aerial/releases
[x] Your issue isn't already mentioned in our issues. You may find a workaround there or a similar request already made.
[x] Your problem isn't mentioned in the troubleshooting page.

If none of this fixes your issue, tell us about the problem you are experiencing or the feature you'd like to request.
Required information
In order to help us sort your issue, we ask that you provide the following information:

[x] Mac model: MacBook Pro 13" 2017 Four Thunderbolt 3 Ports
[x] macOS version: Big Sur 11.0.1
[x] Monitor setup: Two external, VX3276-FHD Display, 31.5", 1920x1080, and Dell U2720Q Display, 27", 3840x2160

If appropriate, please enable Debug mode and Log to disk in Advanced tab and replicate your bug, then attach the AerialLog.txt file (You can access this file through the Advanced tab).
AerialLog.txt
Description of issue / Feature request
I upgraded from Catalina, and Aerial failed to update, I was on 1.6 so I uninstalled and reinstalled to the latest version.  If I download the videos locally, they work, but I have extremely limited space. Previously, I would download them as they played in the old version, and this works for me, as I have no bandwidth constraints, but it seems like they do not work at all unless I pre-download videos. Is this feature no longer available?
Note I do not have any third party Firewall, or even the native MacOS one, enabled.

I see in the FAQ this was removed, unfortunate.

Hey there, just to give you a bit more info on what you said

I upgraded from Catalina, and Aerial failed to update, I was on 1.6

Yep so Catalina introduced new security mechanisms that broke auto updates, that's why you were stuck there with a really old version. And that's why there's a new standalone updater (Aerial Companion) to fix that issue with V2, you can find it here : https://aerialscreensaver.github.io/

so I uninstalled and reinstalled to the latest version.

I did see your note about not using a firewall, black screen can still happen in some scenarios like trying to force streaming.

If I download the videos locally, they work, but I have extremely limited space. Previously, I would download them as they played in the old version, and this works for me, as I have no bandwidth constraints, but it seems like they do not work at all unless I pre-download videos. Is this feature no longer available?

So as you've seen, streaming is no longer available for a lot of reasons. But there's now a new rolling cache option + cache limit that you can configure so it uses minimal disk space and still give you something very close to streaming while keeping a very small cache.

You can go as low as 1 GB if you want, though at least 2 GB would probably be what I'd recommend.
Note there's also a bug currently in the rotation system that's fixed in the latest beta (if you use companion, you can pick desired version > beta to get the latest one).


RDBMS table model

This PR brings a notion of Table Model. We now don't store everything in single-json-column tables, but we try to determine the best column types to represent data. RdbmsWriteFile scans the incoming Vector[Data] and incrementally creates columns. For simple elements like Data.Int, Data.String etc. we create appropriate DB fields. For nested Data.Obj, arrays and such, we use a jsonb column.
More details available here: https://docs.google.com/document/d/1loSdaInrdMaFg08EHqaLa2oxWP6wZziN2-uzLenJCuA/edit?usp=sharing
More column type <-> quasar data type mappings will be added later.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: kciesielski:x: mikolak-netYou have signed the CLA already but the status is still pending? Let us recheck it.

@drostron @rabbitonweb The PR is ready for review. There was a failing test with couchbase, probably requiring restarting the build.

@kciesielski 👌🏾, review commencing.

I've been pulled away by a few other issues. I'll be able to offer a review today or tomorrow.

@drostron PR updated according to your remarks. Failed builds seem to be caused by some non-related issues, please bump the build.

@drostron Equality updated and tests added.

@drostron I pulled all checks to the top-level in tests, also added "must" literal where needed to make test output correct.


Evaluation takes too much time. Is it normal?

The dev data contains 5000 sequences.
INFO:tensorflow:Creating ZeroBridge in mode=eval
INFO:tensorflow:
ZeroBridge: {}
INFO:tensorflow:Starting evaluation at 2017-05-04-07:43:55
I tensorflow/core/common_runtime/gpu/gpu_device.cc:975] Creating TensorFlow device (/gpu:0) -> (device: 0, name: GeForce GTX 1080,
pci bus id: 0000:01:00.0)
W tensorflow/core/framework/op_kernel.cc:993] Out of range: FIFOQueue '_30_dev_input_fn/parallel_read_1/common_queue' is closed an
d has insufficient elements (requested 1, current size 0)
[[Node: dev_input_fn/parallel_read_1/common_queue_Dequeue = QueueDequeueV2component_types=[DT_STRING, DT_STRING], timeou
t_ms=-1, _device="/job:localhost/replica:0/task:0/cpu:0"]]
W tensorflow/core/framework/op_kernel.cc:993] Out of range: FIFOQueue '_30_dev_input_fn/parallel_read_1/common_queue' is closed an
d has insufficient elements (requested 1, current size 0)
[[Node: dev_input_fn/parallel_read_1/common_queue_Dequeue = QueueDequeueV2component_types=[DT_STRING, DT_STRING], timeou
t_ms=-1, _device="/job:localhost/replica:0/task:0/cpu:0"]]
W tensorflow/core/framework/op_kernel.cc:993] Out of range: PaddingFIFOQueue '_29_dev_input_fn/batch_queue/padding_fifo_queue' is
closed and has insufficient elements (requested 64, current size 0)
[[Node: dev_input_fn/batch_queue = QueueDequeueUpToV2[component_types=[DT_INT32, DT_STRING, DT_INT32, DT_STRING], timeout
_ms=-1, _device="/job:localhost/replica:0/task:0/cpu:0"](dev_input_fn/batch_queue/padding_fifo_queue, dev_input_fn/batch_queue/n)]
]
W tensorflow/core/framework/op_kernel.cc:993] Out of range: PaddingFIFOQueue '_29_dev_input_fn/batch_queue/padding_fifo_queue' is
closed and has insufficient elements (requested 64, current size 0)
[[Node: dev_input_fn/batch_queue = QueueDequeueUpToV2[component_types=[DT_INT32, DT_STRING, DT_INT32, DT_STRING], timeout
_ms=-1, _device="/job:localhost/replica:0/task:0/cpu:0"](dev_input_fn/batch_queue/padding_fifo_queue, dev_input_fn/batch_queue/n)]
]
W tensorflow/core/framework/op_kernel.cc:993] Out of range: PaddingFIFOQueue '_29_dev_input_fn/batch_queue/padding_fifo_queue' is closed and has insufficient elements (requested 64, current size 0)
[[Node: dev_input_fn/batch_queue = QueueDequeueUpToV2[component_types=[DT_INT32, DT_STRING, DT_INT32, DT_STRING], timeout_ms=-1, _device="/job:localhost/replica:0/task:0/cpu:0"](dev_input_fn/batch_queue/padding_fifo_queue, dev_input_fn/batch_queue/n)]]
W tensorflow/core/framework/op_kernel.cc:993] Out of range: PaddingFIFOQueue '_29_dev_input_fn/batch_queue/padding_fifo_queue' is closed and has insufficient elements (requested 64, current size 0)
[[Node: dev_input_fn/batch_queue = QueueDequeueUpToV2[component_types=[DT_INT32, DT_STRING, DT_INT32, DT_STRING], timeout_ms=-1, _device="/job:localhost/replica:0/task:0/cpu:0"](dev_input_fn/batch_queue/padding_fifo_queue, dev_input_fn/batch_queue/n)]]
W tensorflow/core/framework/op_kernel.cc:993] Out of range: PaddingFIFOQueue '_29_dev_input_fn/batch_queue/padding_fifo_queue' is closed and has insufficient elements (requested 64, current size 0)
[[Node: dev_input_fn/batch_queue = QueueDequeueUpToV2[component_types=[DT_INT32, DT_STRING, DT_INT32, DT_STRING], timeout_ms=-1, _device="/job:localhost/replica:0/task:0/cpu:0"](dev_input_fn/batch_queue/padding_fifo_queue, dev_input_fn/batch_queue/n)]]
W tensorflow/core/framework/op_kernel.cc:993] Out of range: PaddingFIFOQueue '_29_dev_input_fn/batch_queue/padding_fifo_queue' is closed and has insufficient elements (requested 64, current size 0)
[[Node: dev_input_fn/batch_queue = QueueDequeueUpToV2[component_types=[DT_INT32, DT_STRING, DT_INT32, DT_STRING], timeout_ms=-1, _device="/job:localhost/replica:0/task:0/cpu:0"](dev_input_fn/batch_queue/padding_fifo_queue, dev_input_fn/batch_queue/n)]]
W tensorflow/core/framework/op_kernel.cc:993] Out of range: PaddingFIFOQueue '_29_dev_input_fn/batch_queue/padding_fifo_queue' is closed and has insufficient elements (requested 64, current size 0)
[[Node: dev_input_fn/batch_queue = QueueDequeueUpToV2[component_types=[DT_INT32, DT_STRING, DT_INT32, DT_STRING], timeout_ms=-1, _device="/job:localhost/replica:0/task:0/cpu:0"](dev_input_fn/batch_queue/padding_fifo_queue, dev_input_fn/batch_queue/n)]]
INFO:tensorflow:Finished evaluation at 2017-05-04-08:59:35
INFO:tensorflow:Saving dict for global step 1001: bleu = 16.43, global_step = 1001, log_perplexity = 1.39028, loss = 1.36901, rouge_1/f_score = 0.746607, rouge_1/p_score = 0.945662, rouge_1/r_score = 0.628931, rouge_2/f_score = 0.45424, rouge_2/p_score = 0.582968, rouge_2/r_score = 0.384094, rouge_l/f_score = 0.354937
WARNING:tensorflow:Skipping summary for global_step, must be a float or np.float32.

I have similar issue:
INFO:tensorflow:Starting evaluation at 2017-05-04-20:25:54
INFO:tensorflow:Finished evaluation at 2017-05-04-20:43:21
Not so fast(

I have the same issue; training is relatively fast, evaluation sometimes seems to take forever:

INFO:tensorflow:Starting evaluation at 2017-05-05-08:24:36
INFO:tensorflow:Finished evaluation at 2017-05-05-08:36:39

INFO:tensorflow:Starting evaluation at 2017-05-05-08:42:20
INFO:tensorflow:Finished evaluation at 2017-05-05-08:54:32

Anyone any idea how to properly deal with this, except reducing evaluation after x amount of steps?
What about the warnings btw?

WARNING:tensorflow:Skipping summary for global_step, must be a float or np.float32.

Turning off ROUGE metrics in text_metrics.yml helps me to significantly improve evaluation time

@sld
What is a clean way to turn off ROUGE ? Just delete the following classes with ROUGE in the .yml file?

default_params: &default_params

separator: " "
postproc_fn: "seq2seq.data.postproc.strip_bpe"
metrics:
class: LogPerplexityMetricSpec
class: BleuMetricSpec
params:
<<: *default_params
**- class: RougeMetricSpec
params:
<<: *default_params
rouge_type: rouge_1/f_score
class: RougeMetricSpec
params:
<<: *default_params
rouge_type: rouge_1/r_score
class: RougeMetricSpec
params:
<<: *default_params
rouge_type: rouge_1/p_score
class: RougeMetricSpec
params:
<<: *default_params
rouge_type: rouge_2/f_score
class: RougeMetricSpec
params:
<<: *default_params
rouge_type: rouge_2/r_score
class: RougeMetricSpec
params:
<<: *default_params
rouge_type: rouge_2/p_score
class: RougeMetricSpec
params:
<<: default_params
rouge_type: rouge_l/f_score*

@milanv1 Yeah, I deleted all those lines and it worked.

@milanv1 it worked


Add (Leave) Quotation Marks to Field/Control Names for AL Keywords

Hi,
Right now AL Keywords are saved without quotes when I use the OnSave Features of the Waldo's CRS AL Language Extension.
What do you think about adding quotation marks around AL Keywords.
There is a related issue on the al-code-outline extension:
https://github.com/anzwdev/al-code-outline/issues/302#issuecomment-1003387295

@waldo1001 This is indeed an annoying side effect when running the alCodeOutline cleanup commands (including FormatDocument) combined with an app that has filenames controlled by CRS.
alCodeOutline settings:

CRS settings:

It results in additional / excessive delta's in our commits / PR's.
Result when running alCodeOutline cleanup (" " have been removed)

Result when saving the file (" " are added again on reserved keywords)

Hi,
I have this problem too in all my projects, but I think the best is TO KEEP all this quotes because in some special names like "Code" , "Type", etc which are spread throughout the standard code and in my custom code too, the VSCode AL Language automatic coloring resalta this words erroneusly in blue color because it thinks they are Keywords and not Identifiers.

I personally don't mind the extra quotes .. same reason as Javier.. .
In terms of the extension: it doesn't have any parser in the system, so it doesn't know if whatever it generates is going to be "compileable".  That's why I chose the "will always compile" way.. 🤷♂.
I could be adding additional coding on every part of where it changes/adds code to handle cases like this - but in a way, it's almost like building a parser myself.  And that's simply something I can't, won't and have the time for.
That being said - since Vjeko now has a parser out there, I might throw everything overboard, and go with that.  At least, when it's able to catch what I need ... I talked to him, and he doesn't have the bits I need yet .. 🤷♂.

ALProject14.zip


LeetCode links

First of all, thank you so much for this wonderful repo, and also sorry for asking my question in English as inputting Chinese is too painful for me.
My question is actually not necessarily related to this repo, so please forgive me for posting such question here. I'm a bit confused regarding the leetcode links provided in this repo. It seems that the links point to the Chinese version of the leetcode site correct? What's the relation between the Chinese version site and the English version site? I ask because I'm a regular user on the English leetcode site, but I found that I wasn't able to login to the Chinese version site with my English site user name and password.

I've figured out how to login to the Chinese version site using my user name and password from the English version site. However, I'm still confused as to whether everything on the Chinese site are also on the English site.


Add weighted performance script

Complete:

Parser works
Multiple problem types can be read
Output to screen (size, count, kernel time, total time are written to the screen)
CMake changes are done to make sure it can run as an executable

To-Do:

Actually use --best-solution (recently merged) and read kernel times (currently passing a dummy value)
Write output to csv
Figure out what to do with Validate params
Figure out what to do with the ini file
Add help message

Closing this so it doesn't take CI cycles


impl: custom error response

Implement: https://github.com/envoyproxy/gateway/pull/4259
Issue: https://github.com/envoyproxy/gateway/issues/1400

@zhaohuabing can you make CI happy?

e2es are failing ...


Reduce dependnecies of ra_vfs

In preparation for moving ra_vfs to a separate repo with extensive cross-platform CI, remove dependency on ra_thread_workder and ra_arena.

r? @pnkfelix

The repo is optimistically set up here: https://github.com/rust-analyzer/ra_vfs

r=me once commit nits are addressed.

bors r=pnkfelix


Extend schema - optional machine dict

to be used for sending optional machine metrics/data

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (fd9aac6) 100.00% compared to head (dd23bb4) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #27   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines           76        85    +9
  Branches         3         3
=========================================
+ Hits            76        85    +9

Impacted Files
Coverage Δ

tests/test_post_live_metrics.py
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Poke @daavoo


Refactor GUI presentation

Add answer visualization as text
Add legends for graphical elements
Add more problem information: URL, difficulty ..
Check font sizes on axes and GUI in general


upgraded core addons for 1.25 support (1.24 as well)

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/do-e2e-tests


New documentation plan

I think we have a lot of back and forth about the documentation of the waldo-cli and this is making it very painful.
Let's move to:

Readme is the source of truth for the public CLI documentation. It should roughly have all the documentation that currently exists in https://github.com/waldoapp/waldo-go-cli ; and Readme should reference to that repo at the top: all the source is public, check it out at https://github.com/waldoapp/waldo-go-cli.
we should remove the sections of the product about CI integration. And make sure to redirect https://app.waldo.com/applications/app-xxx/docs to the proper Readme section
we should remove all documentation from Waldo-go-cli. It should simply state: Documentation for how to integrate this plugin can be found in ${readmeUrl}.
We should add a link at the top of the builds page to the Readme section: "Learn how to integrate your CI".
It seems like there is a way to auto fill in a token in Readme when it's added in the query params. The link from above should include the proper token from the app so that the users can see a auto-generated doc that corresponds to their app credentials.
the documentation in Readme should only include information that are useful to our customers. specifically we can remove all mentions of real device from there
we can use this repo waldo-go-agent to keep track of internal documentation

It's exactly the plan I'm doing so LGTM 😅

I’m OK with it. I’m not quite clear on 2 things:

Exactly where in readme this should live (perhaps @manoufeb can put a placeholder doc there).
How to autofill the upload token. (Is there a way to provide a fake upload token in case someone comes to the doc from outside the product?)

Thanks, @eBardX for taking care of this. ❤
Here is the structure I see

Builds & CI/CD

iOS - CI Upload

Bitrise
Fastlane
GitHub Action
...

Android - CI Upload

Bitrise
Fastlane
GitHub Action
...

For the token normally yes, if you don't have the token in the URL params there is a placeholder. I'm waiting for Readme support to come back to me with a better explanation of how to do it.
You can see an example here:
https://developer.mixpanel.com/docs/ios-objective-c-quickstart?projectToken=Hello_John_From_Waldo

Is that Builds & CI/CD section what is currently called just Build now?

Yes

ok, I will make the changes, but leave them unpublished until we have clarity on the upload token.

@manoufeb I have largely finished the docs. I figured out how to define a variable and provide a default value. The variable is called waldoUploadToken. However, I have not yet figured out how to substitute from the URL. There must be some other magic I am missing.
Two outstanding issues (besides that):

Not sure how to reconcile the instructions on the (existing) Upload a new build page with those of the (new) iOS | Uploading your simulator build to Waldo and Android | Uploading your emulator build to Waldo pages.
Not quite sure where to put this: https://github.com/waldoapp/waldo-go-cli/blob/master/RUNS.md

For the upload token Readme.com answered me this:
The example you see implemented by Mixpanel is actually custom developed and not a ReadMe feature.
Generally speaking, to pass user data into ReadMe you'll need to provide it via JSON Web Token, take a look at our custom login documentation for details:
https://docs.readme.com/docs/custom-login-with-readme
https://docs.readme.com/docs/passing-data-to-jwt#user-data-json-format
I hope you find this information helpful! Please let me know if you have any other questions!```

Can you have a look and let me know if we can do it or not?

1. Can we do a manual export simulator build  page under `iOS | Uploading your simulator build to Waldo` & `Android | Uploading your emulator build to Waldo`

2. I think it's covered in the rules section. Can you confirm @laurentsigal ?
![image](https://user-images.githubusercontent.com/19815229/173636591-b21d533a-c35a-465e-917c-7b0afbfebb64.png)

I’ll take a look at the customized solution, but it may be above my pay grade.
As for the other 2 items:

I think @laurentsigal is of the opinion that we should just nuke this: https://docs.waldo.com/docs/uploading-a-new-build
You are correct that in the product, the rules section self-documents nicely. However, the screenshot in the ReadMe docs (https://docs.waldo.com/docs/run-rules) shows incorrect command line syntax and so should be replaced.

I'll upload a new screenshot for 2.

Yeah, that JWT stuff will need to be handled on the backend so that when a customer follows a link from the product to the docs, the customer is logged in and their upload token (or anything else) is passed along. I believe the use of <<waldoUploadToken>> in the new docs is all that is necessary on that side of things. I’ve already provided a default value for it that works great.

@laurentsigal is that something we can easily do or it's not something we want to prioritize?
https://docs.readme.com/docs/custom-login-with-readme
https://docs.readme.com/docs/passing-data-to-jwt#user-data-json-format

I think this is something that we should easily be able to add in webapp, see https://github.com/waldoapp/webapp/issues/3868

@eBardX can you close this one since the following work is on webapp#3868, please.
I don't have access to that repo so I can't close it 😅


Add support for One-Click unsubscribing according to RFC8058

I would like to add the 'List-Unsubscribe-Post' header to emails according to the RFC8085 standard. I've drafted some code, but I'm unsure whether to send it like this directly or make it configurable in some way. When I looked at the code of the CRM module for the mailer (https://github.com/remp2020/crm-remp-mailer-module/blob/13a06bf1251070fc00a1fae77eb330deb7ad005f/src/Presenters/MailSettingsPresenter.php#L81), it complies with the standard for unsubscribing from newsletters.
More information about the new header:
https://datatracker.ietf.org/doc/html/rfc8058
https://aws.amazon.com/blogs/messaging-and-targeting/an-overview-of-bulk-sender-changes-at-yahoo-gmail/
https://blog.google/products/gmail/gmail-security-authentication-spam-protection/
Gmail’s FAQ and Yahoo’s FAQ both clarify that the one-click unsubscribe requirement will not be enforced until June 2024 as long as the bulk sender has a functional unsubscribe link clearly visible in the footer of each message.

Hi, thanks for the PR.
The change itself sounds reasonable and looks OK to me. If it's used against REMP Mailer module in CRM, it really does what it's supposed to do.
However (as you already mentioned) there needs to be additional level of control for users who have their own CRM, or their own unsubscribe landing page. Those people should enable this only when they're sure, that POST request to their unsubscribe URL actually unsubscribes the user, otherwise they wouldn't do what RFC says.
For example when user clicks our unsubscribe button, we present the message with option to pause their newsletter subscriptions for a month instead of full unsubscribe. User is not unsubscribed right away (not even if it's a POST request). We can't have this header present.
I'd say that this could be a seeded config option, which would be configurable on the /mailer/settings/ page; right next to the default mailer page. We need to:

Add new boolean config option in DatabaseSeedCommand.
Try to describe the config in the description so user of Mailer understands when it needs to be enabled (unsubscribe URL needs to unsubscribe user if it's requested with POST)
Alter the feature in Sender to only add the extra header if the config option was enabled.

If you have more spare time, you can try to add this to the PR. If not, we can take over once we're available :slightly_smiling_face: .

Looks good, we'll test it. Thanks! The "recipient" replacement occurs in Mailgun by Mailgun, we'll test that with our sandbox account, don't worry :).

Hi, this is now merged in the master branch and will be part of the next-week release. Since this is only mirrored repository split from our monorepo, We'll be linking the split commit here next time we split the repository.
Thanks for the contribution! :)


False positive bucket_prefix on aws_s3_bucket

Description
Should not be seen as a drift since it's a Terraform internal attribute.
resource "aws_s3_bucket" "test-prefix" {
  bucket_prefix = "test-prefix-"
}

Found changed resources:
    - test-prefix-20210611092741609600000001 (aws_s3_bucket):
        - bucket_prefix: "test-prefix-" => <nil>

Possible Solution
Let's delete this attribute inside the NormalizeFunc of the aws_s3_bucket with the next architecture:
val.SafeDelete([]string{"bucket_prefix"})

Is that issue free for grabs?
I'd like to implement the change as my first contribution, look pretty straightforward.

@a-vorobiev Yeah you can go on !


[Spike] Explore concurrent push to data feeds

User Story
As a developer I want to do a spike to see automate's performance when sending data to data feeds parallelly, I will use this work when I want to send data feed data as quickly as possible without impacting automate.

Explore the maximum number of push connections we can do parellally
Check if automate is impacted or not

Definition of Done
When we are sure about all the things we need to do to parallelize the data push. Find out if there are any challenges. Check if automate is impacted or not?

Automate can send upto 40000. Anything above that starts to throw an error. (https://github.com/chef/automate/issues/5278)


Bind to both IPv4 and IPv6 when localhost is specified (#231).

#231
cc @halter73 @mikeharder @Tratcher

Travis is producing a bunch of Error -99 EADDRNOTAVAIL address not available errors in travis tests on both OS X and Linux: https://travis-ci.org/aspnet/KestrelHttpServer/jobs/132455305

No more filters.

:shipit:

Updated to disallow binding to localhost:0. cc @davidfowl

:bell:

You can :shipit: after addressing my comments.  Or, you could wait until I figure out the design for #766, since it might impact this change as well.


Add a close button for fullscreen

Add a close button so the app can be closed on devices.

Not necessary, as devices have back buttons.


Phone Number Locator

Project title
Phone Number Locator
Project Description
This website can give details about a phone number
Stack:
And in order to tick the check box just but x inside them for example - [x] like this. Please delete options that are not relevant.

[x] Html
[x] CSS
[x] JavaScript

@gurjeetsinghvirdee Kindly assign


Add missing notice about order of run

A notice about the order of run in workspaces

@behnammodi thanks for opening this issue!
We actually vendor in the docs for the CLI, so you are going to want to open a PR over at https://github.com/npm/cli to make this change.

@MylesBorins Thanks


Build foliage binaries

We should have prebuilt foliage binaries readily available for developers to download.
On GitHub I had a GHA CI that, on release tag, builds exes for linux, macos, and windows and adds them to the releage page.
I later disabled it and focused on doing everything with nix instead. It's still there and could be revived.
https://github.com/andreabedini/foliage/blob/main/.github/workflows/haskell.yml
On hydra, I have foliage binaries built for aarch64-darwin, aarch64-linux, x86_64-darwin, x86_64-linux, and also x86_64-linux-musl.
I never managed to get aarch64-linux-musl to work. I haven't tried windows.
See https://github.com/andreabedini/foliage/blob/main/flake.nix

Alternatively: publish to Hackage and tell people to install it with cabal?

@yvan-sraka The CI doesn't seem to be working, can you have a look?

@yvan-sraka The CI doesn't seem to be working, can you have a look?

I quickly iterate on it here https://github.com/input-output-hk/foliage/pull/67, it's now trigger only on new tags pushed, so I should probably test it in a fork :)

If you make it create draft releases, we can just test it and then delete the release.