Cluster items with exact same lat/lang position are always bound together as a cluster

Summary:
Cluster items with exact same lat/lang position are always bound together as a cluster and do not separate away as individual items even at the lowest zoom level.
Steps to reproduce:
Add items with same lat/lang
Expected behavior:
items should seperate
Observed behavior:
items stay clumped together and rendered as cluster
Device and Android version:
lib 0.4.3. Samsung Galaxy S6

I´ve developed a quick workaround, you can find here.
In general, ClusterManager class is extended to include a method to check if markers in a cluster have the same location, if that´s the case, the implementation of OnClusterClickListener distribute the markers around the cluster location an hides the original cluster.
This workaround only works if the user is in the maximum zoom level and all the markers in the cluster have exact the same location (lat/lng). Also, when the user zooms out all the markers bring back to the original cluster.

Re-opening, there is a fix proposed in https://github.com/googlemaps/android-maps-utils/pull/413.

Has the same issue.

Same problem with versions:
implementation "com.google.android.gms:play-services-maps:18.0.0"
implementation 'com.google.maps.android:android-maps-utils:2.2.3'

Reading this issue again, it's not clear to me if the feature request is 1) to show clusters are individual markers (but still stacked on the same lat/lon), or 2) to somehow visualize markers with the same lat/lon as "unstacked" so they are separate and don't appear exactly on the same lat/lon.
@september669 @agent10 @menismu @usergoodvery Could you clarify which of the two above you're looking for?
If 1), you can actually accomplish this with the new method shouldRender() that was added in 2.3.0, along with shouldRenderAsCluster(). See the ZoomClusteringDemoActivity for an example, which implements a ZoomBasedRenderer that overrides these methods.
I tested this demo activity with the same marker locations as follows:
        clusterManager.addItem(new MyItem(18.545723, 73.797726, "Loc1", snippet));
        clusterManager.addItem(new MyItem(18.545723, 73.797726, "Loc2", snippet));
        clusterManager.addItem(new MyItem(18.545723, 73.797726, "Loc3", snippet));
        clusterManager.addItem(new MyItem(18.545723, 73.797726, "Loc4", snippet));
        clusterManager.addItem(new MyItem(18.545723, 73.797726, "Loc5", snippet));
        clusterManager.addItem(new MyItem(18.545723, 73.797726, "Loc6", snippet));
        clusterManager.addItem(new MyItem(18.545723, 73.797726, "Loc7", snippet));

...and when you zoom in you can see the individual markers instead of a cluster, although they are all stacked on top of each other. When you tap on the marker, it does cycle through all 7 of them and show the corresponding text in the popup balloon.
Please take a look and let me know if that accomplishes what you're looking for, or if you're looking for a solution to visualize markers with the same lat/lon in a different way.

IMHO,  @menismu workaround above looks good and it will be a good UI solution.
I mean if you tap on cluster of markers with same location it splits around, and with zoom out splited markers clustered again.


Explicitly create build artifcats (nuget package) in CI/CD

A build artifact is downloadable output which can be created and collected on each CI run. It can be anything from a single file to an entire folder full of binaries. In the case of a .NET Core NuGet library it is a very useful feature to create a super early version of a NuGet package as soon as a build has finished:
In combination with pull request triggers this is a super handy way of giving OSS contributors and OSS maintainers an easy way of downloading and testing a NuGet package as part of a PR.
It is also a nice way of letting users download and consume a "super early semi official" NuGet package which came from the project's official CI pipeline when someone is in absolute desperate need of applying a fix before an official release or pre-release has been created.
More from: https://dusted.codes/github-actions-for-dotnet-core-nuget-packages#github-release-trigger-for-official-nuget-release

I don't think this is absolutely necessary for 1.0. Removing 1.0 label.


Cannot Get Host Details with SDK

I am currently using the SDK to try and generate custom reports.  I could be missing something, but it seems that getting the details for a specific host are missing from the API.  Any chance this could be added in?  Does it already exist and I'm missing it?  If the API call isn't part of the API currently, was there a specific reason it was left out?   Thanks!
https://cloud.tenable.com/api#/resources/scans/host-details

+Alan Fung
From: James Anderson<EMAIL_ADDRESS>Sent: Thursday, October 5, 2017 9:15 AM
To: tenable/Tenable.io-SDK-for-Python<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [tenable/Tenable.io-SDK-for-Python] Cannot Get Host Details with SDK (#57)
I am currently using the SDK to try and generate custom reports. I could be missing something, but it seems that getting the details for a specific host are missing from the API. Any chance this could be added in? Does it already exist and I'm missing it? If the API call isn't part of the API currently, was there a specific reason it was left out? Thanks!
https://cloud.tenable.com/api#/resources/scans/host-details
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/tenable/Tenable.io-SDK-for-Python/issues/57 , or mute the thread https://github.com/notifications/unsubscribe-auth/ABL1UYlNqZM2aohfApRhkOOoQIEbdYniks5spQB5gaJpZM4PvVn- .  https://github.com/notifications/beacon/ABL1UacASz8gsfJ71OADK-rO8rlYj6xEks5spQB5gaJpZM4PvVn-.gif

@jandersonbt There is an update being worked on now that includes the call you mention. We expect it to be released in the next day or so.
In the meantime you could try something like this to get around it:
from tenable_io.client import TenableIOClient
from tenable_io.api.models import BaseModel

host_details = {}

client = TenableIOClient()
scans = [scan.id for scan in client.scans_api.list().scans]

# For each scan, find all hosts and add them to a dictionary keyed by scan_id
for scan in scans:
    scan_hosts = []
    hosts = [host['host_id'] for host in client.scans_api.details(scan).hosts]
    for host in hosts:
        response = client.get('scans/%(scan_id)s/hosts/%(host_id)s',
                                    path_params={'scan_id': scan, 'host_id': host})
        scan_hosts.append(BaseModel.from_json(response.text).info)
    host_details[scan] = scan_hosts

print(host_details)

Which should print something like:
{
  "17": [
    {
      "host-ip": "<IP_ADDRESS>",
      "host_end": "Thu Sept 10 09:09:28 2017",
      "host_start": "Thu Sept 10 09:06:53 2017",
      "operating-system": "Microsoft Windows Server 2012 R2 Standard",
      "host-fqdn": "testserverms1"
    }
  ],
  "19": [
    {
      "host-ip": "<IP_ADDRESS>",
      "host_end": "Thu Sept 11 11:29:24 2017",
      "host_start": "Thu Sept 11 11:26:59 2017",
      "operating-system": "Microsoft Windows 8.1",
      "host-fqdn": "msworkstation1"
    }
  ]
}

Thanks so much for the update!  And thank you for the quick fix!

@jandersonbt thank you for reporting this issue, we have addressed this in the 0.3.0 release.
TenableIOClient().scans_api.host_details(scan_id, host_id)


Enable override plugin

This allows repo admins to force green status on the failing test as an exemption

This is to solve https://github.com/nephio-project/nephio/pull/133

/lgtm

/lgtm


Dont times * 100 on amount

You should not be doing times on the amount. Because the currency cannot always be split into 100.
For example YPN (japanese) will not be multiplied with 100.

Thanks for the heads up. I'll look into it

Any updates @tuurbo ? :)

I'll try to work on it this weekend :D

Nice! Them I can push monday

Hey guys, take a look to http://verraes.net/2011/04/fowler-money-pattern-in-php/

