Test Console

Discussed in https://github.com/jonathanpeppers/dotnes/discussions/39

Originally posted by charlesbetros August 23, 2024
I think a console app that runs the transpile task might be useful. I wrote a really simple one for my local dev. We could do some assembly output comparisons for basic acceptance tests. I haven't put much thought in to selecting a project to "run".

FYI I took this even further with:

#45

> dotnet tool install -g dotnes.compiler --add-source .\bin\Debug\ --prerelease
You can invoke the tool using the following command: dotnes
Tool 'dotnes.compiler' (version '0.1.1-alpha') was successfully installed.

Then you can just run it via:
> dotnes
Usage: dotnes.exe mydll.dll chr_generic.s


Add timestamp for when last updated

Need: The dashboard updates regularly (hourly?). As a user it us unclear when the last update was so I am unsure if dashboard errors are current or if it has not yet updated since a recent metadata/upload fix.
Acceptance criteria:  A visible timestamp saying when the dashboard was last updated

I see now that this is present on the home page, but not on each centers page. Closing.


Different note styles between player and CPU

What feature do you want to get added on the base engine?
Can you make add note style depending of the character, like in Bos and Bosip mod ? We could edit it in the character creator menu, like health icon and color bar.

Just use the Arrow Skin in the song's chart smh -_-

https://github.com/ShadowMario/FNF-PsychEngine/discussions/995

Just use the Arrow Skin in the song's chart smh -_-

I don't think he means that


NuGet restore - Unexpected token =

Required Information
Question, Bug, or Feature?
Type: Bug
Enter Task Name: NuGet restore - Unexpected token =
Environment

Server - Azure Pipelines
I'm not exactly sure, but I think this is Azure Pipelines

Account Name: Agility PR Solutions
Team Project Name: agilitypr
Build Number: 10416

Agent - Hosted or Private:
The build definition's agent pool suggests that it's using a private agent.

Agent: Private
Agent OS: Windows_NT
Agent OS Version: 10.0.14393
Agent Version: 2.115.0

Issue Description
It would appear that any changesets we check-in, fails at the nuget restore step with the error "##[error]Unhandled: Unexpected token ="

Checks-in were successful previous day and no configuration changes were applied.
Task logs
I've enabled the "system.debug" = true; I think these are the logs.
nuget_restore_task_log.txt
Error logs
nuget_restore_pipeline_log.txt
EDIT 1: added task logs with "system.dbug" enabled.
EDIT 2: updated agent details; discovered pipes were using default (private) agent pool, not Azure Pipelines.

@setzuiro, could you set the environment variable "system.debug" = true, run the pipeline again and send the more detailed log? This will allow us to more accurately debug the issue.
A possible way to unblock you would be to add the Nuget Tool Installer Task in your pipeline before the Nuget Command task. Here is a link to the docs for the task: https://docs.microsoft.com/en-us/azure/devops/pipelines/tasks/tool/nuget?view=azure-devops .
Please let me know if this workaround works for you. More detailed logs of the failure would be much appreciated.

@apluche, thanks for the suggestion. I've enabled the system.debug and added the logs to the original post. Let me know if additional logs are required.
I've added the Nuget Tool Installer Task to my pipeline before my NuGet Restore task but I unfortunately still get the same error.

Thanks for the quick turnaround and the logs.
Are you using the VS 2019 hosted agent?

@apluche Thanks for the quick responses, really appreciate it.
I realized that the pipeline builds were configured to use what appears to be private agents; so I've update the original post. Apologies for the misinformation and lack of better understanding on the matter.

According to the System capabilities of the agent pool, these are the configurations; appears to be VS 2015? We're using VS 2017 to push the check-in through TFS.

First occurrence of task failing started at approx 2:39 p.m UTC today.

I'll investigate and try to figure out how to apply your suggestion for the nuget authenticate task.
Thanks

Hi @setzuiro,

Those agents are VS 2015 agents. You can update the agents to VS 2019 agents using the upgrade agents button in Project Settings -> Agent Pools panel. This may fix the original issue as it sounds like a script incompatability.
We pushed out a new version of the task this morning and the version in the logs indicates that you are using the new version.

Please let me know if updating the agents or using the nuget authenticate task unblocks you!

Hi @apluche,
Thanks so much for the continued efforts. I have added the Nuget Authenticate task to run before the Nuget Restore and that has indeed unblocked us for the time being.
Unfortunately, when we try to update the agents, we can see a successful request being made in the network panel, but there's no feedback otherwise and the agents aren't updated.
We'll proceed with the nuget authentiate task as a workaround for the time being, but I think it'd be better to update the agents too.
Please advise, and thank you very much!

Hi @setzuiro ,
Please look at this documentation to see how to update the agents: https://docs.microsoft.com/en-us/azure/devops/pipelines/agents/agents?view=azure-devops&tabs=browser#agent-version-and-upgrades .
Additionally, the Nuget Authenticate task is meant to be used without then using the nuget restore task. You can run a script and invoke nuget directly once you have run the Authenticate task.
Based on this, you can take out the Nuget Authenticate task and try again. We have been releasing new versions of the task in response to a livesite issue - which is likely to have fixed your issue.
Please let me know if you are unblocked.

Hi @apluche,
Thanks for your continued efforts.
As you suggested, I have removed the Nuget Authenticate task and retried with success.
The tasks are now working in their original states again and we have been unblocked.
Thanks again, appreciated the assistance and quick turn around.
I'll close the issue now.


Problem with installation with composer

Hello,
I've tried to install your project to my server but this does not work:
mstrd308@srv03:~ $ php composer.phar require jkroon/google-vision
Using version ^1.0 for jkroon/google-vision
./composer.json has been created
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Installation request for jkroon/google-vision ^1.0 -> satisfiable by jkroon/google-vision[1.0.0].
    - jkroon/google-vision 1.0.0 requires zendframework/zend-hydrator ^2.3 -> satisfiable by zendframework/zend-hydrator[2.3.x-dev] but these conflict with your requirements or minimum-stability.

Installation failed, deleting ./composer.json.
mstrd308@srv03:~ $

What am I doing wrong?
Regards,
mstrd308

Hey @mstrd308. Thank you for contributing.
It seems that the latest Zend Hydrator version this library is using has been pushed back to development. Version 2.3 was included, but marked as "dev". I've revoked the version to 2.2.
Let me know if it's all working now. Any feedback is welcome.


(PUP-6156) Implement access operator for Iterator type

The AccessOperator implementation for the Iterator type was missing.
This commit adds that and a test to verify that it works.

CLA signed by all contributors.


provide a way to disable each extension

Signed-off-by: Augustin Husson<EMAIL_ADDRESS>

:+1:


[mgmt] cosmosdb stable release

https://github.com/Azure/sdk-release-request/issues/5206

API change check
APIView has identified API level changes in this PR and created following API reviews.
@azure/arm-cosmosdb

@jeremymeng do you know why i didn't have merge permission for this pr?

Finally I used merge without waiting to merge this.


Make slippage bounds on gamm CLI be optional instead of required

Actually, this is a bad idea until we have MEV resistance


The requested pixel format 'W10' is not natively supported by the Linux Kernel

Hello I hava a D415 Depth Camera and it won't connect to my laptop.
The main error for this is mentioned below.
I can access the realsense viewer without any problems but when ever I turn my node on by using
roslaunch realsense2_camera rs_camera.launch
it gives this error. Please help me thank you
My kernel version is 4.15.0-60-generic
An exception has been thrown: The requested pixel format 'W10 ' is not natively supported by the Linux kernel and likely requires a patch!
Alternatively please upgrade to kernel 4.12 or later. Last Error: Invalid argument
terminate called after throwing an instance of 'rs2::backend_error'
what():  The requested pixel format 'W10 ' is not natively supported by the Linux kernel and likely requires a patch!
Alternatively please upgrade to kernel 4.12 or later. Last Error: Invalid argument
[camera/realsense2_camera_manager-1] process has died [pid 15139, exit code -6, cmd /opt/ros/kinetic/lib/nodelet/nodelet manager __name:=realsense2_camera_manager __log:=/home/hong/.ros/log/68c59866-ce8c-11e9-8e37-a86daaec59a0/camera-realsense2_camera_manager-1.log].
log file: /home/hong/.ros/log/68c59866-ce8c-11e9-8e37-a86daaec59a0/camera-realsense2_camera_manager-1*.log

Have the same issue. Any update? Thanks!

Have the same issue. Any update? Thanks!


Cleaning and/or solving the frontend tests (warning/logs)

What does it do?
It tries to remove all the warnings in the unit tests and e2e tests
Why is it needed?
to have proper logs when running the tests
How to test it?
you can run both the tests with these commands
Try to remove the warnings, prevent errors and warnings from the:
unit tests, you can run them with the command
yarn test:watch
E2E tests, you can run them with the command
yarn test:e2e
Related issue(s)/PR(s)
It is related to this ticket https://strapi-inc.atlassian.net/jira/software/c/projects/DX/boards/9?modal=detail&selectedIssue=DX-465

In the case of elements like RawTable, whilst we're using prop-type they will fail because of this: https://github.com/facebook/react/issues/6653#issuecomment-215793856
They're not necessarily fixable imo, I don't think certain props should have a default because then the warnings aren't flagged for users 🤔 not sure your thoughts on this?


descriptorsets.cpp: Unnecessary code

In function render()
if (cubes[0].rotation.x > 360.0f)
cubes[0].rotation.x -= 360.0f;
is not needed as trigonometric functions handle the periodicity inherently
if (cubes[1].rotation.x > 360.0f)
cubes[1].rotation.x -= 360.0f;
is also not needed because the rotation is around Y-axis and the trigonometric functions handle the periodicity  inherently.

Although the trig functions will handle values > 360, if you let it run long enough that those values become very large you will start to see numerical inaccuracies appear. The code here is necessary to prevent that.

In the second half of that code
cubes[1].rotation.y += 2.0f * frameTimer; 			if (cubes[1].rotation.x > 360.0f) 				cubes[1].rotation.x -= 360.0f;
Rotation is around Y axis, but X axis is handled

This may not be the most optimal way of doing this, but I' fine with the current code. So closing this.


Syntax issue

Hello,
Thanks for your module, i've tried your config file but i got some syntax issues.
I've change ";" by "," when necessary and it works now :
Here my config file :
{
module: 'MMM-NetworkScanner',
position: 'top_left',
config: {
devices: [
{ macAddress: "1a:1b:1c:1a:1b:1c", name: "Server", icon: "server"},
{ macAddress: "2a:2b:2c:2a:2b:2c", name: "Desktop", icon: "desktop"},
{ macAddress: "3a:3b:3c:3a:3b:3c", name: "Laptop", icon: "laptop"},
{ macAddress: "4a:4b:4c:4a:4b:4c", name: "Laptop", icon: "laptop"}
],
showUnknown: false,
showOffline: true,
keepAlive: 300,
updateInterval: 5
}
}

