Feature: Dashboard filters: datepicker and time granularity

Make sure these boxes are checked before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if any
[x] I have reproduced the issue with at least the latest released version of superset
[x] I have checked the issue tracker for the same issue and I haven't found one similar

When checking "Date" under options in the dashboard filter explore view, only the default time options for since and until are available, with no ability to set time granularity.

The time granularity option and datepicker should be available for dashboard filters.
Superset version
0.19.3
Expected results
Dashboard date filters should have time granularity and datepicker.
Actual results
Dashboard date filters only allow the default time options

Why did you close this item? I have the same problem.

Huh that's odd, I must've hit it by accident. Anyway I did make a PR for this https://github.com/apache/incubator-superset/pull/3508

Notice: this issue has been closed because it has been inactive for 206 days. Feel free to comment and request for this issue to be reopened.

Not stale


Theme demo not operating at advertised URL

I'm pretty certain I checked out your demo in the past and I'm sure it once worked anyway.
Going to the demo as per the theme blueprint and Grav themes listing, I am redirected to https://jtbray.com/#/login which is a 3CX Management Console login.

i think is was just doing something and dont care now...


Handle attachments with more than one period in the filename

Failed to replicate with same input as provided by the user.


Recommend using pipx

I have personally used pipx extensively on my linux device, and seems to be the best way to avoid the dependency hell of python. It basically install a python package in it's own python environment, while also making it accessible in the PATH. It also has significantly less overhead than Docker.
PS: This will reduce the number of installation related issues too.

@harshcurious thank you for your interest.
I have heard about the tool, but don't really have an experience with it and that is why I don't really feel comfortable recommending it. However, if I understand it correctly this would not require any changes in the source code, right?
Note that you don't have to use Docker to make things run. Another approach is below
Without pipx (using venv)
Install time
cd $SOME_FOLDER
python -m venv venv
source venv/bin/active
pip install mltype

When using
cd $SOME_FOLDER
source venv/bin/active
mlt --help

Thanks a lot for the reply. I agree that it is possible to create a python environment (using a variety of tools; I prefer conda) and run it. The issue with that approach is that I will have to remember to activate/deactivate the environment on every run. Pipx does that automatically and provides a direct access to it. I personally use Manjaro (Arch based) as my daily driver and the python packages from AUR broke often. So I have personally started using pipx. It does not require any changes on the part of the package maintainers. As long as pip works, pipx also works.
To use pipx, you need to first install pipx using pip (I do it with the default python installed on the system):
pip install pipx
pipx ensurepath

Details: https://pypa.github.io/pipx/installation/
Once pipx is in the PATH all you need to install a python library is:
pipx install <package_name>

It install the package into its unique isolated environment and the package can be accessed from the terminal (i.e. the binary is in the PATH)
Upgrade is easy too. You can either upgrade all your pipx python packages using pipx upgrade --all or just one particular one using pipx upgrade <package_name>
But I understand your reluctance, there are so many python packager/installers that have failed that recommending Docker seems safer :P
PS: I installed this using pipx and it worked fine, in case someone else wants to use it.

@harshcurious I think it is a great suggestion. Thank you for it. I am sure that people will want to use pipx will stumble upon this issue and will be able to follow your instructions:)
Anyway, as I mentioned, since I am not an expert on this technology I am not going to put it in the README.md.
Thank you!!


Is it possible to delete or archive attributes?

In IRMA I've got some old or expired attributes that are 'polluting' my vault. Like the entry to the beerfridge of a certain field lab ;-). Is it possible to delete or archive these attributes?
Seriously, for longterm users I think this is a necessary function: people change their mail adress, phone number or bank provider. Now the old attributes of the same kind stay in the vault with an 'expired date'.
Also from the context of 'regie op gegevens', every user should be able to delete attributes when not used or required anymore.

Hi Joost, credentials can be removed by long-pressing, i.e., press and hold for approximately 1 second.

This feature is not very discoverable; in a new version there is an explicit 'trash' icon.


[BEAM-2982] Use the SubscriptionProvider in PubsubUnboundedSource

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Run mvn clean verify to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

During expansion, a ValueProvider may not be accessible. This ensures
that if the subscription is based on a value provider, it will only be
evaluated when that ValueProvider is bound, rather than at construction
time.

R: @bjchambers

Coverage increased (+0.01%) to 69.61% when pulling 23e56d0dee8c70b33eb0364a1cde94125399dadf on tgroh:b_2982 into 66b864f2b3f95780f8952bafcafd8796ea931c2b on apache:master.

Coverage decreased (-0.07%) to 69.528% when pulling 368809f9dffda952a1a6cc220fc25472e5153726 on tgroh:b_2982 into 66b864f2b3f95780f8952bafcafd8796ea931c2b on apache:master.

LGTM


fix proxy run to support single and double quotes

Signed-off-by: endika<EMAIL_ADDRESS>

in the posix mode (default one) the split method deletes the single and double quotes
https://docs.python.org/3/library/shlex.html#parsing-rules


Book and Label classes and methods

Changes:

Implement Book class
Implement Label class
Implement Show all books method in App
Implement Show all labels method in App
Implement Add a book method in App

@vflorin1995 looks good 👍


Release 1.1.0

PRE-RELEASE

[x] update CHANGELOG.md
[x] bump version to 1.1.0-rc.x

package.json
CHANGELOG.md

[x] check successful CI runs
[x] create rc tag v1.1.0-rc.x

[x] successful CI run

QA

[x] changelog testing
[x] installation: npm i @gherlint/gherlint
[x] check/Test available command options:

[x] npx gherlint --help - shows all available options
[x] npx gherlint --version - check version is correct

[x] check different feature paths: npx gherlint <path>

[x] single file (features/login.feature)
[x] folder (features)
[x] glob pattern (features/**/*.feature)

[x] check --fix option fixes the fixable lint problems

[x] npx gherlint --fix <path-to-feature-file>

[x] run linter on test feature file(s) - (create feature file(s) to check all available rules)
[x] check config file can be used:

[x] .gherlintrc
[x] .gherlintrc.json
[x] .gherlintrc.js
[x] multiple config files in the project root - check for message

[x] modify between different lint severity levels (error, warn, off) - check for respective lint reports

RELEASE

[x] update CHANGELOG.md (add changelog for fixes/patches if any)
[x] bump version to 1.1.0

package.json
CHANGELOG.md

[x] create release tag v1.1.0

[x] successful CI run

