Sketch not working without Serial Monitor? (IDFGH-7568)

Environment

Development Kit:      [ESP32-DevKitS | ESP32WROOM32E ]

Kit version : [not sure, bought month ago, both are black/gold in color]
Module or chip used:  [ESP32-WROOM-32 : ESP32-D0WD-V3]
Operating System:     [Windows]
Using an IDE?: [Yes : Arduino IDE 1.8.19]
Power Supply:         [USB from PC (500mA) | External 5V from lab power supply (5A max)| PowerBank(3A max)]

Problem Description
I do apologise if this is incorrect place to ask such question - I stumbled upon a similar issues here:
Hang on post-reset boot? (IDFGH-3516) #59, thus decided to give it a try and describe what happens with my small IoT project the best I can.
Usage:
I am using a "module" version of ESP32(just the small square with antennae) and program it using a "fixture" dev board (click-in) if I am using the names correctly.
WiFi enabled using ElegantOTA - for over-the-air sketch uploads.
Bluetooth enabled using ESP32 BLE Keyboard library - for HID BLE Keyboard functionality.
Serial - for Serial Monitors & USB uploads when connected to PC, when module is soldered to custom PCB this Serial is not connected
Serial1 - for incoming data transmissions (simple strings from External Device), only Rx used
Serial2 - for incoming Range measurements data from Range Finder sensor type, only Rx used
Device is supposed to turn on, connect to WiFi(for OTA), advertise itself as BT HID Keyboard, listen to input from RangeFinder, if range is within threshold, then trigger External Device and listen for incoming data, if data received send it as BT HID Keyboard keys.
The problem arises when moving the programmed and otherwise working fine "module" to a custom piece of PCB.
GPIO:

NC - not connected
This is used across ESP32-DevKitS and custom PCB.
All backplate pins connected to GND.
Anything without red arrow and label is not connected.
The Issue:
When using ESP32-DevKitS click-in dev board connected through USB to a PC with open Arduino IDE and Serial Monitor, the device works exactly as expected and described above at Usage section.
However, when "clicking-out" of ESP32-DevKitS and soldering the module to the custom PCB, the device seems to run only once. At first power on, I can briefly see it connect to WiFi (I have IP from DHCP and can open it in a browser), I can briefly see it advertise itself as Bluetooth Keyboard and it even briefly reacts to RangeFinder (toggles ExternalDevice on and off).
After reboot (EN button or power cycle) the module seems to never work again (does not connect to WiFi, does not advertise BT, does not react to Range Finder, does not trigger External Device) unless desoldered, plugged back in and then back out of ESP32-DevKitS "click-in" dev board (and connecting it to PC via USB).

[Does not work] When using ESP32WROOM32E Module on custom PCB connected to PowerBank through PCB's power supply
[Does not work] When using ESP32WROOM32E Module on custom PCB connected to External Power Source (lab power suppy) through PCB's power supply
[Works as supposed] When using ESP32-DevKitS connected via USB to PC with Arduino IDE and Serial Monitor connected
[Works as supposed] When using ESP32-DevKitS connected via USB to External Power Source (lab power supply)
[Works as supposed] When using ESP32-DevKitS connected via USB to regular PowerBank
and best of all:
[Works as supposed] When using empty ESP32-DevKitS connected via USB to PC with Arduino IDE and Serial Monitor connected and custom PCB with soldered-in ESP32 "module", PCB is powered by PowerBank and both ESP32-DevKitS and PCB are connected to each other's GPIO pins and common-grounded.
For example: PowerBank -> custom PCB (with ESP32 soldered in) -> jumper GPIO cables + common GND -> ESP32-DevKitS (empty) -> PC (USB)

I do realize it might seem like the custom PCB issue, but I have tested for continuity and appropriate voltages all over it and it seem to be in accordance to all the datasheets. It was also a paid custom design from a professional company - I don't think I would be able to design it myself.
The biggest indicator "that it is not the custom PCB at fault here" is the fact the PCB was working fine [as supposed] up untill I added the WiFi functionality together with OTA.
I feel like I am missing something obvious here...
Maybe some obscure requirement when using both WiFi and Bluetooth?
Maybe some error with configuration/profile (please see screenshot below)?
I did find a thread, at Arduino Forums, claiming that I cannot do "Serial.begin()" if there is no Serial Monitor connected because that's blocking and will hang my project because... no Serial Monitor is connected - is that even ture? I did try and remove all "Serial" calls with no change in results, though.
Any insight would be highly appreciated.
Arduino IDE Configuration:

Code to reproduce this issue
my_iot.ino
Please ignore line 95: millis_start = millis(); , it was supposed to be removed as any other millis();

Does your PCB expose the UART Rx & Tx Lines?
If so, It would be useful to look at the logs while the device is running in the PCB.

Does your PCB expose the UART Rx & Tx Lines?
If so, It would be useful to look at the logs while the device is running in the PCB.

Do you mean those:

If yes, then no - the PCB does not expose those GPIOs, unfortunately :(
I could try to solder some thin wires to the module and connect to some TTL/USB converter, though?

If this doesn't fix your problem I recommend commenting out anything that says serial


Branch status should automatically updated in status bar.

When using the terminal for checkout branch in remote ssh development, checking out branch does not update bottom left side branch name. I think it should automatically be synced and updated in the status bar also if I do it by the terminal on visual studio code.

As I noticed, I check out to master branch from another branch, but it was showing another branch in the status bar, not another branch. But that should be synced and automatically updated in remote development.

IMHO, it would be more user-friendly.

This https://github.com/microsoft/vscode/commit/13d0dad699bfc4e4529ece16dd577be98945fce1 should've fixed it. Please try it again tomorrow on Insiders and let me know if it works for you.

@joaomoreno let me give you some more information here.
I just installed the latest updates on Insiders and checked. but that the feature is not there.
Let me give you some more specification:
As we connect to the remote server for development using ssh extension. In that case, the terminal we use on insiders also shows the directory and path of the ssh server.
So when I try to switch the branch from the terminal in git, like ex: I am in master branch. and I want to move into another branch say issue-branch-me. But as I am doing it from the terminal, and not from the status bar.
So here the case happens that I am in the issue-branch-me branch but in the status bar it shows the master branch and not updated to the branch I moved in.
IMHO, that should be tracked and updated in the status bar also. because for me it becomes quite problematic.
Hope this helps to understand more. Thank you. :)

@joaomoreno thank you very much... this is working. I am closing this issue. Thanks so much


Applied necessary text changes

Description of changes

Changed the text content present in src/scenes/home/signup/signup.js to match issue requirement
Issue Resolved

Fixes #868

@jjhampton I have made the fixes as suggested. Kindly have a look

@jjhampton I have fixed the pluralization as well.


change timestamp to UTC instead of local time

The InfluxDB API always assumes the timestamp to be UTC.
Setting the timestamp to local time can lead to some bad hiccups.
Especially leads to problems with Grafana not displaying the timestamps correctly.

Closes #4


Note the default GC latency mode

Fixes #9739 .

@tdykstra Could you take a look since Bill is out?

@tdykstra Thanks for the review.


Pagination should have show all.

If I have pagination set to [5,25,50] and I have 23 rows, I am only able to select 5 per page.
I should be able to select 25 to show the entire collection.

I see your point, I think could be a good improvement. I would like hear more what the community think about that.


angular 1.5

Will support be available for angular 1.5?
Unfortunately, Components cannot be documented
Any workarounds?

I have same issue.

it is actually possible:
dunny-item-list.component.js:
/**
 * @name DummyItemListComponent
 * @memberOf DummyItemList
 * @ngdoc component
 */
export const dummyItemListComponent = {
    templateUrl: dummyItemListTemplate,
    controller: DummyItemListController,
};

thank you


ClinGen gene-based retinopathy terms

ClinGen gene-based retinopathy terms:
Please note that the majority of the suggested terms are synonyms of an existing term / terms that follow a dyadic gene-related syndrome format. One of the suggested terms "TSPAN12-related exudative vitreoretinopathy" is just a slight rephrasing of an existing gene-related Mondo terms ("TSPAN12-related vitreoretinopathy (MONDO:0100484)").
Thank you Dr. Vasilevsky for reviewing these suggested terms when you have time!
The ClinGen Retina Gene Curation Expert Panel
Batch_TermRequests_ClinGenRetinaGCEP_08152023.xlsx

Hi @wchankey, thanks for posting your request. We'll review this as soon as we can.

Spreadsheet is here: https://docs.google.com/spreadsheets/d/18eBPSIPhxbtnkh7oO6NfNS44IPGDFQ6S/edit#gid=1389946864

@wchankey we have a few questions about your spreadsheet.
I see a lot of new terms being requested with only 1 child (e.g. "PDE6A-related retinopathy" has only "retinitis pigmentosa 43"), which makes me wonder whether these terms are the same or there are other "PDE6A-related retinopathies". If so, we should make sure they are in Mondo.
Could we schedule a meeting to discuss the spreadsheet? Thank you in advance!
(my email is sabrina(at)tislab(dot)org)

