Line markers not saving/FPS issue

I've been working on a project of outlining the trigger border of dig/emote clue steps and I've noticed a few times now that after marking the outline I'll come back later to find that they've disappeared. Is this a known issue that's being worked on or is it related to adding too many lines in a short timeframe?
Also, after adding a new line it'll lag my game for about 3 seconds each line, which isn't a big deal but is kinda annoying. I've had this issue from the start so I don't think it's related to having too many lines marked (I'd guess I'm between 4-5k lines marked atm). Is this due to a bad interaction with another plugin or something?

Here's an example of a medium clue box I've had to remake three times now outside of the Mory slayer tower. To name a few more off the top of my head: there's also the north lumbridge cow pen dig step, digsite winch emote step, tower of life dig step, and the southern terrorbird dig step. From what I can recall, all the groups of lines that aren't saving started happening with medium clue steps, which I started doing after making a box around all the beginner and easy clue steps.

Yes, I think markers can disappear if they are created in quick succession. This would be because lines are competing when saving and end up overwriting each other. I should be able to fix this by saving less frequently, but not sure how often to save and what should trigger it.
It would be interesting for me to know if you are creating the markers immediately as groups using the shift key or doing it manually in the panel. The latter will most likely be more prone to issues, as I think there are several (unnecessary) save triggers in the panel.
Having 5000 lines could also be an issue, as it will most likely increase the time it takes to save significantly because the Json object it is stored as becomes quite big. I guess you could test this by exporting all your lines and creating a backup in a Pastebin, deleting all markers in the panel, and checking if the issue still persists.
It is really cool to see that you are creating outlines for clue steps. I hope you plan to contribute these outlines to the core Clue Scroll plugin as that plugin is much more suitable for displaying many lines than this plugin is (because they are static).

Thanks for getting back to me so quickly. I've just been shift+right clicking to add lines and haven't really labeled the marker groups yet; eventually I'll probably want to list each group by their item ID to make it easier to implement in the Clue Scrolls plugin though.
With the few seconds of lag after adding each line it does somewhat often create a separate marker group, and almost always when I change the line edge in the plugin menu. It could be a useful feature to have the option to add lines to a pre-existing marker group instead of automatically creating a new group after four or five seconds like it currently seems to be.
When I add a new line it always goes into the Line Markers sidebar list; however, when I exit Runelite, it removes recently added lines/marker groups. I'm not sure if there's a timer that saves locally every so often or maybe the local file is too big and isn't saving new data or something. Temporary fix would be to export my markers before exiting but maybe a manual save button could work too. I'm pretty sure some of the lines I've added were over an hour before I logged off so it's not a super quick thing either.

Can you export all your ~5000 markers and send them to me? You can either save the export to a file and drag & drop it into a comment here on GitHub or upload it to Pastebin and provide the link. That way I can test if I can reproduce the same issues you are experiencing.

It could be a useful feature to have the option to add lines to a pre-existing marker group instead of automatically creating a new group after four or five seconds like it currently seems to be.

Not quite what you wanted, but you can quickly merge two groups by renaming one of them to the exact name of the other group.

LineMarkers2309300920.txt
Oh nice, I was worried I was going to have to manually change each line's marker group so renaming the group is going to be helpful. I exited the game, logged in again, and exported the attached text file so it should be in it's default state.

LineMarker2309301311.txt
I spent the last few hours compiling the marker groups that I had into one for each step and that seems to have fixed the not saving my data issue and the FPS issues. I'm not sure how many marker groups I have now so I attached a file listing the condensed version. I don't recall at what number of marker groups I started having FPS issues but I started having the save issues when I passed 400; 463 is when it stopped saving altogether.
I still had the same number of lines when condensing the marker groups started to improve FPS so it seems like the large amount of marker groups is what was causing issues. Not sure how that could be improved but good to know for future use.

Been a little while, I realized that if I couldn't add new lines in-game after a certain point then I could potentially get around this by just importing the lines. So I went through and compiled the needed data and used a spreadsheet to make a list of all the lines I wanted. I don't have any issues importing the data and everything shows up correctly in-game; however, upon closing the game, it removes the imported lines from the plugin. I've attached the complete list of lines I import and was hoping to find out if this is a resolvable issue or just a limitation of the plugin. Thanks
0CluePack.txt


HW 13 task 4 , the given codes on iteration wouldn't work properly

I would either change it to:
for i in np.arange(train.num_rows):
train_row_features_array = row_to_array(train.row(i), features)
or just:
how do you iterate the train set.........

Upon further testing on the coding, it would both work with the original codes provided and my alternative codes. Therefore I consider it a non issue anymore. My apology.


[Cosmos] Raise exception if invalid retry configuration specified for connection

Description
If an invalid retry policy is specified when creating a Cosmos connection an TypeError is instantiated but not actually raised.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

hi @adamckay, thank you for your contribution! You are completely right in your assessment and have also shown me some other mistakes in our code by doing so... would you mind if I pulled your branch and made some additional changes, or would you like to make them?
from what I can see right now, it would be doing this same change in the _cosmos_connection_policy_async.py file as well as adding a quick bugfix comment to the CHANGELOG.md file stating that wrong retry policies will now get thrown.
Let me know if you prefer I do it or if you'd like to do it as well :)

Hi Simon, thanks for the speedy review! Feel free to make any changes necessary :)
I did also consider how to add a test (though it's perhaps excessive for such a simple change that's unlikely to regress) but I'm afraid I didn't have time to fully grok the codebase to find the best way to add it.

@adamckay thanks again for your contribution, and thank you for using our sdk! :)


Error building OpenSfM

Hi,
After installing all dependencies on Ubuntu with docker using
docker build –t mapillary/opensfm .
I'm trying to build OpenSfm using
sudo apt-get install python3-setuptools
sudo apt-get install python3-wheel
python3 setup.py build
but i got this error:
Configuring for python 3.6...
Traceback (most recent call last):
File "setup.py", line 48, in
configure_c_extension()
File "setup.py", line 39, in configure_c_extension
subprocess.check_call(cmake_command, cwd='cmake_build')
File "/usr/lib/python3.6/subprocess.py", line 306, in check_call
retcode = call(*popenargs, **kwargs)
File "/usr/lib/python3.6/subprocess.py", line 287, in call
with Popen(*popenargs, **kwargs) as p:
File "/usr/lib/python3.6/subprocess.py", line 729, in init
restore_signals, start_new_session)
File "/usr/lib/python3.6/subprocess.py", line 1364, in _execute_child
raise child_exception_type(errno_num, err_msg, err_filename)
FileNotFoundError: [Errno 2] No such file or directory: 'cmake': 'cmake'
Thanks in advance,
Manuel

I have the same exact problem. Did you manage to solve it?

No. Sorry
Obtener Outlook para Android
On Wed, Apr 22, 2020 at 5:54 AM +0200, "yurenpang"<EMAIL_ADDRESS>wrote:
I have the same exact problem. Did you manage to solve it?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Encounter exactly the same problem.
Popping

@BrianG13 @mlm74 make sure you have CMake installed.

hello！I has installed cmake,and I occurred this :
CMake Error at CMakeLists.txt:3 (project):
Running
'nmake' '-?'
failed with:
绯荤粺鎵句笉鍒版寚瀹氱殑鏂囦欢銆?
-- Configuring incomplete, errors occurred!
See also "C:/Users/x00574457/Desktop/md/OpenSfM/cmake_build/CMakeFiles/CMakeOutput.log".
Configuring for python 3.8...
Traceback (most recent call last):
File "setup.py", line 55, in
configure_c_extension()
File "setup.py", line 43, in configure_c_extension
subprocess.check_call(cmake_command, cwd="cmake_build")
File "C:\Users\x00574457\AppData\Local\Programs\Python\Python38\lib\subprocess.py", line 364, in check_call
raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['cmake', '../opensfm/src', '-DPYTHON_EXECUTABLE=C:\Users\x00574457\AppData\Local\Programs\Python\Python38\python.exe', '-DVCPKG_TARGET_TRIPLET=x64-windows', '-DCMAKE_TOOLCHAIN_FILE=../vcpkg/scripts/buildsystems/vcpkg.cmake']' returned non-zero exit status 1.
Did you occurred this?


ZStack don't working in velocity_x 2.6.0 =>  'List<Widget?>' is not a subtype of type 'List'

ZStack don't working in velocity_x 2.6.0 => type 'List<Widget?>' is not a subtype of type 'List' in type cast

Fixed in v2.6.1


Option for getting the LaTeX code

MathJax has a contextual menu for that. It would be great to have such a feature in KaTeX.

The LaTeX code for an equation is embedded in the MathML code, so you can extract it yourself if you want. This behaviour could maybe be put into an extension if you wanted it.
I don't think that the contextual menu is in the scope of KaTeX though, nor do I think we want it. Overriding the default right-click behaviour is usually just annoying. We also couldn't guarantee that it would work since KaTeX can be server-side rendered, but the menu would require some client-side script to run.

@xymostech The contextual menu is just an example. If overriding the right-click is annoying, how about the left-click? Left-clicking the formula creates a popup window with LaTeX source code. I think that's not a bad idea.

I don't think it is appropriate in most professional websites to expose the LaTeX code to the user. Technical users can use the devtools to see more info (as they would to inspect any other source code for the page) and if your site does want this functionality, it's probably best designed and integrated into your site in a way that KaTeX wouldn't be able to do and should be built custom for your site.

@sophiebits @xymostech  Think about math forums. The equations are not there to be read only. For example, if somebody posts a question with a lot of equations, then for people who want to answer the question, it would be much more convenient for them to be able to copy the equations.
I am an administrator of a (big) math forum, and this is the most important reason why I'm still using MathJax (I once replaced it by KaTeX as an experiment, but that failed quickly). MathJax is very flexible: one can disable the option to view the source code, or can even disable completely the contextual menu.

There is copy-tex extension, which puts LaTeX source code to the clipboard when rendered equation is copied. Also, in the extension, there is katex2tex.js, which extracts LaTeX source from rendered KaTeX element. You can use it to appropriate LaTeX source viewing method.