released a beta version of 2.0 (don't use in production), needs more testing.

no longer convert money to cents
psr2
switched internal api calls from xml to json

https://github.com/tuurbo/spreedly/tree/2.0

Looks good. When can we use in production? Is there anything you would like me to test?

@canfiax I've now tested quite a few methods, just didn't want to push a major release until i'm sure there aren't any major issues. You can try it out with composer require tuurbo/spreedly:2.0.x-dev. I'll try to push to master within the next couple days

Plan on releasing 2.0 tonight.

Awesome!! Good job 💪🏽


Half angle calculator

A tool that calculates Half-angle functions sin(x/2), cos(x/2), tan(x/2). I would like to add this tool

this is already implemented in Calculator, which is on the home screen


Centos 7 - Compile issues

Hi All,
I'm using CentOS 7 and running this version i have noticed this is an EL6 package which raises questions but i used the documentation from here.
https://crystallang.org/docs/installation/on_redhat_and_centos.html
[brian@eggbox crystal-monetdb-libmapi]$ rpm -qi crystal
Name        : crystal
Version     : 0.23.1
Release     : 3.el6
Architecture: x86_64
Install Date: Sat 16 Sep 2017 05:37:52 PM BST
Group       : default
Size        : 62248014
License     : Unspecified
Signature   : RSA/SHA512, Fri 14 Jul 2017 06:43:46 PM BST, Key ID 09617fd37cc06b54
Source RPM  : crystal-0.23.1-3.el6.src.rpm
Build Date  : Thu 13 Jul 2017 07:18:33 PM BST
Build Host  : localhost
Relocations : /
Packager    : Juan Wajnerman
Vendor      : Omnibus<EMAIL_ADDRESS>URL         : http://crystal-lang.org/
Summary     : The full stack of crystal
Description :
The full stack of crystal
So when i try to compile my project crystal-monetdb-libmapi i get this.
crystal build --release quick.cr -o bin/quick
crystal: /var/cache/omnibus/src/llvm/llvm-3.8.1.src/lib/CodeGen/LexicalScopes.cpp:160: llvm::LexicalScope* llvm::LexicalScopes::getOrCreateRegularScope(const llvm::DILocalScope*): Assertion `cast(Scope)->describes(MF->getFunction())' failed.
/usr/bin/crystal: line 102: 19983 Aborted                 (core dumped) "$INSTALL_DIR/embedded/bin/crystal" "$@"
make: *** [quick-release] Error 134
Hope i have provided enough information.
Regards,
Brian Hood

Hi @puppetpies!
This is a known issue of Crystal 0.23.1 - we're tracking it on #4719.
As a workaround, adding --no-debug as a compilation flag will allow you to compile.

Cheers man !


Make playground equipment searchable

https://github.com/openstreetmap/id-tagging-schema/blob/bf95421f65018ccd7f398f8459b37462febddcba/data/presets/_playground.json#L26

Note https://github.com/openstreetmap/id-tagging-schema/blob/bf95421f65018ccd7f398f8459b37462febddcba/data/presets/_playground.json#L24
This preset is capable of roughly describing what is tagged, it is not possible to tag with it

I'm a bit confused? Yes the preset comes up when you add the playground tag, I just found it annoying to type it in every time instead of being able to search for it like it prompts you to do after creating a feature.

This preset that you linked has no values for specific equipment, it is a catchall for any playground tag.
If it would be searchable - playground=* would be tagged after selecting it, not a specific value.
And in case adding (or having) more specific values like playground=swing this would remain as an unsearchable catchall anyway.

If it would be searchable - playground=* would be tagged after selecting it, not a specific value.

@matkoniecz The file is misleading, the tag would be playground=yes, which appears to be used for specifying if a facility has a playground, not for generic playground equipment.
@lectrician1 The problem would appear to be that there's no great way to tag generic playground equipment in OSM right now.

Could searching for "playground" show all already-existing presets for playground instead?
I notice that presets have the "terms" JSON element to tell iD to show the preset when specific things are searched.
Could something be implemented that allows the all of the playground features to be shown when searching for "playground", without having to add the "terms": "playground" element to every playground preset? Of course, these should probably be rank below a standard Playground feature, but they should still show up in the search.
This could also apply to other OSM tags. For example, searching for "highway" displays all of the presets with the highway tag (ranked below of course).
Also, another solution could be actually making it searchable and then prompting the user to enter a type of equipment. If the user does not enter a type, iD could create an issue. Then if no type is selected even at submit then it discards the feature.
Idk. Just some ideas.

The problem would appear to be that there's no great way to tag generic playground equipment in OSM right now.

I'm a bit confused by this statement?

Could something be implemented that allows the all of the playground features to be shown when searching for "playground", without having to add the "terms": "playground" element to every playground preset?

We could add a category for the playground presets, like the Golf Features category.

I'm a bit confused by this statement?

It doesn't look like there's a tag that says "this is playground equipment" without specifying what type of equipment it is. This is different from tags like historic=yes which means "this is a historic feature" without further context.

I would be for a category.
In the process of writing this I thought of these things (moved them to new issues):
https://github.com/openstreetmap/iD/issues/8207
https://github.com/openstreetmap/id-tagging-schema/issues/58


Setup fails because 'SATA Controller' already exists

Virtualbox Version: 6.1.6_Ubuntu r137129
Vagrant Version: 2.2.9
OS: Ubuntu 20.04 LTS
When i run vagrant up command the following issue appears and it stops vagrant from continuing
A customization command failed:

["storagectl", :id, "--name", "SATA Controller", "--add", "sata", "--portcount", 2]

The following error was experienced:

#<Vagrant::Errors::VBoxManageError: There was an error while executing `VBoxManage`, a CLI used by Vagrant
for controlling VirtualBox. The command and stderr is shown below.

Command: ["storagectl", "bc0931a1-b0b1-4589-a847-a4d39685910e", "--name", "SATA Controller", "--add", "sata", "--portcount", "2"]

Stderr: VBoxManage: error: Storage controller named 'SATA Controller' already exists
VBoxManage: error: Details: code VBOX_E_OBJECT_IN_USE (0x80bb000c), component SessionMachine, interface IMachine, callee nsISupports
VBoxManage: error: Context: "AddStorageController(Bstr(pszCtl).raw(), StorageBus_SATA, ctl.asOutParam())" at line 1078 of file VBoxManageStorageController.cpp
>

Please fix this customization and try again.

Thanks for the info!
Was this a first set up or did you pull the changes down then try to set it up?

This was the first set up, all i did is clone the repo, cd into rhcsa8env directory then vagrant up and i ended with that error, a similar error was also mentioned in issue #13

I just made some changes to the drive config that may be causing it. I'll take a look

I'm not getting the error on Ubuntu 18. I'll try on 20.

I'm not receiving the error on the following config:
Virtualbox Version: 6.1.12 r139181
Vagrant Version: 2.2.9
OS: Ubuntu 20.04 LTS

Can you run apt update -y && apt upgrade -y && vagrant destroy -f, reboot, then try to build the environment again?

I think because i've got different vagrant machines running that causes the issue, for me the problem wasn't resolved after running vagrant destroy -f , also same behavior happened on windows but the difference was that i was able to get all the 3 machines running, doing vagrant halt then vagrant up again i ended up with the same error.
i wanted to skip setting up samba and nfs (server side) but ended up doing them anyway and now i'm able to see the shares on the client machine
Vagrant.configure("2") do |config|
  config.vm.synced_folder ".", "/vagrant", type: "rsync"
  config.vm.box_check_update = false

  config.vm.define "server" do |server|
    server.vm.box = "bento/centos-8"
    server.vm.hostname = "server"
    server.vm.network "private_network", ip: "<IP_ADDRESS>"
    server.vm.provision "shell", inline: <<-SHELL
      sudo yum update
      sudo yum install -y vdo kmod-kvdo httpd
      sudo echo "<IP_ADDRESS> nfs.com" >> /etc/hosts
  SHELL
  end

  config.vm.define "nfs" do |nfs|
    nfs.vm.box = "bento/centos-8"
    nfs.vm.hostname = "nfs"
    nfs.vm.network "private_network", ip: "<IP_ADDRESS>"
    nfs.vm.provision "shell", inline: <<-SHELL
      sudo yum update
      sudo yum install -y nfs-utils cifs-utils samba-client samba
      sudo systemctl enable --now {nfs-server,smb,firewalld}
      sudo firewall-cmd --permanent --add-service=nfs
      sudo firewall-cmd --permanent --add-service=mountd
      sudo firewall-cmd --permanent --add-service=rpc-bind
      sudo firewall-cmd --permanent --add-service=samba
      sudo firewall-cmd --reload
      sudo mkdir -v /nfs /cifs
      sudo echo "/nfs   *(rw,no_root_squash)" >> /etc/exports
      sudo systemctl restart nfs-server
      sudo useradd samba
      sudo chown samba /cifs && sudo chmod 770 /cifs
      sudo echo '''
                [cifs]
                  comment = samba share
                  path = /cifs
                  write list = samba
                ''' >> /etc/samba/smb.conf
  SHELL

  end
end

Am i missing something with this setup or its just good enough to start practicing ?

Unfortunately i won't be able to erase all the different boxes, i'm going to close the issue and hopefully you'll be able to make it work without having to start from a completely fresh vagrant setup at some point in the future, thank you.

No problem. Thanks for submitting the issue!

i'm also having the same issue. it worked fine when i set it up for the first time. but if i type vagrant halt and try to turn it on again
i get this error after this line  server2: Running 'pre-boot' VM customizations....
server1 and repo are working fine though.
 os         : fedora 32
vagrant : 2.2.9
 vbox     :   6.1.12 r139181


Multiple Selection

Is there any possibility select multiple file ...

Not right now

Hi,
Is this feature gonna be available soon?

Wouldn’t count on it. Done when it’s done!

+1

+1

+1

Notice that this feature request is up for grabs. I don't need multi selection in pay project, so I will not develop this feature in the foreseeable future, and I think jfversluis is also occupied. Nevertheless, pull requests are welcome!

I have implemented this featuer myself in UWP as one of my clients needed it.
It was super easy in UWP but I don't know how easy it is in iOS, Android, WPF and macOS.
I will file a PR when I have added all platforms.

You can already make a PR now and push changes as you go. This way we can discuss the API and the implementation right now

Hi ok I will do that today when I get home.
UWP part is done and is fully working!
I am a bit struggling with Android part so hopefully you can help me with that part.
Thanks!

I just implemented multiple selection for the file picker proposal in Xamarin Essentials, you might borrow the implementation from there: https://github.com/xamarin/Essentials/pull/1147

Closing this since this project will retire, please check out Xamarin.Essentials which now has a FilePicker based on this one.


[BUG]README.md LICENSE linked error

What happened:
README.md LICENSE linked error. LICENSE.md cannot be found.
What you expected to happen:
fix LICENSE.md to LICENSE in README.md
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

App version:
Kubernetes version (use kubectl version):
Install details (e.g. helm install args):
Others:

@MorganMaxg Nice catch! Would you like to submit a PR to fix this?

@MorganMaxg Nice catch! Would you like to submit a PR to fix this?

OK

/assign @MorganMaxg

/documentation

/label documentation


Add delete index if exists method

ISSUE-231: Add delete index if exists method to handle index not found error

@alallema thanks for the review. I implement the review remarks


Small issue with lists

Hey there, I'm currently trying to create output like so:
output.push({ h1: "Collection" })
categories.forEach(cat => {
  output.push({ h2: cat.name })
  cat.files.forEach(file => {
    output.push({ ul: [ {link: { title: file.name, source: file.path }}]})
  })
})

Now, that is obviously adding a ul around every link, how would I .push the opening of an ul separate from the ending so I can keep my loop?

As far I remember, the lists are not interpreted deeply.

Sorry, actually they are.
I'm not sure what the problem is...

Oh that's fine then, was not sure because the documentation did not say anything about that.

To explain my problem with that a bit more: It just adds a ton of new lines after every li this way.

Any workaround?


Merge the SFD Dustmap code into the web interface (especially for AstroNote generation)

Need to fix the AstroNote code to add the SFD dustmaps and use them to generate the relevant extinction for PS1 and ATLAS.

Done for ATLAS. Still to be done for Pan-STARRS. (PS1, PS2, YSE)


update package version number

Changed setup.py version so that I can update the pypi package version.

Should be good to go.


How to use changedRows in OkPacket?

I wonder about changedRows in OkPacket.js

changedRow is not affectedRows.
node-mysql fetch changedRows from OkPacket.message.
but it seems message always empty, so changedRow always 0.

https://github.com/felixge/node-mysql/blob/master/lib/protocol/packets/OkPacket.js#L25
So, when and how to use changedRows value?

All that does is parse the message, and doesn't have anything to do with the MySQL protocol, so I'm not sure. Looking through the history of this repo, it was added from https://github.com/felixge/node-mysql/issues/251
If that doesn't help explain what it does, then I'm at a loss :)

Hi @wilfrem , I have not heard back from you since my post, so I'm going to close this issue as resolved. Please let me know if you're still having issues!


Add quick Regexps course

Regular expressions are extremely important in computer programming but Py4E barely scratches the surface. I suggest adding https://regexone.com/ at some point in the course. It should only take a couple of days to complete but should also provide a more thorough understanding of exactly how regexps work.

Hi @rachel83az .
Can you provide more justification for adding to the curriculum? As noted in our Contributing page, successful critiques of the curriculum will point out ways that OSSU is failing to uphold our curricular guidelines.

Finding a place for regular expressions in the curriculum is tough.  The only place they're mentioned is in the Algorithms section under Automata Theory.   With that said, regular expressions are something developers will use thoroughly in their careers and should be something OSSU covers.
With that said, I think there is a place for a regular expression course under the CS Tools section.

I promised myself I would stay out of this, but...
Since you haven't taken it, you might be unaware, but all the RE related stuff on page 60 of CS 2013 (DFAs and NFAs and their equivalence to RE) is covered by the Compilers course.

Regular Expression is a useful tool, but not worth adding to the curriculum as a whole course. I haven't seen any universities do so

Adding to spamegg's observation, CS2013 specifies learning about regular expressions under the Knowledge Area Algorithms and the topic Basic Automata Computability and Complexity. The OSSU course covering Automata is Theory of Computation. This course lists regular expressions as part of the curriculum.

This proposal has had a few months to gather contributor feedback. Reading through the above:

Study of regular expressions is part of our curricular guidelines. We have courses that address these guidelines.
 There is some support for adding this tool to the CS Tools section of the curriculum, but it is not an obvious majority.

That said, we do keep track of "high-quality courses that, for one reason or another, didn't make it into the curriculum". @rachel83az , would you make a PR to add https://regexone.com/ to the CS Tools section of extras/courses? I will merge that in. It may be that in the future more students will find it there and advocate its inclusion in the main curriculum, something we can revisit after some time.


Run regression tests in github actions

@MiWeiss This is cool! I merged your contribution into the main branch
via c3f95a0 (edited your commit a bit, since I wanted to  add some more
information into the commit message). Thank you!
Addendum I also added you as the contributor to LICENSE.txt.


Added new color variables and updated hex values in the documentation

Notes are here:
https://docs.google.com/document/d/1bRNcUS9t_0WL_ckkMetWRu_jp-KIQOUByd-Rb903zy0/edit
Started working on this yesterday! Still have not done:
an accessibility pass, other than testing these on a crappy screen (new colors look better than old colors fwiw)
One thing to note: I am not using a new color for text-gray-lightest, as the new colors version of this was way too light to be readable. For more context, check out the doc above!
I'm sure there are a bajillion mistakes! 🎱

Thank you for doing this!! I'm having trouble with these new grays :-/
Fill-gray-lighter feels really dark! Here's a comparison: http://take.ms/QMhnP This is much more like our current middle gray fill (fill-gray)
As you can see, the light grey background of Solid is now super dark:

Here's how it looks on the homepage, comparing old and new: http://take.ms/8Mj6A
The new fill-gray seems a bit too close to fill-gray-darker : http://take.ms/kzHNP
 Not sure if we'd have a use for both these dark greys.
I don't feel comfortable shipping these new light and mid grays. Do you agree?
The other colors feel great to me.

yep it's quite dark! i figured we were going to have to compromise somewhere. i mentioned above about text-gray-lightest, probably the same pattern can follow for fill-gray-lighter. Unsure about the 'middle' gray. Will have to check that out tomorrow, going to stop doing work now :).

Fill Grays
fill grays now match all of our gray colors
the main difference between our grays and branding gray is that they have a light gray value between our fill-gray and fill-gray-lighter that's #e0e0e0, which felt too dark to become our fill-gray-lighter. So! The colors are in the branding sheet if need be, but will not be included the Solid colors for now.
Text Grays
Text gray lightest is the same value as before (#999999)
Text-gray, I switched to #222 (same value as fill-gray-darker, and it's the "darkest-gray" in the branding sheet). The next value up is lighter than our current text-gray, so I am less in favor of that.
I'm into this change because it feels less muddy, especially on weaker screens with lower resolution. Here is a compare of the homepage (darker gray on left).

Let's talk about reservations in the solid meeting!

Hey friends! This is ready to go, if anyone wants to take a last look! @allisonchefec @lindseymaratta
Follow up card should include:
-adding teal
-adding purple
-darkening fill-gray-lightest a hair

Nope! Accident because the defaults were added after I started this branch. Good catch!

Adjusted - also fixed the vertical alignment of the !defaults in the variable scss file

The contrast for the green success messaging is pretty low. Definitely fails the WCAG but maybe if we used a lighten function for the background it could work? Could be addressed in a follow up or as part of the accessibility stuff.
https://monosnap.com/file/5RCLwJlcUmOCqglq840ppHpBCENlNC.png
Also still feeling some reservation about the text-grays - seeing the svg-fill grays next to each other is what really makes it feel redundant, but it would be weird to not be consistent. Though if you're confident that the change is worth it, I trust ya!
https://monosnap.com/file/N4GxTpHjuJ8oEBKL0ZVcT4HNSNzlrm.png
Other than these concerns (do what you want with them!) and maybe a rebase to preview the integration with the last release, it lgtm. 👍

Ah good catch @lindseymaratta. I know our svg fills are following our text colors, but in this case this does feel too dark, as did the other medium gray we tried, but then vetoed.
Do we want to revert the updated text-gray-lighter color? Or do we want to just update this svg fill color to be something we're happy with? My vote would be to revert the updated text-gray-lighter color.

yeah I agree with reverting the updated text-gray-lighter - that way we can confidently pair icon and text of the same color

Latest push (maybe final...a little late in the game to be making big changes to this now): I reverted text-gray-lighter back to 666 which should fix the mid SVG gray as well

cool! let's do it!


Expected type did not match the received type

🐛 Bug Report
There is a return type error that is preventing the application from building
Steps to Reproduce
sudo /home/ubuntu/.cargo/bin/cargo install --path .
Expected Behavior
It is supposed to run smoothly and I will then have access to "snarkos account new" but that did not happen, instead I got the above error.
Your Environment

git clone https://github.com/AleoHQ/snarkOS.git --depth 1 which should be pointing to the latest release I assume
rustc 1.72.1 (d5c2e9c34 2023-09-13)
Ubuntu 22

Note: I have attached a link to screenshot of the error.
https://photos.app.goo.gl/RoLNHRsJ4wdqATXm9

Same issue here, on 1d942fdc3924b401f1381f73624301b139df3067
stacktrace of sudo ./build_ubuntu.sh:
   Compiling snarkos v2.2.7 (/home/tom/snarkOS_aleo_node)
error[E0308]: mismatched types
   --> node/rest/src/lib.rs:115:54
    |
115 |                 .error_handler(|error| Response::new(error.to_string()))
    |                                        ------------- ^^^^^^^^^^^^^^^^^ expected `Body`, found `String`
    |                                        |
    |                                        arguments to this function are incorrect
    |
note: associated function defined here
   --> /root/.cargo/registry/src/index.crates.io-6f17d22bba15001f/http-1.0.0/src/response.rs:253:12
    |
253 |     pub fn new(body: T) -> Response<T> {
    |            ^^^
help: call `Into::into` on this expression to convert `std::string::String` into `Body`
    |
115 |                 .error_handler(|error| Response::new(error.to_string().into()))
    |                                                                       +++++++

will be fixed with https://github.com/AleoHQ/snarkOS/pull/3056

The current fix is the second commit in https://github.com/AleoHQ/snarkOS/pull/3050.

Fixed with https://github.com/AleoHQ/snarkOS/pull/3050.


lib/dcmd: don't accept any number when Min == Max

Stop accepting any number when the specified Min and Max are equal to
each other. Instead, return an OutOfRangeError.
Apply this to all number args we have (i.e. IntArg and FloatArg).
Expected breakage should be low to none. One would generally expect that
when specifying a range with the same Min as Max that only that number
is to be accepted.
See also:
https://canary.discord.com/channels/166207328570441728/356486960417734666/1010540772367540284
Signed-off-by: Luca Zeuch<EMAIL_ADDRESS>

After investigating a bit, this change feels
impossible to achieve, as not providing a
Min and Max will result in a [0, 0] range,
thus always failing a parseArgs call that does
not have a range given.
Closing this.

else if (i.Min != 0 || i.Max != 0) && i.Max == i.Min {
		if v != i.Max {
			return nil, &OutOfRangeError{ArgName: def.Name, Got: v, Min: i.Min, Max: i.Max}
		}

Oh wow 🤦
This is why I shouldn't write code with just four hours of sleep.


Feature: Display PDF on script page/main page when script selected

Had multiple pieces of feedback that it would be useful to display the PDF (if available) rather than just the JSON IDs.

I think this should be possible when the PDF is hosted on Azure since it's just a URL rather than actually needing to load the file. Should be possible with an iFrame according to https://stackoverflow.com/questions/17784037/how-to-display-pdf-file-in-html


SMTP password should be stored in a secret instead of config map

Description
The main goal of this task is to store password, used for SMTP authentication, in the secret object.
Additionally, next to passing value via values file/inline, the possibility of using an external secret for smtp password should be added.
Related tasks
https://github.com/FlowFuse/helm/issues/290
Epic/Story
No response

Hey @ppawlowski could I be a pain and request you add an appropriate sizing to this please, and then a rough "Expected" date on when it would be done - not going to be a hard deadline, just trying to size issues and workload


Explicitly use the  header in crashinfo.cpp

What does this PR do?
Explicitly uses <string> as a header for crashinfo.cpp.
Motivation
Windows CI was failing.  https://github.com/DataDog/libdatadog/actions/runs/9402609405/job/25900083549 This looked like it might fix the error
Additional Notes
Anything else we should know when reviewing?
How to test the change?
Describe here in detail how the change can be validated.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 69.24%. Comparing base (bcc567a) to head (fd2e507).


[Enhancement] Add execution of single tfvars file to rover

rover will execute all tfvars files in the folder specified by the var-folder flag. This is not practical if I want to execute single tfvars file in pipeline for different approval groups. It will be great if there is a var-file flag that only execute a single tfvars file.

hi @angde1990 have you tried -var-file?

@arnaudlh you are right Arnaud, -var-file works fine, closing this issue


Updated configuration documentation

Debug toolbar settings look for DEBUG_TOOLBAR_CONFIG in django settings.
Documentation implies this dict is called CONFIG_DEFAULTS.

Thanks!


[OpenAI] prompt_annotations and prompt_filter_results rename

On the service side there was a rename from prompt_annotations -> prompt_filter_results. This breaking change was meant for models Completions and ChatCompletions. In order to move forward with the release we commented out the impacted tests. This PR addresses the rename.

Added backing field for prompt_annotations to be maintained during the transition period of the renaming.
prompt_annotations is only used for deserialization and should not be visible to the user
This field is to be removed once the rename change is propagated and committed to both Completions and ChatCompletions models. (Currently, the rename is only effective in ChatCompletions)

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-ai-openai


ZEPPELIN-3277. NotebookServerTest.testMakeSureNoAngularObjectBroadcastToWebsocketWhoFireTheEvent fails

What is this PR for?
Trivial fix for flaky test, see the comment for details.
What type of PR is it?
[Bug Fix]
Todos

[ ] - Task

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-3277

How should this be tested?

CI pass

Screenshots (if appropriate)
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

hey @zjffdu thanks for making all these improvements, could we make sure there is at least one review before merging?
@Leemoonsoo


Fe/activity pending sdk

Add  MVP _Transaction component + flow (to be refactored into existing Activity/Transaction flow) that relies on Synapse SDK to track tx status for #1650

Summary by CodeRabbit

New Features

Enhanced the Portfolio Activity section with conditional rendering of "Pending" transactions.
Integrated account connectivity in the PortfolioTabManager to display transactions.
Introduced new transaction components for detailed transaction management and display.

Improvements

Implemented helper utilities for generating blockchain explorer links.
Developed custom hooks for transaction status tracking and state management.

Refactor

Created a Redux slice for transaction handling, including actions for managing transaction state.

Chores

Updated the main app component to include transaction updaters.
Added the transaction reducer to the store for state persistence.

0feb777de6897f06836eaa5c4c84623cf31c6473: synapse-interface preview link

Merged into #1695


AUTH-481: add PSa labels to openshift namespaces

openshift-operator-lifecycle-manager
openshift-operators

/hold
blocked by https://issues.redhat.com/browse/AUTH-482

Hi @liouk ^^ could you please run make manifests, then commit any changes, then run make verify just to double check?

Hi @perdasilva, thanks for pointing this out. I've committed the results of make manifests as a separate commit (90cb98fcca562ba3583c07bcb74a020d85e252fa). Also, make verify locally succeeds with the above changes.

Hi @perdasilva, thanks for pointing this out.
I've set the required values in values.yaml, however then running make manifests removes the labels from manifests/0000_50_olm_00-namespace.yaml, even if their values are defined in values.yaml. How can I correctly generate the labels in manifests/0000_50_olm_00-namespace.yaml?

/close


Minus prefix

A minus sign at the beginning excludes the matched candidates.
Ex: If there are three files.

hoge.md
hoge
mdhoge

hoge -md suggests only hoge.

Released in v8.0.0-beta3 🚀

Released in v8.0.0 🚀


chore: update uint8arrays

@vasco-santos was the dist file meant to be added?
Also, will this need to be a breaking change at all? I'm not familiar with the underlying changes to uint8arrays

Dist was my bad, added it to the .gitignore.
Breaking change is not a need here, uint8arrays has a set of utility functions that receive uint8array or string and do things with them. So, it is not instances we pass around.


[FEATURE]: drizzle-kit introspect should generate relation in generated schema.ts

Describe what you want
Currently, running drizzle-kit introspect:{dialect} command lets you pull DDL from an existing database and generate schema.ts.
However, the generated schema.ts only contains the declaration for each table.
Example:
import { pgTable, serial, text, integer, jsonb } from 'drizzle-orm/pg-core';
import { relations } from 'drizzle-orm';

export const users = pgTable('users', {
	id: serial('id').primaryKey(),
	name: text('name'),
});

export const profileInfo = pgTable('profile_info', {
	id: serial('id').primaryKey(),
	userId: integer("user_id").references(() => users.id),
	metadata: jsonb("metadata"),
});

// Missing the relation between users and profileInfo.
// Even though drizzle generated the reference correctly.

It would be useful if drizzle-kit could infer and generate the relations based on foreign key references. This approach is used by other frameworks
https://www.prisma.io/docs/concepts/components/introspection#relations
The generated schema.ts should look like this.
import { pgTable, serial, text, integer, jsonb } from 'drizzle-orm/pg-core';
import { relations } from 'drizzle-orm';

export const users = pgTable('users', {
	id: serial('id').primaryKey(),
	name: text('name'),
});

export const profileInfo = pgTable('profile_info', {
	id: serial('id').primaryKey(),
	userId: integer("user_id").references(() => users.id),
	metadata: jsonb("metadata"),
});

// Auto generated the relation
export const usersProfileInfoRelations = relations(users, ({ one }) => ({
	profileInfo: one(profileInfo, {
		fields: [users.id],
		references: [profileInfo.userId],
	}),
}));

should not be an issue on latest(0.30.0+), if it's still an issue - please reopen
drizzle-kit generates schema.ts and relations.ts


Caching proxy for Homebrew/macOS

If you have a feature request that is relevant to this repository, the runner, then please include the information below:
I'm not 100% sure this is the correct location to file this feature request
Describe the enhancement
Improve caching on macOS, especially homebrew bottles by setting *_proxy variables in the environment or setting HOMEBREW_BOTTLE_DOMAIN.
I saw that there is an ADR for supporting proxies on self-hosted or enterprise runners, but I don't know to what extent GitHub is employing caching proxies to keep things fast on the GitHub hosted runners, especially on macos. Homebrew provides multiple mechanisms for using proxies, but my experimentation seems to suggest that this may not be happening for homebrew bottles and other assets. Homebrew downloads bottles from the bintray API even for relatively common, highly used formula as can be seen in the logs. If there is any caching proxy it seems that homebrew is not using it or it's deviously sneaky.
Code Snippet
Not 100% sure where/how to implement this, but here is the relevant text from man brew:

HOMEBREW_BOTTLE_DOMAIN

By  default,  Homebrew  uses  https://homebrew.bintray.com/  as  its  download  mirror  for  bottles.  If  set,  instructs  Homebrew to instead use the specified URL. For example, HOMEBREW_BOTTLE_DOMAIN=http://localhost:8080 will cause all bottles to download from the prefix http://localhost:8080/.

http_proxy

Sets the HTTP proxy to be used by curl, git and svn when downloading through Homebrew.

https_proxy

Sets the HTTPS proxy to be used by curl, git and svn when downloading through Homebrew.

all_proxy

Sets the SOCKS5 proxy to be used by curl, git and svn when downloading through Homebrew.

ftp_proxy

Sets the FTP proxy to be used by curl, git and svn when downloading through Homebrew.

By setting one or more of these environment variables in the macos runner environment and pointing them to a caching proxy that will forward requests to the appropriate location I think macOS installation times could be reduced, as well as reducing network traffic to external sources.
If HOMEBREW_BOTTLE_DOMAIN is used, then care must be taken: For any install from https://homebrew.bintray.com things will work fine, but some 3rd party taps host their own bottles, so the caching proxy would have to have logic to determine where to fetch the bottle from if this method was used. On the other hand, setting http_proxy and https_proxy is a pretty big hammer and you'll end up caching other stuff people grab with curl etc.
Additional information
FYI, I am a Homebrew-core maintainer.
Thanks for taking the time to read this, and if you think this should be posted elsewhere please point me in the right direction. Cheers!
NOTE: if the feature request has been agreed upon then the assignee will create an ADR.   See docs/adrs/README.md

@alepauly we've never heard complaints about macOS brew installation process and this issue seems to take a lot of efforts to implement and maintenance on the Azure side

Hi @zbeekman, I like this idea but it's not something that we can support and maintain at this time. One thought is to create something like a setup-homebrew Actions that accepts the bottle domain as a parameter and can use it. This generalizes so that anyone can specify their own domain but it could be followed up with community a maintained or other centralized proxy? Or could maybe the setup could take advantage of the actions/cache and implement a proxy based off it?
and sorry for the delay in responding!

Closing since it's not something we can address in this repo. Please reopen to discuss more if needed.


chore(main): release 1.6.3-beta

:robot: I have created a release beep boop
1.6.3-beta (2024-04-17)
Bug Fixes

added babel for typescript compile (1141a73)
bug fixes (019090b)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/appblocks-hub/BB-CLI/releases/tag/v1.6.3-beta :sunflower:


Change "Latest version" to "Stable version" for packages everywhere

Is your feature request related to a problem? Please describe.
"latest" version does not always mean "newest" version. This is very confusing.
Describe the solution you'd like
Document how we use the term "latest" version.
Describe alternatives you've considered
Rename "latest" to "stable" instead?
Additional context

Must be changed in:

repo index
package index
UI (package detail page)
CLI (describe command)

To preven breaking changes, the stable version should be introduced as a new field and the (then deprecated) latest version field can be removed at a later time.

Hii @christophenne @kosmoz. I would like to work on this issue.


Code Quality: UseCurrentScale for properties window

Resolved / Related Issues

[ ] Were these changes approved in an issue or discussion with the project maintainers? In order to prevent extra work, feature requests and changes to the codebase must be approved before the pull request will be reviewed. This prevents extra work for the contributors and maintainers.
Closes #13218...

Validation
How did you test these changes?

[ ] Did you build the app and test your changes?
[ ] Did you check for accessibility? You can use Accessibility Insights for this.
[ ] Did you remove any strings from the en-us resource file?

[ ] Did you search the solution to see if the string is still being used?

[ ] Did you implement any design changes to an existing feature?

[ ] Was this change approved?

[ ] Are there any other steps that were used to validate these changes?

Open properties window for a folder
Confirm the thumbnail doesn't have a black background

Screenshots (optional)
Add screenshots here.

In my case, there is no longer any black background, either in the context menu or in the folder properties.


Allow setting of service account for telegraf-ds

Related to #77 and #92 (in fact I should have done this at the same time, but...)
This is more targeted than what #77 suggests since different configurations may want different accounts/bindings.  However, without the ability to at least set the account in the daemonset, you can't make any progress.

This issue bit me today, I think the serviceaccount mapping doesnt work as expected. I had to add more perms to a default sa to get going (which is not ideal).
I need to catch up on the code changes, i really love the helm telegraf-ds - and i may just contribute and help to get this to a good state.

hi @sudhishkasabaramesh , can you let us know how to fix this problem with existing default service account ?  I tried cluster admin role for default service account , but still i m not able to fix it.

I suggest add the serviceaccount config to the requirements documentation or maybe add yaml definition like this:
apiVersion: v1
kind: ServiceAccount
metadata:
  name: telegraf-reader
  namespace: default
---
kind: ClusterRole
apiVersion: rbac.authorization.k8s.io/v1
metadata:
  name: telegraf-cluster-reader
  labels:
    rbac.authorization.k8s.io/aggregate-view-telegraf: "true"
    rbac.authorization.k8s.io/aggregate-view-telegraf-stats: "true"
rules:
  - nonResourceURLs: ["/stats", "/stats/*"]
    verbs: ["get", "watch", "list"]
  - apiGroups: [""]
    resources: ["persistentvolumes", "nodes", "pods", "deployments", "statefulsets", "nodes/proxy"]
    verbs: ["get", "watch", "list"]
---
kind: ClusterRole
apiVersion: rbac.authorization.k8s.io/v1
metadata:
  name: telegraf-reader-role
aggregationRule:
  clusterRoleSelectors:
    - matchLabels:
        rbac.authorization.k8s.io/aggregate-view-telegraf-stats: "true"
    - matchLabels:
        rbac.authorization.k8s.io/aggregate-view-telegraf: "true"
    - matchLabels:
        rbac.authorization.k8s.io/aggregate-to-view: "true"
rules: [] # Rules are automatically filled in by the controller manager.
---
apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRoleBinding
metadata:
  name: telegraf-reader-rolebinding
roleRef:
  apiGroup: rbac.authorization.k8s.io
  kind: ClusterRole
  name: telegraf-reader-role
subjects:
  - kind: ServiceAccount
    name: telegraf-reader
    namespace: default

With this config kube_inventory input plugin works fine, but kubernetes input not working... always displays this error:
2020-04-03T09:09:10Z E! [inputs.kubernetes] Error in plugin: https://kubernetes/stats/summary returned HTTP status 404 Not Found

@jmorcar have a look at https://github.com/influxdata/helm-charts/tree/master/charts/telegraf-ds
This repo (tick-charts) is replaced by helm-charts one and we fixed the daemonset deployment


Remove orphaned entities

Search through data.raw to find everything that was referenced before Pacifist removed whatever it removes and is no longer referenced after and remove all of those things as well. This effectively removes things like corpses/remnants of removed entities and particles/explosions related to weapons/combat robots that get removed. This PR also adds some new dummies to cover some categories that now end up empty. (Note the dummies change may conflict with #57, but they should be straight-forward to merge cleanly as the changes are actually either identical or just next to each other, not overlapping.)
Note the way this checks for references just looks for any strings matching the names of entries in data.raw, which technically is an overapproximation, meaning that it may think something is referenced when it isn't, which would just result in that thing actually being left in (and therefore visible in the rich text icon selection GUI) accidentally. This is unlikely to happen in practice. The "right" solution would be to determine which strings are EntityIDs or whatever *IDs and only treat those as references, but that seems like overkill here. (I might write the code to do that anyway since someone asked for it in the Factorio Discord when I asked about this, but would probably not bother using it for this project.)
After this pass, there are some explosions left for friendly buildings getting damaged and the atomic bomb explosion for the reactor being destroyed. Those explosions started showing up in the rich text icon selection GUI, so I added a couple lines to PacifistMod.remove_misc() to hide them. I then confirmed that destroying a reactor still produces an atomic bomb explosion as expected.
Together with #57, this should fix #56.

I tried to resolve the conflicts after mergin the other PR, but now I get this and am not quite sure what the reason is:

I couldn't reproduce that error message. But when investigating I noticed that damage-type references are in fields of name type so they weren't getting handled properly anyway since I was ignoring all fields of name type because they're usually the type of that object, not a reference to something else. So I changed the code to never delete damage-types. DamageType can't reference other things anyway, so this won't result in keeping other objects around.
At the same time, it occurred to me that this code shouldn't delete anything labeled as an exception due to the mod rules, so I added a check for that as well. Not sure it does anything, but it would be confusing if it accidentally overrode the exceptions.

I'll have a look at it again in the next days.

Okay, sounds like this strategy doesn't work in general as we can't tell which entities mods will reference by name in their own code. The options I see are:

Abandon this.
Hide instead of remove. (I just pushed this version)
For each mod with compatibility issues, hard code a list of entities that have references (these would be added to the exceptions collection).

(3) sounds brittle and like a lot of work for minor gain, so I'd lean towards (2). Noting that if it accidentally hides extra things, we can add them to the exceptions, but I doubt that's likely to be a problem.
One problem with (2) is that it doesn't hide combat robots, so I want to extend it with an option to "pretend delete" entities, so it will hide combat robots and everything they reference since removing them and everything they reference isn't working for all mods.
(I saw that SeaBlock error before on a different branch and fixed it by marking SeaBlock as a dependency so it would load before Pacifist... but then I got a different error, which I why I never pushed that change.)

With these changes, I got a similar error on Exotic Industries about stone-wall (that is, once I made sure destroyer-remnants wasn't being removed. Adding it to mods_require_walls fixed it, but I'm not sure what's going on if you had tested it before and not needed that.

I'll run it against my test save games later this week when I'm back home. I am thinking about logging which types of things get hidden by this and maybe restrict the algorithm to things like remnants etc that actually show up in the signals and RTF icons.
The defender remnants maybe should be added to the exceptions for Exotic Industries either way, but I'll look into that later as well.
Could you tell me how to reproduce the stone-wall issue with EI? I've been running a save with EI and Pacifist for a while without issues. Maybe you have different mods or mod settings?
In any case, thank you again for the work!


Hi! Sharing?

hello friend, any chance to share the code of this project with me?

Hi, now i'm on big update of ammo.lab
maybe is my next big update

Ok, Thanks!


On popup modal shows "cancel" but not "ok"

This is a really nice addition of date picker. I have tested in mac chrome,safari & ios 9 and I am missing the ok button. It is really there just below the modal,unseen on the phone.

I have changed css to display:none the cancel button for now as they can exit via the x button on top..
.dtp-btn-cancel {   display: none !important; }
and now I get

I am using material-bootstrap via
http://fezvrasta.github.io/bootstrap-material-design/bootstrap-elements.html
Also I have loaded

Thanks for the product again, this thing looks great

The problem is fixed.
Can you confirm it works fine ?
Thank you

I changed out the old version for the new and removed my css none display block and still got the same result


created Pi_generation code

made a global variable PI

This is not a propper algorithm. This would be closed.


Sort out issue with first cell of notebooks.

That is not showing the title an name of the authors.

This is resolved with move to simpler formatting as a part of #21


Scaling over multiple displays stops rendering characters

VSCode Version: 1.25.0 Insiders 28d5806
OS Version: W7x32

Steps to Reproduce:

Open multiple files in a project
Snap VScode to the left side of your leftmost display
Drag the right border to the right side of your next display
Stop seeing characters in the code windows

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

It's quite hard to get some kind of rule from it. In a horizontal way (as in screenshots 3,4), it's quite easy for me to replicate. It even occurs, if the 3rd tile is split again (like screenshot 1,2).

I could not figure out, who it works if I go and stack the tiles vertically. It stop redenering once, but I could not redo it.

@bgraeb does this reproduce in our stable release too?

@bpasero No, does not seem to.

@bgraeb does it reproduce if you run code --disable-gpu ?

@bpasero It's extremely slow, if I rescale the windows (like 15 seconds to rearrange the sidebar), but it the characters are rendered. So it's in a way usable.

@bgraeb how about "code --disable-accelerated-2d-canvas" instead?

@bpasero Characters vanish again. Interestingly, not on the right hand side.
The rescaling time is back to normal

@bgraeb do you have another more modern OS to try with, like Windows 10?

I could try MacOs or a W10-VM later this day.

@bpasero Same issue on MacOS. Need to verify on Windows too?

@bgraeb crazy, I cannot reproduce on Windows 10 so far. Can you show me a screenshot how this looks for you on macOS (just one please). Also, what are your settings?

Just updated: Looks like it got worse.

Settings are:
{
    "window.zoomLevel": 0,
    "workbench.iconTheme": "material-icon-theme",
    "files.autoSave": "onFocusChange",
    "files.associations": {
        "*.xaml": "xml"
    },
    "files.insertFinalNewline": true,
    "files.trimTrailingWhitespace": true,
    "files.defaultLanguage": "shellscript",
    "terminal.integrated.shell.osx": "zsh",
    "editor.cursorStyle": "line",
    "editor.cursorBlinking": "expand",
    "terminal.integrated.cursorStyle": "line",
    "terminal.integrated.cursorBlinking": true,
    "terminal.integrated.shell.windows": "powershell.exe",
    "terminal.integrated.rightClickCopyPaste": true,
    "editor.minimap.renderCharacters": false,
    "terminal.external.windowsExec": "powershell",
    "terminal.external.osxExec": "Terminal.app",
    "git.confirmSync": false,
    "editor.showFoldingControls": "always",
    "workbench.activityBar.visible": true,
    "extensions.autoUpdate": true,
    "editor.smoothScrolling": true,
    "workbench.panel.location": "right",
    "material-icon-theme.showUpdateMessage": false,
    "markdownlint.config": {
        "MD029": false
    },
    "gitlens.advanced.messages": {
        "suppressCommitHasNoPreviousCommitWarning": false,
        "suppressCommitNotFoundWarning": false,
        "suppressFileNotUnderSourceControlWarning": false,
        "suppressGitVersionWarning": false,
        "suppressLineUncommittedWarning": false,
        "suppressNoRepositoryWarning": false,
        "suppressResultsExplorerNotice": false,
        "suppressShowKeyBindingsNotice": true,
        "suppressUpdateNotice": true,
        "suppressWelcomeNotice": true
    },
    "gitProjectManager.baseProjectsFolders": [
        "~/Tresors"
    ],
    "gitProjectManager.maxDepthRecursion": 3,
    "gitlens.keymap": "chorded",
    "workbench.startupEditor": "newUntitledFile",
    "terminal.integrated.copyOnSelection": true,
    "workbench.list.multiSelectModifier": "alt",
    "extensions.showRecommendationsOnlyOnDemand": true,
    "gitlens.historyExplorer.enabled": true
}

@bgraeb does it also reproduce if you run "code-insiders --user-data-dir <some dir>" to really start a fresh session of VS Code?

I foul out, that the rendering stops, when the powershell console below is about 686 in bufferSize, maybe this helps you. If I expand it a little more, it breaks. (btw. Console rendering keeps working.
If I have the window opened like that and expand the file explorer view, the max-buffer size "scale" drops to around 651.
If I open the settings view, only this stops rendering. You can see that below.

About your question:
Just tried it, same issues arise. Interestingly, the welcome-screen stays rendered.

@bgraeb so do you say it does not reproduce if you do not have a terminal open in VS Code?

Terminal does not seem to change a thing. Some of the screenshot show the broken rendering, whilst rendering in the console is intact.

@bgraeb some things to try on those editors where no characters are rendered:

does resizing make them appear eventually (when you make the editor small)
does zooming in (via View menu) changes anything?
does it change when you change the font used for the editor?
does it change when you disable the minimap?

Thank you, for still trying to figure it out.

Question
W7
MacOs

does resizing make them appear eventually (when you make the editor small)
yes
Yes

does zooming in (via View menu) changes anything?
No
No

does it change when you change the font used for the editor?
No
No

does it change when you disable the minimap?
No
No

I'm still trying to figure out, if there is a scheme in editor count or orientation.
My findings so far:

Welcome-Pages, Markdown-Previews, console and Sidebar do not show this issue.
The window has to be big (like 4000px+)
It does not occur, if only two editor-panes are opened.
it seem to occur, if I add at least 3 panes in horizontal order
It does make a difference, if two of these 3 panes are also split vertically. If I have 1 on the left, 1 on the lower right and 2 above this lower right, these two will stop to render.
If I only have 1 on the left, one on the lower right and the settings-pane above it (in the split-view like style), only the settings view is not rendered.
If I have only 3 panes in horizontal order, they will stop rendering, but only if I expand the window way further

@bgraeb thanks for the response. I am still trying to understand if this is a general problem with the UI framework we are using or not. Can you give Atom Beta a try (https://atom.io/beta) to see if you see the exact same thing? They are also shipping with the same version of Electron that we are.

@bgraeb if you are up for a little experiment, can you:

quit any running VSCode Insiders
open C:\Program Files\Microsoft VS Code Insiders\resources\app\out\main.js
find the place where it reads app.commandLine.appendSwitch("disable-mojo-local-storage");
right after paste this: app.commandLine.appendSwitch('force-color-profile', 'srgb');
save (you will need to save as admin I guess)
start VSCode Insiders again
=> does the issue still reproduce?

The change in main.js didn't fix it... I'll try if atom behaves the same, later this day. If you want, maybe we could give it a shot via Skype or something.

@bgraeb thanks for suggesting. in what timezone are you in? I could do a call in the afternoon (Swiss time) probably.

Germany, so not way different, as far as I know ;-)

@bgraeb how does 4 PM sound today? My Skype alias is bpasero
PS: are you able to setup your machine to run VSCode out of its sources as outlined here: https://github.com/Microsoft/vscode/wiki/How-to-Contribute#build-and-run
That would probably make debugging a bit easier.

Should work. I'll try to setup the build process.

Thanks, I can reproduce now. Steps:

configure "Displays have separate spaces" to be unchecked
log out and log in
use the highest resolution on the mac display
connect a secondary display
open a VSCode window over 2 monitors all the way to the end
open 3 splits
make the last editor almost at the end
notice how characters are not drawn

@bpasero Hey! Do you have an update on this?

No

Filed to Electron as https://github.com/electron/electron/issues/13648

Happy to say this appears to be fixed with Electron 3.0.x. Build to verify: Download

Downloaded new VSCode version1.28 for Windows (VSCodeUserSetup-x64-1.28.1.exe). I do not use Electron just VSCode.... Issue still persists... Does it means that actual bug was not yet fixed? Please advise.


Update Examples to use MRTK v2.4

Also updates to Unity 2018.4.17f1 (LTS)
Also includes exposing anchor graph maintenance parameters through WorldLockingContext.

/azp run

/azp run


Update HWT links with the new domain

update HWT url to http://hohmannwannamakertrust.org/

:+1: Thanks Santi!


Update hello.md

K says hi!

Sorry I missed this! Would you still like me to merge it?

No I added this a few days ago but then I recently found out a prominent viewer of yours is called K so I decided to remove it


Modify claim operation to 50 claims per tx

This PR modified claim gas operation to limited 50 claims per transaction. This may create more than 1 ClaimTransaction during 1 cmd but make it possible to claim gas from 10k transactions. Extra output is provided to show the progress.

Can you change it to send only one transaction at a time by default?
You can add a command line option:
claim gas [all]

Send one transaction only without all, otherwise send multiple.

@erikzhang Sure, now it's an optional choice.

https://github.com/neo-project/neo-cli/blob/13773f78ae7f7ccc78ae0aca1c7ff2f86a499f40/neo-cli/Shell/Coins.cs#L70
Shouldn't this line be:
Claims = claims.Take(MAX_CLAIMS_AMOUNT).ToArray(),

?

:yum: , that's comprehensive.

@erikzhang maybe some change like that? https://github.com/neo-project/neo-cli/pull/236/commits/419ef1159cf388b030fe98f5a1cf981a6b30a499


Errors in ConfigFiles.pm

The following errors occur when generating configuration files for mids:
[2016-09-20 16:37:03,296] [DEBUG] GET "/genfiles/view/772/all".
[2016-09-20 16:37:03,299] [DEBUG] Routing to controller "UI::ConfigFiles" and action "genfiles".
Use of uninitialized value $org_server in concatenation (.) or string at /home/dgelin200/cdn/traffic_control/traffic_ops/app/lib/UI/ConfigFiles.pm line 225.
Use of uninitialized value $org_server in concatenation (.) or string at /home/dgelin200/cdn/traffic_control/traffic_ops/app/lib/UI/ConfigFiles.pm line 225.
Use of uninitialized value $org_server in concatenation (.) or string at /home/dgelin200/cdn/traffic_control/traffic_ops/app/lib/UI/ConfigFiles.pm line 225.
Use of uninitialized value $org_server in concatenation (.) or string at /home/dgelin200/cdn/traffic_control/traffic_ops/app/lib/UI/ConfigFiles.pm line 225.
DBIx::Class::Storage::DBI::select_single(): Query returned more than one row.  SQL that returns multiple rows is DEPRECATED for ->find and ->single at /home/dgelin200/cdn/traffic_control/traffic_ops/app/lib/UI/ConfigFiles.pm line 1295
[2016-09-20 16:37:06,773] [DEBUG] MID PARENT.CONFIG:
Removed
Use of uninitialized value in hash element at /home/dgelin200/cdn/traffic_control/traffic_ops/app/lib/UI/ConfigFiles.pm line 948.
Use of uninitialized value in hash element at /home/dgelin200/cdn/traffic_control/traffic_ops/app/lib/UI/ConfigFiles.pm line 948.
Use of uninitialized value in hash element at /home/dgelin200/cdn/traffic_control/traffic_ops/app/lib/UI/ConfigFiles.pm line 948.
Use of uninitialized value in hash element at /home/dgelin200/cdn/traffic_control/traffic_ops/app/lib/UI/ConfigFiles.pm line 948.

Fixed.


Fix missing scope in habtm.rb

Fixes #206

@serprex Do you have an idea why tests are failing? It looks like there is Hash passed as attribute instead of the options?
Failure/Error: has_and_belongs_to_many_without_tenant(name, scope, **options, &extension)

NoMethodError:
  undefined method `arity' for {:tenant_column=>:account_id, :tenant_enabled=>true, :tenant_class_name=>"Account"}:Hash
# ./lib/activerecord-multi-tenant/habtm.rb:13:in `has_and_belongs_to_many_with_tenant'
# ./spec/schema.rb:177:in `<class:Manager>'
# ./spec/schema.rb:174:in `<top (required)>'
# ./spec/spec_helper.rb:85:in `<top (required)>'
# ./spec/activerecord-multi-tenant/associations_spec.rb:1:in `require'
# ./spec/activerecord-multi-tenant/associations_spec.rb:1:in `<top (required)>'

Yes. I'm not sure what's wrong

Tests are green now :)

