Fix pylint errors

It's better, and in consistence with the other repos, to use a file named .pylintrc for defining the configuration for pylint (including the error exceptions). This branch is using a new rcfile instead.

It's better, and in consistence with the other repos, to use a file named .pylintrc for defining the configuration for pylint (including the error exceptions). This branch is using a new rcfile instead.

Commit #0dc5c50 addresses this.

There are many changes in this pull request. The coverage ignoring is discarded (and I agree) but it could break the pipeline. Be careful with this: only if the pipeline is working you can go with this in my view.
The coverage result is not discarded in this PR. The pipeline also executes properly. On one of the comment you have added, we are separately working on it.

There was some good work and progress in with this pull request and as such I am closing it under the remark that the number of exceptions for the linter should be reduced in the future.
This must be kept to a minimum and further work should be done on how to improve it:
disable=C,R,attribute-defined-outside-init,invalid-name,no-init,no-member,no-self-use,
	no-name-in-module,unused-argument,fixme,broad-except,reimported,c-extension-no-member,
	protected-access,bare-except,anomalous-backslash-in-string,redefined-outer-name,
	redefined-builtin


Running 'test' from a root directory results in bogus error.

Version 5.2.1
On Windows, from the root directory of any drive, the following command results in an error:
D:\>haxelib run lime test D:\Sources\IdeaProjects\myFlashApp\project.xml flash -verbose

_/\\\\\\______________________________________________
_\////\\\______________________________________________
_____\/\\\_____/\\\_____________________________________
______\/\\\____\///_____/\\\\\__/\\\\\_______/\\\\\\\\___
_______\/\\\_____/\\\__/\\\///\\\\\///\\\___/\\\/////\\\__
________\/\\\____\/\\\_\/\\\_\//\\\__\/\\\__/\\\\\\\\\\\___
_________\/\\\____\/\\\_\/\\\__\/\\\__\/\\\_\//\\///////____
________/\\\\\\\\\_\/\\\_\/\\\__\/\\\__\/\\\__\//\\\\\\\\\\__
________\/////////__\///__\///___\///___\///____\//////////___

Lime Command-Line Tools (5.2.1)

Error: Incorrect number of arguments for command 'test'

In any other directory, the command executes as expected:
D:\>cd Sources
D:\Sources>haxelib run lime test D:/Sources/IdeaProjects/myFlashApp/project.xml flash -verbose

_/\\\\\\______________________________________________
_\////\\\______________________________________________
_____\/\\\_____/\\\_____________________________________
______\/\\\____\///_____/\\\\\__/\\\\\_______/\\\\\\\\___
_______\/\\\_____/\\\__/\\\///\\\\\///\\\___/\\\/////\\\__
________\/\\\____\/\\\_\/\\\_\//\\\__\/\\\__/\\\\\\\\\\\___
_________\/\\\____\/\\\_\/\\\__\/\\\__\/\\\_\//\\///////____
________/\\\\\\\\\_\/\\\_\/\\\__\/\\\__\/\\\__\//\\\\\\\\\\__
________\/////////__\///__\///___\///___\///____\//////////___

Lime Command-Line Tools (5.2.1)

Initializing project...
Using project file: D:/Sources/IdeaProjects/myFlashApp/project.xml
Reading Lime config: c:\.lime\config.xml
Using target platform: FLASH

Running command: UPDATE
 - Embedding asset: (MANIFEST) manifest/default.json
[...]

If it makes any difference, the Haxe versions were:
D:\Sources>haxe -version
3.4.2 (git build master @ 890f8c7)

D:\Sources>haxelib -version
Unknown command -version
Haxe Library Manager 3.3.0 - (c)2006-2016 Haxe Foundation

This is a bug (or undocumented behavior?) in sys.FileSystem for Haxe
https://github.com/HaxeFoundation/haxe/issues/6489
I'll try and find a workaround

At least it can spit out a useful error message.  'Incorrect number of arguments' is clearly wrong.
So, is sys.FileSystem being called by lime or haxelib at this point?
(By the way... IMO, you shouldn't close issues until you've fixed them -- even if they're in underlying technology...  On the other hand, maybe that's why the intellij-haxe repo has so many open bugs.)

Oh, I closed this with a workaround :smile:
7bcfb5d


Delete index.md

Howdy @Tivieosidi, I'm pretty sure we don't want to be deleting the index page for our warp content, so going ahead and closing this PR.
Feel free to re-open and provide more context.


allow port-forwards to be restarted with the same local port

The current design restricts the user from restarting a port-forward with the same port. The restriction is not technically necessary, and in the event that the associated pod is terminated/restarted lifting this restriction makes it easy for the user to reconnect the port-forward. (the kubectl port-forward process does not stop when the associated pod is terminated).
Signed-off-by: Jim Ehrismann<EMAIL_ADDRESS>addresses #2340

Does this completely resolve the mentioned issue?

Does this completely resolve the mentioned issue?

no, just wanted to make sure this PR is linked to that issue


Is there any way to reduce the VRAM usage?

The usage is up 36G, it is impossible for personal computer.

The usage is up 36G, it is impossible for personal computer.

reduce example data sequences will work...
For example, 250 frame use about 20GB VRAM


200 response automatically added to POST verbs

[branch: dev2.0]
The following code in the Java REST resources:
	@POST
	@ApiOperation(value = "Adds a project to the knowledge base.", notes = "This method adds the information about a new project to the list of projects being managed and assigns to it a unique identifier.", response = Project.class)
	@ApiResponses(value = {
			@ApiResponse(code = 201, message = "The project has been successfully added to the knowledge base and associated to a unique identifier. The resulting project instance is guaranteed to have a non-null identifier.", response = Project.class),
			@ApiResponse(code = 400, message = "The content of the request is not a valid project instance.", response = Error.class),
			@ApiResponse(code = 500, message = "An internal error has occurred while processing the request. The nature of the error can vary. The endpoint will provide additional information in the body of the response.", response = Error.class),
			@ApiResponse(code = 503, message = "The project repository is not accessible.", response = Error.class) })
	public Response addProject(
			@ApiParam(value = "Project to add to the knowledge base.", required = true)
			Project project) {

		// we forward the information to the middleware, which
		// implements our business logic and let it handle the
		// request.
		//
		ApiMiddleware middleware = new ApiMiddleware();

		return middleware.addProject(this.context, project);

	}

produces the following specification:

      "post" : {
        "tags" : [ "projects" ],
        "summary" : "Adds a project to the knowledge base.",
        "description" : "This method adds the information about a new project to the list of projects being managed and assigns to it a unique identifier.",
        "operationId" : "addProject",
        "produces" : [ "application/json" ],
        "parameters" : [ {
          "in" : "body",
          "name" : "body",
          "description" : "Project to add to the knowledge base.",
          "required" : false,
          "schema" : {
            "$ref" : "#/definitions/Project"
          }
        } ],
        "responses" : {
          "200" : {
            "description" : "successful operation",
            "schema" : {
              "$ref" : "#/definitions/Project"
            }
          },
          "201" : {
            "description" : "The project has been successfully added to the knowledge base and associated to a unique identifier. The resulting project instance is guaranteed to have a non-null identifier.",
            "schema" : {
              "$ref" : "#/definitions/Project"
            }
          },
          "400" : {
            "description" : "The content of the request is not a valid project instance."
          },
          "500" : {
            "description" : "An internal error has occurred while processing the request. The nature of the error can vary. The endpoint will provide additional information in the body of the response."
          },
          "503" : {
            "description" : "The project repository is not accessible."
          }
        }
      }
    },

the entry:
          "200" : {
            "description" : "successful operation",
            "schema" : {
              "$ref" : "#/definitions/Project"
            }
          },

should not be included.

Try to remove the response = Project.class from your @ApiOperaiton

Hi thank you! That worked like a charm.

Hi @hyp0th3rmi4 , this fix didnt work for me. I am using swagger-maven plugin version 3.0.0 and swagger-core version 1.5.1-M2. Were you using the same ?

I am facing the same issue as ndhar-1.
@ApiOperation(value = "Post BundleSpec using BundleSpec")   - Even this is adding the extra "200" status code.
@ApiOperation(value = "Post BundleSpec using BundleSpec", code = 201) - I tried this even. Tried to override the value of default error code(which is 200). This dint work either. It still prints 200 also.
What am i missing.?? Thanks in advance.!!


Upgrading to 8.4.5 and signing in with email on web doesn't get past Welcome to Insomnia page in insomnia mac app

Expected Behavior
I should be able to see my collections in the mac app. Today I upgraded to 8.4.5 and my collections are gone. Not in scratch. I hesitantly accepted the E2E BS you made us do with version 8.0.0 and was able to live with it for a while but now once again my collections are not visible.
I clicked Continue with Email

I get this page and click Go Back because I have no idea what email it's even trying to use here since it's not displayed

I get this page and click Continue with email

I enter my email and click Continue with Email

I enter the code on this page

It takes me to this page and I click the Open button

And my insomnia app opens back to this page

What is going on here and how can I login to retrieve my collections?
Actual Behavior
I should be able to login and see my collections.
Reproduction Steps
No response
Is there an existing issue for this?

[X] I have searched the issue tracker for this problem.

Additional Information
No response
Insomnia Version
8.4.5
What operating system are you using?
macOS
Operating System Version
Mac Ventura 13.6
Installation method
I don't remember how I installed it as i've had it for years now. It was either via homebrew or insomnia website.
Last Known Working Insomnia version
8.4.4

@toymachiner62 what happens if you click "Continue with Email" again on the desktop app?

@toymachiner62 what happens if you click "Continue with Email" again on the desktop app?

Takes me back to the web browser. It's just a loop.

what browser are you using and is it incognito mode?

what browser are you using and is it incognito mode?

Chrome and not in incognito mode.

This is strange. We can hop on a call and help you trouble-shoot. Are you part of our community slack? I am not sure what 8.0 to 8.4.5 will have caused you to lose your collections after having to log in. 8.0 is where Kong introduced it, so you must have experienced it in 8.0. Regardless, we can troubleshoot together.

The end to end encryption feature is a feature that has existed even before Kong acquired this product. Previously, e2ee passphrase was used to authenticate accounts, which resulted in people locked out of their accounts when they forgot passphrase. They could not even access their billings. The only thing we did at 8.0+ is to separate end to end encryption from authentication, so people could still access their billings.

