sessionKey pre-pending "sumologic.logger.session"

Sorry in advance if this isn't the right place for this - but I was wondering why the session key adds "sumologic.logger.session" to the beginning of every post.
I have my config set up like:
// generate guid using Guid Node package
const sessionID = Guid.raw();

// set options for sumo-logger
var sumoLoggerOpts = {
    endpoint: "MY_SOURCE_ENDPOINT",
    interval: 20000, // Send messages in batches every 20 seconds
    sendErrors: true,
    sessionKey: sessionID
};

Now I would expect the session ID to equal whatever guid is generated so the message looks like:

{"msg":"session started for user: nwilson with session id: 613dd23b-3422-96b7-3942-c2d08f00bbe2","sessionId":"613dd23b-3422-96b7-3942-c2d08f00bbe2","timestamp":"Thu, 22 Feb 2018 00:19:36 GMT","url":""}

Instead, it looks like:

{"msg":"session started for user: nwilson with session id: 613dd23b-3422-96b7-3942-c2d08f00bbe2","sessionId":"sumologic.logger.session613dd23b-3422-96b7-3942-c2d08f00bbe2","timestamp":"Thu, 22 Feb 2018 00:19:36 GMT","url":""}

I know where to fix it (in sumoLogger.js on line 5, or 28) but I was curious what its purpose is. Thanks!

Really just to ensure a unique identifier in the cookie. If you think it's unnecessary I'm open to a change. Especially one that is sent via pull request :)

Cool, I'll do that then :) Thanks!

I think a prepended unique identifier is good, but should be controlled by the user.  perhaps add it as a property when configuring the instance of the logger and allow the user to set the value?

I was thinking the same thing - however, the developer already sets the session ID. If they want something pre-pended, they can do it in the configuration options already. Unless there's some reason to have it separated?

Actually now that I'm re-reading this, @billsaysthis , you said it's for a unique ID on the "cookie". Shouldn't the cookie's ID be a separate thing from the session ID that goes into what's sent to Sumo?

Marking as enhancement for next iteration. Or pull request if someone wants to add as configurable option ;)


Code examples using custom serializers

Is your feature request related to a problem? Please describe.
I have tried searching the internet for examples, but all the examples with custom serialization was using the byte array methods.
Describe the solution you'd like
It would be nice with a code example that shows how to use gRPC with a custom serializer and not using byte[] arrays, i.e.
using the marshaller that expects:
Action<Something, SerializationContext> as serializers.

Could we have an update on this?

We don't have bandwidth to write this example. Could you help on this? or Flatbufer's gRPC integration might be useful as a practical example using the custom serializer.


Add CLI call feature for overriding any config.json parameter

Following the latest lab meeting discussion, we are considering adding a feature so we can call the Ivadomed framework in the command line, by overriding any parameter from the config.json file.
CLI parameter override already exists in ivadomed but only for a few parameter (namely --train/test/segment, --path-data, --path-output etc.)
As @charleygros mentioned during the meeting, the proposed feature is already implemented within the SCT toolbox, so we can get inspired for this functionality from sct_run_batch

Hey, @mpompolas , So I reviewed the issue. If I am understanding, the basic requirement of the features are:

There should be a Model/Singleton where all default key/parameters are defined with default/values/settings preconfigured (currently this is being take care  of by config.json as a default template).

Work with a model configuration that allow override from CLI arguments.

Ideally and optionally, work with a model configuration that allow override from ANOTHER specification/configuration file (JSON/YAML/ENV etc)

Ideally and optionally, the CLI argument key will be auto checked against the "model" to ensure it is anticipated.

Ideally and optionally, the CLI argument key data type will be checked and validated before the actual CLI commands move forwards

Current Model  (Status Quo)

JSON based configuration as a MANDATORY field for pretty much all ivadomed binaries
Have a default JSON (that user needs to duplicate and update)
CLI parameters are not generatelly from JSON.
CLI parameters are not staticically enforced or centrally managed .
Each /script/ binaries does its own argument parsing, for specific purposes per command SOME of these (but not all) are overwriting specific portion of the configuration JSON

So I dug this a bit more, it is a complex topics with many choices than I thought possible.
This is a good introductory article on the complexity of the issue.

YACS:

it is very MODEL driven. A prototype nodal based default model is defined. Then, it has built in functions to be used to MERGE from file or gets specific parameters via cli
Most notably, it requires YAML configuration migration away from the current JSON approach
still require manually CLI parameter parsing, handling/typechecking

Dynaconf

Support JSON, YAML, ini, env etc and can load everything into a EASILY accessible object/class/dict.
Downside is CLI arguments still require manual handling, parsing.

OmegaConf or Facebook Hydra:

Require migration to YAML/YML. Probably allow the most sophisticated multi-tier YAML overwrite
Very strong type checking
Downside is CLI arguments still require manual handling, parsing.

Confuse:

Very good YAML layering and also CLI generation with validation
Does not do JSON configuration file.

Typer/Click/Google Fire:

More about CLI argument (auto)generation
No JSON/YAML/YML support. Usually are about generate Python function arguments into  ready to use CLI parameters/arguments.

JSONargparse:

seems more about parsing JSON into Arguments for CLI where as our needs is to use CLI to overwrite prototypical/model config JSON

The problems I see are multifold:

There so far does not appear to have JSON ready-to-use CLI generation tool out there that suits our needs.
JSON based configuring/modelling library does not appear to be popular. So far all config layering are done using YAML/YML files.
Since YAML is apparently a superset of JSON, we can technically migrate our entire under the hood configuration system to one of the above WHILE maintaining superficial Python configuration (and essentially adding YAML file based configuration support)
IF we stick with JSON, We probably can only support first level SIMPLE key overwrite at CLI.  Since there are COMMON keywords across multiple first level key.

e.g. We cannot accept e.g. --applied false blindly. So that either means 1) we manually generate the parameter for use at CLI level and map those back to the config.json level which is not really scalable or feasible as the system become more and more complex. or 2) we move towards a more modal based system and HOPEFULLY (since none of the above auto validate CLI level parameters passed to it) we have a way to traverse all the keys and validate the key overwritten is correct.

Possible Path Going Forward

