vogl:  init at 2016-05-13

Motivation for this change
Packaging https://github.com/ValveSoftware/vogl --
Things done

[x] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[x] Linux (SteamOS)

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

--
Things work, modulo two issues:

https://github.com/NixOS/nixpkgs/issues/9415 -- for which there now is a partial solution: https://github.com/deepfire/nix-install-vendor-gl
you need to tick the Use vogl tool checkbox in the Generate trace.. dialog -- no idea about that yet.

This almost works on Nix/SteamOS.  Haven't managed to make it reliable yet, though.

@ndowens, updated my journal..

Currently qt5base is messed up atm, does qt4 work on this package?

@ndowens, no idea.. the qt5-based vogleditor64 binary runs fine here, though.

@ndowens, there's a build failure that ends with:
error: a ‘aarch64-linux’ is required to build ‘/nix/store/2ssy7xzn086xdbb5lpwhg611ksbmmmvr-options-db.xml.drv’, but I am a ‘x86_64-linux’

I'm not sure what to make of it..

Taking a look at it. Idk if I will be able to try and fix it tonight though

@ndowens, I have restricted platforms to intel linux -- let's see if this fixes it.

You are still using qt5 which fails

@Mic92, fixed, thanks!

@ndowens, how does the qt5base problem manifest itself?

The qt version issue was resolved - the following seems to be odd:
builder for ‘/nix/store/4hz579lfl90ja8kyq2nvbp2rahvcx68f-nixpkgs-tarball-17.09pre1234.abcdef.drv’ failed with exit code 137

error: build of ‘/nix/store/4hz579lfl90ja8kyq2nvbp2rahvcx68f-nixpkgs-tarball-17.09pre1234.abcdef.drv’ failed

The command "./maintainers/scripts/travis-nox-review-pr.sh nixpkgs-verify nixpkgs-manual nixpkgs-tarball nixpkgs-unstable" exited with 100.

The issue we're hitting is https://github.com/NixOS/nixpkgs/issues/24200

looks fine to except that one shared object is located in bin:
$ find /nix/store/cjb971c4k5jkzrcrav59z9grljlvgdln-vogl-2016-05-13
/nix/store/cjb971c4k5jkzrcrav59z9grljlvgdln-vogl-2016-05-13
/nix/store/cjb971c4k5jkzrcrav59z9grljlvgdln-vogl-2016-05-13/nix-support
/nix/store/cjb971c4k5jkzrcrav59z9grljlvgdln-vogl-2016-05-13/nix-support/propagated-native-build-inputs
/nix/store/cjb971c4k5jkzrcrav59z9grljlvgdln-vogl-2016-05-13/bin
/nix/store/cjb971c4k5jkzrcrav59z9grljlvgdln-vogl-2016-05-13/bin/voglgen64
/nix/store/cjb971c4k5jkzrcrav59z9grljlvgdln-vogl-2016-05-13/bin/vogltest64
/nix/store/cjb971c4k5jkzrcrav59z9grljlvgdln-vogl-2016-05-13/bin/libvogltrace64.so
/nix/store/cjb971c4k5jkzrcrav59z9grljlvgdln-vogl-2016-05-13/bin/voglbench64
/nix/store/cjb971c4k5jkzrcrav59z9grljlvgdln-vogl-2016-05-13/bin/vogleditor64
/nix/store/cjb971c4k5jkzrcrav59z9grljlvgdln-vogl-2016-05-13/bin/vogl64
/nix/store/cjb971c4k5jkzrcrav59z9grljlvgdln-vogl-2016-05-13/bin/ktxtool64
/nix/store/cjb971c4k5jkzrcrav59z9grljlvgdln-vogl-2016-05-13/bin/glxspheres64

@Mic92, that is by design -- the program assumes this location and breaks when it is located elsewhere.

Alright, now it works on SteamOS as well!
The only quirk is the requirement to tick the use vogl tool checkbox.

@FRidh, done!


subsurface: add map/unmap events

Fixes https://github.com/swaywm/wlroots/issues/1414

Should we avoid iterating over unmapped subsurfaces when rendering? Should we instruct compositors to check whether or not a subsurface is mapped when rendering?

Err, forgot that part. Hmm. It'd be better to handle all of this mess in wlroots. I'll work on a fix.

Fixed

I don't follow how this new patch addresses my concern.

Oh, yeah, the fix was more about the map/unmap semantics for subsurfaces (described in the original issue).
About how compositors should handle it… Right now for popups we iterate over unmapped ones too. I'm not sure iterating over only mapped surfaces is a good idea, as surface iterators can be used for e.g. wlr_surface_send_{enter,leave}. I'd say the compositor should handle it, although Rootston doesn't do it now (not for subsurfaces, not for popups, not for toplevels, not for xwayland surfaces).

Thanks!


[Suggestion] Allow makeRequest to work with non-environment endpoints

Expected Behavior
Suggestion: I'd like to be able to add roles to a space via this migration framework.
For example:
module.exports = async function (migration, { makeRequest }) {
  let role = await makeRequest({
    method: 'PUT',
    url: `/roles/developer-qa-role`,
    data: {
      "name": "Developer/QA",
      "description": "Allows reading Entries and managing API Keys",
      "policies": [...]
    }
  })
}

Actual Behavior
Currently, when I try this I get a 404.  makeRequests generates the url as spaces/XXXXX/environments/master/roles/developer-qa-role, but the roles endpoint is not available on environments, only on spaces.
I tried not including an environment, but since the default is master and createMakeRequest forces the environment into the url, I can't get it to work at the moment.
Possible Solution
A makeRequest that works on spaces and not just environments
Context
We have many spaces with the same content model and use this library to roll the changes out to all the spaces.  I'd like to be able to roll changes such as creating roles out to all of our spaces w/our normal content model versioning strategy instead of needing one-off solutions.

This ticket has been closed due to inactivity. If you still need help resolving your issue, please reach out through our community Slack, or contact Contentful support directly.


vfxt Provider Add option to disable known_hosts check

unable to reproduce.
The code already defeats knownhosts by using the ssh.InsecureIgnoreHostKey() callback. This is safe because if using public ip for controller you will always get a different public ip, and second, if not using public ip it is on the internal trusted network.


Delete OTP entries without button press or pin entry

Device: NitroKey 3A Mini
Nitropy status:
Command line tool to interact with Nitrokey devices 0.4.38
UUID:               840E500EB2CFB1110000000000000000
Firmware version:   v1.5.0
Init status:        ok
Free blocks (int):  238
Free blocks (ext):  456
Variant:            NRF52

When running nitropy nk3 secrets remove Discord, tells me "Please touch the device if it blinks", but immediately shows "Done", and the key is gone.
Remarkably, when I run nitropy nk3 secrets remove Discord again, it will ask my pin number. Upon entering a valid pin, it asks me again to touch the device if it blinks. If I do this, I get:
peter@apx_managed_aur_dev:~$ nitropy nk3 secrets remove Discord
Command line tool to interact with Nitrokey devices 0.4.38
Please touch the device if it blinks
Credential not found. Please provide PIN below to search in the PIN-protected database.
Current PIN (8 attempts left):
Please touch the device if it blinks
Critical error:
An unhandled exception occurred
	Exception encountered: SecretsAppException(code=6a82/NotFound)

--------------------------------------------------------------------------------
Critical error occurred, exiting now
Unexpected? Is this a bug? Would you like to get support/help?
- You can report issues at: https://support.nitrokey.com/
- Writing an e-mail to<EMAIL_ADDRESS>is also possible
- Please attach the log: '/tmp/nitropy.log.0xd5z6n4' with any support/help request!
- Please check if you have udev rules installed: https://docs.nitrokey.com/nitrokey3/linux/firmware-update.html#troubleshooting

Ofcourse, the error above is expected, since the key is now gone from the chips database. Still, it is odd behaviour.
I reckon this is because my key wasn't added with --protect-with-pin, but it's strange that it asks me to touch the device yet immediately shoots to saying "Done" and deleting the key in question.

Hi! Thank you for the report!
This indeed seems to be a bug. Each modification of the state should be confirmed by a user touch. I will check it further.


[Preview 6] Add Blazor auth documentation

Original spec here: https://gist.github.com/SteveSandersonMS/60ca3a5f70a7f42fba14981add7e7f79
A few of the names changed along the way, but the preview 6 implementation is pretty close to this.

OK, I've put a draft of this doc item here: https://gist.github.com/SteveSandersonMS/175a08dcdccb384a52ba760122cd2eda
@guardrex, is this enough for you to work with? Thanks for your help with this!

@SteveSandersonMS Excellent.
@danroth27 @tdykstra Where do you want the content? This spans authn and authz, so how do you want it handled if it's headed into the Security and Identity node where authn/authz are normally split? Alternatively, do you prefer a new topic in the Blazor TOC?

a new topic in the Blazor TOC?

That would be my choice.

