Alerts aren't firing with error 'Alert query returned a non-number value.' when returning a date column

A clear and concise description of what the bug is.
How to reproduce the bug

Create an alert
Use the examples database
Use this query: SELECT year FROM main.wb_health_population LIMIT 1
Set the condition as !=
Set the value a 0
Finish filling out the rest of the alert form

Expected results
The alert is sent because the condition is met
Actual results
Send error with Alert query returned a non-number value.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment
(please complete the following information):

browser type and version:
superset version: superset version
python version: python --version
node.js version: node -v
any feature flags active:

Checklist
Make sure to follow these steps before submitting your issue - thank you!

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

Additional context
Looks like the first failure was with this SHA 2ac28927a326fee6431b5e01f7930e650c636c17

@sadpandajoe @eschutho @betodealmeida @john-bodley
Given that the threshold needs to be a double precision number, do the query always need to return a numeric value?

If that's the case, are we missing a tooltip explaining this constraint? I'm assuming the execution log will show the error.

This was working beforehand with a single digit so if we need double precision, it could potentially break for other people who don't currently have it.

This was working beforehand with a single digit so if we need double precision, it could potentially break for other people who don't currently have it.

@sadpandajoe I believe the error is not related to the double precision but to the fact that the query is returning a non-numeric value.

Per here it seems like year is a DATETIME column and thus shouldn't the specified alert query,
SELECT year FROM main.wb_health_population LIMIT 1

be (assuming SQLite):
SELECT CAST(strftime('%Y', year) AS INT) FROM main.wb_health_population LIMIT 1

@sadpandajoe found that this is a regression from https://github.com/apache/superset/pull/26187. We bumped Pyarrow from 12.x to 14.x and there was a breaking change that impacts the value of year from an into to a timestamp, thus breaking existing alerts if you were (probably incorrectly) returning the year as an int. If we want to maintain our own backward compatibility, we could cast it back to an int automatically. I'm not sure all of the places where this change has impacted the codebase other than alerts, or if we want to consider this a bug fix rather than a breaking change. Thoughts?

For context here's the line where @sadpandajoe's error was coming from. The Python float() method will try to convert any type into a float and will throw an exception otherwise:
>>> float('2023')
2023.0

>>> float('2023-12-01')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ValueError: could not convert string to float: '2023-12-01'
>>>

@eschutho and @sadpandajoe are we sure this is a regression as I'm able to repro this using version 12.0.0 of pyarrow?
Additionally, per https://github.com/apache/superset/issues/26298#issuecomment-1861644976, it seems like the year column is a DATETIME rather than an INT:

I've also been able to repro the behavior—which seems correct—on 3.0.2:
> git checkout 3.0.2
Updating files: 100% (1401/1401), done.
Note: switching to '3.0.2'.
...
HEAD is now at 961eba6d97 chore: Updates CHANGELOG.md for 3.0.2 (rc2)

> rm -rf examples.db
> python3.9 -m pip install -e .
> FLASK_DEBUG=true superset load-examples
> FLASK_DEBUG=true superset run --port 8088

> sqlite3 examples.db
SQLite version 3.41.1 2023-03-10 12:13:52
Enter ".help" for usage hints.
sqlite> .schema wb_health_population
CREATE TABLE wb_health_population (
	country_name VARCHAR(255),
	country_code VARCHAR(3),
	region VARCHAR(255),
	year DATETIME
);

Reading the comments it seems this is not a bug but an expected error when the query returns something that is not numeric. It seems the only follow up could be adding a tooltip to the SQL input to make this restriction more clear as I suggested here.

@john-bodley yeah and if you do an alert based on that and in the successful logs you get -315619200000000000 as a return value which will trigger the alert. So either it's been an existing bug within alerts and reports that just worked or it's a breaking change and we should at least put it in the breaking change log so if people do upgrade it won't break their alerts.

or it's a breaking change and we should at least put it in the breaking change log so if people do upgrade it won't break their alerts.

@sadpandajoe I personally don't view this as a breaking change but more like a fix. Previously, if someone had a query that didn't return a numeric value, the application was wrongfully converting the return value and firing alerts under a incorrect condition evaluation. What the behavior should be is the current one, where errors are thrown when the condition cannot be evaluated. Right now, there are alerts that are being fired under false pretenses and they should stop working, or in other words, they should break.


New version: LinwoodCloud.Butterfly.Nightly version 2.2.2-rc.1

Pull request has been created with WinGet Releaser v2 :rocket:
Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Installation failed with exit code -1978334972
2024-11-04 03:13:59.032 [CLI ] No suitable installer found for manifest Microsoft.VCRedist.2015+.x64 with version 14.40.33816.0

Automated error analysis suggests -1978334972 may mean This package has a dependency missing from your system.
(Automated response - build 897.)

@wingetbot run

Automatic Validation ended with:

Installation failed with exit code -1978334972
2024-11-05 05:29:45.763 [CLI ] No suitable installer found for manifest Microsoft.VCRedist.2015+.x64 with version 14.40.33816.0

Automated error analysis suggests -1978334972 may mean This package has a dependency missing from your system.
(Automated response - build 897.)

[Policy] Needs-Manual-Merge


Fix deprecation warning

Hi,
Thank you for creating this plugin. Just wanted to note that since upgrading to 4.8.1 version of Gradle I have been getting the following warnings:
> Configure project :
Creating a custom task named 'dependencies' has been deprecated and is scheduled to be removed in Gradle 5.0. You can configure the existing task using the 'dependencies { }' syntax or create your custom task under a different name.'.
Creating a custom task named 'init' has been deprecated and is scheduled to be removed in Gradle 5.0. You can configure the existing task using the 'init { }' syntax or create your custom task under a different name.'.
The property(Class) method has been deprecated and is scheduled to be removed in Gradle 5.0. Please use the ObjectFactory.property(Class) method instead.

Again thank you!

Thanks for the information, I'll take care of it.

On 29 Jul 2018, at 8:08 AM, John Roach<EMAIL_ADDRESS>wrote:
Hi,
Thank you for creating this plugin. Just wanted to note that since upgrading to 4.8.1 version of Gradle I have been getting the following warnings:

Configure project :
Creating a custom task named 'dependencies' has been deprecated and is scheduled to be removed in Gradle 5.0. You can configure the existing task using the 'dependencies { }' syntax or create your custom task under a different name.'.
Creating a custom task named 'init' has been deprecated and is scheduled to be removed in Gradle 5.0. You can configure the existing task using the 'init { }' syntax or create your custom task under a different name.'.
The property(Class) method has been deprecated and is scheduled to be removed in Gradle 5.0. Please use the ObjectFactory.property(Class) method instead.
Again thank you!

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/gogradle/gogradle/issues/238, or mute the thread https://github.com/notifications/unsubscribe-auth/AMGhqyqzSOxUM_dRrP-DTgE7T1bFP188ks5uLP0LgaJpZM4VlOpL.


RFC: Login Frontend

Description

Implement frontend changes for login system.
Branch off of auth-backend.

Design
TO BE ELABORATED if Brian doesn't get this done soon and we delegate it.

Issues

Related to #890

Steps left

[ ] Properly set and read auth cookie.
[ ] When user loads schedule, check visibility and allow certain permissions.
[ ] Add ability to update username.
[ ] Use electrodb.


chore: supporting video thumbnails

Description

Deploy Notes
Notes regarding deployment of the contained body of work. These should note any
db migrations, nginx routes, infrastructure changes, and anything that must be done before deployment.

[ ] Need to add environment variables

ENV_VAR=

Screenshots (if appropriate)

@ximxim I think this broke this page--https://onchainsummer-xyz-git-azim-video-thumbnail-support-base-org.vercel.app/anotherblock?spoofDate=2023-08-17
let's check auto play

Review Error for lauchness @ 2023-08-16 13:32:57 UTC
User failed mfa authentication, public email is not set on your github profile. see go/mfa-help

zora declined


chore: do not require identical git hash during DKG

That is waay to constraining. Different peers can't even include non-code fixes for their local deployement and other problems.
Since we're kind of good at the release and versioning and stability things, change the requirement to a code_version_str, which consists of the cargo release version of Fedimint code and optional vendor suffix.

Yeah, it makes sense to allow different git versions.
But now it's not clear if we support the following scenario:
So I create a new fedimintd with a new module and try to setup with the vanilla fedimintd (perhaps accepting the standard modules). Will this ceremony work?

So I create a new fedimintd with a new module and try to setup with the vanilla fedimintd (perhaps accepting the standard modules). Will this ceremony work?

Nope. As things are right now, we don't have any way for peers to vote/agree on a common set and composition of modules, so they have to be exactly the same everywhere.
Maybe one day thought. Then we can remove the whole requirement, and each module will be negotiated independently etc.

Backport failed for releases/v0.3, because it was unable to cherry-pick the commit(s).
Please cherry-pick the changes locally and resolve any conflicts.
git fetch origin releases/v0.3
git worktree add -d .worktree/backport-5313-to-releases/v0.3 origin/releases/v0.3
cd .worktree/backport-5313-to-releases/v0.3
git switch --create backport-5313-to-releases/v0.3
git cherry-pick -x 0800e73e7641d488e14e02294574ff489e3e56e2

I don't think we really need to backport it. People lived with it since now, so they can wait ~2 months for 0.4.


[Sync] kserve/modelmesh-serving-v0.11.0-rc0 to main branch

Motivation
This is Release Sync. Detail information is the here:

Sync Date:
kserve/modelmesh release issue: https://github.com/kserve/modelmesh-serving/pull/390
kserve/modelmesh-runtime-adapter release version : v0.11.0-rc0
odh/modelmesh-runtime-adapter branch: main

Tested it with this ticket: https://github.com/opendatahub-io/modelmesh-serving/pull/107

/retest

openshift ci has a problem so it needs to be merged manually but a new fast image was built and tested by this PR:https://github.com/opendatahub-io/modelmesh-serving/pull/107


Opens and then closes a few moments later

When i open the program, i have the Xbox app also open but the emulator still closes a few seconds after opening.
Please help

This worked for me earlier:

Uninstall everything installed by this application(slimdx, any other libraries installed)
Unplug any physical Xbox peripherals
Open device manager and use "Uninstall Device" on all of the listed "Xbox 360 Peripherals"
Run ScpDriverInstaller.exe and uninstall
Delete the XboxKeyboardMouse folder
Reboot
Install SlimDX (x86) (this is a direct link to the download)
Download and run the installer.

Thanks alot it worked


Compiling With Local Environment Variables (PATH, LD_LIBRARY, PKG_CONFIG_PATH)

