[WIP] Loading spinner for QuickOpen

Currently, QuickOpen gives no indication that it is actively searching - this change adds an animated loading spinner to show that something is happening.
TODO:

[x] Fix up spinner (rotation is off-center) (used placeholder)
[x] Use Job progress API to show progress once MenuJob is complete
[x] Port over fix when running multiple iterations of job in tick
[ ] Test if there are any remaining issues for #540
[x] Investigate why sometimes setLoading(false) doesn't make it when Ripgrep exits quickly
[x] Add deltaTime and totalTime to Tick action
[x] Store menu creation time
[x] Store menu animation time to fade-in spinner for less jarring in cases when it is only briefly displayed

Tested the dev build - it was able to search to completion on root directory on OSX. Need to check with a compiled and code-signed build, though.


I-176: Add wartremover checks (multiple warts)

Continuation of #176.
The following warts were enabled and applied:

Null
Product
FinalVal
JavaConversions
IsInstanceOf
Serializable
JavaSerializable
PublicInference

Coverage remained the same at 47.739% when pulling 567f968a90003eaf88ed425942531572ae25510c on EzequielPostan:I-176-add-wartremover-checks-multiple-warts into f06d58cddbf35dbb8d1be4cf4c03ce8291b9334b on ScorexFoundation:master.

Hi @ceilican
First of all, thank you for your time to review this PR.
As a general comment, I agree with the proposal of reducing the scope of (currently) public vals to private in the cases where that make sense.
With respect of cases where we add type annotations, I wonder what would be the advantage of not adding the types suggested by warts (or in general) in the case where the type annotation is not too verbose (or where they look strange like the String type annotation inside the log function).
The reason why I like having these type inference warts (like Product, Serializable, etc) is because we are using libraries that sometimes involve the type Any (like Akka) and hence almost erase parts of type checking.
I also like types because to me they are simple specifications and, in that line, I think of Scala type inference as a verifier of what I specify for. If I leave open specification, I consider that am loosing part of the compiler help (and not the other way).
This view is of course influenced by my own background. I have no problem to adapt to a different view when collaborating to the project, but it would be interesting for me to read your views on why to avoid type annotations (besides a possible line length increase).

As a general comment, I agree with the proposal of reducing the scope of (currently) public vals to private in the cases where that make sense.

Good!

I also like types because to me they are simple specifications [...]

Me too.

in the case where the type annotation is not too verbose

Exactly. I don't like to see pointless verbosity such as val car: Car = new Car when something like val c = new Car would suffice.    (I'm assuming that c is a local val here... For a public val, I wouldn't abbreviate car to c.)

@EzequielPostan please resolve the conflicts with master !

@kushti
I rebased the branch.
@ceilican
I see that we agree in views and just may differ in how verbose certain cases are.
Do you think I should add any specific changes to the PR (e.g. try to change some public members to private)? or should we merge this in the current state?

@EzequielPostan

Do you think I should add any specific changes to the PR?

I think the two changes you suggested would be excellent:

try to change some public members to private)
try to review the strange ascriptions cases (like the String one inside log function) during the week.

Good, I will review those changes then.


alt not preserved across SYSREQ.C calls.

pri is the return value, so obviously will be clobbered, but I'm pretty sure alt shouldn't be affected.

Just spent several hours debugging a crash that came down to this!

Had a quick look myself, but wasn't sure of a good way to solve it, since it can't be put on the stack as that will mess up the call, and can't be put in a global as you can call a native from inside a function called by a native called by function (CallLocalFunction etc).


Feature request: Save custom algorithms / scratch papers with unique urls for ease of sharing

I have thought about it, and I'm still planning how I should do this within a static page.

I may be able to use apis of Pastbin.com to save and load scratch papers. Thanks for reminding me!

added at f148adb9014865569f0a359484059f458475a06c


Use a modern version of python

The setup.py script seems to indicate that a python version of <3.4 is required. This is horribly out of date, and means that I can't submit it as a package to package managers that don't support python versions that old. Are there any plans to use a more modern python version?

@Pandapip1 are you referring to this line?
https://github.com/plotly/Kaleido/blob/3b590b563385567f257db8ff27adae1adf77821f/repos/kaleido/py/setup.py#L298
That's just saying "if python <3.4 we need to install pathlib." It doesn't say anything about which Python versions are supported by Kaleido, which is all of them AFAIK. Other Plotly projects currently use >=3.6 so next time we make a Kaleido release it will probably move to that and this line can be removed entirely.


TST: refactor optimize/tnc tests

Make tests individual (were all in one previously).
Use assert_ so that things are actually checked.

Good to split the tests.
Instead of
assert_(self.error(self.f45, x, xopt) < 1e-8,
             self.error_msg(fg.__name__, nf, rc))

you could also write
assert_allclose(self.f45(x), self.f45(xopt), rtol=0, atol=1e-8)

This should give a more informative error message, because assert_allclose shows the actual values it compares while assert_ does not. If it's useful you could still add the number of iterations as msg keyword. You can then remove the error and error_msg methods.
Disclaimer: I didn't test the above code, may need a tweak.

Ralf Gommers wrote:

Instead of
assert_(self.error(self.f45, x, xopt) < 1e-8,
             self.error_msg(fg.__name__, nf, rc))

you could also write
assert_allclose(self.f45(x), self.f45(xopt), rtol=0, atol=1e-8)

This should give a more informative error message, because assert_allclose shows the actual values it compares while assert_ does not. If it's useful you could still add the number of iterations as msg keyword. You can then remove the error and error_msg methods.

Yes, it's better indeed.
Should I make another commit or can I amend this one and force-push the
branch?

You can't amend and then make it appear on the same PR. So I normally only add commits, and then if there are too many just squash them when review is done, do a final test and push.

Ralf Gommers wrote:

So I normally only add commits, and then if there are too many just squash them when review is done, do a final test and push.

Ok, thanks for the tip.
I'll merge this in a few days.

Okay, I've tested this now, works fine. I'd recommend one more minor change, because I can never remember which test functions take a decimal and which an atol-like argument. It would be more easy to read if you'd replace 8 with decimal=8.
I think you can go ahead and merge in a couple of days if no one else comments.


Add: Rede NGT [BR]

Channel Name
Rede NGT
Alternative Names (optional)
No response
Network (optional)
No response
Owners (optional)
No response
Country
BR
Subdivision (optional)
BR-SP
City (optional)
No response
Broadcast Area
s/BR-SP
Languages
por
Categories (optional)
Entertainment
NSFW
FALSE
Launched (optional)
No response
Closed (optional)
No response
Replaced By (optional)
No response
Website (optional)
https://redengt.com.br/
Logo
https://i.ibb.co/m82g3bN/descarga-1.png
Notes
https://acesso.ecast.site:3922/live/redengtlive.m3u8

Thank you for the stream: it will be included in the next release.


Login user store in firebase

Description
A clear and concise description of what the PR does.
i store a login user in firebase store

This PR does the following:

Fixes ...

#340
Related Issues
Link any related issues using the format Fixes #issue_number.
This helps to automatically close related issues when the PR is merged.

Placeholder: "Fixes #123"

Changes
List the detailed changes made in this PR.

Added a new feature to ...
Refactored the ...
Fixed a bug in ...

Testing Instructions
Detailed instructions on how to test the changes. Include any setup needed and specific test cases.

Pull this branch.
Run npm install to install dependencies.
Run npm test to execute the test suite.
Verify that ...

Screenshots (if applicable)
Add any screenshots that help explain or visualize the changes.
Additional Context
Any additional context or information that reviewers should be aware of.

This PR is based on the following...

Checklist
Make sure to check off all the items before submitting. Mark with [x] if done.

[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] I am working on this issue under GSSOC

Hey @Satyam1923 ,check the firebase login user store or not

@Varsani2520 isn't same thing already implement in the website why do it again.

@Satyam1923
i need help can you please tell me what excetly i have to store in firestore


PR#1543 Follow up - Adding invitation_msg_id and their_public_did back to record_value.

Signed-off-by: Shaanjot Gill<EMAIL_ADDRESS>

@swcurran Based on discussion with Andrew and Ian regarding this link, invitation_msg_id and their_public_did needs to be added back to record_value for 7.3rc0 release. For backward compatibility, the previously created ConnRecord would need to be re-saved so that it can be reused. Andrew sees no real benefit to doing this as old connections cannot be reused but new ones can be established.
Regarding updating existing data or handling breaking changes, based upon Ian's suggestion, I am working on an ACA-Py upgrade command. I will provide more details on it separately.

Sounds good, @shaangill025 . Let me know what is "must have" for 0.7.3-rc0 is ready and merged, and what can wait.  In the Changelog, I added the following about the previous change.  Is that good enough to go forward for 0.7.3 (final), or do you think we should include the "upgrade" process in 0.7.3?
Thanks!

For those that have an existing deployment of ACA-Py with long-lasting connection records, some action
may be required in order to use RFC 434 Out of Band and the "reuse connection" as the invitee. In PR #1453
(details below) a performance improvement was made when finding a connection for reuse. The new approach
(adding a tag to the connection to enable searching) applies only to connections made using this ACA-Py
release and later, and "as-is" connections made using earlier releases of ACA-Py will not be found as reuse
candidates. We don't think this will affect (m)any deployments and so have not implemented a migration
approach. However, if this issue might affect your deployment, please submit an issue and the ACA-Py
team will work with you to find a solution.

@swcurran This is ready and the changelog looks good for 7.3.0 final. We don't need the upgrade command for this change to ConnRecord. Ian just mentioned that new tags are being added to credential_exchange (most probably in PR#1545) for which backward compatibility needs to be managed but I don't think this is planned for this release. So, the upgrade process can be part of the subsequent release.

Sounds good -- thanks @shaangill025 . @ianco and @andrewwhitehead -- please review and merge this (if OK) when you get a chance.

Is this the only change? Does the connection lookup code need updating, or was it trying to filter on tags that weren't defined?


Which format of prompt did you use? QCMG-A or?

In https://github.com/SooLab/DDCOT/blob/master/utils_prompt.py, there are many formats of input and output, which one did you use to generate the results in Table 1?

Yes, we use the QCMG-A format

It's great work, now i am following your work, can we add wechat? my wechat:SemiZxf


fix(handle-enter-confirm-popup): added key action to confirm popup wi…