Great pick up @plumcraft  - I've updated the example in the ReadMe file


'Replace selection' reduces selected range

Issue Report Checklist

[x] Searched the issues page for similar reports
[x] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[x] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[ ] Could not reproduce inside jupyter qtconsole (if console-related)
[ ] Tried basic troubleshooting (if a bug/error)

[x] Restarted Spyder
[ ] Reset preferences with spyder --reset
[ ] Reinstalled the latest version of Anaconda
[x] Tried the other applicable steps from the Troubleshooting Guide

[x] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
When replacing a simple string in selected text with Replace selection, Spyder changes the amount of text selected.
What steps reproduce the problem?

Open a new Python file (or use temp.py)
Write five lines with # Spyder is not great
Select the five lines
Press Ctrl-R and enter not  and `` in the text boxes
Press Replace selection

What is the expected output? What do you see instead?
I would expect that Spyder keeps the selection of the five lines, but after the replacement operation, just  four lines and part of the fifth line (# Spyder is g) is selected.
 We have confirmed your bug and we'll try to solve it soon.

Thanks @steff456 for finding the cause (and @goanpeca and @ccordoba12) for dealing with this so swiftly.
I applied the patch on my version of spyder 4.1.3 and can confirm that it works (but that was expected, wasn't it).


chore(deps): update module github.com/gruntwork-io/terratest to v0.41.3

This PR contains the following updates:

Package
Type
Update
Change

github.com/gruntwork-io/terratest
require
patch
v0.41.0 -> v0.41.3

Release Notes

gruntwork-io/terratest
v0.41.3
Compare Source
What's Changed

Fix project-name so it doesn't contain special characters for docker-… by @hongil0316 in https://github.com/gruntwork-io/terratest/pull/1203
fix formatting and imports by @ellisonc in https://github.com/gruntwork-io/terratest/pull/1202

Full Changelog: https://github.com/gruntwork-io/terratest/compare/v0.41.2...v0.41.3
v0.41.2
Compare Source
What's Changed

support buildkit for dockcer-compose by @hongil0316 in https://github.com/gruntwork-io/terratest/pull/1201

Full Changelog: https://github.com/gruntwork-io/terratest/compare/v0.41.01...v0.41.2

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, click this checkbox.

This PR has been generated by Renovate Bot.

:tada: This PR is included in version 1.10.5 :tada:
The release is available on:

GitHub release
v1.10.5

Your semantic-release bot :package::rocket:


fix: deep link analytics event triggered twice

This PR resolves https://www.notion.so/artsy/Deeplink-opened-analytics-event-called-twice-13bcab0764a080b78390cf08dbc51759?pvs=4
Description
This PR fixes an issue where deep links calls are getting triggered twice. This was happening because I forgot to check if the navigation is ready before handling the deep link.

Before
After

PR Checklist

[x] I have tested my changes on iOS and Android.
[x] I hid my changes behind a feature flag, or they don't need one.
[x] I have included screenshots or videos, or I have not changed the UI.
[x] I have added tests, or my changes don't require any.
[x] I added an app state migration, or my changes do not require one.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

fix deep link analytics event triggered twice - mounir

Need help with something? Have a look at our docs, or get in touch with us.

This PR contains the following changes:

Dev changes (fix deep link analytics event triggered twice - mounir - MounirDhahri)

  Generated by :no_entry_sign: dangerJS against c1442982240f3cb020895f8f801790bd0272dd73


Patches/record ordering

@jtwiest thanks for this PR! In other places where 'order' configuration gets turned into SQL, something like this is done:
if ($options['order']) {
    $query .= ' ORDER BY '.join(',', static::_mapFieldOrder($options['order']));
}

This provides for applying field mapping and SQL generation when the ['Key' => 'ASC', 'OtherKey' => 'DESC', 'MAX(One, Two) DESC'] order config syntax is used

@themightychris I'm using this in the relationship declaration specifically to do to this:
       'Transactions' => [
            'type' => 'many-many',
            'class' => Transaction::class,
            'linkClass' => StatementTransaction::class,
            'linkLocal' => 'StatementID',
            'linkForeign' => 'TransactionID',
            'order' => 'Related.Time ASC'
        ],

Is this good to get pulled in though? I'm overriding that function in a few places already.

@jtwiest that's fine, the _mapFieldOrder function knows to pass a full string through as-is if that's what you give it, but anywhere in ActiveRecord that supports an 'order' option should still be getting run through that function. Please add before we merge

@themightychris think I addressed your concern, this is now ready for review again.


For Dev Dependency consideration: RubyFMT

Given there's currently a move to clean through rubocop violations,
Thought I'd flag RubyFMT which I've been hearing about on the Yak Shave.
It isn't a static code analyzer but is a formatter more in the style of eslint i believe.
Not tried it on a PR just yet, as the author is saying it's nearly there but not ready for production projects just yet.
Perhaps one to watch, if interested?

All for more robots. Is it as well as rubocop, or instead of, do you think?


Guile: Ensure that module is set before evaluation

Using the Guile socket client, I noticed that my evaluations were not always getting evaluated in the correct context (module).  It was using the BufEnter autocmd, switching the module using the Guile REPL's ,m command when the buffer changed.  It seems that this is not reliable, and sometimes it would fail to find the correct context and switch to the default module (guile-user) instead.  I think it might be somehow related to the presence of the log/HUD buffer, but I'm not sure at all.
This commit strips out all of the BufEnter stuff, and instead makes a separate ,m command immediately before every evaluation.  The module name from opts.context in eval-str.
Whole file evaluations (eval-file) are run in the default module ((guile-user)), but if they begin with (define-module ...) then the end result is still correct: definitions within the file end up in the correct module.
The Racket client uses the same BufEnter system, and therefore almost certainly suffers from the same problem.
Extra discussion:
I tried to use the Guile REPL ,in command (e.g. ,in (my module) (stuff to evaluate)), but unfortunately this command requires the expression to be all on one line.  In the future, I'd like to have the Guile client inject a small "middleware" which can more elegantly handle evaluation of an expression within a specific module, and solve various other problems at the same time.  For example, the default REPL remembers all the evaluation results (so that they can be recalled with $1, $2 etc), but this makes it possible to run out of memory fairly easily.  Definite link to #365 here.

This looks good! I introduced conflicts but I'll handle them.


Error certificate when create connetction on "Managed Service for PostgreSQL" yandex cloud

When I create a new connection to the "Managed Service for PostgreSQL" database for use in collections, I select the root certificate; it is not used for the connection. The following error appears.
connection to server at "***********.mdb.yandexcloud.net" (158.XX.XX.XX), port 6432 failed: could not open certificate file "/root/.postgresql/postgresql.crt": Permission denied
Despite the fact that the US service is connected to the same database using a similar connection string METADATA_POSTGRES_DSN_LIST="postgres://{user}:{password}@{host}:{port}/{database}?sslmode=verify-full&sslrootcert=/certs/ root.crt"
I tried to upload the root certificate into the container, changed the rights to the root directory (for testing), this helps to find the certificate, but then I get another error:
connection to server at "***********.mdb.yandexcloud.net" (158.XX.XX.XX), port 6432 failed: could not open certificate file "/root/.postgresql/postgresql.key"

Thanks, was able to reproduce an error, will be fixed in future releases.


I don't notice the pauses when I'm in fullscreen (OSX)

So if an application is running in fullscreen mode on OSX (such as a video, web browser, or emacs) when stretchly pauses trigger they do not show up over the application. It also doesn't steal the focus from the application so basically the notification goes entirely unnoticed. Since I'm probably in full screen most of the time, this makes it so that stretchly barely helps me at all.

I've tried with atom and firefox and works fine for me. What browser or video player do you use?
Also, are you on 0.10.0? I've released it yesterday.

I think it happens with both firefox and chrome, though I suspect it has been chrome most of the time. By fullscreen, I mean when you click the button that moves the window to its own fullscreen desktop sort of. Not just maximizing it so that it takes up the most space. Not just the way you get it by alt-clicking the green button.
I haven't yet tried 0.10.0, I installed it earlier though!

This was an issue in previous version. Should be fixed in 0.10.0.

I still have the same problem in 0.10.0

I can also just move to a separate virtual desktop in general and the window stays behind.

It works fine for me. Maybe some another app interferes or something? How do you run those apps?

Just clicking the green button that makes OSX create like a separate virtual desktop to run the app in fullscreen. So you can switch between things with control+arrowkeys. When I run it like that the stretchly window doesn't show up. My current workaround has been to make stretchly window also go fullscreen which steals away focus from what I'm doing. But that's fairly annoying. But at least I don't miss out on the break.
I'm not sure what's wrong. Could there be a permission or something that you need to grant stretchly before it can show up above a fullscreen window?

Ah, what is that green button? Not sure if we do the same thing. What I do is cmd+shift+f or the third button:

Do you have one app that way or more? I've tried only with one. I don't know about any permission needed.

Yea the third button. I guess it's not green in your colour scheme :)
I'll try cmd+shift+f later to confirm that it's the same.
I think it's usually more than one app in fullscreen. I may be that I have observed the same thing with only one app but not 100% sure.

Ok so cmd+shift+f is not the same as clicking the third button. I do suspect that you'd get the popup if you do the cmd+shift+f fullscreen but not if you click the third button.

Let's have some testing scenarios:

open chrome and go fullscreen with button, swipe to normal screen, open firefox and go fullscreen - breaks shows up
open chrome and go fullscreen with button, swipe to normal screen, open firefox and go fullscreen, swipe to chrome - break shows

Hm weird now I can't even reproduce my own issue that reliably did not work. Can't tell what the bug is then. Maybe some weird edge case in OSX?

I guess so. Let's close it for now and reopen when we have something reproducible?

Yep! Or at least re-open if it occurs again.


[DRAFT]: CLI v2

What
This is a revamp of k3d´s CLI
Why

CLI command should be testable
Replace viper with koanf to not include packages that are not needed
Provide a graceful shutdown of cobra to cleanup created resources in case of interruption or error
Cleanup current implementation and move business logic parts to separated packages

HI @dtomasi , are you still working on this?

Closing this for now. Feel free to reopen it once you're back on it :)


change module resolution to node on CJS flavor

remove dynamic imports

Closing PR as it is not possible to do what we tried.


feat: allow for overriding data variables and lookups at runtime

Fixes #472
This updates the compiler so that it includes a setLookup function in generated C and JS models, and then updates the runtime packages to allow for supplying lookup overrides via a new options parameter for ModelRunner.runModel.
See this issue comment for more details on the changes included here.  TL;DR this mostly adds new functionality, and there should be no impact on existing projects.
/cc @ToddFincannonEI

I resolved one last minute issue with the runtime-async tests, otherwise I think this is ready to be merged.  As with the other recent branches and PRs, Todd won't have time to review this before it gets merged, so we will merge but then keep the changes queued up on main before we publish official packages.


Failure to build kirkstone

First, Joel, thank you for publishing this layer.
I am trying to build depthai-core on the kirkstone branch with bitbake, but getting a failure.
Full disclosure: I am not a Yocto expert.
Here are some of the key lines of the log surrounding the failure:
FAILED: libdepthai-core.so.2.20.2
...
/home/travis/src/ros/yocto-common-2/tmp/work/skylake-64-oe-linux/depthai-core/2.20.2-r0/recipe-sysroot-native/usr/bin/x86_64-oe-linux/../../libexec/x86_64-oe-linux/gcc/x86_64-oe-linux/11.3.0/ld: /home/travis/src/ros/yocto-common-2/tmp/work/skylake-64-oe-linux/depthai-core/2.20.2-r0/recipe-sysroot/usr/lib/libarchive_static.a(archive_entry.c.o): relocation R_X86_64_32 against `.rodata.str1.1' can not be used when making a shared object; recompile with -fPIC
/home/travis/src/ros/yocto-common-2/tmp/work/skylake-64-oe-linux/depthai-core/2.20.2-r0/recipe-sysroot-native/usr/bin/x86_64-oe-linux/../../libexec/x86_64-oe-linux/gcc/x86_64-oe-linux/11.3.0/ld: failed to set dynamic section sizes: bad value

collect2: error: ld returned 1 exit status
...
ninja: build stopped: subcommand failed.
WARNING: exit code 1 from a shell command.

Full log attached here: log.do_compile.820910.log
Here is the output of bb -e depthai-core: yocto-e.log
Do you have any suggestions?

I haven't tried it on x86_64, only aarch64.  Hmm.  You might start with adding this to the depthai-core recipe:
EXTRA_OECMAKE += " -DCMAKE_POSITION_INDEPENDENT_CODE=ON"

Also be sure you're aware of this one: https://github.com/jwinarske/meta-luxonis/issues/3
If it's enabled it will clean up things needed to link.  I haven't solved this just yet

I looked over Issue #3 before I logged this one. I did not have rm_work in my local conf. (I was able to reproduce that issue when I added rm_work to my local conf, but that is not my issue.)
According to bb -e depthai-core, I found:
PACKAGECONFIG="shared examples tests opencv resource-compile"
So, it looks like it contains "shared" as you suggested.
Grepping through your layer, it looks like this line turns on position-independent code if PACKAGECONFIG contains "shared":
${@bb.utils.contains('PACKAGECONFIG', 'shared', '-DCMAKE_POSITION_INDEPENDENT_CODE=ON', '-DCMAKE_POSITION_INDEPENDENT_CODE=OFF', d)}

During all of my attempts so far, I have been building from the tip of your kirkstone branch (2394f5a). I haven't modified your recipe yet. So, I don't think I have attempted to override PACKAGECONFIG in any way.
Just for kicks, I tried adding your suggested line to depthai-core_2.20.2.bb at line 150, just after the block of EXTRA_OECMAKE.
It failed the same way. I also ran it with -e and found that PACKAGECONFIG was the same as before:
PACKAGECONFIG="shared examples tests opencv resource-compile"

I think I've fixed it.
I added -D CMAKE_POSITION_INDEPENDENT_CODE=ON to the EXTRA_OECMAKE arguments in the libarchive-luxonis_git.bb recipe.
That built successfully on my machine, although I did get a QA warning...not sure how important it is.
WARNING: depthai-core-2.20.2-r0 do_package_qa: QA Issue: depthai-core-dbg: found library in wrong location: /usr/bin/luxonis/examples/.debug/libutility.so
depthai-core-examples: found library in wrong location: /usr/bin/luxonis/examples/libutility.so [libdir]

I sent you a PR for the kirkstone branche. I am curious if this still works for your system.

Resolved with https://github.com/jwinarske/meta-luxonis/pull/7
Merged (after forever; sorry!)


check alive child process when delete task

As there are still shim service lock exists, we may check the container whether has alive child process or not before really delete the task. If has, raise an error to the caller to remind the user to kill all the child processes. Then the delete method without {all: true} will not be stuck.
One shim service lock case in #3149 : ( I think there are may be other cases we have not find out. )
# we use busybox rootfs and /test1 to start container
root@demo:/opt/busybox.test# cat rootfs/test1
#!/bin/sh
sleep 100000&
while true; do
  wait || true
done

# start container test
root@demo:/opt/busybox.test# ctr run -d --rootfs ./rootfs test /test1
root@demo:/opt/busybox.test# ctr t ls
TASK    PID      STATUS
test    17974    RUNNING
root@demo:/opt/busybox.test# ctr t ps test
PID      INFO
17974    &ProcessDetails{ExecID:test,}
17995    -

# start container test-id with pid ns of container test
root@demo:/opt/busybox.test# ctr run -d --with-ns "pid:/proc/17974/ns/pid" --rootfs ./rootfs test-pid /test1
root@demo:/opt/busybox.test# ctr t ls
TASK        PID      STATUS
test        17974    RUNNING
test-pid    18067    RUNNING
root@demo:/opt/busybox.test# ctr t ps test-pid
PID      INFO
18067    &ProcessDetails{ExecID:test-pid,}
18088    -

# container test-id's init process is dead
root@demo:/opt/busybox.test# ctr t kill -s 9 test-pid
root@demo:/opt/busybox.test# ctr t ps test-pid
PID      INFO
18088    -
root@demo:/opt/busybox.test# ctr t ls
TASK        PID      STATUS
test        17974    RUNNING
test-pid    18067    STOPPED

# try to delete the task test-pid, it causes shim service stuck
root@demo:/opt/busybox.test# ctr t delete test-pid
^C
root@demo:/opt/busybox.test# ctr t ls
^C
root@demo:/opt/busybox.test# ctr t ps test
PID      INFO
17974    &ProcessDetails{ExecID:test,}
17995    -
root@demo:/opt/busybox.test# ctr t ps test-pid
^C
root@demo:/opt/busybox.test#

Signed-off-by: lifubang<EMAIL_ADDRESS>

:( I'd rather always kill all in containerd to make it correct by default.

OK, let's wait us make a decision in #3149 .

@lifubang - Is it by contract that a call to delete with un-exited tasks/exec's will forcibly close these processes instead of returning an error? If so I for sure got this wrong in the Windows shim and will submit a fix. Thanks


Bad translations (Simplified Chinese) inside VisualStudio for Mac.

Sorry to bring it here, but I couldn't find a way to feedback bad translations inside VisualStudio for Mac.
Hopefully someone with VisualStudio for Mac sees this?
I've found 3 bad translations so far:

In "Preference" > "Visual Style", where you translate the "Interface" of "User Interface Language" into "界面", which is correct. But you then translate the same "Interface" below into "接口" as in "programming interface", which is absolutely inappropriate. Something like "主题" would be appropriate, or just sticking to "界面" would be much better.
In "Preference" > "Visual Style" > "Interface", where you translate "dark" to "深色", which in Chinese means "deep color", is correct. But you then translate "light" into "轻度" which in Chinese means "mild" and has nothing to do with color/theme. It should be "浅色" which means "light color".
In the right click menu in the Solution area, you translate the "Finder" of "Show in Finder" into "查找器", which is machine translation. Apple's official translation for the Finder app is "访达".

That's what I've noticed so far. I don't know who to bring this up to. If any one of you knows who's responsible for this, would you forward this issue to him/her/them? Those bad translations are just too obvious to ignore.

Oh, I think I found it, just in "Help" >  "Provide a Suggestion". I am closing this issue. Sorry for the bothering.


Bootstrap4的中文版文档何时能出

中文版文档何时能出？
感觉有些例子不够详细，是否能再细化一下？

we don't maintain the translated versions of the documentation...we just link to them from our end. i'd suggest trying to get in touch with whoever is maintaining the chinese version of the docs (should ideally be identified somewhere there, perhaps in the footer?)

Thank you~
You can read Chinese!!! That's amazing!

You can read Chinese!!! That's amazing!

well, in fairness, google translate does ;)


File download results in empty file since v3.0.1

Downloading the log file results in an empty text file since v3.0.1.  I suspect this could have been introduced by bb9acb97b545d45a664a23c7087e837165c7ffe0.

Confirmed: reverting bb9acb9 fixes the issue.  Ideally we wouldn't have to ask for unlimitedStorage permission to generate a small text file, but I need to dig into the permissions model a bit to see if there's something less scary we can ask for.

I found a better solution that requires no permissions.  The filesystem API is completely unnecessary.


Nav should reflect Protocultural PLACE #HASHTAG structure

Header could be Protocultural PLACE #HASHTAG where PLACE and #HASHTAG are html select

Yes but also the hashtag might change when we do an event in a city again,
so it's more event specific
On Saturday, February 13, 2016, Clément Renaud<EMAIL_ADDRESS>wrote:

Header could be Protocultural PLACE #HASHTAG where PLACE and #HASHTAG are
select
—
Reply to this email directly or view it on GitHub
https://github.com/protocultural/protocultural.org/issues/22.

Actually I meant the nav. We should be able to browse 1) by city 2 ) by projects/hashtags

New layout

ok done


Invalid documentation for: weather.js and computers.js

They both have the UTF-8 BOM which means that when hubot tries to parse the documentation it finds it invalid.
Solution: Remove the 3 bytes at the start of the file

#37


[Feature] REP 54: Implement the HeadReady condition

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
This is one of the tracking items of the REP: RayCluster status improvement, the step 5 in the design doc: https://docs.google.com/document/d/1bRL0cZa87eCX6SI7gqthN68CgmHaB6l3-vJuIse-BrY
We should implement the new HeadReady condition by referring to the current utils.IsRunningAndReady
https://github.com/ray-project/kuberay/blob/a43217bd2864961ee188a134807df57fd06e0a77/ray-operator/controllers/ray/rayservice_controller.go#L1211-L1217
Once the condition is implemented, we should base on it to refactor the isHeadPodRunningAndReady function.
Use case
No response
Related issues
https://github.com/ray-project/enhancements/pull/54
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Hi @cchen777, would you like to take on this?

yes @rueian i'd like to take on this !

Closes #2261


Diagnose github release failure

All changed packages have been documented.

You can try these changes at https://cadlplayground.z22.web.core.windows.net/typespec-azure/prs/371/
Check the website changes at https://tspwebsitepr.z22.web.core.windows.net/typespec-azure/prs/371/


change path

PluginDir is /usr/lib/nagios/plugins, I think this notification should be located next to mail-host- and mail-service-notification in /scripts/

Thats right, thanks for the correction!


Make routes load on-demand, and be smart about which ones you keep around in sandbox

Same way we do migration loading.
The API should hint or even enforce the idea that the app sets its routes once and never while appspace data is present.

In-work code now loads routes in a callback. But this can still causes problems wrt #63

This is not actionable, and discussion are in other issues. Closing.


coreX : directories is not populated properly

Dir exist in ls, but not in backend
steps to reproduce and workaround.
dir with contents
c1.system("mkdir /var/run/ahem").get() # got error
c1.system("cat /var/run/redis.pid").get() # read some files on that dir
c1.system("mkdir /var/run/ahem").get() # no error

dir with 0 content
c1.system("mkdir /var/run/sshd/ahem").get() # error
c1.system("touch /var/run/sshd/hai").get() # no error
c1.system("mkdir /var/run/sshd/hmm").get() # no error

@maxux i believe you also got this issue

I can confirm, I have the same issue.

@zaibon i think it is better to merge this issue with #36

duplicate of #36


App crashes in Xcode

Thank you for building this boilerplate. I really appreciate the work you put into this.
Just some stuff I stumbled upon and should be updated in the docs:

Meteor requires to npm install before running meteor
react-native run ios threw an error message on the popup terminal window
npm run ios did not worked either, there is no such script in package.json.

I could start the app using Xcode. However, it only showed the launch screen and threw this error message after a while:
'NSInternalInconsistencyException', reason: 'bundleURL must be non-nil when not implementing loadSourceForBridge'
*** First throw call stack:
(0x18dc4afe0 0x18c6ac538 0x18dc4aeb4 0x18e6e37cc 0x1001427bc 0x100140a8c 0x100177ab0 0x100176cf8 0x100176ad8 0x1000e2d90 0x1000cc7a8 0x193de6e48 0x193ff337c 0x193ff8e24 0x19400d8b0 0x193ff60b8 0x18f7f4884 0x18f7f46f0 0x18f7f4aa0 0x18dbf942c 0x18dbf8d9c 0x18dbf69a8 0x18db26da4 0x193ddffc8 0x193ddac9c 0x1000ccb88 0x18cb3559c)
libc++abi.dylib: terminating with uncaught exception of type NSException
(lldb)

I can also see a type conversion warning in Xcode (I'm using 8.3.3):

Is there anything else you one has to do besides setting the IP address in the settings.js file?

Thanks for bringing these points up! Sorry you're running into issues. I haven't updated this in quite a while so some packages could be out of date. It's on my todo list to update this boilerplate this week.

All of this should be fixed in the latest update!

Thanks!


OmniLight3D Won't Cast Shadows Past a Distance of ~171 Units

Tested versions

Tested in and reproducible in: v4.3.stable.official [77dcf97d8], v4.2.stable.official [46dc27791], v4.1.stable.official [970459615]

System information
Godot v4.3.stable - EndeavourOS #1 SMP PREEMPT_DYNAMIC Thu, 15 Aug 2024 00:25:30 +0000 - X11 - Vulkan (Forward+) - dedicated NVIDIA GeForce GTX 1660 Ti (nvidia; 555.58.02) - Intel(R) Core(TM) i7-10750H CPU @ 2.60GHz (12 Threads)
Issue description
When casting a shadow with an OmniLight3D with a distance more than ~171 units from the object being casted on, and an omni_attenuation of 0, the shadow becomes too unclear to the point of it not being able to be seen. With an omni_attenuation greater than 0, the shadow becomes unclear with an even shorter distance from the object.

Steps to reproduce

Create a 3D scene.
Add a MeshInstance3D node as a child of root with its mesh being a PlaneMesh.
Add another MeshInstance3D node as a child of root with its position being (0,1,0), and its mesh being a BoxMesh.
Add an OmniLight3D node as a child of root with its omni_range being 300, its omni_attenuation being 0, its position being (0,171,0), and its shadow_enabled property toggled on.
Toggle off preview sunlight in the scene.

Minimal reproduction project (MRP)
godot-issue.zip

Related: https://github.com/godotengine/godot/issues/92551

Which version should this bug be patched by?

We don't know what's wrong yet and haven't identified a solution so we don't know yet

Wait, could it have anything to do with the limited size of shadow maps?


Post a notification when finishing moving the downloaded file successfully.

This PR is acceptable, but a test needs to be added to verify the behavior. If you can add them in the next few days these then it may be able to make the 4.0 release.

It seems the travis build failure is irrelevant with my changes, shall we skip that check result? BTW, please review my test case in the new commit, it passed for me.
@jshier


Item name will be returning into new lines on small screen devices in statistic tables

Environment
Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/75.0.3770.100 Safari/537.36
URL
https://penguin-stats-frontend.galvingao.now.sh/result/stage/main_1/main_01-05
Steps to reproduce

Navigate to the URL
Open DevTools of the browser
Resize the screen to let the width be smaller than 580px
Observe the text on the item

What is expected?
the text SHOULD NOT be returned into new lines
What is actually happening?
the text OCCURS TO returns into new lines

It seems the width of the particular field is not high enough. Adjustment is needed on such attribute.
Screenshot on such behavior, notice the red-boxed area:

stage code also has this issue:


correct set version information

also updates the graph types
both separately lgtm'd

/retest

/retest

network again
/retest

/retest

/hold
for rebump

https://openshift-gce-devel.appspot.com/build/origin-ci-test/pr-logs/pull/openshift_cluster-kube-apiserver-operator/264/pull-ci-openshift-cluster-kube-apiserver-operator-master-e2e-aws/1222
was known flakes


.net core  .net framework HttpWebRequest

class Program
{
static void Main(string[] args)
{
string msg = HttpConnectToServer("https://www.okex.com/api/v1/exchange_rate.do");
Console.WriteLine("Core:"+msg);
Console.ReadKey();
}
private static string HttpConnectToServer(string url)
{
//创建请求
HttpWebRequest request = (HttpWebRequest)HttpWebRequest.Create(url);
request.Method = "GET";
        //读取返回消息
        string res = string.Empty;
        try
        {
            HttpWebResponse response = (HttpWebResponse)request.GetResponse();
            StreamReader reader = new StreamReader(response.GetResponseStream(), Encoding.UTF8);
            res = reader.ReadToEnd();
            reader.Close();
        }
        catch (Exception ex)
        {
            return ex.ToString();
            //连接服务器失败
        }
        return res;
    }
}

The same code is a .net core One is a different .net framework results Why?

It seem is the https question. You may need set :
webRequest.AllowAutoRedirect = false;

Such like corefx#23422

.net core 2.1     Request failed
.net core 2.0     Request successfully

i think is better move this issue to https://github.com/dotnet/corefx repo

It was moved: https://github.com/dotnet/corefx/issues/31772


Windows equivalent of .incbin

Instead of doing the big data array approach when you don't have the .incbin directive, why not use a custom resource type and generate a resource file as well as a source file that loads the data from the resource?
You use a user-defined resource type to point the resource to the binary blob.  Each embedded binary gets its own resource id in the generated resource script and the generated C++ source file loads the resources and obtains pointers to their data.  The code would call FindResourceA to get the HRSRC for the blob and then LoadResource followed by LockResource to get the pointer to the data and SizeofResource to get the data length.
For large binary payloads, this avoids all the compiler processing work of C++ source files of large data arrays.

Note: one wrinkle with this is when the resource is embedded in a DLL, in which case the code that fetches the resources will need to make sure that it uses the result of LoadLibraryA on the DLL filename for the HMODULE parameter to the resource functions, or it won't locate the resources properly.

It's too complicated to support this feature because it involves integration with VS, or at the very least MSBuild. It gets complicated when the project already has a resource file and it requires generating some initialization code. I wanted to keep this as straight forward as possible. Ideally you'd have to do nothing but include this header (as is the case with any compiler but VS.) The reason the tool exists at all is mostly because I could not find a work around that was not intrusive like the resource approach. Still not happy with it but I generally view VS as a black sheep of the family of compilers.


Functions with dependencies are not updated

Describe the bug
When rendering content (ie text) my expectation was that i could put the expression into a function on the context object and have the content updated whenever the depencency (ie count) is changed.
To Reproduce
Intuitively I would expect both buttons to show the same text:

  <button l-text="text()" l-on:click="++count">0</button>
  <button l-text="count + ' hi'" l-on:click="++count">0</button>

If this is not intended or should be done any other way, it's probably because the documentation is a bit scarce...

@phoet I don't believe it is actually possible with our current structure to update the dependencies of functions. Use pure functions like this:

  <button l-text="text(count)" l-on:click="++count">0</button>
  <button l-text="count + ' hi'" l-on:click="++count">0</button>

This is possible if you have an external function that returns state

Fair enough.
Since the documentation has references to Vue, I was expecting some kind of similar functionality when it comes to methods used in bindings.
There should at least be a minimal example in the documentation that shows the inteded use.

Good point, the documentation doesn't reflect the intended use. I will update that. @phoet feel free to open a PR to update it yourself too.

I find it especially irritating, as the l-on:click handlers can be functions and update the state. so from my point, following the principle of least surprise, binding a method to l-text should always return an updated value as well.

Issue seems to be here specifically, need to somehow be able to compare value and view[key].toString() and see that it is accessing affected state. Although this is a temporary solution, we're currently working on a full vdom refactor under the vdom-refactor branch.

Fixed in ca44c2e953cce44fce28f9debf3db2bb0b88c18f


Paper CI

@geemi725 Can you fix the CI errors with paper job? Check the output on main.

Reminder on this @geemi725


[RDBMS] az mysql flexible-server create/update: Add --auto-scale-iops to enable or disable autoscale of iops

Related command

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

RDBMS

seems duplicate with #26178


Links on "Signing in with a passkey" page doesn't allow users to sign-in with a passkey

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
https://docs.github.com/en/authentication/authenticating-with-a-passkey/signing-in-with-a-passkey
What part(s) of the article would you like to see updated?
The links for "Navigate to the login page for GitHub.com at https://github.com/login." needs to include the ?passkey=true feature flag for the Sign-in with Passkey button to appear.
Additional information
This feature flag is (only) documented at https://github.com/orgs/community/discussions/54450

@jlaundry Thanks so much for opening an issue! I'll triage this for the team to take a look :eyes:


Crash in dart2js triggered by new CL.

I have a CL (https://dart-review.googlesource.com/c/sdk/+/244241/9) which causes a crash in dart2js (https://logs.chromium.org/logs/dart/buildbucket/cr-buildbucket/8814371261794806161/+/u/test_results/new_test_failures__logs_)
The error is
stdout:
 org-dartlang-sdk:///lib/async/future_impl.dart@42700+1:
 Internal Error: The compiler crashed when compiling this element.
 The compiler is broken.
 When compiling the above element, the compiler crashed. It is not
 possible to tell if this is caused by a problem in your program or
 not. Regardless, the compiler should not crash.
 The Dart team would greatly appreciate if you would take a moment to
 report this problem at http://dartbug.com/new.
 Please include the following information:
 * the name and version of your operating system,
 * the Dart SDK build number (build number could not be determined), and
 * the entire message you see here (including the full stack trace
   below as well as the source location above).
 The compiler crashed: 'package:compiler/src/ir/static_type.dart': Failed assertion: line 1034 pos 9: 'node.promotedType == null ||
             promotedType is ir.NullType ||
             promotedType is ir.FutureOrType ||
             typeEnvironment.isSubtypeOf(promotedType, node.promotedType,
                 ir.SubtypeCheckMode.ignoringNullabilities)': Unexpected promotion of VariableDeclaration(Object? source) in Arguments((source as _Future.T%, source{_Future.T%})). Expected TypeParameterType(_Future.T%), found InterfaceType(Object?)
 #0      _AssertionError._doThrowNew (dart:core-patch/errors_patch.dart:51:61)
 #1      _AssertionError._throwNew (dart:core-patch/errors_patch.dart:40:5)
 #2      StaticTypeVisitor.visitVariableGet (package:compiler/src/ir/static_type.dart:1034:9)

The code in question, if I interpret the offset correctly, is likely around the function:
_completeWithFutureOr(/* FutureOr<T> */ Object? source) {
    assert(identical(source as FutureOr<T>, source));
    if (source is Future<T>) {
      var nativeSource =
          source as Object; // Demote so we can promote to _Future.
      if (nativeSource is _Future) {
        var nativeFuture = nativeSource; // Prevent demotion from assignment.
        while (nativeFuture._isChained) {
          nativeFuture = nativeFuture._resultOrListeners;
        }
        assert(nativeFuture is _Future<T>);
        if (nativeFuture._isComplete) {
          return _setNativeFutureResult(nativeFuture);
        }
        assert(nativeFuture._mayAddListener);
        return _chainCoreFuture(nativeFuture);
      }
      _chainForeignFuture(source);
      return null;
    }
    assert(identical(source as T, source));
    return _setValue(source);
  }

(@42700 is at this position, right after the return type, which is _FutureListener?).
I'll probably keep tweaking the code, and I'll say if it goes away. If not, it's blocking.

That particular assertion has been firing on a variety of programs for a while now. We probably need to revisit how our internal type promotion aligns with the CFE's, but since compilation works fine with assertions disabled, we haven't prioritized it.

Since this code is intended to go into the SDK, it will likely mean completely disabling the assertion-enabled dart2js builders on the buildbot. (Either by marking everything as failing, or by disabling the builders entirely).
So, heads up, that may happen :)

@fishythefish I recall in the past we mentioned that we still needed it because it is more precise for unsound mode, but equally good for sound mode. Is there a hybrid that would make sense today? For example default to use the CFE promotion or bypass this check for null safe libraries?
/cc @johnniwinther

@sigmundch We've discussed this assertion in the past, but I don't have a great understanding of how our analysis compares to the CFE's. Our analysis was useful pre-null-safety, but some of the logic is now shared by the CFE, at least partly because the null safety spec mandated more type promotion. Our analysis may be able to produce narrower/more precise types than the CFE does (@rakudrama might know?), so whatever type we compute should be a subtype of what the CFE yields.
I think it makes sense to use the CFE's promoted type as the starting point for our analysis rather than the unpromoted type. However, I'm not sure whether the cases that trigger this assertion are situations where the CFE performs some type promotion that we could be performing but aren't, or simply bugs in our promotion logic. From discussion with @srujzs, I think there may be at least one bug related to JS interop.

Closing this since fffe5ce6ec59095ab632d7ab1c280b74cf9efd6c landed. We may still want to simplify static type computation in dart2js, but we can follow up on that separately.


Executing Procedure

Hello, so what I'm trying to do is executing a stored procedure in MSSQL2016. The procedure returns a JSON with the new function "FOR JSON PATH". The issue is that I can't bind input parameters like this.
var result string
	results, err := db.Query("EXEC CLIENT_GET ?", "CC")
	if err != nil {
		fmt.Println("Procedured failed..")
		fmt.Println(err)
	} else {
		fmt.Println("Procedured executed successfully")
	}

It keeps me returning this error:
Procedured failed..
mssql: Incorrect syntax near '?'.
2017/12/06 12:57:53 http: panic serving [::1]:55382: runtime error: invalid memory address or nil pointer dereference

i suggest you use 'sqlserver' scheme, and pass named parameter based on your store procedure parameters' name  , assuming your sp first input parameter named 'ID':
db, err := sql.Open("sqlserver", connectionString)

results, err := db.Query("CLIENT_GET", sql.Named("ID", 6))

Unfortunately for us named parameters aren't an option as we have to support an older version of go. Is there an alternative solution? Many thanks

When using ? syntax, I was seeing the following:
2018/02/27 15:44:18 	@p1	{ 1 <RetrieveDetailRequest>
	<Users StartTime="2017-12-12T00:00:00.000Z" EndTime="2017-12-13T00:00:00.000Z">
		<User<EMAIL_ADDRESS>		<User<EMAIL_ADDRESS>	</Users>
</RetrieveDetailRequest>}
2018/02/27 15:44:18 Incorrect syntax near '?'.```

As such, I was able to make it work with the following:

query := "exec dbo.PR_Retrieve_Data @XMLParam = @p1"
rows, err := db.Query(query, string(xmlParam))

Any feedback on the validity of my solution? Should I expect this to work another way?

Yes, using @pN syntax is valid.


So much to like!  💗 the screenshot!

Hi there!
I know this repo is a little old, but I stumbled onto it and wanted to say "Thank you" for taking the time to make a readme with a screenshot.  So many Python programs don't have a GUI so it's a real treat to see one one a Repo's readme.
A few things you may be interested in:

I like that the default for your program is that the GUI comes up :-)  Nice!
Since you wrote the code, there are 2 new abilities that can shorten your code that lines up the Input Elements

size or s parameter can be an int for Text elements.  s=12  is the same as   s=(12,1)
The Stretch element is available.  It's been aliased to be the Push element now

change_look_and_feel('Dark') is now part of the theme APIs so it would be theme('Dark')
One Line Progresss Meter!!! YES!  I just noticed you called it.  There's a PEP8 format for that now too. Non-PEP8 is OneLineProgressMeter and PEP89 is one_line_progress_meter`
JSON

This repo doesn't use them that I saw, but you may be interested in trying out the PySimpleGUI User Settings APIs
If you're using PySimpleGUI, they're already there and available to use
There is a function and and object version
As usual, there are demo programs and documentation on this feature

Here are an alternate layout that will give you the same alignment as before but without having to set a size.  It uses the Stretch element.  When 4.48.0 is released soon, I will be using Push instead of Stretch.  It's the same thing, just a different alias.  If will have a shortcut version P to be extra compact:

    layout = [[sg.Text('Enter 2 folders')],
              [sg.Text('Folder to copy'), sg.Stretch(),
               sg.Input(), sg.FolderBrowse()],
              [sg.Text('Folder where to copy'),sg.Stretch(),
               sg.Input(), sg.FolderBrowse()],
              [sg.Checkbox('Remove source directory', size=20, key='check1'),
               sg.Checkbox('Add a google map based on GPS data', key='check2')],
              [sg.Submit(), sg.Cancel()],
              [sg.T(f'PSG Version = {sg.version}')]]

If you wanted to use the same technique before, you can still use size but set it to an int.  I included the shortcut aliases s and k too for you to see.
    sg.theme('Dark')
    layout = [[sg.Text('Enter 2 folders')],
              [sg.Text('Folder to copy', size=15),
               sg.Input(), sg.FolderBrowse()],
              [sg.Text('Folder where to copy', s=15),
               sg.Input(), sg.FolderBrowse()],
              [sg.Checkbox('Remove source directory', size=20, k='check1'),
               sg.Checkbox('Add a google map based on GPS data', key='check2')],
              [sg.Submit(), sg.Cancel()],
              [sg.T(f'PSG Version = {sg.version}')]]

Keep building stuff!  It's always exciting to see what people build!

Hi, thanks for taking the time to comment!
I appreciate the feedback. I wrote this program as a part of a university intro to programming class, so I won't change the code here since I keep this project for reference. However, I've been using PySimpleGUI for a lot of other smaller projects and I've been really enjoying it, so I will for sure have a look at the new functionality (I especially like the JSON settings).
Also I have to mention, the cookbook and demo programs are fantastic. They make it so much easier to add a GUI to a script compared to other GUI libraries.
Keep up the great work!


Unknown CMake command "detect_cuDNN".

CMake Error at CMakeLists.txt:59 (include):
  include could not find load file:

    mshadowUtils

CMake Error at CMakeLists.txt:60 (include):
  include could not find load file:

    Cuda

CMake Error at CMakeLists.txt:61 (include):
  include could not find load file:

    mshadow

OpenCV found (C:/MXnet/opencv/build)
CMake Error at CMakeLists.txt:104 (detect_cuDNN):
  Unknown CMake command "detect_cuDNN".

Configuring incomplete, errors occurred!
See also "C:/MXnet/build/CMakeFiles/CMakeOutput.log".

download the subfolder mshadow , put it in the mxnet folder

@janelu9 thanks


merge two or more CSV files together

Problem Statement
merge two or more CSV files together

Hey Ravi,
Is there any condition that two files contains data in same format or the two file can totally be different?

@kushal98
consider for same columns length first
Then try with  different columns length


Documentation link not working

Is it just me or is the documentation link down? https://pulsar.gitbooks.io/vue-tour/

Hi @damiendawreck, this link seems to be dead, we're currently working on a new version of Vue Tour and the documentation.
Please refer to this one > https://pulsardev.github.io/vue-tour/


Add scenarios for checking sending messages to different channels, direct with checking of message fields

[ ] Add scenarios for checking sending messages with validation of message fields correctness
[x] Added new steps and etc. (if needed)

Time spent: 4h 52m 10.07.2019

Added checks of correctness by text and channel and also need to add checking incoming message structure by fields.

Now exists steps for check format and type of incoming message payload
Thanks dependabot from github auto-fix of security issue with lodash!


[@azure/openai-assistants] README Instructions Misleading for createRun Function: Issue with assistant.id Placement

Package Name: @azure/openai-assistants
Package Version: 1.0.0-beta.3
Operating system:
[x] nodejs

version: v18.13

[ ] browser

name/version:

[ ] typescript

version:

Is the bug related to documentation in

[x] README.md
[ ] source code documentation
[ ] SDK API docs on https://docs.microsoft.com

Describe the bug
The content of the README.md is incorrect.

To Reproduce
When calling createRun and following the instructions in the README file, I encountered an error. The code snippet I used is as follows:

It appears that assistant.id should be within the same object as instructions. If not, an error message is generated.

After placing assistant.id within the object, it functions correctly.

@Johnsonliuz Thank you for using the Azure OpenAI Assistants SDK, and reporting this. I've corrected the Readme to reflect the current createRun API.


Parallel download of packages

I am trying out vcpkg to check whether its suitable for building a huge dependency list we have.
As a test, I ran:
vcpkg install collada-dom
which goes and installs about 40 packages in total, many of them from boost.
It works beautifully. But is there a way of doing parallelization of package downloading and eventually proceed with the build?
Also, is each of these packages being compiled with all cores? Can I configure the number of cores I want to use for building packages?

How a package is build depends on the individual port, but in general,  the preferred mechanism is ninja, which runs compilation in parallel. Not sure if there is a way to override that behavior.
Downloading one package, while a different one is processed is -as far as I know - not supported at the moment. The difficulty I see there is that downloading, extracting, building and installing all happen as part of a single cmake file, so vcpkg probably doesn't have a mechanism to say "start downloading but wait until I give the signal to proceed with compilation".

While ninja generally uses all available cores, when installing a large number of packages there are a lot of times when only one core is used. It would be nice if unrelated packages could be installed in parallel.

Hi. I watched a twitch streamer trying to use vcpkg and boost (https://www.twitch.tv/garethhubball). Boost wanted to install 294 packages. Each package took about 2 secs meaning it took him 600 seconds or 10 minutes to install  the dependencies. IMHO, that would be too slow for my taste.

Well, most likely,  the stranger didn't actually need all of boost.  Still, it might be beneficial to have the ability to install all of boost in once go instead of by library.

Which reminds me: Do any non-boost packages actually make use of the modular boost structure in their dependency list?

AFAIK, vcpkg doesn't support parallel downloading, for one package, vcpkg builds it in parallel, however, it can't build several packages togather.


Login only with email not working

I'm trying to enable our users to login either with their username or email, but not both. I followed #126
and the username-only login works fine, but the email-only login fails with: "Unable to login with the provided credentials.". This fails both when using email and username_email as the config value of ACCOUNT_AUTHENTICATION_METHOD .
My account-related settings are:
ACCOUNT_USERNAME_REQUIRED = True
ACCOUNT_DEFAULT_HTTP_PROTOCOL = env(
    "BACKEND_ACCOUNT_DEFAULT_HTTP_PROTOCOL", default="http"
)
ACCOUNT_EMAIL_REQUIRED = True
ACCOUNT_EMAIL_VERIFICATION = "mandatory"
ACCOUNT_CONFIRM_EMAIL_ON_GET = True
ACCOUNT_UNIQUE_EMAIL = True

# Allows users to login either
# by username or email.
ACCOUNT_AUTHENTICATION_METHOD = "username_email"

I solved my problem by adding this to settings.py
ACCOUNT_USER_MODEL_USERNAME_FIELD = 'email'

Hi @ifigueroap , hopefully this issue had been solved. If not, I hope this can help you.
I encounter the same issue when I enable users to login with their username and email via setup the ACCOUNT_AUTHENTICATION_METHOD = "username_email".
After that, I try to do some research for this issue, finally, I fixed it by adding parameters to setting.py, like below:
ACCOUNT_AUTHENTICATION_METHOD = 'username_email'
ACCOUNT_EMAIL_REQUIRED = True
ACCOUNT_USERNAME_REQUIRED = True
ACCOUNT_EMAIL_VERIFICATION = 'optional'

AUTHENTICATION_BACKENDS = [
    'django.contrib.auth.backends.ModelBackend',
    'allauth.account.auth_backends.AuthenticationBackend',
]

Appreciate.

Anything new about this?


added brain color mastermind game

PR Description 📜
Please include summary related to the issue you have fixed and describe your PR in brief over here by specifying the issue number on which you were working below
Fixes #2195

The code is a four color code containing the colored buttons displayed.
The code may contain multiples of the same color.
Try to guess the correct code (color and order) by clicking on the colored buttons and then hitting the submit button.
Your guess will then be compared to the code.

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[x] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @SimranLahrani842 ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

You can directly add files without src folder, React needs src folder not js

Ok sir I ll take care

Thank you  @SimranLahrani842 , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀

Hey @SimranLahrani842,
You can't work on a closed issue 👀!
Follow the proper guideline and make a new PR again 😀.
Happy Hacking 💗


FN: No UNVERIFIED-CODE-CONTRACT alert for attack contract

BOT ID - 0x4c7e56a9a753e29ca92bd57dd593bdab0c03e762bdd04e2bc578cb82b842c1f3
Tx - https://bscscan.com/tx/0xce0e119a317a9e67f37f67e281c546b2cf7df3d4f37e4a660faf245461d49134

Unable to repro:
1 findings for transaction 0xce0e119a317a9e67f37f67e281c546b2cf7df3d4f37e4a660faf245461d49134 {
"name": "Contract with unverified code was created",
"description": "0x4b44b52ef15f7aab9a0a6cfdfa8d2c5eeeb6d8c3 created contract 0x1c651B04bDd1C1718EEeafedE0A3c13e87024fa2",
"alertId": "UNVERIFIED-CODE-CONTRACT-CREATION",
"protocol": "ethereum",
"severity": "Medium",
"type": "Suspicious",
"metadata": {
"address_contained_in_created_contract_1": "0x55d398326f99059fF775485246999027B3197955",
"address_contained_in_created_contract_2": "0x657334B4FF7bDC4143941B1F94301f37659c6281",
"address_contained_in_created_contract_3": "0x2120F8F305347b6aA5E5dBB347230a8234EB3379",
"anomaly_score": 0.0006720430107526882
},
"addresses": [],
"labels": [
{
"entityType": "Address",
"entity": "0x4b44b52ef15f7aab9a0a6cfdfa8d2c5eeeb6d8c3",
"label": "attacker",
"confidence": 0.3,
"remove": false,
"metadata": {}
},
{
"entityType": "Address",
"entity": "0x1c651B04bDd1C1718EEeafedE0A3c13e87024fa2",
"label": "attacker_contract",
"confidence": 0.3,
"remove": false,
"metadata": {}
}
],
"uniqueKey": "",
"source": {}
}

Bot health in terms of dropped tx looks OK. I suspect, the etherscan API ran out of quota, but we now have a paid endpoint, which should mitigate this issue.


Improve codegen for Closure<T>

This commit improves the codegen for Closure<T>, primarily for ZST
where the closure doesn't actually capture anything. Previously
wasm-bindgen would unconditionally allocate an Rc for a fat pointer,
meaning that it would always hit the allocator even when the Box<T>
didn't actually contain an allocation. Now the reference count for the
closure is stored on the JS object rather than in Rust.
Some more advanced tests were added along the way to ensure that
functionality didn't regress, and otherwise the calling convention for
Closure changed a good deal but should still be the same user-facing.
The primary change was that the reference count reaching zero may cause
JS to need to run the destructor. It simply returns this information in
Drop for Closure and otherwise when calling it now also retains a
function pointer that runs the destructor.
Closes #874

cc @upsuper, mind testing this out to see if it works for your use case?

Sorry for the delay. As I mentioned in #873 (comment) my main hope is to be able to remove allocation completely, which is probably not a goal worth pursuing with lots of effort for wasm...
And I actually tested this patch, but it seems to somehow break #864 again. I see the following when I try to run wasm-bindgen:
thread 'main' panicked at 'assertion failed: `(left == right)`
  left: `3`,
 right: `1`', crates/wasm-interpreter/src/lib.rs:391:9
stack backtrace:
   0: std::sys::unix::backtrace::tracing::imp::unwind_backtrace
   1: std::sys_common::backtrace::print
   2: std::panicking::default_hook::{{closure}}
   3: std::panicking::default_hook
   4: std::panicking::rust_panic_with_hook
   5: std::panicking::continue_panic_fmt
   6: std::panicking::begin_panic_fmt
   7: wasm_bindgen_wasm_interpreter::Interpreter::call
   8: wasm_bindgen_wasm_interpreter::Interpreter::interpret_closure_descriptor
   9: wasm_bindgen_cli_support::js::closures::rewrite
  10: wasm_bindgen_cli_support::js::Context::finalize
  11: wasm_bindgen_cli_support::Bindgen::_generate
  12: wasm_bindgen::main
  13: std::rt::lang_start::{{closure}}
  14: std::panicking::try::do_call
  15: __rust_maybe_catch_panic
  16: std::rt::lang_start_internal
  17: main

Oops the panic was a mistake and should be fixed now. This PR does remove the allocation if your closure doesn't actually capture anything (like you're calling a function pointer or only capture ZST variables)

Yeah, I can confirm that the allocation goes away if I pass in a closure with no capture or only ZST variable capture. Thanks!

Awesome, thanks!
r? @fitzgen


Bugs in examples/console_test/tools/analisys_csi_data?

In the examples/console_test/tools/analisys_csi_data.py line 65, the assumption is that the first byte of each channel response is the real number and the second one is the imaginary number. However this contradicts the statement in the ESP32 Wi-Fi Channel State Information documentation where it says "Each channel frequency response of sub-carrier is recorded by two bytes of signed characters. The first one is imaginary part and the second one is real part."  Can you please verify the which one is the correct in order to reconstruct the CSI complex data from the byte data? Thanks.

This issue has been fixed，Closing this issue, feel free to reopen with more updates. Thanks.


Re-enable Nexus proxy tests

That seems to have made the CI a bit flaky :| (latest master is red because of timeouts waiting for the nexus mirror to start)


Implements caching for QWATCH queries to improve performance

This PR implements a caching mechanism for QWATCH queries to improve performance and reduce load on the main shard store.
key changes

Added a queryCache to store data relevant to each watched query, reducing the need to scan the entire store on each update.

Modified the QueryWatcher to maintain these caches, updating them as keys are modified.

Updated the QWATCH command to build the initial cache when a query is subscribed to.

Implemented an AdhocQuery channel and handler to execute queries against the cached data.

Updated the ExecuteQuery function to work with the new caching system.

Modified tests to work with the new caching system and added cleanup steps.

Added new constants for SET and DEL operations.

These changes should significantly improve the performance of QWATCH queries, especially for large datasets, by reducing the amount of data that needs to be scanned on each update. The caching system ensures that only relevant data is stored and processed for each query. Additionally, we don't need to filter keys during query execution.
Note: This PR sets the groundwork for future multi-threaded query execution, though additional work will be needed to fully implement scatter-gather algorithms across multiple shards.

What a nice PR <3 Thanks @JyotinderSingh Sir!

Looks great @JyotinderSingh ! We can discuss where the CacheStore should be, but this is great for now.


[firebase_auth] Correct method signature for confirmPasswordReset

Description
The FirebaseAuthPlatform.confirmPasswordReset method should accept String app as an argument to be consistent with the rest of the interface (and existing implementations, particularly for firebase_auth_web).
cc: @lanmonster, @hterkelsen
Related Issues
#1778
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] If the pull request affects only one plugin, the PR title starts with the name of the plugin in brackets (e.g. [cloud_firestore])
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

