BUG REPORT: Since update to 3.1.0 Aerial doesn't save preferences reverting back everything!

Hi there!

[ ] Mac model: MacBook Pro 2016 4xTBP3 - Intel i5 2,9Ghz - 512 SSD
[ ] macOS version: MacOS Monterey 12.5.1
[ ] Monitor setup: Integrated Retina Display

Debug File:
AerialLog copia.txt

Description of issue:
So Aerial was working wonderfully on my actual and current setup until 3.1.0 😔 that I updated with the aerial companion app
(version 1.2.2). Also wanted to mention that I currently have only 30 GB left of storage on SSD... FYI.
I uninstalled the aerial companion app and reinstalled it, but the same bug goes on and on... so what happens exactly?
When I set up everything the way I want (a simple personalization where I only use 5 fix aerial city videos  -videos that I save on my machine- plus deactivating any kind of automatic and new downloads) the setup menu, does not save my choices after all and reverts to its initial state when going back to the menu or launching the screensaver. Also worth to mention that the "welcome splash screen" comes up over and over again when I go into the settings panel to personalize my screensaver settings.
I obviously would like to revert to 3.0.9 -found the file here- but first I'll wait for some feedback as it might be something else since I'm really not a PRO in these fields... who knows what I might be missing... therefore I attached the debug log file so you can have a look at it.
I just LOVE your screensaver and really appreciate your huge effort. 🔝 🔝 🔝
Hopefully you can help me out with this issue! 😃

Hey @LoopNCB
That's super not good. I'll try to have a look at your file tonight and let you know. By any chance, did you move your user folder to some other place ? I did try to improve settings but may have messed something for some people.
If there's anything peculiar about your machine (some sort of corporate management, stuff moved to external drives, etc etc), let me know, it may help me narrow it down !
Sorry for the issue again :(

@LoopNCB on top of what's said above, can you try opening again the settings, with debug mode enabled, and send me the log afterwards ? I'm missing the interesting bits (the debug startup), but if you can't save settings at all that might be an issue.

It's not so much you doing it wrong that some mess I made. I'm not sure why still. If I make a quick build, would you mind installing it and testing it quickly ? I'll force enable the logs and it should give me a better sense of what's happenning.
(short term fix in any case, revert to 3.0.9)

@LoopNCB if you can,

Delete your current AerialLog.txt

Then please install  this build :

https://github.com/glouel/Aerial/releases/download/v3.1.1test1/Aerial.saver.zip

After installing try opening settings, then grab the AerialLog.txt, this one should contain what I need. Thanks for your help !

Thanks a whole lot for your patience, I think I'm getting somewhere. What seems to happen is that the new preferences file isn't found, so you always revert to a brand new configuration. But some details don't fully make sense yet.
If you are able to, can you open "Terminal" on your mac and type :
cd ~/Library/Containers/com.apple.ScreenSaver.Engine.legacyScreenSaver/Data/Library/Preferences/

then, assuming there was no error, type :
ls -la

and paste the output
Optional step :
cd ByHost

then
ls -la

and paste the output too.
I'll try to think of a way to workaround whatever seems not to be working, but it's a bit puzzling yet ! Thanks for your help !

Same issue here though my home folder is not on my boot Volume. Very strong agree that the application is not setting/finding a preference file so I'm just getting a fresh experience on every launch.
Not to cross streams but here's the output you asked for though it's not much at all.

Media-Mac-3:~ media$ cd ~/Library/Containers/com.apple.ScreenSaver.Engine.legacyScreenSaver/Data/Library/Preferences/
Media-Mac-3:Preferences media$ ls -la
total 48
drwx------   6 media  admin   204 Aug 21 15:03 .
drwx------  31 media  admin  1054 Jul 30 21:54 ..
drwx------@  3 media  staff   102 Aug 26 11:01 ByHost
-rw-------   1 media  staff   197 Aug 21 15:03 com.apple.ScreenSaver.Engine.legacyScreenSaver.plist
lrwxr-xr-x   1 media  admin    51 Nov 11  2020 com.apple.security.plist -> ../../../../../Preferences/com.apple.security.plist
lrwxr-xr-x   1 media  admin    58 Nov 11  2020 com.apple.security_common.plist -> ../../../../../Preferences/com.apple.security_common.plist
Media-Mac-3:Preferences media$ cd ./ByHost
Media-Mac-3:ByHost media$ ls -la
total 16
drwx------@ 3 media  staff   102 Aug 26 11:01 .
drwx------  6 media  admin   204 Aug 21 15:03 ..
-rw-r--r--@ 1 media  staff  6148 Aug 26 11:01 .DS_Store
Media-Mac-3:ByHost media$

@dankudos thanks a whole lot for the info, I believe I may have fixed the issue for home folder not on boot volume for upcoming version.
HOWEVER
Your log output is seriously concerning, this is where your "old" and new aerial settings should be, yet they aren't. There's something I'm not understanding here, and I think it may be some x86/arm container mess biting back.
@dankudos can you try this for me ?

open Aerial settings, Advanced, Show log in finder
in finder right click/ctrl click AerialLog.txt to show the contextual menu, press alt to see the alternate options, and 2/3 down the list, press "Copy AerialLog.txt" as path and give me that path ?
from that Finder window, go two level down, then one level up to Preferences/ then ByHost/
what's in that folder ?

I was just getting read to double post that I've provided you bad info due to my own troubleshooting. I manually removed the old pref file earlier to confirm it wasn't "stuck" or otherwise the issue. As you see above no new file was generated. This is why you see the modified date on the ByHost folder as today. I've moved that file back and provided similar output below. You can see this pref file has not been modified since the 3.1 update on 12 August.

Media-Mac-3:ByHost media$ cd ~/Library/Containers/com.apple.ScreenSaver.Engine.legacyScreenSaver/Data/Library/Preferences/
Media-Mac-3:Preferences media$ ls -la
total 48
drwx------   6 media  admin   204 Aug 21 15:03 .
drwx------  32 media  admin  1088 Aug 26 11:58 ..
drwx------@  4 media  staff   136 Aug 26 12:02 ByHost
-rw-------   1 media  staff   197 Aug 21 15:03 com.apple.ScreenSaver.Engine.legacyScreenSaver.plist
lrwxr-xr-x   1 media  admin    51 Nov 11  2020 com.apple.security.plist -> ../../../../../Preferences/com.apple.security.plist
lrwxr-xr-x   1 media  admin    58 Nov 11  2020 com.apple.security_common.plist -> ../../../../../Preferences/com.apple.security_common.plist
Media-Mac-3:Preferences media$ cd ./ByHost
Media-Mac-3:ByHost media$ ls -la
total 32
drwx------@ 4 media  staff   136 Aug 26 12:02 .
drwx------  6 media  admin   204 Aug 21 15:03 ..
-rw-r--r--@ 1 media  staff  6148 Aug 26 12:02 .DS_Store
-rw-------  1 media  staff  6308 Aug 12 11:25 com.JohnCoates.Aerial.A3176DE3-6086-5767-B71F-21EDD8946713.plist
Media-Mac-3:ByHost media$

Ha I see, well this is less concerning then !
I'm not 100% sure I fully understand the issue yet, I tried to do things more properly and hopefully it will be ok this time.
I'll have a test build ready in a few minutes if you feel like testing it. Thanks for your help in any case !

@dankudos if you can please try installing this :
https://github.com/glouel/Aerial/releases/download/v3.1.1test2/Aerial.saver.zip
Hopefully it will work as is. If not, I'm interested if there's an aerial file in Preferences (not byHost) created this time or not.

Seems to be working as expected this time. It did not launch the Welcome setup dialog and read in my existing prefs as well as taking new settings. Also, there's your name on a pref in the desired directory. Grand success!

Media-Mac-3:ByHost media$ cd ~/Library/Containers/com.apple.ScreenSaver.Engine.legacyScreenSaver/Data/Library/Preferences/
Media-Mac-3:Preferences media$ ls -la
total 80
drwx------   7 media  admin   238 Aug 26 12:55 .
drwx------  32 media  admin  1088 Aug 26 11:58 ..
drwx------@  4 media  staff   136 Aug 26 12:02 ByHost
-rw-------   1 media  staff   197 Aug 21 15:03 com.apple.ScreenSaver.Engine.legacyScreenSaver.plist
lrwxr-xr-x   1 media  admin    51 Nov 11  2020 com.apple.security.plist -> ../../../../../Preferences/com.apple.security.plist
lrwxr-xr-x   1 media  admin    58 Nov 11  2020 com.apple.security_common.plist -> ../../../../../Preferences/com.apple.security_common.plist
-rw-------   1 media  staff  8955 Aug 26 12:55 com.glouel.Aerial.plist
Media-Mac-3:Preferences media$

🥳
Hopefully @LoopNCB will confirm this build also fix the original issue.
I had to move the settings from the ByHost directory to one level down in order to be able to share settings with Aerial Companion (Companion couldn't use the ByHost file, sadly). I thought I had it correct, but I made a huge mess 😅
From now on settings will be using this shared file.
Waiting for confirmation that the original issue is fixed and I'll release 3.1.1. Thanks for taking the time to test the new build !

Thanks @LoopNCB for the confirmation that it works now, and no worries, I do appreciate greatly when people take the time to report issues.
Thanks both for your help fixing this mess, I just released 3.1.1 with the fix (it's identical to test2, except logs are no longer enabled by default), so you can resume using/updating via Companion.
Take care and thanks for your kind words !


[Discussion/Feedback] VLM + Multimodal benchmarking

This is an issue to discuss the feasibility / desirability of multimodal benchmarks to be included and supported in lm-eval. With the increase in multimodal (V)LMs and in benchmarks like MMMU designed to test these models, it's worth discussing either whether lm-eval is easily extensible to these tests or whether we wish to have these in scope for the library.

There is already a fork of lm-evalaution-harness built by the LLaVa team called lmms-eval that is focused on VLM evaluation that can serve a PoC. For what is worth, I already have a private fork of lm-eval that has added LLaVa support and is working with MMMU and the LLava 1.5 7B model through the original codebase. I think the main considerations of an implementation are the following:

How tightly do we want to couple VLM and MM benchmarks? Should VLM only be able to run MM benchmarks should we enable both LM and VLM to run MM benchmarks. Do we need to ensure the VLM can run text -only benchmarks
Are VLM first party citizens? Do we need to focus on optimizations for these? For example VLLM just introduced VLM support a couple of days ago. Do we need to pull the latest benchmarks
Significant effort needs to be invested in optimization of the VLMM benchmarks as image data/video data in particle is very big, at what point does it becomes a problem

If the project owners are aligned on these questions, I can clean up and submit a PR for my LLaVa implementation.

Wanted to follow up

Just wanted to give a heads up that I started the PR.


Mategol add me i need help beterbet_07918

Describe the bug
A clear and concise description of what the bug is.
Just loads Source_Prepared
To Reproduce
Steps to reproduce the behavior:
Just run builder and it comes up with that

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
It makes the exe
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):
Windows 11

OS: [e.g. Windows 11, Windows 10]
Python Version: [e.g. 3.11.3]
3.12.7
Additional context
Add any other context about the problem here.

Hey, just use the ModdedPysilon version, it covers every bug and it make sure your discord account never get banned.
Leave me a star if you like my project ;)


Can state change trigger something in IFTTT? WebHooks?

I have the plugin installed and configured for 2 phones in our house. It seems to be working ok(Contact sensor opens when person leaves and closes when they return).
What I would like to do is to trigger either IFTTT or be able to send a web hook when the state of the contact sensor changes. I realize I can create an IOS automation that will sense the state change, but I don't know how to get to IFTTT from there either. :-(
Any suggestions gratefully accepted....

Never mind. I figured it out :-)


