I still get "but could not be resolved – treating it as an external dependency"

Hi
I want to use roll-up with gulp, but I always get "but could not be resolved – treating it as an external dependency" and in the IIFE the import is treated as external variable.
I'm using gulp, because I want to apply roll-up to multiple entries.
What I'm doing wrong ?
const rollup = require("gulp-rollup");
const resolve = require("rollup-plugin-node-resolve");

gulp.task("rollup", function() {
  return gulp
    .src("_01_src/js/form/*.js")
    .pipe(
      rollup({
        entry: "_01_src/js/form/controller.js",
        format: "iife",
        plugins: [
          resolve({
            jsnext: true,
            main: true,
            browser: true
          })
        ]
      })
    )
    .pipe(gulp.dest("_02_build/js/"));
});

Rollup 0.42+ now supports bundling multiple files at once:https://github.com/rollup/rollup/pull/1389
Perhaps try using that instead of gulp, so it'll become easier to find the issue.
Also can you provide a repro and the full error you're seeing in the console?

I don't have a repo yet I'm just trying to move to rollup and do testing with a simple settup.
This is what I want to achieve .
I chose rollup because I want to have IIFE and less bloat compared to webpack.
Also gulp because I use for other tasks related to styles, concat code etc. , not related to javascript.

v6 node comes with npm v3.10.10. Also, workaround works perfectly. Can it be closed now?


json of grafana example dashboard

Would be really nice ... it is possible to spent so much time in designing dashboards

The dashboard that is visible in the screenshot does not longer exist in the way you can see it there. My personal Grafana dashboard that makes use of the rspamd-influxdb script is highly integrated into several other data sources like Telegraf's system metrics collector and the mailCow API (which I use for my personal e-mail server).
Thereby I don't see any deeper sense in sharing my dashboard as you won't be able to use it properly without those dependencies. Sorry.


Tensor Flow Installation issue: couldnot find a version that satisfies the requirement tensorflow

I am trying to install Tensorflow through anaconda. Following is the error when I try to install
"couldnot find a version that satisfies the requirement tensorflow".
I tried to create two sessions - with Python 3.5 and Python 3.6 - but both of them returns the error identically.
Code used in the activated sessions: pip install tensorflow
Screenshot1 - Python 3.6 (Environment name: tenflow)

Screenshot2 - Python 3.6 (Environment name: tflow)

Assuming your Python distribution is 64-bit and not 32-bit, could you try then conda install tensorflow rather than using pip?

Thanks carmezium. My python is 32 bit. Now I realise there isn't tensorflow for 32 bit.

Glad you found the culprit. If there isn't any other question this issue can be closed now.


HttpTrigger Local Function Returns 406 in the Browser

From @aluong on May 8, 2018 22:13
I cloned and the example which targets base image: microsoft/azure-functions-python3.6:v2.0.11737-alpha
When I go to

http://localhost:8080/api/HttpTrigger?name=hi

I see an Http Error 406. In the docker logs I see that the HttpTrigger was hit.
info: Function.HttpTrigger[0]
      => System.Collections.Generic.Dictionary`2[System.String,System.Object] => System.Collections.Generic.Dictionary`2[System.String,System.Object]
      Executing 'Functions.HttpTrigger' (Reason='This function was programmatically called via the host APIs.', Id=f7d69d25-233a-41da-a569-95a2bdaa9ca1)

Copied from original issue: Azure/azure-functions-docker-python-sample#15

I'm running into the same issue - 406 on using  microsoft/azure-functions-python3.6:v2.0.11737-alpha.


Valkyrie crash when editing scenario. MoM

So I mistakenly deleted the local version of my scenario "Delve too Deep." I pulled the scenario folder back off my locally saved Github repository, moved it back into the %appdata MOM folder. The scenario shows up in Valkyrie. It seems to play fine. However, when I try to edit it, it instantly crashes.
Bit of a bummer, as I was in the middle of adding a third alternate story. Managed to finish two before the crashing error.
Version 2.1.0

Can you please try with version 2.1.2?
If it does not fix your problem, please share your scenario in this ticket and I will have a look.

i want to translate Stress and Strain scenario and it crush when i want to open it with editor. Editor is crushing in any downloaded scenario

Just to document it here, we fixed @zcaalock problem by extracting the .valkyrie file manually using a zip extractor and found this issue along the way: https://github.com/NPBruce/valkyrie/issues/932

The problem was related to the "copy" of local packaged scenario.
Now fixed.
@GwanGwanish  your problem may be similar if you tried to "copy" your scenario in Valkyrie before editing.

Will give it a whirl when I can. Y'all doing amazing work with Valkyrie.
On Sun, Nov 4, 2018, 6:22 PM BenRQ<EMAIL_ADDRESS>wrote:

Release 2.1.3 is now available with a fix on this issue
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NPBruce/valkyrie/issues/929#issuecomment-435718383,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AoIZtBHAPMV2UO42i96T5GYOrY1MeUTZks5ur3awgaJpZM4X3Aqx
.


PolkaGPT - All Polka-lover

Project Abstract
PolkaGPT is an application tool for the LLM (Large Language Model) model designed for developers, researchers, and users to search for documents, suggest code, hint at bug solutions, and more. Additionally, the Polkadot-loving open-source community can contribute to the data repository so that PolkaGPT can be updated continuously
Grant level

[x] Level 1:  Up to $10,000, 2 approvals
[ ] Level 2:  Up to $30,000, 3 approvals
[ ] Level 3:  Unlimited, 5 approvals (for >$100k: Web3 Foundation Council approval)

Application Checklist

[x] The application template has been copied and aptly renamed (project_name.md).
[x] I have read the application guidelines.
[x] Payment details have been provided (bank details via email or Polkadot (USDC & USDT) address in the application).
[x] I am aware that, in order to receive a grant, I (and the entity I represent) have to successfully complete a KYC/KYB check.
[x] The software delivered for this grant will be released under an open-source license specified in the application.
[x] The initial PR contains only one commit (squash and force-push if needed).
[ ] The grant will only be announced once the first milestone has been accepted (see the announcement guidelines).
[ ] I prefer the discussion of this application to take place in a private Element/Matrix channel. My username is: @_______:matrix.org (change the homeserver if you use a different one)

I have read and hereby sign the Contributor License Agreement.

Hi @CocDap thanks for your time and effort in submitting the application. Due to the reasons above, and also the fact that it isn't really coupled with Polkadot tech in a novel way, the committee has decided to go ahead and close this application. We appreciate your interest in the W3F grants program, and wish you the best of luck in finding funding.

For some personal reasons, I am responding late. Here are two key differences between PolkaGPT and ChatGPT or Bard:

ChatGPT and Bard are trained on general knowledge, while PolkaGPT focuses on Polkadot-related information (a specific knowledge domain). Additionally, PolkaGPT includes links to YouTube and other learned sources. In terms of in-depth knowledge, PolkaGPT is more specialized, as ChatGPT and Bard may not provide detailed answers.

The Contribute To Earn model is implemented for PolkaGPT, where users contribute data that is verified. Contributions from the community and developers are highly valued, encouraging participation in the Polkadot ecosystem.

Currently, a person's credibility is based solely on their verified social account, which does not necessarily indicate whether they contribute to Polkadot. The proposed idea is to introduce on-chain credit to provide a more accurate measure of credibility for Polkadot enthusiasts. This would involve incorporating additional on-chain factors such as the amount you're staking, nominating, and your participation in activities within Parachains, ...

One more thing, the chatbot is the initial idea, while the combination of Contribute to Earn with On-chain Credit will utilize the tech stack of Polkadot
@Noc2 @keeganquigley Thank you for reviewing my initial proposal.


support PMTiles as a data source

Teach Tangram how to natively decipher 'serverless' PMTiles.
https://github.com/protomaps/PMTiles
"PMTiles is a single-file archive format for tiled data. A PMTiles archive can be hosted on a commodity storage platform such as S3, and enables low-cost, zero-maintenance map applications that are "serverless" - free of a custom tile backend or third party provider."
"PMTiles is a binary serialization format designed for two main access patterns: over the network, via HTTP 1.1 Byte Serving (Range: requests), or via memory-mapped files on disk."
via https://protomaps.com/blog/dynamic-maps-static-storage/
1. Fetch the first 512 kilobytes and parse the directory into a lookup table
2. Look up by key of the tile you want, in this case 8_65_95
3. It's a match! You'll have a byte range offset: 785366 length: 21400
4. Fetch those bytes using Range:bytes=785366-806765 and interpret the data

200 line JS decoder library:  https://github.com/protomaps/PMTiles/blob/master/js/index.mjs
Perhaps something like
sources:
    mapzen:
        type: PMTiles
        url: https://mydata.com/map.pmtiles

though this does not give an indication of the tile type, which would likely be pbf or png given the environment.

+1
From what I can tell https://github.com/protomaps/PMTiles needs to be brought in as a dependency, because Tangram does not expose an API to extend sources at runtime, it's a small bit of code but fast moving at the moment.