I think that we should avoid changing the method signature of confirmPasswordReset because it would be a breaking change, and we are trying to avoid breaking changes to platform interfaces where possible. What I'd suggest instead is that we add app as an optional named parameter to confirmPasswordReset.

@collinjackson I understand where you're coming from from but confirmPasswordReset was only published less than a day ago (#1753), and as far as I can tell, has no active implementations yet. All the other methods on FirebaseAuthPlatform have app as a required argument. At what point to you folks say "this was overlooked and should be corrected"?

I agree with @bgetsug. I should have caught this in code review yesterday. Since this hasn't even landed in package:firebase_auth yet (just the platform interface), I think we should correct the mistake before it actually becomes a breaking change.


Update circe-core, circe-testing to 0.12.1

Updates

io.circe:circe-core
io.circe:circe-testing

from 0.12.0-RC4 to 0.12.1.
Release Notes/Changelog
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention @scala-steward in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.circe" } ]

labels: semver-pre-release

Codecov Report

Merging #52 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #52   +/-   ##
=======================================
  Coverage   45.45%   45.45%
=======================================
  Files           1        1
  Lines          44       44
  Branches        1        2    +1
=======================================
  Hits           20       20
  Misses         24       24

Continue to review full report at Codecov.

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


shell script truncates StringFlag, and affects other string flags