Great. Can merge once you agree to CLA

I think I did that but I'm not sure. What else should I do?

Could you try posting @microsoft-github-policy-service agree again?

@microsoft-github-policy-service agree

Would you be willing to release this as a new version please so that I don't have to pull gem from master branch? Thank you


Dont show convert to relative path for files in local typings or stubs path.

repro steps.

open a folder test
create a folder called typings inside of test,
create a file in typings, lib.pyi
in lib.pyi create class class MyType: ...
create a test.py in the test folder
write from lib import MyType
set cursor on lib

Expected:
no code action to convert lib to relative path
Actual:
it offers convert to relative path, but if selected, nothing happen since we treat any modules under local typings or stubs as third party libraries not user files for relative path purpose.

related to https://github.com/microsoft/pylance-release/issues/5427

This issue has been fixed in prerelease version 2024.3.100, which we've just released. You can find the changelog here: CHANGELOG.md


Add simple-dom tutorial and README section

Undecided if this is going to be part of the lib. Closing for now

Undecided if this is going to be part of the lib. Closing for now


Functor types

Why don't we create an abstract type Functor that has the API:
dom(F::Functor)::Theory
codom(F::Functor)::Theory
function (F::Functor)(x::Ob{:generator})
    ...
end
function (F::Functor)(f::Hom{:compose})
    foldl(compose, map(x->F(x), f.args))
