BindingsBuilder.put()     This method is executed many times

Flutter SDK 2.2.1
get: 4.1.4
BindingsBuilder.put()
This method is executed many times
Entering or exiting HTTP has been increasing and changing
     GetPage(
       name: login,
       page: () => LoginPage(),
       binding: BindingsBuilder<LoginLogic>.put(
         (){
           print("double put - LoginLogic");
           return LoginLogic();
         },
       ),
       transition: Transition.fade,
     )

     class LoginLogic extends GetxController {
       final HttpTool _http = Get.put(
         HttpTool(),
         tag: "-${Tool.getUUId()}",
       );
     }

     static String getUUId() {
         return Uuid().v1().replaceAll("-", "");
     }

[GETX] REPLACE ROUTE /
[GETX] NEW ROUTE /login
[GETX] Instance "HttpTool" has been created with tag "-54a0dd00c50c11eba12b8105d51a53ce"
[GETX] Instance "HttpTool" with tag "-54a0dd00c50c11eba12b8105d51a53ce" has been initialized
[GETX] Instance "LoginLogic" has been created
[GETX] Instance "LoginLogic" has been initialized
[GETX] Instance "HttpTool" has been created with tag "-54c13640c50c11eba12b8105d51a53ce"
[GETX] Instance "HttpTool" with tag "-54c13640c50c11eba12b8105d51a53ce" has been initialized
@jasonlaw
Check the above log to add repeatedly

BindingsBuilder.put
This method is executed twice

@jonataslaw

If BindingsBuilder.put() executed many times, why didn't see the log print("double put - LoginLogic");?

[GETX] REPLACE ROUTE /
[GETX] NEW ROUTE /login
[GETX] Instance "HttpTool" has been created with tag "-54a0dd00c50c11eba12b8105d51a53ce"
[GETX] Instance "HttpTool" with tag "-54a0dd00c50c11eba12b8105d51a53ce" has been initialized
[GETX] Instance "LoginLogic" has been created
[GETX] Instance "LoginLogic" has been initialized
[GETX] Instance "HttpTool" has been created with tag "-54c13640c50c11eba12b8105d51a53ce"
[GETX] Instance "HttpTool" with tag "-54c13640c50c11eba12b8105d51a53ce" has been initialized
Httptool is created repeatedly

getx_examples.zip
@jonataslaw @KevinZhang19870314


No WPT scores shown for content-visibility and view transitions

https://webstatus.dev/features/view-transitions shows numbers in the graph, but no scores in the summary boxes. Also no scores in https://webstatus.dev/?q=view-transitions.
https://wpt.fyi/results/?label=master&label=experimental&aligned&q=feature%3Aview-transitions works.
@jcscottiii can you check what's happening here?

It looks like exactly the same thing is happening with https://webstatus.dev/features/content-visibility.

Notes for myself:
view-transitions data:
{
  "baseline": {
    "status": "limited"
  },
  "browser_implementations": {
    "chrome": {
      "date": "2023-03-07",
      "status": "available"
    },
    "edge": {
      "date": "2023-03-13",
      "status": "available"
    }
  },
  "feature_id": "view-transitions",
  "name": "View transitions",
  "spec": {
    "links": [
      {
        "link": "https://drafts.csswg.org/css-view-transitions-1/"
      }
    ]
  },
  "wpt": {
    "experimental": {
      "chrome": {
        "score": 0.948051948
      },
      "edge": {
        "score": 0.866133866
      },
      "firefox": {
        "score": 0.385614386
      },
      "safari": {
        "score": 0
      }
    },
    "stable": {
      "firefox": {
        "score": 0.385614386
      },
      "safari": {
        "score": 0.37962038
      }
    }
  }
}

content-visibility data:
{
  "baseline": {
    "status": "limited"
  },
  "browser_implementations": {
    "chrome": {
      "date": "2022-11-29",
      "status": "available"
    },
    "edge": {
      "date": "2022-12-05",
      "status": "available"
    }
  },
  "feature_id": "content-visibility",
  "name": "content-visibility",
  "spec": {
    "links": [
      {
        "link": "https://drafts.csswg.org/css-contain-2/#content-visibility"
      }
    ]
  },
  "wpt": {
    "experimental": {
      "chrome": {
        "score": 0.969339623
      },
      "edge": {
        "score": 0.952830189
      },
      "firefox": {
        "score": 0.964622642
      },
      "safari": {
        "score": 0.95754717
      }
    },
    "stable": {
      "firefox": {
        "score": 0.950471698
      },
      "safari": {
        "score": 0.382075472
      }
    }
  }
}

In both cases, Firefox and Safari do not show up as intended because they currently do not have a browser implementation status.
Edge and Chrome do not show up because their latest runs did not succeed.
For view-transitions, here's the crash.
For content-visibility, here's the crash
Currently, we disregard the results for subtest count if there's a subtest with a crash. This is because the number of subtests can be wrong.
The endpoint that returns the score for the summary boxes looks at the latest run. Not the latest run that did not crash (which could be N runs before)

Now that the browsers with implementation statuses have appeared (Edge and Chrome), we see the info for them. We continue to not show the other browsers because their implementation status is missing.
I will close this issue.
Let me know if we should:

Improve the UX to let users know that the score isn't showing that reason (create a new issue)
Store the latest non crashed feature metrics to prevent the need to go back N runs to find it. (comment on #382 as that could influence that implementation)


is there any way to make this use ElevenLabs?

I have been wondering lol

make a library in src/modules that would use eleven's api and link it into tts.py
if you want, i can submit a PR for this, not too difficult.


sortLayoutItemsByRowCol returning different results in Edge/IE 11-10

Hello,
First off, I'd like to say thanks for creating such a great library! It really has been quite fun to work with!
I noticed that I was getting inconsistent ordering of layout items across browsers, particularly in Edge/IE (11-10) vs Chrome/Firefox. After a couple of hours of debugging, I think I tracked down where the discrepancy is happening. It looks like the callback passed to sort() in sortLayoutItemsByRowCol() is returning different results based on browser. I recreated the problem here:
https://codepen.io/guntherjh/pen/qaKLNY
If you open the pen in Chrome vs Edge/IE 11-10, you'll see that the resulting array contains items in different order depending on browser.
Once again, thanks for all your hard work in creating a such a great library and please let me know if you need any further details from me.

Very interesting! I believe the issue is the lack of an equality case.
This adjustment appears to produce consistent results:
https://codepen.io/anon/pen/bwKOJB

I can confirm this fixes the issue in IE10. Do you have other VMs or machines handy so you could check in IE11/Edge?

I am actually on a Windows machine. I confirmed your modification fixes the issue in Edge and IE 11. Thanks for the quick response!

Great. I'll merge it in.

Awesome! Thanks again for your help!


Add real content to all published projects prior to launch v1

I did not realize we were all assigned to this I thought it was just me😭... I just pushed a lot of content updates up in a0908097547bd4e598790095fd1758f5aa6d2e6b. I would consider the content deployable but some pages could still use some love. @connerk -> 3d printing @micacardillo -> mutant toys

This is a stale issue, and not specific, and thus hard to ever close. (AND logic for ALL project sites?) never gonna happen. 🗡
additional information requests to a published project site warrant their own explicit issue.


chore: remove telcoautomation module

see b/302153641#comment5

Did you manually remove it or use git revert?

It was a manual removal, the revert option in the original PR didn't work


Edit image like community with popup

This PR includes:

A popup to edit the image
Changing image from popup changes the image everywhere without reload
Changing image from form changes the image everywhere without reload

@mghazanfar also after update, I am getting error message. Top navigation User menu part not updating avatar, only after reload of page, maybe trigger re-render on navbar or user menu component for it to fetch new image?

@mghazanfar also after update, I am getting error message. Top navigation User menu part not updating avatar, only after reload of page, maybe trigger re-render on navbar or user menu component for it to fetch new image?

On settings page, it changes as both the images. But when we upload image on profile page, data does not update immediately in the avatar inside navbar but it makes a call to getAccounts which changes picture in navbar after a few moments.


CTRL+F is not showing Find & Replace PopUp

Reproducible in vscode.dev or in VS Code Desktop?

[X] Not reproducible in vscode.dev or VS Code Desktop

Reproducible in the monaco editor playground?

[ ] Not reproducible in the monaco editor playground

Monaco Editor Playground Link
No response
Monaco Editor Playground Code
No response
Reproduction Steps
No response
Actual (Problematic) Behavior
When I click on CTRL+F Sometimes it is not showing the Popup Like This

It is throwing error like

It is throwing warning like

When clicking on ALT + W it is showing the below error

Even after reloading the page as well I am getting this type of errors
Expected Behavior
No response
Additional Context
No response

This seems to be like an VS Code issue. Please create a bug in the vscode repository.
Can you share a screenrecording of what you did before?
Without steps to reproduce it, it will be very difficult to debug.


Migrate optparse to argparse.

@alexanderrichards from the meeting, I understood you're still going to add 4 more files to this PR, isn't it?

yes that's right its not finished yet

test this please

test this please

Superseeded by #8849


fix addon name typo in confidential-nodes-aks-overview.md

Making spelling and capitalization for confcom addon consistent across other azure-docs articles like this and this

#sign-off


Make verification faster

Reduces yarn install times by running only once for each dep and symlinking instead of running O(n) yarn install operations

@KevinGrandon Those should be fixed here: https://github.com/fusionjs/fusion-plugin-error-handling/pull/110


MacOS build error: cannot assign value of type 'CGFloat' to type 'Double'

It's possible this is our fault, on newest Flutter we are getting a reminded to upgrade to XCode 13 or above, so I might try that on our build machine to see if it helps?
💪 Building with sound null safety 💪

Running pod install...                                           1,616ms
Building macOS application...
/Users/Dan/development/flutter/.pub-cache/hosted/pub.dartlang.org/window_manager-0.1.4/macos/Classes/WindowManager.swift:379:45: error: cannot assign value of type 'CGFloat' to type 'Double'
            progressIndicator.doubleValue = progress
                                            ^~~~~~~~
                                            Double( )
** BUILD FAILED **

Build process failed

Yep, this was an xcode 12 problem, upgrading to xcode 13 fixed it

still have this problem


NIST Support

Description:
NIST format has the concepts of activities which are typically a full frame action that is happening.  Some of the specifications I have looked at also include a concept of Objects, which these are individual persons/things being tracks at a bounding box level during an Activity.
Link to specification: https://actev.nist.gov/pub/Phase3_ActEV_2021_SDL_EvaluationPlan_20210803.pdf
Notes:

Since activities are temporal and not spatial I opted to have them align themselves on the left side of the screen with space between concurrent activities.
NIST and our track files are both JSON so similar to the kwcoco on the web side I need to load the file up initiall if it is JSON and do some validation to make sure it is a NIST file and not a track.json file.
I added support for exporting and importing NIST files into the diveCLI.
Added some testing procedures for my current implementation of the NIST format.

fixes #862

We may need some discussion about the fullFrameBounds and if it should always read the media to figure out the video size or if I should modify the metadata to store this information.  This is a thing unique to NIST in that there are activities (tracks) not associated with any bounds so I need to generate them.

I sure do love making things harder than they need to be :)

