sts 156 vbump

This PR has the version bump for STS which has the below fixes

Target win-x64 instead of win7-x64
Bump version
Update descriptions & fix path
Target Framework upgrade for ManagedBatchParser proj according to DacFx_160 Nuget.

I have a small warning fix pushed to STS yesterday and since have 2 releases on STS, using the latest one if some other fixes needs to move to ADS! If not let me know I can only use the 155 version which has my fix! Thanks :)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
26997

Relevant Lines:
58921

💛  - Coveralls


Introduce docker build bits for mixer.

At the moment, building a docker image for mixer appears to only work on
linux machines. As such, these builds should only be conducted on linux
hosts.
This setup should closely align with the manager build images and push
images to the same istio-test repo.
This attempts to begin to address #162.

This change is

Current coverage is 70.41% (diff: 100%)

Merging #186 into master will not change coverage

@@             master       #186   diff @@
==========================================
  Files            37         37
  Lines          1359       1359
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            957        957
  Misses          381        381
  Partials         21         21

Powered by Codecov. Last update 7860b21...3390a74

PTAL.
Moved: docker/build.sh to docker/gcloud_build.sh

PTAL. Updated to xenial as the base image.


Message embed images broken

Describe the bug
When Sonia sends embed messages the icon is no longer visible.
To Reproduce
Send a message to Sonia which will trigger a message embed from her.
Expected behavior
The images should be visible on the top right corner.
Screenshots
https://puu.sh/FIBwy/1d01d6da2b.png

It was either a DoS from the images API or from Discord.
Nothing to do from our side.


Analyzer: implement changes to TOP_MERGE.

This pull request specifies a change in the definition of the NNBD_TOP_MERGE algorithm.  This is ready for implementations.  Tests for this change are landing here:

TOP_MERGE member override tests..
Test TOP_MERGE on super-interfaces..

The following tests are currently failing and should pass when the above change is implemented.
language/nnbd/top_merge/top_merge_interfaces_direct_error_test MissingCompileTimeError (expected Pass)
language/nnbd/top_merge/top_merge_interfaces_indirect_error_test MissingCompileTimeError (expected Pass)
language/nnbd/top_merge/top_merge_interfaces_mixin_error_test MissingCompileTimeError (expected Pass)
language/nnbd/top_merge/top_merge_members_direct_error_test MissingCompileTimeError (expected Pass)
language/nnbd/top_merge/top_merge_members_indirect_error_test MissingCompileTimeError (expected Pass)
language/nnbd/top_merge/top_merge_members_mixin_error_test MissingCompileTimeError (expected Pass)

cc @scheglov @stereotype441

I have an issue with language/nnbd/top_merge/top_merge_interfaces_indirect_error_test.
The code:
class A<T> {}

class B<T> extends A<T> {}

class Merge<S, T> extends A<S> implements B<T> {}

class D0 extends Merge<dynamic, Object?> {}

We report an error on class Merge: error: The class 'Merge' cannot implement both 'A<S>' and 'A<T>' because the type arguments are different. (conflicting_generic_interfaces at [test] bin/test.dart:13). And the list of implemented interfaces is fixed on the class level - once we determined that Merge implements A<S> - it does so for all uses.
At the point where we then use class Merge, it does not matter that in this specific case arguments for T and S are compatible.
@leafpetersen It looks to me that the spec speaks about classes as well: a class C in an opted-in library implements the same generic class I more than once.... So, is this a mistake in the test, or the spec means something else, or I don't understand something?

@scheglov You are correct, the design of that test is broken.  I'll revisit it, thanks for catching.  cc @johnniwinther

Proposed test fix here

https://dart-review.googlesource.com/c/sdk/+/143060 will fix language/nnbd/top_merge/top_merge_members_mixin_error_test.


merge package from arlac77/npm-package-template

package.json

