Refactor RNGs

Apologies for the somewhat large PR!
Note: This PR relies on a just-released version of Sciris, so please update (pip install sciris --upgrade, or pip install -e . for Starsim).
New RNGs & distributions

Replaces ss.SingleRNG(), ss.MultiRNG(), ss.ScipyDistribution(), and ss.ScipyHistogram() with a single ss.Dist() class. The random.py and distributions.py submodules have been removed, and dists.py has been added.
The ss.Dist class uses np.random.default_rng() rather than scipy.stats by default, although a scipy.stats distribution can be supplied as an alternative. While comparable in most cases, in some instances Numpy is up to 4x faster (including, critically, for Bernoulli distributions).
ss.Dist has various tools for understanding what it's doing, including dist.show_state(), dist.plot_hist()`, etc.
Also removes ss.options.multirng (this branch is equivalent to it being always on).
Removes duplicate logic for transmission (make_new_cases()), using an approach similar to the previous make_new_cases_singlerng() logic (though we could instead use the make_new_cases_multirng() logic; it should be similar in terms of performance and outcome, though slightly more complex code).
Adds new custom distributions such as ss.choice() and ss.delta().
These distributions can be called directly, e.g. dist = ss.weibull(a=2); dist(5) will return 5 random variates from a Weibull distribution.
Instead of being manually initialized based on the name, the Sim object is parsed and all distributions will be initialized with a unique identifier based on their place in the object (e.g. sim.diseases.sir.pars.dur_inf), which is used to set their unique seed. (Note: this relies on Sciris 3.1.5, which hasn't been released yet, so sc_nested.py has been copied here as a temporary fix.)

Examples
# 1. Create and use a simple distribution

# Old:
import starsim as ss
import scipy.stats as sps
dist = sps.norm(loc=4, scale=3)
rng = ss.MultiRNG('normal')
rng.initialize(container=None, slots=None)
dist.random_state = rng
d = ss.ScipyDistribution(dist)
print(d.rvs(5))

# New:
import starsim as ss
d = ss.Dist('normal', loc=4, scale=3)
print(d(5)) # dist() is an alias to dist.rvs()

# or as a one-liner:
print(ss.normal(4,3)(5))

# 2. Register distributions with the sim (pseudocode)

# Old:
sim = ss.Sim(...)
sim.rng_container = ss.RNGContainer()
sim.rng_container.initialize(seed)
for module in sim.modules:
  items = list(module.pars.items()) + list(module.__dict__.items()) # Isn't recursive
  for key,val in items:
    if isinstance(val, ss.ScipyDistribution):
      val.initialize(sim)

# New:
sim = ss.Sim(...)
sim.dists = ss.Dists(sim)
dists = sim.dists.find_dists() # Parse full Sim object, link to all ss.Dist objects regardless of depth
for dist in dists:
  dist.initialize(sim)

# 3. Show all distributions in the sim

# Old:
sim.rng_container._rngs # Shows RNG names, but have to manually find the corresponding ScipyDistributions

# New:
sim.dists.dists # Shows distributions as well as RNGs

Other changes

This PR also fixes bugs with lognormal parameters, and makes it explicit whether the parameters are for the underlying normal distribution (ss.lognorm_u(), the Numpy/SciPy default, equivalent to ss.lognorm_mean() previously) or the "overlying" lognormal distribution (ss.lognorm_o(), equivalent to ss.lognorm() previously).
Renames ss.dx, ss.tx, ss.vx to ss.Dx, ss.Tx, ss.Vx.
Removed set_numba_seed() as a duplicate of set_seed().

FAQ

Q: Why is this PR so big? Aren't PRs meant to be small?

A: Yes, but distributions are central to how Starsim works, so it wasn't possible to make incremental changes. I promise I opened this PR as soon as I could get the tests to pass!

Q: Code is supposed to be modular, so why combine ss.MultiRNG and ss.ScipyDistribution into a single class?

A: These objects were only ever used together, so creating them sepearately created extra housekeeping work.

Q: Without ss.options.rng, how can we compare results with and without CRN?

A: Using different seeds for sims will give a pretty good approximation of what using a centralized RNG would look like. Alternatively, you could parse the Dist instances and replace all the RNGs with the same one (eek!). From a user perspective, keeping an option that we do not recommend using is confusing.

Q: How is performance on this branch?

A: Not as fast as I'd like. Drawing random numbers is as fast or faster than before (up to 4x faster for Bernoulli distributions, for example), but overall performance is about the same as multirng='multi' was before. Going to spend some time figuring out why.

Q: What happened with slots?

A: These still need to be reimplemented (currently, UIDs are being treated as slots, which is valid except in the case of different numbers of births between sims). However, first, I'd like to write a test that fails with the current implementation, and demonstrate that adding slots back in fixes it. (There is test_repeat_slots, but just showed that the same random numbers are drawn for each slot, whereas we want to show that simulations diverge without using slots but don't diverge when using them.)

Q: What else still needs to be done beyond this PR?

A: See #395. Main issues are performance (including profiling), slots (including writing tests), simplifying how distributions are initialized, and various small "todo" comments for refactoring the code.

Closes #243, #316, #348, #312, #334, #340, #313, #322. Opens #395.

Good, I like the simplifications this enables!
Just starting to form thoughts and questions:

If a users wants to run a simulation with two of the same module (e.g. RSV-A and RSV-B), won't the trace be identical, and hence repeated random numbers? It seems like we'd need to use the name/label in addition to the trace.
We need slots, agent uids alone doesn't achieve CRN. (I see how slots could be provided instead of uids, might just need to clean up the interface?)
I suspect callable parameters are not working correctly.

It seems the call only occurs when the distribution is created (make_dist), so a user would need to regenerate the distribution on every timestep to use a callable with parameters that vary (e.g. age)
The key resolution of callable parameters with non-trivial distributions seems to have been lost (although I am still familiarizing myself with this PR, so apologies if I have simply missed it).

Transmission

The approach in this PR calls rng_source and rng_target twice for each network, breaking CRN if the max slots differ at any point in time.
I believe this code will raise an exception if strict is enabled?
Also just thinking through what will happen when comparing simulations, one with network B and the other with networks A and B.

Regarding performance, the unified approach to transmission uses a lot more random numbers that is needed if not attempting CRN, especially if slot_scale is large

Thanks @daniel-klein , very helpful comments! I agree that a lot of these are likely to be issues. I can work on them, but if you could help write tests for them (that fail on this branch) that would help me fix them. To respond to your specific comments:
Won't the trace be identical for two modules with the same name?
No, because it's based on the location within the python object (via sc.iterobj()), and two objects can't have the same "physical" location. For example:
import starsim as ss

sir1 = ss.SIR(name='sir1')
sir2 = ss.SIR(name='sir2')

sim = ss.Sim(pars=dict(networks='random'), diseases=[sir1, sir2])
sim.run()
sim.dists.dists.keys()

["('people', 'age_data_dist')",
 "('pars', 'sir1', 'dur_inf')",
 "('pars', 'sir1', 'dur_inf', 'module', 'rng_target')",
 "('pars', 'sir1', 'dur_inf', 'module', 'rng_source')",
 "('pars', 'sir1', 'init_prev')",
 "('pars', 'sir1', 'p_death')",
 "('pars', 'sir2', 'dur_inf')",
 "('pars', 'sir2', 'dur_inf', 'module', 'rng_target')",
 "('pars', 'sir2', 'dur_inf', 'module', 'rng_source')",
 "('pars', 'sir2', 'init_prev')",
 "('pars', 'sir2', 'p_death')"]

You can't create two diseases of the same type with the same name anyway (ValueError: Cannot add object "sir" since already present in ndict "{self._type"} with keys: sir), but if you do that, everything else is handled automatically. Even if you somehow could (e.g. they got appended to a list), the trace would still work.
Callable parameters

These are working insofar as test_distributions.py:test_callable() passes. But there might be more to do here. I believe that callable parameters works with arbitrary distributions: they're called, which creates an array, which is used as the parameter for the distribution for those distributions which support it.
dist.make_dist() is called every time dist.rvs() is called. Except in the case of a frozen SciPy distribution, this is not creating a new distribution, but just getting an attribute from np.random.default_rng(), so is fast. You're right though that this doesn't work with frozen SciPy distributions. My personal vote would be to not support the combination of callable parameters with frozen distributions -- I believe this is working against how frozen distributions are supposed to be used, which may be why the code required was so complex.

Transmission
Yes, we could probably optimize how the random numbers are called (I did have a dist.jump() in the loop, but took it out -- we could restore it however). For slots, I think max_slots = n_agents might be fine? We can check what performance difference it makes. But my thinking is that with ~100k agents, and new agents assigned at random, the probability of two agents sharing the same slot and experiencing the same disease challenge on the same timestep is very slight. And even if that does happen, would it actually affect the results? We should write a test (maybe a devtest) to check.

@cliffckerr I think you already addressed most if not all of @daniel-klein's comments. Regarding attainment of CRN, instead of max_slots=n_agents, I think we need slack in the number of slots. Could we allow max_slots=random.rand(n_agents+1,1000) based on the total number agents to be created in the simulation?

Thanks for the nice responses, @cliffckerr!
The module "trace" is really nice, yes!
But the callable parameters still will not work, I can explain when time permits and will create a test to demonstrate. (The code will run, but is technically very wrong.)

So I can't fully create the test I want due to the lack of slots here, but it's something like this:
def test_callable():
    def death_prob(context, sim, uids):
        p = np.ones(len(uids))
        p[-1] = 0 # Last agent has zero prob
        return p

    d = ss.bernoulli(p = death_prob)

    # HERE IS WHAT I WANT:
    uids = [0,1,2] # UIDs are always unique
    slots = [100,205,205] # Slots corresponding to the above UIDs can be repeated
    #draw = d.urvs(uids) # <- Argument is UIDs, slots should be used under the hood

    # BUT TO SHOW THE TEST...
    draw = d.urvs(slots) # <- Argument should be UIDs, but instead put slots here for the purpose of this test

    assert draw[1] != draw[2], 'Draws with different parameter values should be different, even if slot is shared'

    return draw

This test fails. The user should be expected to operate on the UID space, saying which agents are relevant. Behind the scenes, slots are used to do CRN stuff. Slots are not unique, but parameters may differ between two agents that happen to have the same slot. Two agents with the same slot but different distribution parameters cannot receive the same result! This edge case is handled in main, but not here.

I see what you're saying ... I feel this is more a test of slots (and specifically repeated slots) rather than callable parameters though, right? If you replace slots with uids then the test passes, as it's similar to test_distributions.py:test_callable().
But you're right that without slots, CRN isn't fully implemented, so this is definitely an important feature to add back in!

make_dist calls lognorm_convert for lognorm_o assuming the mean and stdev parameters are scalars, not callables (which are handled later). Would be nice to allow callables here!

These changes streamline things a lot, I like it! Agree with @daniel-klein that would be great to see how it looks with slots in there though, since implementing that cleanly is a big part of the challenge

Distributions created as part of pars that eventually feed into convert_plugins get an incorrect trace, and therefore a different seed compared to if the same distribution was supplied to the module directly. I propose that we remove convert_plugins and the dict-based parameterization rather than changing how the trace is handed, it gets quite complicated to initialize otherwise.

Distributions created as part of pars that eventually feed into convert_plugins get an incorrect trace, and therefore a different seed compared to if the same distribution was supplied to the module directly. I propose that we remove convert_plugins and the dict-based parameterization rather than changing how the trace is handed, it gets quite complicated to initialize otherwise.

Agree, we should ensure this is fixed as part of fixing #284, #298, and #301

One thought (maybe just something we should be aware of) - does using the trace to assign RNG seeds make it hard to validate refactoring because changing the data structure could shift the seeds around and make it more common to be unable to validate that results are exactly equal?

One thought (maybe just something we should be aware of) - does using the trace to assign RNG seeds make it hard to validate refactoring because changing the data structure could shift the seeds around and make it more common to be unable to validate that results are exactly equal?

I don't think so, because it's based on dicts, so everything should be key-based rather than position-based. It's true that if you give the same modules different names, you'll get different results. But that's true of the implementation on main as well.

Remaining issues consolidated in https://github.com/starsimhub/starsim/issues/426


Implementation problem in ArchiveTimeStamp class

The ArchiveTimeStamp class implements the ArchiveTimeStamp structure described in RFC4998 (https://tools.ietf.org/html/rfc4998#section-4.1). According to the standard, the "reducedHashtree" field is optional, but the class constructors don't allow this field to be omitted:
public ArchiveTimeStamp(
    AlgorithmIdentifier digestAlgorithm,
    PartialHashtree[] reducedHashTree,
    ContentInfo timeStamp)
{
    this.digestAlgorithm = digestAlgorithm;
    this.reducedHashTree = new DERSequence(reducedHashTree);
    this.timeStamp = timeStamp;
}

public ArchiveTimeStamp(
    AlgorithmIdentifier digestAlgorithm,
    Attributes attributes,
    PartialHashtree[] reducedHashTree,
    ContentInfo timeStamp)
{
    this.digestAlgorithm = digestAlgorithm;
    this.attributes = attributes;
    this.reducedHashTree = new DERSequence(reducedHashTree);
    this.timeStamp = timeStamp;
}

Setting the reducedHashTree parameter throws a NullPointerException.
The other methods are correct. For example:
public PartialHashtree[] getReducedHashTree()
{
	if (reducedHashTree == null)
	{
	   return null;
	}

	PartialHashtree[] rv = new PartialHashtree[reducedHashTree.size()];

	for (int i = 0; i != rv.length; i++)
	{
		rv[i] = PartialHashtree.getInstance(reducedHashTree.getObjectAt(i));
	}

	return rv;
}

Muchas gracias.

A null check for reducedHashTree has now been added.


Add Lock Contact Sensors

Is it possible to add the contact sensor capability into the lock settings?

what is the use case? does your lock have an additional sensor?

my lock has a contact sensor on the door frame to notify me if the door is left open. i also use this to trigger automations such as “if door opens and it’s night, turn on the porch light”

Chevellebro1, I'll look into this, but your pull request would not have worked.  Please refrain from doing that going forward.  Now one question, is the contact sensor a separate device on your smartthings network, or is it a capability of the lock in smartthings?

Supported in 1.4.3.


BufferReader::Read does not block until enough bytes are available, contrary to the comment on Reader::Read

Reader::Read promises to block until there are count bytes available to read.
https://github.com/google/langsvr/blob/main/include/langsvr/reader.h#L44C43-L44C93
But BufferReader::Read doesn't do that.
https://github.com/google/langsvr/blob/main/src/buffer_reader.cc#L37

/// Read reads bytes from the stream, blocking until there are @p count bytes available, or the
/// end of the stream has been reached.

BufferReader reads from a buffer, so Read is correct to read up to the end of the buffer.

ah, of course.


Improving  header in costefficient page !!

I really like your app and want to work on your project ,please assign this issue to me under gssoc-ext 2024


crossorigin can't work on macOS sierra Safari 10.0

[ ] Issue does not already exist
[ ] Issue observed on https://plyr.io

Expected behaviour
Video should be work with crossorigin attribute
Actual behaviour
It can't work, all <source> and <track> not be append
Environment

Browser: Safari
Version: 10.0
Operating System: macOS
Version: Sierra

Steps to reproduce

Use macOS Sierra Safari 10.0 to open https://plyr.io

Relevant links

So this is a bug with Safari as opposed to Plyr?
Probably best to raise it here:
https://webkit.org/reporting-bugs/

@SamPotts look like it's Plyr's bug
Because I put this code in JSBin, it can work
<video poster="https://cdn.selz.com/plyr/1.5/View_From_A_Blue_Moon_Trailer-HD.jpg?v1" crossorigin controls>
  <source src="https://cdn.selz.com/plyr/1.5/View_From_A_Blue_Moon_Trailer-HD.mp4" type="video/mp4">

  <!-- Text track file -->
  <track kind="captions" label="English" srclang="en" src="https://cdn.selz.com/plyr/1.5/View_From_A_Blue_Moon_Trailer-HD.en.vtt" default="">
</video>

So are you seeing this on https://plyr.io?

My gf has the same problem too, she cant see the player on her 2016 mac book air , this problem is very annoying :/

If you don't need the crossorigin attribute - i.e. you're not loading webvtt files from another domain, you can remove the attribute.

Edge 14 has the same problem too.

Is this still happening? Seems very strange to me that only a couple of people would have these issues if it's a browser bug, rather than a browser addon, proxy issue, misconfigured server or something.
There was a related issue in iOS 10 around that time though: https://poodll.com/ios-10-and-html5-video-and-html5-audio-on-https-sites/
Anyway. I'll close this until some confirmation of the opposite (comment here).


"PowerShell on Target Machine" hangs

As the last step in my automatic build and deploy I call a PowerShell script on the remote machine to install my website and update my database using web Deploy and our own program (based on DbUp).
When executing this script the PowerShell task hangs newer finishing the build. Is there a specific way that the remote script should be written to signal it is done?

Did you get it working?

Hi
Sorry for the long delay. I figured out that the hang was caused by a command line program that was waiting for a user input (Console.ReadLIne specifically)


2-column comparison - Visual design

2-column comparison
On desktop, columns should display inline and be equal heights. On mobile, columns should stack and each be the height of their respective content.

Closed by #250


Add new patches to Kubernetes 1.24

Issue #, if available:
Description of changes:
⚠⚠⚠⚠⚠⚠⚠⚠
/hold until https://github.com/aws/eks-distro/pull/2333 merges
⚠⚠⚠⚠⚠⚠⚠⚠
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm

/approve

/retest

/lgtm


add @epelaaez solution for ex5

Name: Emilio Peláez
Score: 3
Chemical accuracy: ~2.339 mHa
Parameters: 32

 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.

Added solutions for exercises 1 and 2 in 6f235c2.

Added solution for exercise 3 in be627f4.

Sorry for the delay. Thank you very much for your submission!


Image to Image - Masking Does not work in SDXL 0.9

Hello SD Team, while using the API and playing with the Image To Image / Masking endpoint, i noticed and has been confirmed by a developer in Discord tha the masking is beeing partially ignored .
Can anyone look into this BUG and fix it? it's important that nothing behind the black pixels are modified by the diffusion.
Awaiting response.
Thank you.

Hi, can you share more details about how you generated these images? Which application did you use with what settings?


Add support for Secret Network v1.2.2

Add support for Secret Network v1.2.2 so you can run Secret Network nodes on Akash.
Refer to the docs for reference.
Please let me know if there are any important changes to be made, new to setting up Akash containers.

@fjij Did you test out the compose file locally? The make install command will most likely fail since the secret network uses a different installation format.

@fjij Did you test out the compose file locally? The make install command will most likely fail since the secret network uses a different installation format.

Thanks for pointing this out. I changed the make target, but this would obviously not work for the other images. Is there a way to add additional run steps into the base Dockerfile for specific images? Because I also noticed that the Secret Network node needs to run a script before make can be run.

I think adaptability has to be considered to support Secretnetwork as we tried to do with other chains. Just look at the history of the current Dockerfile, that will try to give you an idea about how we have tried to make it modular over the time.

I'll spend a little more time looking at the best way to do this and I'll make a new PR when I'm done

@fjij thanks for this - take a look at how juno and cheqd (starport) work - both have more custom installation branches in the Dockerfile that can be run before/after/instead of the make install step.


[feature] support unprepared statement errors. fix #11

If Cassandra returns an unprepared error, the DAO will re-prepare it and try the query again. This fixes #11, and handles the case where Cassandra would go down, and then up again while Kong is still running.
Other changes:

docs: base_dao and the factory have been given a better documentation.
refactor:

kong_query is now the prefered term for _queries elements.
params becomes args_keys and values args for consistance with the driver.
The _execute method was renamed to _execute_kong_query and broken into smaller chunks.
Batch statements now need to use the raw _execute method (the new one, not the one renamed to _execute_kong_query.
Cassandra errors now also have the error code.

Changes Unknown when pulling cdf94a0b9f80ece289ab95de1f3bfeffe5f239a7 on fix/reprepare-statements into ** on master**.


Support guest → host file transfer via clipboard

Tracking issue for this feature.
Milestones:

[x] find and initialize FUSE library
[x] convert server file lists to tangible form for requests
[ ] implement enough FUSE callbacks to see the files
[ ] provide file lists to the local clipboard
[ ] implement file data requests in FUSE callbacks
[ ] research file content caching

Issues in main FreeRDP repo:

#2424 clipboard and files
#4703 Guest to host file copy paste not working

Hi, is this still on-going?

@amosbird sadly, I've never found enough time/motivation to continue with this. There is some work in progress in one of the branches in this repo, but it stalled.


Fix missing wall segments for some extruded polygons

When generating vertices and indices for extruded polygons, we only generate indices for vertices that aren't too close together. However, the array that we were allocating for these indices was sometimes too small, which was most apparent with rhumbline polygons like geojson. This PR allocates a larger array up-front and slices it down to size after all indices have been generated.
I'm not 100% sure how the original index count was computed, @hpinkos @lilleyse is any of that familiar?

Thanks for the pull request @likangning93!

:heavy_check_mark: Signed CLA found.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.
[ ] Works (or fails gracefully) in IE11.

@likangning93 do you have a sandcastle example to reproduce this?  Your fix doesn't look right to me.

Sandcastle on CesiumJS
Sandcastle on branch

I can confirm this fixes the issue I was seeing which came up on the forum here.

I'll take a closer look at this soon

Thanks again for your contribution @likangning93!
No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?
I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

Replaced  by #8177

Congratulations on closing the issue! I found these Cesium forum links in the comments above:
https://groups.google.com/d/msg/cesium-dev/kaAIg-6-XdM/-ob0p_wlBgAJ
If this issue affects any of these threads, please post a comment like the following:

The issue at https://github.com/AnalyticalGraphicsInc/cesium/pull/8035 has just been closed and may resolve your issue. Look for the change in the next stable release of Cesium or get it now in the master branch on GitHub https://github.com/AnalyticalGraphicsInc/cesium.


Support multiple instance types in ASG with Launch Templates

I have issues
I'm submitting a feature request
What is the current behaviour?
When using Launch Templates via the worker_groups_launch_template, you can only have 2 instances types per worker group. It's set here and has a fixed limit of 2 different types, the minimum.
What's the expected behaviour?
You can have up to 20 different instances types in the ASG. Having a broad Mixed Instances Policy is very important when using spot instances as you want a diverse selection of types so that when a spot prices goes to or above the on-demand price, another type can be used.
Are you able to fix this problem and submit a PR? Link here if you have already.
I don't think so.
We can simply add more override sections to aws_autoscaling_group.workers_launch_template.mixed_instances_policy.launch_template, e.g.:
    launch_template {
      override {
        instance_type = "${lookup(var.worker_groups_launch_template[count.index], "instance_type", local.workers_group_launch_template_defaults["instance_type"])}"
      }

      override {
        instance_type = "${lookup(var.worker_groups_launch_template[count.index], "override_instance_type", local.workers_group_launch_template_defaults["override_instance_type"])}"
      }

      override {
        instance_type = "${lookup(var.worker_groups_launch_template[count.index], "override_instance_type_1", "")}"
      }

      override {
        instance_type = "${lookup(var.worker_groups_launch_template[count.index], "override_instance_type_2", "")}"
      }

But we need default values for each override in case they are not used. If we use "" like above, terraform applies the change on every terraform apply:
  ~ module.cluster1.aws_autoscaling_group.workers_launch_template[0]
      mixed_instances_policy.0.launch_template.0.override.#: "2" => "4"

  ~ module.cluster1.aws_autoscaling_group.workers_launch_template[1]
      mixed_instances_policy.0.launch_template.0.override.#: "2" => "4"

I'm not sure how to fix this. Even if we could define the LT outside of the module, it wouldn't help because the instances types are defined on the ASG.

I think it's a provider issue: https://github.com/terraform-providers/terraform-provider-aws/issues/8475

Fixed in https://github.com/terraform-aws-modules/terraform-aws-eks/pull/371

@max-rocket-internet
I tried to use this module (4.0.2) with worker_group_launch_template_mixed follow this docs but terraform return error
Error: module "eks": "worker_group_launch_template_mixed" is not a valid argument.
Can you help me understand what is right way to use eks workers module with mixed_instances_policy?


Confirm terminating roles logic from spec

To make sure we get logic correct for terminating delegations I have created this simple example

Constraints
term = terminating delegation
non-term = non terminating delegation
Priority: The roles in each level are ordered from left to right in the order they would appear under [delegations][roles]
All roles have paths = [‘assets/*’] (just to provide matches for every role only focus on terminating logic now)
Target being searched for = 'assets/always-match.txt’
Expected outcome
Expected role evaluation:
Targets -> A > B > C > D

Copy over comment from @phenaproxima from original google doc

My reading of the Python code at https://github.com/trishankatdatadog/tuf-on-a-plane/blob/ee8049a63ea439d72509f6e876eef08fb50c6ab1/src/tuf_on_a_plane/repository.py#L319-L403 leads me to think that it will be Targets > A > B > C > D > F.

Closing this issue in favor of https://github.com/theupdateframework/specification/issues/168 on Spec itself


Chrome 62 fix: Parse 'raw' requestBody if 'formData' isn't available.

Per #6, Chrome 62 isn't providing a pre-parsed formData object to retrieve the SAMLResponse. Parse the raw ArrayBuffer data to retrieve the SAMLResponse payload and other formData params.

👍

Created https://bugs.chromium.org/p/chromium/issues/detail?id=777898 so that the Chromium team is aware of this change in behavior (which seems buggy to me, not intentional).

👍

@bdruth Thanks for reporting! My apologies for my late reply. I'll have a look at the code, test, merge and update the Chrome plugin asap. Will be somewhere this week.

Code looks good. Thanks again for contributing and thanks for using the extension!


[fix] Cms::Column::Value::Free : generate_public_files

Coverage decreased (-7.7%) to 68.877% when pulling 2a66f3915c29047bb96facf26e64b71c451ed83b on fix/generate-file into f6b9f8ff14904b21d593b9301ca7020e56f09afc on master.

Coverage decreased (-7.7%) to 68.877% when pulling 2a66f3915c29047bb96facf26e64b71c451ed83b on fix/generate-file into f6b9f8ff14904b21d593b9301ca7020e56f09afc on master.


WebSocket Connection Not Sending Variables on Connect

I am working on a cloud api for scratch (https://github.com/ErrorGamer2000/scratch3-api). When I connect to a project on scratch, the most recent values of all of the cloud variables is sent. When connecting to turbowarp, this does not happen. After some experimentation, I found what is happening. When I load the project on tw for the first time, the current values of the cloud variables are loaded from the project's history (to be expected, as this happens when you download a project and re-upload it). The issue, though, is that tw does not count this update as a set action, so the server does not see that they have ever been set. So, after I changed a cloud variable in the project I now receive a set message as soon as I connect. This is a small bug, but would you consider making the initial load of a project act as if a user has set the cloud variables when they are first loaded?

The problem is that when the first connection to a project is received and an internal room is created but the server is never told what variables exist until something changes

This won't be fixed


Fix tests that occasionally hang

Summary
Race conditions are causing some tests to occasionally hang. Adding in some sleeps to protect against this. I believe these issues are unique to the test environment, and should never happen in a normal event loop. But we should keep an eye on this as we are integrating into SDKs.

I haven't been able to trace it down precisely


🚚 Moving Gradle wrapper for types to v1 directory

📝 Description

Originally commented in https://github.com/kotools/types/issues/607#issuecomment-2016975202.

For reducing coupling between Gradle projects, we would like to generate a new Gradle wrapper for the types project and remove the Gradle root project.

✅ Checklist

[x] Generate a new wrapper for the types project by running the wrapper task.
[x] Update the integration workflow for the types project by using the new wrapper.
[x] Update the run configurations for IntelliJ IDEA by using the new wrapper.
[ ] Remove the Gradle settings and the wrapper from the root project.
[ ] Close this issue as completed and update tracking ones if present.

The IDE doesn't load the project when we remove the wrapper from the root directory.
Because this may be inconvenient for contributors, we will revert changes of this issue and close it as stale.


import promise-mysql make a error

Hello, In pages, I want get datas from mysql, so import node module 'promise-mysql'. but make an error, like this:
Failed to compile
../node_modules/mysql/lib/protocol/sequences/Query.js
Module not found: Can't resolve 'fs' in 'd:\jk-work\dev-repos\dl-repos\dl-bs-ssr\node_modules\mysql\lib\protocol\sequences'
This error occurred during the build time and cannot be dismissed.

so , I want to know whether Next,js Pages can use node module to connect mysql in method getInitialProps.
Thank you!

You'll have to create a separate API endpoint as mysql queries can't be ran inside of the browser and getInitialProps is ran in the browser.
Soon you'll be able to build APIs in Next.js: https://github.com/zeit/next.js/issues/7297

thanks so mouch. I will try it later！


Pre-trained embeddings support: Set the models schema from the dataloader output schema

closed by #690


Indexing of Websites installed as apps via Edge

Summary of the new feature/enhancement
In the new (chromium based) version of Edge, you can add sites as Apps, so that they Appeyr in Start Menu and they Launch without a tablist and a search box. Sadly these apps do not get indexed by Power Toys Run. It can only find them, if they run in the background.

Tracked by #3459 and fix is committed


2: Create page in sub-directory/folder [Feature request]

Short description of the problem:
When you have an app with a lot of pages, its easier to manage them in groups. e.g.
Concept A

page 1
page 2
page 3

Concept B

page 4
page 5
page 6

At the moment the ionic g page command creates pages (and their scss / js) at the top level "pages".
However I would like to do
ionic g page conceptA/pagename
and it to create pagename in pages > conceptA > pagename
Otherwise, I have to create a page, and then move it subsequently. This is a bit fiddly as quite a bit is auto-generated, so its not just a drag drop.
What behavior are you expecting?
Works as expected. Just this is an idea for an improvement
Which Ionic Version?
2
Run ionic info from terminal/cmd prompt: (paste output below)
Cordova CLI: 5.4.1
Ionic Version: 2.0.0-beta.3
Ionic CLI Version: 2.0.0-beta.19
Ionic App Lib Version: 2.0.0-beta.9
ios-deploy version: Not installed
ios-sim version: 5.0.4
OS: Mac OS X El Capitan
Node Version: v5.3.0
Xcode version: Xcode 7.3 Build version 7D175

Hello! Thanks for opening an issue with us! As this feature request is more related to the ionic cli i will be moving this feature over there. Thanks again!

This issue was moved to driftyco/ionic-cli#868


Difference between waymo 1.0 and 1.2

As far as I known, there is not "num_gts_in_box" in waymo1.0. If I calculate it and put it into the info pkl and train the data of waymo 1.0. Can I get the same result as I train waymo 1.2?

Waymo 1.2 also add or remove some frames in some sequence. The number of training frames and validation frames is also different between v1.0 and v1.2.

OK, I get it. Thanks very much.

Waymo 1.2 also add or remove some frames in some sequence. The number of training frames and validation frames is also different between v1.0 and v1.2.

OK, I get it. Thanks very much.


"Negrași", "Bârlogu" are listed in the drop-down menu

Description:
When user writes "Negrași" and "Bârlogu" the locations appear in the drop-down menu next to the comune and the county witch is a part according to the law
Precondition:
The website is up an running.

Step 1
Write in the search bar "Negrași"

Expected results
The location appears in the drop-down menu as "Negrași, Argeș (Negrași)"

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Bârlogu"

Expected results
The location appears in the drop-down menu as "Bârlogu, Argeș (Negrași)"


Files deleted from list are still accessible

Something seems not to work properly. After I delete content form Harpooner list it is cached somewhere and not deleted.
Steps to reproduce:

Add files to list
Open Harpooner Menu file and delete it's content
Save the file :w
Close the tab
Try to access any file from the list for example with HarpoonerOpenFile0
File is opened despite Harpooner list being empty
Open Harpooner Menu file again
It has all it's content despite being deleted in step 2

Hey, I didn't face that problem when using IntelliJ Ultimate. I guess the issue stems from API variations across different products. Could you specify which one you're using?

IntelijJ Ultimate 2023.3.3. I'm not sure whether this is the root cause, but having two files Harpooner Menu and Harpooner.xml content between these two is not syncrhonized for some reason. Files removed from Harpooner Menu are visible under Harpooner.xml.

Thanks, I optimized the behavior. Let me know if you encounter further issues


interest in passing functions, not strings for commands?

Having found myself frustrated with argparse, I started looking into Knack for argument parsing.
One of the choices that I'm wondering about in Knack is the choice to specify the command groups with strings, which when used are passed to import_module & a python version independent inspect to get the implementation of the command.
This makes it difficult to use static analysis to verify code using this module, since it's roughly akin to eval.
Additionally, it makes it difficult to programmatically generate a CLI from classes, as it requires going from object instance back to strings for the module and function paths.
Is there any interest in supporting functions rather than strings?
Something akin to:
with CommandGroup(self, "hello") as g:
    g.command("world", hello_world_handler)

@jiasli please take a look


Heading with subheading pattern

https://www.tpgi.com/subheadings-subtitles-alternative-titles-and-taglines-in-html/
https://adrianroselli.com/2020/08/be-wary-of-doc-subtitle.html
We (almost) always do not want to use back to back h1 and h2 for heading and sub heading/subtitle, particularly for things such as cards, feature panels, and hero areas where the additional section the h2 creates is not useful.

Closed with #84


c8d/resolver: Use hosts from daemon configuration

This makes the containerd pull/push respect the insecureRegistries configuration.
Signed-off-by: Paweł Gronowski<EMAIL_ADDRESS>
- What I did
- How I did it
- How to verify it
- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

I added the fallback to the HTTP and changed the (daemon *Daemon) RegistryHosts to correctly parse scheme-less URL with ports.
It's a bit hacky, but it works.

I tried to validate this PR:
ran a registry container
started development container with --link registry
added "insecure-registries" : ["registry:5000"] to my daemon.json
$ docker pull registry:5000/alpine
Using default tag: latest
Error response from daemon: failed to resolve reference "registry:5000/alpine:latest": failed to do request: Head "https://registry:5000/v2/alpine/manifests/latest": http: server gave HTTP response to HTTPS client

tested, but doesn't work for me.
My test setup:

create a registry container
run moby dev container with `--link registry``
register registry:5000 as insecureRegistry in daemon.json

$ docker pull registry:5000/alpine
Using default tag: latest
Error response from daemon: failed to resolve reference "registry:5000/alpine:latest": failed to do request: Head "https://registry:5000/v2/alpine/manifests/latest": http: server gave HTTP response to HTTPS client


Added KYC notation for Opennode

Added KYC notation in the ** Section at bottom.

This has been updated


Add Pathlib

This change add pathlib and changes many os method to support path like objects.
Many of the tests fail because Rust does not support Cstring with nul. Not sure how to bypass that. Would appreciate suggestion on how to solve this.

I did implement TryFromObject for CString recently, but you could also just have a nulcheck function that makes sure a string doesn't have a nul char in it, and returns a ValueError if it does, then just call nulcheck(s, vm)?;. (Or maybe a vm method?)

I did implement TryFromObject for CString recently, but you could also just have a nulcheck function that makes sure a string doesn't have a nul char in it, and returns a ValueError if it does, then just call nulcheck(s, vm)?;. (Or maybe a vm method?)

The problem is that CPython support nul char while rust throws exception.
os.stat("/dev/null\x00")

Should not throw an error.

Ah, I see. Hmm, that is tricky.

It seems only trailing \x00 exists in tests. Trimming them before making PyPathLike would work?

This will actually give the opposite result. As /dev/null should be different then /dev/null\x00

This is not specifically related to this PR, but while reading, i found i have no idea why PyPathLike is not PyPath but Like

Thats how they are called in CPython.

I try to test this problem a bit, but I can't make null-included path working. Could you describe your working environment?
macOS, python 3.8.1, APFS
>>> open('abc\x00', 'w')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ValueError: embedded null byte
>>> import os
>>> os.stat('/dev/null\x00')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ValueError: embedded null byte
>>>

Linux,  Python 3.5.2, ext4
>>> open('abc\x00', 'w')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ValueError: embedded null byte
>>> import os
>>> os.stat('/dev/null\x00')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ValueError: embedded null byte

If you will look at the tests. For example https://github.com/RustPython/RustPython/pull/1840/files#diff-6bc3eb7fdef7557c945c66fb0b60a134R1921

@coolreader18 @youknowone can this be pushed and we will solve the nul later?

It seems Vec<u8> is the reasonble internal data of path types. And they may be used to create ffistring in future when they are actually used. Is it possible to fix now? or do you want to merge this for right now as it is?

It seems Vec<u8> is the reasonble internal data of path types. And they may be used to create ffistring in future when they are actually used. Is it possible to fix now? or do you want to merge this for right now as it is?

I am not sure I understand... Would you like to change the OsString to Vec<u8>?

I guess so. I didn't check cpython implementation well yet, but it seems it is more close to Vec<u8> than OsString. Or enum { Bytes(Vec<u8>), Str(String) } if there are difference by str/bytes.

@palaviv I think fixing it later would be fine, it's not too critical to pathlib, and I think it would rarely happen in practice.

I have merged this and we can do the proposed changes in the future

I checked about PyPathLike impl but it revealed enum is not fit for the use case.


emem: 0.2.20 -> 0.2.22

Motivation for this change
Things done

[✔] Tested using sandboxing
(nix.useChroot on NixOS,
or option build-use-chroot in nix.conf
on non-NixOS)
Built on platform(s)

[✔] NixOS
[ ] OS X
[✔] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[✔] Tested execution of all binary files (usually in ./result/bin/)
[✔] Fits CONTRIBUTING.md.

Thanks! (∩_∩)


MAINT: drop Python 3.8 support

A first step following https://github.com/shapely/shapely/issues/2119 (after also dropping GEOS 3.8 in https://github.com/shapely/shapely/pull/2124), dropping at least Python 3.8.

I think setting NumPy >= 1.20 would make more sense to pair with Python >=3.9. This is because NumPy 1.19.0-1.19.2 support Python 3.6-3.8.

I was just following what we were testing (numpy 1.19.5 with python 3.9). Can also make it explicitly >=1.19.5 (or 1.19.3), but don't mind using 1.20 either (and then update the tests to use that version)

The oldest-supported-numpy is complex, but is most commonly 1.19.3 for for Python 3.9.
I think 1.20 is a cleaner version increment, since it keeps a simpler major-minor increment (rather than a mid-way major-minor-micro increment). See also 1.20.0 release notes if there is anything compelling. But I'm also not too bothered by this detail, as >= 1.19.3 is fine too.
The windows-2019 test is failing due to numpy==1.16.2, which has no cp39 binary wheels.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall first build on drop-38 at 88.122%

Totals

Change from base Build<PHONE_NUMBER>8:
88.1%

Covered Lines:
2619

Relevant Lines:
2972

💛  - Coveralls


Managed BigQueryIO

R: @chamikaramj

R: @chamikaramj
R: @robertwb

Based on an offline discussion, we should do the forking to select the correct write transforms at a single 'BigQueryWriteSchemaWriteTransformProvider' instead of expecting the caller to perform the expansion for the exact implementation. The pipeline options needed for this (for example, dataflowServiceOptions=streaming_mode_at_least_once) should be provided to this call (accessible via input.getPipeline().getOptions()). Also, we can use input.get(INPUT_ROWS_TAG).isBounded() to determine whether this is an unbounded call or not.
This should be possible since all BQ write schema-transforms here share the same configuration.
May be we should keep read general in a similar manner to support future expansions / read methods.
cc: @robertwb

I suggest we do the forking on the SDK side only if it's actually necessary. Keep in mind this approach will require adding the Dataflow runner as a dependency (to access dataflowServiceOptions) in Managed API (a runner-independent module).
This PR used to include such forking but that logic was reverted due to this seemingly unavoidable dependency:
74bc178c79aaed8329b50ed5fa0b50f65bccc3e8

LGTM. Feel free to merge once comments are addressed.

P.S. added one last commit to expose to the Python SDK


fix(doc): css-attribute without interpolation explained

explaines https://github.com/aurelia/templating-binding/issues/100

Maybe it's worth noting that css was introduced to support IE11, as IE11 doesn't like style attribute with interpolation syntax.

Same thing about <require /> element, anything that seems to off standard keyword should be clarified with documentation.

Same thing about <require />  instead of <import /> element, anything that seems to off standard keyword should be clarified with documentation.


A temp solution to enable GluonCV INT8 flow

Description
A simple method to launch int8 inference with gluon is that loading back a quantized JSON file and parameter as a SymbolBlock. However, SymbolBlock function is based on CachedOP which needs both forward and backward computing graph. So, some inference_only operators like quantized operators and subgraph operators cannot be loaded back. Below are the error logs:
Export to JSON...
Load back from JSON with SymbolBlock
[10:01:41] src/operator/subgraph/mkldnn/mkldnn_conv_property.cc:148: Start to execute MKLDNN Convolution optimization pass.
Traceback (most recent call last):
  File "demo_ssd.py", line 58, in <module>
    ids, scores, bboxes = [xx[0].asnumpy() for xx in net(x)]
  File "/home/chenxiny/mxnet-official/python/mxnet/gluon/block.py", line 540, in __call__
    out = self.forward(*args)
  File "/home/chenxiny/mxnet-official/python/mxnet/gluon/block.py", line 1078, in forward
    return self._call_cached_op(x, *args)
  File "/home/chenxiny/mxnet-official/python/mxnet/gluon/block.py", line 797, in _call_cached_op
    self._build_cache(*args)
  File "/home/chenxiny/mxnet-official/python/mxnet/gluon/block.py", line 785, in _build_cache
    self._cached_op = ndarray.CachedOp(out, flags)
  File "/home/chenxiny/mxnet-official/python/mxnet/_ctypes/ndarray.py", line 116, in __init__
    ctypes.byref(self.handle)))
  File "/home/chenxiny/mxnet-official/python/mxnet/base.py", line 252, in check_call
    raise MXNetError(py_str(_LIB.MXGetLastError()))
mxnet.base.MXNetError: [10:01:42] src/pass/gradient.cc:192: Operator _sg_mkldnn_conv is non-differentiable because it didn't register FGradient attribute.

A temporary solution implemented in #14275 is that registering fake grad to subgraph and quantized operators. This will be reverted after the improvement of CachedOP is done.
Environment info (Required)
----------Python Info----------
Version      : 3.6.3
Compiler     : GCC 7.2.0
Build        : ('default', 'Oct 13 2017 12:02:49')
Arch         : ('64bit', '')
------------Pip Info-----------
Version      : 9.0.1
Directory    : /home/chenxiny/anaconda3/lib/python3.6/site-packages/pip
----------MXNet Info-----------
Version      : 1.5.0
Directory    : /home/chenxiny/mxnet-xinyu/python/mxnet
Hashtag not found. Not installed from pre-built package.
----------System Info----------
Platform     : Linux-3.10.0-957.el7.x86_64-x86_64-with-centos-7.6.1810-Core
system       : Linux
node         :
release      : 3.10.0-957.el7.x86_64
version      : #1 SMP Thu Nov 8 23:39:32 UTC 2018
----------Hardware Info----------
machine      : x86_64
processor    : x86_64
Architecture:          x86_64
CPU op-mode(s):        32-bit, 64-bit
Byte Order:            Little Endian
CPU(s):                80
On-line CPU(s) list:   0-79
Thread(s) per core:    2
Core(s) per socket:    20
Socket(s):             2
NUMA node(s):          2
Vendor ID:             GenuineIntel
CPU family:            6
Model:                 85
Model name:            Intel(R) Xeon(R) Gold 6148 CPU @ 2.40GHz
Stepping:              4
CPU MHz:               1000.048
CPU max MHz:           3700.0000
CPU min MHz:           1000.0000
BogoMIPS:              4800.00
Virtualization:        VT-x
L1d cache:             32K
L1i cache:             32K
L2 cache:              1024K
L3 cache:              28160K
NUMA node0 CPU(s):     0-19,40-59
NUMA node1 CPU(s):     20-39,60-79
Flags:                 fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush dts acpi mmx fxsr sse sse2 ss ht tm pbe syscall nx pdpe1gb rdtscp lm constant_tsc art arch_perfmon pebs bts rep_good nopl xtopology nonstop_tsc aperfmperf eagerfpu pni pclmulqdq dtes64 monitor ds_cpl vmx smx est tm2 ssse3 sdbg fma cx16 xtpr pdcm pcid dca sse4_1 sse4_2 x2apic movbe popcnt tsc_deadline_timer aes xsave avx f16c rdrand lahf_lm abm 3dnowprefetch epb cat_l3 cdp_l3 intel_ppin intel_pt ssbd mba ibrs ibpb stibp tpr_shadow vnmi flexpriority ept vpid fsgsbase tsc_adjust bmi1 hle avx2 smep bmi2 erms invpcid rtm cqm mpx rdt_a avx512f avx512dq rdseed adx smap clflushopt clwb avx512cd avx512bw avx512vl xsaveopt xsavec xgetbv1 cqm_llc cqm_occup_llc cqm_mbm_total cqm_mbm_local dtherm ida arat pln pts hwp hwp_act_window hwp_epp hwp_pkg_req pku ospke spec_ctrl intel_stibp flush_l1d
----------Network Test----------
Setting timeout: 10
Timing for MXNet: https://github.com/apache/incubator-mxnet, DNS: 0.0055 sec, LOAD: 2.5571 sec.
Timing for Gluon Tutorial(en): http://gluon.mxnet.io, DNS: 0.0081 sec, LOAD: 2.4796 sec.
Timing for Gluon Tutorial(cn): https://zh.gluon.ai, DNS: 0.0087 sec, LOAD: 2.1053 sec.
Timing for FashionMNIST: https://apache-mxnet.s3-accelerate.dualstack.amazonaws.com/gluon/dataset/fashion-mnist/train-labels-idx1-ubyte.gz, DNS: 0.0121 sec, LOAD: 2.1985 sec.
Timing for PYPI: https://pypi.python.org/pypi/pip, DNS: 0.0079 sec, LOAD: 6.4577 sec.
Timing for Conda: https://repo.continuum.io/pkgs/free/, DNS: 0.0082 sec, LOAD: 1.5200 sec.

@mxnet-label-bot Add [Quantization]


PRs to fix patches in diego-cell upstream project

./diego-cell/garden/jobs/patch_post-start.sh
./diego-cell/cflinuxfs3-rootfs-setup/jobs/patch_pre-start.sh
./diego-cell/rep/jobs/patch_pre-start.sh
./diego-cell/rep/jobs/patch_rep_json.sh

relates with #12

Relevant upstream releases:

Job
Release

garden
https://github.com/cloudfoundry/garden-runc-release/tree/develop/jobs/garden

cflinuxfs3-roots-setup
https://github.com/cloudfoundry/cflinuxfs3-release/tree/master/jobs/cflinuxfs3-rootfs-setup

rep
https://github.com/cloudfoundry/diego-release/tree/develop/jobs/rep

Versions currently used by kubecf (cf-deployment 12.18):

Release
Version
Hash

garden-runc-release
1.19.9
1c678e1c7a3506c0e408860571560081c15e3c6d

cflinuxfs3-release
0.150.0
6421766b2120009c6d422c4b1ed5a12c4290eda4

diego-release
2.41.0
8a183683f26ba982eda1dbeba7f55edc9a1059b0

Commits to branch the PRs from (Using the version tags from previous comment):

Release
Commit
Reasons for patching

garden-runc-release
https://github.com/cloudfoundry/garden-runc-release/tree/e1d3a70ebfc7151cd0f38433fe0ab5ce86f2103a
post-start: curls unix socket support to variable, switched to netcat as more stable

cflinuxfs3-release
https://github.com/cloudfoundry/cflinuxfs3-release/tree/cc3507bdba35c720a8610d8f024ef334be9c9b74
pre-start: Place the rootfs into the ephemeral data directory

diego-release
https://github.com/cloudfoundry/diego-release/tree/e75c353863b51c2428c057f26cbb7178e5930373
pre-start: Place the rootfs into the ephemeral data directory

rep-json: Packages not shared between containers

I am not sure if the rep-json patch is something we can upstream.
It looks to be container/kubecf specific.
Also, about the rootfs placement, we should possibly expand on why we moved it into ephemeral storage, not just that we did.
@viovanov @f0rmiga @mook-as @jandubois  Opinions ?

@fargozhu This ticket about the diego-cell patches is not listing the patches
bosh/releases/pre_render_scripts/diego-cell/garden/jobs/patch_pre-start.sh
bosh/releases/pre_render_scripts/diego-cell/rep/jobs/patch_rep_ctl.sh

Was this intentional ? The pre-start patch does permission changes/fixes. The rep_ctl patch removes an ulimit statement for 100K fds. not sure what the limit is without it. Are both container/kubecf specific to exclude from upstreaming ?

Open question:

Should I do a PR directly on the target repos ?
PR from personal fork ?
PR from SUSE fork ?

Resolutions (Talked with @f0rmiga):

Personal forks.
Make changes, push personal, final discussion about upstreaming
shared-packages vs packages - definitely not right now.
Look into new properties for most changes to make things configurable by us while allowing CF to retain the defaults for their use under BOSH.

garden-runc = commit https://github.com/andreas-kupries/garden-runc-release/commit/34807fd91b2a6c1f1cd0f3b57e9b3c58aa50f9bb

cflinuxfs3 = commit https://github.com/andreas-kupries/cflinuxfs3-release/commit/f61fc8368ca6c767044e9756cb14e0ee565c36d1

diego-release = commit https://github.com/andreas-kupries/diego-release/commit/d298466c6bd64f8ae49b1d6a1e50d78546dbc2e9
Original patch: https://github.com/cloudfoundry-incubator/kubecf/blob/master/bosh/releases/pre_render_scripts/diego-cell/rep/jobs/patch_rep_ctl.sh

@viovanov @f0rmiga @mook-as @jandubois @fargozhu @prabalsharma @jimmykarily @thardeck @svollath @viccuad Ping. Below the final table listing our local patches in the diego-cell area and the commits proposed to become the equivalent upstream PRs.

Two changes (garden) as-is.
Two changes make the place we patch configurable via job properties.
Three patches excluded from upstreaming as SUSE specific (1), and not fully baked (2).

Please review the changes themselves, and the commit messages. The latter are intended to become the PR description (including the back link to this ticket). Please add your comments to this ticket.
All original patches are in kubecf, under the directory bosh/releases/pre_render_scripts/diego-cell. Links are to their github locations.

Patch
Notes
Upstream Change, Proposed

cflinuxfs3-rootfs-setup/jobs/patch_pre-start.sh
Make destination configurable.
https://github.com/andreas-kupries/cflinuxfs3-release/commit/f61fc8368ca6c767044e9756cb14e0ee565c36d1

garden/jobs/patch_post-start.sh
curl/netcat for unix sockets. Upstream as-is.
https://github.com/andreas-kupries/garden-runc-release/commit/34807fd91b2a6c1f1cd0f3b57e9b3c58aa50f9bb

garden/jobs/patch_pre-start.sh
Clear grootfs playground, make accessible. Upstream as-is.
https://github.com/andreas-kupries/garden-runc-release/commit/0c7b7d96027c9a0f735e02491d116460d2ce40f8

rep/jobs/patch_rep_ctl.sh
Make application of fd ulimits configurable, analogous to set_kernel_parameters
https://github.com/andreas-kupries/diego-release/commit/d298466c6bd64f8ae49b1d6a1e50d78546dbc2e9

rep/jobs/patch_pre-start.sh
Hack solution to co-location issues in containerized environs.
Do not upstream

rep/jobs/patch_rep_json.sh
Ditto.
Do not upstream

sle15-rootfs-setup/jobs/patch_pre-start.sh
SLE specific.
Do not upstream

All the ulimit patches were found superfluous. PR #1040 removed them. The PR cloudfoundry/diego-release#512 has been retracted and closed.
The remaining patches for this ticket will be re-checked regarding necessity, and different solutions.

The PR https://github.com/cloudfoundry/garden-runc-release/pull/158
was essentially accepted and merged by upstream, but not directly. See https://github.com/cloudfoundry/garden-runc-release/commit/7098cdca7f3e665defe7228f5e01749660846637
Closed the PR.

The PR https://github.com/cloudfoundry/garden-runc-release/pull/159 is still pending upstream.

The PR https://github.com/cloudfoundry/cflinuxfs3-release/pull/12 is still pending upstream, without any comments, or actions.

Based on https://github.com/cloudfoundry-incubator/kubecf/issues/520#issuecomment-648407686
Since this is filed upstream now, will close. Will track the upstream PR so hopefully, when it's done, we can somehow track it to what we need to consume next.


Default binding  key causes a problem

Whenever I open vim, it will have pre-entered in command line as ;95c. I took 2 hours to figure it out that it's because of default binding <ESC> key: let g:multi_cursor_quit_key='<ESC>'
I could remap this key to another key. However, I'm just curious that I don't see other people who are using this plugin having this issue. Do I miss something? Any help would be appreciated.

@lvarayut I just want to ask you: does this plugin break functional keys? (such as F1, F2 and so on)?

@delminskii No, I doesn't. The only key that has the problem is <ESC>. So, I'm trying to remap it as following:
let g:multi_cursor_use_default_mapping=0
let g:multi_cursor_next_key='\<C-n>'
let g:multi_cursor_skip_key='\<C-k>'
let g:multi_cursor_skip_key='\<C-x>'
let g:multi_cursor_quit_key='\<C-c>' "Remap ESC key to avoid ;95c problem, need the backslash to make it works

But it introduces me the new problem:
Error detected while processing function multiple_cursors#new..<SNR>220_wait_for_user_input:
line   12:
E121: Undefined variable: s:special_keys
E116: Invalid arguments for function index(s:special_keys, s:char) != -1
E15: Invalid expression: index(s:special_keys, s:char) != -1
Press ENTER or type command to continue

I currently can't use this plugin. :(

Having similar problems! Remapping  introduces some visual problems for me.
https://github.com/terryma/vim-multiple-cursors/issues/169

Closing due to inactivity


Improve the looks of the loader page

svg loading symbol(make sure it is either attribution or free use)
use translation for loading text

maybe?


[DOC] Document using Refine with existing projects

Documentation issue
For some users, it's not clear if Refine can be used with existing applications. Although it should be straight forward, we aren't clearly explaining concepts and clarifying that it's possible.
Describe the thing to improve
We need to document using Refine with existing projects.
It may or may not be required to explain it separately for Vite, NextJS or Remix, if we can explain the concept.
Describe the solution (optional)

Add a documentation page about Adding Refine to existing projects
Add examples for Vite, NextJS, Remix. Where refine is mounted only in the /refine route.

References
We have some discussions in the past, which can be used as a reference: https://github.com/refinedev/refine/discussions/3343#discussioncomment-4592073 At the time of this answer, Refine was on v3, but the same concept can be applied to Refine v4 regardless.
https://discord.com/channels/837692625737613362/1186730677694124072

Assign me the task, I will do it as mentioned in the comment #3343 by @aliemir . I will try my best to make that page for the documentation.

Hello @Conqxeror we'll handle this internally, we discussed and there are some important points needs to be handled.

Hello @Conqxeror we'll handle this internally, we discussed and there are some important points needs to be handled.

Ok @BatuhanW 👍🏻 If y'all need any help on this task then do tell me. I would love to work on this task and to grow and learn with you all.


add notes to python code

I would propose to update the description of python code with few notes to make it more legible.
And also give a hint to the student to know how to set up the python environment, cleanly.

thanks!


OperationType EOFCREATE

Add OperationType EOFCREATE
https://github.com/otterscan/otterscan/blob/c7b81575985ed57c723b84b2b8c5ff6c6e8c5131/src/types.ts#L91-L96

for reference: https://eips.ethereum.org/EIPS/eip-7620

Sure.

plans on doing a patch release @wmitsuda ?

plans on doing a patch release @wmitsuda ?

I was not planning to, but I may do it if you need it. Just curious, is it something reth-related? I thought Otterscan API was not working on reth yet.

also note that if you need a docker build and don't mind using develop builds, every commit automatically pushes a develop image tag on dockerhub.

yepp it's for alloy, main dep of reth https://github.com/alloy-rs/alloy/issues/1288

pushed tag 2.6.1 cherry picking this PR: https://github.com/otterscan/otterscan/releases/tag/v2.6.1


Improve start up performance

ALE starts pretty quickly, but it could be better. We should measure start up performance and make a few tweaks to improve start up times.
Ideas

[x] Move code for printing :ALEInfo to another file, loaded only when the command is first issued.
[x] Try replacing ale#Set calls in the main plugin file with let commands using get(g:, ...).

The :ALEInfo idea I mentioned was already how it was, but I made sure not to mess that up when adding the new LSP error debugging.

This should be good enough now.


Minimal change to test CI

Signed-off-by: David Festal<EMAIL_ADDRESS>Hey, I just made a Pull Request!
Minimal change to test CI failures on the most recent code.
:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[ ] All your commits have a Signed-off-by line in the message. (more info)

Closed since this PR confirmed the global CI bug.


Websocket error abortHandshake

I want to convert all my nodejs projects from using the api to using this api wrapper. But I cannot get past the first connection with this package.
I keep getting errors with the demo using yarn and npm:
yarn run v1.22.4
$ node node-demo.js
events.js:288
throw er; // Unhandled 'error' event
^
Error: Unexpected server response: 401
at ClientRequest. (C:\Users\binary-live-api-master\binary-live-api-master\demos\node\node_modules\ws\lib\websocket.js:579:7)
at ClientRequest.emit (events.js:311:20)
at HTTPParser.parserOnIncomingClient [as onIncoming] (_http_client.js:603:27)
at HTTPParser.parserOnHeadersComplete (_http_common.js:119:17)
at TLSSocket.socketOnData (_http_client.js:476:22)
at TLSSocket.emit (events.js:311:20)
at addChunk (_stream_readable.js:294:12)
at readableAddChunk (_stream_readable.js:275:11)
at TLSSocket.Readable.push (_stream_readable.js:209:10)
at TLSWrap.onStreamRead (internal/stream_base_commons.js:186:23)
Emitted 'error' event on WebSocket instance at:
at abortHandshake (C:\Users\binary-live-api-master\binary-live-api-master\demos\node\node_modules\ws\lib\websocket.js:693:15)
at ClientRequest. (C:\Users\binary-live-api-master\binary-live-api-master\demos\node\node_modules\ws\lib\websocket.js:579:7)
[... lines matching original stack trace ...]
at TLSSocket.Readable.push (_stream_readable.js:209:10)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
How can I fix this?

@lechim do you mean the nodejs bot within the main repo?
Short answer is you can't really, there would be so much refactoring I'd just build you own node app. I have a very stable node app base. If you send me you email I can give you a copy. From that you can just write a strategy in node and drop in the start folder. It uses technical indicators npm so you can use anytging from that package for you strays,  or any vanilla or stats lite functions

@wilburforce83 Yeah it seams to me that will be the case. I do not really want to rebuild it.
If you can send me a copy of that code base I will be able to try work from it<EMAIL_ADDRESS>Thank you.


Use clang-3.8

Fun fact: this almost destroyed all of the Jenkins linux workers, which spent an insane amount of time building clang :-)

Funner fact: it's not the answer :)
On Thu, Jan 14, 2016 at 4:39 PM Nicolas Noble<EMAIL_ADDRESS>wrote:

Fun fact: this almost destroyed all of the Jenkins linux workers, which
spent an insane amount of time building clang :-)
—
Reply to this email directly or view it on GitHub
https://github.com/grpc/grpc/pull/4733#issuecomment-171830014.

Jenkins needed a challenge today though.
On Thu, Jan 14, 2016 at 4:44 PM Craig Tiller<EMAIL_ADDRESS>wrote:

Funner fact: it's not the answer :)
On Thu, Jan 14, 2016 at 4:39 PM Nicolas Noble<EMAIL_ADDRESS>wrote:

Fun fact: this almost destroyed all of the Jenkins linux workers, which
spent an insane amount of time building clang :-)
—
Reply to this email directly or view it on GitHub
https://github.com/grpc/grpc/pull/4733#issuecomment-171830014.


The database stores everything in plain text.

In the Application database it stores everything in straight plain text.

Could you leave an example of an entry, so we can see what types of data is being stored in each row?

We've moved hardening tasks like this to a spreadsheet. It isn't relevant to this GitHub repo.


Fix IterativeSolvers.jl for Julia 0.7

wip

Unfortunately the tests require Plots.jl for the test/history.jl smoke test, and one of the dependencies fails on 0.7.
I would really be in favor of just moving plotting tools to a different package like IterativeSolversTools.jl so that IterativeSolvers is very close to being self-contained.


flutter_markdown 0.1.1 flutter packages get fails

Put flutter_markdown: "^0.1.1" in pubspec.yaml
flutter packages get

Error :
Running "flutter packages get" in markdown...
Package flutter_markdown has no versions that match >=0.1.1 <0.2.0 derived from:

markdown depends on version ^0.1.1
---- Log transcript ----
FINE: Pub 2.0.0-dev.16.0
MSG : Resolving dependencies...
SLVR: Solving dependencies:
| - cupertino_icons ^0.1.0 from hosted (cupertino_icons) (locked to 0.1.1)
| - flutter_markdown ^0.1.1 from hosted (flutter_markdown)
| - flutter any from sdk (flutter) (locked to 0.0.0)
IO  : Get versions from https://pub.dartlang.org/api/packages/flutter_markdown.

ERR : Package flutter_markdown has no versions that match >=0.1.1 <0.2.0 derived from:
| - markdown depends on version ^0.1.1
FINE: Exception type: NoVersionException
FINE: package:pub/src/entrypoint.dart 216                                             Entrypoint.acquireDependencies
| package:pub/src/command/get.dart 38                                             GetCommand.run
| package:args/command_runner.dart 194                                            CommandRunner.runCommand
| package:pub/src/command_runner.dart 167                                         PubCommandRunner.runCommand.
| dart:async                                                                      new Future.sync
| package:pub/src/utils.dart 103                                                  captureErrors.
| package:stack_trace                                                             Chain.capture
| package:pub/src/utils.dart 118                                                  captureErrors
| package:pub/src/command_runner.dart 167                                         PubCommandRunner.runCommand
| package:pub/src/command_runner.dart 116                                         PubCommandRunner.run
| /b/build/slave/dart-sdk-linux-dev/build/sdk/third_party/pkg/pub/bin/pub.dart 8  main
| ===== asynchronous gap ===========================
| dart:async                                                                      _Completer.completeError
| package:pub/src/entrypoint.dart                                                 Entrypoint.acquireDependencies
| ===== asynchronous gap ===========================
| dart:async                                                                      _asyncThenWrapperHelper
| package:pub/src/entrypoint.dart 194                                             Entrypoint.acquireDependencies
| package:pub/src/command/get.dart 38                                             GetCommand.run
| package:args/command_runner.dart 194                                            CommandRunner.runCommand
| ===== asynchronous gap ===========================
| dart:async                                                                      new Future.microtask
| package:args/command_runner.dart 142                                            CommandRunner.runCommand
| package:pub/src/command_runner.dart 167                                         PubCommandRunner.runCommand.
| dart:async                                                                      new Future.sync
| package:pub/src/utils.dart 103                                                  captureErrors.
| package:stack_trace                                                             Chain.capture
| package:pub/src/utils.dart 118                                                  captureErrors
| package:pub/src/command_runner.dart 167                                         PubCommandRunner.runCommand
---- End log transcript ----
pub get failed (1)

[✓] Flutter (on Linux, locale en_CA.UTF-8, channel alpha)
• Flutter version 0.0.21 at /work/installs/flutter
• Framework revision 2e449f06f0 (8 days ago), 2018-01-29 14:26:51 -0800
• Engine revision 6921873c71
• Tools Dart version 2.0.0-dev.16.0
• Engine Dart version 2.0.0-edge.da1f52592ef73fe3afa485385cb995b9aec0181a
[✓] Android toolchain - develop for Android devices (Android SDK 27.0.3)
• Android SDK at /home/david/Android/Sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-27, build-tools 27.0.3
• ANDROID_HOME = /home/david/Android/Sdk
• Java binary at: /work/installs/android-studio-3.0/jre/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b01)
[✓] Android Studio (version 3.0)
• Android Studio at /work/installs/android-studio-3.0
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b01)
[✓] IntelliJ IDEA Community Edition (version 2017.3)
• Flutter plugin version 21.2.3
• Dart plugin version 173.4548.30
[-] Connected devices
• None

What happens when you try flutter_markdown: any ?

I get a similar error :
flutter packages get
Running "flutter packages get" in markdown...
Package markdown has no versions that match ^0.11.0 derived from:

flutter_markdown 0.0.9 depends on version ^0.11.0
---- Log transcript ----
FINE: Pub 2.0.0-dev.16.0
MSG : Resolving dependencies...
SLVR: Solving dependencies:
| - cupertino_icons ^0.1.0 from hosted (cupertino_icons) (locked to 0.1.1)
| - flutter_markdown any from hosted (flutter_markdown)
| - flutter any from sdk (flutter) (locked to 0.0.0)
IO  : Get versions from https://pub.dartlang.org/api/packages/flutter_markdown.

ERR : Package markdown has no versions that match ^0.11.0 derived from:
| - flutter_markdown 0.0.9 depends on version ^0.11.0
FINE: Exception type: NoVersionException
FINE: package:pub/src/entrypoint.dart 216                                             Entrypoint.acquireDependencies
| package:pub/src/command/get.dart 38                                             GetCommand.run
| package:args/command_runner.dart 194                                            CommandRunner.runCommand
| package:pub/src/command_runner.dart 167                                         PubCommandRunner.runCommand.
| dart:async                                                                      new Future.sync
| package:pub/src/utils.dart 103                                                  captureErrors.
| package:stack_trace                                                             Chain.capture
| package:pub/src/utils.dart 118                                                  captureErrors
| package:pub/src/command_runner.dart 167                                         PubCommandRunner.runCommand
| package:pub/src/command_runner.dart 116                                         PubCommandRunner.run
| /b/build/slave/dart-sdk-linux-dev/build/sdk/third_party/pkg/pub/bin/pub.dart 8  main
| ===== asynchronous gap ===========================
| dart:async                                                                      _Completer.completeError
| package:pub/src/entrypoint.dart                                                 Entrypoint.acquireDependencies
| ===== asynchronous gap ===========================
| dart:async                                                                      _asyncThenWrapperHelper
| package:pub/src/entrypoint.dart 194                                             Entrypoint.acquireDependencies
| package:pub/src/command/get.dart 38                                             GetCommand.run
| package:args/command_runner.dart 194                                            CommandRunner.runCommand
| ===== asynchronous gap ===========================
| dart:async                                                                      new Future.microtask
| package:args/command_runner.dart 142                                            CommandRunner.runCommand
| package:pub/src/command_runner.dart 167                                         PubCommandRunner.runCommand.
| dart:async                                                                      new Future.sync
| package:pub/src/utils.dart 103                                                  captureErrors.
| package:stack_trace                                                             Chain.capture
| package:pub/src/utils.dart 118                                                  captureErrors
| package:pub/src/command_runner.dart 167                                         PubCommandRunner.runCommand
---- End log transcript ----
pub get failed (1)

Try deleting .packages and try again?
I just created an empty sample project, added flutter_markdown: any, and ran flutter packages get and it worked.
cc @mit-mit for feedback on this tooling flow.

I deleted .packages and tried again, but I got the same error as directly previous.
I made a brand new flutter create project and added the flutter_markdown: any and flutter packages get and it worked.
I played around with the markdown and it looks like it will do what I need.
Thanks Seth.
This issue can be closed, unless there is anything which could be improved in the toolchain to make it easier. Like recommending something that works on these types of error, such as flutter clean (idk if that works).

Great, glad you got it to work!
Let's leave this open for @mit-mit and maybe @kevmoo to take a look, and see if there is a systemic improvement we can make.

flutter packages upgrade should do without deleting files (instead of flutter packages get).
Using any as version constraints is discouraged because it is prone to cause the dependency resolver to go exponential. Use rather narrow version constraints instead.
A new resolver algorithm is work-in-progress that should be faster and provide better errors when   no compatible set of dependencies can be found.

Yes, flutter packages upgrade should have been enough.
I think the core issue here is the extremely noisy pub output that makes it near impossible to see resolution issues. We expect to resolve that in https://github.com/flutter/flutter/issues/13179.

I'm going to close this because we believe it's been resolved upstream in the pub tool itself.


Profile styling and new donation styling file

-profile styling is still in progress

added donation file so the nav bar isnt hiding the inputs and buttons

Startin to look pertyyyyy! Thanks for moving that stuff.


chore: Unify the planning order

Description

What is the purpose of this PR?

Unify the planning order

What problem does it solve?
Are there any breaking changes or backwards compatibility issues?

Related Issue
Type of Change

[x] Miscellaneous tasks

How Has This Been Tested?

[ ] I have run unit tests
[ ] I have tested the changes manually
[ ] I have tested in a staging environment

Checklist

[x] I have read and followed the guidelines in CONTRIBUTING.md
[ ] I have already updated the related templates accordingly (if applicable)
[ ] I have written or updated relevant documentation (if applicable)
[ ] I have added or updated tests to cover my changes (if applicable)
[x] I have reviewed my code for any potential issues

Additional Notes
This approach doesn't make much sense, but we aim to standardize the sequence of steps in the workflows.

Additionally, the previous caching error in cd.yml was caused by the order of the Checkout step. However, since cd.yml usually only runs once, I decided to disable rust-cache for it:
Error: The process '/Users/runner/.cargo/bin/cargo' failed with exit code 101
    at ExecState._setResult (/Users/runner/work/_actions/Swatinem/rust-cache/v2/dist/restore/index.js:9728:25)
    at ExecState.CheckComplete (/Users/runner/work/_actions/Swatinem/rust-cache/v2/dist/restore/index.js:9711:18)
    at ChildProcess.<anonymous> (/Users/runner/work/_actions/Swatinem/rust-cache/v2/dist/restore/index.js:9605:27)
    at ChildProcess.emit (node:events:519:28)
    at maybeClose (node:internal/child_process:1105:16)
    at Socket.<anonymous> (node:internal/child_process:457:11)
    at Socket.emit (node:events:519:28)
    at Pipe.<anonymous> (node:net:339:12) {
  commandFailed: {
    command: 'cargo metadata --all-features --format-version 1 --no-deps',
    stderr: '\x1B[1m\x1B[31merror\x1B[0m\x1B[1m:\x1B[0m could not find `Cargo.toml` in `/Users/runner/work/katharsis/katharsis` or any parent directory\n'
  }
}


feat(res): support 16-bit entry offsets

Fixes https://github.com/skylot/jadx/issues/2267

Hm, with sample provided in https://github.com/skylot/jadx/issues/2267#issuecomment-2334555118 parsing fail at closest
is.checkPos(entriesStart, "Expected first entry start");

because of additional 2 bytes, maybe it is padding, so it might be better to change this line to
is.skipToPos(entriesStart, "Failed to skip to entries start");

And, as soon as, sample also use compact entries, it still can't be parsed, so I am not sure if this is a correct solution.
To be honest, it is better to merge this PR with compact entries support, because this is the only way to verify its correctness.

This PR only adds support for 16-bit entry offsets. Not support for compact entries, not support for non-standard paddings, only 16-bit entry offsets.
But if you insist on implementing all together, then you can close the PR. I don't think I'll create a PR with compact entries support, because my current implementation of the resource parser is too different from yours.

I see.
I will try to implement compact entries support myself and will merge this PR once I managed to parse provided sample.


scripts: kconfig: Do not print warnings for choice overriding

Andy Ross wanted to override a choice selection from a board defconfig
file in a prj.conf, but this could trigger the
assigned-value-didn't-take warning, because the choice symbol set to y
in the board defconfig file ends up as n after another choice symbol is
selected.
(Note: Setting any choice symbol to y is enough to make it the choice
(user) selection. There's no need to set the other symbols to n.)
Fix the warning by checking choices at the choice level rather than at
the level of individual choice symbols. This also makes the warning a
bit more informative in general for choice symbols.

CC @carlescufi @andyross

Codecov Report

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

@@           Coverage Diff           @@
##           master    #8607   +/-   ##
=======================================
  Coverage   64.17%   64.17%
=======================================
  Files         429      429
  Lines       41157    41157
  Branches     6925     6925
=======================================
  Hits        26413    26413
  Misses      11587    11587
  Partials     3157     3157

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 cd1111e...16a5a71. Read the comment docs.


Title in docs page is - PXT

should be target's name.

@pelikhan can you attach a snapshot. I don't know where you see PXT.

See page title on https://pxt.microbit.org/about

Won't Fix


Support for type definition when using suspense

before
after

trpc-swr is a great library, but there is one problem.
That is that type completion does not work properly when using suspense mode to retrieve data.
We have created a PR that fixes this issue. We hope it will be a better library!

Apologies for the wait, Ill be looking at this tonight!

@sannajammeh Thank you. We are looking forward to seeing you.

@sannajammeh Thank you! You have been added to collaborator. Please check it out!


Error on generating addresses

Hi
On register and under my account, when trying to generate new addresses, i get error from easybitcoin.php
PHP Warning:  Could not resolve host:  in /var/www/wlox/wlox-api/lib/easybitcoin.php on line 164
Running latest mbassan.
It does not generate bitcoin addresses even. Checked that rpc is working with daemon.

Remember to set http in protocol under wallets :)


Pip command doesn't work

Hello,
I'm trying to install your package and it seems unavailable
pip install vinted_scraper

ERROR: Could not find a version that satisfies the requirement vinted_scraper (from versions: none)
ERROR: No matching distribution found for vinted_scraper

I get the same error with
pip install vinted_scraper

Do you have any idea why?

My bad I had the wrong version of python. It doesn't work above Python 3.9

Hi,
I'll reuse this issue to try to lower the module's requirements. The only reason for the actual requirements is that they met my local environment. But we can try to be as flexible as we can so everybody can use it.


3 working test mosuckka!

testing change state in serachField and the printing of the proper day of the week and location in currentWeather

oh daggggggggggg boiiiii


How to import pipeline into StreamSets Data Collector container

Hi,
I am new to StreamSets. Currently I have run the following data collector container:
docker run -it -p 18630:18630 -d --name sdc --network mynetwork streamsets/datacollector
My question is how to import the following JSON pipeline file:
wget https://raw.githubusercontent.com/mapr-demos/predictive-maintenance/master/StreamSets/MQTT File Tail.json
into the SCD?
I am trying to google around, but could not find a simpler way to do it. Mostly requires us to register StreamSets account, which I think is not necessary in this case. Please help! Thanks!

This is really a StreamSets question.
You can download this file by cloning the git repository. From there, you should use StreamSets capabilities to upload the json file to StreamSets. There is likely some sort of "upload file" capability, but you would have to work with them to figure that out.
Good luck!


Fix(web): FileUploader cannot remove files with same names

refs #DS-830

Description
FileUploader cannot remove files with the same names
Additional context

generate simple uuid for every instance of FileUploader and use it as a prefix for file id
this avoids having the same file ids on elements that we are querying when removing files from queue

Issue reference

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Follow the PR Title/Commit Message Convention.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[x] Ideally, include relevant tests that fail without this PR but pass with it.

Double-checked with Miloš Hejný. So I am merging this.


Lab should be update to use new AMA workflow

Module/Lab: 09
Exercise: 02
Task: 00all
Step: 00
Description of issue
Lab uses log analytics agent.
This is no longer supported Please update the lab AND the course/exam to reflect this.

Thanks for the update. Lab 2 is being deprecated due to security issues as well as issues arising from the retiral of the Log Analytics agent and Azure Automation Update Management. A readme was posted on GitHub a while back, advising instructors not to run lab 2 or lab 9 because of the impacts due to the retirals. It's possible that people may not have been aware of it
The plan is to provide updated labs likely in the mid-January timeframe that account for the changes. We will keep you posted and apologize sincerely for the inconvenience. (Shouldn't impact the exam as any related questions would have been pulled in an earlier exam scrub).
Thanks,
The Courseware team.


Add support for video embedding on posts with links

Add support for video on posts with links.

Done 8b3fdf2157b645c5a57f1fe5f981e953aeb4782e.


Windows mods

Switched to use Thread instead of Process in change_monitor to fix the pickling_error in Windows.  Also tweaked the way test files are declared to be Windows-friendly

Tests pass, but stopping the couch_change_monitor thread in test_sword.py exits with an unhandled exception. Can someone else confirm this?

Lou,
Can you point me to this solution, and did this resolve the pickling error?

Unfortunately it did not.  What we thought was a fix actually caused different errors on the other platforms (Linux and Mac OS) so I am still working on a solution for this.

Unfortunately it did not.  What we thought was a fix actually caused different errors on the other platforms (Linux and Mac OS) so I am still working on a solution for this.

@jpoyau is working on a new factory model to allow windows platforms to use threads and linux and mac platforms to use process.

It sounds like we're stuck due to the distribution tests that are failing as reported in issue #183.  @jpoyau What is the background?  Any theories about what is going wrong?  How can others help you?

@jpoyau  Looks like you have cracked this! Any points of direction please?

ok, am still getting the pickling error unfortuantely :-( I did checked out the latest tag 0.23.5. Any suggestions?

John,
It ran fine for me on my Windows environment. The distribute tests all passed as well. Thanks for tackling this sticky problem.


WinForms application crashes in RDS environment

stacktrace.txt
User's set up:
"mini-farm of RD servers in VMWare with Windows Server 2016 Datacenter"
For now disabling RefreshView module on Tolis advice.

We will try to answer all questions that do not require research within 24hr.
To prioritize cases that require research we use the following labels in order. For all other issues the posting time is respected.

❇ Exclusive-Services
❤ Bronze Sponsor
❤ Sponsor
❤ Backer
Installation
ShowStopper
Nuget
Contribution
BreakingChange
ReproSample
Deployment
must-have

This case is prioritized.

eXpand.lab release 20.2.302.1 includes commit that relate to this task:

ExcelImporter AutoImportService throws cross thread exception #796

Please test if it addresses the problem. If you use nuget add our LAB NugetServer as a nuget package source in VS.
To minimize version conflicts we recommend that you switch to PackageReference format and use only the eXpandAgnostic, eXpandWin, eXpandWeb packages. Doing so, all packages will be at your disposal and .NET will add a dependecy only to those packages that you actually use and not to all (see the Modules installation-registrations youtube video).
Thanks a lot for your contribution.

Issue is deprioritized as no Assignee found and scheduled for auto-close if no activity in the next 60 days. Thanks a lot for your contribution.

Closing issue for age. Feel free to reopen it at any time.
.Thank you for your contribution.


Test of subqueries on a result

This just adds a test to highlight that the current C-API implementation doesn't take filtering of the result into account when it does a subquery on the result.

Superseded by #1013


Conservative rasterization question

Hello, bgfx support conservative rasterization, or not? Need on DirectX 12 and OpenGL.
I try to make reflections, and want move to high level language. Need also load structured buffers. So need support of buffer that have only size in bytes, and can load any data i.e. void * or char *.
I use Voxel octree structures as SSBO.

There is no support for it right now.

I suggest to add "enable" feature for shader creation.

Sad =(
Need NV conservative raster extensions.

You can modify library to suit your needs. I have no ETA for the feature.

It's added, see:
https://github.com/bkaradzic/bgfx/blob/master/include/bgfx/bgfxdefines.h#L77
https://github.com/bkaradzic/bgfx/blob/master/include/bgfx/bgfxdefines.h#L387

Have for both: OpenGL (extension)  and DirectX 12?

It's added GL, D3D11 (11.3), and D3D12.

atomic* are supported. Just use regular buffers.

Can I declare?
struct Structure {
  ..
};
BUFFER_WR(SomeBuffer, Structure, 2);

Use uvec4, vec4, and pack it manually.

Example of manual packing: https://github.com/bkaradzic/bgfx/blob/master/src/bgfx_compute.sh#L227

As I know, count of heaps are limited?
2016-03-17 14:44 GMT+08:00 Branimir Karadžić<EMAIL_ADDRESS>
Example of manual packing:
https://github.com/bkaradzic/bgfx/blob/master/src/bgfx_compute.sh#L227
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/bkaradzic/bgfx/issues/720#issuecomment-197730710


Helm applied applied only 'half' a spec without errors

Running Helm v2.4.2 with a K8S 1.6.2 cliuster in AWS.
We have a Deployment that has an initContainers in the spec, and noticed yesterday that while the spec was being upgraded, the initContainers part was skipped/ignored due to a typo (comma in an .env.name) but no error during the upgrade.
Getting the deployment with helm get and applying the YAML manaully with kubectl apply -f ... did produce an error.
The Deployment "test-err" is invalid: spec.template.spec.initContainers[0].env[0].name: Invalid value: "NAME,": a valid C identifier must start with alphabetic character or '_', followed by a string of alphanumeric characters or '_' (e.g. 'my_name',  or 'MY_NAME',  or 'MyName', regex used for validation is '[A-Za-z_][A-Za-z0-9_]*')

I would expect helm to return this error when performing the upgrade rather than just not updating that part of the spec.

I am curious whether this is specific to deployments, or whether replica sets have this issue too.

@TimJones, had you modified the Deployment in situ since Helm had installed the previous version? I wonder if the patch submitted during upgrade omitted the "initContainers" portion if it didn't appear to have changed since the last version.

@seh Quite probably, I'll check with my colleagues.
Does Helm only check with what it applied (i.e. making a diff with it's stored configmap) against the new release? Never goes back to the Kubernetes API to confirm what it thinks should be there?

I asked about this behavior in the "helm-dev" channel of the "Kubernetes" Slack team on 28 April 2017. In short, Helm looks at the last manifests it applied when considering applying a new set, and uses the differences between them to submit patches. Helm does not take the current state of the Kubernetes objects into account.
This surprised me. As @technosophos said in that discussion,

It was one of our early "simplifying assumptions" that at some point we probably ought to revisit

though to do so would potentially change Helm into an "enforcer," where you could tell it to ensure that the current cluster state matches the intended state for an already-installed chart. Doing so requires ignoring parts of many of the objects (e.g. the status of a ResourceQuota).

I would expect Helm to be an enforcer at the point of running the install or upgrade command. If I set a tag in a values file I expect it to be applied to the running deployment regardless of the deployment being modified since the previous install or upgrade.


Android Build Error - flutter_uploader-3.0.0-beta.3 - cannot find symbol   mainExecutor = ContextCompat.getMainExecutor(context)

This is error that I get on android. Can you help, how can I fix this?
.pub-cache/hosted/pub.dartlang.org/flutter_uploader-3.0.0-beta.3/android/src/main/java/com/bluechilli/flutteruploader/MethodCallHandlerImpl.java:46: error: cannot find symbol
mainExecutor = ContextCompat.getMainExecutor(context);
^
symbol:   method getMainExecutor(Context)
location: class ContextCompat
Note: /Users/danilo/Developer/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_uploader-3.0.0-beta.3/android/src/main/java/com/bluechilli/flutteruploader/FlutterUploaderPlugin.java uses or overrides a deprecated API.
This is flutter doctor:
`✓] Flutter (Channel stable, 2.2.3, on Mac OS X 10.15.6 19G2021 darwin-x64, locale en-GB)
• Flutter version 2.2.3 at /Users/danilo/Developer/flutter
• Framework revision f4abaa0735 (9 weeks ago), 2021-07-01 12:46:11 -0700
• Engine revision 241c87ad80
• Dart version 2.13.4
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
• Android SDK at /Users/danilo/Library/Android/sdk
• Platform android-30, build-tools 30.0.2
• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 12.2, Build version 12B45b
• CocoaPods version 1.10.2
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 4.0)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin version 51.0.1
• Dart plugin version 193.7547
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)`

I found solution for this:
from build.gradle I was deleted this lines and now is ok.
   subprojects {
        project.configurations.all {
            resolutionStrategy.eachDependency { details ->
                if (details.requested.group == 'com.android.support'
                        && !details.requested.name.contains('multidex') ) {
                    details.useVersion "27.1.1"
                }

                if (details.requested.group == 'androidx.core'
                        && !details.requested.name.contains('androidx') ) {
                    details.useVersion "1.0.1"
                }
            }
        }
    }


added responseFormatInterface

In Response class \Dingo\Api\Http\Format\FormatInterface was type hinted but the interface is not present.
hence created \Dingo\Api\Http\ResponseFormat\ResponseFormatInterface class.
  /**
     * Get the formatter based on the requested format type.
     *
     * @param string $format
     *
     * @throws \RuntimeException
     *
     * @return \Dingo\Api\Http\Format\FormatInterface
     */
    public static function getFormatter($format)
    {
        if (! static::hasFormatter($format)) {
            throw new NotAcceptableHttpException('Unable to format response according to Accept header.');
        }

        return static::$formatters[$format];
    }

This would've been an error in the docblock. Shifting my focus to L5 and Lumen so things here have changed.