Rules preview matches all events when adding a new value check rule

I'm using Nightly build 2018-08-28 23:06:03 on Windows.
When you add a value check rule then all the events are automatically matched in the preview. I guess because all events match the value "empty" or null somehow.
When you start typing the actual value this is not updated so all events keep showing as matched.
This resets to the correct matching when you press the save button or hide and show the preview again.
I'd rather not have all events automatically matched using a null/empty value.
I'm not sure what the best way to deal with this is. It might not be efficient to recalculate every time some character is typed.

Like you said at the end, I don't update the list because it starts to reduce performance when I update the event list on every keystroke.
And I'm guessing the events all match with an empty value because you're using the Contains filter type

How about a timeout after someone is done typing? Let's say after 1000ms of any typing-activity, update the list? This way there is more feedback about the matches, without affecting performance to much.


@@ variables do not always get syntax highlighting

select @@VERSION gets syntax highlighting but select @@TRANCOUNT does not
see

"Even though SOME @@ variables ARE syntax highlighted!" under

http://www.smooth1.co.uk/installs/dbinstalls.html#7.2.3 (Redhat)
and

"However SOME @@ variables do get syntax highlighting as on Redhat" under

http://www.smooth1.co.uk/installs/dbinstalls.html#9.1.1  (Ubuntu)

Thanks for reporting the issue, Currently our syntax highlighter supports a limited subset of Transact SQL keywords and statements. We will be working to expand this in the near future.


Allow checking pr_body for #trivial tag

Was requested in https://github.com/danger/danger/pull/181#issuecomment-211677536.

        1 Warning

  </th>
 </tr>

:warning:
Author @jeroenvisser101 is not a contributor

  Generated by :no_entry_sign: danger


Tags autoComplete timing-out on clustered setup (infinite loop!)

Hi there,
I am using a graphite cluster with 4 carbon-cache processes, on 4 different nodes, that run alongside 4 graphite-web instances. Fronting those, 2 cache-relay instances.
When starting to type a query in Grafana (first node field), Grafana issues two API calls:

metrics/find?from=X&until=Y&query=*
tags/autoComplete/tags (or tags/autoComplete/tags?tagPrefix=%2A&limit=100)

However, it looks like something is really wrong with the tags request. First of all, the request takes forever and eventually time-out. But also, after a few attempts in grafana, the logs start spamming with timeouts for tags and tracebacks reading the tags from the other servers (see below), and it keeps going forever, even though no more requests are being made by grafana whatsoever. It seems that there is an endless loop somehow, like if local=1 was not respected, or something along those lines. Then indeed, the web servers stops being able to even respond to metrics/find requests.
However, if in Grafana, we set the graphite version to <1.1, no tags/autoComplete/tags queries are made, and everything works wonderfully: auto-completion is snappy and graphs are showing perfectly.
Furthermore, when curling a single time the tags/autoComplete/tags API endpoint (see below), the logs go bonkers yet again, and basically never stops hard trying. According to the logs below, it'd seem like, indeed, local=1 (as well as the msgpack`) parameters are missing from the queries, which would create an infinite loop?
curl 'localhost:8080/tags/autoComplete/tags?tagPrefix=%2A&limit=100'

2018-02-26,07:54:21.246 :: RemoteFinder[graphite-cache-3:8080] Error response 500 from http://graphite-cache-3.graphite-cache.dev.svc.cluster.local:8080/tags/autoComplete/tags?tagPrefix=%2A&limit=100
NoneType: None
2018-02-26,07:54:21.405 :: RemoteFinder[graphite-cache-3:8080] Error response 500 from http://graphite-cache-3:8080/tags/autoComplete/tags?tagPrefix=%2A&limit=100
NoneType: None
2018-02-26,07:54:21.421 :: RemoteFinder[graphite-cache-2:8080] Error response 500 from http://graphite-cache:8080/tags/autoComplete/tags?tagPrefix=%2A&limit=100
NoneType: None

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/opt/graphite/webapp/graphite/finders/remote.py", line 260, in request
    preload_content=False)
  File "/usr/local/lib/python3.6/site-packages/urllib3/request.py", line 66, in request
    **urlopen_kw)
  File "/usr/local/lib/python3.6/site-packages/urllib3/request.py", line 87, in request_encode_url
    return self.urlopen(method, url, **extra_kw)
  File "/usr/local/lib/python3.6/site-packages/urllib3/poolmanager.py", line 321, in urlopen
    response = conn.urlopen(method, u.request_uri, **kw)
  File "/usr/local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 668, in urlopen
    **response_kw)
  File "/usr/local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 668, in urlopen
    **response_kw)
  File "/usr/local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 668, in urlopen
    **response_kw)
  File "/usr/local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 639, in urlopen
    _stacktrace=sys.exc_info()[2])
  File "/usr/local/lib/python3.6/site-packages/urllib3/util/retry.py", line 388, in increment
    raise MaxRetryError(_pool, url, error or ResponseError(cause))
urllib3.exceptions.MaxRetryError: HTTPConnectionPool(host='graphite-cache-3', port=8080): Max retries exceeded with url: /tags/autoComplete/tags?tagPrefix=%2A&limit=100 (Caused by ReadTimeoutError("HTTPConnectionPool(host='graphite-cache-3.graphite-cache.dev.svc.cluster.local', port=8080): Read timed out. (read timeout=10)",))

==> /var/log/info.log <==
2018-02-26,07:41:43.888 :: Timed out after 10.000133s for tags for [] *
2018-02-26,07:41:44.004 :: Timed out after 10.000131s for tags for [] *
2018-02-26,07:41:44.309 :: Timed out after 10.000152s for tags for []
2018-02-26,07:41:51.993 :: Timed out after 10.000126s for tags for [] *
2018-02-26,07:41:52.070 :: Timed out after 10.000104s for tags for [] *
2018-02-26,07:41:52.078 :: Timed out after 10.000345s for tags for [] *
2018-02-26,07:41:52.140 :: Timed out after 10.000111s for tags for []
2018-02-26,07:41:52.141 :: Timed out after 10.000168s for tags for []
2018-02-26,07:41:52.215 :: Timed out after 10.000107s for tags for []

local_settings.py
CLUSTER_SERVERS=[<the three non-local carbon caches>?format=msgpack&local=1]
CARBONLINK_HOSTS=['<IP_ADDRESS>:7002']

FETCH_TIMEOUT=10
FIND_TIMEOUT=10

TAGDB='graphite.tags.redis.RedisTagDB'
TAGDB_REDIS_HOST='graphite-redis'

carbon-cache.conf
[cache]
PICKLE_RECEIVER_INTERFACE = <IP_ADDRESS>
PICKLE_RECEIVER_PORT = 2004
USE_INSECURE_UNPICKLER = False

CACHE_QUERY_INTERFACE = <IP_ADDRESS>
CACHE_QUERY_PORT = 7002

DATABASE = whisper
ENABLE_LOGROTATION = False
MAX_CACHE_SIZE = inf
MAX_UPDATES_PER_SECOND = 1000
MAX_CREATES_PER_MINUTE = inf
MIN_TIMESTAMP_RESOLUTION = 1

USE_FLOW_CONTROL = True
LOG_UPDATES = False
LOG_CREATES = False
LOG_CACHE_HITS = False
LOG_CACHE_QUEUE_SORTS = False
CACHE_WRITE_STRATEGY = sorted
WHISPER_AUTOFLUSH = False
WHISPER_FALLOCATE_CREATE = True

GRAPHITE_URL = http://<IP_ADDRESS>:8080

The changes in #2244 seem like they should solve the problem you have currently where each host is trying to forward the tag requests to the others in an infinite loop.
An alternative approach if you're using a single redis tagdb for all 4 instances would be to specify noTags=1 in your CLUSTER_HOST definitions, which will cause the graphite-web host that initially receives the query to resolve the tags and pass the resolved names to each of the non-local graphite-web instances.


typo in android blueprint

Hi,
There's a typo in the android blueprint... It ends with the letter u on line 172.

fixed, thank you


Document how to run tests

I can see tox and coverage and a great suite of tests - would it be possible to document how to run the tests locally?
If I get some guidance here I'd be happy to make a PR myself for it. Also, I'd like to add the coverage report to the pipeline, and update contribution guidelines for checking tests are passing still.

https://github.com/torchbox/wagtailmedia#how-to-run-tests should give you an idea on running the tests.
https://github.com/wagtail/wagtail-localize/blob/main/.github/workflows/test.yml#L105 or https://github.com/torchbox/wagtailmedia/blob/main/.github/workflows/test.yml#L70 for coverage in CI

I couldn't get django-admin test or coverage working locally but pytest does work. It's mentioned briefly here: https://github.com/torchbox/tbxforms/blob/main/CONTRIBUTING.md#publishing It might be better to add that to the readme instead.


[TEAM REVIEW] WV-456 Politician List Page: Add loading notice

Uses candidate_list.html as guide to add loading banner each time form is submitted to notify the user that a GET request has been triggered
PR Breakdown
-includes banner from admin_tools
-adds an id to search input button
-adds displayLoadingBanner function to add banner
-alters submit functions to fire displayLoadingBanner
-creates function to handle firing displayLoadingBanner for search input button onclick

This looks great @jedwardmook, thank you! 👍


Only works with applications specified if the registry key

Great script, but currently only works with applications registered in the specific registry key.
If we were able to specify a full path for apps requiring them and then modify the code to test if a full path is provided, then this would increase flexibility - (scrappily coded) eg:
"KeePass" = "C:\Program Files (x86)\KeePass Password Safe 2\KeePass.exe"
Then modify the code between line 111 and 119 (inclusive)to:
        if ($testFullPath) {
            if (test-path $_.value) {
                $target=$_.value
                #LogAndConsole "Full path specified and exists for $($_.key)"
            }
        }
        if (-not ($testFullPath -or $target))
        {
            try { $apppath = Get-ItemPropertyValue $reg_path -Name "Path" -ErrorAction SilentlyContinue } catch {}
            if ($apppath -ne $null)
            {
                $target = $apppath + "\" + $_.Value
            }
            else
            {
                try { $target = Get-ItemPropertyValue $reg_path -Name "(default)" -ErrorAction SilentlyContinue } catch {}
            }
        }

This could allow flexibility to do so with minimal change, albeit this does not allow for custom file locations (or potential 32 bit/64 bit variations).
Not saying this is appropriate in every environment or situation but looks like it will work in ours