chore(package<EMAIL_ADDRESS>chore(scripts): cover@#overwrite c8 --temp-directory build/coverage ava && c8 report -r lcov --temp-directory build/coverage
chore(package): add xo from template
chore(package): add nyc from template
chore(package): set $.ava.require='esm' as in template
chore(package): set $.ava.files='tests/-test.js,tests/-test.mjs' as in template
chore(package): set $.ava.extensions='js,mjs' as in template

Codecov Report

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

@@           Coverage Diff           @@
##           master     #319   +/-   ##
=======================================
  Coverage   84.11%   84.11%
=======================================
  Files           1        1
  Lines         214      214
=======================================
  Hits          180      180
  Misses         34       34

Continue to review full report at Codecov.

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

Coverage remained the same at 84.81% when pulling db1fae9155eeacae31d1bc338e238f0d53f01445 on template-sync-1 into ff3d02ab687af39652732d9d4f830904b7754216 on master.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Support for Statamic 3.4

Hi folks! Just wondering if support for Statamic 3.4 will take long or is it just turning the corner?

I haven't come across anything that doesn't work in 3.4 but its always worth upgrading in a test/staging environment to iron out any pitfalls.
If you run into anything open issues with the specific errors so they can be looked into.

There should really be anything needed apart from adjusting the composer constraint. Updating this package is on my todo list.

@jasonvarga You're probably right. When I tried to upgrade Statamic, composer stopped me because the eloquent-driver had the older version as a dependency.


Update lodash.merge version for fixing vulnerability

There is a vulnerability in lodash.merge package. Plase update your lodash.merge dependency version to 4.6.2.
Dependabot already did that for you. You can simple release a new npm version for your package.
Thank you.

Published 3.3.0. Thanks


Dead readme links

A bunch of links in readme.md are dead. Seems like all of the ones referring to http://restkit.org/* redirects back to GitHub (https://github.com/RestKit/RestKit). I can't find any API docs, which should theoretically live on http://restkit.org/api/latest, but that also redirects back to GitHub.

Yup, the website is down. Feel free to check the docs on cocoadocs.org, though!

Will the website be back up or should the links be redirected to cocoadocs?

If appreciate a PR redirecting them!
-Samuel E. Giddins
On Jun 7, 2015, at 8:32 PM, mozeryansky<EMAIL_ADDRESS>wrote:
Will the website be back up or should the links be redirected to cocoadocs?
—
Reply to this email directly or view it on GitHub.

Looking around, I see that links to classes won't work. I even found a comment you made on appledocs git issues referencing this: https://github.com/CocoaPods/cocoadocs.org/issues/136.
I'll look through the code/wiki to see if there are any links that can be fixed.
So will the site ever be back up, or should the future work not consider the website other then to redirect to the git repo?

I made a pull request for some broken links which could be fixed: https://github.com/RestKit/RestKit/pull/2230
If you would like to continue the conversation there, I'm willing to help out.
(I'm doing this because I thought RestKit was dead at first due to a lot of links being broken; however, the project is very much still alive.)


fix(charts): quote annotation values

I get errors trying to set service_annotations or deployment_annotations in testing #323. Templating the router service seems to be the initial problem, because the range loop doesn't quote the values as required.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #328   +/-   ##
=======================================
  Coverage   55.69%   55.69%
=======================================
  Files           6        6
  Lines         386      386
=======================================
  Hits          215      215
  Misses        151      151
  Partials       20       20

Continue to review full report at Codecov.

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

I'll see if I can get service annotations working with this fix.

This worked for me. LGTM!
><> cat workflow/values.yaml | grep proxyBuffers
    router.deis.io/nginx.proxyBuffers.enabled: "true"
><> kd get deploy deis-router -o yaml | grep proxy
    router.deis.io/nginx.proxyBuffers.enabled: "true"


[Bug]: PSD + stoichiometry dependant diffusion bug [Half cell]

PyBaMM Version
23.4.1
Python Version
23.4.1
Describe the bug
Hello everyone, I would like to run some half cell graphite simulations using particle size distribution and stoichiometry dependant diffusion within the particles. However I'm getting an error related to discretisation. Did someone already face the same issue? Thanks for the help! Cheers, Marc-Antoine
Steps to Reproduce
import pybamm
import matplotlib.pyplot as plt

options_psd = {"working electrode" : "positive", "particle size": "distribution"}
model = pybamm.lithium_ion.DFN(options = options_psd)

parameter_values = pybamm.ParameterValues("OKane2022")

parameter_values = pybamm.get_size_distribution_parameters(parameter_values, sd_n=0.2, sd_p=0.4)

parameter_values["Maximum concentration in positive electrode [mol.m-3]"] = parameter_values["Maximum concentration in negative electrode [mol.m-3]"]
parameter_values["Initial concentration in positive electrode [mol.m-3]"] = 0.001*parameter_values["Initial concentration in negative electrode [mol.m-3]"]
parameter_values["Positive electrode OCP [V]"] = parameter_values["Negative electrode OCP [V]"]
parameter_values["Lower voltage cut-off [V]"] = 0.001
parameter_values["Upper voltage cut-off [V]"] = 1.6
parameter_values["Positive electrode active material volume fraction"] = parameter_values["Negative electrode active material volume fraction"]
parameter_values["Positive electrode porosity"] = parameter_values["Negative electrode porosity"]

def graphite_LGM50_diffusivity_ORegan2022(sto, T):
    """
    LG M50 Graphite diffusivity as a function of stochiometry, in this case the
    diffusivity is taken to be a constant. The value is taken from [1].

    References
    ----------
    .. [1] Kieran O’Regan, Ferran Brosa Planella, W. Dhammika Widanage, and Emma
    Kendrick. "Thermal-electrochemical parameters of a high energy lithium-ion
    cylindrical battery." Electrochimica Acta 425 (2022): 140700

    Parameters
    ----------
    sto: :class:`pybamm.Symbol`
       Electrode stochiometry
    T: :class:`pybamm.Symbol`
       Dimensional temperature

    Returns
    -------
    :class:`pybamm.Symbol`
       Solid diffusivity
    """

    a0 = 11.17
    a1 = -1.553
    a2 = -6.136
    a3 = -9.725
    a4 = 1.85
    b1 = 0.2031
    b2 = 0.5375
    b3 = 0.9144
    b4 = 0.5953
    c0 = -15.11
    c1 = 0.0006091
    c2 = 0.06438
    c3 = 0.0578
    c4 = 0.001356
    d = 2092

    D_ref = (
        10
        ** (
            a0 * sto
            + c0
            + a1 * pybamm.exp(-((sto - b1) ** 2) / c1)
            + a2 * pybamm.exp(-((sto - b2) ** 2) / c2)
            + a3 * pybamm.exp(-((sto - b3) ** 2) / c3)
            + a4 * pybamm.exp(-((sto - b4) ** 2) / c4)
        )
        * 3.0321  # correcting factor (see O'Regan et al 2021)
    )

    E_D_s = d * pybamm.constants.R
    arrhenius = pybamm.exp(E_D_s / pybamm.constants.R * (1 / 298.15 - 1 / T))

    return D_ref * arrhenius

parameter_values["Positive electrode diffusivity [m2.s-1]"] = graphite_LGM50_diffusivity_ORegan2022

experiment = pybamm.Experiment(["Discharge at 1C until 0.01V (5 second period)"])

sim = pybamm.Simulation(model, parameter_values = parameter_values, experiment = experiment, solver = pybamm.CasadiSolver(rtol=1e-3))
sim.solve()

plt.figure('Half cell model results')
plt.plot(sim.solution['Discharge capacity [A.h]'].entries, sim.solution['Terminal voltage [V]'].entries, label = 'PyBaMM')
plt.xlabel('Capacity [A.h]')
plt.ylabel('Terminal voltage [V]')
plt.legend()
plt.grid(True)

Relevant log output
No response

I suspect the issue is with the stoichiometry dependent diffusion coefficient, because commenting parameter_values["Positive electrode diffusivity [m2.s-1]"] = graphite_LGM50_diffusivity_ORegan2022 works fine. I am not too familiar with how the PSD models work though, so hard for me to tell where the problem is.

Hi @brosaplanella , yes exactly, both PSD and stoichiometry dependent diffusion coefficient work well separately, but not together. According to the error, when PSD and stoichiometry dependent diffusion are used together the integration_dimension goes to 'tertiary' when defining the integral matrix in definite_integral_matrix function (pybamm/spatial_methods/finite_volume). I tried to add a tertiary case without great success until now, thanks for the help!

As a workaround you can delete the variable that is causing problems (assuming you don't need to evaluate it)
model = ...
del model.variables["X-averaged positive particle effective diffusivity distribution [m2.s-1]"]

Just getting back to this for my work, thanks a lot for the workaround @tinosulzer , I really appreciate!
Best,
MA


Create a release process

What would you like to be added:
For a release process, we need consistent versioning implemented in code and in Markdown build file.
The process could do the following:

[ ] gather release notes from merged PRs
[ ] designate an RC
[ ] release the RC with the prefabricated release notes including which PRs this version contains
[ ] mark the generated binary with that specific version
[ ] Optionally, trigger a change in the website for any generated content updates
[ ] Optionally, create release docs
[ ] create the actual release in github with bundled up spec, manifests and binaries for all OS / ARCH combos.

Technical implementation is up to the picker of this ticket. I recommend using GoReleaser.
Why is this needed:

The following tasks might be performed on various repositories:
Controllers
They already have set up containers and built images on a tag push. Extend this by creating artifacts and releases too. The release should contain that version's generated Manifest files as well.
OCM
This repository lacks the complete release process. That includes creating images, doing version bumps and designating release docs.

As per our short discussion: We need to add to the release-process an automated way of updating the link to the latest official release, which is shown on our website.

We can use a parameterized workflow_dispatch in the target repo.


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Implements date filters for donations index page

This continues @duaimei 's work on date filters. It is tested manually and e2e.
Resolves #1101

Closing in deference to #1176


Bug: When restarting server some options are lost from config

Bug Report
When I restart the docker some of my settings disappear, such as the OpenAI key
Steps to Reproduce:
docker run -p 3000:8080 --add-host=host.docker.internal:host-gateway -v ollama-webui:/app/backend/data ghcr.io/ollama-webui/ollama-webui:main

Environment

Operating System: Ubuntu 18.04

Documentation issue, they use environment variables. Let's continue our discussion here: #432
Sidenote, I'm currently extremely busy with day job at the moment so if anyone could get the rolling for the documentation, I'd greatly appreciate it, Thanks!


Show the "Add image to linked Wikipedia article" icon for all images

Summary:
PR #3783 adds a "Add image to linked Wikipedia article" icon to images uploaded via Nearby. As of now, the icon is only shown for the images that meet a certain criteria. I believe it's:
i. Image is uploaded via Nearby
ii. Has a linked Wiki article
iii. The linked wiki article doesn't have any images
For better UX, I think it would be nice if we show the wikipedia icon for all images. We can then enable/disable it based on the criteria. Whenever we disable it, we can show a proper reason for which it is disabled. This would help the users to easily get an idea about when we show and we don't show the icon.
Related issue: #872

hi @sivaraam , what you propose is to remove these criteria and always show the "add image to linked wikipedia article" icon?

I'm suggesting to show the icon in all cases and decide to enable/disable it based on the criteria. When tapping on the icon when it is enabled one would get the dialog as usual. When tapping on the icon when it is disabled one should be shown the reason why it is disabled probably as a tooltip.

@sivaraam I would really like to work on this issue, however, I am unable to repro the icon (I am not able to find an image that passes the criteria). Do you know if there is a way to force the app to give me the icon on a given image?
Regardless, I can begin work on this now after I read through maskaravivek's code and find out how it works. The only issue (as stated earlier) is that I am unsure of how to test any code that I write without having an image that meets the criteria.

@rhulkb27 Thanks for your interest. For some background, I initially suggested this in PR #3783, but it wasn't addressed as people thought it needs to be discussed first and it didn't need to be part of the MVP. That's why I opened this issue. So, you can take this up if others agree to this change.
Regardless, some replies to your questions:

Do you know if there is a way to force the app to give me the icon on a given image?

No, I don't think that's possible. Neither does it make sense to "force" the icon to be shown as that would mean we would not be satisfying the criteria for which the icons are shown which is not ideal for this task.

The only issue (as stated earlier) is that I am unsure of how to test any code that I write without having an image that meets the criteria.

Yeah, it's tricky to get the icon triggered indeed. That's one of the reasons I was suggesting to show the icon for all images and disable it based on criteria. It would likely make the feature more discoverable. Anyways, try using the beta flavour of the app, find some nearby items which are associated with Wikipedia that lack images and upload an image to that nearby item via the nearby upload flow. Doing this the icon should be shown for the image you just uploaded.

Alright, @sivaraam I'm confident that I will be able to implement this change, however as you said we need others to agree that this is a good change before I can begin working on it; do you know who exactly we need to give me the go-ahead from before I can start working on the issue? Maybe we can @ mention them here?

Let me ping some people: @misaochan @maskaravivek @neslihanturan

@misaochan @maskaravivek @neslihanturan Hi, I just wanted to follow up on this issue; don't forget to let me know if you have any issues with this, otherwise, I will implement it tomorrow.

What indicator shall we have for people to know at a quick glance whether or not it is enabled? Should it be greyed out if disabled? Otherwise it would defeat the purpose of having the icon in Contributions at all if users could not tell easily whether the option was available or not for a particular image without tapping it.
Aside from the others that you pinged, @nicolas-raoul 's input would be great. @rhulkb27 I would recommend waiting to implement until a consensus is reached on this.

@misaochan I believe @sivaraam suggested to have a tooltip showing the reason why the image does not pass the criteria.

Should it be greyed out if disabled?

I think it should be. Actually, I assumed it would be greyed out when I mentioned "disabled". Looks like I should have mentioned this explicitly.

Otherwise it would defeat the purpose of having the icon in Contributions at all if users could not tell easily whether the option was available or not for a particular image without tapping it.

Yes, exactly. If we don't grey out then it would end up being very bad UX. I suggested the tooltip as an add-on to explain "why" the button is disabled and greyed-out.

I think it should be. Actually, I assumed it would be greyed out when I mentioned "disabled". Looks like I should have mentioned this explicitly.

Makes sense. I am okay with this change in that case. @neslihanturan what do you think?


Add filtering on ThingService

Prerequisites

[x] I have written a descriptive issue title
[x] I have verified that I am running the latest version of the UI-DSM
[x] I have searched open and closed issues to ensure it has not already been reported

Description

The ThingService should be able to retrieve Things with more filter :

[ ] Allow a collection of ClassKind instead of a simple ClassKind
[ ] Allow to filter on Categories

Steps to Reproduce

System Configuration

UI-DSM version:

[ ] UI-DSM:
[ ] Other:

Environment (Operating system, version and so on):
.NET Framework version:
Additional information:

Does not make any sense anymore since the whole Iteration is query at any time.


Show self in following list and followers list

Hide follow button and unfollow button in this case.

Fix the way of coding for hiding button.


Update action to Node 16

GitHub deprecates Node 12 and actions with github-action-ssh started to fail.

Thank you!


Upgrade log4j

Upgrade log4j and disable message format lookups to improve security.

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


:wave: Always nice to see a familiar face at the end of an internet search!

:smile:

Hello @lawrencegripper.  If you're interested in this, it would be good to get your ideas. The low-level stuff, to send commands to the API and retrieve the JSON responses works well. Last I looked at this I was trying to understand what API structure would work well for the for the SDK.

Hey, yeah I'm not sure I have many thoughts here, I needed some info on how the API behaved and this was super useful but I'm probably not going to be doing much else with it for now, sorry.


DBeaver: Redraw/Repaint of the Database Navigator tree is slighly damaged

DBeaver - Version 6.1.3 CE
DBeaver driver: MS SQL Server  / Microsoft Driver (the new Microsoft Driver)
Operating System: Windows 7 / Windows 8.1 / Windows 10
Database Server: Microsoft SQL Express 2014, 2016, 2017
Redraw/Repaint of the Database Navigator tree is slighly damaged.
o Open a connction/sesion to a SQL Server database with table triggers.
o Expand the schema with Table Triggers displayed.
o Mark more than one table trigger.
o Right click on one of the table triggers and choose Tools -> Disable trigger.
o Press the OK button in the Disable trigger dialog.
o Press the CLOSE button in the Disable trigger progress dialog.
o Left click with the mouse on any of the disabled triggers.
o The Database Navigator tree disappear except the table triggers that was disabled.
o Press the UP/DOWN key on the keyboard to paint/draw the missing tree nodes.
Note 1: The above description does not 100% indicate the problem. You have to try a coule of times.
Note 2: I use the Dark theme in DBeaver and are unsure of this is what causing the trouble.

Please test it in 6.1.4 (there were a few fixes related to trigger management)

I have tried it now in version 6.1.4 CE and I can't make it break like in 6.1.3 CE. Maybe it was solved with the other fixes related to trigger management.

Mark as not reproducible anymore.


(maint) Properly detect cURL on FreeBSD

Add FreeBSD to the supported systems list so that -DUSE_CURL can be set and
features depending on this can be used (e.g. ec2_metadata).

Can one of the admins verify this patch?

CLA signed by all contributors.


Foldable f => From f ... instances

Hi! Would generalizations of instances of the form From [...] be a welcome contribution? Are there reasons to not generalize those?
My example use case:
instance (Foldable f, Ord k) => From (f (k, v)) (Map k v) where
 from = M.fromList . toList

-- usage:
header :: Vector Name = ...
headerLengths = into @(Map Name Int) $ (id &&& BS.length) <$> header

This saves a toList near <$> header, and allows the whole pipeline to work in the original functor (here: Vector) until we actually need the list. WDYT?

Thanks for opening this suggestion! I generally support adding new instances to Witch.
That being said, won't this instance overlap with the list instance?
https://github.com/tfausak/witch/blob/461bf44506ea822100dc2277d43331fd6cee9786/source/library/Witch/Instances.hs#L976
Are you suggesting to replace that instance with this one, or something else?

Good point - the overlap didn't surface for me b/c I did not have the usage of above instance in my code - which your tests helpfully cover :)
My initial thought would be to replace the instance. What's your stance on that?

I’m not necessarily opposed to replacing the instance with a more polymorphic one. So far I’ve tried to avoid polymorphic instances where possible. For example there’s no instance for (Integral a, Integral b) => From a b, but there is an instance for From Int8 Int16. I have found this to be useful for documentation for two reasons: one, it’s immediately clear which types you can convert; and two, if there’s something tricky about a conversion I can document that directly on the instance.

Sorry, clearly I haven't made any progress here. I'm going to close this issue, but I'd be open to a PR introducing this instance.


dependent bot alert: Bump lxml from 4.6.2 to 4.6.3

bumped lxml version

No conflict with the base branch. Okay to merge.


resolver timeout (UDP)

dnscrypt-proxy --local-address=<IP_ADDRESS>:65053 --daemonize dns-payload-size=1252 -R cisco -l /opt/tmp/dnscrypt-proxy.65053.log

dnscrypt-proxy --local-address=<IP_ADDRESS>:65153 --daemonize dns-payload-size=1252 -R dnscrypt.eu-dk -l /opt/tmp/dnscrypt-proxy.65153.log

/ # ip ro
default dev ppp0  scope link
<IP_ADDRESS> dev ppp1  scope link
...
xx.xx.xx.26 dev ppp1  proto kernel  scope link  src xx.xx.xx.52
/ #

65153.log
...
Sat Aug 19 15:10:55 2017 [NOTICE] Starting dnscrypt-proxy 1.9.5
Sat Aug 19 15:10:55 2017 [INFO] Generating a new session key pair
Sat Aug 19 15:10:55 2017 [INFO] Done
...

You can ignore all of these, or try a resolver with a lower latency.

It is clear, but if there was one resolv and it was - dnscrypt.eu-dk, then most likely brakes would be noticeable?

The connectivity between your client and these servers doesn't seem to be very reliable. Is it a VPN?
Packets are lost, or taking too much time to come back. This probably affects everything not just dnscrypt-proxy.
DNS clients automatically retry after a timeout, so this is usually not a problem.
Enabling the cache can help a little bit: https://github.com/jedisct1/dnscrypt-proxy/wiki/Go-faster
As well as choosing a resolver closer to your internet provider or VPN.

Yes one sits on pppoe (cisco) another on pptp (dnscrypt.eu-dk) channels good and stable, it is possible to connect both to pppoe, I think not in them.
Yes since the beginning there was "cache-size=0" value 1500, there were also these messages.

You connection might look stable, but it is dropping packets. There's not much that can be done here, you need to investigate why that happens, as it's outside of the scope of dnscrypt-proxy. Maybe network congestion.
Try using TCP, maybe (change TCPOnly no to TCPOnly yes in the config file).

TCPOnly yes, is aware about this setup. Earlier already I tried, but on speed concedes to UDP to an algorithm. I will try to return once again on this function.

Also try http://icmpcheck.popcount.org/


layers/meta-balena-isg-503:remove the migrate and recovery modules

The initramfs image that exceeds the allocated space. remove some new modules
Changelog-entry:layers/meta-balena-isg-503:remove the migrate and recovery modules

Can one of the admins verify this patch?

@resin-jenkins ok to test

@resin-jenkins test this please

lgtm

I  am not administrator ~

hi @alexgg Could you help merge this Pr?

lgtm

@resin-jenkins ok to test

@resin-jenkins test this please

@resin-jenkins retest this please

@RockSupport how can u-boot & kernel logs be obtained, which UART pins should be used? I believe the RS232 debug pins should be GND, B2/RX1, A2,TX1 at baudrate 1500000 8N1 but I don't see any logs. Thanks

Hi @alexgg ， uboot is same as before,，kernel logs can obtained by dmesg.
and we use uart2_m1 in the board

hi @acostach  only on the 40 pin header
And sorry for unreasonable design
please make sure compile with '-d'
./balena-yocto-scripts/build/barys -m isg-503 --shared-downloads /home/rockemb/Balena/downloads -d
use ttl port

Thank you @RockSupport , I was able to login with the latest git master and a deveopment build, on the 40pin header. Older revisions didn't receive chars in Linux, only in u-boot, but the latest HEAD revision works fine for both.

@acostach maybe miss some config in the dts.
I just debug uboot by uart. did not use after boot
use balena ssh instead

@RockSupport one more question, it looks like a random MAC address is generated for the ethernet on each device boot, which leads to the device always getting a different IP although the dhcp lease isn't expired. Is this expected behavior? We're seeing this with both devices we have using the latest git master revision.
[    1.004254] rk_gmac-dwmac ff360000.ethernet: rk_get_eth_addr: rk_vendor_read eth mac address failed (-1)
[    1.006023] rk_gmac-dwmac ff360000.ethernet: rk_get_eth_addr: generate random eth mac address: fa:23:26:f9:6e:7c
[    1.007912] rk_gmac-dwmac ff360000.ethernet: rk_get_eth_addr: rk_vendor_write eth mac address failed (-1)
[    1.009697] rk_gmac-dwmac ff360000.ethernet: rk_get_eth_addr: id: 0 rk_vendor_read eth mac address failed (-1)
[    1.011547] rk_gmac-dwmac ff360000.ethernet: rk_get_eth_addr: mac address: fa:23:26:f9:6e:7c
[    1.013118] rk_gmac-dwmac ff360000.ethernet: device MAC address fa:23:26:f9:6e:7c

reboot
[    1.005613] rk_gmac-dwmac ff360000.ethernet: rk_get_eth_addr: rk_vendor_read eth mac address failed (-1)
[    1.007389] rk_gmac-dwmac ff360000.ethernet: rk_get_eth_addr: generate random eth mac address: 82:64:48:63:71:53
[    1.009275] rk_gmac-dwmac ff360000.ethernet: rk_get_eth_addr: rk_vendor_write eth mac address failed (-1)
[    1.011048] rk_gmac-dwmac ff360000.ethernet: rk_get_eth_addr: id: 0 rk_vendor_read eth mac address failed (-1)
[    1.012907] rk_gmac-dwmac ff360000.ethernet: rk_get_eth_addr: mac address: 82:64:48:63:71:53
[    1.014475] rk_gmac-dwmac ff360000.ethernet: device MAC address 82:64:48:63:71:53

hi @acostach ，we have write our mac to the devices in factory.So "rk_vendor_read eth mac address" will sucess.
Beside we need keep UUID keep the same  after update from Dunfell to Kirkstone

@RockSupport for the devices we have on our side the "rk_vendor_read eth mac address" always fails, I assume the MAC has not been written in the factory for our units. Can we write it from Linux on our devices so that it stops failing our tests, do you have a line that we can use? Thanks

@acostach we just have an windows tools to write mac.I am not familiar with this tools

I  try to use this tool with balenaOS and write a document for you.

hi @acostach  How can I send the RKDevInfoWriteTool to you?
I  have tested in balenaOS and here is the step how to enter loader
balena ssh deviceIP
reboot loader
press the green button

@RockSupport is the rkdevinfowritetool here https://en.t-firefly.com/doc/download/67.html good?
Also, which port should I connect to the Windows PC, the USB-C port? Are there some pins that I have to jumper for rkdevinfowritetool to talk to the board?

@acostach I think the rkdevinfowritetool here https://en.t-firefly.com/doc/download/67.html is good
with "reboot loader" in ssh or console will enter loader mode

Thank you @RockSupport , rkdevinfowritetool worked for flashing the eeprom with the MAC number. Could you please tell me again the TX, RX and GND pin numbers for the UART? I see you deleted the schematic and I didn't get to save the pin numbers. Thanks

@acostach Sorry I thought the github will send a mail to you

@acostach there is not eeprom store mac addr, the rkdevinfowritetool will write mac info to emmc, But I am not know the detail about that.maybe rk_vendor_read/write can give something useful


Help

Sorry if I am disturbing you but can you give me the apk for this because I'm not skilled enough to compile an app btw can you make a option to switch to PSP XMB rather than PS3 XMB

Oops I'm sorry I'm a idiot I just realized there's a release apk

Well, if you can provide me with one picture (or screenshot) of the authentic PSP XMB (no theme, just default XMB). I probably can do something with it. I never had a PSP before, just a PS3, so my reference is just PS3 currently.

I don't have a PSP but it look pretty much the same with PS3 just remove this bar
 and btw can you turn off the transparent background so my wallpaper doesn't look dark

Here, I added your requested features: Custom background color, adjustable clock bar : Alpha 4
The FPS box shouldn't appear unless the debug setting for FPS is enabled,
P.S : Sorry, I just carried away by the label settings

Anyway, the compilation of this project should be simple and straighforward.

Pull this project source code from git (or download it as ZIP then extract it)
Open your local repo (or extracts) from Android Studio as an Android Project.
Wait until gradle finished resolving and caching all needed dependencies.
Deploy to your device (The play button at top right)

Custom background color can be found at the Setting category, to set it clear / transparent, Put "#00FFFFFF" to the dialog box. It requires Hex color in format "#aarrggbb" where you can control the color transparency by modifying the first 2 digit (the "aa")

Forget to mention that the app setting reset itself after not being used

Please open another Issue since it was different bug 🙂.
Well, new bugs keep appearing since I reworked on how item icon and text is accessed before rendered.


A spec a day keeps the bugs at bay

wat

(Write tests)

but isn't TDD dead? http://david.heinemeierhansson.com/2014/tdd-is-dead-long-live-testing.html

https://twitter.com/rob_rix/status/623697875340521472


Pr darrenfromsandbox

JIRA link (if applicable)
Change description
Adding code to build postgres flexi server
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x ] No

azurerm_key_vault data resource named "keyvault" was already declared - will remove and resubmit


Fix only one of the coordinates being None

After your commit, my ugly plots became beautiful but then some other plots broke :smile:
Was there any reason to use and instead of or on those conditionals?
Otherwise this fixes the case in which one of the coordinates is None while the other one isn't. E.g.:
x=[0, 0, 1], y=[None, 2, 3]

The build is failing but that's because of another reason.


How to handle conversions of Go structs to C++ structs?

Ideas
http://stackoverflow.com/questions/28566242/exchange-data-structures-arrays-between-go-golang-and-c
http://akrennmair.github.io/golang-cgo-slides/
https://groups.google.com/forum/#!topic/golang-nuts/JkvR4dQy9t4

Perhaps it is best to simply reference the C structs as generated by cgo like in crypto_generichash.go


Initial change to compile middleware invoke method when extra args

#530
We could maybe move some of this compilation logic to Common (not sure if we do it in more than one place).
/cc @DamianEdwards @pranavkm

Do you need to worry about methods that could have ref or out parameters? You should probably either add support for them or throw an exception with an error message when building the expression tree.

@JamesNK seems unrelated to this change in particular. What happens when you do methodInfo.Invoke and you have ref and out params? Does it just work?

Yeah it does so this would be a regression.
This is a method I wrote to handle calling pretty much any method info via a compiled expression: https://github.com/JamesNK/Newtonsoft.Json/blob/master/Src/Newtonsoft.Json/Utilities/ExpressionReflectionDelegateFactory.cs#L86-L190
Looking at what I wrote, you might be ok with ref parameters because you don't need to capture the return value. You probably shouldn't try to resolve out params from DI. Either leave them alone or tell the user it is an error to have out parameter here.

:shipit:


Optional timeout on reads and writes

Add an option to have a timeout on transactions. If read or write is not ready, it should timeout the transaction and return an error. Related to issue #8

Sounds nice. However, I don't really get what can be the source of such backpressure within generated map that timeouts CSR access?
Moreover, AFAIK timeout bridges or monitors are usually quite separate things from CSR maps.
Could you please elaborate your idea or provide some practical examples?


gradle.settingsEvaluated and other callbacks should fire even when registered after being fired

settingsEvaluated callbacks and other callbacks like afterEvaluate should fire even when they are registered after the original callback was fired. This can lead to surprising behavior in builds when the callbacks do not fire as expected.
The below example will not print anything if added to a project's build.gradle file.
gradle.settingsEvaluated {
    println("project settings evaluated")
}

Another alternative would be to have a warning displayed that the callback was registered after it was already fired so that build author's can expect that the callback will not be fired.

One thing to Keep in mind, that this could also lead to some unexpected effects, as the code would run later that one might think. That might not be an issue if you only want to read from the settings object, but if you modify stuff, this could have weird effects.

Another alternative would be to have a warning displayed that the callback was registered after it was already fired so that build author's can expect that the callback will not be fired.

If we decide to not allow these late registrations, then I think we should fail. A deprecation warning might be an intermediate step, though.

@gradle/execution would you like to take over this issue?

We don't have the capacity to take a look at this, and it's not related to execution in the first place anyway. Putting this back on triage.

I think we should make settingsEvaluated behave the same way as project.afterEvaluate {}, that is honor all registered callbacks.
See https://github.com/gradle/gradle/issues/3411 and the associated PR.

Putting this back on team triage given we have the opposite ask:

#20047

This feature request is in the backlog of the relevant team and is prioritized by them.

We're not ready to make a decision if we want #20047 or #17914 at the current moment, but we agree that the current behavior should be changed. Isolated Projects may also impact this decision.


Updated Hooks to v12

Closes #2573

Foundry hasn't officially documented the appv2 hooks, we should probably look into those and write our own? Maybe steal from v13? Can be a separate update

Are the AppV2 hooks ticketed? If not could you make sure they are?


Add Elliptic bitcoin dataset

Added elliptic bitcoin transaction dataset
Based on pytorch's version for elliptic bitcoin dataset
https://pytorch-geometric.readthedocs.io/en/latest/_modules/torch_geometric/datasets/elliptic.html
Checklist
Put an x in the boxes that apply.

[x] I have added tests that prove my fix is effective or that my feature works
[x] I have updated the necessary documentation

Amazing @vinayhpandya, thanks for adding this dataset! Apart from @francescofarina's comment, it seems all good to me.

@francescofarina I don't think its absolutely necessary to add pandas, I can try to see if numpy can extract csv values for nodes and edges and that should work as well

@francescofarina I don't think its absolutely necessary to add pandas, I can try to see if numpy can extract csv values for nodes and edges and that should work as well

That would be great - I think it's best to not add pandas if numpy works as well!

I tried using numpy's loadtxt functionality to manipulate the node features but it takes around 9 secs
compared to 3 secs with Pandas, is csv reader a better option?
Time taken is 9.910717964172363 Feature shape is (203769, 167) Time taken for pandas is 3.2206032276153564

@vinayhpandya I think the 9 seconds with numpy are ok. The dataset is processed only once and then stored in memory to be reloaded when needed, so that should be ok 👍🏻

Good catch! It would be indeed better to use numpy here, even if slower.

@TristanBilot @francescofarina I have removed pandas and used numpy's loadtxt to process the files. Although this process is slightly slower than pandas (upto ~7secs)


Error: node-pty failed to load.

Mac 10.13.3 (17D102)
node v9.8.0
yarn run app
yarn run v0.21.3
$ electron app
Plugin hyper-nord (1.0.1) loaded.
App threw an error during load
Error: `node-pty` failed to load. Typically this means that it was built incorrectly. Please check the `readme.md` to more info.
    at createNodePtyError (/../hyper/app/session.js:12:3)
    at Object.<anonymous> (/../hyper/app/session.js:20:9)
    at Object.<anonymous> (/../hyper/app/session.js:113:3)
    at Module._compile (module.js:569:30)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:503:32)
    at tryModuleLoad (module.js:466:12)
    at Function.Module._load (module.js:458:3)
    at Module.require (module.js:513:17)
    at require (internal/module.js:11:18)

https://github.com/zeit/hyper/blob/36cde0f23e27376a9ec139541f7f8b288e772999/app/session.js#L18
Possible Fix
app/session.js
const nodePty = require('node-pty');
//.......
let spawn;
try {
  spawn = nodePty.spawn
} catch (err) {
  throw createNodePtyError();
}

try to clean/reinstall your dependencies: rm -Rf node_modules && yarn

It gets stuck in a loop
yarn run v0.21.3
$ electron-rebuild -f -w app/node_modules/node-pty -m app
✔ Rebuild Complete
✨  Done in 2.44s.
yarn install v0.21.3
[1/4] 🔍  Resolving packages...
success Already up-to-date.
$ electron-builder install-app-deps && yarn run rebuild-node-pty && yarn --cwd node_modules/xterm
  • electron-builder version=19.55.2
  • loaded configuration file=package.json ("build" field)
  • installing production dependencies platform=darwin arch=x64 appDir=/Users/edgarjoya/Documents/LocalDev/hyper/app
yarn run v0.21.3
$ electron-rebuild -f -w app/node_modules/node-pty -m app
✔ Rebuild Complete
✨  Done in 2.33s.
yarn install v0.21.3
[1/4] 🔍  Resolving packages...
success Already up-to-date.
$ electron-builder install-app-deps && yarn run rebuild-node-pty && yarn --cwd node_modules/xterm
  • electron-builder version=19.55.2
  • loaded configuration file=package.json ("build" field)
  • installing production dependencies platform=darwin arch=x64 appDir=/Users/edgarjoya/Documents/LocalDev/hyper/app
yarn run v0.21.3
$ electron-rebuild -f -w app/node_modules/node-pty -m app
✔ Rebuild Complete
✨  Done in 2.43s.

What version of Yarn are you using?  I am on Mac OS X v10.13.2 with Yarn v1.3.2 and am having no issues with my build. From the output of your logs it looks like you are on an older version of Yarn?

@Lucretius Yeap update brew and yarn, works.


Add code lens for running migrations

Closes https://github.com/Shopify/ruby-lsp-rails/issues/51
Adds code lens to run migrations to specific versions in the terminal. This is convenient because it allows developers to quickly rollback or fast-forward to specific schema versions with a click.

Is there a different between migrating to a specific version vs rolling back to that version? For example, is migrating to the second to last version exactly the same as db:rollback?

I noticed this in the docs:

Neither of these rails commands do anything you could not do with db:migrate. They are there for convenience, since you do not need to explicitly specify the version to migrate to.

Thanks for sharing. Do people agree with using 3 code lenses?
Run all migrations | Migrate to this version | Rollback latest migration

I feel like usually people want to run bin/rails db:migrate and bin/rails db:rollback. Having those there will be handy, but it will also help communicate that Migrate to this version may actually be rolling back their database.

Seems like my PRs are getting stale. I'll see if I can find some time to rebase and stuff the logic into different listeners.

Do people agree with using 3 code lenses?

Personally I would be supportive of this order: Migrate to this version | Run all migrations | Rollback latest migration. Idiomatically, if we're putting these lenses on all migrations, the developer would open the migration they want to go to. I would go as far as to say the other lenses should only be visible on the latest migration.

Is there a difference between migrating using the latest migration version and simply running db:migrate to run all of them? I mean the end result of doing each action

Andy is correct, VERSION=<latest> and no args to run all are the same.

Is there a different between migrating to a specific version vs rolling back to that version? For example, is migrating to the second to last version exactly the same as db:rollback?

Yes, I think so. Rollbacks can specify a step count though, so that's slightly different. The default step is one. It is just really convenient to use the VERSION for my use-case because Rails can figure out if I mean I want to go up or down.


bug: Adds the typed pattern rather than the search string to the search register & history

Did you check docs and existing issues?

[X] I have read all the flash.nvim docs
[X] I have searched the existing issues of flash.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
NVIM v0.10.0-dev-541+g96b94f8d7
Operating system/version
Pop!_OS 22.04 LTS
Describe the bug
I'm not sure if this is a bug or a misunderstand + feature request in some sense. When flash is configured to add to the search register and history, what is added is actually what the user types, not the actual search string that is used (which is different in the case of exact and even moreso in fuzzy mode)
It would seem the use case of the option is to make it easily repeatable. If the search string were added to the search register than n, N should just work afaict. But as it is it doesn't
I think this should be state.pattern.search?
https://github.com/folke/flash.nvim/blob/41fab4cb225d9233fec7987bb1445c9768d84caf/lua/flash/jump.lua#L50
Steps To Reproduce

set opts.jump.register = true and opts.search.mode = "fuzzy"
Use the jump (srg...)
Check the search register with "/p
Try repeating the search with "n"

Expected Behavior
I would expect the search register to be filled with the augmented search string that makes it a fuzzy search.
It would seem the use case of the option is to make it easily repeatable. If the search string were added to the search register than n, N should just work afaict. But as it is i'm not sure how to easily repeat fuzzy search.
Repro
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
	vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
	vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
	"folke/tokyonight.nvim",
	{
		"folke/flash.nvim",
		opts = {
			search = {
				mode = "fuzzy",
			},
			jump = {
				register = true,
				history = true,
			},
		},
		keys = {
			{
				"s",
				mode = { "n", "x", "o" },
				function()
					-- default options: exact mode, multi window, all directions, with a backdrop
					require("flash").jump()
				end,
			},
		},
	},
	-- add any other plugins here
}
require("lazy").setup(plugins, {
	root = root .. "/plugins",
})