Let's work out the details. I'll try to get this done by EOD on release day (but no promises on that ... let's say "within 48 hours" ... depends on how smoothly things go that day).

Title: Blazor security
Physical path: aspnetcore/blazor/security/index.md
URL: https://docs.microsoft.com/aspnet/core/security/
TOC: Web apps > Blazor > Security
UID: blazor/security/index

This includes a "security" node and makes the topic the index because we'll almost certainly have additional Blazor security topics in the future.
Title/TOC could also mirror existing security topics with a title of Blazor security and identity and TOC of Security and Identity, but I recommend merely "security" for the node name either way.
Not sure if you prefer "overview" tho on the physical path/URL. Let me know on that.
Under this scheme, the content would be easy to shift in the future. For example, authn and authz could be split into new standalone topics if they ever get too heavy for one topic. The index (overview) content can then focus on generic/cross-cutting security (and identity) coverage.

Can we cross link the Blazor security docs from the top level "Security and Identity" node? We don't want Blazor to be a separate island. It's part of ASP.NET Core.

Yes, we can duplicate the TOC nodes.

I modified my TOC remark to track taking care of it. :point_up:


add contextual prompts for RAG prompts

Abstract
This submission was to showcase the injected RAG contextual prompts.

Thanks! I am not able to build the docker compose, yarn fails with:
23.78 - info Linting and checking validity of types...
25.47
25.47 Failed to compile.
25.47
25.47 ./app/components/chat.tsx
25.47 191:17  Error: Missing "key" prop for element in iterator  react/jsx-key
25.47
25.47 ./app/components/kg.tsx
25.47 63:6  Warning: React Hook useEffect has a missing dependency: 'updateConnectionStatus'. Either include it or remove the dependency array.  react-hooks/exhaustive-deps
25.47
25.47 ./app/components/rag-contextual-prompts-modal.tsx
25.47 22:13  Error: Missing "key" prop for element in iterator  react/jsx-key
25.47
25.47 ./app/components/rag.tsx
25.47 104:6  Warning: React Hook useEffect has a missing dependency: 'updateConnectionStatus'. Either include it or remove the dependency array.  react-hooks/exhaustive-deps
25.47
25.47 info  - Need to disable some ESLint rules? Learn more here: https://nextjs.org/docs/basic-features/eslint#disabling-rules
25.53 error Command failed with exit code 1.
25.53 info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
------
failed to solve: process "/bin/sh -c yarn build" did not complete successfully: exit code: 1

@sebeier I'll have a look

@slobentanzer fixed, please have a try.

@fengsh27 FYI I added a version info to the about page, mainly because I want to know whether what I am looking at is in fact the most current version. It is currently hard-coded in the cn.ts and en.ts files; we should probably make that a variable at some point.

@slobentanzer I submitted the rag contextual prompts button, please have a look.
Additionally, I updated the way to get version info in Welcome page. We can bump version by updating package.version in src-tauri/tauri.conf.json.


Create a VRFConsumer v2.5 example contract

[ ] Add this contract https://docs.chain.link/vrf/v2-5/subscription/get-a-random-number#analyzing-the-contract
[ ] Test it with Chainlink Local (pending)

@andrejrakic perhaps we should consider replacing V2 with V2.5.


[3.0] Switch JavaScript examples to fetch

Why?

It's 2018.
jQuery isn't part of JavaScript. Admittedly, neither is fetch. But at least fetch is included in every modern browser, while jQuery is not.

if it's ok i can do this

Removing the 3.0 tag, as this isn't a real blocker. I need to release v3, and don't have time to work on this myself.


#9 - Configure firewall rules and cvmfs on host machines

@ejr004 can you test this code. Works on the dev-kit for me.


Preserve signature item state when generators are added or removed

This can be achieved easily using keys in yew. We have to be careful that yew only uses keys when all children of a node have keys, so we need to assign keys to the new folder button and the drop zones as well.
Unfortunately this is currently blocked by this bug which messes up the order of elements in keyed lists: https://github.com/yewstack/yew/issues/2327 This bug appears to be actively worked on, so I will revisit this issue once the bug is fixed.

Do note that removing and adding items do not trigger the linked bug and should already work correctly.
The bug is triggered by multiple existing keyed items being reordered to another place with respect to other already existing keyed items in the list.

@WorldSEnder ah that's great. So @zrho could you take a look? Seems this bug will not hold us back after all.

The bug is still triggered: when we drag and drop the items, not only the items themselves but also the dropzones are reordered, leading to dropzones which are not adjacent to the correct item. We could potentially integrate the dropzones into the items themselves, though, so that with drag and drop only single items are moved.

If the dropzones are not direct children of the keyed list and only a single item is moved, I'd expect this to work around the bug, yes.

Fixed by 16cff6a96bb435a1b37378c3f72c689e107660ea


Leverage Qiskit Nature's SymmetryTwoBodyIntegrals classes

What should we add?
Qiskit Nature 0.6 added the new SymmetricTwoBodyIntegrals classes. This plugin may be able to benefit from using them.

This issue has been extracted from #22

Is this still open, I would like to work on this.

Feel free to look into this but keep in mind that this is not an easy first issue if you are new to Qiskit (Nature).
I am also not able to provide a lot of guidance at the moment due to time constraints.


ag:.:49: not enough arguments

Hi, I'm running debian, and have installed ag through
sudo apt-get install silversearcher-ag

I use ag mainly through ctrlp.vim set to use the following command:
let g:ctrlp_user_command= 'ag %s -l --nocolor --hidden -g ""'

and it works perfectly, however when I try to run the same command in zsh or just plain ag Something I get the following error
→ ag something
ag:.:56: not enough arguments

The line number seems to change. This also seems weird:
→ which ag
ag () {
        .
}

As far as I can tell, this happened recently, and might be an issue with my setup, as I ran apt upgrade today, but please point me in the direction of resolving this. Thanks!

This was resolved in another terminal window :man_facepalming:


experiment to try and fix

@chrisns I know that feel, when github action keeps fail :rofl: Remember the dynamic matrix was a pain like this for a stupid problem

setup act on my machine now so can at least reproduce locally
theres some weird problems with multi-line params being passed between actions, am looking into it

@chrisns Do you have the comma separeted solution?

@chrisns Ignore my comment I just saw the commits in master, missed them :)
Thanks @chrisns ! :smiley:  We will get the Hacktober fest t-shirt :rofl:


Error when passing an empty array as binds to function db->query