If we decide to Stick with JSON, probably DynaConf is the way for unified configuration.  We will code the parameter/proper check at CLI level if we want to offer configure eveyrthing at CLI  level. Some sort of scheme needs to be generated (e.g. --modified3dunet.applied true. JSON overwriting/layering is currently non-existent, if my understanding of the code base is correct.
If we decide to add the JSON to YAML transpiling layer and migrate the underlying system to YAML completely and gradually phase users off JSON towards YAML, it  can add some interesting benefits (such as more data type support, less double quote related issues. You can find more pro vs con here.
We can just add the GPU ID in the current arg.parser and kick the can down the road until we really hear user demanding CLI overwrite of parameters and add/adapt those as needed.

So in summary:

Most libraries (except DynaConf) uses YAML to do configuration layering/combination
So far, have not discovered any auto CLI parameters generation process that leverage JSON spec/config file.
We are currently manually managing at CLI/Script level what parameters gets overwritten. Entirely manually managed at PER cli command binary level. No centralization.
Choices are: Stay JSON, hoodwink YAML or only add optional parameters when people ask for it.


Tabbing into the date-field shows overlapping data in Firefox

Tested with version: 82.0.2 (64Bit - Version)
Maybe this bug is only present, if your browser is setup to use other   languages than English and therefore displaying dates with another format than YYYY-mm-dd.
The bug occurs only if you navigate to the date field with the TAB key.

@Sebi2020 check v4 demo at http://chemerisuk.github.io/better-dateinput-polyfill/. I haven't seen such an issue in Firefox.

Please re-read my issue description:

this bug is only present, if your browser is setup to use other languages than English and therefore displaying dates with another format than YYYY-mm-dd.

Here you can see a screenshot showing the issue:

@Sebi2020 reopen the link to demo. Your screenshot is for old version 3, try to use version 4 instead.


Improve Error message when launch fluent fails.

When launch_fluent fails, we should notify the launch path and launcher arguments with which Fluent was attempted to launch.

Sample output.


Not working on real device(Release mode)

Integrated this library for emitting events on an action and on other screen added some listeners to listen for the same events, which is working pretty fine as expected on simulator and device during debugging. But after release builds are generated for iOS and Android devices, its not working at all.
Anyone else has faced same issue?

I test in iOS with --configuration Release and work, how did you implement it?

Integrated this library for emitting events on an action and on other screen added some listeners to listen for the same events, which is working pretty fine as expected on simulator and device during debugging. But after release builds are generated for iOS and Android devices, its not working at all.
Anyone else has faced same issue?

facing same issue, working fine on debug devices and also when I tried the release variant. But when I tried on release build it's not working.

@abdulghanitech Because it won't work, if it's just a simple class in javascript


Rollup or Parcel for CodeMirror

From https://codemirror.net/examples/bundle/

CodeMirror is distributed as a collection of modules. These aren't directly loadable by the browser—though modern browsers can load EcmaScript modules, at the time of writing their mechanism for resolving further dependencies is still too primitive to load collections of NPM-distributed modules.

Problem #1 with using Parcel:
How do I serve the static wasm-file?

As well as the SQLite database

OK, let's try with Vite.
First problem:

By default, sql.js uses wasm, and thus needs to load a .wasm file in addition to the javascript library. You can find this file in ./node_modules/sql.js/dist/sql-wasm.wasm after installing sql.js from npm, and instruct your bundler to add it to your static assets
(https://sql.js.org/#/)

Let's see if I can do this.

https://github.com/jlongster/absurd-sql/issues/35#issuecomment-1193093632
We'll just assume that it always asks for "sql-wasm.wasm" and return its url:
import absurdSqlWasmUrl from "@jlongster/sql.js/dist/sql-wasm.wasm?url";

let SQL = await initSqlJs({
  locateFile: () => absurdSqlWasmUrl,
});


Enable cookies

Is it exist method to enable cookies? Same as
var axios= axios.defaults({jar: true})

No, it's not supported at the moment. Please see #48.


cli: make config available in index.html template

Hey, I just made a Pull Request!
This fixes the original ask in #6830, and more importantly removes the hardcoded set of config values that are being passed through, therefore removing some maintenance burden.
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

@backstage/techdocs-core please take a look


mautrix-signal: --prefix instead of --set PATH

Description of changes
Fixes adding ffmpeg to PATH not working as done in e.g. https://gitlab.com/coffeetables/nix-matrix-appservices/-/merge_requests/11/diffs?commit_id=3bc5c2e64eff2cbc67ec94a938616d549c42ce52
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)

@nh2 tested this on top of NixOS 22.05

22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

CC from recent changes: @NickCao @pimeys @Ma27 @fabaff @expipiplus1

@nh2 LGTM. Please resolve the conflicts and we're good to go :)

Done.


How to ignore read only properties?

I know [JsonIgnore], but I want to know is there any setting can do this.

You can create a custom contract resolver as shown in this answer by Brian Rogers to Is there a way to ignore get-only properties in Json.NET without using JsonIgnore attributes?.


Fix notification email issues

I think that this fix and close #3659 #3856 #3714 #3615
It works in better form than PR #3960 cause pick the FROM addess in case of you have some like FROM = "My Fancy Gogs"<EMAIL_ADDRESS>in cofig,

Thanks!


Release 2024-04-29

Release 2024-04-29
Please merge this Pull Request using 'Create a merge commit' button

Reviewed for changelog.

Reviewed for changelog.


URGENT: Add Bot Metrics to Mixpanel for Tracking

As a Marketing Team member, I want to add Paloambot tracking to mixpanel:

Active Bots (Bar chart)
a. Name of the Bot
b. Number of Bots for that Name

Total Bot TVL (Line chart in USD live bots)

Total Bot Volume (Line Chart of Total Deposits)

I've added mix tracking to all bots.. and pushed code to server tomorrow I will add the reports. @taariq


DNS Timeout for "xxxxx"

Hello,
I'm using the project with LACUS (latest version) and we got for some domains the error:
DNS Timeout for "XXXXXXXXX.com": The resolution lifetime expired after 3.202 seconds: Server Do53:<IP_ADDRESS>@53 answered The DNS operation timed out.; Server Do53:<IP_ADDRESS>@53 answered The DNS operation timed out.
It's working fine for other captures via lookyloo and many urls on monitor, but few of them give me this error 2 or 3 times per day.
Regards

I've seen that too, but it's the DNS resolution by the DNS server that is taking too much time. The best I can do it trying to trigger a retry when that happens, but there is not much else I can do as the resolution isn't made from withing the project.

Yep we can try this :-)


Covariant lists do not serialize property

public interface ILineItem {}

public class LineItem : ILineItem
{
   public string Id {get; set;}
}

public class Order
{
   public string Id { get; set; }
   public IEnumerable<ILineItem> Lines {get; set;}
}

serializing
new Order{
   Id = "order",
   Lines = new LineItem[]{
      new LineItem {Id="123"}
   }
}

results in the following (notice the lines element should have a data element with an array, not be an array)
{
  "data": {
    "type": "order",
    "id": "order",
    "relationships": {
      "lines": [
        {
          "data": {
            "id": "123",
            "type": "lineitem"
          }
        }
      ]
    }
  }
}

I'm facing the same issue when returning an abstract base class type. Is there an easy solution around this issue?

Easiest work around is to make your baseclass something with an “Id” field.
If that that doesn’t work let me know your specific case so I can use it as a starting point to try and handle this a bit better

is this still an issue today? i'm failing to reproduce this in my fork. Granted i upgraded Newtonsoft.Json to the latest version (13.0.2) and refactored the code to use JsonProperty.Converter instead of JsonProperty.MemberConverter.
So this issue might be because of how Newtonsoft.Json v10 or MemberConverter works.
When i test this in my fork, the lines property correctly serializes to an object,
holding a data array with one or relationship objects.
Last time i've checked the spec for JSON:API v1.1. this seems to be correct


Fix B_StagePositions keyerror

Pull Request instructions:

Please follow the update protocol
Answer the questions below in detail. Your responses will be emailed to experimenters.
If the experimenters must do anything new, provide detailed step by step instructions on the wiki
If computer maintainers need to manually update anything, provide detailed step by step instructions
Use markdown syntax in order for your comments to be rendered reliably in the email: "1." instead of "1)", use four spaces for indents.
If you use the keyword "skip email" in the title, it will skip the email updates
Merges from "develop" into "production_testing" should use the keyword "production merge" in the title for reliable indexing of updates
Merges from "production_testing" into "main" should use the keyword "update main"

Describe changes:

corrects logic to move stage to last position

What issues or discussions does this update address?

resolves #1091

Describe the expected change in behavior from the perspective of the experimenter

None

Describe any manual update steps for task computers

None

Was this update tested in 446/447?

[ ] 446/7

If it's not causing problem, this may be a worthless bug fix because eventually no jsons will have the B_Newscale_positions key. I'll test and see today


Chore: add typescript examples to test

Issue: N/A
What Changed
Added a couple more test cases as well as TODO comments for cases we should/could support
Checklist
Check the ones applicable to your change:

[ ] Ran yarn update-all
[x] Tests are updated
[ ] Documentation is updated

Change Type
Indicate the type of change your pull request is:

[x] maintenance
[ ] documentation
[ ] patch
[ ] minor
[ ] major

self merging @shilman


validate CoreEval submission in passed proposals against actual passed proposal

Some proposals pull their CoreEval submissions from the web. Some use a subdir on disk.
The subdir is good because it doesn't make CI dependent on network requests to wherever the CoreEval is stored.
But we need some way to validate that they match. It could be manual and also in CI as advisory.

Note, to pull from a mainnet RPC node itself, the following should work:
agd query gov proposal $PROPOSAL_NUM -o json | jq -r '.content.evals'

Yields the following shape:
[
  {
    "json_permits": "..."
    "js_code": "..."
  }
]


Deferred promise not resolving -  $auth.validateUser() in Ionic ui-router

Hi,
I use $auth.validateUser() in ui-router resolve. When I want to change the state after login most times promise returned by $auth.validateUser() does not resolve.
Situation occurs every time that this.dfd object is already created when validateUser() function is executed.
validateUser: function(opts) {
              var clientId, configName, expiry, location_parse, params, search, token, uid, url;
              if (opts == null) {
                opts = {};
              }
              configName = opts.config;
              console.log(this.dfd);
              if (this.dfd == null) {

Is it safe to set this.dfd to null at the start of validateUser function to workaround the problem?
BTW, there is no such problem with web version of the app.

I am also experiencing the same problem. Did setting this.dfd to null work for you? Are you using master branch or the 0.0.29 release?

Just a note, I'm using the web version and it is not resolving correctly.

Oh nevermind, it's working now. That was a slight misunderstanding on my part.

@mrsweaters Unfortunatelly setting this.dfd to null breaks other things... I've discovered that if one changes state after some time (using setTimeout function for example), the promise successfully resolves, so there must be some time of race condition. I hadn't more time to investigate further..

I've also encountered that bug:(

I figured out at least one reason why this occurs. My error was that I was simultaneously calling submitLogin in a controller method on an ng-click of the submit button AND had the submitLogin() in the ng-submit of the form definition. This caused the submit to occur twice in rapid succession. Apparently the ng-click gets handled first which means the promise handed back from submitLogin is from the ng-click. Since the deferred is a global within ng-token-auth the first one gets clobbered when the second one (the one from the ng-submit) occurs and only the second one is resolved.
Check your network log and see if you happen to be submitting two auth requests in rapid succession for some reason. That could be where the root of the problem for some people.


Welcome[Doc] typo & Contribution to Phase one [ in linux command section ]

@rishabkumar7 @madebygps
Bug :  On welcome page in Phases table [ optional should be " Optional" ]
Improvement : In phase 1 linux bash page want to add [ linux command mind map Link ]
Linux Mind Map
Screenshots

thank you for the issues. We are content at the moment with Linux content and resources. Thank you.


TNT Defuser in inventory whilst spectating

In @CaptainElliott's pull request, he gave spectators TNT defusers...
Here is a screenshot:

This is correct

Oh yes just noticed in the code, but why have this?

If you are OP you are able to defuse TNT in observers

@CaptainElliott Ohh I see! My mistake sorry! I'll close this issue.


Long running processes

Thoughts on supporting long running processes? Reason: I'd love to continue using onchange but I want to use it for a slightly different use-case. In this case, I need to combine it with an initial flag and if it's running on reload kill the current process and spawn a new one.
If it seems reasonable, I can definitely open a PR. I can also make it throttle changes with this implementation.

That seems more like a hot-reloading tool. That's not really what this is meant for.
Can you elaborate more on why you'd want to use this particular tool for something like that, versus something like recluster or herd?

I'm just after a simple CLI that will reload on change, I don't need a specific tool for clustering or hot code reloading. Right now I use nodemon, but the functionality is pretty much equivalent to what this library already does. Figured it'd be easier to kill two birds with one stone - make reloads while already running work and add initial run support, then my use-case is already working.


Sentiment Analysis (not the basic one)

Is your feature request related to a problem? Please describe.
Some of the most appealing causes of suicide are depression, anxiety, and feeling competitively lost.
Describe the solution you'd like
Building an advanced sentiment analysis model, will analyze the sentiment of the user and accordingly provide the level of chatbot. If a single chatbot is made that is available for all moods such as suicidal, depressing, or anxiety it would be hard. So I think the AI chatbots should be divided into a few parts that will handle the different moods. My analysis model will provide the most probable mood case for a user and accordingly, the chatbot will be assigned to the user.
Additional context

@TAHIR0110 @sanjay-kv  I would love to contribute in this issue and I can guarantee my contribution will be effective.

@Cralsic123 assigned!

Describe the feature
Data Collection:  Text reviews, social media posts etc.
Data Preprocessing: Remove noise, lowercase, normalize.
Data Labelling Mark each text as positive, negative, neutral (manual or crowdsourcing).
Building lexicon: Positive, negative, neutral words and relevant phrases.

Score sentiment: Add up the sentiment value of matched words/n-grams in each sentence.
Model Evaluate: Train a simple model (Naive Bayes etc.) and assess performance on unseen data.
Refine: Improve lexicon, n-grams based on evaluation and consider negation handling.
This method builds upon basic sentiment lexicons.

hey @TAHIR0110 can you assign me this?

pls assign me the issue , i have worked with the task model.


No access token has been provided. in CalendarController

I am following the tutorial and found that the "No access token has been provided." error from Laravel in the Calendar chapter.
The Calendar controller will get the access token and set it in the graph api.
        // Get the access token from the cache
        $tokenCache = new TokenCache();
        $accessToken = $tokenCache->getAccessToken();

        // Create a Graph client
        $graph = new Graph();
        $graph->setAccessToken($accessToken);

However, if the session never expired and never refresh the token, the session('refreshToken') valuw will be null and the function getAccessToken() returning the empty string to the CalendarController.
public function getAccessToken() {
  // Check if tokens exist
  if (empty(session('accessToken')) ||
      empty(session('refreshToken')) ||
      empty(session('tokenExpires'))) {
    return '';
  }

Is this a bug or I missed some steps?

Sorry to hear you're having trouble. Can you explain your repro scenario? I'm not sure I'm getting the full picture. You should always have a refresh token. Are you including offline_access in your OAUTH_SCOPES?

Sorry to hear you're having trouble. Can you explain your repro scenario? I'm not sure I'm getting the full picture. You should always have a refresh token. Are you including offline_access in your OAUTH_SCOPES?

Yes, that's the issue. Now I included offline_access in the scopes and it works now. Thanks.


Update cmdline examples to 2.1

@johnpryan if you have a spare minute or two, could you give this a sanity check?


Fix: updated KeyedVector attributes

Remove calls to the deprecated .wv attribute in the KeyedVectors module.
As a hotfix, simply create a dictionary keying vocabulary to word
vectors.

I'm running into this issue too so this merge would be very much appreciated.


Introduce a dynamic config for cassandra all consistency level delete

What changed?
Introduce a dynamic config for cassandra all consistency level delete

Why?
feature flag

How did you test it?

Potential risks

Release notes

Documentation Changes

Pull Request Test Coverage Report for Build 01837a95-0028-40bd-9ed7-ed7aac5bd5c7

48 of 66 (72.73%)  changed or added relevant lines in 18 files are covered.
105 unchanged lines in 21 files lost coverage.
Overall coverage decreased (-0.03%) to 57.236%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

common/persistence/nosql/nosqlplugin/mongodb/plugin.go
0
2
0.0%

common/persistence/persistence-tests/persistenceTestBase.go
0
4
0.0%

common/persistence/nosql/nosqlplugin/cassandra/db.go
8
20
40.0%

Files with Coverage Reduction
New Missed Lines
%

common/types/shared.go
1
41.66%

client/history/client.go
2
38.1%

client/history/metricClient.go
2
45.3%

common/membership/hashring.go
2
83.54%

common/util.go
2
52.74%

service/history/task/transfer_active_task_executor.go
2
72.15%

service/matching/taskListManager.go
2
77.11%

common/persistence/nosql/nosqlplugin/cassandra/workflow.go
3
59.55%

common/persistence/statsComputer.go
3
94.64%

service/history/task/fetcher.go
3
91.75%

Totals

Change from base Build 01837a94-cac1-4f41-a994-e1357753d5a6:
-0.03%

Covered Lines:
85080

Relevant Lines:
148648

💛  - Coveralls


Use @boxyhq/metrics wrapper package for telemetry

What does this PR do?

Uses https://github.com/boxyhq/metrics to collect telemetry.
Type of change

[x] Updated dependencies
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How should this be tested?

[x] Existing unit tests

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code and corrected any misspellings
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

LGTM. Should publish a release version of the metrics package before merging this in

v0.1.0 of metrics is out. Updated the libs here. Tested with lightstep and works.


2.7.570 breaks builds for React apps built with Create-React-App (CRA)

I am using React PDF Viewer, which is based on pdf.js at its core, within a React app created with CRA. Which basically means, that there is Webpack working under the hood, abstracted away through CRA.
Now everything works fine up to 2.6.347, but starting with 2.7.570 and also with 2.8.335 I am running into this error:
./node_modules/pdfjs-dist/build/pdf.js 3657:145
Module parse failed: Unexpected token (3657:145)
File was processed with these loaders:
 * ./node_modules/react-scripts/node_modules/babel-loader/lib/index.js
You may need an additional loader to handle the result of these loaders.
|               numPages: this._numPages,
|               annotationStorage: (annotationStorage === null || annotationStorage === void 0 ? void 0 : annotationStorage.getAll()) || null,
>               filename: ((_this$_fullReader = this._fullReader) === null || _this$_fullReader === void 0 ? void 0 : _this$_fullReader.filename) ?? null
|             }).finally(() => {
|               if (annotationStorage) {

I don't expect anyone here to provide support for either the react package nor CRA. However, as I could clearly identify 2.7.570 as the "deal breaker", I suspect there has been some change that makes a re-configuration of Webpack necessary?
Glad to receive any feedback that could help to shed some light on the root cause :-)
Thank you

I am using React PDF Viewer,

We know nothing about that project here, since it's not affiliated with the PDF.js library in any way; hence please direct any support questions regard the "React PDF Viewer"-project directly to them instead.

However, as I could clearly identify 2.7.570 as the "deal breaker",

Most likely, https://github.com/mozilla/pdf.js/issues/13190#issuecomment-814701619 is relevant here as well.

Slightly off-topic, but: In my opinion, it's reasonable to expect that the "React PDF Viewer"-project should handle their own user support, considering that they're charging users money for their software.
The PDF.js library, in contrast, is completely free and thus it seems somewhat unfair to expect the PDF.js contributors to provide support (for free) to a commercial project.

We know nothing about that project here, since it's not affiliated with the PDF.js library in any way; hence please direct any support questions regarding the "React PDF Viewer"-project directly to them instead.

->

I don't expect anyone here to provide support for either the react package nor CRA.

"Unexpected token" probably refers to the use of the nullish coalescing operator (see https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Nullish_coalescing_operator) used in that line that your environment does not support. Either add a plugin for that or use the legacy version of PDF.js if you need to be able to run in a browser that doesn't support that.

Just to add another observation,  we are seeing a similar error when updating to 2.7.570 using plain pdf.js in a Vue project while building with Webpack:

It looks like in this case it is related to Optional Chaining. I tried to get it working again by adding pdfjs to transpiled dependencies (i. e. in vue.config.js via transpileDependencies), but no luck so far.
I'm still trying to understand why this is happening, since we are building in Node v14 that should support optional chaining and our browserslist should only have browsers that support it as well 🤷 .

@dominik-bln See https://github.com/react-pdf-viewer/react-pdf-viewer/issues/499#issuecomment-819568264

@aspnetde Ah, thank you, that clears it up. Another thing to add to the list that will only work with Webpack 5.


Error: Could not setup bluez OBEX agent

Hi I just installed the binarie on a Arch linux set up and I received the error "Error: Could not setup bluez OBEX agent
" with no other output I am running a really small set up using awesomewm tiling window manager so this might due to dependecies but I am having issues finding whihc one should I install.

@helcone, first of all, check if your bluetooth service is up and running.
This error usually pops up when another bluetooth manager or OBEX manager is running (like blueman or obexftp), make sure these managers are properly closed before launching bluetuith. For blueman, also make sure that its systray icon isn't visible, if it is, click the systray icon, and then click on "Exit".
If you have followed these steps and this issue still persists, I'll look into it.

O yes you were right sorry for the confusion either way I am going to leave this closed as for documentation for someone who stumble into it, keep the great work is a great app thanks

Hi.
Closing blueman didn't work (I ensured the process isn't running) :(
bluetuith v0.0.6 (instaled via go install)
bluez 5.64-2
dbus-x11 1.14.0-1
bluetooth.service - Bluetooth service
     Loaded: loaded (/usr/lib/systemd/system/bluetooth.service; enabled; preset: disabled)
     Active: active (running) since Sun 2022-08-07 23:06:52 CDT; 58min ago
       Docs: man:bluetoothd(8)
   Main PID: 334393 (bluetoothd)
     Status: "Running"

5.15.57-2-MANJARO

@luistoledo

@luistoledo is blueman-applet running? Can you try uninstalling blueman too and see if it works? The blueman-mechanism DBus session may be running, so blueman's OBEX agent may still be registered.
I cannot replicate the issue here. I started blueman, then closed it via the exit option in the blueman systray icon.

thanks for the fast response.
The applet wasn't running on last test.
I found the culprit, it was blueberry! after killing the daemon, then it worked perfectly.
(almost forgot i have it installed, but your last response guide what to search.)
thanks again

Hello, I have followed the steps as suggested previously in this thread. I am on a fresh install of arch, i can connect to devices but not send and receive files. I get the errors:
Warning: Could not setup bluez OBEX agent
Warning: Send/receive files is disabled
I do not have blueman, or obex FTP installed... any help?

Do you have the OBEX daemon(obexd) installed?

Thanks for the response! It is installed but was not running. Once obexd was started the errors went away, but still no option to send files in the menus

The send files button is there now, i am able to recieve files, but sending i get "unable to find service record"

A google search tells me that issue is with the receiving device...it is working. Thank you!


Prevent NullPointerException in ReversedLinesFileReader constructor

In many parts of the Commons IO API, null may be passed as a Charset or the name of one, and Commons IO uses the platform's default character encoding.
I assumed that was the case for ReversedLinesFileReader, and the result was a null pointer exception.
Consider the following minimal example:
import java.io.File;
import java.io.IOException;
import java.nio.charset.Charset;
import org.apache.commons.io.input.ReversedLinesFileReader;

public class TestReversedLinesFileReader {
  public static void main(String[] args) throws IOException {
    ReversedLinesFileReader rfr =
        new ReversedLinesFileReader(new File("TestReversedLinesFileReader.java"),
                                    (Charset) null);
  }
}

Running this program results in:
Exception in thread "main" java.lang.NullPointerException
	at java.lang.String.getBytes(String.java:940)
	at org.apache.commons.io.input.ReversedLinesFileReader.<init>(ReversedLinesFileReader.java:130)
	at org.apache.commons.io.input.ReversedLinesFileReader.<init>(ReversedLinesFileReader.java:78)
	at TestReversedLinesFileReader.main(TestReversedLinesFileReader.java:8)

I assume that the ReversedLinesFileReader constructor is intended to accept null as an argument because of this line:
        final Charset charset = Charsets.toCharset(encoding);

which calls toCharset(Charset) which is a no-op unless its argument is null.  So there would be no purpose to that line except to handle a null encoding argument to the constructor.
The problem comes a few lines later when the possibly-null value encoding is used:
This pull request changes the code so the encoding field is non-null even if the encoding formal parameter is null, and uses this.encoding instead of encoding in 4 locations.
I think that setting the field encoding to a non-null value is the right thing because there are three calls to new String that use encoding as if it is non-null.
An alternate fix would be to:

document the Charset argument as being non-null,
throw an exception within the constructor if a client passes null,
remove the declaration of the charset local variable, and
chang all uses of the charset local variable into uses of encoding.

Coverage remained the same at 89.783% when pulling 90db1a2a1c50a14cb0754e9cc268270f0d780e2e on mernst:ReversedLinesFileReader-encoding into cd154e91439b1e8ba75d9265b643177f7709ddef on apache:master.

@mernst
Thank you for the PR. You are missing a unit test to show what issue this solves, IOW the test fails without the change on the main folder and passes with with change.

@garydgregory I added the test case from the pull request description.  It fails on master and passes in this pull request.

@garydgregory Can this pull request be merged?

I will review tomorrow.

@mernst
I implemented this differently in git master now. You still get credited in changes.xml ;-) Please verify and close.
TY for your patience.

@garydgregory Thanks for the fix!  I appreciate it.  Credit is secondary.


doc(kic) add 2.3.x

Summary
Add KIC 2.3
Reason
https://github.com/Kong/kubernetes-ingress-controller/releases/tag/v2.3.0
Testing
Pending netlify to confirm no table breakage in reference

@scseanchow I've omitted the UDPRoute docs from this although 2.3 does make it available, with the rationale that it'd make more sense to add that as part of whichever release completes GWAPI milestone 2, along with product marketing material. It furthermore makes sense to write a combined guide for the L4 Routes and TLSRoute once they're all ready, as they're fairly similar.
As such the only changes from 2.2.x in this doc are the compatibility tables and a small update on an HTTPRoute feature that we do now support:
diff --git a/app/kubernetes-ingress-controller/2.3.x/guides/using-gateway-api.md b/app/kubernetes-ingress-controller/2.3.x/guides/using-gateway-api.md
index 96f698de18..0dbb8ae76e 100644
--- a/app/kubernetes-ingress-controller/2.3.x/guides/using-gateway-api.md
+++ b/app/kubernetes-ingress-controller/2.3.x/guides/using-gateway-api.md
@@ -205,4 +205,4 @@ Gateway APIs are supported. In particular:
   are not yet implemented.
 - HTTPRoute does not yet support multiple backendRefs. You cannot distribute
   requests across multiple Services.
-- queryParam matches and RegularExpression path matches are not yet supported.
+- queryParam matches matches are not supported.

Let me know if we do want to instead go ahead and provide a UDPRoute guide now.


FTBRanks Intergration

Description
I'd like to see further integration with FTB Ranks.
Currently there are several migration settings that allow importing from other permission mods directly into luckperms.
Proposed Behaviour
Check the mods folder to see if ftbranks is present.
Then Check the world/ranks folder for an existing ranks.snbt and players.snbt and port them into luckperms.
Able to configure this to be done as part of the watch files function to automatically update any changes with luck perms controlling the majority of the permissions. FTB ranks can handle any timed rankups.
Extra Details
No response

I would accept a pull request for the auto migration bit :)


frontend: Allow to use JSON for defining K8s resources

Besides allowing users to set code in JSON, these changes also set the
error bit much more accurately and often as a result of having to parse
the code anyway.
fixes #713
How to test:

[ ] Click the + button, add a YAML resource and apply it: it should work as usual;
[ ] Replace the contents by JSON and apply it: it should work as YAML did
[ ] Change pieces of both language versions in the editor to make it not parse correctly: according to whether it looks like YAML or JSON, there should be an error displayed about it and at most it changes every half a second

@illume , this is failing with the heap allocation. Should we increase the max-old-space setting? Do you have a recommendation for this?

Before I could apply yaml like this: https://github.com/kinvolk/headlamp/blob/main/kubernetes-headlamp.yaml now it doesn't work, and shows the error: "expected a single document in the stream, but found more"

Works as expected for a single JSON but failed for multiple JSON documents like @illume mentioned.

Before I could apply yaml like this: https://github.com/kinvolk/headlamp/blob/main/kubernetes-headlamp.yaml now it doesn't work, and shows the error: "expected a single document in the stream, but found more"
Note, kubectl works with multiple documents in JSON as well. Like this kubernetes-headlamp.json
ED: sorry, before the code didn't work either. An error is only shown after pressing apply. So it's not a regression.

Right. We have this other issue for the multiple streams: #712


Translated Headers

[ x] I have read freeCodeCamp's contribution guidelines.
[x ] My pull request has a descriptive title (not a vague title like Update index.md)
[ x] My pull request targets the master branch of freeCodeCamp.
[ x] None of my changes are plagiarized from another source without proper attribution.
[ x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Sorry, this is incorrect. The ids need to be exactly what they are for FCC to be able to work with these sections properly.


Add RequestConfig builder javadocs

I linked each builder method to its corresponding getter in the built object. However, the builder methods are the primary interface for users, so I think it should hold the primary javadocs instead.

@rhernandez35 Thank you for contributing this PR. It would really cool if you could make similar changes to the config classes in HttpCore.

@ok2c Do you agree with his suggestion that the Javadoc should be moved to the builder methods? I like that idea.

@rschmitt I am fine with it as long as the same change get applied consistently to all similar classes in core and client.


potential depth value error using dmtet

I found the rendered depth image using dmtet doesn't shows depth value that changes gradually. When render using dmtet, the code directly use the rasterized z/w value in clip-space (range from -1 to 1) as depth value, which is not the real depth value in camera coordinate system:

https://github.com/ashawkey/stable-dreamfusion/blob/main/nerf/renderer.py#L942

It seems in order to get the original depth value, an inverse projection is needed. Here is the code that I tried to converted it back to the real depth value:
def depth_clip2real(depth,clip_near,clip_far):
    ''' convert depth from OpenGL CLIP Space to real depth
    input:
        depth (torch.tensor): [height x width] depth from OpenGL CLIP Space
        clip_near (float): clip near distance of camera frustrum
        clip_far (float): clip far distance of camera frustrum
    output:
        depth(torch.tensor): [height x width] depth from real space
    '''
    scale,bias = -(clip_far+clip_near)/(clip_far-clip_near), -(2*clip_far*clip_near)/(clip_far-clip_near)
    depth = bias/(depth+scale)
    return depth

Maybe it would help in providing better depth supervision during training.

Yes you are right, it is more complicated to make it diffentiable.

@Sheldonmao Hi, you are right! It seems the depth from rast is not differentiable (NVlabs/nvdiffrast#114), and I have disabled depth supervision at dmtet stage currently.


Error "ModuleNotFoundError: No Module named deep_translator", code does not execute

deep_translator version: 1.6.1
Python version: 3.10.1
Operating System: Windows 10

Description
I was trying to perform the test translation as it appears in the first example that is in the documentation and when I run the code, it appears like this:
"from deep_translator import (GoogleTranslator,
ModuleNotFoundError: No module named 'deep_translator'"
I don't know if I installed something wrong, but in principle it should work.
What I Did

When I try to run "python -m pip install deep-translator", it gives me this:

Hey @RodrigoSeger , It seems to be working fine on my system :)
deep_translator  : 1.6.1
Python version : 3.9.7
OS : Windows 10
pip version : 21.3.1
Installation step  : pip install -U deep-translator

@RodrigoSeger Hi, apparently you didn't install the package correclty or maybe you are in a different python virtual environment. The best practice is to create a python3 virtual env and run pip install deep-translator there. This way you can make sure that the package has been installed correctly.


Doesn't work in Node (server side)

This uses native fetch, which only works in a browser due to the window object. Look into conditionally using node-fetch for server side.

Two reasons this won't work:

Node doesn't have fetch because it is on the window object
Node doesn't handle new DOMParser()


Update dada2 tools with fixes

My first time on this, so please let me know if I did something wrong here.  Thanks!  ;)

@galaxybot test this

@galaxybot test this

@galaxybot deploy this

Thanks so much for all of the help @martenson :)


Getting error "sqlserverport.sqlserverport.BrowserError: No response from SQL Browser service"

Hi @gordthompson ,
We used sqlserverport in a python based cron job deployed in AKS. Trying to connect to named SQL server instance with port 49002.
It did work in one environment but when moving to other environment when we are trying same it gave below error in the logs
sqlserverport.sqlserverport.BrowserError: No response from SQL Browser service.Verify that the service is available on <> and <> is a valid instance name on it.
We tried servername as below but none of them are working
1.fullyqualifieddns,portnumber
2.only servername without suffix of .com
3. Ip address of server (xxx.xxx.xxx.xxx)
Could you please help us here?

Trying to connect to named SQL server instance with port 49002.

Do you mean that the SQL Server instance is listening on port 49002? If so, then you don't need to use sqlserverport because you already know what port number to use.

@gordthompson  - We are using sqlserver port because its a Named SQL server instance that Linux based AKS cronjob trying to connect

So what does "port 49002" refer to?

@gordthompson -For egress connection from AKS to on-prem SQL instances, we use 1433,49001,49002 etc ports.
Below is the splunk log which we are seeing.

@gordthompson  - Is it getting connection timedout due to max size of bytes i.e 1024 as we can see in splunk log , it has reference to sqlserverport.py,line 61
response = sock.recv(1024)  # max 1024 bytes for CLNT_UCAST_INST

So a process somewhere in the cloud is trying to connect from its outbound port 49002 to the SQL Browser service (UDP port 1434) on one of your on-prem servers. Is that correct?

You could try increasing the value here
https://github.com/gordthompson/sqlserverport/blob/c7829dfa3e1327353b3b4f6078b0bc2a00166ca7/sqlserverport/sqlserverport.py#L56

@gordthompson - i have imported sqlserverport in other application and referring as per example.
How can I set "sock.settimeout(5)" in such case?

If you installed using pip install sqlserverport then the sqlserverport.py file will be under site-packages folder for your Python environment. For my virtual environment, it is
/home/gord/.virtualenvs/venv/lib/python3.12/site-packages/sqlserverport/sqlserverport.py

@gordthompson - would it be possible for you to update  sock.settimeout(5)  to say 20 seconds and publish it?

Fork this repo. In the master branch of your fork, update line 56 in sqlserverport.py with your new timeout value. Then, instead of
pip install sqlserverport

use
pip install git+https://github.com/prate2012/sqlserverport.git

Were you able to change the timeout value, and if so, did it resolve your issue?

No but we did allow 1434 port which fixed the issue


Initialize seqno for setData(vector &) and fix typo

My controller doesn't like random seqnos and I like passing in the data vector directly for better performance.

@hiroyuki, please review. I can't seem to add assignees or reviewers to the PR.


JavaScript

Javascript
User Story
As a boot camp student
I want the prework notes to be structured on a webpage
So that I can easily find and read the information
Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I view the study guide
THEN I can see the four topics I learned listed along with a suggestion on what I should study first

Creatued JS Script file that randomly selects study topic within console.
Established List Topics and Select Topic Functions


How to get page orientation?

Hello,
I'm a bit confused about the proper way to get the page orientation with tesseract. I have tried the following:
using (var pi = page.AnalyseLayout())
{
    pi.Begin();
    return pi.GetProperties().Orientation;
}

But it always returns PageUp.
I have also tried page.DetectBestOrientation(out int orientation, out _) but this returns 270 even for a page that is supposed to be PageUp.

@charlesw Can you help please?

@charlesw: I tried the methods in the samples before but none of them worked for me. Maybe I can send you a sample file?

@tudorserbanuipath You need to have "osd.traineddata" file in the tessdata folder and when initializing the engine set language to "osd" - this worked for me to detect orientation e.g.
using var engine = new<EMAIL_ADDRESS>"osd", EngineMode.Default);