Minor: Add `RuntimeEnvBuilder::build_arc()

Which issue does this PR close?
N/A
Rationale for this change
Small follow on to https://github.com/apache/datafusion/pull/12157 from @devanbenz
It is still somewhat awkward to make RuntimeEnv's, so let's make it slightly easier
What changes are included in this PR?
Adds a convenience method to RuntimeEnvBuilder to build an Arc<RuntimeEnv>
Are these changes tested?
Yes, by CI

Are there any user-facing changes?
A new method

I like the simplification. Is this an established naming pattern in the ecosystem?

I am not aware of this being an established pattern (I think DataFusion uses Arc's a bit more than many Rust purists would prefer)

I also have the same thought, maybe we don't really need Arc in some cases. We could gradually cleanup them later on

Thanks @alamb and @findepi


added player count to boss info

Yes it matters, a lobby with more players is more likely to last longer and so this information is nice to know.

It's really not an interesting number, you can last very long with very few players.


fail to compile on the newest commit

error[E0432]: unresolved imports `icons::SF_UI_ROUNDED_BYTES`, `icons::SF_UI_ROUNDED`
  --> /home/redhawk/.cargo/git/checkouts/iced_aw-d8617147b960a6c7/89ed86d/src/core.rs:30:91
   |
30 |         pub use icons::{BOOTSTRAP_FONT, BOOTSTRAP_FONT_BYTES, NERD_FONT, NERD_FONT_BYTES, SF_UI_ROUNDED_BYTES, SF_UI_ROUNDED, Bootstrap, Nerd};
   |                                                                                ^^^^^^^^^^^^^^^^^^^  ^^^^^^^^^^^^^ no `SF_UI_ROUNDED` in `core::icons`
   |                                                                                |
   |                                                                                no `SF_UI_ROUNDED_BYTES` in `core::icons`
   |
note: found an item that was configured out
  --> /home/redhawk/.cargo/git/checkouts/iced_aw-d8617147b960a6c7/89ed86d/src/core/icons.rs:36:11
   |
36 | pub const SF_UI_ROUNDED_BYTES: &[u8] = include_bytes!("./fonts/SFUIRounded.ttf");
   |           ^^^^^^^^^^^^^^^^^^^
   = note: the item is gated behind the `cupertino` feature
note: found an item that was configured out
  --> /home/redhawk/.cargo/git/checkouts/iced_aw-d8617147b960a6c7/89ed86d/src/core/icons.rs:40:11
   |
40 | pub const SF_UI_ROUNDED: iced::Font = iced::Font::with_name(".SF UI Rounded");
   |           ^^^^^^^^^^^^^
   = note: the item is gated behind the `cupertino` feature

error[E0432]: unresolved imports `crate::core::icons::SF_UI_ROUNDED_BYTES`, `crate::core::icons::SF_UI_ROUNDED`
  --> /home/redhawk/.cargo/git/checkouts/iced_aw-d8617147b960a6c7/89ed86d/src/lib.rs:64:103
   |
64 |                     Bootstrap, BOOTSTRAP_FONT, BOOTSTRAP_FONT_BYTES, Nerd, NERD_FONT, NERD_FONT_BYTES,SF_UI_ROUNDED_BYTES, SF_UI_ROUNDED,
   |                                                                                ^^^^^^^^^^^^^^^^^^^  ^^^^^^^^^^^^^ no `SF_UI_ROUNDED` in `core::icons`
   |                                                                                |
   |                                                                                no `SF_UI_ROUNDED_BYTES` in `core::icons`
   |
   = help: consider importing this unresolved item through its public re-export instead:
           crate::core::SF_UI_ROUNDED_BYTES
note: found an item that was configured out
  --> /home/redhawk/.cargo/git/checkouts/iced_aw-d8617147b960a6c7/89ed86d/src/core/icons.rs:36:11
   |
36 | pub const SF_UI_ROUNDED_BYTES: &[u8] = include_bytes!("./fonts/SFUIRounded.ttf");
   |           ^^^^^^^^^^^^^^^^^^^
   = note: the item is gated behind the `cupertino` feature
   = help: consider importing this unresolved item through its public re-export instead:
           crate::core::SF_UI_ROUNDED
note: found an item that was configured out
  --> /home/redhawk/.cargo/git/checkouts/iced_aw-d8617147b960a6c7/89ed86d/src/core/icons.rs:40:11
   |
40 | pub const SF_UI_ROUNDED: iced::Font = iced::Font::with_name(".SF UI Rounded");
   |           ^^^^^^^^^^^^^
   = note: the item is gated behind the `cupertino` feature

For more information about this error, try `rustc --explain E0432`.
error: could not compile `iced_aw` (lib) due to 2 previous errors

iced_aw = { version = "0.8", features = ["icons", "menu","tab_bar"], default-features = false }

good catch I forgot to remove the  compiler feature switch for it as i converted them to just be under Icons.


#190 fix parameter when row condition is blank

Description

Fixed parameter when row condition is blank.
Originally row:undefined to not sending row: at all
Related Issue :

#190
How Has This Been Tested?

'SET'룰을 적용할 때 컨디션이 입력되지 않았을 때 row: undefined를 보내고 있지 않은지 확인,
편집 후 적용 시 undefined error 가 나는지 확인
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My code follows the code style of this project. it will be added soon
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document. it will be added soon
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Additional Context

run build
deploy to 3


Fix workgroup calculation

Ensure enough workgroups are dispatched to process all particles,
rounding up to the number of particles to avoid truncating the last few
ones.

FYI @josh65536 this is why setting capacity < 64 doesn't render anything, because it was rounding down to 0 and not dispatching any compute thread.


Failed to register native method ProxyJavaScriptExecutor.initialize ... & no static or non-static method ProxyJavaScriptExecutor.initialize ...

when i run reactnative with multidex(dex-3) in note3 (5.0 OS) in my project ,i saw the error at runtime below:
from the logs ,it seem to aim to the native method
com/facebook/react/bridge/ProxyJavaScriptExecutor
private native void initialize(JavaJSExecutor var1);

Logs begin :

01-29 18:46:09.204 21856-22634/com.a E/art: Failed to register native method com.facebook.react.bridge.ProxyJavaScriptExecutor.initialize(Lcom/facebook/react/bridge/ProxyJavaScriptExecutor$JavaJSExecutor;)V in /data/app/com.a-2/base.apk 01-29 18:46:09.214 21856-22634/a W/ExceptionHandler: Exception occurs in worker thread , but it will not crash.
java.lang.RuntimeException: An error occured while executing doInBackground() at android.os.AsyncTask$3.done(AsyncTask.java:300) at java.util.concurrent.FutureTask.finishCompletion(FutureTask.java:355) at java.util.concurrent.FutureTask.setException(FutureTask.java:222) at java.util.concurrent.FutureTask.run(FutureTask.java:242) at android.os.AsyncTask$SerialExecutor$1.run(AsyncTask.java:231) at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1112) at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:587) at
java.lang.Thread.run(Thread.java:818) Caused by: java.lang.NoSuchMethodError: no static or non-static method "Lcom/facebook/react/bridge/ProxyJavaScriptExecutor;.initialize(Lcom/facebook/react/bridge/ProxyJavaScriptExecutor$JavaJSExecutor;)V" at java.lang.Runtime.nativeLoad(Native Method) at java.lang.Runtime.doLoad(Runtime.java:428) at java.lang.Runtime.load(Runtime.java:329) at java.lang.System.load(System.java:982) at com.facebook.soloader.DirectorySoSource.loadLibrary(Unknown Source) at com.facebook.soloader.SoLoader.loadLibraryBySoName(Unknown Source) at com.facebook.soloader.SoLoader.loadLibrary(Unknown Source) at com.facebook.react.bridge.JSCJavaScriptExecutor.(Unknown Source) at com.facebook.react.bridge.JSCJavaScriptExecutor$Factory.create(Unknown Source) at com.facebook.react.ReactInstanceManagerImpl$ReactContextInitAsyncTask.doInBackground(Unknown Source) at com.facebook.react.ReactInstanceManagerImpl$ReactContextInitAsyncTask.doInBackground(Unknown Source) at android.os.AsyncTask$2.call(AsyncTask.java:288) at java.util.concurrent.FutureTask.run(FutureTask.java:237) at android.os.AsyncTask$SerialExecutor$1.run(AsyncTask.java:231)  at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1112)  at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:587)  at java.lang.Thread.run(Thread.java:818)
01-29 18:46:09.284 21856-21856/com.a W/unknown:React: Unable to dispatch keyboard events in JS as the react instance has not been attached

Logs end

thanks for @mkonicek @lexs @mhorowitz , do you see this error logs before ?

envs
1.use the multidex of android (dex NO.3)
2.mobile is 5.x android OS
3.in the app runtime ,when i use    rootView.startReactApplication();
4.from the log ,it is not also related to the  progurad

is it possible that the .so was not loaded successfully( but i unzip the apk,i found all the .so in the libs folde ) ?
i use maven to build project

I'm not sure we support multidex actually. cc @lexs

@mkonicek @yipengmu
I have the same exception but for a different method when I try to run react-native in an existing Android project that uses multidex.
Failed to register native method com.facebook.react.bridge.ReadableNativeMap.getType(Ljava/lang/String;)Lcom/facebook/react/bridge/ReadableType; in /data/app/<app_package>/base.apk:classes2.dex

@mkonicek The way it was enabled is the one described in the link you provided above.
I am assuming that the error has something to do with the multiple dex files that get created now (2 in our case). The Java class loader can see and access see all dex files, but if you are trying to load a class from native code I think you have to make sure that everything you need is in the first file that the loader can see. I am still trying to figure out how to fix this and what are all the affected classes.

Thanks for the explanation @andreipt. I'm curious if you find out how to use multidex with Gradle, we only use multidex with Buck internally. Would be awesome to add it to the docs too.

@mkonicek My final fix was adding the following two lines in the project's proguard config file:
-keep class com.facebook.react.bridge** { *; }
-keep class com.facebook.react.views** { *; }

The .bridge package exception is needed because the following Interfaces are not annotated with @DoNotStrip:
com.facebook.react.bridge.ReadableArray
com.facebook.react.bridge.ReadableMap
com.facebook.react.bridge.WritableArray
com.facebook.react.bridge.WritableMap

There might be more in cause but a quick fix for me was to add the whole .bridge package as a proguard exception.
Also all classes from com.facebook.react.views were not accessible, so I ended up adding this package too as an exception.

@mkonicek So it seems that the problem was actually caused due to a misconfiguration in the proguard file. Since we are using multidex we are also runing proguard on all builds (even debug).
I was missing:
-keepclassmembers,includedescriptorclasses class * { native <methods>; }

and had to update the correct package for:
-keepclassmembers class *  { @com.facebook.react.uimanager.annotations.ReactProp <methods>; }
-keepclassmembers class *  { @com.facebook.react.uimanager.annotations.ReactPropGroup <methods>; }

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with ProductPains! Check out its new home: https://productpains.com/post/react-native/failed-to-register-native-method-proxyjavascriptexecutorinitialize-no-static-or-non-static-method-proxyjavascriptexecutorinitialize-
ProductPains helps the community prioritize the most important issues thanks to its voting feature.
It is easy to use - just login with GitHub. GitHub issues have voting too, nevertheless
Product Pains has been very useful in highlighting the top bugs and feature requests:
https://productpains.com/product/react-native?tab=top
Also, if this issue is a bug, please consider sending a pull request with a fix.
We're a small team and rely on the community for bug fixes of issues that don't affect fb apps.


Cover customers project with tests.

Tests coverage are 0%, must be at least 98%.
Check the possibility of using Testcontainers.

Tests Containers applied, still need to fixed Sonar Issue. Another Issue must be opened.


'Enable Tap to Place' voice command is not working on the device

I have set all as it was described in the tutorial but last voice command 'Enable Tap to Place' works only in Unity Editor but doesn't on the HoloLens 2. I tried to keep cursor on the Rover, look at it etc. Speech confirmation tooltip doesn't pop up! I checked that eyes recognition was working fine on SeeItSayItLabels. Other voice commands that doesn't require focus are working fine. Maybe it is still an issue with "Is Focus Required" property as it was on MRTK 2.3?
I was using Unity 2019.4.9f and MRTK 2.4.

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

ID: 7e526277-fa7f-9d2a-faed-aed2d7555c93
Version Independent ID: ee349afa-a729-60fd-0570-dfc3458e4cc5
Content: Getting started tutorials - 9. Using speech commands - Mixed Reality
Content Source: mixed-reality-docs/mr-dev-docs/develop/unity/tutorials/mr-learning-base-09.md
Product: mixed-reality
Technology: mr-dev
GitHub Login: @jessemcculloch
Microsoft Alias: jemccull

Hi @dudziakl,
Take a look at these two articles if you haven't already:

https://docs.microsoft.com/en-us/windows/mixed-reality/design/voice-input
https://microsoft.github.io/MixedRealityToolkit-Unity/Documentation/Input/Speech.html

@martinwilter Any ideas?

@hferrone, I'm guessing this is caused by the same bug that caused issue #64. Publishing the fix https://github.com/MicrosoftDocs/mixed-reality/issues/64#issuecomment-701024840 should prevent this from happening to future readers.


Hide unselected Networks

When creating/configuring a Network, this decision should dictate what Network options are available under Config->Networks.  This will give the user only the things they are interested in and help streamline their decision-making.  Without metrics, I can only assume that most people will only be interested in the Ethereum mainnet.
If a user needs to activate/create this network, it should still be possible through the Config->Networks->Create button.  This area may need to have certain preset networks available to make it easier for a User to retroactively add ETC/Kovan/TestRPC.

As mentioned in #41, there will be the notion of default networks.
I think a good compromise here would be to show a 'More...' option, which expands to the rest of the non-default networks, meaning only ETH would be displayed by default.
💸 x 3


Fix duplicate item links

Seems to be a case of pointer reuse internally. Forcing a fresh one seems to fix it. Should fix #5

In theory, this seems like the item pointer being passed into GameAPI::AddChatMessage() should be identical. I can pull the changes and ask Eluminator to test, since he encounters the issue somewhat frequently.
Were you able to reproduce the issue consistently prior to this change?

Yep, I was really lucky/unlucky with this particular bug, and after this change it was gone.

I pulled the changes and asked Eluminator to test, but he is still seeing the same issues afterwards.

Hmm, that's odd. I can, in any case, confirm that the data is correct all the way until GameAPI::CreateItem(itemInfo), and since that result is a void* I can't look further into it. But it's definitely happening inside the CreateItem call, or when it's handed over to AddChatMessage. I speculate that it's some sort of undefined behavior, or the compiler is optimizing it wrong.

Yeah, that matches what I've been seeing as well. It seems like there's some strange behavior happening internally either within the GameAPI::AddChatMesssage function or the GameAPI::CreateItem function that only affects some users. My best guess would be that since we are creating the item immediately before calling the function, the item isn't fully registered in the object manager which causes the lookup to fail and the function links the last known item instead.
I'll see if I can put in some debug statements that dump the address of the item pointer when an item is linked. In theory, if the pointers are different, then that should indicate that the problem lies within GameAPI::AddChatMesssage. In either case however, I suspect that this bug may be very difficult/nearly impossible to fix since the issue seems to be within the in-game functions and not with the launcher code.

Seems like the item addresses are distinct, so the most likely culprit is inside of GameAPI::AddChatMessage.
GDCommunityLauncher.log

I'm thinking maybe it's some sort of caching mechanism, and there's some field in the item replica that's not being set correctly to actually invalidate the cache.

As far as I can tell, the ItemReplicaInfo struct should be complete unless it's one of the unk fields or some additional data afterwards. Could be worth playing around with, though it's still odd that it only breaks for certain people.

The most recent hotfix that I pushed up fixed item linking for some people. This change was just adding some extra unknown fields to the ItemReplicaInfo struct, so the culprit was likely the in-game function reading past the bounds of the ItemReplicaInfo struct and getting invalid/corrupt data.
Eluminator reported that he was still seeing item linking issues after the hotfix, so there might be more to it than that. But for now, I'm closing the ticket since the fix seems to be unrelated.


current state

just a quick bumper...how is it going here? love boom1, but it seems broken on modern python versions?

I am getting back on this this month

pushed 0.1 and working on 0.2. The command works minus a few things


HTTPS Support

see Thog/DramaGenerator#13

If it's Let's Encrypt, you could possibly do the renewal automatically!


[Feedback] Catch EOFError during 'az feedback'

Fixes #594. Note that depending on what threads are running when you press Ctrl+C you may get a stack trace for other reasons.

:shipit:


更新

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (701296d) 42.01% compared to head (83ffc04) 42.01%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #20   +/-   ##
=======================================
  Coverage   42.01%   42.01%
=======================================
  Files          14       14
  Lines        4248     4248
=======================================
  Hits         1785     1785
  Misses       2344     2344
  Partials      119      119

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


chore: register the EVM address correctly in single-node scripts

Overview

Checklist

[ ] New and updated code has appropriate documentation
[ ] New and updated code has new and/or updated testing
[ ] Required CI checks are passing
[ ] Visual proof for any user facing features like CLI or documentation updates
[ ] Linked issues closed with keywords

Codecov Report

Merging #2311 (64be04f) into main (376132a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2311   +/-   ##
=======================================
  Coverage   20.34%   20.34%
=======================================
  Files         136      136
  Lines       15800    15800
=======================================
  Hits         3214     3214
  Misses      12278    12278
  Partials      308      308

I assume there might be some docs that need updating

Yes, I created this https://github.com/celestiaorg/celestia-app/issues/2302


Confirm Identity section does not work!

I have installed a fresh copy of the LBRY client on Windows(the problem is likely to exist on any platform..) and attempted to add my card as a proof of identity.
Steps to reproduce error:

Type card number after reading 'To ensure you are a real person, we require a valid credit or debit card. There is no charge at all, now or in the future. Read More'
Add one of my bank debit cards to the information fields
Click verify
'Card declined.'

Expect: I expect to not have my debit card that I use for purchases on other sites declined when trying to verify my identity on LBRY.
What actually happens: My card is declined.
I am running the latest version of the .exe installer from the LBRY website at the time of posting.

Are you behind proxy or VPN? Which country are you in?  Please send an email to<EMAIL_ADDRESS>with the last 4 digits and the answers to the above questions.

I have given 2 days for a reply email and have yet to receive one..
I attempted to re-verify my account following the steps given above with the same result..
Is there a reason why this is not working?

Just following up, did someone reply to your email? If not, please send another as it may have gotten lost.

I eventually received a reply after my last post.
This can be closed now as the issue was resolved.


Unify VM status icons

I'd find more visually pleasant and self-explanatory to use same set of icons to express VM state as we currently use for actions like start a VM:

▶  running vm (in green)
⏸ suspended vm
■ stopped vm

instead of current circled icons.

For running and Down VMs:

I prefer more the former fa-power-off over this fa-square, what others think? @matobet , @jelkosz

For VM being rebooted:

https://github.com/oVirt/ovirt-web-ui/issues/173#issuecomment-297653875 looks great. The square really doesn't feel self-explanatory enough. Maybe it can be rearranged like for example

start
suspend
stop
reboot
console
edit

to group "player control symbols" together. But fa-power-off is also good.

rearranging makes sense

fa-power-off

So, after recent changes:


Keep balances separate visually #117

@Riverhead:

It should be clear what balance is associated with the contract Available for Trading and what balance remains in the user's wallet. The balance in the user's wallet may be considered extraneous information.

Suggestion: Move the account balance down to the Deposit/Withdraw section as that's the only place it's relevant.

(ported from  Keep balances separate visually #117 )

@jorisbontje:

@cassiopaia something to consider for the new designs.


Model which we need to implement

Models which we need to implement before the full benchmark can be run.
models derived from #705
Works with sentence transformers:

[x] all-MiniLM-*-v2
[x] paraphrase-multilingual-*
[x] all-mpnet-*
[x] LabSE
[x] LASER

Needs custom loader:

[x] multilingual-e5-* (added in #807 and #876 as test cases)
[ ]  multilingual-e5-large-instruct (#888)
[ ] e5-mistral-7b-instruct (as it was the first to use Mistral+instructions for embedding) (#888)
[x] GritLM-7b (@Muennighoff will anyone from the paper like to submit an implementation for these?)
[ ] LLM2Vec supervised and unsupervised (@vaibhavad I imagine your group would like to add these)
[ ] BM25 (see comment below)

APIs

[ ] voyage-large-2-instruct (as it is first on the MTEB leaderboard right now) (#887)
[ ] text-embedding-large-3 (due to its relevance in the industry) (#353)

@x-tabdeveloping I know you reached out regarding tokens at some point?

Maybe also BM25? Can be done via e.g. BM25Okapi

I will be able to add LLM2Vec supervised and unsupervised.

Bm25okapi will be slow. It is also not an exact okapi implementation per Robertson et al; I believe the implementation differs in the IDF component in it's use of averaging to avoid division by 0 or negative IDF.
I have a new framework that will be 100x faster written in python (np, scipy, optionally Jax/pystemmer). I can release the technical report.

Sounds great @xhluca, can I ask you to create the implementation for that one as well, then we will have all of desired the models implemented (we might add some later on).

Sure!

Btw bm25s is out: https://github.com/xhluca/bm25s
Let me know if I use some reference implementation from some other baseline (e.g. sentence-transformers) so I can easily add bm25s numbers.
I can also post a technical report now if there's a need to cite it.

@xhluca an implementation of bm25 would be great. You can find a reference implementation in mteb/models.

@xhluca will just ping you here

@xhluca will just ping you here

Sure! I haven't had the chance to look into the reference implementation yet, will do today.

Actually just saw #990, it seems the gist looks pretty decent (have not attempted to run it). Unfortunately it might be a bit hard with encode_queries/encode_corpus appraoch since BM25 does not output a representation per se, so using something like dot product is not really straightforward (there are implementations that do that, but I found that htey tend to run out of memory on 30GB systems).

This PR seems resolved - will close it

👍 ; also checked the other ones in your original message (BM25 etc) as they've all been added now 🚀


classes parameter is not valid while I run 'server.launch(...)"

classes parameter is not valid while I run 'server.launch(...)",
I use help(quiver_engine.server.launch) in python command enviroments, it shows parameters definitions and there are NO 'classes' and 'top' paramaters. It is not same as the examples mentioned in README.MD, is the doc a little bit older?

quiver_engine <IP_ADDRESS>.4 has this problem, but after I check function "server.launch(...)" in source codes at master branch, there is a classes paramter same as README.MD, so I just close this issue and wait 0.1.5.


Remove custom goproxy

Switching travis to default go proxy which should be
GOPROXY="https://proxy.golang.org,direct"
Resolves unexpected EOF when go mod vendor

go proxy back online...


clarify README/warnings

Hi!
Please clarify README -> Warnings and known issues -> 2 chapter.
At the moment it starts with "Generic derivation may not work as expected when the type definitions that you're trying to derive instances for are at the same level as the attempted derivation...". It isn't immediately clear what does "level" mean here. Please, elaborate sense here a little. I still can not grok how to avoid "could not find implicit" sporadic issue.
Thanks!

This doc fragment is moved to  https://travisbrown.github.io/circe/tut/codec.html and clarifying is there:

you can arrange the sealed trait definition so that it is not in the same immediate scope as the attempted derivation


Remove useref and Fleet update changes

ARM (Control Plane) API Specification Update Pull Request

[!TIP]
Overwhelmed by all this guidance? See the Getting help section at the bottom of this PR description.

PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

Purpose of this PR
What's the purpose of this PR? Check the specific option that applies. This is mandatory!

[ ] New resource provider.
[ ] New API version for an existing resource provider. (If API spec is not defined in TypeSpec, the PR should have been created in adherence to OpenAPI specs PR creation guidance).
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix OpenAPI spec quality issues in S360.
[ ] Other, please clarify:

edit this with your clarification

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[x] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[x] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the diagram Step 2, "ARM API changes review", for this PR.

Additional information

 Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.

Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
suppressions guide to get approval.

Getting help

First, please carefully read through this PR description, from top to bottom. Please fill out the Purpose of this PR and Due diligence checklist.
If you don't have permissions to remove or add labels to the PR, request write access per aka.ms/azsdk/access#request-access-to-rest-api-or-sdk-repositories
To understand what you must do next to merge this PR, see the Next Steps to Merge comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
For help with ARM review (PR workflow diagram Step 2), see https://aka.ms/azsdk/pr-arm-review.
If the PR CI checks appear to be stuck in queued state, please add a comment with contents /azp run.
This should result in a new comment denoting a PR validation pipeline has started and the checks should be updated after few minutes.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

https://github.com/Azure/sdk-release-request/issues/5257 is waiting for the PR

Please help in merging the PR as I have addressed all the comments. @MaryGao @tadelesh @XiaofeiCao

"version": "2023-11-01-preview",

same comments as in this file for api-version 2024-05-01-preview too.

Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:5 in f557502. [](commit_id = f55750241673c2fbc3e7fdec4058c1568dafb9cf, deletion_comment = False)

  "description": "Specifies additional XML formatted information that can be included in the\nUnattend.xml file, which is used by Windows Setup. Contents are defined by\nsetting name, component name, and the pass in which the content is applied.",

Here and at other places , are the "\n" needed?

Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:491 in f557502. [](commit_id = f55750241673c2fbc3e7fdec4058c1568dafb9cf, deletion_comment = False)

      "type": "string",

Consider adding "format": arm-id" if applicable , here and every place where resource-id is mentioned

Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:528 in f557502. [](commit_id = f55750241673c2fbc3e7fdec4058c1568dafb9cf, deletion_comment = False)

    "licenseType": {

can this be made enum ?

Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:682 in f557502. [](commit_id = f55750241673c2fbc3e7fdec4058c1568dafb9cf, deletion_comment = False)

      "description": "Specifies the operating system settings for the virtual machines in the scale\nset."

needed? here and at other places

Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:660 in f557502. [](commit_id = f55750241673c2fbc3e7fdec4058c1568dafb9cf, deletion_comment = False)

      "description": "Specifies the boot diagnostic settings state. Minimum api-version: 2015-06-15."

what does Minimum api-version represent, here and at other places ?

Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:676 in f557502. [](commit_id = f55750241673c2fbc3e7fdec4058c1568dafb9cf, deletion_comment = False)

      "description": "Specifies the time in which this VM profile for the Virtual Machine Scale Set\nwas created. Minimum API version for this property is 2023-09-01. This value\nwill be added to VMSS Flex VM tags when creating/updating the VMSS VM Profile\nwith minimum api-version 2023-09-01.",

consider adding timezone and format ?

Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:717 in f557502. [](commit_id = f55750241673c2fbc3e7fdec4058c1568dafb9cf, deletion_comment = False)

    "enabled": {

Standard guidance is: replace boolean/switch properties with a more meaningful enum whenever possible.
A boolean will forever have two valid values (true or false). A string enum type is always preferred. Also, properties should always provide better values just than True and False. For example two switches "isTypeA" and "isTypeB" should be replaced with one enum "type": [A, B, DefaultType]. Enums are always a more flexible and future proof option because they allow additional values to be added in the future in a non-breaking way, e.g. [Enabled, Disabled, Suspended, Deallocated].
Note: do NOT define a 'boolean enum' with two values 'True and False'. This might be easier to 'extend' in terms of types, but semantically its cofusing, and no better than a boolean.

Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:726 in f557502. [](commit_id = f55750241673c2fbc3e7fdec4058c1568dafb9cf, deletion_comment = False)

      "$ref": "#/definitions/FleetProperties",

this has "ProvisioningState" as a new property. so, now this is sent in the response for same api-version. If there are existing users for this api-version, will that not break customers ?

Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:1166 in f557502. [](commit_id = f55750241673c2fbc3e7fdec4058c1568dafb9cf, deletion_comment = False)

      "description": "Resource Id"

Is this an azure resource id ?
Can use "format": arm-id" ?

Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:985 in f557502. [](commit_id = f55750241673c2fbc3e7fdec4058c1568dafb9cf, deletion_comment = False)

    "userAssignedIdentityResourceId": {

"format": "arm-id" ?

Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:1029 in f557502. [](commit_id = f55750241673c2fbc3e7fdec4058c1568dafb9cf, deletion_comment = False)

      "description": "Resource Id"

add more details on what type of id ?
"format": "arm-id"

Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:1201 in f557502. [](commit_id = f55750241673c2fbc3e7fdec4058c1568dafb9cf, deletion_comment = False)

    "provisionVMAgent": {

enum instead of bool, as mentioned at other place?

Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:1305 in f557502. [](commit_id = f55750241673c2fbc3e7fdec4058c1568dafb9cf, deletion_comment = False)

      "type": "boolean",

enum instead of boolean?

Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:1576 in f557502. [](commit_id = f55750241673c2fbc3e7fdec4058c1568dafb9cf, deletion_comment = False)

      "description": "Length of time a Virtual Machine being reimaged or having its OS upgraded will\nhave to potentially approve the OS Image Scheduled Event before the event is\nauto approved (timed out). The configuration is specified in ISO 8601 format,\nand the value must be 15 minutes (PT15M)"

curious, should it always be 15 mins ? or PT15M an example ?

Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:1573 in f557502. [](commit_id = f55750241673c2fbc3e7fdec4058c1568dafb9cf, deletion_comment = False)

"VirtualMachineScaleSetExtensionProperties": {

most of them are matching VirtualMachineExtensionProperties. Can such cases be consolidated into a definition and be used where needed ? avoids duplicate code.

Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:2622 in f557502. [](commit_id = f55750241673c2fbc3e7fdec4058c1568dafb9cf, deletion_comment = False)

    },

id,name,type combo looks to be used at many places. Can they be consolidated into a definition and be used ?

Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:2598 in f557502. [](commit_id = f55750241673c2fbc3e7fdec4058c1568dafb9cf, deletion_comment = False)

      "$ref": "#/definitions/FleetProperties",

Also, Could you update examples reflecting this change ?

Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:1166 in f557502. [](commit_id = f55750241673c2fbc3e7fdec4058c1568dafb9cf, deletion_comment = False)

"version": "2023-11-01-preview",

same comments as in this file for api-version 2024-05-01-preview too.
Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:5 in f557502. [](commit_id = f557502, deletion_comment = False)

What does this comment mean? Are you asking me to define the suppressions again in 2023-11-01-preview version? Something like this: https://github.com/Azure/azure-rest-api-specs/blob/1c63635d66ae38cff18045ab416a6572d3e15f6e/specification/playwrighttesting/resource-manager/readme.md?plain=1#L42

      "$ref": "#/definitions/FleetProperties",

Also, Could you update examples reflecting this change ?
Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:1166 in f557502. [](commit_id = f557502, deletion_comment = False)

I did run oav validate examples and there are no errors or issues with the change.

    },

id,name,type combo looks to be used at many places. Can they be consolidated into a definition and be used ?
Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:2598 in f557502. [](commit_id = f557502, deletion_comment = False)

These definitions were generated using https://azure.github.io/typespec-azure/docs/howtos/migrate-swagger/get-started. So we can keep it as is.

"format": "arm-id"

This is using a sub resource in Compute swagger. Hence keeping it the same. https://github.com/Azure/azure-rest-api-specs/blob/main/specification/compute/resource-manager/Microsoft.Compute/common-types/v1/common.json#L149

Not all comments have been addressed. Please address them and place it back in the queue.

"VirtualMachineScaleSetExtensionProperties": {

most of them are matching VirtualMachineExtensionProperties. Can such cases be consolidated into a definition and be used where needed ? avoids duplicate code.
Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:2622 in f557502. [](commit_id = f557502, deletion_comment = False)

Most of these properties are a lift and shift from compute swagger as direct referencing is not allowed. Hence keeping it the same for consistency.

curious, should it always be 15 mins ? or PT15M an example ?

The value should not exceed 15 min. Updated the docs.

    "enabled": {

Standard guidance is: replace boolean/switch properties with a more meaningful enum whenever possible.
A boolean will forever have two valid values (true or false). A string enum type is always preferred. Also, properties should always provide better values just than True and False. For example two switches "isTypeA" and "isTypeB" should be replaced with one enum "type": [A, B, DefaultType]. Enums are always a more flexible and future proof option because they allow additional values to be added in the future in a non-breaking way, e.g. [Enabled, Disabled, Suspended, Deallocated].
Note: do NOT define a 'boolean enum' with two values 'True and False'. This might be easier to 'extend' in terms of types, but semantically its cofusing, and no better than a boolean.
Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:726 in f557502. [](commit_id = f557502, deletion_comment = False)

Keeping it as boolean as it is boolean in Compute swagger as well.

  "description": "Specifies additional XML formatted information that can be included in the\nUnattend.xml file, which is used by Windows Setup. Contents are defined by\nsetting name, component name, and the pass in which the content is applied.",

Here and at other places , are the "\n" needed?
Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:491 in f557502. [](commit_id = f557502, deletion_comment = False)

\n is being added by typespec compiler automatically. I have not defined it in typespec  file.

Regarding the nextLink here, we have a url type in typespec, our recommandation is to use the native type if existing not format decorator? Could you update as following?
  #suppress "@azure-tools/typespec-azure-core/no-format" "NextLink is of uri format and need to be defined."
  @doc("The link to the next page of items")
  nextLink?: url;

Regarding the nextLink here, we have a url type in typespec, our recommandation is to use the native type if existing not format decorator? Could you update as following?
  @doc("The link to the next page of items")
  nextLink?: url;

/cc @ArcturusZhang @tadelesh

Fixed

I've made a PR to fix service-dir:
https://github.com/Azure/azure-rest-api-specs/pull/29679
This should only affect SDKs.

    },

needs to generate with x-ms-secret, add typescript annotations?

Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2024-05-01-preview/azurefleet.json:3002 in c39e954. [](commit_id = c39e954ad92de7959d7d87d327f4e78c6e349531, deletion_comment = False)

    "protectedSettings": {

needs to generate with x-ms-secret, add typescript annotations?

Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:2469 in c39e954. [](commit_id = c39e954ad92de7959d7d87d327f4e78c6e349531, deletion_comment = False)

    "protectedSettings": {

needs to generate with x-ms-secret, add typescript annotations?

Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:2659 in c39e954. [](commit_id = c39e954ad92de7959d7d87d327f4e78c6e349531, deletion_comment = False)

    "adminPassword": {

needs to generate with x-ms-secret, add typescript annotations?

Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:2996 in c39e954. [](commit_id = c39e954ad92de7959d7d87d327f4e78c6e349531, deletion_comment = False)

    "customData": {

Is the intention that this should be x-ms-secret annotated also, or is it not treated as a secret?

Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:3004 in c39e954. [](commit_id = c39e954ad92de7959d7d87d327f4e78c6e349531, deletion_comment = False)

    "content": {

Is the intention that this should be x-ms-secret annotated also, or is it not treated as a secret?

Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:517 in c39e954. [](commit_id = c39e954ad92de7959d7d87d327f4e78c6e349531, deletion_comment = False)

    "protectedSettings": {

Looks like @secret is a thing...

In reply to:<PHONE_NUMBER>

Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:2659 in c39e954. [](commit_id = c39e954ad92de7959d7d87d327f4e78c6e349531, deletion_comment = False)

protectedSettings

I cannot add @secret to Record unknown.

    "content": {

Is the intention that this should be x-ms-secret annotated also, or is it not treated as a secret?
Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:517 in c39e954. [](commit_id = c39e954, deletion_comment = False)

made the necessary changes

protectedSettings

I cannot add @secret to Record unknown.

@extension("x-ms-secret", true) may work.
This is not great typespec, but it may get the Swagger correct... (typespec seems mostly on @visibility. Not an expert though.)

Fixed by using @OpenAPI.extension()

There are still some comments unaddressed regarding booleans, armid, x-ms-secret annocation. please address/reply.

    },

needs to generate with x-ms-secret also, add Typespec annotations?
Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2024-05-01-preview/azurefleet.json:3002 in c39e954. [](commit_id = c39e954, deletion_comment = False)

Added

      "type": "boolean",

enum instead of boolean?
Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:1576 in f557502. [](commit_id = f557502, deletion_comment = False)

It cannot be changed as it is defined in CRP and they are defined as boolean in ComputeRP

There are still some comments unaddressed regarding booleans, armid, x-ms-secret annocation. please address/reply.

Addressed those comments. And added arm-id formatting and x-ms-secret.

      "description": "Specifies the boot diagnostic settings state. Minimum api-version: 2015-06-15."

This would be good to address.

In reply to:<PHONE_NUMBER>

Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:676 in f557502. [](commit_id = f55750241673c2fbc3e7fdec4058c1568dafb9cf, deletion_comment = False)

      "description": "Specifies the time in which this VM profile for the Virtual Machine Scale Set\nwas created. Minimum API version for this property is 2023-09-01. This value\nwill be added to VMSS Flex VM tags when creating/updating the VMSS VM Profile\nwith minimum api-version 2023-09-01.",

Can the timezone and format be added to description?

In reply to:<PHONE_NUMBER>

Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:717 in f557502. [](commit_id = f55750241673c2fbc3e7fdec4058c1568dafb9cf, deletion_comment = False)

      "$ref": "#/definitions/FleetProperties",

this is worth addressing/replying.

In reply to:<PHONE_NUMBER>

Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:1166 in f557502. [](commit_id = f55750241673c2fbc3e7fdec4058c1568dafb9cf, deletion_comment = False)

      "$ref": "#/definitions/FleetProperties",

FleetUpdate example can be appended to include provisioningState.

In reply to:<PHONE_NUMBER>

Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:1166 in f557502. [](commit_id = f55750241673c2fbc3e7fdec4058c1568dafb9cf, deletion_comment = False)

booleans

enum can't be used instead of booleans as I cannot change the types released in Compute swagger.

      "description": "Specifies the boot diagnostic settings state. Minimum api-version: 2015-06-15."

This would be good to address.
In reply to:<PHONE_NUMBER>
Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:676 in f557502. [](commit_id = f557502, deletion_comment = False)

fixed. Removed unnecessary minimum api version

      "description": "Specifies the time in which this VM profile for the Virtual Machine Scale Set\nwas created. Minimum API version for this property is 2023-09-01. This value\nwill be added to VMSS Flex VM tags when creating/updating the VMSS VM Profile\nwith minimum api-version 2023-09-01.",

Can the timezone and format be added to description?
In reply to:<PHONE_NUMBER>
Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:717 in f557502. [](commit_id = f557502, deletion_comment = False)

Added Timezone format in description.

      "$ref": "#/definitions/FleetProperties",

this is worth addressing/replying.
In reply to:<PHONE_NUMBER>
Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:1166 in f557502. [](commit_id = f557502, deletion_comment = False)

This change was proposed by the sdk team which resulted in Update fields to be removed. Update the update model for swagger in azure fleet by MaryGao · Pull Request #29492 · Azure/azure-rest-api-specs (github.com) If I remove the @parametervisibility from ArmCustomPacthAsync then Fleet PropertiesUpdate remain unchanged.

@MaryGao Can you please let us know why this change is needed?

      "$ref": "#/definitions/FleetProperties",

this is worth addressing/replying.
In reply to:<PHONE_NUMBER>
Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2023-11-01-preview/azurefleet.json:1166 in f557502. [](commit_id = f557502, deletion_comment = False)

Confirmed with Mary that we need provisioningState property in update response. This was missed earlier. So the current changes act as bugfix.
cc @madhura-np

@madhura-np I clarified with service team this property is existing in response since begining, and I think adding optional is allowed in new API version.

    },

why do you need this ? I dont see this definition actually being used anywhere.
Also location must have 'x-ms-mutability':['read', 'create'] extension defined as flaged in the errors

Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2024-05-01-preview/azurefleet.json:1860 in 9704381. [](commit_id = 97043813edfc7ed3f4343ece7b4d970bb7d6fcb2, deletion_comment = False)

Yeah adding the decorator parametervisibility is because we don't want to introduce un-necessary Update model in SDK side. Actually this will not impact the REST API behavior. The property provisioningState with optional and readonly is always there from service end side.
@madhura-np I clarified with service team this property is existing in response since begining, and I think adding optional is allowed in new API version.

I think that is fine

Also location must have 'x-ms-mutability':['read', 'create'] extension defined as flaged in the errors

Looks like this is used to define the VirtualMachineExtension type. Can you please explain what are you trying to do here in terms of modelling
I dont tsee this in the examples as well. Could you please add an example here as well so that users can see how this property is to be used ?

    },

why do you need this ? I dont see this definition actually being used anywhere.
Also location must have 'x-ms-mutability':['read', 'create'] extension defined as flaged in the errors
Refers to: specification/azurefleet/resource-manager/Microsoft.AzureFleet/preview/2024-05-01-preview/azurefleet.json:1860 in 9704381. [](commit_id = 9704381, deletion_comment = False)

It does not have any visibility set to read or create. I cannot refer properties directly from compute namespace.  https://github.com/Azure/azure-rest-api-specs/blob/main/specification/compute/resource-manager/Microsoft.Compute/common-types/v1/common.json#L172
Let me know if I still need to edit it?

Also location must have 'x-ms-mutability':['read', 'create'] extension defined as flaged in the errors

Looks like this is used to define the VirtualMachineExtension type. Can you please explain what are you trying to do here in terms of modelling
I dont tsee this in the examples as well. Could you please add an example here as well so that users can see how this property is to be used ?

Example is present:

Also location must have 'x-ms-mutability':['read', 'create'] extension defined as flaged in the errors

Looks like this is used to define the VirtualMachineExtension type. Can you please explain what are you trying to do here in terms of modelling
I dont tsee this in the examples as well. Could you please add an example here as well so that users can see how this property is to be used ?

Updated examples. Location was having a random value. https://github.com/Azure/azure-rest-api-specs/blob/4ac7e1ff5ec6bb7bc19a24557548b2d54dfd5287/specification/azurefleet/AzureFleet.Management/examples/2024-05-01-preview/Fleets_CreateOrUpdate.json#L394

As discussed in office hours consider if you want to allow customers to update the location field in the payload with a RE-PUT. Please restart the review once you have discussed this internally

As discussed in office hours consider if you want to allow customers to update the location field in the payload with a RE-PUT. Please restart the review once you have discussed this internally

Updated the model to the latest securityPostureReference which does not have location. Removed unnecessary models.

/azp run

/azp run

/pr RequestMerge


Readme.md out of date/not working

Step 5. Deploy
Module does not exist...
Error: Cannot find module '/Users/ajk/OpenEXP/server.js'
    at Function.Module._resolveFilename (module.js:337:15)
    at Function.Module._load (module.js:287:25)
    at Function.Module.runMain (module.js:467:10)
    at startup (node.js:136:18)
    at node.js:963:3

ah, thanks for mentioning this. these steps have changed. I will update the readme soon

@teonlamont can we close this?


New to node.js how to start using it

I have installed node.js on my local machine. I have used gulp before, but not familiar how this works?
So, how to get started? Do I need a create any .js file, like we create in gulpfile.js.

Hi, maybe you should try starting with https://nodeschool.io/#workshopper-list ?

Thanks @saper.


Initial commit for the publication workflow implementation

Will fix #24

The publication workflow is working great, and has successfully published everything I've pointed it at.


Annotations "inheritance"

Review comments fixed:

simple test;
Annotations.java moved to util module;
comment about search added.

Let's add a @StringColumn meta-annotation that is equivalent to
@Column(customValueType=@CustomValueType(
    columnValueType=FieldValueType.STRING,
    columnClass=String.class,
    converter=StringValueConverter.class
)

?
This will allow fun like @StringColumn UUID uuid fields in entity classes :-)

Maybe also make @CustomValueType usable in a meta-annotation?
E.g.: you will be able to add @StringValueType annotation for on class, and that will be <=> @CustomValueType(..., converter=StringValueConverter.class)


Disable toggle select all button while searching

Disable toggle select all button while searching using @sa-si-dev 's suggestion in the last pr.

I checked in this feature now only.
If possible do not check in .min files.


Add the enchanting "spells" as well as formulas?

I just noticed now when building a list of professions for my guild, that the items that are really spells, are not present in data.json.
E.g. "Formula: Enchant Boots - Spirit" is present with itemId 16220, but "Enchant Boots - Spirit" with id 20024 is not. This also means I lose out on the very handy list of reagents. Enchanting is the only profession I checked, but there might be similar cases for other professions. Is it possible that this information can be added?

Would be nice if this could be looked into for wotlk. Noticed this happens with other professions like leatherworking for armor kits. Would it be possible to retrieve all the craftable items from the profession spell tables and insert them into the matching items as I think all the spells include the reagent info but not all item id's do?
Example of the spell table filtered by profession which include all the reagent info:
https://wowhead.com/wotlk/spells/professions/leatherworking
https://wowhead.com/wotlk/spells/professions/enchanting
https://wowhead.com/wotlk/spells/professions/engineering
https://wowhead.com/wotlk/spells/professions/blacksmithing
https://wowhead.com/wotlk/spells/professions/jewelcrafting
https://wowhead.com/wotlk/spells/professions/alchemy
https://wowhead.com/wotlk/spells/professions/mining
https://wowhead.com/wotlk/spells/professions/tailoring


Discord Bot

Tasks

[x] Create some basic functionality for Discord Bot
[ ] Add music feature, like Rhythm
[ ] Add music searching, like pancake bot

some tasks will add later

Literature

Mezgoodle's repo
Video about Discord Bots
Video About Music Bots
Nice video with searching audios on YouTube (only js)

Branch
Discord-Bot
Folder
Discord-Bot

Whoooray!


feat: add manifest functions

The plain std.manifestYamlDoc function produces a valid Yaml but sorts the fields alphabetically. This PR introduces two functions to render the Actions and Workflow Yamls in an opinionated manner. The aim is to make the Yaml look more like it has been handwritten.

An xtd function for this might be cool but I think this is version too opinionated right now.


Saving and loading memory banks

Suppose I save memory bank number 5 (from Save > Current bank)
I restart the application and select memory bank number 0.
Then I load memory bank number 5  (from File > Load bank) this is the memory bank I previously saved from step 1.

Does the application load it into memory bank 0 (the current selected memory bank) or do I have to select memory bank 5 to see it?

The application will load the bank in the currently selected memory bank.


Disable logging requests and responses in config

Add a config option to disable these
It can be preferable to turn the log entries off, particularly for large batch requests

I'm still looking for a way to disable logging, any way to do it?

You can like this:
import logging
logging.getLogger("jsonrpcclient.client.request").setLevel(logging.WARNING)
logging.getLogger("jsonrpcclient.client.response").setLevel(logging.WARNING)


Wrong wallet

I was suppose to enter my litecoin wallet but I entered my Bitcoin wallet instead. Using the soChain I can see the transaction was successful but labelled my Bitcoin wallet as LTC. When I checked the transaction in my Bitcoin wallet the payment cannot be found. I was paid through the wrong wallet I typed so can there be a solution to transfer the funds still as litecoin to my litecoin wallet?

Sorry to hear about your trouble, but I suspect you've made a mistake posting this here.
This project is neither Bitcoin nor Litecoin. I'm going to close this ticket.


modem: gsm_ppp: Support for PPP modems that doesn't support DLCI

Is your enhancement proposal related to a problem? Please describe.
Currently, gsm_ppp uses DLCI to mux between PPP traffic and AT traffic. Quectel EC2X for example, doesn't support this, and switches from PPP to AT either by escape sequence:

Wait for 1 second
Send +++ to the UART
Wait for 1 second
Wait for the OK replied from the modem

or by gpio method:

Pull the DTR pin from low to high
Wait for the OK replied from the modem

To return to PPP:

Sends ATO
Wait for the CONNECT ... message from the modem

Also, EC2X modem output its GPS NMEA messages from a debug UART port instead of DLCI.
Describe the solution you'd like
A kconfig to switch between the DLCI method and the escape sequence/gpio sequence. However, different manufacturers might have different way of doing this and therefore this might becomes ugly in the future.
For the GPS, kconfig might be possible to switch between the DLCI and another UART port in the driver, however I'm not sure how can this be configured in the dts
Describe alternatives you've considered
A seperate PPP driver for modem that doesn't support DLCI?

The GSM muxing is optional and not enabled by default in GSM modem driver. It should not be a big task to add another way of multiplexing the data in the drivers/modem/gsm_ppp.c. Patches are welcome if you want to implement this.


[Addition]: Hummingbird selfwitnessing cluster

Hotspot b58 Addresses
1126TU7k64UCuYHsw3xHqPDa6DEAXmdncMJvM38xLv7ARET4hDK7
112cM4eMZ5Ja1J23qzue6raZzhpR6TTXdDYAGJkCpRPDSqySUoFt
112hrQJz9Jr5i6ZR4Rogc5pivWJ8A9kdSwER8EBymMobwqjoFK3c
112JEiQbWr2FqiEQkvvHS5gjH4PXTf6ct87KhbuFkZLzqZgCwjop
112NaPDhWTpvTHy1L3WF3zEe1FhKCeL6mJKjPy8pYr7ZZPoYeRoD
112PN6DXLdU8Uw3F2Mm61TrCdr6w41duuGHtf1dFo7n9ZX2AHcXb
112UDHXVNSnuVrQBvpRD8yoGCorKT1UkeCNKJAfZDW1cZTUHNsLX
112WoRKNiFh8MDiHtD4zs3u35PcQHGDb2YKpm4mWhNSVPebV6485
112xDqC9xFL1xAcNc7kp6rFWjn6t1gfypc32LHMemapRKGgE5BT
11A7u62d1mC7pn2QWD91Q3vZ8rVZMk3QgYMeqcnaBoZdh5BLYUr
11duFvd86J5iQMspdSD3XqzxoDhBPGjYeYetUnS5WrfL6unQLXR
11JmKvZMeoB3h7YwCbNyWcXK3BwW7dYEyaSA6J2SZUhkRxjBhCQ
11mhZ5ymrwsk5gqnutTtT5nFpVdQ95eDZ8t7K5TumtcuDD1aT3z
11oeMQxPCDqFjiBN65gkZtjCjFZHuVWxfeNVVFroD3omdugLfF1
11RJW9q3h17xgpZNZJin7Y9zzvMS29NQH9QRa71SkXgXJbr3ska
11TnRf7GQ4bwNgiGmeujndNRdAebu1wvSe2aryw3xQPEpDDe9ak
11UWVPuaJHzhqAa6bmJpHnb6SoUfyRFoZaqmc4y5AK2k7eTGgZ5
11XkP78WeEZjncJscALykuRZhRZuhhinS6hDU6EQde4Nv4fhRTv
11XSqPwUxGaBYNqJ9gZKgBmvJgRzYzrSWFJFGw67ALZzJdo4C7A
11yHyyVQjvJw5crM8SjTZVaLPAMKxFBjzP5ndtAEHr2Q4QYbN78
Discord Handle
No response
Reason(s)

All but 1 account for all Hummingbird gateway's didn't show any data transfers.
So data transfer is possible! only 109 out of 3853 showed packet transfer data, all these 109 are in same account.
the gateway's in that account are part of a selfwitnessing cluster so they probably all heared the same
signal.

Most of all Hummingbird gateway's witness only Hummingbird gateway's.

We iterated the witnesses of the witnesses to find the clusters.
We used the node-red server we setup for this to get the results out of api.

All HNT stolen with the total amount of 21067 HNT as by todays date/time.

We conclude that out of the 3852 gateway's we inspected only 109 gateway's in 1 account showed accidental
data transfer. so data transfer for gateway's Hummingbird is possible.

We conclude that out of the 3852 gateway's we inspected only a couple of accounts showed communication with other makers. so communication with other makers for Hummingbird gateway's is possible.

Rssi/ distance / SNR calculations are to perfect, we have found some invalids -59 dbm but we suspect they are from cables touching eachother.

We haven't targetted any maker, it was the result of a count WHERE actors1.actor_role = 'witness'
AND actors2.actor_role = 'challengee' are in same account.

Resolved in https://github.com/helium/denylist/pull/4048


[Feature] Donor Claim blocks

Describe the bug
Claim blocks are not default setting to X amount for donor rank Omen, Unsure about other donor ranks.
This was said in changelog " This means old Store purchases should have kept their baseline rank claim block count."
This may be an only me issue.
To Reproduce
Steps to reproduce the behavior:

Run /claim
Claim menu pops up with set amount of claimblocks at 1024
Thats bad hehe

Expected behavior
Should have 15k I beleive?
Screenshots

Username:
_Conj
Server name (all applicable):
At least Orion
Additional context
Some commands are messed up, but there is an existing ticket for that by itsAmaze
https://github.com/NebulaMC-GG/Bugs-and-Issues/issues/2207

Could you please test this once more and let us know if this issue has resolved for you?  Thank you!

This has been fixed!  Thank you for your report <3


libpng error when converting a jpg to a png with oiiotool

Problem
Hello,
We get an error during the conversion with oiiotool of a jpeg image to a png image.
A segmentation fault occur.
Expected behavior:  No error
Actual behavior:
Here is the input .jpg image:

The convertion command:
(oiio-1.8.15): -> oiiotool -i /a.jpg -o /b.png

gives:
libpng error: profile matches sRGB but writing iCCP instead
Segmentation fault (core dumped)

This error also occurs when we directly use the c++ API to perform the image conversion in our project AliceVision
Versions

OIIO branch/version: Release 1.8.15
OS: Linux
C++ compiler: gxx-4.9

Sorry to have let this fester for a long time, but here is the best I can figure out:
I was never quite able to reproduce a segfault (even going back as far as OIIO 1.8), although for OIIO 2.1 and older, I did get the "libpng error: profile matches sRGB but writing iCCP instead."
For OIIO 2.2 (which is the current supported release), as well as the current master, I don't even get the error message. Everything seems fine with the png image.
So I'm presuming that somewhere along the way, we fixed whatever was up with the PNG writer. Maybe #1910  ? Maybe it was specifically in response to this issue but somehow I forgot to note it here and close the issue, or maybe it was entirely coincidental.
Can you try on your end with 2.2 and verify that it's fixed these days?

I just tested with oiio-<IP_ADDRESS> and png-1.6.37, there is no more crash (just the libpng warning), so it seems to be fixed!
Thanks

Just stumbled across this Github issue, and we seem to be encountering the same thing doing a basic conversion from .tx (mipmapped .tif) to .png using OIIO <IP_ADDRESS> (libpng 1.6.35, GCC 6.3.1), e.g: oiiotool -i /path/to/input.tx -o /path/to/output.png
I still need to dig into it some more (test newer libs, debug/unoptimized builds, etc), but here's the stack trace with our current release build:
#0  0x00007ffff555d6d8 in std::basic_string<char, std::char_traits<char>, std::allocator<char> >::basic_string(char const*, std::allocator<char> const&) ()
   from /lib64/libstdc++.so.6
#1  0x00007ffff755de5f in OpenImageIO_v2_2::PNG_pvt::create_write_struct(png_struct_def*&, png_info_def*&, int&, OpenImageIO_v2_2::ImageSpec&, OpenImageIO_v2_2::ImageOutput*) ()
   from /luma/soft/rez/packages/openimageio/<IP_ADDRESS>/platform-linux/gcc-6.3.1/boost==1.61.0/python-2.7/!nuke/lib64/libOpenImageIO.so.2.2
#2  0x00007ffff75875bd in OpenImageIO_v2_2::PNGOutput::open(std::string const&, OpenImageIO_v2_2::ImageSpec const&, OpenImageIO_v2_2::ImageOutput::OpenMode) ()
   from /luma/soft/rez/packages/openimageio/<IP_ADDRESS>/platform-linux/gcc-6.3.1/boost==1.61.0/python-2.7/!nuke/lib64/libOpenImageIO.so.2.2
Backtrace stopped: previous frame inner to this frame (corrupt stack?)```

@nrusch Can you try with either a newer OIIO 2.2 (2.2.19 was the latest published) or 2.3, and see if this is something that's already fixed?

And in case it persists in the latest versions of either 2.2 or 2.3... does it do this for any PNG file? Or only particular ones? Or only intermittently?

I will try and build a newer version to test, but I think I've narrowed down the issue a bit more: it seems to occur when the input image has an embedded ICC profile and oiio:ColorSpace is set (I'm not sure if the latter is always set on input based on an educated guess, or if it's inferred from other metadata).
Adding either --eraseattrib ICCProfile or --eraseattrib oiio:ColorSpace to the previous conversion makes it work.

OK, it actually looks like just updating to libpng 1.6.37 and rebuilding OIIO (<IP_ADDRESS>) was enough to resolve this issue. I don't know if it's worth making a note of this in the docs, but at the very least this should give anyone else who stumbles across this Github issue a path forward.
I did just notice that libpng is not listed in the "Required dependencies" section of the INSTALL.md, but it is listed in the "Building OpenImageIO on Linux or OS X" section.


PHP commenting is displaying as HTML comments in a pure php file

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.95.3
OS Version: Windows 10

Hello, after updating vscode, when i try to do CTRL + / in order to comment a line in php file, it comments as HTML comments

instead of normal php comments
//
I have the built in extenstions enabled, and external extension PHP Debug by xdebug.org
Also I had php extension pack, disabled it, removed it, and the problem is still happening.
Steps to Reproduce:

open vs code
create a pure php file with extenstion .php
try to comment with CTRL +/
recieve HTML comments instead of php comments

this has always been the case
however there is a new bug
ctrl+/ toggle comment doesn't work after opening a blank php file for the first time
it used to work in 1.75.1, but broke in 1.76.0

@RedCMD I don't understand you man. for me it worked at least until version 1.94. so no relation to what you said

as you can see, the language is set to html
if I set a php tag it works

I don't understand you man
https://github.com/microsoft/vscode/issues/234722

@RedCMD what extenstions do you have installed?
Because now, I have "PHP Debug" and "PHP Extenstion Pack" by Xdebug, and my <?php looks different than yours in the color as yuo can see in the image below, maybe this is the cause of the problem

I encountered the same issue as well. I'm also using Windows 10 and VSCode 1.95.3.

For this to work you need to install a PHP extension. The built-in PHP support is very basic.

I am sorry @aeschli , but i did install php extensions rather than the basics one, and I did mention that in my post of the issue, so please do not close the issue yet

report to the php extension
if they deem it a VSCode issues @ aeschli will reopen this

Hey, @sagisar1. I had the same issue as shown in your screenshot, but I managed to fix it!
Here's what I did:
Press F1 and run command Developer: Inspect Editor Tokens and Scopes
In my case, I noticed the language was detected as HTML instead of PHP.