…th class functions
Description
Keyboard "Enter" key was not functional on confirm pop up. Added an event listener for key down event  whenever the modal is opened and remove when the modal is closed.
Fixes # (issue)
#5111
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Added an API source and enabled Request confirmation before running API settings for it. Now when the API is run, it asks for confirmation, press enter to confirm and esc to close.
Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:red_circle: Total coverage has decreased
// Code coverage diff between base branch:release and head branch: fix/handle-enter-confirm-popup

Status
File
% Stmts
% Branch
% Funcs
% Lines

:red_circle:
total
55.05 (0)
36.91 (-0.01)
33.91 (0)
55.57 (0)

:red_circle:
app/client/src/utils/autocomplete/TernServer.ts
52.71 (-0.23)
40.83 (-0.84)
36.21 (0)
56.74 (-0.25)

/ok-to-test sha=37e092c

/ok-to-test sha=1399cf9


swal.finally not a function

The polyfill your using for your promises doesn't include support for "promise.finally": https://jsfiddle.net/kdf82tc6/1/
This can be solved by using a promise.finally polyfill like this one: https://github.com/matthew-andrews/Promise.prototype.finally
I'm using this lib to wrap some internal logger functions, and having support for promise.finally would be the bees knees

native Promise object doesn't have finally() method:

Why then() doesn't work for you and you need finally()?

All of the previous promise libraries i've used supported finally, its a bit weird to see that native doesn't support it. The idea is to do some cleanup when the promise and any subsequent .then() have resolved.
https://github.com/domenic/promises-unwrapping/issues/18
I'm currently using this for a dialog library in an angular 2 application. Here is a use-case example:
I want to track when a modal is open, and perform some logic on modal open and modal close events. In this case, its applying a blur class to the root application component, so the background comes out blurred and slightly grayscale.
class LoggerService {
  events = new EventEmitter();

   dialogConfirm(title: String, text?: String, confirmText?: String, cancelText?: String): Promise<any> {

    // notify subscribers that the event window has opened.
    this.events.emit({value: 'open'});

    return swal({
      title: title,
      type: 'warning',
      text: text,
      reverseButtons: true,
      confirmButtonText: confirmText || 'Yes',
      showCancelButton: true,
      cancelButtonText: cancelText || 'No'
    })
    .finally(() => {

      // notify any event subscribers that the modal has closed
      this.events.emit({value: 'close'});
    });
  }
}

class TheLeagueAppComponent implements OnInit {
  modalOpen: boolean;

  constructor(
    private _logger: LoggerService
  ) {

  }

  ngOnInit() {
    this._logger.events.subscribe(event => {
      this.modalOpen = event.value === 'open';
    });
  }
}

And lets say I want to do run a dialog, perform some logic in subsequent functions, and then close.
onDeleteMember(id: string) {
  this._logger
    .dialogConfirm('Delete Member', 'Are you sure?')
    .then(result => {
      if (result) {
        return this._http.delete(`api/member/${id}`);
      }
    });
}

Here is an example the effect in action
Blur:
https://gyazo.com/904c86268241534037b01d4da0a741da
Currently, as a workaround, I'm using a then() callback which Promise.resolve(return) resolves the returned value (in the case of a confirm dialog), but ideally I want to emit the event on full promise resolution, not as the first event after the modal has closed.

Hi @hellsan631 !
I read your comment several times and I'm still not sure why exactly you need finally() :)
What you want is
return swal({
  ...
})
.finally(() => {
  // notify any event subscribers that the modal has closed
});

This can be done with
return swal({
  ...
})
.then(() => {
  // notify any event subscribers that the modal has closed
});

Swal2 for historical reasons doesn't use reject. Instead of it, there's the parameter in then() handler which is named isConfirm in most examples.
In other words then() handler will be called in both 'confirm' and 'cancel' cases, so it's practically the same as finally() in AngularJS.

The reason why is the execution sequence. I want to perform the cleanup actions after the entire promise chain has resolved.
Consider this:
success()
  .then(() => console.log('results 1'))
  .then(() => console.log('results 2'));

function success () {
  return swal({
     ...
  })
  .then(() => {
    console.log('cleanup');
  });
}

What would you get in the console when you closed the window?
cleanup
results 1
results 2

This is not ideal, because perhaps you want to execute something after your done with your promise chain, like for instance, disabling/setting a loading bar to 100%. (this is just some rough code, but outlines the problem with not having access to 'finally')
class LoggerService {
  constructor(
    private _loadingBar: LoadingBarService
  ) {}

  confirmProgress () {
    return swal({
       ...
    })
    .then(result => {

      // If we hit 'yes', then enable our loading bar as the first thing in the promise list
      if (result) this._loadingBar.enable();

      return Promise.resolve(result);
    });
    .finally(() => {

      // When the promise chain has finished, set our loading bar to 100%
      this._loadingBar.setPercent(100);

      setTimeout(() => this._loadingBar.disable(), 6000);
    });
  }
}

class UploadComponent {
  constructor(
    private _logger: LoggerService,
    private _uploadService: UploadService
  ) {}

  submit (images) {
    this._logger
      .confirmProgress('Do you want to upload both images?')
      .then(result => {
        return new Promise(resolve => {
          if (!result) return resolve(false);

          this._uploadService
            .upload(images[0])
            .then(() =>  {
              return this._uploadService.upload(images[1])
            })
            .then(resolve);
        });
      });
  }
}

Now you can summize that this can be done in a different way, by setting the progress bars in the UploadComponent. In this case, however the upload component's scope is tightly bound to only deal with the actual submission and upload of the two images. The UI and other services should be able to handle themselves without any necessary interaction between them.
Without a 'finally' method accessible in the initial return promise for swal, interaction like the one described above is impossible.

I just added an example of chaining modals: https://limonte.github.io/sweetalert2/#chaining-modals and you can see that finally() was used there.
All I need to make it works is adding <script src="https://cdn.jsdelivr.net/promise.prototype.finally/1.0.1/finally.js"></script>.

Oh nice. Didn't know that was all I needed to do to enable promise finally support for swal. Thanks!


Language specific documentation missing for module twin

https://docs.microsoft.com/en-us/azure/iot-hub/iot-hub-python-python-module-twin-getstarted#update-the-module-twin-using-python-device-sdk : second point needs to be updated as per the language (same problem in case of C and Node). Carries forward the documentation from .NET platform

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

ID: 6d727c2c-ba23-8369-b093-cbf0c68798ce
Version Independent ID: b969b244-da03-2ce0-f340-d9db9a8ef2f3
Content: Get started with Azure IoT Hub module identity and module twin (Python)
Content Source: articles/iot-hub/iot-hub-python-python-module-twin-getstarted.md
Service: iot-hub
GitHub Login: @chrissie926
Microsoft Alias: menchi

@aishanibhalla Thanks for the feedback and reporting this. I have assigned the issue to the content author to investigate further and update the document as appropriate.

@chrissie926 Please check this issue and update the documentation as appropriate.

@rezasherafat Please look into this feedback and update the doc.

#reassign JimacoMS3

@AshokPeddakotla-MSFT, @aishanibhalla -- thanks for your feedback, but I don't really understand what the ask is here. Can you please try to explain it in more detail?

@AshokPeddakotla-MSFT. @aishanibhalla -- just FYI: this is an old issue, and, as I mentioned, I lack clarity on what changes are being requested. I'll leave it open for a week or so to hear back. After that I'll go ahead and close it. Please update this thread if you can provide additional information/clarity.
Thanks,
Jimaco

@JimacoMS3 No further questions on this. Confirmed with @aishanibhalla and closing this issue.
Thank you!


Add Pep as a maintainer

What this PR does / why we need it:
Access to CNCF services is limited to project maintainers.
As a community liaison I'd need to interact with these services, hence this request
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

goveralls seems to be down. PR is good on travis.

/lgtm
/approve

/hold
/cc @fabiand
can you approve too?

/lgtm
/approve
For the reference - Pep will be helping us with community administrative
stuff.

/hold cancel


Fix of fix in FileBuilder.bat

What is this issue about?

It's the one "John" found.
Originally the FileBuilder.bat did not care about its location, so on Windows 7 it could have worked anywhere, including Program Files, having a space in its path
Then Windows 10 came and Microsoft ensured that batch files work slightly differently than in the past 30+ years, so now FileBuilder.bat automatically detects its location and uses full folder names to access its files. This works both on Windows 7 and 10, most of the time.
Then came Johannes and put QuickNII in Program Files, and Windows breaks parameters along spaces, unless if they are surrounded by quotation marks, which I knew about just simply forgot. So 4 quotation marks has to be added to FileBuilder.bat, which is a trivial thing, just in itself it does not justify re-releasing multiple gigabytes of files.

So, should we move this issue to done?  Do new QuickNII users find easily the solution ? if no, we should publish a new version even if it is a small thing, in my opinion.

@Tevemadar could you see if this issue is still present in the current released version?

I have a user still reporting having problems with Filebuilder. It seems that some user have an issue but not all users. How can we troubleshoot?


TS-4845: NULL dereference in url_sig

(cherry picked from commit 6b4100c1200e309e229c9fdd24004e1d4fb59c9a)
backport PR to 6.2.1

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1151/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/1044/ for details.


[BZ-1318532] ClassCastException on classes with the same FQCN after i…

…ncremental build

Unit test

I'm not 100% sure if this unit test simulates the reproduce step in BZ1318532 correctly. Please take this as a hint. Anyway, this reproduces the ClassCastException.

Test merged and fixed by https://github.com/droolsjbpm/drools/commit/464aad2fd


OAuth2 token endpoint should be able to handle simple form encoded requests

Actually no, the spec is pretty clear on this:

4.1.3.  Access Token Request
The client makes a request to the token endpoint by sending the
following parameters using the "application/x-www-form-urlencoded"
format per Appendix B with a character encoding of UTF-8 in the HTTP
request entity-body:

Broken tools (postman) should not introduce faulty behaviour in hydra. :D


PlayerRevive bug

when i use player-events_remastered with player revive, the death is instantaneous(skips the part when u are down)
i talked with the dev os player revive and he said to me to report here

On disconnect the player will be killed. The responsible code can be found here: https://github.com/CreativeMD/PlayerRevive/blob/1.21/src/main/java/team/creative/playerrevive/server/ReviveEventServer.java#L82


⚠ Bofrost.be has degraded performance

In ca19563, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1352 ms

Resolved: Bofrost.be performance has improved in 9226b45.


