row.columnNames returns column names from current project not the row object

To get a list of column names in a project you can use the GREL expression:
row.columnNames
When working in a single project this seems intuitive. However, when working with the cross() function and manipulating rows from a second project it is unintuitive as row.columnNames always gets the column names from the current project, not the project the row object is from.
For example if doing a cross between two projects ("Project 1" and "Project 2") you might have an expression like this in Project 1:
cell.cross("Project 2", "P2 Column")[0]

When there is a successful match in values between the two projects this expression will return a row object from Project 2
However, the expression:
cell.cross("Project 2", "P2 Column")[0].columnNames

Will retrieve the columnNames from Project 1, even though the row object the variable is applied to is from Project 2. This seems non-intuitive - my expectation was that it would return the list of column names from Project 2 in this scenario.
An example of where this would be useful would be when you want to bring values from all the columns in Project 2 into Project 1. The only way to iterate through a set of columns in a project and is to know the column names so for example you might want to write an expression like:
with(cell.cross("Project 2", "P2 Column")[0], r, forEach(r.columnNames,cn,r.cells[cn].value).join("|||"))

However this fails because r.columnNames returns the column names for Project 1, not Project 2 in this scenario.
The documentation currently says that row.columnNames is:

An array of the column names of the project. This will report all columns, even those with null cell values in that particular row.

Ideally I think that given this variable is accessed via the row object, it should reflect the column names for that row. If this proves not possible then the documentation should be updated to be clear that although the columnNames variable is accessed via the row it actually relates to the current project.
Additional context
The most common use of the columnNames variable is to iterate through each cell in a row. Possibly this would be better implemented directly rather than needing the use of the columnNames variable (see https://github.com/OpenRefine/OpenRefine/issues/194)

This likely has the same root cause as #5633. Can you check to see if the fix in #5634 works for you?

Actually, I included a test for columnNames, so I suspect you'll it's fixed.
https://github.com/OpenRefine/OpenRefine/blob/2f29f502bea20979d703df6b9d8a3e937e7e45b8/main/tests/server/src/com/google/refine/expr/functions/CrossTests.java#L240

Tested and can confirm this is fixed


[Android] How to fetch dev bundle while run-android

Running program on Android simultor.
But it not fetch bundle from dev mod.
instead it fetched bundle from /index.android.bundle?platform=android&dev=false
Is there something I touched wrong?
[20:00:48] <END>   Crawling File System (1457ms)
[20:00:48] <START> Building in-memory fs for JavaScript
[20:00:50] <END>   Building in-memory fs for JavaScript (2844ms)
[20:00:50] <START> Building in-memory fs for Assets
[20:00:53] <END>   Building in-memory fs for Assets (2260ms)
[20:00:53] <START> Building Haste Map
[20:00:56] <START> Building (deprecated) Asset Map
[20:00:57] <END>   Building (deprecated) Asset Map (1756ms)
[20:00:58] <END>   Building Haste Map (5019ms)
[20:00:58] <END>   Building Dependency Graph (11593ms)
[20:00:58] <START> request:/index.android.bundle?platform=android&dev=false
[20:00:58] <START> find dependencies
[20:00:58] <END>   find dependencies (206ms)
[20:00:58] <START> transform

It seems like your app is not in Dev mode. To make sure it is follow these instructions:

Open the Developer menu by shaking the device or running adb shell input keyevent 82 from the command line.
Go to Dev Settings.
Make sure "JS Dev Mode" is checked.

The next time you reload the data, it should now have dev=true from the packager request.

@dsibiski
thanks a lot!
This bothers me for several hours.
close this issue


add doc around what environment variables we support for configuring helm/tiller

I don't remember if there is a doc like this we support. I remember just going through the code to figure this out every time. If we don't already have a doc, we may want to consider adding one.

Should be an easy amendment to https://github.com/helm/helm/blob/c82c0b6046b852f449dcaae768ba57331116dc87/cmd/helm/helm.go#L63-L75 if someone wants to take a crack at it.

Added PR #5123 to document the HELM_TLS_HOSTNAME environment variable.

closing as inactive.


Rename about-pull-requests.md to OMAR

Why:
Closes [issue link]

What's being changed:

Check off the following:

[ ] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[ ] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

Pull requests server extension


Automatic icon inheritance

Is your feature request related to a problem? Please describe.
I have lots of folders with icons. I'd like for every folder to have its icon-less children inherit its icon, with nested folders taking priority over their parents. But doing this on a folder-by-folder basis is incredibly tedious.
Describe the solution you'd like
A setting to automatically apply the “inherit icon” setting to folders, applying their current icon to all of their descendants. Child folders’ icons would take precedence over parents‘.
Describe alternatives you've considered
N/A
Additional context
N/A

I'll close this for now, because inheritance functionality will be deprecated in v2.10.0. By now, you can achieve everything with custom rules as well.


improved fs support with graceful-fs

Includes commits:
(fix) improved fs support with graceful-fs
(fix) updated tests to also use graceful-fs
fixes:
Error: EMFILE: too many open files

@marc136 can you please take a look asap? Otherwise I'll have to switch to a custom implementation. Thanks

Hello @samvloeberghs,
thanks for the request, I just merged it and will also update the version on npm.
Regards,
marc

great! 👍  :)


[qz-3613, qz-3188] Elide superfluous guards in a guarded expression.

Adds a DedupeGuards rewrite for FreeMaps that elides any guards in the continuation of a guard that were evaluated in the predicate of the guard, including any nested guards.
#qz-3613 Done.
#qz-3575 Done.

Test failure in PlannerSql2ExactSpec.


[BUG] FileNotFoundError: [Errno 2] No such file or directory: 'runs/conversation.json'

User: v
FileNotFoundError                         Traceback (most recent call last)
 in <cell line: 44>()
