fix: ignore webpack warnings by source-map-loader

Some third party packages may ship miss-configured sourcemaps, that interrupts the build
See: https://github.com/facebook/create-react-app/discussions/11278#discussioncomment-1780169
To trigger the error, simply install<EMAIL_ADDRESS>add import {} from "stylis-plugin-rtl" somewhere in the code
before:

After:

Not sure if everything should be ignored or only warnings regarding node modules?

Not sure if everything should be ignored or only warnings regarding node modules?

source-maps-loader is used 99.9% of cases for node_modules only
I've changed the impl to use function, maybe this will work

You could use the suggested example of
ignoreWarnings: [/Failed to parse source map/]

from the source map loader docs https://github.com/webpack-contrib/source-map-loader#ignoring-warnings

@Bnaya I've tried rerunning the tests, but seem to be an issue either with caching or lock file - When doing npm install in the create-react-app project does it generate a new package-lock.json?

@Bnaya I've tried rerunning the tests, but seem to be an issue either with caching or lock file - When doing npm install in the create-react-app project does it generate a new package-lock.json?

I've ran npm install (With npm 8.3) and committed the updated lock file.
Lets see if the CI pass

CI passes

I don't know how relevant this is, but I had ejected after upgrading to CRA 5, and I wanted to implement this source-map-loader change, and for the most part it worked. Except every now and then, when the dev server reloads to do file changes, the dev server crashes with "Cannot read 'resource' of undefined."
So the fix for me was to change the relevant line to:
warning.module?.resource.includes('node_modules') &&

(Note the optional chaining)
I don't know under what circumstances warning.module would be non-existent, so hopefully someone more knowledgable than me can chime in.

I will add a null check there, even tho by webpack code/types module shouldn't be undefined.
https://github.com/webpack/webpack/blob/c181294865dca01b28e6e316636fef5f2aad4eb6/lib/WebpackError.js
Regarding using optional chaining, eslint won't let me use it in the project, so i will go with old-style check

I will add a null check there, even tho by webpack code/types module shouldn't be undefined. https://github.com/webpack/webpack/blob/c181294865dca01b28e6e316636fef5f2aad4eb6/lib/WebpackError.js

I was able to replicate the issue. What I did was introduce an ESLint error into one of my TSX files by simply adding:
const unusedVar = 1;

and never actually using unusedVar. This of course violates @typescript-eslint/no-unused-vars. After saving the file, the dev server crashes on re-compile:
TypeError: Cannot read property 'resource' of undefined
    at ignoreSourcemapsloaderWarnings (/Users/denchen/git/ui/config/webpack.config.js:792:26)
    at /Users/denchen/git/ui/node_modules/webpack/lib/IgnoreWarningsPlugin.js:30:8
    at Array.some (<anonymous>)
    at /Users/denchen/git/ui/node_modules/webpack/lib/IgnoreWarningsPlugin.js:29:36
    at Array.filter (<anonymous>)
    at /Users/denchen/git/ui/node_modules/webpack/lib/IgnoreWarningsPlugin.js:28:22
    at Hook.eval [as call] (eval at create (/Users/denchen/git/ui/node_modules/tapable/lib/HookCodeFactory.js:19:10), <anonymous>:7:16)
    at Hook.CALL_DELEGATE [as _call] (/Users/denchen/git/ui/node_modules/tapable/lib/Hook.js:14:14)
    at Compilation.getWarnings (/Users/denchen/git/ui/node_modules/webpack/lib/Compilation.js:4659:37)
    at context.cachedGetWarnings (/Users/denchen/git/ui/node_modules/webpack/lib/stats/DefaultStatsFactoryPlugin.js:487:20)

(node:89082) UnhandledPromiseRejectionWarning: RpcIpcMessagePortClosedError: Cannot send the message - the message port has been closed for the process 89082.
    at /Users/denchen/git/ui/node_modules/fork-ts-checker-webpack-plugin/lib/rpc/rpc-ipc/RpcIpcMessagePort.js:47:47
    at processTicksAndRejections (internal/process/task_queues.js:81:21)
(Use `node --trace-warnings ...` to show where the warning was created)
(node:89082) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:89082) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

As stated in my original comment, I have ejected CRA with very minimal config changes, so I don't know how relevant my error is for non-ejected CRA.

@Bnaya , to be on-par with CRA4 when it comes to webpack dev server logging, could you consider adding
stats: 'errors-warnings'

to webpack.config.json?
This will get rid of all the assets bundling verbosity introduced in CRA5, while keeping warnings and errors displayed.

@Bnaya , to be on-par with CRA4 when it comes to webpack dev server logging, could you consider adding
stats: 'errors-warnings'

to webpack.config.json?
This will get rid of all the assets bundling verbosity introduced in CRA5, while keeping warnings and errors displayed.

I think It's out of the scope of this PR

Just a fly on the wall here, and following this topic.

Still affected by this.

Bump. Any updates on this?

Thanks @Bnaya, that fixed my problem. If you're impatient like me, you can add this fix by editing the file node_modules/react-scripts/config/webpack.config.js, and pasting this after the line performance: false,.
    ignoreWarnings: [
      // Ignore warnings raised by source-map-loader.
      // some third party packages may ship miss-configured sourcemaps, that interrupts the build
      // See: https://github.com/facebook/create-react-app/discussions/11278#discussioncomment-1780169
      /**
       *
       * @param {import('webpack').WebpackError} warning
       * @returns {boolean}
       */
      function ignoreSourcemapsloaderWarnings(warning) {
        return (
          warning.module &&
          warning.module.resource.includes('node_modules') &&
          warning.details &&
          warning.details.includes('source-map-loader')
        );
      },
    ],

Temporary solution for projects using customize-cra & react-app-rewired:
config-overrides.js:
const { override } = require('customize-cra');

const addIgnoreSourcemapsloaderWarnings = config => {
  config.ignoreWarnings = [
    // Ignore warnings raised by source-map-loader.
    // some third party packages may ship miss-configured sourcemaps, that interrupts the build
    // See: https://github.com/facebook/create-react-app/discussions/11278#discussioncomment-1780169
    /**
     *
     * @param {import('webpack').WebpackError} warning
     * @returns {boolean}
     */
    function ignoreSourcemapsloaderWarnings(warning) {
      return (
        warning.module &&
        warning.module.resource.includes('node_modules') &&
        warning.details &&
        warning.details.includes('source-map-loader')
      );
    },
  ];
  return config;
};

module.exports = override(
  addIgnoreSourcemapsloaderWarnings
);

Thanks @Bnaya, that fixed my problem. If you're impatient like me, you can add this fix by editing the file node_modules/react-scripts/config/webpack.config.js, and pasting this after the line performance: false,.
    ignoreWarnings: [
      // Ignore warnings raised by source-map-loader.
      // some third party packages may ship miss-configured sourcemaps, that interrupts the build
      // See: https://github.com/facebook/create-react-app/discussions/11278#discussioncomment-1780169
      /**
       *
       * @param {import('webpack').WebpackError} warning
       * @returns {boolean}
       */
      function ignoreSourcemapsloaderWarnings(warning) {
        return (
          warning.module &&
          warning.module.resource.includes('node_modules') &&
          warning.details &&
          warning.details.includes('source-map-loader')
        );
      },
    ],

This silences repeated sourcemap warnings for @aws-sdk, thank you!

If you okay to lose the sourceMaps in local, Add GENERATE_SOURCEMAP=false in .env.developement file. So that until this fix is released, you can ignore warnings in your local and still have sourceMaps in other env

Is this any closer to being fixed or is there something I can do to suppress the warning on my end? Besides the excellent suggestion to edit webpack.config.js. I would do that but I delete and reinstall node modules frequently enough that it wouldn't make much sense

@TD-DO if you want an extra band-aid, for the band-aid fix, you can also use https://www.npmjs.com/package/patch-package to automatically re-apply the fix anytime your modules are reinstalled.

Bump. Still affected. Can this be reviewed and merged?

Obligatory check in, as the month is now August... Can this please be merged?!

Bump.
These are not useful warnings for me, and are noise from seeing meaningful warnings. I've made an issue for the maintainers, but can only expect so much of other libraries.
WARNING in<EMAIL_ADDRESS>Module Warning (from ./node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from<EMAIL_ADDRESS>file: Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>

The app will work even when those warnings are showing

Bump. Still affected. Can this be reviewed and merged?

Any updates?

We are indeed waiting for this PR to be updated and merged :)

This is just a workaround, but FYI I was able to avoid these warnings by using craco. craco allows you to overlay configuration on create-react-app without ejecting. One of its features is customizing the webpack config. Based on the config change in this PR, I put this in my craco.config.js:
module.exports = {
  webpack: {
    configure: {
      ignoreWarnings: [
        function ignoreSourcemapsloaderWarnings(warning) {
          return (
            warning.module &&
            warning.module.resource.includes('node_modules') &&
            warning.details &&
            warning.details.includes('source-map-loader')
          )
        },
      ],
    },
  },
}

Also seeing this. Would love to see this bug fix get rolled out.

What's happening with this PR.

I have 100's of lines of text being spewed after updating, I would love for this PR to be merged and resolved :)
I understand my apps still work, but I can no longer view any history of content printed to the console with how this currently works.

Any updates on this?

Some update please?

I've been wanting to update to CRA 5 for 9 months now, but waiting on this to do so. Plz?

Is anyone currently working on resolving the conflicts for this PR? I think there are many of us waiting for this fix :)

Waiting for this too. Any updates?

I am getting this error and i am stuck

Another case where dependencies produce warnings.
my project depends on swagger-ui-react -> remarkable -> autolinker
https://github.com/gregjacobs/Autolinker.js/issues/396

any update on this PR? Can someone solve the conflicts and merge it?

Any update pls.


GUI Variant Fixes: off by 1 error and end positions

This fixes https://github.com/bigdatagenomics/mango/issues/290 and https://github.com/bigdatagenomics/mango/issues/295 and
https://github.com/bigdatagenomics/mango/issues/290 shows correct position of variant:

https://github.com/bigdatagenomics/mango/issues/295  no longer cuts off variants:

Coverage remained the same at 71.648% when pulling 2bffb78d88cb9b5ab63ea8789e6a8fb06310c46a on akmorrow13:variantFixes into 12242d9c9ff41dee7111c2e22f8ecd260fc8d278 on bigdatagenomics:master.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/mango-prb/379/
Test PASSed.


RuntimeWarning for not supported items