@toymachiner62 you skipped the Enter Passphrase step because you clicked "Go Back". Please enter your passphrase, so the desktop app can decrypt the cloud synced files. This is not an issue on our end.

I'm having the same issue. The new Insomnia is absolutely terrible


Add MenusTemplate component

In the components/templates folder add a MenusTemplate.tsx file. Copy the structure of one of the other templates (they're all currently the same apart from name - contents will be added later). Update the component name so it matches the file name.
In the pages/[content].tsx file import the template and add it alphabetically in the correct place between the layout tags {page === "menus" && <Menus />}
Add the template data to the config file in data/pages.tsx: { name: "Menus", href: "/menus", content: "menus" }

Hello Emma,
Would love to work on this issue
Thank you

Of course 👍🏻

Thank you


Require Ruby 2.7 & rubocop: Use voxpupuli-rubocop

I think just to be safe, we should mark this backwards-incompatible. I want to work on the todo list after #335 is merged


OFA-Base model checkpoints

Could you please release the pre-trained checkpoint of OFA-Base?
Thanks

We plan to release OFA-Base next week.

Great, thanks!

@noamwies Hi, the pre-trained checkpoint of OFA-Base has been released in this repo. To use OFA-Base, you just need to load ofa_base.pt (see checkpoint.md) and change --arch=ofa_large to --arch=ofa_base in the training scripts.


I want a new property of onDatasetClick

The onElementsClick returns the element under the event point, then returns all elements at the same data index.
But I want to know which dataset is clicked. In other words I want to get a dataset that the clicked element in.
So, I want an onDatasetClick property that returns the element under the event point, then returns all elements from that dataset by the .getDatasetAtEvent(e) of the Chart.js instance.
Reference
Thank you.

Hey @ledsun ,
Apologies for late reply and thanks for reporting.
Will look into it shortly.
Thanks,
Goran

The .getDatasetAtEvent(e) is no longer required for me.
Thank you.

Hi @gor181, is there any way to implement getDatasetAtEvent(event). I need this desperately.


[PY] Samples

### Tasks
- [ ] https://github.com/microsoft/teams-ai/issues/308
- [ ] https://github.com/microsoft/teams-ai/issues/310
- [ ] https://github.com/microsoft/teams-ai/issues/311
- [ ] https://github.com/microsoft/teams-ai/issues/313
- [ ] https://github.com/microsoft/teams-ai/issues/317
- [ ] https://github.com/microsoft/teams-ai/issues/1175
- [ ] TBD

Closing as complete!


Ensure chunking heuristic uses shape instead of maxshape

[x] Closes Issue #149
[x] Tests added

@kmuehlbauer the only thing left to decide is whether / how we want to expose this in the legacy API, or if we even want to pull in the netCDF4 heuristic for that. My suggestion is to keep the behavior consistent by plumbing through the chunking_heuristic keyword even in the legacy case, WDYT?


Upgrade torchvision

I thought we should put this question back on the table: Should we upgrade our torchvision?
@andreanne-lemay & @olix86 --> What are your thoughts?
Small new argument I can across today in favour of an upgrade --> This new function instance_norm is not available in our current torchvision version but is exactly what we need for: NormalizeInstance2D & 3D.

I agree it would be nice to upgrade, but last time I checked we were being held back by our Nvidia driver version #69

I agree it would be nice to upgrade, but last time I checked we were being held back by our Nvidia driver version on rosenberg #69

could we consider updating our driver?

Yes that would be great for us, when we had the discussion in october we were blocked by the fact that ADS needed the old driver / CUDA for their version of TF, not sure if that's still true.


dashboard: your farm page

please add tooltips over the buttons of v2 address and view bootstrap explaining why do I need to click on these

V2 Address -> change to Stellar Address, highlight Where minting payouts get sent to
View Bootstrap -> Download a ZOS image for your farm nodes

Verified


Fixing version: nativescript-angular, nativescript-theme-core

The last commit upgrading the version nativescript-angular broke this template since nativescript-angular version 1.3 is not available yet on npm.
I also upgraded the theme-core module of NativeScript.
Cheers from France

Thanks a lot, @rhanbIT!
<EMAIL_ADDRESS>has been already published to npmjs.com.
We use to update this template in order to prepare and test the release of the nativescript-angular plugin. Therefore, we recommend users to create new projects based on the latest template package published on npmjs.com by tns create my-app --ng.
The version of nativescript-theme-core has been updated as well, so I'm closing this PR in favor of this change: https://github.com/NativeScript/template-hello-world-ng/commit/bc50d1f5799df90acfde3b86414659fa376629d5#diff-b9cfc7f2cdf78a7f4b91a753d10865a2.

Hi @vchimev ,
The module<EMAIL_ADDRESS>has been updated  6 days ago on the package.json via this commit.
But the module has been publish only 4 days ago, that's where the problem was coming from.
Now it works so it's perfect :)


Only include code in test environment

This project uses semver, please check the scope of this pr:

[ ] #none# - documentation fixes and/or test additions
[x] #patch# - backwards-compatible bug fix
[ ] #minor# - adding functionality in a backwards-compatible manner
[ ] #major# - incompatible API change

CHANGELOG

Fixed build to only include this addons code in the test environment as to not bloat non-test environments with extra, unused code.

Any concerns about people running tests in dev environments (e.g. navigating to /tests) - I know they shouldn't, but maybe we want to add some README content + announce internally

Approved, just be aware of ^

👍

Approved


export shouldFail.shouldFailWithMessage

Fixes #1502

This is a harmless, one-liner change. Basically, it lets shouldFail module export the awesome shouldFailWithMessage function so that it could be used by 3rd party developers.

Hi @remon-nashid! I had no idea getting the revert string was as easy as that. I'm currently in the process of adding some tests to the test helpers themselves (#1078), so that we can more rapidly iterate over the API before publishing a standalone package (#1204). Could you add some simple tests to this module too? They don't have to be too extensive, just a couple examples showing how it's supposed to be used.
Thanks a lot!

Thanks for the review @nventuro, I'm adding tests ASAP. I'll be using expectEvent.test.js for inspiration :).

@nventuro I see you bet me to it in #1513 ;). Thanks! Closing this obsolete PR.

@remon-nashid I'm sorry you closed the PR, #1513 doesn't actually address this! It was my bad for not reviewing your PR fast enough :(
It did give me an idea for an API though: what if instead of having shouldFailWithMessage we had a withMessage function that checked for the string, and added it to the other shouldFail functions? So we'd end up with:
shouldFail.withMessage
shouldFail.reverting.withMessage
shouldFail.throwing.withMessage
Unless a message can only be specified for reverts, in which case only the first two make sense :stuck_out_tongue:


feat: add lsp formatting

Description
Problem
Resolves #3400
Documentation
Check one:

[x] No documentation needed.
[ ] Documentation included in this PR.
[ ] [Exceptional Case] Documentation to be submitted in a separate PR.

PR Checklist

[x] I have tested the changes locally.
[x] I have formatted the changes with Prettier and/or cargo fmt on default settings.

Apologise for the late review.
Just want to confirm that we are able to disable the code formatter too in the lsp?

By default, the code will not format on save, you need to enable this in the settings, but if necessary, you can disable it specifically for Noir
settings.json
"[noir]": {
    "editor.formatOnSave": true,
}


Add new_chat_photo command

Noticed this hasn't been added yet, I know this can be handled by genericmessage but having comands for each message type might be a good idea.

Yes I agree, good catch!


new feature: override db name

currently when you have @tables it creates tables with a simple convention of appname-env-tablename
so the following .arc
@app
foo

@tables
bar
baz

creates foo-staging-bar, foo-production-bar, foo-staging-baz and foo-production-baz respectively. this is great for apps! many repos with different .arc files can share an app name and access the same dbs. you can write queries directly but resolving table names is a bit cumbersome so architect/data provides some help.
the new feature idea would be to add a section @db so multiple different apps could access the same set of tables. this is a common thing for older/larger systems that combine many apps but a central persistence store.

No interest/need it seems.


The commit author in the Wiki repository is appearing as "undefined".

Azure DevOps Extensions
WIKI Updater
Platform
Azure DevOps Services
Azure DevOps Server (TFS) Version
No response
Extension Version
3.21.11
Describe the bug
The username set in the "gitname" property is not being correctly set for the ".git/config" file. In other words, after running the "git config --local -l" command in the Wiki repository, it appears as "user.name=undefined". Regarding the "gitemail" property, it is working correctly.
Repo Steps
Set up and run a pipeline in Azure DevOps.
      - task: WikiUpdaterTask@3
        name: publishProjectWiki
        displayName: 'Publish Project Wiki'
        condition: and(eq(canceled(), false), succeeded())
        inputs:
          repo: 'dev.azure.com/grupo-siagri/simer/_git/$(repoWikiName)'
          branch: '$(wikiBranchName)'
          localpath: '$(repoWikiPath)'
          filename: '$(releaseNotesWikiPath)/$(releaseNotesWikiName)'
          replaceFile: true
          dataIsFile: true
          sourceFile: '$(Build.SourcesDirectory)/ReleaseNotes.md'
          updateOrderFile: true
          prependEntryToOrderFile: true
          orderFilePath: '$(releaseNotesWikiPath)'
          message: 'Branch: $(wikiBranchName); Release Notes updated by Azure DevOps Pipeline: $(releaseNotesWikiNameCommitMsg)'
          gitname: '$(gitUserName)'
          gitemail: '$(gitUserEmail)'
          useAgentToken: true

Expected Behavior
After committing to the Wiki repository, it should be defined with the author set in the 'gitname' property.
Logging Information
Pipeline:

.git/config:

commit:

I have a repo for that, looks like it has been an bug since day 1, it is a naming clash between a local variable and a parameter
I will get a fix out ASAP

Being shipped as 3.22.3


API to get chip location

Needed in tt-mlir - an API to query a device location (shelf, rack, y, x)

Closing this issue, migrating to #10745


Create stubs for the Safe-DS library

Is your feature request related to a problem?
To implement DS pipelines in Safe-DS, we need stubs for the Safe-DS library.
Desired solution
Once the stub generator has progressed some more,

generate baseline stubs,
manually polish them,
include them in the src/resources.

Possible alternatives (optional)
No response
Screenshots (optional)
No response
Additional Context (optional)
No response

:tada: This issue has been resolved in version 0.9.0 :tada:
The release is available on:

npm package (@latest dist-tag)
npm package (@latest dist-tag)
v0.9.0
GitHub release

Your semantic-release bot :package::rocket:


[APM-CI][WIP] Pipeline version 0

can these files go under a non-root directory, preferably .ci?

I trying to unified were the Jenkins scripts are, on APM server, Go Agent, Hey APM, and Integration test exists a folder scripts/jenkins where all those files are. We can open a team discussion about the location of the files but IMO should be the same on all projects.

Codecov Report

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

@@            Coverage Diff            @@
##             master     #314   +/-   ##
=========================================
  Coverage     72.29%   72.29%
  Complexity     1183     1183
=========================================
  Files           131      131
  Lines          4472     4472
  Branches        453      453
=========================================
  Hits           3233     3233
  Misses         1029     1029
  Partials        210      210

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 666eb4b...5f431f7. Read the comment docs.

The current pipeline setup takes quite a bit more time to execute. Maybe we can re-shuffle the pipeline setup a bit :)
I don't think we need separate steps for Build and Unit Test as testing also does a build. Similarly, the Smoke Tests also do Unit Tests so I think we can condense all three to one stage.
I think the docs build could also run in parallel to the others.
Currently, the benchmarks are only run for changes in master. We would at least need a way to tag PR benchmark results so that the benchmark dashboard can filter out non-master results. Mixing results from master and PRs would not be a good idea.
But I had also totally different idea on how to approach benchmarking for PRs. I don't think that all PRs need to execute the benchmarks. We could have a phrase like "Jenkins benchmark this please" to benchmark on demand. We would then execute the benchmarks twice - once from the point, the PR branched off and once with the PR changes. Then we could compare the two results and add both results, including the diff as a GitHub comment.
This is just an idea. We can also go the route of benchmarking for every PR and enable comparison via a dashboard.

