Stable simulations after geometry proposal

Just wanted to start a thread to discuss the best way to ensure we get stable simulations after the geometry proposal step.
In order for NCMC to work, we'll need to have a stable simulation for the alchemically-modified lambda=0 state after the geometry proposal step for some timestep.  If we can't get stable dynamics with lambda=0, NCMC is not going to help us improve acceptance probabilities. In principle, this should be relatively straightforward since we should only be dealing with intramolecular valence terms at this point.
Obviously, the best solution would be to see if we can improve the geometry proposal, but there are always possibilities where we can't control what is going on with more complex intramolecular interactions or steric clashes that will occur when we start switching lambda from 0 to 1.
Ideas @pgrinaway has suggested are:

Metropolis Monte Carlo.  Rejects NaNs (good!), but does not respect constraints and makes unbiased Gaussian proposals in each atomic coordinate (bad).
GHMC. Rejects NaNs (good!) and uses gradient information to achieve high acceptance probabilities (also good).  Timestep could potentially be autotuned, or even more sophisticated schemes could be used (e.g. extra-chance GHMC).  We have to change the NCMC engine to accumulate the appropriate form of work for this, however, since it will be the potential energy change accumulated from each step since Metropolization already handles the shadow work.

Do we want to have the flexibility to try multiple integrators inside of NCMCEngine?  If so, what should the API for that look like?
I'm happy to take a stab at this once we decide what the appropriate functionality and API would be.

A couple of extra thoughts here:

I'm not sure if the reason GHMC and Metropolis work here is that they reject NaNs. I think it's not unlikely that neither of them produce NaNs even in their proposals--in the case of GHMC, if the simulation is heading toward NaNland, it might reject before going there. I'm not sure if the reason here really matters, though.

Couldn't we get the Metropolis integrator to respect contraints? It might not be worth it though, since as you point out the proposals are kind of crappy.

Would it be reasonable to simply perform the entire simulation, NCMC and propagation of the coordinates given a compound, with GHMC? It seems to be very well-behaved.

If we decide to go with GHMC, I think it's worth exploring some sophisticated schemes for tuning the GHMC timestep.

*I really like the idea of an NCMCEngine that can take different integration schemes, since I think this will become very useful for tuning, as well as providing an easy way to benchmark algorithms.

I'm not sure if the reason GHMC and Metropolis work here is that they reject NaNs. I think it's not unlikely that neither of them produce NaNs even in their proposals--in the case of GHMC, if the simulation is heading toward NaNland, it might reject before going there. I'm not sure if the reason here really matters, though.

For GHMC, you could turn off the Monte Carlo step and see if it explodes.  If not, it's the thermostat that is helping.
For both GHMC and MMC, we can add a counter that counts how many times NaNs were rejected.

Couldn't we get the Metropolis integrator to respect contraints? It might not be worth it though, since as you point out the proposals are kind of crappy.

This would be super hard because you either need to make proposals in bond-angle-torsion variables and compute Jacobians for the M-H acceptance criteria, or you need to rewrite the OpenMM constraint algorithm to give you the Lagrange multipliers and compute the Jacobians from that.

Would it be reasonable to simply perform the entire simulation, NCMC and propagation of the coordinates given a compound, with GHMC? It seems to be very well-behaved.

I think we can use GHMC in NCMC by modifying the acceptance criteria, but we have to compute the correct criteria for this. We probably want to understand why it works so much better than VV in case we are missing something.

If we decide to go with GHMC, I think it's worth exploring some sophisticated schemes for tuning the GHMC timestep.

For GHMC, you could turn off the Monte Carlo step and see if it explodes. If not, it's the thermostat that is helping.

I think the Monte Carlo step is really what's helping here, since it still explodes when I remove that bit.

This would be super hard because you either need to make proposals in bond-angle-torsion variables and compute Jacobians for the M-H acceptance criteria, or you need to rewrite the OpenMM constraint algorithm to give you the Lagrange multipliers and compute the Jacobians from that.

Ok, nevermind then.

I think we can use GHMC in NCMC by modifying the acceptance criteria, but we have to compute the correct criteria for this. We probably want to understand why it works so much better than VV in case we are missing something.

I agree, I'd like to know as well. Besides the presence of an extra energy computation, is there any reason we should not prefer GHMC or its variants for the propagation of the system (even outside of NCMC)?

Good point. What would the API look like? Which schemes should it support?

I will open another issue for brainstorming this.

GHMC will be much more expensive than VV for propagating NCMC switches. Maybe 2-10 times per timestep depending on the overhead of energy computation, extra force computation, the global kinetic energy sums, and the accept/reject step.

It could still end up being more efficient to use GHMC, of course, but from a cost per timestep view, VV is most efficient.

I think we're good here. Can re-open later if necessary.


add s3_reduced_redundancy option

This time with better formatting for flake8.

Ehh? This looks like a problem with the test process, I don't see any actual errors from my changes?

ping I think this is a failure in the test environment -- manually running the verification it looks fine. Can you confirm?

Coverage decreased (-0.5%) to 59.536% when pulling 44abd6fc1900949ee60e982e106ea0afad14d08c on psifertex:master into 6c5a6d629fc2f1321337efa345e5d7fdc5f41910 on jonls:master.

Merged 👍


tflite depthwise_conv2d produces different(wrong) results

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 16.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: N/A
TensorFlow installed from (source or binary): pip install tensorflow
TensorFlow version (use command below): 1.12.0
Python version: 2.7.12
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: CUDA 9 cuDNN 7.1
GPU model and memory:GTX 1080

So I was trying to use tflite converter to convert a pretrained model. When I use the same input to do inference by both tflite interpreter and native tensorflow I observed vastly different outputs . The difference is caused by the tfLite depthwise2d operator converted from the original  tf.nn.conv2d operator.  So I constructed a mini example to explain the problem:
import numpy as np
import tensorflow as tf

#construct a fake weight coeffecients
np.random.seed(0)
weight_array= np.random.uniform(-1,1, size=(1,3,1,9)).astype("float32")

#construct a simple graph involving conv2d
tf.reset_default_graph()
input_1 = tf.placeholder(dtype=tf.float32, shape=[1,1,600,1])
weights = tf.constant(weight_array)
weights = tf.quantization.fake_quant_with_min_max_args(weights, np.min(weight_array), np.max(weight_array))
out = tf.nn.conv2d(input_1, weights, [1,2,2,1], "SAME")
out_1 = tf.nn.relu6(out)
q_out = tf.quantization.fake_quant_with_min_max_args(out_1, 0, 6)

#convert the graph to tflite in unit8 quantized inference mode
with tf.Session() as sess:
    converter = tf.contrib.lite.TFLiteConverter.from_session(sess, [input_1], [q_out])
    converter.inference_type = tf.contrib.lite.constants.QUANTIZED_UINT8
    converter.inference_input_type = tf.contrib.lite.constants.QUANTIZED_UINT8
    input_arrays = converter.get_input_arrays()
    converter.quantized_input_stats = {input_arrays[0] : (128, 1)}
    tflite_model = converter.convert()

#define a dummy input
input_sig = np.ones([1,1,600,1])
input_data = input_sig.astype("uint8")

#inference using tflite interpreter
interpreter = tf.contrib.lite.Interpreter(model_content=tflite_model)
input_details = interpreter.get_input_details()
output_details = interpreter.get_output_details()

input_shape = input_details[0]['shape']
input_index = input_details[0]['index']
output_index = output_details[0]['index']

interpreter.allocate_tensors()
interpreter.set_tensor(input_index, input_data)
interpreter.invoke()
#get tflite int8 output
tf_lite_output = interpreter.get_tensor(output_index)

scale = output_details[0]['quantization'][0] #scale factor for quantized output
weight_scale =  interpreter._get_tensor_details(0)['quantization'][0] #scale factor for quantized weights
zero_point = output_details[0]['quantization'][1] #the point for quantized output maps to float 0

#get tensorflow float32 output from fake quantized node
with tf.Session() as sess:
    tf_output = sess.run(q_out,  {input_1: input_sig})

#compare, they should be at least close, but nope
print tf_lite_output[0][0][0]
print tf_output[0][0][0]/ scale + zero_point #converted to uint8 output using the quantization information in the output_details

Describe the current behavior
tflite output gives:
[  0,   0,   0,   0,   0,   0, 255,   0,   0]
after converting to uint8 by sc, tensorflow model gives:
[18. 74. 52. 35. 27.  0.  0.  4. 37.]
Describe the expected behavior
I believe those two results should give a very similar result when fed with identical inputs. I also checked that the quantized weight matches with the output from fake quantization node after scaling using the weight_scale. I also manually calculated the operation defined in the graph and produced a result close to what tensorflow model gives. Is there a reason that tflite computation behaves completely different than the native tensorflow ?

Actually found the solution. Changed converter.quantized_input_stats = {input_arrays[0] : (128, 1)}  to converter.quantized_input_stats = {input_arrays[0] : (0, 1)}  and then tf and tflite gave almost same outputs.  Although I am pretty suprised that the output is that sensitive input stats.

hi shenyichen,
Why do you feed same input to Tf and Tflite ?
I expected Tflite to get quantized input. But i see you are feeding the same input.
Are you simulating same float models for both ?


Create .pre-commit-hooks.yaml for support pre-commit hook