Path ends with custom size are not supported
Record type BGNEXTN (30) is not supported
Record type ENDTEXTN (31) is not supported

As the warning says, those fields are not currently supported.
Unfortunately I don't have the time right now to implement support form them, but any contributions are welcome!

Fixed in 81a3998


CSS Formatter

Please create a Built-In CSS Formatter. I don't want to Download Extension Because Extension have many files, storage & takes tooo much time to load the extension.
Thank You

dup of https://github.com/microsoft/vscode/issues/19166. See https://github.com/microsoft/vscode/issues/19166#issuecomment-409928738 for why we can't cover all functionality for all languages.


What is representation for 'iou' in the dense head prediction value?

In the prediction head, the box encode in center(x,y), enter_z(z), dim(l,w,h), rot(sina,cosa). What iou representation?
@zhanggang001

What iou representation

The IoU indicates the iou between the predicted boxes and ground truth boxes. It is a common strategy used in object detection.


Add #![no_std] to the reference impl

This ensures that no accidental uses of libstd can sneak in.
I've verified that this now compiles for ARMv6-M targets via cargo build --target thumbv6m-none-eabi.
(this is great work btw!)

I've added the #![no_std] directive temporarily to make sure it works, but I decided against checking it in. The reference impl isn't published on crates.io, so there won't be any real no_std callers depending on it. (The production blake3 crate does support no_std if you use --no-default-features.) And it's not expected to receive many changes going forward.
The main purpose of the reference impl (besides testing) is to help folks understand the algorithm, together with the spec. I expect that a lot of the people reading it won't be experienced Rust coders, so I've tried to minimize the use of advanced Rust features. I think the benefit of including #![no_std] might be smaller than the cost of making people wonder what it does.


Loss calculation should indicate which loss(es) returned NaN

I've gotten NaN losses, even at high iteration numbers and low learning rates, so it would be nice to know which one(s) caused it to try continuing with it disabled.

Hi. Unfortunately as the current training loop is it's no possible to say exactly which one NaN'd, since all losses are summed before step(). Also, even if it detected NaN in a specific loss, it doesn't necessarily means disabling it will make others stable.
Generally speaking, if you're having NaN there's something wrong on your configuration file or the way you're training. Remember to use a pretrained model and the default template configs at first, to avoid common mistakes.

Well, for the record, I was somewhere between 380k and 385k iterations, and wavelet guiding seems to be the likely culprit since I temporarily turned it off and was able to make it further (390k), then turned it back on and was able to train to 395k.

I see, thanks for reporting. And yes, wavelet guided can be a bit hard to use. Some discriminators don't deal well with it and ends up hurting the training process more than it benefits.


restored inventory items are missing the 'entity' member variable

I'm sure I'm just missing something here. After I got things working through part 10, I noticed that inventory items don't work after being restored from save, all related to 'entity' being null:
"Invalid call. Nonexistent function 'get_entity_name' in base 'Nil'". at this line: https://github.com/SelinaDev/Godot-Roguelike-Tutorial/blob/main/part_10/src/Entities/Actors/Components/healing_consumable_component.gd#L16
"Cannot call method 'queue_free' on a null value." at this line:  https://github.com/SelinaDev/Godot-Roguelike-Tutorial/blob/main/part_10/src/Entities/Actors/Components/consumable_component.gd#L16
I don't have the same problem running the project from this repo, but can't find what I'm doing different. Do you have any ideas?
Thank you!

I figured out this one:  This line here doesn't appear in the tutorial text, but is important:
https://github.com/SelinaDev/Godot-Roguelike-Tutorial/blob/main/part_10/src/Entities/entity.gd#L113

Thank you very much for pointing me to this! I will be sure to uptate the tutorial to include this as soon as I get the chance.

Here's one more thing I found-- there were some edits to main_game_input_handler.gd
The symptoms I noticed were:

If the player hits "I" while my inventory is empty, the game seemingly froze (though it seems like it was just that the main input handler didn't get re-activated)
If the player hits escape while viewing inventory, they are bounced back to the main menu.

>
68,71d68
< 	if inventory.items.is_empty():
< 		await get_tree().physics_frame
< 		MessageLog.send_message("No items in inventory.", GameColors.IMPOSSIBLE)
< 		return null
77,79c74
< 	var has_item: bool = selected_item != null
< 	var needs_targeting: bool = has_item and selected_item.consumable_component and selected_item.consumable_component.get_targeting_radius() != -1
< 	if not evaluate_for_next_step or not has_item or not needs_targeting:
---
> 	if not evaluate_for_next_step or (selected_item and selected_item.consumable_component and selected_item.consumable_component.get_targeting_radius() == -1):
91,92d85
<
<

Thank you again, sorry it took so long. I just updated the site to include these changes.


Update Binding before executing the command of "TextBoxHelper.ButtonCommand"

Describe the changes you have made to improve this project
Now an update of the binding is done before executing the command of "TextBoxHelper.ButtonCommand".
Unit test
guess not
Additional context
See #3924
Closed Issues
#3924

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

@Rerago your changes have been merged, thanks for your contribution 👍


Check for previous/existing GitHub issues/module proposals

[X] I have checked for previous/existing GitHub issues/module proposals.

Check this module doesn't already exist in the module indexes

[X] I have checked for that this module doesn't already exist in the module indexes; or I'm proposing the module to be migrated from CARML/TFVM.

Bicep or Terraform?
Bicep
Module Classification?
Resource Module
Module Name
avm/res/alerts-management/action-rule
Module Details
Module for creation of alert processing rule.
Do you want to be the owner of this module?
Yes
Module Owner's GitHub Username (handle)
judyer28
(Optional) Secondary Module Owner's GitHub Username (handle)
No response

Azure Resource Reference: https://learn.microsoft.com/en-us/azure/templates/microsoft.alertsmanagement/actionrules?pivots=deployment-language-bicep

Hi @judyer28,
Thanks for requesting/proposing to be an AVM module owner!
We just want to confirm you agree to the below pages that define what module ownership means:

Team Definitions & RACI
Shared Specification (Bicep & Terraform)
Module Support

Any questions or clarifications needed, let us know!
If you agree, please just reply to this issue with the exact sentence below (as this helps with our automation 👍):
"I CONFIRM I WISH TO OWN THIS AVM MODULE AND UNDERSTAND THE REQUIREMENTS AND DEFINITION OF A MODULE OWNER"
Thanks,
The AVM Core Team
#RR

I CONFIRM I WISH TO OWN THIS AVM MODULE AND UNDERSTAND THE REQUIREMENTS AND DEFINITION OF A MODULE OWNER

Hi @judyer28,
Thanks for confirming that you wish to own this AVM module and understand the related requirements and responsibilities!
Before starting development, please ensure ALL the following requirements are met.
Please use the following values explicitly as provided in the module index page:

For your module:

ModuleName - for naming your module
TelemetryIdPrefix - for your module's telemetry

For your module's repository:

Repo name and folder path are defined in RepoURL
Create GitHub teams for module owners and contributors and grant them permissions as outlined here.
Grant permissions for the AVM core team and PG teams on your GitHub repo as described here.

Check if this module exists in the other IaC language. If so, collaborate with the other owner for consistency. 👍
You can now start the development of this module! ✅ Happy coding! 🎉
Please respond to this comment and request a review from the AVM core team once your module is ready to be published! Please include a link pointing to your PR, once available. 🙏
Any further questions or clarifications needed, let us know!
Thanks,
The AVM Core Team

Hello @Azure/avm-core-team, please review the following PR:  https://github.com/Azure/bicep-registry-modules/pull/2184.  Thank you.

Hello @Azure/avm-core-team, PR reviewed and committed.

[x] Module published.
[x] GitHub teams are available and assigned to their parents.
[x] CODEOWNERS file updated.

Thank you for your work @judyer28! I'm closing this issue now.


[HTM-845] Fix reporting test failures

See https://github.com/EnricoMi/publish-unit-test-result-action/issues/12


Take develop to main

RI develop to main.

I need to fix the merge conflicts.

Online advertising marketing social media all brands ads


Testing: Unit and E2E frameworks

Let's try out Cypress and see what we think

PR #44 resolves this


Add possibility to save downloaded FIT data to file

For backup purpose and to upload body measurements to platforms other than Garmin and TrainerRoad I would like to store the FIT data in a file. Because the original --no-upload mixes log messages and binary FIT data on stdout it is not possible to just redirect the output to a file. This patch adds --output to the list of arguments to specify a file where the FIT data is stored.
Attention: This patch changes the previous behavior of --no-upload which now doesn't print the FIT data on stdout.

Looks good to me, I've already tested it during merging it on my repo.


Key.Parse - network -> expectedNetwork

I was confused if this is an option or a constraint. It is cleaner in this way.
Network.Parse uses the same naming which is expectedNetwork.

After Named Parameters were introduced in C# this kind of improvement are potential breaking changes, this means that it could break those projects that use this method as follow:
var sk = Key.Parse(wif: aliceKey, network: Network.Main);

I know that is extremely unlikely but it is something we have to have in mind.
Anyway, if you change this method then you should also change the other Parse method because after this PR they will be different.


Aprsdroid TNC - half of Bluetooth frames are missing on radio

I am using Aprsdroid to send the frames over Bluetooth using Lora32 v2.1 TNC. It works intermittently, as some of the frames are not appearing on the radio.
I've leveraged the built-in debug capabilities to look into the Bluetooth frame and how it is processed by software. Here is the output:
Not OK - frame not sent to LoRa:
[DEBUG][bluetooth] Received buffer size 2. Nmea=0. �
[DEBUG][bluetooth] [1/2] c0 -> �
[DEBUG][bluetooth] [2/2] 00 ->
[DEBUG][Bluetooth] Status: 30
[DEBUG][bluetooth] Received buffer size 38. Nmea=0. ����
[DEBUG][bluetooth] [1/38] 82 -> �
[DEBUG][bluetooth] [2/38] a0 -> �

OK - frame sent to LoRa:
[DEBUG][bluetooth] Received buffer size 40. Nmea=0. �
[DEBUG][bluetooth] [1/40] c0 -> �
[DEBUG][bluetooth] [2/40] 00 ->
[DEBUG][bluetooth] [3/40] 82 -> �
[DEBUG][bluetooth] [4/40] a0 -> �
[DEBUG][bluetooth] [5/40] 88 -> �
[DEBUG][bluetooth] [6/40] a4 -> �
[DEBUG][bluetooth] [7/40] 62 -> b
[DEBUG][bluetooth] [8/40] 6c -> l

So it seems that some of the frames are split into two separate chunks, therefore not being recognized as a KISS frame and not sent to radio.
I have this issue with two different boards - TTGO LoRa32 v2.1 and DIY ESP32-WROOM-32D.
Currently using latest main code. Checked out one of the 2023.09 commits, same issue.

please test with TNC2 as configuration on the APRSDroid app

Yes, TNC2 works and I am currently using it as a workaround.

TNC2 is the way to use it ;)