I got an error when trying to pass an empty array as binds to the function query. Debugging, I found that the problem occurs if my query have and exclamation mark that it's not a bind mark. In my case it's an url.
SELECT CONCAT('http://example.com/index.php?op=3')
Besides that, I pass an empty array as binds (empty because I build the bind variable dinamically and the array starts as empty). So, my code is something like this:
$sql = "SELECT CONCAT('http://example.com/index.php?op=3')"; $this->db->query($sql, array());
Running this, I got a loop of notices in a while that never ends. It's the while at the end of function compile_binds at DB_driver.php. For some reason, the $c variable is zero, and the $c-- at the begining of the code makes the while never stops.
In this cennario I have a query with no valid bind marks and an empty array as bind. This is the error case. If I pass null as binds, it works (it's the workaround that I used to solve my problem), but the bug exists.
I think that changing the do...while to while...do will work, since the verification would happen first and $c wouldn't be lesser than zero.

Passing NULL (or not passing anything) is the correct way to do this.
Passing an array, empty or not, indicates that the function should look for bind markers - that's not a bug.


OpenVPN Protocol TLS-AUTH

Hello,
I have a softether developer edition server.
My friend at university cannot connect with tcp port 443 using openvpn protocol.
However, it can successfully connect to the openvpn server that I installed using the angristan openvpn-install script, again using the tcp 443 port.
The script I use: https://github.com/angristan/openvpn-install
When I use the script there is something like this in the client file
<tls-auth>
#
# 2048 bit OpenVPN static key
#
-----BEGIN OpenVPN Static key V1-----
blablabla
-----END OpenVPN Static key V1-----
</tls-auth>

But tls-auth is not available in SoftEther openvpn file.
Unable to connect without tls-auth I guess due to firewall.
How can I add tls-auth to Softether OpenVPN file?
Thanks for your help.

Duplicate of #1798


Autoplay & Play Next Video Feature

PR Fixes:

1  Added Autoplay next video, when the video ends , the next video will be autoplayed.
2  Also added a button, clicking on which plays the next video.

Screencast
Screencast from 15-04-24 02:06:12 PM IST.webm
Resolves #384
Checklist before requesting a review

I have performed a self-review of my code
I assure there is no similar/duplicate pull request regarding same issue

Bro, can I add a button here? That would be so good.

Yeah !! That would be cool !!


Feature/geode 1256

GEODE-1256  Alter rat.gradle to exclude copies of website sources
Added `geode-site/content/**` to the rat.gradle excludes list
such that there are no longer 10 Unknown Licenses.

+1
On Mon, May 16, 2016 at 3:12 PM, Karen Miller<EMAIL_ADDRESS>wrote:

GEODE-1256 Alter rat.gradle to exclude copies of website sources
Added geode-site/content/** to the rat.gradle excludes list
such that there are no longer 10 Unknown Licenses.

You can view, comment on, or merge this pull request online at:
https://github.com/apache/incubator-geode/pull/143
Commit Summary

GEODE-1028: Fix broken link on website's Releases page
GEODE-1256 Alter rat.gradle to exclude copies of website sources

File Changes

M geode-site/website/content/releases/index.html
https://github.com/apache/incubator-geode/pull/143/files#diff-0 (2)
M gradle/rat.gradle
https://github.com/apache/incubator-geode/pull/143/files#diff-1 (1)

Patch Links:

https://github.com/apache/incubator-geode/pull/143.patch
https://github.com/apache/incubator-geode/pull/143.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/apache/incubator-geode/pull/143

+1


Additional examples

File: themes/default/content/docs/intro/concepts/config.md
Could we please add an example on how to load configs based on namespaces that are not the same as the project name?
e.g.
Pulumi.<stack>.yaml
config:
  management:resource_groups:
    - name: infra
    - name: security

Accessing the config using Python (for example)
import pulumi

management_conf = pulumi.Config("management")
resource_groups = management_conf.get_object("resource_groups")

For some reason it wasn't clear to me that I needed to pass the namespace to the Config() constructor. I was using pulumi.Config() instead.

Hi @tksrc
So we have this in the docs right now:
The following example uses an empty constructor. If you are writing code that will be imported into a broader project, such as your own library of components, you should pass your library’s name to the constructor. This string is used as a namespace for all configuration keys.

If you think we should be more clear here then we can certainly add a set of examples?
Paul

Thanks Paul. Maybe its just me, but I usually glance through the docs, look at the examples first and if I do not find what I am looking for, then I read carefully.
I did read the paragraph under https://www.pulumi.com/docs/intro/concepts/config/#code but it was not immediately clear that I could just pass a namespace name to Config().
Perhaps a short example would help support the explanation.

Ok that makes sense :) I'll add something in

This is now mentioned a couple times and there is an explicit example of this form at https://www.pulumi.com/docs/intro/concepts/config/#code - so going to consider this done.


perf: VID ADVZ verify_share use parallelism over multiplicity

closes: #646
This PR does / key places to review:

use parallelism here
new benchmark in vid/benches/advz_multiplicity.rs demonstrates perf improvement
My local environment disallows me from testing whether these changes break the build when gpu-vid feature is enabled. Last time this issue arose @mrain had to check the build using his nvidia gpu.

How to test this PR:
From inside the vid/ directory run
RAYON_NUM_THREADS=N cargo bench --bench=advz_multiplicity --features="test-srs"

On my local machine I observe significant improvement when I increase N from 1 to 6:
N=1:
advz_verify_payload_256KB_multiplicity/1
                        time:   [28.000 ms 29.097 ms 30.711 ms]
advz_verify_payload_256KB_multiplicity/256
                        time:   [364.81 ms 367.66 ms 371.66 ms]

N=6:
advz_verify_payload_256KB_multiplicity/1
                        time:   [27.515 ms 27.925 ms 28.442 ms]
                        change: [-5.5630% -2.1819% +0.6662%] (p = 0.24 > 0.05)
                        No change in performance detected.
advz_verify_payload_256KB_multiplicity/256
                        time:   [87.886 ms 89.704 ms 92.870 ms]
                        change: [-75.541% -74.950% -74.290%] (p = 0.00 < 0.05)
                        Performance has improved.

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against correct branch (main)
[x] Linked to GitHub issue with discussion and accepted design OR have an explanation in the PR that describes this work.
[x] Wrote unit tests
[ ] Updated relevant documentation in the code
[ ] Added relevant changelog entries to the CHANGELOG.md of touched crates.
[x] Re-reviewed Files changed in the GitHub PR explorer

nit: 2 VID tests in advz::bytes_to_field are not working. Try cargo test -p jf-vid --features test-srs --release -- --ignored for details.

These doctests are ignored intentionally due to a Rust limitation that disallows access to private identifiers in doctest: Doctest of private functions - help - The Rust Programming Language Forum 😕

And what's up with all the clippy warnings(?)

Github CI is using a newer version of Rust than our Nix environment. We need to update Nix flake again. We'll do that in another PR.


Cleanup of security-boot-software-updates-stm32mp1

Changes made according to the style guidelines and to improve readability.
QA Steps: Ran Linter/spellcheck, built and visually inspected output. No issues spotted in rendering.
This commit applies to FFTK-2795
Readiness

[x] Merge (pending reviews)
[ ] Merge after date or event
[ ] Draft

Overview
Why merge this PR? What does it solve?
Checklist
Optional. Add a 'x' to steps taken.
You can fill this out after opening the PR. "Did I..."

[x] Run spelling and grammar check, preferably with linter.
[x] Avoid changing any header associated with a link/reference.
[ ] Step through instructions (or ask someone to do so).
[x] Review for wordiness
[x] Match tone and style of page/section.
[ ] Run make linkcheck.
[x] View HTML in a browser to check rendering.
[x] Use semantic newlines.
[x] follow best practices for commits.

[x] Descriptive title written in the imperative.
[x] Include brief overview of QA steps taken.
[x] Mention any related issues numbers.
[x] End message with sign off/DCO line (-s, --signoff).
[x] Sign commit with your gpg key (-S, --gpg-sign).
[x] Squash commits if needed.

[x] Request PR review by a technical writer and at least one peer.

Comments
Any thing else that a maintainer/reviewer should know.
This could include potential issues, rational for approach, etc.

Docs for 880512facd60f8ff5c71ff248afae1294256ca79 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/2584/docs/artifacts/html/index.html


Fixing GH-737 : compute class type with generics handle

Fixes #737
Fixes #209 (to check if it was dependant of #737)
Add generic handle in xjc:javaType
The name tag should be of the form : name="java.util.List<java.lang.String>" for expected List<String>
Need also to provide good adapter that implements the wanted XmlAdapter<String, ExpectedType> where ExpectedType with the example would be List<String> like CommaDelimitedStringAdapter

@lukasj : forced push this PR with wanted changes 😉


Add installation instructions

Follow https://developers.home-assistant.io/docs/add-ons/repository/ to add the repository.yaml file and a "My HA" link for easy install.

Thank you for fixing this and making installation clearer! I'll try it out


some editor shortcuts dont work inside tileset editor

i went to to disable grid snapping in tileset editor and i found that it dont work there
is there another shortcuts for it?
and thanks

Shortcuts still work in the Tileset Editor.
Grid Snapping has the shortcut Shift+G

Shortcuts still work in the Tileset Editor.
Grid Snapping has the shortcut Shift+G

not working in tileset editor i tried it

Maybe Hardware specific?
Did you try it on other devices?

Maybe Hardware specific?
Did you try it on other devices?

no i didn't
but i don't think its the problem
i think they didn't implement it in godot

With Tileset Editor, do you mean that:

Or that:

The second one

The second one
I needed the shortcut because I was editing my tile and when i do the collision layer every time I have to disable grid snapping every time

Oh, that makes it much clearer.
This is then more of a proposal, you want shortcuts to work in the tilemap editor, so open a Issue at godotengine/godot-proposals and close this one.
The main Github Repo is now only for bug tracking.

oh sorry i thought it should work in there


Pin the important Notes

Add a feature that the user can pin his/her important notes according to his/her choice.

!assign

Sure @yogesh-9999


change asar to unpacked files

https://www.electron.build/configuration/configuration search asarUnpack

possibly fixed?


Configurability of Position Widget @Felix-Auinger

Outdated until we have a position / map widget again 😉


Tidy up document-page stylesheet

The only page which still uses the .document_page class is the
detailed guide page. Move the Sass from the .document_page into the
detailed guide Sass file so it is clearer as to what is effected by the
Sass.

This looks good :+1:
One very minor thing, it looks like the bookmarklet might be using this class to find the article ID:
https://github.com/alphagov/whitehall/blob/8941f7abe2c1c7247c407204c06bed1668216f62/app/views/admin/find_in_admin_bookmarklet/_bookmarklet.erb#L11-L13

I was unaware of that bookmarklet. If this is a thing people use we should really come up with a way of properly exposing document-ids rather than scraping them from ids. Updated the selector to continue to work with detailed guides.

:+1:


filter challenge map

Added filters for completed/optional challenges.
It looks like the map is moving over to react so this will need to be reimplemented, but wanted to get feedback on the design of the filters.
Replacing the Collapse all button with a dropdown doesn't use any more screen real-estate which was a concern, but I'm not sure the filter state (i.e. changing color and text) is as clear as it could/needs to be.
closes #6680

Hey @geoff616
Thanks for the PR. Sorry about the delay in actually getting it QA'd. The code looks well written.
A couple of changes I'd like to see:
If a block is empty (all the challenges are filtered out) then the block should also be removed. Otherwise it looks like there is some bug when most of the challenges are completed.
If I select an option in the dropdown, it should stay open so I can select multiples at once.
I've included some screen caps. I know @QuincyLarson is against drop downs but I don't know if there would be a better way in this case.

I think the button would need a drop down arrow to show how it will work.  It should probably expand the other buttons below it, pushing everything down, which will work better on mobile.

 Adding an arrow or some sort of indication of what to expect from the component will decrease confusing behavioural dissonance.
Furthermore, it doesn't have to be a "drop down" per se. It can simply be a component that displays hidden information in a menu.

As @BerkeleyTrue said, I am against drop downs, as they complicate the UI and often don't work properly on mobile. Here's a more formal explanation of why they should be used sparingly: https://www.nngroup.com/articles/drop-down-menus-use-sparingly/
I am not rejecting this out of hand, but rather asking: what percentage of campers are likely to use this feature?
If it's a super-user type feature that a small minority of campers are likely to use, we should just use a hotkey instead of buttons.
If it is something that would be universally useful, could we structure it as a button or series of buttons instead of a dropdown menu? (As noted in the issue, we did originally have buttons for filtering and we got rid of them - the complexity wasn't worth the functionality.)

I do think that we need this feature, one way or another.  I'll admit that most of these use cases are uncommon, but they absolutely do exist and they are not possible to achieve in any other way. All of these are pretty common requests that we've gotten.
It seems to me that it should be possible to achieve this without adding too much interface complexity.

Thanks for all of the feedback, and sorry for the delayed reply.
I don't have data to back this up, but I would guess 20% would use the filters. Especially as campers get father along in the challenges, these filters become more important to help identify what is left to do.
I'll play around with trying not to use a dropdown and update the PR.

Closing as abandoned. Feel free to re-open when ready.
Happy Coding!


Add MCU-based Ultrasonic Sensor readings on ports A1 and A3

Converted UltrasonicSensor class to be able to choose from different UltrasonicSensor implementations (RPI or MCU) based on the port given by the user.
Note: factory method would have been preferred but this option left generated documentation the cleanest.
Added an ultrasonic sensor base class to avoid duplication of identical methods across both implementations

To do

Implement when_activated, when_deactivated, wait_for_active and wait_for_inactive for MCU mode. Will need to add threading and decide how often to read the ultrasonic sensor, not sure how GPIOZero implements this
The MCU currently retains the last value that was read from the ultrasonic sensor even if the sensor has been unplugged. Ideally, we want it to revert back to default state - can't do without firmware change, skip for now
Behaviour of RPI ultrasonic readings is to return the max distance if the reading fails (e.g. if the ultrasonic is not plugged into the correct port), the MCU returns 0.0 in this case so we should probably catch that and return max_distance instead.
Add expansion plate firmware version check on initialization

Bizarre problem discovered (for reference of code, now fixed)
Whilst testing ultrasonic with FaceDetector class, I found they were incompatible if the FaceDetector was instantiated after the UltrasonicSensor on an analog port. Tracked it down to the loading of the dlib shape predictor causing the issue - see comments in code below:
from pitop.pma.ultrasonic_sensor import UltrasonicSensor
from time import sleep
import dlib

ultrasonic = UltrasonicSensor("A1")

# This takes a long time to load and blocks the heartbeat signal from sending to EP
# Causing the EP's ultrasonic sensor configuration to reset (along with all other configs)
# distance readings become 0, and any other commands expecting a specific configuration will also not behave as expected
predictor = dlib.shape_predictor("shape_predictor_68_face_landmarks.dat")

while True:
    print(f"Distance: {ultrasonic.distance}")  # This would now be 0.0 since A1 configuration is reset
    sleep(0.1)

To fix, heartbeat signal shutdown time can be increased as per d803aacad10bd8611b1cbde8f90ccd39a8912e48
Other options:

Look into using a scheduler for the heartbeat signal instead of while True with sleep, since sched has a priority level it may still work even whilst loading the dlib shape predictor.
Load shape predictor in a separate thread - this may not actually work since the heartbeat and shape predictor loading is still happening in two distinct threads anyway. Also a thread that loads the predictor may cause timing issues with the rest of the program.

Increasing timeout to "solve" a hanging thread issue seems like it's ignoring the fundamental issue. However, if it's strictly internal, then I suppose we can release that and create a new Issue for addressing the underlying problem.
Finding the diff very hard to read - can you add example code that shows how the user actually handles this, and how the user code is different when using a factory method? Would also like to know what the "unclean" factory method docs look like so we can make an informed decision.
Is this a breaking change for the SDK?

Increasing timeout to "solve" a hanging thread issue seems like it's ignoring the fundamental issue. However, if it's strictly internal, then I suppose we can release that and create a new Issue for addressing the underlying problem.

Not sure how to solve the underlying issue, but yeah we can create a ticket for it.

Finding the diff very hard to read - can you add example code that shows how the user actually handles this, and how the user code is different when using a factory method? Would also like to know what the "unclean" factory method docs look like so we can make an informed decision.

User doesn't need to do anything differently, it's exactly the same from their perspective. For the factory method, they would be returned the correct class based on their input so again the user code would be no different. I don't know how to deal with documentation for that, I just followed the same way we did this for the camera which seemed like the least friction approach.

Is this a breaking change for the SDK?
Shouldn't be no

To fix, heartbeat signal shutdown time can be increased as per d803aac
Other options:

Look into using a scheduler for the heartbeat signal instead of while True with sleep, since sched has a priority level it may still work even whilst loading the dlib shape predictor.
Load shape predictor in a separate thread - this may not actually work since the heartbeat and shape predictor loading is still happening in two distinct threads anyway. Also a thread that loads the predictor may cause timing issues with the rest of the program.

It's likely this problem cannot be fixed with threading in python due to the GIL, if file is being read for several seconds no other thread can execute during this time. Asyncio loop may be able to continue while this is happening but would be a lot of work to add alongside threading in our sdk.
I think a simple, viable solution might be to run the heartbeat loop as a subprocess, which can run concurrently and only has the downside of being awkward to extract output which shouldn't matter in that use case.

Let's split this out. The timing issue is totally separate from the intention of this PR and it's muddying the conversation.

I think a simple, viable solution might be to run the heartbeat loop as a subprocess, which can run concurrently and only has the downside of being awkward to extract output which shouldn't matter in that use case.

That's a pretty good idea! We'll need this heartbeat time extension for now anyway, but can revert it back once we have a better solution

I think we need to resolve what happens when you try to instantiate ultrasonics on both ports A1 and A3 at the same time.
As I expected, this generally didn't seem to work and I would get errors related to the device id or firmware version, usually this:
Traceback (most recent call last):
  File "ultra.py", line 11, in <module>
    ultrasonic1 = UltrasonicSensor("A3")
  File "/home/pi/pi-top-Python-SDK/pitop/pma/ultrasonic_sensor.py", line 32, in __init__
    name=name
  File "/home/pi/pi-top-Python-SDK/pitop/pma/ultrasonic_sensor_base.py", line 104, in __init__
    self.__compatibility_check()
  File "/home/pi/pi-top-Python-SDK/pitop/pma/ultrasonic_sensor_base.py", line 233, in __compatibility_check
    raise RuntimeError("You are trying to use analog ports for the Ultrasonic Sensor, this is not compatible "
RuntimeError: You are trying to use analog ports for the Ultrasonic Sensor, this is not compatible with a Foundation Plate. Please use an Expansion Plate instead.

And occasionally this:
pi@pi-top:~/pi-top-Python-SDK $ python3 ultra2.py
Traceback (most recent call last):
  File "ultra2.py", line 10, in <module>
    ultrasonic = UltrasonicSensor("A3")
  File "/home/pi/pi-top-Python-SDK/pitop/pma/ultrasonic_sensor.py", line 32, in __init__
    name=name
  File "/home/pi/pi-top-Python-SDK/pitop/pma/ultrasonic_sensor_base.py", line 104, in __init__
    self.__compatibility_check()
  File "/home/pi/pi-top-Python-SDK/pitop/pma/ultrasonic_sensor_base.py", line 236, in __compatibility_check
    firmware_device = FirmwareDevice(plate_id)
  File "/usr/lib/python3/dist-packages/pitopcommon/firmware_device.py", line 80, in __init__
    "Part name provided does not match. {} != {}".format(hex(self.part_name), hex(self.get_part_name())))
pitopcommon.firmware_device.PTInvalidFirmwareDeviceException: Part name provided does not match. 0x2222 != 0x2222

However, I did actually find that I was sometimes able to get correct readings from both ports without error, when instantiating the sensors in two separate python scripts run separately at the command line. So perhaps the hardware/firmware does allow for this? I only have one sensor at the moment so I was swapping it between the ports, and found the reading would stay unchanged for the device that was not plugged in until I replaced it, then it would work again.
When I was getting these errors I run pi-top devices to confirm I was not having issues communicating with the hub/expansion plate and it all seemed ok.
I think we need to either make this work reliably if possible, or improve the error messages, validation and docs to explain that this is not possible.

I would get errors related to the device id or firmware version

I've tracked this down to an incompatibility between the two different i2c methods we have in the SDK. The legacy I2CDevice class (which is non-standard I2C that somehow made into the SDK a long time ago) is used for any registers related to firmware updating (version, part number etc) and hub communication yet we use normal SMBus for interacting with plates.
Since the first instantiated ultrasonic sensor is already reading data at 10 Hz it appears to corrupt the second ultrasonic compatibility check that uses the I2CDevice class (inside FirmwareDevice()). I confirmed this by increasing the time between consecutive reads in the ultrasonic read loop, when I set it to 1 second as opposed to 0.1 seconds it seems to work fine - i.e. the part names and version numbers are not corrupted.
Interestingly, I did a test by running pi-top devices whilst I was running a single ultrasonic sensor on loop in another terminal - this seems to work fine - I expected this to fail in the same way but it doesn't seem to be impacted. So perhaps running in the same process is the problem, but in separate processes it's fine - this is consistent with what you found too.

So perhaps running in the same process is the problem

Just put the compatibility check into its own process and it works fine now, I'd prefer to try and get closer to the root cause before implementing this though

@angusjfw try out 908dae9 and let me know what you think

Yeah, this seems to work reliably now when creating two sensors in one script. When running two scripts it's often fine but can still fail, I assume with the same issue that the data reads corrupt the compatibility check.
This seems reasonable to me for now, we can try to fix the legacy I2CDevice issue later.

When running two scripts it's often fine but can still fail, I assume with the same issue that the data reads corrupt the compatibility check.

That's annoying, I thought it worked fine in separate processes. I just tried the same thing and yeah it's giving me the "no expansion plate attached" error 4/5 times. The weird thing is that running pi-top devices in the terminal never fails regardless of what's running, it always says that I have an Expansion Plate attached - need to dig into why that isn't failing yet the second ultrasonic sensor is

This seems reasonable to me for now, we can try to fix the legacy I2CDevice issue later.

I wish we could just change all the firmware to use SMBus but we're kind of stuck with it now.

The weird thing is that running pi-top devices in the terminal never fails regardless of what's running, it always says that I have an Expansion Plate attached - need to dig into why that isn't failing yet the second ultrasonic sensor is

ok it does fail sometimes, perhaps we just need to implement a global lock on any i2c transactions regardless of whether it's the I2CDevice or SMBusDevice. Kind of a big change though and I'm not sure of the overhead of doing that for every single transaction. To be honest I thought this stuff would be taken care of on the kernel level, clearly not

just fixed references to pitopcommon that were causing this PR to fail. is there any reason not to merge this in?

just fixed references to pitopcommon that were causing this PR to fail. is there any reason not to merge this in?

Thanks! Angus and Liv are happy with it, so if you could merge that would be great 👍


Tensorboard not having scalar tab

hi,
I successfully installed & launched tensorboard but encountered a problem, shown as below.
this is the screen shot shown in jupyter notebook:

this is the screen shot using the native tensorboard:

As you can see there is missing a scalar tab, then I checked jupyter notebook server message:
Starting AddRunsFromDirectory: /home/user/mylogdir
GetLogdirSubdirectories: Starting to list directories via walking.
Adding run from directory /home/user/mylogdir/logs/20200609-104720/train
Adding run from directory /home/user/mylogdir/logs/20200609-104720/validation
Done with AddRunsFromDirectory: /home/user/mylogdir
Beginning EventMultiplexer.Reload()
Reloading runs serially (one after another) on the main thread.
No path found after /home/user/mylogdir/logs/20200609-104720/train/events.out.tfevents.1591670841.ip-XX.XX.XX.XX.2219.146.v2
No path found after /home/user/mylogdir/logs/20200609-104720/validation/events.out.tfevents.1591670846.ip-XX.XX.XX.XX.2219.12667.v2
Finished with EventMultiplexer.Reload()
TensorBoard reload process beginning
Starting AddRunsFromDirectory: /home/user/mylogdir
GetLogdirSubdirectories: Starting to list directories via walking.
Adding run from directory/home/user/mylogdir/logs/20200609-104720/train
Adding run from directory /home/user/mylogdir/logs/20200609-104720/validation
Done with AddRunsFromDirectory: /home/user/mylogdir
TensorBoard reload process: Reload the whole Multiplexer
Beginning EventMultiplexer.Reload()
Reloading runs serially (one after another) on the main thread.
No path found after /home/user/mylogdir/logs/20200609-104720/train/events.out.tfevents.1591670841.ip-XX.XX.XX.XX-132.2219.146.v2
No path found after /home/user/mylogdir/logs/20200609-104720/validation/events.out.tfevents.1591670846.ip-XX.XX.XX.XX.-132.2219.12667.v2
Finished with EventMultiplexer.Reload()
TensorBoard done reloading. Load took 0.002 secs

I think its the tfevent not loading correctly for the filename having a .ipXXXX comes behind.
Related information is as follows:
tensorflow                        2.2.0
tensorflow-estimator              2.2.0
tensorflow-ranking                0.1.3

jupyter                           1.0.0
jupyter-client                    5.3.4
jupyter-console                   6.0.0
jupyter-contrib-core              0.3.3
jupyter-contrib-nbextensions      0.5.1
jupyter-core                      4.6.1
jupyter-highlight-selected-word   0.2.0
jupyter-latex-envs                1.4.6
jupyter-nbextensions-configurator 0.4.1
jupyter-tensorboard               0.2.0
jupyterlab                        1.0.0
jupyterlab-git                    0.9.0
jupyterlab-gitlab                 0.2.0
jupyterlab-server                 1.0.0
jupyterthemes                     0.20.0

jupyter-tensorboard               0.2.0
tensorboard                       2.2.1
tensorboard-plugin-wit            1.6.0.post3

JupyterLab v1.0.0
Known labextensions:
   app dir: /home/tim_su/.local/share/jupyter/lab
        @jupyter-widgets/jupyterlab-manager v1.1.0  enabled  OK
        @jupyterlab/celltags v0.2.0  enabled  OK
        @jupyterlab/git v0.9.0  enabled  OK
        @jupyterlab/shortcutui v0.5.0-beta.2  enabled  OK
        @jupyterlab/toc v3.0.0  enabled   X
        databrickslabs-jupyterlab-statusbar v1.0.0  enabled  OK
        nbdime-jupyterlab v2.0.0  enabled   X

config dir: /home/tim_su/.jupyter
    jupyter_nbextensions_configurator  enabled
    - Validating...
      jupyter_nbextensions_configurator 0.4.1 OK
    jupyterlab_git  enabled
    - Validating...
      jupyterlab_git  OK
    jupyter_tensorboard  enabled
    - Validating...
      jupyter_tensorboard 0.2.0 OK
config dir: /usr/local/etc/jupyter
    jupyterlab  enabled
    - Validating...
      jupyterlab 1.0.0 OK
    nbdime  enabled
    - Validating...
      nbdime 1.1.0 OK
    jupyterlab_git  enabled
    - Validating...
      jupyterlab_git  OK

Hi @TimJLS , did you find a solution to this? Actually, you can open Scalar tab by pressing on "Inactive>Scalar", but I am curious why it happens.

Same here, the page doesn't show data but "No dashboards are active for the current data set" unless I manully switch to "Inactive -> Scalars"

Same here, the page doesn't show data but "No dashboards are active for the current data set" unless I manully switch to "Inactive -> Scalars"

The Scalar tab should work with TB >= 2.2 with https://github.com/lspvic/jupyter_tensorboard/pull/63 ; could you give that a try, please?


Add support for component nagios

http://camel.apache.org/nagios.html

No user posts in 2016 & 2015


⚠ GitLab Server has degraded performance

In bee267c, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 6318 ms

Resolved: GitLab Server performance has improved in 690be95.


prob  macro - conflicting chain and variable on rhs

@trappmartin raised an issue in #997 about what happens when a variable, e.g. x below, is defined in chain but is also passed in explicitly. We can either throw an error (my favorite) or over-write the value from chain with the value passed in explicitly.
using Turing

@model demo(y, ::Type{T} = Float64) where {T} = begin
    x = Vector{T}(undef, 10)
    x .~ Normal()
    y .~ Normal.(x, 1.0)
end
chain = sample(demo(rand(10)), HMC(0.64, 1), 1000)

# log likelihood
logprob"y = rand(10) | model = demo, chain = chain, x = rand(10)"

For something like this, I think we can probably overwrite the value and not throw an error. It seems obvious to me that whoever wrote that query intended x=rand(10) to be the case, regardless of what was in the chain. Take for example someone who had a slightly different model:
using Turing

@model demo(y, ::Type{T} = Float64) where {T} = begin
    x = Vector{T}(undef, 10)
    x .~ Normal()
    z = Vector{T}(undef, 10)
    z .~ Normal()
    y .~ Normal.(x + z, 1.0)
end
chain = sample(demo(rand(10)), HMC(0.64, 1), 1000)

# log likelihood
logprob"y = rand(10) | model = demo, chain = chain, x = rand(10)"

In this case, I understand the query to mean "use x=rand(10) and z values from the chain". It seems to me we should permit this kind of intentioned behavior.

Makes sense :)


updated manifest

added new recalbox 4.0.0 systems to manifest

@rockaddicted or @digitalLumberjack you are most aware than me to valid this merge, if the XML is valid, it's ok.


Is it possible to cancel the restriction that the image size is a multiple of 64 pixels

It's difficulty to calculate accurate resolution with the photos, like normal 1440 x 1920

No, but you don't need to do that. Just slap 1024 or sth and you are fine. You can also use PixelPerfectResolution


Error installing plugin Cannot find module './server'

I am trying to use the plugin following the official guide but i am getting the following error
Error: Could not load js config file                                                                           ││<EMAIL_ADDRESS>Cannot find module './server'      ││   Require stack:                                                                                ││   -<EMAIL_ADDRESS>                                 ││   -<EMAIL_ADDRESS>                                                       ││   -<EMAIL_ADDRESS>                  ││   -<EMAIL_ADDRESS>                 ││   -<EMAIL_ADDRESS>                                             ││   -<EMAIL_ADDRESS>                                                        ││   -<EMAIL_ADDRESS>                                                      ││   at loadJsFile                                                                                ││   (/home/folks/node_modules/@strapi/strapi/dist/core/app-configuration/load-config-file.js:18:13)                ││   at Module.loadFile                                                                                ││   (/home/folks/node_modules/@strapi/strapi/dist/core/app-configuration/load-config-file.js:37:14)                ││   at Object.loadPlugins (/home/folks/node_modules/@strapi/strapi/dist/core/loaders/plugins/index.js:90:41)       ││   at async Strapi.loadPlugins (/home/folks/node_modules/@strapi/strapi/dist/Strapi.js:311:5)                     ││   at async Promise.all (index 3)                                                                                ││   at async Strapi.register (/home/folks/node_modules/@strapi/strapi/dist/Strapi.js:341:5)                        ││   at async Strapi.load (/home/folks/node_modules/@strapi/strapi/dist/Strapi.js:425:5)                            ││   at async Object.develop (/home/folks/node_modules/@strapi/admin/dist/_chunks/index-sNH2VWbC.js:837:28)         ││   at async develop (/home/folks/node_modules/@strapi/admin/dist/_chunks/develop-YNTGWKMC.js:61:5)                ││   at async Command.parseAsync (/home/folks/node_modules/commander/lib/command.js:923:5)
Screenshots

System

Node.js version: v20.11.0
NPM version: 0.2.4
Strapi version: v4.19.1
Database: Postgress
Operating system: ubuntu 22.04

hi, same

I also had this. There's a bunch of peer dependencies not listed on the installation instructions, you need to add these separately then it worked:
yarn add<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>This was the warnings on installing strapi-plugin-loction:
yarn install v1.22.21
warning ../package.json: No license field
[1/5] Validating package.json...
[2/5] Resolving packages...
[3/5] Fetching packages...
[4/5] Linking dependencies...
warning "@notum-cz/strapi-plugin-location > @strapi/design-system ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@notum-cz/strapi-plugin-location > @strapi/design-system ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@notum-cz/strapi-plugin-location > @strapi/design-system ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@notum-cz/strapi-plugin-location > @strapi/design-system ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@notum-cz/strapi-plugin-location > @strapi/design-system ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning "@notum-cz/strapi-plugin-location > @strapi/design-system > @uiw/react-codemirror ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@notum-cz/strapi-plugin-location > @strapi/design-system > @uiw/react-codemirror ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@notum-cz/strapi-plugin-location > @strapi/design-system > @uiw/react-codemirror ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@notum-cz/strapi-plugin-location > @strapi/design-system > @uiw/react-codemirror ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@notum-cz/strapi-plugin-location > @strapi/design-system > @uiw/react-codemirror ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@notum-cz/strapi-plugin-location > @strapi/design-system > @uiw/react-codemirror > @uiw/codemirror-extensions-basic-setup ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>

@pturner1989 how can i do that using npm or can i use both of them in my project?

it still happen

any update on this ?

@raniaamina do you any other option for managing and querying the latitude and logitude.

@raniaamina do you know any other option for managing and querying the latitude and logitude.

Currently just save it as string.

@raniaamina do you know any other option for managing and querying the latitude and logitude.

Currently just save it as string.

are you using filtering based on the latitude and longitude

@raniaamina do you know any other option for managing and querying the latitude and logitude.

Currently just save it as string.

are you using filtering based on the latitude and longitude

It depend on what front end need, simply use regex if you wanna filter the format. This one of them;
https://ihateregex.io/expr/lat-long/
Btw, this conversation is out of topic issue, imo.

Still not working ... and i already installed the dependencies listed on @pturner1989 comment :(

hi, any news ?

If you want to use location based filtering you can do it with custome code
First enable the postgis extension in postgres database and the customize the endpoint like this
SELECT
id,
name
FROM
public.store
WHERE
ST_DWithin(
ST_Transform(ST_SetSRID(ST_MakePoint(77.3433442537524, 28.58118352566312), 4326), 3857),
ST_Transform(ST_SetSRID(ST_MakePoint(longitude, latitude), 4326), 3857), 1000)
ORDER BY
distance;

This is a duplicate of #126

hi, any news ?

Hi, it should be fixed! :)


Abstract nextState for ActionState & BehaviorState not necessary

If we are passing the nextState into the constructor then the nextState defined in the class ends up being trivial as a way to simply fulfill the requirement that the class has a nextState function.

I previously brought up this concern with @mitchkm. He is of the opinion that people writing states should think about what a default transition for every State should look like.


🐛 Bestiary Indicies seems to be incorrectly built when building Flee Mortals! Preview

When building my compendium, I am using multiple sources (5e-tools, TheGiddyLimit/HomeBrew, TheGiddyLimit/Unearthed-Arcana). One source I have is the Flee Mortals! Preview. When I add this one, several weird things happen in the bestiary section...

I would expect that the beasts found in this source would smoothly blend into the others created by the tool. That all index files show the monsters under their directories, and that the directories are all the same capitalization (lower case).

The beast's type (abberation, construct, etc.) creates a capitalized directory, i.e. (Abberation, Construct, etc.).

All beasts from Flee Mortals! Preview are stored in this directory.

The index file (Folder Note) of the lower cased directory is overwritten by the upper case folder. Thus, not indexing the files in its own directory.
It seems the overall bestiary.md index is also messed up.

Configuration

OS: Linux Ubuntu 22.04.03 LTS
CLI version:

ttrpg-convert version 2.2.13
Git commit: 2ed1387
Build time: 2023-11-14T17:01:52+0000

CLI type: LinuxBrew compiled image

Sources:

Flee Mortals! Preview (https://github.com/TheGiddyLimit/homebrew/creature/MCDM Productions; Flee, Mortals! preview.json)
PHB and DMG additionally, but not really necessary.

Easy answer. I haven't bought the book yet. ;-)


[GLUTEN-2463][CH] Allow nullable tuple column as input in Nested::flatten

Changelog category (leave one):

Bug Fix (user-visible misbehavior in an official stable release)

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
close issues: https://github.com/oap-project/gluten/issues/2463 and https://github.com/oap-project/gluten/issues/2459

Can one of the admins verify this patch?

It looks like BlockUtil::flattenBlock becomes the same as 'NestedUtil::flatten'. May replace BlockUtil::flattenBlock

$ ./build_gcc/utils/extern-local-engine/tests/unit_tests_local_engine  --gtest_filter="NestedUtils.*" --gtest_catch_exceptions=0         2>&1 | tee run.log
2023-07-26 15:15:54.787 <Warning> SignalHandler: LD_PRELOAD is not set, SignalHandler is disabled
Note: Google Test filter = NestedUtils.*
[==========] Running 1 test from 1 test suite.
[----------] Global test environment set-up.
[----------] 1 test from NestedUtils
[ RUN      ] NestedUtils.flatten
[       OK ] NestedUtils.flatten (0 ms)
[----------] 1 test from NestedUtils (0 ms total)

[----------] Global test environment tear-down
[==========] 1 test from 1 test suite ran. (0 ms total)
[  PASSED  ] 1 test.

LGTM


Crash when opening the screenshot galary on 1.19.4

Crashes immediately after pressing the screenshot button both on the pause menu and title menu

The current latest versions of Screenshot Viewer for 1.19.3 are not compatible with 1.19.4. I know, I have to update the mod, and it will be done soon. Anyway thanks for submitting this issue.


adr-14

Adding an ADR as a part of DOS-207.

Closing after discussion at last DOS check-in meeting


Updated the documentation regarding CLI options.

Description
The documentation seem to be missing few CLI options when I compared it with the code found in /src/options/cliArgs.ts so I decided to update the documentation and add new CLI options that were missing. Also I added a new column called shorthand notation, this was missing. Also I have added proper types such that we do differentiate between array of string and regular strings.
Changes

Some description were outdated and is now corrected to reflect the
code
The types updated to differentiate between Strings and arrays
Another column added for shorthand notations
more CLI options added
Note about the redudant dashes added

This is great. Thanks @ZobairQ !
(please ignore the build status - it's currently broken for contributors)


feat: replacements for mkdirp and rimraf

These packages' functionality have been integrated into Node's fs module, and can usually be removed.

This could be put in native if there's some way to attach documentation--I feel like at the moment, native can't really be used for anything more complex than polyfills where all you have to do is swap out the name of the function.

perhaps we should also update this? lol
https://github.com/es-tooling/module-replacements/blob/218ebd1658d60e2e572b92f8408bed8300b7f49b/package.json#L39
we could also show how to replace it when it's used in this way

perhaps we should also update this? lol
https://github.com/es-tooling/module-replacements/blob/218ebd1658d60e2e572b92f8408bed8300b7f49b/package.json#L39
we could also show how to replace it when it's used in this way

Good catch! I've added a section at the bottom for replacing rimraf in a CLI context, and gone ahead and done so in this repo.

fs.glob is still experimental; I'd prefer to wait a bit before I'm comfortable recommending it.

i think its true that rimraf users will be using globs sometimes, but also true they probably don't need to (and shouldn't be).
i think lets stick with these docs for now, but see if any issues are raised re having a glob alternative. if it is more important than we thought, we can document using glob, fdir or fs.glob alongside


[BUG] No connection with matter device based on ESP32-S3 with W5500 Ethernet IC

Reproduction steps
I just compiled a matter firmware with esp-idf 5.1 (same with stable 5.0.2) and the latest connectedhomeip example for esp32 for a single lightbulb. Using ESP32-S3 as hardware and a W5500 ethernet IC.
I can pair with Google Home, however the device goes immediately offline after successful pairing. I can from there still share to Apple Home (as another fabric) and everything works fine there. Starting off the pairing with Apple Home works as well, and stays online. Sharing to Google Home works, but goes offline right after.
Bug prevalence
Whenever I do this
GitHub hash of the SDK that was being used
connectedhomeip
Platform
esp32
Platform Version(s)
No response
Anything else?
No response

@dhrishi @shubhamdp

@mgoeller This is  the issue with Google Home, I have reported it to Google's Issue Tracker
I will notify you whenever the next update is available

Can we somehow emphasis this issue?

I can see that the issue is still present in Google Home as of today.

@mgoeller Can you please confirm if this works fine now? If yes, please close the issue


search events results with NotImplementedError

I try the code from examples here:
https://github.com/python-caldav/caldav/blob/b5803fff63464a6e02c216fd24ca06dfc4d02f09/examples/basic_usage_examples.py#L201
It throws a NotImplementedError exception
Traceback (most recent call last):
  File "/home/user/form2calender.py", line 284, in <module>
    main()
  File "/home/user/form2calender.py", line 111, in main
    add_event(form_name, res)
  File "/home/user/form2calender.py", line 252, in add_event
    events_fetched = calendar.search(
  File "/home/user/.local/lib/python3.9/site-packages/caldav/objects.py", line 1072, in search
    (xml, comp_class) = self.build_search_xml_query(
  File "/home/user/.local/lib/python3.9/site-packages/caldav/objects.py", line 1320, in build_search_xml_query
    raise NotImplementedError("searching for %s not supported yet" % other)
NotImplementedError: searching for events not supported yet

Iam using caldav 1.3.9 with python 3.9.2

I try the code from examples here:
https://github.com/python-caldav/caldav/blob/b5803fff63464a6e02c216fd24ca06dfc4d02f09/examples/basic_usage_examples.py#L201

Uff.  I need to make sure that the examples code is actually run in the test suite, and at least prior to releases.  But actually, I've been running the examples code now both towards 1.3.9 and master now, without breakages.  Weird.

It throws a NotImplementedError exception

I think you wrote events when it should have been event.
I probably should consider some better error message than a NotImplemented-error for misspellings like that.  Allowing events, todos and journals as synonyms for event, todo and journal could be another option.


Suppression is not working on spring-security libraries after upgrading to 5.1.1

Hello,
In version 4.0.2 of the plugin we have suppress the spring-security libraries 4.2.x version as we are using Spring 4 and not 5.
    <suppress>
        <notes><![CDATA[
   file name: spring-security-core-4.2.6.RELEASE.jar
   ]]></notes>
        <gav regex="true">^org\.springframework\.security:spring-.*:.*$</gav>
        <cve>CVE-2018-1258</cve>
        <cve>CVE-2019-3795</cve>
        <cve>CVE-2018-11039</cve>
        <cve>CVE-2019-11272</cve>
    </suppress>

In version 5.1.1 of the Plugin this is not working anymore and in fact is showing 0.0 for Highest Severity column:

Dependency
Vulnerability IDs
Package
Highest Severity
CVE Count
Confidence
Evidence Count

spring-security-core-4.2.6.RELEASE.jar
cpe:2.3:a:pivotal_software:spring_security:4.2.6.release:::::::cpe:2.3:a:security-framework_project:security-framework:4.2.6.release:::::::
<EMAIL_ADDRESS>0.0
1
Highest
37

The .xml supression file was upgraded to dependency-suppression.1.3.xsd

4.0.2 of the plugin

The current version is 5.2.2. Does this problem occur with this version, too?

5.2.2 is a not a reliable option for us due to:
https://github.com/jeremylong/DependencyCheck/issues/2216
But I will check without Liquibase

I don't think the network and certificate problems of NIST have anything to do with 5.1.x vs 5.2.x of this plugin.

I don't think the network and certificate problems of NIST have anything to do with 5.1.x vs 5.2.x of this plugin.

This issue is not about NIST: https://github.com/jeremylong/DependencyCheck/issues/2216
Just read the description, thanks.

And yes the problem persist even on 5.2.2:

Dependency
Vulnerability IDs
Package
Highest Severity
CVE Count
Confidence
Evidence Count

spring-security-core-4.2.6.RELEASE.jar
cpe:2.3:a:pivotal_software:spring_security:4.2.6.release:::::::cpe:2.3:a:security-framework_project:security-framework:4.2.6.release:::::::
<EMAIL_ADDRESS>0.0
1
Highest
37

netty-all-4.1.5.Final.jar
cpe:2.3:a:netty_project:netty:4.1.5:::::::*
<EMAIL_ADDRESS>0.0
1
Highest
21

In my testing your suppression file works exactly as expected. However, the OSS Index Analyzer adds an unscored vulnerability:
BREACH attack possible in CSRF tokens (OSSINDEX)

> The CSRF tokens generated by Spring are vulnerable to a BREACH attack.
>
> ...
>
> This only occurs when you turn on CSRF protection in Spring and also have HTTP compression enabled somewhere in your web server stack.
>
> -- [github.com](https://github.com/spring-projects/spring-security/issues/4001)
Unscored:
Severity: 0.0

You can generate the additional suppression for the vulnerability from the OSS Index Analyzer using the HTML report and the suppress button.


feat: update SMT to v0.9.2

Summary
Human Summary

Adds ability to call .Sum() on result of smst.Root() to get the sum as a uint64
Can be done via smt.MerkleRoot(rootHashAsBytes).Sum() AFAIK

AI Summary

Summary generated by Reviewpad on 10 Jan 24 14:20 UTC
This pull request updates the SMT library to version 0.9.2. The changes include modifications in the go.mod and go.sum files, with 3 insertions and 3 deletions.

Issue

Fixes N/A
Enables off-chain sum retrieval from MerkleRoot []byte alias

Type of change
Select one or more:

[x] New feature, functionality or library
[ ] Bug fix
[ ] Code health or cleanup
[ ] Documentation
[ ] Other (specify)

Testing

[x] Run all unit tests: make go_develop_and_test
[x] Run E2E tests locally: make test_e2e
[ ] Run E2E tests on DevNet: Add the devnet-test-e2e label to the PR. This is VERY expensive, only do it after all the reviews are complete.

Sanity Checklist

[ ] I have tested my changes using the available tooling
[ ] I have performed a self-review of my own code
[ ] I have commented my code, updated documentation and left TODOs throughout the codebase

@Olshansk @bryanchriswhite  I only updated the go.mod and go.sum files as I think where you are using this new feature is in other branches LMK if this was wrong


PatchCheck.py does not detect you did not configure "BaseTools/Scripts/SetupGit.py (Bugzilla Bug 2767)

This issue was created automatically with bugzilla2github
Bugzilla Bug 2767
Date: 2020-05-26T00:07:45+00:00
From: ajfish
To:   mdkinney
CC:   edk2+bugs+int+994+563148131503455288, nobody
Last updated: 2020-05-26T21:45:53+00:00

Comment 12705
Date: 2020-05-26 00:07:45 +0000
From: ajfish

Industry Specification: ---
Target OS: ---

I was configuring a system to do edk2 contribution and I did not run BaseTools/Scripts/SetupGit.py and my patch got rejected, but PatchCheck.py passed.
This seems like a bad bug in PatchCheck.py

Comment 12713
Date: 2020-05-26 21:45:53 +0000
From: nobody <>
What means the patch got rejected?
What change is required in PatchCheck.py to fix this issue?


zsh: command not found: upgrade-assistant

Problem encountered on https://dotnet.microsoft.com/en-us/platform/upgrade-assistant/tutorial/install-upgrade-assistant
Operating System: windows
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.
After install the tool, I tried to run the command and the command was not found.
I'm running with a Mac M1.

From the pre-reqs, it looks like this is not supported on the Mac :-(

@pablobfs your initial statement says your OS is windows but then you said you're running on a Mac M1. Is this to say you're running something like Parallels and Windows emulation on your mac?


Font substitution/fallback does not work for non-embedded font

iOS：

Real pdf file:
special marks (1).pdf
pdf url：https://sign-dev.clinflash.com/api/view-pdf/51f311bef5f1acef012e52f46b233ab7-6edb1654addefc935e18fbd63344dea8.pdf
The display is incomplete on the iOS platform, but there is no problem on Android. Version 1.0.55 was introduced.

Your problem is just a font issue. Not a problem with iOS. Apparently, the PDF is not viewable on Chrome's PDF viewer on Windows:

Acrobat on Windows can show the PDF file by using font substitution (STSong-Light is substituted by AdobeSongStd-Light).

Currently, pdfrx does not have control on font substition and no workaround for it.

Found that it can be fully displayed on the iOS simulator

iOS simulator is a simulator; not an emulator. There are such kind of behavior differences.
On the simulator, it uses the preinstalled fonts on macOS, they are not found on iOS apparently.

I also encountered the same issue. My approach is to package this font in the assets on the iOS side, save this font to the application data path when the app initializes, and then let PDFium load this font during initialization.
I found that PDFium supports specifying m_pUserFontPaths during initialization. However, in pdfrx_pdfium.dart, m_pUserFontPaths is always set to null, as shown below:
void _init({String? userFontsPath}) {
  if (_initialized) return;
  print('userFontsPath: $userFontsPath');

  using(
    (arena) {
      final config = arena.allocate<pdfium_bindings.FPDF_LIBRARY_CONFIG>(
          sizeOf<pdfium_bindings.FPDF_LIBRARY_CONFIG>());
      config.ref.version = 2;
      config.ref.m_pUserFontPaths = nullptr;
      config.ref.m_pIsolate = nullptr;
      config.ref.m_v8EmbedderSlot = 1;
      pdfium.FPDF_InitLibraryWithConfig(config);
    },
  );
  _initialized = true;
}

I tried to set m_pUserFontPaths to the application directory, where userFontsPath is the directory where the fonts are stored within the app, like this: /var/mobile/Containers/Data/Application/E396F8AA-87FF-4337-9595-B416D10A34CC/Library/Caches. The code to set the path is as follows:
config.ref.m_pUserFontPaths = userFontsPath != null
          ? Pointer.fromAddress(('$userFontsPath').toUtf8(arena).address)
          : nullptr;

But Xcode reports an error:
Can't show file for stack frame : <DBGLLDBStackFrame: 0x3391bc820> - stackNumber:1 - name:fxcrt::ByteString::ByteString. The file path does not exist on the file system: /../../core/fxcrt/bytestring.cpp

Could you please let me know if this approach is feasible? How should I specify this path value?
Looking forward to your reply.

Acccording to the C++ definition, it is double-pointer:
  // Array of paths to scan in place of the defaults when using built-in
  // FXGE font loading code. The array is terminated by a NULL pointer.
  // The Array may be NULL itself to use the default paths. May be ignored
  // entirely depending upon the platform.
  const char** m_pUserFontPaths;

So the code should be at least like:
/// list of font paths
final fontPaths = <String>[];

...

if (fontPaths.isNotEmpty) {
  final fontPathArray = arena.allocate<Pointer<Char>>(
      sizeOf<Pointer<Char>>() * (fontPaths.length + 1));
  for (int i = 0; i < fontPaths.length; i++) {
    fontPathArray[i] = fontPaths[i].toUtf8(arena);
  }
  fontPathArray[fontPaths.length] = nullptr;
  config.ref.m_pUserFontPaths = fontPathArray;
} else {
  config.ref.m_pUserFontPaths = nullptr;
}

OK, I'll add the minimal support for the font paths on the next version though it could not support all of the platforms.

1.0.99 introduces Pdfrx.fontPaths for loading additional fonts.


Provide some mechanism for specifying wait times and timeouts

Ooo yes please!
Maybe in the Arrives.At<>() helper method.


No ready threads available on scheduler::start()

I have a very simple main function and I run into this assert, causing the program to terminate with an exception
generic/src/rtos/internal/os-lists.cpp:117: os::rtos::thread* os::rtos::internal::ready_threads_list::unlink_head(): Assertion `!empty ()' failed

This is backtrace
[6] from 0x0000000000413c80 in os::rtos::internal::ready_threads_list::unlink_head()+56 at /micro-os-plus/generic/src/rtos/internal/os-lists.cpp:117
[7] from 0x000000000041aab2 in os::rtos::port::scheduler::start()+77 at /micro-os-plus/posix/src/rtos/os-core.cpp:139
[8] from 0x0000000000414917 in os::rtos::scheduler::start()+109 at /micro-os-plus/generic/src/rtos/os-core.cpp:217
[9] from 0x0000000000407023 in main(int, char**)+300 at /micro-os-plus/generic/src/rtos/os-main.cpp:180

When I monitor ready_threads_list I see that before scheduler::start() is called, the two nodes of the threads (main and idle) are added during their respective creation but also removed when they are resume()ed (also during creation).
So my guess is that the scheduler is already running before scheduler::start() is called and it cannot be assumed that any ready threads are present at this point.

So the problem to me seems that the scheduler is already running (reschedule() works normally) before it is actually started.
As a confirmation: When I lock the scheduler before start() at this location it works for me.

I don't know, you are probably right, the synthetic-posix port is not as rigorous as the cortexm one, and might need some adjustments.
I'll get to it soon, right now I'm working on the testing issues for the support packages, but when I'll get to the µOS++ core I'll need the synthetic-posix too.


Handle suppressed exception in DistinctErrorLog

Factor in suppressed exceptions when creating entries in DistinctErrorLog.

No used for uniqueness.


Add is_explicit_relative property in AbstractImportObj

Related PR: #18.
Closes #17.
Prepare for asottile/reorder_python_imports#20.

@asottile Please review.

Looks good!
2 tiny minor things, one will be automatically fixed if you rebase so I won't even mention it, the other is this, not a big deal to fix though -- I am in the process of writing an automatic pre-commit hook to auto-fix this :)
At minimum, please rebase and run the tests once -- add trailing commas or not nbd

I'll merge in the morning -- getting late here where I'm visiting

@asottile Ok,  I've rebased and fixed the other one problem.

Thanks for the PR! I'll make a release

This is part of 0.5.3


[Figma File Search]

Extension
https://www.raycast.com/michaelschultz/figma-files-raycast-extension
Raycast Version
1.79.1
macOS Version
13.2 (22D49)
Description
This keeps crashing after 3,4 days of use. It says could not load files when you try to search for any Figma file

Steps To Reproduce
1.Install Figma search
2. Run it for a few days i.e search for files
3. You'll see the error after few days, you won't be able to search and find files
Current Behaviour
No response
Expected Behaviour
No response

@eshamh are you using personal access tokens or log in with figma to authenticate the extension?

I ran into this as well. I was using the PAT method, and was working prior.

@megaroeny nuts. Was hoping this was a one off. I'll take a look when I can. Sorry about that.

@megaroeny nuts. Was hoping this was a one off. I'll take a look when I can. Sorry about that.

No problem! 👍 I'll try logging in later today to see if that method works. Didn't know that was an option.

Following up, sorry so late...
It worked after reinstalling the extension so that I use the new login method! Not sure if there's a way to notify users about this if they were using a PAT before, but at least it works. 😄


design needed

I saw your project and I love the idea. but to start working on this website we need a design.

How yes i know we need a redisign of the website
but now the website is not our priority


[Feature request/Question] Insert Pixiv tags into images metadata IPTC/XMP

Prerequisites

[x] Did you read FAQ section in readme.md?
[x] Did you search for existing issues in Issues?

Description
I have been trying to figure out a way to to insert the tags from either the txt or json files that can be generated into the actual metadata for the images, but haven't really been able to figure out how i would do this in mass/automated, since it wouldn't really be possible to do this manually with thousands of images.
I know that i could probably use the ExifTool to do this, but haven't really been able to figure out how to use this together with the provided output files generated for singular or groups of images and how i would do that for all the images or at least the first one in a group/series of them.
So would it be possible to integrate this feature into this tool (i know that the %tags% feature exists for the image titles) or maybe just some sort of explanation on how to do this, if possible.

no, as it will modify the file size/timestamp and break duplicate file detection.
You can set writeimagejson = True to export the full image information and maybe write your own script to update the exif?

Figured it out.
By using this command with ExifTool it will apply the tags to to the keyword metadata for all p1 images.
exiftool -charset iptc=utf8 -m -P -r -overwrite_original -ext jpg -tagsfromfile %d%f.json "-keywords<tags" "Download path"
This also preserves the timestamp so it shouldn't interfere with duplicate file detection, unless you have to use both size and timestamp for that. If so another way around that could be to compare Image.tostring() outputs since that doesn't include the images metadata and would not change due to this (It should not at least).
Seems like it would be a decently useful feature to add to this program, since you could easily also add all the image links, captions, and such into the metadata if you want to.
I don't have the knowledge to code this so i wouldn't be able to do it.

This would be quite useful even if it was just sidecar .xmp files
This library can do the work I believe https://launchpad.net/py3exiv2
I could then use the data in professional photo software like DigiKam

merged #980


package name seems to be in lower case, should modify instruction.

it should be
roslaunch fiesta cow_and_lady.launch
rather than
roslaunch Fiesta cow_and_lady.launch
as the package name seems to be in lower case

@Doodle1106  Done.


Incomplete documentation for spacing

The documentation about spacing doesn't mention that the size mixins can be used as well.

Size mixins aren't the same as margin and padding utilities—there's no reason to show them on those pages.

@mdo my bad, I was thinking of the breakpoints, not the size mixins, It isn't mentioned that you can use mt-md-2 mt-sm-0 etc. Have a look at #25716


Feature/ci cd pipeline

This PR contains an initial draft of the work to automate our deployment and operation of the dev services, using Github Actions and Docker Compose.  Similar work for our production services will follow in a separate PR.
NOTES:

Added .github/ and deploy/ directories to project

.github/ contains workflow to be used by Github Actions for CI
deploy/ contains scripts, configuration, and Dockerfiles for deploying each service through CI using docker compose

Created pyproject.toml and reorganized directory to follow PEP 621 specification

In order to make the project pip installable I needed to reorganize a bit such that it aligned w/PEP 621
Specifically, I moved all the top-level code directories under an A2rchi/ directory
I also made changes to imports in an attempt to eliminate all relative imports

Made small changes to DataManager while I was looking at it:

Swapped os.mkdir —> os.makedirs with exist_ok=True

Prevents error(s) if directory already exists

Using os.path.join to reduce chance of configuration error due to missing ‘/’

Made other minor tweaks for syntax / style

Will do a more robust pass through code with refactor branches to propose more style changes

NOTE: tomorrow I will resolve the remaining merge conflicts. I will also do a final deployment to test that things are working properly -- at the moment I cannot debug the final two services (mailbox and scraper) b/c someone is heavily using the subMIT machine I am testing on.

Mostly small nit-picky comments. Nothing much needs to be changed, just want to make sure we are on the same page about a few things.
Overall, also, I wanted to double check that the conversation data is saved reliably when we deploy. I.e. we never overwrite conversation data and it won't be deleted if we remake the repo, etc.

To this point, the conversations are written relative to the configured DATA_PATH, which right now means they're written to /root/data/conversations.json. Our deployment creates a Docker volume the first time we deploy (and re-uses it for subsequent deployments), so conversations will live across deployments and will not be deleted unless we explicitly delete the Docker volume.


volume plot attributes ui issues

the samples per ray input box loses focus after each key stroke(trying to type 1000, requires several clicks) 2)create a volume plotchange the color tablerender the volume plotdelete this plot and create a new one the ui elements are out of sync the color table button will still show the last used color table, while the rest of the widgets reflect the default color table. (see attached screenshot) 3)It would be nice if the histogram used to select opacity updated depending on the limits selected in the color table above.It's very hard to set opacities when the distribution is pegged & you want to use the limits to avoid common values.

-----------------------REDMINE MIGRATION-----------------------
This ticket was migrated from Redmine. As such, not all
information was able to be captured in the transition. Below is
a complete record of the original redmine ticket.
Ticket number: 2622
Status: Pending
Project: VisIt
Tracker: Bug
Priority: Normal
Subject: volume plot attributes ui issues
Assigned to: -
Category: -
Target version: -
Author: Cyrus Harrison
Start: 05/31/2016
Due date:
% Done: 0%
Estimated time:
Created: 05/31/2016 01:39 pm
Updated: 05/31/2016 06:51 pm
Likelihood: 3 - Occasional
Severity: 3 - Major Irritation
Found in version: 2.10.0
Impact:
Expected Use:
OS: All
Support Group: Any
Description:

the samples per ray input box loses focus after each key stroke(trying to type 1000, requires several clicks) 2)create a volume plotchange the color tablerender the volume plotdelete this plot and create a new one the ui elements are out of sync the color table button will still show the last used color table, while the rest of the widgets reflect the default color table. (see attached screenshot) 3)It would be nice if the histogram used to select opacity updated depending on the limits selected in the color table above.It's very hard to set opacities when the distribution is pegged & you want to use the limits to avoid common values.

Comments:

/Users/griffin28/Downloads/VisIt_Redmine_2_Github-master/archive/git_labeled_attachments/2622_Screen_Shot_2016-05-31_at_10.37.37_AM.png

The samples per ray part has been fixed.
Color table name problem similar to #2395


Create typescript definitions

No typescript definitions exist which makes it less desirable from a hotswap perspective with pouchdb proper when using the compilerOptions.noImplicitAny setting.

I do not use typescript at all, but if you do a pull request that adds the type definitions, I'll accept it and publish them

Hi @stockulus, I've created the type definition for the library... here: #87

Given the pull request was merged and accepted, I think this issue should be closed.


Reduce repetition ci

This PR removes duplication within the ci
Fixes https://github.com/compiler-research/xeus-cpp/issues/42

@vgvassilev This PR is ready for review. I don't know where it specifically addresses what you wanted in the issue, but I have tried to remove repetition in the ci.

Nice! Any chance to incorporate the wasm build? It seems quite different...

Nice! Any chance to incorporate the wasm build? It seems quite different...

It may be possible, but I'd advise against it. I think the ci is clearer with the wasm and non wasm builds separate.

Nice! Any chance to incorporate the wasm build? It seems quite different...

It may be possible, but I'd advise against it. I think the ci is clearer with the wasm and non wasm builds separate.

Ok, makes sense. Let's get this one in.


join-with causes the screen to flicker

Hi,
I love AeroSpace, thanks for developing and maintaining this <3
I try to join 2 windows in a vertical layout and I think it basically works but unfortunately the result is unusable as the screen is flickering completely.
To reproduce:

I open 2 windows: 1 finder, 1 iterm
In iterm I type: aerospace join-with left
the 2 windows are displayed now vertically but the screen is flickering I have to quit iterm to stop that or do aerospace move right

Any hint on how to fix that?
aerospace version:
➜  ~ aerospace -v aerospace CLI client version: 0.12.0-Beta 9cf82feb0c85bea77fedcdc6b8cdba75b5bbf1f8 AeroSpace.app server version: 0.12.0-Beta 9cf82feb0c85bea77fedcdc6b8cdba75b5bbf1f8

Sorry, I can't reproduce. Works on my machine
Please try to update to the latest 0.14.1 version. It fixed some of the flickering issues in multi monitor setup
Is the issue reproducible if you disable Displays have separate spaces? https://nikitabobko.github.io/AeroSpace/guide#a-note-on-displays-have-separate-spaces

brew upgrade indeed fix the issue, sorry to have bothered you without trying that first.
I’ll read the doc about setting you mention now.


Navbar Improvement

heyy , Can I make some improvements in navbar design ?

Can I work on Navbar design?

assign it to me

can i improve navbar ?

hey @shivani248 @dilroseR @kriyanshishah  this issue is closed for now ..
if u want u can work on designing issue34


Web - Room - Not here page is not cleared automatically after member is invited

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.77-6
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: N/A
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers): N/A
Issue reported by: Applause Internal Team
Action Performed:

Create a workspace and a room
Copy the URL of the created room
Sign in as a non member
Navigate to the room URL
Not here page appears
Invite member as the admin while member is on the not here page

Expected Result:
Not here page clears
Actual Result:
Not here page clears only after manual refreshing of the page
Workaround:
Unknown
Platforms:

[ ] Android: Standalone
[ ] Android: HybridApp
[ ] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/ccc89caa-5cb8-4e07-8622-e8c47688776b
View all open jobs on GitHub

This should be handled internally because that can be fixed by including the report onyxData in the response from GetMissingOnyxMessages.

This should be handled internally because it can be fixed by including the report onyxData in the response from GetMissingOnyxMessages.


Timothy's melodrama thing. Let's get it done today.

TL;DR
Timothy wants us to share our(Jurj's) code with Melodrama so we can earn some bonus points or something.
---------- Forwarded message ---------
From: Ladue Melodrama<EMAIL_ADDRESS>Date: Thu, Jan 27, 2022 at 11:13 AM
Subject: Melodrama - Email to student body
To: Adam Stirrat<EMAIL_ADDRESS>

Could you forward this email to the student body?

Thanks

Hi Everyone!

Melodrama, Ladue High School's Literary Magazine, is still accepting submissions. We annually publish art, writing, and other forms of media created by Ladue students. If you have any work you are proud of, submit it to Melodrama using this google form . We look forward to showcasing the talents of the Ladue student body!

Last year's Melodrama won first in the nation in the National Scholastic Press Association Fall Best of Show competition. See last year's Melodrama: Last Year's Melodrama

Thanks,
Melodrama Editors-in-Chief

From Timothy:

this is melodrama I'm sure if we show your code and robot theyll accept it plus I think my sister is friends with the people who do it just recommend it to garrison and get nontech to do it
---
Timothy Zhang

No.

You could share one of the commented files.

Yay, my code

This is either done or never going to be done.


WIP: Fix Update-Help failing silently with implicit non-US culture.

PR Summary

HelpCultureNotSupported is caught and ignored with implicit culture (-UICulture not given explicitly, defaults to system) to allow trying again with culture's parents. It does not however check if there are any more fallbacks, so all errors of this type are completely silenced. I store first error of this type, then show it if none of the fallbacks succeed,
Fixes #17696
WIP:

Tests - difficult since only reproduces when (Get-UICulture) -ne 'en-US' and -Path is not given, missing mocks
Update docs which mention this behavior

Opens:

Is this a breaking change? I would say no, even if somebody's script will now throw an error where previously it passed, they should probably know about it.

PR Context

Update-Help with implcit culture fails silently on non-US systems (en-GB in my case), with no feedback on what is wrong even when using -Verbose. With this fix, the user is shown the same error as with explicit -UICulture (Get-UICulture), which clearly describes what they should do:
Update-Help: Failed to update Help for the module(s) '...' with UI culture(s) {en-GB} : No UI culture was found that matches the following pattern: en-GB. Verify the pattern and then try the command again..
English-US help content is available and can be installed using: Update-Help -UICulture en-US.

PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[ ] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[ ] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[ ] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[ ] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[ ] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[ ] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and opened an issue in the relevant tool repository. This may include:

[ ] Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode
(which runs in a different PS Host).

[ ] Issue filed:

[ ] Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.

[ ] Issue filed:

[ ] Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).

[ ] Issue filed:

[ ] Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

[ ] Issue filed:

@dkaszews Thanks for submitting the PR!
@adityapatwardhan, can you please review when you have time? Thanks!

@dkaszews I like the change. Thanks for the PR. For testing, I would add some test hooks to make sure we fall in to the expected code paths.
Look at the files here to see how test hooks can be added:
https://github.com/PowerShell/PowerShell/search?q=testhook&type=code
Also, the test hook is used in product code like this:
https://github.com/PowerShell/PowerShell/blob/17791759356ffa6e2ae657d2ec19a147ce062957/src/Microsoft.PowerShell.Commands.Management/commands/management/Computer.cs#L450-L456

@adityapatwardhan Thanks for the tip! Please take a look if something like this is good.

If found additional issues all related to matching culture:

We use globbing to try and match current culture with the one in help, but because valid CultureInfo values never contain wildcards, this does nothing. E.g.: en-GB has fallback en, but it still won't glob match with en-US. A simple fix would be to suffix (or surround) culture with '*':// UpdateableHelpSystem:594:
IEnumerable<WildcardPattern> patternList = SessionStateUtilities.CreateWildcardsFromStrings(
// - globPatterns: new[] { currentCulture },
globPatterns: new[] { currentCulture + '*' },
options: WildcardOptions.IgnoreCase | WildcardOptions.CultureInvariant);

Second issue is that in UpdateableHelpSystemBase:EndProcessing, when we collapse all HelpCultureNotSupported exceptions for different modules into a single error, the supported culture is actually ignored, always displaying "en-US is supported" without checking that it actually is. That may be true for PowerShell-core modules, but if some module provides help in let's say en-GB or even fr-FR, it will be misleading, as I believe may be the case in #17534 . Fix for that is slightly more involved, as it would require storing the supported cultures in the exception and probably reworking the exception hierarchy, as right now they are only in string.

Which of those should I fix here and which should be a separate issue? I would say the first one can be here, as it's just a single line, but the other I would prefer merging separately.

CodeRefactor reports new issues in UpdateHelpCommand.cs on lines 471 and 476, but I only modified 217 and see similar errors at other places in the file. How can I get it resolved as false positive?

Thanks for the updates. If the CodeFactor issues are not in the code block you touched, you can ignore them.
I would prefer both the changes 1 and 2 in a separate PR as it changes the behavior and is an improvement in fallback lookup.
This PR is focused on fixing the error message on fallback.

@adityapatwardhan Sure, I'll create new issues and separate PRs for those two later

@adityapatwardhan Anything else I should do before this can be merged?

@dkaszews thank you for your contribution!


Wise: Automatically set OTT = true when dealing with European countries

Right now we need to manually set each host account to be compatible with Wise OTT flow and this means that every time a new host joins the platform from Europe or the UK, they'll face some kind of error before they can use Wise for the first time.
We should automate the onboarding process by setting settings.transferwise.ott = true when a collective connects to Wise from any European country.

We may want to use the isMemberOfTheEuropeanUnion helper from lib taxes


Debug missing launch.json popup should say "VSCode extension development"

I think right now it says extension development, which IMHO is too generic

Nice!


dynamicpb: Zero() creates RO message

Sets known map to nil so SetField fails and IsValid returns false.

Not sure what the behaviour should be for Get: https://github.com/protocolbuffers/protobuf-go/blob/v1.20.1/types/dynamicpb/dynamic.go#L137
Mutable checks for nil here: https://github.com/protocolbuffers/protobuf-go/blob/v1.20.1/types/dynamicpb/dynamic.go#L172

Thanks for the PR! I agree that we should make this change; the Zero method should return a read-only message value.
We develop in Gerrit:
https://go.googlesource.com/protobuf/+/refs/heads/master/CONTRIBUTING.md
Would you like to make a Gerrit CL? Or should I make the change there myself?

Ah sorry! Thanks I will try push the change there.


Update vendor packaging

Update to newer type-hinted packaging vendor package.
This is a "no patch" update where the original source code is completely unmodified outside of modifying necessary imports to point to their rez.vendor equivalents.

Looks like a few builds failed temporarily due to some temporary issue with Microsoft packages...? https://github.com/orgs/community/discussions/120966