Extremely slow compile times ~28882ms

Hi there, I'm suffering with really slow compile times when running npm watch script. Hopefully someone wiser than me can advise if I'm doing something wrong (I can't figure out what I've missed 😕).
I'm running laravel-mix v5.04, node v11.15.0, & npm v6.14.4.
Here is my webpack.mix.js file && package.json for reference:
const mix          = require('laravel-mix');
const tailwindcss  = require('tailwindcss');
require('laravel-mix-purgecss');
mix.sass('sass/app.scss', 'css/app.css')
  .options({
    processCssUrls: false,
    postCss: [
      require('autoprefixer'),
      tailwindcss('./tailwind.js')
    ],
  })
  .purgeCss({
    enabled: mix.inProduction(),
    globs: [
      path.join("**/*.html"),
      path.join("js/**/*.js"),
    ],
    folders: ['src', 'templates'],
    extensions: ['html', 'js'],
  })
  .setPublicPath('/')
  .browserSync({
    proxy: 'website-address.test',
    // https: true,
    files: [
        'css/**/*.css',
        'js/**/*.js',
        '**/*.html'
    ],
    browser: "google chrome"
  })

mix.js([
   'js/src/vendor.js'
], 'js/vendor.js')

mix.js([
   'js/src/app.js'
], 'js/app.js')

if (mix.inProduction()) {
    mix.options({
        terser: {
            terserOptions: {
                compress: {
                   drop_console: true
                },
                output: {
                  comments: false,
                }
            }
        }
    });

    mix.version();
}

{
  "name": "Univers",
  "private": true,
  "scripts": {
    "dev": "npm run development",
    "development": "cross-env NODE_ENV=development node_modules/webpack/bin/webpack.js --progress --hide-modules --config=node_modules/laravel-mix/setup/webpack.config.js",
    "watch": "cross-env NODE_ENV=development node_modules/webpack/bin/webpack.js --watch --progress --hide-modules --config=node_modules/laravel-mix/setup/webpack.config.js",
    "hot": "cross-env NODE_ENV=development webpack-dev-server --inline --hot --config=node_modules/laravel-mix/setup/webpack.config.js",
    "prod": "npm run production",
    "production": "cross-env NODE_ENV=production node_modules/webpack/bin/webpack.js --progress --hide-modules --config=node_modules/laravel-mix/setup/webpack.config.js",
    "export-tailwind-config": "tailwindcss-export-config --config=/tailwind.js --destination=sass/base/tailwind-configs --format=scss --flat=true"
  },
  "author": "Studio Andrew Ireland",
  "devDependencies": {
    "browser-sync": "^2.26.7",
    "browser-sync-webpack-plugin": "2.2.2",
    "cross-env": "^7.0.2",
    "imports-loader": "^0.8.0",
    "laravel-mix": "^5.0.4",
    "laravel-mix-alias": "^1.0.2",
    "laravel-mix-purgecss": "^4.1",
    "purgecss-webpack-plugin": "^2.1.2",
    "sass": "^1.26.3",
    "sass-loader": "8.*",
    "tailwindcss": "^1.3.4",
    "tailwindcss-accessibility": "^1.0.0",
    "vue-template-compiler": "^2.6.11"
  },
  "dependencies": {
    "@swup/body-class-plugin": "^1.0.2",
    "@swup/debug-plugin": "^1.0.3",
    "@swup/ga-plugin": "^1.0.1",
    "@swup/gtm-plugin": "^1.0.2",
    "@swup/js-plugin": "^1.0.4",
    "@swup/preload-plugin": "^1.0.3",
    "@swup/scroll-plugin": "^1.0.6",
    "balance-text": "^3.3.0",
    "flickity": "^2.2.1",
    "flickity-imagesloaded": "^2.0.0",
    "gsap": "^3.2.6",
    "imagesloaded": "^4.1.4",
    "overlayscrollbars": "^1.12.0",
    "scroll-lock": "^2.1.4",
    "smoothscroll-polyfill": "^0.4.4",
    "swup": "^2.0.9",
    "vanilla-lazyload": "^15.1.1"
  }
}

Is it also slow when recompiling the code? On the first compile it will be slow, but for recompiling it should be super fast when running npm run watch :)

I also experience compile time problems.
Most of the time it is busy compiling sass code and stuck with sass-loader. For my project initial compiling can take up to 20s and recompiling up to 10s.
Apparently this is a common problem with sass+webpack:
https://stackoverflow.com/questions/56262137/slow-sass-loader-build-times-with-webpack
https://github.com/webpack-contrib/sass-loader/issues/296

this kinda helped me:
https://gist.github.com/Jossnaz/7cf182e794e515d068159ad71fcf7855
I use yarn hot to build
tailwind is set to watch. And incremental watch builds seem to work
initial build time is still almost 30s or so. BUT: the tailwind build runs in a separate process, thus you are free to go coding right away.