end
function (F::Functor)(f::Hom{:generator})
    ...
end

The idea being that the functor type creator just has to specify what happens on the generators of the theory and the term constructors. Then the MonoidalFunctor <: AbstractFunctor would have the following additional method definitions:
function (F::MonoidalFunctor)(x::Ob{:otimes})
    foldl(otimes, map(F, x.args))
end
function (F::MonoidalFunctor)(f::Hom{:compose})
    foldl(compose, map(F, f.args))
end
function (F::MonoidalFunctor)(f::Hom{:otimes})
    foldl(otimes, map(F, f.args))
end

In this setting, in order to make a functor, you would just have to define a type MyFunctor<:MonoidalFunctor which supplies the action on the generators. We could supply a PresentedMonoidalFunctor which works like the current implementation by storing a dictionary mapping the generators to their images under the functor.
struct PresentedMonoidalFunctor{K,V} <: MonoidalFunctor
    g::Dict{K,V}
end
function (F::PresentedMonoidalFunctor)(x::Ob{:generator})
    F.g[x]
end
function (F::PresentedMonoidalFunctor)(f::Hom{:generator})
    F.g[x]
end

I think we could simplify a lot of this code if we store the GATExprs with the
operation as a Function instead of a Symbol. The current implementation looks
like:
julia> X = Ob(FreeSymmetricMonoidalCategory, :X)
julia> dump(X)
Catlab.Theories.FreeSymmetricMonoidalCategory.Ob{:generator}
  args: Array{Symbol}((1,))
    1: Symbol X
  type_args: Array{GATExpr}((0,))

