Fix fetch_all_by_outward_search

Description
Fixed fetch_all_by_outward_search() to expand the search window in increments of the RANGE value. IN the current implementation the search range is increased exponentially instead of linearly. This has now been fixed by storing the initial RANGE value.
Use case
Currently, calling this function with RANGE=10,000 and MAX_RANGE=500,000 will do the search in the following ranges: 10000, 20000, 60000, 240000 and 1200000. This pull request changes this behaviour to search in ranges from 10000 to 50000 in increments of 10000.

Coverage increased (+0.0005%) to 81.445% when pulling 02e6db25e1b4b590d487ed9aa1ab3be65f1859e5 on AsierGonzalez:bugfix/fix_fetch_outward_search into 378db18977278a066bd54b41e7afcf1db05d7db3 on Ensembl:release/97.

Hi @mkszuba,
Thank you for your feedback. I did see that the template mentioned the documentation of the testing but I didn't really know what I should put there. I only did the very simple test of running the old and the new version of the function with the same variants and print the size of the search window and the output. Would that be enough?

It's a start but we would very much prefer if you wrapped this into a unit test which could be run as part of the ensembl test suite. Moreover, running the whole test suite to confirm the change doesn't cause any side effects elsewhere in the code is pretty much a must.
In this particular case, I see there is a test involving fetch_all_by_outward_search() present in getNearestFeature.t which I am fairly sure should now fail.

What you say sounds sensible but I am afraid that I am not familiar with Perl testing so I would appreciate any links to Ensembl docs (or other sources) where it is explained how to run the test suite and how to add new tests. If I have that information I can add a test to check that the function does not return features outside the search range, which would fail in the current implementation. The current test passes for both the old and new implementation because there is a feature in the limit of the search range.
I am also confused about one of the two Travis builds failing. It seems that the failure is due to issues installing Perl v5.14 so I believe this is not related to the changes I made.

What you say sounds sensible but I am afraid that I am not familiar with Perl testing so I would appreciate any links to Ensembl docs (or other sources) where it is explained how to run the test suite and how to add new tests. If I have that information I can add a test to check that the function does not return features outside the search range, which would fail in the current implementation.

Excellent. Documentation for setting up the test environment can be found here, one thing that might be worth adding is that for running a single test file you can use the command prove which should have come with Perl (something like prove -v ensembl/modules/t/getNearestFeature.t). Plus of course there is Travis, whenever it gets to running another build - we tend to keep it rather busy.

The current test passes for both the old and new implementation because there is a feature in the limit of the search range.

That's not good, this test is supposed to check for a feature outside the initial search range... Care to try and modify it so that gives a real rather than a false positive?

I am also confused about one of the two Travis builds failing. It seems that the failure is due to issues installing Perl v5.14 so I believe this is not related to the changes I made.

We are aware if that problem, it is indeed unrelated to your changes.

This is my fault for not properly testing the outward iterator. @AsierGonzalez I propose the following:
If the fourth iteration is 240 bp, then adding a test at L183 should help, like so:
`
Test multiplication error in search window of fetch_all_by_outward_search
@results = @{ $sfa->fetch_all_by_outward_search(-FEATURE => $a, -LIMIT => 5, -RANGE => 10, -MAX_RANGE => 140)};
cmp_ok($results[2]->[0]->display_id, 'eq', 'b', 'Check for closest feature in initial search range');
ok(! defined $results[3], 'Feature $c is not found because it is out of the MAX_RANGE';
`
The distance between $a and $c is 160, hence the original test still finds $c once you've corrected the multiplier. If it limits correctly, it should not find $c. Will that help get this fix in?

Hi all,
Sorry for not replying to the previous messages, I'm swamped with work at the moment and this is in my to-do list. Thank you for the useful feedback and links to the three of you.
@nerdstrike, the idea is to do something like that, if I we reduce the limit to less than 160 bp the current method will stil find $c but the correct implementation won't.
I'll make the changes as soon as I can and I'll let you know.

Thank you! Unfortunately even after this change the test in question continues to pass on master.

I know, I mean to update you on this but I didn't have time.
Last week I tried to set up the test environment as per your suggestions but I didn't succeed. It was really frustating, because I tried to install it both on a MAC and a Debian machine and failed miserably in both. I believe that the problem on the Debian machine is because the test are not suited to use MySQL 8 but there is no information about it in the documentation.
Anywa, th summary is that I gave up and decided to use Travis instead. I was puzzled to see that this test still passes. I need to look into it again but do you have any ideas?

I have finally managed to make the tests run so the fix should be ready to be merged. However, I guess that it should now be merged with the release/99 branch and there are some conflincts. Should I just close this PR and start a new one with the current version? @mkszuba, @ens-bwalts, @nerdstrike

Well done @AsierGonzalez. I leave the integration issues to my colleagues. It should be relatively straightforward to base your changes off master and lose all the testing baggage your PR has acquired.

Thanks a lot for your work on this issue, @AsierGonzalez ! I am afraid it is already too late to have this included into release 99, my recommendation is for you to start with the current state of master, cherry-pick your changes from the current bugfix/fix_fetch_outward_search, then force-push it to GitHub as the new bugfix/fix_fetch_outward_search in order to keep this PR open.
Or since I have just done the cherry-picking myself in order to see if it caused any problems and pushed the result to GitHub (see Ensembl:bugfix/fix_fetch_outward_search), you could just confirm I didn't miss anything and subsequently skip the first two steps :)

Hi @mkszuba, I have checked Ensembl:bugfix/fix_fetch_outward_search and all my changes are there. What should we do now?

Just force-push it to AsierGonzalez:bugfix/fix_fetch_outward_search and we shall take it from there.


Recommend service principle for ACR auth

@ebertrams I added a pointer to the production checklist in response to work item https://msazure.visualstudio.com/DefaultCollection/One/_workitems/edit/9701846/

Thank you Kelly!


Add a "script node" in deployments for arbitrary scripting

Related to #29
Inherits from base node.
Allows the user to start non-ros executables, set up directories, etc.

related to #29


Add test cases to verify that error responses contain status code

Fixes #912

Hi @gmlewis , I hope you has a great weekend. Here's the PR for issue 912


Partially reverting #441 - get @EqualsAndHashCode(callSuper = false) back for data classes

Summary
This pull request partially reverts #441 (this PR was really amazing).
As having @EqualsAndHashCode(callSuper = false) annotation in @Data classes was intentional (I totally forgot it when reviewing the PR), the commit to remove them should be reverted. See also: https://github.com/slackapi/java-slack-sdk/pull/441#issuecomment-622642389
cc @eamelink
Requirements (place an x in each [ ])

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.

Also, I need to add @EqualsAndHashCode(callSuper=false) to the ones that used to have callSuper=true. Will do.

Ah, I had overlooked that! Thanks :)
Having the equality fixed is great; I use it for tests of my library that generates blocks :)


Upgrade hyper and otel dependencies

Changes
Upgrade otel and hyper dependencies, change HttpClient example to hyper over unix socket.
I removed writing the resource for every span, as the field was removed from SpanData.
Merge requirement checklist

[x] CONTRIBUTING guidelines followed
[x] Unit tests added/updated (if applicable)
[x] Appropriate CHANGELOG.md files updated for non-trivial, user-facing changes
[x] Changes in public API reviewed (if applicable)

I removed writing the resource for every span, as the field was removed from SpanData.

@kcking-openai thanks for the PR. Regarding resource, while it was removed from SpanData, SpanExporter::set_resource() method was introduced for one time setting of resource in Exporter. Default implementation was added for it in the SpanExporter trait so that custom exporters don't break. You may want to implement the DatadogExporter ::set_resource() method instead of removing resource from the code.

Thank you @lalitb, I have completed the CLA and pushed a fix for the lint. If you would like an alternative solution to the too_many_arguments lint I could wrap them in a struct, but I wasn't sure it was worth the effort at this point.

And I just pushed a fix for some lints that had broken in a couple unrelated directories.

https://crates.io/crates/opentelemetry-datadog/0.12.0


Room name does not support special characters

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Proposal:
Update our rooms names to allow both dash - and underscore _ instead of just underscore as they do now. And also update our default when a user types a space from underscore to dash. I think dashes just look better than underscores
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
https://user-images.githubusercontent.com/43996225/201206243-0f893eca-c35b-4f60-84a2-1561db8ab0d8.mp4
Expensify/Expensify Issue URL:
Issue reported by: @puneetlath
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1668112427854869
View all open jobs on GitHub

Proposal
Change the regex
[^a-zA-Z\d_-]/g
https://github.com/Expensify/App/blob/5401816f3781da2db128781ab92084ffffc589c8/src/components/RoomNameInput.js#L62

Not a bug, this is a design decision. Closing

Ah sorry I see now this doesn't have a bug label on it. I personally think we should put a freeze on all new features for WAQ so I'm keeping this closed.

@puneetlath feel free to re-open if you'd like. I think my concern is that any person working on a new feature isn't solving bugs. And we have a mountain of bugs in this repo that we need to maintain focus on for the duration of WAQ.


[NOT A BUG] [Help needed] How to draw a building layout over the map

Summary
I am creating a booking app which allows user to select slots in a building and so i would like to draw a building layout in a particular building or coordinate and then let user to select slots from the layout. How to implement the same in a way that i can draw over maps - a layout of the building with clickable or selectable slots.
Here is the rought image of my expected implementation:

Reproducible sample code
<MapView
          provider={PROVIDER_GOOGLE} // remove if not using Google Maps
          style={styles.map}
          initialRegion={{
            latitude: 23.92374455995645,
            longitude: 47.26707199897953,
            latitudeDelta: 0.005,
            longitudeDelta: 0.0121,
          }}>
          {data.map((marker, index) => {
            return (
              <Marker
                key={index}
                coordinate={{latitude: marker.lat, longitude: marker.lng}}>
              </Marker>
            );
          })}

Steps to reproduce
Need help on implementing the above functionality. Can it be implemented using Overlay and/or using Polyline?
Expected result
A building Layout over the map with clickable areas
Actual result
Not able to implement
React Native Maps Version
1.7.1
What platforms are you seeing the problem on?
Android, iOS (Apple Maps), iOS (Google Maps)
React Native Version
0.71.8
What version of Expo are you using?
Not using Expo
Device(s)
All devices
Additional information
No response

For the building outlines, just use polygons.  For the "slots" inside the buildings, use markers with custom shapes: https://github.com/react-native-maps/react-native-maps/blob/master/docs/marker.md#children-components


