K8S complains failure-domain.beta.kubernetes.io/zone is deprecated since v1.17

Here is our ClickhouseInstallation:
 podTemplates:
      - name: clickhouse-in-zone-westeurope-1
        zone:
          values:
            - westeurope-1
        podDistribution:
          - type: ClickHouseAntiAffinity
            scope: ClickHouseInstallation

and here is how statefulset is created
    spec:
      affinity:
        nodeAffinity:
          requiredDuringSchedulingIgnoredDuringExecution:
            nodeSelectorTerms:
            - matchExpressions:
              - key: failure-domain.beta.kubernetes.io/zone
                operator: In
                values:
                - westeurope-1

Then K8S would return a warning like this
Warning: spec.template.spec.affinity.nodeAffinity.requiredDuringSchedulingIgnoredDuringExecution.nodeSelectorTerms[0].matchExpressions[0].key: failure-domain.beta.kubernetes.io/zone is deprecated since v1.17; use "topology.kubernetes.io/zone" instead

https://github.com/Altinity/clickhouse-operator/blob/28ca2237ba6419c9ac1019da2e462a8c42a7a919/pkg/model/normalizer.go#L577

please check in 0.19.0

@minhtd1981 ,
      podDistribution:
      - scope: ClickHouseInstallation
        type: ClickHouseAntiAffinity
        topologyKey: "topology.kubernetes.io/zone"