Instead of storing head(ex) == :otimes we could store the function otimes, this would allow a single abstract functor type with the idea that the codomain category will support all the term constructors of the domain category so that you can implement arbitrary GAT functors:
function (F::Functor)(x)
    foldl(head(x), map(F, x.args))
end

#you still need a base case for generators
Generator = Union{Ob{:generator}, Hom{:generator}}
function (F::Functor)(x::Generator)
    F.g[x]
end

Sounds good to me.

In the @theory macro, could we generate an abstract type representing maps between instances of the theory that only made you define the map on generators?

That would be really cool!

Well, don't get too excited; I'm working out Functors and MonoidalFunctors now, and we'll see how that goes before I attempt that.

Totally agreed about storing Function instead of Symbol for term constructors like compose and id. This should allow us to eliminate complicated reflection machinery in invoke_term and also get a performance boost by going through Julia's standard dispatch mechanism.


How to set the cross domain of a single route

How to set the cross domain of a single route

can you please expand more on your question. What versions are you using? What have you done already to try?


casterName

is it possible to implement the function of displaying who casted buff/debuff?

In core/shared.lua, add a new IDTip.kinds line:
IDTip.kinds = {
	...
	caster = "Caster",
	...
}

Then in clients/shared.lua, change the "SetUnitAura" hooksecurefunc to:
hooksecurefunc(GameTooltip, "SetUnitAura", function(self, ...)
	local id = select(10, UnitAura(...))
	IDTip:addLine(self, id, IDTip.kinds.spell)

	-- add caster name in tooltip
	local caster = select(7, UnitBuff(...))

	-- but if caster name is "player" (ourself), don't bother including it.
	if caster and caster ~= 'player' then
		local name = select(1, UnitName(caster))
		if name then
			IDTip:addLine(self, name, IDTip.kinds.caster);
		end
	end

end)