It wasn't you, I've been kind of half jumping back and forth on this and I have this internal idea of how TS should work and it doesn't.  There should really be an exception to any in the linter for type guards.  I just looked at any as being a forbidden thing and got tunneled into my very bad logic there.

There should really be an exception to any in the linter for type guards.

Yes, there should be.


The format about output.

In the .info.full.txt, the format over each line is #line #topic id (prob, word)... . But what is the meaning of topic id? Where can I find the corresponding topic word? I try to find it in the vocabulary with the id as the line number. But failed. What's more, given a document, how to get its topics?
I am sorry for these trivial questions. Thank you firstly.

The topic id is the id of the topic. You can find the corresponding topic in the corresponding line in .model, which is a row of counts, each of which is the number of occurrences of a word in this topic.
You can get its topics from the file .z.estimate or .z.inference, depending on whether the document is from the training or testing set. These files contains the topic assignments for each word in the documents, and you can just count them if you want the document-topic counts. If you want the document-topic distribution, you can calculate it as
theta_k = (C_k + \alpha) / (\sum_k C_k + K\alpha)
where \theta is the distribution, C_k is the count, and \alpha is the hyper-parameter.

Thanks.
But I am still confused. For example, the topic id of the first line in the train.info.full.txt is 4132, but the line 4132 of train.model is 1    49:1. What's the meaning of it?
Thank you very much.

It means the topic on line 4132 has only one word, word #49, assigned to it once.
Notice however, the topic id may start from 0 (I forgot whether it starts from 0 or 1) and there is a leading line in train.model. You may wish to inspect line 4134 in train.model instead.

I see train.model has a more leading line than .vocab. Do them have the same index? For example, the line 4134 in the train.model corresponds to the word of line 4133 in .vocab?
In my understanding, I first find the topic id in train.info.full.txt, and then find the corresponding topic in the corresponding line(the topic id +2 or +1?) in .model. Have I understood correctly?

Sorry, I forgot train.model is a vocabulary size * number of topics sparse matrix.
So, you should look at the 4132-th column in train.model instead of the 4132-th row. Each row in train.model is a word instead of a topic.

But I only have 100 topics. So the number of columns of train.model is less than or equal to 100. I can't get the 4132-th column?


Chore: Setup angular framework

Description
Install angular framework and dependencies
Fixes #(issue number 1)
Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added necessary inline code documentation
[ ] I have added tests that prove my fix is effective and that this feature works
[ ] New and existing unit tests pass locally with my changes

Closing this request as there are a lot of approved request of this type


[ListItem] Support text prop

would it make sense to support a text prop for ListItem?
Should it replace or complement the existing solution?
Pros:

It reduce the amount of typing:

<ListItem text='mytext1'/>
<ListItem text='mytext2'/>

instead of:
<ListItem>mytext1</ListItem>
<ListItem>mytext2</ListItem>

It improves nested list:

<ListItem text='mytext'>
  ... subitems
</ListItem>

instead of
<ListItem>
  'mytext'
  ... subitems
</ListItem>

@cgestes I like the idea of adding a text property, it compliments the secondaryText property already present in the list-item and cleans up the construction of nested list-items.

I think this makes a lot of sense. It will affect the new MenuItems as well.
Do you guys think we should call the prop primaryText? Also, it will be handled like secondaryText where you can pass in a string or a node.

I like primaryText.

I like primaryText

Hi I am passing in a text property of primaryText but it is not appearing. Any ideas?

@georgerichmond I am running into the same issue -- I am only able to render text using secondaryText attribute.


In sc.pl.tracksplot, groupby is not worked

[x] I have checked that this issue has not already been reported.
[x] I have confirmed this bug exists on the latest version of scanpy.
[x] (optional) I have confirmed this bug exists on the master branch of scanpy.

Note: Please read this guide detailing how to provide the necessary information for us to reproduce your bug.
Minimal code sample (that we can copy&paste without having any data)
ax = sc.pl.tracksplot(adata, markers, groupby = "seurat_clusters")
adata.obs["seurat_clusters"].dtype.name != 'category'

---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-33-097156e92bcf> in <module>
----> 1 ax = sc.pl.tracksplot(adata, markers, groupby = "seurat_clusters")

D:\python\empty for python\lib\site-packages\scanpy\plotting\_anndata.py in tracksplot(adata, var_names, groupby, use_raw, log, dendrogram, gene_symbols, var_group_positions, var_group_labels, layer, show, save, figsize, **kwds)
   1304     if groupby not in adata.obs_keys() or adata.obs[groupby].dtype.name != 'category':
   1305         raise ValueError(
-> 1306             'groupby has to be a valid categorical observation. '
   1307             f'Given value: {groupby}, valid categorical observations: '
   1308             f'{[x for x in adata.obs_keys() if adata.obs[x].dtype.name == "category"]}'

ValueError: groupby has to be a valid categorical observation. Given value: seurat_clusters, valid categorical observations: []

Ture

Versions

[Paste the output of scanpy.logging.print_versions() leaving a blank line after the details tag]

when I detect the with adata.obs["seurat_clusters"].dtype.name != 'category'.  the resault  shows Ture

Hi @john-jiangyong,
The error is that your covariate you group by should be a categorical, while it is not at the moment. You could run:
adata.obs[‘seurat_clusters’] = adata.obs[‘seurat_clusters’].astype(‘category’) To turn the covariate into a categorical.
Note the code above might not be 100% correct as i’m typing from my phone and haven’t verified.


[INTEL MKL] Fix memory leak issue in MklFusedBatchNorm

fix the memory leak bug in mkl batch norm, free the temp weight buffer used for MKL-DNN weight memory primitive.

@harshini-gadige can you take a look at this PR? the one failure //tensorflow/c/eager:c_api_test_gpu seems to be unrelated and is failing in other PRs also.

@harshini-gadige can you take a look at this PR? the one failure //tensorflow/c/eager:c_api_test_gpu seems to be unrelated and is failing in other PRs also.

Sure. Will look into this.

@harshini-gadige The failures are not related to this PR,  other PRs are also failing the same tests. Can you take a look at it again? Thanks.

@harshini-gadige The failures are not related to this PR, other PRs are also failing the same tests. Can you take a look at it again? Thanks.

Looking into this. It takes time and I'll keep posted. Thanks !

@tatianashp  Can I bypass these 2 windows bazel failures and proceed with "Ready to pull"  ?

@tatianashp Can I bypass these 2 windows bazel failures and proceed with "Ready to pull" ?

As per the response I got from the chat, proceeding to get this merged.


Norton LifeLock AV False Positive

It analyzes the file, and removes it.

There’s not a whole lot I can do about this unfortunately.
The program technically exhibits the patterns of a Trojan because it installs and runs Roblox Studio’s exe, but it isn’t done in a malicious manner.

Oh thanks!
On Tue, Nov 3, 2020 at 11:46 AM Max G<EMAIL_ADDRESS>wrote:

Closed #73
https://github.com/CloneTrooper1019/Roblox-Studio-Mod-Manager/issues/73.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CloneTrooper1019/Roblox-Studio-Mod-Manager/issues/73#event-3954096156,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARJLWOJOT6O2PYYFORC7E63SOA6WVANCNFSM4TIWPUJQ
.


python312Packages.azure-mgmt-cosmosdb: 9.5.1 -> 9.6.0

Changelog: https://github.com/Azure/azure-sdk-for-python/blob/azure-mgmt-cosmosdb_9.6.0/sdk/cosmos/azure-mgmt-cosmosdb/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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/)
24.11 Release Notes (or backporting 23.11 and 24.05 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.

Add a :+1: reaction to pull requests you find important.

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

8 packages built:

azure-cli
azure-cli.dist
prowler
prowler.dist
python311Packages.azure-mgmt-cosmosdb
python311Packages.azure-mgmt-cosmosdb.dist
python312Packages.azure-mgmt-cosmosdb
python312Packages.azure-mgmt-cosmosdb.dist


Str.java "rsplit" method

Implementation of rsplit method for strings, test cases included.

@hcheng761 You can run these lint checks locally so you can put them all into one commit, instead of removing once trailing space per commit. Run ant checkstyle to get all the problems in one report.

Thanks for contributing, @hcheng761 ! :+1:


instrumentation know how and span json - user guide update

Proposed changes
Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request. If it fixes a bug or resolves a feature request, be sure to link to that issue.
Types of changes
What types of changes does your code introduce to this project?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x ] Documentation Update (if none of the other choices apply)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x ] I have read the CONTRIBUTING doc
[ ] I have signed the CLA
[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[x ] I have added the necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

@oi-raanne Contents look fine to me. Thanks for putting this together. Got a few minor suggestions on the format and some additional description. Instead of adding those as comments and more iterations, I'll create a new pull request for those. Thanks


Pull to a draft/published post

Search query interface should include 2 links buttons to pull down the PMP content:

[x] Create DRAFT post
[x] PUBLISH a post

Yep.
Also see #24.


Update Base files

Configured Enter, Home, Info, Explain total 4 routes.

Looks Good

Nice work. 😃


fetcher.fetchGuild does not work.

const guild = await bot.helpers.getGuild(id); throws error.
Error: [403] The Authorization token you passed did not have permission to the resource.
{"message":"Missing Access","code":50001}

This is a ss of index.ts.
The link will take you to that line.

That's not answering my question though.

Are you sure that you are actually providing the id of a guild which your bot is in?

This is happening inside the message transformer.
I am not doing anything.

#16 Explains this issue.


BREAK and END are not configurable.

The README says that the BREAK and END keywords are customizable, yet they are not.

I just tried changing it the settings, to no avail, unfortunately. Would be a great feature to have, since I'll be writing tasks in different languages :)
Love the plugin btw

Closing this as stale.


Update to use the simplified JSON building process on resin-odroid

This implements resin-os/resin-yocto-scripts#31
Signed-off-by: Eugene Mirotin<EMAIL_ADDRESS>@agherzan @floion

SGTM

LGTM


[New Badge Request] NextJS

First of all, congratulations and thanks for this awesome project, I will now be using these amazing badges for my projects.

Badge name - NextJS
Category - Skills

Added Next.js Badge 👍


good, when I try to start hubot tells me the following erorr:

D:\documentos\mio\proyectoRobot\hubot-adry\bin>hubot
up to date in 0.704s
The system cannot find the path specified.I'm trying to use it on windows, could you help me

I'll be closing this for now


evm2wasm specific EEI methods

This topic has been touched upon a few times, but never discussed in actual depth.
Not after doing some protoyping with runevm it seems the following features are missing:

getTxGasLimit
getCallDepth (EIP3 proposes this)
getCallType / inStaticMode

Obviously all this is needed if gas calculation is done by evm2wasm as opposed to be part of the EEI.

Another method required for calculating gas as part of runevm is isAccountEmpty(address), which is used in CALL (and for future constantinople support in EXTCODEHASH). Is there any objections to adding this method? otherwise I'll make a PR.


documentation

is it possible to post a documentation or an example how to use the bundle? How to create payments, the forms etc?

I've just created a demo application based on the Symfony Standard Edition. Check it out here: https://github.com/sparklingapp/AdyenBundleExample


API Key

Hey, @dev-aniketj Your Project is really lovely I'm using this project for my college project the problem is that when I use the API that I created using my email id is not working but when I use your API which is being Commanded just above API_KEY variable.
Could you please let me know what's the reason for that?

Please use this link to get the data - https://api.openweathermap.org/data/2.5/onecall?exclude=minutely&lat=26.9124&lon=75.7873&appid=API_KEY
(API_KEY replace with your api key)
Sometimes it takes 2-4 days for new api, please generate another one if the previous is not working, and wait atleast 2-3 hours to get service started.
@souravsk

okay Thanks, @dev-aniketj just one more thing by any chance do you have documentation for this project? Like DFD, ER, etc

No I did not create one, because all the instructions are already given on the website Open Weather Map
Documentation for the Weather App Link - Official Docs


Not working on mobile.

Hi, thanks for the great work!
The example is not working on mobile, is it because mobile browser do not support this functionality?
I tested on Mozilla, Chrome and Safari on iOS.

Hi, I tried to reproduce on those clients. iOS Safari seems to be working for me 😄.
In iOS Firefox and iOS Chrome the demo is indeed broken. I'm having trouble getting logs from those applications. Were you able to get some logs from one of those?
I tried getting Chrome DevTools remote debugging going, but no success.

So I investigated over the weekend and navigator.mediaDevices.getUserMedia doesn't exist in those browsers, so the library can't work there. I added an error message to the demo to make the reason clear.

Added the error message to the demo in d0f9390. I think that resolves this issue, let me know if something else needs fixing 👍 .


support infinite scroll for meteorListView/meteorComplexListView

is it possible to support infinite scroll for meteorListView/meteorComplexListView by default?

For now I can use this code with react-native-infinite-scroll-view, but support for this will be very appreciated. I think it will be cool and very useful for real world apps
I'll provide a simple example for someone who also need this:
npm i --save react-native-infinite-scroll-view

...
import Meteor, {connectMeteor, MeteorListView} from 'react-native-meteor';
import InfiniteScrollView from 'react-native-infinite-scroll-view';

@connectMeteor
class Transactions extends Component{

constructor(){
    super();
    this.limit = 30;
    this.state = {
      limit: this.limit,
      canLoadMore: true
    }
  }

getOptions(){
    console.log(this.state.limit)

    return {
      sort: {updatedAt: -1},
      limit: this.state.limit,
    }
  }

  getMeteorData(){
   Meteor.subscribe('birojasa_transaction', {}, this.getOptions());
}

handleLoadMoreAsync(){
    this.setState({
      limit: this.state.limit + this.limit,
      canLoadMore: false
    });

    //this is a hack (not recommended but working for now)
    setTimeout(()=>{
      this.setState({
        canLoadMore: true
      });
    }, 2000);
  }

render(){
 return (
<MeteorListView
        renderScrollComponent={props => <InfiniteScrollView {...props} />}
        collection="transactions"
        selector={{}}
        options={this.getOptions()}
        renderRow={this._renderRow.bind(this)}
        canLoadMore={this.state.canLoadMore}
        onLoadMoreAsync={this.handleLoadMoreAsync.bind(this)}
      />
)
}

}

Yes it would be really appreciated !
But I have no time for this right now.
PR would be welcome ;)

