RichTextLabel mouse_exited() event fires by Context Menu when control is Right Clicked

Tested versions
v4.3.stable.official [77dcf97d8]
System information
Linux Kubuntu 24.04 KDE Plasma Version: 5.27.11 KDE Frameworks Version: 5.115.0 Qt Version: 5.15.13 Kernel Version: 6.8.0-39-generic (64-bit) Graphics Platform: X11 Processors: 16 × 12th Gen Intel® Core™ i7-1260P Memory: 15.3 GiB of RAM
Issue description
When enabled, the Context Menu of RichTextLabel fires a mouse_exited event when you right click on the label.  In addition, mousing over the Context Menu keeps firing mouse_exited/mouse_entered events. This interferes with the ability to use mouse_exited() with Context Menu at the same time.
However no such event is fired when Context Menu is disabled. Also mouse Left Click has no effect.

Here is the code of the single scene above:
extends Control

func _ready():
	# Checked context Menu
	$RichTextLabel.mouse_exited.connect(func():
		$RichTextLabel.add_theme_color_override("default_color",Color.WHITE)
		print(5)
		)

	$RichTextLabel.mouse_entered.connect(func():
		$RichTextLabel.add_theme_color_override("default_color",Color.RED)
		print(6)
		)

	# Unchecked Context Menu
	$RichTextLabel2.mouse_exited.connect(func():
		$RichTextLabel2.add_theme_color_override("default_color",Color.WHITE)
		print(7)
		)

	$RichTextLabel2.mouse_entered.connect(func():
		$RichTextLabel2.add_theme_color_override("default_color",Color.RED)
		print(8)
		)

Steps to reproduce
Add a RichtextLabel control to a scene with some text.
Enable the Context Menu.
Add mouse_exited event as above.
Right click and event fires even though you did not exit the label.
Mouse over Context Menu and the events keep firing.
Minimal reproduction project (MRP)
Attached is the above scene to demo the issue. Please add to an empty project if you wish to try it.
node_2d.zip

When enabled, the Context Menu of RichTextLabel fires a mouse_exited event when you right click on the label. In addition, mousing over the Context Menu keeps firing mouse_exited/mouse_entered events. This interferes with the ability to use mouse_exited() with Context Menu at the same time.

That's how it is supposed to work, context menu is a separate window and hovering it should emit enter/exit events on both parent window and menu window. Not sure what's the issue.

It's quirky since there is no way to know that there is a Context Menu open to ignore the mouse_exited().  See my tooltip_demo below.
I have a a situation where I display a custom RichTextLabel tooltip and allowing people to copy selected text. The tooltip then goes away when they mouse out.
Unfortunately the right click emits a mouse_exited() , closing the tooltip before they can use the Context Menu.
I can disable it, but people would have to know to use CTRL-C selected text.  Strangely the same reason why the Inspector tooltips do not have a Right Click context menu. You can only use CTRL-C to copy text.
Attached is a demo scene showing the issue.
tooltip_demo.zip

As an example I have a custom RichTextLabel tooltip being displayed. The tooltip closes when you mouse exit from it,
Also the tooltip allows the user to select text and right click to get the context menu. That particular action closes the tooltip, before the user selects from the Context Menu
Attached is a simple demo. Tooltip is displayed when you hover over the main control. When you hover over the tooltip and then away the tooltip closes. If you display tooltip again and then select text ad Right Click the tooltip closes. This should not be happening,  I don't know why it needs to emit a mouse exited since the mouse did not exit.
P.S. This is why the Editor Inspector does not have a Context menu enabled on it's own property tooltips. Whoever coded that had an issue but decided to just allow select text and then use CTRL+C
Attached is a simple demo to show the issue.
tooltip_demo.zip


Update LICENSE

can I also share the copyright?

Welcome to willytest! have fun

Welcome to willytest! have fun


progress bar

hello dear,
plugin works well but when downloading update apk from server , yello progressbar did not shown but apk file is downloading and after some time go to install apk file.
how can i solve this ?

same here. apk is around 30mb large. I did see the progress bar when I tried it with the demo version. Any thoughts?

Progress Bar doesn't update itself in % or fill up when downloading APK
Cordova Version: 8.1.2<EMAIL_ADDRESS>Android Platform Version: android 7.1.4
Plugin Versions
cordova-plugin-app-update 2.0.2 "AppUpdate"
cordova-plugin-appversion 1.0.0 "App Version"
Any fix for this?

Having same issue, any update?

at my site it was the missing content length from the webspace provider. Make sure the content length is sent.

hosted apk on godaddy, how do i send back the content length?


Minor prefactor

smaller refactor before adding a new class  …
mostly:

extract methods in RepositoryUpdater
code that writes out goes into EngineReport and is a bit better

Its out of date. Apart from that I'm happy

Done. Internet was down when I started this, hence the stale master.


tasks/main.yml wrong parameter in import_tasks: install.yml

What happened?

import_tasks: install.yml
become: true
when:
( not __node_exporter_is_installed.stat.exists ) or
( __node_exporter_current_version_output.stderr_lines[0].split(" ")[2] != node_exporter_version ) or
( node_exporter_binary_local_dir | length > 0 )
tags:

node_exporter_install
Did you expect to see some different?

import_tasks: install.yml
become: true
when:
( not __node_exporter_is_installed.stat.exists ) or
( __node_exporter_current_version_output.stdout_lines[0].split(" ")[2] != node_exporter_version ) or
( node_exporter_binary_local_dir | length > 0 )
tags:

node_exporter_install
How to reproduce it (as minimally and precisely as possible):

Environment

Role version:
1.4.0-rc.0

Ansible version information:
ansible --version
  <!-- ansible [core 2.13.4]