My urfave/cli version is
v2.11.2
Checklist

[ X ] Are you running the latest v2 release? The list of releases is here.
[ X ] Did you check the manual for your release? The v2 manual is here
[ X ] Did you perform a search about this problem? Here's the Github guide about searching.

Dependency Management

My project is using go modules.

Describe the bug
When using a shell script, string flags with spaces gets stripped, and affects other flags.
To reproduce
package main

import (
	"fmt"
	"log"
	"os"

	"github.com/urfave/cli/v2"
)

func main() {
	app := cli.NewApp()
	app.Flags = []cli.Flag{
		&cli.StringFlag{Name: "db-pass"},
		&cli.StringFlag{Name: "db-user"},
	}
	app.Action = action
	err := app.Run(os.Args)
	if err != nil {
		log.Fatalln(err)
	}
}

func action(c *cli.Context) error {
	fmt.Println("username", c.String("db-user"))
	fmt.Println("password", c.String("db-pass"))
	return nil
}

#!/bin/sh

DB_PASS='hello world'
DB_USER=user123

go run main.go \
  --db-pass=${DB_PASS} \
  --db-user=${DB_USER}

Observed behavior
When running the shell script ./test.sh:
username
password hello

Expected behavior
username user123
password hello world

Additional context
This only fails when using the shell script. Running it directly in the terminal like so behaves as expected.
DB_PASS='hello world' && DB_USER=user123 && go run main.go --db-pass ${DB_PASS} --db-user ${DB_USER}