@tudorserbanuipath You managed to get it work? I'm facing the very same issue.
@Olby2000 when i added that traineddata file the orientation stayed the same, but the confidence dropped from 250 to <10. Could you point out difference in your implementation to my code snippet:
            using (var engine = new<EMAIL_ADDRESS>"osd", EngineMode.Default))
            {
                using (var img = Pix.LoadFromFile([JPG File]))
                {
                    using (var page = engine.Process(img))
                    {
                        using (var pi = page.AnalyseLayout())
                        {
                            pi.Begin();
                            var ori =  pi.GetProperties().Orientation; //Always PageUp
                        }
                        page.DetectBestOrientation(out int orientation, out float confidence); //orientation always 270, cofidence now <10
                    }
                }
            }


[misc] Code cleanup

Removed unused components import
Fixed doubled properties declarations
Fixed typos
Refactored static variables declarations
Removed unreachable code
Changed tabs to spaces

@veronikaslc please also rebase and resolve conflicts

rebased on the latest dev


Refactor has helper into useQuery or createHooks

has is a very useful helper and should be integrated into the existing Evolu API.
All row columns except for id are nullable by default. A column can be null for two reasons. It's not loaded yet, or it's obsolete. That's why the has helper is so useful, and that's why it belongs to the application itself.
I'm not sure whether it can be refactored into the createHooks, but I'm pretty sure it can be refactored into useQuery. And with memorization, of course.
Some nullable columns are nullable for a reason, while others are just not yet loaded. The best place is createHooks, I believe.

In the past, Evolu had something similar implemented via SQLite views. It worked like that:

CreateHook created a table [tableName]_all and views [tableName] and [tableName]_deleted.
The view [tableName] filtered deleted rows and rows with required columns containing nulls (not yet synced).
Views were updated on start, just like tables.
useQuery provided only views: [tableName] and [tableName]_deleted with correct nonnullable types.
isDeleted column didn't exist. To mark a deleted row, a special invisible column was used.
useMutation returned a function to delete rows.

export const { useQuery, useMutation } = createHooks({
  node: {
    id: NodeId,
    md: NodeMarkdown,
    foo: optional(model.SqliteBoolean)
  },
});

The problem with that design was that all columns added after a release had to be nullable/optional. Otherwise, a user wouldn't see the old data. has helper has the same problem.
As I see it, the API should remain explicit that everything is nullable because of syncing and because a schema can and will change. We really can't distinguish between syncing and old data with obsolete columns. But we can and should add transformations for handling both cases.
Btw, SQLite View is suitable for deleted rows; check https://github.com/evoluhq/evolu/issues/74
The question is, can we describe such transformations declaratively? If so, we can implement them as SQLite views. Note that naive null filtering is not enough because columns can be merged, transformed, or whatever when a schema is evolving.
Therefore I suggest defining such transformations out of SQLite, like has helper does it, and after a while, reconsider moving them into SQLite (for better performance).