The current pipeline setup takes quite a bit more time to execute. Maybe we can re-shuffle the pipeline setup a bit :)

After the improvements, the preview version takes about 10 min, the old one takes about 14 min, parallelizing another block of test maybe we can decrease another minute or so, but I do not think that worth the effort. Never the less if we categorize/group/tag tests with JUnit annotations it is possible to run them easily by groups.

Currently, the benchmarks are only run for changes in master. We would at least need a way to tag PR benchmark results so that the benchmark dashboard can filter out non-master results. Mixing results from master and PRs would not be a good idea.

We can store use the PR id (benchmark-java-PR134) to store in a different index with the same pattern (benchmark-java-*), for branches we can do the same (benchmark-java-BRANCH_NAME)

But I had also totally different idea on how to approach benchmarking for PRs. I don't think that all PRs need to execute the benchmarks. We could have a phrase like "Jenkins benchmark this please" to benchmark on demand. We would then execute the benchmarks twice - once from the point, the PR branched off and once with the PR changes. Then we could compare the two results and add both results, including the diff as a GitHub comment.

It is easy to be able to launch the job to the Jenkins UI for any branch/PR with the parameters that you want, right now, it is possible to do that on this PR.

After the improvements, the preview version takes about 10 min

wow, that's great!

We can store use the PR id (benchmark-java-PR134) to store in a different index with the same pattern (benchmark-java-*), for branches we can do the same (benchmark-java-BRANCH_NAME)

I would rather add the branch name to the metadata of the benchmark results. That makes it easier to filter out PR benchmark results from the dashboards. How can I get access tot he PR id? Is it an environment variable?

It is easy to be able to launch the job to the Jenkins UI for any branch/PR with the parameters that you want, right now, it is possible to do that on this PR.

perfect!

I would rather add the branch name to the metadata of the benchmark results. That makes it easier to filter out PR benchmark results from the dashboards. How can I get access tot he PR id? Is it an environment variable?

sort of, there are some environment variables defined when you process a branch an others when you process a PR
CHANGE_TARGET - is only defined on PRs
GIT_SHA - SHA1 of commit
BRANCH_NAME - the name of the branch, it is defined only on branches
But we could convert a code that I have to make for codecov to a step and we can export it as an environment variable (BRANCH_REF) https://github.com/elastic/apm-pipeline-library/blob/master/vars/codecov.groovy#L20-L44

what's the status of this? can it be merged?

🎉


Use UTC timezone in the deadline counter

Changes proposed in the pull request:

Passing the server timezone to the Javascript countdown for competition deadline
Adding new update

@AppraiseDev/core-team

Thank you, @snukky!


Sync template from workshop 2021-05-10

Closes: coderefinery/2021-05-10-workshop#19
Review: I don't think there is much uncertain here, I checked
history of both to make sure I did it right.  Always worth a look,
though.

Thanks a lot!


Figure out the top bar on Android's onboarding

Is your feature request related to a problem? Please describe.
Figure out wether to keep top app bar in the onboarding flow. Having top app bar + skip/back buttons separately takes a lot of screen space.
Describe the solution you'd like
If the design called for a bar with a standard Android back button, title, and then a skip button E.g. <- Zcash Skip then that would improve the screen height.

After conversation with Geffen, we can use the standard Android titlebar with the back arrow which will give us some space back on the onboarding screens.
This was already implemented, we have a boolean constant in the onboarding flow that will let you toggle between the two modes.  So we can leverage the existing code (and probably remove the boolean condition entirely).


Update nick_aliases.txt

Added work alias, for those worky days.

I can only add aliases that actually have lines, and from a quick grep that isn't the case for SpamAtWork yet.

All right, how about now?  I just said something as SpamAtWork.


Error when using encrypted communication with preconfigured SSL context

Hello,
https://github.com/jacobschaer/python-doipclient/blob/f278a59b21c996bf67ad7afe4783ef04a7f79f08/doipclient/client.py#L777
If I create a DoIPClient instance with use_secure=ssl_context _connect still tries to create a default context.
I think that in client.py, line 777, in isinstance, type is not needed.
How it works on my setup:
if isinstance(self._use_secure, ssl.SSLContext):
Thank you very much for providing this great library! 👍

Please check #38 for the updated implementation and tests.
@alexeckle: Besides that oversight, did the encrypted communication work for you? If so, we could also remove the untested from the docstring
https://github.com/jacobschaer/python-doipclient/blob/f278a59b21c996bf67ad7afe4783ef04a7f79f08/doipclient/client.py#L145-L146


[#70] get username from request is more flexible

Every function which loads username from request has the optional paramter: get_username
piggyback:
removed redefinition of USERNAME_FORM_FIELD from defender.config

@shnela thank you for the PR. Since you have a pretty good use case for using with DRF, maybe it makes sense to update the README with and example of how you are using it, so that others can do the same?
Once the tests pass, I can merge.

Coverage increased (+0.1%) to 94.57% when pulling cd0a22c1c5fd9c86ba6a6f5c1a6e9ef16c71dd91 on shnela:custom_get_user_from_request into 7a2961a12a8d2d2a324a68959a7e5aa835ef5291 on kencochrane:master.

Coverage increased (+0.1%) to 94.57% when pulling cd0a22c1c5fd9c86ba6a6f5c1a6e9ef16c71dd91 on shnela:custom_get_user_from_request into 7a2961a12a8d2d2a324a68959a7e5aa835ef5291 on kencochrane:master.

Coverage increased (+0.1%) to 94.57% when pulling cd0a22c1c5fd9c86ba6a6f5c1a6e9ef16c71dd91 on shnela:custom_get_user_from_request into 7a2961a12a8d2d2a324a68959a7e5aa835ef5291 on kencochrane:master.

Coverage increased (+0.1%) to 94.57% when pulling cd0a22c1c5fd9c86ba6a6f5c1a6e9ef16c71dd91 on shnela:custom_get_user_from_request into 7a2961a12a8d2d2a324a68959a7e5aa835ef5291 on kencochrane:master.

Coverage increased (+0.1%) to 94.57% when pulling cd0a22c1c5fd9c86ba6a6f5c1a6e9ef16c71dd91 on shnela:custom_get_user_from_request into 7a2961a12a8d2d2a324a68959a7e5aa835ef5291 on kencochrane:master.

Coverage increased (+0.1%) to 94.57% when pulling cd0a22c1c5fd9c86ba6a6f5c1a6e9ef16c71dd91 on shnela:custom_get_user_from_request into 7a2961a12a8d2d2a324a68959a7e5aa835ef5291 on kencochrane:master.

Coverage increased (+0.1%) to 94.57% when pulling cd0a22c1c5fd9c86ba6a6f5c1a6e9ef16c71dd91 on shnela:custom_get_user_from_request into 7a2961a12a8d2d2a324a68959a7e5aa835ef5291 on kencochrane:master.

Coverage increased (+0.1%) to 94.57% when pulling cd0a22c1c5fd9c86ba6a6f5c1a6e9ef16c71dd91 on shnela:custom_get_user_from_request into 7a2961a12a8d2d2a324a68959a7e5aa835ef5291 on kencochrane:master.

Coverage increased (+0.1%) to 94.57% when pulling cd0a22c1c5fd9c86ba6a6f5c1a6e9ef16c71dd91 on shnela:custom_get_user_from_request into 7a2961a12a8d2d2a324a68959a7e5aa835ef5291 on kencochrane:master.

ok @coveralls I get the picture, you don't need say it 9 times.

ok, all tests passed.

@kencochrane yes, I will update README with simple example of BasicAuthentication for rest_framework tomorrow.


Users not saved in database properly

Currently, when adding a user to the database, the process creates a 3rd field with value undefined.

This does not affect the login processs in any way, it just saves unnecessary data.


libmad: new package at 0.15.1b

Description of package
mpeg decoder used by audio applications such as cmus
Checklist

[x] Latest upstream version.
[x] I agree to maintain this package (required).
[x] I agree to notify KISS if I can no longer maintain this package (required).

Looks good. :+1:


Update snapshot-copy-managed-disk.md

added missing -o for formatting output as table

@mpowrie : Thanks for your contribution! The author(s) have been notified to review your proposed change.

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

@mpowrie Thanks for helping improve our docs, we appreciate it!
This change looks good, so:
#sign-off


Fixed border-radius typo in MyButton.vue

Issue:
There was a small typo in MyButton.vue, border-radius was misspelled.
What I did
Renamed to border-radius.
How to test
Create a new vue project (e.g. using vue-cli). Run getstorybookand run yarn run storybook.

Thank you for this contribution @herrmannplatz
@kazupon feel free to merge things like this 👍 in the future!

@ndelangen
Ok! I got it!


Module: Javascript Project: Weather APP

Will redesign the weather app with new color codes and also use local storage to showcase the past results.

sure thank mate.

Sir please mark this issue as an closed with hacktoberfest-accepted tag so it will reflect in profile.


Job Boards link produces 500

https://remoet.dev/job-boards
URL: https://remoet.dev/job-boards
Status: 500

Request
:method: GET
:scheme: https
:authority: remoet.dev
:path: /job-boards
Accept: text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8
Accept-Encoding: gzip, deflate, br
Accept-Language: ru
Connection: keep-alive
Cookie: _ga=GA1.1.413882733.1723396998; _ga_Q6X96R180L=GS1.1.17234856<IP_ADDRESS>34858<IP_ADDRESS>; __Host-next-auth.csrf-token=8185ef32c2c7abf01ddb9b666a057098450de450dbf69f81ec69be711bc38df7%7Cf2424f7452d7e46da6a1b59b447f41dc5e7418c3e64ba857b1400d88b9a07530; __Secure-next-auth.callback-url=https%3A%2F%2Fremoet.dev
Host: remoet.dev
Referer: https://remoet.dev/developers
Sec-Fetch-Dest: document
Sec-Fetch-Mode: navigate
Sec-Fetch-Site: same-origin
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/17.6 Safari/605.1.15

Response
:status: 500
Accept-Ranges: bytes
Access-Control-Allow-Origin: *
Age: 415209
Cache-Control: public, max-age=0, must-revalidate
Content-Disposition: inline; filename="500"
Content-Length: 2878
Content-Type: text/html; charset=utf-8
Date: Mon, 12 Aug 2024 18:04:49 GMT
ETag: "030e833b330aed6e34534e346672c400"
Server: Vercel
Strict-Transport-Security: max-age=63072000
x-matched-path: /500
x-vercel-cache: HIT
x-vercel-id: cdg1::p224j-1723485884276-4c68e3e82ac5

Hi 👋 Thanks for the issue! This is soon to be resolved, I am fixing this very soon 🤓


Barnsley

I noticed that this challenge didn't have contribution images so I added.  I also deleted  two contributions that weren't functional.

Wow! ❤


Code blocks in Markdown render poorly

Environment data

VS Code version: 1.89.1
Jupyter Extension version (available under the Extensions sidebar): v2024.3.1
Python Extension version (available under the Extensions sidebar): v2024.4.1
OS (Windows | Mac | Linux distro) and version: Fedora 40
Python and/or Anaconda version: 3.11
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): conda
Jupyter server running: Local

