get-csonline user - filter "users who have not been assigned a specific voice policy" - how? Please provide an example

The text states you can filter for "users who have not been assigned a specific voice policy".  But there's no example, and I can't get it to work.
MicrosofTeams version 4.2.0
These don't work;
get-csonlineuser -filter "TeamsMeetingBroadcastPolicy -eq ''"
get-csonlineuser -filter "TeamsMeetingBroadcastPolicy -eq $null"
get-csonlineuser -filter "TeamsMeetingBroadcastPolicy -eq 'global'"
get-csonlineuser -filter "TeamsMeetingBroadcastPolicy -eq 'default'"
[Enter feedback here]

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

ID: 7008ec10-a056-d8bb-bb4e-b4690a84adfd
Version Independent ID: 2dcb78d9-7f36-208e-0989-184905d73011
Content: Get-CsOnlineUser (SkypeForBusiness)
Content Source: skype/skype-ps/skype/Get-CsOnlineUser.md
Service: skypeforbusiness-powershell
GitHub Login: @tomkau
Microsoft Alias: tomkau

@anwarmahmood1 can you please retry again with the same commands and check at the end of day?  We deployed a fix for the same today

@anwarmahmood1 Please reply.

thanks @yogkumgit
Can confirm
get-csonlineuser -filter "TeamsMeetingBroadcastPolicy -eq ''"
now returns expected results. May I suggest you add that as an example?
Feel free to close.


Scvi in R doesn't update to scvi-tools even after uninstalling and reinstalling (I think :)

Hi!

I'm trying to use SCVI for the first time, following the "ATAC-seq analysis in R", and first installed scvi by typing -m pip install scvi . When I tried to use it on R I got this deprecation message:
/x/y/z/python3.12/site-packages/scvi/__init__.py:31: DeprecationWarning: scvi is deprecated, please uninstall scvi via pip uninstall scvi and install the new scvi-tools package at github.com/YosefLab/scvi-tools warnings.warn(deprecation_msg, DeprecationWarning)
I thus uninstalled scvi and installed scvi-tools. But even after running scvi <- import("scvi-tools", convert = FALSE) on R again, I am still left with the deprecated SCVI.
I've tried uninstalling and reinstalling scvi several times but that doesn't solve the issue. What should I do?

UNIX
`-m pip install scvi `

R (with Reticulate to bridge the gap with python)
scvi <- import("scvi", convert = FALSE)

UNIX
`-m pip uninstall scvi `
`-m pip install scvi-tools `

R (with Reticulate to bridge the gap with python)
scvi <- import("scvi", convert = FALSE)

> scvi$model$PEAKVI$setup_anndata(adata)
Error in py_get_attr(x, name, FALSE) :
  AttributeError: module 'scvi' has no attribute 'model'
Run `reticulate::py_last_error()` for details.

Versions:

'0.6.8'

Okay, so I tried to check the python environment R is using, which I believe should be this:
`> py_config()
python:         /Library/Frameworks/Python.framework/Versions/3.12/bin/python3
libpython:      /Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/config-3.12-darwin/libpython3.12.dylib
pythonhome:     /Library/Frameworks/Python.framework/Versions/3.12:/Library/Frameworks/Python.framework/Versions/3.12
version:        3.12.1 (v3.12.1:2305ca5144, Dec  7 2023, 17:23:39) [Clang 13.0.0 (clang-13<IP_ADDRESS>)]
numpy:          /Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/site-packages/numpy
numpy_version:  1.26.3
scanpy:         [NOT FOUND]
NOTE: Python version was forced by use_python() function`
And compared it to the environment I scvi-tools in:
/Library/Frameworks/Python.framework/Versions/3.12/bin/python3 -m pip install scvi-tools
Both are the same, so I would say they are the same environment.
But when I try to check what is installed in that environment I don't actually get an answer
➜  ~ /Library/Frameworks/Python.framework/Versions/3.12/bin/python3 pip list    ➜  ~
So maybe there is indeed something wrong with the environment? ^^>

I created a new environment with Conda, reinstalled scvi-tools and now it works. Thank you for your suggestions and help :))

I created a new environment with Conda, reinstalled scvi-tools and now it works. Thank you for your suggestions and help :))


php: error while loading shared libraries: libicui18n.so.52: cannot open shared object file: No such file or directory

after upgrading to Ubuntu 15.10, from 15.4 I get the following error:

php: error while loading shared libraries: libicui18n.so.52: cannot open shared object file: No such file or directory

libicui18n.so.52 exists on my system, but Im not sure where phpbrew is looking for it.  Can someone let me know so I can symlink it to the right location?

Can you show me the path of libicui18n.so?
And did you use -- --with-icu-dir=... to specify the prefix of icu?
Can you also show me the ./configure ... command generated in the console? you can use phpbrew --debug install ... command.

hiya sorry for the delay.
turned out it was looking for the :i386 version, which I didnt have. Installed a library with it and then problem solved.   I didnt run it in --debug mode to start with.  Ill know for next time.
many thanks


Build keeps failing, same repetitive errors keep popping up.

Describe the bug
Trying to deploy app, keeps running into build errors...
Link to the Bolt URL that caused the error
https://bolt.new/~/bolt-vite-react-dfnkav
Steps to reproduce

Deploy App
Errors will appear, won't resolve. I'm just wasting tokens trying to resolve the issue.

Expected behavior
The errors would get resolved and the app would deploy
Screen Recording / Screenshot

Platform
Browser name = Safari
Full version = 17.1
Major version = 17
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/17.1 Safari/605.1.15
performance.memory = undefined
Username = krischulz
Chat ID = d6a7b8c098cb

Additional context
The OPENAI_API_KEY in .env and server/config.js match up but it continues to spit out the same error.
Error:
Let me analyze the current state and identify the issues:
The main issue appears to be with the environment variables. The app is using both OPENAI_API_KEY in the Netlify functions and server code, but the client-side code is still trying to access VITE_OPENAI_API_KEY.
The server configuration needs to be updated to use the correct environment variable.
.......................................................
.env
OPENAI_API_KEY=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
VITE_FIREBASE_API_KEY=AIzaSyAADGTSDQ7I51OULeOA5--EDHbCfhe8Kms
VITE_FIREBASE_AUTH_DOMAIN=vite-fc1ac.firebaseapp.com
VITE_FIREBASE_PROJECT_ID=vite-fc1ac
VITE_FIREBASE_STORAGE_BUCKET=vite-fc1ac.appspot.com
VITE_FIREBASE_MESSAGING_SENDER_ID=987481605767
VITE_FIREBASE_APP_ID=1:987481605767:web:4f7ea1ccbd0c9f5c269a22
..........................................................
Server/config.js
import dotenv from 'dotenv';
import { fileURLToPath } from 'url';
import { dirname, join } from 'path';
const __filename = fileURLToPath(import.meta.url);
const __dirname = dirname(__filename);
dotenv.config({ path: join(__dirname, '../.env') });
export const config = {
port: process.env.PORT || 3001,
openaiApiKey: process.env.OPENAI_API_KEY,
nodeEnv: process.env.NODE_ENV || 'development',
corsOrigin: process.env.CORS_ORIGIN || '*'
};
if (!config.openaiApiKey) {
throw new Error('OPENAI_API_KEY is required');
}
......................................................................

Appreciate the feedback! First, we recommend using Chrome as Bolt has the best support for that browser. You can read more here. It also looks like a CORS issue is involved, which we are tracking here.


showtabline won't work

Hi, i got a problem i'm trying to set showtabline=1 but it won't work i still view all the tabs also if I one just one tab open.

Not something I can reproduce anymore


Unable to resolve action maui-actions/apple-provisioning@2

Unable to resolve action maui-actions/apple-provisioning@2, the provided ref 2 is the shortened version of a commit SHA, which is not supported. Please use the full commit SHA 2d313f3be61a3c001d0550951fd049811e351d70 instead.

I'm also getting this, what's the reason behind this? 🤔

Its missing the right tag. all tags should be in format v{version} like v1 or v1.1.1. Fork it and add a tag for yourself

But would be great if @Redth as the repo owner could add it, so that one could actually use this action according to its README.

getting this error as well


Error of file path in lib_mnt.py (extracting .zip in DTMCreation.py)

There is an error on the building of path during extraction of .zip file.
Actually, the extraction create a folder with the same name of the .zip, and when you make a merge (line 486 for example), this folder is forgotten. Thus, the script does not find the right file.
You can see below the console return :
ERROR :
/mnt/data_netapp/raster/monde/Perou/SRTM/srtm_21_14.tif
Archive:  /mnt/data_netapp/raster/monde/Perou/SRTM/srtm_21_14.zip
creating: /tmp/18MUSp0z_hpbk/tmp063det8b/srtm_21_14/
inflating: /tmp/18MUSp0z_hpbk/tmp063det8b/srtm_21_14/srtm_21_14.tif.aux.xml
inflating: /tmp/18MUSp0z_hpbk/tmp063det8b/srtm_21_14/readme.txt
inflating: /tmp/18MUSp0z_hpbk/tmp063det8b/srtm_21_14/srtm_21_14.hdr
inflating: /tmp/18MUSp0z_hpbk/tmp063det8b/srtm_21_14/srtm_21_14.tfw
inflating: /tmp/18MUSp0z_hpbk/tmp063det8b/srtm_21_14/srtm_21_14.tif
gdalwarp  -r cubic -srcnodata -32767 -dstnodata 0  /tmp/18MUSp0z_hpbk/tmp063det8b/srtm_21_14.tif /tmp/18MUSp0z_hpbk/tmp063det8b/srtm_21_14nodata0.tif
ERROR 4: /tmp/18MUSp0z_hpbk/tmp063det8b/srtm_21_14.tif: No such file or directory
ERROR 4: /tmp/18MUSp0z_hpbk/tmp063det8b/srtm_21_14.tif: No such file or directory

Hi, I guess I have taken too long to reply, but did you keep the command line that generates this error ? So that I can test ?
Olivier

Hi, I use MAJA in a docker so the history is not kept, but the command line was something like this :
python3 start_maja.py -z -f folders.txt -g GIPP_S2AS2B_xxx -l LUT_S2AS2B_xxx -t 18MUS -s 18MUS -d 20180101 -e 20190601
I made a correction on my pull request in lib_mnt.py to manage the error, you can see it.
Thanks a lot
Nicolas


Chapter02/car.py returns bad input shape

Chapter02/car.py returns bad input shape with scikit-learn 0.18-2.  How to fix?
-> python car.py
/usr/lib64/python2.7/site-packages/sklearn/cross_validation.py:44: DeprecationWarning: This module was deprecated in version 0.18 in favor of the model_selection module into which all the refactored classes and functions are moved. Also note that the interface of the new CV iterators are different from that of this module. This module will be removed in 0.20.
  "This module will be removed in 0.20.", DeprecationWarning)
Accuracy of the classifier: 78.19%
Traceback (most recent call last):
  File "car.py", line 45, in <module>
    input_data_encoded[i] = int(label_encoder[i].transform(input_data[i]))
  File "/usr/lib64/python2.7/site-packages/sklearn/preprocessing/label.py", line 143, in transform
    y = column_or_1d(y, warn=True)
  File "/usr/lib64/python2.7/site-packages/sklearn/utils/validation.py", line 562, in column_or_1d
    raise ValueError("bad input shape {0}".format(shape))
ValueError: bad input shape ()

I've got the same problem. Here is the solution that I figured out.
Replace the code in line 45 by the following line:
  input_data_encoded[i] = int(label_encoder[i].transform([input_data[i]]))

The function transform() takes parameter as a list of values. Even you have a single value, you still have to convert into a list of a single value, i.e., [input_data[i]].


Cylinder jwt

This PR implements building and parsing of the custom cylinder JSON web token values.
The process is divided between the JsonWebTokenBuilder, which is used to construct encoded, signed token.  As of this commit, the header values and claims values are a flat object of (String, String) pairs.
The token is read and verified by the JsonWebTokenParser.  This parser wraps a Verifier instance which provides the signature verification.  It produces JsonWebToken struct that can be used to examine the token's claims.
As of this PR, the algorithm choice for the JWT (the field "alg" in the header) is hard-coded to "secp256k1", as this is currently the only signing algorithm implemented in Cylinder.

Use of non-hardcoded algorithm names will depend on #23


[NCL-8775] Show SSH details for PNC admins

This is the new PR in replacement of https://github.com/project-ncl/pnc-web-ui-react/pull/552.

Due to missing backend support(see NCL-8756), 401 error is expected for PNC users on build log details page after this PR get merged. cc @matedo1 to confirm if we still want to proceed.

Hi @DnsZhou , I've noticed jira number (PR title, commit message) is wrong


bug in occurs_at in combination with recurrence_dates

When adding dates using recurrence_dates, the occures_at returns a wrong value:
require 'ice_cube'

s = IceCube::Schedule.new(Time.new(2011,10,1, 18, 25))
s.add_recurrence_date(Time.new(2011,12,3,15,0,0))
s.add_recurrence_date(Time.new(2011,12,3,10,0,0))
s.add_recurrence_date(Time.new(2011,12,4,10,0,0))

p s.next_occurrences(12)

# Will return false, but it should return true
p s.occurs_at?(Time.new(2011,12,3,15,0,0))

This is because the method find_occurrences that is called in the occurrences method does only filter the rules, not the recurrence dates.

Hi there - thanks for the bug report - this is fixed in the v0.7 branch which will be merged into master in a few days.  Its unfortunately not ready for use yet, but will also include a bunch more improvement and optimizations.
Thanks for your patience with the issue

version 0.7.0 just merged in and pushed up.
give it a shot at your earliest convenience and please let me know if you have any issues

Yeah, works perfectly.


Indicators: Always display intermediate target periods regardless of whether or not there is data

Stories:

When I review targets versus actuals and % Met on the indicator page, I want to see all of my intermediate targets, so that it's clear how they contribute to the LoP target.
When I've just finished configuring my periodic targets, I want to immediately see them on the Indicators page so that I can be confident they're set up the way I need them.

ACCEPTANCE CRITERIA
After saving new targets, I immediately see all of them on the Indicators page, regardless of whether or not there are any data records associated with any given period.

When there is no data for a period, the target row/s still appears.
Target names are correctly displayed.
Date ranges (wherever applicable) are correctly displayed.
Message is changed from "There are no data records to display." to a more concise, "No data collected" (Please don't put a period after the word "collected" -- it's more of a label than a proper sentence.)
UI detail: The "No data collected" and "New Data" cells span 6 columns, for a cleaner look.

Following are mockups in several target configuration scenarios:

Validated with all target frequency types.

I'm seeing this on the dev server but am not able to reproduce in production. I wonder if it was introduced by the fix. The problem is the long red line extending to the right edge of the table. It should only go around the Date Collected cell.

Tested all target frequencies on Chrome, Firefox, and Safari. The target periods show up regardless of whether or not there are data records associated with them.


Merge error in the README

Oops. I think I screwed up the merge. There are merge artifacts in the README file. :-1:

#89

Merged to master. Have a good day/evening/night/polar-day!


docs: uefi - install nixos into SSD / eMMC

Fix #50

@zilti Does this installation process work for you?

Thank you for the effort, it is really appreciated, but - I don't know if it does work for me or not. Because I have absolutely no intention to currently either learn or use colmena just to install NixOS.

@zilti The use of "colmena" is not a strict requirement in this context. You can also opt for "nixos-rebuild" as a method to deploy the system. You can refer to HeroBrine1st/flake as a practical example.
However, it's important to note that this project is geared towards users who already have a good grasp of NixOS. Offering a detailed, beginner-friendly guide that explains every aspect is not within the current goals of this project.
That being said, contributions are always appreciated, so feel free to submit PRs!

I do have a decent grasp of NixOS, using plain Nix, but also flakes and home-manager for my setup, and disko for partitioning. "Look at that random user's personal config" is not documentation. There is a massively wide gap between that and a "detailed, beginner-friendly guide that explains every aspect".
Good luck with the project!

I do have a decent grasp of NixOS, using plain Nix, but also flakes and home-manager for my setup, and disko for partitioning.

Then convert the colmena demo to a nixos-rebuild one should not be a problem for you.

"Look at that random user's personal config" is not documentation. There is a massively wide gap between that and a "detailed, beginner-friendly guide that explains every aspect".

I mean we have no doc for this currently,  but you can submit a PR for this if you find it troublesome.

Good luck with the project!

I apologize for this if you find my words not friendly, I'm not a native speaker...


Logic error in case statement example

The example here is passing strange:
https://learn.adacore.com/courses/intro-to-ada/chapters/imperative_language.html#imperative-language-case-statement
You have the cardinal point 0|360 assigned to due east. While 0|360 is universally assigned to due north.
All the other cardinal points are similarly out by 90 degrees.
Which only goes to show, even the most rigorous language can't protect you from logic bugs :-)
Excellent tutorial by the way - much appreciated. But with a language that's marketed to safety-conscious engineers it might be better if you didn't send them off in the wrong direction...
         when 0 | 360 =>
            Put_Line (" is due east");
         when 1 .. 89 =>
            Put_Line (" is in the northeast quadrant");
         when 90 =>
            Put_Line (" is due north");
         when 91 .. 179 =>
            Put_Line (" is in the northwest quadrant");
         when 180 =>
            Put_Line (" is due west");
         when 181 .. 269 =>
            Put_Line (" is in the southwest quadrant");
         when 270 =>
            Put_Line (" is due south");
         when 271 .. 359 =>
            Put_Line (" is in the southeast quadrant");
         when others =>
            Put_Line (" Au revoir");
            exit;
      end case;

Thanks for reporting the issue!
It was fixed with PR #580 and is now published on learn.adacore.com.


ARROW-8678: [C++/Python][Parquet] Remove old writer code path

The new code has been the default for 2 release.

going to merge this.


get_libname(): resolve possible symlink returned by dladdr().

At least of Fedora systems, /lib64 -> /usr/lib64 symlink exists and, if
returned there, it messes with the rest of the logic without realpath().

+1

Thanks, merging.


Can't I use this api now ?

Whenever I check balance I get fallowing error.
 {"label":"0","walletAddress":"00","balance":0.0,"totalReceived":0.0,"message":"Internal Server Error: {\"error\":\"Address not found in this wallet\",\"warning\":\"This endpoint has been deprecated, for the best safety and security, use the HD API instead: https://github.com/blockchain/service-my-wallet-v3#enable-hd-functionality\"}"}
This suggest to change endpoins.
Kindly help

Yes.
this line
string route = $"merchant/{identifier}/balance";
needs to change to
localhost:<port>/merchant/...
as stated here.
You can change this locally for now, and I will fix this. Thanks for pointing this out.


[ic-popover-menu] - VoiceOver keyboard navigation bug

Summary of the bug
When trying to navigate through ic-popover-menu using a keyboard in combination with VoiceOver, it is not possible to navigate and click on the back button on any subMenu. The keyboard will skip over it and instead select the whole popover group. Can still navigate and click on other menu items.
🪜 How to reproduce

Go to the ic-popover-menu storybook
Click on the first example
Open up VoiceOver
Navigate to the second subMenu
See the error

📸 Screenshots or code
https://github.com/mi6/ic-ui-kit/assets/117368893/11dad939-a1a3-48b3-8d43-65f857fb07e0
🖥 📱 Device

Type: Desktop
Device: MacBook
Browser version: Firefox 126, Chrome 125

🧐 Expected behaviour
The back button should be focusable and clickable by keyboard, as it is when not using VoiceOver.
Additional info
There are also a few quirks and differences when navigating the popover using a keyboard & VoiceOver vs not using VoiceOver. For example, I could only navigate through the menu items using the left/right arrow keys, and clicking involved ctrl + opt + space. These may just be VoiceOver intended behaviour, however the keyboard experience is much smoother when not using VoiceOver.

The back button is accessible to screen readers, it's just at the bottom of the menu in the tab order (same for keyboard and for voiceover)
this feels deliberate, i'm assuming we wanted the back button to be the last thing someone tabs through - but it does cause a misalignment between visuals and nav order. This is potentially bad practice and maybe a WCAG failure, i'll have a look...


Updater Component is not available

Home Assistant release with the issue:
0.94.0

Last working Home Assistant release (if known):
not known
Operating environment (Hass.io/Docker/Windows/etc.):

arch | x86_64
dev | false
docker | true
hassio | false
os_name | Linux
python_version | 3.7.3
timezone | Europe/Berlin
version | 0.94.0
virtualenv | false
Component/platform:

Updater
Description of problem:
The updater component is not available in the frontend overview. The configured lovelace component stopped working. I cannot find the updater component in the states overview either. This could be related to this issue.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
# Enables configuration UI
#default_config:

# Manually enables config items
#cloud:
config:
#conversation:
frontend:
history:
logbook:
map:
mobile_app:
person:
ssdp:
sun:
system_health:
updater:
zeroconf:

Traceback (if applicable):
2019-06-08 00:18:07 INFO (SyncWorker_6) [homeassistant.loader] Loaded updater from homeassistant.components.updater
2019-06-08 00:18:08 INFO (MainThread) [homeassistant.setup] Setting up updater
2019-06-08 00:18:09 INFO (MainThread) [homeassistant.setup] Setup of domain updater took 1.0 seconds.

Additional information:
No additional information.

Any ideas anyone?

Hello, everybody. Am I the only one missing the updater component? Unfortunately, I'm not experienced enough to find the cause of this error.

This is not working for me either with 0.95.4.

Neither with 0.96.1

I have debugged the code and I think I have found the problem:

The updater URL returns an old version https://updater.home-assistant.io/
{"release-notes":"https://www.home-assistant.io/blog/2019/06/26/release-95/","version":"0.95.0"}

An SSLError while fetching the current version. I can't find these SSL Errors on my production server so I think this is unrelated.

2019-07-23 08:17:38 ERROR (MainThread) [homeassistant.components.updater] Could not contact Home Assistant Update to check for updates
2019-07-23 08:17:46 ERROR (MainThread) [homeassistant.core] Error doing job: SSL error in data received
Traceback (most recent call last):
  File "/usr/lib/python3.7/asyncio/sslproto.py", line 526, in data_received
    ssldata, appdata = self._sslpipe.feed_ssldata(data)
  File "/usr/lib/python3.7/asyncio/sslproto.py", line 207, in feed_ssldata
    self._sslobj.unwrap()
  File "/usr/lib/python3.7/ssl.py", line 767, in unwrap
    return self._sslobj.shutdown()
ssl.SSLError: [SSL: KRB5_S_INIT] application data after close notify (_ssl.c:2609)

I think the problem is related to the outdated JSON at https://updater.home-assistant.io/. So the current version is always newer than the newest version and the updater component remains silent.
I have two recommendations:

get the latest version from the github repository. Unfortunately, I have no idea how to do this, but I think this is the best solution.
always let the updater component display the latest version and a boolean value if the current version is lower.

Translated with www.DeepL.com/Translator

Even if my PR gets merged, I need someone to update the outdated JSON at https://updater.home-assistant.io/ Any ideas who might be able to do that?


Inconsistency with Firefox and localhost on MathJax snippets, and possible Accessibility issue.

Replace the text below with the details of the issue you are facing.
DO NOT simply erase the form and type a free-form response.
Issue Summary
I'm getting different results from the same HTML document, concerning the 2 MathJax snippets,
when loading the file in Firefox:

using localhost:
http://localhost/DEIMS/DEIMS2024-paper-pdf20-6Sep-pretty-DPUB.html
remotely via the web:
http://science.mq.edu.au/~ross/TaggedPDF/DEIMS2024/DEIMS2024-paper-pdf20-6Sep-pretty-DPUB.html

Using localhost, there are extra <div> sections at the end of the HTML <body>,
as seen in the following image, using the Developer Tools view:

Also, when running Accessibility-checker plug-ins, there are some warnings produced, concerning these active <div>s, with ARIA attributes. (see below)
Across the web:

There are no issues with the visual display, nor with activating pop-up menus over the MathJax snippets.
Using alternative browsers, Safari and Chrome, the views produced seem to be identical to the online one from Firefox; that is, no issues are evident using those browsers.
Possible Accessibility issue: should there be aria-atomic="true"?

The Siteimprove Accessibility checker seems to think that with assertive it is better to also set aria-atomic.
While perhaps not a bug, it would be reasonable to do so.
Also, the AInspector accessibility plug-in also has some Manual Check items related to these assertive divs.
Is this really an issue?
Steps to Reproduce:
Explained above.

Any other information you want to share that is relevant to the issue
being reported. Especially, why do you consider this to be a bug? What
do you expect to happen instead?
Technical details:

MathJax Version: 3.2.2
Mac OS X 13.5.2 (Ventura)
Browser: Firefox 130.0  (64 bit) — latest

I am using the following MathJax configuration:
MathJax = {
    loader: {
      load: ['[tex]/noerrors', 'input/tex-base', 'a11y/complexity']
    },
    options: {
      ignoreHtmlClass: 'tex2jax_ignore',
      processHtmlClass: 'tex2jax_process',
      enrichSpeech: 'deep',
      makeCollapsible: true,
      a11y: {
        highlight: 'None',
        backgroundColor: 'White',
        foregroundColor: 'Black',
        speech: true,
        subtitles: true,
        speechRules: 'mathspeak-default'
      }
    },
    tex: {
      packages: ['base', 'ams', 'noerrors', 'noundefined'],
      inlineMath: [['$', '$'], ['\\(', '\\)']],
      processEscapes: true
    }
};

and loading MathJax via
<script id="MathJax-script" async<EMAIL_ADDRESS>
Supporting information:

Please supply a link to a (live) minimal example page, when possible.
Given above.
Download it, and then view locally, using localhost.

If your issue is with the display of the mathematics produced by MathJax, include a screen snapshot that illustrates the problem, when possible.
Screen shots given above.

Check your browser console window for any error messages, and include them here.

No version information available for component input/tex-base<EMAIL_ADDRESS>MathJax: Invalid option "speechRules" (no default value<EMAIL_ADDRESS>

Include the MathJax configuration you are using, and the script tag that loads MathJax itself.

The div elements that you point to are part of the assistive tools and are triggered by activating the expression explorer.  Since the configuration you give doesn't enable them by default, it is likely that you turned them on when looking at the local version (or some other local page earlier on), and since the setting persists for the domain when you change it, that means all local files will have it turned on by default.  But the web page doesn't have it turned on until you do so explicitly (then it will be on for that domain until you turn it off again).  That is why you are having different experiences between the web page and your local page.  That also explains why Chrome and Safari show the same as the on-line version, as you either never turned on the assistive tools in them, or they don't save those settings for file:// URLs (I don't remember if they do or not).
So this is "expected behavior" in this case.
The error message
MathJax: Invalid option "speechRules" (no default value<EMAIL_ADDRESS>
is because the speechRules option was broken into two separate ones in v3.1.3 (see the "note" in the documentation).
Note that you don't need to load input/tex-base yourself, as that is part of the tex-mml-chtml.js file that you are loading.  Removing that should eliminate the other error you are seeing.
As for aria-atomic, yes, that could be added, but the mechanism used for the explorer in v4 (now out in beta release) has been changed and no longer uses aria-live regions, so the issue will be moot in v4.  It would be possible to give you a patch to add aria-atomic if that is important to you.  if I understand correctly, it should only have an effect if you have math notation inside another contain that has aria-atomic set.

See also the discussion in the MathJax Users Forum on aria-atomic.

Hi Davide,
It's been awhile since we last communicated.
I'm impressed with how MathJax has come along.
BTW, thanks for the quick reply, which is precisely along the lines of what I would expect.
I do recall setting Accessibility > Activate a bit over a week ago; but not noticing any immediate difference, just forgot about it.
A bit later when I checked with Siteimprove it showed errors that I wasn't expecting and didn't understand. I did not realise the connection — mea culpa, and thank you for pointing this out.
Thanks also for the Forum link, where Michael Boudreau asks whether the issue has come up for anyone else. You can now reply in the affirmative. :-)
Using Firefox's Accessibility panel view, I now see that there is quite a difference to the way the math snippets are presented there, with and without Activate.
Active:

In particular, with the alternative text of almost equals, supplied by MathJax.
Inactive:

This has MathML <math> tag and structure elements.
Can you comment on the request for interactive semantics?
Is there an attribute that I can supply for something sensible here?
Overall, my aim is to improve the Accessibility of Tagged PDF documents created with LaTeX, by implementing ARIA recommendations via attributes, as much as is possible within that setting. Testing the effectiveness of this requires deriving to HTML, then testing that the ARIA attributes have been correctly passed along, and serve their intended purpose.
— Read my test document and explore the website referred to, to see how effective this can be.
Getting the best possible scores in Accessibility Checkers is definitely part of my aim.
If the aria-atomic issue will be going away, then I see no need to work with any patched version; at least not locally-served. What would be the configuration to test using a v4 beta, say loaded online?
And did I see somewhere that there could be a way to use MathJax directly with a (tagged) PDF?
That would be something fantastic. I'd be prepared to help with producing examples and checking features for such a plug-in.
All the best.
Ross

OK, I found how to use the v4 beta:
<script id="MathJax-script" async<EMAIL_ADDRESS>
Now Siteimprove is happy, but there are mixed results with the AInspector plug-in, according to which MathJax features are enabled. It reports violations as follows.
Violation: Remove the unsupported aria-braillelabel attribute from mjx-container element.
Fixed by switching off Braille generation.
Violation: Remove the unsupported aria-label attribute from mjx-container element.
Role: generic

Would not this be fixed if role="math" were applied to each mjx-container element?
That would probably also remove the issue with aria-braillelabel or allow it to be renamed.
One other way to apparently fix this is to switch off semantic enrichment. But this reduces the mathematical content to a collection of individual letters and digits — not really desirable.

This next image shows with role="math" inserted by hand, after replacing the LaTeX source by the result that MathJax would generate from it.

AInspector is now happy, but

the interactive functionality of MathJax is disabled, and
the visual math styling is wrong.
However it suggests that if MathJax were to be doing this itself, then all would be good, yes?

Sorry for the long delay in getting back to you.  I've been looking into the issues you raised, and found some additional problems while working with VoiceOver.  So we are looking into some fixes, but don't have a final solution yet.  We are looking at role="math", but there are other problems to be resolved as well.

One other way to apparently fix this is to switch off semantic enrichment.

This will disable the assistive tools completely, so I don't think you want to do that.
I'll get back to you when we have something more concrete to offer.
Davide

Hi Davide.
Thanks for the reply.
In the time since my last posting I've come to the view that role="tree" is more appropriate semantically, designating the root node of the tree of pop-up menus and windows produced by MathJax. I recall seeing somewhere that someone else has suggested this already.
The role="math" belongs on the structure element that provides the (La)TeX source as content, or maybe a parent of this. My documents derive from Tagged PDF, produced from LaTeX source. I can provide this "math" role via an attribute to the <Formula> structure element, but cannot affect anything deeper provided by MathJax processing.
I certainly do not want to switch off semantic enrichment. The only point in doing that was to see if this had any effect on validation. But it has many other undesirable aspects.
Hope to hear from you again.
Cheers.
Ross


URI Cache won't update when model has parent

Description
Table runway_uris with uri mappings is not updated when newly created models that extend another class are saved. They are saved when they only extend Eloquent's Model. So having a custom class as the parent makes the difference here.
Steps to reproduce

Create a modal that extends another class