NettyHttpClient: Replace ReadTimeoutException with our own exception.

It may also be a good idea to wrap non-ReadTimeoutExceptions, to provide additional context to all exceptions. But I thought we could start here.

wow, this patch is soooo useful. Just merge it @gianm . It’s tested on two of our clusters.


executable for Linux

change ...
else
platform="unix"
outfile="lite"
compiler="gcc"
cflags="$cflags -DLUA_USE_POSIX"
lflags="$lflags -o $outfile  -no-pie"
fi
and
creat file this bureau
open mousepad or nano
[Desktop Entry]
Encoding=UTF-8
Version=1.0
Type=Application
Terminal=false
Exec=/$HOME/lite/lite
Name=lite
Icon=$HOME/lite/lite.png
Comment=
Path=$HOME/lite
StartupNotify=false
Thank you for application is super

You want a .desktop file? I'm pretty sure that won't be added, since lite isn't distributed with any form of installer that can get you a consistent path for the executable. For example, you suggest it is in the user's home, but I happen to have it under /opt.
Instead of just adding a .desktop file, we should also add some installing script that copies stuff to a fixed place.

You want a .desktop file? I'm pretty sure that won't be added, since lite isn't distributed with any form of installer that can get you a consistent path for the executable. For example, you suggest it is in the user's home, but I happen to have it under /opt.
Instead of just adding a .desktop file, we should also add some installing script that copies stuff to a fixed place.


Update http4s-blaze-client, ... to 0.20.0-RC1

Updates

org.http4s:http4s-blaze-client
org.http4s:http4s-blaze-server
org.http4s:http4s-circe
org.http4s:http4s-dsl
org.http4s:http4s-server

from 0.20.0-M7 to 0.20.0-RC1.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention @scala-steward in the comments below.
Have a nice day!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [{ groupId = "org.http4s" }]

Handled directly


Administrator permissions

Prerequisites

[X] Put an X between the brackets on this line if you have done all of the following:

Checked that your issue isn't already filed: https://github.com/icetee/remote-ftp/issues

