Cannot quick start with Ionic UI (node_modules error)

I'm submitting a ...  (check one with "x")
[x] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request

Current behavior
When going through the ngx-formly Ionic UI quick start I am stuck on step 3 (Import the FormlyModule and UI (pre-defined types/templates)). When adding FormlyIonicModule an error is thrown:
ERROR in<EMAIL_ADDRESS>Module not found: Error: Can't resolve 'ionic-angular' in '/Users/myUser/Projects/ionicTest/node_modules/@ngx-formly/ionic/fesm5'
｢wdm｣: Failed to compile.
Expected behavior
This error is a blocker and is not allowing me to continue with the initialization of the Ionic UI formly quick start.
Minimal reproduction of the problem with instructions

I started a new project with the Angular CLI (7.3.0) quick start:
ng new my-app
From there I followed the quick start instruction of ngx-formly ionic UI:
npm install @angular/forms @ngx-formly/core --save
npm install @ngx-formly/ionic --save
I then followed the instructions of importing the FormlyModule and UI. I ensured my module ts file had these imports:
import {NgModule} from '@angular/core';
import {ReactiveFormsModule} from '@angular/forms';
import {FormlyModule} from '@ngx-formly/core';
import {FormlyBootstrapModule} from '@ngx-formly/bootstrap';
import {FormlyIonicModule} from '@ngx-formly/ionic';
@NgModule({
imports: [
...,
ReactiveFormsModule,
FormlyModule.forRoot(),
FormlyBootstrapModule,
FormlyIonicModule
]
})
I then saw an error appear in my terminal:
ERROR in<EMAIL_ADDRESS>Module not found: Error: Can't resolve 'ionic-angular' in '/Users/myUser/Projects/ionicTest/node_modules/@ngx-formly/ionic/fesm5'
｢wdm｣: Failed to compile.
This stops me from proceeding with initializing the project. The line "FormlyIonicModule" seems to be issue because when adding it is when the error appears.

Please tell us about your environment:
MacOS High Sierra operating system Version 10.13.6
Visual Studio Code IDE Version 1.30.2

Angular version:
Angular 7.2.3
Angular CLI 7.3.0

Browser:
Google Chrome Version 63.0.3239.108 (Official Build)

Language:
typescript 3.2.4

Node (for AoT issues):
Node 10.15.1

ionic should be installed too by following their doc.
for the newest ionic v4, you may need to use our v5.rc version

Going to my package.json and changing the "@ngx-formly/core": & "@ngx-formly/ionic" from "^4.8.2" to "^5.0.0-rc.11" got rid of the error.
I think an adjustment to the installation steps should be made so that it can run smoothly or new users will have the same issue I had.
Thanks

@hoamar
I was also facing the similar issue. The following versions solved my problem
"@ionic/angular": "^4.0.2",
"@ngx-formly/core": "^5.0.0-rc.11",
"@ngx-formly/ionic": "^5.0.0-rc.11"
@aitboudad I think we need to have an updated document, which is very much challenging for the constantly evolving library :) Also, the latest version in the npm is 4.8.2. Once, the rc version is finalized and stable and published in the npm, the problem should be resolved.

the stable version should be released this weekend.

which is very much challenging for the constantly evolving library :)

no worry about this :), after V6 we'll have enough time to focus on the doc and website (not our hight priority for now).

@aitboudad Thank you.


Large number of resource groups that begin with radtest- in Azure subscription

Area for Improvement
There are ~231 resource groups in Radius Dev subscription which begin with radtest-. These would be created during test pipeline runs but they are either not being deleted after the run completes or they are not picked up by purge-test-resources.yaml later and deleted.
Radius Test subscription has a fewer number ~24 resource groups.

Is the purge-test-resources.yaml is only running against Radius Test subscription? Pls review workflow(s) which should delete resource groups upon test runs and check if working as expected.
Delete older resource groups which have names that start with radtest-

Observed behavior
No response
Desired behavior
Resource groups are deleted after test run completion.
purge-test-resources.yaml is running as expected.
Proposed Fix
No response
rad Version
NA
Operating system
NA
Additional context
NA
Would you like to support us?

[ ] Yes, I would like to support you

AB#12488

I created a script that deletes all the resource groups that start with radtest- and have 0 resources. It is running now, and it should finish in less than an hour. After that we can check if we will still see these resource groups get built up.

There are 17 resource groups, within two different subscriptions, that have at least one resource. I checked all these resource groups, and all the resources are storage accounts. There may be a pattern here like storage account not being deleted, functional test failing, and resource group becoming dangling as well as the storage account.

After my PR number or resource groups went up to 25. After the functional tests of the same PR finished running, this number didn't go down. So, yes, we are definitely not deleting resource groups after running functional tests.
Please review the PR linked above.

@ytimocin  Radius Dev still has around 11 resource groups starting with 'radtest'. They seem be created roughly between March8-11 and have a identity and storage account resource. Checking, if they should also be deleted.

@lakshmimsft @ytimocin thanks for catching and fixing this. I'm curious how did we end up with radtest resource groups in Radius Dev, they should only be getting created under test subscription did we find out where did they come from?

@kachawla not exactly sure. Functional tests create resource groups with the prefix of radtest-func not just radtest. Maybe the ones that start with radtest were just tests or previous versions of our functional tests.

@kachawla not exactly sure. Functional tests create resource groups with the prefix of radtest-func not just radtest. Maybe the ones that start with radtest were just tests or previous versions of our functional tests.

@ytimocin I meant they should not be getting creating in the Dev subscription. Just checked and noticed a bunch of new radtest-***-shared resource groups have been created in this week as well. I'm going to log an issue to investigating what's creating them in the dev subscription. All functional tests should target test subscription.

@kachawla not exactly sure. Functional tests create resource groups with the prefix of radtest-func not just radtest. Maybe the ones that start with radtest were just tests or previous versions of our functional tests.

@ytimocin I meant they should not be getting creating in the Dev subscription. Just checked and noticed a bunch of new radtest-***-shared resource groups have been created in this week as well. I'm going to log an issue to investigate what's creating them in the dev subscription. All functional tests should target test subscription.

Interesting. Do you see more than 11 like other than the ones below? I didn't delete these 11 resource groups on purpose because they have resources inside. If you see new resource groups that were created this week, we should definitely create an issue to investigate.
I also couldn't find the creation dates for these resource groups on the portal. Do you know how we can see that? That would be helpful to me.

@kachawla not exactly sure. Functional tests create resource groups with the prefix of radtest-func not just radtest. Maybe the ones that start with radtest were just tests or previous versions of our functional tests.

@ytimocin I meant they should not be getting creating in the Dev subscription. Just checked and noticed a bunch of new radtest-***-shared resource groups have been created in this week as well. I'm going to log an issue to investigate what's creating them in the dev subscription. All functional tests should target test subscription.

Interesting. Do you see more than 11 like other than the ones below? I didn't delete these 11 resource groups on purpose because they have resources inside. If you see new resource groups that were created this week, we should definitely create an issue to investigate.

You are right, these are the only ones I see and all of them were created 3 months ago. I was making a mistake when converting epoch timestamp which made me think they were created this week.

I also couldn't find the creation dates for these resource groups on the portal. Do you know how we can see that? That would be helpful to me.

We add a creationTime tag on test resource groups, you can find them in tags in resource group overview page on the portal.


Do not access 'Errors' if there are no current errors

Related to #74. Fixes access to the 'Errors' key in errs if errs is empty.
This PR needs to be verified.

looks safe enough - thanks.


Crimson isles constants

Add crimson isles constants to src/constants?
ill add it tomorrow.
Tell me if it should be in the other constants location.
It will have the Dojo scores and the belt points, and other stuff.

what constants do you mean?


chore: upgrade deps, fix broken event links

Can we verify the build error? After this I can approve it

@sammartinez we not use GitHub actions to CI testing. Please ignore the CircleCI check failure. We should remove that. In fact, give me the green light, and I'll do so right now.


Add meta for banners and shields

This PR adds a meta class, BannerMeta, to be used on banners and shields. Both items use the same meta.
Feedback required: DyeColor#patternColorId has been added instead of using DyeColor#ordinal, should this be reverted?

It looks like you manually edited the DyeColor enum instead of changing DyeColorGenerator (as it says   * AUTOGENERATED by DyeColorGenerator at the top)

I didn't even see that, my bad.

Requires Minestom/MinestomDataGenerator#12 to continue.


Documentation improvement: Setting toolchain in osx for armgcc and other stuff

This is a documentation issue.
Can you please explain how do I set the toolchain in osx (you only mentioned for windows).
I'm using arm-none-eabi-gcc
Also, what is the proper target name for nrf51dk development board?
Thank you!

@shayo The methods described in the README should work on OSX as well as Windows. We used Windows paths in the README but it should work with any type of path.
Also, if arm-none-eabi-gcc is in your shell's PATH, then it should be automatically detected with no need for any configuration!

Also, what is the proper target name for nrf51dk development board?

NRF51_DK :)

Gotcha. I managed to run the compile. Now I get a crash. I'll open another issue.


fix(text-spacing): WIP creating default font size

Attempts to create a default font-size that is applied to the Light and Dark themes that are applied to the entire MaterialApp.
Replace this paragraph with a description of what this PR is changing or adding, and why. Consider including before/after screenshots.
List which issues are fixed by this PR. You must list at least one issue. An issue is not required if the PR fixes something trivial like a typo.
If you had to change anything in the flutter/tests repo, include a link to the migration guide as per the breaking change policy.
Pre-launch Checklist

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

If you need help, consider asking for advice on the #hackers-new channel on Discord.

This fix is incomplete. I'm closing this as I'm no longer attempting to continue this PR.


when i  use the tridentnet, the rpnl1 loss will be a large number and the rcnnl1loss will be nan,how can i solve this problem?

when i use tridentnet_r101v2c4_c5_addminival_2x with my own dataset.
The RPNL1LOSS and rcnnl1loss is normal at first, but after about 3000 iterarion the rcnnL1loss will be nan and RPNl1loss will be a large number.
I use the code with only one GPU ,
and i donnot change other config unless the gpus in KVstoreParam

Without any details about your own dataset, the best guess should be that the learning rate is too large. Try to decease your lr.

thanks a lot.


Remove duplicate test - closes #93

Current coverage is 100% (diff: 100%)

Merging #96 into master will not change coverage

@@           master   #96   diff @@
===================================
  Files           7     7
  Lines         138   138
  Methods         0     0
  Messages        0     0
  Branches        0     0
===================================
  Hits          138   138
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update 64bfa7a...27a4db4


[vRPex] Skinshop delay

Long delay (~8s) of opening skinshop menu because vRPclient.getDrawableTextures

vRPex has a better way to write Tunnel/Proxy calls, but it requires more thinking to avoid this kind of behavior (calls are waiting the result/execution by default).
Should decrease latency: https://github.com/ImagicTheCat/vRP/commit/b0218861daf7fcadba2c66b8fb119e54a870f582


Migrate from tokio_core to tokio.

I will change the dependency path back to the original location once https://github.com/NikVolf/tokio-named-pipes/pull/1 is merged.