Hi Sabrina,
Thank you for writing back with these questions and for the good idea to meet about the spreadsheet of ClinGen gene-based retinopathy terms (Issue #6562). You’re raising a good point about child terms for PDE6A-related retinopathies (for example), and I would similarly benefit from the chance to ask your advice in a few instances about how you’d recommend structuring the parent-child term relationships.
For the PDE6A-related retinopathy example, the clinicians I work with were not aware of a particular alternative child term that PDE6A-related patients were receiving as a diagnosis besides “retinitis pigmentosa 43”. However, they noted that there is a broad spectrum of disease severity (discussed for example in https://pubmed.ncbi.nlm.nih.gov/35533076/) where the combination of mild and severe cases would be better described as “retinopathy” rather than “retinitis pigmentosa”. As a result, they were thinking to ask Mondo to consider creating a new parent term that was more inclusive “PDE6A-related retinopathy” while preserving the existing “retinitis pigmentosa 43” term as a still-valid child term.
Anyways, I’ll put some upcoming times below when I’ll be available to meet by Zoom.
Thursday August 24th 3:30pm – 6pm (EST U.S.)
Friday August 25th 11:30am – 6pm (EST U.S.)
Monday August 28th 12pm – 6pm (EST U.S.)
Tuesday August 29th 9am – 6pm (EST U.S.)
Wednesday August 30th  9am – 6pm (EST U.S.)
Thank you for your patience (since this is my first submission using the spreadsheet) and for checking your schedule for any compatible times!
William
From: Sabrina Toro @.>
Date: Wednesday, August 23, 2023 at 10:45 AM
To: monarch-initiative/mondo @.>
Cc: Hankey, William Charles IV @.>, Mention @.>
Subject: Re: [monarch-initiative/mondo] ClinGen gene-based retinopathy terms (Issue #6562)
@wchankeyhttps://github.com/wchankey we have a few questions about your spreadsheet.
I see a lot of new terms being requested with only 1 child (e.g. "PDE6A-related retinopathy" has only "retinitis pigmentosa 43"), which makes me wonder whether these terms are the same or there are other "PDE6A-related retinopathies". If so, we should make sure they are in Mondo.
Could we schedule a meeting to discuss the spreadsheet? Thank you in advance!
(my email is sabrina(at)tislab(dot)org)
—
Reply to this email directly, view it on GitHubhttps://github.com/monarch-initiative/mondo/issues/6562#issuecomment-1690098282, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AZ4FRJN7ZVKGYTHLVKUU7UTXWYJQNANCNFSM6AAAAAA3RRKRDY.
You are receiving this because you were mentioned.Message ID: @.***>

From email:
For the recommended names “GPR143-related foveal hypoplasia with or without albinism” and “LRP5-related exudative vitreoretinopathy with or without osteoporosis”, is it possible to move the “with or without” and everything that follows it out of the name and into the definition? Or is there a reason to keep it in the name? In the LRP5 example, the group wanted to include that these may be syndromic or non-syndromic, but did not have a convenient term like “ciliopathy”. We were concerned about not being inclusive of the non-ocular disease that would be important to patients and providers. We had gone through this with the ClinGen disease naming committee, and with/without seemed to be the best when we couldn’t come up with a disease family term. The group members saw your point and one of the chairs indicated that they would be willing to let this one go if you recommend it. Another group member recommended keeping it in “LRP5-related exudative vitreoretinopathy with or without osteoporosis”, but removing it from the other example to give “GPR143-related foveal hypoplasia”?
For the 16 recommended names that have been proposed as a parent term with a single child term, (“PDE6A-related retinopathy”, “CACNA2D4-related retinopathy”, “ELOVL4-related maculopathy”, “MAK-related retinopathy”, “TSPAN12-related exudative vitreoretinopathy”, “KIZ-related retinopathy”, “TOPORS-related retinopathy”, “PRPF8-related retinopathy”, “RD3-related retinopathy”, “BBS9-related ciliopathy”, “BBS10-related ciliopathy”, “dominant BEST1-related retinopathy”, “recessive BEST1-related retinopathy”, “autosomal dominant BEST1-related vitreoretinochoroidopathy”, “IMPG2-related recessive retinopathy”, and “IMPG2-related dominant retinopathy”), why not consolidate the parent term and the child term into a synonym? The group members replied that they still prefer the parent-child setup because the gene-first dyadic term captures the disease spectrum better than the usually too-specific OMIM term. So in that sense they are not synonymous. We also decided not to replace or rename the original term because these have been used by clinicians and patients for so long. Keeping as parent-child also allows for future clinical disease terms to be added, so it future-proofs the proposed structure. You had recommended that if the group feels this way, it might be a good idea to add a comment such as; “we have made the decision for now to keep the parent term separate from the parent term (with a date) for this reason, but it’s possible in the future that this could be merged.”
For the recommended name “autosomal dominant BEST1-related vitreoretinochoroidopathy”, are there other examples of vitreoretinochoroidopathy that are autosomal recessive? Or is it always autosomal dominant, so we can remove the mode of inheritance from the name? “Autosomal dominant vitreoretinochoroidopathy” is an existing term that has been published in relation to BEST1 (PMID: 18611979, PMID: 36712704), so the group was proposing to preserve the existing term in a dyadic format that starts with “BEST1-related“. The group has also been trying to maintain consistency with their previous convention of “gene-related dominant/recessive disease” (for example “RP1-related dominant retinopathy”) when a gene has multiple inheritance patterns. After reconsideration, the group members recommended reorganizing these 3 terms to “BEST1-related dominant vitreoretinochoroidopathy” (with “autosomal dominant vitreoretinochoroidopathy” as a synonym), “BEST1-related dominant retinopathy” (with “Best disease” and “vitelliform macular dystrophy 2” as synonyms), and “BEST1-related recessive retinopathy” (with “autosomal recessive Bestrophinopathy” as a synonym).

New terms were created:

MONDO ID
term labels

MONDO:0700224
PDE6A-related retinopathy

MONDO:0700227
ELOVL4-related maculopathy

MONDO:0700228
LRP5-related exudative vitreoretinopathy

MONDO:0700229
MAK-related retinopathy

MONDO:0700230
GPR143-related foveal hypoplasia

MONDO:0700231
TSPAN12-related exudative vitreoretinopathy

MONDO:0700232
KIZ-related retinopathy

MONDO:0700233
TOPORS-related retinopathy

MONDO:0700234
PRPF8-related retinopathy

MONDO:0700235
RD3-related retinopathy

MONDO:0700236
BBS9-related ciliopathy

MONDO:0700237
BBS10-related ciliopathy

MONDO:0700238
BEST1-related dominant retinopathy

MONDO:0700239
BEST1-related recessive retinopathy

MONDO:0700240
BEST1-related vitreoretinochoroidopathy

MONDO:0700241
IMPG2-related recessive retinopathy

MONDO:0700242
IMPG2-related dominant retinopathy

MONDO:0700243
CACNA1F-related retinopathy

MONDO:0700244
CACNA2D4-related retinopathy

Note that disease names reported in the spreadsheet were added as synonyms and tagged as "ClinGen preferred"
@wchankey These should be available in the next Mondo release, early January.
Thank you for your patience.

Thank you Sabrina for sorting through these complicated requests and creating the most appropriate terms! I’m grateful for your time and the group will appreciate your thoughtful approach to determining the correct terminology. Wishing you excellent holidays ahead,
William
From: Sabrina Toro @.>
Date: Wednesday, December 20, 2023 at 8:05 PM
To: monarch-initiative/mondo @.>
Cc: Hankey, William Charles IV @.>, Mention @.>
Subject: Re: [monarch-initiative/mondo] ClinGen gene-based retinopathy terms (Issue #6562)
You don't often get email from @.*** Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification
New terms were created:
MONDO ID
term labels
MONDO:0700224
PDE6A-related retinopathy
MONDO:0700227
ELOVL4-related maculopathy
MONDO:0700228
LRP5-related exudative vitreoretinopathy
MONDO:0700229
MAK-related retinopathy
MONDO:0700230
GPR143-related foveal hypoplasia
MONDO:0700231
TSPAN12-related exudative vitreoretinopathy
MONDO:0700232
KIZ-related retinopathy
MONDO:0700233
TOPORS-related retinopathy
MONDO:0700234
PRPF8-related retinopathy
MONDO:0700235
RD3-related retinopathy
MONDO:0700236
BBS9-related ciliopathy
MONDO:0700237
BBS10-related ciliopathy
MONDO:0700238
BEST1-related dominant retinopathy
MONDO:0700239
BEST1-related recessive retinopathy
MONDO:0700240
BEST1-related vitreoretinochoroidopathy
MONDO:0700241
IMPG2-related recessive retinopathy
MONDO:0700242
IMPG2-related dominant retinopathy
MONDO:0700243
CACNA1F-related retinopathy
MONDO:0700244
CACNA2D4-related retinopathy
Note that disease names reported in the spreadsheet were added as synonyms and tagged as "ClinGen preferred"
@wchankeyhttps://github.com/wchankey These should be available in the next Mondo release, early January.
Thank you for your patience.
—
Reply to this email directly, view it on GitHubhttps://github.com/monarch-initiative/mondo/issues/6562#issuecomment-1865335564, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AZ4FRJJMNZY5FTXHARTKXELYKODMZAVCNFSM6AAAAAA3RRKRD2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRVGMZTKNJWGQ.
You are receiving this because you were mentioned.Message ID: @.***>


WebGLState::push fixes

Hi. It looks like WebGLState::push() has bugs. It makes sense only after next changes.

@ymiroshnyk, thanks for pointing this out. Would you mind putting together pull-request for these changes?

Yeah, those bugs. I didnt find any use of push in pixi. Is that ok?

@bigtimebuddy, yeh, when I figure out what is pull-request :) Sorry, I just started to use github.

Maybe @ivanpopelyshev will be a chap and help with the pull-request?
Here's Github's page about pull-requests (we also call them PRs):
https://help.github.com/articles/about-pull-requests/

@ymiroshnyk First you have to learn how to work with forks. Then you make your branch based on "dev", commit to it. Then you use "pull request" button in your forked repo.

@ivanpopelyshev Ok. Thanks. Will do that today.

@ymiroshnyk I can make PR if you don't want to, its a couple of minutes for me

@ivanpopelyshev ok. I made it.

@ymiroshnyk out of curiosity what effect did this have? Could this have caused "oops webgl have stopped working" on predominantly samsung devices? We had to revert to version 3 of pixi over the last few days because of quite a number of people sending us that bug report. As always with the "oops" error it's almost impossible to pinpoint what went wrong.

@cvissie As I see in the pixi code, this function is not used at all. And I doubt that much people use it in their projects.
I write plugin for PIXI for NeutrinoParticles, so I need it to inject into pixi rendering pipeline and, of course, save and restore GL states.

@cvissie And using of pop() after push() cause 100% crash on any device/browser.

@ymiroshnyk Would have been nice to have met you sooner :) That looks awesome mate.

Hey guys! I actually made some big changes to state in the next update:
https://github.com/pixijs/pixi.js/pull/3304
Should be a much nicer way to deal with state!

Looks like #3652 was merged, so closing this issue!


Does not compile (neither beta nor nightly)

Compiling the complex example fails. I know, there are many problems with dependencies as well, but I this project hadn't had any activity for some time now. I hope rustless is still alive...

@LukasKalbertodt I'm waiting for https://github.com/chris-morgan/mopa/issues/1, but it looks abandoned. I think I have to refactor my projects to remove the dependency, it will take some time.

Removing dependencies sounds great in this current unstable Rust world... Thanks for working on it!

it seems now the issue you were linking on mopa is fixed, is there's still some other things to fix ? and is there any way to give a hand ?  because i think if it can compile with the beta of rust, then it will be a great step forward to make it "near-production" ready.

As far as I can tell, these are currently some issues:

regex macros are used in rustless or some dependencies. Compiler extensions are unstable and not allowed in beta, therefore regex won't compile. I guess one needs to remove all uses of those macros.
phf_macros is used in rustless or deps. Same as above.
jsonway won't compile, but it seems like you already fixed that

yes actually after my message i suddenly got some free time so i started digging into that
interesting enough regex compile when i compile it alone (with beta2)

Yep, I already noticed that. Without using it's macros (I guess without #![plugin(regex)]), it works just fine.
It would be really great if rustless is compiling again...

I've started to try to make valico compile (I'm going to eat something)
I haven't touched rust for a long time (6month) and I forgot , how can I tell valico to tell my patched version of jsonway ?

@LukasKalbertodt 	@allan-simon 	thanks for your work! Right now there are some troubles with deps such as iron/body-parser and some others. They must be fixed to compile rustless. I'm working on it.

By the way: Are you ( @s-panferov ) targeting beta or nightly or both? I think beta is more important...
And if you could post a list of broken deps, we could try to fix some of them...
PS @allan-simon : I wrote an email to your address on your github profile... If you haven't seen yet.

@LukasKalbertodt right now I am trying to make it work on nightly because some deps now works only on nightly. Rustless itself is very conservative and doesn't use any unstable API, so after it will be fixed to work on nightly, we can adopt it for beta.

@allan-simon valico and jsonway has been already fixed on nightly. Now https://github.com/iron/body-parser/pull/55 is the blocker.

@s-panferov body-parser seems to be fixed now. Whats the status now?

Fixed

Nice!
However, "Fixed" may be misleading, because it still doesn't compile with beta. Maybe we should open another issue for beta. It should compile with beta before 1.0 is released I think...

yes good idea to have a ticket for beta.


FUNCTION_SETS_GLOBAL_VARS does not understand context

The global vars check creates a lot of false positives because of how simple it is. It seems to ignore almost all context. Consider this example:
(
  a='a'
)

bar() (
  b='b'
)

c='c' true

local arr=(
  d='d'
)

e='e'; export e

python - <<'EOF'
f='f'
print(f)
EOF

All of those variable-like patterns should be fine ('d' and 'f' are not even variables), but zsh-sweep complains about all of them. 'f' is the most problematic, because it's not always practical to add a shell-style comment inside of a heredoc in order to suppress it. Another problem is that zsh-sweep complains about implicitly global variables being set outside of functions, even in --script mode, where there is no practical difference between local and global AFAIK.
Notice: running check: CHECK_FUNCTION_SETS_GLOBAL_VARS_PRE…
Notice: running check: CHECK_FUNCTION_SETS_GLOBAL_VARS_POST…
[ZSweep][zsfilt:184]: Error: Error near line #2 ↔  a='a':
[ZSweep][zsfilt:184]: Error: Error near line #6 ↔  b='b':
[ZSweep][zsfilt:184]: Error: Error near line #9 ↔ c='c' true:
[ZSweep][zsfilt:184]: Error: Error near line #12 ↔  d='d':
[ZSweep][zsfilt:184]: Error: Error near line #15 ↔ e='e'; export e:
[ZSweep][zsfilt:184]: Error: Error near line #18 ↔ f='f':
[ZSweep][zscan:125]: Warning: A global variable has been created without declaration, by assignment. Add a line like local VAR or typeset -g VAR so that the variable is defined before use.

Less concerning are the false negatives like this, which zsh-sweep does not see as variables:
{ a='a' }

foo() { b='b' }

(
  a='a'
)

I think that a should be reported here. Was your thought that var created in subshell isn't a global? I think that it's good to expect a typeset in '(…)`, for readability.

I'm reading the error code as "a global was implicitly created inside of a function", and I see no function in that example. I suppose if you go by the description it might be referring to any undeclared variable, which seems like a purely stylistic note and not a diagnostic, since typeset can be used before or after a value is assigned to a variable. In that case, the error code should probably be changed to make its meaning clearer.


lqtDemo

Problem encountered on https://dotnet.microsoft.com/learn/web/aspnet-hello-world-tutorial/install
Operating System: windows
Provide details about the problem you are experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.

lqtdemo

Unclear what kind of problem you are hitting. Closing.
Feel free to reopen with additional info. Thanks!


The rotation is invalid

Here's my code, I set the rotation to the scene, but it didn't work.
 const threeScene = new THREE.Scene();
  const camera = new THREE.PerspectiveCamera(
    75,
    window.innerWidth / window.innerHeight,
    0.1,
    1000
  );
  const xColor = 0xff0000; // 红色
  const yColor = 0x00ff00; // 绿色
  const zColor = 0x0000ff; // 蓝色

  const lineWidth = 10; // 线条宽度

  const xMaterial = new THREE.LineBasicMaterial({
    color: xColor,
    linewidth: lineWidth,
  });
  const yMaterial = new THREE.LineBasicMaterial({
    color: yColor,
    linewidth: lineWidth,
  });
  const zMaterial = new THREE.LineBasicMaterial({
    color: zColor,
    linewidth: lineWidth,
  });

  const xGeometry = new THREE.BufferGeometry().setFromPoints([
    new THREE.Vector3(0, 0, 0),
    new THREE.Vector3(50, 0, 0),
  ]);
  const xAxis = new THREE.LineSegments(xGeometry, xMaterial);

  const yGeometry = new THREE.BufferGeometry().setFromPoints([
    new THREE.Vector3(0, 0, 0),
    new THREE.Vector3(0, 50, 0),
  ]);
  const yAxis = new THREE.LineSegments(yGeometry, yMaterial);

  const zGeometry = new THREE.BufferGeometry().setFromPoints([
    new THREE.Vector3(0, 0, 0),
    new THREE.Vector3(0, 0, 50),
  ]);
  const zAxis = new THREE.LineSegments(zGeometry, zMaterial);

  const axesGroup = new THREE.Object3D();
  axesGroup.add(xAxis, yAxis, zAxis);
  threeScene.add(axesGroup);
  // 设置y轴向下
  camera.up = new THREE.Vector3().fromArray([0, -1, 0]);

  camera.position.set(0, 0, 6);

  const viewer = new GaussianSplats3D.Viewer({
    initialCameraLookAt: [0, 0, 0],
    threeScene: threeScene,
    camera: camera,
  });
  let sleepStep = 0;
  viewer.addSplatScene("./assets/data/16.ply",{
    'rotation': [1, 0, 0, 0]
  }).then(() => {
    viewer.start();
  });

Uploading 16.compressed (1).zip…

There was a bug that I fixed in the latest release v0.2.1 that caused scene transforms to be improperly applied. Could you try that version and see if it fixes your problem?

Ok, I'll try

Just following up, did you have any luck with the latest version?

I found in version 0.2.1 that I still couldn't rotate the scene

https://gitee.com/fan_yan_fang/guassian-demo

For what it's worth, when I use your code above and change the rotation parameter of the addSplatScene() function, it does change the orientation of the scene. Can you post a link to the scene you're trying to view?

I think I made a mistake, rotation is valid in 0.2.1.

Glad to hear it, I'll go ahead and close this one.


Compile Problem requires 'export CUB_INC=/path/to/cub

Thanks for sharing your code.
I got a problem when complie the usr_op, showing as below:
(I'm not familiar with the Cuda-CUB)
-- Reuse cached information from TensorFlow 1.9.0
-- No TensorFlow-CC-LIBRARY detected
-- No TensorFlow source repository detected
-- CUDA_INCLUDE_DIRS: /usr/local/cuda-9.0/include
-- CUDA_SAMPLE_INC: /samples/common/inc
CMake Error at CMakeLists.txt:26 (message):
  requires 'export CUB_INC=/path/to/cub

I'm work with ubuntu 14.04, cuda 9.0, tensorflow-gpu 1.9.0, GTX 1080Ti.
Hope you can fix it, thank you.

See the updated build instructions in the README.md from commit 129cfefebf1dffa074d030be94065bdebc74f830. The copy-and-paste solution would be
cd /tmp
wget https://github.com/NVlabs/cub/archive/v1.8.0.zip
unzip v1.8.0.zip -d $HOME/libs
export CUB_INC=$HOME/libs/cub-1.8.0/
rm /tmp/v1.8.0.zip

Feel free to ask, if there is anything unclear.


Chunk interpolation to select calibration data

I will need a method to select calibration data for the strar tracker. I made some slides to decribe how it is supposed to work:
https://docs.google.com/presentation/d/1oxIcYSQvGnU7IQYy3fGdcv0qXiLpvaXR9YtmnDesj4Y/edit?usp=sharing

@maxnoe @kosack Can you have another look if there is something else to be changed?

@kosack @maxnoe this PR is needed to complete the pointing calibration (for the variance calibration application), can we advance it?

I am a bit stuck here with one of the tests. test_hdf5 is failing in pytest begause the data from the file is not loaded correctly. When i look in the test what x and y values the interpolators have i get some wrong values. However if i try to do the same outside of pytest it works. I used this code to test by myself:
import astropy.units as u
import numpy as np
import tables
from astropy.table import Table
from astropy.time import Time

from functools import partial
from ctapipe.core import Component, traits

from ctapipe.monitoring.interpolation import PointingInterpolator

from ctapipe.io import write_table

t0 = Time("2022-01-01T00:00:00")

table = Table(
    {"time": t0 + np.arange(0.0, 10.1, 2.0) * u.s, "azimuth": np.radians(np.linspace(0.0, 10.0, 6)) * u.rad, "altitude": np.radians(>)

path = "pointing.h5"

write_table(table, path, "/dl0/monitoring/telescope/pointing/tel_001")

with tables.open_file(path) as h5file:
    interpolator = PointingInterpolator(h5file)
    t = t0 + 1 * u.s
    alt, az = interpolator(tel_id=1, time=t)
    print(interpolator._interpolators[1]["alt"].y,interpolator._interpolators[1]["alt"].x)
    print(alt,az)

Has anyone an idea what is wrong here?

@mexanick Thanks, i got the issue fixed.

@maxnoe Hi Max, could you give this another look?

This looks good now. A remaining question would be if you want to add specific ChunkInterpolators (like PointingInterpolator for LinearInterpolator)?
I.e. CalibrationInterpolator, FlatFieldInterpolator, PedestalInterpolator etc.

Can also be done in a follow-up PR of course.

This looks good now. A remaining question would be if you want to add specific ChunkInterpolators (like PointingInterpolator for LinearInterpolator)?
I.e. CalibrationInterpolator, FlatFieldInterpolator, PedestalInterpolator etc.

I'd consider having a factory then. I think, just "CalibrationInterpolator" won't make much sense, but a specific ones like "FFInterpolator" may. I'd address it in another PR.

Yes, factory makes sense for this


chore: update version of aa

The error is not related to this PR. It is caused by tonic dependency.


ADM 395[stub] post request configuration Cross Domain

Summary
post request configuration Cross Domain

generateReporter
could u pair with backend to update this endpoint to match RESTful


Cannot input Chinese (even change IME to Chinese) in Textbox on Kylin V10

Describe the bug
Cannot change input method to Chinese and input Chinese in a textbox defined like below on Kylin V10, But works fine on Windows. The default font has been installed on the system and can be displayed correctly.
Noto Sans CJK TC Regular

I also tried the solution for the issue https://github.com/AvaloniaUI/Avalonia/issues/6155, but still not work.

To Reproduce
Run a applicaiton with textboxes on Kylin V10.
Expected behavior
Can change input method to Chinese and input Chinese in the Textbox.
Avalonia version
11.0.10
OS
Linux
Additional context
No response

你是不是用root权限启动了？

你是不是用root权限启动了？

是的

你是不是用root权限启动了？

是的

输入法不是root权限的  root权限的程序没法加载非root权限的输入法

@yangjieshao @a6754z any chance to translate it to ebglish as well?

你是不是用root权限启动了？

是的

输入法不是root权限的 root权限的程序没法加载非root权限的输入法
A program with root privileges cannot load input methods whitch do not had root privileges

确实是这个原因，非常感谢！
Yes. That's indeed the reason. After change to start the application with non-root privilege, I can enter the Chinese in textbox.
Thanks a lot!


Add links to glitter AR.js/lobby and the AR.js Trello to the Readme.

Is your feature request related to a problem? Please describe.
It would be nice to steer people to a place for more interaction with questions as well as something that shows things aren't dead, yet. :-)
Describe the solution you'd like
Perhaps the following links in the Status section of the Readme:
https://trello.com/b/63F7JlvD/arjs
https://gitter.im/AR-js/Lobby
Describe alternatives you've considered
Don't really have any, though I guess it could also go under 'What's Next' or 'Last but not least'
Additional context
There's a good start on picking this back up, but the pieces still seem disconnected...this might help.

Sounds good! Can someone make a simple 2 PR for this, on both dev branch and master?

I will make an attempt at that.  Sounds like a good way to get started with pull requests. :-)


Route generation takes a LONG time for environments with many routes

Describe the bug
After updating an Openshift cluster to using the v2.5.0 version kubectl-build-deploy-dind image from a much older version of this image, it was noticed that a specific project with multiple hundreds of routes was now taking multiple hours to build, where it took ~20 minutes previously.  The overwhelming majority of this time is spent with the build pod sitting at ~100% CPU usage (1 core) churning through executing the exec-routes-generation.sh script.
To Reproduce
Steps to reproduce the behavior:

Create an example project with multiple hundred routes in its .lagoon.yml
Deploy the project
See build take multiple hours
????
PROFIT

Expected behavior
Deployments should not take multiple hours, regardless of the number of domains present in a project/environment.
Screenshots
N/A
Additional context
Given the massive amounts of domains in this project, it seems to me that what we're seeing here is simply handling/processing a substantial amount of data numerous times. I would assume it would be better to store this data somehow once accessed, so that we don't need to read the .lagoon.yml file, pipe that to shyaml, then filter for the specific bit we need; multiple times. Maybe this routes generation activity is better suited to a higher programming language like Python?

This is fixed by https://github.com/uselagoon/lagoon/pull/3133 right?

This would've been resolved by #3133, but that functionality was reverted as part of the work done in #3181 to resolve issues we saw with badly formatted docker-compose.yml files. For now, this issue remains unresolved as of Lagoon release 2.8.4

This has been fixed now that routes are handled in the build-deploy-tool


Idea: Community page margins are off

This happens to be the case because of the .community .callouts .callout a selector which adds a margin-right: 8px. Unfortunately, it applies to the text link (not just the logo link), which pushes the description 8px out.

Interesting, what browser? This doesn't repro on mine (Checked safari and edge)

Forgot to post the link: https://www.typescriptlang.org/community
Ah this was on Firefox. Indeed, it looks like it works in Edge.

Also, there are unexpected spaces caused by the same margins:


PowerToys Run: search sorting is tricky

Description of the new feature / enhancement
typing "rem" or "remo" or "remot" should bring me "Remote Desktop" first, then Windows Settings then Command line based on my settings, but it seems it is always Windows Settings.
Scenario when this would be used?
I expect results are aggregated then sorted. I still dont understand what is the current behavior
Supporting information
"rem"

"remo"

"remot"

and here are my settings

Program (which should be the "Remote application" program in this case) has the highest rank

Shell which is also second result in my case should have had middle rank

Windows Settings has the lowest rank but showed first

I'm having the same issue where Global sort order modifier does not seem to be affecting the order. I have the Windows walker set to 100 but whenever I search for a program it'll always show the App (which opens a new app) before showing the running app version.
Toggling the above mentioned "Results order tuning" does not seem to make a difference on my end.

Same here, Windows settings seem to be prioritized all the time and are not influenced by any ranking.
@stefansjfw any idea what this could be?

Also reporting in that I'm experiencing this. Global search weights being applied per plugin seems to be doing little to nothing at all, though the tuning seems to manually put recently used tasks at the top.
Even though I have Window Walker priority at 30, recents at 10, and Programs at -20, searching "outlook" or "firefox", it prioritizes the applications as opposed to the actively running windows.

Having the same issue here. When searching for "Fire", it should show "Firefox" first, but it shows "Windows Defender Firewall with Advanced Security".
The only way to force "Firefox" is to pump all the values to 1000 or -1000 (1000 for the fine tunning and the "Programs" plugin, and -1000 for the "Windows Settings" plugin) and even then, it mixes the results in a weird way:

(This is the behaviour with the values pumped to 1000 or -1000)

I'm going to be digging in the code a bit to see if I can find something obvisously wrong, wish me luck

I confirm that the sort order modifier does not guarantee that the result from the category with a higher modifier will pop at the top of the list.
My case:
I want Programs to appear at the top. However, Windows settings hijack this and appear at the top instead, despite the sort order modifiers.
I use v0.73.0.
My configuration is:

Results order tuning: OFF
Windows settings - Global sort order modifier: -1000 (expect them to appear at the bottom)
Program - Global sort order modifier: +1000 (expect them to appear at the top)

I try to launch Telegram.
Expected: when I type "tele", Program "Telegram" pops at the top of the list.
Actual: when I type "tele", Windows settings "Phone and modem" pops at the top.
Only after I type "teleg" does Telegram pop at the top.

Either I don't understand the concept of sort order modifier or there is something wrong with it.

I dig a little bit, and if I didn't make any mistakes, ResultsViewModel.Sort is what sort the result list:
public void Sort(MainViewModel.QueryTuningOptions options)
{
    List<ResultViewModel> sorted = null;

    if (options.SearchQueryTuningEnabled)
    {
        sorted = Results.OrderByDescending(x => (x.Result.Metadata.WeightBoost + x.Result.Score + (x.Result.SelectedCount * options.SearchClickedItemWeight))).ToList();
    }
    else
    {
        sorted = Results.OrderByDescending(x => (x.Result.Metadata.WeightBoost + x.Result.Score + (x.Result.SelectedCount * 5))).ToList();
    }

    // remove history items in they are in the list as non-history items
    foreach (var nonHistoryResult in sorted.Where(x => x.Result.Metadata.Name != "History").ToList())
    {
        var historyToRemove = sorted.FirstOrDefault(x => x.Result.Metadata.Name == "History" && x.Result.Title == nonHistoryResult.Result.Title && x.Result.SubTitle == nonHistoryResult.Result.SubTitle);
        if (historyToRemove != null)
        {
            sorted.Remove(historyToRemove);
        }
    }

    Clear();
    Results.AddRange(sorted);
}

The sorting score equals to "WeightBoost + Score + SelectedCountScore", and by my understanding, WeightBoost and SelectedCountScore is controled by PowerToysRun itself, while Score can be modified by plugin, for example Calculator Plugin set the result score as 300:
public static Result CreateResult(decimal? roundedResult, string iconPath, CultureInfo culture)
{
    // Return null when the expression is not a valid calculator query.
    if (roundedResult == null)
    {
        return null;
    }

    return new Result
    {
        // Using CurrentCulture since this is user facing
        Title = roundedResult?.ToString(culture),
        IcoPath = iconPath,
        Score = 300,
        SubTitle = Properties.Resources.wox_plugin_calculator_copy_number_to_clipboard,
        Action = c => Action(roundedResult, culture),
    };
}

So, if I didn't make any mistake, the problem is the WeightBoost i.e. "global sort order score modifier" which we can control is limited to 1000 maximum, while the Score controled by plugin can be arbitrary large, I think a effective solution is to set a UpperBound for score returned by plugin and make the final score something like "ScoreModifier * UpperBound + PluginScore + SelectedCountScore"?

Same here.. found a couple of similar issues too in the github, linked them to here.

I had been experiencing the same issue, principally that WindowWalker does not show above Programs regardless of settings. This is why I had switched to Flowy launcher for some time.
Setting WindowWalker priority to 1000 and Programs to -10 does seem to make PowerToys Run respect the ordering, although I won't be surprised if it returns to the old behavior of ignoring my configuration.


Compile Error

Trying to compile runs though and will error out:
MacOS Ventura 13.5
rustc 1.71.1 (eb26296b5 2023-08-03)
cargo 1.71.1 (7f1d04c00 2023-07-29)
Output:

Compiling icy_term v0.5.0 (/Users/xxxxxx/Downloads/icy_term/icy_term)
error[E0599]: no method named from_unicode found for struct Box<(dyn BufferParser + 'static)> in the current scope
--> src/ui/main_window_mod.rs:227:50
|
227 |         let translated_char = self.buffer_parser.from_unicode(ch);
|                                                  ^^^^^^^^^^^^ help: there is a method with a similar name: convert_from_unicode
error[E0599]: no method named from_unicode found for struct Box<(dyn BufferParser + 'static)> in the current scope
--> src/ui/main_window_mod.rs:242:58
|
242 |                 let translated_char = self.buffer_parser.from_unicode(ch);
|                                                          ^^^^^^^^^^^^ help: there is a method with a similar name: convert_from_unicode
error[E0599]: no method named from_unicode found for struct Box<(dyn BufferParser + 'static)> in the current scope
--> src/ui/main_window_mod.rs:249:58
|
249 |                 let translated_char = self.buffer_parser.from_unicode(ch);
|                                                          ^^^^^^^^^^^^ help: there is a method with a similar name: convert_from_unicode
error[E0599]: no method named from_unicode found for struct Box<(dyn BufferParser + 'static)> in the current scope
--> src/ui/main_window_mod.rs:496:42
|
496 |         let mut cr = [self.buffer_parser.from_unicode('\r') as u8].to_vec();
|                                          ^^^^^^^^^^^^ help: there is a method with a similar name: convert_from_unicode
error[E0599]: no method named to_unicode found for reference &dyn BufferParser in the current scope
--> src/ui/buffer_view/mod.rs:533:44
|
533 |                     res.push(buffer_parser.to_unicode(ch.ch));
|                                            ^^^^^^^^^^ help: there is a method with a similar name: convert_to_unicode
error[E0599]: no method named to_unicode found for reference &dyn BufferParser in the current scope
--> src/ui/buffer_view/mod.rs:546:44
|
546 |                     res.push(buffer_parser.to_unicode(ch.ch));
|                                            ^^^^^^^^^^ help: there is a method with a similar name: convert_to_unicode
error[E0599]: no method named to_unicode found for reference &dyn BufferParser in the current scope
--> src/ui/buffer_view/mod.rs:551:44
|
551 |                     res.push(buffer_parser.to_unicode(ch.ch));
|                                            ^^^^^^^^^^ help: there is a method with a similar name: convert_to_unicode
error[E0599]: no method named to_unicode found for reference &dyn BufferParser in the current scope
--> src/ui/buffer_view/mod.rs:557:48
|
557 |                         res.push(buffer_parser.to_unicode(ch.ch));
|                                                ^^^^^^^^^^ help: there is a method with a similar name: convert_to_unicode
error[E0599]: no method named to_unicode found for reference &dyn BufferParser in the current scope
--> src/ui/buffer_view/mod.rs:563:44
|
563 |                     res.push(buffer_parser.to_unicode(ch.ch));
|                                            ^^^^^^^^^^ help: there is a method with a similar name: convert_to_unicode
error[E0599]: no function or associated item named new found for struct PETSCIIParser in the current scope
--> src/ui/screen_modes.rs:112:69
|
112 |                 main_window.buffer_parser = Box::new(PETSCIIParser::new());
|                                                                     ^^^ function or associated item not found in PETSCIIParser
error[E0599]: no function or associated item named new found for struct AtasciiParser in the current scope
--> src/ui/screen_modes.rs:122:69
|
122 |                 main_window.buffer_parser = Box::new(AtasciiParser::new());
|                                                                     ^^^ function or associated item not found in AtasciiParser
error[E0599]: no function or associated item named new found for struct PETSCIIParser in the current scope
--> src/address_mod.rs:160:48
|
160 |                 return Box::new(PETSCIIParser::new());
|                                                ^^^ function or associated item not found in PETSCIIParser
error[E0599]: no function or associated item named new found for struct AtasciiParser in the current scope
--> src/address_mod.rs:163:48
|
163 |                 return Box::new(AtasciiParser::new());
|                                                ^^^ function or associated item not found in AtasciiParser
For more information about this error, try rustc --explain E0599.
error: could not compile icy_term (bin "icy_term") due to 13 previous errors

same issue on debian 11.x with rust 1.71

I updated one underlying library.
Fixed.


License ?

Hello, under what license is this project released under ? Thank-you.

MIT License. Feel free to copy modify and redistribute it ;)


Automatically get the pin from the user and press back.

No need for personal accounts to use this app.


Cannot change the keybind for accessing in-game player model selection menu.

While playing in-game, the g key is used to access gestures and select the model you want to use. It is not possible to change this keybind in the mod's settings.

You can modify the mods keybinds in Minecraft's default keybind editor, located at Pause/Options/Controls


Environment changes are only picked up upon full restart

Environment
Windows build number: Microsoft Windows NT 10.0.18362.0
Windows Terminal version (if applicable): 1.1.2021.0

Steps to reproduce

Launch Windows terminal
Modify environment (e.g. via Win+Pause -> Advanced System Settings -> Environment variables)
Open a new tab in the existing instance

Expected behavior

Should see environment change in the new tab

Actual behavior

Environment change only becomes available if the entire Terminal instance is restarted completely

Thanks for the report! This is a duplicate of #1125, but I decided to just leave it open in PR #7243.


Import magic command displays errors for Markdown cells

Describe the bug
When I use the import magic command to import a .dib notebook which contains a Markdown cell, the notebook displays an error instead of the output of the Markdown cell.
See this example to reproduce the bug:

Please complete the following:
Which version of .NET Interactive are you using? (In a notebook, run the #!about magic command. ):
Version: 1.0.355307+4cc71ae4e275fd80852b9c2faa317bc3a0033d2c

OS

[ ] Windows 11
[x] Windows 10
[ ] macOS
[ ] Linux (Please specify distro)
[ ] iOS
[ ] Android

Browser

[ ] Chrome
[ ] Edge
[ ] Firefox
[ ] Safari

Frontend

[ ] Jupyter Notebook
[ ] Jupyter Lab
[ ] nteract
[x] Visual Studio Code
[ ] Visual Studio Code Insiders
[ ] Visual Studio
[ ] Other (please specify)

Hi, I am working on this at #2569


vscode get slower over time

VSCode Version: Code 1.17.2 (b813d12980308015bcd2b3a2f6efa5c810c33ba5, 2017-10-16T13:59:46.104Z)
OS Version: Windows_NT x64 10.0.14393
Extensions:

Extension
Author (truncated)
Version

vscode-css-formatter
aes
1.0.0

vscode-eslint
dba
1.4.3

githistory
don
0.2.3

Angular2
joh
2.10.0

Steps to Reproduce:
1.use vscode and keep ir open for several days, working intensivelly with it.
2.

Reproduces without extensions: Yes/No

I wonder if this issue is being caused by an installed extension. Can you try to run VS Code without extensions? From the command line (NOT the integrated terminal in Code), execute: code --disable-extensions and try your steps again to see if it reproduces. If you see it is an issue with the extension, please file it against the extension repository itself.


rollup should respect  the preset while reading package.json exports

Motivation
When building for worker runtimes such as cloudflare/vercel-edge, there's an issue where the rollup plugin node resolve will follow the default priority which is good for Node, but not for these environments, which means in these scenarios many dependencies needs to be alias manually to their browser version.
Describe the feature
Depending on the preset, we should alter rollup node-resolve exportConditions order priority.
For example for Cloudflare like environments, we could try workerd > browser > import > default >  module > deno > node.
For deno we could try deno > browser > import >   default > workerd > module > node
It looks like there's a lot of edge cases and no convention defined, ie vercel uses edge-light, libsql use web and http ...
We could run into an issue where we don't have the correct export no matter how much we try, hence why we should give the ability to customise this through Nitro options, as well as document it.
https://github.com/rollup/plugins/blob/master/packages/node-resolve/README.md
https://github.com/unjs/nitro/blob/main/src/rollup/config.ts
Additional information

[X] Would you be willing to help implement this feature?

Related:

Allow exported export conditions: https://github.com/unjs/nitro/issues/83
Using web export condition for Worker presets: https://github.com/unjs/nitro/issues/1123
Adoption for runtime-keys: https://github.com/unjs/node-fetch-native/pull/75 (we probably need to prefer this over for worker preset conditions)


Deserialization changes ECTD data

ECTD is not fully supported by UDO and I tried to use abapgit as a workaround. There are some minor issues here as well:
After pulling the following ECTD object, you should see differences between remote and local.
zfh_ifme_tdc_ff_parser_json.ectd.zip
It will remove a trailing newline in the data:
<CONTENT>:3:NEWT
</CONTENT>

becomes
<CONTENT>:3:NEWT</CONTENT>

It also changed SORTLNR from 2 to 1, whatever that is.
There's also an issue with the diff display. There is a very long line in the serialized xml and the diff doesn't word wrap, so I can't see the other part or scroll.

Can you provide a public test repo to reproduce the issue? I cannot import your zip as the IF_IFME_TESTING=>TS_JSON doesn't exist in my NW 752 dev edition.

Try pulling https://github.com/FreHu/abapgit-ectd-bug
Small correction, the diff is partly visible at 1440p:

I renamed the test data container. Shouldn't SAP disallow copying to a local package without a customer prefix in the first place? I'm not familiar with the rules about this.
Strangely, now pulling causes the following error, which I wasn't getting before:

Despite the error, the pull works (refresh the repo after).

Works with 754 SP 7 and latest abapGit


[ZEPPELIN-3042] updating jgit to support post-commit hooks

What is this PR for?
Git functionality in Zeppelin is provided by jgit, but it's fixed to an old version. A newer version of said library supports new things, primarily post-commit hooks.
What type of PR is it?
Improvement
Todos

[ x ] - CI (two failed, but due to yarn)

What is the Jira issue?
ZEPPELIN-3042
Questions:

Does the licenses files need update? -- Yes, only because the license file notes the exact version of included libraries. jgit hasn't changed, license-wise, so it's only a matter of updating the version information
Is there breaking changes for older versions? -- nope
Does this needs documentation? -- possibly, to let people know they get use post-commit hooks now?

thanks for contribution. I think need to configure travis with your account for builds https://zeppelin.apache.org/contribution/contributions.html#continuous-integration , otherwise couldn't check you jenkins CI status

I already have Travis set up and I linked its results in my initial comment. Or am I missing something?

Thanks @kokes for the contribution.
Checked CI result https://travis-ci.org/kokes/zeppelin/builds/299146848
Some profiles are failing but not related to this PR.
LGTM and merge to master if no further discussion.

I noticed that the JGit minimum execution environment became Java 8 from JGit 4.6.
https://projects.eclipse.org/projects/technology.jgit/releases/4.6

Update JGit minimum execution environment to JavaSE-1.8

When I ran Zeppelin by Java 7, the following message was shown and Zeppelin could not be launched.
Caused by: java.lang.UnsupportedClassVersionError: org/eclipse/jgit/lib/Repository : Unsupported major.minor version 52.0
        at java.lang.ClassLoader.defineClass1(Native Method)
        at java.lang.ClassLoader.defineClass(ClassLoader.java:803)
        at java.security.SecureClassLoader.defineClass(SecureClassLoader.java:142)
        at java.net.URLClassLoader.defineClass(URLClassLoader.java:442)
        at java.net.URLClassLoader.access$100(URLClassLoader.java:64)
        at java.net.URLClassLoader$1.run(URLClassLoader.java:354)
        at java.net.URLClassLoader$1.run(URLClassLoader.java:348)
        at java.security.AccessController.doPrivileged(Native Method)
        at java.net.URLClassLoader.findClass(URLClassLoader.java:347)
        at java.lang.ClassLoader.loadClass(ClassLoader.java:425)
        at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:312)
        at java.lang.ClassLoader.loadClass(ClassLoader.java:358)
        at java.lang.Class.forName0(Native Method)
        at java.lang.Class.forName(Class.java:195)
        at org.apache.zeppelin.notebook.repo.NotebookRepoSync.<init>(NotebookRepoSync.java:80)
        at org.apache.zeppelin.server.ZeppelinServer.<init>(ZeppelinServer.java:127)
        at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
        at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:57)
        at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
        at java.lang.reflect.Constructor.newInstance(Constructor.java:526)
        at org.glassfish.hk2.utilities.reflection.ReflectionHelper.makeMe(ReflectionHelper.java:1350)
        at org.jvnet.hk2.internal.Utilities.justCreate(Utilities.java:1009)
        ... 30 more

Is it OK for Zeppelin not to support Java 7?

And it seems that this change causes the failure of the test https://travis-ci.org/kokes/zeppelin/jobs/299146854 because Zeppelin process didn't respond to any requests.

Darn, that's unfortunate (and it didn't click initially, because a lot of the other builds are based on Java 7).
A better interim solution would be to downgrade the library (but still upgrade against the original code) to version 4.5.0, before the restriction of Java > 7 was introduced. This is the first major version to allow for post commit hooks, which is the point of this PR.
I've edited my branch and launched a build on Travis. Let's see how that one goes.

It looks a bit better, only one build errs, of the four tests there are two timeouts, will try it again to see if it helps.

@kokes I confirmed that CI test against JGit 4.9.0 always fails and one against JGit 4.5.4 passed.
JGit 4.9.0

comimt: https://github.com/kjmrknsn/zeppelin/commit/2afba274c83d4e9f4527fcde41e09c7707c89a4c
CI test result: https://travis-ci.org/kjmrknsn/zeppelin/builds/310104872

JGit: 4.5.4

commit: https://github.com/kjmrknsn/zeppelin/commit/27b08c9d0a938dbe1e70ed27c4e6f1ba21c4dafd
CI test result: https://travis-ci.org/kjmrknsn/zeppelin/builds/310107611

So, I think JGit version should be 4.5.4, which is the latest version for Java 7: https://projects.eclipse.org/projects/technology.jgit

I sent a PR to fix this issue: https://github.com/apache/zeppelin/pull/2693


use incremental ninja builds

Technically it should be possible to run ninja build in ./build directory from the previous build and reuse build results from it.
The biggest challenge I see is that we will need to enable all projects in cmake but run tests only for the affected projects.
Open questions:

how much speedup that provides on windows / linux to justify the development?

for "far" commits (~100 commits in between) ninja rebuilds almost everything and we get no speedup

This isn't a safe technique in general: changes in CMake would not be picked up because of the cache, and leftover from the previous builds can introduce/hide issues as well.
CCache gets your most of the benefits without sacrifying correctness.

@joker-eph e.g. on windows moving 10 commits from e2fe26e77b02633207bdd36a14e2dcf210cc2fe7 to d81a843846f4171ffe3f44b66bf3a9fda5c411cf will result
in 60s build with ninja incremental build and using only sccache will take ~600s with almost perfect cache hit (except for dozen targets).
My idea was to run cmake anyway and check if build.ninja has changed to decide if we have start from scratch. On the other hand I don't build with ninja often and will be grateful to know if there are some important nuances.

You would need to nuke CMakeCache.txt before rerunning cmake to force it to repopulate its cache.
This would avoid most CMake configuration issues I think, but that won't avoid files that are moving for example.
Think if I change the TableGen configuration and generate my header in a new folder but I forget to update one of the include path: the previously generated header will be in the build directory and accessible from the previous include path. The build will succeed but any clean build would fail (You can also have false failures with similar conditions).
In practice it may be OK though, these conditions may be rare enough (even though when it happens it can be a pain to debug). Wiping out the build directory on a daily basis may be enough?


Player name empty in status HTTP payload

So I enabled the status HTTP server with the STATUS_HTTP=true flag, and am hitting the endpoint (http://localhost:9000/status.json) for a server I'm running locally. It appears that the Steam API is properly detecting when I log in/out because the number of players in the array in the payload increases/decreases appropriately. However, the information is incomplete - the player name is simply an empty string:
{
  "last_status_update": "2021-09-05T08:59:14.112084+00:00",
  "error": null,
  "server_name": "Eric's test server",
  "server_type": "d",
  "platform": "l",
  "player_count": 1,
  "password_protected": true,
  "vac_enabled": false,
  "port": 2456,
  "steam_id":<PHONE_NUMBER>5623300,
  "keywords": "0.156.2",
  "game_id": 892970,
  "players": [
    {
      "name": "",
      "score": 0,
      "duration":<PHONE_NUMBER>8359375
    }
  ]
}

I have the following relevant environment variables set; do you see any problem?
SERVER_PUBLIC=true
STATUS_HTTP=true
STATUS_HTTP_PORT=9000

See https://github.com/lloesche/valheim-server-docker#status-web-server

All the information in status.json is fetched from Valheim servers public query port. You will notice that some of the fields like player name or player score currently contain no information. However for completeness the entire query response is left intact.

ahh didn't see that, thank you. so this is something that Steam or the Valheim developers will need to address themselves then?

I think I found a bug report for this. if anyone else coming here would like to see this resolved, please upvote here to put it on the Valheim devs' radar:
https://valheimbugs.featureupvote.com/suggestions/160117/enhance-valve-protocol-support
I'll close the issue since there is nothing that can be done from our side


Examine Pattern.compile usage in sdk\appconfiguration

The following files have an instance of Pattern.compile() that should be examined and likely replaced.

[ ] azure-spring-cloud-appconfiguration-config\src\main\java\com\azure\spring\cloud\config\resource\Connection.java
[ ] spring-cloud-azure-appconfiguration-config\src\main\java\com\microsoft\azure\spring\cloud\config\resource\Connection.java

Closing as these files no longer exist


multiple speaker compatability

Hi so first of all great work,
the diarization works great for me on audio files with less than 3 speakers. Given an audio file with more than or close to 8 speakers, results in a very good transcription, but there are still only 3 speakers assigned to more than 8 people (so it just assigns them all as speaker 0 to 2). Changing the max_num_speakers variable in the config yaml doesnt seem to change that, so does creating a num_speakers variable in the manifest file. Is there something i'm missing/ how would adjusting the speaker count work?

the speaker count in the config is for evaluation purposes only, in order to tune the diarization performance, you need to play with speaker_embeddings parameters such as window_length_in_sec and shift_length_in_sec. diar_window_length and sigmoid_threshold might be worth giving a shot too

thanks, i'll give that a try :)

Hi,
I'm have the same issue. Can you provide some instructions on how to tune those parameters to have a higher number of speakers by default, please?
Thanks in advance.

Hi @famda , I haven't tinkered with these before, but it's totally trial and error

No worries, I can make some tests with it. I just need to understand where to start. Can you guide me a little so I can play around with it? 😀

From just messing around with the shift_length and lowering it about 0.25 i was able to detect 7 speakers when running it on an audiofile it only detected 3 before (theres 9 actual people speaking in it). Going Lower than that, didn't make much of a difference, but increased inference time dramatically (running locally on CPU). So i would start there. Changing the sigmoid_threshold didn't do much, but you can try that as well.

You should also try playing with scale windows and weights

Hi, how do you configure the possibility to separate the speaker?
Which is the config flag to set? because in the instruction there is nothing about this..
Can you help me?

Hello, what do you mean exactly by separating the speaker?

Sorry for my bad english!
Actually I have this:
{
"speaker": "Speaker 0",
"start_time": 433460,
"end_time": 435480,
"text": "Adesso approfondiamo un pochino meglio. "
},
{
"speaker": "Speaker 0",
"start_time": 435520,
"end_time": 437420,
"text": "Allora, innanzitutto, da dove venite ragazzi? "
},
both sentences are categorized as “speaker0,” but the first sentence is spoken by a woman and the second by a male speaker.
Is it necessary to set some special configuration parameter to get speaker0 and speaker1?

Hi, I want know can i use task='translate' in  Whisper_Transcription_+_NeMo_Diarization.ipynb file. actually i want to pass a non-english (hindi)  audio in model and then after getting english transcription(by using task = translate). I want to perform speaker diarization.
but i think because my translated transcript and audio both have different language english  and hindi respectively I am  not able to achieve this.
can somebody help me. How can i perform speaker diarization and translation of transcription both.
for testing purpose i am using  Whisper_Transcription_+_NeMo_Diarization.ipynb

@francescocassini usually the default settings work as expected, but you can check my second comment about what to change if they dont
@01Ashish I haven't tested it for translate task yet, but as a starter, you should enable word timestamps in whisper and remove the alignment model and see how it goes

I want to use max_speaker parameter as cli argument like whisperx. Do you have any plan?
Our case is clear for how many peoples speak in audio, so I expect model perform well by assigning correct max_speaker number for each execution. However, I don't know well how to do that.

I want to use max_speaker parameter as cli argument like whisperx. Do you have any plan or solution?
Our case is clear for how many peoples speak in audio, so I expect model perform well by assigning correct max_speaker number for each execution. However, I don't know well how to do that.

You can modify this parameter in the telephonic YAML config found in the configs folder, can you try it on an audio that predicts the wrong number of speakers a d see if it makes a difference?

@MahmoudAshraf97
Thank you for quick reply. but, unfortunately I didn't face wrong number of speaker problem. just considering tune this repository behavior.
I use this repository for creating transcription with docker on cloud services. So..., changing yaml is bit difficult for my case.
Ideally, If I can assign max_speaker value from cli, I can change behavior dynamically via docker run command.
That kind of operation should be supported in this repository in future? If no plan, I will use larger speaker count

It's easy to add, not a big deal, but I want to make sure first that it does affect the inference, because it was reported earlier that this parameter has no effect on inference, only evaluation which is not used here


[camera] Exception for calling availableCameras() with cached engine and FlutterFragment on Android

Steps to reproduce
Create an Android project with add-to-app Flutter module, cached engine and navigation to/from FlutterFragment as in example
Add call of availableCameras() in flutter code after navigation to FlutterFragment.
In application: navigate to FlutterFragment -> close it by back button -> open new FlutterFragment
Expected results
No exception for availableCameras()
Actual results
Exception for availableCameras()
PlatformException(java.lang.NullPointerException, NullPointerException, Cause: null, Stacktrace: java.lang.NullPointerException
at java.util.Objects.requireNonNull(Objects.java:220)
at io.flutter.plugins.camerax.CameraSelectorHostApiImpl.filter(CameraSelectorHostApiImpl.java:49)
at io.flutter.plugins.camerax.GeneratedCameraXLibrary$CameraSelectorHostApi.lambda$setup$1(GeneratedCameraXLibrary.java:1039)
at io.flutter.plugins.camerax.GeneratedCameraXLibrary$CameraSelectorHostApi$$ExternalSyntheticLambda1.onMessage(D8$$SyntheticClass:0)
at io.flutter.plugin.common.BasicMessageChannel$IncomingMessageHandler.onMessage(BasicMessageChannel.java:261)
at io.flutter.embedding.engine.dart.DartMessenger.invokeHandler(DartMessenger.java:292)
at io.flutter.embedding.engine.dart.DartMessenger.lambda$dispatchMessageToQueue$0$io-flutter-embedding-engine-dart-DartMessenger(DartMessenger.java:319)
at io.flutter.embedding.engine.dart.DartMessenger$$ExternalSyntheticLambda0.run(D8$$SyntheticClass:0)
at android.os.Handler.handleCallback(Handler.java:938)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loop(Looper.java:246)
at android.app.ActivityThread.main(ActivityThread.java:8653)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:602)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1130)
, null)

Code sample
Code in example project
Screenshots or Video
No response
Logs
No response
Flutter Doctor output
flutter doctor -v
[✓] Flutter (Channel stable, 3.22.2, on macOS 14.5 23F79 darwin-x64, locale en-BY)
    • Flutter version 3.22.2 on channel stable at /Users/mikalai/flutter/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 761747bfc5 (4 weeks ago), 2024-06-05 22:15:13 +0200
    • Engine revision edd8546116
    • Dart version 3.4.3
    • DevTools version 2.34.3

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/mikalai/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_SDK_ROOT = /Users/mikalai/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.10+0-17.0.10b1087.21-11609105)
    • All Android licenses accepted.

[!] Xcode - develop for iOS and macOS (Xcode 15.4)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15F31d
    ✗ Unable to get list of installed Simulator runtimes.
    • CocoaPods version 1.15.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2024.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.10+0-17.0.10b1087.21-11609105)

[✓] VS Code (version 1.85.1)
    • VS Code at /Users/mikalai/Downloads/Visual Studio Code.app/Contents
    • Flutter extension version 3.90.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 14.5 23F79 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 126.0.6478.127

[✓] Network resources
    • All expected network resources are available.

I checked this issue on both Flutter stable 3.22.2 but I saw a different exception:
E  [ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: PlatformException(channel-error, Unable to establish connection on channel., null, null)
   #0      InstanceManagerHostApi.clear (package:camera_android_camerax/src/camerax_library.g.dart:315:7)
   <asynchronous suspension>
I  Connecting to camera service
D  Added camera: 0
I  Device Level: INFO_SUPPORTED_HARDWARE_LEVEL_FULL
D  Added camera: 1
I  Device Level: INFO_SUPPORTED_HARDWARE_LEVEL_FULL

After try upgrading to Flutter master channel (I'm on 3.23.0-14.0.pre.157), I don't see that exception anymore. By any chance, could you also try on Flutter master channel?

@huycozy
I confirm the reported behaviour on Flutter master channel as well (the stack trace is slightly different). I tested on emulators and device.

Flutter Doctor
[✓] Flutter (Channel master, 3.23.0-14.0.pre.159, on macOS 14.5 23F79 darwin-x64, locale en-BY)
  • Flutter version 3.23.0-14.0.pre.159 on channel master at /Users/mikalai/flutter/flutter
  • Upstream repository https://github.com/flutter/flutter.git
  • Framework revision 484cddd1a4 (5 hours ago), 2024-07-08 04:09:13 -0400
  • Engine revision 69075e7e87
  • Dart version 3.5.0 (build 3.5.0-323.0.dev)
  • DevTools version 2.37.0

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
  • Android SDK at /Users/mikalai/Library/Android/sdk
  • Platform android-34, build-tools 34.0.0
  • ANDROID_SDK_ROOT = /Users/mikalai/Library/Android/sdk
  • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
  • Java version OpenJDK Runtime Environment (build 17.0.10+0-17.0.10b1087.21-11609105)
  • All Android licenses accepted.

[!] Xcode - develop for iOS and macOS (Xcode 15.4)
  • Xcode at /Applications/Xcode.app/Contents/Developer
  • Build 15F31d
  ✗ Unable to get list of installed Simulator runtimes.
  • CocoaPods version 1.15.2

[✓] Chrome - develop for the web
  • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2024.1)
  • Android Studio at /Applications/Android Studio.app/Contents
  • Flutter plugin can be installed from:
    🔨 https://plugins.jetbrains.com/plugin/9212-flutter
  • Dart plugin can be installed from:
    🔨 https://plugins.jetbrains.com/plugin/6351-dart
  • Java version OpenJDK Runtime Environment (build 17.0.10+0-17.0.10b1087.21-11609105)

[✓] VS Code (version 1.85.1)
  • VS Code at /Users/mikalai/Downloads/Visual Studio Code.app/Contents
  • Flutter extension version 3.90.0

[✓] Connected device (3 available)
  • sdk gphone64 x86 64 (mobile) • emulator-5554 • android-x64    • Android 13 (API 33) (emulator)
  • macOS (desktop)              • macos         • darwin-x64     • macOS 14.5 23F79 darwin-x64
  • Chrome (web)                 • chrome        • web-javascript • Google Chrome 126.0.6478.127

[✓] Network resources
  • All expected network resources are available.

Exception
PlatformException(java.lang.NullPointerException, NullPointerException, Cause: null, Stacktrace: java.lang.NullPointerException
at java.util.Objects.requireNonNull(Objects.java:203)
at io.flutter.plugins.camerax.ProcessCameraProviderHostApiImpl.getAvailableCameraInfos(ProcessCameraProviderHostApiImpl.java:92)
at io.flutter.plugins.camerax.GeneratedCameraXLibrary$ProcessCameraProviderHostApi.lambda$setup$1(GeneratedCameraXLibrary.java:1157)
at io.flutter.plugins.camerax.GeneratedCameraXLibrary$ProcessCameraProviderHostApi$$ExternalSyntheticLambda1.onMessage(D8$$SyntheticClass:0)
at io.flutter.plugin.common.BasicMessageChannel$IncomingMessageHandler.onMessage(BasicMessageChannel.java:261)
at io.flutter.embedding.engine.dart.DartMessenger.invokeHandler(DartMessenger.java:292)
at io.flutter.embedding.engine.dart.DartMessenger.lambda$dispatchMessageToQueue$0$io-flutter-embedding-engine-dart-DartMessenger(DartMessenger.java:319)
at io.flutter.embedding.engine.dart.DartMessenger$$ExternalSyntheticLambda0.run(D8$$SyntheticClass:0)
at android.os.Handler.handleCallback(Handler.java:873)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loop(Looper.java:193)
at android.app.ActivityThread.main(ActivityThread.java:6669)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:493)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:858)
com.example.tryfluttermodule         I  , null)

