Omit all but instruction_addr field when symbol address not found

Goal
Stack trace entries whose symbol address cannot be determined will now only have an instruction_addr field.
Previously, it would send 0 values for missing fields, which would confuse the backend.
        "threads": [
            {
                "backtrace": {
                    "contents": [
                        {
                            "instruction_addr":<PHONE_NUMBER>
                            "object_addr":<PHONE_NUMBER>,
                            "symbol_addr": 0,
                        },
...

Testing
Manually tested using:
#if TARGET_CPU_X86 || TARGET_CPU_X86_64
    asm("jmp 0x70000000");
#elif TARGET_CPU_ARM || TARGET_CPU_ARM64
	asm volatile("bx %0" : : "r" (0x70000000));
#else
    #error(Unhandled CPU architecture)
#endif

Manually checked that the correct report is generated:
        "threads": [
            {
                "backtrace": {
                    "contents": [
                        {
                            "instruction_addr":<PHONE_NUMBER>
                        },
                        {
                            "instruction_addr":<PHONE_NUMBER>,
                            "object_addr":<PHONE_NUMBER>,
                            "object_name": "UIKit",
                            "symbol_addr":<PHONE_NUMBER>,
                            "symbol_name": "-[UIApplication sendAction:to:from:forEvent:]"
                        },
...

Manually checked backend display.

Could the manual test code be used to create an E2E scenario?

Unfortunately no. I tried creating e2e tests but they wouldn't give consistent results.

LGTM, just holding off until after today's release...


Fix v0.4.0 rule of hooks

Somehow eslint didn't got the react-hooks config, thus there where no rule-of-hooks checks at linting.

fixed already in the new separate repos of material-code and material-color
fixes for rest of ui-schema (incl. core) are in progress

Release with:

@ui-schema/ui-schema 0.4.2
@ui-schema/ds-material 0.4.0-alpha.6
@ui-schema/material-dnd 0.0.15
@ui-schema/material-slate 0.0.11


make key-generator and encryptor strategy as a spring bean on spring namespace

[ ] key-generator
[ ] encryptor strategy

I will try to solve it.

Thank you, assigned

When I looked at some code of sharding - core and sharding - spring, I found that ShardingDataSourceBeanDefinitionParser will register a SpringShardingDataSource bean to spring.Then according to the configuration in ShardingRuleConfiguration - > TableRuleConfiguration - > KeyGeneratorConfiguration, the constructor of SpringShardingDataSource creates the corresponding ShardingKeyGenerator instance by way of SPI.
When SPI is used to create an instance of a ShardingKeyGenerator, it will make all lot of peices instances of ShardingKeyGenerator of all types, and we can iterate to find type that we want. Now We can optimize this issue if we make the ShardingKeyGenerator as a spring bean, and at the time we need to considerate the decoupling of sharding-core and sharding-spring.
This is the context of this issue as I understand, am I right?

@TaoZhiMLND Sure, you are totally correct. We can discuss how to decouple it if you want.

it seems that we should instance KeyGenerator when constructing KeyGeneratorConfiguration instead of inside TableRule. for spring-namespace we can leverage FactoryBean to create KeyGenerator through our NewInstanceServiceLoader.

After the discussion with @cherrylzhao, we decide make key-generator as a spring bean on spring namespace in the following steps:

[✓] change shardingNamespace.xml configuration.

[✓] create class KeyGeneratorFactoryBean extends TypeBasedSPIConfiguration  implements FactoryBean<ShardingKeyGenerator>,leverage FactoryBean to create KeyGenerator.

[✓] modify class KeyGeneratorConfigurationextends TypeBasedSPIConfiguration，add member variables ShardingKeyGenerator keyGenerator.

[✓] modify class KeyGeneratorBeanDefinitionParser addConstructorArgReference with KeyGenerator by using TAG generator-ref.

[✓] modify class TableRule, shardingKeyGenerator=tableRuleConfig.getKeyGeneratorConfig().getKeyGenerator().

[✓] modify class ShardingRule, use keyGeneratorConfiguration.getKeyGenerator() instead of create by ShardingKeyGeneratorServiceLoader.

[✓] modify correlative tests, add test on ShardingKeyGenerator Bean.

[  ] modify docs and examples.

I'll commit a pr.


Ui, can you be more refined, now the ui font is a bit fuzzy

Ui, can you be more refined, now the ui font is a bit fuzzy

Please provide screenshot. Please provide configuration details that you deleted from the issue template.

Are you talking about the OS X feature? Use defaults write -g CGFontRenderingFontSmoothingDisabled -bool NO
To disable

Closing due to lack of information.


LLVM 3.0 macros and cl_khr_3d_image_writes

LLVM's inclusion of OpenCL C 3.0 feature macros (see d88c55) broke parsing of the builtin headers when using OpenCL C 3.0 as the language standard.
Example errors:
In file included from <built-in>:375:
include/opencl-c.h:10571:37: error: use of type '__write_only image3d_t' requires cl_khr_3d_image_writes extension to be enabled
void __ovld write_imagef(write_only image3d_t image, int4 coord, float4 color);
                                    ^

Current workaround is to undef cl_khr_3d_image_writes. Longer term, figure what should actually be done about this.
Additionally, determine which feature macros should be defined for clspv.


[BUG] linear_regression.ipynb and ridge.ipynb future warning

Describe the bug
When running cuml/linear_regression.ipynb or cuml/ridge.ipynb notebooks, the following warning is displayed in cell #9.

/conda/envs/rapids/lib/python3.6/site-packages/ipykernel_launcher.py:2: FutureWarning: Method .as_matrix will be removed in a future version. Use .values instead.

Steps/Code to reproduce bug
ocker pull nvcr.io/nvidia/rapidsai/rapidsai:cuda10.0-runtime-ubuntu18.04
docker run --runtime=nvidia --rm -it -p 8888:8888 -p 8787:8787 -p 8786:8786 nvcr.io/nvidia/rapidsai/rapidsai:cuda10.0-runtime-ubuntu18.04
bash utils/start-jupyter.sh
localhost:8888
run linear_regression_demo.ipynb or ridge.ipynb
Expected behavior
No warning displayed.
Environment details (please complete the following information):
DGX-1
Additional context
Add any other context about the problem here.

resolving in https://github.com/rapidsai/notebooks/pull/67

Integreated.   Thanks @miguelangel !


Add a decorator for plotting a subclassed model (based on plot_model)

System information.
TensorFlow version (you are using): 2.6
Are you willing to contribute it (Yes/No) : Yes
Describe the feature and the current behavior/state.
I tested tf.keras.utils.plot_model in tf 2.6 and 2.8, to find plot_model  cannot directly plot a subclassed model. I've tried several ideas, and this one requires minimal code to change.
Will this change the current api? How?
It will add a decorator function to keras/utils/vis_utils.py, which is like a wrapper and will call plot_model to do the work.
Who will benefit from this feature?
plot_model support functional model so usually we can do below to plot a subclassed model.
Here is a method on StackOverflow, which I used to use:
class Model(tf.keras.Model):
    def plot(self,input_shape):
        x=layers.Input(input_shape)
        y=self.call(x)
        model=tf.keras.Model(inputs=x,outputs=y)
        return tf.keras.utils.plot_model(model,show_shapes=True)

But everytime I want to plot a new model, I need to copy this plot function and manually change the input_shape. I considered to derive a plotable class who completes the plot function, but it still can't avoid manually changing the input_shape. (Oh yes, I am lazy.)
So I want to find a Don't-Repeat-Youtself way to do so.
My idea is to add a decorator on build function (I'll show my code below). It just catched the input_shape of build, and all the work is still done by Model.build and plot_model.
@plotable()
    def build(self,input_shape):
        super().build(input_shape)

Contributing

Do you want to contribute a PR? (yes/no): yes
If yes, please read this page for instructions
Briefly describe your candidate solution(if contributing):

Here is my sample.
Below are the codes.
def plot_subclass_model(silent=False,show_shapes=True,**kwargs):
    '''
    This is a supplementary decorator to plot a subclassed model.
    Decorate it on Model.build to call `tf.keras.utils.plot_model`.
    Example:

    ```python
    class MyModel(tf.keras.Model):

      def __init__(self):
        super().__init__()
        self.dense = tf.keras.layers.Dense(4, activation=tf.nn.relu)

      def call(self, inputs):
        return self.dense(inputs)

      @plot_subclass_model()
      def build(self,input_shape):
        super().build(input_shape)

    model=MyModel()
    model.build((None,32,32,3))
    ```

    Args:
      silent: Whether to plot the model. If set to True, it will do nothing other than the `build` function. Default is False.
      kwargs: Args passed to `plot_model`. Arg `show_shapes` will pass True by default. See more in `tf.keras.utils.plot_model`.
    '''
    from functools import wraps

    kwargs['show_shapes']='show_shapes'

    def decorate(func):
        @wraps(func)
        def wrapper(self,input_shape):
            result=func(self,input_shape)

            if not silent:
                from tensorflow.keras import layers
                if isinstance(input_shape,(tuple,tf.TensorShape)):
                    inputs=layers.Input(input_shape[1:])
                elif isinstance(input_shape,list):
                    inputs=[layers.Input(s[1:]) for s in input_shape]
                else:
                    raise ValueError(f'Expect input_shape to be a shape or a list of shape, but received {type(input_shape)}')

                outputs=self.call(inputs)
                model=tf.keras.Model(inputs=inputs,outputs=outputs)
                plot_result=tf.keras.utils.plot_model(model,**kwargs)
                try:
                    from IPython.display import display
                    display(plot_result)
                except ImportError:
                    pass
            return result
        return wrapper
    return decorate

unsubscribe
------------------ 原始邮件 ------------------
发件人:                                                                                "keras-team/keras"                                                                                @.>;
发送时间: 2022年3月1日(星期二) 下午3:31
@.>;
@.***>;
主题: [keras-team/keras] A decorator for plotting a subclassed model (based on plot_model) (Issue #16142)
System information.
TensorFlow version (you are using): 2.6
Are you willing to contribute it (Yes/No) : Yes
Describe the feature and the current behavior/state.
I tested tf.keras.utils.plot_model in tf 2.6 and 2.8, to find plot_model  cannot directly plot a subclassed model. I've tried several ideas, and this one requires minimal code to change.
Will this change the current api? How?
It will add a decorator function to keras/utils/vis_utils.py, which is like a wrapper and will call plot_model to do the work.
Who will benefit from this feature?
plot_model support functional model so usually we can do below to plot a subclassed model
class Model(tf.keras.Model):     def plot(self,input_shape):         x=layers.Input(input_shape)         y=self.call(x)         model=tf.keras.Model(inputs=x,outputs=y)         return tf.keras.utils.plot_model(model,show_shapes=True)
But everytime I want to plot a new model, I need to copy this plot function and mannually change the input_shape. I considered to derive a plotable class who completes the plot function, but it still can't avoid mannually changing the input_shape. (Oh yes, I am lazy.)
So I want to find a Don't-Repeat-Youtself way to do so.
My idea is to add a decorator on build function (I'll show my code below). It just catched the input_shape of build, and all the work is still done by Model.build and plot_model.
@plotable()     def build(self,input_shape):         super().build(input_shape)
Contributing
Do you want to contribute a PR? (yes/no): yes
If yes, please read this page for instructions
Briefly describe your candidate solution(if contributing):
Here is my sample.
Below are all the code, and I'll update it soon following the keras_api_design_guidelines .
def plotable(silent=False):     '''     Used on model.build to call tf.keras.utils.plot_model     '''     from functools import wraps     def decorate(func):         @wraps(func)         def wrapper(self,input_shape):             result=func(self,input_shape)             if not silent:                 from tensorflow.keras import layers                 from IPython.display import display                 if isinstance(input_shape,(tuple,tf.TensorShape)):                     inputs=layers.Input(input_shape[1:])                 elif isinstance(input_shape,list):                     inputs=[layers.Input(s[1:]) for s in input_shape]                 else:                     raise AssertionError                 outputs=self.call(inputs)                 model=tf.keras.Model(inputs=inputs,outputs=outputs)                 display(tf.keras.utils.plot_model(model,show_shapes=True))             return result         return wrapper     return decorate
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Hi Meidozuki, thanks for reporting the issue and being willing to contribute!
We do not have general approach for plotting the subclassed model by intention, because subclassed model might contain customized layer and special structure that is hard to plot. The approach you are proposing is aligned with Functional model, which is a chain of Keras layers.

OK, I see. I did meet  a case that a subclass has its own variables and cannot be plotted.
Thanks for reviewing.


exploitdb: 2022-06-28 -> 2022-07-02

Description of changes
Update to latest upstream release 2022-07-02

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

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

1 package built:

exploitdb


Release 3.9

Updating release-3.9 branch with recommended workaround for container images not being found on docker.io

@vincepower looks like you tried merging into master versus the branch.

Closing PR since this is against wrong branch.


Provide advice for tibble::tribble() calls

Hello, thank you for this tool! After using Ruff and rustfmt, I'm very happy to see fast formatting coming to R.
I was testing air on some packages and noticed that tibble::tribble() are reformatted with one element per line, which makes it harder to read:
Before:
tribble(
  ~colA, ~colB,
  "a",   1,
  "b",   2,
  "c",   3
)

After:
tribble(
  ~colA,
  ~colB,
  "a",
  1,
  "b",
  2,
  "c",
  3
)

I saw this was mentioned in #26 but the issue is now closed and I didn't find any mention of this in docs/formatter.md, so I'm just opening this so that it is tracked somewhere. I hope it's not a duplicate.
Thanks again

Tested on main with 093bb96

Yes, right now there is no way to avoid the mess that you see here with tribble() calls.
We really want to avoid hardcoding any specific function names into the formatter. One idea is to turn off formatting altogether. You can't do this yet but it would look something like
# air format: ignore
tribble(
  ~x,   ~y
  1,    2
)

We are also considering more powerful comments that would actually help you enforce a particular format, like
# air format: layout-tabular
tribble(
  ~x,   ~y
  1,    2
)

This would snap you into a rectangular layout as you type to help you out (probably falling back to a "verbatim" what-you-see-is-what-you-get layout if the number of arguments doesn't divide evenly into a rectangle)

Closing in favour of: https://github.com/posit-dev/air/issues/113


CKAN descriptions are Markdown, not text.

When pulling from CKAN instances, the description is always treated as plain text, when it is actually markdown - https://github.com/theodi/data_kitten/blob/master/lib/data_kitten/publishing_formats/ckan.rb#L53
This shows up as an issue when generating certificates using this - it may be that the right thing to do is to fix there, rather than here.

Regarding implementation, CKAN API version 2 (but not 3) seems to provide a notes_rendered with rendered HTML which can be used temporarily (since data_kitten only parses packages with v2 right now).


The program does not work with one specific boiler

I tried the program with different boilers and it was a success! There was not a single problem...
But the program does not work with one boiler. More precisely, the program works, but the boiler does not start...
We are talking about the Baxi Luna3 Comfort 1.240 fi boiler
I did the following:

Made sure that there is no jumper.
Parameter F10 = 0
Parameter AMBON = 0
Parameter MODUL = 0
With such parameters, the boiler does not start and does not respond via the Opentherm bus.
I tried the program on several Baxi Luna3 Comfort 1.240 fi boilers and not a single boiler worked.
The Baxi Luna3 Comfort 1.310 fi and BAXI ECO-4s boilers work perfectly!
Maybe someone has a positive experience with the Baxi Luna3 Comfort 1.240 fi boiler?
Please help me with this enchanted boiler...

Could this be an answer?
#19

Could this be an answer?
#19

It could very well be... And it needs to be checked.
I'm ready to check, but I don't understand where in the code I can change the master ID?


Crash on single player launch

Microsoft.Xna.Framework.Audio.NoAudioHardwareException was unhandled
Message: An unhandled exception of type 'Microsoft.Xna.Framework.Audio.NoAudioHardwareException' occurred in MonoGame.Framework.dll
Additional information: OpenAL drivers could not be found.

Obtain OpenAL.


Get Key from %APPDATA% instead

Feature Request
Is there a way for us to get the ElevenLabs key from %APPDATA% .json file instead?
Is your feature request related to a problem? Please describe.

I would want to be able to ship my Unity Game to other users so I would like to at least stop unwanted users from abusing my key.
Describe the solution you'd like

Creating a way (Or teaching me how to if it's available right now) to retrieve keys from a .json file

You actually can setup a json file named .elevenlabs and it will read your api key that way.
https://github.com/RageAgainstThePixel/com.rest.elevenlabs/blob/48e1fcf42d5586dc556cbcee883fd5c2670d7822/ElevenLabs/Packages/com.rest.elevenlabs/Runtime/ElevenLabsAuthentication.cs#L99

You'd use it like this:
var auth = ElevenLabsAuthentication.LoadFromDirectory("path/to/your/directory", "yourFileName");
var client = new ElevenLabsClient(auth);

But your problem will still be there as anyone can sniff your api calls with wire shark, as the auth is passed in the headers.
Again your best bet is to setup an API proxy.

Honestly tho, this api is pretty expensive thb, your best bet to generate everything you need in editor ahead of time for now. I'll work on the proxy API support when I get a chance.


Make JSON sort keys for replicability

Using json.dumps() doesn't retain the same order, making it difficult to ensure any sort of order. Lets use the native sorting method to ensure at least some kind of order.

At this rate, I'd also suggest using ujson


Boilerplate in get_results() with many derived buffers

Right now, a real-world get_results() function can look like this:
...
return {
    'num_frames': self.results['num_frames'],
    'varsum': self.results['varsum'],
    'sum': self.results['sum'],
    'var': self.result(name='var', data=var),
    'std': self.result(name='std', data=np.sqrt(var)),
    'mean': self.result(name='mean', data=self.results['sum'].raw_data / num_frames),
}

or like this:
def get_results(self):
    results = get_results(self.results)
    return {
        'fourier': self.results['fourier'],
        'complex': self.result('complex', results),
        'amplitude': self.result('amplitude', np.abs(results)),
        'phase': self.result('phase', np.angle(results)),
    }

The buffer name has to be repeated for each buffer. This could be improved by returning a dict of buffername --> ndarray and doing the conversion to a BufferWrapper automatically. That has two advantages: First, it saves some tedious typing, including the call to self.result(). Second, it enforces that results of an UDF are always BufferWrapper.
One could even just pass through buffers that are not in the result dict, i.e. num_frames, varsum and sum in the stddev example, so that one just has to worry about buffers that are to be modified or created.

The buffer name has to be repeated for each buffer. This could be improved by returning a dict of buffername --> ndarray and doing the conversion to a BufferWrapper automatically. That has two advantages: First, it saves some tedious typing, including the call to self.result(). Second, it enforces that results of an UDF are always BufferWrapper.

Thanks, this kind of feedback is really useful. I agree that the repeated names may not be DRY enough. From the perspective of writing a get_results function, returning a numpy array directly would indeed be less typing and a bit more DRY.
A (maybe minor?) point is that, right now, the conversion from array to BufferWrapper at that point is explicit, and what you return from get_results is also what you get back from run_udf. But this may be acceptable, in combination with the easier-to-implement get_results? Let me know what you think.

One could even just pass through buffers that are not in the result dict, i.e. num_frames, varsum and sum in the stddev example, so that one just has to worry about buffers that are to be modified or created.

Possibly, that also means it's no longer possible to remove buffers from the final result of the UDF. Removing results from the final result may be important to hide implementation details that could change in future versions of a UDF. Maybe this should be already declared in get_result_buffers? self.buffer(..., private=True)? What do you think? Important? Feature creep?

One could even just pass through buffers that are not in the result dict, i.e. num_frames, varsum and sum in the stddev example, so that one just has to worry about buffers that are to be modified or created.

Possibly, that also means it's no longer possible to remove buffers from the final result of the UDF. Removing results from the final result may be important to hide implementation details that could change in future versions of a UDF. Maybe this should be already declared in get_result_buffers? self.buffer(..., private=True)? What do you think? Important? Feature creep?

Since get_results() is already an advanced feature that not all UDFs need and that is more of , probably it would make sense to require passing through all buffers, and dropping buffers from a result could be done here? Or do we need some sort of guarantee that whatever is in get_resul_buffers() also is present in get_results()?

Since get_results() is already an advanced feature that not all UDFs need and that is more of , probably it would make sense to require passing through all buffers, and dropping buffers from a result could be done here?

Or do we need some sort of guarantee that whatever is in get_resul_buffers() also is present in get_results()?

I think it would be good to at least know beforehand which "result channels" will exist - the idea of private=True was to declare that this buffer is only used as an intermediate buffer, basically the symmetric case to allocate=False.
To summarize my proposal:

All buffers are declared in get_result_buffers
If a buffer is only computed in get_results, it should be marked via allocate=False so it isn't allocated on workers
If a buffer is only used as intermediary result, it should be marked via private=True
Not including a buffer in get_results means it will either be passed on unchanged, or dropped if private=True
It's an error to omit an allocate=False buffer in get_results
It's an error to include a private=True buffer in get_results

So, your example from above would become:
return {
    'var': var,
    'std': np.sqrt(var),
    'mean': self.results['sum'].raw_data / num_frames,
}

num_frames, varsum, sum aren't declared private in this case and returned unmodified. As an example, if one would like to not include varsum in the final result, one could declare it private in get_result_buffers, then it would be dropped and visualization would skip it.
With the proposed change from #1000, it would become:
return {
    'var': var,
    'std': np.sqrt(var),

    # explicitly get result buffer, to mitigate confusion of __getitem__ vs __getattr__
    'mean': self.results.get_buffer('sum').raw_data / num_frames,
}

Thoughts? Maybe private and allocate can be combined into a single parameter, then we also automatically exclude the invalid case private == True and allocate == False. Maybe use='private' and use='result_only' (to be bike-shedded, please!)


Better newlines and long texts

The current method of converting to strings does not take newlines into consideration, which are probably not that uncommon. The current state of "newlines in markdown tables" seems pretty bad from a quick search on the internet.
Current:
echo '{"message":"Wr\nong"}' | tablemark
| Message |
| ------- |
| Wr
ong  |

Some options:
Replacing \n with
:
| Message     | Code  |
| ----------- | ----- |
| Wr
ong | 2     |

Message
Code

Wrong
2

Extending the table downwards
Keeping gutters/separators
| Message | Code  |
| ------- | ----- |
| Wr      | 2     |
| ong     |       |
| Good    | 0     |

Message
Code

Wr
2

ong

Good
0

Without gutters/separators
| Message | Code  |
| ------- | ----- |
| Wr      | 2     |
  ong
| Good    | 0     |

Message
Code

Wr
2

ong

Good
0

Since only the first produces valid markdown (GitHub-usable), I think that should definitely be an option. Apart from that consideration, I'm mostly into the last one (wrapping and aligning but without gutters/separators).
My rough proposal is:
Wrap at newlines and after any 80 characters run. Do not add gutters and separators, but align content. Provide an option to tweak characters per line or disable automatic line wrapping. Provide an option to replace newlines with
 and disable any wrapping.

Yo @tjconcept - thanks for the writeup. I'm back from vacation ( a terrible non-vacation really since it resulted in a totaled car and leaving early due to hurricane ) and will review this and your other PRs when I get a chance.
Right now I think your proposal is the way to go, although defaulting to
 might be a choice.

I'm sorry to hear about your vacation, but welcome back.
I went ahead and implemented this for the challenge, first as a prototype, then I couldn't stop. It was.. much more complex than I had imagined. Things as word wrapping (visual length ≠ .length due to unicode), and generating and rendering a matrix for each row.
Anyway, I am satisfied with the result. I'm gonna put up a PR for you to peak at the code soon.

though maybe make
 the default.

I was inclined to newlines because my primary use case is "pretty printing" in the terminal. But I guess this package (reasonably) has "valid markdown" as the primary use case.

@tjconcept Either way shouldn't be too bad as a default. A shorthand flag on the CLI would make either one easy enough to trigger, like -n for using newlines or -b for breaks or something.
Unicode lengths may need a little extra love. Might be worth adding in string-length for that but there could also be alternatives.
I've made you a collaborator for both the module & the CLI so you can make pushes. I'll just jump in with reviews as we've been doing, but you've been putting in a load of time for these so it'll be easier for you to work on things. Thanks for your efforts 👍

Unicode lengths may need a little extra love. Might be worth adding in string-length for that but there could also be alternatives.

I ended up writing a new package for this as we need the wrapped result back as an array, not a string, we need the string length counter to be tied to the wrapper (returning the same result), and we need hard wrapping (some packages had hard wrapping, but would still overflow on words longer than the column).
It's not that bad to count too much, as long as you wrap by the same the same metric, and only wraps on explicit white space. It will only result in some excess whitespace.

I've made you a collaborator for both the module & the CLI so you can make pushes. I'll just jump in with reviews as we've been doing, but you've been putting in a load of time for these so it'll be easier for you to work on things.

Awesome! Give PRs an approval when you think they are ready then.

Thanks for your efforts 👍

No problem. I'm scratching my own itches here (this being the last one I think, for now).

Closed by #6


.elems is not a multi method

If I try to override .elems, I get:
Package 'Table::Driver::Postgres' already has a method 'elems' (did you mean to declare a multi method?)

...and if I try to make .elems a multi, I get
Cannot have a multi candidate for 'elems' when an only method is also in the package 'Table::Driver::Postgres'

This might be a bug inside my head, but it might be something Hash::Agnostic can fix.
Thanks!

It may be a bug in your head, or a WAT in the language.  But there is a way around it.  The problem golfed is this:
$ raku -e 'role A { method a { } }; class B does A { multi method a { } }'
===SORRY!=== Error while compiling -e
Cannot have a multi candidate for 'a' when an only method is also in the package 'B'

Note that if it wouldn't be a multi, there'd be no problem:
$ raku -e 'role A { method a { } }; class B does A { method a { } }'

So an only method does not pose an issue.  Yet you want a multi, presumably because you want more than one.
You can get around this by declaring a proto in the class.  A proto is internally also an only method.  So:
$ raku -e 'role A { method a { } }; class B does A { proto method a(|) {*}; multi method a { } }'

works like you probably want.
If you want to access the original method in your new multi, there's a way to do that:
$ raku -e 'role A { method a { "A" } }; class B does A { proto method a(|) {*}; multi method a { self.A::a } }; dd B.a'
"A"

Note that you can call a method from a role like that directly by making the method name fully qualified.
Hope this helped.  Going to close this issue now.  Please re-open if you disagree  :-)


Hide cancelled meetings

When meetings get cancelled, but not deleted, they still show up in MultiMeet. These should be (optionally) hidden.

Fixed by edd711649564246b0f1b4eb233ff2b7a10b2856b


Insufficient 'Getting Started' instructions

I'm currently trying to experiment with Tone.js in a simple react app. I started with the create-react-app boilerplate and webpack config, and then followed the basic instructions of npm install tone.
However, something is broken about the way the tone is bundled and delivered. When I just wanted to test out the simple Hello Tone example in my app, I tried importing Tone. Since the actual import statement wasn't listed in the examples I guessed at import Tone from 'tone', but that resulted in the following error:
Attempted import error: 'tone' does not contain a default export (imported as 'Tone')
I then tried a named import and got a similar result. Eventually, while viewing source on your example page I discovered the cdnjs version I was finally able to load on the page. That's not ideal, but sufficient for my needs.
I'd really hate if others had so much trouble just getting a simple example running on a page. You should at least mention the cdnjs option in the getting started instructions, but ideally it would be great if it could actually be able to be imported into a standard webpack bundle.

Thanks for the issue. I think there used to be a quick guide on how to import Tone.js into your project at the top of the README, but i guess that got lost in the recent transition to typescript. I just added it back at the top of the README.
This should work in create-react-app:
import * as Tone from 'tone';

Awesome, thanks @tambien!


Replace firingUnixTimestampSeconds with duration field

more accurate to use relative time
easier for unit test
cleaner in the history

https://github.com/indeedeng/iwf/pull/388


Move util/common packages from libraries to GameUtils

Only those packages that are used in 2+ libraries / modules should be moved to avoid unnecessary clutter and to reduce size

Commonly used classes have been moved to GameUtils


codegen: fill does not capture full text string

Specific to bing.com
Repro steps:

go to bing.com
click on the search input field
type "git" quickly
type "hub" after a short pause
the codegen shows only git (await page.fill('input[aria-label="Enter your search term"]', 'git');

it seems bing.com tries to autocomplete the field, which does not play well with codegen

I am also facing some issues with entering text in search field. I tried on this https://www.catch.com.au/

I am also facing some issues with entering text in search field. I tried on this https://www.catch.com.au/

The issue that catch.com.au is not working was tricky. They basically overwrite String.replaceAll in their code base which we use internally in a way that its broken and raises a error for our use case:
    String.prototype.replaceAll = function(e, t, n) {
        return this.replace(new RegExp(e.replace(/([\/\,\!\\\^\$\{\}\[\]\(\)\.\*\+\?\|\<\>\-\&])/g, "\\$&"),n ? "gi" : "g"), "string" == typeof t ? t.replace(/\$/g, "$$$$") : t)
    }

(from their frontend bundle) So we can't do much in that case if they overwrite default methods.

I tried to reproduce it a few weeks ago and wasn't. Let's reopen if its still reproducible.


Expose both root/rootless socket

As of now we are only exposing the /run/podman/podman.sock which consumed by docker client or API side, we need to expose both the socket if someone wants to consume API in rootless mode.

For future reference, this is a follow-up of these commits which expose the podman socket as unix socket on the host, instead of having to access it through a ssh tunnel.
https://github.com/code-ready/crc/commit/d983719d34e7ca00adb1dc20290cd8453f7d2909
https://github.com/code-ready/crc/commit/790195bb3baafbb222c4f496575123fa367cdae8


UVI Request

--- UVI JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from  to before v4.19.195",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=ec72482564ff99c6832d33610d9f8ab7ecc81b6d"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "ec72482564ff99c6832d33610d9f8ab7ecc81b6d",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "net/nfc/rawsock.c: fix a permission check bug\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v4.19.195 by commit ec72482564ff99c6832d33610d9f8ab7ecc81b6d. For more details please see the references link."
}
--- UVI JSON ---

This issue has been assigned UVI-2021-1000867


System Requirements

Wow this looks like just what I need! But it doesn't say what platform it is targeted at - OSX or Linux? Any specific distros? Perhaps you can add this to the README...
Thanks for sharing!

I run it on Linux (Ubuntu 16.04) but it should work on OS X as well. It might even work on Windows, but I've never tried ;-)

Is this up for grab? I have windows laptop and I can try to work on it.  I think I can help here, and also it will be a great learning experience to go through all the code and try to understand it.
Please let me know :)

Closing this issue because I documented it


WinRT originate error ?

I'm trying to create a FrameReader object reading from a webcam.

But it throws error:

We have no expertise in the WinRT media APIs. I suggest you try StackOverflow or other appropriate site where experts on those APIs hang out.

It turns out that there exceptions have no negative impact on my code. They will appear when I call the StartAsync on cameras, but my cameras can work without error.


Health indicator should support being disabled

Hi.
There was an issue https://github.com/spring-cloud/spring-cloud-kubernetes/issues/449.
It reproduces in spring-cloud-kubernetes-fabric8-autoconfig:2.0.3.
Hope it fill be fixed soon, thanks!

The conditional is still there
https://github.com/spring-cloud/spring-cloud-kubernetes/blob/c2fd0a1f8816de943de322a1bfa39987523336f9/spring-cloud-kubernetes-client-autoconfig/src/main/java/org/springframework/cloud/kubernetes/client/KubernetesClientAutoConfiguration.java#L79
Can you provide a complete, minimal, verifiable sample that reproduces the problem? It should be available as a GitHub (or similar) project or attached to this issue as a zip file.

Hi. Thanks for fast response. I've made a little demo. Hope it helps.
https://github.com/neonomaly/spring-cloud-fabric8-health-bug

The annotation is still there. The problem is not in the annotation, but the fact that you are using spring-cloud-kubernetes-fabric8-config dependency.
This dependency has a "bootstrap configuration" attached to it, specifically it imports Fabric8AutoConfiguration; as such setting the management.health.kubernetes.enabled=false in application.yaml would have no effect on that bean.
To put in other words, if you would have used spring-cloud-kubernetes-fabric8-autoconfig, everything would have worked (since it does not have a bootstrap step); but since you use spring-cloud-kubernetes-fabric8-config, this would not help.
The solution is trivial: place your property to disable that health indicator in bootstrap.yaml.
To that extent, I have been looking into the documentation and this is a bit blurred. imho, we need to properly fix the documentation, which I gladly will, if Ryan agrees.

It seems to be a little more complicated than that.  This is only a problem when using webflux, when using spring-boot-web there is no need to have a bootstrap.yaml.  I suppose the workaround for now is to put in in bootstrap.yaml but I would like to understand why it is behaving differently when using webflux.

So the difference appears to be how the beans are fetched for the different configurations.
The non-reactive configuration fetched the beans through the application context
https://github.com/spring-projects/spring-boot/blob/main/spring-boot-project/spring-boot-actuator-autoconfigure/src/main/java/org/springframework/boot/actuate/autoconfigure/health/HealthEndpointConfiguration.java#L81
The reactive implementation has them injected
https://github.com/spring-projects/spring-boot/blob/main/spring-boot-project/spring-boot-actuator-autoconfigure/src/main/java/org/springframework/boot/actuate/autoconfigure/health/ReactiveHealthEndpointConfiguration.java#L48
I'm not sure there is much we can do from a Spring Cloud perspective to work around that.

The problem seems to be in the way reactive and non-reactive health contributors are picked up. I'll try to explain.
When you use org.springframework.boot:spring-boot-starter-web, spring-boot under the hood will pick-up every instance of HealthContributor from the application context, as seen in the proper configuration. Specifically notice :
    applicationContext.getBeansOfType(HealthContributor.class)

In this case @ConditionalOnEnabledHealthIndicator("kubernetes") works
In the reactive case, they are injected directly into a method, as seen here and (to my surprise), that conditional does not work here.

Ryan made the same conclusion it seems! but much less verbose... @ryanjbaxter why wouldn't that conditional work in case of the reactive config?

nvm. I debugged this and understood how things happen...

If we look at the Javadoc for getBeansOfType we see that it does not consider the application context hierarchy when looking for the beans.
https://docs.spring.io/spring-framework/docs/current/javadoc-api/org/springframework/beans/factory/ListableBeanFactory.html#getBeansOfType-java.lang.Class-
Ultimately this comes down to just something we are going to have to document for now.  Fetching configuration data via the K8S API is not something we want to maintain long term so doing anything more than that at the moment seems counter productive.

Closed via 6dcad3a


Extend to_dataframe() method to some bc list commands

Description
Extend to_dataframe() method to following pymapdl methods: dklist, dllist, dalist, fklist, sfllist, bfklst, bfllist, bfalist.
Issue linked
No issue linked
Checklist

[x] I have visited and read Develop PyMAPDL.
[x] I have tested my changes locally
[x] I have added necessary documentation or updated existing documentation.
[x] I have followed the PyMAPDL coding style guidelines.
[x] I have added appropriate unit tests that also ensure the minimal coverage criteria.
[x] I have reviewed my changes before submitting this pull request.
[x] I have linked the issue or issues that are solved to the PR if any.
[x] I have assigned this PR to myself.
[x] I have marked this PR as draft if it is not ready to be reviewed yet.
[x] I have made sure that the title of my PR follows Commit naming conventions (e.g. feat: adding new MAPDL command)

I'm taking care of the permissions issue on the CICD. Give me some time @pvargasm . Ping me if I have nothing by the end of the day.

Hi @pvargasm
I think we are going to need to close this PR, so you can open a new one, but do not use your fork please. As the forks don't have access to repository secrets, and those are needed for pulling the docker image, etc.


Parse from bytes::Bytes iterator

Solve my #611 .
BytesIteratorReader lacks document and test, please tell me the convention to add them.

There's an issue with this PR.
This PR does reading by wrapping Iterator<&Bytes> into Read. Which means CodedInputStream has no option to reuse underlying Bytes objects without copying them into memory. When CodedInputStream is constructed from Bytes, and when Bytes is requested (when tokio_bytes codegen option is used, see CodedInputStream::read_raw_tokio_bytes function), the generated messages bytes reference original Bytes object allocation, not create new allocation (which might be much more efficient).
In current form the PR does nothing which cannot be done outside of the crate.


Fix #212 : Update position property for opened accordion content

Fix bug #212 : I only change the 'position' property for opened accordion content to 'static'. Thus, the list created by awesomplete can be positionned as desired.
I made an example in my fork to check the bug : https://github.com/kserin/Phonon-Framework/blob/fix/212-example/examples/components/accordion-example/index.html .

Hi @kserin,
Thank you for your contribution!


Allow kappa=0 in von_mises_lpdf

Summary
Relaxed the boundary check in stan/math/prim/scal/prob/von_mises_lpdf.hpp for kappa so that 0 is a valid value. Fixes #564.
Tests
Added test for kappa = 0, with expected value generated in R with the circular::dvonmises function.
Side Effects
None.
Checklist

[X] Math issue #564

[X] Copyright holder: Marco Colombo
The copyright holder is typically you or your assignee, such as a university or company. By submitting this pull request, the copyright holder is agreeing to the license the submitted work under the following licenses:
- Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
- Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

[X] the basic tests are passing

unit tests pass (to run, use: ./runTests.py test/unit)
header checks pass, (make test-headers)
docs build, (make doxygen)
code passes the built in C++ standards checks (make cpplint)

[X] the code is written in idiomatic C++ and changes are documented in the doxygen

[X] the new changes are tested

To clarify the test I've added: the Von Mises distribution for kappa = 0 corresponds to the circular uniform distribution with density 1 / (2 * PI) (independently of the values for y or mu). And log(1 / (2 * PI)) = -1.837877 (the value checked in the test).

Nice. I think this should have an autodiff test too? Note that computing ops_partials.edge3_.partials_[n] divides by kappa. That's easy to fix, though.
Might be worth fixing von_mises_rng too. The algorithm there looks stable until rho rounds to zero, which happens around kappa<1e-8. At that point the distribution is indistinguishable from uniform so no clever math needed.

Thanks @nhuurre for your suggestions. I've fixed the expression for the derivative to work also in the case of kappa=0. I'll file an issue for the von_mises_rng when I'm done with this. Perhaps you know the answer to my question below. :)
Locally I have added a set of parameters to the distribution test (test/prob/von_mises/von_mises_test.hpp) for kappa=0, but it throws in the finite differences part because kappa becomes -1e-08. So is it correct to write a custom test using the autodiff framework instead of relying on the distribution tests? Any pointers as to where I could look for an example would be very helpful.

I've pushed a distribution test for kappa as close to zero as the finite differences would allow, and another hand-coded test inspired by test/unit/math/mix/scal/prob/normal_test.cpp. The latter doesn't fully work, there's a bit at the end for which I get -nan, so there's something I must be doing wront, but to be honest I don't fully follow the autodiff intricacies.
@bob-carpenter could you have a look if you see what of those tests should be kept and what should be fixed?

I went ahead and fixed von_mises_rng too (and updated the description accordingly). @nhuurre is this what you had in mind?

Yes, although that 1e-8 was just an order-of-magnitude guess. Testing a bit with Python it seems the bound needs to be 1.4e-8 to ensure nonzero rho. Also, the main branch does some "fmod with positivity constraint" adjustment to mu before centering the distribution. Just copying that blindly the uniform branch would be
output[n]
    = (uniform_rng() - 0.5) * TWO_PI
      + std::fmod(std::fmod(mu_vec[n], TWO_PI) + TWO_PI,TWO_PI);

Thanks! I'll post an updated patch after the rest of the flattening PRs go in as they may cause further conflicts.

This is ready for review.


Whitch App

With witch app can i make scenes to set channel? I tried several eg. the Eve app, HomeDash, myHome and Hesperus. With non of these it is possible. Non have implement these characteristics to set scenes with.

There is a sample app from Apple called HomeKit Catalog that has to be compiled on your device since it's not in the App Store. There is also a section in the homebridge readme with more infos.

Ok, thank you for the info.

You're welcome.


[New Port Request] embree4

Library name
Embree4
Library description
high-performance ray tracing library
Source repository URL
https://github.com/embree/embree/releases/tag/v4.3.0
Project homepage (if different from the source repository)
https://www.embree.org/
Anything else that is useful to know when adding (such as optional features the library may have that should be included)
older version of embree v3.x is already in vcpkg: embree3 | Version: 3.13.5

I am interested to tackle this in the future, however should we create yet another port or simply stop putting the version into the port name and just use the manifest version field ?
In a nutshell, I am proposing to deprecate the embree3 port - or making it a stub port - and create the embree port instead of embree4

please keep embree3 and create a new port for embree4. There are still many libraries which depend on embree3. Currently, it is cumbersome to install a specific version of a library from the command line. I believe proper versioning is only available in manifest mode
Having said that, thanks a lot for working on embree4, much appreciated!

Indeed, I am used to manifest mode only those days but I understand this can be painful when it is not used at all and have to force people into doing custom overlay ports...
That said, and assuming we will have a embree4 port. Would it be useful to have a embree meta port which simply point to the latest desired version (and which can also be either versioned and/or overlaid) ?

Such meta ports come with own maintenance issues: they have to mirror the actual port's features. Example: opencv vs. opencv4.
Having two ports can cause problems if they provide the same files (cf. opencv ports) or symbols (may break msbuild autolinking). It is possible to exclude the older version from vcpkg CI if the they provide the same files (cf. opencv3). But then it may break silently.

I'm also waiting for embree4. Thank you!

I also need embree4 in vcpkg. Having this "closed as not planned" is disappointing - embree4 is a well-known and widely used library. If such libraries are not available in vcpkg (or only old versions like embree3 3.13), then what's the point of it?
/CC @dg0yt


fixes size for images of stream block

fixes #1038

fixes size for images in streamblock

Would it be complicated to also add this for images that are contained within paragraphs?

Just to confirm, we'd ideally like the serving size of 360 px width also for images within paragraphs.

fixed size for images in paragraphs as 360


Can table be expanded to fill in unused space

Example URL(s)
https://developers.cloudflare.com/logs/reference/log-fields/zone/http_requests/
Actual Behavior
On Firefox the 3-columns table has a fixed large first column then a scrollable list of 2 columns. By default, on a macOS laptop 13", the 3rd column isn't visible unless scrolled to the right. Although there's unused space on the right.
Expected Behavior
Table to fill in space on the right. Eventually have a smaller first column with word wrapping.
Additional information
No response

We'd need the frontmatter in the source template changed to be layout: list.


use gtests framework

Complete missing tests
Update test with Google Tests Framework

Codecov Report
Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (88b4dd1) 35.20% compared to head (9f5387a) 35.61%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #199      +/-   ##
==========================================
+ Coverage   35.20%   35.61%   +0.40%
==========================================
  Files          14       14
  Lines        2701     2701
  Branches     1166     1166
==========================================
+ Hits          951      962      +11
+ Misses       1121     1114       -7
+ Partials      629      625       -4

Files
Coverage Δ

clic/src/cudabackend.cpp
6.15% <ø> (ø)

clic/src/tier1.cpp
35.39% <0.00%> (-0.10%)
:arrow_down:

... and 5 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Notice of Copyright/License Violation

This is a notice of Copyright violation against the PIVX developers in which this project has been forked from. The MIT copyright license has been modified to explicitely exclude the PIVX developers' copyright.
You are hereby notified of an intention to pursue further action if this issue is not acknowledged and formally addressed with a resolution timeframe commitment wherein. Failure to acknowledge and correct the violation will result in a formal DMCA takedown notice filed with GitHub Proper after no less than five business days from the date of this notice.
Explicit acknowledgement and referencing progress to maintain compliance MUST be noted in this issue's comments or referred to in any subsequent pull requests to this repository.

Can you please explain what you feel is in violation of copyright?  If so, we'd be happy to correct it.
We haven't modified the MIT license in any way.

https://github.com/Nitrous-Core/nitrous/blob/master/COPYING
You have explicitly removed the required PIVX copyright from the license/use agreement file. That file (and all the source code headers) are binding; meaning that any removal or editing of prior copyrights is against the use license.

I understand what you are saying be WE didn't remove it.  We forked the code from another project - Wilo - and all copyrights that were in place when we forked it remained in place.  We would have no reason to "explicitly remove" a PIVX copyright (which appears everywhere else in our software).  If you are saying the copyright is missing specifically from the MIT license file, I see that and I have corrected it.
Again, we removed nothing related to copyrights.  We forked the code from a different project.  If something was removed, it was prior to our fork.  Therefore, I can't be 100% sure what you want me to add.  For now, I have added PIVX to the license page, but if there are other places PIVX should be mentioned and it is not, you'll need to tell me where those places are.  Thanks.

I understand your dilemma and your response, but the fact remains that your source code is in violation of a particular copyright, and it is your responsibility to correct it, even if the source you forked from is also in violation.
I do see that you have corrected the situation in https://github.com/Nitrous-Core/nitrous/commit/dc6c9b04083e736205e8ea47f87a2b113fc314f3 and will thus close this issue and consider it resolved for your part, though I would ask that you provide me a link to the source repository you forked from so I can address the issue there as well.

https://github.com/WiloTeam/Wilo
Doesn't seem to be much there now.

Ok, yep...looks like a dead/scam project.
You have resolved the issue on your own, so that is all that is required at this point. Regarding your edit to your above comment: it doesn't matter where you sourced/forked from; all open source projects are inherently liable for any and all copyright or license/use stipulations.
Had you forked the PIVX repository directly, this likely would not have been an issue at all, but a copy of a copy is still responsible for any and all original copyrights or use licenses even if said copyrights/licenses have been altered elsewhere. The only thing that we (PIVX) can operate on when a project doesn't follow github's forking guidelines is the source at hand (ie, we are not aware of the fact you copied the source from a 3rd party), and so any modifications to the copyright/license are inherited by your project.

Understood.  But the same logic applies to the person forking the source code - they would have absolutely no way of knowing that there is a missing copyright unless they are told.  PIVX isn't in every Github project.  If the project I'm forking from doesn't mention PIVX, It should be safe to assume that PIVX didn't create any of it (ie, we are not aware of the fact that the source we copied was a "3rd party").  I appreciate you bringing this to our attention so that we could fix it.  In the future though, I would suggest approaching with a more cooperative attitude.  We both know that the language you used was a bit dramatic for a copyright missing in ONE text file.  It appears everywhere else. And although I'm not an attorney, the copyright in that specific place seems hardly worth mentioning. You must have been looking pretty hard to find something so insignificant.


fix readthedocs build errors

Branch master currently fails because:
python -m mkdocs build --clean --site-dir $READTHEDOCS_OUTPUT/html --config-file tools/mkdocs.yml
ERROR   -  Config value 'theme': Unrecognised theme name: 'material'. The available installed themes are: mkdocs, readthedocs
ERROR   -  Config value 'plugins': The "exclude" plugin is not installed

It likely needs a
pip install mkdocs-material mkdocs-exclude

but it is not immediately obvious where to put them.

I believe we need to add the additional package requirements to https://github.com/twosixlabs/armory-library/blob/master/.readthedocs.yaml (following https://docs.readthedocs.io/en/stable/config-file/v2.html#packages).


Eloquent Resource resolve() iterate over nested resources.

Laravel Version: 5.5.7
PHP Version: 7.1.8
Database Driver & Version:

Description:
Manually resolve an Eloquent Resource should (in my opinion) resolve any other resources defined in the returned array.
Should be:
(new UserResource($user))->resolve() == ['profile' => [...]]
Not
(new UserResource($user))->resolve() == ['profile' => UserProfileResource {...}]
Steps To Reproduce:
Set a relation as an attribute in a resource then resolve that resource, should result in a nested array.
Current results in an array with an instance of the related resource.
eg:
class UserResource extends Resource
{
    public function toArray($request)
    {
        return [
            'profile' => new UserProfileResource($this->whenLoaded('profile')),
        ];
    }
}

dd(new UserResource($user)->resolve());

For suggestions and feature requests please use the https://github.com/laravel/internals repo.


release to npm

notably, for the client cert support. happy to do this for you if you add me to the project.

I'm not an owner of the npm package so I can't do this.
cc: @3rd-Eden

Released to npm, also added @lpinca to websockets/* npm packages.

(released as 1.1.0)


turn off for some commands

Hi, can you please add some more context to this bug report or feature request? It’s hard for me to understand from an incomplete sentence alone.

@ericfreese yes you're right, sorry about that. I edited the first post.

Do you want to keep these long commands in your history? If the command is not in history, it won't be suggested by this plugin.
You can use HISTORY_IGNORE param or fiddle with the zshaddhistory hook to control what is saved to history. See this thread on the mailing list: http://www.zsh.org/mla/users/2015/msg00147.html
You might also be interested in histignorespace option that allows you to prefix a command with a space to omit it from history.  And here's some more info including removing existing history entries: https://stackoverflow.com/a/13661794/154703

Hm, that certainly helps, but it's hard to cover every case of long commands that I might write

@theonlygusti Please try features/history-ignore branch (PR #456). I have yet to write tests, but I think it will work for your use case.
Say that you don't want suggestions for youtube-dl or e.g. curl:
ZSH_AUTOSUGGEST_HISTORY_IGNORE="(youtube-dl *|curl *)"

Merged to develop. Will go out with the next release.


update reliable

Codecov Report

Merging #209 into release-rc2 will not change coverage.
The diff coverage is n/a.

@@              Coverage Diff               @@
##             release-rc2     #209   +/-   ##
==============================================
  Coverage          28.63%   28.63%
  Complexity           330      330
==============================================
  Files                110      110
  Lines               3573     3573
  Branches             255      255
==============================================
  Hits                1023     1023
  Misses              2470     2470
  Partials              80       80


Block data for each annotator

Description

When the annotators labeled data in workspace, another annotators also labeled this data in the same workspace and task. The problem is that data fetched does not be allocated for each user. In the other word, the first annotator labels for document 1, and the another also labels for document 1, they do not whether teammate have done this document or not. In current solution, annotator coming later will overlaps in the results of that of the first one. This makes many users work on the same task leading to conflict the results.

Solution

Data is allocated for each annotator, and invisible for the others but workload of the project is visualized
Reference: https://rasa.com/docs/rasa-enterprise/user-guide/annotate-nlu-examples/

Hi @anhphong22 ! We had a similar problem while doing an internal annotation project and we found a workaround: we added an "annotator" field in the metadata and we assigned the records randomly between the available annotators, then we asked them to filter the records with their name using the "Metadata" filter in the UI. That way they only see records assigned to them and we avoided overlap.
Does this sound like something that solves your issue?

Hi @ Natalia Elvira
Your solution sounds good. From my experience, I design a database with multiple stage for fetching and syncing data from local to remote. The annotator was assigned or selected what records you want, these records sent status (in progress, subjected to timeout) to remote database, meaning that the others could not employ these records.
From: Natalia Elvira @.>
Date: Monday, 17 April 2023 at 15:59
To: argilla-io/argilla @.>
Cc: Nguyen Thanh Phong @.>, Mention @.>
Subject: Re: [argilla-io/argilla] Block data for each annotator (Issue #2688)
Hi @anhphong22https://github.com/anhphong22 ! We had a similar problem while doing an internal annotation project and we found a workaround: we added an "annotator" field in the metadata and we assigned the records randomly between the available annotators, then we asked them to filter the records with their name using the "Metadata" filter in the UI. That way they only see records assigned to them and we avoided overlap.
Does this sound like something that solves your issue?
—
Reply to this email directly, view it on GitHubhttps://github.com/argilla-io/argilla/issues/2688#issuecomment-1510962685, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AR2ROBAUOKISDRXCOY25UGDXBUA6LANCNFSM6AAAAAAW4RCWRQ.
You are receiving this because you were mentioned.Message ID: @.***>

I'm thinking of a solution to do this using roles and workspaces.  I'm putting together a tutorial inspired by this issue #2705


Add unit tests for render engines

Adds initial framework to run pytest tests.
Test text render engine.

Woah, this is really awesome!!!
I know from the draft state that you're not done. My main thought is that we shouldn't write files into the repo; instead we should use the tmp_path fixture.
My other thought is that I wonder if there's a simple way to automate regenerating the images (e.g. if we change the default font), but that's less important.

Ah, and because of Python import weirdness, you should add import PIL.ImageOps in addition to import PIL at the top of test_render_engines.py.

@tomers, you should probably rebase this on the latest main

I don't like running pytest in pre-commit. Individual commits should be able to break tests. For example, this is desirable in test-driven development where you first commit a broken test and then fix it in the subsequent commits.

I rely on pre-commit for linting and type-checking which I find essential for lowering my commit entropy.
Including pytest will encourage developers to make larger commits which makes review difficult.
Also, I can't think of a single major open-source project that runs pytest in pre-commit. I think there are good reasons for that.

My main thought is that we shouldn't write files into the repo; instead we should use the tmp_path fixture.
I prefer not to. The reason for this is that:

I delete each created file when the test succeed
In case of test failure, the file is in the directory just above the reference expected_renders directory, thus it is easy to compare. In case the new file is the correct one (e.g. following a code change), we can just move the file into that directory. It is very easy to do from the IDE, where the actual image file is near the code, the test code, and the reference image file, and by simple and short drag-and-drop action it can be moved to the expected_renders directory, and be committed.

My other thought is that I wonder if there's a simple way to automate regenerating the images (e.g. if we change the default font), but that's less important.
I don't think we should. I prefer to do it manually, and examine each change. As I wrote above, we can just drag and drop in order to move the new version of the file to its designated place.

I rely on pre-commit for linting and type-checking which I find essential for lowering my commit entropy.
Including pytest will encourage developers to make larger commits which makes review difficult.
Also, I can't think of a single major open-source project that runs pytest in pre-commit. I think there are good reasons for that.
Ok, I agree to your reasoning. I've removed it. Instead, I've added it to the tox cofiguration in pyproject.toml. Please review this, and see if I did it in a proper manner. I want to allow developers to type a short command (e.g. tox to run all tests, including generation of coverage reports and any other artifacts of a test run).

@tomers, I just started to review. Are you still making changes?

Marking as draft for now, just click "Ready for review" when you'd like me to take a look.

This PR is becoming pretty big, so I'll stop here, and submit my future changes in a separate PR

It's a lot more conventional to put tests/ alongside src/. I don't feel particularly strongly about this, but unless there's a particular reason it may be better in terms of standardization to move it there.

I think it would be better to have separate tests directory in each relevant code section, e.g. render engine tests under src/labelle/lib/render_engines/tests. This will prevent building of similar hierarchy under a global tests directory, which might break in the future, if things move along.

I haven't looked in detail about why = None is necessary for the rendering init, but it stands out to me as odd.
Probably we shouldn't need to weaken function signatures in order to make tests easier. Maybe add a kwargs to the parameterize?

In cli.py we explicitly pass these arguments, which might be None, thus we must support the case where the argument value is None, and handle it in the function body, instead of in the declaration.

This still seems fishy to me. Why is the value None in the first place? If they're coming from cli.py then perhaps our defaults there are incorrect. For example, I think this is wrong:
https://github.com/labelle-org/labelle/blob/d75a71e63fb2864d2105b93f926c1ef4bb9b46be/src/labelle/cli/cli.py#L214
Shouldn't it be this?
] = DEFAULT_BARCODE_TYPE

Also, I'm not sure if you saw this remark, or if it's hidden for you: https://github.com/labelle-org/labelle/pull/32#discussion_r1584773683
Thanks for all your work on this!

@maresb I've fixed the default handling of barcode type.
If possible, please review this and merge.
I got lost amongs the different PRs that we got. I'll leave it to you to merge.
I guess I'll have some more breaking changes coming up soon. Would it be ok that you release a new major version with breaking changes, and then we introduce more breaking changes, or go you prefer that we keep work on main branch, without releasing, and then release once we're done?

@tomers, would you be able to address the remaining two default arguments? It's pretty minor, so please just merge yourself afterwards.
My wish would be to merge this PR, then #37, then #25, make one or more patch releases, and then start with the breaking changes. Would that work well with your plans?

@maresb, I am blocked from merging this PR.  Please merge.

I am blocked from merging this PR.

That's annoying, sorry about that! I don't want that to happen.
Looking under https://github.com/labelle-org/labelle/settings/branch_protection_rules/49111150, I see

I think that's what's preventing you from merging as a maintainer. I'd prefer to keep this normally checked in order to avoid accidents, but please feel free to temporarily remove this protection in cases like this.

Unchecking that box gives maintainers the following override:


Add ProbabilisticSVC and ProbabilisiticNuSVC

Closes #15
Apologies to reviewer: the 3rd commit is messy because of some reformatting. The only essential addition is the "probabilistic classifiers" testset.

Codecov Report

Merging #16 (89589a3) into master (5ed884a) will increase coverage by 1.56%.
The diff coverage is 94.11%.

:exclamation: Current head 89589a3 differs from pull request most recent head 2135dfb. Consider uploading reports for the commit 2135dfb to get more accurate results

@@            Coverage Diff             @@
##           master      #16      +/-   ##
==========================================
+ Coverage   90.90%   92.47%   +1.56%
==========================================
  Files           1        1
  Lines         176      186      +10
==========================================
+ Hits          160      172      +12
+ Misses         16       14       -2

Impacted Files
Coverage Δ

src/MLJLIBSVMInterface.jl
92.47% <94.11%> (+1.56%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Closes #15
Apologies to reviewer: the 3rd commit is messy because of some reformatting. The only essential addition is the "probabilistic classifiers" testset.

@ablaom Why restrict this to only SVC and NuSVC. Is there anything stopping us from adding this to all SVM models including epsilon-SVC

Closes #15
Apologies to reviewer: the 3rd commit is messy because of some reformatting. The only essential addition is the "probabilistic classifiers" testset.

@ablaom Why restrict this to only SVC and NuSVC. Is there anything stopping us from adding this to all SVM models including epsilon-SVC

Nevermind. I see why

Typos corrected. Another good catch.


math support in markdown with utterance.es

Github recently introduced math support in markdown, for example I can write $x^2$ in this issue. But math support doesn't appear to work with utterances. This would be, in my view, a really killer feature.

I would love to have this feature as well! $maths$
$$maths$$


Historical retrieval

What this PR does / why we need it:
The current SQL template of point in time join does not scale
Problem 1: ROW_NUMBER() does not scale
In order to calculate a unique ID for each row of the entity dataframe, we compute the following
SELECT ROW_NUMBER() OVER() AS row_number, edf.* FROM {{ left_table_query_string }} as edf

The problem is that BigQuery will need to send all the data to a single worker in order to properly calculate the row number of each row. For our use case, we end up with a OOM error

Solution
The solution is to calculate a deterministic hash that will act as a unique identifier.
Because the entity_dataframe should contain all entity keys, I use FARM_FINGERPRINT() that will compute a deterministic hash for a given input. This hash is computed in a distributed fashion as it only needs the datapoints of a given row.
WITH entity_df AS (
    SELECT
        *,
        FARM_FINGERPRINT(CONCAT(
            {% for entity_key in unique_entity_keys %}
                CAST({{entity_key}} AS STRING),
            {% endfor %}
            CAST({{entity_df_event_timestamp_col}} AS STRING)
        )) AS entity_row_unique_id
    FROM {{ left_table_query_string }}
),

Alternative
I tried GENERATE_UUID() that is non deterministic and the query got wrong because i suspect that it computed it multiple times (depending on how the SQL query gets optimized & parsed). So we ended up with all features being always Null
TODO: Matt can look at how the query is interpreted and see if GENERATE_UUID() is called multiple times
Problem 2: Window functions and ORDER BY are often the bottleneck
As a former data scientist, I often realized that big data engines (Spark, BigQuery, Presto, etc.) are often much more efficient with a series of GROUP BY rather than a Window function. Moreover, we should avoid ORDER BY operations as much as possible.
So this PR comes with a new approach to compute the point in time join. This is solely compose of JOINs and GROUP BYs
Here are the results of my benchmark:
Context
I perform the same query using both templates. For the original template, I switch the ROW_NUMBER() of the entity dataframe by the FARM_FINGERPRINT() one  as explain above.
The api call is the following
result = store.get_historical_features(
    entity_df="""
    SELECT user_id, TIMESTAMP '2021-01-01' AS observed_at
    FROM `my_dataset`
    LIMIT 100000000 -- 100M
    """,
    feature_refs=[
        "feature_view_A:feature_1",
        "feature_view_A:feature_2",
        "feature_view_B:feature_3",
        "feature_view_B:feature_4",
        "feature_view_B:feature_5",
        "feature_view_C:feature_6",
        "feature_view_D:feature_7",
    ]
)

And some idea of the scale of this historical retrieval

feature_view_A contains ~5B rows and ~3.6B unique "user_id"
feature_view_B contains ~5B rows and ~3.6B unique "user_id"
feature_view_C contains ~1.7B rows and ~1.1B unique "user_id"
feature_view_D contains ~42B rows and ~3.5B unique "user_id"

Results
On the original SQL template

Elapsed time: 5 min 59 sec
Slot time consumed: 5 days 5 hr
Bytes shuffled: 5.89 TB
Bytes spilled to disk: 0 B

With the SQL template of this PR

Elapsed time: 3 min 21 sec (-44%)
Slot time consumed: 2 days 12 hr (-52%)
Bytes shuffled: 3.55 TB (-40%)
Bytes spilled to disk: 0 B

So as we can see, the proposed SQL template consume half the resources that the one currently implemented.
Also, because this new SQL template is only composed of JOINs and GROUP BY, it should scale "indefinitely" except if the data is highly skewed (eg: a single "user_id" represents 20% of the dataset).
Which issue(s) this PR fixes:

Fixes #
Does this PR introduce a user-facing change?:

The description is clear and easy for me to understand. The impact of the solution looks great, @MattDelac!


Enable the new model serialization for Batch

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.Batch


[WIP] [FIX] Enable processing of walletFiles

This PR is adding node specific 'fs' module requirement to eac.js-client. This will require extra effort in non-node environments.
What's the reason for having other way of reading the wallet?

The fs module only comes in when a wallet file is used, instead of a key store.
An option will be to move the require for fs into the if condition.
Though, I do not know about that

Why not add fs.readFileSync(opts.walletFile, 'utf8'); in eac.js-cli and leave eac.js-client as it was ?

Reading wallet from the file is a specific why to initialise the wallet, whole point of have higher level abstraction is to be open for other ways to deliver the wallet. That's why we just keep the way to pass it as keystore, so browser will read from localStorage, cli from file etc

implementing suggested separation of concerns.
wil update when ready

@kosecki123 @lsaether ready. Kindly re-review


redundancy

Challenge http://www.freecodecamp.com/challenges/waypoint-target-html-elements-with-selectors-using-jquery has an issue. Please describe how to reproduce it, and include links to screenshots if possible.
not sure if this is a bug or typo but the 2 challenge tasks are repeated here.

Hey @duanwalker thanks for posting!
I don't see this issue.  I'm going to close this now but feel free to reopen if you still notice it and provide some screenshots.


compatibility with webpack

(Fixes #1)
Now it's possible to use this module with webpack, by importing it directly:
import "pure-swipe";

or
require('pure-swipe')

Great, thanks @endel


Please add support for S40ZBTPB Lite

Please add support for the following:
12/16/2023, 12:28:46 PMeWeLink BridgeeWeLinkWARNING[Device022f18] is not supported, do create a GitHub issue with the following info:
[7005] {"bindInfos":{},"subDevId":"f358e829004b12007005","parentid":"1001dc6ebb","fwVersion":"1.1.0","switch":"off"}.
Thank you

Hi @trhessel
Do you still have this message on v12.1.0 of the plugin?

No, all good and I am able to control this via homekit. Thank you for
adding this and following up.
On Tue, Dec 26, 2023 at 9:12 AM Ben @.***> wrote:

Hi @trhessel https://github.com/trhessel
Do you still have this message on v12.1.0 of the plugin?
—
Reply to this email directly, view it on GitHub
https://github.com/bwp91/homebridge-ewelink/issues/532#issuecomment-1869573105,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALCRICND5ASW6HS3XYING6LYLLLORAVCNFSM6AAAAABAXXF4GCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRZGU3TGMJQGU
.
You are receiving this because you were mentioned.Message ID:
@.***>


Incorrect type hint

Affected module
Ingestion Framework
Describe the bug
The type hint in a function, yield_table_tag_details, is incorrect.
To Reproduce
N/A
Expected behavior
The table_name_and_type param type should be Tuple[str, TableType]

I can help make a PR to fix it if need.


Clock icon for scheduled docs in the sidebar.

We’re interested in adding an indictor to the DataDoc sidebar to quickly see which DataDocs are scheduled or not.  Something like the screesnhot below.  Does this seem OK?  If you don’t have any issues I can make a PR.

yup this sounds like a great idea!

@czgu Is it acceptable to create a new field "scheduled" for datadoc. There might be a change in the database to add a column "scheduled" to the table 'data_doc'.


indent: bad indentation in chained methods calls with generics

Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.

Describe the bug
when chaining methods, if one of the method contains generics, the indentation computing fails
In below case it is the reducer function: its callback ask indentation of 2 instead of indentation of 4
const map: Record<string, string> = {};

const map2 = Object.keys(map)
  .filter((key) => true)
  .reduce<Record<string, string>>((result, key) => {
    result[key] = map[key];
    return result;
  }, {});

  6:1  error  Expected indentation of 2 spaces but found 4  @stylistic/indent
  7:1  error  Expected indentation of 2 spaces but found 4  @stylistic/indent
  8:1  error  Expected indentation of 0 spaces but found 2  @stylistic/indent

✖ 3 problems (3 errors, 0 warnings)
  3 errors and 0 warnings potentially fixable with the `--fix` option.

Reproduction
https://stackblitz.com/edit/github-caheqy?file=src%2Findex.ts
Contributes

[ ] I am willing to submit a PR to fix this issue
[ ] I am willing to submit a PR with failing tests

I update my sample: https://stackblitz.com/edit/github-caheqy-gwabam?file=src%2Findex.ts
We still have an error: it only works for the top level items.
Inside a class for example we have the indentation bad lint


How to Reduce the size of camera source in mobile.

Text detection previews full screen to scan text. I want a small rectangle for text detection.
I want to reduce the area of camera preview and also i want to capture only the text within the area by the camera.
I have reduced the camera preview area. but the camera is capturing the full area and recognizing text..... I want only that text to be recognized which is within the preview.

any update on this ?

@shiv71 you can use corner points and filter the results using your recognition area coordinates

Check this https://www.youtube.com/watch?v=kOYP2Xun_6o&list=PLQhQEGkwKZUrempLnmxjt7ZCZJu1W3p2i&index=77


Update README.md

Add schema in the solution.

I test the solution with postgres version 11 on debian buster


Positioning variable depending on breakpoint

I want to set up my toast as 'top-right' on lg+ resolutions and 'bottom-centre' on all others.
Is there a way to do that through the Toaster options?

me too i need this?

these options have or not??

Not yet as far as I can tell. Even if positioning can be disabled so I can position it myself with Tailwind classes that would work. But of course a way to do that through props would be preferred.

You have window.innerWidth to detect width of your screen.
<Toaster position={window.innerWidth >= 1500 ? "top-center" : "top-left"} />

Let's use @aXenDeveloper for now.


GhostBSD fails to boot to desktop when running scfb driver on RX6600 after April 16 update[Bug]:

Version
23.04.15
Explain the issue and what happens
Installed GhostBSD with no manual graphics configuration with scfb driver on previous GhostBSD version 22.04.02.
Did update through update-station. Reboot and GhostBSD fails to get to the desktop. Issue is fixed by manually creating an scfb-driver.conf file in /usr/local/etc/X11/xorg.conf.d/ with the following contents
Section "Device"
Identifier    "Card0"
Driver        "scfb"
EndSection
scfb driver must be used bcause RX6600 not yet supported in GhostBSD with amdgpu driver
Explain how to reproduce the bug?
Install GhostBSD with scfb autoconfiguration and update system to latest version.
What is the Expected behavior (if applicable)
GhostbSD should update and reboot to slick-greeter.
Additional context (if applicable)
probably not that important because of low numbers of scfb driver users

issue moved from ghostbsd/ghostbsd-src to ghostbsd/issues


Depreciated under v2

now -V 1 zeit/hazel

Solved by https://github.com/zeit/hazel/pull/54

albeit it does seem that one needs to append -e URL=NOW_URL to any deploy as it does not automatically resolve the url

Fixed! Cutting a new release shortly.


Faulty output on mimeparse

Just tried pypi-changes, looks super cool! However, I found something weird with mimeparse:
$ pypi-changes .venv/bin/python
🐍 Distributions within /home/juanlu/Projects/LSF/poliastro/library/.venv/bin/python
├── mimeparse 0.1.3 11 years ago remote 0.1.4 3 seconds from now
├── cryptography 3.4.7 7 months ago remote 36.0.0 an hour ago
├── ujson 1.35 5 years ago remote 4.3.0 an hour ago
├── hypothesis 6.14.6 3 months ago remote 6.27.0 9 hours ago
...

Looks like a bug?

A PR fixing this would be welcome :+1:


Grafana dashboard not working

garie version: no idea, I git cloned it today (October 14th 2019)

Relevant code or config
Stock configuration
What you did:

git clone garie
edit start.sh: removed the PAGESPEED_INSIGHTS_KEY
edit garie-config.json: change the URL
sh start.sh

What happened:
Installation from docker-compose, then at runtime I got this message:
... [truncated] lots of messages ...
grafana_1                   | GF_PATHS_DATA='/var/lib/grafana' is not writable.
grafana_1                   | You may have issues with file permissions, more information here: http://docs.grafana.org/installation/docker/#migration-from-a-previous-version-of-the-docker-container-to-5-1-or-later
grafana_1                   | mkdir: can't create directory '/var/lib/grafana/plugins': Permission denied
garie_grafana_1 exited with code 1
... [truncated] lots of messages ...

Problem description:
Seems like the default volume mapping doesn't writing to /var/lib (in my machine it's owned by root). So for a quick test since I'm just evaluating garie I ran sudo sh start.sh (I know it's a bad idea from a security POV and that in a prod settings devops will set this up correctly), and instead I got this:
... [truncated] much less messages since docker images were pulled before...
then lots of line about grafana migration =>
grafana_1                   | t=2019-10-14T13:43:15+0000 lvl=info msg="Executing migration" logger=migrator id="create org_user table v1"
then errors:
grafana_1                   | t=2019-10-14T13:43:38+0000 lvl=eror msg="Anonymous access organization error: 'Viewer': Organization not found"
grafana_1                   | t=2019-10-14T13:43:38+0000 lvl=info msg="Request Completed" logger=context userId=0 orgId=0 uname= method=GET path=/ status=302 remote_addr=<IP_ADDRESS> time_ms=0 size=29 referer=
grafana_1                   | t=2019-10-14T13:43:38+0000 lvl=eror msg="Anonymous access organization error: 'Viewer': Organization not found"

So I checked Grafana docs and changed garie's config/grafana/grafana.ini: org_name = Viewer to org_name = test (to match the provisioning done in config/grafana/provisioning/dashboards/dashboard.yml), and now I got a log that should mean grafana server is started, but localhost:3000 still doesn't work
grafana_1                   | t=2019-10-14T14:05:52+0000 lvl=info msg="HTTP Server Listen" logger=http.server address=<IP_ADDRESS>:3000 protocol=http subUrl= socket=

At this point, I have no idea what else to fix or to look at.
Suggested solution:
Maybe something in Grafana changed since last time you checked?

Hi @DjebbZ ,
Did you manage to get this working?

No, in the end we decided not to use garie. I should close the issue I suppose.


A job not start when device restarted.

Hello. I don't know, actually, what kind of reason, but my job not start when device restarted. This is my code:
BackgroundJob.register({
jobKey: exactJobKey,
job: () => console.log(new Date())
});
    BackgroundJob.schedule({
        jobKey: exactJobKey,
        period: 10800000,
        allowExecutionInForeground: true,
        exact: true,
        persist: true
    });

So,  console.log(new Date()) is working only then I start my app manually. Should this task run automatically?

Did you try running the example project?

I have the same issue. My cfg:
`
var name = 'test';
var period = 30;
var exact = false; // or true, it doesn't matter for that issue (after 1 hour after device restart still no bg job fired)
BackgroundJob.schedule({
jobKey: name,
period: period * 1000,
persist: true,
exact: exact,
allowExecutionInForeground: true
});
`
Maybe the issue occurs when allowExecutionInForeground is set as true?

No it's that you have exact execution chosen. That should be added to the readme.

If this helps, registering the background job this way works.
BackgroundJob.schedule({
jobKey: regularJobKey,
period: 15000
});
This is given in the example and i just tested this on emulator. To restart the emulator, adb -s  reboot


remove performance from readme table of contents

this section was removed in 8b59c35805314d4e79889c18441f90d2b8523de2

Oh well, yes. Why is it still there?

Thanks a lot @slang800, sorry for the long delay.


"thumbnail is invalid" not reset with new thumbnail upload

Not sure how users are running into this yet, but I think they add a bad URL and then later use the upload tool - but it won't clear the error after that.

Re-open
I missed out the main piece (the 'upload' case) -- see #6072.  Thanks to @keikari for highlighting the mistake.


feathers-rest returns Internal Server Error on invalid login

While testing auth I have noticed that feathers-rest returns Error: Internal Server Error on invalid login while both socket transports return {name: "NotAuthenticated", message: "Invalid login.", code: 401, className: "not-authenticated", errors: Object}. Here is a gist with an example: https://gist.github.com/juodumas/ef831aa1f595e7745d7e418108bea461

Thank you for the Gist, that helps. Having a look at it right now.

The problem is that you have to use the feathers-errors error handler. If the server looks like this it should work:
// npm init --yes
// npm i feathers feathers-memory feathers-authentication feathers-hooks feathers-rest feathers-socketio feathers-primus body-parser
const bodyParser = require('body-parser')
const feathers = require('feathers')
const hooks = require('feathers-hooks')
const rest = require('feathers-rest')
const socketio = require('feathers-socketio')
const primus = require('feathers-primus')
const authentication = require('feathers-authentication')
const memoryService = require('feathers-memory')
const errorHandler = require('feathers-errors/handler');

const app = feathers()
        .configure(rest())
        .configure(socketio())
        .configure(primus({transformer: 'websockets'}))
        .configure(hooks())
        .use(bodyParser.json())
        .use(bodyParser.urlencoded({extended: true}))
        .configure(authentication())
        .use(feathers.static(__dirname + '/public'))
        .use(errorHandler())

app.use('users', memoryService())
const users = app.service('users')

users.before({
  create: authentication.hooks.hashPassword('password'),
  update: authentication.hooks.hashPassword('password'),
  patch: authentication.hooks.hashPassword('password')
})

users.create({
  id: 'admin',
  email: 'admin',
  password: 'admin'
})

app.listen(3030)

Looks like I got too trigger happy with bug reporting. Thanks for the explanation.

This was a great bug report. Better to report too many things than to miss an actual bug.

@juodumas 💯 . You did some of the best bug reports we've seen so far. Makes it easier to track down and determine whether it is legit or not. ❤


Dark Mode by BajraYeJoon

Dark Mode Supported Single Web Page
Toggle around to get light or Dark mode.

Feel Free to Contribute 📝
You can add new features, styles and animation to improve your skills 😃
happy learning.💻

@ron2111  PLease review the PR and merge it😃😃


Remove deprecated images from operator

Those images have been deprecated when KCP was abandonned. The content used for local development and has been moved to the 'developer' directory.

Remove access-setup image.
Remove cluster-setup image.
OpenShift GitOps operator updated to 1.5.10.
Refactor dev_setup.sh: * Split the content into multiple files (one file per app). * Use a 'local' folder as the template to deploy pipeline-service

conceptually lgtm
not sure I can speak to the nuances of the bash changes, and what were deltas vs. copy / paste to new files
multiple commits capturing "copy as is to new file" vs. "here is where I tweaked things after the copy" could have been helpful

admittedly I may have had opportunities for that in the recent exporter reorg :-)

/test test-pipeline-service-deployment

/test test-pipeline-service-deployment

/test test-pipeline-service-deployment

/test test-pipeline-service-deployment

/retest

/retest

it seems at least conceivable @Roming22 the upgrade job failure in the upgrade-pipeline-service step is related to some of these change (given the bash changes ??)
Please commit your changes or stash them before you switch branches.
Aborting
command terminated with exit code 1

/test test-pipeline-service-upgrade

Failure logs:
STEP-RUN-PLNSVC-SETUP

+ touch /workspace/workdir/source/destroy-cluster.txt
+ echo 'Execute dev_setup.sh script to set up pipeline-service ...'
Execute dev_setup.sh script to set up pipeline-service ...
+ kubectl -n default exec pod/ci-runner -- sh -c '/workspace/sidecar/bin/plnsvc_setup.sh https://github.com/openshift-pipelines/pipeline-service 0f2d7aaa82f8a992816e5062f6f261d8beeb70a7'
+ export KUBECONFIG=/kubeconfig
+ KUBECONFIG=/kubeconfig
+ REPO_URL=https://github.com/openshift-pipelines/pipeline-service
+ REPO_REVISION=0f2d7aaa82f8a992816e5062f6f261d8beeb70a7
+ git config --global --add safe.directory /workspace/source
+ git fetch origin 0f2d7aaa82f8a992816e5062f6f261d8beeb70a7
From https://github.com/openshift-pipelines/pipeline-service
 * branch            0f2d7aaa82f8a992816e5062f6f261d8beeb70a7 -> FETCH_HEAD
+ git checkout 0f2d7aaa82f8a992816e5062f6f261d8beeb70a7
error: Your local changes to the following files would be overwritten by checkout:
	developer/config.yaml
Please commit your changes or stash them before you switch branches.
Aborting
command terminated with exit code 1

Issue should be fixed by forcing the checkout in ci/images/ci-runner/hack/sidecar/bin/plnsvc_setup.sh.


More README proposed changes

Improve introduction language
Add links to Node/Python clients.

Thank you for the links to the related projects :+1:


Error(s) in loading state_dict for ZoeDepth

RuntimeError: Error(s) in loading state_dict for ZoeDepth:
Unexpected key(s) in state_dict: "core.core.pretrained.model.blocks.0.attn.relative_position_index", "core.core.pretrained.model.blocks.1.attn.relative_position_index", "core.core.pretrained.model.blocks.2.attn.relative_position_index", "core.core.pretrained.model.blocks.3.attn.relative_position_index", "core.core.pretrained.model.blocks.4.attn.relative_position_index", "core.core.pretrained.model.blocks.5.attn.relative_position_index", "core.core.pretrained.model.blocks.6.attn.relative_position_index", "core.core.pretrained.model.blocks.7.attn.relative_position_index", "core.core.pretrained.model.blocks.8.attn.relative_position_index", "core.core.pretrained.model.blocks.9.attn.relative_position_index", "core.core.pretrained.model.blocks.10.attn.relative_position_index", "core.core.pretrained.model.blocks.11.attn.relative_position_index", "core.core.pretrained.model.blocks.12.attn.relative_position_index", "core.core.pretrained.model.blocks.13.attn.relative_position_index", "core.core.pretrained.model.blocks.14.attn.relative_position_index", "core.core.pretrained.model.blocks.15.attn.relative_position_index", "core.core.pretrained.model.blocks.16.attn.relative_position_index", "core.core.pretrained.model.blocks.17.attn.relative_position_index", "core.core.pretrained.model.blocks.18.attn.relative_position_index", "core.core.pretrained.model.blocks.19.attn.relative_position_index", "core.core.pretrained.model.blocks.20.attn.relative_position_index", "core.core.pretrained.model.blocks.21.attn.relative_position_index", "core.core.pretrained.model.blocks.22.attn.relative_position_index", "core.core.pretrained.model.blocks.23.attn.relative_position_index".
   i meet the problem that https://github.com/isl-org/ZoeDepth/releases/download/v1.0/ZoeD_M12_N.pt can't be used for zoedepth/zoedepth_v1.py

Hi @LiHaodong0217 , Could you tell us more about how you got this error, such as environments and commands to reproduce it? Thank you.

@LiHaodong0217 I had this issue as well, solved with run pip install timm==0.6.7 https://github.com/isl-org/ZoeDepth/issues/97#issuecomment-2041500556

Thank you @Linusnie for your help. As stated in readme, it is needed to install requirements_dev.txt to run notebooks and demos. The package timm is included in requirements_dev.txt.
@LiHaodong0217 Could you check if it works for you and report back to us?

Closing the issue as we have got no response. Feel free to reopen it if the problem persists. Thank you.


Dedicated resource for files to support PUT/LIST/REMOVE on stage  to  allow deploy jar/zip/py & etc.

Is your feature request related to a problem? Please describe.
Complex functions & stored procedures allow to list packages they relay on and those files should be available in Snowflake before function or sp can be created.
Describe the solution you'd like
Dedicated resource to move files.
Describe alternatives you've considered
as for now I can achieve desired result with  snowsql provider.

Hello @DmitryMaletin . I am sorry, but I don't fully understand the request. Can you please provide an example of what SQL commands you are executing?

Hi @sfc-gh-swinkler ,
The purpose is deploy python/scala based functions/procedures.
An example blow uses snowsql provider.
resource "terraform_data" "replacement" {
  input = "${sha1(filebase64("./snowpark/python_bundle.zip"))}"
}

resource "snowsql_exec" "python" {
  name = "./snowpark/python_bundle.zip"

  create {
    statements =  "PUT file:///snowpark/python_bundle.zip @MYDEPLOYMENT auto_compress = false overwrite = true;"
  }

  read {
    statements = "LIST @DMYDEPLOYMENT/python_bundle.zip"
  }

  update {
    statements =  "PUT file:///snowpark/python_bundle.zip @MYDEPLOYMENT auto_compress = false overwrite = true;"
  }

  delete {
    statements = "REMOVE @MYDEPLOYMENT/python_bundle.zip"
  }

  lifecycle {
        replace_triggered_by = [terraform_data.replacement]
  }
}

resource "snowflake_function" "python_test" {
  name     = "python_test"
  database = "MY_DB"
  schema   = "PUBLIC"
  arguments {
    name = "arg"
    type = "int"
  }
  comment             = "Example for Python language"
  return_type         = "NUMBER(38,0)"
  null_input_behavior = "CALLED ON NULL INPUT"
  return_behavior     = "VOLATILE"
  language            = "python"
  runtime_version     = "3.8"
  handler             = "python_bundle.hello"
  imports             =<EMAIL_ADDRESS>  statement           = ""
}

We also need this. We are using Dmitry's workaround for now, but it's not ideal. Often, the snowflake_function resource will fail since the file isn't visible in the stage yet. This is despite adding a depends_on to wait until the PUT is complete.

@imre-kerr-sb1
Sadly as for now it's not possible to set timeout directly on resource and retry is also not implemented although it's possible
https://developer.hashicorp.com/terraform/plugin/sdkv2/resources/retries-and-customizable-timeouts
One of the option (far to ideal) is to use https://registry.terraform.io/providers/hashicorp/time/latest/docs/resources/sleep