vim.cmd.colorscheme("tokyonight")
-- add anything else here

Good point, changed it to use pattern.search instead


Summit Events Object: Add a new text-area field called Test

Please confirm the following during testing:

There is a new text area field on the Summit Events Object called Test
The new Test area field is exposed on the Summit Events Page Layout
The Admin profile has FLS

Critical Changes
NA
Changes
NA
Issues Closed

This appears to be a test bit of code so we have decided to delete it.


Better group invite support

Pull Request Etiquette
There are several guidelines you should follow in order for your
Pull Request to be merged.

[X] I have checked the PRs for upcoming features/bug fixes.
[X] I have read the contributing guidelines.

It is sometimes better to include more changes in a single commit.
If you find yourself having an overwhelming amount of commits, you
can rebase your branch.

Description
This PR adds better support for group invites.

Any update on this?


Update scala3-compiler, scala3-library to 3.3.3

About this PR
📦 Updates

org.scala-lang:scala3-compiler
org.scala-lang:scala3-library

from 3.2.2 to 3.3.3
📜 GitHub Release Notes - Version Diff - Version Diff
Usage
✅ Please merge!
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!

🔍 Files still referring to the old version number
The following files still refer to the old version number (3.2.2).
You might want to review and update them manually.
build.sbt

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

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scala-lang" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, old-version-remains, commit-count:1

Superseded by #2120.


Getting default nginx index.html at localhost

I followed the instructions provided in the documentation. Initially, I ran into some issues with the npm install in the backend folder. I solved that by changing the bcrypt dependency version to "*". I managed to run the backend.
However I am unsure about how to edit the root directive in the http block in nginx.conf. Should I put the path to the build directory in my frontend in the root directive? When I do this, and run the backend and frontend, I am still getting the default nginx index.html page instead of the frontend at localhost. Are there any other changes I should make to the nginx.conf file? I am an Ubuntu user.
Thank you.

My suspicion is that the project (which hasn't been touched in two years or so) is a bit out of date. As for the the nginx.conf file, have you tried changing it to the correct build directory on your system?

Thanks for replying!
Yes, for the nginx.conf file, I have changed the root path in the http block to the build directory in my frontend as such:
sendfile        on;
keepalive_timeout  65;

root /home/skynet/orbital2/pern/frontend/build;


What txt files should be put in '/path/to/image/filelist'?

For PACS dataset, what txt files should be put in '/path/to/image/filelist'?

Put the art_painting.txt, cartoon.txt, photo.txt and sketch.txt. In each file, line by line lists image file and the corresponding label. For instance, "photo/elephant/064_0122.jpg 1"


python310Packages.scancode-toolkit: 30.1.0 -> 31.0.0b4, python310Packages.typecode: 21.6.1 -> 30.0.0, python310Packages.plugincode: 21.1.21 -> 30.0.0

Description of changes
Follow-up of #172460

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 172709 run on x86_64-linux 1

33 packages built:

amberol
checkov
detect-secrets (python39Packages.detect-secrets)
python310Packages.container-inspector
python310Packages.debian-inspector
python310Packages.detect-secrets
python310Packages.extractcode
python310Packages.extractcode-7z
python310Packages.extractcode-libarchive
python310Packages.license-expression
python310Packages.parameter-expansion-patched
python310Packages.pip-requirements-parser
python310Packages.plugincode
python310Packages.pyahocorasick
python310Packages.scancode-toolkit
python310Packages.tern
python310Packages.typecode
python310Packages.typecode-libmagic
python39Packages.container-inspector
python39Packages.debian-inspector
python39Packages.extractcode
python39Packages.extractcode-7z
python39Packages.extractcode-libarchive
python39Packages.license-expression
python39Packages.parameter-expansion-patched
python39Packages.pip-requirements-parser
python39Packages.plugincode
python39Packages.pyahocorasick
python39Packages.scancode-toolkit
python39Packages.tern
python39Packages.typecode
python39Packages.typecode-libmagic
reuse


[New game]: Musical floor game

🎮 Game Request
The user has to hover over the colourful tiles to produce music.
Point down the features
The user has to hover over the colourful tiles to produce music.
Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

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

/assign

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


keras tape.gradient error: Input to reshape is a tensor with 1012 values, but the requested shape has 20240 [Op:Reshape]

i use the tape.gradient(g_loss, aa_mutator.trainable_variables) to calculate the gradient of a model called aa_mutator and got the error
File "/home/tialan/Data/gan/code/g.py", line 297, in <module>
    grads_g = tape.gradient(g_loss, aa_mutator.trainable_variables)
  File "/home/tialan/tf/lib/python3.7/site-packages/tensorflow/python/eager/backprop.py", line 1086, in gradient
    unconnected_gradients=unconnected_gradients)
  File "/home/tialan/tf/lib/python3.7/site-packages/tensorflow/python/eager/imperative_grad.py", line 77, in imperative_grad
    compat.as_str(unconnected_gradients.value))
  File "/home/tialan/tf/lib/python3.7/site-packages/tensorflow/python/eager/backprop.py", line 162, in _gradient_function
    return grad_fn(mock_op, *out_grads)
  File "/home/tialan/tf/lib/python3.7/site-packages/tensorflow/python/ops/array_grad.py", line 782, in _ReshapeGrad
    _IndexedSlicesToTensorNoWarning(grad), array_ops.shape(op.inputs[0])),
  File "/home/tialan/tf/lib/python3.7/site-packages/tensorflow/python/util/dispatch.py", line 201, in wrapper
    return target(*args, **kwargs)
  File "/home/tialan/tf/lib/python3.7/site-packages/tensorflow/python/ops/array_ops.py", line 195, in reshape
    result = gen_array_ops.reshape(tensor, shape, name)
  File "/home/tialan/tf/lib/python3.7/site-packages/tensorflow/python/ops/gen_array_ops.py", line 8368, in reshape
    _ops.raise_from_not_ok_status(e, name)
  File "/home/tialan/tf/lib/python3.7/site-packages/tensorflow/python/framework/ops.py", line 6862, in raise_from_not_ok_status
    six.raise_from(core._status_to_exception(e.code, message), None)
  File "<string>", line 3, in raise_from
tensorflow.python.framework.errors_impl.InvalidArgumentError: Input to reshape is a tensor with 1012 values, but the requested shape has 20240 [Op:Reshape]

within the aa_mutator model i build a customized keras layer
class mutate_func_layer(layers.Layer):
    def __init__(self):
        super(mutate_func_layer, self).__init__()
    def call(self, inputs):
        Mut_pos_layer_out, input_pre, Mutation_3 = inputs
        where = where_func(Mut_pos_layer_out)
        return mutate_func(Mutation_3, where, input_pre)

with mutate_func defined as
@tf.custom_gradient
def mutate_func(x, where, input_pre):#(Mutation_3, where, input_pre): @@ x = mutation 3
    print('m3_in_mutate_func')
    print(x.shape)
    aa_aft = gather_nd_func(x, where)
    print('m3_in_mutate_func')
    print(aa_aft.shape)
    aa_aft = K.argmax(aa_aft, axis=-1)
    print('m3_in_mutate_func')
    print(aa_aft.shape)
    aa_aft = tf.reshape(aa_aft, [-1])
    print('m3_in_mutate_func')
    print(aa_aft.shape)
    aa_aft = tf.cast(aa_aft, dtype=tf.float32)
    print('m3_in_mutate_func')
    print(aa_aft.shape)
    aa_seq_out = tf.tensor_scatter_nd_update(input_pre, [where], [aa_aft])
    print('m3_in_mutate_func')
    print(aa_seq_out.shape)
    def grad(upstream):
        return upstream*1, upstream*1, upstream*1

the shape for layers in the mutate_func are printed as
m3_in_mutate_func
(1, 1012, 20)
m3_in_mutate_func
(675, 20)
m3_in_mutate_func
(675,)
m3_in_mutate_func
(675,)
m3_in_mutate_func
(675,)
m3_in_mutate_func
(1, 1012)

the model is able to predict given the input. just for fitting the error shows at the stage of tape.gradient. is the error raised due to the customized layer? Thanks for any help or suggestion

@tianlt,
This error is specific to the size of the dataset that you are using.
You can try the workaround mentioned on similar issue #70313390.
Removing data with less than one batch of data.
train_dataset = train_dataset.shuffle(buffer_size=512).batch(BATCH_SIZE, **drop_remainder=True**).


Update specs to use modern style

Updated the specs to reflect a cleaner style of rspec testing. For info
about spec style visit http://betterspecs.org/

Currently having an issue with Rubocop:
Metrics/BlockLength: Block has too many lines.

Rubocop is complaining about the length of the Rspec.describe block. I can't imagine being able to reduce it's size. Just figured I'd owuld mention that.
Otherwise this file is ready for review. Let me know if the style is suitable and I will proceed to work on more specs.

Rubocop is complaining about the length of the Rspec.describe block. I can't imagine being able to reduce it's size. Just figured I'd would mention that.

And yes, block length limitation has no value for specs. It could be disabled in local specs/.rubocop.yml

I made the requested changes. I had an issue testing the :name property.
df.row[0].name #returns 0
df.row[0][:name] #returns "Homer"

It seems that the have_attributes method tests the first variation. I opted to test the :age column instead.

@zverok Thank you for the feedback. I moved the declarations into the contexts. The ActiveRecord connection is the default source since it is used twice. So far we have gone from 83 lines to 56 .


chore: Improve APK Names

Fixes #421

https://seesparkbox.com/foundry/semantic_commit_messages

https://seesparkbox.com/foundry/semantic_commit_messages

Thanks


MUMUP-tbd : Happy loading, Happy no results

Loading state is too fast, but all sections have a loading gif.

Awesome! :+1:

Good call. :+1:


Adding .ec files but get less reported coverage

Hi,
I have four .ec files by executing four tests on the same app, but the reported coverage from two of the .ec files is HIGHER than the coverage from all four of them. This is strange to me because intuitively the cumulative coverage should be only increased by adding more .ec files. Specifically, I have the following metadata and two ec folders:
q
├── astro.pickle
├── ec1
│   ├── onstop_coverage_1610318914750.ec
│   └── onstop_coverage_1610318965061.ec
└── ec2
    ├── onstop_coverage_1610318902020.ec
    ├── onstop_coverage_1610318914750.ec
    ├── onstop_coverage_1610318947862.ec
    └── onstop_coverage_1610318965061.ec

After executing acv report com.metago.astro -p ~/q/astro.pickle -ec ~/q/ec1 -o ~/q, I got a coverage: 76534 of 627614 | 12.19444%
Then, after executing acv report com.metago.astro -p ~/q/astro.pickle -ec ~/q/ec2 -o ~/q, I got a coverage: 69355 of 627614 | 11.05058%
This is weird because the files in ec1 are also in ec2. How can the cumulative coverage decrease by just adding more .ec files?
The files can be downloaded here:
https://drive.google.com/file/d/1W---ONiffuW9Shje1Hth2lgkghQY-5H0/view?usp=sharing

Hi @jwlin, in this version acvtool calls the stop command each time the report generated. It needs a small change in the code to achieve cumulative measurements. Could you try for now to use acvtool from the another project https://github.com/pilgun/acvcut? It contains a more fresh acvtool version (just use the same way acvtool.py). To generate cumulative ec files call acvtool.py stop command. You will also need to kill your app when you want to start measurements from scratch again. I will later merge this update into to acvtool project.

Hi @jwlin, in this version acvtool calls the stop command each time the report generated. It needs a small change in the code to achieve cumulative measurements. Could you try for now to use acvtool from the another project https://github.com/pilgun/acvcut? It contains a more fresh acvtool version (just use the same way acvtool.py). To generate cumulative ec files call acvtool.py stop command. You will also need to kill your app when you want to start measurements from scratch again. I will later merge this update into to acvtool project.

Thank you @pilgun for the quick response. Do I need to collect the ec files again with the acvtool.py of acvcut? I tried to start over by instrumenting the apk with the acvtool.py of acvcut, but the instrumented apk crashed after installation.
I also tried to generate the report for the same ec files with the acvtool.py of acvcut, but got errors as follows:
$ python acvtool.py report com.metago.astro -p ~/q/astro.pickle -ec ~/q/ec1 -o ~/q
report generating...
concise report...
Traceback (most recent call last):
  File "acvtool.py", line 247, in <module>
    main()
  File "acvtool.py", line 244, in main
    run_actions(parser, args)
  File "acvtool.py", line 96, in run_actions
    ignore_filter=args.stubs)
  File "/home/junwei/acvcut/smiler/reporter.py", line 41, in generate
    save_concise_report(report_dir, ec_files, smalitree)
  File "/home/junwei/acvcut/smiler/reporter.py", line 56, in save_concise_report
    cover_smalitree_cumulatively(st, ec_coverage)
  File "/home/junwei/acvcut/smiler/reporter.py", line 369, in cover_smalitree_cumulatively
    if cl.is_coverable():
  File "/home/junwei/acvcut/smiler/instrumenting/apkil/classnode.py", line 260, in is_coverable
    if self.ignore:
AttributeError: 'ClassNode' object has no attribute 'ignore'

Thank you @pilgun for the quick response. Do I need to collect the ec files again with the acvtool.py of acvcut? I tried to start over by instrumenting the apk with the acvtool.py of acvcut, but the instrumented apk crashed after installation.
I also tried to generate the report for the same ec files with the acvtool.py of acvcut, but got errors as follows:
$ python acvtool.py report com.metago.astro -p ~/q/astro.pickle -ec ~/q/ec1 -o ~/q
report generating...
concise report...
Traceback (most recent call last):
  File "acvtool.py", line 247, in <module>
    main()
  File "acvtool.py", line 244, in main
    run_actions(parser, args)
  File "acvtool.py", line 96, in run_actions
    ignore_filter=args.stubs)
  File "/home/junwei/acvcut/smiler/reporter.py", line 41, in generate
    save_concise_report(report_dir, ec_files, smalitree)
  File "/home/junwei/acvcut/smiler/reporter.py", line 56, in save_concise_report
    cover_smalitree_cumulatively(st, ec_coverage)
  File "/home/junwei/acvcut/smiler/reporter.py", line 369, in cover_smalitree_cumulatively
    if cl.is_coverable():
  File "/home/junwei/acvcut/smiler/instrumenting/apkil/classnode.py", line 260, in is_coverable
    if self.ignore:
AttributeError: 'ClassNode' object has no attribute 'ignore'

Yes, you need to instrument it with the new version and use the new pickle file.

Yes, you need to instrument it with the new version and use the new pickle file.

I tried to instrument the app with the new acvtool.py but the instrumented app crashes...

I tried to instrument the app with the new acvtool.py but the instrumented app crashes...

that's a pitty, would you mind to share the apk? maybe I could take a look this week

that's a pitty, would you mind to share the apk? maybe I could take a look this week

I am a little confused about this tool (the current version). Does it support multiple .ec files and report cumulative coverage? Particularly,

I "acv start -> test manually -> Ctrl + C" for three times, and the tool generates three .ec files. When I acv report, does the tool correctly report the cumulative coverage?
Alternatively, If I manually pull the three .ec files and acv report with -ec option, does the tool correctly report the cumulative coverage?

I am a little confused about this tool (the current version). Does it support multiple .ec files and report cumulative coverage? Particularly,

I "acv start -> test manually -> Ctrl + C" for three times, and the tool generates three .ec files. When I acv report, does the tool correctly report the cumulative coverage?
Alternatively, If I manually pull the three .ec files and acv report with -ec option, does the tool correctly report the cumulative coverage?

The current version (old) may not generate several ec files in a single run. The terminal should take Ctrl+C only once, the ec file gets generated and the tool finalises the app. Extra files could left from other unsuccessful tries.

The current version (old) may not generate several ec files in a single run. The terminal should take Ctrl+C only once, the ec file gets generated and the tool finalises the app. Extra files could left from other unsuccessful tries.

For the second part of the question, the tool can generate a report from all ec files from a directory. The report is cumulative in this case. But not sure that it will be correct, because with this version you can run tests only not cumulatively.

For the second part of the question, the tool can generate a report from all ec files from a directory. The report is cumulative in this case. But not sure that it will be correct, because with this version you can run tests only not cumulatively.

Thank you for the clarification. If I understand correctly, It basically says that if I have multiple test case scenarios, I need to run them all in a single run to get the coverage as a whole, i.e., with only one .ec file. The tool (current version) does not support multiple "single runs" and then reporting the cumulative coverage.

Thank you for the clarification. If I understand correctly, It basically says that if I have multiple test case scenarios, I need to run them all in a single run to get the coverage as a whole, i.e., with only one .ec file. The tool (current version) does not support multiple "single runs" and then reporting the cumulative coverage.

Ok, it took time to rethink your question. I think it's correct to apply tests on separate app runs, pull ec files, then put them into the same directory and generate the report. The report in this case represent the cumulative coverage of all runs. E.g. if you put ec1 and ec2 in the same folder and generate the report you may get at least 12.19444% from your example.

Ok, it took time to rethink your question. I think it's correct to apply tests on separate app runs, pull ec files, then put them into the same directory and generate the report. The report in this case represent the cumulative coverage of all runs. E.g. if you put ec1 and ec2 in the same folder and generate the report you may get at least 12.19444% from your example.

No. ec2 already includes the files in ec1. In other words, ec1 is a subset of ec2, but reports larger coverage than ec2.

No. ec2 already includes the files in ec1. In other words, ec1 is a subset of ec2, but reports larger coverage than ec2.


[react-form-state] non-dirty sub-fields get validated on blur of a fellow sub-field

Overview
Sub-fields of <FormState.Nested /> and <FormState.List /> validate on blur of any other sub-field. This leads to early validation of non-dirty fields, and a buggy looking UI, e.g.:

Consuming repo

facebook-marketplace

See https://github.com/Shopify/facebook-marketplace/issues/695

This is a fun one. I'm planning to do a bit of an overhaul of how the nested valdiators work for v1.0, so this will definitely be one of the main things to fix there.

@debchn so I can make sure I take into account everything going on here:

what version of react-form-state is this?
are these custom validators?
are you using validate or validateRequired for these validators?

Hey @TheMallen, sorry for the late reply (was on vacation).
When I made this issue, we were on v0.2.3 and only using validate (with custom validators) within validateNested.

I think if the following are true this is mitigated:

you're on .5.x
you're using the non-required family of validators / validation helpers

example video: https://monosnap.com/file/Kis2waCreHhLFB7uzNTI2ocRL0zKYD
As far as solving this more generally, the way that validators and Nested/List work is going to have to go through a big change to fix some of the issues with them, I'm hoping to look at that when the current sprint is over.
If you have specific cases for this and the other bugs like it, it'd be super helpful if you could fork the sandbox with a reproducible case :)

@TheMallen using your example form, if I start with blank values in the nested section and then change the "values" input (which has no validation on it) the sibling field "option" still gets validated. Is there a way to avoid that? This is the biggest issue that my team is facing.

The only workaround atm would be not using the required family of validators, since the others bail on an empty input. Otherwise no that's exactly why Nested / List need to be rethought

Thanks @TheMallen
Our team is facing the same issue as @jfangrad's (i.e. we need required validators on our nested fields). Interested in the big re-think—keep us in the loop!
cc @HiFiSamurai

Hi, is there anything planned in the near future to address this?

@TheMallen Hi, we are facing the same issue. If anybody here has any suggestion for checking nested fields that are required, we would love to know.


Fixed demo by moving sample audio into repo (avoids CORS)

Soundcloud doesn't serve audio with the CORS header, so it's not possible to run it through an AnalyserNode. To remedy this I've included a audio sample in the repository itself, in order to avoid the CORS problems. The audio is licensed CC-BY-NC 3.0, available at http://freemusicarchive.org/music/Kai_Engel/Irsens_Tale/Kai_Engel_-Irsens_Tale-_04_Moonlight_Reprise I've included attribution on the demo.html page.