That should work, thank you. Perhaps worth adding this to wiki.

https://github.com/richonguzman/LoRa_APRS_Tracker/wiki/06.-Bluetooth-Connection-%23-Conexión-Bluetooth
its there for a while now ;)

Right, but there's no info about TNC2 preferred over KISS.


Archetype 2

To verify Augraphy's utility, we need to reproduce the document in this notebook.
There are some notes about the effects present in the document, as well as some suggested augmentations you might need when reproducing it. These lists are not exhaustive; you may need to use more, or less, to achieve a faithful reproduction.
We can discuss the reproduction process for this document in this Issue. Discussion about this project in general should go here.

I added an example in the notebook, but it's still need some further improvements. I'm still thinking on how to reproduce those stripped noise at the top area, so right now they are not look exactly the same yet.
Sample:

Augmented:

Closed with PR #154


bug(fe): panic when aggregation on distinct on'result

reproduce
create table t(a int, b int);
explain select count(*) from (
    select distinct on(a,b) * from t
)

output
dev=> explain select count(*) from ( select distinct on(a) * from t);
                                QUERY PLAN
---------------------------------------------------------------------------
 BatchSimpleAgg { aggs: [sum0(count)] }
 └─BatchExchange { order: [], dist: Single }
   └─BatchSimpleAgg { aggs: [count] }
     └─BatchGroupTopN { order: [], limit: 1, offset: 0, group_key: [t.a] }
       └─BatchExchange { order: [], dist: HashShard(t.a) }
         └─BatchScan { table: t, columns: [a] }
(6 rows)

dev=> explain select count(*) from ( select distinct on(b) * from t);
ERROR:  Panicked when handling the request: index out of bounds: the len is 1 but the index is 1
This is a bug. We would appreciate a bug report at:
  https://github.com/risingwavelabs/risingwave/issues/new?labels=type%2Fbug&template=bug_report.yml
server closed the connection unexpectedly
        This probably means the server terminated abnormally
        before or while processing the request.
The connection to the server was lost. Attempting reset: Succeeded.
psql (16.3, server 13.14.0)

duplicate with https://github.com/risingwavelabs/risingwave/issues/19175


Folders are occasionally seen as Files

Microsoft Windows [Version 10.0.16257.1] (running alwsl/alwsl)
I first noticed this issue attempting to build a Jekyll site. I kept receiving various File exists @ dir_s_mkdir.

This seems to be because it cannot find the folder in the cache it expects, so it tries to create it only to discover that a "file" already exists with the same name. The problem with this is that there are no files, and viewing the cache directory with Explorer shows only folders.

Additionally, the first ls of the directory claims to only see 4 folders and 8 files, yet subsequent ls attempts see all 12 folders (note that highlighted ls results are folders).
I cannot figure out why this only happens sometimes (though perhaps it is caused by the attempted Jekyll build process?), but on another instance of this bug I was able to use ls -HALR.

Another time with two ls -al commands back to back:

Additionally, attempting to cat one of these "files" (while WSL still thinks they are files) causes a BSOD with the error kernel_data_inpage_error.

duplicate of #2392


[Communication] - Phone number -Update capabilities and test.yml

As per conversations update phone number to point to js specific phone number.
Updated the capabilities tests as well.

should we update the browser test too?

should we update the browser test recordings too?

@beltr0n I ran it in record mode but it looks like the browsers are beings skipped. I am also a little concerned cause the update test passes locally but seems to timeout on CI all the time

@JoshuaLai Any updates here?


Link list, links should be shorter on smaller breakpoints

Detailed description
Describe in detail the issue you're having.
When browser is set to med or small breakpoints the link list is retaining it's default height. It should not.
Current:

Is this a feature request (new component, new icon), a bug, or a general issue?
Design defect
Is this issue related to a specific component?
Card
What did you expect to happen? What happened instead? What would you like to see changed?
According to the design specs, the height of each link should shrink.
Desired:

What browser are you working in?
Chrome
What offering/product do you work on? Any pressing ship or release dates we should be aware of?
Web simplification
Steps to reproduce the issue

Open the Link List in Storybook and view at different breakpoints.
Compare to design specs in Abstract

Additional notes
When there are only 2 links this isn't a major problem. However, when there are 3 or more the list becomes very long on mobile/tablet. User has to scroll a ton to get to the next content piece.

@larahanlon2 here is the conversation you me and Lila had about this topic...

Oh, I completely missed this. My bad. I guess we can close this issue then.

I think we have #1534 opened to address these changes.


Supporting controllerAs and bindToController usage

I noticed the zxcvbn object is getting directly attached to $scope.  I've been following the pattern of the controllerAs and bindToController using the name 'vm' so everything in scope is available as vm.whatever.  I can't do that with zxcvbn and was hoping you might be able to tweak the code in a way that would allow it?  I'm not exactly sure what all would be involved but wanted to pose the request in case it was desirable for others and not too impactful to the existing code.
Thanks for the consideration,
Billy

@jamesclark92 any idea on this?

I'll look into this

Thanks for checking on it.

+1
I have the same case, I use controllerAs and cannot update to 3.x because it requires value on $scope.

Update: I am in the process of re-writing this to allow for controller as syntax.
Cheeky: I believe injecting $scope should make it work

@m0t0r @m0t0r I have re-written the attribute to be suitable for controller as syntax. I believe the element version of the directive was already compatible.

@jamesclark92 thank you so much, I'll check it.

You may need to just download the source from the dist folder as I'm not sure if ghostbar has updated bower yet
Sent from my iPhone

On 27 Jan 2016, at 21:58, Vitaly<EMAIL_ADDRESS>wrote:
@jamesclark92 thank you so much, I'll check it.
—
Reply to this email directly or view it on GitHub.

Okay  its already been done by me

Thanks a bunch guys!  I'll pull this in and let you know if it caused any issues.

The attribute is working great but I'm not able to make the element work.  Take a look at this plunkr and notice the vm.results is not getting populated:  http://plnkr.co/edit/U6g7na?p=preview
Am I doing something wrong?

@wcjr 3.1.1 contains the fix for this

would mind updating your bower.json version to 3.1.1? Thanks.

@wcjr whoops - done

no worries, at least it wasn't the dreaded "forgot to push tags" thing that always gets me! :-)


How to change the current platform?

I'm using VS Code with the atom keybindings on OSX, but I'm used to have it under linux. Is it possible to change the platform that VS Code uses (from the OSX to linux) and use the respective keybinding?

@xdmx Not sure I understand. On your Linux machine you can install this atom key map too and your MacOS machine you can install it too?

Hi Kenneth, I mean if I can have the same atom key bindings on both OS (based on the Linux one), so having for example ctrl+c to copy on both systems instead of having command+c on MacOS and ctrl+x on Linux, without having to manually remap all of them

Ah @xdmx at this point in time we don't support this. The intent of the keymaps is to mimimic the original keyboard shortcuts for Atom, so you would have to copy paste them manually into your own keybindings.json file.  You can simply copy them from https://github.com/Microsoft/vscode-atom-keybindings/blob/master/package.json#L24, and remove the mac, win, linux keys, but keeping key around.


RulesEngine lifetime and thread-safety

Hello
I can't find it in docs nor in wiki, what's the advised lifetime of the RulesEngine object? Should I create it every time I need it and it will cache the compiled evaluator itself or should I create just one instance and cache it? If so is it thread-safe and I can reuse the same instance at the same time?

@Makciek , it is recommended to use it as single instance to benefit from caching of compilation.
Also, it is thread safe, so you can use same instance at same time

Hello, I'm curious about the thread safety of the localParam. If we aim to record the result of rule evaluation and assign it to a local parameter, could it be overwritten by other rule evaluations with different inputs?
For instance, let's say we construct a rule containing blacklist words. If the text contains any of these blacklist words, we wish to record the offensive word in the localParam and utilize it to generate the FailureMessage.

@vanwx localParam is like a local variable in code, they do not share reference with any other rule or execution. So it should be completely thread safe

@vanwx localParam is like a local variable in code, they do not share reference with any other rule or execution. So it should be completely thread safe

Yeah thanks @abbasc52 . I did write some tests and it's been safe. Also I found some examples on this repo use the localParam to store result state so it's good.


Step three to fixing home page: have working search and map/list of campaigns

after #416 and #417 we can really solve #193 by putting search back and having it work along with a list/map of campaigns

@tonysurma @mheggeseth I would like to work on this, but would like some guidance as to what type of functionality is desired.  Do we just want to be able to search the list?  If so, what types of searches.  @mheggeseth made reference in #193 to displaying a map.  How does that factor in?  Please advise, thanks.

Putting this on feature backlog and will answer @mmoore99 questions in new requirement that clarifies this in our new milestone.


Add ignored_metadata option

Remembering to add metadata fields to the matadata list is problematic, so here is a version that allows you to rather have a black list of tags you don't want which are added to the existing black list.
config :logger_json, :backend,
       metadata: :all,
       ignored_metadata: [:domain, :erl_level],
       json_encoder: Jason,
       formatter: LoggerJSON.Formatters.BasicLogger

I love this... Was just thinking about implementing this today.

This was fixed in the recent releases of LoggerJSON in a slightly different way: https://github.com/Nebo15/logger_json/blob/master/lib/logger_json.ex#L55. Thank you for PR anyways and sorry for such a long reply!


scripts/run_pca.py uses excessive memory

It runs out of memory on a machine with 360 GB of ram.
I guess we can solve it by avoiding loading all the descriptors into memory. We can load a subset of it, fit the PCA on the subset, and then apply the transformation to the remaining data.