One other approach - could use the application compatibility telemetry under "HK_User:[userGuid]\Software\Microsoft\Windows NT\CurrentVersion\AppCompatFlags\Compatibility Assistant\Store"
Add a function:
    #Get application Compatibility Info per user
    New-PSDrive HKU Registry HKEY_USERS|out-null
    $applicationPaths=@{}
    $userKeys=Get-ChildItem -Path "HKU:"
    $userKeys|foreach {
        try {
            $appCompat=Get-Item -Path "HKU:\$($_.pschildname)\Software\Microsoft\Windows NT\CurrentVersion\AppCompatFlags\Compatibility Assistant\Store" -ErrorAction SilentlyContinue
            if ($appCompat) {
                foreach($appPath in $appCompat.property) {
                    $AppExe=split-path $apppath -leaf
                    if ($appexe) {
                        $applicationpaths[$appexe]=$apppath
                    }
                }
            }

        } catch {
        }

    }
    remove-psdrive hku
    return $applicationPaths
}```

Then, under line 119 add:

```       if (-not $target) {
            $target=$applicationPaths[$_.value]
        }```

I agree that it would be nice for the script to somehow automatically find programs that are not in the registry location being searched.  If they can't be automatically found, it would be nice to have a way to edit the program list with the path name so that can be searched if the program is not found in the registry location.
For example, the AppV Client application C:\Program Files (x86)\Microsoft Application Virtualization\Client UI\AppVClientUX.exe does not seem to have a reference in the registry location being searched.
For now, I'm customizing the MS script with an extra programs list and supporting function based on HarmVeenstra's solution to the ( https://github.com/HarmVeenstra/Powershellisfun/blob/main/ASRmageddon Create Common Shortcuts Start Menu/Create_Common_Shortcuts.ps1).  It would be nice if the general support was directly in the MS Script (even we have to manually add to the programs list in script) so everyone has this support (and it has MS code/testing and avoid any mistakes I might make when trying to integrate).


Flame 1.0.0-rc8 Tap regions off

First of all, thank you for working on this game framework, it's really awesome!
I am trying out 1.0.0-rc8 with Flutter 2.0.3 (but I was having the issue with my game right before Flutter 2.0 released, no change in behavior pre-2.0)
The bug:
If the camera is moved, tappable items stay in the same place on the screen even though the camera moves and the object draws relative to where the camera is.  Drawing relative fine, tapping relative bad.
This is a little difficult to explain.
If foo is in position (100, 100) and the camera is at the origin (0, 0) the tap regions are normal, you can tap on your object and it registers.
If foo is in position (100, 100) and the camera is at position(200, 200), you can still tap around physical (100, 100) on your phone and trigger the tap, if you tap on the object you see (translated from the camera move), no tap is registered.
I made an unlisted YouTube video showing this behavior:
https://youtu.be/n3L1GG3wmvo
A similar behavior happens when the anchor point is not in the center.  The video shows this as well.
Source code to play with
import 'package:flame/components.dart';
import 'package:flame/game.dart';
import 'package:flutter/material.dart';

void main() {
  runApp(App());
}

class App extends StatelessWidget {
  final GameManager gameManager = GameManager();
  @override
  Widget build(BuildContext context) {
    return GameWidget(game: gameManager);
  }
}

class GameManager extends BaseGame with HasTapableComponents {

  Tile tapMeTile = Tile(Color(0xFF0000FF));
  Tile cameraFollowTile = Tile(Color(0xFF800080));

  @override
  Future<void> onLoad() {
    tapMeTile.position = Vector2(200, 200);
    cameraFollowTile.position = Vector2(100, 100);
    add(tapMeTile);
    add(cameraFollowTile);
    // camera.follow = cameraFollowTile;
    return super.onLoad();
  }
}

class Tile extends PositionComponent with Tapable {

  bool toggle = false;
  Color untappedColor;

  Tile(Color untappedColor) {
    size = Vector2(50, 50);
    this.untappedColor = untappedColor;
    // anchor = Anchor.center;
  }

  @override
  bool onTapDown(TapDownDetails details) {
    toggle = !toggle;
    return true;
  }

  @override
  void render(Canvas canvas) {
    Rect rect = Rect.fromCircle(
      center: position.toOffset(),
      radius: size.x/2,
    );
    canvas.drawRect(rect, paint());
    super.render(canvas);
  }

  Paint paint() {
    Color color = toggle ? Color(0xFF00FF00) : untappedColor;
    return Paint()..color = color;
  }
}

pubspec.yml
name: flame_test_ontap
description: A new Flutter project.
publish_to: 'none'
version: 1.0.0+1

environment:
  sdk: ">=2.7.0 <3.0.0"

dependencies:
  flutter:
    sdk: flutter
  flame: 1.0.0-rc8

dev_dependencies:
  flutter_test:
    sdk: flutter

flutter:
  uses-material-design: true

Hi @nateonguitar , thanks for reporting this!
This was actually a known side effect of the interaction of camera/viewport + tapables. I proposed a list of solutions on the original PR, you can take a look there it describes what exactly the issue is and how to hack a solution (using the provided transform functions).
We landed on implementing option 4, which involves redefining all event classes on our side, but I haven't gotten to it yet. Hopefully I will have something ready soon! In the meanwhile, sorry for the inconvenience.

Oh dude! I'm excited to see the outcome!
I just skimmed through the other thread, is the anchor position with the tap regions also a known issue?

Oh dude! I'm excited to see the outcome!
I just skimmed through the other thread, is the anchor position with the tap regions also a known issue?

Is that a new bug for rc8 too? The tap regions and anchors should have been working in rc7 at least. I will have a look later!

Watching the video, there is definitely a bug with anchor as well if you can take a look @spydon

I just tested with 1.0.0-rc8 and yes, both the camera move offset and anchor offsets have the same results as my rc7 test.

I have found the issue with the anchors, I will hopefully have a PR up later today.

Fixed on #710

Awesome, thanks for working in this!

Hi guys, first of all, thanks for working on this, I'm excited to see where Flame is headed for v1.0 release!
@erickzanardo @luanpotter @spydon
I think this issue might have been closed prematurely.  Here's a video of the issue still happening.  Before making the video I used rc8 and the issue was still there, but I know you guys have done a lot of work since then.  When I point at main in github the issue still happens.
The code is identical to what I posted earlier except instead of camera.follow = obj I used camera.followComponent(obj) and I changed the line in pubspec.yaml to use github instead using
  # flame: 1.0.0-rc8

  flame:
    git:
      url: https://github.com/flame-engine/flame.git
      path: packages/flame
      ref: main

https://youtu.be/SxAjdFY_EcU

Also, I'd like to mention, I have no pre-1.0 experience with Flame.  I saw that version 1.0 is being worked on so I didn't really care to learn pre-1.0, so if I'm doing anything wrong please let me know, I posted the code above.  If this can be fixed with some documentation update that's a lot easier than fixing it when it does actually work if I used it correctly.

Hey @nateonguitar, you are correct this was closed too early since only one of the the problems were solved (the anchor one), and I don't think we are tracking the new events system (which this depends on) in an issue, right @luanpotter?

I am unsure if this is really a problem, we have planned the new event system to be implemented, but at the moment the methods on the camera can be used to get coordinates into the game world right?

@erickzanardo I assume the users would think that for example tapable should continue working even if they move the camera?
It's not something the common user would think that they would have to re-implement themselves just because they move the camera I think, and therefore classed as bug?

You are right @spydon, I totally forgot about tapbles and draggables

Hi guys, just trying our rc9 which says:

I just tried out the tap regions for having an anchor not centered and it's the same issue.  I can tap off to the bottom right of the square and it triggers the tap event.  The left and top quadrants don't trigger the tap event as expected when using the default anchor point.
I'm bringing it up again because this is the default anchor position.

Hmm, that is odd, I thought I added tests for this after I fixed it.
Just to clarify, this is without moving the camera right?

It feels like this example should be broken if it wasn't fixed in rc9:
https://flame-engine.github.io/flame/#/Controls_Tapables :thinking:

Ah, I see what the problem is here.
It looks like your render function is off:
  @override
  void render(Canvas canvas) {
    Rect rect = Rect.fromCircle(
      center: position.toOffset(),
      radius: size.x/2,
    );
    canvas.drawRect(rect, paint());
    super.render(canvas);
  }

position will be the anchor position in there, not the center which you expect. Try this instead:
  @override
  void render(Canvas canvas) {
    canvas.drawRect(toRect(), paint());
    super.render(canvas);
  }

@spydon That fixed it!


Add an ability to give order the paths in Grabbit Configs

There should be some way of ordering paths in a grabbit config.
This could be manual/user driven (something like order:1, order:2 for each path in the config) or automated / programmatically calculated in some way (by looking at the paths in question and figuring out which paths should go in first)

This is to address the sequencing issue caused by concurrency and the issue described in GH-92.

A potential solution pasted from https://github.com/TWCable/grabbit/pull/144:
1.) We need to create a ranking for paths. Here are some example paths for demonstration:
/foo - 1
/foo/bar - 2
/foo/bar/zoo - 3
/foo/car/do - 1
Start each path out with a ranking of 1, then for each path in sorted order based on path depth, increment the path's ranking by it's parent's ranking if it is found within the visited set; otherwise keep a ranking of 1.
This makes sure paths that can be processed independently are within the same rank group.
After that, we need to make sure Spring Batch will feed the jobs from our job queue, into the thread pool in order, based on ranking. To do this, we would need a Spring ThreadPoolTaskExecutor instance with a ThreadPoolExecutor that is built with a BlockingPriorityQueue implemented with a Comparator that compares our jobs based on path ranking. Long winded, I know..
Still even then, this would only make sure that jobs are feed into the pool in priority order, but it wouldn't prevent certain sized thread pools from picking up and starting work on higher ranked jobs before lower ranked jobs complete. To solve this, we would probably need some sort of system within our validator task that polls the job queue to see if lower ranked jobs are still in flight, and if so, put the thread of the higher ranked job to sleep for some period.

Had a discussion with @jbornemann regarding this, quoting him from the discussion

Any solution will be too complicated compared to the value it would give us. At the very minimum, what we have now (checking to see if path already has an existing parent before running the path's job - and raising a configuration error) I think is acceptable.

Closed the PR i created couple of weeks back. Should this ticket be closed as well, or we keep it in To-Do list for a later date.
@jdigger @jazeren1 @sagarsane @viveksachdeva

Thanks @Shashi-Bhushan .. Yes I think we should keep this open for now.


Table does not render properly when using colspan and relatively large data

The table renders fine when using small strings in the cells. But, when the data in the cells are large...

When the table starts, only the header is rendered even though there is space in the page for the first row.
The bigger problem is that the table cells are not rendered properly.

Please execute the following code to see the issue.
public class EasytableTest2 {

	private static final String FILE_NAME = "excelLike.pdf";
	private static final float PADDING = 50f;

	public static void main(String[] args) throws IOException {

		createAndSaveDocumentWithTables(new PDDocument(), FILE_NAME, createRegularTable(false),
				createRegularTable(true));
	}

	private static void createAndSaveDocumentWithTables(PDDocument document, String outputFileName, Table... tables)
			throws IOException {

		final PDPage page = new PDPage(PDRectangle.A4);
		document.addPage(page);
		float startY = page.getMediaBox().getHeight() - PADDING;

		try (final PDPageContentStream contentStream = new PDPageContentStream(document, page)) {
			for (final Table table : tables) {
				TableDrawer.builder()//
						.page(page)//
						.contentStream(contentStream)//
						.table(table)//
						.startX(PADDING)//
						.startY(startY)//
						.endY(PADDING)//
						.build()//
						.draw(() -> document, () -> new PDPage(PDRectangle.A4), PADDING);
				startY -= (table.getHeight() + PADDING);
			}
		}
		document.save(outputFileName);
		document.close();
	}

	private static Table createRegularTable(boolean large) {

		return Table.builder().addColumnsOfWidth(150, 150, 150).borderWidth(0.25f)//
				.addRow(Row.builder()//
						.add(TextCell.builder().text("head 1").backgroundColor(Color.LIGHT_GRAY).build())//
						.add(TextCell.builder().text("head 2").backgroundColor(Color.LIGHT_GRAY).build())//
						.add(TextCell.builder().text("head 3").backgroundColor(Color.LIGHT_GRAY).build())//
						.build())//
				.addRow(Row.builder()//
						.add(TextCell.builder().text("row span 4").rowSpan(4).build())//
						.add(TextCell.builder().text("row span 2 1").rowSpan(2).build())//
						.add(TextCell.builder().text(dummyData("data 1", large)).build())//
						.build())//
				.addRow(Row.builder()//
						.add(TextCell.builder().text(dummyData("data 2", large)).build()).build())//
				.addRow(Row.builder()//
						.add(TextCell.builder().rowSpan(2).text("row span 2 2").build())//
						.add(TextCell.builder().text(dummyData("data 3", large)).build())//
						.build())//
				.addRow(Row.builder()//
						.add(TextCell.builder().text(dummyData("data 4", large)).build())//
						.build())//
				.build();
	}

	private static String dummyData(String str, boolean gen) {
		if (!gen) {
			return str;
		}
		StringBuilder sb = new StringBuilder(str);
		for (int i = 0; i < 50; i++) {
			sb.append(str);
		}
		return sb.toString();
	}
}

Hi @pradhan-v,
Indeed: that doesn't look very nice. The problem here is that the first row is too big: since the cells on the right are increasing the size of the cell on the left (the one with rowspan) and since the first row is now too big to be drawn on one page it doesn't render correctly.
The question is: what would be a correct rendering? We would need to somehow split the cell on the left over the page border. But there is no concept of cell splitting in easytable.
The "fix" will most probably be that an exception will be thrown in such cases as is already if a single cell is too big.
Anyway: You could work around by using regular cells on the left side that have no borders in between i.e. setting border width to 0 for bottom/top borders.
Hope this helps,
Stefan

Hi @vandeseer
Throwing an exception also would leave the rowSpan feature unusable :(. May be draw the cell with the height set to the table height (endY?). Carry over the remaining height to the next page ?
Setting the top/bottom border widths to 0 does the trick. But, it leaves "holes" at the bottom of the table when the "row spanned" cells span across multiple pages. This is okay, for now.
Thanks.
Pradhan.


Ignore warning for onnx tests

Closes #158.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #167   +/-   ##
=======================================
  Coverage   93.49%   93.49%
=======================================
  Files          24       24
  Lines        1538     1538
  Branches      285      285
=======================================
  Hits         1438     1438
  Misses         60       60
  Partials       40       40

Flag
Coverage Δ

#unittests
93.49% <ø> (ø)

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 7474f78...b09be30. Read the comment docs.


[v3] useQueries : No type inference with two query fns of different return value types.

Describe the bug
When using useQueries with two queries of different return values, its return type defaults to  QueryObserverResult<unknown, unknown>[] .
To Reproduce
Check the function useQueriesTypeIssue in  https://codesandbox.io/s/restless-dust-ifoc9?file=/src/App.tsx .
Expected behavior
The return type of useQueries should be based on the type of the query functions passed in arguments.
Desktop (please complete the following information):

OS: Windows 10
Browser: Chrome, Firefox
Version: Last

Additional context
Was asked to file a bug after a discussion in the official Discord.

I also got no type inference when mapping over an array and using the same queryFn, so I think it's not specific to two different query functions.

This is a known drawback with the current types approach, but I plan on revisiting it post v3 launch.


Usage management and access control - Technical documentation update

Please revise chapter. Divide it to Functionality (description of what is delivered in 1st prototype) and Future work (everything else). Also Further implementation hints and Technologies should be revised and only what we are using in the first prototype should remain here.

This chapter is currently not updated.

Chapter was left as before. Declaration: "No implementations are yet an integrated part of the SWR delivery" was added.


[Bug Report] Cascader Cascader Selector Options Data Source Children displays an exception when there is an item of the same name (label is the same, value is different)

Element UI version
2.10.1
OS/Browsers version
windows10
Vue version
2.6.10
Reproduction Link
https://codepen.io/alwaysbe/pen/EBrapQ
Steps to reproduce
选中[四川/成都市/市辖区]，后表单拿到数据无误，但是选择框中回显的实际为[山东省 / 济南市 / 市辖区]
What is Expected?
选中[四川/成都市/市辖区]，后表单拿到数据为[四川/成都市/市辖区]，并且选择框回显为[四川/成都市/市辖区]
What is actually happening?
择框中回显的为[山东省 / 济南市 / 市辖区]

<el-cascader v-model="form.regions" placeholder="试试搜索：地址" :options="regions" filterable :props="{value:'label',expandTrigger: 'hover'}"></el-cascader>

props.value值必须唯一

in fact,the selected value [Sichuan/Chengdu/Municipal District] and [Shandong Province/Jinan City/Municipal District],only the third part "Municipal District" is the same

https://github.com/ElemeFE/element/blob/dev/packages/cascader-panel/src/store.js#L55
cascader match last leve node only,  and i don‘t think is necessary to handle props.value to be same .

there may some case use cascader to select cities, and the city list may have many Municipal District, in fact its belong to different province, and we need to select the province city detail, so the problems out

It's may be a feat which has some way to get the node data, not only value, and not to handle same value.
So you should create a new issues about this feature.

你好，这个问题我最近也遇到过了，现在有解决这个问题吗。

Fixed in #15935


enhance: streaming HashJoin should maintain degree only for opposite of outer side(s)

Currently, our streaming HashJoin always maintains degree, which is a integer column to record the number of joined rows from the other side. This is useful for outer join to emit the NULL rows. For example,
A left join B on A.bid = B.id

For this case, whenever a B event comes, the Join operator needs to

If B event is delete (-)

Check whether more than 1 AB events has been emitted yet, if so, an + NULL event need to be emitted after this AB row being deleted

If B event is insert (+)

Check whether any AB events has been emitted yet, if not, an - NULL event need to be emitted before this AB row being inserted

As we can see, actually degree only matters for outer joins. Specifically,

Inner join: none
Outer join: the inner side
Full outer join: both sides
Anti/Semi Join: the inner side

(Correct me if wrong)
I think this optimization should be done before we split degree into a separated state.

Related: #3254


fix int set

sorry for bothering, I finally run pixie on local, hope this is the final fix for that.

Thank you for the PR!


Content-Type is overridden by json object

I have a Axios interceptor to change the data of a POST request:
export default function ({ $axios }) {
  $axios.interceptors.request.use((req) => {
    console.log('req: ', req)
    req.data = {
      data: {
        type: 'user',
        attributes: {
          vendor: 'Facebook',
          code: '**'
        }
      }
    }
    return req
  })

  $axios.setHeader('Accept', 'application/vnd.api+json', [
    'post'
  ])
  $axios.setHeader('Content-Type', 'application/vnd.api+json', [
    'post'
  ])
}

But using this req.data changes to content-type to application/json;charset=UTF-8 causing the back end to return a 415 (Unsupported Media Type).
What's the correct way of formatting a json-api object?

Isn't this a question for the Axios project?

Isn't this a question for the Axios project?

I think the issue is that:
    req.data = {
      data: {
        type: 'user',
        attributes: {
          vendor: 'Facebook',
          code: '**'
        }
      }
    }
    ````

is an json object, forcing the content-type to change. So I'm wondering if there's a correct way to format or change this data object so the content-type header doesn't get changed.

I think the issue is that:
    req.data = {
      data: {
        type: 'user',
        attributes: {
          vendor: 'Facebook',
          code: '**'
        }
      }
    }
    ````

is an json object, forcing the content-type to change. So I'm wondering if there's a correct way to format or change this data object so the content-type header doesn't get changed.

That still sounds like a question about how Axios should be used.
Looking at what our front-end developer is doing, it looks like Axios get / post / etc methods take a configuration object as an additional argument (2nd arg for get, 3rd for post, patch), and that should allow you to specify headers:
resp = axios.post('/link/goes/here', jsonapi_payload, {
    headers: { 'Content-Type': 'application/vnd.api+json' }
});

I'm not an Axios expert, but it looks like something like that should work.

That still sounds like a question about how Axios should be used.
Looking at what our front-end developer is doing, it looks like Axios get / post / etc methods take a configuration object as an additional argument (2nd arg for get, 3rd for post, patch), and that should allow you to specify headers:
resp = axios.post('/link/goes/here', jsonapi_payload, {
    headers: { 'Content-Type': 'application/vnd.api+json' }
});

I'm not an Axios expert, but it looks like something like that should work.

Looks like you're correct and this is an Axios issue: https://github.com/axios/axios/issues/340. My apologies.

Looks like you're correct and this is an Axios issue: https://github.com/axios/axios/issues/340. My apologies.

No worries, @alucardu. Looks like that Axios issue you found has a decent resolution so I'm going to close this now.

No worries, @alucardu. Looks like that Axios issue you found has a decent resolution so I'm going to close this now.


Pallet Smart Contract: Canceling a contract with insufficient funds can result in error

What
The resources used since the last cycle will be billed when canceling a contract and the locked balance will also be distributed. This happens via the function _cancel_contract in lib.rs. First the state is set to deleted. Then the function bill_contract is being called which will call handle_lock. This function will extend the locked balance when it is in the create state. If it is in the deleted state (aka canceled) it will handle the distribution of the reserved balance and remove the lock.
There is an issue when canceling a contract if the user doesn't have enough funds. The first place where it goes wrong is in the function handle_grace which is being called right before handle_lock. handle_grace will start the grace period if the available balance is not enough, which it shouldn't do when the current state is the deleted state. The second place is in handle_lock itself. If the state is deleted and if the available balance is not enough the function _distribute_cultivation_rewards will throw an error.
Acceptance criteria

[ ] Both problems mentioned above should be fixed
[ ] One or more tests should be added covering this specific case

Verified on devnet.
after deploying a vm and then leaving the account with almost no balance 0.012 TFT

i was able to delete contract 6858 without any errors.


Backbone.js - Add the "replace" option to "router.navigate()"

See Backbone documentation for the option here.

@morrisallison thanks mate!


Removed deprecated symbols that have a new name

Exported symbols marked as "Renamed to $SYMBOL_NAME. Will be removed in v8" have been removed. The entire project was also searched and all references to deleted entities were updated.
BREAKING CHANGE: removed deprecated symbols that have a new name
Related issue (if exists): #6367

I can’t figure out why “CI / Node 14 build” is crashing. I installed the same version of node locally but the tests pass.
@benlesh, maybe this is some kind of anomaly?

Not sure... It might have something to do with the transpiled output using a feature that doesn't exist in Node 14, and being somehow different than what is happening locally. Different TS version locally maybe?

@benlesh I restarted and it's gone. It looks like it's some kind of floating bug in old node or tests in the repository.

Thank you, @demensky


Wayland support using 'wlr-copy' instead of xclip?

Currently not usable with Wayland clipboard

So this is my ghetto quick implementation during a work call for wl-clipboard:
#[cfg(target_os = "linux")]
pub fn dump_image_to_clipboard(image: &DynamicImage) -> anyhow::Result<()> {
    use std::{process::Stdio, io::Write};

    let mut temp = tempfile::NamedTempFile::new()?;
    println!("{}", temp.path().to_str().unwrap());
    image.write_to(&mut temp, ImageOutputFormat::Png)?;
    let mut cmd = Command::new("wl-copy")
        .args([
            "-t",
            "image/png",
        ])
        .stdin(Stdio::piped())
        .spawn()
        .map_err(|e| format_err!("Failed to copy image to clipboard: {}", e))?;

    // NOTE: We get to do all this reading and writing from the files because wl-copy only accepts
    // files on STDIN
    let mut stdin = cmd.stdin.take().expect("Failed to open stdin");
    let temp_file_path = temp.path().to_str().unwrap();
    let file_content = std::fs::read(temp_file_path).expect(&format!("Unable to open {}!", temp_file_path));
    stdin.write_all(&file_content[..]).expect("Unable to write stdin!");
    Ok(())
}

I've replaced the original function that had x-clip. This works for me just fine, but I've, as a side effect, nuked xclip support. Not ideal.

Nice.
I was planning to add wl-copy as a fallback if xclip fails or vise versa.

I'm going to get a PR to you hopefully this evening from my branch here. What I think the best way to do this is attempt to call both xclip & wl-copy and if one succeeds then don't boil an error up.
If both fail then send the error up the stack.
This avoids the situation on some Wayland systems in which xclip & wl-copy are both installed and only one is called by us, potentially not populating their clipboard correctly.

That would be awesome. 👍🏾

Seems to work:

Can someone validate against my branch for xclip?

I've uninstalled both wl-clipboard & xclip and it would appear the errors are working as anticipated.
With only one or the other installed the image is getting copied to my clipboard. The branch appears to work just fine for me, but additional verification would be good. I'll open a PR and verification can happen there.


build prod command not working with 9.0.0-rc.1

🐞 Bug report
Command (mark with an x)

- [ ] new
- [x] build
- [ ] serve
- [ ] test
- [ ] e2e
- [ ] generate
- [ ] add
- [ ] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Is this a regression?
Yes, the previous version in which this bug was not present was: ....
Description
After updating to --next with ng update @angular/core and @angular/cli I tried to run build prod
yarn ng build --project=x --prod
🔬 Minimal Reproduction

🔥 Exception or Error

[error] TypeError: context.getProjectMetadata is not a function
    at Object.configureI18nBuild (C:\node_modules\@angular-devkit\build-angular\src\utils\i18n-options.js:70:36)

🌍 Your Environment

Angular CLI: 9.0.0-rc.1
Node: 12.3.1
OS: win32 x64
Angular: 9.0.0-rc.1
... animations, cli, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router, service-worker

Package                            Version
------------------------------------------------------------
@angular-devkit/architect          0.803.3
@angular-devkit/build-angular      0.900.0-rc.1
@angular-devkit/build-optimizer    0.900.0-rc.1
@angular-devkit/build-webpack      0.900.0-rc.1
@angular-devkit/core               9.0.0-rc.1
@angular-devkit/schematics         8.3.3
@angular/cdk                       9.0.0-rc.0
@angular/flex-layout               8.0.0-beta.26
@angular/material                  9.0.0-rc.0
@angular/material-moment-adapter   9.0.0-rc.0
@angular/pwa                       0.801.3
@ngtools/webpack                   9.0.0-rc.1
@schematics/angular                8.1.3
@schematics/update                 0.900.0-rc.1
rxjs                               6.5.3
typescript                         3.6.4
webpack                            4.41.2

Anything else relevant?
ng serve is working but there I got an different error after migration
we don't use angular i18n

The architect and schematic packages appear to be incorrect and outdated.
Are these present directly in the project?  These should generally not be present within the project’s package.json.

There is no direct entry in package.json

Can you please provide the output of:
npm ls @angular-devkit/architect

and also share you package.json?
 Are you using it for ng build? It might not support version 9 yet.

Yes it's related to this, when I replace it with:  "builder": "@angular-devkit/build-angular:browser" build is running

Hi there, version 9 is not officially supported yet, but as far as I know there are no breaking changes in Architect v9, are they @alan-agius4 @clydin ? We ran a few sanity checks and<EMAIL_ADDRESS>worked with Angular 9 just well.
In any case I created an issue for tracking https://github.com/just-jeb/angular-builders/issues/624.


SCB-993 print warning log when aggregate conflict

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Format the pull request title like [SCB-XXX] Fixes bug in ApproximateQuantiles, where you replace SCB-XXX with the appropriate JIRA issue.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Run go build go test go fmt go vet to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage decreased (-0.06%) to 62.708% when pulling b5689678aa8c1d01b59e1332de643754d3065ee9 on little-cui:bug into ea31a1a3c9fdbd242817157f61e4aad15bfa5f0f on apache:master.

Codecov Report

Merging #481 into master will decrease coverage by <.01%.
The diff coverage is 41.17%.

@@            Coverage Diff             @@
##           master     #481      +/-   ##
==========================================
- Coverage   60.32%   60.31%   -0.01%
==========================================
  Files         159      159
  Lines       13529    13534       +5
==========================================
+ Hits         8161     8163       +2
- Misses       4789     4790       +1
- Partials      579      581       +2

Impacted Files
Coverage Δ

server/plugin/pkg/discovery/indexer_cache.go
100% <ø> (ø)
:arrow_up:

server/core/key_convertor.go
91.81% <0%> (-1.71%)
:arrow_down:

server/service/event/service_event_handler.go
12.5% <0%> (ø)
:arrow_up:

server/admin/service.go
66.66% <0%> (-5.08%)
:arrow_down:

server/plugin/pkg/discovery/types.go
100% <100%> (ø)
:arrow_up:

server/plugin/pkg/discovery/etcd/cacher_kv.go
67.48% <100%> (ø)
:arrow_up:

server/service/instance.go
67.61% <100%> (-0.07%)
:arrow_down:

pkg/etcdsync/mutex.go
61.29% <0%> (-5.38%)
:arrow_down:

pkg/util/tree.go
81.25% <0%> (-3.13%)
:arrow_down:

pkg/tlsutil/tlsutil.go
73.58% <0%> (-0.95%)
:arrow_down:

... and 3 more

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 ea31a1a...3cb34ae. Read the comment docs.


Dev cookies banner

What does this PR fix/introduce?
// Add a brief description or bullet points about the changes introduced.
// Include an issue number that is associated with this PR.
Added Cookies Banner as requested
Reviewers
// Tag whoever needs to review. If you're not sure, leave blank.
@Adrian-Wrona @andrzej-casper

FYI, I will review this PR today.

Build is working now :tada: 2 minor concerns to resolve left.


[web] Support custom url strategies

Expose a Dart API allowing apps to customize the url strategy.
Companion PR: https://github.com/flutter/engine/pull/19134
Fixes https://github.com/flutter/flutter/issues/33245

@yjbanov @hterkelsen @chunhtai this PR is ready for review.

@yjbanov this is ready for another pass. Main things I did:

Tried to make it null safe but found out the tooling doesn't allow it yet.
Update docs, rename methods, etc.


[FirebaseAuth]: Unhandled Exception: type 'List<Object?>' is not a subtype of type 'PigeonUserInfo' in type cast

Is there an existing issue for this?

[X] I have searched the existing issues.

Which plugins are affected?
Auth
Which platforms are affected?
iOS
Description
i get this error when i click on home to got to the simulator home and put the app in the background
[ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: type 'List<Object?>' is not a subtype of type 'PigeonUserInfo' in type cast
#0      PigeonUserDetails.decode (package:firebase_auth_platform_interface/src/pigeon/messages.pigeon.dart:401:28)
#1      _FirebaseAuthUserHostApiCodec.readValueOfType (package:firebase_auth_platform_interface/src/pigeon/messages.pigeon.dart:1573:34)
#2      StandardMessageCodec.readValue (package:flutter/src/services/message_codecs.dart:477:12)
#3      StandardMessageCodec.readValueOfType (package:flutter/src/services/message_codecs.dart:522:23)
#4      _FirebaseAuthUserHostApiCodec.readValueOfType (package:firebase_auth_platform_interface/src/pigeon/messages.pigeon.dart:1581:22)
#5      StandardMessageCodec.readValue (package:flutter/src/services/message_codecs.dart:477:12)
#6      StandardMessageCodec.decodeMessage (package:flutter/src/services/message_codecs.dart:341:28)
#7      BasicMessageChannel.send (package:flutter/src/services/platform_channel.<…>

these
Reproducing the issue
on the latest ios version in the simulator click the home button while running the app
Firebase Core version
3.2.0
Flutter Version
Channel stable, 3.22.2
Relevant Log Output
[ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: type 'List<Object?>' is not a subtype of type 'PigeonUserInfo' in type cast
#0      PigeonUserDetails.decode (package:firebase_auth_platform_interface/src/pigeon/messages.pigeon.dart:401:28)
#1      _FirebaseAuthUserHostApiCodec.readValueOfType (package:firebase_auth_platform_interface/src/pigeon/messages.pigeon.dart:1573:34)
#2      StandardMessageCodec.readValue (package:flutter/src/services/message_codecs.dart:477:12)
#3      StandardMessageCodec.readValueOfType (package:flutter/src/services/message_codecs.dart:522:23)
#4      _FirebaseAuthUserHostApiCodec.readValueOfType (package:firebase_auth_platform_interface/src/pigeon/messages.pigeon.dart:1581:22)
#5      StandardMessageCodec.readValue (package:flutter/src/services/message_codecs.dart:477:12)
#6      StandardMessageCodec.decodeMessage (package:flutter/src/services/message_codecs.dart:341:28)
#7      BasicMessageChannel.send (package:flutter/src/services/platform_channel.<…>

Flutter dependencies
dependencies:
firebase_messaging: ^15.0.3
firebase_core: ^3.2.0
firebase_auth: ^5.1.2
cloud_firestore: ^5.1.0
firebase_database: ^11.0.3
firebase_storage: ^12.1.1
cloud_functions: ^5.0.3
firebase_analytics: ^11.2.0

Hello @EgHubs I cannot reproduce this issue on my end, can you provide a full sample app?

Hi @EgHubs is there any update on this issue?

Can you try running flutter pub upgrade firebase_auth and re-run the application. It solved the issue on my end.
Checking what upgrade does reveals that it updates the dependencies of the specified package.

I'm having the same issue, no suggestions from this thread or other sources worked so far.
try {
      final googleAccount = await _googleSignIn?.signIn();
      final googleAuth = await googleAccount?.authentication;
      final googleCredential = GoogleAuthProvider.credential(
        accessToken: googleAuth?.accessToken,
        idToken: googleAuth?.idToken,
      );
      await FirebaseAuth.instance.signInWithCredential(googleCredential);
    } catch (error, st) {
      //
    }

And I see it's printing
I/flutter (31925): error caught: type 'List<Object?>' is not a subtype of type 'PigeonUserDetails?' in type cast #0      PigeonUserCredential.decode (package:firebase_auth_platform_interface/src/pigeon/messages.pigeon.dart:432:23)
I/flutter (31925): #1      _FirebaseAuthHostApiCodec.readValueOfType (package:firebase_auth_platform_interface/src/pigeon/messages.pigeon.dart:830:37)
I/flutter (31925): #2      StandardMessageCodec.readValue (package:flutter/src/services/message_codecs.dart:477:12)
I/flutter (31925): #3      StandardMessageCodec.readValueOfType (package:flutter/src/services/message_codecs.dart:522:23)
I/flutter (31925): #4      _FirebaseAuthHostApiCodec.readValueOfType (package:firebase_auth_platform_interface/src/pigeon/messages.pigeon.dart:840:22)
I/flutter (31925): #5      StandardMessageCodec.readValue (package:flutter/src/services/message_codecs.dart:477:12)
I/flutter (31925): #6      StandardMessageCodec.decodeMessage (package:flutter/src/services/message_codecs.dart:341:28)
I/flutter (31925): #7      BasicMessageChannel.send (package:flutter/src/services/platform_channel.dart:218:18)

Versions:
  firebase_core: ^3.3.0
  firebase_auth: ^5.1.4
  google_sign_in: ^6.2.1
  cloud_firestore: ^5.2.1
  firebase_crashlytics: ^4.0.4
  firebase_analytics: ^11.2.1


Thanks for the bug report!
Now that the problem has been reproduced, I'll fix it 👍

Fixed in v0.2.0


add statechange for owned-fraction in rgb21 invoice data

Background:
Now there are Amount, and Ownedfraction in the RGB invoice module, that is
pub struct Amount( u64); and pub struct OwnedFraction(u64);.  We can perform a state change operation for the amount type when we want to build a transfer transition, and there is also the check_add method for the amount type.
Both the StateChange and check_add for the amount type can perform algorithm operation(+, -).
Then for the rgb21 invoice, there is Ownedfraction which is u64 similar to Amount, and now if we want to build a transfer operation for the rgb21 contract, we just push all of them(owned fraction, token index) into the data state. So I think maybe there is also a state change and a check_add method requirement for the Ownedfraction type, and then the user can add an owned fraction when they want to transfer. Finally, they can push (changed owned fraction, token index) into a data state.
Description:
impl StateChange for Ownedfraction

The PR makes sense for me. @zoedberg @crisdut can you confirm the same so we can merge it before the next release?


Type error problem after updating expo version from 36 to 38

I updated the expo version to 38. When I was 36, the app worked normally, but an error occurs in the image. I tried the npm intall and other methods, but they were not solved. How can I solve this problem?

my app file.zip

Hi @HoWonJang for Expo related issues, please file an issue at their repo https://github.com/expo/expo. Thanks!


Closed ClientSession on class deletion

Hi, thanks for the contribution, but unfortunately, we've decided against trying to maintain multiple options for session instantiation, and have removed this option.


Added sink with emitter, so that it can be used to index, update and …

Need some refactoring and cleanup on the emitter. It is more aligned with elasticsearch sink connector to allow index, update and delete documents from/to the index. I'll add more safety mechanisms and test cases later.
With this we can also extend this to flink's table APIs

I'll add more samples for update and delete requests. I think it will be complete when we add support to dynamic sink to support SQL.


Create settings possibility

set commands, set percentage for stuff, set anything, can do that in connection with the musicbot app? maybe small window in there? but gotta check that you cant just change settings for other streamers...
maybe make settings commands that only work when u are streamer?

look at twitch authorization possibility


Expose device path types

This fix allows to expose the DeviceType and DeviceSubType enumerations.
I'm also thinking about moving the device_path modules outside loader_image since, on the UEFI specification, those are two separated protocols. What are your thoughts on this?

I'm also thinking about moving the device_path modules outside loader_image since, on the UEFI specification, those are two separated protocols.

Yep, makes sense. Initially it was only used internally in the loaded_image protocol, but it might be useful in other cases as well.

@GabrielMajeri I think it's ready for you to take a look 🤗


Updating README.md file

Updates
I updated the readme file. Please review and accept

Thoughts
I probably shouldn't, but okay. 👍


compile error in eos-dev docker

Hello, my contract is compile well out of docker container. And I try to deploy in eos-dev container, and failed:
$ docker exec -it eosio /bin/bash -c 'eosiocpp -o xxx.contracts/xxx.wast xxx.contracts/xxx.cpp && eosiocpp -g xxx.contracts/xxx.abi xxx.contracts/xxx.cpp'
xxx.contracts/xxx.cpp:7:10: fatal error: 'eosio.token/eosio.token.hpp' file not found
#include <eosio.token/eosio.token.hpp>
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 error generated.
error Command failed with exit code 1.

How can I fix the head file include error? Thanks

I think eosio.token/eosio.token.hpp is not in the default include path. You can copy eosio.token/eosio.token.hpp into your source code folder and then use #include "eosio.token.hpp" in your cpp file.

Consider transitioning to eosio.cdt, which is available as pre-built binary packages.  eosiocpp is deprecated and will be removed in a future release.  Otherwise you may follow Kayan's advice, or add the eosio.token include path to the command line with -I.


Error reported uploading code coverage data

https://ci.appveyor.com/project/PowerShell/powershell-f975h/build/6.0.0-beta.7-425#L3623
Resolve-Path : Cannot find path 'C:\projects\powershell-f975h\Microsoft (R) Build Engine version 15.3.409.57025 for .NET Core' because it does not exist.
At C:\Program Files\AppVeyor\BuildAgent\Modules\build-worker-api\build-worker-api.psm1:207 char:18
+     $fullPath = (Resolve-Path $Path).Path
+                  ~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (C:\projects\pow...5 for .NET Core:String) [Resolve-Path], ItemNotFoundException
    + FullyQualifiedErrorId : PathNotFound,Microsoft.PowerShell.Commands.ResolvePathCommand

Push-AppveyorArtifactInternal : Cannot bind argument to parameter 'FullPath' because it is null.
At C:\Program Files\AppVeyor\BuildAgent\Modules\build-worker-api\build-worker-api.psm1:209 char:42
+     Push-AppveyorArtifactInternal -FullPath $fullPath -FileName $File ...
+                                             ~~~~~~~~~
    + CategoryInfo          : InvalidData: (:) [Push-AppveyorArtifactInternal], ParameterBindingValidationException
    + FullyQualifiedErrorId : ParameterArgumentValidationErrorNullNotAllowed,Appveyor.BuildAgent.Api.Utils.PushAppveyorArtifactInternalCmdlet

Resolve-Path : Cannot find path 'C:\projects\powershell-f975h\Copyright (C) Microsoft Corporation. All rights reserved.' because it does not exist.
At C:\Program Files\AppVeyor\BuildAgent\Modules\build-worker-api\build-worker-api.psm1:207 char:18
+     $fullPath = (Resolve-Path $Path).Path
+                  ~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (C:\projects\pow...ights reserved.:String) [Resolve-Path], ItemNotFoundException
    + FullyQualifiedErrorId : PathNotFound,Microsoft.PowerShell.Commands.ResolvePathCommand

Push-AppveyorArtifactInternal : Cannot bind argument to parameter 'FullPath' because it is null.
At C:\Program Files\AppVeyor\BuildAgent\Modules\build-worker-api\build-worker-api.psm1:209 char:42
+     Push-AppveyorArtifactInternal -FullPath $fullPath -FileName $File ...
+                                             ~~~~~~~~~
    + CategoryInfo          : InvalidData: (:) [Push-AppveyorArtifactInternal], ParameterBindingValidationException
    + FullyQualifiedErrorId : ParameterArgumentValidationErrorNullNotAllowed,Appveyor.BuildAgent.Api.Utils.PushAppveyorArtifactInternalCmdlet

I think this issue has been resolved by #4957


Fixing compilation issue

It fixes compilation issue , as well as need to passing previous service obj while disabling session affinity
#247

/approve
/lgtm


Removed incorrect logical definiton for fungal sorus development (GO:…

…0099121) and added logical definition to sorocarp development (GO:0048837)

Looks good


Clicking on Hangfire menu item yields error

I was clicking around in the application, just random clicks and entering a few products etc.
When clicking on the HANGFIRE menu item, I got an error - please see attached screenshot. Unfortunately it seems to be random. I have been trying to create the steps necessary to reproduce but have not succeeded.

Thanks, Luke.

Hi, this is an issue which occurs when Hangfire schema is not created by default. Probably for some reason, the schema creation was skipped. I will try as well if i could reproduce this issue. Thanks


Min Color Temp Ignored

I have several integrations that have been running without issues for years. For some reason the min temp color temp is now ignored on some of the integrations. Min color has been set to 2400 but for some reasons the lights are set to 2000. Other integrations don't have this issue and adhere to the min/max set.
Config is made from the UI.
I believe the issue appeared when updating to core_2023.11.0 but not certain. Tried rolling back to core_2023.10.5 with no change.
No changes have been made to lights or adaptive light config.
home-assistant.log

Do you experience this bug without adaptive-lighting?

Do you experience this bug without adaptive-lighting?

The bug is only related to adaptive-lighting so not really sure really what you mean.
Issue was fixed by deleting the entry and adding a new with duplicate settings.

Duplicating the entry will now make sure that the lights don't go outside the restraints set. The lights will set to 2400 kelvin and not 2000. However, they don't adjust up. Right now this entry is stuck at 2403 kelvin where all other entries in the house are at 3378.
Reloading the entry will update the lights.


Why deprecated?

Hi there! I think your project is very promising, especially now with the ability to selfhost chat-ui instances, but I noticed that it is marked as deprecated. Can you please let me know why? Thanks!

Hello,
HuggingChat used to be open to visitors without authentication, and this repository does not (yet) support the authentication layer that has been added, so I marked it as deprecated to save people some time!
However, I may dig into their authentication flow to update this code. In the waiting, I'm open to PRs 😉
Greetings

@secondtruth indeed
@maximerenou i have trouble finding another alternative to your solution. Could you help point me in the right direction please?


chore: add inline style for typing indicator

Description
Added inline style "rcbDot" for typing indicator in ChatBotBody.tsx.
Closes issue #105
What change does this PR introduce?
Please select the relevant option(s).

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update (changes to docs/code comments)
[x] Non breaking change (adding code that doesn't add additional functionality)

What is the proposed approach?
Styles from "rcbDot" have been taken from issue #91.
Checklist:

[x] The commit message follows our adopted guidelines
[x] Testing has been done for the change(s) added (for bug fixes/features)
[x] Relevant comments/docs have been added/updated (for bug fixes/features)

I pushed the changes. Let me know if everything is okay.


How to search in all fields?

Is it possible to search in all fields when using the SEARCH HTTP method?
To search in a given field, I send a query like this one: {"_query": {"fieldname": "search terms"}}.
To search in all fields, I tried this: {"_query": "search terms"}. But the query returned no documents.

Yes!! You can search in all fields, but you have to add "_index": "terms" which means you can search adding the field or without the field.
The document must be something like this:
{
      "fieldname1": {
            "_type": "keyword",
            "_value": "value1",
            "_index": "terms"
      }
}

And the search should work like you tried!
By default the value of _index is field_terms which force use field in the search

Hi,
I try added documents specifying the indexing mode like above with "_index": "terms" but it doesn't work. I can still find the document using ?q=fieldname1:hello but not using ?q=hello.
I don't understand why.
I've checked the index and "_index": "terms" is there:
curl http://xapiand:8880/document/?pretty
{
  "_settings": {
    "number_of_shards": 5,
    "number_of_replicas": 1
  },
  "_schema": {
    "_id": {
      "_type": "integer",
      "_slot": 0,
      "_prefix": "Q"
    },
    "text": {
      "_index": "all",
      "_prefix": "text.",
      "_ngram": false,
      "_cjk_ngram": false,
      "_cjk_words": false,
      "_slot": 989045663,
      "_type": "text"
    },
    "_version": {
      "_prefix": "V",
      "_slot": 1,
      "_type": "positive"
    },
    "_type": "foreign/object",
    "_foreign": ".xapiand/indices/document"
  }
}

Hi!! Indeed! this is a problem! without the field we try to guess the type but we have no way of differentiating between text type and keyword type.  So it's being guessed as a keyword. But I think is fixed by 4b44a70. Thanks for reporting the issue!!

It looks it works. Thanks!


All messages in a room always returned by messages.list() method

When I run the messages.list() method it returns all messages in a room even with a max argument.
Sample code
from ciscosparkapi import CiscoSparkAPI

api = CiscoSparkAPI()

messages = api.messages.list(roomId="ROOMID", max=10)

Expected: 10 messages returned
Actual: every message in the room is returned.

@further-reading : Per the API spec the max parameter specifies the maximum number of items per page ("page size").
Since ciscosparkapi transparently handles paging for you… It respectfully uses your requested page size, and still returns the all of the results - requesting additional pages as needed until all results have been returned. 😁
If you only want a certain number of items returned, you can use the Python standard library's itertools.islice() function to get just what you need:
from itertools import islice

from ciscosparkapi import CiscoSparkAPI

api = CiscoSparkAPI()

messages = api.messages.list(roomId='<your-room-id>')

# Only want the first 50 messages?
for message in islice(messages, 50):
    print(message)

# Want them in a list?
short_list = list(islice(messages, 50))

I have considered adding a convenience slicing ability to the package's GeneratorContainers, which would effectively do the very same thing only with the simple syntax of messages[:50].

Thanks for the explanation! :)
Just to clarify, say I do something like this in a room with 1000 messages:
from itertools import islice
from ciscosparkapi import CiscoSparkAPI

api = CiscoSparkAPI()

messages = api.messages.list(roomId='<your-room-id>', max=10)

#get 11
for message in islice(messages, 11):
    print(message)

Which best describes what the ciscosparkapi package does:

Detects there are additional pages on the initial response, immediately goes through all available pages until all 1000 messages have been fetched.
Fetches the first page worth of messages only. When the loop reaches the 11th message it will then fetch the 2nd page.

In other words, does ciscosparkapi fetch all messages immediately or will it wait until the additional pages are needed first?

I just tested the REST endpoint and it works as expected. So it's something in the CiscoSparkAPI that is not working today.
`import requests
def send_request():
try:
response = requests.get(
url="https://api.ciscospark.com/v1/messages",
params={
"roomId": "<my_room_id>",
"max": "2",
},
headers={
"Authorization": "",
},
)
print('Response HTTP Status Code: {status_code}'.format(
status_code=response.status_code))
print('Response HTTP Response Body: {content}'.format(
content=response.content))
except requests.exceptions.RequestException:
print('HTTP Request failed')`

import requests

def send_request():
    try:
        response = requests.get(
            url="https://api.ciscospark.com/v1/messages",
            params={
                "roomId": "<my_room_id>",
                "max": "2",
            },
            headers={
                "Authorization": "<Bearer my_token>",
            },
        )
        print('Response HTTP Status Code: {status_code}'.format(
            status_code=response.status_code))
        print('Response HTTP Response Body: {content}'.format(
            content=response.content))
    except requests.exceptions.RequestException:
        print('HTTP Request failed')

Based on what @cmlccie said, despite what the api docs say the max parameter is actually setting page size not the maximum returned results. So when requests is used it returns the first page, but CiscoSparkAPI will see that there are more pages and is capable of walking over them to get everything.
I did some testing earlier to confirm. If you don't set a max the most requests will return 99, which is the default page length.

Right, but using the requests module and hitting the endpoint directly, if I put in a max=1 I get one item, equal to one message. If I put in 5, I get 5 items, etc. If I put in arbitrary numbers above 100 I get all of those as well (so, 143 or 229). If I don't use max at all I get 49 in the room I'm testing.
Keeping in mind this is not with the Python wrapper API, this is just directly with the code above.

To clarify where the Cisco docs are confusing:
From list messages

Limit the maximum number of messages in the response.

But pagination clarifies:

When requesting a list of resources the max query parameter may be used to control the number of items returned per page.

So like I said, when you use the requests module you are actually getting just the first page of results as the response. The wrapper though will also see that there are additional pages available and is capable of requesting those too. In other words, all the max argument does in the wrapper is change how many requests it'd need to make to get everything. If there are 1000 messages and you set max at 1000 the wrapper makes one request, if you set max at 500 it makes two requests.

The response from Cisco will have a link to the next page in the header when your result set is paginated.    You will need to recursively make api calls to the next url found in the header to get the additional results.

On Dec 31, 2017 at 7:03 PM,  <Roy Healy (mailto:notifications@github.com)>  wrote:
To clarify where the Cisco docs are confusing:
From  list messages (https://developer.ciscospark.com/endpoint-messages-get.html)

Limit the maximum number of messages in the response.

But  pagination (https://developer.ciscospark.com/pagination.html)  clarifies:

When requesting a list of resources the max query parameter may be used to control the number of items returned per page.

So like I said, when you use the requests module you are actually getting just the first page of results as the response. The wrapper though will also see that there are additional pages available and is capable of requesting those too. In other words, all the max argument does in the wrapper is change how many requests it'd need to make to get everything. If there are 1000 messages and you set max at 1000 the wrapper makes one request, if you set max at 500 it makes two requests.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly,   view it on GitHub (https://github.com/CiscoDevNet/ciscosparkapi/issues/50#issuecomment-354629102), or   mute the thread (https://github.com/notifications/unsubscribe-auth/ARoKxucJHoOk9ciKhCnUGq4l8af-mtZGks5tGCC3gaJpZM4RCovt).

So if I'm reading those docs correctly, when I set "max=1" I am getting one result per page, and since I'm not then asking for any additional pages, I'm getting just my one item. I see the link returned for next, so I could ask for more pages, but I'm not (using my example code above). So my main question I guess is how come the wrapper is automagically returning all additional pages? It may be respecting my request for one result per page, but it is then giving me all pages. I don't see a way in the wrapper to limit page results in the message.list method.
So (again, if I'm understanding this correctly) the wrapper is helpfully giving me all pages available, grabbing the next=rel link and doing it using my max=1 parameter. So if I want an efficient way to get only one item, I should use the direct endpoint and skip the wrapper in this one case, otherwise the wrapper is going to give me everything and then I have to slice and dice to get just my one, which would be very inefficient with large result sets.
Is that an accurate summary, or am I missing something?

A deeper read of the documentation for the wrapper seems to confirm the above... from: http://ciscosparkapi.readthedocs.io/en/latest/user/api.html#messages

This method supports Cisco Spark’s implementation of RFC5988 Web Linking to provide pagination support. It returns a generator container that incrementally yields all messages returned by the query. The generator will automatically request additional ‘pages’ of responses from Spark as needed until all responses have been returned.

This only became an issue for me because of one particular room having 18,000 messages. The speed difference in requesting 100 messages only, vs. getting all 18,000 and then parsing to only give the user 100, is a giant speed difference on the order of seconds vs. minutes.

That's accurate, yeah.
The dev suggested some code earlier to grab a specific number of messages from the output of the api call, however I'm not sure about its efficiency. I asked him to clarify at which point the wrapper will request the next pages.


Add FavIcon functionality

Currently, the schema and core implementation are missing the favicon functionality. My understanding is that SharePoint master pages contain a SharePoint:SPShortcutIcon tag that renders out the default SharePoint icon /_layouts/images/favicon.ico. The icon file is stored in the SharePoint file hive, and there is no way to change it in a controlled manner other than modifying of the master page or implementing script-based workarounds. Modifying of master page is not the recommended approach for SharePoint Online and there is no CSOM or REST API to replace the hard-coded URL with an URL to an icon file that is relative to a site. It would be nice if you could influence SharePoint architects to either change the default location of the favicon in the out-of-the-box master pages to a location on the site, like in the {masterpagecatog} for example, or to add favicon functionality to SharePoint APIs so that we would be able to specify an alternative icon in the provisioning template schema and implement in the sp-sites-core (e.g. FavIcon attribute on the WebSettings element).

Yet another SharePoint pain that kills productive time

Hi,
Thanks for your feedback. Please write this request in the https://github.com/SharePoint/sp-dev-docs/issues repo or in UserVoice. As such, I will close this item. Thanks.


Background images for training

Hi! Thanks for releasing the code!
I'm trying to reproduce the training and thus need to gather all the training and validation backgrounds. Following your description, I used the lsun repo to download and extract the backgrounds. However, now I'm struggling with 1) selecting the "correct" background images and 2) converting them to the right format and to the right location.
The provided script data/copy_lsun_images_to_train_files_dir.py doesn't work for me, since I guess the directory structure isn't correct after extracting the images. E.g. "bedroom_train_lmdb" extracts images to e.g. ./f/8/8/1/2/2/*.webp which isn't compatible with your script. Your script also only looks for .jpg files. Furthermore, I don't know how to select the mentioned 397582 training backgrounds, since e.g. "bedroom_train_lmdb" alone has over 3mio images. Would be grateful for any help!

Hey!
That's odd, IIRC the script used to work with the dataset as extracted. I will take a look this weekend and get back to you.

Hey, any update?

Hello, may I ask if there is any progress？

Hey is there any update?


Why does BasicDemo have many errors???

Hello guys,
I really want check BasicDemo
But I have got errors.  Because I already installed BulletSharp, OpenTK and SharpDX from Nuget Server and It can't resolve if I have error like concflicts:
BasicDemo.cs:
Matrix4 modelLookAt = body.MotionState.WorldTransform * lookat;
Because it shows error Matrix4 and Matrix
Physic.cs:
World.Gravity = new Vector3(0, -10, 0);
error...
Vector3 localInertia = colShape.CalculateLocalInertia(mass);
Conflict
and more...
How do I fix?

You don't have to install Nuget packages manually. All you should have to do is compile BulletSharp.sln in Release mode and then build Demos.sln.
The Nuget package is a different version of BulletSharp: https://github.com/AndresTraks/BulletSharpPInvoke
It is similar, but not compatible with the demos here.
The solutions are a bit badly organized. It's mostly because you couldn't put C++/CLI and C# projects into the same solution in early versions of Visual Studio Express. I'm thinking of ways to improve this.

Okay I will try thanks! PS: Nice idea OpenTK with BulletSharp = It is really good like Unity 3D. Nice job!


regression pgm.sql no longer works with arrays

This used to work on the v3 branch, currently, it always quotes the variable which obviously breaks this.
And using pgm.func results in an undefined;
exports.up = (pgm) => {
    pgm.sql('INSERT INTO states (id) VALUES {values};', {
        values: ['queued', 'busy', 'completed', 'failed'].map(e => `('${e}')`).join(', \n')
    });
};

Suggestion: make pgm.sql aware of PgLiteral, its not the most sugar-coated but it is fairly simple and allows for special cases, which is exactly what this is.

FYI: the rather crappy workaround:
exports.up = (pgm) => {
    const values = ['queued', 'busy', 'completed', 'failed'].map(e => `('${e}')`).join(', \n');

    pgm.sql(`INSERT INTO states (id) VALUES ${values};`);
};

I tried it in v2 and v3 it didn't work in either of them. It added apostrophes around ${values}. 🤷

well it did work for me, I have tests that wouldn't pass if the values weren't in the table :man_shrugging:
But thank you anyway!


How to bind csvserver and lbvserver

I want to bind a csvserver and a lbvserver, but there is no csvserver_lbvserver_binding resource...
bind cs vserver sample_cs -lbvserver sample_lb

or (as I would expect it):
resource "netscaler_csvserver_lbvserver_binding" "sample_cs_sample_lb" {
  csvserver = "${netscaler_csvserver.foo.name}"
  lbvserver = "${netscaler_lbvserver.foo.name}"
}

I think I miss most of the bindings. Any reason those are not provided?

The way we support binding lbvservers and csvservers is through cspolicies.
You have to define a policy with the csverserver and the targetlbvserver.
For a detailed example on how to do this see here.
The command bind cs vserver sample_cs -lbvserver sample_lb is used to set the default lb vserver that will be selected when all bound cspolicies have failed to match the request.
We do not support this functionality currently.
The way this could be implemented is by adding a default_lb_vserver option in netscaler_csvserver and then do the binding in the code of the netscaler_csvserver resource.
A workaround to the missing default lb vserver would be to use a policy with a catch all rule and low priority.
Something like this would work, provided it has the lowest priority in the cspolicies bound to the csvserver.
resource "netscaler_csvserver" "csvserver1" {
  name        = "csvserver1"
  ipv46       = "<IP_ADDRESS>"
  port        = "80"
  servicetype = "HTTP"
}

resource "netscaler_lbvserver" "lbvserver1" {
  name = "lbvserver1"
  ipv46       = "<IP_ADDRESS>"
  port        = "80"
  servicetype = "HTTP"

}

resource "netscaler_cspolicy" "default_policy" {
  targetlbvserver = netscaler_lbvserver.lbvserver1.name
  csvserver  = netscaler_csvserver.csvserver1.name
  policyname = "defaultpolicy"
  priority   = 99
  rule       = "HTTP.REQ.IS_VALID"
}

I see; we don't seem to use cspolicies. What would be the equivalent of a TCP (non-HTTP) catch-all rule?

I think this expresssion would match all ipv4 TCP traffic on the client side
CLIENT.IP.PROTOCOL.EQ(TCP)


install questions

when I run helm install --wait --timeout 10m tobs timescale/tobs
Error: no Secret with the name "tobs-certificate" found
helm list
NAME	NAMESPACE	REVISION	UPDATED                                	STATUS	CHART      	APP VERSION
tobs	default  	1       	2022-06-10 14:51:24.457333106 +0800 CST	failed	tobs-0.10.1	0.10.1
the helm release status is failed,when I deal with same pod not running, not something to continu..

NAME                                                        READY   STATUS      RESTARTS   AGE
alertmanager-tobs-kube-prometheus-alertmanager-0            2/2     Running     0          18m
opentelemetry-operator-controller-manager-7467bdc8b-rjg9p   2/2     Running     0          18m
prometheus-tobs-kube-prometheus-prometheus-0                2/2     Running     0          18m
tobs-connection-secret-mdwqp                                0/1     Completed   0          18m
tobs-grafana-6894864595-7844f                               3/3     Running     3          18m
tobs-grafana-db-j7rcc                                       0/1     Completed   2          18m
tobs-kube-prometheus-operator-5874b95798-55cl4              1/1     Running     0          18m
tobs-kube-state-metrics-56c568fdcc-8mdzq                    1/1     Running     0          18m
tobs-prometheus-node-exporter-fqc4j                         1/1     Running     0          18m
tobs-promlens-7f778cc958-vzdpw                              1/1     Running     0          18m
tobs-promscale-648d58d4f5-kgl52                             1/1     Running     3          18m
tobs-timescaledb-0                                          1/1     Running     0          18m

not opentelemetrycollectors  cr , how can i to do?
when I delete helm release tobs, the pod tobs-opentelemetry-collector-5f5889956d-8nbb2 is creating....

We are looking into issue with no Secret with the name "tobs-certificate" found during installation.
As for:

when I delete helm release tobs, the pod tobs-opentelemetry-collector-5f5889956d-8nbb2 is creating....

We are tracking this issue in https://github.com/timescale/tobs/issues/365

@marsty339 could you check if this is still the case when using tobs 0.11.1?

@marsty339 could you check if this is still the case when using tobs 0.11.1?

helm install --wait --timeout=10m tobs timescale/tobs
Error: secrets "tobs-pgbackrest" already exists
kubectl  get po
NAME                                                         READY   STATUS      RESTARTS      AGE
alertmanager-tobs-kube-prometheus-alertmanager-0             2/2     Running     0             45s
alertmanager-tobs-kube-prometheus-alertmanager-1             2/2     Running     0             44s
alertmanager-tobs-kube-prometheus-alertmanager-2             2/2     Running     0             43s
opentelemetry-operator-controller-manager-7c985ff6f9-l8mnr   2/2     Running     0             55s
prometheus-tobs-kube-prometheus-prometheus-0                 2/2     Running     0             45s
prometheus-tobs-kube-prometheus-prometheus-1                 2/2     Running     0             44s
tobs-connection-secret-b9k4b                                 0/1     Completed   0             55s
tobs-grafana-7cd8df4694-qg7kz                                3/3     Running     0             55s
tobs-kube-prometheus-operator-56c9986ff4-8wx6c               1/1     Running     0             55s
tobs-kube-state-metrics-677c9b4fd4-qvf7f                     1/1     Running     0             55s
tobs-prometheus-node-exporter-6wxgr                          1/1     Running     0             55s
tobs-promscale-f74b845d8-cdknf                               1/1     Running     2 (42s ago)   55s
tobs-timescaledb-0                                           1/1     Running     0             55s
The installation still has errors，And it feels very complicated to install.
tobs 0.11.1 Requires  k8s version 1.23+,but my cluster version is 1.21.4。。。

@paulfantom
helm install --wait --timeout 10m tobs timescale/tobs
Error: secrets "tobs-certificate" already exists
kubectl  get po
NAME                                                         READY   STATUS      RESTARTS       AGE
alertmanager-tobs-kube-prometheus-alertmanager-0             2/2     Running     0              2m17s
alertmanager-tobs-kube-prometheus-alertmanager-1             2/2     Running     0              2m17s
alertmanager-tobs-kube-prometheus-alertmanager-2             2/2     Running     0              2m17s
opentelemetry-operator-controller-manager-7c985ff6f9-2hnw2   2/2     Running     0              2m31s
prometheus-tobs-kube-prometheus-prometheus-0                 2/2     Running     0              2m16s
prometheus-tobs-kube-prometheus-prometheus-1                 2/2     Running     0              2m16s
tobs-connection-secret-ng466                                 0/1     Completed   0              2m31s
tobs-grafana-5dfc6fb499-4vkk5                                3/3     Running     0              2m31s
tobs-grafana-db-9vp8h                                        0/1     Completed   2              2m31s
tobs-kube-prometheus-operator-56c9986ff4-4rz5p               1/1     Running     0              2m31s
tobs-kube-state-metrics-677c9b4fd4-ljh9g                     1/1     Running     0              2m31s
tobs-prometheus-node-exporter-fnfh9                          1/1     Running     0              2m31s
tobs-promscale-f74b845d8-jwf4w                               1/1     Running     3 (113s ago)   2m31s
tobs-timescaledb-0                                           1/1     Running     0              2m31s


Add literals holding original text to ThematicBreak and HardLineBreak

Currently it is not possible to know which text was parsed to ThematicBreak. Was it --- or was it *** ? This information is lost when parsing mark up text in this library.
In order to enable the user of the library to differentiate, it would be useful to keep the text literal in a variable of ThematicBreak. So one could do (in Kotlin)
// prints the original text that was parsed
fun print(node: Node): String {
    return when (node) {
        is ThematicBreak -> node.literal
        ...
    }
}

The same issue exists with HardLineBreak where two or more line breaks are reduced to one HardLineBreak-element. Adding a literal variable would enable to reconstruct the original text that was parsed and one could check how many line breaks the text originally had.
More generally, this feature request goes into the direction of enabling a bijection between the text and its parsed representation. I am not sure if a full bijection makes sense, but for the two node types of ThematicBreak and HardLineBreak it definitely does.

Thanks, done for ThematicBreak. Can you create a separate issue for HardLineBreak please? I'm not sure what you mean by "The same issue exists with HardLineBreak where two or more line breaks are reduced to one HardLineBreak-element".
On printing the original text that was parsed, have you seen this recently merged pull request?:

https://github.com/commonmark/commonmark-java/pull/306

(To be released in the next release soon.)

Released in 0.22.0 now: https://github.com/commonmark/commonmark-java/blob/commonmark-parent-0.22.0/CHANGELOG.md#0220---2024-03-15


Update README.md

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 47.60%. Comparing base (52b3ee7) to head (433a62c).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #192   +/-   ##
=======================================
  Coverage   47.60%   47.60%
=======================================
  Files          57       57
  Lines        4054     4054
=======================================
  Hits         1930     1930
  Misses       1948     1948
  Partials      176      176

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


variables don't bind when using lowercase in the naming

Description
When using the DBeaver @set command to set a variable, when executing the script the parameter binding is looking for the variable name in UPPER CASE.  The variables tab shows the variable created using same case as the @set command.
DBeaver Version
Version <IP_ADDRESS>301060755
Operating System
macOS 13.0.1
Database and driver
PostgreSQL
PostgreSQL JDBC Driver
Steps to reproduce

Open a new script
set a variable using @set, with name using some mix of uppper and lower case letters
use the variable in a SELECT statement and execute

Example of above
`@set userIdList = (SELECT "Id" FROM public."AspNetUsers" WHERE "UserName" IN<EMAIL_ADDRESS><EMAIL_ADDRESS>'bick2@perf.labs'))
-- title: AspNetUsers
SELECT
*
FROM
"AspNetUsers" anu
WHERE
"Id" IN (:userIdList)
ORDER BY
anu."UserName"
;`

Expected results
variables with any mix of upper and lower case letters will be recognized during script run so long as the set and the call match up
Actual results
Bind parameters during script execution is taking variable names and looking for the upper case version of the variable name.  Since variable was set using mix case the variable's value is not populated at run time and a pop up is presented for variable binding.
If the variable is name with all upper case during the @set command and called using : later in the script with mixed case the variable replacement happens successfully.
Additional context
This was working properly on DBeaver version 22.3.1 and looks to have broken in 22.3.2

Thank you for bug report.
As a temporary workaround you can set your variable value directly in the bind dialog.

@unset has the same flaw


mobile-first portfolio project

[HTML&CSS linter used
The following sections added

the toolbar
the headline section (right after the header)

Project Status: INVALIDATED :red_circle:
Hi @mhassa204 ,
While you made a great effort in this project, I can't go ahead and review your code.

Invalid Code Review Request

This is because you have submitted a project, that is missing the HTML and CSS Linters please ensure that they are installed and passing as expected to ensure that there are no linter errors in your project :wink:
Your current linter checks

Expected checks

Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers, and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me @NahnahAJ in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.

Please @mhassa204, do not make changes to the project once it is submitted for review. This can lead to inconsistent advice from the code reviews. Kindly build your project, tests it and ensure that all requirements are met to the best of your ability and submit a PR.


Double and Teselagen.bio.orf.ORF types for sequence-utils don’t exist

We get the following errors when importing from @teselagen/sequence-utils:
node_modules\@teselagen\sequence-utils\calculateTm.d.ts(1,102): error TS2304: Build:Cannot find name 'Double'.

node_modules\@teselagen\sequence-utils\findOrfsInPlasmid.d.ts(1,129): error TS2503: Build:Cannot find namespace 'Teselagen'.

The types seem to be generated from these comments:
https://github.com/TeselaGen/tg-oss/blob/master/packages/sequence-utils/src/getOrfsFromSequence.js#L11
https://github.com/TeselaGen/tg-oss/blob/master/packages/sequence-utils/src/calculateTm.js#L21
We were able to work around the error by defining global types:
declare namespace Teselagen {
   namespace bio {
       namespace orf {
           type ORF = string;
       }
   }
}

type Double = number;

@tnrich

@ryan-stackwave hmm.. not sure why those comments are in there like that. Cleaned them up to hopefully prevent this issue for you and published a new version of @teselagen/sequence-utils

Closing, reopen if necessary


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@01e96f8). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #16   +/-   ##
=======================================
  Coverage        ?   96.59%
=======================================
  Files           ?        9
  Lines           ?      176
  Branches        ?       15
=======================================
  Hits            ?      170
  Misses          ?        6
  Partials        ?        0

Continue to review full report at Codecov.

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


Modularize proto::messsage

Break up src/proto/message.rs into a tree of submodules. Also move a couple of unit tests from message into submodules, because they tested only those types' functionality.
This change is motivated by #66, which--in part--fixes a bug made possible by pub access to the inner Vec of the Unparsed newtype. Refactoring the proto::message types into modules ensures that the message parsing implementations can't directly access the inner fields of the types unless they are deliberately pub. When they were all in the same module, visibility rules didn't matter.

:+1: looks quite a bit more manageable.
Would you be able to run formatting with cargo +nightly fmt? (for ordering of imports) and add sign-offs with git rebase --signoff wiktor/fix-extension-serialize? (I think that's the base branch you're using) and push that stuff with git push --force-with-lease?
Thanks a lot :clap: The files are really smaller now... and it occurred to me that we could move quite some hex-encoded vectors to tests/messages which would further simplify the code... but that's for another PR :sweat_smile:


Protected content

Is it already possible to protect media content with your plugin in any way?
I have followed this discussion (https://github.com/strapi/strapi/issues/8623) and it seems that there is currently no way to secure media content, which means that there are many use cases, where user related content would be shared to anybody: user images, invoices, orders, audio messages, video messages, etc.

I'm also interested know how to ;)

As far as I understood, there are two modes:
{
   publicFiles: true; // false
}

And there is no way to have a granular control: it's either public or private.
How is it possible to turn on ACLs mode?

Subject to object ACLs means fine-grained, object-level access control lists (ACLs) are enabled. Objects may be public if they grant access to allUsers or allAuthenticatedUsers.  Learn more

From the documentation of this plugin:

The bucket should be created with fine grained access control, as the plugin will configure uploaded files with public read access.

Which means, some kind of preperation for custom access for documents/folders. Is it right?

I have recently started configuring this plugin for some users at my company. We are also wanting the files to remain private, however, the plugin will only try to access them via public URL as far as I can tell. Are there any plans to allow Strapi and this GCS plugin to access files in a private bucket?


Remove #![feature(core_intrinsics)] usage

This is for some reason enabled when using alloc instead of std, which prevents crates from using proptest on stable with only the alloc feature enabled.

thanks! It doesn't look like it's being used for anything, so I'm guessing this is historical left-over


Allow to change layout from URL

?layout=comfortable or ?layout=compact

Done in https://github.com/mondeja/simple-icons-website-rs/commit/1d60cef610a6d22408ba025157afd57192feffc8


chore(main): release 2.31.0

:robot: I have created a release beep boop
2.31.0 (2024-08-05)
Features

EMS-3685: added xlsx for 2 new eligibility fields (#2884) (3878733)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v2.31.0
:sunflower:


Godot 4: variables in the Visual Script's members window don't sort by their name

Godot version
v4.0.alpha3.official [256069eaf]
System information
Windows 11, Vulkan Clustered, Intel UHD Graphics 620 v<IP_ADDRESS>0
Issue description
The variables in the Visual Script's members window doesn't sort by name, as shown in the screenshot below:

This is a regression from Godot 3.4. In Godot 3.4, the variables will be sorted by name, as shown in the screenshot below:

I expect all the variables to be sorted by name. Moreover, all the functions in Godot 4 are sorted by their name, why not the variables too?
Steps to reproduce

Create some variables in Visual Script.
Look at the members' window, the variable won't be sorted by name.

Minimal reproduction project
No response

VisualScriptEditor gets the script variables using the get_variable_list [3.x] [master] method.
It seems in 3.x it was sorting the variables but in master this was changed in PR #39649 and no longer does.
The functions are retrieved using the get_function_list, this doesn't sort either but VisualScriptEditor sorts them after retrieving, so sorting the variables should probably be done the same way.


[DRAFT][TEST] test -Werror / no EXTRA_CFLAGS for assembly?

CI supposedly adds -Werror. Let's make sure.
cc: #41553

https://github.com/zephyrproject-rtos/zephyr/runs/4765994915?check_suite_focus=true is full of signed wrapping warnings but can't find the other warning. Let's try with only one revert.

I have quick local reproduction steps now, no need to test in CI: EXTRA_FLAGS is not used when compiling zephyr/tests/arch/x86/static_idt/src/test_stubs.S
git revert 0b226c91616c

west build -b qemu_x86 -d b0 tests/arch/x86/static_idt/ -- -DEXTRA_CFLAGS=-Werror; echo $?
  => 0

ninja -C build clean
ninja -C build CMakeFiles/app.dir/src/test_stubs.S.obj -v | tee _

Now search for -Werror in the ninja output: it's there when compiling offsets.c but missing when compiling test_stubs.S

CMAKE_AFLAGS works: https://github.com/zephyrproject-rtos/zephyr/runs/4769291234?check_suite_focus=true


Increase monitoring frequency to 10 minutes

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #284 (205679a) into master (d66d69a) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #284   +/-   ##
=========================================
  Coverage     58.86%   58.86%
  Complexity     1202     1202
=========================================
  Files           131      131
  Lines          5056     5056
  Branches        588      588
=========================================
  Hits           2976     2976
  Misses         1807     1807
  Partials        273      273

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 d66d69a...205679a. Read the comment docs.


added tests for video generation from system files

should close #55  but blocked by #58 to ensure tests actually run

Should be unblocked now with the merge of #78