Description
When I try to create a file (in a folder which requires administrator privileges it doesn't work. Is there a way to get administrator permissions? I am using a Raspberry Pi as the host.
Steps to Reproduce

Try to create a file which requires administrator permissions
It doesn't create the file

Expected behavior: [What you expect to happen]
The file to be created.
Actual behavior: [What actually happens]
The file isn't created.
Reproduces how often: [What percentage of the time does it reproduce?]
100%
Versions
Atom: 1.60.0
Electron: 9.4.4
Chrome: 83.0.4103.122
Node : 12.14.1
OS: Windows 11 21H2

Prerequisites

[x]  Put an X between the brackets on this line if you have done all of the following:

Checked that your issue isn't already filed: https://github.com/icetee/remote-ftp/issues

Description
When I try to create a file (in a folder which requires administrator privileges it doesn't work. Is there a way to get administrator permissions? I am using a Raspberry Pi as the host.
Steps to Reproduce

Try to create a file which requires administrator permissions
It doesn't create the file

Expected behavior: [What you expect to happen] The file to be created.
Actual behavior: [What actually happens] The file isn't created.
Reproduces how often: [What percentage of the time does it reproduce?] 100%
Versions
Atom: 1.60.0 Electron: 9.4.4 Chrome: 83.0.4103.122 Node : 12.14.1 OS: Windows 11 21H2

@bjhaneline21291 #463

Atom.io has been deprecated. Maybe archive this repository?


Format of known_hosts

If I run
- run: for ip in $(dig @<IP_ADDRESS> gitlab.evilcorp.de +short);
do ssh-keyscan gitlab.evilcorp.de,$ip; ssh-keyscan $ip; done
And paste the output:
#gitlab.evilcorp.de:22 SSH-2.0-OpenSSH_8.2p1
gitlab.evilcorp.de,IP ssh-ed25519 AAA...
gitlab.evilcorp.de,IP ssh-rsa AAA...
IP ssh-ed25519 AAA...
IP ssh-rsa AAA...
... to GIT_SSH_KNOWN_HOSTS, mirror-action fails with:
Host key verification failed.
fatal: Could not read from remote repository. Please make sure you have the correct access rights and the repository exists.
BUT if I run mirror-action with GIT_SSH_NO_VERIFY_HOST: "true" it runs without errors.
I also pulled my repo to my machine, accepted the fingerprint and copied the .ssh/known_hosts to my secrets. The according file was differently formatted (|1| 3Wm...=|z+s...= ecdsa-sha2-nistp256 PubKey) but also won't work.
Any suggestions on how to fix this?
P.S. My Script:
steps:
- uses: actions/checkout@v1
- uses: spyoungtech/mirror-action@master
with:
REMOTE<EMAIL_ADDRESS>GIT_SSH_PRIVATE_KEY: ${{ secrets.GIT_SSH_PRIVATE_KEY }}
GIT_SSH_KNOWN_HOSTS: ${{ secrets.GIT_SSH_KNOWN_HOSTS }}
DEBUG: "true"

Sorry to have missed this for so long, I thought I definitely responded to this before.
The GIT_SSH_KNOWN_HOSTS is simply the contents of a known_hosts file. The action simply dumps the variable contents directly into a file and the ssh configuration is told to use that file.
So, you just need to follow the standard convention. You can probably just copy/paste the host from your own known_hosts file for whichever host you're using.
The format is like
<host> <key format> <key>

For example for GitHub.com you can use this:
github.com ssh-rsa AAAAB3NzaC1yc2EAAAABIwAAAQEAq2A7hRGmdnm9tUDbO9IDSwBK6TbQa+PXYPCPy6rbTrTtw7PHkccKrpp0yVhp5HdEIcKr6pLlVDBfOLX9QUsyCOV0wzfjIJNlGEYsdlLJizHhbn2mUjvSAHQqZETYP81eFzLQNnPHt4EVVUh7VfDESU84KezmD5QlWpXLmvU31/yMf+Se8xhHTvKSCZIFImWwoG6mbUoWf9nzpIoaSjB+weqqUUmpaaasXVal72J+UX2B+2RPW3RcT0eOzQgqlJL3RKrTJvdsjE3JEAvGq3lGHSZXy28G3skua2SmVi/w4yCE6gbODqnTWlg7+wC604ydGXA8VJiS5ap43JXiUFFAaQ==

Hello, I encounter the same issue @catchMyException described, using a copy of my own know_hosts file or the result of the ssh-keyscan -p <port> <hostname> command. The format matches what you said, still I can only seem to be able to push by using the GIT_SSH_NO_VERIFY_HOST parameter.
Copying what is in GIT_SSH_KNOWN_HOSTS to a known_hosts file via echo "<content>" > known_hosts and executing manually what your GH action does is working fine (and passing it an empty known_hosts fails, as expected).
ssh <user>@<host> -p <port> -i ~/.ssh/id_rsa -o IdentitiesOnly=yes -o UserKnownHostsFile=known_hosts

I'll take a look at your code to see whether a quick fix is possible for me to PR.
My GH workflow:
name: Mirror the repository
on:
  push:
    branches: [ main ]
jobs:
  mirror:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v2
      - name: 'Mirror'
        uses: spyoungtech/mirror-action@master
        with:
          REMOTE: ${{ secrets.REMOTE }}
          GIT_SSH_PRIVATE_KEY: ${{ secrets.GIT_SSH_PRIVATE_KEY }}
          GIT_SSH_KNOWN_HOSTS: ${{ secrets.GIT_SSH_KNOWN_HOSTS }}
#          GIT_SSH_NO_VERIFY_HOST: "true"
          DEBUG: "true"

I found what the problem was and made a PR.
Using non-explicit path for the git config --global core.sshCommand parameter resulted in the known_hosts file not to be found (I guess). I replaced the ~ with the actual path /github/home. You might want to put the resolution of ~ in a variable and concatenate it instead of hard coding it as I did: Github could change the home path and therefore break the script.
Check the PR #19
I would be very glad if you could accept it soon as I will have to GIT_SSH_NO_VERIFY_HOST in the mean time.
Thank you for your efforts, this GH action is very handy!


Transactional attribute no longer required when using EF tx

According to the docs, the [Transactional] attribute should not be required to enroll in transactional middleware. I added a call to options.Handlers.AutoApplyTransactions() to UseEntityFrameworkCoreTransactions. I adjusted the unit tests and sample code.
I modified auto apply to skip sagas as I am not sure if it should be treated this way (if not omitted all EF Integration Saga) tests fail.

@wojteksuwala I think I want to keep this as an opt in behavior for now, and I adjusted the docs today to reflect that.
Is it okay if we don't take this one in?

Sure no problem


Conflicting nullability specifier on return types, '_Nullable' conflicts with existing specifier 'nonnull'

Swift compiler warning in:

FSSecutiry.h
FSWatermark.h

Is this warning related to Xcode 8 updates in nullability declaration check? Any thoughts?

please update to 0.3.5, should be fine.

@pJes2 I have updated to version 0.3.5 and the warnings are gone. Thanks!

closing


how do i add ui to pysch engine?

Describe your problem here.
how do i make a ui for pysch engine for the menu?
Are you modding a build from source or with Lua?
source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
no

be more specific

be more specific

he's probably asking how to make custom menu just like from the holiday mod

that's still not specific at all

like how do i put the logobumpin in to the menu?

you can't without source

you can't without source

can't you change the image of logobumpin by putting it into the images directory of your lua made mod?

what do you mean by sourse

can't you change the image of logobumpin by putting it into the images directory of your lua made mod?

they wanted to copy it to the menu, or they explained it wrong

what do you mean by sourse

source code

wait i use visual studio code is that a source code or a lua?

probably source then, but you marked lua here

oh i thought visual studio code is a lua thing i did a mistake with that sorry about that

i fucked up

i really thought that visual studio code is a luA

i change it from Are you modding a build from source or with Lua? to source

so can you help me?

with that?

with that?

are you trying to modify the start screen just like this mod did with Lua?

oh i thought visual studio code is a lua thing i did a mistake with that sorry about that

Visual studio code is a text editor which you can use to code with literally any language. You just gotta have some plugins installed for some specific languages. It can support haxe (aka. source code) and lua too

You can change logoBumpin by putting your png and xml (named logoBumpin) into your /images/ folder in your mod folder


apis: add reservation name, taints and tolerations in ReservationAffinitty

Ⅰ. Describe what this PR does

add taint and tolerance for reservation
Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
V. Checklist

[ ] I have written necessary docs and comments
[ ] I have added necessary unit tests and integration tests
[ ] All checks passed in make test

/lgtm


Bag of holding changes currency values

Could also build custom bag of holding, may resolve issue.

What does it actually do and how could we reproduce this?


Search in tables not working as expected when there are several pages.

Describe the bug
Search functionality in tables is not working as expected, it does filter and show the elements which are currenty displayed in the list, but not the ones that are in a different page of the table.
To Reproduce
Steps to reproduce the behavior:

Go to Products (having more than 10 records so that there are more than one page of results in the table)
Search for a Product that is in a different page than the current one, e.g. page 2.
The records matching the searched string won't show up, the table will be empty, you will only see them if you go to the page where they are.

Expected behavior
The records matching the search string should be shown doesn't matter in which page of the table they are.

@unchase I have tried the fix (applying the changes from your PR). It works if you search from the first page but if you are in, for example, second page of results, again the filtering doesn't work.

Hi, @paferpo
The filtering works correctly on the second page, but you get a response without going to the first page after the search.

Hi, @unchase
You get the response with the filtered results but the view is not updated to show it. You should see the results after your search irrespectively of the page you currently are. Otherwise it's not very user friendly, is it?

Fixed in https://github.com/blazorhero/CleanArchitecture/pull/173/commits/d98f0b06f63582e181c9f58bf76264965ea267e2
Check, please

Now it's perfect, thanks! 🎉


Feature: GetTenantByEmail function

Is your feature request related to a problem? Please describe.
I need to be able get a tenant by email address.
similar to how get tenant by id is available https://github.com/staticbackendhq/core/blob/528ab4d6db58e546a465453b79e474486185a7a8/database/postgresql/sb.go#L217
getting a tenant by email will also be very useful

also note there is a bug here:
https://github.com/staticbackendhq/core/blob/528ab4d6db58e546a465453b79e474486185a7a8/database/postgresql/account.go#L51
should be %s.sb_accounts


Update for Electron v7.2.4

Would you be willing to update this project for the most recent version of electron?
I'd be happy to help update if someone could point me in the right direction of how this project interacts with electron.

Try checking this version which is more up to date:
https://github.com/dceejay/electron-node-red


Amrutha/newtab

Description
(Write your answer here.)
Checklist

[ ] I am making a proper pull request, not a spam.
[ ] I've checked the issue list before deciding what to submit.

Related Issues or Pull Requests
(Write your answer here.)
Add relevant screenshot or video (if any)
(Add here)

I'm closing this as already the changes have been made.


Improve model ploting

Sorry for the crazy number of files changed. It is mostly data and from my dumb idea to have the data downloaded.
This adds a new system for flux calculations and model plotting that is modular and easily extendable to extended sources.
For now, only point sources have dedicated classes. The calculations are fast now that I have delayed using units until the end of the process.
flux and plotting of point sources is now done with functions instead of classes. Key words remain the same. examine the updated demos.
Also, we now have better and full support of using energy/wavelength/frequency for the energy axis of plotting or flux. This is done using native astropy tools.
I also fixed a few bugs. The downloader was ignoring compressed files and re downloading them... but no longer.
I reorganized the io folder as it was getting big.

PS: coverage decreased... but I wanted to fix that on my own after you review.

Current coverage is 69.06% (diff: 66.28%)

Merging #140 into master will decrease coverage by 3.40%

@@             master       #140   diff @@
==========================================
  Files            91         96     +5
  Lines          9676      10259   +583
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           7012       7085    +73
- Misses         2664       3174   +510
  Partials          0          0

Powered by Codecov. Last update 8250410...0f653dc

@giacomov I have successfully updated this to the new framework.
I kept the same idea of have "handlers" for fitted sources except much more of the work is being done by RandomVariates and the implementation is MUCH cleaner now that error propagation is uniform between MLE and Bayesian. This also got rid of the nasty multiple inheritance.
One issue was the fact that you can not use astropy units  with or make arrays of RandomVariates because the constructor becomes confused and turns the RV into a regular array. I looked at modifying the constructor, but it would have been really ugly and the VariatesContainer I have made is a much simpler solution that also deals with unit conversion easily.
The frame work is extendable to polarization plotting, extended objects, etc. with dedicated subclasses.
The actual plot and flux functions are a bit verbose, but this is to handle all the cases that may come up. I've tried to keep duplicate code to a minimum except where it is needed.

@giacomov also fixed a bug in analysis results. you were calling equal_tail for both equal_tail and hpd


fix: externalAddr scheme can be set by user

Closes #747
Removed the hardcoded "http://" scheme from Inbound Endpoint.
Signed-off-by: Derek Trider<EMAIL_ADDRESS>

missing semantic prefix in title.

missing semantic prefix in title.

Done!


Spotify shorted links doenst work

The new Spotify short url template, spotify.link, template does not work in the bot all the time.

it never did, I'll try to make some changes, meanwhile suggest it in the suggestions channel in our discord server!


[Multimonitor][Bug] Cairo "restores" all over the place when waking a multi-monitor system

See screenshot. And this time all drivers (including for DisplayLink) are completely up to date.

For three screens, there are four Cairo menubars, with only one of them (the "closer" of the two on the top-right screen) being in the proper location. None of them respond to any input, either.

What is the resources rectangle on top left monitor?
Is this a bad ms paint copy paste or is there two top menu bar on top right screen ?
If two, how many Cairo processes in task manager?

What is the thing highlighted in pink? !
Just observing:
It seems the second bar on top right screen is the bar missing on bottom screen.
Taskbar is also spanned across two screens, and misaligned.
Which one is the USB DisplayLink screen ?

Do you see better results if all monitors are set to the same scaling setting?

@mdmk That's Stardock Fences. Also, pretty sure the menubar for the bottom (laptop internal) screen is the one that is split between the top two screens about two thirds of the way down them. The offset of the top-right screen is because of the screen positioning in the display settings since Windows 10 unfortunately doesn't allow pixel-perfect monitor alignment. IIRC the top-left monitor is the one connected USB->DisplayLink->DVI. The other top monitor is DP direct from the laptop.
@dremin We'd have to see this happen again before I could say, but if I did change the laptop monitor to 100% scaling like the other two it'd be unreadable for me. Even when the system is not docked I need it scaled up.

Fences is in wrong position so ?
Try without Fences started.
Stardock tools generally does heavy modifications to Windows,  better to leave it off for test purposes.
As a workaround for what @dremin said, you can lower internal resolution.
It will be ugly and blurry, but bigger.
And no longer high dpi, so not very good idea for a significant test, unless it is totally unrelated.
You can also launch Windows magnification tool when needed, create a shortcut on your desk or taskbar, or use Win + Plus (symbol +) to launch magnification tool easily.

@coldacid Understood. I actually somewhat expect the positioning of things to not be correct with per-monitor DPI as that is something I am still working on, but the other issues I see on your screenshot are definitely unexpected. Does this happen every time or has it been mostly OK?

@dremin It's been mostly okay.

Is this any better in this build?

This is fixed in the latest release.


expandParent nope option not allowing children nodes to expand parent nodes when set to true

Describe the Bug
This is a beautiful project so thank you for developing it!
I'm a pro subscriber and when playing around with the dynamic grouping example I noticed some odd behavior. First, the initial parent node can be expanded upon by its children, but the expandParent node option is not set to true or any state for any of its children nodes. When I drag and drop a new group node onto the canvas, and then drag an existing node into the new group node it assigns properly but I cannot expand the parent extent by dragging the children to the group node's boundary.
The latter makes sense as the expandParent node option isn't set for any of the initial nodes.
To further diagnose I hardcoded the expandParent node options (true/false) in the initial elements and noticed the same phenomena.
I'm wondering if this has something to do with the nodes array and how they are sorted since the initial group node always comes first then the second group node.
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue

Use Dynamic Grouping Pro Example
Drag an already existing child node in the initial group node to see that the group node's extent can expand.
Verify node options for initial nodes do not have expandParent set to true.
Hardcoded initial node option expandParent for all simple node types to false results in the same phenomena. Verified Node Options on console log.
Hardcoded initial node option expandParent for all simple node types to true results in the same phenomena. Verified Node Options on console log.

Expected behavior
dragging of children nodes should not expand parent node extent with current settings, yet does so only in the demo for the initial group node. Dragging and dropping a new group node shows behaviors as expected.
Screenshots or Videos
https://github.com/wbkd/react-flow/assets/67077261/8f6705f4-f554-4b51-83a8-94fa9d2cb263
Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 91.1]

Additional context
No response

Hey @rperendy
this is not a bug, but an example of how you can implement custom functionality based on child nodes. In this case we are calculating the group node dimension based on the child nodes. It's this part of the custom group node implementation:
const { minWidth, minHeight, hasChildNodes } = useStore((store) => {
  const childNodes = Array.from(store.nodeInternals.values()).filter((n) => n.parentNode === id);
  const rect = getRectOfNodes(childNodes);

  return {
    minWidth: rect.width + padding * 2,
    minHeight: rect.height + padding * 2,
    hasChildNodes: childNodes.length > 0,
  };
}, isEqual);

If you don't like that behaviour, you can safely remove the minWidth and minHeight code.


Update scalatest, scalatest-mustmatchers to 3.2.6

Updates

org.scalatest:scalatest
org.scalatest:scalatest-mustmatchers

from 3.2.5 to 3.2.6.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalatest" } ]

labels: library-update, semver-patch

Superseded by #27.


Attempt to || ize the experiment runs

Might be worth it later, probably not

https://medium.com/@mjschillawski/quick-and-easy-parallelization-in-python-32cb9027e490 -> the only thing I am afraid of is globals

Ended up not using that article and spawn them as separate processes

Tested with many files, hits ~97% cpu usage on 5800x;
Config files cannot have spaces in them


useSelection should expose a function which focuses & selects a node, same as onNodeClick

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report
[ ] Performance issue
[X] Feature request
[ ] Documentation issue or request
[ ] Other... Please describe:

Current behavior

I've got a storybook plugin which displays Recoil trees both in the graph & in a list, currently if the user clicks a node on the graph they get a nice zoom in focus animation to show the node - but if they click it in the list the node is selected (via setSelections) but the graph doesn't zoom to focus on it.
Expected behavior

Ideally I'd like the same focusing action to occur regardless of if the user clicks the node in the graph or in the list.
Also it'd be really nice to have a way to trigger the hover effect.
Could be there's something returned by useSelection that'd do this already but I've not spotted it
What is the motivation / use case for changing the behavior?

To allow the nice graph features to be used programmatically as well.

Couldn't you set the node selection to do this? I'm concerned this might get state out of whack if you use 2 different ways to focus things.

OK - I thought it did. Thats a bug then. If you want to take a stab at it, go for it. I will need to fix this soon eitherway.

Think you'll need to decide which of setSelections / toggleSelection / removeSelection / addSelection trigger a re-focus & if it only happens when exactly 1 node is selected - I had all of them doing it at one point for my PR but backed it out incase they were being used internally & it'd trigger a focus twice

Now that Im thinking about this, I'm realizing there might be 3 different use cases:

Focus ( Focus a particular node but not select )
Selection by External ( click on something outside of the graph and it select )
Selection by System ( click on something inside the graph and it select )

There is a centerGraph method on the GraphCanvas which is the better way to handle this since the selection helper is optional and not required. If we did want to enhance this, we should put the methods there.
Let me think about this today. Feel free to post any thoughts in the meantime.

See https://github.com/reaviz/reagraph/pull/28#issuecomment-1186578682


FREEMARKER-211 [freemarker-generator] Build fails with JDK 17 LTS with " No XPath support is available"

Did this make it work for you? I've added the jaxen dependency to my project and I'm still getting the same error.


Hamming Distance (Leet code)

The Hamming distance between two integers is the number of positions at which the corresponding bits are different.
Given two integers x and y, return the Hamming distance between them.
I will use c++ in this question.
Porblem Link: https://leetcode.com/problems/hamming-distance/

@Amit-45  assign it to me.

@Ayush7614 Please assign it to me under LGMSOC. I will solve it in java.


The Signup Activity of this app(branch redesign) is not working..

Summary
The Signup Activity of this app(branch redesign) is not working.

Expected behaviour
Writing the remaining code on the signup .kt class.

Device and Android version
Infinix hot 7
Android 10

Short Video
Understand better after watching this short video.
https://user-images.githubusercontent.com/72181295/106356995-77d21e00-6329-11eb-9392-58699f36cf11.mp4

Can i work on this task.


Updates from AD

Updated app structure to support admin/public/common for parity to AD
Updated Web deps
Updated API deps that were throwing warnings
Added basic common utils and components from AD
Updated server scripts/webpack/etc with updates made for AD
Created basic boilerplate app for public app

OK I'm going to abandon this one in favor of https://github.com/dominiek/bedrock/pull/22
There's a few things in here I may pull over still but more piecemeal. The new PR doesn't mess with the directory structure which is for the best. Seemed like a good idea at the time but it's much better to not assume more than one site out of the box. The new stuff will have facilities for serving more than one webpack build but it will be all off by default. 👍


Calibrate Unit D Multisense, add automatic general setup to drc_environment.sh

Calibrated Multisense on Unit D
Added two Valkyrie robot.cfg files, one for unit D and one for unit C (presumably)
Added generalisation to drc_environment that will set up VAL_LINK_IP, VAL_ZELDA_IP, VAL_MULTISENSE_IP, and VAL_UNIT based on whether we are running at MIT or Edinburgh. These environment variables can then be used in robot.pmd and other files to avoid having to fork/duplicate files

This is cleaned up and should be ready to go. Incorporated our changes to the corresponding files from the ed-install-fixes branch.

This looks fine by me. happy to accept. uses the same approach as Pat wrote for Valkyrie


Improve performance by deprecating getCatalog, and preferring alternatives for accessing the catalog.

As https://github.com/dapperlabs/nft-catalog/pull/135 points out, we need a way to deprecate use of the getCatalog function, and have all consumers update to a new way to pull all information from the catalog because as the catalog grows, we are nearing execution limits from simply calling getCatalog() because it copies over the entire catalog map to the memory of the executing script or transaction rather than using the map reference.
I think the best way to do this will be to add support for new ways to iterate over the the catalog similar to how the following PR does https://github.com/dapperlabs/nft-catalog/pull/135
But, rather than entirely removing the getCatalog function, we can make a snapshot of the current catalog, and have the getCatalog function return the snapshotted catalog. This will make it so all existing use of getCatalog will not be affected short-term, and allows for all consumers to switch to the new functions to iterate over the catalog in a way that will not reach execution limits.

https://github.com/dapperlabs/nft-catalog/pull/139 has been pushed to provide an alternative to the existing getCatalog, and provide updates to all of our script usage.
https://github.com/dapperlabs/nft-catalog/pull/140 is open to provide a way to snapshot the current catalog, so we can officially replace getCatalog with a snapshotted version of the catalog.
I'm thinking next Friday, 3/31, we can run the updateSnapshot to stop continuing catalog updates to the getCatalog function.
Any consumers would ideally update their catalog usage before 3/31, and if they do not, then they will not receive future catalog updates until they do update. This is subject to change, pinging some catalog consumers to review, please comment here if you have any suggestions/alternatives!

Ran into some issues when testing the snapshotting of the catalog on testnet, looking into it but in the meantime, the snapshot has not happened yet as planned for today, and I'll give a new update soon.

New alternative approach which should not run into execution limits when snapshotting on testnet or mainnet: https://github.com/onflow/nft-catalog/pull/143
After review, will likely run the snapshot in testnet tomorrow 4/7, and on Mainnet on Monday 4/10, which afterwards will mean that getCatalog is returning a snapshotted version of the catalog.


destroyPlayer() does not remove player in cache

After looking at the source code of the lavalink-client, the LavalinkPlayer should be remove from cache after calling destoryPlayer() method
But it don't.

Source code

Please don't use the weirdly named methods. You're supposed to use getPlayer() as that will check the cache first

Please don't use the weirdly named methods. You're supposed to use getPlayer() as that will check the cache first

I don't want it to create a new player as getPlayer() will create a new player if the cache is not exist
Why I want to create a new player if I already want to destory the player if exist.....

Please don't use the weirdly named methods. You're supposed to use getPlayer() as that will check the cache first

I don't want it to create a new player as getPlayer() will create a new player if the cache is not exist Why I want to create a new player if I already want to destory the player if exist.....

Then there seems to be a flaw in your code. If you just want to destroy the player, there is no need to check if it exists. Also, if you are using the JDA or D4J integration, the player will be automatically destroyed when your bot leaves the voice channel

Im not using either JDA or D4J, Im making a global service handling all of my track stuff (Web controller/Bot/Other)
Im doing the check because I want to know if the player is exist previously, if Im using node.destroyPlayer(), I will not know that.
It also send a destroy rest action which it add extra latency to my service.

You can always check the connected status on the link. If it is connected, you have a player. Also, if you call subscribe() instead of block() te request will be done asynchronously, meaning that there is no added latency

Either way, this was fixed in version 2.2.0


Unknown field "x" on object type "Query". Did you mean "x"?

Before you create the issue

Please search the existing issues at https://github.com/jimkyndemeyer/js-graphql-intellij-plugin/issues?q=sort%3Aupdated-desc before opening a new issue.
Read the FAQ at https://github.com/jimkyndemeyer/js-graphql-intellij-plugin#faq
If you're using an EAP, try with a stable version. If that works, please mention it in the issue along with both versions.

Version and Environment Details
Operation system: Windows 10 1909
IDE name and version: IntelliJ IDEA 2019.3 (Ultimate Edition)
Build #IU-193.5233.102, built on November 28, 2019
Plugin version: 2.3.0
Expected Behaviour
No Errors
Actual Behaviour
Red Words
Steps to Reproduce / Link to Repo with Reproduction and Instructions
schema.json
schema.zip

Hi,
Thanks for using the plugin.
JSON files are not automatically included in schema discovery as they may contain non-graphql content.
You have to set your schemaPath to your schema.json file in a .graphqlconfig file. The quickest way to create the config file is right click, "New", "GraphQL Configuration File" in your project root. Then update the schemaPath.
The plugin should then pick up your Query type and the fields should not longer be shown as unresolved/red.
If you're doing introspection to fetch the schema from an endpoint, I recommend you follow the setup in https://github.com/jimkyndemeyer/graphql-config-examples/tree/master/remote-schema-introspection as this performs better in the plugin than using a JSON file.

Thanks.
I tried to fetch the schema with this plugin and the new file works well.
The schema.json file I've attached above was generated by apollo client, and one difference between two file as I observed is, that the new file wraps "__schema" with "data" while in my old file the "__schema" is a top field.
But I'm still confused of the advice when I using my old file.

Since it looks ok in this pane.


Database connections being left open

Python: 3.6.5
Django: 2.1.2
sentry-sdk: 0.4.2
I upgraded from raven to sentry-sdk yesterday and noticed my app was now keeping database connections open rather than closing them per request.
I wrote a tiny view to test this:
def sentry_test(request):
    """Return the password reset form."""
    from django.contrib.auth.models import User
    print(User.objects.all())
    return HttpResponse("
Hello
")

and when GET the URL repeatedly I see the subsequent connections remain idle after the request has completed:
=> SELECT state, query FROM pg_stat_activity WHERE client_addr IS NOT NULL;
 state  | query
--------+-----------------------------------------------------
 idle   | SELECT "auth_user"."id", "auth_user"."password", ...
 active | SELECT state, query FROM pg_stat_activity WHERE client_addr IS NOT NULL;
 idle   | SELECT "auth_user"."id", "auth_user"."password", ...
 idle   | SELECT "django_migrations"."app", "django_migrations"."name" FROM "django_migrations"
 idle   | SELECT "auth_user"."id", "auth_user"."password", ...
 idle   | SELECT "auth_user"."id", "auth_user"."password", ...
 idle   | SELECT "auth_user"."id", "auth_user"."password", ...

As you can see the connection from the migrate query on startup has remained open too.
If I use raven this behaviour is not exhibited.
I suspect this won't help much, but if I use sentry-sdk and comment out:
        WSGIHandler.__call__ = sentry_patched_wsgi_handler

Then the problem doesn't manifest itself but I've basically turned off sentry at this stage.

Thank you, I can reproduce this. Apparently sentry_sdk.integrations._wsgi._ScopePoppingResponse.close does not behave correctly.

@untitaker yeah when it's converted to an iterator it's no longer a response so no close method. I hacked this together to run a quick test:
@implements_iterator
class _ScopePoppingResponse(object):
    __slots__ = ("_response", "_hub", "_original_response")

    def __init__(self, hub, response):
        self._hub = hub
        self._response = response
        self._original_response = response

    def __iter__(self):
        try:
            self._response = iter(self._response)
        except Exception:
            reraise(*_capture_exception(self.hub))
        return self

    def __next__(self):
        try:
            return next(self._response)
        except StopIteration:
            raise
        except Exception:
            reraise(*_capture_exception(self.hub))

    def close(self):
        self._hub.pop_scope_unsafe()
        if hasattr(self._original_response, "close"):
            try:
                self._original_response.close()
            except Exception:
                reraise(*_capture_exception(self.hub))

and the connection was closed correctly.

Thanks for the separate investigation :) will release asap

0.4.3 is otu

Tested and verified connections are being closed correctly.


Package updates

Update meta data.
Upgrade dependencies.
Add build script.

Seems fine to me for now; was still able to get syntax highlighted for .uno, .unoproj and .ux


resurrect blender

Type of Change

[x] New example
[x] Example updates (Bug fixes, new features, etc.)

Checklist

[x] Example is testable in synthetic monitoring system, or lambda-test: false is added to example frontmatter (---)

[x] Example is tested by executing with modal run or an alternative cmd is provided in the example frontmatter (e.g. cmd: ["modal", "deploy"])
[x] Example is tested by running with no arguments or the args are provided in the example frontmatter (e.g. args: ["--prompt", "Formula for room temperature superconductor:"]

[x] Example is documented with comments throughout, in a Literate Programming style.
[x] Example does not require third-party dependencies to be installed locally
[x] Example pins its dependencies

[x] Example pins container images to a stable tag, not a dynamic tag like latest
[x] Example specifies a python_version for the base image, if it is used
[x] Example pins all dependencies to at least minor version, ~=x.y.z or ==x.y
[x] Example dependencies with version < 1 are pinned to patch version, ==0.y.z

running into issues trying to test the rendering in frontend on my devbox, so I'm just going to optimistically merge and check back in later


Refactor deployment detail component

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

/approve


[BUG] Flutter building apk: Duplicate class found in modules guava-26.0-android.jar and listenablefuture-1.0.jar

Describe the bug
youtube_player_flutter version 7.0.0+7 was giving this error when I tried building the apk:
% flutter build apk --no-shrink
You are building a fat APK that includes binaries for android-arm, android-arm64, android-x64.
If you are deploying the app to the Play Store, it's recommended to use app bundles or split the APK to reduce the APK size.
    To generate an app bundle, run:
        flutter build appbundle --target-platform android-arm,android-arm64,android-x64
        Learn more on: https://developer.android.com/guide/app-bundle
    To split the APKs per ABI, run:
        flutter build apk --target-platform android-arm,android-arm64,android-x64 --split-per-abi
        Learn more on:  https://developer.android.com/studio/build/configure-apk-splits#configure-abi-split

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:checkReleaseDuplicateClasses'.
> 1 exception was raised by workers:
  java.lang.RuntimeException: Duplicate class com.google.common.util.concurrent.ListenableFuture found in modules guava-26.0-android.jar (com.google.guava:guava:26.0-android) and listenablefuture-1.0.jar (com.google.guava:listenablefuture:1.0)

  Go to the documentation to learn how to Fix dependency resolution errors.

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 29s
Running Gradle task 'assembleRelease'...
Running Gradle task 'assembleRelease'... Done                      30.3s
Gradle task assembleRelease failed with exit code 1
joao@Mini-de-Joao opengames % flutter build apk --no-shrink
You are building a fat APK that includes binaries for android-arm, android-arm64, android-x64.
If you are deploying the app to the Play Store, it's recommended to use app bundles or split the APK to reduce the APK size.
    To generate an app bundle, run:
        flutter build appbundle --target-platform android-arm,android-arm64,android-x64
        Learn more on: https://developer.android.com/guide/app-bundle
    To split the APKs per ABI, run:
        flutter build apk --target-platform android-arm,android-arm64,android-x64 --split-per-abi
        Learn more on:  https://developer.android.com/studio/build/configure-apk-splits#configure-abi-split

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:checkReleaseDuplicateClasses'.
> 1 exception was raised by workers:
  java.lang.RuntimeException: Duplicate class com.google.common.util.concurrent.ListenableFuture found in modules guava-26.0-android.jar (com.google.guava:guava:26.0-android) and listenablefuture-1.0.jar (com.google.guava:listenablefuture:1.0)

I solved it by downgrading youtube flutter to version: 6.1.1

Same issue here when starting in debug mode.
Launching lib/main.dart on Android SDK built for x86 64 in debug mode...

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:checkDebugDuplicateClasses'.
> 1 exception was raised by workers:
  java.lang.RuntimeException: java.lang.RuntimeException: Duplicate class com.google.common.util.concurrent.ListenableFuture found in modules guava-26.0-android.jar (com.google.guava:guava:26.0-android) and listenablefuture-1.0.jar (com.google.guava:listenablefuture:1.0)

  Go to the documentation to learn how to Fix dependency resolution errors.

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 11s
Exception: Gradle task assembleDebug failed with exit code 1

@morphy2k were you able to solve this?

I had the same problem and solved it [https://stackoverflow.com/questions/56639529/duplicate-class-com-google-common-util-concurrent-listenablefuture-found-in-modu]

Thank you so much @apps-auth


Client/Server: Changes to 'number of terminals'

Is your feature request related to a problem? Please describe.
As of today we enforce that the end user specify an amount (1 - N) of a given terminal type when adding it to an aspect object.
This limitation no longer matches the problem domain and therefore we need to adjust how we specify the quantities of these.
Describe the solution you'd like
Function aspect:
By default infinite.

No limits/amount when adding a terminal

Product aspect:
By default infinite.

User can check "limit" and specify max amount

Describe alternatives you've considered
None.
Additional context
None.

Implemented needed backend changes on 'your' feature branch @Fruithut inkluding model changes (NodeTerminalLibAm/Dm/Cm)


[CSS Bug]: highlighted extra space should be reduced in Auto Split PDF

Installation Method
Docker
The Problem

Version of Stirling-PDF
0.28.3
Last Working Version of Stirling-PDF
No response
Page Where the Problem Occurred
http://localhost:8080/auto-split-pdf
Docker Configuration
No response
Relevant Log Output
No response
Additional Information
No response
Browsers Affected
Chrome
No Duplicate of the Issue

[X] I have verified that there are no existing issues raised related to my problem.

resolved


[Feature] Request to 8-bit Quantization of Attention with SageAttention

Checklist

[X] 1. If the issue you raised is not a feature but a question, please raise a discussion at https://github.com/sgl-project/sglang/discussions/new/choose Otherwise, it will be closed.
[X] 2. Please use English, otherwise it will be closed.

Motivation
As https://github.com/thu-ml/SageAttention mentioned, the quantized 8-bit attention will improvement the speed of inference about 2x and more with the same accuracy, so shall we give it a try or do some verification?
Related resources
github: https://github.com/thu-ml/SageAttention

contributions are welcome

I'd like to work on this issue. This would be my first open-source contribution, and I'm eager to learn and help out. Please let me know if you have any specific guidelines I should follow.


Getting reports of HTTP/3 broken in Safari/iOS 18

We're getting reports of HTTP/3 connection errors in Safari on iOS 18. I don't have this to test, but the only details I have so far are that the error is "connection reset" or "They just randomly timeout or only load half of the page content":
https://caddy.community/t/caddy-http-3-quic-and-the-new-apple-software-today/25660
I'll try to find out more, and I have no idea if this is something wrong with Safari/iOS, Caddy, or some quirk in quic-go. But I wanted to open this issue to track it.

Thanks for the report. I'm on it.
Struggling to reproduce it though. On macOS, Safari refuses to use H3 in the first place. Next up: testing on iOS devices.

According to the discussion on caddy.community, it looks like this has been resolved by updating quic-go to the most recent version.


Reduce size of swagger-ui folder

Goal: Reduce size from 8 Mb to less than 2 Mb

Remove *.map files
Remove source JS not used (bundle version is enough)

The folder is now 1.4 MB.


Error in introduction section of "The Streight Dope"

Greetings,
I am not sure if this is the correct place or format to log this comment, so please advise.  I am just beginning to consume your notebook on deep learning and am eager to both learn and contribute. Great work!
In the Introduction / Models you give an example of a loss function based on predicting the stock price of AMZN.  This is fine, however there is a comment about being long or short on the stock which actually has nothing to do with the accuracy in predicting the stock price;  that you may have a long or short position on the stock is irrelevant to the accuracy of the model in predicting the stock price.
Kind regards,
Dave

Also, the picture labeled "Assume that you find this beautiful mushroom in your backyard:" is of a cat. A later comment is made that "as any botanist would tell us, the above actually is a death cap". No, it's cat.

Lol there's a mystery here. The right picture shows in GitHub but the wrong picture shows in the html render.

:-)

Also, there is an error / typo in stating the max probability in this example:
notebook: Basic probability theory
section: Basic probability theory
example:
In [71]:
totals / 1000
Out[71]:
[ 0.168       0.15800001  0.15099999  0.155       0.18799999  0.18000001]
<NDArray 6 @cpu(0)>
The text states:
"As you can see, the lowest estimated probability for any of the numbers is about .15 and the highest estimated probability is 1.86.  "
The above " highest estimated probability is ... " should read 0.188  (not 1.86)

Ah yeah. Failed to set random seed, so you get different values every time
it's run. I'll fix it. Thanks.
On Sep 22, 2017 9:39 AM, "Dave Fisk"<EMAIL_ADDRESS>wrote:

Also, there is an error / typo in stating the max probability in this
example:
notebook: Basic probability theory
section: Basic probability theory
example:
In [71]:
totals / 1000
Out[71]:
[ 0.168 0.15800001 0.15099999 0.155 0.18799999 0.18000001]
<NDArray 6 @cpu https://github.com/cpu(0)>
The text states:
"As you can see, the lowest estimated probability for any of the numbers
is about .15 and the highest estimated probability is 1.86. "
The above " highest estimated probability is ... " should read 0.188 (not
1.86)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/zackchase/mxnet-the-straight-dope/issues/228#issuecomment-331497931,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACR4zjwOFh_L7i82XiiPBMwz7ZtCRj-zks5sk-K1gaJpZM4PcqVA
.

Regarding the mushroom cat issue, in the introduction the cats / dogs example has two death cap mushrooms and a box of matches where a cat should be!


TextButton widget not correctly positioned on Android and IOS

I tried to use a Positioned widget around a TextButton at 0.0 but it does not draw the widget on the top-left of the parent container. Even with a CustomMultiChildLayout approach by using its custom layout delegate, the position 0.0 shows some transparent space between the top of the widget and its container.
Same thing using other widget (container, text ...) is fine.
Fun fact : there is no issue on web, only on Android (I could not test on iOS so I don't know if the issue is present there)
Steps to Reproduce
Please use this code (with a Stack container) :

code sample
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return const MaterialApp(
      title: 'Wrong TextButton Offset',
      home: TestPage(),
      debugShowCheckedModeBanner: false,
    );
  }
}

class TestPage extends StatelessWidget {
  const TestPage({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(),
      body: Stack(
        children: [
          Positioned(
            top: 0,
            left: 0,
            child: TextButton(
              style: ButtonStyle(
                  backgroundColor: MaterialStateProperty.all(Colors.red)),
              child: Text(
                "Not Fine",
                style: TextStyle(backgroundColor: Colors.white.withAlpha(150)),
              ),
              onPressed: () {},
            ),
          ),
          const Positioned(
            top: 0,
            left: 0,
            child: Text(
              "Fine",
              style: TextStyle(backgroundColor: Colors.amber),
            ),
          ),
        ],
      ),
    );
  }
}

Or you can use this code too (using a CustomMultiChildLayout) :

import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return const MaterialApp(
      title: 'Wrong TextButton Offset',
      home: TestPage(),
      debugShowCheckedModeBanner: false,
    );
  }
}