That would work. Another option which I used for the City1M dataset is to use Incremental PCA (https://scikit-learn.org/stable/modules/generated/sklearn.decomposition.IncrementalPCA.html) with some batch size that fits in memory. Happy to accept PRs!


Trajectory Fails to Generate

Describe the bug
I was messing around with the new path planner, and several times the trajectory would fail to generate.
Screenshots
Example Screen shots that show the problem

Versions: (please complete the following information):
PathPlanner v2025.0.0 Beta 3

This is likely due to the control point placement of your end point. That's creating a really sharp kink in the curve that the generator can't handle. You need to either lengthen it or change its heading.


Minor clean up to simple client API and docs

Only one more thought: Should both client APIs re-export RelationalError and/or ConnectionError? At the very least I would argue for re-exporting the RelationalError type name (i.e. not its constructors), because it shows up in type signatures so much. ConnectionError only has to be handled in one spot in your app usually, and you probably don't write the type signature for it.

For now I've pushed a change to export RelationalError (and it's constructors) from both client APIs. I find it confusing when a module exports a type but not its constructors, especially when another module exports the constructors. If you're not a fan of that change, I'll just drop the latest commit.

I think you are right to export the constructors. It's nice to have an all-in-one API to import for newcomers.


Make label checks in one file, ignore draft PR

This PR:

condenses two label checkers into one file
Removes it running from draft PR's

Testing:

Commit 2 is done while in draft, has only yml changes, checks should be skipped
Commit 3 has a file edited that would normally trigger this, checks should be skipped
Commit 4 makes PR ready for review, CI should fail
Commit 5 removes the dummy edit, CI should pass

This seems to have somehow broke things, as not all expected checks are running, closing for now


Enable comments for blog posts?

I would like to use the comments plugin by the grav team. It is enabled (see it from admin panel) and I see it is included in this theme in templates/partials/blog-item.html.twig:
{% if page.header.continue_link is sameas(true) and config.plugins.comments.enabled %}
        {% include 'partials/comments.html.twig' %}
{% endif %}

What could be the reason that it is not appearing on the bottom of the blog posts?
Thanks!

Hi @zaszlo , I just downloaded the current Open Publishing Space skeleton (using Quark Open Publishing) and installed the Comments Plugin and it seemed to work:

Are you installing the Comment Plugin in the OPS skeleton or another site using the Quark Open Publishing theme?

Thanks for the feedback, tried with the modified snippet, but still the comment section didn't appear... any other hints on how to debug what is going on?

Did you clear the cache for your test (system.yaml file, both caches or via Admin Panel)? Also confirm that your active theme is the theme files you are editing - thanks!

I did clear the cache with method "all" and am using the My Quark Theme with only minor additions in templates copied from Quark Open Publishing:
item.html.twig
partials/blog-item.html.twig
partials/navigation.html.twig
The partials having the edit that you suggested here. But did that edit on both My theme and on the Quark Open Publishing.
...
        {% else %}
            {{ page.summary }}
        {% endif %}
    </div>

    {% if config.plugins.comments.enabled %}
        {% include 'partials/comments.html.twig' %}
    {% endif %}

</div>