@alex-zaitsev you are right :( Not sure why I could not see that field customizable. Thanks a lot

I could finally fix the warning.
zone:
          values:
            - westeurope-1
        podDistribution:
          - type: ClickHouseAntiAffinity
            scope: ClickHouseInstallation

should be
       zone:
          key: "topology.kubernetes.io/zone"
          values:
            - westeurope-1
        podDistribution:
          - type: ClickHouseAntiAffinity
            scope: ClickHouseInstallation


OMP

Hi, Can CMAQ support openmp + mpi ?

Hi XiaoLinhong,
Please submit your question to the CMAS User Forum.
https://forum.cmascenter.org/
Thank you, Liz


Wdo3650/panel commanding usability

Updated with usability enhancements.

Changes:
-Command bar animates down
-First button margin, padding and color changes
-Other buttons in command bar match hover and active styles from ODN usability change

Looks good however the commanding seems broken and not selectable on small screens. Probably a pointer events issue. We might want to add some space between the buttons on small screens as well.

THIS IS APPROVED, PULL REQUEST CAN BE COMPLETED IF USER DESIRES THIS RESULT. THANK YOU FOR YOUR WORK, MAY TEH FORCE BE WITH YOU.


[wip][Hotfix] Fix reg withdraw hook [OSF-7545]

Purpose
Withdrawing does not manually call on node updated actions on the registration (only on the node), causing withdrawals to not hit the share hook properly.

Changes
Add a manual call to the hook in the retraction model at the end of on complete

Side effects
None known

Ticket
[#OSF-7545]
https://openscience.atlassian.net/browse/OSF-7545

QA Notes
Make sure retracting a registration populates the withdraw tag on Registries and that it generally doesn't break anything :)

Not sure why this one is failing when the nearly identical develop fix is passing (https://github.com/CenterForOpenScience/osf.io/pull/6945), will explore in the AM.

Old master. The other PR only applies now. Closing.


Release 0.14

Note: do not merge until #130 gets merged onto release-0.14

@rberrelleza could you review the release notes? I think that part didn't have a previous PR:
https://github.com/okteto/docs/pull/134/files#diff-fdc6ee7e265beb2f6428769ee199f648a54ed18b263b1d44c154d9bbe004969f


[Storage] Add data plane Fileshare Storage swagger

2020-10-02 swagger for fileshare

@tasherif-msft has this been reviewed by the arch board?

This hasn't been reviewed by the Arch Board or API Stewardship Board, but it's an existing API we're grandfathering in.  The ADP team has been shipping off these specs for years now so we're confident in their overall quality.


visualisation with trained model

Hi, great package!
I am looking at the example in plotting_examples folder.
These seem to work independently from a trained torch model ?
what would be a minimal way / example to use those with a trained model ?
e.g. how to visualise the latent traversal of a trained model
Best regards

Hi @dgm2, thank you!
Yes, sorry you are right. Most of them are independent.
You can have a look at the helper code inside the pytorch lightning utilities, one of the callback classes is specifically for generating latent traversals:
https://github.com/nmichlo/disent/blob/ff462ba567a734041874cc584b97695a81729498/disent/util/lightning/callbacks/_callback_vis_latents.py#L197-L261
This code is more complicated than it need to be for most cases because of some additional handling and quirks. Maybe we can add a specific example for latent traversals.
This method uses helper functions from disent.util.visualize.vis_img to convert tensors to images, disent.util.visualize.vis_latents to generate latent sequences, and disent.util.visualize.vis_util to combine the images together into a grid or make sequential frames.

thanks!
the callback method returns stills, frames, image
how should I input these into plot_dataset_traversals ? or into visualize_dataset_traversal
or what is are corresponding values there? e.g. does stills corresponds to grid as input into plt_subplots_imshow
e.g. this example makes sense?
many thanks!
trainer = pl.Trainer(
    max_steps=2048,
    gpus=1 if torch.cuda.is_available() else None,
    logger=False,
    checkpoint_callback=False,
    max_epochs=1
)
trainer.fit(module, dataloader)
# trainer.save_checkpoint("trained.ckpt")

viz = VaeLatentCycleLoggingCallback()
stills, frames_, image_ = viz.generate_visualisations(trainer_or_dataset=trainer, pl_module=trainer.lightning_module,
                                                      num_frames=4, num_stats_samples=15)

plt_scale = 4.5
offset = 0.75
factors, frames, _, _, c = stills.shape

plt_subplots_imshow(grid=stills, title=None, row_labels=None, subplot_padding=None,
                    figsize=(offset + (1 / 2.54) * frames * plt_scale, (1 / 2.54) * (factors + 0.45) * plt_scale),
                    show=False)

This example seems to make sense, but admittedly it has been a while since I last touched the code.

stills is an array of shape (num_latents, num_frames, 64, 64, 3) containing individual latent traversals.
frames is a concatenated version of stills intended to create videos, so the individual stills over the factors dimension are combined together into an image grid. The final array is approx of shape (num_frames, ~(64 * grid_w), ~(64 * grid_h), 3).
image is a single image that you can plot that has all the latent traversals merged together into a grid, the x axis of this grid will correspond to num_latents and y axis to num_frames (or vice versa) so the shape will be approx: (~(64 * num_latents), ~(64 * num_frames), 3)

You can try and plot images directly with plt.imshow(image). Or create your own visualization/animation with the frames or stills


Add revert with reason functionality for mock contracts

Not being able to revert with a reason on a mock contract is prohibitively limiting.
Say you have contract A and contract B. The latter depends on the former. Say that the former throws specific errors depending on the state of the properties in the two contracts. If I mock contract A, I have to modify a lot of tests to revert with "Mock revert" instead of my bespoke error messages.
This PR does the following:

Adds a new revertReason property in the MockCall struct
Changes the signature of the __waffle__mockReverts function, by adding a second parameter revertReason
Adds a new function revertWithReason in the object returned by the stub function - which makes my change backwards-compatible!
Modifies the tests so that they work with the updated __waffle__mockReverts function

@PaulRBerg Good stuff, merging. Would add a PR with docs?

Released in 3.0.3

Thanks for merging, Marek!

Would add a PR with docs?

I could do this, in case you haven't done it in the meantime?

Released in Waffle 3.1.0. Didn't have a chance to work on docs yet. Would love to get some help :)

Done: https://github.com/EthWorks/Waffle/pull/366.


Viewing Assets from External Portal Returns a 500 Error

Hello,
I get a 500 error when creating or updating assets using the HuduAPI module, adding it to the company's External Portal, and then trying to view it from said portal.
The only way I've found to fix it is to edit the asset in the internal assets via the web dashboard and click the update button.
Is this a known bug?
Please let me know if you need any more information.
Thank you in advance and for making a great tool for connecting to the Hudu API.

Hudu support had said this about a month ago when I asked:

This "500 error" is 100 percent stemming from Luke Whitelock's script after testing. My best accurate guess would be that there's something corrupted with the way that script is handling "Asset Link Fields". When I tested I had no issues sharing assets to any company's "External Portal" that had an "Asset Link" field.

I realize now that this 500 error doesn't happen when I don't try to have a "Site" location asset or link to any asset for that matter.
The error when I use AssetTag, which is what the Hudu support told me to use to link to an Asset.

I started with this integration with TacticalRMM: https://github.com/bc24fl/tacticalrmm-scripts/blob/main/Tactical_Hudu_Sync.ps1
And made quite a lot of changes: https://github.com/ZzBombardierzZ/tacticalrmm-scripts/blob/main/Tactical_Hudu_Sync.ps1
Please let me know if I'm linking to Assets wrong or if there's anything I can do to fix this issue while still linking to an Asset.

Hey @ZzBombardierzZ  this is actually a bug in the function but if you're using valid, properly cased field types it shouldn't cause an issue with the asset layout itself.
I'll be pushing a new version shortly that will include a revised version of the validation that should work.

Hi @greenlighttec, I appreciate the fast response and the new update. The error in PowerShell is no longer there. However, I still get a 500 error when the asset has a linked asset to it. I could be doing something wrong on my end, but I'd greatly appreciate any advice.
You can see the code I'm running here: https://github.com/ZzBombardierzZ/tacticalrmm-scripts/blob/main/Tactical_Hudu_Sync.ps1
My only idea is that maybe I'm getting the linked asset to link to incorrectly?
With the linked asset:

Removing the linked asset:

Thanks,

@greenlighttec Thank you so much! Adding  -Compress -AsArray to this line $siteToLinkJson = $siteToLinkArray | ConvertTo-Json did in fact fix it for me.
Thanks!


bug: remote peer fault

This is a bug report
Problem
Having problem getting a correct connection with default defaultBootstrap, it throws me an ['remote pair fault']
Packages
packages:
    "@waku/react": "^0.0.5-8c435e4",
    "@waku/sdk": "^0.0.24",
    "protobufjs": "^7.2.6",

Proposed Solutions
Describe one or several (or none) solutions to fix the problem describe above.
While connected to the defaultBootstrap like
export default  function Feed({ topic, id }: { topic: string, id:string}) {
  return (
    <LightNodeProvider options={{defaultBootstrap: true}}
    protocols={[Protocols.LightPush, Protocols.Store]}
    >

      <PageHeader
        title="Feed"
        subtitle="See what other think about this topic"
      />
      <WakuFeed topic={topic} id={id} />

    </LightNodeProvider>
  );
}

already tried to add different protocols to see if that makes a different. but it did not.

import { useWaku } from "@waku/react";
import CreateFeed from "./create-feed";
import RetrieveMessages from "./retrieve-messages";

export default function WakuFeed({ topic, id }: { topic: string; id: string }) {
  // Create and start a Light Node
  const { node, error, isLoading } = useWaku();

  if(isLoading) return
Loading...

    if(error) return
Error: {JSON.stringify(error)}

  return (
    <>

        {node && <CreateFeed contentTopic={topic} node={node} />}
        {node && <RetrieveMessages contentTopic={topic} node={node} />}

      {/* <Post key={index} details={message} /> */}
    </>
  );
}

Both retrieve as sending messages do not work.

This is the way I send the message:

  const encoder = createEncoder({ contentTopic });
  const { push } = useLightPush({ node, encoder });
  const [message, setMessage] = useState("");

  const sendMessage = async () => {
    if (!push || message.length === 0) {
      return;
    }
      const protoMessage = PProjectFeed.create({
      id: crypto.randomUUID(),
      message: message,
      comments: [],
      likes: 0,
    });
    const payload = PProjectFeed.encode(protoMessage).finish();

    const result = await push({ payload: payload });

    console.log(result);
    // Check for errors{
    if (result?.failures && result.failures.length > 0) {
      toast({
        title: `Write to ${contentTopic} failed`,
        description: result.failures[0].error,
      });
      return;
    }
    toast({
      title: "Success",
      description: "Message sent",
    });
  };

If this is a bug report and you know how to fix the problem, feel free to include a proposal or open a PR.
Or feel free to omit this section.
@vpavlin said the following in discord:
This error comes from the service node (nwaku) when there is a problem with Lightpush connection
THe js-waku involvement there should be probably to attempt a reconnection with the node or picking up a different node in case of this error (which it does not do at the moment)
Notes

This problem should be fixed within next version of the library.
How new version should be used -  https://github.com/waku-org/lab.waku.org/pull/60
Please, install npm i -S @waku/sdk@next or any other library and tell if it helps with the issue observed.
Important: be aware that there are braking changes, follow in particular this one in https://github.com/waku-org/lab.waku.org/pull/60


[review] Loss of search-as-you-type is a major loss of functionality

As part of the review for https://github.com/NixOS/nixos-homepage/pull/454

This is especially for the options, as it affects packages a bit less.
I'm bad at remembering things. All things. I generally don't remember the option names right, but only parts of them. I actually use the search-as-you-type feature of the site to work around my failings.
Losing search as you type in options would actively harm, at the very least, my ability to give help regarding options, in addition to slow down considerably my own personal use of options.
If this is not possible, I will likely have to implement an additional, web-based or not, fuzzy-finder that searches as you type for option names.

This was a known loss from the beginning since we are using elasticsearch on the backend and not loading all the data in the browser.
I personally was actually annoyed by search-as-you-type (and also some others as well), but i understand that it is useful to others.
Plan is to bring some sort of search as you type functionality via Completion suggester which I think will address your use case.
If you agree with the above proposal I would close this ticket since the work on completion suggesters is being tracked under #34

The linked issue is finding it problematic because of the performance issues. It is unclear whether it would be undesirable if there was not the performance issues of the package set being handled in the browser.
I am 99.99% sure I am not alone that will find, mainly for the options, that the loss of search as you type fuzzy finding is a net regression in usability for the search. You don't have to block on my opinion, but I am saying it, it's not going to be liked.
At the very least, I now know I have to play around finding a useful replacement.

This is being addressed in https://github.com/NixOS/nixos-search/pull/74

First of all: great job on this new search tool, it's wonderfully speedy.
That said, this completion UI strikes me as counterproductive (these observations are based entirely on using it, I haven't looked at the code):

It's slower than the actual search (!?): if I type "sound" and press enter, I'll get the "real" search results before the completions show, while the completion seems to be slow enough to warrant a loading animation

It doesn't add any value — It shows fewer relevant options, since it's only based on option name prefixes

It hides other UI behind a popup that contains little information

It adds a level of indirection before the relevant information: I have to select an option from the dropdown, then click on the option name again in the real results, in order to see the option's documentation. In detail, considering a "happy path" where the desired result is available on the first page of results:

Type search term, e.g. "sound"
Select result from dropdown, e.g. "sound.mediaKeys"
Click outside dropdown to dismiss it
Click on option (e.g. "sound.mediaKeys.enable") to see docs

Without the dropdown on the new interface, it's better (though this is discouraged by the loading animation for the suggestions):

Type search term, e.g. "sound"
Press enter
Click on option to see docs

Whereas with the old search, it's better still (not sure what this "reloading" behaviour is?):

Type search term, e.g. "sound"
Click on option to see docs

Video of the first approach vs the old search: https://sphalerite.org/dump/optionsearch.mp4

@lheckemann option (b) is not possible to do since the whole thing will become too expensive on the backend. suggestions queries are much simpler queries we do to the backend. But we could of course try.
I'm currently working on last issue regarding suggestions (#120).
If you are interested I would love to have a screen sharing session with you to learn more how you think UX should be, since we all come with some assumptions that are impossible to write in text.

I will actually close this issue since it was already implemented and there is already a ticket for further improvements.


Expand refs to list types to address documentation/cdk generation error

Issue #, if available:
#77
Description of changes:
Looking at some other examples of resource providers and the generated documentation, it appears that a $ref to an array type generates the object with a single list property, whereas when the array type is specified at the top level, it generates an array as expected.
This PR "expands" the array references.
Worth waiting for confirmation from the cloudformation team before merging as it's just my theory of the problem right now, and this is a workaround rather than addressing the root cause! :)
Another Example of this Issue
The same issue is present for IoT Provisioning Template tags - the Tags type is an object with a Tags property that's then a list:
Resource definition:

https://github.com/aws-cloudformation/aws-cloudformation-resource-providers-iot/blob/master/aws-iot-provisioningtemplate/aws-iot-provisioningtemplate.json#L63

Documentation:

https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-iot-provisioningtemplate.html#cfn-iot-provisioningtemplate-tags
https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-properties-iot-provisioningtemplate-tags.html

A Correctly Generated Example
Resource definition:

https://github.com/aws-cloudformation/aws-cloudformation-resource-providers-acmpca/blob/e3c591cddff371786b4dacaaef1d02b938b121c7/aws-acmpca-certificateauthority/aws-acmpca-certificateauthority.json#L124

Documentation:

https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-acmpca-certificateauthority.html#cfn-acmpca-certificateauthority-tags

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

I think the underlying issue should be fixed now thanks to https://github.com/aws/aws-cdk/issues/11402#issuecomment-771398728.

Verified the root-cause fix, so this isn't needed :) Thanks!


New examples

#177

by the way @lulldev you aren't showing up in the contributors list most likely because your email isn't added to your github account. You should add the one associated with this commit https://github.com/a-synchronous/rubico/commit/ff1ec6167947268f9f5a6540c64d068f77bbcde3.patch

@richytong about esm. can be more?

also about your email I think you should look into making the one in your commits match up to your actual email. This is done through creating a new gpg key and adding it to your github gpg keys. This is a pretty good guide for it<EMAIL_ADDRESS>


Certain faces cannot be inspected

In this example, the cylindrical faces cannot be inspected.
t1 = SlotOverall(10, 5)
t2 = Pos(Z=2) * SlotOverall(8, 3)
t = loft([t1, t2])
t = offset(t, 1)

Note to myself:
According to the backend Terminal output:
response.radius = shape._geom_adaptor().Cylinder().Radius()
                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'OCP.Geom.Geom_RectangularTrimmedSurface' object has no attribute 'Cylinder'

check out which results shape._geom_adaptor() could have and handle them

https://dev.opencascade.org/doc/refman/html/classGeom__Surface.html

Fixed in 2.6.1


allow better typechecks by setting a max-recursion depth

it's been said many times that certain features cannot be implemented properly due to hitting infinite recursion, and because of that all sort of shortcuts been taken to keep performance acceptable
consider adding a max-depth parameter to the compilerOptions which controls how deep the type checker can go
for developers who don't care about thorough type checks the depth can be 0
others can set it to any comfortable number which gives acceptable results in acceptable time

Very few people could use this flag usefully and many more people would set it to nonsense values and feel that the results were unexpected.


Add bundle size & module badges, contributor guide, and code of conduct

Same as #634, cherry-picked on top of master
This is due to an advise from Sean Larkin: https://twitter.com/TheLarkInn/status/847930416749092864
It adds a badge for the bundle size, gzipped bundle size, a brief contributor guide and the Contributor Covenant code of conduct.

@geelen 🙌


LightDM: No such interface 'org.freedesktop.DisplayManager.AccountsService'

Issue description
LightDM seems not installing dbus interface of AccountsService.
Systemd journal:
lightdm[1335]: Error updating user /org/freedesktop/Accounts/User1000: GDBus.Error:org.freedesktop.DBus.Error.InvalidArgs: No such interface 'org.freedesktop.DisplayManager.AccountsService'

In lightdm package, it is contained:
% nix build -f '<nixpkgs>' lightdm
% ls result/share/dbus-1/interfaces | grep -i accounts
org.freedesktop.DisplayManager.AccountsService.xml

But in the current system:
% ls /run/current-system/sw/share/dbus-1/interfaces | grep -i accounts
org.freedesktop.Accounts.User.xml@
org.freedesktop.Accounts.xml@

Steps to reproduce
services.xserver.displayManager.lightdm.enable = true;

Technical details

system: "x86_64-linux"
host os: Linux 4.18.0, NixOS, 18.09.git.8483983 (Jellyfish)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.0.4
channels(root): "nixos-18.09pre148970.b5b11e9da43"
channels(mnacamura): "nixpkgs-18.09pre148461.bf1b50cbc8f"
nixpkgs: /home/mnacamura/repos/nixpkgs

Hmm, apparently it is some kind of extension to accountsservice: https://github.com/CanonicalLtd/lightdm/commit/6015bce25f241e7580c03594d846769f8236232f
Weird that it is not linked – the lightdm module contains services.dbus.packages = [ lightdm ].

Actually, the dbus module does not use /run/current-system/sw/share/dbus-1/interfaces, it creates a configuration file:
https://github.com/NixOS/nixpkgs/blob/cc288eef072c5feb6dd74740799a1349ef8fa58e/nixos/modules/services/system/dbus.nix#L13
The directory is probably just a side effect of something adding /share to environment.pathsToLink.

Ah I see. So, the configDir is put in /etc:
https://github.com/NixOS/nixpkgs/blob/cc288eef072c5feb6dd74740799a1349ef8fa58e/nixos/modules/services/system/dbus.nix#L69-L72
I examined /etc/dbus-1/*.conf:
% ls /etc/dbus-1/
session.conf  system.conf

% cat /etc/dbus-1/* | perl -pe 's/(\/?<\w+>)/\n$1/g' | grep -i lightdm
<servicedir>/nix/store/sx3mcl8cb2rk18imz9x740fixpvk4zkn-lightdm-1.26.0/share/dbus-1/services</servicedir>
<includedir>/nix/store/sx3mcl8cb2rk18imz9x740fixpvk4zkn-lightdm-1.26.0/etc/dbus-1/session.d</includedir>
<servicedir>/nix/store/sx3mcl8cb2rk18imz9x740fixpvk4zkn-lightdm-1.26.0/share/dbus-1/system-services</servicedir>
<includedir>/nix/store/sx3mcl8cb2rk18imz9x740fixpvk4zkn-lightdm-1.26.0/etc/dbus-1/system.d</includedir>
<includedir>/nix/store/sx3mcl8cb2rk18imz9x740fixpvk4zkn-lightdm-1.26.0/share/dbus-1/system.d</includedir>

I guess lightdm 's $out/share/dbus-1/interfaces should also be included in these paths?

% nix-shell -p dbus pkgconfig
[nix-shell:~]$ pkg-config --variable interfaces_dir dbus-1
/etc/dbus-1/interfaces

Probably, interfaces files should be linked in /etc/dbus-1/interfaces?

No idea – I thought the interface files were only for introspection or something. I tried linking to /run/current-system by adding it to environment.systemPackages but that does not seem to help. You can try the /etc route but I do not see it documented anywhere. Though, I do not see the one in data dirs documented either – perhaps it is really just for applications like d-feet. But the data should be available through dbus interface so I do not see the need for installing the XML files.
On a side note, accountsservice looks for the interface symlinks in $XDG_DATA_DIRS/accountsservice, so something like the following might be good:
--- a/nixos/modules/services/desktops/accountsservice.nix
+++ b/nixos/modules/services/desktops/accountsservice.nix
@@ -43,6 +43,9 @@
     } // (mkIf (!config.users.mutableUsers) {
       environment.NIXOS_USERS_PURE = "true";
     });
+
+    # Extensions
+    environment.pathsToLink = [ "/share/accountsservice" ];
   };

 }

Thanks for suggestion. I tried this:
    environment.pathsToLink = [ "/share/accountsservice" ];
    environment.systemPackages = [ pkgs.lightdm ];

Apparently the interface file appears,
readlink -f /run/current-system/sw/share/accountsservice/interfaces/org.freedesktop.DisplayManager.AccountsService.xml
/nix/store/sx3mcl8cb2rk18imz9x740fixpvk4zkn-lightdm-1.26.0/share/dbus-1/interfaces/org.freedesktop.DisplayManager.AccountsService.xml

but I still see the error.

Hopefully fixed by the above PR.


codespell: add action, pre-commit, config and make it fix some typos

you asked in email to contribute -- here is one ;-)

Thank you for your contribution.
It looks good to me.
@joaoandre-avaiga  Can you have a look at it as well to ensure it fits with the refactoring of the CI/CD you are currently working on?
Thx

Thank you for the contribution! For sure, that looks good for me.

@yarikoptic We are in the middle of a repository merge and a deep refactoring of the Taipy CI/CD.
The tests are failing (in your PR case because of timeout), for external reasons. We are currently working on it.
I propose to merge your PR now even if the test GitHub action is not repaired yet, so we can benefit from your contribution in the new CI/CD.
Thank you.


pbasex

I came across a nice python implementation of the pbasex algorithm:
  https://github.com/e-champenois/CPBASEX

Modifying the provided CPBASEX/examples/sample_pbasex.py provides this O2-ANU1024.txt Abel transform comparison with PyAbel (basex):

test-pbasex.py
There may be a case to work with Elio Champenois  to:

integrate a version of pbasex into PyAbel
improve the algorithm (or parameter option selection) to give a more accurate transform. The rough O2-ANU1024.txt image comparison indicates less resolution, peak intensity, and baseline issues. In particular, the intensity required a xR scaling to match PyAbel, which may even be PyAbel's fault, see #253.

Thoughts?

Thanks for finding this @stggh! It would be nice to have a pBasex implementation in PyAbel.
With any new algorithm, we would want to re-make all of the figures from the paper and see how the algorithm compares with the others. In this case, it looks like the resolution is much worse, but perhaps this is simply the basis set.
Please go ahead and contact Elio (if he doesn't already see this thread) and see if he is interested.

Hi Dan and Stephen,
Thanks for the message. Feel free to use the pbasex code I developed however you wish, including it in your package if you want. I can of course help out or at least go over what I was thinking when I was implementing this, let me know! I'd be curious to hear more about all these Abel inversion algorithms too, I hadn't considered many of these before diving into pBASEX, since the "pros" made a lot of sense to me at the time.
In terms of the example above, I see two immediate issues:

the basis set looks a bit sparse
the baseline issue is due to a transpose issue: beta_2 = -1 and beta_2=2 aren't transposes of each other, so the orientation of the image matters with pBASEX.

Here's what I got with your sample image, using 256 radial basis functions (512 total, up to l=2).
I'm sure it could be improved with more tuning of the basis set (number of functions, width of Gaussians, etc.) or other tricks (better centering, l1 regularization, weighting errors, masking weird center noise in the image), but I tried to go with an "out of the box" approach here.

With l2-regularization:

Thanks @e-champenois! Your calculation looks much better. I will attempt to implement pbasex into PyAbel.

I will attempt to implement pbasex into PyAbel.

After #263, please! :–)
... so that I can finish my stuff...

(this is without any regularization)
time in milliseconds

=== Basis generation ==========================================

Method              n = 513    n = 1025    n = 2047    n = 4097
---------------------------------------------------------------
rbasex                114.3       208.7       536.9      2057.1
two_point               3.2        13.9        50.8       192.7

=== Inverse Abel transform ====================================

Method              n = 513    n = 1025    n = 2047    n = 4097
---------------------------------------------------------------
rbasex                  5.4        21.9        83.2       296.3
two_point               1.4        11.3        79.3       671.5

Fair enough!

Any update about the implementation of pBasex in PyAbel?
cheers!

My apologies, there is much less free time in retirement than I imagined. I have just returned from sailing in the Australian Whitsunday Islands. I will try to get onto #263 and this task soon!
In the mean time, pbasex is compatible with PyAbel, see my example code above.

@stggh, regarding #263, I'd suggest to work on it two steps: first focusing on the (x, y)/(row, col) issues, and then do the center/origin renaming (which I can do myself, if you don't have time).

which I can do myself, if you don't have time

Sorry to have blocked progress on this for so long. Yes, if you have the time, please start the changes. At the moment I am tied up with documentation, in preparation for a laser safety verification of my lab.

I actually wrote to Gustavo on Monday and got the source code from him. It is supposed to be compiled by the users and works only in Unix-like systems (he said that other people have created other versions for Windows). I compiled it in Linux without any issues and ran some tests. Yesterday I wrote back to Gustavo thank him and ask some remaining questions, so I'll report my findings here after he replies. But unfortunately my suspicion about the disk/memory requirements turned out to be correct, and the slowness of the basis generation actually exceeded my expectations...

Now Gustavo wrote that currently he actually uses another implementation, coded in Igor Pro, “with a theta sampling which depends on the radius, which is more efficient and logical”. But since I do not have it yet, here is my summary about the original C implementation.
First, the answers to my questions above:

The projected basis functions are indeed created in the RAM. They are not saved to the disk, but instead the SVD is done in place, and the results are then saved. (See below for the memory consumption). The reconstructed-image basis functions (radial Gaussians × angular Legendre polynomials) are computed on the fly (withing about ±5σ from their peaks), so they do not take any extra memory, but require exp() evaluation several times per pixel.

The basis parameters are adjustable, but only through editing the source code and recompilation (and then making sure that the saved files match the executable). The number of angular terms is adjustable at run time, but every lmax requires its own basis set.

Yes, the matrices will be huge. For some reason the program uses float (32-bit) numbers for some calculations, but the matrices have double (64-bit) elements, so for the default settings with a 256 × 256 resampled image, 128 radial and 2 angular basis functions the U matrix has the size 256 × 256 × 128 × 2 × 8 bytes = 128 MiB (“binary megabytes”) ≈ 134 MB (“decimal megabytes”). Examples for some other sizes are shown below.

The program does not modify the singular values at all. The basis-creation part gets them from a library SVD function, which produces M = rmax × lmax singular values (all non-zero, since G is a full-rank matrix), and saves them to one of the basis files. The image-transform part then reads them from that file and passes directly to the library SVD solver. Gustavo wrote: “If you want, you can remove other singular values under a given threshold by setting them to zero”, but the original pBasex implementation definitely does not, and it's not clear whether other implementations do.

Some timing/size tests:
Here NR = radial bins, NTH = angular bins, NK = radial basis functions = NR / 2 (recommended); NL = angular functions = 2 (default) in all cases.
The basis generation has two time-consuming steps: the Abel integration of the basis functions (“basis”) and the singular-value decomposition of the resulting matrix (“SVD”). The decomposition results are saved to separate files for U, V and S; their sizes are added in the table.

NR(NK)
NTH

128
256
512

128(64)
6 min basis,8 s SVD.16 MB + 128 KB + 1 KB
12 min basis,17 s SVD.32 MB + 128 KB + 1 KB

256(128)

31 min basis,2.5 min SVD.128 MB + 512 KB + 2 KB

512(256)

88 min basis,20 min SVD.512 MB + 2 MB + 4 KB

3 h basis,39 min SVD.1 GB + 2 MB + 4 KB

So the basis size and generation time scale linearly with NR × NTH × NK (3rd power of the image radius), and the SVD time scales as NR × NTH × NK2 (4th power!).
I did not try to create a narrow basis set with NK = NR = 512, which is required to analyze 1-megapixel images with 1-pixel resolution, but it apparently will take >9 h and >2 GB.
The transform timings for a 1 Mpx image are (I have slightly modified the code to print these metrics; the last “Done in ...” is printed by the original code):
Symmetrization: 0.00082 s
To polar: 0.016849 s
SVD: 0.011849 s
PES: 8.6e-05 s
Image: 0.211069 s
Done in 0.241049 seconds

for the default settings (256 × 256 resampled image, 128 functions) and
Symmetrization: 0.000784 s
To polar: 0.060347 s
SVD: 0.094387 s
PES: 0.00012 s
Image: 0.208495 s
Done in 0.365381 seconds

for the 512 × 512 resampled image and 256 functions. Not slow, but not very fast either.
Here I should make a note about the performance of this original implementation. First, its code is not optimized at all, except that it does the left–right symmetrization of the images (but never does the top–bottom symmetrization, even when odd orders are not used) and works with only one half. Otherwise everything is done quite inefficiently (for example, each Legendre polynomial is evaluated at each integration point anew through a recurrent formula, with memory allocation/deallocation...). Second, for all nontrivial calculations it uses the GNU Scientific Library (including the use of its general-purpose integration function to compute the basis projections). The GSL is probably not the most efficient and runs in a single thread (although all the heavy computations can be efficiently parallelized).
I guess that optimizing the computations and using more efficient linear-algebra libraries could improve the timings by a factor of 10–100. The memory usage can be reduced by a factor of 2 (using the top–bottom symmetry) at most. But the time and memory scalings with the image size are still worse than for all other methods, meaning that even with the best optimizations the pBasex method cannot be practically used for high-resolution transforms.
Finally, some additional remarks about this original pBasex implementation and the pBasex article itself.

The Cartesian-to-polar resampling is done such that when the image radius is large enough, the NTH angular samples actually skip some image pixels completely. This means that not all the available data is used, and the signal-to-noise ratio at large radii is worse than it could be with proper resampling. Strangely enough, the documentation does not recommend increasing NTH. (I also have an impression that the “bicubic interpolation” is implemented incorrectly.) An implementation with proper rebinning should work better.

The comparison with the BASEX results for odd angular orders, where pBasex is claimed to perform significantly better than BASEX, is not really correct. The problem is that the original BASEX program replaces all negative intensities in the output image with zeros (a poor decision!), so when the pBasex authors were extracting radial distributions from that output image, they were integrating all the positive noise (present in the image) without integrating the counterweighting negative noise (removed from the image), and the results were of course badly biased. The BASEX program itself calculates radial distributions without this zeroing. The original pBasex program also does correct calculations internally, but has the same flaw that it zeroes all negative intensities in the output. Thus while the pBasex method indeed produces cleaner reconstructed images, I do not think that its radial distributions are actually better than those obtained by other linear transform methods.

@DanHickstein,

Perhaps the assumption is that large basis sets won't need to be used to describe most images...

It depends on the expectations for what “most” means. :–)
I have shown earlier an example of the 0.3% kinetic-energy resolution (FWHM) in my setup (which was not the first to achieve this), which means the radial resolution ~0.15% and hence at least ~700 distinct radial values, so in practice >1000 basis functions are needed to represent this adequately.
But on the other hand, in many experiments the resolution is indeed much worse and cannot be improved by reasonably better instruments. In these cases a low-resolution method will be enough.

@MikhailRyazanov: That's great that you were able to make contact with Gustavo and awesome that he was kind enough to provide the original source code.
Your analysis of the pBASEX code seem quite insightful!
It seems that, generally speaking, the scaling of the computation time with the size of the image is not especially favorable. How does your rBasex method provide similar results with better scaling?
Your final two points (about the cartesian-to-polar conversion and setting negative values to zero) are perfect examples of how confusion can arise when trying to compare Abel transform methods implemented in different software!
So, have you reached a conclusion regarding if/how we should incorporate pBasex? Or is more study needed?

How does your rBasex method provide similar results with better scaling?

It employs the separability into radial and angular parts (which does not work with Legendre polynomials, but works with cosine powers). So instead of solving the huge problem that connects every pixel to every basis function (where these huge matrices appear), it is splits into two subproblems: 1) get the radial distributions for each angular part (a standard procedure, linear in the number of pixels = quadratic in the radius), 2) transform each of these radial distributions (they are 1-dimensional, so it is a matrix-by-vector multiplication, also quadratic in the radius). The 1st subproblem is as overdetermined as the “huge problem”, but has an explicit solution, without the need of the SVD and extra memory. And the 2nd is not overdetermined, so it's just rmax equations for rmax variables (separately for each angular part).
Plus, the basis projections are calculated analytically, which is much faster, and not for each pixel, but just for each radius.

So, have you reached a conclusion regarding if/how we should incorporate pBasex? Or is more study needed?

It can be implemented directly as described in the article (like the original code does). The only tricky part would be to calculate the basis projections efficiently, if we don't want this to take many hours.
Or, as Gustavo wrote, use a different resampling method (I haven't received his new code yet, but my guess is that it's similar to the rasampling in the “polar onion peeling” method).
And, in my opinion, the approach used by @e-champenois (without resampling) is more reasonable. The only problem is that it is intended to transform images of some particular size, so different image sizes require different basis sets to be generated (which is very slow). The original pBasex resampling just takes its samples from any image, and the basis-set size only determines the relative radial resolution, but not the input/output image size.

Not certain if this should be made as a new issue or not, but it is sort of in the vein of this thread.
I have been attempting to use the rbasex implementation to invert some above threshold ionization (ATI) data that requires a very high order. I have found that when the order exceeds 20 (or 21 for odds) their is a sharp change in the noise.
The figure below shows an example of this problem. You can produce the same result with the rbasex "beam block" example. The cutoff is always at order 20 (or 21 for odd). As the order increase the problem only gets worse. If this is just a limit of rBasex then it should probably be mentioned in the documentation that their is a maximum order that is usable.

Can you try other “polar” methods (lin-BASEX, pBasex, POP) to see how they behave with very high orders?
And, before going into computational details, is ATI reasonably described in terms of Legendre polynomials? I have no experience with it, but from the few examples that I've read, its seems to be non-perturbative and involve strong interference between many channels.

@talbert48 Thanks for bringing this to our attention, and for posting the nice data.
I think that we should move this to a separate issue. I opened https://github.com/PyAbel/PyAbel/issues/286 for the discussion.


Remove rollup build project from github

This package is still not quite at a major release so the rollup build product is still in github.

[x] Remove dist and all sub dirs

this has been finished


Add signing capabilities to Travis

Fixes #83
Takes ideas from https://github.com/chrisbanes/insetter
Add gpg signing for release builds.
Uses AES to decrypt the gpg key and deletes it afterwards.

Am I correct in thinking that someone will still have to press release in nexus? If that is the case lgtm

Yes, I will update RELEASING.md after I try to cut a release.
On Sat, Feb 15, 2020, 8:59 AM Mike Nakhimovich<EMAIL_ADDRESS>wrote:

@digitalbuddha approved this pull request.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/dropbox/Store/pull/105?email_source=notifications&email_token=ACCPWM4VESV7IGSMT4EIBSTRDANPJA5CNFSM4KVTCKW2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCVVVE5I#pullrequestreview-359354997,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACCPWM7K6KYB2OWOESPZMXDRDANPJANCNFSM4KVTCKWQ
.


Micro: Add TraceProvider injector.

This allows to inject a TraceProvider into the service constructor,
instead of letting the service constructor create one.

Please rebase to get the docker ceph pipeline green.

Please rebase to get the docker ceph pipeline green.

Thanks for the hint, it now passes! :)


variable argument length

There may just be no way around it. But this issue is for discussion around that.
One thing I lost was the ability to have variable argument-lengths.
For example, with FileInfo.joinPaths.

This has been requested in QTBUG-70677, though without any news since it was opened in 2018. :-/

Bummer. I'm honestly ok with using an array. it seems to work fine, if not minorly annying. I could also see wrapping it in javascript-space:
FileInfo.joinPaths = (...args) => FileInfo._joinPaths

Right, we can execute some JS code as well to fill in those parts. We can resolve the constructor exception the same way and make the call to open automatic again (and rename to _open to indicate it's not meant to be used).

Since I am way stronger with JS, it appeals to me to solve the inter-op issues in JS-space.
This would solve the dynamic args issue, and the no throw in constructor issue.
Should I make a scriptmanager, so the C++ user can do something like this:
ScriptManager manger(jsEngine, ["File", "FileInfo", "Process"]);

It would limit what they were named in JS, but also make it easier to just add what is needed, and since the class would have access to the jsEngine, it could inject js-space wrappers and stuff.
@rweickelt offered a workaround in email. This demonstrates a slightly different approach using qmlRegisterSingletonType with inline javascript-code.

Hmm, to try it out, I did this, inside Qbs4QJS namespace:
class ScriptBinaryFile
{
public:
	ScriptBinaryFile(QJSEngine jsEngine, const QString jsName = "BinaryFile") {
		jsEngine.globalObject().setProperty("BinaryFile", jsEngine.newQMetaObject(&BinaryFile::staticMetaObject));
	}
};

// called like this:
Qbs4QJS::ScriptBinaryFile fbin(jsEngine);

and I get this error:
ERROR: In file included from /usr/include/qt/QtCore/qobject.h:46,
                 from /usr/include/qt/QtCore/QObject:1,
                 from /home/konsumer/Documents/otherdev/Qbs4QJS/default/Qbs4QJS.6f304c2a/qt.headers/../../../binaryfile.h:1,
                 from /home/konsumer/Documents/otherdev/Qbs4QJS/default/Qbs4QJS.6f304c2a/qt.headers/moc_binaryfile.cpp:10:
/usr/include/qt/QtCore/qobjectdefs.h:114:5: error: class definition may not be declared a friend
  114 |     friend Q_DECL_CONSTEXPR const QMetaObject *qt_getEnumMetaObject(ENUM) noexcept { return &staticMetaObject; } \
      |     ^~~~~~

Do I need to do it some other way?

I implemented them, like above, but I can put it back, if it seems like a bad idea. I'm going to try to add the variable length argument stuff, now.

So, I got this working over on #4, then added the dynamic args stuff for FileInfo.joinPaths. It auto-closed, but feel free to re-open, if this seems like a bad solution.


Remote Tunnel Access hangs when connecting from two browser tabs at same time

Insiders on MacOS Big Sur
Version: 1.75.0-insider
Commit: 6d40104789d03d41b8866a1e57847dae14c5cf0d
Date: 2023-01-12T05:36:13.961Z
Electron: 19.1.9
Chromium: 102.0.5005.194
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 20.6.0
Sandboxed: Yes

All on the same machine...
Turn on Remote Access
Open a browser (Edge in this case) instance to connect to server
Leave this browser tab open and turn off Remote Access on the desktop tools
Turn on Remote Access again
Copy the URL
Open a new tab (leaving the existing tab open) and browse to the server

Result: Hangs.  First browser instance will eventually try to reconnect and will hang as well.
In the dev tools console of the browser lots of 404s.

Hard quit VS Code on the desktop
In terminal ps aux | grep vscode-cli

Have to kill one or more of these processes to be able to turn on Remote Access again.

Duplicates https://github.com/microsoft/vscode-remote-release/issues/6543


SearchResult is missing totalPages field

Description
Hi, it seems that the SearchResult class does not contain the totalPages of the Meilisearch search result
At least I could not find it anywhere 😅
We would need this field in order to display the total pages in our frontend.
Expected state
The totalPages can be retrieved via a getter on the SearchResult
Current state
Means to retrieve the totalPages are not present, neither on the SearchResult class nor on the Searchable interface.
Environment (please complete the following information):

Meilisearch version: 1.2
meilisearch-java version: 0.11.5

Hello @matthiasmoke
thanks for the report
Any contribution is welcomed ❤

Update from my side. I found it after all 😄
It's contained in the SearchResultPaginated type. Will close the issue


Google+ API Sunset

With the sunset of the Google+ API, this breaks Watchmen's authentication.

Just in case someone is searching by string, this produces an error like this:
InternalOAuthError: failed to fetch user profile
    at …/node_modules/passport-google-oauth2/lib/oauth2.js:92:28
    at passBackControl (…/node_modules/oauth/lib/oauth2.js:132:9)
    at IncomingMessage.<anonymous> (…/node_modules/oauth/lib/oauth2.js:157:7)
    at IncomingMessage.emit (events.js:187:15)
    at endReadableNT (_stream_readable.js:1081:12)
    at process._tickCallback (internal/process/next_tick.js:63:19)


Allow down migrations

Currently up migrations are run when neoq starts. In order to roll back to previous releases, users need to be able to "roll back" their neoq_jobs schema to previous versions.
Currently, there is no way to specify which schema version the the current version of neoq should use.

Since you're embedding the migrations at build time, you could determine what the "latest" migration version is based on the files in that embed. If the version in the DB is greater than that you could start with raising a log message at the very least. But for an older version to know how to downgrade from a migration it doesn't have might be tricky.

I wonder if this issue can be addressed with a release policy that mandates 2-phase release for all migrations.

The down migration is released
Followed by the release containing the up migration

Obviously not perfect, but it allows the existing migration tooling to be used. Maybe it's a start of something.

Closing since migrations change so infrquently


Picture set refactoring

Coverage decreased (-34.4%) to 62.025% when pulling e5c2559c48843cf719813740fca8460d331bb554 on picture_set_refactoring into 67a416ded824433156fbc44fdf08f66aa466132b on master.


[Feature]: Add endpoint to track file-size of uploads that are usedForTreasuryExport

Why is this issue important?
A lot of the reasons why we need run into memory issues when generating audit reports or treasury exports is due to the upload-file-sizes. Having an easy way for us to find out the overall file-sizes for all uploads will be helpful to debug memory issues.
Current State
This feature does not exist. Today in order to find out the filesizes a USDR staff engineer has to SSH into the ECS task and list files in /var/data EFS drive.
Expected State
When an admin user hits this endpoint GET /api/uploads/metadata/used_for_treasury_export
they will receive the following information
{
    files: [{
         id: <uuid>,
         filename: <string>,
         filesize: <number>,
    }],
    aggregate: {
         total_files_included_in_export: <number>,
         total_filesize_of_all_uploads_in_export: <number>,
    }
{

Implementation Plan
Note: the goal of this ticket is to implement something quickly that will aid us in debugging when our partners run into issues. This does not require us to have any UX consideration and it is not something that should require more than a half-day effort.
Option 1

Store filesize in the uploads table and use this information
Add this filesize attribute and have it be exposed in the usual /api/reporting_periods/:periodId/exported_uploads endpoint: https://github.com/usdigitalresponse/usdr-gost/blob/_staging/packages/server/src/arpa_reporter/routes/reporting-periods.js#L142

Option 2

Create a new endpoint as suggested in the expected state
Get all the file records in memory by querying the database using usedForTreasuryExport
For each record check the filesystem for its filesize
Compile the results in json schema as shown above

Whatever ships this feature fastest is the right solution here. It is not information we intend to expose to our partners anytime soon. If there is a slightly different implementation that gets us the same results then that is great too 👍
Relevant Code Snippets
No response

This might be something we can put directly into datadog as datadog metrics rather than an endpoint.


Added AbortReason logging

Pull Request Test Coverage Report for Build 1710

35 of 38 (92.11%)  changed or added relevant lines in 4 files are covered.
8 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.9%) to 77.6%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Actions/Helpers.ts
16
17
94.12%

src/Actions/Actions.ts
1
3
33.33%

Files with Coverage Reduction
New Missed Lines
%

src/Actions/Pending.ts
2
78.95%

src/EconomicStrategy/EconomicStrategyHelpers.ts
6
77.14%

Totals

Change from base Build 1704:
0.9%

Covered Lines:
642

Relevant Lines:
800

💛  - Coveralls


bug fix proposed

sometime locals var in current state is null. the code below throws exception:
function updateView(firstTime) {
var newScope,
name            = getUiViewName(scope, attrs, $element, $interpolate),
previousLocals  = name && $state.$current && $state.$current.locals[name];
if (!firstTime && previousLocals === latestLocals) return; // nothing to do
newScope = scope.$new();
latestLocals = $state.$current.locals[name];
PROPOSED CODE:
// BUG FIX
var newScope,
name = getUiViewName(scope, attrs, $element, $interpolate);
var previousLocals = null;
if (name && $state.$current && $state.$current.locals)
previousLocals = name && $state.$current && $state.$current.locals[name];
      if (!firstTime && previousLocals === latestLocals) return; // nothing to do
      newScope = scope.$new();
        // BUG FIX
      if ($state.$current.locals)
        latestLocals = $state.$current.locals[name];

another place where locals are referenced without checking for null is:
var current = $state.$current,
name = getUiViewName(scope, attrs, $element, $interpolate),
locals  = current && current.locals[name];
and fix
    var current = $state.$current,
        name = getUiViewName(scope, attrs, $element, $interpolate);
    var locals = null;
    if (current && current.locals)
        locals = current && current.locals[name];

    if (! locals) {
      return;
    }

Interesting, I don't think we've ever had this come up. Can you post an example that demonstrates how the issue is triggered? Thanks.

Yep, I tried to isolate example, i can share in private code for entire file and do skype session, otherwise i have to put entire app project:)

http://plnkr.co/edit/yXEWItLQBxSEKzOwLsfj?p=preview
what I did is only add this line:
$stateProvider.state({ name: 'redir', url: '/redir', controller: function() { }, onEnter: function($state) {$state.transitionTo("home");}});

choose 'home.foo' from the 'go to state' dropdown then choose "redir", result in chrome:
TypeError: Cannot read property 'globals' of null

firebug traceback:
Error: to.navigable.locals is null
transitionTo/$state.transition<@https://rawgit.com/angular-ui/ui-router/0.2.13/release/angular-ui-router.js:3198:11
qFactory/defer/deferred.promise.then/wrappedCallback@https://code.angularjs.org/1.2.25/angular.js:11572:31
qFactory/defer/deferred.promise.then/wrappedCallback@https://code.angularjs.org/1.2.25/angular.js:11572:31
qFactory/ref/<.then/<@https://code.angularjs.org/1.2.25/angular.js:11658:26
$RootScopeProvider/this.$get</Scope.prototype.$eval@https://code.angularjs.org/1.2.25/angular.js:12701:16
$RootScopeProvider/this.$get</Scope.prototype.$digest@https://code.angularjs.org/1.2.25/angular.js:12513:15
$RootScopeProvider/this.$get</Scope.prototype.$apply@https://code.angularjs.org/1.2.25/angular.js:12805:13
setupAsOptions/<@https://code.angularjs.org/1.2.25/angular.js:21683:11
createEventHandler/eventHandler/<@https://code.angularjs.org/1.2.25/angular.js:2853:7
forEach@https://code.angularjs.org/1.2.25/angular.js:325:9
createEventHandler/eventHandler@https://code.angularjs.org/1.2.25/angular.js:2852:5

https://code.angularjs.org/1.2.25/angular.js
Line 10071

or choose 'redir' and then 'home.foo', result in chrome:
TypeError: Cannot read property '@' of null

firebug traceback:
Error: $state.$current.locals is null
updateView@https://rawgit.com/angular-ui/ui-router/0.2.13/release/angular-ui-router.js:3833:15
$ViewDirective/directive.compile/</<@https://rawgit.com/angular-ui/ui-router/0.2.13/release/angular-ui-router.js:3804:11
$RootScopeProvider/this.$get</Scope.prototype.$broadcast@https://code.angularjs.org/1.2.25/angular.js:12980:15
load@https://rawgit.com/angular-ui/ui-router/0.2.13/release/angular-ui-router.js:3558:11
resolveState/</injectables.$template<@https://rawgit.com/angular-ui/ui-router/0.2.13/release/angular-ui-router.js:3458:18
invoke@https://code.angularjs.org/1.2.25/angular.js:3966:14
proceed@https://rawgit.com/angular-ui/ui-router/0.2.13/release/angular-ui-router.js:474:32
invoke@https://rawgit.com/angular-ui/ui-router/0.2.13/release/angular-ui-router.js:470:26
$Resolve/this.study/<@https://rawgit.com/angular-ui/ui-router/0.2.13/release/angular-ui-router.js:449:14
$Resolve/this.resolve@https://rawgit.com/angular-ui/ui-router/0.2.13/release/angular-ui-router.js:553:12
resolveState/<@https://rawgit.com/angular-ui/ui-router/0.2.13/release/angular-ui-router.js:3461:23
forEach@https://code.angularjs.org/1.2.25/angular.js:332:11
resolveState@https://rawgit.com/angular-ui/ui-router/0.2.13/release/angular-ui-router.js:3455:7
transitionTo@https://rawgit.com/angular-ui/ui-router/0.2.13/release/angular-ui-router.js:3157:20
go@https://rawgit.com/angular-ui/ui-router/0.2.13/release/angular-ui-router.js:3007:14
Parser.prototype.functionCall/<@https://code.angularjs.org/1.2.25/angular.js:10846:15
Parser.prototype.statements/<@https://code.angularjs.org/1.2.25/angular.js:10621:29
$RootScopeProvider/this.$get</Scope.prototype.$eval@https://code.angularjs.org/1.2.25/angular.js:12701:16
ngChangeDirective<.link/<@https://code.angularjs.org/1.2.25/angular.js:17974:7
NgModelController</this.$setViewValue/<@https://code.angularjs.org/1.2.25/angular.js:17752:11
forEach@https://code.angularjs.org/1.2.25/angular.js:325:9
NgModelController</this.$setViewValue@https://code.angularjs.org/1.2.25/angular.js:17750:7
setupAsOptions/</<@https://code.angularjs.org/1.2.25/angular.js:21735:13
$RootScopeProvider/this.$get</Scope.prototype.$eval@https://code.angularjs.org/1.2.25/angular.js:12701:16
$RootScopeProvider/this.$get</Scope.prototype.$apply@https://code.angularjs.org/1.2.25/angular.js:12799:18
setupAsOptions/<@https://code.angularjs.org/1.2.25/angular.js:21683:11
createEventHandler/eventHandler/<@https://code.angularjs.org/1.2.25/angular.js:2853:7
forEach@https://code.angularjs.org/1.2.25/angular.js:325:9
createEventHandler/eventHandler@https://code.angularjs.org/1.2.25/angular.js:2852:5

https://code.angularjs.org/1.2.25/angular.js
Line 10071

the issue seems to exist with angular 1.4 and/or ui-router 0.2.14 as well
https://github.com/angular-ui/ui-router/issues/1833 looks similar

i'm currently using a workaround something like this:
$rootScope.$on('$stateChangeStart', function(event, toState, toParams, fromState, fromParams) {
  if (toState.name === "redir") {
    $state.transitionTo("home");
    event.preventDefault();
  }
});

just stumbled upon this problem. I created a route that the sole purpose is to do a redirection, inside onEnter. always give the $state.$current.locals is null

@nateabele Here's a plunkr. I've experienced the same issue.
http://plnkr.co/edit/XjYN14oiBEu5cDVrnA2R?
I found some way to fix it, as well as what i believe is the cause of the issue, but I'm not very familiar with ui-router internals, so I haven't submitted a PR or anything.
var transition = $state.transition = resolved.then(function () {
    var l, entering, exiting;

    if ($state.transition !== transition) return TransitionSuperseded;

    // Exit 'from' states not kept
    for (l = fromPath.length - 1; l >= keep; l--) {
        exiting = fromPath[l];
        if (exiting.self.onExit) {
            $injector.invoke(exiting.self.onExit, exiting.self, exiting.locals.globals);
        }
        exiting.locals = null; //THIS LINE
    }

    // Enter 'to' states not kept
    for (l = keep; l < toPath.length; l++) {
        entering = toPath[l];
        entering.locals = toLocals[l];
        if (entering.self.onEnter) {
            $injector.invoke(entering.self.onEnter, entering.self, entering.locals.globals);
        }
    }

    ....

If you then start another transition inside an onEnter callback, exiting locals have already been blown up, but weren't set to the values from the intermediate state yet, and I assume these are referenced by the global $state, because that then seems to get nulled.
Removing that line from the loop there and adding another loop over fromPath down immediately after all onEnter callbacks are fired, and nulling the exiting locals here, seems to fix this issue, but I wasn't sure what side effects this might have. Doesn't seem to have broken anything for my project though.
In the plunkr, I don't know why three clicks are required. I'd expect one to maybe work because I haven't initialized any state, and then the second click would trigger the error. It doesn't, and instead a third is required. In my project, every transition to a state after initialization/bootstrapping is done initiated from inside an onEnter callback causes the error to occur.

Any Update on this issue?
Facing same issue when I am using onEnter callback. Looks like $state.$current.locals is not initialized when '$viewContentLoading' event is triggered.

Same here.
in the onEnter, getting $state.current doesn't give me the current state we are going to...

Why are you trying to use $state.$current instead of $state.current?

@eddiemonge I could be wrong but I don't believe the issue is with end users using $state.$current,  not sure if you noticed the example to reproduce http://plnkr.co/edit/XjYN14oiBEu5cDVrnA2R?

its not, but it was interesting to me all the same

Is there plans to fix this ? We have this issue multiple times on several routes/views in our app in production and we're a bit stuck...

Is there any updates on this issue? I had face the same problem when trying to use $state.go(...) in onEnter callback. Now I use $timeout to workaround with this problem.
onEnter: ["$state", "$timeout", function($state, $timeout){
    $timeout(function(){
        $state.go(...);
    });
}]

It works but the code is not beautiful.

I have same issue so I am interested in final solution too.

+1

I got the same issue with this code:
 86         $stateProvider.state('auth.logout', {
 87             url: '/logout',
 88             template: 'Logout!',
 89             onEnter: ['$state', '$auth', '$timeout', function($state, $authProvider) {
 90                     $authProvider.logout();
 91                     $state.go('app.index');
 92             }],
 93         });

Changing the "onEnter" to:
 89             onEnter: ['$state', '$auth', '$timeout', function($state, $authProvider, $timeout) {
 90                     $authProvider.logout();
 91                     $timeout(function() {
 92                         $state.go('app.index');
 93                     });
 94             }],

...stopped the error appearing.
As @eldereal has mentioned, it works but it's not entirely elegant (or obvious why the $timeout is there).

This should be fixed by the redirect support in the 1.0 version

This issue was fixed for me, I had UI-router and another ui-router bundled in with ionic :/ removed ui-router and everything resolved!

I was experiencing the same issues mentioned here, but they went away after I upgraded from 0.2.15 to 0.2.18.

I have the same issue with ionic 1.3.2 and ui-router 0.3.2. I double check the config, and I am using only one ui-router.
Here is the stacktrace:

angular.js:14110 TypeError: Cannot read property 'name' of null
    at getUiViewName (angular-ui-router.js:4152)
    at updateView (angular-ui-router.js:4035)
    at angular-ui-router.js:4005
    at Scope.$broadcast (angular.js:18239)
    at angular-ui-router.js:3369
    at processQueue (angular.js:16606)
    at angular.js:16622
    at Scope.$eval (angular.js:17913)
    at Scope.$digest (angular.js:17727)
    at Scope.$apply (angular.js:18021)

Thanks for your help!

It looks like it is related to ionic:
self.init = function() {
var navViewName = $attrs.name || '';
// Find the details of the parent view directive (if any) and use it
 // to derive our own qualified view name, then hang our own details
 // off the DOM so child directives can find it.
 var parent = $element.parent().inheritedData('$uiView');
 var parentViewName = ((parent && parent.state) ? parent.state.name : '');
 if (navViewName.indexOf('@') < 0) navViewName = navViewName + '@' + parentViewName;

 var viewData = { name: navViewName, state: null };
 $element.data('$uiView', viewData);

So I think it is cased by this line
https://github.com/driftyco/ionic/blob/1.x/js/angular/controller/navViewController.js#L44
Am I right?
Cheers,
Andrej

Any news here?


ATS9: Crash on ProxySession::do_api_callout()

[ 0  ] libpthread-2.17.so     waitpid                                                      ( :undefined               )
[ 1  ] traffic_server         crash_logger_invoke(int, siginfo_t*, void*)                  ( Crash.cc:168             )
[ 2  ] libpthread-2.17.so     0x7fc6cfa2f5f0                                               ( :undefined               )
[ 3  ]                       0x0                                                          ( :undefined               )
[ 4  ] traffic_server         ProxySession::do_api_callout(TSHttpHookID)                   ( ProxySession.cc:171      )
[ 5  ] traffic_server         Http1ClientSession::destroy()                                ( Http1ClientSession.cc:79 )
[ 6  ] traffic_server         HttpSM::kill_this()                                          ( HttpSM.cc:6946           )
[ 7  ] traffic_server         HttpSM::state_api_callback(int, void*)                       ( HttpSM.cc:1351           )
[ 8  ] traffic_server         TSHttpTxnReenable                                            ( InkAPI.cc:6096           )
[ 9  ] authproxy.so (deleted) 0x7fc692d66878                                               ( :undefined               )
[ 10 ] authproxy.so (deleted) 0x7fc692d65d15                                               ( :undefined               )
[ 11 ] traffic_server         INKContInternal::handle_event(int, void*)                    ( InkAPI.cc:1101           )
[ 12 ] traffic_server         PluginVC::process_read_side(bool)                            ( I_Continuation.h:190     )
[ 13 ] traffic_server         PluginVC::process_write_side(bool)                           ( PluginVC.cc:569          )
[ 14 ] traffic_server         PluginVC::main_handler(int, void*)                           ( PluginVC.cc:225          )
[ 15 ] traffic_server         EThread::process_event(Event*, int)                          ( I_Continuation.h:190     )
[ 16 ] traffic_server         EThread::process_queue(Queue<Event, Event::Link_link>*, i... ( UnixEThread.cc:176       )
[ 17 ] traffic_server         EThread::execute_regular()                                   ( UnixEThread.cc:236       )
[ 18 ] traffic_server         EThread::execute()                                           ( UnixEThread.cc:345       )
[ 19 ] traffic_server         spawn_thread_internal(void*)                                 ( Thread.cc:92             )
[ 20 ] libpthread-2.17.so     start_thread                                                 ( :undefined               )

(gdb) frame 4
#4  0x000055dc92e8b912 in Http1ClientSession::free (this=0x7fc6178e50c0) at Http1ClientSession.cc:115
115	  this->do_io_write(nullptr, 0, nullptr);

Sounds like the crashes we were seeing that should have been addressed by https://github.com/apache/trafficserver/pull/6469.  But I see that commit is already in the 9.0.x branch.

Mark T. and I looked over our builds, and everything prior to this should be in the build that we see this crasher with:
commit 919b9fa6b3d7366cda181aab0172c32308945360
Author:     Brian Neradt<EMAIL_ADDRESS>AuthorDate: Tue Mar 10 21:31:26 2020 -0500
Commit:     Leif Hedstrom<EMAIL_ADDRESS>CommitDate: Tue Mar 10 20:31:41 2020 -0600

This includes the fixes in #6469 ...

Duplicated with #6527. Closing this because #6527 has more details (symbols, history...)


improve registration results

try to apply Laplacian filter to improve registration of edges.

this function is slowly being deprecated as of v3.2.1


Serve not working in Ubuntu 16.04 64bit

Not sure if this is a known issue -- or an issue with my specific configuration -- but even after installing globally in Ubuntu 16.04 64 via:
npm i serve -g
And then running against files in a static directory (a Vue.js build):
serve ./dist
The terminal displays that the server started, gives the two sets of URLs, and then goes right back to the command line.
No errors -- but it immediately stops serving.
Anyone else see this?
(Just as a sanity check, I installed http-server gloabally via npm, ran it against the same static directory, and all is well. Files are served fine via localhost.)
Thoughts?
PS -- Not that it should matter, but I'm running Ubuntu via VMWare on a Windows box with 4 gigs ram to the vm.
cschweda@ubuntu:~/Webdev/test01$ serve dist

   ┌─────────────────────────────────────────────────────┐
   │                                                     │
   │   Serving!                                          │
   │                                                     │
   │   - Local:            http://localhost:3000         │
   │   - On Your Network:  http://<IP_ADDRESS>:3000   │
   │                                                     │
   │                                                     │
   │                                                     │
   └─────────────────────────────────────────────────────┘

cschweda@ubuntu:~/Webdev/test01$ node -v
v7.5.0
cschweda@ubuntu:~/Webdev/test01$ npm -v
4.1.2
cschweda@ubuntu:~/Webdev/test01$

I can confirm this bug

I tried and had the same bug with
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>For some reason, npm start works on this project. No clue what's different beyond being used as an npm script.

Yep -- this works when run via npm -- but not on the command line.
package.json:
  "start": "serve ./dist --static",

Then:
cschweda@ubuntu$ npm run start

Works as expected.
However:
cschweda@ubuntu$serve ./dist --static

Doesn't work. Simply goes right back to the command line.
cschweda@ubuntu$ serve -v
3.3.1

I can confirm this issue on Void Linux with serve 3.3.1.

Testing this out, it looked like the issue was that when on Linux and xclip isn't installed, the copy-paste package can raise a SIGPIPE when it attempts to copy to the clipboard. The reason there's no error is that the process seems to terminate before the error lister has a chance to fire.
(Still not sure why this isn't a problem when you run serve via $ npm run start...)

Yep. Installed xclip -- and all is well:
cschweda@ubuntu:~$ sudo apt-get install xclip

Then:
cschweda@ubuntu:~$ serve

And everything works fine.

serve -v
3.4.0
Stop working in Linux Mint 17.2 after update.  It just get back to the console with no output at all.

Did you try installing xclip as mentioned above?
sudo apt-get install xclip

Thanks @cschweda yes, that solves the problem sorry.


What should I replace dog.h and squeezenet_params.h with?

Hi, I am using your codes as reference to do my version of CNN, but I faced a problem when compiling the host program. In your host program, there are variables such as sample, conv1_weight used, but I noticed that these are declared in another 2 files, which are dog.h and squeezenet_params.h. Can anyone tell me what should I prepare if I need to do my own CNN for object detection? I want to do the similar thing for SqueezeDet, what are the steps I need to go through?

I have upload on my website [https://github.com/ChenChungKo/squeezenet]


Mock Mode Not Working

With package version "swagger-express-mw": "^0.1.0" mock mode is no longer working with cmd swagger project start -m

My apologies for that. I was already looking into it, so here's a quick fix!

@theganyo  Thank You its working now

Hi @theganyo,
Probably I'm doing something wrong, but clean installation of swagger still not working when running swagger project start -m
As far as I understand, you have updated version of swagger-node-runner, but have not updated version of it in swagger-node.


A user can also link to a referral as part of creating a provisional booking

Changes in this PR
This PR adds a new screen as part of the booking creation journey to allow the user to associate a booking with an assessment
Where a booking is associated with an assessment, we link back to the assessment from the booking page
To note

This cannot be merged until API changes are made - awaiting support for linking bookings to assessments, and filtering assessments from the API by CRN
This feature suggests a rethink of how we approach E2E tests, which is out of scope for now. See this ticket
This feature relies on the applyDisabled flag, which this ticket will likely remove in favour of per-region filtering. Will need to co-ordinate these two changes
The Figma designs show the name of the referral submitter in the referral list. We do not currently get this data from the API. This is covered by these tickets

Screenshots of UI changes
Select assessment

Select assessment with error

Select assessment with no assessments

Select assessments when feature is not enabled for this region

Booking with assessment link

Release checklist
Release process documentation.
Pre-merge checklist

[ ] Are any changes required to dependent services for this change to work?
(eg. CAS API)
[ ] Have they been released to production already?

Post-merge checklist

[ ] Manually approve
release to test
[ ] Manually approve
release to preprod
[ ] Manually approve
release to prod

@tahb @patinipat I'm happy that this is in a good state for review. Still need the OpenAPI changes and to test this against the parallel API changes, but I don't anticipate any great changes to this PR

 A user can also link to a referral as part of creating a provisional booking

PR looks good to me


Disabled nightly tests for straightdope example notebooks

This is a tracking issue for re-enabling all the straight dope nightly notebook tests.

chapter05_recurrent-neural-networks/rnns-gluon. Ref - https://github.com/apache/incubator-mxnet/pull/12326/
chapter08_computer-vision/visual-question-answer Ref - https://github.com/apache/incubator-mxnet/pull/12343

@vishaalkapoor

Thanks @sandeep-krishnamurthy please add me as an assignee.

Sorry, we can't assign to non committers

@sandeep-krishnamurthy @marcoabreu Please feel free to close as https://github.com/apache/incubator-mxnet/pull/12744 addressing this has been merged. Thanks!

@sandeep-krishnamurthy Can you close this issue ?


OCPBUGS-2076: Replace k8s.io/cloud-provider with openshift's version

Service handling logic in k8s.io/cloud-provider v0.25.(0,1,2) is broken. This issue was fixed upstream, but not released yet. We are expecting that this fixes will be included into v0.25.3, but now need to replace released cloud-provider library version with the latest version from release-1.25 branch.
Xref: https://github.com/kubernetes/kubernetes/issues/112793
This commit is expected to be dropped during next rebase.

one of the test failures was related to a node going ready/unready multiple times, but i don't think this is related to the CCM.
/lgtm

/approve

/retest


cached_token

what mean of  cached_token? can you explain it to me in a more general way? l have an erro:no shuch a file :data/coco-train-idxs.p

have you solved the problem? I meet the same.


OCI Artifact manifest

application/vnd.oci.artifact.manifest.v1+json media type definition by OCI https://github.com/opencontainers/image-spec/blob/main/artifact.md

There is another reference by ORAS project
https://github.com/oras-project/artifacts-spec

ORAS Artifact is experimentally supported by Azure container registry https://learn.microsoft.com/ja-jp/azure/container-registry/container-registry-oras-artifacts

OCI Artifact manifest is still in draft, introduced in OCI image spec 1.1.0-rc1

OCI Image spec 1.1.0 has been released https://github.com/opencontainers/image-spec/releases/tag/v1.1.0, and above contents are out-dated https://github.com/containers/oci-spec-rs/issues/144


IMDB list by keyword

Would it be possible to handle IMDB lists by keyword?
I have just tried creating a collection based on https://www.imdb.com/search/keyword/?keywords=disaster-movie, but got an error
| Processing IMDb List: https://www.imdb.com/search/keyword/?keywords=disaster-movie                 |
| Unknown Error: IMDb Error: https://www.imdb.com/search/keyword/?keywords=disaster-movie must begin with either:
| https://www.imdb.com/list/ls (For Lists)
| https://www.imdb.com/search/title/? (For Searches) |

I am slowly finding my way around how to use it and set up everything in the best way possible, and from what I have seen so far IMDB keyword lists seem far superior to related title (or user-curated) lists I have found.

Ah, actually, I have just found out I could use url like https://www.imdb.com/search/title/?keywords=disaster, and it works perfectly fine. Solves my issue. :)


Niklas Wallenborg / fixed scub on star wars text section 2

Not 10 and 11
but fixed scub on star wars text section 2,

@Wallenborg You can merge now


resources/page: Add :slugorfilename attribute

Fixes #4739, #385
It adds a new :slugelsefile attribute that behaves the same way as :slug
with the only difference being that it fallbacks to filename instead of title.
Both issues are closed because they got marked as stale… but I don't see any
point in opening an another issue.

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

Yes, this is reasonable, but:

I suggest we name it :slugorfilename
We need a simple test case for it (there should be a test that can be adapted)

Done.
I now added tests to check if :slugorfilename extends to slug
when set and also added tests for checking if :slugorfilename and
:slug properly fallback to filename and title respectively.

This looks good, thanks.


[CU-2x1wkca] Update location in footer

Background
The location in the footer is still Austin, TX and should be updated to Reading, UK
Changes

Updates location in footer from Austin, TX to Reading, UK

How to test

Ensure the location in the footer reflects the change to "Reading, UK"

Task linked: CU-2x1wkca Update footer


The command is still using Helm 2

--tiller-namespace and --service-account flags are not supported in Helm 3. Please update to Helm 3 compatible command.

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

ID: d3925c0f-2f3a-a53e-d558-c00cca37e961
Version Independent ID: 1a49146e-07a4-bb12-0a5b-0d07590fbcad
Content: Create an ingress controller with an existing Application Gateway
Content Source: articles/application-gateway/ingress-controller-install-existing.md
Service: application-gateway
GitHub Login: @greg-lindsay
Microsoft Alias: greglin

@voonxx
Thanks for your feedback! We will investigate and update as appropriate.

@voonxx
Thank you for the feedback. This issue is currently being tracked internally as mentioned here..


Remove the white space

The white space is causing an error and bundler cannot find any commands!
bundler: command not found: _rails_command
Install missing gem executables with `bundle install`

Nevermind, you must be running be rails with the rails plugin loaded. I see no other choice than to revert that change then.


Use PicoCLI instead of commons-cli

PicoCLI has better support for handling command line args, including:

better help output
strong type conversions
automatic subcommand handling (routing logic)
exception and exit code handling (although we may continue to use our own)
keeps options contained to where they are used, instead of defined up front

It could be worth investigating if this provides any benefit, especially if we are going to add more subcommands as in https://github.com/jpmorganchase/tessera/issues/872 or https://github.com/jpmorganchase/tessera/issues/871.
There are already quite a few options in the DefaultCliAdapter, particularly around key generation, that could be split out.

As an extension to this task, changing the CLI framework also means that the current tests might me less relevant. For CLI stuff it would be great is testing was from the provision of CLI options and verifying the output, which is perhaps a departure to traditional unit tests.
Also, remove IO dependencies and abstract out. tests should interact with api objects and not have filesystem dependencies. FilesDelegate and other util objects can be used and mocked out for tests.


How do you install this extension to VSCode?

Nowhere in the README do you actually instruct how to install this into VSCode. You only say you must have XO installed already, which I do.

Like any other extension in VS Code. There's not much to it to be honest...

Gee, thanks for the help.

Would a section like this help? https://github.com/SamVerschueren/vscode-final-newline#install

Yes, thank you!!!!!

Fixed. Thanks for the suggestion. Sorry for the my first reply, wasn't meant to be rude...


cartodbfy shall discard cartodb_id candidates that contain nulls

Currently it checks for uniqueness but not for nullity.

Just forced this behaviour by importing a file in which I nulled some cdb_ids in the testing environment, and yes, the error is there:
ERROR:  CDB(_CDB_Add_Indexes:23502:column "cartodb_id" contains null values): ALTER TABLE untitled_table_2_merge_1 ADD PRIMARY KEY (cartodb_id)\nCONTEXT:  SQL statement "SELECT _CDB_SQL(sql, '_CDB_Add_Indexes')
(Probably this comment is useless for the record, but this way I make sure I report everything I'm facing...)

Example csv file from iriberri:
cartodb_id,the_geom,untitled_table_1_name,untitled_table_1_city,name,column_1441980553921,city
1,0101000020E610000074D2FBC6D79E0DC05A643BDF4F354440,,Madrid,,,Madrid
2,0101000020E6100000739D465A2A6FE6BF28D53E1D8F214340,,Elche,,,Elche
,,,,,,

https://cartodb.com/superadmin/users/11903/data_imports/09a4ba10-1572-4f47-99f2-a2744def0206
From today:
2015-10-02 13:03:32 UTC: Exception: PG::Error: ERROR:  CDB(_CDB_Add_Indexes:23502:column "cartodb_id" contains null values): ALTER TABLE download ADD PRIMARY KEY (cartodb_id)
CONTEXT:  SQL statement "SELECT _CDB_SQL(sql, '_CDB_Add_Indexes')"
PL/pgSQL function _cdb_add_indexes(regc

fix is ready, to be deployed beginning of next week.


[material-ui][Select] Does it receive different types of value?

I have a doubt about the types of the select component because it only accepts string values.
exemple:
<Select>
        <MenuItem value={10}>VALUES</MenuItem>
        <MenuItem value={20}>VALUES</MenuItem>
        <MenuItem value={30}>VALUES</MenuItem>
</Select>

In MenuItem, I get an error about the number value. In the documentation file select.d.ts, there is:

export interface SelectProps<Value = unknown>
  extends StandardProps<InputProps, 'value' | 'onChange'>,
    Omit<OutlinedInputProps, 'value' | 'onChange'>,
    Pick<SelectInputProps<Value>, 'onChange'> {

   * The `input` value. Providing an empty string will select no options.
   * Set to an empty string `''` if you don't want any of the available options to be selected.
   *
   * If the value is an object it must have reference equality with the option in order to be selected.
   * If the value is not an object, the string representation must match with the string representation of the option in order to be selected.
   */
  value?: Value | '';
}

When I set value?: Value | '' | number, it works fine. Does this mean the select component does not support other types in Material UI? I was thinking of opening a pull request, but it's better to clear my doubt first.
Search keywords: types, select

Hey @RivaldoCM, thanks for reaching out!
You can specify Select's value type via the Value generic parameter by doing <Select<number>>{...}</Select>, and you can do the same for SelectChangeEvent<number>. Here's an example: https://codesandbox.io/p/sandbox/40432-example-vjpfng
Is this what you're looking for?

Exactly, @DiegoAndai . Thanks for the explanation.


Pin flake8 to verion 3.5.0.

This fixes failing tests due to flake upgrade to version 3.6.0

Coverage remained the same at 89.495% when pulling 2b2829bb81e5fe2ae04e3172381e22205ef62e69 on akatsoulas:pin-flake8 into efe3e6a63454430cdb22661d3352e4624f264d2a on mozilla:master.


Fix lint, use containsMatchingElement

Fix contains assertion by switching to containsMatchingElement for Enzyme 2.3.0 compatibility.
Fix lint by downgrading to eslint 1.x (eslint-config-defaults is not yet compatible with 2.x).
Fix lint by disabling no-magic-numbers in tests.
Fix test-client error by adding react/addons to externals.

/cc @ananavati @benbayard @iamdustan @ryan-roemer

lgtm

@iamdustan Thanks! Looks like I don't have permissions on this repo to actually merge.

ah. I can do it for now. Would this be major, minor, or patch release? The public API appears to be the same.

@iamdustan Yeah, I'd go with patch release.

published as<EMAIL_ADDRESS>


Make Modifier implementations show up in the docs

These are really confusing to figure out otherwise.

See github.com/iron-bot/build-doc


[BUG]: Documentation - PowerStore Update Driver from v2.4 to v2.5 using Helm docs has wrong file name

Bug Description
https://dell.github.io/csm-docs/docs/csidriver/upgradation/drivers/powerstore/ - Step 2 "Edit helm/config.yaml file and configure connection information for your PowerStore arrays changing the following parameters:".  I believe this should be referring to the sample secret file. helm/config.yaml does not exist.
Logs
No logs
Screenshots
No response
Additional Environment Information
No response
Steps to Reproduce
Follow steps in documentation
Expected Behavior
Files referenced in documentation steps should be consistent
CSM Driver(s)
CSI Driver for PowerStore 2.5
Installation Type
No response
Container Storage Modules Enabled
No response
Container Orchestrator
Kubernetes 1.24.6
Operating System
RHEL 8.6

We will fix this in upcoming release.

@rajkumar-palani : please have a look


The links don't work because they're inside a code-block

File: notary/reference/server-config.md

Fixed in #9750


Use the new SloanDigitalSkySurvey package

I'm not sure how this will work with the CI build while waiting for SloanDigitalSkySurvey to be published (pull request: https://github.com/JuliaLang/METADATA.jl/pull/3188).

Looks like a test is failing

Yep, sorry, I think I just pushed a fix, let's see...
On Wed, Aug 26, 2015 at 4:24 PM, Jeffrey Regier<EMAIL_ADDRESS>wrote:

Looks like a test is failing
—
Reply to this email directly or view it on GitHub
https://github.com/jeff-regier/Celeste.jl/pull/53#issuecomment-135204176
.

Right now it's failing because the CI build can't get the SloanDigitalSkySurvey package.  I couldn't find how to put a non-published package in a REQUIRE file, so maybe we just have to wait for the pull request to be approved.
Tests pass on my machine.

Yeah, I think there's no way to require a package that isn't published.
On Wed, Aug 26, 2015, 5:01 PM Ryan<EMAIL_ADDRESS>wrote:

Right now it's failing because the CI build can't get the
SloanDigitalSkySurvey package. I couldn't find how to put a non-published
package in a REQUIRE file, so maybe we just have to wait for the pull
request to be approved.
Tests pass on my machine.
—
Reply to this email directly or view it on GitHub
https://github.com/jeff-regier/Celeste.jl/pull/53#issuecomment-135209411
.


Refactored the chipping script from sandbox into the main module

In tree_detection_framework/preprocessing:

I created derived_geodatasets.py to hold the CustomRaster and CustomVector datasets. Also moved the plot function inside CustomRasterDataset.
In preprocessing.py I implemented the create_dataloader, visualize_dataloader, and save_dataloader_contents functions.

In tree_detection_framework/entrypoints:

Added the script tile_data.py to test and perform the preprocessing from command line.

I also placed get_projected_CRS() in utils/geospatial.py. Let me know if you feel the geospatial.py needs to be renamed to something more generic.


update android project files

This should fix all the kotlin and agp version incompatibilities peoples
were having.
This updates the AGP version to the latest version which still works
with java 8. Bump gradle version to the minimum required.
Update kotlin to latest 1.4 point release release.
Replace jcenter with mavenCentral as jcenter is scheduled for shutdown.

How do we do each of the 3 steps?


fix: ignores files with permalink=false

When using permalink=false, the outputPath is undefined.

Hi @saneef, this is interesting.
Could you please make the same Pull Request in the new monorepo with the core library and this plugin?
It's here: https://github.com/nhoizey/images-responsiver/tree/main/packages/eleventy-plugin-images-responsiver
Thanks! 🙏

@nhoizey Created: nhoizey/images-responsiver#90


[Feature Request] Add roles based on existing reaction

It would be very cool if we could execute a function that would go through the existing reactions on the configured messages, and assign any roles that are missing based on those. Use case is that our bot may have been offline for some time, or we're migrating from another bot that missed many reactions. This could ensure there's nothing missing, I think it would make sense to have such an option

Yeah, I've been thinking about something like this too. Our bot doesn't have downtime issues, but there are cases such as reactions from people who isn't anymore in the server that could be removed with a catchUp() function to sync the roles to the reactions.

Would be good if removing the reactions of missing people was optional. And while bot may not have downtime issues, the Discord API having ones isn't that rare, and then these reactions may quickly get missed.
Here is an example code snippet on how I did this if that helps (I just put it quickly together to get the job done, so it's ugly and not safe against edge cases, but TS will catch all of these):
msg.guild.channels.resolve("CHAN_ID").messages.fetch("MSG_ID").then(message =>
            {
                    message.reactions.cache.find(r=>r.emoji.name==="🖥").users.fetch({after:"LAST_USER_ID"}).then(r=>{
                    r.each(u=>{
                        let gm=msg.guild.members.resolve(u.id);
                        if(gm&&!gm.roles.cache.has("ROLE_ID")){
                            gm.roles.add("ROLE_ID");
                            msg.reply('adding missing role to:' + gm.user.tag);
                        };
                    });
                    msg.reply('last: ' + r.last().id + ', size: ' + r.size);
                });
            }
        );

Generally users.fetch() should initially have no params, but it returns up to 100 users at once, so then you gotta use the r.last().id to supply to the after param to get the next ones. If returned amount is <100 or is 0, then it was the last portion. Checking against roles.cache is important, because otherwise rate limits are hit very frequently, and even if something isn't in the cache, it doesn't hurt to check it in this case (different story with removal...)


Fix py modules loading in R API

Fix the issue “attempt to apply non-function”. This is because those Python modules are not loaded properly.

LGTM


Tesseract fails to OCR text with very clear hexadecimal digits

Current Behavior
Here is the image:

And here is what Tesseract produces as OCR:
74
3a

31
3c

72
2C

74
Oa

Expected Behavior
n/a
Suggested Fix
n/a
tesseract -v
$ tesseract -v
tesseract 5.3.4
 leptonica-1.82.0
  libgif 5.2.1 : libjpeg 8d (libjpeg-turbo 3.0.1) : libpng 1.6.40 : libtiff 4.4.0 : zlib 1.3 : libwebp 1.3.2
 Found SSE4.1
 Found OpenMP 201811
 Found libarchive 3.7.2 zlib/1.3 liblzma/5.4.4 bz2lib/1.0.8 liblz4/1.9.4 libzstd/1.5.5
 Found libcurl/8.5.0 OpenSSL/3.0.12 zlib/1.3 libpsl/0.21.2 (+libidn2/2.3.4) libssh2/1.11.0 nghttp2/1.58.0

Operating System
No response
Other Operating System
FreeBSD 14.0
uname -a
$ uname -a
FreeBSD yv.noip.me 14.0-STABLE FreeBSD 14.0-STABLE #1 stable/14-n266076-2001d7f6a272: Sat Dec 30 13:33:21 PST 2023     unknown@nohost.com:/disk-samsung/obj/disk-samsung/freebsd-src/amd64.amd64/sys/GENERIC amd64

Compiler
clang-16
CPU
Intel(R) Core(TM) i7 CPU         930  @ 2.80GHz
Virtualization / Containers
n/a
Other Information
n/a

Try to invert the image before running OCR.

Regular text is OCRed fine when it is white on black, it's only hexadecimal digits that fail.

Please try black on white nevertheless. Does it work then?

The inverted text works better, but still not perfect, with -c thresholding_method=2.

@yurivict : read documenation before posting issue!
tesseract i4183.png - something_that_is_alredy_covered_by_documenation
6d 20 64 6f 63 73 20 68 6f 77 20 74 6f 20 72 75
6e 20 50 79 74 68 6f 6e 20 74 65 73 74 73 2c 20
73 65 65 20 68 74 74 70 73 3a 2f 2f 67 69 74 68
75 62 2e 63 6f 6d 2f 61 70 61 63 68 65 2f 74 76
6d 2f 69 73 73 75 65 73 2f 31 32 31 35 38 0a Oa
2e 69 6e 63 6Cc 75 64 65 20 3c 62 73 64 2e 70 6f


Improve Issue Templates

Describe the solution you'd like
We want to use Github Issues as a tracker for new requests. We want to improve the template we are currently using to fit our new requirements.

@DaoDaoNoCode @dlabaj @maroroman @samuelvl
This is the new yaml format for the requests, I've created a mock project here just to check how it works.
These are the new templates, let me know what you think:
Bug report
name: Bug Report
description: File a bug report
title: "[Bug]: "
labels: ["bug"]
assignees:
  - lucferbux
  - DaoDaoNoCode
  - dlabaj
body:
  - type: markdown
    attributes:
      value: |
        Thanks for taking the time to fill out this bug report! Please, fill this form to help us improve the project.
  - type: input
    id: contact
    attributes:
      label: Contact Details
      description: How can we get in touch with you if we need more info?
      placeholder: ex<EMAIL_ADDRESS>    validations:
      required: false
  - type: checkboxes
    attributes:
      label: Is there an existing issue for this?
      description: Please search to see if an issue already exists for the bug you encountered.
      options:
      - label: I have searched the existing issues
        required: true
  - type: textarea
    attributes:
      label: Current Behavior
      description: A concise description of what you're experiencing.
    validations:
      required: false
  - type: textarea
    attributes:
      label: Expected Behavior
      description: A concise description of what you expected to happen.
    validations:
      required: false
  - type: textarea
    attributes:
      label: Steps To Reproduce
      description: Steps to reproduce the behavior.
      placeholder: |
        1. In this environment...
        2. With this config...
        3. Run '...'
        4. See error...
  - type: textarea
    id: workaround
    attributes:
      label: Workaround (if any)
      description: Any manual steps that allow you to resolve the issue
      placeholder: Tell us the steps you followed to resolve the issue!
    validations:
      required: false
  - type: dropdown
    id: openshift-version
    attributes:
      label: Openshift Version
      description: What version of Openshift are you running?
      options:
        - Baremetal
        - OpenStack
        - RHV
        - AWS
        - OKD
        - CodeReady Containers
    validations:
      required: false
  - type: dropdown
    id: browsers
    attributes:
      label: What browsers are you seeing the problem on?
      multiple: true
      options:
        - Firefox
        - Chrome
        - Safari
        - Microsoft Edge
    validations:
      required: false
  - type: textarea
    id: opendatahub-version
    attributes:
      label: Open Data Hub Version
      description: Please attach relevant kfdef manifest if applicable
      render: yaml
  - type: textarea
    id: logs
    attributes:
      label: Relevant log output
      description: Please copy and paste any relevant log output. This will be automatically formatted into code, so no need for backticks.
      render: shell

feature request
name: Feature request
description: Suggest an idea for this project
title: "[Feature Request]: "
labels: ["enhancement"]
assignees:
  - lucferbux
body:
  - type: markdown
    attributes:
      value: |
        Thanks for taking the time to fill out this feature request! Please, fill this form to help us improve the project.
  - type: input
    id: contact
    attributes:
      label: Contact Details
      description: How can we get in touch with you if we need more info?
      placeholder: ex<EMAIL_ADDRESS>    validations:
      required: false
  - type: textarea
    id: description
    attributes:
      label: Feature description
      description: A clear and concise description of what you want to happen.
    validations:
      required: true
  - type: textarea
    id: describe-alternatives
    attributes:
      label: Describe alternatives you've considered
      description: A clear and concise description of any alternative solutions or features you've considered.
      placeholder: Tell us about alternatives you've considered...
    validations:
      required: false
  - type: textarea
    attributes:
      label: Anything else?
      description: |
        Links? References? Add any other context or screenshots about the feature request here.

        Tip: You can attach images or log files by clicking this area to highlight it and then dragging files in.
    validations:
      required: false


initial code for pagination view.

Please always provide the [Jira issue(s)] your PR is for, as well as test URLs where your change can be observed (before and after):
Fixes #
Test URLs:

Before: https://main--aem-competency--aem-comp.hlx.live/big-bets

After: https://feature-big-bets-listing--aem-competency--aem-comp.hlx.live/big-bets

[ ]  If you are adding a new block or a variation to an existing block, please fill below:
Block library path: https://--aem-competency--aem-comp.hlx.live/tools/sidekick/library.html?plugin=blocks&path=/sidekick/blocks/&index=0

Not required.