class TestPage extends StatelessWidget {
  const TestPage({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(),
      body: CustomMultiChildLayout(
        delegate: MyLayoutDelegate(),
        children: [
          LayoutId(
            id: 1,
            child: TextButton(
              style: ButtonStyle(
                  backgroundColor: MaterialStateProperty.all(Colors.red)),
              child: Text(
                "Not Fine",
                style: TextStyle(backgroundColor: Colors.white.withAlpha(150)),
              ),
              onPressed: () {},
            ),
          ),
          LayoutId(
            id: 2,
            child: const Text(
              "Fine",
              style: TextStyle(backgroundColor: Colors.amber),
            ),
          ),
        ],
      ),
    );
  }
}

class MyLayoutDelegate extends MultiChildLayoutDelegate {
  @override
  void performLayout(Size size) {
    if (hasChild(1)) {
      var childSize = layoutChild(
        1,
        BoxConstraints.loose(size),
      );
      positionChild(
        1,
        Offset.zero,
      );
      print("widget 1: $childSize");
    }

    if (hasChild(2)) {
      var childSize = layoutChild(
        2,
        BoxConstraints.loose(size),
      );
      positionChild(
        2,
        Offset.zero,
      );
      print("widget 2: $childSize");
    }
  }

  @override
  bool shouldRelayout(covariant MultiChildLayoutDelegate oldDelegate) {
    return true;
  }
}