Currently the compiling step uses a pure environment variables (https://github.com/codrsquad/portable-python/blob/688b6b302c4a709ef916d808d42157332ca119e8/src/portable_python/init.py#L513), and ignore the local environment variables.
For example, I want to compile Python with a custom GCC, like use devtoolset-11 in centos 7.9:
foo@bar:~$ scl enable devtoolset-11 rh-python38 -- portable-python build 3.7.17

SCL will introduce the GCC to PATH and LD_LIBRARY, and for now portable-python will ignore these environment variable.
Is this design in purpose? If not, I would like to create a pull request to fix.

The scl enable devtoolset-11 executes the following:
# General environment variables
export PATH=/opt/rh/devtoolset-11/root/usr/bin${PATH:+:${PATH}}
export MANPATH=/opt/rh/devtoolset-11/root/usr/share/man${MANPATH:+:${MANPATH}}
export INFOPATH=/opt/rh/devtoolset-11/root/usr/share/info${INFOPATH:+:${INFOPATH}}
export PCP_DIR=/opt/rh/devtoolset-11/root
# bz847911 workaround:
# we need to evaluate rpm's installed run-time % { _libdir }, not rpmbuild time
# or else /etc/ld.so.conf.d files?
rpmlibdir=$(rpm --eval "%{_libdir}")
# bz1017604: On 64-bit hosts, we should include also the 32-bit library path.
# bz1873882: On 32-bit hosts, we should include also the 64-bit library path.
# bz2027377: Avoid unbound variables
if [ "$rpmlibdir" != "${rpmlibdir/lib64/}" ]; then
  rpmlibdir32=":/opt/rh/devtoolset-11/root${rpmlibdir/lib64/lib}"
  dynpath32="$rpmlibdir32/dyninst"
  rpmlibdir64=
  dynpath64=
else
  rpmlibdir64=":/opt/rh/devtoolset-11/root${rpmlibdir/lib/lib64}"
  dynpath64="$rpmlibdir64/dyninst"
  rpmlibdir32=
  dynpath32=
fi
# Add SCL dyninst to LD_LIBRARY_PATH, both 64- and 32-bit paths.
export LD_LIBRARY_PATH=/opt/rh/devtoolset-11/root$rpmlibdir/dyninst$dynpath64$dynpath32${LD_LIBRARY_PATH:+:${LD_LIBRARY_PATH}}
# Now prepend the usual /opt/.../usr/lib{64,}.
export LD_LIBRARY_PATH=/opt/rh/devtoolset-11/root$rpmlibdir$rpmlibdir64$rpmlibdir32${LD_LIBRARY_PATH:+:${LD_LIBRARY_PATH}}
export PKG_CONFIG_PATH=/opt/rh/devtoolset-11/root/usr/lib64/pkgconfig${PKG_CONFIG_PATH:+:${PKG_CONFIG_PATH}}

The idea was to not rely on env vars on build host, but conservatively set them so that a build would be as portable as possible.
We can review that yes, thinking here you'd want to use existing LD_LIBRARY_PATH and PKG_CONFIG_PATH?
PR welcome, I think the current one is too conservative perhaps + it is trivial to ensure env vars clean on builder with an export LD_LIBRARY_PATH= if need be.

I just made a simple pull request for this feature, #44.


Unit tests for zs_text

Test coverage for zs_text is about 88%. Lets improve this. See Coveralls Report for more details.

Resolved. See commit 600278a4fa7cfbcee434409340b0eab804c4ff64


Mindboggling invalidation f***k-up while computing wing span

I happened to extract the following strangeness, that is related to some wierd invalidation behavior
TEST_F(TiglWing, tiglHeyWhatsGoingOn)
{
    tigl::CCPACSConfigurationManager& manager = tigl::CCPACSConfigurationManager::GetInstance();
    tigl::CCPACSConfiguration& config = manager.GetConfiguration(tiglHandle);

    auto& vtp = config.GetWing(3);

    // somehow, this generates something
    // removing this line makes the test succeed!!!
    vtp.GetWingspan();

    auto vtpSymAx = vtp.GetSymmetryAxis();
    // which is not correctly invalidated now
    // note, that we actually don't change anything
    vtp.SetSymmetryAxis(vtpSymAx);

    auto span = vtp.GetWingspan();
    EXPECT_NEAR(5.9, span, 0.1); // the reported wing span is 0.61 instead of 5.4
}

Here's some behaviour I noticed:

If the first line of wing span computation is removed, everything succeeds.
If the line setting the symmetry is remove, everything succeeds.
Even though, the symmetry plane is actually not changed, the following invalidation messes it up.

The problem was in the method CCPACSWing::InvalidateImpl. In line
https://github.com/DLR-SC/tigl/blob/8e1b13dd136cc90184ede4d538f9f0ad039e8fb3/src/wing/CCPACSWing.cpp#L144
the method Reset() is called, which is simply wrong. Invalidation should never reset any values except from cached results.


Error when loading the plugin

The following error happened 2 hours ago in a build:
Could not import plugin:
Unexpected token {
    vm.js:80:10 createScript
    vm.js:139:10 Object.runInThisContext
    module.js:617:28 Module._compile
    module.js:664:10 Object.Module._extensions..js
    module.js:566:32 Module.load
    module.js:506:12 tryModuleLoad
    module.js:498:3 Function.Module._load
    module.js:597:17 Module.require
    internal/module.js:11:18 require
    /opt/build/repo/node_modules/fs-extra/lib/mkdirs/index.js:3:44 Object.<anonymous>

I am guessing this probably was triggered by you @Tom-Bonnike, but I just wanted to let you in case this was an unresolved bug. Thanks!

Yes, it’s because of the node version like I mentioned in https://github.com/netlify/plugins/pull/30#issuecomment-630302173.

Oh yes. That's definitely a very hard to debug error message. I hope we can print better warnings using engines.node soon.


Removed invalid # from inline.hpp.

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

Project coverage is 97.12%. Comparing base (4b7483e) to head (87bce40).

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #336   +/-   ##
=======================================
  Coverage   97.12%   97.12%
=======================================
  Files         152      152
  Lines        9163     9163
=======================================
  Hits         8900     8900
  Misses        263      263


scala 2.12 integration and JDK version

I am trying to integrate the 2.12 JAR (https://mvnrepository.com/artifact/nl.grons/metrics-scala_2.12/3.5.5), building with jdk1.8 and scala 2.12.0, and I'm running into a problem NoSuchMethodError nl.grons.metrics.scala.BaseBuilder.$init$.
What version of the JDK was the library compiled with?
When I take the metrics-scala source and build it locally with JDK1.8- I have no problem.
thanks,

nevermind i found it- javap -v BaseBuilder.class | grep major


is Registry 2.0 unsupported?

Hi!
I'm trying to deploy using an private docker/registry-2.0 (docker-distribution)
Have I misconfigured my registry or is there simply no support for /v2/ api yet?
$ centurion -p myApp -e production
** Invoke environment:production (first_time)
** Invoke environment:common (first_time)
** Execute environment:common
** Execute environment:production
** Invoke centurion:setup (first_time)
** Invoke centurion:clean_environment (first_time)
** Execute centurion:clean_environment
** Execute centurion:setup
** Invoke list (first_time)
** Execute list
** Invoke list:tags (first_time)
** Execute list:tags
GET: "https://registry.mydomain.com/v1/repositories/myapp/tags"
E, [2015-09-21T11:42:34.014356 #4959] ERROR -- : Couldn't communicate with Registry: #<Excon::Response:0x0000000195d188 @data={:body=>"\r\n<title>404 Not Found</title>\r\n\r\n
404 Not Found
\r\n
nginx/1.9.4\r\n\r\n\r\n", :headers=>{"Server"=>"nginx/1.9.4", "Date"=>"Mon, 21 Sep 2015 09:51:43 GMT", "Content-Type"=>"text/html", "Content-Length"=>"168", "Connection"=>"keep-alive"}, :status=>404, :status_line=>"HTTP/1.1 404 Not Found\r\n", :reason_phrase=>"Not Found"}, @body="\r\n<title>404 Not Found</title>\r\n\r\n
404 Not Found
\r\n
nginx/1.9.4\r\n\r\n\r\n", @headers={"Server"=>"nginx/1.9.4", "Date"=>"Mon, 21 Sep 2015 09:51:43 GMT", "Content-Type"=>"text/html", "Content-Length"=>"168", "Connection"=>"keep-alive"}, @status=404, @remote_ip="x.x.x.x", @local_port=39801, @local_address="x.x.x.x">

Note:
If i do try to replace the v1 to a v2 and visit the link manually:
https://registry.mydomain.com/v2/repositories/myapp/tags
I do indeed get to the place i should be, instead of a 404.

It looks like the V2 registry is not backward compatible in API with the V1 registry. We've never implemented the V2 calls so it looks like that doesn't work, unfortunately.  Since our production code uses either the public registries or Dogestry, we've not had to implement V2 yet. The public registries all still work with the V1 API calls.
From your testing it looks like we could try to just substitute V2 everywhere and see what breaks but my guess is a lot breaks. If you want to work on that we'd appreciate the contribution. Otherwise it will probably be awhile.

I took a stab at this about a month ago due to this being a block for us using centurion for deployments. I made the API version a config parameter with a default of 1 to keep backwards-compatibility. FWIW, we've been using this fork in production for a few weeks now without any issues.
I would certainly not mind creating a pull request with these changes if people are interested. It would require a bit of refactoring at this point due to some of the recent changes here that conflict with some of the changes I made.

Will definatly need v2 support to roll this out. Be happy to exercise your PR

Pull request for this referenced in #153


Support streaming audio only

Add support for streaming audio only, no video.

Resolved in 5a88affd6fa611f3c43bbbdf84df46a477d682fd and af7c63f746bc07d97797842d68786555185b0828


Does SciSpacy support other languages?

I would like to know how to use the scientific and medic models for texts in portuguese

Hi, all of scispacy's models are trained on english text exclusively. If you have some Portuguese labeled datasets, you can retrain spacy's base Portuguese model on your texts, but scispacy only supports English at this time.


feat: progress improvements: less API calls, more info closes #26

fix: progress returned for all environments not what was needed by view
refactor: cleaning up css class names
feat: progress now uses dynamic dashboard to lower api calls
feat: upgrade octopus.client to 3.3.3
fix: progression style cleanup
chore(release): bump version, update changelog
feat: added ability to order release batches
fix: only check progress if there are projects in a release batch
fix: GetEnvironments takes permission in as a query param closes #38
fix: getprogression no longer takes in environmentids closes #37
feat: progress command cleanup
fix: improved exception handling
feat: allow for enabling of stdout logging from octovariable
fix: remove duplicate octopus.client references
fix: environment permissions now respected for projects as well fixes #39
fix: force logs directory to be created
chore(release): bump version, update changelog
fix: logs file powershell fix
chore(release): bump version, update changelog
chore: deploy kraken step template for Octopus Deploy
chore: deploy kraken step template for Octopus Deploy
chore: missing semicolon
refactor: cleaning up method names to be more clear and not exposing permission query to GetBatchEnvironments

Hmm.. I seem to have messed up my rebasing.


Switch from using NodePaths to using Nodes

Across the project, NodePaths were used instead of just Nodes. Although GDExtension doesn't make it obvious, there is a way to export Node.

Thanks, this is really good to know! I thought the NodePaths should be used with the inspector, and sprinkled them all over the place as a result.


Dock

So, i just installed this in a vm. The dock dosent start and i dont know how to activate it. Also, i have installed dolphin for (reasons) and it appears as if i have no icons, this is probably because i have to set them somewhere but if you want this to become a "distro" i think some icons for folders and stuff would also be nice. (again, its probably my fault, but im a noob).

I'm assuming you've installed Plank using sudo pacman -S plank
And to start the Dock you'll need to move the pixarch/flavours/i3/config file to your ~/.config/i3 folder
(I am assuming you have already done this)
This should start the dock.

To get the icons working, you'll have to do a little bit of work as the installation script for this is still yet to be made.
First of all, I am assuming that you have cloned the repo on the ~/ directory.
Navigate to ~/pixarch/applications/dock and open each and every .desktop file
You'll see a line called
Icon=/home/heisenburgh/pixarch/icons/app.ico

Now, all you have to do is replace "heisenburgh" with your own username.
Lastly, move all the .desktop files from the pixarch/applications/dock directory to /usr/share/applications directory and reboot.
Sorry it's such a hassle right now, it's a wip and so much is left to do!

Thanks a lot man, im planning on switching to it to my main rog and want to know all the shit before i do so
Thanks ^_^

Ok so i've done the stuff, for anyone else wondering why it dosent work you have to do this
cd ~/pixarch/applications/dock
sudo pacman -S nano (my choise)
nano Alacrity
And change
Icon=/home/YOURNAME/pixarch/media/icons/aseprite.ico
#You forgot about the media folder
Repeat for the rest

This still dosent fix the dock, but again, its maybe becouse of the vm

This still dosent fix the dock, but again, its maybe becouse of the vm

Shouldn't be because of the VM.
What exactly isn't working?

This still dosent fix the dock, but again, its maybe becouse of the vm

Shouldn't be because of the VM. What exactly isn't working?

The dock itself isnt starting, i have tried launching it with super+space "dock" and still dosent open

Hmmm
This shouldn't happen
Can you remove and re-install the Dock?
Or...
one thing you can do is
Open up a terminal
And just type in plank
Then can you share the Log you get there


cmd/go: add subversion zip support

Pending CL 120998 is missing Subversion zip support. Need to add that.

Will have to deal with this in Go 1.12.

Fixed in 1.14.


Is go 1.9 required or is go 1.8 supported?

containerd v1.0.0-beta.2 builds ok with go 1.8 and appears to work, but one of the tests fails to build because it uses new 1.9 only pprof methods:
# github.com/containerd/containerd/metadata
metadata/db_test.go:322: undefined: pprof.Labels
metadata/db_test.go:323: undefined: pprof.Do
metadata/images_test.go:476: t.Helper undefined (type *testing.T has no field or method Helper)
metadata/images_test.go:499: t.Helper undefined (type *testing.T has no field or method Helper)
FAIL	github.com/containerd/containerd/metadata [build failed]

I see that because of #1215 the CI has been bumped to 1.9.
In LinuxKit we use alpine 3.6 and hence go 1.8. If go 1.9 is a requirement then we could switch to Alpine Edge but we'd obviously rather not do that if we don't have to.
I couldn't find a reference to a minimum go version in the repo, so if 1.9 is required it would be good to add that somewhere.
If 1.8 is still supported perhaps CI could build but not run the tests (assuming the failure rate of #1215 is too high to tolerate).

required

Can this be revisited?  Go 1.9 is not widely available in Linux distributions, and forcing it as a requirement will be hinder the progress of #1508 significantly.  I'm not a go developer, but is it possible to conditionally skip the test that require 1.9 when run with 1.8?

@carlwgeorge there are bugs in 1.8 that are fixed in 1.9.  And its not just test code but the binary code as well that needs it.  1.9 has been out for a while now.

Go 1.9 was released on 2017-08-24.  That was 1 month and 18 days ago.  I'm not trying to be argumentative, just adding context for anyone that reads this later.

@carlwgeorge no worries if 1.9 is required. I vaguely remember the fedora golang maintainer not keen on bumping f26 to 1.9, but since f27 is gonna be released soon anyway, you can just build for rawhide and f27.

Circling back since this is coming up again in #1508.  RHEL 7.5 was released on 2018-04-10 with golang 1.9.  Containerd 1.1.0 was released on 2018-04-23 and now requires at least golang 1.10.  I can upgrade Fedora 28 to containerd 1.1.0, but this new minimum version of golang prevents me from building containerd for EPEL (RHEL/CentOS) like I had hoped.  Hopefully Red Hat will update golang to at least 1.10 soon.


increasing replicas from 1 to 2 or more causes ambassador to crash restart loop

Describe the bug
When increasing replica set from 1 to 2 or more causes replicas to crash
RBAC enabled
To Reproduce
Steps to reproduce the behavior:
kubectl apply -f https://getambassador.io/yaml/ambassador/ambassador-rbac.yaml
Expected behavior
All 3 pods that are part of replica set should be running
Versions (please complete the following information):

Ambassador: [e.g. 0.38]
Kubernetes environment [e.g. Minikube, bare metal, Google Kubernetes Engine]
DCOS/Mesos Kubernetes Framework
Version [e.g. 1.10.6]

Additional context
I have tried remove the statsd, liveness and readness probes and that does not solve the problem
---
apiVersion: v1
kind: Service
metadata:
  labels:
    service: ambassador-admin
  name: ambassador-admin
spec:
  type: NodePort
  ports:
  - name: ambassador-admin
    port: 8877
    targetPort: 8877
  selector:
    service: ambassador
---
apiVersion: rbac.authorization.k8s.io/v1beta1
kind: ClusterRole
metadata:
  name: ambassador
rules:
- apiGroups: [""]
  resources:
  - services
  verbs: ["get", "list", "watch"]
- apiGroups: [""]
  resources:
  - configmaps
  verbs: ["create", "update", "patch", "get", "list", "watch"]
- apiGroups: [""]
  resources:
  - secrets
  verbs: ["get", "list", "watch"]
---
apiVersion: v1
kind: ServiceAccount
metadata:
  name: ambassador
---
apiVersion: rbac.authorization.k8s.io/v1beta1
kind: ClusterRoleBinding
metadata:
  name: ambassador
roleRef:
  apiGroup: rbac.authorization.k8s.io
  kind: ClusterRole
  name: ambassador
subjects:
- kind: ServiceAccount
  name: ambassador
  namespace: default
---
apiVersion: extensions/v1beta1
kind: Deployment
metadata:
  name: ambassador
spec:
  replicas: 2
  template:
    metadata:
      annotations:
        sidecar.istio.io/inject: "false"
      labels:
        service: ambassador
    spec:
      serviceAccountName: ambassador
      containers:
      - name: ambassador
#        image: quay.io/datawire/ambassador:0.39.0-rc0
        image: quay.io/datawire/ambassador:0.38.0
        resources:
          limits:
            cpu: 1
            memory: 400Mi
          requests:
            cpu: 200m
            memory: 100Mi
        env:
        - name: AMBASSADOR_NAMESPACE
          valueFrom:
            fieldRef:
              fieldPath: metadata.namespace
#        livenessProbe:
#          httpGet:
#            path: /ambassador/v0/check_alive
#            port: 8877
#          initialDelaySeconds: 30
#          periodSeconds: 3
#        readinessProbe:
#          httpGet:
#            path: /ambassador/v0/check_ready
#            port: 8877
#          initialDelaySeconds: 30
#          periodSeconds: 3
#      - name: statsd
#        image: quay.io/datawire/statsd:0.38.0
      restartPolicy: Always

kubectl get pod
NAME                          READY     STATUS             RESTARTS   AGE
ambassador-7fd4cdb8b5-fv6k6   0/1       CrashLoopBackOff   7          16m
ambassador-7fd4cdb8b5-t68rx   0/1       CrashLoopBackOff   6          11m

If replicaset=1 ambassador deploys successfully
kubectl delete -f amb/ambassador-rbac.yaml
service "ambassador-admin" deleted
clusterrole.rbac.authorization.k8s.io "ambassador" deleted
serviceaccount "ambassador" deleted
clusterrolebinding.rbac.authorization.k8s.io "ambassador" deleted
deployment.extensions "ambassador" deleted

 kubectl apply -f amb/ambassador-rbac.yaml
service "ambassador-admin" created
clusterrole.rbac.authorization.k8s.io "ambassador" created
serviceaccount "ambassador" created
clusterrolebinding.rbac.authorization.k8s.io "ambassador" created
deployment.extensions "ambassador" created

 kubectl get pod --watch
NAME                          READY     STATUS    RESTARTS   AGE
ambassador-7fd4cdb8b5-cg86d   1/1       Running   0          18s

Crash logs
[2018-08-24 17:15:43 +0000] [19] [INFO] Listening at: http://<IP_ADDRESS>:8877 (19)
[2018-08-24 17:15:43 +0000] [19] [INFO] Using worker: threads
[2018-08-24 17:15:43 +0000] [47] [INFO] Booting worker with pid: 47
2018-08-24 17:15:43 diagd 0.38.0 [P47TMainThread] INFO: Starting periodic updates
2018-08-24 17:15:58 kubewatch 0.38.0 INFO: generating config with gencount 2 (1 change)
/usr/lib/python3.6/site-packages/pkg_resources/__init__.py:1298: UserWarning: /ambassador is writable by group/others and vulnerable to attack when used with get_resource_filename. Consider a more secure location (set with .set_extraction_path or the PYTHON_EGG_CACHE environment variable).
  warnings.warn(msg, UserWarning)
2018-08-24 17:16:03 kubewatch 0.38.0 WARNING: Scout: could not post report: HTTPSConnectionPool(host='kubernaut.io', port=443): Max retries exceeded with url: /scout (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f0d94e2d2b0>: Failed to establish a new connection: [Errno -3] Try again',))
2018-08-24 17:16:03 kubewatch 0.38.0 INFO: Scout reports {"latest_version": "0.38.0", "exception": "could not post report: HTTPSConnectionPool(host='kubernaut.io', port=443): Max retries exceeded with url: /scout (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f0d94e2d2b0>: Failed to establish a new connection: [Errno -3] Try again',))", "cached": false, "timestamp":<PHONE_NUMBER>.199165}
[2018-08-24 17:16:03.269][50][info][config] source/server/configuration_impl.cc:50] loading 0 static secret(s)
[2018-08-24 17:16:03.271][50][info][upstream] source/common/upstream/cluster_manager_impl.cc:132] cm init: all clusters initialized
[2018-08-24 17:16:03.271][50][info][config] source/server/configuration_impl.cc:60] loading 1 listener(s)
[2018-08-24 17:16:03.281][50][info][config] source/server/configuration_impl.cc:94] loading tracing configuration
[2018-08-24 17:16:03.281][50][info][config] source/server/configuration_impl.cc:116] loading stats sink configuration
got SIGHUP
forking and execing new child process at epoch 1
forked new child process with PID=54
got SIGCHLD
PID=54 exited with code=1
Due to abnormal exit, force killing all child processes and exiting
force killing PID=22
exiting due to lack of child processes
AMBASSADOR: envoy exited with status 1
Here's the envoy.json we were trying to run with:
{
  "listeners": [

    {
      "address": "tcp://<IP_ADDRESS>:80",

      "filters": [
        {
          "type": "read",
          "name": "http_connection_manager",
          "config": {"codec_type": "auto",
            "stat_prefix": "ingress_http",
            "access_log": [
              {
                "format": "ACCESS [%START_TIME%] \"%REQ(:METHOD)% %REQ(X-ENVOY-ORIGINAL-PATH?:PATH)% %PROTOCOL%\" %RESPONSE_CODE% %RESPONSE_FLAGS% %BYTES_RECEIVED% %BYTES_SENT% %DURATION% %RESP(X-ENVOY-UPSTREAM-SERVICE-TIME)% \"%REQ(X-FORWARDED-FOR)%\" \"%REQ(USER-AGENT)%\" \"%REQ(X-REQUEST-ID)%\" \"%REQ(:AUTHORITY)%\" \"%UPSTREAM_HOST%\"\n",
                "path": "/dev/fd/1"
              }
            ],

            "route_config": {
              "virtual_hosts": [
                {
                  "name": "backend",
                  "domains": ["*"],"routes": [

                    {
                      "timeout_ms": 3000,"prefix": "/ambassador/v0/check_ready","prefix_rewrite": "/ambassador/v0/check_ready","weighted_clusters": {
                              "clusters": [

                                    { "name": "cluster_127_0_0_1_8877", "weight": 100.0 }

                              ]
                          }

                    }
                    ,

                    {
                      "timeout_ms": 3000,"prefix": "/ambassador/v0/check_alive","prefix_rewrite": "/ambassador/v0/check_alive","weighted_clusters": {
                              "clusters": [

                                    { "name": "cluster_127_0_0_1_8877", "weight": 100.0 }

                              ]
                          }

                    }
                    ,

                    {
                      "timeout_ms": 3000,"prefix": "/ambassador/v0/","prefix_rewrite": "/ambassador/v0/","weighted_clusters": {
                              "clusters": [

                                    { "name": "cluster_127_0_0_1_8877", "weight": 100.0 }

                              ]
                          }

                    }
                    ,

                    {
                      "timeout_ms": 3000,"prefix": "/httpbin/","prefix_rewrite": "/","host_rewrite": "httpbin.org","weighted_clusters": {
                              "clusters": [

                                    { "name": "cluster_httpbin_org_80", "weight": 100.0 }

                              ]
                          }

                    }

                  ]
                }
              ]
            },
            "filters": [
              {
                "name": "cors",
                "config": {}
              },{"type": "decoder",
                "name": "router",
                "config": {}
              }
            ]
          }
        }
      ]
    }
  ],
  "admin": {
    "address": "tcp://<IP_ADDRESS>:8001",
    "access_log_path": "/tmp/admin_access_log"
  },

  "cluster_manager": {
    "clusters": [
      {
        "name": "cluster_127_0_0_1_8877",
        "connect_timeout_ms": 3000,
        "type": "strict_dns",
        "lb_type": "round_robin",
        "hosts": [
          {
            "url": "tcp://<IP_ADDRESS>:8877"
          }

        ]},
      {
        "name": "cluster_httpbin_org_80",
        "connect_timeout_ms": 3000,
        "type": "strict_dns",
        "lb_type": "round_robin",
        "hosts": [
          {
            "url": "tcp://httpbin.org:80"
          }

        ]}

    ]
  },

  "statsd_udp_ip_address": "<IP_ADDRESS>:8125",
  "stats_flush_interval_ms": 1000
}AMBASSADOR: shutting down (1)

Possibly related to #693 (and maybe #568, just maybe)

@zencircle could you check if 0.39.0 release fixed it for you? - https://github.com/datawire/ambassador/releases/latest

This issue was due to pod-to-pod communication failure as result of missing iptables rules. Rules that fixed the problem
iptables -P FORWARD ACCEPT
iptables -t nat -A POSTROUTING -s <IP_ADDRESS>/8 -m set --match-set overlay dst -j MASQUERADE

Note: Our kubernetes CIDR is <IP_ADDRESS>\8
Note: There are other rules required for K8, these are missing ones in my configuration


[AMBARI-23969] UI should load stack services from multiple mpacks.

What changes were proposed in this pull request?
Changed cluster behavior to get stack services from multiple stacks
How was this patch tested?
All tests passing. Have disabled a few tests. Will add new ones later.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2493/
Test FAILed.
Test FAILured.


Introduce metrics type in metering plans

Added mandatory type string field to the metering plans

Coverage remained the same at 86.898% when pulling afc47d7819c8d73f9ac766f2c1859690377efc1b on hsiliev:metrics_type into e17d23b80c0d64802659db502ef8355dfe98514b on cloudfoundry-incubator:master.

The metering/rating/pricing schema check only happen at the provisioning plugin to allow the integrator to use their own schema and add additional fields that their custom provisioning plugin need to resolve metering/rating/pricing plan. Adding the metering metrics type here as required will not break any implementation that we have until we use the type field outside of the plugins.


Adapt to new GHA Runners

Explicitly install openssl on windows?

LGTM, thanks! But it seems to still be failing.

Yes, this was mostly for debugging purposes. @jasagredo and @newhoggy have been figuring out the underlying openssl issue.

Oh!
ld: warning: ignoring file '/usr/local/opt/cardano/lib/libsodium.23.dylib': found architecture 'x86_64', required architecture 'arm64'
ld: warning: ignoring file '/usr/local/opt/cardano/lib/libblst.dylib': found architecture 'x86_64', required architecture 'arm64'
ld: warning: ignoring file '/usr/local/opt/cardano/lib/libsecp256k1.2.dylib': found architecture 'x86_64', required architecture 'arm64'

Sigh...
GHC runtime linker: fatal error: I found a duplicate definition for symbol
   __blst_platform_cap
whilst processing object file
   C:\msys64\mingw64\opt\cardano\lib\libblst.a
The symbol was previously defined in
   C:\msys64\mingw64\opt\cardano\lib\libblst.a(#2:server.o)
This could be caused by:
   * Loading two different object files which export the same symbol
   * Specifying the same object file twice on the GHCi command line
   * An incorrect `package.conf' entry, causing some object to be
     loaded twice.
ghc-9.8.1.exe: Could not load Object Code C:\msys64\mingw64\opt\cardano\lib\libblst.a(#1:assembly.o).

that's a GHC bug, indirectly introduced by using a more recent libblst. Our cross compilers were fixed for this. (Hence it doesn't happen in nix builds).

This is the threepenny ui failure on darwin:
Configuring library for threepenny-gui-<IP_ADDRESS>..
Preprocessing library for threepenny-gui-<IP_ADDRESS>..
Building library for threepenny-gui-<IP_ADDRESS>..
[ 1 of 29] Compiling Foreign.JavaScript.Include ( src/Foreign/JavaScript/Include.hs, dist/build/Foreign/JavaScript/Include.o, dist/build/Foreign/JavaScript/Include.dyn_o )
[ 2 of 29] Compiling Foreign.JavaScript.Resources ( src/Foreign/JavaScript/Resources.hs, dist/build/Foreign/JavaScript/Resources.o, dist/build/Foreign/JavaScript/Resources.dyn_o )
WARNING: /Users/runner/.ghcup/ghc/9.6.4/lib/ghc-9.6.4/bin/./ghc-9.6.4 is loading libcrypto in an unsafe way

Ok, Window 9.8 is fixed now. Windows 9.6, we need to see. That seems to randomly die :-/
I'll try to also fix macOS.

Alright let's try to fix this for macos-lastest as well.

All GHA are back to green.


feat: Visitors Count

Create a visitor count in home page.
Each time a user visits the website count should increase.
Give unique styling as well.

@yamini1183 Assigned, you can now start working on this issue.


Check the model status before traffic gets redirected to the pods

/kind feature
Describe the solution you'd like
As of now, while the model is being loaded into memory, user queries may still reach a pod which may not be ready to answer (since the model is not loaded). The larger the model the larger the time to load it the larger the "downtime"
Anything else you would like to add:
This could be useful in different tasks like model/runtime canary rollout, replicas auto-scaling, etc

I think this ticket should be moved to upstream kserve repo ( https://github.com/kserve/kserve/issues ).
We can keep this as "clone" to simplify the tracking but the solution should be implemented upstream

A such feature is already implemented upstream: https://kserve.github.io/website/0.10/modelserving/data_plane/v2_protocol/#healthreadinessliveness-probes


[T3A4][W10-A4]Wang Xien Dong

Ready to merge

Some comments added. Please close the PR after reading comments.


SDK API Docs for other programming languages

Hi there,
Will you be accepting any contribution for other programming languages?

Contributions are always welcomed :)


UITapGestureRecognizer Help

I have buttons within the bubble of my messages. However, the built in tap gesture recognizer seems to be taking priority over them so I can not actually press the buttons themselves.  I tried to disable the tap recognizer, but that just didn't let me press anywhere in the bubble. Is there anyway around this?

Hey @dealaus
You'll want to look at Apple's docs on UIGestureRecognizer and it's delegate. What you need is there.
Also, be sure to review the contributing guidelines next time you open an issue :smile:


Extract strings into string file

Someone needs to take a pass through all the files and look for any hardcoded English strings we need to move out.

As a starting point, there are some places I marked with //TODO localize strings where I didn't feel like moving strings at the time but wanted to make a note of it to find those places in the future.

Should we just split up the files and sit down during a weekend to do this?

Rather than do this initially we should just enable language switching, then let users report places that aren't translated.  We can target those spots as users find them.


Getting at residuals after FIR-model-regression

Hi all,
I am not sure this is the right place to post (vs. neurostars?) since my question is in between a help call and an enhancement/documentation request - probably more of the former. I am trying to regress out task-related effects from fMRI data via FIR in order to not have to worry about wrong hemodynamics too much. I managed to set up the design matrix and estimate it:
hrf_model = 'FIR' #'Canonical With Derivative'
design_matrix = make_design_matrix(frame_times, paradigm, drift_model='polynomial',
                        drift_order=3, hrf_model = hrf_model,
                        add_regs= motion, add_reg_names= reg_names, fir_delays=range(1,5)) #
plot_design_matrix(design_matrix)
fmri_glm = FirstLevelGLM().fit(fmri_img, design_matrix)

Now I am stuck trying to get at the residuals. Browsing through the examples I can get the contrasts (FIR-results would be interesting per se), but how would I proceed to do sth. along the lines of
resid = y_act - y_hat  #then convert resid into a 4d-nii-image and continue with e.g. nilearn to extract data from nodes?
Nistats looks really superpromising! Thanks!
Ralf.

Dear Ralf,
We haven't included this feature in the nistats API.
I think that, if you're indeed interested in regressing out the effect related to a FIR model, my suggestion is to get the array of values in the design matrix that you have created, and then give them as confound argument to some masker.transform() application, where masker is a nilearn NiftiMasker. This will output some 'residual' times series. you can map them back to 4D NiftiImage using the masker.inverse_transform() method.
HTH,
Bertrand

Dear Bertrand,
that's the solution! Thanks so much! Still trying to do things the old-fashioned pre-python way: brain-space, classical modeling ... habits of thinking...
Best Ralf.


pre-fill the organ donation form

What's this PR do?
For the organ donation form, we are now pre-filling the form with any relevant info we already have. I'm using the zipcode from the first screen since we know for sure that we have that.
How should this be reviewed?
If any of the following fields for your account are filled out, they should be being pulled in. Zip Code should always be pulled in from the first screen.

first name
last name
birthdate
street address line 1
street address line 2
city
state

Relevant tickets
Fixes #6556
Checklist

[ ] Documentation added for new features/changed endpoints.

💯 👍

🚢


Leaflet.JS Library with Angular v15.2.6?

Hello I want use leaflet v1.9.3 with Angular v15.2.6 but I got this exception.
I try many solutions  in the web and use example in official documentation but the problem not resolved yet.
TypeError: this._events[type].push is not a function
    at NewClass._on (leaflet-src.js:532:24)
    at NewClass.on (leaflet-src.js:455:12)
    at NewClass.addHooks (leaflet-src.js:13693:22)
    at NewClass.enable (leaflet-src.js:5905:10)
    at NewClass.addHandler (leaflet-src.js:3846:14)
    at NewClass.init (leaflet-src.js:392:14)
    at proto.callInitHooks (leaflet-src.js:359:26)
    at NewClass.initialize (leaflet-src.js:3269:10)
    at new NewClass (leaflet-src.js:302:22)
    at createMap (leaflet-src.js:4863:11)

Can you provide an example repo of an angular project where you get this error?

Closing due to no response.


Color selector layout issue

I am using the color selector, but I have seeing the following layout issue where the vertical bar is moved under the square. Is anyone else experiencing this issue?

Same issue here.

Fixed this.
If you using bower you can use the this by change it
"dat-gui": "~0.5.1",

by
"dat-gui": "git@github.com:webcaetano/dat.gui.git#53762381014f91f5d22c10515dbdec85a425c04c"

I create a pull-request too with the fix.

I wasn't able to re-create the problem exactly, but this fix should solve the issue. Thanks @webcaetano for the solution


[1248] Expose course enrichments

Context
Course enrichment data, this is required for the frontend course details view
Changes proposed in this pull request

Optional include on courses endpoint

Guidance to review
example: http://localhost:3001/api/v2/providers/2AT/courses/35L8?include=enrichments
TODO: Specs

@benilovj good point. Let's discuss

max of 2, latest published, latest draft


Cannnot login to AzureRM Account from Mac PowerShell

Please help...

This issue was moved to Azure/azure-powershell#5856


Impossible to check whether a current process was launched using the torchelastic command on MacOS

🐛 Bug
When checking whether a current process was launched using the torchelastic command on a system running MacOS, it results in an error stating that module 'torch.distributed' has no attribute 'is_torchelastic_launched'. This is due to the torch.distributed not being available by default on MacOS, which leads to other APIs in the distributed package being hidden, as per the PyTorch documentation.
To Reproduce

Any call of the detect() method from the TorchElasticEnvironment class.
Expected behavior

The detect()method should return False when torch.distributed is not available.
Environment

PyTorch Lightning Version (e.g., 1.5.0): 1.6.3
PyTorch Version (e.g., 1.10): 1.11.0
Python version (e.g., 3.9): 3.8.13
OS (e.g., Linux): MacOS
CUDA/cuDNN version: -
GPU models and configuration: M1 Chip
How you installed PyTorch (conda, pip, source): conda
If compiling from source, the output of torch.__config__.show(): -
Any other relevant information:

Additional context

I discovered this bug when using the Lightning-Hydra-Template, which calls that method by default to check for availability of distributed job scheduling.

I've fixed this bug by simply incorporating an additional if-statement in the said method. Thus I am willing to submit a PR.

Can confirm I am seeing same error with running:
pytorch 0.11.1
`File ~/miniforge3/envs/gst/lib/python3.8/site-packages/pytorch_lightning/plugins/environments/torchelastic_environment.py:65, in TorchElasticEnvironment.detect()
63 """Returns True if the current process was launched using the torchelastic command."""
64 if _TORCH_GREATER_EQUAL_1_9_1:
---> 65 return torch.distributed.is_torchelastic_launched()
66 required_env_vars = {"RANK", "GROUP_RANK", "LOCAL_RANK", "LOCAL_WORLD_SIZE"}
67 return required_env_vars.issubset(os.environ.keys())

AttributeError: module 'torch.distributed' has no attribute 'is_torchelastic_launched'`

@aynetdia Thanks for reporting the issue. I think the patch will be something similar to #12794. Feel free to ping me anytime if you have any trouble submitting a PR :]

pytorch 0.11.1

@martinb-bb Do you mean PyTorch 1.11.0?

@akihironitta  hi, sorry! yes 1.11.0 👍

Hi @akihironitta  Can you tell me when this fix will be pushed to PyPI package? thanks 😄

@martinb-bb We finalised the next patch release in #13019, so it should be available very soon (today I believe).


Issues after upgrading to pnpm v9

After upgrading to pnpm v9, the isolate script is now failing. Here's what I see in the logs:
i  functions: Start isolating the source folder...
info Generating PNPM lockfile...
i  functions: +++ Failed to isolate: Failed to generate lockfile: value.startsWith is not a function

I know that the lockfile format has changed in PNPM v9. Could it be related to that?

I noticed this too yesterday. The imported pnpm functions do not seem to be compatible with v9. If the v9 logic is not backwards compatible with v8 it might be tricky to solve because we can not link two versions of pnpm in the dependencies.
Possibly we could then just release a v2 of this library that is only compatible with v9 of pnpm.
I don't expect to work on this in the coming days or even weeks, so I suggest to stick with v8 for now.

I was about to open this issue.

It seems you can install 2 versions of the same dependency. That means I should be able to use the pnpm lockfile handling functions from v8 next to v9, and that would make it possible to support v9 while staying backwards compatible with v8.
I will try to allocate some time for this soon.

Support was just added in 1.14.0.
I will now adopt the firebase-tools-with-isolate fork to include it.


My second revup foo

Topic: foo2
Relative: foo

#
head
base
diff
date
summary

0
ad3f6a57
f2ff784b
diff
Sep 14 8:02 PM
1 file changed, 1 insertion(+)

Reviews in this chain:
└https://github.com/sqliu343/revup/pull/1 My first revup foo
　└https://github.com/sqliu343/revup/pull/3 My second revup foo


fix: Load different files of same type

Closes: #118.

@lzinik As you can see, this repo has not been updated for 2 years. I invite you to my fork: https://github.com/cyntler/react-doc-viewer


Feature Request to add official support for ScyllaDB and ClickHouse

Describe what you want
Introduction
This feature request proposes adding official support for ScyllaDB and ClickHouse to the Drizzle Object-Relational Mapper (ORM). Integrating these NoSQL databases with Drizzle would significantly enhance its versatility and cater to a broader range of use cases, particularly for applications requiring:
High Scalability and Performance: ScyllaDB excels in handling large data volumes with low latency and high throughput, making it ideal for applications like IoT, gaming, and ad tech.
Real-time Analytics and Big Data Processing: ClickHouse's column-oriented architecture empowers efficient real-time data processing and analytics, valuable for domains like financial services, e-commerce, and big data workloads.
Benefits
Broader Adoption: Supporting ScyllaDB and ClickHouse would attract developers working with these popular databases, expanding Drizzle's user base and fostering a more vibrant community.
Improved Developer Experience: Drizzle's existing ORM abstraction and schema management tools would extend seamlessly to ScyllaDB and ClickHouse, offering developers a consistent and efficient experience across diverse database environments.
Streamlined Development Workflows: Integration with these databases would empower developers to leverage Drizzle's ORM capabilities for data access and manipulation in ScyllaDB and ClickHouse, streamlining development workflows and promoting code consistency.
Request
We kindly request the Drizzle development team to consider adding official support for ScyllaDB and ClickHouse. This would involve:

Implementing necessary drivers and connectors to enable seamless communication between Drizzle and these databases.
Extending Drizzle's ORM syntax and functionalities to accommodate the unique data models and query languages of ScyllaDB and ClickHouse.
Providing comprehensive documentation and tutorials to guide developers on using Drizzle with these NoSQL databases.
Conclusion

Adding support for ScyllaDB and ClickHouse would position Drizzle as a highly versatile ORM solution, empowering developers to leverage its capabilities with a wider range of database technologies. We believe this integration would significantly benefit the Drizzle community and broaden its appeal in the NoSQL development landscape.

ScyllaDB is NoSQL, so that's likely not happening in a very, very long time (likely never). As for ClickHouse, we can definitely consider it but it's not a priority right now.

@alexey-milovidov
look https://t.me/clickhouse_en/59205


[CTKD] all LE cases & BR Accepted cases

As CTKD over Classic requires encryped connections, they should be considered independent cases where the Classic pairing has been done.

Didn't notice there is a isort check...

Hi Abel, I think this PR is ready.

Friendly ping.


[Minor] Correct a typo and enhance the text of update iOS

This PR corrects a typo and enhances the text (by replacing a comma with a period) in the FAQ entry update iOS.

@Ein-Tim Thanks.


[Optimization] Accelerate dgl create block  by bypassing sanity check

/okay to test

/okay to test

/merge


Accessibility Permissions for app being tested

Description
i build an app that adds Vim bindings to the whole macOS. to test this currently the app needs Accessibility Permissions. i can't seem to find a way to make this happen with the current GitHub Action macOS environment.
Area for Triage:
Apple
Question, Bug, or Feature?:
Question
i have looked at the following issues but couldn't find a solution or confirmation that what i'm trying to achieve is not possible:

https://github.com/actions/virtual-environments/issues/1567
https://github.com/actions/virtual-environments/issues/1441
https://github.com/actions/virtual-environments/issues/650
https://github.com/actions/virtual-environments/issues/553

Virtual environments affected

[ ] Ubuntu 16.04
[ ] Ubuntu 18.04
[ ] Ubuntu 20.04
[x] macOS 10.15
[ ] macOS 11.0
[ ] Windows Server 2016 R2
[ ] Windows Server 2019

Image version
Version: 20210423.1
Expected behavior
allow Accessibility Permissions to the app being tested.
Actual behavior
Accessibility Permissions not allowed.
Repro steps
run an app that requires Accessibility Permissions, like for example an app that creates event taps. even taps creation will fail.

as per the similar issues reported above, i tried to enable the Accessibility Permissions with
sudo sqlite3 "/Library/Application\ Support/com.apple.TCC/TCC.db" "insert into access (service, client, client_type, allowed, prompt_count, indirect_object_identifier, flags) values ('kTCCServiceAccessibility', 'mo.com.sleeplessmind.kindaVim', 0, 1, 1, 'UNUSED', 0)"
or
sudo sqlite3 "$HOME/Library/Application\ Support/com.apple.TCC/TCC.db" "insert into access (service, client, client_type, allowed, prompt_count, indirect_object_identifier, flags) values ('kTCCServiceAccessibility', 'mo.com.sleeplessmind.kindaVim', 0, 1, 1, 'UNUSED', 0)"
in the first case i'm told the db is not writable; in the second that there's no such db. if i understand correctly the first case is due to SIP, which you are not planning to disable through some options. so i'm just trying to confirm that there is nothing i can do currently to run my tests through GitHub Actions (without i guess rewriting my app so that in some kind of "test" mode it doesn't create a tap event. most of my tests should be decoupled so that they don't require a tap event to be run).
thanks a ton.

Hello @godbout , as far as I know, it is not possible to enable Accessibility Permissions in runtime because Hosted runners have SIP enabled. These databases can be changed only with disabled SIP (System Integrity Protection).

hi @maxim-lobanov. thanks a ton for the swift answer! seems to confirm what i understood till now (first mac app here). may i just confirm with you that nothing that i could tick in Xcode (entitlements or anything else?) could help on this issue?

@godbout , unfortunately, I can't answer. I don't have much experience with macOS / iOS development, our expertise is mostly related to configuring macOS environment so you shouldn't trust my words 😄
I would suggest to duplicate this question to GitHub Community Forum, I think someone has already faced with this issue and can help

i like this. i'm getting 40 this year and i'm realizing nobody has a fucking clue what they're doing and nobody should be trusted. me included 😂
alright, thanks for the help. i'll check the forum. and i'll probably pull out the event tap from the tests anyways. doesn't have much use there. thanks again!

ok so jfi in case someone is having a similar issue, i ended up adding a Testing configuration, and a Swift Compiler Custom Flag as TESTING for this new configuration.
in my code, i now only activate the event tap if i'm not in TESTING configuration with:
#if !TESTING
    _ = EventTapController.init()
#endif

all my own unit test code is decoupled from Apple's API so it can be tested without interfering with the OS.
the GHA yml now calls the testing as: run: xcodebuild clean -scheme kindaVim -configuration Testing test

ah! a little update. i added UITests that are using the Accessibility API and i pushed to GitHub, before trying to find a way to exclude them from GHA. i was expecting them to fail due to the SIP thing, but they passed 😅 wondering, is there any reason for that? like Xcode has been added to the TCC database even with the SIP on?

ok, strange. tests passed this morning, don't pass anymore now, with:
Found 1 interrupting element:
1770
    t =     0.89s         Window at {{357.0, 183.0}, {461.0, 171.0}} from Application 'com.apple.accessibility.universalAccessAuthWarn'
1771
    t =     0.89s     Invoking UI interruption monitors for Window at {{357.0, 183.0}, {461.0, 171.0}} from Application 'com.apple.accessibility.universalAccessAuthWarn'

so it seems that for whatever reason this morning this went through and now i guess the Accessibility window popping up, which i was expecting from the start. closing again i guess.


build(sync): auto sync

add auto sync fork workflow

Wow, this is great thanks for the PR. Do you have any other changes? Otherwise, I'll merge now

You can merge now, thanks.


Getting access to HTTP header fields from the UserProfileDelegate update method

I'm using your CredentialsGoogleToken to sign in a user on a server, where the user signed in originally on an iOS mobile app using Google Sign in. Because my server operates somewhat independently of the mobile app, it's important for me to pass along a Google Sign in server auth code (https://developers.google.com/identity/sign-in/web/server-side-flow). This is used on the server to obtain a server-side refresh token, in addition to obtaining an access token. I think it makes sense for me to pass that server auth code up to the server as a HTTP header-- since its authentication related.
Thus, it would seem to make sense to add a parameter to your UserProfileDelegate update method that provides access to the HTTP headers. E.g.,
func update(userProfile: UserProfile, from dictionary: [String:Any] andHeaders:Headers)

I am sorry, I don't understand what you are trying to do here.
The update method updates the user profile with the data received from the authentication provider. Do you mean that Google sends an additional token to CredentialsGoogle in an HTTP header, and you want to pass this token to the application using UserProfile?

Thanks for your response. What you are saying is close to what I'm doing with a couple of differences:

I'm using CredentialsGoogleToken not CredentialsGoogle.
It's my iOS app that's sending the HTTP header. I have an iOS mobile app, which connects to my server. This app uses mobile-app side Google Sign In to generate an access token and a server auth code. What I want to do is to ship the server auth code up to my server in an HTTP header. However, if I use CredentialsGoogleToken the way it is currently, I don't have access to the HTTP headers (e.g., using the UserProfileDelegate) to obtain that server auth code.

The server auth code is exchanged for a refresh token on my server. That refresh token has to be stored in my server's database, so it can later be used to generate new access tokens. Hence my need for getting it to my server application.

So, you want to pass the headers of the original request (the request that contains the access token) to UserProfileDelegate, right?

Yes, that's what I want to do.

@crspybits I talked to our Swift@IBM security expert and in her opinion it is not a good idea. We don't understand why the server needs to communicate with Google, it seems that the right way to do that is from the client.
If you still want to pass an auth code to the server, you can write a middleware to extract the desired info from the request, and put that middleware right after the Credentials middleware.

On the We don't understand why the server needs to communicate with Google-- Communicating from the server to Google is the procedure recommended by Google for servers that need offline access to the client's data. See for example:
https://developers.google.com/identity/protocols/OAuth2WebServer#offline
https://developers.google.com/identity/sign-in/web/server-side-flow
http://stackoverflow.com/questions/30637984/what-does-offline-access-in-oauth-mean
In my case, I'm developing a server that enables Google users to authenticate users with other credentials (e.g., Facebook) to access their Google Drive info. So, that's a natural case where offline access is needed.

Hi @crspybits, I was the one that recommended that adding a header field to that delegate API might not be a good idea.
I completely understand your reason for wanting offline support from the token and this particular flow seems perfectly suitable for it. I should admit from what you had written before it seemed you wanted to refresh your client side token with the server side refresh token, which is not good practice, but that seems not to be what you were doing.
The main reason I had was that by making the API generic so that it supports this very specific flow is creating precedence for adding all types of cases specific to different providers. Adding a generic field also leads to lack of type safety and potential vulnerabilities down the line. I find that in general auth flows are very often customized by providers and supporting all specific cases out of the box is hard. In this case, this Google flow is a hybrid of implicit and authz code and so it fits into neither Credential or CredentialToken.
We should probably come up with something specific for Google that supports this specific flow.

Thanks, @gtaban. Please let me know if I can help out.

I was updating the Kitura versions for my project, and I'm looking back at this issue. @gtaban, has any progress been made with this? For the time being, I'm still working with my own /CredentialsGoogleToken fork but would rather get away from that.
Thanks,
Chris.

you can write a middleware to extract the desired info from the request, and put that middleware right after the Credentials middleware.

Hi @irar2 How can this be accomplish ? I need to get some info from the header request from various codable routes. Any help ? Thank you!

Hi, to get access to HTTP headers in Codable routes you can use a type-safe middleware.
See https://developer.ibm.com/swift/2018/06/05/type-safe-middleware/ for an intro and do join our Slack if you have questions!

Thanks @ianpartridge ! I have asked on Slack but still got no answers.
Gonna try it !


Rebuild Sparkeats Home

Requirements
Requirements are open for discussion.
TBD
Notes
This issue may be broken down into smaller parts.
Comment if you are interested in working on this issue.
References

https://eats.seesparkbox.com/refresh

@yosevu Is the goal to rebuild the hompage such that the root route will feature the design found on /refresh?

@yosevu Is the goal to rebuild the homepage such that the root route will feature the design found on /refresh?

Yes, that's the goal. So this actually depends on #398 to Add React Router and Routes. I will add some details to the issue.


Fix typo in file-structure.md

programss > programs

Docs Build status updates of commit 3c63b80:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/techniques/file-structure.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 633e35c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/techniques/file-structure.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


introduce findFiles2 API

The current findFiles API does not allow for the caller to incorporate default excludes (search.excludes and files.excludes) in addition to custom excludes. I currently don't have plans on finalizing this and having this replace findFiles in the near future, but copilot currently needs something like this to improve perf.
You can test it via:
	vscode.commands.registerCommand('extension.testFindFiles2', async (args) => {
		await vscode.workspace.findFiles2('**/deadends*', {
			exclude: '**/*.ts',
			useIgnoreFiles:true,
		}).then(r => r.forEach(item =>
			console.log(item.toString())));
	});

WIP, haven't written tests for this.

I like this and if this works well we should also target this for finalization. Looks like a small change that will help many extensions.
In addition to useIgnoreFiles  (or instead) we could have something like useConfiguredExcludes, e.g I am looking for a flag that makes the API behave just like file quick pick. So if a user/workspace runs file quick pick over node_modules or so than that's their choice and extension should have a simple/default way to play along

In addition to useIgnoreFiles  (or instead) we could have something like useConfiguredExcludes, e.g I am looking for a flag that makes the API behave just like file quick pick. So if a user/workspace runs file quick pick over node_modules or so than that's their choice and extension should have a simple/default way to play along

I think, by default, this API and findTextInFiles can just set useDefaultExcludes:true and it will already follow the excludes that you see in the editor. This is because the *ignoreFiles arguments will already use the configured excludes if not set. Interestingly, although findTextInFiles says that useDefaultExcludes:true is default, it isn't (see this issue https://github.com/microsoft/vscode/issues/204179). To me, it makes the most sense for both APIs to have useDefaultExcludes:true so that setting none of the useDefaultExcludes/*ignoreFiles settings assumes the normal quickpick behavior.
I'll probably bring this to the API sync because I have questions regarding how best to go about a 'first draft' of this API such that it can eventually be finalized.


cannot mix top level fields from two different graphql servers

So i was using hasura graphql engine for my backend and lately they have introduced a feature called schema stitching that combines remote graphql schemas, and i ran into a problem when i tried to mutate from the secondary stitched schema. all the queries and mutations from the primary schema work perfectly is it me or has anyone ran into this?

This sounds like an issue with the server configuration – from the client's perspective it should all act like a single service

This sounds like an issue with the server configuration – from the client's perspective it should all act like a single service

closing as I'm fairly sure this is a server side issue. If you still think it's client side, come by #support in the discord, but you're probably better off finding support on hasura itself

closing as I'm fairly sure this is a server side issue. If you still think it's client side, come by #support in the discord, but you're probably better off finding support on hasura itself


Some problems

I'm trying to use the following command:
sudo hcxdumptool -i eth0 -w capture.pcapng --active_beacon --enable_status=1

but --active_beacon and --enable_status=1 options are unrecognized. Then I removed them but I got a new error:
Requesting interface capabilities. This may take some time.
Please be patient...

failed to get interface list

3 errors during runtime

bye-bye

How can I fix it? Thanks.

hcxdumtool does not work on wired interfaces like eth0. You must select a WiFi interface.
"--active_beacon" and  "--enable_status" are deprecated and hey have been removed.
Starting with the release v6.3.0 (05.05.2023), hcxdumptool uses different options.
In other words, the syntax has changed.
Please see the built-in help.
Just type "hcxdumptool -h" and press Enter.
Or type "hcxdumptool --help" to get more information like a typical command line:
 $ hcxdumptool -i INTERFACENAME -w dumpfile.pcapng -F --rds=1
   i     : name of the interface to be used
   w     : name of file to which packets are written
   F     : use all available channels
   rds=1 : sort real time display by status (last PMKID/EAPOL on top)

Looks like the WiFi adapter is not visible for KALI:
https://unix.stackexchange.com/questions/626328/are-usb-wi-fi-adapters-compatible-with-kali-in-wsl2

Closed this report, because it is related to WSL and/or KALI in combination with WSL.
Please notice (from README.md):
Unsupported: Windows OS, macOS, Android, emulators or wrappers!
and WSL is Windows.


[flutter_tools] add --dart-define option for fuchsia build

Description
add ability to use --dart-define for fuchsia build
Related Issues
Didn't open an issue for trivial changes.
Tests
I added the following tests:
Didn't add tests for trivial changes
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: Replace with a link to your migration guide

@jonahwilliams @zanderso please have a look at this. This is just a trivial change so that I can set -Ddart.vm.product=true

Ddart.vm.product=true should only be set by the flutter tooling when we're targeting release mode. Is this not working correctly?

Also - yes we should add dart-defines! Thanks for getting started with the PR here.
It does need to be tested, let me take a look at the existing build_fuchsia code and get back to you

Ahh, just looked - seems we're not passing the same arguments to the fuchsia kernel compiler that we are to the others:
https://github.com/flutter/flutter/blob/ba57ea09d17a30c05d6771cffdb6d10250b2f359/packages/flutter_tools/lib/src/fuchsia/fuchsia_kernel_compiler.dart#L73
The fix for release mode would probably be to change this line to dart.vm.product instead of release

This is also where you would need to add the dart-defines from the buildInfo class. Something like:
for (final Object dartDefine in dartDefines)
  '-D$dartDefine',

In the list of command line arguments

Oops, but it seems that there are no arguments passed to the kernel compiler command here
[  +51 ms] Building Fuchsia application...
[   +1 ms] executing: /home/michaellee8/flutter_fuchsia_toolchain/flutter/bin/cache/dart-sdk/bin/dart
/home/michaellee8/flutter_fuchsia_toolchain/flutter/bin/cache/artifacts/flutter_runner/flutter/arm64/release/aot/dart_binaries/kernel_compiler.snapshot --target flutter_runner --platform
/home/michaellee8/flutter_fuchsia_toolchain/flutter/bin/cache/artifacts/flutter_runner/flutter/arm64/release/aot/flutter_runner_patched_sdk/platform_strong.dill --filesystem-scheme main-root --filesystem-root
/home/michaellee8/flutter_gallery --packages main-root:///.packages --output build/fuchsia/flutter_gallery.dil --component-name flutter_gallery --aot --tfa --no-embed-sources -Ddart.vm.release=true
-Ddart.developer.causal_async_stacks=false main-root:///lib/main.dart

btw if I am building from another directory (let say ~/flutter_gallery), how do I make sure that my changes in the Flutter SDK will be reflected in the resulting binary? It seems that my app is not working even I hard-coded kReleaseMode to true. It just keeps breaking here
[00471.805] 102361:102540> [INFO:component.cc(312)] Using snapshot without framework for fuchsia-pkg://fuchsia.com/flutter_gallery#meta/flutter_gallery.cmx
[00477.213] 102361:102540> [WARNING:flutter/shell/common/persistent_cache.cc(190)] Could not acquire the persistent cache directory. Caching of GPU resources on disk is disabled.
[00477.213] 102361:102540> [WARNING:flutter/shell/platform/fuchsia/flutter/engine.cc(309)] Requesting intl Profile
[00477.213] 102361:102540> [INFO:persistent_cache.cc(338)] PersistentCache::UpdateAssetPath:
[00477.216] 102361:102540> [ERROR:flutter/shell/platform/fuchsia/flutter/platform_view.cc(583)] Platform view received message on channel 'flutter/isolate' with no registered handler. And empty response will be generated. Please implement the native m
[00477.216] 102361:102540> essage handler.
[00477.306] 102361:102540> [WARNING:flutter/shell/platform/fuchsia/flutter/engine.cc(304)] Sending LocalizationPlatformMessage
[00477.555] 102361:102715> [ERROR:flutter/lib/ui/ui_dart_state.cc(157)] Unhandled Exception: NoSuchMethodError: The getter 'properties' was called on null.
[00477.555] 102361:102715> Receiver: null
[00477.555] 102361:102715> Tried calling: properties
[00477.555] 102361:102715> #0      DiagnosticableNode.getProperties (package:flutter/src/foundation/diagnostics.dart:2943)
[00477.555] 102361:102715> #1      TextTreeRenderer._debugRender (package:flutter/src/foundation/diagnostics.dart:1238)
[00477.555] 102361:102715> #2      TextTreeRenderer.render (package:flutter/src/foundation/diagnostics.dart:1116)
[00477.555] 102361:102715> #3      FlutterError.dumpErrorToConsole (package:flutter/src/foundation/assertions.dart:775)
[00477.555] 102361:102715> #4      FlutterError.reportError (package:flutter/src/foundation/assertions.dart:907)
[00477.555] 102361:102715> #5      SchedulerBinding._invokeFrameCallback (package:flutter/src/scheduler/binding.dart:1076)
[00477.555] 102361:102715> #6      SchedulerBinding.handleDrawFrame (package:flutter/src/scheduler/binding.dart:1013)
[00477.555] 102361:102715> #7      SchedulerBinding.scheduleWarmUpFrame.<anonymous closure> (package:flutter/src/scheduler/binding.dart:822)
[00477.555] 102361:102715> #8      _rootRun (dart:async/zone.dart:1180)
[00477.555] 102361:102715> #9      _CustomZone.run (dart:async/zone.dart:1077)
[00477.555] 102361:102715> #10     _CustomZone.runGuarded (dart:async/zone.dart:979)
[00477.555] 102361:102715> #11     _CustomZone.bindCallbackGuarded.<anonymous closure> (dart:async/zone.dart:1019)
[00477.555] 102361:102715> #12     _rootRun (dart:async/zone.dart:1184)
[00477.555] 102361:102715> #13     _CustomZone.run (dart:async/zone.dart:1077)
[00477.555] 102361:102715> #14     _CustomZone.bindCallback.<anonymous closure> (dart:async/zone.dart:1003)
[00477.555] 102361:102715> #15     Timer._createTimer.<anonymous closure> (dart:async-patch/timer_patch.dart:23)
[00477.555] 102361:102715> #16     _Timer._runTimers (dart:isolate-patch/timer_impl.dart:398)
[00477.555] 102361:102715> #17     _Timer._handleMessage (dart:isolate-patch/timer_impl.dart:429)
[00477.555] 102361:102715> #18     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:168)

Oops, but it seems that there are no arguments passed to the kernel compiler command here
[  +51 ms] Building Fuchsia application...
[   +1 ms] executing: /home/michaellee8/flutter_fuchsia_toolchain/flutter/bin/cache/dart-sdk/bin/dart
/home/michaellee8/flutter_fuchsia_toolchain/flutter/bin/cache/artifacts/flutter_runner/flutter/arm64/release/aot/dart_binaries/kernel_compiler.snapshot --target flutter_runner --platform
/home/michaellee8/flutter_fuchsia_toolchain/flutter/bin/cache/artifacts/flutter_runner/flutter/arm64/release/aot/flutter_runner_patched_sdk/platform_strong.dill --filesystem-scheme main-root --filesystem-root
/home/michaellee8/flutter_gallery --packages main-root:///.packages --output build/fuchsia/flutter_gallery.dil --component-name flutter_gallery --aot --tfa --no-embed-sources -Ddart.vm.release=true
-Ddart.developer.causal_async_stacks=false main-root:///lib/main.dart

Oh my bad here, I thought that  -Ddart.vm.release=true is due to my setting.

The arguments are the flag list here: https://github.com/flutter/flutter/blob/ba57ea09d17a30c05d6771cffdb6d10250b2f359/packages/flutter_tools/lib/src/fuchsia/fuchsia_kernel_compiler.dart#L51

@jonahwilliams I wrote the code to add those flags into the kernel compiler, also refactored it to allow easier testing. Please kindly have a look at it.

I think the remaining test does not have much to do with my patch? Are there any further problem?

@jonahwilliams Please have a look at this, I have apply the fixes as requested except the contains one.

Thank you for fixing this @michaellee8 !

Umm are there any formatter config I can use? I just use Crtl+Shift+I in VS Code which should be using dartfmt.

Unfortunately we don't use autoformatting for the flutter/flutter repo. The style guide is outlined here: https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo
But general, format in way that is readable, don't go too long, and intent things 2 spaces instead of 4

@jonahwilliams Oh i can get it. Are there any problems left?

Hey please don't merge this right now. Let me revert all the bad things caused by auto formatter first.

All damages are cleaned and looks good to merge now. LGTM


Use the Publishing API database for everything

Drop the dependency on the Content Store database.
Update 2024-01-17: The Content API database might be deprecated.
The Publishing API database has some advantages:

Includes everything that has content, whether or not it has its own
route (URL), such as role appointments that don't have their own page
but do appear inside other pages.
Aligns with GOV.UK's policy of using Postgres wherever possible, and
removing MongoDB.  We could rewrite our MongoDB queries as SQL to use
the postgres implementation of the Content Store directly (instead of
via MongoDB), but if we're going to do that, why not rewrite them to
use the Publishing API database?
Is structured better for linking between things, because the links are
in separate tables rather than being part of each document.

And some disadvantages:

It is much bigger, so the nightly backup isn't available as early in
the day, and it takes longer for us to process, so users might only
see content as it was two days ago instead of one day ago.  This delay
might soon be lowered by an optimisation in GOV.UK's backup process
(lower level of compression).
Hundreds of thousands of pages haven't been rendered from Govspeak
into HTML, so we would have to do that in order to extract HTML
elements such as plain text and URLs.

Being done in other pull requests.


Merge pull request #2 from vsouza/master

update

Project URL

Description

Why it should be included to awesome-ios (optional)
Checklist

[ ] Only one project/change is in this pull request
[ ] Addition in chronological order (bottom of category)
[ ] Supports iOS 9 / tvOS 10 or later
[ ] Supports Swift 3
[ ] Has a commit from less than 2 years ago
[ ] Has a clear README in English

          1 Error

:no_entry_sign:
Found 2 link issues

Link issues by awesome_bot

Line
Status
Link

1082
301
https://github.com/MerrickSapsford/Listenable redirects tohttps://github.com/msaps/Listenable

1476
301
https://github.com/MerrickSapsford/Pageboy redirects tohttps://github.com/msaps/Pageboy

  Generated by :no_entry_sign: Danger


arm builds

Would it be possible to publish arm builds for k8dash. Right now the image is x86 only and I want to use it on my raspberry pi 4 cluster. I know how to do this manually and would be willing to investigate how to automate it. https://engineering.docker.com/2019/04/multi-arch-images/

I've had some trouble with node-sass, but for the moment it works well on my Pi 3 cluster.
https://hub.docker.com/r/kimdiallo/rpi_k8dash
But arm builds directly from here would be very welcome.

Thanks for the leg work @kimdiallo. Any chance you'd like to provide a PR?

@kimdiallo I was just able to deploy your rpi version, unfortunately when I expose it with an Ingress it asks for Basic Auth. Did you set up some kind of standard user-name or password? Can this be disabled for login with a token?

@danielsatanik I don't think that depends on the build. Try to follow these instructions[1] and start the pod again.
[1]https://github.com/herbrandson/k8dash#service-account-token

@herbrandson Sorry, i have never noticed your request. So far, unfortunately, this has been very makeshift and built directly on a real Pi. I think with a dockerfile for a reasonable cross build would be the better option. If there is no great hurry, I would be happy to help.

@danielsatanik I don't think that depends on the build. Try to follow these instructions[1] and start the pod again.
[1]https://github.com/herbrandson/k8dash#service-account-token

@kimdiallo I followed the instructions but unfortunately it still asks for authentication. Something else you can think of?

@danielsatanik The only difference that comes to my mind is, that i run this as a NodePort, not with ingress...

@herbrandson Sorry, i have never noticed your request. So far, unfortunately, this has been very makeshift and built directly on a real Pi. I think with a dockerfile for a reasonable cross build would be the better option. If there is no great hurry, I would be happy to help.

There's no hurry, so that would be great if you can!

Just in case I'm already on it since I'm dying for a good dashboard for my Cluster to get started with my little cross amd64/armhf/arm64 cluster that I've setup with k3sup.
I'm using Docker 19.03 with buildx as outlined here: https://mirailabs.io/blog/multiarch-docker-with-buildx/
I'm also throwing out node-sass (js wrapper ontop of c++ code that needs to compiled for each target platform) replacing it with sass (pure js implementation I'm using in production) since that thing increases the build time more then 3 fold, also requiring python 2.7/gcc dependencies ontop.
I'll track my progress here (currently not up-to-date just for future reference):
https://github.com/thiscantbeserious/k8dash

Thanks @thiscantbeserious! Super excited to see how it goes :)

well, just to keep you updated:
https://github.com/ayufan-rock64/linux-build/blob/master/recipes/binfmt-misc.md
Make sure you're using a kernel above 4.8 if you're going for the binfmt_misc route - otherwise you won't be able to run (a good way to verify if binfmt_misc is actually working or not):
docker run --rm -it arm64v8/busybox echo Hello World

There you go I successfully published an multiarch-image for armv6, armv7, arm64 and amd64. I didn't test it yet tought. Do you want me to create a simple shellscript for the build?
https://hub.docker.com/r/youcantbeserious/multiarch-k8dash/tags

So I'm close to finalizing my adjustments, after some trendemous cross-platform testing and walzing trough the docs.
I added a Travis config, which will build automatically for each commit and tag it with the revision and latest tag, pushes both to Docker Hub automatically and in the end also does a Vulnerability-Check with trivy.
Trivy also scans the base-container, not just npm and provides an additional layer of security - which discovered multiple high risk exploits in the alpine-image that's been previously used, so I also switched that out with node:lts-alpine instead.
Travis will require you to setup the Repository variables HUB_USER and HUB_TOKEN (the API-Token you should generate on Docker-Hub, NOT your Password).
Example Travis log:
https://travis-ci.com/thiscantbeserious/k8dash/builds/149215576
Tags on Docker Hub:
https://hub.docker.com/r/youcantbeserious/multiarch-k8dash/tags

Ping @herbrandson PR is up for Review 🙉

Thanks for this @thiscantbeserious ! I'll work on reviewing this weekend.

Hello,
I enjoy using Skooner on x86_64, but I have trouble on my raspberry pi 4.
I get this :
standard_init_linux.go:228: exec user process caused: exec format error
I think the image is not OK for ARM Rpi.
Are there some updates to be able to run skooner on ARM archs ? (or the image tags to use ?
Regards,

Am I missing something in order to run on arm64? When pulling this on a raspberry pi 4 (dpkg reports aarch64) I still encouner an exec format error. This happens when applying the deployment described in https://raw.githubusercontent.com/skooner-k8s/skooner/master/kubernetes-skooner.yaml
Do I need to specify a special tag for the image?

@moll-re Did you figure out how to do this?  I'm just trying a Raspberry Pi install of Skooner and having the same issue.

I found this commit which removed arm64 and arm7 image builds but provides no explanation as to why this was done: https://github.com/skooner-k8s/skooner/commit/76f6a1c4e7dd6d45f85a3a75634bf513087814f6


small text fix on gene pages

Coordinates of predicted trans-membrane domains:
->
Coordinates of predicted transmembrane domains:

I've changed the text. It should be in tomorrow's update.


Etcd Registry clarity - Self hosted Kubernetes

I am unable to understand or find the /registry directory within the self hosted example, where the api server keeps all its state. Perhaps this is due to container linux etcd client being 2.3.7 or have i misunderstood.
cat ./etcdctl.sh
export ETCDCTL_CERT_FILE=/opt/bootkube/assets/tls/etcd-client.crt
export ETCDCTL_KEY_FILE=/opt/bootkube/assets/tls/etcd-client.key
export ETCDCTL_TRUSTED_CA_FILE=/opt/bootkube/assets/tls/etcd-peer.crt
export ETCDCTL_CA_FILE=/opt/bootkube/assets/tls/etcd-ca.crt
export ETCDCTL_PEERS=https://node1.example.com:2379

source ./etcdctl.sh

etcdctl ls --recursive
/coreos.com
/coreos.com/updateengine
/coreos.com/updateengine/rebootlock
/coreos.com/updateengine/rebootlock/semaphore

Expected to see "/registry.../"

This seems to be my own generic misunderstanding with K8 as I get the same issue with self hosted etcd-operator


Fixes #1648

As this will need lots of small changes, I'm opening PR now, and then work through this PR with several commits.

No, it doesn’t

@mholo65 Ah, didn't read your PR description correctly.


Use UEFI's locate_protocol to replace find_protocol

Fixes #79
This change lets users avoid allocating when they just want to find a single implementer of a protocol.

Thanks for adding this functionality


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1230 ms

Resolved: Bofrost.at performance has improved in 1e9b68c.


Change types of id column and sequence for large tables

Is your feature request related to a problem? Please describe.
At least for the extra annotations table we still have the integer type of the id column as integer. This is problematic when importing too often as this will overflow.
Describe the solution you'd like
Change this to bigint. This can be done in a migration just like variants/migrations/0067_smallvariant_seq_id_bigint.py.
Review all other tables and ensure that the type is big enough everywhere.
Describe alternatives you've considered
N/A
Additional context
N/A

This is something that should be backported to the athenea branch.


Support dApp Providers (Bitcoin Provider)

[ ] Create provider core

[ ] Support Types
[ ] Create provider interaction interface

[ ] Create test pages
[ ] Support Bitcoin features

[ ] GetInfo
[ ] GetAddresses
[ ] Sign Transaction
[ ] Sign input (Pbst)
[ ] Inscribe (text, images)
[ ] Send inscription

Version test : http://<IP_ADDRESS>:8900/s/dQTrt3APHH3RFiw
Link dApp test : https://42n2p7.csb.app/

[ ] Can't click button Approve

Version test : http://<IP_ADDRESS>:8900/s/BApc7PB5DkDzQMH

[ ] Error page when click View detail with case sign typed data
Link dApp : https://ex-sub-connect-substrate.vercel.app/

[ ] Don't send request when click Sign typed data v4

Version test : http://<IP_ADDRESS>:8900/s/Wr7ggyfKfwR4LFR

[ ] Don't show transaction time in wallet when use sendTransfer method

[ ] Re-check message details when perform sign Psbt

Version test : http://<IP_ADDRESS>:8900/s/NjmspTzdJE7m6r8

[ ] Re-check information when input account address invalid with psbt hex

=> Related to issue #228

Version test : http://<IP_ADDRESS>:8900/s/NjmspTzdJE7m6r8
Validation passed in case :

Install/Update version
Connect dApp BTC with some methods :
Connect
Sign message
Sing Psbt
sendTransfer
Connect dApp EVM with some methods :
Connect
Sign message
sendTransfer


Implement Redux for the ReactPlugin

At the moment the PlaygroundList component of the ReactPlugin is a massive beast that takes care of everything. It's like a top level global state management thingy – oh wait, there's better solutions for this? :wink:
I think we've discovered the point where integrating Redux starts making sense and everything easier to handle. Would appreciate a PR implementing this!

I'm on it :)


Auto-detect dark threshold

Often as in #418 the dark threshold is incorrect. Printing the darkness values for my camera it seems very trivial to detect a reasonable threshold. Would a contribution be accepted that automatically configures this value if it can be detected?
Here are the darkness values for my camera where you can see the dark frames as consistently >= 99.1:
[82.9427]
[80.70182]
[80.00607]
[99.18099]
[80.11067]
[99.19748]
[80.238716]
[99.23134]
[80.18143]
[99.235245]
[80.00174]
[99.275604]
[79.90408]
[99.26085]
[79.7895]
[99.283424]
[79.90191]
[99.29383]
[79.90885]
[99.29037]
[79.99002]
[99.28993]
[80.08941]
[99.2934]
[80.0395]
[99.28386]
[80.16363]
[99.306854]
[80.44922]
[99.323784]
[80.54948]
[99.3368]
[80.60547]
[99.33507]
[80.609375]
[99.32422]
[80.64192]
[99.33594]
[80.72048]
[99.34201]
[80.932724]
[99.3776]
[80.99089]
[99.390625]
[81.04297]
[99.40972]
[81.08637]
[99.43837]
[81.118484]
[99.442276]
[81.131516]
[99.42751]
[81.35113]
[99.407555]
[81.5217]
[99.38759]
[81.73394]
[99.35156]
[81.86154]

I would definitely accept the contribution but i'm not sure it's "very trivial" to set the threshold automatically.
Some cameras have 2 light frames and 1 dark, some have 1 light frame and 2 dark. Both of our cameras alternate. Camera exposure can vary greatly, what might be a good threshold in one setting might not be the right one in another. What if the user is lit by a non-emitter light source when they run the command? What happens if someone runs this on a visible light camera accidentally?

Since the IR emitter should emit at a relatively constant brightness and dark frames should be relatively constant if you plot the points it most cases there should be two very distinct clusters. We can just do something simple to identify the darker cluster.
More specifically, one idea is we can just split the points into two groups based on the midpoint between the 10th and 90th percentile darkness value. Then if the darker group has a very low range and standard deviation relative to the midpoint we can be pretty confident in the dark threshold being accurate. In the case someone runs it on a regular camera the range of the dark points relative to the midpoint would be very large so we would know to skip setting the dark threshold automatically.
Do you happen to have a flashing IR camera? If so, could you share your darkness values so I can have at least one other sample to test on before submitting a PR?


ClientSetup Enhancements - Parameterize script to allow for additional flexibility

Checklist

[X] I have verified this is the correct repository for opening this issue.
[X] I have verified no other issues exist related to my request.

Is Your Feature Request Related To A Problem? Please describe.
Through customer interaction these are the nice additions we feel would be well received to edit within ClientSetup

Allow entry of Passing domain credential for agent installation
Allow toggle of ContextMenu on or off with extension install
Make adding additional packages to ClientSetup easier
Make adding additional chocolatey sources easier

Describe The Solution. Why is it needed?
These enhancements would make the ClientSetup script more flexible by providing parameters to the script to support more customizable installs.
Additional Context
No response
Related Issues
No response

This could be done with additional parameters:
[Parameter()]
[Hashtable[]]
$AdditionalPackages,

[Parameter()]
[Hashtable[]]
$AdditionalConfiguration

We could then just act on it at the end of ClientSetup:
if($AdditionalPackages){
  #action here
}

if($AdditionalConfiguration){
  #Action here
}

If doing AdditionalPackages can I suggest that we trigger the Jenkins job (like we do in Azure / Ansible) rather than adding further logic?
See: https://github.com/chocolatey/c4b-azure/blob/main/source/Initialise-C4B.ps1#L174-L184

If doing AdditionalPackages can I suggest that we trigger the Jenkins job (like we do in Azure / Ansible) rather than adding further logic?
See: https://github.com/chocolatey/c4b-azure/blob/main/source/Initialise-C4B.ps1#L174-L184

This is on the client side. When setting up a workstation with C4b this would allow you to add additional things to be installed, on top of what is predefined in the ClientSetup.ps1.  At this stage, trigger the jenkins job doesn't quite make sense?

Ah, very true - for some reason I was thinking of that being similar to the additional packages in Azure. Disregard me!


Add keybind error notifications

This PR adds error toast notifications for keybinds.
I'm not convinced all of these messages are necessary, but I figured it would be easier to add them now and discard changes as necessary. I don't feel that strongly either way as to which cases we should provide error messages for; one would assume users would generally know what keybinds they set themselves should do, but unintentional keypresses don't seem that likely either and/or would have unintended consequences anyway.

Very well done. I love that the error messages (except one) are informative and tell the user what went wrong, rather than just saying "FAILED". Just one small suggestion, and we're good to go.


:bug: wrong image name in docs for rpi

https://kairos.io/docs/installation/raspberry/#customizing-the-disk-image

fixed in https://github.com/kairos-io/kairos-docs/pull/138