Add convenience constructor methods to taproot classes

[ ] add Node(left, right) constructor (https://github.com/bitcoinops/taproot-workshop/pull/19#discussion_r326712417)
[ ] add TapLeaf(descriptor_string) constructor (https://github.com/bitcoinops/taproot-workshop/pull/19#discussion_r326712118)

Suggested by @bitschmidty

add Node(left, right) constructor

Implemented in #137


create collectible listings table

Table to hold information on a listing for querying and display purposes
CollectibleListings

id - uuid
collectibleId - FK Collectible.id
price - number
type - [purchase, auction]
expiresAt - Date
createdAt - Date
updatedAt - Date

Could potentially expand this table down the line to include things like:

Flag for archived or not (current strategy is to delete row when listing removed or completed)
Flag for status, potentially to hold an "in flight" state if someone else is already in the purchase flow for the Listing

Flag for archived or not (current strategy is to delete row when listing removed or completed)
Flag for status, potentially to hold an "in flight" state if someone else is already in the purchase flow for the Listing

Maybe archived could be a type in addition to purchase/auction? Assuming the user couldn't ever relist an archived listing.
Status could be helpful, though we could also convey on the release page. If we updated the status anytime a purchase was started, we'd have to make sure that was updated whenever they left, and the data could be inaccurate on the releases page depending on when the purchase was started. 🤔


How do I get the code for the mobile demo?

It was turned into an example called gyro here is the code.

Thank you very much, but I also want to consult, I met problems when doing the project, add a set of ice with the world. The add (world, the stack), and then delete the remove one by one, and then continue to add also with the world. The add (world, stack) ice can not good to add, what reason be?

Please can you open a new thread that includes an example for your issue? Thanks.


Multiple binlog player source types

LGTM

Wait the unit tests are failing on this file, that can't be a coincidence. @tirsen can you make sure they pass?


How do I keep my variables declarations DRY?

First of all, I would like to thank you for this tool.
Problem
I am having a problem trying to keep the variables.tf file DRY. Here's my directories structure:
.
├── common.shared.yaml
└── iam
│   ├── role-01
│   │   ├── main.tf
│   │   ├── outputs.tf
│   │   ├── terragrunt.hcl
│   │   └── variables.tf
│   └── role-02
│       ├── main.tf
│       ├── outputs.tf
│       ├── terragrunt.hcl
│       └── variables.tf
└ terragrunt.hcl

In my root terragrunt.hcl:
...

locals {
  common_vars_yaml = "common.shared.yaml"
}

inputs = merge(
  yamldecode(
    file(
      find_in_parent_folders(local.common_vars_yaml)
    )
  )
)

...

Even though I import these inputs in every terragrunt.hcl for each service, I still need to repeat all variables declarations.
Example
# common.shared.yaml
obj:
  str: value

# variables.tf
variable "obj" {
  type = object({
    str = string
  })
}

The last part above (variables.tf) is copied and pasted for every service that needs variables present in common.shared.yaml.
How could I solve it? Are there any patterns or built-in functions?

Hi,
maybe exists another approach but I was thinking about using generate block
generate = {
  gen = {
    path      = "variables.tf"
    if_exists = "overwrite"
    contents  = <<EOF
# variables.tf
variable "obj" {
  type = object({
    str = string
  })
}
EOF
  }
}

and include it in HCL:
include {
  path = "${get_terragrunt_dir()}/../variables.hcl"
}


Writers' guide to stumptown

We should have a document describing what stumptown-content looks like from the point of view of a technical writer. This should explain:

at a high level, what stumptown-content and stumptown-renderer are, and how they relate
at a high level, which bits of the stumptown-content repo are of interest to writers
what kind of files these bits contain
what recipes are for
how reference pages are written
how guide pages are written
how landing pages are written
how sidebars are specified
what all the currently designed ingredients are, and how they are to be written (e.g. browser_compatibility, examples, etc etc)
how to run stumptown-renderer to see the rendered version of the documentation pages

It does not need to go into the details of build-json or what the JSON interface looks like.

Acceptance criteria:

doc is written and reviewed by stumptown-content team

Parking until strategy is clearer.

FWIW this is done: https://github.com/mdn/stumptown-content/pull/197.


Add port option.

Hey @matt-42,
Thanks for the contribution. Can you please provide some context on why it's needed? I mean, why would someone want to use a non-default PostgreSQL port on their CI? Is there some use case for that?

I just published ikalnytskyi/action-setup-postgres@v2 with this enhancement. Thank you!


feat: pp.numeralTypes option for printing number literals with type ascriptions

This was a pretty printer option in the Lean 3 community edition, and it was useful for debugging how arithmetic expressions elaborated or have been otherwise transformed. For example, the real numbers have two exponentiation operations (Real -> Real -> Real and Real -> Nat -> Real) and it is helpful knowing at a glance which of these is x ^ 2.

I am a bit concerned about the name numeralTypes and its discoverability because, as far as I know, as numeric literals are not called "numerals" anywhere else in Lean. Thus, it would be hard to connect the dots between the option in an autocomplete list and its functionality. Maybe numLitTypes instead?

Sebastian suggested that we go through the RFC process for this, so I'll put this back to draft and we can discuss these issues on Zulip.

I'm re-opening this now that it's implementing an RFC.
Something I checked by hand is that the delaborators ensure that each subexpression is still hoverable in the infoview.

This looks good.

Could you add a sentence to RELEASES.md for visibility?
You wrote such great text on the RFC (manual level explanation, reference level explanation). Should any of that text be copied into docs/? Happy either way.

@semorrison I added a RELEASES entry. Regarding the RFC, I'm planning on eventually going through the RFCs I've written and copying the explanations to relevant parts of the docs (in future PRs).


fix: platform org docs use old resource names

Describe your changes
The documentation for harness_platform_organization uses the wrong resource names. This leads to errors when new users are copy/pasting from the documentation.
Comment Triggers

PR Check triggers

Build: trigger build
Sub Category Field Check: trigger subcategoryfieldcheck

trigger build

trigger subcategoryfieldcheck

@rssnyder We need to regenerate docs once this is done. Can you please help make your branch public so we can do the same?

closing the pr as it has been updated in latest version https://registry.terraform.io/providers/harness/harness/latest/docs/resources/platform_organization


Logo

Apache Tuweni doesn't have a logo.
Tuweni is taken after the Lakota language word Tuweni which both means "nobody" and "never".
See our FAQ here: https://tuweni.apache.org/faq/
See also reference to tuweni here: https://www.wolakotaproject.org/lakota-pronunciation-glossary/
A logo would incorporate the ASF feather and the whole name "Apache Tuweni".
See press kit here: https://apache.org/foundation/press/kit/#:~:text=The “Powered By Apache” logo,the new ASF brand identity.
All ASF logos here for inspiration: https://apache.org/logos/

Logo is there now! See #149


HSEARCH-1999 Test

We couldn't reproduce it but I'd prefer to merge the test than throwing it away

Merged with a change to the commit message https://github.com/hibernate/hibernate-search/pull/924

https://hibernate.atlassian.net/browse/HSEARCH-1999


DATABASE STRUCTURE

I would like to understand how the structure of the database generated in the code works.
I took a look and realized that there are three tables: 'stantment', 'tag' and 'tag_association'
I want to understand what the tag and tag_association table is for, and I would also like to understand the columns in the stantment table, as there is the "search_text" column that breaks my question and places initials in front:
text -> CLICQ is a complete platform for product and process quality management of your business.
column -> NNP: click NNP: a JJ: platform NN: complete NN: to DT: management NN: prosecutor NN: quality NN: from IN: FW products: VBP processes: your VB: business
What would each initials?

This might give an idea on database https://chatterbot.readthedocs.io/en/stable/conversations.html#statement-response-relationship
the search_text column is a string of text containing part-of-speech, lemma pairs.


for youtube share url not getting details like title, image etc

While sharing youtube URL from the app it is not getting a proper response
https://youtu.be/MZWina8plDk

Also facing the same problem. Any solution?

I'm trying to diagnose but if you look at the htmlCode sent to performPageCrawling(), the title is:
<title>YouTube</title>

So now all titles show up as "YouTube" instead of the original. I think it might have something to do with following the redirects.

I'm trying to diagnose but if you look at the htmlCode sent to performPageCrawling(), the title is:
<title>YouTube</title>

So now all titles show up as "YouTube" instead of the original. I think it might have something to do with following the redirects.

Yup, seeing this also!

Yup, seeing this also!

Seems to be fixed with the new version.


Snapshot Menu:  Menu Disappears on Hover

Using the query tool, draw a marquee that will yield large results.
Populate fields in the Query Tool, click Submit.
While the query is running and when the snapshot is added to the timeline, click the gear icon to display the snapshot menu.
Mouseover and click Save to Bookmarks.
The menu disappears when hovering over to click and select a menu item.
The snapshot menu should not preemptively disappear.

It's not very responsive with too much data, but it doesn't go away.

The responsiveness of the menu?  The lack of responsiveness exists prior to the z-index change, but at least one is able to hover over and click on a menu list item vs. the previous situation

Passed on develop.


PENV-458: set correct jet id for db search

- What I did

for first jet with depth=0 JetDropID func return jet with empty jetID
to return result to API, use ToString and JetIDToString methods of JetDropID

- How to verify it
Run tests

Codecov Report

Merging #94 into master will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #94      +/-   ##
==========================================
- Coverage   75.32%   75.30%   -0.03%
==========================================
  Files          45       45
  Lines        2886     2891       +5
==========================================
+ Hits         2174     2177       +3
- Misses        566      570       +4
+ Partials      146      144       -2

Impacted Files
Coverage Δ

api/mappers.go
100.00% <100.00%> (ø)

etl/models/models.go
100.00% <100.00%> (+8.00%)
:arrow_up:

etl/extractor/platform_impl.go
70.99% <0.00%> (-4.59%)
:arrow_down:

instrumentation/converter/platform.go
100.00% <0.00%> (+16.66%)
:arrow_up:

Continue to review full report at Codecov.

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


Recursion error in DependecyUtils

I found an error when using the DependencyUtils.  It seems like it's getting stuck in a loop in the dependency graph and the recursion blows the stack.  This is a snippet of the stack trace:
Caused by: java.lang.StackOverflowError
    at scala.runtime.BoxesRunTime.boxToInteger(BoxesRunTime.java:69)
    at edu.arizona.sista.processors.Sentence.dependencies(Document.scala:73)
    at edu.arizona.sista.utils.DependencyUtils$.getIncoming$2(DependencyUtils.scala:129)
    at edu.arizona.sista.utils.DependencyUtils$.followTrailOut$2(DependencyUtils.scala:145)
    at edu.arizona.sista.utils.DependencyUtils$.edu$arizona$sista$utils$DependencyUtils$$followTrail$3(DependencyUtils.scala:138)
    at edu.arizona.sista.utils.DependencyUtils$.followTrailOut$2(DependencyUtils.scala:150)

Is this caused by some cycle in the dependency graph?
Becky, can you please show us how to replicate this? Is there a test case that can be made, which would fail.

Yes - using Agiga.  In the file I am using (/data/nlp/corpora/agiga/data/xml/afp_eng_199405.xml.gz), sentence 127 ("The evacuees who arrived at Lanarca airport later boarded a British Airways airliner bound for Gatwick airport , in southern Britain .") has the following dependency parse.
Note the loop in the outgoing deps between tokens 1 and 3:
scala> doc.sentences(127).words
res1: Array[String] = Array(The, evacuees, who, arrived, at, Lanarca, airport, later, boarded, a, British, Airways, airliner, bound, for, Gatwick, airport, ,, in, southern, Britain, .)
scala> doc.sentences(127).dependencies.get
res2: edu.arizona.sista.struct.DirectedGraph[String] =
roots: 8
outgoing:
0:
1: (0,det) (3,rcmod)
2:
3: (1,nsubj) (6,prep_at)
4:
5:
6: (5,nn)
7:
8: (1,nsubj) (7,advmod) (12,dobj) (20,prep_in)
9:
10:
11:
12: (9,det) (10,nn) (11,nn) (13,partmod)
13: (16,prep_for)
14:
15:
16: (15,nn)
17:
18:
19:
20: (19,amod)
incoming:
0: (1,det)
1: (3,nsubj) (8,nsubj)
2:
3: (1,rcmod)
4:
5: (6,nn)
6: (3,prep_at)
7: (8,advmod)
8:
9: (12,det)
10: (12,nn)
11: (12,nn)
12: (8,dobj)
13: (12,partmod)
14:
15: (16,nn)
16: (13,prep_for)
17:
18:
19: (20,amod)
20: (8,prep_in)

ps - if you are waiting for me to do more (i.e. make a test) please let me know and I'll try to figure out how to do that

Yes, can you please add a unit test for this in processors (in a separate
branch), which would fail on that sentence?
I'd like to see the exact code you use.
On Wed, Mar 2, 2016 at 1:22 PM, bsharpataz<EMAIL_ADDRESS>wrote:

ps - if you are waiting for me to do more (i.e. make a test) please let me
know and I'll try to figure out how to do that
—
Reply to this email directly or view it on GitHub
https://github.com/clulab/processors/issues/43#issuecomment-191412242.

I added a test for this issue.  It currently fails.  It's at branch issue43

Also, in the same sentence, the same code, there is also an infinite loop (in the Interval(5,7)). It doesn't throw a stack overflow error, it seems like an infinite loop, and I don't know how to write tests for infinite loops.

@marcovzla is going to look at this today.


Create email/content for HL7 senders that are no longer sending us data

Problem statement
Some HL7 senders on the COVID pipeline have stopped sending us production data. Before we finalize our sender migration plan, we want to confirm whether or not we should plan on migrating these senders.
What you need to know

Sender Audit for more information about these senders

List of CP HL7 senders to contact:

Lifepoint
Prescryptive
Medical Network Solutions
Color-labs
Primary
Limsabc-nova
Signetic
Mayo-clinic
Aegis
Nerolab

MedArbor is an HL7 Sender that has NOT finished onboarding to the CP, but they are very close (only need approval through WA remaining, I believe). Angela has been the team member in charge of onboarding them last year, and our sync calls have moved to an "as-needed" basis. Since we are still 4+ months away from onboarding a sender to the UP, I don't think we should stop the train from moving since the work they are doing would be necessary for the UP as well, BUT I think we should try to find out:

Are they capable of (or interested in) sending any other conditions beyond COVID through ReportStream?

Answers we'd like to know

Why have they stopped sending RS data?
Do they plan on sending COVID data in the future?
Do they have any other conditions (beyond COVID) that they'd be interested in sending through RS?

Acceptance criteria

[x] Develop email content for CP HL7 senders that have stopped sending us production data
[x] Develop content/wording for communicating with MedArbor and how we want to communicate that
[x] Identify high-level plan on how/when to send those emails

Content doc


Update dependencies and code style rules; remove .NET Framework 4.6 support

Types of changes

[ ] Bugfix
[ ] Enhancement (new and/or improved behavior)
[ ] Refactoring (no functional changes)
[x] Code style update (no functional changes)
[x] Dependencies added, updated, or removed
[ ] Build related changes
[ ] Repository infrastructure changes (e.g. changes to issue / PR templates)
[ ] CI related changes
[ ] Documentation update
[ ] Other (please describe below)

Proposed changes / fixed issues
Remove support for .NET Framework 4.6, leaving only .NET Standard 2.0.
Update all dependencies. Rebuild with .NET SDK 6.0.
Checklist

[x] My contribution is my original work
[ ] My contribution, or part of it, comes from projects with an MIT-compatible license AND I have updated the THIRD-PARTY-NOTICES file accordingly
[ ] I have added and/or updated related documentation (if applicable)
[x] I have updated the "Unreleased changes" section in CHANGELOG.md according to the modifications I made

Codecov Report

Merging #54 (507458d) into main (1ee3ad9) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #54   +/-   ##
=======================================
  Coverage   65.42%   65.42%
=======================================
  Files           3        3
  Lines         107      107
  Branches       22       22
=======================================
  Hits           70       70
  Misses         26       26
  Partials       11       11

Impacted Files
Coverage Δ

src/Cecil.XmlDocNames/MemberReferenceExtensions.cs
0.00% <ø> (ø)

...Cecil.XmlDocNames/StringBuilderExtensions-Cecil.cs
69.23% <ø> (ø)

...ecil.XmlDocNames/StringBuilderExtensions-common.cs
53.84% <ø> (ø)

Continue to review full report at Codecov.

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


Compatible with Natural Language Dates

In the front-matter, there is a conflict with the Natural Language Dates plugin. When using the "@" symbol to select a time, the Natural Language Dates plugin is not providing the expected prompt. For selecting time, Natural Language Dates is more meaningful. An option to config ignore character will be userfull

Thank you very much


Error running Logshark 4.1

I get an error when I try to run Logshark
D:>Logshark D:\tabserverlogs.zip
Loading Logshark user configuration..
Failed to initialize Logshark request processor!
Please check to make sure that the results database was configured correctly. (F
ailed to open connection to localhost:5432: 28000: no pg_hba.conf entry for host
"::1", user "logshark", database "postgres", SSL off)

Each PostgreSQL database has pg_hba.conf file which describes who can connect to what database, from which IP addresses and how they should authenticate. The error message means none of the records in pg_hba.conf allow to connect user "logshark" to database "postgres" from the "::1" address (IPv6 localhost).
The easiest solution is to add a record to the pg_hba.conf file allowing "logshark" to connect from this IP using username and password. Then restart PostgreSQL.
Here's the link to Postgres documentation re this file: https://www.postgresql.org/docs/current/auth-pg-hba-conf.html

Closing this one. If the issue persists - please let us know.


Redirecting issue of DONATE NOW button

When we click on DONATE NOW button , it is redirecting us to home page while it should not redirect us to home page , instead it should take us on new page . I think this is an issue and I want to solve this issue. please assign this issue to me

Hello @himanshu007-creator , I can add an actually working donate button and I would like to work on it under GSSOC . Clicking on the Donate Now would redirect to a live page for donations

Work in progress. so not an issue.


Plan out Unity 6.0 and XRI 3.0 support

Things that must be done before release:

Remove obsolete methods
investigate supporting downstream unity versions (Still support xri 2?)
per Adam - will be a challenge to support XRI 2 & 3 together

Will be a breaking change and require major version bump
Question:  Continue support for MRTK v3 packages?  Or deprecate?

@ms-RistoRK will be looking at this upgrade for the next couple sprints.
Expected to require major release bump
Talked to Unity:
Namespaces have changed
process is to upgrade to pre-release 2, THEN upgrade to release and then not running the migration tool.

Would it make sense to upgrade the project to 2022 LTS and 2023 LTS first?  I see some slight upgrades for those versions.

Created feature/XRI3 branch on 4/17/24.

Risto is currently working on a new MRTK rig prefab without controller behaviors.
Old prefab will remain but deprecated

Before we ship this version....deprecated MRTK3 method/classes should be removed.

https://github.com/MixedRealityToolkit/MixedRealityToolkit-Unity/issues/531

Holding in discussion until the XRI 3 PR is committed

Updates the "details" of the issue, so goals are clearer

XRI3 branch still has some usages of deprecated XRI classes
ObjectManipulator

C:\dev\repos\mrtk3-atm\org.mixedrealitytoolkit.spatialmanipulation\ObjectManipulator\ObjectManipulator.cs(1065,17): warning CS0618: 'XRBaseInputInteractor.xrController' is obsolete: 'xrController has been deprecated in version 3.0.0.'
C:\dev\repos\mrtk3-atm\org.mixedrealitytoolkit.spatialmanipulation\ObjectManipulator\ObjectManipulator.cs(1065,54): warning CS0618: 'ActionBasedController' is obsolete: 'ActionBasedController has been deprecated in version 3.0.0. Its functionality has been distributed into different components.'

RiggedHandMeshVisualizer

C:\dev\repos\mrtk3-atm\org.mixedrealitytoolkit.input\Visualizers\RiggedHandVisualizer\RiggedHandMeshVisualizer.cs(77,17): warning CS0618: 'XRBaseController' is obsolete: 'XRBaseController has been deprecated in version 3.0.0. Its functionality has been distributed into different components.'

FlatScreenModeDetector

C:\dev\repos\mrtk3-atm\org.mixedrealitytoolkit.input\InteractionModes\FlatScreenModeDetector.cs(34,13): warning CS0618: 'FlatScreenModeDetector.controllerLookup' is obsolete: 'Deprecated, please use MixedReality.Toolkit.Input.TrackedPoseDriverLookup instead.'
C:\dev\repos\mrtk3-atm\org.mixedrealitytoolkit.input\InteractionModes\FlatScreenModeDetector.cs(34,47): warning CS0618: 'ControllerLookup' is obsolete: 'Deprecated, please use MixedReality.Toolkit.Input.TrackedPoseDriverLookup instead.'

InteractionModeManager

C:\dev\repos\mrtk3-atm\org.mixedrealitytoolkit.input\InteractionModes\InteractionModeManager.cs(74,87): warning CS0618: 'XRController' is obsolete: 'XRController has been deprecated in version 3.0.0. Its functionality has been distributed into different components.'
C:\dev\repos\mrtk3-atm\org.mixedrealitytoolkit.input\InteractionModes\InteractionModeManager.cs(74,22): warning CS0618: 'XRController' is obsolete: 'XRController has been deprecated in version 3.0.0. Its functionality has been distributed into different components.'
C:\dev\repos\mrtk3-atm\org.mixedrealitytoolkit.input\InteractionModes\InteractionModeManager.cs(215,36): warning CS0618: 'XRBaseInputInteractor.xrController' is obsolete: 'xrController has been deprecated in version 3.0.0.'
C:\dev\repos\mrtk3-atm\org.mixedrealitytoolkit.input\InteractionModes\InteractionModeManager.cs(252,36): warning CS0618: 'XRBaseInputInteractor.xrController' is obsolete: 'xrController has been deprecated in version 3.0.0.'

HandJointInteractor

C:\dev\repos\mrtk3-atm\org.mixedrealitytoolkit.input\Interactors\HandJointInteractor.cs(35,53): warning CS0618: 'XRBaseInputInteractor.xrController' is obsolete: 'xrController has been deprecated in version 3.0.0.'
C:\dev\repos\mrtk3-atm\org.mixedrealitytoolkit.input\Interactors\HandJointInteractor.cs(35,69): warning CS0612: 'ArticulatedHandController' is obsolete
C:\dev\repos\mrtk3-atm\org.mixedrealitytoolkit.input\Interactors\HandJointInteractor.cs(52,43): warning CS0618: 'XRBaseInputInteractor.xrController' is obsolete: 'xrController has been deprecated in version 3.0.0.'

MRTKRayInteractor

C:\dev\repos\mrtk3-atm\org.mixedrealitytoolkit.input\Interactors\Ray\MRTKRayInteractor.cs(53,53): warning CS0618: 'XRBaseInputInteractor.isUISelectActive' is obsolete: 'isUISelectActive has been deprecated in version 3.0.0. Use a serialized XRInputButtonProvider to read button input instead.'

XRI3 branch still has some usages of deprecated XRI classes
ObjectManipulator

C:\dev\repos\mrtk3-atm\org.mixedrealitytoolkit.spatialmanipulation\ObjectManipulator\ObjectManipulator.cs(1065,17): warning CS0618: 'XRBaseInputInteractor.xrController' is obsolete: 'xrController has been deprecated in version 3.0.0.'
C:\dev\repos\mrtk3-atm\org.mixedrealitytoolkit.spatialmanipulation\ObjectManipulator\ObjectManipulator.cs(1065,54): warning CS0618: 'ActionBasedController' is obsolete: 'ActionBasedController has been deprecated in version 3.0.0. Its functionality has been distributed into different components.'

RiggedHandMeshVisualizer

C:\dev\repos\mrtk3-atm\org.mixedrealitytoolkit.input\Visualizers\RiggedHandVisualizer\RiggedHandMeshVisualizer.cs(77,17): warning CS0618: 'XRBaseController' is obsolete: 'XRBaseController has been deprecated in version 3.0.0. Its functionality has been distributed into different components.'

FlatScreenModeDetector

C:\dev\repos\mrtk3-atm\org.mixedrealitytoolkit.input\InteractionModes\FlatScreenModeDetector.cs(34,13): warning CS0618: 'FlatScreenModeDetector.controllerLookup' is obsolete: 'Deprecated, please use MixedReality.Toolkit.Input.TrackedPoseDriverLookup instead.'
C:\dev\repos\mrtk3-atm\org.mixedrealitytoolkit.input\InteractionModes\FlatScreenModeDetector.cs(34,47): warning CS0618: 'ControllerLookup' is obsolete: 'Deprecated, please use MixedReality.Toolkit.Input.TrackedPoseDriverLookup instead.'

InteractionModeManager

C:\dev\repos\mrtk3-atm\org.mixedrealitytoolkit.input\InteractionModes\InteractionModeManager.cs(74,87): warning CS0618: 'XRController' is obsolete: 'XRController has been deprecated in version 3.0.0. Its functionality has been distributed into different components.'
C:\dev\repos\mrtk3-atm\org.mixedrealitytoolkit.input\InteractionModes\InteractionModeManager.cs(74,22): warning CS0618: 'XRController' is obsolete: 'XRController has been deprecated in version 3.0.0. Its functionality has been distributed into different components.'
C:\dev\repos\mrtk3-atm\org.mixedrealitytoolkit.input\InteractionModes\InteractionModeManager.cs(215,36): warning CS0618: 'XRBaseInputInteractor.xrController' is obsolete: 'xrController has been deprecated in version 3.0.0.'
C:\dev\repos\mrtk3-atm\org.mixedrealitytoolkit.input\InteractionModes\InteractionModeManager.cs(252,36): warning CS0618: 'XRBaseInputInteractor.xrController' is obsolete: 'xrController has been deprecated in version 3.0.0.'

HandJointInteractor

C:\dev\repos\mrtk3-atm\org.mixedrealitytoolkit.input\Interactors\HandJointInteractor.cs(35,53): warning CS0618: 'XRBaseInputInteractor.xrController' is obsolete: 'xrController has been deprecated in version 3.0.0.'
C:\dev\repos\mrtk3-atm\org.mixedrealitytoolkit.input\Interactors\HandJointInteractor.cs(35,69): warning CS0612: 'ArticulatedHandController' is obsolete
C:\dev\repos\mrtk3-atm\org.mixedrealitytoolkit.input\Interactors\HandJointInteractor.cs(52,43): warning CS0618: 'XRBaseInputInteractor.xrController' is obsolete: 'xrController has been deprecated in version 3.0.0.'

MRTKRayInteractor

C:\dev\repos\mrtk3-atm\org.mixedrealitytoolkit.input\Interactors\Ray\MRTKRayInteractor.cs(53,53): warning CS0618: 'XRBaseInputInteractor.isUISelectActive' is obsolete: 'isUISelectActive has been deprecated in version 3.0.0. Use a serialized XRInputButtonProvider to read button input instead.'

These issues have been resolved in the XRI3 feature branch


use latest pgserver (compatible with 3.12)

NB also bumps to postgres 16, no issues there with tests.

This change is


pd: 📬 create tonic router in penumbra-app

this addresses a todo comment found in the pd entrypoint. we are now using tonic 0.10 so we can have the penumbra-app crate assemble a Routes, sparing us the recitation of every service.

i am going to rebase this once #3652 is merged.

i've got a broken smoke test to sort out, marking as a draft until i can sort that out.

NB: this includes #3687.

for the sake of github cross-referencing, this was a quick follow-on to #3652/#3627.


Errors on invoking the example

It's probably work in progress but I better let you know. On invoking example.ps1 I get the below errors. I am using PowerShell v3, if it matters.
Invoke-RestMethod : The 'Accept' header must be modified using the appropriate property or method.
Parameter name: name
At C:\-\GIT\flancy\example.ps1:45 char:1
+ Invoke-RestMethod -Uri http://localhost:8001/process -Headers @{'Accept'='applic ...
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Invoke-RestMethod], ArgumentException
    + FullyQualifiedErrorId : System.ArgumentException,Microsoft.PowerShell.Commands.InvokeRestMethodCommand

Invoke-RestMethod : The 'Accept' header must be modified using the appropriate property or method.
Parameter name: name
At C:\-\GIT\flancy\example.ps1:46 char:1
+ Invoke-RestMethod -Uri http://localhost:8001/process -Method Post -Body "Notepad ...
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Invoke-RestMethod], ArgumentException
    + FullyQualifiedErrorId : System.ArgumentException,Microsoft.PowerShell.Commands.InvokeRestMethodCommand
```

Tried on v4 and v5, it works. It may be v3 issue.

Yes is fixed in v4
https://connect.microsoft.com/PowerShell/feedback/details/757249/invoke-restmethod-accept-header

I'll probably just remove the header sample - while it's correct, it's not really necessary.  Invoke-RestMethod without it will still get the same results.  It's really only needed for something that implements the ability for the client to change the content type based on the requesting headers.

Wound up leaving the -headers.  The problem is that without them, the system freezes when running the script after the post example.  I can add a sleep to fix it, but instead I simply added a comment that in V3 and earlier you cannot use these headers.  I also added a note that it will work without those headers in v3.  No note about the unexpected behaviors that might happen, but I think it's ok.


fix missing handle 'update' event massage

Signed-off-by: fugr<EMAIL_ADDRESS>

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "fix-handle-update-event" git@github.com:fugr/swarm.git somewhere
$ cd somewhere
$ git rebase -i HEAD~2
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

@GordonTheTurtle  thanks a lot!

@fugr could you squash your commits into one commit please?

LGTM

@nishanttotla  how to do that

you can try like:
http://stackoverflow.com/questions/5189560/squash-my-last-x-commits-together-using-git

@nishanttotla   squash all commits into one commit,done

why tests failed?anyone knows?

Test failures seem unrelated. LGTM

LGTM. Thanks @fugr.


Show two apps with controls at the same time

I have controls from both home assistant and tinycam pro integrated in the powermenu. However (as it was in Android 11) I have to switch between the two apps using the dropdown menu.
Is it at all possible to have the control from both apps shown at the same time? I only use a few buttons so it would easily fit on the screen.
Maybe you can somehow add this as an option?
Thanks!

This is a limitation of the Controls library that's in AOSP, it's only built to bind to one app at once. It would take significant rewrite to change that, which I'm not prepared to do so I'm going to say this is out of scope, but anyone is welcome to fork the project and give it a shot.


Use the selected texture filtering in AnimatedSprite2D frame preview

Fixes #57550

See #57596 and the discussion there, I'm not sure this fix handles everything that should be (inheritance of filtering mode, AnimatedSprite3D).

Oops ! Didn't see that other PR. My bad.
Indeed, this one only solves the AnimatedSprite2D issue, and automatically takes the value of texture_filter, and passes it to the SpriteFrames preview when it updates.
It doesn't make too much sense with AnimatedSprite3D, since texture_filter, is a CanvasItem property, which AS3D does not inherit from.
https://user-images.githubusercontent.com/3624853/221428755-ed2b216b-11be-4f89-a775-b40098f3f0b9.mp4
But I guess we can close this PR if it's not the right way to go, and the issue is being worked already.
Cheers !

Note that the SpriteFrames editor can be used for a single SpriteFrames resource, without AnimatedSprite{2D,3D} node.

I just pushed a few changes, and it mostly works, but there are a couple of edge cases which are yet to be fixed.
So now, the SpriteFrames resource gets it's filtering from the CanvasItem, if it's attached to one.
If it's a stand alone SpriteFrame, it gets its filtering option from the project setting.
Unfortunately the mapping done with the switch clause is not very elegant, but it's necessary. That's because CanvasItem::TextureFilter enums do not match the filter enum in the project settings. So a mapping is needed.
There are still 2 problems though.

On a stand alone SpriteFrame, only new animations inherit what's set in the project settings. Existing ones do not update. Need to find a way to update those.
For some reason, when the CanvasItem's filter is set to Inherit, the SpriteFrames does not get the same filter, while it works for all the other values.

I'm not sure if _update_library() is the right place to do this.

I'm not sure if _update_library() is the right place to do this.

What would you suggest ?

I would suggest SpriteFramesEditorPlugin::edit() as this is where the edited resource is changed. Also note that in the case of AnimatedSprite3D, some default filter must also be applied, as in the case of standalone SpriteFrames (because Node3D does not have a texture filter).
But personally, I think it's overcomplicated. I don't see any problem with TEXTURE_FILTER_NEAREST_WITH_MIPMAPS (#74341). See also #67426. There, initially the author suggested a separate setting, but in the end we decided that this was not necessary.
In the TileMap/TileSet editor, this is also not configurable and does not depend on the project settings:
https://github.com/godotengine/godot/blob/61d2c855114c824f5ca27ded0a1fa71cc7b21134/editor/plugins/tiles/tile_map_editor.cpp#L2216
https://github.com/godotengine/godot/blob/61d2c855114c824f5ca27ded0a1fa71cc7b21134/editor/plugins/tiles/tile_set_editor.cpp#L781
@Calinou What do you think about it?

#57550 (duplicate of #74337 or vice versa) was fixed by #74341. This should be closed as a simpler solution has been merged. Nevertheless, thanks for your contribution!

I still think this ought to work out of the box and there's a deeper bug that will need to be looked into: https://github.com/godotengine/godot/pull/57596#issuecomment-1429835376
But for the time being #74341 is likely "good enough".


[libvpx] Fix arm-uwp builds for v142 toolset

This adds improved support for targeting UWP as well as ARM/ARM64 for regular Windows Desktop with the v142 toolset. Without this change, libvpx for the arm-uwp triplet currently fails to build with v142. This change does not modify the existing behavior for v141 and earlier.
Supported triplets when VCPKG_PLATFORM_TOOLSET is set to v142:
arm-uwp*
arm64-uwp*
x86-uwp*
x64-uwp*
arm-windows*
arm64-windows*
x86-windows*
x64-windows*

I followed the directions to update the port version number in the CONTROL file and ran vcpkg x-add-version libvpx, but I'm stilling a failure on the Validate Version Files check for x86_windows. I'm not sure what I did wrong. Any ideas? Thanks.

I followed the directions to update the port version number in the CONTROL file and ran vcpkg x-add-version libvpx, but I'm stilling a failure on the Validate Version Files check for x86_windows. I'm not sure what I did wrong. Any ideas? Thanks.

For the failures on x86-windows, you can run .\vcpkg.exe x-add-version libvpx --overwrite-version . Then submit the changes.

For the failures on x86-windows, you can run .\vcpkg.exe x-add-version libvpx --overwrite-version . Then submit the changes.

For the failures on x86-windows, you can run .\vcpkg.exe x-add-version libvpx --overwrite-version . Then submit the changes.
Excellent, thank you!

For the failures on x86-windows, you can run .\vcpkg.exe x-add-version libvpx --overwrite-version . Then submit the changes.
Excellent, thank you!

Looks good to me, thanks for the PR!


RuntimeError: Internal PLY loading error. Type has no defined byte size.

Hi,
Description: I tried to load a .PLY file with your library, unfortunately it does not work as intended -  resulting in an error: RuntimeError: Internal PLY loading error. Type has no defined byte size.
Version: point-cloud-utils-0.31.0
Workaround: I can still load the files with plydata = PlyData.read(input_file), extract the vertices and faces, and process the data further with your library. Maybe this workaround helps others with a similar issues.
Sincerely, Gregor

I also attached an exemplary file: 22831bc32bd744d3f06dea205edf9704.zip
Meshlab is loading the file correctly:

Error Log:
---------------------------------------------------------------------------
RuntimeError                              Traceback (most recent call last)
Cell In[66], line 14
     12 # load cuboid and sample point cloud
     13 print(input_file)
---> 14 cuboid_verts, cuboid_faces = pcu.load_mesh_vf(input_file)

File /clusterstorage/gkobsik/miniforge3/envs/py3d/lib/python3.10/site-packages/point_cloud_utils/_mesh_io.py:640, in load_mesh_vf(filename, dtype)
    627 def load_mesh_vf(filename, dtype=float):
    628     """
    629     Load a triangle mesh consisting of vertex positions and face indices.
    630     Point Cloud Utils currently supports PLY, OBJ, STL, OFF, VRML 2.0, X3D, COLLADA, 3DS.
   (...)
    638       f : An (m, 3)-shaped integer numpy array of face indices into v
    639     """
--> 640     ret = load_triangle_mesh(filename, dtype=dtype)
    641     return ret.v, ret.f

File /clusterstorage/gkobsik/miniforge3/envs/py3d/lib/python3.10/site-packages/point_cloud_utils/_mesh_io.py:556, in load_triangle_mesh(filename, dtype)
    544 """
    545 Load a triangle mesh into a `TriangleMesh` class.
    546 Point Cloud Utils currently supports PLY, OBJ, STL, OFF, VRML 2.0, X3D, COLLADA, 3DS.
   (...)
    553   mesh : A `TriangleMesh` class containing the loaded mesh
    554 """
    555 ret = TriangleMesh()
--> 556 ret.load(filename, dtype=dtype)
    557 return ret

File /clusterstorage/gkobsik/miniforge3/envs/py3d/lib/python3.10/site-packages/point_cloud_utils/_mesh_io.py:344, in TriangleMesh.load(self, filename, dtype)
    341         os.chdir(previous_dir)
    343 with pushd(mesh_root_path):
--> 344     mesh_dict = load_mesh_internal(mesh_filename, dtype)
    346 self.textures = mesh_dict["textures"]
    347 self.normal_maps = mesh_dict["normal_maps"]

RuntimeError: Internal PLY loading error. Type has no defined byte size.

Same error here, but this only happens on py=3.9. using py=3.10 on a same machine resolves this issue.

Running into the same issue / error (tested with point-cloud-utils==0.30.2 and 0.31.0) using py=3.11.
The python version seems to not be related on my end, but I found that problem can be circumvented using numpy==1.26.4 (the issue is popping up for numpy==2.1.3).


Re-write Processing functions with Functions v2 to check performance increase

Re-writing Azure functions with .NET Core since v2 is supposed to give better performing functions

Done


Consolidate tuning configs

There are several tuning configs in Druid. Use common base config for simplicity.

Done


Update managing_contact_list.md

Adding managing lists video

Hello @iritch42,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


[bug] Managed Omni users cannot use talosctl pcap

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
$ talosctl pcap -s 100000 -i eth0 --cluster <CLUSTERNAME> -n <NODENAME> --output talos.pcap
rpc error: code = PermissionDenied desc = not authorized

When a user of managed Omni tries to take a packet capture via talosctl, they receive a permission denied error. This is due to the fact that talosconfig that omni generates is locked down right now to avoid operators changing things out from under omni.
Expected Behavior
Managed Omni users should be able to use talosctl pcap on their nodes.
Steps To Reproduce

Create a fully managed Omni cluster
Download the talosconfig from the Omni UI
Run  talosctl pcap -s 100000 -i eth0 --cluster <CLUSTERNAME> -n <NODENAME> --output talos.pcap --talosconfig <TALOSCONFIG_PATH>

What browsers are you seeing the problem on?
No response
Anything else?
No response

The fix will be released as part in Omni v0.7.0+

Omni v0.7.0 has been released and I can confirm that I am now able to take pcaps via talosctl, thanks!


Localized messages for transaction errors

If there was an error sending the transaction, we just print it as it is: https://github.com/encointer/encointer-wallet-flutter/blob/3d614a70623cd950ce5508b6624d50f3ae7e2355/app/lib/service/tx/lib/src/submit_to_js.dart#L64
However, especially in the case of errors, it would be great if the user gets additional information about the error (in the users own language if possible). I would suggest a handling like this:
String getTxErrorMsg(Translations dic, String txError) {
    switch (txError) {
    // not 100% sure about the string of the error, you need to check yourself what the error is.
    case 'ecncointerCeremonies.VotesNotDependable':
      return dic.encointer.votesNotDependableError);
    case 'RewardsAlreadyClaimed':
      return dic.encointer.votesNotDependableError);
   //...  some more cases
    default:
      // display plain tx error in case we don't recognize the error
      return txError;
  }
}

Additionally, currently I don't have an overview of all potential errors, we need to play around with the app a bit to find them.

I played with our app and I found

"1010: Invalid Transaction: Inability to pay some fees , e.g. account balance too low" (transactions) This error message appears when a transaction fails due to inability to pay fees, for example, if the account balance is insufficient.
"encointerCeremonies.AlreadyEndorsed" (send endorse second time) This error message occurs when a user tries to endorse the same participant twice.
"encointerCeremonies.VotesNotDependable" (ask coin before finish meetup) This error message occurs when a user tries to request a coin before finishing the meetup.
"encointerCeremonies.NoValidClaims" (participants not scanned and tap claims) This error message occurs when a user tries to submit a claim without scanning the participants or tapping the claim.
"1014: Priority is too low: (39557856 vs 39557856): The transaction has too low priority to replace another transaction already in the pool." (tap submit 2 claims bat-bat 2 jolu) This error message occurs when a user's transaction priority is not high enough, or when they try to replace a previously made transaction.
"encointerCeremonies.RewardsAlreadyIssued" (submit claim pending second time) This error message occurs when a user tries to submit a claim for the second time and the rewards have already been issued.


Issue utilizing v.0.1.15 provider

Terraform Version
Run terraform -v to show the version. If you are not running the latest version of Terraform, please upgrade because your issue may have already been fixed.
Terraform v1.0.9
on darwin_amd64

Affected Resource(s)
Please list the resources as a list, for example:
terraform provider v0.1.15 for MAC and Linux
If this issue appears to affect multiple resources, it may be an issue with Terraform's core, so please mention this.
Terraform Configuration Files
terraform {
  required_providers {
    harness = {
      source  = "harness-io/harness"
      version = "0.1.15"
    }
  }
  required_version = ">=1.0.0"
}

When running terraform init, the following error appears:
Error: Incompatible provider version
│
│ Provider registry.terraform.io/harness-io/harness v0.1.15 does not have a package available for your current platform, darwin_amd64.
│
│ Provider releases are separate from Terraform CLI releases, so not all providers are available for all platforms. Other versions of this provider may have different
│ platforms supported.

Expected Behavior
Terraform should initialize properly
Actual Behavior
What actually happened?
Steps to Reproduce
Please list the steps required to reproduce the issue, for example:

terraform init

Important Factoids
Same error appears when running on Linux, in Terraform Cloud

There's currently an issue with the Terraform Registry we're working with Hashicorp to get fixed. For now you'll need to download the package directly from the github release

@micahlmartin We do face similar issue when Jenkins calls the registry API to get provider metadata which ends up with HTTP 404 error. The endpoint [https://registry.terraform.io/v1/providers/harness-io/harness/0.1.15/download/linux/amd64] is throwing "{"errors":["signing key A1A83E0CC7A69F1D not found for harness-io"]}" error. Works fine for all rest of the versions expect 0.1.15.
Believe this is also caused due to the same Terraform registry issue which we are talking about.

This has now been fixed. You should be able to use the provider with harness/harness instead of harness-io/harness


Fix an issue that does not render Skeleton when width param is nullable

Hello, thanks for your library.
I found an issue while redering Skeleton when width param is nullable.

The photo above is the result of <Skeleton count={1}/>
Span element with class react-loading-skeleton has width:100% style, but skeleton is not rendered.
(It works fine in storybook, I guess it's because of style used inside storybook.)
This is because the parent span(aka. container) of the span with class react-loading-skeleton does not have a width style.

I tried putting width:100% style in the parent span and it was resolved.

Therefore, when the width param is Nullable, width:100% is added to the container span.
I additionally added some test code and all passed.
Thanks.

Hi @kwangminini, thanks for the PR 😀. It looks like you are trying to fix a bug, but I'm not 100% sure what the bug is. Can you create a minimal reproducible example that demonstrates the bug? Feel free to add a new story to our Storybook, or create a standalone GitHub repo.

@srmagura Sorry for not being able to figure out the exact cause.
But now I have figured out the exact cause.
The bug appears when display: flex of Skeleton's parent element.
You can also check it out in the storybook
export const Basic: React.FC = () =>
<Skeleton count={5} width={100}/>
;
Example CodeSandbox
In my opinion, giving width: 100% to the parent element seems to have the least side effect, so I modified it like this.
Thanks

Thanks for that CodeSandbox. That really helps me understand what is going on.
I would prefer not to merge this because I think adding width: 100% could break some people's layouts. Here's a CodeSandbox that shows this. (I'll admit that it is a somewhat contrived example.)
Other people have hit this same issue, where they are confused because the skeleton has 0 width by default when used in a flex container. I think a better "fix" for this problem would be to update Examples section of our README to explain why the skeleton has 0 width by default (because it has no intrinsic width), and how to fix it (use containerClassName to give the skeleton container flex: 1). Let me know if you are interested in making this change to the docs.

Thanks for that CodeSandbox too😀.
Looking at the example, it seems better to edit the README than to merge.
I'm very interested in editing the README.
Modifying the README of open-source, which is used by many developers, is also a pleasure for me.
If you don't mind, can I fix it and make a pull request again?

Yes, a PR that updates the README would be most appreciated!


Support of Docker/Kubernetes CPU limit/reservation

Feature request
Docker (swarm) and Kubernetes have a way to limit CPU usage of a container.
Docker (swarm):
version: '3.4'
services:
    text-embeddings:
        image: ghcr.io/huggingface/text-embeddings-inference:cpu-0.6
        deploy:
            resources:
                limits:
                    cpus: '2'

Kubernetes:
apiVersion: v1
kind: Pod
metadata:
  name: text-embeddings
spec:
  containers:
  - name: text-embeddings
    image: ghcr.io/huggingface/text-embeddings-inference:cpu-0.6
    resources:
      requests:
        cpu: "2000m"
      limits:
        cpu: "2000m"

However, for that to be working optimally (see motivation below), the application in the container has to be aware of this limit and to allocate thread pools accordingly.
Motivation
If thread pools don't match the CPU limit, the container is throttled and performance will drop way beyond expectations (6 times slower in the example below).
For instance on my core i3-8300H (4 cores, 8 threads).
I'm evaluating performance with the following apache bench command (a request containing a single 17KB text to be processed with the sentence-transformers/paraphrase-multilingual-MiniLM-L12-v2 model:
ab -k -n24 -c4 -p req-en-17k-b1-huggingface.json -T application/json localhost:18083/embed

configuration
reqs/sec
avg. CPU usage (top)

no cpu limit
16.87
465%

cpuset=0,1
11.48
185%

cpus=2
1.82
200%

cpus=2 + env vars
11.03
150%

You can see on line 3 that cpu=2 (without environment variables) performance are 6 times slower than using cpuset=0,1.
The problem is neither Kubernetes nor Docker Swarm allow the cpuset option.
You can see on line 4 that adding environment variables controlling the number of threads has a positive impact on performances (almost on par with cpuset=0,1).
no cpu limit configuration:
version: '3.4'
services:
    multiminilml12v2:
        image: ghcr.io/huggingface/text-embeddings-inference:cpu-1.0
        environment:
            - MODEL_ID=sentence-transformers/paraphrase-multilingual-MiniLM-L12-v2
            - NVIDIA_DISABLE_REQUIRE=1
            - RUST_BACKTRACE=full
            - JSON_OUTPUT=true
            - PORT=18083
            - MAX_BATCH_TOKENS=65536
            - MAX_CLIENT_BATCH_SIZE=1024

cpuset=0,1 configuration:
version: '3.4'
services:
    multiminilml12v2:
        image: ghcr.io/huggingface/text-embeddings-inference:cpu-1.0
        environment:
            - MODEL_ID=sentence-transformers/paraphrase-multilingual-MiniLM-L12-v2
            - NVIDIA_DISABLE_REQUIRE=1
            - RUST_BACKTRACE=full
            - JSON_OUTPUT=true
            - PORT=18083
            - MAX_BATCH_TOKENS=65536
            - MAX_CLIENT_BATCH_SIZE=1024
        cpuset: "0,1"

cpus=2 configuration:
version: '3.4'
services:
    multiminilml12v2:
        image: ghcr.io/huggingface/text-embeddings-inference:cpu-1.0
        environment:
            - MODEL_ID=sentence-transformers/paraphrase-multilingual-MiniLM-L12-v2
            - NVIDIA_DISABLE_REQUIRE=1
            - RUST_BACKTRACE=full
            - JSON_OUTPUT=true
            - PORT=18083
            - MAX_BATCH_TOKENS=65536
            - MAX_CLIENT_BATCH_SIZE=1024
        deploy:
            resources:
                limits:
                    cpus: '2'

cpus=2 + env vars configuration:
version: '3.4'
services:
    multiminilml12v2:
        image: ghcr.io/huggingface/text-embeddings-inference:cpu-1.0
        environment:
            - MODEL_ID=sentence-transformers/paraphrase-multilingual-MiniLM-L12-v2
            - NVIDIA_DISABLE_REQUIRE=1
            - RUST_BACKTRACE=full
            - JSON_OUTPUT=true
            - PORT=18083
            - MAX_BATCH_TOKENS=65536
            - MAX_CLIENT_BATCH_SIZE=1024
            # interesting variables below
            - TOKIO_WORKER_THREADS=1
            - NUM_RAYON_THREADS=1
            - MKL_NUM_THREADS=1
            - MKL_DOMAIN_NUM_THREADS="MKL_BLAS=1"
            - OMP_NUM_THREADS=1
            - MKL_DYNAMIC="FALSE"
            - OMP_DYNAMIC="FALSE"
        deploy:
            resources:
                limits:
                    cpus: '2'

Your contribution
I'm afraid I can't do much more than this.
Please note that I don't really now which of my environment variables do really have an impact on performance since I am totally unaware of the internals of text-embeddings-inference.
The issue is known though:
https://danluu.com/cgroup-throttling/
https://nemre.medium.com/is-your-go-application-really-using-the-correct-number-of-cpu-cores-20915d2b6ccb
Some ecosystems have begun to take that into account.
For instance since Python 3.13 you can fool Python into thinking it has less CPU using an environment variable:
docker run --rm -it --name py13 -e PYTHON_CPU_COUNT=2  python:3.13.0a4-slim  python -c "import os; print(os.cpu_count())"
2

Java does that automatically since java 15:
docker run --rm -it --name java23 --entrypoint /bin/bash openjdk:23-slim
root@31e4b2de8fad:/# jshell
jshell>  System.out.println(Runtime.getRuntime().availableProcessors());
8

ocker run --rm -it --name java23 --cpus=2 --entrypoint /bin/bash openjdk:23-slim
root@1935b08ebcf7:/# jshell
jshell> System.out.println(Runtime.getRuntime().availableProcessors());
2

Super interesting!
Can you try:
version: '3.4'
services:
    multiminilml12v2:
        image: ghcr.io/huggingface/text-embeddings-inference:cpu-1.0
        environment:
            - MODEL_ID=sentence-transformers/paraphrase-multilingual-MiniLM-L12-v2
            - NVIDIA_DISABLE_REQUIRE=1
            - RUST_BACKTRACE=full
            - JSON_OUTPUT=true
            - PORT=18083
            - MAX_BATCH_TOKENS=65536
            - MAX_CLIENT_BATCH_SIZE=1024
            # interesting variables below
            - MKL_NUM_THREADS=1
            - MKL_DOMAIN_NUM_THREADS="MKL_BLAS=1"
            - MKL_DYNAMIC="FALSE"
        deploy:
            resources:
                limits:
                    cpus: '2'

?
TEI uses https://crates.io/crates/num_cpus internally which correctly get the number of CPUs. My guess is that only MKL is confused.

@OlivierDehaene thanks for you fast answer!
Your configuration is working fine:

configuration
reqs/sec
avg. CPU usage (top)

no cpu limit
16.87
465%

cpuset=0,1
11.48
185%

cpus=2
1.82
200%

cpus=2 + env vars
11.03
150%

cpus=2 + your conf
10.92
150%

There is a slight difference but it doesn't mean anything (I kept the 11.03 value for the cpus=2 + env vars configuration but its result was 10.90 today).

Do you think it TEI could set those environment variables at the beginning of its startup phase?

Setting these values correctly would be really hard since they are MKL/runtime specific. Plus these should be set before execution so this imply creating a launching script above the TEI bin.
I think it's better to document the problem and let users find the best values for their specific env.
It's more a design decision than anything.

Setting these values correctly would be really hard since they are MKL/runtime specific.
let users find the best values for their specific env
I think it's better to document the problem

So it would be great if the documentation could state a rule of thumb (that would be a sensible default) like this:
For people running the CPU-based Docker image with Docker or Kubernetes CPU limits: set those environment variables and replace 1 with the smallest positive integer greater than or equal to the allocated CPU limit. (if that's the correct rule of thumb because I don't know the relationship between MKL_NUM_THREADS and MKL_DOMAIN_NUM_THREADS).
MKL_NUM_THREADS=1
MKL_DOMAIN_NUM_THREADS="MKL_BLAS=1"
MKL_DYNAMIC="FALSE"

Plus these should be set before execution so this imply creating a launching script above the TEI bin.

TEI running on bare metal definitely doesn't require such a script but, since this repo is including the CPU-based Dockerfile, the CPU-based image would definitely be more user-friendly with a script :).

This issue causes serious performance problems when running on servers with many CPUs.
Example: there are 256 CPUs in a multi-socket system, and a user wants to dedicate 8 CPUs for each text-embedding-interface container in the system. What happens now is: every text-embedding-interface creates 2 x 256 threads. When the threads are squeezed to run on 8 CPUs, the whole service runs so slow that it looks broken.
Worker thread pools need to be populated based on allowed, not all CPUs in the system.
For logs, see Issue https://github.com/opea-project/GenAIExamples/issues/763


Add stream_mode=custom

adds the ability for nodes (including in subgraphs) to emit chunks directly to the output stream, emitted chunks can have any type
when stream_mode=custom isnt requested by the caller emitted chunks are ignored

@hinthornw IMO astream_events is an escape hatch no one should have to use under normal circumstances. I think in a world with multimodal / continuous IO having custom streaming output from your graph should be a first class citizen. Thoughts?


[SMALLFIX] Use CountDownLatch instead of sleeping in a loop

Followup to https://github.com/amplab/tachyon/pull/2218

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/7159/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/7162/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/7172/
Test PASSed.

LGTM


ONVIF Auto Discovery - Not work

ONVIF has the official tool for investigating onvif devices.
Autodiscovery does Not work with OpenIPC onvif. Probably that problem for other software and NVR connect to OpenIPC onvif
https://www.lingodigit.com/onvif_nvcdemo.html#other_version  - ONVIF Device Tool.
https://www.onvif.org/profiles/conformance/device-test-2/

Hello
Please check the latest version and give us feedback
Thank you


Add support for CompletionList.Data.

CompletionList.Data serves as a way for completion items to not redundantly provide Data. Aka, all completion data objects serve as a default for underlying completion items.
The LSP Platform hasn't yet inserted these changes into VS; however, our language server doesn't depend on the LSP platform directly anyhow. This changeset enables the data bag optimization of the LSP platform in a way that will automatically light up once it becomes available.
Updated benchmarks to use new Data capability on the completion list.
Updated our VSCompletionListCapability to include a Data property.
Added tests

Before:

Method
Mean
Error
StdDev
Op/s
Gen 0
Gen 1
Gen 2
Allocated

'Component Completion List Roundtrip Serialization'
7.434 ms
0.1169 ms
0.1036 ms
134.5
585.9375
296.8750
7.8125
3,679 KB

'Component Completion List Serialization'
1.924 ms
0.0257 ms
0.0241 ms
519.8
83.9844
37.1094
21.4844
523 KB

'Component Completion List Deserialization'
5.888 ms
0.1090 ms
0.1908 ms
169.8
507.8125
242.1875
-
3,156 KB

After:

Method
Mean
Error
StdDev
Op/s
Gen 0
Gen 1
Gen 2
Allocated

'Component Completion List Roundtrip Serialization'
2,931.8 us
43.66 us
38.71 us
341.1
304.6875
136.7188
-
1,870 KB

'Component Completion List Serialization'
332.5 us
6.04 us
5.65 us
3,007.8
26.8555
5.3711
-
166 KB

'Component Completion List Deserialization'
2,836.9 us
49.85 us
46.63 us
352.5
277.3438
121.0938
-
1,704 KB

Results:
Roundtrip: 2.5x faster
Serialization: 5.8x faster
Deserialization: 2.1x faster
Razor's language server part of dotnet/aspnetcore#31883

FYI @ToddGrun / @jimmylewis if you want to adopt


With form_for object,  :validate=> :true (rather than a boolean),  firebug spits out a "settings is undefined" error, but the form has the data-validate attribute

Obviously a symbol is not correct usage, but it is never caught and the form gets the attribute data-validate="true" set. The issue is that on the surface the form looks like it is working, but the validation javascript for the form is not generated and firebug then complains rightly about "settings is undefined"
I found the problem on this line:
https://github.com/bcardarella/client_side_validations/blob/master/lib/client_side_validations/action_view/form_helper.rb#L9
if options[:validate] is a symbol, then the variable content_for_name is going to pick it up as its value and you end up in this if statement
https://github.com/bcardarella/client_side_validations/blob/master/lib/client_side_validations/action_view/form_helper.rb#L33
and content_for(:true) will not do much good (unless you have a yield :true somewhere in your view) and the script variable is then assigned nil. form_for basically now returns "#{form}".html_safe instead of the intended "#{form}#{script}".html_safe. So you get the same form back without javascript.
I understand that this is a feature that allows moving javascript code into the header.
Could I suggest that the use of :true should not be permitted? Either raise an error or don't set the data-validate attribute.
This would save symbol/boolean dyslexics like myself a lot of time (too late for me though).
Thank you for your consideration.

Yeah, this should be handled properly. I'll try to make sure I do so for the next release.

So, I actually brain farted when I previously replied to this.
This is actually expected behavior. It is undocumented (but it should be) that when you pass a symbol to :validate it will assign the script to content_for so you can render the script tag else where.
See https://github.com/bcardarella/client_side_validations/issues/98

I think accepting :true is OK. Detecting the edge case of someone using a symbol instead of a boolean is not something I think should be done.

I agree. Shouldn't happen to me any more and this issue should help others who make the same mistake. Appreciate your time and consideration.

I just got bitten by this issue (glad i found this thread!).
To make the info (hopefully) easier to find, I have put this in the wiki:
https://github.com/bcardarella/client_side_validations/wiki/Common-Gotchas

@jpmcgrath thank-you for the wiki entry


findByXxxIn generates invalid SQL when the Collection is empty [DATAJDBC-567]

Veli-Pekka Muli opened DATAJDBC-567 and commented
The generated SQL is WHERE "myschema"."mytable"."mycolumn" IN ()

Affects: 2.0.1 (Neumann SR1)

I am not able to reproduce it - If the passed Collection is empty, then generated SQL will be:
SELECT COUNT(*) FROM "schema"."table" WHERE 1 = 0


[hyprland/window] Implement rewrite option like in sway/window

Fixes #1727

The code should probably be refactored, moving std::string Window::rewriteTitle(const std::string& title) to some common place and let it take the rules as an additional argument. Because right now there is code duplication between sway/window and hyprland/window.
Maybe a file somewhere in include/util?

Yep, move this func to a common place :)
Thanks!

Refactoring done. I have tested it to be working on hyprland and assume it to work on sway as well since it's only a minor change.
Btw, any way to run the linter check locally before pushing?

Yes you can run clang-format :)

Thx!


MacOS PowerApps CLI Auth Steps

On MacOS, where Multi Factor Authentication is enabled for a tenant, the flags --name and --kind were needed to authenticate. MSAL did not launch the native browser without those flags. Without them, the following error was given:
"Error: Only loopback redirect uri is supported, but app://[APPID REDACTED]/ was found. Configure http://localhost or http://localhost:port both during app registration and when you create the PublicClientApplication object. See https://aka.ms/msal-net-os-browser for details"
Attempting to authenticate with --username  & --password failed with:
"Error: AADSTS50076: Due to a configuration change made by your administrator, or because you moved to a new location, you must use multi-factor authentication to access '00000007-0000-0000-c000-000000000000'. "
Until MSAL launches the authentication web view without those flags, the documentation ought to call out the necessity of using them.

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

ID: bf4e3c9e-af16-604e-43d7-179252741bdb
Version Independent ID: ce493d89-9f8c-ac80-6e77-00e5a6f84077
Content: Tutorial - Use Microsoft Power Platform CLI with portals - Power Apps
Content Source: powerapps-docs/maker/portals/power-apps-cli-tutorial.md
Service: powerapps
GitHub Login: @neerajnandwana-msft
Microsoft Alias: nenandw

Hi @JTRemington,
yes, this is a known limitation on macOS: the pac CLI does not yet support interactive OAuth authentication flows. We are working on addressing this in a future release of the pac CLI (which will ship with a future release of the PowerPlatform VS Code extension).
This limitation is also mentioned in the release notes in https://aka.ms/ppcvscode
Since your AAD instance requires ConditionalAccess/MFA for your user credentials, your otherwise correct workaround to use --username/--password authN failed.
To work around the AAD ConditionalAccess requirement, the best practice is to use appID/SPN authentication, by supplying a --tenant, --applicationId, --clientSecret triple. To create one, please refer to the documentation and a PowerShell script as documented in: https://aka.ms/buildtools-spn
Note that the reference PS script needs to run on Windows Desktop Powershell
CC: @kkanakas @neerajnandwana-msft


Add bfs

Have you read the Contributing Guidelines on Pull Requests?
Yes
(Write your answer here.)
Description
Added Added Graph traversal algorithm of Breadth-First Search in C.
(Write your answer here.)
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
#1725
(Write your answer here.)

@srimani-programmer . Kindly review this

@srimani-programmer . Kindly review this

I am closing this Pr. I have created a new pr @srimani-programmer
New Pr  no: #1914

I am closing this Pr. I have created a new pr @srimani-programmer
New Pr  no: #1914


Adding updated pot files to version control.

What's this PR do?
This PR adds the updated POT files to version control.
Where should the reviewer start?
Just confirming that the code looks correct.

@angaither

:pots: :egg: :+1:


Application Insight in isolated Function app

I am facing issue in moving my function app code from in process to isolated process. Here is my application insight code which i am using in my startup.cs. i am not using APPINSIGHTS_INSTRUMENTATIONKEY configuration.
class Startup : FunctionsStartup
{
public override void ConfigureAppConfiguration(IFunctionsConfigurationBuilder builder)
{
builder.ConfigurationBuilder.AddApplicationInsightsSettings(applicationInsightsInstrumentationConectionstring);
}
public override void Configure(IFunctionsHostBuilder builder)
{
builder.Services.AddApplicationInsightsTelemetry();
}
}
Now i want to move it to program.cs for function app isolated process. i have tried different ways but it is not writing logs to application insight.

.ConfigureServices(services =>
{
               services.AddApplicationInsightsTelemetryWorkerService(opt =>
                {
                    opt.ConnectionString = applicationInsightsConectionString;
                    opt.DependencyCollectionOptions.EnableLegacyCorrelationHeadersInjection = true;
                });

            })

@MianNajmi we have a preview package for using app insights in the dotnet isolated worker. See https://github.com/Azure/azure-functions-dotnet-worker/pull/944#issue-1282987627


how to relogin or change github account of Azure Data Studio

Type: Feature Request
I have installed Azure Data Studio on my Macbook pro. And I installed GitHub Copilot Extension. The first time I login my github account successfully. But I need to change to another github account which has copilot subscription. But there is no feature to log out or sign out the github account and relogin with another github account.
Azure Data Studio version: azuredatastudio 1.45.1 (Universal) (88c21b1725a3e79440027bdb7b5a55fb036be0e2, 2023-08-03T00:42:36.656Z)
OS version: Darwin x64 21.6.0
Restricted Mode: No
Preview Features: Enabled
Modes:

I found a solution (not the best one)
Go to AppData\Roaming\azuredatastudio\User\globalStorage\state.vscdb replace your github login by the new one
Be careful, you may lost some saved password.
I hope MS will add the option to manage GitHub account as it is in VS Code

Hi, I can't even login with GitHub account, everytime it opens up a window in browser to sign-in with Azure account.

We could consider supporting commands for GitHub Copilot: Sign In/Sign Out - @lewis-sanchez ?

To elaborate on @phnogues comment, I was able to remove the GitHub creds from my ADS with:
sqlite3 state.vscdb "delete from ItemTable where key like '%github%';"

The above command needs to be run in your globalStorage folder:
Linux:
~/.config/azuredatastudio/User/globalStorage

Windows:
<user>\AppData\Roaming\azuredatastudio\User\globalStorage

Requires sqlite3 CLI to be installed (e.g. sudo apt install sqlite3)

@phnogues  Deleting the file AppData\Roaming\azuredatastudio\User\globalStorage\state.vscdb worked for me.

@lewis-sanchez should this be supported now that https://github.com/microsoft/azuredatastudio/pull/25014 was merged?

@kisantia, yeah, this should be supported. PR #25014 allowed users to log out. The Copilot extension would then prompt the user to sign back in, but in some cases the user would need to close and reopen Azure Data Studio for the extension to trigger the prompt to sign back in with the same or different account.

Closing this issue since it is now supported in insiders and will be available in the next release.


Microsoft Teams classic [Microsoft.Teams.Classic] Version <IP_ADDRESS>56 - already installed error

Hello,
I can't upgrade Teams too  :
C:\Windows\System32>winget upgrade --all
Nom                                           ID                                  Version        Disponible     Source
----------------------------------------------------------------------------------------------------------------------
Driver Easy 6.0.0                             Easeware.DriverEasy                 6.0.0          <IP_ADDRESS>691    winget
Foxit PDF Reader                              Foxit.FoxitReader                   2<IP_ADDRESS>97 2<IP_ADDRESS>70 winget
Microsoft SQL Server Management Studio - 19.2 Microsoft.SQLServerManagementStudio 19.2           20.1           winget
Teams Machine-Wide Installer                  Microsoft.Teams.Classic             <IP_ADDRESS>72     <IP_ADDRESS>56   winget
Microsoft .NET Runtime - 6.0.30 (x64)         Microsoft.DotNet.Runtime.6          6.0.30         6.0.31         winget
6 mises à niveau disponibles.

Les packages suivants ont une mise à niveau disponible, mais nécessitent un ciblage explicite pour la mise à niveau :
Nom     ID              Version  Disponible Source
--------------------------------------------------
Discord Discord.Discord 1.0.9013 1.0.9144   winget

Here is the log :
WinGet-Microsoft.Teams.Classic.<IP_ADDRESS>56-2024-06-03-14-35-28.149.log
Thanks,
Vincent

This appears to only impact 1 package, so should be moved to the winget-pkgs repo.

Is this the "upgrade always available" situation? Do you perform the upgrade, then check again to see the same version numbers, and the upgrade is still there?

Is this the "upgrade always available" situation? Do you perform the upgrade, then check again to see the same version numbers, and the upgrade is still there?

oh yes, you're right, in the control panel I see to correct version.
I uninstall Teams Classic and kept Teams new.
Thanks for help.


Relying on jquery

Currently, the library uses a very small subset of jQuery's functionality, but jQuery itself is very huge.  The process of using browserify appends jQuery to the source file, which results in a large output file.
Can we somehow take jQuery away from gmail.js?
The primary uses of jQuery in gmail.js are:

DOM selection
Utility methods like $.each, $.merge, $.param
$.ajax

Given we use a somewhat modern flavor of Javascript, we can easily replace these functionalities with native syntax. (maybe some polyfills)
I understand that a CDN can also be used here, which would result in a not so huge file.

I'm no big fan of jQuery, and I agree that looking only at the internal use in the library, jQuery does not provide a whole lot of value. In a green-field code-development scenario, I would wipe out jQuery in a single commit and move on.
This is however a reasonably mature library now, and people rely on it to keep working.
If we remove use of jQuery, we will change our API-surface and create a breaking change.
I think the only way to get "rid" of jQuery is to create a new "base-object" (Gmail2?) with a new API only.
People would then have the ability to move to the new API without us causing breakage on the old one, until we eventually just obsolete and remove it completely.
That's a long term development process though, and in the meantime we will be shipping effectively double code-size for GmailJS.
So yeah. No perfect solution in sight, and to me, jQuery has not been enough of an issue to be worth all that effort yet.

Hello @josteink,
Recently Google enabled trusted-types CSP in Google Calendar, therefore rendering injected jQuery code unusable, as it relies on innerHTML assignments upon initialization. For now this policy is not enabled for Gmail, but this may change unexpectedly in near future.
jQuery added support for TrustedHTML for 4.0, but it's still in development, and I did not find specific ETA by quick googling.
I think it would be good to transition from jQuery - make it completely optional and introduce el with raw HTMLElements for gmail-js dom() objects. If consumer provides jQuery in 'Gmail' constructor, it would be a reason to provide compatibilty $el wrappers.
What do you think?

Thanks for a long, informative and reasoned update in this issue.
I think your suggestion sounds quite reasonable.
I’ve always wanted to move away from JQuery, but breaking the API wasn’t something i wanted to do. This provides a real argument for doing so.
Thinking in terms of types, I guess we could achieve this with a new instance member/field which acts as a HTMLElement-transformer used to “encode” elements before leaving (or entering?) the api.
This may not compile directly but it should roughly illustrate the idea:
class Gmail<TOutput> {
    …
    _transform: (elem: HTMLelement) => TOuptput

    …

    dom(name): TOuput {
        const result = someDomNode;
        return this._transform(result); // do this everywhere JQuery is returned today
    }
}

Then one can instanciate a Gmail class with a IdentityTransformer (i => i) to avoid JQuery or instantiate a Gmail with a JQueryTransformer (i => $(i)) to maintain the old API.
That should allow us to change the internals to be JQuery-free and allow JQuery-free usage, while at the same time causing minimal breakage.
What do you think?

Hello @josteink,
This sounds reasonable, and would also work with some other libraries like fabiospampinato/cash, but I think JQueryTransformer could be set automatically if user provides jQuery in counstructor (to keep compatibility), and IdentityTransformer would be used if no jQuery was provided/required.
 I've tried hacking around some methods, and they start looking like this:
api.check.is_peoplekit_compose = function (el) {
    if (el && typeof el.jquery === 'string') {
        return el.find("div[name=to] input[peoplekit-id]").length !== 0;
    } else {
        return !!el && typeof el.querySelector === 'function' && !!el.querySelector("div[name=to] input[peoplekit-id]");
    }
};

I'm not sure how this could be implemented with element transformer approach. I mean, even if no jQuery module/transformer was provided in constructor, some consumers could still rely on providing jQuery instance to such method and expecting it would work.
If this could be omitted and dropped from contract, probably some utility methods like getElement(), getElements(), isElementPresent() etc. could be helpful in transformer interface to do the DOM lookup job.

A more complex, but still pretty simple approach could be to have the transformer not just be a function but an object which can transform both in and out.
In that case we can make all incoming calls that (currently) rely on jQuery just instead handle plain DOM elements.
That would probably also work better in a API cyclic kind of way.
That is: Objects retrieved from the API should be usable. That would be nice.
I must admit I haven't looked into how extensive that kind of change would be, not for the API as a whole nor even for single functions, since as you show above ...
For some functions we try to do some sort of input detection.
That said... Usually I like making refactorings in incremental "guaranteed correct" steps.
So if we at first just start with returned values (which should be fairly simple to prove correct)... Then incoming jQuery based parameters that would (kind of) be a dependency brought on the extension on its own through the APIs it decided to use, right?
Meaning the core/internal GmailJs code should still be able to be made trusted-types compatible (if not already).
And after that we could take the step to deal with incoming parameters, if we still think it looks like a problem worth solving.
That's a three-step plan which I kind of believe in.
Or do you perhaps have any other or better suggestion? 🙂

And for the sake of argument, if we are expecting jquery in api.check.* mfunftions I think it’s fair to treat those as primarily internal for now, and they could be made to either:

Warn and/or crash upon detecting incoming jQuery objects
Silently extract first HMLElement from jquery and run regular DOM code afterwards.

In either case, (external) Jquery-callers will manually need to rewrap the returned value with (their) jquery to keep code working if they expect HTML elements, but jquery supports/encourages that kind of object-agnostic “just in case” code so I don’t think that’s going to cause a lot of outrage.

Just as a FYI: I'll be busy as heck the upcoming week(s), so if you want to take a stab at this feel free to get going and create a PR when you have something/anything to show for (complete or not).
I'll probably have time to provide feedback if nothing else :smile:

Hello @josteink,
Taking into account that jQuery could not be used anymore due to recent TrustedHTML changes in Gmail, would it make sense to avoid using it altogether for DOM manipulations?

I think changes related to TrustedHTML should be kept in a single issue, to make it easier for everyone to track the conversation.
I'll provide an answer in this thread instead:
https://github.com/KartikTalwar/gmail.js/issues/779


paperclip 422 Client Error: Unprocessable Entity for https://api.stuyspec.com/media

Receiving a 422 Client Error for uploading images.

After taking a look at the Rails app logs, I found these two lines:
Command :: file -b --mime '/tmp/5324ab93bc18266f86f202102d084de120180216-14456-1bh4p7j.jpeg'
[paperclip] Content Type Spoof: Filename features-congressofvienna-2-courtesy-of-mr-hanna-jpeg.jpeg (text/html from Headers, ["image/jpeg"] from Extension), content type discovered from file command: text/html. See documentation to allow this combination.

A key takeaway is this: Content Type Spoof: ... (text/html from Headers, ["image/jpeg"] from Extension). This means that the file -b --mime command (gets the mime type of a file) is saying the file is in HTML while the extension of the file indicated it to be a JPEG. After opening the file in a text editor, it was indeed html. The content:
...

  One account. All of Google.

  Sign in to continue to Google Drive

...

The error was coming from a permissions issue. The Volume folder needed to not only be shared with<EMAIL_ADDRESS>but link sharing had to be set to "Anyone with the link can edit."


Is there a way to rotate the output 180⁰ ?

This could be done programmatically. Horrible algorithm and uses a lot of resources.  Effecetly, you want to rotate the 2d array of leds.
https://stackoverflow.com/questions/42519/how-do-you-rotate-a-two-dimensional-array
void rotate180()
{
//copy
CRGB tmp_led[MATRIX_HEIGHTMATRIX_WIDTH];
for (int x = 0; x < 64; x++)
for (int y = 0; y < 64; y++)
{
CRGB tmp_led[xy] = leds[XY(x, y)];
}
//store
for (int x = 0; x < 64; x++)
for (int y = 0; y < 64; y++)
{
leds[XY(x, y)] = tmp_led[XY(MATRIX_HEIGHT-x, y)];
}

Up to now I did not see the point to rate 180deg as you can just physically rotate the panel

Up to now I did not see the point to rate 180deg as you can just physically rotate the panel

that is because i designed my stand to hold the panel upside down, with the hub75 connectors and the power being at the bottom half. This allowed me to hide the wires using my stand without it covering the whole panel, using much more filament for printing