fix: updated tools workflow

Description
Updated tools inside the website and refactored the code of the tools script.
Related issue(s)
Fixes #1604

@derberg I recently checked with the Github API that they have improved their Github Search API results and it's working fine with correct results. Hence, I'm enabling the workflow inside website repository so that we don't have to do this step manually.

/rtm


Possibly unhandled StatusCodeError: 400

When i receive the StatusCode 400 from a server i have this error
Possibly unhandled StatusCodeError: 400 - { errorMessage: "Unable to complete Request", errorCode: 1 }
    at new StatusCodeError (/api/node_modules/request-promise/lib/errors.js:26:15)
    at Request.RP$callback [as _callback] (/api/node_modules/request-promise/lib/rp.js:58:34)
    at Request.self.callback (/api/node_modules/request/request.js:360:22)
    at Request.emit (events.js:110:17)
    at Request.<anonymous> (/api/node_modules/request/request.js:1202:14)
    at Request.emit (events.js:129:20)
    at IncomingMessage.<anonymous> (/api/node_modules/request/request.js:1150:12)
    at IncomingMessage.emit (events.js:129:20)
    at _stream_readable.js:908:16
    at process._tickCallback (node.js:355:11)

Should i post this issue at the request module?

Hi @luanmuniz that is definitely something related to the promise functionality. It means that you either did not register an error handler (.catch( < error handler here> ) or .then(..., < error handler here >)) or you register it too late - which would be an advanced use case.
If you show me your code snippet I could help.

Yes.  I did not register the error handler for this specific scenario. Sorry about that.
I was expecting to just not execute the success function and ignore the error.
Its good that we are forced to handle the error, thank you for that! :D

Indeed, this message comes from Bluebird and is a good design choice. Have a good day!


WidgetStateProperty upkeep

This pull request adds 2 missing constructors in material_state.dart that had been overlooked previously and fixes inaccurate documentation as reported in #154197.
Additionally, WidgetStateColor.fromMap() has been turned into a const constructor to match the others.

https://github.com/flutter/flutter/pull/154695 is going to generate a few conflicts here, so I'll need to make some updates once it's merged.


itk::WatershedImageFilter crashes (assertion error) on certain inputs

Description

itk::WatershedImageFilter crashes (assertion error) on certain inputs.  This will cause the Python kernel to crash or a C++ program to terminate.
Steps to Reproduce

int itkWatershedImageFilterBadValuesTest(int argc, char* argv[]) {
	int result = 0;

	char* path = argv[1];
	std::cout << "The file path is: " << path << std::endl;

	using ImageType = itk::Image< float, 2>;
	auto reader = itk::ImageFileReader< ImageType >::New();
	reader->SetFileName(path);
	auto img = reader->GetOutput();

	auto watershed = itk::WatershedImageFilter< ImageType >::New();
	watershed->SetLevel(1);
	watershed->SetThreshold(0.1);
	watershed->SetInput(img);
	watershed->Update();

	std::cout << "Got through here" << std::endl;

	watershed->SetLevel(3);
	watershed->SetThreshold(10);

	try {
		watershed->Update();
	}
	catch (itk::ExceptionObject &e) {
		e.Print(std::cout);
	}
	catch (char* e) {
		std::cout << "char* exception: " << e << std::endl;
	}
	catch (...) {
		std::cout << "Failed by exception!";
		result = -1;
	}

	return result;
}

floatmax1ultrasoundeye.zip
Expected behavior
Watershed was part of a image processing pipeline I was numerically optimizing.  The inputs are likely poor for the image it's segmenting, but I would expect an exception (at the worst) or simply a single label image.
Actual behavior

for (segment_ptr = segments->Begin(); segment_ptr != segments->End(); ++segment_ptr)
  {
    labelFROM = (*segment_ptr).first;

    // Must take into account any equivalencies that have already been
    // recorded.
    labelTO = m_MergedSegmentsTable->RecursiveLookup((*segment_ptr).second.edge_list.front().label);   <--------- ERROR
    while (labelTO == labelFROM) // Pop off any bogus merges with ourself
    {                            // that may have been left in this list.
      (*segment_ptr).second.edge_list.pop_front();
      labelTO = m_MergedSegmentsTable->RecursiveLookup((*segment_ptr).second.edge_list.front().label);
    }

![image](https://user-images.githubusercontent.com/59569148/93356501-3e32c780-f80d-11ea-843c-f9fd716dd325.png)

Reproducibility

Versions

Environment

Additional Information

I'm confirming whether the example crashes on FloodLevel < 1.0

Did you close accidentally or on purpose?

I'm trying to see if it is a real issue.  The parameters in my test were
out of bounds for the documentation.
On Wed, Sep 16, 2020, 3:20 PM Dženan Zukić<EMAIL_ADDRESS>wrote:

Did you close accidentally or on purpose?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/InsightSoftwareConsortium/ITK/issues/2007#issuecomment-693608993,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOGPH7F7FNZAAXYBISJG3CLSGEFWJANCNFSM4RO7QC2A
.

There's a definitely an issue.  I think the above is the key part, but I'm verifying again.  I'm running an optimizer and the values are clamped between 0 and 0.9 which according to the documentation should be valid for level and threshold.
It might that there's still something where level < threshold or vice versa and maybe that's not a correct input, but given that the filter throws an exception for other reasons I'll likely add another exception to the list iterator above.  Note, it still currently causes an assertion error in the list.
By the way, any thoughts on ImageFilters throwing exceptions?  Wouldn't it be better to simply return a degenerate labeling (all background or foreground)?

I'm rerunning my optimization code, but clamping threshold to a percentage of level (forcing threshold < level).  That might have been erroneous input.  Regardless, I'm going to propose submitting the following:
    labelTO = m_MergedSegmentsTable->RecursiveLookup((*segment_ptr).second.edge_list.front().label);
    while (labelTO == labelFROM) // Pop off any bogus merges with ourself
    {                            // that may have been left in this list.
      (*segment_ptr).second.edge_list.pop_front();
      labelTO = m_MergedSegmentsTable->RecursiveLookup((*segment_ptr).second.edge_list.front().label);
    }

to this:
    if (segment_ptr->second.edge_list.empty())
    {
      itkGenericExceptionMacro(<< "CompileMergeList:: An unexpected and fatal error has occurred.");
    }
    labelTO = m_MergedSegmentsTable->RecursiveLookup((*segment_ptr).second.edge_list.front().label);
    while (labelTO == labelFROM) // Pop off any bogus merges with ourself
    {                            // that may have been left in this list.
      (*segment_ptr).second.edge_list.pop_front();
      labelTO = m_MergedSegmentsTable->RecursiveLookup((*segment_ptr).second.edge_list.front().label);
    }

Please turn it into a PR.

Will do.

@dzenanz
@thewtex
I had it fail on another input with the threshold less than the level.  Let me know if that is concerning or not?  I think my fix will stop the crash, but does it make sense that this throws exceptions so often?   It also throws exceptions in other places which I'm currently just catching and considering a failed segmentation.

Filters are allowed to throw exceptions. That is better than failing silently. With an exception it is clear where the problem is. Re-throwing is an option, I guess. Maybe @blowekamp has an opinion.

I moved to the MophologicalWatershed filter due to stability and reproducibility issues that  were never tracked down in this filter.
Throwing exceptions is certainly preferred to segfault or assert failure for conditions occur without precondition violations.
If "threshold < level" should be a hard precondition it can be verified early in and overloaded ProcessObject::VerifyPreconditions method to throw an exception when violated.

@blowekamp That's good to know.  I think a review of this implementation could be warranted.  For the PR, I'll just fix the assert failure.  Unfortunately, it looks like the exceptions are not predictable from the input data.  I had another failure with a normal image and threshold > level.  Without digging into the actual algorithm we're using here, I'm not convinced why this isn't returning zero labels or a single label instead of throwing an exception.

Updating the PR - I think the unused argc in the test driver is causing OS X to fail.


This will get a Golang rewrite next weekend.  What features are most important?

Having this loaded in tmux would be a blessing while coding, so I'll rewrite it.
What features are most important to you?

This rewrite begins tomorrow.
Top priorities at this time are:

Live(ish) score updates
Play-by-play game view
Keyboard controls
View games by day

Did this get released?

Not yet, no.
The free API was disabled, so it's gonna need a bigger rewrite than I anticipated.

how did you Go  :-p

Polling fails--they block you, so it's a no Go.  At least the last time I tried.


Add array_slice function

Change the commit message and title of this pull request to "Add array_slice function".

Always start commit message with capital letter
The concept of "User-defined" function doesn't really exist in Presto

If I remember correctly, we decided to throw an exception when length < 0 and return an empty array in all other failure cases.

Do something similar to this (taken from DateTimeFunctions)
checkCondition((zoneOffset % 60_000) == 0, INVALID_FUNCTION_ARGUMENT, "Invalid time zone offset interval: interval contains seconds");

This throws a PrestoException with INVALID_FUNCTION_ARGUMENT when condition is not met.

This needs documentation (add it to array.rst)

Arrays are 1-indexed, so the offset needs to be a 1-indexed argument. It should also throw an exception if the offset is zero

Squashed the commits and merged :)


feat: Create-faq

Pull Request: [FEAT] - Create FAQ Endpoint #298
Description
Added functionality to create a new FAQ in the Help Center module. This feature allows administrators to add frequently asked questions with the required fields and handles optional fields properly.
Endpoints

POST /api/v1/faqs

Request Body
{
  "question": "string",
  "answer": "string",
  "category": "string"
}

Response Format

Create FAQ (POST /api/v1/faqs){
  "status_code": 201,
  "success": true,
  "data": {
    "id": "String",
    "question": "String",
    "answer": "String",
    "category": "String",
    "createdBy": "String",
    "createdAt": "String",
    "updatedAt": "String"
  }
}

Manual Testing

Create a New FAQ

Action: Send a POST request to the POST /api/v1/admin/faqs endpoint with a valid CreateFaqDto payload.
Expected Result: You should receive a response with status code 201 and the created FAQ object.

Example Request:
curl -X POST http://localhost:3000/api/v1/faqs \
-H "Content-Type: application/json" \
-d '{
  "question": "What is the return policy?",
  "answer": "Our return policy allows returns within 30 days of purchase.",
  "category": "Policies"
}'

Invalid Input Data

Action: Send a POST request to the POST /api/v1/faqs endpoint with invalid or incomplete data.
Expected Result: You should receive a response with status code 400 and an appropriate error message indicating the missing or incorrect fields.

Checklist

[x] Implemented the endpoint to create a new FAQ.
[x] Added validation for required fields and data types.
[x] Verified that all optional fields (category) are handled correctly.
[x] Ensured proper error handling and logging for invalid input.
[x] Updated the response format to include createdAt and updatedAt fields.

Reference Issue
[FEAT] Create an endpoint for Admin to Create FAQs

hey
@Heba-WebDev
@Homoakin619
@Cobidex please review 🙏 thanks


NewRelic::Agent.add_custom_attributes support for ActiveJob::DelayedJob transactions

Is your feature request related to a problem? Please describe.
In many of our controllers, we do things like...
    ::NewRelic::Agent.add_custom_attributes(
      {
        my_cool_attribute: "blah"
      }
)

to add custom attributes to the resulting transactions, which we can then use in queries when generating charts and whatnot in New Relic. However, we cannot figure out how to implement this same functionality for ActiveJob::DelayedJob transactions so that we can add similar info to our background job transactions.
Feature Description
We would really, really like an easy way to add custom attributes to ActiveJob::DelayedJob transactions.
Describe Alternatives
We have tried creating a DJ plugin like so:
module NewRelicInstrumenter
  class DelayedJobPlugin < Delayed::Plugin
    callbacks do |lifecycle|
      lifecycle.before(:perform) do |_worker, job|
        ::NewRelic::Agent.add_custom_attributes(
          {
            my_test_attribute: "blah"
          }
        )
      end
    end
  end
end

and then adding it into our DJ initializer (config/initializers/delayed_job.rb) like so:
require "new_relic_instrumenter"

Delayed::Worker.plugins << NewRelicInstrumenter::DelayedJobPlugin

but we do not see any custom attributes on the resulting transactions. I'm not sure if we're just missing the right place/way to implement this, or if it's not possible right now, but we would LOVE guidance/tips here.
Additional context
n/a
Priority
[Really Want]

Hi @danielmklein, thanks for reaching out! I'm surprised to hear NewRelic::Agent.add_custom_attributes isn't working within ActiveJob::DelayedJob transactions.
I have a few questions I hope will provide some context to dive more deeply into this issue:

Do you see any errors or other odd messages in the newrelic_agent.log file when attempting either strategy?
What versions run on your application for the following:

Ruby
ActiveJob
DelayedJob
newrelic_rpm

@kaylareopelle thanks for following up!!
I do not see anything unexpected in newrelic_agent.log.
We are running...

Ruby 2.7.4
activejob <IP_ADDRESS>
delayed_job 4.1.9
newrelic_rpm 8.1.0

For some additional context that I just noticed --
If I add custom attributes from within the job's perform method itself, it works. However, when doing this I cannot access DJ-specific stuff like what time the job was queued (unless I am missing something). In other words, this works, and I see the custom attributes on the resulting transactions:
class NewRelicTestJob < ApplicationJob
  queue_as :default

  def perform
    ::NewRelic::Agent.add_custom_attributes(
      {
        job_class: "NewRelicTestJob",
        attribute_source: "inside_the_job"
      }
    )
    puts "Hello from NewRelicTestJob at #{Time.current}!"
  end
end

