Fixing NDK code compilation error in Android Studio

Adding the extra empty ".c" file fixes the error with compilation from the Android Studio.
A known bug: https://code.google.com/p/android/issues/detail?id=66937

This may come from a current NDK bug on Windows, when there is only one source file to compile. You only need to add one empty source to make it work again.<<
http://ph0b.com/android-studio-gradle-and-ndk-integration/

Since I'm a noob in Android-Studio, can you please edit the page of the project (in "README.md" file) and show a full, step-by-step instructions of how to import, build, and run the library's sample in Android-Studio?


obey exit code from rspec to determine success and failure

that is follow up for #58 to get it right in the long run.

speaking of exit codes -- have you ever written an rspec formatter for maven failsafe?


Best Practice for handling local settings while development

Hi there
I was searching through your documentation, which has many very useful inputs, thank you for this.
I am missing one point:
What is the best practice you propose for local development machine settings? It's clear to me that in Azure App Configuration Service you can have labels for different environments. But if you have 5 developers on a project you probably want all of them to add there local dev machine settings with labels. So is there any best practice for this? Or is it just help yourself and add a fallback to local appsettings with a preprocessor constant like #if debug?
Thanks for you help

Hello @LaurinSt
Suggestions may vary depending on your local dev environment. Is this .NET Core application or something else?

Most of the times they are .net core asp.net web api projects. But how would it differ between console project and asp.net project?

These all fall into the same category of .NET apps. I asked because we could've been talking about Java or something else.
I will assume here that you are using AzureAppConfiguration Provider as well.
If you need to have different centralized dev profiles and share between the team, then using labels is a good start.
If you want to allow devs to play locally with some settings (for debugging purpose or whatnot), then you can layer AzureAppConfiguration and local json file (ex. appsettings.dev.json) on top. .NET Core Configuration allows for multiple config providers.
So use AzureAppConfiguration for all centralized settings and local appsettings.dev.json for devs to change stuff locally. Since appsetting.dev.json is after AddAzureAppConfiguration, it should override an existing centralized setting, as well as provide new as appropriate.
Perhaps something like this:
            var config = new ConfigurationBuilder()
                .AddAzureAppConfiguration(...)
                .AddJsonFile("appsettings.dev.json");
                .Build();

Thanks. That's a nice way I was not aware of.

@drago-draganov
Hi drago,
We kind of want the same thing for our .net framework app. When debugging/running locally we don't want to call app configuration at all to save some cost.
For your code:
 var config = new ConfigurationBuilder()                 .AddAzureAppConfiguration(...)                 .AddJsonFile("appsettings.dev.json");                 .Build();
I believe it will still make a call to the Azure app configuration then the local file will override them. Wondering how can we don't call it when local debugging.
Thanks

@iqoOopi,
var configBuilder = new ConfigurationBuilder();

//
// Skip Azure AppConfiguration for dev environment
if (!env.IsDevelopment())
{
    configBuilder.AddAzureAppConfiguration(...);
}

var config = configBuilder
    .AddJsonFile("appsettings.dev.json");
    .Build();

@drago-draganov Oh yes, thanks a lot.


[CI][Packaging][Python] Free-threaded wheel build failing to in docker image install of tzdata

The free-threaded linux wheel builds are failing recently, see eg https://github.com/ursacomputing/crossbow/actions/runs/11146254492/job/30977772790
It seems that the installation of tzdata (does that come as dependency of python in apt?) requires some manual interaction on the cli, regardless of the -y being passed:
   #8 62.14 Configuring tzdata
  #8 62.14 ------------------
  #8 62.14
  #8 62.14 Please select the geographic area in which you live. Subsequent configuration
  #8 62.14 questions will narrow this down by presenting a list of cities, representing
  #8 62.14 the time zones in which they are located.
  #8 62.14
  #8 62.14   1. Africa   3. Antarctica  5. Arctic  7. Atlantic  9. Indian    11. US
  #8 62.14   2. America  4. Australia   6. Asia    8. Europe    10. Pacific  12. Etc
  Error: The operation was canceled.

This was not the case when we initially added those builds.
cc @raulcd @kou