Done.


update_VM_to_full_CIL.sh fails for 2 reasons

error during building
[ 54%] Performing configure step for 'astra-toolbox'
./autogen.sh: line 3: aclocal: command not found
Error running aclocal

Note that it keeps on going and clones CIL-demos anyway, but that has another error
 55%] Completed 'TomoPhantom'
[ 55%] Built target TomoPhantom
Makefile:114: recipe for target 'all' failed
make: *** [all] Error 2
Cloning into 'CIL-Demos'...
remote: Enumerating objects: 323, done.
remote: Counting objects: 100% (11/11), done.
remote: Compressing objects: 100% (7/7), done.
remote: Total 323 (delta 4), reused 4 (delta 4), pack-reused 312
Receiving objects: 100% (323/323), 46.23 MiB | 2.11 MiB/s, done.
Resolving deltas: 100% (134/134), done.
/usr/bin/python: No module named pip
All done

So the install is calling the wrong python.

Needs
sudo apt install autotools-dev automake

as part of the script, and a python3 at the end. Best to remove the --user option as well from the pip. Apparently no longer needed/recommended with python3 (says @casperdcl ).

Installing the packages after the failed run gives another error
** Please tell me who you are.

Run

  git config --global user.email<EMAIL_ADDRESS>  git config --global user.name "Your Name"

to set your account's default identity.
Omit --global to set the identity only in this repository.

fatal: empty ident name (for<EMAIL_ADDRESS>not allowed
Cannot save the current index state
CMake Error at /home/sirfuser/devel/buildVM/builds/astra-toolbox/tmp/astra-toolbox-gitupdate.cmake:80 (message):
  Failed to stash changes

Checking
$ cd ~/devel/buildVM/sources/astra-toolbox/
$ git status
HEAD detached at origin/master
Untracked files:
  (use "git add <file>..." to include in what will be committed)

	python_build
	python_install

Is this an astra bug?

Manually removing astra-toolbox from both sources and builds gives
[ 55%] Performing configure step for 'astra-toolbox'
configure.ac:42: error: possibly undefined macro: AC_PROG_LIBTOOL
      If this token and others are legitimate, please use m4_pattern_allow.
      See the Autoconf documentation.
Error running autoconf

$ cd ~/devel/buildVM/sources/astra-toolbox/
$ git status
HEAD detached at origin/master
Untracked files:
  (use "git add <file>..." to include in what will be committed)

	python_build
	python_install

Is this an astra bug? (i.e. should these be in their .gitignore?)

This is my fault. I create these files in order to be able to build astra.
I see 2 options:

add them to .gitignore
remove them in the SuperBuild

I am afraid we will not be able to do 2 if the configure/build/install steps fail? 1 seems more robust.

will be cosed by https://github.com/SyneRBI/SIRF-SuperBuild/pull/600


KeyVault resource-manager removing secrets.json from readme.md

This removes secrets.json from the readme.md of the resource-manager folder for Key Vault. The APIs in secrets.json are for ARM API documentation only and are not intended to be part of any generated SDK or command line tooling for the key vault management plane.
Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

/azp run automation - sdk

Can one of the admins verify this patch?


Introduce KafkaRecordEntity to support Kafka headers in InputFormats

Today Kafka message support in streaming indexing tasks is limited to
message values, and does not provide a way to expose Kafka headers,
timestamps, or keys, which may be of interest to more specialized
Druid input formats. For instance, Kafka headers may be used to indicate
payload format/encoding or additional metadata, and timestamps are often
omitted from values in Kafka streams applications, since they are
included in the record.
This change proposes to introduce KafkaRecordEntity as InputEntity,
which would give input formats full access to the underlying Kafka record,
including headers, key, timestamps. It would also open access to low-level
information such as topic, partition, offset if needed.
KafkaEntity is a subclass of ByteEntity for backwards compatibility with
existing input formats, and to avoid introducing unnecessary complexity
for Kinesis indexing tasks.

looks like tests are failing due to insufficient diff coverage for some lines. Many of them appear to be where we introduced additional type parameters. @gianm what's the guidance here?

looks like tests are failing due to insufficient diff coverage for some lines. Many of them appear to be where we introduced additional type parameters. @gianm what's the guidance here?

looks like tests are failing due to insufficient diff coverage for some lines. Many of them appear to be where we introduced additional type parameters. @gianm what's the guidance here?

What I've done is either,

Add tests if it's something that really should be tested
Add a suppression if it's something that really shouldn't be tested
Ignore the coverage check if it's something that could be tested, but it isn't that important, and we want to skip it for now. (It's a diff-oriented check, so ignoring it won't affect future patches.)

looks like tests are failing due to insufficient diff coverage for some lines. Many of them appear to be where we introduced additional type parameters. @gianm what's the guidance here?

What I've done is either,

Add tests if it's something that really should be tested
Add a suppression if it's something that really shouldn't be tested
Ignore the coverage check if it's something that could be tested, but it isn't that important, and we want to skip it for now. (It's a diff-oriented check, so ignoring it won't affect future patches.)

ok I will merge it as is then. Adding specific tests for the trace logging seems a little overkill. We can add it later if we feel strongly about it.

ok I will merge it as is then. Adding specific tests for the trace logging seems a little overkill. We can add it later if we feel strongly about it.


Indenting multiple lines at once

It would work like this:

Select block of multiple lines
Press Ctrl+Tab
This will be useful when you want to intend multiple lines to increase readability of your code (e.g. if you were concerned with character limit and as such you've avoided unnecessary white space, but at the end you had plenty of characters left so you've decided to format your code properly)

Nice idea, but your spelling of 'indent' is bothering me. No offense, I just have OCD. But yes, this would be a great feature.

it works, but with TAB/Shift+TAB


Assets moved to different folder doesn't trigger cache clearing

When an existing asset in a folder is moved to another folder the cache clearing isn't called. This means Imager will search to an old path and can't find the asset which will result in an Exception.
Is there a fix for this?

Which cache are you referring to? The template cache?

@aelvan I'm still debugging this issue a little bit further. I will close this issue and get back to you when i found the issue if there is one for imager.


[Feature] Add Helm Chart Checker in CI

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
This is requested from @chenk008 in https://github.com/ray-project/kuberay/pull/175#issuecomment-1061655294.
We maintain both the kustomize and helm manifests. Currently, we do use kustomize to deploy the control plane and validate cluster creations etc in CI. However, we lack of tests to validate helm chart changes. This can be improved by adding helm chart checker in CI.
/cc @ebr @chenk
Use case
Validate helm manifest changes make sure it's working
Related issues
https://github.com/ray-project/kuberay/pull/175
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

It's partially covered by https://github.com/ray-project/kuberay/pull/554.
What remains to be done is to add KinD CI integration tests that validate that the charts work end-to-end.

What remains to be done is to add KinD CI integration tests that validate that the charts work end-to-end.

A basic e2e sanity check that we can use Helm to deploy a 1-head and 1-worker Ray cluster with the latest Ray version would be sufficient.

We can use existing compatibility test code as a starting point for this.

@kevin85421 for the KubeRay 0.4.0 release we should add a e2e CI test that validates that at least the default values.yaml works

Again, not a hard blocker for the 0.4.0 release, but we do need to at least do some manual testing.


Echo effect not playing

I am able to see messages sent and recieved with the Echo effect, but it is not possible to play the animation. I am able to play all of the other effect animations. I observed this on the  android client, but I will test it on windows and web browser when I have a chance.

We don't support the echo animation at this time - it's incredibly difficult to pull off

On a related subject - is this the reason why I cannot tap the invisible ink message to view it, or is there another way besides copying it to read the contents?

you have to swipe on the message to remove the effect, similar to on an apple device

Okay, I can confirm that invisible ink works.
If the Echo effect isn't ever planned to be added, it would be nice to see some text above the send with Echo icon informing that the animation won't be visible for yourself. Similarly  if you press the replay button, it should show an alert dialog explaining the same thing. I'm not a UI designer, so I don't know if that would be the best place to put it, but some message would be nice.

done

Why won't the echo effect ever be added?


Fixing BackupHistory

Fixes #1196 and #1143
I think I've modified the type definition right, and stuff looks 'righter' at my end. Fred need to confirm though!
And while I'm going this, whats the management view on the different return names? Easy enough to standardise, just which is the preferred one:
Raw:
Name
BackupSetID
checkpoint_lsn
Database
database_backup_lsn
DeviceType
Duration
End
first_lsn
FullName
last_lsn
MediaSetId
Path
position
Server
Software
software_major_version
Start
TotalSize
Type
Username
object:
Name
BackupSetId
CheckpointLsn
ComputerName
Database
DatabaseBackupLsn
DeviceType
Duration
End
FileList
FirstLsn
FullName
InstanceName
LastLsn
Path
Position
Software
SoftwareVersionMajor
SqlInstance
Start
TotalSize
Type
UserName

Looking good, but - as discussed - superseded by https://github.com/sqlcollaborative/dbatools/pull/1200


ObjectPropertyNaming defaults do not match Kotlin style guide

Expected Behavior
I expect code that follows the Kotlin style guide to not get flagged by detekt.
Observed Behavior
Code like this:
    companion object {

        private val DEVICE_NAMESPACE_INDEX = ushort(1)

        ...

is being flagged by the ObjectPropertyNaming rule even though the style guide encourages this naming convention for immutable data: https://kotlinlang.org/docs/reference/coding-conventions.html#property-names.
Your Environment

Version of detekt used: 1.0.0-RC10

Hmm. detekt flags these because they are private. The style guide makes no explicit distinction either way, it just says "top-level or object val properties with no custom get function".
Not sure what the intention of the style guide is here. In Java it's not strange to make the constants uppercase whether it's private static final or public static final.

This is a regression. I belive. Didn't happen before RC10.

Commit 17ea132 changed this behavior

Seems to be a merge problem. I'll change it back and add a test


"Photo Ready" option in Umple Online will cut off the diagram if the diagram is scrollable

Possible Solution: 1. Consider providing a downloading .png diagram option
2. Give a link that directs to a new tab to show the full image.

I think the solution is simply be to not turn off the scrollbars. This feature was put in so people could have a nice view (in addition to the view graphiviz view, and mostly for small models. Generating a png I think would be hard to do from arbitrary DOM content. Also people can still hide the text (control-t) and even the menu if they want to make the image fill the screen, so making a separate tab would not have much benefit.

When the diagram grows relatively large, display the full diagram is almost impossible without scrolling because users can't change the scale of each component. Currently, the only way to get a nice full image of the diagram is to generate the GraphViz diagram, but that option is hard to find. So I think adding a link that directs to .svg file at a more visible position would be great. And also change the "Photo Ready" function to be scrollable.

Fixed by issue #1557


Connection fails via https when certificate contains IPv6 address in SAN

The URL is https://[2001:db8::17]/, the server is running with a certificate that includes both DNS:2001:db8::17 and IP:2001:db8::17 as alt_names. Accessing that URL with e.g. curl works just fine, when using urllib3 I am seeing this error instead:
A problem was encountered with the certificate that prevented urllib3 from finding the SubjectAlternativeName field. This can affect certificate validation. The error was Codepoint U+003A at position 5 of u'2001:db8::17' not allowed

Can you please try installing the ipaddress module to see if that resolves your issue?

That module is installed. Actually this seems to be an issue with the cryptography module that can be avoided by omitting the DNS:2001:db8::17 part and is fixed in master there otherwise.

Reopening since even after fixing https://github.com/pyca/cryptography/issues/3943 I'm still seeing another traceback within urllib3 now:
  File "/usr/local/lib/python2.7/dist-packages/urllib3/connection.py", line 337, in connect
    cert = self.sock.getpeercert()
  File "/usr/local/lib/python2.7/dist-packages/urllib3/contrib/pyopenssl.py", line 348, in getpeercert
    'subjectAltName': get_subj_alt_name(x509)
  File "/usr/local/lib/python2.7/dist-packages/urllib3/contrib/pyopenssl.py", line 222, in get_subj_alt_name
    for name in ext.get_values_for_type(x509.DNSName)
  File "/usr/local/lib/python2.7/dist-packages/urllib3/contrib/pyopenssl.py", line 175, in _dnsname_to_stdlib
    name = idna_encode(name)
  File "/usr/local/lib/python2.7/dist-packages/urllib3/contrib/pyopenssl.py", line 173, in idna_encode
    return idna.encode(name)
  File "/usr/local/lib/python2.7/dist-packages/idna/core.py", line 355, in encode
    result.append(alabel(label))
  File "/usr/local/lib/python2.7/dist-packages/idna/core.py", line 265, in alabel
    raise IDNAError('The label {0} is not a valid A-label'.format(label))
IDNAError: The label 2001:db8::17 is not a valid A-label

This looks like it's in the same vein as our idna issues in Requests. Running IP addresses through the encoder is going to cause issues. That's why we ended up only running the idna portion on names that contained Unicode characters. It's possible a minor tweak to idna_encode in the pyopenssl module is necessary for IPv6 IPs in the Subject Alt Name field.

This is a crude hack, but enough to get things working for me with py27 at least. Feel free to reuse/improve.
diff --git a/urllib3/contrib/pyopenssl.py b/urllib3/contrib/pyopenssl.py
index 2762bca..df9946a 100644
--- a/urllib3/contrib/pyopenssl.py
+++ b/urllib3/contrib/pyopenssl.py
@@ -172,6 +172,8 @@ def _dnsname_to_stdlib(name):
                 return prefix.encode('ascii') + idna.encode(name)
         return idna.encode(name)

+    if ':' in name:
+        return name
     name = idna_encode(name)
     if sys.version_info >= (3, 0):
         name = name.decode('utf-8')

Until https://bugs.python.org/issue23239 is fixed correct certificates with SAN IP Addresses won't work on Python 2.7.


Doesn't work with Sentry 7.7.1

As the title says, it appears this plugin doesn't work with the latest version on Sentry. I have installed it but have no JIRA menu item in the projects settings area.

Facing the same issue

Here too :( I do have a JIRA menu but it doesn't save my credentials or gives any error/notice.

Closing this out as Sentry 7.x is no longer supported


Interrupt handling

Complete both masked and unmasked interrupt handling.

Implemented in 6ab43fb (within the timing limitations of the current CPU core). There's a separate Ticket #60 for running Klaus Dormann's testsuite.


chore: bump some sqlx-core dependencies

Do you think we could get this into a release? I think all the changes are semver-compatible.


Modify Contact and Lead Flows that generate one at a time Links

These flows need to look up the Unsubscribe link setup record to get what the friendly name for the URL is to append that to the field. That is passed to the record in the field, and then parsed in the formula Unsubscribe Link Contact/Lead to create the URL.

Can create a subflow for three elements of the contact and lead flows.


forums.plex.tv

Match found in list.34.hosts.ubuntu101.co.za.domains:
forums.plex.tv

Duplicate of #205


Update VS Code tutorial series for .NET 7

On step 6, on the 7.0 tab, make sure the command reflects 7.0 not 6.0 -> dotnet new console --framework net7.0

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

ID: 8cc2e9c2-f333-f6a1-064e-790d950b684c
Version Independent ID: 25a4c9b9-2808-e296-df03-5837000dcbad
Content: Create a .NET console application using Visual Studio Code - .NET
Content Source: docs/core/tutorials/with-visual-studio-code.md
Product: dotnet-fundamentals
GitHub Login: @tdykstra
Microsoft Alias: tdykstra

@mbjr24 thanks for reporting this oversight. There were several more places that needed the same update, and they're getting fixed in PR #32540


Adjust Indicator model LOP_Target field to Accept Decimal Values

Fix issue #711.

Coverage remained the same at 57.778% when pulling e3754505e6a4237886f21f68c80faf7d7ffd197f on Fix/#711 into 697a5deedba99dc8eedf0cd3ad59a6d03a091f5c on dev-v2.


(docs) Include reccomendations for where files should go in a module

CLA signed by all contributors.

Added to docs proper, with edits proposed in https://github.com/puppetlabs/bolt/pull/797


POSIX clipboard (wClipboard) produces on master invalid file lists

When trying to copy paste a folder (containing files) with xfreerdp3 (master), the submitted file list is invalid. A lot of file-replace dialogs are shown for each entry (where as the former file seems to always be 0 Bytes) and the attributes (the creation date) is invalid.
Reproduction steps:

Prepare a folder with files, you can clone the ogon-channels repo for this (small repo, lots of files).
Connect with xfreerdp3 (master) to an RDP server
Copy the folder on the client side and paste it on the server side

Observation:
You get a lot of file-replace dialogs and the file creation attribute is invalid. Also, the contents of the folder themselves are in the top level directory too.
These issue don't happen with xfreerdp2 (stable-2.0 branch).
Screencast of the issue:
Bildschirmaufzeichnung vom 2022-10-27, 09-15-11.webm
In the screencast, first xfreerdp3 (master) is used to try to copy-paste the folder (where you can see, that it fails with the errors mentioned above).
Then xfreerdp2 (stable-2.0) is used, where everything works fine.

The issue is likely in the POSIX clipboard part. A month or two ago, things just worked fine. So, any recent PRs in that area are possible candidates where things went wrong.

@pnowack ok, can not reproduce here on fedora 36 with gnome/nautilus. (copied my whole download folder with many subolders, android studio, ... in it)

@akallabeth that's weird. In any case, the error is with xfreerdp from the master branch and the nautilus version on the client side here 43.0 (am on Arch)
nautilus was ported to GTK4 in 43, but should not affect xfreerdp in any case here, and xfreerdp from the stable-2.0 branch is, as seen in the screencast unaffected by that.
In the screencast, where I close the xfreerdp3 window, the terminal output is seen and lots of Find NextFile failed (18) is shown, does this give a clue here?

Could you retry this with the ogon-channels repo here, which I use here too? In case nautilus 43 really brings an additional factor here (although it works fine with the stable-2.0 version), Ubuntu 22.10 also has GNOME 43.

can not reproduce here on fedora 36 with gnome/nautilus.

Does this also apply to the file changed dates here (i.e. not 1601, but actual date, that is also used on the client side for the respective file(s))?


Typo in LiveComponentHydrator.php

Q
A

Bug fix?
no

New feature?
no

Issues
Typo

License
MIT

Small typo in the message error. "on" instead of "one"

Thanks Jean-David.


Max buffer count reached for L1MetadataArray_test

L1MetadataArray_test_generate_abstract fails with [ERROR RSZ-0060] Max buffer count reached. error on newer OpenROAD. CI: https://github.com/The-OpenROAD-Project/megaboom/actions/runs/8708125043/job/23884744619#step:7:4378
This error looks like a problem with OpenROAD itself or the configuration for the build error. It appeared after Docker image had been updated - OpenROAD-flow-scripts hash used inside the image: 4ac2d01

Fixed by https://github.com/The-OpenROAD-Project/megaboom/pull/43

@eszpotanski I merged #43, but there are some fixes required to .github/ folder to reflect that L1MetadataArray_test only tests up to and including place now. CI is still trying to build L1MetadataArray_test_cts and L1MetadataArray_test_cts_make, which don't exist.
Could you create a PR to fix this?
Thanks!


[AGE-272] [Bug] Improve errors in evaluation view

When errors happen in the evaluation view, there are a couple of problems:

The error in the llm app are shown in black and not in red as expected

The issue is in the LLM app response and not in the UI

The error in the llm app is very general (500 internal error) and does not provide details about the message in the error as we do have in the playground (❌ Model gpt-3.5-turbo does not support JSON response format)

Unclear whether this is still relevant

The error for the evaluation is the same as in the llm app. Instead we should say "LLM app failed" or something similar.

This is resolved by the open PR

From SyncLinear.com | AGE-272

@jp this issue should actually be fixed in the new version of the SDK (we still are returning things as 200 messages but with another output). However the templates have not been updated / deployed yet.

not sure tbh on the interface here

My guess is that FastAPI should return 401 with details


Cannot read properties of undefined (reading 'length')

happening on submit for add resident form.

fixed with using res.redirect('/boat_ramp')


Make the All option actually pick all options

Issue
The "All" as empty string option isn't ok unless fuzzy searching is used, since juju_application="" won't match anything.
Solution
Update the matchers (and injection) to use regexp searches. This is annoying in regexp searches due to the fact that we can't pass a template string to cos-tool (right now)
Testing Instructions
Build a test Grafana with this library. Relate it to something (let's say Kafka). Make sure they are filled in.
Release Notes
Make the All option actually pick all options

Resolves #196. (add it to your PR description so it closes the issue on merge!)
Would love for this to finally pass tests as well :D LGTM!

It should pass now, rebased/merged on top of the working main. Only timing out because GH is GH...


SP-5868

Backport of PDI-19054 - LogRegistry Purge Not Removing Channels with More than 10% of "Active" Channels (9.1 Suite)

:warning: Build finished in 24m 36s
Build command:
mvn clean verify -B -e -Daudit -amd -pl core

:no_entry: Failed Tests
:cloud_with_lightning_and_rain: 1 test(s) failed:
org.pentaho.di.trans.step.mqtt.MQTTStreamSourceTest.testMqttStreamSingleTopic (click to expand)
${result.errorDetails}

Tests run: 4958, Failures: 1, Skipped: 7    Test Results

:information_source:  This is an automatic message


Why the images didn't load?

I have no problem in uploading images, but the images uploaded didn't even load. I can't figured out what's wrong. did you know why?

Check the URL you are getting there - likely your media configuration needs a change.

on settings.py
MEDIA_ROOT = os.path.join(BASE_DIR, 'media')
MEDIA_URL = 'media/'
The link is media/uploads/2017/04/02/blue-square.jpg

Your MEDIA_URL should probably start with a slash (MEDIA_URL = '/media/')

I'm having the same problem. I can see the image there in the media/uploads/2017/04/09 directory, but it's not loading on the page.
Did you ever manage to get to the bottom of it?
My settings are the same as ylto.
MEDIA_ROOT = os.path.join(BASE_DIR, 'media') MEDIA_URL = '/media/'

Do your media handling actually works in Django? The only reason they are not displayed is because Django isn't handling them (in development).

My bad, I forgot to add urlpattern for media. Here's how I solved it
`from django.conf import settings
from django.conf.urls.static import static
urlpatterns = [
# ... the rest of your URLconf goes here ...
] + static(settings.MEDIA_URL, document_root=settings.MEDIA_ROOT)`
https://docs.djangoproject.com/en/1.11/howto/static-files/

my fault was adding the "+ static(settings.MEDIA_URL, document_root=settings.MEDIA_ROOT)" in the wrong urls.py file....


Multi-panel execution mode - Focus multiple panels at a time

Description of the new feature/enhancement
It would be nice to have a feature to write on to multi-panels - where the user can simultaneously write to all panels at the same time. Useful while working with multiple machines which have the same setup on scenarios like checking logs, listing files etc. MobaXterm is already having such a feature. Currently on wt when working with multiple panels, we can only focus on a single panel at a time.

Thanks for the suggestion! This is actually already being tracked by another issue on our repo - please refer to #2634 for more discussion.
/dup #2634


meta multiple requests

client:
fio --name=jfs-test --directory=/jfs/ -ioengine=libaio --rw=read --bs=1m --size=1g --numjobs=1 --direct=1 --group_reporting
juicefs:

log:

Read is called for each block in the chunk. It's okay since the result is cached in the openfiles.


ScrollTo functionality

Note: For large trees - would be nice to have ability to scrollTo a specific node - see behavior from antD Tree -
https://ant.design/components/tree#tree-methods
Describe the solution you'd like
A method on the tree instance that allows developers to programmatically scroll to a specific node and align accordingly.
What value does this proposed solution bring to users?
When auto-selecting - like after a search - allows the tree to focus on the selected node
Displaying a large tree with selections - allows developers to focus on the actual selected value vs user having to manually find previously selected nodes.
Describe alternatives you've considered
Custom event to scroll using document.querySelector or scrollIntoView()

@cda210. Hi Chris, since this is a request from internal to our company, can you please file an RFA request and then have our PM prioritize it by talking to your PM? In the meantime, I’m going to close this issue since active issue counts are seen by potential consumers as a negative. So we try to keep them low. If your internal JIRA ticket is approved then we’ll reopen this issue here when the PR is worked. We want to keep the RFA process for Vertex <—> Vertex the same.

@hnnweb this issue was closed as not completed. If you’d like it to be completed, tell me and I can reopen it. But I’d only reopen it if it’s something you or your team would be interested in building since it’s not a priority for our team.

No, you closed it as "completed"

sooo, this won't be implemented?

See comment https://github.com/dgreene1/react-accessible-treeview/issues/107#issuecomment-1845362722

I could try a PR, but as I'm using it for a personal project I can't guarantee a rush on it.
Do you have any suggested approach?

I do not. I personally wouldn’t use a tree this type of thing. I’d probably use a table with a filter.


Make DemoYesNoSUT work with caching.

Without this PR we fail the expensive-tests due to #168, but that merged recently enough the daily check hasn't run.

Fixes #173


chore(main): release 1.4.0

:robot: I have created a release beep boop
1.4.0 (2023-09-18)
Features

add snap ci (#49) (5cd483f)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/aidanaden/canvas-sync/releases/tag/v1.4.0 :sunflower:


Color of Rotate now to be changed to make it look like an Actionable text on the UI screen

What kind of request is this (question/bug/enhancement/feature request): bug
Steps to reproduce (least amount of steps as possible):
on rancher:v2.2-head with UI tag 2.2.83
Have a cluster(DO cluster-3 nodes) in v2.2.4 - with custom image, giving 30 minutes of expiry for the cluster.
Upgrade to v2.2-head with UI tag 2.2.83.
Warning shown on the UI - This cluster has certs that are expiring or have expired. Roatate Now

Color of Rotate now to be changed to make it look like an Actionable text on that screen

Can be tested with v2.2.5-rc10

verified with v2.2.5-rc11

Rotate Now is visible now and is seen as an Actionable item
ui-index is https://releases.rancher.com/ui/2.2.86/index.html


MudSelect possible race condition on dispose

Bug type
Component
Component name
MudSelect
What happened?
I've hosted a page via BlazorWebView for WinForms on a test application. On the hosted Page, there's nothing but a MudSelect component. When disposing of the BlazorWebView instance, there seems to occur a race condition because of some JS behind the MudSelect, resulting in a total application freeze.
I'Ve tracked this issue down to this specific component. Other Mud components work as expected.
Expected behavior
Dispose should work when using MudSelect.
Reproduction link
https://github.com/CyanGrave/MudBlazorIssueMudSelectDemo
Reproduction steps

Run the provided demo app
click the close button on the window
witness the glorious frezzing =)
...

Relevant log output
No response
Version (bug)
6.0.7
Version (working)
No response
What browsers are you seeing the problem on?
Microsoft Edge
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

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

Reproduction link says 404

i'm sorry. should be visible now.

Reverting to 6.0.6 solved the issue (select list not closing at all), but when can we expect the solution? In 6.0.8?

6.0.6 did not solve the issue. Also the problem is not related to #4220. This issue does not refer to a bug with closing the selection. It mentions a race condition when disposing of a MudSelect component. I also did try 6.0.8, issue still persists.

Have the same issue , in 6.0.9 too. Any MudSelect component, even an empty one, in a winforms app with BlazorWebView, seems not to dispose properly. Changing to standard blazor 'select' component works fine, the app closes as normal then instead of hanging during closing.  Any ideas when this could be fixed ?

Have the same issue , in 6.0.9 too. Any MudSelect component, even an empty one, in a winforms app with BlazorWebView, seems not to dispose properly.

That should be fixed after 6.0.10. Could you provide info? If it still occurs we can reopen the issue.


fix: Remove old @microsoft/recognizers-text-number version with postinstall scripts

#minor
Description
This PR removes the old version or @microsoft/recognizers-text-number from the other recognizers packages that require it.
Specific Changes

Added postinstall scripts in botbuilder-dialogs and botbuilder-dialogs-adaptive package.json files to remove the old version of  @microsoft/recognizers-text-number.

Testing
These images show the dependencies tree before and after running the postinstall scripts.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 84.122%

Totals

Change from base Build<PHONE_NUMBER>:
-0.4%

Covered Lines:
20329

Relevant Lines:
22877

💛  - Coveralls

By using Yarn in the post-install script, you're FORCING a dependency on anyone using your package must have Yarn installed.
There's zero reason for this... why not use NPM?
It's fine if you use Yarn in your dev & builds, but to force it on ALL consumers of the package? 👎👎
For example, if a user doesn't have Yarn installed, this breaks all builds of new Teams projects created with TTK (see my referenced bug above

@andrewconnell Apologies.  We'll take care of it.


Do not propagate blockchain broadcast errors

Description

Fixes pyrsia/pyrsia#1213
This PR don't propagate blockchain broadcast errors because broadcasting doesn't guarantee other nodes will receive the blocks
Screenshots (optional)
PR Checklist

[x] I've read the contributing guidelines.
[x] I've read "What is a Good PR?"
[x] I've included a good title and brief description along with how to review them.
[x] I've linked any associated an issue.

Code Contributions

[x] I've built the code cargo build --all-targets successfully.
[x] I've run the unit tests cargo test --workspace and everything passes.
[x] I've made sure my rust toolchain is current rustup update.

I hope you didn't mind me committing the change in your PR already :)

I hope you didn't mind me committing the change in your PR already :)

Thanks for your comments. :)


Confirmation on how can we restore files from dir when using only Master/Volume

I'm using only Master/volume no filer and storing individual generated links into a db tagged to a user. I'm planning to use this in production and to backup daily, in the event there is a need to recover files from backup, can I just spin up a weed instance to access the backup files?
Will the link path e.g."3,01637037d6 " change ? (in http://localhost:8080/3,01637037d6). My understanding is that only ip/port will change as it will be set up in another environment. (I will still be able to access this file using http://newipaddr:port/"3,01637037d6 )
Note: I've tested this myself and notice no change (able to access eg.http://localhost:8080/3,01637037d6 and view similar files in volume using ./weed export from both machine) when i copy all dir files (.vif, .dat, .idx) into another machine and spinning this other weed instance.
I'm looking for confirmation as I did not see much documentation about this.

This is correct.

@jereloh How do you plan to use it in production if there is no guarantee that, for example, volume 3 will not be moved to another server volume?

It seems that a separate http server will be required to generate the link via x-accel-redirect


Setting for Custom Highlight background color

The ability to change the highlight background color from red to anything else.

There will be no setting for that. Either complete theme support or nothing.


Redis Config URL

This adds the ability to pass a url in your redis.json, similar to MySQL.

Codecov Report

Merging #16 into master will increase coverage by 0.46%.
The diff coverage is 30.61%.

@@            Coverage Diff             @@
##           master      #16      +/-   ##
==========================================
+ Coverage   67.51%   67.98%   +0.46%
==========================================
  Files           5        5
  Lines         197      203       +6
==========================================
+ Hits          133      138       +5
- Misses         64       65       +1

Impacted Files
Coverage Δ

Sources/RedisProvider/Provider.swift
36.06% <30.61%> (+5.15%)
:arrow_up:

Continue to review full report at Codecov.

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

So far so good. It would be great if you could add some tests for the new configuration method.
Either casting and verifying the fields of the created Clients, or connecting to the test database with URL configuration and running commands should work, or even both strategies.

Thanks! Going to merge this and test locally then tag


a folder for more things we want to have in the standard library

Please read the readme in the pull request before commenting.
Happy to bikeshed on the name, before we start moving things here.
Lets have some puns and please vote with some github emoji for the puns you like, so we can actually also settle on one.
Note: You get unlimited votes.

nps -> national proof service

stdwish -> the standard library we wish to have or the std we wish to have

Good to know we aren't the first to need this.
But Max, I don't think you are taking this review seriously.
I expect a pun :P

coqney: since this is our accent?

stockcoq -> a store of coq goodies

coq-a-doodle-doo -> makes me smile

coqorico -> french for coq-a-doodle-doo

coq-a-doodle-doo -> makes me smile

I worry about importing this, but otherwise I love it.

coqlude

coq-a-doodle-doo -> makes me smile

I worry about importing this, but otherwise I love it.

How about shortening it
doodledo

concoqtions

coq-a-doodle-doo -> makes me smile

I worry about importing this, but otherwise I love it.

How about shortening it
doodledo

Maybe this is a good name for a monad library, when we get to that :)

So far we have:
coqlude, which so far I just think is funny.
coqney, which Paul at least thinks is funny.
stockcoq, which I think is funny and appropriate
doodledo, for a future monad library
concoqtions, which I think is funny and appropriate
So for me it is between stockcoq and concoqtions.  If no one else votes, I am going to make a call, at latest tomorrow.

So I am going to go with coqstock, until someone brings this up again.


Update ionic.d.ts

Fix IonicActionSheetOptions bugs

ionic/ionic.d.ts
to author(@spencerwi). could you review this PR?
:+1: or :-1:?
check list

[ ] pass the Travic-CI test?

Ah, good catch. That's an accurate fix.

@spencerwi thanks!
@TimChen44 thanks mate!


Bug Summary

Describe the bug
make docs fails on a new project with:
Error reading page 'modules.md': No module named 'mkdocstrings.handlers.python'

Thanks for raising this issue! I tried to reproduce it by running the following commands:
 2762  pip install cookiecutter
 2763  cookiecutter https://github.com/fpgmaas/cookiecutter-poetry.git
 2764  cd example-project
 2765  poetry install
 2766  make docs

This seemed to work fine for me. Could you maybe add some more details on how to reproduce your issue?

Hi there --
I was able to work around the issue by bumping the constraints on mkdocstrings in the  pyproject.yaml:
mkdocstrings = {version = "^0.22.0", extras = ["python"]}
mkdocstrings-python = "^1.7.0"

(My values may be overly conservative).  In my case, I was getting a mkdocstrings-python < 1.0 with the current settings in ccp .
Also, to get the pages to deploy to gh-pages on release, I needed to tweak the permissions in the release workflow yaml (again, I probably used settings that are too broad).
my repo is here: https://github.com/habemus-papadum/how-now if you want to see the exact settings.
I'm guessing it is due to me using python3.11.  But I feel this issue is resolved on my end so am going to mark it closed.
Thank you for create CCP!