Expected behaviour
When a code block (here, Python) is added to a markdown cell using triple quotes, the block is formatted as code with appropriate language coloration when specified. Here is a working example in a browser-based Jupyter notebook:

Actual behaviour
In VSCode, the block is not well defined and does not include language coloration despite the language being specified with the code block. It looks really terrible, almost as if inline code formatting is being used on the block. If  this is the case, it is not the correct encoding.
Note that this is independent of which theme is being used in VSCode. They all look similar to this, but using the dominant colors of the theme being used.

Steps to reproduce:
Create a code block inside of a markdown cell using the triple quote notation (and language label, optionally)

This occurs with any notebook containing markdown code blocks, but here is a simple example.

Thanks. Looks fine to me tho:

Does this reproduce in the latest VS Code insiders build with all extensions disabled?

Thank you. It looks like a Markdown preview extension was causing this. Sorry for the trouble!


Convert all @Overwrite Mixins to @Inject

Compatibility is vastly reduced by using @Overwrite. The mod may not even be compatible with itself in some cases!
All Mixins that use @Overwrite should be carefully rewritten to use @Inject instead.

This issue should be closed now I think?

I had to use a few overwrites for some mods patches so not completely yet.

There are still 5 Overwrites in the main module as of today.


Request for Example or Barebone Documentation

Hi @aabmets I just came across your project, and I wanted to say that this concept seems very promising! Automating the generation of IPC components for Electron apps could save a lot of development time and effort.
Would it be possible to include a basic example within the codebase or provide a barebone documentation file to help others get started quickly? It would be incredibly helpful to understand the plugin’s intended usage and capabilities.
Thanks for your work on this – looking forward to seeing how it evolves!

@Rajaniraiyn I haven't gotten around to writing documentation, because the custom DSL was until recently still in flux and there were breaking changes. Currently, the library is losing the Vite requirement, because it is not possible to get Vite HMR to reload the main process when IPC automation has generated new bindings. Today I finished writing comprehensive vitests for the library and I'm planning on releasing this library soon on npmjs, before Christmas at the latest.

@Rajaniraiyn Christmas came early https://www.npmjs.com/package/automate-electron-ipc

That’s awesome! Looks like Christmas really did come early.


Cut-and-Paste simple text outlines from other apps

Implement the ability to import a simple text outline created in any external app (e.g. Notes, BBedit).  The external outline would have to be created using either tabs or cmd-] for indentation.  It would be helpful to go ther other way as well (e.g. from Zavala to Notes).  Cutting and pasting an outline, as opposed to an entire file, is what's relevant since external dosuments would genearlly have content that doesn't conform to an outline.

I'm pretty sure that this is a duplicate of #189. If you feel that #189 doesn't cover all your points, we can comment on it or reopen this issue.


Plugin needs improved connection error handling

This has happened several times in the last couple of days. Some sort of network issue causes a plugin connection failure, and instead of self recovering, the plugin goes into a loop with an error every ten minutes. Restarting the plugin from the Indigo console is successful every time.
Could this kind of connection failure be caught and handled with functionality being eventually restored once the network is running again? The ten minute loop seems to imply to me that the retry logic is already there, but there may be an issue with what the retry is attempting to do.
Here's the log.
 There are a lot more of these.)**

Dec 6, 2020 at 8:20:00 AM
   Lutron RRA2/Caséta Warning     Lutron IP Gateway: Error sending IP command, resetting connection:  'int' object has no attribute 'sendall'

Dec 6, 2020 at 8:30:00 AM
   Lutron RRA2/Caséta Warning     Lutron IP Gateway: Error sending IP command, resetting connection:  'int' object has no attribute 'sendall'

Dec 6, 2020 at 8:40:00 AM
   Lutron RRA2/Caséta Warning     Lutron IP Gateway: Error sending IP command, resetting connection:  'int' object has no attribute 'sendall'

Dec 6, 2020 at 8:50:00 AM
   Lutron RRA2/Caséta Warning     Lutron IP Gateway: Error sending IP command, resetting connection:  'int' object has no attribute 'sendall'

Dec 6, 2020 at 8:54:40 AM
   WebServer                       Indigo Touch client connected from <IP_ADDRESS>

Dec 6, 2020 at 8:56:39 AM
   Reloading plugin "Lutron RRA2/Caséta 7.3.3"
   Stopping plugin "Lutron RRA2/Caséta 7.3.3" (pid 10041)
   Lutron RRA2/Caséta             Shutting down Lutron
   Stopped plugin "Lutron RRA2/Caséta 7.3.3"
   Starting plugin "Lutron RRA2/Caséta 7.3.3" (pid 14790)
   Started plugin "Lutron RRA2/Caséta 7.3.3"
   Lutron RRA2/Caséta             Starting up Lutron
   Lutron RRA2/Caséta             Lutron IP Gateway: Running IP Start
   Lutron RRA2/Caséta             Lutron IP Gateway: Connecting via IP to <IP_ADDRESS>:23

Yeah, I worked on the reconnect code some, but I need to take another pass at it.  Hard for me to test without shutting down the system here, which annoys the spouse.  ;)

The good news is, the retry logic seems solid. :) Seems like something is getting broken when the connection initially fails and it isn't being fully reset.

I have a similar issue sporadically. Anything specific from the logs that could help diagnose the issue? I just assume my ancient Mac Pro has random connectivity issues.

I can't help with the original disconnects, that's an OS or network issue.  I think.  But I should be able to get it to reconnect more reliably.  I just need to figure out how to close out everything and start from scratch.  Or if it's a problem that the process is messed up, just restart the plugin.  Which might be the right answer.

Yep, I think my router, switches, or access points might have received a
firmware update. Nothing you can do about that. Restarting the plug-in
seems like a pretty big hammer if it is going to restart with every failed
retry. But if that’s what it takes, that’s what it takes I guess.
On Sun, Dec 6, 2020 at 6:40 PM Joe Keenan<EMAIL_ADDRESS>wrote:

I can't help with the original disconnects, that's an OS or network issue.
I think. But I should be able to get it to reconnect more reliably. I just
need to figure out how to close out everything and start from scratch. Or
if it's a problem that the process is messed up, just restart the plugin.
Which might be the right answer.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/FlyingDiver/Indigo-radiora2/issues/48#issuecomment-739613774,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AINFBCDYX77BOODGJSSNH4TSTQXCHANCNFSM4UPNKNPQ
.

Fixed in next release.  Tested with two scenarios.  First, network disconnect by unplugging the ethernet to the repeater).  Second, power failure by unplugging power to repeater.  In both cases, the plugin reconnected to the repeater properly within 30 seconds of network or power restored.

Fixed in next release.  Tested with two scenarios.  First, network disconnect by unplugging the ethernet to the repeater).  Second, power failure by unplugging power to repeater.  In both cases, the plugin reconnected to the repeater properly within 30 seconds of network or power restored.

Were you able to consider a temporary network failure. You could emulate it by disconnecting ethernet from the host Mac.

Were you able to consider a temporary network failure. You could emulate it by disconnecting ethernet from the host Mac.

No difference.  Unplug the cable at the repeater or unplug it at the Indigo Mac.  Same symptoms, same solution.

No difference.  Unplug the cable at the repeater or unplug it at the Indigo Mac.  Same symptoms, same solution.


handle empty in Generated{Product,Abstract}Algebra

This checks for an empty iterator first and breaks, rather than create N ArrayBufferedOperation instances just to bail out if the sequence has nothing to offer.

:+1:
Note you should probably do this same thing to the case class macros. Shame we have to fix it N times.

@johnynek  Here we go: https://github.com/twitter/algebird/pull/599


Multi control net Blender issue when using more than one model