It's also worth noting that this is only a problem when the StringFlag contains spaces.
If the issue relates to a specific open source Github repo, please
link that repo here.
If you can reproduce this issue with a public CI system, please
link a failing build here.
Want to fix this yourself?
We'd love to have more contributors on this project! If the fix for
this bug is easily explained and very small, free free to create a
pull request for it.
Run go version and paste its output here
go version go1.18.2 darwin/arm64

Run go env and paste its output here
GO111MODULE="on"
GOARCH="arm64"
GOBIN=""
GOCACHE="/Users/raymond/Library/Caches/go-build"
GOENV="/Users/raymond/Library/Application Support/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="arm64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/raymond/go/pkg/mod"
GOOS="darwin"
GOPATH="/Users/raymond/go"
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_arm64"
GOVCS=""
GOVERSION="go1.18.2"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/raymond/Documents/testcli/go.mod"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch arm64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/94/h0_6vpb97570slf3srtqd7cw0000gn/T/go-build3181733792=/tmp/go-build -gno-record-gcc-switches -fno-common"

Commenting with an update in case anyone else stumbles across this issue:
It turns out that modifying the shell script to wrap the variables in double quotes produces the desired results.
#!/bin/sh

DB_PASS='hello world'
DB_USER=user123

go run main.go \
  --db-pass="${DB_PASS}" \
  --db-user="${DB_USER}"