No problem :)

appveyor failed ^^

Hmm, I'll have a look...

Oh... recent changes added new windows stuff... I'll take care of it.

Well no, last time I checked, now I don't have windows

You see, this build is ok:
https://ci.appveyor.com/project/NikolayVolf/parity-tokio-ipc/build/1.0.3
I'll run it few times to see if there are non-deterministic failures

So far every b4a6cdf7 commit build is a success...
https://ci.appveyor.com/project/NikolayVolf/parity-tokio-ipc/history

Interesting... I'll see what I can figure out.

I've temporarily added a timeout to ipc pipe creation. Let me know if I should leave it in or not.
The intermittent stalling / failures must be a result of having multiple threads attempting to access the Windows file handles at the same time (now that everything runs on a multi-threaded runtime). I think what I'm going to have to do is add back the handle argument to tokio_named_pipes::NamedPipe::new, ::from_pipe, etc. and also to parity_tokio_ipc::IpcConnection::connect. That way, a single reactor handle can be used to create them in a single-threaded fashion once again.

Using a reactor handle for NamedPipe creation did the trick. Apparently this is important for coordination with tokio::reactor::PollEvented2 on Windows due to the single-threadyness needed.

cool, just a couple of minor issues

Will update dependency as soon as https://github.com/NikVolf/tokio-named-pipes/pull/2 is merged.

Also instead of creating a stable branch, I'd create a separate branch for each version.

@NikVolf I assume you're fine with taking over the tokio-named-pipes crate, yeah? I'll help maintain if needed.

Aye, and I don’t see any problems doing it in current gh state


fix: retrieve configuration resources by labels

Description
This PR fixes a bug in Liqoctl where Configuration resources were retrieved by name instead of labels.
It also makes private the functions that forge default names on Kubeconfig, Nonce and SignedNonce resources.

/rebase test=true

/merge


AttributeError: 'NoneType' object has no attribute 'active_view'

I just saw this on my console, after I opened a new window, but I am not sure on how to reproduce:
Traceback (most recent call last):
  File "D:\SublimeText\sublime_plugin.py", line 513, in on_selection_modified_async
    callback.on_selection_modified_async(v)
  File "D:\SublimeText\Data\Packages\SublimeLinter\sublimelinter.py", line 394, in on_selection_modified_async
    view = self.get_focused_view_id(view)
  File "D:\SublimeText\Data\Packages\SublimeLinter\sublimelinter.py", line 382, in get_focused_view_id
    active_view = view.window().active_view()
AttributeError: 'NoneType' object has no attribute 'active_view'
Could not determine project file as view does not seem to have an associated window.

Lines:
File: SublimeLinter/sublimelinter.py
373:     def get_focused_view_id(self, view):
374:         """
375:         Return the focused view which shares view's buffer.
376:
377:         When updating the status, we want to make sure we get
378:         the selection of the focused view, since multiple views
379:         into the same buffer may be open.
380:
381:         """
382:         active_view = view.window().active_view()
383:
384:         for view in view.window().views():
385:             if view == active_view:
386:                 return view

Related:

https://github.com/SublimeLinter/SublimeLinter3/issues/186 AttributeError since build 3069

I had the same error but in different location
Traceback (most recent call last):
  File "/Applications/Sublime Text.app/Contents/MacOS/sublime_plugin.py", line 465, in on_pre_save_async
    callback.on_pre_save_async(v)
  File "/Users/xxx/Library/Application Support/Sublime Text 3/Packages/SublimeLinter/sublimelinter.py", line 470, in on_pre_save_async
    if view.window().active_view() == view and self.is_settings_file(view):
AttributeError: 'NoneType' object has no attribute 'active_view'

This may just be an asynchronisity issue that pops ups in weird situations. I'm not sure if we should guard against that, but it should be easy enough, e.g.: https://github.com/SublimeLinter/SublimeLinter3/pull/611/files#diff-e4e8a30eb88a08e1933dc3a042bde8b2R350
@groteworld I don't get this error and you weren't able to reproduce in #186 either. Any ideas/opinions?

We got a guard in place for this now


ARROW-9291 [R]: Support fixed size binary/list types

some progress here:
library(arrow, warn.conflicts = FALSE)

a <- Array$create(vctrs::list_of(as.raw(1:4), as.raw(1:4)), type = fixed_size_binary(4L))
a
#> Array
#> <fixed_size_binary[4]>
#> [
#>   01020304,
#>   01020304
#> ]
v <- a$as_vector()
v
#> <fixed_size_binary<4>[2]>
#> [[1]]
#> [1] 01 02 03 04
#>
#> [[2]]
#> [1] 01 02 03 04
str(v)
#> fixed_size_binary<4> [1:2]
#> $ : raw [1:4] 01 02 03 04
#> $ : raw [1:4] 01 02 03 04

# we can skip the type= we have an R object that inherits from vctrs_list_of,
# with class arrow_fixed_size_binary and a byte_width attribute
raws <- vctrs::new_list_of(
  list(as.raw(1:4), as.raw(1:4)),
  ptype = raw(),
  byte_width = 4L,
  class = "arrow_fixed_size_binary"
)
a <- Array$create(raws)
a
#> Array
#> <fixed_size_binary[4]>
#> [
#>   01020304,
#>   01020304
#> ]
v <- a$as_vector()
v
#> <fixed_size_binary<4>[2]>
#> [[1]]
#> [1] 01 02 03 04
#>
#> [[2]]
#> [1] 01 02 03 04
str(v)
#> fixed_size_binary<4> [1:2]
#> $ : raw [1:4] 01 02 03 04
#> $ : raw [1:4] 01 02 03 04

Created on 2020-07-07 by the reprex package (v<IP_ADDRESS>01)

I think we need a function for the raws <- vctrs::new_list_of(...) part, and probably an equivalent function for the other binary types, but binary(), large_binary() and fixed_size_binary() which would be the obvious choice is already taken.
I think that in this function:
template <>
std::shared_ptr<arrow::DataType> InferArrowTypeFromVector<VECSXP>(SEXP x) {
  if (Rf_inherits(x, "data.frame") || Rf_inherits(x, "POSIXlt")) {
    return InferArrowTypeFromDataFrame(x);
  } else {
    if (Rf_inherits(x, "arrow_fixed_size_binary")) {
      SEXP byte_width = Rf_getAttrib(x, symbols::byte_width);
      if (Rf_isNull(byte_width) || TYPEOF(byte_width) != INTSXP ||
          XLENGTH(byte_width) != 1) {
        Rcpp::stop("malformed arrow_fixed_size_binary object");
      }
      return arrow::fixed_size_binary(INTEGER(byte_width)[0]);
    }

    SEXP ptype = Rf_getAttrib(x, symbols::ptype);
    if (Rf_isNull(ptype)) {
      if (XLENGTH(x) == 0) {
        Rcpp::stop(
            "Requires at least one element to infer the values' type of a list vector");
      }

      ptype = VECTOR_ELT(x, 0);
    }

    // special case list(raw()) -> BinaryArray
    if (TYPEOF(ptype) == RAWSXP) {
      return arrow::binary();
    }

    return arrow::list(InferArrowType(ptype));
  }
}

we should rather dispatch based on an r type for binary and large binary, as it's done here for fixed size binary, and forget the magic conversion of a vctrs_list_of<raw> to a BinaryArray, as we might actually want to create an Array of type List<uint8>:
arrow::list_of(arrow::uint8())
#> ListType
#> list<item: uint8>

Created on 2020-07-07 by the reprex package (v<IP_ADDRESS>01)
but type deduction currently short circuits it.

Let me see if I understand the issues you're concerned with. We have some inconsistency in how types are converted: raw -> uint8, but uint8 -> integer, for example. Here, binary -> list_of(raw), and we've special-cased list_of(raw) -> binary, but you might think that it should be list_of(uint8) given how the non-list type is converted.
I'm fine with getting rid of the special-casing of list_of(raw) to binary and having the binary conversion set an R class (subclass of vctrs list of or whatever), which we use to determine that the list of raw should be converted back to BinaryArray. I'm not particularly worried that someone would have a list of raw vectors in R and be frustrated that that wasn't converted to a list of uint8--seems unlikely, and if you really care, you can always specify a type/schema. But it seems reasonable to be explicit and set a class attribute.

Some more progress today.
library(arrow, warn.conflicts = FALSE)

# with explicit type=
#   no deduction, but testing
raws <- vctrs::list_of(as.raw(1:4), as.raw(1:4))
a1 <- Array$create(raws, type = binary())
a2 <- Array$create(raws, type = large_binary())
a3 <- Array$create(raws, type = fixed_size_binary(4L))
a1$as_vector()
#> <binary[2]>
#> [[1]]
#> [1] 01 02 03 04
#>
#> [[2]]
#> [1] 01 02 03 04
a2$as_vector()
#> <large_binary[2]>
#> [[1]]
#> [1] 01 02 03 04
#>
#> [[2]]
#> [1] 01 02 03 04
a3$as_vector()
#> <fixed_size_binary<4>[2]>
#> [[1]]
#> [1] 01 02 03 04
#>
#> [[2]]
#> [1] 01 02 03 04

# with type deduced from class, we should have functions for this.

a1 <- Array$create(
  vctrs::new_list_of(list(as.raw(1:4), as.raw(1:4)), class = "arrow_binary")
)
a2 <- Array$create(
  vctrs::new_list_of(list(as.raw(1:4), as.raw(1:4)), class = "arrow_large_binary")
)
a3 <- Array$create(
  vctrs::new_list_of(list(as.raw(1:4), as.raw(1:4)), class = "arrow_fixed_size_binary", byte_width = 4L)
)
a1$as_vector()
#> <binary[2]>
#> [[1]]
#> [1] 01 02 03 04
#>
#> [[2]]
#> [1] 01 02 03 04
a2$as_vector()
#> <large_binary[2]>
#> [[1]]
#> [1] 01 02 03 04
#>
#> [[2]]
#> [1] 01 02 03 04
a3$as_vector()
#> <fixed_size_binary<4>[2]>
#> [[1]]
#> [1] 01 02 03 04
#>
#> [[2]]
#> [1] 01 02 03 04

Created on 2020-07-08 by the reprex package (v<IP_ADDRESS>01)
Will tackle FixedSizeList on the same model.
Now that we have the R classes arrow_binary, arrow_large_binary, and arrow_fixed_size_binary, perhaps we don't really need to inherit from vctrs_list_of, I actually had to backpedal it with some methods to make it look sort of ok:
# vctrs support -----------------------------------------------------------

#' @importFrom vctrs vec_ptype_full vec_ptype_abbr obj_str_footer
#' @method vec_ptype_full arrow_fixed_size_binary
#' @export
vec_ptype_full.arrow_fixed_size_binary <- function(x, ...) {
  paste0("fixed_size_binary<", attr(x, "byte_width"), ">")
}

#' @method vec_ptype_full arrow_binary
#' @export
vec_ptype_full.arrow_binary <- function(x, ...) {
  "binary"
}

#' @method vec_ptype_full arrow_large_binary
#' @export
vec_ptype_full.arrow_large_binary <- function(x, ...) {
  "large_binary"
}