I wrote copy-tex, and it would probably not be hard to write a similar extension that overrode some click behavior to reveal the source code.  I think it'd be fine to include such a feature as a KaTeX contrib extension, but agree that it doesn't belong in the "base" KaTeX (because that doesn't do any client-side JavaScript).  @netw0rkf10w are you interested in writing such an extension?  If not, I can give a try, though I'm not sure what the best behavior would be.

@ylemkimon @edemaine  Awesome! I will definitely try it on my forum (not now but maybe in a month).
Thanks @edemaine for the great extension! The current "Select and copy" is already a very good option, but I think it would be nice to have an additional one that does not require selection (for example: right-click on the formula and "Copy LaTeX code", which I think is the best). Unfortunately I am currently not available to do it myself (although you have done almost all the work with copy-tex) as I'm busily writing my PhD thesis :(


Add the no-ignore flag to the args to pass to ripgrep

As a user, when I want to use the quick open without ignoring files/folders that are part of the .gitignore, there is currently no way to set this up. In the following screenshot, dags_dev folder is part of the .gitignore:

My particular use case requires me to have a folder in the .gitignore and use that to test (as to avoid accidentally uploading stuff that should not be uploaded into GitHub), so currently there is no way for me to quickly open files inside that folder

Thanks for logging the issue, @sollymay - I'm working on a fix for this in #3795 (A configuration setting - search.useIgnoreFiles to add the --no-ignore flag when using ripgrep for quick open / find in files)


Added hyperbolic sin and cos to keras backend

Sorry, no CLAs. The changes are trivial. Feel free to recreate from your acc. Also we need real and imag

Thanks, I'm on it. Can we close this? @KOLANICH

@kyscg , are you going to create a new PR with the stuff?

Yes, I will do it soon. Is there anything else you want me to keep in mind???

linalg.einsum though I am not sure that it is avilable in other libs (it is unavailable in CNTK) and  K.eye's batch_shape parameter.


Minor grammar fix

Minor grammar fix at line 241
type=routine
risk=low
impact=sev5

Can you put all grammar fix in one PR? that would be convenient to review


Problem with app.use(path, router)

Dear express community,
I am trying to organize my routes in different files, as described in the docs and in several websites I have been searching for this subject.
All of them suggest to use app.use("path", router obj) to assign a route to a router instance.
This is the code I have so far:
server.js:
const devices = require("./controllers/devices.js");
app.use("/api/devices", devices);
controllers/devices.js:
var router = express.Router();
// Middleware that is specific to this router
router.use(function timeLog(req, res, next) {
  // Middleware here
  next();
});
router.get('/', function(req, res) {
    logger.debug("GET DEVICES");
    db.Devices.find({}).then(function (err, result) {
        res.send(result);
    });
});
exports = router;
However, when I launch my application, I am getting this exception:
(node:17868) UnhandledPromiseRejectionWarning: TypeError: Router.use() requires a middleware function but got a Object
I cannot find what I am doing wrong. The error is pretty clear: I am sending a router instance to 'app.use', which expetcs a function, but this is what I am seeing in all examples and in the documentation.
What is the correct way to organize the routes in different files?
Thank you!

@mauriciocirelli : The problem is in the last line of your devices.js file.
Instead of using exports = router; , you should use module.exports = router;
In node.js, when you wish to export an object as a whole from a module (which in this case is router), you should use module.exports.
Look at the value of devices router that you got using require for your code.

Now when we change it to module.exports, look at the value of devices.

Ohhh c'mon.. This is embarassing.
Still getting use with this nodejs syntax.
Thank you @sampathBlam


Dynamic chunk/entity view distance

Currently, the chunk/entity view distances are set via a server flag, making it impossible to change them during runtime.
Would be very useful to change without having to restart the server entirely.

I would also like to be able to change this parameter for each instance separately :)


Deprecate using version from request class.

This CR deprecates the using version from request class.
The methods WithVersion and GetVersion are deprecated in the
class PartitionsRequest, so we should not use that.
Relates-to: OLPEDGE-2351
Signed-off-by: Oleksii Zubko<EMAIL_ADDRESS>

Codecov Report

Merging #388 into master will decrease coverage by 0.1830%.
The diff coverage is 16.6667%.

@@               Coverage Diff                @@
##             master       #388        +/-   ##
================================================
- Coverage   88.0983%   87.9153%   -0.1829%
================================================
  Files            72         72
  Lines          2403       2408         +5
  Branches        264        269         +5
================================================
  Hits           2117       2117
  Misses          161        161
- Partials        125        130         +5

Impacted Files
Coverage Δ

...aservice-read/lib/cache/MetadataCacheRepository.ts
75.0000% <0.0000%> (-8.3333%)
:arrow_down:

...k-dataservice-read/lib/client/PartitionsRequest.ts
100.0000% <ø> (ø)

...olp-sdk-dataservice-read/lib/client/QueryClient.ts
73.2143% <50.0000%> (-1.3311%)
:arrow_down:

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 77929cc...bbe5f24. Read the comment docs.


SPIKE: Detect if images in Image Registry or ECS are root disabled

Describe the issue
We wish to prevent teams from deploying ECS containers that have a root user left active while running.  We wish the root user to always be stripped out at the end of the build process prior to insertion in the image registry.  This is a platform level scope.
Which Sprint Goal is this issue related to?

Note: 'Milestone' is a ZenHub term that we use synonymously with 'Sprint'.

The 'Milestone' description (created in ZenHub) should clearly list Sprint Goals. This section should indicate which 'goal' this issue is related to.
Additional context

On prem OpenShift operates to this standard

Definition of done

Meeting with aws, tech roles and security to discuss best approach (CloudCustodian? Guardrail? Other?)
Estimate a means of detecting across entire app team project-set ecosystem, any tickets created.
If it's harder to do, then break off the auto-remediation piece into a separate well-defined ticket
We want to detect all ECS and EKS instances and we want to do a basic scan of the image registry to detect root enabled pods.
Design includes notification
time-boxed to 3 days

Duplicate


Annotation cache dir is in /tmp if apcu is not used

Related: #303
In the SchemaFactory, it defaults to sys_get_temp_dir() if the apcu extension is not found:
$cache = function_exists('apcu_fetch') ? new ApcuCache() : new PhpFileCache(sys_get_temp_dir() . '/graphqlite.' . crc32(__DIR__));

This cache dir should be configurable, so that for example, in a Symfony install, we can set it to var/cache/dev

Closed by #310


Implement King's Blood

I'd love to use the snapshot functionality here, but it's not that straightforward. Ideas?

My idea for these situations is to take the snapshot at the beginning of AbilityResolver and put it on context as cardStateWhenInitiated. Then you wouldn't need to record the parent and gold tokens in the event listener, just reference them directly in the handler off of context.cardStateWhenInitiated.parent/gold. Alternatively, the snapshot could occur only for costs that sacrifice a card, but that may be less intuitive than something that's always available.
Then we'd need to go through cards that sacrifice themselves as a cost and update accordingly.
If you want to take a crack at this, then great. If not, I'd at least recommend renaming this.parentCard and this.gold to something that look less like the normal properties of the card. I think the convention we've used elsewhere is this.originalX.


fix: add server support and update readme with install and usage

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Attribute completion shoud not insert optional args

This is wrong, the user should never erase things inserted by completion, we must not insert = "".

The completion could also be changed to give the user a choice, e.g.:
deprecated ${1:= "${2:reason}"}, where the user can hit backspace from tabstop $1 to not use the optional argument, or
`deprecated ${1|,"${2:reason}"|}, using explicit choice syntax to choose between having the argument or not

The completion could also be changed to give the user a choice, e.g.:
deprecated ${1:= "${2:reason}"}, where the user can hit backspace from tabstop $1 to not use the optional argument, or
`deprecated ${1|,"${2:reason}"|}, using explicit choice syntax to choose between having the argument or not

This is a rarely invoked completion, so I'd probably prefer just deprecated.

This is a rarely invoked completion, so I'd probably prefer just deprecated.

That is, it takes n times to learn each snippet tab stops, which works for frequent snippets.

That is, it takes n times to learn each snippet tab stops, which works for frequent snippets.

I can work on this.
Do you know if there are other attributes that should not have the optional args ?
Or is it only the deprecated one ?

I can work on this.
Do you know if there are other attributes that should not have the optional args ?
Or is it only the deprecated one ?

must_use and should_panic suffer from the same problem I think.

must_use and should_panic suffer from the same problem I think.

Fixed in #7176

Fixed in #7176

the same still happens for at least inline

the same still happens for at least inline


Bower install pull child modules dependencies

Hi,
I have  requirement to pull all the dependent modules and the interdependent modules to same level.
This is the project structure.
package
/bower.json
dependencies:['module1','module2'];
module1
/bower.json
dependencies:['module3']
So the issue is when I try to do bower install on package level I am able to get module1 and module2 but not module3 which is a dependent module for module1.
Does any one have a approach.
Thanks

Sunny.

It's not enough information. Please post in new issue github repository that reproduces this.


Update tutorial to reflect new service layout

Refs #1984
To start off with, I've updated the tutorial to walk the user through writing a service class that extends BaseJsonService instead of defining a badge in server.js using the procedural style. This doesn't complete the documentation refresh - there are still other topics we need to cover/update - but I think this represents a chunk of work which is useful to review.
Next steps:

Update/refresh the service testing docs
Walk though defining a service family with an abstract parent class and multiple badge implementations extending it
Add docs on how to port legacy services

(probably in that order)

Warnings

:warning:

This PR modified service code for gem but not its test code. That's okay so long as it's refactoring existing code.

Messages

:book:

:sparkles: Thanks for your contribution to Shields, @chris48s!

:book:

Thanks for contributing to our documentation. We :heart: our documentarians!

  Generated by :no_entry_sign: dangerJS

Chris, thanks for taking this on! I responded to your comments though I didn't read it through fully.
I'll be offline for a few days starting tomorrow, so feel free to merge this, and maybe I'll do a set of follow-up edits when I'm back online.

Thanks for the comments on this @paulmelnikow and @PyvesB .
As well as the comments provided here @niccokunzmann has submitted this PR to my fork proposing some changes which..vary in scope and I haven't properly read yet.
I'll do another pass over this taking into account the suggestions made in both PRs, resolve the conflicts and update this.

Good idea. You can change it. We will avoid it.
On 09/03/2018 09:55 PM, Pyves wrote:

@PyvesB commented on this pull request.

In doc/TUTORIAL.md
https://github.com/badges/shields/pull/2042#discussion_r214750527:

-- CONTRIBUTING.md
-You can read
-- previous successful pull-requests to see how other people implemented their badges.

Usually they start with "[add][add-pr]".
-- later [pull-requests tagged with new-badge][new-badge].
-- [implementations and their commits][blame] in the [server.js][server] file.
-- [merged pull-requests][mergedpr].
+You can also read previous
+merged pull-requests with the 'service-badge' label
+to see how other people implemented their badges.

(2) Setup
I suppose you have git installed.
I don't mind whether it's "you" or "we", but I think we should avoid
"I" as it sounds as if one authority is dictating the tutorial.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/badges/shields/pull/2042#discussion_r214750527,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAieIAo50Q0DxyVbGS7W_qMQJWJ9gFsiks5uXYkugaJpZM4WWXxa.

