Add a GitHub workflow to allow contributors to self-assign issues

Adding a comment as #self-assign to any issue will automatically assign that issue to the commenter. This will do-away the need for waiting on the repo-maintainers and also relieve the maintainers from some administrative chores.
An example of the solution could be found implemented here:

https://github.com/unionai-oss/unionml/pull/136

Since GitHub actions are not supported on this repo, closing this issue.


Added college name generator

Added a COLLEGETYPE list for common types of colleges. Wrote a function to randomly generate a first name and last name using faker, and a college type from the COLLEGETYPE list. Also wrote a test case for the new function.

To help ease testing of future submissions I updated the workflow settings for the project. You should see test results here automatically shortly after future changes.

When ready this closes #21

I'm going to merge this, and then start a new PR to do the clean up previously requested.


transformation between tracker and depth cameras in the mount by intel

Required Info

Camera Model
d435 and t265

Firmware Version
(intel realsense viewer 2.22.0)

Operating System & Version
{Win (10)

Platform
PC

Language
{python}

Segment
{AR}

hi,
i was looking the the mount that was designed by google for the t265 and d435 cameras and the extrinsic that were provided for that frame,
the line

origin xyz="0.009 0.021 0.027" rpy="0.000 -0.018 0.005"

says that the d435 is at xyz meters from the t265 and at rpy degrees,
is this wrt the center of the 2 cameras? if so how do i get the transformation wrt to the rgb camera on d435?
Thanks

Hi, I think you are referring to realsense-ros: https://github.com/IntelRealSense/realsense-ros/blob/occupancy-mapping/realsense2_camera/urdf/mount_t265_d435.urdf.xacro#L9
The parent and child frame are defined right in the lines before. You could use for example rviz to visualize the frames.
RPY is in radians.
To get the transformation to other sensor frames you can see a sample use of the API in this PR: https://github.com/IntelRealSense/librealsense/pull/4355, l. 40 ff..


Extra XML Generated

I've been using this extension for a while now, and I've been enjoying it a lot, it's very useful. However I am noticing one problem, sometimes when you type the ///, it will generate two sets of XML for the same method/property/etc.
Eg.
/// <summary>
///
/// </summary>
/// <summary>
///
/// </summary>
public void Method() {

}

This isn't much a huge problem, as when you hit control + Z (cmd + Z on Mac) it will undo the second one, however it is an extra keystroke, and we developers hate that, you know! (Which is like, the entire reason for this extension...)
Thanks a lot!

I noticed the same. It seems to happen when you have both this extension and the C# extension enabled at the same time.
If you have only one of the two extension enabled then you get only one set of xml comments. With both extensions enabled you get two sets of comments.
I ended up disabling this extension and keeping just the C# extension enabled, since the C# extension is a requirement for other extensions like C# Dev Kit.

I ended up disabling this extension and keeping just the C# extension enabled, since the C# extension is a requirement for other extensions like C# Dev Kit.

Fair, I would like to see this bug fixed if possible though, as I didn't even know the C# Extension has auto-xml


Action Button : Open Display target value is not used #25

Added ctrl + click

That's a great update to #25 #26, and glad to see that it's just a small code change.
Thanks!


Having two interval streams running at the same time sometimes causes them to freeze/halt

Describe the bug
I have two streams created with interval operator:

"main" which emits every 60 seconds
"secondary" which emits every 3 seconds

When they run at the same time, sometimes they "freeze" or "halt" and do not emit for some time. It happens randomly, I didn't find one rule by which it occurs.
Expected behavior
The streams created with interval operator should emit according to the time that was set at the beginning.
Reproduction code
No response
Reproduction URL
https://stackblitz.com/edit/stackblitz-starters-6jltsf?file=src%2Fmain.ts
Version
7.8.1
Environment
No response
Additional context
The code in Stackblitz measures time between each emission for two streams: main and secondary. Then compares this time to the one set in interval operator. If the time measured is 1.5x (or more) times greater than original one then report error to the console.
Below you can see two screenshots that I took. They show that it occurs randomly (sometimes there are no errors):
Here you can see that it took 127 emissions for secondary stream to exceed its original interval time (3 seconds) by around 16s and 35s seconds.

Here it happened quite quickly and for both main and secondary.

Yep, seems like it.


Display generated tags separately

Fixes
Fixes #4192 by @fcoveram
Description
This PR adds a separate display of machine-generated tags and tags provided by sources on the single result page following the Figma mockups.
I added the tooltip for generated tags, but I don't know what to write there.
I started to write the implementation plan for the frontend feature, but realized that this can be done in a single PR. As the project lead, do you think we need a frontend IP, @AetherUnbound?
Testing instructions
Run the app using just frontend/run dev (running dev:only would not update the English strings, so you would need to run dev at least once)
Go to http://localhost:8443/image/a487f4eb-ce05-43e1-acae-73c4ab090cc9 to see how machine-generated and source tags are displayed.

@obulat this looks great with the exception of an (existing) spacing issue that becomes more apparent when we add a text label above the tags.

Observe that the "Source" and "Source Tags" text labels are not in alignment. "Source" is 0.25rem to the right of "Source Tags".
I think we just need to remove horizontal padding from the dd and dt elements, and that this will fix the pages to match the mockups.

I started to write the implementation plan for the frontend feature, but realized that this can be done in a single PR. As the project lead, do you think we need a frontend IP, @AetherUnbound?

I think an IP would still be good, even just as a decision log for some of the questions/points that were made in the project proposal. It would not have to be a big document IMO!

To @zackkrida's point

Observe that the "Source" and "Source Tags" text labels are not in alignment. "Source" is 0.25rem to the right of "Source Tags".

If I recall correctly, we intentionally added this spacing to avoid cropping links' focus ring. In this case, the flickr label.
We can add the same spacing in this section to make it look align. Or perhaps, and correct me if I'm wrong, investigate how to fix this site-wise issue and stop us from adding empty spacings that could break the layout.

We can add the same spacing in this section to make it look align. Or perhaps, and correct me if I'm wrong, investigate how to fix this site-wise issue and stop us from adding empty spacings that could break the layout.

I added a negative margin to the "Source" or "Provider" (whichever comes first) to fix this. You can see in the snapshots that they are aligned now.
Now, I looked at the snapshots again, and I feel like the tags headings are fine when there are both source and generated tags. However, when there are only source tags, the "Source tags" heading is not very clear. Why are they called "source" tags? And since the "What it is?" link is only available on the Generated tags, it is not possible to navigate to the explanation page. What do you think, @fcoveram?

I think we can safely remove the "Source tags" heading when there are no machine-generated tags. The pill set in the media details area is common content in stock and other similar services and does not need further explanation.

I think we can safely remove the "Source tags" heading when there are no machine-generated tags. The pill set in the media details area is common content in stock and other similar services and does not need further explanation.

Updated in 15cb517. I also added a screen-reader-only "Tags" heading.

Currently, the link says "What is this?"; suggestions for the text are welcome.

@obulat I think the classic "Learn more" would be appropriate here.

Hey @fcoveram could you look at this again and let us know what you think? Thanks :smile:

I will take a look at this tomorrow morning 🙏 sorry for the delay


switched products loop to bulk operation query

Updated to better handle large product catalogs :)
Left in the commented out "log" tag, presuming that is there to more easily debug with merchants.

 Rebuild "Auto-tag products by age" for bulk operations

Left in the commented out "log" tag, presuming that is there to more easily debug with merchants.

100% correct. That log tag right there is firmly in the be-generous-to-my-future-self category. :D


[Distributed] Initializer and lifecycle call synthesis via SIL

Implements local initializers and properties and storing the lifecycle hook returned values in them via SIL synthesis.
Opens up the door for https://github.com/apple/swift/pull/38739/ resolve impl via SIL and storing the properties

@swift-ci please smoke test

@swift-ci please smoke test Linux Platform

replaced by more complete [Distributed] Complete remote actor resolve() implementation in SIL #38823


How to insert margin and padding to table cell

I have merged cells and then I have to set margin and padding. Is there any way to set margin (left, right, bottom, top) and padding (left, right, bottom, top) to table's cell?

There are margin properties on a cell and these are described in the API documentation. PowerPoint has no separate concept of padding.


Rasterio oneoh

Changes to make nodata compatible with rasterio 1.0

src.indexes is immutable
no rasterio.drivers context

Had to xfail the slic test which does not work anymore: tests/test_alpha_func.py::test_slic_mask_any
cc @dnomadb any idea why ^ is failing?

already included in #64


LazyColumn doesn't play nicely with Flexbox

https://github.com/cashapp/redwood/pull/746#issuecomment-1401845828 and https://github.com/cashapp/redwood/pull/746#issuecomment-1404003920 (assuming these two are due to the same issue).

Paired with @veyndan this morning and I think we have an idea of the issue. Going to work on this today.


Add hover support for resource paths

Purpose
$title
Fixes #41646
Samples

Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[x] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

How do we came up with the name Query Parameters for the function params?
When we have a rest args for the query params we show the type definition as int[].... This doesn't have any meaning. I think what we need to show is int....
When the param description is not documented it should show an empty string. Instead of that currently it shows a null.
We can have const values as the path parameters. When we hover the action statement we can see the constant as a path parameter but the description is missing even the param is documented.
eg: resource function get path/[A]/path1()

How do we came up with the name Query Parameters for the function params?
When we have a rest args for the query params we show the type definition as int[].... This doesn't have any meaning. I think what we need to show is int....
When the param description is not documented it should show an empty string. Instead of that currently it shows a null.
We can have const values as the path parameters. When we hover the action statement we can see the constant as a path parameter but the description is missing even the param is documented.
eg: resource function get path/[A]/path1()

First two have been addressed with #fde9bfb
Can you provide an example where it shows null when the description is not provided? I couldn't reproduce it.
Created a new issue for the 3rd one as the goto def feature is also not working for constants used as path parameters.

When documentation for a rest parameter is not provided (for a functional argument), it displays null instead of an empty string. Consider the following example.
client class MyClient {

    # Description.
    #
    # + param - parameter description
    resource function accessor path(string param, int... ids) {

    }
}

public function main() {
    MyClient MyClient = new;

    MyClient->/path.accessor("test", 1, 2, 3);
}

This is not specific to a resource access and occurs in function call expressions, as shown in the example below.
# Description.
#
# + param - parameter description
# + return - return value description
function name(string param, int... ids) returns string {
    return "hello";
};

public function main() {
    _ = name("hello", 1, 2, 3);
}


Get all Workspace documents

Is there a possibility to get all workspace documents?
I am working on a path completion via the language server protocol, but at the moment i can only recommend already opened text documents. Is there any notification or request I can use, or is there an easy workaround?
And in addition, opening .csv files does not trigger the didOpen Notification. Why not?
(using vscode)

Is there a possibility to get all workspace documents?

No, the server is expected to figure this out based on the rootUri in the initialize request in the beginning.

And in addition, opening .csv files does not trigger the didOpen Notification. Why not?
(using vscode)

Ask your questions related to Visual Studio Code's LSP implementation in the microsoft/vscode-languageserver-node repository.

Oh, thanks.
I don't know why i didn't think about that ...
Ok, I will do that.