#' @method vec_ptype_abbr arrow_fixed_size_binary
#' @export
vec_ptype_abbr.arrow_fixed_size_binary <- vec_ptype_full.arrow_fixed_size_binary

#' @method vec_ptype_abbr arrow_binary
#' @export
vec_ptype_abbr.arrow_binary <- vec_ptype_full.arrow_binary

#' @method vec_ptype_abbr arrow_large_binary
#' @export
vec_ptype_abbr.arrow_large_binary <- vec_ptype_full.arrow_large_binary

#' @method obj_str_footer arrow_fixed_size_binary
#' @export
obj_str_footer.arrow_fixed_size_binary <- function(x, ..., indent.str = "", nest.lev = 0) {
  invisible(NULL)
}

#' @method obj_str_footer arrow_binary
#' @export
obj_str_footer.arrow_binary <- obj_str_footer.arrow_fixed_size_binary

#' @method obj_str_footer arrow_large_binary
#' @export
obj_str_footer.arrow_large_binary <- obj_str_footer.arrow_fixed_size_binary

So now either :

we have a compatible list, i.e. a list of raw vectors and we request type = binary/large_binary/fixed_size_binary.
we have R objects (that still have to be list of raw vectors) with these dedicated classes

In any case the conversion from arrow to R makes these classes regardless.

Further progress re FixedSizeList:
library(arrow, warn.conflicts = FALSE)

a <- Array$create(list(1:4), type = fixed_size_list_of(int32(), 4L))
a
#> Array
#> <fixed_size_list<item: int32>[4]>
#> [
#>   [
#>     1,
#>     2,
#>     3,
#>     4
#>   ]
#> ]

# back to R
a$as_vector()
#> <fixed_size_list<integer, 4>[1]>
#> [[1]]
#> [1] 1 2 3 4
str(a$as_vector())
#> fixed_size_list<integer, 4> [1:1]
#> $ : int [1:4] 1 2 3 4
#> @ list_size: int 4
#> @ ptype    : int(0)

Created on 2020-07-09 by the reprex package (v<IP_ADDRESS>01)

Looks like this conflicts with the Dictionary PR I just merged

Should be better now 🤞🏾


Use @actions/core.exportVariable instead of set-env

set-env is now deprecated. @actions/core exposes exportVariable
to be used instead.
https://github.com/actions/toolkit/tree/main/packages/core#exporting-variables
I guess the large diffs in .js is due to UNIX vs Windows line endings.
Cherry-picked from #19

Have you tested with your branch in your workflows?

This actually doesn't work using @master right now. I will investigate.

@master doesn't work because they don't have node_modules in them. You'll have to try using stigok/k8s-set-context@releases/v1 for testing

@stigok I tested it with your release work. I still see the warning in logs. Am I missing something?

@shigupt202 I suppose the actions/core lib is not updated here yet

This is https://github.com/Azure/k8s-set-context/pull/15 still open

So this looks okay? Tested the action using stigok/k8s-set-context@releases/v1 with success, however, as @ds-ms says, the warning still appears.
As you say, the only place set-env is still present is at node_modules/@actions/core/lib/core.js:38

Let's fix #15 and then you can rebase and validate. 😃
Or if you're willing, you could update the package version in your branch and run npm install --production locally. And check in the newly added node_modules to this PR.

I'll wait for #15 :)

Is this correct? 8d92b0b

This looks good to me.

I think it's missing a few refs. This upgrade should have had @babel refs in the package-lock.json file, did you amend it?

I simply set @actions/core": "^1.2.6",  in package.json and ran npm install --production. Then added a new commit.
There seems to be some line-ending changes since I'm on Linux. This might be an issue for diffs.

@ds-ms I tested the changes and it's working fine. For some reason @babel references are not present in the packag-lock.json, but I think we can go ahead with these changes as it's removing the warning.

@shigupt202 do you want to pick up the port of the package.json changes to master?

package.json changes are already there in master. But we need to port the other changes done in this PR to master, i.e., replacing set-env with exportVariable. I'll do it.

This change is already present in master because of https://github.com/Azure/k8s-set-context/pull/18

@shigupt202 no it's not. The package-lock has it at 1.1.0 in master

@stigok I think it's 1.2.6. See this


Patch version to 0.1.3

@JuliaRegistrator register branch=main

Registration pull request created: JuliaRegistries/General/92861
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.3 -m "<description of version>" 07473bd6e3a828d3bdc8f09683ab60428fd014f4
git push origin v0.1.3


Access Denied.

Hi Guys.
I am trying to use RDPWrap. I have install the latest version of the RDPWrap and i have updated my ini file to support Windows version 10.0.17763.292.
Everything is green when i run the RDPConf.exe. However when i run RDPCheck.exe and put in the credentials for my account i am faced with an access denied message.
Any ideas?
Thanks in advance.

Search for "Access Denied" in the README.


Add styling to new directory name Input field

resloves #999

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

Thanks! I'm going to close this in favour of #1001, as pointed out in https://github.com/livebook-dev/livebook/issues/999#issuecomment-1038288648 :)

Okay thanks


Please add more preset and plugin examples

I have seen there are now 2017 and including -minimal and -latest presets available.
The minimal uses feature detection to only transpile for features not supported for the Node.js version being used!
Also, what about plugins for advanced features such as (legacy) decorators and async/await?
Good job! Looking forward to more tutorials and general tutorial improvements!
How about code coverage!?

Hey I'm gonna address both your tickets here. I'm currently too busy to update the tutorial, and by no means do I intend for it to be comprehensive. The focus is on simplicity instead of abstractions or extra plugins/presets.
In the late future, I will definitely release a "second edition" that updates the outdated information, but nothing more than improvements to clarity as the original reason I created the tutorial was for it to be focused and small. I feel like adding more content or talking about DRYness is unnecessary complication that shouldn't go into the main tutorial, but maybe an "extras" type thing at the end.
I'll definitely take your suggestions into consideration, but for the time being...I just am too busy for this tutorial to be a priority. Sorry 😕 .


Utilize C# semantic token edits handler

First and foremost, I'm not sure if we should take this PR since it's unclear if it's actually helping performance. Open to feedback and ideas on how to further optimize things here!
The PR adds logic so we now utilize Roslyn's semantic tokens edits handler, with an aim of improving overall performance. Pros and cons are in the section below.
Preliminary manual testing (via stopwatch) seems to suggest Razor is now performing faster when inserting into a large document, however visually I feel like it's a tiny bit slower. One idea is maybe the LSP side is taking longer to process and apply edits. When typing a character into a large document of about 1000 lines with 3-4 tokens per line, using the non-edits approach took about 1.74s on average for Razor to process, while the edits approach was averaging around 1.5s. Again, this is just on the Razor end, LSP has their own logic that might actually be making things slower (will try to figure out a way to test tomorrow to see if this is the case). Or maybe there's a bug somewhere in the code I wrote / something that could be further optimized. 😄

Benefits to using Roslyn's semantic tokens edit handler:

Using the C# edits handler should reduce the amount of data Roslyn sends us by a ton - instead of being potentially returned tens of thousands of tokens when a user types a character, we should now just be returned maybe a dozen or so.

Drawbacks to using Roslyn's semantic tokens edit handler:

Roslyn on their end still has to compute the full document tokens regardless, and on top of that now has to compute edits.
We have to apply the edits ourselves to the cached C# generated document. I tried to minimize the impact of this by only passing through all the tokens once when applying the edits, but this still could potentially be costly. Since we're just doing a single pass, I opted to use a List which I think  is performant enough, but if anyone has any alternate data structure ideas, definitely let me know.

TO-DO:
Add tests - I didn't want to spend time adding any if we weren't going to take this PR, but I plan on adding some later if we decide to proceed.

EDIT: Results from additional testing (via the super scientific Stopwatch approach 😆 ):

SCENARIO
ORIGINAL (ms avg)
EDITS HANDLER (ms avg)

Remove a line in   the middle of the file (x.ToString();)
292
240.8

Pasting a line in   the middle of the file (x.ToString();)
197.25
171.75

Whitespace   insertion (new line) in the middle of file
349.2
239.6

SCENARIO
ORIGINAL (ms avg)
EDITS HANDLER (ms avg)

Remove a line in   the middle of the file (x.ToString();)
4488.6
4214.6

Pasting a line in   the middle of the file (x.ToString();)
5067.8
4809.6

Whitespace   insertion (new line) in the middle of file
5404.6
4073.4