Just to be sure I explain steps clear: I push button in my app and go to Flutter fragment, then **go back** by system back button, then **go to Flutter fragment for the second time**.
I did these steps with debug build. I tried with release build and saw the exception you mentioned when I went to Flutter fragment for the first time (didn't see in debug build). I believe, this case can be something about proguard/R8.
However, my exception takes place in both release and debug build modes.

Thanks for providing the clear steps. I can reproduce NullPointerException now.

Output log
2024-07-09 11:15:03.485 32522-32612 flutter                 com.example.tryfluttermodule         E  [ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: PlatformException(channel-error, Unable to establish connection on channel., null, null)
                                                                                #0      InstanceManagerHostApi.clear (package:camera_android_camerax/src/camerax_library.g.dart:315:7)
                                                                                <asynchronous suspension>
2024-07-09 11:15:03.489 32522-32612 flutter                 com.example.tryfluttermodule         I  PlatformException(channel-error, Unable to establish connection on channel., null, null)
2024-07-09 11:17:12.644 32522-32522 WindowOnBackDispatcher  com.example.tryfluttermodule         W  OnBackInvokedCallback is not enabled for the application.
                                                                                Set 'android:enableOnBackInvokedCallback="true"' in the application manifest.
2024-07-09 11:17:12.749 32522-357   CameraManagerGlobal     com.example.tryfluttermodule         I  Connecting to camera service
2024-07-09 11:17:12.793 32522-357   CameraRepository        com.example.tryfluttermodule         D  Added camera: 0
2024-07-09 11:17:12.803 32522-357   Camera2CameraInfo       com.example.tryfluttermodule         I  Device Level: INFO_SUPPORTED_HARDWARE_LEVEL_FULL
2024-07-09 11:17:12.805 32522-357   CameraRepository        com.example.tryfluttermodule         D  Added camera: 1
2024-07-09 11:17:12.806 32522-357   Camera2CameraInfo       com.example.tryfluttermodule         I  Device Level: INFO_SUPPORTED_HARDWARE_LEVEL_FULL
2024-07-09 11:17:12.806 32522-357   CameraValidator         com.example.tryfluttermodule         D  Verifying camera lens facing on panther, lensFacingInteger: null
2024-07-09 11:17:12.818 32522-32612 flutter                 com.example.tryfluttermodule         I  PlatformException(java.lang.NullPointerException, NullPointerException, Cause: null, Stacktrace: java.lang.NullPointerException
2024-07-09 11:17:12.818 32522-32612 flutter                 com.example.tryfluttermodule         I  	at java.util.Objects.requireNonNull(Objects.java:207)
2024-07-09 11:17:12.818 32522-32612 flutter                 com.example.tryfluttermodule         I  	at io.flutter.plugins.camerax.ProcessCameraProviderHostApiImpl.getAvailableCameraInfos(ProcessCameraProviderHostApiImpl.java:92)
2024-07-09 11:17:12.818 32522-32612 flutter                 com.example.tryfluttermodule         I  	at io.flutter.plugins.camerax.GeneratedCameraXLibrary$ProcessCameraProviderHostApi.lambda$setup$1(GeneratedCameraXLibrary.java:1157)
2024-07-09 11:17:12.818 32522-32612 flutter                 com.example.tryfluttermodule         I  	at io.flutter.plugins.camerax.GeneratedCameraXLibrary$ProcessCameraProviderHostApi$$ExternalSyntheticLambda1.onMessage(D8$$SyntheticClass:0)
2024-07-09 11:17:12.818 32522-32612 flutter                 com.example.tryfluttermodule         I  	at io.flutter.plugin.common.BasicMessageChannel$IncomingMessageHandler.onMessage(BasicMessageChannel.java:261)
2024-07-09 11:17:12.818 32522-32612 flutter                 com.example.tryfluttermodule         I  	at io.flutter.embedding.engine.dart.DartMessenger.invokeHandler(DartMessenger.java:292)
2024-07-09 11:17:12.818 32522-32612 flutter                 com.example.tryfluttermodule         I  	at io.flutter.embedding.engine.dart.DartMessenger.lambda$dispatchMessageToQueue$0$io-flutter-embedding-engine-dart-DartMessenger(DartMessenger.java:319)
2024-07-09 11:17:12.818 32522-32612 flutter                 com.example.tryfluttermodule         I  	at io.flutter.embedding.engine.dart.DartMessenger$$ExternalSyntheticLambda0.run(D8$$SyntheticClass:0)
2024-07-09 11:17:12.818 32522-32612 flutter                 com.example.tryfluttermodule         I  	at android.os.Handler.handleCallback(Handler.java:959)
2024-07-09 11:17:12.818 32522-32612 flutter                 com.example.tryfluttermodule         I  	at android.os.Handler.dispatchMessage(Handler.java:100)
2024-07-09 11:17:12.818 32522-32612 flutter                 com.example.tryfluttermodule         I  	at android.os.Looper.loopOnce(Looper.java:232)
2024-07-09 11:17:12.818 32522-32612 flutter                 com.example.tryfluttermodule         I  	at android.os.Looper.loop(Looper.java:317)
2024-07-09 11:17:12.819 32522-32612 flutter                 com.example.tryfluttermodule         I  	at android.app.ActivityThread.main(ActivityThread.java:8592)
2024-07-09 11:17:12.819 32522-32612 flutter                 com.example.tryfluttermodule         I  	at java.lang.reflect.Method.invoke(Native Method)
2024-07-09 11:17:12.819 32522-32612 flutter                 com.example.tryfluttermodule         I  	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:580)
2024-07-09 11:17:12.819 32522-32612 flutter                 com.example.tryfluttermodule         I  	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:878)
2024-07-09 11:17:12.819 32522-32612 flutter                 com.example.tryfluttermodule         I  , null)

flutter doctor -v (stable and master)
[✓] Flutter (Channel stable, 3.22.2, on macOS 14.1 23B74 darwin-x64, locale en-VN)
    • Flutter version 3.22.2 on channel stable at /Users/huynq/Documents/GitHub/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 761747bfc5 (3 weeks ago), 2024-06-05 22:15:13 +0200
    • Engine revision edd8546116
    • Dart version 3.4.3
    • DevTools version 2.34.3

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.10+0-17.0.10b1087.21-11572160)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.4)
    • Xcode at /Applications/Xcode15.4.app/Contents/Developer
    • Build 15F31d
    • CocoaPods version 1.15.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2023.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • android-studio-dir = /Applications/Android Studio.app/
    • Java version OpenJDK Runtime Environment (build 17.0.10+0-17.0.10b1087.21-11572160)

[✓] VS Code (version 1.90.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.90.0

[✓] Connected device (3 available)
    • iPhone (mobile) • d9a94afe2b649fef56ba0bfeb052f0f2a7dae95e • ios            • iOS 15.8 19H370
    • macOS (desktop) • macos                                    • darwin-x64     • macOS 14.1 23B74 darwin-x64
    • Chrome (web)    • chrome                                   • web-javascript • Google Chrome 126.0.6478.127

[✓] Network resources
    • All expected network resources are available.

• No issues found!

[!] Flutter (Channel master, 3.23.0-14.0.pre.157, on macOS 14.1 23B74 darwin-x64, locale en-VN)
    • Flutter version 3.23.0-14.0.pre.157 on channel master at /Users/huynq/Documents/GitHub/flutter_master
    ! Warning: `flutter` on your path resolves to /Users/huynq/Documents/GitHub/flutter/bin/flutter, which is not inside your current Flutter SDK checkout at /Users/huynq/Documents/GitHub/flutter_master. Consider adding /Users/huynq/Documents/GitHub/flutter_master/bin to the front of your path.
    ! Warning: `dart` on your path resolves to /Users/huynq/Documents/GitHub/flutter/bin/dart, which is not inside your current Flutter SDK checkout at /Users/huynq/Documents/GitHub/flutter_master. Consider adding /Users/huynq/Documents/GitHub/flutter_master/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ad6166a8b8 (3 hours ago), 2024-07-07 19:58:25 -0400
    • Engine revision ca79a56a66
    • Dart version 3.5.0 (build 3.5.0-323.0.dev)
    • DevTools version 2.37.0
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.10+0-17.0.10b1087.21-11572160)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.4)
    • Xcode at /Applications/Xcode15.4.app/Contents/Developer
    • Build 15F31d
    • CocoaPods version 1.15.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2023.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • android-studio-dir = /Applications/Android Studio.app/
    • Java version OpenJDK Runtime Environment (build 17.0.10+0-17.0.10b1087.21-11572160)

[✓] VS Code (version 1.90.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.92.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 14.1 23B74 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 126.0.6478.127
    ! Error: iPhone has recently restarted. Xcode will continue when iPhone is unlocked. (code -14)

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

From Android triage: cc @bparrishMines @camsim99
Looks like the error is coming from here
https://github.com/flutter/packages/blob/main/packages/camera/camera_android_camerax/android/src/main/java/io/flutter/plugins/camerax/CameraSelectorHostApiImpl.java#L49
Do we expect that a cached version of the engine would interact poorly with the instance manager/have any other bugs that have run into that class of problem?

So, filter gets called on a newly created CameraSelector in availableCameras (see source), so I find it odd that the new CameraSelector instance would no longer have an identifier after going back, but I do recall us expecting complications with add-to-app use cases.
@bparrishMines Do you have any insight on this?

@gmackall I'm not that familiar with add-to-app scenario for a cached engine on Android. Do you know if the FlutterPlugin events get triggered. Also do any of the ActivityAware events trigger? They could potentially break the interop, but I'll will probably need to log in an example app to see what specifically is being triggered.

Hi me also have problem when opening camera 0.11 from android host app via flutter
I start the flutter app from kotlin:
startActivity(
                FlutterActivity
                    .withCachedEngine(MyApplication.FLUTTER_ENGINE_ID)
                    .build(this)
            )

Then I got this error when accessing camera API again.
Activity io.flutter.embedding.android.FlutterActivity has leaked IntentReceiver io.flutter.plugins.camerax.DeviceOrientationManager$1@b13e581 that was originally registered here. Are you missing a call to unregisterReceiver()?
                                                                                android.app.IntentReceiverLeaked: Activity io.flutter.embedding.android.FlutterActivity has leaked IntentReceiver io.flutter.plugins.camerax.DeviceOrientationManager$1@b13e581 that was originally registered here. Are you missing a call to unregisterReceiver()?
                                                                                at android.app.LoadedApk$ReceiverDispatcher.<init>(LoadedApk.java:1594)
                                                                                at android.app.LoadedApk.getReceiverDispatcher(LoadedApk.java:1374)
                                                                                at android.app.ContextImpl.registerReceiverInternal(ContextImpl.java:1614)
                                                                                at android.app.ContextImpl.registerReceiver(ContextImpl.java:1580)
                                                                                at android.app.ContextImpl.registerReceiver(ContextImpl.java:1568)
                                                                                at android.content.ContextWrapper.registerReceiver(ContextWrapper.java:668)
                                                                                at io.flutter.plugins.camerax.DeviceOrientationManager.start(DeviceOrientationManager.java:73)
                                                                                at io.flutter.plugins.camerax.DeviceOrientationManagerHostApiImpl.startListeningForDeviceOrientationChange(DeviceOrientationManagerHostApiImpl.java:65)
                                                                                at io.flutter.plugins.camerax.GeneratedCameraXLibrary$DeviceOrientationManagerHostApi.lambda$setup$0(GeneratedCameraXLibrary.java:1606)
                                                                                at io.flutter.plugins.camerax.GeneratedCameraXLibrary$DeviceOrientationManagerHostApi$$ExternalSyntheticLambda0.onMessage(D8$$SyntheticClass:0)
                                                                                at io.flutter.plugin.common.BasicMessageChannel$IncomingMessageHandler.onMessage(BasicMessageChannel.java:261)
                                                                                at io.flutter.embedding.engine.dart.DartMessenger.invokeHandler(DartMessenger.java:292)
                                                                                at io.flutter.embedding.engine.dart.DartMessenger.lambda$dispatchMessageToQueue$0$io-flutter-embedding-engine-dart-DartMessenger(DartMessenger.java:319)
                                                                                at io.flutter.embedding.engine.dart.DartMessenger$$ExternalSyntheticLambda0.run(D8$$SyntheticClass:0)
                                                                                at android.os.Handler.handleCallback(Handler.java:938)
                                                                                at android.os.Handler.dispatchMessage(Handler.java:99)
                                                                                at android.os.Looper.loop(Looper.java:223)
                                                                                at android.app.ActivityThread.main(ActivityThread.java:7656)
                                                                                at java.lang.reflect.Method.invoke(Native Method)
                                                                                at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:592)
                                                                                at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:947)

This is related to my issue https://github.com/flutter/flutter/issues/155492

I had the same problem when used ver. ^0.11.0+2, but if you use 0.10.6 it's everything works. Answer is in change that android implementation of the camera plugin migrate from camera_android to camera_android_camerax


a problem by client at version of v2

There are some problems here by v2 plugin of http client at git address "https://github.com/go-micro/plugins/tree/main/v2/client/http"
by reading origin code , i find the a http request call for other micro service , why does request method designed as 'POST'  and was forced.
the other http client at JAVA Stack such as 'feign'、'rest template' , I can use http method in GET\POST\PUT ..........

Could you provide some code examples to showcase the problems?

`
func (h *httpClient) call(ctx context.Context, node *registry.Node, req client.Request, rsp interface{}, opts client.CallOptions) error {
// set the address
address := node.Address
header := make(http.Header)
if md, ok := metadata.FromContext(ctx); ok {
for k, v := range md {
header.Set(k, v)
}
}
// set timeout in nanoseconds
header.Set("Timeout", fmt.Sprintf("%d", opts.RequestTimeout))
// set the content type for the request
header.Set("Content-Type", req.ContentType())

// get codec
cf, err := h.newHTTPCodec(req.ContentType())
if err != nil {
	return errors.InternalServerError("go.micro.client", err.Error())
}

// marshal request
b, err := cf.Marshal(req.Body())
if err != nil {
	return errors.InternalServerError("go.micro.client", err.Error())
}

buf := &buffer{bytes.NewBuffer(b)}
defer buf.Close()

hreq := &http.Request{
	Method: "POST",
	URL: &url.URL{
		Scheme: "http",
		Host:   address,
		Path:   req.Endpoint(),
	},
	Header:        header,
	Body:          buf,
	ContentLength: int64(len(b)),
	Host:          address,
}

// make the request
hrsp, err := http.DefaultClient.Do(hreq.WithContext(ctx))
if err != nil {
	return errors.InternalServerError("go.micro.client", err.Error())
}
defer hrsp.Body.Close()

// parse response
b, err = ioutil.ReadAll(hrsp.Body)
if err != nil {
	return errors.InternalServerError("go.micro.client", err.Error())
}

// unmarshal
if err := cf.Unmarshal(b, rsp); err != nil {
	return errors.InternalServerError("go.micro.client", err.Error())
}

return nil

}
`
see code for ‘Method: "POST",’ ， it forced  as 'post'
since only the variable "endpoint" can be used to build HTTP requests through the "client interface".
code
type Client interface { 	........ 	NewRequest(service, endpoint string, req interface{}, reqOpts ...RequestOption) Request 	...... }
Why the variable 'endpoint' can't do this as
req := client.NewRequest({microServiceCode}, '{httpMethod} {httpPath}', map[string]string{})
such as
req := client.NewRequest({microServiceCode}, 'GET /path', map[string]string{})
by the way,
the function  named 'Method' of  the Interface named Request can return 'Get'
and the function  named 'Endpoint' of  the Interface named Request can return '/path'

It's not an http client in the traditional sence like net/http is. The client package is a client for the go-micro service, which happens to use the HTTP transport to communicate, with the predefined POST method. If you want to call it with any other method you would have to use an HTTP client such as net/http

thanks's u


WIP: Bazel second iteration

This is a second attempt at switching to Bazel - implemented a lot more consistently and cleanly (hopefully!).
Still WIP - opening this PR to test in CI environment.
NONE

/assign
/hold

/retest

This is now all green.
Our e2e tests still depend on system-provided tools, e.g. helm and kubectl.
In a follow-up I'm going to remove this dependence, so our entire build/test/deploy process is managed and versioned using Bazel.
I'd appreciate if anyone can check out this branch and test out common targets (most of which are documented in the Makefile!)

The last piece here is gofmt. It is a part of the actual Go SDK, and as such is difficult to utilise via Bazel (see https://github.com/bazelbuild/rules_go/issues/511).
For now, we still require gofmt to be available as part of the verify_fmt target. This means users will need a go version matching our CI to get accurate test results locally.
I am investigating ways we can work around this - we may have to switch to some out-of-tree tool like goimports for this, which wouldn't be too bad, but would mean our format could technically be out-of-sync with the go version specified (this doesn't seem like a massive issue)

I'm happy with trying this out for a release or two. If it turns out the benefits don't outweigh the extra complexity, we can roll back to pure makefiles and bash.

I'm going to create a post on the mailing list to explain that we're trying this out.
If people have significant issues, I'd love to hear them so we can assess whether this is a good move long term or not.

Rebased and ran bazel run //hack:update-codegen to resolve openapi and bazel issues 😄 re-adding lgtm