Expected results:
The TextButton must be rendered at the top left of its container like the Text widget.
Actual results:
There is a gap between the top of the TextButton widget and its parent (Stack).

flutter doctor -v

[✓] Flutter (Channel master, 2.5.0-7.0.pre.131, on Pop!_OS 21.04
    5.12.19-051219-generic, locale en_US.UTF-8)
    • Flutter version 2.5.0-7.0.pre.131 at /home/aurechan/workspace/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 061dff8ca7 (6 hours ago), 2021-08-18 04:07:02 -0400
    • Engine revision 99df75c44e
    • Dart version 2.15.0 (build 2.15.0-25.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /home/aurechan/Android/Sdk
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /home/aurechan/Android/Sdk
    • Java binary at: /usr/lib/jvm/java-8-openjdk-amd64/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_292-8u292-b10-0ubuntu1-b10)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[!] Android Studio (not installed)
    • Android Studio not found; download from
      https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/linux#android-setup
      for detailed instructions).

[✓] VS Code (version 1.59.0)
    • VS Code at /usr/share/code
    • Flutter extension version 3.25.0

[✓] Connected device (2 available)
    • SM N770F (mobile) • RF8N51KM3SN • android-arm64  • Android 11 (API 30)
    • Chrome (web)      • chrome      • web-javascript • Google Chrome
      92.0.4515.131

