Draft of cache busting

Draft of cache busting in #54

Looks good, thank again!
Thinking on it, I wonder if it would make sense to move the logic for removing the keys to happen inside of the remote_fetch method only if the response was successful? Something along the lines of:
async fn remote_fetch(
        &self,
        middleware: &mut impl Middleware,
    ) -> Result<HttpResponse> {
        let mut res = middleware.remote_fetch().await?;
        match res.status {
            200..=299 => {
                let cache_key =
                    self.options.create_cache_key(&middleware.parts()?, None);

                if let Some(cache_bust) = &self.options.cache_bust {
                    for key_to_cache_bust in cache_bust(
                        &middleware.parts()?,
                        &self.options.cache_key,
                        &cache_key,
                    ) {
                        self.manager.delete(&key_to_cache_bust).await?;
                    }
                }
            }
            _ => {}
        }
        ...

Oh, also I think we would need to call the logic inside of the run_no_cache method as well. In the case of issue #54 it's a POST request that would be triggering the deletion, but as of the last release we skip the normal run logic upon requests determined to be non-cacheable. Instead the run_no_cache method is called before letting the middleware chain continue
pub async fn run_no_cache(
        &self,
        middleware: &mut impl Middleware,
    ) -> Result<()> {
        self.manager
            .delete(
                &self
                    .options
                    .create_cache_key(&middleware.parts()?, Some("GET")),
            )
            .await
            .ok();
        let cache_key =
            self.options.create_cache_key(&middleware.parts()?, None);

        if let Some(cache_bust) = &self.options.cache_bust {
            for key_to_cache_bust in cache_bust(
                &middleware.parts()?,
                &self.options.cache_key,
                &cache_key,
            ) {
                self.manager.delete(&key_to_cache_bust).await?;
            }
        }
        Ok(())
    }

Looks good, thank again!
Thinking on it, I wonder if it would make sense to move the logic for removing the keys to happen inside of the remote_fetch method only if the response was successful? Something along the lines of:
async fn remote_fetch(
        &self,
        middleware: &mut impl Middleware,
    ) -> Result<HttpResponse> {
        let mut res = middleware.remote_fetch().await?;
        if let 200..=299 = res.status {
            let cache_key =
                self.options.create_cache_key(&middleware.parts()?, None);

            if let Some(cache_bust) = &self.options.cache_bust {
                for key_to_cache_bust in cache_bust(
                    &middleware.parts()?,
                    &self.options.cache_key,
                    &cache_key,
                ) {
                    self.manager.delete(&key_to_cache_bust).await?;
                }
            }
        }
        ...

Edit: really no longer sure about this change, I keep going back and forth on it

You're welcome!
I think with deleting, it could just be the user wants to forfeit the request, so only use the request to clear the cache but not to actually send the request, but this is specific to reverse proxies more so a request isn't sent to the origin server.

Oh, also I think we would need to call the logic inside of the run_no_cache method as well. In the case of issue #54 it's a POST request that would be triggering the deletion, but as of the last release we skip the normal run logic upon requests determined to be non-cacheable. Instead the run_no_cache method is called before letting the middleware chain continue

Ah I missed that, I'll add it in!


Update dependency to Ecto 3 ?

Do you think there is any incompatibility with Ecto 3 ? I think this should support it.
Thanks.

+1, I can't use trans in my phx 1.4 app, unless I explicitly override the ecto 3.0 dependency. Let's see if I run into trouble.
$ mix deps.get

...
Failed to use "ecto" because
phoenix_ecto (version 4.0.0) requires ~> 3.0
trans (versions 2.0.0 to 2.0.4) requires ~> 2.1 *

@speeddragon, @beno I am addressing the Ecto 3.0 update in PR #57

Trans 2.1 is already published in hex.pm. It supports Ecto 3.0 and does not require any changes to client code.
Thank you for your patience! :beers:


Context for Translation-Based Methods

"You should mention in the background section the translation-based approaches to speed up MC on CAD-derived models. It would also be helpful if you could give order-of-magnitude runtimes for MC on those models, though I understand that data may be hard to come by and will not be directly comparable to the runtimes for your method."
IMO this belongs in the background in it's own section.

Note to self: This is briefly mentioned in Chapter 1 and should be tied to the location of this section elsewhere in the document.

I think this is actually covered in chapter 2 at the end of the CSG section. Can you verify @gonuke?
I'm not sure that I have a way to make fair comparisons of these approaches, particularly for the models I've been using for production demonstrations. Though I'm open to exploring ways to get that information if necessary.

I don't think it is in scope to get new/relevant performance data.  You can quote (very) old performance data, but improvements have been ongoing on both sides of that coin.
I could suggest some additional words about human vs. computer effort, but nothing really citable....

I'd honestly rather claim it as a separate method altogether given the simplifications needed to perform the translation process.
I'd be happy to hear any suggestions you have!

Still hung up on this. Wondering if I shouldn't ping Tim T and see if he maybe missed the mention of a couple of these tools. It certainly wasn't an extensive list of these tools and I didn't go into detail regarding file formats or results, but I think awareness of other methods is all that is required here. I'm not trying to compare methods, only improve an existing one.

(backpedals...) Reading that section again, I only mention those tools in the context of visualization and geometry repair. I don't describe them as tools used to provide simulation-ready input. I'll review this tonight and come up with a better description.

Yeah - that's probably not the best way to mention them anyway...
Maybe remove their mention from the end of 2.2, and add a paragraph at the top of 2.4 about tools that exist to convert from CAD to CSG:

a number of them exist (history includes a few more than MCAM/McCAD)
they have some limitations (limited surface representation = CAD modification)
they have some benefits (no source code modification[1] & faster runtime)
you can refer to past efforts to compare them head-2-head with DAGMC being 2-4x slower

[1] I just noticed that I don't think you ever discuss how DAGMC gets coupled to MC physics codes


Update dns initialization settings

[!warning]
This is a public repository, ensure not to disclose:

[x] personal data beyond what is necessary for interacting with this pull request, nor
[x] business confidential information, such as customer names.

What kind of PR is this?
Required: Mark one of the following that is applicable:

[x] kind/feature
[ ] kind/improvement
[ ] kind/deprecation
[ ] kind/documentation
[ ] kind/clean-up
[ ] kind/bug
[ ] kind/other

Optional: Mark one or more of the following that are applicable:

[!important]
Breaking changes should be marked kind/admin-change or kind/dev-change depending on type
Critical security fixes should be marked with kind/security

[ ] kind/admin-change
[ ] kind/dev-change
[ ] kind/security
[ ] kind/adr

What does this PR do / why do we need this PR?

Adds support to initialize config with support to CAPI and IPVXLAN on azure

Adds one #1999

Information to reviewers

Checklist

[ ] Proper commit message prefix on all commits

Change checks:

[x] The change is transparent
[ ] The change is disruptive
[ ] The change requires no migration steps
[ ] The change requires migration steps
[ ] The change upgrades CRDs

Metrics checks:

[ ] The metrics are still exposed and present in Grafana after the change
[ ] The metrics names didn't change (Grafana dashboards and Prometheus alerts are not affected)
[ ] The metrics names did change (Grafana dashboards and Prometheus alerts were fixed)

Logs checks:

[ ] The logs do not show any errors after the change

Pod Security Policy checks:

[ ] Any changed pod is covered by Pod Security Admission
[ ] Any changed pod is covered by Gatekeeper Pod Security Policies
[ ] The change does not cause any pods to be blocked by Pod Security Admission or Policies

Network Policy checks:

[ ] Any changed pod is covered by Network Policies
[ ] The change does not cause any dropped packets in the NetworkPolicy Dashboard

Audit checks:

[ ] The change does not cause any unnecessary Kubernetes audit events
[ ] The change requires changes to Kubernetes audit policy

Falco checks:

[ ] The change does not cause any alerts to be generated by Falco

Bug checks:

[ ] The bug fix is covered by regression tests

Since this isn't dependent on your storage resource script you should rebase it on main and remove all commits except the new ones.

Removed the need for CLUSTER_PROVISIONER and set the related settings to be azure related.

Please give the PR a proper title btw

Title edited.

It basically adds support for ck8s-cluster-api dns IPs, IPVXLAN, and azure. (IPVXLAN is needed for azure but this makes it not azure specific)

Right, what about "Support Azure DNS and VXLAN node network policy" or something like that?

Either way, this is fine to merge whenever you're ready! :)


Add support for DateTime type

Hi,
I've stumbled upon your library, and being an Android developer I've loved the Room inspiration of it.
I've started introducing it in one of my project, and found out the DateTime type is not supported.
I think it would be great to add this datatype to the ones recognized by the library.
Thanks in advance

I've thought about introducing type converters similar how Room handles these conversions. This would open up the possibility of persisting data of custom date implementations, for instance.

I found a workaround for now using the dart built-in features :

var milliseconds = DateTime.now().millisecondsSinceEpoch
DateTime.fromMillisecondsSinceEpoch(milliseconds)

Not sure why DateTime is not supported. I think SQLite support it as format.
@vitusortner  Do you have some technical limitation here? Why do you need to convert?

DateTime is not supported by sqflite (the database access layer).
Type converters might seem like an overkill for such an easy mapping between milliseconds and Dart's DateTime. They surely simplify the interaction with the database. Nevertheless, type converters come in even more handy when thinking about other conversions. For example, whenever an enum value should get stored. It's nicer to just define a type converter in one place and let it do its magic automatically without calling its functions manually.

Yes, make sense. :)

@vitusortner: Further to the above:

For example, whenever an enum value should get stored.

For the special case of enums, wouldn't it make sense to have floor simply store in an int column, and just convert using myEnum.index and myEnum.values[n]?
One step up would be to copy the idea from JPA and use an annotation to allow mapping by user's choice of int value or by string name (FWIW, JPA uses @Enumerated(EnumType.STRING) or EnumType.ORDINAL; ORDINAL is the default there). It's well known there that ordinal is more efficient but does mean you can't add new values to the enum except at the end.
This would be simpler than fully implementing type converters and could maybe get in sooner? But meanwhile I've had to convert my entity to use ints for dates (as you suggest) and for enums.

It's not planned to support DateTime out of the box for now. Same for enums. Instead, type converters will be introduced soon which will solve these problems. Follow the progress and discussion in #165.


R108

Schemas were reviewed by @yalisassoon in https://github.com/snowplow/iglu-central/pull/973 (cherry-picked with insignificant changes)

Done - @chuwy please make sure to update the Releases page in GH, the last 4 releases are undocumented.


Any Blade indents.scm

Hi all, do any one has an indents.scm file for share

I took a crack at this, but I'm pretty sure it's impossible with the current grammar. The indent.begin is used to indent child nodes and the current grammar has them as siblings instead.

Mmmm can I have an example of how this is used? I have never come across this capture.
Is this used to implement a  functionality like this? 🤔
@php
    automatic indentation after pressing return
@endphp

Bingo, you can find a ton of indents.scm below:
https://github.com/nvim-treesitter/nvim-treesitter/tree/master/queries

Oh wow that's a goldmine thanks for sharing all these goodies @calebdw ! As always very much appreciated!
Well, that's not good at all! 😬
I will definitely look into this when I get a chance, but as I am extremely busy just now with university work. It might be a while!
Nova bizarrely still uses their old XML RegEx API to do the indentation hence the oversight.
Regardless, being siblings does not make sense anyways. The stuff should appear as sub-tree! 😊

I have a few questions. I have been playing around in Helix, but I am a bit unsure about the workflow in the modal editors.

Do you use snippets? ie write @ifand then it prints stubs?

If so, is the indent not baked, inside the snippet?

@if(<placeholder>)
     <placeholder>
@endif

Now lets say, after @if the editor indents, when you type @endif do you expect the editor to automatically outdent the line with the @endif as well?

I managed to get the indent working in Helix using the current grammar, for if statements, but there are some caveats:

For normal indentation to work, the @if and @endif must be written and present. (this is like brackets, without the matching pair no indentation happens. At least with helix and their defaults anyways)

I did of course managed to get around this using a specific capture they have for python style grammars, which is @indent @extend but not sure what would be the equivalent in NeoVim

On both occasions, I do need to manually outdent the @endif... is this an expected behaviour?

This is my major concern, if this behaviour is normal, I can crack on coming up with a solution/stubs that can be used across editors. Only problem being, there are no standardisation around _predicates _  and captures in tree-sitter, meaning each editor will  need to have a different indents.scm for sure!

https://github.com/EmranMR/tree-sitter-blade/assets/11975985/1706d8b9-e266-4429-9ba3-4e31fa8322c6

Indents would be backed into the snippet, but that's completely separate from treesitter indents
Indentation should happen as soon as there's an @if and it would end when there's an @endif
You can take a look at the php indents which I just fixed a bug in
I took a look at the indents in the past for blade, and I'm not sure if they're possible without some structural changes
Indents are very much editor specific---tree-sitter does not support indents on its own. So I would say that's not the responsibility of this repo to craft the indents (no other ts grammar repo has indent queries in them).

Indents would be backed into the snippet, but that's completely separate from treesitter indents

Indentation should happen as soon as there's an @if and it would end when there's an @endif

Looks promising!
Because if you have snippets, you can align @if and @endif, then   allow the editor to take over indentation.
So I reckon the video I uploaded shows a correct functioning indentation?

Write @if
Press o, <return> O(from endif) or  you get the indent
And in the end when you write the @endif and press return the editor indents back!


ZEPPELIN-3357. Livy security mode is broken

What is this PR for?
Straightforward fix for livy security issue.
What type of PR is it?
[Bug Fix]
Todos

[ ] - Task

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-3357

How should this be tested?

CI pass

Screenshots (if appropriate)
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

hey @zjffdu thanks for making all these improvements, could we make sure there is at least one review before merging?
@Leemoonsoo


[Machine Learning] Using Imbalanced-Learn to handle imbalanced text data in natural language processing

Proposal Submission
Proposed title of article
[Machine Learning] Using Imbalanced-Learn to handle imbalanced text data in natural language processing
Proposed article introduction
Imbalanced-Learn is a Python module that helps in balancing the datasets which are highly skewed or biased towards some classes. Thus, it helps in resampling the classes which are otherwise oversampled or under-sampled. It offers a number of re-sampling techniques commonly used in datasets showing strong between-class imbalance.
Most models that are built using the imbalanced data will have a bias towards predicting the majority classes in the dataset and ignore the minority classes. Class imbalance is identified by observing the distribution of classes within your data or using visualizing libraries to plot the class distributions.
Imbalanced-Learn has various techniques that will be used to handle this problem. These techniques are such as Random Under sampling technique resample all classes but the majority class, Random Oversampling technique: resample all classes but the majority class.
In this tutorial, we will discuss these techniques in natural language processing and implement them practically using Python. We will also perform all the text preprocessing steps.
Key takeaways

Text preprocessing (we will perform word lemmatization, removing stop words, and removing punctuations).
Performing data augmentation using NLP Transformers.
Installing Imbalanced-Learn.
Exploring Imbalanced-Learn methods and techniques.
Implementing RandomOverSampler to balance the classes.

Article quality
This tutorial is unique because we will implement text preprocessing in detail so that a reader can understand the concepts in natural language processing. We will create a custom function for word lemmatization, removing stop words, removing punctuations, removing extra white spaces, and removing numbers. This ensures we have a clean dataset for natural language processing.  We will also perform data augmentation using NLP Transformers. Finally, we will implement RandomOverSampler to balance the classes.
References
Please list links to any published content/research that you intend to use to support/guide this article.
Conclusion
Finally, remove the Pre-Submission advice section and all our blockquoted notes as you fill in the form before you submit.
We look forwarding to reviewing your topic suggestion.
Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template

Sounds like a helpful topic - let's please be sure it adds value beyond what is in any official docs and/or what is covered in other blog sites. (the articles should go beyond a basic explanation - and it is always best to reference any EngEd article and build upon it). @donnexwafula33
Please be attentive to grammar/readability and make sure that you put your article through a thorough editing review prior to submitting it for final approval. (There are some great free tools that we reference in EngEd resources.)
ANY ARTICLE SUBMITTED WITH GLARING ERRORS WILL BE IMMEDIATELY CLOSED.
Please be sure to double-check that it does not overlap with any existing EngEd articles, articles on other blog sites, or any incoming EngEd topic suggestions (if you haven't already) to avoid any potential article closure, please reference any relevant EngEd articles in yours. - Approved


OnUninstallAppPerCompany

Thank you for the CU2 that preserves the new data in the extensions.
That is awesome and now we can start developing seriously.
However, what do I do if I actually want to delete the data on uninstalling an app.
Can I make a OnUninstallAppPerCompany procedure?
/Peik

No, there is no such procedure, but you can use the Sync-NavApp commandlet and run it with Clean mode.

Ok Thank you,
I used the "schemaUpdateMode": "Recreate" in the launch.json instead.
/Peik


Firstline not working

Issue Type: Bug

I am making an extension and firstline does not work.
When I add
{
"firstLine": "-- #\/Teverse"
}

to my package.json, it won't work.
The first line of the file is:
-- #/Teverse

in test.lua

The workaround for this is if I set add:
{
    "extensions": [
        ".lua"
    ],
}

to package.json. But, every lua file has the custom language.

When using:
"contributes": {
    "languages": [
      {
        //...
        "firstLine": "-- #\/Teverse",
      }
    ]
}

And opening a file with an unknown extension, like test.tssss with the contents:
-- #/Teverse

This is interesting

The file is associated with the language I have chosen. Do you perhaps try to open a file with a file extension that is defined by another extension?

Yeah.
I'm doing .lua, so it is overwritten by lua.
Is there anyway to get around that?

@alexdima
Yeah.
I'm doing .lua, so it is overwritten by lua.
Is there anyway to get around that?

Also,
If there an activation event for on save?

@sasial-dev No, extension matches get higher priority. First line matches are only considered if a file is not recognized by its filename. You can write an extension and change the language of the lua documents after they have been opened.


Issue-basic reducer for example working/data loading

Automated Review Comments
Issues in src/actions/exampleActions.js:

(29, 'Hardcoded string found')
(30, 'Hardcoded string found')
(42, 'Hardcoded string found')
Issues in src/components/Example.js:
(2, 'Hardcoded string found')
(39, 'Hardcoded string found')
(42, 'Line exceeds 80 characters')
(45, 'Hardcoded string found')
(47, 'Line exceeds 80 characters')
(48, 'Line exceeds 80 characters')
(51, 'Hardcoded string found')
Issues in src/main.js:
(2, 'Hardcoded string found')
Issues in src/reducers/exampleReducer.js:
(33, 'Hardcoded string found')
(34, 'Hardcoded string found')
(40, 'Hardcoded string found')
(41, 'Hardcoded string found')
(51, 'Hardcoded string found')
(52, 'Hardcoded string found')
(108, 'Line exceeds 80 characters')
(113, 'Line exceeds 80 characters')
Issues in src/store.js:
(2, 'Hardcoded string found')
(3, 'Hardcoded string found')
(10, 'Line exceeds 80 characters')
Issues in src/utils.js:
(3, 'Hardcoded string found')
(15, 'Line exceeds 80 characters')
(58, 'Hardcoded string found')
(73, 'Hardcoded string found')
(78, 'Hardcoded string found')
(81, 'Hardcoded string found')
(92, 'Hardcoded string found')
(96, 'Hardcoded string found')
(225, 'Hardcoded string found')

I got it.


K8s Dev UX: Search and Replace (M1.1)

h2. Overall Outcome
Customer serves secrets through volume mounted files to an existing application workload deployed into K8s.  This only requires modifying the application deployment manifest and not the application itself.  This essentially matches Novartis’ primary desired use-case.
h2. Epic/Release Outcome
Exact functionality TBD.  However, at a high-level, allow users to bring pre-existing configuration file/content, replace placeholders with secrets from Conjur and then write the resulting updating file/content to a volume shared with the application.
h2. References

PRD: [Conjur Developer Experience|https://cyberark365.sharepoint.com/:w:/r/sites/Conjur/_layouts/15/Doc.aspx?sourcedoc=%7B956DB935-7E3D-4D88-B964-188BCB6F7729%7D&file=Conjur%20Developer%20Experience%20for%20K8s%20PRD.docx&action=default&mobileredirect=true]
Feature Doc: TODO
Solution Design: TODO

https://ca-il-jira.il.cyber-ark.com:8443/browse/ONYX-11465

@Conjur-oss why did you reopen this? This has been completed for months.


Translate to Spanish

Hi Carbon team,
I want to translate the GitHub page to Spanish, is that something you would like to do?
Thanks,
Miguel

@ucdstudent95618 absolutely! I have actually been hoping someone would offer to translate the README! I would really really appreciate that 🙏😄
You can create a new file and I can reorganize it before merging 🙂 Let me know if you need any help.

Awesome! 🙂 I will send you a PR once I am done. I will create a file called README.es.md.


Update docs with getting started guide

Adding a getting started guide
Added getting started guide part 2 in Typescript and Rust
Moved some stuff to a Reference section

@yirenlu92 there were a bunch of fixes that need to be made, so I just went ahead and opened https://github.com/yirenlu92/ren-sway-fork/pull/1 to do them all in one go.

@adlerjohn re duplicating, I just wanted to bring up that it can be more convenient for the user to have all the instructions on a single page, instead of having to jump back and forth between different links.

hi guys, do you have other comments? there will be additional docs work but should we merge these first?

ok I think I went through and made the requested changes...let me know if there's anything else

done, let me know if there's anything else!


Initial HTTPS support

This is the initial support for HTTPS:

rusttls (i.e. ring, i.e. boringssl) based.
HTTPS is enabled by default
--insecure-http disables HTTPS and needs to be explicitly set

Any plans to update the doc with an example execution command, say using a self-signed certificate ?
Please don't consider this as a blocker to merge this PR.

Any plans to update the doc with an example execution command, say using a self-signed certificate ? Please don't consider this as a blocker to merge this PR.

Good point. I'll open an issue to document self-signed cert generation once this PR is merged.

FYI The build error is related to this issue https://github.com/confidential-containers/attestation-service/issues/62

@sameo Now https://github.com/confidential-containers/attestation-service/pull/63 is merged, can you run CI again and can we try to get this PR merged?


Stripe Checkout

Hi!
I couldn't find any better place to send this.. I'm using the html form of stripe checkout. It appears to open correctly on Android (through cordova and web browser) but on iOS trought cordova it only dims the screen for a while and if using safari it opens  checkout window on seperate tab.
Thanks for the help!

Hi @virtapoika,
This space is used for bugs reports about jquery.payments, a library helping the implementation of payment forms. If you want an answer about a specific Checkout integration question you should send an email to our support:
https://support.stripe.com/email
I'll close this issue for now.

Thanks!

guillaumehenriot<EMAIL_ADDRESS>kirjoitti 30.8.2016 kello 13.20:
Hi @virtapoika https://github.com/virtapoika,
This space is used for bugs reports about jquery.payments, a library helping the implementation of payment forms. If you want an answer about a specific Checkout integration question you should send an email to our support:
https://support.stripe.com/email https://support.stripe.com/email
I'll close this issue for now.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/stripe/jquery.payment/issues/245#issuecomment-243396954, or mute the thread https://github.com/notifications/unsubscribe-auth/AF5TeiL263FT8_nUtlSOisvx5438CLI4ks5qlAP5gaJpZM4JwQWQ.


Automatic movement to harpooned point

Getting an issue where in insert mode, you will be randomly moved back to where your harpooned point is.
Deleting the harpoon stops it from jumping
steps to reproduce

open nvim
set harpoon point
exit nvim
reload nvim
harpoon back to file
J down a few lines, enter insert mode
type a few characters
cursor is pinged back to point
delete harpoon point (dd enter)
pinging still continues
Close and reopen nvim again
open file with telescope or similar
pinging stops

Could you include your configuration? The entire harpoon section?
I have yet to experience this, and I've been using it for the last 2 days.

I'm using Lazy as well so I tried to mimic the config here, yet still no luck reproducing the behavior. @ThePrimeagen were you able to replicate yet?
@tris203 We could move this to the Discord channel to troubleshoot if you're open

Further testing, it's not all the time but it seems to be reproducible with this file in particular
and for some reason, it always line 19.
I thought it was the harpooned point, but its not. It jumps me back specifically to line 19

I have experienced where I was pulled back to a similar point over and over again after playing a game, and that is because I left the game early

Are you playing vim be good?

No @ThePrimeagen
I haven't even got VimBeGood installed
It only seems to be in this one specific file

To prove it's from harpoon, you can add a simple print statement in the config.lua file
Lua/harpoon/config.lua
Find the defined select function in the default configuration, I am on phone so I am not going to go grab it
And add at the top of the select function
print("Hello, Harpoon")

:messages
You should see it in the bottom of your screen

Found it, not a harpoon bug Incompatibility with alpha-nvim
harpooning from the dashboard directly is what causes the wierdness


generate_access_token can't use service_account_path format

According to the docstring for generate_access_token, service_account_path can be used to build the required format for a service account:
client = iam_credentials_v1.IAMCredentialsClient()
name = client.service_account_path('[PROJECT]', '[SERVICE_ACCOUNT]')
scope = []
response = client.generate_access_token(name, scope)

But service_account_path returns a format that the IAM API doesn't accept:
>>> print(name)
projects/[PROJECT]/serviceAccounts/[SERVICE_ACCOUNT]

The correct format is: projects/-/serviceAccounts/[ACCOUNT_EMAIL_OR_UNIQUEID]  I don't know if all service account API endpoints accept this format.

The path generated by the library seems to match up with the API definition. See the proto file.
Is the API returning an error if you use the path from client.service_account_path?

Yes, I get a 400 Bad Request.  You should be able to reproduce by running the example code (you'll need to fill in an actual scope, project, and service account).
Could the proto file be wrong?  The docstring for generate_access_token says this:
Args:
    name (str): Required. The resource name of the service account for which the
        credentials are requested, in the following format:
        ``projects/-/serviceAccounts/{ACCOUNT_EMAIL_OR_UNIQUEID}``. The ``-``
        wildcard character is required; replacing it with a project ID is
        invalid.

That matches the language from the discovery document for iamcredentials.

Ah I see, it looks like that docstring is coming from https://github.com/googleapis/googleapis/blob/57e72ebf709dc5ebbd27a746d6b92b05c054dea2/google/iam/credentials/v1/common.proto#L36-L45
If you pass '-' for project manually the helper method will produce the correct path. That said, it isn't ideal that the method accepts project when it must always be '-'.
client.service_account_path('-', '[SERVICE_ACCOUNT')

Hi,
I checked with the some other folks, and it looks like passing - to the method is the correct thing to do here. The API owners may support passing projects in the future, so we want to leave the possibility open in the pattern.


Pre-release versioning

As part of the CD flow, when creating a release I can identify a release as a prerelease within the GitHub user interface. This information however is lost when the package is published to NPM.
Investigate whether the “preid” option on the “npm version” cli command can enable forwarding this information to the npm repository.
The reason for this is all the users appear to be downloading the unstable candidate releases rather than staying on the stable release.

A secondary prerelease channel should be added for this to NPM.
This would allow for a CD pipeline for builds as they occur while keeping the main release channel only updated when a release has been published. So it should only affect the build pipeline workflow.
Proposed name is: homebridge-automower-platform-prerelease
This would help with testing upcoming changes without requiring an official release.


knex seed primary keys don't start at 1

Environment
Knex version: ^0.15.2
Database + version<EMAIL_ADDRESS>OS: MacOS / Docker (CentOS)
Questions about how to use knex
When running a seed without specifying the PK field, it seems that my id's start at 9 instead of 1, every time I rerun the seeds. Even when using Truncate+Cascade. I don't feel like I should have to specify a pk in each seed as I'm expecting my increments to start at 1 with every rerun of the seeds. Or am I wrong?

Why is this expected behavior? This is actually incorrect

Show me your database truncate code.

Also questions how to use knex doesn't belong here, as issue template also states.

Using Postico, right click on the table name, click Truncate..., and rerun the seed is how I currently do it. I'm sorry about the formatting. I didn't really understand the issue template.

However, in my unit tests, I use knex-db-manager with the following in my afterAll block
dbManager.truncateDb( ['knex_migrations', 'knex_migrations_lock'] );

thats strange, since I've written that truncate code for knex-db-manager and I know that it gives an option to query, which truncates also id sequences of corresponding table.
Please check from your database after truncate that those sequences are also set to 0.

also knex truncate with postgresql seems to work just fine
> knex('accounts').then(console.log)
Promise {
  _bitField: 0,
  _fulfillmentHandler0: undefined,
  _rejectionHandler0: undefined,
  _promise0: undefined,
  _receiver0: undefined }
>   knex:client acquired connection from pool: __knexUid2 +2s
  knex:query select * from "accounts" trx1 +2s
  knex:bindings [] trx1 +2s
  knex:client releasing connection to pool: __knexUid2 +2ms
[ { id: '1',
    last_name: null,
    email: null,
    logins: 1,
    balance: 0,
    about: null,
    created_at: null,
    updated_at: null,
    phone: null },
  { id: '2',
    last_name: null,
    email: null,
    logins: 1,
    balance: 0,
    about: null,
    created_at: null,
    updated_at: null,
    phone: null },
  { id: '3',
    last_name: null,
    email: null,
    logins: 1,
    balance: 0,
    about: null,
    created_at: null,
    updated_at: null,
    phone: null } ]
> knex('accounts').truncate().then(console.log)
Promise {
  _bitField: 0,
  _fulfillmentHandler0: undefined,
  _rejectionHandler0: undefined,
  _promise0: undefined,
  _receiver0: undefined }
>   knex:client acquired connection from pool: __knexUid2 +14s
  knex:query truncate "accounts" restart identity trx1 +14s
  knex:bindings [] trx1 +14s
  knex:client releasing connection to pool: __knexUid2 +9ms
Result {
  command: 'TRUNCATE',
  rowCount: null,
  oid: null,
  rows: [],
  fields: [],
  _parsers: [],
  RowCtor: null,
  rowAsArray: false,
  _getTypeParser: [Function: bound ] }
> knex('accounts').insert({}).then(console.log)
Promise {
  _bitField: 0,
  _fulfillmentHandler0: undefined,
  _rejectionHandler0: undefined,
  _promise0: undefined,
  _receiver0: undefined }
>   knex:client acquired connection from pool: __knexUid2 +3s
  knex:query insert into "accounts" default values trx1 +3s
  knex:bindings [] trx1 +3s
  knex:client releasing connection to pool: __knexUid2 +6ms
Result {
  command: 'INSERT',
  rowCount: 1,
  oid: 0,
  rows: [],
  fields: [],
  _parsers: [],
  RowCtor: null,
  rowAsArray: false,
  _getTypeParser: [Function: bound ] }
> knex('accounts').then(console.log)
Promise {
  _bitField: 0,
  _fulfillmentHandler0: undefined,
  _rejectionHandler0: undefined,
  _promise0: undefined,
  _receiver0: undefined }
>   knex:client acquired connection from pool: __knexUid2 +3s
  knex:query select * from "accounts" trx1 +3s
  knex:bindings [] trx1 +3s
  knex:client releasing connection to pool: __knexUid2 +1ms
[ { id: '1',
    last_name: null,
    email: null,
    logins: 1,
    balance: 0,
    about: null,
    created_at: null,
    updated_at: null,
    phone: null } ]

Try to come up reproduction case for this one and I'm sure you will find what is the problem in your code.

Steps taken

Truncated database (restarting identity and cascading)
Ran migrations using npm run knex migrate:latest
Ran seeds using npm run seed:run

Issue seems to have resolved itself. This doesn't quite make sense to me, though. Is there some way you might be able to explain it to me? Why on a fresh database would this happen?

Could I run dbManager in my seed file to ensure proper indexing?

knex(tableName).del() doesn't reset sequence, so that might have been a problem

I don't doubt that might have been the issue. And I was certain it was an issue with how I'd written it, not an issue with the library itself. Would it be bad practice to truncate instead of .del()?

Truncate and del creates different queries. .truncate() does restart identities and does TRUNCATE query and del() does just DELETE table without where statement, so everything gets deleted. Neither one is a bad practice, just different queries.
Documentation of truncate could be better 😐 I don't know if it restarts sequences on every dialect, but with postgres it does.

So calling knex.truncate restarts identities. Does it also cascade?

If worst comes to worst, I'll just use raw to accomplish this, but it would be much better to have it in the library

i don't think it cascades. Yea, I think there is open issue about truncate and cascade somewhere here.

Ah, lemme find it because this may just end up being a duplicate of that issue

https://github.com/tgriesser/knex/issues/1506#issuecomment-446975889


removed the dot on the pytest command on  the README.rst

removed the dot on the pytest command on  the README.rst

Thanks, @avallbona!

For anyone looking for context, see pytest's 3.0.0 release notes:

Introduce pytest command as recommended entry point. Note that py.test still works and is not scheduled for removal.


Bring back host field in HttpGet probe

Address issues:
https://github.com/rancher/rancher/issues/20749
https://github.com/rancher/rancher/issues/19992
Problem:
On redeploy/UI update, httpGet.host in yaml manifest is removed.
Solution:
Bring it back in API scheme
The Host header in HTTPHeaders field can be a substitute for it, but we probably shouldn't break existing manifests using the host field.

LGTM

This was intentionally and explicitly dropped by @ibuildthecloud in this pr https://github.com/rancher/types/pull/73. Unfortunately, not a good comment or linked issue.
We need to understand why he dropped it before we can add it back.

LGTM


Can I generate character sequences with Ludwig?

Also, does it support Transformer model?

Please, read the docs, the faqs and the blogpost that describe Ludwig BEFORE asking questions.
The short answers are "yes, it can generate characters" adn "not yet, the transformer is coming in the next update".

I'm sorry, I found the transformer plans in FAQ. However, I can't find a sequence of character generation example on the list of examples. I'm looking for a way to do what char-rnn does with Shakespeare text but with Ludwig. Care to help me?

Use a text feature and set level: char ;) In the docs: text feature

I have input file called shakespeare.csv of 1mb with:
First Citizen:
Before we proceed any further, hear me speak.
All:
Speak, speak.

All:
Resolved. resolved.
First Citizen:
First, you know Caius Marcius is chief enemy to the people.
All:
We know't, we know't.
...
I have test.yaml with:
input_features:
-
name: shakespeare
type: text
encoder: rnn
cell: lstm
bidirectional: true
level: char
output_features:
-
name: shakespeare
type: text
I use ludwig experiment --data_csv shakespeare.csv --model_definition_file test.yaml butain it doesn't train, gives me "TypeError: 'int' object is not callable".
I suspect there are some things I'm doing wrong, I want to generate new text in the style of Shakespeare. If you know how to do it please tell me so I can do it. Thanks.

you have to respect Ludwig's data format. READ THE DOCS.

 They are obviously not clear enough, at least on my problem. Being Ludwig promoted as easy to implement as it is, I was expecting a little quick help instead of just saying read the docs like a robot. For now I'll wait until Uber hires someone who can respond to my questions properly.

@aletote this entitled and abusive attitude is not constructive and will not get you any help from anyone.


support Pattern subscribe in kafka compat client

Describe the bug
Kafka wrapper not support pattern subscribe, in this link
Since Pulsar now support Pattern type subscribe, we should add the support for this.

Hi @jiazhai , I'd like to work on this. I just want to know how can we get the partitions for those matched topics, which can be used to construct TopicPartition like subscribe(Collection<String> topics, ConsumerRebalanceListener callback) did. Could you please give me some hints ?

@liketic
I am not sure you need to get the partitions. Because pulsar client already handles that.
If you want to subscribe to a list of topics, you can do that in ConsumerBuilder to set a list of topics.
Hope this clarify your questions.

@liketic Thanks for take care this issue. Thanks @sijie for the clarification.
the original change for pulsar java client is in PR #1279
you could also reference #3256.

Thank you @jiazhai @sijie . The method subscribe https://github.com/apache/pulsar/blob/41b286dc0cc7b6ca3aa24acfccd53dfed547c0c5/pulsar-client-kafka-compat/pulsar-client-kafka/src/main/java/org/apache/kafka/clients/consumer/PulsarKafkaConsumer.java#L215
lists all partitions for each topic and create a TopicPartition instance for each pair of them, and update the member variables consumers and unpolledPartitions for later use.
What I mean is if we need to do the same things, try to get all topics matched the pattern and then get all partitions for each topic...  , which means we need a method to list all topics based on a regex in client.
Otherwise, what I can think of is something like:
    @Override
    public void subscribe(Pattern pattern, ConsumerRebalanceListener callback) {
        ConsumerBuilder<byte[]> consumerBuilder = PulsarConsumerKafkaConfig.getConsumerBuilder(client, properties);
        consumerBuilder.subscriptionType(SubscriptionType.Failover);
        consumerBuilder.messageListener(this);
        consumerBuilder.subscriptionName(groupId);
        consumerBuilder.topicsPattern(pattern);
        try {
            consumerBuilder.subscribe();
        } catch (Exception ex) {
            throw new RuntimeException(ex);
        }
    }

@liketic
I see your point now. if we don't care about notifying ConsumerRebalanceListener, then we can just use #topicsPattern(pattern).
so I would suggest introducing a TopicPartitionListener in pulsar consumer to receive the partitions. so pulsar will manage notifying the topic partitions, instead of us doing this in kafka wrapper. so the implementation in kafka wrapper would be something like
    @Override
    public void subscribe(Pattern pattern, ConsumerRebalanceListener callback) {
        ConsumerBuilder<byte[]> consumerBuilder = PulsarConsumerKafkaConfig.getConsumerBuilder(client, properties);
        consumerBuilder.subscriptionType(SubscriptionType.Failover);
        consumerBuilder.messageListener(this);
        consumerBuilder.subscriptionName(groupId);
        consumerBuilder.topicsPattern(pattern);
        consumerBuilder.topicsListener(topicPartitions -> {
               callback.onPartitionsAssigned(topicPartitions);
        });
        try {
            consumerBuilder.subscribe();
        } catch (Exception ex) {
            throw new RuntimeException(ex);
        }
    }

There seems to be a few problems in the current implementations of kafka compact clients, I'm not sure if they have been reported:

KafkaConsumer states its subscribe methods are not incremental, which means new subscriptions will replace the old ones. However, implementions in PulsarKafkaConsumer are not, and seem to create dangling consumers if re-subscribe to the same topics.
https://github.com/apache/kafka/blob/f698f3f8401845e0f73b104c0f96c54d2851b67a/clients/src/main/java/org/apache/kafka/clients/consumer/KafkaConsumer.java#L957
https://github.com/apache/pulsar/blob/54b39e64f361a2ed5242259e04505bcfc66c1c8a/pulsar-client-kafka-compat/pulsar-client-kafka/src/main/java/org/apache/kafka/clients/consumer/PulsarKafkaConsumer.java#L267
ConsumerRebalanceListener.onPartitionsAssigned should be called when the partition becomes active.
Methods in PulsarKafkaConsumer are not totally thread safe, although concurrent collections are used.

It seems several places in PulsarKafkaConsumer are affected In order to use consumerBuilder.topicsPattern(pattern) for subscription, like messageId/offset management/seeking/commit, consumer management, etc. And maybe we should also reimplement the multi-topic subscription using that provided by the underlying pulsar client? That may simplify the implementation as a whole, but need more work and tests than expected.
https://github.com/apache/pulsar/blob/54b39e64f361a2ed5242259e04505bcfc66c1c8a/pulsar-client-kafka-compat/pulsar-client-kafka/src/main/java/org/apache/kafka/clients/consumer/PulsarKafkaConsumer.java#L240

@fantapsody

And maybe we should also reimplement the multi-topic subscription using that provided by the underlying pulsar client?

+1

The module is removed.
Closed as invalid. Perhaps it's something you can report to https://github.com/streamnative/kop.


Dbt oracle docs

Description & motivation
Added profile-oracle and oracle entry to supported databases file. I followed the format from Microsoft Server driver.

This looks great! Thanks for creating the plugin, and for adding to the docs site. (Sorry for the delay in reviewing.) Could you fix the file conflicts, and then I'll merge this in?
By the way, I see in the plugin readme that you implemented dbt-integration-tests + some additional custom testing. We are nearly release-ready with a more comprehensive, flexible adapter testing suite: https://github.com/fishtown-analytics/dbt-adapter-tests. We'll push it to pypi when ready. I'd be happy to open an issue in the dbt-oracle repo with more details, if helpful.

@jtcohen6 It would be very nice if you could open the issue for using the new adapter tests. I remember that the integration tests  used dbt_utils so I there a few test cases that I couldn't make it work without actually addign some code to dbt_utils.

Thanks for fixing the merge conflicts. I'll open an issue in the dbt-oracle repo as soon as our testing suite is on pypi.
Two small procedural things that slipped my notice last week:

Can you change the base branch of this PR from master to current?
Can you add reference/warehouse-profiles/profile-oracle to sidebars.js?

Done!


Advanced-search example added in example directory

Hey here 👋 thanks for the contribution.
This example doesn’t appear to use elasticsearch. Was that intentional?

Can you please check. I made some changes

The examples seem to use regular javascript search methods (.includes()/.filter()), not the external elasticsearch engine. there might be some confusion here since you are using elastic-search in the title

Yeah changed title please check it

Still a lot of changes for a simple substring search.

Typically the examples are meant to show example usage of express itself. This example seems focused on something non-express (implementing a static search). The entire example is just a single express route, for instance, and there are already examples for how to define a route, how to read a query string, and how to reply json. I'm unclear on the value added by this particular example is all.

I'm closing this as there hasn't been any further discussion on this, and it was last expressed that it is likely not really for the inclusion in the examples.


Create Official release of AlmaLinux WSL

Create Official release of AlmaLinux WSL 8.4
Publish WSL image to Microsoft Store for download

Issue update, registed with Microsoft store and created an App. Working with Microsoft to publish it.

AlmaLinuxWSL app is submitted to Microsoft store in last week of December, waiting on approval

Hello everyone,
I am very happy to announce that our AlmaLinux WSL is available from Microsoft Store https://www.microsoft.com/en-us/p/almalinux-8-wsl/9nmd96xjj19f
try it out and give us some feedback


zset module can't shard file between many server, only get data from ？

zset module can't shard file between many server, only get data from ？

I don't understand your question.


feat(controller): allow named function syntax when registering controllers

This pull-request means to introduce two new syntax for registering controllers:
$controllerProvider.register(function FooCtrl($scope) {
  // ...
});

$controllerProvider.register(['$sope', function FooCtrl($scope) {
  // ...
}]);

As it is shown above, using a named function we can remove the need for the first argument on the controller declaration, which was the controller's name.
This feature relies first on Function.name, which is currently implemented in most major browsers, with the exception of Internet Explorer. Because of that, I have also added a fallback method to get a function's name, which was mostly based on @dfkaye's Gist, although I've pretty much skiped the validation part which I think is unecessary for this case.
P.s.: I've made tests for both use-cases and all test (including e2e) seem to be working just fine.

I also dislike it, and since it didn't really pick up, I'm gonna close it.
For debugging, it should be enough to give your controller function a name regardless.


Update datamodels

update resource and data of the datamodel instead of creating temporary datamodels and resources

Updating datamodels works


Add something like -e to only dump the main exe for the game

The files for GDK games can be modded without dumping them, except for the exe. To mod Hi-Fi Rush, I had to dump the whole game, which is pretty big and took a little while, and all I needed was the exe so I could patch it. After that, I just renamed the existing exe in the original game files and copied in the patched one and the mods I added worked fine from there.

A more general form of this would probably be to just allow a regex expression to be used as a filter for file paths to dump in case someone only wants the .exe or only wants the .pck files by providing the appropriate regex expression.

While I understand you added a -f or -filter argument, you've provided no examples to what "regex expressions" are supposed to be.
I've tried in every single way possible and UWP Dumper still dumps whatever it likes. I am trying to dump "Starfield.exe".
If I give it "Starfield.*exe" it dumps any file that has "Starfield" in its naming convention, disregarding the ".*exe" part completely. If I give it "exe", it dumps any file. With or without the "". So what's the proper formatting for this filtering?

Not sure if this applies to every game, but a workaround I've learned is to click on the exe and them hit the delete key to send it to the recycle bin. Then take it out and you can modify it.


Release 0.8.0

The first release candidate for v0.8.0 is ready for testing!
Download the installer at https://github.com/marcelklehr/nodist/releases/tag/v0.8.0-rc1

Now would be a good time to test: https://github.com/marcelklehr/nodist/releases/tag/v0.8.0-rc2 ;)

v0.8 is out now. Woot!
(@gmahomarf  I hope I didn't break bash support again. Feel free to report any issues you find! :) )

Woohoo! @marcelklehr as soon as I get a chance, I'll install this version from scratch and let you know

@gmahomarf In v0.8.{0,1} bash support was broken, I think, because of line endings, but v0.8.2 should work in both git bash and even Cygwin! \o/


使用内置 Bracket Pair 不生效

Issue Type: Feature Request
关于Bracket Pair Colorize 2不再维护使用内置着色器。
在settings.json中配置
{"editor.bracketPairColorization.enabled": true,
"editor.guides.bracketPairs": "active"
}
该方法不生效。页面中代码还是没有颜色区别。
VS Code version: Code 1.69.2 (3b889b090b5ad5793f524b5d1d39fda662b96a2a, 2022-07-18T16:12:52.460Z)
OS version: Windows_NT x64 10.0.19044
Restricted Mode: No

感谢您创建此问题！
然而，由于您没有使用英语，我们很难进行处理。如果可能，请您将此问题描述修改为英文。请您直接编辑此问题，不要提交回复或新问题。
请注意，机器翻译服务通常无法处理用于讨论软件问题所需的技术性语言，所以我们推荐您寻找一位熟悉相关技术的人来协助翻译。
如果您无法做到，没有关系。此问题已经被标记为需要翻译，等待社区成员志愿进行翻译。

Thanks for creating this issue!
Unfortunately, as the description is not in English, it's hard for us to work on. If possible, please edit the original issue to be in English.
In our experience, automatic translation services are not able to handle the technical language needed to effectively communicate about software issues, so we recommend finding a human translator who is familiar with our technology to help.
Otherwise, we have marked this as needing translation and a community member may be able to help out.

@Linyas Is it possible that you have the following setting:
"editor.language.colorizedBracketPairs": []
If that's the case, try removing it.

I think I saw the problem：
I didn't set it that "editor.language.colorizedBracketPairs": [].
but I have setting "editor.language.brackets": [],
I'll try to comment it out, and that's it ok.
thank you for you!
------------------ 原始邮件 ------------------
发件人:                                                                                "microsoft/vscode"                                                                                @.>;
发送时间: 2022年7月25日(星期一) 晚上6:08
@.>;
<EMAIL_ADDRESS>主题: Re: [microsoft/vscode] 使用内置 Bracket Pair 不生效 (Issue #155927)
@Linyas Is it possible that you have the following setting:
"editor.language.colorizedBracketPairs": []
If that's the case, try removing it.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Hey @hediet, this issue might need further attention.
@Linyas, you can help us out by closing this issue if the problem no longer exists, or adding more information.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Clarify git api usage

@lszomoru https://github.com/microsoft/vscode/issues/173204#issuecomment-1426981409

Closing this PR in favour of https://github.com/microsoft/vscode/pull/192139.


fix: Now only adds owners once

#46  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @jhechtf and the rest of your teammates on  Graphite


[Feature request] Numbers representing pallete colors

I find my self counting 1 by 1 to get the id/number of the color for palette to use in code.
We can think on how to map this more efficiently. But for starters, maybe a small number (color int code) in palette window, for each color, could do?

It will end up in the next build!

Available in v0.4.0-beta10.


style: Implementing components page.

fixes #418

Pull Request Test Coverage Report for Build 612

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 77.284%

Totals

Change from base Build 586:
0.0%

Covered Lines:
1243

Relevant Lines:
1539

💛  - Coveralls

please fix the text styles


Bot will occasionally place blocks midair when it falls

Behold:

It seems to happen when the target (in this case the player) is located below the bot and across a gap or hole.
It also happens when you block-place the bot (place blocks in the way of a jump, causing it to fall). Don't ask me why I was trolling the bot though...
Most likely, any case of falling (while attempting to bridge) will cause this to happen.
In my use case, the pathfinder is being used in combination with a self-made PVP system for 1.8-- but when I disable the PVP code, the issue persists (though it seems a little better at not doing that since it's more sure of where it's goal is when I'm not constantly changing it).
As a side note, this is a simulated Bedwars game, and I don't intend to use it on public servers without consent.
That's about all the information I have if you have more questions I'll do my best to answer them.

Can confirm, it also tends to do this when given Levitation, like from a shulker.


CPU gradient definitions

Hi,
Firstly, thanks so much for making this code available. It's a really useful resource for learning more about RNN-T.
I have two questions, if I may:
i.) in cpu_rnnt.h the gradients are defined as:
ProbT g = alphas[idx(t, u)] + betas[idx(t+1, u)];
grad[idx(t, u, blank_)] = -std::exp(log_probs[idx(t, u) * 2] + g - loglike);

and
ProbT g = alphas[idx(t, u)] + betas[idx(t, u+1)];
grad[idx(t, u, labels[u])] = -std::exp(log_probs[idx(t, u) * 2 + 1] + g - loglike);

Following rnnt_notes.pdf, as far as I understand the formula used here is equation 9. From this, I can account for the following part of the above code:
ProbT g = alphas[idx(t, u)] + betas[idx(t+1, u)];
grad[idx(t, u, blank_)] = -std::exp(g - loglike);

but I'm not sure where the log_probs[] comes in here?
ii.) Following on from my first question, it might be obvious that I'm not an expert in calculus and so I was wondering if you'd be able to point me to any resources or give any hints so that I can learn more myself about how to derive the gradient of the loss from eq. 5, 7 and 8 in rnnt_loss.pdf? Even the smallest hint would be greatly appreciated!
Thanks again!

Well, the gradidents here are to log probability, \frac{\partial L}{\partial \log p} = \frac{\partial L}{\partial p} \frac{\partial \exp{\log p}}{\partial \log p} = p \frac{\partial L}{\partial p}.

Thank you! That makes a lot more sense to me now :-)
Referring now to the component g - loglike (in log domain): I understand how g is derived but it's so far been a mystery to me why the -loglike is there.
Would I be right in saying that this is because the loss, L, is defined in log domain, i.e. L = -lnP(y*|x) and so we end up dividing through by this P(y*|x) term (loglike) as a result of applying the chain rule to take the derivative of the log?

how is this different than CTC loss?


Error: Cannot make a conjunction of disjunctions

Hello,
I'm trying to filter tasks using Python SDK. Thus, I want to get from ArmoniK only specifics status within a specific range, so I do need to use multiple nested and and or operands.
According to Filter class:
    def __and__(self, other: "Filter") -> "Filter":
        # [...]
        if self.is_true_disjunction() or other.is_true_disjunction():
            raise Exception("Cannot make a conjunction of disjunctions")

It looks like the filter doesn't support this basic logical expression: (a|b)&c
Is there another way of doing this kind of filtering?

Hi @dbrasseur-aneo,
I recently had an issue with a workaround I used to address this lack. Meanwhile I saw that you opened a branch for this fix, is there any delivery date currently scheduled?
Thanks

Hi ! No delivery date yet, but it should be done after my vacations.


User tutorial api

GET (obtain lists of read tutorial, favourited tutorial)
UPDATE (store newly read tutorial, newly favourited tutorial, unfavourite tutorial)

Wrong repo


3D preview renders images wrong

For some reason, 3D preview on the parts tracker is broken. Here's how unofficial part 19086.dat looks:

Missing cyli can be seen around peg holes, that are present on the old tracker:

This bug has been traced to the library parse and import script specifically:
If the first line after the header is "0 BFC INVERTNEXT", that line is stripped from the file.
This has been fixed and this issue will be closed when the import import is performed.


logout and delete account issues

Describe the bug
Testing Trial Account I found:

Delete Account doesn't Delete account and logout
After delete the account the app just keep loading
When I create a trial account and try to backup an error screen shows up

To Reproduce
Steps to reproduce the behavior:

See error in the video

Expected behavior
The app needs to logout without any issues.
The app needs to display a backup or create account page.
Screenshot and video

https://github.com/GaloyMoney/galoy-mobile/assets/105733968/bf8401e2-173d-473d-979c-f82f664efe90
Smartphone (please complete the following information):

Device: [Poco X4]
OS: [Android ]
Version [13 MUI 14]

Additional context
Add any other context about the problem here.

@nicolasburtey I'd like to work on this one. I've gathered that the appCheck system is validating whether or not a device is allowed to use a the 'trial account' service. None of my devices / emulators are passing this check in dev or in prod. Therefore I can't create trial accounts for investigating. Are there some settings on the Firebase account that I should know about that would be causing this?

Emulators are not getting an appcheck by design. For physical device, I don't think google has published what they use to determine if a device is valid or not. It may likely be that privacy settings are preventing appcheck to be successful
we are on working in running the mobile tests with a backend running also locally, this may be best to wait on that enhancement to work on this ticket.


"NewRelicCapacitorPlugin" plugin is not implemented on ios

Description
I followed the instrumentation guide for my Ionic Capacitor app. Android is working but I am getting errors on iOS.
https://docs.newrelic.com/docs/mobile-monitoring/new-relic-mobile-ionic-capacitor/get-started/introduction-new-relic-ionic-capacitor/
Steps to Reproduce

Install agent with guided install
build the hybrid app
run the app on an ios emulator

Expected Behavior
I should not see errors on the Safari developer console, and the new relic dashboard should display an App Launch data point
Relevant Logs / Console output

Your Environment
MacBook Pro Intel

XCode: Version 14.1
<EMAIL_ADDRESS><EMAIL_ADDRESS>
Additional context
I am not super familiar with iOS development,  so there may be set-ups I need to do on xcode? However, when I was following through the guided installation UI, the only instruction needed on the iOS side was to run the command pod install, and that's what I did.  While Android did provide additional guidelines with working with the android code.

@Toxiapo we will review this issue from our side on Monday.

Thank you @ndesai-newrelic! Please let me know if you need more context from my side.
(This is a monorepo project with NPM+React framework using<EMAIL_ADDRESS>and<EMAIL_ADDRESS>

@Toxiapo can you share example app where you reproduced this issue? I am unable to do it with capacitor react app?

@ndesai-newrelic I do not have an example app, but I can try to create an MVP of my app

@Toxiapo can you do that? it will be very helpful to us

Hi @ndesai-newrelic, I could not replicate the bug using an example app but I believe I know why it was happening. We can close this issue. Thank you!
For anyone who's getting similar errors: The problem was that when I installed the plugin and sync the assets to the ios/android folder, it failed to include the new plugin in the Podfile like the example shown here. To resolve this, I manually added the missing plugin to the Podfile and everything seems to be working fine.
@ndesai-newrelic  I had assumed that 'npx cap sync' would automatically incorporate new plugins into the native platforms, but I haven't yet determined why it didn't work in my configuration.


[fuzzing] add Cargo.lock to Libra fuzzer workspace

Motivation
The Libra fuzzer workspace doesn't have a lock file.  The fuzzer
workflow was broken twice in the last three weeks due to dependency
changes.
This commit locks the crate dependency in the fuzzer's workspace.
Test Plan
Tested locally  with cargo run --bin libra-fuzzer fuzz accumulator_merkle_proof -- -- -runs=1

Instead of this we should put ibra-fuzzer/fuzz in the main workspace. Remove these lines https://github.com/libra/libra/blob/master/testsuite/libra-fuzzer/fuzz/Cargo.toml#L20-L22
Then it will use the toplevel cargo.lock.
I believe the cargo-fuzz tooling puts this in by default just to isolate it, but I don't think it is needed in our case, and we certainly want to be fuzzing exactly same set of libs, etc.

Good idea! Will fix and resubmit.


Entity selection stays not filtered

First of all, great work.
I noticed that in Entity Selection the dropdown doesn't stay filtered and it jumps back and shows all entities.
For example:
When entering "lights....." only the "lights." entities are displayed for a short time (as it should be), then the dropdown list is re-rendered and shows all entities again.

Best Regards

+1
It needs to be sorted alphabetically and filter when text is entered.

Having the same issue

Same issue, which put me off trying anything 😔

I'm aware of this issue that I don't understand for now, but I will fix it. But beside that almost everything seems to work.


isOpen() returns true even if there is no camera

Migrated from http://code.opencv.org/issues/3880#note-4
I am hitting a bug on opencv 2.4.10, ubuntu 14.04 and gcc 4.8.2. isOpen() returns true even if there is no camera in my machine. Then get this message:
GStreamer Plugin: Embedded video playback halted; module v4l2src0 reported: Cannot identify device '/dev/video0'.
OpenCV Error: Unspecified error (GStreamer: unable to start pipeline
) in icvStartPipeline, file /home/coutinho/src/opencv-2.4.10/modules/highgui/src/cap_gstreamer.cpp, line 383
terminate called after throwing an instance of 'cv::Exception'
what(): /home/coutinho/src/opencv-2.4.10/modules/highgui/src/cap_gstreamer.cpp:383: error: (-2) GStreamer: unable to start pipeline
in function icvStartPipeline
Aborted (core dumped)

Confirmed bug on latest 2.4 branch! Using Ubuntu 14.04 x64, no webcam connected to the system.

#include
#include

using namespace std;
using namespace cv;

int main( int argc, const char** argv )
{
    VideoCapture test(0);
    Mat frame;
    if(test.isOpened()){
        cerr << "The camera is open!" << endl;
    }
    return 0;
}

fixed in https://github.com/Itseez/opencv/pull/5202


Keys rotation refactoring

:x: Build TheIdServer 1.0.2-user-ole1-0071 failed (commit https://github.com/Aguafrommars/TheIdServer/commit/352eb4333c by @aguacongas)

:white_check_mark: Build TheIdServer 1.0.2-user-ole1-0072 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/9b7583b33c by @aguacongas)

:white_check_mark: Build TheIdServer 1.0.2-user-ole1-0073 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/8807242658 by @aguacongas)

:x: Build TheIdServer 1.0.2-user-ole1-0074 failed (commit https://github.com/Aguafrommars/TheIdServer/commit/379e886eec by @aguacongas)

:white_check_mark: Build TheIdServer 1.0.3-user-ole1-0084 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/1d82ed2175 by @aguacongas)

:white_check_mark: Build TheIdServer 1.0.3-user-ole1-0087 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/43ec2bcc25 by @aguacongas)


PointShop Admin Bug

My friend has this server and i am the Co Owner but for some reason whenever we go into the pointshop we dont have the little admin tab in the top right corner so we cant give or take away any point plz help

AssHole


Rename note to notification throughout the codebase

Fixes #865

Replaced occurrences of note with notification.
Updated related comments and documentation.

Originated from feedback by @ahoppen regarding terminology confusion. This change aims to maintain clarity across the codebase.
Reference: Apple’s issue tracker rdar://116703667
Please review.

Argh, looks like this conflicted quite heavily with https://github.com/apple/sourcekit-lsp/pull/896. Could you rebase the PR on top of main.
Sorry for causing the conflicts.

The issue has been resolved by https://github.com/apple/sourcekit-lsp/pull/1353.


Update diffusers-interpret to work with the latest diffusers package (0.8.0)

When using diffusers-interpret with the latest diffusers (0.8.0, yes, I need this version because I use Euler discrete scheduler), it gives the following error:
ImportError: cannot import name 'preprocess_mask' from 'diffusers.pipelines.stable_diffusion.pipeline_stable_diffusion_inpaint' (/usr/local/lib/python3.7/dist-packages/diffusers/pipelines/stable_diffusion/pipeline_stable_diffusion_inpaint.py)
Can it be fixed to work with it? 0.3.0 is now very outdated.
Thanks!

I understand the frustration, but I tried my best to make this package compatible with diffusers and Hugging Face did not help.
I suggest diffusers-interpret users help me and write in this Issue and in this PR, stating that they'd that diffusers incorporated those changes, to make packages like diffusers-interpret to be fully compatible for newer versions.
Usually, the Hugging Face community helps when there is a considerable amount of people interested.

To be honest, all the needed changes were made in this PR.
It's not worth it for me to redo the code if they have not approved it previously.
Try to comment in that PR and tag a couple of maintainers to get their attention to the lack of compatibility with packages like this 😞

I started writing in different PRs, let's see if they really make a change. In the meantime, it would be awesome to have a pypi diffusers package adapted to be compatibile with this but on the latest version!

@JoaoLages we can follow this suggestion and open the two feature requests.

@invoke-ai would also like an API that lets us provide our own text_embeddings, as it's necessary for us and anyone else who does any kind of token-weighting.
We're still in the very messy middle phase of our diffusers integration, and we haven't started trying to submit pipeline API changes like this upstream yet, but I think we'd be very much onboard for a PR or feature request for that.
As things stand at the moment, diffusers really expects an application this sophisticated will more or less completely disregard their existing StableDiffusionPipeline class and write their own.
There has been some progress on this front, for example StableDiffusionPipeline now exposes methods like prepare_latents and decode_latents an application-specific subclass can take advantage of. But Patrick seems to really like long methods and hasn't been prioritizing extensibility.
(InvokeAI is interested in adding some interpretation aids. We'll see how this goes.)

Hi, I wanted to follow up as presented demo colab is not working at all. When downloading StableDiffusion pipeline I got error
`Fetching 33 files: 100%
33/33 [00:00<00:00, 1808.93it/s]
TypeError                                 Traceback (most recent call last)
 in <cell line: 8>()
6 device = 'cuda' if torch.cuda.is_available() else 'cpu'
7
----> 8 pipe = StableDiffusionPipeline.from_pretrained(
9     "CompVis/stable-diffusion-v1-4",
10     use_auth_token=True,
/usr/local/lib/python3.10/dist-packages/diffusers/pipeline_utils.py in from_pretrained(cls, pretrained_model_name_or_path, **kwargs)
371                         load_method_name = importable_classes[class_name][1]
372
--> 373                 load_method = getattr(class_obj, load_method_name)
374
375                 loading_kwargs = {}
TypeError: getattr(): attribute name must be string`
I found an error about it on SO: https://stackoverflow.com/questions/74687769/typeerror-getattr-attribute-name-must-be-string-in-pytorch-diffusers-how
Which points diffusers to 0.3.0.
When I tried to update it and use recent version, I got error:
Requirement already satisfied: mpmath>=0.19 in /usr/local/lib/python3.10/dist-packages (from sympy->torch>=1.9.1->diffusers-interpret) (1.3.0) Installing collected packages: diffusers   Attempting uninstall: diffusers     Found existing installation: diffusers 0.24.0     Uninstalling diffusers-0.24.0:       Successfully uninstalled diffusers-0.24.0 Successfully installed diffusers-0.3.0 WARNING: The following packages were previously imported in this runtime:   [diffusers] You must restart the runtime in order to use newly installed versions.
When I force to use 0.24, I got error on importing explainer:
`---------------------------------------------------------------------------
ImportError                               Traceback (most recent call last)
 in <cell line: 1>()
----> 1 from diffusers_interpret import StableDiffusionPipelineExplainer
2
3 explainer = StableDiffusionPipelineExplainer(
4     pipe,
5
2 frames
/usr/local/lib/python3.10/dist-packages/diffusers_interpret/explainers/stable_diffusion.py in
4 import torch
5 from PIL.Image import Image
----> 6 from diffusers.pipelines.stable_diffusion.pipeline_stable_diffusion_inpaint import preprocess_mask
7 from torch.utils.checkpoint import checkpoint
8 from diffusers import StableDiffusionPipeline, LMSDiscreteScheduler, StableDiffusionImg2ImgPipeline, \
ImportError: cannot import name 'preprocess_mask' from 'diffusers.pipelines.stable_diffusion.pipeline_stable_diffusion_inpaint' (/usr/local/lib/python3.10/dist-packages/diffusers/pipelines/stable_diffusion/pipeline_stable_diffusion_inpaint.py)`
So I tried to install it with pip again:
Installing collected packages: diffusers   Attempting uninstall: diffusers     Found existing installation: diffusers 0.24.0     Uninstalling diffusers-0.24.0:       Successfully uninstalled diffusers-0.24.0 Successfully installed diffusers-0.3.0 WARNING: The following packages were previously imported in this runtime:   [diffusers] You must restart the runtime in order to use newly installed versions.
So I can create a pipe for stable diffusion (requires diffusers > 0.4)  OR I can use explainer (requires 0.3). Both of them does not work together in demo colab.


#1875 - Post Dropdown Menu UI for future Save, Hide, Follow & Report functionality

What does this pull request aim to achieve? 💯
closes #1875

adds post dropdown for posts not created by the logged in user
UI displays, buttons do not work yet

[x] I have checked that no one else is working on similar changes.
[x] I have read the latest README and understand the development workflow.
[x] The name of this branch is: feature/<branch_name>, or hotfix/<branch_name> if this is a hotfix, where <branch_name> briefly describes the issue(s) resolved and is prefixed with the issue number(s) (e.g. 1127-1130-update-git-branching-model). The branch is created in a cloned version of this repo, not a forked repo.
[x] This branch is rebased/merged with the latest staging (or the latest production if it is a hotfix).
[x] There are no merge conflicts.
[x] There are no console warnings and errors.
[x] On the right hand side of this PR, I have linked the appropriate issue(s) under "Issues" and added a project progress under "Projects".
[x] The title describes the issue(s) being addressed using format: <issue_numbers> - <brief_issue_description> (e.g. #1127/#1130 - Update Git Branching Model)

@nadinejuraschek please make sure this branch is rebased/merged with the latest staging.

Thanks @nadinejuraschek!


fix: obeys esbuild's external argument

This pull requests fixes this issue https://github.com/lucacasonato/esbuild_deno_loader/issues/31
The plugin now handles esbuild's external argument properly accepting wildcards

I'll add a test before landing this.


Integration with AIM systems

Using VCs for authentication/authorization purposes requires administrators to perform separate steps to add the appropriate roles to user profiles on the AIM system of choice, causing overhead.
We need a mechanism that allows to programmatically apply these changes on the target AIM system (e.g.: Keycloak), regardless of the architecture chosen.
Investigate the possibility of creating plugins that can be plugged at runtime to extend the basic functionality of Issuer Kit or, alternatively, using webhooks to deliver requests similarly to what aca-py does now with the controller.

I was thinking of this in the reverse way. Is there a way in Keycloak to add a plugin that calls Issuer Kit (or invokes Issuer kit functionality) to offer a credential to a user when their Keycloak account gets created.  That would eliminate the separate Issuer Kit/Keycloak consoles.

I need to further investigate how much "programming" can be done from within Keycloak. I am not aware of ways to "push" changes that happen in Keycloak out to other systems: as far as I know, it is delegated to the external system to use the APIs exposed by keycloak to figure out that something happened and act accordingly, dispatching requests to either Keycloak iteself or other resources.
This is why I was proposing a plugin/middleware or webhook delivery: this would abstract from the architecture of the AIM system and allow a custom piece of code that does what is necessary (e.g.: user creation, assigning/removing roles, etc.) when notified/triggered by Issuer Kit.

Agreed.  This is a lower priority and I'm hoping that Todd can try to learn things to help us.  It's not clear how tied we want to be to Keycloak, so we really need to think about that.  For example, I think that the MoJ folks want to use the VC to enable access to systems other than Keycloak, so a tight Keycloak integration might not even need to be a high priority.  Or maybe they make it easy and so it is a light lift.  Either way, we do should not be a big effort on this.

I think it's best to keep the implementation separate from the aggregator (KeyCloak in this case), and use APIs and webhooks to perform the necessary integration.  That way we're not tied to one specific product.

Yes -- if we did something specific for Keycloak, it would be a plugin for keycloak that used part of Issuer Kit as a dependency. And if keycloak supports that, others likely do.
The core identity kit should not have anything specific for Keycloak.


Remove default statsVar Count_Person when other statsVars are selected

Describe the bug
When no places are selected, default statsVar Count_Person is added.
In the old timeline tool, the default statsVar would disappear if users select other statsVars, now it stays.
To Reproduce
URL: https://staging.datacommons.org/tools/timeline#
Browser: Chrome
Steps to reproduce the behavior:

Go to https://staging.datacommons.org/tools/timeline#
Search for a place, e.g. United states
select a new statsVar

Expected behavior
Expect "Count_Person" to disappear after step3

I suppose we've fixed this by removing the default behavior of showing population. I think having the default chart helps usability... was it removed on purpose? If not, should we bring it back?
@qlj-lijuan feel free to ignore this if you've already logged off for the day :)

It was not removed on purpose. I didn't have time to add the default chart after the refactoring. We should add it back.

we should add a message telling users to pick a stat var


feat(tf2-base): update healthcheck

Currently if you have a Server IP set the healthcheck will fail because the server is not listening on all interfaces. With this change it will only use <IP_ADDRESS> to healthcheck if no other Server IP is set, or it is <IP_ADDRESS>.

Thanks!


Verify this feature works as expected.

Closing as the assigned testers are done testing


[Feature Request] gather non-tensors objects for metrics (remove duplicates)

As discussed in #1383, in certain NLP tasks, it is convenient to retain sentences as strings in the dataloader instead of tokenizing them into tensors. I propose implementing a function that gathers non-tensor objects and removes duplicates within the last batch. This function would complement the existing gather_for_metrics() (used for tensors) and enhance the capabilities for distributed evaluation in such scenarios.
Thank you.

Hey, I'd like to pick up this issue.
So basically we would need a new function like gather_for_metrics for objects and that removes duplicates within the last batch, correct?
As gather_object is implemented here: https://github.com/huggingface/accelerate/blob/main/src/accelerate/utils/operations.py
maybe it makes sense do build this in the same file.
What do you think @muellerzr ?

@Lorenzobattistela or to do a check if something is a tensor or not in gather_for_metrics, and call gather_object and de-dupe instead :) This way it will automatically work for anything we pass to it.
Yes that is exactly it! And ideally add some tests to accelerate/test_utils/scripts/test_metrics

Yeah this approach makes sense. I'll start now. Feel free to assign me :)

I thought something like this:
def gather_for_metrics(self, input_data):
        """
        Gathers `tensor` or object and potentially drops duplicates in the last batch if on a distributed system. For objects, duplicates are removed. Should be used
        for gathering the inputs and targets for metric calculation.

        Args:
            input (`torch.Tensor`, a nested tuple/list/dictionary of `torch.Tensor`, `object` or a nested tuple/list/dictionary of `object`):
                The tensors / objects for calculating metrics across all processes

        Example:

        ```python
        >>> # Assuming two processes, with a batch size of 5 on a dataset with 9 samples
        >>> import torch
        >>> from accelerate import Accelerator

        >>> accelerator = Accelerator()
        >>> dataloader = torch.utils.data.DataLoader(range(9), batch_size=5)
        >>> dataloader = accelerator.prepare(dataloader)
        >>> batch = next(iter(dataloader))
        >>> gathered_items = accelerator.gather_for_metrics(batch)
        >>> len(gathered_items)
        9
        ```
        """
        def check_input(input_data):
            if isinstance(input_data, (list, tuple)):
                return all(is_torch_tensor(item) for item in input_data)
            return is_torch_tensor(input_data)

        if not check_input(input_data):
            try:
                obj = gather_object(input_data)
                # deduplicate objects within the last batch
                if self.gradient_state.end_of_dataloader and self.gradient_state.remainder > 0:
                    deduplicated_objects = []
                    seen_objects = set()
                    for item in obj:
                        if item not in seen_objects:
                            deduplicated_objects.append(item)
                            seen_objects.add(item)
                return deduplicated_objects

            except Exception:
                return obj

        tensor = self.gather(input_data)

Did not found a util for dedup, am I missing something? Also I'm thinking a better name for input_data, which is not very descriptive. Maybe tensor_or_object.
What do you think @muellerzr ?
Thanks in advance

@Lorenzobattistela start at the implementation of Accelerator.gather_for_metrics, that's where it should live and be modified. (You should just see a call to gather there + the dedupe)

Actually this is at accelerator.py function gather_for_metrics

You mean accelerate.py  as in src/accelerate/accelerator.py at the function gather_for_metrics? Thats where i was changing

@muellerzr we can close this


feature: Implement "List Active Guild Threads"

Is your feature request related to a problem? Please describe.
The "List Active Guild Threads" endpoint is not currently implemented.
https://discord.com/developers/docs/resources/guild#list-active-guild-threads
Describe the solution you'd like
IGuild should have a method to list active guild threads.

I am getting:
[2023-02-16 21:27:11.183216] [SHOUT] [IgnoreExceptions] Unhandled exception was thrown
Error: type 'Null' is not a subtype of type 'bool' in type cast
Stack trace:
#0      new ThreadListResultWrapper (package:nyxx/src/internal/response_wrapper/thread_list_result_wrapper.dart:40:31)
#1      HttpEndpoints.fetchGuildActiveThreads (package:nyxx/src/internal/http_endpoints.dart:1642:12)
<asynchronous suspension>

I believe there is no has_more in the response, unlike
https://discord.com/developers/docs/resources/channel#list-public-archived-threads