I answered this question in another issue a few hours ago. I get a 26 second initial compile time (There's not really any way around this given the 20k lines of tailwind utilities) but my subsequent builds all take around 150ms.
Here's my setup: https://github.com/JeffreyWay/laravel-mix/issues/2470#issuecomment-699942687

Hey, everyone. I'm not sure if this is related to a Mix bug or not. I use Tailwind, too, but have never suffered from 30 second watch recompiles.
It could be related to many things. If someone can send me a set of steps to follow, I can look into it further.

I also seem to have an issue with extreme long compilation times with Laravel-Mix 5.0.4. When I downgrade to 4.1.4, the problem is gone.
Any update on this issue?

Any update on this issue?

I think laravel mix 6 is the way to go. At least, once plugins started fixing their build issues (new File becomes unavailable)

The command hung for minutes, and when I did a Ctrl+C it kicked into action.
May be worth a try!


Add ZPOPMIN and ZPOPMAX commands

Redis 5.0.0 adds two new commands for working with sorted sets: ZPOPMAX and ZPOPMIN. This PR adds support for the two commands and some corresponding test coverage.
The return values of ZPOP* differ from e.g. ZRANGE WITHSCORES in that:

they always return scores despite not having a WITHSCORES option
the elements of the pair are the other way round fixed upstream

Please feel free to rename zset_force_score_pairs if you can think of a better name.

This actually fails with non-numeric values.  The function that works should look something like:
def zset_force_score_pairs(response, **options):
    "Return the response as a list of (value, score) pairs"
    score_cast_func = options.get('score_cast_func', float)
    it = iter(response)
    ret = list(izip(imap(score_cast_func, it), it))
    return [(i[1],i[0]) for i in ret]

Yes you're absolutely correct, my mistake. Will update

@andymccurdy the release of Redis 5 is now imminent (https://twitter.com/antirez/status/1045702914159902720)
Please could you look over this PR. Thanks!

Redis 5 is released since a while and no support for new redis commands. I will find myself some better library since this obviously isnt maintained any longer.

Hello @rufoa
I didn't check the open PRs and I've also implemented these in #1037, oh well.

No problem. Yours is better anyway! Andy should accept your PR instead of
this one.
On Thu, 1 Nov 2018, 14:04 Itamar Haber<EMAIL_ADDRESS>wrote:

Hello @rufoa https://github.com/rufoa
I didn't check the open PRs and I've also implemented these in #1037
https://github.com/andymccurdy/redis-py/pull/1037, oh well.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/andymccurdy/redis-py/pull/989#issuecomment-435040880,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABxzqNLVt3SXU1LBmYnL8PbpIR7UmPqXks5uqv96gaJpZM4UXeAd
.

the changeset @itamarhaber submitted has been merged in. thanks!


Tag version 2.1.2

The FileProvider fix from czenker is currently not available as a released version. Could you create a release tag?

Is actually v2.2.2 - but that includes the FileProvider fix.  Thanks!


"Supported" is confusing Bing

I asked Bing for supported versions of .net framework and got this back:

It took me a minute to figure out what was happening, but I believe it is confused because out-of-support Windows versions are listed as "Supported", as there are two different meanings in play. Windows supports certain versions of .NET Framework, but those versions of Windows are out of support.
Maybe the OSes should be separated into supported and unsupported OS sections and "Supported editions" could be simplified to "Editions"?

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

ID: 8afd1f96-b008-8b2e-bf3e-89d91f8cca5e
Version Independent ID: 2010bd32-59f6-823c-f193-27439e12798d
Content: .NET Framework system requirements - .NET Framework
Content Source: docs/framework/get-started/system-requirements.md
Product: dotnet-framework
GitHub Login: @gewarren
Microsoft Alias: gewarren

This isn't something that we have direct control over, while I agree it's not ideal the source material that is cited does correctly describe the support details. I think we should close this is it's not a docs issue. Do you agree @gewarren?

That is kind of interesting. I wonder if we should use the word "compatible" instead of "supported"? Or perhaps split it into supported and out-of-support versions, like @mairaw did here: https://github.com/dotnet/core/blob/main/os-lifecycle-policy.md?WT.mc_id=dotnet-35129-website.

I realize that this is closed, but I disagree with the change as it was done.
The issue is that this makes it seem like .NET Framework 4.8 cannot be installed on Windows versions that are no longer in Mainstream support.
If a link to the migration guide article was included on that page with a note that somehow mentions that it lists the operating systems where it can be installed, that would be better.


Release: 0.10.x

Summary
This branch prepares a suite of changes for collective release under the 0.10.x series. This includes several (small) backward-incompatible changes, which are documented below.
PR Tree

#86
#84
#82
#72
#73
#76
#71
#70
#68
#63

Dependency updates

#93
#94
#95
#96
#97
#98

Related issues

#85
#78
#67
#66

Breaking changes

The default_executable_name attribute and parameter has been renamed to executable_name.
This is because there is no other executable name parameter; setting this attribute will unconditionally change the output binary name.

Legacy GVM is only usable with the legacy rules.
Previously, it was possible to use a rather old version of GraalVM (Java 11-era) with the new-style rules. This use is no longer supported because older versions of native-image show incompatibilities with Bazel's process wrapper.

Native image builds will now respect Bazel flags like --compilation_mode=opt.
Defaults have been left to GraalVM to maintain backward compatibility with the default case, which builds binaries with -O2 (optimizations on). However, this is still a breaking change because the behavior will be different if --compilation_mode was being explicitly set to any value other than opt. Note that Bazel's default value of fastbuild also applies here, so passing no value at all will result in a behavior change (opt → fastbuild).

The order of options passed to native-image has changed.
Attributes like extra_args now pass later on the command line (last, actually, for extra_args); because native-image options are last-wins, this should allow overriding any of the flags injected by the rules. This should offer users a path forward even if the flags passed by the rules are incompatible in some way. This change in the order of action arguments may result in cache misses. Otherwise, this change should be transparent to rule users.

Hermetic build environment improvements.
The parameter use_default_shell_env is no longer passed to the underlying native-image compile action; instead, the toolchain environment is assembled for strict use. Native compiler environment variables like INCLUDE, LIB, DEVELOPER_DIR, and SDKROOT no longer need to be passed via --action_env.

Toolchain targets and registration.
See below for an exhaustive description, but basically, the registration calls for toolchains need to change, but only for users who are consuming via Bzlmod.

Breaking changes: Toolchain registration
When registering toolchains in a Bzlmod installation of these rules, the target @graalvm//:all must be changed to two toolchain registrations, based on the desired functionality:
Register the Java toolchain:
register_toolchains("@graalvm//:jvm")

Register the GVM toolchain:
register_toolchains("@graalvm//:sdk")

I have added a short tour above @fmeum of how I've organized these changes. I set use_default_shell_env on Windows which seems to fix the compile issues there (I no longer have access to my previous testing VM -- working on getting that running again).
If you have access to better Windows CI and see issues with that, please feel free to revert (but my understanding is that use_default_shell_env is good and the strictest possible option, so long as we don't need to override env). Since Windows builds pass with this on, I'm guessing Bazel is injecting LIB and INCLUDE under the hood, since native-image relies on these to resolve Visual Studio.
Otherwise, the code you added for hermetic C++ is pretty much unchanged, aside from being refactored to narrowly miss the lambda via the legacy rules. Unwinding them turned out to be a great idea because the whole implementation feels much cleaner and easier to understand now.
Feel free to file PRs against this branch and we can gather/test changes for release at 0.10.x. I've also invited you to collaborate on the repository. Cheers!

@fmeum could you take a look?

@fmeum could you take a look?

On it. It worked in Bazel's CI, so I wonder why this breaks here.

@sgammon https://github.com/sgammon/rules_graalvm/pull/81 works for me locally on Windows.

I also added a missing attribute in a separate commit. This is working for me locally now, I am also rerunning the Bazel CI checks.

@fmeum nice work, it looks like all checks are passing locally. This doesn't include the toolchain repo separation yet but it can, that change should be quick and I'd love to get it in under this release.

@fmeum merged in #86 which offers these changes:

feat: detect and honor compilation_mode flag
feat: build with debug settings if compilation_mode=dbg
feat: ability to build a shared library
feat: build with tool:coverage if coverage is enabled and a test-only target is being built
docs: add doc which explains shared library feature
docs: add doc for built settings integration (more to come)

I figured this would be welcome in this release and they pass all gates. In particular, the -c fastbuild mode is much better for local integration testing.

It looks like lazy fetching hasn't been realized yet: If I check out https://github.com/bazelbuild/bazel/pull/19361 and build some unrelated target (e.g. bazel build //src/main/java/net/starlark/java/eval:eval), GraalVM is still fetched.

@fmeum what do you think could be the issue? is it that graalvm_sdk (the actual toolchain type implementation) still resides in @graalvm?

@fmeum what do you think could be the issue? is it that graalvm_sdk (the actual toolchain type implementation) still resides in @graalvm?

Yes, I think that's it. You probably don't even need the aliases, but two separate repositories with one containing the files and the other containing the toolchain targets. register_toolchains(...) will follow aliases to register the toolchains to learn the constraints on them and in the process would fetch the repo.

See https://github.com/bazelbuild/rules_go/commit/b3ebe1f404abbe6d603e6de317338353b9c7a297 for a commit that realized this in rules_go.

but two separate repositories with one containing the files and the other containing the toolchain targets

@fmeum but isn't this how it's arranged now? toolchain calls only occur within the x_toolchain_config_repo. The call I'm worried about is graalvm_sdk which is a toolchain type implementation -- does that count?
Apologies for the confusion. I will try to review that commit and see if I can figure it out.

The split looks good and a manual register_toolchains("@graalvm_toolchain_config_repo//:all") works as expected. The provided toolchain macro still registers "@graalvm" by default, which triggers an eager fetch.

@sgammon Looks good to me, except for a few references to @graalvm that should probably use @graalvm_toolchains instead.

@fmeum okay, sounds good. I'll fix the toolchains piece, merge the Maven artifacts PR since it's relatively safe, do some final cleanup and issue a release. The remaining Bazel integration features can land in 0.11.x


Add and and or keywords

They could even have special prefixes but this would be really useful because it is common to have very long chains of booleans.
and(
  expr,
  expr,
  ..
)

It would parse to a new ast node that hold a Vec<UntypedExpr>
We can desugar it during typechecking into a binop chain

+1 cause the current and and or functions in aiken/list require looping over a list which is just wasting budget for no reason. Doing this we can have some ergonomic and basically free.
Another alternative is to have better formatting for Boolean chains.

Can the compiler optimize list.and in these cases?

@waalge that's a bit more complicated to this at this stage because the compiler has no notion of stdlib whatsoever. That means you'd need some heuristic to first detect that this is one of the case, and then apply the optimisation.
What @MicroProofs suggests is fairly straightforward as it can be done entirely as a syntactic sugar. That is, something that's fully handled by the parser / type checker; a shorthand to produce a more complicated AST.

I can only take your word for it on that - I have no grasp on the complexity of getting the compiler to recognize things like this pattern.
I think adding new key words is a marginal increase on user cognitive load - there would be two things which look quite similar.  And and or from list can no longer be pulled into the namespace.
I'm not in a position to provide a better fix. Chaining binops is also suboptimal. I use multiple expect True instead.

@waalge if this were to be a thing, I suspect we'd get rid of the list functions in stdlib.
That said though, I think just having the formatting look good would do a lot here

Really? I think you still need to operate on lists. It's only if their elements are determined at compile time then the compiler could recognize a more efficient compilation.


CMAKE_CXX_FLAGS set wrong and fails to build on M1 Mac

Required Info

Camera Model

Firmware Version

Operating System & Version
MacOS

Kernel Version (Linux Only)

Platform
PC

SDK Version

Language
C

Segment

Issue Description
I got this error when trying to build on MacOS Ventura (Apple M1 chip), which is the same error as #11897.
-- The CXX compiler identification is AppleClang <IP_ADDRESS>30022
-- The C compiler identification is AppleClang <IP_ADDRESS>30022
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /Library/Developer/CommandLineTools/usr/bin/c++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /Library/Developer/CommandLineTools/usr/bin/cc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Checking internet connection...
-- Internet connection identified
-- Info: REALSENSE_VERSION_STRING=2.54.1
-- Setting Unix configurations
-- Performing Test SUPPORTS_CXX14
-- Performing Test SUPPORTS_CXX14 - Failed
CMake Error at CMake/lrs_macros.cmake:17 (message):
  Project 'realsense2' requires C++14 or higher
Call Stack (most recent call first):
  CMakeLists.txt:53 (config_cxx_flags)

-- Configuring incomplete, errors occurred!

It seems that check_cxx_compiler_flag(-std=c++14 SUPPORTS_CXX14) fails while appleclang actually supports -std=c++14 flag.
I checked a trace of cmake and found that CMAKE_CXX_FLAGS may be set wrong and this is the cause of the above error.
On my MacOS the MACHINE variable is arm64-apple-darwin22.5.0, and it seems this is not handled correctly in CMake/unix_config.cmake at line 18-31.
    if(${MACHINE} MATCHES "arm-*")
        set(CMAKE_C_FLAGS   "${CMAKE_C_FLAGS}   -mfpu=neon -mfloat-abi=hard -ftree-vectorize -latomic")
        set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -mfpu=neon -mfloat-abi=hard -ftree-vectorize -latomic")
    elseif(${MACHINE} MATCHES "aarch64-*")
        set(CMAKE_C_FLAGS   "${CMAKE_C_FLAGS}   -mstrict-align -ftree-vectorize")
        set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -mstrict-align -ftree-vectorize")

As a result, the flags for arm-* machines (-mfpu=neon -mfloat-abi=hard -ftree-vectorize -latomic) are applied in my MacOS, and these flags causes error on appleclang.
By changing the above code to the following, the error was fixed and I could successfully build librealsense.
    if(${MACHINE} MATCHES "arm64-*" OR ${MACHINE} MATCHES "aarch64-*")
        set(CMAKE_C_FLAGS   "${CMAKE_C_FLAGS}   -mstrict-align -ftree-vectorize")
        set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -mstrict-align -ftree-vectorize")
    elseif(${MACHINE} MATCHES "arm-*")
        set(CMAKE_C_FLAGS   "${CMAKE_C_FLAGS}   -mfpu=neon -mfloat-abi=hard -ftree-vectorize -latomic")
        set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -mfpu=neon -mfloat-abi=hard -ftree-vectorize -latomic")

Hi @na-trium-144  Thanks so much for sharing your solution to a build failure on MacOS Ventura and linking it to the Ventura discussion at https://github.com/IntelRealSense/librealsense/issues/11897
I will also link it to https://github.com/IntelRealSense/librealsense/issues/11815 which is the central place on this support forum where I am directing Ventura users who are having problems.

Hi @na-trium-144  Do you require further assistance with this case, please?  Thanks!

Case closed due to solution achieved and no further comments received.


Policy validation - performance impact

Adding validation policies may affect API throughput. The following general principles apply:

- The larger the API schema size, the lower the throughput will be.
- The larger the payload in a request or response, the lower the throughput will be.
- The size of the API schema has a larger impact on performance than the size of the payload.
- Validation against an API schema that is several megabytes in size may cause request or response timeouts under some conditions. The effect is more pronounced in the Consumption and Developer tiers of the service.

May I ask for any performance test results that supports above mentioned sentences?
Thank you in advance!
[Enter feedback here]

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

ID: d6696a2f-0437-5238-0032-393351594956
Version Independent ID: 2fb919f6-699c-e79a-c76c-6534985e7259
Content: Azure API Management validation policies
Content Source: articles/api-management/validation-policies.md
Service: api-management
GitHub Login: @dlepow
Microsoft Alias: danlep

Hello @pomorsen - Thank you for your feedback! We will review and update as appropriate.

We do not have performance details to share and we advise our customers to run the load tests in their environments. The performance of the API Management service depends on many factors, including the configuration of the service (for example, validation policies, other policies, logging) and responsiveness of the backend services. We hope to soon release tooling to allow easy, on-demand load testing of the API Management service in the desired configurations.


Upgrade to LLVM 16

We using LLVM 16.0.6.

At the moment, we're having a few problems building the project under Ubuntu-latest:
/usr/include/stdio.h:205:27: error: use of undeclared identifier 'L_tmpnam'
extern char *tmpnam (char[L_tmpnam]) __THROW __wur;
                          ^
/usr/include/stdio.h:210:33: error: use of undeclared identifier 'L_tmpnam'
extern char *tmpnam_r (char __s[L_tmpnam]) __THROW __wur;

For some reason (unknown to me at the moment), the standard library failed to compile, so I'm going to ignore that and merge the PR with this issue #380.
NOTE: Compilation is successful on MacOS and ArchLinux.


separate completion list is not working.

Yes, "statement completion shows only shortened description" seem to be broken, I'll investigate what goes wrong.
But I don't understand what is not working in "separate completion list is not working", could you explain.

By ReadMe description, i thought that separate statement completion list for intrinsics is by default on. No issue.

Yes I should mention that it is switched off by default, simply because most people don't want to be bothered by it in regular C/C++ coding.


fix: updating branch

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Add logs for self-monitoring

Get loki urls to travel over Mimir cluster relation data

With the new implementation of the LogForwarder class, we probably want to change the content of relation data to be closer to what we need there; I'd pass, from the coordinator, a dictionary of {loki_unit_name: url}, so we can pass it directly to the log forwarder

I think that publishing the Loki endpoint in the same function as the Mimir config is creating some problems; I'll change things around a bit :)


Support for Disabling Redirects

Description
Hi there 👋 This project looks pretty neat, especially from its usage in markdown-lint-check 🎉
We are investigating the possibility of using markdown-lint-check via GitHub Actions to replace some wget link checking we have, but we have an interesting feature request. Currently this library hardcodes the acceptable redirects at (a maximum) of 8:
https://github.com/tcort/link-check/blob/cbcfa7e0cf51ae290d50994a2afd98b7e39af59e/lib/proto/http.js#L22
In our project documentation, we are dealing with an upstream documentation website that will sometimes move pages, but not setup proper redirects (instead all broken links just go to the homepage). Example pull request to fix some of these: https://github.com/terraform-providers/terraform-provider-aws/pull/12803/files
We are thinking that if we could configure the acceptable redirects to 0, then we could alert on those types of broken links.
Proposal
It would be super neat if this library supported configuring maxRedirects (especially to 0 in our case).
Please let me know if you have questions or if I can help contribute. 👍

Please let me know if you have questions or if I can help contribute.
Hi @bflad yes you can contribute of course. If you have an idea about implementing this, you can propose a pull request on this repository.


Feat: addons/system_state - role creation

System state role, to provide a last play state.

I don't understand this role. Would the use of ARA answer the need?

Yes. This is why i stopped developing this PR and put it in draft, to investigate external tools like ARA deeper.

I am closing this PR. External tools seems to cover the need.


[bump] package version for client

        @fluidframework/build-common:     0.20.0 (unchanged)
 @fluidframework/eslint-config-fluid:     0.22.0 (unchanged)
  @fluidframework/common-definitions:     0.20.0 (unchanged)
        @fluidframework/common-utils:     0.27.0 (unchanged)
                              Server:   0.1017.1 (unchanged)
                              Client:     0.32.2 -> 0.32.3
                     generator-fluid:      0.3.0 (unchanged)
                         tinylicious:      0.4.0 (unchanged)
                         dice-roller:      0.0.1 (unchanged)

I'm confused. You just bumped to 0.32.2. Why do we need this?

I'm confused. You just bumped to 0.32.2. Why do we need this?

I'm confused. You just bumped to 0.32.2. Why do we need this?

The instructions in the OneNote say to release and then bump the version. I had to bump the version before releasing 0.32.2 because it was not done after the 0.32.1 release.

I'm confused. You just bumped to 0.32.2. Why do we need this?

The instructions in the OneNote say to release and then bump the version. I had to bump the version before releasing 0.32.2 because it was not done after the 0.32.1 release.


Add possibility to disable configmap creation

As configmaps are sometimes managed by different parties (in our case it is a Helmchart that takes care of creating and destroying configmaps), it would be helpful to have a possibility to disable leader configmap creation on the library's side.

I added unit test org.springframework.cloud.kubernetes.fabric8.leader.Fabric8LeadershipControllerTest#whenNonExistentConfigmapAndCreationNotAllowedStopLeadershipAcquire

Thanks, it would be good to add some documentation about the property as well.

@ryanjbaxter I added documentation to _configprops.adoc


Feature Discussion: Full Text Search

This is a fork of the issue #49, to specifically talk about Full Text Search
We can use bleve to allow all content in the CMS to have high quality search.

Full text search
Facet based search

This is a global feature, and each Content Item interface needs to support a simple "GetBleveMapping" function interface, that is parsed back to the Main bleve code.
When content is added, deleted, updated bleve also needs to be told.
Designing it this way makes it very easy to integrate bleve, and i have found works well.
I want to add that the way Bleve works with mapping will help the Admin GUi allot too.
This is because Bleve has good support for Numeric ranges and datetime range. If this is a first class thing in the whole system it means the Admin GUI can then express these 2 types perfectly.
You can see what i mean here:
http://www.blevesearch.com/docs/Result-Faceting/
Then there is the Time when a user searches the system.
There are a few user story patterns for how search can operate.
First is a global search page. The idea is that on the left you express ALL the facets (which as just content types), and on the right you show the results.
Here is a Demo of exactly this:
https://ouava.github.io/#/pubsearch

Enter the word "test" in the top right search bar - here is a bug in the search and you have to keep trying searching in the top right. But it will work after 10 seconds.
then you will see results for all content facets that contain the text you searched for.
then you can use the facets on the left to drill down.

As you can see, with this approach, you can use the search with any pre scoping you need also.
Maybe you just want to search for Emails by Person X ? So the content type is only Emails, and the Author is onlx X.
So you might have a users page about Email, and so you can use the global search system to search for emails and show the results using the same facets system (if you want).
Lastly, after you have done your search, how do you get your documents ? The ID's to the Content Items (or Documents) are saved in bleve. All you as a programmer have to do is take the ID, and ask the main CMS for it.
You can store the Content / Docuemnts inside Bleve too BTW. But this CMS system has a nice wy to do that already. SO its seem silly.
Thats it for now. I hope this opens up discussion on this.
Bleve uses boltdb, but it can use other Database engines.
BoltDB is nice because of its portability. You can run it on mobiles, desktops, servers with no code change.
Bleve can scale out using aliases.
See this and demo code : http://www.philipotoole.com/increasing-bleve-performance-sharding/bleve-corrected/

Question was:
Do you think we could extend Item to implement a default mapping interface so the Ponzu developer wouldn't need to do this themselves in their content file?
Answer:
Yes I think you can use a default mapping on the types that users generate. That's exactly what I was also thinking. But you need to provide a few more specific types that bleve needs as first class types in the core of ponzu that users specify in their generated types.

numeric range
datatime Range
maybe one or two other. Dont have time to look right now

These are needed for bleve in more complex search cases which you will hit.
The common GUI widgets can then be fully aware of these first class types
as well as bleve. Numeric range and datetiem range always need specific widgets !
I think it closes off lots of potential bugs too.
Here is another example of similar thinking in terms of first class types to provide data sync. Might be good idea.
I have raised it as a new FeatureIssue here: #53

@gedw99  -
Although it's been some time since we discussed the full text search integration via Bleve, I have an initial version in the ponzu-dev branch. If you'd like to check it out and let me know if:

I have any glaring mistakes
You have improvements or suggestions

I would greatly appreciate it! I'm hoping to merge a cleaned up version of that branch early next week.
All of the search-related code is in this compare: https://github.com/ponzu-cms/ponzu/compare/ponzu-dev

@nilslice thankyou. your really diligent. I figured it would be forgotten.
So i had a look at the code. It works, but is missing a ton of things needed IMHO.
If you want a nice clean example i recommend papernet:
https://github.com/bobinette/papernet/blob/master/bleve/paper.go

see the way boltdb and bleve are cleanly seperated

https://github.com/bobinette/papernet/blob/master/bleve/mapping.json

Nice and easy way to add mapping.

He has wrapped all dependencies like boltDB and bleve nicely. Its elegantly done and keep suse out of trouble.
I also work with it a bit.
You could take the wrappers and then your good  !!
We should be clear about goals and scoping too. My scope would be:

BoltDB storage only for now. We can use Alias's later to allow multiple servers.
Clear delineation between search storage and business data storage because they have different HA patterns later. Also in the workflows you want an emergent messaging style archi so that you can setup as many stores as you need for different things. We have 2 now but i bet you might add minio later. Not suggesting you need to add NATS or that complexity.
Facets support with a data driven GUI. So when new terms are added you see it in the global search page when you type any search term.
golang structure to support bleve tagging ? Would be neat i think. Seen it before.
18n from the start of course

i saw the changes with regards to the main architctural points in the issues.
Really looks great, and more useful to a broader audience i feel now. I have to give it a whirl again.
I think we can start adding examples, so people can quickly get going and increase participation.
Having formal example, also makes it easy for others to see how they can structure their own apps to use ponzu. It can be a little wierd at first since its a CMS

Thank you for the thoughtful response -- I agree there is much more to do to take full advantage of Bleve's features. From the Papernet example, I see how I should wrap Bleve's API a bit more - thank you for sharing that example.
My main goal with this integration at the moment is to enable a user to get efficient search from an API endpoint without requiring much (or any) configuration by the Ponzu user to enable phrase matching against indexed content. The use case for Ponzu is outside client conducting a search via HTTP like /api/search?type=Song&q=Us+and+Them
In a later release, I think doing more of a Admin-focused feature set would be cool, plus adding better support to the API for bleve's numeric and date range queries (which I'm a bit confused by to be honest). When you refer to a Data driven GUI using Facets, I don't really understand what that would look like - I apologize for my rather naive abilities and familiarity with Bleve (and search in general)!
From a Ponzu-user's perspective, I want to know if search indexing should be done by default (ALL your content types are indexed) with the option to disable indexing, OR if it should be enabled on a type-by-type basis. Already the search results via http api endpoint will respect the other rules (item.Hideable & item.Omittable) enforced by a Ponzu user, but I could see performance reasons to not index every content type.. so I'm not sure if auto-enabling is the best way to go..
I agree with your reasoning for external folders for examples to run tests against and demonstrate features -- and I would like that to be a community effort if possible. I will add a repo within the Ponzu organization to host these projects.

I really appreciate you asking me more...
i just wanted to point you to a lib that shows how to use it better. Hope this is ok.
About bleve and the numeric and datatime range queries, they are pretty simple and just more terms. With range queries like tis you need to precalcuate your min and max and then pick the bracket in the middle. Liek "low, medium and high" in their examples. You can find the bottom and top by dong a quick bolt search on each index mapping using range terms. I guess thats a little weird but its how it works and is the only place i needed to add to bleve to get it working.
The word "terms" and "facets" can be confusing for sure. they are the same thing in a way.
Terms are what you use to index data against with different term types.
Facets is a generic industry word for using those terms in the GUI.
At least thats how i have arrived at this understanding on my own. I am probably not absolutely correct :)
BTW marty on the bleve google groups is super supportive. I asked him lots of newbie questios and as long as i put in the effort to get half way he was really helpful.
I think a good example is Amazon shopping to talk around this.
The user starts  at the free text search field on the home page.
The user types "tv", and get a result set of TVs in a list view.
Once the user picks one you are at that TV's item page (which you got from bolt), and on the left the facets show up (which came from bleve), showing the different facets like manufacturer, screen type, screen size, vesa plate, etc, etc.
The facets are pre "checked" (as check boxes and sliders) to match the TV your current looking at BTW:)
Then you can use the facets to widen or narrow your search.
Its very powerful and intuitive. I guess i can say its intuitive since its amazon and so designed for every man and his dog to be able to use :)
So maybe you can see that this is all data driven. The bleve terms mapping that occurred when you indexed a Record (or document has they call it in Bleve) determines the facets. Bleve actually returns back to you the facets ( it calls them facets in the JSON data and golang structs if i recall ) and all you have to do is display it like Amazon does as a tree.
Then you check box or range slide facets and send the query back in.
there is no logic from the programmer to implement. Its very cool and i hope you agree.
The bleve search name space is where all the action is and it does the request response cycle fully for you, and all you need to do is parse it. I  tend to encapsulate it it and not use the JSON.
Mostly because i find separating my binding and transport to me better architecturally, but thats not appropriate for this context.
Hopefully what i wrote above helps ??
You asked if i think search indexing should be done by default.
I think YES :) Just because your CMS is designed to show you all your data types, and so why not index all that data ? Then you can have a generic search page just like Amazon. But  Materials Design of course :) I saw one done with material design in Dart and it was the sexy.
Its also very easy thanks to Bleve being so well designed so why not .
I always set things up so that any mutation to the DB hits the DB and hits bleve. Just that simple.
You also ask if once a user gets to a type should we show the search. Well once you get to a Type, the search facets can be found that its is a member of, just like the Amazon example. SO yes i would say :)
Also with bleve you can scale it horizontally on many servers. They call this Alias's, and its more work but generic. Just so you know its there. I am using mini allot to make any data storage system HA in this way at the moment as a bit of an experiment. Mini has a dsync repo that does one thing and one thing only - Its does record locking across the cluster. This allows minio to scale up to 16 servers, and have full linearizability. https://github.com/minio/dsync
Just mentioning this in case your curious. Its damn useful because you can use it in a myriad of ways. We can talk about that in another discussion if you want.
Example folder. Cool. Its a hack that way i did it but works well enough because its a full tear down and rebuilt so no side effects.  I used sh scripts, but i am guessing you will make it more golang ish.
BTW i checked out buffalo that you wrote about in another issue.
https://github.com/gobuffalo
It seems to be a good match with ponzu. They use different DB storgae. Ponzu uses boltdb and buffalo uses the sqlx ones (sqlite, etc etc). But there feels to be some synergy there in that they both are frameworks.
I typically use bleve with any DB, and use bleve for search and when i get down to a record type fall back to using basic sqlish queries (like " select x from y where firstname = z " type of thing). The two work well i find for almost anything.