Hi @AurelienKun,  Thanks for filing the issue, I am able to reproduce the issue on the latest stable and the master channel on Android and IOS.
Output

Android
IOS
MacOS
Web

❌
❌
✅
✅

stable
master

❌
❌

✅ : No Issue ❌: Issue reproduced

flutter doctor -v
[✓] Flutter (Channel stable, 2.2.3, on macOS 11.5.2 20G95 darwin-arm, locale en-GB)
    • Flutter version 2.2.3 at /Users/mahesh/Documents/flutter
    • Framework revision f4abaa0735 (6 weeks ago), 2021-07-01 12:46:11 -0700
    • Engine revision 241c87ad80
    • Dart version 2.13.4

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.5.1, Build version 12E507
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] VS Code (version 1.58.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.25.0

[✓] Connected device (4 available)
    • iPhone 12 Pro Max (mobile) • 5430F628-A903-402C-A635-D55D7E6EEFCA • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-14-5 (simulator)
    • macOS (desktop)            • macos                                • darwin-arm64   • macOS 11.5.2
      20G95 darwin-arm
    • Chrome (web)               • chrome                               • web-javascript • Google Chrome
      92.0.4515.131

• No issues found!

[✓] Flutter (Channel master, 2.5.0-7.0.pre.8, on macOS 11.5.2 20G95 darwin-arm, locale en-GB)
    • Flutter version 2.5.0-7.0.pre.8 at /Users/mahesh/Documents/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 40e8620a27 (7 days ago), 2021-08-06 09:14:50 +0430
    • Engine revision 431ac604da
    • Dart version 2.14.0 (build 2.14.0-383.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.5.1, Build version 12E507
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] VS Code (version 1.58.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.25.0

[✓] Connected device (4 available)
    • iPhone 12 Pro Max (mobile) • 5430F628-A903-402C-A635-D55D7E6EEFCA • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-14-5 (simulator)
    • macOS (desktop)            • macos                                • darwin-arm64   • macOS 11.5.2
      20G95 darwin-arm
    • Chrome (web)               • chrome                               • web-javascript • Google Chrome
      92.0.4515.131

• No issues found!

The reason this happens is that the default minimum size for TextButtons is 64x36 per https://api.flutter.dev/flutter/material/TextButton/defaultStyleOf.html. The minimum size ("tap target size") for any interactive widget on a touch screen is 48x48, so on Android and iOS the height ofTextButtons are padded to 48. That's why you're seeing the gap. On desktop the default visualDensity (see https://api.flutter.dev/flutter/material/ThemeData/visualDensity.html) effectively eliminates the extra padding.
I realize that this is pretty confusing. On the upside, it doesn't seem to come up that often.
You can avoid the default minimum 48x48 tap target size by specifying tapTargetSize:
TextButton(
  style: ButtonStyle(
    backgroundColor: MaterialStateProperty.all(Colors.red),
    tapTargetSize: MaterialTapTargetSize.shrinkWrap,
  ),
  child: Text('Fine'),
  onPressed: () {},
)


Add past student proposals and final work product submissions

Adding links to proposals and final work product submissions by past GSoC students would prove useful to the newer cohort.
Final work product submissions are public anyway, so IMO it should be fine to include them. Proposals aren't public by default so we can make it optional and students could PR a link to it if they want. :)
Thoughts, @idvoretskyi @caniszczyk?

Would submissions and proposals of community bridge students be also included in this?

Would submissions and proposals of community bridge students be also included in this?

I think we can do that! :)
We can create gsoc and community-bridge subdirectories under a 2019 directory or something like that.

@nikhita @palash25 let's keep this repo GSoC-focused.
We may create a dedicated repo for the Community Bridge process and documents later if needed. Please, don't forget that we're CNCF's participation the Community Bridge is in the pilot stage at the moment.

Hi @nikhita @idvoretskyi, actually there is a repo called gsoc-cn, which contains proposals and some introductions of the different orgs from Chinese GSoC students . Although the introductions are in Chinese, all the proposals are in English. I think we can use a similar structure for this like: https://github.com/gsoc-cn/gsoc-cn/tree/master/resources


Drop OpenSSL

Replace HsOpenSSL with tls.
This replaces the dependency on OpenSSL with a pure Haskell based
version. Also adds tfausak/wuss, which abstracts away the secure
websocket initialization.
Fixes #62

Added min bounds. Let me know if there are further additional requirements.


Profiling

I have discussed this with @jorisv and @fabinsch a bit.
The idea is that we can use a more advanced code profiler such as tracy (which requires instrumenting the code) to better assess performance bottlenecks in some critical (sub-millisecond) pieces of code with high accuracy.

We now have a repo of tracy with our own patches: https://github.com/Simple-Robotics/tracy/tree/patches
Currently trying it out internally


Add model cache_dir config to Baskerville scripts and related config

Added model cacheing support by:

Including a new <k,v> pair in the 'top config' file;
Updating the training script generator to include this; and
Updating the load_model_and_tokenizer function to pass it in as part of the model_kwargs expected by the transformers AutoModelForCausalLM.from_pretrained method.

@msmoore Looks good to me (pending giving it a try)! These are secondary as they're much smaller files but would be good to do these too:

adding cache_dir as an arg (if present in model_kwargs) to the AutoTokenizer.from_pretrained call in arcsf.models.model.load_model_and_tokenizer
Setting wandb cache directories - these are the two env variables that need to be set (link to another project, see lines 108-109), and can be done in arcsf.config.experiment.ExperimentConfig.init_wandb anywhere before the wandb.init call.

I realised everything can be set with env variables which I'm hoping will avoid needing to make any other tweaks to the code.


Added Filter Questions by Difficulty Feature

Added DifficultyFilter component under QnFilter
Added the DifficultyFilter component to BankPage

![Screenshot 2023-10-28 at 11 12 17 PM](https://github.com/CS3219-AY2324S1/ay2324s1-course-assessment-g10/assets/97405333/7de79c47-edb8-4201-a249-43cf2717a9bc)

don't just commit a "delete". Instead, please look up how to restore a file to before a particular point in commit history
https://dev.to/lofiandcode/git-and-github-how-to-revert-a-single-file-dha


" instead of '

[Enter feedback here]
Hy, it works on my tenant only with " instead of '
Is that an typo or depend it on the tenant language?

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

ID: a30c67d5-6cc0-16f3-db2a-8244d7344aa3
Version Independent ID: ab230d40-68c6-3a1f-c4c3-12d4a76e42df
Content: Group membership for Azure AD dynamic groups with memberOf - Azure AD - Microsoft Entra
Content Source: articles/active-directory/enterprise-users/groups-dynamic-rule-member-of.md
Service: active-directory
Sub-service: enterprise-users
GitHub Login: @billmath
Microsoft Alias: billmath

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

@neozoen
Could you please elaborate the query for better understanding. please help with doc screenshot, So, we can assist you further.

Hy, it seams I was a little bit to fast with that post und I had an typo. Now both works:
user.memberof -any (group.objectId -in ['88888888-xxxx-bbbb-yyyy-12345678', '11111111-xxxx-aaaa-yyyy-121212121212']) and
user.memberof -any (group.objectId -in ["88888888-xxxx-bbbb-yyyy-12345678", "11111111-xxxx-aaaa-yyyy-121212121212'7"])
Sorry, my fault

@neozoen
We are going to close this thread as resolved but if there are any further questions, please tag me in your reply and we will be happy to continue the conversation.


status 200 when using set_exception_handler

I'm using rollbar with great success but today I've noticed something that bothers me.
My code is using exception to report invalid GET parameters.
In dev, when I call an invalid URL, I get a 500 error and the error is logged by php.
In prod, where rollbar is enabled, I obtain a blank page with a 200 status.
Having a 200 status can be an issue for a number of reasons (for example because it means a search engine could index pages whose URL is invalid).
In my opinion, most people would like the server to return a 5XX status, so it makes sense to implement this in rollbar's error_handler.
Do you think that's a good idea ?

Hey there, sorry for the long delay!
I don't think we're purposely setting the status to 200. On my end I still get a 500 when I report something to Rollbar.
Are you sure this is the application and not your server in production? (Apache can easily do stuff like this, for example).

Seeing this exact same behaviour too after switching to php7.
It's definitely a rollbar related problem because when rollbar is not enabled everything works as expected.

See also https://github.com/getsentry/sentry-php/issues/280 for what seems like the same issue.

This is ready for review in PR #143

Fixed in #143


end the stream when the websocket closes

When the remote end closes, there's no notification at the server side of the connection right now. This patch ends the stream when the remote end closes.
It might also be a good idea to have a close event to handle cleanup and such.

Hmm. This should already work (close should be emitted when the stream is destroyed). Could you add a test case for this?

Test added!

Released in 1.5.0


object() constructor

In my Python program I have:
ANY = object()

Transcrypt translates this to:
export var ANY = object ();

But the script won't execute.  My browsers (Firefox 109.0.1, Safari 15.6.1) apparently don't recognize "object" as a JavaScript function.  Something missing from the Transcript-generated runtime perhaps?

I should have said: that's perfectly good Python, and the Python program has worked fine for years.

Not sure I understand what your desired end result is. Are you trying to create a bare JavaScript object? If so, you can try using window.Object() which would be more explicit and might have a better chance of being transpiled correctly. You may also have to use the Transcrypt __new__() function in this case as well.
Issue #665 may provide some more clues, and also check the docs on Creating bare JavaScript objects for another possible option.

I'm creating an atom, as in LISP or Erlang.  It 's just a marker.  It's an object that compares equal to itself and unequal to any other Python object using "is" or "==".  That's all it's for.
The proposed solutions are not more explicit, they are obfuscation.  I'm not doing anything with windows.  And I'm writing Python, not Javascript.
If I have to know about special cases that Transcrypt won't translate correctly, and write workarounds in which I have to think about the JavaScript translation, I won't use Transcrypt.

Sorry, I assumed you were translating Python to JS for use in a browser, window is a browser object that the JS Object() belongs to in that case and has nothing to do with the Windows OS.

Yes, I want to run the Python program in a browser window as if it were a
terminal window.
But the Python program will still be used in other environments, so I don't
want to put Transcrypt-specific code into it.
Thanks,
- Allan

On Mon, Feb 27, 2023 at 10:35 AM John S @.***> wrote:

Sorry, I assumed you were translating Python to JS for use in a browser,
window is a browser object that the JS Object() belongs to in that case
and has nothing to do with the Windows OS.
—
Reply to this email directly, view it on GitHub
https://github.com/QQuick/Transcrypt/issues/840#issuecomment-1446755136,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAU6EBN36W47SHPXMSCPZMTWZTQXPANCNFSM6AAAAAAU74XN7I
.
You are receiving this because you authored the thread.Message ID:
@.***>

Thanks!  I took a look at PyScript and I thought it was exactly what I
wanted.
But it doesn't do input() nicely.  It pops up a text entry box and grays
out the rest of the window.  I can type input lines, but I don't see the
program's responses to them until the program terminates!  I don't see a
dialog as I would if I were running the program in a terminal window.
From the PyScript documentation and what I can find with online searches, I
don't think I can get PyScript to behave differently.  (No, I'm not asking
for help with PyScript.)  So I'm abandoning it.
I may come back to Transcrypt some day and deal with the the workarounds I
need.  But for now I'm stepping away from my project for a while.
Anyway, many thanks for the help!
- Allan

On Mon, Feb 27, 2023 at 1:23 PM John S @.***> wrote:

Considering the differences in runtime environments, sometimes having
"Transcrypt-specific code" is unavoidable depending on what you are trying
to accomplish. This is one reason why Transcrypt has pragma notations
(compiler directives). But, even with these differences, you can definitely
create Python code that will work in both environments. It does sometimes
require coding workarounds to solve impedance mismatches between Python and
JavaScript though.
Specifically the # pragma ('ecom')
https://www.transcrypt.org/docs/html/special_facilities.html#skipping-transcrypt-code-fragments-when-running-with-cpython-pragma-ecom-and-pragma-noecom
directive for code that only needs to be used on the JS side and the #
pragma ('skip')
https://www.transcrypt.org/docs/html/special_facilities.html#skipping-fragments-while-generating-code-pragma-skip-and-pragma-noskip
directive for code that only needs to be used on the Python side. I
personally use the skip/noskip directive all the time to keep the Python
linter happy when referencing JS libraries in Python code. The Transcrypt
docs are actually pretty good IMO, but there is a lot there to digest and
involves a subsequent learning curve in order to get the most out of
Transcrypt.
TBH, if you are looking to stay 100% in Python and never have a need to
make any calls to JS libraries in your application, you might want to look
at using PyScript https://pyscript.net (WASM), as it might be a better
fit depending on where you need to deploy it, and if you can tolerate the
initial load time. There's also libraries like Brython and Skulpt that
might be potential alternate options. You just have to figure out which one
is best for your situation and priorities. This post by Anvil
https://yasoob.me/2019/05/22/running-python-in-the-browser/ is getting
a bit dated, but it helped me out a lot when I first decided that
Transcrypt was the best solution for me.
I'm not trying to push you away from using Transcrypt either. It's really
versatile and I absolutely love it despite whatever imperfections and
limitations it may currently have. But you sometimes have to dig into it a
little to achieve a specific result, and I recognize that it isn't always
the best solution for every situation.
—
Reply to this email directly, view it on GitHub
https://github.com/QQuick/Transcrypt/issues/840#issuecomment-1447030468,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAU6EBLZFPNPWKT2TSEGJBLWZUEK7ANCNFSM6AAAAAAU74XN7I
.
You are receiving this because you authored the thread.Message ID:
@.***>


Removed supports_snapshots? methods which were not using supportsFeatureMixin

Few files had "supports_snapshots?" methods defined as against using the supportsFeatureMixin to define which feature is supported and which is not.
Removed such instances and replaced them by "supports :snapshots"

@miq-bot assign @durandom

@miq-bot remove_label wip

@miq-bot add_labels euwe/no, pluggable providers, refactoring


Convert partners section of index template to animated cards

The card deck should be animated as directed by Aure.

Here is the commit that will be included in a future pull request. The transitions and overall design have been approved by Aure.
To Do
(1) Find better versions of the dark logos that don't look great with a dark background
(2) Get summary descriptions from Coleman for the cards that currently have placeholder text


[PCM-171] Allow sound to play if output device can't be set

Coverage increased (+1.5%) to 65.663% when pulling 56ef443c4773fb43fdab872c7674fdb9155ef084 on PCM-171-firefox-play-sound into 7e596e9a5a44082ca3e818f5f2a58713b9388576 on master.

Coverage increased (+2.3%) to 66.463% when pulling 56ef443c4773fb43fdab872c7674fdb9155ef084 on PCM-171-firefox-play-sound into 7e596e9a5a44082ca3e818f5f2a58713b9388576 on master.

Looks good.


Moved release note on #50254 (sql: remove constant folding visitor) to Backward-incompatible section

Fixes  #7857.

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/9c3e9a0f167d81801e2db9eb826176ba29adbec9/
Edited pages:

releases/v20.2.0-alpha.2.md


Cilium fails to attach xdp to bonded interface

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
I have 4 25 gbit mellanox NICs LACP bonded. Looks like the xdp program attempts to connect to the bond and fails.
level=fatal msg="Failed to compile XDP program" error="program cil_xdp_entry: attaching XDP program to interface bond0: invalid argument" subsys=datapath-loader
Looks similar to:
#25453
Cilium Version
v1.14.4
Kernel Version
6.5.0-14-generic #14~22.04.1-Ubuntu
Kubernetes Version
v1.29.0+rke2r1
Sysdump
No response
Relevant log output
No response
Anything else?
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Checking in here, I have read a few other reports, wondering if it has something to do with the device cilium is attaching to natively, which is not something you can specify in the helm chart. That's just a hypothesis though.

I can confirm this. Cilium is not able to handle bond interface properly. When i included bond0 interface in "devices:" section of cilium-config. External Gitlab Webhooks were not able to reach services inside kubernetes cluster. I was getting "Error: Net::OpenTimeout". As a workaround, i removed bond0 interface from cilium config and just entered node-IP interface and webhooks are working fine.

So, I have reachability working just fine, but there seems to be an issue particularly with XDP, and supposedly this is supported in the bonding driver. May have to do with LACP vs active-active, etc after looking under the covers in the linux bonding driver. Would be great to get some input from @dylandreimerink or somebody else from the Cilium team here.

The kernel bonding driver only supports XDP for a limited set of bond modes:
https://elixir.bootlin.com/linux/v6.7.2/source/drivers/net/bonding/bond_main.c#L327


Pack color and depth images into .bag

Required Info

Camera Model
D435i

Operating System & Version
Ubuntu 22.04

Kernel Version (Linux Only)
6.5.0-1025-oem

Platform
PC

Language
{C++/python}

Issue Description
Is it possible to pack color and depth images into a .bag file? I see there is a converter that extracts data from the bag, what the simplest way to make a reverse process?
I want to pack data that comes from another source into RS format, could you please advise how to so this? Is there any info in the bag package formats?

Hi @jenadzitsiuk  I hope that the link below will be helpful in regard to SDK bag structure.
https://github.com/IntelRealSense/librealsense/tree/master/src/media/ros
There are a couple of discussions about converting images into a bag file, though I have not previously seen it done with the RealSense SDK's bag format, which as some differences to the ROS rosbag format
https://stackoverflow.com/questions/31432870/how-do-i-convert-a-video-or-a-sequence-of-images-to-a-bag-file
https://stackoverflow.com/questions/38700271/python-converting-image-time-series-to-ros-bag
The link below discusses the RealSense SDK's bag format compared to the ROS rosbag format, and has a link to a rosbag parser project.
https://github.com/IntelRealSense/librealsense/issues/7845#issuecomment-732456123

Thank you for the links, I will take a look!

You are very welcome.  I'm pleased that I could help.  :)


Render THORChain swaps as trades

Overview
Currently, we have tx parsers to identify 0x and CoW trades and render them as a trade in the transaction history.
We want to do the same for THORChain trades.
References and additional details
references transactions for the thorchain trade below

https://etherscan.io/tx/0x86f4086ab59b105415bc2aaac226a4c7efdd939ab2eb8ea05ecf95428dfb75db
https://blockchair.com/bitcoin-cash/transaction/abe2a8cf3592dfae04624a7b7916f546c32452b85ff17d4b6f60d7129d88d52c

when implementing this, consider

some sort of generic "blockchain event" type interface that can support the concept of other activities, e.g. EPNS notifications in the same chronological view
a trade should be defined as an event that can contain 1 or more transactions, e.g. a 0x/cow swap is a single tx, a thorchain trade is two.
provision for osmosis trades must also be supported in your design architecture

leverage RTK query and hit the lcd endpoint on the node or midgard, to obtain trades a given address has done, probably don't do this in lib for performance reasons.
Acceptance Criteria

We can render THORChain trades in a single row
We dedupe them and avoid rendering multiple instances of the same trade

Need By Date
No response
Screenshots/Mockups
how we currently render 0x/cowswap trades

how we currently render thorchain trades

Estimated effort
No response

@twblack88 - can you comment on this? still something we want to explore or stale and should be closed?

@kaladinlight will be able to bring more context on this as this effectively requires linking THOR Txs between each other i.e a trade consisting of one/many (for UTXOs) inbound Txs, and an outbound Tx at unchained-client Tx parser level.
I'd say this is a major requirement, especially with the current outbound delays and is a key component in really allowing us to deliver the best THOR experience

There are levels to this idea, but I have a personal issue tracked related to getting the thorchain tx parsing updated for the current feature set as a starting point that I can create a formal issue for. This would at least be able to tag each transaction as Swap In, Swap Out, Deposit, etc.
The actual linking of transactions is a whole separate task. There is feasibility to this idea certainly and would improve our UX, so I think this is still worth exploring, but probably best updated to a spike ticket for starters

@kaladinlight do we want to close this since we already have unchained-client parsing of swap Txs, albeit without the linking of Txs which is indeed a whole different task (and effort)?

closing as the new UI makes it pretty clear but we don't need to add extra exapnding/collapsing .
Defer to users asking for that level of complexity.


need a new release that works with python 3.8

Now that https://github.com/opencomputeproject/ocp-diag-pci_lmt/pull/10 is in, there should be a release so users can download a package with this fix so that they can run on python 3.8

@sksekar can you make a new release? thanks


Add extra asgi option with starlette as a dependency.

Follow-up to #1022.
Description
asgi extra installs requirement starlette.
Types of Changes

[ ] Core
[ ] Bugfix
[ ] New feature
[x] Enhancement/optimization
[ ] Documentation

Issues Fixed or Closed by This PR

#1022

Checklist

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] I have tested the changes and verified that they work and don't break anything (as well as I can manage).