update powershell script to configure workspace

Azure PowerShell team is removing Switch-AzureMode, which will be a breaking change to create a solid foundation for the future of Azure PowerShell. The changes are as follows and are targeted for release on Oct. 8th of 2015.
(as mentioned in https://github.com/Azure/azure-powershell/wiki/Deprecation-of-Switch-AzureMode-in-Azure-PowerShell)
Example script was modified to use new AzureRm* commands

@troyanov - Thanks for the updates. @ms-toddabel - Could you review and let me know if this has been addressed through some of your recent updates, or if the updates can be merged?

This article is no longer published.

Closed per author feedback.


PHP: update readme file

For the issue #12481, https://github.com/grpc/grpc.github.io/issues/579.

Add links to grpc.io and cloud.google.com
Make the installation part the same as grpc.io
Update the README file for php example.

Update all related version in README file. I think this PR can be merged after the test finish.

Bazel Debug build for C/C++- #15610
Bazel Opt build for C/C++ - #15610

Hi @stanley-cheung . I updated all related version in README file. Thanks for reviewing it!


Display lightbulb icon for previous suggested replies

Requirements
As a means of offering the ability to access suggested replies for previous messages, we should display a lightbulb icon on the righthand side of previous messages that have suggested replies. This requirement should not be limited to a certain amount of messages within the conversation, but rather all messages that contain suggested replies.
We can show the suggested replies on top of the input bar if one of the last 5 messages has suggested replies. If none of last 5 messages contain suggested replies, we will only show the lightbulb icon.
Design Example

In the design above, the lightbulb would be shown to the righthand side of the first message ("Hey, when are you open?"), and the suggested replies prompt would already be available on top of the input bar for the most recent message.
Design Notes
Please use this design for the lightbulb icon.

Related to #677


OSCQuery values have value 0 until changed at least once

Steps to reproduce:

Make a new avatar
Put around 4 bools into its FX layer
Make sure these are also present in the Parameter List of the avatar
Make sure that at least 2 of them have True as their starting value
Upload the avatar
Go to the OSCQuery page in your browser
All the bools will be False

I also tried making a new layer, with an animation state that had a parameter driver, setting one of these values to True, but that did not change anything, what so ever.
To fix such a value, the value has to be toggled in the game.
After toggling it, it will show up correctly on the OSCQuery page from then on.
So the bug is that wrong parameter values are being broadcasted from VRC, through OSCQuery.
And this is not only broken for Booleans from what I have seen, but for other values as well, such as Int and Float parameters.

An update on this issue:

I added a layer with an empty animation (no keyframes at all), which transitions to another animation state in 1 second, since the exit time is 1.
The second state, that the FX layer gets to after 1 second, turns "value1" to "True".
This is reflected in OSCQuery correctly. This value is set to "True" correctly.
So maybe this is an issue with parameter starting values, that do not change after the avatar is initialized.
Here is a public avatar, where this can be seen: https://vrchat.com/home/avatar/avtr_d33cfc49-a608-4e93-9290-400c4e4976a1
All of these values should be true:

(value1 is set to true by a parameter driver, and only that appears as true in OSCQuery)

Posted this on Canny as well, as this is a serious issue: https://feedback.vrchat.com/bug-reports/p/oscquery-broadcasts-wrong-parameter-values


Contact us page (tracking)

📝 Changed routes:

/contact-us

(view full size)

Commit 05f40014194460d6ad63a2c00fd886e91a788ddf (https://thirdweb-ls81npnik.thirdweb-preview.com).


FIX: Get CMS preview working for non inline-editable elements.

Fixes #905. Depends on silverstripe/silverstripe-admin#1250
This was a feature in an earlier version of the module and most of the code is still in place.
Edit: While this technically depends on the PR from silverstripe-admin, it won't cause any defects if it does get merged in early, and merging it in even without that PR would make it easier for project-specific code to enable previews for elementals.

Have manually tested this, along with the following PRs

https://github.com/silverstripe/silverstripe-framework/pull/10112 (alread merged)
https://github.com/silverstripe/silverstripe-admin/pull/1252/files (unmerged)
https://github.com/silverstripe/silverstripe-admin/pull/1250 (unmerged)

I've installed dnadesign/silverstripe-elemental-userforms which is a non-inline editable content block
Can confirm that split-mode updates as expected

@GuySartorelli you'll need to rebase this PR on top of 4 before it can be merged

@emteknetnz Rebased on 4. Thanks (and @michalkleiner too) for looking at this! It's great to see movement on these preview PRs.

All of the upstream PRs this work depends on have been merged in.


Remove panic from compiler.rs (2)

Adds a new error type: NativeAssertError. It's a replacement for panic!, as it collects the backtrace but it doesn't interrupt the flow of the program.
There is a single todo! left, but I didn't know how to deal with it. We should take a closer look at it.
There are a few expect left, but being such a simple case, I'm not sure is worth handling it.
Unlike #905, I tried to make this non-panic abstraction as similar as possible to the associated panicking code. The compiler.rs was changed to use the non-panic variants
Checklist

[ ] Linked to Github Issue
[ ] Unit tests added
[ ] Integration tests added.
[ ] This change requires new documentation.

[ ] Documentation has been added/updated.

Codecov Report
Attention: Patch coverage is 45.94595% with 20 lines in your changes missing coverage. Please review.

Project coverage is 82.97%. Comparing base (4d24686) to head (3900284).

Files with missing lines
Patch %
Lines

src/error.rs
14.28%
18 Missing :warning:

src/compiler.rs
87.50%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #906      +/-   ##
==========================================
- Coverage   83.01%   82.97%   -0.04%
==========================================
  Files         120      120
  Lines       34313    34341      +28
==========================================
+ Hits        28485    28495      +10
- Misses       5828     5846      +18

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


await conn.closed() seems to not be working

I want to implement a reconnect function in my app but await connection.closed() only waits forever and does not react to network outages and queue going offline

Please provide an example

Sure, this is the code I'm using
const conn = await connect(queue_uri);

    const ch = await conn.openChannel();
    logger.debug(`Connected`);

    await ch.declareQueue({
        queue: queue_name,
        durable: true,
    });

    logger.debug(`Consuming messages`);
    await ch.consume(
        {
            queue: queue_name,
        },
        async (args, __, bindata) => {
            const data = new TextDecoder().decode(bindata);
            const msg = JSON.parse(data);

            logger.debug(`Processing message:`, msg);
            await messageHandler(ch, args, msg);
            logger.debug(`Processed`);
        }
    );

    logger.debug("waiting for connection to close");
    await conn.closed()
    logger.debug(`Finished consuming message`);

You need to call "close" to close the connection. See https://deno.land/x/amqp@v0.23.1/mod.ts?s=AmqpConnection


坚果pro1代手机无法正常运行。第1步都无法跳出去。报错如下

############################
(226.15384615384608, 0.06074766355140192, 0.8392156862745098)
(140, 142, 149)
如果你的脚本无法工作，上报issue时请copy如下信息:
Screen: Physical size: 1080x1920
Density: Physical density: 480
DeviceType: OD103
OS: darwin
Python: 2.7.10 (default, Jul 15 2017, 17:16:57)
[GCC 4.2.1 Compatible Apple LLVM 9.0.0 (clang-900.0.31)]
minLayer = 0, maxLayer =<PHONE_NUMBER>
capture ok : w = 1080, h = 1920, s = 1088, f = 1, size = 8355840
minLayer = 0, maxLayer =<PHONE_NUMBER>
capture ok : w = 1080, h = 1920, s = 1088, f = 1, size = 8355840
采用方式1获取截图
minLayer = 0, maxLayer =<PHONE_NUMBER>
capture ok : w = 1080, h = 1920, s = 1088, f = 1, size = 8355840
('scan_start_y: ', 690)
(1514805594, 338, 1109, 788, 849)
adb shell input swipe 500 1600 500 1602 723
minLayer = 0, maxLayer =<PHONE_NUMBER>
capture ok : w = 1080, h = 1920, s = 1088, f = 1, size = 8355840
('scan_start_y: ', 690)
(1514805598, 340, 1195, 788, 849)
adb shell input swipe 500 1600 500 1602 787
minLayer = 0, maxLayer =<PHONE_NUMBER>
capture ok : w = 1080, h = 1920, s = 1088, f = 1, size = 8355840
('scan_start_y: ', 690)
(1514805602, 340, 1195, 788, 849)
adb shell input swipe 500 1600 500 1602 787
minLayer = 0, maxLayer =<PHONE_NUMBER>
capture ok : w = 1080, h = 1920, s = 1088, f = 1, size = 8355840
('scan_start_y: ', 690)
(1514805606, 340, 1195, 788, 849)
adb shell input swipe 500 1600 500 1602 787

Screen: adb server is out of date.  killing...

daemon started successfully *
Physical size: 1080x1920
Density: adb server is out of date.  killing...
daemon started successfully *
Physical density: 480
DeviceType: adb server is out of date.  killing...
daemon started successfully *
OD103
OS: win32
Python: 3.6.3 (v3.6.3:2c5fed8, Oct  3 2017, 18:11:49) [MSC v.1900 64 bit (AMD64)]

adb server is out of date.  killing...

daemon started successfully *
minLayer = 0, maxLayer =<PHONE_NUMBER>
capture ok : w = 1080, h = 1920, s = 1088, f = 1, size = 8355840
adb server is out of date.  killing...
daemon started successfully *
2634 KB/s (210720 bytes in 0.078s)
采用方式0获取截图
adb server is out of date.  killing...
daemon started successfully *
minLayer = 0, maxLayer =<PHONE_NUMBER>
capture ok : w = 1080, h = 1920, s = 1088, f = 1, size = 8355840
adb server is out of date.  killing...
daemon started successfully *
2406 KB/s (210720 bytes in 0.085s)
scan_start_y:  690
Traceback (most recent call last):
File "123.py", line 288, in
main()
File "123.py", line 277, in main
piece_x, piece_y, board_x, board_y = find_piece_and_board(im)
File "123.py", line 202, in find_piece_and_board
for j in range(board_x_start, board_x_end):
TypeError: 'float' object cannot be interpreted as an integer

通为坚果Pro 1报错

#501


Extras are not resolved without explicit version lock (due to pre-release presence in dependency tree)

Pipenv fails resolution when extras are used.
$ python -m pipenv.help output
Pipenv version: '11.10.1'
Pipenv location: '/usr/lib/python3.6/site-packages/pipenv'
Python location: '/usr/bin/python3'
Other Python installations in PATH:

2.7: /usr/bin/python2.7

2.7: /usr/bin/python2.7

3.6: /usr/bin/python3.6m

3.6: /usr/bin/python3.6

2.7.14: /usr/bin/python

2.7.14: /usr/bin/python2

3.6.3: /usr/bin/python3

PEP 508 Information:
{'implementation_name': 'cpython',
 'implementation_version': '3.6.3',
 'os_name': 'posix',
 'platform_machine': 'x86_64',
 'platform_python_implementation': 'CPython',
 'platform_release': '4.13.15-200.fc26.x86_64',
 'platform_system': 'Linux',
 'platform_version': '#1 SMP Tue Nov 21 21:10:40 UTC 2017',
 'python_full_version': '3.6.3',
 'python_version': '3.6',
 'sys_platform': 'linux'}

System environment variables:

XDG_RUNTIME_DIR
SSH_AUTH_SOCK
GIO_LAUNCHED_DESKTOP_FILE_PID
GUESTFISH_PS1
LANG
DISPLAY
SHLVL
LOGNAME
XDG_VTNR
PWD
SESSION_MANAGER
HISTSIZE
XAUTHORITY
QTDIR
JOURNAL_STREAM
DESKTOP_AUTOSTART_ID
COLORTERM
XDG_SESSION_ID
GIO_LAUNCHED_DESKTOP_FILE
DESKTOP_SESSION
XDG_SESSION_DESKTOP
GDMSESSION
USERNAME
TERMINATOR_UUID
WINDOWPATH
GUESTFISH_OUTPUT
GNOME_DESKTOP_SESSION_ID
VTE_VERSION
DBUS_SESSION_BUS_ADDRESS
MODULESHOME
MAIL
XDG_DATA_DIRS
LESSOPEN
HISTCONTROL
TERMINATOR_DBUS_NAME
XDG_MENU_PREFIX
TERMINATOR_DBUS_PATH
GDM_LANG
HOSTNAME
XDG_SESSION_TYPE
OLDPWD
TERM
QTLIB
XDG_CURRENT_DESKTOP
GUESTFISH_RESTORE
GUESTFISH_INIT
SHELL
MODULEPATH
HOME
XDG_SEAT
QT_IM_MODULE
USER
QTINC
SSH_ASKPASS
LOADEDMODULES
PATH
XMODIFIERS
LS_COLORS
ZSH
PAGER
LESS
LSCOLORS
VIRTUALENVWRAPPER_PROJECT_FILENAME
VIRTUALENVWRAPPER_WORKON_CD
VIRTUALENVWRAPPER_SCRIPT
WORKON_HOME
VIRTUALENVWRAPPER_HOOK_DIR
LC_CTYPE
GOPATH
_
PYTHONDONTWRITEBYTECODE
PIP_PYTHON_PATH

Pipenv–specific environment variables:
Debug–specific environment variables:

PATH: /home/fpokorny/bin:/usr/local/cuda-8.0/bin:/usr/local/bin:/usr/lib64/qt-3.3/bin:/usr/local/bin:/usr/local/sbin:/usr/bin:/usr/sbin:/home/fpokorny/bin
SHELL: /usr/bin/zsh
LANG: en_US.UTF-8
PWD: /tmp/ao

Contents of Pipfile ('/tmp/ao/Pipfile'):
[[source]]
url = "https://pypi.org/simple"
verify_ssl = true
name = "pypi"

[packages]
selinon = {extras = ["celery"], version = ">=0.1.0rc5"}

[dev-packages]

[requires]
python_version = "3.6"

$ pipenv install 'selinon[celery]'
Adding selinon[celery] to Pipfile's [packages]…
Pipfile.lock not found, creating…
Locking [dev-packages] dependencies…
Locking [packages] dependencies…

Warning: Your dependencies could not be resolved. You likely have a mismatch in your sub-dependencies.
  You can use $ pipenv install --skip-lock to bypass this mechanism, then run $ pipenv graph to inspect the situation.Hint: try $ pipenv lock --pre if it is a pre-release dependency
Could not find a version that matches selinon[celery]
Tried: 0.1.0rc2, 0.1.0rc3, 0.1.0rc4, 0.1.0rc4, 0.1.0rc5, 0.1.0rc5, 0.1.0rc6, 0.1.0rc6, 0.1.0rc7, 0.1.0rc7, 0.1.0rc8, 0.1.0rc8, 0.1.0rc9, 0.1.0rc9, 1.0.0rc1, 1.0.0rc1, 1.0.0rc2, 1.0.0rc2, 1.0.0rc3, 1.0.0rc3, 1.0.0rc4, 1.0.0rc4
There are incompatible versions in the resolved dependencies.

However, if I explicitly specify version, the Pipfile.lock is correctly created:
$ pipenv install 'selinon[celery]==1.0.0rc4'

Expected result
The Pipenv.lock is correctly created even without specifying version explicitly.
Actual result
The lockfile is not correctly created, pipenv exits in ROUND 1 during resolution:
...
Using pip: -i https://pypi.org/simple

                          ROUND 1
Current constraints:

Finding the best candidates:

Finding secondary dependencies:
------------------------------------------------------------
Result of round 1: stable, done

Locking [packages] dependencies…
Using pip: -i https://pypi.org/simple

                          ROUND 1
Current constraints:
  selinon[celery]

Finding the best candidates:
Using pip: -i https://pypi.org/simple

                          ROUND 1
Current constraints:
  selinon[celery]

Finding the best candidates:

Warning: Your dependencies could not be resolved. You likely have a mismatch in your sub-dependencies.
...

Steps to replicate
Just try to install selinon[celery] in a new environment:
$ pipenv install 'selinon[celery]'

Hey, sorry for the issue.  Firstly, no need to use quotes for this, but secondly, this is not an issue with installing extras, it's an issue with installing pre-release candidates (note the rc<n> on the end).
See that as part of the error (which is missing a space I can see) it also says:
Hint: try $ pipenv lock --pre if it is a pre-release dependency

You should do that, I think your problems will be solved. Let me know if they arent, and again apologies, we are planning to make this 'just work' without requiring input if you are specifying only one thing (see #2022)

Hey, sorry for the issue. Firstly, no need to use quotes for this

I use zsh where quotes are necessary - I just copy+paste the provided input.

, but secondly, this is not an issue with installing extras, it's an issue with installing pre-release candidates (note the rc on the end).
See that as part of the error (which is missing a space I can see) it also says:
Hint: try $ pipenv lock --pre if it is a pre-release dependency

You should do that, I think your problems will be solved. Let me know if they arent, and again apologies, we are planning to make this 'just work' without requiring input if you are specifying only one thing (see #2022)

Great, the -pre flag works. I searched opened issues, but I missed this one.
Thanks for such a prompt response!

Sorry for the hiccup!

No problem at all. Thanks!


fix build on windows, readonly_data_p return TRUE always because IsBadWriterPtr is obsolute API on windows.

...

Ooops. It's not good to fix.

Sorry about noizy. please take a look.

I don't think readonly_data_p() should not always return true.