What
Support https://pre-commit.com/
How
Refer https://pre-commit.com/index.html#developing-hooks-interactively
Usage:
repos:
  - repo: https://github.com/jellydn/sort-package-json
    rev: "1686d9d2ddfb065c4514c637ee4b9985dfbf01dd" # Use the sha / tag you want to point at
    hooks:
      - id: sort-package-json

  - repo: https://github.com/pre-commit/mirrors-prettier
    rev: "v4.0.0-alpha.8" # Use the sha or tag you want to point at
    hooks:
      - id: prettier
        # Those are not supported by biomejs yet, refer https://biomejs.dev/internals/language-support/
        types_or: [html, css, markdown]
  - repo: https://github.com/biomejs/pre-commit
    rev: "v0.1.0" # Use the sha / tag you want to point at
    hooks:
      - id: biome-check
        exclude: "package.json"
        additional_dependencies: ["<EMAIL_ADDRESS>
Tested on my PR https://github.com/jellydn/next-app-starter/pull/1606/files

Thanks but I don't think this is necessary.


chore: try react-scripts-rspack

the cold build time reduce from 51.39s to 4.85s

just a POC of using react-scripts-rspack as  drop-in replacement of react-scripts, feel free to close this

@hardfist looks cool. Just taking it for a spin and randomly checking things.
Is HMR working for you on our project? Saving the file with an edit doesn't seem to do anything.
Note that we're disabling FAST_REFRESH by default, though I have it on myself, and I don't even believe that rspack uses that env var.
https://github.com/excalidraw/excalidraw/blob/c170403b1322cc6de96c142c3bc64cd676bad6fa/.env.development#L24

https://user-images.githubusercontent.com/8898718/230746826-dfa33fcc-c66e-424d-8831-8a68767aeb3e.mp4
it works on my computer, can you give your systeminfo by npx envinfo --system --npmPackages '@rspack/*' --binaries --browsers, I'm gonna see why it not work

good job


[SVG] Cannot export the SVGScene into image

We are trying to export the canvas to png using add.renderer.generateTexture and app.renderer.extract, but the svg object isn't showing in the image.
After digging into the source code and found out this line is causing the issue
https://github.com/ShukantPal/pixi-essentials/blob/0c61c083b7e1bd5206b015699c8d13a0c1e948b5/packages/svg/src/SVGScene.ts#L177
By removing this line, the svg object now can export as expected
- this._cull.cull(renderer.renderTexture.sourceFrame, true);

Although the app works fine without this line, we are not sure wether if there's side effect or not.

Hi @rnike, this is a performance optimization that culls out elements that are off-screen. If you try to render the SVG into a texture without updating the culling, it will not render the previously culled items.

@ShukantPal Thanks for the reply, I found out there is a transform in renderer while using the add.renderer.generateTexture method, this cause the cull be culling with the incorrect area.
 https://github.com/pixijs/pixijs/blob/c04c09c50874fef9296555af23e25fe58cc808a9/packages/core/src/renderTexture/GenerateTextureSystem.ts#L83C1-L96C1
Shall we also handle the renderer.projection.transform when culling the object?
For example adjust the xy of the cull rectangle with the transform from renderer, the following workaround fix our case.
        this.tempSourceFrame.copyFrom(renderer.renderTexture.sourceFrame);

        if(renderer.projection.transform){
            this.tempSourceFrame.x -= renderer.projection.transform.tx
            this.tempSourceFrame.y -= renderer.projection.transform.ty
        }

        // Cull the SVG scene graph
        this._cull.cull(this.tempSourceFrame, true);

The transform is to map (x, y) to (0, 0) in the texture (aka the top left of the texture). But the culling should be done in world space, without discounting the projection transform.
What may be happening is that skipUpdateTransform is being set (since I assume the Transformer is mounted in your scene graph somewhere). That will make the children of the Transformer not update their world bounds. I suggest removing the transformer from your scene, rendering it into a texture, and then re-adding it back.

Unfortunately,  removing Transformer isn't solving our problem.
What we are trying to do is to export the specific area of the stage to image.
       // this is what pixi do while calling `add.renderer.generateTexture`

        const texture = RenderTexture.create({
          width: region.width,
          height: region.height,
        });

        app.renderer.render(app.stage, {
          renderTexture: texture,
          transform: new Matrix().translate(-region.x, -region.y),
          skipUpdateTransform: false,
          blit: true,
        });

output

If increase the size of the texture and don't set the transform, the svg will display but the output image size will be greater than expected
        const texture = RenderTexture.create({
          width: region.width + region.x, // add region.x to width
          height: region.height + region.y, // add region.y to height
        });

        app.renderer.render(app.stage, {
          renderTexture: texture,
          // transform: new Matrix().translate(-region.x, -region.y),  don't set the transform
          skipUpdateTransform: false,
          blit: true,
        });

output

If don't increase the size of texture and don't set the transform for render, the svg graphics will not be draw in the image because of the incorrect export area and culling.
        const texture = RenderTexture.create({
          width: region.width,
          height: region.height,
        });

        app.renderer.render(app.stage, {
          renderTexture: texture,
          // transform: new Matrix().translate(-region.x, -region.y), don't set the transform
          skipUpdateTransform: false,
          blit: true,
        });

output


refactor(linter): use regex visitor in no-regex-spaces

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#6063  👈
#6062
#6058
#6056
#6055
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @camchenry and the rest of your teammates on  Graphite


Tweak error timestamp formatting to be more readable

This uses a . instead of : as the decimal separator and trims the decimal part to 3 digits, as millisecond precision is usually sufficient.

Thanks!


xen 4.5 end-of-support, should be deprecated

Issue description
Security support for Xen 4.5 (our current default version) ended January 2018,
so we should change the default to 4.8 in 18.03 (and remove 4.5 in the future).

@oxij what's your opinion on this?

See #36770.


New option to configure firefox binary

Coverage increased (+0.1%) to 90.757% when pulling ed3c9f6979fbbd95473d62c4467ddaa566b550b8 on feature/firefox_binary into 43ebade7080d72d75675ba0458b2eef622e68be0 on master.


Per-column labels are not supported

Per-column labels would be very nice! Did you try to implement this and found it wasn't worth the trouble?

Hi Matias,
I agree it would be nice but I don't require them in my project. I also unfortunately won't have time in the coming weeks to implement this. It really shouldn't be too hard to implement though. So if you have the need I heartily encourage you to extend fullcalendar-columns and submit a pull request! :)
Best,
Michael

OK, I'll have a go at it!

Any update on this? The column label is required else it's unusable for me...

Haven't had the time to start on this yet but please have a go at it if you like, @JoeyHoutenbos.

I added columns display in this PR - https://github.com/mherrmann/fullcalendar-columns/pull/4 - so if you're interested you can test it out. You will also notice a new variable columnData that will be reported during drag / drop, resize and time selection as well which is passed to columns.
Instead of setting numColumns, you would need to pass an array to columns, ie.:
columns: [ { id: 1, name: 'Col 1' }, { id: 2, name: 'Col 2' } ]

Hey #aromka, I'm interested in your solution. Could you please post a jsfiddle?

Thank you Roman! Two few questions:

Why did you have to overwrite reportSelection? Is it something to do with column labels, or related to a different issue?
Why did you have to set result.endOrig = result.end.clone(); in computeRange?

Cheers! :)
M

How this code add the event to the calendar? 'Cause I don't see any method: http://fullcalendar.io/docs/event_data/addEventSource/
someone could help me to understand? I'm want using the aromka solution for have the resource label displayed.

Aromka I've contacted you on gmail. I want to propose my question:
http://stackoverflow.com/questions/33245038/how-to-redraw-fullcalendar-view
And I want also ask to you if I can get the column number when an eventi is moved to another column. In particolar if an event is on column 0 and the user nove the event on column 1 I want recognize this, how I can do this? Thank you.

Hi aromka I want to ask you a question:
When I save an event on a column, for example column 0 it's possible check when a en event is moved on another column? If the event is moved on another column get the column number with resource label. That's only. I also want report you a bug, if I insert a lot of resources with a long name the label overlap the other cell, could you fix this. Thank you.

Hi aromka I want to ask you a question:
When I save an event on a column, for example column 0 it's possible check when a en event is moved on another column? If the event is moved on another column get the column number with resource label. That's only. I also want report you a bug, if I insert a lot of resources with a long name the label overlap the other cell, could you fix this. Thank you.
PS: mrherrman I want improve your solution so don't delete my post 'cause other user could find usefull!!
Don't ban me to this repository again. How old are you? Show respect for anyone on your project and try to improve it.

@mherrmann

I overwrite reportSelection because I needed to get the columnData (which has the ID - in my project I had an option to toggle on / off columns, so I needed to know for which columns I should be rendering events).
If you select time in a second column and look at the date - it gives you a fake date because of how this plugin renders the columns, so if you have 2 columns for September 1 and select time - first column will return the proper date, but second column will return September 2, so endOrig should return you the proper date.

@Back2TheFuture - for long names you can just use css to prevent long names to overlap (ie. https://css-tricks.com/snippets/css/truncate-string-with-ellipsis/)
For moving the events to different columns - you should be able to get it from select function - something like:
select: function(dtStart, dtEnd, jsEvent, event) {
    console.log(event.column, event.columnData);
}

Hi @Aromka thanks for the support could you please provide a jsfiddle for the solution? Thank you :)

This has now been implemented in release v1.6. The API is a little different from @aromka's PR, for backwards-compatibility reasons. Also @aromka, I did not include all the changes you made because I am not sure yet whether they are "general" enough for everybody to be useful. I believe you should be able to get the same functionality with the new API by keeping a little more state in your "own" objects, rather than in columnData as you proposed. Thank you very much for your contribution. I'm not 100% sure the new API will fit your requirements as nicely, but I am certain that it will benefit others.


Add sha1 to updateFiles

see https://developer.github.com/v3/repos/contents/#update-a-file

We need to preserve the existing method signature, so the new additions would have to be new methods.
At this point, createContent has so many parameters that it should switch to a builder pattern.


more useful error/warn msg in rxjs/src/internal/util/subscribeTo.ts in Rxjs 6

Feature Request
I misstyped imports of observable in one source file: "rxjs", other imports were from "rxjs/index", then I was encountering following warning / error in all browsers except Chrome, which means, my forkJoin / combineLatest is resolving in Chrome ONLY.
const msg = `You provided ${value} where a stream was expected.`
      + ' You can provide an Observable, Promise, Array, or Iterable.';

Describe the solution you'd like
maybe an extra warning message: please watch out how you imported Observable / Combinelatest / forkJoin or whatever ?

This is similar to https://github.com/ReactiveX/rxjs/issues/4873 - as a lib, we can't control import site. It should be controlled by build / lint or anything else. In short, library can't know how you imported from which path.

thanks for the link :) , how come it behaved differently in different browsers?

it's likely possible based on runtime behavior different or build config or lot of other possibilities out there.

But you do agree it is hard to debug, right?

The issue in here is not about the easiness of debugging, what we can do as a lib. Import site control is not something lib can do at all.


Define (default) license for shared code repositories

@mmore and I would prefer a (weak) copy-left license by default; reasons:

avoid code base fragmentation : copy-left
avoid commercial usage clashes : weak

Proposal: EPL-v2 (OSI / FSF approved)

@MarkusTiede is it possible to use licenses like GPL-v3 within the Eclipse foundation?

@MMore - yes; the EPL-v2 has the concept of „Secondary License“ which is specifically made for GPL-v2 and higher combination scenarios. We used this e.g. as a license proposal for OpenPrevo.

However GPL is then the secondary license (with EPL-v2 being first) and the consumer may choose by himself.

I think it's a (very) good compromise to try to avoid the tragedy of anticommons (which might apply with the EPL as e.g. Patents are explicitly allowed) and tragedy of commons (which might (somehow) apply with the GPL).

Gaining (informal) feedback from the OSI as well as Eclipse Foundation the most preferred license with respect to flexibility (being consumend / adopted without major constraints from a copy-left) at the moment is Apache-2.0. We will also very likely take recommend this as a default license for Baloise wide activities.

We at https://github.com/aposin also decided to use Apache-2.0.


Updated CF templates

Description of changes:
Updated CF templates to use the latest instance types.
Updated the PHP versions to the ones that are currently supported by Wordpress
Fixed master template name and link on README.md
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@jamesiri Please pull this in!
Thanks

Only new versions parameter values were added so it doesn't mean "new php versions added". There should be new LaunchConfiguration resources for the new PHP versions. PR shouldn't be merge, otherwise it will break all the deployment if someone choose one of the "new versions" because there is no LaunchConfiguration for those.


Get cabal tests working more reliably

Some diagnostic output and fiddling with the cabal setup script and possibly tests to get it working reliably in CI.

Seems like maybe the minor script changes fixed it 🤷♂


Post photometry to skyportal in flux space and handle negative fluxes

Requesting review from @profjsb (can't request him as a reviewer formally)
This PR

Modifies the make_photometry function to convert photometry from magnitude space to flux space. Photometry from negative difference images is given a negative sign in flux space, gracefully handling negative photometry for variable sources and transients with reference image flux.

Modifies the alert_put_photometry function to PUT the photometry to skyportal in flux space.

@dannygoldstein terribly sorry, completely forgot about this one! Will revisit after #29 is merged, as the changes here will need to go on top that.

Now that #29 has been merged, we should get back to this one.

@dannygoldstein I copied your code over to the new alert_broker_ztf.py script and filed a PR on your fork https://github.com/dannygoldstein/kowalski/pull/1, please merge.

@dannygoldstein I copied your code over to the new alert_broker_ztf.py script and filed a PR on your fork https://github.com/dannygoldstein/kowalski/pull/1, please merge.

@dannygoldstein do you want to merge the pr on your fork so that we can merge this one?

I think this should be in good shape now


apply ComponentProvider to Application

[x] add an interface to Application
[x] add ComponentProvider to Application

finally, I found that the writing test for the Application is so complicated and needs more preparation, so I'll create another PR.

I'll try to fix module-test


Optimize count with an overload which calls walkLength

Following the discussion in #5545 , I created this separate PR for the count optimization. The implementation uses walkLength to compute the count. If walkLength will be deprecated, then I will replace the call with the code, otherwise we can close the other PR and keep just this oprimization

Why is that needed.
The ddoc is bigger then the function ?
which is a thunk.
a templated thunk no less.
introducing symbol bloat.
You could just have made an alias you know ...

You could just have made an alias you know ...

Would break code, due to the different default behavior. count allows a user-defined pred, walkLength allows a maximal count.
Though imho on the long run on of these functions should be deprecated (or at least removed from the docs).

The ddoc is bigger then the function ?

Because documentation is a lot more important than code. This is a common pattern in Phobos ;-)

@wilzbach I created an overload anyway so making an alias won't break any code, but it seems cleaner to me to have an overload, rather then an alias.

(keeping this open for one or two days to give people enough time to veto this approach as a general good practice)

Aliases are generally better, but here we have overloading so that wouldn't work well.
This improves the situation whereby someone calls count with the default predicate - avoid the goofy walking the whole range. That's fine, as is to allow two distinct library functions sometimes do the same thing under certain circumstances.

@andralex done


fix: add required fuse package to fedora

"fuse" package needed for fusermount
"fuse3" package provides fusermount3
When the fuse package isn't installed:
$ ./oudedetai --install-app
...
Installing registry file: /tmp/LBS.2r05magl/disable-winemenubuilder.reg
2024-11-14 14:57:56 ERROR: Error occurred in run_command() while executing "['/home/user/LogosBible10/data/bin/wine64', 'reg', 'query', 'HKCU\\Software\\Wine\\Fonts', '/v', 'Codepages']": Command '['/home/user/LogosBible10/data/bin/wine64', 'reg', 'query', 'HKCU\\Software\\Wine\\Fonts', '/v', 'Codepages']' returned non-zero exit status 127.
2024-11-14 14:57:56 WARNING: Failed to get registry value: HKCU\Software\Wine\Fonts\Codepages
2024-11-14 14:57:56 ERROR: wine.wine_proc: wine.get_registry_value returned None.
Failed to install reg file: /tmp/LBS.2r05magl/disable-winemenubuilder.reg
2024-11-14 14:57:56 CRITICAL: Failed to install reg file: /tmp/LBS.2r05magl/disable-winemenubuilder.reg
Killed
$ /home/user/LogosBible10/data/bin/wine64 reg query HKCU\\Software\\Wine\\Fonts /v Codepages
fuse: failed to exec fusermount: No such file or directory

Cannot mount AppImage, please check your FUSE setup.
You might still be able to extract the contents of this AppImage
if you run it with the --appimage-extract option.
See https://github.com/AppImage/AppImageKit/wiki/FUSE
for more information
open dir error: No such file or directory

This using Fedora-40 and fedora-41

Here's what we have in system.py:
def get_package_manager():
    # Check for package manager and associated packages
    if shutil.which('apt') is not None:  # debian, ubuntu
        # ...
    elif shutil.which('dnf') is not None:  # rhel, fedora
        config.PACKAGE_MANAGER_COMMAND_INSTALL = ["dnf", "install", "-y"]
        config.PACKAGE_MANAGER_COMMAND_DOWNLOAD = ["dnf", "install", "--downloadonly", "-y"]  # noqa: E501
        config.PACKAGE_MANAGER_COMMAND_REMOVE = ["dnf", "remove", "-y"]
        # config.PACKAGE_MANAGER_COMMAND_QUERY = ["dnf", "list", "installed"]
        config.PACKAGE_MANAGER_COMMAND_QUERY = ["rpm", "-qa"]  # workaround
        config.QUERY_PREFIX = ''
        # config.PACKAGES = "patch fuse3 fuse3-libs mod_auth_ntlm_winbind samba-winbind samba-winbind-clients cabextract bc libxml2 curl"  # noqa: E501
        config.PACKAGES = (
            "fuse3 fuse3-libs "  # appimages
            "mod_auth_ntlm_winbind samba-winbind samba-winbind-clients cabextract "  # wine  # noqa: E501
            "p7zip-plugins "  # winetricks
        )

So we need to add fuse along with fuse3? It seems strange to me that both would be needed, but it could be the case.

I'm not sure if just fuse (no fuse3) would work, I know we need fuse for fusermount

Add fuse-libs just to be safe?

"fuse" package needed for fusermount
"fuse3" package provides fusermount3

In my ubuntu 24.04 system, fusermount is a symlink to fusermount3:
$ ll /bin/fusermount
lrwxrwxrwx 1 root root 11 Apr  8  2024 /bin/fusermount -> fusermount3*

Closing this PR. It will be superseded by #232.


[docker][docs] Removed extra equals after mail user env variable

Removed extra = after MAIL_USER env variable to be consistent with other env variables.

Hi @mans2singh since we migrate our docs into apache/dolphinscheduler in PR https://github.com/apache/dolphinscheduler/pull/9291, you should change this file in https://github.com/apache/dolphinscheduler/tree/dev/docs instead of apache/dolphinscheduler-website. Do you have time to migrate it to apache/dolphinscheduler? If you need some help, feel free to ping me.

@zhongjiajie - Thanks will do that. Mans

looking forward

Thanks for the first contribution.
any progress? I will close this PR ^-^


Uncaught Error: Cannot find module 'jquery-ui/autocomplete'

[Enter steps to reproduce below:]

...
...

Atom Version: 1.8.0
System: Ubuntu 16.04
Thrown From: browser-plus package, v0.0.67
Stack Trace
Uncaught Error: Cannot find module 'jquery-ui/autocomplete'
At module.js:340

Error: Cannot find module 'jquery-ui/autocomplete'
    at Module._resolveFilename (module.js:338:15)
    at Function.Module._resolveFilename (/usr/share/atom/resources/app.asar/src/module-cache.js:383:52)
    at Function.Module._load (module.js:289:25)
    at Module.require (module.js:366:17)
    at require (/usr/share/atom/resources/app.asar/src/native-compile-cache.js:50:27)
    at Object.<anonymous> (/home/ducomin/.atom/packages/browser-plus/lib/browser-plus-view.coffee:4:1)
    at Object.<anonymous> (/home/ducomin/.atom/packages/browser-plus/lib/browser-plus-view.coffee:1:1)
    at Module._compile (/usr/share/atom/resources/app.asar/src/native-compile-cache.js:103:30)
    at Object.defineProperty.value [as .coffee] (/usr/share/atom/resources/app.asar/src/compile-cache.js:208:21)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:313:12)
    at Module.require (module.js:366:17)
    at require (/usr/share/atom/resources/app.asar/src/native-compile-cache.js:50:27)
    at HTMLEditor.module.exports.HTMLEditor.getViewClass (/home/ducomin/.atom/packages/browser-plus/lib/browser-plus-model.coffee:26:7)
    at ViewRegistry.module.exports.ViewRegistry.createView (/usr/share/atom/resources/app.asar/src/view-registry.js:135:97)
    at ViewRegistry.module.exports.ViewRegistry.getView (/usr/share/atom/resources/app.asar/src/view-registry.js:86:21)
    at atom-pane.PaneElement.itemRemoved (/usr/share/atom/resources/app.asar/src/pane-element.js:180:37)
    at Function.module.exports.Emitter.simpleDispatch (/usr/share/atom/resources/app.asar/node_modules/event-kit/lib/emitter.js:25:14)
    at Emitter.module.exports.Emitter.emit (/usr/share/atom/resources/app.asar/node_modules/event-kit/lib/emitter.js:125:28)
    at Pane.module.exports.Pane.removeItem (/usr/share/atom/resources/app.asar/src/pane.js:568:20)
    at Pane.module.exports.Pane.destroyItem (/usr/share/atom/resources/app.asar/src/pane.js:637:16)
    at atom-tabs.TabBarView.closeTab (/usr/share/atom/resources/app.asar/node_modules/tabs/lib/tab-bar-view.js:321:26)
    at Object.addElementCommands.tabs:close-tab (/usr/share/atom/resources/app.asar/node_modules/tabs/lib/tab-bar-view.js:95:26)
    at atom-tabs.commandsWithPropagationStopped.(anonymous function) (/usr/share/atom/resources/app.asar/node_modules/tabs/lib/tab-bar-view.js:86:36)
    at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (/usr/share/atom/resources/app.asar/src/command-registry.js:260:29)
    at CommandRegistry.handleCommandEvent (/usr/share/atom/resources/app.asar/src/command-registry.js:3:61)
    at CommandRegistry.module.exports.CommandRegistry.dispatch (/usr/share/atom/resources/app.asar/src/command-registry.js:160:19)
    at AtomEnvironment.module.exports.AtomEnvironment.dispatchContextMenuCommand (/usr/share/atom/resources/app.asar/src/atom-environment.js:1106:28)
    at EventEmitter.outerCallback (/usr/share/atom/resources/app.asar/src/application-delegate.js:340:25)
    at emitThree (events.js:97:13)
    at EventEmitter.emit (events.js:175:7)

Commands
  3x -4:25.9.0 core:paste (atom-text-editor.editor.mini.is-focused)
     -2:44 core:confirm (atom-text-editor.editor.mini.is-focused)
  2x -2:11.7.0 core:paste (atom-text-editor.editor.mini.is-focused)
  4x -0:32.5.0 core:cancel (atom-pane.pane.active)
     -0:16.9.0 browser-plus:open (atom-text-editor.editor.is-focused)
     -0:05.1.0 core:cancel (atom-workspace.workspace.scrollbars-visible-always.theme-one-dark-syntax.theme-one-dark-ui.seti-ui-no-icons)
     -0:01.0 tabs:close-tab (li.tab.sortable.right-clicked)

Config
{
  "browser-plus": {}
}

Installed Packages
# User
angularjs, v0.3.4
browser-plus, v0.0.67
color-picker, v2.2.2
emmet, v2.4.3
file-icons, v1.7.17
highlight-selected, v0.11.2

# Dev
No dev packages

just install browser plus and am experiencing this as well

on windows 10 the same Error.
[Enter steps to reproduce below:]

...
...

Atom Version: 1.8.0
System: Unknown Windows Version
Thrown From: browser-plus package, v0.0.67
Stack Trace
Uncaught Error: Cannot find module 'jquery-ui/autocomplete'
At module.js:340

Error: Cannot find module 'jquery-ui/autocomplete'
    at Module._resolveFilename (module.js:338:15)
    at Function.Module._resolveFilename (C:\Users\Walddys Emmanuel\AppData\Local\atom\app-1.8.0\resources\app.asar\src\module-cache.js:383:52)
    at Function.Module._load (module.js:289:25)
    at Module.require (module.js:366:17)
    at require (C:\Users\Walddys Emmanuel\AppData\Local\atom\app-1.8.0\resources\app.asar\src\native-compile-cache.js:50:27)
    at Object.<anonymous> (file:///C:/Users/Walddys Emmanuel/.atom/packages/browser-plus/lib/browser-plus-view.coffee:4:1)
    at Object.<anonymous> (file:///C:/Users/Walddys Emmanuel/.atom/packages/browser-plus/lib/browser-plus-view.coffee:1:1)
    at Module._compile (C:\Users\Walddys Emmanuel\AppData\Local\atom\app-1.8.0\resources\app.asar\src\native-compile-cache.js:103:30)
    at Object.defineProperty.value [as .coffee] (C:\Users\Walddys Emmanuel\AppData\Local\atom\app-1.8.0\resources\app.asar\src\compile-cache.js:208:21)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:313:12)
    at Module.require (module.js:366:17)
    at require (C:\Users\Walddys Emmanuel\AppData\Local\atom\app-1.8.0\resources\app.asar\src\native-compile-cache.js:50:27)
    at HTMLEditor.module.exports.HTMLEditor.getViewClass (file:///C:/Users/Walddys Emmanuel/.atom/packages/browser-plus/lib/browser-plus-model.coffee:26:7)
    at ViewRegistry.module.exports.ViewRegistry.createView (C:\Users\Walddys Emmanuel\AppData\Local\atom\app-1.8.0\resources\app.asar\src\view-registry.js:135:97)
    at ViewRegistry.module.exports.ViewRegistry.getView (C:\Users\Walddys Emmanuel\AppData\Local\atom\app-1.8.0\resources\app.asar\src\view-registry.js:86:21)
    at atom-pane.PaneElement.itemRemoved (C:\Users\Walddys Emmanuel\AppData\Local\atom\app-1.8.0\resources\app.asar\src\pane-element.js:180:37)
    at Function.module.exports.Emitter.simpleDispatch (C:\Users\Walddys Emmanuel\AppData\Local\atom\app-1.8.0\resources\app.asar\node_modules\event-kit\lib\emitter.js:25:14)
    at Emitter.module.exports.Emitter.emit (C:\Users\Walddys Emmanuel\AppData\Local\atom\app-1.8.0\resources\app.asar\node_modules\event-kit\lib\emitter.js:125:28)
    at Pane.module.exports.Pane.removeItem (C:\Users\Walddys Emmanuel\AppData\Local\atom\app-1.8.0\resources\app.asar\src\pane.js:568:20)
    at Pane.module.exports.Pane.destroyItem (C:\Users\Walddys Emmanuel\AppData\Local\atom\app-1.8.0\resources\app.asar\src\pane.js:637:16)
    at atom-tabs.TabBarView.onClick (C:\Users\Walddys Emmanuel\AppData\Local\atom\app-1.8.0\resources\app.asar\node_modules\tabs\lib\tab-bar-view.js:688:17)

Commands
     -0:24.1.0 command-palette:toggle (atom-text-editor.editor.power-mode.is-focused)
     -0:21 core:move-down (atom-text-editor.editor.mini.is-focused)
     -0:20.4.0 core:confirm (atom-text-editor.editor.mini.is-focused)
     -0:20.4.0 browser-plus:open (atom-text-editor.editor.power-mode)

Config
{
  "core": {}
}

Installed Packages
# User
activate-power-mode, v0.7.0
autoclose-html, v0.23.0
browser-plus, v0.0.67
browser-plus-zoom, v0.0.1
double-tag, v0.2.0
file-icons, v1.7.17
language-matlab, v0.2.1
minimap, v4.24.7
minimap-bookmarks, v0.3.1
minimap-cursorline, v0.2.0
minimap-find-and-replace, v4.5.1
minimap-highlight-selected, v4.4.0
minimap-selection, v4.4.0
navigate, v0.1.16
pp, v0.0.1
preview-plus, v1.2.7
tag, v0.4.0

# Dev
No dev packages

I'm experiencing the same, using Fedora 23 on a fresh install

Same

Same here

The latest update still produces the same error:
Uncaught Error: Cannot find module '../node_modules/jquery-ui/autocomplete.js'

I still experience this as well.

I've referenced a quick fix in #94 that still works after the update

did you try the latest patch.

Yes, I updated to 0.0.74 and still receive the error

@skandasoft jquery-autocomplete package should be referenced on package.json, no?

Any news on this? I've uninstalled and reinstalled browser-plus several times, am on the newest release (0.0.76) and still receive this error.

I wish some body could help with this..because it is not happening on my m/c and it is as simple as file dependency and it is there in https://github.com/skandasoft/browser-plus/tree/master/node_modules/jquery-ui .

Is there any additional information I could give you that would help? I did a global install of jquery-ui via npm too (figured it was worth a shot) but it didn't help. I am on the newest beta of Atom (1.10.0-beta7) if that matters.

In terminal, have you run apm rebuild-module-cache? I updated browser-plus on another computer and it would not open correctly. After exiting atom, running the command, and re-opening the project it worked fine

I hadn't thought to do that @jennuine. When I tried running that command it repeatedly failed with the error Cannot read property 'link' of undefined on the first package.

It sounds like something wrong with your atom installation.. Have you updated it? Also, try running apm rebuild.
If it continues to fail, you'll probably have to uninstall then reinstall atom

I use the portable version of atom so each new beta I remove everything and add a fresh copy. Last time I installed was about 4 days ago. This issue has persisted between the last several installs.
I have also run apm rebuild a couple times - that works without issue.  I'm probably going to give up on this since it's the only package not working.
Thanks for your help @jennuine.

I fixed it by copying node_modules/jquery-ui from the repo directly into /Users/<redacted>/.atom/packages/browser-plus/node_modules.
For some reason, the jquery-ui folder is missing when installing browser-plus from the Package Manager.
Atom Version: 1.10.0
Electron Version: 0.37.8
System: Mac OS X 10.11.6
Thrown From: browser-plus package, v0.0.76

That did it, it's working again! Thanks so much @jayjun 👍

@skandasoft The above fixed it for me. Do you think adding jQuery UI to the dependencies in package.json would make any difference for other still having the problem?

I am having the same issue. After reading over the thread I added "jquery-ui": "^1.12.0" to package.json then ran npm install. That did not work. I was able to successfully get it working by simply cloning the repo into ~/.atom/packages/ then running cd browser-plus && npm install. I hope that helps.

thanks @cpitkin @jennuine @jayjun @mikesprague . Closing the issue.

I ran into this issue just today, and I was able to fix it using the methods suggested within. Just leaving this here in case anyone stumbles upon it like I did.


10.0.26100.2454 Not Supported

termsrv.zip

[10.0.26100.2454]
SingleUserPatch.x64=1
SingleUserOffset.x64=9B55B
SingleUserCode.x64=mov_eax_1_nop_2
DefPolicyPatch.x64=1
DefPolicyOffset.x64=9897F
DefPolicyCode.x64=CDefPolicy_Query_eax_rcx_jmp
LocalOnlyPatch.x64=1
LocalOnlyOffset.x64=8EB71
LocalOnlyCode.x64=jmpshort
SLInitHook.x64=1
SLInitOffset.x64=AF200
SLInitFunc.x64=New_CSLQuery_Initialize
[10.0.26100.2454-SLInit]
bServerSku.x64=11EE5C
bRemoteConnAllowed.x64=11EE70
bFUSEnabled.x64=11EE80
bAppServerAllowed.x64=11EE68
bMultimonAllowed.x64=11EE74
lMaxUserSessions.x64=11EE60
ulMaxDebugSessions.x64=11EE7C
bInitialized.x64=11EE58

Thank you. It's working!

I have tried the solution proposed above but it does not work for me.

Thank you!! S2

I have tried the solution proposed above but it does not work for me.

Did you add a blank line at the end of your rdpwrap.ini file after you added the above lines?

https://github.com/sebaxakerhtc/rdpwrap.ini/commit/6dfb41400c47e962b2b48987381a01d97f3ddbbb

Thanks @bobo334! This works!

I updated .ini and shown "fully supported",  restarted several times, but unable to connect as 2nd user, mstsc shows like "the connection limitation has been used out". What should I do?


Incomplete required item list for Beneath Cursed Sands

Raw chicken doesn't show up as a valid option for meat even though it can be given to the crocodile.

Fixed with 13bc545f477ebb627e34d0b246e56e5bafa124c0


netcore image: Setting locale failed

perl : warning : Setting locale failed. [/app/src/Domain.fsproj]
perl : warning : Please check that your locale settings: [/app/src/Domain.fsproj]
  	LANGUAGE = (unset),
  	LC_ALL = "en_US.UTF-8",
  	LANG = "en_US.UTF-8"
      are supported and installed on your system.

Many follow-up errors come:
  Something went wrong while downloading System.Runtime.InteropServices 4.3
  Message: Error during extraction of /app/packages/System.Runtime.InteropServices/system.runtime.interopservices.4.3.0.nupkg.

For example.

The project is F# only with a few references. I can screen share if you want... Skype? h.feldt

Sure, just give me a ping and I'll showcase it
On 31 May 2018, at 15:48, Dave Curylo wrote:

I'm not able to Skype right now on this network, but can a bit later.
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/fsprojects/docker-fsharp/issues/41#issuecomment-393535803

Fix with
    apt-get --no-install-recommends install -y locales autoconf libtool pkg-config make automake nuget mono-devel msbuild ca-certificates-mono && \
    rm -rf /var/lib/apt/lists/* && \
    echo 'en_US.UTF-8 UTF-8' > /etc/locale.gen && /usr/sbin/locale-gen && \

Yes, this is copy-n-paste from your dockerfile; the only change is adding locales and then generating them.

Thank you for sorting that one!


Add NOPD Case Number field to Incident Details

User Story
As a user in New Orleans
I want to be able to add the "NOPD Case Number" to a case
So that it makes it easier to investigate cases
Business Context:
In the Incident Details page, we want to see a new field called "NOPD Case Number". This should be editable in the Edit Incident Details screen. It should also show up in the Referral Letter.
In Scope

Add "NOPD Case Number" field in Edit Incident Details and Incident Details (for context, NOPD Case #s typically look like: A-12345-24)
Add this to referral letter in the "Incident Details" section underneath date, time, location (which are currently displayed)

Out of Scope

Open Questions
Mockups

Acceptance Criteria

Scenario:
Given that I have entered or edited the "NOPD Case Number" field
When I click "Save"
Then the information is saved to this incident
And shows up in the referral letter once it is generated for a new case

Notes

xxx

Checklist
Kickoff

[ ] We have we indicated which organizations and user permissions require this
[ ] We have indicated whether we have any dependencies to other cards/work
[ ] We have indicated which environment this can be tested in
[ ] We have included a mockup if this is design related
[ ] We have included clear ACs to test it
[ ] We have estimated the story

Deskcheck

[ ] This work has been demoed in CI (or higher) and led by the dev pair who worked on this card
[ ] We have run through and passed all the ACs
[ ] Our documentation has been updated
[ ] Our onboarding checklist has been updated

Will require a new field in the cases table (migration), will need to alter the letterBody template (probably using a seeder)


Resolve conflicted TimeEntry in Timeline

📒 Description
This PR will introduce the logic the resolve the conflict of Timeline entry.
🕶 Types of changes
New feature (non-breaking change which adds functionality)
🤯 List of changes

[x] Introduce new methods in library to override exact Start/End time (unix timestamp) of given GUID. The Context::SetTimeEntryStop and Context::SetTimeEntryStart doesn't work well since it only overrides the hour and minute (The second remains)
[x] Logic to resolve the conflicts

👫 Relationships
Close #3187
🔎 Review hints
Reproduce from https://github.com/toggl/toggldesktop/issues/3187#issuecomment-519121568

It's ready to review @skel35. Thank you in advance 👍

@skel35 @MartinBriza so it's ready for reviewing 👍


[modeling] Add SSD model on dev branch

Add SSD model on dev branch
Github repo: ssd_keras by @pierluigiferrari

WIP : Converting a model
Goal
(boxes, scores, labels) = model.predict_detection('sample.jpg')
Working on

test_load_weights.py
keras_ssd300.py
ssd_detection.py
keras_layer_DecodeDetections.py

Tasks

[ ] model = ssd_300() is for training
[ ] When model.predict_detection() is called, the DecodeDetections layer will be added at the end of the model.

To run the test file, the pretrained model weights are needed. Download files from here, and put them in KerasObjectDetector/keras_ssd/pretraind/ folder.

Starting to add SSD model with @hngskj

Use the weights below.
PASCAL VOC models: 07+12: SSD300*, SSD512*


on running FirebaseAuth.instance.signOut();   throwing the error "Failed assertion: line 3803 pos 12: '_history.isNotEmpty': is not true."

Steps to Reproduce

Run flutter create bug.
Update the files as follows: ...
...

Expected results:
Actual results:

Logs

Hi @useit4me
From the little I can see, the issue is related to a FlutterFire plugin rather than to Flutter itself. Please open the issue in the dedicated repository.
Closing, as this isn't an issue with Flutter itself.
If you disagree, please write in the comments,
providing your flutter doctor -v, your flutter run -v,  your pubspec.yaml
a minimal reproducible code sample that does not use 3rd party plugins,
and I will reopen it.
Thank you


Change StylePart to StringStyle.Part

Instead of having StylePart as a top-level enum, nest it in an extension of StringStyle:
extension StringStyle {

    public enum Part {
        ...
    }

}

What do people think? Comment below!

Very much in favor of this.

Yea, +1!

I'll take this.

Closed by https://github.com/Raizlabs/BonMot/commit/65429d7b6b41ede68192001e09e0e0da42527eee


VHD commands not found on Windows 10

HI, I'm facing disk-related issues with WSL2 and am trying to use Powershell to compact the Virtual Hard Disk and return storage from WSL2 to Windows.
I am trying to use the following commands -
wsl --terminate ubuntu-18.04
Optimize-VHD -Path .\ext4.vhdx -Mode full

However, the command is not present on my version of Windows 10 (Home) and Powershell (v5.1). I get the following error -
Optimize-VHD : The term 'Optimize-VHD' is not recognized as the name of a cmdlet, function, script file, or operable
program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.
At line:1 char:1
+ Optimize-VHD ext4.vhdx
+ ~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (Optimize-VHD:String) [], CommandNotFoundException
    + FullyQualifiedErrorId : CommandNotFoundException

From what I read this command is only available on the Windows 10 (Pro) Edition. Is there a way for me to use this functionality to solve my issue without having to buy the Pro version?

That error message, which is pretty clear, says PowerShell can not find the command Optimize-VHD.
This command is found in the Hyper-V module.
So - can you Import-Module -Name Hyper-V?
And if not have you loaded the RSAT tools for Windows 10.  Seehttps://www.microsoft.com/en-gb/download/details.aspx?id=45520 and go from there.

That error message, which is pretty clear, says PowerShell can not find the command Optimize-VHD.
This command is found in the Hyper-V module.
So - can you Import-Module -Name Hyper-V?
And if not have you loaded the RSAT tools for Windows 10.  Seehttps://www.microsoft.com/en-gb/download/details.aspx?id=45520 and go from there.

That error message, which is pretty clear, says PowerShell can not find the command Optimize-VHD.
This command is found in the Hyper-V module.
So - can you Import-Module -Name Hyper-V?
And if not have you loaded the RSAT tools for Windows 10. Seehttps://www.microsoft.com/en-gb/download/details.aspx?id=45520 and go from there.

Thanks, I tried both of these, but it seems that Hyper-V is not available using either with the Windows 10 (Home) Edition (only with Enterprise, Pro, or Education). It seems weird to allow WSL2 on the Home edition, but not the Hyper-V tool required to manage the partitions created from using the former.

That error message, which is pretty clear, says PowerShell can not find the command Optimize-VHD.
This command is found in the Hyper-V module.
So - can you Import-Module -Name Hyper-V?
And if not have you loaded the RSAT tools for Windows 10. Seehttps://www.microsoft.com/en-gb/download/details.aspx?id=45520 and go from there.

Thanks, I tried both of these, but it seems that Hyper-V is not available using either with the Windows 10 (Home) Edition (only with Enterprise, Pro, or Education). It seems weird to allow WSL2 on the Home edition, but not the Hyper-V tool required to manage the partitions created from using the former.

It seems that this question can best be answered via PowerShell Community Links
For bug reports specific to PowerShell, please visit, PowerShell at GitHub.
The issues section of this repository is for content fixes regarding documentation. Feel free to re-open in case you would like to suggest improvements to the article itself.

It seems that this question can best be answered via PowerShell Community Links
For bug reports specific to PowerShell, please visit, PowerShell at GitHub.
The issues section of this repository is for content fixes regarding documentation. Feel free to re-open in case you would like to suggest improvements to the article itself.


jQuery selector, explicit path required

Hello @AlfonsoFilho,
Calling ClipPath on a jQuery object needs an explicit declaration of the path, e.g. in the example it's correct, but in the readme it's a bit misleading:
$('.image').ClipPath({path:'5% 5%, 100% 0%, 100% 75%, 75% 75%, 75% 100%'});
instead of
$('.target').ClipPath('5% 5%, 100% 0%, 100% 75%');
Thanks.

Apologies for that. In addition, instead of simply correcting the readme, I changed the ClipPath contract on jQuery implementation. So you should use a string. But this is not a breaking change. Use of object with path property still supported but deprecated.


docs: add workaround to use unix sockets with the nixos module

I might write a NixOS module in the future if I find the motivation but I don't want to promise anything.

I'll still go ahead and merge it since it obviously needs to make it into a NixOS release and stuff, but someone has thankfully finally made a NixOS module: https://github.com/NixOS/nixpkgs/pull/353651
So further reviews of it would be appreciated.


Updated translation-status.md [skip ci]

The statuses are not quite right. We will update them when ready. Thanks.


Shouldn't we be checking all files in the repo?

This occurred to me when I was adding the diff-only option. If diff-only is false, then shouldn't we be examining all source files in the repo that use one of the designated file extensions? Currently, we are only examining files that have been changed despite what diff-only option is set to.
However, we can only do this if the repo is checked out before this action is run, but there's the concern of clever people also checking out their repo's submodules (which shouldn't be examined by this action). In case the repo is setup to use submodules, we can use the .gitmodules file to ignore any existing submodules. This wouldn't be hard at all since the .gitmodules file uses an ini file syntax which python's std configparser module can handle nicely.
I suppose we can use the REST API to list all files in the repo, but the REST API's responses are limited to 30 entries per page. If the repo has more than 30 files, then we'd have to subsequently keep making REST requests until we have a list of all the files in the repo. It would definitely be easier if the repo was checked out beforehand, and we use python to "walk" through the repo's files and examine any applicable source files.

It’s best to help users check or output prompts to tell them that they need to have uses: actions/checkout@v2 to checkout repo before running cpp-linter-action. Otherwise, they may also encounter the same problem as me in #24.
If their code has a submodule or subtree, they should also check out if they need it. cpp-linter-action only focuses on the work of clang-format and clang-tidy.
So if the repo was checked out beforehand, there is no need to download the source code through the REST API, right?

It’s best to help users check or output prompts to tell them that they need to have uses: actions/checkout@v2 to checkout repo before running cpp-linter-action.

I did alter the example yml in the readme with a comment about this. I do like the idea of having something in the log about non-existent files. Currently, all we're logging about this is
INFO:CPP Linter:Downloading file from url: https://github.com/shenxianpeng/test-cpp-linter-action/raw/fd6ec9c55cf93df9e01dad042f80e7835fc2fa5f/demo.cpp

but it should say more to make the user aware that the action is trying to recover from no git checkout.

So if the repo was checked out beforehand, there is no need to download the source code through the REST API, right?

True, but it doesn't hurt too much to leave it in as a recourse.

If their code has a submodule or subtree, they should also check out if they need it. cpp-linter-action only focuses on the work of clang-format and clang-tidy.

Checking out submodules is often done for complex projects' build CI. Furthermore, the compiler database file that clang-tidy looks for is generated at build time (ei using cmake -bbuild .). If we do look in the user's repo for all source files using one of the specified extensions, then this action will inadvertantly crawl any submodule(s) that were also checked out. If this issue becomes a desired feature, then I'd use the .gitmodules file (something generated and preferably maintained by git bash commands) to ignore any source files in the checked out submodules. Did I understand your comment correctly?

example of a .gitmodules file
[submodule "RF24"]
	path = RF24
	url = https://github.com/nRF24/RF24.git
[submodule "RF24Network"]
	path = RF24Network
	url = https://github.com/nRF24/RF24Network.git
[submodule "RF24Mesh"]
	path = RF24Mesh
	url = https://github.com/nRF24/RF24Mesh.git
[submodule "pybind11"]
	path = pybind11
	url = https://github.com/pybind/pybind11.git

The paths are relative to the repo's root (which can be specified using the input option repo-root).

I added a conditional warning in the log about missing files when no git checkout was used.

If their code has a submodule or subtree, they should also check out if they need it. cpp-linter-action only focuses on the work of clang-format and clang-tidy.

I mean we don't need to consider submodules, users should consider it by themselves. The working process of cpp-linter-action is to have a complete code, then to analyze the specified code changes and post results comments.
I don't know if I misunderstand your comment about submodules.

I added a conditional warning in the log about missing files when no git checkout was used.

Tested and saw the warning looks good in log

Yeah I think there's a misunderstanding. I'm trying to say the same thing you are: Don't pay attention to submodules. Although I keep going into detail about how to identify if a path in the repo is a submodule (using .gitmodules).
The important thing I'm questioning is:
Should we examine all source files or only the files that changed?
Currently, we are only examining the files that changed.

set the diff-only option to "true" if examining only changed files
set the diff-only option to "false" if examining all applicable source files

I think I'll start another branch on my fork to see what this looks like.

I think I see. most CI checks are only checking the changed files, such as SonarQube Quality Gates. I would like to set diff-only option to "true" by default.
In my case examining only changed files is good, if the codebase needs format I'd like to do a bulk change commits before, then introduce an action for future changed files examination.

The way I have it setup now (in master-py branch): If we set the diff-only option to "true" it will only checks the lines (of the changed files) shown in the diff. Do you still want to check the entire file if diff-only is true?

Oh... diff-only refers to lines, not files.
I tested diff-only to "true" in the test-cpp-linter-action repo, compared with diff-only is "false", I don't see any different output, maybe the test code is not enough. so I'm not sure if need to changed diff-only option to "true", I follow your opinion.

The diff-only option was meant as a entrypoint to posting comments directly on the line in the file (#11), but it may not be as useful as I thought. The difference between lines and files does seem confusing. Maybe I should rename that to changed-lines-only (or diff-lines-only) instead. Then, I could add an option called changed-files-only. Can you think of better names? (I seem terrible with naming things).

In GitHub, the diff files button called "Files changed", maybe we name them files-changed-only and lines-changed-only?
Your naming has always been very professional, and my pleasure to accept any changes.

I like your rationality. I'm going with your name ideas. TBC...

I'm close to submitting a PR, but I have to refine handling the new ignore option. There may be a bug concerning the root folder of the repo. And, I'd like to add a possible syntax that allows excluding an entire folder except for a specific file/folder within the excluded folder.
My current idea:
      - uses: actions/checkout@v2
      - uses: shenxianpeng/cpp-linter-action@master
        id: linter
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
        with:
          style: file
          # search entire repo for source files
          file-changed-only: false
          # ignore everything in demo folder except the demo.cpp file. Notice the bang (`!`) prefix
          ignore: |
            # yml comments are not recognized here
            demo
            !demo/demo.cpp

You can find my progress in this latest run

I tried out using the new log commands to implement in-file comments, but I found some differences compared to using the REST API to post comments.

The message's body can't be multiple lines.
The message looks different in the file's diff compared to the message in the workflow's summary.

However, despite these differences, it is much easier (and even faster) to use log commands.
What you think?

Do you want to keep the use of log commands to make comments?
Do you want to keep the use of REST API to make comments?
Do you want to use both log commands and REST API to make comments?

oh I forgot about the retention period! We shall have both then. PR coming soon...

The link to super-linter example is a year old, but the annotations still exist 👍🏼

If there are both, it might be better to have a switch to control log command and REST API comment on or off.

This thought had occured to me also. How about a new option called thread-comment? If it set true, comment will appear in the thread (PR or commit). Otherwise no comment in the thread.
I also think we should switch away from using a table in the readme to document the input options. They are getting numerous, and the ignore option needs its own list to fully explain the capabilities.

Do you mean to call  https://github.com/2bndy5/cpp-linter-action/commit/7ce1dd61bf0c5c33a2e18e2c0ee8df75695d9eb1 as thread-comment?, not sure if it is a commit comment, my understand below is a thread comment.

If there are other better ways to show input options, please do it. Btw, I still like using readme to display the basic usage.


Request an example of constructing a VTK object containing point/cell data in Legacy style

Hello elrnv!
Thank you three contributors for your work！I'm learning to use vtkio and it feels great!
After studying the examples in the readme, I already know how to construct a VTK object(with default attributes) using vtkio, and bufferd write it into a .vtk file. Here is my Rust file:
test_vtkio.rs :
use std::io::{BufWriter, Write};
use vtkio::model::*;

// The whole idea：
//    1. Construct a String to store the vtk object;
//    2. Construct a VTK object containing data;
//    2. Write VTK object into String;
//    3. Write the String into the .vtk file.

fn main() {
    // Prepare a file for loading vtk content
    let file_path = "./test2.vtk";
    let vtk_file = std::fs::File::create(file_path).unwrap();
    let mut vtk_writer = BufWriter::new(vtk_file);

    // Prepare a String to hold vtk file information
    let mut vtk_strings = String::new();

    // Construct the content of the vtk file
    let tri2 = make_triangle_legacy();

    // Write the content of the vtk file into String
    tri2.write_legacy_ascii(&mut vtk_strings)
        .expect("Write vtk info into bytes failed!");

    // Print out the written string
    println!("{}", vtk_strings.as_str());

    // Write the String into the .vtk file
    write!(vtk_writer, "{}", vtk_strings.as_str()).expect("Write VTK error!");
}

fn make_triangle_legacy() -> Vtk {
    Vtk {
        version: Version { major: 4, minor: 2 },
        title: String::from("Rust vtk io crate test."),
        byte_order: ByteOrder::BigEndian,
        file_path: None,
        data: DataSet::inline(UnstructuredGridPiece {
            points: IOBuffer::F64(vec![
                0.0, 0.0, 0.0, 1.0, 0.0, 0.0, 0.0, 1.0, 0.0, 1.0, 1.0, 0.0,
            ]),
            cells: Cells {
                cell_verts: VertexNumbers::Legacy {
                    num_cells: 2,
                    vertices: vec![3, 0, 1, 2, 3, 3, 2, 1],
                },
                types: vec![CellType::Triangle; 2],
            },
            data: Attributes {
                ..Default::default()
            },
        }),
    }
}

And the above code will generate a vtk file: test2.vtk.  Its contents are:
test2.vtk：
# vtk DataFile Version 4.2
Rust vtk io crate test.
ASCII

DATASET UNSTRUCTURED_GRID
POINTS 4 double
0 0 0 1 0 0 0 1 0 1 1 0

CELLS 2 8
3 0 1 2 3 3 2 1

CELL_TYPES 2
5
5

POINT_DATA 4

CELL_DATA 2

The test2.vtk file can be well loaded and visualized by paraview without any data：

However I'm having trouble writing Dataset attributes to a .vtk file in Legacy style using vtkio. I don't know how to write the following point_data into the test2.vtk file in Legacy style,
POINT_DATA 4
VECTORS displacement float
0 0 0
0.5 0 0
0 0 0
0.5 0 0

Please teach me how to construct a VTK object containing point/cell data, thanks a lot！

Come on folks! any help?

Hi @hunter1992,
You just need to add a non-default Attributes struct. In your example, this will be something like
...
            data: Attributes {
                point: vec![
                    Attribute::vectors("displacement")
                       .with_data(vec![0.0f32, 0.0, 0.0, 0.5, 0.0, 0.0, 0.0, 0.0, 0.0, 0.5, 0.0, 0.0]),
                ],
                ..Default::default()
            },
...

There is more documentation about how to create attributes here: https://docs.rs/vtkio/latest/vtkio/model/enum.Attribute.html .
The best place to see examples of how to create vtk files take a look at the tests subdirectory. For instance, there is an example of how to create a vector attribute here.
Hope this helps!


KAFKA-8897: Warn about no guaranteed backwards compatibility in RocksDBConfigSetter

Kafka Streams cannot guarantee backwards compatibility of code in
a user defined RocksDBConfigSetter since it does not control backwards
compatibility of RocksDB. RocksDB may remove methods without deprecating
them as it has already done in the past with CompactionOptionsFIFO#setTtl().
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Call for review: @mjsax @ableegoldman

Updated this PR. Call for review @ableegoldman @vvcephei

Java 11 / 2.12: kafka.admin.ReassignPartitionsClusterTest.shouldPerformMultipleReassignmentOperationsOverVariousTopics
Java 11 / 2.13 and Java 8 passed.
Retest this please.

Java 8 passed. Both Java 11 failed -- results are not available any longer.
Retest this please.

Merged to trunk and cherry-picked to 2.4.


Unauthorized error

Hello, I am trying to send data from OPCUA Server to InfluxDB 2.0 using your plugin, but I am getting an unauthorized error. First I thought that I used wrong token, which didn't have needed permissions but I checked that and I don't see any problem here. So I would be really thankfull if you could help me.
Here is log containing error message
[2020-10-12T13:28:13.747] [INFO] main - Starting Influx OPCUA Logger v2.0.0-alpha, brought to you by FACTRY (www.factry.io)
[2020-10-12T13:28:13.750] [INFO] main - Initialising influxClient
[2020-10-12T13:28:13.822] [INFO] influx - <IP_ADDRESS>:8086 responded in 63ms running
[2020-10-12T13:28:13.832] [INFO] main - Initialising buffer
[2020-10-12T13:28:13.838] [INFO] main - Connecting OPCUA
[2020-10-12T13:28:13.842] [INFO] buffer - Start processing of bufferfile { file: '/home/data/_queue/1602493702208_0.json' }
[2020-10-12T13:28:13.866] [ERROR] influx - A 401 Unauthorized error occurred: {"code":"unauthorized","message":"unauthorized access"}
[2020-10-12T13:28:13.867] [ERROR] buffer - Error writing points { error:
'A 401 Unauthorized error occurred: {"code":"unauthorized","message":"unauthorized access"}' }
[2020-10-12T13:22:01.403] [INFO] opcuaclient - Established connection. { Endpoint: 'opc.tcp://<IP_ADDRESS>:12345' }
[2020-10-12T13:22:01.484] [INFO] opcuaclient - Established session with server. { ID: 26 }
[2020-10-12T13:22:01.486] [INFO] opcuaclient - Added Polled Metric. { measurement: 'U-L1L2[V]',
tags: { simulation: 'true', location: 'Levice' },
nodeId: 'ns=4;s=Sensor_Simulator:InputRegisters:0:0' }
[2020-10-12T13:22:01.486] [INFO] opcuaclient - Added Polled Metric. { measurement: 'U-L2L3[V]',
tags: { simulation: 'true', location: 'Levice' },
nodeId: 'ns=4;s=Sensor_Simulator:InputRegisters:1:0' }
[2020-10-12T13:22:01.487] [INFO] opcuaclient - Added Polled Metric. { measurement: 'U-L3L1[V]',
tags: { simulation: 'true', location: 'Sala' },
nodeId: 'ns=4;s=Sensor_Simulator:InputRegisters:2:0' }
[2020-10-12T13:22:01.520] [INFO] opcuaclient - Installed subscription on session. { ID: 84 }
[2020-10-12T13:22:02.397] [INFO] buffer - Start processing of bufferfile { file: '/home/data/_queue/1602493702208_0.json' }
[2020-10-12T13:22:02.403] [ERROR] influx - A 401 Unauthorized error occurred: {"code":"unauthorized","message":"unauthorized access"}
[2020-10-12T13:22:02.404] [ERROR] buffer - Error writing points { error:
'A 401 Unauthorized error occurred: {"code":"unauthorized","message":"unauthorized access"}' }
And here is my config file
[opcua]
url = "opc.tcp://<IP_ADDRESS>:12345"
[influx]
url = "http://lukas:password@<IP_ADDRESS>:8086/snmpbucket"
token = "oI67t9BXnHnoYtWRl7YBen9pbZGQngrFje39gBr1WC8TMPk46gclqPFGOS3OK_-eb84EUOg_zRPJth74hp1Paw=="
org = "myorg"
bucket = "snmpbucket"
writeInterval = 10000

That seems to be an error coming from the influx client library itself. Are you certain the token is valid, and provides access to the bucket you want to write to?

Thanks for the reply, yes I tried creating two new tokens one with access just to the bucket I want to write and when that didn't work, I tried using one that has access to everything in Influx and I am still getting same error and nothing is sent to database.

Can you access the database from grafana or chronograph using those credentials?

hi, i am still stuck at this problem. i am getting the same errors. With the same credentials in the config file, i am able to access database in Grafana. but, unable to send data to the database from the logger.


Does not fit more than ten options

hello, i want to add more than 10 options but i whenever i tried to add it truncate middle can you please help me to figure-out this issue.

Hi,
I will check it out and update you.
Thank you.
On Thu 5 Mar, 2020, 1:29 AM krutitrivedi<EMAIL_ADDRESS>wrote:

hello, i want to add more than 10 options but i whenever i tried to add it
truncate middle can you please help me to figure-out this issue.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/malekwasim/WMSegmentControl/issues/3?email_source=notifications&email_token=AGYR3TPL6I226OW4C55ZP2LRF2XLRA5CNFSM4LBR7ZD2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4ISQ36YQ,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGYR3TOEOPFIMCDNMS2KX6LRF2XLRANCNFSM4LBR7ZDQ
.

Hi,
This control was created to replace the default segment control. So it
doesn't have the scroll option.
However if you want multiple options, you can use the collection view for
your purpose.
If you need any help please let me know.
Thank you.
On Thu 5 Mar, 2020, 6:49 AM Wasim Malek<EMAIL_ADDRESS>wrote:

Hi,
I will check it out and update you.
Thank you.
On Thu 5 Mar, 2020, 1:29 AM krutitrivedi<EMAIL_ADDRESS>wrote:

hello, i want to add more than 10 options but i whenever i tried to add
it truncate middle can you please help me to figure-out this issue.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/malekwasim/WMSegmentControl/issues/3?email_source=notifications&email_token=AGYR3TPL6I226OW4C55ZP2LRF2XLRA5CNFSM4LBR7ZD2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4ISQ36YQ,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGYR3TOEOPFIMCDNMS2KX6LRF2XLRANCNFSM4LBR7ZDQ
.


DCOS-40495  Nodes Table Region Column shows "N/A (Local)"

Remove the (Local) addition when the local region is "N/A"
jira: https://jira.mesosphere.com/browse/DCOS-40495
Testing
In Nodes tab, in Region column, if there is no local region, the name should be shown just as N/A instead of N/A (Local).
Alternatively, to test it, try replacing the function regionRenderer in /plugins/nodes/src/js/columns/NodesTableRegionColumn.tsx the following way:
export function regionRenderer(
  masterRegion: string,
  data: Node
): React.ReactNode {
  data;
  masterRegion = "N/A";
  const regionString = "N/A";
  const regionName =
  regionString +
    (masterRegion === regionString && masterRegion !== "N/A"
      ? " (Local)"
      : "");

  return (
    <TextCell>
      {regionName}
    </TextCell>
  );
}

Trade-offs
None that I am aware of.
Dependencies
None.
Screenshots
Before

After

ping @weblancaster

:tada: This PR is included in version 2.12.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Shenlei151 flexibility enhance

Fixes AI4Finance-Foundation/FinRL#481
I adjust a few lines of code in FinRL to make it more flexible for problems that I met with in my own project, though these changes may be aiming for my project, I think it might be useful to extend the feature of FinRL including:
feature extension:

support different costs_pct with different stocks(fix it by modifying finrl/finrl_meta/env_stock_trading/env_stocktrading.py function init)
able to hold stocks when initial instead of only cash(fix it by modifying finrl/finrl_meta/env_stock_trading/env_stocktrading.py function init)
support to define time that trading is forbidden(fix it by modifying finrl/finrl_meta/env_stock_trading/env_stocktrading.py function _buy_stock and function _sell_stock)
able to get state of everyday when trading(fix it by adding finrl/finrl_meta/env_stock_trading/env_stocktrading.py function save_state_memory and finrl/drl_agents/stablebaselines3/models.py function DRL_prediction)
bug:
5.fix bug that when buying stocks it didn't consider conmmision, which will cause the cash < 0 (fix it by modifying finrl/finrl_meta/env_stock_trading/env_stocktrading.py function _buy_stock)
These features are needed in my project, I hope it will help enhance the flexibility of FinRL
Also, I put part of my project code to explan why and how to use the features that I add in RL_stock.py, I hope it will help you understand the changes I made.

Thanks for your codes.
It's great. I suggest to add type hint in all functions. In the new version, type hint will be added.

@shenlei515  some research team contacted me about errors, they have a submission ddl, so I reverted this PR.

@shenlei515 some research team contacted me about errors, they have a submission ddl, so I reverted this PR.

Yes, I see some errors caused by my changes, I think my change only apply to my situation, maybe it need some modifications to be adapted for other situations. I am sorry for the result caused by my unreasonable PR.

It is OK, not a big issue.
We may need to make the codes more standard so that everybody can contribute more easily.

@shenlei515
Could you please describe the function of RL_stock.py? How to use it?
Thanks you very much.
In addition, I think RL_stock.py is not suitable to be in the current folder.

What's the length of initial_list? What does initial_list mean?

@zhumingpassional
RL_stock is a example to explain how to use the changed RLlib and why I need to modify the RLlib
First element of initial_list is the cash we have in the beginning, rests of it are the share of stocks we have at first. So the length of initial_list is supposed to be 1+stock_dimension.

Thanks for your reply.
I suggest initial_list is seperated to two parts (parameters), initial_amount and num_stock_shares, since they have different units (money, and num of shares).

@shenlei515
Thanks for your reply.
I suggest initial_list is seperated to two parts (parameters), initial_amount and num_stock_shares, since they have different units (money, and num of shares).

Yes, that make more sense.

@shenlei515  Could you please push a PR to revise it if you have enough time? Only for this point (initial_list is seperated to two parts).

I have pulled a request to revise it.

Merged. Thanks for your valuable codes.


Preview

👷 Deploy Preview for timely-brigadeiros-dd9fc4 processing.

Name
Link

🔨 Latest commit
1bec4cb70ccc0ad4c882991f50b40bdaa454a7d3

🔍 Latest deploy log
https://app.netlify.com/sites/timely-brigadeiros-dd9fc4/deploys/64d247c6922ece00079069d7


refactor(rust): 'tcp connection delete' command to  use 'rpc' abstraction

Current Behavior
'tcp connection delete' command currently uses the 'connect_to' function, and calls std::process::exit , which needs to be changed as mentioned in #3594

Proposed Changes

Refactored the tcp connection delete command by using Rpc utils, as proposed by #3594

Replaced connect_to by node_rpc abstraction.
Added a test to create a node and then delete it in command.bats, to test the functionality of refactored delete command.
3 Removed now unused functions connect_to , parse_response and delete_tcp_connection.

Checks

[x] All commits in this Pull Request are signed and Verified by Github.
[x] All commits in this Pull Request follow the Ockam commit message convention.
[x] I accept the Ockam Community Code of Conduct.
[ ] I have accepted the Ockam Contributor License Agreement by adding my Git/Github details in a row at the end of the CONTRIBUTORS.csv file in a separate pull request to the build-trust/ockam-contributors repository.

Looks great @h20220145 👍  Could you please submit a PR adding your Git/Github details in a row at the end of the CONTRIBUTORS.csv file to the build-trust/ockam-contributors repository?


Wrong translation for leisure=resort in french

Hi,
Current translation of resort in french is "Relaxation" but it's wrong , resorts are not used only for relaxation, but also used for recreations and accommodations, correct traduction is "Complexe touristique" or "Complexe hôtelier".

Thanks for the report @abdeldjalil09..  Our translations are handled through Transifex (link to French translation) and you can fix this yourself if you sign up for an account there.

But i can't find "Resort"

Yes Transifex UI is kind of confusing.. Does this link work?
https://www.transifex.com/ideditor/id-editor/viewstrings/#fr/presets/101404809?q=Resort

Yes, thanks


libbde: 20210605 -> 20220121

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/libbde/versions.
meta.description for libbde is: "Library to access the BitLocker Drive Encryption (BDE) format"
meta.homepage for libbde is: "https://github.com/libyal/libbde/"
meta.changelog for libbde is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 20220121 with grep in /nix/store/x6x1kc62w2nnfxi4fa09n34fjv4lz2bx-libbde-20220121
found 20220121 in filename of file in /nix/store/x6x1kc62w2nnfxi4fa09n34fjv4lz2bx-libbde-20220121

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
libbde

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/x6x1kc62w2nnfxi4fa09n34fjv4lz2bx-libbde-20220121 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/x6x1kc62w2nnfxi4fa09n34fjv4lz2bx-libbde-20220121
ls -la /nix/store/x6x1kc62w2nnfxi4fa09n34fjv4lz2bx-libbde-20220121/bin

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

1 package built:

libbde

Maintainer pings
cc @eliasp for testing.

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

1 package built:

libbde


Adding Custom OnScrollListener forcing the section header scrolling

Hi,
I have added my custom OnScrollListener which is AbsListView.OnScrolllistener on HeaderListView ,because its own onscrolllistener is required for minimum Api LEVEL 23(my minimum is 15).
It is forcing the section  headers scrolling.I want to fix the sectional headrers.Kindly give me some advice.
Below is my custom implementation:-
MyClass implements AbsListView.OnScrollListener
{
@Bind(R.id.receipt_list)
HeaderListView mReceiptsList;
    mReceiptsList.getListView().setOnScrollListener(this);

@Override
public void onScrollStateChanged(AbsListView view, int scrollState) {
this.currentScrollState = scrollState;
this.isScrollCompleted();//the method to detect scroll completion
}
@Override
public void onScroll(AbsListView view, int firstVisibleItem, int visibleItemCount, int totalItemCount) {
    this.currentFirstVisibleItem = firstVisibleItem;
    this.currentVisibleItemCount = visibleItemCount;
    this.totalItemCount = totalItemCount;
}

   /***By this methos I detect if there's been a scroll which has completed***/
private void isScrollCompleted() {
    if (this.currentVisibleItemCount > 0 && this.currentScrollState == SCROLL_STATE_IDLE && this.totalItemCount == (currentFirstVisibleItem + currentVisibleItemCount)) {

        if(canLoad) { //class variables to manipulate the data
            pageNumber =pageNumber +1;
            refreshScreen();
        }
    }
}

}

I have had a similar problem I think. It worked more like expected for me when I set the listener directly on the HeaderListView.
mReceiptsList.setOnScrollListener(this);


Update the hasonefield package

This is fixed in #43 . I went with that one because it maintains backwards compatibility with SS4. Thanks to @jsirish (and everyone else who sent this in!). Cheers anyway @KallivdH :+1:


Canceled rentals still show a balance due

Some rentals should have their balance adjusted to zero when canceled others should keep the balance (depending on cancellation time)

closed by #282


[Gush org] Remove priority labels for issues and pull-requests

Taking this from the ZeroMQ C4: https://hintjens.gitbooks.io/social-architecture/content/chapter1.html#self-organization

In ZeroMQ, we removed all assigned tasks from the community. For example,
we don't accept feature requests. If someone wants a feature,
they either send us a patch, or offer someone money to make the change,
or they wait. This means people only make changes they really need to make.

In the Gush organization have various priority labels (critical, high, nice to have) to indicate the importance, but there is a major problem with this system.
It only indicates the importance for that person, for example #500 is marked critical (since late 2015), and feature requests are another beast (why is this marked 'low priority'? I "need" it now!).
An even better example is the BitBucket-site issue tracker, multiple issues were marked critical for over 5 years! If something is critical is should be fixed right now, not tomorrow or two hours from now.
Plus, in the past some issues were marked critical while they were a simple user error.
You get the point... it gives a wrong impression and false information.
Instead we can remove remove all priority labels and thread all issues as equal.
For features we follow a simple rule: "If you want a feature, either send a patch (pull request), offer someone money to make the change, or wait. Remember this is a community-driven project, not everyone needs the same features".

I couldn't find where is the priority in the code you spoke about. I think this is not that critical 😊, because i think is more about the use case, and how admins use it. It is ok to have more important issues than others. And if gush is already supporting this then it should be fine, i don't remember using them.

I'm talking about the Gush repository 😜 https://github.com/gushphp/gush/labels

I do pay attention to those labels. And if you label it critical i know it is important for you 👍 . That is good and is communication between us more than anything.


CounterDev analytics not working

Expected Behavior

I have configured counterDev but the site don't seem to be able to track anything from my visits to the site.
counterDev seems to expect a very simple string like

But the generate code is much more complex, like:

Current Behavior

Possible Solution

Steps to Reproduce

Screenshots

Browsers Affected

[ ] Chrome
[x] Firefox
[ ] Edge
[ ] Safari 11
[ ] Safari 10
[ ] IE 11

Detailed Description

Thank you @SrNetoChan for reporting the issue. I will take a look.

If it is of any helps, my site is www.aneto.pt

@SrNetoChan Look like counter.dev has changed how the tracker is included in the site. This PR should fix the issue: #674.
Please update your theme to the latest commit and  let me know if it fixes the issue.

That was my thinking. But I wasn't sure if you were just doing it in another way.
Thanks for the fix!


Fix H5 slice indexing during H5 reading

Make sure that the slice read from the H5 file is correctly indexed depending on the XY/YX axis order.

Fixes https://github.com/ilastik/ilastik4ij/issues/22


Added '.xz' extension in HTTP download.

Hi @smira ,
I recently tried to mirror debian/experimental through aptly. However, I was not able to do it because the only Packages file available in main/binary-< arch > directory is Packages.xz.
Example: ftp.debian.org(..)experimental/main/binary-i386/.
This extension (.xz) is not catched in download.go. So I reused your github.com/smira/go-xz to do it.
Have a good day,

@rmedaer thanks, change looks good to me! Do you mind fixing unit-tests which got broken? (Go 1.5, https://travis-ci.org/smira/aptly/jobs/126150884)

Ohhh ! Sorry for the tests ... I'll change that know and add it to the PR ;-)
Thks for your quick feedback !

I added some tests both for the compression testing and the download test suite. However I still have to modify/adapt some tests to ensure everything is good ! I'll do it ASAP.

There still seems to be two failures for system tests:
============================================================
t04_mirror:UpdateMirror4Test Testing mirror management: update mirrors: wrong checksum in release file, but ignore
Traceback (most recent call last):
  File "system/run.py", line 73, in run
  File "/home/travis/gopath/src/github.com/smira/aptly/system/lib.py", line 94, in test
    self.check()
  File "/home/travis/gopath/src/github.com/smira/aptly/system/lib.py", line 202, in check_output
    self.verify_match(self.get_gold(), self.output, match_prepare=self.outputMatchPrepare)
  File "/home/travis/gopath/src/github.com/smira/aptly/system/lib.py", line 295, in verify_match
    raise Exception("content doesn't match:\n" + diff + "\n")
Exception: content doesn't match:
---
+++
@@ -2,6 +2,7 @@
 Downloading & parsing package files...
 Downloading http://<IP_ADDRESS>:52952/dists/hardy/main/binary-amd64/Packages.bz2...
 Downloading http://<IP_ADDRESS>:52952/dists/hardy/main/binary-amd64/Packages.gz...
+Downloading http://<IP_ADDRESS>:52952/dists/hardy/main/binary-amd64/Packages.xz...
 Downloading http://<IP_ADDRESS>:52952/dists/hardy/main/binary-amd64/Packages...
 WARNING: http://<IP_ADDRESS>:52952/dists/hardy/main/binary-amd64/Packages: sha256 hash mismatch "494414ded24da13c451b13b424928821351c78fce49f93d9e1b55f102790c206" != "8a21688ae769f2b4ffcaa366409f679d"
 ERROR: unable to update: malformed stanza syntax

============================================================
t04_mirror:UpdateMirror6Test Testing mirror management: update mirrors: wrong checksum in package, but ignore
Traceback (most recent call last):
  File "system/run.py", line 73, in run
  File "/home/travis/gopath/src/github.com/smira/aptly/system/lib.py", line 94, in test
    self.check()
  File "/home/travis/gopath/src/github.com/smira/aptly/system/lib.py", line 202, in check_output
    self.verify_match(self.get_gold(), self.output, match_prepare=self.outputMatchPrepare)
  File "/home/travis/gopath/src/github.com/smira/aptly/system/lib.py", line 295, in verify_match
    raise Exception("content doesn't match:\n" + diff + "\n")
Exception: content doesn't match:
---
+++
@@ -2,6 +2,7 @@
 Downloading & parsing package files...
 Downloading http://<IP_ADDRESS>:58492/dists/hardy/main/binary-amd64/Packages.bz2...
 Downloading http://<IP_ADDRESS>:58492/dists/hardy/main/binary-amd64/Packages.gz...
+Downloading http://<IP_ADDRESS>:58492/dists/hardy/main/binary-amd64/Packages.xz...
 Downloading http://<IP_ADDRESS>:58492/dists/hardy/main/binary-amd64/Packages...
 Building download queue...
 Download queue: 1 items (30 B)

============================================================

They should be easy to fix up. I can fix it myself as well if you don't have time.

Moved to #505, closing this one


How to focus on group at index in formArray

Question
Hi! thanks for the lib, very nice one. I'm extending custom form field array (aka repeat) with the functionality of add a new group at index next of group selected and all works fine with the add(index) method. But when I want provide functionality of focus on the new item at index added, the focus going to the last element and not in the previous group created.
Since is a question I don't provide stackblitz example, but if you need it I can share my repo https://github.com/begandroide/ngformly-uitesis/tree/feature/section
Regards.

I found a workaround working with formState and expressionProperties:
formState = { selectedIndex: 0};

fields = [
{
      key: 'questions',
      type: 'repeat',
      fieldArray: {
        wrappers: ['container'],
        fieldGroupClassName: 'row',
        templateOptions: {
          label: 'Title',
          btnText: 'Add another investment',
        },
        focus: true,
        fieldGroup: [
          {
            defaultValue: '',
            className: 'col-sm-4',
            type: 'input',
            key: 'name',
            expressionProperties: {
              focus: 'field.parent.key == formState.focusedIndex',
              blur: 'field.parent.key != formState.focusedIndex'
            },
            templateOptions: {
              label: 'Test:',
            },
          },
        ],
      },
    },
]

@begandroide I'm going to close this as it looks like you've found a solution for the problem that you were having.  If there are any more things feel free to open a new issue!


Add 'Hunters Paradise' ruleset.

Resolves https://github.com/orendain/DemeoMods/issues/178.
One way to describe this ruleset:
**Rules**
- Starting cards: 1x HuntersMark (usable every round), 1x CallCompanion, 1x SummonElemental
- New cards always come from the pool: Charm, Bone, Strength Potion, Speed Potion and all pet summoning cards
- Level 1 POIs: 6 healing, 6 chests
- Level 2 POIs: 6 healing, 12 chests
- Level 3 POIs: 6 healing, 12 chests

Approving so you can merge when you like. Up to you if you want to change Guardian's armor.

Fixed, thanks 👍🏾.  Merging.


Ported detrend normalize from MyDetrend.m

Line 29 in test_detrend:
signal.clean(signals=test_data['TC'], t_r=2)
should be:
signal.clean(signals=test_data['TC'], t_r=2, high_pass = 0.008)
To approximately match the matlab command.

Reviewed 3 of 4 files at r1, 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Read Response with timestamped SenML

Implementation of read response with timestamped SenML
This aims to implement : https://github.com/eclipse-leshan/leshan/issues/1553

I rebase the code on master and push some fixes.
You could review it if wanted. (I advice to review commit by commit)
We still need to decide :

Do you want to handle timestamped value for ObserveResponse too ?
(I mean for observe response, not notification (notification as already timestamp value)

I didn't have time to test about reducing timeout value but I pretty sure that 3 seconds is not needed

I reviewed the code and everything looks fine. I also pushed commit with timestamped for Observe Response.

I reviewed the code and everything looks fine. I also pushed commit with timestamped for Observe Response.

:pray:
I started to work on it again. I should be able to push some code tomorrow.

@mgdlkundera, I added some more fix/features.
You could review/test it if wanted. (I advice to review commit by commit)
@mjviljan is possible to test it ? (let me know if this is a problem for you to test a PR)

@mjviljan is possible to test it ? (let me know if this is a problem for you to test a PR)

Yes, I can test this also. Unfortunately only some time next week, so if you want to merge this before that, don't wait for me.
But otherwise I'll come back to this once I've tried it out.

@mjviljan thx for letting me know that. We can wait for next week.

Hi, sorry for the delay. I tested this now for my case. Unfortunately this doesn't fix that — but I guess that's just (more) proof that the data I receive from the device may not be really LwM2M standard-compliant.
Anyway, thanks for suggesting me to test this! At least I now have some new knowledge to continue with.

I tested it and I also look through the code and it is ok.

I tested it and I also look through the code and it is ok.

Thx :pray:
I rebased it on master, then squashed all commit in one (dceda02a1fd6f99c93b6742c49fc98e5cdb89ddc), then integrated in master.
It will be available in 2.0.0-M15.


Allow Combos to be given a title

Would be nice to have combo names, especially for the ones that have given names in the community like Dramatic Scepter

Closing as not planned. Will see if in the future will become a requested feature.


Add Microsoft Edge

Microsoft Edge is a new browser, forked from IE but that is not IE itself. Windows 10 will include both Edge & IE11.
caniuse-db now separates those 2 browsers, see https://github.com/Fyrd/caniuse/commit/0f31db424f056ec1b934c9788100cae69cab60f2.
The browser should be added to the list.

Sure. I will start it after PostCSS 5.2 release.


feat: Added == and != operator overrides to Vec.cs classes

Thanks for the PR! Sign-off is good and changes are done right :) /master is intended to be a snapshot of the latest stable version, so this should be a PR against the /develop branch.
However! Equality operators for vectors are a bit of a footgun, so I've intentionally avoided them for SK's Vecs. Floating point comparison in particular is the root issue, two floats are "never the same" unless they're straight up copies of eachother! I see you're doing a Distance, which is generally the right approach, but it should be a "less than small value/epsilon" rather than "equal to 0". Distance is also one of the pricier methods as it involves a Sqrt, so a small improvement would be to use DistanceSq and a squared epsilon.
But because there's a lot of these small details here, it's kinda dangerous to be doing this in an operator overload, as it's obscuring complex behavior and performance implications. I've seen math libraries that do this before, and they have always had performance problems hiding in sneaky places!
If you want to do this as a static and/or instance method instead, I'd accept that as a PR :)

Alright! Thanks Nick!
I'll reword that PR and post it again later