Thanks for adding the RELEASE.md file!

Here's a preview of the changelog:

Follow-up to #1022, by adding asgi as an extra.

@patrick91 is this ready to merge?


Fixed #9 Deploy website to heroku

The website is deployed on heroku --- > BlogSite

@Subhra264  You added your commits to the master branch, please next time avoid doing that.
Also run these commands then I'll merge.
git commit --amend -m "Create Procfile"
git push origin <branch name> -f

@Subhra264  You added your commits to the master branch, please next time avoid doing that.
Also run these commands then I'll merge.
git commit --amend -m "Create Procfile"
git push origin <branch name> -f


Refresh button in search

VSCode Version: 1.8.1
OS Version: Windows 10

The refresh button in search should be always enabled instead of an empty entry.
Steps to Reproduce:

Select the Search Tab
Write something
The refresh button is disabled

It only gets enabled when you hit enter, after that you can reproduce the steps below and it'll work. However if you clear the search result, the refresh button will be disabled until you hit enter again.
Thanks.

I'm not sure about this, it doesn't seem like 'refresh' should trigger a search from an empty page. Like how in a web browser, I can't 'refresh' until I've loaded something.

Exactly, when not empty it should enable the button as you can see in the image.


Added a simple PER CI test.

The new simple_per_test.yml failed because the workflow file is the new one but the script files it uses are not ready in the main branch.


