text:
Planning for migration to JuliaGeometry

CC: @JoshuaLampert @juliohm
The long-term aim of this package is to be transitioned under the JuliaGeometry organization.
I'll add notes here as I think it through, but here's some details I figure we should iron out:
Package Name/Scope
Should the package name remain MeshIntegrals.jl or change?
What is the intended scope of this package?

Right now the scope is fairly specific to solving integral problems. I've defined and exported a Jacobian function, which is used internally for calculating differential elements (e.g. the dV for a volume integral), but theoretically could be useful in other contexts. I have visions of AD rules for partial derivatives of Meshes geometry parametric functions, but I suspect those should properly be added to Meshes itself. Are there any other specific interests we should be addressing?

Logistics
How to perform a repo transfer on GitHub?

I'm assuming it's under Settings > Danger Zone > Transfer Ownership but I've never done this before.
If the package name changes, do we still perform a repo Transfer or start a new repo with ported contents?

How to handle registry in General?

If the package name stays the same, how do we re-link the General registry toward the new GitHub page?
If the package name changes,

Register new name in General pointing to new repo
How do we gracefully obsolete this package name?

Any changes to maintainers/roles?
Are there any impacts to GitHub Actions being under an organization vs an individual free account?

Hi @mikeingold, thanks for starting the issue.

Should the package name remain MeshIntegrals.jl or change?

I believe that you already have some notions of differentiation implemented, so I think a broader name would be appropriate. Perhaps MeshCalculus.jl is a good name?

What is the intended scope of this package?

Right now the scope is fairly specific to solving integral problems. I've defined and exported a Jacobian function, which is used internally for calculating differential elements (e.g. the dV for a volume integral), but theoretically could be useful in other contexts. I have visions of AD rules for partial derivatives of Meshes geometry parametric functions, but I suspect those should properly be added to Meshes itself. Are there any other specific interests we should be addressing?

Perhaps we should list the pros and cons of implementing these AD rules and integrals directly in Meshes.jl before deciding what to do with the migration. These building blocks can be useful in different applications, we just need to figure out where to place dependencies and how to increase integration.

How to perform a repo transfer on GitHub?

I'm assuming it's under Settings > Danger Zone > Transfer Ownership but I've never done this before.
If the package name changes, do we still perform a repo Transfer or start a new repo with ported contents?

I think the best option is to create a new package with a different name in JuliaGeometry. That way people can still use MeshIntegrals.jl in the latest version before the migration. The general registry doesn't allow renaming packages, just the URLs where they are host.

Any changes to maintainers/roles?

You guys are the leaders of this development. JuliaGeometry would just be a new home with greater visibility, you would have all admin rights necessary to evolve the project as you wish.

Are there any impacts to GitHub Actions being under an organization vs an individual free account?

I believe JuliaGeometry is also a free account. The actions are run likewise as far as I know. If you had a paid personal account that could be different.
@JoshuaLampert what is your take on the next steps?

I mostly agree with everything you wrote above.
Regarding the name I think MeshCalculus.jl is a good name if we want to extend scope to also include proper support for differentials and AD rules and potentially more. However, if this part will live directly in Meshes.jl and will just be used in this package (whatever it will be called), I wonder whether it makes sense to change the name.
I think I slightly tend to implement everything AD related directly to Meshes.jl probably as a package extension using one of the AD packages. In that case this package could use these routines from Meshes.jl and the extension, but still only compute integrals. Thus, a name MeshCalculus.jl could be misleading and MeshIntegrals.jl would be more appropriate.
Regarding the transfer: If we create a new repo in JuliaGeometry and keep this one, we would loose the commit, issue, PR etc. history, which is not that a big deal, but conserving the history would still be nice for future reference. I consider this as an advantage of transferring the repo to the new org without renaming it. We would then follow the steps described here.
Regarding GitHub Actions and similar: I think that should be no big deal. Maybe we will need to add a new DOCUMENTER_KEY and a COVERAGE_TOKEN and similar (or does JuliaGeometry use org wide tokens?), but that should be no problem, I think. The only thing we need to adjust is this line, I assume.

For clarity on AD rules, just to make sure we’re all talking about the same thing:
Right now, MeshIntegrals.jl uses general solution methods of the form integral(f, geometry). The differential element is calculated using a Jacobian, taking the partial derivatives of the geometry’s parametric function with respect to each argument. For example, if box = Box(Point(0,0), Point(1,1)), then jacobian(box, (x,y)) for some valid parametric coordinates (x,y) would return (Vec(1,0), Vec(0,1)).
Currently, we’re performing that Jacobian inside this package via a finite-difference approximation. Ideally, some kind of AD rule would be defined, but in my opinion the most appropriate place for those to be defined is in Meshes.jl directly, since that’s where the functions themselves are defined. Defining AD Chain Rules in MeshIntegrals.jl for Meshes.jl functions would be type piracy and also probably fragile to upstream changes.

Thanks @mikeingold. I totally agree!

Makes total sense 💯 Looking forward to AD rules :)

Ok, so do I understand correctly that we agree on:

[ ] keeping the name MeshIntegrals.jl,
[ ] moving the repo to JuliaGeometry,
[ ] add AD rules directly to Meshes.jl in the future (probably as a package extension),
[ ] replace the FD formula in MeshIntegrals.jl by the AD rules then defined in Meshes.jl?

If so, I think the next steps are:

[ ] Granting admin rights for JuliaGeometry to @mikeingold (and myself).
[ ] Move the repo via Settings > Danger Zone > Transfer Ownership to JuliaGeometry.
[ ] See if GitHub CI including coverage and Documenter works and potentially fixing it (adding necessary tokens etc.). This includes fixing this line.
[ ] Update the new org in the General registry as described here.
[ ] Adding AD rules to Meshes.jl (some time in the future).
[ ] Replacing the FD formula in MeshIntegrals.jl by the AD rules from Meshes.jl (some time in the future).
[ ] Party :partying_face:

Did I miss anything or are there any objections?

Here goes nothing 🤟

The package has been transferred to JuliaGeometry and I've got a PR pending to update the General registry
https://github.com/JuliaRegistries/General/pull/117915

See also #117 for updating the link to the docs.

I just pinged the Slack channel to make sure the General PR doesn't languish.

@JoshuaLampert, do you have access/rights to the full repo settings, e.g. tokens, secrets, etc?

No, I don't. Do you, @mikeingold?

Actually, it looks like I no longer have access to any repo settings. 😕

Maybe you can grant ownership to a specific repo in the org?

@JoshuaLampert I have been interested/wanting AD rules in Meshes.jl for awhile. I have decent knowledge of ChainRules.jl etc. but sometimes get stuck on the math things for more complex functions. I think Meshes.jl doesn't have too much complex math going on so it shouldn't be that bad. Anyways, what I'm trying to say is I'd like to help in this effort. We need an ongoing issue we can track progess. We can make this issue in Meshes.jl?

@JoshuaLampert I have been interested/wanting AD rules in Meshes.jl for awhile. I have decent knowledge of ChainRules.jl etc. but sometimes get stuck on the math things for more complex functions. I think Meshes.jl doesn't have too much complex math going on so it shouldn't be that bad. Anyways, what I'm trying to say is I'd like to help in this effort. We need an ongoing issue we can track progess. We can make this issue in Meshes.jl? Sorry for kinda hijacking this thread

#35  vaguely tracks this as a standing goal for us in this package.
I’ve done some minimal exploration and none of the AD packages I’ve tried worked out of the box. Ideally, we could differentiate the parametric functions for geometries and get a Meshes.Vec for each partial. The finite-diff approximation version of this is around here. Theoretically, the simplest version of this kind of derivative looks something like
derivative(s::Segment, coord) = maximum(s) - minimum(s)

but I don’t currently have enough experience writing chain rules to know how to effectively implement this in practice.
I’m also not sure if writing these directly for parametric functions is the best path, or if we can write them for lower-level functions and have that support roll up automatically.

Thanks @kylebeggs for chiming in! Your help would be very much appreciated. Let's maybe continue the discussion over at #35.

Awesome, looks like the rights issue has been resolved. Thanks, @juliohm!
@JoshuaLampert, do you have access to the full Settings tab, including keys/tokens now?

Yes, I have. Thanks a lot @juliohm!

The migration to JuliaGeometry is now successfully completed. The remaining points of the TODO list above are tracked in #35.

authors: JoshuaLampert juliohm kylebeggs mikeingold


text:
nested if binder not re-evaluating nodes

Notice in the JSFiddle below how the if binder retains an old state of its child elements upon unbinding and doesn't re-calculate the child element when bound again. I understand this is because the observable on the if binding itself isn't changing but one would expect it to re-evaluate the children upon binding again regardless.
https://jsfiddle.net/rpefhkhg/
I have a proposed re-write of the if binder with forceful unbinds and bind on its children. This in my opinion is an in-efficient solution and I'm hoping the authors may have a slightly graceful approach to this problem. Adding it below. Notice inline comments to see the changes.
Rivets["public"].binders["if"] = {
    block: true,
    priority: 4000,
    bind: function(el) {
      var attr, declaration;
      this.bound = null; // Set to null
      if (this.marker == null) {
        attr = [this.view.prefix, this.type].join('-').replace('--', '-');
        declaration = el.getAttribute(attr);
        this.marker = document.createComment(" rivets: " + this.type + " " + declaration + " ");
        this.bound = null; // Set to null
        el.removeAttribute(attr);
        el.parentNode.insertBefore(this.marker, el);
        return el.parentNode.removeChild(el);
      }
    },
    unbind: function() {
      var _ref1;
      this.bound = null; // Set to null
      return (_ref1 = this.nested) != null ? _ref1.unbind() : void 0;
    },
    routine: function(el, value) {
      var key, model, models, _ref1;
      if (!!value === !this.bound || this.bound === null) {
        if (value) {
          models = {};
          _ref1 = this.view.models;
          for (key in _ref1) {
            model = _ref1[key];
            models[key] = model;
          }
          (this.nested || (this.nested = new Rivets.View(el, models, this.view.options()))).bind();
          this.marker.parentNode.insertBefore(el, this.marker.nextSibling);
          return this.bound = true;
        } else if(el.parentNode != null) { // Parent may be missing
            this.nested.unbind(); // Unbind before moving child
            el.parentNode.removeChild(el);
          return this.bound = false;
        }
      }
    },
    update: function(models) {
      var _ref1;
      return (_ref1 = this.nested) != null ? _ref1.update(models) : void 0;
    }
  };

I have an updated JSFiddle with a much thorough and improved JS fix. The link below demonstrates all problem cases I've encountered.
https://jsfiddle.net/cvo4qomt/
    // ak: Fix for issue #618 - if binders need their view to be built again
    View.prototype.rebind = function() {
      var binding, _i, _len, _ref1, _results;
      _ref1 = this.bindings;
      _results = [];
      for (_i = 0, _len = _ref1.length; _i < _len; _i++) {
        binding = _ref1[_i];
        if(binding.type === "if"){
            _results.push(this.build());
            _results.push(this.bind());
        }else{
            _results.push(binding.bind());
        }
      }
      return _results;
    };

    // ak: Fix for issue #618 - replace element text with original token
    TextBinding.prototype.unbind = function() {
      var delimiters, _results;
      delimiters = this.view.templateDelimiters;
      _results = TextBinding.__super__.unbind.apply(this, arguments);
      this.el.textContent = delimiters[0] + this.keypath + delimiters[1];
      return _results;
    };

    // ak: Fix for issue #616 - sync after updating models
    TextBinding.prototype.update = function() {
      var _results = [];
      _results.push(TextBinding.__super__.update.apply(this, arguments));
      _results.push(TextBinding.__super__.sync.apply(this, arguments));
      return _results;
    };

The routine for rv-if needs to be changed as follows:
-          (this.nested || (this.nested = new Rivets.View(el, models, this.view.options()))).bind();
+          // ak: Fix for issue #618 - nested if content needs to be re-bound
+          if(this.nested){
+            this.nested.rebind();
+          }else{
+            this.nested = new Rivets.View(el, models, this.view.options());
+            this.nested.bind();
+          }

Hi @tanshul22 , is it a duplicate of #611 ?

Hi @jccazeaux, I tried setting bound to false (and to null as in first comment above) for starters and it does seem to solve some basic problems but I have an extremely large and complex application with multiple nested if binders and setting bound to false doesn't seem to solve all my issues. I've updated the second comment above with all my latest changes. Some of these may be workarounds as I'm still learning the source code. My fixes do seem to work though.

Notice how TextBinding.unbind needs to re-create the original text content. I believe my solution may be the right solution.
As you'll notice, unbinding an if or unless needs to unbind all nested views as well. I'm guessing there may be a better way to do this but this seemed extremely necessary.
As you'll notice, I've introduced a View.rebind. This may seem hacky and it very well might be. I couldn't figure out another graceful way to solve this. But just like unbind, it seems very necessary that if and unless binders traverse their nestings during a rebind.

@jccazeaux, I believe your fix for #611 does seem to be working and definitely seems like a much better fix than my rebind function. It does seem like my implementation of TestBinding.unbind is still necessary which is what was originally throwing me off. Maybe its worthwhile to split this up into its own bug. Adding code snippet below for easy access.
    // ak: Fix for issue #618 - replace element text with original token
    TextBinding.prototype.unbind = function() {
      var _ref1, _ref2, _ref3, _results;
      _ref1 = this.view.templateDelimiters;
      _ref2 = this.options.dependencies ? ' < ' + this.options.dependencies.join(' ') : '';
      _ref3 = this.options.formatters.length ? ' | ' + this.options.formatters.join(' | ') : '';
      _results = TextBinding.__super__.unbind.apply(this, arguments);
      this.el.textContent = _ref1[0] + this.keypath + _ref2 + _ref3 + _ref1[1];
      return _results;
    };

Why would it be necessary to reset text content with original template code? Unbind's role is to shut down observers, not to reset template code. Am I missing something?

Please take a look at the following fiddle. How would you propose one go about resetting the TextBinding template in this case? Maybe TextBinding.unbind isn't the correct function for such an operation but resetting the TextBinding template seems necessary in this case.
https://jsfiddle.net/havessex/1/

Closing this issue as its considered a duplicate of #611. Opened #623 to address the issue with TextBinding.

source: gharchive/issue

repo: mikeric/rivets

license: mit

license_type: permissive

license_source: bigquery


text:
upsertMany should split to batch

Describe the bug

Stack trace
To Reproduce
upsertMany with a lot rows.
Expected behavior
Additional context
Versions

Dependency
Version

node
?

typescript
?

mikro-orm
?

your-driver
better-sqlite

What does "a lot" mean, how complex your entity is? Note that I can see from your screenshot that you don't await the upsert call.

Describe the bug

**Stack trace**
To Reproduce
upsertMany with a lot rows.
Expected behavior
Additional context
Versions
Dependency	Version
node	?
typescript	?
mikro-orm	?
your-driver	better-sqlite

https://antfu.me/posts/why-reproductions-are-required

for SQLIite SQLITE_MAX_COMPOUND_SELECT=500 (https://www.sqlite.org/limits.html), it's easy to rich the limit when using batch, maybe related to how sqlite is generated, I think mikro orm should support the automatic batch split or allowed pass a batch size options.
I think this error is common enough, it's easy to get the point.

authors: B4nan rubiin wenerme

repo: mikro-orm/mikro-orm

license: MIT

license_source: github-api


text:
Add onInit lifecycle hook

Fired after the entity instance was created, good place to fill your virtual/shadow properties:
@Entity()
export class Author {

  @Property()
  firstName: string;

  @Property()
  lastName: string;

  @Property({ persist: false })
  fullName: string;

  @OnInit()
  onInit() {
    this.fullName = `${this.firstName} ${this.lastName}`;
  }

}

Closing as resolved. Available for testing via next tag.


text:
Gradient transform

This provides the Grad transform on graphs. Call Grad().process_graph(graph) to obtain the transformed graph.
Current state: Grad outputs something that looks sensible for all graphs I've tested, including closures and graphs with if/while loops. I also tried up to sixth order gradient on a while loop and it didn't crash, which is encouraging (sixth order takes 30 seconds, so I stopped there). However, I didn't run any of these graphs (can't, since the necessary operations are not implemented), so I don't know if they are indeed correct (the graphs can be a bit hard to read, unfortunately).

Progress report.
I've implemented backpropagators for a few primitives and if, ported the finite difference code from master, and added a zeros_like primitive. Then I ported several tests from master.
The good news is that the algorithm seems to be correct: correct gradients are obtained on simple arithmetic, if statements, closures and nested while loops. Worked on first try, too. There are, however, a few issues:

Using the current VM, zeros_like can't get the shape of a closure's gradient, because that involves fetching all closure variables, and some of them will cause Continue exceptions to be raised. I used a simple workaround: zeros_like(closure) generates the ZERO object, which has __add__ and __radd__ overriden to return the other argument, and __getitem__ to return ZERO. That seems to do the trick, but I used the same trick in master (though more as an optimization) and I recall it causing a few issues with higher order gradients. So I don't trust that solution very much. Might be worth looking into the Env idea.

Tuples are made via the make_tuple primitive, which takes a variable number of arguments. This means it's impossible to build a graph for J(make_tuple) (so I can't test second order right now). I can see the following solutions:

Add varargs support. An inference pass could try to eliminate them for optimization.
Special case make_tuple in the gradient pass.
Generate J(fn, n) where n is the number of arguments passed to the function. This would let J specialize on the number of arguments, in which case a graph could be generated for make_tuple with n arguments.
Replace make_tuple(a, b, c) by the composition of a binary function, for example cons(a, cons(b, cons(c, ()))).

For now I'm trying the last option: replacing make_tuple by the composition of cons_tuple calls. Seems to be working so far, just need to scaffold the tests for second order gradients to make sure everything's in order.

The functionality should be complete, and the tests pass, but there are still a few things I want to improve, like notation and documentation.
Note that this PR includes the modifications in #35, #45 and #46. You should review these before this one.

I'm closing this PR and making a new one.

source: gharchive/pull-request

repo: mila-udem/myia


text:
Remove c prefixes

Feel free to close this if you have something ready for master yourself.
I just wanted to get these changes out of my diff for migrating to macro-compat.

I'll merge this now ... I have a couple of follow ups removing the use of Expr as well.

authors: dwijnand milessabin

repo: milessabin/shapeless


text:
Add implicit OrElse combinator

Spin off from milessabin/kittens#73

Like Option.orElse on the type level
Like Either on the value level

I only added fold and unify to OrElse, but other methods from Either might also be useful.

Codecov Report

Merging #791 into master will decrease coverage by 0.05%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #791      +/-   ##
==========================================
- Coverage   88.01%   87.95%   -0.06%
==========================================
  Files          63       64       +1
  Lines        1493     1494       +1
  Branches        5        4       -1
==========================================
  Hits         1314     1314
- Misses        179      180       +1

Impacted Files
Coverage Δ

core/src/main/scala/shapeless/orelse.scala
0% <0%> (ø)

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 3188e7d...890483d. Read the comment docs.

Why does codecov say 0% of diff hit? That can't be true.

The codecov result are pretty flaky IME. I've almost reached the point where I think it might just be better to remove it from the build.

Anyhow ... good stuff ... many thanks ... merging :-)

I think I may have made a mistake. I wanted to submit a version with strict Primary and lazy Secondary, but instead I submitted one where both are lazy. But I'm not sure which one is more useful in general.

authors: codecov-io joroKr21 milessabin


text:
Memory leak of create_string_buffer

create_string_buffer in ParcelThread is leaking, due to the loop in Server.listen.  The instance is created in a while loop that never exits, and create_string_buffer counts on being freed by the garbage collector.  But since the loop never exits, the reference is never released.

buffer is no longer allocated here.

authors: madopal millerjs

repo: millerjs/parcel


text:
Tuya Dehumidifier

I know you are working on it. When might climate devices be updated into the plugin?

Is your feature request related to a problem? Please describe.

Describe the solution you'd like

Describe alternatives you've considered

Additional context

Duplicate of #63
Please see my responses there.
No need to open a new issue for this.


text:
Where does it expect pdb to be?

Hello, and thanks for this amazing software!
My preferred workflow is: profiling an app on Windows machine, then downloading the MTuner report along with the pdb file, and running MTuner via Wine on a linux machine. Naturally I don't expect support of this scenario, but maybe I'm missing something easy?
The problem is that it doesn't load debug symbols, so all functions and locations appear as "unknown" in the GUI. I tried putting the pdb file (and, for the record, the exe one) in the same folder as the MTuner report and tried to point Local Store path to the directory with the pdb file, but it still doesn't seem to load them. What can I do?
Thanks!

It also says "file may be corrupt" in 2/3 of times, even though the app exits with code 0 and the mtuner file is 200mb - 2gb :(

Thanks for the quick answer. What can I do on my side to prevent files from corruption?

Oh. How to "force" load the file then? It doesn't show anything when it fails to load the file with this error

repo: milostosic/MTuner


text:
[skip e2e] Add benchmark ref logs for glove test

Signed-off-by: yudong.cai yudong.cai@zilliz.com

/kind improvement

/lgtm

authors: cydrain landyliu

repo: milvus-io/knowhere


text:
milvusdm error

when I execute milvusdm --yaml M2M.yaml i encounter an error.
2021-04-15 19:50:23,301 ｜ ERROR ｜ milvus_to_milvus.py ｜ transform_milvus_data ｜ 44 ｜ Error with: cannot reshape array of size 350208 into shape (171,64)2021-04-15 19:50:23,301 ｜ ERROR ｜
My milvus version is 1.0.0.

You can check the logs of Milvusdm, which is reading the data as float data.
According to the logs, you can confirm that the dimensions of the data and the number correspond to the collection.

Hi @shiyu22. I have kinda the same issue and here is my logs:
2021-04-21 17:37:25,603 ｜ INFO ｜ milvus_to_hdf5.py ｜ read_milvus_data ｜ 49 ｜ Ready to read all data of collection: mycollection/partitions: [None] 2021-04-21 17:37:25,603 ｜ DEBUG ｜ read_milvus_meta.py ｜ get_collection_info ｜ 71 ｜ Get collection info(dimension, index_file_size, metric_type, version):[(512, 33554432, 2, '1.0.0')] 2021-04-21 17:37:25,640 ｜ DEBUG ｜ read_milvus_data.py ｜ read_milvus_file ｜ 73 ｜ Reading milvus/db data from collection: mycollection/partition: None 2021-04-21 17:37:25,640 ｜ DEBUG ｜ read_milvus_meta.py ｜ get_collection_dim_type ｜ 95 ｜ Get meta data about dimension and types: [(512, 3)] 2021-04-21 17:37:25,641 ｜ DEBUG ｜ read_milvus_meta.py ｜ get_collection_segments_rows ｜ 108 ｜ Get meta data about segment and rows: [('1616691254686440000', 4384)] 2021-04-21 17:37:25,641 ｜ DEBUG ｜ read_milvus_data.py ｜ read_rv_binary_file ｜ 31 ｜ Reading binary data in local file: /home/-----/milvus/db/tables/mycollection/1616691254686440000/1616691254686440000.rv 2021-04-21 17:37:25,648 ｜ ERROR ｜ milvus_to_hdf5.py ｜ read_milvus_data ｜ 56 ｜ Error with: cannot reshape array of size 8978432 into shape (4384,64)
Not really clear what I should do here. Any advice?

Hi, Alexey @scipe,
Maybe there is an issue with reading binary data, you can find the code in read_milvus_data.py, the first 8 bytes is the size of the .rv file, so we truncated the data.
Could you please send me the file name '1616691254686440000/1616691254686440000.rv', and I can test the code on my machine? Of course, you can also modify the source code to test it yourself, and the method to follow again can be found in this issue with step 2.

authors: liaodaihai scipe shiyu22

repo: milvus-io/milvus-tools


text:
[Bug] Wrong output when inf is the end in 'mindspore.ops.linspace'

Environment
Hardware Environment(Ascend/GPU/CPU):
device gpu
Software Environment:

MindSpore version (source or binary): v2.2.13
Python version (e.g., Python 3.7.5): 3.8.18
OS platform and distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 18.04

Describe the current behavior
When running the following code, mindspore.ops.linspace get a different result from pytorch
Describe the expected behavior
tensor([nan, inf, nan, nan, nan])
Steps to reproduce the issue
import mindspore as mind
import numpy as np

output_tensor = mind.ops.linspace(start=0, end=float('inf'), steps=5)

# Print the output tensor
print("mindspore:", output_tensor)

#pytorch v2.0.0+cu117
import torch
import numpy as np

output_tensor = torch.linspace(start=0, end=float('inf'), steps=5)

# Print the output tensor
print("torch:", output_tensor)

when float('nan') is the end parameter, the output is different too.

repo: mindspore-ai/mindspore


text:
PXP-572: Handle blue/green deployment failure on deployment

Summary
Currently with an application that is having Blue-Green deployment topology, we don’t have an effective method to guarantee that the Blue deployment is running BEFORE sending the Green deployment however Mindvalley does not follow this practice yet so need additional handling to check for the deployment status, and ask for confirmation if there is any failure.

Ticket: https://mindvalley.atlassian.net/browse/PXP-572
What's Changed

[x]  Added additional confirmation if there is failure on last deployment

CASE 1: BLUE deployment is succeeded

CASE 2: BLUE deployment is not  succeeded

There are two things from this PR:

It doesn't cover the case of an application that doesn't have the blue version. If it's the case, the CLI simply throwing an error from the API like this

❯ ./target/debug/wukong deployment execute
Not detecting any flags, entering deployment terminal......
Current application: mv-wukong-api-proxy
✔ Step 1: Please choose the namespace you want to deploy · Prod
You've selected `Prod` as the deployment namespace.

✔ Step 2: Please choose the version you want to deploy · Green
You've selected `Green` as the deployment version.

✔ Step 3: Checking the status of the latest Blue deployment...
Error - API Response Error: cdPipeline:3:3: cd_pipelines_version_config_not_defined

We should handle this, and allow the deployment to proceed if there's no blue deployment

Also, we should have test case to cover this logic. If there's test case we will be able to detect the flaw in the logic.

Handle the case where by there is only one version of the deployment which ignores the check
For second point we still have the limitation of doing the test cases for interactive sessions

Step 2 suddenly jump to Step 4 in this case

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: jk-gan, mfauzaan
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: jk-gan, mfauzaan, onimsha
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

authors: jk-gan mfauzaan sauron-droid

repo: mindvalley/wukong-cli


text:
Incompatibility with Fresh Animations (1.20.1)

I'm using Fabric 1.20.1 and the latest versions of
• [ETF] Entity Texture Features - 4.5.0
• [EMF] Entity Model Features - 0.2.13
• Animatica - 0.6
• Armorful - 2.0.1
When I install your mod, all Illagers are broken and don't have features from Fresh Animations like eyes and model movements.
It looks like this:

Also some of Illagers from Illagers Invasion mod are broken too (legs and head).
I'd just like to know what can be done about this problem, because I like your mod.

Are you sure Animatica isn't causing the issue? I don't directly affect the rendering, only the armor layer renderer which shouldn't be affected by fresh animations

I just deleted Animatica and nothing has changed. The illagers look the same as in the image above.
By removing your mod, the animations work fine.
I also wanted to note that on Modrinth in the latest version of your mod for 1.20.1, the main file is the "sources" file, with which the game doesn't launch, somehow requiring Minecraft 1.20:

Hm, I'll republish it. Seems to have corrupted during upload

After some investigation, I found out that the illagers' models are broken because of the Illager Invasion mod.
I apologize for my misunderstanding.

I take back my words. I was wrong again.
Fresh Animations as intended works on Fabric together with Illagers Invasion correctly.
However, when installing your mod, models of vanilla illagers lose animations and eyes, and some of the Illagers Invasion models break too.

Potentially it's an illager invasion compat issue then?


text:
Digging chests with admin pickaxe causes crash related to nodeupdate

Newest maptools, newest 0.5.0-dev as of today. Digging locked chest with admin pickaxe gives this error:
ACTION[Server]: singleplayer takes maptools:pick_admin from creative inventory
ACTION[Server]: singleplayer digs default:chest_locked at (31,17,24) using an Admin Pickaxe.
WARNING[Server]: Undeclared global variable "nodeupdate" accessed at D:\Newprog\minetest\bin\..\mods\maptools/tools.lua:59
ERROR[Main]: ServerError: AsyncErr: ServerThread::run Lua: Runtime error from mod 'default' in callback node_on_punch(): D:\Newprog\minetest\bin\..\mods\maptools/tools.lua:59: attempt to call global 'nodeupdate' (a nil value)
ERROR[Main]: stack traceback:
ERROR[Main]: 	D:\Newprog\minetest\bin\..\mods\maptools/tools.lua:59: in function 'callback'
ERROR[Main]: 	D:\Newprog\minetest\bin\..\builtin\game\item.lua:487: in function <D:\Newprog\minetest\bin\..\builtin\game\item.lua:480>

P.S. IIRC nodeupdate function is deprecated in 0.5.0

#7

I have this problem in 5.0.0 with any node

Try updating this mod.

This bug is fixed for me, thank you.

repo: minetest-mods/maptools

license: Zlib


text:
🛑 MineVN Website is down

In c908a13, MineVN Website (http://minevn.net) was down:

HTTP code: 0
Response time: 0 ms

Resolved: MineVN Website is back up in 8b35d8e.


text:
🛑 MineVN Maven & Resource Pack is down

In 9feeb40, MineVN Maven & Resource Pack (http://app.minevn.net/pack) was down:

HTTP code: 0
Response time: 0 ms

Resolved: MineVN Maven & Resource Pack is back up in b48c88e.


text:
telnet: Unable to connect to remote host: Connection refused

I've tried telnet localhost bgpd in terminal, xterm, and the mn example sudo mn, all of them succeeded.
However, when I ran this command on a project that was slightly modified based on BGP Path Hijacking Attack Demo, the following error occurred:

Connecting to shell
Trying 127.0.0.1...
telnet: Unable to connect to remote host: Connection refused

Then I ran pingall, which succeeded in the mn example, to get things below:

*** Ping: testing ping reachability
h1-1 -> X X X X X X X X X X X
h1-2 -> X X X X X X X X X X X
h1-3 -> X X X X X X X X X X X
h2-1 -> X X X X X X X X X X X
h2-2 -> X X X X X X X X X X X
h2-3 -> X X X X X X X X X X X
h3-1 -> X X X X X X X X h4-1 X X
h3-2 -> X X X X X X X X X h4-2 X
h3-3 -> X X X X X X X X X X h4-3
h4-1 -> X X X X X X h3-1 X X X X
h4-2 -> X X X X X X X h3-2 X X X
h4-3 -> X X X X X X X X h3-3 X X
*** Results: 95% dropped (6/132 received)

Did you debug this further to see what is up with the BGP hijacking demo?

authors: lantz weWillGetThere

repo: mininet/mininet


text:
Fix region setting AWS SDK for .NET example

I tried to use MinIO with the AWS SDK for .NET and got the same error descripted on this post:
https://stackoverflow.com/questions/68678953/error-when-configuring-minio-in-c-sharp-the-authorization-header-is-malformed/69199735
So I found that the solution was to change the RegionEndpoint setting to AuthenticationRegion, as the region is used on the authentication process.
Also the AWS SDK sets the RegionEndpoint as null when ServiceURL is set.

Hi @ferhrosa - sorry, I just noticed this pull (not sure why I didn't get a notification for this).
Let me see if I can get someone internally to double check this, I don't have enough insight into .NET to validate the fix.

repo: minio/docs


text:
Added hmpps auth mock for SP user pact tests.

Added hmpps auth mock for SP user pact tests. Hmpps.getUserGroups now returns HAPPY_LIVING as default

🙋♂ I think the pull request template would be useful for anyone reading this later -- when we have a bit of time, could we fill the description according to that, please?

authors: andrewburrowsmoj sldblog


text:
Failure to upgrade sidekiq to version 6

During the upgrade to Rails 6 we defaulted sidekiq to version 5.7.2 because version 6 requires zeitwerk (the Rails 6 default autoloader) which we don't use.
more information here:
https://github.com/ministryofjustice/laa-apply-for-legal-aid/pull/1240

Sidekiq is currently at 6.1.2 and appears to be working fine with both Zeitwerk and Classic autoloaders. Closing this issue.


text:
Stop or limit audit logs

Audit logs are generating a lot of data which is causing a slowdown in the ELK cluster. Opening this issue as one of the suggestions was to stop shipping audit logs.
https://github.com/ministryofjustice/logstash-formula/blob/master/logstash/client.sls#L9

This is needed, idea would be to allow pillar enabling disabling of components, which should be simple to implement
something like the below (a rough gist)
client.sls:
# Default enabled components
{{ if pillar.get("logstash:salt:enabled", true }}
{{ logship('salt-minion.log', '/var/log/salt/minion', 'salt', ['salt','salt-minion','log'], 'json') }}
{% endif %}
# Optional components default to falise
{{ if pillar.get("logstash:auditd:enabled", false }}
{{ logship('audit.log', '/var/log/audit/audit.log', 'audit', ['audit'], 'json') }}
{% endif %}

Pillar entries similar to the below could then be used
staging.sls:
logstash:
    auditd: true

Closed by #83. Opening https://github.com/ministryofjustice/platforms-team/issues/102 as the process of actually clearing out all users of auditd shipping

repo: ministryofjustice/logstash-formula


text:
Glue job create table

Added functionality to glue job so that if an extraction timestamp partition of data is found to already exist the script also checks if the database and table exists and creates if it does not.
As well as the glue job python script changes were needed to:

the glue job infra terraform to give athena start query execution permissions to the iam role
the athena query bucket, to drop the timestamp suffix so we can predict the name of the bucket when deployed to accounts other than the dev sandbox

Fixes ministryofjustice/data-platform#367

Worth a sanity check if you would like to restrict Athena access to a region and to a pattern workgroup name.

Probably Julia but i think this might come at a later date if this becomes an agreed thing rather than a PoC. We would then know more about what a workgroup for athena to query data products would be called if it were to have a dedicated one.


text:
Unit Tests for Cloudtrail/Support modules in Baselines

User Story
As a MP Engineer
I need to write some unit tests for the Cloudtrail and Support sub-modules sin the Baselines module
So that I can increase the confidence in the modules
Value / Purpose
As a follow-on from https://github.com/ministryofjustice/modernisation-platform/issues/6490 this ticket is to add missing unit tests for some of the sub-modules in the MP Baselines module.
There are roughly 9 resources for unit testing.
Useful Contacts
@richgreen-moj
Additional Information
Follow-on from https://github.com/ministryofjustice/modernisation-platform/issues/6490
Proposal / Unknowns
No response
Definition of Done

[ ] Missing unit tests added
[ ]  New unit tests pass
[ ]  Another team member has reviewed

cloudtrail ✅ - write a follow-up story to test (8 resources) https://github.com/ministryofjustice/modernisation-platform/issues/7307
support ✅ - write a follow-up story to test (1 resource) https://github.com/ministryofjustice/modernisation-platform/issues/7307
These are the resources we need to test.

I'm picking up this one as I've not done much with unit tests & go before. If anyone wants to pair with me on it then do shout. :)

Branch containing the completed support module unit test - https://github.com/ministryofjustice/modernisation-platform-terraform-baselines/tree/issue/7307

Added unit tests for the Support and CloudTrail modules.
https://github.com/ministryofjustice/modernisation-platform-terraform-baselines/pull/547

just checked and the unit test is failing

Go issue resolved with PR https://github.com/ministryofjustice/modernisation-platform-terraform-baselines/pull/551

All looks good

authors: ep-93 markgov mikereiddigital richgreen-moj sukeshreddyg


text:
LAA data platform sandbox

Environment details
Application Name

LAA data platform sandbox
Github team slug

laa-technical-architects
Environments

[x] Development
[ ] Test
[ ] Preproduction
[x] Production

Tags

tag
value

application
LAA data platform sandbox

business-unit
LAA

owner
donovan.artz@digital.justice.gov.uk

Networking options
Connectivity

would it be possible to access cloudwatch data in CP?
also, would like to expose API endpoint here to applications in LAA's AWS VPCs?
Additional features

[x] Bastion
[x] Additional VPC Endpoints
[ ] Extended DNS Zones

Other information

Closing for now, LAA to use their existing sandpit environment for the time being.

authors: davidkelliott donovan-moj


text:
NACS - SPIKE - Policy Not Set

What: A request from LAN to investigate the policy not set feature on the NACS portal
User Story:
Value / Purpose

Agreed with Matti to update the documentation listed here to cover the full workings of the policy priority system.
https://dsdmoj.atlassian.net/wiki/spaces/NAS/pages/3789324291/Admin+Portal+User+Guide#Policies

Updated the Policy Priority hint on the NACS Portal on the following PR - https://github.com/ministryofjustice/network-access-control-admin/pull/560

repo: ministryofjustice/nvvs-devops


text:
SPIKE: PagerDuty membership in IaC

Background
It is possible with a terraform integration to handle PagerDuty setting in IaC as seen on this repo. This ticket is to investigate the feasibility of using this for the ops-eng team for our PagerDuty account and if it then can be automated in some way. Seek Jacob Woffenden for more information on their roll out.
Proposed user journey
Check out the Terraform PagerDuty integration, see if it is a good use case for adding/removing users from PagerDuty, what steps are needed, pros / cons of using this over manually adding/removing users, would this conflict if other teams are adding/removing users in the IaC setup or manually. What can we automate using this feature

Check out the Terraform PagerDuty integration:

https://github.com/PagerDuty/terraform-provider-pagerduty
https://registry.terraform.io/providers/PagerDuty/pagerduty/latest/docs
The integration will add / remove users from PagerDuty that are in its users.tf file only.
Data platform team have implemented their users and team in IaC via this repo https://github.com/ministryofjustice/data-platform

See if it is a good use case for adding/removing users from PagerDuty:
It can be if we want to track users via IaC, but it it is quicker to go into PagerDuty and manually add/remove users.
What steps are needed:
Create new repo, setup code owner and branch protection rules, get the terraform files / create project structure similar to data platform setup, set up S3 and DynamoDB in our AWS or CP AWS account to store the state and state lock file, get all the current names and emails from the users in pager duty, add these details to the users.tf file, set up GH action for tf plan and tf apply, get API tokens set up in PagerDuty and add as repo secrets for the GH Actions. Set up Dependabot to update gh action and terraform versions, create a script that will read the users.tf file to compare against PagerDuty API to detect newly added or removed users as other teams may have added / removed users to PagerDuty outside of our process, get the script to update the file with the changes, create a branch, commit and raise a PR. Create a repo create PR from issue script whose input will be the users email and name that will auto update the file and create a new PR. Create another create PR from issue script to remove user based on email address. Add unit tests for the script code.
Positives / Negatives of using this over manually adding/removing users:
Positives:

Have a single file that is a record of who is a user in PagerDuty (if implement scripts to manage the drift in users)

Negatives:

Another repo to manage, update, maintain, etc
More S3 / DynamoDB either in our AWS or CP AWS account
Quicker to open PagerDuty to add / remove a user
Puts all the names and email addresses of the users in a public view repository (Data platform do not seem to mind about this) (this includes external users and personal email addresses)
If a user is removed by another team team, do we need to contact them to see if that was meant to happen before we remove them from our users.tf file, at the moment we wouldn’t even know / care.
There are multiple owner / admins who could invite users to PagerDuty so cannot control who is added / removed via IaC
Wouldn’t know how many seats are being used / available without going into the tool and checking
If other teams are adding users and we are as well could we exceed the available seats

Would this conflict if other teams are adding/removing users in the IaC setup or manually:
Yes and No, the Terraform integration will only consider the users in its own state via the users.tf file. This means Data-platform can add/remove users in addition to our IaC. The issue is when a user is defined in more than one IaC repository. There can be a condition where a user is added or removed from PagerDuty in our repository then removed or added back onto PagerDuty by the IaC of another repository ie Data-platform, the two repositories would need to be aligned to make this work correctly. There could well be other teams in the future that want to use Terraform integration with PagerDuty that could increase the issue just described.
What can we automate using this feature:

By adding additional code and the Create a PR from Issue we can automate adding and removing a user from the users.tf file to reduce manually editing the file and creating PR's
Future automation would be able to edit the users.tf file directly.

Side Notes:

In theory future automation could just call the same API the PagerDuty Terraform integration is calling.
If we move adding / removing users from PagerDuty from the Ops-Eng team to an IT help desk team we do not even need any automation?
But then we have HelpDesk and project teams adding users manually and via IaC, do or would we need to keep an eye on this?
By handing over the PagerDuty to Helpdesk it will free up the seats used by the Operations Engineering team.
Servicedesk can manage PagerDuty API keys, but will they audit them?
There are some very old API keys in PagerDuty!
We currently have 196 users, we try to keep on top of the users, but if user management went to the HelpDesk team would they conduct an audit to see if these users still exist or not / require access to PagerDuty?
Some PagerDuty users are external or using a personal email account, so we have no idea if that user still requires access to PagerDuty or not.


text:
Note-take with reactions as well as pinning items

Currently Response will copy any pinned items in a comms channel to the incident details. This allows us to keep a record of the important stuff but jettison the chatter.
However, pinning is an awkward process which quickly clutters a channel. It would be nicer if we could mark important content with the :memo: emoji reaction.

Changed to "as well as". Having both might turn out to be confusing, but there's no point trying to remove existing functionality from Response.

repo: ministryofjustice/opg-incident-response


text:
Review after date expiry is upcoming for user: richrace

Hi there
The user @richrace has its access for this repository maintained in code here:
https://github.com/ministryofjustice/github-collaborators/blob/main/README.md#github-external-collaborators
The review_after date is due to expire within one month, please update this via a PR if they still require access.
If you have any questions, please post in #ask-operations-engineering on Slack.

user removed.

authors: ASTRobinson AntonyBishop


text:
데릭 / 10주차 / 1문제 / [티어전 상대: 웡빙]

🧑🏫 수행내역
🧑💻 개인문제풀이 1문제

[PGS] 기능 개발 / Level 2 / 1시간

🤼 티어전

[티어전][데릭, 🏅웡빙][PGS] 모의고사 / Level1 / 40분

✏ 기타사항

고생하셨습니다 ☺

repo: minsson/algorithm-study-in-swift


text:
🛑 Matrix is down

In c57c69d, Matrix (https://matrix.mint.lgbt:8448/_matrix/federation/v1/version) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Matrix is back up in 7cb3f86 after 7 minutes.

repo: mint-lgbt/status


text:
Smithing is broken on 1.2

this mod is not compatible with smithing changes brought with the beta 1.2 branch of Banner Lords, causing instant crashes when entering smithy...

Confirmed instant crash on entering smithy.  Happens even if smithing changes are turned off and all other mods are turned off too.

Also I unblocked everything using powershell and always run as admin.
I confirmed that stopping the mod stops the issue.  Will keep it on because I love the tweaks and I hate crafting anyway.

LIES!  I figured out it was the "Hide Locked items" option that caused the crash. Without it on, every other smithing option works fine.

So I tried that fix I even deleted every file related the bannerlord tweaks and reinstalled them then manually changed the JSON file in the configs folding for the game in "My Documents" and I still get the instant crash. no matter what fixes I try it still crashes on entry. I have confirmed that no matter what setting I have enabled or disabled the game still crashes. I even have a separate mod I used to use that is a stand alone hide unknown parts that I reinstalled and tried again. with bannerlord tweaks off no crash and all unknowns hidden perfectly, with bannerlord tweaks = instant crash. if anyone else has this issue let me know.

And you definitely unchecked "Hide Locked Items" in the mod options?  I only say that - not to frustrate you or talk down to you - because I felt like a total idiot when I turned off everything in the "crafting options" but didn't notice the "smithing options" all the way down.
Regardless, good luck bro.

Yes, sadly. And no fear of talking down I ain't easily offended and deal
with nonsense like this a lot, but yes I went down to the smithing options
and I unchecked both of them. I even after posting deleted all bannerlord
tweaks files and started from scratch. Same thing.
I even made sure that the settings file was correct. I went to "my
documents" then into bannerlord
folder>configs>modsettings>zzbannerlordtweaks, and I opened the
bannerlordtweakssettings.JSON file which is where the settings you change
in-game are saved and why you have to close down and reboot each time you
make a change. And I made sure that file said "false" hiding locked items.
It's baffling so I am just doing without smithing for now I guess.
On Wed, Apr 22, 2020, 1:27 PM bonerstorm69 notifications@github.com wrote:

And you definitely unchecked "Hide Locked Items" in the mod options? I
only say that - not to frustrate you or talk down to you - because I felt
like a total idiot when I turned off everything in the "crafting options"
but didn't notice the "smithing options" all the way down.
Regardless, good luck bro.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mipen/BannerlordTweaks/issues/67#issuecomment-617983284,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APJRAZ74SFEL2RJM2S753O3RN5ALLANCNFSM4MNQTOIA
.

Just thought i'd contribute to the discussion on here that upon reading the mod option fix, to uncheck "Hide Locked Items" I tried this and only this and Smithing starting working again.
This was a new save I made today with mods I had just installed, but I doubt that has anything to do with. I suppose it means it's "fresh" though. Everything is default for me basically.
A temporary fix would be to make that option not default.
But obviously it would be nice to have it enabled for those who want it. I don't think i'll use it either way, locking items is seemingly useless right now in the base game, i'm sure that's going to change in the future so we'll be prone to more bugs like this.

Yeah I agree I don't really need that feature. I just stash anything I
truly want to keep. It's all the other smithing feature I miss. I got to
the point on one character I could make some sweet swords. Now if I want to
make an awesome sword. I gotta log. Disable banner tweaks, then log back in
Smith, then log, then log back in with banner tweaks active... Tedious and
inconvenient but oh well.
On Wed, Apr 22, 2020, 4:07 PM RyanWilson-byte notifications@github.com
wrote:

Just thought i'd contribute to the discussion on here that upon reading
the mod option fix, to uncheck "Hide Locked Items" I tried this and only
this and Smithing starting working again.
This was a new save I made today with mods I had just installed, but I
doubt that has anything to do with. I suppose it means it's "fresh" though.
Everything is default for me basically.
A temporary fix would be to make that option not default.
But obviously it would be nice to have it enabled for those who want it. I
don't think i'll use it either way, locking items is seemingly useless
right now in the base game, i'm sure that's going to change in the future
so we'll be prone to more bugs like this.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mipen/BannerlordTweaks/issues/67#issuecomment-618065513,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APJRAZ7VVLPOLKVX2G3QYHTRN5TARANCNFSM4MNQTOIA
.

Pull request from joshimoo might be fixing the issue.
https://github.com/mipen/BannerlordTweaks/pull/61

Yeah the patch does fix crafting but I am currently on the 1.2 beta branch and have not tested it below that. Will update it on the weekend when taleworlds releases the office 1.2.
Hopefully @mipen will then merge it?

Not sure if this was reported yet but it looks like either the smithing code or experience code is causing a max increase in smithing skill when refining steel and or iron.  After the exp is gained you have max smithing skill (level 1023) with an extra skill point that can't be assigned.
I don't have either the parts or hide options checked.  Only have extra stamina.
Game Version e1.2.1.227199 (Current public build, not dev branch)

@Spar10Tech it's unrelated to the smithing patch and probably related to the experience patch.

I was able to verify that with all the experience options disabled in the config the max skill event does not happen.
Also just wanted to mention the smithing parts learning seems to have increased in the vanilla build as you work with better materials.

Yeah they really boosted smithing learning. Smithing is so broken though. I
can now craft swords that are mediocre and have negative ticks because my
smithing is like 30 and the skill required is like 90 and still sell them
for 4-6k easily. All it takes is a one/two hand sword that has long reach
and high dmg and you basically have infinite money. Only takes a couple
steel and a few iron and boom 5k... Then in late game I can make swords
with fine steel that sell easily in the 30k range.
On Wed, Apr 29, 2020, 8:59 AM Spar10Tech notifications@github.com wrote:

I was able to verify that with all the experience options disabled in the
config the max skill event does not happen.
Also just wanted to mention the smithing parts learning seems to have
increased in the vanilla build as you work with better materials.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mipen/BannerlordTweaks/issues/67#issuecomment-621267356,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APJRAZ3CN7J5AVXXB6PHTVDRPA6FZANCNFSM4MNQTOIA
.

You should try polearms, warsword something (tier 5) blade + tier 4 stick (regular hardwoord), 65k easy money, 180 dmg, 165 distance. You feel like a death reapper with the massive scythe :D

Lol yeah I had a mod that expanded the crafting options on one game, it was
called more smithing parts or something and I made these javelins that were
130 long, 128 Pierce dmg, and 29 speed. It was so broken. They took 3 fine
steel and 1 wood... Sold for 69k.... One shot kills no matter where you hit
someone. I had to get rid of that mod it broke the game for me.
On Wed, Apr 29, 2020, 9:23 AM Joshua Moody notifications@github.com wrote:

You should try polearms, warsword something (tier 5) blade + tier 4 stick
(regular hardwoord), 65k easy money, 180 dmg, 165 distance. You feel like a
death reapper with the massive scythe :D
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mipen/BannerlordTweaks/issues/67#issuecomment-621283277,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APJRAZ3XH4LG77AZNHAQF23RPBBA7ANCNFSM4MNQTOIA
.

Hey. I tried the untick-smelting-option fix and it didn't work on a save file I started today.

Dammit.  As of today's update from TW, the "Unlock parts from smelted weapons" option also causes Smithy to freeze up.  No crash after unchecking that option. This is in addition to the freeze from "Hide locked items."
That's sad... it was one of my favorite tweaks. Do the devs do posts anywhere about these stealth updates???

I tried the unchecking the "Unlock parts from smelted weapons" box and that didn't work for me. But to be honest I'm not that fussed about troubleshooting or getting it working, I think I've played enough M&B recently.

Lol yeah I feel you. I dropped like 90 hours on it... Now I can take a
break, read the patch notes to stay updated on content etc... And jump back
in later this summer (or winter for you southern hemispherians).
On Thu, Apr 30, 2020, 9:13 AM Graeme notifications@github.com wrote:

I tried the unchecking the "Unlock parts from smelted weapons" box and
that didn't work for me. But to be honest I'm not that fussed about
troubleshooting or getting it working, I think I've played enough M&B
recently.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mipen/BannerlordTweaks/issues/67#issuecomment-621917682,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APJRAZ2UD5SGCCPWQ2TL2YDRPGISPANCNFSM4MNQTOIA
.

Aaaand I've got the 1.1.7 version of BannerLord tweaks but it still crashes on entering smithy with any smelting options enabled =_(

Unticking both options as mentioned fixed the crash for me on e1.2.1 (BLT 1.1.7). Might have been just one of them that caused it, but I unticked both just because.

Please update to 1.3.4 if you are on bannerlord 1.3
We will shortly release 1.4.0 for the new bannerlord 1.4 beta :)
The smithing tweaks should be working correctly in both 1.3.4 or 1.4.

authors: Imsvale MarcLacharite RyanWilson-byte Spar10Tech Strobelit Vispilio bonerstorm69 joshimoo vonenasty86

repo: mipen/BannerlordTweaks


text:
根据子集Mip-img计算父级宽度

添加根据子集img宽度和个数，得到父级宽度

想问一下，这个组件的背景需求是怎样的，为什么会需要计算宽度呢

我们公司的一个M站 m.diyiyou.com里面的任意一个游戏下载的详情页面里面有一个地方是展示游戏图片的地方，它的设计是左右滑动的，因为是会溢出，所以光Css就满足不了需求了，就需要通过子集获取到总宽度然后给父级，然后父级加overflow-y:auto,达到滑动效果，(这是我第一次开发组件，很多地方不懂，代码可能不完善，后期维护修改方便不)

能给一个线上页面看下效果吗

http://m.cr173.com/x/199431 我理解是不是和这个页面的『详情』里的图片滑动似的呢？他这里没有用组件，你可以参考一下

http://m.muzisoft.com/mip/soft/237579.html 或者这个页面

好吧，谢谢你，我知道怎么处理了，对了，我这个代码有没有什么问题，才接触MIP，后期肯定需要开发组件的，想了解下怎样才能顺利通过

根据官网开发文档：https://www.mipengine.org/doc/2-tech/4-mip-widget.html
代码要符合fecs规范
我看你的代码基本上是OK的，customElem.prototype.build这里不要用build函数，用createdCallback更合适

既然这个pr 不要了，closed吧

repo: mipengine/mip-extensions


text:
error in dijkstra() function

When I set the $source = "F" and $target = "E" to dijkstra() function, the function return 5 and path is F-E. The true result must 4 and path is F-D-E

Thanks a lot @NgVanDuy . fixed the bug. its working fine now. also sorry for late reply, just saw your message today.

authors: NgVanDuy mirahman


text:
Settings to enable/disable add to cart

Is there a settings that will allow disabling/enabling the add to cart button? I know I can set the available stock to zero to do this. But then again, I don't want to show the product as out of stock.
Use-case for this is when you want to show your customers your upcoming product and its variants. Another is having your saleor act as a product catalog.

There is no such thing currently, as you said, the only way to achieve that is by decreasing the stock quantity to zero. Actually, I think that this is quite an interesting feature. I'm not sure if or how other e-commerce platforms support it, but we could think of adding some kind of "preview" for customers.

authors: kwatog maarcingebala

repo: mirumee/saleor


text:
Add GraphQL query for fetching staffNotificationRecipients

What I'm trying to achieve
Currently the only way to manage which staff receive emails is through the GraphQL queries, as there is no UI for this on the dashboard. However, once staffNotificationRecipients are created, there is no way to obtain their IDs to use in other mutations such as updating or deleting them.
Describe a proposed solution
Add a GraphQL query to fetch staffNotificationRecipients.

There is a field that returns this information in the shop query. You can use it like so:
{
  shop {
    staffNotificationRecipients {
      email
      id
      active
      user {
        email
      }
    }
  }
}

Ah I completely missed that. Thanks very much.

authors: khalibloo maarcingebala


text:
Product repetitions in homepage featured products

On Home page FEATURED products, I'm getting multiple instances of the same product (number of instances seems to match the number of product variants). I think it is the issue with the backend because the Graphql query returns the same product multiple times.
Screenshots

System information
Saleor platform (docker)
storefront 44940260295f1fd495036c237fb3ddb57407aef8
api 053173ccc72c9d03d6f707fe78a8112940c1cf01

Hey, thanks for opening the issue, I've noticed this bug already a few days ago - I think it's related to data loaders which we added to optimize the homepage database queries. I suppose there is a missing distinct somewhere in the dataloader implementation that causes the issue.

example query presenting bug:
query FeaturedProducts {
    shop {
      homepageCollection {
        id
        products(first: 20) {
          edges {
            node {
              id
              name
              category {
                id
                name
              }
            }
          }
        }
      }
    }
  }

As you said @maarcingebala the bug was caused by missing distinct but not in the dataloader implementation. I found it in ProductsQueryset after some debugging (preceded by binary search in working and nonworking commits :smile:). Please take a look at PR  #5892

authors: maarcingebala przlada


text:
Fix invisible buttons

Fixes bug which appeared after today's dashboard's update, causing some buttons to become invisible when hovered.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1294   +/-   ##
=======================================
  Coverage   70.37%   70.37%
=======================================
  Files         117      117
  Lines        6158     6158
  Branches      786      786
=======================================
  Hits         4334     4334
  Misses       1661     1661
  Partials      163      163

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 9b70aa1...0e4b22e. Read the comment docs.

authors: codecov-io dominik-zeglen


text:
Migrate to the new Sentry client

sentry-sdk is now the recommended Sentry client for Python.
Pull Request Checklist

[ ] Privileged views and APIs are guarded by proper permission checks.
[ ] All visible strings are translated with proper context.
[ ] All data-formatting is locale-aware (dates, numbers, and so on).
[ ] Database queries are optimized and the number of queries is constant.
[ ] Database migration files are up to date.
[ ] The changes are tested.
[ ] The code is documented (docstrings, project documentation).
[ ] GraphQL schema and type definitions are up to date.
[ ] Changes are mentioned in the changelog.

Here is the report for 8b702026ca752d7fba266625cc7a891fb80cc825 (mirumee/saleor @ update-sentry-configuration)
Base comparison is 3a50f4e5dadeaceb7706a8b01e7f1c754ecdd81c.
No differences were found. (click me)

# api.benchmark checkout
  test name                           	left count 	right count	duplicate count
  ------------------------------------	-----------	-----------	---------------
  add billing address to checkout     	         34	         34	             20
  add shipping to checkout            	          7	          7	              0
  checkout payment charge             	         14	         14	              0
  complete checkout                   	          6	          6	              0
  create checkout                     	         48	         48	             24

# api.benchmark homepage
  test name                           	left count 	right count	duplicate count
  ------------------------------------	-----------	-----------	---------------
  retrieve main menu                  	          5	          5	              0
  retrieve product list               	          4	          4	              0
  retrieve secondary menu             	          5	          5	              0
  retrieve shop                       	          2	          2	              0

# api.benchmark product
  test name                           	left count 	right count	duplicate count
  ------------------------------------	-----------	-----------	---------------
  product details                     	         13	         13	              3

# api.benchmark variant
  test name                           	left count 	right count	duplicate count
  ------------------------------------	-----------	-----------	---------------
  retrieve variant list               	          9	          9	              2

authors: django-queries maarcingebala


text:
Icons not displayed properly in IntelliSense

The property and method / function icons displayed in IntelliSense are the ones from the standard product theme, but not every time. It seems to occur only when I go 1 level deep on a chain (eg. obj.prop1), one more level and it breaks (eg. obj.prop1.prop2), sometimes it's not even one level - awkward, I know. I may be reading too much into it or maybe I'm missing something? Is it supposed to work like that?
Expected behavior
New product icons show up everywhere in IntelliSense.
Screenshots

Also, I have not seen any new icon replacing the blue property icon. It always shows up like this.

Is there no new icon or is it a bug?
Great theme btw, man. 🎉

Ah, there must be other symbols that are using the same icon that I missed. I can fix that.
And right now it's only those two icons, I haven't gotten around to adding the rest but that's coming soon. Thanks for the feedback!

Looks awesome! Great work!

authors: davidmszrs misolori

repo: misolori/vscode-fluent-icons


text:
🛑 Misskey.gg POST test is down

In be5f1e0, Misskey.gg POST test (https://misskey.gg/api/get-online-users-count) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Misskey.gg POST test is back up in 993008e after 18 hours, 15 minutes.

repo: miss-key/uptime


text:
Mot de passe perdu devrait etre case insensitive

si l'utilisateur réinitialise son mot de passe et que l'email n'est pas exactement celui en base (qui est toujours en minuscule), aucun email n'est envoyé.

PR https://github.com/mission-apprentissage/flux-retour-cfas/pull/2582

re7 ok

authors: Anne-Camille svengau

repo: mission-apprentissage/flux-retour-cfas


text:
fix(server): create fiabilisation mapping using referentiel api

Fix du job de création de la table de mapping fiabilisation en utilisant l'API du référentiel au lieu de la collection legacy

:tada: This PR is included in version 3.2.0-beta.6 :tada:
The release is available on:

GitHub release
v3.2.0-beta.6

Your semantic-release bot :package::rocket:

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

GitHub release
v3.2.0

Your semantic-release bot :package::rocket:

authors: antoinebigard sbenfares


text:
🛑 Catalogue API is down

In a6970b4, Catalogue API (https://catalogue.apprentissage.beta.gouv.fr/api) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Catalogue API is back up in e0a9e87.

repo: mission-apprentissage/upptime


text:
[Feature Request] Variable to file

See: https://www.reddit.com/r/neovim/comments/1evafhr/comment/lis68mi/

Can be achieved via scripts now: https://kulala.mwco.app/docs/scripts/overview/


text:
make cmd non-blocking

Addresses #41

I also tested this and seems like a good solution.
But the only problem so far is that if the API responds too long, it immediately shows the results of an old request

authors: coltorl yujinyuz


text:
Disallow duplicate keys in Blockly editor

Describe the bug
Dictionary in the blocks editor allows duplicated keys which could be possibly avoided by the blocks editor itself. This can create confusion among newcomers.

Affects

[ ] Designer
[x] Blocks editor
[ ] Companion
[ ] Compiled apps
[ ] Buildserver
[ ] Debugging
[x] Other... (please describe)

Other: Documentation should be updated warning users about the same.
Expected behaviour
The current behaviour is as expected.
Steps to reproduce
Shown above.

The current behavior follows what Python does:

x = {'a':1, 'b':2, 'a':3}
x
{'a': 3, 'b': 2}

It might be better to remain "bug compatible", especially if we're thinking about someday
interacting with Python libraries.

There can be a case where there is a typo causes two keys to match exactly and hence, may result in unexpected behaviour.
IntelliJ shows a warning

Similarly, it'd be great if we could identify little mistakes in the blocks statically and report it to the user for them to take further action. This is particularly useful for big projects and where the user overlooks the blocks and couldn't pinpoint the source of the error at runtime.

Making this a warning could be useful. Documentation improvement is fine. I do think that more analysis would be better. It could make for a good GSOC project given that adding new warnings is very incremental so we just do as many as we can. We might want to create an issue to track common problems we would like to report to the user.
Sent from my iPhone

On Dec 19, 2020, at 08:26, Pavitra Golchha notifications@github.com wrote:

There can be a case where there is a typo causes two keys to match exactly and hence, may result in unexpected behaviour.
IntelliJ shows a warning
Similarly, it'd be great if we could identify little mistakes in the blocks statically and report it to the user for them to take further action. This is particularly useful for big projects and where the user overlooks the blocks and couldn't pinpoint the source of the error at runtime.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Evan's idea sounds good.

Making this a warning could be useful. Documentation improvement is fine. I do think that more analysis would be better. It could make for a good GSOC project given that adding new warnings is very incremental so we just do as many as we can. We might want to create an issue to track common problems we would like to report to the user.

I think it would be better to prevent the user from connecting blocks this way, rather than displaying a warning. In my opinion, one of the coolest things about blocks languages is that you can prevent users from making static semantic errors before they even occur. This not only prevents users' code from ever being in a bad state, but it also allows them to learn what is and is not allowed with minimal frustration. We've already taught them (through type checking) that if something connects, it is valid, and if something doesn't connect, it is invalid. If instead we allow something to connect, and then show a warning to say this it is invalid, that is different than current behavior and causes frustration.
That being said, one thing I do like about warnings is that they can be used to provide additional information to the user about /why/ something is bad. For example, in this case you might have a warning like "Dictionaries cannot have duplicate keys, because then the correct value is unknown. See [[blah blah blah]] for more information".

Good thought by Beka.
Keep in mind that this can be done only when the keys can be checked
statically, e.g., when they are literals.
== Hal
Hal Abelson
hal@mit.edu
Prof. of Comp. Sci. and Eng.
MIT Dept. of Elec. Eng. and Comp. Sci.
On Sat, Dec 19, 2020 at 11:13 AM Beka Westberg notifications@github.com
wrote:

Making this a warning could be useful. Documentation improvement is fine.
I do think that more analysis would be better. It could make for a good
GSOC project given that adding new warnings is very incremental so we just
do as many as we can. We might want to create an issue to track common
problems we would like to report to the user.
I think it would be better to prevent the user from connecting blocks this
way, rather than displaying a warning. In my opinion, one of the coolest
things about blocks languages is that you can prevent users from making
static semantic errors before they even occur. This not only prevents
users' code from ever being in a bad state, but it also allows them to
learn what is and is not allowed with minimal frustration. We've already
taught them (through type checking) that if something connects, it is
valid, and if something doesn't connect, it is invalid. If instead we allow
something to connect, and then show a warning to say this it is invalid,
that is different than current behavior and causes frustration.
That being said, one thing I do like about warnings is that they can be
used to provide additional information to the user about /why/ something is
bad. For example, in this case you might have a warning like "Dictionaries
cannot have duplicate keys, because then the correct value is unknown.
See [[blah blah blah]] for more information".
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mit-cml/appinventor-sources/issues/2352#issuecomment-748492687,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGAXNZKWCBKVF23ZA7JSCDSVTFPVANCNFSM4VCH5SJQ
.

That's the thing though, this scenario is valid code and the result is absolutely known and can be reasoned about. If a key is used multiple times, only the last value will be present as it effectively overwrites the existing mapping. I think it would be better for the type checker to not be involved with this because it's not an issue of typing. Having a warning is better like @BeksOmega notes because we can use it as an opportunity to explain why and therefore help the developer better understand how to build a more efficient app. There is also the issue of backward compatibility, whereby if someone has done this and we push a new version that disallows it how do we handle that without breaking their app. Using a warning doesn't put us in that position.

On the latter point, we could automatically disable intermediate keys that are effectively no-ops because they'll be overwritten. This is similar to what IntelliJ does for setting a variable many times without ever reading it. However, we don't have a way to differentiate things automatically disabled due to semantic equivalence versus user-initiated disabling. We might want to modify Blockly to allow that or see if we can build it as a plugin.

How about we show users a warning and also add a functionality where user can right click on the dictionary block and we can give them option saying "Merge duplicate keys" which when selected will go through the keys and remove duplicate keys or merge them into 1? @ewpatton

@devarsh-mavani-19 I think we just want to stick with the warning for now. There's an initial implementation in #2522 that we may want to clean up and merge.

repo: mit-cml/appinventor-sources


text:
mIoU = {(interaction.sum / union.sum).cpu().mean().item() * 100:.3f}") ZeroDivisionError: division by zero

This error coming while running eval_seg_model on ade20k. How to resolve this error.
eval command used: python eval_seg_model.py --path img1.jpg --weight_url  l1.pt --dataset ade20k --crop_size 1024 --model l1

I have the same problem. Have you solved it? How did you solve it?

there is something wrong with your path of dataset

repo: mit-han-lab/efficientvit


text:
Question about pagedattention

Hi, thanks for your great work! I have a small question.
In kernels/csrc/fused_attention/applyBiasRopeUpdateKVCache.h, I saw that you set store_contiguous_qkv = true. Does it mean that pagedattention is not used when quantize KV Cache into int8/int4?
Looking forward to your reply, thank you!

Hi,
Thanks for your interests in QServe. We do use page attention when KV Cache is quantized into int8/int4. Please ignore that flag.

repo: mit-han-lab/qserve


text:
[Support]

Config Creator error
Followed the steps in readme, installed homebridge-cmd4 and jq. I have input the device name and ip correctly in Settings, but when I click Config Creator, I got this error.
ERROR: no Homebridge config.json found in "/"! Please copy "cmd4Config_AA_MyPlace.json" to cmd4 JASON Config manually.
No such file "cmd4Config_AA_MyPlace.json" was found in the entire system.
Paste of Logs:
[20/05/2023, 11:15:21 pm] [Cmd4] Initializing Cmd4 platform...
[20/05/2023, 11:15:21 pm] [Cmd4] Class Cmd4Platform
[20/05/2023, 11:15:21 pm] Loading 5 accessories...
[20/05/2023, 11:15:21 pm] [Sonoff-sv] Initializing mqttthing accessory...
[20/05/2023, 11:15:22 pm] [Sonoff-sv-humidity-sensor] Initializing mqttthing accessory...
[20/05/2023, 11:15:22 pm] [Sonoff-sv-temperature] Initializing mqttthing accessory...
[20/05/2023, 11:15:22 pm] [Sonoff] Initializing mqttthing accessory...
[20/05/2023, 11:15:22 pm] [Security] Initializing security-system accessory...
[20/05/2023, 11:15:22 pm] [Security] Default mode (Off)
[20/05/2023, 11:15:22 pm] [Security] Arm delay (0 second/s)
[20/05/2023, 11:15:22 pm] [Security] Trigger delay (0 second/s)
[20/05/2023, 11:15:22 pm] [Security] Audio (Disabled)
[20/05/2023, 11:15:22 pm] [Cmd4] Cmd4Platform didFinishLaunching
[20/05/2023, 11:15:22 pm] [Cmd4] No queued polling characteristics
[20/05/2023, 11:15:22 pm] Publishing bridge accessory (name: Homebridge, publishInfo: {
  username: 'CC:23:4D:A3:CE:33',
  port: 51826,
  pincode: '***-**-***',
  category: 2,
  bind: [ 'br0', [length]: 1 ],
  mdns: undefined,
  addIdentifyingMaterial: true,
  advertiser: undefined
}).

Cmd4 Config:
{
        "platform": "Cmd4",
        "name": "Cmd4",
        "debug": true
}

Advantage Air System:

Control Unit:  MyPlace
Temperature Sensors:  No

Homebridge Environment:

Node.js Version: v16.17.0
NPM Version: 8.15.0
Homebridge Version: v1.6.1
homebridge-cmd4 Version: v4.50.2
homebridge-cmd4-AdvantageAir Version:  v3.9.0
Operating System: Raspbian

Hey @antoniokly, just double checking but did you also install homebridge-cmd4 as well?

Sorry just saw you said that, should not be reading before my morning coffee… Give me a little and I will come back with some suggestions haha

@antoniokly can you confirm for me if you installed homebridge-cmd4 via the Homebridge web UI install tab or via the instructions found on Cmd4’s page?

I installed homebridge-cmd4 via the Homebridge web UI

Thanks for confirming that, I think I was still on the wrong train of thought before. We are both using the same hosting machine, so I am confident we will get you up and running.
Can I get you to run the following command for me?
config1=$(find /usr/local/lib -name ConfigCreator.sh 2>&1|grep -v find|grep -v System|grep -v cache|grep node_modules|grep cmd4-advantageair|grep ConfigCreator.sh)

when it is done, then run the following command and paste the response back here.
echo "$config1"

Hey @mitch7391 and @antoniokly, based on the error message from ConfigCreator, it seems that homebridge storage path is set to / which is very unusual.  Would @antoniokly please post the system information of your homebridge installation?  It can be found on your homebridge web UI, it looks something like this:

On Raspbian installation, the storage path should have been /var/lib/homebridge/ or something like that. It should not be /.

response of echo "$config1"
/usr/local/lib/node_modules/homebridge-cmd4-advantageair/ConfigCreator.sh

I notice I have an older version of Raspbian, in the mean time, let me set up another raspberry with the latest OS. Thanks for the investigation.

I have set up another Homebridge with latest Raspbian, and it is working! Probably not worth to fix the issue I had on the older version of Raspbian. Thanks so much
BTW, just nice to have, can the Zone control displayed as a Fan with rotation speed in stead of a Dimmer with brightness?

can the Zone control displayed as a Fan with rotation speed in stead of a Dimmer with brightness?

The Zone control is currently using lightbulb accessory as a proxy, and it is certainly possible to use Fan accessory as a proxy instead but that involve a bit of modification in the code.  Mitch and I have preferred lightbulb accessory as a proxy during development stage so that  the zone controls do not confuse with the fanSpeed control.  What I have done is to change the Zone control 'lightbulb' icon to appear like a square ceiling vent outlet (example shown below):

Glad to hear it is all sorted @antoniokly! Really frustrating when it is just a reinstall job, definitely not the first time we have seen it.

BTW, just nice to have, can the Zone control displayed as a Fan with rotation speed in stead of a Dimmer with brightness?

In regards to this, we had to make a few broad choices with how we are defining devices due to how many Advantage Air offer now; I am not sure if it would confuse it as a light and be affected by collective commands like “set all lights to 50%”. I’ll refresh myself on the code and see if it’s something that is as simple as changing the config ‘linkedType’ over and you can test it out yourself :)

Always posting at the same time @uswong 😂

What I have done is to change the Zone control 'lightbulb' icon to appear like a square ceiling vent outlet

I have had many great suggestions from users over the last few years, it’s just hard to keep everyone happy haha one user wanted the zones to be ‘window covering’ accessory, which would also work in place of the light for zones.

What I have done is to change the Zone control 'lightbulb' icon to appear like a square ceiling vent outlet

This is what I would do too, to trick myself into thinking it is a vent. Mine are temperature controlled so mine are just the Switch accessories.

Mine are temperature controlled so mine are just the Switch accessories.

Hey Mitch, it is actually very easy to get your Zone control with temperature sensor to appear like lightbulb accessory by just changing the accessory type in the config from Switch to Lightbulb.  No change in the AdvAir.sh code required. As such, it won't affect any existing users!
If that is the desire for the way forward, we only need to make a minor modification to ConfigCreator.sh to create lightbulb accessory without the brightness characteristic instead of a Switch accessory for Zone control with temperature sensor. On second thought, I would have preferred it to be consistently using Lightbulb accessory as the proxy for all Zone control with or without temperature sensor.
I did tested using window covering accessory but I did not like it.
This does not affect my system and that of @antoniokly, as our systems are without temperature sensor.

Mine are temperature controlled so mine are just the Switch accessories.

In fact, Mitch, you can also make the Switch accessories to "Display As" "Light" in Homekit, then can change the Lightbulb icon to appear like a square ceiling vent outlet.

Hey Mitch, it is actually very easy to get your Zone control with temperature sensor to appear like lightbulb accessory by just changing the accessory type in the config from Switch to Lightbulb.

The reason I had not gone ahead with this was it creates the illusion that you can control your zone percentage open as a temperature sensor user (percentage open for these users is determined by the controller itself). When I had tested, you could set your zone percentage, but it would revert back to what the controller had decided it would be when the accessory updated. I felt users would think this was a mistake in our work and submit issues for it haha... That being said, it could be nice see the percentage open as long as users know they cannot set the value... Maybe we should think about this more!

I did tested using window covering accessory but I did not like it.

Glad we agree here :D

In fact, Mitch, you can also make the Switch accessories to "Display As" "Light" in Homekit, then can change the Lightbulb icon to appear like a square ceiling vent outlet.

You are right! And I do not know how this escaped me as I do it for the broadlink plugin lol just changed them over now in HomeKit and have them as the square vent looking ones now! Love it!

it could be nice see the percentage open as long as users know they cannot set the value... Maybe we should think about this more!

This is definitely a good idea... I like it!

BTW, just nice to have, can the Zone control displayed as a Fan with rotation speed in stead of a Dimmer with brightness?

@antoniokly you may have notice in the latest update, but we have made this an option now!

authors: antoniokly mitch7391 uswong


text:
Issue with vagrant 2.3.0

Ran into this issue after upgrade to Vagrant version 2.3.0 (Ubuntu 18.04)

$ vagrant up
Vagrant failed to initialize at a very early stage:
There was an error loading a Vagrantfile. The file being loaded
and the error message are shown below. This is usually caused by
an invalid or undefined variable.
Path: /opt/vagrant/embedded/gems/2.3.0/gems/vagrant-2.3.0/plugins/commands/serve/command.rb:24
Line number: 0
Message: uninitialized constant VagrantPlugins::Google::Protobuf

It works well on Vagrant 2.2.19 and previous versions.
Tried plugin versions 2.7.0 and 2.6.0 with the same result.

I am also experiencing the same issue.

I also am having this error on an Apple M1 Monterey.

Same problem in the Apple M1 Monterey.

Same problem in Apple Intel 2019 Macbook

I fixed the issue by prepending :: on Empty = Google::Protobuf::Empty
i.e. Empty = ::Google::Protobuf::Empty
in file path /opt/vagrant/embedded/gems/2.3.1/gems/vagrant-2.3.1/plugins/commands/serve/command.rb

@ericraio 's fix worked for me on vagrant version 2.3.2 and plugin version 2.7.0, as well.

authors: MrZeLee RyandenOtter azmng dcode denders-conception ericraio

repo: mitchellh/vagrant-google


text:
how to generated options.html & filters.html

Steps to reproduce the problem:

git clone https://github.com/mitmproxy/mitmproxy.git
2.cd docs
yarn
modd

Any other comments? What have you tried so far?
11:12:11: prep: ./node_modules/.bin/node-sass -o ./src/themes/mitmproxydocs/static/css ./style/style.scss --error-bell
11:12:11: daemon: cd src; hugo server -D
>> starting...
Building sites … ERROR 2018/11/26 11:12:11 "/Users/dc/Documents/interest/gitHub/mitmproxy/docs/src/content/concepts-options.md:47:5": failed to render shortcode "readfile": "/Users/dc/Documents/interest/gitHub/mitmproxy/docs/src/layouts/shortcodes/readfile.html:5:12": execute of template failed: template: shortcodes/readfile.html:5:12: executing "shortcodes/readfile.html" at <readFile>: error calling readFile: stat /Users/dc/Documents/interest/gitHub/mitmproxy/docs/src/content/generated/options.html: no such file or directory
Total in 51 ms
Error: Error building site: "/Users/dc/Documents/interest/gitHub/mitmproxy/docs/src/content/concepts-filters.md:13:5": failed to render shortcode "readfile": "/Users/dc/Documents/interest/gitHub/mitmproxy/docs/src/layouts/shortcodes/readfile.html:5:12": execute of template failed: template: shortcodes/readfile.html:5:12: executing "shortcodes/readfile.html" at <readFile>: error calling readFile: stat /Users/dc/Documents/interest/gitHub/mitmproxy/docs/src/content/generated/filters.html: no such file or directory
exited: exit status 1
>> sleeping... 1000000000

I just delete {{< readfile file="/generated/options.html" >}} in mitmproxy/docs/src/content/concepts-options.md:47:5 and {{< readfile file="/generated/filters.html" >}} in mitmproxy/docs/src/content/concepts-filters.md:13:5 and it will work fine but it miss something.
System information
Mitmproxy: 4.0.4
Python:    3.7.0
OpenSSL:   OpenSSL 1.1.0i  14 Aug 2018
Platform:  Darwin-18.0.0-x86_64-i386-64bit

#3667
I solved the issue by running the ./build-current script

Thanks @jannst!

authors: MrChens jannst mhils

repo: mitmproxy/mitmproxy


text:
Closes #1960 'Z' removes unmarked flows

'Z' now keeps the marked flows. Unmarked flows are cleared.

Awesome, thanks! :smiley:


text:
dbt model warnings on production

Description/Context

There are several warnings/errors on dbt production run recently. Some of these are due to schema changes like https://github.com/mitodl/mitxpro/pull/2716
Failure in test not_null_stg__mitxpro__app__postgres__ecommerce_product_product_is_visible_in_bulk_form (models/staging/mitxpro/_stg_mitxpro__models.yml)
Got 554 results, configured to fail if >10

compiled Code at target/compiled/open_learning/models/staging/mitxpro/_stg_mitxpro__models.yml/not_null_stg__mitxpro__app__po_f7cabc3547213d75b7ba80e3ce1b9e2c.sql

Warning in test not_null_int__edxorg__mitx_users_user_email (models/intermediate/edxorg/_int_edxorg__models.yml)
Got 2 results, configured to warn if != 0

compiled Code at target/compiled/open_learning/models/intermediate/edxorg/_int_edxorg__models.yml/not_null_int__edxorg__mitx_users_user_email.sql

Warning in test dbt_expectations_expect_table_row_count_to_equal_other_table_int__mitxpro__ecommerce_basketitem_ref_stg__mitxpro__app__postgres__ecommerce_basketitem_ (models/intermediate/mitxpro/_int_mitxpro__models.yml)
Got 1 result, configured to warn if != 0

compiled Code at target/compiled/open_learning/models/intermediate/mitxpro/_int_mitxpro__models.yml/dbt_expectations_expect_table__be61db397128264027b33e0ea0cf7c8f.sql

Warning in test dbt_expectations_expect_table_row_count_to_equal_other_table_int__mitxpro__ecommerce_basketrunselection_ref_stg__mitxpro__app__postgres__ecommerce_basketrunselection_ (models/intermediate/mitxpro/_int_mitxpro__models.yml)
Got 1 result, configured to warn if != 0

compiled Code at target/compiled/open_learning/models/intermediate/mitxpro/_int_mitxpro__models.yml/dbt_expectations_expect_table__ceb0a2596c8b34ba2b4e53af85ac683b.sql

Warning in test not_null_int__edxorg__mitx_user_courseactivities_user_email (models/intermediate/edxorg/_int_edxorg__models.yml)
Got 2 results, configured to warn if != 0

compiled Code at target/compiled/open_learning/models/intermediate/edxorg/_int_edxorg__models.yml/not_null_int__edxorg__mitx_user_courseactivities_user_email.sql

Warning in test not_null_int__edxorg__mitx_courserun_enrollments_user_email (models/intermediate/edxorg/_int_edxorg__models.yml)
Got 2 results, configured to warn if != 0

compiled Code at target/compiled/open_learning/models/intermediate/edxorg/_int_edxorg__models.yml/not_null_int__edxorg__mitx_courserun_enrollments_user_email.sql

Warning in test not_null_int__combined__users_user_email (models/intermediate/combined/_combined_models.yml)
Got 2 results, configured to warn if != 0

compiled Code at target/compiled/open_learning/models/intermediate/combined/_combined_models.yml/not_null_int__combined__users_user_email.sql

Warning in test not_null_int__mitx__courserun_enrollments_user_email (models/intermediate/mitx/_int_mitx__models.yml)
Got 2 results, configured to warn if != 0

compiled Code at target/compiled/open_learning/models/intermediate/mitx/_int_mitx__models.yml/not_null_int__mitx__courserun_enrollments_user_email.sql

Warning in test not_null_int__combined__courserun_enrollments_user_email (models/intermediate/combined/_combined_models.yml)
Got 2 results, configured to warn if != 0

Review and fix these issues

I thought @asadali145 was working on this already.

I thought @asadali145 was working on this already.

Yes, he had PR https://github.com/mitodl/ol-data-platform/pull/825 for adding course platforms, but I don't see any PR to rename visible_in_bulk_form to is_private in data platform.

We pushed the release to production yesterday. I wasn't sure about creating a PR here. I can work on one if you want me to @rachellougee.

@asadali145 if you have time to create a PR for renaming visible_in_bulk_form to is_private next week, that'd be great. There are other warnings I will look into.

I can do that today. Will create a PR in a few minutes.

I am going to assign it to myself.

There are other warnings that need to be looked into as well. The first error should be gone tomorrow

Confirmed these errors/warnings are gone, the only warning left is https://github.com/mitodl/ol-data-platform/blob/main/src/ol_dbt/models/intermediate/mitxpro/_int_mitxpro__models.yml#L570

Warning in test dbt_expectations_expect_table_row_count_to_equal_other_table_int__mitxpro__ecommerce_basketitem_ref_stg__mitxpro__app__postgres__ecommerce_basketitem_ (models/intermediate/mitxpro/_int_mitxpro__models.yml)
Got 1 result, configured to warn if != 0

This is known issue with basket_id being out of sync for raw table replications raw__xpro__app__postgres__ecommerce_basketitem and raw__xpro__app__postgres__ecommerce_basket. That out-of-sync basket id is not propagated into intermediate. I think we can close this

@rachellougee Is there anything that I need to work on for this issue?

@asadali145 you don't need to fix anything, I've confirmed that your changes to is_private in ecommerce_product is now gone. All other warnings are unrelated to your changes, that particular warning about basket_id is not related to dbt model, its raw table out of sync issue, so in short, no further action needed

authors: asadali145 pdpinch rachellougee

repo: mitodl/ol-data-platform


text:
Added support for reset password prompt on login

What are the relevant tickets?

Part of https://github.com/mitodl/hq/issues/3473
Description (What does it do?)

This adds the functional behavior for prompting the user to reset their password if they're attempting to login but don't have a password set or any other credential mechanism (e.g. touchstone).
Note: this does not address any look and feel discrepancies vs the designs, it is purely a functional change.
I also added some limited nix flake setup for me to have a workable local dev environment - it doesn't impact functionality here.
Screenshots (if appropriate):
If you don't have "forgot password" enabled for the realm:

If you do have it enabled:

How can this be tested?

Create a user with no password in the admin interface
Verify "Forgot password" is turned off
Attempt to login as that user and verify that you get an error message that we can't log them in. In practice we shouldn't have this setup this way, but it needed to be handled as it is a potential state and having a helpful error message is better than none.
Turn on the "forgot password" feature for the realm.
Try to login again, you should get the prompt to reset your password and it should navigate to the reset password page.
At this point we've simply navigated the user to the password reset page - anything further isn't effected by this PR (i.e. if password reset works in general, it'll work here too).

@collinpreston done

repo: mitodl/ol-keycloak


text:
🛑 Vividcraft is down

In 843cfbc, Vividcraft (https://vividcraft.com.au) was down:

HTTP code: 502
Response time: 919 ms

Resolved: Vividcraft is back up in 3b1f6b8 after 12 minutes.

repo: mitrasheamuskhanal/INS-Tracker


text:
Update taskCreation.ts

silly typo

Thanks! 👍

Of course!
Thank you.
On Mon, Jul 31, 2023 at 9:37 PM Yoshiki Miura @.***>
wrote:

Thanks! 👍
—
Reply to this email directly, view it on GitHub
https://github.com/miurla/babyagi-ui/pull/159#issuecomment-1659553606,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABYJOGLEE5YM7NRGXR4UHDXTCBZVANCNFSM6AAAAAA27EZJAI
.
You are receiving this because you were mentioned.Message ID:
@.***>

authors: ideabrian miurla

repo: miurla/babyagi-ui