[Skype Web Control] Wrong behaviour when using dialogAction button

Bot Info

SDK Platform: Node.JS
SDK Version: 3.14.0
Active Channels: Skype, SWC
Deployment Environment: Azure App Service
App-ID: 6cd7cd5b-47ad-40a8-a319-9dd7846977b7
SWC SDK: https://swc.cdn.skype.com/sdk/v1/sdk.min.js

Issue Description
My bot is deployed both in Skype and in Skype Web Control. When using Skype Web Control version, users get the wrong behavior when clicking in dialogAction buttons.
Code Example
exports.createWelcomeCard = function (session) {
    var welcome = new builder.HeroCard(session)
        .title('Olá!')
        .subtitle('Pretende aceder à sua área pessoal?')
        .text('Na área pessoal poderá consultar o estado das suas faturas.')
        .images([
            builder.CardImage.create(session, 'xxx')
        ])
        .buttons(
        [
            builder.CardAction.dialogAction(session, "access_login", "", "Sim "),
            builder.CardAction.dialogAction(session, "get_goodbye", "", "Não ")
        ]);
    return welcome;
};

Reproduction Steps

Talk to bot
See Welcome Card
Click "Sim"
Expected behaviour shoud be  a prompt "Por favor indique-me..." but when in SWC the welcome card shows up again

Expected Behavior
The image below shows the expected behaviour as seen on the Skype App.

Actual Results
When in Skype Web Control (SWC), the bot always shows the same welcome card after clicking "Sim" or "Não".

Any news on this issue ?

Can you please share your code that handles the clicked responses?

@JasonSowers sure, no problem. So after clicking "Sim" ("Yes") the dialog flow will be:
 bot.beginDialogAction('access_login', 'login'); which will trigger the
// Login message flow     bot.dialog('login', [         function (session) {             session.beginDialog('askNif');         },         function (session, results) {             // All EU VATs accepted             var nif = results.response.toUpperCase();             session.sendTyping();             api.login(nif)                 .then(result => {                     // Save user variables to TableStorage                     if (result.length === 1) {                         session.userData[config.tableStorage.supplierID] = result[0].supplier_id;                         session.userData[config.tableStorage.supplierName] = result[0].supplier_name;                         session.userData[config.tableStorage.nif_key] = result[0].nif;                         console.log(result[0].lastUpdate);                         console.log(session.userData[config.tableStorage.lastUpdate]);                         if (session.userData[config.tableStorage.lastUpdate] === undefined || session.userData[config.tableStorage.lastUpdate] === ' Informação não disponível') {                             if (result[0].lastUpdate !== undefined) {                                 session.userData[config.tableStorage.lastUpdate] = result[0].lastUpdate;                             } else {                                 session.userData[config.tableStorage.lastUpdate] = ' Informação não disponível';                             }                         }                         session.send("✔ NIF validado para " + result[0].supplier_name);                         session.send("⏰ Última actualização da base de dados: " + session.userData[config.tableStorage.lastUpdate]);                         session.beginDialog('homepage');                     } else {                         session.beginDialog('retryNif');                     }                 }).catch(err => {                     if (err.error === 'No NIF') {                         session.beginDialog('retryNif');                     }                     else {                         errorHandler(err, session);                         session.beginDialog('/');                     }                 });             session.endDialog();         }     ]);
This dialogflow is basically the same for all buttons and works pretty well in the Skype App and in the emulator.

@JasonSowers any news? We want to deploy this chatbot to our clients also on SWC as well but right now it doesn't seem the best thing to do...

Hi @fms-santos
Please provide the code you use to show this welcome card.  Is it in response to an event, or response to a user's message?