METHODOLOGY: Each scenario was tested 5 times. The stopwatch was started at the beginning of Razor's edit handler, and was stopped right before the the edit handler returned.
CONCLUSION: Based on these results (keep in mind they're being done with Stopwatch, so take them with a grain of salt), it seems like it may be slightly beneficial to utilize C#'s edit handler, although in most cases the difference in performance is very subtle. The overall benefit is most apparent in a very large file and will likely be unnoticeable in smaller files.
--
Perfview results:

How did it perform on the experimental instance? Definitely want to give that a good workout when dealing with this kind of interlaced services stuff.

We would also definitely want numbers (even if it's just Stopwatch) before we took this as it does complicate things, so we wanna be sure we're getting value for it.

Results from testing (via the super scientific Stopwatch approach 😆 ):

SCENARIO
ORIGINAL (ms avg)
EDITS HANDLER (ms avg)

Remove a line in   the middle of the file (x.ToString();)
292
240.8

Pasting a line in   the middle of the file (x.ToString();)
197.25
171.75

Whitespace   insertion (new line) in the middle of file
349.2
239.6

SCENARIO
ORIGINAL (ms avg)
EDITS HANDLER (ms avg)

Remove a line in   the middle of the file (x.ToString();)
4488.6
4214.6

Pasting a line in   the middle of the file (x.ToString();)
5067.8
4809.6

Whitespace   insertion (new line) in the middle of file
5404.6
4073.4

METHODOLOGY: Each scenario was tested 5 times. The stopwatch was started at the beginning of Razor's edit handler, and was stopped right before the the edit handler returned.
CONCLUSION: Based on these results (keep in mind they're being done with Stopwatch, so take them with a grain of salt), it seems like it may be slightly beneficial to utilize C#'s edit handler, although in most cases the difference in performance is very subtle. The overall benefit is most apparent in a very large file and will likely be unnoticeable in smaller files.

Looks like GetSemanticTokens_CSharp_Nested_HTML failed. The only way this should have changed the output of those tests (or any Semantic Tokens) is if the previous output was in the wrong (seems unlikely at this stage but not impossible). Let me know if you want an extra eye to look over those baseline tests, I know they're not the easiest things to follow.

I tried out @NTaylorMullen's approach to capturing PerfViews and it looks like using the edit handler is indeed faster. 😄
The below results are from an edit in a jumbo file (~12k tokens) and pasting a line in the middle of the file. The numbers indicate the total amount of time taken between start/stop:

C# is faster using the old approach, which is expected. But Razor overall using the edits handler appears to be faster.
Next steps before merging will be to add tests and investigate/fix up the failing test. 👍

C# is faster using the old approach, which is expected. But Razor overall using the edits handler appears to be faster.
Next steps before merging will be to add tests and investigate/fix up the failing test. 👍

Holllllllyy 4x faster!! 👏 . Sick gainz

Update: Thank goodness for testing - when I started writing and running tests, I discovered the logic I wrote to apply C# edits onto the old token set is incorrect. 😬  Exploring fixes right now.

Update: Thank goodness for testing - when I started writing and running tests, I discovered the logic I wrote to apply C# edits onto the old token set is incorrect. 😬 Exploring fixes right now.

oo awesome you caught it early! 👏

Unfortunately I ended up switching the application of C# edits back to using Insert/Remove range. ☹ there was an edge case I discovered with the initial implementation where we weren't applying an edit if it began at a later index than the original token set array length.
I brainstormed alternatives but nothing seemed to stick as well as Insert/Remove range. The positive news is I PerfView tested the updated implementation using the same methodology as above, and the Insert/Remove range still seems to be super fast compared to the original with a large file. The Razor Language Server Client had a total outbound time of 733ms (compared to the original 3693ms).
Does anyone have thoughts on or concerns with the updated approach? I also added tests so hopefully things will be good to go soon (besides future feedback on the PR).


Access denied to fetch pretrained model

Hi, I want ton replicate the results on google colab using this notebook using the default settings in script_inference.sh but this is url is throwing 403 error
https://github.com/kwonminki/Asyrp_official/blob/a6c82dc760849dd4c1e878990ce38e824967e5b0/diffusion_latent.py#L84-L85

It seems that SDEdit no longer provides checkpoints.
Please check it for CelebA-HQ model.
https://github.com/curryjung/DiffStyle_official/issues/1

https://github.com/ermongroup/SDEdit/issues/23#issuecomment-1624585137


Order nodes arrangement by data value

Is it possible to rearrange the nodes by a data value?
Ex. for a data structure:
{     "name": "flare",     "children": [         {             "name": "analytics",             "children": [                 {                     "name": "cluster",                     "children": [                         {                             "name": "AgglomerativeCluster",                             "value": 3938                         },                         {                             "name": "CommunityStructure",                             "value": 3812                         },                         {                             "name": "HierarchicalCluster",                             "value": 6714                         },                         {                             "name": "MergeEdge",                             "value": 743                         }                     ]                 },
Now I want to arrange the nodes by the "value"-attribute in ascending order, so that the nodes with the lowest value is shown left and the node with the highest value is shown right (in a horizontal layout, left to right).
Thank you!

Check this - https://github.com/bumbeishvili/org-chart/issues/104

Sorry but I'm a little bit confused ...
Here my code:
`
sortNode(nodeId) {
    const attrs = this.getChartState();
    const node = attrs.allNodes.filter(({ data }) => attrs.nodeId(data) == nodeId)[0];
    if (!node) {
        console.log(`ORG CHART - SORT - Node with id "${nodeId}" not found in the tree`);
        return this;
    }

    // sort node childs
    const sortedNode = node.sort((a, b) => d3.ascending(a.data.orgData.orderNo, b.data.orgData.orderNo));
    attrs.allNodes.map(node => {
        if (node.id == nodeId) {
            node = sortedNode
        }
        return node
    })

//so here the node order is correct, but I don't know how to order the associated data so the chart represents the new order
`
Thank you for your hints.

OK I sorted the data array before generating the chart. That's ok for me. Thank you.


Certificate thumbprint while deploying

Hi, is there any documentation or resources on how to deploy Psi bot to any cloud services. Specifically, how do we dynamically make a container , set up certificates and use the certificate thumbprint.
Locally, we do it manually by adding it to the local machine, but how do we actually do this in cloud deployment

You could build a container that supports ACME challenges on it's own. You could have a reverse proxy that does it for you, like nginx with acme companion. Storing the certificates could be done using Azure Key Vault, volumes, managed disks or similar. If you are running on AKS, you can have a cert bot doing ACME and storing the certificate in the etcd storage.
There are many ways to receive, store and consume the certificates and many ways to run the samples and it all depends on what kind of compute offering you want to run the bot on. The handling of certificates in that sense is not special for running app hosted media bots. It has more to do with best practices surrounding cert handling for the target IaaS / PaaS offering.

If you are looking for consulting in that area, you can look up my contact details in my profile, btw.

Hey thanks for the tips, will look into it

Also, a minor note: the most recent media SDK (<IP_ADDRESS>-alpha) allows the application to provide its server certificate programmatically as an X509Certificate2 object. There is a new MediaPlatformSettings.MediaPlatformInstanceSettings.Certificate API. This may be more convenient than installing the certificate in the local machine cert store.

Thank you everyone !


🪐  JupyterLab: Hidden Files

This wasn't as simple as first thought, as per this guidance, enable hidden files on the JupyterLab server.

Add   --ContentsManager.allow_hidden="True" to entrypoint.sh
Enable via View Menu.

PR merged.

Documented in this PR.


Greenkeeper/@types/node 8.0.40

Coverage remained the same at 73.962% when pulling 5d3a774927ff02ae724e5757c9dae138b2459001 on<EMAIL_ADDRESS>into 21066acd6ff04ce92b80935bdd2192733079d33f on master.

Coverage remained the same at 73.962% when pulling 6720255b86a87c1796317debfbb2d743fe4105ea on<EMAIL_ADDRESS>into 91ce4cbb8c4abb82ca8df57c7b8b9985cf2db482 on master.


feat: Make the Title field searchable

:tada: This PR is included in version 2.0.0-next.20 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Jenayah: Watch yeuphim.tv

Jenayah requests the watch of the watch_keyword yeuphim\.tv. See the MS search here and the Stack Exchange search here.

yeuphim\.tv has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by tripleee in Charcoal HQ


Add dynamodb permissions to the batch service role

The batch service role was missing permissions to access the dynamodb table used by step functions: https://github.com/Netflix/metaflow-tools/blob/master/aws/cloudformation/metaflow-cfn-template.yml#L1066
This PR adds those permissions if step functions are enabled.
gitter convo: https://gitter.im/metaflow_org/community?at=607df813a2ac0d38e7c7ad0c

This all looks good to me! Great catch
Also thanks for linking the Gitter Convo, that provided a bunch of context. For historical reasons, I'm going to copy the content below.

kelly: Question about step function IAM permissions. We have a step that is doing a next() with a foreach param, and when it runs as a step function, we get an error that the aws batch execution role doesn't have permission to call PutItem to the step functions dynamo table. This is in the step_functions_decoratory.py in task_finished(), so it makes sense to me that the batch execution role would need the dynamo permissions, but when I look at the cloudformation template in metaflow-tools, it doesn't seem to be granting those permissions. Can someone confirm that the batch execution role does need permissions to the step function dynamodb table?

savin: @kldavis4 there are two kinds of roles in play for AWS Batch execution - one role to be able to launch the AWS Batch job in the first place and another one for the launched job to be able to chat with other AWS services (S3, DynamoDb etc.). The second role does indeed require PutItem permission for DynamoDb - https://github.com/Netflix/metaflow-tools/blob/master/aws/cloudformation/metaflow-cfn-template.yml#L986

thanks @GregHilstonHop


Azure SignalR doesn't process messages in order under high load

Describe the bug
SignalR messages sent to the client are processed in order (when sent from the same source to the same clients). This behaviour is observed when hosting signalr yourself. However when Azure SignalR is used then some messages are received out of order when there is a high volume of messages. I couldn't find anything about this in the docs, but according to @davidfowl this should be the case, https://twitter.com/bacobart/status/1557296764100743168.
Expected Behavior
When adding AzureSignalR messages received on the client are in order.
Steps To Reproduce
Reproduction at https://github.com/bacobart/azure-signalr-out-of-order-repro
Without adding .AddAzureSignalR() in the server messages are processed in order as expected. When .AddAzureSignalR() is added some messages are processed out of order.
Exceptions (if any)
none
Further technical details
.net 6.0.400
Microsoft.AspNetCore.SignalR.* are 6.0.8
Microsoft.Azure.SignalR is 1.18.1

@vicancy Do we send messages over different connections from the same source https://github.com/bacobart/azure-signalr-out-of-order-repro/blob/8d2ba0012f9815c6ff3f32e92f1a320dab04adaa/SignalrSpamServer/SpamService.cs#L42? The code above shouldn't result in out of order messages

@bacobart, your observations are correct. Currently the message order is only preserved when messages are sent from within the hub call (or its execution context) originated either from a client message or a client connected event. When the hub context is created via DI, the Azure SignalR SDK does not currently have the code to ensure that the order is always correct. We are aware of this problem and have a plan to fix it in the future SDK releases. As a possible temporary workaround, you can move the  Task.Factory.StartNew(() => SpamTask(cancellationToken), TaskCreationOptions.LongRunning);  code inside OnConnectedAsync() method.
The internals of this are rather complex, but in short, the problem stems from the fact that the SDK maintains multiple server connections to the SignalR service. Messages sent to a client over different server connections would not necessarily arrive to the client in the original order. To be able to arrive in the correct order, these messages would need to be sent over the same server connection. When the messages are sent from within the hub call (or from OnConnectedAsync) SDK has a chance to set an async local variable before the hub code gets executed. This asynclocal helps to maintain the same server connection selection for each SendAsync within the hub call context (e.g. any of its execution context derivatives, e.g. Thread.Start, QUWI, Task.Factory.StartNew, etc). When the messages are sent outside of this async local context (e.g. using hubContext created via DI), the SDK is not able to pick the same server connection each time SendAsync is called.
Again, we have a plan to fix this in the future. Please let us know if the proposed workaround is not suitable for you and we'll look for more options before the final solution is available.

@KKhurin Is there any update on this?

@KKhurin Is there any update on this?

I am attempting to use Azure SignalR with Azure OpenAI streaming completions, but the message chunks are being received all out of order on the client side. Is there a workaround for this?

Is there a timeline on this being fixed? This is rendering Azure SignalR unusable for our application.

A timeline for a fix would be appreciated!

Any update to this issue?


Error Response

This error occurred when pulling the images.

Yes please check the README.md & #3 , the image URL has changed


Warn users when CLI is out of date (esp for bug fixed commands)

Description
Outline the issue here:
Saw a user mention on Twitter that they had struggled with a particular az command and it turned out it was due to an old version of CLI (presumably newer versions included a bugfix). It would be nice if the CLI warned users, either, when:

the CLI is out of date
there is a known bugfix for a particular command being executed in the CLI

The first one certainly seems easier, but might be considered a nuisance by people.

Environment summary
Install Method: How did you install the CLI? (e.g. pip, interactive script, apt-get, Docker, MSI, nightly)
Answer here: n/a
CLI Version: What version of the CLI and modules are installed? (Use az --version)
Answer here: n/a
OS Version: What OS and version are you using?
Answer here: n/a
Shell Type: What shell are you using? (e.g. bash, cmd.exe, Bash on Windows)
Answer here: n/a

We can add a periodically check mechanism to CLI. But it won't fix the old CLI which we've already shipped since we can't patch old CLI when users don't know it is old.

This has been implemented as part of az --version.


Arrow 6.0.0 update

Based on https://github.com/blue-yonder/turbodbc/pull/325.

Codecov Report

Merging #326 (aa206f6) into master (1d05735) will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #326      +/-   ##
==========================================
+ Coverage   98.37%   98.40%   +0.03%
==========================================
  Files         127      127
  Lines        2955     2953       -2
==========================================
- Hits         2907     2906       -1
+ Misses         48       47       -1

Impacted Files
Coverage Δ

...urbodbc_arrow/Library/src/set_arrow_parameters.cpp
95.92% <100.00%> (+0.41%)
:arrow_up:

Continue to review full report at Codecov.

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

Went through the Result docs and added the required .ok() check with the previous error message. Now I feel better about this change.


.send command

There is username and emoji in front of send command and it kinda ruined it for me and members of my server, is there any way to remove it or can you add it in future updates? Much love by the way, bot is great!

What is your OS?

Win10

Can't remove that i think? Only on linux you can edit the source files to your liking.

Well that's understandable, however previously It was completely fine and I don't understand this change at all.

I believe it was supposed to be like an "Announcement" kinda thing. But I also disagree with it. It leaves less wiggle room to be creative with it. I mean before you could manually add the user tag and an emoji if you wanted to announce something. it's not useful for other uses tho.

I agree. I was using .send as startup command to announce, that the bot was activated. Now it looks weird.

This is something i personally use. Posted by Sobaco in the Nadeko server.
Info on how to remove the username and the emoji from .send command message:
Directory - NadekoBot/NadekoBot.Core/Modules/Administration
File - SelfCommands.cs
Line - 473 (await ch.SendMessageAsync($"**{Context.User}** 📣 " + rep.Replace(msg)?.SanitizeMentions());)
Text to remove - $**{Context.User}** 📣 " + (So it would look like await ch.SendMessageAsync(rep.Replace(msg)?.SanitizeMentions());)
That being said, any time nadeko updates you would need to re-add this fix. I wish it would just be removed entirely.


Compare plot integration

Hi @teolvs,
I'm using your code to build a visualization library for Brightway. I may have some issues with understanding parts of your code and I want to keep the conversation with you until I'm confident that your code is properly integrated in our library if you'll have the time for that. First few things related to your prepared library:

I'd be glad if you have some time to check whether the logic that you implemented is intact, because we had to make few changes to your code base: https://github.com/LowerSilesians/brightway/tree/main/bw_visualization/compare_plot,
Could you provide a meaningful example for documentation and manual testing with only US EEIO database?

Hello @ViCt0r99,
I am glad to see this work has been used to build a a visualization library for Brightway. I would be happy to deal with you to help you integrating in your library to explain features or parts of the code. I took a look on the changes in your repository. It seems that you have mainly reorganised the code architecture to clean up the calls between each method. It was a bit messy indeed.
In fact, I started to build the library exclusively on US EEIO database, and I adapted it for EcoInvent in a second step. Thus, you can found meaningful examples if you go back in the old commits in the notebook (~13 january).
I compared "Metal windows, doors, and architectural products; at manufacturer" and "Wooden windows, door, and flooring; at manufacturer". Quite interesting example since you have actually impact transfer if you look at several impact categories such as :  ('Impact Potential', 'GCC'), ('Impact Potential', 'HTOX'), ('Resource Use', 'LAND'), ('Resource Use', 'WATR')
Hopefully this answered your question.


fix 'range scans' example

Due to the fact that you want to iterate over all keys that are before or equal to max starting from min the bytes.Compare() check should look like the commit suggests.

@justphil cool, thanks for catching that. :+1:


I added the AntiMicroX Tool to the README

AntiMicroX is a GNU/Linux tool for generating SDL2 configurations.

We’ve got more than enough of these now and I’d much rather stick to platform-agnostic tools. The list in the Readme intends to be a recommendation, not an large list of alternatives.

The database is compatible with SDL v2.0.10 and newer. Older SDL versions are no longer supported.

It does not look like that portion of the program has changed since I last worked on it. IIRC, the SDL GameController mapping strings generated by that window conformed to the standard around SDL 2.0.4. It works out well enough for the capabilities of the program but it cannot take advantage of some of the more advanced mapping options available in later versions of SDL.

@Ryochan7 yeah, I’m inclined to remove stuff that’s fallen out of date, I’d like to feature your Gitlab-hosted version of gamepad tool as our top recommendation and list the others as lesser alternatives — would you consider including Win/Mac/Linux binaries with your releases?


move mixer/synth from codal to PXT

I'm not sure we want to merge it just yet.
This fixes stuttering sounds in arcade. It also brings back ringTone, but it clicks every half second.
Tested on Arcade STM only so far. You can try to test it on Adafruit and possibly include there.

There are memory problems here. Will push a fix today

Ready to go? @mmoskal


"View collected data" - what log search tile?

In this page, in the View Collected Data section it says "In the Azure portal, in the selected workspace, select the Log Search tile." I cannot find this Log Search tile anywhere. All I can find is this link to a log query editor which is confusing and does not seem to offer any basic log viewing functionality.

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

ID: 56ea7a65-07e2-1af6-9ce2-a1909aedd31b
Version Independent ID: d9a077ff-5241-36ad-524a-33ee850e110f
Content: Configure the Azure Log Analytics agent for hybrid Windows computers
Content Source: articles/azure-monitor/learn/quick-collect-windows-computer.md
Service: log-analytics
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

@sshorkey Thanks for the question! We will investigate this issue and get back to you soon.

Hi @sadiqhAhmed-msft, any update on this?

@sshorkey - You are correct that with the number of recent changes to the portal as a result of delivering a better/cohesive log search experience, the steps are out of date and the article is pending an update.  In the meantime, select Monitor from the left-hand pane in the portal, then select Logs.  In the blank log query page, select the appropriate workspace from the top-left of the query page and then type in the log search query.
Hope that helps.  #in-process as this article has been tagged as a work item with intention to address in our next sprint.

#please-close as the article has been updated to align with the experience in the portal.


Crashing after choosing schedule

Name of program: VGSK2 2020 04 100 NML sv

NGBM1 2021

Should hopefully be fixed in contact with solving #22


RS4 update 2

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/case-study-expanding-the-design-process-for-mixed-reality.md
:white_check_mark:Succeeded
View

mixed-reality-docs/hololens-rs4-preview.md
:white_check_mark:Succeeded
View

mixed-reality-docs/images/academyCollab1000.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/academyMat1000.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/academyTeam1000.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/academyWork1000.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/storyboardHigh1000.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/storyboardLow1000.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/test.jpg
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Implement verbatim option

--verbatim options are included in cli argument, but has NOOP currently, so make it work.

I have to make sure that I need glob crate for this


Remove version: qBittorrent.qBittorrent version 4.4.5

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

Close with reason: URL works;


Send email when daily data missing

We should look into feasibility of email alerts when a condition is met. A storm last Friday night might've knocked out the phone modem at the Tucson station. Regardless of reason, data stopped coming in with the phone down. We remotely rebooted the phone on Sunday to fix the situation. An email alert if daily data don't come in (shortly after midnight is the scheduled time) would let us know sooner about such situations, and help us get things back in order more quickly.

I'll create a new repository to work on this using blastula and a scheduled .Rmd on Posit Connect.  The email needs to be sent from somewhere—I can prototype with a burner gmail account for now, but eventually should think about where you want the email to come from.  It could be that we need to ask Matt Rahr or someone else at UA to create an email address for this purpose.

Would the existing<EMAIL_ADDRESS>work? I'm the wizard behind the curtain on that one.

Perhaps!  I'm still figuring out how authentication works with blastula, but I'll try with my arizona.edu account to see if I can get it to work.

Might need help from IT to get emails from an arizona.edu address to work, but might also be possible with help from Microsoft365R
https://github.com/Azure/Microsoft365R#outlook

I should check the Posit Connect example on the blastula site again—it's not clear how it sends the email and if it is using some mechanism unique to Connect.  If it is, then this might avoid the issue of having to set up an email address for this.

Apparently our instance of Posit Connect is not sending alert emails when scheduled reports run.  We are in contact with the administrator to get this feature enabled.

Sounds like this is maybe not going to be possible, at least not soon.  Next step is to test out Outlook authentication with arizona.edu emails https://github.com/Azure/Microsoft365R#outlook


Changing background color and color of component boxes on hovering

What's the style idea?
When we hover on the component boxes, we can change the color and background color .
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[ ] I am a IWOC'24 contributor.
[X] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

@Rakesh9100 Please assign me this issue under JWoC'24

Proceed @pranjal12-web


Home assistant fails to load toon.climate

First of all, great work. all sensors are working as expected except the thermostat. I get the following error with home assistant 0.89:
Error loading custom_components.toon.climate. Make sure all dependencies are installed
Traceback (most recent call last):
File "/usr/local/lib/python3.7/site-packages/homeassistant/loader.py", line 166, in _load_file
module = importlib.import_module(path)
File "/usr/local/lib/python3.7/importlib/init.py", line 127, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1006, in _gcd_import
File "", line 983, in _find_and_load
File "", line 967, in _find_and_load_unlocked
File "", line 677, in _load_unlocked
File "", line 728, in exec_module
File "", line 219, in _call_with_frames_removed
File "/config/custom_components/toon/climate.py", line 9, in
from homeassistant.components.climate import (ClimateDevice,
ImportError: cannot import name 'STATE_AUTO' from 'homeassistant.components.climate' (/usr/local/lib/python3.7/site-packages/homeassistant/components/climate/init.py)

The climate constants have bin moved in HA home-assistant/home-assistant#20945
I am working on a fix.

Can you try the new master?

Works like a charm again. Thanks a lot... keep up the good work!


Can’t turn nginx off

Add parameter to disable nginx

Created upstream issue to track: https://github.com/voxpupuli/puppet-nginx/issues/1208


Title: TicTacToe

Describe your issue.
We all played this game with a pen and paper in childhood.
This is a simple web version of the game which you can play with your friends.
Screenshots (if applicable).

Additional context (if any)
No response

@s0hamn Go ahead 🚀
PS: Make sure that you don't use any libraries.


Bug Report in my_script.py and notebook.ipynb

Current behavior
I noticed a bug while developing with the to.sql example in the my_script.py and the notebook.ipynb where a value error was being caused since the scripts attempt to create a table named test but if that table already exists (by running the script multiple times) it will cause a ValueError
Stack Traces
my_script.py
Traceback (most recent call last):
  File "/home/jordan/Desktop/OpenSource/hamilton-js/examples/pandas/materialization/my_script.py", line 91, in <module>
    materialization_results, additional_outputs = dr.materialize(
  File "/home/jordan/Desktop/OpenSource/hamilton-js/hamilton/driver.py", line 1007, in materialize
    raw_results = self.graph_executor.execute(
  File "/home/jordan/Desktop/OpenSource/hamilton-js/hamilton/driver.py", line 161, in execute
    fg.execute(nodes, memoized_computation, overrides, inputs)
  File "/home/jordan/Desktop/OpenSource/hamilton-js/hamilton/graph.py", line 556, in execute
    return execute_subdag(
  File "/home/jordan/Desktop/OpenSource/hamilton-js/hamilton/execution/graph_functions.py", line 157, in execute_subdag
    dfs_traverse(final_var_node, dep_type)
  File "/home/jordan/Desktop/OpenSource/hamilton-js/hamilton/execution/graph_functions.py", line 146, in dfs_traverse
    value = adapter.execute_node(node_, kwargs)
  File "/home/jordan/Desktop/OpenSource/hamilton-js/hamilton/base.py", line 504, in execute_node
    return node.callable(**kwargs)
  File "/home/jordan/Desktop/OpenSource/hamilton-js/hamilton/function_modifiers/adapters.py", line 492, in save_data
    return data_saver.save_data(data_to_save)
  File "/home/jordan/Desktop/OpenSource/hamilton-js/hamilton/plugins/pandas_extensions.py", line 505, in save_data
    results = data.to_sql(self.table_name, self.db_connection, **self._get_saving_kwargs())
  File "/home/jordan/.local/lib/python3.10/site-packages/pandas/util/_decorators.py", line 333, in wrapper
    return func(*args, **kwargs)
  File "/home/jordan/.local/lib/python3.10/site-packages/pandas/core/generic.py", line 3008, in to_sql
    return sql.to_sql(
  File "/home/jordan/.local/lib/python3.10/site-packages/pandas/io/sql.py", line 788, in to_sql
    return pandas_sql.to_sql(
  File "/home/jordan/.local/lib/python3.10/site-packages/pandas/io/sql.py", line 2440, in to_sql
    table.create()
  File "/home/jordan/.local/lib/python3.10/site-packages/pandas/io/sql.py", line 927, in create
    raise ValueError(f"Table '{self.name}' already exists.")
ValueError: Table 'test' already exists.

notebook.ipynb
Node df_to_sql encountered an error
Traceback (most recent call last):
  File "/home/jordan/Desktop/OpenSource/hamilton-js/hamilton/execution/graph_functions.py", line 146, in dfs_traverse
    value = adapter.execute_node(node_, kwargs)
  File "/home/jordan/Desktop/OpenSource/hamilton-js/hamilton/base.py", line 504, in execute_node
    return node.callable(**kwargs)
  File "/home/jordan/Desktop/OpenSource/hamilton-js/hamilton/function_modifiers/adapters.py", line 492, in save_data
    return data_saver.save_data(data_to_save)
  File "/home/jordan/Desktop/OpenSource/hamilton-js/hamilton/plugins/pandas_extensions.py", line 506, in save_data
    return utils.get_sql_metadata(self.table_name, results)
  File "/home/jordan/.local/lib/python3.10/site-packages/pandas/util/_decorators.py", line 333, in wrapper
    return func(*args, **kwargs)
  File "/home/jordan/.local/lib/python3.10/site-packages/pandas/core/generic.py", line 3008, in to_sql
    return sql.to_sql(
  File "/home/jordan/.local/lib/python3.10/site-packages/pandas/io/sql.py", line 788, in to_sql
    return pandas_sql.to_sql(
  File "/home/jordan/.local/lib/python3.10/site-packages/pandas/io/sql.py", line 2440, in to_sql
    table.create()
  File "/home/jordan/.local/lib/python3.10/site-packages/pandas/io/sql.py", line 927, in create
    raise ValueError(f"Table '{self.name}' already exists.")
ValueError: Table 'test' already exists.

Screenshots
notbook.ipynb with the same error in the my_script.py

Steps to replicate behavior

Run the my_script.py file multiple times or the code block in the notebook.ipynb file that starts with "materialize a result, i.e. execute the DAG!"

Library & System Information
Python version: 3.10
Hamilton library version: 1.29.0
OS: Ubuntu 22.04
Expected behavior
The code should not cause a ValueError
Additional context
This is a relatively simple fix and I will submit a PR with the code changes.

Closing since this issue has been solved.


Problem with AMC natural players

Hi! Whenever i try to copy the stats of a player from fminside with AMC as his natural position it doesn't really copy the stats. This doesn't happen with players which AMC is their secondary position. Thank you very much in advance.

Hello again! Yes, i'm using the latest version on chrome. Here are some players which their stats don't copy:
https://fminside.net/players/4-fm-24/76002390-james-rodriguez
https://fminside.net/players/4-fm-24/2000225781-thiago-perugini
https://fminside.net/players/4-fm-24/2000107899-matias-gomez
https://fminside.net/players/4-fm-24/96132920-jakub-kauziski

Ah, i forgot to tell this only happens when i try to copy the stats for pes 2021.

thanks for the report and the examples, im gonna try to fix it on the following days

thanks for the report, fixed on #4ccf75d0dbde18594ef1813e66e402e8c47bf61f


Add a script to convert json files into sound files and play.

Hearing and learning a new word would be great rather than just reading it.

Maybe this would do the job: https://pythonprogramminglanguage.com/text-to-speech/

I ll work on this

For time being a cli can work

I can add that to this

I will work on this
will use gtts


Add source-layer for Mapbox-hosted vector tiles

If I want to add vector tiles from my Mapbox.com account, there is a required field that your Layer component leaves out: source-layer. Is it possible to get this added as a prop?
map.addSource("states", {
      type: "vector",
      url: "mapbox://" + mapID
    });

 map.addLayer({
      'id': 'states',
      'type': 'fill',
      'source': layerName,
      'source-layer': layer_from_mapbox, ///this maps to the tileset name referenced by mapID above
      'maxzoom': zoomThreshold,
      'paint': {
        'fill-color': {
          'property': vtMatchProp,
          'type': 'categorical',
          'stops': stops
        },
        'fill-outline-color': '#cccccc'
      }
    });

I experienced this too, then discovered the layerOptions object can be stuffed with anything else you need that isn't supported by default.
So my object which gets spread into props for Layer is thus:
{
    id: 'bar',
    key: Date.now(),
    type: 'symbol',
    layout: {
        'icon-image': 'marker-15'
    },
    sourceId: 'foo',
    layerOptions: {
        'source-layer': 'foo'
    }
}

Thanks @davecranwell - that looks like it will solve it!


Official Python Impelementation?

Hi, thanks for the library. Doing this in python opens the door for ease of use when integrating with CV.
Is it fair to say that this is an official implementation of a KVS consumer?
If so, any way that this could be refactored a bit and shipped as an sdk on pypi? Similar to what AWS has done with their Java Parser?
This is great but it ships a bit heavy having to clone the entire repo, edit files, and include in a local/remote folder to be able to make use of this implementation!

Hi Alex,
This is unofficial in that its community supported opposed to officially supported by the AWS Kinesis Video Service itself. That being said, is no reason it can't be packaged in PyPi with a similar support model.
As time permits I will publish as a PyPi package.
Regards,
Dean.


add support for .scss and .sass files

Add support for .scss and .sass files.

I was just about to code this myself, and yeah this should get merged.


FAQ: How to view participating countries - Step 3 is wrong

Where to find the issue
https://www.coronawarn.app/en/faq/#interoperability_countries
To view more information in the app, tap

'EXPOSURE LOGGING ACTIVE/STOPPED'
Then tap 'Transnational Exposure Logging'.
At last click on 'Participating Countries'.

Describe the issue
On CWA Android 1.6.1 only two taps are necessary to show Participating Countries. Step 3 in the instructions is not right because Transnational Exposure Logging Participating Countries is one element in the UI.

Tap on EXPOSURE LOGGING ACTIVE

Tap on Transnational Exposure, Logging Participating Countries

In Transnational Exposure Logging, scroll down to the list of countries

Suggested change
Check that CWA iOS behaves the same way as CWA on Android.
If yes, then replace Step 3 with:
"scroll down to the list of countries".

@MikeMcC399 iOS behaves like Android.

@Ein-Tim
Thanks for the confirmation about iOS!


The website is 404

Describe the bug
Website returns 404
To Reproduce
Navigate to https://codewars.com
Expected behavior
Website works
Screenshots

Browser

OS: Windows
Browser chrome
Version 96.0.4664.45

Additional context
No

Just after I submitted it, it started working

Yeah, we were affected by this https://status.cloud.google.com/incidents/6PM5mNd43NbMqjCZ5REh


Check GraalVM native support in Ballerina external modules

Description:
$Subject
Describe your task(s)
The following modules should be checked with GraalVM native support

Observability modules

[x] ballerinax/jaeger
[ ] ballerinax/prometheus

Database modules

[ ] ballerinax/mysql
[ ] ballerinax/mssql
[ ] ballerinax/oracledb
[ ] ballerinax/postgresql
[ ] ballerinax/java.jdbc
[ ] ballerinax/redis

Database drivers

[ ] ballerinax/mysql.driver
[ ] ballerinax/mssql.driver
[ ] ballerinax/oracledb.driver
[ ] ballerinax/postgresql.driver

Driver support is tracked in https://github.com/ballerina-platform/ballerina-standard-library/issues/3692

Driver support is tracked in #3692

Ack

The GraalVM native support for ballerinax/prometheus is blocked due to this issue.

All the external modules are now GraalVM compatible


IndexMap sometimes reports a lower capacity after removing elements

I made a small test script that adds and removes elements from an IndexMap:
use indexmap::IndexMap;

fn main() {
	let mut map = IndexMap::with_capacity(10);
	for i in 0..1000 {
		map.insert(i, i);
		println!("{} / {}", map.len(), map.capacity());
		if i % 5 == 0 {
			map.swap_remove_index(i / 2);
			map.swap_remove_index(i / 3);
			map.swap_remove_index(i / 4);
		}
	}
}

Sometimes, when the length of the map (left) decreases, the capacity (right) decreases as well.
410 / 431
411 / 432
412 / 432
413 / 432
411 / 431
412 / 431
413 / 431
414 / 432
415 / 432
414 / 430
415 / 430
416 / 430
417 / 430
418 / 431
416 / 429
417 / 429
418 / 429
419 / 430
420 / 430
419 / 429
420 / 429
421 / 429
422 / 430
423 / 430
421 / 427
422 / 427
423 / 427
424 / 427
425 / 427

From what I understand, the "capacity" is the total number of elements the map can hold in the chunk of memory it has allocated. If you add an item when the map is full, I would expect the capacity to increase as the map allocates more memory. I would never expect the capacity to decrease.
This behavior occurs with both swap_remove_index and shift_remove_index.

This is an artifact of hashbrown::raw::RawTable, and you can reproduce it with std::collections::HashMap too, just removing by key instead of index. Capacity is reported as items + growth_left, but the latter is only sometimes updated on removal. That is, a removed entry may be marked EMPTY and increase growth_left, but it sometimes needs a DELETED tombstone and does not change growth_left, which is when you see the reported capacity decrease.
IndexMap has two relevant capacities, the RawTable and the item Vec, and we report the lesser of those.

Why does DELETED not change growth_left? Should I take this to mean that the capacity of the map actually decreases, i.e. if I create a map with a capacity of 10, later on an allocation might be required to add items even if the length does not exceed 10?

OK, so I guess that means I shouldn't use IndexMap in a scenario where I don't want to trigger memory allocations after the initial creation of the map?

Currently, the only guarantee about avoiding allocation is if you dynamically check the capacity().
With hashbrown 0.11 (#181) we could add a method that internally uses RawTable::try_insert_no_grow, and on failure we could rehash to scrub DELETED entries and try again. Maybe we could also figure that out now using the reported capacity() and what we know of the expected load factor of the raw buckets().

In the meantime, maybe it would be good to add something to the documentation about this? I'm sure I'm not the only one confused about why the capacity can decrease over time.

Adding docs seems like the sensible action to resolve this issue, for the rest we refer to hashbrown.

Hashbrown's capacity says this:

This number is a lower bound; the HashMap<K, V> might be able to hold more, but is guaranteed to be able to hold at least this many.

We could say something like that, and perhaps also note that removal may cause that to decrease.

@tesselode if you're interested, maybe you could comment on rust-lang/hashbrown#255 why you're concerned about this, and whether you think such a manual vacuum operation would work for you.


Fix typos

🐝

:tada: This PR is included in version 1.7.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fixed some tests and removed series information which was contained in each data point

I don't have time to fix the rest of the tests but there are not a lot left to fix...


Drop wchar_t support

Conditionally providing this when __STDC_ISO_10646__ is defined is a bad idea.  The non-portable nature of its code point value already proved that it's a bad choice of code point type (it's still relevant to some applications, but where it's relevant happens to be where __STDC_ISO_10646__ is not defined, e.g., on FreeBSD and Windows).

Support for wchar_t isn't only provided when STDC_ISO_10646 is defined.  That macro only guards the definition of the iso_10646_wide_character_encoding encoding.  The actual encoding for wide string literals is provided by the execution_wide_character_encoding type alias.  When STDC_ISO_10646 is defined, that would presumably alias iso_10646_wide_character_encoding, but is otherwise present and aliases some other implementation defined encoding.
I agree that wide string literals are non-portable, but text_view still provides some support for working with wide strings in a portable way.  For example, consider a multi-byte encoding that allows single byte code unit values to appear as the second byte of a multi-byte sequence.  Naively splitting a string based on the intended single byte code unit value would incorrectly split the multi-byte sequence.  String splitting implemented with the text_view code point iterators avoids this potential problem.

But supporting execution_wide_character_encoding cannot be called supporting wchar_t either.  Either you uses locale information to fully support it, or drop it, otherwise I can't see any usefulness coming out of it.

execution_wide_character_encoding refers to the encoding that is used at compile-time by the compiler to encode wide string and character literals; the encoding that is controlled by the '-fwide-exec-charset=' gcc option.  Naturally, this encoding may differ from encodings specified by run-time locale settings.

That's not nature.  Locale is designed to work with wchar_t, see C 7.29.6, and many applications uses such a convention.  It creates library composability issue if such a convention is not maintained.  So I claim that relying on locale information is best you can do; relying on other source of information to perform automatic conversion is as bad as forking the standard.

I understand.  In C and C++ applications, the wchar_t interfaces operate using the encoding specified by locale settings established at run-time.  No disagreement.  C++ [lex.charset]p3 is clear that the locale settings govern the interpretation of wchar_t code unit sequences at run-time.
However, that doesn't change the fact that there is an additional encoding involved at compile-time; the encoding used to encode wide string and character literals.  This encoding is not based on run-time locale settings (for the application; it may be based on the run-time locale settings in effect for the compiler invocation itself).  This is the encoding used by the compiler to translate universal character names appearing in wide string and character literals to the code unit sequence it emits for the literal (which may, of course, encode a replacement character).  This encoding may correspond to the basic execution wide character set, or may be an extension of it.  It is possible for this encoding to be a super set of the basic execution wide character set and a subset of the (run-time selected) execution wide character set.  If it isn't compatible with the (run-time selected) execution wide character set, well, chaos ensues, but that is status quo.
It may be that the encoding I'm referring to here has limited use.  In fact, it may be that it is only useful for doing compile-time constexpr or TMP manipulation of wide string and character literals.  However, I do believe the support for this is sound.

The purpose of adding universal character names is not let you to put them in "" or L"".  It sometimes works, by accident, by historical reasons, but that should not be a practice that we are promoting.  Of course many homemade way to construct applications are sound and it's also sound to build some library to solve these needs, but I don't think the standard care (of course you can put these in a paper and ask in LEWG).
Another way to look at it is that you can put the interface at an acceptable level of genericity, and leave some parts implementation-defined ("implementations may support additional blah blah blah").  For instance, libstdc++ has an iconv codecvt in ext/.  It's not mandated by the standard, but there might be people using it.

My goal is genericity and sound, consistent behavior.  I'm definitely open to controversial support being left as optional or implementation defined, so long as the interface doesn't preclude such support.

Closing this issue.  I still intend to support wide string encodings.


The homepage load time is too slow

There's definitely some optimizations that can be made to make the website load faster. This issue is related to #75 & #70.
Right now for me the page loads in about 6+ seconds. The biggest problems I can see are:

Many resources coming from rethinkdb.com that aren't loading in parallel.
The fonts are blocking rendering of the page.
We aren't minifying anything I believe and aren't setting proper cache headers.
We aren't using a CDN (We can use Cloudflare for free which will also give us HTTPS #104).

https://developers.google.com/speed/pagespeed/insights/?url=www.rethinkdb.com&tab=desktop
I think with these methods we can at least get in the sub-three second mark, if not two second.

This load time significantly increased. Now the performance score is 100/100.


Adds ability to toggle elements in sphinx docs

Also adds blt logo to RTD
Tasks:

[x]  Add toggle js and css elements
[x]  Test using a toggled item in docs
[x]  Add version of logo with transparent background

Here's an example of toggling:

See: https://llnl-blt.readthedocs.io/en/feature-kweiss-toggle-docs/tutorial/setup_blt.html

Can you change this in the conf.py while you are at it?  This controls the code snippet background color.
https://github.com/LLNL/blt/blob/9f024f1d3c8b0e70c92d01bc3eacfb69a847db04/docs/conf.py#L90
"default" is fine.. just not "sphinx"

@white238 -- good suggestion about the pygments theme. default is a nice improvement!

LGTM


Translate concepts/configuration/scheduler-perf-tuning.md in Korean

This is a...

[x] Feature Request
[ ] Bug Report

Problem:
No concepts/configuration/scheduler-perf-tuning.md in Korean
Proposed Solution:
Translate concepts/configuration/scheduler-perf-tuning.md in Korean
Page to Update:
https://kubernetes.io/...

/language ko


buildable package conditional exports  and bundle size inconsistencies

What version of Turborepo are you using?
1.4.6
What package manager are you using / does the bug impact?
pnpm
What operating system are you using?
Mac
Describe the Bug

Next.js app does not seem to recognize conditional exports mjs/cjs extensions from buildable package
Inconsistent bundle size when an app imports third party package vs when the app uses the same third party package via an internal buildable package

Expected Behavior

Next.js app should be able to resolve conditional exports with mjs/cjs extensions from buildable package
Bundle size when an app imports third party package vs when the app uses the same third party package via an internal buildable package should be close

To Reproduce
A full reproduction is available here: repo
The root README.md contains the few commands to copy/paste if needed

To je more precise re. bundle size, not sure why next includes readable-stream package when using a lib built upon @emotion/cache

Closing due to staleness. Will mention though that this wouldn't be something that is in Turborepo's domain! (at least not today)


I hope to add a tool to convert regular images such as PNG, JPG into vector images such as SVG

Description of the new feature / enhancement
I hope to add a tool to convert regular images such as PNG, JPG and other vector images into SVG and other vector images, this image conversion function is like color extractor or OCR, which is becoming more and more important for users, especially designers and developers
Scenario when this would be used?
When a developer or designer needs to use certain images in his product to adapt to different size devices, so that the image looks the same on different devices，Usually to do something similar, you need some specialized software, they are either expensive or complex, but developers and designers only have this little need, if PowerToy can do it, it really helps
Supporting information
No response

Neat idea but No plans


MH-13589 Check dependencies at build time

The plugin maven-dependency-plugin can be used to check dependencies at build time.
The goal of this task is to add that plugin to as many modules as possible implying the correction of errors it detects.
Notes:

This PR does not cover all modules
This PR doesn't take much care of the order the dependencies appear but tries to fit in the respective files. I tend to go for alphabetical order but don't do this consistently as the files being edited are already quite inconsistent...
This PR does not update NOTICES (it also does not add any dependency ;-)

This PR does not update NOTICES (it also does not add any dependency ;-)

iirc, the maven part of the NOTICES was auto-generated anyway…
If we have covered most modules, we can also activate this by default in the main pom.xml. Then you will only need to define exceptions.

This PR does not update NOTICES (it also does not add any dependency ;-)
iirc, the maven part of the NOTICES was auto-generated anyway…

@gregorydlogan Note sure how important it is to have the dependencies in the NOTICES, but please not that the current state is "not that good". We probably should either remove them or fix them.

If we have covered most modules, we can also activate this by default in the main pom.xml. Then you will only need to define exceptions.

I haven't counted the exact number of modules not covered now, but I estimate it to approx. 20. Not sure about how to configure per-module infos (ingoreDeclaredDependencies) etc, so I would rather stay with the current solution.

My idea was to have the plugin enabled by default without any ignores. Modules can then overwrite this by applying their own configuration in exactly the same way you did it here. But you can drop it wherever no configuration is necessary and new modules are forced to deal with it ;-)
Another option would be to pre-configure the logger as an allowed dependency since it is the most common and it's probably less individual configuration. The downside is that people can still integrate it if unnecessarily.

I think we should move towards having it enabled for all modules (including modules that are added) to ensure we don't end up in the current state again ;-)
But that would require to make it work with all modules which I currently can't (I left the problematic ones out... would take more time).
I would not ignore any unused declared dependencies on globally.

@lkiesow Fixed build -> PR ready for review.

@staubesv NOTICES being a shambles is a known issue.  Keeping it up to date manually sucks, not sure how better to go about it.  One of those things which will likely never be up to date, but it is being tracked.

Stats now don't look that bad any longer. Thanks for the work:
% grep -rl maven-dependency-plugin modules --include=pom.xml | wc -l
166
% grep -rL maven-dependency-plugin modules --include=pom.xml | wc -l
45


Use native ES6 classes over Uptown

This removes the dependency on Uptown and uses ES6 classes which are available in Node 4, the lowest Node we support.
Transpilation is done for the browser distribution to maintain compatibility for all browsers.
I've also added use of const/let over var while I was at it.
DO NOT MERGE
As of right now, this change would break downstream minim-api-description and minim-parse-result due to https://github.com/babel/babel/issues/6299 (Babel-transpiled class can't extend from native class), a fix for this can be found in Babel 7 (there are betas available but no final release). We could move peasant and other minim libraries to Babel 7.
An alternative, perhaps we can move minim-* packages to work similar to minim. Where the node sources are not transpiled and instead written for plain Node 4 and then just transpired for the web distribution.

@kylef Do we need this anymore?

I'd love to move to ES6 at some point, but I'd say it isn't a priority (if it isn't broken don't "fix" it). Will close PR as it certainly needs to be done again as things have changed and there is large conflict which will be difficult to fix. If done again we should do it more incrementally piece by piece.


Create knative-serving-conformance-testing-workflow.md

Steps to follow to gain knative conformance certification

Changes

/kind

Fixes #

Release Note

Docs

/assign @evankanderson

Once we are ready, we can create the issue template in .github/SOME-FILE-I-DONT-REMEMBER so people don't need to copy the table.
This works for me for now.

Can you update the description to link to the discussion document?

Original doc: https://docs.google.com/document/d/1_gWhCeaU42DtHFkwVJOAbLVyuVaW5EhRkulLqQSTxkc/edit

thanks,
/lgtm

/lgtm

/assign @smoser-ibm
/lgtm
Simon, if you're happy with this, reply in a comment with /approve on its own line, and it will be be approved and auto-submit once tests pass.

/approve

@smoser-ibm a gentle ping :)


Issue Running the Example code

Hello,
Apologies if this is a very basic question, I am having an issue running your code EarlyArrivalCount; I followed all the steps but when I try running the code, I get the following error message (in Linux and in Windows). Please note that when I set
val writeToElasticsearch = false, it runs and echoes data in the console.
Looks like it's complaining about elasticsearch, I've confirmed that it is running, I also had to make the change recommended in one of the posting:
config.setInteger(ConfigConstants.JOB_MANAGER_WEB_PORT_KEY,9999)
Any help, advice will be greatly appreciated, Thanks in advance.
Frank
Error Message
log4j:WARN No appenders could be found for logger (org.apache.flink.api.java.typeutils.TypeExtractor).
log4j:WARN Please initialize the log4j system properly.
log4j:WARN See http://logging.apache.org/log4j/1.2/faq.html#noconfig for more info.
05/09/2016 10:37:23	Job execution switched to status RUNNING.
05/09/2016 10:37:23	Source: Custom Source(1/1) switched to SCHEDULED
05/09/2016 10:37:23	Source: Custom Source(1/1) switched to DEPLOYING
05/09/2016 10:37:23	Filter -> Filter -> Map(1/1) switched to SCHEDULED
05/09/2016 10:37:23	Filter -> Filter -> Map(1/1) switched to DEPLOYING
05/09/2016 10:37:23	TriggerWindow(SlidingTimeWindows(900000, 300000), com.dataartisans.flink_demo.examples.EarlyArrivalCount$EarlyCountTrigger@1f1c7bf6, WindowApply at apply(WindowedStream.scala:209)) -> Map -> (Sink: Unnamed, Sink: Unnamed)(1/1) switched to SCHEDULED
05/09/2016 10:37:23	TriggerWindow(SlidingTimeWindows(900000, 300000), com.dataartisans.flink_demo.examples.EarlyArrivalCount$EarlyCountTrigger@1f1c7bf6, WindowApply at apply(WindowedStream.scala:209)) -> Map -> (Sink: Unnamed, Sink: Unnamed)(1/1) switched to DEPLOYING
05/09/2016 10:37:23	Source: Custom Source(1/1) switched to RUNNING
05/09/2016 10:37:23	TriggerWindow(SlidingTimeWindows(900000, 300000), com.dataartisans.flink_demo.examples.EarlyArrivalCount$EarlyCountTrigger@1f1c7bf6, WindowApply at apply(WindowedStream.scala:209)) -> Map -> (Sink: Unnamed, Sink: Unnamed)(1/1) switched to RUNNING
05/09/2016 10:37:23	Filter -> Filter -> Map(1/1) switched to RUNNING
(49551,1356998700000,GeoPoint(-73.97789764404297,40.75187683105469),2)
05/09/2016 10:37:31	TriggerWindow(SlidingTimeWindows(900000, 300000), com.dataartisans.flink_demo.examples.EarlyArrivalCount$EarlyCountTrigger@1f1c7bf6, WindowApply at apply(WindowedStream.scala:209)) -> Map -> (Sink: Unnamed, Sink: Unnamed)(1/1) switched to FAILED
java.lang.RuntimeException: Could not forward element to next operator
at org.apache.flink.streaming.runtime.tasks.OperatorChain$CopyingChainingOutput.collect(OperatorChain.java:319)
at org.apache.flink.streaming.runtime.tasks.OperatorChain$CopyingChainingOutput.collect(OperatorChain.java:300)
at org.apache.flink.streaming.runtime.io.CollectorWrapper.collect(CollectorWrapper.java:48)
at org.apache.flink.streaming.runtime.io.CollectorWrapper.collect(CollectorWrapper.java:29)
at org.apache.flink.streaming.api.operators.TimestampedCollector.collect(TimestampedCollector.java:51)
at com.dataartisans.flink_demo.examples.EarlyArrivalCount$$anonfun$5.apply(EarlyArrivalCount.scala:96)
at com.dataartisans.flink_demo.examples.EarlyArrivalCount$$anonfun$5.apply(EarlyArrivalCount.scala:95)
at org.apache.flink.streaming.api.scala.WindowedStream$$anon$3.apply(WindowedStream.scala:206)
at org.apache.flink.streaming.runtime.operators.windowing.WindowOperator.emitWindow(WindowOperator.java:302)
at org.apache.flink.streaming.runtime.operators.windowing.WindowOperator.processTriggerResult(WindowOperator.java:335)
at org.apache.flink.streaming.runtime.operators.windowing.WindowOperator.processWatermark(WindowOperator.java:362)
at org.apache.flink.streaming.runtime.io.StreamInputProcessor.processInput(StreamInputProcessor.java:156)
at org.apache.flink.streaming.runtime.tasks.OneInputStreamTask.run(OneInputStreamTask.java:63)
at org.apache.flink.streaming.runtime.tasks.StreamTask.invoke(StreamTask.java:218)
at org.apache.flink.runtime.taskmanager.Task.run(Task.java:584)
at java.lang.Thread.run(Thread.java:745)
Caused by: java.lang.RuntimeException: Could not forward element to next operator
at org.apache.flink.streaming.runtime.tasks.OperatorChain$CopyingChainingOutput.collect(OperatorChain.java:319)
at org.apache.flink.streaming.runtime.tasks.OperatorChain$CopyingChainingOutput.collect(OperatorChain.java:300)
at org.apache.flink.streaming.runtime.io.CollectorWrapper.collect(CollectorWrapper.java:48)
at org.apache.flink.streaming.runtime.io.CollectorWrapper.collect(CollectorWrapper.java:29)
at org.apache.flink.streaming.api.operators.StreamMap.processElement(StreamMap.java:37)
at org.apache.flink.streaming.runtime.tasks.OperatorChain$CopyingChainingOutput.collect(OperatorChain.java:316)
... 15 more
Caused by: org.elasticsearch.client.transport.NoNodeAvailableException: None of the configured nodes are available: []
at org.elasticsearch.client.transport.TransportClientNodesService.ensureNodesAreAvailable(TransportClientNodesService.java:305)
at org.elasticsearch.client.transport.TransportClientNodesService.execute(TransportClientNodesService.java:200)
at org.elasticsearch.client.transport.support.InternalTransportClient.execute(InternalTransportClient.java:106)
at org.elasticsearch.client.transport.support.InternalTransportClient.execute(InternalTransportClient.java:97)
at org.elasticsearch.client.support.AbstractClient.update(AbstractClient.java:122)
at org.elasticsearch.client.transport.TransportClient.update(TransportClient.java:345)
at com.dataartisans.flink_demo.sinks.ElasticsearchUpsertSink.invoke(ElasticsearchUpsertSink.scala:80)
at org.apache.flink.streaming.api.operators.StreamSink.processElement(StreamSink.java:37)
at org.apache.flink.streaming.runtime.tasks.OperatorChain$CopyingChainingOutput.collect(OperatorChain.java:316)
... 20 more
05/09/2016 10:37:31	Job execution switched to status FAILING.
05/09/2016 10:37:31	Source: Custom Source(1/1) switched to CANCELING
05/09/2016 10:37:31	Filter -> Filter -> Map(1/1) switched to CANCELING
05/09/2016 10:37:31	Filter -> Filter -> Map(1/1) switched to CANCELED
05/09/2016 10:37:31	Source: Custom Source(1/1) switched to CANCELED
05/09/2016 10:37:31	Job execution switched to status FAILED.

Hi @FrankRojas, which version of Elasticsearch are you running? The example is based on Elasticsearch 1.7.3, so not the most recent version.

Thank you Fabian, that was it; I switched to the correct version of Elasticsearch and it started working.
Thank you for sharing your knowledge.
Frank


Report shipper version in User-Agent headers

Closes #147

@kanatohodets it's in an atomic commit, so it's fairly easy to separate. I'll open a new PR.

Done: #164 -- I'll rebase this once that's merged.

[...] we can probably remove the enormous QPS limits in the near future.

Yup, I plan to change those once #165 lands :)


Enabling https causes an infinite loop of reloads

Describe the bug
When we enable https as mentioned in the documentation with this command:
svelte-kit dev --https

The website, once accessed over https will cause an infinite loop of reloads.
Logs
Browser logs (brave browser, fork of chromium):

Browser logs (firefox)

To Reproduce
Add the --https flag to the dev script in package.json.
Expected behavior
The website should load over https without infinite reload loops
Stacktraces
Nothing
Information about your SvelteKit Installation:

Diagnostics

The output of npx envinfo --system --npmPackages svelte,@sveltejs/kit,vite --binaries --browsers

  System:
    OS: Linux 5.11 Pop!_OS 20.10
    CPU: (4) x64 Intel(R) Core(TM) i5-5350U CPU @ 1.80GHz
    Memory: 794.92 MB / 7.67 GB
    Container: Yes
    Shell: 5.8 - /usr/bin/zsh
  Binaries:
    Node: 14.16.1 - ~/.nvm/versions/node/v14.16.1/bin/node
    Yarn: 1.22.10 - ~/.nvm/versions/node/v14.16.1/bin/yarn
    npm: 6.14.12 - ~/.nvm/versions/node/v14.16.1/bin/npm
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Firefox: 87.0
  npmPackages:
    @sveltejs/kit: next => 1.0.0-next.107
    svelte: ^3.34.0 => 3.38.2

Your browser
Tested on Brave v90 and firefox v87

Your adapter (e.g. Node, static, Vercel, Begin, etc...)
Node, but i used svelte-kit dev

Severity
Annoying
Additional context
N/A

Duplicate of #844

My bad, didn't show up when i searched for similar issues


fix lua53 pkg-config library name

I'm torn on whether we should do this or just create a liblua5.3.dylib. Lua does the former, Lua51 does the latter.

Oh, interesting. I'll try to have a closer look tomorrow.

I see the difference, indeed. Any idea why the lua51 formula has that symlink, instead of fixing the .pc? Non pkg-config users expecting it?

Ah, it may well be our fault. That patch has been around forever, but we apparently wrote it slightly differently for lua51 and lua53. I'm fairly happy to accept this then. We can re-review if anyone complains, I guess.

Merged in https://github.com/Homebrew/homebrew-versions/commit/028e86632561f01d4b9451ca6f7e77f25dff671a. Thanks @Habbie! :beer: