Add namespace property in generatorRef for ClusterExternalSecret

Is your feature request related to a problem? Please describe.
I have to use the same generator in all user namespaces(multi tenant cluster).
Currently ClusterExternalSecret can distribute ExternalSecret but generatorRef can not accept namespace.
So I have to create the same generator and its auth secret in every namespaces for now.
For instance, I have to create the following manifests in all namespaces.
Especially I do not want to create secrets-aws-iam-cred in all namespaces for security.
apiVersion: v1
kind: Secret
metadata:
  name: secrets-aws-iam-cred
  namespace: every-namespace
type: Opaque
data:
  AWS_ACCESS_KEY_ID: ******
  AWS_SECRET_ACCESS_KEY: ******
---
apiVersion: generators.external-secrets.io/v1alpha1
kind: ECRAuthorizationToken
metadata:
  name: aws-ecr-token-generator
  namespace: every-namespace
spec:
  region: ap-northeast-1
  auth:
    secretRef:
      accessKeyIDSecretRef:
        name: secrets-aws-iam-cred
        key: AWS_ACCESS_KEY_ID
      secretAccessKeySecretRef:
        name: secrets-aws-iam-cred
        key: AWS_SECRET_ACCESS_KEY
---
apiVersion: external-secrets.io/v1beta1
kind: ExternalSecret
metadata:
  name: aws-ecr-token
  namespace: every-namespace
spec:
  refreshInterval: "3h"
  target:
    name: aws-ecr-token
  dataFrom:
  - sourceRef:
      generatorRef:
        apiVersion: generators.external-secrets.io/v1alpha1
        kind: ECRAuthorizationToken
        name: aws-ecr-token-generator

Describe the solution you'd like
Add namespace property in generatorRef struct for ClusterExternalSecret.
It will be passed to ExternalSecret and use it only if the ExternalSecret is created by ClusterExternalSecret.

ClusterExternalSecret

apiVersion: external-secrets.io/v1beta1
kind: ClusterExternalSecret
metadata:
  name: aws-ecr-token
spec:
  externalSecretName: aws-ecr-token
  namespaceSelector:
    matchLabels:
      aws-ecr-token: enabled
  refreshTime: "1m"
  externalSecretSpec:
    refreshInterval: "3h"
    target:
      name: aws-ecr-token
    dataFrom:
    - sourceRef:
        generatorRef:
          apiVersion: generators.external-secrets.io/v1alpha1
          kind: ECRAuthorizationToken
          name: aws-ecr-token-generator
          namespace: kube-system # Add

ExternalSecret created by ClusterExternalSecret

apiVersion: external-secrets.io/v1beta1
kind: ExternalSecret
metadata:
  creationTimestamp: "2024-03-20T13:54:40Z"
  generation: 1
  name: aws-ecr-token
  namespace: default
  ownerReferences:
  - apiVersion: external-secrets.io/v1beta1
    blockOwnerDeletion: true
    controller: true
    kind: ClusterExternalSecret
    name: aws-ecr-token
    uid: ebec1975-2d04-4e08-90a2-29ceaa5a9949
  resourceVersion: "94778801"
  uid: 47bd5d56-4e26-4be1-95ed-2c1420f81293
spec:
  dataFrom:
  - sourceRef:
      generatorRef:
        apiVersion: generators.external-secrets.io/v1alpha1
        kind: ECRAuthorizationToken
        name: aws-ecr-token-generator
        namespace: kube-system # Use only if owner is ClusterExternalSecret
  refreshInterval: 3h0m0s
  secretStoreRef:
    kind: ClusterSecretStore
    name: aws-ecr-token-generator
  target:
    creationPolicy: Owner
    deletionPolicy: Retain
    name: aws-ecr-token
status:
  conditions:
  - lastTransitionTime: "2024-03-20T13:54:40Z"
    message: could not get secret data from provider
    reason: SecretSyncedError
    status: "False"
    type: Ready

Describe alternatives you've considered
I struggled to find a workaround but fail #3289

The same as https://github.com/external-secrets/external-secrets/issues/2009. We need cluster-scoped Generators to support it.

@shuheiktgw Thank you for replying.
Cluster-scoped generators are great! Is it planned or someone is already working on it?
I thought creating new cluster-scoped APIs will be a hard work and came up with the idea cross-namespaced reference for ExternalSecret created by ClusterExternalSecret.

@shuheiktgw Is there any plan to release this feature any time soon? Either cluster-scoped generators or namespace parameter?
Thanks

I stepped down from a maintainer a while ago so you may want to reach out to the current maintainers🙂

Yeah, it has came up a couple of times. Generally we want to keep the isolation mechanism that namespaces provide.
Allowing cross-namespace references breaks this idiom, it opens the door for unprivileged data access and eventually exfiltration.
With namespaced resources we can easily leverage rbac for access control. Cross-namespace references make it hard/impossible to properly isolate it.
Hope that makes sense.
 :) Are you aware that it's a thing now?

@konradasb Are you okay with the proposed solution? It has been released with 0.11.0.

@Skarlso Sorry but I didn't know that😳 Thank you for a great job, I'll try it🚀


Camera is stretched vertically making the card unscannable.

Device: Samsung Galaxy S4
OS: Android 21
CardScan version: 1.0.5001
The gray piece is the same size of a normal visa credit card, its just a template so I dont leak sensitive information.

We have a Note 3 that is from the same era as the s4, let me take a look at it today to see if I can reproduce this.

@kingst Whats the status?

The Note 3 worked fine and I've been looking through our data and I can't seem to find any data on the S4, so I don't think it's that popular of a phone anymore. I will say that the GPU on an S4 is likely to struggle with our machine learning model so it might take a long time, but it's also true that the aspect ratio needs to be right or else or model doesn't work.

We just made a number of changes that should fix this. You'll still see a bit of distortion in the cards on your S4 but it doesn't affect the machine learning pipeline which uses the video (the on screen content is just a preview). The version that should be better is 1.0.5102, let me know if it's better.


Locking the thread inside a observe causes all Realm instances to hang

Goals

Be able to use the observe without locking other Realm instances.

I notice this happening in our application when the dataset was bigger than 1000 items, the application started getting slow and hanging for 1.5 seconds when interacting with the objects of the example below.
I was thinking about dispatching this workload to another thread, but this is not allowed since it's not easily possible to transfer live objects between threads.
Expected Results
When doing a heavy work inside the closure from the observe function, other Realm instances should not hang.
Actual Results
All Realm instances get stuck while doing a locking process inside the observer.
Steps for others to Reproduce

Code Sample

Given the code below in a part of the project:
notificationToken = objects?.observe { (changes: RealmCollectionChange) in
      Thread.sleep(forTimeInterval: 5.0) //Simulates a heavy work with objects
}

By doing the code above the application gets very slow, but only because the realm instances are hanging, even work that is being done outside the main thread gets stuck.
If you call this very shortly after:
let realm = try! Realm()
realm.refresh() //hangs for 5 seconds

Version of Realm and Tooling

Realm framework version: 5.4.3 and older
Realm Object Server version: nil
Xcode version: 11.7
iOS/OSX version: All
Dependency manager + version: Submodules

Hi @ppamorim Observing on a user defined should solve your issue. This thread would be solely used for observing and performing your processing tasks.
On main thread
try! self.realm.write {
                let dog = Dog()
                dog.name = "Jim"
                self.realm.add(dog)
            }

On your defined queue
self.notificationToken = self.realm.objects(Dog.self).observe(on: self.myQueue) { (change: RealmCollectionChange<Results<Dog>>) in
                Thread.sleep(forTimeInterval: 5.0)
            }

@leemaguire That's far from ideal.

By default change notifications are delivered on the thread which you called observe() on. Sleeping for five second (or doing an equivalent amount of work) within the notification callback with then naturally occupy that thread for that amount of time. There's no reason why this should block other threads, though. We don't hold any locks while calling the notification callback and the notification system is specifically designed to support different threads picking up notifications at different rates (or even a thread with observers hanging forever inside a callback).

"There's no reason why this should block other threads," this enter in conflict with the fact that realm objects are not thread-safe and I cannot use the very same query. If there is a collection of items stored in the main thread it's not possible to attach an observer and listen to it without locking other realm instances. It's what I am experiencing.
It's not really blocking other threads, but it's the functions refresh and write keeping locked until the notification block completes. I am not sure exactly what is happening.
My solution was to isolate the required data, transform to swift objects and process the data in another thread.

Closing this issue as it has been inactive for some time now. If you have any other questions please open a new issue.


[BUG] Different nodes return different top K vectors with same model

What is the bug?
I have deployed the neural search plugin on a cluster with 2 nodes. I see that the k variable returns different vectors for each node for the same index using the same model and same query. This results in inconsistent results.
How can one reproduce the bug?
I have configured the fields in the mapping as follow:
      "_fulltext_source": {
          "type": "keyword",
          "ignore_above": 8191,
          "index": false,
          "store": false
      },
      "_fulltext_vectorized": {
          "type": "nested",
          "properties": {
            "knn": {
              "type": "knn_vector",
              "dimension": 768,
              "method": {
                "name": "hnsw",
                "engine": "lucene"
              }
            }
        }
      }

This is the query that I am running.
{
  "query": {
    "bool": {
      "must": {
        "bool": {
          "should": [
            {
              "nested": {
                "query": {
                  "neural": {
                    "_fulltext_vectorized.knn": {
                      "query_text": "cowboy",
                      "model_id": "T5fHiIoBJC8u9rWiXx8s",
                      "k": 10
                    }
                  }
                },
                "path": "_fulltext_vectorized",
                "score_mode": "max"
              }
            }
          ]
        }
      }
    }
  }
}

Make sure you have 2 nodes and an index of 1 shard with a replica of 1.
Then run above query in the request body with the preference parameter again each node and see that the results are different.

The expected behavior is that the results returned for each node should be the same. The top embeddings should be retrieved and used, consistently, on all shards for the same index.
What is your host/environment?
Running OpenSearch in Docker containers with latest version of Ubuntu.
Do you have any screenshots?
Here you see the different results for the same query, but running it against different nodes.
The top hit is not so good here.

While the top hit here is good and it has a higher score.

Do you have any additional context?
A temporary workaround is to increase k to a higher number.

@juntezhang can you add some sample data which can help reproducing the issue?

@juntezhang
This behavior happen because the underline graph data structure is different in primary and replicas. To provide some context:
The way replication happens during the indexing of document is:

First the document is indexed in primary shard.
Once the primary acks the document is sent to replica shard.
As this is document level replication and the underline data structure(hnsw) is not build consistent this can happen.

Solution:
Opensearch has a segment replication feature which rather than sending documents to replicas the whole segment is copied primary shard to replica.
Ref: https://opensearch.org/docs/latest/tuning-your-cluster/availability-and-recovery/segment-replication/index/
Blog: https://opensearch.org/blog/segment-replication/

@juntezhang no I didn't reproduce the bug. But I am aware of this scenario can happen based on my understanding of HNSW and Opensearch. I also know that with Segment replication this should not happen, if the replicas and primaries are in sync, because the same graph is copied to the replicas.

I will check out if this bug is reproducable with an index with segment replication.

Yes please do try that.


ocis init --diff creates a temp file that is not deleted when no patch file is created

When running ocis init --diff and no changes are detected, a ocis.yaml.tmp is created but not deleted - compared when a diff is found and a patch is created, the temp file gets removed automatically.
@dragonchaser as you have created the diff code, mind to fix this 😃

This is a bug, needs fixing.


Delete embed properties which are set to "" or empty array

Discord's api now doesn't support parsing of json embeds with empty properties like this:
{"embed":{"title":"","description":"Hello! This won't work because title is empty and thumbnail url too!","color":4321431,"thumbnail":{"url":""}}}
So, I suggest to delete the property instead of letting it be "" (or empty array in case of fields), because discord doesn't support this now and this only causes problems.
It will look like this: {"embed":{"description":"Yay! No empty properties, now it works!","color":4321431}}
Same thing for content

Do you mind elaborating a bit? It seems to work. Except the text that is "" is not eg. the title is not shown by Discord.
This is the same behavior with the embed builder. Perhaps I've miss understood?
Test code:
import requests
import json

webhook_url = ""

requests.post(
    webhook_url,
    headers={ "Content-Type": "application/json" },
    data=json.dumps({
        "embeds": [{
            "title": "",
            "description": "Hello! This won't work because title is empty and thumbnail url too!",
            "color": 4321431,
            "thumbnail": { "url": "" }
        }]
    })
)

Discord output:

It might work with webhooks, however on discord.js lastest version does not work with the client itself not a webhook

Added on commit bd9406169248dc93be5354ad807a085e3e632d89
Here is how it works, once you enter a key with an empty valid using the JSON editor, the editor will not be updated so not to interrupt, instead, the file JSON will not have that key.
You can test it by writing an empty key, then print json in console or switch to the GUI tab then back to the JSON editor, the key will not be there.
Also in the commit, the same will happen to keys that are not accepted by discord.
Test it out with this test data
{
    "content": "Lorem ipsum",
    "embeds": [
        {
            "title": "",
            "description": "Lorem ipsum dolor sit amet, consectetur adipiscing elit",
            "color": 4321431,
            "number": 1234,
            "text": "Hello",
            "emptyObject": {},
            "thumbnail": {
                "url": ""
            },
            "author": {
                "name": "Me",
                "icon_url": "https://cdn.discordapp.com/embed/avatars/0.png",
                "icon": "https://cdn.discordapp.com/embed/avatars/0.png",
                "url": "https://discord.com"
            },
            "test": {
                "test1": {
                    "test2": ""
                }
            },
            "testing": {
                "testing1": {
                    "testing2": "testing3"
                }
            },
            "fields": [
                {
                    "name": "Field name",
                    "value": "Field value",
                    "inline": true
                },
                {
                    "name": "Another Field name",
                    "value": "And another value",
                    "inline": true,
                    "outline": false
                }
            ]
        }
    ]
}


Section 3.1.1 - clear up data re. Mahomes and Manning

In trying to explain that load_player_stats() brings in passing, rushing, and receiving yards for each player, I quickly switched to Peyton Manning to showcase that the data provides, for example, his rushing EPA (-0.740, if you were wondering) which is a stat that is not all that useful considering what we know about Peyton.
I switch back to using Mahomes in the examples right after. I think doing a bit of editing in that section will make it a bit more clear what I am trying to highlight about the data in that getting information pertaining to Mahomes' rushing EPA is perhaps relevant (especially compared to Peyton).

Completed in page proofs.


[Question] Is there a way to enable/disable key mapping for certain modes

Not if you use the default mappings but you do that if you set custom mappings. Check the README.


ghttp: add cookie security configurations

Add configurations for cookie security properties:

HttpOnly
Secure
SameSite

Pending Unit Test
Some questions:

Should I keep backward compatibility on Cookie.SetCookie's httpOnly parameter?
How to handle multiple optional parameters to keep code clean and clear?

Codecov Report

Merging #1549 (c91b839) into master (99455e3) will decrease coverage by 0.02%.
The diff coverage is 82.35%.

@@            Coverage Diff             @@
##           master    #1549      +/-   ##
==========================================
- Coverage   69.79%   69.77%   -0.03%
==========================================
  Files         438      438
  Lines       42421    42452      +31
==========================================
+ Hits        29609    29621      +12
- Misses      10917    10934      +17
- Partials     1895     1897       +2

Flag
Coverage Δ

go-1.15
?

go-1.16
?

go-1.17
69.77% <82.35%> (+<0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

net/ghttp/ghttp_server_config.go
59.03% <ø> (ø)

net/ghttp/ghttp_server_config_cookie.go
53.84% <57.14%> (+3.84%)
:arrow_up:

net/ghttp/ghttp_server_cookie.go
85.60% <100.00%> (+5.04%)
:arrow_up:

os/gspath/gspath_cache.go
84.21% <0.00%> (-10.53%)
:arrow_down:

net/gtcp/gtcp_pool.go
34.09% <0.00%> (-4.55%)
:arrow_down:

os/glog/glog_logger_rotate.go
65.85% <0.00%> (-1.47%)
:arrow_down:

container/gtree/gtree_avltree.go
97.21% <0.00%> (-0.60%)
:arrow_down:

Continue to review full report at Codecov.

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


feat: port Analysis.Convex.Function

I don't know what to do about the excessive use of etaExperiment. Is it better to get rid of them (i.e., try to find another hack to work around 2074) or keep it there and deal with it once lean4 has a solution?

I don't know what to do about the excessive use of etaExperiment.

It is better to keep the eta enablers there.

bors merge


🚀 Feature Request: Add  Github & Open Collective donation buttons to the app

Problem Overview 👁🗨

Users currently do not have a way to donate to Grey Software via Github & Open Collective through the Material Math app
What would you like? 🧰

Creation of Github & Open Collective donation buttons next to the Paypal donation button.
What alternatives have you considered? 🔍

NA
Additional details ℹ

Open Collective link: https://opencollective.com/grey-software
Github sponsors link: https://github.com/sponsors/grey-software

A naive implementation of simply adding more buttons next to the existing paypal button on the title page causes the buttons to render beneath the title, decreasing readability.
I think we should create a separate page for the donations. That way, we can easily add more donation buttons later without affecting the UI. I'm working on that now.

I can't find a free CDN distributing the icons for PayPal, OpenCollective, and Github. I'll just save a low resolution 28x28 version of each logo in the statics/icons directory.

Hi, I think this issue has been resolved?


Respect tags even when the backend does not support them

Coverage remained the same at 100.0% when pulling c345a1bd796b74255091b14f44b34d42c2c55c4b on respect-tags into 391c4d8dfef125ca7a5682aa00ab43cac76cdbfb on master.


Revived Pattern: Number Series

Revived from the archive, and with some useful added bits from having had to explain this system to several people in the past year.

Agreed.  Call this a v1 and maybe I can crosscheck against Kennie's guides for v2  😄


Feature/job openings

This change is

Reviewed 19 of 19 files at r1.
Review status: all files reviewed at latest revision, 10 unresolved discussions.

byuSuite/Apps/JobOpenings/controller/JobOpeningsCategoriesViewController.m, line 39 at r1 (raw file):

#pragma mark - Navigation

- (void)prepareForSegue:(UIStoryboardSegue *)segue sender:(id)sender {

Run formatter

byuSuite/Apps/JobOpenings/controller/JobOpeningsDetailViewController.m, line 17 at r1 (raw file):

@property (weak, nonatomic) IBOutlet UILabel *idLabel;
@property (weak, nonatomic) IBOutlet UILabel *opensLabel;
@property (weak, nonatomic) IBOutlet UIWebView *descriptionView;

Check to see if we should use the ByuWebView. I think there is one, but I don't know what functionality it adds...

byuSuite/Apps/JobOpenings/controller/JobOpeningsDetailViewController.m, line 26 at r1 (raw file):

    [super viewDidLoad];

    self.titleLabel.text = self.posting.postingTitle;

Is this different from the navigation title?
self.title = self.posting.postingTitle;

byuSuite/Apps/JobOpenings/controller/JobOpeningsDetailViewController.m, line 32 at r1 (raw file):

    NSString *htmlString = [self.posting.jobDescription stringByDecodingHTMLEntities];
    [self.descriptionView.scrollView setShowsVerticalScrollIndicator:YES];
    [self.descriptionView.scrollView setShowsHorizontalScrollIndicator:YES];

Do we need the horizontal? Can we put this in the storyboard?

byuSuite/Apps/JobOpenings/controller/JobOpeningsDetailViewController.m, line 37 at r1 (raw file):

-(void)webViewDidFinishLoad:(UIWebView *)webView {
    [self.descriptionView.scrollView flashScrollIndicators];

What is this?

byuSuite/Apps/JobOpenings/controller/JobOpeningsFamilyViewController.m, line 67 at r1 (raw file):

        return @"";
    } else if (self.families.count == 0) {
        return @"No Job Openings";

Instead of this, let's actually create a footer with the text in the storyboard. Then we'll just set the visibility to true if no families come back

byuSuite/Apps/JobOpenings/controller/JobOpeningsPostingsViewController.m, line 23 at r1 (raw file):

    [super viewDidLoad];

    self.navigationItem.title = self.family.jobTitle;

You can just use self.title here

byuSuite/Apps/JobOpenings/model/JobOpeningsClient.h, line 16 at r1 (raw file):

+(void)getJobCategoriesWithCallBack:(void (^)(NSArray<JobOpeningsCategory *> *categories, ByuError *error))callback;
+(void)getJobPostingsWithCategory:(JobOpeningsCategory *)category andCallBack:(void (^)(NSArray<JobOpeningsFamily *> *families, ByuError *error))callback;

If you only need the id, just pass the id in

byuSuite/Apps/JobOpenings/model/JobOpeningsFamily.m, line 20 at r1 (raw file):

    me.jobTemplateId = [dictionary nullSafeObjectForKey:@"job_template_id"];
    me.jobTitle = [dictionary nullSafeObjectForKey:@"job_title"];
    id listingData = [dictionary nullSafeObjectForKey:@"job_listing"];

Comment about why you had to do this

byuSuite/byuSuite.xcodeproj/project.pbxproj, line 3750 at r1 (raw file):

				75DCE5BB1799CE25006E14B5 /* ByuJobOpeningsFamilyTableViewController.h */,
				75DCE5BC1799CE25006E14B5 /* ByuJobOpeningsFamilyTableViewController.m */,
				75DCE5BD1799CE25006E14B5 /* ByuJobOpeningsFamilyTableViewController.xib */,

Why didn't things like this get removed? Double check that you removed all the old stuff

Comments from Reviewable

Review status: 11 of 19 files reviewed at latest revision, 10 unresolved discussions.

byuSuite/Apps/JobOpenings/controller/JobOpeningsCategoriesViewController.m, line 39 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
Run formatter

Done.

byuSuite/Apps/JobOpenings/controller/JobOpeningsDetailViewController.m, line 17 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
Check to see if we should use the ByuWebView. I think there is one, but I don't know what functionality it adds...

Looking through ByuWebView I don't think there's any added functionality for just displaying an HTML string like we're doing here.

byuSuite/Apps/JobOpenings/controller/JobOpeningsDetailViewController.m, line 26 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
Is this different from the navigation title?
self.title = self.posting.postingTitle;

No this is the label I created in the storyboard for the job title. The navigation title is set to Posting in the old code so I just set that in the Storyboard itself. It wouldn't work to use this title as the navigation title since these titles are often quite long.

byuSuite/Apps/JobOpenings/controller/JobOpeningsDetailViewController.m, line 32 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
Do we need the horizontal? Can we put this in the storyboard?

It can't be set in the storyboard. We don't need horizontal.

byuSuite/Apps/JobOpenings/controller/JobOpeningsDetailViewController.m, line 37 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
What is this?

This displays the scroll bar for a second or two after the text is loaded so the user knows they can scroll. Otherwise it is not so apparent.

byuSuite/Apps/JobOpenings/controller/JobOpeningsFamilyViewController.m, line 67 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
Instead of this, let's actually create a footer with the text in the storyboard. Then we'll just set the visibility to true if no families come back

Done.

byuSuite/Apps/JobOpenings/controller/JobOpeningsPostingsViewController.m, line 23 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
You can just use self.title here

Done.

byuSuite/Apps/JobOpenings/model/JobOpeningsClient.h, line 16 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
If you only need the id, just pass the id in

Done.

byuSuite/Apps/JobOpenings/model/JobOpeningsFamily.m, line 20 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
Comment about why you had to do this

Done.

byuSuite/byuSuite.xcodeproj/project.pbxproj, line 3750 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
Why didn't things like this get removed? Double check that you removed all the old stuff

I haven't removed the old stuff yet. I was going to do that at the very end in case I needed to review the old code one last time.

Comments from Reviewable

Reviewed 8 of 8 files at r2.
Review status: all files reviewed at latest revision, 2 unresolved discussions.

byuSuite/Apps/JobOpenings/controller/JobOpeningsDetailViewController.m, line 37 at r1 (raw file):
Previously, ecbrady (Erik Brady) wrote…
This displays the scroll bar for a second or two after the text is loaded so the user knows they can scroll. Otherwise it is not so apparent.

Put a comment

byuSuite/byuSuite.xcodeproj/project.pbxproj, line 3750 at r1 (raw file):
Previously, ecbrady (Erik Brady) wrote…
I haven't removed the old stuff yet. I was going to do that at the very end in case I needed to review the old code one last time.

Deal. Let's nuke it now. The rest look good :)

Comments from Reviewable

Review status: 17 of 37 files reviewed at latest revision, 2 unresolved discussions.

byuSuite/Apps/JobOpenings/controller/JobOpeningsDetailViewController.m, line 37 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
Put a comment

Done.

byuSuite/byuSuite.xcodeproj/project.pbxproj, line 3750 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
Deal. Let's nuke it now. The rest look good :)

Done.

Comments from Reviewable

Reviewed 20 of 20 files at r3.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Scaffolder file system action examples

Hey, I just made a Pull Request!

Added examples for fs:delete and fs:rename scaffolder actions
Added tests for the examples
Made it so that the examples code structure aligns with this recently approved PR: https://github.com/backstage/backstage/pull/18831
This PR contributes to this issue: https://github.com/backstage/backstage/issues/18832

:heavy_check_mark: Checklist

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

Thanks @freben 🙂


No module named jintrospect

Not sure what triggers it, but after Ghidra has been running a while I come across this issue where when the python window tries to do the Resetting python pop-up an error occurs (that's when I notice the issue at least) and then from then on python scripts will fail as well as attempting to use the python window.
Closing the tool doesn't seem to fix, only a complete restart of Ghidra.
windows 10
ghidra 9.1.2
java 11.0.2
python 2.7.16

Is there any chance your user temp directory is getting cleared?

good thinking, I will check my APPDATA\Local\Temp\mumbel-Ghidra\jython_cachedir\python-src next time it happens (I restarted Ghidra as I was submitting this, so its all fresh ATM).
I'd guess it would have to be mumbel-Ghidra related as I do have month old files in my Temp

It's worth checking but since you are on Windows, I wouldn't expect it to be a problem.  I know macOS will just delete files right out from under you in its temp directory.

I have also come across weirder issues with jython_cachedir not having the correct permissions maybe, like ghidra is unable to create the directory (so maybe its a mumbel-Ghidra permissions issue) which ultimately I manually delete mumbel-Ghidra, but I've only had that happen once and no clue what caused it or why I noticed it (in terms of ghidra messages)

I've also encountered this a few times. It was usually when I'd leave it ghidra and the python interpreter running and go to class. It seems to happen after being idle for long periods of time.

I have it sitting all day today.

@ryanmkurtz  its more on the order of weekly, probably bi-weekly.  I basically never exit ghidra or reboot that machine

Ah ok.  I just went in and deleted the jython_cachedir from the temp directory and when I reset the interpreter I got a popup saying:
"Resetting Python... - Uncaught Exception: ImportError: No module named jintrospect"
Is that the exact error?

Correct. (at Py.java:334)

@jeff5, do you happen to know if Jython ever cleans out its cachedir?

I don't think so, on its own initiative. It will create a new cache file for a JAR if it finds it can't read an existing one. (They number up, ISTR.)
We made a couple of changes in 2.7.2 to the cache, in location (cwd) and permissions (user only). This was in response to a CVE. You seem to be choosing your own location already, which was sensible. Hard to see why the permissions would be wrong if they work at all.
Obviously you can toast it when you need to if Jython is stopped. Deleting and remaking files when running always seems to be a problem on Windows. Files are open unexpectedly and can't be deleted. Sometimes it's the code. In other cases I resort to blaming the AV, meaning it's a mystery.

Ok, thanks for the response!

Okay, finally hit.  Sure enough my jython_cachedir (created 5/5/20) has 0 bytes, 0 Files, 0 Folders.

Yes, about 2 weeks.  Are you willing to run procmon with a filter on that directory so we can see who's doing the delete?

Yeah I think that should be okay
Think a filter w/ capturing path contains jython_cachedir is sufficient?

I was just playing with it and if that is your only filter, you will get a ton of extra stuff.  To further reduce it you can add a 2nd filter that is Operation is SetDispositionInformationFile

It does look like Jython at least has the capability to delete a cache file.  I'm not sure what triggers it though:
https://github.com/jythontools/jython/blob/b8d7aa4cee50c0c0fe2f4b235dd62922dd0f3f99/src/org/python/core/packagecache/CachedJarsPackageManager.java#L456-L472
private Map<String, String> readCacheFile(JarXEntry entry, String jarcanon) {
    String cachefile = entry.cachefile;
    long mtime = entry.mtime;

    debug("reading cache of ''{0}''", jarcanon);

    DataInputStream istream = null;
    try {
        istream = inOpenCacheFile(cachefile);
        String old_jarcanon = istream.readUTF();
        long old_mtime = istream.readLong();
        if ((!old_jarcanon.equals(jarcanon)) || (old_mtime != mtime)) {
            comment("invalid cache file: {0} for new:{1}({3}), old:{2}({4})", cachefile, jarcanon,
                    old_jarcanon, mtime, old_mtime);
            deleteCacheFile(cachefile);
            return null;
        }
        ...

You should be able to see that comment() message if you launch ghidra with -Dpython.verbose=comment

so annoyed with myself right now, I had that running this whole time, but had some network issues last night and killed everything.  Completely forgot about this setup when I started stuff this morning

Doh!

Ah ok. I just went in and deleted the jython_cachedir from the temp directory and when I reset the interpreter I got a popup saying:
"Resetting Python... - Uncaught Exception: ImportError: No module named jintrospect"
Is that the exact error?
For me personally, I get "Resetting Python Uncaught Exception: java.lang.NullPointerException".

A new issue would be good, especially if you can provide the entire stack trace.

came into a procmon exception on my login screen and the jython directory gone.  Yay! /s

This ticket is cursed!  Maybe there is a procmon log file?

Doesn't look like there is by default.

Instead of fighting this why not just move jython_cachedir to Application.getUserSettingsDirectory()/"dev" which is where the ghidra scripts go?

If there is some piece of code in ghidra or libs that deletes this directory and its not just my system, what does moving the directory solve?

If there is some piece of code in ghidra or libs that deletes this directory and its not just my system, what does moving the directory solve?

https://github.com/NationalSecurityAgency/ghidra/blob/bdbbca30a155502b79a38874c0186a7ac496dfeb/Ghidra/Framework/Generic/src/main/java/ghidra/framework/Application.java#L601-L615
On windows the user temp and user cache directories are the same. The documentation notes that the returned temp directory can removed by periodic system cleanup. So it's not the best place to be putting these to begin with.
Also, moving the directory would have the benefit of only having to wait a bit when starting the python interpreter for the first time used in ghidra as opposed to the first time every ghidra session.

On windows the user temp and user cache directories are the same. The documentation notes that the returned temp directory can removed by periodic system cleanup. So it's not the best place to be putting these to begin with.

That's backwards.  On Linux and Mac they are the same.  On Windows, they are 2 distinct locations.  We've seen Mac rip temp files right out from underneath you in the past, but I wasn't aware of Windows doing that unless you invoke a Disk Cleanup.
Like @mumbel said, the concern is that if Jython is deleting the directory, moving it won't help.  If @mumbel wasn't willing to help with the debug, I would probably just move the jython cache directory to the cache dir (which would only do something different on Windows) and I would also have the interpreter's reset action check for the existence of the directory each time, in case it is Jython that is deleting it.  That may end up being what we do anyway, but it would be great to know who is actually deleting the files so we can understand the problem we are implementing the solution for.

@ryanmkurtz oh yes, my mistake, it appears I remembered incorrectly.
Why does the python interpreter use the user temp directory when ghidra scripts use %USERPROFILE%\.ghidra\.ghidra_9.2_DEV\dev\ghidra_scripts\bin for example? Just seems to be a bit inconsistent.

Yes, that is inconsistent.  Adding too much stuff to the "settings" directory is bad for systems with roaming profiles.  We'll have to consider this as part of the final solution, once we have all the info.

Well, this just occurred after running ghidra for a few hours. I was actively using ghidra and then ran the decompiler parameter id and calling convention ids and then came back about 45 minutes later. Next time I tried to launch a python script I got the error. I had the debugger running So I have some warnings that might be helpful?
log
Jun 21, 2020 11:03:49 PM org.python.core.packagecache.SysPackageManager warning
WARNING: failed to write cache for 'C:\Users\astre\.gradle\caches\modules-2\files-2.1\org.python\jython-standalone\2.7.2\15592c29538abd36d15570eda9fa055ed1a618ba\jython-standalone-2.7.2.jar' (The system cannot find the path specified)
Jun 21, 2020 11:03:49 PM org.python.core.packagecache.SysPackageManager warning
WARNING: failed to write index file (The system cannot find the path specified)
Jun 21, 2020 11:03:51 PM org.python.core.packagecache.SysPackageManager warning
WARNING: failed to write cache for 'C:\Users\astre\.gradle\caches\modules-2\files-2.1\org.python\jython-standalone\2.7.2\15592c29538abd36d15570eda9fa055ed1a618ba\jython-standalone-2.7.2.jar' (The system cannot find the path specified)
Jun 21, 2020 11:03:51 PM org.python.core.packagecache.SysPackageManager warning
WARNING: failed to write index file (The system cannot find the path specified)
ERROR make_strings.py: Error running script: make_strings.py
org.python.core.PyException: ImportError: No module named jintrospect  (RunScriptTask.java:65)
org.python.core.PyException: ImportError: No module named jintrospect
	at org.python.core.Py.ImportError(Py.java:329) ~[jython-standalone-2.7.2.jar:2.7.2]
	at org.python.core.imp.import_first(imp.java:1200) ~[jython-standalone-2.7.2.jar:2.7.2]
	at org.python.core.imp.load(imp.java:1016) ~[jython-standalone-2.7.2.jar:2.7.2]
	at ghidra.python.GhidraPythonInterpreter.<init>(GhidraPythonInterpreter.java:120) ~[Python.jar:?]
	at ghidra.python.GhidraPythonInterpreter.get(GhidraPythonInterpreter.java:89) ~[Python.jar:?]
	at ghidra.python.PythonScript.runInNewEnvironment(PythonScript.java:110) ~[Python.jar:?]
	at ghidra.python.PythonScript.run(PythonScript.java:48) ~[Python.jar:?]
	at ghidra.app.script.GhidraScript.executeNormal(GhidraScript.java:365) ~[Base.jar:?]
	at ghidra.app.script.GhidraScript.doExecute(GhidraScript.java:220) ~[Base.jar:?]
	at ghidra.app.script.GhidraScript.execute(GhidraScript.java:198) ~[Base.jar:?]
	at ghidra.app.plugin.core.script.RunScriptTask.run(RunScriptTask.java:57) ~[Base.jar:?]
	at ghidra.util.task.Task.monitoredRun(Task.java:126) ~[Generic.jar:?]
	at ghidra.util.task.TaskRunner.lambda$startTaskThread$1(TaskRunner.java:94) ~[Docking.jar:?]
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1130) [?:?]
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:630) [?:?]
	at java.lang.Thread.run(Thread.java:832) [?:?]
ERROR Unexpected Exception: ImportError: No module named jintrospect  (ConsoleComponentProvider.java:332)
org.python.core.PyException: ImportError: No module named jintrospect
	at org.python.core.Py.ImportError(Py.java:329) ~[jython-standalone-2.7.2.jar:2.7.2]
	at org.python.core.imp.import_first(imp.java:1200) ~[jython-standalone-2.7.2.jar:2.7.2]
	at org.python.core.imp.load(imp.java:1016) ~[jython-standalone-2.7.2.jar:2.7.2]
	at ghidra.python.GhidraPythonInterpreter.<init>(GhidraPythonInterpreter.java:120) ~[Python.jar:?]
	at ghidra.python.GhidraPythonInterpreter.get(GhidraPythonInterpreter.java:89) ~[Python.jar:?]
	at ghidra.python.PythonScript.runInNewEnvironment(PythonScript.java:110) ~[Python.jar:?]
	at ghidra.python.PythonScript.run(PythonScript.java:48) ~[Python.jar:?]
	at ghidra.app.script.GhidraScript.executeNormal(GhidraScript.java:365) ~[Base.jar:?]
	at ghidra.app.script.GhidraScript.doExecute(GhidraScript.java:220) ~[Base.jar:?]
	at ghidra.app.script.GhidraScript.execute(GhidraScript.java:198) ~[Base.jar:?]
	at ghidra.app.plugin.core.script.RunScriptTask.run(RunScriptTask.java:57) ~[Base.jar:?]
	at ghidra.util.task.Task.monitoredRun(Task.java:126) ~[Generic.jar:?]
	at ghidra.util.task.TaskRunner.lambda$startTaskThread$1(TaskRunner.java:94) ~[Docking.jar:?]
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1130) [?:?]
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:630) [?:?]
	at java.lang.Thread.run(Thread.java:832) [?:?]

Don't worry about where it says the cache is as it's probably due to how I have things setup to debug with vscode.

Ok thanks for helping to debug it.

@ryanmkurtz   procmon finally hit and didn't die... but neither did the python interpreter.
yesterday morning, didnt look at procmon I guess till this morning, cleanmgr.exe did:
SetDispositionInformationFile on   ...\jython_cachedir\python-src (result NOT EMPTY)
SetDispositionInformationFile on   ...\jython_cachedir\packages (result SUCCESS)
SetDispositionInformationFile on   ...\jython_cachedir (result NOT EMPTY)
and this morning another (just the 1)
SetDispositionInformationFile on   ...\jython_cachedir\python-src (result NOT EMPTY)

Ok.  It needs both SetDispositionInformationFile and Delete: True for it to represent a delete attempt.  I guess you had Delete: False?

Do you have this task setup to run?

You can right click on it to manually run it, and see if it blows away your cachedir.

Yeah, that was the trigger, same single message.  Guess the first time yesterday did successfully remove packages/
Interpreter still works and opened another image in a new CodeBrowser (while keeping the other open) to force the refresh

Is your VM low on diskspace?

It looks like cleanmgr.exe also takes a file's Last Accessed time into account before deleting it.
I think based on your procmon results we can assume that it's Windows cleaning it up and not Jython.  The right thing to do would be to get it out of the temp directory, which we always expected, but it's nice to understand what's going on.


Only ACE editors

It's been great having the flexibility of custom editors, but having only ACE editors will be very consistent and manageable. Only concern is structured input, like multiple fields, which can be achieved using JSON somehow.

A step towards this by making ace theme view-specified, in b6ddbdc566862d05216f51a6297948bd1642b948.


add clearer error message when checkUpdated failed

add clearer error message when checkUpdated failed

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: ayj
If they are not already assigned, you can assign the PR to them by writing /assign @ayj in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi @hanxiaoshuai. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/assign @hzxuzhonghu

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: hzxuzhonghu
If they are not already assigned, you can assign the PR to them by writing /assign @hzxuzhonghu in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/ok-to-test
/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: hzxuzhonghu
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS [hzxuzhonghu]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@hanxiaoshuai: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth-mcp.sh
087a2274860f38cb11a33eae74a38fedb5df9c6e
link
/test e2e-mixer-no_auth-mcp

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test e2e-mixer-no_auth-mcp

@hanxiaoshuai: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth-mcp.sh
087a2274860f38cb11a33eae74a38fedb5df9c6e
link
/test e2e-mixer-no_auth-mcp

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test e2e-mixer-no_auth-mcp

@hanxiaoshuai: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
087a2274860f38cb11a33eae74a38fedb5df9c6e
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaoshuai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
087a2274860f38cb11a33eae74a38fedb5df9c6e
link
/test istio-unit-tests

prow/istio-presubmit.sh
087a2274860f38cb11a33eae74a38fedb5df9c6e
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@hanxiaoshuai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
087a2274860f38cb11a33eae74a38fedb5df9c6e
link
/test istio-presubmit

prow/istio-unit-tests.sh
087a2274860f38cb11a33eae74a38fedb5df9c6e
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaoshuai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
087a2274860f38cb11a33eae74a38fedb5df9c6e
link
/test istio-unit-tests

prow/istio-presubmit.sh
087a2274860f38cb11a33eae74a38fedb5df9c6e
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

/retest

/test e2e-bookInfoTests-envoyv2-v1alpha3


SlidableListItem: Items are bleeding with light color when dark theme is requested

Best seen in the Mail sample. When items are dark, they are bleeding gray color on the top and bottom. Effect is emphasized when scrolling

Only noticed the issue on high DPI screens. Added 1px margin to the command content to make sure it doesn't bleed, seems to have resolved the issue


Script overwrites installed coding standards

Problem
In version 0.5.0, when I run wp-enforcer it removes the standards already set for a project.
Description
I have multiple coding standards installed in my project (WP Coding Standards and PHPCompatibility) and a script to set them for use with the project's PHPCS. When I run wp-enforcer, it only checks for wp-coding-standards/wpcs and removes the other standard on every run, which causes checks to fail because PHPCS can't find the expected sniffs.
Possible Resolution
It would be nice for WP Enforcer to honor the existing config and not remove the standards set, either by not forcing the standards inside the script or by appending wpcs to the installed_paths only if it's not present already.

Morgan, thanks for bringing this to my attention!

Heh, just ran into this myself; I'm going to refactor to use dealerdirect/phpcodesniffer-composer-installer, which will simplify the code in WP Enforcer and reduce conflicts.


"Browse" button in video conference mute does not work at all

Microsoft PowerToys version
v0,53,3
Running as admin

[X] Yes

Area(s) with issue?
Video Conference Mute
Steps to reproduce
Go to video conference mute, scroll down to camera, and press "Browse"
OS Edition	Windows 11 Home
Version	21H2
Installed on	10/4/2021
OS build	22000.434
Experience	Windows Feature Experience Pack 1000.22000.434.0
HP Omen 15 dh1xxx
Intel(R) Core(TM) i7-10750H CPU @ 2.60GHz   2.59 GHz
16GB of Ram (15.8 usable)
Nvidia GeForce 1660 ti
✔ Expected Behavior
The "Browse" button should pull up a file explorer window to choose a file.
❌ Actual Behavior
Nothing happens
Other Software
No response

Duplicate of #13215

Issue is a duplicate
Thank you for your issue! We've linked your report against another issue to centralize the thread.
Thanks for helping us to make PowerToys a better piece of software.


Removed typo

Short description of what this resolves:
Typo
Changes proposed in this pull request:

Remove typo

Ionic Version: 1.x / 2.x
Fixes: #

Thanks!


TSLint is brought to you by: Racists

Apologies to any contributors who aren't employees of Palantir. But to those that are:
Please find jobs elsewhere and stop helping Palantir do horrible things.
Also, stop using my tools (such as Lerna), I don't support you, and I don't want my work to benefit your awful company.

https://news.vice.com/en_us/article/paw9ky/toddler-died-after-getting-sick-in-ice-custody

https://twitter.com/ACLU/status/1033084026893070338

How's it feel to contribute to this institution?

So that's why they have been busy...

@palantirtech That will be actually your stand? Yes, we do that, but also we contribute to open source so who cares? Really? :/

They're human garbage, I asked nicely, now I'm going to force them.

To anyone looking for an alternative to TSLint:
After leaving TSLint I startet developing my own linter, the Fimbullinter project. For a quick start guide, see the documentation of Wotan, the linter runtime
There's a plugin to make Fimbullinter a drop-in replacement of TSLint: Valtýr
Another plugin allows you to use your existing (custom) TSLint rules while benefitting from all other features of the linter: Heimdall
Also note that it has many builtin rules that are either not available in TSLint or superior to their TSLint counterpart.

I asked nicely, now I'm going to force them.

@jamiebuilds Everyone already knows that you asking "nicely" is just a pretense before you call in the mob, but nice of you to admit it. The mob asked nicely too if business owners wanted insurance.
Garbage is as garbage does.

@jamiebuilds is a well known clown, see the following and ignore him
https://github.com/Microsoft/web-build-tools/issues/673

Please stop mentioning me on this issue and contacting me about this, I'm done talking about it since Microsoft employees (Scott Hanselman) have gone out of their way to contact my friends and try to convince them to say shit about me publicly.

No. The Open Source Definition specifies that Open Source licenses may not discriminate against persons or groups. Giving everyone freedom means giving evil people freedom, too.

I actively support the notion of denying resources, a platform, and freedom, to evil people. I don't care if this isn't "true" freedom. Those people don't deserve freedom. Hell, I'd even go so far as to say they don't deserve to share this world with us.

@kode54
I actively support the notion of denying resources, a platform, and freedom, to furries. I don't care if this isn't "true" freedom. Those people don't deserve freedom. Hell, I'd even go so far as to say they don't deserve to share this world with us.

I do remember @jamiebuilds I also remember that he does not like being tagging. I guess that I'm really on edge, because as a person that was pushed to migrate I really fell the needs and the suffering of those who are in the same path as I am. Because of that I just would like an official statement from Palantir. As with Microsoft, if they are actually working with the US government, you are in your right to do so, but you can also be clear as water regarding what are you doing.
Note to @jamiebuilds: Please, I don't know if you just want attention, but please stop trolling good companies for whatever reasons you may have. I do suffer this, the people I come from are being hunted and pointed, and I don't like when someone takes some statement like this just to make his name viral.

And they can also be as unclear as they like. Anyone who thinks the US government is evil can go live under a rock on a island somewhere and avoid contact with anything that was produced as a result of the US government. Alternatively, they can try doing research outside of the bubble of propaganda they apparently consume.
Enough with the brownshirt tactics.

 I also delight in your painting leftism as if it is a bad thing. I’ll just ignore that other comment.

Without getting into that, the ICE policy well predates the current administration

Political matters should be away from the OSS community.
Everyone is free to choose their party and who they follow.
This is childish and irresponsible, instead you should create a medium article that no one will read.


[PE-D][Tester B] More specific error message when for invalid date input (32-04-2024)

Rescheduling a meeting on an illegal date (32-04-2024) returns an error message for an invalid input (either date format is wrong or start datetime is not before end datetime). However, it does not flag out that it is an invalid date.
Maybe it can be improved if the error message says "the date inputted must be a valid date". Or maybe a further improvement is to check for validity of date seperately to give a separate error message.

Labels: severity.Low type.DocumentationBug
original: qinboan/ped#5

Duplicate of #222


CAMEL-10354 OWASP Dependency Check

Added dependencycheck profile to run OWASP dependency check plugin
Note that some recent OpenJDK distribution in such as fedora, RHEL, CentOS doesn't contain ECDHE cipher which is required to download from nvd.nist.gov. In order to run this plugin on these environment, you'd need to install a JCE crypto provider like bouncycastle (e.g. dnf install bouncycastle on fedora24) and remove ECDHE from jdk.tls.disabledAlgorithms property defined in the jre/lib/security/java.security.
https://github.com/jeremylong/DependencyCheck/issues/523

Thanks the PR has been merged. Do you mind closing this? :-)

Thanks! closing


Export ImageField url.

is there any way to export the absolute path of the images ?

You can use field dehydration. Something like this:
class MyResource(resources.ModelResource):
    class Meta:
        model = MyModel
        fields = ('photo',)

def dehydrate_photo(self, obj):
    return obj.photo.url


#1122/Add tests for ObjectPropertyNaming for private val overrides

Following the discussions on #1122 I am adding a test to cover a supposed bug (cannot repro tho) and improve other ObjectPropertyNaming tests slightly.

Just to give more feedback, this is also happening to us as well, after updating to RC9. We did not specify a private property pattern before, but even after specifying the same one as for const and public priperties, it would still report it. I can take a look at the rule if you think it is not reproducible! An example value is: “private val PATTERN_0 = foo” inside a companion object withe the regex pattern being: [A-Za-z][_A-Za-z0-9]*

@pavlospt further investigation would be very helpful in this case.
reference #1122

@schalkms great, will try to dive more into it.

@pavlospt further investigation would be awesome! Need to get all that regressions fixed :X :)


More Efficient Ember Encoding

It was previously UTF-8, we've switched to ASCII to match the gist, is that the correct encoding?

Are we keeping this off 0.21 because it changes undefined (by us or spec) behavior?  It looks binary compatible, and I wouldn't necessarily mind backporting it.

Updated Charset


Leshan server demo multiple value resource

Currently there's an issue where if you read or observe a multiple value resource as a whole it succeeds, but if you try to read or observe instances inside said resource, the server fails when parsing the response payload

Thx for reporting this. :pray:
Could you share a more concrete use case so I will try to reproduce it.

(Let me know if you plan to give us more details information about that)

ah sorry, what do you need, I can share the leshan client I'm using to access the leshan sample server

I need exact information or code to reproduce what you consider as an issue. :pray:

I tried to reproduce  :

with leshan demo (client and server )
version : master (3afc6382f0ca32592a762ada87bf0391609bd7e7)
on object 3442 and resource 1110 (Multiple String Value)
using content format TLV.

And it works for me.
So, @RaiseYourChicken I would really appreciate if you could provide more detailed information :

version used
what is used at client side and server side
which resource/resource instance you try to read/observe
which content format
maybe some logs or even a wireshark capture if needed.

I don't know, anything that could help to target the issue.

@RaiseYourChicken I close this issue but please reopen it if you have any more information to share ... :disappointed:


Bug fixit and minor file reorganization.

Test failure in vmodule revert, haha. Suitable punishment for bundling all patches together :)

@rjnagal: I accumulated all these patches while trying to make a new heapster release. The test failure is due to file re-org. Updated the test now.

Still failing integration:
I0313 17:53:28.043127    6702 framework.go:393] Cannot find Replication Controller "monitoring-heapster-controller". Skipping deletion - replicationController "monitoring-heapster-controller" not found
I0313 17:53:28.045736    6702 framework.go:393] Cannot find Replication Controller "monitoring-influx-grafana-controller". Skipping deletion - replicationController "monitoring-influx-grafana-controller" not found
I0313 17:53:28.049257    6702 framework.go:378] cannot find service "monitoring-influxdb" - service "monitoring-influxdb" not found
I0313 17:53:41.043606    6702 kube_test.go:252] waiting for pods to be running
--- FAIL: TestHeapsterInfluxDBWorks (260.81 seconds)
assertions.go:154:
Location:	kube_test.go:311

Error:		No error is expected but got pod not in running state after 60 seconds

FAIL
exit status 1
FAIL	github.com/GoogleCloudPlatform/heapster/integration	260.821s
godep: go exit status 1

@rjnagal: Fixed the integration tests. This PR is ready for review.

LGTM.
waiting on integration tests.


.NET Core 3/netstandard 2.1 support?

Should it work?
After upgrading my projects, which work fine with .NET Core 2.1 and codegen 0.6.1, any project with codegen fails to compile, complaining about locating "netstandard, Version <IP_ADDRESS>, ...", with CodeGeneration.Roslyn at the top of the call stack.

I would love an update to .NET Core 3.x too :)

.NET Core 3.0 workaround
There's one quite simple workaround for now, for the cases where you don't have v2.1 of runtime installed, and only 3.0 SDK+RT.
Set an environment variable: DOTNET_ROLL_FORWARD_ON_NO_CANDIDATE_FX=2
Documentation: https://github.com/dotnet/core-setup/blob/master/Documentation/design-docs/roll-forward-on-no-candidate-fx.md

This should be fixed by #178 - await 0.7 release.


No Config Argument

Ok so when I start the bot this shows:
2016-08-08 01:06:24,885 [       cli] [INFO] PokemonGO Bot v1.0
2016-08-08 01:06:24,887 [       cli] [INFO] No config argument specified, checking for /configs/config.json
Traceback (most recent call last):
File "pokecli.py", line 493, in
main()
File "pokecli.py", line 126, in main
if bot:
UnboundLocalError: local variable 'bot' referenced before assignment
Other Information
OS: Windows 10
Git Commit: (run 'git log -n 1 --pretty=format:"%H"' and paste it here)
Python Version: (run 'python -V' and paste it here)

pls help

Bad config file. If this is still an issue seek help on the slack channel.


Old migrations

Just worth to noting that old migrations from 0.2.3 won't work as the TimescaleDateTimeField was moved. There are probably not that many using this package since that version, but could be worth mentioning.

Yes, the last contribution did introduce some braking changes, but as you mentioned we are probably not that many using the package. I do want to introduce a changelog though and I'll be more than happy to merge one in as well.

Yes, the last contribution did introduce some braking changes, but as you mentioned we are probably not that many using the package. I do want to introduce a changelog though and I'll be more than happy to merge one in as well.


fetch ui language either from toolbar (via a defined api) or from some cookie (default)

Already fixed


Icebreaker can't be added due to an issue with the bot

Getting this error while adding Icebreaker to Teams "Icebreaker can't be added due to an issue with the bot"
Any suggestions here..

Thank you for submitting this issue, as we work to ensure consistent responses with items logged in our issues list, we are working to address these items that did not get responses. Please let us know if you are still facing this issue and require assistance and we will be happy to discuss. If no further action is needed at moment and no response is received, we will close out the issue in 3 business days. You may also always open a new issue or respond here if something does come back up. Thank you!


Update rabbitmq-c installation script

Support of autotools have been removed from https://github.com/alanxz/rabbitmq-c
We have to use clang instead.

Same PR on php-amqp: https://github.com/pdezwart/php-amqp/pull/295


Using CNN-LSTM to classify time series images

Your product is great, but I recently found that  CNN is not effective for classifying images with different periods, so I would like to ask how you can add CNN-LSTM to the library to make it possible to classify images with time series information. This would really be a huge improvement for timing  images! thank you very much.

Do you deal with video data?

这是来自QQ邮箱的假期自动回复邮件。

您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。

No, I use CNN to classify crop images with different growth periods

Sorry, we currently have no plans to support such a task.

这是来自QQ邮箱的假期自动回复邮件。

您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。

这是来自QQ邮箱的假期自动回复邮件。

您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。


Update zoom-tutorial.md

Changed order of Basic SAML Configuration steps to be in line with the actual order in the UI.
Corrected point b by adding the required /saml/SSO path to the Reply URL.

@jeevansd - can you review the proposed changes?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


nixpkgs search hanging the system

I saw #3594, but I believe it should be made as another issue, as first I can't tell whether this is a regression as I'm not a long term user of nix, and secondly the issue is not a few seconds of wait, but a system freeze for a few minutes.
I'm not using a standalone nixos, I'm running nixpkgs on top of a Debian Wheezy, running X.Org X Server 1.12.4 and gnome-shell 3.4.2. My system is not really recent, but decent enough to not feel the need to upgrade for many years (quad-core 2.4GHz, 4GB in dual-channel, WD Black Caviar 10kRPM disks). But whenever I do nix-env -qa or have nox updating the cache, I'm sure I can go get a coffee, take a shower or do something else, because I know my computer is hanging for about 10 minutes, that ends with gnome-shell getting killed, and all operations back to normal. During that hanging, there's no way to interact with the computer and it's impossible to ssh in.
I did not do all the tests I might think of (like keeping a ssh session opened and see what happens), I've heard from some that going back and forth to the console restores it (though I'd have to restart Xorg, as for an unknown reason I need to restore the console switching keybinds)... Because I need to crack on working on my current project, and waiting for the system to pass the hanging for 10 minutes for each test is not negligible :-)
So I wonder whether this is a known issue with a known solution that I could have found if my duckduck-fu was better? Or is it something related to my system configuration, like some dbus mess (because when you don't know, it's always dbus' fault!) happening making both nix-env and gnome-shell angry at each other for ten minutes?
Even a find or a grep -r within the nixpkgs takes less time and does not hang my system! So I tend to believe there's more to that than just an I/O issue.
That said, nix is really being the missing tool on my debian to have both a stable/simple system for everyday use, with the ability to have up to date version of packages finely managed for development purposes! For that, thanks a lot! ;-)

Hmm, that behavior sounds like running out of RAM, which can cause heavy lags and killing of larger programs. However, it should only need half a gigabyte (x86_64-linux, nix-1.8, recent nixpkgs):
$ command time nix-env -qa > /dev/null
11.38user 0.77system 0:15.58elapsed 78%CPU (0avgtext+0avgdata 573664maxresident)k
58976inputs+0outputs (12major+144836minor)pagefaults 0swaps

The real timing I measured is irrelevant to you, as it's on an SSD (and maybe with partially hot cache).

ok, below is me installing an updated avr package. I went out for 45 minutes, so I'm sorry for the delay, and I used nox because when it's not refreshing, it's lightning fast.
% command time nox avr
Refreshing cache
1 avr8-burn-omat-2.1.2 (nixpkgs.avr8burnomat)
    GUI tool for avrdude
2 avra-1.3.0 (nixpkgs.avra)
    Assember for the Atmel AVR microcontroller family
3 avrdude-6.1 (nixpkgs.avrdude)
    Command-line tool for programming Atmel AVR microcontrollers
4 avrdudess-2.2.20140102 (nixpkgs.avrdudess)
    GUI for AVRDUDE (AVR microcontroller programmer)
5 avr-gcc-libc (nixpkgs.avrgcclibc)
    AVR development environment including binutils, avr-gcc and avr-libc
6 pypy2.4-avro-1.7.6 (nixpkgs.pypyPackages.avro)
    A serialization and RPC framework
7 python2.7-avro-1.7.6 (nixpkgs.python27Packages.avro)
    A serialization and RPC framework
8 python3.3-avro3k-1.7.7-SNAPSHOT (nixpkgs.python33Packages.avro3k)
    A serialization and RPC framework
9 python3.4-avro3k-1.7.7-SNAPSHOT (nixpkgs.python34Packages.avro3k)
    A serialization and RPC framework
Packages to install: 5
replacing old ‘avr-gcc-libc’
installing ‘avr-gcc-libc’
13.99user 7.57system 37:45.85elapsed 0%CPU (0avgtext+0avgdata 2128584maxresident)k
1829624inputs+19168outputs (26090major+814086minor)pagefaults 0swaps

(triage) still a problem?

sadly yes, it still is… and I have not a clue of what would be causing this. I guess it could be system related, having it hang the system on heavy I/O.
But if I'm right, why would nix-env do such a heavy I/O burden at each call?  Eventually, if someone could help me split the process of nix-env step by step so we could identify which part of the search is causing this, I might discover what is causing the hang (and discover what is causing the I/O load).

Did you try removing nix completely and installing a recent version? It’s been nearly two years, a lot has changed in the meantime.

not removing and reinstalling (feels too windowsy), but I did upgrade it regularly. Do you think a reinstall is a better idea than upgrading?

Hm, it sounds to me like the database is blown up or something; on the other hand I haven’t used nix-env in quite a while; it has never performed very well, since it needs to evaluate everything in nixpkgs to list anything.
e.g. installation should be done with the attribute name (-iA) instead of the derivation name, then it is instantaneous.

so, how do you look up the "attribute" names without evaluating everything in nixpkgs?

@guyzmo Ah, there’s two ways I use:

Install nix-repl and use auto-completion
Use search-tool of your choice (grep -r, ack, ag)  on nixpkgs to find good candidates (tip: packagename.= as regex string)

If we want to make that more end-user friendly:
3. Write a small script that automates the second trick and prints possible matches.
4. Write a caching package tool that only evaluates everything if that changed. Similar to locate. It should be easy, since is already added to the store and the backend for the tool could generate the indexing database in a nix expression.
I actually quite like the approach of 4.
There’s also an online search, but I don’t particularly like that, since a) it doesn’t reflect the local nixpkgs and b) it needs network access.

I’m gonna close now.


Add min, max Spark aggregate functions

Before this PR, Gluten maps Spark's min and max functions to Velox Presto's
min and max functions. Presto truncates timestamp data to milliseconds, but Spark
requires timestamp data precision to be in microseconds. This caused wrong results.
Therefore, we need to implement min and max functions for Spark.

move min and max aggregation function implements to lib folder.
add TimestampMinAggregate and TimestampMaxAggregate for Spark to align
the data precision requirement.

Gluten output
val table = "insert_desc_same_time_zone"
val tsCol = "timestamp_typed_col"
val minTimestamp = "make_timestamp(2022, 1, 1, 0, 0, 1.123456)"
val maxTimestamp = "make_timestamp(2022, 1, 3, 0, 0, 2.987654)"
spark.sql(s"CREATE TABLE $table ($tsCol Timestamp) USING parquet")
spark.sql(s"INSERT INTO $table VALUES $minTimestamp, $maxTimestamp")
spark.sql(s"SELECT min($tsCol), max($tsCol) from $table").show(false)
+------------------------+------------------------+
|min(timestamp_typed_col)|max(timestamp_typed_col)|
+------------------------+------------------------+
|2022-01-01 00:00:01.123 |2022-01-03 00:00:02.987 |
+------------------------+------------------------+

Expected output
val table = "insert_desc_same_time_zone"
val tsCol = "timestamp_typed_col"
val minTimestamp = "make_timestamp(2022, 1, 1, 0, 0, 1.123456)"
val maxTimestamp = "make_timestamp(2022, 1, 3, 0, 0, 2.987654)"
spark.sql(s"CREATE TABLE $table ($tsCol Timestamp) USING parquet")
spark.sql(s"INSERT INTO $table VALUES $minTimestamp, $maxTimestamp")
spark.sql(s"SELECT min($tsCol), max($tsCol) from $table").show(false)
+--------------------------+--------------------------+
|min(timestamp_typed_col)  |max(timestamp_typed_col)  |
+--------------------------+--------------------------+
|2022-01-01 00:00:01.123456|2022-01-03 00:00:02.987654|
+--------------------------+--------------------------+

cc @rui-mo  and @PHILO-HE , thanks.

@mbasmanova and @Yuhta , could you help to review? Thanks.

@mbasmanova and @Yuhta , could you help to review? Thanks.

@mbasmanova and @Yuhta , could you help to review? Thanks.

Kindly ping, could you help to review? Thanks.

@jinchengchenghh , do you have more comments? Thanks.

@mbasmanova , could you please help review this PR? Thanks.

@rui-mo , is it ok to merge this PR? Thanks.

@rui-mo , do you have more comments?

Thanks. Just several nits. Would you also review the PR description to make it more concise? Perhaps we don't need too much details for the example part.

Updated, thanks.

cc: @mbasmanova

@zhli1142015 PR description mentions 2 differences between Presto and Spark, however, (2) and (3) seems to be the same between these 2 engines.

Map type is not orderable type.

Map type is not orderable in Presto as well.

Nested nulls are allowed within complex types.

Presto does allow complex types with NULL. Perhaps, the differences are more subtle. It would be nice to clarify.
presto:di> select min(x), max(x) from unnest(array[array[1], array[1, null]]) as t(x);
 _col0 |   _col1
-------+-----------
 [1]   | [1, null]
(1 row)

CC: @rui-mo

@zhli1142015 PR description mentions 2 differences between Presto and Spark, however, (2) and (3) seems to be the same between these 2 engines.

Map type is not orderable type.

Map type is not orderable in Presto as well.

Nested nulls are allowed within complex types.

Presto does allow complex types with NULL. Perhaps, the differences are more subtle. It would be nice to clarify.
presto:di> select min(x), max(x) from unnest(array[array[1], array[1, null]]) as t(x);
 _col0 |   _col1
-------+-----------
 [1]   | [1, null]
(1 row)

CC: @rui-mo

This is interesting, I capture the difference from Velox code. I thought they describe expected behavior for Velox Presto functions. Please let me know if I misunderstand them.
Thanks.
Nested Nulls:
https://github.com/facebookincubator/velox/blob/0adc62e3aaf647596ab476f6300a4165e60f6b91/velox/functions/prestosql/aggregates/tests/MinMaxTest.cpp#L457
Map type:
https://github.com/facebookincubator/velox/blob/0adc62e3aaf647596ab476f6300a4165e60f6b91/velox/functions/prestosql/aggregates/MinMaxAggregates.cpp#L1050C30-L1050C31

Map type is registered for max and min functions:

This looks like a bug. CC: @kgpai @kagamiori @amitkdutta

Presto does allow complex types with NULL. Perhaps, the differences are more subtle. It would be nice to clarify.
presto:di> select min(x), max(x) from unnest(array[array[1], array[1, null]]) as t(x);
 _col0 |   _col1
-------+-----------
 [1]   | [1, null]
(1 row)

Please see below query, I believe the Presto min and max functions do not support comparisons of nested nulls; but failure is only triggered when comparing null values.
presto> select min(x), max(x) from (values (array[1, 2]), (array[1, null])) as t(x);

Query 20240717_073504_00009_fswnv, FAILED, 1 node
Splits: 1 total, 0 done (0.00%)
[Latency: client-side: 37ms, server-side: 29ms] [0 rows, 0B] [0 rows/s, 0B/s]

Query 20240717_073504_00009_fswnv failed: ARRAY comparison not supported for arrays with null elements

@zhli1142015 I still see PR description as before and needing updates. It is currently not accurate.

Map type is registered for max and min functions:

Looking closely at Presto's registration, I see it says that input must be orderable, which excludes maps.
  signatures.push_back(exec::AggregateFunctionSignatureBuilder()
                           .orderableTypeVariable("T")

Looks like we just need to remove 'case TypeKind::MAP'. Any chance you could make a PR to do that?
            case TypeKind::MAP:
              [[fallthrough]];

@zhli1142015 Would you resolve the conflicts? Thanks.

@zhli1142015 Would you resolve the conflicts? Thanks.

Done, thanks.

Sorry but could you rebase again, I'm still seeing conflicts in velox/functions/lib/aggregates/CMakeLists.txt


Load script

follow up PR for stored script mechanism as discussed in #188

Coverage increased (+0.5%) to 88.065% when pulling f574237fc2ee02a63a9f82f7560fc7f969ddbe99 on loadScript into 2ee6cc9cc8c654b25e2eaae2aa7b7558a7f1d9ee on master.


post_save / post_log question

I have questions about post_save and post_log. In my requirement I want to take action on the object at the time it is being logged.
My first attempt was using post_save. Here's the issue: when I receive post_save for the LogEntry, at this moment, the object that is being logged is not saved yet. Is it possible to have the post_save of LogEntry after the object was saved?
Second attempt was using the post_log signal instead. Here the object being logged is saved already, but another requirement is missing: The post_log signal does not include the actor. Is it possible to include it?

@aqeelat do you have any idea here?


Update docs to encourage avoiding platform-dependent types

To avoid situations like #2594.

Done in https://github.com/realm/realm-cocoa/issues/2604 and https://github.com/realm/realm.io/pull/1123


Provide custom kernlespec path when launching Jupyter (for non-RAW cases)

Today we store the custom kernel specs (generated by the extension) in the standard location.
These are only used for spinning up kernels when zmq is not available.
Suggestion is to place them in a custom directory & when lauching jupyter provide the path to this folder, so that jupyter will look for kernelspecs from there are well.
Benefit:

We can delete these kernelspecs anytime we want
These will never appearh in Jupyter lab/notebook (hence will never confuse users)
We don't have to worry about users modifying these kernelspec files (e.g. adding custom env variables)

Planning on addressing this in debt (this iteration we deleted the old kernelspecs, I think we shoudl have just gone wit this approach, then we'd never create kernelspecs on user machines)

this has been completed


Add 60 FPS patches for prisoner timer, spray effects, and map transcribe animation

Adding fixes for the below 60 FPS issues so we can test these all in one place:

The "Ritual" prison monster's footsteps go crazy fast.
The animation process whenever James transcribes notes onto his map is twice as fast (too fast).
The visual range of the Lying Figure's spray attack is twice as long. The spray can look like it's hitting James when it actually isn't.
The visual range of the Hyper Spray's spray is twice as long. The spray can look like it's hitting enemies when it actually isn't.

These are ready to test for v1.0, v1.1 and DC.

Hi Murugo, thank you for your work, looks amazing.
Btw, you can put your username in the copyright notice on top of the source files, it's your work after all! Also in the SixtyFPSPatch.cpp, you could add your user next to mine.
If not, I'm ready to merge

Ok I'm gonna merge, I can change it later if you want to!


[B] The text does not fit the top position of the keyboard, so I cannot see what I am editing

Notify 1.3.1 (fdroid)
OnePlus CPH2399
Android 13

Thanks @serrq for pointing it out will be fixing it in the next update.

Hi @aritra-tech,
Can I try for this issue?

Sure @Devendra34
All the best

Any update @Devendra34?

Hi @aritra-tech,
In XML when we have EditText inside a ScrollView, we get our desired auto scroll experience properly..
The same when we try in compose with TextField inside Scrollable Column, it is not supported by default.
I searched and experimented various ways to solve this issue, but I was not getting a satisfactory result till yesterday.
Recently I found one possible solution, which I'll try to push today. Will document the selected approach in PR description as well.
You can expect a PR by EOD.

Okay got your point.
You can refer to these two articles

Keyboard handling in Jetpack Compose
Keyboard Handling In Jetpack Compose — All You Need To Know

Hi @aritra-tech,
Here's my latest update:
Screen_recording_20231005_115541.webm
As seen in the demo video, the experience is better due to auto scroll to current cursor position
I will rebase and push the stable code

Hi @aritra-tech,
Here's my latest update:

As seen in the demo video, the experience is better due to auto scroll to current cursor position
I will rebase and push the stable code

Nice work @Devendra34

Hey @Devendra34 any update?
Just asking because I will be releasing the latest version tomorrow so wanted this issue to be fixed.

Hi @aritra-tech,
Apologies for the delay, it was a super tight week!
I have raised #123, you may go through it

Hi @aritra-tech, Apologies for the delay, it was a super tight week! I have raised #123, you may go through it

No issues bro.


TimingFunction for all AnimationType Fix #564

If a timing function is defined for "spring" animation type, do not use damping and velocity
see explanation in #564
demo app updated to be able to select the timing function all animation types

          1 Error

:no_entry_sign:
Any changes to library code need a summary in the Changelog.

          1 Warning

:warning:
Consider adding supporting documentation to this change. Documentation can be found in the docs directory.

SwiftLint found issues
Warnings

File
Line
Reason

AnimationsTimingFunctionViewController.swift
23
Delegates should be weak to avoid reference cycles.

Animatable.swift
699
There should be no space before and one after any comma.

Animatable.swift
701
There should be no space before and one after any comma.

  Generated by :no_entry_sign: Danger


remove minor bug in root path

Before submitting

[ Yes ] Did you read the contributor guideline?

What does this PR do?

removes a minor bug in line 193 root = Path(args.data_root).absolute() / args.src_lang
args.data_root is /<src_lang>, no need to add args.src_lang to the path again

python examples/speech_to_text/prep_covost_data.py -d /N/slate/piyush/covost/ru -s ru -t en --vocab-type char
Traceback (most recent call last):
File "examples/speech_to_text/prep_covost_data.py", line 280, in
main()
File "examples/speech_to_text/prep_covost_data.py", line 276, in main
process(args)
File "examples/speech_to_text/prep_covost_data.py", line 195, in process
raise NotADirectoryError(f"{root} does not exist")
NotADirectoryError: /N/slate/piyush/covost/ru/ru does not exist

You are supposed to use -d /N/slate/piyush/covost instead (see also the instructions).

You are supposed to use -d /N/slate/piyush/covost instead (see also the instructions).

My bad than, I got confused by -
parser.add_argument(
"--data-root", "-d", required=True, type=str,
help="data root with sub-folders for each language /<src_lang>"
)

My bad than, I got confused by -
parser.add_argument(
"--data-root", "-d", required=True, type=str,
help="data root with sub-folders for each language /<src_lang>"
)


Change AKS Storage Class

Hello
I'm not sure if I should be posting it here, so sorry in advanced.
When I looked in this doc I see no reference as to how to change the reclaimPolicy for the default storageClass when I provision an AKS cluster
Is there such an option? Currently it is set to delete and I would like to change it to Retain
Thanks in advance
Omer

@jungopro all the examples I am able to find do not appear to let you change it during the initial creation of the cluster
@iainfoulds do you know if this is configurable during the creation phase of a cluster or if we can only set this option when creating a persistent disk after the cluster has already been created?

Thanks for the question, @jungopro. This isn't something that can be edited during the cluster creation. The best practice would be to create an additional storage class and define the reclaim policy of 'retain', then reference this in your deployments rather than one of the default storage classes. An example of this type of storage class definition can be found here - https://docs.microsoft.com/en-us/azure/aks/concepts-storage#storage-classes
@MicahMcKittrick-MSFT #please-close

Thanks for confirming @iainfoulds
@jungopro let us know if you have any additional questions


How to create a new .dat file?

I'm trying to make new .dat files for trajectory optimization. How to parse it or create it in Matlab or other environments?

I know it now


#560 Improve dataloader

Release type: enhancement

[x] Custom caching
[x] Caching key overrides using cache_key_fn

Custom Caching
Added custom caching mechanism from original data loaders  spec
from typing import List, Union, Any, Optional

import strawberry
from strawberry.types import Info
from strawberry.asgi import GraphQL
from strawberry.dataloader import DataLoader, AbstractCache

from starlette.requests import Request
from starlette.websockets import WebSocket
from starlette.responses import Response

class UserCache(AbstractCache):
    def __init__(self):
        self.cache = {}

    def get(self, key: Any) -> Union[Any, None]:
        return self.cache.get(key)   #fetch data from persistent cache

    def set(self, key: Any, value: Any) -> None:
        self.cache[key] = value   #store data in the cache

    def delete(self, key: Any) -> None:
        del self.cache[key]   #delete key from the cache

    def clear(self) -> None:
        self.cache.clear()  #clear the cache

@strawberry.type
class User:
    id: strawberry.ID
    name: str

async def load_users(keys) -> List[User]:
    return [User(id=key, name="Jane Doe") for key in keys]

class MyGraphQL(GraphQL):
    async def get_context(self, request: Union[Request, WebSocket], response: Optional[Response]) -> Any:
        return {
            "user_loader": DataLoader(
                load_fn=load_users,
                cache_map=UserCache()
            )
        }

@strawberry.type
class Query:
    @strawberry.field
    async def get_user(self, info: Info, id: strawberry.ID) -> User:
        return await info.context["user_loader"].load(id)

schema = strawberry.Schema(query=Query)
app = MyGraphQL(schema, graphiql=True)

Cache key override
For cases where the custom cache is overkill, the DataLoaders now support  cache_key_fn  enabling custom hashing function support.  DataLoader can now hash complex objects even non-hashable based on user-defined hashing logic. (examples are in the documentation readme, included in the PR)

Hi, thanks for contributing to Strawberry 🍓!
We noticed that this PR is missing a RELEASE.md file. We use that to automatically do releases here on GitHub and, most importantly, to PyPI!
So as soon as this PR is merged, a release will be made 🚀.
Here's an example of RELEASE.md:
Release type: patch

Description of the changes, ideally with some examples, if adding a new feature.

Release type can be one of patch, minor or major. We use semver, so make sure to pick the appropriate type. If in doubt feel free to ask :)
Here's the tweet text:
🆕 Release (next) is out! Thanks to @techno_tut for the PR 👏

Get it here 👉 https://github.com/strawberry-graphql/strawberry/releases/tag/(next)

Closing raising a different PR with RLEASE.d


Add simple brushing to bar chart vis

When the user holds the mouse button and moves over the bars, they should be brushed (selected) and highlighted in all other visualizations. @bikramkawan, this should work very similar to what you already have implemented for the string column.
Note that nothing needs to be split and that the brushing itself is independent of increasing the size of the brushed elements. This issue is just about letting the user select some of the bars.

Done.
@bikramkawan, please keep your issues up to date.


Enhanced Configurations and Filtering for Project Structure Generation

Hi Austin-Lin,
Congratulations for this fantastic extension. I have been using it a lot !!!
However, I did a lot of post-processing, so I decided to add some functionalities to automatise these. As they proved useful for me, I though I might share with you and others.
In this pull request, I've introduced significant enhancements to the "VSCode Project Structure" extension to improve flexibility and provide more granular control over the output:
Configurable Output Directory:

The default directory for configuration files and output files has been set to the docs folder.

Users can now change this default directory via the extension settings, offering increased flexibility in project organization.
Enhanced Project Structure Generation with Filtering:

Introduced a new command that generates a project_structure_filtered.txt  file. This file reflects the same folder/file tree as the original output, based on the .project_structure_ignore file. Additionally, for a more granular view, the full content of files is now included in the output only for those files that match patterns in the new .project_structure_filter file. This provides users with a focused view of critical or relevant files in their project.

I believe these enhancements can benefit users, offering them more options and control over how they view and document their project structure. This new extension works on my machine (Windows 11, using VS Code Insiders), but I have not tested it in other environments. Feel free to test these changes to ensure compatibility and maintain the robustness of the extension.
I look forward to any feedback or suggestions.
PS: I did raise an issue with this suggestion a few months back. Since you did not reply, I guess you may not be available these days. I have published this enhanced version of your extension temporarily under the name "VSCode-Project-Structure-Plus", giving you full credit for the original version. As soon as you incorporate my PR, I will remove/deprecate the other one. Thank you for your understanding.
Kind regards,
Fabrizio

I fixed a few bugs in the meantime.


[CICD-43] Add circle ci build number support

With this changes we can add circle ci build number to the built file

I will merge this for now for testing purpose

@archit Sure let me do version-commit-circleci.
And for the upstream one, let me do that when we got extra time.... Right now the code is not in the best form it should be, even though it does the job

The code looks fine. I compared it to version-commit, and there is only a diff of 3 lines. I think the upstream folks will consider it. Its very easy to make a PR for upstream via GitHub, just start a new PR, but point the original branch to point to the original fork, not this repo's master.

The thing is that right now they want the revision data to be depending on the source files only (based on their README), so a circle CI build number should not be added. But let me try anyway to see how they feel


feat(core): basic copy

part of #361
Add support to copy and blocking_copy
Currently only the fs implementation is done.

As a new capability, I intend to add a new behavioral test.

As a new capability, I intend to add a new behavioral test.

This PR is good enough to be merged. Maybe we can add behavior test in next PR?


monthly task update devcontainer.json

used tag 1-1-bullseye which pointing to the latest bullseye sha for rust

Hi @samruddhikhandale ,
things to try section is tested successfully and attached is the snapshot to validate the test scenario is working as expected after the bump in basic image version:

@bamurtaugh whose approval do we need to merge this PR? Looks like the Community PR Approvals check is still spinning

Not sure - maybe @chrmarti knows (or can at least approve)?


VMs running via QEMU only expose SSE and SSE2 CPU features

Even when the host machine supports more advanced instruction sets like AVX/AVX2 - VMs only expose SSE and SSE2. At the moment it's not possible to test AVX code in these VMs. Xhyve exposes features up to AVX, but not AVX2, even when the host supports these extensions.
Is there any plan to perhaps add an option that would influence how many CPU features would be provided to the guest machine? I think at least AVX as exposed by xhyve would be great.

I can see if I can add more CPU features when using QEMU. As long as the OS boots I don't think it needs to be an option.

Available now in v0.19.0.


Crash when set app in background

Hi,
I installed the example app on my Android device. When I put the app in background during the camera is running I got the following exception:
flutter: CameraException(Disposed CameraController, buildPreview() was called on a disposed CameraController.)
How can we avoid this?

Hi @MSL-BIT !
Thanks for testing. I'll take a look at it.

Please try use the version 0.9.1


Upgrade mongoengine to the latest version (0.15.3)

This pull request updates mongoengine to the latest version (0.15.3) from our fork (https://github.com/MongoEngine/mongoengine/pull/1833) which contains a work-around for a large regression in memory usage introduced in mongoengine v0.13.0.
I confirmed reverting that commit fixes an issue with increased memory usage.
Using our fork with a reverted commit:
(virtualenv) vagrant@ubuntu-xenial:/data/stanley$ python test_mongoengine.py
Python: 2.7.12 (default, Dec  4 2017, 14:50:18)
[GCC 5.4.0 20160609] on Linux-4.4.0-130-generic-x86_64-with-Ubuntu-16.04-xenial
mongoengine: 0.15.3
sleeping...

(virtualenv) vagrant@ubuntu-xenial:/data/stanley$ ps -eo rss,pid,euser,args:100 --sort %mem | grep -i 'test_mongoengine' |grep -v grep | awk '{printf $1/1024 "MB"; $1=""; print }'
18.3281MB 12696 vagrant python test_mongoengine.py

Using upstream version:
Python: 2.7.12 (default, Dec  4 2017, 14:50:18)
[GCC 5.4.0 20160609] on Linux-4.4.0-130-generic-x86_64-with-Ubuntu-16.04-xenial
mongoengine: 0.15.3
sleeping...

(virtualenv) vagrant@ubuntu-xenial:/data/stanley$ ps -eo rss,pid,euser,args:100 --sort %mem | grep -i 'test_mongoengine' |grep -v grep | awk '{printf $1/1024 "MB"; $1=""; print }'
70.6875MB 6348 vagrant python test_mongoengine.py

Latest version will allow us to support MongoDB 3.6.
When / if mongoengine merges a patch for that issue upstream, we should move away from our fork.

Finally managed to also get unit tests on Travis to also run under MongoDB 3.6 - https://travis-ci.org/StackStorm/st2/builds/412583451
I will go ahead and merge this into master.

It looks like tests run much slower under MongoDB 3.6.
It might be related to this - https://jira.mongodb.org/browse/SERVER-34604

A lot of changes added afterwards is not what I reviewed really.
And timing for Travis longest task is back returned to almost half-an-hour :/


Tracing functionality

Add ability to trace event execution when the trace (or isTrace in the future https://github.com/chark/scriptable-events/issues/11) flag is enabled on an event. This could involve such capabilities:

Viewing event "stack traces".
Visualizing event flow/connections by utilizing gizmos.

This looks interesting: https://github.com/Unity-Harry/Unity-AssetDependencyGraph


[Boss] Eden (High-Level)

A bit of a weird boss: Instead of fighting a singular enemy, you're fighting more or less everything in an area.
As you enter the arena, all that's visible is the Tree of Knowledge in the center of the area
Phase 1: Weeding the Garden
A wave of highly upgraded jungle creeps appears on the edges of the arena and makes its way towards the center, fighting any hero it encounters. These creeps are invincible until being force-fed the fruit of knowledge (periodically provided from a helpful snake resting on the tree of knowledge). After being fed the fruit, the creeps lose their invincibility and may be slain. After the first wave of creeps is defeated, a second spawns, and after that a third. Once the third wave is defeated, the fight moves onto the next stage.
Phase 2: Against the Serpent
The serpent reveals its gigantic true form and coils around the Tree. It has three different modes of attacking: the first is a simple slam of its head/tail, the second is a beam that charges up after a certain amount of mana is spent in the arena, and the third is the most interesting: the snake will passively give intelligence to everyone in the arena, the amount increasing the closer to the serpent you are and the more spells you cast at it. If your intelligence goes above a certain threshold, the serpent temporarily possesses your hero, forcing it to attack your allies. When the possession ends, the intelligence bonus resets, and the decrease in your total mana pool is dealt to you as damage. When the serpent is killed, all possesion effects ends, all heroes are knocked back to the edges of the arena, and the next phase begins.
Phase 3: Escape
Your meddling in the garden has attracted the attention of a higher power. You should probably run. To escape, you need to reach the Tree of Knowledge and eat some of its fruit yourself. Standing in your way is a host of seraphim, who will periodically descend and try to prevent you from leaving. Seraphim have medium-low health, high attack damage/speed, and will begin dragging you back to the edges of the arena if more than one is currently attacking you (the more that are attacking you, the faster they pull). After 15-20 seconds, everyone still in the arena is instantly blasted into oblivion (a giant sunstrike that deals 999999 damage or something would be cool). Only one hero needs to escape for the encounter to be cleared, although the others will still have to make their way out if they don't want to die.
Drops:
TBD depending on if Baumi likes this or not.
Visual Inspirations (Not created by me):
Seraphim:

Too complicated. Closed.


fix(apiconvert): never skip basic auth username/password

About this change—what it does

prevents skipping basic auth username/password in certain service integrations

resolves #1411
Why this way

API always wants us to send these details, regardless of if they're changed or not

acceptance tests: https://github.com/aiven/terraform-provider-aiven/actions/runs/6852128028


get-help about_activedirectory_filter incorrect attribute

The help references email as an AD attribute multiple times. It should be mail

Yes, reference is wrong but about_activedirectory_filter help is not a part of DSCResource. It is a part of ActiveDirectory module help. You can probably submit feedback here https://technet.microsoft.com/en-us/library/hh531527(v=ws.10).aspx

my apologies, got some bad info on where to submit issues on powershell.