Support suda to persist undo (#25)

Write the undo file manually.
Only test under *unix.

I need to investigate it carefully so please be patient 👍

I'm sorry for the late this. Well, what is the ownership and permission of the undofile? I think it must be exactly equal to the file which suda read otherwise others may able to read without permission right?

:h undodir

'undodir' 'udir' E5003
'undodir' 'udir'	string	(default "$XDG_DATA_HOME/nvim/undo")
global
List of directory names for undo files, separated with commas.
See |'backupdir'| for details of the format.
"." means using the directory of the file.  The undo file name for
"file.txt" is ".file.txt.un~".
For other directories the file name is the full path of the edited
file, with path separators replaced with "%".
When writing: The first directory that exists is used.  "." always
works, no directories after "." will be used for writing.  If none of
the directories exist Neovim will attempt to create last directory in
the list.
When reading all entries are tried to find an undo file.  The first
undo file that exists is used.  When it cannot be read an error is
given, no further entry is used.
See |undo-persistence|.
This option cannot be set from a |modeline| or in the |sandbox|, for
security reasons.

if user set undodir to path, running suda without permission for target file, neovim will complain
Error detected while processing function suda#BufWriteCmd[3]..suda#write[46]..<SNR>59_write_undo:
line   18:
E828: Cannot open undo file for writing: xxxx.un~

otherwise, the undofile are always stored in $XDG_DATA_HOME/nvim/undo directory, which is the user home subpath.
No matter what the undodir value is,  the permission of undofile is the same as the parent process of neovim.
We can simply tray catch the error or ignore the complaint if we want.

I'm not professional on this site thus this is a question but what happens if

Alan is sudoers thus he can read /secret
Bob is NOT sudoers but he belongs alan group
Alan read /secret by suda and /home/.config/nvim/undo/secret has written

In above case, can Bob read /home/.config/nvim/undo/secret? Or is my presupportion wrong?

I am not sure what you care about.
$XDG_DATA_HOME/nvim/undo is extended to /home/kevin/.local/share/nvim/undo in my system, kevin is my user name.
This path is absolutely owned by the user.
If the user sets the undodir to the path without permission, not only we can't manually write undofile, but also neovim itself will not be able to write the undofile.

$XDG_DATA_HOME/nvim/undo is extended to /home/kevin/.local/share/nvim/undo in my system, kevin is my user name.
This path is absolutely owned by the user.

Bob belongs to the alan group implied that Bob can read alan's files in my explanation. Thus the file existing in user's home directory is not enough for safe reason in my point of view.

If the user sets the undodir to the path which without permission, not only we can't manually write undofile, but also neovim itself will not be able to write the undofile.

Opposite. I worry about if undodir is permission-less like 777. In that case, I think everyone can read files written by suda.vim which, for example, /etc/shadow.

For example, the above image is a result of ls -al in my undodir on macOS. It said that permissions of undo-files are 644. It means even non-staff users can read those files if they can reach the undodir, right?
This is why I worry about this PR. So please explain why this is safe.

If you concern the undofile leak the password, we can change the mode of undofile written by suda to 600, group and others can't read the file, but the user can go on using this undofile in neovim.

I think about this and I believe that this PR can be a security hole.
Undo files produced by suda MUST have exactly same owner, group, and permissions as the original file. Otherwise, cracker can read secret contents through undo files produced by suda WITHOUT being super user

If you concern the undofile leak the password, we can change the mode of undofile written by suda to 600, group and others can't read the file, but the user can go on using this undofile in neovim.

That's not enough. I'm talking about cases even the user cannot read the content without being superuser. Thus the permission level must be exactly same with the original file (and I don't think it's handy enough)

The same permissions will inevitably cause undo to fail to be read from neovim normally.
With using undofile, sensitive information must be left on undo.
Maybe we should leave the choice to the user, give an option disabled by default, and point out the risk of using that option.

The same permissions will inevitably cause failure about reading undofile from neovim normally.
With using undofile, sensitive information must be left on undo.
Maybe we should leave the choice to the user, give an option disabled by default, and point out the risk of using that option.

I'm really sorry but it's not acceptable even the feature has disabled by default. There is too sensitive data in the world and I don't want my plugin to be a security hole.
Thanks for your patient contributions and I'm sorry again for the rejection.
Feel free to keep using your fork by your own risk.

I think it's depend on the specific system configuration. It's prefect to have 600 on user's home directory so that only the user (who is sudoer) can access the undo files, whatever the permission on these sudo files themself. There may be a concern that the user now no longer need the sudo password to read the undo file. But again, it depend on the configuration, sudoer may not need password to get root permission from the beginning. Moreover, nowadays many computers are single-user use (like my laptop).
I'm appreciate to see the author is so careful about security. But I also think this feature is valid and not-so-danger on some cases.

I see... Maybe I am overthinking things, but I am not sure, so I am forced to be cautious. I'm starting to think that we could offer the functionality on an opt-in basis as suggested and write a warning in the help, but I'm not sure and it's a bit difficult to write a warning.


POST events, graceful failure.

Calls do not resolve nicely on the client side in the event of a post error.

This is fixed in develop.


Erratic behavior: Popup window do not close

This is an erratic error. I dont know if it is a browser cache, satellizer issue or Provider's issue. I thought it would be a Google issue because of this: https://github.com/google/google-api-javascript-client/issues/74
But I faced this issue with LinkedIn too.

Click in the button "sign in with Google or Linkedin"
The user authorize the application
Then the popup does not close and the website refresh itself in the popup.

Any ideas?
Thanks in advance

Have you tried it on https://satellizer.herokuapp.com? I will need more information to know why the popup did not close.
What you saw inside the popup is a normal behavior. It redirects to itself, check for code param, and closes the popup so fast that you usually wouldn't see it load in the popup window.

I have tried the app. It is hard to me to get more information. If I get anything I will post here.
Maybe it is something related with AngularJS and $locationProvider.html5Mode(true).

Satellizer: bower: "satellizer": "0.13.*",
Chrome Version 46.0.2490.80 (64-bit)
OS: OSX 10.9.5
AngularJS 1.4.x integration with $locationProvider.html5Mode(true)

Hi @paulocheque. Did you find why is this happening?

Not really, but I have two hyphothesis:

it was related to AngularJS. Probably some async issue, since I was using some Angular variables in Satellizer.

Some browser cache issue. So I added a ?random=new Date().getTime() into my GET parameters to avoid that.

I made many changes and it appears to have fixed.
Thanks!

@paulocheque @nass600 Can you try something for me?
Edit satellizer.js code starting on line 753. Add console.log here and comment out the line which closes the popup.
if (popupWindowOrigin === documentOrigin && (Popup.popupWindow.location.search || Popup.popupWindow.location.hash)) {
  var queryParams = Popup.popupWindow.location.search.substring(1).replace(/\/$/, '');
  var hashParams = Popup.popupWindow.location.hash.substring(1).replace(/[\/$]/, '');
  var hash = utils.parseQueryString(hashParams);
  var qs = utils.parseQueryString(queryParams);

  // Print current URL location to console
  console.log(Popup.popupWindow.location);

  angular.extend(qs, hash);

  if (qs.error) {
    deferred.reject(qs);
  } else {
    deferred.resolve(qs);
  }

  $interval.cancel(polling);

  // Do not close the popup
  //Popup.popupWindow.close();
}

Then click on this popup and open Chrome Developer Tools to check console output, as well as potential errors that could cause this issue.

One more thing, @paulocheque are you doing any page redirects on root path? For example, when you go to http://yoursite.com, does it try to automatically take you to http://yoursite.com/login?

My problem is I am not getting anything in (Popup.popupWindow.location.search || Popup.popupWindow.location.hash) so that way the flow does not reach the if clause and therefore the popup is not being closed.
Maybe it has something to do with cache as @paulocheque suggests so I am going to try the random query param trick to see if this solves the issue.
Thanks guys

Well, after some hours of research I have narrowed the error to a cache issue in the angular application web server (Nginx). The random query param didn't work for me all the time.
I have page redirects on root path and they don't seem to affect the behaviour.
It seems that the redirect_uri was being cached somehow when my API redirects the successful response back to my angular app.
The lines I added to the virtual hosts were:
  location / {
    if_modified_since off;
    add_header Last-Modified "";
  }

Not sure if this is correct patch (because now the app loads slower) but it works for me right now.

Hey, some of you might be having the same problem as mine of the popupWindowOrigin not being equal as the redirectUri. It makes some authentication methods work and other not to work. In my case Facebook worked but Google didn't. I fixed it by adding the following code at line 752. It removes and trailing slashes:
//Treat popup window origin and redirectUri             		popupWindowOrigin = popupWindowOrigin.trim();             		if(popupWindowOrigin.endsWith('/')){             			popupWindowOrigin = popupWindowOrigin.substr(0, popupWindowOrigin.length-1);             		}             		redirectUri = redirectUri.trim();             		if(redirectUri.endsWith('/')){             			redirectUri = redirectUri.substr(0, redirectUri.length-1);             		}

@nass600 check out this issue https://github.com/sahat/satellizer/issues/727 - I had problem very similar to yours, and although I was also suspecting nginx at the beginning, I figured out later that the problem was in query params being hijacked by uiRouter's redirect. I still think that caching is somehow involved with all this but more as a mean that manifested the real problem, not the cause.

I'll check the issue. Thanks @Martinsos

This bug still persists on iOS Firefox with 0.14.* @sahat

I had a similar issue where the popup was closing for some users and not for others. In the end, it turns out it happened because some users had been visiting the www.example.com and others example.com. This causes the popupWindowPath === redirectUriPath check in Satellizer's pollPopup method to remain unsatisfied due to the difference in domains.
The solution was to force all users to the non-www url.

@nass600 @aligajani
check out #885 and #829 - no resolutions yet but I think they're related to your issues

@chungconscious  True, it works for me most of the time though.
Nonetheless, ping me when there's a solution please :)

+1

having this problem with authenticating w/Linkedin on our service
lmk if there's anything I can do to help with testing fixes etc

Hi @sahat  - I'm having the same problem and it all started when I implemented a page redirect on the root to a subfolder: / redirects to /app/ in express
How might I get around this issue? I did modify my authProvider as such
$authProvider.linkedin({clientId: 'XXXXXXX', redirectUri: window.location.origin + '/app/'});
Seems to happen a more times than not

Adding a note that it's still not solved for us. Problem happens on Chrome at about 70% of the login attempts (with FB or Linkedin) and a 100% of the attempts on Safari. Help!!

@knigal This sounds a lot like the problem that I had, I described it, including the solution, here: https://github.com/sahat/satellizer/issues/727 - I recommend you give it a look.
Can you check at which moment the error happens? Does request get sent to your server? It probably crashes before that, after it receives response from LinkedIn and tries to redirect the popup window, but redirect probably fails at that moment for some reason.
Is maybe promise rejected, have you checked that out? If yes, what is the error message?

Thanks @Martinsos your tip is right on target in my case.
The problem was solved similarly a couple of days ago. That's was good since login stopped working completely in Safari, and panic was seeping in :)
In case this would be of any use to others, I'll add a short summary:
I tracked down the problem to Satellizer's Popup.prototype.polling function. After authentication with the provider, it is testing:
if (popupWindowPath === redirectUriPath) {

This lines confirms that the url of the popup is same as value of the provider's redirectUri.
The issue was indeed a race condition between this polling function and a ui-router redirect which was set up to fwd the root of the app to a substate with a different url:
    $urlRouterProvider.when('/', () => {
        $state.transitionTo('some.other.state');
    });

I've added some logging which exposed that the value of popupWindowPath never equaled redirectUri. Essentially the test was failing because the following test was never true:
if ("https://app.mapme.com" === "https://app.mapme.com/other/path") {

So the authentication was completed successfully, but the root fwd rule was messing things up.
To fix it, I ended up doing as following:

configure redirectRui:

    .config(function ($authProvider) {
        $authProvider.baseUrl = ... ;
        const redirectUri = window.location.origin + '/auth-endpoint';
        $authProvider.facebook({
            clientId: appId,
            redirectUri,
        });

as well as adding a ui-router state, such as:
    $stateProvider.state('authEndpoint', {
            url: '/auth-endpoint',
        })

Also note that I added to the providers (fb/linkedin) acceptable redirectUri page the new redirectUri path: https://app.mapme.com/auth-endpoint so that they don't redirect to my application's root.
Once all that was set up, Satellizer's polling function always unblocked and the authentication popup gets closed, meaning no more issues!
So, the issue is 100% solved for me. Thanks everyone!


tower of hanoi

Hello I want to add C++ code for Tower of Hanoi, I'm a beginner in Hacktober fest so let me contribute.

Hello I want to contribute in this repository. I will give the optimized solution. I would be grateful if you assign this problem to me

Hello
I’m sorry cause I’ve already started working on this
On Sun, 16 Oct 2022 at 1:39 PM, Charu Chaturvedi @.***>
wrote:

Hello I want to contribute in this repository. I will give the optimized
solution. I would be grateful if you assign this problem to me
—
Reply to this email directly, view it on GitHub
https://github.com/1dharode/Hacktoberfest2022/issues/1492#issuecomment-1279917754,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASM5CAION6NENSREEWWUSZLWDOZ27ANCNFSM6AAAAAARGG7MC4
.
You are receiving this because you authored the thread.Message ID:
@.***>

i want to contribute in this repository

Sir please assign me this issue.

hello sir,
@1dharode Sir would you kindly assign me this issue. I have learning c++ since last 1.5 years and I am a beginner at hacktoberfest so I would like to contribute

I have already made a pull req. regarding this question - I have completed this problem in python! Please accept my PR

assign me plz.

Hello, I want to contribute to this repo please assign this to me, I've written an optimized code for tower of Hanoi

Hello ! I am learning cpp from last 6months and i think i could solve this problem.Kindly assign me this issue.

Hi there 👋
I'm excited to contribute to Hacktoberfest 2023, and I'd like to work on this issue. It looks like a great opportunity to learn and contribute to the project.
Please assign this issue to me, and I'll start working on it right away. If you have any specific instructions or guidelines, please let me know, and I'll make sure to follow them.
Thanks a lot for considering my request!
Best regards,
Prateek Sethi

please assign me this issue for hacktoberfest2023 @DHEERAJHARODE

I wish to contribute. Kindly assign this to me.


nginx: allow overriding SSL trusted certificates when using ACME

Motivation for this change
Some ACME providers (like Buypass) are using a different certificate
to sign OCSP responses than for server certificates. Therefore,
sslTrustedCertificate should be provided by the user and we need to
allow that.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@ofborg test acme

Tests are passing, but this still needs a release notes entry.

@vincentbernat can you write a changelog entry? Then this could go in.

Sorry, I have missed the previous hint. I have updated with a changelog entry.


fix: shellcheck fixes for roxctl.sh

Signed-off-by: Pablo Caderno<EMAIL_ADDRESS>Description
Fix shellcheck warnings for roxctl.sh as per #3544

/ok-to-test

Closing this stale PR.
@kadern0 please reopen once you rebase and resolve conflicts.


AMQJS0010E localStorage is not supported by this browser - react native Expo

I instaled AsyncStorage with yarn add @react-native-async-storage/async-storage and and passed the object to init. But when compiles I get the error: AMQJS0010E localStorage is not supported by this browser.
Line 841 , file mqttws31.js

I saw this too.


How to force Clappr player to stay fullscreen after change video?

How to force Clappr player to stay fullscreen after change video?
I write a function that triggered on PLAYER_ENDED event, function loads next video with this load method:
enter code here`enter code here`player.load([{source: 'video_url_exmpl'}], null, true);

But when event was triggered and the new video loaded player cancels Fullscreen mode. I set up option:
exitFullscreenOnEnd: false,

I write a plugin that supposes to toggle fullscreen but browser throw a warning message:
Failed to execute 'requestFullscreen' on 'Element': API can only be initiated by a user gesture

Here is my Clappr player initialization and settings:
 player = new Clappr.Player({
        source: sourceUrl,
        parentId: "#player",
        autoPlay: true,
        width: '100%',
        height: '100%',
        exitFullscreenOnEnd: false,
        playInline: true,
        recycleVideo: Clappr.Browser.isMobile,
        poster: '',
    }).on(Clappr.Events.PLAYER_ENDED, function() {
           player.load([{source: 'video_url'}], null, true);
          }
    });

@RussellSk , please note that playInline and recycleVideo are playback options. (and therefore must be passed in 'playback' property).
The Player exitFullscreenOnEnd option is handled by an internal plugin called end_video which listen to CONTAINER_ENDED and CONTAINER_STOP events to exit (or not) fullscreen at end of the video. (default is true).
With the current Clappr internal mechanisms it is not possible to stay in fullscreen if you change video source with load() Player method. The technical reason is when the source change, player "core" will destroy the current container and build a new one for the new source. (though i did not test on mobile with recycleVideo option enabled and exitFullscreenOnEnd set to false).
Another technical reason may be because the video source url may be removed from video element at the end of video or if video is stopped. (depending on the source format and playback capabilities).
So, in an HTML5 video scenario, to change video source without exit fullscreen, you may needs to work directly with the playback video element. (which is "hacky", but should be possible).

@kslimani Thank you for your reply, but how can I load in HTML5 next video or change video "on the fly" without reloading core of the player and therefore stay in Fullscreen mode? Tell me where to start digging...
Or it is impossible?

@RussellSk go to Clappr demo page and try the following code :
var player = new Clappr.Player({
  parentId: "#player-wrapper",
  source: 'http://clappr.io/highline.mp4',
  height: 360,
  width: 640,
  exitFullscreenOnEnd: false,
  playback: {
    playInline: true,
    recycleVideo: true,
  },
  events: {
    onEnded: function () {
      var setSource = function (newSrc) {
        this.core.options.source = newSrc;
        var container = this.core.getCurrentContainer();
        var playback = this.core.getCurrentPlayback();
        if (container) {
          container.options.source = newSrc;
        }
        if (playback) {
          playback.options.source = newSrc;
          playback._setupSrc && playback._setupSrc(newSrc);
        }
      }.bind(this);

      // Set another .mp4 source and start again player
      setSource('https://static.playmedia-cdn.net/resources/sample/h264_sintel_trailer-1080p.mp4');
      this.play();
    }
  }
});

Please note this code is an ugly trick which may let some player components (like plugins) in an inconsistent state. This work only if source are in same format (ie: all source are .mp4 files).
But should give you some hints to where to start to dig.

Another approach could be also to create one or several Playback plugin which can take a list of video as option and play video one after one. (without using core.load() and working directly with html5 video element).

We're working on a change in the internal player structure to prevent destroying the playback in most cases.
Actually, we already have done this partially in our chromecast implementation, and we’ll probably port this back into Clappr this week. We just need to ensure the new behavior won’t break anything.

@kslimani Thank you for your help, your example solve the issue. :)
@towerz How can I contribute to the project, what can I do? If you have some exercises.
Thanks.

@kslimani I'm sorry for asking you again but when I'm using .m3u8 format, the player plays everytime the same source URL... Without changing, Is there some additional function in Clappr that extract valid url for core?

@RussellSk
Updated code for hls playback :
var player = new Clappr.Player({
  parentId: "#player-wrapper",
  source: 'http://www.streambox.fr/playlists/x36xhzz/x36xhzz.m3u8',
  height: 360,
  width: 640,
  exitFullscreenOnEnd: false,
  playback: {
    playInline: true,
    recycleVideo: true,
  },
  events: {
    onEnded: function () {
      var setSource = function (newSrc) {
        this.core.options.source = newSrc;
        var container = this.core.getCurrentContainer();
        var playback = this.core.getCurrentPlayback();
        if (container) {
          container.options.source = newSrc;
        }
        if (playback) {
          playback.options.source = newSrc;
          playback.options.src && (playback.options.src = newSrc);
          playback._setupHls && playback._setupHls();
          playback._setupSrc && playback._setupSrc(newSrc);
        }
      }.bind(this);

      // Set another .mp4 source and start again player
      setSource('http://www.streambox.fr/playlists/x31jrg1/x31jrg1.m3u8');
      this.play();
    }
  }
});


Expanding on custom directory icons

First I just want to say that I am brand new to git and contributing to open source projects. This might be a simple issue.
I want to modify the prompt_dir() function in powerklevel9k.zsh-theme to include more conditions to replace the icon depending of the state_path.
This issue is related to #606.
This can be easily done if you just add dir states and separate conditions. For instance, here I've added a "DROPBOX""state that uses "DROPBOX_ICON" for paths that contain  '~/dropbox'*, almost exactly the same as the current/etc conditions.
  # save state of path for highlighting and bold options
  local path_opt=$current_path

  typeset -AH dir_states
  dir_states=(
    "DEFAULT"         "FOLDER_ICON"
    "HOME"            "HOME_ICON"
    "HOME_SUBFOLDER"  "HOME_SUB_ICON"
    "NOT_WRITABLE"    "LOCK_ICON"
    "ETC"             "ETC_ICON"
    "DROPBOX"         "DROPBOX_ICON"
  )
  local state_path="$(print -P '%~')"
  local current_state="DEFAULT"
  if [[ $state_path == '/etc'* ]]; then
    current_state='ETC'
  elif [[ "${POWERLEVEL9K_DIR_SHOW_WRITABLE}" == true && ! -w "$PWD" ]]; then
    current_state="NOT_WRITABLE"
  elif [[ $state_path == '~/dropbox'* ]]; then
    current_state="DROPBOX"
  elif [[ $state_path == '~' ]]; then
    current_state="HOME"
  elif [[ $state_path == '~'* ]]; then
    current_state="HOME_SUBFOLDER"
  fi

My question is; how could we do this within .zshrc instead of modifying powerline9k.zsh-theme?
I've seen other users utilize the custom_user prompt status in .zshrc to easily add icons and the like, but to my knowledge there is not a custom_dir status to do the same thing, and even so would it allow conditional formatting to icons?

@romkatv
Looks nice in powerlevel10k! So this is not addressed in powerlevel9k?

Looks nice in powerlevel10k!

Thanks!

So this is not addressed in powerlevel9k?

No.


Samples and Flash

@maks @democloid
Pico-tracker, works correctly only with samples, mono 16 bit 22050Hz. all 8 tracks work well together with long and short samples at the same time.
16 stereo 44100Hz - do not work correctly, sometimes they do not turn on.
Samples - https://discord.com/channels/1166483348634480650/1167241783026139186/1176193011521163344
I checked, for some reason you can't work with samples in more than 3 tracks. They just don't work.
Track 1 - vocals
Track 2 - drums
Track 3 - pad
and then the samples don't work.
I tried to make them different lengths, sometimes 2 tracks work, sometimes three.
Now I tried to upload many parts
All samples 8 secs long
4 bass files
4 pad files
1 drum file
4 vocal files
bass/pad/vocals/drums - each file has a size of 353kb
Short samples work fine. in 8 tracks / 16 instruments
But if you add samples of 8 seconds or more, problems occur.
There is enough memory though, the samples come out to 5 megabytes.
Project - https://discord.com/channels/1166483348634480650/1167241783026139186/1176193494587547648
My Pico 16mb Black. All samples mono 16 bit 22050Hz.

Did some testing on what works and what fails. I confirmed that the 27s mono 16bit 22050Hz file seems to work fine (although with some distortion?) but the stereo 16bit 44100Hz doesn't, never finishes copying to flash.
I used the failing file to create shorter samples, of length: 15s, 10s, 5s and 2s (file sizes: 2.52MiB, 1.68MiB, 861KiB and 345KiB)
15s and 10s also fail to load, 5s and 2s load fine and work as expected, so it's not a problem with stereo or 44100Hz files.
Will need to investigate further.

@democloid  The problem with memory is if you use long samples.
Also try uploading a large number of 16/44100 stereo samples.
Also try using my project to check.


Merge everything to Master

done with adding all my things to develop so its ready for the merge from my side


update dep to v0.5.0

[ ] The purpose of this PR is explained in this or a referenced issue.
[ ] The PR does not update generated files.

These files are managed by the codegen framework at Azure/autorest.go.

[ ] The PR targets the latest branch.
[ ] Tests are included and/or updated for code changes.
[ ] Updates to CHANGELOG.md are included.
[ ] Apache v2 license headers are included in each file.

Can one of the admins verify this patch?

grep -L -r --include *.go --exclude-dir vendor -P "Copyright (\d{4}|(c)) Microsoft" ./ | tee /dev/stderr | test -z "$(< /dev/stdin)"

also  i presume there is no update on the lock files because those already came in from the Auto PR ?

Refers to: .travis.yml:29 in c468073. [](commit_id = c46807304bedcabbb2ded1b6a68dccc954404d6b, deletion_comment = False)


Handle scope and also removing resource when VERSION=v2.0

Handle scope when VERSION=v2.0

Thank you for the change! We'll need a corresponding test for this too.

Can look on that coming days, had some issues with poetry today.

Thank you so much, this should solve #198.
Looks good to me 🎉

.. some tests fail, but I don't think that it's related to this PR. I'll try to look into it this weekend. We should also drop Python3.6 support.

Fixed pipelines, bumped version and removed Python3.6 support.
@tim-schilling @sondrelg , any comments? 😊

Thank you so much! 😊
This will be released on PyPI any minute now.
Github release found here: https://github.com/snok/django-auth-adfs/releases/tag/1.9.6


upgrading Swashbuckle and fixing swagger-ui flaw

Can you describe the OAuth authentication stuff?

To back things up: App Service Authentication needs a couple of things done uniquely for its login flow. In order to get things to work, I took the swagger implementation and tweaked it. The unfortunate side effect of this is that whenever this file updates, we need to merge those updates with our logic. What you see here is mostly that -- the only place I've changed anything is where the code is wrapped in // tags. Those indicate deviations from the original file.
This newest version introduced some new behavior -- like support for multiple auth schemes (you can now use both google and facebook for example). But it coordinates that logic by using the state parameter, which App Service Auth does not pass back to us. So I had to make an extra tweak to get this working.
I'd like to come up with a better model than mucking with this file, but this is the quickest way to get things working with the latest build.

Looks good to me!

@pvbchak - Can you please merge this before you release the next server release?


HHH-12878 - Include statement when throwing StaleStateException in Expectations.BasicExpectation

Change to allow the statement to be included in the error message when a StaleStateException is thrown from Expectations.BasicExpectation. This would make the exception much more helpful when reading the log.
https://hibernate.atlassian.net/browse/HHH-12878

Adjusted the commit message and merged, thanks!


Update excluded JFR tests in JDK8 ProblemList with proper links to upstream JBS issues

I've created upstream PRs for fixing TestCompilerCompile and TestCompilerPhase. The PR for TestCompilerCompile has been merged, but will take a while to backport (because I need to do it for 22, 21, 17, 11, then finally 8). Since, it might take some time to backport and the fix only affects the test, the best temporary solution is probably to keep the test excluded. So for now I'd just like to update the ProblemList links to point to the correct JBS issues so people can understand why these tests must be disabled.

The ECA check is failing. Should I manually sign it, or am I already covered since I work at Red Hat?

I have been told there is a way to indicate in your Eclipse profile that you work for a member company (Red Hat) and would not need to explicitly sign the ECA.  I have not successfully seen how that is done, so I signed the ECA following these instructions (https://adoptium.net/docs/eca-sign-off/).

I have been told there is a way to indicate in your Eclipse profile that you work for a member company (Red Hat) and would not need to explicitly sign the ECA. I have not successfully seen how that is done, so I signed the ECA following these instructions (https://adoptium.net/docs/eca-sign-off/).

Thanks @smlambert! I'll follow those instructions.


K8S-dashboard judge whether the ssl certs is provided based on rbac enable or not.

Traceback (most recent call last):
  File "/pai/deployment/paiLibrary/paiService/service_template_generate.py", line 113, in generate_template
    generated_template = template_handler.generate_from_template_dict(template_data, service_conf_dict)
  File "/pai/deployment/paiLibrary/common/template_handler.py", line 28, in generate_from_template_dict
    map_table
  File "/usr/lib/python2.7/dist-packages/jinja2/environment.py", line 989, in render
    return self.environment.handle_exception(exc_info, True)
  File "/usr/lib/python2.7/dist-packages/jinja2/environment.py", line 754, in handle_exception
    reraise(exc_type, exc_value, tb)
  File "<template>", line 24, in top-level template code
  File "/usr/lib/python2.7/dist-packages/jinja2/environment.py", line 389, in getitem
    return obj[argument]
UndefinedError: 'dict object' has no attribute 'ssl'
2020-01-06 07:52:59,218 [ERROR] - deployment.paiLibrary.paiService.service_management_start : Some error occurs when starting service k8s-dashboard
Traceback (most recent call last):
  File "/pai/deployment/paiLibrary/paiService/service_management_start.py", line 85, in start
    service_template_generater.run()
  File "/pai/deployment/paiLibrary/paiService/service_template_generate.py", line 128, in run
    self.generate_template()
  File "/pai/deployment/paiLibrary/paiService/service_template_generate.py", line 116, in generate_template
    raise e
UndefinedError: 'dict object' has no attribute 'ssl'

#4109


Support missing attributes in QIDO-RS

Expected Behavior
Doing a QIDO-RS request should return the following attributes:
Study Level:

Attribute Name
Tag

Modalities in Study
(0008,0061)

Number of Study Related Series
(0020,1206)

Number of Study Related Instances
(0020,1208)

Series Level:

Attribute Name
Tag

Number of Series Related Instances
(0020,1209)

Instance Level:

Attribute Name
Tag

Rows
(0028,0010)

Columns
(0028,0011)

Bits Allocated
(0028,0100)

Number of Frames
(0028,0008)

Actual Behavior
These attributes are not returned.

These attributes are now supported and will be returned by default according to the standard.
This required updating the database to support the new Instance Level attributes (Rows, Columns, Bits Allocated and Number Of Frames).
If you need to update an existent database, you can use the "DICOMcloud.SqlServerDatabase" project Schema Compare utility by right clicking on the project and hit compare, select your database as the target. For detailed instructions you can see this post:
https://msdn.microsoft.com/en-us/library/hh272690(v=vs.103).aspx


Define function that generates the differential equation

These function must take the generic agents and mechanisms and return a list that can be solved by the integrator.

This is now completed.
The function is saved as .py. It is executed in the code through a string. This simplifies issues importing libraries.
!!! Using Colab, exec() is relatively safe. But in the local package passing a string to execute is a big red flag.


fragment running and present in TableFragments but not in catalog

Job 3166 with its Materialize operator in fragment 3074 keeps reporting metrics. However, the job name is "unknown".
The actor is running, as confirmed by await-tree as well.
However, cannot find the job in the dev dashboard, including the pages for catalogs, dependency graph, and fragment graph.
Can still jump into its fragment graph through the Goto form. But the Relations section does not change accordingly.

Only after the throughput goes to zero (likely the backfill is finished), the jobs start to appear on the dashboard correctly.

It looks like a backfilling job. During backfill, the Catalog is not written yet, while the TableFragments have been written.
It would be better to persit a catalog but marking as invisible
cc. @yezizp2012

Should be resolved by #19134 .


[.Net] Add AutoGen.OpenAI package that uses OpenAI v2 SDK

Why are these changes needed?

Related issue number
#3193

also fix #3404
Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

I like it, more clean and lean. The AzureOpenAIClient is neat and better than the previous code where you had to do a configuration and "hack" it into each agent.

100% same here,, the LLMConfig is too pythonic and not quite a good fit for .NET coding style


Tw palette v2

[Description of your change]
This is in Process
I have added a Tailwind Custom Palette that should allow us more control in the future. Currently we are now able to have access to native events, which allows us to do things like open a link in a new tab. In progress is dark mode and scrolling with keys.
Relevant Basecamp task(s), GitHub Issue(s), or other links

Basecamp task:
-https://3.basecamp.com/4448010/buckets/16001607/todos/4658655283#__recording_4660176260

Review URL (If not sandbox)

Sandbox

Hey @seano424 — when you're ready for the full review do you mind doing an npm run build? That'll compile and minify the scripts we ship to the end user!


Parameterize the top_k_categorical_accuracy metric

Modify to have parameterized version of accuracy metric

Hey @somnathrakshit, thanks for the PR!
This change makes parametrisation of the top_k_accuracy metric possible, but doesn't enable us to do anything else. The way I thought about it, is to have the top_custom_categorical_accuracy function in the user code, outside of Magpie and enable a field called metrics=[] in the Magpie API to pass arbitrary functions to be computed. That way users can compute whatever they want. We should also enable some sensible defaults to be computed when nothing is passed in.
Hope that makes sense!

Yes, it does. I will try to do it over the weekend but not sure if I will get time or not.


Document persistence manager options and the use of com.gopivotal.manager.SessionFlushValve

The Tomcat persistence manager documentation is pretty lacking. Many people are asking for more detail in the comments section.
When using this manager for a shared session store (possibly what it was not originally designed for) many write ups list the time delay between the creating a session and flushing the sessions to the store being a issue with non-sticky sessions. I assume that this valve is some way of mitigating this:-
<Valve className="com.gopivotal.manager.SessionFlushValve" />
Here's the best description of the params that I found:-
<Context>

...

    <!-- The idle time in seconds before persisting session is
         maxIdleBackup + processExpiresFrequency * engine.backgroundProcessorDelay.
         The backgroundProcessorDelay is around 10 seconds. This configuration should make Tomcat
         to persist sessions after 20 seconds of idling and hold them in memory as long as each
         instance wants -->
    <Manager className="org.apache.catalina.session.PersistentManager"
      saveOnRestart="true"
      minIdleSwap="-1"
      maxIdleSwap="-1"
      maxIdleBackup="20"
      processExpiresFrequency="1">
      <Store className="org.apache.catalina.session.JDBCStore"
        connectionURL="jdbc:mysql://localhost/tomcat_data"
        driverName="com.mysql.jdbc.Driver"
        connectionName="tomcat"
        connectionPassword="tomcat"
        sessionIdCol="session_id"
        sessionValidCol="valid_session"
        sessionMaxInactiveCol="max_inactive"
        sessionLastAccessedCol="last_access"
        sessionTable="tomcat_sessions"
        sessionAppCol="app_name"
        sessionDataCol="session_data"
      />
    </Manager>
...

</Context>

Problems in this solution

If Tomcat dies unexpectedly, the new sessions served by dead instance are lost. Most likely with the configuration above, sessions newer than 30 seconds will disappear. Also new changes that have been done to session data will be lost.

http://pedanttinen.blogspot.co.uk/2012/01/tomcat-clustering-and-session.html

The valve com.gopivotal.manager.SessionFlushValve flushes (ie, saves), the HTTP session to the backend store upon a successful HTTP request (ie, one in which no exceptions thrown in pipeline).  The effect being the session is been saved prior to returning the response.
If you were using the backend as your clustering strategy as opposed to Tomcat node clustering, you can use this valve to ensure session state always persisted between requests.
If, however, you are using Tomcat clustering, this valve is probably not needed since Tomcat has replicated the state to another node.
Does this help?

My understanding is this:-
Whichever /persistence/ backend you use if it's mysql postgres or redis wrapped in:
<Manager className="org.apache.catalina.session.PersistentManager">
...
</Manager>

It should be governed by the defaults of the rules listed in that example above so sessions are not instantly stored.
So when you add your special:

It overrides this behaviour?
Does the session flush block Tomcat in anyway?

Yes and yes.
Every HTTP request is accompanied with a call the store's save(Session) method.  See https://github.com/pivotalsoftware/session-managers/blob/master/common/src/main/java/com/gopivotal/manager/SessionFlushValve.java#L150-L157.
This particular valve forwards the request to the valve, and the saves the session.  Because it's part of the pipeline, it's work must complete before a response is returned to the client.
Thus, only enable the valve is Tomcat clustering not enabled.  At a high level, the session is either replicated to the store (in the case of using the valve) or to another node (if using clustering).

Hmm now I'm more confused. Do you use this persistence as well as the built in clustering then? What is the preferred configuration? I'm using it specifically so I can avoid the delta based clustering..
Without clustering enabled if you login to a Tomcat node, create a session. Then shutdown redis. Then visit the other node that request should miss that session.

There are 3 components/features that can be used/enabled:

clustering
persistence
flush valve

The first 2 are orthogonal:

Enabling clustering replicates sessions across nodes
Using a persistence manager reconstitutes state across reboots

Enabling the 3rd, the flush valve, allows you to effect clustering using the persistence backend; it implies you are not using Tomcat clustering.
So ...
If you want a standalone instance that saves sessions across reboots, enable persistence.
If you want a cluster that saves sessions across reboots, enable clustering and persistence.
If you want cluster-like behavior without using Tomcat clustering, enable persistence and the valve.

Okay I've tested and that is the case, thanks for the clear explanation. Option 3 in your list is what I'm going for as I should have a Redis that's always available, either managed on ec2 or using Elasticache.

Can you add your comment to the markdown docs?


chore: vec to hashset

Hi Luciano,
most of the operations you do with claims and algorithms are just contains.
I think that moving to an HashSet will make these operations faster.
Since you are building a benchmark for this project, it could be worth a try 😄

for small sets BTreeSets might be faster but until we have some microbenchmarks I think this is still an awesome improvements! :) thanks a lot


Think about allow add custom preview handlers

Excuse my poor english in advance.
After .svg and .md files preview (tremendous job) released I have thought about allow somekind of possibility to add custom preview handlers, following a "plugin/extension scheme".
-Could be with a "plugin folder". Every time powertoys starts analyze it and load valid found plugins.
-Could be with a menu/open file dialog inside powertoy module tab where you add explicitly the plugins.
-Could be downloading it from somekind of repository/plugin "store" where everybody can share the creations...
I understand what I'm asking is difficult and-or enormous; but it will be fine only think about it or consider it.
And some of the job is done, using in the custom plugins the base classes of c# IPreviewHandler wrapper you have done it's so straightforward.
And I thought in the cons of this idea too:
-The plugins could cause security problems
-Plugins must be specifically written, can't be made "out of the air".
-It requires more code in the Powertoy module in order to analyze, load and manage the plugins
-An many many cons I can't think surely...

We don't have plans to introduce a plugin system. But maybe we implement #1527 which would support many file types


0.10.11 QUIC version support

Caddy 0.10.11 now support quic  v39 now,but can not support the lower version like v38 37 etc.
why?

Because the lib Caddy uses, https://github.com/lucas-clemente/quic-go, removes support from versions that Chrome no longer supports.


Adding new file types

Is there a process for users to add file types that aren't recognised.  For instance, I work with  wavpack and dsd files and it would be great to have them recognised as audio file types.

If those are common enough extensions, feel free to just add those into lsd. You can do that by adding them in https://github.com/Peltoche/lsd/blob/946f2430661e57e0eda641a3f70601d4b46a3b71/src/icon.rs#L225

We also have #707 in the pipeline for enabling custom icons.

Sorry for asking in a closed channel, but is "justfile" legit to be a common extension?

Sure, feel free to add an icon if that is what you are going for.


MarshalBinary() error return value redundant

err is always nil, is there any reason behind it? I know it probably won't be fixed to avoid breaking changes.
// MarshalBinary implements encoding.BinaryMarshaler.
func (uuid UUID) MarshalBinary() ([]byte, error) {
	return uuid[:], nil
}

https://github.com/google/uuid/blob/9b3b1e0f5f99ae461456d768e7d301a7acdaa2d8/marshal.go#L27

This is satisfying the encoding.BinaryMarshaler interface so it must return the nil.


Allow jsx and fix react config/test

Please let me know if more test cases can be added to those test-fixtures files.

Released, thanks @haolinj


Where is the code?

I would like to knit with JS. Can I help you expedite the process of getting the code in the repo somehow? I saw your talk and really enjoyed it; now I would like to play around with knitting myself! 👍

Hi

Eight months...

Almost 18 months later... this is taking more time than Winds of Winter.

💔

"I will be putting all of my code and researches in this repo VERY SOON. I promise !"


Work breakdown of Quick Starts Initiative

The team will review the quick starts initiative product brief and begin to breakdown the individual feature work so we can organize and estimate and plan the project effectively.
Acceptance Criteria

[ ] Read the product brief (requirements) ☕ + 📖
[ ] Work with @jpvajda to break the work down into epics
[ ] Sequence epics
[ ] size epics
[ ] start to construct the basis of a master project plan.

@zstix for input on the AC 👋


firestoreImport() throws  Cannot read property 'batch' of undefined

Expected behavior
firestoreImport() API should import a collection from an exported JSON file.
Actual behavior
firestoreImport() call failed with an exception
Steps to reproduce the behavior

exported an "events" collection from an instance of Firestore.

Running a unit test with ...
const col = db.collection("events");
const events = require("../../fixtures/events.json");
await firestoreImport(col, events);  // << throws TypeError: Cannot read property 'batch' of undefined

In the debugger I found that ...
This line in imports.ts  executes const batch = startingRef.firestore.batch();  but startingRef.firestore is undefined.

My bad!  Arguments were swapped.


Reduce compile times of monitoring and sandbox

On my machine, compiling the project takes a lot of time: 5min 34s
cargo build -Ztimings reveals that this is caused by the 2 packages monitoring and sandbox:

I've reduced the compilation time by using dynamic dispatches (warp::filters::BoxedFilter) instead of static ones (warp::filters::Filter) in those 2 packages.
Those are the times I get by running cargo build --release after a cargo clean in the root of the packages:

warp::filters::Filter
warp::filters::BoxedFilter

monitoring
1min 38s
1min 04s

sandbox
4min 42s
41s

Compiling the whole project has been reduced too: 1min 33s (vs 5min 34s)
Dynamic dispatches have a runtime cost, but it seems that those 2 packages don't need to be highly efficient, is it ?
$ rustc --version
rustc 1.58.0-nightly (e249ce6b2 2021-10-30)

Related links:

https://rust-classes.com/chapter_6_3.html#chapter-63---warp-and-slow-compile-times
https://github.com/seanmonstar/warp/issues/507

@sebastiencs you are my hero.


Test does not start on iOS during patrol develop until Hot Restart is triggered

Steps to reproduce

Do the setup as mentioned in the patrol official website.
Run using patrol develop command with your project.

Actual results
Once after running the patrol develop -t integration_test/example_test.dart, the application is launching without TEST STARTING screen but it will appear once after hot restart clicked after it enabled and the testing will carry on.
Logs
MacBook-Pro Next-Gen-App % patrol develop -t integration_test/example_test.dart --verbose
In next major release, patrolTest method will be intended for UI tests.
If you want to use Patrol in your widget tests, use patrol_finders package.

For more information, see https://patrol.leancode.co/patrol-finders-release
Disable this warning by setting the PATROL_FINDERS environment variable.

Verbose mode enabled. More logs will be printed.
Received test target: /Users/lallu/Documents/GitHub/Next-Gen-App/integration_test/example_test.dart
Generated entrypoint /Users/lallu/Documents/GitHub/Next-Gen-App/integration_test/test_bundle.dart for development
Received Android flavor: jk
Received iOS flavor: jk
$ flutter --no-version-check devices --machine
No device specified, using the first one (iPhone 14 Plus)
Received device: iPhone 14 Plus
Received 8 --dart-define(s) (0 custom, 8 internal)
Received internal --dart-define: PATROL_WAIT=0
Received internal --dart-define: PATROL_APP_PACKAGE_NAME=com.paytronix.nexgenfl
Received internal --dart-define: PATROL_APP_BUNDLE_ID=com.paytronix.nexgenfl
Received internal --dart-define: PATROL_ANDROID_APP_NAME=NGA JK Cafe
Received internal --dart-define: PATROL_IOS_APP_NAME=NGA JK Cafe
Received internal --dart-define: INTEGRATION_TEST_SHOULD_REPORT_RESULTS_TO_NATIVE=false
Received internal --dart-define: PATROL_TEST_LABEL_ENABLED=true
Received internal --dart-define: PATROL_HOT_RESTART=true
• Building app with entrypoint test_bundle.dart for iOS simulator (debug)...
$ flutter build ios --no-version-check --config-only --no-codesign --debug --simulator --flavor jk --target /Users/lallu/Documents/GitHub/Next-Gen-App/integration_test/test_bundle.dart --dart-define PATROL_WAIT=0 --dart-define PATROL_APP_PACKAGE_NAME=com.paytronix.nexgenfl --dart-define PATROL_APP_BUNDLE_ID=com.paytronix.nexgenfl --dart-define PATROL_ANDROID_APP_NAME=NGA JK Cafe --dart-define PATROL_IOS_APP_NAME=NGA JK Cafe --dart-define INTEGRATION_TEST_SHOULD_REPORT_RESULTS_TO_NATIVE=false --dart-define PATROL_TEST_LABEL_ENABLED=true --dart-define PATROL_HOT_RESTART=true
        Building com.paytronix.nexgenfl for simulator (ios)...
$ xcodebuild build-for-testing -workspace Runner.xcworkspace -scheme jk -configuration Debug-jk -only-testing RunnerUITests -sdk iphonesimulator -destination generic/platform=iOS Simulator -quiet -derivedDataPath ../build/ios_integ OTHER_SWIFT_FLAGS=$(inherited) -D PATROL_ENABLED
        /Users/lallu/Documents/GitHub/Next-Gen-App/ios/Runner/iOSDatePicker.swift:56:41: warning: 'keyWindow' was deprecated in iOS 13.0: Should not be used for applications that support multiple scenes as it returns a key window across all connected scenes
                let win  = UIApplication.shared.keyWindow
                                                ^
        /Users/lallu/Documents/GitHub/Next-Gen-App/ios/Runner/iOSPicker.swift:56:41: warning: 'keyWindow' was deprecated in iOS 13.0: Should not be used for applications that support multiple scenes as it returns a key window across all connected scenes
                let win  = UIApplication.shared.keyWindow
                                                ^
        /Users/lallu/Documents/GitHub/Next-Gen-App/ios/Runner/iOSDatePicker.swift:56:41: warning: 'keyWindow' was deprecated in iOS 13.0: Should not be used for applications that support multiple scenes as it returns a key window across all connected scenes
                let win  = UIApplication.shared.keyWindow
                                                ^
        /Users/lallu/Documents/GitHub/Next-Gen-App/ios/Runner/iOSPicker.swift:56:41: warning: 'keyWindow' was deprecated in iOS 13.0: Should not be used for applications that support multiple scenes as it returns a key window across all connected scenes
                let win  = UIApplication.shared.keyWindow
                                                ^
        note: Building targets in dependency order
        /Users/lallu/Documents/GitHub/Next-Gen-App/ios/Pods/Pods.xcodeproj: warning: The iOS Simulator deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 9.0, but the range of supported deployment target versions is 11.0 to 16.2.99. (in target 'CocoaAsyncSocket' from project 'Pods')
        /Users/lallu/Documents/GitHub/Next-Gen-App/ios/Pods/Pods.xcodeproj: warning: The iOS Simulator deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 9.0, but the range of supported deployment target versions is 11.0 to 16.2.99. (in target 'PromisesObjC' from project 'Pods')
        /Users/lallu/Documents/GitHub/Next-Gen-App/ios/Pods/Pods.xcodeproj: warning: The iOS Simulator deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 9.0, but the range of supported deployment target versions is 11.0 to 16.2.99. (in target 'GoogleUtilities' from project 'Pods')
        /Users/lallu/Documents/GitHub/Next-Gen-App/ios/Pods/Pods.xcodeproj: warning: The iOS Simulator deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 10.0, but the range of supported deployment target versions is 11.0 to 16.2.99. (in target 'FirebaseCoreInternal' from project 'Pods')
        /Users/lallu/Documents/GitHub/Next-Gen-App/ios/Pods/Pods.xcodeproj: warning: The iOS Simulator deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 10.0, but the range of supported deployment target versions is 11.0 to 16.2.99. (in target 'FirebaseCore' from project 'Pods')
        /Users/lallu/Documents/GitHub/Next-Gen-App/ios/Pods/Pods.xcodeproj: warning: The iOS Simulator deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 10.0, but the range of supported deployment target versions is 11.0 to 16.2.99. (in target 'FirebaseInstallations' from project 'Pods')
        /Users/lallu/Documents/GitHub/Next-Gen-App/ios/Pods/Pods.xcodeproj: warning: The iOS Simulator deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 9.0, but the range of supported deployment target versions is 11.0 to 16.2.99. (in target 'nanopb' from project 'Pods')
        /Users/lallu/Documents/GitHub/Next-Gen-App/ios/Pods/Pods.xcodeproj: warning: The iOS Simulator deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 9.0, but the range of supported deployment target versions is 11.0 to 16.2.99. (in target 'GoogleDataTransport' from project 'Pods')
        /Users/lallu/Documents/GitHub/Next-Gen-App/ios/Pods/Pods.xcodeproj: warning: The iOS Simulator deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 9.0, but the range of supported deployment target versions is 11.0 to 16.2.99. (in target 'HTTPParserC' from project 'Pods')
        /Users/lallu/Documents/GitHub/Next-Gen-App/ios/Pods/Pods.xcodeproj: warning: The iOS Simulator deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 9.0, but the range of supported deployment target versions is 11.0 to 16.2.99. (in target 'ReachabilitySwift' from project 'Pods')
        /Users/lallu/Documents/GitHub/Next-Gen-App/ios/Pods/Pods.xcodeproj: warning: The iOS Simulator deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 9.0, but the range of supported deployment target versions is 11.0 to 16.2.99. (in target 'Telegraph' from project 'Pods')
        note: Run script build phase 'Run Script' will be run during every build because the option to run the script phase "Based on dependency analysis" is unchecked. (in target 'Runner' from project 'Runner')
        note: Run script build phase 'Thin Binary' will be run during every build because the option to run the script phase "Based on dependency analysis" is unchecked. (in target 'Runner' from project 'Runner')
        warning: Run script build phase 'xcode_backend build' will be run during every build because it does not specify any outputs. To address this warning, either add output dependencies to the script phase, or configure it to run in every build by unchecking "Based on dependency analysis" in the script phase. (in target 'RunnerUITests' from project 'Runner')
        warning: Run script build phase 'xcode_backend embed_and_thin' will be run during every build because it does not specify any outputs. To address this warning, either add output dependencies to the script phase, or configure it to run in every build by unchecking "Based on dependency analysis" in the script phase. (in target 'RunnerUITests' from project 'Runner')
✓ Completed building app with entrypoint test_bundle.dart for iOS simulator (94.5s)
Will uninstall apps before running tests
$ xcrun simctl uninstall 34EBC3CC-F7EA-40F4-8730-EA5891362A5F com.paytronix.nexgenfl
$ xcrun simctl uninstall 34EBC3CC-F7EA-40F4-8730-EA5891362A5F com.paytronix.nexgenfl.RunnerUITests.xctrunner
• Running app with entrypoint test_bundle.dart for iOS simulator on simulator iPhone 14 Plus...
$ xcodebuild -showsdks -json
Interactive shell mode enabled.
Logs: waiting for them...
$ flutter --no-version-check logs --device-id 34EBC3CC-F7EA-40F4-8730-EA5891362A5F
$ flutter attach --no-version-check --debug --device-id 34EBC3CC-F7EA-40F4-8730-EA5891362A5F --app-id com.paytronix.nexgenfl --target /Users/lallu/Documents/GitHub/Next-Gen-App/integration_test/test_bundle.dart --dart-define PATROL_WAIT=0 --dart-define PATROL_APP_PACKAGE_NAME=com.paytronix.nexgenfl --dart-define PATROL_APP_BUNDLE_ID=com.paytronix.nexgenfl --dart-define PATROL_ANDROID_APP_NAME=NGA JK Cafe --dart-define PATROL_IOS_APP_NAME=NGA JK Cafe --dart-define INTEGRATION_TEST_SHOULD_REPORT_RESULTS_TO_NATIVE=false --dart-define PATROL_TEST_LABEL_ENABLED=true --dart-define PATROL_HOT_RESTART=true
Hot Restart: waiting for attach to the app...
Assuming SDK version 16.2 for iphonesimulator
Looking for .xctestrun matching jk_*iphonesimulator16.2*.xctestrun at /Users/lallu/Documents/GitHub/Next-Gen-App/build/ios_integ/Build/Products
Found 1 match(es), the first one will be used
Found /Users/lallu/Documents/GitHub/Next-Gen-App/build/ios_integ/Build/Products/jk_iphonesimulator16.2-arm64-x86_64.xctestrun
$ xcodebuild test-without-building -xctestrun /Users/lallu/Documents/GitHub/Next-Gen-App/build/ios_integ/Build/Products/jk_iphonesimulator16.2-arm64-x86_64.xctestrun -only-testing RunnerUITests -destination platform=iOS Simulator,name=iPhone 14 Plus -resultBundlePath /Users/lallu/Documents/GitHub/Next-Gen-App/build/ios_results_1696933915157.xcresult
        : Waiting for a connection from Flutter on iPhone 14 Plus...
        Command line invocation:
            /Users/lallu/Downloads/Xcode.app/Contents/Developer/usr/bin/xcodebuild test-without-building -xctestrun /Users/lallu/Documents/GitHub/Next-Gen-App/build/ios_integ/Build/Products/jk_iphonesimulator16.2-arm64-x86_64.xctestrun -only-testing RunnerUITests -destination "platform=iOS Simulator,name=iPhone 14 Plus" -resultBundlePath /Users/lallu/Documents/GitHub/Next-Gen-App/build/ios_results_1696933915157.xcresult

        User defaults from command line:
            IDEBuildOperationResultBundlePath = /Users/lallu/Documents/GitHub/Next-Gen-App/build/ios_results_1696933915157.xcresult
            IDEPackageSupportUseBuiltinSCM = YES

        Writing result bundle at path:
                /Users/lallu/Documents/GitHub/Next-Gen-App/build/ios_results_1696933915157.xcresult

        2023-10-10 16:01:57.989840+0530 RunnerUITests-Runner[5948:612402] [SceneConfiguration] Info.plist contained no UIScene configuration dictionary (looking for configuration named "(no name)")
        2023-10-10 16:01:58.000052+0530 RunnerUITests-Runner[5948:612402] Running tests...
        2023-10-10 16:01:59.107425+0530 RunnerUITests-Runner[5948:612402] PatrolServer: INFO: PATROL_PORT is null, falling back to default (8081)
        2023-10-10 16:01:59.107537+0530 RunnerUITests-Runner[5948:612402] PatrolServer: INFO: PatrolServer constructor called
        2023-10-10 16:01:59.107615+0530 RunnerUITests-Runner[5948:612402] PatrolServer: INFO: PATROL_ENABLED flag is defined
        2023-10-10 16:01:59.108806+0530 RunnerUITests-Runner[5948:612402] PatrolAppServiceClient: created, port: 8082
        2023-10-10 16:01:59.108836+0530 RunnerUITests-Runner[5948:612578] PatrolServer: INFO: Starting server...
            t =      nans Checking existence of `"Allow" Button`
        2023-10-10 16:01:59.111304+0530 RunnerUITests-Runner[5948:612578] PatrolServer: INFO: Server started on http://<IP_ADDRESS>:8081
        2023-10-10 16:01:59.111354+0530 RunnerUITests-Runner[5948:612578] Server loop done, error: (null)
            t =      nans Ignoring failure to get hierarchy for remote element in process 3856 (Error getting main window kAXErrorServerNotFound)
            t =      nans Open com.paytronix.nexgenfl
            t =      nans     Launch com.paytronix.nexgenfl
            t =      nans         Setting up automation session
            t =      nans         Wait for com.paytronix.nexgenfl to idle
        : Syncing files to device iPhone 14 Plus...                           7.7s
        :
Hot Restart: attached to the app (press "r" to restart)
Hot Restart: logs are not connected yet
        : Flutter run key commands.
        : r Hot reload. 🔥🔥🔥
        : R Hot restart.
        : h List all available interactive commands.
        : d Detach (terminate "flutter run" but leave application running).
        : c Clear the screen
        : q Quit (terminate the application on the device).
        :
        : A Dart VM Service on iPhone 14 Plus is available at: http://<IP_ADDRESS>:61396/AqaF9uFAzBg=/
        : The Flutter DevTools debugger and profiler on iPhone 14 Plus is available at: http://<IP_ADDRESS>:9100?uri=http://<IP_ADDRESS>:61396/AqaF9uFAzBg=/
Hot Restart: logs connected
        Showing iPhone 14 Plus logs:
Hot Restart for entrypoint test_bundle.dart...
        :
        : Performing hot restart...
        : Restarted application in 1,031ms.
        Patrol (native): initialize() started
        Patrol (native): initialize() succeeded
        00:00 +0: example_test counter state is the same after going to home and switching apps
        Patrol (native): configure() started
        Patrol (native): configure() succeeded
        Patrol (native): pressHome() started
        2023-10-10 16:02:19.572585+0530 RunnerUITests-Runner[5948:612402] PatrolServer: INFO: pressing home button...
            t =      nans Pressing Home button
        [en]
        en
        null
        2023-10-10 16:02:20.044754+0530 RunnerUITests-Runner[5948:612402] PatrolServer: INFO: done pressing home button
        2023-10-10 16:02:20.047033+0530 RunnerUITests-Runner[5948:612402] PatrolServer: INFO: result: ()
        Patrol (native): pressHome() succeeded
        ══╡ EXCEPTION CAUGHT BY FLUTTER FRAMEWORK ╞═════════════════════════════════════════════════════════
        The following assertion was thrown while executing callbacks for FrameTiming:
        'package:flutter/src/widgets/binding.dart': Failed assertion: line 881 pos 16: 'sendFramesToEngine':
        is not true.
        flutter:
        Either the assertion indicates an error in the framework itself, or we should provide substantially
        more information in this error message to help you determine and fix the underlying cause.
        In either case, please report this assertion by filing a bug on GitHub:
          https://github.com/flutter/flutter/issues/new?template=2_bug.md
        flutter:
        When the exception was thrown, this was the stack:
        #2      WidgetsBinding.drawFrame.<anonymous closure> (package:flutter/src/widgets/binding.dart:881:16)
        #3      SchedulerBinding._executeTimingsCallbacks (package:flutter/src/scheduler/binding.dart:329:19)
        #4      _invoke1 (dart:ui/hooks.dart:158:13)
        #5      PlatformDispatcher._reportTimings (dart:ui/platform_dispatcher.dart:555:5)
        #6      _reportTimings (dart:ui/hooks.dart:107:31)
        (elided 2 frames from class _AssertionError)
        flutter:
        The TimingsCallback that gets executed was:
          Closure: (List<FrameTiming>) => void
        ════════════════════════════════════════════════════════════════════════════════════════════════════
        00:00 +0: example_test counter state is the same after going to home and switching apps [E]
          Test failed. See exception logs above.
          The test description was: counter state is the same after going to home and switching apps
        flutter:
        00:00 +0 -1: (tearDownAll)
        00:00 +1 -1: Some tests failed.
Hot Restart for entrypoint test_bundle.dart...
        :
        : Performing hot restart...
        : Restarted application in 892ms.
        Patrol (native): initialize() started
        Patrol (native): initialize() succeeded
        00:00 +0: example_test counter state is the same after going to home and switching apps
        Patrol (native): configure() started
        Patrol (native): configure() succeeded
        Patrol (native): pressHome() started
        2023-10-10 16:02:51.741735+0530 RunnerUITests-Runner[5948:612402] PatrolServer: INFO: pressing home button...
            t =      nans Pressing Home button
        [en]
        en
        null
        2023-10-10 16:02:52.204464+0530 RunnerUITests-Runner[5948:612402] PatrolServer: INFO: done pressing home button
        2023-10-10 16:02:52.204596+0530 RunnerUITests-Runner[5948:612402] PatrolServer: INFO: result: ()

Patrol version
patrol_cli v2.2.0
Patrol Doctor output
Program adb found in /Users/lallu/Documents/platform-tools/adb
Env var $ANDROID_HOME set to /Users/lallu/Library/Android/sdk
Program xcodebuild found in /usr/bin/xcodebuild
Program ideviceinstaller found in /opt/homebrew/bin/ideviceinstaller

Flutter Doctor output
[✓] Flutter (Channel stable, 3.10.6, on macOS 13.3 22E252 darwin-arm64, locale en-IN)
[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    ! Some Android licenses not accepted. To resolve this, run: flutter doctor --android-licenses
[✓] Xcode - develop for iOS and macOS (Xcode 14.2)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.3)
[✓] VS Code (version 1.83.0)
[✓] Connected device (3 available)
[✓] Network resources

Kindly check the xcode configuration for my flavor - jk. Please ping in case of any configuration issues in the xcode

Same issue for me. Any solution for this.

same issue for me @bartekpacia . Any solutions pls.?

Is the problem that, during Hot Restart, if you tap on the app with a finger/mouse, it turns black with a violet text "Test starting/finished"?

we have flavors for our application. It will be better if you tell us the configuration for the flavors in xcode

same here ios but in my case, r shows test starting but stuck at
Hot Restart for entrypoint test_bundle.dart...
 Building app with entrypoint test_bundle.dart for iOS simulator (debug)...
        2023-10-23 11:55:23.601 xcodebuild[28248:195610] DVTCoreDeviceEnabledState: DVTCoreDeviceEnabledState_Disabled set via user default (DVTEnableCoreDevice=disabled)
✓ Completed building app with entrypoint test_bundle.dart for iOS simulator (200.5s)
• Running app with entrypoint test_bundle.dart for iOS simulator on simulator iPhone 14 Pro...
        2023-10-23 11:57:52.667 xcodebuild[32664:210348] DVTCoreDeviceEnabledState: DVTCoreDeviceEnabledState_Disabled set via user default (DVTEnableCoreDevice=disabled)

patrol: ^2.4.0-dev.3
flutter version 3.13.8

We got it to work by

Create a new flavor for eg: integrationtest
At Targets > Runner > Build Settings, find FLUTTER_TARGET, set the new flavor entry point to integration_test/test_bundle.dart

@jacjac11 @bartekpacia thanks a lot. It worked finally for us. It's the solution here

The solution got worked and hence closing this ticket.


Sginal should be polyfill to global

This package is named a "*-polyfill", but its behavior is quite different from other typical polyfills. Most polyfills register the target to the global or tagetprototype, such as:
promise-polyfill
object.assign
Array.prototype.map
We should consider whether this approach is appropriate, and whether the current implementation makes it difficult for multiple packages to share the same polyfill.

I maintain the second two, and no, they do not automatically modify the global - quite intentionally, as that behavior is dangerous.
Providing an opt-in mechanism to do so would be great, though, although i hope it would wait until the proposal is at stage 3.


getCurrentInstance().ctx.$refs.keepAlive.$.__v_cache, can not run in prod

What problem does this feature solve?
When I want to close tabs, I delete the corresponding route cache in this way
What does the proposed API look like?
<router-view v-slot="{ Component, route }">      <keep-alive ref="keepAlive">      <component :is="Component" :key="route.fullPath" v-if="$route.meta?.keepAlive" />      </keep-alive>      <component :is="Component" :key="route.fullPath" v-if="!$route.meta?.keepAlive" />   </router-view>
<script lang="ts" setup> const ins = getCurrentInstance() as any; let removeCache: Function; onMounted(() => {    removeCache = (key) => {       const keepAliveInstance = ins.ctx.$refs.keepAlive.$;       const cache = keepAliveInstance.__v_cache;       cache.delete(key);     };   }); </script>
The tabs tag close event executes this function, delete the corresponding route cache in this way

About custom caching strategy, it does many people need😔
#4339  #5105

Then let's keep it in those issues.


Add season list to database.

Maybe the only feasible way to actually update this database without requiring a re-deploy.
Add a season table with two columns, the int representation of a season and the string version?
Then, to add a new season, a new row can be added. Maybe a new cmdline 'mode' can be added to support this?

Merged!


Fix incorrect pending transaction value

Pending transactions shows incorrect siacoin transfer value until confirmed
https://discord.com/channels/809849352516141067/827641929008152635/1243532719166853221
Double check renterd calculation as well.

adding screenshot here:
last three transactions were all -1000 SC

@n8maninger the siacoin values are calculated as BigNumber(t.inflow).minus(t.outflow) for both pending and regular transactions - is this approach incorrect? I just tested sending 50 SC, and it looks like inflow is 49 and outflow is 0 in the pending transaction data.

Ah interesting, @n8maninger if I do the opposite and send 100 siacoin to hostd, the incoming 100 siacoin value shows as inflow 100, outflow 0.


Frontend-homework-6

Created a .env file

CREATE-POST

GET

GET-BY-ID

Metric
Value

alert_status
b:white_check_mark: OK/b

bugs
0

code_smells
0

reliability_rating
A (0 Bugs)br/Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)br/Note: A being best, E being worst

vulnerabilities
0

Metric
Value

alert_status
b:white_check_mark: OK/b

bugs
0

code_smells
0

reliability_rating
A (0 Bugs)br/Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)br/Note: A being best, E being worst

vulnerabilities
0


Do interactors work with Playwright?

As far as I could tell, there’s no documented support for Playwright.
Can this library made to be work with Playwright? Especially when the DOM is available only lazily?

Hi @paambaati! Thanks for your question.
The short answer is that there is no current integration between interactors and playwright. However there is no reason that one couldn't be written. The key thing is that the actual interactor code must be running in the browser. That means they must be bundled and loaded by playwright. If you are using the Playwright Test framework, then there is some extra work required because the actual test runner is running in NodeJS, while the page context is running in the browser.
You would need to add the interactors via some mechanism such as an init script https://playwright.dev/docs/api/class-page#page-add-init-script and then invoke them with a JSHandle https://playwright.dev/docs/api/class-jshandle
Eventually we could bundle this into a  @interactors/with-playwright-test package so that you could just import them directly and skip this manual setup.


deploy issue

After hexo d:
Committing

/post/92d72c3a.html uploaded successfuly
Additions complete.
ftp.put failed.
Updates failed.
Commit failed.
FATAL Something's wrong. Maybe you can find the solution here: http://hexo.io/docs/troubleshooting.html
Error: 421 Data timeout. Reconnect. Sorry.
at module.exports.Ftp.parse (G:\OneDrive\Github\时光之笔\node_modules\jsftp\lib\jsftp.js:233:11)
at module.exports.Ftp.parseResponse (G:\OneDrive\Github\时光之笔\node_modules\jsftp\lib\jsftp.js:148:8)
at Stream. (G:\OneDrive\Github\时光之笔\node_modules\jsftp\lib\jsftp.js:119:24)
at Stream.emit (events.js:189:13)
at Stream.reemit (G:\OneDrive\Github\时光之笔\node_modules\duplexer\index.js:70:25)
at Stream.emit (events.js:189:13)
at Stream.write (G:\OneDrive\Github\时光之笔\node_modules\event-stream\index.js:192:12)
at Stream.stream.write (G:\OneDrive\Github\时光之笔\node_modules\through\index.js:26:11)
at Stream.ondata (internal/streams/legacy.js:15:31)
at Stream.emit (events.js:189:13)
at drain (G:\OneDrive\Github\时光之笔\node_modules\through\index.js:36:16)
at Stream.stream.queue.stream.push (G:\OneDrive\Github\时光之笔\node_modules\through\index.js:45:5)
at emit (G:\OneDrive\Github\时光之笔\node_modules\split\index.js:35:14)
at next (G:\OneDrive\Github\时光之笔\node_modules\split\index.js:44:7)
at Stream. (G:\OneDrive\Github\时光之笔\node_modules\split\index.js:49:5)
at Stream.stream.write (G:\OneDrive\Github\时光之笔\node_modules\through\index.js:26:11)
at Socket.ondata (_stream_readable.js:689:20)
at Socket.emit (events.js:189:13)
at addChunk (_stream_readable.js:284:12)
at readableAddChunk (_stream_readable.js:265:11)
at Socket.Readable.push (_stream_readable.js:220:10)
at TCP.onStreamRead [as onread] (internal/stream_base_commons.js:94:17)

Sorry for the late reply. I am currently refactoring this plugin. If the problem still exists, please feel free to continue reporting. See #148


Finalize Task API

move Task API to vscode.d.ts
support correlating a task with a terminal.

@alpaix and @rickvanprim I added events to the task execution that signals when the underlying process starts and when it ends. The start contains the process id, the end the exitCode.
Please let me know if this will work for you (minus the polling for now).
See also https://github.com/Microsoft/vscode/issues/49665

@dbaumer I'll be happy to migrate as soon as it's available. What's the timeline for enabling the new API in insiders and public? When will it be available for testing?

@alpaix I pushed a change today that adds the above API to vscode.d.ts (namespace tasks). I left the old workspace API in proposed but I would like to remove it until end of the week. Can you confirm that you moved to the new task namespace.

@dbaeumer We do not plan to support older API as the Live Share remote tasks functionality is an experimental feature. We'll release a final revision along with the latest stable Task API when it's available in non-insiders VSCode version.


Mongodb High CPU

There are a lot of query to DB in crawler, we should optimize it.
Try to use cache in some case to decrease the number of query to db

fixed #333


auto connection after program start

hi,
is it possible that the ftp connection starts automaticly after I started the program?
(without clicking "connect")?

This is not something that is currently not built in.


Support for latest webview_flutter 4.0.1

When trying to add the package with the latest webview_flutter this error shows:
Because webview_flutter >=4.0.0 depends on webview_flutter_platform_interface ^2.0.0 and every version of webview_win_floating depends on webview_flutter_platform_interface ^1.9.5, webview_flutter >=4.0.0 is incompatible with webview_win_floating.

I just take a look at the Migrating from 3.0 to 4.0 in webview_flutter
Hmm... a lot of API changes...
It may take me some time to upgrade to [webview_flutter_platform_interface 2.0.1]
Before I finish it, please... wait... orz

It's all done ... maybe
For the new flutter_webview API,
in 4.0.1, the WebView seems not destroyed after widget dispose,
but in 3.0.3 the WebView will be destroyed.
I post a new issue to [flutter_webview] #118539
I will push to pub.dev after I can figure out how to destroy webview in new API 4.0.1
Currently I only push to github.
Currently Windows webview will be dstroyed when WebViewWidget disposed.
Please modify your pubspec.yaml if you want to upgrade to new API now:
dependencies:
  webview_win_floating:
    git:
      url: https://github.com/jakky1/webview_win_floating.git
      ref: master
  webview_flutter: ^4.0.1

and the windows webview registration code is changed:
WindowsWebViewPlatform.registerWith();
// WebView.platform = WindowsWebViewPlugin(); //old version

The README is also updated.

Thank you! Hopefully you'll be able to figure out how to destroy the webview as well.


TyperError:init() got an unexpected keyword argument 'assist'

Can you help me with these error messages?

It is very weird. Where is the keyword "assist" from? I am not sure that mmdet has such a keyword in the backbone setting.

Thanks for your quick reply. In the mmdetection repository, I saw a similar TypeError error message.https://github.com/open-mmlab/mmdetection/issues/5734
Maybe there is something wrong with my environment configuration?

Thanks for your quick reply. In the mmdetection repository, I saw a similar TypeError error message.open-mmlab/mmdetection#5734 Maybe there is something wrong with my environment configuration?

I recommend you to use pycharm to debug to find the "assist" keyword

I can't find this file ‘utdac_detection_coco.py', please tell me where is this file

 I can't find this file ‘utdac_detection_coco.py', please tell me where is this file

browse the internet for "how to use the custom dataset in mmdetection"
and I think this link may help: https://github.com/mousecpn/mmdetection/blob/master/docs/2_new_data_model.md


can i start a func which have a argument?

RT

Did you try it?  I think it is not possible because of the function pointer type, but try it.

Did you try it? I think it is not possible because of the function pointer type, but try it.

Thanks for your replay, i will try it. If it is not possible, i thinks i can use global variable and  i will close this issue.

The reason for my project not using this is for this primary reason. Otherwise, it looks like a nice library.
I also tried to use an anonymous function (lambda), but that didn't work as it does not seem to be supported by this library.
I opted for https://github.com/avandalen/VirtualDelay instead.

The reason for my project not using this library is for this primary reason. Otherwise, it looks like a nice library.
I also tried to use an anonymous function (lambda), but that didn't work as it does not seem to be supported by this library.
I opted for https://github.com/avandalen/VirtualDelay instead.

But it only support UNO, i what to find a ticker that can support esp8266 or esp32


Rollable Tables only output the roll amount

Rollable Tables only output the roll in chat, not the content entered in the Rollable Table (whether text, entities or compendium), no modules running, foundry 0.8.9, latest version of the system, table settings look to be correct.

I assume you mean something like this is happening. What are you expecting? What's your use case here?

Thanks for the thorough response! Very helpful. I've pushed a fix for that in v0.3.13, previous chats should even show the info i was accidentally hiding!

Awesome, thank you so much
On Sat, 9 Oct 2021, 12:19 hitcherland, @.***> wrote:

Thanks for the thorough response! Very helpful. I've pushed a fix for that
in v0.3.13, previous chats should even show the info i was accidentally
hiding!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hitcherland/FoundryVTT-Heart/issues/9#issuecomment-939271976,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AT4UTVXT5YNPMR3DBV55WUDUGAJJTANCNFSM5FGVJZOQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

This appears to be broken again in 0.4.2:

Whoops, refixed in 0.4.3

Thanks!


Refactor BooleanExpressionVisitor

Currently, BooleanExpressionVisitor uses visit and leave. I've got the feeling that it would be better style to only use visit in combination with recursive calls, but I'm not sure about that.

By that we could also rewrite the current logic:
connect(and);
this.connectorDeque.push(" AND ");

to something more readable


Yank old versions of diesel_codegen

We seeing quite periodic questions/bugreports about something broken in diesel that boils down to someone tries to use the old diesel_codegen.
I suggest to yank at least the 0.16 version to indicate that this crate shouldn't be used anymore.

@weiznich What if I want to use diesel 0.16 though ? :/
I agree that those questions are way too frequent but yanking old versions doesn't sound like a good solution to me.

A other solution would be to publish a empty version as 0.17 or so that does print an error message saying not to use this crate anymore in compilation with recent diesel versions.

I'm with @Eijebong on this, I don't think yanking old versions of this crate is a good idea.

So what's about releasing diesel_codegen 0.17 that gives error messages not to use that crate?
If someone wants to use diesel 0.16 it is an intentional decision and that person will use diesel_codegen 0.16. If someone uses diesel_codegen by accident (or old guides) there is a message what's wrong. Old projects using diesel 0.16 will continue to work because going from 0.16 to 0.17 indicates a breaking change.

I'm not sure how that's helpful. The issue is people with existing 0.16 projects updating Diesel to 0.99 or 1.0 and leaving diesel_codegen there. It's not like they'd randomly decide to go to 0.17.

If there's a way to get the version of diesel used in a build script, we could add one which makes the compilation fail and release 0.16.whatever

The issue is people with existing 0.16 projects updating Diesel to 0.99 or 1.0 and leaving diesel_codegen there.

I'm not sure if that's really the issue. (Let's just ask the next person with that problem :wink:)

It's not like they'd randomly decide to go to 0.17.

If they are updating diesel I assume they will also update diesel_codegen to the latest version. In that case will releasing a newer version emitting some error message point them into the right direction.

If there's a way to get the version of diesel used in a build script, we could add one which makes the compilation fail and release 0.16.whatever

So we just need to parse Cargo.toml/Cargo.lock? 🙈

So we just need to parse Cargo.toml/Cargo.lock? 🙈

TBH that actually sounds fairly reasonable


[BUG] Generic HTTP Actions in Java Client does not work with AwsSdk2Transport

What is the bug?
This bug is in feature: #377 where the generic Http actions are enabled. While the generic Http actions work fine on local open search instances, they give 403 when working with AWS open search.
One reason can be, as I see  is that in the PR: https://github.com/opensearch-project/opensearch-java/pull/910/files , the changes are done to  RestClientTransport.java and ApacheHttpClient5Transport.java , but the changes to AwsSdk2Transport is not done.
cc: @reta
How can one reproduce the bug?
Try calling the AWS open search. The below opensearch client gives 200 when working with non generic flow, but fails for generic flow.
below is sample code:
private String searchWithGenericClient(OpenSearchClient openSearchClient) throws IOException {
        Response response = openSearchClient.generic()
                .execute(
                        Requests.builder()
                                .endpoint("/" + "notifications" + "/_search?typed_keys=true")
                                .method("GET")
                                .json("{"
                                        + "    \"query\": {"
                                        + "        \"match_all\": {}"
                                        + "    }"
                                        + "}"
                                )
                                .build()
                );
            return response.getBody().get().bodyAsString();
    }

open search version:  2.10.1
 <dependency>
            <groupId>org.opensearch.client</groupId>
            <artifactId>opensearch-java</artifactId>
            <version>2.10.1</version>
        </dependency>

Output:
Exception in thread "main" org.opensearch.client.opensearch._types.OpenSearchException: Request failed: [security_exception] 403 Forbidden
	at org.opensearch.client.transport.aws.AwsSdk2Transport.parseResponse(AwsSdk2Transport.java:477)
	at org.opensearch.client.transport.aws.AwsSdk2Transport.executeSync(AwsSdk2Transport.java:396)
	at org.opensearch.client.transport.aws.AwsSdk2Transport.performRequest(AwsSdk2Transport.java:193)
	at org.opensearch.client.opensearch.generic.OpenSearchGenericClient.execute(OpenSearchGenericClient.java:168)
	at com.anubhav.dao.OpenSearchDao.searchWithGenericClient(OpenSearchDao.java:52)
	at com.anubhav.dao.OpenSearchDao.main(OpenSearchDao.java:37)

What is the expected behavior?
We need the generic flow to work for AWS opensearch as well. We are in the design stage of our product where, we need to decide generic flow vs object based flow. We want to use generic flow, but due to this error, we are stuck.

No screenshot. Only code snippet already attached.
Do you have any additional context?
Looks like the change is not done for all flows for https://github.com/opensearch-project/opensearch-java/issues/377  and PR: https://github.com/opensearch-project/opensearch-java/pull/910/files

Thanks @reta for assigning to yourself.
Complete code:
        software.amazon.awssdk.http.SdkHttpClient httpClient = ApacheHttpClient.builder().build();
        org.opensearch.client.opensearch.OpenSearchClient client = new OpenSearchClient(
                new AwsSdk2Transport(
                        httpClient,
                        // replace with endpoint in aws opensearch
                        HttpHost.create("https://xxxxxxxxxx.us-east-1.aoss.amazonaws.com").getHostName(),
                        "aoss",
                        Region.of("us-east-1"),
                        AwsSdk2TransportOptions.builder().build()));

        Response response = client.generic()
                .execute(
                        Requests.builder()
                                .endpoint("/" + "notifications" + "/_search?typed_keys=true")
                                .method("GET")
                                .json("{"
                                        + "    \"query\": {"
                                        + "        \"match_all\": {}"
                                        + "    }"
                                        + "}"
                                )
                                .build()
                );
        System.out.println(response.getBody().get().bodyAsString());
```java

## Setup:
Use AWS open search server-less: https://aws.amazon.com/opensearch-service/features/serverless/

@anubhav3008 do you wanna take it? thank you! :pray:

@anubhav3008 do you wanna take it? thank you! 🙏
@reta , I can take it up. But i not sure how to fix it. I am trying to understand the code flow as of now.

Which version would this fix be in?

@BrendonFaleiro https://github.com/opensearch-project/opensearch-java/blob/main/CHANGELOG.md, 2.10.3. If you want a release quickly open an issue "Release v. Next" and we can do it soon.


Update configure-kubenet.md

PR for https://supportability.visualstudio.com/AzureContainers/_workitems/edit/80297

@asudbring
Can you review the proposed changes?
IMPORTANT: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


feat: run command that does compile/build and then up

The goal here is to align better with the terminology used in go, e.g. go build and go run. We can have lunchpail build and lunchpail run.

one thing we'll need to figure out here: throughout the code are calls to compilation.Name() (perhaps soon build.Name() via #329).
these calls assume that the build name is a static value of the executable itself. we may need to add a way to set the build name temporarily, just for the duration of the current command.


Return the last event in a window -- join clause issue ?

Hello,
Concerning this query:
WITH LastInWindow AS
(
SELECT
MAX(Time) AS LastEventTime
FROM
Input TIMESTAMP BY Time
GROUP BY
TumblingWindow(minute, 10)
)
SELECT
Input.License_plate,
Input.Make,
Input.Time
FROM
Input TIMESTAMP BY Time
INNER JOIN LastInWindow
ON DATEDIFF(minute, Input, LastInWindow) BETWEEN 0 AND 10
AND Input.Time = LastInWindow.LastEventTime
Is not the "ON DATEDIFF(minute, Input, LastInWindow) BETWEEN 0 AND 10" useless, as the join also contains "AND Input.Time = LastInWindow.LastEventTime" ? So the datediff is already 0, no?

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

ID: f0ebe9e5-9a83-b5e6-5dd6-34c841cae021
Version Independent ID: 8c19161b-2eef-ecc9-37fe-5e665da9b2cb
Content: Common query patterns in Azure Stream Analytics - Azure Stream Analytics
Content Source: articles/stream-analytics/stream-analytics-stream-analytics-query-patterns.md
Service: stream-analytics
GitHub Login: @AliciaLiMicrosoft
Microsoft Alias: ali

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

@kdwitte69
Thank you for your feedback!
Since this issue isn't directly related to improving our docs, and to gain a better understanding of your issue, I'd recommend working closer with our support team via an [Azure support request] (https://docs.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request). Or you can leverage our Q&A forum by posting your issue there so our community, and MVPs can further assist you in troubleshooting this issue or finding potential workarounds.
[Teams Q&A forum] (https://docs.microsoft.com/en-us/answers/topics/46488/office-teams-windows-itpro.html) for technical questions about the configuration and administration of Microsoft Teams on Windows.
[Microsoft Teams Community forum] (https://answers.microsoft.com/en-us/msteams/forum?sort=LastReplyDate&dir=Desc&tab=All&status=all&mod=&modAge=&advFil=&postedAfter=&postedBefore=&threadType=All&isFilterExpanded=false&page=1)
Thank you for your time and patience throughout this issue.


fix codecov report

Problem
Codecov isn't working anymore. It started failing after the sbt-scoverage bump.
Solution
Use the previous version of the plugin
Checklist

[x] Unit test all changes
[x] Update README.md if applicable
[x] Add [WIP] to the pull request title if it's work in progress
[x] Squash commits that aren't meaningful changes
[x] Run sbt scalariformFormat test:scalariformFormat to make sure that the source files are formatted

@getquill/maintainers

:+1:

👍


Rename Action from Build to Release

Rename Build Action to Release to better reflect what it does

Merged myself cause simple change and not possible to test outside of a release :P


My cleanup of the shortcut-property-access-on-unknown-property-name optimization relied on IsExternal() to tell us whether an object is eligible. But we must also exclude HostDispatch, which returns false on IsExternal().

@Yongqu


get visible contents from reader

how can you get the contents of the starting and ending locator for content that is visible in the reader?
Lets say we have pagination enabled and our reader render's two pages. How can I get content of only the page that is visible.
Here's an example:
PAGE 1:

PAGE 2:

I want to get the html content of PAGE 1 ONLY. not of the entire chapter. How can I do that?

@aferditamuriqi

Use setting as paginated and columns to 1

this is not what I meant. I want to get the text that is rendered in the first page inside javascript, not on screen.
I want to use this text to see what contents of the epub the user can currently see.
@aferditamuriqi

That's not part of the features and ePub reader has .
You might es t to consider writing your own script and inject


final resolution size

how can i change the  final size resolution to improve the quality?
im getting a 349 x 512 picture from a 4.800 x 7.024 original

Closing this as unrelated to Hub. Please reopen it if there was a misunderstanding.


sm move assignment operator

I'm not very happy with the semantics (current move ctor actually copies and this move assignment does the same) however this makes e.g. SM reset much cleaner (by assignment a fresh instance).

thanks @dsiroky


Renames docker image to zipkin, as it isn't ambiguous

Moving forward, we have a single docker image. Instead of a suffix, we
can standardize on the simple word "zipkin"

cc @abesto @klingerf @gena01 @adrianco

+1

re-cutting 1.1.2

forgot to add repos on quay.io and dockerhub. cutting release again


Add a new step to the load process

As a developer I want to be able to initialize the program state before the level is loaded in order to avoid errors when users switch between different savegames.
Assumptions/Preconditions

The level loading procedure currently consists of the following steps:

{Flags|NodeGeometry|SegmentGeometry}.OnBeforeLoadData
ICustomManager.OnBeforeLoadData (for every manager)
(SerializableDataExtension loads custom game data)
ICustomManager.OnAfterLoadData (for every manager)
(LoadingManager sets up detours)
ICustomManager.OnLevelLoading (for every manager)

It is currently impossible for managers to reset their fields before geometries are calculated.

Tasks

Add a new OnInit step that is called before geometries are calculated.
Update the documentation of AbstractCustomManager.

OR for bonus points: Move the documentation to the wiki.

Technical notes

The step needs to be added to ICustomManager, AbstractCustomManager
Log calls to OnInit to TMPE.log
check whether existing code from constructors/OnBeforeLoadData code should be moved to OnInit

Acceptance criteria

The new step is implemented and is executed before calculating geometries
TMPE still works when the user switches to another game level

@kianzarrin have the various hotswap updates effectively resolved this issue?

yes


Do backups

Write a cron job to make periodic compressed signed backups of the DB.

Closed in e2d53da4 .