Add access to Microsoft teams information in activity

It seems like Micosoft Teams embeds relevant information in "channelData" with its own schema.
This PR ports schema for Microsoft Teams from the dotnet SDK.
https://github.com/microsoft/botbuilder-dotnet/tree/master/libraries/Microsoft.Bot.Schema/Teams/Generated
This is quite important to have a bot to send proactive messages to a channel since this is the only way to access the real channel id which the bot has been added to.

@fredrikluo Thank you!  I'll take a closer look soon.  There was an entire effort to get Teams support on the other platforms that hasn't happened in Java yet.  It may be a bit before something happens with this PR while we schedule next steps for the Java SDK.


Repo tidying

Tidied a few things up in the repository :)
Please see individual commit messages for more details

Looks good to merge!

@adambuckland is this good to merge?

Yes, sir! Should still be good to merge


Feat: keychain rotate passphrase

This is a rough implementation supporting rotating keychain passphrases. I need some suggestions on how to proceed. I haven't received a reply so I've copied the original conversation here for reference:
async changeKeychainPassword(oldPassword, newPassword){
    if (typeof oldPassword !== 'string' || typeof newPassword !== 'string') {
      throw new Error(`Invalid pass type '${typeof oldPassword}'`);
    }
    if (newPassword.length < 20) {
      throw new Error('pass must be least 20 characters')
    }
    // TODO: Need to decide on how to import/generate opts
    const newDek = newPassword
      ? crypto.pbkdf2(
        newPassword,
        this.opts.dek.salt,
        this.opts.dek.iterationCount,
        this.opts.dek.keyLength,
        this.opts.dek.hash)
      : ''
    const oldDek = privates.get(this).dek
    privates.set(this, { "dek":newDek })
    var keys = await this.listKeys()
    await keys.forEach(async key =>{
      // TODO: Decide how to handle deleting and importing the "self" key.
          // importKey and removeKey throw an error when handling "self"
      if (key.name != "self"){
        var keyAsPEM = await this._getPrivateKey(key.name)
        await this.removeKey(key.name)
        this.importKey(key.name, keyAsPEM, oldDek)
      }
    })
  }

src/keychain/index.js
What's left is to decide how to handle the "self" key (this.importKey() and this.removeKey() throw an error on "self") and how to handle changing passwords in the context of src/index.js (see here). I'm trying to be cautious/minimal with the changes I make so I'd appreciate some suggestions on how to proceed.

@vasco-santos Please take a look at the new changes and let me know what you think. Regarding tests, I've incorporated testing for validation but I'm not exactly sure how to go about the following:
it('can rotate keychain passphrase', async () => {
    //TODO...
})

I think I've come up with a possible solution. If its ok with you then everything should be good to go. I also decided to create a new keychain here (instead of using ks) to avoid influencing any other/future tests that dont use the new rotated password.

Also, before submitting the PR try to get the CI green please 🙏🏼 I added small inline suggestion to fix the lint, but I probably did not get to all of them. You should run both npm run lint and npm run test

Sorry about that, I shouldve read the contrib guidelines more carefully. Anyway, I've ran lint and fixed any issues that I could trace back to me but I'm not sure If I can even get the CI to green because linting fails regardless of my edits. I've fixed the mistakes from the previous review and wrote the test as described.
However, I keep getting "Error: Cannot read the key, most likely the password is wrong or not a RSA key" here when testing on browser, despite it working fine in test:node. Am I missing/overlooking something?

Sorry about that, I shouldve read the contrib guidelines more carefully. Anyway, I've ran lint and fixed any issues that I could trace back to me but I'm not sure If I can even get the CI to green because linting fails regardless of my edits. I've fixed the mistakes from the previous review and wrote the test as described.

No problem, thanks for looking into it. I already submitted a suggestion fixing the lint. I will try to replicate the browser issue and get back to you

@vasco-santos Thanks for the help, I wasn't aware of that. I've removed the redundancy and everything works as expected now.

Released in 0.31.6

@vasco-santos Thanks for all the help!


Different color for $ in console syntax highlighting?

It's all feels a little too similar here.
Check out this Discord highlighting for comparison:

Here's GitHub's rendering:
$ foo # example stuff
bar

I like the different color personally. So :+1:


⚠ Spryr has degraded performance

In 2b1df9d, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 5375 ms

Resolved: Spryr performance has improved in 34e1eb4 after 4 minutes.


pack_install(tokenize) failed on macOS

Hi,
pack_install(tokenize).
only gave an option to use the .zip.  When I tried, it failed because of missing libraries on my mac.  Is there a list of dependencies I should install first?
Thanks.

Hi, @vjktm. Thanks for reporting your issue.
Installation should be self-contained, assuming there haven't been any breaking changes to swi-prolog standard libraries. I was just able to install on swi-prolog 8.4.3, so this shouldn't be an issue, afaik.
A zip file is indeed all that is downloaded, then swipl takes care of unpacking the source into your local packages.
Could you please report here

what version of swi-prolog you are running
the example command you used to install the library
the error output you received when you tried to to use_module

Thanks!

Hi @shonfeder , thanks for the response.
swi-prolog 8.5.20
command: pack_install(tokenize).
This time I got:
pack_install(tokenize). % Contacting server at https://www.swi-prolog.org/pack/query ... ok Install<EMAIL_ADDRESS>from https://github.com/shonfeder/tokenize/archive/v1.0.1.zip Y/n?  % Contacting server at https://www.swi-prolog.org/pack/query ... ok % "v1.0.1.zip" was downloaded 180 times Package:                tokenize Title:                  A simple tokenization library Installed version:      1.0.1 Author:                 Shon Feder<EMAIL_ADDRESS>Maintainer:             Shon Feder<EMAIL_ADDRESS>Packager:               Shon Feder<EMAIL_ADDRESS>Home page:              https://github.com/shonfeder/tokenize Download URL:           https://github.com/shonfeder/tokenize/release/*.zip Install "tokenize-1.0.1.zip" (16,717 bytes) Y/n?  % % PL-Unit: tokenize  % .................... done % % All 20 tests passed % % PL-Unit: tokenize  % .................... done % % All 20 tests passed % echo "(none)" % (none) true.
Then I did:
use_module(library(tokenize)). which worked.  It did not work without library/1
So, now it seems to work.
Previously it could not load some libraries, so I did the following in the mac terminal:
brew update
brew upgrade
I also did:
brew install libb2 since it could not load on the first try.
It seems all this fixed the problem.  Appears that my brew installation was not up to date.
I think my issue is solved!
Thanks for your response.
?- tokenize("what is your name?",Tokens). Tokens = [word(what), space(' '), word(is), space(' '), word(your), space(' '), word(name), punct(?)].
Thank you for your library and your response.


feat: add breaking change rules

:tada: This PR is included in version 1.1.0 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Lobby instructions always say !sau join

Bug Report
The lobby instructions don't respect the actual server prefix.
Steps to Reproduce

Change the command prefix of the bot
Start a lobby, or add a spectator.

Expected behavior
The "Join the Game!" instructions should use the server's default prefix.
Screenshots

This issue is resolved in version 2.6.0-beta.4

This issue is resolved in version 2.6.0


Export Options such as PDF, PNG, and SVG Formats

Describe the feature
The Export Options feature in Blox AI empowers users with versatile export capabilities by offering PDF, PNG, and SVG formats for sharing and printing purposes.
PDF Format: Ensures compatibility and high-quality output for sharing via email or embedding in documents.
PNG Format: Preserves graphics with a transparent background, ideal for social media sharing or web use.
SVG Format: Maintains sharp visuals for editing and scaling, suitable for designers and developers.
These export options cater to diverse user needs, ensuring seamless communication and flexibility in handling diagrams across various platforms and contexts.
Add ScreenShots

I will add an export option there and then accordingly the user can select how they want to save their work.
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@sanskriti-lal will be needing only in pdf and, need the pdf in a formated manner: -

First the document
Next at last the flowchart png.

will u be able to do so ??

Yea sure, then I'll just add the pdf option there.

@sanskriti-lal  Pdf format :
Title of the document
Body of the document
Flowchart image extracted from the Excalidraw

Do read the contributing.md and u can send the final result to me via whatsapp

Ok cool

please assign this to me


Test reports not merged when running "multiple" execution path and tests are failing

When ending up in multipleRunsExecute https://github.com/Ullink/gradle-nunit-plugin/blob/master/src/main/groovy/com/ullink/gradle/nunit/NUnit.groovy#L222 and exception is thrown in execute https://github.com/Ullink/gradle-nunit-plugin/blob/master/src/main/groovy/com/ullink/gradle/nunit/NUnit.groovy#L315 merging of test results are not done, i.e. outputFile https://github.com/Ullink/gradle-nunit-plugin/blob/master/src/main/groovy/com/ullink/gradle/nunit/NUnit.groovy#L236 will not exist afterwards.

The merging of tests results only works for nunit2 result format. So you need to specify restulFormat='nunit2' . Perhaps that is the issue?
Added a functional test that runs nunit in parallel so you can see a working example https://github.com/Ullink/gradle-nunit-plugin/pull/59 .

I think the problem is that it throws exception as nunit is returning a non-zero exit code when there are failing test cases, thus causing the merge skipped


db download :: download.ripcm.com/phigaro/ broken

Hello,
can you provide a mirror of download.ripcm.com/phigaro/.
google drive is not useable with command line tools. (wgat, curl) that breaks some automatic build mechanism of phigaro.
can you provide a wget-able (or curl-able) location for the dbs.
regards
Eric

thanks for this good news
and sorry for the late reply.
Eric

Hummm not sure it is currently possible to download data from
I have a ERROR 403: Forbidden

Sorry, for answering only now.
There was a server maintenance at that day, since that time it works
http://download.ripcm.com/phigaro/


Verify sensor calibration value

I wanted to calibrate the sensor but it's waaay to technical for me unfortunately.
The flow meter comes with this graph: http://www.microelectronicos.com/datasheets/UCTS0058.pdf

Could you tell me what a reasonable calibration value is based on this graph? Or does it not work like that?

I did a test with the default settings using 5 liters and it was off by almost 4 liters:
Currently 14.00 l/min, 8.94 l total.

Now on to calibrating..

I ended up using this for now:
const double cap = 30.0f;  // l/min
const double kf = 11.4f;   // Hz per l/min
FlowSensorProperties sensorConfig = {cap, kf, {1, 1, 1, 1, 1, 1, 1, 1, 1, 1}};
_sensor = new FlowMeter(_pin, sensorConfig);