However, adding attributes from a DJ plugin does not work (I know I've already said this, just wanted to get both these in one place). Digging into the source code a bit, I noticed that if I put a breakpoint inside the block here, NewRelic::Agent::Transaction.tl_current is nil, which I think has something to do with why attributes added here don't make it (because there's maybe not yet an actual transaction to put them on?):
module NewRelicInstrumenter
  class DelayedJobPlugin < Delayed::Plugin
    callbacks do |lifecycle|
      lifecycle.before(:perform) do |_worker, job|
        # `NewRelic::Agent::Transaction.tl_current` is nil here
        ::NewRelic::Agent.add_custom_attributes(
          {
            job_class: job.payload_object.job_data["job_class"],
            attribute_source: "NewRelicInstrumenter:DelayedJobPlugin",
            job_creation_time: job.created_at
          }
        )
        puts "Hello from NewRelicInstrumenter::DelayedJobPlugin at #{Time.current}!"
      end
    end
  end
end

I hope this helps! Please let me know if there's any other info I can provide -- I am going to keep poking at things and will follow up if I have a breakthrough. 😄

Good news!! After experimenting with DJ callbacks, this works:
module NewRelicInstrumenter
  class DelayedJobPlugin < Delayed::Plugin
    callbacks do |lifecycle|
      lifecycle.around(:invoke_job) do |job, *args, &block|
        # Forward the call to the next callback in the callback chain
        puts "Hello from NewRelicInstrumenter::DelayedJobPlugin at #{Time.current}!"
        ::NewRelic::Agent.add_custom_attributes(
          {
            job_class: job.payload_object.job_data["job_class"],
            attribute_source: "NewRelicInstrumenter:DelayedJobPlugin"
          }
        )
        block.call(job, *args)
      end
    end
  end
end

In conjunction with (as before), this in the initializer in config/initializers/delayed_job.rb:
require "new_relic_instrumenter"

Delayed::Worker.plugins << NewRelicInstrumenter::DelayedJobPlugin

That's wonderful news, @danielmklein! Thanks for sharing your solution! Is there anything else our team can do to help resolve this issue?

@kaylareopelle not currently. Is there a good place where we might be able to document this for future travelers?

@danielmklein - there sure is! We have a specific page in the Ruby Agent's documentation for Delayed::Job Instrumentation that would be a good fit for this solution.
New Relic's docs are open source as well. You're welcome to contribute to this page directly by clicking the "Edit Page" button in the top right corner. If you'd prefer we take care of the changes, let me know and I'll create a ticket for our team to add the content.

@kaylareopelle thanks for the guidance! I've opened a PR on the docs repo here: https://github.com/newrelic/docs-website/pull/4887. We shall see how it fairs in the review process 😂


Specifying COMPOSER_VERSION=2 in .env doesn't seem to work

Steps to reproduce

Add COMPOSER_VERSION=2 to .env file.
Run warden env down && warden env up -d
Run warden shell
Run www-data@pacvac-php-fpm:/var/www/html$ composer --version

Expected result
    Composer version 2 [...]

Actual result
Composer version 1.10.19 2020-12-04 09:14:16

It's expected because of this commit: https://github.com/davidalger/warden/commit/b64fd71c376a1af5def964072cf5d35d9b16ebec

This does work for us, so I do not expect a general bug here.
What does warden version output? Are you using 0.10.0 or later?
Try running warden env pull to update the underlying Docker images. Additionally, try adding the --force-recreate flag to warden env up.

Hmm, you're right, the --force-recreate flag seems to have fixed it! Thanks


POL-433 Secure connections to/from Kafka

This PR provides a sample config that can be used to secure the connection from policies-engine and kafka.
There is also a script to help to locally validate the setup.

@josejulio Since we just bumped Quarkus to 1.13.6.Final, policies-engine now depends transitively on kafka-clients 2.8.0.

That was my first and last time resolving a conflict with the GitHub UI 😄 (I don't really like conflicts resolution based on merge)

@josejulio Since we just bumped Quarkus to 1.13.6.Final, policies-engine now depends transitively on kafka-clients 2.7.0.

Should we remove it from the POM?

We should probably depend on quarkus-kafka-client instead of the Apache dependency.

We should probably depend on quarkus-kafka-client instead of the Apache dependency.

Updated. Also did some tests and it's connecting to kafka using the secure channel.


[mixedreality] Upgrade @azure/mixed-reality-remote-rendering to ESM/vitest

Packages impacted by this PR

@azure/mixed-reality-remote-rendering

Issues associated with this PR

https://github.com/Azure/azure-sdk-for-js/issues/31338

Describe the problem that is addressed by this PR
Migrates the @azure/mixed-reality-remote-rendering package to ESM/vitest via automation.
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Are there test cases added in this PR? (If not, why?)
Provide a list of related PRs (if any)
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[ ] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[ ] Added a changelog (if necessary)

API change check
API changes are not detected in this pull request.


OpenAI TTS API Key

System Health details
Couldnt find the health card
Checklist

[X] I have enabled debug logging for my installation.
[X] I have filled out the issue template to the best of my ability.
[X] This issue only contains 1 issue (if you have multiple issues, open one issue for each issue).
[X] This issue is not a duplicate issue of currently previous issues..

Describe the issue
OpenAI TTS Key - There is no documentation to advise where this needs to be put. I've tried putting it int he configuration.yaml file and sleect OpenAI TTS from the service. But the voices dont play.
Reproduction steps

as aboe
2.
3.
...
Debug logs
na

Diagnostics dump
No response

Hi @Steveelliott111.
The OpenAI TTS integration is entirely separate to the Chime TTS integration. Please review the integration's GitHub repository for the steps to install and configure it.


cloud server

Hi! first congratulations on the work, this is helping me a lot I wanted to know if you have any tips regarding my scenario.
I use a debian server in the cloud with the screenly monitor installed, the communication between the pi and the monitor is done through a vpn. The system can see the pi normally, even the addon works perfectly, but does not succeed on the assets, a message No Screenly API detected! - No data could be collected ...
Has anyone gone through this situation?
I'm Brazilian, sorry for my bad English

Hello @c4rv88
thanks for you congratulation 😄
Normally, the message appears if this happened in line: index.php:326
$playerAPICall is TRUE when the player is reachable and $playerAPI must not return the value "authentication error 401"!
Do you use the Screenly OSE Player with https:// protocol?
Do you activate the Basic authentication in the Screenly OSE Player settings and not enter the right data into the monitoring?
When both options are not the problem test if you can reach the player via curl command from your server via VPN, please.
curl xxx.xxx.xxx.xxx
Best,
didiatworkz

I already did a clean installation, I did not enable https. Even so, the error persists. I did the test with curl and I can access my pi. I will do other tests, thanks for the tips.

You're welcome

Hi!
I haven't been successful in my settings yet!
Does he have any ip redirection?
My tests:
test ping :  Server - client interface tun0 [ok]
teste Ping: Server - client Interface  wlan0 [no sucess]
Test Ping:  client -  server interface tun0[ok]
teste ping: Client - Server Interface eth0[ok]
any tips?
Thank you!

Hello @c4rv88
there is no restriction from my side...
What I wonder is how you set up your VPN tunnel between the client and server (SSH, PiVPN or OpenConnet).
Maybe  @nsuva can help here as well, since he has already set up a working system via SSH tunnel.
If you look at your ping test, it looks like there might be a firewall not open or a forwarding is missing.
But unfortunately, I am no expert. :(
I used the system in my company in the local network and it definitely works ;)
Best
didiatworkz


Accessing global variables from within the fast_map function

Hello,
I love the idea of the package, however I was wondering is there any way to pass immutable global variables into the function body using fast_map()? Normal python map() supports this, so by running this:
from fast_map import fast_map
import time

def task_with_multiple_params(a, b):
    return a + b + c

if __name__ == '__main__':
    c=5
    print(c)
    print('Using function with multiple parameters:')
    for s in map(task_with_multiple_params, [1,2,3], [1,2,3]):
        print(s)

the output is 7 9 11
But when I run the same code using fast_map() instead of map() I get the following error:
NameError: name 'c' is not defined
I understand this could be a problem of the multiprocessing library which doesn't allow mutable global variables to be used in functions when multithreaded so I was wondering if a workaround for this library was possible.  Python map() function gets around this by allowing global variables since everything gets done on a single thread so there are no simultaneous READ/WRITE operations at the same variable memory adress.
Additional reason for posing this question is that for example in my code unfortunately using multiple parameters with same data copied over the length of the list is hard to implement since there are multiple large global matrices, so creating a list of 32000 elements of the same 50000x50000 matrix (to make it iterable) is very memory intensive, and the matrices are not changed as a result of the function. So for example this works normally:
from fast_map import fast_map
import time

def task_with_multiple_params(a, b, c):
    return a + b + c

if __name__ == '__main__':
    c=[5,5,5]
    print(c)
    print('Using function with multiple parameters:')
    for s in fast_map(task_with_multiple_params, [1,2,3], [1,2,3],c):
        print(s)

But isn't feasible.
Thanks in advance,

Hello, sorry for late response, I tried to reproduce the error but running the code below with fast_map instead of map worked well for me and the NameError didn't occur:
from fast_map import fast_map
import time

def task_with_multiple_params(a, b):
    return a + b + c

if __name__ == '__main__':
    c=5
    print(c)
    print('Using function with multiple parameters:')
    for s in map(task_with_multiple_params, [1,2,3], [1,2,3]):
        print(s)

I used python 3.7.15 on Lubuntu 18.04 with 0.0.8 version of fast_map.


Bug in active detection?

Using #32, moving the active detection probability slider around has very little impact on the outcomes.
At 500 simulations, otherwise default settings, I'm seeing:

Active prob 4%: control prob 68%, average duration of infectiousness 2.04, $R_e$ 0.98, prob of active detection 0
Active prob 49%: control prob 68%, average duration of infectiousness 2.04, $R_e$ 0.98, prob of active detection 0.02
Active prob 82%: control prob 70%, average duration of infectiousness 2.01, $R_e$ 0.96, prob of active detection 0.05

Why is active detection not happening?

[Some thinking aloud]
Since the passive detection delay is 2, we can potentially actively detect new infections that arise in the first unit of time of the duration of infectiousness of a passively-detected individual. This on average is 0.5 infections (birth rate being used is 0.5). Thus I would expect something like 0.25 active detections per passive detection, ignoring active-active chains of detection.
Since the active detection delay is 2, and the latent duration is 1, any such actively-detected infection should arise in its own first unit of infectious time. Since infection times are uniform, this is an average of 0.25 new infections which could be actively detected after an active detection.

Maybe this isn't wrong but just deeply counter-intuitive?
Fixing delays and rates, if I passively detect a case, on average

It's caused 0.5 infections
I detect p_active_detect * 0.5 next-infections actively
I detect p_passive_detect * (1 - p_active_detect) * 0.5 next-infections because active happens first when it happens

If I don't passively detect a case

It causes on average 1.5 next-infections
I actively detect 0 of these, because I missed this infection
I passively detect p_passive_detect * 1.5

So, the marginal proportion of next-infections passively detected isn't hugely affected by whether or not a particular infection is detected (we lose a small chunk of a small number to active detection). But the reverse is not true, because for every infection we miss, we get a lot more infections we could never actively detect.

Paging @swo because my brain now hurts

I think my first antidote is #14 ; we should have some unit tests for the "do detections" function, and make sure it's doing what we think it should
I think my second antidote is #23 (unless overcome by another issue), so we can visualize what is happening in the simulations!

Agree that those will help make sure that the "problem" is actually just unintuitive emergent behavior.

we can get a passive-passive chain when active detection probability is 100% and active detection delay is 0.0.

I think this is OK? (Unless you meant that passive detection delay were 0?) Two people in a chain can be passively detected after they transmit.

Right. The later infection is actually detected first. Generation-views hurt my brain.

I bet this is related to #53. That bug would (sort of) make active vaccination behave as if it had a random delay from t_exposed that was 1 + Uniform(0, 3), so it would lose out to passive when it shouldn’t.

Agree that #53 likely will solve this
And/or we can use #55 to see what is going on!


[Feature Request]: automatically set GOMEMLIMIT

Is there an existing issue for the same feature request?

[X] I have checked the existing issues.

Is your feature request related to a problem?
Automatically set GOMEMLIMIT to optimize MO memory GC behavior.
Describe the feature you'd like
ref: https://github.com/matrixorigin/matrixone/issues/11225
To control the behavior of Go GC, we need to automatically set GOMEMLIMIT env to container memory limit.
We also need to provide an optional config to tune the default GOMEMLIMIT percent:
kind: CNSet | DNSet | LogSet
spec:
    memoryLimitPercent: 80

If .memoryLimitPercent  configured, the GOMEMLIMIT will be set to .resources.limits.memory * memoryLimitPercent / 100.
Note:

if container memory limit is not set (which is usually a bad practice), GOMEMLIMIT should not be configured.
if .overly.env.GOMEMLIMIT is set, the calculated GOMEMLIMIT will be overwritten to the configured value.

Describe implementation you've considered
No response
Documentation, Adoption, Use Case, Migration Strategy
No response
Additional information
No response

closed via #404


Broken link to Containers in the section See also

The link Containers takes me to the page "welcome-back-to-cpp-modern-cpp".

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

ID: 71492b71-7570-f06d-c905-e78b359b0b8b
Version Independent ID: 6cc78e56-7034-c776-f532-68b8ccf75572
Content: C++ Standard Library Containers
Content Source: docs/standard-library/stl-containers.md
Product: visual-cpp
Technology: cpp-standard-libraries
GitHub Login: @TylerMSFT
Microsoft Alias: twhitney

Fixed. Now links to the parallel containers topic.
Will be live by Monday afternoon (3/30)


cmd option of clab exec does unexpected variable expansion

❯ dclab exec -t private/exec.clab.yml --cmd 'echo ${HOSTNAME}'
INFO[0000] Parsing & checking topology file: exec.clab.yml
INFO[0000] Executed command "echo devbox-u20" on the node "clab-linux-l1". stdout:
devbox-u20

The command echo ${HOSTNAME} should have been executed inside the container, but apparently, the hostname was resolved on the clab host, event though single quotes were used to prevent that.
Needs investigation as to why this happens

this was an issue with vars expansion in my shell


UI: Add deep linking for tabs in Settings page

Signed-off-by: Aayush Dutt<EMAIL_ADDRESS>
Description

Add deep linking in Settings page for environtment, service-mesh, and metrics tabs
This PR fixes #670
Notes to reviewers:
Not sure if this is the best way to achieve this. Reviews/suggestions are welcome.
Checklist

[x] DCO signed.
[x] Title of the PR starts with area name, e.g. meshery:, mesheryctl:, ui:.

Hello! This is exciting. 😄

A good way to put this new feature to use it to change from this:

to this:

A good way to put this new feature to use it to change from this:

to this:

The component used here, doesn't need to be a button.

Good suggestion. I'll put that up in a new issue

Tested. This works!
I don't know that it would be an improvement to dynamically name anchors based on the name of the tabs. I don't think doing so makes the code any less fragile or easy to sustain.

@leecalcote I agree that would be better
Here are some considerations though:

We will need to re-render the page since the names of the tabs will not be known to the constructor.
Current structure of the page is not using name attributes, instead, it is using the imported component and rendering is based on the tab index in state.


Provide unit test for gencerts subcommand

As a maintainer, I want to be able to ensure the gencerts subcommand performs the following behavior:

produces valid and usable certificates
generates the certificates in the expected location
exercises interesting use cases for arguments and their side effects

Note: Follow-up work from https://github.com/OpenBazaar/openbazaar-go/issues/868

This is resolved as part of https://github.com/OpenBazaar/openbazaar-go/pull/1602


Implement @overload decorator for operation overloading

Implementation issue for overloaded consumes and conditional overloads
The default modeling of 'overloaded' operations with mutiple inputs conditionally linked to multiple outputs can be modeled using a basic operation that uses unions, and overloads to represent the conditionality:
model Text is string {
  @Cadl.Http.header("content-type") contentType: "text/plain";
};
model Image is bytes {
  @Cadl.Http.header("content-type") contentType: "application/octet-stream";
}

@route("/doImageOrTextStuffOnly")
@post op doStuff(data: Text | Image, @query option?: string): string | int32;

@overload(doStuff)
op doStuffWithText(data: Text): string;

@overload(doStuff)
op doStuffWithBytes(number: Image): int32

Client generators can use this to generate overloads based on their capabilities [clients might simply implement the base operation, or  might implement each of the overloaded operations using separate names, or the same name].
To support this

[ ] Implement @overload decorator $overload(operation: OperationType) and associated accessors
[ ] Relax checks for multiple operations on the same route when decorated with @overload
[ ] Update OpenAPI3 emitter to ignore the operations marked with @overload

You can't add properties to is bytes and is string like that.
It should be something like:
model Text {
  @header contentType: "text/plain",
  @body body: string;
}


Have you trained the  part of eyes? There is no blink in our results. How can you achieve the performance in your demos.

@Claaudiaaa Hi, we did not model the eyes. We actually do not know what triggers the blinking in our results. However, they do exist with certain video-driven results.


KMP String Algoritm

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
Resolved issue  #2729
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
(Write your answer here.)

@ankitaggarwal23 @HarshCasper @siddharth25pandey @iamrajiv
Please review my PR

@zhcet19 Code copied. Check above


OnEdited method not response

OnEdited method not response
version: gopkg.in/telebot.v3 v3.3.6
    b.Handle(OnEdited, func(c Context) error {
	fmt.Println(c.Message().Text)
	return nil
    })

I have a very similar issue, where b.Handle won't response after 4 times, so example, I ask for firstname, photo, lastname, middlename, the middlename does not gets handle anymore, I don't know if this is related, as I'm have not done further testing but I think there is some issue like a bug.

Please cover this logic with tests or look for the existing one. Try to reproduce the update Telegram sends you and see if it fires the OnEdited handler.

Same here

How to reproduce? Looks working for me with this example
https://github.com/dvordrova/telebotIssue703

I also can't reproduce. It works fine for me


IPMI SOL activate and trigger shutdown -Fr  command from Host

Test steps:

Boot to OS
Start

ipmitool -I lanplus -U root -P 0penBmc sol activate -H xx.xx.xx.xx

Trigger command from host OS partition

shutdown -Fr

Check the Host Rebooted back to partition

Check the SOL connection is still active

@sushmm99 can you check


text-shadow with blur radius is flipped wrong

The flipped version of 0 1px 3px rgba( 0, 0, 0, 0.4 ); is currently text-shadow: 0 -1px 3px rgba( 0, 0, 0, 0.4 );. But this is wrong, it should be the same since the second value is the y-coordinate.
deb10e9 adds two new test cases, will try to fix them.
BTW: text-shadow supports multiple text shadows separated by comma.

It looks like the regexp for colors is too greedy.

Reported as #41. Fixed by #44. Release in v1.1.3.
Thanks!


FPGA: support multiple FPGA devices

FPGA plugin, Admission Webhook and CRI hook currently support only one FPGA device. They should be extended to support multiple devices. This will help to support more complex FPGA workflows that use more than one accelerated function.

I've added support for multiple devices per container to webhook in https://github.com/rojkov/intel-device-plugins-for-kubernetes/tree/multiple-fpgas.
The CRI-O hook is the next step.


Update tangletestnet addresses and add WETH warp route

Description

Updates tangletestnet addresses and metadata.
Adds a new warp route - WETH from holeksy to tangletestnet.

@paulbalaji Below is the core config for new tangletestnet deployments:

I have also updated the warp route to include the correct relayer address:

Since we're using trustedRelayerISM here, I am trying to run just one relayer.

I have few questions:

I want to transfer WETH from holesky to tangletestnet. Do I need to include both the chains when generating the agent config? or just the destination chain? The docs specify --chains chain1 which is confusing.

2 .When trying to generate agent config for --chains tangletestnet I see these warnings:

Could you please tell me how to get this working and if I am missing something here. I am following Deploy Hyperlane with Local Agents  from the docs.

overall LGTM, please run yarn changeset add to add a minor changeset

I want to transfer WETH from holesky to tangletestnet. Do I need to include both the chains when generating the agent config? or just the destination chain?

The relayer needs to have metadata on the chains that it should watch/deliver to, so if you want the relayer to go between chain1/chain2 it should have both in the config

When trying to generate agent config for --chains tangletestnet I see these warnings:

When you deployed the core contracts there should be a merkleTreeHook that was also deployed. There's currently a bug where this doesn't get correctly to the artifacts in some cases. If you add that merkleTreeHook address, it should be able to generate the agent config
I believe this is the one you deployed https://testnet-explorer.tangle.tools/address/0x1d7A5b6e18c16103c365e975B258Cb5039971E9b

I believe this is the one you deployed https://testnet-explorer.tangle.tools/address/0x1d7A5b6e18c16103c365e975B258Cb5039971E9b

Yes that seems correct.
@paulbalaji Thanks for the response here! One more question.
For the transfer to work, does the tangletestnet related config should be in core config of holesky?
hyperlane core read --chain holesky does not include tangletestnet in it's defaultHook, defaultISM domains.

Yup that's right, if you want to use the existing mailbox we have there we'd have to enroll tangletestnet into the default chains there.
You can also deploy your own mailbox/contracts on holesky, and the trusted relayer could relay between your two mailboxes

Okay. So deploy to mailbox and other contracts on holesky, it's the same process right.
hyperlane registry init and hyperlane core init and hyperlane core deploy.
Also when deploying on holesky, can I just do hyperlane core init to keep core config to default or hyperlane core init --advanced ?
I believe after deploying the contracts on holesky, when I generate agent config:
hyperlane registry agent-config --chains holesky,tangletest
I need to make sure the core contracts on holesky points to the addresses I just deployed.
Also, hyperlane core deploy does not deploy interchainGasPaymaster contract. So this will not work right? How will the sender on holesky send the gas fee to paymaster so that relayer can use it on tangletestnet? Is this a bug as well - not able to deploy interchainGasPaymaster with hyperlane core deploy?
Also the docs mention, there needs to be 1 paymaster contracts for 1 relayer. Not sure how to do this.

It should be the same process yes

I need to make sure the core contracts on holesky points to the addresses I just deployed.

yup, if you have your local registry you can use --overrides <pathtoregistry> with the cli to override the default addresses

Also, hyperlane core deploy does not deploy interchainGasPaymaster contract. So this will not work right? How will the sender on holesky send the gas fee to paymaster so that relayer can use it on tangletestnet? Is this a bug as well - not able to deploy interchainGasPaymaster with hyperlane core deploy?
Also the docs mention, there needs to be 1 paymaster contracts for 1 relayer. Not sure how to do this.

Yeah I'm looking into the IGP cli situation at the moment. To unblock you running in the interim, worth trying to set interchainGasPaymaster to the zero address


fix(reposition-scroll-strategy): use last calculated position to re-align the overlay

Uses the element's last calculated position to re-align it, instead of recomputing the position completely. This prevents the overlay from jumping into the viewport when the user scrolls away.

Initially I did it with overlayRef.getState().positionStrategy, but that looked weird, because it assumes that the scroll strategy will only be used on a connected overlay (which will be the case most of the time, but we don't enforce it). Similarly, we have the dipose method which is only relevant for the global strategy, but it has to be implemented everywhere.

Closing in favor of https://github.com/angular/material2/pull/7805.


Strip references to pyviz off project sites

As part of the ongoing rename effort, we need to replace the wording on various project sites to refer to holoviz rather than pyviz:

[x] datashader
[ ] holoviews
[x] geoviews
[x] hvplot
[ ] panel
[ ] colorcet

Does this include renaming sphinx_pyviz_theme to sphinx_holoviz_theme?  I think it ought to, unless it is very painful to do so.

The sites should also each situate the project in the larger holoviz effort, with at least a sentence or two on each site explaining what part of holoviz they are, and linking to both holoviz.org and pyviz.org.

Does this include renaming sphinx_pyviz_theme to sphinx_holoviz_theme?

I just made a separate issue for that one #225

I think this task is done at the github level, though some packages still need to be released before it takes effect publicly.


runtime: Add indent for Protobuf

This adds the indent file for Protobuf from https://github.com/uarun/vim-protobuf/blob/master/indent/proto.vim
I've contacted the original author (Johannes Zellner) and he is ok with it being upstreamed.
Protobuf will fail to build unless the files are indented with 2 spaces, so it makes sense to have this be the default for any vim user editing proto files. Indenting in another way will throw an error like: Found an incorrect indentation style "	". "  " is correct.

So you are submitting a new runtime file on behalf of Johannes Zellner? I'd very much prefer, if whoever volunteers to maintain the file sends it directly here. Can you ask him to at least ACK here then? Please add an entry to the MAINTAINERS file ?
Also this line here:
let b:undo_indent = "setl cin<"

doesn't undo all options set.

So you are submitting a new runtime file on behalf of Johannes Zellner? I'd very much prefer, if whoever volunteers to maintain the file sends it directly here. Can you ask him to at least ACK here then? Please add an entry to the MAINTAINERS file ?
Also this line here:
let b:undo_indent = "setl cin<"

doesn't undo all options set.

I contacted him now to see if he wants to remain as the maintainer, otherwise I'll change it to be myself. I've updated the undo_indent now.

Hi, this is Johannes and I'm happy if David takes over maintainership.
In fact I don't use it since more than 20 years any more ;-)

Hi, this is Johannes and I'm happy if David takes over maintainership. In fact I don't use it since more than 20 years any more ;-)

That settles it, I've updated the maintainer and added myself to the MAINTAINERS file, thank you!

thanks both!


Style transfer - how to access/change parameters

Hi all,
Thanks for the great work.
I have been using the style transfer to make some nice images, however in the latest versions being able to change the style loss in advanced parameters does not seem to work any more. I get the same result regardless of the values i put into the style loss mult.
Please can you give me some guidance on how i can access them - i can't find the documentation for this?
I am using Google Colabs to run it. And this has occurred in every release of turicreate after 6, previous the V6, i can alter the style params.
This is my code
!pip install turicreate
!pip uninstall -y tensorflow
!pip install tensorflow-gpu
import turicreate as tc
tc.config.set_num_gpus(1)
from google.colab import drive
drive.mount('./drive')
import os
os.chdir('drive/My Drive/trainingdata')
style = tc.load_images('style')
content = tc.load_images('content')
params = {
'print_loss_breakdown': True,
'style_loss_mult': [ 1e-2, 1e-2, 1e-2, 1e-2],
'finetune_all_params': True,
}
model = tc.style_transfer.create(style, content, max_iterations = 10000, _advanced_parameters=params)
This runs using the GPU and the style is transferred. But if i try to tweak the params to get more or less style transferred - i get the same result every time.
Please let me know what i am doing wrong
Thanks

Looking at the API docs_advanced_parameters is getting treated as part of the kwargs. Looking at the implementation for tc.style_transfer.create, _advanced_parameters will get added to the params variable. However it doesn't seem like params is ever passed through to the C++ implementation.
So yes, I agree _advanced_parameters is not getting used anymore. I don't think anything has change here since TuriCreate 6.0.
@saddif - did _advanced_parameters work in 6.0 or any version after that?
Related to #3190

hi, i think i mentioned this problem in - "Turicreate 6.2.2 Style Transfer parameters not working? #3190" - the last working version i have tried was V5.8.
I'll try and run some with V6.0 and onwards and report back.
Thanks again for all the hard work on turicreate!
Saddif

Hi Toby,
So i have retried the different versions to see which ones work with advanced parameters
params = {
'print_loss_breakdown': True,
'style_loss_mult': [ 1e-2, 1e-2, 1e-2, 1e-2],
'finetune_all_params': True,
}
model = tc.style_transfer.create(style, content, max_iterations = 5000, _advanced_parameters=params)
v5.8 - runs with GPU
!pip uninstall -y mxnet
!pip install mxnet-cu100==1.4.0.post0
Works as expected
V6.0 runs with GPU - using mxnet (same code as above)
Strange results - looks like advanced parameters are being used since the output is not like 6.2.2 etc
V6.1 runs with GPU - using mxnet (same code as above)
Advanced parameters not accessed - result is same as V6.2.2 etc
V6.2 runs with GPU -
!pip uninstall -y tensorflow
!pip install tensorflow-gpu
Advanced parameters not accessed - result is same as V6.2.2 etc
Hope that helps you - looks like something changed in V6.0 - the results are very strange and very different to V5.8, and V6.1 onwards there is no access to advanced parameters.
Saddif

@TobyRoseman
Do you have an update on this issue? is there a way to access the advanced parameters in the current release?
Thanks

@saddif - no updates.

It looks like the parameters you want are still in the code. There is just no way to modify them using our API. If you want to change those values, you could just change them in the code which is installed on your computer.
In order to do this, first find the location on your computer of the file you need to change. This can be done by running:
print(tc.style_transfer.style_transfer.__file__)
Then open that file and change the parameters you care about.

Hi Toby, Thanks for the reply (sorry about creating another issue about this)
I have tried use Turicreate V6 onwards on Google colabs when trying to use the GPU, and now it doesn't appear to work (before it was working fine - i was not able to change the parameters). Please could you help with some advice?
Google colabs has CUDA 11.2 installed - this stops turicreate from working with the GPU (even if i uninstall tensorflow and install tensorflow_gpu).
I have downgraded CUDA to 10 (i read that turicreate will only work with this?), and uninstalled tensorflow and then installed tensorflow_gpu - 1.5, 1.3 and 1.13.1. When i check for GPU using
import tensorflow as tf
print(tf.test.is_gpu_available())
print(tf.config.list_physical_devices('GPU'))
i get the following result - True but also an error - the same error appears when trying to run a model
True
AttributeError                            Traceback (most recent call last)
 in ()
1 import tensorflow as tf
2 print(tf.test.is_gpu_available())
----> 3 print(tf.config.list_physical_devices('GPU'))
/usr/local/lib/python3.7/dist-packages/tensorflow/python/util/deprecation_wrapper.py in getattr(self, name)
104     if name.startswith('dw'):
105       raise AttributeError('Accessing local variables before they are created.')
--> 106     attr = getattr(self._dw_wrapped_module, name)
107     if (self._dw_warning_count < _PER_MODULE_WARNING_LIMIT and
108         name not in self._dw_deprecated_printed):
AttributeError: module 'tensorflow._api.v1.config' has no attribute 'list_physical_devices'
What do you think i can try next?? What are the exact CUDA, tensorflow_gpu requirements for turicreate 6.4.1?
Thanks in advance! I really like turicreate, but can't afford a GPU-rig myself, hence why i am stuck with using Colabs.

@TobyRoseman hi, sorry I think i posted a follow up comment with more information about the tensorflow issue into the wrong ticket - this one #3190

@TobyRoseman - thanks for the above explanation and to try access the parameters in the .py file directly.
Unfortunately this doesnt seem to work (i have managed to get colabs and gpu working, but the parameters still dont change)- i know it should but not sure why - these are my steps in colabs

Install Turicreate

Check the tensorflow using
import tensorflow
print(tensorflow.version)
2.0.4

!pip uninstall -y tensorflow

!pip install tensorflow-gpu==2.0.4

check location of style transfer file with
print(tc.style_transfer.style_transfer.file)
/usr/local/lib/python3.7/dist-packages/turicreate/toolkits/style_transfer/style_transfer.py

change the params in the above file:
params = {
"batch_size": batch_size,
"vgg16_content_loss_layer": 2,  # conv3_3 layer
"lr": 0.001,
"content_loss_mult": 1.0,
"style_loss_mult": [1e-1, 1e-1, 1e-1, 1e-1],  # conv 1-4 layers
"finetune_all_params": True,
"pretrained_weights": False,
"print_loss_breakdown": False,
"input_shape": (256, 256),
"training_content_loader_type": "stretch",
"use_augmentation": False,
"sequential_image_processing": False,
# Only used if use_augmentaion is True
"aug_resize": 0,
"aug_min_object_covered": 0,
"aug_rand_crop": 0.9,
"aug_rand_pad": 0.9,
"aug_rand_gray": 0.0,
"aug_aspect_ratio": 1.25,
"aug_hue": 0.05,
"aug_brightness": 0.05,
"aug_saturation": 0.05,
"aug_contrast": 0.05,
"aug_horizontal_flip": True,
"aug_area_range": (0.05, 1.5),
"aug_pca_noise": 0.0,
"aug_max_attempts": 20,
"aug_inter_method": 2,
"checkpoint": False,
"checkpoint_prefix": "style_transfer",
"checkpoint_increment": 1000,
}
I closed the file and reopened it to check the paramters had been changed - and they had.
Then all the usual to start the learning
import turicreate as tc
tc.config.set_num_gpus(1)
style = tc.load_images('style')
content = tc.load_images('content')
model = tc.style_transfer.create(style, content, max_iterations = 5000) - i have done various combinations of this to see if i need to call advanced params e.g. definig advanced params in colabs, and adding _advanced_parameters = params to this model run.
The model runs with GPU :
Using 'image' in style_dataset as feature column and using 'image' in content_dataset as feature column
Downloading https://docs-assets.developer.apple.com/turicreate/models/resnet-16.mlmodel
Download completed: /var/tmp/model_cache/resnet-16.mlmodel
Downloading https://docs-assets.developer.apple.com/turicreate/models/vgg16-conv1_1-4_3.mlmodel
Download completed: /var/tmp/model_cache/vgg16-conv1_1-4_3.mlmodel
Using a GPU to create model.
WARNING:tensorflow:From /usr/local/lib/python3.7/dist-packages/tensorflow_core/python/ops/resource_variable_ops.py:1630: calling BaseResourceVariable.init (from tensorflow.python.ops.resource_variable_ops) with constraint is deprecated and will be removed in a future version.
Instructions for updating:
If using Keras pass *_constraint arguments to layers.
but there is no difference to setting style loss to 1e-1 or 1e-4 (the default setting)
What do you think i could try next?
btw i did not change the CUDA and Cudnn - they are the default versions in colabs i.e. 11.0, V11.0.221 and cudnn = 7.65
if i do change them to V10 and cudnn 7.4 i.e. what should work with tensor_flow GPU 2.0.4- in a fresh colabs book - then GPU is not accessed and only works with CPU. Although i dont think this is the issue.
Thanks
Saddif

Unfortunately this doesnt seem to work (i have managed to get colabs and gpu working, but the parameters still dont change)- i know it should but not sure why -

What makes you think it isn't working?

but there is no difference to setting style loss to 1e-1 or 1e-4 (the default setting)

What do you mean by "no difference"? Are you just doing a visual inspection of outputted images or does it actually produce the exact same image? It might be that this hyper-parameter is just not that important.

What do you think i could try next?

I'd encourage you to dig into that part of the TuriCreate codebase.

@TobyRoseman thanks for the reply
Yes i am doing a visual comparison between the two style loss settings - previously on Turicreate 5.8 setting it to 1e-1 for all four values would make a profound difference in the output vs 1e-4. However, with v6.4.1 there is almost zero difference (maybe a pixel change here and there) - hence why i don't think it is doing anything.
But you maybe correct - maybe it is working but with the new tensorflow-gpu there is very little change.
Anyway - i guess i will have to wait for an update that will allow me to change the style loss in a similar fashion to earlier versions.
Do you when there will be an update - or everything shifting to createML (which isn't as good IMHO).
Thanks again for your help and answering all these questions


[feat] Prevent js file use from server configuration object

As discussed in Issue #212, please find here the corresponding pull request.
So, if you agree with adding such feature, you could merge it.

Code is fixed and prettify
Tests are updated
Documentation is updated

Hope I did well according your contributing note and nothing were missed.
Best regards,
Ahry

Thank! Will be looking through this.

Hi there. I've been pretty busy lately. However, I'm not sure about the idea of extending the config object to include functions. There is probably a better way, and I will be playing around with TypeScript project that includes node-fhir-server-core to really investigate this. I appreciate your analysis though!

I may understand your point of view and of course contrary to the sample I provided, people can provide metadata builder function instead of bare metadata object. It also means that we can dynamically create metadata, a functionality that is missing right now. We can only create them statically at server start.
Moreover it's useful when including you're library into a typescript project not to reference JavaScript files, and it's the main point.

@Ahryman40k Do you have a sample TypeScript project somewhere where I can see this fix in action?

I'm writing a typescript library that encapsulate your own as helper to produce FHIR microservices quickly, but it still under development.
Capabilities route already works and metadata are dynamically added.
I try to share you an example next week because I'm currently in holiday until next year.
Thank you for your interest.

Please find a sample there: https://github.com/Ahryman40k/asymmetrik-sample
A README file is present to run sample.
Don't forget it's a POC and not a real development project.

Thanks @Ahryman40k ! I'll be looking at this to really understand the problem.

@zeevosec Have you take a look ?

Would love to see this merged. We are using your great lib in one of our typescript based projects. Would help a lot to maintain a clean code base and setup tests.

Me too ;)

Hey I am going to revisit this and probably merge it.

Do I need to sync with your repo then resubmit my work ?
I know I did it long time ago.

Hello any updated for implemeting typescript to the project ?
Thank you


FormEditorApi controller not found.

Hi. I have installed the latest version from NuGet, and installed it on Umbraco 7.4.3
In the backoffice a get /umbraco/backoffice/FormEditorApi/PropertyEditor/GetAllConditionTypes 404, for all Api call to the /umbraco/backoffice/FormEditorApi
Do you have any idea?

Hey @eriksimonic,
That's weird. Do you see FormEditor.dll in your /bin-folder?

Yes. It is in the bin directory

If i run the application in x64 bit the FormEditor works.. only in may local IIS was a bit troublesome.
So for now I am testing the functionality, it works even trought an Macro witch is great, I can create a reusable form.
I am only missing a Workflow setup, I know that you have a event on witch I can hook may custom logic, But it would be realy neet to have a Workflow interface  "IFormEditorWorkflow", and you could just create a new class and when the data is saved, the workflow is called with the whole model and formdata.. so I could have different processing paths for different forms.. for now we have only 2x options.. and is a bit limiting.
BTW.. did I mention that is quite a nice extension!

Hey @eriksimonic :)
I'm glad you solved it. I'm a bit puzzled, though... I just installed a fresh Umbraco 7.4.3 with the latest FormEditor (<IP_ADDRESS>), all from NuGet, and ran it on IISExpress with no trouble at all. FormEditor isn't built specifically for x64, as far as I know. So... strange.
With respect to workflow - I'd love to hear your ideas, if you have something concrete?

Hi.
The workflows or plugable runtimes or something like this.
I would implement it like this.
In the property editor I would add a new tab, Custum Actions
There I could select a custom developed class that implements and interface ie IFormEditorAction,
public class FormEditorActionParamater
{
public string propertyAlias {get;set;}
public string propertyName {get;set;}
public Bool Required {get;set;}
public object Value {get;set;}
public string BaseType {get;set;}
}
public class FormEditorActionParameters
{
public class FormEditorActionParamater  Parameters  {get;set;}
}
public interface IFormEditorAction
{
void Execute(ref Dictionary<string,object> postedValues, IPublishedContent formContentNode, EditForm formDefinition);
string ActionName {get;}
FormEditorActionParameters ActionParameters {get;}
}
With this construct it would be posible to populate custum action required parameters, when the action is selected, and with a search for instance of we could find all avalable action classes..
Does make it any sense?


Get-SPOExternalUser : Could not load type 'Microsoft.SharePoint.Administration.CentralAssetRepository' from assembly

Hi
First thing I did is download and install the latest latest SharePoint Online Managment Shell :
16.0.9119.1200
`
connect-SPOService -ul 'https://tenant-admin.sharepoint.com'
Get-SPOExternalUser -PageSize 3
`

I have tried rebooting but there is still an issue - oh and it freezes the ISE when I get this error.

@officedocsbot assign @yogkumgit

Thanks for getting back to me .  Can you confirm you tried to reproduce the scenario I showed above both a first release and a UK based live tenant? I am not so sure support can help here given the module loads in a PS 5.3 shell  and I can connect to the tenant admin url .

Hi @get-itips  ,  Now raised a support request on this . Thanks for the advice.

Hi,
Is this issue fixed now?
I found no solution to automatically remove guests accesses on sites collection.
Thanks


deps: update to SpiderMonkey 124.0.2

Updates to SpiderMonkey 124.0.2 per https://github.com/bytecodealliance/gecko-dev/pull/43 and https://github.com/bytecodealliance/spidermonkey-wasi-embedding/pull/11.

Updating the Rust toolchain got the build to pass correctly here.


config.yaml should provide default model location across commands

I suspect config.yaml in offering very specific, different path possibilities for model locations on the host (generate, serve, chat) might offer too fine-grained a configuration.
I have models on my system that I'd already downloaded in /srv/models.... I didn't realize there were three model paths in the config.yaml to start so I got a traceback when i tried ilab serve and it couldn't find the model in the default model path.
Probably, under general: - have a path to a directory where models are stored that's more global, and just put the name of the model or the subpath in the individual commands' config.
Only sticky point is perhaps if you want a mix of local and remote models.... but then you could override the local model dir (which could be thusly named, local_model_dir) with a url to an endpoint in the individual commands' config areas. eg
general:
  local_model_dir: /srv/models
chat:
  model: 'Mistral AI'/mixtral-8x7b-instruct.gguf
generate:
  model: https://blahblah.endpoint/api:1234

Note: filing this against 0.17.0 on Fedora 39


Cleanup .fseventsd before unmounting

Currently the output DMG contains a .fseventsd directory, which a read-only image doesn't really need. Maybe it would make sense to delete it before unmounting the rw image? (Unless it can break something that I'm not aware of.)

Doesn't that folder recreates again before unmounting?  If not, I think this is a good cleanup.

@aonez In my tests it works - there's no .fseventsd in the output image. (I'm on macOS 10.15.7)


Duplication Files/Folders should work same as Nodes

Godot version:
3.1 2e87703
OS/device including version:
Windows 10
Issue description:
When I try to duplicate Node I must only choose one or more and click CTRL + D.
Godot add to original name some numbers. I can duplicate as much as I want.
When I try to duplicate File or Folder in File System Dock I must choose only one(If I choose more, only last selected will duplicate) and when I click CTRL + D, I must write the proper name because by default Godot doesn't change name in this popup. When I click Duplicate button, editor lost focus on File System Dock and now point at the some Node from Scene Dock. If I wanna duplicate this file more time, I must again click at the desired file and click CTRL + D. It works a worst than in Scene Dock

Somewhat related to #20641

Based on @akien-mga's closing comment on #20641, is this something that should be done? That comment specifically calls out that the dialog is desired, whereas this issue would suggest removing the dialog in place of auto-incrementing names. There is a bit of a conflict between those two methodologies for the browser.

Reading the last comment in #20641, it seems the feature's current behavior is intended and won't be changed, so I'll close this.


Fails to compile with Intel MPI

MPI_status variable conflicts with intel mpi library.

Possible fixed in https://github.com/ORNL-Fusion/SIESTA/pull/7


Warning about same key in console upon rendering tasks

I get a warning in console says that task components have same id

Fixed it already


stam2

run tests

Pull Request Test Coverage Report for Build 32

8 of 10 (80.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 97.677%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

accounts/views.py
8
10
80.0%

Totals

Change from base Build 19:
-0.2%

Covered Lines:
799

Relevant Lines:
818

💛  - Coveralls


Add support for connecting ACR registry during AKS create

This PR adds the ability to connect an ACR registry to an AKS cluster at cluster creation time by performing the service principal role assignment implicitly. Two new optional properties are added to the create command:

acrName: the name of the registry to be attached
acrResourceGroup: the name of the resource group where the ACR can be found. If not specified, it is assumed to be the same as the RG where the AKS cluster is being created.

This functionality has now been added in #891


perlPackages.NetPrometheus: init at 0.07 (and StructDumb at 0.09)

Motivation for this change
Adding these packages to add support for Prometheus in Hydra. Should be backported so Hydra can make use of them.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

bot-generated code is a bit simpler:
  NetPrometheus = buildPerlModule {
    pname = "Net-Prometheus";
    version = "0.07";
    src = fetchurl {
      url = mirror://cpan/authors/id/P/PE/PEVANS/Net-Prometheus-0.07.tar.gz;
      sha256 = "1dh498b26wdaip053hw52317jjmb2n2r5209a1zv5yfrlxpblqm7";
    };
    propagatedBuildInputs = [ RefUtil StructDumb ];
    buildInputs = [ TestFatal ];
    meta = {
      description = "export monitoring metrics for F<prometheus>";
      license = with stdenv.lib.licenses; [ artistic1 gpl1Plus ];
    };
  };

  StructDumb = buildPerlModule {
    pname = "Struct-Dumb";
    version = "0.09";
    src = fetchurl {
      url = mirror://cpan/authors/id/P/PE/PEVANS/Struct-Dumb-0.09.tar.gz;
      sha256 = "0g9rziaqxkm00vh30g1yfwzq3b1xl23p8fbm4rszqsp641wr2z9k";
    };
    buildInputs = [ TestFatal ];
    meta = {
      description = "make simple lightweight record-like structures";
      license = with stdenv.lib.licenses; [ artistic1 gpl1Plus ];
    };
  };

I think Po4a could be simplified too, bot ignored it because it is not a CPAN package.

Updated, nice catch, thank you!


Edgetpu link error in Cmake build

Description
I am trying to incorporate edgetpu functionality into an existing application which already has it's own build system that uses Cmake. I am getting a link error for 'rdft' undefined reference. It's not clear if this is a build issue, missing library, or an issue with specific version of tensorflow or edgetpu from another issue solution to the 'rdft' undefined reference.
/usr/bin/ld: /usr/local/lib/libtensorflow-lite.a(spectrogram.cc.o): undefined reference to symbol 'rdft'
/usr/bin/ld: /usr/local/lib//libtensorflow_framework.so.2: error adding symbols: DSO missing from command line

Click to expand!
Issue Type
Build/Install
Operating System
Ubuntu
Coral Device
M.2 Accelerator B+M
Other Devices
No response
Programming Language
C++
Relevant Log Output
/usr/bin/ld: /usr/local/lib/libtensorflow-lite.a(spectrogram.cc.o): undefined reference to symbol 'rdft'
/usr/bin/ld: /usr/local/lib//libtensorflow_framework.so.2: error adding symbols: DSO missing from command line

/usr/bin/ld: /usr/local/lib/libtensorflow-lite.a(spectrogram.cc.o): undefined reference to symbol 'rdft'
/usr/bin/ld: /usr/local/lib//libtensorflow_framework.so.2: error adding symbols: DSO missing from command line

Looks like the issue is when building the libtensorflow-lite.a. Please try to build it without any edgetpu code and check with tenosrflow team team with all the reproducable steps.. Thanks!


question: why is ScreenUtil.init obtaining a context from Navigator?

I'm working on fixing a bug within the widgetbook package.
In Widgetbook, we have two navigators. One for navigating through multiple of the catalogued widgets and one navigator for previewing/rendering the selected use-case.
When previewing using a Widget that uses the flutter_screenutil package, the code 200.h returned the correct height. However, whenever the UI is refreshed (for example by calling setState()), 200.h does not longer return the correct height.
I debugged into the code of ScreenUtil.init (see this code) and realized something that I'd consider odd.
The lines
final navigatorContext = Navigator.maybeOf(context)?.context as Element?;
final mediaQueryContext =
    navigatorContext?.getElementForInheritedWidgetOfExactType<MediaQuery>();

obtain a new context.
However, based on how ScreenUtil is used, the context that is obtained might not be the one that contains the inherited MediaQuery causing 200.h to return a different value.
I changed the source code of flutter_screenutil to
final mediaQueryContext =
    context?.getElementForInheritedWidgetOfExactType<MediaQuery>();

With this code, the MediaQuery of the context is the one I'm expecting.
Question:
What's the reason to first fetch the Navigator and extract another (possibly different) context from there?

hello, i'm going to rethink the context here


Hide ChoiceResult and Dtmf Result, to use RecognizeResult Only

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.Communication.CallAutomation


Cscfairmeta 100 add a button to get the dataset as a file

Added a button for data download in different formats

This will be reviewed once Metax-test is back online


Fix coverage workflow

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

Comparison is base (3707fc5) 100.00% compared to head (e050c4f) 100.00%.
Report is 28 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #320   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           82        87    +5
  Lines         2253      2281   +28
  Branches       245       209   -36
=========================================
+ Hits          2253      2281   +28

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


Missing output

Can you explain why some commands, such as Install-Module -Name SqlServer -Scope CurrentUser -Force, display no output in the logs, even with system.debug enabled? I have to append -Verbose sometimes to see the output, but that is different than the behavior I'm seeing locally.
Locally, $VerbosePreference is set to SilentlyContinue (same as what I'm seeing in Azure DevOps).

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

ID: a0e444c2-69c6-4d71-6fd0-94d0f4097d3a
Version Independent ID: 3a727833-1cc3-c2fb-c5b3-22605eee2721
Content: PowerShell task - Azure Pipelines
Content Source: docs/pipelines/tasks/utility/powershell.md
Product: devops
Technology: devops-cicd
GitHub Login: @vtbassmatt
Microsoft Alias: macoope

Thanks for your question. It looks like your question isn't an issue with the documentation, but an issue or question specific to your implementation or a bug in the product itself.
For help with a specific task, here are a couple of options where you might consider asking your question:

Azure DevOps on Stack Overflow
Azure DevOps Support Bot

If you've encountered an issue with the product itself, to make sure it gets in front of the right people, please submit your bug here.
If you have an idea for improving the product, please submit your idea here.


Show warning for mismatched repo URLs

Description
Updates the preview sticky header to render a warning if the wheel's code repository doesn't match the action repository.
Demos
With matching repos
https://napari-hub-preview-matching-repos.vercel.app

With mismatched repos
https://napari-hub-preview-mismatch-repo.vercel.app

Added @neuromusic @liaprins-czi @DragaDoncila for review of the label and either @kne42, @klai95 or @justinelarsen for code review 😄

Thanks for the feedback @liaprins-czi I've updated the text to match the styling now:
https://napari-hub-preview-mismatch-repo.vercel.app

oops, didn't realize it was missing. added!

@codemonkey800 here is the link to the relevant section in the doc, now that it's been merged: https://github.com/chanzuckerberg/napari-hub/blob/main/docs/setting-up-preview.md#seeing-a-warning-message

@codemonkey800 here is the link to the relevant section in the doc, now that it's been merged: https://github.com/chanzuckerberg/napari-hub/blob/main/docs/setting-up-preview.md#seeing-a-warning-message

Awesome, thank you @DragaDoncila!!


Oksuzian082624a

Hi @oksuzian,
You have proposed changes to files in these packages:

data
Scripts
ProjPy

which require these tests: build.
@Mu2e/write, @Mu2e/fnalbuild-users have access to CI actions on main.
:hourglass: The following tests have been triggered for ce6e1d94e4a0d9f263850bf21b7157095ee2fff6: build  (Build queue is empty)
About FNALbuild. Code review on Mu2e/Offline.

approved, please address the blank line if you can before tagging.


Bump Marathon to 1.9.100

High-level description
What features does this change enable? What bugs does this change fix?
Bump Marathon to 1.9.100. Fixes blocked issue MARTHON-8706. Also includes some performance improvements and a fix for external volumes.
Corresponding DC/OS tickets (required)
None (this just bumps master to the version contained in 2.0)
Related tickets (optional)
Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[ ] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain.
[ ] Test Results: [link to CI job test results for component]
[ ] Code Coverage (if available): [link to code coverage report]

@mesosphere-mergebot changelog-not-required this is just bumping to the Marathon version contained in 2.0

@mesosphere-mergebot label Ready For Review

@mesosphere-mergebot changelog-not-required this is just bumping to the Marathon version contained in 2.0

@mesosphere-mergebot bump-ee

@mesosphere-mergebot sync

@mesosphere-mergebot sync


Display the sleep type on sleep

Description of the new feature / enhancement
As you published the new sleep feature i checked my system for it's settings and horrified to find out that my efforts to guard the health of my ssd ... using sleep ... has turned to be the opposite thing since hybrid sleep was turned on .
Please Display a screen notice on screen when sleep happens : " Sleep " or " hybrid sleep " and add these setting to the power toys , since it's hidden in the power settings .
There is no way to tell that your pc goes to hybrid sleep instead of sleep with these fast nvme's .
Scenario when this would be used?
Sleep
Supporting information
No response

@dend I unfortunately don't have much background in this area. For context, is there a reason why hybrid sleep is turned on instead of keeping this user's previous sleep setting?

The app should not in any capacity influence the computer's sleep settings. Awake does nothing to the computer's power settings/profile.
@byshai - can you please clarify the steps to reproduce your hybrid sleep observation?

@byshai - still cannot repro on my end. Can you please provide steps to reproduce? Without this information, we'll close the issue until additional information is available.

This is a feature request ! not an issue .

@byshai OK, so if I am reading this correctly - you want to see the type of sleep your computer will go to based on the power settings?

Yes

@dend IMO there is a bug with Windows System command “sleep” executing via PowerToys Run. For some reason it turn off the computer. Is there possibility to switch from Hibernate to regular Sleep?

@byshai there are two separate commands Hibernate and Sleep. It seems that Sleep command’s behavior is broken

I suspect this ticket is the duplicate of #10557

Closing this as it's not directly related to Awake.


Download as PDF

Is your feature request related to a problem? Please describe.
Download the resume as PDF.
Describe the solution you'd like
Download the resume in same design in PDF

Hey, I'd like to give it a shot. Can you assign it to me?

@sabesansathananthan Hey I have added the functionality as you asked. Please review and tell any requested changes.


Compatibility with Three.js clipping planes

Description of the bug
Hi, we are the creators of the IFCLoader of Three.js. We are developing a framework to develop BIM applications in the browser.
We have just started to use this amazing library to be able to have a more realistic feel. It works perfectly: the only problem is that when we use clipping planes, the ambient occlusion of the model doesn't go away.
Is there a way to make the clipping planes work together with the AO post processing?
To Reproduce
Use clipping planes in an object that has ambient occlusion applied.
Expected behavior
The AO of the clipped parts shouldn't be rendered.
Screenshots
In the following image you can see a model cut horizontally, and how the ambient occlusion of the whole building is still there:

Library versions used

Three: 0.132.2
Post Processing: 6.23.1

Desktop

OS: Windows 10
Chrome

Hi,
it's possible that the NormalPass doesn't see your clipping planes. Are you using local clipping planes? If so, does AO look correct if you use global clipping planes instead?

We're using local clipping. Global clipping works actually, but we're trying to avoid it since it will limit what we can do. Is this something we can solve with local clipping?

MRT seems the best way to go for this and for other advanced postfx. :)

Jumping on the bandwagon here, would you happen @vanruesc to have that Normal Pass computed from Depth available somewhere? It would be helpful for scenes with no normals - for example where geometry primitives are written to depth via custom shaders, like pointclouds. If this could be added to the standard set of Pass of the postprocessing lib - with its set of limitations - that would be nice! Or one could write its own ShaderPass which would use depth as input and do the differentiation on the shader-side.

@jo-chemla That pass hasn't been implemented for postprocessing yet. There are also currently no plans on adding it  since v7 supports rendering normals via MRT.

It would be helpful for scenes with no normals - for example where geometry primitives are written to depth via custom shaders, like pointclouds.

I think this is a more complex topic because it touches on transparency, the geometry used to render a single "point" and the usefulness of the rendered normals.
If the points are simple planes then rendering the normals in the vertex shader would be much easier than calculating finite depth differences. However, those flat normals wouldn't be very useful. Transparent objects are also drawn on top of other things which creates conflicting normals for the affected pixels. To avoid this, normals are usually only rendered for opaque meshes.

Thanks for getting back! We do render opaque geometry and have a use for normals computed screen space - for example this video  is rendered purely from a massive pointcloud where points are 1px texels (no per-vertex normals, so the texels are not oriented along the normal but simply face the camera) and it has lighting computed from screen-space normals.
I did implement this ScreenSpace Normals Effect following this simple example - can do a PR if you think this can be useful, although with some minor artifacts.
And good luck with v7, a lot of promising features seem upcoming!


fix(deps-peer): allow usage of<EMAIL_ADDRESS>

Allows usage of<EMAIL_ADDRESS>

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


error: VerifyContext has a problem

Describe the bug
The isScam is false, but the VerifyContext will be .scam.
A clear and concise description of what the bug is.

SDK Version

Client: Swift
Version 1.9.8

To Reproduce
Steps to reproduce the behavior:

Go to https://pancakeswap.finance
Click on 'Connect Wallet'

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Device (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

should change guard isScam == nil else { to guard isScam ?? false else {

And I scan opensea.io and https://uniswap.org/ always show .unknown. and the api https://verify.walletconnect.com/attestation/267c86f8624562d268a5ddb29ec439f7f05b84e0a03c216490cff10d5cb61d2c is always 404.

hey @xwal, please upgrade to 1.9.9

I have the same issue.
response from https://api.blowfish.xyz/v0/domains
{
"domain": "uniswap.org",
"status": "PROCESSED",
"riskScore": 0.0,
"labels": [
"source=blowfish_whitelist",
"source=brand_whitelist",
"source=defi_llama_whitelist"
],
"warnings": []
},
Why the verifyContext still showing unknown? @llbartekll

@wongzigii did you upgrade to 1.9.9?

@wongzigii did you upgrade to 1.9.9?

I can't upgrade to 1.9.9. When I upgrade to 1.9.9, the connection failed.

pairClientWeb socket is not connected to any URL.

And I scan opensea.io and https://uniswap.org/ always show .unknown. and the api https://verify.walletconnect.com/attestation/267c86f8624562d268a5ddb29ec439f7f05b84e0a03c216490cff10d5cb61d2c is always 404.

I have the same problem. Did you upgrade to 1.9.9 finally?

Yes, I have upgraded.

And I scan opensea.io and https://uniswap.org/ always show .unknown. and the api https://verify.walletconnect.com/attestation/267c86f8624562d268a5ddb29ec439f7f05b84e0a03c216490cff10d5cb61d2c is always 404.

I have the same problem. Did you upgrade to 1.9.9 finally? @xwal I am not sure is this releated to the domain pollution. #1009

And for unknown case, I have ignored it. Some wallets e.g. Rainbow ignore the "Unverified" state - up to you @wongzigii


use components which was installed globally for createDiscreteApi

https://github.com/zhaojh329/oui/commit/c6f58a3d82d2302d080476eb953a21dfee307565
There are one problem remains that theme and internationalization have not been synced.
Fortunately, naive-ui will expose vue app created by createDiscreteApi in the next version.

https://github.com/zhaojh329/oui/commit/4b0124c26d8251e8ce39dfae2623a8bb118144e5


Bluetooth: BAP: BA: Replace bools with atomic

Replace the boolean values for the BAP Broadcast Assistant with an atomic value. This prevents a rare, but possible, race condition.

Added DNM until I spend additional time verifying that atomic values is actually a proper solution (seems not) and whether it needs to be replaced with or accompanied by a mutex.


[MM-25658] Replace Data Grid on Channel and Teams page in System console with Common Component

Summary

Replace the data grids on Teams and Channels page with the new common component created by the wonderful @fmunshi .
Ticket Link

https://mattermost.atlassian.net/browse/MM-25658
Screenshots

/cla-update

/cla-check

/check-cla

Just noticed an issue - previously clicking on any team / channel row would take you to the page for that team / channel... I think we're gonna want this to behave the same way, so you're going to have to setup a way to add an onclick handler to a full datagrid row
I think refactoring the datagrid Row type to be
export type Row = {
    cells: {
        [key: string]: JSX.Element | string;
    }
    onClick?: () => void;
}

this would then require changing existing implementations to work with this new type too.

A "Quick" explanation for css naming:
Given this example HTML:
<button class="Button Button___primary">

    Click Me!

</button>

A component has three naming considerations as follows:

The component itself representing the Block in BEM
The components child elements representing the Element in BEM
The components modifiers representing the Modifier in BEM

Blocks:
There are technically 3 Blocks above – the outer button and the two inner icons. The icons are Blocks as they are also components that are imported into the button component. So, the Block class names are:

.Button for the Button component
.Icon for the Icon components

Block names use UpperCamelCase (also known as PascalCase).
Generally speaking, Block classes are used to visually style the component and its child elements and not define any placement styles within its parent container.
Elements:
There are 3 Elements above – the two icon Elements and the span tag containing the button's text. While the icons are components and therefore are Blocks in their own right, they are also Elements from the buttons perspective, so they receive Element class names as well:

.Button_leftIcon
.Button_label
.Button_rightIcon

Element names start with the Block name and add the Element name using lowerCamelCase (also know as just camelCase) separated by a single underscore. Element class names are only really necessary if you need to modify the Element's styles in the context of the parent Block.
Elements within a parent block that are components should be styled from the Blocks perspective using the Element class name (i.e. .Button_leftIcon) and not the Element's component name (i.e..Icon).
Generally speaking, Element classes are used to position the element within the Block and not strictly to apply visual styling. Though visual styling can be applied if it doesn't make sense to support the needed styling within a child component's default styles or if the Element doesn't necessarily warrant the creation of a separate component.
Modifiers:
The overall button and the two icons above have modifiers applied:

.Button___primary on the button to override the generic button with primary styling
.Icon___small on the two icons to override the generic icon size to make them small icons

Note that this modifier name is from the perspective of the icon and not specifically the button because the icon already supports the size the button needs. If you needed an icon size within the button that the icon didn't already support and doesn't make sense to add to the icon component directly, you could use an element modifier class name like .Button_rightIcon___superExtraSmall. This approach should be used sparingly though!

Modifier names start with the Block name and add the Modifier name using lowerCamelCase (also know as just camelCase) separated by three  underscores to help visually distinguish them from Element names that use a single underscore.
Modifiers can also be used for component state changes like .Button___toggled when a button is toggled to apply different visual styling when that state is triggered. That said, I also personally like to use HTML attributes to indicate state changes as some html elements already have state attributes. Eg. [disabled] or [data-toggled="true"] or ...
(Been meaning to write this as a general draft doc for CSS, so will copy from here)

thanks for the explanation @deanwhillier

@hahmadia I found your PR! I'd like to follow the mattermost repo more but I've a day job. I may be peeking around from time to time. "See" you around!

@hahmadia I found your PR! I'd like to follow the mattermost repo more but I've a day job. I may be peeking around from time to time. "See" you around!

@clarmso
Welcome!! So cool to see you around! Consider joining www.community.mattermost.com . The entire Mattermost team is active (including myself) over their  and we are always happy to help out if you ever come across issues or problems.

@deanwhillier can you please review the PR & confirm if the changes are good from code perspective. Will pickup the QA once it's done. Don't want to duplicate QA efforts if there are any new changes based on your review.

/update-branch

/update-branch

MM-26568 - Resolved

@srkgupta talked to @hahmadia over zoom, determined all three need real quick fixes, we talked through the css changes should be pretty quick for him :)

@srkgupta talked to @hahmadia over zoom, determined all three need real quick fixes, we talked through the css changes should be pretty quick for him :)

Thanks @michaelgamble

Addressed all problems @srkgupta


docs: added ic-menu documentation to select and search bar code mdx

Summary of the changes
Added ic-menu documentation to select and search bar code mdx
Related issue
#177
Checklist

[Y] I have manually accessibility tested any changes, if relevant.
[Y] I have raised this pull request against the develop branch

closed as docs for ic-menu not required as users do not use component directly


Add information about userns-remap

Add details about userns-remap
Fixes #1366
@estesp PTAL

@pvnovarese PTAL

Addressed great feedback, split this off into its own topic and made some "What next" links in the configuration index, one of them pointing to the security landing page.

Please review at https://deploy-preview-3678--docker-docs.netlify.com/engine/security/userns-remap/

Thanks @mstanleyjones! We can chat about multiple ranges.. its a fairly advanced topic with probably a very small set of potential users.

I'm one of those potential users, it would be great to get more info and further tested examples about the multiple ranges topic.

@fabiomartinelli can you tell us more about your use  case for using multiple ranges?