Here you go. It's basically the root dialog. When a user first writes anything to the bot, the bot goes to:
bot.dialog('/', [
        function (session) {
            if (session.userData[config.tableStorage.supplierID]) {
                session.beginDialog('homepage');

I must emphasize again that everything works as expected on the Skype app for Windows, iOS, Android and also on the emulator and web chat.

@fms-santos we're looking into the issue now! Do you have a test page where we can try this bot?

Yes, I do have. http://mockedpinfospot.azurewebsites.net/

@brene it seems that it's fixed. Can you confirm? What was the issue? Should I change something on my side?

@fms-santos we're still trying to figure out what's wrong and what's the difference between our implementation and the native Skype client. Sorry for the inconvenience.

Hey @brene and @JasonSowers , I was wondering if this issue is still being solved or at least taken into account.

@fms-santos I cannot speak for @brene and his team but I will reach out and see what I can find out.

@fms-santos I was waiting for the SWC team to respond, but you have been waiting forever on this issue.  They have put bot related development on the back burner for the time being so I would not count on this being fixed anytime soon.  I have no timeline or information other than that, unfortunately.

Ok thank you @JasonSowers . That's a shame.

@brene @JasonSowers This is a serious bug in the functionality of SWC I really hope they will take the time to fix it soon.

At this time there is no new information or timeline I have for you.  I do hope this gets addressed and will continue to poke around about it periodically.  Unfortunately, it's not something in our code base that we could just fix for you guys.

@JasonSowers thanks Jason, I really hope your pokes will work eventually.

Thank you for opening an issue against the Bot Framework SDK v3. As part of the Bot Framework v4 release, we’ve moved all v3 work to a new repo located at https://github.com/microsoft/botbuilder-v3. We will continue to support and offer maintenance updates to v3 via this new repo.
From now on, https://github.com/microsoft/botbuilder repo will be used as hub, with pointers to all the different SDK languages, tools and samples repos.
As part of this restructuring, we are closing all tickets in this repo.
For defects or feature requests, please create a new issue in the new Bot Framework v3 repo found here:
https://github.com/microsoft/botbuilder-v3/issues
For Azure Bot Service Channel specific defects or feature requests (e.g. Facebook, Twilio, Teams, Slack, etc.), please create a new issue in the new Bot Framework Channel repo found here:
https://github.com/microsoft/botframework-services/issues
For product behavior, how-to, or general understanding questions, please use Stackoverflow.
https://stackoverflow.com/search?q=bot+framework
Thank you.
The Bot Framework Team


Constrain phenotypes by multiple conditions

Now #56 is done do we want to be able to constrain by multiple phenotype conditions?

The server side code supports this so to implement this we just need to decide what the user interface should look like.

Could this  look like Canto Condition selection?

See also https://github.com/pombase/website/issues/1463
for alleles that do not have a particular condition.

Could this look like Canto Condition selection?

I wasn't able to implement that in the same way as Canto, but I've done something similar.  It's all working, but I won't deploy the change until we've all had a chance to test it and discuss it.  We might need help text and a documentation update.

Is this on dev? I don't see it looking like this.

It was tricky to enable just for the dev site so it's only available on my desktop at the moment: https://desktop.kmr.nz

I think this change is ready to go.  I've done some more testing and it all seems OK.
It's running on my desktop if you'd like to give it a try: https://desktop.kmr.nz/

I couldn't test because the phenotype search box doesn't appear to be working?

Sorry, I was working on something else.  It should be OK now.

I think this change is ready to go. I've done some more testing and it all seems OK.

I decided that the easiest plan is to release the change to the dev server: http://dev.pombase.kmr.nz/query
If it's all OK, I'll update the main site.

The excluded conditions are dev-site only because they aren't working properly.  It's still using the old system.

OK! I was on dev.
So you still need to make the excluded conditions behave like the  present conditions?
Also we need different headers , they are all constraints:
Constrain conditions
Present
Excluded
?

So you still need to make the excluded conditions behave like the present conditions?

Yep.
I'd like to have a chat on Zoom about excluding conditions to make sure it's useful before implementing it.

It might also be useful to constrain on allele type (i.e. deletion) so that you can retrieve all annotations to a phenotype without, for example over-expression.

This is almost implemented so we should figure out what we still need to do in the next couple of weeks if possible

The remaining question is how things should work if multiple conditions are excluded.  For example if you exclude "high temperature" and "+ NaCl" should that filter out an annotation with just "high temperature" as a condition?  Or should it only filter out annotations with both "high temperature" and "+ NaCl" as conditions?
But maybe I'm overthinking this.  :-)

I think it should be both, but let's chat about it

we can chat about this tomorrow!

I just tried and I don't thin the advances search is working on your desktop (tried FYPO and GO)

If you need to test the excludes constraints, the dev site is a better bet.

For Kim:

disable constraining on multiple conditions
enable exclude constraints

disable constraining on multiple conditions
enable exclude constraints

Done.

Thanks!

The “Exclude condition” option restricts the results to exclude any gene with phenotype annotations to the phenotype term of interest including the specified condition, for any allele, but only where they are not present with another condition.

I think we should re-word this slightly to be clear that we exclude annotations not genes.
How's this:
The “Exclude condition” option excludes from the results any annotations to the phenotype term of interest that include the specified condition

for any allele, but only where they are not present with another condition.

I don't understand that bit.

Let's refine tomorrow...

The “Exclude condition” option excludes from the results any annotations to the phenotype term of interest that include the specified condition (now that annotations to the same gene with a different condition, or no condition specifies will remain).

The docs now have the changes we discussed on Zoom.

I think this is done?

Yep, one small suggestion. Would it be possible to have a "I" icon right next to the "contain condition" option linking to the appropriate place in the docs, since its a bit complicated?

Would it be possible to have a "I" icon right next to the "contain condition" option linking to the appropriate place in the docs, since its a bit complicated?

Good plan.  I've made that change to the code.  The main site will be updated in 30 minutes or so.

Good plan. I've made that change to the code. The main site will be updated in 30 minutes or so.

Done.


Static closures resolving

Fixes #1636 PHP Warning, wich still actual in all PHP versions.

Coverage remained the same at 100.0% when pulling addaf3e8f0e179839e50908893014b7adfc9d4c4 on ElisDN:static-closure into 090498eb089ef328b3ef96b4ad33478684fbf062 on slimphp:4.x.

Sorry... It does not work correctly in global scope...

See https://github.com/slimphp/Slim/pull/3067#discussion_r616037003


Sent Ethernet frames may have extra octets at the end

Packet buffer data after the IP packet is not removed before the packet is output, resulting in extra bytes in the outgoing Ethernet frame after the IP payload data. This occurs also with VxLANs.
One way to see this is to forward very short IP packets from an untagged ethernet interface to a VLAN interface. In that case the received Ethernet padding is not removed and after VLAN tag insertion the outgoing frame is longer than it should be.

Commit c226ac7 fixes the issue for IPv4 packets but not for IPv6 packets.


prorab-src-dir problem

$(call prorab-src-dir, src) lists sources twice because " src" is passed with preceding space. Add protection from that

done


Why is it that when I run hellokan, it runs so much differently than expected

Here's the result of my run

I think this happened because there is still some hidden degree of random in the code (between device). You can try increasing the training set or changing up the seed of the create_dataset() function

Same here. Is it possible to be instability with floating point in the implementation of LBFGS?

Adding this line in the beginning helps
torch.use_deterministic_algorithms(True)

This should be solved with the most up-to-date version. Temporarily close this issue.

This should be solved with the most up-to-date version 0.2.0. Close this issue for now.

thank you


Inaccurate prize breakdowns.

Because the current contract handles everything in wei and the lack of decimals in solidity the prices are not always distributed 100%.
Solution would to use a larger unit.

Don't need to worry about this, users would be working with much higher units that wei. So having the contract deal with wei is ideal.


updated maintainers.md (xDnsServer)

Pull Request (PR) description
This PR is to update the xDnsServer maintainer within the Maintainers.md markdown file.
This Pull Request (PR) fixes the following issues
N/A
Task list

[x] This PR changes documentation

This change is

@gaelcolas Can you look at this PR and add required permissions.


Common code for both client and server

This is a (please label accordingly):

[X] tech debt

Find a sensible way to share code between client and server

[ ] shared js modules
[ ] assets generated by client tools that need to be injected into server templates

We have the config module, that's shared.
https://github.com/wellcometrust/wellcomecollection.org/tree/master/config


Volume scroll: Only when a hotkey is held

Thank you for this extension and carrying on some useful features I really loved. I was wondering if it would be possible to implement a feature/option that only changed volume when scrolling if right click (or any hotkey really) was held down while scrolling. I find myself often accidentally changing volume without this and it would be a nice-to-have thing.
Thanks again for the effort, its greatly appreciated.

@ArchiveAnon that should be possible. Right now I'm working on improving some of the code. I'm also making tests for the code to make sure everything is working before releasing the next version.

@ArchiveAnon would the alt or ctrl key work? if I want to get this new feature out fast those two would be the easiest otherwise it will take a bit longer.

Honestly, any key works. Just can be a bit of an annoyance if you scroll and accidentally mute the video you're watching or blast the volume haha. Thanks for taking the time to do this!

Alright, the easiest way for me to add this feature is "ctrl", "alt" or "shift" key press. Because in the event for the scroll wheel there are properties that tell if those keys are pressed when scrolling.
https://github.com/VampireChicken12/youtube-enhancer/blob/dev/src/features/scrollWheelVolumeControl/index.ts#L30

Although I can do it the harder way which will allow for a wider range of keys to choose from.
https://keyjs.dev/#keycodes-table

If you have Discord feel free to add me to talk about this new feature and other features. vampirechicken is my Discord username

I decided that the following keys will be alright
Alt key, Ctrl key, Shift key

@ArchiveAnon version 1.12.0 includes "Scroll wheel volume control modifier key feature"


Extraction missing activations

article link: https://www.sciencedirect.com/science/article/pii/S0028393218300265?via%3Dihub
coordinate extracted by recent run of ACE: 10.0  48.0  38.0
article on neurosynth-compose: https://compose.neurosynth.org/base-studies/5SXcXCJzaNx6/3LhSKuCuFRkR

from the recent scrape, there are only coordinates from the final table:
https://academic.oup.com/cercor/article/11/9/796/294145?login=true
910  26.0  42.0  32.0
911  48.0  19.0  23.0
912  50.0  19.0   2.0
913  45.0  21.0  15.0


[docs] Start a doc on ClangImporter C->Swift name mapping

Includes enums in all their flavors, plus swift_wrapper structs.
Planned future content:

Methods, initializers, properties

"Omit-needless-words" rules
NSError
Boolean properties
Inferred default arguments
Factory methods as initializers

Protocols (mainly just that they check for collision)
CF types (low-priority since non-Apple people can't make them)
Custom names, including import-as-member and import-as-accessor
swift_private

@swift-ci Please smoke test

swift/AST/SwiftNameTranslation.h is for Swift-to-ObjC, not ObjC-to-Swift. Most of what's talked about here is in swift/ClangImporter/ImportName.h, but not all of it.

@swift-ci Please smoke test

ah, OK. Notice that both directions of translation are interesting for cross-language automatic renaming.

Notice that both directions of translation are interesting for cross-language automatic renaming.

I think I'll rename the file to CToSwiftNameTranslation, leaving room for a future SwiftToCNameTranslation.

I think I'll rename the file to CToSwiftNameTranslation, leaving room for a future SwiftToCNameTranslation.

Sounds good, Thank you!

@swift-ci Please smoke test

We've got three approving reviewers so far, so if anyone else has comments we can take it in post-commit review.


[New game]: Emoji Enigma

🎮 Game Request
Game logic and basic description
Emoji Enigma is an engaging memory card game that challenges players to match pairs of emojis hidden behind cards. The game board consists of a grid of cards, each containing a unique emoji. Players must remember the location of the emojis as they flip cards to uncover matches. The objective is to clear the board by matching all pairs within the shortest time possible.
How to Play:
Click on any facedown card to reveal the emoji underneath.
Click on a second card to see if it matches the first.
Match all pairs of emojis to complete the game.
Enter your username to record your score on the leaderboard
Point down the features
Game points

Memory Challenge: Test your memory skills by remembering the positions of emojis as you flip cards.

Leaderboard: Compete with friends and other players by achieving top scores on the leaderboard.
Dynamic Gameplay: Each playthrough shuffles the emojis, providing a fresh challenge every time.

Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @Roshr2211 !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign

Hey @Roshr2211 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Minimal PR to enable appveyor support

I push it to my fork to see what happens on appveyor. see https://ci.appveyor.com/project/SteffenForkmann/visualfsharp/build/<IP_ADDRESS>

It needs some work, am working on it - I've got it enabled in my fork too, e.g.https://ci.appveyor.com/project/dsyme/visualfsharp/build/<IP_ADDRESS>.
I created the PR to clear the idea with @latkin, who agrees with the direction in principle as an interim

@latkin can you please activate this project in appveyor?

Appveyor should now be enabled (@KevinRansom did it)
We have a problem running Perl on the CI server, see https://ci.appveyor.com/project/dsyme/visualfsharp/build/<IP_ADDRESS>#L1003, not sure what the error is, perhaps its just not installed (or a wrong version)
Will try the other unit tests now

Great - we have "Waiting to hear about b310e2a — Waiting for AppVeyor build to complete" in this PR now, thanks @KevinRansom  :)

I know time is a factor, but riht now it doesn't look like any portable tests are either compiled or run. The portable stuff is a frequent culprit for build issues, so that would be nice to get some coverage on.

@latkin I agree.  Even having a single portable build would be nice.  Would doing only the most restrictive portable build (259) as a start suffice?
My experience has always been that it will show most problems that arise in any of the other targets.

From looking at candidate build.cmd, seems that some of the profiles are indeed included when building the runtime, I was only speaking of the tests (which suffer from similar build issues on occasion).

AFAIK we have a 30minute window on AppVeyor - I may be wrong (that may be for Travis).  We are pretty close to that.  Let's see where we are, get some of each of the test suites enabled including running all of the core unit tests, and then add stuff.

FYI: You get 40 minutes on AppVeyor, at any level of account.  If we were
to upgrade to paid accounts, the build system would also be quicker, as the
hardware used gets nicer.
On Wed, Jan 28, 2015 at 5:23 PM, Don Syme<EMAIL_ADDRESS>wrote:

AFAIK we have a 30minute window on AppVeyor - I may be wrong (that may be
for Travis). We are pretty close to that. Let's see where we are, get some
of each of the test suites enabled including running all of the core unit
tests, and then add stuff.
—
Reply to this email directly or view it on GitHub
https://github.com/Microsoft/visualfsharp/pull/169#issuecomment-71952131
.

Last run we we're on 38min (running just the basic FSharp.Core.Unittests) - 3 test failures for reasons,due to timeouts and one due to a \r\n v. \n difference which surprised me (is AppVeyor checking out the repo with non-standard endings/)
Yes, upgrading would help. It looks like we would need Pro for it to make any real difference.  However the @Microsoft/visualfsharp team's plan is to move to (or augment with) eventually whatever Roslyn use, which is fair enough. Presumably Microsoft will have a paid plan or their own installations for that.

OK, I merged this to enable AppVeyor on commits for the day, please keep an eye on the history https://ci.appveyor.com/project/KevinRansom/visualfsharp-radou/history

perl is the one embedded in git C:\Program Files (x86)\Git\bin\perl.exe and is not ok ( perl version because msys perl dont have win32 cpan module

@dsyme i think i can fix perl ( download + install in appveyor-build.cmd ) if needed.
But there is time limit, is it usefull?

@enricosada do you know if msys has support for an alternative module for controlling processes? We could use that if needed.

dont know really, i'll try to install module Win32 from cpan before call perl

@enricosada - yes, it would be useful to fix perl, we can then selectively run subsets of the fsharp and fsharpqa tests

no luck with install Win32 module on perl embedded in msys git
( http://stackoverflow.com/questions/28283602/use-win32process-with-perl-of-msys-git )
Anyway now with upgraded appveyour we can install activestate perl, i'll try to do that


Add easy setup, with Windows Installer or NSIS for example

I'd love to download a single nodist-0.3.11.setup.exe file, which I only have to run to be guided through a simple installation. Asking me on the way if I'd like x64 for example.

Good idea! Thanks to @nullivex this is now the default installation method.


Refinement of cells containing start and end waypoints

Once created, the mesh needs to be refined when start and end waypoints are known. The cells containing these waypoints should be split down to the max split level. This will ensure that the paths created are not thrown off by travelling to/from cell centres that are (potentially) a long way away from the waypoints and maybe in the wrong direction.

The MeshiPhi component of this has now been implemented in #40, maybe this issue should be moved to the PolarRoute repo?

We also wish to split cellboxes which are large are bordering small(er) inaccessible cells, down to the resolution of the bordering inaccessible cells

Relevant MeshiPhi changes being made in #50


Strange status code behavior

This is a Bug Report
The EG appears to return a 202 status code when the subscription is set to async and a 200 status code when the subscription is set to sync.  This current breaks the SDK.
However, when you create both an async and sync subscription on the same path+method within the Event Gateway, the async subscription appears to return a 200 status code instead of a 202.  Only briefly noticed this, so having another person verify will be useful.  If confirmed, this  currently breaks the SDK.
Similar or dependent issues:

https://github.com/serverless/event-gateway-sdk/issues/49

That's correct, if there are only async subscription EG will return 202 Accepted. If there is a sync subscription created on the same method and path it takes control of the returned response so it can be anything actually.
I'm closing this one as it's more an issue about SDK.


Bump up rk-entry version to v1.0.5

Codecov Report

Merging #9 (729d721) into master (f04c763) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master       #9   +/-   ##
=======================================
  Coverage   93.47%   93.47%
=======================================
  Files          18       18
  Lines         843      843
=======================================
  Hits          788      788
  Misses         42       42
  Partials       13       13

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 f04c763...729d721. Read the comment docs.


SPEC data files: recognize and read

The spec2nexus.spec module will read SPEC files into Python.
This is a good case study for:

[ ] a lazy loader (since SPEC files may have many scans)
[ ] a cache of recognized SPEC files (since there is no existing standard for recognizing a SPEC file by name)

Not seeing the need yet for either of these optimizations.


jetson.inference.detectNet waiting time of 5 minutes

Hey there,
We use the following line of code self.net = jetson.inference.detectNet("ssd-mobilenet-v2", threshold=0.5) for object detection and it takes about 5 minutes to load.
Is there a way to speed up the process or to preload the model if we start the robot?
We want to use it in a path finding problem and every time we restart the kernal of the notebook we need to wait for about 5 minutes.

It should only take that long the first time you run the model, because it then saves the optimized TensorRT engine out to disk.
Do you see the .engine file get saved in the same directory of the model?  It should be a *.engine file under jetson-inference/data/networks/ssd-mobilenet-v2

No there is no *.engine file in this destination
There are only the two following files: ssd_coco_labels.txt and ssd_mobilenet_v2_coco.uff

Do you have write access to the folder?
If you run detectnet program outside of Jupyter notebook, does it make the .engine file?

Thanks @dusty-nv , there was a problem with the permissions for this folder & files, after changing the permission the *.engine file has been created and the loading time was really fast


Cannot use the rockyou text

yeah i added featur that he will try to open also {wordlist}.txt file, probably you didnt added *.txt on the end


Add missing async task creation when updating callbacks

This is based on the discussion made in #142 so I hope this change enables the use of the add-on with python3.11.
I tested this change using the example in the README and my LG WebOS TV with python 3.11.3 in Mac OS.
It tries to fix #142.

I tried the examples here: https://github.com/home-assistant-libs/aiowebostv/tree/main/examples and all of the ran without problems.

Thanks, I will try to release a package later today


options, deploy: fix setting device_id when verbosity <= 1

Fixes a long standing annoyance where vab would not deploy when verbosity <= 1...

#293 🫠


Build error with iOS 18.1 and Xcode 16.1.0 Beta 2

Hello you great library developers. It seems there is some work to do to support the latest Xcode/iOS.

Yes I am also seeing this issue

I have to say that (1) it's only using Charts for the previews and (2) it would seem like this actually an SDK issue as it appears to be reporting that they've shipped a version of Charts with the SDK that is not built with the correct toolchain?

I can confirm that if you just remove Charts + the preview, the framework builds fine on Xcode 16.1B2. Here's a fork with just that change for now. https://github.com/marcpalmer/Pow


infra/makemake: fix cachix

Or wait for upstream fixes at https://github.com/nix-community/buildbot-nix/pull/241?

We can fix it with the next update.

Oh no, still 502...


Way to close Donkey (and underlying Vertx)

Is your feature request related to a problem? Please describe.
During development using namespace reloading tools like integrant, there should be a way to release the resources created by com.appsflyer.donkey.core/create-donkey. Otherwise new instance of Donkey will be created each time namespaces are reloaded. As far as I can tell, these resources are not garbage collected.
Describe the solution you'd like
A function close-donkey that would release all resources created by create-donkey.
Describe alternatives you've considered
Creating donkey instance only once, and storing it in somewhere where namespace reloading would not affect it. Haven't figured any way to do this.
Additional context
None

My current work-around is:
(defn- close-donkey [^Donkey donkey]
  (-> donkey
      (.-config)
      (:vertx)
      (.close)))

@jarppe it will be added in the next release.
Please note that config is not documented and is not part of the public API. You should not use it in your code as it may be removed in future versions without notice.


Deploy Docker container

Deploy Docker container

We're planning to containerize the app with Docker (#20) and deploy on Google Cloud Kubernetes Engine.
@AbdouSeck has agreed to set up a Google Cloud account for our organization.
@KonradSchieban has helped us with Docker configuration.
@br3ndonland purchased the domain name mbtaccess.org.

This issue was addressed by PR #48


[Request] VSCode keybindings for micro

Can you please add vscode keybindings for micro?

This might be a starting point: #1142 (word navigation and selection)

Ok I implemented this in form of a config file here

Brilliant work, Thank you!


Major CompSci coming through...

Starting to clean up the ugly, undocumented layout left behind by the so-called 'Software Engineers';
Server.js has been calculated to be correct by a designated Computer Science Major;

Fix blank newlines and merge conflict (with your own module!) and it should be ready to go

Is there a way to see the merge conflicts on Github? I can't find a diff in Github to show where it's having issues..

Well.. That's awkward. But it worked... :/


For help: "zh_Hant_TW" and "zh_Hans_CH"

In my project, you need to use "en", "zh_Hant_TW" and "zh_Hans_CH".
But I haven't found a way to make them exist at the same time. Can anyone help me? Thank you.
I saw this in the official document. Does it mean that traditional Chinese cannot be used?
const List<Locale> defaultSupportedLocales = <Locale>[
  Locale('ar'),
  Locale('de'),
  Locale('en'),
  Locale('es'),
  Locale('fr'),
  Locale('hi'),
  Locale('pt'),
  Locale('ru'),
  Locale('zh'),
];

The following is the information printed out by the console.

`======== Exception caught by widgets ===============================================================
The following message was thrown:
Warning: This application's locale, zh_Hans_CH, is not supported by all of its localization delegates.

• A FluentLocalizations delegate that supports the zh_Hans_CH locale was not found.

The declared supported locales for this app are: en, zh_Hans_CH, zh_Hant_TW

See https://flutter.dev/tutorials/internationalization/ for more information about configuring an app's locale, supportedLocales, and localizationsDelegates parameters.
====================================================================================================

======== Exception caught by widgets ===============================================================
The following message was thrown:
Warning: This application's locale, zh_Hans_CH, is not supported by all of its localization delegates.

• A FluentLocalizations delegate that supports the zh_Hans_CH locale was not found.

The declared supported locales for this app are: en, zh_Hans_CH, zh_Hant_TW

See https://flutter.dev/tutorials/internationalization/ for more information about configuring an app's locale, supportedLocales, and localizationsDelegates parameters.
====================================================================================================

Edit WinXaito: formatting

Which version of fluent_ui do you use ?
Apparently, this is fix on the master branch (for the 4.0.0): https://github.com/bdlukaa/fluent_ui/pull/459
Can you check with the branch master ?

@WinXaito
Hi,Sir
I provided the following information:
fluent_ui: ^3.12.0 （The latest stable version）
intl: ^0.17.0
The "Flutter intl"（Latest version） From Android Studio Markerplace.
Flutter version 3.3.1 on channel stable.
I have tried to pre release version 4.0.0,it is ok.
Thank you very much for your reply.


Pipeline printing

Small problem but the pipeline prints only the ink_phase as shown.
https://github.com/sparkfish/augraphy/blob/2f58a48dc324fb653965374a9a9ac5a0dfdb475a/augraphy/base/augmentationpipeline.py#L302-L304

Thanks! I will fix this in the next update.

This is fixed in this pull request: https://github.com/sparkfish/augraphy/pull/192/.


net/haproxy: Avoid duplicate port option (linked to commit new option…

… "checkport" for health checks SHA-1: f14eebe9e2e97edf790d5616d3650aca1bfcb812)

@fraenki seems to be on vacation at the moment, this could take a little longer
Thanks for your submission 👍

Looks good to me! And good catch, btw.
@fichtner Please merge :)

Merged, thanks!

@fichtner There are no other changes pending on my side, so feel free to bump version.

done :)


Random resources without notes yet.

GDAL/OGR Quickstart
States by continent (data)
Mapbox GeoJSON polygon ex.
Leaflet examples
Graph coloring
GeoJSON
Countries dataset
Topo/Geo converter
Leaflet talk
Polymaps cloropleth
The most helpful tutorial (Bostock)
ogr2ogr
boundary geo-referencing guide
ADM0
Github natural earth
gdal cheat sheet
US states cloropleth

🌏


User profile: missing info

On the user profile page (when clicking on own user icon and then "my profile"), some info is missing.
The same data as the one in upsert user view should appear.
(reported by @Gerd-ECRIN )

Infos complete


Fix typo in CMakeLists file

@cla-bot check

Thank you for your contribution. Great to see you trying out realm for flutter 👍

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 93.366%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
380

Relevant Lines:
407

💛  - Coveralls


Disable duplicated rule on detekt project

This rules do the same so I'm disabling the one from ktlint. I could also remove the duplication on @Suppress but #7497 already does that and I would prefer not to fix the conflicts.
Related with #7491

Warnings

:warning:
This PR is approved with no milestone set. If merged, it won't appear in the detekt release notes.

  Generated by :no_entry_sign: dangerJS against 6a4cd5e2c164e0d0f655b26213e9f7da136b55bb


RN 0.61.2 Picker component displays only dropdown picker. Dialog box is not working.

Hello, Picker component dialog mode doesn't work.

+1, Same issue (Currently in RN 0.60.5)


Out of Memory error when loading model - Only using 1 GPU

Hello,
I'm trying to load models using the streamlit app. I have 4 NVIDIA L4, but it only uses 1 and reaches an out of memory error on flux-schnell. Is there a way to make it load on several GPUs ?
Error :
OutOfMemoryError: CUDA out of memory. Tried to allocate 126.00 MiB. GPU 0 has a total capacity of 21.95 GiB of which 54.12 MiB is free. Process 31512 has 0 bytes memory in use. Of the allocated memory 21.70 GiB is allocated by PyTorch, and 13.95 MiB is reserved by PyTorch but unallocated. If reserved but unallocated memory is large try setting PYTORCH_CUDA_ALLOC_CONF=expandable_segments:True to avoid fragmentation. See documentation for Memory Management (https://pytorch.org/docs/stable/notes/cuda.html#environment-variables)

Are you loading in bfloat16? The model should fit in 24GB.

I'm using the default install mentionned in readme. It works with ComfyUI, closing this issue.

This was resolved by switching from the reference code in the repo to comfyui ?
Curious because I get the exact same error on a 24gb card trying to run demo_gr.py with default everything, nothing else in ram, have 2x24gb cards.
I am rather interested in getting the reference code demo to run.

Turns out what was missing was --ofload which says it offloads the model when idle but seems to also be necessary to boot the demo


arch: arm: s32k1xx: Author Gregory Nutt: update licenses to Apache

Summary
Gregory Nutt has submitted the SGA and we can migrate the licenses to Apache.
Impact
NONE
Testing
NONE

@xiaoxiang781216 please ignore  warning: #include outside of 'Included Files' section


[Snyk] Upgrade @types/node from 20.4.4 to 22.9.0

Snyk has created this PR to upgrade @types/node from 20.4.4 to 22.9.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

⚠ Warning: This PR contains major version upgrade(s), and may be a breaking change.

The recommended version is 179 versions ahead of your current version.

The recommended version was released on a month ago.

[!IMPORTANT]

Warning: This PR contains a major version upgrade, and may be a breaking change.
Check the changes in this PR to ensure they won't cause issues with your project.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:

🧐 View latest project report
📜 Customise PR templates
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

:tada: Snyk checks have passed. No issues have been found so far.
:white_check_mark: security/snyk check is complete. No issues have been found. (View Details)

❌ Deploy Preview for sprightly-pony-a06b5f failed.

Name
Link

🔨 Latest commit
dc790e8d882b8184603d3174d63361cbdab4b604

🔍 Latest deploy log
https://app.netlify.com/sites/sprightly-pony-a06b5f/deploys/674ea7cc1ebaa10008a99d13

👷 Deploy Preview for sprightly-pony-a06b5f processing.

Name
Link

🔨 Latest commit
dc790e8d882b8184603d3174d63361cbdab4b604

🔍 Latest deploy log
https://app.netlify.com/sites/sprightly-pony-a06b5f/deploys/674ea7cc1ebaa10008a99d13

👷 Deploy Preview for stalwart-kulfi-7bc69f processing.

Name
Link

🔨 Latest commit
dc790e8d882b8184603d3174d63361cbdab4b604

🔍 Latest deploy log
https://app.netlify.com/sites/stalwart-kulfi-7bc69f/deploys/674ea7cc8ed4120008f22df7


Add rule name in errors

At the moment it is hard to see which rule is producing an error. It was be nice to add the name of the rule that produces an error so that it is easy to find more information about the rule and reconfigure if needed.
On the other hand the error number does not provide much useful information. The current output is as follows:
index.html [3,95]: (E009) use only relative links

It would be better to have this as:
index.html [3,95]: (href-style) use only relative links

This would make it consistent with other front end linters like ESLint and LessHint

+1

See the wiki page option by error code for a temporary solution as proposed in #58. I plan on making the output match what @whistler gave above but have some work to do on grunt-htmllint, etc. to get there.

@mlochbaum Which changes should be done on grunt-htmllint? I reckon we can implement this and then update grunt-htmllint and gulp-htmllint, no?


Right Tool for the Job Issues

Right Tool for the Job  lets you pick from shovel, trap disarm kit and thieves' tools but none of these seem to work right now - no item in inventory, nothing on the ground and the class action doesn't go on cooldown.

works on my version of the mod, 1.6.1. I can't reproduce this :(

Video proof: https://imgur.com/6Okvlui

Found the problem, seems to be a conflict with Cleric Subclasses mod.  Both use improvedUI Assets so it must be one of those files.  It's a pretty minor feature so I'll just ignore it, everything else seems to work fine and I'm playing Shadowheart as Twilight 🤷

Seems that the Cleric mod was updated to rely on ImprovedUI Assets too, so it shouldn't give you issues if you get the updated version.

I'm using the latest version, still doesn't work.  I've tried changing the order by no difference except removing cleric subclasses completely.

are you using the correct modfixer and removed the old ones? Have you removed the mod that blocks the warning message? Have you updated to the latest improcedUI?

looks like you figured it out, thank you.


[HUDI-4753] more accurate record size estimation for log writing and spillable map

Change Logs
Describe context and summary for this change. Highlight if any code was copied.
Impact
No public API or user-facing feature change.
Risk level: none | low | medium | high
Low
Contributor's checklist

[x] Read through contributor's guide
[x] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

@yihua Hey Yihua, could you help review this PR.

rebased w/ latest master.

@hudi-bot run azure

@hudi-bot run azure


Uncaught Error: Call to undefined function cache_get() due to PHP version mismatch

Issue Type

Bug Report / Support Request

Your Environment
Vagrant 2.2.5
VirtualBox 6.0.10r132072
ansible 2.8.2
  config file = /etc/ansible/ansible.cfg
  configured module search path = [u'/home/dane/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/lib/python2.7/dist-packages/ansible
  executable location = /usr/bin/ansible
  python version = 2.7.16 (default, Apr  6 2019, 01:42:57) [GCC 8.3.0]

Your OS

Linux (Ubuntu 19.04)

Full console output
vagrant@local:/var/www/drupalvm$ drush si
PHP Fatal error:  Uncaught Error: Call to undefined function cache_get() in /var/www/drupalvm/includes/module.inc:762
Stack trace:
#0 /var/www/drupalvm/includes/module.inc(962): module_implements('system_theme_in...')
#1 /var/www/drupalvm/modules/system/system.module(2514): module_invoke_all('system_theme_in...')
#2 /var/www/drupalvm/includes/theme.inc(798): _system_rebuild_theme_data()
#3 /var/www/drupalvm/includes/theme.maintenance.inc(57): list_themes()
#4 /var/www/drupalvm/includes/bootstrap.inc(2935): _drupal_maintenance_theme()
#5 /var/www/drupalvm/includes/errors.inc(179): drupal_maintenance_theme()
#6 /var/www/drupalvm/includes/bootstrap.inc(2647): _drupal_log_error(Array, true)
#7 [internal function]: _drupal_exception_handler(Object(Error))
#8 {main}
  thrown in /var/www/drupalvm/includes/module.inc on line 762
Drush command terminated abnormally due to an unrecoverable error.                                                                                [error]
Error: Uncaught Error: Call to undefined function cache_get() in /var/www/drupalvm/includes/module.inc:762
Stack trace:
#0 /var/www/drupalvm/includes/module.inc(962): module_implements('system_theme_in...')
#1 /var/www/drupalvm/modules/system/system.module(2514): module_invoke_all('system_theme_in...')
#2 /var/www/drupalvm/includes/theme.inc(798): _system_rebuild_theme_data()
#3 /var/www/drupalvm/includes/theme.maintenance.inc(57): list_themes()
#4 /var/www/drupalvm/includes/bootstrap.inc(2935): _drupal_maintenance_theme()
#5 /var/www/drupalvm/includes/errors.inc(179): drupal_maintenance_theme()
#6 /var/www/drupalvm/includes/bootstrap.inc(2647): _drupal_log_error(Array, true)
#7 [internal function]: _drupal_exception_handler(Object(Error))
#8 {main}
  thrown in /var/www/drupalvm/includes/module.inc, line 762

Summary
When I try to run any Drush command (or just drush) inside my VM, I get the above error. This is a Drupal 7 codebase.
I think the problem is that even though I've configured the VM to use PHP 7.1, the CLI still uses PHP 7.3. And the MySQL extension is missing for PHP 7.3 (it's only present for 7.1).
As a workaround, I can run this in the VM: sudo apt-get install php7.3-mysql
However, it seems like there's a bug in DrupalVM here, with one of two possible fixes:

The PHP CLI version should be pinned similarly to the web server version, so that it has the same extensions available
The native PHP should have the MySQL extension installed

My config.yml:
vagrant_hostname: local.foo.edu
vagrant_machine_name: foo
vagrant_ip: <IP_ADDRESS>

drupal_build_composer: false
drupal_build_composer_project: false
drupal_install_site: false

drupal_core_path: "/var/www/drupalvm"

php_version: "7.1"

Seems similar to https://github.com/geerlingguy/drupal-vm/issues/1498

I'm not sure why the 7.3 version would still be installed...
The php-versions role sets it here:
https://github.com/geerlingguy/ansible-role-php-versions/blob/master/vars/Debian.yml#L14
And that overrides what's set in the php-mysql role here:
https://github.com/geerlingguy/ansible-role-php-mysql/blob/master/vars/Debian.yml#L2

This is definitely related to #1963, which I'm about to fix through an upstream role update.

I believe the above commit fixes this issue. Please try the latest master release and see if it's still an issue (I will be tagging a new release soon, as well).


SDKS-3254 - Fix to Stop triggering the RefreshToken call twice

JIRA Ticket
Please link jira ticket here
Description
This is the Flow, now it should call only one
--> POST https://openam-sdks.forgeblocks.com/am/oauth2/realms/alpha/token/revoke
--> GET https://openam-sdks.forgeblocks.com/am/oauth2/realms/alpha/authorize?client_id=AndroidTest&scope=openid profile email address phone&response_type=code&redirect_uri=org.forgerock.demo%3A%2Foauth2redirect&code_challenge=HhJPdFv9jRiV9s7GzczE2nsxI2oxkd0oycwLZe4SMWU&code_challenge_method=S256&state=H_9N4Zd2GMxU7bHxIeaf4A

POST https://openam-sdks.forgeblocks.com/am/oauth2/realms/alpha/access_token

@witrisna addressed comments ..please look


My config

Thought I'd share my config as in about 10 mins I've got this looking exactly how I wanted.  bufferline was always a touch too opinionated for myself. My general principle with theming is "if I don't need to see something, it shouldn't be a visual burden". This is just perfect and I love the power.
My config

To enable the theme change, I load in my theme's colors and with a colorscheme autocommand, refresh cokeline on any change. I have also added in a modified button.

Click to see configuration
local M = {}

function M.setup()
	local present, cokeline = pcall(require, "cokeline")
	if not present then
		return
	end

	local colors = require("colors").get()

	cokeline.setup({

		hide_when_one_buffer = true,
		cycle_prev_next_mappings = true,

		default_hl = {
			focused = {
				fg = colors.purple,
				bg = "NONE",
				style = "bold",
			},
			unfocused = {
				fg = colors.gray,
				bg = "NONE",
			},
		},

		components = {
			{
				text = function(buffer)
					return buffer.index ~= 1 and "  "
				end,
			},
			{
				text = function(buffer)
					return buffer.index .. ": "
				end,
				hl = {
					style = function(buffer)
						return buffer.is_focused and "bold" or nil
					end,
				},
			},
			{
				text = function(buffer)
					return buffer.unique_prefix
				end,
				hl = {
					fg = function(buffer)
						return buffer.is_focused and colors.purple or colors.gray
					end,
					style = "italic",
				},
			},
			{
				text = function(buffer)
					return buffer.filename .. " "
				end,
				hl = {
					style = function(buffer)
						return buffer.is_focused and "bold" or nil
					end,
				},
			},
			{
				text = function(buffer)
					return buffer.is_modified and " ●"
				end,
				hl = {
					fg = function(buffer)
						return buffer.is_focused and colors.red
					end,
				},
			},
			{
				text = "  ",
			},
		},
	})
end

return M

When I first used cokeline I was surprised that cycle_prev_next_mappings = true was not the default option. It's something I am so used to with all other buffer plugins.
Btw...happy to move this to a "Users Config" thread

@noib3 -  Closing the issue and tagging you for thanks

Glad you like it!

Btw...happy to move this to a "Users Config" thread

We could probably add a section to the README showing off various configurations. I'll probably do that later/tomorrow, or feel free to send another PR and I'll merge it.


Create paderewski.txt

@qubix00n Thank you for your patience while waiting for a review and processing your request.
Pull request merged. Thank you!


add optgroups to group second-level links

It would be cool/more semantic/more accessible to add optgroups to group together the related second-level links, rather than just visually indenting them. It could work like this:
The list (can't get Github list markup to work, sorry!):

Vegetables
Fruit
2a. Apples
2b. Bananas
2c. Oranges

The select:
      <optgroup label="Vegetables">
        <option>Vegetables main page</option>
      </optgroup>
      <optgroup label="Fruit">
        <option>Fruit main page</option>
        <option>Apples</option>
        <option>Bananas</option>
        <option>Oranges</option>
      </optgroup>
    </select>```

Added in V2 branch.
Not ready for release yet, but it's there ;-)


indicator correlation page

LGTM. FYI - @dlaliberte, please merge this into your branch after this hits dev.

I don't wanna merge this into dev in this state, I would just merge it into the other branch and close this PR

SGTM.


Peek on assertion failure duplicates message

In the image below, the message 'Expected value to be true. Actual 'false'' (which comes from the error thrown by the assertion library) is repeated:

A suggested improvement could be the following:

This is just the design of VS Code -- the title of the peek is the first line of the error message.


how do pdf.js support pinch and expand gesture?

Attach (reccomened) or Link to PDF file here:
Configuration:

Web browser and its version:
Operating system and its version:
PDF.js version:
Is an extension:

Steps to reproduce the problem:
1.
2.
What is the expected behavior? (add screenshot)
What went wrong? (add screenshot)
Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):

Closing as duplicate of #2582.


sort with uniq values in array

I have an array with duplicate values [1,2,3,1,10,1]. How can I sort an array with unique values?
And get [1,2,3,10]
The only way to do this is with awk, are there any other options?

Hey @Lichtsinnig, it's better be done by awk, yes. I can't think of a more efficient solution to this at the moment

Привет @Lichtsinnig , это лучше сделать awk, да. Я не могу придумать более эффективного решения этого вопроса в данный момент

I tried to make a configuration with awk but it failed :(

well, I thought you were able to get that done already. In that case I'm not positive, if awk would manage that demand.
Actually, it shouldn't be so hard to expand json processor functionality with sort and unique operators applied to arrays.

Config
input:
  type: http_server
buffer:
  type: memory
pipeline:
  processors:
    - process_field:
        codec: json
        path: list
        processors:
          - awk:
              program: |
                        BEGIN{a[$1]++}END{for(i in a) {print a[i]}}
output:
  type: stdout
  stdout:
    delimiter: ""

I get one element, but not the whole array :(
{"EventID":4720,"TargetUserNameT":"U","list":"1","u":123}

I can't land my hands on it right now, but I think you just need to look up for any function which uses only for and if statements and implement the same logic with awk

keep in mind, though, that you need to manually craft an appropriate json's array out of your resulting string (default result_type in process_field)
With that said I'd rather have json enhanced with the operators below as sorting is quite trivial operation to perform, at my opinion. So feel free to contribute if @Jeffail would tag this as enhancement

Yeah I think in this case the cleanest approach would be to add a sort, unique and maybe sort_unique operators to the json processor.

@Lichtsinnig Would you like to implement those operators, then?

It would be cool if this functionality was implemented!

Bloblang now has methods sort and unique


Client not aware of URL prefix for DefectDojo

Our instance of DefectDojo is installed at https://domain.com/dojo , such that the API is found at  https://domain.com/dojo/api/v1/
It appears that the /dojo root path is not observed by the helper functions in defectdojo.py.
For example:
In defectdojo.py,
create_enagement() calls get_user_uri(), which returns /api/v1/users/$user_id
In my case, this user URI should be /dojo/api/v1/users/$user_id
The error that DefectDojo returns when creating an engagement is below. I'll submit a PR shortly with a proposed fix.
{"error_message": "Couldn't match resource_uri /api/v1/users/19/ with /dojo/api/v1/users/", "traceback": "Traceback (most recent call last):\n\n  File \"/usr/local/lib/python2.7/dist-packages/tastypie/resources.py\", line 219, in wrapper\n    response = callback(request, *args, **kwargs)\n\n  File \"/usr/local/lib/python2.7/dist-packages/tastypie/resources.py\", line 450, in dispatch_list\n    return self.dispatch('list', request, **kwargs)\n\n  File \"/usr/local/lib/python2.7/dist-packages/tastypie/resources.py\", line 482, in dispatch\n    response = method(request, **kwargs)\n\n  File \"/usr/local/lib/python2.7/dist-packages/tastypie/resources.py\", line 1384, in post_list\n    updated_bundle = self.obj_create(bundle, **self.remove_api_resource_names(kwargs))\n\n  File \"/usr/local/lib/python2.7/dist-packages/tastypie/resources.py\", line 2175, in obj_create\n    return self.save(bundle)\n\n  File \"/usr/local/lib/python2.7/dist-packages/tastypie/resources.py\", line 2310, in save\n    self.is_valid(bundle)\n\n  File \"/usr/local/lib/python2.7/dist-packages/tastypie/resources.py\", line 1296, in is_valid\n    errors = self._meta.validation.is_valid(bundle, bundle.request)\n\n  File \"/usr/local/lib/python2.7/dist-packages/tastypie/validation.py\", line 73, in is_valid\n    form = self.form_class(**self.form_args(bundle))\n\n  File \"./dojo/api.py\", line 79, in form_args\n    pk = self._get_pk_from_resource_uri(rel_field, resource_uri)\n\n  File \"./dojo/api.py\", line 64, in _get_pk_from_resource_uri\n    resource_uri, base_resource_uri))\n\nException: Couldn't match resource_uri /api/v1/users/19/ with /dojo/api/v1/users/\n"}

Hi @aaronweaver and  @mattvaldes,
Since the last update I'm receiving the following error (similar that the first comment):
{"error_message": "Couldn't match resource_uri //api/v1/users/1/ with /api/v1/users/", "traceback": "Traceback (most recent call last):\n\n  File "/usr/local/lib/python2.7/dist-packages/tastypie/resources.py", line 221, in wrapper\n    response = callback(request, *args, **kwargs)\n\n  File "/usr/local/lib/python2.7/dist-packages/tastypie/resources.py", line 461, in dispatch_list\n    return self.dispatch('list', request, **kwargs)\n\n  File "/usr/local/lib/python2.7/dist-packages/tastypie/resources.py", line 493, in dispatch\n    response = method(request, **kwargs)\n\n  File "/usr/local/lib/python2.7/dist-packages/tastypie/resources.py", line 1401, in post_list\n    updated_bundle = self.obj_create(bundle, **self.remove_api_resource_names(kwargs))\n\n  File "/usr/local/lib/python2.7/dist-packages/tastypie/resources.py", line 2205, in obj_create\n    return self.save(bundle)\n\n  File "/usr/local/lib/python2.7/dist-packages/tastypie/resources.py", line 2340, in save\n    self.is_valid(bundle)\n\n  File "/usr/local/lib/python2.7/dist-packages/tastypie/resources.py", line 1313, in is_valid\n    errors = self._meta.validation.is_valid(bundle, bundle.request)\n\n  File "/usr/local/lib/python2.7/dist-packages/tastypie/validation.py", line 73, in is_valid\n    form = self.form_class(**self.form_args(bundle))\n\n  File "/home/xalvarez/django-defectDojo/dojo/api.py", line 85, in form_args\n    pk = self._get_pk_from_resource_uri(rel_field, resource_uri)\n\n  File "/home/xalvarez/django-defectDojo/dojo/api.py", line 70, in _get_pk_from_resource_uri\n    resource_uri, base_resource_uri))\n\nException: Couldn't match resource_uri //api/v1/users/1/ with /api/v1/users/\n"}
The problem is that this error appear in the last release 1.1.4, but if I use the version 1.1.3:
pip install -I defectdojo_api==1.1.3
I don't receive any error.
I think that this bug is not really solved.
Best regards.

@p3r1c0 @aaronweaver it appears I did not test this enough.
After a quick review, I think https://github.com/aaronweaver/defectdojo_api/blob/master/defectdojo_api/defectdojo.py#L37
should be
if len(host.split('/') > 4
The len() function is needed to check for the presence of the URL prefix (otherwise it's always True, which worked in my case).
The 4th position is needed because the code example specifies the host with a trailing slash.
host = 'http://localhost:8000/'
which is different from the code I was using.
I'll have to do some testing to verify more completely, but happy to hear other opinions.

I think the error is on line 38.
https://github.com/aaronweaver/defectdojo_api/blob/master/defectdojo_api/defectdojo.py#L38
self.custom_path = '/' + '/'
This doesn't make sense to me.

Same here, version 1.1.3 works, with 1.1.4 I receive the same error ... {"error_message": "Couldn't match resource_uri ...

I can confirm, please reopen.


CI/CD infrustructure/build simplification

What problem does this solve?
What do you want to do?
What solution would you like?
Lay out what an ideal solution would look like, from a user-facing perspective.
[Optional] How could this be implemented?
Briefly sketch out how this could be put in place.
[Optional] What alternatives have you considered?
Mention other possible solutions to solve the problem presented. If you have strong opinions, explain why they wouldn't be good enough.
If workarounds exist, briefly describe them and explain why they're inadequate.
Related work
Explain how any other related features or bugs tie into this feature request.

Out of scope - it's better for dexterous_developer to focus on development experience and the elements it can uniquely address rather than trying to resolve other build and ci related issues


unable to use clip functionality for HTML component

If the HTML component is placed outside the clip, it will remain visible when using the clip function.

Clip is function is used to make only 3 lines visible, all other react konva components are hidden expect for HTML components,
Please help me how to handle this situation.
Thank you in advance

HTML is not part of the canvas rendering, so clip doesn't affect it. I recommend not to use  for and icon. It is better to use  for "temporary" components, such as tooltips or some inputs.


docs: try to fix depth of index

Signed-off-by: Julian Oes<EMAIL_ADDRESS>

It seemed alright in local testing, so hopefully it's ok.


Since ABI's now hold a process WP, they should be handed

out one per process rather than keeping a single global instance.
Differential Revision: https://reviews.llvm.org/D54460
git-svn-id: https://llvm.org/svn/llvm-project/lldb/trunk@346775 91177308-0d34-0410-b5e6-96231b3b80d8

@swift-ci please test


Call to undefined function locale_get_display_name()

Before submitting your issue, please make sure that you've checked all of the checkboxes below.

[x] You're running the latest release version of Deployer.
[x] Ensure that you're running at least PHP 5.6.4, you can check this by running php -v
[x] You've ran composer install --no-dev from the root of your installation.
[x] You've ran npm install --production from the root of your installation.
[x] You've ran rm -rf bootstrap/cache/* from the root of your installation.
[x] You have restarted the queue listener and node socket server.

Error
Next ErrorException: Call to undefined function locale_get_display_name() (View: /home/sites/deployer/resources/views/user/profile.blade.php)
Steps to reproduce
Install from fresh and visit profile page of your user once logged in.
Environment info
Operating System: Debian Jessie 8.7
PHP Version: 7.0.16

hmm, that is interesting. I was under the impressive that it is built into PHP > 5.3 😡 I will have to look at another way to do that, for now you can just remove that function from the PHP file, it is "just" so that it displays the language name in the dropdown along with the locale code.
Would you mind sharing the output of php -i ?

Yeah not used that function before but maybe it's something I can fix within PHP setup.
Here's the output https://d.pr/n/8Ntn/2YNFOx64.txt

I believe on Ubuntu for example, you would need the (for example) php5.6-intl package, which installs and enables the intl extension which is disabled by default.

@alsotop Works, thanks! Installed php7.0-intl.

Ah! Of course. I shall tweak it so the function is only called if it exists


In Customization

Hi Team,
We have already working AWS account where we don't have option to create new vpn and public subnet. How can we customize this git as per our environment .
Customization like :
1)- Use existing VPC and respective perquisites.
2)- How to do all config in public subnet.
3)- ECS cluster creation and using ECR for docker image etc.

Hi @sangram23,
There are multiple samples within this repository.  Which one specifically are you looking to reuse/customize?

I am looking for Amundsen where i can use already configured  service in our environement  for below stack:
VPC stack
Amazon RDS for PostgreSQL
Amazon Redshift cluster
Bastion host

That sounds like a very bespoke deployment.  The code provided in this repo is to be used for reference purposes or experimentation.  It doesn't cover every possible deployment pattern or architecture.
The Amundsen sample deployment stack[1] is written in CDK and modularized so that you can deploy each component individually.  So if you wanted to skip the VPC, RDS Postgres, Redshift, and Bastion stacks, you can certainly do that.  It may require manually retrieving the required parameters to launch the remaining stacks (as those parameters may have original come from the stacks that you don't want to deploy).
Modifying these stacks for public subnets may require directly modifying the CDK code.  If you've never used CDK before, you can find some getting started material in our docs [1] and some really good introductory workshops [2].
Not everything in these stacks can take advantage of public subnets.  For example, Neptune is only accessilble from inside of a VPC and only ever gets assigned private IP addresses (no public IP addresses are ever assigned, even if you specify using a public subnet).
[1] https://github.com/aws-samples/amazon-neptune-samples/tree/master/amazon-neptune-and-aws-cdk-for-amundsen
[2] https://docs.aws.amazon.com/cdk/v2/guide/getting_started.html
[3] https://workshops.aws/card/cdk


Unclosed connector Exception in consul.aio

Thee is simple script with consul.aio:
import asyncio
import consul.aio

loop = asyncio.get_event_loop()
c = consul.aio.Consul()

@asyncio.coroutine
def go():
    session = yield from c.session.create(ttl=20)
    assert session

    response = yield from c.session.destroy(session)
    assert session

if __name__ == '__main__':
    loop.run_until_complete(go())

If I run it I'll get an exception:
Unclosed connector
connector: <aiohttp.connector.TCPConnector object at 0x7f9dc0121cc0>
Exception ignored in:

Python 3.4.3
aiohttp 0.16.3
python-consul 0.3.19

Want to have a shot a providing a pull request? I don't personally use asyncio.
@jettify might be able to help too...

New asyncio/aiohttp more strict to unclosed connections. We need to call aio.HTTPClient._connector.close() to close http connections.

I'll do PR

I've done some monkey patching few days ago:
class HTTPClient(consul.aio.HTTPClient):
    def __del__(self):
        self._connector.close()

but there is still exception with ResourceWarning: unclosed .

Cool, I've forgotten about __del__ method. I'll add this to my PR too.

Ah, I might have merged too soon. Is there an additional change to come?

yes, I'll add __del__ method with ResourceWarning, I need like more 5-10 min

:+1:

Pushed to Pypi as v0.3.20

Thank you for the fix but ResourceWarning is still here:
$ python -Wdefault /tmp/1
Exception ignored in: sys:1: ResourceWarning: unclosed <socket.socket fd=7, family=AddressFamily.AF_INET, type=2049, proto=6, laddr=('<IP_ADDRESS>', 44245), raddr=('<IP_ADDRESS>', 8500)>

loop.run_until_complete(c.http._session.close())

before
loop.close()

seems to help.


Add PHPUnit 7.0

By the way, why not order than alphabetically?

Nice! :+1:
I'm not sure what exactly do you mean, but true/false is so the finished projets are seen first. So people know they can use it and it's not only "one day promise".


Generate diagrams for all sim.v models

This PR depends on #57 because it uses conda and should be merged after #57.
This PR adds the Symbolator to the Symbiflow Documentation.
The plugin allows generating model diagrams of Verilog files.
It can be used to obtain diagrams of sim.v files from the Symbiflow Architecture Definitions.
In order to insert the symbols in Sphinx you need to add the follwing RST directive to you file:

.. symbolator:: path/to/verilog/file.v

FYI - See an example at https://github.com/SymbiFlow/python-symbiflow-v2x/pull/12

@mithro: Is this OK to merge or are do you have any other comments?

FYI - This branch needs to be rebased and retitled. This is generating Symbols not diagrams.
Look at the sphinxcontrib-verilog-diagrams repo for what is meant by diagrams.


Error with pymedphys 0.40

I'm trying to install OMG Dosimetry on new computers at work, and I get an error message in the analysis module when trying to compute gamma. The error is with pymedphys trying to find the file EXTRA-DEPENDENCIES,.txt but not finding it.
The error is resolved after I install version 0.39.3.

Seems a bug inside PyMedPhys (as a matter of fact it was relesed yesterday).

PyMedPhys is looking for
...REPO_ROOT / lib / pymedphys / dependency-extra.txt.
Conda (or venv) uses site-packages folder instead of lib, for package storage, i.e., the file is in:
...REPO_ROOT / site-packages / pymedphys / dependency-extra.txt.
By my understanding, PyMedPhys does not support installation by other means than pyenv for virtual environment management. But I do not have experience with pyenv.
One provisional solution for the issue is to force the installation of PyMedPhys v0.39
https://github.com/jfcabana/omg_dosimetry/blob/84ea76b72da0cfb7a7b273ea1a5c08bc7d8f5658/pyproject.toml#L25


Possible to integrate justpy (ag-grid) into Dash framework?

Hi, I am currently using the Python Dash framework to build visualizations and would love to integrate justpy (ag-grid) into it. Does anyone know that it is possible? oo

You probably can't run justpy and dash together, but I may be wrong.

Hello @elimintz! Thank you so much for creating justpy! I have no web dev skills or background but justpy made it easy to access to ag-grid and highcharts.
I was also trying to achieve the same thing as @benedictthen1 because my school project requires to use both dash and ag-grid. I realised that justpy uses Starlette server so I tried to mount flask and dash servers on Starlette by adding a few lines of codes in justpy.py file.
Flask works well but for dash, it is somehow showing strange characters instead of showing ploty dash visualizations.
justpy.py

In Browser,

Does anyone happen to know why? I'm really struggling on this for my school project.

I'm sorry I can't help you with this. Maybe you can convince the prof to use JustPy instead of Dash?😁

I'm sorry I can't help you with this. Maybe you can convince the prof to use JustPy instead of Dash?😁

Hi @elimintz we where just evaluating both frameworks, I felt a little bit more inclined to use justpy (bc tailwind and also I'm not that familiar with dash, but seems less flexible). Anyway, what are the reasons to choose justpy over dash ?
Many thanks !!

I'm sorry I can't help you with this. Maybe you can convince the prof to use JustPy instead of Dash?😁

Hi @elimintz we where just evaluating both frameworks, I felt a little bit more inclined to use justpy (bc tailwind and also I'm not that familiar with dash, but seems less flexible). Anyway, what are the reasons to choose justpy over dash ?
Many thanks !!

Hi @pabloazurduy
Thank you for considering JustPy.
Just to be clear, there are reasons to prefer dash. One main reason is that it has been around much longer and is more mature.
That being said, JustPy is more flexible and the syntax is cleaner in my opinion. Also, if you prefer Highcharts as your charting library, then JustPy is a better way to go.
What I would recommend you do is create a small application with both that mimics some of the functionality you are looking for and then decide. Alternatively, run and modify a few examples from both frameworks and see what you like better.
Would be happy to learn what you decide in the end and what were your reasons.

Hi @pabloazurduy
Thank you for considering JustPy.
Just to be clear, there are reasons to prefer dash. One main reason is that it has been around much longer and is more mature.
That being said, JustPy is more flexible and the syntax is cleaner in my opinion. Also, if you prefer Highcharts as your charting library, then JustPy is a better way to go.
What I would recommend you do is create a small application with both that mimics some of the functionality you are looking for and then decide. Alternatively, run and modify a few examples from both frameworks and see what you like better.
Would be happy to learn what you decide in the end and what were your reasons.


Updated composer from v1.6.4 to v1.6.5

Release notes here:
https://github.com/composer/composer/releases/tag/1.6.5

Could not review pull request. It may be too large, or contain no reviewable changes.


Cannot build containerd-stress

Description:
I get a bunch of similar error messages when building containerd-stress (or other directories in /cmd)
cannot use spec (type *"github.com/containerd/containerd/vendor/github.com/opencontainers/runtime-spec/specs-go".Spec) as type *"github.com/opencontainers/runtime-spec/specs-go".Spec in field value

Similar errors pop up when I'm building other parts of containerd. It is always unable to use a vendored package.
Build environment:

Version

contanierd source
1.0.1

runc
1.0.0-rc4+dev

go
1.9.3

Steps to reproduce the issue:
cd /cmd/containerd-stress/
export GOPATH=$(echo $GOPATH):$(pwd)

go build

Errors:
Output of the above steps is:
./main.go:142:8: cannot use spec (type *"github.com/containerd/containerd/vendor/github.com/opencontainers/runtime-spec/specs-go".Spec) as type *"github.com/opencontainers/runtime-spec/specs-go".Spec in field value
./main.go:215:24: cannot use w.spec (type *"github.com/opencontainers/runtime-spec/specs-go".Spec) as type *"github.com/containerd/containerd/vendor/github.com/opencontainers/runtime-spec/specs-go".Spec in argument to containerd.WithSpec
./main.go:260:47: cannot use &pSpec (type *"github.com/opencontainers/runtime-spec/specs-go".Process) as type *"github.com/containerd/containerd/vendor/github.com/opencontainers/runtime-spec/specs-go".Process in argument to t.Exec

Using Makefile
Alternatively, when i run make in the repository's root, I get the following output:
cmd/ctr/main.go:78:10: cannot use plugins.Command (type "github.com/containerd/containerd/vendor/github.com/urfave/cli".Command) as type "github.com/urfave/cli".Command in array or slice literal
cmd/ctr/main.go:79:13: cannot use "github.com/containerd/containerd/cmd/ctr/commands/version".Command (type "github.com/containerd/containerd/vendor/github.com/urfave/cli".Command) as type "github.com/urfave/cli".Command in array or slice literal
cmd/ctr/main.go:80:13: cannot use "github.com/containerd/containerd/cmd/ctr/commands/containers".Command (type "github.com/containerd/containerd/vendor/github.com/urfave/cli".Command) as type "github.com/urfave/cli".Command in array or slice literal
cmd/ctr/main.go:81:10: cannot use "github.com/containerd/containerd/cmd/ctr/commands/content".Command (type "github.com/containerd/containerd/vendor/github.com/urfave/cli".Command) as type "github.com/urfave/cli".Command in array or slice literal
cmd/ctr/main.go:82:9: cannot use "github.com/containerd/containerd/cmd/ctr/commands/events".Command (type "github.com/containerd/containerd/vendor/github.com/urfave/cli".Command) as type "github.com/urfave/cli".Command in array or slice literal
cmd/ctr/main.go:83:9: cannot use "github.com/containerd/containerd/cmd/ctr/commands/images".Command (type "github.com/containerd/containerd/vendor/github.com/urfave/cli".Command) as type "github.com/urfave/cli".Command in array or slice literal
cmd/ctr/main.go:84:16: cannot use "github.com/containerd/containerd/cmd/ctr/commands/namespaces".Command (type "github.com/containerd/containerd/vendor/github.com/urfave/cli".Command) as type "github.com/urfave/cli".Command in array or slice literal
cmd/ctr/main.go:85:8: cannot use pprof.Command (type "github.com/containerd/containerd/vendor/github.com/urfave/cli".Command) as type "github.com/urfave/cli".Command in array or slice literal
cmd/ctr/main.go:86:6: cannot use run.Command (type "github.com/containerd/containerd/vendor/github.com/urfave/cli".Command) as type "github.com/urfave/cli".Command in array or slice literal
cmd/ctr/main.go:87:12: cannot use "github.com/containerd/containerd/cmd/ctr/commands/snapshots".Command (type "github.com/containerd/containerd/vendor/github.com/urfave/cli".Command) as type "github.com/urfave/cli".Command in array or slice literal
cmd/ctr/main.go:87:12: too many errors
Makefile:139: recipe for target 'bin/ctr' failed
make: *** [bin/ctr] Error 2

I have no such issue on current master (d3a7a92d305f67aa69f2601049dfe42de4d19b3e), which revision are you on? containerd vendors all its dependencies, so this shouldn't happen.

I think the problem is that you're setting GOPATH incorrectly.  Don't see it to pwd, leave it at the default value and build with `go build ./cmd/containerd-stress

Yes, this is due to an incorrectly set GOPATH environment variable.
Please follow the instructions in https://github.com/containerd/containerd/blob/master/BUILDING.md to correctly build containerd.

Yes the issue was with GOPATH. I was setting it to the root of repository. Hence the build  from inside /cmd/ was failing.
I solved the issue by running go get github.com/containerd/containerd. This downloaded the containerd repo inside $GOPATH/src/github.com/containerd.
make works fine now.


Add check to cli

Add checks to cli to check if file defined in the flags is present.

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@b415bd2). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop     #543   +/-   ##
==========================================
  Coverage           ?   87.73%
==========================================
  Files              ?      180
  Lines              ?     3759
  Branches           ?        0
==========================================
  Hits               ?     3298
  Misses             ?      461
  Partials           ?        0

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 b415bd2...5f67279. Read the comment docs.

It was proposed to introduce mechanism of handling critical situations both in irohad, iroha-cli. Consider changing this PR after we will decide on that


TON BRAINS 'PolkaDot Brains' Fix BTC price and MD

Grant Application Checklist

[x] The application-template.md has been copied, renamed ( "project_name.md") and updated.
[x] A BTC address for the payment of the milestones is provided inside the application.
[x] The software of the project will be released under the Apache license version 2.0 as specified in the terms and conditions.
[x] The total funding amount of the project is below CHF 30k at the time of submission.

Sign Off

Sign Off


Switch to golang/dep for dependencies

Switch to the official https://github.com/golang/dep tool for dependency management

Hi, I want to get familiar with monocular and also with the new dep dependency manager, so I would like to pick up this issue. I see that you use the deis/go-dev docker image for go tooling, currently version v0.22.0, which includes go version 1.7.3.
dep was added in deis/go-dev:v0.24.0, which includes go 1.8.1.
But because dep is still under heavy development, it probably makes sense to even use the latest deis/go-dev:v1.4.0 image, which includes go 1.9.
Long story short: are there reasons to not use go 1.9?
https://github.com/deis/docker-go-dev/releases

@slintes as discussed on Slack feel free to take this :).
Sorry forgot to answer your question about 1.9, no particular reason, feel free to update to the latest go-dev image!


MINOR: add df.to_unoptimized_plan() to docs, remove erroneous comment

Which issue does this PR close?

Closes #.
Rationale for this change

What changes are included in this PR?

adds dataframe.to_unoptimized_plan() to docs
Are there any user-facing changes?

Codecov Report

Merging #3348 (26e33b8) into master (a49e01e) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3348   +/-   ##
=======================================
  Coverage   85.48%   85.48%
=======================================
  Files         294      294
  Lines       54072    54072
=======================================
+ Hits        46223    46225    +2
+ Misses       7849     7847    -2

Impacted Files
Coverage Δ

datafusion/core/src/dataframe.rs
89.02% <ø> (ø)

datafusion/expr/src/logical_plan/plan.rs
78.22% <0.00%> (+0.16%)
:arrow_up:

datafusion/core/src/physical_plan/metrics/value.rs
87.43% <0.00%> (+0.50%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Get deployments working for 3.7 through 4.4

Fixes https://github.com/fusor/mig-operator/issues/231

"The selector field defines how the Deployment finds which Pods to manage."
This became required in v1 whereas in v1beta1 it was optional. The UI and Controller already had matchLabels configured so no need to add it there.


Fix DL Communications locations

Data entry seems to have included multiple quotation marks both single and double. Changed manually to exclude quotation marks. An alternative to this PR would be to sanitize the input more thoroughly.

That would do it! Did you fix the Google sheet as well?

Just did now! Thanks for the reminder.


dm pull acts confusingly

I had intended to write dm clone, but wrote dm pull, which then said this:
luke@beast:~/moby-counter$ dm pull origin luke4/moby_counter_demo
Pulling luke4/moby_counter_demo from origin:luke5/moby_counter_demo
Response '{"jsonrpc":"2.0","error":{"code":-32000,"message":"Both local and remote filesystems don't exist. \u0026{Peer:cloud.datamesh.io User:luke5 ApiKey:XXX Direction:pull LocalNamespace:luke4 LocalFilesystemName:moby_counter_demo LocalCloneName: RemoteNamespace:luke5 RemoteFilesystemName:moby_counter_demo RemoteCloneName: TargetSnapshot:}","data":null},"id":5577006791947779410}
' yields error Both local and remote filesystems don't exist. &{Peer:cloud.datamesh.io User:luke5 ApiKey:XXX Direction:pull LocalNamespace:luke4 LocalFilesystemName:moby_counter_demo LocalCloneName: RemoteNamespace:luke5 RemoteFilesystemName:moby_counter_demo RemoteCloneName: TargetSnapshot:}

Also the error shouldn't show the password in plain text!

issues:

the model is slightly complicated (you can have namespaces locally)
behaviour for clone, pull, push tries to make it make sense - it's complicated
have we implemented and documented the complicated rules correctly?

possible solutions:

stop having namespaces locally?
distinguish between admin-on-hub and admin-as-local-user? disallow local namespaces in the latter case?

Closing as old and low prio, reopen when needed


Minor CSS bug with codeblock highlighting

Hi, there is a style bug on the online documentation.
The highlighting don't fill all the line when the codeblock have a scrollbar

Unfortunaly I can't try to fix the CSS I don't have computer for a few more weeks

This is a mobile related issue with mkdocs, doesn't belongs to FastAPI
We use hl_lines when adding a highlight to a line
Example:
> ``` Python hl_lines="4  9 10 11 12  25 26 27"
> some_code_here
> ```

It adds highlighting to the whole line so the problem probably belongs to markdown_extensions in mkdocs.yml

Oops my Bad,
I'll open an issue on the mkdocs repo

Thanks for the help here @ycd ! :clap: :bow:
Thanks for reporting back and closing the issue @flapili  :+1:


Revisiting workspace list

The workspace list for frank lab is getting a bit unwieldy, and I propose to update the system.
@lfrank @khl02007
Could you make a python script that queries your dj database and returns a flat list of workspaces with meta information?
Then at the end of the script it would be something like this:
sortingview.set_workspaces(workspaces)

where workspaces is a list of workspace dicts each of the form:
{
  'workspaceUri': 'workspace://...',
  'label': 'workspace-name',
  'metaData': arbitrary meta information that you want to include (but must be json-able - and not huge)
}

Then on the web app, rather than getting a big list, you'd have a search bar, and you'd be able to see the meta data for the workspaces, as well as convenient access to recently opened workspaces.
the idea is that script would be run periodically, or whenever the database has been updated
Thoughts?

I think that would be relatively straight forward; we just need to think about which meta data to include. The easiest way to do this would be to rerun it each time a new sorting is added. I'm not quite clear on how to include data that are json-able, though. Do you have an example of that?

@lfrank, for now you don't need to include any meta data (maybe it's all encoded in the workspace name/label)
By json-able I just mean 'able to be serialized as json'. So, no numpy arrays or other Python objects that aren't simple strings, numbers, dicts, lists, etc.
There is a PR open on this (#72), and that will include a migrate script that will allow you to migrate from the existing workspace list in the interim.


fix spacing on help messages when verbs have aliases

Fixes #6
Fixes #517
Fixes #636
Uses suggestion from @twaalewijn in https://github.com/commandlineparser/commandline/pull/636#issuecomment-645501830 to fix layout of help messages. Uses existing extension method for the concat instead of the exact one provided in the example.

Can confirm that the fix in this PR also causes the tests that I added in my branch to succeed again. 👍


Fix mismatch in parameter names

Function parameters should be named the same in both declaration and
definition.

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

Friendly ping? :)

Thanks for these fixes :)


audittool "WarnAsError" tripped up by white space in shell.properties

When adding tests to shell.properties, whitespace after the test number can cause errors in audittool.

Equally true for before the number.
#102, installed on bodhi, gives this output
╭─   ~/tmp ·······················································································  12s jimk@bodhi  18:28:49 ─╮
╰─❯ audittool.sh ~/home/prod/asset-manager-85/W8LS73821                                                                          ─╯
starting /home/jimk/home/prod/asset-manager-85/W8LS73821
INFO  Passed	/home/jimk/home/prod/asset-manager-85/W8LS73821		Archive EXIF Test
INFO  Passed	/home/jimk/home/prod/asset-manager-85/W8LS73821		No Files in Root Folder
INFO  Passed	/home/jimk/home/prod/asset-manager-85/W8LS73821		Image EXIF Test
INFO  Passed	/home/jimk/home/prod/asset-manager-85/W8LS73821		Web Image Attributes
INFO  Passed	/home/jimk/home/prod/asset-manager-85/W8LS73821		No folders allowed in Image Group folders
INFO  Passed	/home/jimk/home/prod/asset-manager-85/W8LS73821		File Sequence Test
INFO  Passed	/home/jimk/home/prod/asset-manager-85/W8LS73821		File Size Test
╭─   ~/tmp ························································································  4s jimk@bodhi  18:28:58 ─╮
╰─❯ !gr                                                                                ─╯
╭─   ~/tmp ····························································································· jimk@bodhi  18:29:01 ─╮
╰─❯ grep ErrorsAs /usr/local/bin/audittool/at9.04-2021-02-08/shell.properties                                                    ─╯
# ErrorsAsWarning
ErrorsAsWarning= 999, 42 , 115


Deprication warning of required dependencies

What version of million are you using?
2.5.13
Are you using an SSR adapter? If so, which one?
None
What package manager are you using?
npm
What operating system are you using?
Mac and Linux
What browser are you using?
Chrome, Firefox , Safari
Describe the Bug
When trying to npm install I get the following errors for both<EMAIL_ADDRESS>and<EMAIL_ADDRESS>deprecated<EMAIL_ADDRESS>🚨 This package has been deprecated in favor of separate inclusion of a polyfill and regenerator-runtime (when needed). See the @babel/polyfill docs (https://babeljs.io/docs/en/babel-polyfill) for more information.
deprecated<EMAIL_ADDRESS><EMAIL_ADDRESS>is no longer maintained and not recommended for usage due to the number of issues. Because of the V8 engine whims, feature detection in old core-js versions could cause a slowdown up to 100x even if nothing is polyfilled. Some versions have web compatibility issues. Please, upgrade your dependencies to the actual version of core-js.
What's the expected result?
To create and download node modules without errors
Link to Minimal Reproducible Example
https://issueIsInTheTerminal.terminal/dontKnowHowToUseStackblitz
Participation

[ ] I am willing to submit a pull request for this issue.

Note: Both deprecation warnings come from the babel-plugin-transform-react-createelement-to-jsx package, which hasn't been updated since a few years:

babel-plugin-transform-react-createelement-to-jsx is also GNU GPL v3, and there are mixed opinions if it is allowed for MIT licensed projects to still use the MIT license when using GNU GPL licensed projects.
It might make sense to just drop the dependency and re-implement @aidenybai

Fixed on latest @AfroXLandfee


Fixed bug in MarketHoursDatabaseJsonConverter

In the constructor, holiday dates were formatted using the default date format set in the system preferences, instead of always using the fixed US format (as in the Convert method).

Nice catch!


Add generalize code for deprecated code in rails5

Add generalize code for deprecated code in rails 5

Great, thank you !

Great, thank you !


Does this support .Net Framework 4.8 ? Or is the documentation lacking ?

That is a error in the documentation. .NET Framework 4.8 is fully supported.


Tapas website update - no list

Source information
Tapas 1.4.20
Source language
English
Steps to reproduce

Click "browse" > "sources" > choose "tapas" extension.
Open webview to login
Comic should be listed in mihon but got "no result found" instead.

Expected behavior
List of comic would be loaded.
Actual behavior
"No result found"
Mihon/Tachiyomi version
0.16.5
Android version
Android 13
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 0.15.3.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

i have this problem too 😭

Please 🙏 fix it 🙏 😢

For clarity, no results show up when you go to the Latest or Popular sections on the extension, but if you do text search specifically we do get results. Unfortunately, the filter search does not work at all.

For clarity, no results show up when you go to the Latest or Popular sections on the extension, but if you do text search specifically we do get results. Unfortunately, the filter search does not work at all.

Tried but  not be able to search mature titles.

For clarity, no results show up when you go to the Latest or Popular sections on the extension, but if you do text search specifically we do get results. Unfortunately, the filter search does not work at all.

Tried but not be able to search mature titles.

Exactly, the filter search doesn't work.


fs rpc improvements and fix flaky upload banner

rpc improvements

use FSPathUpdate (keybase/kbfs#1663) instead of FSActivity
trigger RPCs when subscribed path is updated
suppress unnecessary RPC when we know we already subscribed paths
we now auto-update folder list even when it's updated on other devices

fix flaky upload banner

don't consider files being written into the journal when determining
if we are uploading, since the journal update may be delayed


Not able to train a model on a 3D stack

I spent several hours annotating about 10 of 50 frames of a 50 frame stack and attempted to update the CF model which did run on the entire stack.  It threw an error.  Is this a bug or is this a missing feature?
If this cannot be fixed, could a potential work around be to annotate each frame as a separate image, then update the CF model as a custom model and run that model on the 3D stacks?
Thank you.

From my understanding, you are trying to train a cellpose model on a 3D image? Cellpose will only take in 2D images so you need to split the image into individual layers. I have a similar script here.

Cellpose is capable of 3d segmentation as described in the documentation on this page and this page.
@TroySeman what exactly was the error you encountered?

You ignored my request for help then regurgitated two links which I did read over a month ago and closed my ticket.  I've been in IT for over 25 years because I do not "handle" customers this way.
That is also the reason your documentation is sketchy and unusable.  Its not been given to neophytes to review, you haven't learned how to make it mean something.
I already know I cannot train a 3D model in cellpose.  that was not the request.  I have a trained 2D model that seems to be running.  The software appears to be confused about the axes.
I'm attaching two more images that may describe the problem better.  The software seems to be confuses about axes.  the segmented layers appear orthogonal to the image.
In absence of real human help, I've asked chat GPT to provide some code.  I haven't had time to try it yet, My intention is to explicitly direct Napari to load the images to defined coordinates instead of having the software guess at it.  I do not expect meaningful assistance from this forum based upon your delayed reply and the lack of legible documentation. Indeed after my criticisms here I expect my posts to be deleted entirely.


zerossl-bot.sh wrapper is using the wrong variable for ARGS

Problem statement
zerossl-bot.sh wrapper is using the wrong variable for ARGS
Steps to reproduce

Install zerosll wrapper root@vmubuntu2004srv:~# bash <(curl -s https://zerossl.com/get-zerosslbot.sh)
Get API key from https://app.zerossl.com/developer
Setup DNS on own domain pointing to your IP
Setup Apache site using mydomain.comroot@vmubuntu2004srv:~# cat /etc/apache2/sites-available/mydomain.com.conf
<VirtualHost *:80>
        ServerName mydomain.com
        DocumentRoot /var/www/html
</VirtualHost>
# vim: syntax=apache ts=4 sw=4 sts=4 sr noet

Open port 80 and 443
Make sure you can visit http://mydomain.com
Run zerossl cerbot wrapperroot@vmubuntu2004srv:~# zerossl-bot --apache -d mydomain.com -m<EMAIL_ADDRESS>--zerossl-api-key<PHONE_NUMBER>23123123123123123123

Visible results
The certificate is issued by Let's Encrypt instead of ZeroSSL
Expected results
The certificate should be issues by ZeroSSL
Proposed solution
The last line of zerossl-bot.sh is wrong - is using ARGS instead of CERTBOT_ARGS
Working script
#!/bin/bash

CERTBOT_ARGS=()

function parse_eab_credentials()
{
    PYTHONIOENCODING=utf8
    ZEROSSL_EAB_KID=$(echo $1 | python -c "import sys, json; print(json.load(sys.stdin)['eab_kid'])")
    ZEROSSL_EAB_HMAC_KEY=$(echo $1 | python -c "import sys, json; print(json.load(sys.stdin)['eab_hmac_key'])")
    CERTBOT_ARGS+=(--eab-kid "$ZEROSSL_EAB_KID" --eab-hmac-key "$ZEROSSL_EAB_HMAC_KEY" --server "https://acme.zerossl.com/v2/DV90")
}

while [[ "$#" -gt 0 ]]; do
    case $1 in
        --zerossl-api-key=*)
            ZEROSSL_API_KEY="${1:18}"
        ;;
        --zerossl-api-key|-z)
           ZEROSSL_API_KEY="${2}"
           shift
        ;;
        --zerossl-email=*)
            ZEROSSL_EMAIL="${1:16}"
        ;;
        --email|--zerossl-email|-m)
           ZEROSSL_EMAIL="${2}"
           CERTBOT_ARGS+=(-m "${2}")
           shift
        ;;
        *) CERTBOT_ARGS+=($1) ;;
    esac
    shift
done

if [[ -n $ZEROSSL_API_KEY ]]; then
    parse_eab_credentials $(curl -s -X POST "https://api.zerossl.com/acme/eab-credentials?access_key=$ZEROSSL_API_KEY")
elif [[ -n $ZEROSSL_EMAIL ]]; then
    parse_eab_credentials $(curl -s https://api.zerossl.com/acme/eab-credentials-email --data "email=$ZEROSSL_EMAIL")
fi

echo ${CERTBOT_ARGS[@]}
certbot ${CERTBOT_ARGS[@]}

@bitroniq Thank you for your report! Can you confirm this MR would fix the problem?
https://github.com/zerossl/zerossl-bot/pull/34/files

@bitroniq Thank you for your report! Can you confirm this MR would fix the problem? https://github.com/zerossl/zerossl-bot/pull/34/files

PR looks good to me - this should solve the problems
Here is my review with the comment:
https://github.com/zerossl/zerossl-bot/pull/34#pullrequestreview-1374835219
The thing is though that the official downloader

https://zerossl.com/get-zerosslbot.sh
Is pulling the file from
https://zerossl.com/zerossl-bot.sh

And the https://zerossl.com/zerossl-bot.sh is broken (wrong last line - ARGS instead of CERTBOT_ARGS.

@bitroniq Changed get-zerosslbot.sh to always use the latest version from Github master. Does this fix the problem? The version on zerossl.com is outdated.

@Blackbam that will fix the problem. Thx!
BTW. Will you update the script here https://zerossl.com/get-zerosslbot.sh as well to always get latest from github?

@bitroniq Fixed ✌

@bitroniq Fixed ✌

Great! I see and tested it, everything works as expected now.
Thank you!


Fix crash by improving VRAM usage patterns when doing .swap()

This pull request fixes #1362 by making cross attention control manage VRAM in a better way. It also enables cross-attention control on larger outputs - it still uses a lot of VRAM but is more active and aggressive about offloading attention slices to system RAM when VRAM fills up.
(When this happens the performance becomes quite bad quite quickly - this is expected, and in non VRAM-constrained use cases performance should not be impacted.)

There is a new problem here which is that very large generations with .swap (>1024x1024) the offloading of attention slices from the GPU to CPU can use very large amounts of system RAM. if you're running inside a docker container and some limit is hit (32GB on my runpod.io instance), the process just gets killed without throwing exceptions or anything.

@lstein This holds up on my end. Please run some quick tests on your system too and see if there's anything out of the expected.


metas sorted

@rultor merge

@rultor merge

@eugenedar OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@eugenedar Oops, I failed. You can see the full log here (spent 15s)
+ set -e
+ set -o pipefail
++ dirname ./run.sh
+ cd .
+ echo 1099072
+ echo '2.0-SNAPSHOT BUILD'
2.0-SNAPSHOT BUILD
+ date
Fri 22 Jul 2022 01:30:48 PM CEST
+ uptime
 13:30:48 up 23 days, 15:31,  0 users,  load average: 0.00, 0.03, 0.08
+ cat
.rultor.yml in yegor256/objectionary-secrets doesn't allow EugeneDar/eo-json to use its assets (there are 8 friends), see http://doc.rultor.com/reference.html#assets
+ exit -1
'cid' file is absent, container wasn't started correctly

@rultor merge

@rultor merge

@eugenedar OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@eugenedar Oops, I failed. You can see the full log here (spent 16s)
+ set -e
+ set -o pipefail
++ dirname ./run.sh
+ cd .
+ echo 1099644
+ echo '2.0-SNAPSHOT BUILD'
2.0-SNAPSHOT BUILD
+ date
Fri 22 Jul 2022 01:35:39 PM CEST
+ uptime
 13:35:39 up 23 days, 15:36,  0 users,  load average: 0.00, 0.00, 0.05
+ cat
.rultor.yml in yegor256/objectionary-secrets doesn't allow EugeneDar/eo-json to use its assets (there are 9 friends), see http://doc.rultor.com/reference.html#assets
+ exit -1
'cid' file is absent, container wasn't started correctly

@rultor merge

@rultor merge

@eugenedar OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@eugenedar Done! FYI, the full log is here (took me 3min)

@rultor merge

@eugenedar The pull request is closed already, so I can't merge it

@rultor merge

@eugenedar I'm sorry, I don't understand you :( Check this page and try again please

@rultor merge

@eugenedar The pull request is closed already, so I can't merge it