TypeScript does not follow object keys when narrowing types

TypeScript Version:  4.0.3

Search Terms:
typescript narrowing types via object keys
Code
type A = {
    kind: 'A',
    aField: 'aField'
};

type B = {
    kind: 'B',
    bField: 'bField'
};

function getField(p: A | B) {
    if (p.kind === 'A') {
        // TypeScript knows that p is A
        return p.aField; // intellisense: kind, aField
    }
    // TypeScript knows that p is B
    return p.bField; // intellisense: kind, bField
}

function getField2(p: A | B) {
    switch (p.kind) {
        case 'A':
            // TypeScript knows that p is A
            return p.aField; // intellisense: kind, aField
        case 'B':
            // TypeScript knows that p is B
            return p.bField; // intellisense: kind, bField
    }
}

// TypeScript does not follow object keys when narrowing types
function getField3(p: A | B) {
    return {
        'A': (p as A).aField, // expected to work without type assertion (p.aField), since TypeScript has enough information
        'B': (p as B).bField, // expected to work without type assertion (p.bField), since TypeScript has enough information
    }[p.kind];
    /*
    Since I'm indexing with p.kind,
    TypeScript should know that for the branches where keys match one of p.kind possible values,
    p should pe narrowed to its type, the same way as when TypeScript manages to follow the if and switch branches.
    */
}

Expected behavior:
TypeScript should be able to follow object keys when narrowing types the same way it does when following if and switch branches.
Actual behavior:
TypeScript does not follow object keys when narrowing types even there is enough information.
Playground Link:
https://www.typescriptlang.org/play?target=99&ts=4.0.3#code/C4TwDgpgBAglC8UDeBYAUFTUDWBLAdgCYBcUA5DGQDTpZQCGAYrhADYnlMvtnoC+AbnTpQkKACEEyWljxFSZcdRmYARszYcy67oV5pBwtADMArvgDGwXAHt8UAOYRgG9gAowpOAB8JASmkMLFxjKA8AOjlCBHhECjIA1CC6TAB6VKgAFXAIAGULACdcMGAcfBsAdwBnKGAAC3pSsChcGpgVFILnUwL7MHCuTQEodJb8YDZWVoh8KohSKKoGV0IOvg7R7Mh8opKyypr6xqhm1okOruAevvCdIZGMggnWKbnZ+ZwCQiW79n4jMyWax2RzOFYAJg8XigvnEiQ6VQquGAFjqYX6UXhyRSFnoc3IlGIHRSWE2OR2xVK2HK1VqDSaLTaxJJUEu1xOAxWw1GT0m03eCy+S0Gf2xdFx+MUZCJYpZZO2hUp+1pRwZZ3EzJJbN6HN+hG5j3GfLec0FRB+KzW-zQ6HleUVe0INggNXKpWMNhelSgNlUACsIFYcBAQDUKnUZlB8PQCgVKgQHLUclV0ICrLZ7E4XLoAMxQ2Aw-yBOja+xJFkE6XohhtPyc3RLUYQAAekCsEGiwBsUAqNgK2B7yLqNlMpVE0DxcwKwPsERFhD8SyqBAs0C29t2pQaNRmI4caIIHoKAFtGhnNeQlKQPDX-LcVo2Mi22xNO93e-3B-UR2OcjWpzO6L3roi5QMulhruSDpbniUC7qY+5jEep4zmsADaGJfAAukI2KpAAVB0uQrtAACSZDHmMhAtgmX5oph5odOuFJ7FUw6mOwyp0scR50tAqgFPQlgRmGEZdMGoZQChqI+vg0A2KEDHRGANhVMuqisNAABu9CsKYLo0NizRsSOnFiNGsaVB2tTdsihw5Es9TQFU9DHtAFT0CAt7hpGzHQVJQn0E4hzdh6XoVHxLShEJ0SIsiMkCUJqIuuEHT4ak1pGOgQA
Related Issues:
None so far.

It would be completely invalid to "narrow" here, since an operation like this would fail:
function getField3(p: A | B) {
    return {
        'A': (p as A).aField.toUpperCase(),
        'B': (p as B).bField.toUpperCase()
    }[p.kind];
    /*
    Since I'm indexing with p.kind,
    TypeScript should know that for the branches where keys match one of p.kind possible values,
    p should pe narrowed to its type, the same way as when TypeScript manages to follow the if and switch branches.
    */
}

There'd have to be some notion of "expressions which can be safely evaluated and discarded" and only apply narrowing there, but this doesn't seem like a common enough case to invest engineering effort in such an isolated scenario.

@RyanCavanaugh thanks for feedback !


Is the controller fast enough for pulsing without interrupts?

Additionally, you should add a system timer interrupt for every 60th of a second, as per this site on the SNES controller. Do this on a separate file that we can both use as I also have to draw every 60th of a second!
In particular:

Every 16.67ms (or about 60Hz), the SNES CPU sends out a 12us wide, positive going data latch pulse on pin 3. This instructs the ICs in the controller to latch the state of all buttons internally. Six microsenconds after the fall of the data latch pulse, the CPU sends out 16 data clock pulses on pin 2. These are 50% duty cycle with 12us per full cycle. The controllers serially shift the latched button states out pin 4 on every rising edge of the clock, and the CPU samples the data on every falling edge.

But nice work on the controller so far!

This seems fine if we have a GPU interrupt!


Add semantic token modifiers to callable variables.

this is related to https://github.com/microsoft/pylance-release/issues/5785
basically, user could have something like this
func: Callable[...] = ...
it is a variable, but it can be used like function
also, user could have something like this
class Foo():
     def __call__(self): ...

func = Foo()
func()

here func is a variable again but it can be used like function
we could add semantic token modifier to these variables so users who want to distinguish them can use editor.semanticTokenColorCustomizations to do so.

how we will implement or how much perf cost it will have, we need to dig in but before that, we probably want to see how many people will upvote this.

I briefly looked, and we get decls for all identifiers (basically names), but we only get the type of the decl, not at each specific position, which will cause us to do the narrowing and other things Eric mentioned above.
So, if we do it based on the declared type, there won't be any extra cost. However, if we want it to be precise and correct at each position where identifiers are used, there will be extra performance cost.
That said, since the semantic token provider and checker (the one that produces diagnostics) will share the same cache, and since the checker will call getType on every identifier anyway, in reality, the overall cost should be the same. It would be just a matter of who does it first and gets the blame.
For the same variable/parameter being colored differently if we decide to do it per each position, since we just mark the token with extra data and people who want it need to opt in, it might be okay?
But I am not here to draw conclusions but to provide more context so people can discuss with more information in hand.

This is related to https://github.com/microsoft/pylance-release/issues/5612
Currently the same symbol can be treated either as a function or a variable depending on how it is first declared.


After deleting a managed node group from the eks_managed_node_groups cluster security group was also deleted.