Change toolbar color in my schedule page

My schedule page is very similar to All sessions page. It make user confused.
So I want to change toolbar and statusbar color.

Any thoughts?

I could change toolbar color, but I could not change statusbar color with transparecy...


octavePackages.image-acquisition: 0.2.2 -> 0.2.6

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for octavePackages.image-acquisition is: Functions to capture images from connected devices
meta.homepage for octavePackages.image-acquisition is: https://octave.sourceforge.io/image-acquisition/index.html
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.2.6 with grep in /nix/store/5anl78inx8f91mxhwbaf91ipa2dg8cpy-octave-9.2.0-image-acquisition-0.2.6
found 0.2.6 in filename of file in /nix/store/5anl78inx8f91mxhwbaf91ipa2dg8cpy-octave-9.2.0-image-acquisition-0.2.6

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

octavePackages.image-acquisition

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/5anl78inx8f91mxhwbaf91ipa2dg8cpy-octave-9.2.0-image-acquisition-0.2.6 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/54719fedf20eb62ae17446470bb8deb5d96fec76#octavePackages.image-acquisition

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/5anl78inx8f91mxhwbaf91ipa2dg8cpy-octave-9.2.0-image-acquisition-0.2.6
ls -la /nix/store/5anl78inx8f91mxhwbaf91ipa2dg8cpy-octave-9.2.0-image-acquisition-0.2.6/bin

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

1 package marked as broken and skipped:

octavePackages.image-acquisition

Maintainer pings
cc @KarlJoad for testing.

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

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

Closed for #338851, to add additional fixes.


Fixes

Fix the build on kernel older than 3.16
Remove unecessary check
Fix a wrong variable type

More details can be found in which commit's message

Please fix the warnings:
src/lib/common/sol-missing.h:80:6: warning: "HAVE_DECL_IFLA_INET6_ADDR_GEN_MODE" is not defined [-Wundef]

Ohhh, i saw the error in that commit. Sorry for the noise, i'll remove it.

barbieri, about the if !
Unlike the other AC_CHECK_*S' macros, when a symbol is not declared, HAVE_DECL_symbol is defined to 0' instead of leaving HAVE_DECL_symbol undeclared. When you are sure that the check was performed, use HAVE_DECL_symbol just like any other result of Autoconf:
#if !HAVE_DECL_SYMBOL
extern char *symbol;
#endif

closing ... fixes on
https://github.com/solettaproject/soletta/pull/94


fix: Remove deprecated Cargo.toml feature flag

The cargo feature flag workspace-inheritance stabilized post Rust v1.64.0 and is no longer required.
This would remove warnings when compiling the marginfi programs with anchor.

Warnings
warning: /home/ubuntu/marginfi-v2/programs/marginfi/Cargo.toml: the cargo feature `workspace-inheritance` has been stabilized in the 1.64 release and is no longer necessary to be listed in the manifest
  See https://doc.rust-lang.org/nightly/cargo/reference/unstable.html#workspace-inheritance for more information about using this feature.
warning: /home/ubuntu/marginfi-v2/observability/indexer/Cargo.toml: the cargo feature `workspace-inheritance` has been stabilized in the 1.64 release and is no longer necessary to be listed in the manifest
  See https://doc.rust-lang.org/nightly/cargo/reference/unstable.html#workspace-inheritance for more information about using this feature.
warning: /home/ubuntu/marginfi-v2/programs/liquidity-incentive-program/Cargo.toml: the cargo feature `workspace-inheritance` has been stabilized in the 1.64 release and is no longer necessary to be listed in the manifest
  See https://doc.rust-lang.org/nightly/cargo/reference/unstable.html#workspace-inheritance for more information about using this feature.
warning: /home/ubuntu/marginfi-v2/test-utils/Cargo.toml: the cargo feature `workspace-inheritance` has been stabilized in the 1.64 release and is no longer necessary to be listed in the manifest
  See https://doc.rust-lang.org/nightly/cargo/reference/unstable.html#workspace-inheritance for more information about using this feature.
warning: /home/ubuntu/marginfi-v2/clients/rust/marginfi-cli/Cargo.toml: the cargo feature `workspace-inheritance` has been stabilized in the 1.64 release and is no longer necessary to be listed in the manifest
  See https://doc.rust-lang.org/nightly/cargo/reference/unstable.html#workspace-inheritance for more information about using this feature.
warning: /home/ubuntu/marginfi-v2/tools/llama-snapshot-tool/Cargo.toml: the cargo feature `workspace-inheritance` has been stabilized in the 1.64 release and is no longer necessary to be listed in the manifest
  See https://doc.rust-lang.org/nightly/cargo/reference/unstable.html#workspace-inheritance for more information about using this feature.

I see we use Rust v1.66 in CI -- if we haven't updated we probably should not merge this.
Closing for now.


UnboundLocalError: local variable 'u' referenced before assignment

View details in Rollbar: https://rollbar.com/Zooniverse/Zooniverse_Aggregation/items/8/
Traceback (most recent call last):
  File "./aggregation_api.py", line 2366, in <module>
    project.__aggregate__()#workflows=[84],subject_set=[494900])#,subject_set=[495225])#subject_set=[460208, 460210, 460212, 460214, 460216])
  File "./aggregation_api.py", line 382, in __aggregate__
    clustering_aggregations = self.__cluster__(raw_markings,image_dimensions)
  File "./aggregation_api.py", line 556, in __cluster__
    shape_aggregation = self.cluster_algs[shape].__aggregate__(raw_markings,image_dimensions)
  File "/home/greg/github/reduction/engine/clustering.py", line 118, in __aggregate__
    cluster_results,time_to_cluster = self.__cluster__(markings,users,tools,reduced_markings,image_dimensions[subject_id])
  File "/home/greg/github/reduction/engine/blob_clustering.py", line 354, in __cluster__
    aggregate_polygons,aggregate_stats,total_incorrect_area = quad_root.__aggregate__(image_area)
  File "/home/greg/github/reduction/engine/blob_clustering.py", line 138, in __aggregate__
    c_polygons,c_stats,incorrect_area,polygons_by_user_density = c.__aggregate__(total_area)
  File "/home/greg/github/reduction/engine/blob_clustering.py", line 138, in __aggregate__
    c_polygons,c_stats,incorrect_area,polygons_by_user_density = c.__aggregate__(total_area)
  File "/home/greg/github/reduction/engine/blob_clustering.py", line 138, in __aggregate__
    c_polygons,c_stats,incorrect_area,polygons_by_user_density = c.__aggregate__(total_area)
  File "/home/greg/github/reduction/engine/blob_clustering.py", line 178, in __aggregate__
    print u
UnboundLocalError: local variable 'u' referenced before assignment

