Add assertEventType helper

I don't mind introducing such a helper but I'm worried that it will give users a false sense of safety. After all - this can throw, how do you respond to the thrown error?

Good point, I wonder if that can be communicated (or is already communicated) in the name and API. I only see this useful when using typescript and in Dev mode.

Maybe an alternative could be a type predicate function, not as ergonomic though IMO


.js file?

Does this run on client-side? or node.js only?
Ex.
<script src="node_modules/prefume.js/dist/prefume.js"></script>

Perfume is only client-side because of window.performance used to calculate the browser metrics.
I just updated the docs, have you tried using the Universal Module Definition
import Perfume from 'node_modules/perfume.js/perfume.umd.js';

PS: The library it's still on working progress ;)

I will give it a try, Thanks.


feat: add copy button

closes https://github.com/denoland/denobyexample/issues/27

If you insist. Does this look good?

@crowlKats PTAL!


Dist code was not included in the 1.0.4 release.

Updating this dependency to latest from npm broke our moov related code. Diving into the node_modules it looks like the dist directory was not included. Had to roll back to 1.0.3 for the time being.

up 👍 ! 🙏

Fixed in the v1.0.5 release. Thank you for your patience, all.


Update to frontend 0.0.30

Update Design Sytem to use the latest version of govuk-frontend.
Because the SCSS are conflicting, app- prefix to the SCSS exports was added in this PR.

You can preview this change here:
Built with commit b916a44e4c6267dbd7a31451e2b75de11edc99b8
https://deploy-preview-285--govuk-design-system-preview.netlify.com


Valset: better unjail error message when jail lock didn't expire

closes https://github.com/confio/poe-contracts/issues/34

We should back port this to the 0.5.x branch.
Maybe best if we squash PRs that will be back ported, so git cherry-pick becomes easy


Monitor is undetected

it is a Samsung Odyssey Neo g9.
Only source available is "leave unchanged"

Hi Migle,
Thanks for reporting this! This post suggests it might only work over HDMI.
Unfortunately it's up to Samsung to support DDC/CI on the DisplayPort.
Cheers,
Fidel

Ahhh ok makes sense! so I got it to show over HDMI but it shows the incorrect inputs, neither of which work.


a DOMException occurs with the ap.list.switch(0) method

In using your Aplayer project, at the end of the playlist, a DOMException occurs with the ap.list.switch(0) method: "The fetching process for the media resource was aborted by the user agent at the user's request."

Can't reproduce


Why its so laggy when it start and when i press alt + space it lag, not responding

please help me fix this.

@plsAlow
Right-click on PowerToys icon in the tray menu and select Report Bug.

Drag and drop the report into a Github comment.


Broken Link for the index.html

I considered creating an PR but I think its probably faster done with an issue.

The first link Official lnd recovery guide to https://docs.lightning.engineering/lightning-network-tools/lnd/recovery does not work anymore.
I gues the correct "new" link should be:
https://docs.lightning.engineering/lightning-network-tools/lnd/disaster-recovery

Thanks a lot, I updated the link!


Add Sana .pth files into download counting in huggingface repo

Cool!

Lint is finally passing :smile:
I'm merging. Expect 2-3 days before getting it deployed to production.

OHH, Great to know. Thanks a lot for your support and quick apply @Wauplin @Vaibhavs10

Hi experts. Sorry for interrupting. The downloading count does not recover in our collection. For example, https://huggingface.co/Efficient-Large-Model/Sana_1600M_1024px
Could you pls help to check the problem?
Cc: @Wauplin @Vaibhavs10

Hey @lawrence-cj I can confirm to you that this PR has been deployed to production. I can now see 7400+ downloads for https://huggingface.co/Efficient-Large-Model/Sana_1600M_1024px for instance. Is that more in line with what you were expecting?

Oh, thank you so much for it @Wauplin. Actually, I would have expected more as we have 70K downloads of our component models. But that's ok, I'm ok if that's the backend data.


Loading this up on mac and I realized sqlite is hanging render.js

Detection
Oddly enough, I didn't experience a single slowdown during initial development on my Windows machine, but hopping over to a mac with less power and I'm noticing right away, the UI hangs on start and during sqlite actions. Difficult to notice after the initial load, but definitely still there.
Possible solution
Maybe there is a way to lazy load on start or async these actions altogether. Pretty sure I can solve this with another render process, but I don't wanna. 😬

Tested on mac and this proves to be viable. It also seems to have fixed the fontawesome icons, which were not loading even after the db loaded up. Interesting, that cdn is called in html, which I would have thought was already loaded in before render.js even started churning... hmm.


International usage please

Making it so that it is easier to use everywhere @coding-is-fun

This cannot be done


Add Nuxt

Resolves #57
Resolves #86

The goal is to at least match/exceed the current master state.
Btw @FunctionalMetatable try searching WhenFlagClicked on google. It's pretty high up don't you think :)

I'm noticing a problem with the markdown, hmm. Try going to http://localhost:3000/blog/md-tutorial. Also, try making a new tutorial and creating code.

I'm noticing a problem with the markdown, hmm. Try going to http://localhost:3000/blog/md-tutorial. Also, try making a new tutorial and creating code.

fixed

Although apparently searching "when flag clicked" gives different results :P

including an instagram account for some reason lol

I've deployed it to replit
Preview: https://flagclicked-2.9gr.repl.co/

I'd like to note this:
When we use axios, we have to use a baseURL param. This is defaulting to localhost:3000 which makes all requests go to localhost:3000 instead of the proper base URL.
I think we should write a plugin for axios instead of using the default @nuxtjs/axios

I'm guessing axios was fixed?

this period annoys me

these buttons suck

I don't think the auth page should be a studio page since the studio page hurts my eyes
It should be a project instead
(Also, it logged me out after I logged in. I had to log in twice, which was annoying).
The tutorial page is great but the pfp is too far to the left

The buttons in the header need more padding as well.
Code wise, it LGTM.

Suggestions:
add this css to the buttons:

make the editor be atleast 300px
Bugs:
This:

@FunctionalMetatable your prettier settings are making it hard to read commits :(

@FunctionalMetatable your prettier settings are making it hard to read commits :(

I should globally install prettier
anyways everything from cubey and markverb1s suggestions are done except navbar padding

Oh, yeah. We need to hide the my tutorials and new tutorials button in the navbar when the user is logged out and replace it with maybe the about page

Let's also support a bit of mobile since, non nuxt had it, and it's really bad right now

Let's cleanup the footer a bit too. Let's fix the when flag clicked block and add GitHub repo there too. Let's remove the tutorials link there too.

Let's hide "this week's featured tutorials when there is none. Contact us page needs a wip page for now. Let's fix scratch-blocks issues . Oh and we should add the credits page to the footer.

@FunctionalMetatable I want to merge this after I address this problem:

Any idea why this is happening? This error is flooding terminals.

@FunctionalMetatable I want to merge this after I address this problem:

Any idea why this is happening? This error is flooding terminals.

normal with nuxt: babel made this happen and I did a bit of research few days ago and next release will fix this

rip the old https://www.whenflagclicked.org/

I disconnected it from replit, why is it still on a replit screen

I disconnected it from replit, why is it still on a replit screen

the domain is still linked to replit; you just deleted the repl i think

Why does it not work?

I haven't tested admin yet, I need to clear the database somehow

I haven't tested admin yet, I need to clear the database somehow

just put this in server-middleware/server.js right before export default app:
auth.databases.users.update({ username: "TheColaber" }, { $set: { admin: true } })

I'm currently waiting for Fosshost's applications to be unsuspended...

If only I was 2 days faster, now I have to wait 2 weeks later :(
Anyway, lets merge this thing


Server Binary Build Issue with Python 3.12: Awaiting Fix in Tree-Sitter-Languages Repo

Before submitting your bug report

[X] I believe this is a bug. I'll try to join the Continue Discord for questions
[X] I'm not able to find an open issue that reports the same bug
[X] I've seen the troubleshooting guide on the Continue Docs

Relevant environment info
- python 3.12

Description

follow up #579
building the the server binary from source with python 3.12 is currently not working
need to wait for a fix to the issue in the tree-sitter-languages repo

To reproduce
see description
Log output
No response

see https://github.com/grantjenks/py-tree-sitter-languages/issues/30

After the move away from the Python server this issue is no longer applicable

I forgot about it, thanks for closing it.


In combined package yaml files, allow versions to be given with ints or floats

is particularly nice for yaml files, since you can now do, ie:
versions: [1.5]
...instead of:
versions: ['1.5']
...which feels more consistent, since you can also do:
versions [1.5.3]

Hey Paul,
Float support for version numbers was actually removed because it causes a bug.
Specifically: 0.9 !< 0.10 in float, but should be as version number.
Sent from my iPhone

On Sep 21, 2015, at 2:45 PM, Paul Molodowitch<EMAIL_ADDRESS>wrote:
is particularly nice for yaml files, since you can now do, ie:
versions: [1.5]
...instead of:
versions: ['1.5']
...which feels more consistent, since you can also do:
versions [1.5.3]
You can view, comment on, or merge this pull request online at:
https://github.com/nerdvegas/rez/pull/248
Commit Summary
In combined package yaml files, allow versions to be given with ints or floats
File Changes
M src/rez/package_resources_.py (19)
M src/rezplugins/package_repository/filesystem.py (8)
Patch Links:
https://github.com/nerdvegas/rez/pull/248.patch
https://github.com/nerdvegas/rez/pull/248.diff
—
Reply to this email directly or view it on GitHub.

Ah, good point. I'll close this pull request


pull request for e2e test at 12/30/2016 10:56:10

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


chore: refactor L1BlockInfo::tx_estimated_size_fjord

Found tx_estimated_size_fjord in our block-building pipeline's hot path so had a look.

Apparently, it's taking a non-trivial 9% of the cake when State::commit takes 10% and Evm::transact itself only takes 38%.
This PR adds some info and constants around tx_estimated_size_fjord, and micro-optimizes it by removing redundant U256 conversions. 91% of tx_estimated_size_fjord is still spent on flz_compress_len so we may need to optimize the current Rust port, or go with an Assembly binding.

@hai-rise could you please also submit these improvements to the https://github.com/bluealloy/revm/tree/release/v50 otherwise these improvements won't make it into the release


Modify CMakeLists.txt to enable code to compile and run

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Motivation and Context / Related issue

Previously, the code would fail to make due to linker errors about Unicode and PyAlloc and maybe other things.
Now that the lines about Minesweeper and testing have been commented out, it compiles and runs.
How Has This Been Tested (if it applies)

I did a new clone of the repo, checked out this branch, and tried compiling and running, and it works.
Checklist

[ ] The documentation is up-to-date with the changes I made.
[ ] I have read the CONTRIBUTING document and completed the CLA (see CONTRIBUTING).
[ ] All tests passed, and additional code has been covered with new tests.

Merging the PR now. Ideally, we will want to come back to these errors and try to get the tests working relatively soon.


Add test cases for ensuring canary rollout

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

/approve


asdasd

@
- [ ] Turn on GitHub Pages
 -[ ] Outline my portfolio
- [ ] Introduce myself to the world


morley.csv

I want to create csv file for my dataset.

Hi @RobertMelvin, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;


feat: auto create student record on register

:tada: This PR is included in version 1.3.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add regression tests: article LikeCoin donation

Checklist

[ ] Test donation: LikeCoin;

Refs

revision: https://mattersnews.slack.com/archives/C88CK7Q7L/p1673520847520339?thread_ts=1673520666.618439&cid=C88CK7Q7L
donation: https://mattersnews.slack.com/archives/GA0R09FHN/p1673452980150929

should be postponed due to the setup difficulty with headless E2E testing


rtio: Split the rx and tx buffer unions

Transmit/write buffers are expected to be constant values given to the operation to transmit but not mutate. Seperate the OP_TX and OP_RX operation description unions.
Fixes #76353

Seems I need to look at renaming the transceive union members as well

Duplicated union members 'tx_buf' and 'rx_buf'.

good catch!

Grepping for rtio around the tree: nothing missing!
drivers/i2c/i2c_sam_twihs_rtio.c  - ok
drivers/i2c/i2c_mcux_lpi2c_rtio.c - ok
drivers/i2c/i2c_nrfx_twi_rtio.c   - ok
drivers/spi/spi_sam.c             - ok
drivers/spi/spi_mcux_lpspi.c      - ok
include/zephyr/drivers/sensor.h   - no mention of sqe->buf anywhere
include/zephyr/drivers/spi.h      - no mention of sqe->buf anywhere
include/zephyr/rtio/rtio.h        - ok
subsys/rtio/rtio_handlers.c       - ok

An alternative possible, though I have no preference for either! And it does not work well for op_callback.
		/** OP_TX */
		struct {
-			uint32_t tx_buf_len; /**< Length of buffer */
-			const uint8_t *tx_buf; /**< Buffer to write from */
-		};
+			uint32_t buf_len; /**< Length of buffer */
+			const uint8_t *buf; /**< Buffer to write from */
+		} tx;

I really liked this, so I went ahead and put the idea to use here

Moved the tdk board changes to #77098


Hi, is there a pure Matlab version of your code? Unfortunately, I failed to use Mosek for Julia due to some limitations, and I am not familiar with Julia lang.

Sorry, the whole project is written in Julia language. However, it is very straightforward to run the code: download Julia language from here, get your Mosek license here (free for academic use), and simply follow instructions in README.md. Good luck!


Observable.from error in Swift 3 app

I've have an app I'm converting from Swift 2 to Swift 3, using code that's pretty much identical to the discoverCharacteristics idiom from the README:
Observable.from($0.discoverCharacteristics([my_characteristic_uuid]))

Xcode 8.0 is producing the following error against the parameter of Observable.from() in the above:
/path/to/our/project/XferControlMessage.swift:109:43:
Cannot convert value of type 'Observable<[Characteristic]>'
 (aka 'Observable<Array<Characteristic>>') to expected argument type '[_]'

This error reads like the compiler perhaps isn't choosing the from<S: Sequence> variant defined in Observable+Creation.swift?  As I understand, the whole point of Observable.from is to convert Observable<[Characteristic]> into an observable sequence Observable<Characteristic>.  So this definitely seems like either a code or documentation bug, but I'm new enough to Swift, and especially the nuances of Swift 3, that I'm not immediately clear on the root cause here.
This is using RxBluetoothKit 3.0.4 and RxSwift 3.0.0.

Hi,
We've dropped support for our own Observable.from operator, because in 3.0 version RxSwift contains similiar one that is called the same way.
Information in our README.md is not up to date, we've missed it and I will fix it right away.
Now, to fix it if you have specified services/characteristics I would advice you to follow Convenience calling methods section of our README.
To use Observable.from from RxSwift you need to pass to it array of elements, instead of observable on array.
So it would look like:
    .flatMap { $0.discoverServices([serviceId]).flatMap { Observable.from($0) } }```
Or writing it one level lower:

peripheral.connect
.flatMap { $0.discoverServices([serviceId]) }
.flatMap { Observable.from($0) }

This should do the work.
To make it nicer you could write your own extension on Observable, which basically calls `Observable.from($0)` but maybe it would be nicer to have it in more meaningful way.

I'm jumping to README and updating it now. :)

Thanks again and let me know how it works for you :)

It's fixed and in master right now :)

Fantastic, that resolved it. Thanks so much for the prompt reply!  I'll leave off adding an extension until I've gotten more experience with RxSwift & RxBluetoothKit.


Widget dissapears immediately

When opening a hdf5 file, the widget briefly (milliseconds) pops-up, to close again immediately.
Other widgets and interactive elements have no problems.
I just used the sample notebook to replicate the issue.
Tried multiple browsers: FF, Edge, Chrome on Win10.
Tried versions v0.0.10, 9 and 8.
Using jupyterlab v3.0.16

Hi @debbes80, thanks for trying jupyterlab-h5web !
Here are a few diagnostics to run to pinpoint the issue.

Is it working when you double-click on a HDF5 file ?

Could you verify that both the labextension and the server extension are installed and enabled ?
You can do this by running the following commands in a terminal:

jupyter labextension list
jupyter serverextension list

Both should show a line telling that jupyterlab_h5web is enabled and OK.

Can you open the Network tab of the developer consoler (F12 on FF) before opening a file ? When opening a file, do you see any blocked/failed h5web requests (url begins with /h5web/) ?

I get the same effect when double-clicking a file
Yes, both were ok
the request is there and it is also ok

I made a venv with only jupyterlab (tried 3.0.16 and 3.0.10) and jupyterlab_h5web, but still getting the same effect.

Odd, I tried with<EMAIL_ADDRESS>and it works for me but it is on Linux... I will try to do the same on Windows to see if I can reproduce the issue.
In the meantime, we can still push the diagnostics a bit further:

Do you see the message JupyterLab extension jupyterlab-h5web is activated! in the browser developer console ?

What steps do you do to install jupyterlab and jupyterlab_h5web in a fresh environment ? Do you use pip in a terminal or something else ?

By shear luck I managed to find out it was jupyter-server causing the issue. I'm using poetry for my venvs and did an update of the packages before my try. It updated jupyter-server (1.11.0 -> 1.11.1) and the widget works fine.
Looking at the jupyter-server release notes, I don't know if https://github.com/jupyter-server/jupyter_server/pull/587 might be the cause.


Does it support UTF-8 in filenames?

Does it support UTF-8 filenames (regardless of OS version and locale settings)?

You might be interested in wide-character versions of dirent functions.
They work irrespective of the os version and locale.
The single/multibyte versions of the dirent functions expect that directory
names are represented in the currently selected windows codepage and they
return file names expressed in the currently selected codepage, which is
not necessarily UTF-8.
ma 28. kesäk. 2021 klo 15.35 Chris Koźmik @.***>
kirjoitti:

Does it support UTF-8 filenames (regardless of OS version and locale
settings)?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tronkko/dirent/issues/39, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACCPOG3PKTHWCTCCT3FKGWLTVBT7TANCNFSM47N4ED5A
.

Do you know if setting locale as UTF-8 would guarantee opendir() (non wide-char version) to work with UTF-8 filenames (it works on my machine)?
std::locale::global(std::locale("LC_CTYPE=.utf8"));
Similarly, does setting locale as UTF-8 would guarantee _wopendir() to work?

Setting locale appears to guarantee that opendir() and fopen() work with UTF-8 file names: Microsoft says that multibyte versions of C runtime functions support UTF-8 after April 2018 update and that you should link statically if you wish to support older window versions.  See
https://docs.microsoft.com/en-us/cpp/c-runtime-library/reference/setlocale-wsetlocale?view=msvc-160#utf-8-support
Pretty cool!


Update pleiades optfile after compute node OS changed to TOSS

After compute node switched from SUSE Linux Enterprise (SLES)
to  Red Hat Enterprise Linux-based Tri-Lab Operating System Stack (TOSS), version 3
on Dec 20, 2021, update this optfile for NAS computer (pleiades).
The main change is related to MPI module, now using: mpi-hpe/mpt.2.25 (since the
previous module: mpi-sgi/mpt.2.14r19 is not longer available).
Also use directly Env var "MPI_ROOT" and "NETCDF" (set from the modules) to set-up
include dir path and libs ; and remove "-lnetcdf" since it's added by genmake2.
What changes does this PR introduce?
update optfile "linux_amd64_ifort+mpi_ice_nas"
What is the current behaviour?
no longer working since compute-node upgrade
What is the new behaviour
fixed
Does this PR introduce a breaking change?
no
Other information:
latest testreport results with updated optfile:
http://mitgcm.org/testing/results/2021_12/tr_pleiades-ieee_20211231_0/
Suggested addition to tag-index
o tools/build_options:

update pleiades optfile "linux_amd64_ifort+mpi_ice_nas"

@owang01 I am going to make new adjustments to this optfile, so probably better to review this after it's pushed.

Just a comment regarding this point:

Also use directly Env var "MPI_ROOT" and "NETCDF" (set from the modules) to set-up
include dir path and libs

This goes along with @mjlosch comment in PR #583, and the advantage of this is:

the same optfile can be used without MPI (in this case, without loading module mpi-hpe/mpt.2.25), and also without NetCDF modules (for a simpler built in case no NetCDF is needed). And without MPI, since the NetCDF module, include path and libs are different than with MPI, using module set env var "NETCDF" still works while hard coded paths would not.
the optfile is easier to maintain: for instance, this new optfile could have been used with the previous OS (SLES), providing the right MPI module is loaded. This is an interesting backward compatibility during the transition period when old SLES nodes are still available (but no longer the default nodes).

There is one small difference with optfile "linux_ia64_cray_ollie" (that Martin mentioned in PR #583 comments) is that,
since this "ice_nas" optfile is specific to pleiades, and we know, on this computer which env. var are set by which module, there is no attempt to consider other env. var setting that just the 2 mentioned above (e.g., NETCDF_ROOT, NETCDF_HOME, MPI_HOME do not appear here); I think it's simpler this way and a bit more clear (just my thought).

I run both the ieee (=default) and the -fast testreport with this recent compiler (comp-intel/2020.4.304) and looks good
( http://mitgcm.org/testing/results/2022_01/tr_pleiades-ieee_20220110_0/ and http://mitgcm.org/testing/results/2022_01/tr_pleiades-fast_20220110_0/ ). The 2 fails are due to a license access problem, not a real issue with the compiler version itself.
I updated the comment in this optfile, and will merge this little PR soon.


Performance Issues - Possible Bug with Concurrent Users

I currently have two docker containers: 1 mapproxy, 1 geoserver both on AWS ECS - they are simultaneously being used on an EC2 instance that then stores the images in s3. Geoserver requests are from postgresql. I'm using uwsgi as part of the configuration. I'm attempting to scale it using multiple 64 vCPU EC2 instances but I cant even get one to properly use the available cores on 1 instance.
No matter the combination I use the caching stays pretty much the same (processing 11,000 tiles per 30 secs). CPU usage also never goes above 30ish% average. Attached are the specs.

Looks like after more testing on my side - the Processes (Workers) and Threads from uwsgi are completely over-written after doing SSH log testing against https://pypi.org/project/uwsgitop/ .


Project Check #7 - Ethan Tran & Alex Kumar

Task: Working on AI bot:

 completed making a simple frontend to work with
 continued work on finding the private key
 increased accuracy of response and made it so you can put multiple inputs
3 hours each

future goals for AI bot:

  complete the frontend to output response for ai API better
 Try to find potential ways to decrease time for API call
 possible create new key

Task: creating community forum backend

 create backend endpoints CRUD
 Create comments and name input
 READ old inputs from other users
 DELETE comments already made
 UPDATE your old comments
 SQL table properly working
 6 hours

Future goals for baclend

  Ensure all endpoints are properly working
 add more functionality to the edit endpoint
 make the delete endpoint more specific
 possibly add UID

[x] Python 2D Animation Hacks
[x] Sorting Algorithms Hacks


Evidence-based Software Engineering

You need a section covering the engineering aspects of programming; in particular the evidence for the opinions that get bandied about.
A book to get the section started: Evidence-based Software Engineering

What issue?
On Tue, 30 Jul 2024, 12:27 am github-actions[bot] @.***>
wrote:

This issue has been automatically marked as stale because it has not had
recent activity during last 60 days 😴
It will be closed in 30 days if no further activity occurs. To unstale
this issue, remove stale label or add a comment with a detailed explanation.
There can be many reasons why some specific issue has no activity. The
most probable cause is lack of time, not lack of interest.
Thank you for your patience ❤
—
Reply to this email directly, view it on GitHub
https://github.com/EbookFoundation/free-programming-books/issues/11146#issuecomment-2257227618,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BJPG6SKG62PD37KPBJCYCYDZO3MYHAVCNFSM6AAAAABIRSHKE2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENJXGIZDONRRHA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***
.com>

That book would be good to add. I would put it in the "Software Architecture" category, which looks like a grab bag at the moment and we could consider splitting out a "software engineering" category.
Please us e this PR to add the book, splitting the section should be a separate category.

@eshellman
Hello Eric,
I'd like to work on this issue. Could you please assign it to me?


[Docker] Errors while checking binaries.

2022-04-26 11:53:31.128 28 | exec <INFO> Executing about binary check: python --version
2022-04-26 11:53:31.130 28 | helper <ERROR> Process undefined for about binary check error: Error: spawn python ENOENT
2022-04-26 11:53:31.157 28 | helper <ERROR> pipenv binary not found at: /usr/local/bin/pipenv
2022-04-26 11:53:39.813 28 | exec <INFO> Executing about binary check: /usr/local/bin/TwitchDownloaderCLI --version 2>&1
2022-04-26 11:53:39.816 28 | helper <ERROR> Process undefined for about binary check error: Error: spawn /usr/local/bin/TwitchDownloaderCLI ENOENT

python not being found is a bit weird, shouldn't affect anything though
twitchdownloader outputs the version info in stderr which makes it a bit annoying to capture

In my tests there was no pipenv and no twitchdownloadercli in the docker-container (2days ago)
which pipenv
which TwitchDownloaderCLI
were both empty

pipenv is not downloaded as this is the only application running in it, not sure if twitchdownloader is ever placed into $path, gonna have to check


make it work with css so we can make it responsive

After tinkering with this stuff for waaaaay too long I came to understand that web components the way they are implemented here are NOT allowing for outside css changes, therefor making it impossible to style it iteratively from a useful dev experience perspective.
Maybe I missed something? How can we use breakpoints, dark mode, all the goodies etc, to style the container?

got grumpy

Man, my mistake for not checking any of the DeepChat.dev website on mobile. It completely ignores mobile devices and doesn't render correctly at all. IMO a complete oversight by this library to not address this ;(
Sorry again, but I spent way too much time assuming this would be baked in, coming from a frontend perspective.

next repo

Sorry to hear Deep Chat didn't address your usecase.
May I ask what kind of styling was not being rendered/ignored on mobile? Also, what type of phone/OS/browser were you using?
Thanks!

It just did not come with ANY responsiveness in the framework, and all was left to user land to fix. So nevermind, like I said before, this is an extreme hassle to whoop into shape. I had to workaround so much that I decided to go look for something developer friendly...but thanks for all the hard work anyway1


Challenge text is wrong

Challenge Use Clockwise Notation to Specify the Padding of an Element has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/50.0.2661.94 Safari/537.36.
The challenge asks to "Use Clockwise Notation to give the ".green-box" class a padding of 40px on its top and left side, but only 20px on its bottom and right side." If the clockwise notation is "top, right, bottom, left" the correct would be asking to give the " '.green-box' class a padding of 20px on its top and left side, but only 40px on its bottom and right side.". Because the correct answer to this challenge is:
  .green-box {
    background-color: green;
    padding: 20px 40px 20px 40px;
  }

My code:

margin

padding

padding

@lucrp, read this again. "Use Clockwise Notation to give the ".green-box" class a padding of 40px on its top and left side, but only 20px on its bottom and right side."
your code is wrong.  try again.

@dwd2010 , when I type the correct answer to the question   .green-box {     background-color: green;     padding: 40px 20px 40px 20px;   } I get this errors:

So I think that the question would be "Use Clockwise Notation to give the the opposite: ".green-box" class a padding of 20px on its top and left side, but only 40px on its bottom and right side."

@lucrp, it is not error. I trying to say you got wrong answer.

@dwd2010 You're right. I didn't read it right. Now I get it. Thank you and excuses for opening an Issue that was my mistake. Next time I'll ask for help at Gitter first.


Added localization for iOS plist files

Summary
Permission dialogs on iOS use native mechanism to apply localization to usage description strings in them. This PR adds ability to populate native iOS translation resource files with translated values according to Apple guide. This is also applicable for other localizable strings in Info.plist file.

Updated the lane 'ios replace_assets' in Fastlane file with script, which finds and copies the iOS-specific localization strings into iOS project and replaces the {applicationName} with actual value.
Added new strings to en.json file, based on the strings from ios/Mattermost/Info.plist
iOS project file updated to include translation resource files (for all currently supported languages in Mattermost project), so they will be included to the app package.

Checklist

[ ] Added or updated unit tests (required for all new features)
[ ] Has UI changes
[x] Includes text changes and localization file updates
[ ] Have tested against the 5 core themes to ensure consistency between them.

Device Information
This PR was tested on: iPhone simulator, iOS 16.2
Release Note
* Added localization support for iOS usage description strings on permission dialogs.

Hello @Partizann,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

First of all this is great but this ticket https://mattermost.atlassian.net/browse/MM-52217 that must be done asap will definitely conflict with this

That's fine. Let's wait for the update of Info.plist values and I will update this PR accordingly

Qq: will the fallback continue to be english?

Yes, it will fallback to the value from Info.plist

Can you find a way to localize the extensions. Maybe it'll require a different approach

I'll take a look on that

@Partizann can we work together on this one to include the changes suggested in the ticket I provided? I don't mind if you want me to make those changes myself on this PR. lmk

It seems that when running npm run i18n-extract the strings are being stripped from the en.json file as there are not defined anywhere in the JS codebase.

Sure, @enahum. I would prefer that you make the change described in ticket, because IMHO including the question Allow $(PRODUCT_NAME) to use XXX into each usage description string is redundant. There is already a title in each permission dialog set by iOS

@Partizann I need you to please enable Maintainers are allowed to edit this pull request.

@Partizann worse case scenario if you cannot do that I'll submit another PR and close this one

closing in favor of #7303

Can you find a way to localize the extensions. Maybe it'll require a different approach

Here is my suggestion #7351


[Bug]: 'Paper' and the skill cards are both seemingly counted as the 'same' item in the certificate crafting!

Describe the Bug
'Paper' and the skill cards are both seemingly counted as the 'same' item in the certificate crafting!
To Reproduce

Install just this mod and JEI for testing
Load new world
Craft the Mining certificate with 5 paper!
Profit

Catch the bug In Action!

Crash Report or Log
No response
OS
Ubuntu
Minecraft Version
1.19.2
ModLoader
Forge
Mod Version
1.19.2-1.3.0

I've resolved the issue. The upcoming update will ensure that it is functioning correctly.


[feature request] add a separator between "Open scene" and "Save scene"

Hi,
It happened at least twice that I clicked on "Save scene" instead of "Open Scene" and that I unwillingly overwritten a poor and innocent scene.
My screen is 15" with a resolution of 1366 x 768. Even if it's not a high density screen or minuscule screen, menu items are small enough to miss my target when I click a little too fast.

There is a separator between the two by now.


Message sended/received twice with IPv6Enabled

Hi,
I have IPv6Enabled and when I send a message from the client to server (two Unity applications on the same machine) I get them twice on the server – one via IPv4 and the same via IPv6.
Thought the NetManager checks if IPv6 works and then only sends the message once via IPv6 (if not sends it automatically via IPv4 only).
How can I fix this (except by disabling IPv6)?

@michael-balle hi. Can you send minimal example project with reproduce?

SvNetworkManagerClient.txt
SvNetworkManagerServer.txt
SvNetworkMessageTypes.txt

@michael-balle seems that it's because you broadcasting and received both ipv4 and ipv6 address.

Check on connected client will not save here, because connecting taking some time. You can use GetPeersCount(ConnectionState.Any)

Great, thank you.


Changing the gap of a column does not update in real time

See here: https://screenshot.click/03-35-v91h6-5hcgd.gif
There are some other no-longer-updating-in-realtime bugs, eg for draw-to-insert, so there may be a common root cause

The problem is related the elements being inside render props, I couldn't figure out why exactly


0.1.x integration

Adding support for IP aliases, MTU and iptables ingress rules configuration

LGTM!


{data} property included in the exported CSV?

return $csv->output($study->name . '.csv');
At the bottom of the JSON file this is included:
{"data":412655}
How can I prevent this from happening?
Thanks!

@baj84 this question has been asked several times on this issue tracker. Please read the documentation where AbstractCSV::output is explained.


Chapter 9 Challenge 8 Advanced Challenge 3

Implement the final challenge for chapter 9

Completed in pr #1044

Completed in pr https://github.com/saving-satoshi/saving-satoshi/pull/1044


Github pull request application limiting Solid reach to talent

To apply to be a Solid panellist, editor, administrator or creator one needs a GitHub account and know how to navigate GitHub enough to submit a pull request.
Because GitHub is a tool with specific professional and geographically distributed users these steps act as filters to the applications.
Submitting a pull request is very public which is good because when the Solid Director approves and application it is possible to refer back to the approval. However, the public nature may also mean applicants hesitate to apply at all.
My proposal is to include a text "If you are interested in applying to become a panellist, editor, administrator or creator then email<EMAIL_ADDRESS>with your CV and motivation letter."
This email can be received by administrators and put forward to the director for review. When the application is reviewed positively one of the administrators can submit a pull request with the directors review to have the benefit of reference.
The thinking behind this design is that we may tap into a wider and more diverse pool of talent.

https://github.com/solid/process/pull/163


Replace the f32 font_size value of TextStyle with a FontSize enum

Objective
Replace the font_size field of TextStyle with an enum similar to Bevy UI's Val type enabling users to set font sizes that depend on the size of the viewport or the text's parent container.
Fixes #9525
Solution
Replaces f32 font size with a FontSize enum. Currently implemented variants:

Px: size in logical pixels,
Vh: size in percentage of viewport height,
Default: use default font size contained in the new DefaultFontSize resource.

Only updated text.rs example so far, want feedback on the API etc before rewriting every example.
Many problems left to solve:

Expect a lot of bugs with scale factor and Text2d.
The hello bevy text in the text example scales correctly when you resize the window but almost immediately runs out of font atlases.
Should there be Vw, VMin and VMax variants, or is only Vh really useful?
How should percentage values be implemented?
Is using Default for the name of an enum variant bad style?

Small window with FontSize::Vh(50.):

Larger window:

Changelog

Added FontSize enum with Px, Vh and Default variants.
The font_size field of TextStyle is now a FontSize.
Added DefaultFontSize resource. Also implemented as an enum with Px and Vh variants but no Default variant.

Migration Guide
The font_size field of TextStyle is now a FontSize. The FontSize::Px variant is the equivalent to setting an f32 value previously.

Is there a reason why you aren't using the Val enum here?

Is there a reason why you aren't using the Val enum here?

It's not clear if all the variants of Val would make sense. Even if they do, we might add other variants to Val in the future that don't (or FontSize variants that don't make sense for Val).
Val doesn't have a Default variant, maybe we could use Auto but might be confusing.
Even if all the variants of Val do make sense for font sizes, the behaviour is probably going to be quite different and sharing the same docs for both could be confusing.

I would be very happy to see vh for font size, but I'm not so sure about adding DefaultFontSize

I'm feeling similarly iffy about DefaultFontSize. It's a very implicit global override, without a clear use or unification. Global theming should probably be a thing, but I'd want something more cohesive than scattered resources for some but not all properties.

Yep this is very experimental at this stage, it's blocked by a lot of things and I haven't put a great amount of thought into it. I think it's an important feature though.
Agree that DefaultFontSize is a deadend.
The main blocker is how we create a new texture atlas for every different font_size even if there is only a fraction of a pixel difference. I think we need some combination of:

Scaling of glyphs instead of creation of a new font atlas when the sizes are close
Primitive garbage collection that disposes of font atlases with sizes no longer in use
Draw text that uses the responsive size variants using distance fields

None of which seems that difficult to implement but I don't have unlimited time or that much experience to draw on.
(1) is conceptually easy but the implementation details might be very tricky (like how to determine whether the fonts are close enough, what if we have multiple windows each with a different scale factor?).
(2) Looks like it could be hacked together quite easily, just delete the oldest font atlas that isn't in use if we go past the font atlas limit. Perhaps this should be part of the changes to assets though, I'm not sure?
(3) I thought this would be challenging but not so sure now. There is plenty of prior art on how to render sdf fonts. It might be the simplest least hacky approach.

This PR would fix #5471 if merged.

As a response to the first point I could see practical use for Vh, Vw, and Vmin. Those being the following:

Vh; is the smaller of the two in most cases. If the software is meant for a landscape orientation, this works well as the text will be certain to always fit on the screen.
Vw; this can be useful for windows that are taller than wide (such as they would be on phone) as this would be the smaller of the two, once again insuring that the text is always visible
Vmin; it's the minimum of the two above, so it's ideal for when your GUI should be adaptive and work across multiple platforms.
I don't see any practical use for Vmax, but there is no reason to be limiting with no reason. If it would work and have no issues... why not let it be there?

As a response to the third point I could see practical use for Vh, Vw, and Vmin. Those being the following:

Vh; is the smaller of the two in most cases. If the software is meant for a landscape orientation, this works well as the text will be certain to always fit on the screen.
Vw; this can be useful for windows that are taller than wide (such as they would be on phone) as this would be the smaller of the two, once again insuring that the text is always visible
Vmin; it's the minimum of the two above, so it's ideal for when your GUI should be adaptive and work across multiple platforms.

I don't see any practical use for Vmax, but there is no reason to be limiting with no reason. If it would work and have no issues... why not let it be there?

Well no features is also a feature. It can seem harmless to add extra variants but everything has to be maintained and documented and kept up to date etc. A huge range of similar options can be overwhelming for users too.
Your argument makes sense about Vw and VMin though. I think I agree now, they should both be included.

I'd also like to throw in a potential Percent variant version if possible in regards to my own issue on this subject, #9280. This has killed game projects in the past where I've been at a loss on how to reliably size the text for a variety of viewing sizes. If anyone has a recommendation in the mean time on how to display cards on a screen as UI buttons without resizing the window screwing up the cards via their text, please let me know.

I agree with @JustAnotherCodemonkey a Percent (of the height of the parent) value would be really useful.
Our use case is to have a UI node have a shrink animation, but it's really tedious to have the characters on it shrink at the same time with the way font size is implemented.


PGD fails when running targeted attack with PyTorch or TensorFlow

Describe the bug
When running a targeted attack, the final compute_success calculation may fail due to dimension mismatch. Here's an example trace:
File "/workspace/armory/scenarios/audio_classification.py", line 181, in _evaluate
x_adv = attack.generate(x=x, y=y_target)
File "/workspace/art/attacks/attack.py", line 74, in replacement_function
return fdict[func_name](self, *args, **kwargs)
File "/workspace/art/attacks/evasion/projected_gradient_descent/projected_gradient_descent.py", line 181, in generate
return self._attack.generate(x=x, y=y, **kwargs)
File "/workspace/art/attacks/attack.py", line 74, in replacement_function
return fdict[func_name](self, *args, **kwargs)
File "/workspace/art/attacks/evasion/projected_gradient_descent/projected_gradient_descent_pytorch.py", line 216, in generate
100 * compute_success(self.estimator, x, y, adv_x, self.targeted, batch_size=self.batch_size),
File "/workspace/art/utils.py", line 627, in compute_success
attack_success = compute_success_array(classifier, x_clean, labels, x_adv, targeted, batch_size)
File "/workspace/art/utils.py", line 599, in compute_success_array
attack_success = adv_preds == np.argmax(labels, axis=1)
File "<__array_function__ internals>", line 6, in argmax
File "/opt/conda/lib/python3.7/site-packages/numpy/core/fromnumeric.py", line 1188, in argmax
return _wrapfunc(a, 'argmax', axis=axis, out=out)
File "/opt/conda/lib/python3.7/site-packages/numpy/core/fromnumeric.py", line 58, in _wrapfunc
return bound(*args, **kwds)
numpy.AxisError: axis 1 is out of bounds for array of dimension 1

This does not happen when using ProjectedGradientDescentNumpy. I think this is due to the difference in the parameters sent to compute_success:
https://github.com/Trusted-AI/adversarial-robustness-toolbox/blob/main/art/attacks/evasion/projected_gradient_descent/projected_gradient_descent_numpy.py#L308
https://github.com/Trusted-AI/adversarial-robustness-toolbox/blob/main/art/attacks/evasion/projected_gradient_descent/projected_gradient_descent_tensorflow_v2.py#L212
https://github.com/Trusted-AI/adversarial-robustness-toolbox/blob/main/art/attacks/evasion/projected_gradient_descent/projected_gradient_descent_pytorch.py#L216
In particular, numpy passes in targets instead of y, which means that targets = self._set_targets(x, y) has been called. targets will be one-hot-encoded, where y could be label-encoded. Printing just before the call in PyTorch, I get:
y: [1], targets = [[0. 1. 0. 0. 0. 0. 0. 0. 0. 0. 0. 0. 0. 0. 0. 0. 0. 0. 0. 0. 0. 0. 0. 0.
  0. 0. 0. 0. 0. 0. 0. 0. 0. 0. 0. 0. 0. 0. 0. 0.]]

I think the fix is simple: replace y with targets in the call to compute_success in PyTorch and TensorFlow.

I am fine making a PR to this, if you would like.

I am fine making a PR to this, if you would like.

Hi @davidslater Thank you very much for catching and fixing this bug!

Hi @davidslater Thank you very much for catching and fixing this bug!


Re-activate Tokio Console

Actually I'm not sure this is going to work.
I can connect with Tokio-console to a Sui-node binary compiled by these changes, and with TOKIO_CONSOLE=true env var.  However, no tasks are displayed.
I followed all the recommendations in https://docs.rs/console-subscriber/0.1.7/console_subscriber/#enabling-tokio-instrumentation
You have to enable TRACE logs for Tokio and tracing, which just spews an unacceptable amount of logs.  And even after that I still am not getting any instrumentation.  Granted I had no transactions, but I should be able to see something happening just with consensus and checkpoints, etc.
Originally posted by @velvia in https://github.com/MystenLabs/sui/issues/4438#issuecomment-1234863624

Here's one place where this would be useful to test:  any of the integration tests in https://github.com/MystenLabs/narwhal/issues/841
These are in a live lock, and it would be tremendously useful to run their binaries on tokio-console.


stub ignores hash_including

Hi!
I'm trying to stub a request using hash_including. The request in question has several parameters, one of them is a random generated number which I simply can't generate beforehand therefore I'm trying with hash_including.
While trying to use the snippet
stub_request(:post, "http://api.example.com/").
            with(body: hash_including('method' => 'get_contacts')).
            to_return(status: 200, body: "", headers: {})

I receive this NetConnectNotAllowedError:
WebMock::NetConnectNotAllowedError: Real HTTP connections are disabled. Unregistered request: POST http://api.example.com/ with body<EMAIL_ADDRESS>with headers {'Accept'=>'*/*', 'Accept-Encoding'=>'gzip;q=1.0,deflate;q=0.6,identity;q=0.3', 'User-Agent'=>'Ruby'}

You can stub this request with the following snippet:

stub_request(:post, "http://api.example.com/").
  with(:body =><EMAIL_ADDRESS>       :headers => {'Accept'=>'*/*', 'Accept-Encoding'=>'gzip;q=1.0,deflate;q=0.6,identity;q=0.3', 'User-Agent'=>'Ruby'}).
  to_return(:status => 200, :body => "", :headers => {})

registered request stubs:

stub_request(:post, "http://api.example.com/").
  with(:body => hash_including({"method"=>"get_contacts"}))

I tried using hash_including with different keys (e.g. params key), tried string keys, tried switching body to query in the stub, tried hash rockets.
Anybody have a clue why would hash_including not work for this stub?

Your request doesn't have Content-Type header specified, therefore WebMock doesn't know how to represent the the request body as a hash and then match it to the hash_including matcher.
Please add Content-Type: 'application/json' header to the request.

That's what you get for running an old gem! Thank's a lot, don't know how I've missed it!


Chrome for testing is only for automated testing bar on all screenshots

Really appreciate the chance to fix to a chrome version for testing. The lack of this ability has been very difficult for us. We work in a regulated industry where we have extensive browser-based tests which take screenshots. These screenshots end up in validation documentation that is viewed by external auditors.
I'm noting when using a chrome-for-testing build:

A header that says "Chrome for testing vXXXX is only for automated testing...."
Well, yes. That's what we're doing, we are driving this with selenium. We don't need or want a header on every screenshot telling us so. Can we have a flag to turn this off?

There's an internal-only FR for that: https://crbug.com/1476789 ("Re-introduce --disable-infobars as a CfT-only feature").

This has now landed: https://chromium-review.googlesource.com/c/chromium/src/+/5012709
However, a release hasn't yet been made. Wait a few days and it should be available.


add umbc

Thank you for contributing!Preview these changesLink expires: Sat Sep 29 06:36:14 UTC 2018Note: it may take a moment for the branch to deploy. Check all pending travis builds have completed

Thank you for contributing!Preview these changesLink expires: Mon Oct  1 15:27:58 UTC 2018Note: it may take a moment for the branch to deploy. Check all pending travis builds have completed


Compile fail on Travis due to ClassNotFound

See https://travis-ci.org/bridgedb/BridgeDb#L4544-L4549

Locally confirmed.

Calling the following make the main compile work:
GitHub/bridgedb/org.bridgedb.uri.ws.server$ mvn compile

OK, running the testing still does not work; I disabled that for now ...


Is there a away to validate the whole filesystem-based dynamic configuration ?

It's seems --mode validate only check the main envoy.yaml config. It can't check filesystem-based dynamic resource, even yaml format error. Is any way to do the whole check ?

plus one on this, what's the best way to go about validating your lds/cds config files?


[RadioGroup] Create new RadioGroup component

[x] I have followed (at least) the PR section of the contributing guide.

@atomiks The docs demo is missing the "indicator", I think that's one (or perhaps the main?) point of confusion. There should be (whatever "should" means) a little dot inside the circle when it's selected. Like this https://helios.hashicorp.design/assets/components/form/radio-card/radio-card-states-2acdd2c7a5775a7479f7c497d3947804.png

@colmtuite that's just one particular design though. The indicator can be anything, such as a plain filled-in white circle which is an outline when unchecked.


make the output spreadsheet format the same as the input format

@BijalBPatel has ideas for doing this.  relates to #14  and #12

closing this as it's wrapped up in #14


Update landing page and registry viewer to use Node.js 18

Which area/kind this issue is related to?

/area registry-viewer
/area landing-page
Issue Description

The current version of Node.js for the landing page and the registry viewer, version 16, is planned to lose support September 11, 2023. Due to this, it is recommended to update to Node.js 18 which is the next LTS version available.
Acceptance Criteria

[ ] Update container base images under Dockerfile
[ ] Update container image under devfile
[ ] Update any necessary dependencies under package.json
[ ] Update documentation requirements

Acceptance criteria added, refinement date set to August 25th.

Acceptance criteria added, refinement date set to August 25th.

Changed to September 8th.


Git checkout failing on master branch on windows due to long file names

Describe the bug
Git checkout failing of master branch on windows due to long file names
Desktop (please complete the following information):

Windows 10
Master branch v11.1.0

I have the same issue, it happens because of Test's long snapshot names

Thanks for reporting ... we will have a look at this.

This should be fixed


Update Diagrams in EM Reporting Instructions

Document is here:
https://easternresearchgroup.sharepoint.com/:w:/r/sites/CAMD-ECMPS-ERG/Shared Documents/ECMPS-ERG/Beta Instructions/[2024-11_em-reporting-instructions.docx](https://easternresearchgroup.sharepoint.com/:w:/r/sites/CAMD-ECMPS-ERG/Shared Documents/ECMPS-ERG/Beta Instructions/2024-11_em-reporting-instructions.docx?d=wd138cd1e3c664689b50e32f8d073a426&csf=1&web=1&e=vreQXP)?d=wd138cd1e3c664689b50e32f8d073a426&csf=1&web=1&e=vreQXP
Each figure needs to be reviewed/updated. Also, the following are changes:
Figure 1: Submission Comment and Long Term Fuel Flow Data are missing
The "Emissions" box should just say:
ORIS Code
Version
Year
Quarter
Submission Comment
Figure 2: Remove the "Emissions" box

Figure 2:
Remove duplicate dailyBackstopData element
hourlyOperatingData should have a dotted box
Figure 6:
remove MonitoringSystemID
Figure 10:
the first "d" in dailyBackstopData should be lower case
Figure 20:
remove fuelCode
Figure 23:
violationOfCO2StandardIndicator & violationOfCO2StandardComment should have the "of" capitalized

@annalbrecht The changes have been made

Changes look good. Marking this as done.


[FEATURE REQ] Update headshot_url for 2024 Draft Class in NFL Roster database

Is there an existing issue for this?

[X] I have searched the existing issues

Is your feature request related to a problem? Please describe.

Player profile headshot urls are missing for the 2024 NFL draft class in the NFL Rosters database. Would also be nice if those URLs were added to the Draft database.

Describe the solution you'd like
Add new player headshot image urls to the NFL Roster database for players who were taken in the 2024 NFL draft. Also, add a new table column for headshot url in the NFL Draft database.
Describe alternatives you've considered
I tried to access the base url for the player profile urls (https://static.www.nfl.com/image/private/f_auto,q_auto/league/...) to see if I could contribute, but I don't have access that server.
Additional context
No response

These headshots are not available yet from the NFL. You can check this yourself--Caleb William's NFL.com page does not have a headshot.. So unfortunately, we can't provide links to headshots that don't yet exist.
As this is an issue on the NFL's end, and not anything with our code, I'm going to close this issue. If the NFL adds these headshots, I'm confident that our code will ingest the new headshots appropriate, but in the event that it doesn't, feel free to re-open this issue.


only font_roboto_16 & font_roboto_28 available for micropython?

Hi,
How can I use font_roboto_12 and font_roboto_22 in micropython as mentioned in the docs?  I can find only lv.font_roboto_16 & lv.font_roboto_28.
Thanks,
Kaiyuan

Please send questions to the forum.
If it's micropython specific, you can use the micropython category in the forum.


Name shown in chat notifications even if hide names is checked

Hello,
I've check the "hide names" option in health estimate tokens' options, and the token names still appear in chat. After checking out the source code, it seems to me that in logic.js@outputStageChange, you only replace the actor's name IIF the token's "display name" option is set to "Never displayed".
As a GM, I like the names to be displayed, always or hovered, for a lot of token ; in addition to named NPCs, I'm using Token Mold to generate adjective to try to generate a small detail on the fly. Due to that, my players got a good laught out of a "Fulfilled Zombie" :)
I did too, not gonna lie, but I'm worried now that it might spoil some names.
Since I think the fix for this is transforming the condition to be || actor.data.displayName == 0, it's a breaking change from the historic behavior. Thus I'm reluctant to create a PR for this. I'm also reluctant to create a PR adding another "Hide names" option, it seems redundant.
What's your thought on that please? Is it better to change the actual implementation or to create another option? Or a better third option maybe?
Note: I'm using Foundry V9, and Health Estimate is fully updated on this version.

There's two keybinding controls (Hide Names and hideHealthEstimate) for hiding the names, but I reckon that the feature could should better and assumes people run the game the same way I do.
It also made me notice players' tokens with the Never Displayed option will be obfuscated because of that logic.
I'm thinking about checking for actor.data.displayName <= 20 (when selected/hovered by owner) and also check for !token.actor.hasPlayerOwner so you'd be able to use whatever private option and also not obfuscate players' tokens.

There's two keybinding controls (Hide Names and hideHealthEstimate) for hiding the names, but I reckon that the feature could should better and assumes people run the game the same way I do.

Oh, it didn't cross my mind to check keybindings. I don't really use them, but when hidden with this method it behaves exactly like I want: name always displayed on canvas, and names in notification only visible to me and not the players 👍

I'm thinking about checking for actor.data.displayName <= 20 (when selected/hovered by owner) and also check for !actor.actor.hasPlayerOwner (ignore the double actor, the first one is actually a token object) so you'd be able to use whatever private option and also not obfuscate players' tokens.

Would you consider adding this condition too displayName == 40 /* always for owner */ && !actor.actor.hasPlayerOwner please? I like having all my tokens' names visible, either due to Token Mold prefixes, or simply to have an npc's name right under my eyes.
If it makes it incompatible with another feature, I understand, and I'll use the keybindings instead.

Closing this as it is in the future V10 release.


Update ug, dg and ppp

Codecov Report

Merging #199 (488587f) into master (b1459bc) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #199   +/-   ##
=========================================
  Coverage     67.88%   67.88%
  Complexity      812      812
=========================================
  Files           132      132
  Lines          2821     2821
  Branches        342      342
=========================================
  Hits           1915     1915
  Misses          854      854
  Partials         52       52

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Use latest conceptual framework for transcipt segment start/end

Feature description
We're altering when/how to set the start/end values of SequenceLocation objects in transcript segment elements of fusions, rather than setting both start and end every time. Now, start and end should be selectively used based on orientation and position in a fusion. More details are in respective working docs/written on the board next to Jeremy's desk.
Use case
n/a
Proposed solution
No response
Alternatives considered
No response
Implementation details
No response
Potential Impact
No response
Additional context
No response
Contribution
None

I landed per https://github.com/cancervariants/fusor/pull/166#issuecomment-2256201904 during review. Would you be able to link the working docs / examples referenced in this issue for comment?

Offhand, there's an initial illustration of how to think about orientation here -- https://docs.google.com/document/d/1EE1-evt6NStIKUENWdirR9lXo6rtBhQvCUxvxaGYlXs/edit from Jeremy. I believe he's also writing up something a bit bigger as well.

@ahwagner @jarbesfeld and I have been working on updating the locations in a new draft PR #172


Page.RSSLink is deprecated

Hello,
I have just discovered your theme, that is the best I found with tags widget :)
However, when I build my site, I have this warning that could become an issue in the future:
WARN 2020/05/26 11:46:07 Page.RSSLink is deprecated and will be removed in a future release. Use the Output Format's link, e.g. something like:
    {{ with .OutputFormats.Get "RSS" }}{{ .RelPermalink }}{{ end }}

Thanks, i'll fix it as soon as possible.

fixed https://github.com/Tazeg/hugo-blog-jeffprod/commit/bbe726e57038b97df811731330fe2c35f5043964


Webhooks are not dumped in bug-report

These are critical parts of Istio operations, we should include them

also no Namespaces


support path activations

fix #13, support local (path) activations.
@keertip, @pq

@keertip @pq, ping

LGTM!


Sanitize filenames

This changes the way that the interview_label is created so that it matches not just the rules for variable names but also the filename sanitization methods that are used by werkzeug.utils.secure_filename(). Docassemble upstream limits the filenames in the playground to those that pass the werkzeug.utils.secure_filename() sanitization rules, so this helps ensure that packages we create in can be edited in the Playground.
This patch also makes the interview_label lowercase by default and renames the template and YAML to match the interview_label.
Fix #561
~Edit: give me a sec to make sure the filenames of a generated interview still all match up, sorry!~ Ready to be reviewed.
If you want you can try running the Weaver with the filename that started this all: 01 - Intake NOTES w all Variables.docx

Not quite right yet. This filename is getting a zero-length variable for short_filename which doesn't seem correct
01 - Funky Filename ---.docx
.


Implement asTypeOf, refactor internal APIs

Several refactor changes that shouldn't affect non-nonsensical Chisel:

Implement asTypeOf (resolution for #443)
fromBits currently untouched. Will be chisel3-deprecated if possible as a future PR.
Refactor code that needs to create a super-type of several types into cloneSupertype (eliminates the premise of #446)

This should be the only thing that changes externally visible API, in that the checks are now more consistent. oneHotMux now checks against inconsistent input types.

Split out Mux (raised in #446), and simply using above
Simplify Vec construction using above
Add a typeEquivalent internal API. This and cloneSupertype eliminates the need for typesCompatible, which is inexplicably a Mux operation.
Move cloneTypeWidth from Data to Bits. It used to silently drop the width parameter when called on any non-Bits, which is just sketchy. This makes uses of it explicit and always correct, minus the Bool case.

Eventually want to get rid of cloneTypeWidth. The only stragglers are Bits.pad (which can instead create a new UInt/etc from scratch) and Reg (which clears widths behind the scenes for you). Not sure how to address the Reg case.

Deprecate flatten, hopefully will remove it eventually using more local operations. The stragglers are mainly Bindings directionality checks, which can be made more local with a resolution to #440.

Originally the intention was to remove fromBits and the less clean APIs that it relied on (like flatten), but many more yaks were shaved. flatten hasn't been removed yet, but should be possible in the near future,

Aggregate.getElements now public.
Data.flatten is still private, since I don't think it's a good Data-level API or is well named (since it returns Element). It might make sense as a utils function or something.

Everything else fixed. More discussion on role if type in typeEquivalent may be necessary

There's a conflict in Bits.scala, think you can resolve it?

Conflict fixed.
CloneSupertype now works for fixed point (requiring the binary point to be the same) and parts of Analog have been updated to fit the new API.
Merging as soon as tests go green...


Adding Strict-Transport-Security HTTP header

Coverage decreased (-0.06%) to 55.648% when pulling 0e424f464fbe738565f99e11631b592569109d18 on tlisonbee:highlander-sts into a1530c0d5bbea104c79989179a7cbd4a42cb05f4 on Nike-Inc:highlander.

Coverage decreased (-0.06%) to 55.648% when pulling d9b06044718df4bdd3170bc301201b769dd0546f on tlisonbee:highlander-sts into a1530c0d5bbea104c79989179a7cbd4a42cb05f4 on Nike-Inc:highlander.


fail to upload models due to max try

What is the issue?
pushing c165271d7cbb...  73% ▕████████████████       ▏  44 GB/ 61 GB  4.3 MB/s    1h3m Error: max retries exceeded: Put "https://dd20bb891979d25aebc8bec07b2b3bbc.r2.cloudflarestorage.com/ollama/docker/registry/v2/repositories/taozhiyuai/qwen2-57b-a14b-instruct/_uploads/cb319ba7-5ab8-40c3-a59b-03a007ffbae6/data?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=66040c77ac1b787c3af820529859349a%!F(MISSING)20240607%!F(MISSING)auto%!F(MISSING)s3%!F(MISSING)aws4_request&X-Amz-Date=20240607T235721Z&X-Amz-Expires=86400&X-Amz-SignedHeaders=host&partNumber=48&uploadId=AHVfUfgfMc11-7PxdRLFlN2yqnUV68oRPODmHTg-a_dG_1nK5pBAEcAFf7Jt-79U2UCiAWGFDI-cFvkNxjzWL23Ly7xfr6VkHhyZWUETuxPDm3ADSL8KM2P51ZZ-Vye1olRgYFzt2L6NhedUCxtrAogdpZKh6lGd25X25VLx7NOLr0V4Obd-W9w5HuPYyVIbegFqH8cHYrPbFbiso8zt8kzb3LwqINTnKF7IDzakHe5rzmztMAH-J5HI0Lu6ELbmS-k3qeZF4-Xwuf9rNp9jsBSkJOvkBbECUpt2FLKchLqT2uAVH4-zxRqVXu9UGkA-d-XzB4FglNJrk6fGd__EGRw&X-Amz-Signature=06249a03866edad37e421d5a34bf016714a2fcb46019545544c03dcdce29a470": write tcp <IP_ADDRESS>:62212-><IP_ADDRESS>:443: write: broken pipe
how to ignore max try.
OS
macOS
GPU
Apple
CPU
Apple
Ollama version
1.40

pushing c165271d7cbb... 100% ▕███████████████████████▏  61 GB
Error: 500: {"errors":[{"code":"UNKNOWN","message":"unknown error","detail":{"Path":"/docker/registry/v2/repositories/taozhiyuai/qwen2-57b-a14b-instruct/_uploads/1bf7af90-ba95-4e9d-8ec1-d4ced1f8096a/data","DriverName":"s3aws"}}]}

I AM F*ING CRAZY!

Hi @taozhiyuai can you update your ollama version to 0.1.42 and try again? I see that you are using Docker. Can you try without Docker?  What Memory do you have on your mac as I see that you are pushing a 61GB model.  it could be a memory issue. I have at my office a 192 GB Mac Station. I could try to reproduce this issue to see if it works with more memory.

not docker, I install from https://ollama.com/download/Ollama-darwin.zip
my Mac is m3 max 128g which is ok for my model.
some versions of ollama raise upload issue. I have upgraded and trying.thx for help.

Hi @taozhiyuai can you update your ollama version to 0.1.42 and try again? I see that you are using Docker. Can you try without Docker? What Memory do you have on your mac as I see that you are pushing a 61GB model. it could be a memory issue. I have at my office a 192 GB Mac Station. I could try to reproduce this issue to see if it works with more memory.

0.1.42 is ok. some previous version had the same issue, some not. I hope the upload is resumable.

If it's Ok with version 0.1.42 could you please close the Issue?
Thank you

If it's Ok with version 0.1.42 could you please close the Issue? Thank you

but I hope upload is resumable in the future. 0.1.42 is ok. but I think new version in the future may not. previous version had the same issue. so please make uploading is resumable.

I agree that some problems went and came back, but Ollama is more stable at each version.


Input.action_just_pressed and Input.action_just_released doesnt work in _physics_process on Android

Godot version
4.0.beta1
System information
Vulkan Android
Issue description
on Android with TouchScreenButton node, input_action_just_pressed and Input.action_just_released doesnt work in _physics_process function but it does work with _process function however.
This works on godot 3.5 so im guessing its a bug on godot 4 beta 1
Steps to reproduce
heres an example project with the details:
https://github.com/ShadowwDev/Godot4Beta1-InputBugTest
Minimal reproduction project
Project download
InputBugAndroid.zip

I can confirm this on 4.0.beta2.

I was developing a project with Godot4 beta10 and ran into that same problem. When calling Input.is_action_just_pressed... on android it does not work if it is inside physics_process, it only works inside _process.

same issue with godot 4 beta 16. Input.is_action_just_pressed doesnt work on android

Confirmed bug in Godot 4.0

This is still happening. Wish that it is fixed soon. The _process(delta) workaround does work.

On 4.0.2, this works on desktop when enabling Emulate Touch From Mouse in the advanced Project Settings, so the issue is Android-specific (or Android/iOS).

I'm using v4.0.1.stable.official [cacf49999] and Input.is_action_just_pressed and Input.is_action_just_released aren't working on android. I use them for jumping.

try 4.0.2

try 4.0.2

Still not working on android

Confirmed bug in Godot 4.1

Is this issue because is_action_just_pressed checks if the input was pressed on this frame where each frame is a call to _process and not _physics_process? Maybe on Android, the framerate you're getting means that _physics_process isn't called frequently enough to hit the right frames when the input was changed (but on PC the framerate works out)?

On 4.0.2, this works on desktop when enabling Emulate Touch From Mouse in the advanced Project Settings, so the issue is Android-specific (or Android/iOS).

Is the emulation started here? input.cpp#L502
TouchScreenButton events on Android here input.cpp#L858
I think because TouchScreenButton is a graphical element, the functions are only triggered after _physics_process.
Workaround: #83301  = get_physics_frames() + 1
I was going to compare it to a normal Button, but there is no action property. Docs
If it should be solved like with the TouchScreenButton, then you could probably reproduce the problem in the desktop version.

Sorry I must have missed this bug report.
It may be a regression from my PR #77055 .
I have a fair idea of what may be happening, and why it occurs on Android (because it has separate input thread).
Previously both a press AND release updated the physics tick on the action, so it may have been reporting the action_just_pressed on the next physics tick when released (because it occurred too late for the current user code to intercept it). It would have still been likely buggy.
There's a PR (#83301) to just increase the physics tick registered for the press. This would likely "fix" the issue, but isn't ideal because it introduces a tick of delay to input, I'll put a note on that PR.
The actual problem makes total logical sense if you think about it. Every physics tick you have an opportunity to process presses that come in that tick. If the press comes in after the _physics_process, then it can be missed. This isn't occurring on most platforms as they process input synchronously before a tick, but on Android it can occur after (by random chance).
I think this should be pretty easy to fix properly, I'll do a PR. :+1:

Indeed, moving some of the Input code into the _process function fixed this issue for me. Thx ❤
It really needs to be more clear what is going wrong. Something like debug warnings, idk.

Indeed, moving some of the Input code into the _process function fixed this issue for me. Thx ❤
It really needs to be more clear what is going wrong. Something like debug warnings, idk.

The fault is not your side, it's an engine bug. There is PR to fix it, it just needs to be reviewed and merged. There's no point making a PR for warnings, as it is just as simple to fix it.
In some cases putting input code in the _process() can be used to mostly workaround it in the meantime (although even that appears to be susceptible, to a lesser extent).

Then what's the point of using _process at all? Isn't everything better
off being inside of a _physics_process?
On Thu, Oct 19, 2023, 09:51 lawnjelly @.***> wrote:

Indeed, moving some of the Input code into the _process function fixed
this issue for me. Thx ❤
It really needs to be more clear what is going wrong. Something like debug
warnings, idk.
The fault is not your side, it's an engine bug. There is PR to fix it, it
just needs to be reviewed and merged. There's no point making a PR for
warnings, as it is just as simple to fix it.
In some cases putting input code in the _process() can be used to mostly
workaround it in the meantime (although even that appears to be
susceptible, to a lesser extent).
—
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/66318#issuecomment-1770171442,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOWMWNDMO57ZTMJM5MTAFZ3YADEXPAVCNFSM6AAAAAAQUJBIGKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZQGE3TCNBUGI
.
You are receiving this because you commented.Message ID:
@.***>

@synalice
https://github.com/godotengine/godot/pull/83438#issuecomment-1765333406
You could try this demo and check the input latencies and experiment with engine settings.
Normally you should also process all inputs under _physics_process (at 60 FPS, because '_physics_process' is executed before '_process), but on Android that doesn't work so well yet. There is a bugfix for the is_action_just functions, but this increases the input latency (at 60 FPS it is at least 16.7ms).


Inconsistent use of white points

One of my biggest concerns with using CIELAB and CIELCH was how inconsistent the chroma calculation was. It would swing quite large, and something always felt off. It bothered me greatly.
Recently I dug into it and found the reason for the large swing in chroma and why it was even closer to zero was due to the fact that white point usage was consistent. As CIELAB was in D50 and sRGB was in D65, there was a conversion to XYZ and then chromatic adaption. It was in this area that CIELAB was thrown greatly off course.
We can see the heights to which chroma could reach when evaluating sRGB achromatic colors:
srgb: maximum chroma: 0.014636
srgb: maximum a: 0.003817
srgb: maximum b: 0.014130

But after re-evaluating all the matrices with the same chromatic adaption white point, we can see a vast improvement.
srgb: maximum chroma: 0.000012
srgb: maximum a: 0.000011
srgb: maximum b: 0.000005

I want to code up an experimental branch that adjusts all the matrices. I am interested to see if color.js (and the CSS spec) fix the XYZ matrices or they adjust the chromatic adaption matrices.
NOTE: I could not exactly figure out how color.js calculated their ProPhoto matrices. I was really, really close, but not spot-on. I'm not sure if their primaries were different or their white point was different from mine. But if all the matrices get updated, it won't matter as we would then match them.
Ref https://github.com/LeaVerou/color.js/issues/87

I did figure out ProPhoto primaries and white points. They kept the white point to 4 significant digits like D65, and capped the primaries to that as well. So I will match my primaries with the consistent white points.


Test connection failed javax.net.ssl.SSLHandshakeException

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
http api
新增：检测内网平台https+非443端口。提示报错。（内网平台https，无ssl证书）
Expected Behavior
测试连接失败 javax.net.ssl.SSLHandshakeException
Steps To Reproduce
No response
Environment
HertzBeat v1.4.3

Debug logs
No response
Anything else?
No response

你不能去用HTTPS检测一个没有证书的网站，用HTTP

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑🤝🧑👫🧑🏿🤝🧑🏻👩🏾🤝👨🏿👬🏿

You can't use HTTPS to detect a website without a certificate, use HTTP


Improve image-upload condition to know if population succeeded

What this PR does / why we need it:
This pull request aims to act as a minor correction/follow-up for this PR: https://github.com/kubevirt/kubevirt/pull/10534.
Instead of checking both the pod phase and the pvcPrime annotation, it's more straightforward to check the target PVC phase: When using populators, a bound target PVC will always mean that the population has succeeded. This will help us address some corner cases in https://bugzilla.redhat.com/show_bug.cgi?id=2241658.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes # Partial fix of https://bugzilla.redhat.com/show_bug.cgi?id=2241658
Special notes for your reviewer:
Release note:

NONE

@mhenriks to take a look at this one
/cc @mhenriks

/lgtm
/approve

/test pull-kubevirt-e2e-k8s-1.28-sig-compute

/test pull-kubevirt-e2e-k8s-1.28-sig-compute-migrations

/cherry-pick release-1.0

/cherry-pick release-1.0


Embedded webpage in canvas doesn't render in HTML export.

Hi,
Embedded webpage in canvas doesn't render in HTML export.

Thank you for looking after.
Win 10 PRO
Firefox 122.01
Obsidian 1.5.3
your plugin version 1.8.0

What do you mean by embedded webpage? Do you mean just an iframe? Or are you using a plugin?

here is a screenshot

Ah, okay I will look into this

This should be added in 1.8.1-0b

Hi,
It WORKS exactly as I expected. THANK YOU VERY MUCH 👍 🥇

Okay great I will close this


[Prayer] Prayer bar is rendering under the character instead of above.

Type
Incorrect behavior
Operating System
Windows
OS Version
No response
Bug description
Prayer bar is rendering under character instead of above as of 8/8/2022
Screenshots or videos

RuneLite version
RuneLite version: 1.8.30
Launcher version: 2.4.2
Logs
client.log
launcher.log

Do you have "Improved Tile Indicators" plugin and does the issue disappear if you disable that plugin?

Yep, disabling that does indeed fix it, but it's not ideal. Should I open an issue there?

@derektedor yes https://github.com/LeikvollE/tileindicators/issues


☂ Proposal: desktop context menu fidelity

The desktop text selection toolbars, as introduced in https://github.com/flutter/flutter/pull/73882 and https://github.com/flutter/flutter/pull/74286, have major limitations that make them not as good as native.  We should either find a way to use the native context menus, or we should work to fix each of these problems.

We can't draw the menu outside of the current window.  Also, the menu can't receive mouse events (like hover) unless the current menu is focused.
The only buttons available are the default mobile buttons (cut/copy/paste/select all).  We should support all of the same buttons that display natively.
It should be possible for Flutter devs to customize the menu.
The native Mac menu has a flashing animation when clicked.
Submenus should be possible and easy to implement.
Currently, only EditableText widgets can show the menu. Right-clicking elsewhere doesn't show it.  Fixing this probably requires refactoring TextSelectionControls to work with both the existing setup and menus (like showMenu).
Menu items that are wider than the default width are ellided, but on native the menu grows to accommodate them.
On Windows and Linux we use a Material-style menu, but we might want to build a menu that looks like the native one instead.
The native Mac context menu is slightly transparent and has a blur effect (screenshots below).

Will this focus on Desktop Mac only? Adding labels just in case.

Will this focus on Desktop Mac only? Adding labels just in case.

We should support all of the same buttons that display natively.

Doing this without using native menus would be non-trivial, due to services

We should support all of the same buttons that display natively.

Doing this without using native menus would be non-trivial, due to services

Something missing from that list: Menus on macOS have a subtle glass effect.

Something missing from that list: Menus on macOS have a subtle glass effect.

@pedromassango This includes the Windows and Linux context menus as well.  I've updated the description to make that more clear and added the labels.

@pedromassango This includes the Windows and Linux context menus as well.  I've updated the description to make that more clear and added the labels.

@shyndman What version of MacOS is that screenshot from?  I've been meaning to upgrade to Big Sur to check if it still looks like that but I haven't gotten around to it.

@shyndman What version of MacOS is that screenshot from?  I've been meaning to upgrade to Big Sur to check if it still looks like that but I haven't gotten around to it.

Ah, good point. I'm still on Catalina too.
Only presuming it's the same, since their design system appears to be leaning even further into glass effects.

Ah, good point. I'm still on Catalina too.
Only presuming it's the same, since their design system appears to be leaning even further into glass effects.

@justinmc, I confirmed on a Big Sur computer:

@justinmc, I confirmed on a Big Sur computer:

@shyndman Thanks a lot for trying that out!  Added to the list above.

@shyndman Thanks a lot for trying that out!  Added to the list above.

This has recently bitten us, as we have a styled right-click menu we built that matches our app, and we'd like to use it instead of this.
Is there any mechanism to turn this off? It seems to be swallowing the secondary pointer event, not allowing it to bubble up.

@esDotDev Hmm we definitely want people to be able to turn this off.  Have you tried passing null to selectionControls?

Just tried it, doesn't seem to be working. There is code like this inside of TextField:
      case TargetPlatform.linux:
      case TargetPlatform.windows:
        forcePressEnabled = false;
        textSelectionControls ??= desktopTextSelectionControls;
        paintCursorAboveText = false;
        cursorOpacityAnimates = false;
        cursorColor ??= selectionTheme.cursorColor ?? theme.colorScheme.primary;
        selectionColor = selectionTheme.selectionColor ?? theme.colorScheme.primary.withOpacity(0.40);
        break;

Maybe some NoControlsControls() we can pass?

Also, some general feedback on this topic, I think we're heading in the wrong direction trying to emulate the OS menus.
If you look at most high profile cross platform apps, they almost all use app-specific context menus.

Slack
Sublime Text
Android Studio
Spotify

All use app-specific context menus. From a developer perspective, they tend to:

Match the app better
Are much easier to test/debug
Are much more flexible and easier to theme
Avoid the "uncanny valley" issues discussed here, where it feels slightly "off", this is generally worse UX than just having a bespoke menu.

Just seems like a nice, general purpose context menu, widget, that can wrap anything, and be styled freely, is really the way to go. We built something recently like this, and it worked very nicely. You can avoid any issues with over-hang on the window just using a quadrant-based positioning scheme.
From a framework SDK POV, these things will be ever-moving targets and very very hard to perfectly match. You'll end up with different OS dot releases, with subtly different expected behavior, and it will just be a bunch of technical debt.

I'm guessing that here is the GestureDetector that is swallowing your right clicks.  And here is the definition of that GestureDetector.  If that's true, then I think we need something that gets rid of the secondary tap handlers on that GestureDetector, not just something to get rid of the selection controls.
I agree with you on allowing bespoke context menus.  I've been doing a bunch of work to allow the context menus to be customized on mobile, and I had planned to do the same thing on desktop, but we decided to keep everything private for now while our understanding of everyone's expectations for desktop matures a bit.
Currently, the TextSelectionControls setup that we've got is dependent on the EditableText-based widgets like TextField.  On desktop though, it's obvious that you need to be able to right click anywhere in the app and get a context menu to show up.  I think TextSelectionControls needs major refactoring, or we need to build something totally separate for desktop for now.

Thanks Justin, thats great to hear. We plan to make our ContextRegion a package, so I'll post that here when it's done as at least some reference (I described it a bit more above in an edit).
Should I file another issue for the disabling of controls?

Yes please do and tag me in the new issue!  Thanks for the explanation in the comment edit.  I like that approach.

Yes please do and tag me in the new issue! Thanks for the explanation in the comment edit. I like that approach.

Hey Justin, I've created a new issue for disabling here:
https://github.com/flutter/flutter/issues/74255
Somewhat relevant to this issue is an example of how we might want to style native menus:
https://user-images.githubusercontent.com/736973/113606909-d8b9f300-9605-11eb-8701-8e187961b1f9.mp4
Things to note here:

we'd like total control over margins and paddings and heights of the btns
we might want to different hover colors depending on type (in our case, "delete" gets a special dark grey color, while the others are orange on hover)
we need to disable an item based on some condition (in our case, "make cover" is disabled if this image currently the cover)
We might want to assign an icon to field
We would like to use a non-system font to better match the application
We'd like to provide our own divider

Popular apps where these sorts of costume (non native looking) menus can be seen are Spotify, Steam and Slack

Thanks for all of the examples!  I think all of this is doable in the long term.  I'd like to get a solution somewhat like what we have for Android with TextSelectionToolbar and TextSelectionToolbarTextButton.
CC @shihaohong who I believe plans to work on nested menus.

fwiw there is no following behavior expected on desktop, resizing a window, or attempting to scroll, should close the context menu.

@esDotDev I've gotten around to refactoring the context menus to be based on widgets: https://github.com/flutter/flutter/pull/107193

https://github.com/flutter/flutter/pull/107193 has been merged and so the default desktop toolbars (DesktopTextSelectionToolbar and CupertinoDesktopTextSelectionToolbar) are now public and customizable.


ci: add matrix for command

I added matrix for command to reduce ci time

Hi @manudeli, I really appreciate the PR, but I'm not sure spinning out a different job for linting vs testing is the best approach since I'll already be using an additional 9 checks for E2E testing across node 16,18,20 across windows, mac, and linux

I understand your concern that the number of actions might become too large. However, I think it's okay to easily merge this Pull Request and add window, mac, and linux later and fix it again if there are any problems. How about proceeding like this?

I understand your concern that the number of actions might become too large. However, I think it's okay to easily merge this Pull Request and add window, mac, and linux later and fix it again if there are any problems. How about proceeding like this?

While I agree merging it would be okay, I'm still not sure the change makes sense for the 2-3 second speed up. I'm going to close the PR we can reopen it in the future if the build times become a problem.


Discussion: Improving performance of memory analyzer

I spent the last few days trying to optimize the performance of memory analyzer. At this point, I wanted to start a discussion on the results, and how we can merge it back into the redis-rdb-tools.
Creating test RDB files
I first needed a set of rdb files to work against. Synthetic rdb files, those generated from redis-benchmark didn't cover all patterns. So I made a set of RDB files from real world data. See

Importing StackExchange Archives into Redis
Import US H1B applications to Redis

For benchmarking, I created askubuntu.rdb file using the stackexchange script. This has about 3.6M keys. Using the unstable version of redis-rdb-tools, generating the memory csv took about 18 minutes on my laptop. This was with python-lzf installed.
(Trying to) Improve Python Code
Looking at the python code, I realized a few things -

To calculate the memory, we don't actually need to load the entire object from the RDB. For example, for strings, just knowing the length of the string and whether it is a numeric value is enough to calculate the memory.
The RdbCallback interface is a big overhead. It is a good interface for other things like JSON conversion where we need all the keys and values, but for memory calculations it's a poor abstraction.
The python code itself isn't written in a IO performant manner.

To address these concerns, I rewrote the memory analyzer in a new python module called indexer.py. With these changes, the CSV generation took about 9 minutes for askubuntu.rdb.
This wasn't good enough, so I tried to profile the python code. It seems we are generating too many objects, and that the IO is not buffered inspite of all my attempts to get it to buffer. I have a few ideas on how to tackle the object creation, but I haven't got on to that yet.
I paused the efforts, and decided to try something else.
Rewriting in C
I wondered what the performance would be like if this was written in C. So I started hacking, and wrote a version that just printed the keys. To my surprise, the program ran successfully in just about 9 seconds! That was a massive improvement, so I started adding all the features.
At this point, I have a 80% working C version. I have also been successful in calling this C function from python, so we can introduce the C code without breaking the current python CLI interface.
Next Steps / Discussion Items

I am hoping to complete the C version, integrate it with the python CLI, and ensure all the existing bugs are fixed. For example, proper quoting of key names, proper escaping of characters in the key, revisit the memory calculation. Do you see any specific concerns, or are there specific implementation bugs I should address while I am re-writing it in C?
I am thinking of dropping support for 32 bit version of redis in the memory calculation. In other words, I am assuming 64 bit redis is used. Do you see a problem with this?
I am introducing a new field in the CSV - "savingsIfCompressed". Since the values are compressed in the rdb but are not compressed in memory - if we were to simply compute ( length - compressed length), we would get a decent idea if it's worth compressing the value pair before saving it in Redis.
I am planning to convert the newly added expiry timestamp to a relative timestamp. Meaning, instead of "key expires at this time", say "key expires in 4 hours". This makes the memory analysis independent of current time. The RDB file already has the creation time in an aux field, so the calculation is easy. In case the aux field is not available, rdb generation time can be found out 1) using file creation time, 2) finding the least expiry time, 3) asking the user.
The current CSV has a column lengthOfLargestElement. This was added so that users can tweak the -ziplist- configurations. This value is easy enough to compute for certain encodings like hashtable, set, skiplist and so, but is expensive for embedded encodings like quicklist, ziplist etc. This is because we have to first uncompress the entire string, and then traverse each element to find the largest element. For hashtables, skiplists etc, the elements are stored as strings with their length prefixed, so we don't have to load the entire string. But the realization is that if rdb is already using an embedded encoding, it is already memory optimized, and we know the lengthOfLargestElement is less than 512 anyways. So - my thought was to only find this value for un-optimized encodings like hashtable/skiplist, and ignore it for embedded encodings. Do you agree with this decision?

P.S.
The source code is in the perf branch over here https://github.com/hashedin/redis-rdb-tools. The code is neither feature complete nor is it written well enough for me to open a pull request. But hopefully I can get some early feedback.
Specifically, two files should be of interest -

rdbtools.c the hacked C version of the code.
indexer.py - the faster version of the current memory parser.

Impressive, and very much needed.
Here are a few notes / concerns:

I see value in keeping a python-only implementation (obviously some, minority of users can't afford using native code)
I see value in keeping the  support for 32 bit, Redis may be used on embedded devices or people who just don't want to waste memory on storing 64 bit pointers if their database is small.
I see value in keeping the memory estimator capability of estimating the memory footprint of different redis versions (not necessarily the last one).
maintaining another parser will be a big overhead (both need to be updated, and tested).
maintaining another memory estimator, formulas, will also be a big overhead, so maybe at least for this one we can try to keep the logic in python and just rely on C parsing.
the savingsIfCompressed seems kinda narrow feature, although a wider one that sends the actual data to the user so that he can try different compression algorithms, will require totally different (and wasteful) approach.
I think that in some cases, people may want to estimate the memory of an RDB file streamed directly form the network, or from a gzip stream, so seek may not be an option.
I agree that it makes little sense to decode ziplist encoded items, considering they are loaded as is into redis. so as long as this parser is only used for memory estimator, maybe we can drop the LengthOfLargestElement altogether.
If we do want to help users optimize zip lists, then i do think we need to decode them and let users see what they're made of, and also i think that LengthOfLargest may not be enough.. we should have a histogram of different sizes.

BTW, over the last couple of years I was trying to convince Antirez to change the RDB parsing code inside redis to a SAX style parser with callbacks, and create a libredis.so that other applications can use. this way the parsing code can be shared and no need for forks or alternative implementations. so far no luck.

Great feedback, exactly what I was hoping for. Thanks! I will work on this feedback this week.

re. 32 bit version, and keeping it working for older versions - Noted, will keep it that way.
re. maintaining two parsers - agree, don't want to do that. Let me try a shared library in C that works across platforms that does the core parsing. Then we can continue to keep the current callbacks, except the callbacks are called from c.
re. Seek - Great catch! I had plans of issuing a SYNC command to download the RDB as part of this library, but my thought was to first save it locally and then parse. But you are right, users would want to use this over streams / gzip. An older version of my code added seek() to the RIO interface, and it defaulted to "read and ignore" in non-file interfaces. Do you think that makes sense?
re. savingsIfCompressed - The idea is to provide a hint "compressing will help" rather than provide an exact value of how may bytes will be saved. The thought is, if LZF saves memory, perhaps you should try out a different algorithm and test it.
re. optimizing ziplists - Not so much as optimizing ziplists. More about optimizing hashes and sortedsets that would otherwise use hashtable/skiplist. A histogram is exactly what I have in mind - actually two histograms. One for number of elements in hashtable/sortedset, and another for the length of field/value/element. The point I was trying to make - this histogram doesn't have to bother about hashes/sortedsets that are already encoded as ziplists.

For the last two points, some context - I'm trying to get towards providing recommendations at some point. Compressing strings if it makes sense, combining multiple strings into a hashmap ala instagram style, optimizing into ziplists etc. are some examples.

Quick update - I've been able to get the memory analysis a lot better. But I haven't got the time to work on the python callbacks. I would like to at least merge the memory analysis part before redis conf, trying to find the time to make it happen.


Feature Request: Display current time

Hello!
I used google chromecast screensaver as photo viewer and there was current time on bottom right corner. I found this very convenient. Could you add such feature in Roku-GooglePhotos screensaver?

I like this feature request. Will keep it in the feature request for action.

In progress

Completed - Needs functional test during UAT. Moving to resolved.  Feature will be released as part of v3.5