Next, I ran the command Extensions: Disable All Installed Extensions, and the PHP syntax highlighting started working again.

To identify the culprit, I re-enabled my extensions one by one. It turned out that the EJS Language Support extension was causing the issue in my case.
If you're experiencing something similar, you can follow these steps to troubleshoot and identify which extension might be breaking PHP highlighting for you.
Hope this helps! 😊

Hello @RedCMD , now knowing that the ejs extension creates the conflict with php,  who should I report to now? I think it should be ejs or vs code, but I think that is more of vs code related

EJS first
if they find it to be a VSCode issuse, then it'll come back here


ci: running cargo build in release as build test

This solves the issue #6045 where there are breakages in the rust build , so to fix that ,we build in release mode and this can be done by running cargo build --release. There is already a makefile command which runs this so we just had to add that in the cicd pipeline.

Checks

[x] All commits in this Pull Request are signed and Verified by Github.
[ ] All commits in this Pull Request follow the Ockam commit message convention.
[ ] There are no Merge commits in this Pull Request. Ockam repo maintains a linear commit history. We merge Pull Requests by rebasing them onto the develop branch. Rebasing to the latest develop branch and force pushing to your Pull Request branch is okay.
[ ] I have read and accept the Ockam Community Code of Conduct.
[ ] I have read and accepted the Ockam Contributor License Agreement by adding my Git/Github details in a row at the end of the CONTRIBUTORS.csv file in a separate pull request to the build-trust/ockam repository. The easiest way to do this is to edit the CONTRIBUTORS.csv file in the github web UI and create a separate Pull Request, this will mark the commit as verified.

