Nested data type incorrect empty insert issue

Steps to reproduce:

Create table with the following structure:

xxxx :) CREATE TABLE test.cmdts_test (  cmdts Nested( cmdty_id Nullable(UInt16), cmdty_name Nullable(String), cmdty_group_id Nullable(UInt16), cmdty_group_name Nullable(String)) ) ENGINE = Log

CREATE TABLE test.cmdts_test
(
    cmdts Nested(
    cmdty_id Nullable(UInt16),
    cmdty_name Nullable(String),
    cmdty_group_id Nullable(UInt16),
    cmdty_group_name Nullable(String))
)
ENGINE = Log

Ok.

0 rows in set. Elapsed: 0.003 sec.

Insert data by leaving one of the fields empty(not NULL, but empty square brackets)

xxxx :) INSERT INTO test.cmdts_test VALUES ([5], ['myname'], [], ['fafa'])

INSERT INTO test.cmdts_test VALUES

Ok.

1 rows in set. Elapsed: 0.003 sec.

Now select from that table

xxxx :) SELECT * FROM test.cmdts_test;

SELECT *
FROM test.cmdts_test

Exception on client:
Code: 89. DB::Exception: Unknown compression method: 108: while receiving packet from localhost:9000, <IP_ADDRESS>

Connecting to localhost:9000.
Connected to ClickHouse server version 1.1.54383.

With NULL value passed, works as expected, however. I guess it would be nice to assume if no value is passed for that nested field, then the values should be NULL. Or at least the insert shouldn't pass.

I think this issue is related with https://github.com/yandex/ClickHouse/issues/2231

Yes.
milovidov-Latitude-E7470 :) INSERT INTO test.cmdts_test VALUES ([5], ['myname'], [], ['fafa'])

INSERT INTO test.cmdts_test VALUES

Received exception from server (version 18.11.0):
Code: 190. DB::Exception: Received from localhost:9000, <IP_ADDRESS>. DB::Exception: Elements 'cmdts.cmdty_id' and 'cmdts.cmdty_group_id' of Nested data structure 'cmdts' (Array columns) have different array sizes..

Fixed.


Canvas layer reprojection still has (small) anomalies

Describe the bug
The display of a uniform canvas layer reprojected should still be uniform.
If you use 6.3.1, then there are still the terrible drawing issues with reprojection. The current version (6.3.2-dev.1589259231810) shows some black dots. If you pan/zoom around, then it throws errors which are, I think, due to issues in proj4js (which should be fixed by https://github.com/proj4js/proj4js/pull/342). I couldn't find an easy way of using that version in CSB.
Also, it appears that what you see is browser and OS specific.
To Reproduce
Steps to reproduce the behavior:

Go to 'https://codesandbox.io/s/dev-ol-canvas-reproj-ylso5'
See that the display area is not (quite) uniform. There are little black dots in places.

Expected behavior
The display should be uniform.

Possibly related to #10987 although that affected the color values (a difference of 1 in the red value is over 6000 meters in a DEM), but your problem is only affecting the opacity https://codesandbox.io/s/dev-ol-canvas-reproj-dqnwh (specifying imageSmoothing: false in the sandbox will fix it by forcing your brokenDiagonalRendering method)

Indeed it does -- thank you. I'll see if I can find something to fix in the non-broken diagonal rendering -- it could, of course, be a browser rendering bug as well. In which case, it probably makes sense to improve the detection of broken rendering to include this case,


Implement listen()

Implement listen() function.

Does this look better now? Can we merge this first?


Add cupy.scipy.linalg.bandwidth functionality

What it says on the tin.
One thing worth noting is that since the ElementwiseKernel tests for every cell in a matrix, this method would take longer for unbanded matrices, but I doubt people would be using bandwidth for unbanded matrices anyways so I think it's fine. That being said, if anyone has ideas on making the code more efficient for high-bandwidth matrices that would be cool.
Linked Issue : https://github.com/cupy/cupy/issues/6324

I've added more tests, which allowed me to find that F-contiguous arrays were transposed twice by mistake. I've done a bit of a rewrite to make things a bit more clear (by introducing a new variable A which is C contiguous).
Also, if it's fine, I've updated mypy to 0.981 due to issues I had with pre-commit hooks.

/test mini

/test mini

Sorry for the delay!, seems that the tests failures are real :(
https://ci.preferred.jp/cupy.linux.cuda121/129650/

Actually, do you recommend I add the mypy update into a separate PR as well? If this PR keeps getting bogged down, then at least mypy would get updated faster.

/test mini

Hmm, just realised that this only is an issue with Windows systems, while Linux systems (like the ones I've been using) have been working fine. I'll try and get a Windows system up and running this weekend to figure out a fix to this.

Alright, fixed!
Turns out the underlying problem is that cupy.mgrid has dtype int32 on Windows, but int64 on Linux. Using a type placeholder should fix things.

/test mini

/test mini

Test failures are unrelated


@export_range's "radians_as_degrees" changes behaviour based on the default value in code

Tested versions
Reproduced in v4.3.stable.official [77dcf97d8]
System information
Godot v4.3.stable - Garuda Linux #1 ZEN SMP PREEMPT_DYNAMIC Thu, 12 Sep 2024 17:17:51 +0000 - X11 - GLES3 (Compatibility) - Mesa Intel(R) HD Graphics 630 (KBL GT2) - Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz (8 Threads)
Issue description
When using "radians" or "radians_as_degrees" hints in @export_range, the behaviour changes based on the default value within the code:

If the default values in the code are numbers, values inputted in the inspector field as degrees will not be converted to radians and will remain as degrees. (90deg in the inspector will result in the value of the variable being equal to 90)
If the default values in the code are numbers multiplied by PI, values inputted in the inspector field as degrees will be converted to radians. (90deg in the inspector will result in the value of the variable being equal to 1.5708)

I believe this is unintuitive, as even a 0 as a default value will end up in degrees in the inspector not being converted to radians. In my specific case I was making a turret base class, with export variables used to adjust possible vertical/horizontal spread. Having set default values to 0, it was unclear why a +-5deg spread value set in the inspector resulted in shots fired in whatever direction possible ( rotation.y = randi_range(-h_spread, h_spread) does very different things if h_spread is in degrees vs radians.
Steps to reproduce
Create an empty project with an empty Node3D and add the following script to it. Then, in the inspector, set all export fields created by this script to any value in degrees. Run the scene and observe the output printed.
extends Node3D

@export_range(0,360, 1, "radians") var R_deg = 1
@export_range(0,360, 1, "radians") var R_rad = 1*PI
@export_range(0,360, 1, "radians_as_degrees") var RaD_deg = 1
@export_range(0,360, 1, "radians_as_degrees") var RaD_rad = 1*PI

func _ready() -> void:
	# comments after each line below represent the values printed if all inspector export values are set to 90
	print("Marked 'radians', default code value is 1: " + str(R_deg)) #90
	print("Marked 'radians', default code value is 1*PI: " + str(R_rad)) #1.5708
	print("Marked 'radians_as_degrees', default code value is 1: " + str(RaD_deg)) #90
	print("Marked 'radians_as_degrees', default code value is 1*PI: " + str(RaD_rad)) #1.5708

Minimal reproduction project (MRP)
N/A

It's because these export variables don't have type hints, so the type is inferred from its default value.
The type is int when initialized to 1, and it's float when initialized to 1*PI. Integers can't represent radian values properly, so radians_as_degrees only changes the suffix.
I think there could be a warning when radians_as_degrees is used on an int property.


cannot load saved file!

it seems when a data file is saved, it cannot be loaded back!
for example 5_bobcats_F4_F9_F13_M10_M15_BBMMinput.csv first 5 entries:
ID,CummulTime,Northing,Easting
F4,0,4234527,258558
F4,300,4231777,259222
F4,1440,4229992,255965
F4,1740,4231788,255459
F4,2100,4233396,255891

loads fine.  after it is saved, the file looks like:
"","id","time","ydata","xdata"
"1","F4",0,4234527,258558
"2","F4",300,4231777,259222
"3","F4",1440,4229992,255965
"4","F4",1740,4231788,255459
"5","F4",2100,4233396,255891

but the app will throw an error message "Error : Missing required columns; must have time and lat-long, x-y or UTM"

looks like currently the code is renaming the first column to xdata and we end up with 2 xdata columns!  debug statement shows the gpsdata at the end of preprocessDataframe() to contain:
     xdata             id                 time            ydata
 Min.   :   1.0   Length:1580        Min.   :     0   Min.   :4224793
 1st Qu.: 395.8   Class :character   1st Qu.: 19815   1st Qu.:4231487
 Median : 790.5   Mode  :character   Median : 39720   Median :4234742
 Mean   : 790.5                      Mean   : 55348   Mean   :4240917
 3rd Qu.:1185.2                      3rd Qu.: 72750   3rd Qu.:4252112
 Max.   :1580.0                      Max.   :209460   Max.   :4258177
     xdata
 Min.   :244694
 1st Qu.:247323
 Median :252713
 Mean   :252169
 3rd Qu.:257614
 Max.   :259963

This is one for me to tackle. I'll address this once I'm done with the NSF annual report, probably tomorrow (7/10)

more debugging shows that loadDataframeFromFile() read.csv call will name a data "X" when the column name is blank and then that X column gets renamed to xdata and hence resulting in 2 xdata columns; didn't realize R will allow this.  so perhaps the fix is not to write the column with no header...

oops, that caused a new error "Warning: Error in read.table: duplicate 'row.names' are not allowed" when loading dataset 5_bobcats_F4_F9_F13_M10_M15_BBMMinput.csv, debugging...

oops, changing the read.csv() call broke reading other csv files NOT written by write.csv().  instead, i've changed the write.csv() call and that seems to work now


[Experiment] Epoch-based map

Refs: #8

Swaps atomic snapshots from vanilla CLHT with epoch-based snapshots described in #8
Value pointer uniqueness required by atomic snapshots is no longer needed. So, the Map can be easily adapted to store arbitrary types, not only interface{}. Also, I didn't bother with getting rid of nilVal tag struct, but it should be straightforward to do so
Bucket linked lists are removed in favor of increasing the bucket size to 2 CL (128B). This means up to 7 entries per bucket vs. 9 entries in the baseline implementation
Keys and values are now placed in a single kvs array, so that each key and the corresponding value are located in the same CL

The performance is slightly better (~10%) than in the baseline, except for the Range operation which got ~50% slower (#25 may fix that).

Codecov Report

Merging #29 (ce64ee8) into main (1ad5cea) will decrease coverage by 5.11%.
The diff coverage is 90.51%.

@@            Coverage Diff             @@
##             main      #29      +/-   ##
==========================================
- Coverage   97.64%   92.53%   -5.12%
==========================================
  Files           5        5
  Lines         509      482      -27
==========================================
- Hits          497      446      -51
- Misses         10       28      +18
- Partials        2        8       +6

Impacted Files
Coverage Δ

map.go
89.91% <90.51%> (-7.35%)
:arrow_down:

Continue to review full report at Codecov.

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


[BUG] <After modifying the yml configuration file, re-refresh or reload or clear the cache, the data before modification is still displayed>

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
No response
Expected Behavior
No response
Steps To Reproduce
对在线修改yml体验感较差，修改后缓存很久才能被清掉。
暂时最直观有效的处理办法就是重启服务，然后就生效。
Environment
HertzBeat version(v1.3.2):

Debug logs
No response
Anything else?
No response

🫡 we will fix it.


Ignore TODO tags in commented-out-code

Summary
Extend the regex to ignore TODO tags (with or without parentheses) that are immediately followed by a colon. For example,
# TODO(tjkuson): Rewrite in Rust

is no longer flagged as commented-out code.
Closes #7031.
I also updated the documentation to inform users that the rule is prone to false positives like this!
EDIT: Accidentally linked to the wrong issue when first opening this PR, now corrected.
Test Plan
cargo test

Moved the tag checking logic to before the regex matching, as I believe it is less expensive. Also, now checks for lowercase tags (for example, # todo(tom): Rewrite in Rust).


Reload does not work with Linux + Android

Hi Benoit, first of all, many thanks for the nice demo!
I was playing with it using Android + Linux. The app builds and installs correctly (using react-native install-android) and even runs correctly on the first time. (The weird thing though is, on the first run it does not try to pull the JS using react packager - I see no trace of any requests there.)
However, when as I try to reload JS from the phone, the JS gets transferred correctly (confirmed by the packager logs) but I'm ending up with a blank screen, and the following errors in the log:
D/ReactNativeJS(13909): 'Running application "nativeApp" with appParams: {"initialProps":{},"rootTag":1}. __DEV__ === false, development-level warning are OFF, performance optimizations are ON'
D/ReactNativeJS(13909): 'Running application "nativeApp" with appParams: {"initialProps":{},"rootTag":1}. __DEV__ === false, development-level warning are OFF, performance optimizations are ON'
E/ReactNativeJS(13909): 'Error: \n stack: measureMethods@http://localhost:8081/src/index.android.bundle?platform=android&dev=true:53888:11\nhttp://localhost:8081/src/index.android.bundle?platform=android&dev=true:53166:25\nrequireImpl@http://localhost:8081/src/index.android.bundle?platform=android&dev=true:64:17\nrequire@http://localhost:8081/src/index.android.bundle?platform=android&dev=true:26:19\nhttp://localhost:8081/src/index.android.bundle?platform=android&dev=true:52928:34\nrequireImpl@http://localhost:8081/src/index.android.bundle?platform=android&dev=true:64:17\nrequire@http://localhost:8081/src/index.android.bundle?platform=android&dev=true:26:19\nhttp://localhost:8081/src/index.android.bundle?platform=android&dev=true:52806:34\nrequireImpl@http://localhost:8081/src/index.android.bundle?platform=android&dev=true:64:17\nrequire@http://localhost:8081/src/index.android.bundle?platform=android&dev=true:26:19\nhttp://localhost:8081/src/index.android.bundle?platform=android&dev=true:52762:21\nrequireImpl@http://localhost:8081/src/index.android.bundle?platform=android&dev=true:64:17\nrequire@http://localhost:8081/src/index.android.bundle?platform=android&dev=true:26:19\nhttp://localhost:8081/src/index.android.bundle?platform=android&dev=true:926:52\nrequireImpl@http://localhost:8081/src/index.android.bundle?platform=android&dev=true:64:17\nrequire@http://localhost:8081/src/index.android.bundle?platform=android&dev=true:26:19\nhttp://localhost:8081/src/index.android.bundle?platform=android&dev=true:896:218\nrequireImpl@http://localhost:8081/src/index.android.bundle?platform=android&dev=true:64:17\nrequireImpl@http://localhost:8081/src/index.android.bundle?platform=android&dev=true:34:32\nrequire@http://localhost:8081/src/index.android.bundle?platform=android&dev=true:26:19\nglobal code@http://localhost:8081/src/index.android.bundle?platform=android&dev=true:73756:9\n line: 53888\n message: Can\'t find variable: process', { [ReferenceError: Can't find variable: process]
E/ReactNativeJS(13909):   line: 53888,
E/ReactNativeJS(13909):   column: 11,
E/ReactNativeJS(13909):   sourceURL: 'http://localhost:8081/src/index.android.bundle?platform=android&dev=true' }
E/ReactNativeJS(13909): 'Error: \n stack: http://localhost:8081/src/index.android.bundle?platform=android&dev=true:54631:11\nrequireImpl@http://localhost:8081/src/index.android.bundle?platform=android&dev=true:64:17\nrequire@http://localhost:8081/src/index.android.bundle?platform=android&dev=true:26:19\nhttp://localhost:8081/src/index.android.bundle?platform=android&dev=true:2808:20\nrequireImpl@http://localhost:8081/src/index.android.bundle?platform=android&dev=true:64:17\nrequire@http://localhost:8081/src/index.android.bundle?platform=android&dev=true:26:19\nhttp://localhost:8081/src/index.android.bundle?platform=android&dev=true:2336:30\nrequireImpl@http://localhost:8081/src/index.android.bundle?platform=android&dev=true:64:17\nrequire@http://localhost:8081/src/index.android.bundle?platform=android&dev=true:26:19\nhttp://localhost:8081/src/index.android.bundle?platform=android&dev=true:2310:25\nrequireImpl@http://localhost:8081/src/index.android.bundle?platform=android&dev=true:64:17\nrequireImpl@http://localhost:8081/src/index.android.bundle?platform=android&dev=true:34:32\nrequire@http://localhost:8081/src/index.android.bundle?platform=android&dev=true:26:19\nglobal code\n line: 54631\n message: Can\'t find variable: process', { [ReferenceError: Can't find variable: process]
E/ReactNativeJS(13909):   line: 54631,
E/ReactNativeJS(13909):   column: 11,
E/ReactNativeJS(13909):   sourceURL: 'http://localhost:8081/src/index.android.bundle?platform=android&dev=true' }
E/ReactNative(13909): Got JS Exception: TypeError: undefined is not an object (evaluating 'require('BatchedBridge').flushedQueue')

(Note that the first error is actually Can\'t find variable: process', { [ReferenceError: Can't find variable: process] - you need to scroll to the right in the error log).
Any idea why is this failing? I actually tried Windows too, but that fails for different reasons, so let's leave this for now.

Hi @docent, thanks.
About your issue, I am not really familiar with Android and Linux as I work on a Mac.
Have you looked at the react-native repo issues. Android support is still quite new and they may still have bugs unresolved. Have you tried to install the demo project with react-native init to see if it works as expected as well.

Yes. The default demo project is working as expected, and so are other demos that I tested. Everything works including reload etc.
Do you know why your app does not try to pull the JS immediately after start? That is somewhat strange, all the other demos would do that.

I have same issue with android in windows 7. If I reload, the app exists showing uncertain error.


Questions about channel re-scaling approach

Hello,
Thanks for your good work.
I have two questions about the proposed channel re-scaling approach.
Q1)
It describes that this work merges channel layer-scale [76] and an effective squeeze-excitation network [48] as channel re-scaling approach.
But I cannot see how it merge. I can only see effective SE module below.
Could you please help give some advices about the proposed channel re-scaling?
https://github.com/naver-ai/rdnet/blob/6739093bf0af0ec53af163ff15e91d46d66dbe9e/rdnet/rdnet.py#L88
Q2)
Could you give some advices when to use ESE module?
Did you try to mix two block types like this scenario?
Block ==> ... ==> BlockESE ==> BlockESE ==> Block ==> ...
Thanks.

@DennisInTw
The layer scale is defined in the following lines.
https://github.com/naver-ai/rdnet/blob/6739093bf0af0ec53af163ff15e91d46d66dbe9e/rdnet/rdnet.py#L134-L135
In our experience, the ESE module is more efficient in stage 3 or stage 4. Performance improves when used in stage 1 or 2, but computation increases significantly.
RDNet already utilizes the ese module only in stages 3 and 4.
https://github.com/naver-ai/rdnet/blob/6739093bf0af0ec53af163ff15e91d46d66dbe9e/rdnet/rdnet.py#L385
Thank you.

Thanks for your help.


By using Entitas, is there a proper way to serialize a frame?

Hi,
As we all know that ECS makes data and behavior separated. So I wanna get a proper way to serialize all entities and their components (excluding rendering components).
Given that a component is not supposed to present methods, how can I serialize them? It's useful to save a state and recover it at a later time.
Cheers

You can write a flag component like "RecordableComponent" and use JSON.net to serialize instances. However, I'm almost sure that this is not what you want by practice. For cool things like kill cam and rewindable gameplay, it's important to filter out unnecessary data, otherwise, the player's memory/harddisk would be blown up quickly because you're storing data at the 60Hz frequency.
So as you can see, it's not an Entitas related problem. IIRC, Zaks made a talk about how to implement a playback system, you can google that video, it's insightful.

Thanks for your reply. It's really not an easy thing to do, I think so too.
I found the following demo but, its solution so damn sucks if the systems grow more and more complex.
https://github.com/coding2233/Entitas-Replay-Demo
I won't store the state data at every frame. Instead, I can save a state every 30 frames, and save the user-action every frame.
What you said 'RecordableComponent' may be not enough, cause when I have collected all Recordable Entities, I still don't know what to record. I need manually organize all components I want to record in a recording-system. Maybe the best way is writing a code generator to automatically generate a component includes all components that have the attribute "Recordable".
Considering server/client communication, it's necessary to do serializing/deserializing. The code may look like
    public RecordSystem(Contexts contexts)
    {
        _recordableEntities = contexts.game.GetGroup(GameMatcher.Recordable);
    }

    public void Execute()
    {
        Dictionary<int, byte[]> serialized = new Dictionary<int, byte[]>();
        foreach (GameEntity e in _recordableEntities.GetEntities())
        {
            serialized[e.EntityId] = e.GetComponents<IRecordableComponent>().SerializeToBytes()
        }

       //do next...
    }

JSON.Net needs more device resources, and bytes serialization such as protobuf breaks ECS rules.

You can write a flag component like "RecordableComponent" and use JSON.net to serialize instances. However, I'm almost sure that this is not what you want by practice. For cool things like kill cam and rewindable gameplay, it's important to filter out unnecessary data, otherwise, the player's memory/harddisk would be blown up quickly because you're storing data at the 60Hz frequency.
So as you can see, it's not an Entitas related problem. IIRC, Zaks made a talk about how to implement a playback system, you can google that video, it's insightful.

Thanks for your reply. It's really not an easy thing to do, I think so too.
I found the following demo but, its solution so damn sucks if the systems grow more and more complex.
https://github.com/coding2233/Entitas-Replay-Demo
I won't store the state data at every frame. Instead, I can save a state every 30 frames, and save the user-action every frame.
What you said 'RecordableComponent' may be not enough, cause when I have collected all Recordable Entities, I still don't know what to record. I need manually organize all components I want to record in a recording-system. Maybe the best way is writing a code generator to automatically generate a component includes all components that have the attribute "Recordable".
Considering server/client communication, it's necessary to do serializing/deserializing. The code may look like
    public RecordSystem(Contexts contexts)
    {
        _recordableEntities = contexts.game.GetGroup(GameMatcher.Recordable);
    }

    public void Execute()
    {
        Dictionary<int, byte[]> serialized = new Dictionary<int, byte[]>();
        foreach (GameEntity e in _recordableEntities.GetEntities())
        {
            serialized[e.EntityId] = e.GetComponents<IRecordableComponent>().SerializeToBytes()
        }

       //do next...
    }

JSON.Net needs more device resources, and bytes serialization such as protobuf breaks ECS rules.

I recorded and serialized the content data of the component. Next I wanna find a way to know what component the data belongs to.

public interface IRecordableComponent {}

[Game]
public class PositionComponent : IComponent, IRecordableComponent
{
    public float x;
    public float y;
}

[Game]
public class RecordableComponent : IComponent {}

public class RecordSystem : IExecuteSystem
{
    readonly GameContext _context;

    public RecordSystem(Contexts contexts)
    {
        _context = contexts.game;
    }

    public void Execute()
    {
        Dictionary<int, IComponent[]> entityData = new Dictionary<int, IComponent[]>();
        var entities = _context.GetGroup(GameMatcher.Recordable);
        foreach (var e in entities)
        {
            var comps = e.GetComponents().Where((c) => c is IRecordableComponent).ToArray();
            entityData[e.creationIndex] = comps;
        }

        if (entityData.Count > 0)
        {
            Debug.Log(JsonConvert.SerializeObject(entityData));
        }
    }
}

If you are making a multiplayer game, don't use JSON, it's still too big
unless your game is simple enough. We usually use code generator to create
contracts for State, Input and other stuff then serialize them as a
dict/hashtable byte[]. Google GafferOnGames, there are lots of network
essential posts on it and you will see how picky it is to transfer data in
a real game. In short, every bit you saved is $$$ ;)
Jovi Du<EMAIL_ADDRESS>于 2018年12月30日周日 下午11:18写道：

I recorded and serialized the content data of the component. Next I wanna
find a way to know what component the data belongs to.
public interface IRecordableComponent {}
[Game]public class PositionComponent : IComponent, IRecordableComponent
{
public float x;
public float y;
}
[Game]public class RecordableComponent : IComponent {}
public class RecordSystem : IExecuteSystem
{
readonly GameContext _context;
public RecordSystem(Contexts contexts)
{
    _context = contexts.game;
}

public void Execute()
{
    Dictionary<int, IComponent[]> entityData = new Dictionary<int, IComponent[]>();
    var entities = _context.GetGroup(GameMatcher.Recordable);
    foreach (var e in entities)
    {
        var comps = e.GetComponents().Where((c) => c is IRecordableComponent).ToArray();
        entityData[e.creationIndex] = comps;
    }

    if (entityData.Count > 0)
    {
        Debug.Log(JsonConvert.SerializeObject(entityData));
    }
}

}
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/sschmid/Entitas-CSharp/issues/849#issuecomment-450567316,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADO9GQMjGWkWWrEAeHaeLuh2gY4itEFjks5u-NlXgaJpZM4ZeQuj
.


PieController requires LinearScale in webpack production mode

Expected behavior
Either the application bundled in production mode doesn't require LinearScale, or LinearScale is required in development mode, too.
According to the documentation, I think that Chart.js is intended to not require LinearScale.
Current behavior
During runtime of a Chart.js-Application, which doesn't include LinearScale, bundled in webpack's production mode, I get an Uncaught ReferenceError: LinearScaleBase not defined:

Reproducible sample
https://github.com/chkpnt/chartjs-issue11157/blob/main/App.js#L3-L9
Optional extra steps/info to reproduce
Starting the application with npm run start(which uses webpack's development mode) works, while npm run start-prod isn't working. In order to get the application to work in production mode, LinearScale needs to be included in the bundle.
Possible solution
No response
Context
No response
chart.js version
v4.2.1
Browser name and version
Firefox 110.0
Link to your project
https://github.com/chkpnt/chartjs-issue11157

Another example of this issue;
`import axios from "axios";
import {
Chart,
ChartConfiguration,
TimeScale,
LinearScale,
PointElement,
LineElement,
Title,
Tooltip,
Legend,
Point,
ChartDataset,
} from "chart.js";
import "chartjs-adapter-moment";
Chart.register(
TimeScale,
LinearScale,
PointElement,
LineElement,
Title,
Tooltip,
Legend
);
export class ChartView {
URL: string = "https://localhost:5001";
set: Set = new Set();
chart: Chart;
chartColors: any = {
red: "rgb(255, 99, 132)",
orange: "rgb(255, 159, 64)",
yellow: "rgb(255, 205, 86)",
green: "rgb(75, 192, 192)",
blue: "rgb(54, 162, 235)",
purple: "rgb(153, 102, 255)",
grey: "rgb(201, 203, 207)",
};
type: string = "FileSizeOutbound";
config: ChartConfiguration<"line"> = {
type: "line",
data: {
datasets: [
{
label: this.type,
borderColor: this.chartColors.red,
data: [],
},
],
},
options: {
responsive: true,
scales: {
x: {
type: "time",
time: {
unit: "month",
},
},
},
},
};
constructor() {}
AddData(dataset: ChartDataset, point: Point) {
if (!this.set.has(point.x)) {
dataset.data.push({
x: point.x,
y: point.y,
});
this.set.add(point.x);
}
}
OnRefresh() {
setInterval(() => {
var dataset = this.chart.data.datasets[0];
var period = "1d";
axios
.get<Point[]>(
${this.URL}/eAdaptor/RateRequestStore/${this.type}/${period}
)
.then((response) => {
for (var i = 0; i < response.data.length; i++) {
var point = response.data[i];
this.AddData(dataset, point);
}
this.chart.update();
})
.catch(function (error) {
console.log(error);
});
}, 1000);
}
Load(id: string) {
var ctx = (document.getElementById(id) as HTMLFormElement).getContext("2d");
this.chart = new Chart(ctx, this.config);
this.OnRefresh();
}
}
export default ChartView;
`


Excluded Resources are being included in CRDs

What happened?
CRDs are not honoring the list of ignored-resources in generator-config.yaml
We can see this in the SecurityGroup resource currently on master:
SecurityGroup is a managed resource in ec2/v1beta1.
SecurityGroup is only a type in ec2/v1alpha1.
The ec2/v1alpha1/generator-config.yaml specifies SecurityGroup as an ignored resource.
However both versions are present in the CRD.
When the storageVersion is a managed-resource, this is a minor bug. When the storageVersion is the type, as in https://github.com/crossplane/provider-aws/pull/859, it can cause the resource to lose the status fields which crossplane requires.
How can we reproduce it?

Checkout https://github.com/crossplane/provider-aws/pull/859
Run make generate and note that the CRD contains two different specs for v1alpha1 and v1beta1.
The v1beta1 resource is missing status fields, and is the storageVersion.
Apply the CRDs to the cluster and create a v1alpha1 route.
kubectl get route.ec2.aws.crossplane.io will be missing status fields:

% kubectl get route.ec2.aws.crossplane.io
NAME      AGE
example   96s

The status is missing because the CRD for Route contains a storage version which does not supply the crossplane-required status fields.

What environment did it happen in?
Kind cluster
Kubernetes version:
Client Version: version.Info{Major:"1", Minor:"22", GitVersion:"v1.22.1", GitCommit:"632ed300f2c34f6d6d15ca4cef3d3c7073412212", GitTreeState:"clean", BuildDate:"2021-08-19T15:38:26Z", GoVersion:"go1.16.6", Compiler:"gc", Platform:"darwin/arm64"}
Server Version: version.Info{Major:"1", Minor:"22", GitVersion:"v1.22.1", GitCommit:"632ed300f2c34f6d6d15ca4cef3d3c7073412212", GitTreeState:"clean", BuildDate:"2021-08-25T19:57:36Z", GoVersion:"go1.16.7", Compiler:"gc", Platform:"linux/arm64"}

We could mark all generated types with skipversion to keep them out of CRDs. But this seems unelegant.

@AaronME I just tested this one and it seems like the only viable option here. We need to add // +kubebuilder:skipversion to here.
For other options, having the information of a collision is a hard one for ACK since the types are generated by a different mechanism than CRD file and even if we take it from the user, I'm not sure if it's worth opening another config interface in generator-config.yaml as you said.
I can work on it only next week. @AaronME @haarchri If anyone is interested in opening a PR to ACK, it'd be great.

@muvaf Opened https://github.com/aws-controllers-k8s/code-generator/pull/236


New SignStream collection ready for inclusion in the DAI

There are now several new "Carey" files ready to be processed.
Important to know about these files:  As with some other collections, each sign is its own utterance.  The utterance itself is the "video clip" and the sign inside the utterance will be what we want for the "sign clip".
Happy to explain further.
These will constitute a new ASLLRP isolated sign collection.
We also have the new RIT sentence file.  (Our other RIT files are isolated signs). But the RIT file is a much lower priority!

This is not as high a priority as the other things we are working on now, since this does not HAVE to be done by the end of July.
thanks !!!

Carey files have been uploaded :-).  Thank you.
What remains is the new RIT sentence file (a new file type, as the other RIT files are isolated signs).


Invalid memory access in nfa_save_listids/nfa_restore_listids

I'm trying to debug a difficult to reproduce crash, which happens with my configuration but not when reducing anything to find a minimal example.
While investigating I noticed that in regexp_nfa.c sometimes nfa_save_listids/nfa_restore_listids operates on the array listids which is too small.  I added a variable which stores the last allocated size and have seen prog->nstate become fairly large (around 900) while the buffer is only big enough for 5 or so ints.
I don't know if this behavior is exceptional and caused by some other bug but the logic here seems faulty somehow since the allocated size nstate has seemingly little to do with prog->nstate and there are no bound checks or reallocations.
I'm hoping this admittedly vague description is enough to notice an error since I have not been able to come up with a reproducible example.

Do you have a stack trace?
Can you reproduce it with valgrind?
Or can you reproduce it with the address sanitizer (i.e. rebuild vim after uncommenting the following line in vim/src/Makefile:
SANITIZER_CFLAGS = -g -O0 -fsanitize=address -fno-omit-frame-pointer

It it crashes, it will then provide useful information on stderr.

Thanks.  Address sanitizer: http://vpaste.net/IRY0P
Valgrind output: https://gist.github.com/andymass/0d4f7157ddec6d12666f419acd8b9597
Very noisy, but note "Invalid write of size 4" (could be red herring).
Backtrace: http://vpaste.net/szNC7

Thanks.  Address sanitizer: http://vpaste.net/IRY0P
Valgrind output: https://gist.github.com/andymass/0d4f7157ddec6d12666f419acd8b9597
Very noisy, but note "Invalid write of size 4" (could be red herring).
Backtrace: http://vpaste.net/szNC7

I can guess that the problem is that the number of states changes
between calls to recursive_regmatch().  I'll make a patch for that.
It would still be nice to have a pattern and text to reproduce this
with and have a test for it.  It's probably something with & in the
pattern.
--
hundred-and-one symptoms of being an internet addict:
212. Your Internet group window has more icons than your Accessories window.
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

I've come up with a reproducible example (sanitizer output included): https://gist.github.com/andymass/4fc164de7e4cffe0272b56a1379da904
Unfortunately, it still crashes after Bram's update.  Maybe for a different reason.
@dpelle, sorry, the line numbers differed because of my own tracing code.  I am using master.

I've come up with a reproducible example (sanitizer output included): https://gist.github.com/andymass/4fc164de7e4cffe0272b56a1379da904
Unfortunately, it still crashes after Bram's update.  Maybe for a different reason.
@dpelle, sorry, the line numbers differed because of my own tracing code.  I am using master.

I cannot reproduce the problem.
It's unlikely it is actually trying to use freed memory, more likely
pointer computations cause that memory location to be accessed.
Although the change for the listids is probably valid, it might be
completely unrelated to this problem, since it's happening when using
reginput.
--
If evolution theories are correct, humans will soon grow a third
hand for operating the mouse.
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

I think I see what is happening:

searchpos() is called with a complex pattern.
in nfa_regexec_both(), the global variable nstate is set to prog->nstate (large) then nfa_regtry()->nfa_regmatch() are called.
in nfa_regmatch(), vim does a ui_breakcheck() which eventually calls clip_get_selection().
the TextYankPost autocmd is fired for the "* register which calls match_file_pat()->vim_regexec()->...->nfa_regexec_both().  This sets nstate to prog->nstate for the autocmd filename pattern (small).
back in the context of (2), nfa_regmatch() calls recursive_regmatch() which calls nfa_save_listids(), which writes to invalid memory because *listids is only allocated according to the new global nstate set in (4) which is too small.

In short it is an autocmd's pattern interfering with the global variable nstate in the middle of another regex matching.
To reproduce it is necessary to either be running vim locally with X or with ssh -Y (trusted X forwarding) but ./configure && make suffices.

Andy Massimino wrote:

I think I see what is happening:

searchpos() is called with a complex pattern.
in nfa_regexec_both(), the global variable nstate is set to prog->nstate (large) then nfa_regtry()->nfa_regmatch() are called.
in nfa_regmatch(), vim does a ui_breakcheck() which eventually calls clip_get_selection().
the TextYankPost autocmd is fired for the "* register which calls match_file_pat()->vim_regexec()->...->nfa_regexec_both().  This sets nstate to prog->nstate for the autocmd filename pattern (small).
back in the context of (2), nfa_regmatch() calls recursive_regmatch() which calls nfa_save_listids(), which writes to invalid memory because *listids is only allocated according to the new global nstate set in (4) which is too small.

In short it is an autocmd's pattern interfering with the global variable nstate in the middle of another regex matching.
To reproduce it is necessary to either be running vim locally with X or with ssh -Y (trusted X forwarding) but ./configure && make suffices.

We can either make sure regexec is never used recursively, or not use
TextYankPost for the clipboard.
That last one would be the simplest.  I wonder what people do with
TextYankPost anyway?  Is it relevant for the clipboard registers?
With some effort it would be possible to make regexec work recursively.
There is some support already (but I wonder when it works, since several
global values are not saved and restored).
--
You are Dead.  Do you wish to restart, load, or quit?
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

Is it possible that other dangerous callbacks are processed in RealWaitForChar?  A cursory inspection shows there might be job, channel, resize, and title updates which can execute vim script.  If so, the problem still exists even if you were to remove the TextYankPost autocmd for the star register and these bugs will be even harder to reproduce in the future.
I'm not certain what a use case is for TextYankPost on the star register, but if this autocmd is the only possible issue, a simpler fix would be:
diff --git a/src/regexp_nfa.c b/src/regexp_nfa.c
index d72430fad..1787c4d39 100644
--- a/src/regexp_nfa.c
+++ b/src/regexp_nfa.c
@@ -7185,7 +7185,9 @@ nfa_regexec_both(
        prog->state[i].lastlist[1] = 0;
     }

+    block_autocmds();
     retval = nfa_regtry(prog, col, tm, timed_out);
+    unblock_autocmds();

     nfa_regengine.expr = NULL;

Although as mentioned this might not cover all the possible recursive cases.

With some effort it would be possible to make regexec work recursively.

Could the remaining globals just be stuffed into rex and/or *prog?
Another alternative would be to use a more restricted version of ui_breakcheck/RealWaitForChar which cannot perform unsafe callbacks during regex execution.

Please check if the problem still exists after patch 8.1.0192.
It should make executing a regexp recursively work.
But I haven't yet been able to trigger that in a test.
--
hundred-and-one symptoms of being an internet addict:
263. You have more e-mail addresses than shorts.
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

Looks like a good change.  I was able to get nfa_regexec_both to be entered recursively (modifying the source to printf when this occurs) with version 8.1.0194 without crashing using this example: https://gist.githubusercontent.com/andymass/8c9d570bb4445208302a317cfd421131/raw/4fe36a1c3cbe300398f2ab31ca5025004cdb1329/crash2.vim .  Crashes in vim 8.1.0191.


crashed the app

This happens if there's no json in box

Can you please elaborate on this?
Did you do this without signing in? Clicking the editor link in the top nav bar?

This issue was fixed in later sprint


[Explorer] add radio button to choose output format when downloading data

Download options are in the API now.


Make a polymorphic function test case

Is your feature request related to a problem? Please describe.
It is not clear that polymorphic functions work correctly with codegen yet.
Describe the solution you'd like
Make a hash : (X,Y) -> u64 function and supply some different values for X and Y. Then check that codegen works with these function definitions.

Body polymorphism has been confirmed as bugged. Thinking about this a little bit it is clear that this should not work with a single body, thus the name poly-morphism: multiple arrows.


#669 New flag for GKE ingress

Fixes #669
I know this needs to be mirrored to versioned_docs/2.7, I just want to verify that the edits look good.

@btat Sorry about that, moved too fast. I'll update accordingly

...and now that you've pointed it out, I think the release note for this also uses : instead of =.


Added a built-in function split_char()

Hi,
I am adding a new built-in function split_char() that splits a given string by a given character and returns a list.

@sanatdeshpande1 looks great!

@andrefpoliveira Thanks. :)


Create documentation

Create documentation for the syntax and the functionality of the language.
Since the language is currently in an usable state, this is pretty important.

There is still stuff to write, for example documentation on modules and standard libraries, but for the time being it's fine. May reopen the issue later on just write to the wiki as-is.


How to access the current environment?

I have an angular app and I'm using @aws-amplify/ui-angular and aws-amplify.
How do I access the current environment that I'm hooked up to? Just to be sure, I have two environments amplify env list:
| Environments |
| ------------ |
| dev.         |
| *prod        |

I see: amplify/.config/local-env-info.json and it has:
  "envName": "prod"

So how to do I get that envName at runtime?

Closing in favour of #4364
As a workaround , you can check for aws-exports.js file , some variables might contain envName attached to some resource variables. For example :
You can do some easy regex to filter that out. This comment might help: https://github.com/aws-amplify/amplify-cli/issues/4364#issuecomment-635768030


Multiline column descriptions result in unclosed string literals

Describe the bug

In BigQuery, the new base_<node/source>_columns models fail with a syntax error when a multiline column description is present.
Steps to reproduce

Take a single line description, and convert to a multiline description using block notation.
description: >
  My column description

Expected results

I expected dbt project evaluator to run without issue.
Actual results

Instead, the new base_<node/source>_columns models failed with "Unclosed string literal" syntax errors.
Screenshots and log output

Inspecting the failed BigQuery job shows that it is interpreting the multiline string as unclosed individual strings.

System information
The contents of your packages.yml file:
packages:
  - package: dbt-labs/dbt_project_evaluator
    version: [">=0.9.0", "<0.10.0"]

Which database are you using dbt with?

[ ] postgres
[ ] redshift
[x] bigquery
[ ] snowflake
[ ] trino/starburst
[ ] other (specify: ____________)

The output of dbt --version:
I don't have ready access to the CLI at the moment, but these are the versions listed at the start of a Cloud IDE invocation.
dbt=1.6.9, dbt-bigquery=1.6.9
Additional context

I think I've traced the issue to the active line in the wrap_string_with_quotes macro.
It seems like you all are trying to move away from the macro in #419, but I don't know whether dbt.string_literal has a triple-quoting option to support multiline BigQuery strings.
Are you interested in contributing the fix?

Yes! I've played around in #420, and have confirmed that it resolves the issue in our own project.

Thanks for raising it!
We will fix it in a patch in the next day or so.
Ideally, it also requires updates to dbt-bigquery which I did a PR for https://github.com/dbt-labs/dbt-bigquery/pull/1089 , but we can include the new bigquery__string_literal in this package for now.

Sweet! Thanks, Benoit.

Hey all, any update on this? Any ETA?

Thanks for the follow-up. It slipped my mind.
We have a fix in #422 but it wasn't merged yet as we had some comments about it.
If you need the fix ASAP you can install from the branch wrap-strings-quote-update until we get it merged and released.

@b-per FWIW, we upgraded to 0.11.0 this afternoon, and we're still encountering the Syntax error: Unclosed string literal errors in base_node_columns and base_source_columns. Essentially, we're observing the same behavior noted in https://github.com/dbt-labs/dbt-project-evaluator/issues/448.

Does it fail as well?
It does fail:

$ dbt show --inline 'select {{ dbt_project_evaluator.wrap_string_with_quotes("ab\nc")}} '
19:49:03  Running with dbt=1.7.10
19:49:04  Registered adapter: bigquery=1.7.7
19:49:06  Found 478 models, 5 snapshots, 18 seeds, 294 tests, 179 sources, 1 exposure, 39 metrics, 1738 macros, 0 groups, 7 semantic models
19:49:06
19:49:07  Concurrency: 4 threads (target='dev')
19:49:07
19:49:09  BigQuery adapter: https://console.cloud.google.com/bigquery?project=inq-warehouse&j=bq:US:c1f8323b-a1e2-4bf8-9b6f-6d9238bbace3&page=queryresults
19:49:09  Encountered an error:
Runtime Error
  Database Error in sql_operation inline_query (from remote system.sql)
    Syntax error: Unclosed string literal at [8:5]$ dbt show --inline 'select {{ dbt_project_evaluator.wrap_string_with_quotes("ab\nc")}} '
19:49:03  Running with dbt=1.7.10
19:49:04  Registered adapter: bigquery=1.7.7
19:49:06  Found 478 models, 5 snapshots, 18 seeds, 294 tests, 179 sources, 1 exposure, 39 metrics, 1738 macros, 0 groups, 7 semantic models
19:49:06
19:49:07  Concurrency: 4 threads (target='dev')
19:49:07
19:49:09  BigQuery adapter: https://console.cloud.google.com/bigquery?project=inq-warehouse&j=bq:US:c1f8323b-a1e2-4bf8-9b6f-6d9238bbace3&page=queryresults
19:49:09  Encountered an error:
Runtime Error
  Database Error in sql_operation inline_query (from remote system.sql)
    Syntax error: Unclosed string literal at [8:5]

And here's the corresponding job in BigQuery:

Thanks. There is a problem with this macro then, because it works for me locally.

can you do a dbt clean and dbt deps to check if the dbt show command would work?
if you go to dbt_packages/dbt_project_evaluator/macros/wrap_string_with_quotes.sql , what do you see? Is the macro bigquery__string_literal there?
Did you overwrite the macro bigquery__string_literal(value) with you personal version in your dbt project?

can you do a dbt clean and dbt deps to check if the dbt show command would work?

here's the full output of the three commands:
$ dbt clean && dbt deps && dbt show --inline 'select {{ dbt_project_evaluator.wrap_string_with_quotes("ab\nc")}} '
20:49:07  Running with dbt=1.7.13
20:49:08  Checking /workspaces/inquirer-dbt/logs/*
20:49:08  Cleaned /workspaces/inquirer-dbt/logs/*
20:49:08  Checking /workspaces/inquirer-dbt/dbt_packages/*
20:49:08  Cleaned /workspaces/inquirer-dbt/dbt_packages/*
20:49:08  Checking /workspaces/inquirer-dbt/target-base/*
20:49:08  Cleaned /workspaces/inquirer-dbt/target-base/*
20:49:08  Checking /workspaces/inquirer-dbt/target/*
20:49:08  Cleaned /workspaces/inquirer-dbt/target/*
20:49:08  Finished cleaning all paths.
20:49:10  Running with dbt=1.7.13
20:49:11  Installing dbt-labs/dbt_utils
20:49:11  Installed from version 1.1.1
20:49:11  Up to date!
20:49:11  Installing calogica/dbt_expectations
20:49:11  Installed from version 0.10.3
20:49:11  Up to date!
20:49:11  Installing dbt-labs/codegen
20:49:11  Installed from version 0.12.1
20:49:11  Up to date!
20:49:11  Installing data-mie/dbt_profiler
20:49:11  Installed from version 0.8.1
20:49:11  Up to date!
20:49:11  Installing dbt-labs/dbt_project_evaluator
20:49:11  Installed from version 0.11.0
20:49:11  Up to date!
20:49:11  Installing brooklyn-data/dbt_artifacts
20:49:11  Installed from version 2.6.2
20:49:11  Up to date!
20:49:11  Installing elementary-data/elementary
20:49:12  Installed from version 0.14.1
20:49:12  Up to date!
20:49:12  Installing dbt-labs/audit_helper
20:49:12  Installed from version 0.11.0
20:49:12  Up to date!
20:49:12  Installing grafana/leaner_query
20:49:12  Installed from version 0.1.12
20:49:12  Up to date!
20:49:12  Installing Velir/ga4
20:49:12  Installed from version 6.0.1
20:49:12  Up to date!
20:49:12  Installing calogica/dbt_date
20:49:12  Installed from version 0.10.1
20:49:12  Up to date!
20:49:15  Running with dbt=1.7.13
20:49:16  Registered adapter: bigquery=1.7.7
20:49:16  Unable to do partial parsing because saved manifest not found. Starting full parse.
20:49:37  [WARNING]: Did not find matching node for patch with name 'dim_query_statement' in the 'models' section of file 'models/marts/bigquery_audit_log.yml'
20:49:37  [WARNING]: Did not find matching node for patch with name 'int_dbt_execution_info' in the 'models' section of file 'models/intermediate/int_bigquery_audit_log.yml'
20:49:37  [WARNING]: Did not find matching node for patch with name 'int_importance_calculation' in the 'models' section of file 'models/intermediate/int_bigquery_audit_log.yml'
20:49:38  [WARNING]: Test 'test.leaner_query.not_null_dim_query_statement_job_key.dde1be6c77' (models/marts/bigquery_audit_log.yml) depends on a node named 'dim_query_statement' in package '' which was not found
20:49:40  Found 478 models, 5 snapshots, 294 tests, 18 seeds, 179 sources, 1 exposure, 39 metrics, 1738 macros, 0 groups, 7 semantic models
20:49:40
20:49:40  Concurrency: 4 threads (target='dev')
20:49:40
20:49:41  BigQuery adapter: https://console.cloud.google.com/bigquery?project=inq-warehouse&j=bq:US:f0f374a1-2a89-4feb-983a-fbe9facb879a&page=queryresults
20:49:41  Encountered an error:
Runtime Error
  Database Error in sql_operation inline_query (from remote system.sql)
    Syntax error: Unclosed string literal at [8:5]

if you go to dbt_packages/dbt_project_evaluator/macros/wrap_string_with_quotes.sql , what do you see? Is the macro bigquery__string_literal there?

It is. Here's the full contents of the file dbt_packages/dbt_project_evaluator/macros/wrap_string_with_quotes.sql:
{% macro wrap_string_with_quotes(str) %}
  {% if str is none %}
    {{ return('cast(NULL as ' ~ dbt.type_string() ~ ')') }}
  {% else %}
    {{ dbt.string_literal(str) }}
  {% endif %}
{% endmacro %}

{#
  To be removed when https://github.com/dbt-labs/dbt-bigquery/pull/1089 is merged
#}
{% macro bigquery__string_literal(value) -%}
  '''{{ value }}'''
{%- endmacro %}

Did you overwrite the macro bigquery__string_literal(value) with you personal version in your dbt project?

We did not.

We will get to the bottom of it 😄
What I am thinking now then is that one of the other packages you have is definining bigquery__string_literal() themselves.

once all the packages are installed, can you search for bigquery__string_literal to see which package defines it?
if you open manifest.json and search for the bigquery__string_literal macro entry, what package is listed?
could you also try adding the following to your dbt_project.yml?

dispatch:
  - macro_namespace: dbt
    search_order: ['dbt_project_evaluator', 'dbt']

Ok, some progress here! Specifically, the change to the dispatch config resolved the issue. Note that I addressed your first two points before editing the dispatch config:

once all the packages are installed, can you search for bigquery__string_literal to see which package defines it?

I'm only seeing it defined in ./dbt_packages/dbt_project_evaluator/macros/:
$ dbt clean && dbt deps && grep -iR "bigquery__string_literal" ./dbt_packages
16:41:25  Running with dbt=1.7.13
16:41:25  Checking /workspaces/inquirer-dbt/target/*
16:41:25  Cleaned /workspaces/inquirer-dbt/target/*
16:41:25  Checking /workspaces/inquirer-dbt/dbt_packages/*
16:41:26  Cleaned /workspaces/inquirer-dbt/dbt_packages/*
16:41:26  Checking /workspaces/inquirer-dbt/logs/*
16:41:26  Cleaned /workspaces/inquirer-dbt/logs/*
16:41:26  Checking /workspaces/inquirer-dbt/target-base/*
16:41:26  Cleaned /workspaces/inquirer-dbt/target-base/*
16:41:26  Finished cleaning all paths.
16:41:28  Running with dbt=1.7.13
16:41:29  Installing dbt-labs/dbt_utils
16:41:29  Installed from version 1.1.1
16:41:29  Up to date!
16:41:29  Installing calogica/dbt_expectations
16:41:29  Installed from version 0.10.3
16:41:29  Up to date!
16:41:29  Installing dbt-labs/codegen
16:41:29  Installed from version 0.12.1
16:41:29  Up to date!
16:41:29  Installing data-mie/dbt_profiler
16:41:29  Installed from version 0.8.1
16:41:29  Up to date!
16:41:29  Installing dbt-labs/dbt_project_evaluator
16:41:29  Installed from version 0.11.0
16:41:29  Up to date!
16:41:29  Installing brooklyn-data/dbt_artifacts
16:41:29  Installed from version 2.6.2
16:41:29  Up to date!
16:41:29  Installing elementary-data/elementary
16:41:29  Installed from version 0.14.1
16:41:29  Up to date!
16:41:29  Installing dbt-labs/audit_helper
16:41:29  Installed from version 0.11.0
16:41:29  Up to date!
16:41:29  Installing grafana/leaner_query
16:41:30  Installed from version 0.1.12
16:41:30  Up to date!
16:41:30  Installing Velir/ga4
16:41:30  Installed from version 6.0.1
16:41:30  Up to date!
16:41:30  Installing calogica/dbt_date
16:41:30  Installed from version 0.10.1
16:41:30  Up to date!
./dbt_packages/dbt_project_evaluator/macros/wrap_string_with_quotes.sql:{% macro bigquery__string_literal(value) -%}

if you open manifest.json and search for the bigquery__string_literal macro entry, what package is listed?

After running dbt clean && dbt deps && dbt compile, this object contains the only occurrences of bigquery__string_literal in manifest.json
"macro.dbt_project_evaluator.bigquery__string_literal": {
  "name": "bigquery__string_literal",
  "resource_type": "macro",
  "package_name": "dbt_project_evaluator",
  "path": "macros/wrap_string_with_quotes.sql",
  "original_file_path": "macros/wrap_string_with_quotes.sql",
  "unique_id": "macro.dbt_project_evaluator.bigquery__string_literal",
  "macro_sql": "{% macro bigquery__string_literal(value) -%}\n  '''{{ value }}'''\n{%- endmacro %}",
  "depends_on": { "macros": [] },
  "description": "",
  "meta": {},
  "docs": { "show": true, "node_color": null },
  "patch_path": null,
  "arguments": [],
  "created_at":<PHONE_NUMBER>.5902495,
  "supported_languages": null
},

could you also try adding the following to your dbt_project.yml?

That resolved the issue!
$ dbt clean && dbt deps && dbt show --inline 'select {{ dbt_project_evaluator.wrap_string_with_quotes("ab\nc")}} '
16:53:21  Running with dbt=1.7.13
16:53:21  Checking /workspaces/inquirer-dbt/target-base/*
16:53:21  Cleaned /workspaces/inquirer-dbt/target-base/*
16:53:21  Checking /workspaces/inquirer-dbt/target/*
16:53:21  Cleaned /workspaces/inquirer-dbt/target/*
16:53:21  Checking /workspaces/inquirer-dbt/dbt_packages/*
16:53:21  Cleaned /workspaces/inquirer-dbt/dbt_packages/*
16:53:21  Checking /workspaces/inquirer-dbt/logs/*
16:53:21  Cleaned /workspaces/inquirer-dbt/logs/*
16:53:21  Finished cleaning all paths.
16:53:24  Running with dbt=1.7.13
16:53:25  Installing dbt-labs/dbt_utils
16:53:25  Installed from version 1.1.1
16:53:25  Up to date!
16:53:25  Installing calogica/dbt_expectations
16:53:25  Installed from version 0.10.3
16:53:25  Up to date!
16:53:25  Installing dbt-labs/codegen
16:53:25  Installed from version 0.12.1
16:53:25  Up to date!
16:53:25  Installing data-mie/dbt_profiler
16:53:25  Installed from version 0.8.1
16:53:25  Up to date!
16:53:25  Installing dbt-labs/dbt_project_evaluator
16:53:25  Installed from version 0.11.0
16:53:25  Up to date!
16:53:25  Installing brooklyn-data/dbt_artifacts
16:53:25  Installed from version 2.6.2
16:53:25  Up to date!
16:53:25  Installing elementary-data/elementary
16:53:25  Installed from version 0.14.1
16:53:25  Up to date!
16:53:25  Installing dbt-labs/audit_helper
16:53:25  Installed from version 0.11.0
16:53:25  Up to date!
16:53:25  Installing grafana/leaner_query
16:53:25  Installed from version 0.1.12
16:53:25  Up to date!
16:53:25  Installing Velir/ga4
16:53:25  Installed from version 6.0.1
16:53:25  Up to date!
16:53:25  Installing calogica/dbt_date
16:53:26  Installed from version 0.10.1
16:53:26  Up to date!
16:53:28  Running with dbt=1.7.13
16:53:30  Registered adapter: bigquery=1.7.7
16:53:30  Unable to do partial parsing because saved manifest not found. Starting full parse.
16:53:50  [WARNING]: Did not find matching node for patch with name 'dim_query_statement' in the 'models' section of file 'models/marts/bigquery_audit_log.yml'
16:53:50  [WARNING]: Did not find matching node for patch with name 'int_dbt_execution_info' in the 'models' section of file 'models/intermediate/int_bigquery_audit_log.yml'
16:53:50  [WARNING]: Did not find matching node for patch with name 'int_importance_calculation' in the 'models' section of file 'models/intermediate/int_bigquery_audit_log.yml'
16:53:50  [WARNING]: Test 'test.leaner_query.not_null_dim_query_statement_job_key.dde1be6c77' (models/marts/bigquery_audit_log.yml) depends on a node named 'dim_query_statement' in package '' which was not found
16:53:52  Found 478 models, 5 snapshots, 294 tests, 18 seeds, 179 sources, 1 exposure, 39 metrics, 1738 macros, 0 groups, 7 semantic models
16:53:53
16:53:53  Concurrency: 4 threads (target='dev')
16:53:53
16:53:54  Previewing inline node:
| f0_  |
| ---- |
| ab
c |

@b-per Thanks for all the assistance, @b-per. FWIW, there's one additional instance of bigquery__string_literal in my manifest.json after making the change to dispatch (both nested under the macros key:
"macro.dbt.string_literal": {
  "name": "string_literal",
  "resource_type": "macro",
  "package_name": "dbt",
  "path": "macros/utils/literal.sql",
  "original_file_path": "macros/utils/literal.sql",
  "unique_id": "macro.dbt.string_literal",
  "macro_sql": "{%- macro string_literal(value) -%}\n  {{ return(adapter.dispatch('string_literal', 'dbt') (value)) }}\n{%- endmacro -%}\n\n",
  "depends_on": {
    "macros": ["macro.dbt_project_evaluator.bigquery__string_literal"]
  },
  "description": "",
  "meta": {},
  "docs": { "show": true, "node_color": null },
  "patch_path": null,
  "arguments": [],
  "created_at":<PHONE_NUMBER>.2609057,
  "supported_languages": null
},
"macro.dbt_project_evaluator.bigquery__string_literal": {
  "name": "bigquery__string_literal",
  "resource_type": "macro",
  "package_name": "dbt_project_evaluator",
  "path": "macros/wrap_string_with_quotes.sql",
  "original_file_path": "macros/wrap_string_with_quotes.sql",
  "unique_id": "macro.dbt_project_evaluator.bigquery__string_literal",
  "macro_sql": "{% macro bigquery__string_literal(value) -%}\n  '''{{ value }}'''\n{%- endmacro %}",
  "depends_on": { "macros": [] },
  "description": "",
  "meta": {},
  "docs": { "show": true, "node_color": null },
  "patch_path": null,
  "arguments": [],
  "created_at":<PHONE_NUMBER>.0822887,
  "supported_languages": null
},


Triggers - Build Completion - Branch filters wildcard support

I am currently using Azure Devops, and even though this documentation has screenshots of Branch filters wildcard support on visual designer, the dropdown for branches is only letting me select from the available branches. I even tried typing the branch specification (features/*), but it does not let me do that so there is no question of pressing enter after typing it as typing is not available. I managed to overcome the above issue by using yaml fo Continuous Integration trigger but since Build Completion trigger does not support yaml as yet and there seems no way to do it through the visual designer, I am stuck. Could somebody please help.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: f5673ec1-c7d1-e0ac-043a-8fc3d5b226a9
Version Independent ID: b4872831-51c0-6cf8-1c56-de20beb38a80
Content: Build pipeline triggers - Azure Pipelines & TFS
Content Source: docs/pipelines/build/triggers.md
Product: devops
GitHub Login: @andyjlewis
Microsoft Alias: alewis

Sorry, I realized that we need to click the dropdown first and then type it in the 'Filter my branches' textbox and then press enter. Please feel free to close this.


Add File()

similar to EnvVar()
It's just a smal helper to make NewValueSourceChain more usefull :)

will be used in https://github.com/woodpecker-ci/woodpecker/pull/2951 ...


Can import package - Angular 12 support

error when npm install,

Hi,
Thank you for reporting this issue, this was due to Angular 12 support.
This should be working now, be sure to install the newest version I've published, with:
npm install<EMAIL_ADDRESS>
Please tell me if it's working on your side
Have a good day

still same issue 😢

Can you tell me which Angular version are you using?

Can you tell me which Angular version are you using?

I think this is due to your version of Node (16.3.0) that is not supporter by Angular.
Try to use Node v14 instead (see how to do here: https://subscription.packtpub.com/book/web_development/9781838558758/1/ch01lvl1sec03/installing-node-js-14-with-nvm)

i  install  node 14 and works fine , thanks 🙏🙏


fixed optimize not work when config.out is a function in nodejs V6

Thanks! Note this project is looking for a maintainer to ensure these kinds of fixes get release promptly.


Admin panel allow to create user with email address already used by another user.

Gogs version (or commit ref): https://dl.gogs.io/gogs_v0.9.97_linux_amd64.tar.gz

Git version: 2.9.3

Operating system: Ubuntu 14.04.5 LTS (GNU/Linux 2.6.32-042stab116.2 x86_64)

Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[x] SQLite

Can you reproduce the bug at https://try.gogs.io:

[ ] Yes (provide example URL)
[x] No - Need access to admin panel
[ ] Not relevant

Log gist:
2016/10/05 02:07:58 [...routers/repo/repo.go:101 handleCreateError()] [E] CreatePost: updateUser: e-mail has been used [email: <email address>]

Description
Step to reproduce

Configure admin user profile email.
Use admin panel to create user with same email address as admin.
Login to new user with username. Create new repo will result in "Error 500" page.

Expected behavior
Admin panel should not allow new user with email address already used by another account.

Duplicated with #3681

oops, sorry, please close!

@J-Siu nah, no problem, your help are welcome :)


Add 3 new sounds

-Images should be picked from the royalty free selection here https://unsplash.com/
-the image should be related to the sounds provided
-the sounds should be relaxing: thunderstorm, zen bells, city sounds etc.
-royalty free libraries for music: https://pixabay.com/music/
-no music but sounds
-sounds should not be same/similar to what already available

this was done correct? what would you say the max should be

@DavidKozdra do you mean max in terms of amount of sounds/musics?
I'm suggesting to increase slowly (3 music at time) so I can check we're not overload the website and repository - if you meant instead weight wise, I would say music max around 3mins loop-able, and image low resolution. Let me know if you have any question :)

Closed by #100


initialValues prop is undefined when initialised from the state on first render

redux-form v6.5.0
initialValues prop is undefined when initialised from the state on first render
Lifecycle

The state is empty when the constructor is called
An action is dispatched to load data into the store
InitialValues data are loaded by the selector
@@redux-form/INITIALIZE is called and the form is populated with the values
this.props.initialValues is undefined
render() is called once

Expected behaviour:

this.props.initialValues should be populated and componentWillReceiveProps called

Actual behaviour:

this.props.initialValues is populated only when the form is edited

Options

enableReinitialize option doesn't fix the problem, and I intend to keep it to false.

Example
import React, { PropTypes } from 'react'
import { Field, reduxForm } from 'redux-form'
import { connect } from 'react-redux'

import { Button, Input } from '../../components'
import { FORM } from './constants'
import { fetchData } from './actions'
import { makeSelect } from './selectors'

export class Form extends React.PureComponent {

  static propTypes = {
    initialValues: PropTypes.object,
    fetchData: PropTypes.func.isRequired,
    handleSubmit: PropTypes.func.isRequired,
  }

  constructor(props) {
    super(props)
    this.props.fetchData() // Load data from the server into the store
    this.props.initialValues === undefined //=> true, which is OK since there are no data yet
  }

  componentWillReceiveProps(nextProps) {
    // componentWillReceiveProps is not called, until next props change (ie: form change)
  }

  render() {
    return (
      <div>
        <form onSubmit={this.props.handleSubmit}>
          <Field
            name="name"
            component={Input}
          />
          <Button>
            Save
          </Button>
        </form>
      </div>
    )
  }
}

export default connect(
  (state, props) => ({
    initialValues: makeSelect()(state, props),
  }),
  { fetchData }
)(reduxForm({
  form: FORM,
  onSubmit: (data) => console.log(data),
})(Form))

React.PureComponent doesn't have React's lifecycle methods, use React.Component instead

@leanderhoedt Of course they have lifecycle methods. https://facebook.github.io/react/docs/react-api.html#react.purecomponent.

Because this issue saw almost no activity for a few months, I'm closing it.


Add myself to contributors, re. rust-cryptoki

Cf. https://github.com/parallaxsecond/rust-cryptoki/pull/7

Thanks, good to have you here 😃
One check failed because you just need to sign-off your commit, with the Signed-off-by ... field (git commit -s thing).


Rewrite g4py interface to accomodate changes due to refactoring of CADMesh

37372e2ec74613d4174b0d0e8038cd2621005a1a caused the Python interface to fail because the code has been restructured.


hgtdb_20170630.tar.xz

I am trying to do "download a pre-built database as of 2023-01-02, and compile it.", but I compile database "hgtdb_20170630.tar.xz" , it doesn't have "taxon.map.gz". And what should I do?

Hello @YingyingZ9 The 2017 database was too old and it was designed for HGTector v0.x. It won't work with the current HGTector. With that being said, it may be possible for you to hack the database to make it compatible.

Hello @YingyingZ9 The 2017 database was too old and it was designed for HGTector v0.x. It won't work with the current HGTector. With that being said, it may be possible for you to hack the database to make it compatible.

Thank you for your reply. I've encountered a new problem，when I enter it “hgtector search -i ***/hgtdb_20230102/japonica/.faa.gz -o search_dir -m diamond -p 16 -d ****/hgtdb_20230102/diamond/db.dmnd -t ****/hgtdb_20230102/taxdump” ，I got this result：Homology search started at 2023-10-18 09:08:28.411085.
Settings:
Search method: diamond.
Self-alignment method: native.
Remote fetch enabled: no.
Reading input proteins...
japonica: 55801 proteins.
Done. Read 55801 proteins from 1 samples.
Dropping sequences shorter than 30 aa... done.
Reading local taxonomy database... done.
Read 51329 taxa.
Batch homology search of japonica started at 2023-10-18 09:08:29.332318.
Number of queries: 55786.
Traceback (most recent call last):
File "/home/zhouyingying/data_HD/miniconda3/envs/hgtector/bin/hgtector", line 96, in
main()
File "/home/zhouyingying/data_HD/miniconda3/envs/hgtector/bin/hgtector", line 35, in main
module(args)
File "/home/zhouyingying/data_HD/miniconda3/envs/hgtector/lib/python3.11/site-packages/hgtector/search.py", line 177, in call
res = self.search_wf(
^^^^^^^^^^^^^^^
File "/home/zhouyingying/data_HD/miniconda3/envs/hgtector/lib/python3.11/site-packages/hgtector/search.py", line 692, in search_wf
res = self.diamond_search(seqs)
^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/zhouyingying/data_HD/miniconda3/envs/hgtector/lib/python3.11/site-packages/hgtector/search.py", line 1539, in diamond_search
raise ValueError(f'diamond failed with error code {ec}.')
ValueError: diamond failed with error code 1.


WIP: Split

Split the Service Bus Explorer into some assemblies.
Use the new .NET Standard based libraries for Event Hubs, Relay and Service Bus.
Moved some classes into the new assemblies.

Replaced this PR with a later one.


How to run it ?

I am not familiar to djingo.
My question is , when I git clone it from website, how can i run it ?
I have using python setup.py build and python setup.py install

(venv)guo@pingdeMacBook-Air-2 coupons$ pwd
/Users/guo/Code/repository/02.Learn/00.python_learn/django-coupons/build/lib/coupons
(venv)guo@pingdeMacBook-Air-2 coupons$ l
total 48
-rw-r--r--  1 guo  staff    22B  1  4 16:50 __init__.py
-rw-r--r--  1 guo  staff   2.7K  1  4 16:50 admin.py
-rw-r--r--  1 guo  staff   1.9K  1  4 16:50 forms.py
drwxr-xr-x  3 guo  staff   102B  1  4 16:51 locale
drwxr-xr-x  6 guo  staff   204B  1  4 16:51 migrations
-rw-r--r--  1 guo  staff   4.0K  1  4 16:50 models.py
-rw-r--r--  1 guo  staff   598B  1  4 16:50 settings.py
drwxr-xr-x  6 guo  staff   204B  1  4 16:51 south_migrations
drwxr-xr-x  3 guo  staff   102B  1  4 16:51 templates
drwxr-xr-x  7 guo  staff   238B  1  4 16:51 tests
(venv)guo@pingdeMacBook-Air-2 coupons$ python admin.py runserver
Traceback (most recent call last):
  File "admin.py", line 2, in <module>
    from django.contrib import admin
  File "/Users/guo/Code/repository/01.Work/grid-cdnzz/venv/lib/python2.7/site-packages/django/contrib/admin/__init__.py", line 4, in <module>
    from django.contrib.admin.helpers import ACTION_CHECKBOX_NAME
  File "/Users/guo/Code/repository/01.Work/grid-cdnzz/venv/lib/python2.7/site-packages/django/contrib/admin/helpers.py", line 8, in <module>
    from django.contrib.admin.utils import (
  File "/Users/guo/Code/repository/01.Work/grid-cdnzz/venv/lib/python2.7/site-packages/django/contrib/admin/utils.py", line 7, in <module>
    from django.contrib.auth import get_permission_codename
  File "/Users/guo/Code/repository/01.Work/grid-cdnzz/venv/lib/python2.7/site-packages/django/contrib/auth/__init__.py", line 7, in <module>
    from django.middleware.csrf import rotate_token
  File "/Users/guo/Code/repository/01.Work/grid-cdnzz/venv/lib/python2.7/site-packages/django/middleware/csrf.py", line 14, in <module>
    from django.utils.cache import patch_vary_headers
  File "/Users/guo/Code/repository/01.Work/grid-cdnzz/venv/lib/python2.7/site-packages/django/utils/cache.py", line 26, in <module>
    from django.core.cache import caches
  File "/Users/guo/Code/repository/01.Work/grid-cdnzz/venv/lib/python2.7/site-packages/django/core/cache/__init__.py", line 34, in <module>
    if DEFAULT_CACHE_ALIAS not in settings.CACHES:
  File "/Users/guo/Code/repository/01.Work/grid-cdnzz/venv/lib/python2.7/site-packages/django/conf/__init__.py", line 48, in __getattr__
    self._setup(name)
  File "/Users/guo/Code/repository/01.Work/grid-cdnzz/venv/lib/python2.7/site-packages/django/conf/__init__.py", line 42, in _setup
    % (desc, ENVIRONMENT_VARIABLE))
django.core.exceptions.ImproperlyConfigured: Requested setting CACHES, but settings are not configured. You must either define the environment variable DJANGO_SETTINGS_MODULE or call settings.configure() before accessing settings.

I would recommend you to start with the official Django tutorial first: https://docs.djangoproject.com/en/1.9/intro/tutorial01/
Considering your question: this is just a pluggable application which can be run within a Django project but not standalone. You could follow the first steps of the tutorial and then have a look at the readme file of this project on how to run it.
I will close this ticket since it is not really related to this project. You should get familiar with Django first, if there are any questions then, on how to run this application, fee free to reopen the ticket. Good luck! :)


Add docs about passing a function to a tooltip title

This documents the option of passing a function to tooltip's title.
/cc @atom/feedback

Looks good to me! :shipit:


update gaudi resource driver chart to v0.2.0

@eero-t, @byako, PTAL

@byako This looks (still) OK to me.


Unable to run tests in es6 branch

Checked master, npm install, run build and test: everything fine
When i checked es6 branch i got a couple of errors:
No babel -> installed locally or global (same effect)
No mocha-phantomjs -> installed global
npm run build > bundled version created fine
npm test -> got the following error:

<EMAIL_ADDRESS>pretest D:\repositories\rivets
babel src/parsers.js --out-file spec/lib/parsers.js --modules umd

<EMAIL_ADDRESS>test D:\repositories\rivets
mocha-phantomjs spec/runner.html

SyntaxError: Parse error

npm ERR! Test failed.  See above for more details.

I checked spec/lib/parsers.js and is using es6 syntax, instead of umd
Using node 4.2 under windows

Hit the same issue, have fixed and submitted a PR. Feel free to test!


[wip] refactor validates_timeline to supports :timeline

@miq-bot add_label pluggable providers, refactoring, darga/no

@mzazrivec I wonder if timeline is a pure UI thing. Can you point me to the code or tell me what a Manager or a Vm has to implement, so that it can be used in a timeline? Because the amazon Vm do not support timeline, but I dont know why

Can you point me to the code or tell me what a Manager or a Vm has to implement, so that it can be used in a timeline?

@mzazrivec @bmclaughlin - is it that every provider that has an event_catcher automatically supports the timeline thing?

@durandom needs rebase ... assign back to me after it's ready

@miq-bot add_label wip

@bmclaughlin @mzazrivec ping

Can you point me to the code or tell me what a Manager or a Vm has to implement, so that it can be used in a timeline?

is it that every provider that has an event_catcher automatically supports the timeline thing?

@durandom timelines do show events. You need the events defined in the settings.yml and the event needs to be correctly tied with the timeline object (Provider, Vm, etc.). Timelines show Provider's Events, caught by event catcher. But I think they show also MIQ events, e.g. produced by Alerts.
Then I think the Timelines toolbar button, should be enabled when there are events present, so same as for the C&U, which is enabled when there are metrics.

@durandom yeah I think it should be available to all. Just depends on the events presence, I think now it checks presence of any event, without timeframe. Not sure where are all the Timelines, should be Provider, Vm, Cluster, Availability Zone, maybe more.

@durandom it looks like the change that I made was due to Amazon not providing events at that time per this BZ [1]
[1] https://bugzilla.redhat.com/show_bug.cgi?id=1023177

@miq-bot remove_label wip

@mzazrivec @Ladas this PR became a bit bigger than expected :smile:
I figured, that every model, that include EventMixin should basically support a timeline, because all methods that are used are in that mixin. Now have a look at all those models in the files list... Does that mean, that all those now have a timeline button? Does that make sense?

@moolitayer @cben can you review/check this? Thanks!

@durandom SupportsFeatureMixin looks very interesting. Can you explain what adding it does to entities(specifically is the system already using if to decide on buttons routes screens etc.)? Is there a need to define :supports in our entities?
from ours I counted: container, container_group, container_node, container_project, container_replicator afaik those should all have timelines

@moolitayer have a look at https://github.com/ManageIQ/manageiq/pull/8886 and the inline docs, which explain it quite good I think. We are in the middle of a transition from validate_ to supports_ calls - and most of the buttons make already use of this.

Nice, so includers of EventMixin get supports :timeline.
Once they have SupportsFeatureMixin they can also report that and EmsTimeline button checks for that support.
@simon3z Looks good to me with regards to containers, verified the relevant entities have timeline button  & timeline is accessible.

@Ladas can you have a look at this if this makes sense?

Looks good to me :+1:

@blomquisg what do you think? For @Ladas this is good 😄

cool, thanks @blomquisg ❤
now, lets see how this pans out :)


Misleading Syncing Preferences window

Describe the bug
Users having a FreshRSS account report they are unable to connect to it using Vienna.
They get messages "A server with the specified hostname could not be found". https://github.com/FreshRSS/FreshRSS/issues/3417
When I tried to reproduce the problem with a test account, I noticed in my debug info something similar to (redacted):
Task <A7944B2B-497A-4087-9A08-C634E50C7800>.<1> finished with error [-1003] Error Domain=NSURLErrorDomain Code=-1003 "A server with the specified hostname could not be found"
....
NSErrorFailingURLStringKey=https://https://freshrss.example.com/api/greader.php/accounts/ClientLogin?accountType=GOOGLE&service=reader

(note the double sequence https://https:// in the URL)
It occurred to me that Vienna's syncing preferences dialog is misleading. The window suggests to input a complete URL, while the user should only provide a hostname and a path.
In the above example, I immediately got correct access once I replaced https://freshrss.example.com/api/greader.phpwith freshrss.example.com/api/greader.php in my settings.
Screenshots
In previous versions of Vienna, the preference window was more explicit about what was expected.
Here is a 3.5.3 screenshot : an https:// string does show on the window, as an advice to the user to avoid inputting any scheme.

This changed once we switched to a storyboard based window (commit 5dfb7a1)
The current display is completely misleading:

Additional information:
Change occurred with Version 3.6.

@Eitot , do you see a more elegant (and localizable) solution than just replacing the URL: label with URL: https:// ?

In any case, what about accepting both inputs with and without scheme? You can always add an explicit scheme such as https:// if is not provided by the user. That is what Web browsers have been doing for years


Inconsistency in mocking Cache

Laravel Version: 5.3.x
PHP Version: 7.0
Database Driver & Version: Irrelevant

Description:
The lately added cache() helper is extremely helpfully, but also apparently created some inconsistencies when testing with mocks. Considering this simple example:
public function testCache()
{
    Cache::shouldReceive('get')->once()->with('key');
}

This test will pass with Cache::get('key') but fail with cache('key'), with the following error:
Mockery\Exception\NoMatchingExpectationException: No matching handler found for Mockery_12_Illuminate_Cache_CacheManager::get('key', NULL)

Correct me if I'm wrong, but given that cache() is (or at the very least, widely considered to be) a mere shortcut to Cache::get() and Cache::put(), one would expect their behaviors to be the same, regardless of how the helper is implemented under the hood.
Steps To Reproduce:
As above.

looks correct to me, due to the way __call works

the global function passes through arg 2 because it is set as a default


chore(main): release 1.0.2

:robot: I have created a release beep boop
1.0.2 (2024-11-24)
Bug Fixes

remove publish workflow (301c2eb)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.0.2
:sunflower:


Update on parameters and hyper parameter tuner.

For issue #170

feature: parameter validation
feature: parameter hyper space setting
feature: auto hyper parameter tuning
fix: 'model_class' parameter auto setting

@bwanglzu Validators are mostly lambdas, and dill serializes lambdas while pickle does not. This branch was created before the feature request issue, and was originally for some docstring updates, thus the name.

Tuner is perfectly fine to be a standalone function instead of a class, so I switched.

int and float both inherits numbers.Number. Third party library data types like numpy.int64 is also a subclass of numbers.Number. All of them should work in a parameter hinted as type numbers.Number.

Overall, I like the hyper-parameter tuner. For all model specific hyper-parameters, it will require the model define the default value space.


ci: remove issue-labeler

Summary

Related issue (if exists)
#2681

Types of changes

[ ] Docs change / Dependency upgrade
[ ] Bug fix
[ ] New feature / Improvement
[ ] Refactoring
[ ] Breaking change

Checklist

[ ] I have added changeset via pnpm run changeset.
[ ] I have added tests to cover my changes.

any Context ?

any Context ?

Forget to mention it. We have achieved the same goal at https://github.com/web-infra-dev/rspack/pull/2681.