42
43 # Replace with your LLM instance
---> 44 interactive_conversation(llm)
2 frames
/usr/local/lib/python3.10/dist-packages/swarms/structs/conversation.py in save_as_json(self, filename)
230         """
231         # Save the conversation history as a JSON file
--> 232         with open(filename, "w") as f:
233             json.dump(self.conversation_history, f)
234
FileNotFoundError: [Errno 2] No such file or directory: 'runs/conversation.json'

Upvote & Fund

We're using Polar.sh so you can upvote and help fund this issue.
We receive the funding once the issue is completed & confirmed by you.
Thank you in advance for helping prioritize & fund our backlog.

@vyomakesh09 please try again and lmk


Stop fetching live stream while paused #2

Have you read the FAQ and checked for duplicate issues: yes
What version of Shaka Player are you using: 2.2.5-debug
Can you reproduce the issue with our latest release version: yes
Can you reproduce the issue with the latest code from master: yes
Are you using the demo app or your own custom app:demo
If custom app, can you reproduce the issue using our demo app:
What browser and OS are you using: win7x64+chrome
What are the manifest and license server URIs: vm2.dashif.org/livesim/utc_head/testpic_6s/Manifest.mpd
(you can send the URIs to<EMAIL_ADDRESS>instead, but please use GitHub and the template for the rest)
What did you do? Opened stream https://shaka-player-demo.appspot.com/demo/#asset=//vm2.dashif.org/livesim/utc_head/testpic_6s/Manifest.mpd;lang=ru
Paused it and waited for >5 minutes
What did you expect to happen? Player will not fetch segments when paused even after >5 minutes (up to timeShiftBufferDepth="PT5M")
What actually happened? Palyer still feteches segments after > 5 minutes of being paused
P.s. the same issue https://github.com/google/shaka-player/issues/982 - must be fixed , but in fact it is not.

Guys, hi! Will it be fixed in 2.3.0?

Yes, I'm sure it will.  We have identified the cause and will have a fix out in the near future.

The fix has been cherry-picked for v2.2.7.

Confirm, thanks!


rendering about arkit (D6, D7)

I used the LiveLink app to collect ARKit data for rendering.
I took the values of the first 51 blendshapes and used the command：
python -m main.render ./test_audios_ours_arkit_result/demo.npz ./test_audios_ours_arkit ./test_audios_ours_arkit_result/ for rendering,
but the result is as follows.
https://github.com/user-attachments/assets/f6b2c0df-65cd-462e-bc0c-b7eac41b7eee
Why is this happening?
Do the original ARKit data I collected need any preprocessing?

I used the LiveLink app to collect ARKit data for rendering. I took the values of the first 51 blendshapes and used the command：
python -m main.render ./test_audios_ours_arkit_result/demo.npz ./test_audios_ours_arkit ./test_audios_ours_arkit_result/ for rendering,
but the result is as follows.
20240816-164637.mp4
Why is this happening? Do the original ARKit data I collected need any preprocessing?

Hi, please refer to this issue https://github.com/X-niper/UniTalker/issues/5, where we privide the meaning of the 51 blendshapes. You may need to align the orders of the blendshapes


[Feature]: Tests to run on CI (GitHub Actions)

Feature Request

I have:  Tests to run on CI (GitHub Actions)

searched for such a feature request (https://github.com/yagop/node-telegram-bot-api/labels/enhancement) and found none

Introduction

There is no "workflows" folder and no "test.yml"  file present in Actions folder. Therefore, tests not running on push or pull request.
Example

We can add it to GitHub Actions. Below is the code snippet that can resolve this issue:

Very disappointed that github actions is not being used (if only on top of the existing ci/cd).


Warn the student if they’re using the REPL on a module whose code has changed.

Jaakko: "This should be pretty easy"

Decide later on the UI, after evaluating some options.


Exceptions vs. ParseErrors

There's some inconsistency between errors that are thrown within Handlebars.
When a matching error (or any other non-parse error) is thrown, we get to consume an Exception object with robust information. Info like line number and column in Exception is exceptionally useful when integrating Handlebars with build tools and providing users with useful output. For matching errors like if doesn't match title - 3:3 the message is somewhat useful, but not robust.
When a parsing error is thrown, we get an incredibly robust message such like:
Parse error on line 13:
...></section>{{/if}}
---------------------^
Expecting 'INVERSE', 'OPEN_ENDBLOCK', got 'EOF'

But there's no other useful information on the Error thrown. No line number, no column. So we're left to parse the message for the line number and column (based on where the caret is on that line of the message).
So what we're left with is Handlebars emitting two very different kinds of errors, with different information. Each is useful, but not complete. I'd like to request some kind of unification of these two errors. Line number and column is useful on each, and the same robust message that is provided with parse errors would be grand to have on Exception errors thrown. Additionally, I'm curious as to why - 3:3 appears at the end of the message for non-parse errors, as that's a rather cryptic way to represent the line and column already provided on the Exception object itself.

The goal for this task is to examine thrown exeptions throught the compiler and parser and unify the message and the properties of these exceptions.


Consider providing all functionality from single 'choc' (or similar) command

IMHO, it will improve adoption / discoverability if all chocolatey commands stem from a single entry point -- choc or whatever seems the most effectual.  I think most people will agree that chocolatey is a little slow to type (and easy to mistype ;0)
Just saw this tweet...
https://twitter.com/MotoWilliams/status/259731598503723008
The issue is that Chocolatey has all these aliases to things that are inside chocolatey -- and the docs kind of push you in this direction.

cinst
cuninst
clist
cpack
cup

Etc, etc...
While these are convenient for people who know them, they're not really discoverable to people who are new.  It's easy to remember one command -- a bit harder to remember a handful.
So I think that there are a few things that could be done to make the situation better

Stress a single command entry point
Better inline docs  - the output of chocolatey /? is an offender here.  Known issues, release notes  and legal stuff appear before the usage examples, for instance.
Inline docs / aliases / profile registration in Powershell.  Powershell's built-in help system is fantastic and man-page like.  Using this (better) could go a really long way.
A better landing page than the Wiki -- a fancy chocolatey.github.com that is noob friendly.  There's a lot going on at the chocolatey.org - I think it could be simplified with a focused landing page.

Other package manager command line ref examples (note they all have short, easy to type names ;0) -

npm
pip
gem
nuget
brew
apt-get

I know you guys are already familiar with all of these.. links provided solely to document their command usage.
I love Chocolatey and will continue to build packages for it... but I see these areas as needing improvements (and yeah, I know I should probably use the forums for this ;0)

This is called choco.


about kafka0.10

does this tool support kafka0.10  version？

@NLP1r I just updated the repository so it compiles and works with Kafka 0.10. README is also updated to indicate the kafka versions that are supported by Kafka Monitor.


React 18 Support Missing useSyncExternalStore on initial render.

There is an issue caused by the new changes for react 18 concurrent rendering support. On the initial render of an app (tested with a nextjs SSR application) a critical error is thrown stating the app will re-rendered client side only due to this missing hook.
This is  caused by the returned values not being memoized. The array syntax works fine when given a mutable array to edit (this way the array reference remains constant), but the single array syntax breaksdown when using groups, selectors, etc.

Bug fixed! The issue was being caused by some values not being mutable (mainly group inatnce)


Problem with Arch Linux

I was updating my Babel-CLI package and at final i have this warnings:
[salahaddin@TulipanROG ~]$ sudo npm update -g babel-cli
/usr/bin/babel-doctor -> /usr/lib/node_modules/babel-cli/bin/babel-doctor.js
/usr/bin/babel -> /usr/lib/node_modules/babel-cli/bin/babel.js
/usr/bin/babel-node -> /usr/lib/node_modules/babel-cli/bin/babel-node.js
/usr/bin/babel-external-helpers -> /usr/lib/node_modules/babel-cli/bin/babel-external-helpers.js
/usr/lib
└──<EMAIL_ADDRESS>
npm WARN optional Skipping failed optional dependency /babel-cli/chokidar/fsevents:
npm WARN notsup Not compatible with your operating system or architecture<EMAIL_ADDRESS>
My system architecture is:

fsevents is an OS X-only package, that's why it's an optional dependency of chokidar.  Closing.


Project update (taco.json update) should happen only if user chooses "y" when presented with the choice

Taco JSON should be updated only if user chose to update the project when presented with the choice
Fixing redundant status printing done when plugins are added/removed/updated

do we have a test case covering this?

Test updates are pushed.

:+1:


Odd relationships showing up in term "neighborhood" view

response to biotic stimulus
has some relationship to "cross-streak test evidence"
looks strange?

Term:
http://amigo.geneontology.org/amigo/term/GO:0009607
Lineage graph:
http://amigo.geneontology.org/amigo/term/GO:0009607#display-lineage-tab
"Neighborhood" as above.

These additional relations only occur in neighborhood_graph_json and neighborhood_limited_graph_json, topology_graph_json and others are clean.
There was something we had with Planteome that concerned this I think...

I'm pretty sure this would be very confusing to the average AMiGO user.

@cmungall Is this actually operating as intended (see https://github.com/geneontology/amigo/issues/220 ), short the failure to have a proper default icon, or do we need/want to have a slightly tighter filter in the owltools function?
@ValWood The neighborhood view is there to give a complete look at the immediate area around a term using all of the links available in go-plus, including, apparently, some of the more "exotic" ones.

Maybe add the labels for the relations? Then they would make more sense if people didn't want to look up what they are.

@ukemi Yes, a little more verbiage and explanation all around if this is the desired/intended behavior. The way the plumbing works though, we'd need to create an internal list of likely relations (or modify the loader to add the label information).

Maybe, but this particular one really doesn't make any sense ;) Why would you put evidence here?

Evidence is for annotation not for ontology terms.
or if not,  I  want to add
"inferred from direct assay"  and "inferred from mutant phenotype" to every term in the
catalytic activity branch, for example.
Why add confusing things?

I agree, it's confusing. How can a user tell where this is coming from ?

@ukemi Or possibly a quirk/problem in go-plus, as this is not a general problem?

All relations should have labels. Let's just filter out if we don't have a relation label for some reason. We should also show source of axioms.
In this case the relationship is simultaneously correct (if kind of weird) and of zero use to almost all AmiGO users.
In this case we should have our import chain only import eco-basic, as we don't need anything else in eco other that is-a / labels /syns

All relations should have labels. Let's just filter out if we don't have a relation label for some reason. We should also show source of axioms.
In this case the relationship is simultaneously correct (if kind of weird) and of zero use to almost all AmiGO users.
In this case we should have our import chain only import eco-basic, as we don't need anything else in eco other that is-a / labels /syns


iOS google oauth issue "The operation couldn’t be completed. (DCTAuth error 0.)"

Hi all
Running RN 47.2 with iOS 9. Followed the readme and compile all works fine except authorize. Getting some kind of weird error after successful login and callback. Not sure why there would be this error here. Did anyone encounter this or resolved it?
(file was linked via cocoapod)

Are you by any chance including a client_secret in your Google config? I had mine set up like this:
google: {
    callback_url: `[IOS SCHEME]:/google`,
    client_id: 'YOUR_CLIENT_ID',
    client_secret: 'YOUR_SECRET'
  }

and I was getting that same error. When I removed the client_secret it worked.
I set it up like this so I could use the config for Android too, but that appears the break iOS. The docs could be a little clearer about using an Android-specific configuration.

i am getting this problem with twitter oauth after getting error app is crashed. any one who have solution for this

^ was this ever resolved?


severe distortion and sudden change

Hi, I tested a case and found severe distortion. Is this expected behavior?

https://github.com/user-attachments/assets/c67c22c1-fc52-40d1-ab58-37fbaa719846

Can you try the CLI script and see if the point condition is correct?

Do you mean the render point clouds? This is the point cloud condition.
https://github.com/user-attachments/assets/0a9c5f15-605a-420d-b2a2-f5e032da107e


increase the default epochs to 1000 for linear regression

Signed-off-by: Xun Zhang<EMAIL_ADDRESS>Description
Increase the number of iteration for epochs to improve convergence of linear regression.
Issues Resolved
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #394 (a1ce465) into main (23cbbb4) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head a1ce465 differs from pull request most recent head 6a69747. Consider uploading reports for the commit 6a69747 to get more accurate results

@@            Coverage Diff            @@
##               main     #394   +/-   ##
=========================================
  Coverage     91.43%   91.43%
  Complexity      528      528
=========================================
  Files            59       59
  Lines          1529     1529
  Branches        122      122
=========================================
  Hits           1398     1398
  Misses           89       89
  Partials         42       42

Flag
Coverage Δ

ml-commons
91.43% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Firefox

What does it need to be available on Firefox?
If the steps are known and it's straightforward,
I could devote some time to do it.
nice plugin @leonhartX !

Tried loading it - it has a problem with "declarativeContent" in the manifest.json permissions (I think it needs more context to it) and with options_page (deprecated, will be removed).
I removed declarativeContent and options_page and the extension loaded. login token creation works, the buttons in the Functions page in firefox are there, but they aren't pressable.
created this for now https://github.com/leonhartX/gas-github/pull/97
it's probably a permission issue with the declarativeContent that needs fixing

Amazing Plugin!
Is there some alternative for Firefox?


Impossible to get back to scale = 1 with mousewheel if min-scale = 1

Hi,
I have set min-scale to 1 and max-scale to 3. I can zoom with the mouse wheel and it works well, but then when I unzoom with the mouse wheel, it can't get back precisely to the 1.0 scale (min-value), it remains a little bit bigger.
If I zoom and unzoom many times with mouse wheel I can get close to the 1.0 scale but never exactly it. Playing with the wheel-velocity helps a little and allows me to get closer ton min-scale, but does not fix the issue.
It's problematic because it leaves room for a vertical scroll even when the content is same height as the viewer.
Thank you for the great package, it's very useful. If you have a buymeacoffeelink I'd send a lil something for sure.
I have other less important issues that I might post here soon, I try to gather more informations.
Have a good day!

@jeanfrancoisgb hi, I will fix the bug in 2 days

Hi! Any update?
I can do it and do a pull request if you prefer?

Also just tested, same issue happend with pinch dezoom on a touch screen.

@jeanfrancoisgb tomorrow i am going  to fix and close

@jeanfrancoisgb fixed and published, "version": "4.2.4". Please open and describe another bug for same issue happens with pinch dezoom on a touch screen you mentioned above


Environment configuration error

  cc1: error: attribute ‘avx512fp16’ argument ‘target’ is unknown
  In file included from include/simsimd/curved.h:40,
                   from include/simsimd/simsimd.h:106,
                   from c/lib.c:54:
  include/simsimd/dot.h: In function ‘simsimd_dot_f16_sapphire’:
  include/simsimd/dot.h:1622:5: error: unknown type name ‘__m512h’; did you mean ‘__m512bh’?
   1622 |     __m512h ab_vec = _mm512_setzero_ph();
        |     ^~~~~~~
        |     __m512bh
  include/simsimd/dot.h: In function ‘simsimd_dot_f16c_sapphire’:
  include/simsimd/dot.h:1645:5: error: unknown type name ‘__m512h’; did you mean ‘__m512bh’?
   1645 |     __m512h ab_real_vec = _mm512_setzero_ph();
        |     ^~~~~~~
        |     __m512bh
  include/simsimd/dot.h:1646:5: error: unknown type name ‘__m512h’; did you mean ‘__m512bh’?

have you solved this error？

Not yet.


AlertController not show correct style for android

Currently, I am following code example and I put it on iframe
<iframe id="demo-android" src="./onboarding/?production=true&ionicplatform=android"></iframe>
import { Component } from '@angular/core';

import { AlertController } from 'ionic-angular';

@Component({
  templateUrl: 'template.html'
})
export class BasicPage {

  constructor(public alerCtrl: AlertController) { }

  doAlert() {
    let alert = this.alerCtrl.create({
      title: 'New Friend!',
      message: 'Your friend, Obi wan Kenobi, just approved your friend request!',
      buttons: ['Ok']
    });
    alert.present()
  }

}

See issue on image:

I can not find any solution for that. Please help me out
thanks

No idea with supporter this is reason that I hate to use ionic framework


After flashing Aliexpress ZB-GW04-USB EFR32MG21 to ELU0141_MG21_zb_ncp_6103_211204.gbl, I get the "Couldn't communicate with the adapter" error...

`sudo python3 Elelabs_EzspFwUtility.py flash -f ./data/EFR32MG21/ELU0141_MG21_zb_ncp_6103_211204.gbl -p /dev/ttyUSB2`
2023/01/23 12:06:08 Elelabs_EzspFwUtility:   EZSP adapter in bootloader mode detected:
2023/01/23 12:06:08 Elelabs_EzspFwUtility:   Gecko Bootloader v1.9.2
2023/01/23 12:06:08 Elelabs_EzspFwUtility:   Allready in bootloader mode. No need to restart
2023/01/23 12:06:09 Elelabs_EzspFwUtility:   Successfully restarted into X-MODEM mode! Starting upload of the new firmware... DO NOT INTERRUPT(!)
.....
.....

2023/01/23 12:06:44 Elelabs_EzspFwUtility:   Firmware upload complete
2023/01/23 12:06:44 Elelabs_EzspFwUtility:   Rebooting NCP...
2023/01/23 12:06:59 Elelabs_EzspFwUtility:   Couldn't communicate with the adapter in Zigbee (EZSP) mode, Thread (Spinel) mode or bootloader mode

I can get the device consistently into bootloader mode and use sudo python3 Elelabs_EzspFwUtility.py flash... to load any of the four firmware options, but each seems to load and then throw the "Couldn't communicate with the adapter" error. Nothing I do allows for a probe anymore, whether in bootloader nor EZSP mode...
`sudo python3 Elelabs_EzspFwUtility.py probe -b 115200 -p /dev/ttyUSB2`
2023/01/23 12:13:09 Elelabs_EzspFwUtility:   Couldn't communicate with the adapter in Zigbee (EZSP) mode, Thread (Spinel) mode or bootloader mode

BTW..
`ls -l /dev/serial/by-id`
total 0
lrwxrwxrwx 1 root root 13 Jan 23 12:05 usb-1a86_USB_Serial-if00-port0 -> ../../ttyUSB2
lrwxrwxrwx 1 root root 13 Jan 23 11:04 usb-Silicon_Labs_HubZ_Smart_Home_Controller_C1301815-if00-port0 -> ../../ttyUSB0
lrwxrwxrwx 1 root root 13 Jan 23 11:04 usb-Silicon_Labs_HubZ_Smart_Home_Controller_C1301815-if01-port0 -> ../../ttyUSB1

Also, my home assistant (and other containers) in docker are all down so there should not be any other service accessing /dev/ttyUSB2.
Is the device hosed? Any help is much appreciated, thanks!

ELU0141_MG21_zb_ncp_6103_211204.gbl firmware is only for Elelabs products based on EFR32MG21 MCU.
The problem with other vendors of the MG21 USB sticks is the hardware pinout. They are probably using the different UART TX and RX pins for communication with the USB-UART converter.
So right now your product is just not communicating properly with the your PC/RPi.
You need to put the product into the bootloader mode and flash the firmware for your particular hardware. Typically you should ask your vendor for it.
P.S. I'll use this opportunity to promote our Elelabs Zigbee USB sticks ;)
https://elelabs.com/products/elelabs-usb-adapter.html

ELU0141_MG21_zb_ncp_6103_211204.gbl firmware is only for Elelabs products based on EFR32MG21 MCU.
The problem with other vendors of the MG21 USB sticks is the hardware pinout. They are probably using the different UART TX and RX pins for communication with the USB-UART converter. So right now your product is just not communicating properly with the your PC/RPi.
You need to put the product into the bootloader mode and flash the firmware for your particular hardware. Typically you should ask your vendor for it.
P.S. I'll use this opportunity to promote our Elelabs Zigbee USB sticks ;) https://elelabs.com/products/elelabs-usb-adapter.html

Thanks, NilsBohr!  Due to your response, I was able to determine that the proper firmware for my device is located at https://github.com/xsp1989/zigbeeFirmware/blob/master/firmware/Zigbee3.0_Dongle-NoSigned/EZSP/README.md.
Now my results to the command, sudo python3 Elelabs_EzspFwUtility.py probe -p /dev/ttyUSB2
are
	2023/01/24 07:52:00 Elelabs_EzspFwUtility:   Firmware: 6.10.3-41
	2023/01/24 07:52:00 Elelabs_EzspFwUtility:   EZSP v8

If I can't get this device working for my docker home assistant installation, I'll be ordering the Elelabs USB adapter.
Much appreciated!


Update mkdocs dependency to remove lock on jinja version

What does this PR do?
This PR updates the mkdocs version to fix the bug of jinja and remove the lock of jinja version.
Motivation
Have a working doc and CI.
More
~- [ ] Added/updated tests~
~- [ ] Added/updated documentation~
Additional Notes
Follows https://github.com/mkdocs/mkdocs/pull/2800
There is no difference between assets generated by mkdocs 1.2.2 and mkdocs 1.2.4.

Declined in favor of #8891.


Provide both English names and abbreviated / symbolic names for parameter functions

Many of the functions defined in this package do not represent the English names of parameters, but rather abbreviations (grav, year_anom) or approximations of mathematical symbols (R_d, cp_v).
I think it's best practice to use ordinary written names in scientific scripts to minimize barriers to script comprehension --- but I accept that some users prefer to write code that uses abbreviations. As a compromise, is it possible to provide both verbose names and "short aliases" for key functions? Some examples are

molecular_mass_dry_air
gravitational_acceleration
heat_capacity_constant_volume

We can implement the verbose names and aliases in the Planet module, and then ask that the verbose names be overloaded by specific implementations as in
https://github.com/CliMA/CLIMAParameters.jl/blob/master/src/Planet/planet_parameters.jl
One advantage is that module code will then be self-documenting:
Planet.mean_sea_level_pressure(::AbstractEarthParameterSet) = 1.01325e5 # hPa

I am happy to submit a PR for this, though unfortunately I don't know what the ordinary names would be for all of the functions so I am unable to finish the job completely.

Greg
Do you think there are standard names from the CF conventions and related
WMO catalogs we could adopt/coopt? Those tend to be recognized widely and
used in downstream tools I think?
Chris
On Wed, Nov 4, 2020 at 09:23 Gregory L. Wagner<EMAIL_ADDRESS>wrote:

Many of the functions defined in this package do not represent the English
names of parameters, but rather abbreviations (grav, year_anom) or
approximations of mathematical symbols (R_d, cp_v).
I think it's best practice to use ordinary written names in scientific
scripts to minimize barriers to script comprehension --- but I accept that
some users prefer to write code that uses abbreviations. As a compromise,
is it possible to provide both verbose names and "short aliases" for key
functions? Some examples are

molecular_mass_dry_air
gravitational_acceleration
heat_capacity_constant_volume

We can implement the verbose names and aliases in the Planet module, and
then ask that the verbose names be overloaded by specific implementations
as in
https://github.com/CliMA/CLIMAParameters.jl/blob/master/src/Planet/planet_parameters.jl
One advantage is that module code will then be self-documenting:
Planet.mean_sea_level_pressure(::AbstractEarthParameterSet) = 1.01325e5 # N / m^2
I am happy to submit a PR for this, though unfortunately I don't know what
the ordinary names would be for all of the functions so I am unable to
finish the job completely.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/CliMA/CLIMAParameters.jl/issues/41, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA27DYE4JYMRNQDFX7IV24TSOFPVJANCNFSM4TKDSM5A
.

That seems like a fine idea. We can provide both English names (for scripts) and names that correspond to standard mathematical notation (for equations that appear deep inside the code). I'm not sure the place for abbreviations; they often seem to be ad hoc.

My 2 cents: I'm definitely a fan of self-documenting names, but I think I would prefer sticking with shorter names here. The reason being is that these parameters are not type-stable and, to reduce the number of names of variables, we often use aliases like
_grav = FT(grav(param_set))

so that we don't 2x the number of clima parameters. Following verbose names would look like
_mean_sea_level_pressure = FT(mean_sea_level_pressure(param_set))

And my hunch is that this might get ugly. We could just FT(mean_sea_level_pressure(param_set)) where needed, but I would look around CM.jl to see how that may impact some people (i.e., @trontrytel).

to reduce the number of names of variables, we often use aliases

With English names available, mathematical notation becomes an option for type-converted names:
g = convert(FT, gravitational_acceleration(earth_parameters)))

And my hunch is that this might get ugly.

If we provide both aliases and English names, then one can write _grav = FT(grav(param_set)) if they desire, or use the form above that I might prefer instead --- hopefully we can find something that works for everyone.
I do hope that we can ultimately hide type conversion concerns from users, because the proliferation of FT(*) is detrimental to the readability of scripts. Then the "2x" problem would not be a concern.

I agree with @charleskawczynski . I think having too long and descriptive names will make the code less readable.
I also thought the full descriptions are provided inside the CLIMAParameters.jl. For example for gravitational acceleration we have:
https://github.com/CliMA/CLIMAParameters.jl/blob/f779db6fa86f0bfce9de4ee31757fed1fe8b75ed/src/Planet/Planet.jl#L122-L123

Putting aside the debate about literate style versus mathematical notation, what do we think about providing English aliases in addition to the short names that are currently provided?
It's true there are docstrings in `Planet.jl'; however, these are not currently pasted in the module where numerical values of the parameters are defined, which is where you have to look if you are trying to evaluate whether the numerical values are correct, or what you want.

I'm generally in favor of clear (and often that means verbose) variable and parameter names (and way back when wrote something to that effect in the CliMA roadmap). However, readability is important. If we stay close to names we use in equations, it will be both readable and clear, i.e., I don't think we need anything more verbose for common parameters such as cp_v, R_d, g or grav (although all should have a plain language definition that is easily accessible, as they do in CLIMAParameters.jl.
That being said, I think a lot of our variable names (e.g., in the numerics) may benefit from more clarity (as evidenced by the questions just about every new user asks about what they mean).

Is it possible to define the base functions differently (or a variant with a special type) e.g.
instead of
""" Molecular weight dry air (kg/mol) """
function molmass_dryair end

in the canonical def,  do something more like along the lines of
""" Molecular weight dry air (kg/mol) """
function molmass_dryair (return (cfname="molecular_weight_dry_air", standard_units="kg/mol") )

that would make it possible to get standard names programmatically, which can be useful for REPL and for logging etc...

Thank @tapios. I just want to make sure that I head off any miscommunication. I'm not suggesting changing the names of anything, but rather adding aliases.
Let's take cp_v as an example. My suggestion is to provide
""" Heat capacity at constant volume (J / K). """
function heat_capacity_constant_volume end
const cp_v = heat_capacity_constant_volume

With this design, a user implementing a simulation on an Earth-like planet with reduced gravity might write
using CLIMAParameters.Planet
import CLIMAParameters.Planet: gravitational_acceleration

struct StrangeEarth <: AbstractEarthParameterSet end

gravitational_acceleration(::StrangeEarth) = 7.0

# Later on...
grav(StrangeEarth()) # returns 7.0

The problem of acronyms is especially bad when you cannot even Google the acronym, e.g. TSI and MSLP don't turn up relevant results. I think this makes the package less understandable and accessible.
Just wanted to add a :+1: for aliases, although I'm very much in favor of verbose names.


Default to OFF

Please add an option to default to the OFF state.
Thanks,
David

add crosshair.enabled setting (boolean), default to true

Thank you for your quick response! This is a very helpful tool.


Allow bigram as input

This PR implements #22 to skip "bigram-ifying" if an input is already a bigram by checking if the input is an array.
Used nested ternaries for the logic -- would understand if you'd prefer not having those, though.

Codecov Report

Merging #23 (8515d37) into main (8b9e821) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #23   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         2
  Lines          105       113    +8
=========================================
+ Hits           105       113    +8

Impacted Files
Coverage Δ

index.js
100.00% <100.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 8b9e821...8515d37. Read the comment docs.

released, thanks!


FreakyPinCodeControl sizing keyboardbuttons

Hello!
I´m using your "FreakyPinCodeControl" in an iOS app. I don´t know how to change the size of the keyboardbuttons and how to reduce the space between them. Can you show me an example?

Yes, I did, but ItemSpacing and KeyboardSizeRequest have no Effect?

I just tried it out and it's not working, So I will try to see if I can find some time to fix this, But I am not 100% sure how soon I will look into it!
You can either wait or just copy the control to your app and make changes to it if this is urgent

After looking into this further, I have rewritten the logic that controlled the keyboard & display of the buttons, now they will be evenly distributed based on the available space and will also fit the height available to the control, Also I have added two new properties that should let you control the HeightRequest and WidthRequest of the keyboard buttons if you are not happy with the existing logic changes and the SizeRequest property is now deprecated in favour of the new properties and will be removed in some future release.
Note: once you use these above-mentioned properties, the above-mentioned logic that decides the sizes of the buttons will get automatically disabled as it is part of a grid and the only way to enable it again would be to recreate the view.
More information here: https://github.com/FreakyAli/Maui.FreakyControls/pull/122/files

Oh yeah FYI ItemSpacing is not for the keyboard but the pin code items, you should find a new property in the next release called KeyboardSpacing

Ok  thanks, I will try this when the next nugetversion is released.

Fixed in the latest pre-release

Ok thanks a lot.
Is there also a possibilty to hide the cancel button or to set the size of this button separately? Since I can only set one text size for the keyboard, it is difficult to find a suitable size that fits the numbers and the cancel text. It would also be nice if the cancel button could have no text but an icon instead

Is there also a possibility to hide the cancel button or to set the size of this button separately?

For now no, if this was something you needed you should've mentioned this in this or a separate bug.

Since I can only set one text size for the keyboard, it is difficult to find a suitable size that fits the numbers and the cancel text.

This is intentional for consistency.

it is difficult to find a suitable size that fits the numbers and the cancel text. It would also be nice if the cancel button could have no text but an icon instead

Raise a feature request and i might look into it


src coffee main atom

src coffee main atom

This looks like an accidental PR, so I'm closing it. If this was not accidental, reopen with a proper description etc. Thanks!


[WIP]Add histogram aggregation function feature

https://github.com/apache/pinot/issues/8493

Codecov Report

Merging #8724 (9f2fa24) into master (d2396dd) will decrease coverage by 7.23%.
The diff coverage is 47.08%.

@@             Coverage Diff              @@
##             master    #8724      +/-   ##
============================================
- Coverage     69.68%   62.45%   -7.24%
+ Complexity     4577     4336     -241
============================================
  Files          1729     1720       -9
  Lines         90183    90184       +1
  Branches      13415    13450      +35
============================================
- Hits          62848    56324    -6524
- Misses        22971    29719    +6748
+ Partials       4364     4141     -223

Flag
Coverage Δ

integration1
26.96% <1.45%> (-0.14%)
:arrow_down:

integration2
25.31% <1.45%> (-0.11%)
:arrow_down:

unittests1
66.13% <47.08%> (-0.03%)
:arrow_down:

unittests2
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...ache/pinot/segment/local/customobject/AvgPair.java
74.19% <ø> (ø)

...not/segment/local/customobject/DoubleLongPair.java
100.00% <ø> (ø)

...inot/segment/local/customobject/FloatLongPair.java
100.00% <ø> (ø)

.../pinot/segment/local/customobject/IntLongPair.java
100.00% <ø> (ø)

...pinot/segment/local/customobject/LongLongPair.java
100.00% <ø> (ø)

...ot/segment/local/customobject/MinMaxRangePair.java
75.86% <ø> (ø)

...not/segment/local/customobject/QuantileDigest.java
58.70% <ø> (ø)

...inot/segment/local/customobject/SerializedHLL.java
100.00% <ø> (ø)

...t/local/customobject/SerializedQuantileDigest.java
87.50% <ø> (ø)

.../segment/local/customobject/SerializedTDigest.java
85.71% <ø> (ø)

... and 256 more

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 d2396dd...9f2fa24. Read the comment docs.


show and if do not bind correctly with table TDs

If you have a table in which some columns are shown based on some condition, both the show.bind and if.bind fail to behave properly on TD cells in particular and they also behave differently.
The following code case shows this:
http://plnkr.co/edit/NSgsbS?p=preview
Some examples where this is of value include:
Table view configurations
Toggling columns on or off based on a condition or the data type
Hiding columns based on available real estate (adaptive column models)

I added to the test (but since I had frozen it I had to fork) with a simple DIV table using display styles for table, table-row and table-cell.  I am seeing the same behavior
http://plnkr.co/edit/4dIZkZ?p=preview

Maby you should use $parent. in repeaters?
${model.issue}
${model.issue}

yes you are correct.  updating the TD tag only in the body (where there was a repeater) fixes the issue.
bit by $parent again evidently.  Hopefully for the last time.


Some tweaks

Now eldritch smite can knock prone enemies Huge or smaller.
A mismatch in prerequisite of potent spellcaster is addressed.
 (This is put here cause I just implemented it for self-use and want to share it with others. Drop this feature if you wish)

We ended up removing potent spell caster on warlocks. Please only send the smite change on another request after merging latest dev.


Add Docker Build Files

Add Docker Build Files

Looks good! Quick comment: can you add a README.md to briefly describe how to build the docker using the scripts?


Accessory not reachable

I am using HomeSpan 1.9.0 on an ESP32 (original). It is connected to my wifi network, and I can ping it's ip address without any issues. I can also see it broadcasting using an app from the App Store called Discovery, however when I try to connect to it, it will fail and state the accessory is not reachable.
`
rst:0xc (SW_CPU_RESET),boot:0x13 (SPI_FAST_FLASH_BOOT)
configsip: 0, SPIWP:0xee
clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
mode:DIO, clock div:1
load:0x3fff0030,len:1184
load:0x40078000,len:13260
load:0x40080400,len:3028
entry 0x400805e4

Welcome to HomeSpan!
Apple HomeKit for the Espressif ESP-32 WROOM and Arduino IDE

** Please ensure serial monitor is set to transmit
Message Logs:     Level 2
Status LED:       Pin 2
Device Control:   Pin 0
Sketch Version:   n/a
HomeSpan Version: 1.9.0
Arduino-ESP Ver.: 2.0.14
ESP-IDF Version:  4.4.6
ESP32 Chip:       ESP32-D0WDQ6 Rev 1 dual-core 4MB Flash
ESP32 Board:      doitESP32devkitV1
PWM Resources:    16 channels, 8 timers, max 20-bit duty resolution
Sodium Version:   1.0.12-idf  Lib 9.4
MbedTLS Version:  mbed TLS 2.28.4
Sketch Compiled:  Apr  8 2024 14:53:32
Partition:        app0
MAC Address:      78:E3:6D:18:2E:8C
Device Name:      Environmental Sensor
HTU21D Temperature - Constructor: 27.72C
HTU21D initial Temperature - end Constructor: 27.72C
HTU21D Humidity - Constructor: 66.42%
HTU21D initial Humidity - end Constructor: 66.42%
*** HomeSpan Info ***
➤ Accessory:  AID=1
➟ Service AccessoryInformation:  IID=1, UUID="3E"
⇨ Characteristic Name("Environmental Sensor"):  IID=2, UUID="23", Perms=PR
⇨ Characteristic Manufacturer("Snowball's Chance "):  IID=3, UUID="20", Perms=PR
⇨ Characteristic SerialNumber("78:E3:6D:18:2E:8C"):  IID=4, UUID="30", Perms=PR
⇨ Characteristic Model("HS Sensors"):  IID=5, UUID="21", Perms=PR
⇨ Characteristic FirmwareRevision("0.1a"):  IID=6, UUID="52", Perms=PR+EV
⇨ Characteristic Identify(1):  IID=7, UUID="14", Perms=PW
➟ Service HAPProtocolInformation:  IID=8, UUID="A2"
⇨ Characteristic Version("1.1.0"):  IID=9, UUID="37", Perms=PR
➤ Accessory:  AID=2
➟ Service AccessoryInformation:  IID=1, UUID="3E"
⇨ Characteristic Name("Temperature Sensor"):  IID=2, UUID="23", Perms=PR
⇨ Characteristic Manufacturer("Snowball's Chance"):  IID=3, UUID="20", Perms=PR
⇨ Characteristic SerialNumber("78:E3:6D:18:2E:8C"):  IID=4, UUID="30", Perms=PR
⇨ Characteristic Model("HomeSpan Temperature Sensor"):  IID=5, UUID="21", Perms=PR
⇨ Characteristic FirmwareRevision("0.1a"):  IID=6, UUID="52", Perms=PR+EV
⇨ Characteristic Identify(1):  IID=7, UUID="14", Perms=PW
➟ Service TemperatureSensor:  IID=8, UUID="8A"
⇨ Characteristic CurrentTemperature(27.7216):  IID=9, UUID="11", Perms=PR+EV, Custom-Range=[-50,100]
➤ Accessory:  AID=3
➟ Service AccessoryInformation:  IID=1, UUID="3E"
⇨ Characteristic Name("Humidity Sensor"):  IID=2, UUID="23", Perms=PR
⇨ Characteristic Manufacturer("Snowball's Chance"):  IID=3, UUID="20", Perms=PR
⇨ Characteristic SerialNumber("78:E3:6D:18:2E:8C"):  IID=4, UUID="30", Perms=PR
⇨ Characteristic Model("HomeSpan Humididty Sensor"):  IID=5, UUID="21", Perms=PR
⇨ Characteristic FirmwareRevision("0.1a"):  IID=6, UUID="52", Perms=PR+EV
⇨ Characteristic Identify(1):  IID=7, UUID="14", Perms=PW
➟ Service HumiditySensor:  IID=8, UUID="82"
⇨ Characteristic CurrentRelativeHumidity(66.4182):  IID=9, UUID="10", Perms=PR+EV, Range=[0,100]
Service                             UUID         AID  IID  Update  Loop  Button  Linked Services

AccessoryInformation                  3E           1    1     YES    NO      NO  -
HAPProtocolInformation                A2           1    8      NO    NO      NO  -
AccessoryInformation                  3E           2    1     YES    NO      NO  -
TemperatureSensor                     8A           2    8      NO   YES      NO  -
AccessoryInformation                  3E           3    1     YES    NO      NO  -
HumiditySensor                        82           3    8      NO   YES      NO  -
Configured as Bridge: NO
Database Validation:  Warnings=0, Errors=0
*** End Info ***
Generating new random Accessory ID and Long-Term Ed25519 Signature Keys...
Accessory ID:      A6:7A:52:47:99:E6                               LTPK: 1F432CE7E29CCF99EDEF8C582FD63025BDA3343F99D5825C5C2F18CCD4ABC118
No Paired Controllers
Resetting Database Hash...
Accessory configuration has changed.  Updating configuration number to 1
Environmental Sensor is READY!
Trying to connect to [Wifi Network].  Waiting 1 sec...
*** Log Level set to 2
HTU21D Temperature - Sensor Loop 27.72C
HTU21D Humidity - Sensor Loop 66.43%
WiFi Connected!  IP Address = <IP_ADDRESS>
Starting MDNS...
HostName:      HomeSpan-A67A524799E6.local:80
Display Name:  Environmental Sensor
Model Name:    HomeSpan-ESP32
Setup ID:      HSPN
Starting HAP Server on port 80 supporting 14 simultaneous HomeKit Controller Connections...
DEVICE NOT YET PAIRED -- PLEASE PAIR WITH HOMEKIT APP`

I have the same problem too((((

Any update on this - as noted above, please start all testing with a clean version Example 1.


degit fails installing

Need to install the following packages:
<EMAIL_ADDRESS>Ok to proceed? (y) y
! could not fetch remote https://github.com/chromaui/intro-storybook-react-template
! could not find commit hash for HEAD
I got it to work with

npx git-pwa clone --branch master https://github.com/chromaui/intro-storybook-react-template .

for this reason I theorize that the default branch is wrong????

@rhildred thanks for taking the time and reach out to us with your issue. Appreciate it 🙏 !
To answer your question, no, the default branch was left as master to avoid situations like this, as the degit package defaults to the master branch. I've encountered a similar issue as you're mentioning randomly, and rerunning the command usually solved it and was able to grab the template.
One thing to consider, though, is probably some local configuration you have on your environment or something related that's introducing the issue or something wrong with the package.


Type inference issue for method arguments

Please consider the following code snippet.
public class Base{
	class Derived extends Base{
	}

	public void bar(Optional<Base> o) {
	}

	public void foo() {
		bar(Optional.of(new Derived()));
	}
}

For this code the JSS is unable to find a declaration for the method call to bar because the argument type is solved as Optional<Derived> where Java actually would infer Optional<Base>. At least I think that this causes the issue.

Hi @mlangkabel
I tried your problem and couldn't get a declaration for bar too.
However, with the code below
import java.util.Optional;

public class Base{
	static class Derived extends Base{
	}

	public void bar(Optional<Base> o) {
	}

	public void foo() {
		Optional o = Optional.of(new Derived());
		bar(o);

		Derived d = new Derived();
		bar(Optional.of(d));

		bar(Optional.of(new Derived()));
	}
}

the first call of bar(o) was solved as Base.bar(java.util.Optional<Base>). The two other calls of bar couldn't be solved.

+1
I'm seeing similar issues related to type inference, but in a slightly different use-case:

class ComplexGenericArgument {

    static class Items<T> {
        public <Output extends T> Output apply(Transform<? super Items<T>, Output> t) {
            return null;
        }
    }

    static class Transforms {
        public static <I, O> Transform<I, O> of(DoFn<I, O> fn) {
            return null;
        }
    }

    static class Transform<Input, Output> {

    }

    static class DoFn<Input, Output> {

    }

    static class MyFn extends DoFn<Integer, Long> {

    }

    public void main() {
        Items<Integer> items;
        items.apply(Transforms.of(new MyFn())); // this fails to be resolved.
    }

}

java.lang.RuntimeException: Unable to calculate the type of a parameter of a method call. Method call: items.apply(Transforms.of(new MyFn())), Parameter: Transforms.of(new MyFn())
...
Caused by: java.lang.IndexOutOfBoundsException: Index: 0
	at java.util.Collections$EmptyList.get(Collections.java:4454)
	at com.github.javaparser.symbolsolver.javaparsermodel.contexts.MethodCallExprContext.matchTypeParameters(MethodCallExprContext.java:404)
	at com.github.javaparser.symbolsolver.javaparsermodel.contexts.MethodCallExprContext.resolveMethodTypeParameters(MethodCallExprContext.java:364)
	at com.github.javaparser.symbolsolver.javaparsermodel.contexts.MethodCallExprContext.solveMethodAsUsage(MethodCallExprContext.java:92)
	at com.github.javaparser.symbolsolver.javaparsermodel.JavaParserFacade.solveMethodAsUsage(JavaParserFacade.java:504)
	at com.github.javaparser.symbolsolver.javaparsermodel.TypeExtractor.visit(TypeExtractor.java:274)
	at com.github.javaparser.symbolsolver.javaparsermodel.TypeExtractor.visit(TypeExtractor.java:39)
	at com.github.javaparser.ast.expr.MethodCallExpr.accept(MethodCallExpr.java:86)
	at com.github.javaparser.symbolsolver.javaparsermodel.JavaParserFacade.getTypeConcrete(JavaParserFacade.java:394)
	at com.github.javaparser.symbolsolver.javaparsermodel.JavaParserFacade.getType(JavaParserFacade.java:286)
	at com.github.javaparser.symbolsolver.javaparsermodel.JavaParserFacade.solveArguments(JavaParserFacade.java:240)

I believe I have managed to pinpoint the problem and will have a PR shortly.
I'm afraid I can't tell for sure whether we are being bitten by the same issue.

Hi @staslev
Here is the result of my test with your code
ComplexGenericArgument.Items.apply(ComplexGenericArgument.Transform<? super ComplexGenericArgument.Items<T>, Output>)
ComplexGenericArgument.Transforms.of(ComplexGenericArgument.DoFn<I, O>)

Is this supposed to be the right solution ?
What i've changed is in MethodCallExprContext#matchTypeParameters. I added this test case
                if (actualType.asReferenceType().typeParametersValues().size() > 0) {
                    matchTypeParameters(tp, actualType.asReferenceType().typeParametersValues().get(i), matchedTypeParameters);
                }

in the 2nd else if(...). But this is just a test i've added after the first call (with the same error than you). I need to do more research to understand what the method is used for, etc.

Thanks for testing!
I think the root cause is that actualType.asReferenceType().typeParametersValues() comes up empty if it does not have type parameters which may not be ideal for instance, in the following case:
class NonGeneric extends VeryGeneric<Integer> {...

NonGeneric will state it has no type arguments, and if a method's signature demands that all the generic type parameters of its argument are fulfilled, it may try to traverse an empty list.
I've been playing around with collecting the generic params from everyone up the inheritance chain, so that such signatures happy.

Yes you're right, the error IndexOutOfBoundsException happens when typeParametersValues() is empty. Your PR fixed that issue.
Regarding your question, typeParametersValues try to match each generic type from expectedType with the one in actualType but does not consider parent's types. Thus adding the if condition does not change the method behavior (except the exception thrown).
I'm not sure i understood what you meant ^^

Regarding the first example, the fact is that we would probably need special support for generic arguments that are inferred:
bar(Optional.<Derived>of(new Derived())); // would fail
bar(Optional.of(new Derived())); // would succeed

It is like we are postponing taking a decision on the type of Optional.of(new Derived()) until we consider the context. It seems a tricky one to me.


Show something meaningful stats in the home page

Currently, the main page is showing top reward address. It's meaningful only when the consensus algorithm is a PoW-based one.

FYI


[QUESTION] Is it possible to do validation after the RCPT TO section in the SMTP session?

New question checklist

[x] I have read the Contribution Guidelines
[x] I have read the documentation
[x] I have searched for existing GitHub issues

Question

When using Truemail as my main engine for validating emails I found that every time I validate Yahoo emails the results are always valid/true. After I try to debug the problems, turn out that Yahoo accepts all email addresses in the RCPT TO section, so even if the email doesn't exist the return from the Yahoo MX server will always be OK (250).  After I continued to the DATA section in the SMTP session the Yahoo MX server just returned the mailbox not exist message.
Here is the detail:
$ swaks -s mta5.am0.yahoodns.net:25 -f<EMAIL_ADDRESS>-t<EMAIL_ADDRESS>=== Trying mta5.am0.yahoodns.net:25...
=== Connected to mta5.am0.yahoodns.net.
<-  220 mtaproxy511.free.mail.bf1.yahoo.com ESMTP ready
 -> EHLO vpndev.aktiva.co.id
<-  250-mtaproxy511.free.mail.bf1.yahoo.com
<-  250-PIPELINING
<-  250-SIZE 41943040
<-  250-8BITMIME
<-  250 STARTTLS
 -> MAIL<EMAIL_ADDRESS><-  250 sender<EMAIL_ADDRESS>ok
 -> RCPT<EMAIL_ADDRESS><-  250 recipient<EMAIL_ADDRESS>ok
 -> DATA
<-  354 go ahead
 -> Date: Tue, 30 Apr 2024 04:24:03 +0000
 -> To<EMAIL_ADDRESS> -> From<EMAIL_ADDRESS> -> Subject: test Tue, 30 Apr 2024 04:24:03 +0000
 -> Message-Id<EMAIL_ADDRESS> -> X-Mailer: swaks v20190914.0 jetmore.org/john/code/swaks/
 ->
 -> This is a test mailing
 ->
 ->
 -> .
<** 552 1 Requested mail action aborted, mailbox not found
 -> QUIT
*** Remote host closed connection unexpectedly.

So is there any workaround for this problem? I tried various validators that are available online and the result is valid.
Is the only way is just to do/continue the SMTP session until the DATA section? If it is, doesn't it will send the message to the target email if the email is valid?

Hi, @RichiMaulana! It could be... I guess for validating yahoo emails you should use another approach.


Is there a way to extract a top image from an article?

article_trafilatura = trafilatura.bare_extraction(trafilatura.fetch_url(url), include_images=True)

When I set include_images=True, I get article src, but it seems to miss the  top article image src. Is it meant to be this way or is it still not implemented or am I doing something wrong? I have tried passing different urls from different websites but still I cannot find a way how to extract that image.

Hi, trafilatura is optimized for text retrieval, if the images you're looking for are not within the main text they will not be displayed in the processed document. Can you please give me an example of a webpage where the top image is missing?

@adbar Fox example this article: https://edition.cnn.com/2021/03/08/uk/meghan-harry-oprah-interview-recap-scli-gbr-intl/index.html. Maybe I am missing something but aren't all the article photos inside the main text frame?

@ArturasDruteika Do you have other similar examples?

Hello
I have this examples
https://thepopinsider.com/news/shang-chi-trailer/
https://www.cnet.com/news/explosive-marvel-shang-chi-trailer-reveals-the-new-mandarin/

The second one now works but not this one:
https://thepopinsider.com/news/shang-chi-trailer/

Hi @adbar , please check url https://www.scmp.com/lifestyle/families/article/2160779/three-reasons-more-chinese-indonesians-are-learning-mandarin-and . When I fetch the article with include_images=True, the graphic tag have no src attribute.

Hi @phongtnit, the SCMP article you mention is a different bug, I opened another thread.
The second link above and the others mentioned earlier are now fixed in a4d0bd1dc903d79b812f897d9e7cfe07ba6550f6. @phongtnit @ArturasDruteika @AliaHamwi

Hi, I know this is an old thread, but I have several examples where it fails to fetch images:

https://openstax.org/books/anatomy-and-physiology/pages/6-1-the-functions-of-the-skeletal-system#fig-ch06_01_04
https://openstax.org/books/anatomy-and-physiology/pages/1-1-overview-of-anatomy-and-physiology

Basically, all images on that domain seem to get missed on the output XML

Hi @pranav-profjim, images are not my priority but I reopen the thread and will have a look at your cases later.

test page: https://docs.aiproxy.io/guide/quick-start
This type of link also cannot be resolved, is it possible to fix it, or can you provide suggestions for improvement?

@pofey Images are not my priority, I'm not sure how to fix it but let's keep the issue open to have a look at it.


bump dependencies

We need to bump dependencies to solve inconsistencies with SPM packages that have the same dependencies as SwiftyInsta has.

Great 💪
Do you mind updating the .podspec too?

@sbertix i just bumped the pod specs


fix: make default services inferrable

Current dependencies on/for this PR:

main

PR #140   👈

This comment was auto-generated by Graphite.

Merge Activity

Sep 27, 1:31 PM: @pevisscher merged this pull request with Graphite.


The whoami command does not exist

The readme mentions a whomai command:
https://github.com/heroku/force/blob/2ad08e292c109bf3902d9ec498bece53c6e331e4/README.md#whoami
But this command does not exist, or has been removed and someone has forgotten to remove the section in the readme.

Sorry, wrong question.


Still wrong URL "Create Index API"

Still wrong URL "Create Index API". Same issue as #33469

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

ID: 2ab1cea8-3915-f271-06ca-bf8885a80409
Version Independent ID: 3a9a5ee2-97fa-affa-7bdc-d8f26f6bfb25
Content: Tutorial: Index semi-strutured data in JSON blobs - Azure Cognitive Search
Content Source: articles/search/search-semi-structured-data.md
Service: cognitive-search
GitHub Login: @HeidiSteen
Microsoft Alias: heidist

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

@vrlb I'm seeing "Create Index (Azure Cognitive Search REST API)"
Is there a different link that should be there? In the PR logs I see that this got changed twice recently.

@vrlb @MarileeTurscak-MSFT  Sorry, I just fixed it this morning and should have circled back on this issue right away.  Thanks everyone for seeing this through.  I think it's all good now.
#please-close


fix(bulk-import): bulk import ux improvements

Resolves:

https://issues.redhat.com/browse/RHIDP-3792
https://issues.redhat.com/browse/RHIDP-3607
https://issues.redhat.com/browse/RHIDP-3798
https://issues.redhat.com/browse/RHIDP-3783
https://issues.redhat.com/browse/RHIDP-3837
https://issues.redhat.com/browse/RHIDP-3858
https://issues.redhat.com/browse/RHIDP-3888
https://issues.redhat.com/browse/RHIDP-3893

Fixes:

Removed the Repositories tab view and simply listing the added repositories
Moved the DeleteDialogContext to shared-react package. Updated the rbac UI plugin
In the Add Repositories list page show the PR contents in the sidepanel if the repo has a PR waiting for approval
Text on the educational card is now centered
The import PR description now links to <base-url>/catalog and the <base-url>/bulk-import/repositories?repository=<repo-url> which opens the sidepanel for the repository with the Pull request details on the app
Fixed the dark theme issue
updating the selected rows in the Added repositories list now triggers the API with the updated size per integration param
Fixed the issue with the drawer closing when the added repository list updates

Screenshots:
Showing an info panel with the PR link if the job is waiting for approval

Tabbed layout in the Added repositories list page is removed

Text on the educational card is now centered

The import PR description links to Software catalog <base-url>/catalog & associated import job <base-url>/bulk-import/repositories?repository=<repo-url>

Shows an alert instead of 404 error page when the user is not authorized to access the plugin (https://issues.redhat.com/browse/RHIDP-3833)

Add repositories page in dark theme on showcase app

Thanks for these improvements. I quickly tested this and it looks good overall (especially with the dark theme); But I noticed the following behavior:

When selecting more than one repository from the organizations tab, I noticed that the form with the PR details is only filled for the first repo in the list. See the screenshots below. Is that expected?

Right, this is the issue that I am trying to fix which is why I made the PR WIP

Does this PR really fix https://issues.redhat.com/browse/RHIDP-3798 (as depicted in this PR description)? It seems to me that @jerolimov's point was to make the link open the bulk import details for the selected repo, so they can edit the PR directly. But right now, the link points to the current instance Backstage catalog (which was the point of https://issues.redhat.com/browse/RHIDP-3792):

Oh I missed that, on it right away

Thank you for all these @debsmita1 !
Some quesitons:

Showing an info panel with the PR link if the job is waiting for approval
Can we make this message to match the design? without the description text?

Shows an alert instead of 404 error page when the user is not authorized to access the plugin (https://issues.redhat.com/browse/RHIDP-3833)
I guess this is fine for now, but it will be great using the [full page layout] (https://www.figma.com/design/GeVb7rc390TkxeQNnJIcBD/RHDH-Theme-(pages)?node-id=110-933&t=tedPPRSTpsge64S3-1) for this use cases in the future (probably we need another ticket).

Add repositories page in dark theme on showcase app
The text on the educational card in this screenshot doesn’t seem to line up with the illustrations

Thank you for all these @debsmita1 !
Some questions:

Showing an info panel with the PR link if the job is waiting for approval

Can we make this message to match the design? without the description text?

Fixed

Shows an alert instead of 404 error page when the user is not authorized to access the plugin (https://issues.redhat.com/browse/RHIDP-3833)

I guess this is fine for now, but it will be great using the [full page layout] (https://www.figma.com/design/GeVb7rc390TkxeQNnJIcBD/RHDH-Theme-(pages)?node-id=110-933&t=tedPPRSTpsge64S3-1) for this use cases in the future (probably we need another ticket).

I created this ticket https://issues.redhat.com/browse/RHIDP-3889 , to address the above

Thanks @debsmita1, i have one observation
The icon and text are not aligned properly

Thanks @debsmita1, i have one observation
The icon and text are not aligned properly

Fixed

getting , here there is error + some data

getting below, here there is error + some data

Indeed, errors returned by GitHub do not prevent the backend from fetching the rest of the data. To help with troubleshooting, those errors are collected and returned in the errors field. So depending on the permissions of the GH integrations, some data might be returned, but errors might be returned as well.
IMO, we should still display the data returned along with any potential errors from the errors field.

getting below, here there is error + some data

Fixed it.
Showing the error along with the data that is fetched


hands tflite checkpoint in github repo does not match tflite checkpoint installed with pip

I spent a good amount of time inferencing directly with the tflite models for palm detection and hand landmarks.
As stated in the title the tflite checkpoints for palm detection and hand landmark detection in this repo seem to be outdated.  They perform much worse when compared to the tflite checkpoints installed when I run "pip install mediapipe"
The checkpoints on this repo expect 2944 anchors, meaning the palm detection has an output of 2944 by 19, but from the pip install the checkpoint expects 896 anchors, meaning the palm detection output is 896 by 19.  Expected input image sizes also differ between the tflite checkpoints supplied in this repo and the tflite checkpoints installed by pip for both palm detection and hand landmarks.
Are they checkpoints purposefully different? If not, it would save a lot of headache to update the checkpoints in this repo.

Thanks for reporting this. It might be a temporarily out of sync issue between the github repo and the pypi package push. We will try our best to keep them in sync per release.


Not able to open file explorer even after disenabling defult explorer option

Description
Hi,
I am not able to open file explorer anymore even after the setting shows file explorer is disabled as default app. Not able to toggle the setting too. Kindly provide some temporary fix.
best,
Samarth
Steps To Reproduce
No response
Expected behavior
to be able to toggle default explorer option
Files Version
<IP_ADDRESS>
Windows Version
19042.1466
Relevant Assets/Logs

even thou here it shows that files is not the default option, it still is.

To revert to explorer you can use the UnsetFilesAsDefault.reg file in this archive.
https://raw.githubusercontent.com/files-community/files-community.github.io/main/data/SetFilesAsDefault.zip.
Please let us know if it fixes the issue.

The toggle option was not working. But a file did! thanks a ton!
btw, amazing product! Had been looking for something like this for a while!


ISPN-13768 IracTombstoneCleanupTest fails randomly

https://issues.redhat.com/browse/ISPN-13768
Some changes on the test transport, so we can mark a command as forbidden and wait for completion. I was not able to replicate the failure though.

CI is going crazy with the changes in the test. The executions either times out or have multiple failures. Going to investigate.

TBH, I'm not sure if your PR is actually doing anything.
I wasn't able to reproduce the error locally anymore and I haven't seen the test failing in CI since ever to try to refresh my memory of what is going on.

TBH, I'm not sure if your PR is actually doing anything. I wasn't able to reproduce the error locally anymore and I haven't seen the test failing in CI since ever to try to refresh my memory of what is going on.

Yeah, TBH I am ok if we don't merge this. I was not able to reproduce the behavior described in the JIRA, basically, the only additional thing here would be forbidden commands, but that was covered in other ways before.

Ok! I'll close the PR and I'm going to close the JIRA since the issue does not happen for a long time.
I'm afraid this PR is not doing anything because you are sending the commands before checking the forbidden set; nothing prevents the command to be delivered in the next test method.


Pakfile Explorer Bugs/Suggestions

File size displays as 0.0 KB when importing new files

Delete doesn't work when multiple files are selected

Null bytes are displaying in the VMT editor (they don't exist in the original file)

Deleting all files in a folder also deletes the folder. It would be nice for the folder to be retained so you can replace files by removing all of them and then importing new ones

Lumper crashed when I moved multiple vtfs/vmts into an empty folder

Adding another thing that we just discussed in voice, "Replace"/"Export" top-left buttons don't make it clear they completely export/replace everything. Previously it was "Export Contents"/"Replace Contents", I think BIRD changed it to lower the width of that top bar. We should change it to "Export All" / "Replace All".


adding "snap to pixel" feature

Would you consider adding "snap to pixel" feature to both of the renderers?
After making a little demo yesterday, I found that my sprite blurs when it is not positioned at
integer points.
The sub-pixel bluring effect looks the same with one in the article
http://seb.ly/2011/02/html5-canvas-sprite-optimisation/

I'd appreciate support for this in Pixi itself (currently patching it in). How do you envision arranging it in user-land? It's doable if all display objects are in a single, pixel-aligned container, but it becomes highly impractical when nested objects are used, where the misalignment may occur at any level in the tree.

The short answer is take care not to assign floating values to your positions, if something is a delta change round/floor/ceil it before assigning.
You're right, this isn't an easy problem to solve, but the solution isn't to make objects everywhere have an expensive math operation to catch mistakes. Canvas renderer currently has a roundPixels option that increases rendering speed by flooring positions as images are drawn, but it doesn't work like that on WebGL. We would have to round/floor/ceil positions in js which would negatively effect the renderer's performance.
There is also the issue that some applications want to round positions as the move, some want to floor, and some want to ceil, some want to let the browser interpolate normally. It depends on the application. If the answer is "it depends on the application" it probably doesn't belong in the core. I'm not convinced that this belongs in pixi core because it means adding a branch to every update transform call at best, or many branches at worst. I'm still of the opinion that you should just round/floor/ceil your values you assign to positions as you do it in your application, if it makes sense to do so in your application.

Take for example a spaceship moving slowly around a space. You don't want to round its x and y, because then you can't achieve the slow motion. The spaceship has lots of elements, rotating turrets, etc. Also, there is zooming and rotation of the view going on.
You only want the leaves in the display tree, the actual sprites, to snap to the pixel grid. The only practical place to apply this would be somewhere between updateTransform and the actual rendering.
I fully sympathize with not wanting to add code to fix a very specific problem in such a general location. I'm all for separation of concerns. (Why does BaseTexture deal with loading/caching images from urls? :-] )
So would you consider adding an optional callback inside/after DisplayObject.transform? It would receive the updated matrix, allowing it to modify it further, e.g. applying rounding/flooring/whatever to the translation values.
If you're worried about branching, you might implement it somewhat like this:
DisplayObject.prototype.setTransformCallback(cb) {
	if (cb != null) {
		this.updateTransform = function() {
			DisplayObject.prototype.updateTransform.call(this);
			cb(this.worldTransform);
		};
	} else {
		delete this.updateTransform; // revert to using the prototype version
	}
}

(Why does BaseTexture deal with loading/caching images from urls? :-] )

Historical and usability reasons, it was written like that a long time ago before we had a loader and unfortunately people started using it and expect it to be that way :( I hate it.

So would you consider adding an optional callback inside/after DisplayObject.transform? It would receive the updated matrix, allowing it to modify it further, e.g. applying rounding/flooring/whatever to the translation values.

I have thought so long and hard about this one. The challenge is making it happen in a way that is performant, and I haven't found a method with low enough overhead to make it acceptable, yet.

If you're worried about branching, you might implement it somewhat like this:

Well we definitely can't do that since that will change the hidden class of the object. But I understand what you are going for. I'll try to think about this problem more and see if we can't get a good solution.

It would change the hidden class, but only for users who explicitly want to have that callback and are ready to take the performance hit.
Also, if the new updateTransform were not to be a closure but a function defined once (with the callback stored as a property), it would add only one hidden class.
Anyway, I've looked around the source a bit and found things like displayObjectUpdateTransform and _originalUpdateTransform. I'm sure you're able to find a way to work this in :)

@bartvanheukelom Yeah it is possible, those aren't adding/removing properties though just chaning them. And displayObjectUpdateTransform is just a shortcut to calling up the prototype chain to reduce the call stack in hot methods.
In other words, those examples add speed but yours reduces it :P let me think on this a bit.

Also I noticed that PIXI.Text gets blurry when not using roundPixels. It's very difficult to not use floating numbers on the final position since the anchor/pivot usually are float values (0.5 for center), so this is not on the user control anymore - using integers on x/y will not ensure that the evaluated position is actually an integer.
Maybe just PIXI.Text should round it's position in this case, and not the entire core.

Hello! With v3 you can snap to using renderer.roundPixels = true

@GoodBoyDigital This issue still exists when rotation/angle is used unfortunately. Can the same algorithm that is used by CSS3 rotateZ, etc, be used here?

@skaraman im not sure i follow i'm afraid? If you provide an example of the issue and what needs solving that would be ace. Thanks!

@GoodBoyDigital actually it was to do with retina resolution - http://www.html5gamedevs.com/topic/20776-pixel-percision-on-rotation-for-renderer/

I'm trying to avoid this pixel snapping functionality. I want to have a lot of slow moving sprites and the motion looks awful as they snap to each pixel. I've tried renderer.roundPixels = false but it has no effect (Pixi.js 4.5.1 - Canvas).
Is there any way to fix this?
I can easily repro it in the "tinting" demo with these line modifications:
//dude.scale.set(0.8 + Math.random() * 0.3); // Comment out this line
//dude.rotation = -dude.direction - Math.PI / 2; // Comment out this line
dude.speed = 0.1 //2 + Math.random() * 2; // Make them move slowly

Hi all, any ideas as to why renderer.roundPixels = false seems to have no effect in my case above? Thanks for any suggestions.

RoundPixels is off by default. How big coordinates are you working with? May be its precision problem.


Optimize code style for Project 01

you about identifer ?

Yes please, leave cell identifier there, we do not need to hard code it to specific place.


Entry point not found error with texteditor module

I am running windows 10 and have installed the texteditor module with 'pip install texteditor'
when importing via 'import texteditor' i received the attached error message.
I have uninstalled and reinstalled but with the same results.

after uninstalling multiple times and reinstalling it finally worked.  But.. new issue.  will be logging it


The "Explain" block attempts a POST request that returns a 405 (method not allowed) status when attempting to explain code

When using the "Explain" block, which is supposed to explain code using ML, it returns a blank response. The reason for this is that it's trying to make POST request that returns a 405 response (method not allowed).
The endpoint it is trying to reach is "https://blocks-sandbox.githubnext.com/api/explain", one possible cause is that you have to remove the 'sandbox' part from it.
Some other reasons why this might be happening might be:

The server does not accept POST requests to that URL or there’s a redirect involved converting the request to a GET.
Your browser is making a preflight request, which uses the OPTIONS HTTP method. This is to check whether the server will allow the POST request (the 405 status code is sent in the response to the OPTIONS request, not the POST request).
The POST method might not be allowed on which you are sending the request and the request is simply denying you to make any POST request by sending 405 error response

See the screenshots below for more details.
Screenshot of the POST request that's not being allowed

Example 'blank' response returned by the explanation block

The 'Explain' block

Relocated to the right repo (githubnext/blocks-examples)