config file = /etc/ansible/ansible.cfg
configured module search path = ['/home/andrey.grigorian/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
ansible python module location = /usr/lib/python3/dist-packages/ansible
ansible collection location = /home/andrey.grigorian/.ansible/collections:/usr/share/ansible/collections
executable location = /usr/bin/ansible
python version = 3.10.4 (main, Jun 29 2022, 12:14:53) [GCC 11.2.0]
jinja version = 3.0.3
libyaml = True
</li>

-->

Variables:

__node_exporter_current_version_output

Ansible playbook execution Logs:

TASK [cloudalchemy.node_exporter : Create the node_exporter group] *************************************************************************************************************************************
fatal: [rhea0109a.gaia.ucom.am]: FAILED! => {"msg": "The conditional check '( not __node_exporter_is_installed.stat.exists ) or ( __node_exporter_current_version_output.stderr_lines[0].split(\" \")[2] != node_exporter_version ) or ( node_exporter_binary_local_dir | length > 0 )' failed. The error was: error while evaluating conditional (( not __node_exporter_is_installed.stat.exists ) or ( __node_exporter_current_version_output.stderr_lines[0].split(\" \")[2] != node_exporter_version ) or ( node_exporter_binary_local_dir | length > 0 )): list object has no element 0\n\nThe error appears to be in '/home/andrey.grigorian/.ansible/roles/cloudalchemy.node_exporter/tasks/install.yml': line 8, column 3, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n\n- name: Create the node_exporter group\n  ^ here\n"}

Anything else we need to know?:

This role has been deprecated in favor of a the prometheus-community/ansible collection.


chore: add tests on the published npm artifact

Since we automate npm publishing, it should be fairly easy to add tests that ensure the published artifact installs/works correctly.
We can achieve this by npm or yarn installing the published version in CI after we publish it.

We might need to retry the installation as sometimes there's a delay between the time npm publish finishes to the time the package is available

One note: it sometimes takes several seconds after npm publish to be able to npm install a package.

A better way to do it might be to use a local registry for testing https://verdaccio.org/docs/en/e2e


Update profile without a profile picture

Describe the bug
When trying to update my profile's displaying name it gives me an error. It says that I have an invalid "invalid image"/profile picture. I currently have the default twitter profile picture which leads me to think that is the problem.
To Reproduce
Steps to reproduce the behavior:

Go to profile
Click on edit profile
Change name while having default profile picture
See error, "invalid image"

Expected behavior
Being able to update my profile's display name even though I have the default profile picture.
What device are you on?
Mac book pro, Mac OS Big Sur 11.2

@MusashiAsada , good point, I have issue which can be solved with it #1085

What's the image URL? Could be failing the regex pattern

Here's the URL: https://abs.twimg.com/sticky/default_profile_images/default_profile_normal.png

yeah the regex is for a different type of profile picture URL: https://pbs.twimg.com/profile_images/1295038552661725184/LzFeB7Dj_400x400.jpg
your has default_profile_images in it, I'll fix that.

Thank you! I don't completely understand what the issue is but I'm trying to learn. I just wanted to contribute with something to make this as good as possible!

fixed by #1155


Create a script to generate all the firrtl primitive op wrappers

This will look similar to the script we have for generating verilog modules.
I think this would be nice so that we could leverage it in the firrtl backends for both magma and coreir.
What do you think?

Sounds reasonable.
One issue is that my firrtl backend for magma emits a wrapped module for each input width (i.e. Add8 where the input types are UInt<8>). It does this on demand depending on what is used the in the magma circuit being compiled.
I just read in the spec that you can leave the width of the IO types to a module unspecified.

Alternatively, if the bit width is omitted, it will be automatically inferred by FIRRTL’s width inferencer, as detailed in section 9.

This sounds like a better strategy. Basically, define a generic "Module" version of each primitive with the widths unspecified. I think we still need to define different wrappers for different IO types (i.e. an unsigned add vs a signed add).

Also, because it doesn't hurt to ask :)
https://github.com/freechipsproject/firrtl/issues/632

From Adam:

The problem with simply added the inst syntax is that FIRRTL primops are polymorphic in their input/output types. Thus, at declaration of inst my_adder of add, we don't know the input or output types of my_adder.

So looks generating wrapped modules with the inline annotation is our best bet.

Yeah that does make sense. The only other way to do that would be if they could parameterize instances (which I do not think they could do)
inst my_adder_signed of add(SInt)

But yeah, I do agree that it is preferable to use wrappers that are generic over the width. We might as well utilize that feature. I also definitely agree that we will need separate ops for signed and unsigned.
For example something like
Module firrtl_slt :    in : UInt[2]   out UInt<1>   in0 <= toSInt(in[0])   in1 <= toSInt(in[1])   out <= lt(in0,in1)

@leonardt, Are you going to be able to do this before the meeting next Monday? Also related, do you know if it is possible to have separate files for the same firrtl circuit?

@rdaly525 sure, I can have it done by Wednesday (busying prepping for AHA presentation tomorrow)

Awesome thanks!

@rdaly525 I've started on this, but it looks like we're not going to be able to use this approach for parameterized primitives like shr(e, n) where n is a parameter (static shift). I can't generate a module for each parameter, so it looks like you'll need to either generate these modules on demand (when you know which specific parameter you need) or special case the code generation to handle firrtl primitives directly (that's what I did in magma).

Is that the only case where this is an issue? I could imagine a workaround to be just treat it as a dynamic shift and let firrtl compiler change it to a static shift.

ops with parameters are

pad
shl, shr
bits (bit extraction, selecting a range of bits)
head, tail (truncate n bits off the beginning/end)

This is unneeded now


Missed data in geocoding/data/it

Missed prefix:
'390577' => 'Siena'
'390575' => 'Arezzo'
'390574' => 'Prato'
'390583' => 'Lucca'

Hi,
Could you please clarify what you mean? I find all these prefixes in the resource files.
https://github.com/googlei18n/libphonenumber/blob/master/resources/geocoding/it/39.txt#L142

http://libphonenumber.appspot.com shows that numbers with these prefixes all geocode properly.
@michelep - You must be using an old version of libphonenumber. Please sync to the latest version and try again, and re-open if you are still encountering a problem. Thanks.


google colab with cuda11.1

hi and tnx for the code.
is there any chances that your code run properly on google colab because i tried many ways and installed mmdetection manually and i changed all AT_CHECK to TORCH_CHECK but the main problem is the compile.sh file and the fact that its not possible to change env in colab even if we install anaconda and that cuda 11.1 is default cuda in colab and even when i change it to 10.1 the mmcv cant install so i think its sort of impossible to run your code but if anyone could get results on colab every help is appreciated.

I think the code could run on colab as long as you follow our INSTALL.md, such as (1) cuda 10.0 (2) torch==1.3.1 (3) mmcv==0.2.14 (4) gcc==4.9

@navidasj96 @csuhan I am facing same issue is there anyway around it? I can not install mmcv-full on colab if downgrade cuda to 10.0. I installed mmcv then but again when i install mmdet it tries to install mmcv-full and so it fails again. I can install all other libs of exact versions like torch etc but mmcv does not work.


Programmatically setting a value for MultipleSelector

Hey, I am trying to set a value using a MultipleSelector row, and can't figure out how to select one of the values by doing something such as row.value = "example"
In my code its for the purpose of editing an item after creating it.
In the example:
row = FormRowDescriptor(tag: Static.categories, rowType: .MultipleSelector, title: "Categories")
row.configuration[FormRowDescriptor.Configuration.Options] = [0, 1, 2, 3, 4]
row.configuration[FormRowDescriptor.Configuration.AllowsMultipleSelection] = true
row.configuration[FormRowDescriptor.Configuration.TitleFormatterClosure] = { value in
switch( value ) {
case 0:
return "Restaurant"
case 1:
return "Pub"
case 2:
return "Shop"
case 3:
return "Hotel"
case 4:
return "Camping"
default:
return nil
}
} as TitleFormatterClosure
if I was to do:
row.titleForOptionValue("Restaurant");
or row.titleForOptionValue(0);
or row.value = "Restaurant"
or row.title = "Restaurant"
Then the selected option still shows up as nothing
How can I get it so that it displays one of the options by default when the form loads?
Thanks

Figured it out. I need to set the row.value before configuring the TitleFormatterClosure.
Example:
row = FormRowDescriptor(tag: Static.categories, rowType: .MultipleSelector, title: "Categories")
row.configuration[FormRowDescriptor.Configuration.Options] = [0, 1, 2, 3, 4]
row.configuration[FormRowDescriptor.Configuration.AllowsMultipleSelection] = true
row.value = "Restaurant"
row.configuration[FormRowDescriptor.Configuration.TitleFormatterClosure] = { value in
switch( value ) {
case 0:
return "Restaurant"
case 1:
return "Pub"
case 2:
return "Shop"
case 3:
return "Hotel"
case 4:
return "Camping"
default:
return nil
}
} as TitleFormatterClosure

I have the same issue but the problem is that after setting the selected value, the user can no longer make any changes. Were you able to solve that and can share?


Added missing code parts in Chapter 9

Added following missing code parts:

make map constants public
in Listing your inventory: ItemMenuResult
rusntate::showinventory -> create WantsToDrinkPotion
Runstate::prerun -> added call to ecs.maintain

Fixed Typo inventory.rs to inventory_system.rs

Thanks! This was my first pull request ever :) Glad I could help.
As I don't know how else to contact you, I'll just ask here. Seems like a lot of people like your tutorial and I think it would maybe be a cool thing to have a discord server, where people can ask if they have any problems or discuss about what they are working on. Just an idea

You can find me on twitter as @herberticus, and I can be found on the Roguelikes discord in the roguelikedev channel during US-Central Time daytime. I think this will get you there: https://discord.gg/tCu7Gs


allow use buttons (left and right) in navbar without disappear in the mobile screen

Hi guys, this is an enhancement for the navbar.
Example:
http://www.google.com/design/spec/layout/structure.html#structure-app-bar

Not sure what you mean by this. You can already do this

Oh, sorry i didn't know that. The documentation has an example?

Documentation for what?
On Wed, Feb 4, 2015 at 10:55 PM, tinchoz49<EMAIL_ADDRESS>wrote:

Oh, sorry i didn't know that. The documentation has an example?
—
Reply to this email directly or view it on GitHub
https://github.com/Dogfalo/materialize/issues/616#issuecomment-72990205.

--
Doggy sends his greetings from Mars.

@Dogfalo - It's not documented, I would probably add soon.
@tinchoz49 - If you need to show some buttons both in desktop and mobile - do not enclose them in "side-nav" container. For example those 3 icons on right (shown in link http://www.google.com/design/spec/layout/structure.html#structure-app-bar) will not be inside a ul that has class "side-nav".

thanks @snow01!
Ticket closed!


[🐞]

Which package is affected?
Headless Kit
Describe the bug
Issue: Modal Backdrop Covers Entire Screen and Buttons Flicker
Description
I’m encountering an issue with the Qwik modal component in my application. I have a modal that works correctly in one part of my app but misbehaves in others. When I use bind:show={admin.modalShow} to control the modal's visibility, the backdrop covers the entire screen unintentionally, and buttons inside the modal either flicker or do not appear.
Code Example
Here's a simplified version of my component setup:
<Modal.Root closeOnBackdropClick={true} bind:show={admin.modalShow}>
  <Modal.Trigger class="btn" onClick$={async () => {
    await admin.store.update({ action: "set-form", record: props.id as "Ingredient" })
  }}>
    <PhPencil class="h-5 fill-blue-500" />
  </Modal.Trigger>
  <Modal.Panel class="bg-white dark:bg-gray-950 text-gray-50 p-4 data-[state='open']:h-4/5
    data-[state='open']:rounded-lg data-[state='open']:shadow-lg">
    <Modal.Close class="btn">

Close

    </Modal.Close>

Create New Food Item

    <Resource value={admin.forms}
      onResolved={(v) => {
        return <CreateForm data={v} key={`form-${props.id}`} recordId={props.id} submit={"update"} />
      }}
      onPending={() =>
Loading...
}
    />
  </Modal.Panel>
</Modal.Root>

What I've Tried

Verified that admin.modalShow is a Signal<boolean> and logs changes to console.
Checked CSS styles to ensure no unintended styles cause the backdrop to cover the screen.
Inspected DOM elements to confirm correct rendering.
Used the console log to trace the state changes of admin.modalShow.

Observations

When bind:show={admin.modalShow} is not used, the modal displays and functions correctly.
The issue arises only when the modal is controlled using the bind:show signal.
The DOM structure looks correct, but styles seem off when bind:show is in effect.

CSS Example
Here’s the CSS applied to the modal:
Reproduction
Hope to do something soon
Steps to reproduce
No response
System Info
System:
    OS: macOS 14.5
    CPU: (8) arm64 Apple M2
    Memory: 279.22 MB / 16.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 21.6.2 - ~/.nvm/versions/node/v21.6.2/bin/node
    npm: 10.8.2 - ~/projects/publish_notes/frontend/node_modules/.bin/npm
    pnpm: 9.1.0 - ~/Library/pnpm/pnpm
    bun: 1.1.21 - ~/.bun/bin/bun
    Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
  Browsers:
    Chrome: 127.0.6533.89
    Safari: 17.5
  npmPackages:
    @builder.io/qwik: ^1.7.3 => 1.7.3
    @builder.io/qwik-auth: ^0.2.2 => 0.2.2
    @builder.io/qwik-city: ^1.7.3 => 1.7.3
    @builder.io/qwik-labs: github:BuilderIo/qwik-labs-build#main => 0.0.1
    @qwik-ui/headless: ^0.4.4 => 0.4.4
    @qwik-ui/styled: ^0.1.0 => 0.1.0
    @qwik-ui/utils: ^0.2.1 => 0.2.1
    typescript: ^5.5.4 => 5.5.4
    undici: ^6.19.5 => 6.19.5
    vite: ^5.3.5 => 5.3.5

Additional Information
No response

Solved on Discord.


add missing call to ob_clean

Fix #134.
This PR just add a missing call to ob_clean, like showed in Advanced Shutdown Handler of the documentation.

I think that we need to check ob_get_contents before calling ob_clean
https://github.com/slimphp/Slim-Skeleton/blob/master/src/Application/ResponseEmitter/ResponseEmitter.php#L28


Improve code comments

Closes #644
Partially address #676
Partially address #677

I know the issues this PR addresses are marked for V1.0.0, but I thought it would be good to generally clean the code now. That way people won't use the older style as an example within the code base.

It's marked for 1.0.0 but that doesn't mean it needs to wait for that. For me it's a way of prioritizing tasks mostly for myself and just getting them assigned so that by 1.0.0 all of the tasks are completed. I have no objections to taking on tasks that are assigned for other milestones earlier especially if they're not related to core gameplay. :D


[Snyk] Upgrade tape from 5.6.3 to 5.6.4

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade tape from 5.6.3 to 5.6.4.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 22 days ago, on 2023-07-02.

Release notes

Package name: tape

5.6.4 - 2023-07-02v5.6.4

5.6.3 - 2023-01-15v5.6.3

    from tape GitHub release notes

Commit messages

Package name: tape

41fc81a v5.6.4
70de437 [Performance] use `call-bind` for autobinding
2043b2e [Deps] update `deep-equal`
a6a5eee [Dev Deps] update `@ ljharb/eslint-config`, `aud`
e244e64 [Refactor] `Test`: skip binding for a non-function value
26a75bb [readme] Link to explain what TAP is
0e80800 [Dev Deps] pin `jackspeak` since 2.1.2+ depends on npm aliases, which kill the install process in npm < 6
c656ee5 [Tests] simplify tests
7e7c3d0 [Deps] update `minimist`
c1b619d [readme] improve t.throws description for Function
d5d675d Merge tag 'v4.16.2'
a892b65 v4.16.2
0731b5f [Fix] `throws`: avoid crashing on a nonconfigurable or nonextensible `expected`
834453c [actions] update rebase action
01edce8 [Deps] update `defined`, `minimist`, `object-inspect`, `string.prototype.trim`
1b3ad24 [Dev Deps] update `@ ljharb/eslint-config`, `array.prototype.flatmap`, `aud`
15e2175 [meta] add missing npmrc config

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Codecov Report
Patch and project coverage have no change.

Comparison is base (c8440f5) 90.54% compared to head (30501c7) 90.54%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #284   +/-   ##
=======================================
  Coverage   90.54%   90.54%
=======================================
  Files           9        9
  Lines         296      296
=======================================
  Hits          268      268
  Misses         28       28

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Trying to fix the JaCoCo yml to generate a badge with code coverage

Still configuring the Jacoco badge to build the project and show our code coverage

I updated the CodeCoverage.yml to build the project correctly. Still working on how to generate the report in the badge.


Add and Edit upload/download - input fields too long

The entry fields are so large they appear on a different line. This has changed - they were previously on the same line. It looks bad.

Edit validation setup - parser fields are also too long

Also Edit validation setup - email field is too long

Fixed 0.9.7


Run tsc compiler in CI

Would it be reasonable to run this in CI as a guard against mistakes like I made in https://github.com/flatgeobuf/flatgeobuf/commit/3b4e22c33aac74cbb96ec6c6b299c20a692d8948 ?

Yep, thanks. 🙂


Move media queries to  instead of

I'm loading in the contents of a css file and emogrifier is correctly moving all the media queries to a STYLE tag in the HEAD of the output.  Since most email clients strip out the HEAD, Foundation for Email recommends that the media queries be placed at the top of the BODY tag.  Is it possible to make that change or create a setting that will do that?

Hi @Gabe-SpotMe, do you have any links about the HEAD getting stripped in most email clients and about the requirement from Foundation? Thanks!

Sure, here's their online inliner.
Take a look at the notes on the right of the page.

I just put one up.  I'm a relatively novice developer so I apologize if I missed anything.  My changes worked in my tests though...

@Gabe-SpotMe Great!
Could you please add some unit tests that fail without your changes and pass with it? And could you please add a changelog line?

I added the changelog line and added a note to the readme.  Unfortunately I've never written a unit test before so I'm not quite sure how to do that...

@Gabe-SpotMe I'll be glad to help you with the unit tests.

Have you already done a "composer install" with your local copy of this Repository? (see https://github.com/jjriv/emogrifier/blob/master/CONTRIBUTING.md#install-the-development-dependencies )
Have you already run the existing unit tests using "vendor/bin/phpunit Tests/"? (see https://github.com/jjriv/emogrifier/blob/master/CONTRIBUTING.md#unit-test-your-changes )

I did the composer install and that worked fine.  But when I try to run existing unit tests I get the following error:
cannot open file "Tests/.php".

@Gabe-SpotMe Could you please post the exact command line you've used to run the tests? Thanks!

I'm running Windows 10.  I navigated to the emogrifier/vendor/bin directory and tried to run 'phpunit Tests/'
it then shows the following error:
cannot open file "Tests/.php".
If I run just 'phpunit' at that same folder, it does successfully give me PHPUnit 4.7.27 help content.

You'll need to be in the package's root directory and then run:
vendor/bin/phpunit Tests/

(Composer bin commands expect to be called from within the package root, not from the vendor/bin/ directory.)

On Windows, you also might need to replace the forward slashes with backslashes (depending on which shell you're using).

Any progress on this?

@Renkas I'm currently waiting for a reply by @Gabe-SpotMe.
@Renkas Would you be willing to take over?

Fixed by #396.


go tool vet is not running on all examples

When running make lint the vetting tool is not ran on all our examples by default. This let a vetting error slip through the cracks in the ping-thrift-gen example.
For some reason go tool vet only runs on this example after installing the example:
$ go install github.com/uber/ringpop-go/examples/ping-thrift-gen/

When you run make lint after an installation you will see the following error
$ make lint
ERROR: ./examples/ping-thrift-gen/main.go:95: github.com/uber/ringpop-go/examples/ping-thrift-gen/gen-go/ping.Pong composite literal uses unkeyed fields
make: *** [lint] Error 1

This will prevent the precommit hook that is setup to finish which makes it harder to commit. The easiest way to remove this error is by removing the ping-thrift-gen from pkg via
$ rm -rf $GOPATH/pkg/darwin_amd64/github.com/uber/ringpop-go/examples/ping-thrift-gen # OSX specific

We should find the root cause for the vetter to not be running correctly and fix the vetting issue that it finds when it does run.

The error in question is a valid error, it's complaining about:
https://github.com/uber/ringpop-go/blob/master/examples/ping-thrift-gen/main.go#L95
	return &gen.Pong{"Hello, world!", identity, &pHeader}, nil

The vet tool raises an error when you don't specify keys for literals for types that are defined outside of your package. You can fix this by specifying keys (e.g., &gen.Pong{Message: "Hello, world!"}).
The reason this is an error is to make it easier for package owners to add fields to structs without breaking all code that depends on this package. If some package uses B.Pong{"Hello world"}, now that call will fail when the owner of B adds a new field to Pong, since unkeyed literals must specify all fields.
The vet tool uses the package cache to determine the types. E.g., if the Pong type was actually a slice, then you wouldn't need keys.

Thanks for the explanation why it needs the files in pkg. Now we need to find a way to get these files available on travis-ci so the make lint test starts failing. Because now we ran into this 'by accident' just wondering if there are more (important) files failing on vet as well.

To get packages into the pkg directory, you need to do go install, or go test -i or go build -i.
In TChannel, we run the tests first, which will do go test -i to install all the dependencies and store them in pkg, then we run the lint rules so vet has the package files available.
I still need to write that common Makefile or simple build system for our open source repositories so we don't have to duplicate Makefiles everywhere, just haven't gotten the chance yet


[Feature] Use .brainglobe directory to store deformations

would .brainglobe/deformations be a good directory from me to store the data @adamltyson?

Alternatively I could store the deformation in the directory of the atlas, in the same folders as the atlas files. I'll review the v2 spec and start conforming to this

Yeah, for now. We can/should revisit once the API itself is updated.

add just a _v1.0 string

I assume you mean add the actual version, not just _v1.0!

well all the versions now will be v1.0! as it would just be versioning for the deformation matrix. But yeah in the future if the deformations are updated we would update the version number.
One issue is that there are versions of spaces and versions of deformations... This might get confusing

Oh yeah, I misunderstood. Should a deformation version be a combination of space versions? I.e. a specific deformation version for specific versions of source and target spaces?

but then again I assume the space won't update without changing it's name entirely??

if there was theoretically a ccfv4 based on MRI. would this still be allen_mouse in the brainglobe api


SetUserContext doesn't work on IOS 10.4.3

I wrapped setUserContext into function and call it in the ComponentDidMount of my app.
export const sentrySetUserContext = (user) => {
  Sentry.setUserContext(getUserInfo(user));
};

const getUserInfo = (user) => ({
  userID: user.id.toString(),
});

To test, I add a button which calls Sentry.nativeCrash().
I received the error in the Sentry console but without information about UserID.
It only shows my IP.
IOS version: 10.4.3
Sentry version: 0.21.1
I did not test on Android.

Hey, can you enable verbose logging:
Sentry.config("DSN", {
  logLevel: SentryLog.Verbose
}).install();

You should see a output similar to this:
'react-native-sentry (setUserContext):', { userID: '12341' }

I wasn't able to reproduce this, please make sure your setUserContext is called.

I see the output:
 but on the website (Sentry console), i did not see theses informations.
I created a new project using create-react-native-app and did the same steps, and it seems to work. I'll investigate more to understand what happened.
Thanks you

I still don't know why the userID is not shown in the Sentry console.
Is it possible to see easily the content that is sent to your Sentry server when a crash occurs ? It seems that the information about user is lost, but i do not know where.
Thanks

I come with some insights:
Using Charles, I saw a difference in the body request between my app (which doesn't work) and the mini-app (which works).
The function called on both app is :
Sentry.setUserContext({ userID: "12345", extra: { country: 75 } })

Content of data sent to Sentry API in my App (not working):
"user": {
		"id": "12345",
		"username": "(null)",
		"email": "(null)",
		"extra": {
			"country": 75,
		}
	}

And content of data sent to Sentry API with mini-app (works):
"user": {
		"userID": "12345",
		"extra": {
			"country": 75,
		}
	}

I don't know why it added "username" and "email" keys and set them to "(null)". Because "(null)" is not a valid email, it won't display the user context in the Sentry console.
My little fix was to set username and email with fake values in setUserContextfunction.
I'm using<EMAIL_ADDRESS>

Thank you very much for investigating this so thoroughly, I will release a fix for this soon.


Fix small typos in ecco package

in ecco_check.F and ecco_readparms.F, correct character length when
checking for m_tr[Heat/Salt] and m_horflux_vol

What changes does this PR introduce?
Bug fix
Does this PR introduce a breaking change?
No

@NoraLoose Good catch ! and your fix looks good.
I propose to merge this PR soon (e.g., tomorrow) if no one complains


typo: Connnect -> Connect

Added some code fence languages as well

We're in the process of pulling these commits to the private repo, so we can verify the content on staging. We'll post a new PR # shortly and then close this PR.

I'm starting on pulling this PR to the private repo now.
Thanks,
Trisha

@nschonni - we've moved your suggestions into the private repo for staging and validation.
https://github.com/MicrosoftDocs/azure-docs-pr/pull/61953
Thank you.


[ANNOTATION] add the csv file which include filenames for test train split with labels

Downloading an entire dataset can be resource-intensive and inefficient. Instead, having a CSV file with predefined training and test splits offers significant advantages. This CSV file can be used for various purposes, such as fine-tuning models, maintaining consistent splits across experiments, and evaluating performance on unseen data. Moreover, it allows users to easily validate and compare the results of different models or algorithms without the need to repeatedly split the dataset. This approach not only saves resources but also ensures consistency and reproducibility in machine learning workflows.
Please upload the csv file.!!!

From the README:
You can also manually access the dataset at https://ladi.s3.amazonaws.com/index.html, or via AWS S3 at s3://ladi, and the provided dataset class in training/LADI-v2-dataset

The CSV files are in the S3 bucket.


Fix failure to ship description

Towards #169

Codecov Report

Merging #208 into master will increase coverage by 2.6%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master     #208     +/-   ##
=========================================
+ Coverage   62.81%   65.42%   +2.6%
=========================================
  Files          36       36
  Lines        1912     2059    +147
  Branches      139      182     +43
=========================================
+ Hits         1201     1347    +146
- Misses        711      712      +1

Impacted Files
Coverage Δ

lib/web_catalog/api_extractor.es6.js
100% <0%> (ø)
:arrow_up:

lib/web_catalog/post_processors.es6.js
94.05% <0%> (+0.63%)
:arrow_up:

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 6cc4fa4...3dc24cc. Read the comment docs.


[Project Addition]: Fashion MNIST Dataset Image Classification using DL

Deep Learning Simplified Repository (Proposing new issue)
:red_circle: Project Title :  Fashion MNIST Image Classification
:red_circle: Aim : This project focuses on image classification task using deep learning model on the Fashion MNIST dataset.
:red_circle: Dataset : https://www.kaggle.com/datasets/zalando-research/fashionmnist
:red_circle: Approach : I will use a CNN model to perform the task of image classification. I will use Early Stopping in the developed model. Additionally, a GUI using Tkinter to provide a user-friendly way to interact with the model.

📍 Follow the Guidelines to Contribute in the Project :

You need to create a separate folder named as the Project Title.
Inside that folder, there will be four main components.

Images - To store the required images.
Dataset - To store the dataset or, information/source about the dataset.
Model - To store the machine learning model you've created using the dataset.
requirements.txt - This file will contain the required packages/libraries to run the project in other machines.

Inside the Model folder, the README.md file must be filled up properly, with proper visualizations and conclusions.

:red_circle::yellow_circle: Points to Note :

The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.
"Issue Title" and "PR Title should be the same. Include issue number along with it.
Follow Contributing Guidelines & Code of Conduct before start Contributing.

:white_check_mark: To be Mentioned while taking the issue :

Full name : Anjali
GitHub Profile Link : https://github.com/deedGhost
Email ID<EMAIL_ADDRESS>Participant ID (if applicable): Anjali (discord: .sleepyded)
Approach for this Project : Deep learning model for Image Classification.
What is your participant role? (Mention the Open Source program)
GSSOC'24 Contributor

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

@deedGhost  , im looking forward to contribute Fashion MNIST Image Classification
can yu assign me task
discord name : harsha00429
mail<EMAIL_ADDRESS>github: https://github.com/VoonnaHarsha

Hi @deedGhost can you share a brief on your approach? What are the architectures/methods you are planning for this project?
Hi @VoonnaHarsha as this issue is raised by @deedGhost, hence this can be assigned to only Anjali. You can check out other issues, which are open and opened by me (@abhisheks008)

@abhisheks008 I will be using a CNN model as CNNs are widely used for image classification tasks. These architectures will have convolutional layers, pooling layers, and fully connected layers. I will be incorporating early stopping method to avoid the overfitting problem and then will compare it with the normal model. Additionally, I am planning to create an interactive GUI for the working of this model.

Okay! So you are planning to create a web app for the same. Then please follow the project structure as,
Project Dummy
|- Dataset
  |- dataset.csv
  |- README.md
|- Images
  |- img1.png
  |- img2.png
  |- img3.png
|- Model
  |- project_dummy.ipynb
  |- README.md
|- Web App
  |- app.py
  |- templates
  |- demo.mp4
  |- README.md
|- requirements.txt

For your reference you can check out these projects which are already in this repo, Brain Tumor Detection, Alzheimers Detection.
Check out the Web App README template here.
Check out the project README template here.
@deedGhost looking forward to hearing from you.

@abhisheks008 Okay sir! Thank you for the related materials. I will start working on it.

Assigned to you @deedGhost

Hey @abhisheks008
I want to ask which images do I need to add in the Images folder, the ones I used to test the model or something else??

Hey @abhisheks008 I want to ask which images do I need to add in the Images folder, the ones I used to test the model or something else??

The EDA results, accuracy graphs, confusion matrix so on. All the resultant images that you have generated while developing the project. Also these EDA results will gonna be a part of the README too.


[question] ram version of gym atari

Hi Antonin,
Need a pointer to run something as simple as
env_id = 'MsPacman-ramNoFrameskip-v0'
env = make_atari_env(env_id, num_env=4, seed=0)
env = VecFrameStack(env, n_stack=4)
model = PPO2(MlpLnLstmPolicy, env, verbose=0)
model.learn(total_timesteps=training_length)

but that did not work
I probably need to unpack the make_atari_env, so when I look under the hood I see
env = gym.make(env_id)
env = NoopResetEnv(env, noop_max=30)
env = MaxAndSkipEnv(env, skip=4)
env = wrap_deepmind(env, episode_life=True, clip_rewards=True, frame_stack=True)
env = DummyVecEnv([lambda: env])

but have not yet been able to get it to work, any pointers?
Thanks

Hello,
I recommend you to read Atari preprocessing.
Also, Atari Ram is quite different as the observation is not an image anymore (see obs definition, it is a vector of size 128) so you cannot use completely the same pre-processing.


Thanks

./gradlew assembleDebug

Please refer to the link.


allowPartiallySucceededBuilds for deployment jobs

Hey Devops Gurus,
Is there to allowPartiallySucceededBuilds for deployment jobs?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 5aeeaace-1c5b-a51b-e41f-f25b806155b8
Version Independent ID: fd7ff690-b2e4-41c7-a342-e528b911c6e1
Content: Deployment jobs - Azure Pipelines
Content Source: docs/pipelines/process/deployment-jobs.md
Product: devops
Technology: devops-cicd-process
GitHub Login: @juliakm
Microsoft Alias: jukullam

It looks like you've a question about the product itself, rather than an issue with the documentation. Consider these resources:

Azure DevOps Services
Azure DevOps Support Bot
Azure DevOps on Stack Overflow


Update doobie-core, doobie-h2, ... to 1.0.0-RC4

About this PR
📦 Updates

org.tpolecat:doobie-core
org.tpolecat:doobie-h2
org.tpolecat:doobie-hikari
org.tpolecat:doobie-postgres
org.tpolecat:doobie-scalatest

from 1.0.0-RC2 to 1.0.0-RC4
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
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 me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.tpolecat" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.tpolecat" }
}]

labels: library-update, early-semver-pre-release, semver-spec-pre-release, version-scheme:early-semver, commit-count:1

Superseded by #258.


In PSGallery, this project's link to the repo is 404

If you view this project in the PS Gallery: https://www.powershellgallery.com/packages/OpenAuthenticode/0.2.0 there is a line to the 'Project SIte". Clickin on this generates a 404.

Thanks for the issue, this was fixed with https://github.com/jborean93/PowerShell-OpenAuthenticode/commit/11e21d87d59e75d8d7967ebe7a371347540f1d83 and is just waiting for a new release.


Val metrics (a1, a2, a3) almost 0 and don't improve despite loss converging - DepthFormer

DepthFormer doesn't appear to be training when looking at validation metrics despite the training loss converging - data is being loaded correctly, verified by visualising the depth maps before training loop in IterBasedRunner. Implemented warm_up=True in decode head decode loss but still seeing no improvement.

Yes, I'm confused now. The convergence problems seem not to be solved by a simple warm_up strategy. I observe this issue in all the methods I implement. Moreover, even the official repos of bts/adabins/transdepth suffer a similar problem. Specifically, I think the gradience is vanishing.
Now, a direct solution is to run the script several times. If the loss is converging in the first several steps, the following training stage will be ok. I would appreciate it if anyone can give me a hand to solve this issue.

Here is one possible solution: modify the schedule to make the training stage start with a relatively small learning rate.

Close for now.


[1.8.9] Incorrect behavior on gui sizes on old worlds

Regardless of whether standardsettings.txt has guiScale set or not, the mod always resets gui scale to auto if an old world is loaded first thing after launch.

oh thanks for the issue i know what the problem is here! will be fixed in the next update, idk when that will come out tho

should be resolved in this prerelease:
https://github.com/KingContaria/StandardSettings/releases/tag/v1.2.2-pre1

I can test in a couple hours

I'm getting some really wierd behavior when dying with hardcore after loading up an old world first thing

hmm that might just be terrible vanilla behavior. It looks like vanilla doesn't delete worlds when you hit the delete button. Is this true?

settings are being reset when joining an old world, is this intended?

can you show me your mods folder?

okay so i tested it a bit myself, and it turns out that the issues with sound still playing and the world not getting deleted are a vanilla bug (i tested it without any mods, not even installed fabric) so i assume the timer still playing is just a sideproduct from that bug and has nothing to do with standardsettings.
i cannot however reproduce the settings being changed on old worlds, are you sure you dont still have an older version of standardsettings in your mod folder?

https://bugs.mojang.com/browse/MC-30646

Yeah I can't reproduce settings being changed on an old world, looks good to me!

okay, thanks for your testing its very appreciated! :)


change title on README

commented on the code given review,reviewed pull request,approved changes,comments made before merging.

thanks git helped in making me know how a pull request is reviewed


Setting position fixed

Hi developer, Great plugin.
I am facing one issue. While scrolling page popover should fixed. but it goes up while scrolling.

I'm having the same problem

It keeps fixed to the element that called it (As excepted)

except when that element is within a fixed container, like a fixed menu.

@martinbroos I mentioned this in issue 185 with how I implemented it when in a fixed container such as a menu wrapper.

@atavernor thanks, I will go check if this works in my situation.

@atavernor in your comment you suggest adding the isFixed method. In my case I know for sure it's fixed but how do i make the popover fixed and add the calculations the right way.
I tried this by using the onShow callback:
            onShow: function (element) {                 element.css('position', 'fixed');             }
The popover will be fixed, but as expected the calculations are wrong and it gets positioned offscreen. How did you fix this?

I just added the following to the getElementPosition function to use the position of the container rather than the trigger link as the reference position (but you should note that this was added to the previous version of the plugin before the fixed positioning features were added)
           getElementPosition: function() {
                // If the container is the body or normal conatiner, just use $element.offset()
                var elRect = this.$element[0].getBoundingClientRect();
                // If fixed position
                if  (this.isFixed(this.$element)){
                        var containerRect = this.options.container[0].getBoundingClientRect();
                        return {
                            top: elRect.top,
                            left: elRect.left,
                            width: elRect.width,
                            height: elRect.height
                        };
                }

            .....

@martinbroos If you want,  I can send you my entire modified plugin. I don't want to post it here directly though as I had to make a lot of name spacing changes to get it to work and it could cause confusion if others were to download it. Let me know and I'll email to the address in your profile if thats OK.

@atavernor thanks for the offer but I don't want to use a custom version of the plugin I would rather extend it so it can still be updated. Isn't this a good addition for the plugin? That their would be a setting for displaying it fixed.

@martinbroos I wasn't suggesting that you use the modified one - just that you could more easily see  what I did by seeing the whole thing rather than pasted snippets. The offer is still open if it helps.
I have not had time recently but I always fully intended to make these mods to the original as a PR.

@atavernor ah oke my mistake, please sent it to me. Maybe I can find some time to make a pr based on your changes. Thanks!

This feature seems very useful with the plugin, I'm looking forward to your pr and merge to the master branch.

Please, that popover should not be fixed by default! Like it was mentioned:
"It keeps fixed to the element that called it (As excepted)"
Keep in mind where the element should be created.

You can fix this by using the container setting. This should contain a selector of the container element that is fixed. By default this is the body tag.
It's not possible at this moment to auto detect if a popover trigger is inside of a fixed container, this would still be an nice addition. But most of the time you know which elements are fixed and have popover child so then initialize the plugin again with the container setting should get what you want.

 $('#txtLoginMail').webuiPopover({
        container: ".pop",  <<<<<<<<<<<<<<     Definitely add it
        trigger: 'manual',
        closeable: true,
        arrow: true,
        placement: 'bottom-left',
        width: 200
    });


Text Utility ReactAPP

Add a new project to the list
This is a react app which provides text utility feature like :
UPPERCASE TEXT
LOWERCASE TEXT
REMOVE SPACES
COPY TEXT
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

Looks Good 🎉


Add Wiki link to the help command

As per the moment only planning to add the link to the General link command.
Could add the "per topic" links, but as per the moment they do not exist.

Added links for:
Wiki
Issues
Repo

I will reduce the space between Miscellany and Relevant links from 3 spaces to 2

Fixed in commit: https://github.com/OriolFilter/Steam_Lobby_Discord_Bot/commit/732f772616227a4c8a7ec06efc4259c32269534f

Proceeding with test deployment before closure of the issue.

Deployment successful. Proceeding with closure.


Would you like to release the tutorials about how to train a new cogview on other language?

I want to train a new cogview with small size on other language, I found the pretrain script, and has some questions:

how to use vae to get the vqvae_hard_biggerset_011.pt?
how to generate the pretrain file ali_vqvae_hard_biggerset_011.lmdb? And would you like to release the ali_vqvae_hard_biggerset_011.lmdb in the future?
Thanks a lot

The training scripts of vqvae will be released after reformatting. 2. You can use vqvae_hard_biggerset_011.pt to compress images into image tokens, but a tutorial and example dataset will be release after some days.

@Sleepychord Thanks for quick response. Got it


⚠ ESPN has degraded performance

In 78d78fa, ESPN (https://espn.com) experienced degraded performance:

HTTP code: 200
Response time: 1445 ms

Resolved: ESPN performance has improved in baeb528.


Enhancement request: QOS setting in the NOTI->MQTT

Hi guys
I still beleive this is the absolute BEST integration out there for MQTT, the others provided are way too narrow, this is done 'the right way'.
One thing that I'm missing seems to be a way to set QOS for the MQTT messages.
The QOS can be 0, 1 or 2, and describes how much a guarantee there is for the message to be delivered.

@fribse

I still beleive this is the absolute BEST integration out there for MQTT, the others provided are way too narrow, this is done 'the right way'.
One thing that I'm missing seems to be a way to set QOS for the MQTT messages.
The QOS can be 0, 1 or 2, and describes how much a guarantee there is for the message to be delivered.

This should be set for each message In Dictionary or as one setting in config?
I suppose one setting in config would be better.

Well, it is usually settable per message type, as it's different what kind of priority the messages get.
But I don't think it matters that much.

@fribse could you share your notiDict and mqttDict that you are using while integrating it to HA with mmm-remotecontrol? I would add it as an example here

Yes of course.
I'm not really clear in my head (suffering from Covid-19), so please bear with me, if I'm doing it wrong.
But a lot of work went into this, and you are a gem for helping me out with the syntax corrections, so of course I will help out with some working example.
First of the config.js part:
    {
      module: 'MMM-MQTTbridge',
      disabled: false,
      config: {
        mqttServer: 'mqtt://mqrruser:mqttpassword@mqtthost:1883',
        mqttConfig:
        {
          listenMqtt: true,
          interval: 2000,
        },
        notiConfig:
        {
          listenNoti: true,
          ignoreNotiId: ['CLOCK_MINUTE', 'NEWS_FEED'],
          ignoreNotiSender: ['clock','newsfeed','currentweather','calendar']
        }
      }
    }

Then the mqttDictionary.js
var mqttHook = [
    {
      mqttTopic: "cmnd/bathroom/mirror",
      mqttPayload: [
        {
          payloadValue: "MONITOROFF",
          mqttNotiCmd: ["MONITOROFF"]
        },
        {
          payloadValue: "MONITORON",
          mqttNotiCmd: ["MONITORON"]
        },
        {
          payloadValue: "UNDIM",
          mqttNotiCmd: ["UNDIM"]
        },
        {
          payloadValue: "DIM",
          mqttNotiCmd: ["DIM"]
        },
      ],
    },
  ];

var mqttNotiCommands = [
    {
      commandId: "MONITOROFF",
      notiID: 'REMOTE_ACTION',
      notiPayload: {action: 'MONITOROFF'}
    },
    {
      commandId: "MONITORON",
      notiID: 'REMOTE_ACTION',
      notiPayload: {action: 'MONITORON'}
    },
    {
      commandId: "DIM",
      notiID: 'REMOTE_ACTION',
      notiPayload: {action: 'BRIGHTNESS', value: '50'}
    },
    {
      commandId: "UNDIM",
      notiID: 'REMOTE_ACTION',
      notiPayload: {action: 'BRIGHTNESS', value: '255'}
    },
  ];

module.exports = { mqttHook,  mqttNotiCommands};

Then there is the notiDictionary.js
var notiHook = [
  {
    notiId: "USER_PRESENCE",
    notiPayload: [
      {
        payloadValue: '1',
        notiMqttCmd: ["SCREENON"]
      },
      {
        payloadValue: '0',
        notiMqttCmd: ["SCREENOFF"]
      }
    ]
  }
];
var notiMqttCommands = [
  {
    commandId: "SCREENOFF",
    mqttTopic: "state/bathroom/mirror/monitor",
    mqttMsgPayload: "OFF"
  },
  {
    commandId: "SCREENON",
    mqttTopic: "state/bathroom/mirror/monitor",
    mqttMsgPayload: "ON"
  }
];

module.exports = { notiHook, notiMqttCommands };

In Home Assistant I've built a sensor and a switch.
The binary sensor reflects the state of the mirror, and the switch turns the monitor on/off
binary_sensor:
  - platform: mqtt
    name: "MagicMirror Bathroom"
    state_topic: 'state/bathroom/mirror/monitor'
    device_class: window

The switch uses two images to show the state of it.
switches:
  - platform: template
    switches:
      magicmirror_bathroom:
        friendly_name: "Magicmirror"
        value_template: "{{ is_state('binary_sensor.magicmirror_bathroom', 'on') }}"
        turn_on:
          service: mqtt.publish
          data:
            topic: "cmnd/bathroom/mirror"
            payload: "MONITORON"
        turn_off:
          service: mqtt.publish
          data:
            topic: "cmnd/bathroom/mirror"
            payload: "MONITOROFF"
        entity_picture_template: >-
          {% if is_state('binary_sensor.magicmirror_bathroom', 'on') %}
            /local/images/mirror_on.png
          {% else %}
            /local/images/mirror_off.png
          {% endif %}

The images are placed in \config\www\images

Hi, i  took over the ownership of the module and started with a new version today.
The new version will support:

Setting the type of session (clean or not)
A default QoS level for publishing and subscribing
Setting QoS level for publishing and subscribing based on the topic
A default retain setting for publishing
Setting retain config for publishing for each message
All other options the MQTT library supports for subscribing and publishing

As of version 2.0.0 of the module QoS, retain flags (and many more things) are supported now. Please see the release notes for details.


Phase set information

I would like to phase and impute my dataset with shapeit5. I originally planned on using shapeit4, incorporating read-supported phase sets as calculated by whatshap4. (See #13 from https://odelaneau.github.io/shapeit4/#documentation). I have a diverse WGS dataset, and being able to phase singletons/rare variants would be beneficial.
Does Shapeit5 use phase sets, like Shapeit4? If so, is there anything I need to do to ensure it uses that information?

Sounds good @odelaneau. I'm working on using shapeit5 in several projects, including phasing the 1000 genomes project samples. Speed is less of a concern for these projects than for yours!
Related question: I have noticed that shapeit4 has different default settings for several common parameters. (I am away from my computer at the moment, but I can provide a list of these parameters when I get back.) For a data set of either several hundred or several thousand samples, would you recommend using shapeit4's default settings? Or even simply using shapeit4 to perform the initial phasing?
-Joe


Inspector with a MultiNodeEdit switches to show a single node after "play" or "stop"

Godot version:
3.0.2 (Steam)
OS/device including version:
Linux x64; X11; Debian Testing
Issue description:
If multiple items are highlighted in the scene tree, and thereby the inspector shows a MultiNodeEdit, playing the scene or project (or stopping one already running) causes the inspector to change to show the settings for only a single node (that which has the border highlight).
Fortunately, going back one step in the inspector history allows one to quickly return to the previous MultiNodeEdit.
Steps to reproduce:

Load or create any scene with more than one node
Select multiple nodes in the scene tree
Observe that the Inspector is displaying a MultiNodeEdit
Click "Play Scene", or press F6
Observe that the Inspector is no longer displaying a MultiNodeEdit

I can't reproduce this in 3.2.1, so I'll assume this issue has been fixed since. Please comment if you can still reproduce this.


Errors should be more explicit. Closes #44.

With this branch:
$ python foo.py
Traceback (most recent call last):
  File "foo.py", line 13, in <module>
    response = client.search('San Francisco', term=u'ș'.encode('utf8'))
  File "/nail/home/kmitton/pg/other/yelp-python/yelp/endpoint/search.py", line 44, in search
    self.client._make_request(SEARCH_PATH, url_params)
  File "/nail/home/kmitton/pg/other/yelp-python/yelp/client.py", line 48, in _make_request
    return self._make_connection(signed_url)
  File "/nail/home/kmitton/pg/other/yelp-python/yelp/client.py", line 54, in _make_connection
    self._error_handler.raise_error(error)
  File "/nail/home/kmitton/pg/other/yelp-python/yelp/errors.py", line 107, in raise_error
    response
yelp.errors.InvalidParameter: {'msg': 'Bad Request', 'text': u'One or more parameters are invalid in request: term', 'code': 400, 'id': u'INVALID_PARAMETER'}

Compared to master:
...
yelp.errors.InvalidParameter:

tox green on 2.7 and 3.4 with my credentials in tests/json/credentials_secret.json for integration testing, and relatively-green without those creds.

@virgytam gave a ship-it for this branch in #47 .


[Misc]: How is the continous batching feature of vLLM implemented?

Hi, I'm curious about the imlementation of continous batching.
It is not mentioned in detail in the vLLM paper, and the code can only use this feature, but there is no detailed code on exactly how it is implemented. Is it a serial execution of the attention layer like Orca, or is it implemented like collapsing all requests within a batch into 1 dimension, then using tree-attention-mask to control valid attention score? Thank you very much!

right！I also wonder how vllm implement orca. And when we batch different requests, how to async get respective generation results


highlight known issue on exchange sst

https://github.com/vesoft-inc/nebula-exchange/issues/71
https://discuss.nebula-graph.com.cn/t/topic/8438/18?u=wey

It seems repartitionWithNebula: False is with issues, we should highlight the known issue/guide user to turn it on faster before we fixed/changed default value.
quoted from second URL in above

4.需要在-c 的exchange.conf配置文件里，强制指定： repartitionWithNebula: true，文档位置：配置参数 - Nebula Graph Database 手册
这里有bug，具体信息参考：exchange 3.0 的生成的sst只包含不带tagID的数据 1
https://github.com/vesoft-inc/nebula-exchange/issues/71
https://github.com/vesoft-inc/nebula-exchange/pull/49 1

We do not provide the function of generating local SST files. Currently, we only support uploading to HDFS after generation. It is not clear whether the problem is caused by this process, we need @Nicole00 to help confirm.

The parameter repartitionWithNebula  defaults to true and the documentation has been updated


bug (istio-operator): revisioned IstioOperator resources affect non-control-plane resources

Bug description
When creating, updating, or deleting an IstioOperator resource with a revision, non-control-plane and non-revisioned resources are affected, e.g. ingressgateways are upgraded/downgraded/deleted
[ ] Docs
[x] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[x] User Experience
[ ] Developer Infrastructure
Expected behavior
IstioOperator resources with a revision should only install/manage control-plane resources, not the entire Istio installation.
Steps to reproduce the bug
istioctl install --set tag=1.5.8

istioctl operator init

kubectl apply -f - <<EOF
apiVersion: install.istio.io/v1alpha1
kind: IstioOperator
metadata:
  namespace: istio-system
  name: canary-control-plane
spec:
  profile: default
  revision: canary
EOF

# wait for istio-operator to finish reconciling

# notice that the ingressgateway deployment was updated to 1.6.8

kubectl -n istio-system delete istiooperator canary-control-plane

# wait for istio-operator to finish reconciling

# notice that the ingressgateway deployment and pods were deleted

Version (include the output of istioctl version --remote and kubectl version and helm version if you used Helm)
$ istioctl version --remote
client version: 1.6.8
pilot version: 1.5.8
pilot version: 1.5.8
istiod version: 1.6.8
data plane version: 1.6.8 (1 proxies)

$ kubectl version
Client Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.0", GitCommit:"9e991415386e4cf155a24b1da15becaa390438d8", GitTreeState:"clean", BuildDate:"2020-03-26T06:17:09Z", GoVersion:"go1.14", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.2", GitCommit:"52c56ce7a8272c798dbc29846288d7cd9fbae032", GitTreeState:"clean", BuildDate:"2020-04-30T20:19:45Z", GoVersion:"go1.13.9", Compiler:"gc", Platform:"linux/amd64"}

I now realize that a more correct way to create a canary control plane is with the following config:
apiVersion: install.istio.io/v1alpha1
kind: IstioOperator
metadata:
  namespace: istio-system
  name: canary-control-plane
spec:
  profile: empty
  revision: canary
  components:
    pilot:
      enabled: true

Instructing the operator to only manage the pilot resources creates a more true canary as it doesn't affect any existing installed resources. If the documentation was more specific in this regard it would have prevented a lot of trial and error on my part.

https://istio.io/latest/docs/setup/upgrade/#data-plane

Unlike istiod, Istio gateways do not run revision-specific instances but are instead in-place upgraded to use the new control plane revision.

I think this indicates what you are looking for, but I agree docs should provide more details and examples like you tried https://github.com/istio/istio/issues/26443#issuecomment-673612789
cc: @istio/wg-docs-maintainers

@shamsher31 thanks, I fully understand now. The issue that led to my misunderstanding was that on the 1.6 version of that page, there is no mention of the gateways: https://istio.io/v1.6/docs/setup/upgrade/#data-plane

I still think it's probably worth calling out in the documentation that you can create a revisioned IstioOperator from the empty profile which only installs the pilot component to achieve a true touches-the-control-plane-only revision.

I still think it's probably worth calling out in the documentation that you can create a revisioned IstioOperator from the empty profile which only installs the pilot component to achieve a true touches-the-control-plane-only revision.

@istio/wg-docs-maintainers PTAL.

Here's the current recommended way to handle this issue: https://preliminary.istio.io/latest/docs/setup/upgrade/gateways/

@frankbu that's excellent, thanks!


bug related to ```

Hello,
Is it possible to add escaping for the  ```  character within the code? Otherwise, a bug occurs, and the formatting breaks. Additionally, since Telegram has a 4096-character limit and I send messages in two parts when this limit is exceeded, could the library include a feature to automatically split the text while preserving the formatting?
sample code:
print("```")

Thank you.

I don't see any issues happening, can you give me more information?

Since our splitting is done by another library, and in order not to destroy the simplicity of this library (not to introduce more complex modules or cause disaster), I think you can write a helper function yourself, splitting by newline and calculating the message.

Since our splitting is done by another library, and in order not to destroy the simplicity of this library (not to introduce more complex modules or cause disaster), I think you can write a helper function yourself, splitting by newline and calculating the message.

Is this a private library? It's difficult to write my own function, if you just cut by the new line, the tags of the code blocks will break. And other large blocks, especially nested ones.

we use https://github.com/miyuchina/mistletoe parse markdown text

What do you use to cut a block of code that is more than 4k characters?

I posted to https://telegra.ph/api

What do you use to cut a block of code that is more than 4k characters?

https://github.com/eternnoir/pyTelegramBotAPI/blob/master/telebot/util.py#L330

What do you use to cut a block of code that is more than 4k characters?

https://github.com/eternnoir/pyTelegramBotAPI/blob/master/telebot/util.py#L330

It doesn't support markdown.

from typing import List, Optional, Tuple
import re

def chunk_markdown(text: str, max_chunk_size: int = 3000) -> List[str]:
    """
    Split markdown text into chunks of specified maximum size while preserving code blocks.

    Args:
        text: Input markdown text
        max_chunk_size: Maximum allowed size of each chunk

    Returns:
        List of markdown text chunks
    """
    chunks: List[str] = []
    current_chunk: List[str] = []
    current_size: int = 0
    in_code_block: bool = False
    code_block_header: Optional[str] = None
    pending_code_block_end: bool = False

    # Split text into lines while preserving empty lines
    lines: List[str] = text.splitlines(keepends=True)

    def add_chunk() -> None:
        """Helper function to add accumulated lines as a new chunk."""
        nonlocal current_size
        if current_chunk:
            chunks.append(''.join(current_chunk))
            current_chunk.clear()
            current_size = 0

    def is_code_block_start(line: str) -> Tuple[bool, Optional[str]]:
        """
        Check if line is a code block start and extract its header.

        Returns:
            Tuple of (is_start: bool, header: Optional[str])
        """
        stripped = line.lstrip()
        if stripped.startswith('```'):
            return True, line.rstrip()
        return False, None

    def is_code_block_end(line: str) -> bool:
        """Check if line is a code block end."""
        return line.lstrip().startswith('```') and not line.lstrip()[3:].strip()

    for i, line in enumerate(lines):
        line_size = len(line)
        next_line = lines[i + 1] if i + 1 < len(lines) else None

        # Handle code block boundaries
        if not in_code_block:
            is_start, header = is_code_block_start(line)
            if is_start:
                in_code_block = True
                code_block_header = header
                pending_code_block_end = False
        else:
            if is_code_block_end(line):
                in_code_block = False
                code_block_header = None
                pending_code_block_end = False

        # Start new chunk if current would exceed max size
        if current_size + line_size > max_chunk_size and current_chunk:
            if in_code_block and not pending_code_block_end:
                current_chunk.append('```\n')  # Close code block in current chunk
                pending_code_block_end = True
            add_chunk()
            if in_code_block and code_block_header:
                current_chunk.append(f'{code_block_header}\n')  # Reopen code block in new chunk
                pending_code_block_end = False
            current_size = line_size
        else:
            current_size += line_size

        # Skip adding closing tag if it's already pending
        if pending_code_block_end and is_code_block_end(line):
            continue

        current_chunk.append(line)

    # Add final chunk
    add_chunk()

    return chunks

Maybe we can implement a tokenizer internally. I'll look into how to split the parser blocks.

This problem has been completely solved: https://github.com/sudoskys/telegramify-markdown/blob/pypi_0.2.0/playground/telegramify_case.py


[zh-TW] Suggest that "Configure" be translated into "組態".

Suggestion

English String: Configure
Current Translation: 設定
Suggested Translation: 組態

There's a corresponding issue that I posted in vscode-loc: https://github.com/microsoft/vscode-loc/issues/1309
Version

CMake Tools Version: v1.14.24

Screenshots

@csigs Could you take a look at this?

@hwhsu1231 After an internal review and discussing with our localization team, the decision has been made that the current translation is appropriate.
Thank you for your suggestion, but we are closing this issue as "won't fix", based on our internal review results.
Thanks again!


Implement update functionality for VPCEndpoint resource

Is your feature request related to a problem?
ec2-controller supports Create/Delete functionality for VPCEndpoint resource, but not Update.
Describe the solution you'd like
I should be able to modify/update an ACK-managed VPCEndpoint resource using its controller
Describe alternatives you've considered
Updating resource myself using aws-sdk, cli, or console

ModifyVpcEndpoint is an ignored operation in generator.yaml.
Note, resource will not be in ACK.ResourceSynced condition if users do not provide a policyDocument in the Spec because AWS will set a default policyDocument if not provided. The resulting diff will trigger update (not implemented)

@brycahta sounds like policyDocument should be lateInitiailized?

/reopen

/reopen

sdkUpdate returns ackerr.NotImplemented

/lifecycle frozen

Do we know what the comment in the unimplemented sdkUpdate(...) fn refers to?
	// TODO(jaypipes): Figure this out...

Is there something tricky here to be figured out? If so, what is the context?

@cannonpalms I was going to reply "that is the generated code when update operations are too complex to handle", but looks like it's that AND we probably have a code-generator bug https://github.com/aws-controllers-k8s/code-generator/blob/14cef51778d471698018b6c38b604181a6948248/templates/pkg/resource/sdk_update_not_implemented.go.tpl#L1-L10


CSP and the forbidden usage of the eval function

We are running the latest version of Jaxon successfully.
After applying a secure CSP set of rules, a problem popped up.
The execution of JS is done via the JS eval function.
CSP can be configured to allow the eval function, but that is considered to be less secure.
We have tested an alternate way, by using the "new function" instead.
But CSP is clever enough to block these calls as well.
Losening the CSP is making the calls to work again, but this does not fullfill security requirements.
Is this on the scope or is there an alternative way to cope with these requirements?

Hi,
I tried, but I couldn't find a trivial solution to replace calls to eval() on the library.
The issue is when you need to access an attribute with dot, let say for example css.color.
You cannot simply use brackets because let color = element[css.color] would be an incorrect replacement for eval("var color = element.css.color").
The library requires to do that for expressions with an arbitrary number of dots, to read or to assign an attribute value, and even with the element name itself stored in a variable.
There are also cases where the raw js code is sent from the PHP app.
Maybe a solution could be to be able to dynamically create js functions directly from the PHP library, which is not possible now since the content of the response to ajax requests is json formatted.

Another idea could be to dynamically add

Hi,
It's indeed a tough issue. The whole purpose of the script (and call) function is to call JS.
Ans that's kind of forbidden if you want to enforce CSP (which definitely makes sense).
An idea for jaxon might be to generate JS functions (now the call to the function is generated, like jaxon_functionname) and in addition to this, create a more extended version of a function in the prepare phase. Then put an event handler on a piece of html (i.e.
Now the PHP part can do an assign of something to the DIV, which kicks of the event handler which kicks of the original function. Assigning HTML to the DOM is one of the parts which is allowed to do as there is no eval function in this part of jaxon.
This might be working, though not trivial to implement.
In our implementation we have done analysis on all the calls.
It seems we are using a lot of DOM manips via jquery which can be moved easily to client side.
We'll start working on these. Then a few plain functions for which we will try to set event handlers.
We are only using portions of the framework, so not all evel's are touched in our usage.
We will see if we can work around it for now.

An update on the experiments.
The solution in this SO https://stackoverflow.com/questions/359788/how-to-execute-a-javascript-function-when-i-have-its-name-as-a-string
Has been tested in the call function.
If we replace
eval(code.join(''));
By
executeFunctionByName(command.func, window, command.data);
The whole thing works.
Being able to use calling of a function + parameters is already a big step forward.

Hi,
I just made the first commit.
https://github.com/jaxon-php/jaxon-js/commit/039faae14d18d66b43d647b5c92a45288a79a65b
It appears that the css object, which properties can be assigned using a PHP call like $response->assign('element_id', 'css.color', 'blue') is actually named style in the DOM element.
With that additional change, the above solution works. I tested on Chrome and Firefox.
See https://github.com/jaxon-php/jaxon-js/blob/039faae14d18d66b43d647b5c92a45288a79a65b/src/tools/dom.js#L119
However, this change only covers cases when the js function already exists. For the functions in https://github.com/jaxon-php/jaxon-js/blob/v4.x/src/cmd/script.js, we also need to be able to dynamically create the js function from the code received in the ajax response.
I'm thinking of adding a <script> tag in the page code in js.


[API] Valid player Id with no 'records' returns wrong error message

Hello
While using the API I've stumbled upon a wrong error message. When you have a tracked user, who has no records the API call will result in Invalid player id.
Example Id: 5827 (Tripplemarty)
Expected result: Empty array

My fault. I gave in Id instead of PlayerId.


Fix search bar / Fix spaces & coding style

Fix Searchbar

Searchbar should looking nice on all resolutions (up to 1080p, could not test to 4k)

Fix spaces & coding style

I noticed that this CSS does contain different coding styles, fixed that (changed to the most used style)
I also noticed that many spaces are wrong placed, fixed that

nice, thanks @Nitair


[W5.11][T15-A3]Siva Subbiah

I have made the find command case insensitive and updated the user guide and test cases( Improvement 1)

Don't forget to update FindCommand's COMMAND_USAGE as well!


vgo does not work on mac os x

What version of Go are you using (go version)?

$ go version
go version go1.12.4 darwin/amd64

go version go1.12.4 darwin/amd64 vgo:devel +b0a1c5df98
Does this issue reproduce with the latest release?
What operating system and processor architecture are you using (go env)?
go env Output
$ go env

I have followed this URL to try the vgo:
https://research.swtch.com/vgo-tour
➜  hello vgo run hello.go
# command-line-arguments
runtime.strequal: relocation target runtime.memequal not defined for ABIInternal (but is defined for ABI0)
runtime.cgocall: relocation target runtime.asmcgocall not defined for ABIInternal (but is defined for ABI0)
runtime.cgoCheckWriteBarrier: relocation target runtime.systemstack not defined for ABIInternal (but is defined for ABI0)
runtime.cgoCheckTypedBlock: relocation target runtime.systemstack not defined for ABIInternal (but is defined for ABI0)
runtime.cgoCheckUsingType: relocation target runtime.morestackc not defined for ABI0 (but is defined for ABIInternal)
runtime.(*TypeAssertionError).Error: relocation target runtime.memequal not defined for ABIInternal (but is defined for ABI0)
runtime.panicwrap: relocation target internal/bytealg.IndexByteString not defined for ABIInternal (but is defined for ABI0)
runtime.memhash: relocation target runtime.aeshash not defined for ABIInternal (but is defined for ABI0)
runtime.(*itab).init: relocation target runtime.memequal not defined for ABIInternal (but is defined for ABI0)
runtime.lock: relocation target runtime.procyield not defined for ABIInternal (but is defined for ABI0)
runtime.notesleep: relocation target runtime.asmcgocall not defined for ABIInternal (but is defined for ABI0)
runtime.notetsleep_internal: relocation target runtime.asmcgocall not defined for ABIInternal (but is defined for ABI0)
runtime.(*mheap).sysAlloc: relocation target runtime.memmove not defined for ABIInternal (but is defined for ABI0)
runtime.mallocgc: relocation target runtime.publicationBarrier not defined for ABIInternal (but is defined for ABI0)
runtime.mallocgc: relocation target runtime.memclrNoHeapPointers not defined for ABIInternal (but is defined for ABI0)
runtime.mallocgc: relocation target runtime.systemstack not defined for ABIInternal (but is defined for ABI0)
runtime.persistentalloc: relocation target runtime.systemstack not defined for ABIInternal (but is defined for ABI0)
runtime.persistentalloc1: relocation target runtime.morestackc not defined for ABI0 (but is defined for ABIInternal)
runtime.typedmemmove: relocation target runtime.memmove not defined for ABIInternal (but is defined for ABI0)
runtime.typedslicecopy: relocation target runtime.memmove not defined for ABIInternal (but is defined for ABI0)
runtime.memclrHasPointers: relocation target runtime.memclrNoHeapPointers not defined for ABIInternal (but is defined for ABI0)
/usr/local/go/pkg/tool/darwin_amd64/link: too many errors

At this point, rather than using vgo, a much better option is to just use Go 1.12 with modules enabled.
Also, a better introduction is now https://blog.golang.org/using-go-modules, which was partially adapted from that older vgo-based introduction you were reading.
See also #27563, including https://github.com/golang/go/issues/27563#issuecomment-429000191 which states:

Once Go 1.12 is out, Go 1.10 will be deprecated, at which point all supported versions of Go will have module support. Then we can remove vgo entirely (maybe a month or so after the Go 1.12 release).

Yeah, vgo is no longer supported. It served its purpose, but at this point just use go.


Review/refactor authentication logic for octokit

I may be wrong but the API requests using octokit don't seem to be properly authenticated.
See https://octokit.github.io/rest.js/v19:
new Octokit({
auth: "secret123",

Even if current approach is correct (see https://github.com/project-stacker/stacker-build-push-action/blob/main/src/installer.ts#L23), authentication seems to be set only for that specific call, not on the else branch/

Note the bug manifests in failed GitHub calls to get the stacker release information (github applies rate limiting for all calls and the limit for unauthenticated calls is low).


Tests broken on master due to missing setuptools.tests.fixtures

Not sure what's going on here. I can replicate this locally only after doing git clean -idx followed by a new python boostrap.py.

I can reproduce the issue. tox-3.2.1 is the problem, latest successful Travis build was using 3.1.2.

It's something about the way tox 3.2 creates the venv; because after creating the env, invoking .tox/python/bin/pytest manually, replicates the error.

For now we can pin to 3.1.2 (#1454) until we can figure out what's going on.
I'm guessing the issue is that tox is starting to test by default against an as-installed version of the packages that it uses, rather than as-seen-by-the-repo-root. Since we don't install the tests, setuptools.tests.fixtures shouldn't be importable in that case.
I imagine the solution is to start using some sort of relative imports to pull in the fixtures, or something like that.

This is interesting. The tests only fail if invoked with the console script (pytest), but don't fail if invoked by runpy (python -m pytest).

I think it's because in the newer tox, pip is called with python -m pip, and this result in setuptools being already present in the environment.

@jaraco That strengthens my suspicion. If you invoke pytest directly, it deliberately removes the current working directory from your PYTHONPATH. python -m pytest doesn't do that for whatever reason.

> ls -d .tox.good/python/lib/python3.7/site-packages/setuptools*
.tox.good/python/lib/python3.7/site-packages/setuptools.egg-link
.tox.good/python/lib/python3.7/site-packages/setuptools.pth
> ls -d .tox.bad/python/lib/python3.7/site-packages/setuptools*
.tox.bad/python/lib/python3.7/site-packages/setuptools/
.tox.bad/python/lib/python3.7/site-packages/setuptools.egg-link
.tox.bad/python/lib/python3.7/site-packages/setuptools-39.0.1.dist-info/
.tox.bad/python/lib/python3.7/site-packages/setuptools.pth

@pganssle: This is to be expected, no? https://docs.python.org/3/library/sys.html?highlight=sys path#sys.path

Similarly the pip uninstall command behaves differently whether invoked by script or runpy:
setuptools master $ .tox/python/bin/python -m pip uninstall setuptools
Uninstalling setuptools-40.0.0.post20180811.post-20180811:
  Would remove:
    /Users/jaraco/Dropbox/code/main/setuptools/.tox/python/bin/easy_install
    /Users/jaraco/Dropbox/code/main/setuptools/.tox/python/bin/easy_install-3.7
    /Users/jaraco/Dropbox/code/main/setuptools/.tox/python/lib/python3.7/site-packages/setuptools.egg-link
Proceed (y/n)? ^COperation cancelled by user
You are using pip version 10.0.1, however version 18.0 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
setuptools master $ .tox/python/bin/pip uninstall setuptools
Uninstalling setuptools-39.0.1:
  Would remove:
    /Users/jaraco/Dropbox/code/main/setuptools/.tox/python/bin/easy_install
    /Users/jaraco/Dropbox/code/main/setuptools/.tox/python/bin/easy_install-3.7
    /Users/jaraco/Dropbox/code/main/setuptools/.tox/python/lib/python3.7/site-packages/easy_install.py
    /Users/jaraco/Dropbox/code/main/setuptools/.tox/python/lib/python3.7/site-packages/pkg_resources/*
    /Users/jaraco/Dropbox/code/main/setuptools/.tox/python/lib/python3.7/site-packages/setuptools-39.0.1.dist-info/*
    /Users/jaraco/Dropbox/code/main/setuptools/.tox/python/lib/python3.7/site-packages/setuptools/*
Proceed (y/n)? ^COperation cancelled by user
You are using pip version 10.0.1, however version 18.0 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.

I actually think it's a good thing to test against the installed version. I think we should temporarily pin tox, then get our house in order so that the tests can be run against an installed version - maybe move setuptools/tests to a sibling directory of setuptools.
I've had enough bugs in the past that would have been found by testing against the installed version that I prefer to do the tests that way.

I've encountered other issues where testing against a non-development version leads to other problems (such ImportMismatchErrors or other conflicts). My preference would be to continue to test the code as it sits rather than requiring it to go through an install step before each test run (to keep the two copies in sync).

Changing the install/freeze commands is one way to make it work again with newer tox:
install_command={envbindir}/pip install {opts} {packages}
list_dependencies_command={envbindir}/pip freeze

Here's something interesting - after creating a fresh venv, when invoked with runpy, pip doesn't know it has a setuptools installed if the current directory is the working copy.
setuptools master $ python -m venv .tox/env
setuptools master $ ls .tox/env/lib/python3.7/site-packages/
__pycache__                 pip                         pkg_resources               setuptools-39.0.1.dist-info
easy_install.py             pip-10.0.1.dist-info        setuptools
setuptools master $ .tox/env/bin/python -m pip uninstall setuptools
Not uninstalling setuptools at /Users/jaraco/Dropbox/code/main/setuptools, outside environment /Users/jaraco/Dropbox/code/main/setuptools/.tox/env
Can't uninstall 'setuptools'. No files were found to uninstall.

Relevant is python issue31874.

If only there were some way to instruct Python not to add the current directory to sys.path. I don't think that's possible, and #1455 may be our best path forward for now.


The accessory sent an invalid response. Pleas try again.

The Garage control in the "Eve" application gives error "The accessory sent an invalid response. Please try again". I even removed my customized GarageDoorOpener accessory, cleared out the /persist directory, removed the accessory from HomeKit, and added it again and I still have this issue.
Any thoughts on what might be causing this or how to see what is being sent to HomeKit that might be causing the response to be invalid?

I guess Eve is doing its own characteristic validation. I'd suggest you use a different HomeKit app and see what's the actual value HomeKit is receiving

Or enable the debug logging so we can see what's the value by adding "DEBUG=*" before node

It was the iPhone application. I used Apple’s HMCatalog app and had not issue.  Thank you.
On 21-Feb-2016, at 19:43, Khaos Tian<EMAIL_ADDRESS>wrote:
Or enable the debug logging so we can see what's the value by adding "DEBUG=*" before node
—
Reply to this email directly or view it on GitHub https://github.com/KhaosT/HAP-NodeJS/issues/219#issuecomment-186966927.


add mysql2 gem for windows

Dears
I am very thankful for your amazing effort to make rubyinstaller possible by just couple clicks .
we  would love if you could support MySQL by default as it not easy to install it in windows
if you make quick search you will find many are stuck in this step  , me included as i tried many possible solutions and worked with me once and failed many . I think it is very critical for beginners to install MySQL , especially the one who would only focus on learning ruby and rails without worrying about how to install this or that ,Just like using ruby-installer
it would be extremely amazing if Ruby-installer come with per-configured MySQL gem or at least make it easier to do "gem install mysql2" as it with most ruby gems
Thank you

While I understand your frustration, this isn't really a goal of RubyInstaller.
Likewise in this case it is really up to the gem provider to provide "fat binaries" for gems in which they compile the gem themselves and release it. Which the maintainer may or may not do and in some cases simply cannot.
In the case of this particular gem, it looks like @luislavena did some awesome work with the current maintainers and it will have fat binaries next release.

At the moment, if you have a mingw compiler available, the best option in your Gemfile is:
gem 'mysql2', github: 'brianmario/mysql2', ref: '72dedddac9de02b50d44ef0fbea6a99aedac486c'
I will build fat binaries for the next release in the coming months.

I released mysql2 version 0.3.18 with windows binaries today! Please give it a spin, report any issues to https://github.com/brianmario/mysql2.


Allow the failure severity to be configured

This is supposed to be the final PR for #6400 introducing a new severity concept. It allows users to configure the minimum severity that leads to build failures

Warnings

:warning:

It looks like you're editing the un-versioned copy of our website. This affects only users on the 'next' version of detekt, and it's correct only if you intend to document a future change or feature. If you intended to make a change also for the current version of detekt, please make sure you edit the equivalent file inside website/versioned_docs/ as well.

Messages

:book:

Thank you very much for making our website better :heart:!

  Generated by :no_entry_sign: dangerJS against 176ed4b00876cba383d4938afa24fc9a75541dfa

@detekt/maintainers:
Before I finish this up with tests and documentation I would like your take on the ignoreFailures parameter in the DetektExtension (see https://github.com/detekt/detekt/pull/6417/files#diff-76e96187fb9c234b865786caa99de176c851268f7a284a2fd3d79402dd426f27) It used to be declared by the CodeQualityExtension which we no longer extend. This property would collide with the newly introduced failOnSeverity parameter. I see 2 options here:

Keep both parameters and set failOnSeverity to never if ignoreFailures is set to true. Then we should also validate that the failOnSeverity parameter is not set.
Replace  ignoreFailures  with failOnSeverity. I do not know if ignoreFailures is some sort of community standard and should be available.

I vote for option 2. It seems to be the better long term option with lower maintenance.

There's a third option which is my preference:

Keep both parameters, use failOnSeverity to set the severity level that is considered a failure. If there are any issues raised at this level it won't fail the task/build if ignoreFailures is true

They're two different flags with two different features. The only reason we no longer extend CodeQualityExtension was to unblock https://github.com/detekt/detekt/pull/6496 - it wasn't because the properties in CodeQualityExtension were no longer relevant.

There's a third option which is my preference:
* Keep both parameters, use `failOnSeverity` to set the severity level that is considered a failure. If there are any issues raised at this level it won't fail the task/build if `ignoreFailures` is `true`

That is also an option that is easy to implement but what is a use case? Why would a user for example specify failOnSeverity=error and ignoreFailures=true?

True that if there's a "never" option for the new property there wouldn't normally be a reason, though it makes it a little simpler to hook up a way to toggle failures on and off if there's a boolean property available.

I do not know if ignoreFailures is some sort of community standard and should be available.

It's available on all Gradle code quality tasks, test tasks, and the validate plugins task, so it's standard for every task in Gradle core that performs verification.

what is a use case? Why would a user for example specify failOnSeverity=error and ignoreFailures=true?

@marschwar Imagine that the code checked into the repo is error+false, as a default for most developers + CI. But some developers might want to toggle ignoreFailures=true locally, because they have a different process, they could write experimental code, make it work, then clean it up.

what is a use case? Why would a user for example specify failOnSeverity=error and ignoreFailures=true?

@marschwar Imagine that the code checked into the repo is error+false, as a default for most developers + CI. But some developers might want to toggle ignoreFailures=true locally, because they have a different process, they could write experimental code, make it work, then clean it up.

But they would still change the gradle build file. I do not see why changing ignoreFailues to true is any easier than changing failOnSeverity to never.
But I do not want to argue against the approach. I was just curious if I had been missing something. The question I was asking myself was if it helps the users or will it confuse them to have both.

@marschwar is there something blocking this to be merged? What can we do to make this happen?

Thank you @BraisGabin for the reminder. I did just resolve the merge conflicts. I did not plan to make any more additions to this PR. To me it is good to go.

Then, let's merge it! It improves the project :)


Chat response: actions are always visible

Each chat response shows actions, which is quite noisy, I would have expected the actions for a response only to become visible when I have focused on a response or hovered over it, similar to the "X" button a chat request:

We changed this when we moved the toolbar. The problem is that it now takes up space in that position, so if it only appears on hover, it leaves a big empty gap. Also, we removed the border lines between rows, and the toolbar provides a nice visual separation. I wouldn't mind getting rid of the thumbs-up again to slim it down. Also most users won't see the extra 'report issue' button.


Deploy Nuxt 3 sites with universal rendering on Azure Static Web Apps - Misleading and outdated content.

The "Configure dynamic routes" section of this page is misleading and outdated. Users do not need to manually configure all of the potential dynamic routes. Nuxt automatically generates a staticwebapps.config.json file which includes a navigation fallback:
"navigationFallback": {     "rewrite": "/api/server"   }
This sends the pages unknown to SWA (Dynamic pages) back to /api/server which nuxt then successfully handles by rendering the page.
This has caused a lot of confusion for Nuxt developers using SWA and should be updates :)

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: a1b3f061-f523-e60d-9633-92841373318e
Version Independent ID: 87f53c02-bfa8-8d07-b166-5cbc4d5775a1
Content: Tutorial: Deploy Nuxt sites with universal rendering on Azure Static Web Apps
Content Source: articles/static-web-apps/deploy-nuxtjs.md
Service: static-web-apps
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

@TheSirLan
Thanks for your feedback! We will investigate and update as appropriate.

Hi @TheSirLan upon further review we feel your feedback would be best received by the doc author to update this doc as necessary. cc: @craigshoemaker

Thanks for your feedback and your contribution to Azure docs.
Feedback for this repository is moving away from GitHub to a system specific to the Microsoft Learn platform. Issues for this repository will soon be disabled, and additional comments from GitHub will no longer be possible. However, we are now tracking and triaging this issue in the new feedback system.
To learn more about our feedback systems, please see Provide feedback for Microsoft Learn content.
#please-close


Add xlswriter

Describe the feature

Is it possible to add xlswritrer extension?
https://pecl.php.net/package/xlswriter
https://github.com/viest/php-ext-xlswriter

@cevin It already exists for PHP 7.0 and above.
https://github.com/shivammathur/homebrew-extensions#extensions


Adding São Paulo data

Hello! Please just stick to one pull request. You can keep modifying the existing branch and it will update this pull request. No need to close and re-open.

Thanks for this addition.
Are you able to find any license information to link to? Also, can you include some description of what GeoSampa is?

Where can I put description information about GeoSampa? I have tried to put it inside GeoSampa address as a description and the test complained

Where can I put description information about GeoSampa?

You can put it in the description of the pull request if you want.

Where can I put description information about GeoSampa?

You can put it in the description of the pull request if you want.

OK, I have put it into note tag inside Json file. Sorry I did it before read your message, but GeoSampa is the official GeoPortal of the municipality of São Paulo city, maintained by the Coordination of Production and Information Analysis (GeoInfo) of the Municipal Department of Urbanism and Licensing

Got it, thanks!

Hi @iandees,
Do you have any idea why is it taking so long to get processed?
Thanks,

@ingalls it looks like data-pls isn't linking to the right build in the PR here. It looks to maybe be this job in batch? https://batch.openaddresses.io/job/73128

@iandees Looking into this.
@feromes The reason why it isn't processing quickly is that currently it is running on the same AWS Batch instance as our weekly builds in order to save significant $. In my last deploy yesterday I created the inifial infra to run a second job queue running on the same instances with a higher priority which will let CI jobs skip the queue once there is capacity. I haven't finished wiring this up yet.

Superseded by #6949