Although when we added those (https://github.com/apache/arrow/pull/43965), the above output was also already present in the logs. So the fact that it is hanging here might not necessarily be caused by tzdata itself.
Looking at some more of the failures the last days, some of those are also being caused by internet / download connection issues.

Issue resolved by pull request 44362
https://github.com/apache/arrow/pull/44362


finalize: handles requests and responses across packets

This is now ready
The point of this PR is for libhtp to handle requests/responses ending and beginning in the middle of network provided chunks.
That is the added test case 97-requests-cut.t and 98
So htp_connp_REQ_FINALIZE probes a whole request line to say if we handle it as body or a new request, unless the connection is closed, in which case we complete the request.
Several test cases result change :

adding a line return to 91-request-unexpected-body.t so that the probing can restart at the new request
EmptyLineBetweenRequests : we do not have any longer tx->request_ignored_lines == 1 as the ignored line is now part of the previous request body
LongResponseHeader as the test framework directly returns on error before closing the connection, we do not make it to tx->request_progress == HTP_REQUEST_COMPLETE. Should we rather change the test framework ?
cf https://github.com/OISF/libhtp/blob/0.5.x/test/test.c#L357 versus https://github.com/OISF/libhtp/blob/0.5.x/test/test.c#L417

See https://redmine.openinfosecfoundation.org/issues/2655 for the latest changes to htp_connp_REQ_FINALIZE
Follows #278 by adding another way to complete the request part of a transaction : when we see a second response (before reassigning to a new in_tx)

If we look for an entire request line, does it matter how long that request line is? Also, does it matter if the request line itself might not be complete yet in the data that is fed to us?

If we look for an entire request line, does it matter how long that request line is?

Not really except for memory consumption (but it is already the case) : we buffer until a new line arrives or we hit the buffering limit field_limit_hard

Also, does it matter if the request line itself might not be complete yet in the data that is fed to us?

In this code no, we only check the method, but we could add furthers checks (especially the protocol part at the end of the request line)


proposal: add option to extend atomicity to io.Closer

Currently the PendingFile implements io.Closer by simply embedding *os.File, so a "normal" Close() closes the underlying tempfile without renaming it. This means using the PendingFile as a io.Closer needs explicit action on both the happy and sad paths:

in the happy path (no errors), Close() doesn't rename the file, so you need CloseAtomicallyReplace()
in the sad path , Close() leaves the dangling tempfile, so you need Cleanup()

This PR is my proposal of adding an option where at least the happy path behaves more like a normal file: writing and closing leaves you with the intended file; sad path needs extra steps (usually removing the file; in our case Cleanup()).
WDYT?

Sounds reasonable, thanks for the detailed description!


[DDC-3455] Test for RuntimeException in AnnotationExporter::exportClassMetadata()

@TorbenBr is the exception expectation correct? Or is this test supposed to fail in this PR?

Closing as incomplete


[REQUEST] VLLM Support

Reference Issues
NA
Summary
I would like to know how to configure VLLM as LLM
Basic Example
I have Mistral and LLama running in VLLM and would like to configure them in Kotaemon
Drawbacks
I need it running with VLLM
Additional information
No response

Hi @micuentadecasa you can totally do something like this: https://docs.vllm.ai/en/latest/serving/openai_compatible_server.html
Start VLLM OpenAI compatible server and add new new OpenAI-based model in resources with modified base_url

@taprosoft  many thanks for the info

thanks for the fast support


Bug Testing and General Feedback

I just noticed this mod today (somehow it flew under my radar until now) and was interested in seeing what it had to offer, so I've done some testing and found a few oddities here and there that might help with the overall polish. Despite that though, it's a very simple but very effective mod, so good work!

The recipe for the Crimson Banner doesn't work if you place the recipe on the right side of the Arcane Workbench. Left side works fine, however. Picture:

Some Crimson Armour recipes have a similar problem to the one above. Crimson Cult Boots don't craft if recipe is in top half of the Arcane Workbench, Crimson Cult Helm doesn't work on bottom half of the Arcane Workbench, and the Crimson Cult Hood doesn't work in the bottom half of the Arcane Workbench. Here's the boots recipe as an example:

There's a few problems with the Ancient Tile recipe. First is that it also seems to have the same problem as above again, where it doesn't work if done in the bottom half of the table. Second is that since I'm helping out with Thaumic Augmentation development, I've noticed there's a small recipe conflict between the current WIP version of it and Crimson Revelations, since both add a recipe for four Ancient Stone in a square. This'll probably need some further thought or discussion, but it's not too serious at the moment, especially since that version of Augmentation is still very much in development.

Suggestion for Crimson Praetor Armour recipes: it might be worth increasing the essentia cost a bit, but I was using an Ancient Infusion Altar to craft them which does lower essentia costs when crafting.

And that's about all of the problems I can find. Aside from that, this mod is excellent, the writing is great and the recipes are very helpful, so it's a good simple way to get access to all of the content that Thaumcraft 6 didn't manage to add before its hiatus. Augmentation is planned to get some of that content actually, but implemented in a different way. I shan't go into too much detail though, spoilers and all that.
While I'm here, I'll send you an invite to the Thaumcraft Addon Discord, it's a good place to discuss addon development or just about anything else really, so feel free to join if you want: https://discord.gg/bBsBJWW

Recipes are fixed and Praetor Armor costs are increased. Ancient Tile recipe was left as four Ancient Stone in a square, but I'd be willing to change it once that version of Thaumic Augmentation is out.


Make Landing Page More Prescriptive in How to Get started

We should improve the landing page to better illustrate how to get started with the visual, i.e.:

Add data
Choose existing template or import one
Save or export

We should also consider what to display if the report is being viewed rather than being edited, as this may not completely make sense, depending on context.

Resolved in build 105.


In IE11 screeshot of graphs and charts present in the html is not getting downloaded

Bug reports:
Hi niklasvh,
We are trying to implement download functionality in our angularjs application where the user can take a screenshot of the HTML page which contains graphs and charts rendered using highchart.js.File with the screenshot should be downloaded and stored in the users local machine. We are leveraging canvas2image.js to download the files in jpeg and png format and jspdf for downloading the screenshot in pdf format. We do not have any server involved in this action and the entire process is handled at the frontend. Strangely this functionality works smoothly in Chrome and Firefox but in IE11 it fails and nothing is downloaded. We are getting DOM7011: The code on this page disabled back and forward caching. in the console if we use html2canvas latest version i.e. 1.0.0-alpha.12 . If we try to use any lower version of html2canvas.js i.e. 0.0.4 then the files get downloaded but we see white patches in place of graphs and charts in all the browsers. Please suggest what should be the appropriate solution for this. Thanks in advance!!
Specifications:

html2canvas version tested with: v1.0.0-alpha.12
Browser & version: IE 11.1029.15063.0
Angular JS 1.5 & highcharts.js 6
Operating system: Windows 10 Enterprise

did you find the solution for this.

Not really!
On Fri, Apr 12, 2019 at 10:23 AM shree2713<EMAIL_ADDRESS>wrote:

did you find the solution for this.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/niklasvh/html2canvas/issues/1537#issuecomment-482436585,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APQUwTREO3D9UjccZ6Fq1Cyk0_FitXW3ks5vgBEtgaJpZM4UIbxB
.

--
Best Regards,
Chetna


Hello

Compile my Mod Please

BRUH
lmfao
this repository is only for bbpanzu's mods

I can’t

I have and error from visual studio

find out how to

so there's this really cool thing, don't know if you have heard this. It's really interesting how it works, have you possibly tried, I dunno, checking https://github.com/ninjamuffin99/Funkin or https://github.com/KadeDev/Kade-Engine on how to compile the game? interesting right?


protocol specification issue

while I did see just now, on npm package page
"No API for user-provided protocol extensions."
there is some support for it.
however; you drop everything to lower case, and force me to think in your terms because you don't provide a reverse map always (or the wrong value, which should be the reverse mapped value).

on( 'request', (req)=> {
  req.resource,   //      '/gameState'
  req.requestedProtocols,    //  [ 'gundb' ]
  req.protocolFullCaseMap[req.requestedProtocols[0]]   // correctly cased 'gunDb'
  req.accept( req.requestedProtocols[0], null );    // have to pass lower case version
}

on( 'connect', (conn)=>{
    conn.protocol
} )

resource is a useful, switable value, but is lost to the  'connect' event (it's not part of connection)
you do pass protocol as part of the connection, but the protocol there is what was sent in the accept.
if I specify a single string protocol for the client, I receive an array that is one element; this is okay, for consistency.  The strings in the array 'requestedProtocols' are always lower case.  In order to get the correct capical case I have to pass it through protocolFullCaseMap.
When I accept() the connection I have to pass the lower case version, and the library apparenlty send the correctly cased one out.  The 'connect' event receives the lower case version.
I also do not see a way to pass information between 'request' and 'connect' other than the selected protocol(?).... like the req object has no relation to the conn object
I don't think there's a guarantee that I will get a 'connect' for a webSockServer object always, after a 'request' that does an accept, before another 'request' ?    or - is it possible to get two 'request' events without a 'connect' (assuming the request does an accept)

I don't think there's a guarantee that I will get a 'connect' for a webSockServer object always, after a 'request' that does an accept?

Yes, there is.

before another 'request' ?

Who cares? Of course you can get different WS connection requests at the same time.
Please, be specific in the issue you are requesting.

in the first case; I meant 'immediately' and/or not before another 'request'
I have to wait until 'connect' to attach on( 'message' ) so I don't know which handlers I should be using in connect, because I don't get the 'resource' from the 'request'
I can save the resource in the webSocketServer object from the 'request' and for the 'connect' to receive the resource; but, if connect doesn't finish before another request, then another request would of course overwrite that...
I remember vaguely in the connection is the remote address; which could let me use that as a key in a map or something; but then multiple browsers from behind the same firewall connecting at the same time wouldn't be identifiable as to which 'request' this 'connect' was in response to.


Figure out how to analyze code coverage with Android Studio

In build.gradle
    buildTypes {
        debug {
            testCoverageEnabled = true
        }
    }

command
 ./gradlew createDebugCoverageReport

coverage.tar.gz
DB004 Code coverage data

http://ksoichiro.blogspot.com/2014/05/androidgradle-jacoco.html


How to use

Can you tell me how to use this code. I am pushed to my github page but it display 404

Hi, @FantasyLions .. Looks like you got it working.  :O) Cheers

Thanks for your help! This problem caused by my domain have not CNAME to github page  -_-#

Oh, I see.. you have another domain besides the github.io one. Yep.. good ol' DNS redirects are great. :O)


Fix poetry install

Pay attention to several changes added to the source, feel free to change if it doesn't fit.

Fix #152 and #154
add flake8 and pre-commit as dev dep.
add pre-commit hook
add poetry test to travis ci

Oh, now I understand. Yaml config for pre-commit doesn't work without this tool. Let's keep it. It is my mistake, sorry. Everything is perfect. I'll drop flake8 from here and merge it :)

Thanks for guiding me along. Go ahead.
:) It's nobody mistake. I am not an expert, still learning. I hope that you will be patient with me LoL.

Merged. Thank you for your help. I'll be happy to see new contributions from you if you have free time. We already have much bugs and ideas in issues and much more undiscovered things :)


import PropTypes from 'prop-types';

Make it compatible with latest react versions.

@Progoogler Can you please merge this?

It would be better to merge this pull request, in my project the library doesn't work at all 😔

@Friendly-Robot This PR was never merged. Can you please actually merge it?

When I cloned this repo, it turned out that the code in the camera.js file wasn't actually changed to what this PR is proposing, can it be changed? It would make upgrading to a newer React Native version a lot easier.


#asIndexKeyOfSize: cleanup

Instead of having
 (aString asIndexKeyOfSize: self keySize) asInteger

everywhere, this PR moves #binaryKey: to the index. We can use that to calculate the key with the right length

Wouldn't it be better to call #binaryKey rather #indexKey because it is the key in the index?
Furthermore #asIndexKeyOfSize should probably return the integer directly.

I made an improved version


Update completions submodule

Be great to update this.

What is the period that they are updated? Been 7 months since the last update.


yandex-music: 5.28.4 -> 5.31.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for yandex-music is: Personal recommendations, selections for any occasion and new music
meta.homepage for yandex-music is: https://music.yandex.ru/
meta.changelog for yandex-music is: https://github.com/cucumber-sp/yandex-music-linux/releases/tag/v5.31.2
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 5.31.2 with grep in /nix/store/gswmbx6zk7gblyvbcb9cmvpk48r62xw9-yandex-music-5.31.2
found 5.31.2 in filename of file in /nix/store/gswmbx6zk7gblyvbcb9cmvpk48r62xw9-yandex-music-5.31.2

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

yandex-music

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/gswmbx6zk7gblyvbcb9cmvpk48r62xw9-yandex-music-5.31.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A yandex-music https://github.com/r-ryantm/nixpkgs/archive/e956046208c0b5e128569e04fc4ab6b7651992c5.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/e956046208c0b5e128569e04fc4ab6b7651992c5#yandex-music

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/gswmbx6zk7gblyvbcb9cmvpk48r62xw9-yandex-music-5.31.2
ls -la /nix/store/gswmbx6zk7gblyvbcb9cmvpk48r62xw9-yandex-music-5.31.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

yandex-music

Maintainer pings
cc @ein-shved for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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

I prefer not to merge this because of this issue.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for yandex-music is: Personal recommendations, selections for any occasion and new music
meta.homepage for yandex-music is: https://music.yandex.ru/
meta.changelog for yandex-music is: https://github.com/cucumber-sp/yandex-music-linux/releases/tag/v5.34.1
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 5.34.1 with grep in /nix/store/650gkjvr80mq8srsanamwj53b49r080m-yandex-music-5.34.1
found 5.34.1 in filename of file in /nix/store/650gkjvr80mq8srsanamwj53b49r080m-yandex-music-5.34.1

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

yandex-music

Instructions to test this update (click to expand)

Either download from the cache:
nix-store -r /nix/store/650gkjvr80mq8srsanamwj53b49r080m-yandex-music-5.34.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nixpkgs-update-cache.nix-community.org/' \
  --option trusted-public-keys '
  nixpkgs-update-cache.nix-community.org-1:U8d6wiQecHUPJFSqHN9GSSmNkmdiFW7GW7WNAnHW0SM=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The nixpkgs-update cache is only trusted for this store-path realization.)
