BLD: Small tweaks to GH Actions workflows

Nothing big, just some small fixes that should make development easier.

Remove Node.js matrix for main CI, since the version number is hard-coded currently; this should remove the need to click through multiple pages when accessing the main CI (edit: actually maybe you still need to click through things twice, hm. at least it should simplify the code.)

In the future we could add back this matrix and use it to try multiple Node.js versions -- however I don't think it's an urgent priority, since Node.js is only used on the development side (so being exhaustive about versions working probably isn't too important, at least compared to Python versions in the standalone CI)
Another concern here is that trying multiple Node.js versions should probably be a separate process from installing/testing the Python code; I think the most efficient way to set this up this would be something like adding a different workflow for JS testing, then adding another workflow for uploading to McHelper that is dependent on both the "JS testing" and "Python testing" workflows passing [but IDK how to do that right now]

Make some documentation consistent btwn YAML files (e.g. Empress -> EMPress)

Remove the "Empress" from the workflow names, renaming them to just "Main CI" and "Standalone CI" -- I think this makes the README badges/etc. easier to interpret

The following artifacts were built for this PR: empire-biplot.qzv, empire.qzv, empress-tree.qzv, just-fm.qzv, plain.qzv

Thanks @fedarko !


Hardware state handling and vector drawables

Quite a lot for one PR - sorry...
The state handling in the chat fragment is not the most elegant right now, but works (I hope). The AlertDialogs could be replaced by something prettier and not so intrusive. Also I'm not 100% convinced about the data connection icon.
Vector graphics should be self explanatory - we can basically use them anywhere but in the notification (due to support lib).
FAB for centering location:

Internet connection monitoring

GPS Hardware:

New twitter button:

:heart: :heart: :heart:
looks really good! will have a deeper look tomorrow.

closes #118


How to Configure NPMPLUS to Reverse Proxy Itself?

Hi,
I’m trying to set up NPMPLUS to reverse proxy itself so that I can access the web interface securely via a custom domain name (e.g., https://npm-plus.mydomain.com). However, I’m having trouble getting this configuration to work correctly.
I’ve tried setting up a proxy host with the following settings:
Scheme: http
Forward Hostname/IP: <IP_ADDRESS>
Forward Port: 81 (since the web interface runs on port 81)
SSL: Enabled using my wildcard certificate

Despite these settings, I keep encountering SSL warnings or issues with the web interface not loading properly. Could you please provide guidance or best practices on how to configure NPMPLUS to proxy itself effectively?
Thank you for your help!
Let me know if this captures what you want to ask or if you'd like to modify anything.

you need to use https as Scheme


Fails on Raspberry Pi OS Debian 10 Buster?

Fails on Raspberry Pi OS Debian 10 Buster?
Installed via pip3 install vcgencmd...
# pip3 install vcgencmd

Looking in indexes: https://pypi.org/simple, https://www.piwheels.org/simple

Requirement already satisfied: vcgencmd in /usr/local/lib/python3.7/dist-packages (0.1.1)

# python3

Python 3.7.3 (default, Jan 22 2021, 20:04:44)

[GCC 8.3.0] on linux

Type "help", "copyright", "credits" or "license" for more information.

>>> import vcgencmd

>>> vcgencmd.measure_temp()

Traceback (most recent call last):

  File "<stdin>", line 1, in <module>

AttributeError: module 'vcgencmd' has no attribute 'measure_temp'

>>>

Version 0.1.0 seems to work, but 0.1.1 does not?  So the direct download and compile of 0.1.0 worked.  But pull of 0.1.1 from the pip3 repository fails.  Guess 0.1.1 is not released correctly?

The pypi registered "vcgencmd" package is not mine :)
The one you are installing is this one: https://github.com/sushantnadkar/vcgencmd.git
Before you unstall a pypi package, checkout it's pypi info:
https://pypi.org/project/vcgencmd/

Yup, I did not realize there was a conflict.  Thanks for the response, always nice when people support their code!


Migrate IGC connector to its own repository

Objectives:

Demonstrate building an external proxy connector (outside core Egeria codebase)
Demonstrate how to dynamically include / bind such a connector into a running environment (eg. Helm-orchestrated containers)

It looks likely that we'll need to change the packaging mechanism for Spring Boot in the server chassis to use ZIP instead of the default JAR -- to be able to load external dependencies (like connectors) at runtime (using loader.path) rather than forcing everything to be pre-bundled into a single jar file.
Unless we want to have some interim deployment step that re-bundles the jar file by injecting external connector jars into its BOOT-INF/lib?
See:

https://stackoverflow.com/questions/30745132/external-library-folder-for-spring-boot
http://weyprecht.de/2018/05/02/how-to-really-package-and-deploy-a-spring-boot-application/
https://docs.spring.io/spring-boot/docs/current/maven-plugin/usage.html

This will presumably also affect the ability to do java -jar XXX.jar which is how the current docker containers launch the spring boot app, so will need to change there? However reading the docs it does seem as if the change is needed almost for this very reason - to get additional flexibility in the loading process...

You can still run the executable jar file using java -jar XXX.jar, this approach just allows you to additionally provide a -Dloader.path=... to the same command.  So you can run:
java -Dloader.path=... -jar XXX.jar

Without using this approach, the -Dloader.path=... is ignored, as is any -cp, CLASSPATH environment variable, etc.
So this way we could even still use the same 'vanilla' Egeria Docker image for even these custom connector proxies -- just by init'ing the container first to mount in the additional connector jar(s) and changing the entrypoint / CMD args to include the mounted path with -Dloader.path=...


⚠ ClamAV has degraded performance

In cfcbb0f, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 240 ms

Resolved: ClamAV performance has improved in dab2fd4 after 5 minutes.


Upgrade cookie consent banner to use a custom element tag

This upgrades the cookie consent banner to use a custom element tag:
<cookie-consent-banner />

The one thing I need direction on is the package.json config such as what files to expose and how.


Switch to go.mod and update the aws-sdk

Hey @cmattoon -
The current version of the code uses an outdated aws-sdk which doesn't support the AssumeRoleWithWebIdentity IAM call. The architecture also depends on the node running the container having access to the secrets.
I'm using your container on EKS with the OIDC identity stuff enabled which allows pods to assume roles in AWS, thus I can limit the permissions i need to set on a node.
I updated that with some other packages as well as the dependency system you were using before hand 😊
I think i'm also using a newer version of golang but that's not as important.
I'll write up the PR for this shortly.

fyi this is a good article explaining the new permission model https://aws.amazon.com/blogs/opensource/introducing-fine-grained-iam-roles-service-accounts/

Hey @cmattoon monthly bump - Any thoughts on the above? I think the last time I checked the CI the docker config needed an update for the secret


Add Camel K

Pre-submission checklist:
Please check each of these after submitting your pull request:

[x] Are you only including a repo_url if your project is 100% open source? If so, you need to pick the single best GitHub repository for your project, not a GitHub organization.
[x] Is your project closed source or, if it is open source, does your project have at least 300 GitHub stars?
[x] Have you picked the single best (existing) category for your project?
[x] Does it follow the other guidelines from the new entries section?
[x] Have you included a URL for your SVG or added it to hosted_logos and referenced it there?
[x] Does your logo clearly state the name of the project/product and follow the other logo guidelines?
[x] Does your project/product name match the text on the logo?
[x] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
[x] ~5 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

Deploy preview for landscape ready!
Built with commit d768d5a3bd6bdf487d3b193a0288bcaa37fc99f1
https://deploy-preview-1437--landscape.netlify.com

LGTM

Logos need to include the project name, though we can ignore the lack of a K here. Please review after this builds and give it another LGTM if it looks OK.

Yeah, right.. we don't have a squared one at the moment. Thanks for fixing.

LGTM


Add a link in the Release note part of the PR template

Before asking a question, make sure you have

[X] Searched existing Stack Overflow questions
[X] Googled your question
[X] Searched open and closed GitHub issues
[X] Read the documentation

What is your question?
Add a link in the Release note part of the PR template

/type enhancement

/remove question

/unlabel question


broken link

Image is broken for Grinder Final File CloudShell :
missing link: https://github.com/RenatoGregio/AzureResourceInventory/raw/main/images/GrinderFinalFileDesktopCloudShell.png

Fixed.
Thx Hara!


Implement QT interface

Implements interface used by quantum-tensors
Working towards rewriting performance issues in QT-QG2

I cannot merge that for fundamental reasons.
The version here need to base on a version quantum-tensor NOT the current master.

The latest QT wasn't converted to a npm package I can reference. How do you want to proceed?

Work on this branch. I will merge only after we release a new stable version of QT, after changes.

See #226 instead.


⚠ Mobitopup has degraded performance

In 664144d, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 9037 ms

Resolved: Mobitopup performance has improved in f434cd2.


1122 admin add intake

Implements #1122

This is working great, only thing I noticed is that you can create an intake where start date === end date:

I added a check in the database mutation that will throw if the start date is greater than or equal to the end date, as well as a pgtap test for it. Let me know if you can reproduce this on the front end again though! Thanks.

The validation doesn't seem to trigger when there is an intake open, this is what I did:

Create a new intake that starts in 1 to 2 minutes
Refresh the page once the new intake "starts"
Create a future intake
No validation

Result:

I can't reproduce this

I can't reproduce this

I think this is probably fine the way it is, it is a very very edge case.


glfwCreateWindow should honor wShowWindow for the first window

Because it does not, the universal way to launch programs hidden or minimized does not work for GLFW-based games: ShellExecute(IntPtr.Zero, "open", "game.exe", null, null, SW_HIDE) (also does not launch minimized regardless of shortcut properties).
Steps

Build boing example
Create a desktop shortcut for it (right click on .exe -> Send To -> Desktop (create shortcut))
Open the shortcut properties and set "Run" to "Minimized"

Expected
The game starts minimized
Actual
The game opens regular window and steals focus

Agreed, at the very least it should be an option for GLFW to honor this. I've been meaning to do this but haven't had the time. Just removing the workaround in createHelperWindow didn't work reliably for me.

There is now the GLFW_WIN32_SHOWDEFAULT window hint for enabling this behavior per-window.


BUG - Expansion depth in odata v2 is only limited to two

Related Feature
Description
When I'm using @sap-ux/fe-mockserver-core as a mock server for opa5 test. I use odata v2 request and want to expand a property for 3-layer deep. Only 2 layers are expanded, and the third level is ignored.
Steps to Reproduce
Steps to reproduce the behavior:

Send an odata v2 request with $expand parameter more than 2 layers
See error: only 2 layers can be expanded.

Expected results
All the requested expansion structure can be returned as result
Actual results
Only 2 layers can be expanded.
Screenshots
Version/Components/Environment
Add any other context about the problem here
OS:

[x] Mac OS
[x] Windows
[x] Other

Root Cause Analysis
Problem
https://github.com/SAP/open-ux-odata/blame/9f3cc504ae9caf4255316692de820ffbe4d3d49d/packages/fe-mockserver-core/src/request/odataRequest.ts#L241
That only the second level of expansion goes into the recursive method, the deeper level isignored
Fix
{describe the fix}
Why was it missed
{Some explanation why this issue might have been missed during normal development/testing cycle}
How can we avoid this
{if we don’t want to see this type of issues anymore what we should do to prevent}

HI @LombardsAI would it be possible for you to provide a sample metadata / query where this could be reproduced easily ?

Hi @nlunets, if you can follow this link: https://github.tools.sap/epd-specification/specmgmt/blob/main/app/ui-property/webapp/localService/metadataV2.xml to get the metadata file and use https://{{your_server}}/odata/v2/PropertyConfigureService/Property?$expand=attributeSetting/attributeFields/texts&$filter=displayId eq '{{mock-property-id}}' to reproduce the error（third level expand "texts" are not returned in the response

@LombardsAI your mockdata (btw private URL), doesn't contain data for attributeFields, so as such no expansion is possible for the last level.
Even attributeSetting is completely auto generated based on the non-nullability of the field.
If you fill in mockdata you will see your data

Hello @nlunets,
I'm afraid the problem is that in this line of code https://github.com/SAP/open-ux-odata/blame/9f3cc504ae9caf4255316692de820ffbe4d3d49d/packages/fe-mockserver-core/src/request/odataRequest.ts#L241 the expand is split by "/", and in this line https://github.com/SAP/open-ux-odata/blame/9f3cc504ae9caf4255316692de820ffbe4d3d49d/packages/fe-mockserver-core/src/request/odataRequest.ts#L244 only the second element in the split array goes into the recursive method.
You can ping Fang,David in teams and I can demo this to you by screen sharing.


Hotspot from image in cube map

Hi,
Thanks for the great library.
I'm using for a project the cube map option, I would like to know if there is a simple way to calculate hotspot information, from face image coordinates.
Thanks anyway.

I would recommend setting the hotSpotDebug option to true, which will print coordinates to the console on mouse clicks. This makes it easy to find coordinates to place hot spots.
If you want to determine coordinates directly from cube map images, you'd have to work out the math yourself.


Device Properties

Added Manager::getDeviceProperties() which exposes some device parameters that are relevant for Kompute
devprops = mgr.get_device_properties()
print(devprops)

Device Name:                   SwiftShader Device (LLVM 10.0.0)
Maximum Workgroup Count:       65535, 65535, 65535
Maximum Workgroup Invocations: 128
Maximum Workgroup Size:        128, 128, 64
Timestamps Supported:          False

(Single Include header changed more than I want, not sure why.)

Modified to return raw vk:: struct and Python dict
mgr.get_device_properties()

{'device_name': 'SwiftShader Device (LLVM 10.0.0)',
 'max_work_group_count': (65535, 65535, 65535),
 'max_work_group_invocations': 128,
 'max_work_group_size': (128, 128, 64),
 'timestamps_supported': False}


Possible race condition between bosh2 stop --hard and resurrection

Earlier today I ran bosh2 stop --hard on a VM. BOSH stopped it properly. I went to do things to the deployment, and got a deployment lock. When I investigated that, the director was CCKing the stopped VM, and recreated it. Afterwards bosh stop --hard did nothing (presumably because the VM is listed as state detached, and bosh start does nothing (presumably because the VM has a vm_cid associated with it). I now have a running VM that I cannot get rid of, or update.
Updating the database to change the state got things in a workable state again.
  Name       east-us-proto-bosh
  URL        https://<IP_ADDRESS>:25555
  Version    260.0.0 (00000000)
  User       admin
  UUID       0330dc9a-5850-4a7f-87ea-91135d2c7fa8
  CPI        azure_cpi
  dns        disabled
  compiled_package_cache disabled
  snapshots  disabled

Deployment

Azure CPI v22
Did this process again, and reproduced the same issue. In total, I tried twice with the new bosh cli, and once with the ruby bosh cli. The ruby bosh cli did not see the VM recreated after it was stopped, however in all 3 cases, the hm log showed:
W, [2017-03-23T18:41:06.734481 #7348]  WARN : (Resurrector) notifying director to recreate unresponsive VM: db-eastus-dev-bosh bosh/028063d4-2fa2-4dbe-b6d2-869826bfb774

So, not sure if this is a BOSH issue, or a bosh-cli issue.

I'm a little unclear on this - you mention you got a deployment lock and then investigated it. How were you retaining the deployment lock, and what were you investigating?
Health monitor is a bit naive in how it notifies director to recreate vms. Did any tasks show up around your other tasks which may have been [trying] to interfere?

"got a deployment lock" meant "got an error indicating the deployment was locked". The locking task was the resurrector trying to bring back the machine that I ran stop --hard against. It did resurrect, but bosh still thought the job was stopped and couldn't do anything, until I manually updated the database to fix the detached state.

I wasn't able to reproduce this issue.
Closing this due to inactivity. Feel free to reopen if you are still running into this issue.


selenium.common.exceptions.JavascriptException: Message: ReferenceError: Store is not defined

hi there
I'm facing this error
selenium.common.exceptions.JavascriptException: Message: ReferenceError: Store is not defined
thank you

Tested this and the issue exists.
The root cause is that the underlying WebWhatsAPI that is used has changed. This calls fort he code to be refactored which will take a but of time.
You can check out the latest version of the Whatsapp API here for more information - https://github.com/mukulhase/WebWhatsapp-Wrapper


Add "Get Involved" page to main OH site

List ways in how people can interact with Open Humans. Starting from blogging/tweeting/slacking (off) over sharing data to running their own projects

I'd like to better understand how this proposed page fits into the whole site? What's going to link to it? How does someone find it? How important should it be? Is it redundant with anything that's already present? (Should that be cut?)
I'm concerned because I think there's been a process of adding "page for X" without thinking about the overall layout and UX of finding stuff across all our pages, and we end up with a lot of content people aren't aware of -- including our own team.

As we've discussed over the last week: Right now it's really hard to see (or even explain) how users can interact with OH. The "get involved" page should be a hub for all calls to action, sorted by level of involvement.
Ultimately it would be great if this could be front & center and replace a good parts of the different sub-sites which are right now hard to find (e.g. I can't find our public data "lists" without poking on the site), but this will probably be a larger thing for the UI/UX overhaul that we discussed as a potential next project after the site has been cleaned from historical artifacts and runs on modern software.
Until then I see that this could be a good extension for the about us page.

Closing this as obsolete.
We do need better communication as part of UX/UI improvements. But guides should be the least-favorite solution. We'll leave which/what guides are needed until later, preferably in response to more deliberate usability testing & observations. :)


segfault running rag.py

After watching the video I wanted to give it a go.
But was greeted with a segfault :(
 python examples/rag.py

 > Creating library 'Agreements'...
[1]    20130 segmentation fault  python examples/rag.py

I happens somewhere in the library.add_files(os.path.join(sample_files_path,"Agreements")) line.
tested with Python 3.9.18

Hi, yes see the discussion here for the fix/workaround: https://github.com/llmware-ai/llmware/discussions/48

Sorry about that.   We don't see that in our testing.  I did a quick google and based on the answer here:
https://stackoverflow.com/questions/52026652/openblas-blas-thread-init-pthread-create-resource-temporarily-unavailable
Can you try adding this to your test code to see if that works around the issue (if so, we'll look at adding it to our initialization code):
import os
os.environ['OPENBLAS_NUM_THREADS'] = '1'

Also can you share some specifics on the system you are testing on and the output from:
uname -a

That looks like your system is maxed out in terms of running processes and can't even create a new thread to run docker compose kill.  Are you able to get a list of processes with ps or top and see if there are infact many processes running and kill some manually (or restart your WSL instance?)

Yes sorry I can't be of more help yet.  We've tested on native Linux (x86_64 and arm64) but have not had bandwidth yet to test on WSL.   So there may be something unique there going on that we haven't encountered yet.
We are currently working on native Windows support and can add WSL testing/validation to the list of items.  Is there anything special about your self-compiled kernel or do you think testing on vanilla WSL2 should be sufficient?


renamed the Nacl_cube_Octagon.ipynb to nacl_cube_octagon

To follow the naming convention of Python file, I just renamed the Nacl_cube_Octagon.ipynb to nacl_cube_octagon
Rename Nacl_cube_Octagon.ipynb to nacl_cube_octagon.ipynb

I am not sure, but I guess in this case NaCl should be fine because they represent Na and Cl molecules. Please review that, otherwise @Mosaddeq107 is correct but I guess if I am not wrong we can keep it the old way too,


Parser sees var name starting with _ as a wildcard binding

Description:
string '_&"*%_var = "sample value";

Steps to reproduce:
Affected Versions:
SL beta 2
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

Parsing as string '_ MISSING[;] MISSING[function]&"*%_var = "sample  MISSING[=]value MISSING[+]"; MISSING[;]
which is the excepted behavior as characters have not been excluded with \.
With https://github.com/ballerina-platform/ballerina-lang/issues/30470 we will make _ reserved keyword. Yet one can use _ as a variable-reference with a quote. Let's keep this issue to address that problem.


Added preview to authoring [#165887899]

Depends on this LARA PR to be merged (https://github.com/concord-consortium/lara/pull/459) and npm updated with new version.


Complete installation guide

I tried to install the source on the server and Windows, but I still had problems!
Is it possible to learn this source completely?

Hey @Meytiz, just made an update so the second issue (NameError: name 'Live' is not defined. Did you mean: 'like'?) should be resolved. As for the first one, can you provide me with a screenshot in order to betten understand the error messages you're receiving ?

C:\Users\M!IND\Downloads\IAM>python3 cookies.py
Python was not found; run without arguments to install from the Microsoft Store, or disable this shortcut from Settings > Manage App Execution Aliases.

Ok so you need to edit the environment's variables in order to resolve the python not found error. For the second one try entering this command:
pip install -r ./files/requirements.txt

C:\Users\M!IND\Downloads\IAM>py cookies.py
Using cookies from C:\Users\M!IND/AppData/Roaming/Mozilla/Firefox/Profiles\8sc9n5ae.default-release\cookies.sqlite.
Cookie import failed: JSON Query to graphql/query: HTTP error code 401.

how fix that?

where is the problem from?

Hey @Meytiz check this tutorial to resolve the python not found issue. In order to resolve the second issue (Cookie import failed: ...) log in to your Instagram account from firefox. Then enter the command python3 cookies.py in your terminal and finally enter: python3 IAM.py again in your terminal.

I did this. But the problem still exists.

Ok give me a couple of seconds to fix it and I'll let you know.

Try again now maybe. I changed the login method. With the same procedure. Login to your instagram acc via firefox, python3 cookies.py and python3 IAM.py

ok, git pull yeah?

Exactly !

And let me know

:(

Shit. Ok I'll try to fix it and I'll get back to you in a couple of hours.

tnx bro

fixed man?

Hey @Meytiz sorry it's not ready yet. It's pretty big script. I will try to fix it and upload it later today or tomorrow.

:(

Hey @Meytiz I'm really sorry but I truly can't work on your request right now, I'll try to fix it this or the next week.

Hey @Meytiz I'm really sorry but I truly can't work on your request right now, I'll try to fix it this or the next week.

python3 cookies.py
Using cookies from /Users/{usernmae}/Library/Application Support/Firefox/Profiles/ghyuqd3e.default/cookies.sqlite.
Cookie import failed: JSON Query to graphql/query: HTTP error code 401.

use py cookies.py

@new92 :(((((((((((

use py cookies.py

hey i am on macos. used python instead of python3. same issue

python3 cookies.py
Using cookies from /Users/{myusername}/Library/Application Support/Firefox/Profiles/ghyuqd3e.default/cookies.sqlite.
Cookie import failed: JSON Query to graphql/query: HTTP error code 401.

Hey @Meytiz so sorry for the delay 😭 I fixed the bug and the script works just fine ! Tested it myself 👍

Hey @ShivamB25 ok so you need to firstly download Firefox (if not already installed) and login to your instagram account before running cookies.py and IAM.py scripts.

I tested it today but it didn't work for me!

[::] Posts to like/unlike (num) >>> 10
Traceback (most recent call last):
File "C:\Users\M!IND\Downloads\IAM-main\IAM-main\IAM.py", line 3341, in
main()
File "C:\Users\M!IND\Downloads\IAM-main\IAM-main\IAM.py", line 2619, in main
browser = webdriver.Chrome()
^^^^^^^^^
NameError: name 'webdriver' is not defined
C:\Users\M!IND\Downloads\IAM-main\IAM-main>

@Meytiz try again now, I fixed the problem. Just make sure you update IAM before running it. Using the git pull command.

[!] WARNING: Not all packages used in IAM have been installed !

Traceback (most recent call last):
File "C:\Users\M!IND\Downloads\IAM-main\IAM-main\IAM.py", line 117, in
init(autoreset=True)
^^^^
NameError: name 'init' is not defined. Did you mean: 'int'?

@Meytiz fixed the bugs try one more time please

[::] Posts to like/unlike (num) >>> 10
Traceback (most recent call last):
File "C:\Users\M!IND\Downloads\IAM\IAM.py", line 3341, in
main()
File "C:\Users\M!IND\Downloads\IAM\IAM.py", line 2619, in main
browser = webdriver.Chrome()
^^^^^^^^^
NameError: name 'webdriver' is not defined

Now ?

yeap

Still the same error

[::] Posts to like/unlike (num) >>> 10
Traceback (most recent call last):
File "C:\Users\M!IND\Downloads\IAM\IAM.py", line 3343, in
main()
File "C:\Users\M!IND\Downloads\IAM\IAM.py", line 2621, in main
browser = webdriver.Chrome()
^^^^^^^^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 'Chrome'

What about now ?

[::] Users to follow (num) >>> 10
[?] Users to follow their followers (num) >>> 1
[::] Username No1 >>> ft360_ir
[✓] User added.
[+] Initiating following process...
[!] Error !
[*] Error message ==> name 'bot' is not defined
[1] Return to menu
[2] Exit
[::] Number (from the above ones) >>>

Hey @Meytiz let me fix it and I'll get back to you


Fix for undefined window when using o3dgc.js in a web worker.

Fixes an error when window is undefined when using o3dgc.js from a web worker.
CC @kmammou @pjcozzi

@kmammou this is for our glTF work.  We want to be able to show how to do mesh decompression in a web worker.
Also, would you be able to move O3DGC to its own repo (perhaps amd/o3dgc) instead of just having it in a subdirectory in this repo?  This will be better for the glTF ecosystem especially with O3DGC as the first - and premiere - mesh compression extension.


WIP: Cloud store using https://github.com/lytics/cloudstorage

do not merge, just for review

This change is

related to issue #63

closing since going with stow


Fix unhandled exceptions when invalid class indexes are entered with …

…RemoveFromClassCommand

Codecov Report

Merging #251 (4ce819e) into master (f0af483) will decrease coverage by 0.04%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #251      +/-   ##
============================================
- Coverage     59.70%   59.66%   -0.05%
  Complexity      677      677
============================================
  Files           129      129
  Lines          2591     2593       +2
  Branches        308      309       +1
============================================
  Hits           1547     1547
- Misses          957      959       +2
  Partials         87       87

Impacted Files
Coverage Δ

...gic/commands/classcommands/DeleteClassCommand.java
0.00% <0.00%> (ø)

...commands/classcommands/RemoveFromClassCommand.java
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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

LGTM


Move latex functions to its own module

Closes #15. Closes #29.

Codecov Report

Merging #30 (4e9723e) into master (af1aca2) will decrease coverage by 0.39%.
The diff coverage is 40.00%.

@@            Coverage Diff             @@
##           master      #30      +/-   ##
==========================================
- Coverage   88.44%   88.04%   -0.40%
==========================================
  Files          22       23       +1
  Lines         822      828       +6
  Branches      174      175       +1
==========================================
+ Hits          727      729       +2
- Misses         76       80       +4
  Partials       19       19

Impacted Files
Coverage Δ

mlresearch/__init__.py
86.66% <ø> (ø)

mlresearch/utils/__init__.py
100.00% <ø> (ø)

mlresearch/latex/_utils.py
19.23% <25.00%> (ø)

mlresearch/latex/__init__.py
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Bump tags to upgrade packages

Signed-off-by: Aadhitya A<EMAIL_ADDRESS>Description
This PR fixes #76
Notes for Reviewers
This PR is not a 100% fix but a routine to update the packages, some checks need to be done on Meshkit framework.
Signed commits

[x] Yes, I signed my commits.

Seems Meshkit v0.2.36 is not compatible when upgraded in the go.mod so I didn't touch that line for now...

Need to open up a new one next week... Closing this for now

@alphaX86, I tried to update this a while back and ran into a dependency issue. This failed due to a conflict with meshery-adapter-library, which in turn failed due to a conflict with meshkit. I have PRs open for each of these. I believe meshkit needs to be updated first, then meshery-adapter-library, and finally this library to resolve these dependency issues.

meshkit PR: https://github.com/meshery/meshkit/pull/155
meshery-adapter-library PR (will need to be updated once meshkit PR is merged): https://github.com/meshery/meshery-adapter-library/pull/79

Understood @jbyers19 👍

PR has staled much, closing....


fix: 🐛 use PropsWithChildren for FormContainerProps type

The children prop was not defined on FormContainerProps resulting in type errors when rendering a FormContainer with children.

@TimMikeladze I never seen this as a problem with the use of FormContainer. Is your issue referencing the upcoming changes of React 18?

I have this same issue with React 18.

@TimMikeladze I never seen this as a problem with the use of FormContainer. Is your issue referencing the upcoming changes of React 18? Because this package is currently not tested with new typing structures of React 18, the current FormContainer is using FunctionComponent type and that supports children by default

I noticed this when updating @types/react and @types/react-dom to 18.0.8 and 18.0.3. I haven't dug further into this issue though.

I released 3.3.0 which hopefully supports now both version (React 17 + 18) types. Thanks @TimMikeladze for your contribution

now it is working !


Pass source contents directly to rubyfmt instead of passing file path

This changes the document formatting LSP logic to pass file contents directly to rubyfmt through stdin instead of using the file path (see "Motivation" for more on why we need this). As part of Stripe's internal rubyfmt rollout, this also adds a flag to "opt-in" formatting directories -- again, more in "Motivation" for why.
Motivation

In the current Stripe development environment, there's a brief pause between when you save a file and when it gets synced to your devbox. If you enable format on save in VSCode and save before file syncing happens, passing the file path to rubyfmt will then run formatting on a not-yet-updated version of the file, which will then make edits and overwrite your changes. However, the LSP has access to the local copy, so this change makes it so that we directly pass the up-to-date source to rubyfmt (rubyfmt supports reading from stdin like cat myfile.rb | rubyfmt) to get around this issue.
The other wrench here is that internally, we previously were gradually rolling out rubyfmt behind an allowlist based on filename, but because we're now passing the source instead of the path to the rubyfmt binary, we need to be able to selectively format directories, hence the additional option added in the first commit.
Test plan

This passes existing tests to maintain existing functionality, and I manually tested the allowlist option.

For what it's worth, with the Sorbet Compiler we didn't do anything to prevent people from putting # compiled: true in their files other than just publicly and repeatedly ask them as much. We also set things up to mention our team on all new usages of # compiled: comments


STORAGE - [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: Hostname mismatch, certificate is not valid for 'db.xxx.supabase.co'. (_ssl.c:1002)

I'm trying to use the python SDK to upload a file to my bucket but is getting an error that says certificate verify failed.
Below is my code
from supabase.client import create_client, Client as SupabaseClient

supabaseURL = os.getenv("SUPABASE_URL")
supabaseKey = os.getenv("SUPABASE_KEY")

print(supabaseURL)
print(supabaseKey)
supabase: SupabaseClient = create_client(supabaseURL, supabaseKey)  # type: ignore

supabase.storage.from_("bucket").upload(
                f"subtitles/{caption.id}_{caption.language}.srt",
                os.path.abspath(output_path),
            )

Here's the full error
2023-06-30 15:16:20,937:ERROR - Error while translating captions: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: Hostname mismatch, certificate is not valid for 'db.xxxxx.supabase.co'. (_ssl.c:1002)
Any idea why this is happening? My bucket is public.

I'm pretty sure this isn't related to supabase-py, but searching online it seems like the SSL certificate_verify_failed error can be caused by any number of reasons - SO thread hope one of the answers help.


Not working

Hi,
At the begining i would like to say that i'm not programmer and i'm really bad at it. I'm a photographer and I want to use your code with my photo project. Unfortunatelly I've got problem with compiling your code. I've tried everything, install bunch of libraries and even arduino ide 1.8.5, but leds or even lcd not working. I've got this error:
`Arduino:1.8.5 (Windows 10), Płytka:"Arduino/Genuino Uno"
C:\Users\ThinkPad\Desktop\student\led\ArduinoUNO\Light-painter-master\Light-painter-master\Light_painter\Light_painter.ino:510:7: warning: extra tokens at end of #else directive
#else if CONSISTENT
   ^

C:\Users\ThinkPad\Desktop\student\led\ArduinoUNO\Light-painter-master\Light-painter-master\Light_painter\Light_painter.ino: In function 'void getFileName(char*, uint8_t, uint8_t)':
C:\Users\ThinkPad\Desktop\student\led\ArduinoUNO\Light-painter-master\Light-painter-master\Light_painter\Light_painter.ino:610:64: warning: default argument given for parameter 3 of 'void getFileName(char*, uint8_t, uint8_t)' [-fpermissive]
void getFileName(char* fileName, uint8_t index, uint8_t ext = 0) {
                                                            ^

C:\Users\ThinkPad\Desktop\student\led\ArduinoUNO\Light-painter-master\Light-painter-master\Light_painter\Light_painter.ino:110:6: note: previous specification in 'void getFileName(char*, uint8_t, uint8_t)' here
void getFileName(char* fileName, uint8_t index, uint8_t ext = 0);
  ^

`
Can you help me?

Everything is ok. I just forgot to check I2C adress of LCD screen. Sorry to bother!


Included link to SQL AHB video tutorial

@jjaygbay1

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Phlow views do not update when definition changed

For example when this definition:

is changed to:

It updates the title, the item list. It does not update the columns.

The inspector view is updated on Phlow view definition change, including the toolbar:

Same as before, a Phlow view does not update if it is not in a scene graph.


Performance issues

I tried to use this library to display some basic data from a database.
There were around 8 columns and a maximum of around 100 rows.
The problem is that its not possible to use the "table" with more than around 5 rows. Whenever scrolling is necessary, everything just becomes extremely laggy. It uses such high amounts of CPU that some devices/browsers even show a performance impact message. Scrolling is not even possible when the number of rows is >50, it then runs with <1FPS.
I looked at the code and found 2 mayor problems:

The rows are wrapped in a "Column" widget. This is super inefficient. I replaced it with a Listview.builder and got a huge performance increase.
The textinputfields in every cell are somehow pretty hard to render. Replacing them with standard text widgets leads to a massive gain in performance. However, the edit functionality is lost, which is why I transformed the "save" button into a "popupmenubutton" and introduced the options edit/delete.

Combining both points, everything works smooth like it should. Since point 2 is somewhat a conceptional change, I would at least recommend point 1.

You should consider forking the repository and making your changes available for others


Incorrect way to complete challenger

The Objective is like this:
function myTest(val) {
  var answer = "";

  switch (val) {
  case 1:
    answer = "alpha";
    break;
  case 2:
    answer =  "beta";
    break;
  case 3:
    answer = "gamma";
    break;
  case 4:
    answer = "delta";
    break;
  }

  return answer;
}

myTest(1);

But you can complete the challenger with:
function myTest(val) {
  var answer = "";

  switch (val) {
  case 1:
    return "alpha";
    break;
  case 2:
    return  "beta";
    break;
  case 3:
    return "gamma";
    break;
  case 4:
    return "delta";
    break;
  }

  return answer;
}

myTest(1);

This is okay. Both methods are an equally valid way to solve the problem. Thanks and happy coding!


How to pass optional group 'name' to the chart.

Hi,
I have a very similar issue as #23. The optional value you can pass to .group([value,[name]]) isn't available via angular as far as I can see. I do not mean legend like in #23 or chart group, but I think he was talking about 'name' and not 'legend'. My problem is how do you pass a group name to the chart via angular? dc-group only accepts one value and not the second optional parameter.
Thanks,
Sergio

@sergio91 I may need to expand the API to support this, for the time being using the strategy outlined here https://github.com/TomNeyland/angular-dc/issues/15#issuecomment-52341163 might work.

@sergio91 I may need to expand the API to support this, for the time being using the strategy outlined here #15 (comment) might work.

@TomNeyland thank you for the quick response.  Great stuff you are making here :).

This has been fixed in dc.js v2.0.0 beta 10 with https://github.com/dc-js/dc.js/pull/886/. You can now pass an array for the group option.

@sergio91 This is exposed via:
dc-chart-group The 'chart' prefix is to disambiguate the two different 'group' concepts.


Errata readme.md

I think $resource should be there as a function parameter.
Loading from a remote data source:
angular.module('App', ['ngResource', 'localytics.directives'])
.controller('BeerCtrl', function($scope,$resource) {
$scope.beers = $resource('api/beers').query()
});

Oh I see, you meant the $resource Dependency Injection?
Good catch!
I accepted your request #118.
Many thanks!


remove space in constant

I know that this file is auto-generated, but I saw the issue and wanted to report it in some way. :-/
⇾ "[EA] There is chance that it should be 'RADIUS_MPPE_KEY_LEN' here (without leading/trailing space)." found by "Php Inspections (EA Ultimate)"

This change is

Hey, @voku. Thank you for noticing it. It seems that this problem comes from radius.php (line 7) it should be fixed there as well. Let's fix it and get the green test run.

Hey, @voku, would you please fix the constant declaration in radius.php as well?

merged as https://github.com/JetBrains/phpstorm-stubs/pull/818


How to flip the recorded video

I'm recording video through front camera. I just want to flip the video as what it looks like in the preview. Please help me..!!

Easiest way on your PC should be using ffmpeg:
.\ffmpeg.exe -i original.mp4 -vf "transpose=0,transpose=1" flipped.mp4

This sample has been migrated to a new location where we can accept Pull Requests (check README for more information).
As recommended by GitHub, we are closing all issues and pull requests now that this older repo will be archived.
If you still see this issue in the updated repo, please reopen the issue/PR there. Thank you!


Apply filtering to the Stack dataset

What
Eliminate low-quality text from the Stack dataset.
How
Apply the filtering rules developed in the RedPajama project.

18a79d9db52a9d0b4cd9d9db62dd20a6c0f8dfff


2021-03-10 v. 0.1.4: migrated to null safety & added tool description

You need to fix errors from dart analyze output. Check it on local machine, on GitHub Actions it is not showing.

Errors successfully fixed.
But I noticed, that pub tool command:
flutter pub outdated --mode=null-safety
displays an incorrect result:
direct dependencies: all support null safety.
dev_dependencies: all support null safety.
But it's not true! Be carefully, and checks the dependencies manually.


tf.keras.layers.Embedding convert error

System information

Linux Ubuntu 16.04):
tf-nightly

Hello,
I have problem when converting "tf.keras.layers.Embedding".
"from_keras_model" works well but   "from_saved_model" doesn't.
Code
embed_model = tf.keras.models.Sequential([
    tf.keras.layers.Embedding(10, 10)
])
dinput1 = np.random.randint([1],10)
out=embed_model(dinput1)

MODEL_DIR = "embed_model"
embed_model.save(MODEL_DIR, save_format="tf" )
converter = tf.lite.TFLiteConverter.from_saved_model(MODEL_DIR)
#converter = tf.lite.TFLiteConverter.from_keras_model(embed_model)

converter.experimental_new_converter = True
tflite_embed_model = converter.convert()

open("dec_model.tflite", "wb").write(tflite_embed_model)

Error Log
Traceback (most recent call last):
File "/local1/sshwang/virtual_envs/pytorch_1/lib/python3.5/site-packages/tensorflow/lite/python/convert.py", line 180, in toco_convert_protos
enable_mlir_converter)
File "/local1/sshwang/virtual_envs/pytorch_1/lib/python3.5/site-packages/tensorflow/lite/python/wrap_toco.py", line 38, in wrapped_toco_convert
enable_mlir_converter)
Exception: Failed to find function '__inference__wrapped_model_251'. The imported TensorFlow GraphDef is ill-formed.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "embedding_model.py", line 21, in
tflite_embed_model = converter.convert()
File "/local1/sshwang/virtual_envs/pytorch_1/lib/python3.5/site-packages/tensorflow/lite/python/lite.py", line 920, in convert
return super(TFLiteConverterV2, self).convert()
File "/local1/sshwang/virtual_envs/pytorch_1/lib/python3.5/site-packages/tensorflow/lite/python/lite.py", line 752, in convert
self).convert(graph_def, input_tensors, output_tensors)
File "/local1/sshwang/virtual_envs/pytorch_1/lib/python3.5/site-packages/tensorflow/lite/python/lite.py", line 498, in convert
**converter_kwargs)
File "/local1/sshwang/virtual_envs/pytorch_1/lib/python3.5/site-packages/tensorflow/lite/python/convert.py", line 555, in toco_convert_impl
enable_mlir_converter=enable_mlir_converter)
File "/local1/sshwang/virtual_envs/pytorch_1/lib/python3.5/site-packages/tensorflow/lite/python/convert.py", line 183, in toco_convert_protos
raise ConverterError(str(e))
tensorflow.lite.python.convert.ConverterError: Failed to find function '__inference__wrapped_model_251'. The imported TensorFlow GraphDef is ill-formed.

@adonisues,
I am facing a different error stating ValueError: None is only supported in the 1st dimension. Tensor 'embedding_1_input' has invalid shape '[None, None]'. Please find the gist of it here.
Could you please provide the complete code to reproduce the issue reported here. Thanks!

@adonisues As you are using model.save, you need to use from_keras_model. Once you update that part of code, then everything works as expected. Please check the gist here.
Please verify once and close the issue if this was resolved for you. Thanks!


Allow OpenXR extensions to add properties to the OpenXRCompositionLayer node

This is a follow-up to PR https://github.com/godotengine/godot/pull/89880
There are a number of OpenXR vendor extensions that add new structures that allow configuring composition layers in special ways.
This PR allows GDExtensions to add properties to the OpenXRCompositionLayer node, and inject these new structures into the composition layer's configuration using the values of these properties.
I just posted PR https://github.com/GodotVR/godot_openxr_vendors/pull/117 to 'godot_openxr_vendors' as an example, which adds support for XR_FB_composition_layer_secure_content - here's how it looks in the editor:

Thanks!


Issue working with crispy forms?

I'm trying your uploader widget in a form, but couldn't get it to display. No matter what I did, it seemed to show the default FileUpload widget. Then I started debugging the issue one at a time, and finally when I stopped using {% crispy form %} to display the form and instead used {{ form.as_p }} to display it, your widget worked. Is there anything in particular about your widget that might not make it work with crispy forms? I am using a few custom widgets with the crispy library and have not had any issues. I will raise the issue in the crispy library github repo as well. Do let me know if you need any logging information or anything else about my current configuration.
My environment:
Python 3.12
Django 4.2.16
Latest version of django-image-uploader-widget==1.0.0

Just as a follow up turns out crispy forms does require some special things for custom widgets. But everything I have done in my custom widget, is there in your custom widget as well. So I'm not sure why it isn't working. I will raise the issue on crispy-forms repo as well. here is the docs for crispy-forms

Well, really i never tested this widget with crispy forms, and this is a good time for add a native support for it. I will work on this issue for next weeks.

hey thanks so much for this! i was looking at a similar approach, defining a custom template within crispy FormHelper.


Add concurrency limit for remote write

Prior to this commit, unbounded number of write would have been trigged from avalanche based on number of samples and batch size. Adding a new flag to limit the concurrency will be useful to simulate prometheus remote write sharding.
Signed-off-by: Arunprasad Rajkumar<EMAIL_ADDRESS>

Nice! Any plans to rebase, code changed slightly?

@bwplotka Done. PTAL. Thanks


Jsm/dont use scheme for model urls

For observation attachments and layers, the URL for a model is sent back from the API call, and is determined on the server. Existing code was using the server's request protocol which may not match that of the client side. This change does not use the server's request protocol and instead uses // which will match the browser's protocol.
This is an issue if MAGE is used via HTTPS but proxied through to HTTP. The observation.url property, for example, would come back as http://... rather than https://.... This fixes that issue.
Note: this will work correctly in a browser but I'm not sure if other API callers will respect //. Another solution would be to add a configuration variable / env var e.g. MAGE_CLIENT_URL_USE_HTTPS which, when set to true would return https as the scheme/protocol rather than whatever is on the request. I leave that to the MAGE devs to determine which is the better fix.

Pull Request Test Coverage Report for Build 524

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.01%) to 53.796%

Totals

Change from base Build 469:
-0.01%

Covered Lines:
2780

Relevant Lines:
4830

💛  - Coveralls

Are you seeing problems?  If so, what are you using to proxy?

Yes, we are proxying MAGE requests and only support HTTPS. We are using NGINX, for example, but that is not really relevant. The problem is that HTTPS is proxied to the actual MAGE server which is listening on HTTP. So when the server builds the URL (path) using req.getProtocol(), it returns "http" since that is actually what the request URL's scheme is. But once this makes it back to the client (browser), the client uses this URL to manage attachments (and other things). But our proxy is not listening on HTTP, only HTTPS, so we end up with a 404 from NGINX.
We could fix this by listening for HTTP traffic in NGINX as well and proxying it appropriately, but IMO the MAGE server shouldn't be dictating the protocol that a client should use, so using // instead of http[s] fixes this in the browser.
We could also fix this by running MAGE internally on HTTPS but we generally run internal APIs without SSL and use NGINX to proxy SSL traffic through.

@jsalankey I believe it is relevant, you need to set X-Forwarded-Proto header.
https://expressjs.com/en/4x/api.html#req.protocol
I believe something like this will work for NGINX:
proxy_set_header X-Forwarded-Proto

That's great -- didn't realize Express would support that. Will give it a try.

That works -- closing this.


serve the generated main.dart file so source_maps work for it

Description
Writes the generated main.dart file for flutter web apps to the web asset server so it can be served.
This fixes source maps for the root entrypoint.
Related Issues
https://github.com/flutter/flutter/issues/50753
Tests
I couldn't find any existing integration tests where I could ensure this file is served, if one does exist please point me at it and I would be happy to update it :).
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

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

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

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

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

Actually I found the right test, although it is currently skipped.
Note that if I remove the IOOverrides zone the test does pass though.

I had some issues with it failing locally - though if you comment out the skip/remove the IOOverrides and it passes that is good enough for me

I haven't unskipped the test for now - mostly just noting that the new expect does in fact pass when ran locally.


Block the Master database from changes made in slave.

In other words, one way syncronization. Is it possible?
I've thought that the current parameter "MSS_PerformSlaveJobs" regulates this behaviour but it does not.

One way sync is the easiest, just don't include MSS_PerformSlaveJobs in the master. The only thing you have to take into account is, if slave have a modification time later that master, the slave entry will prevail.

Looking back into this, you should include MSS_PerformSlaveJobs to false in settings.


wasm: disable C++ tests.

Signed-off-by: Piotr Sikora<EMAIL_ADDRESS>

https://github.com/envoyproxy/envoy/pull/21487 was merged instead.


ERROR in ./~/react-tap-event-plugin/src/injectTapEventPlugin.js

I got this error. Any idea how to resolve this?
ERROR in ./~/react-tap-event-plugin/src/injectTapEventPlugin.js
Module not found: Error: Cannot resolve module 'react/lib/EventPluginHub' in /home/chris/Projects/Web/tailpos/static/node_modules/react-tap-event-plugin/src
 @ ./~/react-tap-event-plugin/src/injectTapEventPlugin.js 23:2-37

ERROR in ./~/react-tap-event-plugin/src/TapEventPlugin.js
Module not found: Error: Cannot resolve module 'react/lib/EventConstants' in /home/chris/Projects/Web/tailpos/static/node_modules/react-tap-event-plugin/src
 @ ./~/react-tap-event-plugin/src/TapEventPlugin.js 22:21-56

ERROR in ./~/react-tap-event-plugin/src/TapEventPlugin.js
Module not found: Error: Cannot resolve module 'react/lib/EventPluginUtils' in /home/chris/Projects/Web/tailpos/static/node_modules/react-tap-event-plugin/src
 @ ./~/react-tap-event-plugin/src/TapEventPlugin.js 23:23-60

ERROR in ./~/react-tap-event-plugin/src/TapEventPlugin.js
Module not found: Error: Cannot resolve module 'react/lib/EventPropagators' in /home/chris/Projects/Web/tailpos/static/node_modules/react-tap-event-plugin/src
 @ ./~/react-tap-event-plugin/src/TapEventPlugin.js 24:23-60

ERROR in ./~/react-tap-event-plugin/src/TapEventPlugin.js
Module not found: Error: Cannot resolve module 'react/lib/SyntheticUIEvent' in /home/chris/Projects/Web/tailpos/static/node_modules/react-tap-event-plugin/src
 @ ./~/react-tap-event-plugin/src/TapEventPlugin.js 25:23-60

ERROR in ./~/react-tap-event-plugin/src/TapEventPlugin.js
Module not found: Error: Cannot resolve module 'react/lib/ViewportMetrics' in /home/chris/Projects/Web/tailpos/static/node_modules/react-tap-event-plugin/src
 @ ./~/react-tap-event-plugin/src/TapEventPlugin.js 27:22-58

Maybe this is related? https://github.com/callemall/material-ui/issues/5573

freeze react version.
after rm -rf node_modules && npm install
% cat package.json | grep react
      react: ~15.3.0,
      react-addons-create-fragment: ~15.3.0,
      react-addons-css-transition-group: ~15.3.0,
      react-addons-pure-render-mixin: ~15.3.0,
      react-addons-transition-group: ~15.3.0,
      react-addons-shallow-compare: ~15.3.0,
      react-addons-update: ~15.3.0,
      react-dom: ~15.3.0,

@j138 Thanks for the reply. How can I freeze react version. I am new in this thing. :(. I tried this
npm install -g npm-freeze

# Create version manifest in your project folder
npm-freeze manifest
rm -rf node_modules
npm install
npm-freeze check

but still error persist.

I freezed<EMAIL_ADDRESS>that resolved problem.
s/^15.3/~15.3/g
BTW, resolved this issue.
callemall/material-ui#5573
I just fixed after update<EMAIL_ADDRESS>and<EMAIL_ADDRESS>


case statement trailing comma bug

Trailing commas cause odd behavior matching everything except the match value.
ary = [1, 2, 3]

ary.each do |n|
  case n
  when 1
    puts "one #{n}"
  when 2,
    puts "two #{n}"
  end
end

Output:
one 1
two 3

Expected
one 1
two 2

Flipping case1 and case2 makes the output stranger.
two 1
one 1
two 3

Crystal 0.30.0 (2019-08-05)

LLVM: 8.0.0
Default target: x86_64-apple-macosx

a comma means another expression to test against.
This is not a bug, just a bad edge case. I don't know how to improve it.

To be clear: Properly formatted the code looks like this:
ary = [1, 2, 3]

ary.each do |n|
  case n
  when 1
    puts "one #{n}"
  when 2, puts "two #{n}"
  end
end

This should explain the "strange" behaviour. There is no way to tell whether a trailing comma in a case expression might be a mistake or not.

Actually, we. could maybe disallow a newline after a comma in this case. The problem then becomes how to include many expressions that would look bad in a single line... but I'm open for suggestions!

(I say this because I was bitten by this too)

Maybe forcing a \ at the end of the line:
  case n
  when 1
    puts "one #{n}"
  when 2, \
    3
    puts "two #{n}"
  end

Or require parenthesis:
  case n
  when 1
    puts "one #{n}"
  when (2,
        3,
        4)
    puts "two #{n}"
  end

It's easy to know what the when condition includes.

FYI, Ruby requires () for method calls with arguments in when.
case 1
when 2,
  puts "string"
end

-:3: syntax error, unexpected tSTRING_BEG, expecting do or '{' or '('
  puts "string"

@lugia-kun That's an interesting fact. But won't make a difference for this issue anyway.


Fix broken link in CONTRIBUTING.md

The link to the project issues page showed a 404 error.

Hey @LuckyLuke19
Thanks for contributing.
Approved.


Improve Table component

Overview

support all possible data types from Table Schema
support inline editing (with corresponding widget e.g. dates)
TBD

We can use the inline functionality from the component Reactdatagrid that we are using: https://reactdatagrid.io/docs/inline-edit
If you are talking about the different types from https://specs.frictionlessdata.io/table-schema/#types-and-formats

string

number

integer

boolean

date

array

object

time

datetime

year

yearmonth

duration

geopoint

geojson

any
Looks like they already have some custom editors for the above types:

[x] string the default text input editor

[x] number @inovua/reactdatagrid-community/NumericEditor

[x] integer @inovua/reactdatagrid-community/NumericEditor

[x] boolean @inovua/reactdatagrid-community/BoolEditor

[x] date @inovua/reactdatagrid-community/DateEditor

[x] any the default text input editor

[ ] array a component to add, remove and reorder values from a list

[ ] object maybe just a textarea with the json and we can validate a valid json

[ ] time I think we would need to develop

[ ] datetime  @inovua/reactdatagrid-community/DateEditor

[ ] year NumericEditor with validation

[ ] yearmonth NumericEditor with validation

[ ] duration NumericEditor with validation

[ ] geopoint a component that would open a openstreetmap and you could navigate/search a point

[ ] geojson same as the json one but with a different validator

I think it would make sense to break this issue in multiple ones, maybe one just for the initial inline edit using the default input text and then an issue for each type of column. What do you think?

A question about applying the changes to the data resource. Would we use frictionless framework to apply the changes, using the Transform steps? https://framework.frictionlessdata.io/docs/guides/transform-steps Do we have on the framework a method to update an individual cell? I couldn't find it looking at the docs and quick looking at the source code.

@aivuk
Thanks. My thinking currently is to implement steps.table_patch that takes as an argument a structure like:
- row_position
  field_position
  value
- ...

which will be produced by the Table component e.g. listening on onUpdate prop


JSdoc/TSdoc inserting @returns despite function not returning data

VS Code Version: 1.54.3
OS Version: Windows_NT x64 10.0.19041

Steps to Reproduce:

Write a function that uses an empty return statement (potentially as a guard), most minimalistic example:function t(){
  return;
}

Type /** above the function followed by enter to initiate jsdoc autocomplete/insertion
Observe that a @returns clause was added, despite the function never returning any data:/**
 *
 * @returns
 */
function t(){
  return;
}

Does this issue occur when all extensions are disabled?: Yes

I hope it's fine for me to add additional description here.
I am not sure whether this really qualifies as a bug, since this may have been a conscious decision. I would however personally expect JSdoc/TSdoc to only insert the @returns clause when the return keyword was followed by some data. I noticed this behavior when I was using return statements as guards and thought it may be worth reporting.
A somewhat more advanced, but very related, idea would be to also omit the @returns when a promise of type void is returned. This is probably not doable in js, but might be OK in typescript (especially when explicit return types are provided).

This is intentional, since you might be stubbing out the function JSDoc first.
You can turn off @returns generation altogether in Settings as of TypeScript 4.3.


Implement TS trait for slices

Use impl_shadow! macro to implement TS for slices of type T: TS.
This allows for structs like this:
#[derive(ts_rs::TS)]
#[export, export_to = "./bindings/"]
struct Foo {
    arr: Box<[u8]>
}

This is great for saving memory when dealing with immutable collections of unknown size, since there is no capacity to store like there is with Vec<T>

Thanks for the PR. Could you add a test & remove the version bump?

Sure thing, I was just adding a fix to the derive macro.

Are you sure about the version bump removal though? You won't be able to push the update to crates.io due to the version number being the same, and it is a new feature, which normally would require a bump to the minor version to conform to semver

Done!

@escritorio-gustavo yeah, thanks for removing it. I don't publish every PR as a new release, so i'd like to do that separately. It'd also be a backwards-compatible addition, so if we'd publish this PR as a new version, i think it should be 7.0.1.


Newly created build tool and command plugin templates should have conditional support for XcodeProjectPlugin #6446

This is a SwiftPM 5.9 nomination of #6446 and it's follow-on fix #6451.
Motivation
Package plugins can be applied to packages in Xcode without any modification. But Xcode also has an additional XcodeProjectPlugin module that provides extra API for applying plugins to Xcode projects. Since this is a common use case, it makes sense for newly created plugins to have conditional use of that extra API by default. If there is support for applying plugins to the project formats of other IDEs in the future, it would make sense to add those to the templates as well.
This support is currently always added but we could consider having options in the API for whether to add it.
Modifications

extended the template-generated source code
added to the unit tests

rdar://108166582

@swift-ci smoke test


HTTP headers and custom data added in Custom Asserters not cleaned up

Describe the bug
I use Custom Asserter to add a header/custom data to an HTTP Request for some tests. I use AssertConvoBegin to attach it and AssertConvoEnd to remove it after the test. The problem is that if a test fails then AssertConvoEnd method isn't triggered and the header/data isn't removed - it is persistent in next convo tests and it makes them fail, because the handler method depends on the existence of this particular header/data.
To Reproduce
It's easier to show it.

Expected behavior
I expect the header/data to be removed even if the test fails.
Maybe you have a different idea how to attach custom headers/data to some tests?  Only to some, I don't want them in all tests (I set the header Authorization in requestHook.js), but this is the only thing I want in all tests!
Botium Flavour:

Botium Box not used
Botium CLI not used
Botium Bindings 2.0.16

The problem is:

requestHook executes after my custom asserter (LOCATION). I would be able to clean it using  delete requestOptions.body.data if it was the other way around (I could see if any the data or headers persisted from previous requests and delete them - then custom asserter would add only the correct one. If I did it now it would delete what I set in my customAsserter in # begin.
I can't get requestOptions in RESPONSE_HOOK. only the response data is there.

I just found a way to reproduce the scenario:

I changed the message in convo file to the one that fails the test -> assertConvoStep doesn't execute, assertConvoEnd doesn't execute. the header/data set before in assertConvoBegin is persisted and available in next requests - I don't want that.

this is the only way I found to do this. first the header is attached, then the request is sent, then process.nextTick is executed and the request is still sent with the data I set.

Hi, you invested quite some work in this issue, and you were on the right track.
First, you can use a logic hook (not an asserter) to attach a piece of information to the current session context of the HTTP connector:
....
"LOGIC_HOOKS": [
    {
      "ref": "LOCATION",
      "src": {
        "onConvoBegin": "container.pluginInstance.view.location = { long: args[0], lat: args[1] }'"
      }
    }
  ]
....

The view is used as Mustache view object in the HTTP connector, so you can now do something like this to manipulate the HTTP header - same for the body template:
...
"SIMPLEREST_HEADERS_TEMPLATE": "{  {{#location}}\"X-HEADER-LOCATION\": \"{{long}}/{{lat}}\"{{/location}}  }",
...

Didn't try it, but it should work, if I made no typos ...

Hi!
Thanks for the response.

I tried using your solution, but it didn't work. is there a particular version of Botium that enabled Logic Hooks?
I also tried pointing the LogicHooks[0].src to a particular file like I did with Asserters e.g. "./botium/logichooks/latlong.js" and it didn't trigger this code. to be honest, I don't know what is the class structure (which particular methods it should have to work. I tried writing onConvoBegin method, but it wasn't executed.

I can see an article for creating CUSTOM ASSERTERS https://botium.atlassian.net/wiki/spaces/BOTIUM/pages/24477705/Developing+Custom+Asserters but I can't find one for creating CUSTOM LOGIC HOOKS. https://github.com/codeforequity-at/botium-core/tree/master/src/scripting/logichook/logichooks all have different class structures.
I would need something like this https://github.com/codeforequity-at/botium-core/blob/master/src/scripting/logichook/userinput/ButtonInput.js so I could set LOCATION long lat and it would add location field to meMsg. is it possible to do a custom one logic hook like that?

This one works for me. Don't know exactly what botium core version is required, but is part of botium core for a long time now.
the classes you mentioned have different class structure because not every logic hook or asserter has to implement all hooks. user input is a good choice if you want to add this behaviour on a step-by-step base, but not when doing it in the #begin section
botium.json:
{
  "botium": {
    "Capabilities": {
      "PROJECTNAME": "experitest",
      "CONTAINERMODE": "simplerest",
      "SIMPLEREST_URL": "https://box.botium.at/iambotium/reply",
      "SIMPLEREST_METHOD": "POST",
      "SIMPLEREST_RESPONSE_JSONPATH": "$.reply",
      "SIMPLEREST_BODY_TEMPLATE": { "username": "botium", "message": "{{msg.messageText}}", "location": "{{location.long}}/{{location.lat}}"},
      "SIMPLEREST_HEADERS_TEMPLATE": "{ {{#location}}\"X-HEADER-LOCATION\": \"{{long}}/{{lat}}\"{{/location}} }",
      "LOGIC_HOOKS": [
        {
          "ref": "LATLONG",
          "src": {
            "onConvoBegin": "container.pluginInstance.view.location = { long: args[0], lat: args[1] }"
          }
        }
      ]
    },
    "Sources": {},
    "Envs": {}
  }
}

latlong.convo.txt:
latlong

#begin
LATLONG 1|2

#me
Hello, Bot!

#bot

log output:

2019-12-18T15:05:29.303Z botium-SimpleRestContainer constructed requestOptions {
"uri": "https://box.botium.at/iambotium/reply",
"method": "POST",
"followAllRedirects": true,
"timeout": 10000,
"headers": {
 "X-HEADER-LOCATION": "1/2"
},
"body": {
 "username": "botium",
 "message": "Hello, Bot!",
 "location": "1/2"
},
"json": true
}
--

ah, ok. thanks for the snippet. I had LOGIC_HOOKS as a field under $.botium and not $.botium.capabilities. it works!
but now I would have to convert SIMPLEREST_BODY_TEMPLATE and SIMPLEREST_HEADERS_TEMPLATE values to strings and use moustache templating.
is it possible to put the { lat, long } object somewhere so I can add the location field in requestHook?
yes, there is. anything you add on container.pluginInstance.view object is added to an argument that is passed to requestHook function

so, this is solved for you ?
(added some items to the backlog to make such things easier in the future)

everything is solved. thanks for helping.
just one additional thing I changed.
using
container.pluginInstance.view.context.location
instead of
container.pluginInstance.view.location
so I have all the additional fields added on the context object. a lil bit cleaner imo
module.exports = ({ requestOptions, context }) => {
  // set the authorization token for every request
  requestOptions.headers.Authorization = `Bearer ${process.env.TOKEN}`;

  if (context.location) {
    requestOptions.body.location = context.location;
  }
};


Cannot Build after upgrading

Has followed the migration, but still cannot build for ios. Tried google and all methods not help fixing the issue.
CodeSign /Users/myUser/flutterApp/flutterExample/flutterExample/build/ios/Debug-iphonesimulator/Runner.app (in target 'Runner' from project 'Runner')
cd /Users/myUser/flutterApp/flutterExample/flutterExample/ios
export CODESIGN_ALLOCATE=/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/codesign_allocate
Signing Identity:     "-"
/usr/bin/codesign --force --sign - --entitlements /Users/myUser/flutterApp/flutterExample/flutterExample/build/ios/Runner.build/Debug-iphonesimulator/Runner.build/Runner.app.xcent --timestamp=none /Users/myUser/flutterApp/flutterExample/flutterExample/build/ios/Debug-iphonesimulator/Runner.app

/Users/myUser/flutterApp/flutterExample/flutterExample/build/ios/Debug-iphonesimulator/Runner.app: resource fork, Finder information, or similar detritus not allowed
Command CodeSign failed with a nonzero exit code
https://flutter.dev/docs/development/ios-project-migration
XCode Version: 11.4 for IOS 13.4
Flutter Doctor:
flutter doctor --verbose
[✓] Flutter (Channel stable, v1.12.13+hotfix.9, on Mac OS X 10.15.4 19E266, locale en-HK)
• Flutter version 1.12.13+hotfix.9 at /Users/myuser/flutter
• Framework revision f139b11009 (5 days ago), 2020-03-30 13:57:30 -0700
• Engine revision af51afceb8
• Dart version 2.7.2
[!] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
• Android SDK at /Users/myuser/Library/Android/sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-29, build-tools 29.0.2
• ANDROID_HOME = /Users/myuser/Library/Android/sdk
• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
✗ Android license status unknown.
Try re-installing or updating your Android SDK Manager.
See https://developer.android.com/studio/#downloads or visit https://flutter.dev/setup/#android-setup for detailed instructions.
[✓] Xcode - develop for iOS and macOS (Xcode 11.4)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 11.4, Build version 11E146
• CocoaPods version 1.8.4
[✓] Android Studio (version 3.6)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin version 45.0.1
• Dart plugin version 192.7761
• Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
[✓] VS Code (version 1.39.2)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.5.1
[✓] Connected device (1 available)
• iPhone 11 Pro • 6BB233AA-74B3-44F7-90FF-93984A2F73FB • ios • com.apple.CoreSimulator.SimRuntime.iOS-13-4 (simulator)
! Doctor found issues in 1 category.

for the android toolchain issue
refer this
robbinespu.gitlab.io/blog/2020/03/03/flutter-issue-fixed-android-license-status-unknown-on-windows/

first, check the code then try shifting to flutter channel to beta the run project.

Hi @wilson0506
Please take a look at this comment
As the comment suggest,  it's not a Flutter issue
Closing, as this isn't an issue with Flutter itself,
if you disagree please write in the comments and I will reopen it
Thank you


fix: a bug about crashes down when there are two slashes.

What is it?
#6004

[ ] Feature / enhancement
[ ] Bug
[ ] Docs / tests / types / typos
[ ] Infra

Description

Checklist:

[ ] My code follows the developer guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have ran pnpm change and documented my changes
[ ] I have made corresponding changes to the Qwik docs
[ ] Added new tests to cover the fix / functionality

here as result

However I tested in my local

@wmertens everything is fine


New command: 🔐 spo folder roleinheritance reset

Usage
m365 spo folder roleinheritance reset [options]
Description
Restores the role inheritance of folder
Options

Option
Description

-u, --webUrl <webUrl>
URL of the site where the file for which to reset role inheritance is located

-f, --folderUrl <folderUrl>
The site-relative URL of the folder

--confirm
Don't prompt for confirmation

Examples
Reset inheritance of folder with site-relative url /Shared Documents located in site _https://contoso.sharepoint.com/sites/project-x_
m365 spo folder roleinheritance reset --webUrl "https://contoso.sharepoint.com/sites/project-x" --folderUrl "/Shared Documents"

Additional Info
we may also reuse command we already have using Cli.executeCommandWithOutput. We may reuse the m365 spo folder get command and the m365 spo listitem roleinheritance reset.

@pnp/cli-for-microsoft-365-maintainers and another one on the permission management list 😉. Any comments on this one ?

Looks great @Adam-it

Looks great @Adam-it

Ok, opening it up

Can I work on this?

Sure @nanddeepn
Thank you for contributing! 👏

Hi @Adam-it @milanholemans @martinlingstuyl
m365 spo folder get command does not return listItemId, listId, or listTitle which can be passed to m365 spo listitem roleinheritance reset command.
If we need to reset the roleinheritance of sub-folder, then we can use
${args.options.webUrl}/_api/web/GetFolderByServerRelativeUrl('${args.options.folderUrl}')/ListItemAllFields/resetroleinheritance
If we need to reset the roleinheritance of list, we already have m365 spo listitem roleinheritance reset and we cannot use it here because always listTitle cannot be same as URL, as we might have spaces in list title and not in URL.
Considering we already have m365 spo listitem roleinheritance reset for resetting roleinheritance of list, should we target this command only for resetting roleinheritance of sub-folders?
Or, please correct me if I am missing anything here.
Thank you.

@nanddeepn thanks for the comment.
I agree we should go with GetFolderByServerRelativeUrl 👍
BTW for reseting roleinheritance of list we have m365 spo list roleinheritance reset not listitem

Thank you @Adam-it
GetFolderByServerRelativeUrl does not support resetting the roleinheritance at the list level.
In this case we have 2 options:
Option 1:
If folderUrl is specified as list name, we will call m365 spo list roleinheritance reset internally.
Option 2:
We can have this command spo folder roleinheritance reset only for resetting the roleinheritance of folder inside list/library.
Please suggest.

TBH I was thinking of the second option. The general aim was to target a command which is supposed to do the same as listitem one (as we may also reset folders using listitem one) but in order to make easier to find for the user we thought of creating dedicated file and folder commands which were suppose to do the same as the lisitem one. So I would say reseting the list permissions was never the intention of this command and for this list  roleinheritance reset command should be used


[Embedded] Enable SIMD types in embedded Swift

@swift-ci smoke test

@swift-ci smoke test

Looks great -- could you also add a test into test/embedded?

@swift-ci smoke test


[BUG] When running a simulation the sd_vehicle_interface node doesn't launch

We have developed an interface to an rFpro simulator that subcribes and publishes ROS topics directly.  When we click GO on the Aslan UI the vehicle doesn't start to drive because the sd_vehicle_interface node does not launch correctly.  When we manually launch this node using roslaunch the vehicle starts driving correctly.
The bug is that the sd_vehicle_interface doesn't start automatically and we believe it's supposed to.
We can launch the node manually and the vehicle starts to drive correctly using the following command
roslaunch sd_vehicle_interface sd_vehicle_interface.launch sd_vehicle:=twizy sd_gps_imu:=none sp_speed_source:=ndt_speed sd_simulation_mode:=true
We're running Aslan on Ubuntu 16.04.

Hi @mdempse1
Please use this command when running
rosparam set /use_sim_time true


[10.x] Add Model::defaultRouteKeyName()

This is a WIP as I have not written tests yet.
Add the ability to default for route keys.
Why is this necessary?
It's not strictly necessary at all, just a touch of developer experience.
Doesn't this already work with the primary key?
Yes! However, OWASP recommends using longer, random strings in URL routes. Laravel friend @tpetry also recommends using UUIDs as a supplement to incrementing int IDs rather than a replacement.
We use both big ints and UUIDs for our models, so the primary key remains id, but for all public references via our API, it's uuid.
This is a nice helper to encourage security best practices while also not incurring the performance penalties and general frustration of using UUIDs as PK/FK.
Alternatives
Add the following to every model (possibly by employing a trait) or create a BaseModel class that all Models extend from:
public function getRouteKeyName()
{
    return 'uuid';
}

Why do we need another indirection when we can already use getRouteKeyName? Is it only because you want to use a property instead of overwriting a method?
If thats the case, you could also do:
protected static $routeKeyName = null;

public function getRouteKeyName()
{
     return static::routeKeyName ?? $this->getKeyName();
}

But I still don't see the need. There is already a way to do this with the getRouteKeyName() method. You can also abstract it away with a common trait.


[Visual Editor] Fails render @inner_content of layout

When a page in rendered with this layout https://github.com/BeaconCMS/beacon/blob/main/guides/recipes/create-a-blog.md#create-the-layout it fails to render the slot and instead render plain text:

@cibernox and @cbroeren this one was recently reported and I believe it fails because the <%= @inner_content %> expression is inside a node that also contains an expression and is nested. If you move the <%= @inner_content %> into the root (first level) it works.

I dived a bit more into this, and noticed that it's failing to render whenever a conditional slot is expected to be rendered, cq :if={true} vs :if={false}.

(Example layout template)

     Issue 274 - Layout with conditional inner content

true

    <%= @inner_content %>

false

    <%= @inner_content %>

(Visual Editor Result)

In our Visual Editor we check whether the ast has a rendered_html value, and if so, use that. If it doesn't have it, we render the inner content as html. https://github.com/BeaconCMS/beacon_live_admin/blob/e4f18f83ae1c1c0c60e4cfdf2b6ae5c114f0fe91/assets/svelte/components/LayoutAstNode.svelte#L14-26
The ast for the sections that conditionally should be rendered, do have the rendered_html, the other ones don't.

(Different ast values for conditional blocks)

From the visual editor perspective, we can only work with the ast we have, and I don't think there's an easy way to cover for this difference from inside the editor itself, because the existence of rendered_html is based on different conditions. Beacon is adding this value based on whether the element has eex attrs, and then sets the rendered_html when it should be rendered. https://github.com/BeaconCMS/beacon/blob/6db870adff76db2bc45d742ec07cc08f134fe3fb/lib/beacon/template/heex/json_encoder.ex#L257-L262
https://github.com/BeaconCMS/beacon/blob/3cd6d605ef456516563f5b1416eb1aea9ea05a48/lib/beacon/template/heex.ex#L63
@leandrocp any ideas on where you think we should make a change or add a check on the eex conditional rendering or the rendered_html being set?

Essentially it only supports @inner_content in the root level: {:else if node.tag === "eex" && node.content[0] === "@inner_content"} as proven in this test https://github.com/BeaconCMS/beacon/blob/6f0e4ffdb9a9e03ccc6b6fce424e828075a6e1af/test/beacon/template/heex/json_encoder_test.exs#L464-L488
But it falls into {:else if node.rendered_html} if the node is nested as children of a parent node that contains eex expresions, as per your example:
  test "layout inner_content inside eex" do
    layout_template = ~S|
    my_header

      Today
      <%= @inner_content %>

    |

    page_template = ~S|

page

    |

    assert_output(
      layout_template,
      [
        %{
          "tag" => "header",
          "attrs" => %{},
          "content" => ["my_header"]
        },
        %{
          "tag" => "div",
          "attrs" => %{":if" => "{true}"},
          "content" => [
            %{"tag" => "span", "attrs" => %{}, "content" => ["Today"]},
            %{
              "tag" => "eex",
              "attrs" => %{},
              "content" => ["@inner_content"],
              "metadata" => %{"opt" => ~c"="},
              "rendered_html" => "\n    <div>page</div>\n    "
            }
          ],
          "rendered_html" => "
\n  Today\n  \n    <div>page</div>\n    \n
"
        }
      ],
      %{inner_content: page_template}
    )
  end

Since the parent div has a rendered_html field, it will ignore the children and just render it. That's tricky to solve, @cibernox mentioned "flag it somehow and the rendered_html have some marker of where to render the nested content" when we first discussed it, something like:
    "metadata": {
       slot_position: 94 // The slot is in the 94th position of the string
     },

Instead of string position, we could maybe mark the index position in the AST:
%{
  "tag" => "div",
  "slots" => [1] # second position in `content` list
  # ...
}

That's important because we can't lose what comes before and after the slot, ie: it should still render the span element so users can edit it, and any other elements that comes after.
Do you think that's something you work with in the visual editor?

Essentially it only supports @inner_content in the root level: {:else if node.tag === "eex" && node.content[0] === "@inner_content"} ... But it falls into {:else if node.rendered_html} if the node is nested as children of a parent node that contains eex expressions

That’s not just supported in the root level (from the visual editor side). If it’s nested it will also work because it reaches the same logic for children. It’s just only rendering the @inner_content when it’s the only content in the ast that’s marked as eex (theoretically the first, but I think it should check for only).

Since the parent div has a rendered_html field, it will ignore the children and just render it. That’s tricky to solve,

Correct. But I try to understand the tricky part if that’s because of your next comment:

That’s important because we can’t lose what comes before and after the slot, ie: it should still render the span element so users can edit it, and any other elements that comes after.

In the visual editor, if the ast doesn’t have the rendered_html, it will render it’s content correctly using the list of content items (like the span from your example). Then it will act the same as when the condition is :if={false} or for any other non-conditional parent.
Note: This also affects / applies to conditional elements for pages, similar to the layout. In that case, it will currently also just use the rendered_html and not render each content item separately, so also not being able to select them separately in the editor (the visual part, we still show it in the list of Content in the sidepanel though).

This is all considered just from this conditional use case. If it's just about this :if attribute, I would say there might not be a reason for the ast to have or not have the rendered_html. But I know we need to add that there's more logic involved in when and why that's being set (whether it has any other eex attributes for example). So there might be other use cases where the inner content (or any page content) will not be rendered correctly or as separate (selectable/editable) elements.
So the real question is maybe: What eex attributes could/should affect this rendered_html being set? here (maybe_add_rendered_html) and here (has_eex_in_attrs?)
And/or when should we have and use the rendered_html as precedence over the content list? Extracting the content from the html string from inside the visual editor sounds really tricky and not favorable.


parse string log level from config

Requested by @avinson
Signed-off-by: David Lawrence<EMAIL_ADDRESS>(github: endophage)

LGTM


Use null aware operators on _ButtonStyleState

Added null aware operators to prevent crash caused by using the null assertion operator on a null value.
Fixes #90374
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

This PR has been idle for 2 months so I'm going to close it. If you're interested in resuming work on it, please reopen.


Desktop view

Portfolio Project
update index.html file
update styles.css file
update node_modules, add linter to eliminate error
update read.MD file
create media queries for desktop view and implements changes as instructed.
modify styles.css, create background images for work section in desktop view

Please apart from the problem am having do i also have any problem please bare with me it is because of time


[Feature request] Bookmarking chapters in ComicRack format

The ComicRack format is capable of adding chapters like
<Page Image="0" Bookmark="Cover"/>
And it will show up on any e-readers as chapters in the table of contents.
Can we please have the possibility of bookmarking without having to manually edit the XML file.

I didn't even know that Bookmarks were a part of the ComicRack spec but I just double checked on a copy from archive.org and turns out it does.
I don't know when I'll have time to implement this but I plan to.

Just to throw in my two cents:  I feel like bookmarks are more ephemeral and personal, and should live outside of the embedded metadata within a comic file.  While bookmarks are metadata, it's personal metadata, and maybe bookmarks (and other things like current page, finished status etc) are best managed by reader applications outside of the comic file.  Sounds like good fodder for conversation over at the anansi-project

This is true and ComicRack follows this idea in some instances but not others. For example the personal rating vs the community rating.
I am also thinking that I will complete the few elements that ComicTagger does not currently support from ComicRack, and then support a separate format for any changes, such as in this case I think the current usage is for labeling chapters and should probably be renamed as such, whether in a similar format as ComicInfo.xml or another type.
I have made a PR at anansi-project/comicinfo#13 to get it added to the v2.0 spec for completeness.

@VictorGamerLOL what programs have you used that implement the bookmark feature?
We are discussing it in anansi-project/comicinfo#13 and were wondering how programs implemented it.

@lordwelch I did not use any program for this. I just read about it in the kindle comic converter wiki, which said that you must use ComicRack metadata to change the default metadata, and the example they used there had bookmarks of chapters.


1.8.0-beta: ts.transpile now errors for .js files

The ts.transpile method now throws an error when given a file with a .js extension:
test.js:
var ts = require("typescript");
var diags = [];
var output = ts.transpile("const a = 10; export default a;", { module: 1 }, "input.js", diags);
console.log(output);
console.log(JSON.stringify(diags));

output:
Heracles:plugin-typescript frank$ node test.js
/Users/frank/work/plugin-typescript/node_modules/typescript/lib/typescript.js:1682
                throw new Error("Debug Failure. False expression: " + (message || "") + verboseDebugString);
                ^

Error: Debug Failure. False expression: Output generation failed
    at Object.assert (/Users/frank/work/plugin-typescript/node_modules/typescript/lib/typescript.js:1682:23)
    at transpileModule (/Users/frank/work/plugin-typescript/node_modules/typescript/lib/typescript.js:47824:18)
    at Object.transpile (/Users/frank/work/plugin-typescript/node_modules/typescript/lib/typescript.js:47832:22)
    at Object.<anonymous> (/Users/frank/work/plugin-typescript/test.js:3:4)
    at Module._compile (module.js:425:26)
    at Object.Module._extensions..js (module.js:432:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:313:12)
    at Function.Module.runMain (module.js:457:10)
    at startup (node.js:138:18)

The actual error causing this is TS5055:
Cannot write file 'input.js' because it would overwrite input file.

This breaks e.g. the systemjs built-in typescript transpilation:(https://github.com/systemjs/systemjs/blob/6243d0c7700ca802699796cfae0f9e213072e077/dist/system.src.js#L1210)
Looks to be caused by the changes made for allowJs option in https://github.com/Microsoft/TypeScript/pull/5471.

May we see a 1.8 branch with this fix? Currently the entire 1.8 branch is broken for us, a 1.8.3 release would be appreciated.

@denis-sokolov this has been ported in release-1.8 a while back in https://github.com/Microsoft/TypeScript/pull/6791.
do you see the same issue on<EMAIL_ADDRESS>

@mhegazy, ah, sorry, I did not find that it was ported. Yes, I did all my testing on 1.8.2, I never had 1.8.0 installed. I shall double-check, just in case. Anything else to be on the lookout for?

We believe this issue has been fixed. if you are still running into issues please file a new issue and give us more information to be able to diagnose it.

For other people who stumble here. My problem was caused by the es6-default-import-commonjs problem, and was solved by upgrading ts-loader from 0.7.2 to 0.8.1.


Protobuf Submodule Reference Broken

What version of gRPC and what language are you using?
1.38.0
What operating system (Linux, Windows,...) and version?
N/A
What runtime / compiler are you using (e.g. python version or version of gcc)
N/A
What did you do?
Cloned the GRPC repo recursively using
 git clone --recurse-submodules --shallow-submodules https://github.com/grpc/grpc.git

What did you expect to see?
A successful checkout
What did you see instead?
...
fatal: remote error: upload-pack: not our ref 436bd7880e458532901c58f4d9d1ea23fa7edd52
fatal: the remote end hung up unexpectedly
Fetched in submodule path 'third_party/protobuf', but it did not contain 436bd7880e458532901c58f4d9d1ea23fa7edd52. Direct fetching of that commit failed.

Also, navigating to that submodule (third_party/protobuf @ 436bd78) from the GitHub web UI returns a 404 confirming the dead link.

Also hitting this problem, seems like protobuf is now something else?

Looks like this is the new home of protobuf: https://github.com/protocolbuffers/protobuf

That is really unfortunate.
If this repos stays renamed, building an old version of grpc will be not pleasant ...

That is really unfortunate.
If this repos stays renamed, building an old version of grpc will be not pleasant ...

I'm using the zip file now rather than git to fetch the source - and it's actually much faster to fetch it.

That is really unfortunate.
If this repos stays renamed, building an old version of grpc will be not pleasant ...

I'm using the zip file now rather than git to fetch the source - and it's actually much faster to fetch it.

Just downloaded grpc-1.33.2.zip and it didn't contain files in third_party/protobuf. Am I missing something?

Sorry, I spoke too soon. I couldn't get the canned sources to build, either.
Richard

From: Eric Wolf @.>
Sent: 28 July 2021 16:59
To: grpc/grpc @.>
Cc: Whitehead, Richard @.>; Comment @.>
Subject: Re: [grpc/grpc] Protobuf Submodule Reference Broken (#26809)
[External Email]
That is really unfortunate.
If this repos stays renamed, building an old version of grpc will be not pleasant ...
I'm using the zip file now rather than git to fetch the source - and it's actually much faster to fetch it.
Just downloaded grpc-1.33.2.ziphttps://urldefense.proofpoint.com/v2/url?u=https-3A__codeload.github.com_grpc_grpc_zip_refs_tags_v1.33.2&d=DwMCaQ&c=Qwsh1H-X9ypOoLLEcAIltQ&r=w6ludjNtDQpNasrV7-vv8UaUPqkFSZ8tefXB_suiJPM&m=y79o4NLPxaC-znSa-MhInDbC110x3Br3pb1T2IMlG9M&s=fNKrt8-dLKV63MLgoqyaAEZgPpZesI7TYCxC1CxHfZ8&e= and it didn't contain files in third_party/protobuf. Am I missing something?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_grpc_grpc_issues_26809-23issuecomment-2D888427916&d=DwMCaQ&c=Qwsh1H-X9ypOoLLEcAIltQ&r=w6ludjNtDQpNasrV7-vv8UaUPqkFSZ8tefXB_suiJPM&m=y79o4NLPxaC-znSa-MhInDbC110x3Br3pb1T2IMlG9M&s=1b22ACUkKKeWj1Tx2arXojH5EGs2VaBFSOssGqd2-kM&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AUPE7CW2NH3E7TGR7TEET5LT2ASO5ANCNFSM5BESVHGQ&d=DwMCaQ&c=Qwsh1H-X9ypOoLLEcAIltQ&r=w6ludjNtDQpNasrV7-vv8UaUPqkFSZ8tefXB_suiJPM&m=y79o4NLPxaC-znSa-MhInDbC110x3Br3pb1T2IMlG9M&s=eoYbqnpKBuvGaJVxwAHYQHx71xs3nfVZJ-mTR7upGXc&e=.

CONFIDENTIAL-
This email and any files transmitted with it are confidential, and may also be legally privileged. If you are not the intended recipient, you may not review, use, copy, or distribute this message. If you receive this email in error, please notify the sender immediately by reply email and then delete this email.

Duplicate of https://github.com/grpc/grpc/pull/26811, which tracks the fix as well. A protobuf repository change broke many open source tools.

https://github.com/ripple/rippled is also affected by this. Why is the issue closed? Is the fix merged?

This is closed as a duplicate. There were a few reports discussing the gRPC-specific problem, and it's easier to keep track of one (see #26811 ). The protobuf team is working on solving the root cause.