@rghdrizzle seems you are not signing your commits when pushing to Github, here's a tutorial on how to sign commit and more on generating and adding GPG keys. For this PR, I will be signing the commits so that we can merge immediately.

@metaclips
Oh Thats strange ,for some reasom when I rebased from github browser , it didnt sign , my first commit was signed though.


Condition based on macro variable - classic pipelines

Hello,
how can we use macro variables in conditions? Can we only use template expressions here? If yes, is there any way that we can declare it in classic pipelines?
Thanks

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

ID: 21e5cee4-eaae-3a96-db91-540ac759e83a
Version Independent ID: 9bdc837c-ffe0-d999-f922-f3a5debc7f92
Content: Conditions - Azure Pipelines
Content Source: docs/pipelines/process/conditions.md
Product: devops
Technology: devops-cicd
GitHub Login: @juliakm
Microsoft Alias: jukullam

@roofiq did you find a solution for your question?

Hey @ffeldmann, i guess Yes. But share your issue, would be glad to help.

@roofiq thanks for the offer, I solved my issue :)


Replication factor to 1 but still have 3

Hey ;) I try to have only one broker.
I need set offsets.topic.replication.factor to 1 default is 3.
Here is my values.yaml
cp-zookeeper:
    servers: 1
    prometheus:
        jmx:
            enabled: false