Description
Please provide a clear and concise description of the issue you are encountering, and a reproduction of your configuration (see the examples/* directory for references that you can copy+paste and tailor to match your configs if you are unable to copy your exact configuration). The reproduction MUST be executable by running terraform init && terraform apply without any further changes.
If your request is for a new feature, please use the Feature request template.

[ x ] ✋ I have searched the open/closed issues and my issue is not listed.

⚠ Note
Before you submit an issue, please perform the following first:

Remove the local .terraform directory (! ONLY if state is stored remotely, which hopefully you are following that best practice!): rm -rf .terraform/
Re-initialize the project root to pull down modules: terraform init
Re-attempt your terraform plan or apply and check if the issue still persists

Versions

Module version [Required]:

Terraform version: v1.1.7

Provider version(s): 3.72.0

Module version: 18.0.6

Reproduction Code [Required]

Steps to reproduce the behavior:

Not using workspaces

Didn't clear local cache

Delete a managed node group from eks_managed_node_groups then run terraform plan and terraform apply
Expected behavior

Delete a managed node group from eks_managed_node_groups without any side effects.
Actual behavior

After deleting a managed node group from the eks_managed_node_groups cluster security group was also deleted.
Terminal Output Screenshot(s)

Additional context

Its not the cluster security group that is deleted, its the security group that is created with each node group https://github.com/terraform-aws-modules/terraform-aws-eks/blob/d969e94bbea89ed616713bdab849f042ceda87ac/modules/eks-managed-node-group/main.tf#L362

Its not the cluster security group that is deleted, its the security group that is created with each node group
https://github.com/terraform-aws-modules/terraform-aws-eks/blob/d969e94bbea89ed616713bdab849f042ceda87ac/modules/eks-managed-node-group/main.tf#L362

Thanks for the reply, I created a new eks cluster and try to reproduce the issue, but the auto-delete EKS control plane sg issue is not happening, so I try to find a clue from the cloud trail, turns out the sg was accidentally deleted before we delete a managed node group from the eks_managed_node_groups. sorry for opening the issue.


Netlify deployments

Is your feature request related to a problem? Please describe:
I am running into a bunch of issues when trying to deploy to netlify. I cross reference my netlify dashboard and do not see this project in there (i am also not sure why i would?) I was never asked to sign into netlify or anything so that is weird.
Describe the solution you'd like:
Some kind of netlify integration with my Bolt account and netlify so they can talk to eachother. Espically when trying to publish to production.
Describe alternatives you've considered:
None at the moment, I am stuck on trying to deploy.
Additional context:
The chat often references a "netlify CLI" but i would not have known that unless I aksed.

Hi! 👋 Thanks for the comments!
You can publish to netlify anonymously, so you are still provided with a deployment URL.
There's also a URL that allows you to 'claim' the page - which will allow you to manage it directly through your Netlify account.
Are you able to see these URLs when deploying?

In my case, I had to remind the LLM that it is able to deploy the project for me anonymously as you suggest @kirjavascript... using this prompt:
"The maintainers of bolt.new have commented in github issues that you should be able to deploy this project anonymously to netlify for me. Could you please try to do that? I am almost out of tokens for our session (29k left), so please limit text conversation with me as much as you can."
After that it successfully deployed and provided the URL as demonstrated in ViteConf.
Before that it was encouraging me to search my Stackblitz account for integration settings with my Netlify account, but also described how I'd manually deploy it through github/netlify.
Sidenote: This was the first time I could deploy an app through bolt.new with no extra features before running out of free-tier tokens, so I'd say you guys are making great progress 👍

This should now be resolved, thanks!


⚠ Secret Website 3 has degraded performance

In 36f24a7, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 2664 ms

Resolved: Secret Website 3 performance has improved in 09038e1.


Remove geo replication for GCR

We have been doing geo-replication for a long time, but it's costly and we don't tell our end-users about it.
It costs about 4x to store images in Artifact Registry vs Container Registry. 0.10 USD vs 0.026 USD for a multiregion GCS bucket.

Storing in just gcr.io/knative-nightly which is the US multi-region should be sufficient.
/cc @dprotaso @cardil

/lgtm
/approve

/cherry-pick release-1.15

/cherry-pick release-1.14

Note - for the changes to take effect in release branches downstream repos need to update their knative.dev/hack dependency

Also @upodroid to save on cost should we delete the images in those regions?

I will delete the registries in knative-nightly project today.
I haven't migrated the prod one yet.


Release Google.Cloud.Run.V2 version 2.2.0

Changes in this release:
New features

Adds support for custom audiences (commit 5233add)

Created release for Google.Cloud.Run.V2-2.2.0


compact/canonicalize in execute (after evaluation)

[ch2936]

@djspiewak hm so the only place I've found where we're materializing before storing or returning results is in SegmentsWrapper.sliceToSegments, which we call (for one) in MimirPTableStore#write. We need to call materialized in SegmentsWrapper.sliceToSegments though because an ArraySegment contains an array of its actual values.

Maybe a better approach then would be to make materialized check to see if columns are already materialized and leave them as-is.
Also I vaguely remember inserting a call to materialized somewhere in slamdata-backend.

@djspiewak I checked SDBE and there are no calls to materialized.

Hurray for faulty memories! Good to know.


Split runtime/common/crypto into a crate

This PR simply moves the crypto utilities and some other dependencies to a crate so that they're easier to consume by downstream libraries (in this case, a Rust gateway). The alternatives are to copy the code verbatim; or else pull down altogether too many dependencies also used by the runtime.
The gateway just needs the ECDH box functions, so it's not awful to copy/re-implement those, but then it's harder to apply patches from this repo (but the code is pretty static and well tested, so maybe nothing will change).

Thanks for the review. I was holding off on merging this because it wasn't 1000% clear that it'd be useful, but it turns out that it will be now. I'll also need to reboot the rust client PR in the oasis-sdk repo to fetch the runtime public key, so this week will be one of great closure :)

I ended up not using this because the crate functions invoke the global allocator, which isn't preferred. I copied the one function out, which should be fine.


Test the images

The images are a completely untested part of the codebase, it would be good to put some tests in there.

We have some tests around the images now. They can be expanded upon once the image system is overhauled


Clicking links and touch scrolling fail on mobile

After updating from 1.6.6 master to current master, scrolling by swiping and clicking links doesn't work. I compiled it with and without the mobile cmake option and with and without setting the build type to Release.
Scrolling with the scrollbar still works.
Hardware: PinePhone
OS: Mobian with Phosh

There were a couple of tweaks in mouse/touch event processing in v1.7, so something must now be incompatible with the touch events on PinePhone/Mobian. I'll poke around a little and see if I can detect anything on my end, but unfortunately I don't have a PinePhone to debug with so success is not guaranteed...

@justjosias Have you checked if this is fixed for you in v1.8?

It works quite well now. Thank you! All the touch events appear to be operational.


Refactor createSubscription to use async/await instead of Promises

What is missing from the application? Please describe.
On the ecars-pwa createSubscription function we are mixing Async/Await with Promises.
Describe the solution you'd like
Refactor to use only async/await

Fixed by #87


strconv.Atoi: parsing "3.220000e+01": invalid syntax"

time="2021-03-09 12:33:32" level=warning msg="fail to get value for TemperatureDegC: strconv.Atoi: parsing "3.220000e+01": invalid syntax" file="extensions/edgex_source.go:153" rule=led_off
Please let us know which version has the fix.
PS: Device is Raspberry Pi 4.
Thanks
Vijay Kumar

There is no value descriptor needed for edgex v2 and eKuiper. eKuiper will only support edgex v2 since v1.2.1 thus this problem won't be fixed.


feat: add public notebook for evaluating models in Vertex AI Studio and model garden

Description
feat: add public notebook for evaluating models in Vertex AI Studio and model garden
chore: update introduction texts and titles for gen ai evaluation service tutorials

Added a section titled: "Understand the EvalTask class"


[WR-357, WR-395] KeyCloak and config_ignore

This PR installs the KeyCloak module and fixes the Drupal config infrastructure to ignore the KeyCloak settings. This includes:

Install KeyCloak module and dependencies
Remove config_split module (as per WR-395) and remove src/sync/split files
Enable config_ignore and ignore KeyCloak and Backup / Migrate settings
Replace config_split settings in settings.php with the simpler approach of excluding specific modules in non-dev environments
Remove redundant environment variables

Attempting to sync results in the following errors
 [error]   (Currently using Missing or invalid module The following module is marked as installed in the core.extension
configuration, but it is missing:
 * config_split

  The "config_split" entity type does not exist.
manually installing the config split module followed by executing the drush cr and drush cim commands get everything up and running.
we may need to do this in two stages.  First do all the configuration changes without uninstalling the config split module.  Once everyone (or at least AWS-DEV) is on that master with the new config, then uninstall the module.

Thanks for the review. I'll try again with a dump from live to test the workflow.

I was able to reproduce with a dev dump. I solved it using the following:
drush cdel core.extension module.config_split
drush cr

then the sync script passed. Thinking about how to make this productionizable.

Pushed a small hack to force-delete the config_split module and keep going with sync. @rjkuyvenhoven, please try again. Thanks!

Thanks for the confirmation. However, I realized that this will not work for AWS because we run Composer during the Docker build phase, not at runtime. There's no Drupal environment that drush can find during the build phase. I am pausing this branching and decoupling Keycloak from config_split.

I was able to perform the following workflow successfully on my local:

Restore the latest dump from DEV
Uninstall the config_split module from the UI
Run sync successfully
If @rjkuyvenhoven doesn't mind, I will call this a success and merge it to master as is. Local devs will not need to manually uninstall the module because the Composer post-install script takes care of that already as you previously confirmed.

ok, sounds good to me.


Add pcp/pmcd status check.

@twiest @kwoodson Will kill the main "pcp start" process which will cause the container to exit.

@twiest latest version removes full paths in scripts

:+1:


[$250] IOS - IOU-Send Money Button is sometimes triggered after double tapping on add attachment

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open test app
Tap on a contact
Tap on +
Double tap on add attachment
Close the popup menu, just typing outside somewhere else
Double tap on add attachment again

Expected Result:
I expect adding attachments pop up to open
Actual Result:
The send money button is triggered and opens after double tapping on add attachment button
Workaround:
Unknown
Platform:

Where is this issue occurring?

iOS

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
Email or phone of affected tester (no customers): any
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

https://user-images.githubusercontent.com/93399543/173239482-f8e4e697-e242-4d76-9309-03b7245fdcbf.mp4
Upwork job URL: https://www.upwork.com/jobs/~0170e1181be2b46bb0
Issue reported by: Applause - Exploratory
Slack conversation:
View all open jobs on GitHub

Applying External label

Upwork job URL - https://www.upwork.com/jobs/~0170e1181be2b46bb0

i think you are struggling with the loops

@1810205045 can you clarify what you mean? Are you interested in taking this job? If so, please start by reading the Contributing Guidelines document.

I think this is pretty low value, I don't think people will double tap "add attachment" often, and if they get to that page it should be very easy to navigate back & try to add an attachment again. Does anyone else think this is something that should be fixed / addressed at this point?

Got a few people agreeing with me that this is low value, closing out


"Live Demo" section: Touched column remains always empty

On the https://react-hook-form.com/ website the "Live Demo" section at the bottom doesn't populate touched fields in the "Touched" column.

I'm totally new to react-hook-form (currently a user of redux-form, that's how I ended up on a website looking for alternatives). So, out of curiosity, I'll see if I can quickly fix the issue myself, as it's probably something simple :)

oops thanks i will take a look tonight (if you haven't)

Will submit a PR in a moment :)


Add service's website information to wappalyzer fingerprint database

@shelld3v just wanted to confirm if this is different from https://github.com/projectdiscovery/wappalyzergo/issues/3?

@shelld3v just wanted to confirm if this is different from #3?

Yes, what I wanted to say is that, when wappalyzer detected a technology, Wordpress for example, it should provide the link to that service (homepage), and https://wordpress.com is, in this situation.

@shelld3v I've included this information in https://github.com/projectdiscovery/wappalyzergo/issues/3, closing this one it will be handled as part of another issue.

@shelld3v this is now added with - https://github.com/projectdiscovery/wappalyzergo/pull/35


how can solve this ?

ValueError: not enough values to unpack (expected 3, got 2)

getting same error

(cnts, _) = cv2.findContours(blueMask.copy(), cv2.RETR_EXTERNAL,
change line 74


Migrate solution to .net core 3.0 for LINQPad 6

@lvond are you sure it's working ? I just tried your version, it compiles fine. I then zipped and renammed the zip extension to lpx6 to install it. The driver seems to load just fine, it finds the old connections back, but it's impossible to add a new one (can't specify a path, it defaults to c:*.csv).

@lvond thanky you very much for help with LINQPad6 migration.
I'll check this PR, merge it and add official nuget for installation soon.

@lvond are you sure it's working ? I just tried your version, it compiles fine. I then zipped and renammed the zip extension to lpx6 to install it. The driver seems to load just fine, it finds the old connections back, but it's impossible to add a new one (can't specify a path, it defaults to c:*.csv).

By impossible do you mean the textbox is not editable so you cannot edit the default path? Is at least Drag&Drop working for you? I had no problem with both of ways. Just spotted little defect in connection dialog - linqpad icon is missing.

@lvond I don't even get the connexion properties window. Here is what I do and what I get :

Start LinqPad 6 (
Click on "Add connection"
"Choose Data Context" window opens, select "CSV Context Driver" v1.6.7443.20506 (its location being LocalDataFolder)
Click "Next >" at the bottom right of "Choose Data Context" window
Small "Loading..." popup appears, then disappears
New connexion with what I guess are default parameters appears in the connexion list with the CSV icon and "c:*.csv" as connexion name. When expanded, it shows "No files found." in purple.

If I try to edit that connexion, or any prior connexion no connexion property window appears at all either. Only way I found to add or edit connexions is to launch LP 5, do the job there and then restart LP6 where it's automaticaly imported.
Note that I can query any connexion imported from LP5 with LP6. I just don't get the Connexion Property window.

Thank you again @lvond !
I'm going to merge the PR, solve remaining issues and publish official nuget for the plugin.


Tracking issue for Next Release

CC @eriknelson, @s0ph0s-2, @cbojar, @herbmillerjr & @hippyjake as the currently active people working on the project.
This is a tracking bug for the Next Release milestone.
The current list of tasks for Next Release follows (With the currently assigned people in brackets):

[x] Async title page reload (@eriknelson)
[x] Abstract Admin functionality (@eriknelson)
[x] Make plugins that use data.json more robust (@cbojar)
[ ] Implement 'silent mode' (@eriknelson)
[ ] Track down unknown memory leak (@leestripp & @eriknelson)
[ ] Implement command cooldowns using cinch-cooldown (@s0ph0s-2)
[ ] Implement Rate limiting (Currently no one, mostly needs input and then a simple PR)
[ ] Add variable title limits per show (Currently no one)
[ ] Switch Twitter API from polling to streaming (@rikai)
[ ] Migrate to google-api-client 0.9.x (Currently no one)

I've created this as a central place to collaborate on finishing off these tasks in a timely fashion. The original stated goal date of this release was 'Mid-September', with the caveat of possibly taking to the end of September.
If it's still possible to hit that goal, that'd be really nice. However, if outside forces prevent working on stuff, that's understandable as well and stating as much as fine. It'll just let other people take over in your stead. :+1:
Feel free to use this issue for discussion of collaboration, passing off of information or anything else relating to getting these tasks done.
This issue will be updated as tasks are completed.

Also, would doing some sort of 'group sprint' this weekend be something that would help people along?
I'm willing to be available via IRC if we wanted to make a group effort! :100:

That kind of thing seems to help the Solus project a lot. I can only take part in such when it's my turn to elder-sit the parental unit (like last weekend, when I started contributing). But whenever it lands on one of those weekends I'm game. Sorry I can't help with one for this release cycle.

Due to the lack of responses, i think i'll delay this until next weekend, that way everyone will have a week to plan for it. :+1:

@eriknelson, @s0ph0s-2, @cbojar, @herbmillerjr & @hippyjake Any of you guys able to make it at any point between now and sunday? :)

I have exams this upcoming week, so I might have some time after I finish studying.
On 2016 sep 15 15:54 -0400, rikai<EMAIL_ADDRESS>wrote:

@eriknelson (https://github.com/eriknelson), @s0ph0s-2 (https://github.com/s0ph0s-2), @cbojar (https://github.com/cbojar), @herbmillerjr (https://github.com/herbmillerjr) & @hippyjake (https://github.com/hippyjake) Any of you guys able to make it at any point between now and sunday? :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub (https://github.com/rikai/Showbot/issues/70#issuecomment-247434734), or mute the thread (https://github.com/notifications/unsubscribe-auth/ATl-TSJd0tLb8ZDVj8k9tR570lc1mBcnks5qqaJ7gaJpZM4J4Sjk).

Yeah, I should be able to drop in here and there.
On Sep 15, 2016 3:54 PM, "rikai"<EMAIL_ADDRESS>wrote:

@eriknelson https://github.com/eriknelson, @s0ph0s-2
https://github.com/s0ph0s-2, @cbojar https://github.com/cbojar,
@herbmillerjr https://github.com/herbmillerjr & @hippyjake
https://github.com/hippyjake Any of you guys able to make it at any
point between now and sunday? :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rikai/Showbot/issues/70#issuecomment-247434734, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAxYaQHazX3hd6909a7dSTLigV4RWe7-ks5qqaJ7gaJpZM4J4Sjk
.

Alright, just give me a ping if you're active in the IRC and working on something so i can keep track. I'm going to be working on the tweetstream stuff myself this weekend, but i'd be happy to see discussion/activity on any implementation details or whatever over the course of this weekend.
Hoping for a big push to check off the rest of those boxes and release! :+1: :dancer: :meat_on_bone:

Sadly, we missed our release date goal due to everyone being extremely busy, but that's understandable.
That said, I'd still like to get things crossed off the list, so if you guys could give me a status update on things when you've got a chance, that'd be wonderful! :meat_on_bone:

I hope to finish up my issue this weekend, but I can't fully commit to it until I know how much other work I'll have.
On Oct 3, 2016, 11:03 -0400, rikai<EMAIL_ADDRESS>wrote:

Sadly, we missed our release date goal due to everyone being extremely busy, but that's understandable.
That said, I'd still like to get things crossed off the list, so if you guys could give me a status update on things when you've got a chance, that'd be wonderful! 🍖
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub (https://github.com/rikai/Showbot/issues/70#issuecomment-251131114), or mute the thread (https://github.com/notifications/unsubscribe-auth/ATl-Tc25UwaMwKbgQ_JhF-ldZZQFyhcSks5qwRk7gaJpZM4J4Sjk).

Initial work on the tweetstream code is now in the FEATURE/tweetstream branch.
The code is pretty minimal to get it up and running, but I'm running into issues restricting the output to only the users tweets, rather than replies from others and retweets as well.

@s0ph0s-2 Had a chance to look at the updated cinch-cooldown?


Events being called multiple times on server restart

Hi, first of all, nice work!
If the server restarts, the io.On(Socket.EVENT_CONNECT, ... ) function is called again, thus registering all the events inside this method.
So the next time an event is emitted it is called multiple times.
Shouldn't the Socket.EVENT_CONNECT event be fired only once?

Maybe if you post some code, we can help you


Backbone Views

I've added a bunch of views and wired them up. There are some more minor changes that need to be done to make this golden. I wanted to update the progress...

Nice. thank you.


Add Option To Create CNAME R53 Record For Cluster

Describe the solution you'd like.
Allow users to pass in a R53 zone id and a DNS name to create a Route 53 Record
Describe alternatives you've considered.
Can add it outside of the module, but would be nice if this logic was in one place.

Hi Thomas,
This kind of feature request is not something we are going to support in this module, because of the nature of the resource modules. RDS Aurora has no relation to Route53 service in order to operate in the first place.
Please use route53 module to manage records or zones.


Move snapshot service to edge

Summary of changes
Changes introduced in this pull request:

Changed forest-tag value to edge from latest.

Reference issue to close (if applicable)

Closes #106
Other information and links

Yeah, edge is not backward compatible with latest so you need to change a few flags. You can deploy this version under a different Terraform key and test that it works.


Update Gnosis SDK Packages

Description
We are updating the Gnosis Safe dependencies as we found an issue while testing the integration of kwenta.io into the Safe:
https://github.com/gnosis/safe-apps-list/issues/107
Kwenta app was not able to calculate gas fees in Optimism because they call a method that is not currently supported in the version you are currently using.
I updated as well the way you get the safe info as the method was renamed and moved.
Tested locally inside the Safe using Kwenta seems to solve the issue and the app works as expected.
Checklist

[x] The version field in package.json is incremented following semantic versioning
[x] The box that allows repo maintainers to update this PR is checked
[x] I tested locally to make sure this feature/fix works
[x] This PR passes the Circle CI checks (if merging a fork run yarn with node version 12.22.7 to ensure no errors)

Hey @Adamj1232 thanks. I sent the contact details to our BizDev so he can ping Shen ☺


A spinning wheel should be shown while onInit waits for the promise

Category

[ ] Question
[ ] Typo
[ ] Bug
[X] Enhancement / Suggestion
[ ] Additional article idea

Expected or Desired Behavior
When I override onInit and return Promise I would like SPFx to show a spinning wheel until that promise is fulfilled.
Observed Behavior
If it takes long time to fulfil the promise in onInit, nothing is shown where the web part is supposed to be.
Steps to Reproduce
Add this to a WP
public onInit<T>(): Promise<T> {
  return new Promise<T>((resolve: (args: T) => void, reject: (error: Error) => void) => {
    setTimeout(() => {
        resolve(undefined);
      }, 5000);
  });
}

This is fixed in the code. Should be available in the next drop. Please let us know if you still find the issue after the next drop.

Works as expected now. Thanks for the awesome job!


Update details on ManiSkill/SAPIEN about new GPU parallelized sim + parallelized rendering

Added some updates.
Furthermore I have two questions about things to add (that are clearly biased towards my own work so I do not add them in this PR):

I would like to add ManiSkill under the simulators list as most robot learning people will use that API as opposed to direct SAPIEN for GPU simulation / parallel rendering but it's not exactly at the physics/simulation engine of the tech stack (and I am obviously very biased here).
I would like to also add a category to the comparison table about parallel rendering as it is a mostly unique feature of ManiSkill/SAPIEN, and I believe mjx also supports parallel depth-only rendering via mjx.ray although I don't know if that is an official feature (and I haven't tried it myself yet).
I leave the two points above a decision to be made by the other maintainers.

Thanks!

Thank you very much for the information!

(1) Thank you for your valuable suggestions. I've also noticed that SAPIEN and ManiSkill have always been merged together, with the latest APIs often appearing with the release of a new ManiSkill. Do you think it's a good idea for us to combine them both? Like in the title, we name it SAPIEN&ManiSkill or something similar.
(2) It sounds great! Thank you very much for your suggestions and help. We already have a table for parallel rendering, but if you could add more information and make the table complete, that would be great!

Sure I think this can be useful. The projects are built/maintained by the same group essentially, and IsaacGym has it's own section despite not being the low level simulation framework. (Actually IsaacGym is deprecated in favor of IsaacLab now). I can make a PR to refactor a bit later

Where is the table for parallel rendering section? I only found the details for FPS for a single environment

(1) Thank you so much!
(2) Oh yes. here we compared the rendering speed for a single env. Parallel rendering is also very very important feature and previous simulators can not achieve it well. People struggle it a lot. Thanks ManiSkill for the contribution and let's make a table for it!


Take storake keys as reference (K -> &K)

This change encourages the use of ownes types as PrimaryKey, which makes it very easy to deserialize bytes into owned keys. This can be implemented for Addr.
Since you cannot deserialize borrowed bytes into an owned object, a copy is created in parse_key. However, parse_key is never used.
In order to avoid copying around owned keys when thes are needed in multiple locations, the key arguments of all the storage functions were turned into references.
This is basically how the type system of standard library maps like https://doc.rust-lang.org/std/collections/struct.HashMap.html works: You have an owned key type K that is used plain in a few places. But get takes a reference. In HashMap the K can be of unknown size (e.g. [u8]). This is something we cannot do as long as we want to keep parse_key, which deserializes into an owned K.

Do not merge as is. Target branch is a copy of the fork from #260.

The fact of parse_key deserializing to an owned value is irrelevant, in my opinion.

This is crucial. Before you had K in and K out. If K was a reference, you created the requirement described by Ethan here: https://github.com/CosmWasm/cosmwasm/issues/868#issuecomment-816531741 which is almost impossible to fulfill by anything other than renamed &[u8]s. You cannot put K in and *K out.
But what you can do is put &K in and get K out, which is shown here.
This makes a few things a bit less fancy. But I think those get nicer over time when you migrate K from refs to values consistently. One problem you run into is that the literal b"foo" gives you an &[u8; 3] instead of an &[u8], which makes some things slightly less pretty than before. But this is really just a problem of test code.

Agreed. But as you mentioned, parse_key is currently not being used. I think we're creating our own parsers when we need them.
Anyway, both options are OK with me.

Let's revisit this when we tackle #198
For now, going for the much more straightforward #264


KAFKA-12944: Assume message format version is 3.0 when inter-broker protocol is 3.0 or higher (KIP-724)

Also:

Deprecate log.message.format.version and message.format.version.
Log broker warning if the deprecated config values are ignored due to
the inter-broker protocol version.
Log warning if message.format.version is set via ConfigCommand.
Always down-convert if fetch version is v3 or lower.
Add tests to verify new message format version based on the
inter-broker protocol version.
Adjust existing tests that create topics with an older message format
to
have the inter-broker protocol set to 2.8.
Add upgrade note.

Note that the log compaction change to always write new segments with
record format v2 if the IBP is 3.0 or higher will be done as part of
KAFKA-13093 (with Kafka 3.1 as the target release version).
(cherry picked from commit a46b82bea9abbd08e550d)
Reviewers: David Jacot<EMAIL_ADDRESS>David Arthur
<EMAIL_ADDRESS>Jason Gustafson<EMAIL_ADDRESS>Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

There were a few mechanical conflicts I had to resolve. No re-review required, but worth running the PR build before merging.

Test failures are unrelated.


Formatted attributes

closes #38

overall looks great :white_check_mark:
just one thing:
you moved methods from export.rb --> attributes.rb
can you do the same for export_spec.rb --> attributes_spec.rb?

doing to make a branch for #37


unit test for SGSparseMatrix

@iglesias sorry, I rewind my commit from the fork which closed the original pull request. I have corrected the indentation, and changed the variable names "rows and cols" to "features and vectors"

Thanks for the new pull request, @yingryic. See the comments above and also have a look at the few comments regarding style left.

@iglesias I have made some change, please review

Merging for the moment since everything is fine. But it would be nice if you answer the comments above and address them if you like in new pull request.


Can't play video (Android)

We've been using this plugin for 2 months in 2 different apps.
Something changed about 2 weeks ago in our dev environments.  Our apps don't play videos anymore.  I have the same problem with the demo project.
I assume(d) that it is something we did, but I can't figure it out.  We reinstalled nativescript.  We reinstalled Android SDK.  We downgraded nativescript versions.  Nothing seems to work.
So - I know we are probably doing something dumb.  I just don't know what.
In the demo app, the video never loads.  So it won't play.  It won't get a duration.  It doesn't take up screen space.
I added logging to videoplayer.android.js.  Video.prototype.initNativeView is never called.
Any clues would be greatly appreciated.

Is it local or remote video?

I just tried the demo and yea the video isn't playing. Really weird, last push I tested and all was fine. Not sure what happened. Will need some debugging to see what's happened.

Well, I hate to say it but I'm glad.  We've been banging our heads against a wall for 3 days trying to figure out what we did.  Thanks for looking at it.
BTW - it is for local and remote.

Yea it's not rendering so I imagine it's no initializing.
On Thu, Jul 6, 2017, 12:57 PM simplec-dev<EMAIL_ADDRESS>wrote:

Well, I hate to say it but I'm glad. We've been banging our heads against
a wall for 3 days trying to figure out what we did. Thanks for looking at
it.
BTW - it is for local and remote.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/bradmartin/nativescript-videoplayer/issues/80#issuecomment-313472242,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFulhOaaUpcKqTvB9XG6iRxrix0Bs4Pmks5sLSATgaJpZM4OP2Nz
.

@simplec-dev - have you tried NS 3.0 with the latest release of the plugin? Also can or have you tried different variations of NS with the latest release? I don't have time right now to figure out what changed to cause the issue so hopefully you can narrow it down to when it worked and what versions you have everything and we can find out. The last release I know I tested on android and iOS before publishing and the demo worked. So I'm guessing something in the core but of course that's a guess so possibly wrong. Lets just get different variations tested of core and this plugin to find when it last worked okay and then we can possibly ping the core team and see if they have someone to help debug it. I would but super low on free time this week and next :)

It was definitely working with NS3.0 / Angular.  Both of our apps are angular.
We upgraded to 3.1.1 around the time that this started because of an update to Telerik Pro UI plugins.  My first thought was that it was the problem.  Yesterday we uninstalled nativescript and installed 3.0.1 on a machine.  We also reverted the package.json so the version numbers were downgraded.  We still had the same problem.
We have one app that was style NS2.x.  When that was built, local video didn't work either.
My gut says it is something in core modules.  But, I'm relatively new to nativescript and haven't developed many plugins.  So it is just instinct based on other experience.

@NickIliev - any chance you have a minute to look into this. The demo worked when I released 3.0.8 of this plugin with the current NS version at that time, about 24 days ago. Think that was 3.0.1. Really curious if you can get the demo to run or know of anything that changed that might cause an issue with the video not initializing.

Just checked my android instance, not working for me either

When I downgraded tns core modules to 3.01 the video player started working again.  Something in 3.1.0 breaks this and the exoplayer plugin.
npm remove tns-core-modules --save
npm install<EMAIL_ADDRESS>--save
I'm also running the 3.0x runtimes; 3.1 should be compatible with 3.0 but it was my first test to see if it was the runtimes causing the issue.  Then I did the core modules...

I can confirm this. Android-only issue. iOS works fine on latest core-modules.

@bradmartin indeed the demo is not working with the latest tns-core-modules version. From the error I have received, it appears that this.mediaPlayer has value of null

Think I have a similar issue in mu Carousel plugin. After upgrading to 3.1.x all those new setNative properties are failing, all values are undefined.

Yea @nickilliev I think this is a bigger issue than just the video demo :(
I'll try to look into it today
On Mon, Jul 10, 2017, 2:09 AM Ned<EMAIL_ADDRESS>wrote:

Think I have a similar issue in mu Carousel plugin. After upgrading to
3.1.x all those new setNative properties are failing, all values are
undefined.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bradmartin/nativescript-videoplayer/issues/80#issuecomment-314023568,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFulhFkQNbjGgBtbYwdsDULYZdXR2mRdks5sMc4LgaJpZM4OP2Nz
.

@NickIliev - Just another datapoint.  3.1 also break the NS-exoplayer plugin on android...  3.0 works fine.

@simplec-dev - until the core-modules issue is pinpointed and we figure this out try downgrading the core-modules as mentioned ^^^

Thanks Brad.  I can't downgrade because of a ng-template bug that got fixed.  Apps are in a long QA cycle and we can avoid video for a while.  Thanks again.

@simplec-dev think I know the issue, will publish fix once confirmed and provide details and then go beat up @bnussey

Sorry to bother you on this one @NickIliev - the source for the android plugin had initNativeView and not createNativeView also not returning the native view instance. At any rate it did work with 3.0 so something changed to cause it to fail with 3.1 but at any rate the plugin has been fixed with version 3.0.9 @simplec-dev please check it out and let me know if you still have any issues. The demo works fine so I'm assuming all good to go now 😄

Our apps have functional video again.  Thanks Brad.

Nice one @bradmartin

@bnussey in california right? need a good excuse to visit 😆

Yeah mate in LA! Always welcome to visit, as long as it isn't to beat me up haha ;)


Refactor unit tests

Just created the following in the W3C unit tests:
  private $maxTimeInMilliseconds = 5;

  private function assertEquality($expected, $result)
  {
    $this->assertTrue($result == $expected, $expected ."\n\n". $result);
  }

  private function assertPerformance($startMicrotime, $stopMicrotime)
  {
    $tDiff = $stopMicrotime - $startMicrotime;
    $milliseconds = floor($tDiff * 1000);
    $this->assertTrue($milliseconds < $this->maxTimeInMilliseconds, $milliseconds .' is longer than '. $this->maxTimeInMilliseconds);
  }

  private function assertElementReturn($element, $config, $expected)
  {
      $base = '\Eightfold\Html5Gen\Html5Gen';
      $tStart = microtime(true);
      $result = $base::$element($config);
      $tStop = microtime(true);
      $this->assertPerformance($tStart, $tStop);
      $this->assertEquality($expected, $result);
  }

Recommend placing a base test class and refactoring the units tests. The main block of tests is the same for all elements (call assertElementReturn):

Performance test less than maxTimeInMilliseconds.
Equality test the expected string and the result from generation.

Some of this is complete - now it's a matter of updating the other tests to use this BaseTest.php class: https://github.com/8fold/html5-generator-php/blob/develop/tests/BaseTest.php

[ ] https://github.com/8fold/html5-generator-php/blob/develop/tests/UnitBlockArticleTest.php
[ ] https://github.com/8fold/html5-generator-php/blob/develop/tests/UnitBlockBodyTest.php
[ ] https://github.com/8fold/html5-generator-php/blob/develop/tests/UnitBlockFormTest.php
[ ] https://github.com/8fold/html5-generator-php/blob/develop/tests/UnitBlockGroupingTest.php
[ ] https://github.com/8fold/html5-generator-php/blob/develop/tests/UnitBlockHeadTest.php
[ ] https://github.com/8fold/html5-generator-php/blob/develop/tests/UnitBlockHtmlTest.php
[ ] https://github.com/8fold/html5-generator-php/blob/develop/tests/UnitBlockTableTest.php
[ ] https://github.com/8fold/html5-generator-php/blob/develop/tests/UnitBlockTableTest.php - promote assertSpeed() to BaseTest.php.
[ ] https://github.com/8fold/html5-generator-php/blob/develop/tests/UnitBlockTableTest.php
[ ] https://github.com/8fold/html5-generator-php/blob/develop/tests/UnitInlineEmbeddedTest.php
[ ] https://github.com/8fold/html5-generator-php/blob/develop/tests/UnitInlineTextLevelTest.php
[ ] https://github.com/8fold/html5-generator-php/blob/develop/tests/UnitScriptingTest.php
[x] https://github.com/8fold/html5-generator-php/blob/develop/tests/UnitW3CTest.php


Add support connect style middleware

With the release of my next-connect, it is now possible to use Connect-style middleware (function (req, res, next) {})
This PR attempts to have the deprecated default export to have this.

Code coverage will be hit. The PR to fix #38 will hopefully fix this.


[RFE] Add labels to CAPI kubeconfig secrets

Is your feature request related to a problem? Please describe.
CAPI v1.5.0 and higher requires that the secrets used for kubeconfigs have the label  cluster.x-k8s.io/cluster-name=${CLUSTER_NAME}. If we upgrade Rancher Provisioning (i.e. v2prov) to use CAPI 1.5.0 without these labels then provisioning will be broken with an failed to retrieve kubeconfig secret for Cluster error.
See the CAPI migration guide: https://github.com/kubernetes-sigs/cluster-api/blob/main/docs/book/src/developer/providers/migrations/v1.4-to-v1.5.md#other
Describe the solution you'd like
I'd like to see the label added to secrets early so that its not a problem if someone updates to use CAPI v1.5.0.

/backport v2.7.x

does this need a separate ticket for 2.9? I see the PR above for it, so I'm assuming we should track the work for it separately.

Closing as this has essentially been absorbed into https://github.com/rancher/rancher/issues/43109 / https://github.com/rancher/rancher/issues/43945 and will be indirectly validated per these tickets.


Improve release process

The current deploy process is ... well... laughable.  'Happy to admit it.  In my defense, I've been using this project as a bit of a sandbox to see how far one can push the limits of a static-file only, bundler-less, SPA.
But the time for that has past.  If nothing else, having to use htm and preact's template-string-based, like-JSX-but-not-actually-JSX syntax is getting old.
Not sure what other's think, but I've been playing with Parcel and it seems to work pretty well.  Setting that up as part of the build process, with a GH Action for deploying to the (coming soon) github pages version of this app probably makes sense.

Note: There's an ENV object that is used to configure bugsnag (bug reporting), that gets updated as part of the deploy process.  How/if that needs to be maintained is up for debate, but some sort of production bug reporting is a good idea.

npmgraph.js.org is now served directly out of the /docs directory on the /main branch.  The CI workflow requires that directory to be consistent with the source. (I.e. it fails if committer has not run npm build since making changes.)
There's still room for improvement but this is a start.


'batched' function looks buggy

https://github.com/facebookresearch/dlrm/blob/b631a99cf5ff320272d18a776cda85b5207bdf19/torchrec_dlrm/dlrm_main.py#L368
I don't think batched function is doing what we want it to be doing. What is the intended output from it?
Test case:
import itertools
from typing import Iterator

def batched(it: Iterator, n: int):
    assert n >= 1
    for x in it:
        yield itertools.chain((x,), itertools.islice(it, n - 1))

lis = [i for i in range(10)]
n = 5
for b in batched(lis, n):
  print(b)
  for item in b:
    print(item)

Output:
<itertools.chain object at 0x793e86d9d870>
0
0
1
2
3
<itertools.chain object at 0x793e86d9e050>
1
0
1
2
3
<itertools.chain object at 0x793e86d9d540>
2
0
1
2
3
<itertools.chain object at 0x793e86d9cc10>
3
0
1
2
3
<itertools.chain object at 0x793e86d9e740>
4
0
1
2
3
<itertools.chain object at 0x793e86d9d2d0>
5
0
1
2
3
<itertools.chain object at 0x793e86d9f7c0>
6
0
1
2
3
<itertools.chain object at 0x793ebd8bf520>
7
0
1
2
3
<itertools.chain object at 0x793e86c607c0>
8
0
1
2
3
<itertools.chain object at 0x793e86c603d0>
9
0
1
2
3

已收到邮件  阿里阿豆故咋一马斯！如未及时回复，请致电15868848097  QQ：812737452


Update dependencies and resolve Flow issues

Update dependencies and resolve Flow issues
This update bumps Fancy to v2.0.1, which removes react-scripts as a
dependency (this was added accidentally).
Other updates include the removal of 2 lodash dependencies and
resolving various Flow issues.

Coverage remained the same at 100.0% when pulling dab668eb7c73146c484a0a4c3e72f565a37015fd on update-deps-fix-flow into 44c495198ed95d29424fe57282c4ffb43e4d0640 on master.


When exporting the functions to the global space it will shows a notice or a warning if the functions already exists in the global space.

This is useful if you use another libraries that also export functions to global space.

Rebased and pushed as 1858cc3b6fba0cc20f3d79dcf852860db52ca612


Question: Is it possible to send multipart/form-data request?

Hello!
I'm working with an API that requires me to send some files as multipart/form-data. Is this possible to achieve with restish? This is how the command's help looks like:
  ## Argument Schema:

    {
      id: (string)
    }

  ## Input Example

    {
      "filename": [
        "string"
      ]
    }

  ## Request Schema (multipart/form-data)

    {
      filename: [
        (string format:binary)
      ]
    }

I've tried several approaches but I wasn't able to figure out something that works. Some examples:
1.
restish my-api upload-file 650d9c463c92333e6aa6cc0 @file
ERROR: Caught error: not sure how to marshal multipart/form-data

restish my-api upload-file 650d9c463c92333e6aa6cc0 filename: @file
ERROR: Caught error: not sure how to marshal multipart/form-data

I've also tried adding -H "Content-Type: multipart/form-data" to those commands but the result was the same. Am I doing something wrong or is there just no support for multipart/form-data? I will really appreciate any help, thanks!

@Klarys there's no special logic to handle multi-part forms in Restish yet. For now you can use curl to do it, e.g. see https://everything.curl.dev/http/multipart. I'll consider this a feature request to support it in Restish and will see what I can do.

Thank you for answering and considering this as a future feature request. This functionality is crucial for my use case, so it would be really great to see it implemented.


Build script should ask for session duration in hours

Seconds is a bit too granular for user input. I'm suggesting that we prompt for hours but store the value in seconds.

Resolved in 82a70f8d76a4bcb8ca2f03452d102659ebb4e065


Use CMAKE_CONFIGURATION_TYPES for build variants

Right now, the CMake build variants defaults to Debug, Release, MinSizeRel and RelWithDebInfo.
In my CMakeLists, I have manually added more types:
set(CMAKE_CONFIGURATION_TYPES "Debug;Debug_ARM;Release;Release_ARM" CACHE STRING "" FORCE)
I would like these options to be selectable when picking a build variant (sidenote: CLion has this behaviour).

Duplicate of https://github.com/microsoft/vscode-cmake-tools/issues/128


Add tests

WIP - Add testing-library/svelte & jest

Unit tests are still incomplete, but merging to enable ci test runs.
Will add in separate PR.


search word under cursor with *

In command mode, * over a word should jump to the next instance of that word in the buffer. I think this can piggyback on top of built-in search and replace without much trouble.

@zenhob using the atom-macros package I was able to define a couple macros that do exactly this:
@findNextWordUnderCursor = ->
    dispatchEditorCommand 'editor:select-word'
    dispatchEditorCommand 'find-and-replace:use-selection-as-find-pattern'
    dispatchEditorCommand 'find-and-replace:find-next'

@findPreviousWordUnderCursor = ->
    dispatchEditorCommand 'editor:select-word'
    dispatchEditorCommand 'find-and-replace:use-selection-as-find-pattern'
    dispatchEditorCommand 'find-and-replace:find-previous'

I think this behavior is more true to vim's * than the select-next command.

@robru, in #561 I have some fixes for * and # – is this related?

@jacekkopecky eh, define "related" ;-)
The above code achieves the desired behavior only if you install another package to enable it. It's basically just a workaround, not really relevant to vim-mode directly, no.


Test Cases appears to be wrong. Because the number of arguments for the function keeps varying

Challenge Seek and Destroy has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/56.0.2924.87 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:
function destroyer(arr, a, b) {
  // Remove all the values

  return arr[3];

}

destroyer([1, 2, 3, 1, 2, 3], 2, 3);

@arsubbu Thank you for reporting this issue. The test cases are correct. Please take a look at the objects linked on the challenge description to learn how to deal with them.
This is a standard message notifying you that this issue seems to be a request for help. Instead of asking for help here, please click the "help" button on the challenge on freeCodeCamp, which will take you to the help chatroom for that specific challenge. You can also view our full list of official chatrooms.
If you think I'm wrong in closing this issue, please reopen it and add further clarification. Thank you, and happy coding!


Big esp scaling issue

When you look away from esp but still see it it resizes very weirdly

I know this is a pretty old thread and you're waiting for a response, I think its because the esp is hitbox based.


Feature request: Color error line with "red"

First thank you very much for this incredible thing!

In #26 you said you don't want it to be language aware. I am okay with that. But messing both with IDEA Community, phpStorm and Clion demo, it seems IDEA application uses same API to mark erroring places.
It would be cool if CodeGlance could pull this info out of editor component and background those error lines with configurable(!) error color. Configurable because of themes, and I am not sure if themes/editor can expose their "Error stripe mark" setting (maybe they can).
Although built-in "linemap" already does this (colors line red), some single character errors like semicolons, parens and such (especially in closures) are still hard to spot.
I believe single "giant" "error line" background in CodeGlance would help when pinpointing place of error, and fixing errors would get much easier.

I would extend this request: color lines with the same colors as Intellij's scrollbar does.
My use case is: look for a changed lines in a file. Normally I spend time examining marks on a scrollbar, or press Ctrl-D and edit directly in a diff viewer.
Clear highlight of changed lines would help avoid this. Plus, it would be easier to navigate warnings etc.

How does this look? I haven't been using IDEA for some time but I plan to use it more again and after 5 years I came to check state on this and I see no movement whatsoever. Last repo activity seems to be from 3 years ago.
Is this very good project effectively dead?


iPad problem

Hi all,
I really like your work but i have a problem
I included your package to my xamarin forms project and i've created a page where:

I get the device position
I center the tkcustommap to my position
I place the mapspan to mapspan(myPosition,distance.fromkilometers(1))
I place a pin to myPosition
I place a circle centered in myPosition with radius driven by a slider

all of this work on Android and iPhone but on iPad the mapspan is always equal to default and nothing appear(pin and circle)
One thing to notice:
sometimes my app work on iPad if i build it in debug or ad-hoc mode
if I build in AppStore mode it doesn't work on iPad
Ty for your work

Check out this: https://github.com/TorbenK/TK.CustomMap/issues/29

Sorry i skipped that issue because on some devices everything works well
I'll try with the istance
Thank you

It works
Thank you


Add target ID checking to OpenOCD Flashing

The proposed enhancement is to add an ID check to the OpenOCD flashing routines (most likely in the scripts which call OpenOCD and give it commands). An OpenOCD script implementing ID check would...

Reset Halt
Read a target ID from a register on the MCU
Verify that the target ID matches the expected target ID

If a mismatch, alert the user to the fact that the .elf target does not match the connected target
Optionally direct them to the right location to re-configure their target .elf for the connected target

Only attempt to flash the device if the above check is confirmed

This could potentially prevent attempting to flash mismatched targets (e.g. flash a .elf that is built for MAX32690 MCU to a MAX32666 MCU), which sometimes can cause situations where it becomes difficult to read/write the device's flash again.

I have ran into this issue commonly and believe it is probably a good idea. @Jake-Carter have you messed with the flash scripts before?

Agreed.  Did some digging and there is a "family code" entry in the USN from the flash info block we can use for this.
Just put this together for the MAX78000.cfg.  It will catch any mismatched connection and shut down the OpenOCD server.
$_CHIPNAME.cpu configure -event reset-end {
	# The family code we expect to read from the MAX78000
	set FAMILY_CODE 0x85

	# Unlock the info block
	mww 0x40029040 0x3a7f5ca3 1
	mww 0x40029040 0xa1e34f20 1
	mww 0x40029040 0x9608b2c1 1

	# Read from info block.
	set info_block_pg1 [mrw 0x10800000]

	# Write to the info block again to re-lock it.
	mww 0x40029040 0xFFFFFFFF 1

	# Extract family_code.
	set usn_16 [expr { $info_block_pg1 >> 15 }]
	set usn_family_code [expr { ($usn_16 & 0xFF00) >> 8 }]
	if { [expr { $usn_family_code != $FAMILY_CODE }] } {
		echo "ERROR: Family code mismatch!  Expected $FAMILY_CODE but received 0x[format %X $usn_family_code]"
		echo "ERROR: Are you connected to the right microcontroller? (MAX78000)"
		shutdown 1
	}
}

Some more improvements are needed to improve the error messages and behavior.
On a mismatch, VS Code's flash tasks will look like this.  It will fail, but is not helpful.
 *  Executing task: arm-none-eabi-gdb --cd="/home/jhcarter/repos/msdk/Examples/MAX78000/Hello_World" --se="build/Hello_World.elf" --symbols=build/Hello_World.elf -x="/home/jhcarter/repos/msdk/Examples/MAX78000/Hello_World/.vscode/flash.gdb" --ex="flash_m4_run /home/jhcarter/repos/msdk/Tools/OpenOCD cmsis-dap.cfg max78000.cfg" --batch

The target architecture is set to "armv7e-m".
Open On-Chip Debugger 0.11.0+dev-g56a818e4c (2023-10-09-16:10)
Licensed under GNU GPL v2
For bug reports, read
        http://openocd.org/doc/doxygen/bugs.html
Remote communication error.  Target disconnected.: Connection reset by peer.

while make flash.openocd looks like this.  The "program" command somehow bypasses the shutdown or continuously reconnects.
~/repos/msdk/Examples/MAX78000/Hello_World (dev/micro-ros*) » make flash.openocd        jhcarter@jhcarter-X670E
Loaded project.mk
****************************************************************************
* Analog Devices MSDK
* - User Guide: https://analog-devices-msdk.github.io/msdk/USERGUIDE/
* - Get Support: https://www.analog.com/support/technical-support.html
* - Report Issues: https://github.com/Analog-Devices-MSDK/msdk/issues
* - Contributing: https://analog-devices-msdk.github.io/msdk/CONTRIBUTING/
****************************************************************************
Open On-Chip Debugger 0.11.0+dev-g56a818e4c (2023-10-09-16:10)
Licensed under GNU GPL v2
For bug reports, read
        http://openocd.org/doc/doxygen/bugs.html
DEPRECATED! use 'adapter driver' not 'interface'
Info : CMSIS-DAP: SWD  supported
Info : CMSIS-DAP: Atomic commands supported
Info : CMSIS-DAP: Test domain timer supported
Info : CMSIS-DAP: FW Version = 2.0.0
Info : CMSIS-DAP: Serial# = 04231702dde703f600000000000000000000000097969906
Info : CMSIS-DAP: Interface Initialised (SWD)
Info : SWCLK/TCK = 1 SWDIO/TMS = 1 TDI = 0 TDO = 0 nTRST = 0 nRESET = 1
Info : CMSIS-DAP: Interface ready
Info : clock speed 2000 kHz
Info : SWD DPIDR 0x2ba01477
Info : max32xxx.cpu: Cortex-M4 r0p1 processor detected
Info : max32xxx.cpu: target has 6 breakpoints, 4 watchpoints
Info : max32xxx.cpu: external reset detected
Info : starting gdb server for max32xxx.cpu on 3333
Info : Listening on port 3333 for gdb connections
target halted due to debug-request, current mode: Thread
xPSR: 0x21000000 pc: 0x100008b8 msp: 0x2001ffe0
Info : SWD DPIDR 0x2ba01477
target halted due to debug-request, current mode: Thread
xPSR: 0x81000000 pc: 0x00002124 msp: 0x20003ff0
ERROR: Family code mismatch!  Expected 0x85 but received 0x84
ERROR: Are you connected to the right microcontroller? (MAX78000)
shutdown command invoked
Info : SWD DPIDR 0x2ba01477
target halted due to debug-request, current mode: Thread
xPSR: 0x81000000 pc: 0x00002124 msp: 0x20003ff0
ERROR: Family code mismatch!  Expected 0x85 but received 0x84
ERROR: Are you connected to the right microcontroller? (MAX78000)
shutdown command invoked
** Programming Started **
Info : Padding image section 0 at 0x10008f24 with 12 bytes
** Programming Finished **
** Verify Started **
** Verified OK **
** Resetting Target **
Info : SWD DPIDR 0x2ba01477
ERROR: Family code mismatch!  Expected 0x85 but received 0x84
ERROR: Are you connected to the right microcontroller? (MAX78000)
shutdown command invoked
shutdown command invoked


Blue buttons appear on transaction details loading screen

Operating System (OS and version): Ubuntu 14.04
OpenBazaar version: 1.1.4
Reproducible: Always
Steps to reproduce: Click on transaction details
Observed Behavior: Blue buttons appear on top of the loading screen
Expected Behavior: Those buttons must not appear The buttons should use current color scheme

The buttons should use the theme color.
Also what does Load Index do? We've never used the term "Index" anywhere else in the app.

This is not a bug, those buttons should appear. They are useful if you get stuck on the loading screen, which happens in rare cases, and requires restarting the app if you don't have these buttons.
The loadIndex button looks like it isn't loading a translated phrase. We can change it in the language file to "Go To Discover".
They should have the custom color classes added to them too.
On my TODO list is to add a timer so they only show up if you're on the loading screen for more than 3 seconds.

those buttons should appear. They are useful if you get stuck on the loading screen

:thumbsup:

Similar to #1449 we should not be blocking window controls or the main header. If this overlay didn't block the main header bar the Discover button would still be available and also the reload button. So maybe we can simplify this?

That's a different reload button, it only resends the same address.


Add support for the Julia language

A small PR to add support for running Julia in code blocks. I tested it with some simple code blocks with imports, functions and such and all seems to go smoothly.

Thanks @AstroBarker, this looks great!


Fixed minor misspelling

Fixed misspelling which I found during walking through the codebase.

Thanks @mshr-h !


Throw exception when Focus() fails.

Describe the bug
When calling Focus() from the constructor or OnInitialized it silently fails with weird behavior.
Also when calling Focus when a component or it's parent component has IsVisible == false this silently fails.
To Reproduce
Call Focus() from constructor, OnInitialized or when the component/parent is disabled.
Expected behavior
When Focus() cannot be performed an Exception should be thrown.
Avalonia version
11.1.3
OS
No response
Additional context
No response

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/3ag3jpquii3of
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/3ag3jpquii3of
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

Focus method returns a boolean indicating operation result. It would be very wasteful to throw an exception every time.

If focus() or any other function fail because it is run to early this is 100% a bug and an exception makes sure that the developers are informed of the bug. Correct usage will not cause an exception. Also if focus() can fail for more than one reason how do I know that reason from getting a boolean?
And even if so to me exceptions are way more clean than requiring testing with if's. Getting to collect error handling into a single try...catch is one of the beautiful things with C#. Especially as one need to surround code with try...catch statements anyway means that handling another error doesn't produce more error handling code.


add redirect for framework cleaning (DCOS_OSS-1622)

Description
Broken link to documentation when deleting service from DC/OS
Urgency

[x] Medium

Can one of the admins verify this patch?

Can one of the admins verify this patch?

ok to test

retest this please

👍

retest this please


Syncing requirements.txt to MCE requirements

Description
Updating the bundle-automation requirements.txt file to be the same as MCE.
Related Issue
If applicable, please reference the issue(s) that this pull request addresses.
Changes Made
Updated hack/bundle-automation/requirements.txt.
Screenshots (if applicable)
Add screenshots or GIFs that demonstrate the changes visually, if relevant.
Checklist

[ ] I have tested the changes locally and they are functioning as expected.
[ ] I have updated the documentation (if necessary) to reflect the changes.
[ ] I have added/updated relevant unit tests (if applicable).
[ ] I have ensured that my code follows the project's coding standards.
[ ] I have checked for any potential security issues and addressed them.
[ ] I have added necessary comments to the code, especially in complex or unclear sections.
[ ] I have rebased my branch on top of the latest main/master branch.

Additional Notes
Add any additional notes, context, or information that might be helpful for reviewers.
Reviewers
Tag the appropriate reviewers who should review this pull request. To add reviewers, please add the following line: /cc @reviewer1 @reviewer2
/cc @cameronmwall
Definition of Done

[ ] Code is reviewed.
[ ] Code is tested.
[ ] Documentation is updated.
[ ] All checks and tests pass.
[ ] Approved by at least one reviewer.
[ ] Merged into the main/master branch.

This PR only affect automation tooling for this repository. No changes to the product.

/lgtm


[runtime] Use an unnamed bitfield to avoid -Wglobal-constructor

(instead of an unused field that we'd rather not initialize)
constexpr can't be 100% enforced in a template, so instead it gets silently dropped if the instantiated function doesn't fulfill all the requirements of being constexpr. In this case, that was a constructor not explicitly initializing all fields, even the one we marked unavailable. This meant we were using a non-constexpr constructor to instantiate a global, which semantically requires static initialization.
(The actual initialization is still optimized away at the LLVM level. But the Clang frontend doesn't know that.)
Note that the warning will still fire unless you update your Clang build; I just today cherry-picked the change that handles unnamed bitfields correctly (apple/swift-clang#136).

@swift-ci Please test

Now that's technology. This field shouldn't survive too much longer but this is still worthwhile in the meantime.


GRPC support

Tasks

Introduce new API type v1beta1
Update the .proto parser
Test scenarios for all types of gRPC communications
Analytics support for gRPC APIs

[ ] Update the proto parser
[ ] Test scenarios for all types of gRPC communications

Update - 27th August 2024

Implementation changes from PR https://github.com/wso2/apk/pull/2369/files have been added
Parser updated for single .proto files

To be completed:

Add support for multiple .proto files provided as a .zip file
Add authentication support (OAuth2, mTLS and APIKey)
Add analytics support
Look into providing seperate dashboard for gRPC APIs

Update - 3rd Septermber 2024

Research was done for parser support for .proto files
.proto files do not have a Java native parser to parse the .proto files
Recommended approach is to compile the .proto files into a .desc file using the protoc compiler
The protoc compiler does not have native support in Java and therefore the files need to be provided as descriptor files or the files need to be compiled during runtime as part of a Java background process
As of now, proceeding with assuming the .desc files are provided as input for the config generator call

Update - 5th Sepember 2024

After discussion with the APK team, it was decided that complete parsing and validation of .proto files were not necessary
Since only the service names across all files are necessary for the creation of all relevant GRPCRoutes, the .proto files are parsed to obtain the service names


Fixed https://github.com/console-rs/console/issues/177

Fixed https://github.com/console-rs/console/issues/177.I also added the first 16 colors to the color enum

Please change the title to your pull request to briefly reflect what it does.


Bump versions

Codecov Report

Merging #66 (f2e0bae) into develop (147e8f2) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop      #66   +/-   ##
========================================
  Coverage    85.18%   85.18%
========================================
  Files           18       18
  Lines          405      405
  Branches        53       53
========================================
  Hits           345      345
  Misses          60       60

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 147e8f2...f2e0bae. Read the comment docs.

Codecov Report

Merging #66 (f2e0bae) into develop (147e8f2) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop      #66   +/-   ##
========================================
  Coverage    85.18%   85.18%
========================================
  Files           18       18
  Lines          405      405
  Branches        53       53
========================================
  Hits           345      345
  Misses          60       60

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 147e8f2...f2e0bae. Read the comment docs.


Capability map IG pillar - add missing capabilities

:white_check_mark: Checklist

[x] This pull request has a meaningful title.
[x] If your changes are not yet ready to merge, you have marked this pull request as a draft pull request.

:ballot_box_with_check: Maintainers' checklist

[x] This pull request has had the appropriate labels assigned
[x] This pull request has been added to the SATRE backlog project board
[ ] This pull request has been assigned to one or more maintainers

:arrow_heading_up: Summary
This PR should add (and remove if needed) any missing capabilities from the Information Governance pillar, that @machintim has added to his capability decomposition.
So far it adds:

Internal audit

:closed_umbrella: Related issues

:raising_hand: Acknowledging contributors

[ ] All contributors to this pull request are already named in the table of contributors in the README file.
[ ] The following people should be added to the table of contributors in the README file:

Please update the capability map to include changes from other PRs.

@JimMadge is anything in this PR not already included in information_governance.md?

@JimMadge is anything in this PR not already included in information_governance.md? If @machintim could confirm whether he thinks Internal Audit is a useful capability, maybe he could suggest how we word the component.

No they are duplicate capability components with some minor wording differences.

I have moved internal audit as it didn't feel like a separate capability when I did the decompositions. It is now part of quality management as a process and role.
https://dmail.sharepoint.com/:w:/r/sites/SATRE/Shared Documents/WP3 - Formalisation/SATRE WP3 - Output.docx?d=w21f6b79f144a4dd28e1e0b94d85289ed&csf=1&web=1&e=MtEHch


Support build to local Docker daemon directly.

Please upvote if you would like this feature to be added.

Implemented in #266.


Report missing opt-out

Up for discussion:
Do you think we need a reporting function for people who didn't opt out of certain categories? Like a Jam game that used external music but forgot to opt out of the Audio category? Or a Compo game without sound effects at all?
I could imagine something like this:

Each category gets a Report button
If that button has been pressed by 3 or more users, the category is brightly marked "opt-in challenged" (maybe only visible to the creator) with a description of what it means
The creator can then either choose to opt out of that category or write a comment why they think the category should be kept in.
Any other user pressing the report button will be shown the creator's comment and can decide whether they still want to report it and maybe add a response to the comment.
If the creator doesn't react to an opt-in challenge, the category will be opted out by the end of the voting.
Challenging an opt-in should only be possible during the first 1-2 weeks to give the creator a fair amount of time to react.
If no consensus is reached after some back-and-forth between creator and voters, either the category is kept as opt-in or a moderator needs to step in.

This is just a collection of thoughts. Do you think this is too much policing? Should it be done at all? Maybe in a less severe form?

Related to #972 high five!

Right now notifications, patching (i.e. fellow user submitted edits), and groups (special subscriptions) aren't implemented, but ultimately these will help users fix this. I want this to be user to user as much as possible.


Can't stake using a keplr wallet connected to a Google account

Related to:

https://github.com/NibiruChain/web-app/issues/1115
https://github.com/cosmos/relayer/issues/1493

@CalicoNino , is this one fixed from the changes you added to @nibiruchain/nibijs , or there still work to be done?

I believe this should be fixed. I mean I have staked with a newly created Google account.


set locale breaking change

Laravel Version: 5.7.3
PHP Version: 7.2
Database Driver & Version:

Description:
You cant set locale using config(['app.locale' => 'fr']); you have to use app()->setLocale('fr') now.
With config(), app('translator') isnt aware of the change and use the base value.
Steps To Reproduce:
on fresh 5.6 & 5.7
Route::get('/', function () {
    config(['app.locale' => 'fr']);
    //app()->setLocale('fr');
    return view('welcome');
});

Do you know which commit/PR caused this breaking change?

Why not just use app()->setLocale('fr') ? I had this problem migrating to 5.7 but I found the solution you already know much better now

Does it occur on a fresh 5.5 install?
Either way, if it is occuring on a fresh 5.6 and 5.7 install, then you can open a PR at http://github.com/laravel/docs to update the docs.


build: fix rpc mumbai

Description

Type of change

[ ] Docs change / dependency upgrade
[ ] Configuration / tooling changes
[ ] Refactoring
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Requires changes in customer code

High-level change(s) description - from the user's perspective

Related Issue(s)
Fixes
Related pull request(s)

Fixed on testnet-prod branch.


Closes #2398 Re-enable notifications for pending offline products saved

Description

Use WorkManager instead of service to send notifications periodically.
Use PeriodicWorkRequest with a retryInterval of 60 minutes and flexInterval of 10 mins, this means that the user will receive at max one notification in 50 mins if there are any products saved offline and the user is connected to a network.
Reformat imports in MainActivity.java
Remove files WifiUploadReceiver.java and UploadService.java since it is not being used anymore.
Remove permission android.permission.FOREGROUND_SERVICE which is not being used anymore.

Related issues and discussion
Fixes #2398
Screen-shots, if any

Checklist
Make sure you've done all the following (You can delete the checklist before submitting)

[x] Apply the AndroidStyle.xml style template to your code in Android Studio.
[x] Code is well documented
[ ] Include unit tests for new functionality
[x] All user-visible strings are made translatable
[x] Code passes Travis builds in your branch
[x] If you have multiple commits please combine them into one commit by squashing them.
[x] Read and understood the contribution guidelines.

In the previous version, notification will be displayed as soon Wifi is found. With this new version, the check will be done every XXmin. @teolemon is it ok for you ?

@deniger We can no more use the previously implemented way of running a service in the background and listening for changes in the network. No service can run in the background in Android 8 and above (Battery Optimization by Google). I found this to be the aptest way of implementing this. The minimum interval time can be 15 mins which would ideally be like real-time, but I avoided being so aggressive in posting notifications to the user so kept it 60 mins.

thanks @huzaifaiftikhar for this info.
I'll take a look this morning


Publish to Open-VSX

Feature description
What do you want to see added to the extension? I want to see it published to the open-vsx marketplace
How would it help you? Allow users of VSCodium and code-server to use the extension
How often would you use this feature? N/A

Thanks for the suggestion, this has now been integrated to the CI.
Extension is available here.
The namespace should be verified soon.

Many thanks!


Fix compose comment for proposals

Fixed change in height (expand/collapse) of editor when long comment is added
Added loader when txn is initiated
Added toast confirmation after comment is posted
Preview link

Summary by CodeRabbit

New Features

Enhanced comment functionality on proposals with real-time updates and auto-focus capabilities.
Improved editor components with keyup actions, visibility settings, and dynamic resizing.

Enhancements

Updated sorting function to actively subscribe to new comments.
Refined comment submission process with visual feedback through toasts and loading indicators.

Bug Fixes

Adjusted event handling in editors to ensure smooth interaction and performance.

Ready @Megha-Dev-19

@ori-near @frol This Pull Request by @Megha-Dev-19 should address many of the commenting issues on proposals that we discussed in the call yesterday. I did a small adjustment to clear the editor when the submission transaction is finished, and also added "Don't ask again" configuration to the tests ( which is slightly different when saving to socialdb compared to calling another smart contract like devhub.near ).
Here is a video of what the automatic test now does. As you can see it creates the comment and submit. There is no "blue toast" from the VM like when calling another contract, but there's the disabled submit button with a spinner. In the end there's a also a confirmation toast. What this test does not do is to show the actual submitted comment in the end, and if we want this too, it is possible by mocking the results of the subscribed social.index.
https://github.com/NEAR-DevHub/neardevhub-bos/assets/9760441/e9777a28-548f-4603-81d6-3710a682efab


Update MultiBoxTracker.java

If you remove the camera from detected object, then the tracker will stop tracking. And the rectangle will disappeare.

@googlebot I signed it!


Helm chart version v0.16.0-rc.3 not available

The latest chart v0.16.0-rc.3 is not available.
Although I can change image.tag to v0.16-rc.3 for app version, allowCrossNamespaceRefs is only added to v0.16-rc.3 and is not available in chart version v0.15.1.
Then I'm not able to use the feature of allowCrossNamespaceRefs.
Can you please release a new chart version v0.16-rc.3?

0.15.1 is the latest stable chart.  If you want to use non-stable charts you will need to use the --devel flag
helm search repo tf-controller
NAME                            CHART VERSION   APP VERSION     DESCRIPTION
weaveworks/tf-controller        0.15.1          v0.15.1         The Helm chart for Weave GitOps Terraform Contr...

helm search repo tf-controller --devel
NAME                            CHART VERSION   APP VERSION     DESCRIPTION
weaveworks/tf-controller        0.16.0-rc.3     v0.16.0-rc.3    The Helm chart for Weave GitOps Terraform Contr...

@dwalker-sabiogroup Thank you! Yes, if I install with --version v0.16.0-rc.3, it works!


does not detect?

I used to use this app very nicely and it ran very smoothly
but now it kind of just STOPPED detecting drivers
it straight up gave me the error
"initializing emulated Xbox 360 controller...
connection error: bus not found
Press [ENTER] to exit"
I think there is a problem with vgiem but I never found any sollution

oh wait i just found a sollution
i needed to repair the driver there was a missing file 😝

oh wait i just found a sollution
i needed to repair the driver there was a missing file 😝


Adding new input/field in create user form

This is  a request

Prerequisites

[ +] Are you running the latest version?
[ +] Are you reporting to the correct repository?
(enso is made of many specialized packages: https://github.com/laravel-enso)
[ +] Did you check the documentation?
[+ ] Did you perform a cursory search?

Description
I want to add new field for password in creating user- form, but when I add it to user.json file than create user doesn't work at all.

Steps to Reproduce

Expected behavior

Actual behavior

Update in config/enso/forms.php the validations key to match your APP_ENV value.
Now the validator will let you know that "type": "password" does not exist in this package. Then go read the docs about "type" attribute and what it is used for.

Can I add this "type: password" on this package and if yes can you help me how?
Thanks a lot.

You really don't like reading docs right?

for password you need to use "type": "input"

Yes, I've read the doc in fact but I thought it was like "input type="something..", that's why I've asked you. Now I'm going to try/prove this.
Thanks again.


[SMALLFIX] Better exception handling

Exception does not allow self-suppression, so when netty packet writer handles the same exception,   it throws runtime exception:
java.lang.IllegalArgumentException: Self-suppression not permitted
	at java.lang.Throwable.addSuppressed(Throwable.java:1043)
	at alluxio.client.block.stream.NettyPacketWriter.updateException(NettyPacketWriter.java:363)
	at alluxio.client.block.stream.NettyPacketWriter.access$600(NettyPacketWriter.java:69)
	at alluxio.client.block.stream.NettyPacketWriter$WriteListener.operationComplete(NettyPacketWriter.java:459)
	at alluxio.client.block.stream.NettyPacketWriter$WriteListener.operationComplete(NettyPacketWriter.java:436)

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/17568/
Test PASSed.


use type hints

https://blog.jetbrains.com/pycharm/2015/11/python-3-5-type-hinting-in-pycharm-5/
probably a better way to enforce the product_spec format than all the checks I do

maybe not cause python still does not actually enforce types


fix: managed namespace not getting properly removed from argocd instance cluster secret

What type of PR is this?

Uncomment only one  /kind line, and delete the rest.
For example, > /kind bug would simply become: /kind bug

/kind bug

/kind cleanup
/kind failing-test
/kind enhancement
/kind documentation
/kind code-refactoring

What does this PR do / why we need it:
fixes a bug where a managed namespace was not being removed from the managing argocd instance's cluster secret after that namespace was updated to be managed by a different argocd instance
Have you updated the necessary documentation?

[ ] Documentation update is required by this PR.
[ ] Documentation has been updated.

Which issue(s) this PR fixes:
Fixes https://issues.redhat.com/browse/GITOPS-1644
How to test changes / Special notes to the reviewer:

Install Gitops-operator
Create namespaces: test-target, test-argo1, test-argo2
Label test-target with argocd.argoproj.io/managed-by=test-argo1
Create ns-scoped argocd instances in test-argo1 and test-argo2
Create sample application in test-argo1:

apiVersion: argoproj.io/v1alpha1
kind: Application
metadata:
  name: guestbook
  namespace: test-argo1
spec:
  project: default
  source:
    repoURL: https://github.com/redhat-developer/gitops-operator
    path: ./test/examples/nginx
    targetRevision: HEAD
  destination:
    server: https://kubernetes.default.svc
    namespace: test-target
  syncPolicy:
    automated: {}
    retry:
      limit: 1

Check that guestbook app in test-argo1 is "healthy" and "synced"
Create guestbook app in test-argo2

apiVersion: argoproj.io/v1alpha1
kind: Application
metadata:
  name: guestbook
  namespace: test-argo2
spec:
  project: default
  source:
    repoURL: https://github.com/redhat-developer/gitops-operator
    path: ./test/examples/nginx
    targetRevision: HEAD
  destination:
    server: https://kubernetes.default.svc
    namespace: test-target
  syncPolicy:
    automated: {}
    retry:
      limit: 1

Update label on test-target ns to: arogcd.argoproj.io/managed-by=test-argo2
Refresh guestbook app in test-argo1
Check that guestbook app in test-argo1 is now "missing" and "unknown" and that guestbook app in test-argo2 is now "healthy" and "synced"

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: jaideepr97:x: Jaideep RaoJaideep Rao seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

CLA pls @jaideepr97

@iam-veeramalla
When I generated a new personal access token I think it didn't connect to my account somehow, so my commits started getting split between 2 commiters (both me), but It seems I can only sign the CLA with my actual account, not sure how I would be able to sign the CLA with the personal access token created commiter
I've deleted my personal access token since then hoping future PRs don't have this problem, but for now I don't know how to satisfy this check

@iam-veeramalla When I generated a new personal access token I think it didn't connect to my account somehow, so my commits started getting split between 2 commiters (both me), but It seems I can only sign the CLA with my actual account, not sure how I would be able to sign the CLA with the personal access token created commiter I've deleted my personal access token since then hoping future PRs don't have this problem, but for now I don't know how to satisfy this check

Not sure either, We will figure this out together :)


Add mechanism to expose IFWI version number

IFWI versions have specific layout. Request here is to add capability to expose that

@aravindksg : what do you mean about layout?

typedef struct global_ifwi_version{
UINT16 Product_version;
UINT16 Release_version;
UINT16 Build_version;
UINT16 Hotfix_version;
UINT16 Engineering_version;
}ifwi_version_t;

Closing issue since we can't add vendor specific struct


config saves calibrations by resolution

This will sadly introduce a nesting that I would rather avoid, but it would be helpful to be able to maintain the previously calibrated intrinsics for each camera so that extrinsics could be calculated at maximal resolution, and then the resolution could be lowered for purposes of recording / triangulating. I suspect that this will sidestep (a bit) some issues with the reading of frames. 640x480 isn't great, but I suspect that it should provide sufficient data for doing single person tracking with mediapipe or DLC, though wouldn't really be appropriate for aruco marker tracking (with markers of reasonable size).

Follow up reflection to note that I'm able to read 1280x720 @30fps from 4 cameras without issue. I'm guessing I could pull from 5 easily enough, even if having to downgrade to 24fps. That should capture a decent volume with redundancy and is a good entry-level self-limiting point. I expect that future scale up would benefit from substantial increases in setup complexity (e.g. by using an orange pi to function as a camera server).

This is such a notably low priority that I'm just going to close it for now.


Check for recently placed identical orders during balancing

Creates a Map that stores recently placed orders along with a timestamp. During balancing we check if an identical order has been placed recently before placing the order. After placing an order we record the order in the Map.

Fixed by setting the reduceOnly flag to true in https://github.com/blockworks-foundation/liquidator-v3/commit/8be33e400c7e702b4b9f85f7fa6acb6e067a2bcc so closing this.


Update read_wi() to support Camtrap DP v1.0-rc.1

Suggested in camtraptor July 2023 coding sprint
Similar to inbo/camtrapdp#9, read_wi() should also read data into Camtrap DP v1.0-rc.1.

The status of read_wi() is undecided, see https://github.com/inbo/camtrapdp/issues/62. Closing issue here.


BMS name cannot be permanently set

Describe the bug
Dear Louis,
This is a fantastic piece of software! Thank you so much for creating this. I installed it on my Venus system this week and it worked right away with no modifications. I am running an Overkill BMS with my battery, and running Venus on a Raspberry Pi 3b.
I want to give my BMS a custom name because the default "SerialBattery(LTT/JBD)" is long and shorter names are easier to display on the GUI. I can rename it to "BMS" using "Device" and "Name" in the settings. However, the name change does not stick, and the name defaults back to "SerialBattery(LTT/JBD)" when Venus OS is rebooted.
Can we get the BMS name change (and possibly other SerialBattery settings) to survive a reboot?  Has anyone else noticed other customized properties that revert to default when rebooting Venus?
Thanks,
Lauren
How to reproduce
Steps to reproduce the behavior:

In the Device menu, change the Name of the new device "SerialBattery(BMStype)" to a custom name, like "BMS"
Go back to the Pages screen and verify that the name change is being displayed properly.
Reboot Venus OS.
After the system boots up, the custom name change is gone and the device name is back to the default "SerialBattery(BMStype)".

Expected behavior
Name changes made on the Device menu in Venus should survive a reboot of the system and not have to be set again.
Driver version
1.0.20230531
Venus OS device type
Raspberry Pi
Venus OS version
3.01
BMS type
Smart BMS (LLT, JBD, Overkill Solar)
Cell count
4
Connection type
Serial USB adapter to TTL
Config file
[DEFAULT]

; If you want to add custom values/settings, then check the values/settings you want to change in "config.defau>
; and insert them below to persist future driver updates.

; Example (remove the semicolon ";" to uncomment and activate the value/setting):
MAX_BATTERY_CHARGE_CURRENT = 60.0
MAX_BATTERY_DISCHARGE_CURRENT = 100.0

Relevant log output
serial connection works fine

Any other information that may be helpful
No response

Duplicate of https://github.com/Louisvdw/dbus-serialbattery/issues/100.
Try the latest nightly from the dev branch and it will work.

i am using nightly dev v1.0.20231010dev and sadly it does not work.
after renaming all and then rebooting the original name is restored.

Please post the content of your config.ini


Release 0.9.0

As the current state of affairs is pretty good, I think we can create a 0.9.0 release now and keep up the development on new features

I have prepared and tested the release in this branch:
https://github.com/NextGenTel/akka-tools/tree/release-0.9.0
All @ovevang needs to do is to check out the branch and run
sbt publish

No need to push anything to github

Nice!
On Wed, Sep 23, 2015 at 9:47 AM, Morten Kjetland<EMAIL_ADDRESS>wrote:

I have prepared and tested the release in this branch:
https://github.com/NextGenTel/akka-tools/tree/release-0.9.0
All @ovevang https://github.com/ovevang needs to do is to check out the
branch and run
sbt publish
No need to push anything to github
—
Reply to this email directly or view it on GitHub
https://github.com/NextGenTel/akka-tools/issues/5#issuecomment-142521522
.

A little correction: After the release is available in maven central we must update "current version" in README in master-branch

Done, including current version update in docs. Sonatype release completed, maven central update pending.

Nice
On Wed, Sep 23, 2015, 16:34 Øyvind Vevang<EMAIL_ADDRESS>wrote:

Closed #5 https://github.com/NextGenTel/akka-tools/issues/5.
—
Reply to this email directly or view it on GitHub
https://github.com/NextGenTel/akka-tools/issues/5#event-417047096.


https://raw.githubusercontent.com/The-Blockchain-Association/sec-notice-list/master/ba-sec-list.json

No description provided.

Transfer transfer = new Transfer();
//Supported networks
List<Blockchain> blockchains = new ArrayList<>();
//Evm series, the first parameter is Ethereum, the second parameter is the ChainId, like 1 is the ChainId for Ethereum.
blockchains.add(new Blockchain("ethereum", "56"));
transfer.setBlockchains(blockchains);

transfer.setProtocol("TokenPocket");
transfer.setVersion("1.0");
transfer.setDappName("Test demo");
transfer.setDappIcon("https://eosknights.io/img/icon.png");
//business id defined by developer
transfer.setActionId("web-db4c5466-1a03-438c-90c9-2172e8becea5");
//data，If it is a native token, you can add on-chain data
transfer.setMemo("0xe595a6");
transfer.setAction("transfer");
//sender
transfer.setFrom("0x52bc44d5378309ee2abf1539bf71de1b7d7be3b5");
//receiver
transfer.setTo("0x32ff06198da462f1c519d30f4d328b3fef295d19");
//contract address, if you are transferring ether, this parameter is optional
transfer.setContract("0xdAC17F958D2ee523a2206206994597C13D831ec7");
//token amount. For example, we transfer 0.01 USDT here, which is equivalent to passing 0.01 USDT.
transfer.setAmount(0.01);
//Required
transfer.setDecimal(18);
transfer.setSymbol("USDT");
transfer.setDesc("Only for ui display, not on the chain");
//if developer set callback url, after the wallet operation is completed, the result will be called back to the callbak URL through the post application json method
transfer.setCallbackUrl("http://<IP_ADDRESS>:9011/taaBizApi/taaInitData");
TPManager.getInstance().transfer(this, transfer, new TPListener() {
    @Override
    public void onSuccess(String s) {
        //The result of the transfer. Note that developer needs to confirm the final result on chain with the hash. We just return the hash after sending transaction, it can not guarantee a successful transaction.
        Toast.makeText(EthTransferActivity.this, s, Toast.LENGTH_LONG).show();
    }

    @Override
    public void onError(String s) {
        Toast.makeText(EthTransferActivity.this, s, Toast.LENGTH_LONG).show();

    }

    @Override
    public void onCancel(String s) {
        Toast.makeText(EthTransferActivity.this, s, Toast.LENGTH_LONG).show();

    }
});

👌

#404

##420

[ ] ****

@Imebeez

404Sent from my iPhoneOn Jul 16, 2023, at 10:47 AM, Nina naydenova @.***> wrote:
#404
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>