When I enable more than one model I am getting this error.

can you paste your scripts?


checkov: 2.3.92 -> 2.3.95

Diff: https://github.com/bridgecrewio/checkov/compare/refs/tags/2.3.92...2.3.95
Changelog: https://github.com/bridgecrewio/checkov/releases/tag/2.3.95
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 221456 run on x86_64-linux 1

1 package built:

checkov


feat: add xkcd tutorial steps for v0.2.0

Primary Reviewer
@lachieh
Feature or Problem
This PR adds the remaining tutorial steps from the website to the XKCD actor, and the nice Chakra UI from wasmcon, effectively letting users follow the initial tutorial directly from the app itself.
Related Issues
Will need a followup updating the wadms to use this actor's v0.2.0 version once published
Release Information

Consumer Impact

Testing

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin
[ ] x86_64-windows

Tested on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin
[ ] x86_64-windows

Unit Test(s)

Acceptance or Integration

Manual Verification

new look


Unable to create virtual machine

I seem to be having some issues creating vms using the arm resource library.
This is the code i'm using:
func CreateVm() {
	client := resources.NewClient(helpers.SubscriptionID)
	client.Authorizer = helpers.GetServicePrincipalToken()
	gr := resources.GenericResource{}
	grName := "Test50"
	gr.Name = &grName
	grLocation := "northeurope"
	gr.Location = &grLocation // Somehow get a list of these
	grProperties := make(map[string]interface{})
	grProperties["notes"] = "Test-machine-image" // I think this is the description (maybe?)
	grProperties["customImageId"] = "/subscriptions/5dcf0555-####-####-####-############/resourcegroups/test/providers/microsoft.devtestlab/labs/test-lap/customimages/test-machine-image"
	// Get a list of these, and calculate something that fits from what the user wants
	grProperties["size"] = "Standard_A2"
	grProperties["userName"] = "" // Why is this empty, that is strange. Need to experiment
	grProperties["password"] = nil // Same as userName
	grProperties["isAuthenticationWithSshKey"] = false
	grProperties["disallowPublicIpAddress"] = false
	gr.Properties = &grProperties
	result, err := client.CreateOrUpdate("bla2077483059", "Microsoft.Compute", "virtualMachines", "", "test50", "2015-06-15", gr)
	if err != nil {
		panic(err)
	}
	fmt.Println(result.Name)
}

Whenever that runs i end i the panic call with this error:
panic: resources.Client#CreateOrUpdate: Failure responding to request: StatusCode=400 -- Original Error: azure: Service returned an error. Code="BadRequest" Message="The request message is invalid." Status=400
It might be because i'm unable to find the correct parameters for the CreateOrUpdate call, but the resource api docs doesn't exactly describe what should be done: https://msdn.microsoft.com/en-us/library/azure/dn790569.aspx

Ended up solving this in another using the actual compute part of the arm api.


Allow multiple cases for a single block inside switch statements

Coverage increased (+0.05%) to 82.59% when pulling d789ee13c4f8a8c9d96dc6959e4c1d8797662b00 on lukelafountaine:allow-multi-case-single-block into 6dcf68a698b76041735f7fd58ec15f37a948625d on silvermine:master.

Coverage increased (+0.05%) to 82.59% when pulling d789ee13c4f8a8c9d96dc6959e4c1d8797662b00 on lukelafountaine:allow-multi-case-single-block into 6dcf68a698b76041735f7fd58ec15f37a948625d on silvermine:master.

Coverage increased (+0.05%) to 82.59% when pulling d789ee13c4f8a8c9d96dc6959e4c1d8797662b00 on lukelafountaine:allow-multi-case-single-block into 6dcf68a698b76041735f7fd58ec15f37a948625d on silvermine:master.

Coverage increased (+0.05%) to 82.59% when pulling d789ee13c4f8a8c9d96dc6959e4c1d8797662b00 on lukelafountaine:allow-multi-case-single-block into 6dcf68a698b76041735f7fd58ec15f37a948625d on silvermine:master.


compilation with Smack disabled

The integrity layer accidentally enabled Smack unconditionally.
Also found a minor issue in coreutils while testing.

Looks OK, please merge.


ESPProvision Wifi Provisioning - Additional BLE Endpoints not working (IDFGH-13630)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

IDF version.
v5.3.1
Espressif SoC revision.
ESP32-S3
Operating System used.
macOS
How did you build your project?
VS Code IDE
If you are using Windows, please specify command line type.
None
Development Kit.
ESP32-S3-DevKitC-1U
Power Supply used.
USB
What is the expected behavior?
When adding a custom endpoint using the example code at https://docs.espressif.com/projects/esp-idf/en/stable/esp32s3/api-reference/provisioning/wifi_provisioning.html?highlight=wifi provisioning#additional-endpoints, I expect the ESP32-S3 to broadcast that endpoint in addition to the pre-defined prov-* endpoints included as part of the Wifi Provisioning service
What is the actual behavior?
Everything appears to load OK and no errors are reported, however when inspecting the BLE device using a scanner, the custom endpoint does not appear. Also, calling 'ESPDevice.sendData(...)from the iOS app returns sendDataError with errorBLE characteristic does not exist`.
Steps to reproduce.
Here is the code I'm using:
`void provision_ble(void)
{
if (!wifi_sta_initialised)
{
    sta_netif = esp_netif_create_default_wifi_sta();
    wifi_sta_initialised = true;
}
wifi_init_config_t cfg = WIFI_INIT_CONFIG_DEFAULT();
ESP_ERROR_CHECK(esp_wifi_init(&cfg));

/* Configuration for the provisioning manager */
wifi_prov_mgr_config_t config = {
    .scheme = wifi_prov_scheme_ble,
    .scheme_event_handler = WIFI_PROV_SCHEME_BLE_EVENT_HANDLER_FREE_BTDM};

/* Initialize provisioning manager with the
 * configuration parameters set above */
ESP_ERROR_CHECK(wifi_prov_mgr_init(config));

esp_err_t err = wifi_prov_mgr_endpoint_create("custom-data");
if (err != ESP_OK)
{
    ESP_LOGE(TAG, "Error creating custom endpoint: %d", err);
}

char *ssid = ipb_config_get_str("wifi.ap.ssid");
const char *pop = ipb_config_get_str("wifi.ap.pass");
char service_name[21]; // Ensure the array is large enough to hold the string and null terminator
strncpy(service_name, ssid, sizeof(service_name) - 1);
service_name[sizeof(service_name) - 1] = '\0'; // Ensure null termination

wifi_prov_security_t security = WIFI_PROV_SECURITY_1;
wifi_prov_security1_params_t *sec_params = pop;

free(ssid);

const char *service_key = NULL;

uint8_t custom_service_uuid[] = {
    /* LSB <---------------------------------------
     * ---------------------------------------> MSB */
    [removed]};

wifi_prov_scheme_ble_set_service_uuid(custom_service_uuid);

ESP_ERROR_CHECK(wifi_prov_mgr_start_provisioning(security, (const void *)sec_params, service_name, service_key));

err = wifi_prov_mgr_endpoint_register("custom-data", prov_sys_reboot, NULL);

wifi_prov_print_qr(service_name, pop, PROV_TRANSPORT_BLE);

}
esp_err_t prov_sys_reboot(uint32_t session_id, const uint8_t *inbuf, ssize_t inlen,
uint8_t **outbuf, ssize_t *outlen, void *priv_data)
{
ESP_LOGI(TAG, "Calling sys reboot");
if (inbuf)
{
ESP_LOGI(TAG, "Received data: %.*s", inlen, (char *)inbuf);
}
char *response = rpc_sys_reboot(NULL);
*outbuf = (uint8_t *)strdup(response);
if (*outbuf == NULL)
{
ESP_LOGE(TAG, "System out of memory");
return ESP_ERR_NO_MEM;
}
outlen = strlen(response) + 1; / +1 for NULL terminating byte */
free(response);
return ESP_OK;

}`
Debug Logs.
No response
More Information.
Other than the custom endpoint, all other aspects of the BLE provisioning process work correctly.