works with buffs, but does not display the name of the caster on debuffs on dummy for example.
and also in a duel on debuffs applied by me it writes the name of the target, not mine, as well on debuffs on me do not write the name of the player who applied it
mine debuffs

my debuff on enemy

debuffs on dummy

without addons but with idtip doesn't work at all

Should be included in idTip itself in the next release with nameplate aura support also.

Should be included in idTip itself in the next release with nameplate aura support also.

Thanks for the answer!
It doesn't work the way i meant in my request.
For example:
instead of the name of the npc I get "nameplateNumber" and instead of my name on my buff I get "player"


Add placeholder text to service update search

Was asked today how to search for audit events with specific dates in the admin app and I had to check to code to find out.
Adding placeholder text as a stopgap solution until we decide to overhaul this search page.

:+1:
+2 for the branch name.

👍

Placeholder text is evil, but as a stopgap it should still go out.


Keep getting this error: Cannot read property 'kind' of undefined

Bug Report or Feature Request (mark with an x)
- [x ] bug report -> please search issues before submitting
- [ ] feature request

Command (mark with an x)
- [ ] new
- [ ] build
- [x ] serve
- [ ] test
- [ ] e2e
- [ ] generate
- [ ] add
- [ ] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Versions

Angular CLI: 7.1.2
Node: 8.11.4
OS: win32 x64
Angular: 7.1.2
... animations, cli, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router
Package                           Version
@angular-devkit/architect         0.11.2
@angular-devkit/build-angular     0.11.2
@angular-devkit/build-optimizer   0.11.2
@angular-devkit/build-webpack     0.11.2
@angular-devkit/core              7.1.2
@angular-devkit/schematics        7.1.2
@angular/cdk                      7.1.1
@ngtools/webpack                  7.1.2
@schematics/angular               7.1.2
@schematics/update                0.11.2
rxjs                              6.3.3
typescript                        3.1.6
webpack                           4.23.1
Repro steps
Happens many times a day when saving a file and the cli re-compiles
The log given by the failure
Failed to compile.
: TypeError: Cannot read property 'kind' of undefined
at getAssignmentTargetKind (node_modules\typescript\lib\typescript.js:10049:28)
at Object.isAssignmentTarget (node_modules\typescript\lib\typescript.js:10092:16)
at checkObjectLiteral (node_modules\typescript\lib\typescript.js:46107:45)
at checkExpressionWorker (node_modules\typescript\lib\typescript.js:50748:28)
at checkExpression (node_modules\typescript\lib\typescript.js:50696:42)
at checkExpressionWithContextualType (node_modules\typescript\lib\typescript.js:50524:26)
at checkApplicableSignature (node_modules\typescript\lib\typescript.js:47963:35)
at chooseOverload (node_modules\typescript\lib\typescript.js:48281:26)
at resolveCall (node_modules\typescript\lib\typescript.js:48231:26)
at resolveCallExpression (node_modules\typescript\lib\typescript.js:48534:20)
at resolveSignature (node_modules\typescript\lib\typescript.js:48800:28)
at getResolvedSignature (node_modules\typescript\lib\typescript.js:48844:26)
at checkCallExpression (node_modules\typescript\lib\typescript.js:48936:29)
at checkExpressionWorker (node_modules\typescript\lib\typescript.js:50759:28)
at checkExpression (node_modules\typescript\lib\typescript.js:50696:42)
at resolveDecorator (node_modules\typescript\lib\typescript.js:48741:28)
Desired functionality
No errors
Details:
This started when Angular 7 was released. Unfortunately I wasn't able to find much information about this issue. Neither on Stackoverflow nor in the issue trackers. Well at least not since the release of Angular 7