Automator: update<EMAIL_ADDRESS>dependency in<EMAIL_ADDRESS>

Generated by Automator - 2023-01-23T16:17:08+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio_release-1.17
b344b1bfd7175e8c7f06d03e080b6b82836fb23d
link
true
/test integ-security-multicluster

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-security-multicluster


tcmalloc incompatible with macOS Monterey?

Previous to Monterey tcmalloc was incompatible with apps launched from the Dock.  As of Monterey tcmalloc causes apps to crash even when launched from the command line.  I've tried 2.7 and 2.9.1.
See also https://trac.macports.org/ticket/63242 which I assume is the same issue.
The resulting stack is pretty useless as far as I can tell, but here you go.
Process 56526 stopped
* thread #1, queue = 'com.apple.main-thread', stop reason = EXC_BAD_ACCESS (code=1, address=0x0)
    frame #0: 0x0000000000000000
error: memory read failed for 0x0
Target 0: (myapp) stopped.
(lldb) thread backtrace
* thread #1, queue = 'com.apple.main-thread', stop reason = EXC_BAD_ACCESS (code=1, address=0x0)
  * frame #0: 0x0000000000000000
    frame #1: 0x00007ff818508bb2 libobjc.A.dylib`load_images + 1358
    frame #2: 0x000000010006341c dyld`dyld4::RuntimeState::notifyObjCInit(dyld4::Loader const*) + 170
    frame #3: 0x0000000100068bfd dyld`dyld4::Loader::runInitializersBottomUp(dyld4::RuntimeState&, dyld3::Array<dyld4::Loader const*>&) const + 167
    frame #4: 0x0000000100068beb dyld`dyld4::Loader::runInitializersBottomUp(dyld4::RuntimeState&, dyld3::Array<dyld4::Loader const*>&) const + 149
    frame #5: 0x0000000100068beb dyld`dyld4::Loader::runInitializersBottomUp(dyld4::RuntimeState&, dyld3::Array<dyld4::Loader const*>&) const + 149
    frame #6: 0x0000000100068beb dyld`dyld4::Loader::runInitializersBottomUp(dyld4::RuntimeState&, dyld3::Array<dyld4::Loader const*>&) const + 149
    frame #7: 0x0000000100068cac dyld`dyld4::Loader::runInitializersBottomUpPlusUpwardLinks(dyld4::RuntimeState&) const + 108
    frame #8: 0x000000010007c32e dyld`dyld4::APIs::runAllInitializersForMain() + 222
    frame #9: 0x000000010005a358 dyld`dyld4::prepare(dyld4::APIs&, dyld3::MachOAnalyzer const*) + 3438
    frame #10: 0x00000001000594b4 dyld`start + 388

Confirmed with a c++ proprietary program, linked to log4cxx (https://logging.apache.org/log4cxx/latest_stable/). We're trying to set up a minimal reproducible example.

Attached a highly simplified application which replicates the crash using log4cxx as @natale-p suggested.  This simply uses log4cxx and instantiates a logger.  It crashes in the registerClass method when adding an element to the map for the ClassNamePatternConverter class.  This is the 11th entry being added to the map.

Install log4cxx from brew, from source with the link from natale-p, etc.
Uncompress on a Mac with Monterey.
'cd test/build'
'cmake ..'
'make'
'src/test'

test.zip

An even simpler version which does not require log4cxx.

Uncompress on a Mac with Monterey.
'cd test/build'
'cmake .. -DCMAKE_BUILD_TYPE=Debug'
'make'
'src/test'

test 2.zip

The root cause is a missing implementation for a macOS method that is documented as "optional" while is actually used in the code. For example, the following code:
#include <stdio.h>
#include <stdlib.h>

int main(int argc, char *argv[])
{
    void *ptr = malloc(128);
    printf("malloc(128) returned %p\n",ptr);
    free(ptr);
    return 0;
}

When linked with tcmalloc is crashing (see https://developer.apple.com/forums/thread/693430?answerId=695428022#695428022). With the patch proposed here (https://github.com/gperftools/gperftools/pull/1315) the code compiles & runs fine.

Having:
#include <stdio.h>
#include <stdlib.h>

int main(int argc, char *argv[])
{
    void *ptr = malloc(128);
    printf("malloc(128) returned %p\n",ptr);
    free(ptr);
    return 0;
}

compiles without issue

Hello, the issue is not in compiling, but when executing with the tcmalloc library.

I assume 852fb6df031560c5e353b497222b76394190e27d has the issue ?

Closing for now. Please reopen if you're still having trouble.