@radiegtya can you share the MeteorListView code that implemented with infinite-scroll-view?

Any updated on it? This feature is a must!


Safari extension port

First of all, hanks for this amazing extension !
I've managed to create a safari port with the following Apple Developper command :
xcrun safari-web-extension-converter ./build

The extension seems to work fine on iOS, iPadOS macOS; If you want I can create a pull request with the added Xcode Project files
Best regards,
Frédéric Duppel

Whoa so cool I can put this on my phone now. Had no idea this was so easy

Very cool. There I found a step-by-step of how to enable that extension in Safari. 🙏 https://www.vshsolutions.com/blogs/convert-chrome-extension-to-safari-web-extension/

Created Pull request #75 for Xcode source files & Safari macOS app

Thx @fred-corp for suggesting this, and good to hear you enjoy the extension!
The PR worries me a bit and I'm not sure what to do with it yet. It adds/changes a whopping 82 files, including all sorts of binaries, duplications of other files in the repo, and other files.
What also doesn't help: I don't currently have access to a Mac (only an iPad) so no good way to maintain an Apple-only part of the code-base.
On the other hand it is super cool that it works across Apple devices including tablet and smartphones.
Would you be okay to let the PR and issue sit for (quite) some time? As announced in #69 my personal and work schedule around this December is absolutely insane, and I have no real time to do anything besides crucial fixes.
Regardless: thanks for sharing! And: happy puzzling!

Hi @jeroenheijmans
Thanks for checking out the PR :)
I completely understand that the way I added the code is not optimal; I did it this way so other people could maybe install the extension on their device more easily.
I'll try to experiment with file linking in Xcode, and maybe try to find a way to automate the app building with GitHub workflows


Update share-links.rst: typo

Summary
Typo.
Ticket Link
No need…

Hello @grinapo,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.


View not displaying on iOS 15

I'm trying to run it on iOS 15 and it isn't displaying. Any clues of what maybe going on?

如果你使用的是场景，在`    func sceneDidBecomeActive(_: UIScene) {
// Called when the scene has moved from an inactive state to an active state.
// Use this method to restart any tasks that were paused (or not yet started) when the scene was inactive.
    #if DEBUG
        PerformanceMonitor.shared().start()
    #endif
}`中里添加，因为PerformanceView文件中的func keyWindowScene()过滤的activationState是foregroundActive

Hi,
any fix for this I facing the same

Hi, I am having the same issue here

I don't think there is any support for this Library anymore, no update in 2 years

It works for me. Inside AppDelegate, just use:
    func applicationDidBecomeActive(_ application: UIApplication) {
        PerformanceMonitor.shared().start()
    }

You guys need to uncomment / remove the code snippet in PerformanceView line: 81 if you do not use WindowScene .
I commented it and fixed
/*
if #available(iOS 13, *) {
self.windowScene = PerformanceView.keyWindowScene()
}
*/


Use HTTP persistent connections for API requests

Implemented the use of HTTP persistent connections for API requests by default (refer issue #20)

Requests to Gateway will now re-use persistent http connections which reduces load and increases response time
Default connection poolmaxsize=10 to align with Session object defaults. Note: pool use applies to multi-threaded use of pyPowerwall only, e.g. as with the pyPowerwall Proxy Server
Added env PW_POOL_MAXSIZE to proxy server to allow this to be controlled (persistent connections disabled if set to zero)
Added env PW_TIMEOUT to proxy server to also allow timeout on requests to be adjusted
For release prep, pyPowerwall version changed to v0.6.0 and proxy version to t17 (release notes still required)

Thank you @mcbirse !  This is a great improvement with elegant approach and code.  Happy to merge!


Document creating ConsumerGroup with multiple consumers

I haven't found a simple way to init a ConsumerGroup with multiple consumers.

Is it necessary to define and pass a whole CustomPartitionAssignmentProtocol through the protocol param?
Is it any sample code for that?

Environment

Node version: 7.10.1
Kafka-node version: 2.2.3

Thanks in advance.

Just start another instances of the ConsumerGroup with the same group ID. The kafka broker will automatically coordinate between the multiple instances and balance partition ownership among your instances.
It's not required to define a custom assignment protocol. We use the built ins roundrobin protocol.


This is my first commit ever!! :)

i  am just learning so plz ignore this  :)

Well, we wished we could ignore it, but you just notified 1.600+ GitHub accounts about this pull request.
Please use private repos for learning git and other stuff, just don't open randomly new pull requests on OSS public projects.


Always convert lifetimes to static

Fixes a bug where kernels with a lifetime parameter, but no type parameters, would not have lifetimes converted to static. This caused a compilation error for those kernels.

I've added a test for this case in https://github.com/juntyr/rust-cuda/commit/40e02ed0106117440691108471591dc8b01f2e2f


[BUG] Unable to create backups after snapshot is generated

Hi!
I'm having issues with the backup system. I've just found a way to make an error message disappear (the one described in issue #4187, btw redeploying everything after having removed the backup target from the config fixed it up, thanks for the tip!) and another problem came up.
Bug description
When clicking "create backup" in the volume management UI, a snapshot is created and nothing happens. Through the rancher UI, I tried to manually run the cronjob for a scheduled backup, the job takes forever to complete because for every available volume there is the following error in the pod log:
failed to complete backupAndCleanup for volume-name: timeout waiting for the backup.
No backup is shown in the longhorn UI.
To Reproduce
In the rancher UI, use the button "run now" to start a scheduled backup job and check the log.
Log or Support bundle
The log of the pod running the cronjob shows this error multiple times:
level=error msg="failed to run job for volume" concurrent=1 error="failed to complete backupAndCleanup for volume-name: timeout waiting for the backup of the snapshot sched-ba-16b52a09-ce71-429b-9ee5-618087750787 of volume volume-name to start" groups=default job=sched-backup labels="{\"RecurringJob\":\"sched-backup\"}" retain=30 task=backup volume=volume-name.
Edit: I've just sent an email to<EMAIL_ADDRESS>with the support bundle attached.
Environment

Longhorn version: v1.3.0
Installation method (e.g. Rancher Catalog App/Helm/Kubectl): kubectl
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version: k3s version v1.22.7+k3s1

Number of management node in the cluster: 2
Number of worker node in the cluster: 4

Node config

OS type and version: debian bullseye 11.4, kernel release 5.10.0-16-amd64
CPU per node: 2 vCPU
Memory per node: 4096MB on the control nodes, 5120MB on the worker nodes
Disk type(e.g. SSD/NVMe): SSD
Network bandwidth between the nodes: 1Gbps

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal): Proxmox hypervisor
Number of Longhorn volumes in the cluster: 15

Additional context
Previously, when running longhorn version 1.2.4, backups were working flawlessly; issues appeared after updating to v1.3.0.
By manually opening a shell inside a longhorn-manager pod, I can successfully create files and folders in the nfs share through the mount in /var/lib/longhorn-backupstore-mounts/; I'd say this is not an issue related to NFS permissions.

Can you provide more details about the reproducing step...?