@therawk have same @angular-devkit/build-angular 0.11.2, full recompile fixes it

We’d need a bit more to look into this for example a code snippet that is causing this error.
If you instrument the checkSourceFile function in typescript.js to print node.path on entry, you can at least check what file started the chain of operations that led to the crash. Typescript don’t have any builtin functionality for tracking checker crashes.

@alan-agius4 Uh :|
To be honest I have no idea how to do that. I searched google but the only thing I found are actual error logs.
Would you mind telling me what I need to do? Sorry

@therawk

Open node_modules/typescript/lib/typescript.js
locate function checkSourceFile
Do a console.log(node.path)

Execute build/serve command.

I just tried this and here's the output. Does this help at all?

By any chance do you have forkTypeChecker set to false in your angular.json?

Ok. I'll migrate my project to a new one - wanted to re-work the design anyway. If I notice something I'll report here.
The error is just annoying. As @djleonskennedy said recompiling fixes it

Let us know (in a new issue) if you hit this issue again and you notice a pattern for reproduction.

Simply stop application and execute "npm install" again. It fixed my issue.

The problem in our case was that the locally installed version of @angular/cli was different to globally installed version of @angular/cli.

I am getting this issue as well despite updates and making sure that local/global cli are same version

I am still getting this issue with ngx-build-plus.
I simply use the ng build --watch --project=dummy-project and the recompile turns into this error.
@angular-devkit/architect  0.13.6
@angular-devkit/build-angular  0.13.6
@angular-devkit/build-optimizer  0.13.6
@angular-devkit/build-webpack  0.13.6
@angular-devkit/core  7.3.6
@angular-devkit/schematics  7.3.6
@angular/cli  7.3.6
@ngtools/webpack  7.3.6
@schematics/angular 7.3.6
@schematics/update  0.13.6
rxjs  6.4.0
typescript  3.2.4
webpack 4.29.6

Hi All, I just installed angular ssr and this is happening to me too.

Ignore my previous comment. It was just high hope. It did not happen for a while so I thought it is fixed but the error started to appear again.

Your source contains var x , when edit to var x= int or string or any then error fixed

The same mistake in Angular 7 project. Observation has shown for several weeks that this may occur at different moments of work, the strangest of them, when the application is just running locally, I do not work in it for several hours, after a while, the application automatically recompiles and I get this error

@myWorkingGithub: I have the same thing: usually when the app is running locally, and I haven't touched it for about half an hour, I get the error... Also Angular 7.

did anyone has found solution for that problem? if yes please let me know i am facing same problem

Please solution!
This error appears randomly when updating a ts file that imports swal!!!

@myWorkingGithub, @izzeda, @shripadgodse, @dsldiesel
Kindly Stop compiler with cntr+c & Recompile entire project.
This will end the error.

@myWorkingGithub, @izzeda, @shripadgodse, @dsldiesel
Kindly Stop compiler with cntr+c & Recompile entire project.
This will end the error.

But this is temporary solution. Same issue is happening after some time

I'm getting the same error. Also on Angular 7 (7.2.5) with TypeScript 3.2.2.
It happens randomly after changing a file and recompilation.

@myWorkingGithub, @izzeda, @shripadgodse, @dsldiesel
Kindly Stop compiler with cntr+c & Recompile entire project.
This will end the error.

But this is temporary solution. Same issue is happening after some time

If you replace any file while the server is initialized then you will get these error every time. If you just edit your file then it would simply recompile.
@klinki
You should also stop the server before changing/replacing the file.

@harshsyno I see. I don't usually replace files. (I assume by replacing you mean deleting the file and creating it again?). But this might be related to git (maybe changing branch or something...). I'd expect it to just update the file, but maybe that's not the case.

@harshsyno I see. I don't usually replace files. (I assume by replacing you mean deleting the file and creating it again?). But this might be related to git (maybe changing branch or something...). I'd expect it to just update the file, but maybe that's not the case.

@klinki I understand your concern. But the solution I got working is simply starting the server again.

Same problem here... but it works when i only do a normal ng serve. But when i try to do a ng serve --configuration=prodution it fails with the same error message mentioned above.

just do ionic serve again
it's worked for me

What's the reason to close this issue. It seems to me this is not resolved at all.

Hi all, the above issue should have been solved with https://github.com/angular/angular-cli/pull/13992 which is available in @ngtools/webpack 7.3.8`.
Please update to the most recent Angular CLI version.
If the problem persists after upgrading, please open a new issue, provide a simple repository reproducing the problem, and describe the difference between the expected and current behavior.
Thanks.

@jer-tx , I had the same problem.
What helped to me was to delete both "node_modules" and "package-lock.json". Then I run the command
ng add @nguniversal/express-engine --clientProject <angular-project>
and it complained about missing node modules. I installed them manually one by one (only what was really necessary) . And finally it worked!
After that you can re-install everything else by
npm run i


Eufy / Homebridge error

Hello,
I always get the following error (also attached) when adding eufy plugin to home bridge. I rested the plug-in and home bridge but still getting the same issue.
how to solve?
2021-05-09, 9:04:55 AM [eufy] Add Accessory: Aquarium - 634496A2F6234697
2021-05-09, 9:04:56 AM [eufy] Error connecting to accessory: Error: Socket closed without sending response
at Timeout. (/home/hoobs/.hoobs/node_modules/node-eufy-api/lib/tcp-socket.js:153:48)
at listOnTimeout (internal/timers.js:549:17)
at processTimers (internal/timers.js:492:7)
2021-05-09, 9:04:56 AM [eufy] Removing accessory: Aquarium

This looks suspiciously like this issue: https://github.com/sebmos/node-eufy-api/issues/2 (and the corresponding issue in the original Python version of this package: https://github.com/google/python-lakeside/issues/11). Unfortunately, there's no workaround - Eufy needs to fix the firmware on these light bulbs to solve this issue.

Ah ok!
Thank you so much for your reply.
On Sun, 9 May 2021 at 17:34 Sebastian @.***> wrote:

This looks suspiciously like this issue: #2
https://github.com/sebmos/node-eufy-api/issues/2 (and the corresponding
issue in the original Python version of this package:
google/python-lakeside#11
https://github.com/google/python-lakeside/issues/11). Unfortunately,
there's no workaround - Eufy needs to fix the firmware on these light bulbs
to solve this issue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sebmos/node-eufy-api/issues/17#issuecomment-835819037,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQJ6STDCQOJ3LIHKLYROOO3TM2MNTANCNFSM44OSGZ3A
.

--
Regards,
Shaheen Alkhudhari


RuntimeError: module compiled against API version 0xb but this version of numpy is 0xa. What's woring with it. Thanks for help

Traceback (most recent call last):
File "init.pxd", line 987, in numpy.import_array
RuntimeError: module compiled against API version 0xb but this version of numpy is 0xa
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/media/wxl/00063DAF000FECC3/FastMaskRCNN-master/train/train.py", line 28, in
from libs.datasets import download_and_convert_coco
File "/media/wxl/00063DAF000FECC3/FastMaskRCNN-master/train/../libs/datasets/download_and_convert_coco.py", line 17, in
from libs.datasets.pycocotools.coco import COCO
File "/media/wxl/00063DAF000FECC3/FastMaskRCNN-master/train/../libs/datasets/pycocotools/coco.py", line 55, in
from libs.datasets.pycocotools import mask as maskUtils # change importing
File "/media/wxl/00063DAF000FECC3/FastMaskRCNN-master/train/../libs/datasets/pycocotools/mask.py", line 3, in
import libs.datasets.pycocotools._mask as _mask
File "_mask.pyx", line 23, in init libs.datasets.pycocotools._mask
File "init.pxd", line 989, in numpy.import_array
ImportError: numpy.core.multiarray failed to import
I reinstall numpy. But the problem is still exist. I donn't know how the fix it. Thanks for your answers