this is a great fix, I hope it gets merged in!


[3.7.1] Won't set Chromium 'release_version' on Linux

Hi,
I'm getting this error when running specs with Capybara:
NoMethodError:
            undefined method `<' for nil:NilClass

# /home/user/.rvm/gems/ruby-2.5.3@gemset/gems/webdrivers-3.7.1/lib/webdrivers/chromedriver.rb:22:in `latest_version'
# /home/user/.rvm/gems/ruby-2.5.3@gemset/gems/webdrivers-3.7.1/lib/webdrivers/common.rb:37:in `desired_version'
# /home/user/.rvm/gems/ruby-2.5.3@gemset/gems/webdrivers-3.7.1/lib/webdrivers/common.rb:16:in `update'
# /home/user/.rvm/gems/ruby-2.5.3@gemset/gems/webdrivers-3.7.1/lib/webdrivers/selenium.rb:7:in `driver_path'
# /home/userr/.rvm/gems/ruby-2.5.3@gemset/gems/selenium-webdriver-3.141.0/lib/selenium/webdriver/chrome/driver.rb:39:in `initialize'
# /home/userr/.rvm/gems/ruby-2.5.3@gemset/gems/selenium-webdriver-3.141.0/lib/selenium/webdriver/common/driver.rb:44:in `new'
# /home/userr/.rvm/gems/ruby-2.5.3@gemset/gems/selenium-webdriver-3.141.0/lib/selenium/webdriver/common/driver.rb:44:in `for'
# /home/user/.rvm/gems/ruby-2.5.3@gemset/gems/selenium-webdriver-3.141.0/lib/selenium/webdriver.rb:86:in `for'
# /home/user/.rvm/gems/ruby-2.5.3@gemset/gems/capybara-2.18.0/lib/capybara/selenium/driver.rb:23:in `browser'
.
.
.

Debugging the file you can see the regexp in method chrome_version is not working propperly when the Chromium version has a dot before the actual version number:
[18, 27] in<EMAIL_ADDRESS>   18:       def latest_version
   19:         raise StandardError, 'Can not reach site' unless site_available?
   20:
   21:         # Versions before 70 do not have a LATEST_RELEASE file
   22:         byebug
=> 23:         return Gem::Version.new('2.46') if release_version < '70.0.3538'
   24:
   25:         # Latest webdriver release for installed Chrome version
   26:         release_file     = "LATEST_RELEASE_#{release_version}"
   27:         latest_available = get(URI.join(base_url, release_file))
(byebug) release_version
nil
(byebug) chrome_version[/\d+\.\d+\.\d+/]
nil
(byebug) chrome_on_linux
"Using PPAPI flash.\n73.0.3683.75"
(byebug) "Using PPAPI flash.\n73.0.3683.75"[/(\d|\.)+/]
"."
(byebug) "Using PPAPI flash\n73.0.3683.75"[/(\d|\.)+/]
"73.0.3683.75"

# Testing modified regexp
(byebug) "Using PPAPI flash.\n73.0.3683.75"[/\d(\d|\.)+/]
"73.0.3683.75"
(byebug) "Using PPAPI flash\n73.0.3683.75"[/\d(\d|\.)+/]
"73.0.3683.75"
(byebug) "Using PPAPI flash 73.0.3683.75"[/\d(\d|\.)+/]
"73.0.3683.75"

I'm runnig Linux Mint 19.1 with Chromium Version 73.0.3683.75 (Official Build) Built on Ubuntu.
Thanks

Hi @camador, the fix for this was merged yesterday and will be available in v3.7.2 soon. Please see #43  for details.

Ooops, sorry for posting solved issues. Thanks


Use of undeclared type PropertyListEncoder on Swift 5.0

PropertyListEncoder and PropertyListDecoder are both unavailable prior to Swift 5.1, causing a build error for Swift 5.0.
From what I've gathered they're available on the swift-5.1-branch, but missing on the swift-5.0-branch.

LGTM

  Generated by :no_entry_sign: Danger Swift against 21f51ad1890e7dc750e3c6bda8f08ccc4ac5c364

Are you planning to make any more changes or I can merge this?

I'm not planning any further changes, you can merge this 👍


Using out of scope variables in own show_data method

Hi,
the past days I've been working on creating my first plot recipe for Makie and extending its show_data method for DataInspector.
I'm not going to lie: I had my moments where I thought that I'll never get it working. 😅 But I think I've now understood how things work and my plot is created as I want it to. Yay for that already!
For now my own plot type is relatively simple and can be described as a wrapper around the scatter primitive. Extending on the functionality of DataInspector, I'm then trying to access an outside variable to use its value as the tooltip string.
using GLMakie

outside_var = "Hello from the other side."

# simple MyPlot recipe and plot! method go here

function GLMakie.Makie.show_data(inspector::DataInspector, plot::MyPlot, idx, ::Scatter)
    a = inspector.plot.attributes
    # method body goes here
    a._display_text[] = outside_var
    # rest of method body
end

Hovering a scatter point then naturally produces the UndefVarError: outside_var not defined.
What would be the best way to let the show_data method access my outside variable?
Currently I can think of two ways to do this:

Should I put it in the Attributes of the @recipe(MyPlot) body and then use it in show_data like plot[:outside_var]? Would outside_var then have to be made an Observable for the tooltip to be able to track its current value or will it just be displayed on every new mouse-over/hover? I assume the former is the case. This seems like the natural approach to me but I'm unsure how to specifically tackle it.
Should I add an extra (kw)arg to show_data itself? If so, which other function would have to be changed/extended then as well? This would seem like hopping through a few too many hoops to achieve such a simple thing.

This whole question might seem a bit silly, sorry if that's the case. I've put in quite a bit of time to get to this point (i.e. generally understanding the recipe system) and would just appreciate a bit of guidance if possible. Thanks in advance for your tips, tricks and advice! 🙂

I don't think adding (kw)args to show_data makes much sense. It's called on mouse position updates, after some filtering (i.e. the inspectable attribute). You'd have to mess with the constructor and every show_data method to add (kw)args, I think.
I'm a bit surprised that your example doesn't work. Something like
x = 1.0
module A
foo(x) = nothing
end
using .A
A.foo() = x
A.foo()

works for me. Maybe you just need a global outside_var in there?
Otherwise I would add it to the plots attributes. Attributes will convert the inputs to nodes on its own and show_data is currently called on mouse movement, so you can use a._display_text[] = plot.outside_var[].
@recipe(MyPlot, points) do scene
    attr = Attributes(
        outside_var = "init"
    )
end

# plot! code

function GLMakie.Makie.show_data(inspector::DataInspector, plot::MyPlot, idx, ::Scatter)
    a = inspector.plot.attributes
    # method body goes here
    a._display_text[] = plot.outside_var[]
    # rest of method body
end

You'd have to mess with the constructor and every show_data method to add (kw)args, I think.

Yes, that's what I expected. Thanks for the confirmation.

Maybe you just need a global outside_var in there?

I've tried that and it sadly didn't work. :/

Otherwise I would add it to the plots attributes. Attributes will convert the inputs to nodes on its own and show_data is currently called on mouse movement, so you can use a._display_text[] = plot.outside_var[].

Great, thanks for the info. I'm going to try that out and report back if it works as expected.

I'm a bit surprised that your example doesn't work.

So was I and that's why I posted on here. Now I was curious and tried my simplified MWE from above again. This time it worked as expected and the tooltip updates when outside_var changes.