ignore - misuse of rollbar (Greg's fault)


Generated worker blob includes main dependency graph

The sample main/worker setup from the README includes webworkify in the worker blob, even though it's not required from the worker.
In the application I'm currently working on this results in a blob of approximately 1.1MB, for the following worker, due to the sheer size of the rest of the bundle:
module.exports = function(self) {
  console.log('hello world!');
};

The culprit seems to be usage of Object.keys(sources) which will include all sources up to that point.
Is this by design?

I had this problem and ended up writing https://github.com/kumavis/meowserify
however, its not very good, could be improved greatly.

@timkurvers ever come up with a clever solution here?

@kumavis Not really unfortunately. I ended up building a separate worker-bundle using browserify.

Check out #30 for a solution.


Faster, More Stable Kernel Expectations

Analytic kernel expectations (wrt Gaussian distributions) for the RBF kernel, Linear kernel and Sum kernels (when composed of RBF and Linear kernels) have been reimplemented in a more efficient and numerically stable way by better exploiting Cholesky factorisations and simplifications as well as avoiding superfluous transpositions and dimension augmentations.
Main changes:

new implementations of  RBF and Linear kernel expectations as well as the RBF-Linear cross kernel expectation
more efficient mean function expectation handling
Streamline quadrature expectations and unified the expectations files
clearer distinction between quadrature_expectation and expectation which makes @quadrature_fallback superfluous
"Private" _expectation methods are kept private
Improve handling of Sum kernels (with 1 bug fixed on when cross expectations simplify)
switch to assert_allclose instead of the less advisable assert_almost_equal
switch around (feat, kern) to (kern, feat) in expectations for consistency
improve test coverage and reduced error tolerance
add, correct and improve docstrings
make covariance name consistent for all Gaussian probability_distributions: p.cov
add shape inference for guassian covariances

Codecov Report

Merging #646 into master will increase coverage by 0.06%.
The diff coverage is 97.71%.

@@            Coverage Diff             @@
##           master     #646      +/-   ##
==========================================
+ Coverage   98.12%   98.18%   +0.06%
==========================================
  Files          47       46       -1
  Lines        3724     3738      +14
==========================================
+ Hits         3654     3670      +16
+ Misses         70       68       -2

Impacted Files
Coverage Δ

gpflow/__init__.py
100% <ø> (ø)
:arrow_up:

gpflow/models/gplvm.py
97.54% <100%> (ø)
:arrow_up:

gpflow/conditionals.py
100% <100%> (ø)
:arrow_up:

gpflow/probability_distributions.py
100% <100%> (ø)
:arrow_up:

gpflow/expectations.py
97.65% <97.65%> (+0.38%)
:arrow_up:

gpflow/mean_functions.py
92.85% <0%> (-4.77%)
:arrow_down:

gpflow/kernels.py
99.4% <0%> (+0.88%)
:arrow_up:

Continue to review full report at Codecov.

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


fix wordRotationQuatenion bug

Please check if the PR fulfills these requirements

[ ] The commit message follows our guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
What is the current behavior? (You can also link to an open issue here)
What is the new behavior (if this is a feature change)?
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
Other information:

Codecov Report

Merging #703 (83346d3) into dev/0.7 (b211d66) will increase coverage by 0.15%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           dev/0.7     #703      +/-   ##
===========================================
+ Coverage    44.45%   44.60%   +0.15%
===========================================
  Files          408      408
  Lines        21731    21662      -69
  Branches      2972     2957      -15
===========================================
+ Hits          9660     9662       +2
+ Misses       12071    12000      -71

Impacted Files
Coverage Δ

packages/core/src/Transform.ts
86.25% <100.00%> (+0.76%)
:arrow_up:

packages/draco/src/decoder.ts
0.00% <0.00%> (-0.92%)
:arrow_down:

packages/loader/src/scene-loader/Oasis.ts
0.00% <0.00%> (ø)

...ackages/loader/src/scene-loader/ResourceManager.ts
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


Revert "Api fuzzer extensions to support simulating traffic congestion"

Reverts grpc/grpc#27820
The PR breaks the windows build on master.
E.g.:
https://source.cloud.google.com/results/invocations/6092d075-1d0c-4587-9be1-3e83ed550b77/targets/github%2Fgrpc%2Ftoplevel_run_tests_invocations%2Frun_tests_c_windows_dbg_native/tests
The failure has been correctly reported by the presubmit tests on that PR already, but it seems that you didn't check the test results and merged anyway.  Please be more careful next time and don't merge unless you've mapped failures on the PR to pre-existing flakiness issues. (go/grpc-merge-rules).

windows portability test has also been broken by the original PR: https://source.cloud.google.com/results/invocations/e8534864-8b1f-4e19-bc48-5dbd38e6e160/targets/github%2Fgrpc%2Ftoplevel_run_tests_invocations%2Frun_tests_c_windows_dbg_native_x64_default/tests

How does this differ from #27973?


feedtrng: give an option to choose the source device

e.g., /dev/cuaU1 instead of now hardcoded /dev/cuaU0

Done. See https://github.com/jj1bdx/freebsd-dev-trng/compare/0.1.0...0.1.1


feat: go-ipfs 0.8.0 + docs for repeated flags

Text still WIP, but wanted to start before I forgot. This explains how repeated flags are used in the ipfs pin remote service commands.

cc @jessicaschilling

cc @jessicaschilling

@aschmahmann should we merge this and regenerate https://docs.ipfs.io/reference/http/api/  ?
go-ipfs 0.8.0 already shipped :D

@lidel I assume merging this was the right call? (or did we need to wait for a release)


Add minority and parent question to student application

Reporting to FEDs. There are no ties to assessments. Robin following up on "prefer not to answer".
Acceptance criteria:

[ ] Add a Y/N
[ ] Add if Y, add which minority group student self identifies
[ ] Determine correct question for "how many parents do you live with"

Additional context
-Parent question to confirm:
As an applicant
I want to provide my parent(s) marital status So That the correct parent information is requested*
Additional Context

This can replace the current question "How many parents do you live with?"

Acceptance Criteria

[ ] When parent information is required, the following application question is included:
**My parent(s) marital status is:

Single

Married/Remarried

Common-Law

Seperated/Divorced/Widowed**

[ ] Given (Single OR Seperated/Divorced/Widowed), When (application is submitted), Then (1 Parent Information Request is generated)

[ ] Given (Married/Remarried OR Common-Law), When (application is submitted), Then (2 Parent Information Request are generated)

Awaiting confirmation of question and LOV from CSFA - will bring forward for enhancement once confirmed.

Business confirmed on May 13 no decision has been made on this issue. Given it will not affect assessment or award, it can be removed from MVP scope list.


Issue 49 Fix bug when typing in address in payment screen when in Solana Mode

Closes #49

This PR has become stale. Closing for now


Prioritize custom routes over the defaults

[x] Changes have been thoroughly tested to not break existing functionality.
[ ] New functionality has been documented or existing documentation has been updated to reflect changes.
[ ] Visual changes are explained in the PR description using a screenshot/recording of before and after.

Move down all custom routes after the default routes to enable route overriding.

Isn't this the reverse of #7810 which fixed an issue?

Please investigate that issue and provide more information if you would like me to merge this

Please investigate that issue and provide more information if you would like me to merge this

I've checked and tested the problem of #7809 and now I can see the dilemmatic situations since the "global" routes (login, logout, profile, etc.) are placed at the same level with the tenant slug.
However, the fix #7810 came with a drawback that avoided us to override any default routes. The best options I can think so far is stick to the current version, close this PR, and enable customizations for all global routes.

Please suggest how you want to achieve customisation of global routes, as you mention


Bump to 5.1.0-0

Pull Request Test Coverage Report for Build 998

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

Totals

Change from base Build 989:
0.0%

Covered Lines:
3594

Relevant Lines:
4924

💛  - Coveralls

Pull Request Test Coverage Report for Build 998

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

Totals

Change from base Build 989:
0.0%

Covered Lines:
3594

Relevant Lines:
4924

💛  - Coveralls


Add buton that will open wordle variation with answers already inputted

Need to research if this even possible without XSS on worldle side

It is impossible


error when trying to install geopandas in windows

Environment Information

geospatial version: 0.9
Python version: 3.9
Operating System: Windows 10 (enterprise)

Description
I followed the necessary steps for installing the geospatial package with combination of conda - mamba. When reaching the step for installing geopandas, I get an error message "could not solve for environment specs, the following packages are incompatible" (please see picture below).
The same issue was present also when to install the package in my personal computer running windows 11.
I also tried installing installing geopandas using only conda - without installing mamba.
What I Did
I run the following commands in git-bash, where in the last one I got the attached traceback message:
conda create -n geo python=3.9
conda activate geo
conda install -c conda-forge mamba
mamba install geopandas

I have the same problem

Sorry for the delay. Please try installing geospandas from the conda-forge channel. I works fine on my Windows 11.
mamba install geopandas -c conda-forge


Strange IAM permission

I'm trying to understand why does this plugin need following permissions?
- ec2:CreateNetworkInterface
- ec2:DescribeNetworkInterfaces
- ec2:DetachNetworkInterface
- ec2:DeleteNetworkInterface

Those are needed if the lambda is deployed to a VPC.
You can read more here: https://docs.aws.amazon.com/lambda/latest/dg/vpc.html

The Lambda function execution role must have permissions to create, describe and delete ENIs. AWS Lambda provides a permissions policy, AWSLambdaVPCAccessExecutionRole, with permissions for the necessary EC2 actions (ec2:CreateNetworkInterface, ec2:DescribeNetworkInterfaces, and ec2:DeleteNetworkInterface) that you can use when creating a role. You can review the policy in the IAM console. Do not delete this role immediately after your Lambda function execution. There is a delay between the time your Lambda function executes and ENI deletion. If you do delete the role immediately after function execution, you are responsible for deleting the ENIs.

Thanks @juanjoDiaz  for the clarification.
If I'm not deploying the Lambda to VPC, would omitting those IAM permission still work?

It should work AFAIK.

Cool, thanks for the clarification.


fix(e2e): attempt to stabilise TestNativeIntegrations

To recap what's being done in this fix:

Relax timeout and specify Xmx for Quarkus native build for native_test.go
Insert warm up test step before the native build test so that the time for downloading jars can be skipped for native build testing (native build itself would take long time especially with Camel Quarkus 2.11)
Split install-it CI workflow into two jobs: install-it and native-it, so that native-it can be retried separately and also any leftovers in install-it test suite won't consume resources for native-it.

Release Note
NONE

This really doesn't solve the failure. I identified a different cause so am creating another fix.

--maven-cli-option -Dquarkus.native.native-image-xmx=5g seems to manage to make TestNativeIntegrations pass. Let's see how it goes.

@phantomjinx Can you please review this?  I split install-it into two jobs.


[Twinkle-0001] Create POC for Swing GUI Automation Testing #3

More about integration between Java and Cucumber - Cucumber JVM

More about Gherkin syntax - Gherkin Language

Before try to implement UIAutomation logic read this - Windows Accessibility features

Example of integration Java with Windows API for working with UI - ui-automation


expand utility emissions base data to other regions

Currently we're using the eGRID database from the USA.  We'd like to expand utility emissions data to other regions, such as Europe, UK, India, etc.  We need help identifying trusted data sets for calculating emissions in those regions.

@udosson What do you think of using these data sources for Europe:

https://www.eea.europa.eu/data-and-maps/data/approximated-estimates-for-the-share-3 has a spreadsheet which shows the percentage of energy coming from renewables for each country.  Note that 0.55 means 55%
https://www.eea.europa.eu/data-and-maps/daviz/co2-emission-intensity-5 has a CSV file of the carbon intensity of each country's electricity generation measured in g CO2/KWH

Funny, huh - in Europe you measure in grams of CO2, here we measure in tons!


Non random UUID's

track; https://github.com/gobuffalo/uuid/issues/1

track; https://github.com/gobuffalo/uuid/issues/1

This package is not vulnerrable to this bug because we use rand.Read(uuid[:])[1]
which internally calls io.ReadFull(Reader, b) [2]

https://github.com/komuw/yuyuid/blob/83038e1813dd8d9656f9d53db764ddb5c3dcd00b/yuyuid.go#L73
https://github.com/golang/go/blob/bf86aec25972f3a100c3aa58a6abcbcc35bdea49/src/crypto/rand/rand.go#L22-L23


Step sample Of the CLI option is incomplete.

For the CLI option, the step 2 sample is incorrect and will not work as expected.
The proper sample for this to work should be this one:
az monitor app-insights component linked-storage link --resource-group "XXXXXXXXX" --app "XXXXXXXXXXXX" --storage-account "XXXXXXXXXXX" --subscription "XXXXXXXXXXXXXX"
The missing part is the subscription id reference at the end. After testing with the customers that's the only wat for this to work.
Please update the document for future references.
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: f2faf9c8-54aa-aff6-12ea-c073e1c30a23
Version Independent ID: 8e48a797-9036-e3a4-772f-07b45a246e83
Content: Configure BYOS for Profiler and Snapshot Debugger - Azure Monitor
Content Source: articles/azure-monitor/profiler/profiler-bring-your-own-storage.md
Service: azure-monitor
Sub-service: profiler
GitHub Login: @hhunter-ms
Microsoft Alias: hannahhunter

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

@mauriciorlx, thank you for sharing the feedback.
I tried using the cmd as available in the document and it worked as expected. Here is the cmd that I used:
az monitor app-insights component linked-storage link --resource-group "AppInsights_Resource_Group" --app "AppInsights_Name" --storage-account "StorageAccount_Name"
I am not sure why you might have seen an issue when not using the -subscription parameter, because that is optional. For more details see az monitor app-insights component linked-storage link
One of the scenarios that I could think of (however, I could not test it as I have access currently to only one subscription_), is when you have logged into multiple subscription using az login cmd. In that case, to avoid any conflict the subscription id might be required.
Please let me know if you have any questions.

@mauriciorlx, We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply and reopen the issue. We will gladly continue the discussion.


Widget Size Fix

It could fix #1981. I could not reproduce it with my phone, but the minValues did not match google's recommendation.
https://developer.android.com/guide/practices/ui_guidelines/widget_design.html#anatomy

This makes sense to me and explains why it's only sporadically reproducible. Up to @cketti whether we merge it into 5.2-MAINT but I'm happy to see it go in.


LeftMenu doesnot fit to the explicit width

Hi ,
I am a new bee to swift and i have integrated AMSlideMenu to the project. I have used this library for a couple of Obj projects. Ok right now what am trying to find out is adjust the width of the left menu. I have overridden the width on a Mainviewcontroller.swift which is a subclass of AMSlideMenuMainViewController. The left menu is not framed to the width i have asked for. When i slide or open the left menu ,it looks like left menu screen is taking the full screen . I have only left menu in this project no right menu
import UIKit
class MainMenuViewController: AMSlideMenuMainViewController {
override func viewDidLoad() {
    super.viewDidLoad()

    // Do any additional setup after loading the view.
}

override func didReceiveMemoryWarning() {
    super.didReceiveMemoryWarning()
    // Dispose of any resources that can be recreated.
}

override func segueIdentifierForIndexPathInLeftMenu(indexPath: NSIndexPath!) -> String! {
    switch (indexPath.row) {

    case 0 :
        return "showHomeScreen1"

    case 1 :

        return "showHomeScreen2"

    default :

        return "showHomeScreen1"
    }
}

override func initialIndexPathForLeftMenu() -> NSIndexPath! {
    return NSIndexPath(forRow: 0, inSection: 0);
}

 override func leftMenuWidth() -> CGFloat {
    return 160
}

}

I haven't learned swift language yet, so I can't help you here, sorry.
You can ask those king of questions in http://stackoverflow.com


Type check don't work with analyzer 5.8.0

import 'package:analyzer/dart/ast/ast.dart';
import 'package:analyzer/src/dart/ast/ast.dart';

external NodeListImpl<ExpressionImpl> get arguments;

void main() {
  // Expression id = arguments.first; // works
  var id = arguments.first; // not, but works with < 5.8.0 version

  String? name;

  if (id is Identifier) {
    if (id is PrefixedIdentifier) {
      // The getter 'identifier' isn't defined for the type 'ExpressionImpl'.
      name = id.identifier.name;
    } else {
      // The getter 'name' isn't defined for the type 'ExpressionImpl'.
      name = id.name;
    }
  }

  print(name);
}

Don't use XyzImpl.
The reason you get this is that type promotion using x is T2 (where x has type T1) happens only when T2 is a subtype of T1. But Identifier is not a subtype of ExpressionImpl, it is a subtype of Expression.


Bidder's Params For Pollux Adapter

Related: https://github.com/prebid/Prebid.js/pull/1431

Thanks @polluxnetwork !


Odoo Support

Could you consider Odoo chart support? I have tried a custom image with arm support but some path doesn't match with the official bitnami chart values.
Thanks for amazing job btw.

@joepa37 I looked at odoo, but found that it wasn't infrastructure related software. I don't think it's the software I'm using or interested in in the near future. It's hard to use the time for this. It is also mentioned in the README, but supporting all bitnami's charts is not currently the goal of this repository. Also, even if I write those images, few people will use them.
I recommend making a chart that uses the official image or modifying bitnami's chart to use the official image.


Reduce the API surface of bevy_ecs::Query

Query::new and QuerySet::new functions were marked and documented as unsafe in # 798, which essentially closes #796. Before closing #796, I wanted to see what would happen if these were marked as pub(crate). Everything in bevy's repo continues to build since the usages are only in bevy_ecs. As mentioned in the issue, it is unlikely a user will create their own IntoSystem impl.
In addition, Query::iter_unsafe, Query::get_unsafe and Query::get_component_unsafe are not used in any of bevy's crates. I imagine the latter methods were added to mirror some of World's methods like World::query_unchecked, but I believe there is some value in reducing the API surface of bevy_ecs::Query for learnability and leaving the unsafe interfaces to direct interactions with World.
These changes are split into two commits. I would understand if only the first or none of these changes are desired, but I wanted to open this to float the idea.

Cool I support making the constructors pub(crate) (although its worth pointing out that this prevents people from building their own IntoSystem implementations that use Query/QuerySet).
I think I want to keep the unsafe methods around. There will be some patterns (like nested conflicting query iterations) that are safe in practice, but that can't be statically enforced. I don't want to close off the option to drop down to unsafe code if a consumer is willing to take the risk / self validate.

Ah yea, I respect the motivations of not trying to prevent something potentially useful for a user.
Before I opened this, I wasn't able to create a motivating example in a query system where safe methods did not already handle it and where the unsafe methods were used correctly.

like nested conflicting query iterations

Do you have an example in mind where this would be okay to do? The following tests a few variations:
use bevy_ecs::{Schedule, prelude::*};

#[derive(Debug)]
struct Foo(pub i32);

fn main() {
    let mut world = World::default();
    let mut resources = Resources::default();

    world.spawn((Foo(42),));

    let mut schedule = Schedule::default();
    schedule.add_stage("update");

    schedule.add_system_to_stage("update", s0.system());
    schedule.add_system_to_stage("update", s1.system());
    schedule.add_system_to_stage("update", s2.system());

    schedule.initialize(&mut world, &mut resources);
    schedule.run(&mut world, &mut resources);
}

// This is fine with safe interface to read only components.
// It will build and print `Foo(42)`.
fn s0(query: Query<&Foo>) {
    for x in query.iter() {
        for _y in query.iter() {
            println!("{:?}", x);
        }
    }
}

// This will build, but will thankfully panic at runtime anyway.
// "&mut test::Foo conflicts with the component access [test::Foo] in this prior query: &mut test::Foo'"
fn s1(q0: Query<&mut Foo>, q1: Query<&mut Foo>) {
    unsafe {
        for x in q0.iter_unsafe() {
            for _y in q1.iter_unsafe() {
                println!("{:?}", x);
            }
        }
    }
}

// This will build and run, but can acquire two mutable borrows
// to the same component and thus is incorrect usage.
// `cargo miri run` will report undefined behavior.
fn s2(query: Query<&mut Foo>) {
    unsafe {
        for x in query.iter_unsafe() {
            for _y in query.iter_unsafe() {
                println!("{:?}", x);
            }
        }
    }
}

Something like this:
fn system(mut queries: QuerySet<(Query<&mut A>, Query<(&mut A, &B)>)>) {
  for x in queries.q0().iter_unsafe() {
     if some_criteria_that_ensures_no_conflict {
         if let Ok((a, b)) = queries.q1().get_unsafe() {
         }
     }
  }
}

I imagine a common criteria would be checking IDs beforehand in some form.
fn system(q0: Query<With<A, Entity>>, q1: Query<&mut A>) {
    for e0 in q0.iter() {
        for e1 in q0.iter().filter(|&e1| e1 != e0) {
            let entities = unsafe { (q1.get_unsafe(e0), q1.get_unsafe(e1)) };
            if let (Ok(a0), Ok(a1)) = entities {
                print!("{:?}: {:?}", e0, a0);
                println!(", {:?}: {:?}", e1, a1);
            }
        }
    }
}

Anyway, thanks for patiently hearing me out and sharing the motivations.
The commit that removed the methods has now been reverted ^^


chore: port order.basic again

There seem to have been a number of changes since this was ported, and this takes the opportunity to restore the #aligns.

bors d+

bors r+


OpenAPI file is not valid for the OpenAPI specification

I'm trying to import this file: https://github.com/microsoftgraph/msgraph-sdk-powershell/blob/dev/openApiDocs/v1.0/Groups.yml
into Azure API Management Service

One or more fields contain incorrect values:
Parsing error(s): Cannot have multiple operations with the same operationId: group_DeleteAcceptedSenderGraphBPreRef
Parsing error(s): The input OpenAPI file is not valid for the OpenAPI specification https://github.com/OAI/OpenAPI-Specification/blob/master/versions/3.0.1.md (schema https://github.com/OAI/OpenAPI-Specification/blob/master/schemas/v3.0/schema.yaml).

It looks like the OpenAPI docs in this repository are not compliant to the openapi specification.

Hi @Znote could you please try with the latest open API docs from the repo. If it was not valid, Microsoft Graph  PowerShell SDK would be missing the Groups module.


Add a function to set window size limit

In Window module, it will be more convenient if add a function to set min/max size of window.
Windows provide a message named WM_GETMINMAXINFO, and we can use lParam to simply limit a window's size.
Just use two sf::Vector2us to store the setted minimum and maximum, and post them when get WM_GETMINMAXINFO.
I've only considered this problems about Windows. I'm sorry if it will cause any problem of crossing platform.

I assume this will then not even allow change the window size to a smaller or larger size?
So you wouldn't need to handle the resize event and re-set the window size when it drops below or goes above a certain value, correct?

It's close.
More clearly, I don't want that a window drops below or goes above the setted value while user is dragging.
I scanned the code in WindowImplWin32.cpp, and saw
// The system request the min/max window size and position
case WM_GETMINMAXINFO:
{
    // We override the returned information to remove the default limit
    // (the OS doesn't allow windows bigger than the desktop by default)
    MINMAXINFO* info = reinterpret_cast<MINMAXINFO*>(lParam);
    info->ptMaxTrackSize.x = 50000;
    info->ptMaxTrackSize.y = 50000;
    break;
}

But Windows 11 doesn't stop windows going larger than the desktop. This leads to a real maximum of 50000 * 50000 pixels.
I'd like to change this to
MINMAXINFO* info = reinterpret_cast<MINMAXINFO*>(lParam);
if (m_windowMaxTrackSize.x > 0)
    info->ptMaxTrackSize.x = m_ windowMaxTrackSize.x;
if (m_windowMaxTrackSize.y > 0)
    info->ptMaxTrackSize.y = m_ windowMaxTrackSize.y;
if (m_windowMinTrackSize.x > 0)
    info->ptMinTrackSize.x = m_ windowMinTrackSize.x;
if (m_windowMinTrackSize.y > 0)
    info->ptMinTrackSize.y = m_ windowMinTrackSize.y;

and m_windowMinTrackSize and m_windowMaxTrackSize are two sf::Vector2us which can be changed by a function in class sf::WindowImpl.


Response: Internal server error

[MasterHamsterKombatBot] [2024-09-13 17:13:42] [INFO] [Account S]: 🛸 Getting basic account data.
[MasterHamsterKombatBot] [2024-09-13 17:13:45] [ERROR] [Account S]: ✖ Status code is not 200
[MasterHamsterKombatBot] [2024-09-13 17:13:45] [ERROR] [Account S]: ✖ Response: Internal server error

the same.

No worries after some minz we all get banned. Lol

no answer?

unknown errors based on bearer token as I understand because requests to api with certain tokens cause an error, without token api says wrong token, performance is gradually restored, but what is the exact cause is unclear.

Mine is fixed. No problems. The bot is running.


Create Magic8Ball.py

Can you please edit the code to be compatible with python 3

it should be ok now


Module Causes huge lag in Foundry

I've been waiting for a module like this for a long time, so I'm super happy you made it! Unfortunately, I've been noticing when I try to use it, it seems to cause Foundry to either break entirely, or at least suffer from huge lag. So far the only way I've been able to fix it is by closing out of Foundry and restarting it from scratch. It doesn't seem to cause the lag right away, only when I actually hover my mouse over a video file.
My best guess is that some of my images are quite large, like, 150 MB files in some cases, so maybe that is behind it. I'm not sure if that's something that can be fixed if so, but I figured I'd point it out just in case.

Hi GambetTV, thanks for the issue!
I did notice some performance issues when it came to folders with a lot of video files, so I'll probably revisit this sometime soon and see if I can improve the performance somehow, probably by deferring the loading of the video a bit. I'll look into it and see what I can do!

Hi @GambetTV,
I deferred the video loading in version 1.0.1. While this is in general less convenient (as you don't get proper previews anymore and it will take a moment to load the file when hovering), I hope that this will improve performance issues, especially with list with a lot of and/or large files.
Please let me know if this issue is resolved for you. Thanks!


Moved eightbit graph trimming to before output_nodes definition

Currently fails when output_nodes have inputs that are removed by graph trimming. Example below:
import tensorflow as tf
from quantize_graph import GraphRewriter
from tensorflow.python.framework.graph_util import \
    convert_variables_to_constants

g = tf.Graph()
with g.as_default():
    x = tf.placeholder(shape=(2,), dtype=tf.float32, name='input_node')
    y = tf.get_variable(name='quantize_target', shape=(2,), dtype=tf.float32,
                        initializer=tf.random_normal_initializer(0., 1.))
    z = tf.add(x, y, name='output_node')
    init = tf.initialize_all_variables()

with tf.Session(graph=g) as sess:
    sess.run(init)
    frozen_def = convert_variables_to_constants(
        sess, g.as_graph_def(), ['output_node'])

rewriter = GraphRewriter(frozen_def, 'eightbit')
output_def = rewriter.rewrite(['output_node'])

Traceback:
Traceback (most recent call last):
  File "quantize.py", line 22, in <module>
    output_def = rewriter.rewrite(['output_node'])
  File "<path>/quantize_graph.py", line 333, in rewrite
    self.eightbitize_nodes_recursively(output_node)
  File "<path>/quantize_graph.py", line 455, in eightbitize_nodes_recursively
    input_node = self.nodes_map[input_node_name]
KeyError: u'quantize_target/read'

Can one of the admins verify this patch?

I signed it!
On 1 Aug 2016 10:36 AM, "googlebot"<EMAIL_ADDRESS>wrote:

Thanks for your pull request. It looks like this may be your first
contribution to a Google open source project. Before we can look at your
pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/
https://cla.developers.google.com/ to sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll
verify. Thanks.

If you've already signed a CLA, it's possible we don't have your
GitHub username or you're using a different email address. Check your
existing CLA data https://cla.developers.google.com/clas and verify
that your email is set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.
If you signed the CLA as a corporation, please let us know the
company's name.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/3594#issuecomment-236467020,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAoOq5WXZxDd5gs0f0EezJvwKlrVBA9Tks5qbT-VgaJpZM4JZKRU
.

Assigning @petewarden for reviewing.

Just realized this will probably result in issues when output nodes themselves are trimmed. Will ponder further...

@jackd should we wait for you to ponder the issue before we move forward with review / testing?

@danmane Sorry, busy time at work. Changed the interface of GraphRewriter.remove_unneeded_nodes to allow for an optional whitelist, and the corresponding call in eightbitizing code to node remove any of the output nodes. As a result, identity outputs are not pruned from the graph. In certain circumstances this is slightly sub-optimal: for example, it would be possible to remove an identity mapping resulting in an output node by removing the input from the graph and relinking dependencies to the output... but I wasn't able to resolve control edge tests with this approach. In these rare cases, I find it difficult to imagine a single extra identity call should be too strenuous for a system.

@petewarden could you please review this?

@jackd Sorry for the slow process here :( Can you please fix merge conflicts, and then I will make sure that someone takes a look promptly.

Pinging @jackd -- can you fix the merge conflicts?

Closing due to inactivity. Please merge the conflicts and re-open if you want us to take another look.


Indexing Engine CP model: Update rules/goals, add JSON examples

FWIW, I feel like some of the components here could probably be improved (in terms of duplication between this and other parts of the website and app), but this is good enough for now, so will go ahead with merging it.


Documentation

Third class documentation written, code neither working in Collab, in windows, ubuntu, lots of paths issue, no dockerisation nothing.

Thank you for raising these issues. To better assist you, could you please provide the following information:

Specific code snippets or error messages you encountered, which will help us identify the issue more quickly.
Your runtime environment configuration on Colab, Windows, or Ubuntu, such as Python version and library versions.
A more detailed description of the path issues you mentioned, such as which paths are causing problems.

Regarding the lack of Docker file, we will prioritize this and update the documentation accordingly.
Thank you again for your feedback.  If you have any more suggestions or issues, please feel free to let us know!


Add Providers to the list view of ClusterOutputs

rancher/dashboard#1568

Added the namespace column to both types. Visually there's no difference however.


[Bug] Right-click object menu displays partially off-screen in Obsidian on Ipad

Details
When "right-clicking" or tapping-and-holding to open the object menu, the object menu displays partially off-screen, and cannot be dragged or scrolled to view cutoff items.
Workaround
Move the canvas so that the intended object can be selected from a higher part of the screen. Move back to continue previous work.
This works, but is pretty annoying for workflow ergonomics.
Environment:

iOS 15.1 (2020 iPad Pro 12.9")
Obsidian v1.0.5
Excalidraw v1.5.16

Video
https://user-images.githubusercontent.com/89762294/149054776-b1c3e001-8362-4c0e-9ce9-315571419c76.mp4
Details

Create an object, right-click and delete it
Create a new object, delete from menu is off-screen. Tried scrolling to access, but menu display is static

This is not an Obsidian specific bug, but seems to be related to how the context menu handles screen offsets.

When there is no offset, excalidraw handles context menu positioning well:

Hi @zsviczian! Are you confirming that this is a bug with excalidraw, or that it's instead something I should be adjusting on my machine? Thanks!

From what I can see, this is an excalidraw bug in the core Excalidraw package, not Obsidian specific.


doc: Update supported expressions

Which issue does this PR close?

Closes #.
Rationale for this change

We added some more supported expressions. We should update the document.
What changes are included in this PR?

How are these changes tested?

cc @parthchandra @huaxingao

LGTM

cc @sunchao

Is there a document listing unsupported expressions that individuals could potentially work on if they're interested? Thank you.

Is there a document listing unsupported expressions that individuals could potentially work on if they're interested? Thank you.

We don't have such document. Spark expressions are too many so the unsupported list might be very long.
We can probably list some frequently used expressions which are not supported yet.

I created the umbrella ticket https://github.com/apache/arrow-datafusion-comet/issues/240 to cover these unsupported expressions. I will add some expressions there so the people who are interested can work on.

Merged. Thanks.


Remind users they can edit their declaration later

--
This is a requirement of the story.
@superroz can you check the wording?

Looks good to me. Might be worth adding the actual deadline info when we know what it is.


Unable to order form input options when using eloquent driver

Bug description
When using eloquent driver, the form input options are stored as JSON data, which doesn't allow for specific ordering. Options should be stored in an array to allow for specific ordering
How to reproduce
Ordering options in flat files works fine, but when using eloquent driver, options are stored in a JSON object, causing the options to be stored in ascending order. Reordering options in the control panel doesn't work, as the options are always stored in ascending order
Logs
No response
Environment
Environment
Application Name: Statamic
Laravel Version: 11.10.0
PHP Version: 8.2.10
Composer Version: 2.7.2
Environment: local
Debug Mode: ENABLED
URL: empower-eats.public.test
Maintenance Mode: OFF

Cache
Config: NOT CACHED
Events: NOT CACHED
Routes: NOT CACHED
Views: CACHED

Drivers
Broadcasting: log
Cache: file
Database: mysql
Logs: stack / single
Mail: smtp
Queue: sync
Session: file

Statamic
Addons: 1
Sites: 1
Stache Watcher: Enabled
Static Caching: Disabled
Version: 5.11.0 PRO

Statamic Addons
statamic/eloquent-driver: 4.5.0

Statamic Eloquent Driver
Asset Containers: eloquent
Assets: eloquent
Blueprints: eloquent
Collection Trees: eloquent
Collections: eloquent
Entries: eloquent
Forms: eloquent
Global Sets: eloquent
Global Variables: eloquent
Navigation Trees: eloquent
Navigations: eloquent
Revisions: file
Taxonomies: eloquent
Terms: eloquent
Tokens: eloquent

Additional details
No response

Can you provide a screencast or screenshot of what you are seeing and also the underlying data please?

Hi Ryan,
Thanks for getting back to me so quickly.
Please find screen recording and screenshot attached.
Cheers,
Alex
Uploading Archive.zip…

Here's a screencast as well...
https://github.com/statamic/eloquent-driver/assets/138436068/eeae327b-3fde-4d80-a626-9df3e050f90b

Thanks - just for clarity it's blueprint related rather than form related.
The underlying database engine reorders the JSON for the field type on save, so we would need to work around that and do something similar to how we handle sections: https://github.com/statamic/eloquent-driver/blob/8172d109b20f19ba9bb71e98210b07c977511a61/src/Fields/BlueprintRepository.php#L201

Gotcha! Sorry for the confusion, appreciate your help

Sounds good, thanks! Might have a crack at it this weekend if you haven't already gotten to it before me. Cheers mate

Great, if you get to it before me I would suggest you extend the DataReferenceUpdater in core to recurse over the fields, as it will save quite a bit of logic creation. I appreciate your use case is form blueprints, but the solution will need to be able to handle selects inside replications (inside replicators....).

@Baumi-Dev I've done some work on this - https://github.com/statamic/eloquent-driver/pull/321 - still need to apply it to fieldsets but you could test if what I've done so far works for you?

Going to close this off as it will be solved by https://github.com/statamic/cms/pull/10467 in core.


Duplicate Windows

The popup will sometimes show multiple windows.
This is almost certainly due in part to the previous workarounds for the disappearing popup issue #1
So, it's not opening with two Windows, but multiple instances are somehow winding up running at the same time.

Found two opened, used the web controls to show which was in control of the port, closed the other. A second opened again. Repeated and checked for the lock file, which was not present.
It looks like somehow the instance that has the lock file winds up not being the same one that has control of the port.
How?
They shouldn't be able to be open without control of the lock file... clearly one is.
Need to look into, is the first instance losing control of the lock file or is the second opening without it and winding up with control of the port?


refactor(cli): split the 2 big functions into small ones

+ change the style for declaring functions to be consistent with other files

Codecov Report

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

@@          Coverage Diff          @@
##           master   #109   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           3      3
  Lines         134    134
=====================================
  Hits          134    134

Continue to review full report at Codecov.

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

@Garbee all your requests are adressed


Provide some guidance on dependencies and known good platforms

It would be useful to have some guidance on which Linux distribution(s) this has been tested working on, and what version of QT etc is needed to get this to build correctly.
For example, it seems it needs QT 5.15, but it also seems to not completely cleanly build on Debian 11 (see other issue) which has 5.15.2.
Also, adding a note somewhere that the CMake instantiation for l8w8jwtgui has to have the "-DL8W8JWT_ENABLE_EDDSA=On" parameter for the sub-library l8w8jwt specified or the build will fail would help people.

Hi there @firthmj ,
Good idea! This is all still very fresh, so I'm sure there's many things missing in the README.md :)
I tested (and released) this on x64 Kubuntu 20 (LTS) with Qt 6.2.2 using the Qt Creator.
The Linux package in the Releases page contains all the dependencies bundled for making it work on (K)ubuntu.
I actually don't know how the process is easiest/least painful for people who want to build from source on other distros using the CLI. Maybe you can help with that? I'm open for pull requests and I'd be gladly reviewing them 😃

I'm probably slightly unusual in that I use cmake from the CLI quite a lot for other things, so just having some guidance on parameters to pass to cmake would be useful.
But for any project, having a "stake in the ground" Linux distribution where it is known to build and work is very useful, so knowing it has been tested on Ubuntu 20 is very helpful.
I think I will submit a PR to get it to build and run on Debian 11 (which has QT 5.15, as I said before) - it isn't a big delta.


Getting access to 'this'

Is there a pattern or way to get access to the wrapped component's this from within any of the composed methods?
For example, I would like to be able to do this:
withHandlers({
    onError: props => (e) => {
      this.setState({error: e})
    }
  })

if you want to do that, you may need to use withState(), so the code will be sth like this
javascript
withState('error', 'setError', null),
withHandlers({
onError: ({ setError }) => (e) => {
setError(e);
}
})
Let me know if it helps you 😉

But this puts them on the props object right? It seems like this library kind of forces you to use props for everything, but maybe I'm not using it correctly.
Do I have to change from a pattern like this:
    Data.Query.run(q).then(results => results.requests)
      .then(requests => this.setState({ requests, loading: false, error: undefined }))
      .catch(e => this.setState({ error: e, loading: false }));

to
    Data.Query.run(q).then(results => results.requests)
      .then(requests => {
        this.setState({ requests });
        this.props.setError(undefined);
        this.props.setLoading(false);
      }
      .catch(e =>
        .then(requests => {
        this.props.setError(e);
        this.props.setLoading(false);
      }
     );

withState('state', 'setState', null),
withHandlers({
    onError: ({ setState }) => () => {
      setState({ requests, loading: false, error: undefined });
    }
})

Always using props is much nicer, you are decoupling yourself from any particular api, since you are just calling a passed in prop.


Fix bug in iceflow model

I finally found the bug in the iceflow model that was producing unstabilities in the code. The problem was coming from the second time the gradient surface was being calculated. The previous version had
surface_gradient_s[2:end] = diff1(surface_gradient) ./ dx
instead of
surface_gradient_s[2:end] = diff1(surface_h) ./ dx

Well spotted, thanks Facu!


ValueError: cannot convert float NaN to integer

I'm trying to run rbc detection on a specific file. Getting following error. Help.
yolo3_one_file_to_detect_them_all.py:291: RuntimeWarning: invalid value encountered in greater
netout[..., 5:] *= netout[..., 5:] > obj_thresh
Traceback (most recent call last):
File "yolo3_one_file_to_detect_them_all.py", line 434, in
main(args)
File "yolo3_one_file_to_detect_them_all.py", line 421, in main
correct_yolo_boxes(boxes, image_h, image_w, net_h, net_w)
File "yolo3_one_file_to_detect_them_all.py", line 334, in correct_yolo_boxes
boxes[i].xmin = int((boxes[i].xmin - x_offset) / x_scale * image_w)
ValueError: cannot convert float NaN to integer

I just got the same error. I'm pretty sure x_scale is 0 causing the division to result in NaN. But I'm not sure why x_scale ended up at 0.

I got the same error. When I debugged this, every value in netout was nan. I'm not sure exactly why the prediction resulted in a bunch of nan. It seems, though, that the detect_them_all script is designed to work only with the pretrained weight mentioned in the repository's readme and with a default set of labels that work for the repository's datasets. My problem was that I was trying to run it with my own trained weights from my own dataset.

is it fixed? I got same issue

It works only with the pretrained weights mentioned in the file. Check output folder. Sometime it gives the output even with the error.

no any correct output on my test.  xmin, ymin, xmax, ymax are NaN on all boxes[i]

any one find solution??

do you fix it? could you share the solution?

that this issues already fixed ?

In my case despite error output was coming in the output folder. Check the output folder guys.

any updates?


SocketChannel input/output closed behavior

It is possible for a SocketChannel to have its input and output closed. When in this state is it possible that the FD is not cleaned up by the OS until an explicit channel.close() is called (this is the case on mac).  The socket is observed to be in the CLOSED state until the process is terminated.  The question is there anything useful that can be done on a SocketChannel after the input and output have been closed?  Should Netty be responsible for calling close() (at some point) under these conditions?

@normanmaurer @ejona86 @nmittler - FYI.

@Scottmitch can we just close this for now ? I think it is "ok" as it is.

@normanmaurer - I just did a quick test and our EPOLL module on Linux appears to result in the OS cleaning the FDs after shutdown. I don't think we resolved this issue ... did you verify this no longer occurs on mac?


Properly reset pointer tracking data in output_archive

This fixes an issue that is caused by not cleaning out the pointer
tracking data in output archives during serialization preprocessing
while repeatedly serializing the data because of futures not being
ready

This fixes the serialization bug. Thank you very much. The size is now correctly computed in the final pass and matches the preprocessed 'estimate'.
I do still have one bug where the final size does not match the preprocess_archive estimate, but as it turns out, the final size is always (so far) smaller than the estimate, so I can for now ignore it and use the slightly larger estimated size when it happens (about once every N thousand parcels so far, so it is not a big deal). The difference is only 4 bytes or something AFAICT, so it isn't a problem. I will do some larger runs on daint with this fix and see if things are stable.


Add Travis CI Badge to README

Addresses #39.

Very good! merged :)
Feel free to add more badges, if you know any that may fit this repo.


delete a job doesnt always update

Describe the bug
i deleted a job with kubectl when i looked in lens. it was still there.
when i tried to delete it , i got this message

multiple efforts to delete the job thru lens failed.  checking with kubectl confirmed there were no jobs in this namespace.
10 minutes has passed already

Duplicate of #3821


[Type] add non_empty_vec

Pull Request Test Coverage Report for Build 887049250

42 of 42 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.0005%) to 99.967%

Totals

Change from base Build 886406615:
0.0005%

Covered Lines:
3030

Relevant Lines:
3031

💛  - Coveralls


Wired bugs, can anybody help me?

Platform:

Environment:
# Python

# Java

# Ruby

# Golang

Here is the error output:

I tried these ways to fix it, but it's seem useless.
First

Second

Then

There were lots of solutions in StackOverFlow:
Question:0 - StackOverFlow
Question:1 - StackOverFlow
Question:2 - StackOverFlow
Question:3 - StackOverFlow
But none of them can fix my issues...
Thanks for your help.
Best regards!

I think the error is in your environment config. Try changing
# Python

# Java

# Ruby

# Golang

to this:

Thx, I'll have a try. @rdev

@rdev
Year, it works for me!
the again.


Mac: Git checkout and merge sometimes produce bad results

Note that both of these issues reproduce without VFS4Git
Merge
See https://github.com/Microsoft/git/issues/63
Checkout

git clone https://github.com/wilbaker/VFSForGit
git checkout dont_assert_on_file_attributes_merge_weirdness
git status
git checkout bad_checkout_master
git status

Repeat steps 2 through 5 until git status shows a bunch of modified files.

Closing for now for lack of a solid repro. We'll reactivate if one becomes available. Also, haven't seen this since we went to Git 2.20 series.


Refactoring all api response data classes to have BaseClass as parent class, reducing cookie-cutter code.

Is your feature request related to a problem? Please describe.
Lots of to_dicts and @classmethod def from_dict type stuff could be reduced down to just one occurrence with a parent class. Tested with PM's Patch data class, just need to set up elsewhere. Will save 100s of lines of code.
Describe the solution you'd like

Fixed in #209


allow for quadratic objectives

The idea (or hack) is

store objective variable for nonlinear objectives in the model
add an auxiliary constraint with the objvar to represent the nonlinear objectives.
whenever the objective gets change, we

set the objective coefficient of objvar to 0 the objective coefficient of objvar
set bounds of objvar to -\infty, \infty (and so the auxiliary constraint is redundant)
release objvar
create a new objvar and a new auxiliary constraint (if the new objective is nonlinear)

There are a couple of test missing: passing from linear objective to quadratic and the other way  around... but what can I say

Wow, you've been busy!
I thought we were going to do this hack on the SCIP.jl side, because it would be easier?

Oh, my, objvar everywhere ;-)
So there is this comment "this is just wrong" in reformSense.... Do you know the fix?

I think that if you set up the objective var with the auxiliary constraint, everything should be fine, even when changing between minimization or maximization, because the objective coefficient will be changed appropriatly. That is, I don't think objvar needs special treatment in these methods. Let's talk on monday.

Also: we could just always use the auxiliary constraint, even for linear objectives. This would eliminate some checks and duplicate code.

Hey.. I don't remember :) I just found it natural to do it on the CSIP side... how would you do it on the SCIP.jl side?  I don't actually think it would have been easier, because somehow we have to add this variable, but we do not want this variable to be considered by CSIP in any way... right?
Regarding the "this is just wrong"... the thing is that I am always assuming it is minimization, so setting a nonlinear (quadratic) objective means: max/min t s.t f(x) <= t
In the max case, this doesn't make sense and so in the reformSense... I change it to
min -t s.t. f(x) >= t
The issue is that after that change, in the second call of reformSense... I set it back to max t s.t f(x) <= t which doesn't make sense.
So, despite the comment, it is not wrong.. it just doesn't make sense and one could just reformSense... once and that's it.

Also: we could just always use the auxiliary constraint, even for linear objectives. This would eliminate some checks and duplicate code.

I agree.
Monday sounds good.

there is another TODO: when setting the objective we should remove the objective coefficient of all csip variables (but if we follow your idea of handling the linear objective in the same way, then we do not have to worry)

This might be skipped in favor of using the "nonlinear expressions" everywhere.

https://github.com/leethargo/CSIP/pull/22 implements this pr


update deprecation of MAP_ANON from core.sys.posix.sys.mman

remove MAP_ANON for linux from core.sys.posix.sys.mman
(has been deprecated since sep 2013)

add platform specific headers for mman (OSX, FreeBSD)
and deprecate MAP_ANON for those in the posix header

please rebase

done

Auto-merge toggled on


Add index

Add manually created index.ts to project.
I deliberately left out some internal functions and classes and not exported them. To make this clear, I created empty exports for internal files in the index.ts to not mistakenly think they were just forgotten to get exported.
Not sure if that covers every use-case so far in skills, but it is a start IMHO. Interested in your feedback @ddgenome and @ipcrm.
I've migrated a skill to use the index: https://github.com/atomist-skills/github-auto-merge-skill/pull/9

This index provides everything used by the k8s pod health skill.

Thanks @ddgenome. I'll go ahead and merge this and we can iterate in master.

Opps, hang on. I'm seeing your comments now. Addressing them.

They weren't recommendations, just talking points/things to consider.

Closing in favour of #14


Exit after reading test dataset

Hey,
I've tried to run with tested Rao's dataset, but the run was abandoned due to the following issues:

Reading input from GM12878_combined/1kb_resolution_intrachromosomal/chr1/MAPQGE30/chr1_1kb.
Building matrix for chromosome 1 at resolution 1000bp with 249233 rows.
Reading KR normalization counts
Initializing matrix to zero elements
terminate called after throwing an instance of 'std::bad_alloc'
what():  std::bad_alloc
Aborted

It looks like you might not have enough memory for such a large matrix. Does the same command work for chromosome 22 for example (GM12878_combined/1kb_resolution_intrachromosomal/chr22/MAPQGE30/chr22_1kb)? Or at a lower resolution (GM12878_combined/100kb_resolution_intrachromosomal/chr1/MAPQGE30/chr1_100kb)?


Unable to create wallet via nano s

I have the latest ledger bitcoin application installed on my nano s with browser support enabled.
I get an Exception: Invalid channel when trying to proceed in the Hardware Keystore prompt.
If I disable browser support, I get 'NoneType' object has no attribute 'handler'.

huh, going back in the prompts with browser support disabled and going forward again now works


[internals] improper data in anonymous function view

investiguate this strange data
  1) test view with anonymous function, wrong arg (AnonymousFunctionViewTest)
     test/anonymous_function_view_test.exs:86
     ** (CaseClauseError) no case clause matching: {[1, 2, 3]}
     code: view [1, 2, 3] do
     stacktrace:
       test/anonymous_function_view_test.exs:90: (test)

Should display [1, 2, 3] instead of the same but wrapped in a 1-tuple.

When an anonymous function is defined and then immediately applied (fn x -> x + 1).(0), if it is compiled code, then the Erlang VM executes it as a case. (Explaining the wrapping in a tuple, as that's how I did it for custom a def)
So nothing can be done on my end.

https://github.com/erlang/otp/pull/4545

Detect this case and unwrap the values to raise error similar to Elixir


Applying periodic BCs to parcel mergers

Here is the PR. I have also checked it against my original fix, the results are identical to good precision.

@sjboeing This looks good, but we also need to fix this in 2D.

@sjboeing I applied also the changes in the 2D version of your PR. I then cherry-picked the commit and opened a PR to merge into the master branch, see #298.

@sjboeing If you agree with my commit https://github.com/matt-frey/epic/pull/297/commits/9889e42cd8b9017b29d7ccb3eeaafa52e99558f5, I am happy this to be merged in. Thanks!


getViewport -> makeViewport

For #2748

Change List

In the docs: getViewport -> makeViewport

I confirmed that this fixes the example in codepen.

@mccalluc Many thanks for both finding AND fixing this!


Make Path serializable

Path objects, as they are exposed today, are opaque - once a line/curve/shape is added to them, it is not possible to inspect them (beyond getting bounds and measurements).
I would like to have an efficient way to serialize (and deserialize) Paths.  Some use cases:

Capture and save a drawing/signature as a vector (rather than being forced to rasterize it).
Have an efficient way to store and retrieve Paths that were created at runtime from other data (e.g. from SVG path data)

There was previously discussion around using SVG Path data strings, which Skia has some built in support for, here: https://github.com/flutter/engine/pull/4784 - which was rejected for being too SVG specific and potentially bloating the engine.
It would be fine for me if we could even just get the verbs, points, and weights from an SkPath - although it might be nice to expose weights in a more friendly way, or perhaps to stick to a normalized subset of the possible path verbs in Skia (e.g. by decomposing certain curves to a another equivalent curve).  This could be done without adding (much) additional code to the engine, at the cost of relying on Skia's internal Path implementation details a bit more.

And to clarify, my suggestion here is to expose SkPath::getVerbs and SkPath::getPoints.
There is no current way I can see to get the weights from an SkPath, and it really looks like it's only ever used for Conics and is set to 1 (set to 0 in all other cases).

Another option is to have anyone who creates a Path manually keep track of how they did it and serialize that.

I do have (yet another) issue with Paths.
My goal: I am trying to serialize my Path objects and send them to another client so that the client can display exactly the same Path.
This seems not doable at the moment as I cant access anything from the Path object.
I need a Path object instead of a plain array of captured points as this will decrease the packet size a lot - which is a lot of money I dont have to spend for a more powerful server.


⚠ Samsung has degraded performance

In f4d9775, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2834 ms

Resolved: Samsung performance has improved in 193e089.


support tman and other test frameworks.

branch can not be merge auto, please rebase to resolve it, so sorroy. @zensh

👍 +1


Create CarbonBlackViaAWSS3_ConnectorDefinition.json

Required items, please complete
Change(s):

See guidance below

Reason for Change(s):

See guidance below

Version Updated:

Required only for Detections/Analytic Rule templates
See guidance below

Testing Completed:

See guidance below

Checked that the validations are passing and have addressed any issues that are present:

See guidance below

Guidance <- remove section before submitting

Before submitting this PR please ensure that you have read the following sections and filled out the changes, reason for change and testing complete sections:
Thank you for your contribution to the Microsoft Sentinel Github repo.

Details of the code changes in your submitted PR.  Providing descriptions for pull requests ensures there is context to changes being made and greatly enhances the code review process.  Providing associated Issues that this resolves also easily connects the reason.

Change(s):

Updated syntax for XYZ.yaml

Reason for Change(s):

New schema used for XYZ.yaml
Resolves ISSUE #1234

Version updated:

Yes
Detections/Analytic Rule templates are required to have the version updated

The code should have been tested in a Microsoft Sentinel environment that does not have any custom parsers, functions or tables, so that you validate no incorrect syntax and execution functions properly.  If your submission requires a custom parser or function, it must be submitted with the PR.

Testing Completed:

Yes/No/Need Help

Note: If updating a detection, you must update the version field.

Before the submission has been made, please look at running the KQL and Yaml Validation Checks locally.
https://github.com/Azure/Azure-Sentinel#run-kql-validation-locally

Checked that the validations are passing and have addressed any issues that are present:

Yes/No/Need Help

Note: Let us know if you have tried fixing the validation error and need help.

References:

Guidance for Detection checks
General contribution guidance
PR validation troubleshooting

Hello @mpmisha, Can you please update the branch from master once. Unable to see KQL validation errors

Hello @mpmisha @oded-weber, I have generated the CCP template but while connecting the connector I was facing issue for DCR

I have shared the template and above screenshot with Alon. Asked him if has any idea about the issue


Update restore-statements-filelistonly-transact-sql.md

@Duracell1989 : Thanks for your contribution to the Azure documentation! The author, @barbkess, has been notified to review your proposed change.

@Duracell1989 : Thanks for your contribution to the Azure documentation! The author, @barbkess, has been notified to review your proposed change.

@Duracell1989 : Thanks for your contribution to the Azure documentation! The author, @barbkess, has been notified to review your proposed change.

@barbkess please review


Add logic to handle revoked certs

Not sure if treating CrlStatus::RemoveFromCrl as not being revoked is correct @TKussel ?

Not sure if treating CrlStatus::RemoveFromCrl as not being revoked is correct @TKussel ?

To quote the OpenSSL doc:

In practice, removeFromCRL is not particularly useful because it is only used in delta CRLs

This is true for us as well, we fetch the full CRL, not the delta. No removeFromCRL should occur.

One last thing, would it be possible to write a test case for this?


Xnor/rtic upgrade

upgraded to rtic 1.0
removed _bss from sdram.. quick search made me think it isn't zeroed, which is what _bss indicates

Moving up the latest is fine. You'll also need to update the CI to use that version.

Moving up the latest is fine. You'll also need to update the CI to use that version.

done