Yes, sure.
Through the longorn gui no error is shown. I have a recurring job called sched-backup that should be executed during the night, in the morning no backup is created.
I tried to manually backup a volume by clicking on the "create backup" button the gui: the snapshot is correctly generated but no backup is shown in the volume details and in the backups tab.
To further troubleshoot, I opened the rancher ui, selected the longhorn namespace and looked at the cronjobs. The cronjob called sched-backup is show and has the correct cron expression. The cronjob has a green status, as if everything worked.
I tried to manually perform the backup by clicking on "run now" in the options of the cronjob: a pod is created to handle the job and I checked the logs of that pod.
The pods takes forever to complete the job, after the completion the job is marked as "successful" and rancher reports no errors. By opening the log of the pod I see multiple times the error explained in the previous comment.
I think the pod's log should be in the support bundle, I've generated the archive immediately after the manually started job finished. If needed I can provide you the full log.
Let me know if there is something you need me to provide you or if there are some tests to be performed.

I tried to manually perform the backup by clicking on "run now" in the options of the cronjob: a pod is created to handle the job and I checked the logs of that pod.
What's this? Rancher UI? I don't see this option on the job page.

there may be something wrong with backup creation init. Need to check the support bundle for more details.

I sent you the support bundle as soon as I opend the issue. If the email to<EMAIL_ADDRESS>did not came through, let me know how can I provide it to you.

I saw tons of the logs like the followings:
2022-08-07T15:44:51.547638859+02:00 time="2022-08-07T13:44:51Z" level=warning msg="Cannot take snapshot backup" backup=backup-989d2d761c4844c3 controller=longhorn-backup error="could not find snapshot 'sched-ba-353e2804-4e70-48b3-8f39-a91d375c366e' to backup, engine 'unifi-controller-vol-e-36c0bc0d'" node=nem-03
2022-08-07T15:44:51.547722181+02:00 time="2022-08-07T13:44:51Z" level=warning msg="Error syncing Longhorn backup longhorn-system/backup-989d2d761c4844c3" controller=longhorn-backup error="longhorn-backup: fail to sync backup longhorn-system/backup-989d2d761c4844c3: could not find snapshot 'sched-ba-353e2804-4e70-48b3-8f39-a91d375c366e' to backup, engine 'unifi-controller-vol-e-36c0bc0d'" node=nem-03

This means the volume snapshots are removed before the backup creation is done or the backup is updated...

Can you try to remove all stuck backups then see if the backup creation works?

Hi,
thanks for you time.
How can find stuck backups?
In the rancher UI I do not have running jobs, everything is marked as successful.

You can check the backup state in GUI or kubectl. If a backup is an old one and the state is not Completed, you can consider it as a stuck backup.

I am observing the same behavior as the OP on 1.3.1: fixed mount path issue according to #4187, but then observe these events during backups:
31m         Warning   SnapshotError      snapshot/snapshot-f7362672-09d8-490b-a954-b4dc09ed03c1        lost track of the corresponding snapshot info inside volume engine 31m         Warning   SnapshotUpdate     snapshot/snapshot-f7362672-09d8-490b-a954-b4dc09ed03c1        snapshot becomes not ready to use
Backup jobs eventually say they have completed (and I can see evidence of -snapshots- being created/kept at those timeswhen viewing individual volumes in the UI). However, the Backups tab shows no information, and there are no mention of -backups- for the individual volumes.
Not sure if it is relevant, but it seems like everything is listed as a Snapshot in the UI and never a Backup? I have a separate deployment of 1.3.1 where Backups are working, but none are listed as Backup in the UI there either...

ched-ba-353e2804-4e70-4

@jlphillipsphd Can you also send us the support bundle for analysis? Thank you.

I have some updates, I think this issue is related to node reboots.
I've updated to v1.3.1 and backups were working, then I had to reboot to apply some kernel updates and the problem appeared again.
These are the details of what I've done:

updated longhorn following the instructions (in my case, I just run a kubectl command), after this I manually run a cronjob from the rancher UI and backups worked;
applied updates to the OS using apt (without rebooting), after this step the recurring job was automatically run and backups were performed correctly.

I had to reboot half of the machines to apply kernel updates, I followed this procedure:

in the longhorn UI, disabled node scheduling and requested eviction for the corresponding nodes
waited for each volume to be moved, no issues here
drained the nodes using kubectl
waited for kubectl to complete the drain procedure, in the longhorn namespace every pod was moved correctly
rebooted the OS
uncordoned the nodes

At this point, backups stopped working (I tried to manually run the cronjob from the rancher UI) and the same exact problem appeared (errors in the log included).
Then I enabled node scheduling and disabled eviction in the longhorn UI and backups still don't work.
I tried to re-schedule everything in the longhorn namespace multiple times but it did not fix the issue. I even tried to re-update longhorn (running the kubectl command again) but nothing was done by kubernetes and nothing changed.

@derekbit I will look into getting the bundle, but need to explore it for concerns about sensitive data...
I hadn't realized it might have been related to node maintenance instead of the update itself... I also went through a maintenance cycle (for each node: cordon, drain appropriately, update, reboot, uncordon) shortly after the update from 1.2.4 to 1.3.1.

@derekbit I will look into getting the bundle, but need to explore it for concerns about sensitive data...

You can check the error during back and its associated snapshot.
Sounds the snapshot is somehow deleted before doing backup.

Didn't know which component(s) to examine in detail, but here are some additional details and a work-around that I found for my situation...
Went into the UI, selected a volume, and "Create Backup". A snapshot gets created, but not a backup. Here's what I observed...

In the instance manager log I see:
time="2022-08-18T13:58:39Z" level=warning msg="Snapshot backup timeout" backup=backup-b66f715210e84c74 controller=longhorn-backup error="proxyServer=<IP_ADDRESS>:8501 destination=<IP_ADDRESS>:10001: failed to backup snapshot e2ff9529-06c1-4d72-a215-c9b26b90c59b to backup-b66f715210e84c74: rpc error: code = Unknown desc = failed to create backup to nfs://nfs.longhorn-backup.svc.cluster.local:/ for volume pvc-63cfac91-a2d6-40f4-9a31-57184571f2bf: rpc error: code = DeadlineExceeded desc = context deadline exceeded" node=[redacted]
Notice that I am using k8s to manage the NFS server for the backups (there is an FQDN for the NFS server that I am using). I modified the backup target to use the IP address of the NFS service instead of the FQDN of the service. I don't know why I had to change this since it was working in 1.2.4, but I guess some component is trying to access the NFS and can't use the DNS properly (if containers try to use internal DNS resolution during start-up I know this fails because DNS is only available -inside- of the containers).
So, after switching from the FQDN of the service to the IP address of the service, everything is working as intended.
I know this doesn't help actually identify the issue as to why the FQDN approach isn't working (but, maybe someone can chime in who knows when the DNS resolution is being used which might result in this failure). I am hoping I can put together a test case using a development deployment soon, but I need to leave my production deployments alone for now since it's working for me in this situation. It would be ideal to use the FQDN of the service though in case I ever need to remake the service for some reason and get a new IP address from k8s.

Hello, it seems I also have the issue "snapshot becomes not ready to use" when they are created.
I don't know if I can be helpfull, but I'll be happy to help if needed.
For me it's when I create a shapshot that I get this error not a backup.
I have setup a backup every night and lunch time and a few snapshot during the day

So this just happend to me to after upgrading longhorn from 102.3.1+up1.5.3 > 103.3.1+up1.6.2 using the rancher UI.
Turns out I had an S3 backup target set via helm values when I first installed longhorn, which I changed via the longhorn UI after the installation.
But since I didn't change it in the values of the helm chart, it got restored to the old value when I upgraded longhorn and backups then "stopped" working.
Well, not really stopped: The snapshot is working fine and then it just runs into a timeout because the backup target doesn't exist. I guess this could be communicated more clearly in the UI.
Restoring the backup target to the previous setting got things working again.


⚠ Samsung has degraded performance

In f7bb5e9, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2238 ms

Resolved: Samsung performance has improved in 33dff8d.


Send multiple messages to multiple topics from one element of a Spark collection

not really motivated by a use case


Button link broken

https://learn.microsoft.com/en-us/azure/azure-arc/resource-bridge/network-requirements
#PingMSFTDocs

Azure documentation issue guidance
Thanks for opening an issue in the Azure technical documentation repository.
We use GitHub issues as the primary channel for customer and community feedback about the Azure documentation.
Creating an issue
We prefer that you create documentation feedback issues using the Feedback link on the published article - the feedback control on the doc page creates an issue that contains all the article details so you can focus on the feedback part.
You can also create a feedback issue here in the repo. If you do this, please make sure your issue lists:

[ ] The relevant Azure service or technology.
[ ] A link to the published documentation article that you have feedback about.
[ ] Clear, specific feedback that the author can act on.

Pull requests and article contributions
If you know the change that is needed in an article, we encourage you to submit the changes directly using a pull request. If the change is large, or if you want to contribute an entire article, follow these guidelines:

[ ] Don't surprise us with a big pull request or a pull request with a new article! Submit an issue that describes the details of the proposed large change or new article.
[ ] Include the service or technology area.

We'll route the issue to the appropriate content team for review and discussion.
Tech support and product feedback
If you would like to contact Microsoft about other things, such as product feedback or tech support, please review these guidelines:

If you need technical support using Azure, the paid and free support options are described here: https://azure.microsoft.com/support/options/.

Each article in the Azure technical documentation contains a product feedback button - it's best to submit product feedback directly from a relevant article. Otherwise, you can submit product feedback for most Azure products in the following product feedback forum: https://feedback.azure.com/d365community/forum/79b1327d-d925-ec11-b6e6-000d3a4f06a4.

@hyoshioka0128
Thanks for your feedback! We will investigate and update as appropriate.
I have verified that the URL in my previous query was operational. Try once more, please.

@SaibabaBalapur-MSFT  - Thanks.
The button worked with Edge and Chrome, but not with FireFox 113.0.1 (64-bit).

@hyoshioka0128
Thanks for your contribution. Please add your feedback in below link, so our production team can review it and update the same.
feedback · Community (azure.com)


[5.7] IRGen: Add a frontend option to force single LLVM module emission in multithreaded mode

This allows to experiment with single module LLVM emission without
having to change drivers that expect multiple output files.
rdar://94744623
(cherry picked from commit 821ba4707979019247b4c7114072ac1eba705b85)

@swift-ci test

main PR: https://github.com/apple/swift/pull/59344


Feature request - Gradle Properties options

To pass Gradle property as argument we have 2 choices:

Use env syntax and Gradle convention for environment variables:

 uses: eskatos/gradle-command-action@v1
      with:
         env:
           ORG_GRADLE_PROJECT_someSecret: 1234
         arguments: build

The issue of this approach is that it doesn't support properties with dots (very common practice, because of properties file)

Pass property as command-line argument:

 uses: eskatos/gradle-command-action@v1
    with:
        arguments: build -PsomeSecret=1234

Works fine until you have 1-2 property, but becomes issue if you have many of them, multiline support would help but it also have issues in YML, requires special syntax and not very convenient to use, it also easy to forgot slash in command
So I suggest adding support for gradle properties in yml syntax:
 uses: eskatos/gradle-command-action@v1
      with:
         properties:
            someSecret: 1234
         arguments: build