I've pushed some additional edits addressing the various bits of feedback. Thanks, all. @PyvesB maybe you could give this a second look. Cheers


Revert "fix(command): allow reveal_file to be used with dir"

Reverts nvim-neo-tree/neo-tree.nvim#1501 due to unintended errors

Actually I can fix this asap


Revoke Derived key UI

If deleteKey=true and publicKey and derivedPublicKey are both set, then we display this simpler revoke screen.

Looks good.

Thanks for adding this. This was very must needed.


Define plug-in system

We need some robust mechanism to control how plugins/components are installed, enabled and disabled at runtime.

For a first step I would suggest to symlink files instead of copying them.
We could add a folder settings/components.d and settings/scripts.d/ and components/scripts that should be enabled would be linked there.
@MiczFlor @s-martin  Does that sound sane?

Sounds like a good idea and if it works for Linux it should work for us :smiley:

Hi @ZyanKLee @s-martin
what is the goal here? I think we need to separate local installs and their file changes from the git repo. This can be done with configuration files. Not all components will have a beautiful separation of core functionality and individual tweaks.
With symlinks, that will not make a difference. The local repo files will still be changed. Making copies decouples the two. Then it is independent.
Possibly I am missing something :) ?

well, having copies instead of symlinks always an option if changes are made manually. But if a script is going to be used as-is but is not part of the core-functionality, I would suggest to not use scripts, but symlinks to point at the "git objects". But perhaps my idea went short of the goal and instead we should think about putting the configuration itself outside of the RPi-Jukebox-RFID folder. Having it at ~/.config/phoniebox/ would allow us to update the git repo without caring much about anything else in the same parent directory.

is it feasible to also look into how components communicate with the "core"?
currently almost all RFID readers are handled by one python script, correct?
does that scale? can we improve maintainability by separating code that is specific to one device into a component and hook those components into the Reader.py based on a config variable?
same questions for buttons, displays, etc

should we provide a common interface for components to communicate with other parts and the core? every read and write access to the file settings/ files should in my opinion go through the core.

I agree that we should think about that.
I don't know enough about Python, but we should just define an interface and let each RFID reader implement that (maybe providing the Neuftech reader as default implementation).

Adressed for future3 in #1771


infinite loop / missing data when converting to a DataSet

I am using the IDataReader.ToDataset() extension method provided by the  Microsoft.Azure.Kusto.Cloud.Platform nuget package to convert a IDataReader to a DataSet.
It seems that with the MockDataTable that I am setting up using this nuget package, it seems to be going into a infinite loop. I took a look today, and it seems that it might be caused by the fact that this package isn't mocking IDataReader.IsClosed and IDataReader.NextResult(),
I got around the infinite loop temporarily by doing
            mockDataReader.SetupAllProperties();
            mockDataReader.SetupSequence(m => m.IsClosed).Returns(false).Returns(true).Returns(true).Returns(true);
            mockDataReader.SetupSequence(m => m.NextResult()).Returns(false);

but that seems to not be enough, because the DataSet is returned but has no columns or rows
the extension method provided by the Kusto nuget package seems to be using DataTable.Load() to load the data, which the source code is here: https://github.com/microsoft/referencesource/blob/master/System.Data/System/Data/DataTable.cs#L4459 , but is fairly complicated, so i would need some time to see what is happening
I'm wondering if it would honestly be easier to just implement IDataReader itself rather than using Moq to mock the individual methods, as that would still provide immense value as implementing IDataReader is complicated for a single developer to do, and having the extension method is handy, and implementing it normally without mock would make it so all edge cases would be less likely to happen. Honestly i think the API could still stay the same, as you still have the MockDataTable object that you pass into SetupReturns(), and no would needs to know  or care that it is an actual IDataReader implementation
I am willing to help contribute either way, however i am busy at the moment so I wanted to file this issue before i forget

Hello @mgrandi , thanks for the details. I did have a look at the call stack and figured out that it is much complicated or even not possible, because of the new() objects like DataTable being created on the fly. I don't think that the code is that easily unit testable.
Here is the call stack (methods), that i could trace out.
using System.Data;

public static DataSet ToDataSet(this IDataReader reader)
{
    return reader.FromDataReader("DataSet");
}

using System.Data;
using Kusto.Cloud.Platform.Utils;

public static DataSet FromDataReader(this IDataReader reader, string name)
{
    using (reader)
    {
        DataSet dataSet = new DataSet(name);
        int num = 0;
        bool moreData = true;
        while (moreData)
        {
            DataTable dataTable = ExtendedDataTable.FromDataReader(reader, out moreData);
            if (string.IsNullOrWhiteSpace(dataTable.TableName))
            {
                dataTable.TableName = $"{name}_Table_{num++}";
            }
            dataSet.Tables.Add(dataTable);
        }
        dataSet.AcceptChanges();
        return dataSet;
    }
}

using System;
using System.Data;
using Kusto.Cloud.Platform.Utils;

public static DataTable FromDataReader(IDataReader reader, out bool moreData)
{
    moreData = false;
    try
    {
        DataTable dataTable = new DataTable();
        DataTable schemaTable = reader.GetSchemaTable();
        dataTable.Columns.CollectionChanged += Columns_CollectionChanged;
        dataTable.Locale = ExtendedCultureInfo.InvariantCulture;
        dataTable.Load(reader);
        dataTable.Columns.CollectionChanged -= Columns_CollectionChanged;
        DataColumn dataColumn = schemaTable.Columns["ColumnType"];
        DataColumn dataColumn2 = schemaTable.Columns["ColumnOrdinal"];
        if (dataColumn != null && dataColumn2 != null)
        {
            foreach (DataRow row in schemaTable.Rows)
            {
                int index = Convert.ToInt32(row[dataColumn2]);
                dataTable.Columns[index].ExtendedProperties.Add("ColumnType", Convert.ToString(row[dataColumn]));
            }
        }
        moreData = !reader.IsClosed;
        return dataTable;
    }
    finally
    {
        if (!moreData)
        {
            reader.Dispose();
        }
    }
}

Which then calls the code you referred, DataTable.Load(reader, ...).
As you mentioned, I done see a real value in trying to make this work. However, I came across A Test Helper that could help in Mocking this, if there is some time to spend. Please have a look.

thanks for taking a look!
would it be worth it to change the logic or add a second set of classes  to actually implement a full IDataReader? (dependingon if doing that breaks any existing usage of the current code)
it seems that the current set of classes work for some use cases but there are others where like Kusto are expecting a full IDataReader and it might be worth to implement one, and it might actually be easier to do rather than using Moq methods alone like the current code does


Version object

A formal version object would be a useful addition; it would prevent all the existing Version/VersionString pairing that goes on in every module that uses Carbon.
It could also help in establishing formal module versioning through module metadata directives.

What would I call such an object, with Carbon.Version being assigned to a field already?

Is this actually important?


velodyne: Update to version 1.2.0

Bump from Velodyne version 1.1.2 (Nov 2013) to the latest release,
1.2.0 (Aug 2014).  This version includes a fix to support yaml-cpp
1.5+ so resolves #325 [1].
[1] https://github.com/bmwcarit/meta-ros/issues/325
Signed-off-by: Ash Charles<EMAIL_ADDRESS>

:+1: Looks good. I probably just forgot to do the whole packagegroup-ros-world testing with the last pull request, so I am happy that you noticed it and resolved it. :100:


Add fthomas/refined to Adopters

The alternative title of this change is "I love sbt-ci-release". :-)
Since I switched to sbt-ci-release in my projects, publishing is almost pain-free. Many thanks @olafurpg for this great plugin!

Sorry about that, I've removed the tag from the commit message.

Releasing has gotten even easier now with Scala Steward! Just push a tag and merge update PRs 😁


Spikes don't deal damage on rotated tips

When rotated a cardinal direction, spikes don't do damage when hit straight on the tip. Only when touched sideways or stepped up on via the "stair effect".

Yeah, well, this is stupid, this is because minecraft only runs onCollidedWithBlock if you are actually inside that block space, not touching it.  But apparently 0.05 pixels is enough for it to care.


Support shift (keyboard) to select/unselect multiple rows

It would be great if we could use the shift key (in the keyboard) to select multiple rows.
For example - click on the "select" of row#1, press down the shift key, click on the "select" of row#5 -> result is that rows 1,2,3,4,5 are selected.
Same with unselect.

I'll consider to implement it but actually, it's a good idea to have some ways to trigger selection, I will also consider to implement on react-bootstrap-table2  thanks

@AllenFang Was this implemented in react-bootstrap-table2?

@AllenFang how can this be implemented?

@AllenFang, I have found that this is a very important feature for wide adoption on list and select libraries.  I find that the majority of the community libraries have skimped on this one.  I've just implemented this functionality on my own dropdown library (in beta) successfully, and would be glad to contribute here, or share my own code as a starting place, or even just write out the detailed technical requirements if that will help.
This feature took lots of careful testing because handling keyup & keydown for shifts w/ arrows up/down or even changed mid-select is a lot of conditions while allowing for mouse click and shift-click to interoperate with the selections w/o issue.  Add to that, managing the transitions across rows which were pre-selected before the shift-selection was begun, etc.  I also found that the checkbox state boolean is not always the same as the row state boolean, depending on which elements are being used to construct the row and its wrapper(s).  There are also some choices to consider as possible props for what happens when a selection is in place and a non-shift-selection occurs, which methods to clear all are supported or defaulted, etc.  For larger datasets, performance becomes a consideration since certain points of selection state can only be understood or verified by searching the whole set.  I also implemented a SelectionGroup dropdown which allowed for multiple list boxes to be configured.  I wouldn't suggest that this was productive for most use cases or library weight, but it was very useful for me to learn that several issues that happen when multiple list boxes are deployed to keep state integrity between each, and to leave hooks for handling z-index layering, etc.
I am in process of migrating from react-bootstrap-table to -table2 on a smallish project where I must have shift-select, so I will need to go a different direction if I can't find a way to "hack it" in or else find the feature added.
Please let me know if I can help.  thx

Dear, pls advice. I use react-bootstrap-table
I can't select few checkboxes when holding shift. It's select only one.
How to set up to be able select checkboxes according click start and finish holding shift ?
Thank you in advance !
Oleg