Not sure what went wrong last time I tried it out. I probably messed sth up somewhere in the process.
Interestingly enough though, it still doesn't work once I use my actual implementation in which I don't add methods to `GLMakie.Makie´. Instead I implemented the Makie extensions within another used module. Here's a proper MWE because I think it might be helpful to know how I'm doing things:
module MakieMyPlot

using Makie

@recipe(MyPlot, pos) do scene
    Attributes(
        markersize = 10,
    )
end

function Makie.plot!(myplot::MyPlot{<:Tuple{Vector{Point2f0}}})
    scatter!(myplot, myplot[:pos]; markersize=myplot[:markersize])
    return myplot
end

function Makie.show_data(inspector::DataInspector, plot::MyPlot, idx, ::Scatter)
    a = inspector.plot.attributes
    scene = Makie.parent_scene(plot)

    proj_pos = Makie.shift_project(scene, plot, to_ndim(Point3f0, plot[1][][idx], 0))
    Makie.update_tooltip_alignment!(inspector, proj_pos)
    ms = plot.markersize[]

    global outside_var
    a._display_text[] = outside_var
    a._bbox2D[] = FRect2D(proj_pos .- 0.5 .* ms .- Vec2f0(5), Vec2f0(ms) .+ Vec2f0(10))
    a._px_bbox_visible[] = true
    a._bbox_visible[] = false
    a._visible[] = true

    return true
end

end

using .MakieMyPlot
using GLMakie

fig, = myplot(rand(Point2f0, 20))
outside_var = "Hello from the other side."
inspector = DataInspector(fig.scene)
fig

outside_var remains undefined. So the question seems to be: Which scoping rule am I blissfully ignoring right now?

Main.outside_var works

Main.outside_var works

That's true. But once I want to avoid global variables by wrapping things in a function, I'm stuck again.
function create_plot()
    fig, = myplot(rand(Point2f0, 20))
    outside_var = "Hello from the other side."
    inspector = DataInspector(fig.scene)
    fig
end

I suppose my best bet is to add my outside_var as a positional argument to the recipe as this will allow for more generic usage of the recipe and doesn't constrain that outside_var is directly defined in Main.
@recipe(MyPlot, pos, outside_var) do scene
    Attributes(
        markersize = 10,
    )
end

pos = rand(Point2f0, 20)
outside_var = "Hello from the other side."
myplot(pos, outside_var)

Inside show_data I can then write
a._display_text[] = plot.outside_var[]

This seems to have solved my problems/questions and works even if myplot is called from within a function. Thanks for your time and answers. :)


Ability to use the terminal while the webui is running

Is your feature request related to a problem? Please describe.
Until all the features available in terminal are usable in webui, I'd like to be able to use the terminal while the webui is running.
Describe the solution you'd like
I'd like to be able to use the terminal while the webui is running.
Describe alternatives you've considered
Running two instances is an option but requires extra system resources and can cause conflicts if both try to use the gpu at the same time
Additional context
Perhaps a way to enter terminal commands in the webui directly is another option

Entering invoke REPL commands in the web UI would be pretty cool and, I believe, fairly straightforward to implement - we can parse the command and send it to the generator just like we do currently with the web ui parameters. Error handling could be tricky, though...

Have been thinking about this, and if we do it, I think the one "UX" feature we'd want to include would be a command "starter" - Basically, copy whatever settings/info I have in the WebUI into the CLI command, which I can then edit/add to.
Very useful in cases where a CLI feature hasn't been made available in the WebGUI yet, and prevents having to retype everything

I found an older issue #543 with a similar request and a good idea - add a checkbox that runs the prompt as a REPL command.
Unfortunately, if we just parsed the REPL command and passed the parameters to the web UI's server, we would still be limited to features supported by the web UI. So I think making this work in a useful way (i.e. exposing CLI-only features on the UI) this is a bit trickier than I imagined initially.

It doesn't necessarily need to be available in the webui, though I understand why we'd want that. But my original thought was to just have the command line invoke>  remain available in the terminal while the webui is running

Right. I'm going to take the coward's way out and say this is beyond the scope of the current server, and ask @Kyle0654  to consider this for the WIP backend.

I've considered it 🙂. I think it would end up being a CLI that uses the web API though, otherwise we risk service conflicts in the new architecture.

I would like to use the Invoke AI model as a serving for API calls - so this feature would be super helpful to interact with the backend.


Add method to prevent dropdown from opening on click

Could be useful to manage dropdown state with mouse enter and mouse out on desktop.
If mousernter event is detected, block click event until mouseout occurs. By doing this it will be possible to prevent dropdown from collapsing on click for users with mouse. At the same time, the click functionality is preserved for users without mouse, and the dropdown will also be operable on touch devices.

Done


create image from deploy transaction if necessary

If chaincode startup fails with "image not found", we need to create the image from the transaction and retry. This is especially needed if there's no replay on validators.

@muralisrini is this still an issue? Or can it be closed?

@christo4ferris This can be closed...we attempt to recreate the image if not found when launching a chaincode.


Attention slicing isn't lowering memory usage like it should

What happened?
I'm working on Stable Diffusion, lowering it through Torch-MLIR from a PyTorch model. The model has a tunable parameter called attention slicing which splits the batch matmul for the attention heads into several separate batch matmuls that are run in sequence as seen here. When I lower this to Linalg and run it through IREE-Runtime, however, I'm not seeing any significant memory savings. I've written and attached an isolated test case and while there is some change in memory usage, it is very small.
What my coworkers and I suspect is happening is that IREE-Runtime is running the dispatches in parallel or some other kind of optimization that is "undoing" the attention slicing. How can we ensure that IREE-Runtime runs this code the same way it does in PyTorch or, barring that, how can we make IREE-Runtime run in a memory-conscious way? Is there some way to set a memory cap so that the runtime will sacrifice some speed for memory usage?
Steps to reproduce your issue

Run iree-compile --iree-input-type=tm_tensor --iree-hal-target-backends=llvm-cpu --iree-stream-resource-index-bits=64 --iree-vm-target-index-bits=64 sliced.mlir -o sliced.vmfb.
Run iree-benchmark-module --module=sliced.vmfb --device=local-task --function=forward --input=128x640x640xf32 --input=128x640x640xf32 --input=128x640x640xf32.
Repeat for the unsliced variant.
Compare the memory usage.

What component(s) does this issue relate to?
Runtime
Version information
0cdfe9888d4f6544bc182c09cac1538285452d80
Additional context
No response

Here is the benchmarking info for the unsliced test case, and here is the sliced one.

Oh, and here is the script used to generate those just in case.

There's a few things that could be happening; the most likely one can be isolated with --iree-stream-partitioning-favor=min-peak-memory and if that makes no real difference then it's likely something with fusion. But I suspect that's it. We default to --iree-stream-partitioning-favor=max-concurrency.
It'll be a losing battle to try to get any framework to match any other framework exactly with such high-level inputs as what's an optimization in one may be a pessimization in another. By splitting like this you've said "hey compiler, we have a tremendous amount of independently/concurrently executable work" and IREE happens to be taking advantage of that by assuming your intent was to execute that work fast. Of course, if what you care about more is peak memory consumption then that is the wrong assumption for the compiler to make, but it doesn't know that :)
20% savings seems nice, but what are you targeting?
fyi if you pass --print_statistics to iree-benchmark-module it should show you peak/churn for what we allocate - it's a much less noisy number as it only includes the allocations influenced by the compiler instead of application stuff.

What kind of savings is pytorch seeing? We can compare that with the numbers with min-peak-memory and see how far off we still are.
There's still an entire phase of partitioning we aren't doing where we reorder dispatches to reduce peak usage - depending on the output we see that may be the next lowest-hanging fruit. We may also be missing some dispatch fusion we should have based on the patterns used during splitting and we can verify that we aren't keeping transient tensors live any longer than required.

It looks like IREE is working as intended after testing with different amounts of slicing: no slicing, automatic slicing, max slicing. Max slicing is seeing savings of ~40% which lines up with PyTorch. It looks like the memory usage oddities we're seeing are unrelated to the attention slicing and that the peak memory usage was misleading.

Yay! You should still give --iree-stream-partitioning-favor=min-peak-memory a shot to see if it helps.

Interestingly min-peak-memory is faster on RDNA2 unet by 4ms. It has no effect on RDNA3. All these tests were without attention slicing enabled. Need to do a more controlled study.
https://github.com/nod-ai/SHARK/issues/1202#issuecomment-1474660228


create_user: Refactor notify_created_user function.

This PR refactors the notify_created_user function to call format_user_row twice with different
parameters instead of modifying the person object returned by format_user_row.
This change makes the code somewhat more easy to understand than it was before.
Discussion - https://github.com/zulip/zulip/pull/22994#discussion_r1088445491 and https://github.com/zulip/zulip/pull/24529#issuecomment-1450802875

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Posted one comment, otherwise lgtm.

@timabbott updated the PR.

Looks great, merged, thanks @sahil839!


Still getting Method not found...

Hi guys,
Im f*cking crazy whith PhpStorm when tried to fix a popular error Method 'Bla bla' not found in class.
I have been searching for days in google whith hope to find out the way for this problem but no luck.
Almost every singel topic I have read are pointing me to this laravel-ide-helper but after Install thousand times (with fresh laravel project), PhpStorm still not recognize those damn method.
I also install laravel plugin in PhpStorm but still not work too, what can I do now?
Here is my code.
<?php

namespace App\Http\Controllers;

use App\Article;
use App\Menu;
use Illuminate\Http\Request;
use App\Http\Requests;
use App\Http\Controllers\Controller;
use Illuminate\Http\Requestst;
use Illuminate\Support\Facades\Input;

class PagesController extends Controller
{
    public function index()
    {
        $article = Article::latest('published_at')->get(); // Method 'latest' not found in class...
        $data = array(
            'articles' => $article
        );
        return view('pages.index' , compact($data));
    }
}

Please help.

Helper have not all methods. For example latest() not in helper list...

Article::latest is a method of your Model I supose ... You have to make
"php artisan ide-helper:models" to fix your Problem ...
best regards
Kvintiljanov Aleksandr<EMAIL_ADDRESS>schrieb am Do., 3. Dez.
2015 um 19:14 Uhr:

Helper have not all methods. For example latest() not in helper list...
—
Reply to this email directly or view it on GitHub
https://github.com/barryvdh/laravel-ide-helper/issues/283#issuecomment-161736141
.

I ran "php artisan ide-helper:models" -> Yes, it wrote some line in my models file and still not work
I ran again "php artisan ide-helper:models" -> No, it created a new file called _ide_helper_models.php file but still not work.
I access file _ide_helper.php and add
/**
         * Add an "order by" clause for a timestamp to the query.
         *
         * @param string $column
         * @return \Illuminate\Database\Query\Builder|static
         * @static
         */
        public static function latest($column = 'created_at'){
            return \Illuminate\Database\Query\Builder::latest($column);
        }

Still not work, even I clear cache and restart PhpStorm

@vietnguyen09 , make sure you don't have duplicate class declaration, as after _ide_helper_models.php is created, it copies model declarations in. You should take docblocks from that file and put them in your actual models and then remove the _ide_helper_models.php file. You might still miss some Models related methods (especially you extend Model instead of \Eloquent), but we have issues discussing those particular cases in detail, please refer to them.

latest is a method in query builder
https://laravel.com/api/5.3/Illuminate/Database/Query/Builder.html#method_latest
But this works in my laravel 5.2 installation with ide-helper, I have exactly what you posted as code in my _ide_helper.php file and it seems to be enough.
In the past I had a problem with phpstorm laravel plugin when I did not open the project directory directly. If I opened a folder few folders outside, it did not work well with models. Probably not related, but just saying. There is a path setting in phpstorm laravel plugin but I couldnt get it to work. Now I only open the folder where my composer.json sits....

@barryvdh old and outdated, all mentioned things work for me, I vote for closing :}


fix grep path for windows

@nicks @victorwuky @westongcooper

hang on, lemme test this on my windows machine

hmmm...i think we need to do a lot more than this, when i tested this on windows

xargs doesn't exist
uname doesn't exist
rm -fr doesn't exist

i think we might need to do something more clever here, gmas

@nicks you're right. I'm the original developer of this extension, but this entire time, I've neglected the fact that I'm actually running Tilt within MinGW which ends up being a (sometimes strange) mixture of linux/windows. In this particular case, Tilt's os.path* methods are detecting windows and converting paths to windows paths, but then those vars are being passed to local linux (mingw) commands. So grep resolves, but chokes on the windows paths.
Not really sure what to do here, then... if I add windows-equivalent commands, those will break inside mingw...
Suggestions?


Implement Add/Sub for Nullable<Date/Timestamp>

Fixes #1109
Missing tests, I'll write them tomorrow (too sleepy right now 😴 )

I don't think we need to make a macro for this.

This will be merged in #1123


FGType example in docs doesn't work

https://stk.readthedocs.io/en/latest/stk.molecular.functional_groups.html#stk.molecular.functional_groups.FGType.get_functional_groups
The example listed here doesn't work, I get TypeError: __init__() missing 1 required positional argument: 'name'

Fixed in #95


Disable IPv6 for /plans/provision/virtual in CI

As discussed, IPv6 does not work in CI, and adds significant overhead to testing with testcloud.
Pull Request Checklist

[x] implement the feature

There are just two failing tests:

/tests/prepare/install
/tests/prepare/recommend

Both failed because of what seems to be a temporary repository problem of CoreOS.

@psss was it faster?


feat: counting elements in an interval with given residue

Also, is there a better place to put these theorems?

bors r+
Thanks!