{# added check for content display flag - hibbittsdesign.org #}
{% if not (grav.uri.param('chromeless')) %}
...

Interestingly if I add
    {% if config.plugins.comments.enabled %}
        OK
        {% include 'partials/comments.html.twig' %}
    {% endif %}

The OK is rendered fine. I installed the comments plugin from admin console and checked that it is enabled, did not touched it otherwise.

Hmm... I've made a new release of the theme with the above fix, perhaps try that?
I've also released a new Open Publishing Space skeleton with the new theme, you could download and test that to make sure there are no issues with your specific setup etc. https://getgrav.org/downloads/skeletons

It was my bad that I didn't created a new comments.yaml into my config/plugins and didn't changed the default enabled route from blog to home.

Glad you got things going!

Hi Paul,
I'm tested Open Publishing Space skeleton with the Comments Plugin.

I got this error:

Whoops \ Exception \ ErrorException (E_NOTICE)
Trying to access array offset on value of type null

I copy the contents of user/plugins/comments/comments.yaml in user/config/plugins/comments.yaml.
However, I notice that there is user/data/comments folder.  This seems normal to me because there are no comments on the articles.

Thanks for letting me know @LeMoussel 🙂


check whether Metadata contains Timestamp

Fixes #5787
Description
We check whether blob.Metadata.ContainsKey("Timestamp") before calling blob.Metadata["Timestamp"] to avoid key not found exception.
Please let me know if the "skip if not exist" behavior is good or not.
Specific Changes

skip if blob.Metadata.ContainsKey("Timestamp") is False in GetTranscriptActivitiesAsync

Hi @rmt2021
This class has been deprecated, and it looks like BlobsTranscriptStore would not have this issue as metadata and values are set in one call:
https://github.com/microsoft/botbuilder-dotnet/blob/main/libraries/Microsoft.Bot.Builder.Azure.Blobs/BlobsTranscriptStore.cs#L436
Nonetheless, the fix you've provided seems better than throwing a null reference exception. Thank you for looking into this.


Move tests into a new folder

Otherwise, they all get built individually and then again as part of a new main test target.
Related to #2581

this one doesn't pass the CI for some reason - is it due to tokio::main macro?

I think I accidentally left in the main.rs in tests in the commit itself 😅

I think I accidentally left in the main.rs in tests in the commit itself 😅

if you were using remote branches on the main repo, then I would have fixed it 😝

if you were using remote branches on the main repo, then I would have fixed it :stuck_out_tongue_closed_eyes:

I wasn't sure if I should do this or stick to my fork! Is it preferred that I push to this repo?

if you were using remote branches on the main repo, then I would have fixed it 😝

I wasn't sure if I should do this or stick to my fork! Is it preferred that I push a branch in this repo?

I guess it's the most widely used option (because it's easier and trivial to pull someone's branch and work together). using your fork is also allowed though!

I just tried, and it looks like I don't have access to push to this repo currently


Change workflow to matrix-test multiple go versions.

Just a small contribution, but testing multiple versions can be useful from time to time.
Change workflow to matrix-test multiple go versions and lower minimum go version for tests.

Thank you very much for your contribution, really anything is welcome. I hope you can use the library in your projects, greetings.


Please, update mode for the newer version of the game!

Hi there! I came back to the game with the new update and accidentally discovered that this mode is not working on this version. This mod was too funny can't imagine a game without it xd...

yep

Hi, it si fix please check moddb

...moddb? If you mean thunderstore, it is still not updated :P
And if you don't mean thunderstore, can you please provide the link for the mod?


tab names not centered to tabs themselves

no clue why this happens! i installed everything properly, i'd think?

actually wait i think i might know why this happens hold on


Change the nixpkgs pin

The existing commit fails to build on my machine with the error:
api.c:22:10: fatal error: 'endian.h' file not found
#include <endian.h>
         ^~~~~~~~~~
1 error generated.
make[3]: *** [Makefile:693: libseccomp_la-api.lo] Error 1
make[2]: *** [Makefile:918: all-recursive] Error 1
make[1]: *** [Makefile:498: all-recursive] Error 1
make: *** [Makefile:407: all] Error 2
builder for '/nix/store/y77qz7z2s4skis94n82ji10lfiqcn0mv-libseccomp-2.3.3.drv' failed with exit code 2

This PR fixes the problem.

I'd rather not move away from the unstable channel just yet.
The fix to the problem you're encountering for the moment is to set allowBroken = false in your Nix configuration.

Maybe we can add allowBroken=true; in our import of nixpkgs, just so it
doesn't depend on the user's config?
On Mon, Apr 9, 2018 at 3:25 AM, John Wiegley<EMAIL_ADDRESS>wrote:

Closed #153 https://github.com/jwiegley/hnix/pull/153.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jwiegley/hnix/pull/153#event-1563178113, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ABGlYE_yFiv8aExdp4eQeJRzScNDVOtuks5tmwz9gaJpZM4TLaO6
.

The commit I'm changing to is on the unstable channel.    The nixpkgs repo is the same as nixpkgs-channels in this case.


Added test case to create a data source from bucket page

Added test case to create a data source from bucket page.
Signed-off-by: Chebrolu Harika<EMAIL_ADDRESS>

Closing this as this scenario is taking care by #70


detect by 360 with thread inject

Thanks for the heads up. when I get some time I'll make some adjustments.


Improve notification

Describe the remaining issues

Both desktop and mobile versions have these same UI issues

[x] Unselected notification button
Expected:

Actual:

[x] Sending queue
Expected:

Actual:

[x] Notification cards
Expected:

Actual:

[x] Expanded card
Expected:

Actual:

[ ] Missing ! and Error badges

[ ] sending and files must have the first letter uppercase and need to adjust the spacing
Expected:

Actual:

[ ] All items (including cards and badges) of the mobile version in design seem to have different padding and sizes compared to the desktop one

[ ] Maybe try to use shrinkWrap for ListView instead of using fixed height

[ ] Some fontWeight are not as same as the design

[ ]  We need to remove the unread counter on the notification icon after tapping in

[ ] About a "Cancel" button - Should we need to add it to the app?

Updated the design in this PR : https://github.com/atsign-foundation/atmosphere_pro/pull/342

Updated the design in this PR : #342

@sachins-geekyants Added some new issues!


LOTR - make a TILE hierarchy for GPC+RC

LOTR - make a hierarchy for GPC+RC

what should this hierarchy be called?

The hierarchy will be called lotr_tile.
this is the building block of the lot fabric

@dkravche When you Push to origin the GPC_TILE  please "close with comment"

Done


PP: content change on the 1st prototype page

https://verify-local-patterns.herokuapp.com/service-patterns/parking-permit/example-service/start-page
Change this:
In some parts of Argleton, you'll need a parking permit to park your car or other vehicle.
Parking permits allow you to park within a given boundary, 24 hours a day and 7 days a week. - not needed as there's duplication between this page and residents parking permit page
To this:
You may need a parking permit to park your vehicle if you live in some parts of Argleton.

This varies between councils.
This issue does not document a problem.

@sanjaypoyzer I reworded it into an issue (see Title), either way I can see why we need to be repeating the same content on several pages of the prototype. Can this be looked into?

Ok, I've reworded it to be clearer.
Still not really sure why this is a problem though. It's on both those pages because the information would be useful at both parts of the journey.

In all our testing with users, we've seen that reading is often incomplete, and users forget what they've read as often as remembering it. As a principle, information should be available to the point of need. Requiring users to remember information they may or may not have read elsewhere is probably unwise.
also: https://www.nngroup.com/articles/recognition-and-recall/

Ok, happy to close it now


Failed to deploy traefik using rancher fleet

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What version of the Traefik's Helm Chart are you using?
10.24.0
What version of Traefik are you using?
2.8.0
What did you do?
We are using rancher (2.6.3) fleet (0.3.8) to deploy traefik into k3s clusters not previous deployed with traefik. The first deployment by fleet always fails for every cluster.
If we manually delete the failed traefik release, the subsequent deployment of traefik by fleet will success.
Here is the fleet configuration we are using:
# For full list of the sample fleet.yaml, please refer to https://github.com/rancher/fleet/blob/master/docs/gitrepo-structure.md

# The default namespace to be applied to resources. This field is not used to
# enforce or lock down the deployment to a specific namespace, but instead
# provide the default value of the namespace field if one is not specified
# in the manifests.
# Default: default
defaultNamespace: kube-system

helm:
  # Use a custom location for the Helm chart. This can refer to any go-getter URL.
  # This allows one to download charts from most any location.  Also know that
  # go-getter URL supports adding a digest to validate the download. If repo
  # is set below this field is the name of the chart to lookup
  chart: traefik

  # A https URL to a Helm repo to download the chart from. It's typically easier
  # to just use `chart` field and refer to a tgz file.  If repo is used the
  # value of `chart` will be used as the chart name to lookup in the Helm repository.
  repo: https://helm.traefik.io/traefik

  # The version of the chart or semver constraint of the chart to find. If a constraint
  # is specified it is evaluated each time git changes.
  version: 10.24.0

  values:
    # Use nodeport for traefik routing access. Default config with LoadBalancer would not work correctly
    # On a cluster wihtout LoadBalancer component.
    service:
      type: NodePort
    ports:
      web:
        nodePort: 80
      websecure:
        nodePort: 443
      # To access dashboard, portfoward 9000, then go to http://<IP>:9000/dashboard/
      # Otherwise, enable the following nodeport
      # # traefik:
      # #  nodePort: 9000
      # #  expose: t

What did you see instead?
Everytime, the first deploy by fleet will fail with the following message:
Release "traefik" failed: failed post-install: unable to build kubernetes object for deleting hook traefik/templates/dashboard-hook-ingressroute.yaml: unable to recognize "": no matches for kind "IngressRoute" in version "traefik.containo.us/v1alpha1"

What is your environment & configuration?
Rancher Server 2.6.3
Fleet: 0.3.8
K3s cluster engine: 1.20.6+k3s1, 1.21.14
Additional Information
No response

@mloiseleur , We have disabled both traefik and servicelb during the deployment of k3s:
--no-deploy=traefik,servicelb

After we moved to rancher 2.6.8, it seems the problem disappeared. We will monitor the status, and reopen it if we can reproduce the problem again.


ADBDEV-6472: Don't overwrite gpexpand template on the same host

Don't overwrite gpexpand template on the same host
In RedOS, when both the master and a segment are on the same host, running
gpexpand correctly was impossible.
Typically, the master is excluded from the gpexpand host array. However, if both
the master and a segment are on the same host, the master's host gets included
in the array. The issue arises due to the scp command corrupting and truncating
the gpexpand_schema.tar file when the host, source and destination paths are the
same.
Don't issue a scp command if we are dealing with the local host with and paths
are the same. Note that gpexpand always uses relative filepaths for scp's
destination, and that they are relative to /home/gpadmin/.
This problem does not occur in GPDB 7, as it uses rsync instead of scp, which
does not have said issue.

Allure report https://allure.adsw.io/launch/84435

Allure report https://allure.adsw.io/launch/84549

Allure report https://allure.adsw.io/launch/84625

Allure report https://allure.adsw.io/launch/84665

Allure report https://allure.adsw.io/launch/84782

Allure report https://allure.adsw.io/launch/85097

Allure report https://allure.adsw.io/launch/85142

Currently, this bug is reproducible only on RedOS.

under ubuntu this command
scp host:file host:file

also truncates file on host, where host is remote host

Allure report https://allure.adsw.io/launch/85283

Allure report https://allure.adsw.io/launch/85306

Allure report https://allure.adsw.io/launch/85418

Allure report https://allure.adsw.io/launch/85456

Allure report https://allure.adsw.io/launch/85551

Allure report https://allure.adsw.io/launch/86002


Online Status of User not updated on Login

Details:
When a user first logs in and opens a chat, the chat user is viewed as offline even if the user is actually online.
Steps to recreate:
Log in with an account in one device, say account A.
Login with account B on another device.
Open the chat with A while logged in with B.
You will see that A is offline, while A is actually online on the other device.
Would Like someone to fix it.

I'd like to take a peek into this. Assign me

Sure, have a look and please explain how you will solve it. I will assign you the issue then.

Hey, I'd have to set this project up on my machine, can you provide me with the .env variables REACT_APP_PROFILE_API and REACT_APP_API_KEY?
I understand the issue has something to do with socket.io but to dig deep, I have to have .env files setup correctly

REACT_APP_PROFILE_API is https://api.multiavatar.com. You will get the API key once you create an account in their website

I have fixed the issue but now I see that you also have been working on the same issue, it is not gonna work out if we both try to fix the issue with different approach.
I'll raise a PR shortly, merge it if you think it solves your issue.

I worked on implementing the feature, and hence the bug came around. Had some other work so could stick around to fix it, and so I raised this issue.


Screen Designer - tables don't have space for more than one row

Should increase height to allow more than one row to show.

Fixed


"Unable to read data from the transport connection" on AL: Publish with debugging from VS Code

Describe the issue
When trying to Publish with debugging from VS Code I get this error;
[2022-09-28 15:07:01.78] Publishing AL application using launch configuration 'Your own server'.
[2022-09-28 15:07:02.06] Publishing package to tenant 'default'
[2022-09-28 15:07:02.06] Targeting server 'https://<redacted>.westeurope.cloudapp.azure.com', server instance 'BC' and tenant 'default'.
[2022-09-28 15:07:02.07] Using user name and password authentication. User name used is: 'admin'.
[2022-09-28 15:07:02.08] Sending request to https://<redacted>.westeurope.cloudapp.azure.com:7049/BC/dev/apps?tenant=default&SchemaUpdateMode=synchronize&DependencyPublishingOption=default
[2022-09-28 15:07:21.10] Error: An error occurred while sending the request.
Unable to read data from the transport connection: A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond..
A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond.

[2022-09-28 15:07:21.10] Error: An error occured while processing the request.
Request ID: 4a710b00-a478-4fa5-ab90-877af6fa86b4
Session ID: 44e7e026-1334-4f88-97d5-f13a4c2669ff

If you are targeting a cloud instance, supply these IDs if contacting Microsoft support.

Scripts used to create container and cause the issue
Created the container with the SetupNavContainer.ps1 from the C:\DEMO\ folder. The Azure VM is created from https://aka.ms/getbcext.

Windows (Windows Server 2022 Datacenter)
Standard D4s v5 (4 vcpus, 16 GiB memory)

settings.ps1
$templateLink = "https://raw.githubusercontent.com/Microsoft/nav-arm-templates/master/getbcext.json"
$hostName = "<redacted>.westeurope.cloudapp.azure.com"
$storageConnectionString = ""
$containerName = "bcserver"
$isolation = "Default"
$vmAdminUsername = "vmadmin"
$navAdminUsername = "admin"
$azureSqlAdminUsername = "sqladmin"
$Office365UserName = ""
$Office365CreatePortal = "No"
$artifactUrl = "bcartifacts/sandbox//be/daily"
$navDockerImage = ""
$registryUsername = ""
$registryPassword = ""
$sqlServerType = "SQLExpress"
$azureSqlServer = ""
$appBacpacUri = ""
$tenantBacpacUri = ""
$databaseBakUri = ""
$includeAppUris = ""
$enableSymbolLoading = "No"
$includeCSIDE = "No"
$includeAL = "No"
$clickonce = "No"
$enableTaskScheduler = "Yes"
$licenseFileUri = "<redacted>"
$publicDnsName = "<redacted>.westeurope.cloudapp.azure.com"
$workshopFilesUrl = ""
$style = "demo"
$RunWindowsUpdate = "Yes"
$TestToolkit = "Libraries"
$AssignPremiumPlan = "Yes"
$CreateTestUsers = "No"
$CreateAadUsers = "No"
$Multitenant = "Yes"
$WindowsInstallationType = "Server"
$WindowsProductName = "Windows Server 2022 Datacenter"
$ContactEMailForLetsEncrypt = "<redacted>"
$RemoteDesktopAccess = "<redacted>"
$WinRmAccess = ""
$BingMapsKey = ""
$requestToken = ""
$createStorageQueue = "No"
$AddTraefik = "No"
$nchBranch = ""
$passwordKey = <redacted>
$adminPassword = "<redacted>"
$Office365Password = ""

Full output of scripts
PS C:\Users\vmadmin> C:\DEMO\SetupNavContainer.ps1
Using artifactUrl https://bcartifacts.azureedge.net/sandbox/20.5.45456.47005/be
Country be
Version 20.5.45456.47005
Locale nl-BE
Running container (this might take some time)
BcContainerHelper is version 4.0.4
BcContainerHelper is running as administrator
Hyper-V is Disabled
UsePsSession is True
Host is Microsoft Windows Server 2022 Datacenter - ltsc2022
Docker Client Version is 20.10.17
Docker Server Version is 20.10.17
Removing Session bcserver
Removing container bcserver
Removing entries from hosts
Removing bcserver from container hosts file
Removing bcserver-* from container hosts file
Removing Desktop shortcuts
Removing C:\ProgramData\BcContainerHelper\Extensions\bcserver
Fetching all docker images
Fetching all docker volumes
ArtifactUrl and ImageName specified
Image mybc:20.5.45456.47005-be already exists
Using image mybc:20.5.45456.47005-be
Publishing port 8080
Publishing port 443
Publishing port 7046
Publishing port 7047
Publishing port 7048
Publishing port 7049
PublicDnsName is <redacted>.westeurope.cloudapp.azure.com
Creating Container bcserver
Style: sandbox
Multitenant: Yes
Version: 20.5.45456.47005
Platform: 20.0.45448.46957
Generic Tag: <IP_ADDRESS>
Container OS Version: 10.0.20348.887 (ltsc2022)
Host OS Version: 10.0.20348.887 (ltsc2022)
Using process isolation
Using locale nl-BE
Disabling the standard eventlog dump to container log every 2 seconds (use -dumpEventLog to enable)
Using license file <redacted>
Downloading C:\ProgramData\BcContainerHelper\Extensions\bcserver\my\license.flf
Additional Parameters:
--env RemovePasswordKeyFile=N
--storage-opt size=100GB
--env CustomNavSettings=EnableTaskScheduler=true
Files in C:\ProgramData\BcContainerHelper\Extensions\bcserver\my:
- AdditionalOutput.ps1
- InstallCertificate.ps1
- license.flf
- MainLoop.ps1
- SetupCertificate.ps1
- SetupNavUsers.ps1
- SetupVariables.ps1
- SetupWebClient.ps1
- SetupWindowsUsers.ps1
- updatehosts.ps1
Creating container bcserver from image mybc:20.5.45456.47005-be
40050bfd95c84bb590e7844a042a3dcc7b79691a56d01b974286f1af6a7501a9
Waiting for container bcserver to be ready
Initializing...
Setting host.containerhelper.internal to <IP_ADDRESS> in container hosts file
Starting Container
Hostname is bcserver
PublicDnsName is <redacted>.westeurope.cloudapp.azure.com
Using NavUserPassword Authentication
Starting Local SQL Server
Starting Internet Information Server
Certificate File Thumbprint 08FCCEE4A4276A93635C4906788DAFD187A13549
Import Certificate to LocalMachine\my
Modifying Service Tier Config File with Instance Specific Settings
Modifying Service Tier Config File with settings from environment variable
Setting EnableTaskScheduler to true
Starting Service Tier
CertificateThumprint 08FCCEE4A4276A93635C4906788DAFD187A13549
Registering event sources
Creating DotNetCore Web Server Instance
Using application pool name: BC
Using default container name: NavWebApplicationContainer
Copy files to WWW root C:\inetpub\wwwroot\BC
Create the application pool BC
Create website: NavWebApplicationContainer with SSL
Update configuration: navsettings.json
Done Configuring Web Client
Enabling Financials User Experience
Using license file 'c:\run\my\license.flf'
Import License
Dismounting Tenant
Mounting Tenant
Mounting Database for default on server localhost\SQLEXPRESS with AllowAppDatabaseWrite = False
Sync'ing Tenant
Tenant is Operational
Creating http download site
Creating Host Windows user
Setting SA Password and enabling SA
Creating admin as SQL User and add to sysadmin
Creating SUPER user
WARNING: The password that you entered does not meet the minimum requirements.
It should be at least 8 characters long and contain at least one uppercase
letter, one lowercase letter, and one number.
Assign Premium plan for ADMIN
Container IP Address: <IP_ADDRESS>
Container Hostname  : bcserver
Container Dns Name  : <redacted>.westeurope.cloudapp.azure.com
Web Client          : https://<redacted>.westeurope.cloudapp.azure.com/BC/?tenant=default
Dev. Server         : https://<redacted>.westeurope.cloudapp.azure.com
Dev. ServerInstance : BC
Dev. Server Tenant  : default
Setting bcserver to <IP_ADDRESS> in host hosts file
Setting bcserver-default to <IP_ADDRESS> in host hosts file
Setting bcserver-default to <IP_ADDRESS> in container hosts file

Files:
http://<redacted>.westeurope.cloudapp.azure.com:8080/ALLanguage.vsix

Container Total Physical Memory is 16.0Gb
Container Free Physical Memory is 9.8Gb

Initialization took 47 seconds
Ready for connections!
Reading CustomSettings.config from bcserver
Creating Desktop Shortcuts for bcserver
Skipping app 'C:\Applications.BE\Microsoft_Permissions Mock_20.5.45456.47005.app' as it is already installed
Skipping app 'C:\Applications.BE\Microsoft_Test Runner_20.5.45456.47005.app' as it is already installed
Skipping app 'C:\Applications.BE\Microsoft_Any_20.5.45456.47005.app' as it is already installed
Skipping app 'C:\Applications.BE\Microsoft_Library Assert_20.5.45456.47005.app' as it is already installed
Skipping app 'C:\Applications.BE\Microsoft_Permissions Mock_20.5.45456.47005.app' as it is already installed
Skipping app 'C:\Applications.BE\Microsoft_Library Variable Storage_20.5.45456.47005.app' as it is already installed
Skipping app 'C:\Applications.BE\Microsoft_System Application Test Library_20.5.45456.47005.app' as it is already installed
Skipping app 'C:\Applications.BE\Microsoft_Tests-TestLibraries_20.5.45456.47005.app' as it is already installed
TestToolkit successfully imported
Container bcserver successfully created

Use:
Get-BcContainerEventLog -containerName bcserver to retrieve a snapshot of the event log from the container
Get-BcContainerDebugInfo -containerName bcserver to get debug information about the container
Enter-BcContainer -containerName bcserver to open a PowerShell prompt inside the container
Remove-BcContainer -containerName bcserver to remove the container again
docker logs bcserver to retrieve information about URL's again
Copying .vsix and Certificate to C:\ProgramData\bccontainerhelper\Extensions\bcserver
Container output
Initializing...
Setting host.containerhelper.internal to <IP_ADDRESS> in container hosts file
Starting Container
Hostname is bcserver
PublicDnsName is <redacted>.westeurope.cloudapp.azure.com
Using NavUserPassword Authentication
Starting Local SQL Server
Starting Internet Information Server
Certificate File Thumbprint 08FCCEE4A4276A93635C4906788DAFD187A13549
Import Certificate to LocalMachine\my
Modifying Service Tier Config File with Instance Specific Settings
Modifying Service Tier Config File with settings from environment variable
Setting EnableTaskScheduler to true
Starting Service Tier
CertificateThumprint 08FCCEE4A4276A93635C4906788DAFD187A13549
Registering event sources
Creating DotNetCore Web Server Instance
Using application pool name: BC
Using default container name: NavWebApplicationContainer
Copy files to WWW root C:\inetpub\wwwroot\BC
Create the application pool BC
Create website: NavWebApplicationContainer with SSL
Update configuration: navsettings.json
Done Configuring Web Client
Enabling Financials User Experience
Using license file 'c:\run\my\license.flf'
Import License
Dismounting Tenant
Mounting Tenant
Mounting Database for default on server localhost\SQLEXPRESS with AllowAppDatabaseWrite = False
Sync'ing Tenant
Tenant is Operational
Creating http download site
Creating Host Windows user
Setting SA Password and enabling SA
Creating admin as SQL User and add to sysadmin
Creating SUPER user
WARNING: The password that you entered does not meet the minimum requirements.
It should be at least 8 characters long and contain at least one uppercase
letter, one lowercase letter, and one number.
Assign Premium plan for ADMIN
Container IP Address: <IP_ADDRESS>
Container Hostname  : bcserver
Container Dns Name  : <redacted>.westeurope.cloudapp.azure.com
Web Client          : https://<redacted>.westeurope.cloudapp.azure.com/BC/?tenant=default
Dev. Server         : https://<redacted>.westeurope.cloudapp.azure.com
Dev. ServerInstance : BC
Dev. Server Tenant  : default
Setting bcserver to <IP_ADDRESS> in host hosts file
Setting bcserver-default to <IP_ADDRESS> in host hosts file
Setting bcserver-default to <IP_ADDRESS> in container hosts file

Files:
http://<redacted>.westeurope.cloudapp.azure.com:8080/ALLanguage.vsix

Container Total Physical Memory is 16.0Gb
Container Free Physical Memory is 9.8Gb

Initialization took 47 seconds
Ready for connections!
Container setup complete!

Screenshots

Additional context

I've try'ed to search in the Event Log for clues, but not sure if one of these warnings is responsible for this error.

does it happen all the time?
When I completely remove the Resource Group from Azure and build a new VM it seems the error message is gone for a few days. I change often the $artifactUrl in the settings.ps1 to jump between Belgium and W1 containers.

did it use to work?
Yes

the way to recreate the container inside the VM is by using the function:
Replace-BcServerContainer -artifactUrl $artifactUrl

Not sure what it does different - but I would use that.
You could also add traefik to the VM and have both BE and W1 containers there.

Good advise! I'll try this and create a new issue if this problem reoccurs.


Add automated releasing

Add automated build/release on updates to the main branch and any new tags
Closes https://github.com/operator-framework/combo/issues/96

I had one more comment but I think we're getting close.

Just thought of a follow-up for this. We'll probably want to create a Bundle format that we can submit for both OLM and RukPak. Not required for this PR but I'll create a follow-up.

Just thought of a follow-up for this. We'll probably want to create a Bundle format that we can submit for both OLM and RukPak. Not required for this PR but I'll create a follow-up.

@tylerslaton Does this issue cover that?

@oceanc80 It sure does, good catch!


Merging template updates

This is a big one, template update 2.14 implemented, updated Mesmer, Cellpose, MultiQC, respective tests.
Dev is version-bumped to 1.1.0dev
Seqera container for MultiQC 1.22.1 to circumvent issue in gitpod - should this be only provided by the user?
https://github.com/nf-core/molkart/issues/59
https://github.com/nf-core/molkart/issues/62
https://github.com/nf-core/molkart/issues/70
*** introduces bug that multiqc doesn't show custom qc results - neither with crops, nor quantifictions - should be fixed ASAP, but this PR is giant already.
PR checklist

[x] This comment contains a description of changes (with reason).
[x] If you've fixed a bug or added code that should be tested, add tests!
[x] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[ ] If necessary, also make a PR on the nf-core/molkart branch on the nf-core/test-datasets repository.
[x] Make sure your code lints (nf-core lint).
[x] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
[x] Check for unexpected warnings in debug mode (nextflow run . -profile debug,test,docker --outdir <OUTDIR>).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[x] CHANGELOG.md is updated.
[x] README.md is updated (including new tool citations and authors/contributors).

One additional comment. Singularity tests failed for me after the template update.
nextflow run . -profile test,singularity --outdir ./test
The error message I received was:
ERROR ~ Error executing process > 'NFCORE_MOLKART:MOLKART:MINDAGAP_DUPLICATEFINDER (mem_only)'

Caused by:
  Process `NFCORE_MOLKART:MOLKART:MINDAGAP_DUPLICATEFINDER (mem_only)` terminated with an error exit status (255)

Command executed:

  duplicate_finder.py \
      spots.txt \
      90

  cat <<-END_VERSIONS > versions.yml
  "NFCORE_MOLKART:MOLKART:MINDAGAP_DUPLICATEFINDER":
      mindagap: $(mindagap.py test -v)
  END_VERSIONS

Command exit status:
  255

Command output:
  (empty)

Command error:
  INFO:    Environment variable SINGULARITYENV_NXF_TASK_WORKDIR is set, but APPTAINERENV_NXF_TASK_WORKDIR is preferred
  INFO:    Environment variable SINGULARITYENV_NXF_DEBUG is set, but APPTAINERENV_NXF_DEBUG is preferred
  FATAL:   container creation failed: image driver mount failure: image driver squashfuse_ll instance exited with error: squashfuse_ll exited: fuse: failed to open /dev/fuse: Operation not permitted

Can you check on your end whether it works on HPC @kbestak ?


Test: Improved --wait behaviour

Test for https://github.com/Microsoft/vscode/issues/18199 and https://github.com/Microsoft/vscode/issues/28661
Complexity: 3

[ ] Windows - @egamma
[x] macOS - @weinand
[x] Linux - @joaomoreno

VS Code can be started with the --wait argument from the command line and this will cause the terminal to wait until the VS Code instance is closed. A popular use case is to configure VS Code as git editor to edit git commit messages (see https://stackoverflow.com/questions/30024353/how-to-use-visual-studio-code-as-default-editor-for-git)
The implementation of this feature was rewritten to fix https://github.com/Microsoft/vscode/issues/18199. Verify using the CLI (e.g. code --wait <path to some file>):

you get back to the prompt when starting an initial instance and closing that instance
same when starting a second instance and closing that instance or all instances
same when starting an initial instance, opening a second window and closing the first window

In addition, we will close the window when the last editor is closed of that instance (this is https://github.com/Microsoft/vscode/issues/28661). Verify:

the window closes and you get back to the prompt when closing the last editor of the instance that was opened with --wait

Finally, verify that VS Code can still be used as git editor as outlined in https://stackoverflow.com/questions/30024353/how-to-use-visual-studio-code-as-default-editor-for-git

Works like a charm!!


Always adds to the default deck, even if it is renamed

A potential resolution to https://github.com/JASchilz/AnkiSyncDuolingo/issues/56. Grabs the default deck by ID instead of by name. This way, it can add to the default deck even if the user renames it.
I was inspired to open this PR because I renamed by default deck to "zDefault" so it would show at the bottom of my list. Before this change, the "Default" deck would be created by AnkiSyncDuolingo, but it would add the cards to the "zDefault" deck.
This PR is one attempt at solving the problem, but I can imagine its behavior may still be confusing. For example in https://github.com/JASchilz/AnkiSyncDuolingo/issues/56 the user renamed the deck to something else, and their cards were getting added to that deck instead of "Default". This PR would just make the cards sync to the renamed default deck without creating an empty "Default" deck, which is especially confusing if the user doesn't remember they renamed the default deck.
Some alternative solutions which may be better than this PR:

Adding the cards to a newly created "Default" deck

May not work well if users rename the default deck
At least this would be consistent with the add on description / readme

Adding a setting for where cards should go by default

More engineering work
Can cover the weird cases where people rename the default deck
Essentially requested in https://github.com/JASchilz/AnkiSyncDuolingo/issues/48
Considering Anki plugins have a built in JSON config editor, this may be possible to implement with no UI work

Looks like a strict improvement over the current strategy.


Hooks not effecting rows when called using create_or_update in django 4.2

from dataclasses import dataclass
from typing import List, Optional

from django_lifecycle import BEFORE_CREATE, BEFORE_SAVE, BEFORE_UPDATE, LifecycleModel
from django_lifecycle.decorators import HookConfig

@dataclass(order=False)
class UpdateFieldsHookConfig(HookConfig):
    # Is only required for a single hook,
    update_fields: Optional[List[str]] = None

update_fields_hook = UpdateFieldsHookConfig

class UpdateFieldsLifecycleModel(LifecycleModel):
    class Meta:
        abstract = True

    def save(self, *args, **kwargs):
        skip_hooks = kwargs.pop("skip_hooks", False)
        save = super().save

        if skip_hooks:
            save(*args, **kwargs)
            return

        # unsure about next line, if running twice is OK, but ought to be...
        self._clear_watched_fk_model_cache()
        is_new = self._state.adding

        if "update_fields" in kwargs:

            if is_new:
                methods = self._get_hooked_methods(BEFORE_CREATE, **kwargs)
            else:
                methods = self._get_hooked_methods(BEFORE_UPDATE, **kwargs)

            methods += self._get_hooked_methods(BEFORE_SAVE, **kwargs)
            fields = set()
            for hooked_method in methods:
                for hook in hooked_method.method._hooked:
                    if hook.update_fields:
                        fields = fields.union(set(hook.update_fields))
            kwargs["update_fields"] = list(set(kwargs["update_fields"]).union(fields))

        save(*args, **kwargs)

usage: same as before but you gotta swtich to our mixin and hook decorator, just add update_fields to the hook params
@update_fields_hook(BEFORE_CREATE, update_fields=["anchor_status"])

My basic fix for it. caused by this change in django https://docs.djangoproject.com/en/4.2/releases/4.2/#setting-update-fields-in-model-save-may-now-be-required which makes it so only fields which are being passed into the update kwags, so if you're updating a field based on an updated field the code will run, but not change in the db. This code above works, but maybe there's better solutions, if I'm able to get a PR approved.

Thanks for reporting this bug! 🐞
That could be a solution, but I don't like having to specify which fields are going to be updated in the decorator because in your hooked methods you may or may not update fields.
One option could be to automagically check which fields have changed (via has_changed) and add them to update_fields, but this may not be for everyone.
Another option could be to have a variable in the model to make the developer responsible of adding which fields have updated?
In your code:
class MyModel(LifecycleModel):
    title = models.CharField(max_length=50)
    slug = models.SlugField(max_length=50)

    @hook(BEFORE_CREATE)
    def set_slug_from_title(self):
        self.slug = slugify(title)
        self._lifecycle_update_fields.add("slug")

and in LifecycleModelMixin:
class LifecycleModelMixin:
    def save(*args, **kwargs):
        # ...
        if "update_fields" in kwargs:
            kwargs["update_fields"] = list(set(kwargs["update_fields"]).union(self._lifecycle_update_fields))

        super(*args, **kwargs)
        # ...

Maybe both options, configurable via a setting?
What do you think that it could be the best option?

One option could be to automagically check which fields have changed (via has_changed) and add them to update_fields, but this may not be for everyone.

This might be the least painful way, but I do see some dangers with this, as if someone is explicitly trying to save only a certain field, then this will add all changed fields, which would be very hard to debug why.

self._lifecycle_update_fields.add("slug")

Not sure if I see this as a better choice then having it in the hook itself... or is your worry about having it in the hook that it will be required to be listed in every hook if there's multiple decorators on a method?
But this does seem to be a fairly good way to resolve this issue, so long as its highlighted in the docs.

Maybe both options, configurable via a setting?

This seems to be the best option! :100: by default it can be off (to not cause unexpected side effects to other peoples code), and have it be a setting (either on the model or global level). Or another mixin LifecycleUpdateFieldsMixin which auto adds the fields. :thinking:
Main points would be:

Clearly noted in the docs that something is required to be done in the case of create_or_update in dj >=4.2
By default no magic updating update_fields
Settable per model or global


Upgrade to consul 0.8.2

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Crossfade between loading states

Hi!
CrossfadePlugin seems to imply Crossfade animation between the described image loading states. In fact, Crossfade does not work as state -> state but as empty -> state.
This is more like a bug.
Example:
CoilImage(
    modifier = Modifier.matchParentSize(),
    imageModel = { url },
    component = LocalImageComponent.current,
    previewPlaceholder = painterResource(id = Icons.ic_logo),
    loading = loadingPlaceholder,
    failure = failurePlaceholder,
)

In this example, I expect behavior:
loadingPlaceholder crossfades to success/failure, but I get: loadingPlaceholder -> clear image box -> crossfade to success/failure.

Hey @TemMax, it would be very helpful to provide any screenshot or video for this along with the details of the component parameter that you've used.


⚠ Bofrost.at has degraded performance

In d2e31b3, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2315 ms

Resolved: Bofrost.at performance has improved in 65686fc.


Update backpack_role

Update permissions from the backpack role to get every object in the cluster. We're currently missing some metadata such as:

network-attachment-definitions.k8s.cni.cncf.io
networks.operator.openshift.io
dnses.config.openshift.io

The update looks a bit aggressive, but will make our life easier in the future :)
Signed-off-by: Raul Sevilla<EMAIL_ADDRESS>

needs a rebase.

Can we merge this one?


Add feature note about the ilab command redesign

This adds a feature note about the redesign as requested by @leseb. Please let me know if this is not descriptive enough!
resolves #1333

I believe this is a must-have before releasing 0.17 (today?)

@leseb we will probably do the release tomorrow - @cdoern can you implement this change so we can merge it before then?

done!


[Feature Request] Literal strings in JSON post body (no environment variable replacement)

Insomnia Version: <IP_ADDRESS>12
Operating System: macOS Mojave 10.14.5

Details
Would like to be able to turn off environment variables for JSON body posts or have an option to use a literal string without environment variable replacement. Using some handlebars variables that use the same format inside of a string that I want treated literally, not replaced. Since they aren't known environment variables, Insomnia is throwing errors.
Example:
{
   "myfield": "{{ handlebars_variable_doesnt_exist_in_environment }}"
}

You can do this in the request settings, or surround your value in {% raw %}...{% endraw %}

@gschier can you ellaborate?

@reberthkss If you click on the arrow next to a particular request, then choose Settings, you'll get the options pictured above. Checking the one that's not checked will prevent variable substitution in the body of the request. Would be nice if there was a more general way to do this rather than on per-request basis though.


Add --exclude option to upgrade command to exclude a specific app when upgrading packages using --all

Description of the new feature / enhancement
I want to upgrade packages but i don't want to update a specific app. It will be extremely helpful to have --exclude following the name, ID package to exclude when using the winget upgrade --all

Proposed technical implementation details
No response

Use winget pin add <ID> to exclude a package from winget upgrade --all


replace transforms: Don't go through all nodes

This makes it so that the replace transformer stops recursing through the AST as soon as it finds the level where it finds some statements.
I ran level-2 5 times on each version. Times are in milliseconds (and ordered from lower to higher)

Version
Attempt 1
2
3
4
5

Before
1595
1658
1691
1709
1769

After
1534
1541
1546
1566
1580

This averages on a ~8% speedup on the entire run of the program.
I compared the output of level-2 before and after this change and they are identical.
I haven't looked at other transformations, but maybe the same optimization can be applied in other places as well.

Nice!

Yeah, makes sense because all the replacements are top level!


Add GIMP Plugin to debian packaging

Added changes to the debian folder that I've been using to build the GIMP plugin on launchpad.
Also added clang to the build dependencies, though it's not needed and can be removed.
Removed debian/tmp/ from the install files, since it doesn't seem to be necessary.

@googlebot I consent


Which census block groups or census tracts keep coming up in every index or score experiment?

Also, which don't show up consistently? Which tools capture which tracts? What tool or indicator allows for example permafrost or coal communities to be highlighted?
This might be helpful in identifying core inputs relevant across geographies and programs.

As a part of this you'll probably need to pick a cut off for each tool/indicator/score! A very useful feature idea.


Parser for lambda lists

There are a lot of generic half-baked ones. It also may include the discussion of what can be added/changed about lambda-lists structure – such as pattern matching.

https://github.com/s-expressionists/Ecclesia includes such utilities as a library.

Right, I (or you can) can add that to the Current status part. But the "lambda-lists structure" is more of a point, I guess I maybe shouldn't dismiss the existing libraries as half-baked, or at least do it less. I would want a point like that, maybe more general, referring to how variables are declared/bound at all....I struggle to word that precisely.

Right, I (or you can) can add that to the Current status part. But the "lambda-lists structure" is more of a point, I guess I maybe shouldn't dismiss the existing libraries as half-baked, or at least do it less. I would want a point like that, maybe more general, referring to how variables are declared/bound at all....I struggle to word that precisely.


Better diagram for Wed demo

Description
Better diagram for Wed demo
Type of change

Documentation update

Testing steps
N/A

Codecov Report

Merging #92 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #92   +/-   ##
=======================================
  Coverage   48.79%   48.79%
=======================================
  Files          11       11
  Lines         912      912
=======================================
  Hits          445      445
  Misses        413      413
  Partials       54       54

Continue to review full report at Codecov.

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


Use LC-X for topo db key

This PR reverts part of https://github.com/atlanticwave-sdx/sdx-controller/pull/328, to use LC-X as database key. Because using domain name as db key caused some issues on provisioning connection.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>1
Details

0 of 7 (0.0%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.02%) to 58.374%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sdx_controller/handlers/lc_message_handler.py
0
2
0.0%

sdx_controller/messaging/rpc_queue_consumer.py
0
5
0.0%

Files with Coverage Reduction
New Missed Lines
%

sdx_controller/handlers/lc_message_handler.py
1
24.0%

Totals

Change from base Build<PHONE_NUMBER>4:
-0.02%

Covered Lines:
1086

Relevant Lines:
1918

💛  - Coveralls


test(tsm1): TagKeys/TagValues stops scanning when context is cancelled

Closes #
Describe your proposed changes here.

[ ] CHANGELOG.md updated with a link to the PR (not the Issue)
[ ] Well-formatted commit messages
[ ] Rebased/mergeable
[ ] Tests pass
[ ] http/swagger.yml updated (if modified Go structs or API)
[ ] Documentation updated or issue created (provide link to issue/pr)
[ ] Signed CLA (if not already signed)

alchemy please merge


New profile

@massg  try this year for Hacktoberfest and have a look at the updated repo and contribute.
HAPPY HACKING


Questions about the 3D match training data

Hello, thanks for sharing the well-organized code. I have downloaded the training data of 3dmatch from the link provided in scripts/download_datasets.sh.  I found there only 7000+ scene pairs with 0.3 overlap threshold while there are tens of thousands frames in even one sequence of the original 3dmatch dataset. Have you conducted some sampling operations and are these all the training data?
Thanks in advance :)

The commercial depth sensors generate noisy samples that are not suitable for registration. A typical pipleline first integrate 50-100 frames into one point cloud. We also followed this procedure and created N^2 pairs within one sequence and then removed pairs < 0.3 overlap.

Got it. Thanks for your reply.

BTW, it seems that the judgement of overlap is conducted in point clouds with 0.1 voxel size downsampling while 0.05 for the test data, is that true ?


:any is incredibly powerful. Is it too much?

:any can produce all kinds of ridiculous selectors, to capture all kinds of cool situations
Some things that are non-desirable I have questions and concerns about are:

we lose the ability to generalize to a parent by selecting multiple different children. ( this is probably good as this didn't always make sense and we can select parents another way )
these selectors look crazy ( a little bit ), I'm worried they may produce sets which are hard to reason about. Maybe I'm just not used to it (since this is a change and an extension in behaviour to what I've used for ages) and other people shall find it useful, but maybe it is troublesome.

I need to test out the interface more ( in the product is a better idea )
and consider if I need to put any ( or no any ) behind an option, or roll any back completely.

We definitely want any. But right now any is not working correctly.
One reason is, we basically only apply a considewration of any when we are in
lcs_read mode ( in other words forming the lcs transcript after we have found the max
value )
We ought to apply consideration of any upstream, when we are actually filling out the matrix with quotients
We ought to consider any when we are forming those scores


Qa extreme precipitation fix legendtoggle

When users click on the legend buttons in a chart, the visibility of the corresponding series in the chart is toggled on/off. The main purpose of this QA fix is to add this functionality to the Intensity chart.

 Tools: Extreme Precipitation Events


Running R Script as Executable

Hi ,
Im new to RTVS . Im trying to run the R Script with the windows application . please help me in resolving this . Actually when i press one button in windows application this r script need to executed automatically

I mean to say is there any way to run the R Script as like as exe ..

Have a look here https://github.com/MikhailArkhipov/RTVS-cs


Not able to use reg add. Getting syntax error

Windows Terminal version
1.16.10262.0
Windows build number
10.0.22621.0
Other Software
No response
Steps to reproduce
Try to add reg value but getting syntax error
reg add HKEY_CURRENT_USER\software\microsoft\office\16.0\outlook\search /t REG_DWORD /v test /d 1
Expected Behavior
expected to add the reg value
Actual Behavior
syntax error occur

Can you share a screenshot?
This isn't exactly something Terminal is in control of, but we can still try to help.

Pleasee see the screenshot.
If I run the command in PowerShell Terminal there is no issue.
If I run the command using normal cmd there is no issue

Huh.
This might sound weird, but are you typing the command in, or copy-pasting it/? I wonder if there's a paste filter in conhost that might translate characters that look like . or / into the ASCII ones.

Hmm, I copy the command in NotePad then it works. Seems Terminal not able to automatic change the ASCII or otherway around

Note to team:
we should either...

make this the epic for special character filtering for paste, or...
find a dup (possibly #9380, #6134)

/dup #6134


Reset to factory default

Summary of the new feature/enhancement

Proposed technical implementation details (optional)

closing this. need more details


fix(core): respect specified schema when populating (select-in)

Closes #2675 issue.

Codecov Report

Merging #2676 (8a554df) into master (4393337) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master     #2676   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          193       193
  Lines        11283     11283
  Branches      2568      2568
=========================================
  Hits         11283     11283

Impacted Files
Coverage Δ

packages/core/src/entity/EntityLoader.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

makes sense, thank you!


#1348 - map subprocess stderr to caller stderr

Linking the issue for the requested info on motivation... https://github.com/boto/botocore/issues/1348

@jamesls @JordonPhillips Another use case where stderr getting swallowed makes it much harder to process MFA prompts (particularly when called via credential_process), https://github.com/segmentio/aws-okta/issues/143. Anything we can do to unblock this pr?

Plus one to this. I just wrote a similar patchset for this before checking the PRs, after spending several hours writing something similar to segment's aws-okta command mentioned above. I would love to see this in the aws cli to simplify MFA workflows.
My patchset checks if stdin/stderr are ttys, and only then pass them through. Would this be an acceptable solution to the maintainers?

FWIW AWS libraries in other languages already do this:
https://github.com/aws/aws-sdk-go/blob/master/aws/credentials/processcreds/provider.go#L347

Can we please merge this PR? This is literally breaking pretty much all of the tooling that needs to request more details before providing credentials. If there's anything that still needs to be done I'm happy to volunteer.

Closing because this problem doesn't impact me any more (and having it in my dashboard was becoming irksome) - if someone else wants to clone it - please do.


Remove quantity_to_string Print Statement

Description
Removes a redundant print statement from the quantity_to_string method.
Status

[x] Ready to go

Codecov Report

Merging #315 into 0.3.0-branch will decrease coverage by <.01%.
The diff coverage is n/a.

@@               Coverage Diff                @@
##           0.3.0-branch     #315      +/-   ##
================================================
- Coverage         75.66%   75.66%   -0.01%
================================================
  Files                17       17
  Lines              5133     5132       -1
================================================
- Hits               3884     3883       -1
  Misses             1249     1249

Impacted Files
Coverage Δ

openforcefield/utils/utils.py
91.08% <ø> (-0.06%)
:arrow_down:

Continue to review full report at Codecov.

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


[Secrets Store CSI Driver] Azure Vault

Use: https://github.com/Azure/secrets-store-csi-driver-provider-azure

Read : https://medium.com/containers-101/gitops-secret-management-with-azure-csi-secret-store-960800a550e6

/reopen

/reopen

/reopen

/reopen


Update Vector Search to 0.10.3 and update numpy minimum version to 2.0.0

What
Release 0.10.3:

https://github.com/TileDB-Inc/TileDB-Vector-Search/releases/tag/0.10.3
https://pypi.org/project/tiledb-vector-search/0.10.3/
https://anaconda.org/conda-forge/tiledb

Note that we leave - tiledb >=2.26,<2.27 because Vector Search 0.10.3 does not update Core.
Beyond the 0.10.3 update, I also updated recipe/conda_build_config.yaml from:
numpy:
  - 1.22
  - 1.22
  - 1.23

to
numpy:
  - 2.0.0
  - 2.0.0
  - 2.0.0

And then ran ~/repo/tiledb-vector-search-feedstock conda smithy rerender.
Testing
CI passes.

I'm fine with migrating to numpy 2 since tiledb-py-feedstock was already migrated (conda-forge/tiledb-py-feedstock#238) and the conda-forge migration is almost complete.

Sounds good, thanks.

Note though that it would be nice to stop manually managing the python/numpy versions, and just use the upstream conda-forge pins. Is there a reason we can't build vector search for python 3.12? (or are we just trying to save storage space?)

I'm not sure about this - cc @dudoslav / @ihnorton?