Multiple Test Notifications

Describe the bug
The test pass notification is invoked more than once on completion, causing the plugin to dispatch more than one waifu to tell you that you did a good job.
To Reproduce
Run this test: https://github.com/Unthrottled/hdd/blob/master/src/test/java/io/acari/water/WaterRepositoryTest.java
Expected behavior
Well you get one sound played, but you get multiple notifications the (1 more) at the bottom of the window
Screenshots
This is the event log and the notification after one test run.

Platform (please complete the following information):

Version WMP 1.0

IntelliJ IDEA 2020.1 (Community Edition)
Build #IC-201.6668.121, built on April 8, 2020
Runtime version: 11.0.6+8-b765.25 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
Linux 4.15.0-109-generic
GC: G1 Young Generation, G1 Old Generation
Memory: 512M
Cores: 12
Non-Bundled Plugins: io.acari.DDLCTheme, zd.zero.waifu-motivator-plugin
Current Desktop: ubuntu:GNOME

Additional context
Only one listener is being set, there is just more than one test pass notification being sent to the listeners.

I can look into the fix.

I also had experienced this in #110 but it only happens when there are multiple projects opened, I can only reproduce this issue with the one you've provided. Do you have any idea why it shows up there and not on the other tests?
My initial investigation, I tried to recreate it in a different project
Can't reproduce with

Parameterized Test

n Number of individual test

Can reproduce with this test.

So turns out that there are different ways IntelliJ executes a test. As you would notice from the highlights, there are hierarchies of subtest or list of tests where I can't reproduce the issue, but to the one where I can reproduce it only shows one list of test but runs multiple tests.
Note: I just got curious why this happens 😂
I'll check the PR #125 tomorrow as I was caught up by the rabbit hole when investigating this 😅.
Me:
When you've thought that you've resolved the issue of multiple test invocations.
Jetbrains unit test event:

If you where curious here is what I found:
Both of these classes:

https://github.com/JetBrains/intellij-community/blob/master/platform/smRunner/src/com/intellij/execution/testframework/sm/runner/GeneralTestEventsProcessor.java#L220
https://github.com/JetBrains/intellij-community/blob/master/plugins/gradle/java/src/execution/test/runner/GradleTestsExecutionConsoleManager.java#L142

Both execute the notification topic publishing code. Which is consistent with the double notifications that I was experiencing.
I looked into wiring our own listener, but I did not try to hard because I came up with a hack that fixes it, so I could work on more interesting problems.

Woah, that is indeed a good find! Now that makes sense.

I looked into wiring our own listener, but I did not try too hard because I came up with a hack that fixes it, so I could work on more interesting motivator problems.

I think that it was the simplest implementation we can do to prevent multiple notifications, it was good ✨

Resolved by #125


Juju offer default name breaks when charm has multiple relations to expose and no name is specified

Description
when i'm trying to expose 2 offers from the very same charm and don't specify the name, the default appears to be the application name
this triggers continuous changes to the terraform state
culprit seems to be https://github.com/juju/terraform-provider-juju/blob/main/internal/juju/offers.go#L94-L97
potentially a more sensible default or making the name mandatory might do the trick
Urgency
Casually reporting
Terraform Juju Provider version
0.15.1
Terraform version
v1.7.2dev
Juju version
3.5.5
Terraform Configuration(s)
data "juju_model" "model" {
  name = var.model
}

data "vault_generic_secret" "lego_credentials" {
  path = var.secret_path
}

resource "juju_secret" "lego_plugin_secret" {
  model = data.juju_model.model.name
  name  = "lego-httpreq-plugin-secret"
  value = {
    httpreq-endpoint            = var.lego_config.httpreq_endpoint
    httpreq-http-timeout        = var.lego_config.httpreq_http_timeout
    httpreq-propagation-timeout = var.lego_config.httpreq_propagation_timeout
    httpreq-username            = data.vault_generic_secret.lego_credentials.data["username"]
    httpreq-password            = sensitive(data.vault_generic_secret.lego_credentials.data["password"])
  }
  info = "Secret for LEGO charm, httpreq config information included"
}

resource "juju_access_secret" "grant" {
  secret_id    = juju_secret.lego_plugin_secret.secret_id
  model        = data.juju_model.model.name
  applications = [juju_application.lego.name]
}

resource "juju_application" "lego" {
  name  = var.name
  model = data.juju_model.model.name
  trust = var.trust

  charm {
    name    = var.charm.name
    channel = var.charm.channel
    base    = var.charm.base
  }

  config = {
    email                   = var.lego_config.email
    plugin                  = "httpreq"
    plugin-config-secret-id = juju_secret.lego_plugin_secret.secret_id
  }
}

resource "juju_offer" "certificates" {
  model            = data.juju_model.model.name
  application_name = juju_application.lego.name
  name             = "certificates"
  endpoint         = "certificates"
}

resource "juju_offer" "send_ca_certificate" {
  model            = data.juju_model.model.name
  application_name = juju_application.lego.name
  name             = "send-ca-cert"
  endpoint         = "send-ca-cert"
}

### Reproduce / Test

```shell
terraform init && terraform apply

Debug/Panic Output
No response
Notes & References
No response

Thank you for reporting this issue. We will discuss it and let you know about its resolution as soon as possible.

This is the same behavior as the juju cli: https://github.com/juju/juju/blob/f75387c60dd05478d64f515961f282ab3c6fc5f3/cmd/juju/crossmodel/offer.go#L153-L155
Requiring a name would be a compatibility issue at this point and only necessary when making offers on different endpoints of a single application. However the documentation can be updated to be clear on the behavior.


Add possibility to configure the ServiceBehavior

Users of EMG.Utilities.ServiceModel will be able to customize the ServiceBehavior attribute by doing something like
service.ConfigureService(svc => svc.AddressFilterMode = AddressFilterMode.Any);

If we want, we can add an helper into EMG.Utilities.ServiceModel.ECS for the snippet above:
service.AcceptToAllIncomingRequests();
// equivalent of service.ConfigureService(svc => svc.AddressFilterMode = AddressFilterMode.Any);

If we want, we can add an helper into EMG.Utilities.ServiceModel.ECS for the snippet above:

Yeah I would like that, because every time you use ECS you would need to do that configure


Update .eslintrc.json

Is there a reason you didn't do
"env": {
    // ...
    "webextensions": true
 }

Just curious :)

@alexweininger Sorry for the late response.
I actually have not tried that but I have an own chrome extension project where I have added "chrome" to globals, which works fine.

@gson88 Ok thanks!


[dv] Verify fetch enable

Verify the fetch enable signal. Write a sequence that can toggle it for the new sequence library. Use this sequence to randomly toggle fetch enable in some tests. It should have no impact on architectural execution (i.e. timing of instructions will differ but co-simulation checking should not find any issues).
Add a new checker that ensures that when fetch is disabled Ibex isn't fetching and executing new instructions (assertions may be sufficient)

estimate 4

@GregAC Please go ahead I'm still working on the debug side unfortunately..

Fetch enable testing added via this PR https://github.com/lowRISC/ibex/pull/1844


Feature Request: Navigation through Highlighted References

In Visual Studio there are 2 keyboard shortcuts that allow you to navigate through highlighted references of the current open document.  I think it would be very useful to have these in VS Code.
For languages like c# and typescript VS Code already has the symbol information to perform the highlight, I think now it's just a matter of being able to map a key to navigate next/previous.
In Visual Studio:
Edit.NextHighlightedReference
Edit.PreviousHighlightedReference

navigate through highlighted references

@Sharpiro You the symbol highlight, right?

@jrieken Yes, it highlights, but it doesn't allow you to navigate to the next/previous instance of that highlighted word. There is a Shift+F12 feature that shows references, but it is a slow and clunky method of navigating a piece of code...
Such a handy feature of VS2015/2017... Ctrl+Shift+Up/Down.

Understood, thanks.


Icon Update: Stackshare in wrong category

The stackshare icon is located in the wrong category (currently in Development), it belongs in the Logos category
Also, the logo always has a background rectangle that has a border radius

This is missing from the icon set

Oh, stack-share is a logo!!!??
I thought it is a kind of action in development.
I'll fix it later

I've moved the stackshare icon from development to logos and redesigned the  -fill icon.
Have a look at it here(https://remixicon.com/#icon-stackshare-fill) .


.tf.json files not cleaned up

I receive an error when I run subsequent commands after the initial command (terragrunt plan/apply). I believe this is because .tf.json files aren't cleaned up from .terragrunt-cache.
The working directory already contains files. The -from-module option requires
an empty directory into which a copy of the referenced module will be placed.

To initialize the configuration already in this working directory, omit the
-from-module option.

Ah, good find. Would you be up for a PR to fix this?

@brikis98 Moving my comment on #571 to here.
I noticed that in that issue you had mentioned requests for cleaning up all files/folders. I was wondering if this was still desired, and if so, should this be the default behavior?
I did a quick search on the open issues to see if there was any discussion on this, but I wasn't able to find any additional issues that mentioned that request. I don't think I know enough about Terragrunt to know if there would be a use case where you wouldn't want to clean up all files/folders.

I'm scratching my head... I think we don't delete all files to avoid deleting the .terraform folder Terraform uses as a cache, so you don't have to download modules, providers, and configure state every time. I can't remember if we avoid deleting other files for some reason off the top of my head though 😕

Oh, you know what, I think the Terraform state file, assuming you're using local state, is one of the items that'll be in that folder... So deleting it outright would be a bad idea.

@brikis98 Would a valid approach for cleaning up .terragrunt-cache be to delete all files/folders in it except for the .terraform folder and terraform.tfstate?
I'll see if I can make the fix some time this week.

I was actually just digging through that code in #739 and, unfortunately, it's a bit more complicated. Here's what has to be maintained that I'm aware of:

.terraform folder and its contents. Contains modules, provider code, remote state config.
terraform.tfstate. Contains local state config.
terraform.tfstate.backup. Contains local state config backup.
.terragrunt-source-version. Used by Terragrunt to know if a new version of source should be downloaded.

However, it's potentially even trickier. For example, a user could run terraform apply -state-out=foo to tell Terraform to store the state in a file called foo rather than terraform.tfstate. There's also terraform plan -out=foo to store a plan file, and possibly other data you don't want to lose.
So for now, in the cleanupDownloadDir method, I just make sure not to delete anything in the working dir other than .tf files. Perhaps the easiest fix is to just expand the glob expression to also delete .tf.json files?


Update SLAS helper to require channel_id

Starting 7/31/2024, SLAS will start requiring the channel_id as a required argument when retrieving an access token with grant_type: 'client_credentials for security purposes. This PR updates the loginGuestUserPrivate SLAS helper function to require channel_id.

Someone can correct my javascript knowledge if its incorrect, but I believe that marking a function with async means that return value is automatically a promise, so although we don't use await statements, we return a Promise<TokenResponse>

Yes you're right, the function marked with async automatically returns a promise. It will not wait for getAccessToken to resolve before returning, but I guess subsequent operations are handled in getAccessToken.


Visual Metronome per NPC ID

Create a visual metronome above an NPC based on it's ID. For instance above Xarpus I'd want a 4 tick visual metronome. Over another NPC I would want a 5 tick visual metronome. etc. This would maybe require a new plugin but would be heavily requested by many.

Currently visual metronome is the only way I know to have a tick timer. Although the tob plugin has room tick timers only on entry. I would like a visual metronome over NPC Id's. To give an example here we have Xarpus. In order to scythe walk correctly I need to step back 2 ticks after his NPC ID changes (he's marked red). In this clip that happens at tick 3 so I need to step back on tick 1 during a 4 tick timer. During p3 Verzik I need a tick timer that is every 7 ticks instead of every 4.
Could also add an offset value so you wouldn't have text directly on top of HP percentage like in the clip shown. It could be as simple as the below:
NPCID:TickCounter:Offset
8,340:4:-24
8,372:4:+12
8,374:7:0
This plugin would take that list and as soon as NPC ID 8,340 spawns (xarpus) it would count a 4 tick timer on that NPC's location (not the player location). Another 4 tick timer for the 8,372 NPC ID (verzik p2) and then a 7 tick timer for 8,374 (verzik p3). Let me know what you think.
https://imgur.com/Xeka2RF

Hey, I see how this would be coded and how people would find it useful but yeah I think it would be a better idea for a separate plugin.
first of all, it's a big idea and would take a decent amount of time to make. I can't really just group it into another section of this plugin, which is already starting to have a pretty long list of settings.
and also, I'm pretty busy with other work now, and don't really have time to code a whole new plugin anymore. it sounds like a good idea though, I think someone else would be better off coding this, or you could try it yourself.


mimalloc MI_TRACK_ASAN option can not be used together with MI_USE_CXX

I am using mimalloc v2.1.7 and compile & running it on Ubuntu, Linux 5.15.0. When I turn on both two options, with MI_OVERRIDE turned on, compile my own program with asan options, the program can not run as expected, but failed in mimalloc instead.

I just tried:
$ cmake ../.. -DMI_TRACK_ASAN=ON -DMI_USE_CXX=ON
$ make -j
$ ./mimalloc-test-stress

and it seems to work. Can you provide more info?


container props and default done

Did not listen to propchanges (componentWillReceiveProps).
default for done was using an fat-arrow function. Accessing this in fat-arrow will bind it to the initial component.
The call later to done with .call(config, ..) did nothing because it was already bound.

Ah yeah, I think this just got me. Had a module with module.exports = container(React.createClass({... but all the components I created ended up with the same props.

@chrisbarmonde Yes =) but i didn't create multiple instances, just one that got mounted and unmounted a couple of times. I always got the same props and i passed in a callback when the callback got called it was on an old unmounted parent.
But i've been wondering, if i export a container and use it multiple times on one page would it work ? Wouldn't all the components share props and such because all share same config ?

This has been release in v0.9.2. Thanks for your help!


Optimize notifications stream

Abort reading the log past the oldest message in your feed, since presumably no one will have mentioned your feed before you started posting on it.
Also fix an error having to do with messages where the branch property is an array

merged into 1.7.1


Change ambiguous wording for timeouts

The previous wording made it sound like any connection that was ongoing
for more than 10 seconds (e.g. a download of a large file) would be
disconnected.

Yes, much better!


Remove nokogiri dependency

Using nokogiri to parse our xml messages is adding a tremendous amount of time to bundle install.  Given we are using very little of its functionality, I'd like to investigate using rexml instead.

Removed in 0.6.0 - please open an issue if this breaks something.


Raycast from mouse to tile is inaccurate

When trying to select tiles in the bottom left corner of the screen, you will notice that sometimes the wrong tile is selected (same when building on it). I think there is an issue with how we trace from the mouse position to get the hovered tile.

Fixed


[PE-D][Tester D] listing teams works even with additional arguments

listing teams using the "listt" command works even with additional arguments after the word "listt", e.g. "listt aagaqgiqgqg"

Labels: type.FunctionalityBug severity.Low
original: nikele2001/ped#5

This should be fine, but need to add notes in the UG to explain the expected outputs.


Remove explicit install instructions, update refs to cert-manager

Hi! I'm a cert-manager maintainer and I spotted this repo while looking at something else. It's always great to see these integrations with cert-manager :grin:
I noticed that the instructions recommended a very old version of cert-manager which is no longer supported. It might be better for users if they're linked to the cert-manager installation instructions on our website, which we try to keep up to date for each version we release.
I also added a link and updated so that "cert-manager" was always written in the way we like to style it. See https://cert-manager.io/docs/faq/style/ for details on that.

Great, thanks a lot! smile (and sorry for the misspell of cert-manager )

Ah don't worry about it at all, it's no big deal - I just like to change it when I see it! Thanks so much :heart:


init shared object with shape array

Describe the solution you'd like
init shared object with std::array

fixed with a310a4f


Removed deprecated -d flag from docs

Hi,
This addresses #15909 :)

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "15909-daemon-flag" git@github.com:lyndaoleary/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

Sorry @GordonTheTurtle, I forgot oops :laughing:

LGTM, thanks again Lynda!
ping @fredlf @jamtur01 @moxiegirl @SvenDowideit

Oh, I edited your top comment, and added fixes #15909, so that GitHub will automatically close the linked issue if this is merged. There are some forms you can use but (I checked, to be sure), "addresses ..." isn't in the list. You can find what you can use here; https://help.github.com/articles/closing-issues-via-commit-messages/

LGTM


[FRONTEND WITH FRAMEWORK]: Angular(5.3) -> Getting route information

Description

How to fetch route information?
What is the use of getting information from route?
How to access query parameters/fragments?
Explain with syntax/example
Reference :- https://angular.io/guide/router

(Week 5 Issue)
Note:-

Don't copy paste, we would be merging after reviewing the PR.
Changes should be made inside "Frontend_Web_Development_React_Angular_Vue/Angular/Week5" directory
Tasks would be assigned on First Come First Serve Basis

Domain
Frontend Dev React/Angular/Vue
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

@RIYANSHGARG any updates

I will create pr by tommorow.


cf login -a api.local.pcfdev.io --skip-ssl-validation API endpoint: api.local.pcfdev.io FAILED Server error, status code: 404, error code: 0, message:

Hi Guyz,
PCFdev is up and running in my windows machine. But when i do a cf login from CMD terminal i get the above error message. I am currently in a corporate network and i have tried using Acrylic too as per the documentation. When i ssh into the machine i am able to login to the endpoint successfully but am unable to do it directly from the windows cmd.
==> default: Waiting for services to start...
==> default: 0 out of 48 running
==> default: 0 out of 48 running
==> default: 0 out of 48 running
==> default: 0 out of 48 running
==> default: 4 out of 48 running
==> default: 48 out of 48 running
==> default: PCF Dev is now running.
==> default: To begin using PCF Dev, please run:
==> default:    cf login -a api.local.pcfdev.io --skip-ssl-validation
PS C:\Users\mjt5kor\Documents\pcfdev-v0.15.0\pcfdev-v0.15.0(2)\pcfdev-v0.15.0> cf login -a api.local.pcfdev.io --skip-ss
l-validation
API endpoint: api.local.pcfdev.io
FAILED
Server error, status code: 404, error code: 0, message:

Do you have a working internet connection? To resolve the *.local.pcfdev.io hostname it needs to go to the internet (it will resolve to <IP_ADDRESS> if im not mistaken) but it wont work without internet connection.

yes @Turbots am able to ping... I meant that if I putty(ssh) session directly into <IP_ADDRESS> it works but when i try to cf login from the windows cmd am getting the "Failed" error.
C:\Users\mjt5kor>ping <IP_ADDRESS>
Pinging <IP_ADDRESS> with 32 bytes of data:
Reply from <IP_ADDRESS>: bytes=32 time<1ms TTL=64
Reply from <IP_ADDRESS>: bytes=32 time<1ms TTL=64
Reply from <IP_ADDRESS>: bytes=32 time<1ms TTL=64

I know it can take a while and that it is kinda lame advice, but did you already try stopping and starting the pcfdev environment? I have noticed it can help in some configuratoins (especially when first time failes). you should also try posting the vagrant logfile here.

Thanks @Turbots i did a vagrant up from the scratch and looks like the issue persists. I guess this has to do with the proxy settings in my environment. The setup works fine when i take an ssh session, the problem is when i do directly using windows cmd. I am trying to tinker around with my proxy setting..Will update if i am successful :/

hey @sclevine any recommendations here ??

Hi @jaikanthm,
Could you post the full output of running the Windows start script? If you curl api.local.pcfdev.io (or use the Windows equivalent) in the same terminal, what body do you receive for the 404? What does the JSON response look like if you navigate to api.local.pcfdev.io in a browser? If you cf login from a new terminal (without the proxy environment variables set), does that fix your issue?
I wonder if your proxy environment variables are preventing the cf CLI from talking to api.local.pcfdev.io.
-Stephen

I encountered also an error but not 404. Mine was:
cf login -a api.local.pcfdev.io --skip-ssl-validation
API endpoint: api.local.pcfdev.io
FAILED
Server error, status code: 504, error code: 0, message:
Some facts:
pinging it is okay
doing putty to it is okay
calling GET on http://api.local.pcfdev.io/ returns this
{
"code": 10000,
"description": "Unknown request",
"error_code": "CF-NotFound"
}
calling GET on http://api.local.pcfdev.io/v2/info returns this
{
"name": "pcfdev",
"build": "b40a5acf9677bd2061a3a9027c600ce04b277bbe",
"support"<EMAIL_ADDRESS>"version": 0,
"description": "",
"authorization_endpoint": "http://login.local.pcfdev.io",
"token_endpoint": "http://uaa.local.pcfdev.io",
"min_cli_version": null,
"min_recommended_cli_version": null,
"api_version": "2.54.0",
"app_ssh_endpoint": "ssh.local.pcfdev.io:2222",
"app_ssh_host_key_fingerprint": "a6:d1:08:0b:b0:cb:9b:5f:c4:ba:44:2a:97:26:19:8a",
"app_ssh_oauth_client": "ssh-proxy",
"logging_endpoint": "wss://loggregator.local.pcfdev.io:443",
"doppler_logging_endpoint": "wss://doppler.local.pcfdev.io:443"
}

@xtrycatchx you are experiencing a different, likely unrelated error, please open a new issue for this. It would also be helpful to include the output of running the distribution script for your platform.

Hi @sclevine this is the output for the powershell script..both with proxy and without proxy.
Curl o/p is below as well....
When i open api.local.pcfdev.io using internet explorer its asks me whether to open or save a file but nothing happens when clicked (strange)
PS C:\Users\mjt5kor\Documents\pcfdev-v0.15.0\pcfdev-v0.15.0(2)\pcfdev-v0.15.0> .\start-windows.ps1
PCF Dev has reserved 3 GB out of 7 GB total system memory.
Bringing machine 'default' up with 'virtualbox' provider...
==> default: Clearing any previously set forwarded ports...
==> default: Clearing any previously set network interfaces...
==> default: Preparing network interfaces based on configuration...
default: Adapter 1: nat
default: Adapter 2: hostonly
==> default: Forwarding ports...
default: 22 (guest) => 2222 (host) (adapter 1)
==> default: Running 'pre-boot' VM customizations...
==> default: Booting VM...
==> default: Waiting for machine to boot. This may take a few minutes...
default: SSH address: <IP_ADDRESS>:2222
default: SSH username: vagrant
default: SSH auth method: private key
default: Warning: Remote connection disconnect. Retrying...
==> default: Machine booted and ready!
==> default: Checking for guest additions in VM...
==> default: Configuring and enabling network interfaces...
==> default: Configuring proxy for Apt...
==> default: Configuring proxy environment variables...
==> default: Machine already provisioned. Run vagrant provision or use the --provision
==> default: flag to force provisioning. Provisioners marked to run always will still run.
==> default: Running provisioner: shell...
default: Running: inline script
==> default: stdin: is not a tty
==> default: Waiting for services to start...
==> default: 0 out of 48 running
==> default: 0 out of 48 running
==> default: 0 out of 48 running
==> default: 0 out of 48 running
==> default: 8 out of 48 running
==> default: 48 out of 48 running
The SSH command responded with a non-zero exit status. Vagrant
assumes that this means the command failed. The output for this command
should be in the log above. Please read the output to determine what
went wrong.
PS C:\Users\mjt5kor\Documents\pcfdev-v0.15.0\pcfdev-v0.15.0(2)\pcfdev-v0.15.0> curl api.local.pcfdev.io
curl :
Access Denied (content_filter_denied)
Your request was denied because of its content categorization: "Suspicious"
Based on mandatory information security and privacy rules access to this category is not allowed.
Detailed technical information:
URL: http://api.local.pcfdev.io/

Could you try out PCF Dev 0.16.0 (just released), and report if this error persists?

okay, i'll try this and will get back to you @sclevine

Awesome @sclevine  will do that and update

hi @sclevine this is the error i get when i use pcf dev 0.16. I tried it both in a corporate network behind a proxy and also on an open network at home same issue.
PS E:\pcfdev-v0.16.0-windows> cf install-plugin .\pcfdev-v0.16.0-windows.exe
Attention: Plugins are binaries written by potentially untrusted authors. Install and use plugins at your own risk.
Do you want to install the plugin .\pcfdev-v0.16.0-windows.exe? (y or n)> y
Installing plugin ./pcfdev-v0.16.0-windows.exe...
OK
Plugin pcfdev v0.0.0 successfully installed.
PS E:\pcfdev-v0.16.0-windows> cf dev start
Please retrieve your Pivotal Network API from:
https://network.pivotal.io/users/dashboard/edit-profile
API token>
FAILED
Error: failed to reach Pivotal Network: Post https://network.pivotal.io/api/v2/products/pcfdev/releases/1622/product_fil
es/4895/download: dial tcp: lookup network.pivotal.io: getaddrinfow: No such host is known.
PS E:\pcfdev-v0.16.0-windows> cf dev start
Please retrieve your Pivotal Network API from:
https://network.pivotal.io/users/dashboard/edit-profile
API token>
FAILED
Error: invalid Pivotal Network API token
PS E:\pcfdev-v0.16.0-windows> cf dev start
Please retrieve your Pivotal Network API from:
https://network.pivotal.io/users/dashboard/edit-profile
API token>
FAILED
Error: invalid Pivotal Network API token
PS E:\pcfdev-v0.16.0-windows> cf dev start
Please retrieve your Pivotal Network API from:
https://network.pivotal.io/users/dashboard/edit-profile
API token>
FAILED
Error: invalid Pivotal Network API token
PS E:\pcfdev-v0.16.0-windows> cf dev start
Please retrieve your Pivotal Network API from:
https://network.pivotal.io/users/dashboard/edit-profile
API token>
FAILED
Error: failed to reach Pivotal Network: Post https://network.pivotal.io/api/v2/products/pcfdev/releases/1622/product_fil
es/4895/download: dial tcp: lookup network.pivotal.io: getaddrinfow: No such host is known.
PS E:\pcfdev-v0.16.0-windows>

Some of those errors indicate that you are unable to resolve network.pivotal.io, and others indicate that your API token is invalid. Are you copying and pasting the API token from the provided URL? Does ping network.pivotal.io succeed from the same terminal you're running cf dev start in? What do your proxy environment variables look like (with any sensitive info redacted)? Can you confirm that you've destroyed all old versions of PCF Dev?

Quite strange unable to ping network.pivotal.io....but able to ping google and other sites
C:\Users\jai>ping google.com
Pinging google.com [<IP_ADDRESS>] with 32 bytes of data:
Reply from <IP_ADDRESS>: bytes=32 time=11ms TTL=55
Reply from <IP_ADDRESS>: bytes=32 time=10ms TTL=55
Reply from <IP_ADDRESS>: bytes=32 time=23ms TTL=55
Reply from <IP_ADDRESS>: bytes=32 time=18ms TTL=55
Ping statistics for <IP_ADDRESS>:
Packets: Sent = 4, Received = 4, Lost = 0 (0% loss),
Approximate round trip times in milli-seconds:
Minimum = 10ms, Maximum = 23ms, Average = 15ms
C:\Users\jai>ping network.pivotal.io
Pinging network.pivotal.io [<IP_ADDRESS>] with 32 bytes of data:
Request timed out.
Request timed out.
Request timed out.
Request timed out.
Ping statistics for <IP_ADDRESS>:
Packets: Sent = 4, Received = 0, Lost = 4 (100% loss),

Able to ping github.io as well ...unable to ping network.pivotal.io alone
Pinging github.io [<IP_ADDRESS>] with 32 bytes of data:
Reply from <IP_ADDRESS>: bytes=32 time=187ms TTL=49
Reply from <IP_ADDRESS>: bytes=32 time=188ms TTL=49
Reply from <IP_ADDRESS>: bytes=32 time=186ms TTL=49
Reply from <IP_ADDRESS>: bytes=32 time=186ms TTL=49

network.pivotal.io doesn't actually respond to ping. DNS resolution seems to work (even though it appears to fail occasionally when the CLI does it: lookup network.pivotal.io: getaddrinfow: No such host is known.). Can you provide the rest of the requested info?

Hey @sclevine  good news... I was able to paste the API token...The API token when pasted in powershell was blank hence i was pasting it 2 times... Now the download starts at least from my home network.. Will keep you posted how it goes...
Regarding downloading behind proxy it fails saying "proxy authentication required"

I'm closing this due to inactivity for now. Feel free to re-open if you're still experiencing issues.

Just to note - I was able to resolve this in windows by adding an environment variable no_proxy equaling local.pcfdev.io, api.local.pcfdev.io and *.local.pcfdev.io


[WIP] Generic channels

Implementation PRs coming later today, opening this for comments.
This PR:

changes MessageChannel to be an extension of Channel, allowing things such as TitleChannel as well as custom channels
adds TitleChannel for sending titles

I'd appreciate more verbose type parameter names -- especially since you have a lot of them here.

If anyone wants to see, here is my idea of a message channel: https://gist.github.com/Wolfizen/8dfc62fd2d5b0a201e0a

adds TitleChannel for sending titles

Why do you need that in the API?

Closing in favour of https://github.com/SpongePowered/SpongeAPI/pull/1043 for now.


Fix auth0 new tenants

 Issue by landonreed
Thursday Aug 30, 2018 at 19:09 GMT
Originally opened as https://github.com/catalogueglobal/datatools-server/pull/113

With https://github.com/catalogueglobal/datatools-ui/issues/420, this PR allows new Auth0 tenants to work with Data Tools.  The tenant must be configured with an Auth0 rule that assigns app_metadata and user_metadata to a value scoped by http://datatools.  For example:
function (user, context, callback) {
  const namespace = 'http://datatools/';
  if (context.idToken && user.user_metadata) {
    context.idToken[namespace + 'user_metadata'] = user.user_metadata;
  }
  if (context.idToken && user.app_metadata) {
    context.idToken[namespace + 'app_metadata'] = user.app_metadata;
  }
  callback(null, user, context);
}

Upon receiving a request via its HTTP API, datatools-server will handle verifying the token and re-map the scoped metadata objects to the app_metadata and user_metadata fields. This ensures that the application continues to function for older Auth0 tenants that still have access to the fields on the user profile and limits the changes required on the front end.

landonreed included the following code: https://github.com/catalogueglobal/datatools-server/pull/113/commits

 Comment by codecov-io
Tuesday Sep 04, 2018 at 14:22 GMT

Codecov Report

Merging #113 into dev will decrease coverage by 0.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##               dev    #113      +/-   ##
==========================================
- Coverage     5.13%   5.12%   -0.02%
  Complexity      91      91
==========================================
  Files          129     129
  Lines         6751    6776      +25
  Branches       880     884       +4
==========================================
  Hits           347     347
- Misses        6379    6404      +25
  Partials        25      25

Impacted Files
Coverage Δ
Complexity Δ

...nveyal/datatools/manager/auth/Auth0Connection.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1db34f2...666fb0b. Read the comment docs.


Solved the issue of page rendering at bottom

Solved issue #274  Now the page /fullservicefund opens at the top when the respective route is hit.
https://github.com/Priyaaa1/StartConnect-Hub/assets/154724111/b3d9c899-dfbf-42e9-80a0-f899e4934654

@Priyaaa1 Pls review the PR!!!


Removed IMouseDevice from service locator

Part of #1013

Also could you add a description of why this needs doing to the PR so that people looking back in 5 years time can see the reasons things ended up how they did?

I haven't touched keyboard device yet, since it will require changes at least in focus management. This PR is already huge, so it's better to do in a different one


Correct commands to use SUSE crm instead of RH pcs

Changed commands for "2. Restart the pacemaker service on all cluster nodes" for SUSE to use "crm" instead of the RH "pcs"

@petersatsuse : Thanks for your contribution! The author(s) have been notified to review your proposed change.


SCSS fails in prod build

I'm submitting a ...  (check one with "x")
[ x ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here, instead see use [gitter](https://gitter.im/mgechev/angular2-seed) or [stackoverflow](https://stackoverflow.com/questions/tagged/angular2)

Current behavior
I have enabled prod mode and also enabled scss. SCSS works fine if i build in dev mode. But in prod mode it is not able to find css or scss file. The output error when compile in prod is:

Expected behavior
Compile in prod (serve.prod) without sass errors.
Minimal reproduction of the problem with instructions
My project structure is this:

I realized the @media from sass files cause this error and i dont know how can i fix it, can anyone help me? @mgechev
Please tell us about your environment:
Windows 10, Visual Code IDE.

Angular Seed Version: latest

Node: node --version = v6.5.0

@AdrianAgui , main.css line 1009 you forgot }.
This issue can be closed ;)

@vyakymenko main.css is  the file created on the compile process, is not my main.css.
The problem here is with @media sass, without @medias in my code build prod works well, but when i put one the build fails (the error above).

@AdrianAgui , it's not a seed issue. Can u share your code, I think error in one of your scss file ?

I found the error, and yes, is a seed issue. In tasks/seed/build.html_css.ts file, when the build is prod, is used plugins.cached('process-component-css') on some functions, I replaced this by plugins.util.noop() which is used in dev build.
I dont know why is working now but I suggest you take a look in build prod gulp task.
Thanks.


add sql state and vendor code for this

Fixes #4654

Codecov Report

Merging #4768 into master will not change coverage.
The diff coverage is 0%.

@@            Coverage Diff            @@
##             master    #4768   +/-   ##
=========================================
  Coverage     56.09%   56.09%
  Complexity      441      441
=========================================
  Files          1157     1157
  Lines         20878    20878
  Branches       3863     3863
=========================================
  Hits          11711    11711
  Misses         8437     8437
  Partials        730      730

Impacted Files
Coverage Δ
Complexity Δ

.../jdbc/core/resultset/EncryptResultSetMetaData.java
3.57% <0%> (ø)
0 <0> (ø)
:arrow_down:

...jdbc/core/resultset/ShardingResultSetMetaData.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 93ecfd2...71a1329. Read the comment docs.

Pull Request Test Coverage Report for Build 10279

0 of 2 (0.0%)  changed or added relevant lines in 2 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.005%) to 59.647%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sharding-jdbc/sharding-jdbc-core/src/main/java/org/apache/shardingsphere/shardingjdbc/jdbc/core/resultset/EncryptResultSetMetaData.java
0
1
0.0%

sharding-jdbc/sharding-jdbc-core/src/main/java/org/apache/shardingsphere/shardingjdbc/jdbc/core/resultset/ShardingResultSetMetaData.java
0
1
0.0%

Files with Coverage Reduction
New Missed Lines
%

sharding-orchestration/sharding-orchestration-core/sharding-orchestration-core-registrycenter/src/main/java/org/apache/shardingsphere/orchestration/core/registrycenter/util/IpUtils.java
3
76.0%

Totals

Change from base Build 1084:
-0.005%

Covered Lines:
12453

Relevant Lines:
20878

💛  - Coveralls


check serialize/deserialize interoperability (with other languages)

create post in the SR channel after March CC.

finished one round of this. will do this again for GA release.


Add tests

The easiest way to test this would be to see if it can convert the CRDC-H model in Google Sheets into valid LinkML, but this is a bad idea since the CRDC-H model in Google Sheets will probably be changing soon. Instead, we should probably come up with a dummy spreadsheet whose sole purpose is to test this generator.
We also need a test to ensure that the output is valid LinkML YAML. The good news is that the LinkML model is itself written in LinkML, so it should be validatable. Unfortunately, most of those validation methods are designed around other formats (such as validating JSON, JSON-LD, RDF and so on) but I haven't found one for validating the YAML directly.

Might be a good idea to do this for the data in https://harmonization.datacommons.cancer.gov/additional-resources

[ ] Once we do this, we should use GitHub Actions' "matrix" feature to test our code against multiple versions of Python.


Menu disappears under cards

Hallo,
When I want to delete a card, the dialog box goes under the other cards.

/Calmar

@HrCalmar  Yes... That's a problem with @kibibit/kb-frosted-cards.
I'm trying to fix this but can't find an easy solution for now.
You can keep the theme itself without the kb-frosted-cards extension and it will work fine. the kb-frosted-cards extension is mainly for using other backgrounds than the default one. Since in the screenshot it seems like you're using the default background, the experience won't be much different without the kb-frosted-cards extension.

@HrCalmar  Yes... That's a problem with @kibibit/kb-frosted-cards.
I'm trying to fix this but can't find an easy solution for now.
You can keep the theme itself without the kb-frosted-cards extension and it will work fine. the kb-frosted-cards extension is mainly for using other backgrounds than the default one. Since in the screenshot it seems like you're using the default background, the experience won't be much different without the kb-frosted-cards extension.


Atmospheric extinction

Add option for taking Atmospheric extinction into account.
From the Duffett-Smith book "Practical Astronomy With Calculator" the formula is:
deltaM = 0.2 / cos(z)
z is a zenithal distance (90 - alt), deltaM is a correcction for magnitude.
Note: formula is incorrect for zenith distances greater than 85 degrees.
coefficent is

0.1 for highest mountains
0.2 for very good lowland locations
0.35 for typical lowland
0.5 for humid climates

See also: https://github.com/Stellarium/stellarium/blob/e0f00c384e10e0f23117b35212fb0f708f14c0e4/src/core/RefractionExtinction.cpp#L35


Saving from a partial SCSS file that has variables gives error

Hello,
Whenever I try to save from a partial SCSS file the package gives me an error and I would have to go to the main SCSS file and save from there.
For example,
style.scss file imports _variables.scss and _ui.scss.
Both style.scss and _ui.scss use variables from _variables.scss.
If I make a change to _ui.scss and save from there the package gives an "Undefined variable" error.
But if switch tab and save from style.scss, everything compiles.
Is there an option that I am missing? or would this be in the next version?
thank you

Hey!
Have a look at the main parameter.
If you want to compile style.css when you save _ui.scss you should setup this comment as first line-comment:
// main: ../relative/path/to/style.css

Clear?


Latest release of Microsoft.EntityFrameworkCore.Cosmos 6.0.4 has dependency Microsoft.Azure.Cosmos >= 3.21.0 which is less than minimum recommended version 3.25.0.

We are continuously addressing and improving the SDK, if possible, make sure the problem persist in the latest SDK version.

Describe the bug
Latest release of Microsoft.EntityFrameworkCore.Cosmos 6.0.4 has dependency Microsoft.Azure.Cosmos >= 3.21.0 which is less than minimum recommended version 3.25.0.
To Reproduce
Dependency shown under https://www.nuget.org/packages/Microsoft.EntityFrameworkCore.Cosmos
If referencing from a project,

CMD> dotnet list package --include-transitive
Transitive package Resolved
Microsoft.Azure.Cosmos 3.21.0
Expected behavior
A clear and concise description of what you expected to happen.
https://github.com/Azure/azure-cosmos-dotnet-v3/blob/master/changelog.md#-recommended-version
The minimum recommended version is 3.25.0.
Actual behavior
[NuGet Gallery | Microsoft.EntityFrameworkCore.Cosmos 6.0.4](https://www.nuget.org/packages/Microsoft.EntityFrameworkCore.Cosmos/6.0.4)
Dependencies
net6.0
Microsoft.Azure.Cosmos (>= 3.21.0) <-- Which does not meet minimum recommended version 3.25.0
Microsoft.EntityFrameworkCore (>= 6.0.4)
Microsoft.Azure.Cosmos (>= 3.21.0)
Environment summary
SDK Version: net6.0
OS Version Windows
Additional context
Workaround by explicitly adding min version reference like

Hi @aluty,
The Cosmos SDK team does not own that project. It seems like they already migrated to latest Cosmos version: https://github.com/dotnet/efcore/pull/27680. I'm going to transfer it to the correct repository so they can follow up with a release schedule.


normalize-by-median: accept paired & unpaired data at the same time

Workaround: process one set and save the hash; once completed repeat but load the hash file and process the other set

Working on this

Thank you @susinmotion !


KAFKA-8436: use automated protocol for AddOffsetsToTxn

More detailed description of your change,
if necessary. The PR title and PR message become
the squashed commit message, so use a separate
comment to ping reviewers.
Summary of testing strategy (including rationale)
for the feature or bug fix. Unit and/or integration
tests are expected for any behaviour change and
system tests should be considered for larger changes.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Retest this please

@mimaison Could you take a look for this one?

@mimaison Could you have a look for this one?

@mimaison Could you take a look for this one?

@mimaison Could you take a look for this one?

@mimaison Thanks for the review! Addressed

ok to test

@mimaison Looks 2 checks passed

@mimaison Does the change look good to you?


Implement PEP 639, Metadata 2.4

Depends on merge/release of https://github.com/pypa/packaging/pull/828
Closes #16620
TODO: Move to release of packaging with license-expression parser

@befeleme would you be able to take a look at this to review the implementation details?

@befeleme would you be able to take a look at this to review the implementation details?

Yes, hopefully tomorrow.

After reading the code I've got just one question: I can't locate the validation of the License-Expression per https://peps.python.org/pep-0639/#add-license-expression-field ("[Python Package Index (PyPI) MUST validate that they contain a valid, case-normalized license expression with valid identifiers and MUST reject uploads that do not"). Or did I miss it?

After reading the code I've got just one question: I can't locate the validation of the License-Expression per https://peps.python.org/pep-0639/#add-license-expression-field ("[Python Package Index (PyPI) MUST validate that they contain a valid, case-normalized license expression with valid identifiers and MUST reject uploads that do not"). Or did I miss it?

That's implemented upstream in pypa/packaging, see https://github.com/pypa/packaging/pull/828

Ah, looking at the whole warehouse/forklift/metadata.py I see packaging validations are applied automatically and no additional changes regarding that are needed here. Makes sense.

Re @miketheman:

I'm thinking that tracking usage/progress of this can be deferred, but that might be a nice metric to emit via DB query later - I can imagine wanting to see new projects using the syntax, older projects converting to the new syntax, etc - but definitely beyond scope of this changeset.

That is all technically captured in the immutable release metadata, so aside from watching a pot boil in real-time in datadog I agree adoption metrics are well beyond scope.

Migrations are going out for this ahead of merge during this window to avoid unnecessary disruption: http://status.python.org/incidents/q5q5347n3s2b

Thanks everyone!


c# build logging not working

after months of working on a project I'm completely unable to continue working as my project suddenly refuses to run in the godot editor,  citing build failure but the view logs button doesn't work, and building in visual studio succeeds! so there is no way to run my project nor to discover the cause of the alleged error! I tried getting help on the discord c# channel but eventually was advised to submit a ticket here
in short,  my project builds in visual studio (for Mac) but  wont run in godot, (after running ok for months)

@samtsejerome Please fill in the issue template instead of replacing it when reporting bugs.

Godot version: 3.2 mono
OS/device including version: Macbook Pro running Macos 10.15.5
Issue description:
Steps to reproduce: run the project I've beenworking on for months
Minimal reproduction project: doesn't happen in a mininal project.

I wouldn't raise an issue if it werent completely blocking me after months of work but it is and the support channels were a dead end too

Minimal reproduction project: doesn't happen in a mininal project.

Someone would need access to your full project to be able to diagnose this.

I can share the  project if needed
Jerome Mayeux
Sent from Polymail ( https://polymail.io/?utm_source=polymail&utm_medium=referral&utm_campaign=signature )
On Sun, 12 Jul 2020 at 19:27 Hugo Locurcio < Hugo Locurcio ( Hugo Locurcio<EMAIL_ADDRESS>) > wrote:

Minimal reproduction project: doesn't happen in a mininal project.

Someone would need access to your full project to be able to diagnose
this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub (
https://github.com/godotengine/godot/issues/40321#issuecomment-657258331 )
, or unsubscribe (
https://github.com/notifications/unsubscribe-auth/AQIJ2UYNEX2JS6F3MQ3T4NLR3H6BVANCNFSM4OX4WQKQ
).

I can't read the text in the screenshots. Try running Godot from the terminal, e.g.: Godot.app/Contents/MacOS/Godot --path path/to/project --editor --verbose. With the Print Build Output option enabled, try to build the project and copy the full terminal output here.

Hi I only work in this project on Sunday’s and was unable to run my project after hours of trying so put  my Laptop  away, I’ll try and find somrctime before my next session to follow your suggestion, but failing that it won’t be until next Sunday I’m afraid
Regsrds,
Jerome.
Jerome Mayeux
Sent from Polymail ( https://polymail.io/?utm_source=polymail&utm_medium=referral&utm_campaign=signature )
On Sun, 12 Jul 2020 at 19:44 "Ignacio Roldán Etcheverry" < ">"Ignacio Roldán Etcheverry" > wrote:

I can't read the text in the screenshots. Try running Godot from the
terminal, e.g.: Godot.app/Contents/MacOS/Godot --path path/to/project
--editor --verbose. With the Print Build Output option enabled, try to
build the project and copy the full terminal output here.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub (
https://github.com/godotengine/godot/issues/40321#issuecomment-657260023 )
, or unsubscribe (
https://github.com/notifications/unsubscribe-auth/AQIJ2UY2NNEJLL56DOFWCO3R3H77LANCNFSM4OX4WQKQ
).

I ran the command-line you suggested and the error is inexplicable as the solution has been building for months as is.:

is there a way we can communicate about this without it being on the public internet..?

is there a way we can communicate about this without it being on the public internet..?

We prefer discussion to be public so everyone can benefit from it. Godot developers generally don't offer free private support (I don't).

Ok I only asked because the log output has my full name and personal folders in
Jerome Mayeux
Sent from Polymail ( https://polymail.io/?utm_source=polymail&utm_medium=referral&utm_campaign=signature )
On Sun, 19 Jul 2020 at 15:34 Hugo Locurcio < Hugo Locurcio ( Hugo Locurcio<EMAIL_ADDRESS>) > wrote:

is there a way we can communicate about this without it being on the
public internet..?

We prefer discussion to be public so everyone can benefit from it. Godot
developers generally don't offer free private support (I don't).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub (
https://github.com/godotengine/godot/issues/40321#issuecomment-660653919 )
, or unsubscribe (
https://github.com/notifications/unsubscribe-auth/AQIJ2U6GU7AVVKSYYUSCVS3R4MAAFANCNFSM4OX4WQKQ
).

any idea why my build is failing (based on the output) ?

I can't say for sure without an example project, but looking at the error I would bet this was happening because you were trying to build a 3.2.2+ project on an older version. There were some breaking changes introduced in that version.


make COMPLEX column filterable in Druid code

This patch removes the assumption made by Druid core that COMPLEX columns can't support filters.
I'm writing a custom aggregator.. extension takes full control of column representation by writing custom GenericColumnSerializer, and stores bitmaps for fast filtering based on values in this column.

Would it make sense to add a canFilter method to ColumnCapabilities to have a finer level of control than just mapping filterability directly to ValueType?

Would it make sense to add a canFilter method to ColumnCapabilities to have a finer level of control than just mapping filterability directly to ValueType?

Actually I think existence of Filterable.FILTERABLE_TYPE is redundant. For me, ColumnCapability.hasBitmapIndexes() is already the indication whether column supports filtering or not. So, one thing to do is to remove Filterable.FILTERABLE_TYPE altogether and adjust the code in ColumnSelectorBitmapIndexSelector to not use that.
however, I would like to keep things simple for this PR and not cause any unwanted side effects as above suggestion needs to get tested appropriately.

Actually I think existence of Filterable.FILTERABLE_TYPE is redundant. For me, ColumnCapability.hasBitmapIndexes() is already the indication whether column supports filtering or not.

They don't mean the same thing: for example, long columns are filterable but do not have bitmap indexes. IMO a capability makes sense and is a good way to solve this.

@gianm agree that columns can be filterable without bitmap indexes. however, independently,
only use of Filter.FILTERABLE_TYPE is in ColumnSelectorBitmapIndexSelector.columnSupportsFiltering(column) which looks like
  private static boolean columnSupportsFiltering(Column column)
  {
    ValueType columnType = column.getCapabilities().getType();
    return Filters.FILTERABLE_TYPES.contains(columnType);
  }

and at its call sites, it is used to check whether to work with an empty bitmap or get one from column. it appears, everything would stay exactly same even if I changed above implementation to...
  private static boolean columnSupportsFiltering(Column column)
  {
     return column.getCapabilities().hasBitmapIndexes();
  }

with above done, there is no more any usage of Filter.FILTERABLE_TYPE .
I must be missing something.

@himanshug probably it makes sense for this new capability to always be true for string, long, float, double; and for it to be false by default for complex, but it can be overridden.

@SuppressWarnings("unused") I believe

@clintropolis thanks, dint realize it was same as that for intellij... let me add that.

Hmm, I might be confusing it with intellij 😛

If it's not that it's probably
 // noinspection unused

?

no, I think you are right ... teamcity is from jetbrains and uses same set of inspections. so I guess it would be same... but the build will confirm soon.


Error publishing zip

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az webapp deployment source config-zip
Errors:
Expecting value: line 1 column 1 (char 0)
Traceback (most recent call last):
python3.7/site-packages/knack/cli.py, ln 206, in invoke
    cmd_result = self.invocation.execute(args)
cli/core/commands/__init__.py, ln 523, in execute
    raise ex
cli/core/commands/__init__.py, ln 581, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
cli/core/commands/__init__.py, ln 574, in _run_job
    six.reraise(*sys.exc_info())
...
lib/python3.7/json/decoder.py, ln 337, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
lib/python3.7/json/decoder.py, ln 355, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az webapp deployment source config-zip --resource-group {} --slot {} --name {} --src {} --verbose --debug

Expected Behavior
Environment Summary
Darwin-18.5.0-x86_64-i386-64bit
Python 3.7.3
Shell: bash

azure-cli 2.0.63

Extensions:
azure-devops 0.3.0

Additional Context

@stefano-corricelli, please some context on top of the auto-generated issue would be helpful.
However from the looks of things it looks like you are passing a json file or string that is incorrectly formatted.
cc: @panchagnula

@stefano-corricelli can you re-try it looks like the API is not returning a valid JSON at times - have seen this recently myself once - also this doesn't block the Zip Deployment itself, this is the API that we query to get the status of the Zip Deployment - so most like Deployment worked - but command has failed.

Not sure if this is related but I am trying to deploy and am getting an error:

az webapp deployment source config-zip --resource-group "XXXX" --name "XXXX" --src "code.zip"
Getting scm site credentials for zip deployment
Starting zip deployment. This operation can take a while to complete ...
Configuring default logging for the app, if not already enabled
Zip deployment failed. {'id': '410e1c8837cf45b398466a85c17e1473', 'status': 3, 'status_text': '', 'author_email': 'N/A', 'author': 'N/A', 'deployer': 'Push-Deployer', 'message': 'Created via a push deployment', 'progress': '', 'received_time': '2019-05-31T16:22:26.2990039Z', 'start_time': '2019-05-31T16:22:26.595878Z', 'end_time': '2019-05-31T16:22:49.7990506Z', 'last_success_end_time': None, 'complete': True, 'active': False, 'is_temp': False, 'is_readonly': True, 'url': 'https://XXXX.scm.azurewebsites.net/api/deployments/latest', 'log_url': 'https://XXXX.scm.azurewebsites.net/api/deployments/latest/log', 'site_name': 'XXXX', 'provisioningState': None}. Please run the command az webapp log tail
-n XXXX -g XXXX

And then while watching the logs I tried again and saw:

az webapp log tail -n XXXX -g XXXX
2019-05-31T16:22:11  Welcome, you are now connected to log-streaming service.
2019-05-31T16:23:11  No new trace in the past 1 min(s).
2019-05-31 16:22:25 ~1XXXX POST /api/zipdeploy isAsync=true&X-ARR-LOG-ID=189bc58a-a895-4573-a082-2083029e5cd2 443 - <IP_ADDRESS> python-requests/2.22.0 - - XXXX.scm.azurewebsites.net 202 0 0<PHONE_NUMBER> 2303
2019-05-31 16:22:35 ~1XXXX GET /api/deployments/latest X-ARR-LOG-ID=1cffe7e7-0b4b-4541-bda5-7d39f0a8d284 443 - <IP_ADDRESS> python-requests/2.22.0 - - XXXX.scm.azurewebsites.net 202 0 0 1153 1095 6789
2019-05-31 16:22:42 ~1XXXX GET /api/deployments/latest X-ARR-LOG-ID=55c66ae6-5383-4367-aec1-67cc1a55b20f 443 - <IP_ADDRESS> python-requests/2.22.0 - - XXXX.scm.azurewebsites.net 202 0 0 1153 1095 2620
2019-05-31 16:22:47 ~1XXXX GET /api/deployments/latest X-ARR-LOG-ID=517ed0d3-1ed8-4c78-95c9-650199b59e17 443 - <IP_ADDRESS> python-requests/2.22.0 - - XXXX.scm.azurewebsites.net 202 0 0 1153 1095 1109
2019-05-31 16:22:52 ~1XXXX GET /api/deployments/latest X-ARR-LOG-ID=57390637-d345-4f02-b06b-d1686d5999d3 443 - <IP_ADDRESS> python-requests/2.22.0 - - XXXX.scm.azurewebsites.net 200 0 0 1027 1095 187

I also confirmed on Kudu that the files were not deployed.

closing this due to inactivity please let us know if this is still an issue. thanks!


Stringify returns incorrect values on CompoundTags

There is flawed logic in the Stringify method of the CompoundTag class, and likely the ListTag as well. When appending children to the StringBuilder, it fails to call Stringify on them, therefore appending the ToString representation.

Fixed with 82399f2 and deployed in 1.3.1.


Allow enum types starting with underscore

GraphQL enum types, which start with an underscore are converted to elm modules that start with an underscore, which is not valid elm code.
APIs generated with graph.cool use an enum type, wich starts with an underscore. Here is the corresponding part of the introspection query:
{
          "inputFields": null,
          "name": "_ModelMutationType",
          "description": null,
          "interfaces": null,
          "enumValues": [
            {
              "name": "CREATED",
              "description": null,
              "isDeprecated": false,
              "deprecationReason": null
            },
            {
              "name": "UPDATED",
              "description": null,
              "isDeprecated": false,
              "deprecationReason": null
            },
            {
              "name": "DELETED",
              "description": null,
              "isDeprecated": false,
              "deprecationReason": null
            }
          ],
          "fields": null,
          "kind": "ENUM",
          "possibleTypes": null
        }

@mkaulig thank you for reporting, and thanks for the very clear issue! I should be able to get a fix out for this and #12 within the next few days.


Rejected requests counter always being updated and saved.

Per the ReadMe:

By default, rejected calls are not added to the throttle counter.
...
If you want rejected requests to count towards the other limits, you'll have to set StackBlockedRequests to true.

Looking at the code, I can see that Repository.Save is called within ThrottlingCore.ProcessRequest, which is called from ThrottlingFilter roughly 10 lines before the overlimit check is performed.
Screenshot from Redis showing a counter of 13 (this was with a perMinute limit of 10):

I've tried this with setting StackBlockedRequests explicitly to false as well as leaving the default, both with the same result.

I have found that this is also the case with the ThrottlingFilter, but not the ThrottlingHandler.


The Keyboard focus moves out of the Pop up on pressing Tab.

“Check out Accessibility Insights! - Identify accessibility bugs before check-in and make bug fixing faster and easier.”
GitHubTags: #A11yMAS;#A11yTCS;#A11ySev2;#VisualStudioCode-Win32-Apr2021;#Visual Studio Code Client;#DesktopApp;#WCAG2.4.3;#AINotInScope;#FTP;#FocusOrder;#Keyboard;
Environment Details:
Screen Reader: NVDA/JAWS
Operating system: Windows 10
Repro Steps:

Launch VS Code application.
Tab till "Azure" and press enter.
Tab till "app service " dropdown and press enter.
Tab till "Files" dropdown and press enter.
Tab to any file and press enter.
Edit by adding "make changes" and save(ctrl + S) the "SRC" file..
Tab till the Pop up Saving "hostingstart.html".
Check if the keyboard focus moves out of the Pop up on pressing Tab.

Actual:
The Keyboard focus moves out of the Pop up on pressing Tab.
Note: This issue is observed with all the Notification Pop up that appear for "Initializing SSH" Pop up scenario.
Expected:
The Keyboard focus should remain in the Pop up on pressing Tab.
User Impact:
Keyboard users will be confused if the keyboard focus moves out of the Pop up on pressing Tab key.
Please find the below attachment for reference:

@v-gitsay The upper right toolbar in the notifications is following best examples of aria toolbar https://www.w3.org/TR/wai-aria-practices/examples/toolbar/toolbar.html
Which means that the toolbar is only one tab stop, it should not be multiple tab stops as that would make the navigation harder for the user. You should use left / right to navigate inside a tooblar.
Everything is behaving as designed here, thus closing this issue and we have heard positive feedback from users regarding this. Thanks


upgrade web3py

requested by Discord community member: https://discord.com/channels/869983523371642921/869986794970693723/1091468590261227540

Hi there,
web3.py v5 support will end on August 31, 2023. See issue #3044

Also, i noticed a little issue with forta_agent dependencies in python:
web3 v5 has bug with python 3.11 (calls getargspec instead of getfullargspec or another alternative, py-3.11 changed this name)
During dev I can just use python 3.9 and no issues come.
For deployment it means that:

either i patch the calls and use the latest containers, life is good
either i compile with python 3.9, which means that i have to deploy with an old container (like node:12-alpine) so that it uses python 3.9 too

This requires the node deps to be specified in package-lock format 1 or 2, which is under older node versions like 12, during dev.
All in all a little messy ^^'


Dependency Clean Up Request

Make the dependency on tensorflow-lite optional (use 'feature' for meson_option)
Refactor backend options with 'feature' (for 'auto' mode)

:octocat: cibot: Thank you for posting issue #2333. The person in charge will reply soon.

Make the dependency on tensorflow-lite optional (use 'feature' for meson_option)
Refactor backend options with 'feature' (for 'auto' mode)

Rationale:
It's quite strange to require tf-lite when you build w/o any meson options. With 'feature', you can enable tf-lite only if it's installed (no error if tf-lite doesn't exist) However, for distro builds (Tizen, Ubuntu, ...), we need 'force-enable', which will emit errors if it doesn't exist.
CC: @DonghakPark

Okay, i will clean up dependency ASAP

It's not an ASAP item. We do not have a deadline for this "long-term maintenance". If you have any other pending tasks with deadlines, you should do those first.


Update Preview icon to align with Edge

Environment
Windows build number: 19628
Windows Terminal version (if applicable): 1.0.1402

Steps to reproduce
Install Windows Terminal Preview
Expected behavior
The Terminal Preview icon and Edge Insider icon's preview-labels line up nicely and use the same font and size.
Actual behavior
Given the basic design, it feels obvious that the Terminal Preview's "Preview" label is modelled after the Microsoft Edge icon.

But the labels don't line up nicely and a different font (weight) is used for the Terminal compared to Edge.

It's a very insignificant issue. But I'd just like to point it out because - as someone who uses both regularly - it annoys me. 🙃

@cinnamon-msft we'll need to move the pill around a little bit. I'll see if I can do that for fun in my icon generation script.
Had some ideas about merging the pill in as a separate image instead of part of the terminal icon which will give us breathing room on the larger tiles where there's more than space enough for the icon

It's also worth noting that on lower-resolution displays the Edge Dev icon still shows the badge with text, but the terminal icon removes the text and just shows a blue bar.

@k-yle given that our badge is shorter than Edge's, we thought it imprudent to have a tiny black smear you didn't stand a chance of reading.

@k-yle given that our badge is shorter than Edge's, we thought it imprudent to have a tiny black smear you didn't stand a chance of reading.

FYI, the badge does display with text when pinned to the start menu as a "small" size icon:


blockstore: always send a coding shred for chained merkle root conflicts

Problem
#835 has a chance to send a chained merkle root conflict duplicate proof consisting of 2 data shreds.
With 2 data shreds we cannot verify that they are from adjacent FEC sets. We require that the shred from the lower FEC set be a coding shred.
Summary of Changes

Modify the backwards check to always send the coding shred from lower FEC set. We know this is available because there must be an erasure meta.
Modify tests to check the upgrade scenario, where the first received coding shred index is not reliable (skip the check in this case).

This ensures that we always send a coding shred for the lower FEC set, meaning the gossip handler can verify that duplicate proofs are for adjacent FEC sets.
Without this verification an attacker can arbitrarily pick non adjacent FEC sets to "prove" that every block is duplicate
Contributes to https://github.com/solana-labs/solana/issues/34897

Codecov Report
Attention: Patch coverage is 85.18519% with 4 lines in your changes are missing coverage. Please review.

Project coverage is 82.8%. Comparing base (d1afd78) to head (ccf8d64).
Report is 6 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff            @@
##           master    #1353    +/-   ##
========================================
  Coverage    82.8%    82.8%
========================================
  Files         867      867
  Lines      368897   368881    -16
========================================
+ Hits       305591   305720   +129
+ Misses      63306    63161   -145


Write Product Development Chapter

Write about the 3.4 Product Development Chapter

Description of the Production Vision Board included

wrote description of product development and some description on product vision board


Exec tachikoma update<PHONE_NUMBER>3325

:hamster::hamster::hamster:   Strategy none
Powered by Interval Pull Request as a Service - Tachikoma.io Announcement: 2015-06-28 Updated permissions - Tachikoma.io

no change


mocha.global() function doesn't work

Prerequisites

[x] Checked that your issue hasn't already been filed by cross-referencing issues with the faq label
[x] Checked next-gen ES issues and syntax problems by using the same environment and/or transpiler configuration without Mocha to ensure it isn't just a feature that actually isn't supported in the environment in question or a bug in your code.
[x] 'Smoke tested' the code to be tested by running it outside the real test suite to get a better sense of whether the problem is in the code under test, your usage of Mocha, or Mocha itself
[x] Ensured that there is no discrepancy between the locally and globally installed versions of Mocha. You can find them with: node node_modules/.bin/mocha --version(Local) and mocha --version(Global). We recommend that you not install Mocha globally.

Description
I'm trying to use mocha,globals() function to set up a global for the project:
//runner.js
const Mocha = require('mocha');
var myInfo = 'Hello world';
(async () => {
    try {
        const mochaConfig = {timeout: 100000};
        const mocha = new Mocha(mochaConfig);
        mocha.global(['myInfo']);
        mocha.files = ['./test1.js'];
        mocha.run((failures) => {
            process.on('exit', () => process.exit(failures ? 1 : 0));
        });
    } catch (err) {
        process.exit(1);
    }
})();

//test1.js
describe('example spec 1', function() {
    it('first test 1',async function() {
        console.log(myInfo); //error
        console.log(global.myInfo); //undefined
        console.log(this.test.globals()); //undefined
        // what should I write to receive myInfo global ?
    });
});

Steps to Reproduce

Expected behavior: I can access myInfo variable as global variable or somehow else in body of tests
Actual behavior: I can not access myInfo variable as global or any other ways

Reproduces how often: Each run
Versions

mocha version 9.1.3
.- node version 14.17.6
Your operating system

windows 10
64-bit

PowerShell

Additional Information
May be I missed something but just from official site information I didn't find any way to get access to a variable added as global using mocha.global() method.
And also is it possible to access to the globals in Worker threads if parallel option is set true

Mocha's functions as describe or it are on the global object, therefore you don't need to require/import them in your tests. So if you need a property in global scope, then just add it to global.whatever  = .... This is javascript and has nothing to do with Mocha.
mocha.global() has a different purpose, see also mocha.checkLeaks().

And also is it possible to access to the globals in Worker threads if parallel option is set true

Each worker process has its own global, global properties are not transmitted from main to child processes. Only environment variables are passed through from main to children.


Update GetAckTimeout to use the backoff algorithm

Problem
SecureSession::GetAckTimeout and UnauthenticatedSessionTable::GetAckTimeout always use the idle specified by MRP without taking into account the PeerActiveMode.
Also, the current formula calculation:
GetRemoteMRPConfig().mIdleRetransTimeout * (CHIP_CONFIG_RMP_DEFAULT_MAX_RETRANS + 1); represent a static time and not a time that changes such as represented by mrpBackoffTime.
If someone uses GetAckTimeout to configure the exchange response timeout, it may end up into ending the exchange before all MRP retransmissions is done.

The cirque failure is interesting. It fails when trying to connect to a network:
INFO:NetworkCommissioning:Connect to a network
DEBUG:chip.EM:Flushed pending ack for MessageCounter:220181293 on exchange 44264i
DEBUG:chip.CSM:FindOrEstablishSession: PeerId = [6:0000000000000001]
DEBUG:chip.CSM:FindOrEstablishSession: No existing OperationalSessionSetup instance found
INFO:chip.CTL:Found an existing secure session to [6:0000000000000001]!
DEBUG:chip.CTL:OperationalSessionSetup[6:0000000000000001]: State change 1 --> 5
DEBUG:chip.DMG:ICR moving to [AddingComm]
DEBUG:chip.DMG:ICR moving to [AddedComma]
INFO:chip.EM:<<< [E:44265i M:190906594] (S) Msg TX to 6:0000000000000001 [AD8E] --- Type 0001:08 (IM:InvokeCommandRequest)
INFO:chip.IN:(S) Sending msg 190906594 on secure session with LSID: 26183
DEBUG:chip.DMG:ICR moving to [CommandSen]
INFO:chip.EM:>>> [E:44265i M:220181294 (Ack:190906594)] (S) Msg RX from 6:0000000000000001 [AD8E] --- Type 0000:10 (SecureChannel:StandaloneAck)
DEBUG:chip.EM:Found matching exchange: 44265i, Delegate: 0x15ad0f8
DEBUG:chip.EM:Rxd Ack; Removing MessageCounter:190906594 from Retrans Table on exchange 44265i
DEBUG:chip.IN:SecureSession[0x7fffe0048bd0]: MarkAsDefunct Type:2 LSID:26183
INFO:chip.SC:SecureSession[0x7fffe0048bd0]: Moving from state 'kActive' --> 'kDefunct'
INFO:chip.DMG:Time out! failed to receive invoke command response from Exchange: 44265i

I suspect that it uses to work because GetAckTimeout was using the idle interval by default (5000 in this case) which ends up giving more than a minute of additional time for the response timeout to fire (70478ms for idle versus 4224 for active in this case).
It does not seems right that connect was working just because the code was assuming an idle timeout. I will dig to see what I can do.

It does not seems right that connect was working just because the code was assuming an idle timeout

Hmm.  What are the parameters involved there?  I assume the connect is an async process, so it sends the ack, but the exchange timeout is being hit?  What "time it will take the server" are we providing as input to that connect call?  It must be at least 30 seconds, since this is during commissioning, right?  So 30s + active MRP bits (34s) is not enough in this case?  Maybe we should be using a higher timeout for ConnectNetwork in AutoCommissioner?

The python test suite is not using the AutoCommisioner but instead it send raw commands. See

Well, that would be the bug, the.  It's violating the spec, if it allows less than 30s on the server for any commissioning-related command.
@mrjerryjohns @tcarmelveilleux @yunhanw-google could you please take a look at the python bits there?

@vivien-apple Does just adding interactionTimeoutMs=60 to that SendCommand call help for now?  That won't get MRP bits added, given what command.cpp is doing, but it might be enough to at least get this test to pass...

@vivien-apple Does just adding interactionTimeoutMs=60 to that SendCommand call help for now? That won't get MRP bits added, given what command.cpp is doing, but it might be enough to at least get this test to pass...

Yes, I have tried already to add interactionTimeoutMs=120000 and as you can see  Cirque is green ;)


Properly resolve paths to temporary response files when autolinking

Previously, .autolink files were passed as flags throughout the driver so that they could be prefixed by "@". This PR adds a new Job.ArgTemplateKind, responseFilePath to make sure we resolve the underlying path properly. This fixes most of the Driver/tools_directory integration test (there's at least one other remaining unrelated issue preventing it from passing)

@swift-ci test


Add config to change the default query row limit

Currently, if no LIMIT if specified, a query will only return 10 results. Add a config to change the default number.

In general, keeping a lower number of configurations keeps a product easier to use and document. Along these lines, I think a config variable like this does not add much value to pinot.
My vote is a -1
I am happy to hear thoughts from others though.


Fix typescript linting

Fixes the issue found in #348 where the linter considered a used enum to be unused.
Does this by using the recommended typescript rules which properly account for that.
This also meant that quite a few files needed to be updated because things that previously weren't caught are now caught (e.g. explicit any isn't allowed anymore)
The biggest change is is utils.ts where the buildProblem function I made needed to be cleaned up. The changes to it are a lot nicer since now the type inference gets the return type correct is most cases now. The clean up to the mapper types also means making generics for the simulator are a lot nicer

ew github ui doesn't let me change message

Weird the tests didn't fail in your PR but I know the issue I think

Weird the tests didn't fail in your PR but I know the issue I think

Hm strange, my guess is that the meta/config data in the DFA JSON could be decided in real time rather than by the details of the NFA JSON?

Dw, it's just cause we don't add the extra project data to the graph in the conversion because there are now actual functions in the projectStore for working with graph data


[Java] Adding examples about Dictionary-encoded Layout

#204

Please @lidavidm if you could help me with a review, thanks.


Update dependencies- to dev branch

see PR #21

Thanks! I no longer have write permissions to this repo, but I notified the Push team so that someone can merge this PR.

Coverage decreased (-22.2%) to 77.778% when pulling d6e6417dbdae0b9c4d76f115e4487611c32928b4 on jkingoliver:master into 903a198754e552cbb41fd7326b1bc78bd96a1c7d on ibm-bluemix-mobile-services:development.


docs: Remove S from extraVolumeMounts

Remove S from extraVolumeMounts in documentation

extraVolumeMounts is the correct field name since it adds entries to the volumeMounts section. Instead of changing this field name, can you fix the typo in the message in NOTES.txt?

Sure, done. Thank you for clarification.

Regenerate examples with new version error.

@dmitryax please review


Remove panicking code from casper_types crate (Issue #2801)

closes #2801

Replaced unnecessary panicking code with results in types folder.

bors merge

bors merge


VisualRust.Shared is missing from installation package

The generated installation package VisualRust.msi is missing VisualRust.Shared.dll that is required for VisualRust.Build.dll to run. This results in this exception during build:
------ Build started: Project: RustApplication2, Configuration: Debug AnyCPU ------
C:\Program Files (x86)\MSBuild\VisualRust\VisualRust.Rust.targets(19,5): error : Could not load file or assembly 'VisualRust.Shared, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=401f5d177ee7d4a2' or one of its dependencies. The system cannot find the file specified.
   at VisualRust.Build.Rustc.ExecuteInner()
   at VisualRust.Build.Rustc.Execute()
Done building project "RustApplication2.rsproj" -- FAILED.

VIsualRust build used:
https://ci.appveyor.com/project/vosen/visualrust/build/job/ubhcib4nvxt95i72/artifacts
Building VisualRust.Shared manually and additing to C:\Program Files (x86)\MSBuild\VisualRust\ helps.

Thanks, it should be fixed now


Goldpinger

Hi team!
As you know, we have used goldpinger in a real environment provided us a good view about the network situation of the Kubernetes cluster. As it was an useful utility, we considered to include it in this monitoring module.
Thanks!

 Add goldpinger to our monitoring package


Upgrade to Python 3.12

Motivation
Move to a Python version with active support
Changes

Workflow

⚠ Create your PR as draft ⚠
Work on you PR until the CI passes (if something not related to your task is failing, you can ignore it)
Mark it as ready for review

Test logic is modified? -> Get a review from RFC owner. We're working on refining the codeowners file quickly.
Framework is modified, or non obvious usage of it -> get a review from R&P team

:rocket: Once your PR is reviewed, you can merge it!
🛟 #apm-shared-testing 🛟
Reviewer checklist

[ ] Relevant label (run-parametric-scenario, run-profiling-scenario...) are presents
[ ] If PR title starts with [<language>], double-check that only <language> is impacted by the change
[ ] No system-tests internal is modified. Otherwise, I have the approval from R&P team
[ ] CI is green, or failing jobs are not related to this change (and you are 100% sure about this statement)
[ ] A docker base image is modified?

[ ] the relevant build-XXX-image label is present

[ ] A scenario is added (or removed)?

[ ] Get a review from R&P team

It'll probably break local install, to reduce complaints, could you add  on run.sh a detection if it's python 3.9, and exit with an error message :

Please run rm -rf venv && ./build.sh -i runner 🙏

ok, LGTM. Let me list repos where we need to do a PR. I'll put the list in the first message


Fix viewer asset name

@rgauss is your preference to update this repo? Or to rename the asset attached to the release?

@rgauss is your preference to update this repo? Or to rename the asset attached to the release?

I think oscal-viewer.zip is more specific (thinking of it sitting in a downloads folder for 6 months) so let's keep that.


feat: Add TimeSlider animation to maps + consolidate chart logic pt II

This PR:

adds an ability to animate the maps,
continues improvements to sharing the chart logic.

Chart logic sharing
I consolidated the chart logic of every chart to use the variables - data - state pattern, where we first compute necessary variables (getters, axis labels, etc), then use them to create the base, sorted data for a chart (chartData, scalesData, segmentData, allData) which is then used to create a proper chart state.
Next steps

Consolidate duplicated logic to compute shared things in the charts on a scale level (e.g. a hook to compute the getters for XBand dimension)
Extract computation of chart-specific data (e.g. wide data for XBand dimension, etc)
Extract computation of segments and color scales
Update MapState type

Important: It's not possible to animate the map properties (colors, radiuses, etc) when using interleaved mode:

WebGL2 based deck.gl features, such as attribute transitions and GPU accelerated aggregation layers cannot be used.

https://deck.gl/docs/api-reference/mapbox/overview#limitations

@bprusinowski wonderful 🎉


Can't download podcast (you are not an admin or uploader)

I have a fresh Ubuntu 22.04 installed on a VM and installed Podfetch via Portainer with the Postgres file here on Github. I then mounted an SMB share and linked my _data to that share via symlink (i'm guessing my issue, but I can't get my SMB share to mount as a Volume in portainer. just get error 500 after setting up an SMB volume and setting it to this container). I then enabled basic authentication and made myself a user with admin rights. After signing in to PodFetch when I try to add a podcast I get an error that "You are not an admin or uploader" though that is the role that shows up in the cli. Any thoughts? Even if I can get pointed in the right direction to mount the SMB shares correctly I would appreciate it.
 This is my first foray into docker and portainer, so i'm fumbling around in the dark trying to learn as I go.

This is likely because the PodFetch container user does not have the correct permissions. You need to set the user and group id to the user that also starts the container. This is most likely the id 0

So I think finally got the volume mapped correctly after a quick walk away from the desk, and chaning uid/gid to 0 in the yaml file lets me add Podcasts. However, I can't hit the "download to server" button, it only lets me download locally to my computer.
Maybe I have this setup wrong still, but I don't know where
`version: '3'
services:
podfetch:
image: samuel19982/podfetch:latest
user: ${UID:-0}:${GID:-0}
ports:
- "2080:8000"
depends_on:
- postgres
volumes:
- /var/lib/docker/volumes/SMBShare/_data:/app/podcasts
environment:
- POLLING_INTERVAL=300
- SERVER_URL=http://localhost:80 # Adjust to your server url
- DATABASE_URL=postgresql://postgres:changeme@postgres/podfetch
- DB_CONNECTIONS=10 # optional
postgres:
image: postgres
environment:
POSTGRES_USER: ${POSTGRES_USER:-postgres}
POSTGRES_PASSWORD: ${POSTGRES_PASSWORD:-changeme}
PGDATA: /data/postgres
POSTGRES_DB: ${POSTGRES_DB:-podfetch}
volumes:
- postgres:/data/postgres
restart: unless-stopped
volumes:
postgres:`

My best guess is that: Is the frontend url correctly configured? Are you accessing PodFetch via localhost in your browser?

That would be something I forgot to set again when I re-installed portainer.
Hopefully last question, and it's really more of a Portainer question, so maybe it would better asked there. However, I have my CIFS share mounted under volumes, but Podcasts are saving to a local version, not the NAS share. What is the correct way to map this with Docker containers?

I figured it out. If I mount it on the host and bind it in my volumes, it works.
The only other thing I cant figure out is how to name these correctly. They are going into named folders, but they're all named podcast.mp3. I think this is why Audiobookshelf is picking them all up as separate things instead of podcasts of the same series. Is there a list of how to use the naming conventions to address this by chance?

I figured it out. If I mount it on the host and bind it in my volumes, it works.
The only other things are

I cant figure out is how to name these correctly. They are going into named folders, but they're all named podcast.mp3. I think this is why Audiobookshelf is picking them all up as separate things instead of podcasts of the same series. Is there a list of how to use the naming conventions to address this by chance?
I can't seem to do anything besides add podcasts once I put PodFetch behind my reverse proxy (caddy). Updated the ENV for the URL to podcasts.mydomain.com and enabled BAISC_AUTH. Can log in and add series (which then download) but can't delete episodes or unsubscribe.

There is an option in the settings menu called direct paths. This maps all episodes of a podcast into the root directory. That way audiobookshelf will detect it as one podcast.
That sounds like a misconfiguration. Did you change the frontend url correctly and also added https://?

Direct Paths fixed it
Apparently I was just impatient, looks like it's working fine now.

Thank you so much for all the help with this. Everything seems to be working great now.

You're welcome :). Have fun with PodFetch.


Add missing dockerfile markup tests

As requested in #1771  :)

@l-vo Thanks a lot!

You're welcome :)


Fix UT testLargeFileInLowerLevel timeout

Description

This UT is testing large file merging, which requires a large amount of memory, and the CI environment has a small memory. As a result, the file merging is limited due to the merging memory control, and the merging cannot be completed.
To fix it, we need setEnableCompactionMemControl= false for this UT.

Codecov Report
Attention: 17 lines in your changes are missing coverage. Please review.

Comparison is base (f3b3d39) 49.03% compared to head (eab991a) 49.12%.
Report is 12 commits behind head on master.

Files
Patch %
Lines

...e/dataregion/memtable/AlignedWritableMemChunk.java
0.00%
12 Missing :warning:

...he/iotdb/db/utils/datastructure/AlignedTVList.java
85.29%
5 Missing :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##             master   #11568      +/-   ##
============================================
+ Coverage     49.03%   49.12%   +0.09%
- Complexity    24753    24815      +62
============================================
  Files          2781     2789       +8
  Lines        174238   174548     +310
  Branches      20855    20896      +41
============================================
+ Hits          85437    85750     +313
+ Misses        88801    88798       -3

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


not working..

Expected Behavior
(What did you expect to happen?)
Actual Behavior
(What actually happened?)
Steps to Reproduce the Problem

Specifications

Browser:
OS: Mac/Linux/Windows

The app crashed sometime between Thursday Feb 6 and Friday Feb 7.   See comments #82, #83, #84, #84, #85 and #86.


Tests with function calls in the name are not found

defmodule MyTest do
  use ExUnit.Case

  test inspect(&MyModule.hello/1) do
    # ...
  end
end

^Such tests are not found.
The same, but with interpolation, works okay:
  test "#{inspect(&MyModule.hello/1)}" do

I hadn't seen test names with arbitrary expressions like that. I'll add support for it.

I use it quite a lot to have module, function and arity in sync with implementation. Thanks :)

Supporting arbitrary calls turned out to be quite tricky, probably requires a refactor. Similar to #19.
But I managed to support the specific case "inspect" because it results on a somewhat predictable test name.

@jfpedroza Thanks! This helps a lot.
I'd like to ask support this for describe as well :)

Makes sense. I just realized we don't support dynamic describes at all.
Can you open a separate issue for that? :)

Sure


fix(semantic release): Adding semantic-release

:tada: This PR is included in version 1.0.2 :tada:
Your semantic-release bot :package::rocket:


Not compatible with RaspiRobot V3f?

Hello!
I'm using raspi-io together with johnny-five for a pi rover project. I encountered an issue when trying to setup the motors.
According to the python example I got with the rover kit the following pins should be used:
    RIGHT_PWM_PIN = 14
    RIGHT_1_PIN = 10
    RIGHT_2_PIN = 25
    LEFT_PWM_PIN = 24
    LEFT_1_PIN = 17
    LEFT_2_PIN = 4
    SW1_PIN = 11
    SW2_PIN = 9
    LED1_PIN = 8
    LED2_PIN = 7
    OC1_PIN = 22
    OC2_PIN = 27
    OC2_PIN_R1 = 21
    OC2_PIN_R2 = 27
    TRIGGER_PIN = 18
    ECHO_PIN = 23

My code:
const five = require('johnny-five');
const board = new five.Board({
  io: new raspi({enableSoftPwm: true})
});

board.on('ready', function() {

var left = new five.Motor(['GPIO24', 'GPIO4', 'GPIO17']);
var right = new five.Motor(['GPIO14', 'GPIO25', 'GPIO10']);

  console.log("Start");
  left.forward(55);
  right.forward(55);

  board.wait(2000, () => {
   left.reverse(55);
   right.reverse(55);
  });

  board.wait(4000, () => {
   left.stop();
   right.stop();
  });
});

The error I get is that GPIO14 does not support mode pwm, when I try the "left" motor it works as intended. Any help at all would be much appreciated!

Try setting enableSoftPwm to true in the constructor and that will fix it.
The Raspberry Pi doesn't actually support PWM on GPIO14, but it's possible to emulate PWM in software. It comes at a bit of a cost though: it increases CPU usage and it's not quite as accurate. I came to a different conclusion on how surface this information to the user than the author's of the Python library, as I think it's important for the user to be aware of this.

const raspi = require('raspi-io');
const five = require('johnny-five');
const board = new five.Board({
  io: new raspi({enableSoftPwm: true})
});

board.on('ready', function() {

var left = new five.Motor(['GPIO24', 'GPIO4', 'GPIO17']);
var right = new five.Motor(['GPIO14', 'GPIO25', 'GPIO10']);

  console.log("Start");
  left.forward(55);
  right.forward(55);

  board.wait(2000, () => {
   left.reverse(55);
   right.reverse(55);
  });

  board.wait(4000, () => {
   left.stop();
   right.stop();
  });
});```

I do that already I think in the "new raspi({enableSoftPwm: true})" ? Or should it be somewhere else? Still no luck, also tried having it inside the "new five.Motor" but it won't work.

What puzzles me is that one of the motors are working as intended but not the other?

Oh whoops, sorry I totally missed you already had enableSoftPwm. Hmm, OK yeah that's strange. enableSoftPwm only works in the raspi constructor.
Can you give me the full stack trace of the exception?

According to the changelog GPIO14 and GPIO15 are not available with<EMAIL_ADDRESS>or higher. See https://github.com/nebrius/raspi-io/blob/master/CHANGELOG.md#600-2016-3-20.

Oh RIGHT! Thanks @fivdi, I forgot my own work there for a second :D

Ohhhh, I see the issue now. GPIO14 is reserved for use with the serial port. If you move it to a different pin it should work

The motor board is not soldered but can only be plugged into the raspberry pi in one way so I guess you could call it hard wired.

@nebrius sorry to be a pain but do you have any updates regarding this issue? Anything I can do?

@robertjarske sorry for the delay, I just got back from Burning Man so I've been off the grid for a long time.
I actually don't know if this will work cause I don't really remember this part of the implementation, but in an ideal world it will work: try passing enableSerial: false to the Raspi IO constructor and see if it starts working. If not, then I should add support for that.

@nebrius I tried setting enableSerial to false but unfortunately I get the same error  as before

@robertjarske Thanks for trying that. I'll leave this open so I can investigate getting this behavior changed. I can't promise I can make this change (there may be unintended side-effects of setting a pin mode on a serial port pin after the app is closed), but this sounds like a good change to make.

@nebrius thank you for looking into it

@robertjarske I know this is considerably late, I'm sorry for that. I just finished a major rewrite I started around the time of my last comment. Now it has logic to not filter out the serial pins if enableSerial is set to false. It should work for you now.

I'm going to go ahead and close this issue as the immediate problem is fixed, but if you run into any other problems please reopen and we'll work out the next step.


Manual cherry-pick of https://github.com/istio/istio/pull/26260

Manual cherry-pick  of #26260. Needed to prevent rare Pilot deadlock and memory leak in Syncz debug endpoint. @howardjohn @hzxuzhonghu
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[X] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Hi @Monkeyanator. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

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

/ok-to-test

@Monkeyanator: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio_release-1.7
b5e1b0f8e6608e319738b822c6cdd4d95f4a224b
link
/test release-notes_istio_release-1.7

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

@Monkeyanator: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio_release-1.7
7bbba82ac8c38aef1c7cd7209ebe4aaa9c182e5d
link
/test release-notes_istio_release-1.7

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

@Monkeyanator: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio_release-1.7
667518bb9dd7bc216d32584a9efcea1f2a6fc67b
link
/test release-notes_istio_release-1.7

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

@googlebot I consent.


Allow minting on polygon and sidechain inside app

Currently we do not support in-app NFT minting.
We need to add a context menu that allows the user to mint an object as NFT directly through the app.
Not every dragged and dropped object is an NFT so a menu would be better.

as discussed with avaer, we will not be supporting this currently


reorder tabs around on the right dock (inspector, history, node) is bugged

Godot version
4.2 beta3 mono
System information
Windows 10
Issue description
can't really add anything extra
Steps to reproduce
do it
Minimal reproduction project
n/a

Check latest beta, should be fixed now.

This was fixed by https://github.com/godotengine/godot/pull/83966

Closing as fixed, if the problem still occurs in the latest version please comment here


navbar fixing

Fixing the navbar visibility issue.

@Naga-Himaja i created a fresh issue for this, you can start working


Support composite types in the type service

The type service should support composite types such as TreeSets, HashMaps and ArrayLists as well as standard arrays.
They should be incorporated into the type service and should be generic (ie the type service should not store "java.util.ArrayList<java.lang.String>" as a type)
They should also be supported by the dynamic input component possibly using md-chips.

The UI fails to meet our Accessibility policy. It is unmaintainable and will be removed shortly.