@olegshepel It would take me some time to do a complete write-up on this, but I will do a quick brain dump here.  Maybe that will be useful.  The essential concept is to define and manage the user's "shift-selection" operation as a mini-lifecycle, and then commit their "selection-groups" to the underlying component's props and state only where and when needed to cause a re-render.  You could do what I'm about to describe by creating your own component to wrap and implement the underlying table component along with your shift-select handling.  This would keep things very tidy and away from your own app's implementation.  I have done this, but only for specific applications which have too much other stuff going on to share here.  For now, I'm going to explain the basics in a much more "hacky" manner, so that you might get a mental image of the concept requirements.  From there, you could compose and implement much more deeply or elegantly as your case may need, or as your own application separation or quality concerns are to be met.  (you might also be fine with the hack :)
Here are some basic ideas to consider:

your onClick handler can detect shift via the event.shiftKey boolean (or else you could listen for shift with your own listener setup in componentDidMount and removed in componentWillUnMount
the onClick handler can save the row position  (which is also your data array position) that has taken the click event by adding it to your own "management" array.  This can be done a couple ways, but to avoid state, I'll show it this way:  this.myShiftArray[clickPositionValue]

You will have to determine your own requirements for what a shift+click would mean to your user or implementation, but probably some conditional logic in the handler like...:
3. unless myShiftArray is empty, the first click on a row would pass event.shiftKey = false
4. a second click event to the handler which still has event.shiftKey = false would mean the user simply clicked a different row and is not currently performing a shift+click operation.  In this case, simply overwrite this.myShiftArray = [clickPositionValue].  This leaves the shift-selection-group's "start" value as the position last clicked before a shift+click event occurs.
5. alternatively {if...else if...else, etc.}, when event.shiftKey = true, your handler would then check to see that this.myShiftArray.length === 1, so it can know that a prior click has occurred and the user is now performing a shift+click to complete the selection from that point (use min/max points to allow upward/downward selection).  This is when you would perform a for loop to update your component's row selections to match the group of rows between the two points which the user has just "shift-selected".
This could mean a couple possibilities:
a. setState() on your "selected" array which would pass down as props after render
b. or maybe you do a dom.click() on each node to simulate the user's action (using something like event.target.parentElement.querySelectorAll("div"), etc.) to find those nodes
c. or updating just the style or class value for those particular nodes if you are not passing selected items down to the component.  You might find that passing the selections down to the underlying component could get very costly in render time if you are dealing with table's re-rendering hundreds of dom nodes.  With react-bootstrap-table2 for instance, I let it handle the background style operation internally via its native row click, but I do not use the underlying onSelect or selected props, because the render time is way to long for a user who is in the middle of shifting and selecting, etc.  With other libraries, you may have different options.
d. your own invention...
Whether a, b, or c, You may want to skip rows which are already selected on the dom (could use another this.alreadySelected for this or else check the dom nodes for a "selection-row" class, etc. ) or are marked as nonSelectable by the underlying component (I use a this.state.nonSelectableIds array which gets updated as relevant).
NOTE:  With b or c you will want/need to skip component updates via shouldComponentUpdate to avoid re-renders... again, it depends on the lib onto which you are bolting this functionality.  You can set a boolean like this.skipShouldUpdateCase = true when you perform b or c.  Then in shouldComponentUpdate, you will want to do something like if ( this.skipShouldUpdateCase ) { this.skipShouldUpdateCase = false; return false; }.
...here is only slightly better than pseudo-code to look at for some idea:
`handleClick = ( event, dataRow, dataRowIndex ) => {
const divNodes = event.target.parentElement.querySelectorAll("div")
, isSelected = divNodes[dataRowIndex].classList.contains("selection-row")
, isShift = event.shiftKey
if ( this.myShiftArray === undefined ) this.myShiftArray = []
if ( !isShift ) {
    if ( !isSelected ) {
        //normal select, overwrite this.myShiftArray
        this.myShiftArray = [dataRowIndex]

    } else {
        //DeSelect, remove from this.myShiftArray
        this.myShiftArray.length && this.myShiftArray.splice( 0, 1 )
    }

} else if ( isShift ) {
    if ( !isSelected ) {
        if ( !this.myShiftArray.length ) {
            this.myShiftArray = [dataRowIndex]

        } else {
            this.myShiftArray.push( dataRowIndex )
            const rowPosArray = this.doShiftClickForLoop( divNodes, Math.min(...this.myShiftArray), Math.max(...this.myShiftArray) )
            this.refreshSelectionArray( divNodes, rowPosArray, true ) //push = true, splice = false, etc.
        }

    } else {
        this.myShiftArray.length && this.myShiftArray.splice( 0, 1 )
    }

}

}`
Good luck, I hope this helps.  If you get your prototype in place in a repo somewhere, I could certainly help a bit within reasonable time constraints.


Document nodejs >= 14 requirement due to ES2020 features

elm-watch makes use of ES2020 features such as the nullish coalescing operator ??. This is only supported in nodejs >= 14.

Yeah, I think it's worth it, as some Linux distributions package rather old Node.js versions in their stable repos, e.g. Debian stable (bullseye) is at node 12, current Ubuntu LTS (20.04) even defaults to node 10, and even the upcoming Ubuntu LTS (22.04) will come with node 12 apparently.
As an occasional node user who just want to use stuff like elm-watch on node, but not develop for node, I am typically fine with using whatever my distro currently provides and maybe not aware of upstream support lifecycles, so adding a hint to check whether my installed version is sufficient would be useful for me.

Can you try npm install elm-test with the Node.js and npm that comes with your distro? They use the "engines" field in package.json to specify the minimum required Node.js version.
https://github.com/rtfeldman/node-test-runner/blob/fd52f043b9ae6ce20596af123af3be1a7967a383/package.json#L6-L8

Can you try npm install elm-test with the Node.js and npm that comes with your distro? They use the "engines" field in package.json to specify the minimum required Node.js version. I’m thinking if that’s good.

Sure. Unfortunately it doesn't have any effect, neither during installation nor when runnig elm-test.
I've tried it on a Linux Mint (Una, based on Ubuntu Focal) machine which comes with Node.js 10.19.0, so it should have failed in some way... On Debian Bullseye I wouldn't have any issue anyway with elm-test as it is currently at 12.22.5.


nerdctl inspect does not allow inspecting networks, volumes

Description
docker inspect allows inspecting networks, volumes, images, containers.
nerdctl only allows containers and images
Steps to reproduce the issue

nerdctl inspect bridge
2 docker inspect bridge.

Describe the results you received and expected
.
What version of nerdctl are you using?
1.7.6
Are you using a variant of nerdctl? (e.g., Rancher Desktop)
None
Host information
No response

#3017 was not meant to fix this.
Can we reopen?

@yankay @AkihiroSuda can we reopen this?


Broken link

The "More information: Use the Early Bound Entity Classes in Code", under section "Organization service" above seems to be broken. When navigated gives a Access denied exception.

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

ID: ace4bd59-0cbe-d4c6-b4e1-3dc2c8e0acd0
Version Independent ID: 88ebb402-85ff-04d0-21e0-34d007d7aeef
Content: Programming models (Developer Guide for Dynamics 365 for Customer Engagement apps)
Content Source: ce/developer/programming-models.md
Service: crm-online
GitHub Login: @JimDaly
Microsoft Alias: jdaly

@alokkms The link works fine for me. Can you try opening this doc in incognito mode and see if it returns the same Access Denied error ?

Closing this issue as the link works fine.


C3 homepage redesign ideas

Layout Inspiration:
https://gleam.run/
https://dotnet.microsoft.com/en-us/languages/csharp
Component inspiration
https://nextjs.org/
https://vercel.com/frameworks/nextjs
This is a bit busy, but has some interesting elements
https://crystal-lang.org/
Other language websites
https://ocaml.org/
https://jule.dev/
https://odin-lang.org/
https://ziglang.org/
https://www.rust-lang.org/
https://www.python.org/
https://go.dev/
https://www.scala-lang.org/
https://www.haskell.org/
https://www.roc-lang.org/
https://haxe.org/
https://www.swift.org/
Frameworks
https://svelte.dev/
https://www.solidjs.com/

Link to tutorials for solving specific needs

Good tutorials to get people on the road to having stuff up and running could be really interesting here, then people can learn "just in time" while trying to solve the problem of making their game starting from a tutorial and template/starter repo


Python: Add commits of type docs to the changelog

See https://github.com/googleapis/python-pubsub/pull/117#issuecomment-641789106
Also clarify that it is normal to not see all the commits listed in the release notes. (This has been a point of confusion a few times in the Python repositories).
CC @plamut

@busunkim96 👋 this can be configured. I think we can use the same approach we used for Ruby for Python:
https://github.com/googleapis/release-please/blob/master/src/releasers/ruby-yoshi.ts#L29

this can be configured.

Great!
How do we configure it? Can you point to some documentation please?

@jcornaz The PR shows an example of this

Closing as the original issue is resolved.

@jcornaz The PR shows an example of this

I don't understand. That's a code change in this repository.
Does it mean that the only way to "configure" is to actually fork this repository and change the code?
Nevertheless as this issue is closed should I open a new issue for adding the documentation?

@jcornaz 👍  I think opening a new issue with your question would be best.

@busunkim96 I created #511

Has this been reverted by https://github.com/googleapis/release-please/pull/1104#pullrequestreview-820573534?


Redis pubsub messages are not properly handled

Expected Behavior
I'm not an expert in redis streams, but we're working on seeing what kind of metrics we can get out of it in order to see if it's viable to implement dapr-pubsub using redis streams.
The expected behavior is that when dapr has successfully delivered a message from sender to receiver the message should be handled according to the specification of the underlying pubsub driver (redis streams in this case).
This means calling the XACK command on the message(s) to remove them from the list of "pending messages" (this is documented at https://redis.io/commands/xack)
Actual Behavior
It looks to me as if dapr never XACK's a message, which casues it to stay in redis in a pending state. Over time, this buildup is dangerous and it also causes unecessary re-delivery of messages.
This buildup of pending messages can be seen by issuing a XPENDING command, or by using the redis_exporter docker image (this way there's no need to know the names of streams/consumer groups before hand, so it's a little easier):
docker run -d -e REDIS_EXPORTER_CHECK_STREAMS="*" --net=host docker.io/oliver006/redis_exporter
curl http://localhost:9121/metrics | grep redis_stream

The cause seems to be this part of the redis implementation:
https://github.com/dapr/components-contrib/blob/e881e4ba28e34fdb26a89533a8ee881fb5d87b79/pubsub/redis/redis.go#L150-L153
Messages are acked if the err object is empty, but by implementing a bit of extra logging:
				err := handler(&msg)
				if err != nil {
					r.logger.Infof("error: %s", err)
				}
				if err == nil {
					r.logger.Infof("acking message %s in stream %s and group %s", message.ID, stream, consumerID)
					r.client.XAck(stream, consumerID, message.ID).Result()
				}

At least on my machine, every message causes an error:
INFO[0000] error: unknown status returned from app while processing pub/sub event 1753be78-2a1c-46c0-b6e8-a06362a06c14: ok  app_id=dummy-plugin-dev instance=trond-x1 scope=dapr.contrib type=log ver=edge
INFO[0000] error: unknown status returned from app while processing pub/sub event 833f4d53-898e-4c12-a957-025c4b2b8af6: ok  app_id=dummy-plugin-dev instance=trond-x1 scope=dapr.contrib type=log ver=edge
INFO[0013] error: unknown status returned from app while processing pub/sub event f1ffc01d-cee8-4a76-9591-9d0a78325fc6: ok  app_id=dummy-plugin-dev instance=trond-x1 scope=dapr.contrib type=log ver=edge
INFO[0013] error: unknown status returned from app while processing pub/sub event 81cf3630-b87d-4052-8ac8-44a9ecc3bbc0: ok  app_id=dummy-plugin-dev instance=trond-x1 scope=dapr.contrib type=log ver=edge

So as far as I can see there are several  issues here, silently swallowing errors is one. I'm not well-versed enough in Go to be able to figure out how the "handler" call works.
EDIT: It almost looks to me like the err object is filled by an "ok" - indicating that there might not be an error at all, so the bug is simply that an err object is returned when a message is handled successfully - but I'll leave that investigation to someone with a better understanding of the codebase.
Steps to Reproduce the Problem
Use one of the dapr quickstarts utilizing redis pubsub, send a few messages, and use one if the procedures listed above to grab the list of pending messages
Release Note
RELEASE NOTE: FIX Sucessfully delivered messages are now properly XACKed when using Redis pubsub

Investigating this a little further, the unexpected "ok" seems to come from the fact that our app didn't properly respond to dapr using an expected payload (https://docs.dapr.io/reference/api/pubsub_api/#expected-http-response) - we simply returned an return {'status': 'ok'} which dapr didn't know how to handle.
That said, I would still consider this a fairly important bug:

the redis pubsub silently discards errors instead of at least properly logging them

the runtime handler responsible for parsing the response after forwarding a message looks like this:
https://github.com/dapr/dapr/blob/2c0633d3ecccc429b804f3e355d8e9add78cc79b/pkg/runtime/runtime.go#L1124-L1125
The way things are right now, this error is show anywhere because of the way redis pubsub silently discards errors. Furthermore, the error could be made way more user-friendly, for example:
return errors.Errorf("unknown status returned from app while processing pub/sub event %v: status %v is not one of SUCCESS, RETRY, DROP. Assuming error", cloudEvent.ID, appResponse.Status)

In general (and I know I've said this before in other issues), dapr needs some solid improvement in the logging department. It is far to difficult to understand what's going on when things arent going perfectly to plan.


predict single instances

Hello, sorry for bothering. I've just trained the model and I was wondering how could I predict (using the model) the comment of a single instance of code and see the result.
Thank you for your time

Like the testing code, you don't need consider the file processing or pytorch dataloader. Replace the source_ids in line491 with your tokenized comment. Then print text after line497.


Installation freezes at "Unzipping Dart SDK" in android studio

Recently I had installed flutter plugin in android studio. And when I went for new project, it asked me flutter sdk. I chosen "Install SDK". And it was freezes at ""Unzipping Dart SDK". I kept as it is for more than 3hrs, but was there is no any response.
Please can any one help me to continue....

cc @devoncarew

same problem. I met this when run "flutter dector", it just hung at Unzipping Dart SDK... forever.
I work on Windows

This issue was moved to flutter/flutter-intellij#1961


[#1] Create build pipeline.

Tests are running/passing: https://github.com/Lokulus/ClaimsTransformation/actions/runs/1158370508


Cannot read property 'setSecurity' of undefined - Node JS, Strong-Soap

I try to deploy my node application inside the "IIS 8". I have installed iisnode.
The basic application is running perfectly.
But I go with SOAP, I am getting below error.
Application has thrown an uncaught exception and is terminated:
TypeError: Cannot read property 'setSecurity' of undefined
    at C:\inetpub\iisnode\www\salesorder\so.js:32:11
    at C:\inetpub\iisnode\www\salesorder\node_modules\strong-soap\src\soap.js:48:5
    at C:\inetpub\iisnode\www\salesorder\node_modules\strong-soap\src\soap.js:30:16
    at ReadFileContext.callback (C:\inetpub\iisnode\www\salesorder\node_modules\strong-soap\src\parser\wsdl.js:370:11)
    at FSReqWrap.readFileAfterOpen [as oncomplete] (fs.js:336:13)

Code:

var express = require('express');
var app = express();

app.get('/node/salesorder/myapp/salesorderquery', function(req, res){
    var json = req.query;
  var soap = require('strong-soap').soap;
  var url = '../wsdl/SalesOrderQuery.wsdl';
  var requestArgs = {
    SalesOrderByElementsQuery_sync : {
      ProcessingConditions : {
        QueryHitsMaximumNumberValue : 2 , //json.QueryHitsMaximumNumberValue,
        QueryHitsUnlimitedIndicator : false,
        LastReturnedObjectID : "0000000000011ED18B8B8EA6C678AAE6"
      }
    }
  };
  var options = {
    attributesKey: '$attributes'
  };

  soap.createClient(url, options, function(err, client) {
    client.setSecurity(new soap.BasicAuthSecurity("**APP","******"));
     client.FindByElements(requestArgs, function(err, result, envelope) {
      res.send(JSON.stringify(result));
    });
  });
})

app.listen(process.env.PORT);

Please help me solve this problem.
Regards Sankaran A

Hi Sankaran-
The code you've posted isn't using node_redis as far as I can see, so we can't help you. Perhaps you want to ask in strong-soap.

Ok thanks @stockholmux . I will raise in strong-soap.


Don't set the Signon instance name in production

Even on AWS. This differs from the current production environment, in
which this is unset, I believe intentionally.
The Signon service assumes that this will be unset in the production
environment, e.g. it won't enforce the use of 2 step verification for
admin users if this is set.

Thanks for taking a look @1pretz1 :)


⚠ Jenkins Plugins Artifactory has degraded performance

In 981554d, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1294 ms

Resolved: Jenkins Plugins Artifactory performance has improved in f87d3b6 after 4 hours, 7 minutes.


Allow snapshot to be loaded from an http request and add progress

Issue summary

Once #761 is in, would be very nice to allow for loading a snapshot from a url, as Lotus does it.
They also added in a progress bar for the snapshot import, and I think this would be a very nice UX to add. For this progress to show up well, it has to be before any of the async tasks are started, as the logger conflicts with the progress bar output (see fetch-params progress bar when requests are done async), but should be pretty straightforward
This issue is definitely low priority since it wouldn't add any new functionality.
Other information and links

I can look into it this week :)

Just for more guidance, I made this PR for importing snapshots https://github.com/ChainSafe/forest/pull/789


How to display single-spa app in the browser without root-app

Question
How can I serve my single-spa angular app and display it in the browser without going through the root app?
If I run the single-spa command: ng s --ssl --project my-app --disable-host-check --port 9001 --live-reload false, then it serves the app and says to open https://localhost:9001. But when I open it, it doesn't display the app on the screen. (But it does display if I run the root-app, where it's registered properly according to the spa docs).
So how can I get it to display when I go to https://localhost:9001?
Environment
Libs:
Angular 17.3.7,
"single-spa": ">=4.0.0",
"single-spa-angular": "^9.1.2",
"zone.js": "~0.14.3"

It happens because of the custom webpack builder and the singlespa-custom webpack. I don't exactly know what the code that "single spa" custom webpack does, but it's necessary to work with single spa.
As much as I know you can't do that without defining another "project" in your angular.json. You should configure that project like a non-singlespa project, using angular 17 esbuild builder for example.
Then, in your package.json you can write a serve script for both of them, specifing the appropriate --project flag.
If someone else knows a better way to do this I'd like to know!

I'm not sure I follow you exactly, but I did get a sort of answer on ChatGPT. If you add this to the end of main.single-spa.ts, it works (unless you run into this zone.js problem with Angular 17):
// For standalone mode
// @ts-ignore
if (!window['singleSpaNavigate']) {
  platformBrowserDynamic().bootstrapModule(AppModule).catch(err => console.error(err));
}

That's awesome!
I was suggesting that solution, but yours is much better.


Hello , i am new to this library is there any way to set max size of image being uploaded here . i want the maximum size of image being uploaded to be 200kb but  setting the compressInitial to 0.1 or 0.2 does not ensure that

There is not such a way. Using the package with jpeg or webp with the corresponding compression rationyou can optimize the size f the image while not losing quality

Non feature non issue , so closing it


fix: Interactive color legend

Fixes #1058.
This PR fixes two problems with interactive color legend issues.

Stacked bar chart was using all segments defined in the data to construct series (coordinates to draw the chart) – when one segment was de-selected using interactive color legend, it would introduce NaNs and thus break the chart. Now we use plottable segments to create series.
Interactive color legend not affecting the chart – as we store abbreviations / labels inside interactive filters state, we need to also get segment abbreviation / label when applying interactive filters to the data (previously we would try to access segment value).

How to test
Re-create the charts specified in #1058 to see that they now work correctly.

Thanks for reviewing @ptbrowne!


The Icon Missing

Your Icon Has Missing @Dxgamer7405 For The Credits.

I Saw It's Showing The Small HaxeFlixel Icon, Beacuse It's Missing The Icon Name For The Credits

I Saw It's Showing The Small HaxeFlixel Icon, Beacuse It's Missing The Icon Name For The Credits

ye of course


Unhandled promise rejection TypeError: Cannot read property 'response' of undefined

Hi. I'm having a lot of trouble trying to incorporate facebook login and using sagas to do it. In my sagas, I have:
function* loginUser(username) {
  yield put(actions.loginUser.request(username));
  const { response, error } = yield call(api.loginUser, username);
  if (!error) {
    yield put(actions.loginUser.success(username, response));
  } else {
    yield put(actions.loginUser.failure(username, response));
  }
}

and then my async call looks like:
function loginUser(username) {
  fetch('/auth/facebook')
    .then(function (response) {
      if (response.status >= 400) {
        throw new Error('Bad response from server');
      }
      return response;
    })
    .then(
      response => ({ response }),
      error => ({ error: error.message || 'Something shitty happened' })
    );
}

However, I keep getting a promise rejection error. The line const { response, error } = yield call(api.loginUser, username); is where it throws. I know that the response is coming back because i console.log it and it's there. But for some reason, the yield call is not able to read the response.
Would you happen to know if I'm doing anything wrong?

Isn't a return statement missed in your loginUser function ?

@yelouafi yes, it is. thank you for getting back so quickly.

Where is the return statement missing and what should it be? I am having this exact same issue. Thanks!

just before the fetch, function loginUser creates a Promise (by using fetch) but it doesnt return it to the caller


Using author's extension in mine

Hi.
Nice extension!
I'm also developing a Chrome Extension and I need to take screenshots too.
Is there a way to integrate your extension (or its part) in my code?
And is there a way to take screenshot of particular selector, like document.getElementById('p1') ?
Thanks.

Take a look at the captureVisibleTab API
https://developer.chrome.com/extensions/tabs#method-captureVisibleTab
To get a specific element, you’ll have to identify its coordinates on the
page and extract that from the screenshot of the visible tab, e.g.,
position and draw it onto a smaller canvas object.
On Fri, Sep 6, 2019 at 2:39 PM Aroxed<EMAIL_ADDRESS>wrote:

Hi.
Nice extension!
I'm also developing a Chrome Extension and I need to take screenshots too.
Is there a way to integrate your extension (or its part) in my code?
And is there a way to take screenshot of particular selector, like
document.getElementById('p1') ?
Thanks.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mrcoles/full-page-screen-capture-chrome-extension/issues/96?email_source=notifications&email_token=AABBLS7UOZ3DEAEEVBLB5ADQIKPXXA5CNFSM4IUL57EKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HJ36X2Q,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABBLSZ5HRAHGJBY72HL5BDQIKPXXANCNFSM4IUL57EA
.


IOS build fix.

system call is disabled on this platform.

does it still compile on MacOS with this?

Yes just tried.


Remove Pac4jWrapper from the Pac4jAuthHandler API

Stateful - and Stateless - Pac4jAuthHandlers should not expose Pac4jWrapper in their constructors,
Instead they should expose the elements from which a Pac4jWrapper is constructed, and create the
wrapper internally when constructed. There is no need to externalize the Pac4jWrapper creation.
Since the stateful handler auto creates the callback handler with which it is associated, it can pass its
internal Pac4JWrapper into the callback handler when it creates it.

Once this is complete, so the Pac4j Clients object is exposed in the constructors, we should migrate to the new Config API (so the handler accepts a Config object).
We should arguably make the Config part of the Pac4jHandlerOptions class.


Search - Backend

We have ElasticSearch in place and populating.  We just need to make sure we have sensible data structures and endpoints in place for performing faceted searches across the collection of resources and studies.  While this should not include location based searches (unless we change our minds), it should include a robust ability to find resources/studies/training that meet criteria that a participant might be interested in.  We can re-use this feature to provide suggestions for participants.

Added in PR #83


Update to use new fluentbit, mdm, and mdsd

Which issue this PR addresses:
Fixes https://issues.redhat.com/browse/ARO-1384 and https://issues.redhat.com/browse/ARO-1386
What this PR does / why we need it:
Updates the fluentbit, MDM, and MDSD images to use latest Mariner 2 versions.
Test plan for issue:
E2E to verify that the images work.
Is there any documentation that needs to be updated for this PR?
N/A

2023/04/20 02:12:37 pkg/deploy/assets/rp-production.json has been modified after make generate
2023/04/20 02:12:37 pkg/deploy/assets/gateway-production.json has been modified after make generate
exit status 1

Looks like you just need to commit these 2 files and you should be good to go

2023/04/20 02:12:37 pkg/deploy/assets/rp-production.json has been modified after make generate
2023/04/20 02:12:37 pkg/deploy/assets/gateway-production.json has been modified after make generate
exit status 1

Looks like you just need to commit these 2 files and you should be good to go

oops, fixed


setPingConnectionInterval for replicated servers configuration is not propagated to underlying MasterSlaveServersConfig

Hey,
When I set setPingConnectionInterval in the replicated servers configuration it's not propagated to underlying master slave configuration:
      config.useReplicatedServers()
          .setPingConnectionInterval(1000)
          .addNodeAddress(nodeAddresses.toArray(new String[0]));

My current understand that ReplicatedServersConfig is used to create MasterSlaveServersConfiguration In the MasterSlaveConnectionManager in https://github.com/redisson/redisson/blob/master/redisson/src/main/java/org/redisson/connection/MasterSlaveConnectionManager.java#L389.
In this method setPingConnectionInterval is not copied and default value is used in MasterSlaveServersConfiguration.

Hi! Right, Thanks for pointing out.

Fixed


(WIP) Make WorkspaceClient and AccountClient friendly with autospeccing

Changes
We cannot create_autospec(WorkspaceClient), because all public properties are created within the constructor:
>>> WorkspaceClient
<class 'databricks.sdk.WorkspaceClient'>
>>> ws = create_autospec(WorkspaceClient)
>>> ws
<MagicMock spec='WorkspaceClient' id='4394200816'>
>>> ws.workspace.list()
Traceback (most recent call last):
  File "/Applications/PyCharm CE.app/Contents/plugins/python-ce/helpers/pydev/_pydevd_bundle/pydevd_exec2.py", line 3, in Exec
    exec(exp, global_vars, local_vars)
  File "<input>", line 1, in <module>
  File<EMAIL_ADDRESS>line 643, in __getattr__
    raise AttributeError("Mock object has no attribute %r" % name)
AttributeError: Mock object has no attribute 'workspace'

This PR makes it easier to introspect the class instances and makes mocks more reliable.
All existing behavior stays the same.
See:

https://docs.python.org/3/library/unittest.mock.html#autospeccing
https://stackoverflow.com/a/35921144/277035

Tests

[ ] make test run locally
[ ] make fmt applied
[ ] relevant integration tests applied

Codecov Report
Attention: 119 lines in your changes are missing coverage. Please review.

Comparison is base (1d04851) 57.72% compared to head (c089093) 57.78%.
Report is 2 commits behind head on main.

Files
Patch %
Lines

databricks/sdk/__init__.py
68.35%
119 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #480      +/-   ##
==========================================
+ Coverage   57.72%   57.78%   +0.05%
==========================================
  Files          38       38
  Lines       26162    26357     +195
==========================================
+ Hits        15103    15231     +128
- Misses      11059    11126      +67

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

random integration test passed:


Linearity Check Fails

The following should break the linearity check, but this typechecks.
bug : Char[] -> Char[]
bug c = let
            [_] : Char[] = c
        in (case True of
                True -> c;
                False -> ['a'])

Note that
bug : Char[] -> Char[]
bug c = let
            [_] : Char[] = c
        in (case True of
                True -> c;
                False -> c)

and
bug : Char[] -> Char[]
bug c = let
            [_] : Char[] = c
        in c

both successfully report Linear variable c is used more than once.

You can also substitute [0..1] with [] or  [Private] and the issue is the same.

FROM haskell:9.2.5

RUN git clone https://github.com/Z3Prover/z3.git --branch z3-4.11.2
WORKDIR z3
RUN python scripts/mk_make.py
WORKDIR build
RUN make
RUN make install

#RUN wget -L https://github.com/Z3Prover/z3/releases/download/z3-4.11.2/z3-4.11.2-x64-glibc-2.31.zip
#RUN unzip z3-4.11.2-x64-glibc-2.31.zip

WORKDIR /
RUN git clone https://github.com/granule-project/granule.git --branch v<IP_ADDRESS>
WORKDIR granule
RUN stack setup
RUN stack install

RUN echo "--include-path /workspaces/Graded-Modal-Parsers/src/StdLib" >  ~/.granule

ENTRYPOINT ["tail", "-f", "/dev/null"]

I have a dev container build for <IP_ADDRESS> as well and the issues exists there as well.

bug : Int -> Int
bug x = let () = drop@ Int x in (if True then x else 7)


Add a success message to the 'Join' page

To repro:

Visit https://operationcode.org/action
Fill in the form
Submit the form

Currently there is no success flash or other visual indication that your form has been submitted.
There should be a success flash and maybe a message that explains what to explain next.

@rickr Agree. Something along these lines.

In fact, need to add to Stripe donation as well.

Closing this since the login process has changed.


Rerendering when other parts of the context change

I am using React Router with React Redux. I'm having a problem where any component that uses the routing context (e.g. <Link>) rerenders whenever the Redux store updates. Since I have a lot of links and frequently update the store, I'd like to prevent this rerendering.
My suspicion is that it's rebuilding the routing context object each time the context changes, which causes every consumer of the context to update. So, I think that could be fixed by caching the context in Route and Router, as prescribed in the React documentation's advice on context.
My workaround is to reimplement Link so that it memoizes its anchor and can receive props from withRouter, which seems to have solved the problem. But I would rather not have to do that.

What versions of each library are you using? It shouldn't be doing this.

The latest one: 5.0.0.

What about React Redux?

Oh whoops. React 16.8.0, Redux 4.0.1, and React Redux 6.0.1.

I should have made sure everything was up to date first. Upgrading to React-Redux 7.0.1 fixed it.


Couple of changes to cmd.js

Commits documented and signed off. Ready to discuss as needed.

I'm curious why you need the --exit option? The auto-closing behavior seems to work fine for me. What's your use case?

Seeding the torrent when done downloading.
So far the only work around Ive found was to relaunch with seed instead of
download which is not efficient on several gb torrents (long rehash).
Is'nt it best practice to keep sharing by default?
BR,
V
On Sun, 7 Feb 2016 at 21:35, Feross Aboukhadijeh<EMAIL_ADDRESS>wrote:

I'm curious why you need the --exit option? The auto-closing behavior
seems to work fine for me. What's your use case?
—
Reply to this email directly or view it on GitHub
https://github.com/feross/webtorrent/pull/600#issuecomment-181107223.

Yeah, you make a good point. My concern is that lots of folks use the command line to fetch files and then do other things with them. Example:
webtorrent magnet_uri --stdout | some-other-program

We can make it so that --stdout implies --exit.
In the future, please send separate PRs for each change you want to make so they can be discussed and merged individually.

I like this PR, except for the changes you made to the UI. Can you remove those? It doesn't make sense to me to hide the detailed peer info under --verbose when we're already clearing the entire terminal and taking up the whole terminal buffer.
In another PR/issue, I'm open to making a more minimalist download UI similar to the npm3 one:

And we can hide the current one behind --verbose.
Sound good?

Sounds good.
I ll open new PRs for the port and exit including the implicit for stdout.
Feel free to drop this one.
Cheers,
V
On Sun, 7 Feb 2016 at 21:51, Feross Aboukhadijeh<EMAIL_ADDRESS>wrote:

I like this PR, except for the changes you made to the UI. Can you remove
those? It doesn't make sense to me to hide the detailed peer info under
--verbose when we're already clearing the entire terminal and taking up
the whole terminal buffer.
In another PR/issue, I'm open to making a more minimalist download UI
similar to the npm3 one:
https://camo.githubusercontent.com/4bf12754e72cf475c5e00a864744ffeaf6c06c19/687474703a2f2f69312d6e6577732e736f667470656469612d7374617469632e636f6d2f696d616765732f6e657773322f6e706d2d332d302d626574612d69732d6f75742d616464732d70726f67726573732d626172732d746f2d7468652d7061636b6167652d696e7374616c6c6174696f6e2d73637265656e2d3438353437342d332e6a7067
And we can hide the current one behind --verbose.
Sound good?
—
Reply to this email directly or view it on GitHub
https://github.com/feross/webtorrent/pull/600#issuecomment-181114367.

Thanks for opening new PRs. That's really awesome of you. I'll look for those.


PIP-136: Sync Pulsar policies across multiple clouds

Sync Pulsar policies across multiple clouds
Implementation : PR: https://github.com/apache/pulsar/pull/16425
Motivation
Apache Pulsar is a cloud-native, distributed messaging framework which natively provides geo-replication. Many organizations deploy pulsar instances on-prem and on multiple different cloud providers and at the same time they would like to enable replication between multiple clusters deployed in different cloud providers. Pulsar already provides various proxy options (Pulsar proxy/ enterprise proxy solutions on SNI) to fulfill security requirements when brokers are deployed on different security zones connected with each other.

However, sometimes it's not possible to share metadata-store (global zookeeper) between pulsar clusters deployed on separate cloud provider platforms, and synchronizing configuration metadata (policies) can be a critical path to share tenant/namespace/topic policies between clusters and administrate pulsar policies uniformly across all clusters. Therefore, we need a mechanism to sync configuration metadata between clusters deployed on the different cloud platforms.
Goal
Replicated metadata event topic
All regions in a cluster that share the same metadata-store (eg: global zookeeper which persists policies) are already in sync but they are not in sync with regions which are in different clusters and not sharing the same metadata-store. We want to sync clusters that are not sharing the same config metadata store and in order to synchronize metadata store, we can pick one region from each cluster and set up a replicated topic across those regions where they can exchange change of metadata event and try to sync cluster with the change of events occurred at different clusters.
This PIP will introduce metadata event-topic which will be replicated between isolated clusters that are not in sync and don’t share the same metadata store. In order to provide replication guarantee, each broker when it receives metadata update, first publishes metadata-change event to this replicated topic and then asynchronously applies the update to the metadata store by consuming metadata event.
Below data structure shows the payload of change events published into the event topic. Each event contains metadata value along metadata path, source cluster name and last-updated time of the event. Source-cluster name and last-update time help destination clusters to handle stale or duplicate events.
public class MetadataEvent {
    private String path;
    private byte[] value;
    private Set<CreateOption> options;
    private Long expectedVersion;
    private long lastUpdatedTimestamp;
    private String sourceCluster;
    private NotificationType type;
}

Handling race condition
Users can update the same policy with different values concurrently in different regions. Every region will eventually receive the updates from other remote regions where policy has been modified and Pulsar has to handle this scenario by merging (or selecting distinct value) the concurrent updates in a consistent manner across all regions. Therefore, each update contains modified-time and the name of source-region which has updated the value. Pulsar region compares local update and remote update based on latest modified timestamp and lexicographical ordering of source-region name and determines a final selected value deterministically across all regions and eventually all regions will have one distinct consistent value for the concurrently modified policy in the metadata store.
For example in the below diagram, Region-A and Region-B received an update for policy P1 at the same time T1. Both regions exchange the local event update with each other and both the regions have to pick only one distinct event from both the updates so, both the regions will have a consistent same update in the metadata store. First, each region compares events based on event updated timestamp and then based on the lexicographic ordering of source-region name. In this example, modified timestamp T1 is the same for both the events so, the next Pulsar selects event with source region-name A over source-region B based on lexicographic sorting on source region name. Therefore, both regions will eventually update metadata with a distinct event that occurred at region-A.

Implementation
Event publisher and handler
Publisher
Every broker receives a metadata-update publishes event message to the persistent topic (metadata-event-topic) which replicates to other clusters. Metadata store publishes change-event when it receives create/update/delete operation for metadata. After publishing the message, metadata-store immediately tries to perform create/put/delete operation asynchronously so, the metadata-store doesn't have to change the existing put/delete API which returns the metadata-store update Stat after making an update in the metadata-store. With this approach, local region tries to apply metadata-change event twice

async update when broker receives metadata-change request from user.
Metadata sync-topic listener receives the event and applies the change to metadata store.
However, broker has race-condition handling which handles duplicate updates for the same event.

Consumer Handler
Every region consumes events from this topic and applies changes to the metadata store accordingly. This PIP introduces MetadataEventSynchronizer to publish metadata events and consume events from the topic and handles updates in the metadata store.
MetadataEventSynchronizer creates a failover consumer on the metadata-event-topic so, that only one of the brokers’ synchronizer can consume and handle the event update.
MetadataEventSynchronizer.java
**
 * Metadata synchronizer to notify and synchronize metadata change events.
 */
public interface MetadataEventSynchronizer {

    /**
     * Notify metadata change event.
     * @param event
     *            metadata change event.
     * @return
     */
    CompletableFuture<Void> notify(MetadataEvent event);

    /**
     * Register notification listener to sync metadata event in local cluster.
     * @param event
     */
    void registerSyncListener(Function<MetadataEvent, CompletableFuture<Void>> event);

    /**
     * Name of current cluster served by the Synchronizer.
     * @return clusterName
     */
    String getClusterName();

    /**
     * close synchronizer resources.
     */
    void close();
}

Broker changes
Configuration
# topic name to share metadata changes from local metadata store
private String metadataSyncEventTopic;

# topic name to share metadata changes from configuration metadata store
private long configurationMetadataSyncEventTopic;

MetadataEventSynchronizer implementation
PulsarMetadataEventSynchronizer which implements MetadataEventSynchronizer and handles metadata events’ notification and processing.
Event topic consumer and publisher
User can enable this feature by configuring metadataSyncEventTopic / configurationMetadataSyncEventTopic into broker and broker initializes MetadataEventSynchronizer component which creates failover consumer to listen and handle metadata’s change events. It also enables broker to publish metadata changes into event topic.
PIP:
https://github.com/apache/pulsar/pull/15223
Rejected alternative

https://github.com/apache/pulsar/issues/13728

Use System-topic:
Use System-topic to synchronize metadata across the cluster. It might not be the correct choice to utilize system-topic to handle metadata-store transportation. Because system topic helps broker to persist topic policies in that local cluster whereas Metadata-event synchronizer helps broker to copy metadata-store across two independent clusters which don't share metadata-store/global-zookeeper. Users will also not be able to use system-topic  for metadata sync due to the below reasons:

storage and reliability: Not every user prefers or uses the system topic for the metadata storage due to multiple reasons such as legacy-system, higher reliability on metadata-store compared to system-topic stored in bookies.

Schema compatibility; System topic right now supports only topic level policies with a specific schema whereas the metadata change event requires a different schema for the metadata-store update.

Merging and handling capabilities: Metadata change event not only requires different schema but also requires special handling for create/update and merging capabilities. It will require unnecessary enhancement on system-topic to support merging capabilities.

Compaction requirement: system topic also requires compaction which all systems don't enable because compaction comes with an extra server-side cost which is very expensive for large scale and multi-tenant systems,

However, system-topic can work with metadata-synchronizer. System topic persists topic policies. Broker reads this compacted system topic to retrieve topic policies and applies them to the loaded topic. The broker can replicate metadata-store data to another destination broker that is part of a separate cluster using a metadata-synchronizer, and the destination broker can later persist policies in the local cluster by publishing them to system-topic.

@rdhabalia
since this PIP is mentioned in release notes of 2.11.0
see https://pulsar.apache.org/release-notes/versioned/pulsar-2.11.0/
-> can this issue be closed as fully implemented?


Change the transcription language to spanish

Hi guys! Thanks for the voicemail stack, it worked like a charmed and I have installed it on our connect instance.
Everything works fine with the voicemail solution, we are able to get the sms and email with the link to the audio file and the the transcription just fine as long as the voice message is left in english, but we handle only spanish callers.
Could you can please point me to where I could configure the transcription language? Not a developer but I have some AWS training.
Thanks for your time!
Eduardo

Hi Eduardo,
Im glad it is working well for you! If you want to change the language to Spanish, you can do so by setting an attribute in your VM-Greeting contact flow called 'languageCode'. The potential values for the language code are detailed here: https://docs.aws.amazon.com/transcribe/latest/dg/what-is-transcribe.html
Thanks!

Hi! Thanks for the help, I tried setting the attribute at the beginning of the VM-greet CF, please see attached files. When I tried calling the voicemail now it acts as if the voicemail is being recorded but the email and SMS are not sent anymore.
Can you help me identify what I did wrong please?
Thanks!
Eduardo

Hi Eduardo,
Im glad it is working well for you! If you want to change the language to Spanish, you can do so by setting an attribute in your VM-Greeting contact flow called 'languageCode'. The potential values for the language code are detailed here: https://docs.aws.amazon.com/transcribe/latest/dg/what-is-transcribe.html
Thanks!

Hi! Can you help me out? i uploaded the screen captures of how I tried to add the attribute, but it stopped working completely.
Thanks a lot!

Hi Eduardo,
You need to set the contact attribute block to 'Use Text' instead of 'Use attribute'. Right now it is setting the language code to another attribute called 'es-US'. This should fix your problem

Hi Eduardo,
You need to set the contact attribute block to 'Use Text' instead of 'Use attribute'. Right now it is setting the language code to another attribute called 'es-US'. This should fix your problem

Thank a lot! That was it, i got it working now in spanish.


Add RemoveResponseHeaders

Adds RemoveResponseHeaders method that allows removal of certain headers from response recording, associated plumbing, and test coverage. Intent is to make it easy to remove useless headers added by proxies and the like.

Thanks!


Refresh the FAQ

What type of PR is this?
/kind documentation
What this PR does / why we need it:
Following the feedback from @robscott on #1862, this PR consolidates some of the FAQ answers and adds links to relevant docs.
Which issue(s) this PR fixes:

Fixes #1862
Does this PR introduce a user-facing change?:

NONE

/lgtm
/hold for other maintainer approval though.


[BUG] When using View menu, folders or images are shown on a single line

Describe the bug
When using V1.6, if I select View, Folders or View, Albums when you select one, everything is on a single line rather than wrapping.
Version:
V1.6
Tool:
N/A
To Reproduce
As described in 'describe the bug'
Expected behavior
If selecting View, Folders or View, Albums then when clicking on a folder, pics in that folder, etc are shown in the next pane wrapped to fill the space.
Affected Image(s)
N/A
Screenshots

What screen resolution do you have? I'm not able to reproduce this issue currently and will need more information.

I also have this problem. My main monitor resolution is 2560x1600, but it also happens in the second one at 1920x1080. Both are at 100% scale.

Can you check if this fixes the issue?
https://github.com/RupertAvery/DiffusionToolkit/actions/runs/7855752123/artifacts/1235591850

Yes! It fixed it.

I'm using 2560x1440 at 100% scale. Can confirm the link provided fixes it for me also.

Can you check if this fixes the issue?
https://github.com/RupertAvery/DiffusionToolkit/actions/runs/7855866422/artifacts/1235608381

looks like it fixed it for me too, 5120x1440 display at 150% scale

Fixed in the latest version (v1.6.1)


Missing processor

I'm getting this error:
<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
<plist version="1.0">
<array>
	<dict>
		<key>RecipeError</key>
		<string>Unknown processor 'XcodeBuildNumberEmitter'.</string>
	</dict>
</array>
</plist>

I deleted and made again the override. I'm always getting this error and I can't find this processor.

Whoops, sorry. I added it back in c007f8f3e7e683ea3b46fc90b3b5ff8a7e017efa. Can you test again and verify?

Hi!
Now I'm getting this error:
  File "/usr/local/bin/autopkg", line 2887, in <module>
    sys.exit(main(sys.argv))
  File "/usr/local/bin/autopkg", line 2881, in main
    exit(subcommands[verb]["function"](argv))
  File "/usr/local/bin/autopkg", line 2327, in run_recipes
    search_github=make_suggestions,
  File "/usr/local/bin/autopkg", line 430, in load_recipe
    auto_pull=auto_pull,
  File "/usr/local/bin/autopkg", line 430, in load_recipe
    auto_pull=auto_pull,
  File "/usr/local/bin/autopkg", line 430, in load_recipe
    auto_pull=auto_pull,
  [Previous line repeated 982 more times]
  File "/usr/local/bin/autopkg", line 400, in load_recipe
    auto_pull=auto_pull,
  File "/usr/local/bin/autopkg", line 307, in locate_recipe
    recipe_file = find_recipe(name, override_dirs + recipe_dirs)
  File "/usr/local/bin/autopkg", line 205, in find_recipe
    return find_recipe_by_identifier(id_or_name, search_dirs) or find_recipe_by_name(
  File "/Library/AutoPkg/autopkglib/__init__.py", line 318, in find_recipe_by_identifier
    matches = glob.glob(pattern)
  File "/Library/AutoPkg/Python3/Python.framework/Versions/Current/lib/python3.7/glob.py", line 20, in glob
    return list(iglob(pathname, recursive=recursive))
  File "/Library/AutoPkg/Python3/Python.framework/Versions/Current/lib/python3.7/glob.py", line 71, in _iglob
    for dirname in dirs:
  File "/Library/AutoPkg/Python3/Python.framework/Versions/Current/lib/python3.7/glob.py", line 72, in _iglob
    for name in glob_in_dir(dirname, basename, dironly):
  File "/Library/AutoPkg/Python3/Python.framework/Versions/Current/lib/python3.7/glob.py", line 83, in _glob1
    return fnmatch.filter(names, pattern)
  File "/Library/AutoPkg/Python3/Python.framework/Versions/Current/lib/python3.7/fnmatch.py", line 51, in filter
    pat = os.path.normcase(pat)
  File "/Library/AutoPkg/Python3/Python.framework/Versions/Current/lib/python3.7/posixpath.py", line 55, in normcase
    if not isinstance(s, (bytes, str)):
RecursionError: maximum recursion depth exceeded in __instancecheck__```
This happen using my override and the original recipe.

Try deleting and recreating your override. That error suggests that a recipe identifier is also listed as its own parent, causing a recursion loop.

Hi Nick!
That solve the issue.
Thank you very much !!!!


Rebase master changes to v1.1-branch

Address https://github.com/kubeflow/tf-operator/issues/1321
This branch will be used to serve tf-operator users. It goes into maintenance mode.
We will do a rebase and merge to keep commit history. This would be similar like we cut a new v1.1-branch from master.

Seems we can do that due to CLA issue. Let me manually sync the branch

➜  tf-operator git:(v1.1-branch) ✗ git push -f  upstream v1.1-branch
Total 0 (delta 0), reused 0 (delta 0), pack-reused 0
remote: error: GH006: Protected branch update failed for refs/heads/v1.1-branch.
remote: error: Required status check "cla/google" is failing.
To github.com:kubeflow/tf-operator.git
 ! [remote rejected]   v1.1-branch -> v1.1-branch (protected branch hook declined)
error: failed to push some refs to 'github.com:kubeflow/tf-operator.git'


pip installation?

It seems that this lib is not yet available on the pip  registry. If you could installation steps to the doc, that would be great!

This project has been on PyPI since October 2011:
https://pypi.org/project/humanize/
There are a couple of badges on the README, and it can be installed in the usual way, but I've updated the README and docs to include installation instructions:
Installation
From PyPI
python3 -m pip install --upgrade humanize

From source
git clone https://github.com/github.com/python-humanize/humanize
cd humanize
python3 -m pip install -e .


feat: add gcloudRun and use getExecOutput

This introduces two new functions gcloudRun and gcloudRunJSON which use @actions/exec#getExecOutput to execute gcloud commands. It checks if the command was successful. On failure, it throws an error and prints stderr and stdout. On success, it returns the stderr, stdout, and combined output.
Introducing this function also greatly simplified some of the other functions, which had a lot of duplicate logic, etc.
I also refactored the tests a bit to reduce the number of times we actually install gcloud. This made the tests significantly faster (~5min -> ~40s).

@bharathkkb these failing tests are also failing at HEAD (I just kicked off that run). It looks like a credential or something might have expired?

@sethvargo yup they expired. I will update in a bit and update here

Yay green now :)

@bharathkkb can I get a 👍 on this now that tests are passing?


Add retry for IPyWidgets CDN download failures

For #238

[ ] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[ ] Title summarizes what is changing.
[ ] Has a news entry file (remember to thank yourself!).
[ ] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[ ] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).

Codecov Report

Merging #4179 (184ba03) into main (dcaac00) will decrease coverage by 6%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##            main   #4179     +/-   ##
=======================================
- Coverage     71%     65%     -7%
=======================================
  Files        384     384
  Lines      25153   25153
  Branches    3598    3598
=======================================
- Hits       17906   16361   -1545
- Misses      5861    7555   +1694
+ Partials    1386    1237    -149

Impacted Files
Coverage Δ

src/client/datascience/liveshare/serviceProxy.ts
5% <0%> (-83%)
:arrow_down:

src/client/datascience/liveshare/liveshareProxy.ts
6% <0%> (-72%)
:arrow_down:

...ent/datascience/jupyter/liveshare/responseQueue.ts
24% <0%> (-72%)
:arrow_down:

...rc/client/datascience/jupyter/debuggerVariables.ts
15% <0%> (-69%)
:arrow_down:

...atascience/jupyter/liveshare/guestJupyterServer.ts
11% <0%> (-63%)
:arrow_down:

src/client/datascience/jupyterDebugService.ts
17% <0%> (-61%)
:arrow_down:

src/client/datascience/jupyter/kernelVariables.ts
14% <0%> (-59%)
:arrow_down:

...science/jupyter/liveshare/guestJupyterExecution.ts
19% <0%> (-59%)
:arrow_down:

...cience/data-viewing/jupyterVariableDataProvider.ts
14% <0%> (-58%)
:arrow_down:

src/client/datascience/jupyter/jupyterDebugger.ts
17% <0%> (-54%)
:arrow_down:

... and 143 more


useconsolelifetime does not invoke StopApplication when running in Console

I am trying to understand the pattern for a graceful shutdown of an azure function and its host. Following the documentation for UseConsoleLifeTime, I tried the below (simplest of many variations and attempts).
The documentation for UseConsoleLifeTime explicitly states that it "Listens for Ctrl+C or SIGTERM and calls [StopApplication() to start the shutdown process. This will unblock extensions like RunAsync and WaitForShutdownAsync."
However, in the below, StopApplication never gets called (in the VS 2022 Debugger) when I hit Ctrl+C in the console.
`using Microsoft.Extensions.Hosting;
public partial class Program:IHostApplicationLifetime
{
CancellationToken IHostApplicationLifetime.ApplicationStarted => throw new NotImplementedException();
CancellationToken IHostApplicationLifetime.ApplicationStopping => throw new NotImplementedException();
CancellationToken IHostApplicationLifetime.ApplicationStopped => throw new NotImplementedException();

public static void Main()
{
    var host = new HostBuilder()
        .ConfigureFunctionsWorkerDefaults()
        .UseConsoleLifetime()
        .Build();

    host.Run();
}

void IHostApplicationLifetime.StopApplication()
{
    var test = "Got to here";
    throw new NotImplementedException();
}

}`

@hseidel can you please share the documentation you're referencing? The pattern you're using is not actually registering the IHostApplicationLifetime implementation doesn't seem to be correctly used.

@Fabio @.***>
Thanks for this response. The documentation I was referring to is located here: https://docs.microsoft.com/en-us/dotnet/api/microsoft.extensions.hosting.hostinghostbuilderextensions.useconsolelifetime?view=dotnet-plat-ext-6.0 .
I think your response is correct, as I was able to register the functions correctly using asp.net host running under an Azure app service.  My goal is to implement the desired functionality using an Azure Function, but I think I need to better understand how underlying asp.net hosting capabilities are surfaced in azure functions.  I am fine letting this issue close.
From: Fabio Cavalcante @.>
Sent: Tuesday, March 22, 2022 5:44 PM
To: Azure/azure-functions-dotnet-worker @.>
Cc: hseidel @.>; Mention @.>
Subject: Re: [Azure/azure-functions-dotnet-worker] useconsolelifetime does not invoke StopApplication when running in Console (Issue #834)
@hseidelhttps://github.com/hseidel can you please share the documentation you're referencing? The pattern you're using is not actually registering the IHostApplicationLifetime implementation doesn't seem to be correctly used.
—
Reply to this email directly, view it on GitHubhttps://github.com/Azure/azure-functions-dotnet-worker/issues/834#issuecomment-1075673798, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABWE3DKOKGP35V7WSVWEIE3VBI5KPANCNFSM5RE6SQEQ.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

Thank you for the update, @hseidel !