cp-kafka:
    brokers: 1
    configurationOverrides:
        offsets.topic.replication.factor: 1
        default.replication.factor: 1
        min.insync.replicas: 1
    prometheus:
        jmx:
            enabled: false

cp-schema-registry:
    enabled: true
    prometheus:
        jmx:
            enabled: false

cp-kafka-rest:
    enabled: false

cp-kafka-connect:
    enabled: false

cp-ksql-server:
    enabled: false

cp-control-center:
    enabled: false

And still have 3 factors and error:
[2020-11-29 17:08:04,178] ERROR [KafkaApi-0] Number of alive brokers '1' does not meet the required replication factor '3' for the transactions state topic (configured via 'transaction.state.log.replication.factor'). This error can be ignored if the cluster is starting up and not all brokers are up yet. (kafka.server.KafkaApis)
Command:
helm install kafka -n kafka -f .\values.yaml confluentinc/cp-helm-charts
What Am I doing wrong ?

Looks like you also need to set the configuration for confluent metrics:
  configurationOverrides:
    "offsets.topic.replication.factor": 1
    "default.replication.factor": 1
    "confluent.metrics.reporter.topic.replicas": 1

Looks like you also need to set the configuration for confluent metrics:
  configurationOverrides:
    "offsets.topic.replication.factor": 1
    "default.replication.factor": 1
    "confluent.metrics.reporter.topic.replicas": 1

As the error says, transaction.state.log.replication.factor needs to be set to 1, as well

I have the same problem.  https://github.com/confluentinc/cp-helm-charts/issues/509
there are params that are not passed to the applications

@survivant I'm not seeing any error about replication factors in your post

the logs wasn't complete
...
[2021-04-07 19:07:42,935] WARN The replication factor of the schema topic _schemas is less than the desired one of 3. If this is a production environment, it's crucial to add more brokers and increase the replication factor of the topic. (io.confluent.kafka.schemaregistry.storage.KafkaStore)
[2021-04-07 19:07:43,230] INFO ProducerConfig values:
...

my point was more that I have specify 1 replica, 1 server... in all the settings that I found in the chart's values.yaml  and at the end, the applications are still looking for 3 replicas (probably a default value).
there are settings that are not passed using Helm charts

You need to pass through a configurationOverrides for kafkastore.topic.replication.factor or env-var for SCHEMA_REGISTRY_KAFKA_STORE_REPLICATION_FACTOR
https://github.com/confluentinc/cp-helm-charts/blob/master/charts/cp-schema-registry/templates/deployment.yaml#L88-L94
The default, indeed, is 3 - https://docs.confluent.io/platform/current/schema-registry/installation/config.html#kafkastore-topic-replication-factor

@survivant You need to pass through a configurationOverrides for kafkastore.topic.replication.factor or env-var for SCHEMA_REGISTRY_KAFKA_STORE_REPLICATION_FACTOR
https://github.com/confluentinc/cp-helm-charts/blob/master/charts/cp-schema-registry/templates/deployment.yaml#L88-L94
The default, indeed, is 3 - https://docs.confluent.io/platform/current/schema-registry/installation/config.html#kafkastore-topic-replication-factor

This is incorrect.  It should be just replication.factor (or SCHEMA_REGISTRY_REPLICATION_FACTOR) as per the linked source code.  Confirmed that it works.

replication.factor alone is not a valid config
Related issue https://github.com/confluentinc/cp-docker-images/issues/448

All I know is that this works, results in SCHEMA_REGISTRY_REPLICATION_FACTOR being set correctly on the Deployment, and no errors in the log:
    cp-schema-registry:
      configurationOverrides:
        replication.factor: 1

And this does NOT work, the service doesn't respond, results in the logged error above (Number of alive brokers '1' does not meet the required replication factor '3'):
    cp-schema-registry:
      configurationOverrides:
        kafkastore.topic.replication.factor: 1

Chart version 0.6.1

In my case I had to set these variables in my values file:
kafka:
  replicas: 1
  config:
    offsets.topic.replication.factor: 1
    transaction.state.log.replication.factor: 1
    transaction.state.log.min.isr: 1


Handlebars Partial blocks don't work for Patterns

Hi there,
do handlebars partial blocks work only for the styleguide templates or can I use them in Pattern templates as well?
I could use embed but partial-blocks are neat.
Here's what I am trying to do:
In patterns/components/MyComponent/index.hbs:

   {{>@partial-block}}