[x] latest version (1.1.0) is available in npm registry (https://www.npmjs.com/package/@gherlint/gherlint)

version 1.1.0 is released

v1.1.0
npmjs


Generalize From impls for Bson::Array and Bson::Document

Currently we only implement From<Array> and From<Document>; however, we should be able to implement the more generalized FromIterator<Bson> and FromIterator<(String, Bson)> traits for these two objects.

Agree.

This is available as part of the current API; Document implements FromIterator, and Array is an alias of Vec<Bson>, and since Vec<T> already implements FromIterator<T>, FromIterator<Bson> is available for Array.


[Bugfix]Fix the problem of "," being divided in []

Purpose of this pull request

Check list

[ ] Code changed are covered with tests, or it does not need tests for reason:
[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[ ] Update the release-note.

Since the quotes [] cannot be parsed, special processing is required.

@ruanwenjun @EricJoy2048 PTAL


Add Cypress to CI chain

Context
Now that the initial Cypress minimal implementation is merged, we have the basic structure in place to automate some of the Operations Workstream's manual testing process.
We can catch regressions earlier in the workflow by including Cypress in our CI chain.
Work to be done

Investigate the best approach. It's probably between using GitHub Actions or Cypress's native Dashboard. The native Dashboard is likely a better developer and operations experience but, has an ongoing monthly cost.
Implement the best approach

Still overcoming a permissions issue, though this is partly done.
I suspect the tests will break in CI until the programmatic log-in work is complete, so finalising this issue is likely blocked by https://github.com/shapeshift/web/issues/948.

This is now ready for review: https://github.com/shapeshift/web/pull/1012


缺少文件

from ._nms import lib as _lib, ffi as _ffi
from ._crop_resize import lib as _lib, ffi as _ffi
from ._roi_align import lib as _lib, ffi as _ffi
from ._roi_crop import lib as _lib, ffi as _ffi
这些文件没有，我尝试了make.sh，结果生成了.so文件，请问这是咋回事啊。

我也报这个错了


feat: Added lazy option and fixed tr_harmonizer dependency

Added lazy option to all Genotypes Classes which when false (default) sorts output genotypes in the same order as the list of samples given (if given).
Also updated tr_harmonizer dependency so it won't error when TRTools is installed.

I just thought of something that might be potentially super important, @mlamkin7 and @gymreklab .
According to the numpy docs, Genotypes.subset() will trigger a copy of the entire genotype matrix to be made in memory because we have to use advanced indexing to reorder the columns of the genotype matrix:

Advanced indexing always returns a copy of the data
https://numpy.org/doc/stable/user/basics.indexing.html

So we should probably check that this code doesn't double the amount of memory of read(). I can look into a way to check that.

A couple ideas:

Now that I'm thinking about it, naming it lazy might be confusing since lazy is also a parameter of cyvcf2 and people might think that this is that parameter. What do you think about calling it something like reorder_samples and defaulting it to True?
Can you add type hints to be consistent? So lazy=False would become lazy: bool = False, for example
The Genotypes classes also have an __iter__() method that is supposed to work similarly to the read() method except it's supposed to read things line by line to reduce memory-use. Unfortunately, we won't be able to use subset() to reorder samples for that. What should we do in that case, do you think? I'd prefer the behavior of __iter__() to continue to align with read() as much as possible

Updated 1. and 2., but 3 does pose an issue since I believe it returns one variant at a time? we may have to sort each time.

Confirming now that running subset() together with read() does indeed double the peak memory usage :(
results
Running read() and then subset() gives us a peak memory usage of 8.8 MiB. Judging by the intensity (in color) of the flamegraph bars for the read() and subset() functions, roughly half of the 8.8 MiB is due to read() while the other half is due to subset().

Running just read() alone gives us a peak memory usage of 5.1 MiB. Most of this is due to the initial allocation of memory for the genotype matrix by read().

For more info on how to interpret the flamegraphs, take a look at the filprofiler docs
reproducing this
in case we need it for later, here are the steps to reproduce this
Using commit 56dddf46b193f38e23cf40809745873f4031bdb7, run this command
fil-profile run tests/bench_genotypes_mem.py

with this script
#!/usr/bin/env python

from filprofiler.api import profile

from haptools.data import GenotypesPLINK

gt = GenotypesPLINK("../happler/tests/data/19_45401409-46401409_1000G.pgen", chunk_size=1)
gt2 = GenotypesPLINK("../happler/tests/data/19_45401409-46401409_1000G.pgen", chunk_size=1)

def read():
    gt.read()

def read_and_subset():
    gt2.read()
    gt2.subset(samples=gt2.samples, inplace=True)

profile(read, "fil-result/bench_read_GenotypesPLINK")
profile(read_and_subset, "fil-result/bench_read_subset_GenotypesPLINK")

using this file


kick out audience or broadcaster if they are not host

hi, I want to implement live streaming and want to kick out audience or even broadcaster if they are not host, if I use socket io, I can communicate with the system but I want to know is there any event listeners which can help me in achieving this scenario.
right now I am thinking to save each user in database with some unique id and if host want to kick someone, host will trigger a api which will popup a alert or modal with navigation.goBack(), is it use case good or I can use something else, thank you

i found out these rest apis https://docs.agora.io/en/interactive-live-streaming/reference/channel-management-rest-api?platform=react-native, I will try those and check if it works in my use case.


Non-US States in "State" Information

I was surprised to find a non-US State, and what I presume is a City, in the State field for one of the Domains.

Domain name
Domain type
Agency
Organization name
City
State
Security contact email

usaseanconnect.gov
Federal - Executive
Department of State
U.S. - ASEAN Connect
Jakarta
Ja
<EMAIL_ADDRESS>

For legacy (pre-Nov 2023) domains, City and State are asserted values that are associated with the domain request and don't have anything to do with a domain's operation. This is certainly an oddity – and it also doesn't matter 😄 – so I've updated it to Washington D.C. It should update on the next push.


Lenovo Legion 5 15ARH05

There's already a profile for 15ARH05H model, and it seems to work fine so far with my 15ARH05.
https://pcsupport.lenovo.com/us/en/products/laptops-and-netbooks/legion-series/legion-5-15arh05/
Possibly could be almost 1:1 copy, the only issue I noticed is a different AMD GPU bus id, which is PCI:5:0:0 instead of PCI:6:0:0:
lspci | grep -i vga
01:00.0 VGA compatible controller: NVIDIA Corporation TU117M [GeForce GTX 1650 Ti Mobile] (rev a1)
05:00.0 VGA compatible controller: Advanced Micro Devices, Inc. [AMD/ATI] Renoir [Radeon Vega Series / Radeon Vega Mobile Series] (rev c7)

Edit: currently testing my setup at https://github.com/SoSeDiK/nixos-hardware/tree/lenovo-legion-15arh05, will open a PR later at some point.

Could this be a case similar to #1038? Definitely sounds like there is more than one version, and it is unlikely than the original creator simply missed the correct bus ID.

It is similar indeed, Lenovo seems to have three versions for this model:

Lenovo product search

My test setup for my model works so far and seems to be correct on paper, but I'm holding back due to a couple of unresolved issues with NVIDIA GPU (not properly working sleep/suspend on both open and closed driver).

I missed the H being missing in your3 title and first line. Woops.
Regarding your sleep issues, have you seen #336723 or directly this Discourse thread?

I'm not on GNOME (mostly running Hyprland), so sadly the fix doesn't apply.
Suspend works most of the time (at very rare occasions screen turns off, but laptop still works; in that case screen turns on from any key/mouse action and then suspend will work from the second try), the issues mostly arise upon waking up from the sleep as I've had the laptop freeze a bunch of times.
I also can't get neither S01x nor D3 to work despite reports on the Internet that it works for other people with the same/similar laptop/GPU models (the usual workaround of closed driver with NVreg_EnableGpuFirmware=0 didn't work).

Details
cat /proc/driver/nvidia/gpus/0000:01:00.0/power
Runtime D3 status:          Not supported
Video Memory:               Active
GPU Hardware Support:
Video Memory Self Refresh: Not Supported
Video Memory Off:          Supported
S0ix Power Management:
Platform Support:          Not Supported
Status:                    Disabled

For some more context, I'm on nixos-unstable with 6.12.5-xanmod1 kernel and stable 565.77 closed nvidia driver.

I suspect the finegrained power management (hardware.nvidia.powerManagement.finegrained) to be the cause of my issues. Disabling it and using instead NVreg_DynamicPowerManagement=0x01 (because otherwise it's disabled by default for Turing GPUs) enables RTD3:

cat /proc/driver/nvidia/gpus/0000\:01\:00.0/power
Runtime D3 status:          Enabled (coarse-grained)
Video Memory:               Active

GPU Hardware Support:
 Video Memory Self Refresh: Not Supported
 Video Memory Off:          Supported

S0ix Power Management:
 Platform Support:          Not Supported
 Status:                    Disabled

I've done a few dozen suspend & hibernate cycles without issues: laptop properly shutdowns and wakes up.
I've also switched to the open nvidia driver and had no issues either after a few cycles (same kernel and driver version as before).
Setting NVreg_DynamicPowerManagement=0x02 makes it Not supported again, not sure why. Can't get Video Memory Self Refresh and S0ix to work either, even though they supposedly are supported by the hardware?

I suspect the finegrained power management (hardware.nvidia.powerManagement.finegrained) to be the cause of my issues.

I just looked into this and realised the card does not (officially) support the option on the open modules. The finegrained option mentions the chapter 22 of the nvidia docs, which do say it supports Turing and newer, but the open kernel modules docs say that they only support RTD3 from Ampere (30 series) onwards. Hence why the closed driver should make it work, but, well, your empirical testing seems to show that no.
Have you tested manually adding the NVreg_DynamicPowerManagement=0x01 to the kernel boot options, with the closed driver? Instead of using the hardware.nvidia.powerManagement.enabled option. I'm thinking there's something going on with the default, which is 0x03 and it disables RTD3 features

Have you tested manually adding the NVreg_DynamicPowerManagement=0x01 to the kernel boot options, with the closed driver? Instead of using the hardware.nvidia.powerManagement.enabled option. I'm thinking there's something going on with the default, which is 0x03 and it disables RTD3 features

Default 0x03 disables dynamic power management by default for Turing, so the manual override is needed indeed.
Basic coarse-grained 0x01 power management works on both open and closed drivers, fine-grained 0x02 doesn't.
There are a bunch of reports like https://github.com/pop-os/nvidia-graphics-drivers/issues/216 on the Internet, which make me think that the GPU itself should support fine-grained too, but the workarounds didn't work for me.

I've dug up some threads from years ago, notably like this one: https://forums.developer.nvidia.com/t/nvidia-d3-runtime-power-management-only-works-on-window/196553/3.
According to the users with the same issue, NVIDIA blocked fine-grained RTD3 on Linux on Ryzen 4xxx and 3xxx models (I have Ryzen 5 4600H) despite the hardware support being there and it working fine on Windows. I couldn't find any official statements from NVIDIA or sources besides users just reporting the issue, but seems plausible, and sucks if that still the case after all these years.
My system advertises support for S0ix mode too, defaulting to S3:
cat /sys/power/mem_sleep
s2idle [deep]

But NVIDIA's unhappy, and booting with s2idle breaks suspend, causing system freezing.


Use a sparse bundle for /nix

This is a wild idea, and I'm not sure yet if we should go through with it, really. It came about from a short discussion in #nix-darwin. (Not sure where to find IRC logs.) Two parts:

Is creating a disk image (sparse bundle) faster than an APFS volume? -> Some test runs on GitHub Actions still seem to indicate we can't really get below ~5s with this method. (Though Mac runners have very inconsistent perf)

Then antifuchs came up with the brilliant idea: What if we commit a prepared, empty disk image? -> I did see a 3s run, so maybe? Again, perf is super inconsistent.

But, there are also downsides which make this questionable:

No idea what the perf impact is of Nix builds on a disk image, compared to an APFS volume.
Arbitrary upper bound to image size. We can probably go wild, because the 80G image in this PR is 33K compressed.

Another idea that's not in this PR: distribute an image with the Nix store and DB already present. Though, we'd still need to compress the image, so I think this essentially just moves the unarchive step around in the process. Sounds like there's little benefit.
With the super inconsistent performance of Mac runners, I'm really not sure any of this is worth the trouble. This PR is really just to have the effort documented and maybe some discussion. 🙂
For reference, the command used to create the image is:
hdiutil create -size 80GiB -fs APFS -type SPARSEBUNDLE -volname nix -nospotlight ./nix.sparsebundle

Interesting approach! If we somehow can show that this actually cuts down the (average) install time, I think it is worthwhile.
I think that distributing a complete disk image with the store and db fits slightly better with our current approach of building and publishing the nix store as part of the GitHub release. It will make the Linux and MacOS installation scripts deviate slightly more, but if we can avoid the committed disk image I think its a win. Though, the empty disk is a one time thing so I could live with that too.


Pigeon And ShrinkRay

Adds a pigeon mob with the base yml from the bat.
Screenshots

First time Making a pull . Its just a simple pigeon mob using almost the same yml as the bat.
:cl:

add: Added Pigeon Mob

New Shrinkray Texture added.


style: Fit to Pylint

Description

Fit to Pylint.

Checklist

[x] My code follows the existing style, code structure, and naming taxonomy
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have performed a self-review of my own code and included any verifying manual calculations
[x] I have added or updated unit tests that prove my fix is effective or that my feature works, and achieves sufficient code coverage.  New and existing unit tests pass locally and in the build (below) with my changes
[x] My changes generate no new warnings and running code analysis does not produce any issues
[x] I have added or run the performance tests that depict optimal execution times
[x] I have made corresponding changes to the documentation

Oh, thanks for letting me know. looks like most of them are resolved, but I'll have a look again


mooinfo 1.1.3

Create try to add mooinfo 1.1.3
Important: Do not tick a checkbox if you haven’t performed its action. Honesty is indispensable for a smooth review process.
In the following questions <cask> is the token of the cask you're submitting.
After making any changes to a cask, existing or new, verify:

[ ] The submission is for a stable version or documented exception.
[ ] brew audit --cask --online <cask> is error-free.
[ ] brew style --fix <cask> reports no offenses.

Additionally, if adding a new cask:

[ ] Named the cask according to the token reference.
[ ] Checked the cask was not already refused.
[ ] Checked the cask is submitted to the correct repo.
[ ] brew audit --cask --new <cask> worked successfully.
[ ] brew install --cask <cask> worked successfully.
[ ] brew uninstall --cask <cask> worked successfully.

is there a way to test it as local mooinfo.rb ?

[ ] brew audit --cask --online <cask> is error-free.
[ ] brew style --fix <cask> reports no offenses.

Did only create /Users/hjh/Documents/GitHub/homebrew-cask/Casks/m/mooinfo.rb
And stuck in testing it
hope i did all correct or please test it ! if possible
Grazie :)


How to use "App Configuration" task in an Azure DevOps pipeline in special cases ?

Hello,
In an Azure DevOps Pipeline, to get key-values from an App Configuration to use them in a yaml template, I use this yaml task :
    - task: AzureAppConfiguration.azure-app-configuration-task.custom-build-release-task.AzureAppConfiguration@5
      inputs:
        azureSubscription: 'ServiceConnectionName'
        AppConfigurationEndpoint: 'https://ressourcename.azconfig.io'

Questions

Why a task name so long !
Where is the documentation of this yaml tasks v5 ?
How to use a private endpoint or a connectionString with this AppConfiguration task ? How to configure it ?
Which access rights need the Azure AD account used in the ServiceConnection (for the "azureSubscription" parameter) ?
If secrets are in a Key Vault in another azure subscription,  which access rights need the Azure AD account ?

Thanks for your help ?

In task "AzureAppConfiguration@5", I need to set the AzureDevOps service connection, mapped on the resource group in an azure subcription where is the App Configuration to use.
But, if this App Configuration contain reference to secrets in Key Vaults which are on anothers resource groups and anothers Azure subcriptions, how to do ?

I see what you mean, for that case, you'll have to go to the relevant Key Vault and assign Key Vault Secret User role to the service principal. From the Key Vault menu, select  Access configuration and ensure Azure role-based access control is selected as the permission model.

This is documented in
https://learn.microsoft.com/en-us/azure/azure-app-configuration/pull-key-value-devops-pipeline#add-role-assignment

The link above does not have documentation for the YAML task.
Where can we find the YAML task documentation?
At work we have been using AzureAppConfiguration@5 for over a year.
Is there a newer version?
What has changed in the past year?

Hello @KarsonAlford yes there is a newer version of the Azure App configuration task, please see App configuration task release notes, for this task describing the improvements between versions. We don't have specific YAML documentation for the App configuration task.


Package 'python' has no installation candidate?

So, I've been trying to get this to work for a while, but that first line of code I have to paste in terminal just throws me a Package 'python' has no installation candidate. I'm not sure how to fix this, can you help me?

So, I've been trying to get this to work for a while, but that first line of code I have to paste in terminal just throws me a Package 'python' has no installation candidate. I'm not sure how to fix this, can you help me?

Is it with a Virtual Machine? cause i had this issue with the VM and then i found a fix

yea im on vmware

Python 2.x is provided by the python and reached end of life years ago. You will need python3; however, the Python code being run (and written for Python 2) may be incompatible with Python 3.


GPT-synthetic dataset

Hi, thanks for your nice work.  Could you please release the GPT-synthetic dataset?

I've also been eagerly awaiting the release of the GPT-synthetic dataset.

Hi ChocoWu and Mowenyii,
Sorry for the late reply. We have updated the datasets in "datasets" folder. Thank you.


GraphRbacManagementClient.users.update ignores extension properties

I added some extension properties using the Graph-Explorer, based on the documentation Directory schema extensions | Graph API concepts
And now, when I try to update these properties via the azure-graph (NPM-Package), then the values of my extension properties will be ignored.

Thank you for opening this issue! We are routing it to the appropriate team for follow up.

The packages azure-graph and `@azure/graph are for the older Azure AD Graph API which is being deprecated in favor of the newer Microsoft Graph API.
Based on the blog post Microsoft Graph or the Azure AD Graph, Azure AD Graph is currently in a mode where new features will not be added and only security updates will be provided.
What is being asked in this issue is a feature request that would require the service endpoints to be updated.
Therefore, we recommend you use the @microsoft/microsoft-graph-client package instead and log any issues you find with it in https://github.com/microsoftgraph/msgraph-sdk-javascript.


Proposal: MassTransit.Observability.Tests

Currently, there are no tests for metrics.
This PR is just a draft showing how MetricCollector could be used to verify tags.
MetricCollector  is very new. It was introduced in v9.0.0-preview.6.24353.1.

@NooNameR
I refactored the test.

InMemory transport didn't work for me. I replaced ActiveMq with RabbitMq.
I initialized many MetricCollectors using reflection on InstrumentationOptions.
Change the assert logic to snapshot testing. See: Metrics_Specs.Should_get_metrics_with_tags.verified.txt
I realized that MetricCollector exists in .net8. I moved the test class to MassTransit.RabbitMqTransport.Tests project.

Observations:

Sometimes the test fails because the messaging.masstransit.destination attribute is inconsistent.
messaging.masstransit.destination is missing on many metrics. https://github.com/MassTransit/MassTransit/pull/5416
Custom tags can be added only in IReceiveObserver.PreReceive. Tags added in other methods are missing. I think this is fine and could be documented as expected behavior.
Custom tags added in ISendObserver are missing. I think this should be fixed.

Custom tags can be added only in IReceiveObserver.PreReceive. Tags added in other methods are missing. I think this is fine and could be documented as expected behavior.
Custom tags added in ISendObserver are missing. I think this should be fixed.

you should be using filters for this not observers

InMemory transport didn't work for me. I replaced ActiveMq with RabbitMq.

I'll take a look into this

I have just created a test and everything works with InMemory: https://github.com/MassTransit/MassTransit/pull/5429


Italic metadata are consistent in Italic fonts (universal?)

I am working on fixing Nunito (variable) that has one source with an ital axis, and we slice it in post-process to generate 2 VFs. Some stuff related to the italic font were missing and Fontbakery didn't raise any warning.
Also this is useful cause GlyphsApp doesn't still set the italic style linking properly to link 2 VFs, and you can only see it if you look inside the font tables. These are also common practice and wouldn't kill to have it in Universal Profile.
I attach here the test case, a "good" and a "bad" with the current report:
ItalicStyleLinking.zip
All these checks should be triggered if "Italic" is part of the file name, or NameID2, or NameID17.

[x] #3663
[x] #3664
[x] #3665
[x] #3666
[x] #3667
[x] #3024
[x] #2934
[x] #3668
[x] #3669
[x] #3670

@felipesanches I finished my review but I would also like opinion from @vv-monsalve and @m4rc1e before doing the fragmented issues.

@RosaWagner, thanks for all the effort put in here, but I feel this is "a fish too big to fry". There's way too much stuff in a single issue, so I would be happy if you could indeed split it into smaller / more focused issues, as you mentioned a few messages above.
@m4rc1e and @vv-monsalve, have you already read this issue? What are your thoughts?

Done @felipesanches
I think 3 weeks are enough to wait for feedback. Let's go with no comment, no problem.
I didn't link the test case in all of them, but it is in this issue, with all the issues mentioned.

thanks a lot! :-D

I was finally able to go through all the items included. It was a good compiling @RosaWagner.
Although I have to say that having them in separated issues really made it more accessible, so I see better now Felipe's point on requesting it to be that way ;)
I've left comments on the separated issues, but the in first one #3663 is the reasoning done for all the others.

+1 for adding these checks, I just ran into the exact same situation as you describe in the original post — a font made from a DS document with an ital axis was missing some italic style linking details and FontBakery didn't raise a single issue. I think these checks would be very helpful!

@colinmford Glad to read you’re using fontbakery. Fistbump.

thanks, @yanone

Wohoo 🎉


SWAP is NaN

Because free in version 3.3.10 doesn't display -/+ buffers/cache any more, there's only two columns and line 3 (for swap) just doesn't exist.
A simple fix could be to count number of lines and act consequently…

Thank you for your bug report!
I will make a fix for that next week ... super simple - as you pointed out :-)

Ahhhhh I see, you already made a pull request for that also! Thank you!

Not yet, but it's coming ;)
Actually I patched my fork, but I don't really know how to proceed here. Fork again and fix at your point on master or what?

OK, I will do it …
As i saw, the last column (1st line) also changed the meaning, So I anyway have to make a more deeper investigation. I am thinking maybe to access also /proc/meminfo … so this will take a bit to make that work again as wanted.
I will let you know, when it is done.
Thank you once again for your report
Sebastian

Am 10.06.2016 um 13:08 schrieb Buzut<EMAIL_ADDRESS>Not yet, but it's coming ;)
Actually I patched my fork, but I don't really know how to proceed here. Fork again and fix at your point on master or what?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/sebhildebrandt/systeminformation/issues/9#issuecomment-225155359, or mute the thread https://github.com/notifications/unsubscribe/AEf9iPnntroovgjcXf_cDaPDPdfNWf1_ks5qKUWlgaJpZM4Iy0GP.

I still did not found the time to finalize/fix all  ... sorry (but I am working on it)

Finalizing V 3.0 of systeminformation. I now have a proper implementation for both versions of free. Thank you for your support on this!

mem is now also supporting changed output of free from procps-ng 3.3.10


Fix Pub/Sub Lite IO and SDF performance issues with shuffles

Imported from Jira BEAM-14116. Original Jira may contain additional context.
Reported by: dpcollins-google.

Unable to assign user @dpcollins-google . If able, self-assign, otherwise tag @damccorm so that he can assign you. Because of GitHub's spam prevention system, your activity is required to enable assignment in this repo.


Add review guidelines

As discussed with @mattfarina yesterday, I took a stab at an initial draft for chart review guidelines.
/cc @kubernetes/charts-maintainers

Nicely done @unguiculus
As we come up with more detail it can be added.


Large polygons do not triangulate

From the forum:
When I draw polygons with large distance between points, Cesium throws this error:
"Uncaught DeveloperError: At least three positions are required."
or
"Uncaught TypeError: Cannot call method 'subtract' of undefined."

My code:

    var lonLatToCartesian = function(x, y, z) {
        return ellipsoid.cartographicToCartesian(Cesium.Cartographic.fromDegrees(x, y, z));
    };

    var positions = [lonLatToCartesian(-44, -5,0), lonLatToCartesian(-100,-1,0), lonLatToCartesian(70,0,0),lonLatToCartesian(-44, -5,0)];
    var polygon = new Cesium.Polygon();
    polygon.setPositions(positions);
    primitives.add(polygon);

if I change the point (70,0,0) to (60,0,0), it works.

https://groups.google.com/forum/?fromgroups=#!topic/cesium-dev/xRqn3PQRrvQ
This is most likely because polygon triangulation happens on a tangent plane.  At the least, we should try for a better error message.

Duplicate with #2788


Add Schema changes for Readonly schema.

Fixes: #146
Developer's Certificate of Origin 1.1
   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.

@srikant-ch5 can you link the issue?

@srikant-ch5 can you link the issue?

In the PR title ? I have updated it now.


Adds fix for error checking in sampleTerrain

This PR follows up on a couple of loose ends from https://github.com/CesiumGS/cesium/pull/10631

There is now a catch clause to make sure that if there's an error thrown in sampleTerrainMostDetailed, the BoundingSphereState.FAILED is returned.
If a terrainProvider without availability is set, such as EllipsoidTerrainProvider, flyTo works for that too - it was previously broken. Use this Sandcastle to confirm this functionality.

Thanks for the pull request @sanjeetsuhag!

:heavy_check_mark: Signed CLA found.
:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.


Tests failing with current OpenWhisk

Environment details:

local deployment, vagrant, native ubuntu, Mac OS, Bluemix, ...
MacOs
version of docker, vagrant, ubuntu, ...
Docker version 18.06.1-ce, build e68fc7a

Steps to reproduce the issue:

checkout the repo
initialize the environemt ./tools/travis/setup.sh
./gradlew test

Provide the expected results and outputs:
All the test should pass
Provide the actual results and outputs:
Two tests fail
runtime.actionContainers.Python2ActionContainerTests > python2action should run zipped Python action containing a virtual environment with non-standard entry point FAILED
java.lang.NoClassDefFoundError: Could not initialize class common.WhiskProperties
runtime.actionContainers.PythonActionContainerTests > python3action should run zipped Python action containing a virtual environment with non-standard entry point FAILED
java.lang.NoClassDefFoundError: Could not initialize class common.TestUtils
at runtime.actionContainers.PythonActionContainerTests.$anonfun$new$10(PythonActionContainerTests.scala:221)
Additional information you deem important:

issue happens only occasionally or under certain circumstances
changes you did or observed in the environment

A possible solution: Create a whisk.properties file - there’s a limitation which we need to remove which requires this file to be present currently.

Where it should be and what it should contain?
--
Michele Sciabarra
<EMAIL_ADDRESS>----- Original message -----
From: rodric rabbah<EMAIL_ADDRESS>To: "apache/incubator-openwhisk-runtime-python"<EMAIL_ADDRESS>"Sciabarra.com ltd"<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Re: [apache/incubator-openwhisk-runtime-python] Tests failing with current OpenWhisk (#43)Date: Mon, 10 Dec 2018 08:12:55 -0800
A possible solution: Create a whisk.properties file - there’s a
limitation which we need to remove which requires this file to be
present currently.— You are receiving this because you authored the thread. Reply to this
email directly, view it on GitHub[1], or mute the thread[2].
Links:

https://github.com/apache/incubator-openwhisk-runtime-python/issues/43#issuecomment-445872502
https://github.com/notifications/unsubscribe-auth/AdPB74wo9FKP_EVB7fEAxdX93T_XPSqlks5u3ogHgaJpZM4ZLlOm


Finish migrating to the host_api

Migrate all remaining direct uses of the fastly_ functions to the host_api. This further reduces the number places that we're explicitly managing memory lifetimes, and improves the ergonomics of working with strings at the host call boundary.

Nice. My one residual concern on this is if we should be matching the host interface classes more closely in name to something that could be represented from the WIT. Specifically that in the overall structure here, ideally that would just be the C++ bindgen (ie, in future).

Yeah, that's a good point. I'd be happy to rename these classes in the future, especially if we're starting to look at adding c++ codegen at some point


microservices

I use workerman for microservices.
For example:
index.php:
<?php

require_once __DIR__ . '/../vendor/autoload.php';
use Workerman\Worker;

/*if (ini_get('pinba.enabled')) {
    pinba_schema_set('workerman');
    pinba_script_name_set("workerman/index.php");
}*/

$services = [
    "http://<IP_ADDRESS>:9011" => 'ms1',
    "http://<IP_ADDRESS>:9012" => 'ms2',
    "http://<IP_ADDRESS>:9013" => 'ms3',
    "http://<IP_ADDRESS>:9014" => 'ms4',
];

foreach ($services as $socket_name => $service_name) {
    $Worker = new Worker($socket_name);
    $Worker->name = $service_name;
    $Worker->count = 4;
    $Worker->onWorkerStart = function () use ($Worker, $service_name) {
        /*if (ini_get('pinba.enabled')) {
            pinba_script_name_set("workerman/$service_name.php");
        }*/
        $Worker->onMessage = include "$service_name.php";
        /*$Worker->onClose = function($connection) use ($service_name) {
            if (ini_get('pinba.enabled')) {
                pinba_flush("workerman/$service_name.php", PINBA_FLUSH_RESET_DATA);
            }
        };*/
    };
}

Worker::runAll();

ms1.php
<?php

$redis = new Redis();

$redis->pconnect('<IP_ADDRESS>');

return function($connection, $data) use ($redis)
{
    $r = 'hello world';

    $connection->close($r);
};

nginx.conf:
location ~ ^/ms1$ {
    proxy_pass http://<IP_ADDRESS>:9011;
}
location ~ ^/ms2$ {
    proxy_pass http://<IP_ADDRESS>:9012;
}

It works little slower then swoole but it's much more useful:

When I need to update my code I just use "php index.php reload" (or I need to use reload graceful?)
When I have an error in one microservice workerman restarts it while another microservices work fine.

But I have couple problems.

If I need to add one new microservice I need to do "restart" (and workerman will restart all microservices).
If I need to reload one new microservice I need to do "reload" (and workerman will reload all microservices).

How to reload one microservice?
How to start one new microservice without restart?
How to use Worker("unix:http:ms1.sock") instead of Worker("http://<IP_ADDRESS>:9011") for http proxy in nginx like this:
location ~ ^/ms1$ {
    proxy_pass unix:/var/www/workerman/ms1.sock;
}

PS: Thanks a lot for your library.

These are very good questions.
I wrote an example that I hope will help you.
index.php
<?php

require_once __DIR__ . '/../vendor/autoload.php';
use Workerman\Worker;

$services = include "config.php";

// Create new service
function create_worker($service_name, $info)
{
    $Worker = new Worker($info['listen']);
    $Worker->count = $info['count'];
    $Worker->user = $info['user'];
    $Worker->name = $service_name;
    $Worker->onWorkerStart = function () use ($Worker, $service_name) {
        $Worker->onMessage = include "$service_name.php";
    };
    return $Worker;
}

foreach ($services as $service_name => $info) {
    create_worker($service_name, $info);
}

// onMasterReload will called in master scope when run `php index.php reload` command.
Worker::$onMasterReload = function () {
    $all_workers = [];
    foreach (Worker::getAllWorkers() as $worker) {
        $all_workers[$worker->name] = $worker;
    }
    $services = include "config.php";
    // Start new worker.
    foreach ($services as $service_name => $info) {
        if (!isset($all_workers[$service_name])) {
            $worker = create_worker($service_name, $info);
            if (!$worker->reusePort) {
                $worker->listen();
            }
        }
    }
    // Change old worker config. Here is worker->count.
    foreach ($all_workers as $service_name => $worker) {
        if (isset($services[$service_name])) {
            $worker->count = $services[$worker->name]['count'];
        }
    }
};

// Parse command "php index.php reload {$service_name}"
if ($argv[1] == 'reload' && isset($argv[2])) {
    $reload_service_name = $argv[2];
    echo "reload $reload_service_name\n";
   // Send signal SIGUSR1 to the worker processes.
    exec("php index.php status | grep ' $reload_service_name' | awk '{print $1}' | xargs kill -SIGUSR1");
    exit("reload complete\n");
}

Worker::runAll();

config.php
<?php
return [
    "ms1" => [
        // As you see, you can use unix socket like this.
        'listen' => 'unix:///var/www/workerman/ms1.sock',
        'count'  => 1,
        'user'   => 'www-data',
    ],
    "ms2" => [
        'listen' => 'http://<IP_ADDRESS>:9012',
        'count'  => 2,
        'user'   => 'www-data',
    ],
    "ms3" => [
        'listen' => 'http://<IP_ADDRESS>:9013',
        'count'  => 3,
        'user'   => 'www-data',
    ],
    "ms4" => [
        'listen' => 'http://<IP_ADDRESS>:9014',
        'count'  => 4,
        'user'   => 'www-data',
    ]
];

How to reload one microservice?

Use command php index.php reload $service_name for example php index.php reload ms1.

How to start one new microservice without restart?

Add new service config in $config.php then run command php index.php reload

How to use Worker("unix:http:ms1.sock") instead of Worker("http://<IP_ADDRESS>:9011") for http proxy in nginx like this:

The codes like this.
new Worker('unix:///your/path');

Thanks a lot.
The reloading of one service works perfectly and creating of new service too.
But I have two problems with unix socket:

after start workerman creates socket file /var/www/workerman/ms1.sock and after reload/restart it already exists and I have error:

stream_socket_server(): unable to connect to unix:///var/www/workerman/vpnservers.sock (Unknown error) in file /var/www/workerman/vendor/workerman/workerman/Worker.php on line 2224

in line:

$this->_mainSocket = \stream_socket_server($local_socket, $errno, $errmsg, $flags, $this->_context);

It works only if I delete file /var/www/workerman/vpnservers.sock

'new Worker('unix:///your/path');' uses tcp transport and doesn't understand http protocol so nginx with option 'proxy_pass http://unix:/var/www/workerman/vpnservers.sock;' couldn't get anything from workerman.

I think it is not easy to add support restart/reload and http protocol for unix sockets so I will use ports instead of unix sockets.

1、To fix unable to connect to unix.. we have modify function create_worker like this.
function create_worker($service_name, $info)
{
    $Worker = new Worker($info['listen']);
    $Worker->count = $info['count'];
    $Worker->user = $info['user'];
    $Worker->name = $service_name;
    $Worker->onWorkerStart = function () use ($Worker, $service_name) {
        $Worker->onMessage = include "$service_name.php";
    };
   // === Here ===
    if ($Worker->transport == 'unix') {
        $Worker->reusePort = false;
    }
    return $Worker;
}

2、If you want to support http protocol with unix socket you can try to set the protocol like this.
$worker = new Worker('unix:///your/path');
// == Here ==
$worker->protocol = '\Workerman\Protocols\Http';

Thanks a lot.

@walkor thank you so much. how to stop one of child processes completely in this example?


[PAD] Ability to customize error messages

Current PAD users are familiar with Robin error messages
After converting a Robin script to Fx, customers could get different errors messages
The goal here is to override some of the Fx messages with the same Robin equivalents known by customers

We should strive for high quality error messages in Fx. If there are specific errors cases that can be improved, we should look at those.
Power Fx is the same across the platform, so the goal here is consistency across the platform (eg, with Power Apps, Dataverse, etc).


Added some tests and change API request method

Bug, "Get" methods in Slack are actually POSTs, since Slack's API is a JSON RPC

:rocket: PR was released in<EMAIL_ADDRESS>:rocket:


factor reordering when adding a plot label to a facet_wrap plot

Describe the bug
When you add a on plot label to a facetted chart the factor ordering is reset. In most cases this doesn't have a noticeable effect, but when you have deliberately ordered a factor in a particular way it is quite annoying.
Code to reproduce
Please supply a reproducible example -- the minimum amount of code which recreates the problem you experienced.
Please remove any unnecessary packages or code that are not related to the problem you are experiencing.
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Duplicate of #217


Markers with label not showing in google chrome after deploying on IIS production

My application is using markerwithlabel library.It works well while in development environment on google chrome.But after deploying the application on IIS production , the markers are not visible in google chrome while they are visible in IE and Firefox.Please suggest a solution as most of the users are having google chrome installed on their machine.

Please provide more details.
Error details, screenshots, links if the apps are public or not, and things like that.
Otherwise is nearly impossible to know what's the issue and how to help you to solve it.


added spec and fix for bug when setting value to null after mount

Attempts to fix bug outlined in https://github.com/monterail/vue-multiselect/issues/190#issuecomment-282792542
Setting the selected :value to null after the control had been mounted was causing the watcher to generate an internal value of [null] and then map that array using the 'trackBy' property.
Have tried to keep both code and spec style consistent with rest of project.

Should already be resolved.


fix: pagination issues second try

This is just the revert of the revert.

Is it possible to deploy the docs for this branch?

@jguddas please add a changeset


Build breaks with React Native starting from 2.9.0

Bug report

[X] I confirm this is a bug with Supabase, not with my own application.
[X] I confirm I have searched the Docs, GitHub Discussions, and Discord.

Describe the bug
After "move from websocket to ws and support native browser WebSocket API" (#263) cannot build anymore in React Native (Webpack).
In particular ws module cannot find some libraries url,stream, etc. Adding Polyfills causes max call stack errors.
To Reproduce
Steps to reproduce the behavior, please provide code snippets or a repository:

Init new project
Try to compile

Expected behavior
Replacement of websocket to ws must not be a breaking change.
System information

OS: [ Windows]
Version of supabase-js: [latest]
Version of Node.js: [19.8.1]

Additional context
Same codebase, same build steps works as usual using @supabase/supabase-js 2.36.0 - realtime 2.8.4

@piccinnigius which version of realtime-js are you on when using supabase-js? this should be fixed in realtime-js v2.9.1. try upgrading supabase-js in your react native app.

@w3b6x9 thanks for replying.
I'm using latest supabasejs that comes with realtime v2.9.1 :/

@piccinnigius which react native version are you on?

@w3b6x9 I've been on 0.71.14 and to check if on newer versions works I've upgraded to 0.73.1

@piccinnigius that's odd b/c it's working fine for me. here are my setup steps:

npx react-native@latest init TestProject (v0.73.1)
npm install @supabase/supabase-js (v2.39.2) - confirmed that it's on realtime-js v0.29.1
npm install react-native-url-polyfill and imported at the top of index.js - looks like this is a necessity b/c supabase-js is using URL
Set up Supabase client, created and subscribed to channel

did you have the same steps? can you share exactly your setup and the output you're seeing?

@w3b6x9 i can confirm that usually works.
I think that part of the problem is related to using webpack on React Native.
In second hand we can refeer to ws readme that says:

Note: This module does not work in the browser. The client in the docs is a reference to a back end with the role of a client in the WebSocket communication. Browser clients must use the native WebSocket object. To make the same code work seamlessly on Node.js and the browser, you can use one of the many wrappers available on npm, like isomorphic-ws.

The real question is: why if in React Native Websocket is available and NATIVE_WEBSOCKET_AVAILABLE  is true with this section messing up resulting in using ws package?
Anyway this is the output:

Details
Cannot apply update due to error: {"loc": "1:287-302", "message": "Module not found: Error: Can't resolve 'zlib' in 'D:\testPrj\packages\native-shared\node_modules\@supabase\realtime-js\node_modules\ws\lib'
BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.
If you want to include a polyfill, you need to:
- add a fallback 'resolve.fallback: { "zlib": require.resolve("browserify-zlib") }'
- install 'browserify-zlib'
If you don't want to include a polyfill, you can use an empty module like this:
resolve.fallback: { "zlib": false }", "moduleIdentifier"<EMAIL_ADDRESS>"moduleName"<EMAIL_ADDRESS>x [16:37:08.169Z][Console] Cannot apply update due to error: {"loc": "1:1311-1328", "message": "Module not found: Error: Can't resolve 'stream' in 'D:\testPrj\packages\native-shared\node_modules\@supabase\realtime-js\node_modules\ws\lib'
Did you mean './stream'?
Requests that should resolve in the current directory need to start with './'.
Requests that start with a name are treated as module requests and resolve within module directories (node_modules).
If changing the source code is not an option there is also a resolve options called 'preferRelative' which tries to resolve these kind of requests in the current directory too.
BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.
If you want to include a polyfill, you need to:
- add a fallback 'resolve.fallback: { "stream": require.resolve("stream-browserify") }'
- install 'stream-browserify'
If you don't want to include a polyfill, you can use an empty module like this:
resolve.fallback: { "stream": false }", "moduleIdentifier"<EMAIL_ADDRESS>"moduleName"<EMAIL_ADDRESS>x [16:37:08.175Z][Console] Cannot apply update due to error: {"loc": "1:459-476", "message": "Module not found: Error: Can't resolve 'stream' in 'D:\testPrj\packages\native-shared\node_modules\@supabase\realtime-js\node_modules\ws\lib'
Did you mean './stream'?
Requests that should resolve in the current directory need to start with './'.
Requests that start with a name are treated as module requests and resolve within module directories (node_modules).
If changing the source code is not an option there is also a resolve options called 'preferRelative' which tries to resolve these kind of requests in the current directory too.
BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.
If you want to include a polyfill, you need to:
- add a fallback 'resolve.fallback: { "stream": require.resolve("stream-browserify") }'
- install 'stream-browserify'
If you don't want to include a polyfill, you can use an empty module like this:
resolve.fallback: { "stream": false }", "moduleIdentifier"<EMAIL_ADDRESS>"moduleName"<EMAIL_ADDRESS>x [16:37:08.187Z][Console] Cannot apply update due to error: {"loc": "1:514-531", "message": "Module not found: Error: Can't resolve 'crypto' in 'D:\testPrj\packages\native-shared\node_modules\@supabase\realtime-js\node_modules\ws\lib'
BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.
If you want to include a polyfill, you need to:
- add a fallback 'resolve.fallback: { "crypto": require.resolve("crypto-browserify") }'
- install 'crypto-browserify'
If you don't want to include a polyfill, you can use an empty module like this:
resolve.fallback: { "crypto": false }", "moduleIdentifier"<EMAIL_ADDRESS>"moduleName"<EMAIL_ADDRESS>x [16:37:08.187Z][Console] Cannot apply update due to error: {"loc": "1:26-43", "message": "Module not found: Error: Can't resolve 'stream' in 'D:\testPrj\packages\native-shared\node_modules\@supabase\realtime-js\node_modules\ws\lib'
Did you mean './stream'?
Requests that should resolve in the current directory need to start with './'.
Requests that start with a name are treated as module requests and resolve within module directories (node_modules).
If changing the source code is not an option there is also a resolve options called 'preferRelative' which tries to resolve these kind of requests in the current directory too.
BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.
If you want to include a polyfill, you need to:
- add a fallback 'resolve.fallback: { "stream": require.resolve("stream-browserify") }'
- install 'stream-browserify'
If you don't want to include a polyfill, you can use an empty module like this:
resolve.fallback: { "stream": false }", "moduleIdentifier"<EMAIL_ADDRESS>"moduleName"<EMAIL_ADDRESS>x [16:37:08.188Z][Console] Cannot apply update due to error: {"loc": "1:1619-1634", "message": "Module not found: Error: Can't resolve 'http' in 'D:\testPrj\packages\native-shared\node_modules\@supabase\realtime-js\node_modules\ws\lib'
BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.
If you want to include a polyfill, you need to:
- add a fallback 'resolve.fallback: { "http": require.resolve("stream-http") }'
- install 'stream-http'
If you don't want to include a polyfill, you can use an empty module like this:
resolve.fallback: { "http": false }", "moduleIdentifier"<EMAIL_ADDRESS>"moduleName"<EMAIL_ADDRESS>x [16:37:08.189Z][Console] Cannot apply update due to error: {"loc": "1:1648-1665", "message": "Module not found: Error: Can't resolve 'stream' in 'D:\testPrj\packages\native-shared\node_modules\@supabase\realtime-js\node_modules\ws\lib'
Did you mean './stream'?
Requests that should resolve in the current directory need to start with './'.
Requests that start with a name are treated as module requests and resolve within module directories (node_modules).
If changing the source code is not an option there is also a resolve options called 'preferRelative' which tries to resolve these kind of requests in the current directory too.
BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.
If you want to include a polyfill, you need to:
- add a fallback 'resolve.fallback: { "stream": require.resolve("stream-browserify") }'
- install 'stream-browserify'
If you don't want to include a polyfill, you can use an empty module like this:
resolve.fallback: { "stream": false }", "moduleIdentifier"<EMAIL_ADDRESS>"moduleName"<EMAIL_ADDRESS>x [16:37:08.191Z][Console] Cannot apply update due to error: {"loc": "1:1703-1720", "message": "Module not found: Error: Can't resolve 'crypto' in 'D:\testPrj\packages\native-shared\node_modules\@supabase\realtime-js\node_modules\ws\lib'
BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.
If you want to include a polyfill, you need to:
- add a fallback 'resolve.fallback: { "crypto": require.resolve("crypto-browserify") }'
- install 'crypto-browserify'
If you don't want to include a polyfill, you can use an empty module like this:
resolve.fallback: { "crypto": false }", "moduleIdentifier"<EMAIL_ADDRESS>"moduleName"<EMAIL_ADDRESS>x [16:37:08.194Z][Console] Cannot apply update due to error: {"loc": "1:1721-1737", "message": "Module not found: Error: Can't resolve 'https' in 'D:\testPrj\packages\native-shared\node_modules\@supabase\realtime-js\node_modules\ws\lib'
BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.
If you want to include a polyfill, you need to:
- add a fallback 'resolve.fallback: { "https": require.resolve("https-browserify") }'
- install 'https-browserify'
If you don't want to include a polyfill, you can use an empty module like this:
resolve.fallback: { "https": false }", "moduleIdentifier"<EMAIL_ADDRESS>"moduleName"<EMAIL_ADDRESS>x [16:37:08.200Z][Console] Cannot apply update due to error: {"loc": "1:1747-1762", "message": "Module not found: Error: Can't resolve 'http' in 'D:\testPrj\packages\native-shared\node_modules\@supabase\realtime-js\node_modules\ws\lib'
BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.
If you want to include a polyfill, you need to:
- add a fallback 'resolve.fallback: { "http": require.resolve("stream-http") }'
- install 'stream-http'
If you don't want to include a polyfill, you can use an empty module like this:
resolve.fallback: { "http": false }", "moduleIdentifier"<EMAIL_ADDRESS>"moduleName"<EMAIL_ADDRESS>x [16:37:08.201Z][Console] Cannot apply update due to error: {"loc": "1:1771-1785", "message": "Module not found: Error: Can't resolve 'net' in 'D:\testPrj\packages\native-shared\node_modules\@supabase\realtime-js\no
de_modules\ws\lib'", "moduleIdentifier"<EMAIL_ADDRESS>"moduleName"<EMAIL_ADDRESS>x [16:37:08.201Z][Console] Cannot apply update due to error: {"loc": "1:1794-1808", "message": "Module not found: Error: Can't resolve 'tls' in 'D:\testPrj\packages\native-shared\node_modules\@supabase\realtime-js\no
de_modules\ws\lib'", "moduleIdentifier"<EMAIL_ADDRESS>"moduleName"<EMAIL_ADDRESS>x [16:37:08.204Z][Console] Cannot apply update due to error: {"loc": "1:1822-1839", "message": "Module not found: Error: Can't resolve 'crypto' in 'D:\testPrj\packages\native-shared\node_modules\@supabase\realtime-js\node_modules\ws\lib'
BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.
If you want to include a polyfill, you need to:
- add a fallback 'resolve.fallback: { "crypto": require.resolve("crypto-browserify") }'
- install 'crypto-browserify'
If you don't want to include a polyfill, you can use an empty module like this:
resolve.fallback: { "crypto": false }", "moduleIdentifier"<EMAIL_ADDRESS>"moduleName"<EMAIL_ADDRESS>x [16:37:08.252Z][Console] Cannot apply update due to error: {"loc": "1:1918-1935", "message": "Module not found: Error: Can't resolve 'stream' in 'D:\testPrj\packages\native-shared\node_modules\@supabase\realtime-js\node_modules\ws\lib'
Did you mean './stream'?
Requests that should resolve in the current directory need to start with './'.
Requests that start with a name are treated as module requests and resolve within module directories (node_modules).
If changing the source code is not an option there is also a resolve options called 'preferRelative' which tries to resolve these kind of requests in the current directory too.
BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.
If you want to include a polyfill, you need to:
- add a fallback 'resolve.fallback: { "stream": require.resolve("stream-browserify") }'
- install 'stream-browserify'
If you don't want to include a polyfill, you can use an empty module like this:
resolve.fallback: { "stream": false }", "moduleIdentifier"<EMAIL_ADDRESS>"moduleName"<EMAIL_ADDRESS>x [16:37:08.252Z][Console] Cannot apply update due to error: {"loc": "1:2002-2016", "message": "Module not found: Error: Can't resolve 'url' in 'D:\testPrj\packages\native-shared\node_modules\@supabase\realtime-js\node_modules\ws\lib'
BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.
If you want to include a polyfill, you need to:
- add a fallback 'resolve.fallback: { "url": require.resolve("url/") }'
- install 'url'
If you don't want to include a polyfill, you can use an empty module like this:
resolve.fallback: { "url": false }", "moduleIdentifier"<EMAIL_ADDRESS>"moduleName"<EMAIL_ADDRESS>
Thanks in advance

@piccinnigius have you tried polyfill-ing all the modules listed in the error message?

@w3b6x9 yes, this results in maximum call stack exceeded error

@w3b6x9 I've fixed by marking ws as external in webpack configuration. Idk if something needs to be done on realtime-js side.
Thanks for your help.

@piccinnigius nice, glad that marking it external got it working.

The real question is: why if in React Native Websocket is available and NATIVE_WEBSOCKET_AVAILABLE is true this section messing up resulting in using ws package?

i'm still wondering this that you pointed out. i'll look into this and see if i can come up with anything.


.setLimit() has no effect on Scan?

Hi,
I've tried to use the .setLimit() on a Scan, but it still returns with all rows in the table after the startrow.
Below the code snippet, which produces this error.
public static void main(String[] args) throws IOException  {
    Connection c = BigtableConfiguration.connect(projectId, instanceId);
    Table t = c.getTable(TableName.valueOf(tableName));

    Scan s = new Scan().withStartRow(Bytes.toBytes(startRow)).setLimit(5);
    ResultScanner resultScanner = t.getScanner(s);

    for (Result r : resultScanner) {
        System.out.print(Bytes.toString(r.getRow()) + "\t");
        System.out.println(Bytes.toString(r.value()));
    }
}

Gradle dependecy:
dependencies {
    implementation "com.google.cloud.bigtable:bigtable-hbase-beam:1.5.0"
}

I tried with different versions of Bigtable dependencies but the result was the same.
Do I misuse something?
Thanks!

We have not kept up with all of the features in newer HBase versions.  setLimit seems to have been added in HBase 1.4.
For now, you can use a PageFilter to limit the number of records.  We'll find a way to use setLimit in Cloud Bigtable.

Thank you for your answer! We will go with PageFilter in this case.


Should this be "return"?

I think you meant return here, or else it should be lowercase r.

Yep, thanks


fix(userspace): safely check string bounded access

What type of PR is this?
/kind bug
Any specific area of the project related to this PR?
/area engine
What this PR does / why we need it:
This prevents out-of-bound string access from happening.
Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
NONE

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

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce]

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

LGTM label has been added.  Git tree hash: bc98e5d7d9639e4ee22f733bdd5e2ae4440bbd0a

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

Needs approval from an approver in each of these files:

OWNERS [FedeDP,jasondellaluce]

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

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

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP,jasondellaluce]

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


x/image/font: no method to measure exact string drawing bounds

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go1.7.4 darwin/amd64
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/raphaelgeronimi/Local/Others/Go workspace"
GORACE=""
GOROOT="/opt/local/lib/go"
GOTOOLDIR="/opt/local/lib/go/pkg/tool/darwin_amd64"
CC="/usr/bin/clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/zw/0zvr11l11975vn4gk7s4hrx00000gn/T/go-build732769753=/tmp/go-build -gno-record-gcc-switches -fno-common"
CXX="clang++"
CGO_ENABLED="1"
What did you do?
I need to measure the exact drawing bounds of a string with a Face from x/image/font, to draw it in a bitmap of the proper size, and position the origin of the string properly so as to not clip any extraneous left, top, bottom or right typographic "limb".
What did you expect to see?
A function or method to measure the exact bounds, as:

It is a very common need
Measuring them is not trivial for a library end-user as it requires some typographic knowledge (e.g., kerning), and the amount of duplicated code goes beyond the usual loop 3-liners (with standard formatting, this takes 40 lines).

What did you see instead?
No method found.
I wrote a patch with the missing functions and methods BoundString and BoundBytes, that I am ready to contribute.

Possible dup: https://github.com/golang/go/issues/14436

cc @nigeltao


Fix false positive rewriteRelativeImportExtensions error on non-TS extensions

Fixes #60392
Fixes #60396
This check was unnecessary for the actual rewriting, but somehow forgot to include it for error checking.

Should this be patched in for 5.7 stable?

Yes—I thought we hadn’t split off yet?

@typescript-bot pick to release-5.7

@typescript-bot cherry-pick to release-5.7


POSIX: Update TryGetGVFSHooksPathAndVersion to retrieve the version of GVFS.Hooks rather than the current process

Code that verifies that the proper version of the hooks is on the path does not work on Linux and Mac.
https://github.com/microsoft/VFSForGit/blob/69c3b45a253a710489222f954780c0a0c38d3bdc/GVFS/GVFS.Platform.POSIX/POSIXPlatform.cs#L50

Closing as duplicate of #1044


⚠ Bofrost.at has degraded performance

In e839b56, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 897 ms

Resolved: Bofrost.at performance has improved in 6192102 after 6 minutes.


Cannot import formatToOpenAIToolMessages

Seems like langchain/agents/format_scratchpad/openai_tools is not exported because i receive the following error when i try to import the function formatToOpenAIToolMessages:
Error [ERR_PACKAGE_PATH_NOT_EXPORTED]: Package subpath './dist/agents/openai/output_parser' is not defined by "exports" in ....

The problem wast the dist in the url path


Service failure page

This service failure page will be modified once the actual design page is available
Also, added comments on SEND_EMAIL flag

Closing this as the changes are now under https://github.com/co-cddo/domains-register-a-govuk-domain/pull/117


Timesheet - 05/08/2021 - Ian Schoonover

Task 1
Created / Updated Core Team Payment Calculation Sheets for July 26 2021 - August 4th 2021
Proof of Work

https://docs.google.com/spreadsheets/d/10iKkBZkOR2aEAULZXS84MuiGcP8dE3TuHTxeXXNZVN0/edit#gid=0
https://docs.google.com/spreadsheets/d/1eyiCrpGmdYR-cI3iH3aToSlXtB_OkgrwNoAcYnqGTAA/edit#gid=0
https://docs.google.com/spreadsheets/d/1iXnpiDHtGQi5DmnCmpvZCKSBTSXDZOQZ_NAPR3UXFGc/edit#gid=0
https://docs.google.com/spreadsheets/d/1FSW16Q1CYQxnci0CnKw17gwV_q0KY_kbXbjbWbu1PrA/edit#gid=0
https://docs.google.com/spreadsheets/d/1vJXUoi_fHslz8gwsrJprIBpjnC6e_mThubvqMur8B4s/edit#gid=0
https://docs.google.com/spreadsheets/d/1CS3NV00YEwZAzTW88p-a6UICKV0O1bmm5yiGM_nI4kA/edit#gid=0
https://docs.google.com/spreadsheets/d/14JyEXeFjmq2Sh-NDEwZLlpPvGmir-FwwfMtl9nuNE-I/edit#gid=0
https://docs.google.com/spreadsheets/d/1bCsiHVXZjQsvs5uwA_ldaEeLFTdjOhyi3JlntzEOpCM/edit#gid=0

Time Spent
2 hours

Total Time Spent
2 hours
Account Number
4b6b2d11f5cd03baa9cbd563e76d1209d4491b2b88186ab97b73460340c2a6ed

Hey @nax3t, when you timesheet is ready, can you add the "Ready for Review" label?

@tspearing Done! :) Thank you.


Updated - WebApp Does not login but Teams Work

Hello, I have successfully deployed all the setup and configuration; however, I can open the web app via URL < it just goes round and round in a circle and does not open anything with a message saying Please Wait.
I have managed to upload the app to Teams, where it opens, but When I click on Login, it opens the login window. I try to log in, but nothing happens,  And I go back to the Login Window.
https://github.com/OfficeDev/microsoft-teams-emergency-operations-center/assets/156114106/ad7d8802-55b2-4203-b9d6-30b36ba513c4
I  have tried to open this in the Web Version of Teams, but it is the same.
It appears it is unable to authenticate. I have restarted the Webapp.
What can I do to log in and troubleshoot?
Thank you

Hello, Sorry to bother you. I found the mistake in API> I used the instructions from the Browser console and found the API name was wrong, and as soon as I updated. it started to work in Teams. However, I can not make the web app work. It is still stuck at please wait and keep on going in a circle. thank you

@sanjaymittaluk, Thank you for reporting this.
Can you please open TEOC app in web version of teams and share the console logs and network trace with us for troubleshoot the issue?

URL IS -  azurewebsites.net/#/tab

@sanjaymittaluk, Can you please confirm is you are able to create and update incident successfully in web version of teams.
And share network trace with us for further troubleshooting.
Meanwhile we will try to troubleshoot this issue from our end

Yes, No problem with anything else so far. I did, however, have an issue where I could not open or edit an incident, but then I removed it from the List and created it again, and it worked.

Yes, No problem with anything else so far. I did, however, have an issue where I could not open or edit an incident, but then I removed it from the List and created it again, and it worked.

@sanjaymittaluk, Thank you for your confirmation.
azurewebsites.net/#/tab URL will always show loader that is how it is designed.
Can you please try creating one new incident now and confirm if it is working fine for you?
In this case if everything is working fine for you we will close this issue.

v-asshrirao


An error has occurred, please try to create your incident again

@sanjaymittaluk , Can you please share console logs and network trace with us for troubleshooting.

@sanjaymittaluk, For troubleshooting we will need logs for below scenarios.

When you try to create incident and its failing.
When incident created successfully after few attempts but you are not able to open it or edit it.

Can you please help us with logs and network trace for both scenarios.

@sanjaymittaluk, For troubleshooting we will need logs for below scenarios.

When you try to create incident and its failing.
When incident created successfully after few attempts but you are not able to open it or edit it.

Can you please help us with logs and network trace for both scenarios.

I am not sure what has happend but we can not create incident anymore and this is completely broken

@sanjaymittaluk, Can you please try to create incident and copy the logs from console in text file and share with us along with network trace. Without analysing the logs we will not be able to find the root cause and help you on this issue further.

@sanjaymittaluk, Can you please try to create incident and copy the logs from console in text file and share with us along with network trace. Without analysing the logs we will not be able to find the root cause and help you on this issue further.
Request you to kindly attach logs and network trace in GitHub ticket itself because we will not be able to open logs if you share over an email.

teams.microsoft.com-1706115136500.txt

@sanjaymittaluk, Thank you for sharing the logs. We will analyse this and try to get back to you ASAP.

@sanjaymittaluk, Thank you for sharing the logs. We will analyse this and try to get back to you ASAP.

do you think , it could be because one of the app service is on free with 60 minutes limit ?

Hello, do you know if we need to do anything about an issue which is not allowing us to create an incident? I saw an update that a patch has been issued. Do you know if it's automatically applied, or do we need to update it somewhere?

Hi @sanjaymittaluk,
May I know which hot fix are you referring to. The hot fix we have been waiting is not deployed yet. We will keep you posted once its deployed.
Thanks.

the one where

Hi @sanjaymittaluk,
May I know which hot fix are you referring to. The hot fix we have been waiting is not deployed yet. We will keep you posted once its deployed. Thanks.

it is the one where we could not create a new incident

Hi @sanjaymittaluk,
Yes, the Create Incident failure is happening due to an issue with a graph call and i am referring to the hot fix for the same. This is not deployed yet. We will let you know when it's ready. Thanks for your patience on this.
Thanks.

Hi @sanjaymittaluk,
Yes, the Create Incident failure is happening due to an issue with a graph call and i am referring to the hot fix for the same. This is not deployed yet. We will let you know when it's ready. Thanks for your patience on this. Thanks.

Thank you, would it be automatic or does it require any action from us

@sanjaymittaluk,
Thank you for your patience on this.
We have released the hotfix for this issue. Request you to kindly sync the code by following steps mentioned in this link
If you are still facing the issue we suggest you to clear cache and try again.

@sanjaymittaluk, Thank you for your patience on this. We have released the hotfix for this issue. Request you to kindly sync the code by following steps mentioned in this link
If you are still facing the issue we suggest you to clear cache and try again.

You guys are Gods.. All is good now, and I can create a new Incident.  So that you know, All I had to do was Click on Sync and wait. Permissions were already there.

@sanjaymittaluk,Glad to hear that it worked. We will go ahead and close this ticket now.


Function.prototype.apply should accept 2 arguments only

Just had a bug in production where someone accidentally used Function#apply instead of Function#call. Roughly like so:
function getUrl() { return 'some string' }
axios[method].apply(axios, getUrl(), someArg, someOtherArg);

I looked into why Flow hasn't caught this. getUrl() returning a string means that it conforms to an Iterable expected of a 2nd argument. But if apply didn't accept more than 2 arguments, this would have been caught.
To reproduce:
(function () { }).apply({}, [], 1) // no error

Flow version: 0.100
Expected behavior
It should error with "expected 2 arguments" or some such.
Actual behavior
No error is reported.

Link to Try-Flow or Github repo: https://flow.org/try/#0GYVwdgxgLglg9mABFApgZygCgJQG8C+AUKhgHQCGADpQDYCemARIwDSKNhxSLkBOv5Oo2wBuIA

TS gets this right: https://www.typescriptlang.org/play/#src=(function () { }).apply(%7B%7D%2C%20%5B%5D%2C%201)
Surprisingly, according to this test, https://github.com/samwgoldman/flow/blob/ef5d4a4b896ad9cb1fb49227e253c7135b349255/tests/function/apply.js apply(..., "some string") should be an error, but it seems that's no longer the case (since version 0.39)?

I believe this is related to spread?


Release PEG.js 0.12.0

The 0.11.0 release is near! Here are the remaining steps:

[x] Upgrade all used libraries to their newest versions
[ ] Drop support for

[ ] Node 6.x
[ ] Node 8.x
Internet Explorer

[ ] Test Node.js support

[ ] 10.x
[ ] 12.x
[ ] 14.x (Upcoming in 04/2020)

[ ] Test browser support (latest versions only)

[ ] Chrome
[x] Firefox
[ ] Opera
[ ] Safari
[ ] Edge

[ ] Misc (Optional)

[ ] Update issue and pr templates for GitHub (example)
[ ] Update the wiki
[ ] Drop forum support; Encourge discussions in GitHun issues instead
[ ] More tests
[ ] Increase covereage
[ ] @pegjs/standalone

[ ] Release!

[ ] Update CHANGELOG.md
[ ] Add note about the ill-fated 0.11.0 and it's eventual rewrite 😅
[ ] Update version to 0.12.0
[ ] Publish to NPM
[ ] Create a release on GitHub
[ ] Update website (#578)
[ ] Tweet

@dmajda @Mingun
I've updated the CHANGELOG.md file, and before release I'm thinking of adding slightly more information for the major changes.
For now, I'm doing other stuff (including an overhaul of the website), but while I'm doing that I'de be grateful if you guys looked over the list and point me to anything I should change or you think I missed.

Hi @futagoza! I don’t follow PEG.js development anymore, so I can’t really give much comments beside that the changelog looks really impressive and that I’m glad that you managed to get to the release. Keep up the good work!

As though nothing is forgotten. I left several comments under commit 87dcc13d0f87843d8eb5980f8fa1f493d13a7db0, generally with wishes to add references to specific issues

@Mingun just pushed 4723176 with what you noted

Anything us folks can do to help finalize this release?

Any news on this?

@futagoza anything in particular you'd like help on? Seems like a few folks here might be willing and I see all the commit activity.

Is there any update on when v0.11.0 may be released?

@futagoza need help releasing the version?

@lindner @srenault @tilgovi @GreenImp @cr-roypeled Sorry for the late reply 🙇
I'm kind of overloaded with my work right now but have been (on my time off) working on a rewrite of the PEG.js library written in TypeScript. It originally started off as a way to fix a bug in the new plugin system for 0.11 (the intention was to find the root of the problem and brainstorm a fix using TypeScripts type system) before finally releasing it, but sprawled out of control into a full-blown rewrite  😅 that is also changing the current AST to enable a better development flow in general.

please don't block peg
if you want to rewrite, fine, but it's been two years, and common sense things aren't getting released that are ready to go
we need you to release es module support.  it's in 11 right now
major breaking changes just mean everyone who uses peg has to leave.  that's not the right way to handle typescript.  handling typescript in the existing codebase is a couple simple string transformations.
it's been two years
please let us go back to a simple regular release schedule of minor improvements
if you try to wait forever and then release something unrelated, half of us will leave because the tool rotted on the vine, and the others who waited won't be able to use the work
release 0.11 and don't change the ast

or at least turn it over to a maintainer who's willing to actually maintain the library, instead of turning it into a hobby project to replace it
this is a serious tool in use by thousands of other projects

i want to add typescript support
i went to add esmodule support today only to realize that what's in github and what's on npm aren't the same, by a pretty wide margin, and that es modules are actually in there, but the developer doesn't consider them releasable
can you please give us instructions on how to add features today?  there are things in 11 that i would backport to 10, so that everyone can have them.
i want to add typescript.  i did it just now against 11.  it was about an hour of work.  i have full type checking against my ASTs, and typescript pointed out a bunch of bugs in my PEG.
But if I add it to 11, will it get released?
Can you please get this library back to where other people can contribute, and things will actually get merged?  This is a very heavily used project.

buddy, i don't mean to be rude, but the version of peg on npm is more than three years old.  saying "i'm scrapping and starting with an even bigger thing" is a problem
if you're three years in and scuttling the work, it's time to try a different development pattern.  this is a very heavily used library and we're all waiting on important things that won't get merged
i just implemented typescript support against 11.  it took me about an hour.  i don't know whether to put it up, because i don't think you're going to merge 11.
i could write it against 10 in the same hour.  you could publish 0.10.1, with support for es modules and typescript, and this project could immediately go back to being healthy again.
we really don't want these big overarching redesigns blocking the library for three years.
we need this library to be back in a place where patches can be committed and published again.
are you willing to accept patches against 10?  that's the right thing to do here.
please let this go back to being a healthy community project.  either pick that 0.10.0 is accepting patches, or merge and publish 0.11.0.
let us help.  don't leave the library in a state where nobody but you can work.

This is the way you treat a personal itch-scratching project, not a project that other people depend on.


fix emu-import bug

Today I was trying to update https://github.com/tc39/proposal-record-tuple from<EMAIL_ADDRESS>to the latest but from version 10.0.1 and onwards the generated spec was almost entirely empty as all the <emu-import> tags were being ignored.
Stepping through with a debugger I get to the querySelectorAll("EMU-IMPORT") and see that it returns an empty NodeList. Changing it to lowercase fixes the issue, I'm presuming there is some recent non-locked down sub dependency of jsdom that has recently changed, otherwise I don't know how this is working for others.

Sorry, yeah, this is a bug in a transitive dependency, which updated fairly recently. It's working for others because they still have a package-lock which names the older version of nwsapi, presumably. I'd prefer to just pin version 2.2.0 of nwsapi until this bug is fixed, rather than changing the selector. (You can do that in your own package.json right now without needing ecmarkup to make that change.)

Cool. I'll do that instead. Thanks!


re-write useHover hook

The original solution only works when ref is set on the first render.
However, under some circumstances it might be available only after first render.
In this case, the original solution will break.
There is no cleanup function set because once the node is removed, the event listeners attached to it will also be removed automatically.
What do you think?
Reference article(https://medium.com/@teh_builder/ref-objects-inside-useeffect-hooks-eb7c15198780)

Yeah I'm planning on updating it this week.


Make Gateway <-> Federation connections expire

Gateway clients keep a permanent web socket open with the Federation. Consider making the gateway client re-register every 5 minutes or to help federations detect with gateways that go offline
Users should be able to deactivate gateways that go offline for extended periods and no longer announce themselves to the federation

They could also just send a ping message or something.

They could also just send a ping message or something.

I feel like re-registering after 4 minutes if timeout is after 5 is much simpler since it re-uses the existing API endpoint. If registration entries can be updated using signed messages there is also the benefit of being able to change e.g. the announce address and the next registration will update it.

Maybe we get to optimize the de-register re-register cycle by observing transaction routing success, i.e
From Federation's perspective:

Successful routing of a tx by a gateway is a confirmation of liveness. Keep this gateway registered
Unsuccessful attempt to route tx will de-register a gateway from a federation. Same as a ping by the clients, provided we don't lose funds. Gateway uses this as a signal to re-register with the federation

From Gateway's perspective:

De-register (or just de-allocate capital as a form of partial de-register) from a federation after some configured period's lapses without any tx activity
Use the next tx attempt as a signal to re-register and re-allocate capital to federation?

Call: Websocket disconnections aren't a problem for gateway because it cal always re-connect. This is more about clients expecting a gateway to be online when if isn't online -- e.g. when generating invoices or paying.

Closed by #1506.


Fix #379 - cachet::install should be app::install

This change also uses the latest official release of Cachet by default and fixes a php issue with later versions of the alpine image.
This is reported as problems #379 and is related to https://github.com/CachetHQ/Cachet/issues/4186

Any news about this ?

This change seem ok, any news?

please approve?


Add servant-checked-exceptions package

This adds the servant-checked-exceptions package:
https://hackage.haskell.org/package/servant-checked-exceptions

Previous bounds issues dealt with in https://github.com/fpco/stackage/commit/138154f379ad90b4446e188b31c09273b6faf49a.

Previous bounds issues dealt with in 138154f.

@lwm Was this comment supposed to be for #2489?

Previous bounds issues dealt with in 138154f.
@lwm Was this comment supposed to be for #2489?

Those were the bounds you saw on the failing Travis run here, so just wanted to be clear (but maybe, it wasn't clear at all :)). :+1:

@cdepillabout nice project! This was #1 on my servant wish list :-)

@bergmark Thanks!  If you have any suggestions, bugs, etc, feel free to file an issue/PR! https://github.com/cdepillabout/servant-checked-exceptions


How to use your code on cityscapses

Hi, @kazuto1011  Thanks for your work. I would like to know how to use your code on CityScapes? The init.caffemodel can be used for cityscapes or not?

Oh I have misunderstood that the init.caffemodel has been used for coco-sutff training! I think you can train cityscapes images by modifying the #classes of the model and integrating with cityscapes dataloader someone's provided, but sorry I'm not sure about the validity with the COCO pre-trained init.caffemodel. According to the general cityscapes manner, an imagenet pre-trained model should be used, right? I will fix the initialization part with the officially provided init model.


go/adbc/driver/snowflake: driver cannot support query "SHOW TABLES"

The SHOW TABLES is the basic functionality for driver to list the table information of database or schema.
Query: SHOW TABLES
  Message:
Apache.Arrow.Adbc.AdbcException : [Snowflake] arrow/ipc: could not read message schema: arrow/ipc: could not read message metadata: unexpected EOF

  Stack Trace:
CallHelper.TranslateCode(AdbcStatusCode statusCode) line 794
CallHelper.Call(IntPtr fn, CAdbcStatement& nativeStatement, CArrowArrayStream* arrowStream, Int64& nRows) line 616
AdbcStatementNative.ExecuteQuery() line 312
AdbcCommand.ExecuteQuery() line 144
AdbcCommand.ExecuteReader(CommandBehavior behavior) line 176
AdbcCommand.ExecuteReader() line 160
ClientTests.CanClientExecuteQuery(AdbcConnection adbcConnection, TestConfiguration testConfiguration) line 107
ClientTests.CanClientExecuteQueryWithShowTable() line 159

@zeroshade - https://github.com/snowflakedb/gosnowflake/releases/tag/v1.7.2 has the fix for what I think is the same issue outlined here. The current gosnowflake library is at 1.6.23. Can we do an update?

The 1.7.2 update did not resolve the issue, but there is a way to make this work using pagination and the existing code.
Use the query SHOW TABLES LIMIT 100 and parse the results. Retain the last table found and then run the query ```SHOW TABLES LIMIT 100 FROM `last_table````. Continue to do this until you have parsed all of the tables.
I will add a test that demonstrates how this works.

@davidhcoe Does this issue only impact on the SHOW TABLES query or other simple query like SELECT has the similar issue?

@ruowan @davidhcoe have either of you tried using ADBC v0.10.0 with the "SHOW TABLES" query? At least in my local test it appears to succeed without issue

We are using the nuget package that @davidhcoe shared us last time on Feb.5. In my local testing, it still failed. @zeroshade could you have a try to running SHOW TABLES on the TEMP_SHOW_TABLES database which failed in my local test?
Btw, the TEMP_SHOW_TABLES has around 900 tables. The table count may be a factor causing this issue.  Also, I am wondering whether it only happens on SHOW TABLES or other query like simple select has such issue?

It is specific to @ruowan's environment and the number of tables that exist. The problem persists with the latest code. The only known workaround is the paging.

I test the paging locally; our existing solution duration is 1.3s but with the paging. the Duration has performance downgrade. And I guess the SHOW TABLES LIMIT FROM execute and retrieve all result only return the limit count of result which cause this performance downgrade.

SHOW TABLE
SHOW  TABLE WITH PAGING
Limit

1.3s
13.6s
100

1.5s
8.3s
200

1.5s
7.6s
400

Most likely a significant part of the issue is that unlike normal select queries, snowflake doesn't return the results of SHOW TABLES using Arrow, it returns it as JSON that we have to parse and manipulate into Arrow. This is likely an issue that needs to be addressed on the server side of Snowflake


File gets nested twice under different parents

Describe the bug
I have a webpack.mix.js file in a php project that gets nested under both artisan and package.json files. Clicking on any of the nested files opens the same file.
I'm using the latest version of the extension with no custom config applied.

Reproduction
Provided a screenshot
System Info
System:
    OS: Windows 10 10.0.22621
    CPU: (12) x64 AMD Ryzen 5 3600 6-Core Processor
    Memory: 8.76 GB / 15.93 GB
  Binaries:
    Node: 18.18.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.19 - C:\Program Files\nodejs\yarn.CMD
    npm: 10.1.0 - C:\Program Files\nodejs\npm.CMD
    pnpm: 8.7.6 - C:\Program Files\nodejs\pnpm.CMD
  Browsers:
    Edge: Spartan (44.22621.2283.0), Chromium (117.0.2045.43)
    Internet Explorer: 11.0.22621.1

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

Closed in favor of #142. Sorry for the duplicate