@gedw99 -
Thank you so much for all your input and assistance getting this feature to a usable point! Although I didn't implement it to your full recommendation, I think it is a place where the majority of Ponzu users will see the benefits and once more interest is present, we can revisit the topics discussed to further the functionality.
I am going to close this issue now, since #116 is merged. However, please keep ideas logged and let me know if you think of other improvements to the existing implementation.
Steve

@gedw99

Example folder. Cool. Its a hack that way i did it but works well enough because its a full tear down and rebuilt so no side effects. I used sh scripts, but i am guessing you will make it more golang ish.

I added an examples repository, if you'd like to share any of your examples.
See it here:
https://github.com/ponzu-cms/examples


Fix parsing doubles in locales that have a different decimal point.

This fixes https://github.com/skystrife/cpptoml/issues/45 in a portable way.

Thanks!


GPU tcmalloc: large alloc error while dm.data.process()

I'm trying to run the tutorial notebook on my GPU and when running the tutorial script:
train, validation, test = dm.data.process(
    path='sample_data/itunes-amazon',
    train='train.csv',
    validation='validation.csv',
    test='test.csv')

I get the following output:
Reading and processing data from "sample_data/itunes-amazon/train.csv"
0% [############################# ] 100% | ETA: 00:00:00
Reading and processing data from "sample_data/itunes-amazon/validation.csv"
0% [############################# ] 100% | ETA: 00:00:00
Reading and processing data from "sample_data/itunes-amazon/test.csv"
0% [############################# ] 100% | ETA: 00:00:00tcmalloc: large alloc<PHONE_NUMBER> bytes == 0x55dd7d8ac000 @  0x7ff96db501e1 0x7ff96df237e8 0x7ff91ede0f7b 0x7ff91ede1647 0x7ff91ede7058 0x7ff91ede7d99 0x7ff91edaf775 0x7ff91edd247e 0x55dcf69a9114 0x55dcf69a9231 0x55dcf6a0de8f 0x55dcf69626f9 0x55dcf6963805 0x55dcf697e943 0x55dcf69bd06a 0x55dcf69bde28 0x55dcf6a0d626 0x55dcf69a868b 0x55dcf6a0d6c9 0x55dcf69626f9 0x55dcf69a8917 0x55dcf6a0a0a6 0x55dcf69626f9 0x55dcf6963a30 0x55dcf697e943 0x55dcf69bd06a 0x55dcf69bde28 0x55dcf6a0e3d7 0x55dcf69a868b 0x55dcf6a0d6c9 0x55dcf69629da
tcmalloc: large alloc<PHONE_NUMBER> bytes == 0x55ddf58ae000 @  0x7ff96db501e1 0x7ff96df237e8 0x7ff91ede0f7b 0x7ff91ede1647 0x7ff91ede7058 0x7ff91ede7d99 0x7ff91edaf775 0x7ff91edd247e 0x55dcf69a9114 0x55dcf69a9231 0x55dcf6a0de8f 0x55dcf69626f9 0x55dcf6963805 0x55dcf697e943 0x55dcf69bd06a 0x55dcf69bde28 0x55dcf6a0d626 0x55dcf69a868b 0x55dcf6a0d6c9 0x55dcf69626f9 0x55dcf69a8917 0x55dcf6a0a0a6 0x55dcf69626f9 0x55dcf6963a30 0x55dcf697e943 0x55dcf69bd06a 0x55dcf69bde28 0x55dcf6a0e3d7 0x55dcf69a868b 0x55dcf6a0d6c9 0x55dcf69629da

And then it hangs.
I checked if the GPU is visible through torch and it is.
Is there a way of reducing the memory allocation?

We ran into the exact same issue. Did you manage to find a workaround if I may ask?

how much CPU & GPU ram do you have? we require a minimum of 12GB of CPU memory for loading fastText embeddings.

We tried running it using Google Colab which has 25GB of CPU memory and 10GB GPU memory. It crashes due to too much memory usage.
Also tried running it on our local system with 16GB and 4GB respectively, same result.


New feature: call hierarchy support

It is a feature common in other ides like eclipse or intellij
It shows the graph of all transitive calls of a function, in a tree:

The nodes are selectable (shows info in the right pane) and navigable with ctrl+click
In the right pane other info like the possible multiple calls in the same funcion, with the line number (also navigables to the concrete line)

It could use @wz1000 HieDB, like the recent find references implementation
It is supported in lsp 3.16:

https://microsoft.github.io/language-server-protocol/specifications/specification-3-16/
https://github.com/microsoft/language-server-protocol/issues/468

This would need add support for that feature in haskell-lsp:

lsp spec: https://microsoft.github.io/language-server-protocol/specifications/specification-3-16/
lsp-haskell issue: https://github.com/alanz/haskell-lsp/issues/134


FINERACT-1575 IPv6 workaround

Description
A bean has been added as a workaround for supporting IPv6 address, which are used in fully managed serverless environments (like Firebase, AWS Amplify, GCP, Azure, Azure App Service).
Checklist
Please make sure these boxes are checked before submitting your pull request - thanks!

[ ] Write the commit message as per https://github.com/apache/fineract/#pull-requests

[ ] Acknowledge that we will not review PRs that are not passing the build ("green") - it is your responsibility to get a proposed PR to pass the build, not primarily the project's maintainers.

[ ] Create/update unit or integration tests for verifying the changes made.

[ ] Follow coding conventions at https://cwiki.apache.org/confluence/display/FINERACT/Coding+Conventions.

[ ] Add required Swagger annotation and update API documentation at fineract-provider/src/main/resources/static/api-docs/apiLive.htm with details of any API changes

[ ] Submission is not a "code dump".  (Large changes can be made "in repository" via a branch.  Ask on the developer mailing list for guidance, if required.)

FYI our guidelines for code reviews are at https://cwiki.apache.org/confluence/display/FINERACT/Code+Review+Guide.

Same here, never used it before. But, just to be on the cautious side: can we add an "@ConditionalOnProperty" with a property that is set by default to false? That way those who want to use this can, and those who are unaware are not getting something they didn't order.
Talking of unaware: could we add some documentation to explain what this is for and why people would want it?

Closing as it's obsolete. Please reopen if you're planning to work on it.


template_tags error seen for DUNE

Hi,
See this error below - (template_tags function not defined in dune.py file)
10/21/2024 22:33:27.629: MoverTask[fardet-vd_fd_mc_2023a_reco2_reco2_dunevd10kt_nu_1x8x6_3view_30deg_geov3.fcl_reco2_v09_81_00d02_merged_skip000000_lim000020_stage_makecaf_20241009T023641.root]: data file already exists at the destination and has correct size. Not overwriting
10/21/2024 22:33:27.641: Mover:
Mover failed: hd-protodune_detector_run0000028005_physics_standard_reco_stage2_calibration_protodunehd_keepup_root-tuple_merged_skip000050_lim000014_testing_20241009T023519.root status: queued error:
Traceback (most recent call last):
File "/exp/dune/app/home/dunepro/venvs/declad/lib64/python3.11/site-packages/pythreader/task_queue.py", line 210, in run
result = task.run()
^^^^^^^^^^
File "/exp/dune/app/home/dunepro/2x2_Minerva/declad_merging/declad/declad/mover.py", line 389, in run
metacat_dataset_did = custom.metacat_dataset( self.FileDesc, metadata, self.Config)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/exp/dune/app/home/dunepro/2x2_Minerva/declad_merging/declad/declad/custom/init.py", line 163, in metacat_dataset
return rucio_dataset(desc, metadata, config)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/exp/dune/app/home/dunepro/2x2_Minerva/declad_merging/declad/declad/custom/init.py", line 155, in rucio_dataset
meta.update(template_tags(metadata))
^^^^^^^^^^^^^
NameError: name 'template_tags' is not defined

Closing this as fixed in #44 , which was mistagged as fixing #41...

After fetching latest tag 2.3.1
dunepro@dunedecladgpvm02 declad]$ git status
HEAD detached at 2.3.1
Untracked files:
(use "git add ..." to include in what will be committed)
declad/custom/init.py
nothing added to commit but untracked files present (use "git add" to track)
10/22/2024 13:06:26.596: Mover:
Mover failed: fardet-vd_fd_mc_2023a_reco2_reco2_dunevd10kt_nu_1x8x6_3view_30deg_geov3.fcl_reco2_v09_81_00d02_merged_skip000020_lim000020_stage_makecaf_20241009T024029.root status: downloading metadata error:
Traceback (most recent call last):
File "/exp/dune/app/home/dunepro/venvs/declad/lib64/python3.11/site-packages/pythreader/task_queue.py", line 210, in run
result = task.run()
^^^^^^^^^^
File "/exp/dune/app/home/dunepro/2x2_Minerva/declad_merging/declad/declad/mover.py", line 399, in run
metacat_dataset_did = custom.metacat_dataset( self.FileDesc, metadata, self.Config)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/exp/dune/app/home/dunepro/2x2_Minerva/declad_merging/declad/declad/custom/init.py", line 163, in metacat_dataset
return rucio_dataset(desc, metadata, config)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/exp/dune/app/home/dunepro/2x2_Minerva/declad_merging/declad/declad/custom/init.py", line 155, in rucio_dataset
meta.update(template_tags(metadata))
^^^^^^^^^^^^^
NameError: name 'template_tags' is not defined
Should this routine be added to dune.py in custom area -
        def template_tags(metadata):
            return {}

adding the dummy routine to dune.py appears to have fixed the issue.

That should work but you're not supposed to have to add that; it should have defined it that way if the import didn't work...

I am likely doing something wrong but can not see it. -
here is PYTHONPATH -
PYTHONPATH=/cvmfs/fermilab.opensciencegrid.org/products/common/db//../prd/sam_web_client/v3_6/NULL/python

So wait, you 're running 2.3.1 and still getting a "NameError: name 'template_tags' is not defined"?!?
--
Marc W. Mengel
Computer Services Senior Developer
Data Management and Applications
Fermi National Accelerator Laboratory
630 840 8256 office
www.fnal.govhttps://email.fnal.gov/owa/redir.aspx?C=JkVsWjSyHEuWV2kFJWNPyC9OyMKf7dEIp_yxPk9yfhfadE3mdLfI6T8Ztf8BVaJjzoIe8oJ5H0M.&URL=http%3A%2F%2Fwww.fnal.gov
https://email.fnal.gov/owa/redir.aspx?C=JkVsWjSyHEuWV2kFJWNPyC9OyMKf7dEIp_yxPk9yfhfadE3mdLfI6T8Ztf8BVaJjzoIe8oJ5H0M.&URL=http%3A%2F%2Fwww.fnal.gov
Connect with us!
Newsletterhttps://email.fnal.gov/owa/redir.aspx?C=-29AtpIe40SAvGF4aiNr86KvVjap7dEIuxIdg8Q5bYa9sT7QTFVhxhVnl8zwXkEuBSuAQLRkEfg.&URL=http%3A%2F%2Fwww.fnal.gov%2Fpub%2Ftoday | Facebookhttps://email.fnal.gov/owa/UrlBlockedError.aspx | Twitterhttps://email.fnal.gov/owa/redir.aspx?C=-29AtpIe40SAvGF4aiNr86KvVjap7dEIuxIdg8Q5bYa9sT7QTFVhxhVnl8zwXkEuBSuAQLRkEfg.&URL=https%3A%2F%2Ftwitter.com%2FFermilabToday

From: Doug Benjamin @.>
Sent: Tuesday, October 22, 2024 1:22 PM
To: fermitools/declad @.>
Cc: Marc W Mengel @.>; State change @.>
Subject: Re: [fermitools/declad] template_tags error seen for DUNE (Issue #42)
[EXTERNAL] – This message is from an external sender
I am likely doing something wrong but can not see it. -
here is PYTHONPATH -
PYTHONPATH=/cvmfs/fermilab.opensciencegrid.org/products/common/db//../prd/sam_web_client/v3_6/NULL/python
—
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_fermitools_declad_issues_42-23issuecomment-2D2429955326&d=DwMFaQ&c=gRgGjJ3BkIsb5y6s49QqsA&r=mA4qFs7Hfq-joWhWXnsXfA&m=JKSlKOSfa_B3GcLlggnLQ6W9dmxdbZljKXgcdDcRq4ph5Hwa4APpWHA1Akrk8SHH&s=jItokMxl7TJXGnbhBka_RG10iH8b9RhFMsGcU0kYSvA&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AAB6GV7INUEAJYSWEEENM6LZ42JURAVCNFSM6AAAAABQLRHFS6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMRZHE2TKMZSGY&d=DwMFaQ&c=gRgGjJ3BkIsb5y6s49QqsA&r=mA4qFs7Hfq-joWhWXnsXfA&m=JKSlKOSfa_B3GcLlggnLQ6W9dmxdbZljKXgcdDcRq4ph5Hwa4APpWHA1Akrk8SHH&s=LLj05SwXAZFjOMYu1ZwqxQishNV8zTlMno7o99nq2sU&e=.
You are receiving this because you modified the open/close state.Message ID: @.***>

yes hence the hack

Aaargh.. I'm not reading the stack trace right; you're getting that failure in "/exp/dune/app/home/dunepro/2x2_Minerva/declad_merging/declad/declad/custom/init.py"
which is the symlink to the custom/dune.py ...

Yes we need to add one there in the custom module, if we're calling it in the custom module... sigh.

It's the mover.py code that's protected against it not being in the custom module; but when I moved that rucio_dataset() code into the custom module, it then needs to be in there... Sigh.

Tagged 2.3.2

It works thanks


In Python3 - license text is displayed as a byte type

When selecting to view a license request from the license requests table, the text shows prepended with b' and the quotes are escaped.
The full display is below:
Test License [Please Ignore]

Fullname

Test License [Please Ignore]

License author name

Short Identifier

testid

Source/URL

http://source1.com

Osi Approved

true

Notes

None

Standard License Header

None

Text

b'

this doesn\'t match

'
Submission Date

Dec. 16, 2020, 11:12 p.m.

Resolved in PR #276

Resolved in PR #276

Resolved with PR #276

Resolved with PR #276


[4284] Update end academic cycle estimation

Context
Current end academic cycle estimation code predicts 2 months too long for course end date for course lengths 1 year or greater, update code to calculate the correct value.

 [S] end_academic_year estimate calculation needs tweaking in SetAcademicCycles service


feat: Add organisation-wide Prettier config

Description
Per this issue and discussion on our internal slack, this PR adds our organisation-wide Prettier config.
Note, this PR does not:

Reformat all the code. For reasons relating to git history, maybe it's more desirable to adhere to the new code style when each file is edited?
Enforce Prettier conformation with CI tool

 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.

I don't see any usage instructions. Is that not in scope for this PR?

@yknl Hey Ken, I left this PR to quite a bare minimum. I'd say usage is mostly down to a given developers set up/IDE config.
I could add a package json script to prettier everything, but then we should probably format the whole codebase now?

Is there a way to add an ignore file for prettier and gradually remove files from the list as we go?

Indeed there is a prettier ignore config we can use
We can add the result of find src -name "*.ts" to it – although would need some motivation in place to begin removing the files as we go. If we do this, we can format a handful of files now, and set up CI for it as well?

That sounds like a good approach to me.

Done, I've had to disable the eslint semi colon rule, which is now enforced by Prettier.

Thanks @kyranjamie! I see that npm run lint:formatting checks the files for conformance, but doesn't actually tell you what needs to be changed. I tried running the prettier command node_modules/.bin/prettier src/wallet.ts --write on a file but it seems to have added semicolons to every line. I'm guessing that it's not picking up the config. What is the correct command? Should it be added to scripts?

It doesn't say any files because of the .prettierignore, remove a few lines and run again and it'll show them.

It doesn't say any files because of the .prettierignore, remove a few lines and run again and it'll show them.

I meant that it doesn't tell you which lines in the files are in violation of what rules. But I suppose that is not a necessity as you can just run the prettifier and look at the diff.
Also, looks like there isn't a problem with the config, I just thought we were not requiring semi colons. And since we've voted to require semi colons, I think this PR is good to go.


Prevent needless getDiagnostic calls

Minor performance improvements in obvious places, e.g. calling $this->getDiagnostics in a loop.

Rebased, it now merges.

Thanks for the help!

:bow:


Not seeing Authorization headers

Hi @gmlewis 👋
I am using your library with oauth2.StaticTokenSource, and when the requests are logged I see no REDACTED header fields in the curl output.
Below you can see my whole http client configuration, excuse the mess :D
	transport := &oauth2.Transport{
		Source: oauth2.StaticTokenSource(
			&oauth2.Token{
				AccessToken: token,
			},
		),
		Base: &transportHeaders{
			modifiedSince: time.Now().AddDate(0,0,-1).Format("Thu, 22 Dec 2017 07:51:19 UTC"),
		},
	}

	ct := dbg.New(dbg.WithTransport(transport))

	transporta := &httpcache.Transport{
		Transport:           ct,
		Cache:               diskcache.New("cache"),
		MarkCachedResponses: true,
	}

	hcli := &http.Client{Transport: transporta}

The output is typically:
2022/03/11 10:43:53 curl -X GET \
  https://github.enterprise/api/v3/orgs/finn/teams/billing/repos?per_page=100 \
  -H 'Accept: application/vnd.github.mercy-preview+json' \
  -H 'If-None-Match: W/"3a3ae4bfaa7c0fadae157f58146911ccf7822215d16afc34ae7bef87e11f8dda"' \
  -H 'User-Agent: go-github'

Cheers,
emil

Hi @emilva - thanks for the report.  I'm unable to reproduce the problem you are seeing with that snippet.
Here's what I get:
$ go run main.go
# command-line-arguments
./main.go:25:9: cannot use &transportHeaders{…} (value of type *transportHeaders) as type http.RoundTripper in struct literal:
	*transportHeaders does not implement http.RoundTripper (missing RoundTrip method)
./main.go:26:4: unknown field 'modifiedSince' in struct literal of type transportHeaders
./main.go:32:17: undefined: httpcache
./main.go:34:24: undefined: diskcache
./main.go:38:2: hcli declared but not used

Can you please send me a complete program (without the token obviously) that can reproduce the issue you are seeing?

of course, I just sent parts of the configuration.
I managed to reproduce it with this simple go file:
package main

import (
	"context"
	dbg "github.com/gmlewis/go-httpdebug/httpdebug"
	"github.com/google/go-github/v43/github"
	"golang.org/x/oauth2"
)

func main() {

	ctx := context.Background()
	ts := oauth2.StaticTokenSource(
		&oauth2.Token{AccessToken: "xyz"},
	)
	tc := oauth2.NewClient(ctx, ts)

	ct := dbg.New(dbg.WithTransport(tc.Transport))
	client := github.NewClient(ct.Client())
	opts := github.ListOptions{Page: 100}
	client.Teams.ListTeams(ctx, "finn", &opts)

}

Output from go run main.go:
2022/03/11 13:51:28 curl -X GET \
  https://api.github.com/orgs/finn/teams?page=100 \
  -H 'Accept: application/vnd.github.v3+json' \
  -H 'User-Agent: go-github'

Thank you, @emilva !   Whups!!!  My bad.  I'll update the repo on how to use it.  Please try this instead:
func main() {

	ctx := context.Background()
	ts := oauth2.StaticTokenSource(
		&oauth2.Token{AccessToken: "xyz"},
	)
	// tc := oauth2.NewClient(ctx, ts)
	tc := &oauth2.Transport{Source: ts, Base: dbg.New()}

	client := github.NewClient(&http.Client{Transport: tc})

	// ct := dbg.New(dbg.WithTransport(tc.Transport))
	// client := github.NewClient(ct.Client())
	opts := github.ListOptions{Page: 100}
	client.Teams.ListTeams(ctx, "finn", &opts)

}

With this version, I now get this output:
$ go run main.go
2022/03/11 10:00:58 curl -X GET \
  https://api.github.com/orgs/finn/teams?page=100 \
  -H 'Accept: application/vnd.github.v3+json' \
  -H 'Authorization: <REDACTED>' \
  -H 'User-Agent: go-github'

Fixed documentation in: https://github.com/gmlewis/go-httpdebug/releases/tag/v0.0.5
Closing issue as resolved.

great, thank you @gmlewis!


[WIP] Fix Ubuntu py38 build

For some reason the python3.8 build installated the pytest.txt prior to tensorflow:
https://github.com/tensorflow/addons/runs/598441627
This caused issue since numpy was not available for scikit-image installation. I couldn't find documentation about why this order would be changed.
This is a more explicit order regardless.
Marking WIP -- confirming locally since CI won't test this until it merges

Maybe for pull requests, we could tests against python 3.5 and 3.8, and skip 3.6 and 3.7? That would help because it's not the first time we get problems when inserting a new python version. That would help with forward compatibility too.

Maybe for pull requests, we could tests against python 3.5 and 3.8, and skip 3.6 and 3.7? That would help because it's not the first time we get problems when inserting a new python version. That would help with forward compatibility too.

I changed it so all python builds are tested, these are done independently now so we shouldn't see any test time increase.

Since the pool of concurrent builds is shared with the rest of the TF org, if possible, it'd be best to skip builds that have a very low probability of failure. Once more and more repos are using github actions, we'll see the benefits.

https://help.github.com/en/actions/getting-started-with-github-actions/about-github-actions#usage-limits
I think the tf org has 180 concurrent builds total. And another limit is 50 concurrent macos jobs.

Great thanks!


[PW-8262] Fix invalid redirect url

Summary
If the payment response type is has the action type redirect the redirect url of the method AbstractPaymentMethodHandler::pay should be the url of the PSP, like Klarna for example, instead of the payment.adyen.redirect_result route which should be the target after visiting the PSP page.
Tested scenarios
Use Klarna for example within a headless context.

@acampos1916 @rikterbeek @peterojo @Morerice @AlexandrosMor @candemiralp @RokPopov @raoulritter We would need this for a project with a tight timeline. Could you please have a look at this PR? Since the AbstractPaymentMethodHandler is not easily modifiable without nearly copy the whole class it would be nice to have this fix ASAP.

Since this change the plugin works flawlessly with the Shopware PWA and Shopware Frontends, expect the credit cards, but there you provide a good documentation how to implement it.
Also Shopware does it the same way within their PayPal plugin, see https://github.com/shopware/SwagPayPal/blob/master/src/Checkout/Payment/PayPalPaymentHandler.php#L125
I guess you are handling the redirect correctly within the twig templates by a js script so this issue didn't popped up earlier.

Hello @GM-Alex,
Thank you for your suggestion and changing your PR. We created an internal investigation ticket to find out the actual cause of the issue because we haven't got any complaints about this issue before.
After this investigation we may approve your PR or create a new one depending on the issue/solution.
Best Regards,
Can

Hi @peterojo,
I'm afraid to tell you this but you are wrong. In line 382 (https://github.com/Adyen/adyen-shopware6/pull/361/commits/e2b872a37e7d3b85c7549c627f6533189008053a#diff-9c1eb916d49e2b822a34215d39a653eac2a404a5bc2b5a7aaba33e765083b777R382) you can see that we only redirect to the PSP (for example Klarna) if the action type is redirect and the url is set. So payments methods which do not involve a redirect action are directly redirected to payment.adyen.redirect_result. In addition payments which require a redirect cannot be finalized without the redirect, so this is obviously false and so your implementation is.
Furthermore I now tested my changes with the storefront and the headless approach and it works every think as expected. In addition the type parameter which defines if a payment requires a redirect and is checked by my changes comes from your module / api endpoints and for example for credit card, a none redirecting payment the type is, surprise, not set to redirect.
So please apologize my open words but it seems that you don't tested my modifications nor have a full understanding of your own plugin. So this behavior is imho very disrespectful to you clients, which can expect a working plugin, and also for contributors. I spend now some hours to prove this, which should be your job...
Please feel free to prove me wrong, by providing a counter example, with a step by step guide where my modification breaks the logic. But I'm nearly 100% sure that you aren't able to do so since I now tested my modifications with the default storefront.
Since this is an urgent modification we need for the upcoming going live of our customer please "really" review this PR again. We also can meet via Zoom, Teams or what ever you preferred to get this fixed asap. Thank you. Again sorry for the harsh words but this treatment feels not well.

@peterojo As assumed above you are fixing your faulty implementation via JS, within your ConfirmOrderPlugin. To verify this use my PR and disable the ConfirmOrderPlugin at the main.js file of your plugin, see https://github.com/Adyen/adyen-shopware6/blob/e55dcf65bc0ed1e142e83cec024e27116b74f191/src/Resources/app/storefront/src/main.js#L10 and rebuild the storefront by using bin/build-storefront.sh. Do the same without my modifications.
You will see that Klarna works, as expected, with my changes and don't work without it. Also my changes have not effect since you are handle it via your JS, so they also will not break anything. And this also shows that you didn't even tested my modifications...
FYI @acampos1916 @rikterbeek @Morerice @AlexandrosMor @RokPopov @raoulritter @candemiralp

Hi @GM-Alex!
Thanks for your feedback! Apologies for the miscommunication in my earlier response, and I didn't mean any disrespect. Your PR will not break other payment methods in the storefront usage of the plugin, but our documented approach for headless integrations which I have explained earlier is recommended for a reason. We would love to hear your feedback if you encountered any issues when making the additional payment-status call to get the Adyen API payment response?
I have reached out through our support agent to schedule a call so we can discuss it further.

The issue is that the default implementation from Shopware is not working without it properly.
So what's the reason for the approach for headless integrations which you have explained earlier?
I think not using the way of implementation which is Shopware providing makes no sense here. In addition since this is not breaking anything and works auto of the box with Shopware PWA and Shopware Frontends this is the way to go. Furthermore you would be also able to remove JS code for the storefront implementation with this change. Less code leads to less complexity and less complexity leads to better code. There is imho absolutely no reason to not change it.


Updates for MVAPICH2, Enable multigroup by default

These are updates from recent testing with MVAPICH2.
Note part of this is trivial (the MVAPICH2 bits) but it also enables the multigroup oserver by default if using IOSERVER nodes. This is based on conversations with @weiyuan-jiang. We don't affirm that this is an ideal setup (number of IONODES, etc), but it at least gets people closer to a better setup automatically.
Should be taken in concert with https://github.com/GEOS-ESM/GMAO_Shared/pull/198 (but not required, just should)

I did a C180 L181 run of GEOSgcm before and after this PR. Stock took 33:30 to run and with this 33:35. Given the vagaries of discover's disks, that's pretty much no impact.

Found a bug. Blocking until I can fix it!

Unblocking. Simple fix:
   # Finally multigroup requires at least two backend pes
   if ($NUM_BACKEND_PES < 2) set NUM_BACKEND_PES = 2


Add basic embeddings of content

All the data teams have done experiments that require embeddings of GOV.UK content.  Currently we each create our own embeddings, usually on our laptops, which is slow and annoying.  Instead, could we have a standard set of embeddings as part of the GovGraph data pipeline, accessible to everyone?
~BigQuery ML makes it easy to get embeddings by uploading a model (e.g. BERT) and writing a single query.~ It's severely limited to very small models of a certain type.

This is a great idea! +1

From a GCP point of view Vertex AI Matching Engine provides a low latency vector database.
The below might also be of interest.
Get text embeddings
Embeddings for Image


Support P2SH

The 3rd address type you can send to is a P2SH address.
An example: kaspa:precqv0krj3r6uyyfa36ga7s0u9jct0v4wg8ctsfde2gkrsgwgw8jgxfzfc98 (devfund)
These addresses are:

67 length addresses (kaspa: + 61 byte string)
version byte: 8

Fixes #45

Codecov Report
Patch coverage: 80.70% and project coverage change: -0.39 :warning:

Comparison is base (6475f5f) 93.33% compared to head (13832ec) 92.95%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #46      +/-   ##
==========================================
- Coverage   93.33%   92.95%   -0.39%
==========================================
  Files          23       23
  Lines        1231     1278      +47
==========================================
+ Hits         1149     1188      +39
- Misses         82       90       +8

Flag
Coverage Δ

unittests
92.95% <80.70%> (-0.39%)
:arrow_down:

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

Impacted Files
Coverage Δ

src/transaction/utils.c
87.09% <33.33%> (-12.91%)
:arrow_down:

src/sighash.c
93.80% <66.66%> (-1.61%)
:arrow_down:

src/address.c
90.90% <75.00%> (-5.25%)
:arrow_down:

src/transaction/deserialize.c
91.42% <76.92%> (-1.91%)
:arrow_down:

src/transaction/serialize.c
100.00% <100.00%> (ø)

unit-tests/test_address.c
100.00% <100.00%> (ø)

unit-tests/test_tx_parser.c
100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Send to P2SH: https://explorer.kaspa.org/txs/e8a7f889ab0a1d3ce560245c65b2c3a002445034fc71632a7a3cdc54754dcf88
Send to Schnorr: https://explorer.kaspa.org/txs/a3a25c622d287552db6c5b3d81e3838c4a7d7f1e969cda3b02f4e720cd5e7195
Send to ECDSA: https://explorer.kaspa.org/txs/8fe76222138fe5189b8b58cb7e7b8df4f697f7ec44bc80b6bfd37f2931e0122b


fix issue where 'out' dates still selectable when should be disabled

[bugfix] When shouldShowDaysOut is true and earliestSelectableDate is set to a date later than the out days of the previous month the out days are still selectable and colored wrong until the month is changed. To fix I removed the KVO on isHidden since isUserInteractionEnabled is used to calculate isDisabled. I'm not sure if this is the best way to fix it, but in my tests it does fix the issue and doesn't cause any other side effects.
public override var isHidden: Bool {
    didSet {
      isUserInteractionEnabled = isHidden ? false : true
    }
}

[improvement] I found that a boolean logic was reversed in changeDaysOutShowingState so I used labelled arguments to make it more consistent.
[improvement] I found that there is some duplicate code for setDayOutViewsVisible so I refactored this into a new protocol CVCalendarContentPresentationCoordinator that contains the shared logic applied to any calendar which can toggle the presentation of out days.

Looks great, thanks! 👍


extend test documentation

There are clang-tidy specific tests in the test suite, but clang-tidy is not mentioned in the documentation as a dependency to run the tests.

@gyorb clang-tidy already mentioned at here https://github.com/Ericsson/codechecker/blob/master/docs/README.md#linux as a mandatory dependency for a development environment. Isn't it enough?


[BUG] Adding new asset file does not seem to work

Information

Version: v2.6.0
OS: Windows
APK From? Any

Steps to Reproduce

Do the usual to decompile an app
Add a new file, one that did not exist earlier into the assets folder
Rebuild
The resulting APK does not contain the new file

I'm guessing this is not an actual bug and that I simply do not know how to achieve this.

Forgive me, I'm retarded. APKTool is working correctly.


Marketplace not working.

Issue Type: Bug
5/7/2021
I have a fairly high internet connection (60Mbps) but still I can't connect to vscode marketplace from yesterday and even I can't browse vscode website on web. Please fix this problem ASAP.
Thanks & Regard,
Wraient
<EMAIL_ADDRESS>VS Code version: Code 1.57.1 (507ce72a4466fbb27b715c3722558bb15afa9f48, 2021-06-17T13:26:56.255Z)
OS version: Linux x64 5.10.0-kali7-amd64
Restricted Mode: No

Can you share more details? What region you live in? Do you still see this issue?
Can you load marketplace.visualstudio.com in the browser?

no, I can't load marketplace.vscode or any microsoft website by then but now the issue is sorted out. Once I got an pop-up on vs code saying "do you trust this autor of this directory?" I clicked on yes and from then it started working. I live in India and about the browser I guess it was because of release of windows 11 and everyone downloading it and server getting trafficed. so that's it. Thanks for your kind words.


Add ability to disable interceptor

Once the Disable button is clicked, the extension shouldn't mock fake responses and should return the actual response returned by the server for any further requests made

Closing this as #101 merged into master


update zh-trans content/zh/docs/tasks/administer-cluster/reserve-comp…

ref:https://github.com/k8smeetup/website-tasks/issues/3699

/cc @tengqm

/cc @tengqm
done

/lgtm

/approve


Error when using Clob type schema definitions

Bug Report
For English only, other languages will not accept.
Before report a bug, make sure you have:

Searched open and closed GitHub issues.
Read documentation: ShardingSphere Doc.

Please pay attention on issues you submitted, because we maybe need more details.
If no response more than 7 days and we cannot reproduce it on current information, we will close it.
Please answer these questions before submitting your issue. Thanks!
Which version of ShardingSphere did you use?
4.0.0-RC2
Which project did you use? Sharding-JDBC or Sharding-Proxy?
Sharding-JDBC
Expected behavior
Successfully read from tables with columns of clob types ( with @Lob annotation).
Actual behavior
Got the following error:
org.springframework.dao.InvalidDataAccessApiUsageException: java.lang.String cannot be cast to java.sql.Clob; nested exception is java.lang.IllegalArgumentException: java.lang.String cannot be cast to java.sql.Clob
The same code works for original DataSource from springboot.
Reason analyze (If you can)
Steps to reproduce the behavior, such as: SQL to execute, sharding rule configuration, when exception occur etc.
Example codes for reproduce this issue (such as a github link).

Duplicate with #2956.


Change Solr Port

I'm using this docker-compose.yml, but I need to change the port for the Solr instances so they are unique.  I'm trying to use podman in Fedora 32.  I mounted /var/solr to my host and updated the solr.xml in each solr to have a unique Jetty port, I also mounted /etc/default, and created a "solr" file in each solr that states a unique SOLR_PORT.  The "solr" file is a copy of solr.in.sh renamed to just "solr" and has the SOLR_PORT line uncommented and set to the same value as the Jetty port.  I also modified the docker-compose.yml file to use 8981:8981 for solr1, 8982:8982 for solr2, and 8983 for solr3.  So the ports match in solr.xml, "solr", and docker-compose.yml.  However, it always seems to use 8983, since only the solr 3 container runs and the others fail with:
2020-06-16 08:45:44.922 WARN  (main) [   ] o.e.j.x.XmlConfiguration  => java.security.PrivilegedActionException: java.io.IOException: Failed to bind to <IP_ADDRESS>/<IP_ADDRESS>:8983
Caused by: java.net.BindException: Address already in use

Containerization has been a revolution in how systems are deployed because it isolates processes to such a high degree that it doesn't matter what port software runs on within the container.  I don't know about "podman" but it seems you are trying to get each process to use a different port which just seems like a fundamental disconnect with what containerization enables.


Improving maintainability: run lint

Hello.
This pull request is part of https://github.com/eirslett/storybook-builder-vite/issues/188.
Resolves https://github.com/eirslett/storybook-builder-vite/pull/193#issuecomment-1006256729.
Done:

[x] Run lint on all codebase.
[x] Fix code inspection warnings from a JetBrains IDE for the packages/storybook-builder-vite directory.
[x] Create .git-blame-ignore-revs file.

@eirslett, @IanVS
For setup, run once:
git config blame.ignoreRevsFile .git-blame-ignore-revs

Also, you can enable the Hide whitespace changes option in the Files changed tab settings.
Best wishes,
Sergey.

Thanks for this, I will review it after #193 is merged, since it is difficult right now to tell what's a part of this PR and what's a part of the other.

Rebased onto main branch.

We will need to change the SHA in .git-blame-ignore-revs now, since this was squash-merged.

We will need to change the SHA in .git-blame-ignore-revs now, since this was squash-merged.

@IanVS fixed in #198.


When will the SSR CLI be ready for production

The mobile terminal rating requirements are very high.
I'm going to migrate from NextJS to SolidJS.

If SvelteKit taking close to a year and not being ready is an indication it's too early for me to set a date. When we get past this alpha stage we should have a better idea. But right now we are still understanding the solution and scope.

We are working hard towards a beta but ours APIs are stabilizing, you can start playing with the alpha if you are ready for some breaking changes in the future. Closing this for now..


Adding ulimits and log-config in HostConfig

We are sending a pull request for this.

#99 Already sent.

#99 was just merged :+1:


chore(deployment): Change the release branch names to alpha and beta

The tags on npm have main in the currently. By using alpha and beta as the branch names this should remove the "main" from the names

This looks reasonable, but the main reason for using main-beta and main-alpha is that main* are protected branches. Do we have the protection setup for alpha and beta before merging this?

I simplified the rules quite a bit:

blocking the ability to target these branches to merge (should target main-v4, main or main-v2 IMO)
only allow pushing to these branches if you're an admin

The workflow would be. Make a change, put up a PR, get it approved & merged. If you want to deploy to alpha or beta, force push to those branches.
As I'm writing this out though I wonder if it would be better to remove 'alpha' and 'beta' in favour of targeting main-v4, main or main-v2. The package.json config should allow all three 'main' branches to be released to their correct channels.


fix(curriculum): remove all instances of var in RPG game

Summary changes
This PR is responsible for removing all instances of the var keyword in the RPG game.
Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

closes #53164

@jdwilkin4 in this file
I think you changed variable name xp to let
-assert.match(code, /xp/);
+assert.match(code, /let/);

Hey @Kashinathpat !
Thanks for catching that.
I will push a commit to fix that.
Also, a neat trick you can do for PR's is to use the suggestion tool.
On GitHub, you can click on a line in the PR like this

Then click on the suggestion tool

Then you are able to make a change

And that change will show up for the author of the PR and they can commit that change right from GitHub 👍

It has been super helpful for all of the maintainers and makes reviewing PR's much easier 👍

Ugh, of course, the pick.toString() transpiles the code and uses var. Because of that code will need to be matched.
function pick(guess) {
  const numbers = [];
  while (numbers.length < 10) {
    numbers.push(Math.floor(Math.random() * 11));
  }
  text.innerText = "You picked " + guess + ". Here are the random numbers:\n";
  for (let x = 1; x < 5; x++) {

  }
}
console.log(pick.toString());

// console
function pick(guess) {
  var numbers = [];
  var _LPC25 = 1;
  var _LP25 = Date.now();
  while (numbers.length < 10) {
    if (_LPC25++ % 100 === 0 && Date.now() - _LP25 > 100) {
      (function (e) {
        console.log("Potential infinite loop detected on line " + e + ". Tests may fail if this is not changed.");
      })(265, 2);
      break;
    }
    numbers.push(Math.floor(Math.random() * 11));
  }
  text.innerText = "You picked " + guess + ". Here are the random numbers:\n";
  var _LPC26 = 1;
  var _LP26 = Date.now();
  for (var x = 1; x < 5; x++) {
    if (_LPC26++ % 100 === 0 && Date.now() - _LP26 > 100) {
      (function (e) {
        console.log("Potential infinite loop detected on line " + e + ". Tests may fail if this is not changed.");
      })(269, 2);
      break;
    }
  }
}

Ugh, of course, the pick.toString() transpiles the code and uses var

Yeah, that's fine.
The goal of this PR is to remove all instances of var in the lessons that the campers see.
A direct quote from the issue linked to this PR

We should simply stop teaching var as the confusion isn't worth it.

As long as we remove all instances from the teaching lessons and just jump straight into let, then we are fine.

We won't be transpiling const and let for long: https://github.com/freeCodeCamp/freeCodeCamp/pull/53127, but I stalled, because I wasn't sure how best to handle the fact we're also not transpiling () => {} and so cannot just look for function any more.
I'll get to that Soon(ish) if you want to match let.

I'll get to that Soon(ish) if you want to match let.

That would be great, but I also don't think there is a rush since I don't think that should be a blocking factor for this PR.
The purpose of the issue brought up by Quincy and this resulting PR is to remove all instances of teaching var to campers.
So I think it is fine, if there is a separate PR later on to refactor the tests to only match let


Please update the information on how to proceed after the  request deadline is over.

Not mentioned - What to do after 30th June,2020 ?
How Can I raise a request for GoLocal for South Africa tenant now ?
Can you please update the information on how to proceed after the  request deadline is over ?

@Navesai We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.
Please feel free to re-open this issue if you see an improvement is required in the specific area of this documentation. We will certainly update this documentation with that valuable information. Thanks again, for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.

@Navesai We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.
Please feel free to re-open this issue if you see an improvement is required in the specific area of this documentation. We will certainly update this documentation with that valuable information. Thanks again, for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Working synthesis version for ref encoder; results are no good

Synthesis is running without error, however the results are not as expected and the duration predictor is not performing well.


Adding "base" to files.exclude breaks VS Code

VSCode Version: Code 1.17.1 (1e9d36539b0ae51ac09b9d4673ebea4e447e5353, 2017-10-10T14:24:50.851Z)
OS Version: Windows_NT x64 10.0.14393
Extensions:

Extension
Author (truncated)
Version

code-gnu-global
aus
0.2.2

vscode-astyle
chi
0.7.1

python
don
0.7.0

toggle-excluded-files
eam
1.6.0

base-converter
hoo
0.0.1

auto-comment-blocks
kev
1.0.1

vscode-devskim
MS-
0.2.0

cpptools
ms-
0.13.1

sublime-keybindings
ms-
2.9.1

material-icon-theme
PKi
2.2.2

reloaded-cpp
rel
0.1.6

vscode-coverage-gutters
rya
1.1.3

perforce
sle
2.2.1

(1 theme extensions excluded)

Steps to Reproduce:

Open any folder in VS Code
Create a .vscode directory
Create a file "settings.json" within this .vscode directory
Add the following to the file:
{  "files.exclude": {      "base": true  } }
Save and close VS Code
Reopen VS code to the original folder (folder which contains .vscode directory)
VS Code window will be blank except for top menu bar, keyboard shortcuts will give errors.

Reproduces without extensions: Yes

@natedawg1013 great catch 👍


[Bugfix] no naming rule check when create mv

What type of PR is this：

[x] bug
[ ] feature
[ ] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #9762
Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

run starrocks_fe_unittest

[FE PR Coverage Check]
:heart_eyes: pass : 7 / 8 (87.50%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/analyzer/MaterializedViewAnalyzer.java
7
8
87.50%
[85]

run starrocks_admit_test

https://github.com/Mergifyio backport branch-2.4

[FE PR Coverage Check]
:heart_eyes: pass : 7 / 8 (87.50%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/analyzer/MaterializedViewAnalyzer.java
7
8
87.50%
[85]


RHEL8 DHCP Log streaming blank

I was able to get the install done with the help of "npm audit" during some failures.  I can log into the IP:3000 and for the most part seems to be working.  However when I click on Logs, and the DHCP Log shows up, regardless of what I type in the search, or leave it blank, and then click "start" it never shows anything in this window.
I've tried using "npm install console" and "npm install websocket", which downloaded and installed those, but it's still not showing anything.
Selinux has been disabled, even before the install.  I checked permissions, and changed/edited the glass_config.json file to point to the proper log and leases files.
All other screens seem to be showing fairly accurate information.  Leases can be searched on the active leases section.
I've even tried creating a sym link for /var/log/dhcp.log, in case the log file was being called some other way.

I've never had logging working correctly, but are you actually seeing logs into /var/log/dhcp.log?

Yes, and I have verified the settings for the Admin point to the
correct file.
On Fri, Jan 28, 2022 at 4:10 AM madtempest @.***> wrote:

I've never had logging working correctly, but are you actually seeing logs
into /var/log/dhcp.log?
—
Reply to this email directly, view it on GitHub
https://github.com/Akkadius/glass-isc-dhcp/issues/90#issuecomment-1024067247,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AU2YRTCEL27PCUP3ADZIETLUYJTPVANCNFSM5AKDO2WQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

Via CLI/SSH or console, I can "tail -f /var/log/dhcp.log" and watch the log
fly by.
On Fri, Jan 28, 2022 at 1:39 PM Scott Smith @.***> wrote:

Yes, and I have verified the settings for the Admin point to the
correct file.
On Fri, Jan 28, 2022 at 4:10 AM madtempest @.***>
wrote:

I've never had logging working correctly, but are you actually seeing
logs into /var/log/dhcp.log?
—
Reply to this email directly, view it on GitHub
https://github.com/Akkadius/glass-isc-dhcp/issues/90#issuecomment-1024067247,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AU2YRTCEL27PCUP3ADZIETLUYJTPVANCNFSM5AKDO2WQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

@wcarman1 I'm trying to do the same. I'm a novice user running a small network at home. I've opened port 3000 on the firewall as per instructions here: https://github.com/Akkadius/glass-isc-dhcp/issues/62. Did you open port 8080 on the same firewall zone? Or was it something else? Thanks.

@llima27 sorry for the delay, I believe it was on the zone that you use to access the webserver. I believe for me it was public but it may be different for you though, it depends on your setup. I also moved to KEA/ Stork and away from ISC overall so I don't have my setup to look at anymore.

@wcarman1 No worries! I'll give it a try. I'll add both 8080 and 3000 to public to see if I can access the logs from the web.


Move retrieval of user data to op-be-user

We are retrieving user data (password, username, etc.) directly within our auth service. We should move that process to the op-be-auth service.

@Feyzanrs can you please merge tne first/hsm/1/create-first into phase1, then close the issue.

@Feyzanrs can you please merge the first/hsm/1/create-first into phase1, then close the issue.

with great pleasure :)

@Feyzanrs can you please merge the first/hsm/1/create-first into phase1, then close the issue.

with great pleasure :)

Thanks <3 :)


upgrade 2.0.2 to 2.4 failed

No "save settings" button in address/admin/install.php
Did not get option for lite install on set up of portal.
reset head back to 2.0.2 (1e311572f52f31bf9a8fe288dbaa1c3dcc2a8e57) and ran
./install.sh again.
Went to set up portal page, save button was there and all went ok but now portal unreachable, map does not work, station still sending data to flightaware though.
I have had this problem before
https://github.com/jprochazka/adsb-receiver/issues/174#issuecomment-215992038
can reach
localhost/install/.
and save settings button is there but when i try to change timezone (europe) and save the settings, it reverts back to africa.
Portal pages, map, performance etc do not function
Think I have found the answer as i had an existing image which i was trying to upgrade.
https://github.com/jprochazka/adsb-receiver/blob/master/README.md
Until I get time to do a fresh install, any ideas of a fix/rollback on my existing 2.0.2 please ?

Actually check the log file /var/log/lighttpd/error.log and post any messages you have there which occur when you try and visit the portal.

2016-10-11 13:39:52: (mod_fastcgi.c.2702) FastCGI-stderr: PHP Fatal error:  Uncaught exception 'Exception' with message 'DateTimeZone::__construct(): Unknow$
2016-10-11 13:39:52: (mod_fastcgi.c.2702) FastCGI-stderr: Stack trace:
2016-10-11 13:39:52: (mod_fastcgi.c.2702) FastCGI-stderr: #0 /var/www/html/blog.php(60): DateTimeZone->__construct('')
2016-10-11 13:39:52: (mod_fastcgi.c.2702) FastCGI-stderr: #1 {main}
2016-10-11 13:39:52: (mod_fastcgi.c.2702) FastCGI-stderr:   thrown in /var/www/html/blog.php on line 60

Can you get me the full line starting with:

2016-10-11 13:39:52: (mod_fastcgi.c.2702) FastCGI-stderr: PHP Fatal error: Uncaught exception 'Exception' with message 'DateTimeZone::__construct(): Unknow$

There is more information I need after the $.
Thinking you might just be missing a setting is all.

2016-10-11 13:39:52: (mod_fastcgi.c.2702) FastCGI-stderr: PHP Fatal error: Uncaught exception 'Exception' with message 'DateTimeZone::__construct(): Unknow$$Unknown or ba$Unknown or bad timezone ()' in /var/www/html/blog.php:60

Take it you are running the lite version of the portal. If so add the following to /var/www/html/settings.xml.
<?xml version="1.0" encoding="utf-8"?>
<settings>
...
    <setting>
        <name>timeZone</name>
        <value>America/New_York</value>
    </setting>
...
</settings>

Make sure to set the proper time zone afterwards in the admin area.
Let me know if you encounter any other issues.

Completed the change to  /var/www/html/settings.xml
enter admin area, timezone selections are there in list, save button is there, but any change will not save.
Reverts back to 1st item on list Africa.
Tried localhost/admin/install.php
The data directory is writable.
The settings.class.php file is writable.
No Save button or next button at foot of page.

Completed the change to /var/www/html/settings.xml using nano (no file existed btw)

If this has the advanced portal features enabled, which I am guessing is the case if the .xml files are not present in the folder /var/www/html/data/, use the MySQL queries added to the post above.
If you are using SQLite the same queries should work.
sqlite /var/www/html/data/portal.sqlite
DELETE FROM adsb_settings WHERE name = 'timeZone';
INSERT INTO adsb_settings (name, value) VALUES ('timeZone', 'America/New_York');

Hi Joe, I had lite install and upgraded using the readme.
when i used install.sh, I did not get option for lite/data junkie
Now i have a receiver feeding to fr24,fa etc but portal will not work.
I wanted lite as before.
Do I make the change to /var/www/html/data/ as above ?

Hi Joe, files are in /var/www/html/data/
administrators.xml
blogposts.xml
flightnotifications.xml
placeholder.txt
settings.xml

Take a look at the error log once more lets see if we fix one issue and ran into another.
cat /var/log/lighttpd/error.log

Hi Joe,
Here we go.
2016-10-12 17:24:24: (mod_fastcgi.c.2702) FastCGI-stderr: PHP Fatal error:  Unca                                                                             ught exception 'Exception' with message 'DateTimeZone::__construct(): Unknown or                                                                              bad timezone ()' in /var/www/html/blog.php:60
2016-10-12 17:24:24: (mod_fastcgi.c.2702) FastCGI-stderr: Stack trace:
2016-10-12 17:24:24: (mod_fastcgi.c.2702) FastCGI-stderr: #0 /var/www/html/blog.                                                                             php(60): DateTimeZone->__construct('')
2016-10-12 17:24:24: (mod_fastcgi.c.2702) FastCGI-stderr: #1 {main}
2016-10-12 17:24:24: (mod_fastcgi.c.2702) FastCGI-stderr:   thrown in /var/www/h                                                                             tml/blog.php on line 60

That is still the same one...
OK so I got this right all the above XML files you listed exist.

Completed the change to /var/www/html/settings.xml using nano (no file existed btw)

This file was empty?
If so lets do this...
Backup all .xml files located in the directory /var/www/html/data EXCEPT settings.xml and administrators.xml.
mkdir ~/backups/
sudo cp /var/www/html/data/*.xml ~/backups/
sudo rm -f ~/backups/settings.xml
sudo rm -f ~/backups/administrators.xml

Delete all .xml files in the directory /var/www/html/data/.
sudo rm -f /var/www/html/data/*.xml

Delete settings.class.php.
sudo rm -f /var/www/html/classes/settings.class.php

If you downloaded the latest master branch containing v2.4.0...
cd ~/adsb-receiver
./install.sh

Skip installing install/reinstall everything if you wish EXCEPT the portal. Be sure to Install the portal when asked to. and follow all the instructions including going to the web based installer when told to.
Restore you old XML files.
(only necessary to restore any blog posts or the flight notification list.)
sudo cp ~/backups/*.xml /var/www/html/data/

Hi Joe that worked and I now have portal but d1090 map view does not work.I just get spinning java wheel.
ctrl,shift j result is "Uncaught ReferenceError: google is not defined"
Nearly there !

That error is due to a missing Google Maps API key.
You may want to go ahead and update/rebuild dump1090-mutability using install.sh.
dump1090-mutability did not use a key for the Google Maps API service in the past which was fine up until recently when they changed their policy and began requiring it. They switched it to use Open Street Maps by default to get around this problem. Rebuilding and reinstalling dump1090-mutability should fix your issue.

Hi Joe,
No luck with re install of mutability from ./install.sh, maps do not show work.
Perhaps d1090 mutability has changed recently as I note that it no longer supports internal  web server.
[http://discussions.flightaware.com/ads-b-flight-tracking-f21/dump1090-mutability-not-showing-map-t38041.html]
I also noted the change in directory / location for gmap.html
in build/dump1090/public_html/
But the gmap.html file looks very different to previous ones I have seen, In particular line 9 where the API key insertion location is .
Is there any way to cure the map issue with google maps or how to use other map suppliers ?
Help anyone ?
Thanks

Forget about Google Maps for now it is no longer supported in the dump1090-mutability code. Yes there are many changes in the newer version of dump1090-mutability as they migrated away from Google Maps to OpenStreetMaps.

After you upgraded make sure your browser cache is cleared out.
Go directly to the dump1090-mutability map to see if it works. (http://YOUR_DEVICE_IP_ADDRESS/dump1090/gmap.html)
If that works try to visit the portal map. _(http://YOUR_DEVICE_IP{ADDRESS/dump1090.php)