class Plant extends BaseModel
{

class BaseModel extends Model
{

Create a new instance and save

Plant::create([ 'id' => 1 ]);

New entries are not saved to runway_uris

Environment
Application Name: Test
Laravel Version: 11.14.0
PHP Version: 8.3.8
Composer Version: 2.7.7
Environment: local
Debug Mode: ENABLED
URL: local.test
Maintenance Mode: OFF
Timezone: UTC
Locale: en
Cache
Config: NOT CACHED
Events: NOT CACHED
Routes: NOT CACHED
Views: CACHED
Drivers
Broadcasting: log
Cache: file
Database: sqlite
Logs: stack / single
Mail: log
Queue: sync
Session: file
Statamic
Addons: 1
Sites: 3 (Nederlands, Deutsch, English)
Stache Watcher: Enabled
Static Caching: Disabled
Version: 5.12.0 PRO
Statamic Addons
statamic-rad-pack/runway: 7.7.2

What does your custom BaseModel class look like? Does it have a public static function booted() method by any chance?

What does your custom BaseModel class look like? Does it have a public static function booted() method by any chance?

Nope, it just has some common methods, nothing special. I'm not sure if it matters, but Plant uses mutators.

Hmm, can you share the content of your BaseModel class?

<?php

namespace App\Models;

use App\Services\Helpers\StringSanitizer;
use Illuminate\Database\Eloquent\Model;

class BaseModel extends Model
{
    protected function sanitizeName(string $name): string
    {
        return StringSanitizer::of($name)
            ->trim()
            ->removeExtraSpaces()
            ->uppercaseWords()
            ->get();
    }
}

It does seem to work when I run php please runway:rebuild-uris

Okay I found another thing, it does put one entry in the runway_uris table, but keeps overwriting that one record with the latest model I saved (the model_id also isn't correct, it's 11241, not 0):

When I remove my own generated ids and instead use auto-incremented ids (using ->id()) it suddenly works, any idea why?

Okay I'm just plain stupid, added public $incrementing = false; to my model and now it works, sorry for wasting your time

No worries!


HubMap

[ ] @hkir-dev to use the names here https://docs.google.com/spreadsheets/d/1tK916JyG5ZSXW_cXfsyZnzXfjyoN-8B2GXLbYD6_vF0/edit#gid=1379088218 to match ID with Label name and pcl ID (prefLabel should be the key)
[ ] @shawntanzk to spot check NS-forest

PCL ID column added after the CT/1 columns: ASCT+B_Tables_Published_v1.0_v1.1 - Brain_v1.1-PCL.xlsx

check with Josef how to handle NS-forest @dosumis
Hold off until above and purls resolve


CFE support for language versions

This includes:

support for the new JSON package format
using language version info in the JSON package format to determine whether to analyze a library in NNBD opt-in or opt-out mode
allowing a single library's opt-in / opt-out status to be overridden by an appropriately formatted comment at the top of the file (note: there are corner cases for part files under this scenario)
appropriate fallback logic for analyzing projects that lack a JSON package file

@johnniwinther
Please edit the description as needed, the current description is clone from the Analyzer support issue
Please add Estimate to the issue

Generally I'd say this has been fixed.
CFE (with 81d4ac07d488fb48383586b29f6503b674a8b917) reads the json file and uses that as the default appropriately. It allows libraries to overwrite via the comment syntax (and has supported that for ~half a year).

allowing a single library's opt-in / opt-out status to be overridden by an appropriately formatted comment at the top of the file (note: there are corner cases for part files under this scenario)

What corner cases? As I recall there is no corner cases: The part and non-part should just match..?

appropriate fallback logic for analyzing projects that lack a JSON package file

Details? I've heard nothing else, than:

Use .dart_tool/package_config.json; if it doesn't exist
Use .packages; if it doesn't exist
Use default version of the compiler;

Whichever way picks the "default" version it still allows to be overwritten by the first dart version comment in the code.
/cc @lrhn can you shed light on any of this?

The part and library file must have the same version marker. That is the entirety of that corner case. It does mean that having the same version is not enough, if one has a marker and the other gets the version from a default, then it's still invalid.
For the compiler, you should just read the package_config.json/.packages files as specified, and if there are none, use the SDK version.
The analyzer might be able to figure something out from the pubspec.yaml itself, if it runs before anyone has called pub get, but that's not something the compiler should worry about.
I don't know if Fasta need to be prepared for someone else providing the resolution to use for a particular file. That's not a priority for now, we need to get language version working correctly, then we can add features later.

What's the status?  Are we ready to close this?

I believe so. The CFE supports everything it's intended to support.


New version: PhaseSpaceIO v0.3.0

Registering package: PhaseSpaceIO
Repository: https://github.com/jw3126/PhaseSpaceIO.jl
Created by: @jw3126
Version: v0.3.0
Commit: 44c56285ee8dada7400c58ad627b678b00018e94
Reviewed by: @jw3126
Reference: https://github.com/jw3126/PhaseSpaceIO.jl/issues/11

Next time, please add the compat entries. I recommend using CompatHelper.jl


Fine-tuning models :  accuracy and loss never converges

System information

Have I written custom code (as opposed to using example directory):  Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):  Windows 7 Professional
TensorFlow backend (yes / no):  Yes
TensorFlow version:  1.13.1
Keras version:  2.2.4
Python version:  3.6
CUDA/cuDNN version:  10
GPU model and memory:  GeForce GTX 745 2 Go

Describe the current behavior
Hi !
I'm having a strange behavior during fine-tuning.
I'm trying to fine-tune Inception on two classes : dogs and cats, with the example from the documentation. But when I launch the learning, the accuracy stays around 0.5 and the loss is around 7 (that's a bit huge isn't it ?).
It seems that there's nothing wrong with the code, so what's happening there ?
I've tried the same code and dataset on another computer and models but the behavior is still the same !
Describe the expected behavior
The model accuracy tends to 1 and the loss to 0.
Code to reproduce the issue
The code : https://gist.github.com/Zappyford/5b6595b8a6596bc14551dbdda5339075
The dataset : https://mega.nz/#!tdYmlAoJ!RMkJjWBKubk7rFZQBngZ52vPF1rXi6FPA6eTQOJYses
Other info / logs
logs.txt

Hi!
This issue isn't related to a bug/enhancement/feature request or other accepted types of issue. There are several tutorials in web. Here is one close to your work. Check this also.
To ask questions, please see the following resources :
StackOverflow
Keras Slack channel
Keras.io support
Gitter
Thanks!
If you think we made a mistake, please open another issue and fill all the information asked by the template. Thanks!


cc13x2_cc26x2: define flash layout and support MCUboot bootloader

The following series introduces changes required for support MCUboot on the Texas Instruments SimpleLink CC13x2/CC26x2 MCU family and includes all the missing bits in the DTS files of the following boards:

CC1352P1 LaunchXL (CC1352P)
CC1352R1 LaunchXL (CC1352R)
CC1352R SensorTag (CC1352R)
CC26x2R1 LaunchXL (CC2652R)

The flash layout is defined as below (the same for all boards):

boot_partition:     40 KiB (0xa000)
slot0_partition:   136 KiB (0x22000)
slot1_partition:   136 KiB (0x22000)
storage_partition:  32 KiB (0x8000)
unused:              8 KiB (0x2000)

When building application for chain-load with MCUboot, the CCFG will be included only with the bootloader image, so it should be programmed first (to make sure CCFG contains proper configuration).
Example memory map when building default MCUboot for cc26x2r1_launchxl board:
west build -b cc26x2r1_launchxl bootloader/mcuboot/boot/zephyr
[...]
Memory region         Used Size  Region Size  %age Used
           FLASH:       27514 B        40 KB     67.17%
             RAM:       23168 B        80 KB     28.28%
      FLASH_CCFG:          88 B         88 B    100.00%
           SRAM1:          0 GB         8 KB      0.00%
        IDT_LIST:          0 GB         2 KB      0.00%

@gmarull twister tests fails on linking (power.c is not built when CONFIG_PM_DEVICE is enabled). This was removed in soc/arm/ti_simplelink/cc13x2_cc26x2/CMakeLists.txt.

@gmarull twister tests fails on linking (power.c is not built when CONFIG_PM_DEVICE is enabled). This was removed in soc/arm/ti_simplelink/cc13x2_cc26x2/CMakeLists.txt.

https://github.com/zephyrproject-rtos/zephyr/pull/62244

@vaishnavachath

Thanks for adding the MCUBOOT support, all of the launchpad also have an external SPI flash IIRC which can alternatively be used as Slot 1/scratch partition to achieve larger usable FW slots

AFAIR, swap-using-scratch MCUboot's algorithm is no longer recommended (that's the reason why I didn't include scratch_partition here).

This is what we did in BeagleConnect Freedom : https://git.beagleboard.org/beagleplay/cc1352/zephyr-beagle-cc1352/-/blob/sdk/boards/arm/beagle_bcf/beagleconnect_freedom.dts#L193
But that can definitely be an enhancement

Sounds fine, based on what I see, most of bits required for external SPI NOR flash support are missing in LaunchPad's DTSes at the moment anyway.
How about introduce later DTS overlay for all LaunchPads which would make all the partitions bigger and move slot1_partition and storage_partition to external SPI NOR? This would allow to use dual-image feature with just internal flash or in combination with external one.

Yes sure, the current changes looks good.


Update doco for  tap()

This might not be perfect, but it's better than what's there. :)

Thanks!


upgraded deps

This patch worked for me to be able to use FA with webpack 4. However, I did not test for other webpack versions, But I think it's ok.

No answer at the moment, no problem, @tapz have repackaged this to npm here https://github.com/gowravshekar/font-awesome-webpack/issues/41#issuecomment-413213495
Thank you both !


Phone refuses to connect to wifi hotspots with WPA3/WPA2 mixed mode enabled

I tried both my access point and a wifi hotspot on another Android phone. The Pinephone with GloDroid couldn't connect and said that the password was probably wrong (which it wasn't), and it connected successfully once I switched the Security setting from WPA2/WPA3-Personal to WPA2-Personal in the Wi-Fi hotspot settings of the other Android phone that I used as a hotspot. I also tried WPA3-Personal (without WPA2), and unsurprisingly, this didn't work either.
This is build 2024w44 (Android 15), installed on an SD card on a Pinephone, specifically UBports Community Edition with 2GB RAM.
I understand that lack of WPA3 support is a hardware limitation in the Pinephone's SoC. But at least in this answer in an OpenWrt forum thread, it seems to be implied that at least WPA3/WPA2 mixed mode compatibility is something that could be fixed in the firmware.
Can this indeed be fixed in GloDroid? Or is it an implementation in AOSP, which needs to be fixed upstream?
Thank you for your continued hard work on this project, and have a happy and healthy new year.

I just tried running postmarketOS 24.12 on the same Pinephone, and I got the exact same limitation with that OS. It could only connect to a WPA2-only access point. Neither WPA3-only nor WPA3/WPA2 mixed mode worked. This was with the iwd wifi manager selected when I built the postmarketOS image for this phone. I haven't tested this with wpa_supplicant.
Still, isn't the whole point of WPA3/WPA2 mixed mode to allow support for older devices?


Remove question about the Nile

https://github.com/python-discord/sir-lancebot/blob/13620a6461a78e43e8b16b9ad35691ff30622332/bot/resources/evergreen/trivia_quiz.json#L194
There's no way to definitively say if the Nile river or Amazon river is larger than the other. While the Amazon indisputably discharges more water, I believe there's some debate about where the Amazon definitively starts, and thus there's no way to say which is longer.
Because this question is ambiguous, it should be removed.

Could we possibly add the thing about the dispute in the info part?

@MrKomodoDragon That's fine if we change the question to be one that has a definitive answer.

Hey Guys,
i am completely new to open source but an intermediate in python. I would like to take over this request if that is ok for you.
Best Regards,
Arez1337

@Arez1337 we actively encourage you to do so. Let us know in the #dev-contrib channel if you have any questions.

Ok, I requested a pull for my updated repository with the removed question https://github.com/python-discord/sir-lancebot/pull/598


Sidetrail job failures

Problem:
The Sidetrail test patches some code, which is failing

+ cp /codebuild/output/src117552917/src/github.com/awslabs/s2n/tests/sidetrail/working/s2n-cbc/../../../../crypto/s2n_hmac.c crypto/
--
411 | + cp /codebuild/output/src117552917/src/github.com/awslabs/s2n/tests/sidetrail/working/s2n-cbc/../../../../crypto/s2n_hmac.h crypto/
412 | + patch -p5
413 | patching file crypto/s2n_hmac.c
414 | Hunk #1 succeeded at 29 with fuzz 2 (offset 3 lines).
415 | Hunk #2 succeeded at 241 (offset 3 lines).
416 | Hunk #3 succeeded at 324 (offset 10 lines).
417 | patching file crypto/s2n_hmac.h
418 | Hunk #2 succeeded at 73 (offset 1 line).
419 | + cp stubs/s2n_hash.c crypto/
420 | + cp stubs/s2n_hash.h crypto/
421 | + mkdir -p error
422 | + cp ../stubs/s2n_errno.c error/
423 | + mkdir -p tls
424 | + cp /codebuild/output/src117552917/src/github.com/awslabs/s2n/tests/sidetrail/working/s2n-cbc/../../../../tls/s2n_cbc.c tls/
425 | + patch -p5
426 | patching file tls/s2n_cbc.c
427 | Hunk #3 succeeded at 107 with fuzz 2.
428 | + mkdir -p utils
429 | + cp /codebuild/output/src117552917/src/github.com/awslabs/s2n/tests/sidetrail/working/s2n-cbc/../../../../utils/s2n_safety.c utils/
430 | + cp /codebuild/output/src117552917/src/github.com/awslabs/s2n/tests/sidetrail/working/s2n-cbc/../../../../utils/s2n_safety.h utils/
431 | + patch -p5
432 | patching file utils/s2n_safety.c
433 | Hunk #1 FAILED at 55.
434 | 1 out of 1 hunk FAILED -- saving rejects to file utils/s2n_safety.c.rej
435 |
436 | [Container] 2020/07/21 21:40:26 Command did not exit successfully codebuild/bin/run_sidetrail.sh /sidetrail-install-dir ${CODEBUILD_SRC_DIR} exit status 1
437 | [Container] 2020/07/21 21:40:26 Phase complete: BUILD State: FAILED
438 | [Container] 2020/07/21 21:40:26 Phase context status code: COMMAND_EXECUTION_ERROR Message: Error while executing command: codebuild/bin/run_sidetrail.sh /sidetrail-install-dir ${CODEBUILD_SRC_DIR}. Reason: exit status 1

Solution:
Update the patches.  Is there a better way to generate these patch files?

Does this change what S2N sends over the wire? If yes, explain.
Does this change any public APIs? If yes, explain.
Which versions of TLS will this impact?

Requirements / Acceptance Criteria:
What must a solution address in order to solve the problem? How do we know the solution is complete?

RFC links: Links to relevant RFC(s)
Related Issues: Link any relevant issues
Will the Usage Guide or other documentation need to be updated?
Testing: How will this change be tested? Call out new integration tests, functional tests, or particularly interesting/important unit tests.

Will this change trigger SAW changes? Changes to the state machine, the s2n_handshake_io code that controls state transitions, the DRBG, or the corking/uncorking logic could trigger SAW failures.
Should this change be fuzz tested? Will it handle untrusted input? Create a separate issue to track the fuzzing work.

Out of scope:
Is there anything the solution will intentionally NOT address?

Fixed with #2166


Lazagne and pypykatz

Hi @skelsec,
Great work, your project is very cool. I have started something like that some time ago just to retrieve wdigest passwords but your project seems more stable and do lot more.
I wanted to know if you were agree to add pypykatz in lazagne. It would be really nice, if a clear text password is found or the user hash, it will automatically retrieve softwares'passwords which used DPAPI to store their credentials (chrome, etc.). I managed password decryption using DPAPI if I have the user password or his hash.
So I think, both project are complementary. I could add it as an external dependency like that if some bugs appear, I will redirect issues to your github.
However, even if you accept, I will have a big problem to do it. I'm still really dependent on Python 2.7. Lazagne has been added as a post exploitation module of pupy, and it manages only Python 2.7 right now. But when I checked your code, I don't think it will be so hard to be compatible from Python 2.7 to newer Python versions (but I may have wrong).
So tell me what you think about it, it could be cool to join our work.
Have a nice day !
Alessandro

Hi,
Of course, I will mention you, I always try to mention libs / code / ideas when I use it.
I understand the testing problem. I could not be able to test it in so much hosts. I will check your code when I will have more time and if it's easy to do the portability between python versions, I will do a PR with pleasure. Right now, I haven't seeing your code so I have no idea how it works.
Have a nice day !


Please invite me to the community

Name
durgesh4993
Additional Context
I am Durgesh Kumar Prajapati, Pre- Final year student of B.Tech (CSE). I am a full stack PHP & MERN Developer. And also worked as an Open Source Developer. As an Open Source Contributor I have contributed to XWiki, The Tor Project and Omega Up Organization.

Invitation sent to join the GitHub Organization. Welcome to Infraform 🎉Please check your mail to accept the invitation sent by GitHub! 🤖About usInfraform is a remarkable open-source solution tailored for serverless platforms, aimed at empowering creators to craft their infrastructure solutions. 💻 We're thrilled to welcome you into our vibrant community! ✨ We eagerly anticipate your valuable contributions! 🤓 Don't forget to explore our community on Github and connect with us on LinkedIn 💜 Feel free to spread the word about our community to like-minded individuals interested in development, DevOps, technology, and collaborative contributions. 👉💜👈


The created image is referenced as "non-supporting cloud init"

Hi there
I followed this procedure.
Yet when I use the create image on the portal wizard, it states "The selected image does not support cloud init." in the "Advanced tab" of the creation wizard.
It works when provisioning from the CLI, yet my goal here is to expose this possibility in the portal. How to achieve this ? Is there some kind of metadata I could add on the image in some way ?

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

ID: 2821df0b-e273-692f-aa09-028b6f942a9e
Version Independent ID: a0b698f8-fb2c-5e5c-8b00-936f723df5cc
Content: Prepare Azure VM image for use with cloud-init
Content Source: articles/virtual-machines/linux/cloudinit-prepare-custom-image.md
Service: virtual-machines-linux
GitHub Login: @danis
Microsoft Alias: danis

Thanks for the feedback! We are currently investigating and will update you shortly.

@duckie Can you please share which specific Azure VM image are you using to enable cloud-init?
Also, please provide the link to the documentation, if you are following any, for steps you are taking on Azure portal. If not, screenshots will be helpful to understand it better.

@duckie Is there any update?

Hello,
I'm experiencing the same issue.
I can provision a Virtual Machine from the CLI using the --custom-data parameter (and it works), but when I use the portal, my image is stated not compatible with cloud-init.
My image is built on top of OpenLogic Centos 7.6.
Here is my waagent.conf:
Provisioning.Enabled=n
Provisioning.UseCloudInit=y
ResourceDisk.Format=y
ResourceDisk.Filesystem=ext4
ResourceDisk.MountPoint=/mnt/resource
ResourceDisk.EnableSwap=n
ResourceDisk.SwapSizeMB=0
ResourceDisk.MountOptions=None
Logs.Verbose=n
OS.RootDeviceScsiTimeout=300
OS.OpensslPath=None
OS.SshDir=/etc/ssh
OS.EnableFirewall=n

Hello
I am using  OpenLogic:CentOS-CI:7-CI:latest
Best

@duckie @mgraff Also, please provide the link to the documentation, if you are following any, for steps you are taking on Azure portal. If not, screenshots will be helpful to understand it better.

Hello @Karishma-Tiwari-MSFT,
Here is the command to provision from the CLI with custom data (clout-init.txt) :
az vm create --resource-group cloudinittest --name cloudinittest --image  OpenLogic:CentOS-CI:7-CI:latest --admin-username cloud-user --admin-password "my_password" --custom-data cloud-init.txt

When provisionning the same image from the portal, we cannot add custom data because the portail says "The selected image does not support cloud-init" (which obviously is not true) :

@Karishma-Tiwari-MSFT answered for me.

Thanks @mgraff @duckie for sharing the details. Let me investigate the issue and get back to you on this.

@mgraff @duckie I am also seeing 'The selected image does not support cloud init.' when I choose one of these marketplace images, for example,  CentOS 7.6

I am reaching out to Product teams and will share the updates here.

Hello @Karishma-Tiwari-MSFT
You are not trying the right image, Centos is published in various versions. The ones seen in the marketplace are not cloud init enabled.
However, you can observe the phenomenon the the "Ubuntu Server 18.04 LTS" image of the marketplace.
Additionally, the publishing protal for the marketplace as a check box asking if the published image is cloud init enable. But the equivalent for private managed images is nowhere to be found.
Thanks for your help.

@duckie Thanks for the explanation. It helped in understanding that the issue is for the private managed images. I am in touch with the product teams and will share as soon as I have an update.

Hello @Karishma-Tiwari-MSFT
You are not trying the right image, Centos is published in various versions. The ones seen in the marketplace are not cloud init enabled.
However, you can observe the phenomenon on the "Ubuntu Server 18.04 LTS" image of the marketplace.
Additionally, the publishing portal for the marketplace has a checkbox asking if the published image is cloud-init enabled. But the equivalent for private managed images is nowhere to be found.
Thanks for your help.
Edit: After discussing with someone on your side, the "cloud init" support field has not been made available on the public market publication portal but is yet reserved for some "core" images. So adding it to the portal for everyone to use would be nice too.

Can you please let me know when I can find check-box for enabling cloud-init on publishing VM offer?
I am trying to publish VM based on Ubuntu and want to have cloud-init available in portal.

@rgalexey The product team is aware of this issue. I will share the updates as I hear back. Currently, there is no option to do it for private managed images from Azure portal. It can be done from Azure CLI.

@duckie @rgalexey @mgraff  Thank you all for bringing this to our attention. The product team have acknowledged this and actively working on the option to have this feature in Azure portal. I will share when I have the ETA for this.
At this point, the workaround is to use Azure CLI option to provision it, enabled with cloud init custom data.
We will now close this for now. I will be tracking this feature and share as soon as I get the ETA and when it goes live. If you have additional questions let me know and we can reopen and continue.

Any update on when this might be available in the portal?

It is not rolled out in Azure portal yet. The product team has this feature as part of their discussions. I will share more as I get a clarity from the product team.

also will be happy to have this.. thanks.

A lot of people are waiting for this feature.
It would be better to keep this issue open until the feature is available.

Do you happen to have a rough estimate as to when this will be resolved? End of year?

This feature for Azure portal is in the roadmap. Custom data option will be available, if you do not have Cloud-Init enabled on the VM.

Custom data document: https://docs.microsoft.com/en-us/azure/virtual-machines/custom-data
Blog post : https://azure.microsoft.com/en-us/blog/custom-data-and-cloud-init-on-windows-azure/

Since this feature is in product and would take some time to be released, I will be closing this issue. I will share the information here when it is released to Azure portal.

I really dont get the "this is gonna take a long time so let's close it". What is the logic here ? Dont we close issues once it is actually fixed ?


Day313

Resolves #639.

Pull Request Test Coverage Report for Build 1365

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 1355:
0.0%

Covered Lines:
5

Relevant Lines:
5

💛  - Coveralls


Small tweak to 'build ladybird' install / run steps

"Build Prerequisites" explains what commands to do to get ready.
The next section is "Build steps : Using ladybird.sh"
The first step is running ladybird.sh
However at this point none of the prior steps would have placed ladybird.sh on the users system.
Please add these steps
$ git clone https://github.com/LadybirdBrowser/ladybird/
cd ladybird
cd Meta

There's no reason to cd Meta, the commands in the "Using ladybird.sh" section invoke them from the toplevel dir:
https://github.com/LadybirdBrowser/ladybird/blob/master/Documentation/BuildInstructionsLadybird.md?plain=1#L161-L169
# From /path/to/ladybird
./Meta/ladybird.sh run ladybird

As for the git clone step, I would expect anyone trying to build a browser from git to have cloned the repository already?

Agreed, see https://github.com/LadybirdBrowser/ladybird/issues/430#issuecomment-2211451171 for previous discussion.


Add support for Guitar extension in Wiimote plugin

Hi :)
I wanted to use the accelerometer / gyrometer in combination with my Wiimote guitar so I patched your (awesome) tool and added Wiimote guitar support.
Originally I just wanted to hack it in for personal use, but the changes were pretty simple and clean enough that I might as well share it with the world.
Please take a look at my pull request. Feel free to critique everything you don't like. Please also tell me if and where I should add more documentation about my changes.
I have tested my changes with the original Guitar that came with Guitar Hero 3 and everything seems to be working correctly. Attached you can find an example config that I used for testing.
clonehero.txt

Thanks for this. The code looks pretty good. I don't own a guitar attachment, so I can't test this, but I don't have much reason to doubt this works.

Thank you for merging this. If there's ever a bug report about the guitar extension, feel free to tag me :)


Publish universal wheels

AutobahnPython itself is a pure Python project, source compatible with Py2/3

fixed via https://github.com/crossbario/autobahn-python/pull/665


'fisher update' hangs.

On C9 Ubuntu machine.

Yes, this is a problem in machines other than OS X and ubuntu/trusty64, the only two platforms I have tested this well. The problem has to do with my use of find it seems. The milestone I am working towards will fix this by adding a --cache to fisher to retrieve the cache plain contents.
https://github.com/fisherman/fisherman/issues/11

Thanks for working on this, @bucaran.

:+1:

@RafaelCosman @simnalamburt
Guys, I was mistaken and replied to this message shortly after another user had reported a similar issue and claimed it had to do with find(1). This in fact, has nothing to do with that, (probably), and it's just the default behavior of update.
Fisherman has strong ties to the UNIX philosophy, at least it says so in the manual :smile: and one of the things that are common across commands is that streams are prioritized.
So, fisher update will hang because is waiting for input. This allows cool things like:
fisher --cache | fisher udpate

or
fisher update < bundle

So what is the recommend way to list/update all installed plugins?

Hey @RafaelCosman
Just, fisher update plugin1 plugin2 ....
fisher update -h
usage: fisher update [<name or url> ...] [--me] [--quiet] [--help]

       -m --me  Update Fisherman
    -q --quiet  Enable quiet mode
     -h --help  Show usage help

If you want to update all the plugins:
fisher --cache | fisher update

If you want to update Fisherman itself:
fisher update --me

I think making fisher update to update all plugins is the best default. Most package managers like npm, gem, etc update all installed plugin when executed with ~~ update and I think most users are used to it.
Yes fisher --cache | fisher update fits very well with UNIX philosophy but fisher update is also good default, and I think both can coexist with isatty(3) just like as the python binary does.
How do you think?

@simnalamburt Alright. Why not?  :+1:

Fixed in 0.3.0

Perfect


Seeds should germinate to replace 0 height organisms

Allows for replacement of grass and grass-like organisms.

Should actually be a separate structure property.


Linear min & max sats buttons

Describe the changes you have made in this PR
Sats buttons 4 linear values from min to max, with nice rounding & formating (1M, 1k, 100). min & max values are unchanged.
Type of change

feat: New feature (non-breaking change which adds functionality)

Checklist

[x] Self-review of changed code
[x] Manual testing
[ ] Added automated tests where applicable
[ ] Update Docs & Guides
For UI-related changes
[x] Darkmode
[x] Responsive layout

can you please try and fix code quality checks in the code?

@matjaz I don't think a linear distribution of the amounts between min and max is what I would want as a user. When I tip someone I use those buttons as kind of shortcuts for smaller amounts.
I don't think people would use a 1M sat button a lot, do you? :thinking:

Using LNURL, LN SERVICE specify min & max. So it's up to service what values they want.
Example I was considering is service balance withdrawal where max is balance.
When you tip someone you don't use LNURL (no min&max).
When min & max are not specified it fallbacks to current values(1k, 5k 10k, 25k).
There is no point of having disabled buttons.
What do you suggest, @reneaaron ?
@pavanjoshi914 fixed formatting.

https://lntictactoe.com & https://www.lnflip.com enables balance withdrawal.
Let's say user has a balance of 2500 sats, most of the buttons are disabled. For 150000 sats button values are too low.
So for tag

payRequest, Alby could have a max value of wallet balance or LNURL maxSendable
withdrawRequest, LN SERVICE defines maxWithdrawable

They also have built-in fees which covers withdrawal fees. Probably they also have throttling for withdrawals.

Any feedback on this?
I don't see the point in buttons with higher values than available. For both, payRequest and withdrawRequest.


Request for version info in command documentation

Firstly, thanks for the work on this project, really love showing outputs to customers.
I am reviewing commands (tests) that are listed in the documentation but I am running v0.0.125 and getting
CommandNotFoundException: The term 'Test-MtCisaPrivilegedPhishResistant' is not recognized as a name of a cmdlet, function, script file, or executable program.
It could be that the test is not included in my version or that I am doing something wrong in my .Tests.ps1 custom test. To be sure it is not the former, I would like to suggest the documentation includes the version this test (command) was introduced.
https://maester.dev/docs/commands/
Alternatively, can I view which commands (tests) are included in my version? Am I on the right track in this section of "Maester.psd1"?

You can run this to get the cmdlets that are available in your current version.
Get-Command -Module Maester

@matthewjlevy that cmdlet wasn't added until 0.0.126.
If you do an Update-Module Maester you should be able to use it.


Mark testbot-intel-nuc tests as required

testbot-intel-nuc tests are required to pass PRs on this repo


Intel doesn´t work

If Intel is picked up it appears in Log but there is no entry in the briefing tab.

If you cgoose the standard ArmA vanilla intel there will be no intel if you doesn’t change a variable directly on the intel object.
https://community.bistudio.com/wiki/BIS_fnc_initIntelObject
Here is the Bohemia function for intel. Maybe there is the “bug”

@Katalam Interesting, didn't know about that function, will check that out.
@3DSFreaker I actually forgot about this bug, but now it's pushed to the top of the ToDo list.


[Packaging] System package manager-friendly vendoring

Greetings, glorious mangal maintainers! I quietly package awesome software for Gentoo Linux. Because mangal is clearly both awesome and software, packaging mangal for Gentoo is what I am now doing.
Except... I can't. Cue sadness cat.

mangal: We Don't Need No "vendor/".
The mangal repository (and thus source tarballs derived from this repository) currently omits the top-level vendor/ directory required to reproducibly build and install mangal from source.
That's usually fine. In the absence of a vendor/ directory, go subcommands silently download and cache dependencies listed in go.mod into the user-local module cache (e.g., ~/go/pkg/mod/). Go developers and brave end users manually building mangal from scratch are accustomed to that.
That's less fine for foolhardy packagers (such as myself) attempting to package mangal for source-based redistribution at the system level. But "source-based redistribution at the system level" is the literal definition of Gentoo Linux. In other words, Gentoo isn't happy. Let's make Gentoo happy.
Gentoo: Actually, We Really Need "vendor/".
Most system-level package managers (e.g., emerge under Gentoo Linux) and language-level package managers (e.g., the Swift Package Manager (SPM)) sandbox at least filesystem access for security. Increasingly many also sandbox network access for the same reason. This is where our packaging trouble starts and ends.
Since mangal omits a vendor/ directory, go subcommands silently:

Download dependencies, thus violating network sandboxing during installation.
Cache dependencies, thus violating filesystem sandboxing during installation.

Admittedly, Gentoo can trivially circumvent filesystem sandboxing by just (A) caching to a temporary non-sandboxed work directory while building and then (A) deleting that directory after building. That's fine.
Gentoo cannot trivially circumvent network sandboxing, however. In theory, Gentoo could vendor mangal dependencies into a new source tarball that we host and maintain on your behalf... which is the problem. Gentoo ...okay, I mean me doesn't want to host and maintain separate source vendor tarballs for every single Go application in existence.
 Vendoring Go dependencies is non-trivial, error-prone, time-consuming, and anti-fun. Moreover, every single other Linux distribution (of which there are a countably infinite number) would need to reduplicate that same work. This is probably why most Linux distributions that currently package mangal sidestep this issue by just redistributing your binary Linux executables. That is what Arch does, for example. But that's bad – or at least fundamentally incompatible with the Gentoo religion.
We're zealous fanatics about source-based installation. We couldn't be wrong! :sweat_smile:
mangal: Yah. We're Still Unconvinced.
Omitting vendor/ is typically the best practice for Go middleware like libraries and frameworks. Conversely, committing vendor/ is typically the best practice for actual Go applications shipping executable binaries... like mangal. Sources or it didn't happen, so:

Should You Commit the Vendor Folder in Go? Conclusion:

The answer to the age-old question of, “should I commit the vendor dependencies in my Go project to source control?” is “almost always.“

To vendor, or not to vendor.
Very well. I can see that neither the lolcat nor the two blog articles I linked to above have convinced you. In that case, I shall now bludgeon you with verbose words.
There's really only one valid reasons to omit vendor/: repository size. But that's not a particularly valid reason in the modern era, because GitHub ...so, Microsoft is what I'm saying is already committed to taking the bandwidth and storage hit on your behalf.
There are many valid reasons to commit vendor/, however – including:

Compatibility with package managers that sandbox. Gentoo. Always Gentoo.
Deterministic builds. mangal builds are currently non-deterministic. By deferring to go subcommands that bang on remote servers for online installation, mangal builds tacitly assume that URLs, versions, and licenses are perfectly stable across all dependencies – forever. But that's never the case. Packaging is a constantly shifting landscape wrapped in a badly documented enigma inside a vanishing mystery of 404 errors. The Node.js world discovered this the rough way when left-pad spontaneously blew everything up.
Better faster harder CI/D. Continuous integration (CI) and deployment (CD) workflows suddenly become substantially faster and more robust, because go no longer needs to bang on remote servers for online installation. Since requisite dependencies are locally available under vendor/, CI/D turnaround (and thus turnaround for tests and PRs) measurably improves.

Gentoo: I Am Very Tired Now.
Thanks so much for all the automated manga downloading, mangal magicians. My manga backlog may currently be crying, but everyone here is a spectacular gift to humanity.

Wooah! Firstly, thank you for all the kind words, really appreciate it! Secondly, your points are very reasonable, you convinced me by 110% 😁 From now on I'll vendor for all my go projects
Just to be sure that I won't do anything wrong - all I have to do to is to run this?
go mod vendor

Or there's something extra needs to be done?

Yep, mangal does not depend on any C code, so I guess that's it. I'll roll vendored version soon. Thank you so much =)

vendor/ in the main, should be ok now!

Wow. So much amazing enthusiasm! You make binging manga even more fun, @metafates.
As soon as I package mangal for Gentoo with a working ebuild at my overlay, I'll submit a PR here adding Gentoo-specific installation instructions to your README.md. Until then, let's read manga until we pass out at 5:32 AM on a Wednesday evening again.

Haha, okay, I'll be waiting 😁


feat: implement rlp Decodable and Encodable traits for Signature

Motivation
Signature should implement alloy_rlp::Encodable and alloy_rlp::Decodable traits.

Solution
Implementation of alloy_rlp::Encodable and alloy_rlp::Decodable traits for Signature with unit tests.
Should close #82.

PR Checklist

[X] Added Tests
[X] Added Documentation

thank you for the contribution! there is some ongoing refactoring work to put the signature type in alloy-core, so some of this work may become part of that

thank you for the contribution! there is some ongoing refactoring work to put the signature type in alloy-core, so some of this work may become part of that

@prestwich Yes I saw that the issue was already linked to a big PR but as I saw that the signature aspect was not yet implemented on it, I told myself that I could open the specific PR for that.
Feel free to use this as you prefer.

closing as superseded by https://github.com/alloy-rs/core/pull/459


issue: Error "Not able to sort using field Rank

In Explorer, Jira issues displays the following error:
Error code: 400
Error message: Not able to sort using field Rank
Jira: v7.4.5
Update;
At startup of vs code, Jira Issues calls jira.baseUrl/rest/api/2/search that returns a json file: search.json. This file doesn't contains any Rank information

What does your jql expression look like?

I didn't change anything about jql expression, I just installed your plugin and gives the necessary settings jira.baseUrl, jira.username, jira.password, jira.port ("").After I restarted VS code and in the explorer, the error raised directly

The default JQL expression I have in there if none is provided looks like this

assignee = currentUser() AND resolution = EMPTY ORDER BY Rank ASC

Looks like I was making too many assumptions and ordering by rank only works if enabled first. In light of this I'll update the default expression to exclude ordering by rank.
@magneticcore  Thank you very much for bringing this to my attention.

fixed and released under version 1.2.1


test: add tests for Model timestamp

Description

add tests for the current behaviors.

Checklist:

[x] Securely signed commits
[] Component(s) with PHPDoc blocks, only if necessary or adds value
[] Unit testing, with >80% coverage
[] User guide updated
[x] Conforms to style guide

Only SQLSRV returns datetime with milliseconds.
1) CodeIgniter\Models\TimestampModelTest::testDoNotAllowDatesInsertArrayWithoutDatesSetsTimestamp
Failed asserting that two strings are identical.
--- Expected
+++ Actual
@@ @@
-'2023-11-25 12:00:00'
+'2023-11-25 12:00:00.000'

/home/runner/work/CodeIgniter4/CodeIgniter4/tests/system/Models/TimestampModelTest.php:93
phpvfscomposer:///home/runner/work/CodeIgniter4/CodeIgniter4/vendor/phpunit/phpunit/phpunit:106

https://github.com/codeigniter4/CodeIgniter4/actions/runs/6987412341/job/19014063118?pr=8249


Add new option to the DevTools extension to run build_runner

I'd like the option to run build_runner without always having to look for the command and run into the terminal. Would it be possible for the DevTools extension to run build or even watch?

This is an interesting idea! Extensions don't have this capability at the moment, but I've asked on the Flutter Discord if something like this would ever be supported. If it is or will be, I think this would be a good candidate for a DevTools extension for build_runner. I'll file an issue there if it's feasible.


wait for page build completely?

hi,
is there any way to wait for a page to build completely?
My case is that I run a startup after a reset in which I read several statuses from the EEprom and output them as text on the page.
I also output the text serially.
I see the serial text although the page is still under construction.
This has the consequence that the text is not displayed on the page.
Can I control the output of the text on the page and wait until the page has built up?
Translated with www.DeepL.com/Translator (free version)

@kellers76 I'm afraid this is one of those cases where a small sample application would go a long way to helping you out.
If you can post a zip file of a small program maybe using a const array instead of eeprom and a small number of text fields with serial output to the serial monitor that reproduces your problem I might be able to give you a solution.
I don't have the time to look over your full application so do not post that version.
Paul--

@kellers76 I'm afraid this is one of those cases where a small sample application would go a long way to helping you out.
If you can post a zip file of a small program maybe using a const array instead of eeprom and a small number of text fields with serial output to the serial monitor that reproduces your problem I might be able to give you a solution.
I don't have the time to look over your full application so do not post that version.
Paul--

hello Paul,
I get it reproduced in a trimmed down original code of mine, however it's not as small as you might need.
I don't want to take up too much of your time.
What I am noticing, I have a total of 3 EEprom queries after the gslc_SetPageCur where I am reading the last statuses. If I have 2 queries active then the text comes, if I activate the 3rd query then the text doesn't come anymore....
ske.zip

hello Paul,
I get it reproduced in a trimmed down original code of mine, however it's not as small as you might need.
I don't want to take up too much of your time.
What I am noticing, I have a total of 3 EEprom queries after the gslc_SetPageCur where I am reading the last statuses. If I have 2 queries active then the text comes, if I activate the 3rd query then the text doesn't come anymore....
ske.zip

@kellers76 No problem.   I'll look into this tomorrow morning.
How do i run your program to reproduce the error?
Paul--

@kellers76 No problem.   I'll look into this tomorrow morning.
How do i run your program to reproduce the error?
Paul--

@kellers76 Ok, this turned out to be easy fix.
You were overwriting the same text field.   On your third eprom read StatusBack==0 and you are replacing your previous message with spaces. So you don't see any messages.
I modified your sample to use three different text fields that you set up.
first eprom read uses m_pElemOutTxt122
second eprom read uses m_pElemOutTxt141
third eprom read uses m_pElemOutTxt141_198
I also modified button callback E_ELEM_BTN19 (BACK) to Main Page - Hauptseite
to set a few Boolean to show all three messages your demo uses.
Anyway, this should give you an idea of what was wrong.
Attached is my modified version of your sample.
Paul--
ske.zip

@kellers76 Ok, this turned out to be easy fix.
You were overwriting the same text field.   On your third eprom read StatusBack==0 and you are replacing your previous message with spaces. So you don't see any messages.
I modified your sample to use three different text fields that you set up.
first eprom read uses m_pElemOutTxt122
second eprom read uses m_pElemOutTxt141
third eprom read uses m_pElemOutTxt141_198
I also modified button callback E_ELEM_BTN19 (BACK) to Main Page - Hauptseite
to set a few Boolean to show all three messages your demo uses.
Anyway, this should give you an idea of what was wrong.
Attached is my modified version of your sample.
Paul--
ske.zip

oh god, this is really embarrassing...
in german i would say, "i didn't see the forest because of the many trees".
Sure, I overwrite the text with the next query if it has the value 0...
Sorry, I must look better next time

oh god, this is really embarrassing...
in german i would say, "i didn't see the forest because of the many trees".
Sure, I overwrite the text with the next query if it has the value 0...
Sorry, I must look better next time


feat: add row-reverse and column-reverse values to direction property…

… of Container component
PRI-325
Description
Add row-reverse and column-reverse values to direction property of Container component
Development checks

[ ] Add changeset according to guidelines (if needed)
[ ] Read CONTRIBUTING.md and Component API principles
[ ] Make sure that additions and changes on design follow Toptal's BASE design, and it's been already discussed with designers at #-base-core
[ ] Annotate all props in component with documentation
[ ] Create examples for component
[ ] Ensure that deployed demo has expected results and good examples
[ ] Ensure the changed/created components have not caused accessibility issues. How to use accessibility plugin in storybook.
[ ] Self reviewed
[ ] Covered with tests

Breaking change

[ ] codemod is created and showcased in the changeset
[ ] test alpha package of Picasso in StaffPortal

All development checks should be done and set checked to pass the
GitHub Bot: TODOLess action

PR commands

List of available commands:

@toptal-bot run package:alpha-release - Release alpha version
@toptal-anvil ping reviewers - Ping FX team for review

PR Review Guidelines

When to approve? ✅
You are OK with merging this PR and

You have no extra requests.
You have optional requests.

Add nit: to your comment. (ex. nit: I'd rename this variable from makeCircle to getCircle)

When to request changes? ❌
You are not OK with merging this PR because

Something is broken after the changes.
Acceptance criteria is not reached.
Code is dirty.

When to comment (neither ✅ nor ❌)
You want your comments to be addressed before merging this PR in cases like:

There are leftovers like unnecessary logs, comments, etc.
You have an opinionated comment regarding the code that requires a discussion.
You have questions.

How to handle the comments?

An owner of a comment is the only one who can resolve it.
An owner of a comment must resolve it when it's addressed.
A PR owner must reply with ✅ when a comment is addressed.

@toptal-anvil ping reviewers

@toptal-anvil ping reviewers

Looks great! Thank you. I have rebased the PR and updated one failing snapshot test (not directly related to your PR)

Thank you @mkrl


EntityDescription is not rendered if second entity is clicked

We noticed that if we click an Entity, and then immediately click another one, the EntityDescription is not rendered.
Looking at https://resium.reearth.io/examples/?path=/story/entity--description it seems that this example exhibits a similar behavior.
Clicking each entity in turn may or may not load the EntityDescription. Sometimes it does, sometimes it doesn't.

I believe it has something to do with this line:
https://github.com/reearth/resium/blob/main/src/EntityDescription/EntityDescription.ts#L69
When you click on a second entity while the infobox is active, the classList no longer contains cesium-infoBox-bodyless, which means it cannot find the node.

I've been noticing this problem as well, and it's preventing me from upgrading to 1.17.2 from 1.16.1. If this could be looked into it would be amazing

I spent a little time today figuring out a workaround to inject the class that it wants. It's not ideal but it seems to work for my purposes until the fix is accepted.
I put the following in the onSelectedEntityChange parameter on <Viewer. Change your ref as needed.
onSelectedEntityChange={(selectedEntity) => {
  const container = refViewer?.current?.cesiumElement?.infoBox?.container;
  const node = container.querySelector('.cesium-infoBox');
  node.classList.add('cesium-infoBox-bodyless');
}


Interface

I have been working on the interface and it is going pretty well. There are a lot of new features that will be added in the new update! I am not going to list them here because you can view the commit history and see the releases page once the draft is saved/the release is published.

#15


about volumeId health detected

version: 0.94
when i removed a volume id's dat and idx file,  the file on the volumeId also can be read from weed mount directory (maybe cache in memory?), and curl  http://xxx:9333/dir/status?pretty=y shows the volumeId also can be writable  ,  it means the volumeID can be allocated to write a file, then write a.txt file to the volumeid shows success(actual in cache not flush to disk?) ,and read succes now, but when restart the volume server directly or copy the volumeid back before restart,  Input/output error shows when read a.txt from weed mount directory ,the a.txt file lost

maybe make sure the .dat and .idx files are not removed.

Similar scenarios may also have the same problem ， like  dat file  corrupt, the file write to the corrupt volumeId maybe lost

@chrislusf   no mechanism to confirm the volumeId actual can be write before allocate the volumeid,
This is not the most important issue，the issue is the file write to the allocate unhealthy(maybe file corrupt or deleted..etc) volumeid success(application also believes that success ), actually, It didn’t write to volume disk at that moment,when restart volume server , the file lost . File loss is intolerable for file system because of  own mechanism.     when the file write to the the unhealthy volumeid should return fail errors immediately ,so the application can be sense that


Reproducibility difficulty

Hello,
We were unable to run an analysis with the new version of the tool while it worked with a previous version. There are two ways to solve this problem: fix the code in a new version (which was done in our case) or fix the versions used. cf https://github.com/BiaPyX/BiaPy-GUI/issues/7
In fact, it is possible to fix a version of the biapy GUI, but it is not possible to do it for the biapy container. Indeed, you have chosen to always use the latest version of the biapy container.
The ideal would be to allow the user to set the version of the biapy container that the GUI will use. This would ensure the reproducibility of the results. Moreover, when publishing in peer-reviewed journals, it is requested to provide the version of the software used in order to be able to redo the analysis.
Best regards and thanks for your great tool!
Benoît

Hi,
We are unable to implement an option to use a different container than the one the GUI expects due to internal checks that depend on BiaPy's core code. That being said, the reproducibility issues you mentioned are something we are keen to resolve.
So far, we've been making updates to have a stable version of BiaPy and its GUI. Is true that these updates have introduced several changes that impact backward compatibility. However, this is something we've been aware of, and we’ve already addressed it in BiaPy’s core code (we’ll be releasing an updated version in the next few days). While the GUI is a bit more complex, we’re working on it as well. Our goal is to implement an automatic translation of older configuration files (.yaml) to newer versions to ensure reproducibility. Currently, we’re considering configuration files from up to a year ago (January 2024). It would be helpful if you could share your configuration file, so we can make sure the changes we’re implementing also take your case into account, especially if it's older than January 2024.
Additionally, now that we’ve reached a stable version, we’ll create a version history of the GUI alongside its specific container. This list will be linked in the repository and on our website, ensuring 100% reproducibility going forward.

We have release a new version of the GUI, v.1.1.3, with the backward compatibility mentioned and also an option to choose the containers that GUI version supports among the ones listed in https://biapyx.github.io/GUI_versions/: you will find a dropdown in the last window, e.g., "Run workflow".
Currently, this last version can only use the last BiaPy core version, v3.5.5, but in it may support future releases, depeding on how the core code changes. This new GUI is prepared to automatically recognize these new containers that can be used and will extend the container list in the mentioned dropdown. Hope this brings a bit more of flexibility and the option to use always the same container version ensuring reproducibility. We decided to implement this feature like that to ensure also all containers listed there are 100% compatible with the GUI version the end user is running.

Hi,
Thank you for your reply and for your responsiveness. I am very aware that biapy is a highly evolving code (which is really good). So, it is great news to learn that we will now be able to have version control. I think this will be useful for users who are concerned about being able to reproduce their analyses.
Benoît


Mono/C# - Cannot reference external library

Godot version:
3.0.6 Stable Mono, also same issue with 3.1 beta mono
OS/:
Win 10
Issue description:
Cannot reference external DLL in C#. My solution compiles fine but when launching from within the editor it fails to find the required runtime DLLs. Getting the following output on the console
Unhandled Exception:
System.DllNotFoundException: fmodstudio.dll
  at (wrapper managed-to-native) FMOD.Studio.System.FMOD_Studio_System_Create(intptr&,uint)
  at FMOD.Studio.System.create (FMOD.Studio.System& studiosystem) [0x00006] in D:\Godot Projects\g\fmod_studio.cs:597
  at Node._Ready () [0x00015] in D:\Godot Projects\g\Node.cs:16

I'm using Fmod library here which requires the DLLs be present during runtime. In my VS project I have the DLLs set to be copied over to the build path. And also I can't seem to add them as a reference I get this error on VS
Please make sure the file is accessible, and that it is a valid assembly or COM component.

If not in the build path where exactly do I need to place the DLL files for them to be found?

@Kiotenhariyo I was successfully able to reference FMOD as an unmanaged lib and it's all working as intended on Godot 3.1 beta 3 🙌
The DLL/lib files must be placed either in the Godot exe directory or at the root of your project directory.  When exporting they must be present in the same directory as the executable.
https://www.mono-project.com/docs/advanced/pinvoke/#windows-dll-search-path
So glad this issue is resolved! Writing C++ modules will still result in better performance but being able to use existing C# wrappers is incredibly useful.

I guess this can be closed. Feel free to open a new issue if it's desired to have a way to tell the editor to copy these files when exporting the game.

@Kiotenhariyo I was successfully able to reference FMOD as an unmanaged lib and it's all working as intended on Godot 3.1 beta 3 🙌
The DLL/lib files must be placed either in the Godot exe directory or at the root of your project directory. When exporting they must be present in the same directory as the executable. https://www.mono-project.com/docs/advanced/pinvoke/#windows-dll-search-path
So glad this issue is resolved! Writing C++ modules will still result in better performance but being able to use existing C# wrappers is incredibly useful.

@Kiotenhariyo I was successfully able to reference FMOD as an unmanaged lib and it's all working as intended on Godot 3.1 beta 3 🙌
The DLL/lib files must be placed either in the Godot exe directory or at the root of your project directory. When exporting they must be present in the same directory as the executable. https://www.mono-project.com/docs/advanced/pinvoke/#windows-dll-search-path
So glad this issue is resolved! Writing C++ modules will still result in better performance but being able to use existing C# wrappers is incredibly useful.

I'm finding this a pain. Works when exported, but not when you run from the editor.


Blocks cleanup

This gets rid of the custom StaticBlock and FixedListBlock classes which are not needed anymore.

Coverage decreased (-0.2%) to 94.36% when pulling f35d3354ecaf364e2f2187f84168dfab56d46345 on blocks-cleanup into 9fd4b8e411cb8c3cb8e4df183a97edc4452b6b9b on master.


added new notebook for glint correction

@BexDunn I think it might just be that it's a big notebook so Github sometimes doesn't load it properly... it worked OK when I tried this morning:


button

As it seems you haven't edited Data.js which is required to list your button on the site. please add your directory name in src/Data.js according to instructions in Readme to get this PR accepted

Closing this PR as the button is already added to the site as this results in conflicting changes


[CA-1434] VPC pools for environments that don't already have one

Set up VPC pools for envs that don't already have one.
https://broadworkbench.atlassian.net/browse/CA-1434
Related firecloud-develop PR: https://github.com/broadinstitute/firecloud-develop/pull/2589

@yonghaoy  thanks for the feedback!

VPC-SC GCP projects is only used by AoU at this moment. And AoU only use Terra dev and prod. So I did not create vpc-sc pool in other envs.
Is the requirement changing?

VPC-SC gcp projects can be used outside of AoU so this is to help set that up. I'm not 100% sure if we will need projects with network monitoring (and flow logs) outside of service perimeters, in which case maybe it makes sense to create a new pool for non-serviceperimeter projects?
Question for you:
why do perf and prod have "notebooks.googleapis.com"? (The other envs do not)

@yonghaoy thanks for the feedback!

VPC-SC GCP projects is only used by AoU at this moment. And AoU only use Terra dev and prod. So I did not create vpc-sc pool in other envs.
Is the requirement changing?

VPC-SC gcp projects can be used outside of AoU so we need to set up a pool for this scenario. We may also need projects with network monitoring (and flow logs) that will not use service perimeters, in which case maybe it makes sense to create a new pool for non-serviceperimeter projects since the current pool lives in for_vpc_sc_unclaimed folders? (I think part of this answer will come from https://broadworkbench.atlassian.net/browse/CA-1429)
Question for you:
why do perf and prod have "notebooks.googleapis.com" for vpc and non-vpc pools? (The other envs do not have it. Should they? Looks like WSM also has it.)

RE: Notebook API is optional for Rawls but required for WSM(WSM supports creates Cloud API Notebook, but I don't know if Rawsl needs). I may copied/paste from WSM template when setting up Rawls Prod. I think you can leave as it or add that API for other envs in case Rawls need to support Terra-cli(https://github.com/DataBiosphere/terra-cli/tree/main/notebooks). Sorry for the confusion.

@yonghaoy thanks for the feedback!

VPC-SC GCP projects is only used by AoU at this moment. And AoU only use Terra dev and prod. So I did not create vpc-sc pool in other envs.
Is the requirement changing?

VPC-SC gcp projects can be used outside of AoU so we need to set up a pool for this scenario. We may also need projects with network monitoring (and flow logs) that will not use service perimeters, in which case maybe it makes sense to create a new pool for non-serviceperimeter projects since the current pool lives in for_vpc_sc_unclaimed folders? (I think part of this answer will come from https://broadworkbench.atlassian.net/browse/CA-1429)
Question for you:
why do perf and prod have "notebooks.googleapis.com" for vpc and non-vpc pools? (The other envs do not have it. Should they? Looks like WSM also has it.)

RE: Notebook API is optional for Rawls but required for WSM(WSM supports creates Cloud API Notebook, but I don't know if Rawsl needs). I may copied/paste from WSM template when setting up Rawls Prod. I think you can leave as it or add that API for other envs in case Rawls need to support Terra-cli(https://github.com/DataBiosphere/terra-cli/tree/main/notebooks). Sorry for the confusion.

Ideally we wouldn't have any extraneous permissions. I think we should check with @broad-dsp-appsec to see if it's ok for the terra/rawls projects in the prod and perf environments to have the extra api enabled.

@yonghaoy thanks for the feedback!

VPC-SC GCP projects is only used by AoU at this moment. And AoU only use Terra dev and prod. So I did not create vpc-sc pool in other envs.
Is the requirement changing?

VPC-SC gcp projects can be used outside of AoU so we need to set up a pool for this scenario. We may also need projects with network monitoring (and flow logs) that will not use service perimeters, in which case maybe it makes sense to create a new pool for non-serviceperimeter projects since the current pool lives in for_vpc_sc_unclaimed folders? (I think part of this answer will come from https://broadworkbench.atlassian.net/browse/CA-1429)
Question for you:
why do perf and prod have "notebooks.googleapis.com" for vpc and non-vpc pools? (The other envs do not have it. Should they? Looks like WSM also has it.)

RE: Notebook API is optional for Rawls but required for WSM(WSM supports creates Cloud API Notebook, but I don't know if Rawsl needs). I may copied/paste from WSM template when setting up Rawls Prod. I think you can leave as it or add that API for other envs in case Rawls need to support Terra-cli(https://github.com/DataBiosphere/terra-cli/tree/main/notebooks). Sorry for the confusion.

Ideally we wouldn't have any extraneous permissions. I think we should check with @broad-dsp-appsec to see if it's ok for the terra/rawls projects in the prod and perf environments to have the extra api (which is already enabled for WSM projects) enabled.

Or we can create a new pool without the extra permissions in prod and perf if we want

I'll handle the perf and prod changes as part of a separate PR to keep things more contained.

I'm assuming prod and dev already have have VPC pools?

yep!

@yonghaoy github tells me Merging is blocked The base branch restricts merging to authorized users. Learn more about protected branches. Should we have access to merge to master? / could you merge it?

@yonghaoy github tells me Merging is blocked The base branch restricts merging to authorized users. Learn more about protected branches. Should we have access to merge to master? / could you merge it?

Hi @andy7i The repo access was messed up when I setup 2 reviewer policy with GHA auto push. Now it should be fixed. Can you try again?

@yonghaoy works for me now -- thanks!