Re-evaluate usage of environment variables for static values in docker-compose.yml

What happened (or feature request):
We use environment variables instead of templating in docker-compose.yml, and it seems this adds unnecessary complexity in some cases.
IMO we should use templating (actually replacing into the docker-compose.yml) for all values that are static. For example there's no reason I understand that these should be driven by environment variables:
    container_name: ddev-${DDEV_SITENAME}-db
      - "${DDEV_IMPORTDIR}:/db"
      - "${DDEV_DATADIR}:/var/lib/mysql"
      com.ddev.approot: $DDEV_APPROOT
      com.ddev.app-url: $DDEV_URL

They all seem to be static values, known at ddev start time and unchanging over the life of the universe.
We do have a couple of actual dynamic variables that are well suited for environment variables and that might be difficult to handle otherwise:
      - DDEV_UID=$DDEV_UID
      - DDEV_GID=$DDEV_GID

Basically, I propose that all static values be added into the docker-compose via templating, and runtime values be left as environment variables.
What you expected to happen:
Version: Please include the output of ddev version and the project's .ddev/config.yaml.
How to reproduce this:
Anything else do we need to know:
Related source links or issues:

The biggest reason I see to use env vars for these is to keep the docker-compose.yaml portable, minimally between developers on the same project. DDEV_IMPORTDIR is /Users/tannerjfco/.ddev/sitename/import for me, but likely not for others. I find it kind of nice that our compose file can work for pretty much any ddev site, and it keeps from having to update the compose template if values in config.yaml change.
The biggest downside I see is that docker-compose on its own ends up being pretty much useless for a ddev site.

I know we have a working idea that .ddev should be portable, and you're right if it is then this is a super-good reason. I still question that idea. I think it's just a mess of trouble for everybody. I'd think a "ddev config" would be good for everybody's ddev soul.

Adding hibernate for now because I would like to see user feedback and how people are ultimately using it and whether this is an issue that comes up.

Duplicate of #250 - We do need to move to explicit docker-compose.yaml now instead of peppering things with environment variables. The key thing is we've moved on to writing docker-compose.yaml explicitly.


Export: add field value delimiter to string fields

If you export data and reimport it and there are description fields containing commas, those would lead to wrong data imports.
On fields which can contain user defined strings, it would be great to have a double quote field value delimiter, to avoid such problems. This is also covered by the CSV RFC: https://tools.ietf.org/html/rfc4180 (paragraph 2.5 and 2.6)

it would be great to have a double quote field value delimiter, to avoid such problems.

This has already been implemented (#422). Have you found a specific area where it isn't working?

I explicitly found it for "Description" of the Prefixes export. I'll have a look in my other exports tomorrow if there are more occurences.

Oh my mistake, I was only looking at import. Yeah, we can add this to the to_csv() method on models with export capability.

Great, thanks! :smile:


What is SDK manager? I tried every folder and .exe in my AndroidDev/SDK folder

What is SDK manager? I tried every folder and .exe in my AndroidDev/SDK folder but it keeps pulling an error
https://docwiki.embarcadero.com/RADStudio/Sydney/en/SDK_Manager

my emulator works fine in android studio but in vsc I have to use this tool and it's unclear what it wants me to point it to!
Could you please include a default path list and names of files it needs to be pointed to! Android studio only needs the root SDK path and it's fine!
It sees the VMs I made in AS but it cant start them!

ls
 analytics.settings   dirlist.txt   fonts                 platforms   system-images
 build-tools          emulator      licenses             patcher                              skins       tools
 cmdline-tools        extras        mklink-android.bat   platform-tools                       sources

If you have follow the below. It should work correctly.
https://github.com/toroxx/vscode-avdmanager#setup-the-android-sdk
The sdkmanager is one for the tool inside the SDK cmdline-tools bin folder.
It helps the tools to look up which SDK package is installed in your system.
Usually after setup the avdmanager.sdkPath or ANDROID_SDK_ROOT or ANDROID_HOME.
The AVDManager tools will help you find out everything.
Except you have multiple sdk folders in your system.
You can check vscode output to find out which SDK is used (as below image).

my VSC doesnt show that screen with the thumbs up. it shows this

I want to know basically which path goes in each of these because of that.

Can you tell me what I need to do to show that screen these or which of the  2nd pic those paths correspond to? Am i meant to reference bin/sdkmanager.bat directly or just /bin?

oops wrong button

avd home= \SDK
cmd version = \SDK\cmdline-tools\latest\bin
emulator = \SDK\tools\emulator.exe
exe = \SDK\tools\bin\avdmanager.bat
sdk man = \SDK\tools\bin\sdkmanager.bat
sdk path = SDK
this is what i put  in under settings but i get this error (as above)

The config path check will show when your start the AVDManager in VSCode.
if you don't see those messages in the AVDManager output panel.
You can scroll it to the top. It will always show on the top.
Your Android SDK may have some problems. The folders are unusual.
Where did you download it? my guide or other places?
If your settings are correct. The emulator can be downloaded in the SDK tool panel.
Try to download it and start the AVD.

The AVDManager output messages will help solve the problems.
It will be more helpful if you show me more information.

I downloaded the SDK with android studio and moved it to another drive and reinstalled windows a few times. Then i put the .android folder inside it and simlinked it with mklink to the C:/users/username folder
for now i start it from cmd or from android studio but it's annoying having to open extra windows to run it so I downloaded this addon and into VSCodium IDE from the in-IDE store.
VSCodium is like a reskin for VSC that runs in portable mode that may or may not run without VSC installed.

The tool should work on the VSCodium. Just have no idea why you need that tricky.
Are you going to share the same SDK folder with all your tools?

The point is to make the SDK standalone and usable across physical machines and OSs.
Are you able to share the expected settings for each of the paths so I can check if one is wrong? I suspect SDK/tools isn't the correct path to some of the executables for this job.

Here is the easier way:
Preparing the SDK

Assume that everything will place in the folder C:\android-sdk.
Create folder C:\android-sdk and C:\android-sdk\cmdline-tools.

mkdir C:\android-sdk
mkdir C:\android-sdk\cmdline-tools

download the SDK command line tool (e.g. commandlinetools-win-9477386_latest.zip) from below link
https://developer.android.com/studio#command-line-tools-only

open commandlinetools-win-9477386_latest.zip, you will see the cmdline-tools

unzip and rename the folder name from cmdline-tools to latest

move the latest to C:\android-sdk\cmdline-tools.
The SDK folder will be like below

C:\android-sdk
└─cmdline-tools
    └─latest
        ├─lib
        └─bin
           ├─avdmanager.bat
           └─sdkmanager.bat

Set  the environment ANDROID_HOME to C:\android-sdk

Setup the VSCode and VSCode-like tools (E.g. VSCodium)

install AVD Manager

Click the Android icon to open the AVD Manager

You may see below. Click Download Emulator to download the emulator

If you click download. Wait until you see the output showing the download 100%.

restart your VSCode and open AVD manager

Complete

Setup Android Studio

Open the Android Studio, Click More Actions, SDK Manager

On `Android SDK Location‵, Click Edit

Change Android SDK Location‵ to the SDK path (e.g. C:\android-sdk`)

Click Next and complete the setup.

After all, the VSCode and Android Studio will share the same SDK tools.
Hope it helps.

That didn't work out for me either.
https://github.com/toroxx/vscode-avdmanager/assets/82642883/16d02df3-a80f-4746-9ac7-e7e3b0f3c9e4

this image you shared is good, but can you share this image on your system? I have copied all the paths from this image into VSC but it still is not working.

Starting the emulator in AS works, and flutter run -d <emulator name> also works
All I did was redirect AS to the base SDK and localdata directory and it worked fine. Are you sure your addon doesn't have any hardcoded drive letters it is looking for? The SDK and .android directorys aren't on my C:/ drive as I mentioned...
Does this app depend on path variables being set in its own terminal? I could do something like
set ANDROID_HOME = D:/<path>/SDK
but I'm not sure where .android comes in. But I can't see any reason linking it to the D: drive or symlinking it in C: shouldn't work...

You can try the below commands on your VSCode.
Those commands which is the tool used to check the avdmanager and sdkmanager installed correctly.
C:\android-sdk\cmdline-tools\latest\bin\avdmanager.bat list avd

C:\android-sdk\cmdline-tools\latest\bin\sdkmanager.bat --list_installed

You may see like below.

Of course not. If hardcore other OS like Linux may not work
Here is the SDK on D drive

Hello, I found the solution to the problem.
1 - Download Java.
https://jdk.java.net/20/
2 - Set the environment variable.
JAVA_HOME=C:\Users\Julio\AppData\Local\Android\jdk-20.0.2
3 - Add the binaries under System Paths.

I'm not sure if this is the issue, but i found this
java -version
java version "19.0.2" 2023-01-17
Java(TM) SE Runtime Environment (build 19.0.2+7-44)
Java HotSpot(TM) 64-Bit Server VM (build 19.0.2+7-44, mixed mode, sharing)

SDK\tools\bin\avdmanager.bat
Exception in thread "main" java.lang.NoClassDefFoundError: javax/xml/bind/annotation/XmlSchema
...
Caused by: java.lang.ClassNotFoundException: javax.xml.bind.annotation.XmlSchema

here is me googling why this github can't start the AVD using my android virtual device I set up in android studio 20 minutes ago

I had to do this because my sanity demaded to know that it wasn't something wrong with my environment, which has been stable for almost a year now.

It was as we thought. There are 3 or 4 managers in my SDK. Maybe this is because different programs are using it like:
VSC
AS
quasar
flutter
\SDK\cmdline-tools\7.0\bin>avdmanager.bat  list avd
Available Android Virtual Devices:

The following Android Virtual Devices could not be loaded:
    Name: Pixel_6_Pro_API_33
\SDK\.android\avd\Pixel_6_Pro_API_33.avd
   Error: Google pixel_6_pro no longer exists as a device

\SDK\cmdline-tools\latest\bin>avdmanager.bat  list avd
Available Android Virtual Devices:
    Name: Pixel_6_Pro_API_33
  Device: pixel_6_pro (Google)
    Path: Z:\CREATIVITY_PROJECTS\VRNelectronics\AndroidDev\SDK\.android\avd\Pixel_6_Pro_API_33.avd
  Target: Google APIs (Google Inc.)
          Based on: Android 13.0 (Tiramisu) Tag/ABI: google_apis/x86_64
    Skin: pixel_6_pro
  Sdcard: 512M

\SDK\tools\bin>avdmanager.bat  list avd
Exception in thread "main" java.lang.NoClassDefFoundError: javax/xml/bind/annotation/XmlSchema

So for that I will use the second one.

what i dont understand is this popup

which global does it mean?


Error Message when opening http://localhost:8040

I received the following error after creating a new solo install of manifold in portainer. Error only occurs after uploading the first model.

Can you see any error messages in /usr/src/app/log/production.log inside the container?

Ah, looks like a permission problem; does the user Manyfold is running as have access to the folder you've mounted at /libraries?

I've got a feature issue open at #2505 that should flag this up properly so it can be avoided in future. Glad you got it solved!


Hardware requirements

Your work is very interesting！！ I have a few questions I'd like to ask.
How many 40GB A100 GPUs are needed if the batch size is 3 and the sample size is around 12?  What are the hardware requirements for your training? How long is the training time needed?

Thanks for your reply! Since I only have six 40GB A100 GPUs, I have changed sample_n_frames to 6 and batchsize to 2. I would like to ask if these configurations can be reduced if it is for fine-tuning with consisiti2v model. Which parameters can be adjusted to lower the GPU memory requirements?

This solution is excellent! I completely forgot that gradient_accumulation_steps can be used to simulate a larger batch size. Thank you so much!


Vault importing does not work as expected

Describe the bug
We are using the spring cloud config server to load configuration into our spring boot app using git and hashicorp vault.
The configuration for the server is like this
spring:
  profiles:
    active: git,vault
  cloud:
    config:
      server:
        native:
        git:
          uri: https://our-git-uri
          searchPaths:
            - '{application}'
        vault:
          host: our-vault-host
          scheme: https
          port: 443
          authentication: our-authentication
          kv-version: 2

The configurations being loaded contain vault import configuration similar to this
spring:
  config:
    import: >-
       vault://arbitrary/secret/path1
       vault://arbitrary/secret/path2

The Spring cloud config server loads the configuration successfully and then proceeds to import the properties from the vault paths specified in the configuration, which is exactly what we want. I would then expect these properties to be returned to the application that requested the configuration. They are not - hence the bug report.
I have done some investigation and worked out the following:
There seem to be two barriers to these properties being returned in the response to the configuration request.
Problem 1
The PassthruEnvironmentRepository only 'passes through' configuration in MapPropertySources but the vault properties are stored in EnumerablePropertySources. it is really easy to change this behaviour with the following code change
in PassthruEnvironmentRepository::findOne change MapPropertySource to EnumerablePropertySource
			if (!this.standardSources.contains(name) && source instanceof EnumerablePropertySource<?>) {
				result.add(
						new PropertySource(name, getMap((EnumerablePropertySource<?>) source, includeOrigin), source));
			}

then PasstheuEnvironmentRepository::getMap can be simplified and still work the same
	private Map<?, ?> getMap(EnumerablePropertySource<?> source, boolean includeOrigin) {
		Map<Object, Object> map = new LinkedHashMap<>();
		if (includeOrigin && source instanceof OriginLookup) {
			Map<?, ?> input = (Map<?, ?>) source.getSource();
			OriginLookup<String> originLookup = (OriginLookup<String>) source;
			for (Object key : input.keySet()) {
				Origin origin = originLookup.getOrigin(key.toString());
				if (origin == null) {
					map.put(key, source.getProperty(key.toString()));
					continue;
				}
				String originDesc;
				if (origin instanceof TextResourceOrigin) {
					TextResourceOrigin tro = (TextResourceOrigin) origin;
					originDesc = tro.getLocation().toString();
				}
				else {
					originDesc = origin.toString();
				}
				Object value = source.getProperty(key.toString());
				map.put(key, new PropertyValueDescriptor(value, originDesc));
			}
		}
		else {
			for (String key : source.getPropertyNames()) {
				// Spring Boot wraps the property values in an "origin" detector, so we
				// need
				// to extract the string values
				map.put(key, source.getProperty(key));
			}
		}
		return map;
	}

As MapPropertySource extends EnumerablePropertySource this is backwards compatible and does not change the normal functionality, but now the properties returned from the arbitrary vault path import are available in the property source map.
Problem 2
The next thing that stops the properties from the arbitrary vault paths being returned is the NativeEnvironmentRepository. Even though we do not have the native profile enabled the NativeEnvironmentRepository is used to filter out what is returned. the NativeEnvironmentRepository::matchesLocation method does not return true for the arbitrary vault paths and I cannot fathom a clean way to make this actually happen. This is where I got stuck.
I am very happy with the way that the cloud config server chooses which configuration files to load but it we have a requirement that each active profile may load up different properties from arbitrary vault paths that have nothing to do with the application or profile of the configuration file that they are specifies in. We do not even control those paths on the app side.
It is very frustrating that it so nearly works exactly as we want, but that the properties get filtered out at the end because they are not linked to the active profile in the same way.
We would like to use the cloud-config-server as a one-stop-shop for loading configuration but this is stopping us.
Can this be fixed?

Can you please do me a favor and create 3 separate issues so we can cleanly address each one of these?

Hi Ryan,
I am only experiencing one issue - the list of problems was showing you what I had investigated - I don't think they make sense as individual issues.
Conrad

if you specify spring.config.import statements in configuration from the config server its going to be too late for Boot to take them into consideration I believe.  Boot will already have constructed the list of locations to look for configuration data at that point

Thats not the behaviour we observe. The spring config server loads the configuration for the application and profile and then it performs an import for all of the imports specified in those configuration files. It is exactly the behaviour we want. It picks up the configuration from the files checked out of git and imports the properties from the vault paths. Unfortunately it then does not return the properties loaded from the vault paths.

Can you provide a complete, minimal, verifiable sample that reproduces the problem? It should be available as a GitHub (or similar) project or attached to this issue as a zip file.

In order to provide a complete minimal verifiable sample you would need a spring config server, a hashicorp vault installation and a spring boot application. Not sure how to provide all of that. I'll see what I can do


Make issue - Install on OS X

Hi,
I am a new machine learning learner, and trying to install LightGBM in RStudio, I got the following error (highlighted in bold below).
I don't really understand, but it seems there is an error with Apple Clang and Make. I have Xcode up to date as well. I have updated cmake to the 3.9.0 version via Homebrew, but I dont understand what is wrong.
Can you please help me ?
Thanks in advance!
Nicolas
------------------------------------------------------------------------------------ install_github("Microsoft/LightGBM", subdir = "R-package")
Downloading GitHub repo Microsoft/LightGBM@master
from URL https://api.github.com/repos/Microsoft/LightGBM/zipball/master
Installing lightgbm
'/Library/Frameworks/R.framework/Resources/bin/R' --no-site-file --no-environ --no-save --no-restore --quiet CMD INSTALL
'/private/var/folders/1s/24913f111ml7kdpt08ppxrqh0000gn/T/Rtmpai7ABg/devtools7bf221cbbf06/Microsoft-LightGBM-ac975e7/R-package'
--library='/Library/Frameworks/R.framework/Versions/3.4/Resources/library' --install-tests

installing source package ‘lightgbm’ ...
** libs
installing via 'install.libs.R' to /Library/Frameworks/R.framework/Versions/3.4/Resources/library/lightgbm
-- The C compiler identification is AppleClang <IP_ADDRESS>20042
-- The CXX compiler identification is AppleClang <IP_ADDRESS>20042
-- Check for working C compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc
-- Check for working C compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/c++
-- Check for working CXX compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
_CMake Error at CMakeLists.txt:18 (message):
AppleClang wasn't supported.  Please see
https://github.com/Microsoft/LightGBM/wiki/Installation-Guide#osx_

-- Configuring incomplete, errors occurred!
See also "/private/var/folders/1s/24913f111ml7kdpt08ppxrqh0000gn/T/Rtmpai7ABg/devtools7bf221cbbf06/Microsoft-LightGBM-ac975e7/R-package/src/build/CMakeFiles/CMakeOutput.log".
make: *** No rule to make target `_lightgbm'.  Stop.
Error in eval(ei, envir) : Cannot find lib_lightgbm.so

removing ‘/Library/Frameworks/R.framework/Versions/3.4/Resources/library/lightgbm’
Installation failed: Command failed (1)
Warning message:
GitHub repo contains submodules, may not function as expected!

as said in the document: https://github.com/Microsoft/LightGBM/tree/master/R-package
you need to install gcc first  : https://github.com/Microsoft/LightGBM/wiki/Installation-Guide#osx
and run export CXX=g++-7 CC=gcc-7 before installation


Nothing happens following power+S2 step

I wish I could give a more precise title, but I simply don't have the information with which to do so.
The BeagleBone Black is connected via a USB cable and adapter that I've confirmed support data transfer. The board works fine (boots to eMMC when powered on without S2 being pressed). I open the app, click the USB Mass Storage button, enter my password, hold down the S2 button on the board while powering it on, and… nothing happens. The app just continues to say to press S2, and only the power light on the board lights up. Clearly the app is failing to boot the board, but as far as I can tell there's no diagnostic output to help figure out where exactly the process is breaking down.
BeagleBoot 0.9.0 on macOS 10.14.5

Hey @tjnycum Can you confirm if the app asks for the password after clicking the USB Mass Storage button?

And, after the app prompts for connecting the board holding the S2 switch, then only you have to connect the board.
Following this, if the app is still stuck there, then press the reset or S1 switch on the board.
If these things don't work, can you please give this https://github.com/ravikp7/node-beagle-boot a try, clone it, npm install and npm start

Thanks


Add homepage to package.json

Before submitting the PR, please make sure you do the following

[ ] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[ ] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[ ] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[ ] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpx changeset and following the prompts. All changesets should be patch until SvelteKit 1.0

Any changes to this file should be indented with tabs like the rest of the file, not with spaces.
I'm not sure how valuable of a change this is, but I would argue that, if we did add a homepage, it should be https://kit.svelte.dev/ - There is a separate package.json field for the repository URL.

Any changes to this file should be indented with tabs like the rest of the file, not with spaces.
I'm not sure how valuable of a change this is, but I would argue that, if we did add a homepage, it should be https://kit.svelte.dev/ - There is a separate package.json field for the repository URL.

Done.

Where does this information appear? We have this info in the root package, and if we wanted to put it in sub-packages, we'd want to put it into all of them rather than just one.

This will appear on npm, there will be links to the homepage/github repo then, which I agree should be done for all packages. To me it's always a little strange to see a package on npm without any link to github, which makes me think it's closed source or something like that.

this change seems like a good idea to me, but agree that it should be done for everything under packages/

this change seems like a good idea to me, but agree that it should be done for everything under packages/

Done!

I was starting to suggest that the homepage for the adapters should be their directory in GitHub since that's where the docs for them are, but I think I changed my mind on that because the README already gets published on npm, so there's no value in clicking through to GitHub to see it
Lgtm!

Is this supposed to be visible on npm? I don't see it<EMAIL_ADDRESS>

It should be, but maybe it's because npm only parses the links based on the latest tag, not the next tag.


Sending this link will crash the app

https://www.bloggeratlarge.com/new-orleans-5-fab-boutique-hotels-in-the-french-quarter/

Issue created by BlueBubbles Bot in the BlueBubbles commnity on behalf of @zach in #android-bugs.

I think this was fixed in a newer build


ArrayIndexOutOfBoundsException

SpongeVanilla version: 1.12.2-7.1.6
Java version: openjdk 1.8.0_212
Operating System: Ubuntu 18.04.2 LTS
Plugins: ViaVersion 2.1.2

When the server receives pick item packet (https://wiki.vg/Protocol#Pick_Item) with slot smaller than 0 or bigger than 35 the ArrayIndexOutOfBoundsException appears in the console. If you send this packet many times the server could crash.

Could you include a log of the exception? I'm sure @Faithcaio knows some of this already, but I Want to make sure I can understand a bit as well.
What's expected and what's happening?

https://pastebin.com/PMNHP4hn

Thats actually vanilla code throwing the AIOOBE.


Empty inventory - not-enough-keys-error.

Spark link
No response
Server Version
1.20.4
Server Platform
Paper
Describe the bug
Potentially related to:
https://github.com/Crazy-Crew/CrazyCrates/issues/664

Having empty inventory slots before your key results in a not-enough-keys error
Have crate key in hotbar slot 9, every hotbar slot except hotbar slot 5 filled with items (not crate keys) (hotbar slot 5 empty)
Run /cc open  - should result in not-enough-keys error.
You can use this setup on any row of the inventory for the same result.
This will happen with any crate key.

Error that appears on Console when this bug is encountered
! [Mon 11:13:38 WARN  Minecraft] An error has occurred while trying to take a key from a player.
! [Mon 11:13:38 WARN  Minecraft] Player: Cobriana
! [Mon 11:13:38 WARN  Minecraft] Key: StarMachine
! [Mon 11:13:38 WARN  Minecraft] An error has occurred while trying to take a key from a player.
! [Mon 11:13:38 WARN  Minecraft] Player: Cobriana
! [Mon 11:13:38 WARN  Minecraft] Key: StarMachine

https://i.imgur.com/w6V6Lc5.png
Crate Type
CSGO
Steps to reproduce
See above.
Additional context
Using https://github.com/Crazy-Crew/CrazyCrates/commit/31d6a1b256772d1f49eeb23a9481763d7f0a66e2
Paper Build 462
Config.yml: https://pastebin.com/jC7FRGAs
Agreements

[X] I have searched for and ensured there isn't already an open issue regarding this.
[X] I ticked all the boxes without actually reading them
[X] My version of Minecraft is supported by CrazyCrew.

Looks to be fixed.


⚠ Bofrost.be has degraded performance

In 22d34fe, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1380 ms

Resolved: Bofrost.be performance has improved in 65b7018.


Fix incorrect Win32 error checking & minor refactors

Problem: In the function that adjusts the process privileges there is a mistake in which GetLastError is called after CloseHandle, though clearly the last error check is meant for the privileges related call before hand and the current state appears like a mistake.
Solution: Fix the above problem, and also various other fixes:

Remove the static variable done since the PlatformId is only called during initialization
Fix incorrect parameter passed to the Win32 API privileges function

Ideally, I would also call an error handling function in case there is a failure, but it seems this is not customary in the early initialization code.

thanks


Add documentation for all annotations

Follow-up to https://github.com/godotengine/godot/pull/62713, actually documents every annotation (except @rpc, I need an example for that one).
Also makes annotations cross-linkable in the docs, and fixes an improperly registered @export_placeholder annotation (was missing a parameter).
Note that EditorSettings.interface/inspector/default_float_step does not exist in EditorSettings.xml, so I had to remove a cross-link in favor of a code tag.

PS. Not sure how to better organize the GDScript exports article to reduce duplication of the same, but it definitely needs a tune up and a partial rewrite, I think. Some things are outdated in GDScript basics as well. And the new annotations are missing from gdscript.py in the docs repo. I'm leaving these notes in case someone wants to do this, but I'm fully ready to tackle it myself later.

FYI I started a suggestion for the @rpc annotation, but while researching found a lot of places that should link there (and should be linked) and need to be updated, i.e. the docs for Node.rpc() and some on GlobalScope and also MultiplayerAPI.
I'd suggest leaving the @rpc one for now and I will do a followup PR once we merged this, touching up all of those places at once.

I'd suggest leaving the @rpc one for now and I will do a followup PR once we merged this, touching up all of those places at once.

That works for me 🙂

Thank you, good work :)

Is anyone planning to/working on/has made a PR for the @rpc annotations? I am not necessarily saying I'll do that, but I'd at least be willing to contribute to a discussion about them

@nathanfranke Yes, Max is going to work on it. See a comment a couple above yours.


Processing a whole document with QuickUMLS

What is the recommended method for processing a whole document with QuickUMLS?
I am currently reading in the whole document and passing to QuickUMLS? This is not very nice though. Is it best to process line by line?  Should you preprocess with sentence segmentation and then send to QuickUMLS (I thought it does sentence segmentation first anyway)?
I was going to implement a document processing script and create pull request to add to the repo.
Thanks.

I've only used it against whole documents. What is the issue with it?
Feel free to make a PR for a document processing script :)


Cover more test/widgets tests with leak tracking #4

Description
This PR changes testWidgets by testWidgetsWithLeakTracking in number of flutter/test/widgets tests according to the https://github.com/dart-lang/leak_tracker/issues/134.
Marks leaks

https://github.com/flutter/flutter/issues/134661

Tests

Updates implicit_animations_test.dart to use testWidgetsWithLeakTracking;
Updates implicit_semantics_test.dart to use testWidgetsWithLeakTracking;
Updates inherited_dependencies_test.dart to use testWidgetsWithLeakTracking;
Updates inherited_model_test.dart to use testWidgetsWithLeakTracking;
Updates inherited_test.dart to use testWidgetsWithLeakTracking;
Updates inherited_theme_test.dart to use testWidgetsWithLeakTracking;
Updates init_state_test.dart to use testWidgetsWithLeakTracking;
Updates interactive_viewer_test.dart to use testWidgetsWithLeakTracking;
Updates intrinsic_width_test.dart to use testWidgetsWithLeakTracking;
Updates invert_colors_test.dart to use testWidgetsWithLeakTracking;
Updates keep_alive_test.dart to use testWidgetsWithLeakTracking;
Updates key_test.dart to use testWidgetsWithLeakTracking;
Updates keyboard_listener_test.dart to use testWidgetsWithLeakTracking;
Updates layout_builder_and_global_keys_test.dart to use testWidgetsWithLeakTracking;
Updates layout_builder_and_parent_data_test.dart to use testWidgetsWithLeakTracking;
Updates layout_builder_and_state_test.dart to use testWidgetsWithLeakTracking;
Updates layout_builder_mutations_test.dart to use testWidgetsWithLeakTracking;
Updates layout_builder_test.dart to use testWidgetsWithLeakTracking;
Updates linked_scroll_view_test.dart to use testWidgetsWithLeakTracking;
Updates list_body_test.dart to use testWidgetsWithLeakTracking;
Updates list_view_builder_test.dart to use testWidgetsWithLeakTracking;
Updates list_view_correction_test.dart to use testWidgetsWithLeakTracking;
Updates list_view_fling_test.dart to use testWidgetsWithLeakTracking;
Updates list_view_horizontal_test.dart to use testWidgetsWithLeakTracking;
Updates list_view_misc_test.dart to use testWidgetsWithLeakTracking;
Updates list_view_relayout_test.dart to use testWidgetsWithLeakTracking;
Updates list_view_semantics_test.dart to use testWidgetsWithLeakTracking;
Updates list_view_test.dart to use testWidgetsWithLeakTracking;
Updates list_view_vertical_test.dart to use testWidgetsWithLeakTracking;
Updates list_view_viewporting_test.dart to use testWidgetsWithLeakTracking;
Updates list_view_with_inherited_test.dart to use testWidgetsWithLeakTracking.

Pre-launch Checklist

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

cc @polina-c

@polina-c thanks for the review!


Admin capabilities

Users can no longer make an account on the site unless they have been invited
Users can update their first name and last name
Users can update their email address
Removed unnecessary logic and templates

(Out of the box) Admins can invite other admins to join the site. Confirmation email will be sent, and invited admin can set their new password.
r @armant

r @dzhang55

@weronica please see comments.

@armant @dzhang55 I addressed all of your comments


Ensure node still exists when promise resolves

Hi, thanks for this and github-buttons!
I added a simple check for an edge case where the node might not exist by the time the import promise resolves. In my case, the buttons are mounted once a route is signed into, but then there is a potential immediate redirect, in which case React has mounted and unmounted the buttons before the import resolves, by which time this._.current is null.

Thanks! Just released 1.2.1.


Add migration to rebuild cached nomenclature date

Once #2686 is resolved we should include a migration to update the field so we can properly depend on the values there.

#2686 is resolved.  Wondering if this should be a task, or a migration.

it should be fixed with soft_validation fixes. We need a task to run soft_validations and fixes on a project.

it should be fixed with soft_validation fixes. We need a task to run soft_validations and fixes on a project.

Yes, but also in theory we should never need a soft validation fix in the long run.  This proposal here is to get everything on the same page in the most efficient way possible.

If you add a migration for this please make sure it won't take too long to execute otherwise will cause deploy to fail.

I think I'll just make a rake task for now.

If you do it, do not forget also set cached_is_valid.

cached_is_valid is already correct via migration

There is number of issues getting accumulated which could be resolved with soft validation dashboard #2546


Question about post-targets and ngx-deploy-npm:deploy

Hey there,
Firstly thanks so much, this library is awesome!
My question is do the post-targets all run in parallel or in series? I took a look at the code and i'm not familiar with rxjs but I suspect it's in parallel?
The reason I ask is that my GitHub Action for my NX monorepo workflow is as follows:

Version all affected local npm packages using your library. Sometimes there is no need for a version bump
Push the commit + tags back to the origin (commit messages have [skip ci] so doesn't fire off a new GitHub Action).
Build the package.json dependencies for the local npm libraries so that they have the latest versions of cross dependencies in them
publish all affected local libraries to npm. ⬅ This step is failing because it's trying to push a version that already exists as there is no change to some of the libs.

I wanted to look at using the post-targets as described in this issue to kick off my build-package-json target and then the publish target, but if they happen in parallel It might build the package before dependencies have been injected.
Right now I have just added continue-on-error: true to my publish action, but it's not great since if it fails to publish for a legitimate reason the build will still succeed.
Also, just to confirm, both my version and publish targets have configurations within them, I can't see a way to specify which configuration the post-target should use, but from looking at the code I think it will inherit whichever configuration the version target was using, is that correct?
Would love some clarification/advice here, thanks!

Hi @eXigentCoder,

My question is do the post-targets all run in parallel or in series? I took a look at the code and i'm not familiar with rxjs but I suspect it's in parallel?

By design post-targets run in series to avoid race-conditions.

I wanted to look at using the post-targets as https://github.com/bikecoders/ngx-deploy-npm/issues/70 to kick off my build-package-json target and then the publish target, but if they happen in parallel It might build the package before dependencies have been injected.

Yes, it's exactly why we introduced post-targets, they only run on versioned projects so I think it's what you need in your case.

I can't see a way to specify which configuration the post-target should use, but from looking at the code I think it will inherit whichever configuration the version target was using, is that correct?

You can pass a configuration to your post-target like this: "postTargets": ["my-proj:target:config"].
Hope it helps.

Awesome stuff, thanks so much and keep up the great work, appreciate it! ❤


:triangular_flag_on_post:(lld): remove objkt ff

✅ Checklist

[x] npx changeset was attached.
[ ] Covered by automatic tests.
[x] Impact of the changes:

common
tezos account

📝 Description
Remove objkt ff since we don't support this webview anymore
There is no use of objkt in LLM so no need to change in LLM

Before
After

❓ Context

JIRA or GitHub link: LIVE-13894

🧐 Checklist for the PR Reviewers

The code aligns with the requirements described in the linked JIRA or GitHub issue.
The PR description clearly documents the changes made and explains any technical trade-offs or design decisions.
There are no undocumented trade-offs, technical debt, or maintainability issues.
The PR has been tested thoroughly, and any potential edge cases have been considered and handled.
Any new dependencies have been justified and documented.
Performance considerations have been taken into account. (changes have been profiled or benchmarked if necessary)

/screenshots

/generate-screenshots


Add .env file to react projecttemplates in order to run the CRA serve…

Added .env files to the react project templates in order to run the CRA server independently.
Fixes #7286

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: dannyBies sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@ryanbrandenburg Would you be able to verify this works correctly?

@dannyBies this never replicated for me. Oughtn't this be fixed by https://github.com/aspnet/AspNetCore/blob/7269dbb73f4aa0ebf5ebb8bc07a076ee181851be/src/Middleware/SpaServices.Extensions/src/ReactDevelopmentServer/ReactDevelopmentServerMiddleware.cs#L72? If the answer is that this is "more correct" I can get behind that, but I'm still wondering how you're seeing this and I'm not.

@ryanbrandenburg
The code in ReactDevelopmentServerMiddleware.cs only gets executed when I start the ASP.NET Core app in development mode.
I followed this guide in which I run the CRA server independently from the Core app. Because I'm just running npm start the ReactDevelopmentServerMiddleware does not get executed and the environment variables are not added.

Ah, that explains it. This should be fine then.

Sorry guys, I messed up somehow :).  I've created a new PR (#13525) that only contains the relevant changes.


Grey profile image border when you can't edit photo

Fixes https://openedx.atlassian.net/browse/MA-1760

@BenjiLee Please review?!

:+1:

Thanks for review @miankhalid


Request: Tags mode display

Hello
I have some big select (over 200 items) and want to make multiselect, but there is one problem - unselect something is really big problem, you should find it again and click.
At first i write a solution that works on "change" event of select and draw labels down to select button, but then i realised that it would be a cool solution for this plugin and rewrite it to plugin source.
It looks like this:
You can click "tag" item to remove it.
Should i add customize options for this "tags" and create pull request? Or this is not realy required feature?

Duplicate of #565.


Proposal: expose /api/v1/write enpoint for remote_write storage API

Proposal
Sometimes is is useful to have a single big Prometheus instance with all the data from other Prometheus instances located in different networks / datacenters.
Solution
The most straightforward solution for this use case is to expose /api/v1/write endpoint for remote_write storage API on the big Prometheus instance, so other instances could write to it using remote storage protocol.

Have you tried Prometheus federation mechanism for this purpose yet?

I'm not sure if anybody goes to allow direct writing to Prometheus TSDB apart from Prometheus server on the same machine itself and it seems as quite a bad idea to me too.
What might work as a workaround - set up all the Prometheus servers writing via remote write to a local TSDBs (InfluxDB, CrateDB, whatever) and then set up writing data from that "small" TSDB instances to a "single big" one. So, "single big" Prometheus will have access to all this data in the "single big" TSDB via remote read.

There is another use case for the exposed remote write API - seamless integration of push model via external adapter service that can collect metrics in various popular formats such as InfluxDB's line protocol, various Graphite formats and formats based on message queues.

Hi, this is basically a request for adding push support to Prometheus, which has come up a lot in the past, and which we've been very careful about so far.
First, a bit of background why push support hasn't been added yet to Prometheus so far: Prometheus is primarily a pull-based monitoring system (which just happens to include a TSDB, but we don't see Prometheus as a TSDB primarily), several of its core features assume that the server is in control of scraping metrics and attaching timestamps at its own configured pace. For example, recording and alerting rules are evaluated based on the server's notion of the current time. So the underlying metrics must arrive in lockstep with rule evaluations, which is best ensured with a pull model. Service discovery integration (with target metadata attaching etc.) and automatic target health monitoring are other features that rely on the pull model. So if we added a push endpoint and advertised it loudly, we would be worried that many users who don't know 100% what they are doing would shoot themselves in the foot, thinking that Prometheus supports "push" now.
However, I started a rough discussion doc a long while back about pros/cons of adding push support to Prometheus: https://docs.google.com/document/d/1H47v7WfyKkSLMrR8_iku6u9VB73WrVzBHb2SB6dL9_g/edit#heading=h.2v27snv0lsur
The consensus at the last dev summit was to add an item to the Prometheus roadmap to only allow backfilling of entire time series (not appending individual samples like via remote write): https://prometheus.io/docs/introduction/roadmap/#backfill-time-series
And yeah, in the originally described case, federation is the usually adopted architecture, which does require making individual Prometheus servers reachable from the central one.

And yeah, in the originally described case, federation is the usually adopted architecture, which does require making individual Prometheus servers reachable from the central one.

I need to qualify that comment: federation is not recommended for transferring all data from one Prometheus server to another, as neither the source TSDB is optimized for that, not is it resource-efficient to do single humungous scrapes. It's more meant to transfer over a select set of aggregated series in the thousands, not in the millions. So unless the data that needs to be federated into the global Prometheus server is small-ish, there is indeed no great solution for this at the moment with Prometheus. Mind you, remote write is super inefficient resource-wise too, though.
Maybe you are rather looking for something like Thanos, which gives you a global view, long-term storage, while being efficient at the transfer of large amounts of data too (much more so than remote write or federation).

To sum up, if you use Prometheus to push its metrics via remote write to another Prometheus (not possible, but requested in this issue) or some LTS storage, this is still pull model, because what matters is how data were collected, right? (:
So if I understand this right, the main blocker for adding this one is to avoid abuse and pushing arbitrary samples from other non-pull collectors? (which is fair point)
I'm asking as we consider adding remote write receiver endpoint for Thanos system: https://github.com/improbable-eng/thanos/pull/659 and we need to educate users which one to choose (sidecar +  query for  fresh data) or remote write.

You are effectively treating it as a TSDB mainly then, not a monitoring system first.

Yup, totally agree.

Mind you, remote write is super inefficient resource-wise too, though.

Why? From my experience it works quite well for Prometheus scraping 50K-100K samples/sec. Probably, it must be optimized somehow for Prometheus scraping millions of samples per second?

@valyala On the sending side, remote write totally blows up memory usage (or at least used to, not sure about the current state of optimization) because normally sample ingestion is a very highly memory-tuned process that avoids allocations as much as possible, appends samples into the TSDB using internal TSDB series IDs when possible (rather than storing full label sets for every sample), etc. For remote write, every sample has to be re-encoded, buffered, and then sent over the wire in its fully expanded form, in protobuf format, in near-time (instead of larger compressed batches). In contrast, Thanos just ships completed (and very well-compressed) on-disk blocks, which requires much less memory memory and other resources.
I'm actually glad to hear that remote write still works well for you with 50K-100K samples/sec.

We've looked at this as part of our bug scrub and @juliusv's responses already appear to cover it.

It looks like this issue must be re-opened according to https://twitter.com/TwitchiH/status/1339665862949826563 . See more details at https://docs.google.com/document/d/1iO1QHRyABaIpc6xXB1oqu91jYL1QQibEpvQqXfQF-WA/edit .

Similarly to how we do not demand you take any particular action in your GitHub org, please refrain from demanding any particular action in our GitHub org.
Simply enabling an end point is not design; as per the document you linked, we will be working within a design doc first to come to a holistic solution which follows Prometheus design principles.

#8307 is open ; any positive feedback is welcome.


Make the link that appears in the chat response more prominent.

Steps to reproduce the problem:

Created an assistant with "Website Browsing" tool.
Asked a question to the tool, which resulted in the following response which has a link:

This link text can be me more prominent with may be a different color/font and underlined.

We now have the links in blue and underlined.


Staging for version increment automation

Signed-off-by: pgodithi<EMAIL_ADDRESS>Description
This is the staging PR for to execute workflows that auto raise the version increment PR's
Example: https://github.com/prudhvigodithi/notifications/pull/1
Issues Resolved
Part of: https://github.com/opensearch-project/opensearch-build/issues/1375
From solution: https://github.com/opensearch-project/opensearch-build/issues/1375#issuecomment-1159476686
Related issue: https://github.com/opensearch-project/notifications/issues/469
Check List

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

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

Referred old version increment PR:
https://github.com/opensearch-project/notifications/pull/465

Codecov Report

Merging #476 (53f139e) into main (d34bf56) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main     #476   +/-   ##
=========================================
  Coverage     71.01%   71.01%
  Complexity      112      112
=========================================
  Files           124      124
  Lines          3923     3923
  Branches        626      626
=========================================
  Hits           2786     2786
  Misses          961      961
  Partials        176      176

Flag
Coverage Δ

dashboards-notifications
86.40% <ø> (ø)

opensearch-notifications
61.70% <ø> (ø)

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

Continue to review full report at Codecov.

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

Renaming to task updateVersion following https://github.com/opensearch-project/opensearch-plugin-template-java/pull/32.

Hey @praveensameneni @lezzago can you please take a look.
Thank you

@lezzago @praveensameneni can you please review this PR soon?


skrooge cannot import using woob

Describe the bug
A clear and concise description of what the bug is.
Steps To Reproduce
Steps to reproduce the behavior:

Configure a banking backend for woob
Run skrooge
Click File > Import > Import using engine

Expected behavior
Import succeeds
Screenshots

Additional context
The given command that complains giving error 255 will run without errors if pasted in a terminal
Notify maintainers
@jokogr
Metadata

system: "x86_64-linux"
host os: Linux 5.18.4, NixOS, 22.11 (Raccoon), 22.11pre-git
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.9.1
nixpkgs: /nix/store/hcsl8zxvg8jnncid40xnm3drlvdcqd4c-source

@pasqui23 please try to reproduce with latest stable once it's been updated (#280507)

; woob bank                                                                                # took 4s zsh
Welcome to bank v3.6

Copyright(C) 2010-2024 Romain Bignon, Christophe Benz
This program is free software: you can redistribute it and/or modify
it under the terms of the GNU Lesser General Public License as published by
the Free Software Foundation, either version 3 of the License, or
(at your option) any later version.

Type "help" to display available commands.

Loaded backends: n26

bank> list
                 Account                     Balance    Coming
------------------------------------------+----------+----------
2024-03-23 15:32:31,635:WARNING:urllib3.connectionpool:3.6:connectionpool.py:871:urlopen Retrying (Retry(total=1, connect=None, read=None, redirect=None, status=None)) after connection broken by 'NameResolutionError("<urllib3.connection.HTTPSConnection object at 0x7f8e8ee04050>: Failed to resolve 'api.tech26.global' ([Errno -2] Name or service not known)")': /oauth2/token
2024-03-23 15:32:31,636:WARNING:urllib3.connectionpool:3.6:connectionpool.py:871:urlopen Retrying (Retry(total=0, connect=None, read=None, redirect=None, status=None)) after connection broken by 'NameResolutionError("<urllib3.connection.HTTPSConnection object at 0x7f8e8ee06350>: Failed to resolve 'api.tech26.global' ([Errno -2] Name or service not known)")': /oauth2/token
Bug(n26): HTTPSConnectionPool(host='api.tech26.global', port=443): Max retries exceeded with url: /oauth2/token (Caused by NameResolutionError("<urllib3.connection.HTTPSConnection object at 0x7f8e8ee040d0>: Failed to resolve 'api.tech26.global' ([Errno -2] Name or service not known)"))
(If --auto-update is passed on the command-line, new versions of the module will be checked automatically)
Use "logging debug" option to print backtraces.
------------------------------------------+----------+----------

so I don't think that I can yet control if this has been fixed sorry.


Vulnmeet presentation

This presentation is as open source as it is WIP.

@Mamietti, can this be merged into master?


[FLINK-12411][table-planner][tests] Workaround limited support of not nullable fields in window aggregation

What is the purpose of the change
This does not fix the problem. It just makes the e2e test pass. For a
proper fix of the underlying problem see FLINK-12249.
Brief change log

Produce nullable type in CASE ... WHEN statement. The added NULL value will never be generated. It's sole purpose is to deceive the planner.

Verifying this change
It changes only test_streaming_sql.sh.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@twalthr This is a temporary fix for the failling test_streaming_sql.sh. WDYT? should we add that until we fix it properly in FLINK-12249


Getting domain_names.map is not a function error in HOSTS

Checklist

Have you pulled and found the error with jc21/nginx-proxy-manager:latest docker image?

Yes

Are you sure you're not using someone else's docker image?

Yes

Have you searched for similar issues (both open and closed)?

Yes

Describe the bug
Almost everything is in the title ^^
When I enter the Hosts > Proxy Hosts menu, none of the 20 or so Hosts I have configured appear, and I see the following error :

domain_names.map is not a function
In Chrome's console, the following can be seen :
TypeError: domain_names.map is not a function
    at module.exports (7.bundle.7.js?v=2.10.3:1:6179)
    at i.render (main.bundle.js?v=2.10.3:310:15407)
    at i._renderHtml (main.bundle.js?v=2.10.3:303:68288)
    at i._renderTemplate (main.bundle.js?v=2.10.3:27:115832)
    at i.render (main.bundle.js?v=2.10.3:27:131271)
    at ae (main.bundle.js?v=2.10.3:27:122435)
    at main.bundle.js?v=2.10.3:27:140662
    at Function.Dt (main.bundle.js?v=2.10.3:3:28769)
    at i._getBuffer (main.bundle.js?v=2.10.3:27:140642)
    at i._renderChildren (main.bundle.js?v=2.10.3:27:140463)

Nginx Proxy Manager Version
2.10.3
To Reproduce
Steps to reproduce the behavior:

Go to Hosts > Proxy Hosts
See error

Expected behavior
See the list of Proxy Hosts
Screenshots
Already provided screenshot with the description.
Operating System
Docker on Alpine Linux.
Additional context
I just updated to the latest version, it was working fine before.
But I don't know on which version I was before...

Hi,
Now on 2.10.4 and still seeing the same error.
Any pointers ?

I try to provision my nginxproxymanager via writing to the sqlite DB. I get the same message when I insert the certificate entry. How did you (@pmorange) solve the problem?

I try to provision my nginxproxymanager via writing to the sqlite DB. I get the same message when I insert the certificate entry. How did you (@pmorange) solve the problem?

Hi,
I was able to solve that by checking in debug mode the json I receive on the webpage. You can do that for instance in Brave and Chrome by opening the developer mode.
My problem was that in the name or description (can't remember which one exactly), I had badly closed " . You can check all the names you have and see if any one of them has a missing quote.
Hoe that helps.

@milkpirate or @pmorange After following the database migration from mysql to sqlite #1529 I have the same error. I have looked for badly closed double quotes but have not found any. I did have an issue with the text Let's Encrypt on certificates since the sql escape ' remained after migration. I have now corrected that and changed it from Let's Encrypt to Let's Encrypt in the table audit_log. Can you be more specific on which table and which field has an issue with double quotes?

I guess table/field containing JSON related. JSON is very picky with its quotes and does not accept '. Further the " must be escaped in the field content/value.
But anyway, I migrated to cloudflare tunnels. Its free, there are a lot of tutorials online (e.g. youtube) and it offers a lot (!) of functionality for free (e.g. SSL certs, Oauth for the endpoints aso.). You might need a domain thought but there are really cheap offers for as low as a few bucks / year (if you're not picky with your TLD - have a look here: https://tld-list.com/).
Sure one then is dependent on a cloud provider but still, for me the feature richness is worth it.

I was able to solve it and the explanation is here.
#https://github.com/NginxProxyManager/nginx-proxy-manager/discussions/1529#discussioncomment-8959737


reimbursable Journal Entry

Codecov Report

Merging #268 (93d7e81) into master (2fd5804) will decrease coverage by 0.05%.
The diff coverage is 66.66%.

:exclamation: Current head 93d7e81 differs from pull request most recent head 602a3e8. Consider uploading reports for the commit 602a3e8 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #268      +/-   ##
==========================================
- Coverage   96.30%   96.26%   -0.05%
==========================================
  Files          31       31
  Lines        3793     3799       +6
==========================================
+ Hits         3653     3657       +4
- Misses        140      142       +2

Impacted Files
Coverage Δ

apps/workspaces/models.py
98.73% <ø> (ø)

apps/mappings/utils.py
95.34% <50.00%> (-4.66%)
:arrow_down:

apps/mappings/models.py
100.00% <100.00%> (ø)

Codecov Report

Merging #268 (602a3e8) into master (2fd5804) will decrease coverage by 0.10%.
The diff coverage is 50.00%.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #268      +/-   ##
==========================================
- Coverage   96.30%   96.21%   -0.10%
==========================================
  Files          31       31
  Lines        3793     3801       +8
==========================================
+ Hits         3653     3657       +4
- Misses        140      144       +4

Impacted Files
Coverage Δ

apps/workspaces/models.py
98.73% <ø> (ø)

apps/workspaces/tasks.py
94.17% <0.00%> (-1.87%)
:arrow_down:

apps/mappings/utils.py
95.34% <50.00%> (-4.66%)
:arrow_down:

apps/mappings/models.py
100.00% <100.00%> (ø)

Codecov Report

Merging #268 (602a3e8) into master (2fd5804) will decrease coverage by 0.10%.
The diff coverage is 50.00%.

:exclamation: Current head 602a3e8 differs from pull request most recent head 984e731. Consider uploading reports for the commit 984e731 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #268      +/-   ##
==========================================
- Coverage   96.30%   96.21%   -0.10%
==========================================
  Files          31       31
  Lines        3793     3801       +8
==========================================
+ Hits         3653     3657       +4
- Misses        140      144       +4

Impacted Files
Coverage Δ

apps/workspaces/models.py
98.73% <ø> (ø)

apps/workspaces/tasks.py
94.17% <0.00%> (-1.87%)
:arrow_down:

apps/mappings/utils.py
95.34% <50.00%> (-4.66%)
:arrow_down:

apps/mappings/models.py
100.00% <100.00%> (ø)

Codecov Report

Merging #268 (984e731) into master (2fd5804) will decrease coverage by 0.10%.
The diff coverage is 55.55%.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #268      +/-   ##
==========================================
- Coverage   96.30%   96.21%   -0.10%
==========================================
  Files          31       31
  Lines        3793     3800       +7
==========================================
+ Hits         3653     3656       +3
- Misses        140      144       +4

Impacted Files
Coverage Δ

apps/workspaces/models.py
98.73% <ø> (ø)

apps/workspaces/tasks.py
94.17% <0.00%> (-1.87%)
:arrow_down:

apps/mappings/utils.py
95.34% <50.00%> (-4.66%)
:arrow_down:

apps/mappings/models.py
100.00% <100.00%> (ø)

apps/sage_intacct/models.py
97.43% <100.00%> (-0.01%)
:arrow_down:

Codecov Report

Merging #268 (d465dd6) into master (2fd5804) will decrease coverage by 0.05%.
The diff coverage is 60.00%.

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

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #268      +/-   ##
==========================================
- Coverage   96.30%   96.25%   -0.05%
==========================================
  Files          31       31
  Lines        3793     3796       +3
==========================================
+ Hits         3653     3654       +1
- Misses        140      142       +2

Impacted Files
Coverage Δ

apps/mappings/utils.py
100.00% <ø> (ø)

apps/workspaces/models.py
98.73% <ø> (ø)

apps/workspaces/tasks.py
94.17% <0.00%> (-1.87%)
:arrow_down:

apps/mappings/models.py
100.00% <100.00%> (ø)

apps/sage_intacct/models.py
97.43% <100.00%> (-0.01%)
:arrow_down:


add ssl cert validation, unless OTP version is too old

Verify ssl requests when downloading packages.

the cacerts bundle will probably double the size of the thing but it's not like we have much of a choice :/

Yea :(, it almost doubles it. From 350k to 500k.

+1. I guess we can always make the cacert depth larger on demand if it comes up?


Fix parsing of mailboxes with angle brackets in the display-name.

Previously angle brackets would be considered to be the start of the email address even if they were in a quoted-string. This adds a proper parser for mailbox.
This does still lack support for parsing comments. That would not be too difficult to add.

It looks like https://github.com/lettre/lettre/pull/839 may be a more complete version of this.

I guess this can be closed now


Flaky test: line ending selector ... opens the line ending selector modal

Failed on CircleCI: https://circleci.com/gh/atom/atom/4524
Package tests failed for line-ending-selector:
........F...

line ending selector
  Status bar tile
    clicking the tile
      it opens the line ending selector modal
        timeout: timed out after 60000 msec waiting for something to happen

Finished in 61.95 seconds
12 tests, 59 assertions, 1 failure, 0 skipped

Happened again in https://circleci.com/gh/atom/atom/5050.

Happened again in https://circleci.com/gh/atom/atom/5264

I've :arrow_up:'d line-ending-selector since master had some changes around these specs.


Create a page for showing in-progress infra scanning

Is your feature request related to a specific problem?
We need a config UI page that communicates to the user that a scan is taking place and to wait a bit while it runs
Describe the solution you'd like
A simple title like Infrastructure Scanning In Progress as well as an active loading type animation
Describe alternatives you've considered, if any
A description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Points: 1

@LKCSmith since we aren't adding infra scanning until 1.8, im moving this ticket


Update Check-OfficeProPlusPorts.ps1

Incorrect quote marks around DefaultDisplayPropertySet

Hi @mprentice22153, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;


Duplicate treatment columns to make sure treatment effect is not lost, particularly with binary treatment variables.

Also fixes outcome feat importance scale.
Fixes #70
Fixes #69

Looks good to me!


Update MiniMessage to 4.2.0-SNAPSHOT

Thanks! I wrote the 1.0.0 update before velocity 3.1.0 was released and unfortunately minimessage didn't work with velocity 3.0.1.


WIP: Fix SQL quoting

show is keyword in MySQL and has to be quoted to reference a column name. Use grave accents (ASCII code 96) for quoting to match rest of the source code. It's non-standard SQL, but it's supported by SQLite and MySQL.
Issue #5106

Codecov Report

Merging #5117 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff           @@
##           master   #5117   +/-   ##
======================================
  Coverage    37.4%   37.4%
======================================
  Files         307     307
  Lines       45552   45552
======================================
  Hits        17037   17037
  Misses      26060   26060
  Partials     2455    2455

Impacted Files
Coverage Δ

models/user_openid.go
28.98% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

But PostgreSQL nope, AFAIR.

@er1z I assume it is handled somehow by the libraries because the rest of Gitea code base uses it too.


Feature Request: DXF preview

Description of the new feature / enhancement
The addition of SVG file preview has been extremely useful. Could this be expanded to include the DXF vector file format?
Scenario when this would be used?
Current process requires opening heavy supporting apps to preview the vector, such as Autodesk, Inkscape, Corel Draw, Illustrator, etc. Using the same mechanism as SVG file preview would offer similar benefits.
Supporting information
DXF is an open-source vector file format created and supported by Autodesk.

would be great if you could provide 1 or 2 sample files here.

Sure thing - here's 2 sample files. Note that I wasn't allowed to upload a DXF, so I've made them a .TXT extension. Please change the extension back to DXF to test. Thanks!
DXF-Sample 1.dxf.txt
DXF-Sample 2.dxf.txt

From what I understand, all of the File Explorer add-ons use WebView2.
That means that the previews use a web browser (MS Edge) under the hood, to open the SVG.
Since DXF is not a web standard, I think it is unlikely to come into fruition.

That is not true, we are using it for certain things but not everything. Our markdown previewer is an actual application for instance.

That is not true, we are using it for certain things but not everything. Our markdown previewer is an actual application for instance.

I'm sorry, I didn't know that. Thanks for clarifying. ☺

No worries. Preview pane can use a lot of stuff. Also remember a web view 2 control needs to be hosted by an app :)

Would be very helpful if this could be extended for .dxf preview, Thanks.

It would be great if PowerToys could also support DWG files as well. Typically, to open/preview them, we are required to install large CAD packages. These packages also take a lot of time to load the file on startup. So, if Peek can support both DXF and DWG, it will be a great feature.

Bump this, we need support for DXF files. It's an industry standard CAD format, it ought to be supported.


HER model don't learn again after load previously learned model.

It seems HER model don't learn again after load previously learned model.
model = HER.load('her_bit_env', env=env)
model.learn(1000)
model.save("her_bit_env")

I got exactly the same effect after test.
Stable baselines v.2.6

Hello,
I tried to reproduce the described behavior but couldn't ...
from stable_baselines import HER, DQN, SAC, DDPG
from stable_baselines.her import GoalSelectionStrategy, HERGoalEnvWrapper
from stable_baselines.common.bit_flipping_env import BitFlippingEnv

model_class = DQN  # works also with SAC and DDPG

N_BITS = 20

env = BitFlippingEnv(N_BITS, continuous=model_class in [DDPG, SAC], max_steps=N_BITS)

goal_selection_strategy = 'future' # equivalent to GoalSelectionStrategy.FUTURE

# Wrap the model
model = HER('MlpPolicy', env, model_class, n_sampled_goal=4, goal_selection_strategy=goal_selection_strategy,
                                                verbose=1)
# Train the model
# it reaches a success rate of ~30% (mean reward:-17)
model.learn(20000)

model.save("./her_bit_env")

# WARNING: you must pass an env
# or wrap your environment with HERGoalEnvWrapper to use the predict method
model = HER.load('./her_bit_env', env=env)

# train again
# this time it reaches 100% success rate (mean reward=-9)
model.learn(20000)

PS: Please provide a minimal working example next time (it was not enough to reproduce the behavior)

Hello,
Works fine in my custom gym.
I noticed  learning e.g 5 times * n steps  is not the same as once  5*n  steps.
Second is better.

I noticed learning e.g 5 times * n steps is not the same as once 5*n steps.

Can you elaborate on that? does not seems right.

@araffin In light of your comment in https://github.com/hill-a/stable-baselines/issues/301#issuecomment-489308597, does HER have a perfect recovery or no?

as it is using off-policy algorithms, it does not by default. the most important thing missing would be the replay buffer, that you can save easily by yourself.


about running under Windows

Hello, can your code run under Windows? I have successfully run your code under Ubuntu. But under windows I always get the error “ImportError: cannot import name '_jpegpy' from partially initialized module 'lib.data_preprocess.utils.jpegpy' (most likely due to a circular import)”, this problem has been solved under Ubuntu by generating .so files, I would like to bother to ask why

I am experiencing many issues running this code on windows. The particular problem you mentioned I solved by modifying the /lib/data_preprocess/utils/jpegpy/jpegpy.py file like in the screenshot below.
I am now working on handling a KeyError from the key './train_images\manipulated_sequences\Deepfakes\183_253\frame_0'. I modified the json file to begin with '.\train_images.... as well as still receiving the same key error.

@ez4encef Downgrading python in my environment to 3.10 got me past this issue at least. Now I am having a "No such file or directory" issue with the efficientnet-b3-5fb5a3c3.pth file mentioned in the latest issue posted.

我在 Windows 上运行此代码时遇到了许多问题。您提到的特定问题我通过修改 /lib/data_preprocess/utils/jpegpy/jpegpy.py 文件解决了，如下面的屏幕截图所示。
我现在正在努力处理来自密钥“./train_images\manipulated_sequences\Deepfakes\183_253\frame_0”的KeyError。我修改了配置文件以“.\train_images...”开头。以及仍然收到相同的密钥错误。

Hi, I still get the same error after using your method

I also get the same error

Thanks!

I am experiencing many issues running this code on windows. The particular problem you mentioned I solved by modifying the /lib/data_preprocess/utils/jpegpy/jpegpy.py file like in the screenshot below.
I am now working on handling a KeyError from the key './train_images\manipulated_sequences\Deepfakes\183_253\frame_0'. I modified the config file to begin with '.\train_images.... as well as still receiving the same key error.

Thank you so much! This one works!


How to use msal in VSCode for web extension?

Core Library
MSAL Node (@azure/msal-node)
Core Library Version
1.14.4
Wrapper Library
Not Applicable
Wrapper Library Version
None
Public or Confidential Client?
Public
Description
I'm now trying to migrate a VSCode extension to a VSCode for web extension which needs to get AAD token. And the login feature in VSCode extension runs well.
Cause the VSCode for web extension is run on web browser, so I first choose  msal-browser, it doesn't work.

Then I choose msal-node package, but when I call getAuthCodeUrl, it shows I have untrusted_authority. That's weird because in VSCode extension we use the same API and it worked! Or it has a network problem in commiuncate with VSCode for web extension?
  ERR [Extension Host] [Fri, 02 Dec 2022 08:24:40 GMT]<EMAIL_ADDRESS>: Info - getAuthCodeUrl called
vscode-appc:/Users/longcheng/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:609
[Extension Host] [Fri, 02 Dec 2022 08:24:40 GMT]<EMAIL_ADDRESS>: Info - getAuthCodeUrl called
vscode-appc:/Users/longcheng/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:87
  ERR [Extension Host] [Fri, 02 Dec 2022 08:24:40 GMT]<EMAIL_ADDRESS>: Verbose - initializeRequestScopes called
vscode-appc:/Users/longcheng/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:609
[Extension Host] [Fri, 02 Dec 2022 08:24:40 GMT]<EMAIL_ADDRESS>: Verbose - initializeRequestScopes called
vscode-appc:/Users/longcheng/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:87
  ERR [Extension Host] [Fri, 02 Dec 2022 08:24:40 GMT] : [a903a039-b998-487e-a9fa-e0dcf64df253]<EMAIL_ADDRESS>: Verbose - buildOauthClientConfiguration called
vscode-appc:/Users/longcheng/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:609
[Extension Host] [Fri, 02 Dec 2022 08:24:40 GMT] : [a903a039-b998-487e-a9fa-e0dcf64df253]<EMAIL_ADDRESS>: Verbose - buildOauthClientConfiguration called
vscode-appc:/Users/longcheng/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:87
  ERR [Extension Host] [Fri, 02 Dec 2022 08:24:40 GMT] : [a903a039-b998-487e-a9fa-e0dcf64df253]<EMAIL_ADDRESS>: Verbose - building oauth client configuration with the authority: https://login.windows-ppe.net/common/
vscode-appc:/Users/longcheng/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:609
[Extension Host] [Fri, 02 Dec 2022 08:24:40 GMT] : [a903a039-b998-487e-a9fa-e0dcf64df253]<EMAIL_ADDRESS>: Verbose - building oauth client configuration with the authority: https://login.windows-ppe.net/common/
vscode-appc:/Users/longcheng/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:87
  ERR [Extension Host] [Fri, 02 Dec 2022 08:24:40 GMT] : [a903a039-b998-487e-a9fa-e0dcf64df253]<EMAIL_ADDRESS>: Verbose - createAuthority called
vscode-appc:/Users/longcheng/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:609
[Extension Host] [Fri, 02 Dec 2022 08:24:40 GMT] : [a903a039-b998-487e-a9fa-e0dcf64df253]<EMAIL_ADDRESS>: Verbose - createAuthority called
vscode-appc:/Users/longcheng/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:87
  ERR [Extension Host] [Fri, 02 Dec 2022 08:24:40 GMT]<EMAIL_ADDRESS>: Verbose - Attempting to get cloud discovery metadata in the config
vscode-appc:/Users/longcheng/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:609
[Extension Host] [Fri, 02 Dec 2022 08:24:40 GMT]<EMAIL_ADDRESS>: Verbose - Attempting to get cloud discovery metadata in the config
vscode-appc:/Users/longcheng/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:87
  ERR [Extension Host] [Fri, 02 Dec 2022 08:24:40 GMT]<EMAIL_ADDRESS>: Verbose - Did not find cloud discovery metadata in the config... Attempting to get cloud discovery metadata from the cache.
vscode-appc:/Users/longcheng/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:609
[Extension Host] [Fri, 02 Dec 2022 08:24:40 GMT]<EMAIL_ADDRESS>: Verbose - Did not find cloud discovery metadata in the config... Attempting to get cloud discovery metadata from the cache.
vscode-appc:/Users/longcheng/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:87
  ERR [Extension Host] [Fri, 02 Dec 2022 08:24:40 GMT]<EMAIL_ADDRESS>: Verbose - Did not find cloud discovery metadata in the cache... Attempting to get cloud discovery metadata from the network.
vscode-appc:/Users/longcheng/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:609
[Extension Host] [Fri, 02 Dec 2022 08:24:40 GMT]<EMAIL_ADDRESS>: Verbose - Did not find cloud discovery metadata in the cache... Attempting to get cloud discovery metadata from the network.
vscode-appc:/Users/longcheng/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:87
  ERR [Extension Host] [Fri, 02 Dec 2022 08:24:40 GMT]<EMAIL_ADDRESS>: Verbose - Did not find cloud discovery metadata from the network... Attempting to get cloud discovery metadata from hardcoded values.
vscode-appc:/Users/longcheng/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:609
[Extension Host] [Fri, 02 Dec 2022 08:24:40 GMT]<EMAIL_ADDRESS>: Verbose - Did not find cloud discovery metadata from the network... Attempting to get cloud discovery metadata from hardcoded values.
vscode-appc:/Users/longcheng/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:87
  ERR [Extension Host] [Fri, 02 Dec 2022 08:24:40 GMT]<EMAIL_ADDRESS>: Verbose - Did not find cloud discovery metadata from hardcoded values... Metadata could not be obtained from config, cache, network or hardcoded value. Throwing Untrusted Authority Error.
vscode-appc:/Users/longcheng/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:609
[Extension Host] [Fri, 02 Dec 2022 08:24:40 GMT]<EMAIL_ADDRESS>: Verbose - Did not find cloud discovery metadata from hardcoded values... Metadata could not be obtained from config, cache, network or hardcoded value. Throwing Untrusted Authority Error.
vscode-appc:/Users/longcheng/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:87
[Extension Host] {"errorCode":"endpoints_resolution_error","errorMessage":"Error: could not resolve endpoints. Please check network and try again. Detail: ClientConfigurationError: untrusted_authority: The provided authority is not a trusted authority. Please include this authority in the knownAuthorities config parameter.","subError":"","name":"ClientAuthError"}

FYI, the error log is on above.
Do you have any experience on how to use MSAL in VSCode for web extension? Which package should I use?
MSAL Configuration
const MSAL_CONFIG: Configuration = {
  auth: {
    clientId: '448e8446-7f2d-4c49-927e-e8a6cc9dcac2',
    authority: 'https://login.windows-ppe.net/common/'
  },
  system: {
    loggerOptions: {
      loggerCallback(loglevel: LogLevel, message: string, containsPii: boolean) {
        console.error(message);
      },
      piiLoggingEnabled: false,
      logLevel: LogLevel.Verbose
    }
  }
};

Relevant Code Snippets
const authCodeUrlParameters = {
      scopes: ['https://analysis.windows-int.net/powerbi/api/user_impersonation'],
      redirectUri: `http://localhost:3000`
    };

    this.publicClientApp
      .getAuthCodeUrl(authCodeUrlParameters)
      .then((response) => {
        console.log(response);
      })
      .catch((error) => console.log(JSON.stringify(error)));

Identity Provider
None
Source
Internal (Microsoft)

@realDuang Your msal config seems correct, I will create a work item to document usage of VSCode for Web Extension for msal.
Are you using single tenant or multi-tenant application?

@realDuang whenever the non_browser_environment error comes up, that means MSAL can't access the window browser API. I'm assuming this is somehow caused by the VSCode for web environment acting as a sandbox of some sort.
I think msal-node is probably the best bet since we can probably work around the untrusted authority issues.
Please try adding knownAuthorities to your config as such:
const MSAL_CONFIG: Configuration = {
  auth: {
    clientId: '448e8446-7f2d-4c49-927e-e8a6cc9dcac2',
    authority: 'https://login.windows-ppe.net/common/',
    knownAuthorities: ['https://login.windows-ppe.net/common/]
  },
  system: {
    loggerOptions: {
      loggerCallback(loglevel: LogLevel, message: string, containsPii: boolean) {
        console.error(message);
      },
      piiLoggingEnabled: false,
      logLevel: LogLevel.Verbose
    }
  }
};
Relevant

Let us know if this helps.

@realDuang Your msal config seems correct, I will create a work item to document usage of VSCode for Web Extension for msal. Are you using single tenant or multi-tenant application?

I think it is a single tenant application, because vscode for web seems like not support multi-tenant.

@realDuang whenever the non_browser_environment error comes up, that means MSAL can't access the window browser API. I'm assuming this is somehow caused by the VSCode for web environment acting as a sandbox of some sort.
I think msal-node is probably the best bet since we can probably work around the untrusted authority issues.
Please try adding knownAuthorities to your config as such:
const MSAL_CONFIG: Configuration = {
  auth: {
    clientId: '448e8446-7f2d-4c49-927e-e8a6cc9dcac2',
    authority: 'https://login.windows-ppe.net/common/',
    knownAuthorities: ['https://login.windows-ppe.net/common/]
  },
  system: {
    loggerOptions: {
      loggerCallback(loglevel: LogLevel, message: string, containsPii: boolean) {
        console.error(message);
      },
      piiLoggingEnabled: false,
      logLevel: LogLevel.Verbose
    }
  }
};
Relevant

Let us know if this helps.

Oh thanks for you help! It is success to work around the authorities check. But it still failed to run.
{"errorCode":"endpoints_resolution_error","errorMessage":"Error: could not resolve endpoints. Please check network and try again. Detail: ClientAuthError: openid_config_error: Could not retrieve endpoints. Check your authority and verify the .well-known/openid-configuration endpoint returns the required endpoints. Attempted to retrieve endpoints from: https://login.windows-ppe.net/common/v2.0/.well-known/openid-configuration","subError":"","name":"ClientAuthError"}

I have tried to open https://login.windows-ppe.net/common/v2.0/.well-known/openid-configuration url and it worked from my side. And it's weird that the error log says it is a network problem but I can't find any network request during the code run.

@realDuang I think the problem here is that since VSCode for Web runs on the browser, the Node HTTPS client that msal-node uses is somehow blocked.

Yeah I agree with you. So that's why I open this issue, and hope to find out a best practice in VSCode for Web azure login.

@realDuang I see that VSCode for web extensions have a few limitations. One of them is that all network calls have to be made through Fetch API. MSAL Node's default network client is an HTTP client that uses Node's native HTTPS module (not fetch). However, MSAL Node is designed to accept a custom-made network client that you can pass in through the configuration object.
Here's the default network client for reference.
You can rewrite that client to use fetch and see if it works for you.

@hectormmg Great solution! It works when I change the network client as customized fetch client! Thanks for your advice!

@realDuang the auth code is always visible in the browser URL bar, it's taken into account by the protocol design. There are mechanisms such as PKCE that help make this secure, since the code_verifier and code_challenge need to match for the auth code to be accepted and traded for a token. Even if an attacker extracts the auth code, if PKCE is used it should be relatively safe.

Thanks for your reply! realy helped a lot!
After I fix the network client problem, there seems have a CORS problem in getToken api. The api seems blocked the vscode.dev domain *.vscode-cdn.net.

Shouldn't this be an open api? If not, how could we add this into cors policy? I think the vscode.dev cdn domain can be trusted in azure login.

@realDuang please look at our (docs](https://github.com/AzureAD/microsoft-authentication-library-for-js/blob/916f926d8fd2d7776000b50256b169e27759e00a/lib/msal-browser/docs/errors.md#access-to-fetch-at-url-has-been-blocked-by-cors-policy) for more info on CORS


Implement Span#add_lazy_link

This PR adds Span#add_lazy_link. After some clarification on the specification, a concrete Link class is an implementation option, and the one that I decided to go with as it seems to make the most sense for Ruby.
This fixes: #37.

We talked about this briefly in the SIG meeting, but I wanted to capture that conversation here. This implementation follows the Java implementation pretty closely, which, does not appear to be very lazy. The properties of the Link are lazily accessed, but the Link itself is eagerly initialized.
See:

Link Interface
Span#add_link
SimpleLink (implements Link)

We discussed the option of providing a callable that returns a link as a way to make this more lazy. This would be in line with the LinkFormatter and concrete Link class option. The downside is that a callable will require an object allocation for the callable, plus an additional allocation for the Link, when called. If attributes are created using static data, this would be additional overhead and possible not very desirable. However, if attributes are generated dynamically in way that is potentially expensive, or would result in additional allocations, this could be preferable.
I have a few ideas of how we can handle this:
Option 1 - Not very lazy
Leave things as is (a Link is an already realized instance, whose properties are accessed lazily, by the tracer when needed).
Option 2 - Lazy Link by callable
We could support both use cases by allowing add_lazy_link to accept either a Link instance, or a callable that returns a Link. The SDK would have to handle the conversion at the point in the time the link is accessed, which might lead to some complications.
Option 3 - Lazy attributes on Link
We could possibly get the best of both worlds by making the Link class its self optionally lazy. Assuming that attributes are the the potentially expensive thing to make (and that the span_context is essentially free) we could change Link#initialize to accept a span_context and attributes, or a span_context and a block that returns a frozen hash of attributes, where the block would be executed lazily when attributes are first accessed. This option would give us lazy attributes, and the SDK would only handle Link instances and not have to worry about managing callables in addition, as it would with Option 2.
I think legitimate use cases will help to drive the best decision. Whatever we choose, we should make sure that the approach will adaptable for add_lazy_event, where a lazy event has a name and attributes. If a name is cheap, but attributes are expensive, then the lazy attribute approach would apply there as well.

I went ahead and implemented this using Option 2 - lazy link by callable instead of Option 3 lazy attributes. This pushes the complexity to Span, but makes the implementation for Link trivial as it allows us to skirt concurrency issues around an attribute_formatter being executed more than once. Span might still have to deal with this situation, but could do so using a single lock when evaluating links, rather than a lock per link.


No object reference

Describe your bug here.
Everytime I try charting and then play the chart I've created it always pop-ups this error ;
Crash_sight.txt
I don't have any idea what went wrong in the files that bumped this one out. Any ideas/suggestions on how could I fix this?
Anything related to the source code?
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
Yea, I tried adding a new strumline

It seems that something is wrong with your Noteskin.
Is your chart trying to specify one?
It seems to be related to the code in StrumNote.hx that refers to Pixel Stages.

so it is about the noteskins


Add deprecation notice

If I understand it correctly, the API this was built for is deprecated.
In which case it probably makes sense to add a warning to the Readme.

There is at the top:
"WARNING: The Media API is deprecated and should not be used for new projects. More information."

@czarneckid
my apologies.

No worries. I bumped up the notice in the README to the top so it's a bit more prominent.


[Snyk] Upgrade: react, react-dom

This PR was automatically created by Snyk using the credentials of a real user.![snyk-top-banner](https://github.com/andygongea/OWASP-Benchmark/assets/818805/c518c423-16fe-447e-b67f-ad5a49b5d123)
Snyk has created this PR to upgrade multiple dependencies.
👯 The following dependencies are linked and will therefore be updated together.

:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

Name
Versions
Released on

reactfrom 18.2.0 to 18.3.1 | 385 versions ahead of your current version | 3 months agoon 2024-04-26
react-domfrom 18.2.0 to 18.3.1 | 385 versions ahead of your current version | 3 months agoon 2024-04-26

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:

🧐 View latest project report
📜 Customise PR templates
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

This was not me. This automated tool has now been disabled. Closing.


Small terminology change in docs

Switching "pimp" terminology for "enrich", per https://groups.google.com/forum/#!topic/scala-internals/kVWg7HKldjs. Many prefer the latter because the former has the potential to offend, and at the very least isn't self-explanatory to people unfamiliar with the decade-old TV show "Pimp My Ride". I hope this is uncontroversial!

If it's good enough for Scala internal it's good enough for me. I can merge once the build passes

LGTM 👍 Thank you @acjay, I agree pimps is not a good name. I personally use "extension methods", but enrichment works too.


[Bug]

Summary
A clear and concise description of what the bug is, and what you expected to happen.
How to reproduce

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

Please include screenshots.
Additional context
Add any other context about the problem here.

34324776

Summary
A clear and concise description of what the bug is, and what you expected to happen.
How to reproduce

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

Please include screenshots.
Additional context
Add any other context about the problem here.


How often does the check for new PRs happen and is there a way to make it more frequent?

Concourse checks for new version of resources every minute by default or you check_every can be used.

@jtarchie thanks for the feedback. I tried adding check_every of 5s and 1s and on the face of it doesn't seem to make a different. Would greatly appreciate some additional help.
resources:
- name: sync
  type: pull-request
  check_every: 1s
  source:
    uri: git@github.com:syncaide/sync.git
    repo: syncaide/sync
    base: develop
    access_token: ((access-token))
    private_key: ((private-key))

There is no visual indicator that a check is happening. It just happens in the background.


Add French

Note : Used double quote for 'today': "aujourd'hui"

Sorry, was easier to close and add myself instead of dealing with the merge conflicts.


privatedns release

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-arm-privatedns


Gather cluster related logs for kuttl job

Currently we are not collecting any cluster related logs on kuttl jobs. If a job fails, we have no idea about the reason of failure.
Gathering cluster logs using e2e-collect-logs.yml playbook will help to troubleshoot the issue.
Depends-On: https://github.com/openstack-k8s-operators/ci-framework/pull/451
As a pull request owner and reviewers, I checked that:

[x] Appropriate testing is done and actually running

/lgtm

recheck

recheck

recheck

recheck

recheck

/approve
/lgtm


Version bump of vaadin to 23.2.3

PR to test the build.

The itests failed.
Using explicit system properties: Map()
[1/1] validatePrepareFrontend
1 targets failed
validatePrepareFrontend 5 invalid!
File should exist: mill-vaadin/out/itest/0.10.7/test.dest/skeleton-starter-flow-spring-v23.2/target/flow-frontend
File should exist: mill-vaadin/out/itest/0.10.7/test.dest/skeleton-starter-flow-spring-v23.2/target/flow-frontend/comboBoxConnector.js
File should exist: mill-vaadin/out/itest/0.10.7/test.dest/skeleton-starter-flow-spring-v23.2/target/flow-frontend/index.js
File should exist: mill-vaadin/out/itest/0.10.7/test.dest/skeleton-starter-flow-spring-v23.2/target/flow-frontend/package.json
File should exist: mill-vaadin/out/itest/0.10.7/test.dest/skeleton-starter-flow-spring-v23.2/target/flow-frontend/vaadin-map/synchronization/index.js


Remove liveness probe from worker pod

Worker pod does not expose grpc endpoint and as such should not have liveliness probe
https://github.com/temporalio/temporal/issues/2582

Hi!
Thanks for the report and the fix! 👌
I close the issue as the fix is part of release v0.6.2: https://github.com/alexandrevilain/temporal-operator/releases/tag/v0.6.2
Thanks again,
Alexandre!


Refactor the way settings work

All the per-setting globals are replaced with a single global object,
'config'; its attrs are the weechat settings. They're automatically
updated on config change, and the setting names and defaults are
specified as part of a map in the PluginConfig class definition.
To add a new setting, add the setting name and default to
PluginConfig.settings, and if it's not a boolean, add an appropriate
PluginConfig.get_ method as well.
Signed-off-by: Ben Kelly<EMAIL_ADDRESS>

I like what you did here. Running it myself as a test now.

superseded by #300