[#3 Attention Performance] add attention support to  tritonintelgpu-distribute-to-warps

tt.splat, tt.broadcast, tt.reduce are used in attention, these ops need to be handled

add initial support for tt.reduce/broadcast/splat

code in poc works fine


Memory leaks with NavMeshSurface.UpdateNavMesh()

I'm updating the surface in runtime with UpdateNavMesh().
Usually the operation takes about 100ms in my setup, but sometimes there are random memory leaks that make the operation take several seconds.
The log is spammed with following warnings:

Internal: JobTempAlloc has allocations that are more than 4 frames old - this is not allowed and likely a leak
To Debug, enable the define: TLA_DEBUG_STACK_LEAK in ThreadsafeLinearAllocator.cpp. This will output the callstacks of the leaked allocations

The script in 6_modify_mesh example scene has these leaks for me as well.
Is this an issue, or I'm doing something wrong?
Here's my code (I'm using the 2018.3 branch with the latest stable Unity build): https://hastebin.com/acebemuzim.cs

I've always had this, and still have it. Would love to know if anything can be done about it.

I've been getting this issue forever as well. I couldn't pinpoint exactly where in project this was coming from since it's too big... but it looks like all the answers are pointing to this.
AFAIK, It seems that it's a bug coming from the Unity Engine. Here's a thread from the Unity Forums (just updated recently, trying looking at the latest comments).
https://forum.unity.com/threads/jobs-lags-jobtempalloc-has-allocations-that-are-more-than-4-frames-old.513124/page-2
I'm in the same boat though, would love know if there's anything we can do about it. Should we submit an official issue to Unity? Are there newer version of Unity that may have fixed this?

I'm having this issue also. Seems to be the Y-Height of the NavMeshBuilder causing the issues. My guess is they calculate the entire y height in one iteration of their voxel code.


Mouse doesnt move with native mouse set to false

Hi,
For some reason when not using native mouse mode, the mouse does not move for me at all. Like it is unable to target anything with right click, even with detections available. Any idea what could be causing this? I use a logitech mouse myself with ghub and perhaps it conflicts with the logitech dll you seem to be using, I havent been able to get it to work at all though. Thank you for your work, I did also send you a message on discord regarding this.

Will be added back in the next update.


systemd mysql-default-wait-ready does not handle inital startup of mysqld

Cookbook version
8.2.0
Chef-client version
12.18.31
Platform Details
Ubuntu 16.04
Scenario:
Create a new mysql instance fails because it seems that mysqld is doing some initialization on first startup which likely takes some time and does not open the socket before this is ready.
Because of this mysql-default-wait-ready kills it immediately.
Script output with set -x on inital startup:
+ daemon_pid=27285
+ ret=0
+ /bin/true
+ /usr/bin/mysqladmin --no-defaults --socket=/run/mysql-default/mysqld.sock --user=UNKNOWN_MYSQL_USER ping
+ RESPONSE=mysqladmin: connect to server at 'localhost' failed
error: 'Can't connect to local MySQL server through socket '/run/mysql-default/mysqld.sock' (2)'
Check that mysqld is running and that the socket: '/run/mysql-default/mysqld.sock' exists!
+ mret=1
+ [ 1 -eq 0 ]
+ [ 1 -ne 1 -a 1 -ne 11 ]
+ grep -q Access denied for user
+ echo mysqladmin: connect to server at 'localhost' failed
error: 'Can't connect to local MySQL server through socket '/run/mysql-default/mysqld.sock' (2)'
Check that mysqld is running and that the socket: '/run/mysql-default/mysqld.sock' exists!
+ /bin/kill -0 27285
+ ret=1
+ break
+ exit 1

Same initial startup with a sleep 10 at the beginning of the script:
+ daemon_pid=28842
+ sleep 10
+ ret=0
+ /bin/true
+ /usr/bin/mysqladmin --no-defaults --socket=/run/mysql-default/mysqld.sock --user=UNKNOWN_MYSQL_USER ping
+ RESPONSE=mysqladmin: connect to server at 'localhost' failed
error: 'Access denied for user 'UNKNOWN_MYSQL_USER'@'localhost' (using password: NO)'
+ mret=0
+ [ 0 -eq 0 ]
+ break
+ exit 0

Steps to Reproduce:
Simple service like this does the job :
mysql_service 'default' do
  port '3306'
  version node['mysql']['version']
  initial_root_password root_password
  action :create
end

systemctl stop mysql-default.service; rm -rf /var/lib/mysql-data/mysql-default /var/log/mysql-default /etc/mysql-default; chef-client
Expected Result:
MySQL daemon initial startup should not fail.
Actual Result:
Initial startup fails.
/usr/share/mysql/mysql-systemd-start from the Ubuntu  MySQL package seem to be a proper implementation of the same functionality.

I am unable to replicate this issue using test kitchen on my local development station with the example you've given.
/usr/share/mysql/mysql-systemd-start appears to wait approximately 30 seconds with for i in $(seq 1 30) while mysqld-wait-ready will wait indefinitely. I'm not sure how you are running into the situation where it's dying.
On initialization depending on the size of the data directory the process could take a while to get ready. Do you have an extremely large data directory?

Hi @iennae. No there is no user generated data in MySQL when this happens, the only difference from a standard setup is that the MySQL data directory is on an encrypted disk.
It is actually the first startup after instance creation that is initiated by the recipe itself when the service fails to start. The only data in the data directory is what was generated by by the recipe itself during instance creation time data directory initialization. If that first startup is done the systemd scripts in the recipe seem to work okay.
I'm not able to do additional tests with this right now unfortunately as I already deleted that node and went with another solution. I could try to reproduce this and gather additional details later if needed.
I'm not using this cookbook right now because of other issues, so if I'm the only one facing this issue or if you are unable to reproduce this please feel free to cancel/ignore this issue.

Ahh. If you do come back to this and find that you can replicate this with more information, I'd be happy to help debug. As I can't reproduce at this time, I'm going to go ahead and close this for now. Thanks so much for getting back to me.


Outdated screenshots

Hello,
I work for Everbidge under the content creation team and noted that this article has some out-of-date screenshots.
We are now moving over to our new UI, and this includes some extra options in the SSO set up page.
Here is what the new screenshot should look like:

Thank you,
Emma Charles
Technical Support Enablement Specialist
01206 443121
www.everbridge.co.uk

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

ID: 6e2d66e3-fdd2-3567-b759-54c8a2d2ed5a
Version Independent ID: a2321350-e67f-e1b2-a2a1-fb55f5c3dd51
Content: Tutorial: Azure Active Directory integration with Everbridge
Content Source: articles/active-directory/saas-apps/everbridge-tutorial.md
Service: active-directory
Sub-service: saas-app-tutorial
GitHub Login: @jeevansd
Microsoft Alias: jeedes

@EmmaC57289
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

@jeevansd  Is there any update on this request please?

@jeevansd  @JamesTran-MSFT Is there any update on this please?

@EmmaC57289
Thank you for following up on this!
Since there hasn't been any update, and if this is an urgent doc enhancement, I'd recommend following our Microsoft Docs contributor guide and creating a PR to update this doc. This way @jeevansd can just review and sign off as needed.
Thank you for your time and patience throughout this issue.

@v-hagamp Can you please get this article updated? @v-rant Please test this application again and then we can also get the App gallery template updated.

#please-close


Megatron-4913 : Redirect to login page after password reset

@thakkarnirav-duclo Closing this as we decided to create branch with name dc_22.0.1_1


WG Tunnel

Package name
com.zaneschepke.wireguardautotunnel
Hash(es)
F-droid
69:49:37:7F:86:39:D0:3E:90:AF:27:94:26:40:97:34:31:99:8F:CF:34:1E:0B:BE:87:0F:76:40:C8:7F:24:4D
Github
52:04:D8:2E:76:6E:8A:A1:4D:CB:B0:6D:C7:0A:EB:AE:2B:DD:81:2D:4D:62:03:CD:52:1A:8A:68:5D:7D:3D:80
Source code link (Optional)
https://github.com/zaneschepke/wgtunnel

As of now, contributing through issues won't be accepted. Please check the updated https://github.com/soupslurpr/AppVerifier/blob/master/CONTRIBUTING.md for instructions. Thanks.


docs: add universal concurrent builds

The current Universal Story suggests to run both the "browser" and "server" builds sequentially, this means that as applications grow, their build time will increase at a fast rate since the total time is the sum of both builds (although "server" builds are normally faster than the "browser", it's still adds a considerable time overhead).
This PR attempts to add a quick tip to run this builds in parallel. Since the CLI doesn't commit changes outside the output directory (and when it does, with the Universal Story both builds target different directories), it seems that both builds can run simultaneously (we are currently doing this on a large app with 300+ components and 15+ locales).

if you want to run them concurrently then you can simple do
"build:client-and-server-bundles": "npm run build:browser & npm run build:server"

Yes for sure, but the output on the log would get all mixed together, whereas concurrently will identify each output line for each command you are running, which is particularly useful when errors get thrown (which might be different errors for each build target depending on the setup).
Concurrently allows you to customize the log output as well if you need it to be more clear, for example:
output from npm run build:browser & npm run build:server

output from concurrently --names "BROWSER,SERVER" "npm run build:browser" "npm run build:server"

(this is how we normally run the build command)
output from concurrently --names "BROSER,SERVER" -c "bgBlue.bold,bgGreen.bold" "npm run build:browser" "npm run build:server"

There are other features that might come in handy as applications grow and build scripts get more complex.
Of course this is just my opinion, and I'm more than happy to edit to just use the & operator, but since the concurrency section is more of a "tip" than a guide, I thought it could help to suggest using a tool that is flexible.

The build system is currently being refactored and updated, this will be something that is definitely looked into once that new build system is implemented.
Thanks for putting this together, hopefully you're able to assist once the new build system is in place.


3.3 Regression: Editor rendering problem after connecting Skeleton2D property for a Polygon2D.

I am using Windows 10 and encountered this problem in Godot 3.3 stable, this does not happen in 3.2.3 stable (or any previous versions that I've used).
I noticed previously that when trying to edit a character in a project I've been working on, that I would have problems copying a Polygon2D node and adjusting the UV, in 3.3 the polygon would turn invisible. I noticed that if I just opened the same project in 3.2.3, the polygon was still visible. I couldn't reproduce it and couldn't find it mentioned here yet so thought it was an error on my side (I would think Polygon2D + Skeleton2D are still ubiquitous in 2D projects?).
But then today I thought of the extra step of attaching a Skeleton2D node to the Polygon and found that there the problem arises.
This is the same project, with the same node selected, first opened in 3.3:

Compare with it opened in 3.2.3:

I think you could have a similar effect previously when you hooked up a polygon to a Bone2D that wasn't part yet of the rest pose of a skeleton 2D ( but having it turn invisible by just connecting it to the skeleton makes it real hard to position the bones).
Minimal reproduction project (used in screenshots)
PolygonUVTester.zip

Fixed by #48647.


virt-controller: run container as non-root user

Create new virt-controller user and run the container under this user.
Part of the fix for #113

All tests pass locally.


Remove LandingZoneOldListAPIView

This seems to be a relic of a very old implementation used by BIH metabolomics. Most likely not in use anymore, but I will ask Mathias once he's not too busy.
Could also just grep access logs in production to see if the URL for this view has been recently hit..

Mathias confirmed this can indeed be removed.

Done.


[WIP] Improve GetConsensusCommand to hinder node from producing redundant blocks.

This PR is aiming at fix #2099

:x: Build AElf <IP_ADDRESS>29 failed (commit https://github.com/AElfProject/AElf/commit/4fa9b20501 by @shiwk)


added script for receipt generator

Hi, thanks for the Pull request, but can you explain how will this help in the game?

@shivangdubey  This is a basic receipt generator game this will keep on adding the items price and at last when we are done with our shopping we will press q and it will automatically generate thank you for shopping with us your total bill is whatever the amount will be at last.

@shivangdubey should i attach the screenshot?

@shivangdubey This is a basic receipt generator game this will keep on adding the items price and at last when we are done with our shopping we will press q and it will automatically generate thank you for shopping with us your total bill is whatever the amount will be at last.

That's nice, but I couldn't get how will this help in Number guessing game?

@shivangdubey This is a basic receipt generator game this will keep on adding the items price and at last when we are done with our shopping we will press q and it will automatically generate thank you for shopping with us your total bill is whatever the amount will be at last.

Got it, simple and subtle.


Fix a typo onlick -> onclick

Because of this mistake, it was necessary to call toggleControlsForm method directly from HTML. Now, it works and the href can be a simple #.

Wow, this was a toy project from 6 years ago... how did you stumble on this?
Anyway, thanks for the PR! I can't believe I thought href="javascript: was a good idea back then...

@qiemem I stumbled upon your project while compiling a list of HTML+JS fractal generators. Check out https://github.com/valera-rozuvan/bookmarks-md/blob/master/fractals.md . Yours is still accessible at https://dl.dropboxusercontent.com/u/21087540/fractalviewer.html = )

Oh, nice!
BTW, I just made a gh-page for it: https://qiemem.github.io/FractalViewer/

@qiemem Cool. Updated my link collection. https://github.com/valera-rozuvan/bookmarks-md/commit/8ac40eaef8de0a6f2a92d4926d68bd3eb6fd48bb


Move webpack dist into docker build

This is potentially controversial.
The thinking behind this is to roll a docker container with the production js/css in it, such that the buildkite/buildkite docker stuff can depend on something that is known good and working. To do this, it runs webpack in the Docker build process and puts the dist/ dir in the container and serves it with a plain 'ol http server rather than webpack-dev-server.

Won’t this change stop hot reloading from working?

@ticky yup, it's not designed for live-dev, just for "known working, instant on" frontend container. We can add a live-reloading, code mounting, webpacking on file changing container as an additional thing (was my thinking).


Fixes unknowns alert

"0% of COVID-19 vaccinations reported unknown age. The map below displays data for cases where age was unknown." was showing up even when there wasn't unknown data available and there for no map.

@benhammondmusic Now im seeing no alert at all under race/ethnicity

Show Unknown Percent Alert without MAP text and with CHART text; show MISSING DATA for map

Show Unknown Percent Alert with MAP/CHART text

Show Missing Data Alert (no race breakdowns for the state)

Show Alert (no data, state is partly suppressed for COVID)

Don't Show Missing Data or 0% Alert (survey data is complete and by design has no unknowns)


release 1.28.1

This is a minor bugfix release. It does not contain any changes to the rkt code, but it updates depencies and runtime versions for bugfixes:

vendor: update go-systemd to v15 (#3759). rkt stopped working when running in a service with systemd v234. This update fixes it.
scripts: update rkt-builder version to 1.3.0 (#3754). This updates the default Go runtime to 1.8, fixing https://github.com/rkt/rkt/issues/3738.

/cc @squeed @iaguis @kirek007

@iaguis Meeep, typos :(
Should be better now, thanks for checking.

All good. Thanks!


Module not found: Error: Can't resolve '@docusaurus/useIsomorphicLayoutEffect' on docusaurus 2.4.1

docusaurus-search-local version 0.44.5
the bad things is because of the cherrio 1.0.0 released, version 0.30.x not work now...

Try our older versions combined with adding a "resolutions" (for yarn, and "overrides" for npm) in your package.json, to use a specific version of cheerio such as 1.0.0-rc.3.
I'll dig the issue with docusaurus v2 later once I got time.

If you're using Docusaurus v2, please try our older versions combined with adding a "resolutions" (for yarn, and "overrides" for npm) in your package.json, to use a specific version of cheerio such as 1.0.0-rc.3.如果你使用的是 Docusaurus v2，请尝试使用我们的旧版本，并在你的 package.json中添加 “resolutions”（用于 yarn，以及用于 npm 的 “overrides”），以使用特定版本的 cheerio，例如 1.0.0-rc.3。
I'll dig the issue with Docusaurus v2 later once I got time.等我有时间后，我会用 Docusaurus v2 来解决这个问题。

Get it, Thank you!

I ran into this problem today. I solved it with the following changes to the packge.json:

add an override for cheerio (as a dependency of @easyops-cn/docusaurus-search-local) to use 1.0.0
downgrade @easyops-cn/docusaurus-search-local from 0.45.0 to 0.44.5

  "overrides": {
    "@easyops-cn/docusaurus-search-local": {
      "cheerio": "1.0.0"
    }
  },
  "dependencies": {
    ...
    "@easyops-cn/docusaurus-search-local": "0.44.5",
    ...
  },


feat(acs): add Communication Toc entries

These were added following, scbedd's instructions to prepare the Azure Communication Services reference documentation.
There will be one of these per language and this is the first so please feel free to let me know if there are any problems which may also be found in the other languages.
Thank you

Docs Build status updates of commit a72c71b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-mapping/reference-unified.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 9ec7011:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-ref-autogen/overview/ManagedPartner.yml
:warning:Warning
View (azure-node-latest)
Details

docs-ref-mapping/reference-unified.yml
:white_check_mark:Succeeded

docs-ref-autogen/overview/ManagedPartner.yml

[Warning-DuplicateUids] Uid(azure.nodejs.sdk.landingpage.services.managedapplications) has already been defined in docs-ref-autogen/overview/ManagedApplications.yml.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


make leaves untracked files in vendor directory

It appears that dep version 0.4.1 (e.g. installed by homebrew) exhibits this behaviour whereas the current HEAD version of dep does not.
Workaround
brew uninstall --force dep && go get -u github.com/golang/dep/cmd/dep

Steps to reproduce

brew install dep
Ensure that homebrew version is picked up. dep version should report v0.4.1.
cd to the riff directory
issue make
issue git status and observe untracked files

Not reproduced with latest dep version (a.k.a. v0.5.0).


Setup and configuration help requested

I have a suite of micro services which are currently deployed using a golang routing mechanism coupled with some proxying by an nginx setup, this is getting to be cumbersome to maintain. I would like to move to Consul + Fabio which offers dynamic proxying and hopefully easy management of my services including versioning etc. As part of this effort I am trying to do a simple Proof of Concept to help me understand the capabilities and gain the knowledge on how to use Fabio.
Backends used
Consul
Objective 1: POC
I have 3 local tomcat instances running on separate ports
Deploy Fabio as my Landing Site for the services
Expose an Endpoint: http://localhost:80  (later this would become a FQDN)
Provide  http(s) on the endpoint
Fabio container would be used as following

                                               +-----> HTTPS(S) —--> tomcat-a
                                              |
Request —> HTTP(S) --> fabio—> HTTP(S)  ——> tomcat-b
                                              |
                                               +—----> HTTP(S) -- —>tomcat-c

With some advise from Frank, I tried doing the following
curl -XPOST -H 'Content-Type: application/json' -d '{"ID": "tomcat1", "Name": "tomcat1", "Tags": ["urlprefix- /"], "Address": "<IP_ADDRESS>", "Port": 8887, "EnableTagOverride": false, "Check": {"HTTP": "http://<IP_ADDRESS>:8887/", "Interval": "15s"} }' 'http://localhost:8500/v1/agent/service/register

curl -XPOST -H 'Content-Type: application/json' -d '{"ID": "tomcat2", "Name": "tomcat2", "Tags": ["urlprefix- /"], "Address": "<IP_ADDRESS>", "Port": 8888, "EnableTagOverride": false, "Check": {"HTTP": "http://<IP_ADDRESS>:8888/", "Interval": "15s"} }' 'http://localhost:8500/v1/agent/service/register

curl -XPOST -H 'Content-Type: application/json' -d '{"ID": "tomcat3", "Name": "tomcat3", "Tags": ["urlprefix- /"], "Address": "<IP_ADDRESS>", "Port": 8889, "EnableTagOverride": false, "Check": {"HTTP": "http://<IP_ADDRESS>:8889/", "Interval": "15s"} }' 'http://localhost:8500/v1/agent/service/register

All the services show registered in consul and "green" with health check passing.
curl -s http://localhost:8500/v1/catalog/services | jq .
{
  "consul": [],
  "fabio": [],
  "tomcat": [
    "urlprefix- /"
  ]
}

I can access http://localhost:9999 and it gets me to tomcat instance presumably to one of the services that are up, however my expectation is that I should be able to go to http://localhost:9999/tomcat.
So i try to use the following registration process
curl -XPOST -H 'Content-Type: application/json' -d '{"ID": "tomcat1", "Name": "tomcat", "Tags": ["urlprefix- /tomcat"], "Address": "<IP_ADDRESS>", "Port": 8888, "EnableTagOverride": false, "Check": {"HTTP": "http://<IP_ADDRESS>:8888/", "Interval": "15s"} }' 'http://localhost:8500/v1/agent/service/register'

curl -XPOST -H 'Content-Type: application/json' -d '{"ID": "tomcat2", "Name": "tomcat", "Tags": ["urlprefix- /tomcat"], "Address": "<IP_ADDRESS>", "Port": 8887, "EnableTagOverride": false, "Check": {"HTTP": "http://<IP_ADDRESS>:8887/", "Interval": "15s"} }' 'http://localhost:8500/v1/agent/service/register'

curl -XPOST -H 'Content-Type: application/json' -d '{"ID": "tomcat3", "Name": "tomcat", "Tags": ["urlprefix- /tomcat"], "Address": "<IP_ADDRESS>", "Port": 8889, "EnableTagOverride": false, "Check": {"HTTP": "http://<IP_ADDRESS>:8889/", "Interval": "15s"} }' 'http://localhost:8500/v1/agent/service/register'

curl -s http://localhost:8500/v1/catalog/services | jq .
{
  "consul": [],
  "fabio": [],
  "tomcat": [
    "urlprefix- /tomcat"
  ]
}

curl -s http://localhost:8500/v1/catalog/service/tomcat | jq .
[
  {
    "Node": "MMTYRQSETTAGQ",
    "Address": "<IP_ADDRESS>",
    "TaggedAddresses": {
      "lan": "<IP_ADDRESS>",
      "wan": "<IP_ADDRESS>"
    },
    "ServiceID": "tomcat1",
    "ServiceName": "tomcat",
    "ServiceTags": [
      "urlprefix- /tomcat"
    ],
    "ServiceAddress": "<IP_ADDRESS>",
    "ServicePort": 8888,
    "ServiceEnableTagOverride": false,
    "CreateIndex": 65,
    "ModifyIndex": 67
  },
  {
    "Node": "MMTYRQSETTAGQ",
    "Address": "<IP_ADDRESS>",
    "TaggedAddresses": {
      "lan": "<IP_ADDRESS>",
      "wan": "<IP_ADDRESS>"
    },
    "ServiceID": "tomcat2",
    "ServiceName": "tomcat",
    "ServiceTags": [
      "urlprefix- /tomcat"
    ],
    "ServiceAddress": "<IP_ADDRESS>",
    "ServicePort": 8887,
    "ServiceEnableTagOverride": false,
    "CreateIndex": 70,
    "ModifyIndex": 72
  },
  {
    "Node": "MMTYRQSETTAGQ",
    "Address": "<IP_ADDRESS>",
    "TaggedAddresses": {
      "lan": "<IP_ADDRESS>",
      "wan": "<IP_ADDRESS>"
    },
    "ServiceID": "tomcat3",
    "ServiceName": "tomcat",
    "ServiceTags": [
      "urlprefix- /tomcat"
    ],
    "ServiceAddress": "<IP_ADDRESS>",
    "ServicePort": 8889,
    "ServiceEnableTagOverride": false,
    "CreateIndex": 73,
    "ModifyIndex": 74
  }
]

Now when I try to access http://localhost:9999, i get nothing and http://localhost:9999/tomcat gets me a 404.
Objective 2: Trying out the actual steps
Deploy Fabio as a SSL termination endpoint for the services
gateway for api services with multiple path prefixes
“/core/api”
“external/api”
configuration in step 2 would provide a REST endpoint to services and should support the operations.
proxied backend services could also have ssl certs(autogenerated) to protect the endpoints.
Ability to add more services and configuration on the fly.Configuration to be persisted and should survive restarts.
Here are the fabio routes displayed

Any help advise is really appreciated

Anybody out there that can guide me here?

Consider timezone. I have a look later.

The fact that fabio shows the routes means that the services are registered properly and healthy and that they have the right tag. The first thing I've noticed is the extra space in urlprefix- /tomcat which isn't necessary but I've checked that the parser ignores it. You should remove it nevertheless since technically you are now matching with host <space> and path /tomcat.
I think your mistake is that you think that fabio is doing some path mangling/stripping/rewriting but that is not the case. Fabio uses the host/path for routing only, i.e. it uses the host/path only to determine which route to pick and then forwards the request as is. For example:
# curl http://localhost:9999/tomcat -> http://<IP_ADDRESS>:8888/tomcat
# curl http://localhost:9999/ -> 404 Not Found (no route for '/'. check logs)
route add /tomcat http://<IP_ADDRESS>:8888/

# http://localhost:9999/ -> http://<IP_ADDRESS>:8888/
route add / http://<IP_ADDRESS>:8888/

Frank
Thank you for taking a look at it.

Fabio uses the host/path for routing only, i.e. it uses the host/path only to determine which route to pick and then forwards the request as is.

I don't think I understand, Does this mean that for each micro-service that I would like to LB/proxy I would need to use a different port and have them served at "/" ?
My Intention was to do the following,  a request to the Fabio external end-point
ex: http(s)://mysite.xyz.com/tomcat    gets resolved to one of the underlying instances (proxy + load balancing)
                                               +-----> HTTPS(S)    ------> tomcat-a
                                              |
               Request —> HTTP(S) --> fabio -------> HTTP(S)  -----> tomcat-b
                                              |
                                               +—----> HTTP(S)    -- —>tomcat-c

Are you saying I have to do something like this
http://localhost:9999/ -> http://<IP_ADDRESS>:8888/
http://localhost:9997/ -> http://<IP_ADDRESS>:8887/
http://localhost:9996/ -> http://<IP_ADDRESS>:8889/
Can you please correct me If I am wrong
thanks
kris

Does your tomcat instance respond to http://<IP_ADDRESS>:8888/tomcat ?

This worked for me:

Download tomcat 7.0.72, unpack, run bin/startup.sh and check that it is running

$ curl -i localhost:8080/tomcat
HTTP/1.1 404 Not Found
Server: Apache-Coyote/1.1
Content-Type: text/html;charset=utf-8
Content-Language: en
Content-Length: 963
Date: Tue, 04 Oct 2016 20:11:45 GMT

...

Register tomcat in consul

$ curl -XPOST -H 'Content-Type: application/json' -d '{"ID": "tomcat1", "Name": "tomcat", "Tags": ["urlprefix-/tomcat"], "Address": "<IP_ADDRESS>", "Port": 8080, "EnableTagOverride": false, "Check": {"HTTP": "http://<IP_ADDRESS>:8080/", "Interval": "15s"} }' 'http://localhost:8500/v1/agent/service/register'

Run fabio without arguments (assuming consul on localhost:8500)

...
2016/10/04 22:09:44 [INFO] Updated config to
route add tomcat /tomcat http://<IP_ADDRESS>:8080/ tags "urlprefix-/tomcat"
2016/10/04 22:09:44 [INFO] consul: Health changed to #10
...

Test access via fabio

$ curl -i localhost:9999/tomcat
HTTP/1.1 404 Not Found
Content-Language: en
Content-Length: 963
Content-Type: text/html;charset=utf-8
Date: Tue, 04 Oct 2016 20:10:13 GMT
Server: Apache-Coyote/1.1

...

@krism74 I'm going to close this issue for now. Feel free to comment and I'll re-open if necessary.


Need GraphQL Implementation

Hi Kamil Myśliwiec,
I am trying @nestjs/ng-universal with GraphQL and I am getting the following error:
ERROR in ./node_modules/graphql/index.mjs 64:0-70:117
Can't reexport the named export 'visitWithTypeInfo' from non EcmaScript module (only default export is available)
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>@ ./server/app.module.ts
@ ./server/main.ts
Note: Please let me know what and where I want to add/modify the configuration for running GraphQL with this module.

Please, provide a minimal repository which reproduces your issue.

facing same issue.


fix: #56 hide whole feed feature is now working again

hide whole feed feature is now working again!

Thanks :)


Test on older Minecraft versions

Test on:

[ ] 1.19
[ ] 1.18
[ ] 1.17
[ ] 1.16
[ ] 1.15
[ ] 1.14
[ ] 1.13

Not as easy as I'd hoped, closing this one and referring back to #40 for continued development

As of v1.6, I am supporting:

[ ] 1.19
[ ] 1.18
[ ] 1.17
[ ] 1.16.5


⚠ BeyondHD has degraded performance

In 6f99bfd, BeyondHD ($BHD) experienced degraded performance:

HTTP code: 200
Response time: 1771 ms

Resolved: BeyondHD performance has improved in 885b9c2 after 14 minutes.


Bug?: CLI options not present in $params when run through spark via terminal

Describe the bug
I don't know if this is a bug or intentional design choice. if you will be accessing the $params array via direct to spark, you will only get the segments. For example:
<?php

namespace CodeIgniter\Commands;

use CodeIgniter\CLI\BaseCommand;
use CodeIgniter\CLI\CLI;

class TestCmd extends BaseCommand
{
    protected $group        = "Test";
    protected $name         = "test";
    protected $usage        = "test param1 param2 -option1 value1 ";

    public function run(array $params)
    {
        CLI::write(print_r($params, true));
    }
}

If I will run php spark test as df -gh jk in the terminal, I am expecting result to be
Array
(
    [0] => as
    [1] => df
    [gh] => jk
)

but got
Array
(
    [0] => as
    [1] => df
)

This is important if we will be making commands and a developer relies only on $params to check for options. If he will write $option = $params['gh'] ?? null, overlooking the more comprehensive $options = $params['gh'] ?? CLI::getOption('gh'), he will always get null even if passing the gh option.
This is not a problem if we are making programmatic calls, like using $this->call() inside other commands as we can explicitly include the options in the $params.
CodeIgniter 4 version
develop
Affected module(s)
CodeIgniter::runController()
Context

OS: Win10
Web server Apache 2.4.41
PHP version 7.4.8

@paulbalandan can be closed in favor of your PR at https://github.com/codeigniter4/CodeIgniter4/pull/3401  ?

@samsonasik that was my first guess and even retried it now. Still fails. command() only works for programmatic calls. But if we will be using it directly thru terminal, it will still give only the segments. This behavior is in CodeIgniter::runController() where the $params passed is the segments array only. I don't know if that is intended really.

Options are not present in the $params array. To get the option value CLI:getOption() method should be used

Options are not present in the $params array. To get the option value CLI:getOption() method should be used

Yes, I'm aware of that. But my question is why? Arguments can be accessed by either $params or CLI::getSegment(), but options can only be accessed by CLI::getOption, if made thru spark

This behavior is not consistent.
$ php spark test param1 param2 -opt1 opt1val

CodeIgniter v4.1.4 Command Line Tool - Server Time: 2021-10-15 18:28:19 UTC-05:00

Array
(
    [0] => param1
    [1] => param2
)

But if you use command():
command('test param1 param2 -opt1 opt1val');

Array
(
    [0] => param1
    [1] => param2
    [opt1] => opt1val
)

I think $param should have [opt1] => opt1val even if called via spark.

In the current implementation, param and option are completely different.
Therefore the command() taking options in params is incorrect behavior.
https://github.com/codeigniter4/CodeIgniter4/blob/0045a6f5f2d7d35cad9123ca5f8efbe84f51eaf6/system/Common.php#L180
But there is no way to pass options to command()/BaseCommand::call().
The BaseCommand::run() method should have had options as a parameter.
        abstract public function run(array $params, array $options)

https://github.com/codeigniter4/CodeIgniter4/blob/0045a6f5f2d7d35cad9123ca5f8efbe84f51eaf6/system/CLI/BaseCommand.php#L101
But adding a parameter is a breaking change.

I think the opposite. IMO command() represents correct behavior. The way I see it, CI4 was designed initially to work only for browsers (and what the user guide says) that's why code in system/Codeigniter.php is working only on the "params" (i.e. URI segments). When support for spark was added later, this behavior was left unchecked (check CodeIgniter::runController())

Good conversation here. I commented on the PR, but just as a reminder CI4 supports three types of requests:

Web: URI segments become parameters, sent to Controllers via Routes, output controlled by Headers to browser
Spark: accessed by CLI via the spark command, arguments are Command arguments, sent to Commands by CommandRunner, output controlled by CLI class
PHP CLI: accessed by CLI via php public/index.php, arguments become URI segments, sent to Controllers via Routes, output varies

2 & 3 are often conflated because 3 is somewhat of a niche case, but (as of now) still needs to be supported.


[Android][curl] Wrong flag in libcurl.pc on android compilation

Robert Schumacher Can I ask improvement of libcurl.pc when compile android?
pkg-config libcurl.pc --libs
-Llibcurl.pc/../../lib -lcurl -lc -lgcc -ldl -pthread libcurl.pc/../../lib/libssl.a libcurl.pc/../../lib/libcrypto.a /home/tal/boinc/3rdParty/android/android-ndk-r15c/platforms/android-16/arch-arm/usr/lib/libz.so
It give libz.so Although it compile libcurl.a with libz.a from vcpkg. Can we fix libcurl.pc that it will give the correct libz flag location? zlib.pc/../../lib/libz.a?
@ras0219-msft @ras0219

@JackBoosY Maybe you can also help?
Maybe the issue is how the curl is compile.
Maybe in android it find the libz that come with the ndk instead to take zlib that come with the ndk.
How I can force find_package to detect the zlib from vcpkg?
https://github.com/curl/curl/blob/3085ccfae996bb0fa606d2c7bc6783dc15d76a30/CMakeLists.txt#L636
https://github.com/curl/curl/blob/3085ccfae996bb0fa606d2c7bc6783dc15d76a30/CMake/Macros.cmake#L116

Also why curl/control file was deleted?
How curl know that he will compile with zlib?
https://github.com/microsoft/vcpkg/commit/e6dcc079c81161786eb7b052209a2047e79f2c6c#diff-a6411f1fd1bac0e7c25451306467d99c12bccbe5a300bd10c208edfcae05ba7b

Also why ports/curl/CONTROL file was deleted?

The deprecated CONTROL file has been replaced by vcpkg.json.

How curl know that he will compile with zlib?

This controlled by configuration options in portfile.cmake, and by find_... commands in curl's own CMakeLists.txt. If the behaviour of the find commands isn't satisfying, there is some influence via options in the portfile.

@dg0yt Thank you for the info!
Can you teach me? How to compile curl with zlib from vcpkg?
We can talk on discord if you like.

No discord on my side...
I was already trying to look at the context. What you could do is: provide logs. In buildtrees/curl, there are "config-...-out.log" from the cmake configuration step. You will be able to find out which zlib it discovered.
curl uses plain find_package(ZLIB), and vcpkg doesn't seem to provide any particular configuration hints. Seems to work well for plain linux, so why shouldn't it work for android?

Because for android there is another set of libraries like that come with ndk, and I think the vcpkg curl find the libz of ndk instead libz from vcpkg. Maybe for other user that ok, but for me, I want the program will stay up to date with the latest zlib.

I will search the logs now and upload it here, we can look at them together.

Because for android there is another set of libraries like that come with ndk, and I think the vcpkg curl find the libz of ndk instead libz from vcpkg. Maybe for other user that ok, but for me, I want the program will stay up to date with the latest zlib.

Sure. A) My linux box comes with zlib, too. B) I must use a more recent zlib on android, too. I just don't use vcpkg for android yet.

config-armv6-android-rel-out.log
config-armv6-android-dbg-out.log

-- Looking for dlopen in dl;-pthread;OpenSSL::SSL;OpenSSL::Crypto
-- Looking for dlopen in dl;-pthread;OpenSSL::SSL;OpenSSL::Crypto - found
-- Found ZLIB: /home/tal/boinc/3rdParty/android/android-ndk-r15c/platforms/android-16/arch-arm/usr/lib/libz.so (found version "1.2.11")

The version is not correct.

How we can give hint for find_package(ZLIB) that will point out to vcpkg install library?

If you want me to provide more logs, I will happy to upload them here.

I don't think I can help you to dig deeper for NDK 15 armv6.
The documentation for the ZLIB find module is: https://cmake.org/cmake/help/latest/module/FindZLIB.html
The key hook is ZLIB_ROOT, but for vcpkg you would need to deal with release and debug variants.
The ZLIB find module does just find_path for the header, and find_library for the lib. The source code is at
https://github.com/Kitware/CMake/blob/master/Modules/FindZLIB.cmake
It contains some variables which might be used for configuration, too (e.g. ZLIB_LIBRARY)
(Make sure that your cmake toolchain doesn't set one of these variables.)

maybe it does.
What about find_package hint path. Can You tell me how I can give the current path of zlib as hint path?

https://stackoverflow.com/questions/49816206/cmake-find-package-specify-path

find_package hint path

Read about <PackageName>_DIR here:
https://cmake.org/cmake/help/latest/command/find_package.html#full-signature-and-config-mode
I don't see that there is a dir which provides cmake configuration for zlib in vcpkg.

for vcpkg there is a variable that tell me where is the current install/arm-android/lib of?

maybe it ${CURRENT_PACKAGES_DIR}/lib ?

I will make a patch and let you know

0001-fix-dependency-zlib.zip

I will try it on my local curl android compilation

I take example from here:
https://github.com/microsoft/vcpkg/blob/7f1708740708b7fd912af5c0ade23a3b48712ce1/ports/qt5-base/portfile.cmake#L155-L156

@dg0yt
I create this patch and it didn't work. Can you help?
From 60ccc12b04e4d81c01bad0b4cf58e86ba1aae48e Mon Sep 17 00:00:00 2001
From: Tal Regev
Date: Sat, 22 May 2021 20:36:20 +0300
Subject: [PATCH] fix-dependency-zlib

---
 CMakeLists.txt | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 63944877a..0cdb4f2e8 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -635,7 +635,8 @@ check_library_exists("${CURL_LIBS}" dlopen "" HAVE_DLOPEN)
 set(HAVE_LIBZ OFF)
 set(HAVE_ZLIB_H OFF)
 set(USE_ZLIB OFF)
-optional_dependency(ZLIB)
+find_library(ZLIB NAMES z zlib PATHS "${CURRENT_INSTALLED_DIR}/lib" NO_DEFAULT_PATH)
+find_library(ZLIB NAMES z zlib zd zlibd PATHS "${CURRENT_INSTALLED_DIR}/debug/lib" NO_DEFAULT_PATH)
 if(ZLIB_FOUND)
   set(HAVE_ZLIB_H ON)
   set(HAVE_LIBZ ON)
--
2.28.0.windows.1

@dg0yt
I think I fix this issue in #18092
Can you take a look?

Seems similar to #17163. What is the value of CMAKE_FIND_ROOT_PATH? So this was the problem for me and this possible solutions

@autoantwort I can print CMAKE_FIND_ROOT_PATH from profile.cmake and write it here.

@autoantwort from profile.cmake file the CMAKE_FIND_ROOT_PATH is empty.
I will try to see the value of CMAKE_FIND_ROOT_PATH from the CmakeLists.txt of lib curl.

@autoantwort
CMAKE_FIND_ROOT_PATH=/home/tal/boinc/3rdParty/android/android-ndk-r15c/home/tal/boinc/3rdParty/android/vcpkg/installed/armv6-android/debug/home/tal/boinc/3rdParty/android/vcpkg/installed/armv6-android
It the same like in your issue, the CMAKE_FIND_ROOT_PATH if point out first for the ndk, then to vcpkg.


Update Minio and minio-go in rke-tools to latest version

Update Minio Image to the latest: RELEASE.2020-07-13T18-09-56Z
https://github.com/minio/minio/releases/tag/RELEASE.2020-07-13T18-09-56Z
Update minio-go in rke-tools
https://github.com/minio/minio-go/releases/tag/v6.0.57

Tested with v2.4-head(2d58a3fcd)

Created a cluster with backup/restore configured to minio
Backup/restore using minio was functional.
RKE tools shows the updated version v0.1.60.
The minio version points to 2019-09-25T18-25-51Z

Tested with v2.4-head(e6e3632a8)

Create a DO cluster with backup/restore configured to minio.
Create few workloads. Perform a backup snp1. Create another workload.
Perform restore to snp1.

Backup and restore was functional using minio
Rke tools was updated to the version rancher/rke-tools : v0.1.61
root@soumyauserminiotest1:~# sudo docker ps -a|grep rke-tools
d0352d14233e        rancher/hyperkube:v1.18.6-rancher1                "/opt/rke-tools/entr…"   8 minutes ago            Up 8 minutes                                      kube-proxy
3f13ba1acc99        rancher/hyperkube:v1.18.6-rancher1                "/opt/rke-tools/entr…"   9 minutes ago            Up 9 minutes                                      kubelet
651488425f6f        rancher/hyperkube:v1.18.6-rancher1                "/opt/rke-tools/entr…"   9 minutes ago            Up 8 minutes                                      kube-scheduler
d42eb2648c29        rancher/hyperkube:v1.18.6-rancher1                "/opt/rke-tools/entr…"   9 minutes ago            Up 8 minutes                                      kube-controller-manager
80cab40ca440        rancher/hyperkube:v1.18.6-rancher1                "/opt/rke-tools/entr…"   9 minutes ago            Up 9 minutes                                      kube-apiserver
e988e4fa4409        rancher/rke-tools:v0.1.61                         "/bin/bash"              9 minutes ago            Created                                           service-sidekick
86c76fbce208        rancher/rke-tools:v0.1.61                         "/bin/bash"              10 minutes ago           Exited (0) 10 minutes ago                         cluster-state-deployer

Minio version was also updated to the latest rancher/minio-minio:RELEASE.2020-07-13T18-09-56Z.
This was observed by enabling and running the default pipeline in rancher

Tested with master-head(f7e83e3e4)
Create a DO cluster with backup/restore configured to minio as a standard user
Create few workloads. Perform a backup snp1. Create another workload.
Perform restore to snp1.
Backup and restore was functional using minio
Rke tools was updated to the version rancher/rke-tools : v0.1.61
root@soumyamasterdotest1:~# sudo docker ps -a|grep rke-tools
ffa3548281d9        rancher/hyperkube:v1.18.6-rancher1                "/opt/rke-tools/entr…"   26 minutes ago       Up 26 minutes                                         kube-proxy
6d00725d5b17        rancher/hyperkube:v1.18.6-rancher1                "/opt/rke-tools/entr…"   26 minutes ago       Up 26 minutes                                         kubelet
7da1788c1c35        rancher/hyperkube:v1.18.6-rancher1                "/opt/rke-tools/entr…"   27 minutes ago       Up 4 minutes                                          kube-scheduler
c9352defd59d        rancher/hyperkube:v1.18.6-rancher1                "/opt/rke-tools/entr…"   27 minutes ago       Up 19 minutes                                         kube-controller-manager
c23f7c9bfdbe        rancher/hyperkube:v1.18.6-rancher1                "/opt/rke-tools/entr…"   27 minutes ago       Up 27 minutes                                         kube-apiserver
c3143dd5f8a5        rancher/rke-tools:v0.1.61                         "/bin/bash"              27 minutes ago       Created                                               service-sidekick
e77fff1af0e2        rancher/rke-tools:v0.1.61                         "/bin/bash"              58 minutes ago       Exited (0) 58 minutes ago                             cluster-state-deployer

Updated a default value for memory limit.

Validated with master-head (c761bcd4d)

Minio image verification

Enable and run the default pipeline rancher/pipeline-example-go in Rancher.
Minio image was updated to the latest rancher/minio-minio:RELEASE.2020-07-13T18-09-56Z
Minio workload came up successfully. No crashloop backoff errors observed.

Verified that the memory limit for minio deployment was updated to 500M


To do

[ ] Icons on scrollbar
[ ] Dropdowns
[ ] Sliders looking more like windows 98 sliders

somehow, sometime this theme needs to be upstreamed :)


Segregate networks by genesis block

We will have multiple different networks going simultaneously. Eventually, a public testnet, and public mainnet, and developer networks. Ethereum has at least 6 networks. It's important that the networks remain separate. If peers A and B are on different networks, and accidentally connect to one another, one may "catch up" to the other if they have the same genesis block, or they will both consider each other to be invalid and ban each other.
We should add the genesis block hash to our peer discovery messages, or something along those lines. Maybe after doing #1647.

Also segregate config files by network. People keep getting error messages because they switched profiles and the profiles have different genesis ledgers

the network segregation should happen as part of @cmr's libp2p integration work.

This is basically done. We have a hacky version with "chain ids" and the libp2p work uses different network identifiers.


sonobuoy check

What type of PR is this?

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespaces from that line:

/kind feature

/kind api-change
/kind bug
/kind cleanup
/kind design
/kind documentation
/kind failing-test
/kind feature
/kind flake

What this PR does / why we need it:
sonobuoy check in 1 master + 1 worker cluster
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
make sure your commit is signed off

/lgtm

/approve

/lgtm


Feature - Add Additional Intro Labs

Add additional intro labs, in particular how it works and installation.  Also corrected some previous intro content.

This update is outside the content folder so we have duplicate files, and it won't render correctly in homeroom.  Can we please merge the new files into content folder?

@theckang Yep moved everything into the content folder.


Implement genotype confidence percentile

Need to do this to normalise gt-conf for overall covg of the sample.
This has been implemented in Mykrobe and minos (in python)

@martinghunt code is here
https://github.com/iqbal-lab-org/minos/blob/ed975d1099c6403eff79b04b0d2064eebfa95e73/minos/genotype_confidence_simulator.py

I did wonder about a single header and cpp file , which did this

Am I right to assume this is for paper?

Happy to discuss tomorrow; is actually not too hard to implement. The decision when we designed the 4way, and when we had not implemented and tested GCP , was just to use same depth for all samples. Now, we have 24 samples at mildly varying different depths coming up, plus this issue in the 4way. So, the argument for doing this immediately is stronger.

Do agree, it looks like the the cost is low for implementing this, with high benefits. However, other tools (e.g. snippy, nanopolish, medaka) I think do not implement such normalisation (they don't have the concept of multisample variant calling). If pandora does this normalisation, do we still need to present ROC curves on several subsampling depths (e.g. 30x, 60x and 100x)? Or do we just give the full data for each tool?

you don't need multisample var calling to need the normalisation - as soon as you use the same threshold for multiple single-sample VCFs, normalisation is an issue.
I don't think this matters. The plan for snippy/illumina is to give a baseline for comparison - how good an error rate you get with normal approaches, using our dataset and evaluation framework, and how high recall can go with one reference.
Having GCP means we can define a meaningful threshold independent of coverage, so our samples have comparable filtering. It doesn't mean we perform the same independent of depth. We will still do better with more depth, certainly going up for 30-60x illumina anyway. so the value of subsampling illumina is unchanged - especially it means snippy performance will change.
Having said that, i think there is diminishing returns, 90% of the value is obtained by just running at one depth, as mostly this section of the paper is to show how high recall can go with one ref.

Deleted previous rambling post

We definitely not doing 24 way illumina at multiple depths, too hard (many samples, different depths)
Subsampling and gcp are orthogonal
Snippy has essentially zero error, mad to worry about fairness of filtering
We can't normalise likelihoods of tools where we don't know how implemented. We ate fixing our own implementation limitation

[ ] Port https://github.com/iqbal-lab-org/minos/blob/ed975d1099c6403eff79b04b0d2064eebfa95e73/minos/genotype_confidence_simulator.py to C++ in a branch

I thought the idea was to have this as a separate "header-only" library? i.e. it shouldn't be a branch on pandora.
I think we need to have another call to discuss the details of this implementation before we go galloping off in the wrong direction :horse_racing:

Oh sorry. Not sure I understood all that you said previously, but I have an answer below (we indeed need to discuss this in more details very soon, as I am starting implementation now).
Header-only library is indeed less coupled and way more reusable, modularised and maintainable than some files in a branch. So it is indeed a better implementation design, but will take more time to do it in a proper way. For example, a core concept we will have in this header-only library is the Genotyper, which does not actually exist in pandora. In pandora, we have the concept of VCFs (class VCF) which has the ability to genotype itself. We would need to decouple these concepts by refactoring into at least two classes, VCF and Genotyper. We will also need to define a public interface to the Genotyper class that is not coupled with pandora concepts, so other users (e.g. gramtools) can implement this public interface and use it. Probably more issues will appear on the run...
In summary, I see three options:

[Fastest but with bad design] Just create a branch and do the genotype confidence percentile inside pandora itself (don't worry about reusability, don't change components interface, keep things tightly coupled, no refactoring). But we will have tests and make sure that genotype confidence percentile is being done correctly. This will get us from implementation to results very fast, can be convenient to finish the paper;
[Medium ground] Create a separate header-only library to deal with this, but don't do any refactoring. Basically, this header-only library will just be usable by pandora (e.g. no refactoring and things will be tightly coupled), but it will be a separated library;
[Slowest but with good design] Proper refactoring of pandora and creation of header-only library with proper specification of a Genotyper abstract class so that is usable to anyone who can concretely implement such Genotyper class;

Please add other options or correct any of these options. Our ultimate goal is always 3. I just don't know if we start by 1 for the sake of the paper, and then refactor the solution into 3 later. I also don't know how fast is 1 and how slow is 3.

I will be probably need GCP soon. So I'm biased towards option3. I don't particularly mind though because I know you're on the paper deadline ;-O

Yes, i'm reluctant to do option 3 before the paper, because a refactor could take a while, and generate more issues on the way. Let me expand on those options

Write a python script that normalises pandora VCFs. The simulation of SNPs could be the same as the minos code, but the genotyping would be basically whatever pandora does.

Leandro's branch implementation.

2.header file, but no refactoring. What does this actually mean? Would the following work as a solution?
simulate_snps(number of snps, genomewide average coverage, error rate, model)
// model can be neg binomial or poisson
returns a vector of pairs like this (covg on true allele, covg on wrong allele)
get_confidence_distribution(genomewide average coverage, error rate, vector of pairs)

calls the internal pandora genotyping code, gets the confidences, makes a list of confidences

make_confidence_to_percentile_map(list of confidences)

returns a map.

The first and third functions could exist in a separate library and would be reusable by gramtools.
The second function is coupled to pandora, but in a future update we could refactor and create a new library function that takes a genotyper class

complete refactor, glory of conan the barbarian rebuild civilisation.
do the above, with refactoring.

2. **[Medium ground]** Create a separate header-only library to deal with this, but don't do any refactoring. Basically, this header-only library will just be usable by pandora (e.g. no refactoring and things will be tightly coupled), but it will be a separated library;

3. **[Slowest but with good design]** Proper refactoring of pandora and creation of header-only library with proper specification of a `Genotyper` abstract class so that is usable to anyone who can concretely implement such `Genotyper` class;

I think both of these options are actually the same. What I mean by that is in both cases we implement a header-only GCP library. The only thing that differs is how pandora interacts with this library. And that is a separate issue altogether.
It would be easy enough to patch an interface within pandora that allows us to pass a genotyper to the GCP library no?

In this example dependency graph, the direction of the arrow indicates "depends on". Regardless of whether we go with Option 2 or 3 that you mentioned @leoisl , it would be best for us to implement an interface in this way so that there is no direct dependency between pandora and gcplib.

Write a python script that normalises pandora VCFs. The simulation of SNPs could be the same as the minos code, but the genotyping would be basically whatever pandora does.
Leandro's branch implementation.

So these options are conflicting. The first is porting pandora genotyping code to python, and use what we have in minos. Second is porting minos GCP code to c++ and use pandora genotyping code. It is easier and less error prone to do the second.

2.header file, but no refactoring. What does this actually mean? Would the following work as a solution?

I vote to go for this option, that you explain here, with the change that the second function (i.e. the interaction between the GCP library and its user) is defined in an interface as explained by @mbhall88  in https://github.com/rmcolq/pandora/issues/216#issuecomment-614045916 . However, I won't worry much about this interface right now, we can refactor it as we need, and do the things in demand.
I think things are very abstract now, and we can easily get lost in the discussion. Will start a repo, and start something so that we can have something more concrete to discuss on.

I disagree. I think discussing these things beforehand is paramount. Otherwise, we get attached to code that we have written and say things like "let's just use what we've written so far for now and we'll change it later".
If everyone else disagrees then I'm happy to back down.

I don't disagree. But let's try and clarify what we can do - i don't think we're that far from being able to do what you want.
I think this whole thing reduces to less than 5 functions. i listed 3 key ones above and suggested 2 would fit in this library and one be implemented by the caller. We could expand on your diagram Michael, and add in APIs and specify callers and responsibilities. This would give brice a chance to look at that and work out how it would/not work with gramtools (similar to minos), and leandro to do the same for pandora. How about that?

Basically i think this lib is simple enough that we can go right ahead and map out the full API, and then think. Something to simulate a set of SNPs (covg on two alleles), somethign to genotype those and get confidences, something to turn that into percentiles, something to get a gt-conf and return a percentile.
If you have those the caller can decide whether to apply to a VCF post hoc, or do it line by line, or do it before the VCF when you first genotype a site - the caller has the freedom to apply as they choose

Yeah, you are right, we should design the full API before touching any line of code, I think I was just too eager to code already. Anyway, I think the best is to think that whatever code is developed before we have the proper API design is prototype and has the risk to be totally discarded (and we should do it). I have a first version of the Genotyper interface for you to take a look (making a PR at  https://github.com/leoisl/GCP). I think it is important to keep this interface as loosely coupled as possible, so it can be used with whatever caller.
The other concepts (Model, Simulator, etc) are simpler to design I think, as they are just going to be used by users (not extended and implemented and the Genotyper interface).


Use fetchFromGitHub when possible

Currently, fetchgit is used for all dependencies. However, it would be possible to use fetchFromGitHub for sources coming from GitHub. This is supported by the nixpkgs buildGoPackage function (see https://github.com/NixOS/nixpkgs/blob/ff2ea911b3051e3f2d1b44c3c8c966f5fb9be648/pkgs/development/go-packages/generic/default.nix#L91).
Note that fetchFromGitHub is the recommended fetcher to fetch sources coming from GitHub in nixpkgs.
Could we add an heuristic to use fetchFromGitHub when a dependency url starts with https://github.com/?

fetchFromGitHub doesn't fetch submodules (which I believe that go get does). I submodules usage in the Go world is rare, but not completely unheard of.


A.04.IR01.IR02.hierarchy

This issue has been extracted from the issue list on:https://ies-svn.jrc.ec.europa.eu/issues/2685
Comment
The purpose is not formulated correctly:
'Purpose: Type of the cited resource must be provided'
Proposed Change
Change to:
'Purpose: Checks that a resource type is provided'

Proposed Resolution
Agreed with proposed change.
Change 'Purpose: Type of the cited resource must be provided' to 'Purpose: Checks that a resource type is provided'


How to create the virtual environment?

Thank you for your exciting work !!!
I found conflicts when I  conda env create
Could you update the environment.yml？

I solved the problem of environment configuration by installing docker!
Thank you for your code, I will study it.


Did what was told to!

You're free to make edits🙇♂

The college name getting erased out isn't a good thing 😐

What else do you suggest?😥

When that typing effect gets totally completed for the college name, that area can be made blank for some seconds and then some phrase like Learn, Grow, Build can take it's place.
I'm just suggesting, the college name getting erased shouldn't be there :thinking:

ok will try!


Syntax error of delete statement on MySQL 5.7

When I execute this delete query on MysqlJdbcContext (MySQL 5.7):
ctx.run(quote {
  query[TestTable].filter(_.id == 1).delete
})

a following syntax error occurs:
[error] java.sql.SQLSyntaxErrorException: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'AS x3 WHERE x3.id = 1' at line 1
[error]         at com.mysql.cj.jdbc.exceptions.SQLError.createSQLException(SQLError.java:120)
[error]         at com.mysql.cj.jdbc.exceptions.SQLExceptionsMapping.translateException(SQLExceptionsMapping.java:122)
[error]         at com.mysql.cj.jdbc.ClientPreparedStatement.executeInternal(ClientPreparedStatement.java:953)
[error]         at com.mysql.cj.jdbc.ClientPreparedStatement.executeUpdateInternal(ClientPreparedStatement.java:1092)
[error]         at com.mysql.cj.jdbc.ClientPreparedStatement.executeUpdateInternal(ClientPreparedStatement.java:1040)
[error]         at com.mysql.cj.jdbc.ClientPreparedStatement.executeLargeUpdate(ClientPreparedStatement.java:1350)
[error]         at com.mysql.cj.jdbc.ClientPreparedStatement.executeUpdate(ClientPreparedStatement.java:1025)
[error]         at com.zaxxer.hikari.pool.ProxyPreparedStatement.executeUpdate(ProxyPreparedStatement.java:61)
[error]         at com.zaxxer.hikari.pool.HikariProxyPreparedStatement.executeUpdate(HikariProxyPreparedStatement.java)
[error]         at io.getquill.context.jdbc.JdbcContextVerbExecute.executeAction$$anonfun$1(JdbcContextVerbExecute.scala:40)
[error]         at io.getquill.context.jdbc.JdbcContextVerbExecute.withConnectionWrapped$$anonfun$1$$anonfun$1(JdbcContextVerbExecute.scala:33)
[error]         at io.getquill.context.jdbc.JdbcContext.wrap(JdbcContext.scala:54)
[error]         at io.getquill.context.jdbc.JdbcContextVerbExecute.withConnectionWrapped$$anonfun$1(JdbcContextVerbExecute.scala:33)
[error]         at io.getquill.context.jdbc.JdbcContext.withConnection$$anonfun$1(JdbcContext.scala:63)
[error]         at scala.Option.getOrElse(Option.scala:201)
[error]         at io.getquill.context.jdbc.JdbcContext.withConnection(JdbcContext.scala:65)
[error]         at io.getquill.context.jdbc.JdbcContextVerbExecute.withConnectionWrapped(JdbcContextVerbExecute.scala:33)
[error]         at io.getquill.context.jdbc.JdbcContextVerbExecute.withConnectionWrapped$(JdbcContextVerbExecute.scala:13)
[error]         at io.getquill.context.jdbc.JdbcContext.withConnectionWrapped(JdbcContext.scala:16)
[error]         at io.getquill.context.jdbc.JdbcContextVerbExecute.executeAction(JdbcContextVerbExecute.scala:40)
[error]         at io.getquill.context.jdbc.JdbcContextVerbExecute.executeAction$(JdbcContextVerbExecute.scala:13)
[error]         at io.getquill.context.jdbc.JdbcContext.executeAction(JdbcContext.scala:16)
[error]         at Sample$.$anonfun$3(Sample.scala:43)
[error]         at Sample$.delete(Sample.scala:43)
[error]         at delete.main(Sample.scala:37)
[error]         at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
[error]         at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
[error]         at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
[error]         at java.lang.reflect.Method.invoke(Method.java:498)

This error means that if we declare an alias for a table, we must use the alias when referring to the table:
DELETE t1 FROM test AS t1, test2 WHERE ...

The generated query by ProtoQuill has a syntax error:
DELETE FROM test_table AS x3 WHERE x3.id = 1

Ref: https://dev.mysql.com/doc/refman/5.7/en/delete.html
Version: 4.0.0
Module: quill-jdbc
Database: mysql 5.7
Expected behavior
Delete query doesn't have syntax errors. Generated query must be:
DELETE x3 FROM test_table AS x3 WHERE x3.id = 1

Actual behavior
Delete query has syntax errors. Generated query is:
DELETE FROM test_table AS x3 WHERE x3.id = 1

Steps to reproduce the behavior
This issue cannot be reproduced using mirror contexts.
On MySQL 5.7, a table definition is
mysql> CREATE TABLE test_table(
  id bigint(20) unsigned NOT NULL AUTO_INCREMENT PRIMARY KEY,
  name varchar(32) NOT NULL
) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4;

mysql> desc test_table;
+-------+---------------------+------+-----+---------+----------------+
| Field | Type                | Null | Key | Default | Extra          |
+-------+---------------------+------+-----+---------+----------------+
| id    | bigint(20) unsigned | NO   | PRI | NULL    | auto_increment |
| name  | varchar(32)         | NO   |     | NULL    |                |
+-------+---------------------+------+-----+---------+----------------+

and the reproduce code is:
@main def delete =
  import io.getquill._
  val ctx = new MysqlJdbcContext(SnakeCase, "sample.context")
  case class TestTable(id: Long, name: String)
  import ctx._

  ctx.run(quote {
    query[TestTable].filter(_.id == 1).delete
  })

GitHub:

https://github.com/taretmch/protoquill-sandbox/blob/db8d4a0c650c04b680550575c68e8a496265bc81/src/main/scala/IssueReproduceSample.scala
https://github.com/taretmch/protoquill-sandbox/blob/db8d4a0c650c04b680550575c68e8a496265bc81/database/init/db.sql#L19-L22

Workaround
I cannot find workarounds.
@getquill/maintainers

For Scala 2 version there was recently a fix https://github.com/zio/zio-quill/issues/2545. Suffering from this issue in my Scala 3 project.

Should be fixed in 4.3.0. @mem-memov could you please try?

Thank you very much for this fix. I'll apply it tomorrow and report here the results.

I've checked version 4.3.0. It still produces aliases in DELETE statements at compile time.
Sorry, don't know how to load 4.3.0-SNAPSHOT as Maven dependency
<dependency>
    <groupId>io.getquill</groupId>
    <artifactId>quill-jdbc-zio_3</artifactId>
    <version>4.3.0-SNAPSHOT</version>
</dependency>

It produces an error: Could not find artifact io.getquill:quill-jdbc-zio_3:pom:4.3.0-SNAPSHOT in apache.snapshots (https://repository.apache.org/snapshots)
Same error with 4.4.0-SNAPSHOT

Same occurs on 4.3.1-SNAPSHOT:
query[Token].filter(_.id(lift(token.id))).delete
Produces:
DELETE FROM token x3 WHERE x3.id = 1;
Instead of:
DELETE x3 FROM token x3 WHERE x3.id = 1;

The same occurs on 4.6.0

my compile time query is: DELETE FROM my_table x8 WHERE x8.id = 123, instead, it should be no alias... my quill version is 4.6.0

If you override the MySQLDialect and set
  override def useActionTableAliasAs: SqlIdiom.ActionTableAliasBehavior = SqlIdiom.ActionTableAliasBehavior.Hide

I think it produces a more correct result for mysql.

@dispalt really appreciate the info here, it works!

Hm, my delete queries still come with Aliases, despite using SqlIdiom.ActionTableAliasBehavior.Hide. @renjiandreamore does this really work for you?

Ah, sorry, I'm using Scala 2 version.

Can someone try with v4.7.3, please? We merge the support of Mysql5 dialect in v4.7.0

@guizmaii
I still get table alias and fails in AWS Aurora MySQL
val zioVersion = "2.0.13" val quillVersion = "4.8.0" or "4.7.3" scalaVersion := "3.3.1"
Quill Query (compiled in 7ms): DELETE FROM sla x2 WHERE x2.sla_id = ? [info] 41 | quote { [info] 42 | slaQuote.filter(_.sla_id == lift(id)).delete [info] 43 | } [info] 44 | }

Are you using the new dialect?

@guizmaii
I am using a similar pattern to what is mentioned in here https://zio.dev/ecosystem/templates and present here
https://github.com/ScalaConsultants/zio-scala3-quickstart.g8
I am not sure I can choose the dialect here because I am relying on io.getquill.jdbczio.Quill.MySql
Layer defined as
  private val mysqlLayer = Quill.Mysql.fromNamingStrategy(Literal)

repo
object SlaRepositoryLive:

  val layer: URLayer[Quill.Mysql[Literal], SlaRepository] = ZLayer {
    for {

      quill <- ZIO.service[Quill.Mysql[Literal]]
    } yield SlaRepositoryLive(quill)
  }

Quill.MySql uses MySQLDialect
   class Mysql[+N <: NamingStrategy](val naming: N, override val ds: DataSource)
    extends Quill[MySQLDialect, N] with MysqlJdbcTypes[MySQLDialect, N] {
    val idiom: MySQLDialect = MySQLDialect
    val dsDelegate = new MysqlZioJdbcContext[N](naming)
  }
  object Mysql {
    def apply[N <: NamingStrategy](naming: N, ds: DataSource) = new Mysql[N](naming, ds)
    def fromNamingStrategy[N <: NamingStrategy: Tag](naming: N): ZLayer[javax.sql.DataSource, Nothing, Mysql[N]] =
      ZLayer.fromFunction((ds: javax.sql.DataSource) => new Mysql[N](naming, ds))
  }

I have overrode idiom in MySql with the following code but I still get the same error
object DbContext {
  val live: URLayer[DataSource, Quill.Mysql[SnakeCase]] =
    ZLayer.fromFunction((ds: DataSource) =>
      new Quill.Mysql(SnakeCase, ds) {
        // workaround for https://github.com/zio/zio-protoquill/issues/159
        override val idiom: MySQLDialect = MySQL5Dialect
      }
    )
}

Exception in thread "zio-fiber-80" java.sql.SQLSyntaxErrorException: (conn=38) You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near 'a WHERE a.foo_id IN (1)' at line 1

looks like there is still an alias but without AS


[BEAM-11227] Try reverting #14295: Moving from vendored gRPC 1.26 to 1.36

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

There remains a CVE but I do not believe it actually affects Beam in normal situations.

Right. The gRPC upgrade is the main culprit.

Run Java_Examples_Dataflow PreCommit

Java Precommit failed:
19:45:43 org.apache.beam.runners.flink.FlinkSavepointTest > testSavepointRestoreLegacy FAILED
19:45:43     java.util.concurrent.ExecutionException at FlinkSavepointTest.java:255
19:45:43         Caused by: org.apache.flink.runtime.messages.FlinkJobNotFoundException at Dispatcher.java:894

19:28:32 org.apache.beam.sdk.metrics.MetricsTest$AttemptedMetricTests > testAllAttemptedMetrics FAILED
19:28:32     java.lang.AssertionError at MetricsTest.java:346

Run Java PreCommit

Flink again. https://ci-beam.apache.org/job/beam_PreCommit_Java_Phrase/3355/console
20:46:52 org.apache.beam.runners.flink.FlinkSavepointTest > testSavepointRestoreLegacy FAILED
20:46:52     java.util.concurrent.ExecutionException at FlinkSavepointTest.java:255
20:46:52         Caused by: org.apache.flink.runtime.messages.FlinkJobNotFoundException at Dispatcher.java:894

Run Java PreCommit

org.apache.beam.sdk.io.FileIOTest.testMatchWatchForNewFiles failed: https://ci-beam.apache.org/job/beam_PreCommit_Java_Phrase/3357/
java.lang.AssertionError: FileIO.MatchAll/Reshuffle.ViaRandomKey/Values/Values/Map/ParMultiDo(Anonymous).output:
Expected: iterable with items [<Metadata{resourceId=/tmp/junit1162320239170804688/watch/first, sizeBytes=42, isReadSeekEfficient=true, checksum=null, lastModifiedMillis=0}>, <Metadata{resourceId=/tmp/junit1162320239170804688/watch/second, sizeBytes=37, isReadSeekEfficient=true, checksum=null, lastModifiedMillis=0}>, <Metadata{resourceId=/tmp/junit1162320239170804688/watch/third, sizeBytes=99, isReadSeekEfficient=true, checksum=null, lastModifiedMillis=0}>] in any order
     but: not matched: <Metadata{resourceId=/tmp/junit1162320239170804688/watch/first, sizeBytes=0, isReadSeekEfficient=true, checksum=null, lastModifiedMillis=0}>

Run Java PreCommit

Run Java PreCommit

Run Java PreCommit

Run Java PreCommit

Run Java PreCommit

Run Java PreCommit

Run Java PreCommit

Run Java PreCommit

Memo: next time I need to revert something, use git revert -m 0 <commit>, where 0 is parent number in commit log's "Merge" section.


Remarks needs formatting

It's great that in Remarks there is a list of values and corresponding names. However it would be easier if it was formatted properly with newlines in between different OS entries.

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

ID: efc66629-0be3-ad72-843c-1aa87cdeabe7
Version Independent ID: 08524449-919b-6f0a-512b-ee65add3545f
Content: CloudServiceConfiguration.OsFamily Property (Microsoft.Azure.Batch.Protocol.Models)
Content Source: xml/Microsoft.Azure.Batch.Protocol.Models/CloudServiceConfiguration.xml
Service: batch
GitHub Login: @erickson-doug
Microsoft Alias: douge

@matthchr - Hi there! I'm the docs author at the Azure .NET SDK docs site, and a customer submitted this issue with documentation generated from the /// comments in your API. Can you please help us find the right person to resolve it? Please be sure to @ them in your response. Thanks!

Issue moved to Azure/azure-sdk-for-net #4475 via ZenHub


Cannot sort when find a document

Hello,
I tryed a lot of things but I have the save problem. When I execute this next code, I have this problem :
Error: Cannot sort on field(s) "reportId" when using the default index
dbPouchDB.createIndex({
          index: {
            fields: ['reportId']
          }
        }).then(function(result) {
          dbPouchDB.find({
            selector: {reportId: 10},
            fields: ['reportId'],
            sort: [{reportId: 'desc'}],
            limit: 1
          });
        }).then(function(result) {
        }).catch(function(err) {
        });

I look at indexes and I have two. First one is the created index for reportId, and second one is the _all_docs.
When I look at the code from pouchdb-find, I see when the _all_docs index is defined, I cannot sort any field from "find" method.
Any ideas ?
Best regards.

This looks like a bug. Can you write a test case to reproduce (e.g. a live HTML file)? Glancing at your code I feel like we already have a number of tests that cover this case, so I'm not sure why it's failing.

I cannot get you a live HTML file because I use angularJs and a lot of modules I will show you.
I'm using these dependencies :

"angular-pouchdb": "^4.3.0",
"pouchdb-find": "0.10.3"

In my app.js, I wrote :
var app = angular.module('moduletest.app', [   ...   'pouchdb',   ...
In my file that I use pouchDB object :
`
angular.module('test.reports').service('TestReports', TestReports);
TestReports.$inject = ['$q', '$log', 'Database', 'Constants', 'pouchDB'];
function TestReports($q, $log, Database, Constants, pouchDB) {
function getMaxReportId() {
  var deferred = $q.defer();

  // Storage PouchDB
  var dbPouchDB = pouchDB('my_database');
  if (!dbPouchDB) {
    console.log('Error pouchDB');
  } else {
    dbPouchDB.createIndex({
      index: {
        fields: ['reportId', 'type']
      }
    }).then(function(result) {
      return dbPouchDB.find({
        selector: {
          type: 'report',
          reportId: {$type: 'number'}
        },
        fields: ['reportId'],
        sort: [{reportId: 'desc'}],
        limit: 1
      });
    }).then(function(result) {
      if (result.docs.length === 0) {
        logger.trace('Max reportId not found!');
        deferred.resolve(parseInt(10));
      } else {
        deferred.resolve(parseInt(result.docs[0].reportId));
      }
    }).catch(function(err) {
      logger.trace('Error : ', err.message);
      deferred.reject(err.message);
    });
  }
  return deferred.promise;
}

}
`
With this code, I have the problem I told you in the last post.
When I execute the next code to see indexes, I see two indexes.
Here is the code :
dbPouchDB.getIndexes().then(function(result) { 	console.log(result); });
And the result is (here is the png):
"_all_docs"
"_design/idx-1307a71464d6d5b4f4460b6b9f983c38"
Sorry for the late response, but I hope you can help me about this problem for me.
Best regards.

I'm sorry, without a live test case it's really hard to deduce the error. I'll try to reproduce your error when I get a chance but I can't promise I'll be successful.
BTW I notice that you are using deferreds and that in your first code sample you forgot a return inside the promise chain. You may benefit from this article on promises which points out some common promise mistakes and how to fix them: https://pouchdb.com/2015/05/18/we-have-a-problem-with-promises.html

Yes, I know. You'll try to reproduce and we'll see.
Just to understand with promises. You told me I forgot a return inside the promise chain, but I don't understand even if your post "we-have-a-problem-with-promises" is very good. I have this code :
dbPouchDB.createIndex(...).then(function(result) {    return dbPouchDB.find(...); }).then(function(result) {    ... }).catch(function(err)) { });
Where is the problem ? Maybe, if I arrive to fix the return problem, maybe I will resove my problem on document sort.
Thanks a lot for this problem.

@opaulet92
dbPouchDB.createIndex({
          index: {
            fields: ['reportId']
          }
        }).then(function(result) {
          // ----------> MISSING RETURN HERE <-------------
          dbPouchDB.find({
            selector: {reportId: 10},
            fields: ['reportId'],
            sort: [{reportId: 'desc'}],
            limit: 1
          });
        }).then(function(result) {
        }).catch(function(err) {
        });

I'll try to reproduce but it's much easier when you provide a live test case, e.g. a JSBin.

Hello,
I found the problem. It was not because I forgot a return (just I forgot it when I wrote this test case).
In fact, It is a problem from 'sort' parameter on find method. For example, here is an example that It's works :
dbPouchDB.find({             selector: {               updateDate: {$gt: ''},               type: 'report',               customerId: String(customerId),               deleteDate: {$type: 'null'}             },             sort: [{updateDate: 'desc'}],             limit: limit           });
In fact, when I was debug the source code, I found there is 3 rules :

the sort and index parameters must be same and sort.length must be <= index.length
the index and selector parameters must be same
match field "selector[1st index]" must be [eq, gt, gte, lt, lte].

It is really weird thoses rules. In fact, in my example, I must write in selector "updateDate: {$gt:''}".
So, this is in fact this issue : https://github.com/nolanlawson/pouchdb-find/issues/137

Hi, I am having the same issue. My code:
dbFactory.createIndex({
    index: {
        fields: ['chapter']
    }
}).then(function(result) {
    return dbFactory.find({
        selector: { book: 'levenslied', chapter: {'$exists': true}},
        sort: [{ chapter: 'asc' }]
    })
}).then(function (res) {
    log.info('great!')
}).catch(function (err) {
    log.error(err)
});

I have read your blog about promises and returns and I think the code above should work. I also incorporated @opaulet92 suggestion by adding chapter: {'$exists': true}} to the selector.
@nolanlawson could you point me in the direction of the right solution to this? Manually sorting afterwards is really not an option.

You need to add book to your index and then sort on both
All misspelling thanks to my iPhone.
On Tue, Sep 6, 2016 at 1:28 PM +0200, "Harm Doedens"<EMAIL_ADDRESS>wrote:
Hi, I am having the same issue. My code:
dbFactory.createIndex({
index: {
fields: ['chapter']
}
}).then(function(result) {
return dbFactory.find({
selector: { book: 'levenslied', chapter: {'$exists': true}},
sort: [{ chapter: 'asc' }]
})
}).then(function (res) {
log.info('great!')
}).catch(function (err) {
log.error(err)
});
I have read your blog about promises and returns and I think the code above should work. I also incorporated @opaulet92 suggestion by adding chapter: {'$exists': true}} to the selector.
@nolanlawson could you point me in the direction of the right solution to this? Manually sorting afterwards is really not an option.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

thanks @garrensmith for your suggestion. I have added book to my index and I am sorting on both now:
dbFactory.createIndex({
    index: {
        fields: ['book', 'chapter']
    }
}).then(function(result) {
    return dbFactory.find({
        selector: { 'book': 'levenslied', 'chapter': {'$exists': true}},
        sort: [{'book': 'desc'},{ 'chapter': 'asc' }]
    })
}).then(function (res) {
    log.info('great!')
}).catch(function (err) {
});

But now I am getting the same error as in #153
Any Ideas?

@garrensmith adding book to the index and sort, was not the solution. I found #140 to state that '$exists': true != '$gt': null. So I replaced the former with the latter and now it works!
Thanks anyway for your suggestion!

Great. Can you close the issue
On 6 September 2016 at 16:15, Harm Doedens<EMAIL_ADDRESS>wrote:

@garrensmith https://github.com/garrensmith adding book to the index
and sort, was not the solution. I found #140
https://github.com/nolanlawson/pouchdb-find/issues/140 to state that '$exists':
true != '$gt': null. So I replaced the former with the latter and now it
works!
Thanks anyway for your suggestion!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nolanlawson/pouchdb-find/issues/204#issuecomment-244963505,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAK9AvhGVkDVf2WGUL-NRKDYRkj-DZaoks5qnXV8gaJpZM4JU7TY
.

Sorry I can't close it, I don't see a close button, maybe the OP can.
Also I think the workaround is quite awkward and may be a bug?

Hello,
I'll come back in fiew weeks. I'll try to find out my problem with the other answers.

All is fine.
Thanks.


Fixed subsite support

Array not string was being passed into $record->extend('MetaTags', $tags)

Hello @chtombleson we would want to keep it as an array going forward as then you can change any of the meta tags with referring the array keys.