By means of an update, I can get this working on Android, but not on iOS.
My (working) Android call is:
val byteArray = "forceEmptyBox".toByteArray()   espDevice?.sendDataToCustomEndPoint("custom-api", byteArray, object : ResponseListener {
However, when I call the iOS equivalent, I get the error:
let jsonData = "forceEmptyBox".data(using: .utf8)!   self.espDevice?.sendData(path: "custom-api", data: jsonData) { responseData, error in

It turns out that I simply needed to restart the iOS device and scanning then worked correctly.

It turns out that I simply needed to restart the iOS device and scanning then worked correctly.

That's great @gadget-man. If any issue comes please feel free to reopen the issue.


Error in setup when fee estimation is not ready

The setup ended since the fee estimation was not ready.
Error! Hit unexpected exception
msg: 'Fee estimation faild. Fallbackfee is disabled. Wait a few blocks or enable -fallbackfee' code:-4

Yes, I was doing a new wallet setup.
Jeffrey J. Gotts
Owner
Gotts Engineering, LLC
“It is curious in how great a degree human progress depends on the individual. Humans, numbered in thousands of millions, seem organized into an ant-like society. Yet this is not so. New ideas, the impetus of all development, come from individual people, not from corporations or states. New ideas, fragile as spring flowers, easily bruised by the tread of the multitude, may yet may be cherished by the solitary wanderer.”
Fred Hoyle - 1957
Sent from Proton Mail mobile
-------- Original Message --------
On Dec 7, 2022, 8:48 AM, jamesob wrote:

Thanks for the report.
Were you performing a new wallet setup when this occurred? What was the step that you were on when you hit this error message?
As a temporary workaround, you can restart bitcoind with the -fallbackfee option:
-fallbackfee=
A fee rate (in BTC/kvB) that will be used when fee estimation has insufficient data. 0 to entirely disable the fallbackfee feature. (default: 0.00)

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Accidentally triggering the Pull to refresh action.

If the scroll view has bouncing enabled it is very easy to accidentally trigger a pull to refresh action while scrolling the view up. The view will bounce using the momentum and velocity, reaching the threshold and thus triggering the action.
I think it should only be triggered while a user is dragging the view down (still touching the screen with their finger).

There's no apparent good way to detect when touches begin or end on a ScrollView, since this is usually done with a DragGesture, which then in turn overrides the scrolling behaviour.
You could track how fast the user's scrolling and use that data to figure out if a bounce happened, but that seems brittle. I'll give this some more thought.

I'm running into the same issue but unsure how to solve it.    Thanks.

I tried everything I could think of to try to track if the user's actually touching the screen while pulling, but various gestures either get hijacked by the ScrollView (at least in the onEnded part) or interfere with regular scroll.
I also tried figuring out if a bounce happened by analyzing a snapshot of the latest scroll positions to detect if there's been a sudden change in scrolling direction, but it has too many false positives.
I guess you can always turn bouncing off.

How about using introspect to keep track of scrollview state? That way you could know the touches

I'll give that a try but I'm not sure if we want to make a dependency on the introspect lib as it's not maintained as much anymore. Let me know if this that big of an issue for you guys.


Fixed the default DCOS version in v20170701 to 1.9.0 and removed unsu…

…pported
DCOS versions

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
If applicable:

[ ] documentation
[ ] unit tests
[ ] tested backward compatibility (ie. deploy with previous version, upgrade with this branch)

Release note:

@dmitsh @jackfrancis @amanohar


Missing info in info pane

Describe the bug
All the right column info (release date, SDKs, etc) is missing when I select an installed version of Xcode, but they do show up for versions that aren't installed.
Also, no information is displayed in the PlatformsView in InfoPane when I select a version of Xcode (installed or not), but they do show up in the Platforms window.
UPDATE: They did show up in the Platforms window the first time I opened it, but now it is blank too. 🤔
To Reproduce
Select an installed version of Xcode in the list.
Maybe the issue is related to using a non-default install location for Xcode?

Expected behavior
All the information that shows for uninstalled versions should show for installed versions too.
And the Platforms list should be populated (maybe a separate issue?)
Screenshots

Version

OS: 14.3.1
Xcodes: 2.0.2

I tried diagnosing and fixing the issue myself so I could make a PR instead of an Issue, but when I try to run Xcodes locally the build crashes on launch in some assembly code, and I can't get the previews to work either (probably for the same reason), and unfortunately I don't have time right now to diagnose that and the info issue. 😕

@rtharston  Can you change your datasource to be Xcode Releases instead of Apple - that should fix your issue.
It still is a bug, where the platforms should still show.

It is already set to Xcode Releases. Note that all the info is present when I select uninstalled versions of Xcode. It is only missing for installed versions.

hmmm, ok can you try deleting the available-xcodes.json and downloadable-runtimes.json from the local cache path?
Then hit refresh. If the platforms don't show up then, it means it's having trouble connecting to https://devimages-cdn.apple.com/downloads/xcode/simulators/index2.dvtdownloadableindex to download them.

That was it!
When I updated from version 1.10 all my settings were reset and a new set of json files were created in ~/Library, so I changed the settings to point at my old json files. That also explains why I remember seeing the Platforms info show up the first time I opened the application, it was still using the new files!
So I take it the format changed between 1.x and 2.0?

The format of available-xcodes i believe wouldn't have changed. downloadable-runtimes is new in 2.0 so that would have been added.
Glad things worked out for you!

By the way, I'm loving the new info pane! Adding the platforms install buttons for each Xcode version is a great touch.

Then maybe I just needed to hit 'refresh' to recreate downloadable-runtimes. I hit the same issue on my desktop computer this morning so I can try just hitting refresh on that computer when I get home. I imagine that will do it.

It should do that on it's own when you first open the app, but if you haven't closed the app in a while, you'll need to manually do ti.

When it first opened it was still pointing at Library instead of my custom location.
So I guess there are two questions here. Why did 2.0 reset my custom location? And should changing the location cause a refresh?

Yep, hitting refresh fixed everything on my desktop. I guess I didn't think to try that since the Xcodes list was populated.


Added example usages of RangeWithOptions, MultiRangeWithOptions, and MultiGetWithOptions

Added examples to the godoc regarding RangeWithOptions, MultiRangeWithOptions, and MultiGetWithOptions methods.

looks good. minor comments

fixed the the ts.add with "time-serie-1" to "time-serie-2" on the last addition on the examples. Also changes labels to labels1.


Support broadcast for user-defined functions !! 😄 💯

current:
julia> f(x) = 2x + 1
f (generic function with 1 method)

julia> x = Variable([1, 2, 3])
name: nothing
values: 3-element Vector{Int64}:
 1
 2
 3
creator: User-Defined(nothing)

julia> f.(x)
ERROR: MethodError: no method matching length(::Variable{Vector{Int64}})
Closest candidates are:
  length(::Union{Base.KeySet, Base.ValueIterator}) at abstractdict.jl:58
  length(::Union{LinearAlgebra.Adjoint{T, S}, LinearAlgebra.Transpose{T, S}} where {T, S}) at /Users/julia/buildbot/worker/package_macos64/build/usr/share/julia/stdlib/v1.6/LinearAlgebra/src/adjtrans.jl:195
  length(::Union{DataStructures.OrderedRobinDict, DataStructures.RobinDict}) at /Users/yuchi_ymgc/.julia/packages/DataStructures/ixwFs/src/ordered_robin_dict.jl:86
  ...
Stacktrace:
 [1] _similar_for(c::UnitRange{Int64}, #unused#::Type{Any}, itr::Variable{Vector{Int64}}, #unused#::Base.HasLength)
   @ Base ./array.jl:575
 [2] _collect(cont::UnitRange{Int64}, itr::Variable{Vector{Int64}}, #unused#::Base.HasEltype, isz::Base.HasLength)
   @ Base ./array.jl:608
 [3] collect(itr::Variable{Vector{Int64}})
   @ Base ./array.jl:602
 [4] broadcastable(x::Variable{Vector{Int64}})
   @ Base.Broadcast ./broadcast.jl:682
 [5] broadcasted(::Function, ::Variable{Vector{Int64}})
   @ Base.Broadcast ./broadcast.jl:1307
 [6] top-level scope
   @ REPL[3]:1

after this PR:
julia> f(x) = 2x + 1
f (generic function with 1 method)

julia> x = Variable([1, 2, 3])
name: nothing
values: [1, 2, 3]
creator: User-Defined(nothing)

julia> y = f.(x)
name: nothing
values: [3, 5, 7]
creator: JITrench.Broadcasting{typeof(+)}

Computational graph:

To implement this future, we add a new field, req_broadcast to Variable.
After this PR, all arguments of  DiffableFunction are checked in this field, and when it is true, arguments become a target for broadcast.
This may slow down the computation.
Since there are only a limited number of function calls that need to be broadcast, good optimization should reduce the overhead of this check.

LGTM !!


[Feature Request]: Make it easier to discover projects built with autogen

Is your feature request related to a problem? Please describe.
Especially those with OSS implementation. For example, Agent-E and SciAgents.
Make it easy to search, browse and try out.
Describe the solution you'd like
Build a website to search for agents built with autogen
In future, make a playground for easily testing such agents.
Additional context
No response

search for agents built with autogen

workaround:
try https://github.com/search?q="import+autogen"&type=code
and https://github.com/search?q="from+autogen+import"&type=code


Plugin does not work anymore in gradle 8.2

It seems bumping to gradle 8.2 now breaks because of this settings plugin.
Here is the error stacktrace coming from gradle:
FAILURE: Build failed with an exception.

* What went wrong:
Configuration cache problems found in this build.

1 problem was found storing the configuration cache.
- Plugin 'net.linguica.maven-settings': external process started 'env'
  See https://docs.gradle.org/8.2/userguide/configuration_cache.html#config_cache:requirements:external_processes

See the complete report at file:///Users/runner/work/1/s/build/reports/configuration-cache/bf5990qu3jevy8o8crnmpt4x7/8vbc41lcq8gwzfj2h91grztxz/configuration-cache-report.html
> Starting an external process 'env' during configuration time is unsupported.

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.
> Get more help at https://help.gradle.org./

BUILD FAILED in 8m

Deprecated Gradle features were used in this build, making it incompatible with Gradle 9.0.

You can use '--warning-mode all' to show the individual deprecation warnings and determine if they come from your own scripts or plugins.

For more on this, please refer to https://docs.gradle.org/8.2/userguide/command_line_interface.html#sec:command_line_warnings in the Gradle documentation.
76 actionable tasks: 76 executed
Configuration cache entry discarded with 1 problem.
Error: The process '/Users/runner/work/1/s/gradlew' failed with exit code 1
    at ExecState._setResult (/Users/runner/work/_tasks/Gradle_8d8eebd8-2b94-4c97-85af-839254cc6da4/2.231.0/node_modules/azure-pipelines-task-lib/toolrunner.js:943:25)
    at ExecState.CheckComplete (/Users/runner/work/_tasks/Gradle_8d8eebd8-2b94-4c97-85af-839254cc6da4/2.231.0/node_modules/azure-pipelines-task-lib/toolrunner.js:926:18)
    at ChildProcess.<anonymous> (/Users/runner/work/_tasks/Gradle_8d8eebd8-2b94-4c97-85af-839254cc6da4/2.231.0/node_modules/azure-pipelines-task-lib/toolrunner.js:839:19)
    at ChildProcess.emit (node:events:513:28)
    at maybeClose (node:internal/child_process:1100:16)
    at Process.ChildProcess._handle.onexit (node:internal/child_process:304:5)
##[error]Error: The process '/Users/runner/work/1/s/gradlew' failed with exit code 1
Finishing: Assemble and run unit tests (:tv:app module)```

@mark-vieira @aalmiray can you check this issue? it is a major issue because AZ Artifacts recommends this plugin in their official documentaion and this now does not work with gradle 8.2

@mark-vieira any news on this issue?

I created a new version of this plugin. You can find it here:
https://plugins.gradle.org/plugin/io.github.rmanibus.maven-settings
https://github.com/rmanibus/gradle-maven-settings-plugin


Integration failed to setup

Describe the bug
The integration fails to setup, shows "Failed setup, will retry" as statys
To Reproduce
Add Dreambot L10S Ultra using Xiaomi
Expected behavior
Should initialise the integration and gather entities
Screenshots

Additional Information (please complete the following information)

Model Name [e.g. dreame.vacuum.p2028]: L10S Ultra
Firmware Version [e.g. 1156]: 4.3.6_3018
Home Assistant Version: 2023.8.4
Configuration Type [With or without map support]: With map using Xiaomi MI Home
Errors or warnings shown in the HA logs (if applicable): https://pastebin.com/ePrsU7RY

I've tried to remove completely the integration, tested with stable release and also with the new 2.0 beta, same results

Restored the robot to factory defaults and it started to work again, sorry!

Indeed that was it! thanks!


feat: port MeasureTheory.Function.Jacobian

[ ] depends on: #4832

This PR/issue depends on:

leanprover-community/mathlib4#4832
By Dependent Issues (🤖). Happy coding!

bors d+
Please wait for Besicovitch to land.

This PR/issue depends on:

leanprover-community/mathlib4#4832
By Dependent Issues (🤖). Happy coding!

bors r+


"buy" action in dashboard is throwing error

https://thirdweb.com/mumbai/0x2D502F5C8Da741FC572D5D2CB113Cd77355e4d33/market/0xc1B5eBBCe602c2079ac967aB12eddDa4Cf4aFE8D
exception:

@ayshptk can you look into this please?


ChatGPT独享账号，到手即用

Describe the feature
ChatGPT独享账号，到手即用
http://bwww.vip/transitive.html

You are really embarrassing. Do you think people who come here are missing an account?


Send later not functioning predictably

I schedule a couple of emails to send later, but for some reason the scheduling is all messed up.
When I set a specific snooze time, that works fine, but send later doesn't seem to work right.
It might be that it is snoozing according to some other time zone. I'm based in Israel, so ET +7hrs. Not clear.
In the image below, you can see the time on my computer's clock (top right 8:48PM), snooze later time (6:03PM) and time it is actually planning to sent on the message window (in 3 hours).

I'd love for this feature to be predictable and reliable.
Thanks!
OSX El Capitan
Version 0.4.38-27d21d1 (0.4.38-27d21d1)

hey @osakkers, thanks for reporting this! This is no good. Can you tell me what time you scheduled it for specifically?

6:03 PM - it's the second message in the drafts folder which you can see in the background and you can see the timestamp on the far right.

Update: I checked my drafts folders again and the messages still haven't sent (even though previously it said they were due to send in 3hrs). It seems that it simply is failing to send. So it seems it's not a time zone/clock issue. See latest image:

thanks for the update @osakkers , still looking into this

@osakkers could you also tell me at what time you scheduled it?

Around noon if I recall correctly...
On May 19 2016, at 12:11 am, Juan<EMAIL_ADDRESS>wrote:

@osakkers could you also tell me at what time
you scheduled it?

—
You are receiving this because you were mentioned.
Reply to this email directly or [view it on GitHub](https://github.com/nylas/N
1/issues/2258#issuecomment-220159558)![](https://github.com/notifications/beac
on/ASSWolmDKtl3ehepwY5gV1grbmEfwe3Oks5qC4CbgaJpZM4Igic_.gif)

great, thanks! I think we know what is causing this, we should ship a fix soon. Thanks!

Awesome. You guys are the best!
Sent from my iPhone

On May 19, 2016, at 12:47 AM, Juan<EMAIL_ADDRESS>wrote:
great, thanks! I think we know what is causing this, we should ship a fix soon. Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

Hey folks—we've dedicated more server resources to running the Send Later queue and it's resolved this issue. Cheers!

Hi, I scheduled a number of messages last night and not one of them sent this morning, despite my computer being on.
I also didn't get an alert or anything. Do I need to set reminders to check every time I send later? It's pretty disastrous if messages don't send!
Is there anything in place to ensure they get sent?
Help much appreciated.

send later dont work at all! The messages stay in the draft section then disappears, no one gets them, not in sent folder.
I really liked the Nylas app but the basic things are not working on this email as advertised. I really tried hard to love this app even i subscribed but I think i have to cancel my subscription now :(


ENH: Increase itk::XMLFileOutputWindow class coverage

Increase itk::XMLFileOutputWindow class coverage.
PR Checklist

[X] No API changes were made (or the changes have been approved)
[X] No major design changes were made (or the changes have been approved)
[X] Added test (or behavior not changed)

Testing the contents is left for a future PR: a way to easily compare the contents of ASCII files to a baseline should be added to the testing infrastructure.


Update DEVELOPER.md

Summary
fixed some typo
SUMMARY_GOES_HERE
Full changelog

[Implement ...]
[Add related tests]
...

Issues resolved
Fix #XXX

done

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@sakshamtaneja21 Could you go through the [https://cla-assistant.io/Kong/kong?pullRequest=5183](CLA Assistant process) so we can accept your PR? thank you!

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

I am closing this PR down since we could not get a CLA signature. Please reopen if you decide to go with it later on, @sakshamtaneja21 .

please check @kikito  i have done with CLA


Multi-organism terms: regulation of protein localisation and cytokine secretion

I need to create some multi-organism terms for the annotation of PMID:25063865. Putting here as a reminder and in case of comments.
ONE:
Leishmania LPG2 (Q25266) excludes mouse Synaptotagmin XI (Syt11) from the phagosome:
modulation by symbiont of host cellular process ; GO:0044068
——[isa]modulation by symbiont of host protein localisation to phagocytic vesicle ; GO:NEW1
———[isa]negative regulation by symbiont of host protein localisation to phagocytic vesicle ; GO:NEW2
modulation by symbiont of host protein localisation to phagocytic vesicle ; GO:NEW1
Any process in which an organism modulates the frequency, rate or extent of protein localisation to the host phagosome. The host is defined as the larger of the organisms involved in a symbiotic interaction.
Exact synonym: modulation by symbiont of host protein localisation to phagosome [GOC:bf]
negative regulation by symbiont of host protein localisation to phagocytic vesicle ; GO:NEW2
Any process in which an organism stops, prevents, or reduces the frequency, rate or extent of protein localisation to the host phagosome. The host is defined as the larger of the organisms involved in a symbiotic interaction.
Exact synonym: inhibition of host protein localisation to phagosome
Exact synonym: disruption of host protein localisation to phagosome
Exact synonym: suppression of host protein localisation to phagosome
TWO:
Leishmania GP63 cleaves Syntaxin XI (a negative regulator of cytokine secretion) and thereby promotes cytokine secretion:
modulation by symbiont of host innate immune response ; GO:0052167]
—[isa]positive regulation by symbiont of host immune response ; GO:0052166
——[isa]positive regulation by symbiont of host cytokine secretion ; GO:NEW
positive regulation by symbiont of host cytokine secretion ; GO:NEW
Any process in which an organism activates, maintains or increases the frequency, rate or extent of cytokine secretion in the host organism. The host is defined as the larger of the organisms involved in a symbiotic interaction.
exact synonym: induction of cytokine secretion [PMID: 25063865]

3 new terms added:
modulation by symbiont of host protein localization to phagocytic vesicle; GO:0036521
negative regulation by symbiont of host protein localization to phagocytic vesicle; GO:0036522
positive regulation by symbiont of host cytokine secretion ; GO:0036523


fix: added owner check for WS pages

Details

This change introduces access to various pages related to Workspace, Only owner can access them
Fixed Issues

$ https://github.com/Expensify/App/issues/21998
PROPOSAL: https://github.com/Expensify/App/issues/21998#issuecomment-1615160146
Tests

Create a workspace User A and add User B
Login with User B
With User B visit following links by directly pasting them

https://staging.new.expensify.com/bank-account/new?policyID=<policy_id>
https://staging.new.expensify.com/workspace/<policy_id>/members
https://staging.new.expensify.com/workspace/<policy_id>/invite
https://staging.new.expensify.com/workspace/<policy_id>/invite-message

Verify if following message shows up -

[X] Verify that no errors appear in the JS console

Offline tests

QA Steps

[X] Verify that no errors appear in the JS console

PR Author Checklist

[X] I linked the correct issue in the ### Fixed Issues section above
[X] I wrote clear testing steps that cover the changes made in this PR

[X] I added steps for local testing in the Tests section
[X] I added steps for the expected offline behavior in the Offline steps section
[X] I added steps for Staging and/or Production testing in the QA steps section
[X] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[X] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[X] I included screenshots or videos for tests on all platforms
[] I ran the tests on all platforms & verified they passed on:

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[X] MacOS / Chrome / Safari
[X] MacOS / Desktop

[X] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[X] I followed proper code patterns (see Reviewing the code)

[X] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[X] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[X] I verified that comments were added to code that is not self explanatory
[X] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[X] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[X] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[X] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[X] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[X] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[X] I verified the JSDocs style guidelines (in STYLE.md) were followed

[X] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[X] I followed the guidelines as stated in the Review Guidelines
[X] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[X] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[X] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[X] I verified that if a function's arguments changed that all usages have also been updated correctly
[X] If a new component is created I verified that:

[X] A similar component doesn't exist in the codebase
[X] All props are defined accurately and each prop has a /** comment above it */
[X] The file is named correctly
[X] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[X] The only data being stored in the state is data necessary for rendering and nothing else
[X] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[X] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[X] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[X] All JSX used for rendering exists in the render method
[X] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[X] If any new file was added I verified that:

[X] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[X] If a new CSS style is added I verified that:

[X] A similar style doesn't already exist
[X] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[X] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[X] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[X] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[X] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[X] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[X] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

Mobile Web - Chrome

Mobile Web - Safari

Desktop

iOS

Android

Doing screen recording for the same.
@aimane-chnaif I don't have clue for testing on IOS and android app, can you please share how I can proceed with same

@aimane-chnaif , I have uploaded the screen recording for PR

@aimane-chnaif , let me know if there are any reviews. Thanks.

friendly bump @aimane-chnaif

@aimane-chnaif . Thank you for Heads up. I have uploaded the videos for them.

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/AndrewGable in version: 1.3.42-0 🚀

platform
result

🤖 android 🤖
cancelled 🔪

🖥 desktop 🖥
cancelled 🔪

🍎 iOS 🍎
cancelled 🔪

🕸 web 🕸
cancelled 🔪

🚀 Deployed to production by https://github.com/jasperhuangg in version: 1.3.42-26 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Fix breaking tests

Resolves #115 and #116.

Removed node 7 from the matrix!

Merged


code of conduct file

Hi @kahanikaar @SanchiMittal @07souravkunda
I found this repository is not having a code of conduct file ,
kindly let me know if can work on this issue
kindly assign me

#16

thank you @07souravkunda
kindly do add the DWOC and level tag also please since both issue and pull request needs to be added DWOC and level tag.


[mac] document and adjust to standards based timing concepts

Hi OpenThread community and above all @edmont , I'm heading over to you guys to adjust OpenThread to a recent unification of timing concepts in the Zephyr IEEE 802.15.4 implementation.
In a nutshell this small change set proposes to align some of the timing sensitive aspects of OpenThread even more closely with concepts from the underlying standards. Unification and closer specification of timing along original IEEE 802.15.4 concepts enables implementation of additional non-Thread IEEE 802.15.4 features in Zephyr (namely TSCH).
Most of this change set is documentation - only a small change is being made to the interpretation of the TX timestamp that seems not to be particularly restricted in OpenThread so far. I propose that we make use of this leeway to jointly move closer to standards-conforming timing primitives.
AFAIK, Zephyr and its derivatives and radio drivers (notably the nRF Connect SDK and nRF radio driver) are the only platforms that depend on this code and those have already made the corresponding changes on their side.
See https://github.com/zephyrproject-rtos/zephyr/issues/59245 for more context as well as @edmont 's regression test result with real hardware based on this change set.

Notice that apart from Zephyr there are some other platforms affected by this change

@edmont How should we go about these platforms?

Notice that apart from Zephyr there are some other platforms affected by this change

@edmont How should we go about these platforms?

Lets allow @jwhui to ping relevant maintainers.

@hemanth-silabs / @suveshpratapa - could you two review this PR, thanks.

I believe I fixed the prettify problem - not sure how to fix the version problem - could you point me to some docs that give me a hint?

I believe I fixed the prettify problem - not sure how to fix the version problem - could you point me to some docs that give me a hint?

You simply need to increment the value relative to the existing value on main.

You simply need to increment the value relative to the existing value on main.

Ok, read through the script and found out where it looks for the version. Then bumped OPENTHREAD_API_VERSION in include/openthread/instance.h

@jwhui - All checks pass now except for Simulation 1.1. But is that error really due to my change? It seems that the pending timestamp is missing from the TLV set in the child update request. I'd assume that this wouldn't be influenced by my change?

@fgrandel  I've re-triggered the checks. They pass now.

@Irving-cl @abtink @jwhui Do you think this is ready for merge now? The other requested reviewers seem to be missing in action. Please be aware that this PR blocks dependent issues in Zephyr.

@fgrandel Not missing in action - there wasn't any deadline requested I think ;)

@EskoDijk Yes, I'm not sure how typical expected response times are in the OT community - I assumed 2 weeks would be fine. Can you approve the PR from your side? I think I addressed all your concerns.

Sorry one question I forgot to ask earlier: why are the references made to the radio's local clock in the file time.h , otPlatTimeGet(), while there is also a time function otPlatRadioGetNow() more directly within radio.h ?
Shouldn't the function template in radio.h have similar restrictions as the one in time.h ? I wonder why exactly is the one in time.h also mentioning "radio" there because at first sight this seems to be only for a time service of the platform, independent of the radio functions.
In my simulation platform, the otPlatRadioGetNow() by the way internally just calls otPlatTimeGet(). So it could be that these are just used interchangably but I'm not sure how it was intended.

why are the references made to the radio's local clock in the file time.h , otPlatTimeGet(), while there is also a time function otPlatRadioGetNow() more directly within radio.h ?

@EskoDijk Good catch! Short answer: I just happened to stumble upon otPlatTimeGet() first because that's the only clock used in Zephyr's platform abstraction layer. otPlatRadioGetNow() is just an alias. But that is of course an oversight on my side. I'll fix that.
That being said, I believe that the OT clock architecture is not very stringent. If you check where the different clocks are used, I doubt the implementations work as independently as you suggest. The platform clock is used in several places where probably either the radio clock should be used instead or a synchronized version of the two would be needed. We're currently fixing this class of bugs in Zephyr which is where this PR originates from. But that is a big issue and definitely out of scope for this simple PR.

I'll fix that.

Fixed.

@fgrandel  I've created a branch here: https://github.com/EskoDijk/openthread/tree/pr-csl
with a few more edits and clarifications. You can use that in this PR if needed.
From what I can see, with the current code, it's not really PHY-independent in any case. (e.g. submac / csl tx scheduler). Though with the changes it's better prepared for future PHYs.

it's not really PHY-independent in any case.

True. But now it is a little more of an implementation issue and a little less of an API/design issue. This PR doesn't try to solve all problems. ;-)

@fgrandel I've created a branch here: https://github.com/EskoDijk/openthread/tree/pr-csl with a few more edits and clarifications. You can use that in this PR if needed.

Thank you so much for investing such a lot of effort. Very much appreciated. I need to run now but I'll have time later today to do a diff and integrate your suggestions.

@EskoDijk Thank you so much for investing such a lot of effort. Very much appreciated. I need to run now but I'll have time later today to do a diff and integrate your suggestions. As you can see your comments have made this a much better quality PR.

Just added a commit with some more updates; with a fix for the timestamp time reference in CSL debug log printing. There may be more coming; this was based on CSL testing with my platform which is not finished yet.

Just added a commit with some more updates; with a fix for the timestamp time reference in CSL debug log printing. There may be more coming; this was based on CSL testing with my platform which is not finished yet.

I probably need to recheck everything - still seeing some definition issues. The aCslSampleTime in otPlatRadioUpdateCslSampleTime() probably needs to reference just the start of the sample window, i.e. start of SHR. It's not a timestamp but just a time.
Also radio.hpp has very similar comments/parameters to radio.h , but aren't updated by this PR, maybe we should just leave those be for now. These don't interface to the platform directly.

I probably need to recheck everything - still seeing some definition issues.

@EskoDijk No problem - we've decoupled this PR from our internal issue by now so please take your time. I propose you recheck everything and once you're confident that you have completed, let me know and I'll review the changes in your own branch and integrate them. Thorough review is good.

@fgrandel Ok, I'm done now with review and testing on the simulation platform.
For convenience I've also created a PR here https://github.com/fgrandel/openthread/pull/1  to track the differences between your code and my updates on it.
Main change is to document that otPlatRadioUpdateCslSampleTime() uses the (now default) timestamp reference of end of SFD. This is based on how the existing code used this timestamp, and on how mCslSampleTime is used in sub_mac. Probably here using the default time reference helps to keep the code understandable.
Note that the SHR time duration was already part of the "Ahead" time component of the CSL sample window; this is now better documented.

Main change is to document that otPlatRadioUpdateCslSampleTime() uses the (now default) timestamp reference of end of SFD. This is based on how the existing code used this timestamp, and on how mCslSampleTime is used in sub_mac. Probably here using the default time reference helps to keep the code understandable.

To be honest, I don't like that change. The spec clearly defines CSL timing in terms of the start of MAC header, see my explanation in https://github.com/openthread/openthread/pull/9322#discussion_r1291207899 about the difference between low level PHY-independent timestamps (i.e. SFD/RMARKER timestamping) as opposed to the special case of CSL (plus the lengthy inline documentation of Thread 1.3.1 wrt CSL timing and the derivation of corresponding timestamps introduced in this PR).
I would therefore not introduce any change to the definition of MAC-level CSL timing in this PR, it's out of scope. So if at all it should be changed then this should be kept in a separate PR. It's not what I intended in this PR. This PR is exclusively about introducing SFD-based timestamps where the standard does not explicitly deviate from them.

@EskoDijk Oh, and of course I have nothing against closing this PR in favor of another PR that you base on top of mine with additional changes and then merge that one. I just don't have time to go that deep into CSL timing specific to OT. It's much beyond "scratching my own itch".

@fgrandel

To be honest, I don't like that change. The spec clearly defines CSL timing in terms of the start of MAC header

Right, overlooked that part. This is indeed the reference point in the spec for CSL. Let's use that! I updated my platform code to it and it seems to work ok.

@Irving-cl @abtink @jwhui Ready for merge?

@hemanth-silabs / @suveshpratapa - could you two review this PR, thanks.

@suveshpratapa / @hemanth-silabs - just reminding you of this PR that needs to be reviewed.  Thanks.


Vault: asset() & total_assets()

Motivation

Rename Vault -> vault
Use a config time constant to

Return the ContractId of the managed asset
Return the balance of the managed asset

Closed as this app is no longer being worked on


Unable to remove boundaryGap in time axis

Version: 3.2.2
I can't find a way to remove boundaryGap when xAxis is type of time. Here is an example:

Notice that when zoom is changed, beginning of the line doesn't start at the beginning of the chart.
I've tried boundaryGap:["0%","0%"] and other combinations but nothing seems to work.
If I use 'category' type, it works as expected:

Real-world example:

Is this something that can be done with numeric or time types?
Thanks as usually!

sample code:

boundaryGapTime.zip

It's not because of the boundaryGap. Same issue with https://github.com/ecomfe/echarts/issues/3637.

Oh, didn't notice that one, thanks for linking it.


New onboarding flow

Flows
That have been worked through (these may change depending on what is is technically possible)
https://www.figma.com/file/mhImzKiTrvaMpSm1lQ5KcL/New-Onboarding-Flow?node-id=0%3A1
Screens and components
(these are all the screens used throughput the various flows and their different interaction states)
https://www.figma.com/file/mhImzKiTrvaMpSm1lQ5KcL/New-Onboarding-Flow?node-id=71%3A20259
Prototype showing some of the interactions within the flows
https://www.figma.com/proto/mhImzKiTrvaMpSm1lQ5KcL/New-Onboarding-Flow?node-id=30%3A2749&viewport=1173%2C-926%2C0.20209833979606628&scaling=min-zoom
Reach out to me if anything doesn't seem right / has been missed

this is testable on https://staging.augur.net/#!/markets @matt-bullock


Unity Issue when Closing Game

I've noticed that when I quit to desktop, a Unity error is thrown (Unity 2019.4.37f1_019e31cfdb15).
I've tried closing the game while Seamstress is disabled and this doesn't happen in that scenario.

Devotion issue. Happens on more than just Seamstress


Rename language client names

@karthiknadig This change will only be visible if the language client crashes, right?

This will be visible anywhere VS Code wants to talk about that particular language server. Like in logs, error messages, notifications, etc.

It should be a safe change though, since I don't think we rely on that name to perform any logic?

Yes, it should be safe. Just to double check make sure that name is not used as an id anywhere.