Somewhere else:
{{#> patterns.components.MyComponent.index }}
   Lorem ipsum
{{/patterns.components.MyComponent.index }}

Expected result:

   Lorem ipsum

Instead I get an error
Error: The partial @partial-block could not be found

@giuseppeg
That's interesting. This is definitely not by design, so we'll look into it.

@giuseppeg
Can you confirm that every instance of the partial in question is being invoked using the block syntax  ({{#>)? In my testing, I got this same error, but realized that it was because I had used the partial multiple times (for different pattern variations), some with {{embed}}.
If you need to use either the block syntax or {{embed}} on a component partial, you can also try adding logic to check for @partial-block:

  {{#if @partial-block}}    {{! <-------- for using the block syntax }}
    {{> @partial-block}}
  {{else}}
    {{#block "content"}}    {{! <-------- for using embed or extend }}
      Default content
    {{/block}}
  {{/if}}

Sorry @erikjung I was out for the weekend.
Yes I've used the block syntax.

If you need to use either the block syntax or {{embed}} on a component partial, you can also try adding logic to check for @partial-block

Do you mean if I want to use a template as embed AND partial? If so, no it is not my case.
It seems that drizzle is just not looking at the patterns/ folder for partials.

@giuseppeg
I looked into this a bit further, and it appears you can use {{>@partial-block}} as long as some care is taken for base patterns.
Here is an example that builds with no error:
https://github.com/cloudfour/drizzle/compare/debug-issue_87
This is not intuitive, but it kind of makes sense. All patterns are registered as partials, and they are also rendered as patterns unless you put hidden: true in the front matter. It is in this initial rendering that we encounter the error: @partial-block won't exist until that pattern is used as a partial with the block syntax.
So if you intend to make a "base" pattern to be extended, there are two options for using the partial block syntax:

Make the base pattern hidden:

---
hidden: true
---

{{>@partial-block}}

Use a condition to only output @partial-block if it exists:

{{#if @partial-block}}
  {{>@partial-block}}
{{/if}}

It's probably worth mentioning this in the docs :)

@erikjung awesome man! Thank you so much for debugging this for me :)
I will try your suggestions tomorrow.

@erikjung the approach seems to be working, however it explodes when I use it as simple partial without the block syntax:
---
hidden: true
---

  {{#if @partial-block}}
    {{>@partial-block}}
  {{else}}
     Default Content
  {{/if}}

Works
{{#>partial-name}}
{{/partial-name}}

Doesn't work: RangeError: Maximum call stack size exceeded
{{>partial-name}}

This should display the default content right?

@giuseppeg
That should work fine. I added another example to this test branch to verify the plain partial usage, and I wasn't able to reproduce that error: https://github.com/cloudfour/drizzle/compare/debug-issue_87
Can you pull that branch and verify?

Here is a diff of the part that cause the Maximum call stack size exceeded
If I remove
{{>"patterns.components.nav.item-link" href="#foo"}}

the build doesn't break
diff --git a/src/patterns/components/nav/example.hbs b/src/patterns/components/nav/example.hbs
new file mode 100644
index 0000000..7289bad
--- /dev/null
+++ b/src/patterns/components/nav/example.hbs
@@ -0,0 +1,14 @@
+{{#>patterns.components.nav.index}}
+
+  {{>"patterns.components.nav.item-link" href="#foo"}}
+
+  {{#>patterns.components.nav.item}}
+    {{#>patterns.components.nav.link href="#baz"}}
+      Link
+
+        🕶
+
+    {{/patterns.components.nav.link}}
+  {{/patterns.components.nav.item}}
+
+{{/patterns.components.nav.index}}
diff --git a/src/patterns/components/nav/index.hbs b/src/patterns/components/nav/index.hbs
new file mode 100644
index 0000000..ed50b2c
--- /dev/null
+++ b/src/patterns/components/nav/index.hbs
@@ -0,0 +1,10 @@
+---
+hidden: true
+---
+
+  {{#if @partial-block}}
+    {{>@partial-block}}
+  {{else}}
+    Nav List
+  {{/if}}
+
diff --git a/src/patterns/components/nav/item-link.hbs b/src/patterns/components/nav/item-link.hbs
new file mode 100644
index 0000000..a3ec35d
--- /dev/null
+++ b/src/patterns/components/nav/item-link.hbs
@@ -0,0 +1,12 @@
+---
+hidden: true
+---
+
+
+    {{#if @partial-block}}
+      {{>@partial-block}}
+    {{else}}
+      Link
+    {{/if}}
+
+
diff --git a/src/patterns/components/nav/item.hbs b/src/patterns/components/nav/item.hbs
new file mode 100644
index 0000000..ba6dc63
--- /dev/null
+++ b/src/patterns/components/nav/item.hbs
@@ -0,0 +1,10 @@
+---
+hidden: true
+---
+
+  {{#if @partial-block}}
+    {{>@partial-block}}
+  {{else}}
+    Item
+  {{/if}}
+
diff --git a/src/patterns/components/nav/link.hbs b/src/patterns/components/nav/link.hbs
new file mode 100644
index 0000000..bbf8d23
--- /dev/null
+++ b/src/patterns/components/nav/link.hbs
@@ -0,0 +1,10 @@
+---
+hidden: true
+---
+
+  {{#if @partial-block}}
+    {{>@partial-block}}
+  {{else}}
+    Link
+  {{/if}}
+

@giuseppeg
Thanks for the additional info. It looks like this is an issue that affects Handlebars itself: https://tonicdev.com/erikjung/57c72b19ad12611300cb9f58
^ Same error conditions (no error when removing the patterns.components.nav.item-link partial)

@erikjung I see, thanks for taking the time to dig into this issue!
I am using handlebars for the first time so I thought I was doing something wrong.
[unrelated] I am working (hacking) on a fork of drizzle that integrates suitcss and what I think is a great (better) folders structure, I am also considering to make a template-engine agnostic builder pkg that uses consolidate.js or metalsmith-layouts behind the scenes – basically I may end up writing my own styleguide generator tool from scratch 😂 let me know if you folks are interested in a collaboration or want to discuss about this in details.

@giuseppeg We use SUIT on most of our projects, so that's great 👍
The template handling is also something we plan on improving for 1.0. We've received some good feedback on it, and using Consolidate seems to make a lot of sense.

basically I may end up writing my own styleguide generator tool from scratch

That's how we ended up making Drizzle (forked Fabricator)!
Our team welcomes contributions and collaborations. If you have ideas, proposals or wish to discuss some aspect of the hack/fork/project you're working on to see if your ideas would be a good fit for Drizzle proper, please don't hesitate to create issues or ping any of us!


Error: getaddrinfo ENOTFOUND https

This is most likely user error, please join the Discord server for further assistance as Github issues are for actual bugs within the code.


Upgrade manual does not show in kubedb.com

Upgrade Manual is there. But, needs to fix the name. or, may be in another folder?


Missing Darkvision Macro

Hi there,
the readme says the Darkvision spell requires a macro.
However, there is no such macro.

This was an old version of the spell, it has recently been changed. However I am pushing a new version with an updated spell/macro now

This was an old version of the spell, it has recently been changed. However I am pushing a new version with an updated spell/macro now


Enum tuple variant changed kind

solves second lint from https://github.com/obi1kenobi/cargo-semver-checks/issues/884

Done


fix(ui): Fix accessibility problems in Clusters

Description
Problems

Visit /main/clusters and then click a row
Minor: Table header text should not be empty

Click Next
Serious: Elements must meet minimum color contrast ratio

Element has insufficient color contrast of 4.32 (foreground color: #2c8820, background color: #edffeb, font size: 10.5pt (14px), font weight: normal). Expected contrast ratio of 4.5:1
<button type="button" class="mr-2 btn btn-success" data-testid="">

Element has insufficient color contrast of 2.08 (foreground color: #5fb2f7, background color: #ebf6ff, font size: 9.0pt (12px), font weight: normal). Expected contrast ratio of 4.5:1
<button type="button" class="download uppercase text-primary-600 p-2 text-center text-sm border border-solid bg-primary-200 border-primary-300 hover:bg-primary-100 flex items-center">

Element has insufficient color contrast of 4.32 (foreground color: #2c8820, background color: #edffeb, font size: 9.0pt (12px), font weight: normal). Expected contrast ratio of 4.5:1
<button class="border-2 font-600 inline-flex items-center justify-center rounded-sm uppercase text-sm border-success-500 bg-success-200 hover:bg-success-300 hover:text-success-800 text-success-700 p-2" type="button">Download Helm values</button>

Analysis and Solutions

Replace Th with Td element.

Replace the following with PatternFly Button element:

Upgrade and Delete buttons: PanelButton from src/Components

Next or Finish button: PanelButton from src/Components

Download YAML file and keys: HTML button element

Download Helm values: SuccessButton from ui-components package

Bonus to remove another leftover blue background from StackRox branding.

Remove titleClassName prop so collapsible cards have default header style.

Checklist

[ ] Investigated and inspected CI test results
[x] ~Unit test and regression tests added~

Testing Performed

yarn lint in ui
yarn build in ui and then compute branch - master for the following

wc build/static/js/*.js
main.js -173 = 4231696 - 4231869
total -1848 = 10310740 - 10312588
ls -al build/static/js/*.js | wc
files 0 = 80 - 80

yarn start in ui

Manual testing

Visit /main/clusters

See Upgrade and Delete buttons disabled in first picture, and with Delete enabled in second picture.

Click a row to open side panel.

See Next button.

Click Next.

See Finish button.
See collapsible headings with default heading style.
See both Download buttons, and then click to see disabled and loading state (temporary change to leave it spinning long enough to take the second picture).

Integration testing

yarn cypress-open in ui/apps/platform

clusters/clusterDeletion.test.js
clusters/clusters.test.js
clusters/clusterCertificateExpiration.test.js
clusters/clusterHealthStatus.test.js
clusters/clustersManager.test.js
clusters/delegateScanning.test.js
clusters/redirectFromDashboard.test.js

ocp-4-13-ui-e2e-tests failure

Cluster initialization failed because one or more operators are not functioning properly.


[NR-102311] Memory issue fix

Runner doesn't wait for timeout after investigation
Change it to thread sleep for runner to resolve memory issue
Turn on ANRMonitor

Other than the try/catdh I don't see anything that would address a memory leak.

So first I have discovered that runner.wait(TIMEOUT) doesn't take affects, while the loop itself keeps spinning, supposedly gc should take affect to clean up WaitableRunner but it didn't.
After I change it into thread.sleep, the memory leak is no longer happening from the profiler view. The theory I came up with is by put the loop thread into sleep allows other process takes place, aka gc is kicking in to clean up WaitableRunner.
Profile compare as below:

However, the wait(TIMEOUT) waits on notification from the runner which may be up to the TIMEOUT duration, or more usually right away. sleep() will cause the thread to block every loop, which is not optimal. I'd like to know why, as you say,  runner.wait(TIMEOUT) doesn't take affect. Under normal conditions, the runner should be signaled immediately, and not wait.