For the cached download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A yandex-music https://github.com/r-ryantm/nixpkgs/archive/161b487f39dd8d92886cbb06f6f17e9c074c72b0.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/161b487f39dd8d92886cbb06f6f17e9c074c72b0#yandex-music

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/650gkjvr80mq8srsanamwj53b49r080m-yandex-music-5.34.1
ls -la /nix/store/650gkjvr80mq8srsanamwj53b49r080m-yandex-music-5.34.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

yandex-music

Maintainer pings
cc @ein-shved for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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


Remove type definitions file: react-query/dist/index.d.ts

After I updated to the latest version 0.4.2 I had a problem with typescript. There seams to be a confusion between the types located in react-query/dist/index.d.ts and @types/react-query/index.d.ts.
When I deleted the first one (react-query/dist/index.d.ts) everything was working as expected. Since there is a dependency to @types/react-query what is the need for the dist/index.d.ts?

Until it's removed, here's a workaround. In your tsconfig.json, add:
    "baseUrl": ".",
    "paths": {
      "react-query": ["./node_modules/@types/react-query"]
    }

Removed in the latest release :) Thanks!


Effects.Typewriter fails when TimeScale is 0

What is the current behavior?
Using the "Phone Chat" sample, no chat test displays when starting the conversation if the timescale is set to zero
Please provide the steps to reproduce, and if possible a minimal demo of the problem:
Import "Phone Chat" sample and open scene.
Add the following to the PhoneChatDialogueHelper script:
void OnEnable()
{
    Time.timeScale = 0;
}

What is the expected behavior?
Chat text display even if timescale is zero
Please tell us about your environment:

Yarn Spinner Version: 2.2.4
Unity Version: 2021.3.21.f1

Other information
Some way to change the effects to run if timescale is zero. Since it uses Time.deltaTime, this prevents it from progressing. Either a way to flip over to unscaledDeltaTime or an easier way to extend the effect so that I can override it. Updating only the Effect doesn't seem to be enough

This is a duplicate (essentially) of #202


Old AGOL Items: Scott

I've found the following items in AGOL that are owned by you that look old, unused, and/or broken. Please take a look at them and see if they can be deprecated/deleted. If they are part of the SGID or some other major project, consider opening a new issue using the full deprecation checklist/process.
I don't mean to call anyone out for their old data. I just wanted to have a place to track these things as I found them as part of the HTTP/HTTPS update process.
This will be a living issue while we track down http links. The list may be edited from time to time.

[ ] Mines (Web Map): http://utah.maps.arcgis.com/home/item.html?id=2666fdd09efa4156957dd5f809eb2f39
[x] PrintProxy Test (Web Mapping Application): http://utah.maps.arcgis.com/home/item.html?id=177c2b166a8d4cb79d888f28f950b33a (If this one needs to stay, could you please update any links to https? The tool shows the following links:  1 x serviceURL: http://localhost:8010, 1 x serviceURL: http://mapserv.utah.gov, 1 x serviceURL: http://print.agrc.utah.gov)
[ ] Sevier Photos (Web Map): http://utah.maps.arcgis.com/home/item.html?id=e30449c6e2004500a62aa89cf87ec351
[ ] WMS Level 21 (Web Map): http://utah.maps.arcgis.com/home/item.html?id=a6743565018941fba743ffaa223a9955
[x] Web App Builder (Web Mapping Application): (not shared publicly, search by name)

This is a very old POC app that I built. I've marked it as deprecated and I'll delete it in a few weeks.
/remind me to delete this item on July 15

/remind me to delete this item on July 15

PrintProxy Test (Web Mapping Application)

This one needs to stay and https isn't going to work with localhost, to my knowledge. I've updated the other URLs in the app.

Where was my reminder yesterday?
@jacobdadams You can delete all of the unchecked items in this issue...

I don't know if the reminder thing is working always...

I've deleted all of my items.


[Feature Request] az ad sp create-for-rbac can show the id/objectId of the created service principal

Context
The current output of az ad sp create-for-rbac only shows the appId:
$ az ad sp create-for-rbac --role Reader --scope /subscriptions/0b1f6471-1bf0-4dda-aec3-cb9272f09590
{
  "appId": "72366639-0238-4559-b44a-9db41c9a3a4b",
  "displayName": "azure-cli-2022-01-17-06-01-22",
  "password": "xxx",
  "tenant": "54826b22-38d6-4fb2-bad9-b7b93a3e9c5a"
}