Tho, there is an issue with it, because key with dot requires escaping in YML:
some.secret should be escaped like:
"[some.secret]"
Still fine, but not perfect. There is another option. Support dot as yml separator:
properties:
  some:
     secret: 1234

For 1 simple property it may be overkill and escape will work just fine
But check this real-life example:
 - name: Build
      uses: eskatos/gradle-command-action@v1
      with:
        arguments: build --info -PsomeService.prod.apiKey=${{ secrets.SOME_SERVICE_API_KEY }} -PsomeService.prod.secret=${{ secrets.SOME_SERVICE_API_SECRET }} -PsomeService.prod.projectId=${{ secrets.SOME_SERVICE_PROJECT_ID }}

As you see, now it's not easy to follow what is going on. it will be better with nested properties syntax:
 - name: Build
      uses: eskatos/gradle-command-action@v1
      with:
        arguments: build
        properties:
           someService:
               prod:
                  apiKey: ${{ secrets.SOME_SERVICE_API_KEY }}
                  secret: ${{ secrets.SOME_SERVICE_API_SECRET }}
                  projectId: ${{ secrets.SOME_SERVICE_PROJECT_ID }}

So much better even if debug and dev environments are configured
Maybe it should be considered to do the same for system properties, which are also very common
If you think it reasonable, I may try to implement it and contribute

Hi @gildor,
Sorry for the late response.
I can see how this can be annoying. But then you would have to deal with more YAML, I'm not sure I'm fond of it. It's a tradeoff between long arguments and 'complex' YAML.
I'm not against such an addition and agree that system properties should be implemented too if we go down that route.
The behaviour when one sets the same property in arguments and properties should be clarified though.

Thanks for reply

The behaviour when one sets the same property in arguments and properties should be clarified though.
Agree, probably properties should be added after build arguments, it makes implementation simple (no need to parse and alter arguments somehow) and allows debug what kind properties are passed to gradle in build logs (because properties would be converted to arguments)

t's a tradeoff between long arguments and 'complex' YAML.
It's true, not a huge fan of more yaml, but we have what we have, and more readable workflow file is with a clear separation between tasks arguments and build properties makes sense here

Agree, probably properties should be added after build arguments, it makes implementation simple (no need to parse and alter arguments somehow) and allows debug what kind properties are passed to gradle in build logs (because properties would be converted to arguments)

Converting the properties to arguments sounds good.
About precedence, I guess the choice is between prepending or appending to arguments.
IOW, yaml properties have precedence, or arguments have.
I guess the former, appending, is reasonable.

Yes, appending as simple and straight forward solution so properties have precedence

One of the issues is GitHub actions runner does not provide access to YAML configuration. It supports literal values only, and even YAML lists are not supported yet.
On the other hand, the following might be not that bad (I'm not sure if multi-line arguments already work, but it can definitely be supported):
 - name: Build
      uses: eskatos/gradle-command-action@v1
      with:
        arguments: |
          build --info
          -PsomeService.prod.apiKey=${{ secrets.SOME_SERVICE_API_KEY }}
          -PsomeService.prod.secret=${{ secrets.SOME_SERVICE_API_SECRET }}
          -PsomeService.prod.projectId=${{ secrets.SOME_SERVICE_PROJECT_ID }}

or
 - name: Build
      uses: eskatos/gradle-command-action@v1
      with:
        arguments: build --info
        properties: |
          someService.prod.apiKey=${{ secrets.SOME_SERVICE_API_KEY }}
          someService.prod.secret=${{ secrets.SOME_SERVICE_API_SECRET }}
          someService.prod.projectId=${{ secrets.SOME_SERVICE_PROJECT_ID }}

@vlsi I really like your idea.
@gildor I've opened #88 to add support for multi-line arguments, and will close this issue in preference to that solution.
Please add a comment if you feel a structured properties approach has some clear advantages.
Thanks!


mina client create-token --sender PUBKEY crashes the daemon

Preliminary Checks

[X] This issue is not a duplicate. Before opening a new issue, please search existing issues: https://github.com/MinaProtocol/mina/issues
[X] This issue is not a question, feature request, RFC, or anything other than a bug report. Please post those things in GitHub Discussions: https://github.com/MinaProtocol/mina/discussions

Description
Attempting to create-token crashes the daemon with the following error
2023-05-06 20:15:46 UTC [Fatal] Unhandled top-level exception: $exn
Generating crash report
        exn: {
  "commit_id": "e976627007d99415932392a84983de4e4a8db38f",
  "sexp": [
    "monitor.ml.Error",
    [
      "Failure",
      "Tokens disabled. Read (Create_new_token\n ((token_owner_pk B62qk8XYSEM6uNwkmMhPY8onoC4oy8qDpX19zyxfsQpEuRsyacg9YMv)\n  (disable_new_accounts false)))"
    ],
    [
      "Raised at Stdlib.failwith in file \"stdlib.ml\", line 29, characters 17-33",
      "Called from Bin_prot__Utils.Make_binable_gen.bin_size_t in file \"src/utils.ml\", line 91, characters 38-54",
      "Called from Mina_base__Signed_command_payload.Body.Stable.V1.With_version.bin_size_t in file \"src/lib/mina_base/signed_command_payload.ml\", line 244, characters 6-381",
      "Called from Mina_base__Signed_command_payload.Poly.Stable.V1.bin_size_t in file \"src/lib/mina_base/signed_command_payload.ml\", line 402, characters 6-127",
      "Called from Mina_base__Signed_command_payload.Poly.Stable.V1.With_version.bin_size_typ in file \"src/lib/mina_base/signed_command_payload.ml\" (inlined), line 402, characters 6-127",
      "Called from Mina_base__Signed_command_payload.Poly.Stable.V1.With_version.bin_size_t in file \"src/lib/mina_base/signed_command_payload.ml\", line 402, characters 6-127",
      "Called from Mina_base__Signed_command_payload.Stable.V1.With_version.bin_size_t in file \"src/lib/mina_base/signed_command_payload.ml\", line 416, characters 4-120",
      "Called from Mina_base__Signed_command.Poly.Stable.V1.bin_size_t in file \"src/lib/mina_base/signed_command.ml\", line 13, characters 6-165",
      "Called from Mina_base__Signed_command.Poly.Stable.V1.With_version.bin_size_typ in file \"src/lib/mina_base/signed_command.ml\" (inlined), line 13, characters 6-165",
      "Called from Mina_base__Signed_command.Poly.Stable.V1.With_version.bin_size_t in file \"src/lib/mina_base/signed_command.ml\", line 13, characters 6-165",
      "Called from Mina_base__Signed_command.Stable.V1.With_version.bin_size_t in file \"src/lib/mina_base/signed_command.ml\", line 23, characters 4-171",
      "Called from Core_kernel__Binable0.to_bigstring in file \"src/binable0.ml\", line 179, characters 17-31",
      "Called from Core_kernel__Binable.to_string in file \"src/binable.ml\", line 7, characters 40-58",
      "Called from Codable.Make_base58_check.to_base58_check in file \"src/lib/codable/codable.ml\", line 90, characters 46-78",
      "Called from Base__Fn.compose in file \"src/fn.ml\" (inlined), line 25, characters 22-27",
      "Called from Mina_base__Transaction_hash.hash_command in file \"src/lib/mina_base/transaction_hash.ml\", line 37, characters 19-72",
      "Called from Mina_base__Transaction_hash.User_command.create in file \"src/lib/mina_base/transaction_hash.ml\", line 111, characters 59-73",
      "Called from Network_pool__Transaction_pool.Make0.Resource_pool.Diff.apply.go in file \"src/lib/network_pool/transaction_pool.ml\", line 1020, characters 28-67",
      "Called from Base__Result.map in file \"src/result.ml\", line 122, characters 19-24",
      "Called from Async_kernel__Deferred1.M.map.(fun) in file \"src/deferred1.ml\", line 17, characters 40-45",
      "Called from Async_kernel__Job_queue.run_job in file \"src/job_queue.ml\" (inlined), line 128, characters 2-5",
      "Called from Async_kernel__Job_queue.run_jobs in file \"src/job_queue.ml\", line 169, characters 6-47",
      "Caught by monitor coda"
    ]
  ],
  "backtrace": [
    "Raised at Stdlib.failwith in file \"stdlib.ml\", line 29, characters 17-33",
    "Called from Bin_prot__Utils.Make_binable_gen.bin_size_t in file \"src/utils.ml\", line 91, characters 38-54",
    "Called from Mina_base__Signed_command_payload.Body.Stable.V1.With_version.bin_size_t in file \"src/lib/mina_base/signed_command_payload.ml\", line 244, characters 6-381",
    "Called from Mina_base__Signed_command_payload.Poly.Stable.V1.bin_size_t in file \"src/lib/mina_base/signed_command_payload.ml\", line 402, characters 6-127",
    "Called from Mina_base__Signed_command_payload.Poly.Stable.V1.With_version.bin_size_typ in file \"src/lib/mina_base/signed_command_payload.ml\" (inlined), line 402, characters 6-127",
    "Called from Mina_base__Signed_command_payload.Poly.Stable.V1.With_version.bin_size_t in file \"src/lib/mina_base/signed_command_payload.ml\", line 402, characters 6-127",
    "Called from Mina_base__Signed_command_payload.Stable.V1.With_version.bin_size_t in file \"src/lib/mina_base/signed_command_payload.ml\", line 416, characters 4-120",
    "Called from Mina_base__Signed_command.Poly.Stable.V1.bin_size_t in file \"src/lib/mina_base/signed_command.ml\", line 13, characters 6-165",
    "Called from Mina_base__Signed_command.Poly.Stable.V1.With_version.bin_size_typ in file \"src/lib/mina_base/signed_command.ml\" (inlined), line 13, characters 6-165",
    "Called from Mina_base__Signed_command.Poly.Stable.V1.With_version.bin_size_t in file \"src/lib/mina_base/signed_command.ml\", line 13, characters 6-165",
    "Called from Mina_base__Signed_command.Stable.V1.With_version.bin_size_t in file \"src/lib/mina_base/signed_command.ml\", line 23, characters 4-171",
    "Called from Core_kernel__Binable0.to_bigstring in file \"src/binable0.ml\", line 179, characters 17-31",
    "Called from Core_kernel__Binable.to_string in file \"src/binable.ml\", line 7, characters 40-58",
    "Called from Codable.Make_base58_check.to_base58_check in file \"src/lib/codable/codable.ml\", line 90, characters 46-78",
    "Called from Base__Fn.compose in file \"src/fn.ml\" (inlined), line 25, characters 22-27",
    "Called from Mina_base__Transaction_hash.hash_command in file \"src/lib/mina_base/transaction_hash.ml\", line 37, characters 19-72",
    "Called from Mina_base__Transaction_hash.User_command.create in file \"src/lib/mina_base/transaction_hash.ml\", line 111, characters 59-73",
    "Called from Network_pool__Transaction_pool.Make0.Resource_pool.Diff.apply.go in file \"src/lib/network_pool/transaction_pool.ml\", line 1020, characters 28-67",
    "Called from Base__Result.map in file \"src/result.ml\", line 122, characters 19-24",
    "Called from Async_kernel__Deferred1.M.map.(fun) in file \"src/deferred1.ml\", line 17, characters 40-45",
    "Called from Async_kernel__Job_queue.run_job in file \"src/job_queue.ml\" (inlined), line 128, characters 2-5",
    "Called from Async_kernel__Job_queue.run_jobs in file \"src/job_queue.ml\", line 169, characters 6-47"
  ]
}
2023-05-06 20:15:46 UTC [Fatal] Prover terminated due to signal, terminating daemon
2023-05-06 20:15:46 UTC [Info] Running async shutdown handler: "Close transition frontier, if exists"
2023-05-06 20:15:46 UTC [Info] Running async shutdown handler: "Remove daemon lockfile"
2023-05-06 20:15:47 UTC [Fatal] libp2p_helper process died unexpectedly: "died after receiving sigkill (signal number 9)"
2023-05-06 20:15:47 UTC [Error] Encountered $error while asking libp2p_helper for peers
        error: {
  "commit_id": "e976627007d99415932392a84983de4e4a8db38f",
  "string": "libp2p_helper process died before answering"
}
2023-05-06 20:15:48 UTC [Fatal] Unhandled top-level exception: $exn
Generating crash report
        exn: {
  "commit_id": "e976627007d99415932392a84983de4e4a8db38f",
  "sexp": [
    "monitor.ml.Error",
    [ "Mina_net2__Libp2p_helper.Libp2p_helper_died_unexpectedly" ],
    [
      "Raised at Mina_net2__Libp2p_helper.handle_libp2p_helper_termination in file \"src/lib/mina_net2/libp2p_helper.ml\", line 144, characters 8-45",
      "Called from Async_kernel__Deferred0.bind.(fun) in file \"src/deferred0.ml\", line 54, characters 64-69",
      "Called from Async_kernel__Job_queue.run_job in file \"src/job_queue.ml\" (inlined), line 128, characters 2-5",
      "Called from Async_kernel__Job_queue.run_jobs in file \"src/job_queue.ml\", line 169, characters 6-47",
      "Caught by monitor at file \"src/lib/gossip_net/libp2p.ml\", line 234, characters 31-31"
    ]
  ],
  "backtrace": [
    "Raised by primitive operation at Core__Signal.send in file \"src/signal.ml\", line 162, characters 6-61"
  ]
}
2023-05-06 20:15:48 UTC [Error] Encountered $error while asking libp2p_helper for peers
        error: {
  "commit_id": "e976627007d99415932392a84983de4e4a8db38f",
  "string": "helper process already exited (doing RPC ListPeers)"
}
(monitor.ml.Error
 ("Merkle_mask.Masking_merkle_tree.Make(Inputs).Attached.Dangling_parent_reference(\"467d43a0-aadf-37c5-9a53-ee02ac662a18\", \"File \\\"src/lib/mina_lib/mina_lib.ml\\\", line 1396, characters 49-56\")")
 ("Raised at Merkle_mask__Masking_merkle_tree.Make.Attached.assert_is_attached in file \"src/lib/merkle_mask/masking_merkle_tree.ml\", line 118, characters 10-57"
  "Called from Merkle_mask__Masking_merkle_tree.Make.Attached.merkle_root in file \"src/lib/merkle_mask/masking_merkle_tree.ml\", line 266, characters 6-26"
  "Called from Mina_base__Ledger.Ledger_inner.merkle_root in file \"src/lib/mina_base/ledger.ml\" (inlined), line 259, characters 25-38"
  "Called from Mina_commands.get_status.(fun).active_status.(fun) in file \"src/lib/mina_commands/mina_commands.ml\", line 278, characters 6-31"
  "Called from Mina_commands.get_status.(fun).active_status in file \"src/lib/mina_commands/mina_commands.ml\", line 276, characters 4-1023"
  "Called from Async_kernel__Deferred1.M.map.(fun) in file \"src/deferred1.ml\", line 17, characters 40-45"
  "Called from Async_kernel__Job_queue.run_job in file \"src/job_queue.ml\" (inlined), line 128, characters 2-5"
  "Called from Async_kernel__Job_queue.run_jobs in file \"src/job_queue.ml\", line 169, characters 6-47"))

The issue was reported in #5724 and closed after a fix was introduced in #5743, yet I am observing the error today.
Inspecting src/lib/mina_base/signed_command_payload.ml at commit e976627007d99415932392a84983de4e4a8db38f, one sees that

So clearly this command will throw an exception and apparently it is not executed inside a try block which handles the exception, causing the daemon to immediately crash. Presumably a similar fate awaits the daemon when executing the create-token-account and mint-tokens commands.
Steps to Reproduce

Run the 1.4.0beta2 daemon and wait for it to sync.
Attach an account and unlock it, e.g. mina accounts create or an existing account (let's call it PUBKEY).
Make sure the account is funded with at least 0.25 MINA.
Do mina client create-token --sender PUBKEY

Expected Result
Create a new token or do nothing; at the very least, not crash the daemon.
Actual Result
Daemon immediately crashes 😢
How frequently do you see this issue?
Always
What is the impact of this issue on your ability to run a node?
Low
Status
Mina daemon status
-----------------------------------

Max observed block height:              249255
Max observed unvalidated block height:  0
Local uptime:                           5m37s
Chain id:                               5f704cc0c82e0ed70e873f0893d7e06f148524e3f0bdae2afb02e7819a0c24d1
Git SHA-1:                              e976627007d99415932392a84983de4e4a8db38f
Configuration directory:                /path/to/.mina-config
Peers:                                  25
User_commands sent:                     0
SNARK worker:                           None
SNARK work fee:                         100000000
Sync status:                            Bootstrap
Block producers running:                0
Coinbase receiver:                      Block producer
Consensus time now:                     epoch=52, slot=3571
Consensus mechanism:                    proof_of_stake
Consensus configuration:
        Delta:                     0
        k:                         290
        Slots per epoch:           7140
        Slot duration:             3m
        Epoch duration:            14d21h
        Chain start timestamp:     2021-03-17 00:00:00.000000Z
        Acceptable network delay:  3m

Addresses and ports:
        External IP:    MY_IPv4_ADDRESS
        Bind IP:        <IP_ADDRESS>
        Libp2p PeerID:  12D3KooWHn2eWhB6JYrWcKrCz5SxXpQVoziQTC8P46S7oB3ej4gV
        Libp2p port:    8302
        Client port:    8301

Metrics:
        block_production_delay:             7 (0 0 0 0 0 0 0)
        transaction_pool_diff_received:     8
        transaction_pool_diff_broadcasted:  0
        transactions_added_to_pool:         0
        transaction_pool_size:              0

Additional information
No response

Hey @Isaac-DeFrain thanks for posting it.
Yet it is not really an issue. Thing is that the actual logic was removed even before the mainnet launch but we removed that CLI sub-commands you've used only recently.
Closing it.

@shimkiv I don't understand. Are saying that the commands have been removed and a new mainnet release will be made that does not have these commands? Because as of today, the latest mainnet release (1.4.0beta2) has these commands. That's the only reason I opened the issue.

I'm talking about the implementation logic behind these sub-commands (removed long time ago) and leftovers in a form of these sub-commands (present for mainnet).
We are not going to prioritise this at least because:
a). I'm pretty sure that no one is really using these commands because mainnet does not provide any tokens capabilities. All tokens stuff will be available in a form of zkApps as part of future Berkeley Hard-Fork.
b). This issue does not really impact the work of the Daemon. Unless the Node Operators intentionally will call mentioned sub-commands.
c). All leftovers were removed in Berkeley-compatible branches.


Framerate Dramatically Limited by Frame Alignment and Point Cloud Calculation

In the main loop of RealSense2OpenPose3D.cpp, the lines...
fsAligned = align.process(fsAligned); //Align the depth to the color frame
rs2::frame depthAligned = fsAligned.get_depth_frame(); //Get the aligned depth frame

//Generate a point cloud and a vertex array from that to hand to "updateKeypoints()" so that it can find the world coordinates
rs2::pointcloud ptCld = rs2::pointcloud();
rs2::points depthPoints = ptCld.calculate(depthAligned);
depthVertices = depthPoints.get_vertices();
...account for about 45-60ms of delay when using the chrono library to do some speed testing. This limits the framerate to 12fps to be extra safe. If OpenPose were to run faster than RS2OP3D, then there is the possibility of losing real-time-ness.
I cannot see how to multi-thread this process to speed it up. Perhaps there is another way that these depth points in 3D space can be acquired instead of with this slow vector. Or maybe there are improvements that can be made to the SDK to reduce latency.
Any help on this would be appreciated.

After talking to a co-worker about this problem, it seems there might be a solution related to the rs2_deproject_pixel_to_point() function.
"Documentation" page: https://dev.intelrealsense.com/docs/projection-in-intel-realsense-sdk-20
Function definition: https://github.com/IntelRealSense/librealsense/blob/5e73f7bb906a3cbec8ae43e888f182cc56c18692/include/librealsense2/rsutil.h#L46

I was able to apply the above mentioned fix which resulted in a 2-3 times improvement. on my system, the keyframe generation can now run at 52-41fps up from 12-16fps before. So, OpenPose can now be run without the frame limit on so long as it runs slower than RS2OP3D.
Please update your versions to 1.2 if you have the old one!


Cassandra: dbeaver displays the day prior for all date columns in results.

Version 4.04. I query for the July 3 and Dbeaver displays July 2!  This does not happen with native cqlsh or Toad.
`CREATE KEYSPACE test WITH replication = {'class': 'SimpleStrategy', 'replication_factor': '1'} AND durable_writes = true;;
CREATE TYPE test.udt_citystate (
city text,
state text
);
CREATE TABLE test.people (
Name text,
date date,
citystate FROZEN<udt_citystate>,
PRIMARY KEY (Name, date));
insert into test.people (Name,date,citystate) values ('FRED','2017-07-03',{city:'Houston',state:'Texas'});
select * from test.people where name = 'FRED' and date = '2017-07-03';`

Also, please notice in the results where the complex user defined type is displayed as null.
Great tool.  I have been working around these little bugs but really liking it so far.

I can't reproduce both these issues.

What Cassandra server version do you have?

Any details?

Cassandra 3.0.9.  Sorry for the slow response.

Tested on Cassandra 2.2 and 3.10. Still can't reproduce.
Please recheck that you use 4.0.4 version (these bugs were fixed a few versions ago).
If you've upgraded to 4.0.4 make sure you removed previous version completely (do not extract zip archive over previous installation).

I did a fresh 64 bit install on a brand new windows 7 vm.   dbeaver 4.05 testing against cassandra 3.10.
Date is still off by one day.  I think it may have something to do with my date locale settings but they are just the default US ones.

Interesting.
Please post debug logs (https://github.com/serge-rider/dbeaver/wiki/Log-files) - maybe there are some errors/warnings there.

dbeaver-debug.txt

Is it possible its just an outdated driver?  I see dbeaver using DataStax Java Driver for Apache Cassandra [2.1.11]
According to this link that driver will produce bad metatdata on Cassandra 3.0.  Although its odd you don't see the same problem.
http://docs.datastax.com/en/developer/driver-matrix/doc/javaDrivers.html#java-drivers

I don't see anything suspicious in logs.
2.1.11 is version of DBeaver extension. DataStax Cassandra driver version is 3.0.5.
Changing locale doesn't affect anything.
So I still have no clue. I have almost the same environment.
BTW are complex types still broken in your case?

I replaced the datastax driver jar file with the 3.2 version and no change (no errors either).  I also changed the column name thinking the [date] name would be throwing it off but no luck their either.  I switched from grid to text view...nope.  When I have time i will dig out wireshark...
Good news is that the complex type issue is resolved.

I made a simpler test...

and ran wireshark during the select...

The highlighted hex is the actual date value returned.  \43c6 = 17350.  Adding those days to cassandras start date for that datatype (1970-01-01 + 17350) I get the correct date 2017-07-03.
Upshot is the date is crossing the network correctly.

Thanks for details.
Complex types: yes, there was an error in metadata reading. Actually you don't need to restart DBeaver - just refresh keyspace list.
Date issue: well, I think I'll have to sit in debugger for some time. This is not the first report about such problem.

I have a clue. Probably there is a bug in Cassandra LocalDate handling (which relies on local time zone).
Will try to fix this..

That sounds promising.  I would assume that people want to see the actual stored value and that local time zone translation should not be a default.

I've reproduced this with PST timezone.
With new fix it is working ok.

  It is now showing a month difference.

@revelation1 I'll take a look.

Thanks for report, indeed this was a bug in DBeaver.
I didn't remember this before. Perhaps it is a result of Cassandra driver upgrade in 4.1

This is MySQL-specific. In some cases recent MySQL driver (8.*) cant recognize server timezone properly and thus interprets timezones incorrectly.
Workaround: set driver property "serverTimezone" in driver  properties to a correct value. Valid timezone list: https://github.com/dbeaver/dbeaver/wiki/JDBC-Time-Zones
I think I'll add timezone selector in the main MySQL connection properties page.
Another workaround is to downgrade MySQL driver to 5.x version (note: it is not fully compatible with MySQL 8.* servers).

Hi Serge,
Thank you for the fast response and implementation in v5.1.2!
Unfortunately the issue persists even after changing the serverTimezone to UTC (through either the "Driver properties" or the new "Server Time Zone:" feature)
If you have any other advice on how to solve this I would really appreciate it!
DB timezone: UTC
MySQL version: 5.6.39-log
Driver: mysql:mysql-connector-java:RELEASE [5.1.44]
Thanks again!


Cop idea: prefer no parentheses: expect(subject).to eq(true) not expect(subject).to(eq(true))

Idea for cop suggestion:
Prefer expect(subject).to eq(true) not expect(subject).to(eq(true)). The same when not_to so expect(subject).not_to eq(true) not expect(subject).not_to(eq(true)).
There's no single example where second approach is being used in https://github.com/rubocop-hq/rspec-style-guide.

Great idea.
There's this in the Ruby Style Guide:

Always omit parentheses for
Methods that are part of an internal DSL (e.g., Rake, Rails, RSpec):
good
validates :name, presence: true

Maybe add a cop here, and eventually a generic part if it to RuboCop, so that rubocop-rails could benefit from it as well, by marking validates and other Rails-specific DSL methods as such to be avoided.
Can't think of legit exceptions to this rule, e.g. when parenthesis would make sense.

Can't think of legit exceptions to this rule, e.g. when parenthesis would make sense.

We just have to be careful of to in other contexts, notably the second to with this change matcher
expect { man.you_cant_handle_the_truth! }.to change { man.the_truth? }.from(true).to(false)

I actually fixed that with below custom config:
Style/MethodCallWithArgsParentheses:
  Enabled: true
  IgnoredMethods:
    # long list of other methods that are also better without parentheses
    - to
    - not_to


[FIXED] Code Completion/Highlighting doesn't work on Rust files created from the editor

UPDATE
Turns out I have to put mod (file name); in the main file... my bad
Lapce Version
v0.2.0
System information
Windows 10
Describe the bug
Creating a .rs file from the editor will disable code completion and highlighting in that file for some reason...
You would have to create it in vscode, quit then open lapce again.
Additional information

screenshot

Rust analyzer likes to complain if a file is not part of the module tree, but you should still see tree-sitted highlighting even so, I believe.
I actually can't reproduce your issue - I created a new file in a project, and I get highlighting just fine.

Maybe this issue was fixed after the 0.2.0 release. Can you please try a nightly build? You can download one from the newest (upmost) run from this list: https://github.com/lapce/lapce/actions/workflows/release.yml


Improve label binding

Closes #1943.
When trying to bind a label to content, we now traverse into SequenceElem's children and StyledElem's child. Also removed FieldAccessError::Internal to fix "this is a bug" errors on unlabelled content when trying to access the "label" field (see the discord for some discussion of this).
This all means that examples like the below (from #1943) will now work as "expected".
#let foo(x) = {
  show figure.where(kind: "foo"): it => {
    let counter = counter(figure.where(kind: "foo"))
    let number = counter.display(it.numbering)
    [#number #it.body]
  }

  figure(x, kind: "foo", supplement: "Foo", numbering: "1")
}

#foo[Hello World]<howdyy>

@howdyy

#let foo(x) = {
  context {
    // Reset counter if needed
  }
  figure(x, kind: "foo", supplement: "Foo", numbering: "1")
}

#foo[Hello World]<howdy>

@howdyy

Potential downside is that you can no longer attach a label to a SequenceElem or a StyledElem.

Potential downside is that you can no longer attach a label to a SequenceElem or a StyledElem.

Does this imply that [#figure(...) abc def g] <label> will label the figure rather than the combination of the figure and the text?

Does this imply that #[#figure(...) abc def g] <label> will label the figure rather than the combination of the figure and the text?

No, it will label the text g.

And what about #[abc #figure(...) def g] <label>?

And the same happens here, it will label the text g.
The search for content to label still seeks backwards within a sequence's children.

You're right, there is then now the issue with StyledElem when queried and placed again losing its styles. I think this could then be resolved by not peeking into StyledElem when labeling, and instead dealing with its child when referencing. Though this may not be ideal either if its child is a sequence, for example.
Sequences are definitely a lot trickier. I agree that there should be a way to choose whether to label the sequence vs an element inside of it. But I imagine any sort of user interface for this would become difficult in nested sequences. In my opinion, it's more helpful to traverse into a sequence by default when labeling, rather than labeling the sequence itself.

For styles, I can somewhat see this as it's less evident to a user that styles result in an element in the first place. For sequences, I can see it less because there are multiple elements and it's not clear to me why the last element is special compared to the others.

Let's close this for now and have more discussion first and then maybe make a change later. Thanks still!


[$500] Chat -  If there is a previous money request on chat, the last message is not visible in LHN

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

Version Number: 1.3.98-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Issue found when executing PR https://github.com/Expensify/App/pull/29939
Action Performed:
Pre-requisite: must be logged in and have a chat where you have previously requested money

Open a chat where you have a previous money request
Send any message (note that it appears in LHN)
Create a new money request
Disable internet connection
Delete the last money request

Expected Result:
The last sent message should be visible in LHN
Actual Result:
The previous money request is visible in LHN, not the last message
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[x] iOS: mWeb Safari
[x] Windows: Chrome
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/4e52ec0a-4fba-4090-9a63-2c6eee25dcab
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0170cc5bfea21c8189
Upwork Job ID:<PHONE_NUMBER>388102656
Last Price Increase: 2023-11-10

This seems to be working as expected.

@Christinadobrzyn Can we close this?

Awesome, thanks @s77rt and @tienifr! I'll close!


Change Label

https://github.com/atharmohammad/Code-N-Collab/blob/c7891023f767cf57309867f8f5f8ed824a022fe4/src/Components/Rooms/Rooms.js#L86
replace * Choose a Unique Name text with  * What you want to be called ?

I am willing to contribute in this issue!

I am willing to contribute in this issue!

Go ahead! But please make sure to make PR on develop branch


Add image of possible splitting modes.

https://github.com/dcelisgarza/DPhil_Thesis/blob/a3103d899af5aaff8ceec62333286bd0c791486e/src/Topology.tex#L49-L49

done ages ago


Add a Time type to Nmap::Command

Add a custom type to validate nmap time values (ex: 10, 10s, 10m, 10h, 10ms).

Implemented in 2bb0394.


Page 8614 Config. Package Card Actions do not trigger in Event Subscriber functions

When creating an event subscriber function to any of the Actions on Page 8614 - Config. Package Card, they do not trigger. I think the reason is that the Action Names property has not been changed from the default (where Action[xxx]) is assigned to the Action. I have found that the event subscriber functions only fire when the Action Name property is changed from the default:

If one looks at the actions in the subform page (8625), these actions names have been changed from the defaults, and the OnAction triggers all fire successfully from event subscribers based on these actions:

I hope this information is sufficient.

Hi @ClintonSaunders,
We have made some changes to how event subscribers work with actions a few months back.
What versions are you running?

Hi there,
Sorry, I am guilty of basing my issue on the version of AL that is running with the latest hotfix of NAV 2018. I will log the issue with the NAV 2018 team if it has already been fixed in the dev preview.
I will setup a Sandbox with the latest dev preview software and test my issues there first in future before logging them on the Github issue list.
Thanks for the prompt feedback..
Clinton
From: Thomas Pedersen<EMAIL_ADDRESS>Sent: Monday, 07 May 2018 09:02
To: Microsoft/AL<EMAIL_ADDRESS>Cc: Clinton Saunders<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Microsoft/AL] Page 8614 Config. Package Card Actions do not trigger in Event Subscriber functions (#2156)
Hi @ClintonSaundershttps://github.com/ClintonSaunders,
We have made some changes to how event subscribers work with actions a few months back.
What versions are you running?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/AL/issues/2156#issuecomment-386976281, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AdahyHH5poF3vLx6JW1cjtArPniEucM7ks5tv_FigaJpZM4T0bBU.

Since you haven't written back in some time, I'll close the issue. if you are still having an issue then feel free to open a new one.


ModalDropdown not redrawing if default index changes

I've been using Modal Dropdown, it has been working very well for me.  But I have one drop-down which sets the 'current month' for a calendar app.  Everything works great if I change the current month using the modal dropdown.  But there is another path for changing the month, and when this path is used, the value changes, the scene re-renders, but Modal Dropdown displays the old month.
I have the following two lines as part of the dropdown:
defaultIndex={this.state.curMonthIdx}
defaultValue={this.state.curMonthStr}
As I mentioned, other things in the scene rerender.  The Modal Dropdown displays the old data.  What should I do to cause the correct redraw?

@drdavej
the default value can not change dynamic.
Maybe this can help you understand what happened.
https://github.com/sohobloo/react-native-modal-dropdown/issues/43#issuecomment-307299640

OK, was able to track things down.  In my scene's constructor, added this:
constructor()
{...
this._monthDropdown = null;
...}
In the render, when I am creating the ModalDropdown, I have:
<ModalDropdown
...
ref={el => this._monthDropdown = el}
...>
Then in my other routine that can switch the month, I have this:
calendarGoForward()
{...
this._monthDropdown && this._monthDropdown.select(newVal);
...}
The trick was using the 'ref...' to save a reference to the ModalDropdown, so that later we could call its select method.


[FEATURE] Back item

Hi @lim_bo56
Could you add an item that allows you to close the gui or run a command so that it can go back to another menu which I create for example with deluxemenus.
An example:
back_item:
enabled: true
name: '&aBack'
material: ARROW
amount: 1
lore: false or (- '')
slot: 2
onClick:

profilegui

If there are multiple pages, can you make sure that this item is on all pages?

Hello @zArtemis-java. I'm pretty sure you can already create an item that runs a command. You could create an item that runs the command to open the other menu. Besides that, I think a close button will be a good addition as we currently don't have one.
Thank you for your suggestion. Next time make sure you use the feature request template.


Issues/1626 maven dep conflicts

GitHub issue resolved: #1626
Fixes #1600
Briefly describe the changes proposed in this PR:

Can we target this against the master branch instead?


 I spotted that some of them were relatively straightforward (albeit a bit technical) using some stuff already in the library.
This should typecheck when https://github.com/agda/cubical/pull/859 is merged.
Btw, a lot of the stuff in Cubical/Homotopy is (unfortunately for you...) written by me, so feel free to drop me a message if you have any questions about this part of the library.

I merged the PR over at agda/cubical now

Thanks! - merged now


Fixes #90

Fixes #90
Changes:

Changed the way offsets were calculated
Changed the way BgBounds were calculated

Screenshots for the change:

Properly name the PR

@iamareebjamal On it, Sir.

@iamareebjamal Done

Proper PR title would be fix: Bound calculation in Preview custom view


Async validation change

Validation could be called multiple times before the first validation resolves. Only the lates validation results should be applied to the validation model.

And maybe declaratively set an interval that invokes

Find a way to asynchronously & declaratively  debounce validation execution.

Merged.


No way to erase logs between restarts

Right now, if you restart the container and pull the log with the tool 'dry', you can see the whole log history since the container was created (not run / restarted), so the history can be massively long.
It would be good to have a way to clear the log when restarting the container.

Your question is pertaining to Docker, nothing to do with this git repo.
Found these on StackOverflow:
To delete the logs on a Docker for Linux install, you can run the following for a single container:

echo "" > $(docker inspect --format='{{.LogPath}}' <container_name_or_id>)

You can configure docker to automatically rotate logs with the following in an /etc/docker/daemon.json file:

{
"log-driver": "json-file",
"log-opts": {"max-size": "10m", "max-file": "3"}
}

@gerry, I thought that the container had to set up the log rotation, not realizing that it's better to let docker do it. thanks for the info, I'll close this thread.


support parser.javascript.exportsPresence and importExportsPresence

parser.javascript.exportsPresence
parser.javascript.importExportsPresence

https://github.com/webpack/webpack/blob/87660921808566ef3b8796f8df61bd79fc026108/lib/dependencies/HarmonyImportDependency.js#L125

webpack-test/cases/cjs-interop/non-existing-export

recover this warning file after support this feature https://github.com/web-infra-dev/rspack/pull/4858#discussion_r1419869788


Deleting Events

Having button on the page to delete past events

Created a function in main.js that deletes events. Waiting for designs for the delete button before resolving

Add delete function to past events @bmfahey


No connection to friendica account

I cannot connect to a Friendica account. What am I doing wrong?

Your site needs to have the Diaspora Protocol plugin/addon enabled (url admin/site or ask your administrator).
Your channel also needs to enable the Diaspora Protocol (url settings/featured)
Add the Friendica person in the text box at url /connections using<EMAIL_ADDRESS>syntax


feat(web-components): add internal prop to allow for validation without text or icon

Summary of the changes
To allow the use of text field in pagination bar, as it was designed, an internal prop has been introduced. This prop allows for inline validation on all validation statuses.
Related issue
#612
Checklist

[x] I have added relevant unit and visual regression tests.
[x] I have manually accessibility tested any changes, if relevant.
[x] I have ensured any changes match the Figma component library.

I think in the ticket it mentions adding an event which would expose the validation message that would've shown up so that pagination can access it. Please could you add this in?

This has now been added


alternate image view

The idea of this PR is get an better style in Image view on website.
The image is larger and other column values is reshaped into rows.
Also correct the link of images in thumbnails

Coverage remained the same at 72.785% when pulling 7101d70058bd2ec8b6aca187e17cf5b22317808e on babsey:image_view_info into 9341bf6f288c54433e1097c220c06db7ec8d47f1 on open-research:master.


add advanced editor feature on comments

Similar to the one we have on the publish page. This would go on file / channel pages

Seems like a good way to explore the code.  I'd like to try this if it's still wanted :)

Yes, that would be awesome, thank you!


icons command produces invalid yaml file

Version(s) affected
1.0.x
Description
As mentioned here: https://github.com/Spomky-Labs/pwa-bundle/discussions/100#discussioncomment-8693164
The command output should be verified
How to reproduce
Execute the command
Possible Solution
No response
Additional Context
No response

Actually, this is normal as the icons can be used in various sections.
I added a comment after the command is executed


[Bug] log4j2 grpc report missing traceId

Search before asking

[X] I had searched in the issues and found no similar issues.

Apache SkyWalking Component
Java Agent (apache/skywalking-java)
What happened
Added to the jvm parameter in the project.
-Dlog4j2.contextSelector=org.apache.logging.log4j.core.async.AsyncLoggerContextSelector
Traceid cannot be obtained as a result
What you expected to happen
-Dlog4j2.contextSelector=org.apache.logging.log4j.core.async.AsyncLoggerContextSelector
after join and report traceid
How to reproduce
jvm param add
-Dlog4j2.contextSelector=org.apache.logging.log4j.core.async.AsyncLoggerContextSelector
Anything else
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Are you going to submit a fix?

yes, i try.

I would like to move this into the discussion panel if you have done. Issue panel for SkyWalking usually tracks things as release blocks or has owners fix them soon.


https://raw.github.com/alcatraz/alcatraz-packages/master/packages.json has error

{
"name": "SwiftJsonToObj",
"url": "https://github.com/aotian16/SwiftJsonToObj",
"description": "json to object plugin",
"https://github.com/aotian16/SwiftJsonToObj/raw/master/screenshot/SwiftJsonToObj.png"
},
{
"name": "Xmas",
"url": "https://github.com/onmyway133/Xmas",
"description": "It's beginning to look a lot like Christmas",
"screenshot": "https://github.com/onmyway133/Xmas/raw/master/Screenshots/xmas.png"
},

Created a pull request to fix this https://github.com/alcatraz/alcatraz-packages/pull/444

Thanks!


Add rootFolderNames & rootFolderNamesExpanded iconThemes definitions for root folders

We need to custom iconThemes of the workspace's root folders, we want to custom different iconThemes for each root folders according to the root folder's name:

fixed by https://github.com/microsoft/vscode/pull/195319

To verify:

create a new file icon theme. You can use https://github.com/microsoft/vscode/tree/main/extensions/theme-defaults/fileicons as an example
add properties rootFolderNames & rootFolderNamesExpanded

Gave it a good crack, but unable to get things working.
Hopefully someone else will have better luck

Got it to work. Needed something like this in an icon theme file, in which fakeproject would be the name of a root folder:
	"rootFolderNamesExpanded": {
		"fakeproject": "_file_light"
	},
	"rootFolderNames": {
		"fakeproject": "_folder_open_light"
	},

@aeschli was suggesting to just edit the extensions/theme-defaults/fileicons/vs_minimal-icon-theme.json in our repo, so one could run out of sources, pick the Minimal icon theme and see the changes.

suggesting to just edit the extensions/theme-defaults/fileicons/vs_minimal-icon-theme.json in our repo,

Thats the only difference, I created a new extension.
Rest was the same.
Though its weird it didn't work, should have.

Oh, sorry, this should have been a real test plan item with complexity 3.
I came up with this repo to test it: https://github.com/aeschli/icon-theme-test2
It implements an icon theme with the new feature

clone the repo, opn in VS Code
run the launch config. This should open on a test workspace with root folders and also select the theme
Verify that the root folder called rootFolder1has a different icon thanrootFolder2` (it's just some random image)


Config( triage ): Add "6.0" to "wontfix_verions" list.

Version 6.0 has finished it's life cycle.
@wuhuizuo @heipark

/cc @heipark

/ok-to-test

/merge


Auto create all relevant run configurations on new project and import

fixes #1089, #1063

sorry, I am repurposing this to also fix #1063
I didn't realize at first that this makes more sense

@aslo @joaoandremartins my fault - I changed the scope of this PR.
I just pushed a change to also auto create the debug and the deploy run configurations. The only remaining TODO here is to auto create the Cloud (if the SDK is specified).

I'll implement auto creating of the Cloud in a separate PR and keep this one scoped to run configurations.

 Now, if there is more than 1 ae standard artifact, then none is seleted by default, requiring the user to edit the local run config prior to running it:

If there is only 1, then its selected by default and immediately runnable:

Note that the run config is valid

Editing the run config shows the single artifact selected
Also note that I think this fixes https://github.com/GoogleCloudPlatform/google-cloud-intellij/issues/1092#issuecomment-264221295. I removed the code from resetEditorFrom to select the single artifact. Instead this selection now happens externally when the run config is created.


[SPARK-48846][PYTHON][DOCS] Fix the incorrect naming and missing params in func docs in builtin.py

What changes were proposed in this pull request?

Fix the incorrect naming and missing params in func docs in builtin.py.
Why are the changes needed?

Some params' name in pySpark docs are wrong, for example:

Does this PR introduce any user-facing change?

No.
How was this patch tested?

Passed GA.
Was this patch authored or co-authored using generative AI tooling?

No.

cc @HyukjinKwon  @allisonwang-db

Let's file a new JIRA next time :-).

Let's file a new JIRA next time :-).

My fault, I probably copied the wrong id. I will definitely pay attention next time. When you have time, please help update the information of the right issue.
https://issues.apache.org/jira/browse/SPARK-48846 @HyukjinKwon Thanks.