but love to be able to calibrate it more (e.g. get rid of those 1's).

Kudos for attempting to calibrate! :)
The diagram ends at a flow rate of 9 l/min, but the true capacity is determined by how much you're able put through it without overwhelming it.
Did you really push 30 l/min through it?
That would be a lot for such a flow sensor, the impeller can start to tumble above it's true capacity, which would in turn lead to missed pulses.
You can actually calculate the k-factor from the diagram.
At 2 l/min it shows slightly above 20 Hz, thus about 11 Hz per l/min.
At 9 l/minit shows roughly 110 Hz, thus roughly 12 Hz per l/min.
So, if you ended up with a k-Factor of 11.4 Hz per l/min, you're pretty spot on with respect to the diagram.
Your sensor's datasheet has a table of 'electrical characteristics' that says [4.1Q] ±10% (in line 8, 'Flow pulse characteristics').
I would assume that this figure is intended to mean what I call k-Factor, which would indicate that the datasheet is inconsistent!

but love to be able to calibrate it more (e.g. get rid of those 1's).

They're actually helping you with that (if we want to trust the diagram again for a moment).
The diagram has a table showing corresponding flow rates l/min and pulse frequencies in Hz.
You can just divide them in order to get a starting point:
Assume a capacity of 10 l/min, since this is where the diagram ends.
2/22 = 1.0
3/33 = 1.0
4/44 = 1.0
5/55 = 1.0
6/67 = 0.08955223881
7/79 = 0.08860759494
8/91 = 0.08791208791
9/103 = 0.08737864078
10/114 = 0.08771929825

We're missing the value for 1 l/min but to get startet we assume it to similarly be 1/11 = 1.0.
So what we're ending up with as a starting point could be this:
const double cap = 10.0f;  // l/min
const double kf = 11.0f;   // Hz per l/min
FlowSensorProperties sensorConfig = {cap, kf, {1.0f, 1.0f, 1.0f, 1.0f, 1.0f, 0.08955223881f, 0.08860759494f, 0.08791208791f, 0.08737864078f, 0.08771929825f}};

awesome, thanks for the feedback. i will try it later today and post the results.

Hey @thijstriemstra, did the suggested calibration config work for you?

hi @sekdiy it didn't, my own parameters seemed to work better. But I'll revisit this hopefully this summer, otherwise next spring. Feel free to close the ticket though.

That's surprising. Was my calculation off or did the calibration code produce errors?
Closing in case you only get around to answering next year. :)
Feel free to open again as soon as you're ready!


Error implementing interface in IE8 in a complex inheritance hierarchy

The code example works fine in all other IEs and chrome, firefox. However in IE8 it throws an error. Whilst it is a wierd inheritance hierarchy, the inconsistency in IE8 is something a developer can spend ages trying to work out.
	test = {};

	test.RandomInterface = function(){};
	test.RandomInterface.prototype.random = function(){};

	test.CommonImplement = function(){};
	test.CommonImplement.prototype.commonInterfaceMethod = function(){};

	test.Class = function(){};
	topiarist.extend(test.Class, test.RandomInterface);
	topiarist.inherit(test.Class, test.CommonImplement);
	test.Class.prototype.classMethod = function(){};
	test.Class.prototype.commonInterfaceMethod = function(){};
	test.Class.prototype.random = function(){};

	test.ClassImplementor = function(){};
	topiarist.extend(test.ClassImplementor, test.Class);
	topiarist.inherit(test.ClassImplementor, test.CommonImplement);
	test.ClassImplementor.prototype.myMethod = function(){};
	test.ClassImplementor.prototype.commonInterfaceMethod = function(){};
	test.ClassImplementor.prototype.classMethod = function(){};

	var div = document.createElement("div");
	div.innerHTML="loaded";
	document.body.appendChild(div);

	new test.ClassImplementor();

Switching these two lines:
topiarist.extend(test.ClassImplementor, test.Class);
topiarist.inherit(test.ClassImplementor, test.CommonImplement);
to
topiarist.extend(test.ClassImplementor, test.CommonImplement);
topiarist.inherit(test.ClassImplementor, test.Class);
fixes the issue. Again painful to work out on your own.

This could be related to http://yourls.caplin.com/c where there is a stack overflow in IE8 for modules with dependencies chains deeper than 12.
A comment from the internal related issue:
In bootstrap.js, the mergePackageBlock and requireAll functions are appended to the window object.
In IE8, functions on the window object are limited to 12 calls in a single call stack. This means any modules with a dependency chain of 13 or greater depth will cause a maximum call stack error, as requireAll will be called 13 or more times.

I have spoken to @fauna5 and seeing as how we could not reproduce the failure, this will be closed as a non-issue.


KAFKA-6955: Use Java AdminClient in DeleteRecordsCommand

Committer Checklist (excluded from commit message)

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

LGTM

@ijuma @vahidhashemian I am not sure that we want to delete methods from scala AdminClient now. we should give users some time to migrate. KafkaAdminClient.deleteRecords API released in recent 1.1.0  release.
maybe we can drop the AdminClient at once after migrating all other methods (consumer group, api versions)?

@omkreddy The Scala AdminClient was never a public API and we removed it from the original KIP (KIP-107) as a result. In addition, we added documentation explaining that it was deprecated when the Java AdminClient was added in the 0.11.0 release. Are you aware of anyone using it?

@ijuma I am not aware of anyone using deleteRecordsBefore API. But I know few tools/companies using AdminClient methods for consumer group monitoring.
This is similar to AdminUtils.scala, which is not a public API, but used in some projects.
I just felt instead of removing methods, it may be appropriate to drop AdminClient at-once after migrating all the API.
I don't have strong reservations about deleting deleteRecordsBefore from scala client.

The main reason for doing it now is that we are doing a major version bump. And unlike the list groups functionality, deleteRecordsBefore was available for a short period in the Scala AdminClient before it became available in the Java AdminClient.

@ijuma, LGTM. Thanks for the tweaks :)

retest please

retest this please


update also the Gemfile.lock

I forgot about Gemfile.lock changes

oh yeah, I forgot we still have a Gemfile.lock file there too.


Add License information to Gemspec

This will make it show up on rubygems.org. I'm doing due diligence on our gems and need to find out the licenses for all the gems. Having it show up on rubygems.org cuts out the step of having to go to the github repo.

It's in the latest release candidate, but not in the latest stable release.  Of course, it's absurd that I haven't released a new stable gem in over two years (even though I've been using the RCs in production for the entire time).  Sorry!  That RC will be promoted to stable within a week.


Inability to distinguish genesis ledger accounts from post genesis accounts from GQL

There is currently no method to determine if an account was part of the genesis ledger or created afterwards. This distinction is crucial because accounts created after the genesis ledger incur a 1 Mina account creation fee. This would be useful to the frontend to show that a 1 Mina creation fee was incurred to create the account. This feature exists in MinaExplorer.

#1308 does not have a test for the genesis_ledger property, nor can I find this property in the explorer. Reopening.


upgrade website deps and fix the build bug

Description

Fixed the build issues
Upgrade website deps to latest stable version

Related Issues
#171
Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[ ] All existing and new tests are passing.
[ ] I updated the version in pubspec.yaml and CHANGELOG.md.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] The analyzer (flutter analyze) does not report any problems on my PR.
[ ] I read and followed the Flutter Style Guide.
[ ] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

Validated updates locally - everything builds fine and works correctly.
Thanks a lot @mhadaily


provide EvaluatableExpression by current document's wordpattern by default (instead of old fixed regular expression based heuristic)

#89084 added the option for either a language extension or a debug extension to registerEvaluatableExpressionProvider to work around the old fixed regular expression that led to issues because it can never match "any language". For more reasons while the old fixed regular expression is not good - see that issue.
The Debug hover API that was shipped with 1.43 is good as it allows extensions that know about the context or at least about expressions that could be evaluated to register themselves and implement something reasonable.
But the old issue stays: most documents already have, with its language definitions, a wordPattern - and as long as there is one I'm quite sure it is better than the general regex approach.
I therefore suggest to do the following:

check if an EvaluatableExpressionProvider is registered - in this case use that (we don't break any extensions that do so)
if this isn't the case: internally do the same as MockDebug, using the existing word definition of the current document for this information [I actually suggest to also have, depending on a new hoverInComments setting an automated filtering on DocumentSymbol as suggested by @weinand)
only if this isn't possible (is there any language definition without a wordPattern?), then use the old fixed regular expression

This should provide a much better "match" than the current one while still providing the option to do something "better" if there's an extension active for it.

Yes, I agree the word defintion would be a good default strategy. But changing this, is no longer an option because it would be a breaking change for all extension relying on the old behavior.
I suggest to file issue against those extensions that would profit from using the "word definition".

because it would be a breaking change for all extension relying on the old behavior

I wonder if there actually is one?
I'm very positive about that more extensions would benefit of this change than extensions that have an issue with that change (if any).
The point is to only use this if the "2020 new" option is not used, and if the old behavior is already deemed possibly less useful than the word definition (if it exist, if it doesn't then the old code would be used, too).
... and if you feel very strong about "possible more breaking than helping" we could have a setting defaultToHoverByWord that enables/disables this approach (and as those can be set language specific one could even configure it per language) - if you have that disabled by default (and therefore always use the old fallback) then there's no "external extension break" possible.


Hotfixes

Codecov Report

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

@@           Coverage Diff           @@
##           master      #73   +/-   ##
=======================================
  Coverage   37.47%   37.47%
=======================================
  Files          17       17
  Lines         451      451
=======================================
  Hits          169      169
  Misses        282      282

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2184ef9...0726713. Read the comment docs.


Difficult to use NodeJS with mraa and upm

The version of NodeJS shipped with Edison is 0.10. This is not really a stable version and not usable for serious things.
The upgrade to a more recent and stable version (even 0.12.7) is not easy and then you enter in a rather complicated voyage to have mraa and upm working with Node.
The best solution would be definitively ship with the Image at least a Node4 (LTS).
In the meantime, you should provide an end-to-end documentation on how to create a Node environment where you can do serous developments and test.

You're preaching to the choir (and tbh the wrong place communities.intel.com
is the right place)
I believe @alext's repo has 0.12.x pkgs. Until then you'll have to compile
your own node version. Sorry!
On 29 Apr 2016 7:33 p.m., "luigisaetta"<EMAIL_ADDRESS>wrote:
The version of NodeJS shipped with Edison is 0.10. This is not really a
stable version and not usable for serious things.
The upgrade to a more recent and stable version (even 0.12.7) is not easy
and then you enter in a rather complicated voyage to have mraa and upm
working with Node.
The best solution would be definitively ship with the Image at least a
Node4 (LTS).
In the meantime, you should provide an end-to-end documentation on how to
create a Node environment where you can do serous developments and test.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/intel-iot-devkit/mraa/issues/501

Well, I have already posted on communities.intel.com, and there the suggestion was: "open an issue on github".

Looking at https://communities.intel.com/thread/101617 need to differentiate the issues:

issue with old NodeJS version
issue with MRAA
issue with UPM
issue with MQTT

Here is a place to rise MRAA components' issues only.
Questions with NodeJS version issue in the Edison image need to address on communities.intel.com.
Information how to compile NodeJS you may find on https://nodejs.org

@luigisaetta, do you want to pursue mraa-related pieces further or should we close this one?

OK, you can close.


Fix MV-42 icons indicating mic and cam on/off

This PR fixed bugs from MV-42 (Display icons indicating whether someone's camera/mic is on or off).

BTW the history of this PR is a bit complicated, but instead of rewriting it you can squash and merge


Enable the creation / update of RuntimeCRs with Provisioner as managing system in KCP DEV and STAGE

Description
For preparing the rollout of KIM on our KCP landscape DEV + STAGE (not on PROD!), please configure in KEB for all plans following behaviour:

the controlling system for Shoot-Clusters is still the Provisioner (kyma-project.io/controlled-by-provisioner: "true", see PR)
please create / update also the RuntimeCR when sending a call to Provisioner

This will simulate the real behaviour later on production. But KIM will only run in "Dry-Run" mode (creating Shoot-Specs but these specs won't be considered by Gardener)  and we can compare KIM's produced Shoot-Spec with the Shoot-Spec which was created by the Provisioner.
Reasons
Allow testing of KIM by enabling it to create/update Shoot-Specs.
Attachments

Related to

https://github.com/kyma-project/infrastructure-manager/issues/250

Settings for creation of  Runtime CRs with kyma-project.io/controlled-by-provisioner: "true" on STAGE. For all plans.


Cannot Sign in when using packaged bundle

might because tauri cannot hosting a fullstack project

Yeah, it's because we have to start the nodejs for running Elk, but Tauri doesn't include Node.js 🤔

@danielroe and me will figure this out on Wednesday, if anyone wants to join I guess we can make that possible too

It's Thursday (in my timezone); did you figure it out?
(I don't see any commits or PR from yesterday - not trying to push; just curious)

This should be fixed now @danielroe did some fantastic fixes and even got signing in CI to work (check out the GH releases tab for a sneaky prerelease 😉)

Maybe we could improve the filename of release files. Adding OS name like darwin or macos / linux

Now it allows to login but once I approve permissions for app I get blank screen with following:

IIRC the scheme in this case is tauri:

Now it allows to login but once I approve permissions for app I get blank screen with following:

can you open a new issue for this and also provide the output of tauri info there?


KAFKA-9559: Change default serde to be null

Implementation of KIP-741 per KAFKA-9559. Changes the default serde from byteArray to null. This allows us to throw ConfigExceptions instead of NPEs which will give users a better idea of where their serdes are misconfigured.
Committer Checklist (excluded from commit message)

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

How confident are we, that we covered most cases to avoid NPE and throw ConfigException? Seems it's still many places that we need to cover, and I am wondering if we follow the right approach?
For example, instead of getting keySerde in some "outer layer" and forward a potential null that we check later, should we not pass around the config instead and call defaultKeySerde is we need it and let it through the ConfigException directly?
Atm, the code seems rather scattered and it might be better to try to unify it more.

I've definitely been relying on the test coverage to catch NPEs. That being said, I think it's been working pretty well. The tests have consistently been highlighting issues or bugs in the changes I made.
I think we can pass config around in some places (left a note for you about that on the direct comment), but I'm not sure that is enough to make this feel comprehensive / safe. Even if we pass around configs in the wrappingNullableUtils, we're still throwing ConfigExceptions in other places in the code. I'm not sure how to be confident that we won't hit NPEs other than relying on current code coverage. Thoughts? @mjsax and cc @ableegoldman


Can't transfer BTC from defi-app on defichain

I am trying to transfer all my coins back to cake since I can't add more liquidity to LPs in defi-app 2.1.4
USDT transfer ok.  DFI transfer ok. Still have a few DFI in defi-app wallet.
Issue: BTC transfer won't go.  Error is below.

I think I have enough DFI to pay for the transfer - I followed Daniel Zirkel's instructions to convert DFI tokens to coins

I'm having the same issue with BTC withdrawals.  Tried multiple times using multiple amounts over the past hour.  I get the same issue for ETH and USDT transfers as well.

Hi @jonjomatic , thanks for your feedback & screenshot. Could you kindly send your app logs. Please give us time to investigate this issue.

Hi,
The app logs should be included as a zip file attachment in my prior email.  I can resend if you don’t have them.
Regards,
Jon

On Dec 29, 2020, at 11:11 AM, Bella<EMAIL_ADDRESS>wrote:

Hi @jonjomatic , thanks for your feedback & screenshot. Could you kindly send your app logs. Please give us time to investigate this issue.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Here you go
On Tue, Dec 29, 2020 at 11:17 AM Jonathan Ho<EMAIL_ADDRESS>wrote:

Hi,
The app logs should be included as a zip file attachment in my prior
email.  I can resend if you don’t have them.
Regards,
Jon
On Dec 29, 2020, at 11:11 AM, Bella<EMAIL_ADDRESS>wrote:

Hi @jonjomatic https://github.com/jonjomatic , thanks for your feedback
& screenshot. Could you kindly send your app logs. Please give us time to
investigate this issue.
[image: 1 App log]
https://user-images.githubusercontent.com/65014479/103308070-87b9c200-4a4c-11eb-924f-e6273b940679.png
[image: 2 Compress]
https://user-images.githubusercontent.com/65014479/103308073-89838580-4a4c-11eb-8c0d-7b374bd9ed33.png
[image: 3 Drag and drop]
https://user-images.githubusercontent.com/65014479/103308083-8c7e7600-4a4c-11eb-8817-40e4bf8e9e03.png
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/DeFiCh/app/issues/195#issuecomment-752209424, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ASJJS7H5P47ITLFCYCAODZDSXISU5ANCNFSM4VMVPK4A
.

Hi @jonho79 , all right. I've informed the team to look out for your email. Alternatively, you can drag and drop it here so I can escalate this. Thank you.

App Logs JH.zip

OK drag and dropped app log

Hi @jonho79 , well received, please give us time to investigate this issue. Thank you.

Same issue here but with USDT

Hi - attached my logs now
app logs 20210101.zip

Hi - attached my logs now
app logs 20210101.zip

It is working with version 2.1.3

It is working with version 2.1.3

It is working with version 2.1.3

I rolled back to 2.1.3 and it didn't work for me

It is working with version 2.1.3

I rolled back to 2.1.3 and it didn't work for me

Some more logs @izzycsy - Also cannot transfer DFI out now, though I did manage to swap BTC -> DFI on the DEX in defi-app 2.1.4
app log 20200102.zip

Some more logs @izzycsy - Also cannot transfer DFI out now, though I did manage to swap BTC -> DFI on the DEX in defi-app 2.1.4
app log 20200102.zip

@jonjomatic well received, please give us time to investigate this. In the mean time, there’s a temporary solution to this, it could be that you've too little DFI UTXOs balance — which is required to perform swap and other functions. You could try convert a little DFI tokens to DFI UTXOs, and see if it works. Attached a link that might help you.
https://www.reddit.com/r/defiblockchain/comments/kb10d3/stepbystep_changing_dfitoken_to_dficoin/

@jonjomatic well received, please give us time to investigate this. In the mean time, there’s a temporary solution to this, it could be that you've too little DFI UTXOs balance — which is required to perform swap and other functions. You could try convert a little DFI tokens to DFI UTXOs, and see if it works. Attached a link that might help you.
https://www.reddit.com/r/defiblockchain/comments/kb10d3/stepbystep_changing_dfitoken_to_dficoin/

Hi izzycsy,
I have the same issue to send my BTCTOKEN to Cake. This my error-message: Not enough balance on wallet accounts, call utxostoaccount to increase it.

Hi izzycsy,
I have the same issue to send my BTCTOKEN to Cake. This my error-message: Not enough balance on wallet accounts, call utxostoaccount to increase it.

Hi,
I have the same issue with version 2.1.4 cant transer DBTC out of DEFI App to cake

BR

Hi,
I have the same issue with version 2.1.4 cant transer DBTC out of DEFI App to cake

BR

Hi @jonjomatic @jonho79 @sudden-break @telefonjoker @TTSIP , thanks for your feedback.
Update is out! https://defichain.com/downloads/

Please give a thumbs up to this comment if your issue has been resolved, if not, please send your app logs and we'll further investigate.

Hi @jonjomatic @jonho79 @sudden-break @telefonjoker @TTSIP , thanks for your feedback.
Update is out! https://defichain.com/downloads/

Please give a thumbs up to this comment if your issue has been resolved, if not, please send your app logs and we'll further investigate.

Hi izzycsy,
thank you for your recommendation. But the problem still exists with the version 2.1.5. How can I send you the app logs?

Hi izzycsy,
thank you for your recommendation. But the problem still exists with the version 2.1.5. How can I send you the app logs?

For me the same still exists with 2.1.5 i have 4 real DFI in the app so transaction fee is covered
Do we have to do something with the new version or just update it ?
BR

For me the same still exists with 2.1.5 i have 4 real DFI in the app so transaction fee is covered
Do we have to do something with the new version or just update it ?
BR

Hi @izzycsy - Happy new year to you.
Same for me, the update to v2.1.5 doesn't resolve the issue and confirm I have DFI coins (below).  Have attached the latest app log.  I couldn't attach the binary, even when zipped, github said the file is too big. Cheers.
>getbalance
11.00087626

app log 20210104.log

Hi @izzycsy - Happy new year to you.
Same for me, the update to v2.1.5 doesn't resolve the issue and confirm I have DFI coins (below).  Have attached the latest app log.  I couldn't attach the binary, even when zipped, github said the file is too big. Cheers.
>getbalance
11.00087626

app log 20210104.log

Hi @telefonjoker a guide on how-to-attach logs are in the comment at the beginning. Simply drag and drop. Uou'll have to use the temporary method if the error still persists https://www.reddit.com/r/defiblockchain/comments/kb10d3/stepbystep_changing_dfitoken_to_dficoin/. Please give the team to continue to look into it. Will await for your logs.

Hi @telefonjoker a guide on how-to-attach logs are in the comment at the beginning. Simply drag and drop. Uou'll have to use the temporary method if the error still persists https://www.reddit.com/r/defiblockchain/comments/kb10d3/stepbystep_changing_dfitoken_to_dficoin/. Please give the team to continue to look into it. Will await for your logs.

Hi @TTSIP , on the top right hand corner of your app, there should be an update button, if not you can replace it by downloading it from https://defichain.com/downloads/.

Hi @TTSIP , on the top right hand corner of your app, there should be an update button, if not you can replace it by downloading it from https://defichain.com/downloads/.

Hi @jonjomatic , happy new year to you too. Well received, I'll update the team on the latest app log sent. Thank you.

Hi @jonjomatic , happy new year to you too. Well received, I'll update the team on the latest app log sent. Thank you.

Hi Izzycsy,
here are my logs. Thank you for your efforts.
Archiv.zip

Hi Izzycsy,
here are my logs. Thank you for your efforts.
Archiv.zip

Hi @TTSIP , on the top right hand corner of your app, there should be an update button, if not you can replace it by downloading it from https://defichain.com/downloads/.

Hi here u have the logs
Apploggs.log
Binary.log
The Converting makes no sence ... because the issue is not that i would not have DFI coins ... i have the issue is that DBTC cant be withdrawen from APP to Cake ... (With DETH it works)
BR

Hi @TTSIP , on the top right hand corner of your app, there should be an update button, if not you can replace it by downloading it from https://defichain.com/downloads/.

Hi here u have the logs
Apploggs.log
Binary.log
The Converting makes no sence ... because the issue is not that i would not have DFI coins ... i have the issue is that DBTC cant be withdrawen from APP to Cake ... (With DETH it works)
BR

I have the same issue but my send button is greyed out and inactive, help please.
app_logs.log.zip

I have the same issue but my send button is greyed out and inactive, help please.
app_logs.log.zip

Not sure if this is related but I was having the same issue, not being able to send BTC from the app. With version 2.1.5 I was able to get the app to say the BTC was sent but have not received into cake after an hour. The app stopped loading transactions so tried closing/re-opening and now can't get app to load again. I was able to find the transaction on blockchain explorer though: https://explorer.defichain.com/#/DFI/mainnet/tx/8c1cc5651285582c9ea4d9e6ff3dcf0c3448ee6ce2cc0065d4a8fa264cb984e6

Not sure if this is related but I was having the same issue, not being able to send BTC from the app. With version 2.1.5 I was able to get the app to say the BTC was sent but have not received into cake after an hour. The app stopped loading transactions so tried closing/re-opening and now can't get app to load again. I was able to find the transaction on blockchain explorer though: https://explorer.defichain.com/#/DFI/mainnet/tx/8c1cc5651285582c9ea4d9e6ff3dcf0c3448ee6ce2cc0065d4a8fa264cb984e6

Updated to 2.1.5
Issue is still persisting and I am getting this statement "Not enough balance on wallet accounts, call utxostoaccount to increase it."
Here are my logs
App Log JH.zip

Updated to 2.1.5
Issue is still persisting and I am getting this statement "Not enough balance on wallet accounts, call utxostoaccount to increase it."
Here are my logs
App Log JH.zip

Hi @kwamelaryea & @AlanAdamson , thanks for your feedback. Your issue seems different, let me link you to the right on https://github.com/DeFiCh/app/issues/123. @kwamelaryea App logs are well received, please give us time to investigate. For future comments regarding unable to send coin, please comment in the link provided. Thank you.

Hi @kwamelaryea & @AlanAdamson , thanks for your feedback. Your issue seems different, let me link you to the right on https://github.com/DeFiCh/app/issues/123. @kwamelaryea App logs are well received, please give us time to investigate. For future comments regarding unable to send coin, please comment in the link provided. Thank you.

Hi @jonho79 , thank you for the update. Well received, please give us time to investigate. Thank you for your understanding.

Hi @jonho79 , thank you for the update. Well received, please give us time to investigate. Thank you for your understanding.

Same issue for me ever since v2.1.4
Swapped DFI tokens to coins following the reddit guide.
DFI to Cake = No problem.
ETH to Cake = "Not enough balance on wallet accounts, call utxostoaccount to increase it."
BTC to Cake = Haven't tried it yet.
App Log:
app-log.zip
Bin Log (in a 4x10mb zip file):
*!! Had to rename files for github upload file support. Rename files to:
bin-log-zip.001, bin-log-zip.002, bin-log-zip.003, bin-log-zip.004
bin-log.001.zip
bin-log-002.zip
bin-log-003.zip
bin-log-004.zip

Same issue for me ever since v2.1.4
Swapped DFI tokens to coins following the reddit guide.
DFI to Cake = No problem.
ETH to Cake = "Not enough balance on wallet accounts, call utxostoaccount to increase it."
BTC to Cake = Haven't tried it yet.
App Log:
app-log.zip
Bin Log (in a 4x10mb zip file):
*!! Had to rename files for github upload file support. Rename files to:
bin-log-zip.001, bin-log-zip.002, bin-log-zip.003, bin-log-zip.004
bin-log.001.zip
bin-log-002.zip
bin-log-003.zip
bin-log-004.zip

Same problem. Cannot transfer BTC to Cake. Same error message.
applogs.zip

Same problem. Cannot transfer BTC to Cake. Same error message.
applogs.zip

Hi @izzycsy @uzyn  - Is there any update to share?  Or is there a CLI workaround?
A bit uncomfortable to know that funds can't be moved if needed.

Hi @izzycsy @uzyn  - Is there any update to share?  Or is there a CLI workaround?
A bit uncomfortable to know that funds can't be moved if needed.

Hi @MadqaZ , app log well received, however I'm unable to open the binary logs. I've forwarded your app log to the team anyway.
@Workoholic-Jap , I'm unable to open your .zip file, could you resend them please? Thank you.
@jonjomatic I do not have CLI workaround at the moment. I apologise for the delay. The team is trying to fix the issue.
Thank you all for your patience.

Hi @MadqaZ , app log well received, however I'm unable to open the binary logs. I've forwarded your app log to the team anyway.
@Workoholic-Jap , I'm unable to open your .zip file, could you resend them please? Thank you.
@jonjomatic I do not have CLI workaround at the moment. I apologise for the delay. The team is trying to fix the issue.
Thank you all for your patience.

Hi @izzycsy How useful is the bin.log file for the team in terms of trying to figure out this issue? Do you really need it?
The bin.log file is ~150Mb, too big to share on Github, so other than zipping and splitting it into 10Mb volumes, all that is left is sharing it through something else than Github. I can make you a WeTransfer link, for example.
I just tested the file, and it works fine, although I do pack/unpack it with 7zip (https://www.7-zip.org/), not with the standard Windows compressor, since that one doesn't seem to recognize the split volume sequence. Also remember to rename the files as I mentioned before ;)

Hi @izzycsy How useful is the bin.log file for the team in terms of trying to figure out this issue? Do you really need it?
The bin.log file is ~150Mb, too big to share on Github, so other than zipping and splitting it into 10Mb volumes, all that is left is sharing it through something else than Github. I can make you a WeTransfer link, for example.
I just tested the file, and it works fine, although I do pack/unpack it with 7zip (https://www.7-zip.org/), not with the standard Windows compressor, since that one doesn't seem to recognize the split volume sequence. Also remember to rename the files as I mentioned before ;)

Hi izzycsy,
have you now been able to localize the problem?
BR
Claus

Hi izzycsy,
have you now been able to localize the problem?
BR
Claus

Update: v2.1.6

Still the same error when trying to send ETH to Cake account.
Still have to re-index at startup

app-log.zip

Update: v2.1.6

Still the same error when trying to send ETH to Cake account.
Still have to re-index at startup

app-log.zip

@izzycsy for your information:
The mistake comes up if u use teh dex in both ways: Example u swap DFI to BTC and afterwards u swap BTC to DFI
If u only swap it in one way withdraw works ... if u use both directions not.
Perhaps this helps the developers
Thanks
BR

@izzycsy for your information:
The mistake comes up if u use teh dex in both ways: Example u swap DFI to BTC and afterwards u swap BTC to DFI
If u only swap it in one way withdraw works ... if u use both directions not.
Perhaps this helps the developers
Thanks
BR

@TTSIP Interesting. I did use the Dex to swap DFI to BTC, but not the other way around. But I did swap ETH to DFI.

@TTSIP Interesting. I did use the Dex to swap DFI to BTC, but not the other way around. But I did swap ETH to DFI.

Hi @izzycsy
I found a solution for me.
my problem was that my tokens were spread over two addresses. One of these addresses was no longer visible, but a large part of the BTC tokens were stored on it. I created this address again in the app with "create new address". I deselected the button "Automatically create a new address" and was able to enter the old address again. "Now there was the following problem. Only BTC tokens and no DFI tokens were on this address. That's why I kept a small LM BTC-DFI in which the rewards were paid out exactly to this address. I now waited until the reward totaled 0.01 DFI. Now it is again possible to transfer this BTC token to Cake. The solution is that all addresses must have at least 0.01 DFI. Realizing this is unfortunately not easy.

Hi @izzycsy
I found a solution for me.
my problem was that my tokens were spread over two addresses. One of these addresses was no longer visible, but a large part of the BTC tokens were stored on it. I created this address again in the app with "create new address". I deselected the button "Automatically create a new address" and was able to enter the old address again. "Now there was the following problem. Only BTC tokens and no DFI tokens were on this address. That's why I kept a small LM BTC-DFI in which the rewards were paid out exactly to this address. I now waited until the reward totaled 0.01 DFI. Now it is again possible to transfer this BTC token to Cake. The solution is that all addresses must have at least 0.01 DFI. Realizing this is unfortunately not easy.

Hi @AlanAdamson @MadqaZ , update v2.1.6 is out https://defichain.com/downloads/.
Do give a thumbs up to this comment if the synchronisation issue has been resolved. Thank you.

Hi @AlanAdamson @MadqaZ , update v2.1.6 is out https://defichain.com/downloads/.
Do give a thumbs up to this comment if the synchronisation issue has been resolved. Thank you.

Update: The following issues are under investigation. Please give a thumbs down to confirm that I tagged you correctly in the issue you're facing, hope I didn't miss anyone out. Thank you.
Unable to send BTC
@jonjomatic @jonho79 @kwamelaryea @Workoholic-Jap @TTSIP
Unable to send ETH
@MadqaZ
Unable to send USDT
@sudden-break
For those that have not tried this, below are steps on how to transfer your coins from DeFi app to via <cakedefi.com>, you may try sending a small amount first to test.

Click on Balances in <cakedefi.com>
In the BTC row, click on Deposit button
You'll see the Transfer Network options BTC / DeFi Blockchain
Under the QR Code states your DeFi address, copy it
Go to DeFi app > BTC wallet > Send
Paste your DeFi address into To address field

Update: The following issues are under investigation. Please give a thumbs down to confirm that I tagged you correctly in the issue you're facing, hope I didn't miss anyone out. Thank you.
Unable to send BTC
@jonjomatic @jonho79 @kwamelaryea @Workoholic-Jap @TTSIP
Unable to send ETH
@MadqaZ
Unable to send USDT
@sudden-break
For those that have not tried this, below are steps on how to transfer your coins from DeFi app to via <cakedefi.com>, you may try sending a small amount first to test.

Click on Balances in <cakedefi.com>
In the BTC row, click on Deposit button
You'll see the Transfer Network options BTC / DeFi Blockchain
Under the QR Code states your DeFi address, copy it
Go to DeFi app > BTC wallet > Send
Paste your DeFi address into To address field

Thanks for the update @izzycsy logs attached below
app-logs.log.zip

Thanks for the update @izzycsy logs attached below
app-logs.log.zip

@izzycsy Unable to send ETH & BTC for me ;)

@izzycsy Unable to send ETH & BTC for me ;)

Not only am I unable to set BTC, I also can't send ETH or USDT.  I get the same error message for the failed transfers.
"Not enough balance on wallet accounts, call utxostoaccount to increase it."

Not only am I unable to set BTC, I also can't send ETH or USDT.  I get the same error message for the failed transfers.
"Not enough balance on wallet accounts, call utxostoaccount to increase it."

Hi @kwamelaryea , thanks for the latest log.
Hi @MadqaZ @jonho79 , could you resend your logs please?
Thanks for replying, the team is currently investigating the issue. Thank you for your patience.

Hi @kwamelaryea , thanks for the latest log.
Hi @MadqaZ @jonho79 , could you resend your logs please?
Thanks for replying, the team is currently investigating the issue. Thank you for your patience.

@izzycsy
app-log-2.1.6.log.zip

@izzycsy
app-log-2.1.6.log.zip

Sorry @izzycsy,
now I have the same problem as last week again. Not enough balance on wallet accounts, call utmost-account to increase it. But the accounts have enough amounts for me.

Should I send you the Logfiles again?
BR
Claus

Sorry @izzycsy,
now I have the same problem as last week again. Not enough balance on wallet accounts, call utmost-account to increase it. But the accounts have enough amounts for me.

Should I send you the Logfiles again?
BR
Claus

Hi @telefonjoker , thanks for your feedback. Yes please send your logs. May I know which coins are you unable to send?

Hi @telefonjoker , thanks for your feedback. Yes please send your logs. May I know which coins are you unable to send?

Hi @izzycsy, I would like to send the BTC´s to Cake. Here are my Logfiles.
BR
Claus
Logs_telefonjoker.zip

Hi @izzycsy, I would like to send the BTC´s to Cake. Here are my Logfiles.
BR
Claus
Logs_telefonjoker.zip

Hi @jonjomatic @jonho79 @kwamelaryea @Workoholic-Jap @TTSIP @MadqaZ @telefonjoker @sudden-break , sharing what @DanielZirkel posted recently, a temporary workaround for this issue, attached is the link that might help you https://www.reddit.com/r/defiblockchain/comments/kvsw4l/stepbystep_workaround_sending_btc_eth_usdt_and/. Do update if it worked for you. The team is trying to fix this issue, thank you for your patience.

Hi @jonjomatic @jonho79 @kwamelaryea @Workoholic-Jap @TTSIP @MadqaZ @telefonjoker @sudden-break , sharing what @DanielZirkel posted recently, a temporary workaround for this issue, attached is the link that might help you https://www.reddit.com/r/defiblockchain/comments/kvsw4l/stepbystep_workaround_sending_btc_eth_usdt_and/. Do update if it worked for you. The team is trying to fix this issue, thank you for your patience.

Hi all,
I can confirm that the work around through the console that DanielZirkel posted on reddit does work.  I was able to transfer some BTC back into my Cake account.  I havent tried for ETH or USDT but plan to in the next few days.  Thanks for alerting me to the work around,  much appreciated
Regards,
Jon

Hi all,
I can confirm that the work around through the console that DanielZirkel posted on reddit does work.  I was able to transfer some BTC back into my Cake account.  I havent tried for ETH or USDT but plan to in the next few days.  Thanks for alerting me to the work around,  much appreciated
Regards,
Jon

@izzycsy I see issues is being closed, but just to confirm, the workaround works for ETH too. Just successfully send some to my Cake account. Thank you all for the help!

@izzycsy I see issues is being closed, but just to confirm, the workaround works for ETH too. Just successfully send some to my Cake account. Thank you all for the help!

I tried to test the CLI workaround this afternoon by sending some DFI to cake.
Got a tx hash output OK.
So far it hasn't been acknowledged by cake as it usually would on receipt. Have sent to this cake DFI address from defi-app before.
Maybe there is a backlog for deposits at cake so will check again tomorrow.
-------- Original Message --------
On 13 Jan 2021, 22:19, MadqaZ wrote:

@izzycsy I see issues is being closed, but just to confirm, the workaround works for ETH too. Just successfully send some to my Cake account. Thank you all for the help!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

I tried to test the CLI workaround this afternoon by sending some DFI to cake.
Got a tx hash output OK.
So far it hasn't been acknowledged by cake as it usually would on receipt. Have sent to this cake DFI address from defi-app before.
Maybe there is a backlog for deposits at cake so will check again tomorrow.
-------- Original Message --------
On 13 Jan 2021, 22:19, MadqaZ wrote:

@izzycsy I see issues is being closed, but just to confirm, the workaround works for ETH too. Just successfully send some to my Cake account. Thank you all for the help!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hi @jonjomatic @jonho79 @kwamelaryea @Workoholic-Jap @TTSIP @MadqaZ @telefonjoker @sudden-break @Huzi1988 @mikej289 @floxcgncgn , update v2.1.7 is out https://defichain.com/downloads/.
Do give a thumbs up to this comment if your issue has been resolved, otherwise, you may reopen this issue with new information (include screenshots) and your app logs. Thank you.

Hi @jonjomatic @jonho79 @kwamelaryea @Workoholic-Jap @TTSIP @MadqaZ @telefonjoker @sudden-break @Huzi1988 @mikej289 @floxcgncgn , update v2.1.7 is out https://defichain.com/downloads/.
Do give a thumbs up to this comment if your issue has been resolved, otherwise, you may reopen this issue with new information (include screenshots) and your app logs. Thank you.

@izzycsy,
it works for me!
Thank you!

@izzycsy,
it works for me!
Thank you!

Good day support,
After the last update, I got a new message while transferring DEFI tokens in the console. Pls see attached
error.txt
App_config.zip

Good day support,
After the last update, I got a new message while transferring DEFI tokens in the console. Pls see attached
error.txt
App_config.zip

Hi @izzycsy the workaround doesn't work for me, it appears to be an address validation issue.
I have updated to version 2.1.7 but now I have some sync issues but I am restarting the wallet again to see if I can fix.

Hi @izzycsy the workaround doesn't work for me, it appears to be an address validation issue.
I have updated to version 2.1.7 but now I have some sync issues but I am restarting the wallet again to see if I can fix.

Good day support,
After the last update, I got a new message while transferring DEFI tokens in the console. Pls see attached
error.txt
App_config.zip

After a recent update to 2.1.8 still has the same issue, ps see attached log and error text file.
app_log_2.1.8.log.zip
defi 2.1.8 error.txt

Good day support,
After the last update, I got a new message while transferring DEFI tokens in the console. Pls see attached
error.txt
App_config.zip

After a recent update to 2.1.8 still has the same issue, ps see attached log and error text file.
app_log_2.1.8.log.zip
defi 2.1.8 error.txt

Hi @izzycsy
Can I reopen this issue, please? I have updated the app to version 2.1.7 but I still have the same issue. However, now I can't use the DEX to convert my BTC back to DFI and I get the following error "recipient () does not refer to any valid address" also the app hangs a bit on boot up and I get an unresponsive white screen.

I have attached new app logs
app_logs.log.zip

Hi @izzycsy
Can I reopen this issue, please? I have updated the app to version 2.1.7 but I still have the same issue. However, now I can't use the DEX to convert my BTC back to DFI and I get the following error "recipient () does not refer to any valid address" also the app hangs a bit on boot up and I get an unresponsive white screen.

I have attached new app logs
app_logs.log.zip

Hi @izzycsy
Can I reopen this issue, please? I have updated the app to version 2.1.7 but I still have the same issue. However, now I can't use the DEX to convert my BTC back to DFI and I get the following error "recipient () does not refer to any valid address" also the app hangs a bit on boot up and I get an unresponsive white screen.

I have attached new app logs
app_logs.log.zip

Try the latest, version 2.1.9 is available already

Hi @izzycsy
Can I reopen this issue, please? I have updated the app to version 2.1.7 but I still have the same issue. However, now I can't use the DEX to convert my BTC back to DFI and I get the following error "recipient () does not refer to any valid address" also the app hangs a bit on boot up and I get an unresponsive white screen.

I have attached new app logs
app_logs.log.zip

Try the latest, version 2.1.9 is available already

Good day support,
After the last update, I got a new message while transferring DEFI tokens in the console. Pls see attached
error.txt
App_config.zip

After a recent update to 2.1.8 still has the same issue, ps see attached log and error text file.
app_log_2.1.8.log.zip
defi 2.1.8 error.txt

After update 2.1.9, the same thing : Add-on auth TX failed: Insufficient funds

Good day support,
After the last update, I got a new message while transferring DEFI tokens in the console. Pls see attached
error.txt
App_config.zip

After a recent update to 2.1.8 still has the same issue, ps see attached log and error text file.
app_log_2.1.8.log.zip
defi 2.1.8 error.txt

After update 2.1.9, the same thing : Add-on auth TX failed: Insufficient funds

Good day support,
After the last update, I got a new message while transferring DEFI tokens in the console. Pls see attached
error.txt
App_config.zip

After a recent update to 2.1.8 still has the same issue, ps see attached log and error text file.
app_log_2.1.8.log.zip
defi 2.1.8 error.txt

After update 2.1.9, the same thing : Add-on auth TX failed: Insufficient funds

Hi @izzycsy, any news about above

Good day support,
After the last update, I got a new message while transferring DEFI tokens in the console. Pls see attached
error.txt
App_config.zip

After a recent update to 2.1.8 still has the same issue, ps see attached log and error text file.
app_log_2.1.8.log.zip
defi 2.1.8 error.txt

After update 2.1.9, the same thing : Add-on auth TX failed: Insufficient funds

Hi @izzycsy, any news about above

@kwamelaryea It appears that you are attempting to send BTC to a Bitcoin network address. BTC on DeFiChain is a DeFi Standard Token (DST) that is only transmittable to other DeFiChain addresses (usually starting with 8 or d).
Do swap from DST BTC to BTC on Bitcoin, you can go through sites like cakedefi.com or swap DST BTC to DFI on DEX.

@kwamelaryea It appears that you are attempting to send BTC to a Bitcoin network address. BTC on DeFiChain is a DeFi Standard Token (DST) that is only transmittable to other DeFiChain addresses (usually starting with 8 or d).
Do swap from DST BTC to BTC on Bitcoin, you can go through sites like cakedefi.com or swap DST BTC to DFI on DEX.

Hi @MarsaXod @kwamelaryea , update v2.1.10 is out https://defichain.com/downloads/.
 Thank you.

Hi @MarsaXod @kwamelaryea , update v2.1.10 is out https://defichain.com/downloads/.
 Thank you.

hi all, need help. I am unable to swap DFI for BTC via Dapp. Any advise?
Defi App log.log

hi all, need help. I am unable to swap DFI for BTC via Dapp. Any advise?
Defi App log.log

Hi @1230-1 , thanks for the screenshots and feedback, you're facing this issue DEX: recipient () does not refer to any valid address, I'll link you to https://github.com/DeFiCh/app/issues/305 which we can further discuss on the problem. Let’s keep this thread related to what the original author opened. Thank you for your understanding.
Logs well received.

Hi @1230-1 , thanks for the screenshots and feedback, you're facing this issue DEX: recipient () does not refer to any valid address, I'll link you to https://github.com/DeFiCh/app/issues/305 which we can further discuss on the problem. Let’s keep this thread related to what the original author opened. Thank you for your understanding.
Logs well received.

I cannot send out, the continue button doesn't light up to be selected, something is bad with lots of money in the balance=/

I cannot send out, the continue button doesn't light up to be selected, something is bad with lots of money in the balance=/

Hi @kwamelaryea @MadqaZ @rxandr , update v2.1.12 is out https://defichain.com/downloads/.
Do give a thumbs up to this comment if your issue has been resolved, otherwise, kindly provide with new information on your app's behaviour (include screenshots) and app logs. Thank you.

Hi @kwamelaryea @MadqaZ @rxandr , update v2.1.12 is out https://defichain.com/downloads/.
Do give a thumbs up to this comment if your issue has been resolved, otherwise, kindly provide with new information on your app's behaviour (include screenshots) and app logs. Thank you.


Include date of scheduled maintenance in email

Currently when scheduling a maintenance, the sent email doesn't contain the maintenance date. Would be nice if the subscriber could see the date also in the email itself.

Are we sure about this? I'm looking at the code and we have a scheduled_at https://github.com/cachethq/Cachet/blob/master/resources/views/emails/incidents/maintenance-html.blade.php#L10

Maybe we are missing a text saying the scheduled maintenance will be at?

looks like it was added in a later version: https://github.com/cachethq/Cachet/commit/b68647df8eba8b8450f55a6ebe52e2fdd1027676#diff-daf631c07d2ae386bb01155338fcc1a1

That commit is included in v2.0.0-beta1, v2.0.0-beta2, v2.0.0-RC1, and future releases.


How to use with ollama

I am running llama3 in the terminal.
I have openAI.sublime-settings as:

// Settings in here override those in "OpenAI completion/openAI.sublime-settings"
{
	"url": "http://localhost:11434",
	"token": "sk-your-token",
	"model": "llama3",
}

but when I invoke new message, whatever I type in the "Question:" bottom panel returns: "OpenAI error
model 'gpt-4-0613' not found, try pulling it first".

Seems like you're running "OpenAI: New Message" right from the start instead of to run "OpenAI: Chat Model Select" first to select a model from the list.

https://github.com/yaroslavyaroslav/OpenAI-sublime-text#ai-assistance-use-case please check this part of readme for detailed instructions.

I did try that, but didn't see llama3 in the menu. (I can scroll down a bit further, but it's not there.) I thought since I specified "model": "llama3" it would be in the menu or be set in any case.

Yeah, now I see.
You have a few issues within your config. Here's the correct example:
{
    "url": "http://localhost:11434",
    "token": "sk-your-token",
    "status_hint": [
        "name",
        "prompt_mode",
        "chat_model"
    ],
    "assistants": [
        {
            "name": "Local llama assistant",
            "model": "llama3",
            "assistant_role": "You are a senior code assistant",
            "prompt_mode": "panel"
        }
    ]
}

Please pay attention that now the "assistants" array appears within the setup and assistant related settings were moved there. The second thing to consider is that there are 4 required settings to make things work (like to add the next to it predefined assistant), they are "name", "model", "assistant_role", "prompt_mode".
I'd recommend to read comments in settings when you'll have time for obtaining more in depth knowledge of how this thing work https://github.com/yaroslavyaroslav/OpenAI-sublime-text/blob/master/openAI.sublime-settings#L47-L73

BTW: Here was the error:
...
RuntimeError: unable to instantiate 'OpenAI completion.openai_panel.OpenaiPanelCommand'
Traceback (most recent call last):
  File "/Applications/Sublime Text.app/Contents/MacOS/Lib/python38/sublime_plugin.py", line 535, in create_window_commands
    o = cls(window)
  File "/Users/reagle/Library/Application Support/Sublime Text/Installed Packages/OpenAI completion.sublime-package/openai_panel.py", line 20, in __init__
    self.load_assistants()
  File "/Users/reagle/Library/Application Support/Sublime Text/Installed Packages/OpenAI completion.sublime-package/openai_panel.py", line 26, in load_assistants
    self.assistants: List[AssistantSettings] = [
  File "/Users/reagle/Library/Application Support/Sublime Text/Installed Packages/OpenAI completion.sublime-package/openai_panel.py", line 27, in <listcomp>
    AssistantSettings(**{**DEFAULT_ASSISTANT_SETTINGS, **assistant})
TypeError: __init__() got an unexpected keyword argument 'model'

The above exception was the direct cause of the following exception:

RuntimeError: unable to instantiate 'OpenAI completion.openai_panel.OpenaiPanelCommand'
Empty file I belive
Empty file I belive
...

Yep, there was a mistake in the provided snipped above fixed it.


Upgrade go to 1.22.4

Recently terraform-docs raised their minimum Go version to 1.22.1, which has caused some of my autoupdate PRs to fail:
build attempt 1...
    => error
    [INFO] Installing environment for https://github.com/terraform-docs/terraform-docs.
    [INFO] Once installed this environment will be reused.
    [INFO] This may take a few minutes...
    An unexpected error has occurred: CalledProcessError: command: ('/opt/go/bin/go', 'install', './...')
    return code: 1
    stdout: (none)
    stderr:
        go: downloading go1.22 (linux/amd64)
        go: download go1.22 for linux/amd64: toolchain not available
    Check the log at /pc/pre-commit.log
build attempt 2...
    => error
    [INFO] Installing environment for https://github.com/terraform-docs/terraform-docs.
    [INFO] Once installed this environment will be reused.
    [INFO] This may take a few minutes...
    An unexpected error has occurred: CalledProcessError: command: ('/opt/go/bin/go', 'install', './...')
    return code: 1
    stdout: (none)
    stderr:
        go: downloading go1.22 (linux/amd64)
        go: download go1.22 for linux/amd64: toolchain not available
    Check the log at /pc/pre-commit.log

I based this off #231, and collected the sha from here.

this looks good -- I unfortunately need to wait a bit to merge and release this due to a pretty annoying regression in cpython


integration: Handle when genSpend tries to spend immature outputs

genSpend in the integration test may sometimes try to spend immature outputs because of the hardcoded 25btc spend.  Lower the hardcoded amount to 20 and make it ok to fail when trying to spend as that doesn't really affect the overall test.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
9 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.02%) to 53.266%

Files with Coverage Reduction
New Missed Lines
%

addrmgr/network.go
2
73.58%

peer/peer.go
2
74.75%

database/ffldb/db.go
5
91.2%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
27538

Relevant Lines:
51699

💛  - Coveralls


update to package:web

Fixes #28

Using web.crypto.subtle.importKey will report an error in web worker mode, and still use js interop mode. But the dart:html dependency is removed, and this modification passed the test


Readthedocs points to wrong version of source code for .postN release

I though I fixed this in #836, but it looks like readthedocs is still pointing to the main branch for v0.9.3.post2.  Not a big deal, but flagging it here for fixing in the next release.

I did a bit of looking into this.  Here's what I think is happening:

In the build process, readthedocs checks out the code using the command
git clone --no-single-branch --depth 50 https://github.com/python-control/python-control .
This gets the latest code, but that code does not have the version set => we end up with __version__ being set to 'dev'.
In doc/conf.py, if version is 'dev' then the source code links point to the main branch.

Possible fixes:

Checkout the code via pip
Figure out if there is a tag on the commit and use that to set the version number


PSPs not available for assignment to Projects

This page says that PodSecurityPolicies can be assigned to Projects and their member namespaces. It also recommends against doing so.
I'm unable to find a way in 2.3.5 to do this, and I suspect that it was removed without updating the docs.

disregard. it only shows up if PSP support is enabled in the cluster.


Sharp issue in docker image

I am having issue running Sharp in docker image
Something went wrong installing the "sharp" module
/app/bundle/programs/server/npm/node_modules/sharp/build/Release/sharp.node: invalid ELF header

Ensure "linux" is used at install time as well as runtime

Consult the installation documentation at https://sharp.pixelplumbing.com/en/stable/install/

Search for this error at https://github.com/lovell/sharp/issues

Could someone please help me out with this
I have a meteor app that I build on mac using :
meteor npm install --arch=x64 --platform=linux sharp
meteor build --server-only --directory $1 --architecture os.linux.x86_64
However when I run the docker container I get above error and I am unable to start. the app.
Any help here will be hugely appreciated thanks

This looks like an old version of sharp, please can you upgrade.


Fixed usage of helpers

Had the same problem as https://github.com/jaredly/hexo-renderer-handlebars/issues/2. Root cause was that the overridden helpers were not called (the original ones were used). Please review my fixes.

looks reasonable. Thanks!


Make use of traffic volumes

Some initial ideas that volumes could be used for:

present predictions in relation to volumes (x crashes per volume y)
identify the extent to which volumes contribute to risk

As pointed out this is predicated on having access to a data source, which right now we're hoping will be Waze. We have applied to be part of the Research Program and I'll update this card as soon as we hear something back.


Can't pass paths via command-line

As we're preparing to update grunt-ts to support TypeScript 2.0's features, it has come to our attention that it's not possible to pass paths via the command-line.
tsc --paths  returns  error TS6064: Option 'paths' can only be specified in 'tsconfig.json' file.
This is a problem for us as we only operate via the command-line and support a more dynamic experience via the Grunt options (which can be determined programmatically, etc.).
User @agubler reported this to us here: Issue: https://github.com/TypeStrong/grunt-ts/issues/339
Is --paths not supported on the command-line just because it's hard to reliably accept JSON from the command-line?  Would you consider a pull request to add this?

Any time you put informatoin inside of a build config and not in the tsconfig.json, you break TypeScript's editor support (e.g. the editor also wants to know which files are built).  If your tool really wants to dynamically generate file lists, the most user-supportive thing to do would be to generate a tsconfig.json (and check it in to the project repo).

Hi,
I appreciate this feedback and I agree with your sentiment with regards to using source-controlled build artifacts.
That said, an important feature of tools like Grunt-ts is allowing developers to customize their build system to use a configuration that is different than their editing environment if they want to, and that's what I'm asking about.

@evmar our use case the project tsconfig.json file is up to date and valid, but using grunt-ts to create variations from this base project file depending on the grunt configuration. The plain dev pipelines uses the tsconfig "as is" however for creating a the distribution we use different output and flags, additionally a user could also have pipelines for different targets (es5, es6 etc).

Is --paths not supported on the command-line just because it's hard to reliably accept JSON from the command-line? Would you consider a pull request to add this?

sorry for the delayed reply. yes. it is only supported in tsconfig.json. so is roots

Would you consider a pull request to add this?

Not sure it is worth the hassle. we have to come up with a serialized format for paths, and that can be complicated, and very hard to read. unless you say it is JSON.

@agubler consider using the new configuration inheritance feature added in TS 2.1 instead.


Default value for

It would be helpful to list the default value for the <nullable> option. I believe all the other options on this page have a default listed.

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

ID: e548e000-8190-b1bb-51fd-7e7b2d5bcb45
Version Independent ID: 19f59267-c4c1-eb0e-7e48-45ffd6e2d9d1
Content: C# Compiler Options - language feature rules
Content Source: docs/csharp/language-reference/compiler-options/language.md
Product: dotnet-csharp
Technology: csharp-language-reference
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Thanks for pointing this out @mrlife
The default is disable.
I've added this to our backlog, and added the "up for grabs" label to this issue.

Hi @BillWagner, I've picked this up and added the extra detail required.  PR submitted for review.


test..com is valid domain

$result = $extract->parse('test..com');
$result->isValidDomain();  // gives true

Registrable Domain ist .com in this case... I think this domain name should be invalid.

Thanks for report, valid issue. I will fix it before 1st April.

Thanks for report, fixed in the latest release 1.2.4. Please check :+1:


buggy analyzer at pub.dartlang.org ?

Today (25.10.2018) many flutter packages at pub.dartlang.org ended up analyzed as UNIDENTIFIED, with errors like

line 14 col 62: Undefined class 'State'.

Here are some:
https://pub.dartlang.org/packages/adhara#-analysis-tab-
https://pub.dartlang.org/packages/firestore_ui#-analysis-tab-
https://pub.dartlang.org/packages/mvc_pattern#-analysis-tab-
https://pub.dartlang.org/packages/photos_library#-analysis-tab-
https://pub.dartlang.org/packages/signature#-analysis-tab-
last one is mine and yesterday was the same version analyzed as completely healthy package
I can see that analyzer is from today checking against upgraded versions of flutter (0.9.5 -> 0.10.1), but I am using my plugin in flutter app that depends on Flutter 0.10.1 and all is working fine.

Confirm
There is the same issue in pub
https://github.com/dart-lang/pub-dartlang-dart/issues/1728


Agent does not use workflow service account

The agent should use spec.serviceAccountName, but instead it always uses default.

@sarabala1979 could I ask you to fix this please?


PP-363: Decisions search filter fixes

Drupal setup

On your local drupal installation, checkout this branch: PP-363-search-filter-fixes
Run make drush-cim drush-cr
If you don't have any data, run make ahjo-migrations (NOTE: you'll need the local proxy URL for this one. Also, edit the user permissions on the test site and add the following permissions for anonymous users: "Access external documents from Ahjo API")
Rebuild and reindex the decisions index: https://helsinki-paatokset.docker.so/fi/admin/config/search/search-api/index/decisions

To test

Checkout branch and start the app with npm start (make sure the index is set for decisions)
Read the code changes. This has a lot of changes to the category and DM selection, so those parts of the code should especially be checked

Date as selected filter

Test the date selector. The selected values should appear as a tag in the selected filters area
You should be able to reset the date search by deselecting the tag

All decisionmakers filter, deselecting decisionmakers

There should be a new item in the decisionmakers list: "Kaikki päättäjät"

Selecting this acts the same as having the filter empty, but should appear as a tag and shouldnt cause any errors

You should now be able to deselect the selected item in the decisionmaker dropdown by clicking it again

ID based decisionmaker and category selection

Decisionmaker and top category selection should now be based on the indexed "Sector ID" and "Top category code" fields
Compare the fi and sv search results to the version on the test site:

https://nginx-paatokset-test.agw.arodevtest.hel.fi/fi/paattajat
https://nginx-paatokset-test.agw.arodevtest.hel.fi/sv/beslutsfattare

On the test site, you should get vastly different results when filtering by category and decision maker (not the special case dm selections)
On the local search app, the results should be more similar between language versions (they might vary by a few decisions, though)
Also test that the URL query parameters work by reloading the page after adding filters

* When de-selecting predefined dates (last week, last month etc.) by pressing the pill, the selection remains in the dropdown.

This should be fixed now. Can you verify that it works for you too?

* Drupal dropdown styles seems to be having some side effects on the policymakers dropdown: https://nginx-paatokset-test.agw.arodevtest.hel.fi/fi/paattajat (hovering selection causes both bg and text colors to be black)

Those should be fixed on the local search fixes branch.


Move what-is-kubernetes to overview page

Move https://kubernetes.io/docs/concepts/overview/what-is-kubernetes/ to https://kubernetes.io/docs/concepts/overview/
This PR also makes sure that:

a new redirect
make sure that the table of contents is no longer rendered

Fixes: #35251

/label refactor
Changes like this need a discussion across the team.

/assign @kcmartin

Once this PR has been reviewed and has the lgtm label, please assign kcmartin for approval


SQL request with LIMIT statement should be allowed

Describe the bug
The SQL importer does not allow requests containing a LIMIT statement.
It is safe to forbid keywords like UPDATE or DROP, but LIMIT should be allowed
Mathieu
To Reproduce
Try to import data from a MySQL database containing a (table_name) table : connect to the database, and make a SELECT * FROM (table_name) LIMIT 10;

Go to '...'
Click on '....'
Scroll down to '....'
See error

Current Results
What results occured or were shown.
Expected behavior
A clear and concise description of what you expected to happen or to show.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS, Windows 10, Linux, Ubuntu 18.04]
Browser Version: [e.g. Chrome 19, Firefox 61, Safari, NOTE: OpenRefine does not support IE but works OK in most cases]
JRE or JDK Version:[output of "java -version" e.g. JRE 1.8.0_181]

OpenRefine (please complete the following information):

Version : OpenRefine 3.1

Datasets
If you are allowed and are OK with making your data public, it would be awesome if you can include or attach the data causing the issue or a URL pointing to where the data is.
If you are concerned about keeping your data private, ping us on our mailing list
Additional context
Add any other context about the problem here.

The LIMIT and OFFSET clauses are used internally in OR  to restrict the number of rows in the preview and create project SELECT queries.

@tcbuzor yes we understand that - but users should still be allowed to set their own LIMITs and OFFSETs - the code should gracefully handle these (for instance, by replacing a user-provided LIMIT by a smaller LIMIT during preview).

I think it would be reasonable for LIMITs and OFFSETs to be set via a specific control if necessary rather than within a query (thinking this could make it easier to manage rather than having to parse LIMIT/OFFSET statements out of a user authored query)

More broadly the problem with the current code is that it chunks queries manually with OFFSET and LIMIT when importing a project from a SQL query. This sort of chunking can be quite inefficient, depending on the query. I think it would be preferable to use the existing JDBC API for that, with setFetchSize and setMaxRows, to let the JDBC stream the query for us directly. In the case of PostgreSQL at least, this is much more efficient as the driver will use server-side cursors to stream the query.

Let's do it as @ostephens suggested. It make sense for me. Maybe we can optimise for different DB if the performance is a concern.


Support DRF 3.12.x

The code was already 99% compatible with DRF 3.12.x since the Pipfile  support DRF ">=3.7.0,<3.13"
However, djangorestframework version support wasn't changed in the setup.py file

Coverage decreased (-0.6%) to 88.006% when pulling 866fd3f43520476a31410d0886ecde0b121685c2 on alexandredufour:feature/support-drf-3.12.x into 162e360bdc0be4a8dde0e5bd10f653b2b740d221 on rhblind:master.

@rhblind Thanks for the quick merge.
Much appreciated.
Do you have any idea about when this will be available in a PyPi release ?

@alexandredufour Yes, I'll try to make a release before the weekend 👍

Just pushed it to pypi now 👍


unexpectedly found nil while unwrapping an Optional value

Inside Statement.swift
this code
public subscript(idx: Int) -> Blob {
        let bytes = sqlite3_column_blob(handle, Int32(idx))
        let length = Int(sqlite3_column_bytes(handle, Int32(idx)))
        return Blob(bytes: bytes!, length: length)
    }

lead to unhandled exception!
The cell stores empty value.

fatal error: unexpectedly found nil while unwrapping an Optional value

Please, fix it.
I've fixed by this:
public subscript(idx: Int) -> Blob {
        let bytes = sqlite3_column_blob(handle, Int32(idx))
        guard let b = bytes else {
            return Blob(bytes:[UInt8]())
        }
        let length = Int(sqlite3_column_bytes(handle, Int32(idx)))
        return Blob(bytes: b, length: length)
    }```

Looks like you're using an old version, this has already been fixed in 0.11.2 (#561)


Support devcontainer feature build/test/publish.

A large part of the ecosystem of devcontainers is the use of features to extend the environment, and potentially the IDE.  Currently the scripts here do not support building, testing, or publishing devcontainer features, meaning either the feature must be baked into the base image (hindering reusability), or creating custom scripts to directly call the cli.  This organization does have an additional repository that contains a GitHub action that can handle features, but unfortunately doesn't support azure at all.

@Clockwork-Muse - if I've understood correctly, you're looking for something like the functionality of https://github.com/devcontainers/action but for Azure DevOps?

Yeah... but this repo doesn't support this feature for GitHub actions either...

This repo (devcontainers/ci) is focussed on running build steps inside a dev container, not on managing features/templates.
The devcontainers/action repo is targetted at packaging dev container features and templates (but only has a GitHub action)
I wanted to make sure I understood which functionality you are looking for. It sounds like it's the feature testing/publishing flow that you are interested in (but for Azure DevOps)?

Yes, primarily.


SalesChannel Configuration inheritance does not apply

PHP Version
8.0
Shopware Version
<IP_ADDRESS>
Actual behaviour
You can configure the plugin ThumbnailPattern storewise using the standard shopware config inheritance and apply a different pattern per sales channel, but in the frontend only the "All Sales Channel" config gets applied
Expected behaviour
The configuration I applied to the sales channel is applied for the thumbnail pattern and other configuration
Steps to Reproduce?
Remove inheritance for a single sales channel in plugin configuration /admin#/sw/extension/config/FroshPlatformThumbnailProcessor - set a different value - clear cache & recompile themes - check frontend for the thumbnail url pattern.
The SystemConfigService#get takes the sales channel id as a secondary parameter, so it should be injected into https://github.com/FriendsOfShopware/FroshPlatformThumbnailProcessor/blob/master/src/Service/ThumbnailUrlTemplate.php#L37

Thank you. I will have a look at it soon.
Meanwhile, feel free to create a PR :-)

Hey @florianliebig
I just had time to take a look into it.
There is no possibility to add media per saleschannel and there might be problems with import/exports adding this feature into the ThumbnailProcessor.
Keep in mind: everyone (plugins f.e.) requesting thumbnails will have to announce any saleschannelId to have it working. Sure, we might fail over to "all saleschannels" but there will occur unwanted behaviours.
That is the reason I think of disallowing setting this per saleschannel for this plugin.
What do you think?

Some more thoughts...
On the other hand... why not adding this feature for thumbnailprocessor :-) Should 3rd-parties just announcing the saleschannelId


Update PodmanDesktop.download.recipe

Hi, @gregneagle
The PodmanDesktop recipes are currently failing with
Processor: URLTextSearcher: Error: No match found on URL: https://api.github.com/repos/containers/podman-desktop/releases/latest

This PR has updated regex
Output from a successful -v run
autopkg run -v PodmanDesktop.munki.recipe
Looking for com.github.autopkg.gregneagle-recipes.download.PodmanDesktop...
Did not find com.github.autopkg.gregneagle-recipes.download.PodmanDesktop in recipe map
Rebuilding recipe map with current working directories...
Looking for com.github.autopkg.gregneagle-recipes.download.PodmanDesktop...
Found com.github.autopkg.gregneagle-recipes.download.PodmanDesktop in recipe map
**load_recipe time: 0.008504416007781401
Processing PodmanDesktop.munki.recipe...
WARNING: PodmanDesktop.munki.recipe is missing trust info and FAIL_RECIPES_WITHOUT_TRUST_INFO is not set. Proceeding...
URLTextSearcher
URLTextSearcher: Found matching text (url): https://github.com/podman-desktop/podman-desktop/releases/download/v1.14.1/podman-desktop-1.14.1-universal.dmg
URLDownloader
URLDownloader: Storing new Last-Modified header: Tue, 05 Nov 2024 10:25:54 GMT
URLDownloader: Storing new ETag header: "0x8DCFD843B03A713"
URLDownloader: Downloaded /Users/paul.cossey/Library/AutoPkg/Cache/com.github.autopkg.gregneagle-recipes.munki.PodmanDesktop/downloads/podman-desktop-1.14.1-universal.dmg
EndOfCheckPhase
CodeSignatureVerifier
CodeSignatureVerifier: Mounted disk image /Users/paul.cossey/Library/AutoPkg/Cache/com.github.autopkg.gregneagle-recipes.munki.PodmanDesktop/downloads/podman-desktop-1.14.1-universal.dmg
CodeSignatureVerifier: Verifying code signature...
CodeSignatureVerifier: Deep verification enabled...
CodeSignatureVerifier: Strict verification not defined. Using codesign defaults...
CodeSignatureVerifier: /private/tmp/dmg.jXvmIY/Podman Desktop.app: valid on disk
CodeSignatureVerifier: /private/tmp/dmg.jXvmIY/Podman Desktop.app: satisfies its Designated Requirement
CodeSignatureVerifier: /private/tmp/dmg.jXvmIY/Podman Desktop.app: explicit requirement satisfied
CodeSignatureVerifier: Signature is valid
Versioner
Versioner: Mounted disk image /Users/paul.cossey/Library/AutoPkg/Cache/com.github.autopkg.gregneagle-recipes.munki.PodmanDesktop/downloads/podman-desktop-1.14.1-universal.dmg
Versioner: Found version 1.14.1 in file /Users/paul.cossey/Library/AutoPkg/Cache/com.github.autopkg.gregneagle-recipes.munki.PodmanDesktop/downloads/podman-desktop-1.14.1-universal.dmg/Podman Desktop.app/Contents/Info.plist
MunkiImporter
MunkiImporter: Using repo lib: AutoPkgLib
MunkiImporter:         plugin: FileRepo
MunkiImporter:           repo: /Users/Shared/munki_repo
MunkiImporter: Copied pkginfo to: /Users/Shared/munki_repo/pkgsinfo/utilities/PodmanDesktop-1.14.1.plist
MunkiImporter:            pkg to: /Users/Shared/munki_repo/pkgs/utilities/podman-desktop-1.14.1-universal-1.14.1.dmg
Receipt written to /Users/paul.cossey/Library/AutoPkg/Cache/com.github.autopkg.gregneagle-recipes.munki.PodmanDesktop/receipts/PodmanDesktop.munki-receipt-20241111-165645.plist

The following new items were downloaded:
    Download Path
    -------------
    /Users/paul.cossey/Library/AutoPkg/Cache/com.github.autopkg.gregneagle-recipes.munki.PodmanDesktop/downloads/podman-desktop-1.14.1-universal.dmg

The following new items were imported into Munki:
    Name           Version  Catalogs  Pkginfo Path                          Pkg Repo Path                                         Icon Repo Path
    ----           -------  --------  ------------                          -------------                                         --------------
    PodmanDesktop  1.14.1   testing   utilities/PodmanDesktop-1.14.1.plist  utilities/podman-desktop-1.14.1-universal-1.14.1.dmg

Thanks. I noticed the Podman.munki recipe had failed this morning, and I was going to find out who wrote it to see if they'd updated it :-)

Excellent timing! 😆


Allow users to specify measurement types in table template

In GitLab by @nwalczak on Aug 19, 2019, 10:51
Instead of just specifying number of rows, specify specific measurement types.
This will allow us to fill in information about expected controls.  For instance, with a flow measurement, we know there will be beads, and with RNAseq there will be a WT.  We can use this to create auxiliary tables to help with parsing.

In GitLab by @nwalczak on Sep 25, 2019, 12:46
created branch 87-allow-users-to-specify-measurement-types-in-table-template

In GitLab by @nwalczak on Sep 25, 2019, 12:46
mentioned in merge request !60

In GitLab by @nwalczak on Sep 25, 2019, 12:50
created branch 87-allow-users-to-specify-measurement-types-in-table-template-2

In GitLab by @nwalczak on Sep 25, 2019, 12:50
mentioned in merge request !61

In GitLab by @nwalczak on Sep 26, 2019, 16:29
closed via merge request !61

In GitLab by @nwalczak on Sep 26, 2019, 16:29
closed via commit 2a253187750f793e9c96e7bb1e7f940aebb1e1c5

In GitLab by @nwalczak on Nov 25, 2019, 11:21
mentioned in commit 2a253187750f793e9c96e7bb1e7f940aebb1e1c5

In GitLab by @tramyn on Nov 25, 2019, 11:43
@nwalczak @jakebeal Could we go ahead and delete this branch now that a release has been made for milestone 2.1?

In GitLab by @tramyn on Nov 25, 2019, 11:48
I spoke too early.
It looks like there were no tagged release made for milestone 2.1 and we jumped from 2.0 to 2.2 milestone.
Since this issue has been resolved and closed, I would like to go ahead and close the branch related to this issue.
Before I do this, @nwalczak and @jakebeal are you both opposed to this decision?

In GitLab by @jakebeal on Nov 25, 2019, 12:31
I'm fine with it if @nwalczak doesn't think anything will be lost.


remove weird file

https://github.com/WalletConnect/walletconnect-website/blob/master/wallet.io
I think this just slipped in shomehow?

hmm didn't notice this... totally must've slipped through a PR.
Removed -> https://github.com/WalletConnect/walletconnect-website/commit/51f483c38a09125508a8a9d46d92d90e923b5adb


Video sometimes a single frame with MutliStreamRecorder?

Hey,
I'm running off of master and using chrome (Version 46.0.2490.86 (64-bit) - on OSX 10.9.5) but there's about a 1/3rd? ish chance of the video blob being a single frame... Audio always seems to work correctly, but video capture is random. You can see in the screenshot below that sometimes the video blob is around 13kb, and sometimes 1.3mb.

@muaz-khan Any thoughts on what might be causing this?

Just an update on this, I dug into the issue and the root cause seems to be related to:
whammy.frames = dropBlackFrames(internalFrames, -1);
Thoughts?


[OOB] Guide for RKE to RKE2 Windows Migration

There will be docs PR for this, so this issue is exempt from "code freeze" requirements.

https://github.com/rancher/docs/pull/4217


Add renteeAddress to rent data key

Currently, we are using only nft address and token id for the unique key of rent data
But, there'd be a error for multiple result for that unique key, because many user can rent the same NFT with nft address and token id.
So, we've to add one more key with rentee address.

Add renteeAddress argument to these three functions.

unrentNFT()
settleRentData()
getRentData()

Apply to rent market and rent content and other service


Add Develop section, add link in We need your help

Coverage remained the same at 88.3% when pulling 70d7e2822beb9f73e9aa0bbc1a53e583f4d6ab97 on bennlich:patch-1 into 8a0c95bdf0eb15c8e14f5431d60545df253b0dbf on cantino:master.

Coverage remained the same at 88.3% when pulling 70d7e2822beb9f73e9aa0bbc1a53e583f4d6ab97 on bennlich:patch-1 into 8a0c95bdf0eb15c8e14f5431d60545df253b0dbf on cantino:master.

Thanks @bennlich!


fix(pipeline): remove unneeded cfn exports

The pipeline package was exporting CodeRepositoryGRCUrl and SonarqubeSecretArn which meant the stack could only be deployed once per region. Nothing seems to be importing these exports. After discussing with the maintainers, we determined that simple outputs are sufficient.
BREAKING CHANGE: CodeRepositoryGRCUrl and SonarqubeSecretArn are no longer CloudFormation exports. Any systems that relied on those being exports will need to be updated to look up the output value by the stack name.
Fixes #239

Thanks for submitting this!


Some photos not shown when navigating between folders

When making an observation in gallery screen: when changing the folder once selecting a photo and then switching back to the first folder, a portion of the youngest photos is not shown.

@jtklein I can't replicate this, and I'm also not sure what portion of the newer photos are not being shown. Can you enumerate your repro steps? I know everyone's phone is going to be different but everyone starts out in "Camera Roll" and probably has a "Camera folder.

So, I narrowed it down and can replicate every time on my phone: When changing to a folder and then back to the first folder the photos that are younger than the youngest one in the second folder are not shown. I have made a screen recording on my iPhone with the version from TestFlight.
https://user-images.githubusercontent.com/17345891/206203741-e925dd55-1d87-4e95-b9e2-dacd4a54125b.mp4