However, id(in MS Graph)/objectId(in AD Graph) is used when directly calling Role Assignments - Create REST API or granting admin consent (https://github.com/Azure/azure-cli/issues/20792#issuecomment-1014183586, https://github.com/Azure/azure-cli/issues/12137#issuecomment-596567479).
Proposal
az ad sp create-for-rbac can show the id/objectId of the created service principal.
Additional information
az role assignment create can resolve the id/objectId from appId when the current logged-in account has Graph read permission. If the current logged-in account doesn't have Graph read permission, --assignee-object-id must be specified.

enhance create-for-rbac


Error: Redis client error

Hi,
I try to use this plugin to enable Redis with Zeek...
Redis and Zeek are installed on the same Docker container.
If
redis-cli -h <IP_ADDRESS> ping
PONG
But when I start the container, here is the issue:
service redis-server start
Starting redis-server: redis-server.
zeek -i ens35 local Log::default_logdir=/opt/zeek/logs LogAscii::use_json=T
error: Redis client error {"label":"index.js","timestamp":"2024-10-02 08:20:59"}
error: Redis client error {"label":"index.js","timestamp":"2024-10-02 08:20:59"}

Hey @AntoninHL - good to meet you and thank you for taking the time to report an issue.

Could you please run the following command to check Redis is listening via a unix socket rather than a TCP socket.

redis-cli -s /var/run/redis/redis.sock ping

It might be worth also checking that Node.js has correct permissions to access the Redis socket file

ls -l $ZEEKJS_REDIS_SOCKET_PATH
If neither of those approaches resolves the issue ping me again and I can have another look.


Zap to Slog

:tada: This PR is included in version 1.0.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


test: reliable waiting for postgres to start up

Apparently postgres will create sockets before it's actually ready for
clients to talk to it! Weird, but we have a better way of doing this
thanks to the NOTIFY_SOCKET protocol we just learned about.
Co-authored-by: Shahar "Dawn" Or<EMAIL_ADDRESS>

Commenting so that some other PRs are the least recently updated. They are used as test fixtures.


Separate explorer.autoReveal into 2 options (Explorer: Auto Reveal)

This would make it a lot easier for keyboard users to use the the sidebar. For example, I would be able to cut, copy, paste, and delete without using the mouse. Currently, I have to use the mouse to deselect the current file. It shouldn't be selected by default. It should only be revealed by default. For the users who want it to be selected by default, there should be an option.

Current option:
Explorer: Auto Reveal

Controls whether the Explorer should automatically reveal and select files when opening them

Separate this option into 2 options:
Explorer: Auto Reveal

Controls whether the Explorer should automatically reveal files when opening them

Explorer: Auto Select

Controls whether the Explorer should automatically select files when opening them.

Can you explain more of your use case to me please surrounding not being able to copy and paste?

Current behavior:
Because my current file is selected, I have to use the mouse in order to deselect it. Then, I can select the files that I want to select, so I can do file operations on them.
In this example, main.c is selected, but I do not want it selected. I only want to select the 3 other files. so I can move them into the destination folder.

Ideal behavior:
If I disable the hypothetical option explorer.autoSelect, then I just have to use shift + down arrow to select the files. Then I cut and paste them with other keyboard shortcuts. I would be able to do this easily because I would control my selection with the keyboard.

Here is an alternative that would allow me the same functionality. Add a command called explorer.toggleSelect, perhaps with a default shortcut of the spacebar. That way, I would be able to deselect main.c, then select the files I want to select.

Here I never touch the mouse and am able to do all the operations. Am I missing something?

TL;DR the main problem is that I can't do operations on multiple selected files with the keyboard. The minor problem is that the current file is always selected and I cannot unselect it.
Here's the difference: you went into the explorer and did operations on one file at a time. When this is done, vscode operates on the file under the cursor (blue border workbench.colorCustomizations: focusBorder.) You renamed one file. You copied and pasted one file.
I am talking about multiple files. This can be done with the keyboard with shift + down/up arrow. The file I came from is always selected by default, and I cannot unselect it. I will show this.
https://user-images.githubusercontent.com/22121365/222293887-1bc42359-5ca0-49bb-9b28-1bcbf503ea98.mp4
I don't have software that shows my keystrokes, so I'll explain what happens.

Show file structure, close folders & explorer
Reveal file in explorer

Try with keyboard:

Go down to the 2 files, then select them with shift + down arrow twice
A file was cut that I didn't select dontMoveMe.c
Go up to the destination folder with the keyboard
Paste
The paste failed, and my files are no longer cut

Try with keyboard and a little bit of mouse at the end

Go down to the 2 files, then select them with shift + down arrow twice
A file was cut that I didn't select dontMoveMe.c
Go up to the destination folder with the mouse
Paste
The paste succeeded mostly (but a file was move that I didn't want to move)

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Drop PartialEq requirement in create_owning_memo

Is your feature request related to a problem? Please describe.
create_owning_memo requires T to implement PartialEq. This is unnecessary because f returns a boolean that determines if the value has changed. It can also be confusing because it makes it unclear whether the equality is being used from PartialEq or from the boolean.
Describe the solution you'd like
Drop the requirement for PartialEq.
Describe alternatives you've considered
Use of a wrapper type that has a dummy implementation for PartialEq.
Additional context
create_owning_memo was added in the new 0.6.7 release.

Thanks, probably just left in accidentally in the initial PR to add it. Any interesting in making a PR to fix it?


refactor(hasura): change workSituation to string

notebook.workSituations » notebook.workSituation

:tada: This PR is included in version 1.22.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Don't show goals if no goals and no balance

The page of a collective freshly created doesn't look great with no goals and no balance

Can I know what do you want !!
Sent from Mail for Windows 10
From: Xavier Damman
Sent: Thursday, December 13, 2018 7:10 PM
To: opencollective/opencollective
Cc: Subscribed
Subject: [opencollective/opencollective] Don't show goals if no goals and nobalance (#1538)
The page of a collective freshly created doesn't look great with no goals and no balance
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Fix broken external links

Fix broken external links like the image below:

After fixed

Notes

As I observe how Livebook handle external links, it seems like Livebook filters out links that are not directly within the paragraphs.

Thanks!


NIFI-12723: Error Handling in Settings, Bulletin Board, and Counters

NIFI-12723:

Error handling in General and Flow Analysis Rules.
Fixed saving flag in management Controller Service listing.
Error handling in Reporting Tasks and Registry Clients.
Error handling in Bulletin Board.
Error handling in Counters.

Will review...


Support overriding the OTEL service name

As @v1v pointed out, we could support setting a different service name, instead of the default junit2otlp:

using a CLI flag: https://github.com/mdelapenya/junit2otlp/pull/7#discussion_r750517623
using the OTEL_SERVICE_NAME environment variable https://github.com/mdelapenya/junit2otlp/pull/7#discussion_r750519506

Let's define the precedence order in the case both are defined

First approach for precedence:

if flag is present, use flag
if env var is present, use env var
else use junit2otlp as fallback

It would be cool to support passing the service version, setting empty as fallback


Android - VBA - Let's finish in chat screen is displayed on finish of Pending account flow

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

Action Performed:

Log in with a applause.expensifail account (that does not have any bank account already added)
Enable staging Web secure server in Account Preferences if disabled
Navigate to the add bank account modal
Verify the Add bank account modal is displayed with 2 options to add bank accounts (Log in and Manual)
Select the Manual method to add a bank account
Use the "Pending State Withdrawal Account" credentials
When Onfido triggered complete all the steps

Expected Result:
Verify that when you finish the flow - the modal asks for 3 amounts of test transactions
Actual Result:
"Let's finish in chat!" screen is displayed when finishing the 'Pending state withdrawal accoun' flow
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platform:

Where is this issue occurring?

Android

Version Number: 1.2.19-1
Reproducible in staging?: y
Reproducible in production?: Can't check
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
https://user-images.githubusercontent.com/43996225/198099671-692123e4-2a96-4da3-b825-d8689dd5798c.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause internal team
Slack conversation:
View all open jobs on GitHub

I don't understand this issue and the screenshot doesn't match the description of the problem. Can I get some clarification as to what is happening and why that's incorrect?

@tylerkaraszewski Sorry about the wrong attachment. Uploaded the correct video

Issue is reproduced with build <IP_ADDRESS>
https://user-images.githubusercontent.com/93399543/201097896-927cfdb2-533c-4c20-a840-9130233ae8fb.mp4

Lol, immediately overdue. Not looked at yet.

Still haven't looked.

Closing per the related Slack convo. This isn't a bug.


Fix the crash when filtering RangeData in local_trajectory_builder for very long runs.

The pr fix the crash for the long runs(like car). The points in the local frame have very large position value, and it causes the crash when filtering. So I transformed the accumulated_range_data_ in the first tracking frame to avoid large position value, then filtering it.
Tested long runs.

I created the bug https://github.com/googlecartographer/cartographer/issues/937 and will have a look at the voxel filter itself.

I fixed it with https://github.com/googlecartographer/cartographer/pull/938 in the VoxelFilter itself, so there is no need to work around by transforming to the tracking frame.
@zjwoody Can you verify the fix with a long run?

@gaschler , I tested #938 , it fixed the crash in long runs now, thanks!

@zjwoody Thanks for verifying!
I'll close this PR for now.


Questions about hyper-parameters in the finetuning stage

Hi, it's a wonderful work.
I am trying to replicate the results of the paper that have been fine-tuned to datasets. As you know, it often needs to take a long time to adjust hyper-parameters to get well trained transformer-based models. If you can show all the settings, it will be really helpful.
I saw some settings in #105, #45, but it lacks other size of models and datasets such as DeiT-Ti, iNat.
Could you please help summarize all size of model on different finetuning datasets mentioned in the paper?
Maybe a table form is very clear, just like the following:

model type
pretrained dataset
finetuned dataset
lr
bs
wd
sched
epochs
warmup
...

ViT-B

ViT-L

...

DeiT-Ti

DeiT-S

...

I really appreciate it.

Hi @lostsword,
Thank you for your suggestion.
As soon as I have some time I complete this table and add it to the Readme.
I'll keep you informed.
Best,
Hugo

Hi @lostsword, Thank you for your suggestion. As soon as I have some time I complete this table and add it to the Readme. I'll keep you informed. Best, Hugo

OK. Thanks a lot.

Hi!
This would help a lot! @TouvronHugo  any update on when it would be possible?


If loxilb is not present, svc external IP is not assigned.

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


[Staging and Production] New Component: The 'project page' link does not navigate to the main project overview page

Steps

Go to osf.io
Go to a project for which you have admin access
On the project overview page, click add component and create a new component
Ensure that you are redirected to the component overview page and there is a blue alert message at the top
Click on 'project page' link

Expected
The link would redirect the user to the main project overview page
Actual
The link just refreshes the component page and the blue alert disappears

This problem has been fixed on production


Issue when connecting Azure Bot Service to bot behind Traffic Manager

We have recently enabled Traffic Manager to sit between the Azure Bot Service channel registration and our bot. When we did this, our web chat component immediately starts failing. I believe we are seeing Authentication Exceptions in the process. The exact message is: "POST to xxxxxxx failed: Bad certificate Bad certificate The remote certificate is invalid according to the validation procedure.". And it seems the bad cert exception is occurring when we hit the traffic manager. Any ideas on how we can fix this?

Ah, I think we are hitting this: https://social.msdn.microsoft.com/Forums/sqlserver/en-US/74920104-ddc9-4ef7-bb10-3c73f3fe8205/certificate-problem-with-traffic-manager?forum=WAVirtualMachinesVirtualNetwork. Closing for now.


Add Switch app link to admin pages

Adds a link to signon to the admin header.
Before

# After

Trello: https://trello.com/c/KJlQrRWB/1445-1-add-switch-app-to-whitehall

 (1) Add 'Switch app' to Whitehall


Set detail URLs in model

Currently the template can render detail URL's. However, the modules do not set those value son the model.

Closed by 0111e95c628c7dc5c7346b547ab2c8adba930e0e


Needs release token

Problem
We turned on MQ on this project, but didn't have any status checks, so things were merging immediately. I've now enabled required checks, but the release can't be cut.
Proposed solution
Switch to using a release token, which means we need to generate a token for this repo.
Alternatives considered
No response
Additional context
No response
Submission Checklist

[X] Once I hit submit, I will assign this issue to the Project Board with the appropriate tags.

I have added ACVM_RELEASES


Error with DistributedDataParallel

Hi, I trying to run informer training with DistributedDataParallel, and get error
-- Process 1 terminated with the following error:
Traceback (most recent call last):
  File "/home/dbliznyuk/.conda/envs/test_crash/lib/python3.8/site-packages/torch/multiprocessing/spawn.py", line 20, in _wrap
    fn(i, *args)
  File "/home/dbliznyuk/test_crash_skript/linformer-pytorch/test_ddp_vanila_torch.py", line 71, in demo_basic
    outputs = ddp_model(torch.randint(20000, (3, 5120)))
  File "/home/dbliznyuk/.conda/envs/test_crash/lib/python3.8/site-packages/torch/nn/modules/module.py", line 722, in _call_impl
    result = self.forward(*input, **kwargs)
  File "/home/dbliznyuk/.conda/envs/test_crash/lib/python3.8/site-packages/torch/nn/parallel/distributed.py", line 511, in forward
    output = self.module(*inputs[0], **kwargs[0])
  File "/home/dbliznyuk/.conda/envs/test_crash/lib/python3.8/site-packages/torch/nn/modules/module.py", line 722, in _call_impl
    result = self.forward(*input, **kwargs)
  File "/home/dbliznyuk/test_crash_skript/linformer-pytorch/linformer_pytorch/linformer_pytorch.py", line 364, in forward
    tensor = self.linformer(tensor, **kwargs)
  File "/home/dbliznyuk/.conda/envs/test_crash/lib/python3.8/site-packages/torch/nn/modules/module.py", line 722, in _call_impl
    result = self.forward(*input, **kwargs)
  File "/home/dbliznyuk/test_crash_skript/linformer-pytorch/linformer_pytorch/linformer_pytorch.py", line 321, in forward
    tensor = checkpoint(layer, tensor)
  File "/home/dbliznyuk/.conda/envs/test_crash/lib/python3.8/site-packages/torch/utils/checkpoint.py", line 163, in checkpoint
    return CheckpointFunction.apply(function, preserve, *args)
  File "/home/dbliznyuk/.conda/envs/test_crash/lib/python3.8/site-packages/torch/utils/checkpoint.py", line 74, in forward
    outputs = run_function(*args)
  File "/home/dbliznyuk/.conda/envs/test_crash/lib/python3.8/site-packages/torch/nn/modules/module.py", line 722, in _call_impl
    result = self.forward(*input, **kwargs)
  File "/home/dbliznyuk/test_crash_skript/linformer-pytorch/linformer_pytorch/linformer_pytorch.py", line 61, in forward
    tensor = tensor + self.fn(tensor, **kwargs)
  File "/home/dbliznyuk/.conda/envs/test_crash/lib/python3.8/site-packages/torch/nn/modules/module.py", line 722, in _call_impl
    result = self.forward(*input, **kwargs)
  File "/home/dbliznyuk/test_crash_skript/linformer-pytorch/linformer_pytorch/linformer_pytorch.py", line 235, in forward
    head_outputs.append(checkpoint(head,Q,K,V))
  File "/home/dbliznyuk/.conda/envs/test_crash/lib/python3.8/site-packages/torch/utils/checkpoint.py", line 163, in checkpoint
    return CheckpointFunction.apply(function, preserve, *args)
  File "/home/dbliznyuk/.conda/envs/test_crash/lib/python3.8/site-packages/torch/utils/checkpoint.py", line 74, in forward
    outputs = run_function(*args)
  File "/home/dbliznyuk/.conda/envs/test_crash/lib/python3.8/site-packages/torch/nn/modules/module.py", line 722, in _call_impl
    result = self.forward(*input, **kwargs)
  File "/home/dbliznyuk/test_crash_skript/linformer-pytorch/linformer_pytorch/linformer_pytorch.py", line 162, in forward
    P_bar = Q/torch.sqrt(torch.tensor(self.dim).type(Q.type()))
RuntimeError: Expected all tensors to be on the same device, but found at least two devices, cuda:1 and cuda:0!

Seems like this error connected with parameter sharing
Code for reproducing
import os
import tempfile
import torch
import torch.distributed as dist
import torch.nn as nn
import torch.optim as optim
import torch.multiprocessing as mp
from linformer_pytorch import LinformerLM
from torch.nn.parallel import DistributedDataParallel as DDP

def setup(rank, world_size):
    os.environ['MASTER_ADDR'] = 'localhost'
    os.environ['MASTER_PORT'] = '12355'

    # initialize the process group
    dist.init_process_group("gloo", rank=rank, world_size=world_size)

def cleanup():
    dist.destroy_process_group()

def demo_basic(rank, world_size):
    print(f"Running basic DDP example on rank {rank}.")
    setup(rank, world_size)

    # create model and move it to GPU with id rank

    model = LinformerLM(
            num_tokens=30522,  # Number of tokens in the LM
            input_size=5120,  # Dimension 1 of the input
            channels=128,  # Dimension 2 of the input
            dim_d=None,
            # Overwrites the inner dim of the attention heads. If None, sticks with the recommended channels // nhead, as in the "Attention is all you need" paper
            dim_k=128,  # The second dimension of the P_bar matrix from the paper
            dim_ff=128,  # Dimension in the feed forward network
            dropout_ff=0.15,  # Dropout for feed forward network
            nhead=16,  # Number of attention heads
            depth=12,  # How many times to run the model
            dropout=0.1,  # How much dropout to apply to P_bar after softmax
            activation="gelu",
            # What activation to use. Currently, only gelu and relu supported, and only on ff network.
            checkpoint_level="C2",  # What checkpoint level to use. For more information, see below.
            parameter_sharing="none",  # What level of parameter sharing to use. For more information, see below.
            k_reduce_by_layer=0,
            # Going down `depth`, how much to reduce `dim_k` by, for the `E` and `F` matrices. Will have a minimum value of 1.
            full_attention=False,
            # Use full attention instead, for O(n^2) time and space complexity. Included here just for comparison
            include_ff=True,  # Whether or not to include the Feed Forward layer
            w_o_intermediate_dim=None,
            # If not None, have 2 w_o matrices, such that instead of `dim*nead,channels`, you have `dim*nhead,w_o_int`, and `w_o_int,channels`
            emb_dim=128,  # If you want the embedding dimension to be different than the channels for the Linformer
        ).to(rank)
    ddp_model = DDP(model, device_ids=[rank])

    loss_fn = nn.CrossEntropyLoss()
    optimizer = optim.SGD(ddp_model.parameters(), lr=0.001)

    optimizer.zero_grad()
    outputs = ddp_model(torch.randint(20000, (3, 5120)))
    labels = torch.randint(20000, (3, 5120)).to(rank)
    loss_mx = labels != -100
    output = outputs[loss_mx].view(-1, 30522)
    labels = labels[loss_mx].view(-1)
    loss_fn(output, labels).backward()
    optimizer.step()

    cleanup()

def run_demo(demo_fn, world_size):
    mp.spawn(demo_fn,
             args=(world_size,),
             nprocs=world_size,
             join=True)

if __name__ == "__main__":
    run_demo(demo_basic, 2)

Also, with DataParallel training going normal

Hey @blizda!
It should be fixed now. It was a simple 1 line change from
P_bar = Q/torch.sqrt(torch.tensor(self.dim).type(Q.type()))

to
P_bar = Q/torch.sqrt(torch.tensor(self.dim).type(Q.type())).to(Q.device)

Check it out with the latest commit or by downloading 0.19.3 from pip

Yes, this fix work, thanks


Back-End - Preliminary Research on NYT Article Search API and API Key Registration

Description:
Tasks:

[ ] Conduct preliminary research on the New York Times Article Search API.

[ ] If not done already, register for an API key to access the New York Times Article Search API.

Assignees:
Back-End Team

task completed


Missing alternate key name in URI

In version 5.11.2 some of our odata requests to the MS Dynamics CRM OData service started to fail. We have found that the behaviour of the library changed concerning filter criteria containing alternate keys. Take the following code snippet as an example (where someAltKey is an alternate entity key):
client.For<account>()
	  .Filter(entity => entity.someAltKey == altKeyValue)
	  .Top(1)
	  .FindEntriesAsync();

In version <IP_ADDRESS> the resulting query URI is as expected and resembles the above code:
https://[...].dynamics.com/api/data/v9.1/accounts?$filter=someAltKey eq '[...]'
In version 5.11.2 instead of emitting $filter in the URI, the entity will be accessed directly using the alternate key, resulting in an invalid URI:
https://[...].dynamics.com/api/data/v9.1/accounts('[...]')
As someAltKey is an alternate entity key, the URI should include the name of that property:
https://[...].dynamics.com/api/data/v9.1/accounts(someAltKey='[...]')

This should be addressed by #725, please check version 5.12.0.


Add Byond 513.1529

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

Note: This program does not support a completely silent install, due to requiring administrator privileges to install. It will prompt with UAC.
Note: The program may produce an error code 1223. The program still installs successfully.  This is due to some bug with the silent NSIS flag. If the flag is removed, it will no longer produce the error code.
Microsoft Reviewers: Open in CodeFlow

I am seeing an error at the end of setup.  I suspect this is by design and this is blocked on  microsoft/winget-cli#137.


Add support for Windows newlines in the log

Really not an issue for most text editors, but Notepad requires CRLF and it's a quick easy fix

Usually the mode you open a file handles these things for you, this is probably the wrong way

Hmm, I'll take a look and see if I can't find a better way

It's worth mentioning that pretty much the only program that this affects is Notepad, as pretty much every other program handles LF gracefully.  Though Notepad is the default, I imagine that most people who are actually checking the log probably have another editor on hand.  Another detail pointed out in this thread is that different line endings could break parsing, though I don't imagine that should be a huge issue here?
That said, I can't find any node documentation that suggests that line endings can be auto handled based on the file mode.  The only solution I've found exposed is os.EOL, which involves another require, but would be a better solution.
Apologies if I've missed something better, I've never worked directly with node.js before, so this is mostly pulled from some quick searches and skimming the docs.

If you could use os since it's in node by default that's fine, I was mostly
mentioning wb and w mode since almost every other language supports it and
handles new lines cross platform perfectly, node might not have that
Might also want to look to see if there is an option in fs to write to a
file in non binary mode
On Mon, Jul 2, 2018, 7:44 PM Zac Davidsen<EMAIL_ADDRESS>wrote:

It's worth mentioning that pretty much the only program that this affects
is Notepad, as pretty much every other program handles LF gracefully.
Though Notepad is the default, I imagine that most people who are actually
checking the log probably have another editor on hand. Another detail
pointed out in this thread https://github.com/nodejs/node/issues/5038
is that different line endings could break parsing, though I don't imagine
that should be a huge issue here?
That said, I can't find any node documentation that suggests that line
endings can be auto handled based on the file mode. The only solution I've
found exposed is os.EOL, which involves another require, but would be a
better solution.
Apologies if I've missed something better, I've never worked directly with
node.js before, so this is mostly pulled from some quick searches and
skimming the docs.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/meepen/salien-bot/pull/175#issuecomment-401970013,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF5lSUrZcQg58CBubYgrZi8y1Ao-VZqrks5uCrBfgaJpZM4U__lJ
.

  Would you prefer to keep os in a const global like fs and then just drop os.EOL in for the newline character, or just pull os.EOL into a let and not keep all of os around?

const os = require("os"); should be fine


Flet app for Android: Back Navigation using native OS back button is exiting the app

Description
I've have 2 routes, initial / route and /store route
from /store view, while pressing system back navigation button ....it's triggering disconnect event.
how to navigate back to initial / view on pressing it
Code example to reproduce the issue:
#/app/main.py
import flet as ft
async def main(page: ft.Page):
    print()
    page.theme = ft.Theme(color_scheme_seed= ft.colors.GREEN_500,
                          page_transitions= ft.PageTransitionsTheme(
                              android= ft.PageTransitionTheme.CUPERTINO,
                              linux= ft.PageTransitionTheme.CUPERTINO
                          ))
    page.padding = 0
    async def route_change(route):
        page.views.clear()
        page.views.append(
            ft.View(
                "/",
                [
                    ft.AppBar(title=ft.Text("Flet app"), bgcolor=ft.colors.SURFACE_VARIANT),
                    ft.ElevatedButton("Visit Store", on_click=gotostore),
                ],
            )
        )
        if page.route == "/store":
            page.views.append(
                ft.View(
                    "/store",
                    [
                        ft.AppBar(title=ft.Text("Store"), bgcolor=ft.colors.SURFACE_VARIANT),
                        ft.ElevatedButton("Go Home", on_click=gohome),
                    ],
                )
            )
        await page.update_async()
    async def gohome(e):
        await page.go_async('/')
    async def gotostore(e):
        await page.go_async('/store')

    async def view_pop(view):
        print('view',view)
        page.views.pop()
        top_view = page.views[-1]
        await page.go_async(top_view.route)
    page.on_route_change = route_change
    page.on_view_pop = view_pop

    await page.go_async('/')
ft.app(target=main)

Describe the results you received:
I've have 2 routes, initial / route and /store route
from /store view, while pressing system back navigation button ....it's triggering disconnect event.
how to navigate back to initial / view on pressing it
https://github.com/flet-dev/flet/assets/35882740/423b81bc-29c3-45e7-a34b-a5c62508ed42
Describe the results you expected:
I expected it to switch back to initial / route from /store route on pressing system back button
Additional information you deem important (e.g. issue happens only occasionally):
Please use bottom back navigation button provided by underlying android OS to reproduce the issue.
There is no issue while using back button present in App Bar.
Flet version (pip show flet):
Name: flet
Version: 0.20.0.dev2135
Summary: Flet for Python - easily build interactive multi-platform apps in Python
Home-page:
Author: Appveyor Systems Inc.
Author-email<EMAIL_ADDRESS>License: Apache-2.0
Location: /home/lekshman/goldfinance/app_env/lib/python3.11/site-packages
Requires: cookiecutter, flet-runtime, packaging, qrcode, watchdog, websocket-client, websockets
Required-by:

Operating system:
command ran on Kali Linux
flet run --android /app

Additional environment details:
Tested on Flet Mobile Android Testing App
Client Version 1.4.0

Ah, it's the issue inside Flet app for Android. That makes sense as pressing device back button is sending it to "Projects" screen instead of previous route inside the app. I'm afraid not all scenarios could be covered/tested with Flet app. There we have like Flet app inside another Flet app. We could try "redirecting" back button or maybe "block" with https://stackoverflow.com/questions/44635450/is-there-any-way-intercept-back-keydown-in-flutter-app-on-android
The main question is if the app is normally working when built with flet build apk. Have you tried that?

@lekshmanmj Maybe try adding "back" and "forward" buttons like that of a browser...

Ah, it's the issue inside Flet app for Android. That makes sense as pressing device back button is sending it to "Projects" screen instead of previous route inside the app. I'm afraid not all scenarios could be covered/tested with Flet app. There we have like Flet app inside another Flet app. We could try "redirecting" back button or maybe "block" with https://stackoverflow.com/questions/44635450/is-there-any-way-intercept-back-keydown-in-flutter-app-on-android
The main question is if the app is normally working when built with flet build apk. Have you tried that?

Have not built apk but I tested  in normal mobile browser and it works. Thanks

I've tested with an actual apk and sadly the back button exits the app completely, any possible solution?

But it doesn't work for apk, any solutions? I have tried lots of things but nothing seems to work :(

Android back button event capture and use in routing and exit solved and proved with other people here: https://github.com/flet-dev/flet/discussions/4495#discussioncomment-11634586


[$1000] Thread title briefly appears in LHN after deleting thread message

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

Action Performed:

Open chat
Send a message
Click on "Reply in thread"
React with emoji on thread message
Send message on thread
Delete that message

Expected Result:
After deleting the thread message, the thread title should not be displayed from LHN
Actual Result:
The thread title is briefly visible in LHN even after deleting the thread message
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

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

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

Version Number: 1.3.32-5
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
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/4123a85c-250c-4704-84cc-7bddc21a4287
https://github.com/Expensify/App/assets/43996225/87ac183e-f741-4dbd-a4c5-d62dbe668c04
Expensify/Expensify Issue URL:
Issue reported by: @ayazhussain79
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1687373680904369
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01afe64c0c4da4cc48
Upwork Job ID:<PHONE_NUMBER>343728128
Last Price Increase: 2023-06-26

Blimey this is niche. It only appears for a second, and it's only a problem if the first message of the thread had an emoji reaction.
Let's fix, but I'm making it weekly. It doesn't even affect people who use #focus mode.

@ginsuma Thanks for the proposal. I don't think your RCA is correct. What you are looking at is the onyx to be merged data with the existing one in the IndexedDB. The created field does not have to be included because it's not going to change. We are adding a new report action but updating an existing one.
BTW, are you still able to reproduce the bug? It does not seem reproducible from my end

@twisterdotcom Can you please retest from your end as well? This seems to be fixed already

@twisterdotcom @s77rt I found this bug six days ago, but what if someone else came across this issue after my post and resolved it.
Can you please check that when it was found and fixed ?

Huh, weird yes. It is fixed. Okay, let's close. Does anybody know where we solved this?
@ayazhussain79 I sent you the bug reporting bonus contract. I'll close this now.

@twisterdotcom offer accepted, Thank you


NTR: exhausted T cell

Preferred term label:
exhausted T cell
Synonyms
Tex cell
Definition (free text, please give PubMed ID)
An effector T cell that displays impaired effector functions (e.g., rapid production of effector cytokines, cytotoxicity) and has limited proliferative potential.
Source: PMID:26544946
Parent term
CL_0000911 'effector T cell'
related ticket: https://github.com/obophenotype/human-phenotype-ontology/issues/5699

look for markers and include this info I can find info


Add ip validation check

Due to the config saving ip addresses and the host address may change. a warning should be logged when the upnp config address does not match any ip on the host.

Good idea :)


Object declared type not being passed to Json output formatter

While working on a solution for the following SO question, I noticed that we are not giving information about the 'declared type' to the JsonOutputFormatter.
http://stackoverflow.com/questions/34326013/how-to-specify-polymorphic-type-in-asp-net-mvc-6
Code of interest (There is an overload here which takes in a parameter for 'object type'): https://github.com/aspnet/Mvc/blob/dev/src/Microsoft.AspNet.Mvc.Formatters.Json/JsonOutputFormatter.cs#L81
Also currently its not simple to supply this information on JsonResult, should we expose a property to enable users to supply the declared type explicitly? (the obvious workaround is to use ObjectResult)

We really really really really strongly discourage serializing a 'type' in JSON. A client can use HTTP to 'new' up any CLR type.

@rynowak so by-design on this? But one thing I'm confused on is that the SO question was about deserialization, yet the code of interest is about serialization...

By design IMO. I don't think we're giving up anything valuable here.

@Eilon just fyi...more detailed example:
Server:
public BaseClass GetInfo()
{
   return new Foo() { }
   OR
   return new Bar() { }
}

Client:
jsonSerializer.Deserialize<BaseClass>(jsonReader) ...this should create an instance of Foo or Bar...i think the user is saying that without the type info in the serialized data, the deserialization would create an instance of BaseClass instead

I long liked using obj.GetType() to determine what to serialize, but I think the world is against people like me. And maybe people like you :smile:
However, to many people the signature is a contract, and only aspects of that contract can be present. Anything not in the contract is a violation. If you said you're returning a BaseClass then only those properties may be present - nothing more.


Change twitter logo to X

I would like to work on this issue . Pls assign me this !

Please assign this issue to me (Riyasharma28) under GSSoC'24.

I would really like to work on this .Please asssign me this @Suchitra-Sahoo

please assign me this

Did the issue assigned to me?
Thank You
On Mon, 13 May 2024 at 18:29, Komalpardasani @.***>
wrote:

please assign me this
—
Reply to this email directly, view it on GitHub
https://github.com/Suchitra-Sahoo/AgriLearnNetwork/issues/235#issuecomment-2107512574,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A57TROLM474KL2JX4XW54LTZCC2KRAVCNFSM6AAAAABHTX7B72VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMBXGUYTENJXGQ
.
You are receiving this because you commented.Message ID:
@.***>

i would like to change the twitter logo to the updated one. Please assign this to me.

This issue is assigned.


[DRAFT] Use protos to represent remote data model

Towards #1748

@shobhitagarwal1612 @sufyanAbbasi PTAL?


Per project docs

I'd like to see documentation per VS project, similar to that in doc/ORGANIZATION.md, or to abstract comment on files. It could contain:

responsibilities
target platforms (standalone window app / embeded control / server / api)
used frameworks (pure c++/ .NET Framework / .NET Standard / c++/cli / winapi / winrt / wpf / winforms / rendering lib)
its place in the "stack"
target (native exe / dll / source only)

This'd make contibuting much easier, but I'm affraid that's subject to frequent changes

@bitcrazed is gonna work on a mega doc outlining the components soon :)


Added bug_report and feature_request issue forms

fixes #685
Describe your changes
I have added two issue forms (bug report and feature request) to the repo.
Screenshots - If Any (Optional)
Bug Report-

Feature Request-

Issue ticket number and link
Checklist before requesting a review

[x] My code follows the style guidelines of this project
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have performed a self-review of my code
[x] If it is a core feature, I have added thorough tests.
[x] New and existing unit tests pass locally with my changes

Please open the PR against the dev branch (Unless the PR contains only documentation changes)

@vladklokun What do you think of this?

@vladklokun What do you think of this?

I think having the templates as issue forms is nice for those opening the issues.
However, I’m not a big fan of the fact that in this version descriptions are rendered in one line, thus flattening any lists, like the one in Steps to Reproduce. I prefer that our hints for filling out forms would be properly rendered as Markdown (lists etc). So I think we should provide the example data for filling out the section—as in how to fill out Steps to Reproduce—in the placeholder key.

wont merge


Buycraft plugin API set coupon code redeem_limit to "unlimited"

I see coupon code redeem limit and expire date have been made no longer mutually exclusive.
How do we set the redeem_limit to "unlimited" now using the API?
The Coupon.Expire class requires an int for the limit

Fixed in #95


Corrected typo in example

https://github.com/MicrosoftDocs/office-docs-powershell/issues/4260

Docs Build status updates of commit f0fcda7:
:clock10: Full build: the average full build time is 11 min(s) 46 sec(s), this is based on the last 1000 build(s) for this repository.


AzureStaticWebAppLatest@0?

There's no such thing as AzureStaticWebAppLatest@0, at least not in service-schema.json. When ran, it produces the following error:

A task is missing. The pipeline references a task called 'AzureStaticWebAppLatest'. This usually indicates the task isn't installed, and you may be able to install it from the Marketplace: https://marketplace.visualstudio.com. (Task version 0, job 'buildAndDeploy', step ''.)

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 3a4b67ed-5493-4e5d-ee03-0a78c121bc7e
Version Independent ID: e853127e-57f8-8d9a-da56-8eeed9017a67
Content: Tutorial: Deploy static-rendered Next.js websites on Azure Static Web Apps
Content Source: articles/static-web-apps/deploy-nextjs-static-export.md
Service: static-web-apps
GitHub Login: @craigshoemaker
Microsoft Alias: aapowell

@wojtekmaj
Thanks for your feedback! We will investigate and update as appropriate.

@wojtekmaj how did you configure your pipeline? Adding the task to my azure-pipelines.yml didn't produce the same error. See images below.


[PE-D][Tester E] Unecessary case-sensitivity

Intentionally typoed Tutorial as tutorial
There should be no ambiguity that I intended to create a tutorial, and the command should just work anyway

Labels: severity.Low type.FeatureFlaw
original: immanuelhume/ped#8

It is intended. Will introduce unnecessary bugs if otherwise. We can argue that for any commands in any operating system.


Fix link for mesosphere dc/os platform

https://www.meetup.com/Docker-Online-Meetup/ currently returns "Error 503 Max restarts limit reached" which is a temporary problem on the server side (i assume).
Reminder: re-run tests before merging and check if links are all fine now


Usability: Template computer/code setup and new code registry configs

Motivation and impact
As a user, I want to have a way to easily setup computer/code and start using AiiDA quickly.
As a user, I want to go to a centralized place to share or find the configurations of my settings.
As a user, when I set the for the same computer and codes for another AiiDA profile, I can do it without duplicating all processes but very easily accomplish it.
As an aiida-code-registry developer, I need all the config sharing processes automatically and put less effort on review the new computer/code configs added in the repo.
As an aiidalab-widgets-base developer, I want to use a official supported template system to set up computer/code for ComputationalResourceWidget for setting computer/code from "database".
The motivation may contain one or more use cases.
Desired Outcome

Setup computer/code from template foramt config files
Registry page to show/share new configure files

Background
For beginners and even for the experienced AiiDA user, setting up computers and codes is still a tedious mission. If using the interactive mode, although it is good that options are prompted up and the user can set every option one by one carefully, it requires going through all options even if some are not necessary and time-consuming for a similar setup that has shared options with other code/computer setup.
AiiDA provides the non-interactive mode to set up the computer/code from a config YAML file, which lowers the burden for users who need to set up the computer/code next time. However, the non-interactive mode requires a YAML file as the input, and not clear which options are mandatory, and let alone it is not clear which default value will be used without checking the command help message or even the source code. Let alone for the computer setup it is a two-stage process, user needs to set up the computer for attributes that are common information for the computer that is stored in the database using verdi computer setup. Then running verdi computer configuration to set up information of the computer that is specific to the user or required to modify after the node is stored in the database.
The computer/code can be set up from a YAML file, and we provide a repository aiida-code-registry to store the YAML files for public computers and codes to share with others. Need to mention that the interactive setup command can accept a URL of a remote YAML file for setup. This makes it possible to not download/clone the aiida-code-registry repo to use the YAML to set up computer/code.
Complexity
The template system need to be supported with verdi, which the DynamicEntryPointCommandGroup can be potentially used to prompt the required parameters dynamically for user's input. But it is not clear how to make it works.
The computer/code registry page need redesigned and well documented in the AiiDA-core docs to encourage users to contribute and maintain it.
Further comments (by GP)
you are using this for the AiiDAlab right?
Can you please comment here on what is the status and if/what are the missing actions?
E.g.:
is the repo easily findable for a generic AiiDA user, e.g. from the AiIDA-core docs?
are there clear instructions on how to contribute a new code/computer?
are there outstanding technical issues, e.g. how to always ask to customize some aspects of a computer such as account or allocation or project?
are at least the (public, i.e. not private inside a university) computers/codes we use there? E.g. Daint, Eiger, Lumi-C, ...
can we identify a person in charge of maintaining the repo?
I think some of these are in the issues of the repo https://github.com/aiidateam/aiida-code-registry, but probably not all of them. If we find a person in charge, this person can triage a bit the tasks and prioritise them (and possibly distribute them, e.g. organising a "coding day"?)

Thanks @unkcpz; I think
@unkcpz and @sphuber I think we should align #4 and #5 a little, I guess the key points are something like:

#4: Easier to setup codes with existing plugins
#5: Easier to setup (simple) codes without a plugin

What do you think?

@unkcpz, I feel the roadmap item, is still focussing a bit too much on the "how" and not the "why"
For the title I suggest something like:

Usability: Make it easier for users to setup known computers and codes

what do you think?

Thanks @chrisjsewell, all very good suggestions, I agree with the rename for both of them.

Okay, I think I need to use your template strictly. I copy and paste from the item I create on Monday. Too hurry up ;) Sorry for the carelessness.

Okay, I think I need to use your template strictly. I copy and paste from the item I create on Monday. Too hurry up ;) Sorry for the carelessness.

Ah no worries. Just want to make sure the first roadmap items are good exemplars for future ones 😄


Remove devices after deleting from Mi Home

Or after deleting from second Home Assistant (c) @to4ko

Autoremove supported in latest version


Update the CLI UI for the command execution.

Just something, i though need a bit enhancement.

hello hello..
anyone home...


Adding date to native logging

Would there be interest in adding date information to foreman's native log system?
This would involve changing foreman/engine/cli.rb line 60 from:
output += "#{Time.now.strftime("%H:%M:%S")} #{pad_process_name(name)} | "
To:
output += "#{Time.now.strftime("%e %b %Y %H:%M:%S")} #{pad_process_name(name)} | "
Sample output (current):
20:44:32 web.1  | message
Sample output (proposed):
4 Feb 2016 20:44:32 web.1  | message
Another option might be ISO8601 time format.

No, let's keep this header short. Foreman generally will refuse changes that are only beneficial to running foreman in production as I am trying to keep the scope narrow as a development tool.


Disabling end scroll indicator on Android

Question
Hi,
I was just wondering if there are any ways to disable/hide the "end scroll indicator" (see image bellow) on Android?

Thank you

please use overScrollMode


ci: update changelog action

What is the problem this feature will solve?
No response
If you have solution，please describe it
No response
What alternatives have you considered?
No response

https://github.com/cuisongliu/quickon_cli/blob/master/.github/workflows/changelog.yml


Add new exchange https://latoken.com/

Pls Add new exchange
https://latoken.com/

Will be included in the next app release (v2.49.1)


Wrong definition of funPinMode?

The definition of funPinMode() seems to be broken.
#define funPinMode( pin, mode ) { GpioOf(pin)->CFGLR = (GpioOf(pin)->CFGLR & (~(0xf<<(4*((pin)&0xf))))) | ((GPIO_Speed_10MHz | GPIO_CNF_OUT_PP)<<(4*((pin)&0xf))); }

Please note that it does not use the mode  argument.
This define works well for me:
#define funPinMode( pin, mode ) { GpioOf(pin)->CFGLR = (GpioOf(pin)->CFGLR & (~(0xf<<(4*((pin)&0xf))))) | ((mode)<<(4*((pin)&0xf))); }

Created a PR: https://github.com/hexaedron/ch32v003fun/pull/1

Created https://github.com/cnlohr/ch32v003fun/pull/267

Fixed here: https://github.com/cnlohr/ch32v003fun/pull/268


[web] Fix url strategy null safety

The pushState/replaceState methods should accept a null state argument.
The state parameter is already nullable in the engine.
Fixes https://github.com/flutter/flutter/issues/78017

@ditman the browser accepts a null state. We should too.
It's possible to read the state from the browser history (which is nullable), then use it later with a pushState or replaceState call. This is what was happening here.

It's possible to read the state from the browser history (which is nullable), then use it later with a pushState or replaceState call. This is what was happening here.

Ahhh, of course, typical :/


support new extension point names format

WHY are these changes introduced?
Adds support for both the Admin::XX::XX syntax and the admin.x.x syntax when extracting the surface of an extension. This will allow to fix the issue where No extensions found for the surface: admin is returned in the embedded admin console.
WHAT is this pull request doing?
Just  changing the regex to support both :: or . as delimiter. (Can current extension points use a . that would influence this?)
Measuring impact
How do we know this change was effective? Please choose one:

[ ] n/a - this doesn't need measurement, e.g. a linting rule or a bug-fix
[ ] Existing analytics will cater for this addition
[ ] PR includes analytics changes to measure impact

Checklist

[x] I've considered possible cross-platform impacts (Mac, Linux, Windows)
[x] I've considered possible documentation changes
[x] I've made sure that any changes to dev or deploy have been reflected in the internal flowchart.

/shipit


decompiler: Remove 'using namespace std' from headers

closes #4703
I've added 'using namespace std' in '.cc' files to reduce diff and ease transition. Eventually, it would probably be good to remove it from the '.cc' files as well.
I did not regenerate code from the *.y and *.l files because it would cause more diffs than necessary (for this PR), but if you did regenerate, it would generate correct/compatible code.
The following header files contained using namespace std

opcodes.hh
capability.hh
error.hh
filemanage.hh
xml.hh
interface.hh
partmap.hh
marshal.hh

I understand this is a huge diff/PR, and I don't expect it to be merged/reviewed all at once because I assume there are internal efforts on some of these files that would cause major conflicts.
However, I hope this changeset can be considered for incremental integration through cherry-picking file changes that are not being touched internally and reviewing them a few at a time. I can update this PR and resolve conflicts incrementally as that happens.
Truthfully, the methodology used to make these changes was very manual, with a lot of compilation trial and error and text find-replace of C++ standard library symbols. I made a best-effort attempt to review the changes to ensure I only changed code symbols and not comments or program strings. Along with a manual review of all the changed files, I also searched the ///.*?std:: regex (to catch documentation comments). I also attempted to run the decompiler integration tests, but I received a lot of errors due to missing test files. I built Ghidra on this branch, analyzed some large files, and saw no errors.
There is growing interest in using the C++ decompiler as a standalone C++ library, and these changes will make that usage more robust and seamless.
Please let me know if there's anything you'd like to see changed to make this PR more acceptable.
Thank you for your time and consideration!

Implemented in c2522c81c6f7eae1c3ba63bc0995332816f15b24


[Template] with implicit operators broke compilation

Steps To Reproduce
https://deck.net/efa24e49e859f6ebb14dfd44234a74c4 <-- Replace with a link to your Deck
public class C
{
    [Template("({0})")]
    [Script("return c")]
    static public implicit operator C(string c)
    {
        return null;
    }
}

public class Program
{
    public static void Main()
    {
        var a = new C();
        C b = method();
    }

    static public string method()
    {
        return "";
    }
}

Actual Result
Bridge.assembly("Demo", function ($asm, globals) {
    "use strict";

    Bridge.define("Demo.C");

    Bridge.define("Demo.Program", {
        main: function Main () {
            var a = new Demo.C();
            var b =
    var $m = Bridge.setMetadata,
        $n = [System,Demo];
    $m($n[1].C, function () { return {"att":1048577,"a":2,"m":[{"a":2,"isSynthetic":true,"n":".ctor","t":1,"sn":"ctor"},{"a":2,"n":"op_Implicit","is":true,"t":8,"pi":[{"n":"c","pt":$n[0].String,"ps":0}],"tpc":0,"def":function (c) { return (c); },"rt":$n[1].C,"p":[$n[0].String]}]}; });
    $m($n[1].Program, function () { return {"att":1048577,"a":2,"m":[{"a":2,"isSynthetic":true,"n":".ctor","t":1,"sn":"ctor"},{"a":2,"n":"Main","is":true,"t":8,"sn":"Main","rt":$n[0].Void},{"a":2,"n":"method","is":true,"t":8,"sn":"method","rt":$n[0].String}]}; });
});

@AndreyZM, thanks for nice steps to reproduce. We will fix this defect in next releases.
A workaround is to add explicit cast to string:
https://deck.net/d9b0a2f80754b3158c7cd610698c1685
public class C
{
    [Template("({0})")]
    //[Script("return c")]
    static public implicit operator C(string c)
    {
        return null;
    }
}

public class Program
{
    public static void Main()
    {
        var a = new C();
        C b = (string)method();
    }

    static public string method()
    {
        return "";
    }
}


Unable to open up the zip file as it is password protected

Dear Author/Prajwal, Please share the password for zip file [for entire code]

can open now


Mining error with Testnet v19 RC.7

Describe the issue
Get an error message like this when trying to mine:
CalcCbTxMerkleRootMNList -- failed: RemoveMN: Can't delete a masternode a554045a49fb5e84a01af9e8bff2a3bf77846b27c36530a7552524f65164893a with a duplicate platformNodeID=0000000000000000000000000000000000000000
Can you reliably reproduce the issue?
If so, please list the steps to reproduce below:

Run 19-RC7 and try to mine

What version of Dash Core are you using?
v19.0.0-rc.7
Any extra information that might be useful in the debugging process.
Earlier today I registered a HPMN (TXID: a554045a49fb5e84a01af9e8bff2a3bf77846b27c36530a7552524f65164893a), then issued a protx revoke. Later I tried to register a new HPMN (TXID: 3979cfb79c4562e819aca69ffae2ea84b9b8f29bd89bdc68be67b88c6f31bf99) re-using the same collateral, but changing the necessary parameters (platform node id, IP, owner key). That second tx appears stuck in the mempool.

Unable to reproduce so far...

This should fix it

Fixed