I upgrade numpy. The numpy is 1.12.1

https://github.com/matterport/Mask_RCNN
Look at this repo.. They have implemented it. All the best..

I fixed this by reinstalling all the dependencies (including numpy) in a dedicated virtualenv.
virtualenv -p python3.5 py35
. py35/bin/activate
pip install ...


reid - how to use reid?

Hi,
I looked at the demo and explanation here: https://github.com/tryolabs/norfair/tree/master/demos/reid
At high-level, I understand what's going on but is it possible for you to explain what's happening in embedding_distance() here?:
https://github.com/tryolabs/norfair/blob/009a1b171ab14336d79d7b7b02dfa5f45066c79e/demos/reid/src/demo.py#L15
I want to do something similar for face reid and I have a way of representing faces using embeddings. I want to understand what are you doing here: what embeddings are you comparing against what? what kind of distance are you using?,  What are matched_not_init_trackers, unmatched_trackers, snd_embedding?, etc.
Appreciate your help with this! Thanks!

Hello. This will be a long answer, so sorry for making you read all this, hopefully it will make sense.
As you can see in the Detection class, you can provide an embedding to the detections. The Tracker will be storing embeddings of the associated TrackedObjects that it generates.
We don't store all the embeddings of each detection associated to a tracked object, since that could require lots of memory to store all the embeddings, but you can tell the Tracker how many embeddings should it have available at once for each tracker by setting the past_detections_length argument to whatever number you would like.
Now I will try to explain this embedding_distance thing. The Tracker has two distances, which you can see in it's arguments when defining the Tracker instance:
The distance_function:
This one is used to match Detection instances with TrackedObject instances. So whenever you call the Tracker.update passing your detections, norfair will use that distance_function to see which detection corresponds to which object you were already tracking, and which detections might be related to a new object you have never seen.
It typically uses positional information (i.e: where the TrackedObject and the Detection are located in the image), specially considering that for TrackedObjects that were created just a few frames ago you might not have that many embeddings available to compare it with a Detection, since it was just created. Of course, you could use both positional information and information from the embeddings if you wanted inside that distance_function, but is not the way we normally use it.
The embedding_distance:
This one is used to associate different TrackedObject instances. The thing is that the previous method with the distance_function might not work perfectly, and we might need to check if two different TrackedObject instances could actually correspond to the same object in real life.
Lets put an example: if the object suddenly and abruptly accelerates and then the corresponding Detection and TrackedObject are far from each other. In that case, the associated TrackedObject will not match with that (or probably any) Detection, becoming part of the list of unmatched_trackers, and will remain there until it matches with a Detection again, or until it is destroyed by the Tracker if enough frames without matching anything have happened.
On the other hand, since that Detection didn't match with any previous TrackedObject, the Tracker will create a new TrackedObject. This new TrackedObject will not be yet be returned by the Tracker when you call the Tracker.update method, since the Tracker waits to have evidence for many frames that the object actually exists and it wasn't just a random false positive of your detector. Therefore, this new TrackedObject is part of the list of matched_not_init_trackers, and will remain there until the Tracker 'initializes' it (i.e: has enough evidence for many frames to consider it a real object and starts returning it in the Tracker.update method), or until the Tracker destroys it if it doesn't match future detections.
One might be able to recognize that an object in the matched_not_init_trackers list might actually correspond to a tracker in the unmatched_trackers list by using the embedding_distance, to then 'merge' the new uninitialized tracker with the old unmatched tracker to a single TrackedObject instance with the old id.
This embedding_distance usually uses information from the embeddings, since positional information might not be very useful because the TrackedObject was lost due to the erratical movement of the object.
The demo
Now I will answer your specific questions from that demo. In that demo, we are using just a color histogram of the bounding box (you can see it in the function get_hist, that is called in this line when defining the embedding).
The snd_embedding variable (read it as 'second embedding') in out embedding_distance, is an embedding of what we called the old unmatched tracker in the previous explanation. So first we try to get the embedding of the last detection we had for that unmatched tracker, if we don't have an embedding available in that detection, then we start looking at the list of past detections (remember that we store only a few of those) starting from the latests and see if we have an embedding available to put in that variable.
Similarly, we look for past embeddings available for the new not yet initialized TrackedObject, and compare them with the embedding of the old unmatched Tracked Object. The distance we use in that demo is the correlation of the color histograms (as you can see in this line, where we compare the histograms with cv2.HISTCMP_CORREL.

@aguscas , Thanks a lot for such a great explanation! It makes perfect sense! I'll use this to come up the logic for my tracking. Closing the issue for now and will open a new one later if I have any further questions.
Again, I really appreciate the time you took to explain it! Have a great day!


add 1.11 on CI and migrate to TestItemRunner

CI errors are caused by problems with change_plotly_version in 1.11, which will be addressed in the next PR for migrating to scratch spaces


Add a duplicate observation checker that checks all places and stat vars

related issue: issue #68

Thanks for the review!


Improve the documentation about domain and translation

Hello,
I find the documentation http://docs.readthedocs.org/en/latest/alternate_domains.html and http://docs.readthedocs.org/en/latest/localization.html not verry clear in the case of translation.
For me the following questions are not clear (and because I don't know how to do it I can't make pull request to fix the doc) :

How I choose the default language to print?
How can I make all the translation inside the same domain and having multiple project?

thanks

+1 We've made some changes to translations and domain configuration that we should definitely clarify.


Transferred 0 bytes is wrong

Obviously 4 bytes got transferred - but it gets reported wrong:
$ nc -l -p 4444 <IP_ADDRESS>
foo

$ echo "foo" | nc <IP_ADDRESS> 5555

$ tcpproxy -l 5555 -d <IP_ADDRESS>:4444
Listening on <IP_ADDRESS>:5555
New connection from <IP_ADDRESS>:59981
Error writing bytes from remote client <IP_ADDRESS>:59981 to upstream server
Connection reset by peer (os error 54)
Transferred 0 bytes from upstream server to remote client <IP_ADDRESS>:59981

I believe the message is correct but perhaps the functionality could use adjustment. It's reporting the number of bytes written from the server (nc on :4444) to the client (nc on :59981), which is zero (the number of bytes sent back).
The message that would have contained the number of bytes copied from the client to the server is not printed; in its place is a message reporting the error that resulted in the termination of the connection.
In this case, we are perhaps being too technical with the definition of "error" here - the client is presumably just terminating the connection without going through the teardown protocol with a TCP FIN and whatnot. We probably should just ignore that error and treat it as a normal connection close.

How did you end up in that state? When I do the same (but first nc, then tcp proxy, then nc), the foo is sent and then the "client" nc instance blocks waiting. Pressing ctrl-c in the client nc tty closes the connection cleanly, making tcpproxy report
Listening on <IP_ADDRESS>:5555
New connection from <IP_ADDRESS>:3490
Transferred 4 bytes from remote client <IP_ADDRESS>:3490 to upstream server
Transferred 0 bytes from upstream server to remote client <IP_ADDRESS>:3490

Interesting.
Yes, then it sounds like the logging output might be just confusing.

How did you end up in that state? When I do the same (but first nc, then tcp proxy, then nc), the foo is sent and then the "client" nc instance blocks waiting. Pressing ctrl-c in the client nc tty closes the connection cleanly, making tcpproxy report

Not for me.
When I press ctrl-c I get the reported message.
When I send echo "foo" | nc -c <IP_ADDRESS> 5555 I get the same message.
Maybe the host OS makes a difference?
Where are you trying?
I am (still) on macOS 12.6

Could be. Apple ships extremely outdated versions of most cli utilities for licensing reasons. (I'm on Linux.)
Can you try again after installing tcpproxy from git?
cargo install --git https://github.com/mqudsi/tcpproxy.git --branch master

Could be. Apple ships extremely outdated versions of most cli utilities for licensing reasons. (I'm on Linux.)

I also tried a recent netcat via homebrew.
Recent master looks better 👍
$ tcpproxy -l 5555 -d <IP_ADDRESS>:4444
Listening on <IP_ADDRESS>:5555
New connection from <IP_ADDRESS>:56001
Transferred 4 bytes from remote client <IP_ADDRESS>:56001 to upstream server
Transferred 0 bytes from upstream server to remote client <IP_ADDRESS>:5600

Awesome, thanks for the confirmation and thanks for posting here and helping improve tcpproxy!
I'll publish an update to cargo soon, as there are a few other fixes I've pushed recently.


ng-bing

In  the title template it says ng-bing instead of ng-bind. This is causing problems. May I fix it?

Just created a pull request for the same.. please confirm..

Just merged it. Thanks for your help!


log tpm2 commands in debug

It would be useful to neatly format commands sent to the tpm in cmd_exec.py
Need to consider security impact though, as this could lead to logging of various passwords, for example when calling tpm2_createek

hands off


GraphQL Question

Just thought I'd take a moment to thank you for this awesome repo. I regularly struggle for technical sources online suitable for enterprise level development; this is one of the best I've seen on this subject.
One question comes to mind, how would you organise GraphQL within this space? Originally l considered a singular graphql model isolated... but that seems inelegant having thought on it.

Thanks for putting all this great content together @Sairyss. 🚀


check result of ContainerMetadata.Retrieve

Signed-off-by: yanxuean<EMAIL_ADDRESS>ref #621
/cc @jellonek

This change is

@jellonek   PTAL

Small nits but rest is perfect.

Review status: 0 of 4 files reviewed at latest revision, 2 unresolved discussions.

pkg/metadata/containers.go, line 188 at r1 (raw file):

				}
				if ci == nil {
					return fmt.Errorf("containerInfo of container %s is not find in Virtlet metadata store", containerMeta.GetID())

Please change the message to "containerInfo for container %q not found in Virtlet metadata store".

pkg/metadata/containers_test.go, line 39 at r1 (raw file):

		}
		if containerInfo == nil {
			t.Fatal(fmt.Errorf("containerInfo of container %s is not find in Virtlet metadata store", container.ContainerId))

Please change the message to "containerInfo for container %q not found in Virtlet metadata store".

Comments from Reviewable

done
@jellonek PTAL

Reviewed 2 of 4 files at r1, 2 of 2 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Reviewed 2 of 4 files at r1, 2 of 2 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable