Enhanced caching

Hi AdmiringWorm.
This pull request enables caching of arrays generated splitting collection-resources (i.e. resources like "Alpha,Bravo,Charlie,Delta,...". Instead of performing Split() each time a collection-resource is accessed, it is executed only once, on first access. Afterwards, the cached array is returned, so reducing complexity from O(n) to O(1), being n the cardinality of the collection-resource. To give an idea of performance improvement, generation of 100.000 first names takes 18.563msec without caching and 214msec with caching.
For the sake of preserving coherence across code, caching has been applied to all collection-resources, even to the short ones.
I did my best to design code so to preserve strongly-typing, thus avoiding magic strings.
This version has a limitation. When multiple unsupported cultures are used, an array is cached once for each culture, even if the fallback mechanics return the same neutral resource. Unfortunately, .net core seems not to allow to programmatically retrieve the list of supported resources. So, cache key fallback is not straightforward.
The class ResourceCollectionCacher, which holds the cache, is thread-safe.
Some tests have been written to test the functionality, even in the case of runtime culture switches.

Make sure Unit Tests runs successfully before open up PR's in the future please.

My tests did entirely pass, if you intended the nunit tests. It's not clear to me why appveyor and travis-ci show that red cross. Let me know if I can make anything to solve this issue. Thanks a lot for the review.

this is the reason they fail: br-BR is an invalid culture identifier
I also ran those on my local computer with the same result

Sorry for the mistake. Oddly, tests passed with br-BR on my machine. Now I used ru-RU.

@supix It could be that somehow a custom culture have been installed on your computer, which
make that culture code valid. Not sure though.

Current coverage is 93.78% (diff: 95.38%)

Merging #26 into develop will increase coverage by 0.04%

@@            develop        #26   diff @@
==========================================
  Files            19         21     +2
  Lines           367        386    +19
  Methods         140        145     +5
  Messages          0          0
  Branches         49         51     +2
==========================================
+ Hits            344        362    +18
- Misses           16         17     +1
  Partials          7          7

Powered by Codecov. Last update 0870eb1...51641ce

If you are happy with this pull request, I'm happy with merging it now

Thanks a lot AdmiringWorm.

@supix no problem.
I don't know when I'm going to push out a new release though,
but in the meantime if you have a need for it I have published it to my myget feed.
https://www.myget.org/F/wormie-nugets/api/v2.
It is currently under a pre release version: 2.4.0-beta1

@AdmiringWorm Nice. Thanks.
Downloaded, but get an error from NuGet. It's not important. I can arrange anyway. But for your information, here is the error message.
Install failed. Rolling back...
Could not install package 'Faker.Net.Portable 2.4.0-beta1'. You are trying to install this package into a project that targets '.NETFramework,Version=v4.5.1', but the package does not contain any assembly references or content files that are compatible with that framework. For more information, contact the package author.

Trying to add manually the reference to the compiled assembly does work.

a fuck, I forgot about that. I think nuget changed how they handle portable libraries, specifically how the
specify wich framework they support.

@supix I repackaged the nuget package, could you please try updating
to beta2 and see if it works now?

It is correctly installed now. But it takes about 20 seconds to generate 100.000 names. As if the caching was disabled. :-)

Ah, hmm. I must have messed up something. Was working in several branches so most likely messed something up there. Give me a moment and let me check what happened

ah, yes as I suspected. Was working in a different branch wich hadn't been updated yet.
beta3 should now be the correct one.

Works! Fast! 👍


Use namespacing for globals

Javascript best practice is to use namespacing rather than globals (recommended by Mary at https://github.com/USGS-VIZLAB/water-use-15/pull/125#discussion_r179135637).
Namespacing appears to be easier than I thought - it can simply mean defining a single global (an object literal, i.e., a curly-braced object with named elements, e.g., myObject = { a : 1, b : "hello" }) and adding any desired global information to that single global object. (https://addyosmani.com/blog/essential-js-namespacing/#beginners, http://www.dyn-web.com/tutorials/object-literal/)
I didn't understand why people would call this namespacing instead of "only using one global variable", but Andy pointed out that the dot notation works for both packages and objects (e.g., d3.scale, myVizzyNamespace.data), so why not use it as a local namespace as well? Seems plenty practical as a way to lower the odds of mistaking a global for a local or having an outside package clobber one of our variables.
Immediately-invoked Function Expressions (IIFE)s can be used to generate all or part of a namespace global variable, but I don't think we'll have a need in this particular vizzy. See https://addyosmani.com/blog/essential-js-namespacing/#beginners for that idea and other ways to get fancy if and when we need to.
Our vizzy currently has a number of global variables - state and county boundary datasets, flags for whether circles have been created or not, etc. - that should use namespacing.

this was done already, right? waterUseViz.stuff?

I'd vote we close this issue. We'll do better with the next one, but we got the major ones.

Well, we have a bunch of things in the backlog (this included). I'm not sure we are going to close them. I think we will just leave them sitting in that swim lane. If it stresses you out to see so many open issues (it does for me), never look at the Issues view, and go to our project view instead (https://github.com/USGS-VIZLAB/water-use-15/projects/1). This is really where we've been tracking what needs to happen.

good point, I'm just doing a sweep of what I think I could help with. I could do this...but I just don't see it worth my time right now.


Unclear explanation for Stream Analytics job query

Step 1 of  "Run the Stream Analytics job" is unclear.  It should make clear that the query should be changed to:
SELECT *
INTO
[adlsg1output]
FROM
[blobinput]

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

ID: 0ce1146c-b5b4-3413-6d7c-d0065cbb263c
Version Independent ID: 26781dc5-918b-9c26-8e2f-76396f74f155
Content: Stream data from Stream Analytics into Azure Data Lake Storage Gen1
Content Source: articles/data-lake-store/data-lake-store-stream-analytics.md
Service: data-lake-store
GitHub Login: @twooley
Microsoft Alias: twooley

Thanks for the feedback! We are currently investigating and will update you shortly.

@pugillum When you click on the query, it gives the default query to understand the query pattern.

As mentioned in the document, “you can run the sample query by replacing the placeholders with the job input and output aliases”.

Note: You can modify the query according to your requirement.
Need help with your query? Check out some of the most common Stream Analytics query patterns here.
Hope this helps.

Hi, thanks for the explanation, I did manage to get it working.  The document indeed mentions "you can run the sample query by replacing the placeholders with the job input and output aliases" but that isn't reflected in the image:

@pugillum The image shown in the article is default query pattern.
Note: You can modify the query according to your requirement.
We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


Is there an explanation as to why DataContract attribute from base class is inherited?

This is contrary to the expected and defined behavior of DataContract attribute.
http://json.codeplex.com/discussions/357850

I didn't know that behavior when I wrote it.


Feature/#330 zakano algorithm

PR for issue #330 .
Ready for a PR?

[x] I have updated the CHANGELOG with an entry including a description, a link to my profile, and a link to the issue ticket. This change is filed under an Enhancement or Bug Fix, is shown under the relevant release tag name, and is included in my PR branch.

Implementation Notes:
Pulled in serializable-fn to source the zakano code as a string.
Breaking changes:
None

@emilyseibert @dehli Added additional logging to capture runtime errors during Zakano code execution.


Wrong indication on page Create certificates to allow the backend with Azure Application Gateway

In the section Create certificates to allow the backend with Azure Application Gateway it is said in Step 1:

Use the steps 1-9 mentioned in the section Export authentication certificate from a backend certificate (for v1 SKU) above to export the public key from your backend certificate.

But the referenced section has only 8 steps, it does not have a 9th step.

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

ID: 5813d794-d0f4-24dd-077f-e640282329d2
Version Independent ID: fbbc4e6b-6565-4531-f8a7-4e21011e4636
Content: Certificates required to allow backend servers - Azure Application Gateway
Content Source: articles/application-gateway/certificates-for-backend-authentication.md
Service: application-gateway
GitHub Login: @vhorne
Microsoft Alias: absha

@coxantohi Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

fixed
#please-close


[Question] How to use the overridings ?

Hello @ysc3839, thanks for this wonderful tool. :+1:
I'm a non coder, I'm just a regular user. I would like know to how to use these overridings ? it doesn't work as said here: Remove '#' to override font style.
of course the Mod is perfectly working on the latest version of Telegram Desktop (2.4.4), just this override thing, it doesn't work. :(
Can you please show me how ? write me an example please.
Thanks in advance.
(Remove '#' to override font style
size: 0
width: 0
weight: 0
italic: false
underLine: false
strikeOut: false
charSet: 0
outPrecision: 0
clipPrecision: 0
quality: 0
pitchAndFamily: 0)

What's your full config file?

FontMod.yaml.txt
(I've just added .txt extension because GitHub refused to upload it in .yaml extension.)

The style: &style is duplicated in your config. Keep only one.
And if you want to change size, just remove the '#' before the size.
For example, before is # size: 0, after is size: 20 (20 is the size number).

The style: &style is duplicated in your config. Keep only one.
And if you want to change size, just remove the '#' before the size.
For example, before is # size: 0, after is size: 20 (20 is the size number).

Thank you very much. Can't we make it for only one font ? Like, for example, if I want to change the size of this:
  DAOpenSansRegular:
    replace: Segoe UI
    <<: *style

Can't I ?

Can't we make it for only one font?

You can. Just write the size in that font.
For example:
  DAOpenSansRegular:
    replace: Segoe UI
    size: 20

The <<: *style just like copy paste the contents of style into the font.

Thanks. :)
I shall close this now.


[Enhancement Request] Use Llama 2 or other open source LLM for AI backend?

Describe the Enhancement
Llama 2 is a very powerful LLM that also happens to be free and open source. It would make a very good backend for the generative features. Other alternatives good alternatives could be OpenLlama or GPT4All
Describe the current behavior
AI isnt real
Describe alternatives to your enhancement
No response
Screenshots
No response
Additional Context
No response

We're likely going to use [https://www.langchain.com](Lang Chain) which should ideally let us do exactly that!


Unset from object property array not working

Unfortunately unset from array is not working.
Sample function code:
var_dump(this->_data);
unset this->_data["key_a"];
var_dump(this->_data);

compiled C code:
zval *key_param = NULL, _0, _1, _2;
zval key;
ZEPHIR_INIT_THIS();

ZVAL_UNDEF(&key);
ZVAL_UNDEF(&_0);
ZVAL_UNDEF(&_1);
ZVAL_UNDEF(&_2);

ZEPHIR_MM_GROW();
zephir_fetch_params(1, 1, 0, &key_param);

zephir_get_strval(&key, key_param);

zephir_read_property(&_0, this_ptr, SL("_data"), PH_NOISY_CC | PH_READONLY);
zephir_var_dump(&_0 TSRMLS_CC);
zephir_read_property(&_1, this_ptr, SL("_data"), PH_NOISY_CC | PH_READONLY);
zephir_array_unset_string(&_1, SL("key_a"), PH_SEPARATE);
zephir_read_property(&_2, this_ptr, SL("_data"), PH_NOISY_CC | PH_READONLY);
zephir_var_dump(&_2 TSRMLS_CC);
ZEPHIR_MM_RESTORE();

result:
array(2) {
  ["key_a"]=>
  string(6) "marcin"
  ["key_b"]=>
  string(5) "paula"
}
array(2) {
  ["key_a"]=>
  string(6) "marcin"
  ["key_b"]=>
  string(5) "paula"
}

Zephir version: 0.9.3a-dev
PHP: 7.0
OS: Ubuntu 16.04
May I provide additional informations, which help resolve this issue?
I tried to modify C code by myself . After remove PH_SEPARATE attribute, issue is gone.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/34490939-unset-from-object-property-array-not-working?utm_campaign=plugin&utm_content=tracker%2F280146&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F280146&utm_medium=issues&utm_source=github).

@dreamsxin Could you please take a look?

@dreamsxin as you can see above it's used within phalcon itself causing some problems

@sergeyklay zephir parse add object-property-unset, or variable add separate flag.

That's weird, because in this PR as you can see test is failing https://github.com/phalcon/cphalcon/pull/14176/files

@Jurigag Can use zephir development version test again, the change https://github.com/phalcon/zephir/commit/843f5c8590ca2146e7b71db6ff6109a77ff2647c#diff-773a733807b93616c23cc11607b87a30R573

I'm seeing that this issue still occurs in 0.12.2.

@ruudbon could you please send a pull request with a test shows the issue


Add serde feature for serializing Commits

This adds 'serde' as optional dependency and makes the crate::Commit object serializable by implementing serde::Serialize.
Signed-off-by: orhun<EMAIL_ADDRESS>

I'd be curious what your use case is

I'm trying to implement a simple changelog generator and it has a wrapper that contains the Commit object. Recently I had to serialize this struct for the template engine. Although right now I'm only serializing a field from the Commit, I think having a serde feature for more extended use cases might be good.

especially to have Serialize without Deserialize (which is more annoying since we went the zero-copy route).

I actually have implemented Deserialize for Commit but I didn't like it due to Box::leak. If you think we can have a workaround for the lifetime issue that unicase::UniCase brings, I'm happy to add this to the PR.
diff:
diff --git a/src/commit.rs b/src/commit.rs
index 9cbc65e..9896421 100644
--- a/src/commit.rs
+++ b/src/commit.rs
@@ -13,7 +13,7 @@ const BREAKING_PHRASE: &str = "BREAKING CHANGE";
 const BREAKING_ARROW: &str = "BREAKING-CHANGE";

 /// A conventional commit.
-#[cfg_attr(feature = "serde", derive(serde::Serialize))]
+#[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
 #[derive(Default, Clone, Debug, PartialEq)]
 pub struct Commit<'a> {
     ty: Type<'a>,
@@ -134,7 +134,7 @@ impl fmt::Display for Commit<'_> {
 /// whitespace before newlines.
 ///
 /// See: <https://git-scm.com/docs/git-interpret-trailers>
-#[cfg_attr(feature = "serde", derive(serde::Serialize))]
+#[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
 #[derive(Debug, Clone, Copy, Eq, PartialEq, Hash)]
 pub struct Footer<'a> {
     token: FooterToken<'a>,
@@ -170,7 +170,7 @@ impl<'a> Footer<'a> {
 }

 /// The type of separator between the footer token and value.
-#[cfg_attr(feature = "serde", derive(serde::Serialize))]
+#[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
 #[derive(Debug, Clone, Copy, Eq, PartialEq, Hash)]
 #[non_exhaustive]
 pub enum FooterSeparator {
@@ -274,6 +274,16 @@ macro_rules! unicase_components {
                     serializer.serialize_str(self)
                 }
             }
+
+            #[cfg(feature = "serde")]
+            impl<'a> serde::Deserialize<'a> for $ty<'_> {
+                fn deserialize<D>(deserializer: D) -> Result<Self, D::Error>
+                where D: serde::Deserializer<'a> {
+                    let s = String::deserialize(deserializer)?;
+                    let ty = Self(unicase::UniCase::new(Box::leak(s.into_boxed_str())));
+                    Ok(ty)
+                }
+            }
         )+
     )
 }
@@ -402,7 +412,7 @@ mod test {
     #[test]
     fn test_commit_serialize() {
         let commit = Commit::parse("type(my scope): hello world").unwrap();
-        serde_test::assert_ser_tokens(
+        serde_test::assert_tokens(
             &commit,
             &[
                 Token::Struct {
@@ -415,7 +425,7 @@ mod test {
                 Token::Some,
                 Token::Str("my scope"),
                 Token::Str("description"),
-                Token::Str("hello world"),
+                Token::BorrowedStr("hello world"),
                 Token::Str("body"),
                 Token::None,
                 Token::Str("breaking"),

Thanks for the explanation!  Definitely makes sense to use this with templating.

Thanks for doing this!
I assume you'll want a new release cut with this change?

I assume you'll want a new release cut with this change?

Sure, that'd be great.

v0.9.2 is now up on crates.io


cant get it to run propperly

Description
hi, I am trying yo get it running on my manjaro machine.
It always displays the default manjaro pic and i tried the w3m google.com and it doesnt display pictures.
since I am prettey new with linux I would like to ask you to tell me what to do for further steps
br.
Neofetch version
Neofetch 7.1.0
Screenshot
Config file
Verbose log
https://pastebin.com/zCzSyPmu

Run neofetch -vv 2> neofetchlog
Upload the contents of neofetchlog to pastebin, gist or equivalent.

#1392
From your log:

[!]\e[0m Image: Failed to find terminal window size.
[!]\e[0m Image: Check the '''Images in the terminal''' wiki page for more info,

You need xdotool or xwininfo.


Incorrect prefer_const_constructors warning

In the following code:
void main() {
  final foo1 = Bar({const Foo(): ""});  // 1
  final foo2 = const Bar({Foo(): ""});  // 2
}

class Foo {
  const Foo();

  bool operator==(dynamic other) => false;

  int get hashCode => 1;
}

class Bar {
  const Bar(Map<Foo, String> x);
}

I get a warning for the foo1:

info: Prefer const with constant constructors. (prefer_const_constructors ...

But, with const at foo2, there is an error:

Error compiling to JavaScript:
main.dart:4:22:
Error: Constant evaluation error:
final foo2 = const Bar({Foo(): ""});
^
main.dart:4:27:
Info: The key 'Foo {}' does not have a primitive operator '=='.

'Foo' is from 'main.dart'.
final foo2 = const Bar({Foo(): ""});
^
Error: Compilation failed.

Dart 2.7.0

Thanks!
@scheglov Looks like analyzer is failing to report an error.

@bwilkerson I took a liberty to rename the issue to reflect the actual bug - previous title confused me (that's why I moved it to linter).

https://dart-review.googlesource.com/c/sdk/+/151623


[Snyk] Security upgrade sass from 1.55.0 to 1.79.0

Snyk has created this PR to fix 1 vulnerabilities in the npm dependencies of this project.
Snyk changed the following file(s):

package.json
package-lock.json

Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Uncontrolled resource consumption SNYK-JS-BRACES-6838727
  696

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Uncontrolled resource consumption

👷 Deploy Preview for relaxed-rolypoly-ea9b8f processing.

Name
Link

🔨 Latest commit
0eefbfc54805ddb5313d21895bf3cf3496a823d1

🔍 Latest deploy log
https://app.netlify.com/sites/relaxed-rolypoly-ea9b8f/deploys/66eac9266c83380008874a75


Not able to link to any repositories in my org

URL to the linked Repo/Org: mapbox/{any repo}
Steps to reproduce the problem:

sign in to https://cla-assistant.io/ with my Github credentials
authorize the CLA Assistant OAuth app to my account and org
click Configure CLA on https://cla-assistant.io/

What is the expected result?
I can choose repositories from the drop-down menu in the config window.
What happens instead?
No repositories show up in the drop-down menu in the config window. I am able to choose a gist, however.

I don't know why I can't see the images I uploaded above, but the public URLs are there if you want to look at them.

cc @trevorspecht @julianrex

@MapboxCI can you check whether you have admin rights for this organisation, please?
See https://github.com/cla-assistant/cla-assistant/issues/436#issuecomment-487910904

@KharitonOff based on your comment above it sounds like the Github account used with CLA Assistant needs to be an org member and not just a repository collaborator. Since that is the case, and @MapboxCI is not an org member, we will look at other options. Thanks for your help!
/cc @julianrex


Waste Free SMC available for adaptive tempered and tempered SMC.

Although SMC base.py was ready for waste-free, tempered and adaptive tempered are coupled to using classic SMC (iterate MCMC and keep the last sample). This PR makes Waste-free available at all levels of the hierarchy.
https://github.com/blackjax-devs/blackjax/issues/710

By the way, whilst we are on that, I remember being a bit dubious about waste free for parallel environments: when you have a lot of processors the analysis in the paper does not hold anymore.
@hai-dang-dau have you ever done some back of the envelope calculations on this? I'd like to be able to recommend people to switch off waste free (and maybe increase the number of intermediary distributions in exchange?) if they have a certain number of processors.

It's true that waste-free SMC shines best when the number of processors is small relative to the number of particles. (Say, a dozen of processors for several thousand of particles.) I think this is the case for most practical applications.
If your number of particles is comparable to that of processors (say, 500 processors and 500 particles), then yes standard SMC is better.
However, if you have that many processors, you probably have a lot of memory as well. When you increase the number of particles to about a million (still with 500 CPUs), waste-free starts to be preferable again.

Well, not really: a commercial grade GPU has fairly little memory so it's
not all that black and white. I'll have a think.
On Fri, 16 Aug 2024, 11:09 hai-dang-dau, @.***> wrote:

It's true that waste-free SMC shines best when the number of processors is
small relative to the number of particles. (Say, a dozen of processors for
several thousand of particles.) I think this is the case for most practical
applications.
If your number of particles is comparable to that of processors (say, 500
processors and 500 particles), then yes standard SMC is better.
However, if you have that many processors, you probably have a lot of
memory as well. When you increase the number of particles to about a
million (still with 500 CPUs), waste-free starts to be preferable again.
—
Reply to this email directly, view it on GitHub
https://github.com/blackjax-devs/blackjax/pull/721#issuecomment-2293228828,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEYGFZ5MOG624WOTBJF2BQTZRXFWRAVCNFSM6AAAAABMSYWGTWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOJTGIZDQOBSHA
.
You are receiving this because you commented.Message ID:
@.***>

In my previous post, by "processors" I meant the number of MCMC chains that can be run on parallel. Although GPU has a lot of parallel processing capacity, I'm not sure how this translates into the number of MCMC chains. The delicate point is that one MCMC chain itself involves a lot of parallelizable operations (ie matrix multiplications for calculating the likelihood or for generating preconditioned proposals). It is these primitive operations that are usually parallelized automatically by JAX/XLA, not the higher level operations that were taken into account in my reasoning above.
Having said that, it's true that we developed and tested waste-free SMC with CPUs in mind; so it'd definitely be relevant to have recommendations tailored to GPU-accelerated environment.

Feel free to submit after fixing the code style

@junpenglao the review gets dismissed whenever we commit something new, even if it is just github's commiting a reviewer suggestion


Allow accessing argument-less predicates on enum views without defining a custom view class

For example, this:
class SPIMode(amaranth.lib.enum.Enum, shape=2):
    Dummy = 0b00
    Swap  = 0b11
    Put   = 0b01
    Get   = 0b10

    @property
    def has_output(self):
        return self in (SPIMode.Swap, SPIMode.Put)

    @property
    def has_input(self):
        return self in (SPIMode.Swap, SPIMode.Get)

could have a default view class that acts as-if it was defined like this:
class SPIMode_View(...):
    @property
    def has_output(self):
        return Array(x.has_output for x in SPIMode)[self]

    @property
    def has_input(self):
        return Array(x.has_input  for x in SPIMode)[self]

(An actual implementation would use __getattr__, of course. This also means that a custom view class can override it easily.)
To avoid any ambiguity in types of arguments or unwanted combinational explosion, this would only occur for functions marked with @property. For all other functions a diagnostic would be shown indicating that a custom view class must be defined.

The big question for this proposal, besides "should we do this", is "what happens to values which don't have associated enum elements"?


Make the wrapper and init task unincubating

When running gradle tasks with the latest Gradle I get:
wrapper - Generates Gradle wrapper files. [incubating]```

Shouldn't we make those two tasks unincubating?

+1. We just need to remove the [incubating] part from the description and mention in the release notes.

Closing as complete!


The Service Framework should support the Dependency Injection pattern of defining Interfaces as dependencies

Reality Collective - Reality Toolkit Feature Request
Is your feature request related to a problem? Please describe

The Dependency Injection Pattern defines a method where at design time, the developer can define the interfaces that a service needs in order to properly instantiate
https://learn.microsoft.com/en-us/dotnet/core/extensions/dependency-injection
This controls the order in which services instantiate and enable tighter "loosely coupled" integration between services and ensures a dependent service is available to wire up events when a service starts.
How would you classify your suggestion

Architecture / Services

Describe the solution you'd like

The ability to define service interface dependencies in the design of a service and have the service framework automatically recognise the dependency on startup.
Describe alternatives you've considered

Manual

This is definitely something we must look into.

Migrated to:
https://github.com/realitycollective/com.realitycollective.service-framework/issues/58


Add test with a bunch of smaller inserts and simultaneous reading?  or periodic reading everything?

My use case is tons of small writes and occasional reads of the world.  I use modernc but am not married to it.  If you are agreeable I can add it.

You mean you want to add your use case to the benchmark? Yes, of course.


Add icon for .yamllint files

yamllint is a Python package that allows to lint YAML files. I've seen that markdownlint has his own icon for .markdownlint files, even if the icon is not official. So I suggest to include an icon for .yamllint configuration files. Maybe adding a check for an existent YAML icon could be enough.

Would this be adequate?

(Sorry about the long wait…)

Thank you!


Failed to read SimpleSSD configuration

Hi,
I have installed the SimpleSSD-FullSystem, when I run build/X86/gem5.opt configs/example/fs.py, "Aborted (core dumped)" occurs. Then, I debug it with gdb,  the error information is:../sysdeps/unix/sysv/linux/raise.c: No such file or directory.
Did you have similar problems? how to solve this problem?
Thanks for your reply!

Hi,
I've never seen such error.
You may want to check how to run gem5,
And SimpleSSD-FullSystem.
Thanks.
P.S. Content of current SimpleSSD homepage is outdated, and we are plan to update homepage until Sep. 2018.

Thanks for your reply! I try to run it  with ARM architecture, error information is as follows:
gem5 Simulator System.  http://gem5.org
gem5 is copyrighted software; use the --copyright option for details.
gem5 compiled Jul 24 2018 09:03:14
gem5 started Jul 25 2018 01:06:32
gem5 executing on ubuntu, pid 2160
command line: ./build/ARM/gem5.opt configs/example/fs.py
Traceback (most recent call last):
File "", line 1, in
File "/home/simplessd/SimpleSSD-FullSystem-release/src/python/m5/main.py", line 435, in main
exec filecode in scope
File "configs/example/fs.py", line 374, in
test_sys = build_test_system(np, simplessd)
File "configs/example/fs.py", line 111, in build_test_system
security=options.enable_security_extensions)
File "/home/simplessd/SimpleSSD-FullSystem-release/configs/common/FSConfig.py", line 412, in makeArmSystem
self.realview.attachPciDevice(
File "/home/simplessd/SimpleSSD-FullSystem-release/src/python/m5/SimObject.py", line 1130, in getattr
raise AttributeError, err_string
AttributeError: object 'VExpress_EMM' has no attribute 'attachPciDevice'
(C++ object is not yet constructed, so wrapped C++ methods are unavailable.)


Settings: have a consistent opening experience

Refs: https://github.com/Microsoft/vscode/issues/50496
I find it a bit weird that only the keybinding to open settings opens the new experience and not our other actions (e.g. from the gear menu). I would suggest to behave consistently and have clear guidance how to get back to the previous experience, possibly with a setting to always use the old experience if desired.

I see that this was only there for testing purposes so I guess we can close this issue and should remember to change back to open the classic settings.

I forgot about the gear menu. I'll change that next time I make the new editor the default.


Remove emojis for common cases

The markdown mode is too emoji heavy for common cases, after enabling this on some projects at work the first 2 questions i was asked were about what the emoji tick and cross mean, people assume it's related to some check that was performed on the gem.

gem
before
after
compare

ddtrace
0.18.1
0.18.2
:x:

i18n
1.4.0
1.5.1
:x:

nokogiri
1.9.1
1.10.0
:white_check_mark: nokogiri@ 1.9.1...1.10.0

Remove the check mark from success, since it is already clear that the script
worked since it rendered the compare link. The green tick is confusing because
we haven't tested or checked anything, but it looks like the gem is approved?
Remove the red x from failures and replace with an error message, the red X
assumes that the developer has some understanding of what the red X means, but
without any context it's easy to assume the Gem has failed a check.  But it
could just be that the gem isn't hosted on Github.
New output would look like

gem
before
after
compare

ddtrace
0.18.1
0.18.2
Failed to find projects GitHub URL

i18n
1.4.0
1.5.1
Failed to find projects GitHub URL

nokogiri
1.9.1
1.10.0
nokogiri@ 1.9.1...1.10.0

Thanks for the pull request, and sorry for merging slow.


Update toolbar icons with high DPI vector icons

Check with UI/UX team for possibiolity of existing set of Trimble/SketchUp style icons that can be used.

It seems the main UI is moving towards trendy monochrome icons. I'd very much stay with colorful, easy to find icons if possible. Usability > trends.

e1c97d5487b81b3cdeba498040f190b0f9b15e56


Enable installation from source utilizing andrewrothstein.git_build and Homebrew

thanks for the contribution! published to the galaxy as v3.1.0


"List locales" API does not return configured fallback_locale

Dear Phrase Team
We're trying to use the "List locales" API endpoint (documented here: https://developers.phrase.com/api/#get-/projects/-project_id-/locales) to get a list of all our configured locales in our project including their fallback_locale.
The API documentation says in the SCHEMA description at the bottom of the page that the  fallback_locale is included in the response payload, but in reality it is not.
Exemplary request - "List all locales":

When comparing this to the response payload of the "Get a single locale" API (see: https://developers.phrase.com/api/#get-/projects/-project_id-/locales/-id-), this one includes the fallback_locale correctly.
Exemplary request - "Get a single locale":

It would be great if you could change your implementation of the "List all locales" endpoint to match the API documentation and include the fallback_locale in the response payload.
Please let me know if you need any further details to troubleshoot this issue.
Thanks

@felixmagnus the schema seems indeed off here, we'll investigate how that happened. Are you relying on the fallback_locale to be present in the list response?

@felixmagnus we added the fallback_locale to the list endpoint as well to comply with the specification. Sorry for this mistake!

Hey @docstun
Thanks a lot for the quick response and the fix!


refactor sort exec stream and combine batches

Which issue does this PR close?
to make way for #360 I'll need the combine batches fn, so i'll refactor the current sort exec stream and add some integration test by the way
Closes #.
Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

Codecov Report

Merging #515 (cfe3429) into master (b84789a) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #515   +/-   ##
=======================================
  Coverage   76.09%   76.09%
=======================================
  Files         155      155
  Lines       26575    26578    +3
=======================================
+ Hits        20221    20225    +4
+ Misses       6354     6353    -1

Impacted Files
Coverage Δ

datafusion/src/physical_plan/common.rs
87.50% <100.00%> (+3.28%)
:arrow_up:

datafusion/src/physical_plan/sort.rs
91.95% <100.00%> (+0.21%)
: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 b84789a...cfe3429. Read the comment docs.


Extension publication versions are Postgres version specific

Closing for now, this has been pushed to a later date.


Bugfix/not property

Fix for the issue that currently, the NOT property of a ruleGroup below root is not set, thus always undefined. Can be verified in the live-demo.
Also added the not-property to RuleGroup's type definition.

Codecov Report

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

@@          Coverage Diff          @@
##           master   #113   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          13     13
  Lines         243    243
  Branches       21     21
=====================================
  Hits          243    243

Impacted Files
Coverage Δ

src/RuleGroup.jsx
100% <ø> (ø)
: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 89fc37f...ffd1072. Read the comment docs.

Thanks, @romanlamsal!  v2.5.1 has been published with this fix.


[BUG] linkcheck fails due to external http://dicom.nema.org/ is "no more"

Describe your problem in detail.
we have link checks workflow failing as in https://app.circleci.com/pipelines/github/bids-standard/bids-specification/5690/workflows/f27d68ca-5cc4-43b5-8540-9fd585ad6207/jobs/16482
URL        `http://dicom.nema.org/medical/dicom/current/output/chtml/part16/chapter_L.html#chapter_L'
Name       `DICOM Body Part Examined'
Parent URL file:///home/circleci/project/site/modality-specific-files/microscopy.html, line 1634, col 57
Real URL   http://dicom.nema.org/medical/dicom/current/output/chtml/part16/chapter_L.html#chapter_L
Check time 0.021 seconds
Size       315B
Result     Error: 404 Not Found

there is https://www.dicomstandard.org/ and https://www.dicomstandard.org/current in particular but links to html and chtml but those are broken too since lead to old urls .
So on the https://www.dicomstandard.org/contact?sf_cntrl_id=ctl00%24Contentplaceholder1%24C003 I have submitted  a report about breakage.
I guess let's wait for a few days - may be we will hear back, but we might want to point to some other location, e.g. https://dicom.innolitics.com/ciods/cr-image/general-series/00180015 , and even though it would be suboptimal, it should resolve CI redness for now.

got a reply "We had a planned outage and are working to bring it back up" so should be an intermittent issue.

that link is now back online, we should be all set on this one


Adopt zero trust

New How to for Security tab.

Approved by Alex and Spike
Backlog item so can merge now. But Protect Hosts PR should merge first (or link will fail)
Laurence can merge

Deploy preview for calico ready!
Built with commit 0e774c17c6e989ab9226a0c2bb1fce65d375841a
https://deploy-preview-2745--calico.netlify.com

Deploy preview for calico ready!
Built with commit 7b6afe7ec88f5da88cacb88ed7e11d88cbd69cad
https://deploy-preview-2745--calico.netlify.com


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_62_76nqk5r/clone/recipe/meta.yaml:14: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_62_76nqk5r/clone/recipe/meta.yaml:19: missing_wheel: For pypi packages, wheel should be present in the host section
WARNING: /tmp/abs_62_76nqk5r/clone/recipe/meta.yaml:24: python_build_tool_in_run: The python build tool setuptools is in run depends
ERROR: /tmp/abs_62_76nqk5r/clone/recipe/meta.yaml:33: missing_pip_check: For pypi packages, pip check should be present in the test commands
ERROR: /tmp/abs_62_76nqk5r/clone/recipe/meta.yaml:41: missing_documentation: The recipe is missing a doc_url or doc_source_url
WARNING: /tmp/abs_62_76nqk5r/clone/recipe/meta.yaml:41: missing_description: The recipe is missing a description
Errors were found


feat: allow importing built-in glamorous components

What:
allow something like import { Div, Span } from 'glamorous'

Why:
Convenient helpers improve user experience.

How:
Create a build script that appends exports to the es file.

Checklist:

[x] Documentation
[x] Tests
[x] Code complete
[x] Added myself to contributors table
[x] Followed the commit message format

Closes #123

Sidenote:
This is not related directly to this commit, but my liter shows an error in dist/glamorous.es.js:
'A module cannot have multiple default exports.'
That's caused by line 1177: exports.default = reactHtmlAttributes$2;

Hi! Thanks so much for this! I'm going to pull it down and run it to double check, but it looks pretty great.
Here are the linting failures. You may want to run npm start validate to check things before you push your changes :)

I would love it if we could also get a test in the dist-tests to check whether you can import things as we want to. Can do it with just one or two elements I think. Also documentation updates would be useful.

As for your linter issues, I think your linter is busted unless I'm missing something. That file is fine I think.

Codecov Report

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

@@          Coverage Diff          @@
##           master   #131   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          10     10
  Lines         130    130
  Branches       33     33
=====================================
  Hits          130    130

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 b742428...589f065. Read the comment docs.

[x] fix bug where component names collide with JavaScript constructors
[x] tests added
[x] documentation added

[x] change html prefix to Tag postfix when name collisions
[x] automatically create a constructor names list
[x] change appendFile to appendFileSync

[x] add a comment preceding exports
[x] provide info about Tag suffix in documentation

[x] export GlamorousComponents instead of GlamorousComponentFactories
[x] remove unnecessary function
[x] improve documentation
[x] make tests more robust

Wonderful! Thank you!


Fix update notifier

This PR makes sure we don't print "Update Available" notification when the current version is the same as the latest version.

fixes https://github.com/dbos-inc/dbos-cloud/issues/405


Refactor ab3 mcscheduler

refactored seedu.address -> mcscheduler
find and replaced all addressbook -> mcscheduler
refactor code imports (also changes things like INDEX_FIRST_WORKER to TypicalIndexes.INDEX_FIRST_WORKER automatically)
Resolves #103

Codecov Report

Merging #121 into master will increase coverage by 0.08%.
The diff coverage is 78.73%.

@@             Coverage Diff              @@
##             master     #121      +/-   ##
============================================
+ Coverage     69.95%   70.04%   +0.08%
- Complexity      858      863       +5
============================================
  Files           137      138       +1
  Lines          2962     2984      +22
  Branches        409      411       +2
============================================
+ Hits           2072     2090      +18
- Misses          765      769       +4
  Partials        125      125

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/mcscheduler/AppParameters.java
75.00% <ø> (ø)
9.00 <0.00> (?)

src/main/java/mcscheduler/Main.java
0.00% <ø> (ø)
0.00 <0.00> (?)

src/main/java/mcscheduler/MainApp.java
0.00% <0.00%> (ø)
0.00 <0.00> (?)

src/main/java/mcscheduler/commons/core/Config.java
68.18% <ø> (ø)
7.00 <0.00> (?)

...ain/java/mcscheduler/commons/core/GuiSettings.java
69.23% <ø> (ø)
5.00 <0.00> (?)

...main/java/mcscheduler/commons/core/LogsCenter.java
78.37% <ø> (ø)
10.00 <0.00> (?)

...c/main/java/mcscheduler/commons/core/Messages.java
0.00% <ø> (ø)
0.00 <0.00> (?)

...rc/main/java/mcscheduler/commons/core/Version.java
82.50% <ø> (ø)
19.00 <0.00> (?)

...ain/java/mcscheduler/commons/core/index/Index.java
100.00% <ø> (ø)
11.00 <0.00> (?)

...er/commons/exceptions/DataConversionException.java
100.00% <ø> (ø)
1.00 <0.00> (?)

... and 131 more

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 55e9cb8...5f61201. Read the comment docs.


Nrn admin poc

[GitHub Actions]
Deployment SUCCESS
Commit 1ea55e01e427f3231c478a7d1322c9d0039ded74 successfully deployed to https://nrn-admin-1lei9ov2n.now.sh
Deployment aliased as https://nrn-admin-nrn-admin-poc.now.sh

[GitHub Actions]
E2E tests FAILED
Download artifacts (screenshots + videos) from checks section at the top

[GitHub Actions]
Deployment SUCCESS
Commit 7c9ba7e96568a42502446464f0c8dab79368c433 successfully deployed to https://nrn-admin-lmavr87rp.now.sh
Deployment aliased as https://nrn-admin-nrn-admin-poc.now.sh

[GitHub Actions]
E2E tests FAILED
Download artifacts (screenshots + videos) from checks section at the top

[GitHub Actions]
Deployment SUCCESS
Commit 53259ef243eed0f4b29db2ef47efc686c59017ad successfully deployed to https://nrn-admin-lmavr87rp.now.sh
Deployment aliased as https://nrn-admin-nrn-admin-poc.now.sh

[GitHub Actions]
E2E tests SUCCESS

[GitHub Actions]
Deployment FAILED
Commit 1125cdd6d26e16efa0d6866f27e4e1db54a300f6 failed to deploy to https://nrn-admin-9a6f61p2g.now.sh (click to see logs)

[GitHub Actions]
E2E tests FAILED
Download artifacts (screenshots + videos) from checks section at the top

[GitHub Actions]
Deployment SUCCESS
Commit 7b16c58d9174478b32464b8da8300d89fbf7b26e successfully deployed to https://nrn-admin-84921w7xp.now.sh
Deployment aliased as https://nrn-admin-nrn-admin-poc.now.sh

[GitHub Actions]
E2E tests SUCCESS

[GitHub Actions]
Deployment SUCCESS
Commit<PHONE_NUMBER>977eb814aaabd64e0679962920c5f6 successfully deployed to https://nrn-admin-a9lh4eht6.now.sh
Deployment aliased as https://nrn-admin-nrn-admin-poc.now.sh

[GitHub Actions]
E2E tests SUCCESS

[GitHub Actions]
Deployment SUCCESS
Commit 6b5760467578983df11e21353eb4e494523fa35c successfully deployed to https://nrn-admin-dknnx6pzx.now.sh
Deployment aliased as https://nrn-admin-nrn-admin-poc.now.sh

[GitHub Actions]
E2E tests SUCCESS

[GitHub Actions]
Deployment SUCCESS
Commit fd9aca89712d729aea5bfaa6a2e554e3b363d6fd successfully deployed to https://nrn-admin-cbwdbv0pc.now.sh
Deployment aliased as https://nrn-admin-nrn-admin-poc.now.sh

[GitHub Actions]
E2E tests FAILED
Download artifacts (screenshots + videos) from checks section at the top


The image set "Splash" has 3 unassigned children.

Just wanted to report an error I may have tracked down.
The image set "Splash" has 3 unassigned children. which refers to splash-2732x2732.png, splash-2732x2732-1.png and splash-2732x2732-2.png.
I think the cause is due to these sizes no longer being included in Apple's specs.
https://developer.apple.com/design/human-interface-guidelines/foundations/app-icons#specifications

Is there is fix for this? I noticed that the universal splash images aren't being generated and also a number of icons. only 7 assets are being created for ios:
CREATE ios icon<EMAIL_ADDRESS>(1.07 MB)
CREATE ios splash<EMAIL_ADDRESS>(485 KB)
CREATE ios splash<EMAIL_ADDRESS>(485 KB)
CREATE ios splash<EMAIL_ADDRESS>(485 KB)
CREATE ios splash-dark<EMAIL_ADDRESS>(482.67 KB)
CREATE ios splash-dark<EMAIL_ADDRESS>(482.67 KB)
CREATE ios splash-dark<EMAIL_ADDRESS>(482.67 KB)
"@capacitor/assets": "^3.0.1",
Xcode 15.0.1


Sidr not opening on Google Chrome 36

Sidr is working well on IPAD2, Android, Firefox and even Internet Explorer 8 but in Google Chrome v36 the content seems to be missing.
Here is a temporary project Link. The left top button is the link button for sidr.
http://2016.webtransformer.de/leistungen.
I am using the latest sidr version.

For the love of God! It took me a lot of time to notice this.
If you have a touch screen clicking with the mouse on the toggle will not open the menu, but using your finger on it will open it.

Preparing a new version with a fix for this https://github.com/artberri/sidr/tree/reborn


How can I get the text preceded by the api?

only the heat map and the score map are displayed in the output folder, but I wanted to know how I can get the output text.

As @ShroukMansour replied, you need to crop the original image with each bounding box, and recognize them. This repo is only for text detection, and the repo for text recognition is here. Good luck!


Updated spec.sh to extract information about NVME devices

@jsfakian Please signoff the commit using the git's -s option (DCO check will be failing otherwise).


Invoke loadingDone when stopping SmartQuery

Had a problem with skipped queries being stuck with loading = true forever.
I think that stop() should invoke loadingDone() so that stopped query isn't reporting itself as loading.

Thanks! :+1:


how can I take photo with bebop2?

bebopVision.open_video() won't return current images. All I get were the old images.

Did you read the documentation and the sample code?  That is not how you get images.
https://pyparrot.readthedocs.io/en/latest/vision.html

I had run the sample code demoBebopVision.py
The program returned the images from the previous execution.

What OS are you on?  sounds like your ffmpeg program didn't clean out the old images.

The latest macOS. So please tell me how can I clean out the old images. Thank you very much.

that makes it easy!  It should be deleting them automatically though.  Since Mac OS is my primary environment, I'll go look into that.  In the meantime, find the images directory which should be inside your pyparrot installation and remove everything in there.

Thank you. This is really helpful. Feels like removing all images under the directory after each execution can be the simplest solution for me. I was keeping rebooting and reseting my bebop LOL

Yeah this is definitely in the directory and not on the bebop since you are using ffmpeg.  If you use the demoBebopVisionGUI version that won't happen (which is what I use 99% of the time).

I fixed this today.  ffmpeg now cleans out the images directory each restart by default (but it can be turned off if you prefer that).

Also fixed the bug where ffmpeg wasn't being killed properly!


[Searchable Snapshot] Design a new ShardsAllocator to handle remote shards

Is your feature request related to a problem? Please describe.

Searchable snapshots will introduce a new remote searcher role - https://github.com/opensearch-project/OpenSearch/issues/3895
As a part of this, we would like to design a new shard allocation logic which will assign the searchable snapshot shards to the nodes with this new role
There is a POC implementation which was done as a part of #4420 and exists at https://github.com/kotwanikunal/OpenSearch/pull/4

Describe the solution you'd like

Design for a new ShardsAllocator which ensures that the remote shards are assigned to the nodes with a remote_searcher role (Ref: https://github.com/kotwanikunal/OpenSearch/pull/4)
Ensures that the hybrid nature of having both the local and the remote shards on the single node are handled appropriately wherein the rebalancing logic for the local shards does not affect the remote shards, remote shards have the appropriate recovery process to handle node failures (Handled by the POC using basic checks, will need a deeper dive)
The AllocatorDecider(s) are capable of handling both the remote and local shards for the hybrid setup (Ref: https://github.com/kotwanikunal/OpenSearch/pull/4)

Additional context

#4420
#4306

The design was published at #4759


enable luarocks test

Is your feature request related to a problem? Please describe.
I would like luarocks test to work out of the box, ie., let luarocks be able to run the gitsigns.nvim testsuite
   > building
   > running tests
   >
   > Error: could not detect test type -- no test suite for gitsigns.nvim?

Describe the solution you'd like
Looking at the luarocks code, one can:

add a .busted file at the root
add test.type = "busted" in the rockspec .

https://github.com/luarocks/luarocks/wiki/test
Describe alternatives you've considered
I am able to run the tests manually but luarocks test creates a convention that can work across packages and as a package manager, it makes my work easier. Also going through luarocks, you can specify dependencies just for tests.
Additional context
I think I hit https://github.com/lewis6991/gitsigns.nvim/issues/847 so I thought this would be a good time to enable tests in the nixpkgs gitsigns-nvim package but then hit > Error: could not detect test type -- no test suite for gitsigns.nvim?. I can perfectly work around it but having lua packages/neovim plugins support luarocks test would make that a ton easier. gitsigns being very popular, it can set a good example.

FYI I've kinda given up on Luarocks so this isn't something I'll want to work on.

Also the testsuite uses nvim -l as the lua interpreter, not sure if luarocks can support that.

this can be used as a wrapper for nvim -l with luarocks. I plan on packaging it for luarocks sometime soon.


Examples have some Python warnings in the output

Description of the modifications
The output for the solve commands
https://fluent.docs.pyansys.com/version/stable/examples/00-fluent/exhaust_system.html#start-calculation
https://fluent.docs.pyansys.com/version/stable/examples/00-fluent/external_compressible_flow.html#solve-for-25-iterations
are showing some warnings in the monitor_streaming module, in the compressible flow example none of the iteration output is shown either.
Useful links and references
No response

Also in the external_compressible_flow example:

And I think those parentheses around the file name (in the text, not in the code line) could be removed as well


Add support for ulid

https://github.com/dylanhart/ulid-rs
motivation is https://github.com/ulid/spec
should be a simple task of copying the uuid code with only slight changes

Is this issue still open , I am looking to contribute . I really love this project and I guess this would be a great start

Yes, this task is available.


DOCK-2440: Removed default limit value for pagination in LambdaEvent endpoints

Description
This PR removes the default limit value so that we can paginate through all events in the app logs component.
The search bar was already implemented so that change was not needed for this PR.
This screenshot of the dockstore repo app logs show that the component shows all 119 events instead of the previous max of 100.

Review Instructions
Check the app logs of a large repo and confirm that the component shows more than 100 events.
Issue
https://ucsc-cgl.atlassian.net/browse/DOCK-2440
Security and Privacy
If there are any concerns that require extra attention from the security team, highlight them here.
e.g. Does this change...

Any user data we collect, or data location?
Access control, authentication or authorization?
Encryption features?

Please make sure that you've checked the following before submitting your pull request. Thanks!

[x] Check that you pass the basic style checks and unit tests by running mvn clean install
[x] Ensure that the PR targets the correct branch. Check the milestone or fix version of the ticket.
[x] Follow the existing JPA patterns for queries, using named parameters, to avoid SQL injection
[x] If you are changing dependencies, check the Snyk status check or the dashboard to ensure you are not introducing new high/critical vulnerabilities
[x] Assume that inputs to the API can be malicious, and sanitize and/or check for Denial of Service type values, e.g., massive sizes
[x] Do not serve user-uploaded binary images through the Dockstore API
[x] Ensure that endpoints that only allow privileged access enforce that with the @RolesAllowed annotation
[x] Do not create cookies, although this may change in the future
[x] If this PR is for a user-facing feature, create and link a documentation ticket for this feature (usually in the same milestone as the linked issue). Style points if you create a documentation PR directly and link that instead.

In the UI, do we implement pagination by retrieving all of the events at once on init, then "paging" via the UI code, and is that why we need to remove the limit?

Also, I'd say it's a bug in the web service that it doesn't enforce a max limit -- it should.

Created SEAB-5993 for that.

closing for new PR


TUM username pattern is hard coded in front end

Is your feature request related to a problem?
If your user does not follow the TUM username pattern you are not shown the dialog for registration. This is pretty unintuitive and bad for external use cases. I suggest being able to configure the username pattern via a property in application-artemis.yml.
Describe the solution you'd like
Make it configurable...
Describe alternatives you've considered
None
Additional context
TUM_USERNAME_REGEX in app.contants.ts should be replaced.

I might look into it even if by fix, for now, it just replacing the pattern with .*

will be fixed with #2174


Add pytest for DataContainer statistics

Closes https://github.com/dask-contrib/dask-sql/issues/1050

Codecov Report

Merging #1061 (7553bb9) into main (afbe4df) will decrease coverage by 0.18%.
The diff coverage is 75.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main    #1061      +/-   ##
==========================================
- Coverage   81.30%   81.12%   -0.18%
==========================================
  Files          78       78
  Lines        4375     4376       +1
  Branches      786      778       -8
==========================================
- Hits         3557     3550       -7
- Misses        645      648       +3
- Partials      173      178       +5

Impacted Files
Coverage Δ

dask_sql/datacontainer.py
95.45% <50.00%> (-1.72%)
:arrow_down:

dask_sql/context.py
93.75% <100.00%> (+0.36%)
:arrow_up:

dask_sql/physical/rex/core/literal.py
58.09% <0.00%> (-2.86%)
:arrow_down:

dask_sql/_version.py
32.76% <0.00%> (-1.13%)
:arrow_down:

dask_sql/physical/rel/custom/predict_model.py
91.66% <0.00%> (-0.23%)
:arrow_down:

dask_sql/utils.py
99.06% <0.00%> (-0.01%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Added 'Introduction in data science', small fixes

Added Introduction in Data Science;
Removed empty list items;

awesome :clap: thank you!


Acceptance Criteria #2

WHEN I view the structure of the HTML elements
THEN I find that the elements follow a logical structure independent of styling and positioning

the HTML shows a logical structure

the HTML shows a logical structure


Support for french layout keyboard

Support for AZERTY
Many people are willing to use pyautogui with foreign keyboard layout, that why I intended to propose this update. I, for example, created a project destined for French users.I was unable to use this tool properly without these modifications.
Functions added

setKeyboardLayout(target)
Change keyboard layout, needed for OSX and Windows operating system.
NOTE: Support for QWERTY and AZERTY only

Reminder: If setKeyboardLayout is not called, qwerty is selected by default. This function is optional.
This function is does nothing on Linux O.S.
By the way, thank for this very good job, asweigart!

@Ousret As of today (0.9.48), this is not solved on Win7 x64, Python 3.6, French keyboard layout.
I still have: Hello world§. Any idea for a temporary fix? Thanks in advance!

letters = "poiuytrezaqwsxdcfvgbhnjklm"
translated = "poiuytrewqazsxdcfvgbhnjkl,"
tr = dict(zip(letters, translated))
def translate(key):
return "".join(map(lambda x: tr.get(x.lower(), x),key))
tiens


The androidx-palette module is only supported by Kotlin Multiplatform

This is a great library, I love it!
But I encountered some problems in using.
Is it possible to separate out an androidx-palette-core module that only supports Kotlin Multiplatform from the androidx-palette module, since I don't need compose support.

That's a good point 👍. I'll add it to the list for the next release

Thank you so much!


chore(deps): upgrade runtime dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-runtime-dependencies-k8s-26-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3202

Questions ?
Please refer to the Backport tool documentation


Improve PS replica lag

PS8.1 replica had filled up disks with relay logs. The 8.1 source was doing too many transactions for replica to handle. This PR reduces disk flushing on replica.

Do you want to reject the config changes then and modify sysbench to do less?

no, i think we leave it with a tuned read-only worker :+1:


Unsecret no results in Workflow D.4 query

https://arax.ncats.io/?source=ARS&id=66cf5381-18cc-4756-8cd6-232500d7db8e

{
  "description": "Success. 0 results.",
  "logs": [
    {
      "level": "INFO",
      "message": "Query time: 54156ms"
    },
    {
      "level": "WARNING",
      "message": "MolePro error: \"Multi-edges queries not yet implemented\""
    },
    {
      "level": "INFO",
      "message": "MolePro response status: #\"HTTP/1.1 501 NOT IMPLEMENTED\""
    }
  ],
  "message": {
    "knowledge_graph": {
      "edges": {},
      "nodes": {}
    },
    "results": []
  },
  "query_graph": {
    "edges": {
      "e00": {
        "object": "n01",
        "subject": "n00"
      },
      "e01": {
        "object": "n02",
        "subject": "n01"
      },
      "e02": {
        "object": "n03",
        "subject": "n02"
      },
      "e03": {
        "object": "n04",
        "subject": "n03"
      }
    },
    "nodes": {
      "n00": {
        "categories": [
          "biolink:ChemicalSubstance"
        ],
        "ids": [
          "DRUGBANK:DB00150"
        ]
      },
      "n01": {
        "categories": [
          "biolink:Protein"
        ]
      },
      "n02": {
        "categories": [
          "biolink:MolecularActivity"
        ]
      },
      "n03": {
        "categories": [
          "biolink:MolecularActivity"
        ]
      },
      "n04": {
        "categories": [
          "biolink:ChemicalSubstance"
        ],
        "ids": [
          "KEGG.COMPOUND:C02700"
        ]
      }
    }
  },
  "status": "Success",
  "validation_result": {
    "message": "",
    "n_edges": 0,
    "n_nodes": 0,
    "size": "2 kB",
    "status": "PASS",
    "version": "1.1.1"
  }
}

@webyrd request for update

Will triage by August 12, 2021.

https://arax.ncats.io/?source=ARS&id=d80a01b8-d3e9-4699-b248-b7e1c2c01fa2

Will said these tickets are being worked on currently


XBMC4XBOX FREEZING ON FILEMANAGER

Issue is that whenever user opens file manager in xbmc4xbox clean install it freezes the xbox when opening file manager from within xbmc4xbox with titan bios. It just gets stuck saying working then becomes unresponsive.

It works when I use add source in xbmc4xbox and filemanager works in unleash x just not in xbmc4xbox.

The short answer is that this is a bug that needs to be fixed in XBMC.
The longer answer is that the higher UDMA speeds are exposing some of the bad assumptions and unsafe practices within XBMC's codebase. This is sort of similar to the temp issues when using an XboxHD+
Specifically, I think there's probably race conditions where XBMC fails to read the correct DVD drive state on startup which is causing the File manger to get confused and hang (or crash).
I kind of hint at this in the Titan README:
DVD drive status can be flakey in XBMC under higher UDMA modes (SMBus issues? most likely)

If you were to eject your DVD drive or try going to the system info tab before opening the system file manager, I assume it'll work just fine because XBMC will re-fetch the DVD drive state.
Alternatively you could try lowering your UDMA speed to 4, or possibly even trying a different revision of DVD drive in the Xbox which 'might' have just enough difference in response times to avoid the race.
The correct longterm solution is that one of us technical contributors will probably have to go and formally update XBMC to start rectifying these modern edgecases on behalf of the community.

I can confirm that loading the system info then going back does indeed fix the issue for the filemanager in xbmc. It only pops up again when restarting xbox then I have to go back into system info to fix it. Using the eject button caused it to freeze as well except this time it froze with the rol blinking as if it's reading a disk.

It seems like dashloader isnt running right either.

Dashloader being pretty important to fix as there is no easy way to setup xbmc unless you wanna start using shortcut xbe but I rather use dashloader app but it does not launch.

Please try to file separate issues for separate problems, it makes it easier to track and fix things as applicable.
I was able to reproduce your issue with the file manager 'freezing' tonight, specifically only after swapping a Samsung-605B into the Xbox I primarily developed and tested Titan on. With a Thompson drive in this system, there are no issues with XBMC or File Manger. With a Samsung 605B, it's behaving similar to how you describe (but eject/sys info seems to help workaround the issue).
Practically speaking Titan doesn't touch or modify anything in the kernel that has to do with the DVD drive, so it's definitely a bit odd. The Xbox kernel also makes zero special-casing for the different types of DVD drives.
I'm surprised the UDMA change didn't have any effect but I'm still confident that XBMC is partially at fault here. It's also possibly aggravated by the fact that many Samsung drives have begun to fall out of spec and ultimately fail (error 12, etc.) over the past couple of years.
TL;DR: I will still look into root-causing this issue, but Thompson drives do not seem to experience this with XBMC.
Thanks for the report!
PS: Also can you share info about your hardware / setup? Eg. Xbox version, SATA Adapter, SATA Drive Type, DVD Drive Type, Modchip

Hardware V1.4 Startech sata adapter 80 wire ide cable upgrade. It's tsoped flashed no modchip needed here. 4tb sata hgst drive.

Good thing is xbmc4xbox gamers edition fixes the filemanager issue

Just as an additional comment, I tested with an unmodified version of M8+ and Samsung 605B with a 10TB HDD in the system and I got the same file manager 'hanging' problem. So it isn't an issue introduced by the Titan patches. It's some sort of artifact between a 'big' HDD versus a slow DVD drive?
I looked briefly at the DVD drive initialization logic during bootup. The kernel will wait for up to 1 second for the DVD drive to initialize itself and reach a 'ready' state before it gives up and continues through the boot process. I'm assuming that my Samsung 605B and your Philips are failing to complete their media detect / initialization during this window.
I could try patching it to be a bit longer (say 5 seconds) just as a quick test and 'dirty' solution, but it doesn't exactly explain why a DVD drive would be slower to initialize or report its status as ready with a 'bigger' HDD in the system. It's a bit of an odd issue, for sure.

Idk now that I have everything running The only thing I hate that doesn't work is IGR which is very annoying to always have to turn off xbox. Please make that a priority over everything as that is the main feature of a tsop that is actually very useful.

FIX IGR FOR NEXT RELEASE.

I resolved all my issues by switching to the far superior cerbios now that it support v1.4 and I'm never looking back this will be the last time I have to rip my hdd out my xbox to backup everything to reformat it for a special bios.

If you can't appreciate somebody volunteering their free time to implement such features and open sourcing this, then you shouldn't be here in the first place. Also, nobody forces you to use this modification. If you're happy with a different solution, like the mentioned Cerbios, then more power to you. But you're not doing any favors and just wasting time if you think that such kind of messages would be in any kind helpful. You didn't pay for anything for this and the project is explicitly marked "highly experimental ". Nobody owes you anything and nobody said that you must install this on your Xbox.

this issue is biting me hard as well, and for me, it is a major annoyance.
It would be awesome if someone could fix it, in my case it just shows empty, but whenever I run a donglefree dvd playback app, the movie plays fine, so the DVD drive is not the culprit. Going to the system info tap or ejecting the drive while in any version of XBMC other than XBMC4Gamers doesn't fix it. but sadly XBMC4Gamers has no multimedia support at all. So I don't know of any way to get XBMC to play movies while using Titan.

this issue is biting me hard as well, and for me, it is a major annoyance.
It would be awesome if someone could fix it, in my case it just shows empty, but whenever I run a donglefree dvd playback app, the movie plays fine, so the DVD drive is not the culprit. Going to the system info tap or ejecting the drive while in any version of XBMC other than XBMC4Gamers doesn't fix it. but sadly XBMC4Gamers has no multimedia support at all. So I don't know of any way to get XBMC to play DVD movies while using Titan.
If nobody bothers to fix it at all I'll just have to use Cerbios or something else. I'm willing to have UDMA5 instead of 6 since Titan boots so much faster and it looks much more stock than the mess that Cerbios is upon boot (I prefer having a boot animation as stock as possible). But this DVD thing is so, SO annoying to not be able to play movies anymore due to it. I don't want to discredit the work: thanks for the hard work on these kernel patches but this DVD issue needs fixing so it's awesome for me.

I don't think multimedia support has been removed. However, there is not menu tab to play DVD's from.  IIRC, you can play a DVD going through the File Manager in XBMC4Gamers. In the File Manager, select the DVD to play it.


Results should not be displayed from deprecated collections

The following query returns a number of deprecated collections:
query MyQuery {
  collection(where: {typeId: {_eq: 1}}) {
    id
    slug
    createdAt
    updatedAt
    deletedAt
  }
}

For example:
{
      "id": 11,
      "slug": "op-govgrants-phoenix",
      "createdAt": "2023-09-30T16:39:04.552222+00:00",
      "updatedAt": "2023-09-30T16:39:04.552222+00:00",
      "deletedAt": null
},

A number of temporary collections were deleted in this recent PR.
The issue might be related to (not) running the importOssDirectory action and could therefore be a very quick fix. If not, and it's more complicated, then this is P2 / grabbag category.

At the end of importOssDirectory, just mark the deletedAt column of any verified collections that weren't imported.
Then in the front-end we should only search for Collections with a null deletedAt.


Server EM can block on Cloud::WebsocketClient#connect DNS lookups

The new #2139 server Watchdog is barking:
I, [2017-04-12T16:50:46.858737 #31110]  INFO -- Watchdog<WebsocketBackend>: watchdog start
W, [2017-04-12T16:51:01.763966 #31110]  WARN -- Watchdog<WebsocketBackend>: watchdog delayed by 10.462s @
	/home/kontena/kontena/kontena/server/vendor/bundle/ruby/2.3.0/gems/eventmachine-1.2.3/lib/eventmachine.rb:673:in `bind_connect'
	/home/kontena/kontena/kontena/server/vendor/bundle/ruby/2.3.0/gems/eventmachine-1.2.3/lib/eventmachine.rb:653:in `connect'
	/home/kontena/kontena/kontena/server/vendor/bundle/ruby/2.3.0/gems/faye-websocket-0.10.7/lib/faye/websocket/client.rb:50:in `initialize'
	/home/kontena/kontena/kontena/server/app/services/cloud/websocket_client.rb:105:in `new'
	/home/kontena/kontena/kontena/server/app/services/cloud/websocket_client.rb:105:in `connect'
	/home/kontena/kontena/kontena/server/app/services/cloud/websocket_client.rb:69:in `block in ensure_connect'
	/home/kontena/kontena/kontena/server/vendor/bundle/ruby/2.3.0/gems/eventmachine-1.2.3/lib/em/timers.rb:56:in `fire'
	/home/kontena/kontena/kontena/server/vendor/bundle/ruby/2.3.0/gems/eventmachine-1.2.3/lib/eventmachine.rb:194:in `call'
	/home/kontena/kontena/kontena/server/vendor/bundle/ruby/2.3.0/gems/eventmachine-1.2.3/lib/eventmachine.rb:194:in `run_machine'
	/home/kontena/kontena/kontena/server/vendor/bundle/ruby/2.3.0/gems/eventmachine-1.2.3/lib/eventmachine.rb:194:in `run'
	/home/kontena/kontena/kontena/server/app/initializers/eventmachine.rb:10:in `block in <top (required)>'
I, [2017-04-12T16:51:02.455117 #31110]  INFO -- Cloud::WebsocketClient: cloud connection opened to wss://socket.kontena.io

Tracing this out, the eventmachine.rb:673#bind_connect calls into the native EventMachine.connect_server method, which ends up in EventMachine_t::ConnectToServer. While the connect itself is non-blocking, it has to first resolve the socket.kontena.io address, and that seems to be a blocking operation: https://github.com/eventmachine/eventmachine/blob/v1.2.3/ext/em.cpp#L1206
	int gai = name2address (server, port, SOCK_STREAM, (struct sockaddr *)&bind_as, &bind_as_len);
...
	if (connect (sd, (struct sockaddr *)&bind_as, bind_as_len) == 0) {

This should be fixed by rewriting the server Cloud::WebsocketClient to use a Celluloid actor with Kontena::Websocket::Client instead - then the DNS lookups would happen in the celluloid actor thread, and not block the EventMachine reactor used for the agent websocket connections.


Provide user assistance when not receiving required attributes

Currently if Rapid does not receive identity data from upstream IdP it will carry on and provide JWT to remote services with empty strings, even for data that we have marked SHOULD be available.
We should re-visit this process to determine if this is what rapid services reasonably expect to happen in this scenario vs users not being provided a JWT and redirected at all.
If this is considered valid we should highlight this edge case in documentation.
If this is considered invalid we could possibly inject in the flow and advise the user their details are not complete, push them through attribute validator and advise they should seek the assistance of their helpdesk before continuing.

There was another instance of this occuring within AAF support on issue 3824 in January 2016


Plugins: Add Leaflet.TileLayer.GLOperations

New plugin Leaflet.TileLayer.GLOperations.
Demo

Lots of overlap with TileLayer.GL, and I'm not a big fan of regl, but looks useful enough.


fix batch inserting with bind by index.

just a small pr to show the problem. I'm not sure what the correct behavior should be, but bindNull should probably also be fixed.

great. i just created this pr to get some feedback, which i never got but good that its fixed :)


Handle validation errors raised by autocomplete_create method

Right now, zero error handling is done when we invoke this method. There are probably a lot of ways we could go about this, but building in handling of validation error (and also returning the results of that error, if there are any), allows users of this feature to use fairly standard Django practices to validate their object before saving and have that be understood by the autocomplete view. One example of how to do this is given in the updated testapp code.
Fixes https://github.com/wagtail/wagtail-autocomplete/issues/134

Added a small error message display on the front-end. Not sure if it's really the best, but, well, it's something. Suggestions welcome.


CW-2981 extension

Appreciate this might need some refining and/or further discussion.
A minimal example of the EIP-2981 spec implemented in CW-land. i.e. #7
It's a very light wrapper/extension for CW721 compliant contracts that adds two additional query messages:

RoyaltyInfo to be called when selling an NFT, to find out what royalties are owed
CheckRoyalties to be called on the contract, to see if it implements CW-2981

This adds royalty information at token mint time.
The main thing are the queries that are called when the marketplace sells an NFT. Different contracts might implement them differently, e.g. to do more complex logic than the straight percentage here, but the returned queries should conform to the shape here.
Aside - could move those (and the extension handlers) to a package, if it makes them more reusable.

Will address the comments and add some more tests 👍

If this is GTM could it be merged? Otherwise before long I'll have to update the PR because things will have shifted.... 🙃

If this is GTM could it be merged? Otherwise before long I'll have to update the PR because things will have shifted.... upside_down_face

Well if things shift it will need to be updated anyway. :upside_down_face:
Needs one more thumbs up.

Okay I think that's all the review points closed off.

@the-frey Do you think we can get this PR in this year xp

@orkunkl I mean, if people want to review it then maybe we can get it in lol

Okay @orkunkl and @JakeHartnell have updated again which means it has cleared your reviews. If you are both still ok to merge, could you review pls 🙏


feat: Lang::intern_symbols

Add Lang::intern_symbols to facilitate lang installations
Use Lang::intern_symbols on the REPL
Create an alias for Rc<RefCell<State>> and use it
Rewrite some functions to one-liners

Closing in favor of #1189, which moves us closer to where we want to be


Reviewer comment

The reviewer think that the model lacks references to Robbie the Robot.

The code has been modified to accommodate comments from a pedantic reviewer.


can't find crate for glutin_window

I got Visual Rust build #80 from AppVeyor, Rust 1.1, and Visual Studio 2015 installed. I created a fork of rocket and added the a .rsproj file and .sln.
https://github.com/ctaggart/rocket
Unfortunately, I'm getting this error and I don't understand why. cargo build works just fine.

/cc @aochagavia

I have no idea about this :wink: ... Never used VisualRust

Currently there is no cargo support in VisualRust yet (see #3). When you try to build, it will just call rustc, which doesn't know abou the dependencies. Until carge integration is implemented, you can have a look at this page, the section "Using VS External Tools" describes how to manually set up VS to invoke cargo.

I found you can build the library separately and get the .rlib in the target/debug and all .rlib files in target/debug/deps and copy them to "C:\Program Files\Rust stable 1.1\bin\rustlib\i686-pc-windows-gnu\lib".
(I also tried adding AdditionalLibPaths="$(ProjectDir)" to the Rust.target file in MSBuild/VisualRust. If it worked, the rlibs could go in the project directory making things a bit cleaner. But it doens't work for some reason...)

Finnaly I create VisualRust.Build.Cargo.dll and VisualRust.Build.Cargo.target. It's simple works fine for me.

Fixed by #234


ReSetup base engine and game projects Vulkan and CI

[x] Setup base project
[x] Setup base game
[ ] Setup Github Action CI
[x] Setup Vulkan rendering of a basic triangle
[ ] Setup ImGUI base
[ ] Setup ImGUI configs settable from the raytracer
[ ] Fix shader compilation moved to the wrong path for Visual Studio

Cannot progress on ImGUI stuff since from the looks of it, it needs more stuff to be setup as a part of the renderer, so storing current work in the setup-project-imgui branch

From Vulkan-Tutorial:
If you like a challenge, then you can still try to use a different queue family specifically for transfer operations. It will require you to make the following modifications to your program:

- Modify QueueFamilyIndices and findQueueFamilies to explicitly look for a queue family with the VK_QUEUE_TRANSFER_BIT bit, but not the VK_QUEUE_GRAPHICS_BIT.
- Modify createLogicalDevice to request a handle to the transfer queue
- Create a second command pool for command buffers that are submitted on the transfer queue family
- Change the sharingMode of resources to be VK_SHARING_MODE_CONCURRENT and specify both the graphics and transfer queue families
- Submit any transfer commands like [vkCmdCopyBuffer](https://www.khronos.org/registry/vulkan/specs/1.0/man/html/vkCmdCopyBuffer.html) (which we'll be using in this chapter) to the transfer queue instead of the graphics queue


Rename config/apps/AppConfig.py to follow snake case convention

🐛 Describe the bug
Listed file name should follow snake case convention.
Also handle the related imports of this file and make sure that everything works.

@taranjeet I would like to try this out.


go1.22.2 - example/webrtc Audio works, but video doesnt

Your environment.

Version: Ubuntu
Browser: chrome - 123.0.6312.105

What did you do?
I copy/pasted your code from the examples/webrtc and then ran CGO_ENABLED=1 go build my-module, followed the instructions on copy/pasting the SDP.
What did you expect?
I expected it to stream my camera and testaudio to browser
What happened?
it streamed testaudio audio just fine, but not the camera. testvideo streaming doesnt work as well.
Here is my full code
package main

import (
	"fmt"

	"github.com/pion/mediadevices"
	"github.com/pion/mediadevices/pkg/frame"
	"github.com/pion/mediadevices/pkg/prop"
	"github.com/pion/webrtc/v3"

	// If you don't like x264, you can also use vpx by importing as below
	// "github.com/pion/mediadevices/pkg/codec/vpx" // This is required to use VP8/VP9 video encoder
	// or you can also use openh264 for alternative h264 implementation
	// "github.com/pion/mediadevices/pkg/codec/openh264"
	// or if you use a raspberry pi like, you can use mmal for using its hardware encoder
	// "github.com/pion/mediadevices/pkg/codec/mmal"
	"github.com/pion/mediadevices/pkg/codec/opus" // This is required to use opus audio encoder
	"github.com/pion/mediadevices/pkg/codec/x264" // This is required to use h264 video encoder

	// Note: If you don't have a camera or microphone or your adapters are not supported,
	//       you can always swap your adapters with our dummy adapters below.
	_ "github.com/pion/mediadevices/pkg/driver/audiotest"
	_ "github.com/pion/mediadevices/pkg/driver/videotest"
	// _ "github.com/pion/mediadevices/pkg/driver/camera" // This is required to register camera adapter
	//_ "github.com/pion/mediadevices/pkg/driver/microphone" // This is required to register microphone adapter
)

func main() {
	config := webrtc.Configuration{
		ICEServers: []webrtc.ICEServer{
			{
				URLs: []string{"stun:stun.l.google.com:19302"},
			},
		},
	}

	// Wait for the offer to be pasted
	offer := webrtc.SessionDescription{}
	Decode(MustReadStdin(), &offer)

	// Create a new RTCPeerConnection
	x264Params, err := x264.NewParams()
	if err != nil {
		panic(err)
	}
	x264Params.Preset = x264.PresetMedium
	x264Params.BitRate = 500_000 // 500kbps

	opusParams, err := opus.NewParams()
	if err != nil {
		panic(err)
	}
	codecSelector := mediadevices.NewCodecSelector(
		mediadevices.WithVideoEncoders(&x264Params),
		mediadevices.WithAudioEncoders(&opusParams),
	)

	mediaEngine := webrtc.MediaEngine{}
	codecSelector.Populate(&mediaEngine)
	api := webrtc.NewAPI(webrtc.WithMediaEngine(&mediaEngine))
	peerConnection, err := api.NewPeerConnection(config)
	if err != nil {
		panic(err)
	}

	// Set the handler for ICE connection state
	// This will notify you when the peer has connected/disconnected
	peerConnection.OnICEConnectionStateChange(func(connectionState webrtc.ICEConnectionState) {
		fmt.Printf("Connection State has changed %s \n", connectionState.String())
	})

	s, err := mediadevices.GetUserMedia(mediadevices.MediaStreamConstraints{
		Video: func(c *mediadevices.MediaTrackConstraints) {
			c.FrameFormat = prop.FrameFormat(frame.FormatI420)
			c.Width = prop.Int(640)
			c.Height = prop.Int(480)
		},
		Audio: func(c *mediadevices.MediaTrackConstraints) {
		},
		Codec: codecSelector,
	})
	if err != nil {
		panic(err)
	}

	for _, track := range s.GetTracks() {
		track.OnEnded(func(err error) {
			fmt.Printf("Track (ID: %s) ended with error: %v\n",
				track.ID(), err)
		})

		_, err = peerConnection.AddTransceiverFromTrack(track,
			webrtc.RtpTransceiverInit{
				Direction: webrtc.RTPTransceiverDirectionSendonly,
			},
		)
		if err != nil {
			panic(err)
		}
	}

	// Set the remote SessionDescription
	err = peerConnection.SetRemoteDescription(offer)
	if err != nil {
		panic(err)
	}

	// Create an answer
	answer, err := peerConnection.CreateAnswer(nil)
	if err != nil {
		panic(err)
	}

	// Create channel that is blocked until ICE Gathering is complete
	gatherComplete := webrtc.GatheringCompletePromise(peerConnection)

	// Sets the LocalDescription, and starts our UDP listeners
	err = peerConnection.SetLocalDescription(answer)
	if err != nil {
		panic(err)
	}

	// Block until ICE Gathering is complete, disabling trickle ICE
	// we do this because we only can exchange one signaling message
	// in a production application you should exchange ICE Candidates via OnICECandidate
	<-gatherComplete

	// Output the answer in base64 so we can paste it in browser
	fmt.Println(Encode(*peerConnection.LocalDescription()))

	// Block forever
	select {}
}

Doesnt work with screen as well(as the video track, after changing GetUserMedia as per docs).

just installed go1.19, same issue..

this happens to me as well, however on chrome the browser does not create any offer, so I switched to firefox that creates the offer but when connected the video is running without showing any image

This problem happens to me too.
The audio works, but video does not.
Anyone knows how to fix it?
Thanks

This problem happens to me too.
The audio works, but video does not.
Anyone knows how to fix it?
Thanks


feat: add arrow2nd/nimotsu

#5665 arrow2nd/nimotsu: CLI tool to track packages in Japan
$ aqua g -i arrow2nd/nimotsu

Thank you always!


ENHANCE AND STRUCTURE THE CONTRIBUTORS SECTION

Related Issue
Fixes:  #1524
Description
The existing documentation for the Contributors section lacks visual representation, which diminishes the engagement of users. It fails to adequately acknowledge the efforts of contributors, potentially leading to decreased motivation for further participation. Currently, there are no clickable links to contributors' GitHub profiles, making it difficult for users to explore their work. The layout is poorly organized, cluttered, and lacks structure, which can confuse readers. Additionally, the absence of quantitative metrics, such as the number of contributions or issues resolved by each contributor, limits the recognition of their impact. To enhance and structure the Contributors section, it is essential to add dynamic visual elements, include a heartfelt acknowledgment message, make contributor names clickable, and organize the information with clear headings and metrics that highlight each contributor's achievements. Removing any redundant or outdated information will further streamline this section, ensuring it effectively celebrates the community's contributions.
Type of PR

[ ] Bug fix
[x] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)

Checklist:

[x] I have made this change from my own.
[x] I have taken help from some online resources.
[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers and screenshots after making the changes.

@PriyaGhosal  the task has  been completed.

@alo7lika
Your pull request has been successfully merged! Don't stop here—continue contributing by addressing more issues and aiming to become a top contributor. Exclusive swag awaits for those who go the extra mile!
Great contribution ! 🎉


ES6 imports on lambda assets

The live version is still broken.
https://github.com/aws-samples/aws-cdk-intro-workshop/pull/125

Thank you for the catch. I will make sure to push an update shortly.


feat: Create override method mailgunVariables receiving variables as json string

@ederfmatos Added in version 1.0.7.


Update install instructions for clarity and to include requested info…

…rmation.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.gitignore
:white_check_mark:Succeeded

docs-conceptual/azurermps-5.7.0/install-azurerm-ps.md
:white_check_mark:Succeeded
View (azurermps-5.7.0)

docs-conceptual/azurermps-6.3.0/install-azurerm-ps.md
:white_check_mark:Succeeded
View (azurermps-6.3.0)

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

:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-conceptual/azurermps-5.7.0/install-azurerm-ps.md
:warning:Warning
View (azurermps-5.7.0)
Details

.gitignore
:white_check_mark:Succeeded

docs-conceptual/azurermps-6.3.0/azurerm-ps-in-docker.md
:white_check_mark:Succeeded
View (azurermps-6.3.0)

docs-conceptual/azurermps-6.3.0/install-azurerm-ps.md
:white_check_mark:Succeeded
View (azurermps-6.3.0)

docs-conceptual/azurermps-6.3.0/other-install.md
:white_check_mark:Succeeded
View (azurermps-6.3.0)

docs-conceptual/azurermps-6.3.0/toc.yml
:white_check_mark:Succeeded
View (azurermps-6.3.0)

docs-conceptual/azurermps-6.3.0/uninstall-azurerm-ps.md
:white_check_mark:Succeeded
View (azurermps-6.3.0)

docs-conceptual/azurermps-5.7.0/install-azurerm-ps.md

Line 114: [Warning] Invalid file link:(~/docs-conceptual/azurermps-5.7.0/uninstall-azurerm-ps.md).

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

@sptramer You have a broken link. You need to copy the uninstall article to the other versions. And update the TOCs to include the new article.

@sdwheeler Should've informed you but this PR is still in the middle of editing stages. I had to push up so that I could sync the work across machines. I'll sign off and ping you directly when it's ready for final review.

:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-conceptual/azurermps-5.7.0/install-azurerm-ps.md
:warning:Warning
View (azurermps-5.7.0)
Details

docs-conceptual/azurermps-5.7.0/other-install.md
:warning:Warning
View (azurermps-5.7.0)
Details

.gitignore
:white_check_mark:Succeeded

docs-conceptual/azurermps-6.3.0/azurerm-ps-in-docker.md
:white_check_mark:Succeeded
View (azurermps-6.3.0)

docs-conceptual/azurermps-6.3.0/context-persistence.md
:white_check_mark:Succeeded
View (azurermps-6.3.0)

docs-conceptual/azurermps-6.3.0/install-azurerm-ps.md
:white_check_mark:Succeeded
View (azurermps-6.3.0)

docs-conceptual/azurermps-6.3.0/other-install.md
:white_check_mark:Succeeded
View (azurermps-6.3.0)

docs-conceptual/azurermps-6.3.0/toc.yml
:white_check_mark:Succeeded
View (azurermps-6.3.0)

docs-conceptual/azurermps-6.3.0/uninstall-azurerm-ps.md
:white_check_mark:Succeeded
View (azurermps-6.3.0)

docs-conceptual/azurermps-5.7.0/install-azurerm-ps.md

Line 114: [Warning] Invalid file link:(~/docs-conceptual/azurermps-5.7.0/uninstall-azurerm-ps.md).

docs-conceptual/azurermps-5.7.0/other-install.md

Line 19: [Warning] Invalid file link:(~/docs-conceptual/azurermps-5.7.0/azurerm-ps-in-docker.md).

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.gitignore
:white_check_mark:Succeeded

docs-conceptual/azurermps-5.7.0/azurerm-ps-in-docker.md
:white_check_mark:Succeeded
View (azurermps-5.7.0)

docs-conceptual/azurermps-5.7.0/install-azurerm-ps.md
:white_check_mark:Succeeded
View (azurermps-5.7.0)

docs-conceptual/azurermps-5.7.0/install-azurermps-maclinux.md
:white_check_mark:Succeeded
View (azurermps-5.7.0)

docs-conceptual/azurermps-5.7.0/other-install.md
:white_check_mark:Succeeded
View (azurermps-5.7.0)

docs-conceptual/azurermps-5.7.0/toc.yml
:white_check_mark:Succeeded
View (azurermps-5.7.0)

docs-conceptual/azurermps-5.7.0/uninstall-azurerm-ps.md
:white_check_mark:Succeeded
View (azurermps-5.7.0)

docs-conceptual/azurermps-6.3.0/azurerm-ps-in-docker.md
:white_check_mark:Succeeded
View (azurermps-6.3.0)

docs-conceptual/azurermps-6.3.0/context-persistence.md
:white_check_mark:Succeeded
View (azurermps-6.3.0)

docs-conceptual/azurermps-6.3.0/install-azurerm-ps.md
:white_check_mark:Succeeded
View (azurermps-6.3.0)

docs-conceptual/azurermps-6.3.0/install-azurermps-maclinux.md
:white_check_mark:Succeeded
View (azurermps-6.3.0)

docs-conceptual/azurermps-6.3.0/other-install.md
:white_check_mark:Succeeded
View (azurermps-6.3.0)

docs-conceptual/azurermps-6.3.0/toc.yml
:white_check_mark:Succeeded
View (azurermps-6.3.0)

docs-conceptual/azurermps-6.3.0/uninstall-azurerm-ps.md
:white_check_mark:Succeeded
View (azurermps-6.3.0)

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

#sign-off
@sdwheeler Ready for your final review

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.gitignore
:white_check_mark:Succeeded

docs-conceptual/azurermps-5.7.0/azurerm-ps-in-docker.md
:white_check_mark:Succeeded
View (azurermps-5.7.0)

docs-conceptual/azurermps-5.7.0/install-azurerm-ps.md
:white_check_mark:Succeeded
View (azurermps-5.7.0)

docs-conceptual/azurermps-5.7.0/install-azurermps-maclinux.md
:white_check_mark:Succeeded
View (azurermps-5.7.0)

docs-conceptual/azurermps-5.7.0/other-install.md
:white_check_mark:Succeeded
View (azurermps-5.7.0)

docs-conceptual/azurermps-5.7.0/toc.yml
:white_check_mark:Succeeded
View (azurermps-5.7.0)

docs-conceptual/azurermps-5.7.0/uninstall-azurerm-ps.md
:white_check_mark:Succeeded
View (azurermps-5.7.0)

docs-conceptual/azurermps-6.3.0/azurerm-ps-in-docker.md
:white_check_mark:Succeeded
View (azurermps-6.3.0)

docs-conceptual/azurermps-6.3.0/context-persistence.md
:white_check_mark:Succeeded
View (azurermps-6.3.0)

docs-conceptual/azurermps-6.3.0/install-azurerm-ps.md
:white_check_mark:Succeeded
View (azurermps-6.3.0)

docs-conceptual/azurermps-6.3.0/install-azurermps-maclinux.md
:white_check_mark:Succeeded
View (azurermps-6.3.0)

docs-conceptual/azurermps-6.3.0/other-install.md
:white_check_mark:Succeeded
View (azurermps-6.3.0)

docs-conceptual/azurermps-6.3.0/toc.yml
:white_check_mark:Succeeded
View (azurermps-6.3.0)

docs-conceptual/azurermps-6.3.0/uninstall-azurerm-ps.md
:white_check_mark:Succeeded
View (azurermps-6.3.0)

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

#sign-off
@sdwheeler Should be ready for final review now.


Order of certs produced by openssl pkcs7 -inform DER -print_certs

I created a dummy chain of 4 levels of certificates with XCA.
Then I produced the .p7b of the chain.
If I run:
openssl pkcs7 -inform DER -in chain.p7b -print_certs
it outputs the certificates in the following order:
leaf
int2
int1
root
Is this intended?
Should the order be reversed?
Should it depend on the DER?
Thanks.

They are printed in the order returned from the load.

Thanks.


Buttons in bottomView don't seem to work

Adding a button to the cell's bottomView doesn't seem to respond to taps. Any ideas on what's causing this?

Found a solution which is to override hitTest in the ElongationCell subclass:
    override func hitTest(_ point: CGPoint, with event: UIEvent?) -> UIView? {
        let scalableViewContainsPoint = scalableView.frame.contains(point)
        let backViewContainsPoint = bottomView.frame.contains(point)

        if scalableViewContainsPoint {
            return scalableView.hitTest(convert(point, to: scalableView), with: event)
        }

        if backViewContainsPoint {
            return bottomView.hitTest(convert(point, to: bottomView), with: event)
        }
        return nil
    }

@v-ken thank you. If you want to contribute, please, make PR request and I'll merge it. Note that you need to change this method both in ElongationCell & ElongationHeader files.

@Jauzee Thanks for the quick response. Unfortunately I don't think it's that simple. The solution I posted may cause adverse effects to gestureRecognizerSwiped implementation in ElongationViewController. The overridden hitTest in the cell was explicitly returning scalableView and bottomView which is then used here:
https://github.com/Ramotion/elongation-preview/blob/master/ElongationPreview/Source/ViewControllers/ElongationViewController.swift#L265-L293
I don't yet have a solution for this at this stage.

Just created a pull request:
https://github.com/Ramotion/elongation-preview/pull/2

@v-ken thank you for contributing!


Ad-hoc modifications with provenance tracking

Perhaps this breaks some fundamental assumption about artifact provenance, but it would be useful to still be able to keep track of the provenance of an artifact after some processing has been applied.
For example, I have a tree artifact that I load, set all the missing branch lengths to zero, and then save it back to an artifact. The resulting artifact has now lost all the provenance information. It would be useful if I could annotate in the provenance, that the changes that I made have the intention of "Set missing branch lengths to zero".
a = Artifact.load('tree.qza')
t = a.view(TreeNode)

for n in t.traverse():
    if n.length is None:
        n.length = 0

patched_tree = Artifact.import_data('Phylogeny[Rooted]', t)
patched_tree.save('tree.patched.qza')

Maybe there could be some kind of action where you could say:
patched_results = qiime2.UnknownAction("Describe a thing",
                                       inputs=[a, b, c],
                                       results=[('TypeX', view_x), ('TypeY', view_y)])

Or maybe the import has already been done, so it would take the patched_tree in your example.

Something like this would be rather handy, my overall goal is to be able to make a note that I made a change, but that the originating object has a particular history associated with it. This should likely be highlighted somehow in the view.qiime2.org.