[Bug] You cannot call the model's crud column like this "example_id" (I think only if it doesn't mean a relationship)

Bug report
What I did
I named a column in my crud table "update_id" and this does not mean a relationship.
php artisan make:migration:schema create_test_table --model=0 --schema="update_id:integer:unique"
php artisan migrate
php artisan backpack:crud test
I'm trying to open the page of this crud
What I expected to happen
The crud page should open without errors
What happened
The page opens, in the upper right corner there is a warning that the page has not loaded and a request to refresh the page. Table elements were not loaded.
What I've already tried to fix it
I have tried manually connecting the columns

protected function setupListOperation()
{
    //CRUD::setFromDb(); // columns
    CRUD::addColumn(['name' => 'update_id', 'type' => 'number']);
}

Got error

Call to a member function getRelated () on bool
Backpack\CRUD\app\Library\CrudPanel\CrudPanel::Backpack\CRUD\app\Library\CrudPanel{closure}
vendor/backpack/crud/src/app/Library/CrudPanel/CrudPanel.php:357

Backpack, Laravel, PHP, DB version
When I run php artisan backpack:version the output is:

PHP VERSION:
PHP 8.0.3 (cli) (built: Mar 31 2021 10:09:09) ( NTS )
Copyright (c) The PHP Group
Zend Engine v4.0.3, Copyright (c) Zend Technologies
LARAVEL VERSION:
v8.34.0@81892ca110795a9c46c7e198cba7763bfd2af0bf
BACKPACK VERSION:
4.1.40@ecbe7676535d3c9121227329d62cad9978d29fee

Hello @larowka
Thanks for the report. It's something we are aware of, unfortunatelly the model has an update() method, so for Backpack, the criteria to be a relationship (or a possible one) are meet.

Name contains someting_id
There is a method in model that is something().

Interestingly I had a conversation with @tabacitu about this a few days ago, and we definatelly don't like the way it is right now.
So our plan for the next major version is to introduce a breaking change, that to work with auto crud functionalities, we will add to the checklist:

Check the return type of something() to see if it is a relation.

So it will be mandatory for developers to setup the relations like:
public something() : BelongsTo
{
return $this->belongsTo();
}

In the meanwhile, your solution is the only available we can provide.
Sorry for the trouble, we expect to launch the next version in the next months.
I am going to close this, let me know if I can help you with something else.
Best,
Pedro


Task stage listeners

Now that we've broken down each segment processing into stages, a useful API would be listeners that can be notified after each stage is complete. Then they can look at (and alter) the state of the segment, do any needed processing, and store state in the Execution attributes ( #31).
Listeners may be arbitrary annotated methods:
   @AfterSourcesMapped
   void afterSourcesMapped(Execution exec, CreateOrUpdateSegment<MyType> segment);

As a part of this task we'll implement the following listeners:
@AfterSourceRowsConverted
@AfterSourcesMapped
@AfterTargetMatched
@AfterTargetMerged
Not sure if we care about @AfterTargetCommited, so skipping it now


BulkUpdate - too many server round trips

Using Sql Profiler I noticed that for a BulkUpdate, 6 server trips are done:

Identify PK - this can be extracted from EF meta information
The Temp table is created - the query is strange: from MyTable t left join MyTable source on 1=0
Returns table collation - ??
Bulk insert - The creation of the temp table and the insertion could be done at the same time
6..  Merge, Drop table - these could be done in one server trip

Can you write an example of reading IdentifyInfo from EF.
It does look unusual but creates what is required:
https://dba.stackexchange.com/questions/916/how-do-i-copy-a-table-with-select-into-but-ignore-the-identity-property
Can't find where that is called from?
Not sure about this since Creating is using pure SQL and Inserting is done using SqlBulkCopy that requires destination table already to exist.
This could be put in single query but I don't think gain would be significant since in Bulk Ops it's not the call to Sql but the amount of Data that was the bottleneck. Also for Readability, and Testing purposes keeping it separate is beneficial.

I haven't tried this on EF, just on NHibernate and MongoDb.
The guys from zzzprojects are compiling the expression to extract the Id but it must be a better way.
After a quick search:
https://stackoverflow.com/questions/15718301/how-i-can-read-ef-dbcontext-metadata-programmatically


Direct Line Attachment File Extensions

Whenever I try to get an attachment from Direct Line, it shows up in the browser's download manager with the name '0', despite there having been a name assigned to the file on the bot server. There is also no extension.
Is there anyway to be able to download the file with the proper extension from the Direct Line API?

hi, @XelaMon
I also want to use Direct Line API, but I couldn't get messages from bot, may you have a look at #672 ?
Thanks.

Is your bot talking the v1 protocol or the v3 protocol?

v3 protocol

@dandriscoll I'm looking at moving to v3 in the near future, but as of now my bot is on v1. If I can see attachment extensions if I upgrade to v3, I'd definitely do that!

Hi @XelaMon have you had a chance to update to v3 ?


2 (or more lines) of text on button?

When the text for the buttons are too long, it truncates it all.  Is it possible to reduce the size of the font and go to 2 lines?

@craigpearce5  Feel free to send Pull Request

If anyone implements this, could I request/suggest that it be an Option.  I wouldn't want the font size to get unreadable when the filename is long.  I bet we could get two lines without making the font smaller.

I'm still pretty new at swift, but I'm guessing it would be accomplished by adding some code here.
private func addButton(title:String)->SCLButton {
    // Update view height
    kWindowHeight += kButtonHeight
    // Add button
    let btn = SCLButton()
    btn.layer.masksToBounds = true
    btn.setTitle(title, forState: .Normal)
    btn.titleLabel?.font = UIFont(name:kButtonFont, size: 14)
    contentView.addSubview(btn)
    buttons.append(btn)
    return btn
}

Perhaps btn.titleLabel?.numberOfLines = 2?
Don't have my mbp with me otherwise I would test that.


Can't call to_hash with nested Sequel::Postgres::JSONBHash

See the code below:
> Hashie::Mash.new(blah: Sequel::Postgres::JSONBHash.new({})).to_hash
ArgumentError: wrong number of arguments (given 1, expected 0)
from /home/braulio/.rvm/rubies/ruby-2.7.5/lib/ruby/2.7.0/delegate.rb:343:in `to_hash

A workaround is to convert Sequel::Postgres::JSONBHash with to_hash before using it inside Hashie::Mash

@jeremyevans FYI

Possibly expected as JSONBHash is a delegate class of Hash, not a subclass of hash.  Unless hashie is expected to convert delegate classes of Hash, this is unlikely to be a bug in hashie.

I agree with Jeremy here. Since that class is not a Hash, it's unexpected I put to Mash's constructor.


Modifies typings according to new pattern

Hey guys, I'm sorry for creating a new PR for the same issue #34 .
Today I checked with the guys from the project https://github.com/typings/registry, when I tried to registry this project in their list of typings and according to them, because the project is hosting the typings there's a better way to do that.
Which is by adding the typings property on the package.json file and that required me to make some changes on the typings definition.
I've tested on my local project and this is working.
Also, because the typings are on the project and registered on the package.json file there's no need for an external tool to install the typings.
Again, I'm sorry for creating a second PR

Coverage remained the same at 90.59% when pulling 3480ff4d9c6d7079620e7a2268d412ef6a4e490d on vintem:develop into d8bc4ee16a3a9dd0496b0c9f334c4b0204d7985b on furqanZafar:develop.

Thanks for the PR, good job :+1: , been super busy lately, hope it wasn't too difficult to figure out the types

Actually, it was very easy. I followed the API documentation for that.

Just wondering...Why was this merged on to develop and not master?

Following up from @atifsyedali's comment - has this been pushed to master?

plz


Use Reflection before invoking the method

Invoking directly the method code can lead to unexpected code execution. Before executing the method a great filter could be the use of ReflectionFunctionAbstract::getReturnType. If not mandatory i think it sould be at least configurable.
Anyway, great package!

Good idea, can you create a PR for this?


Make AI suggested data for Roads visible at lower zoom levels

Description
Being able to review large areas of the map and see where AI suggested road data are located, could be very handy to speed up, data creation in OSM.
Please I clarify: being able to SPOT where the data is, Not to edit the data, that will require zooming to an editable zoom level.
Thanks

Custom Data also needs to be seemed at lower zoom levels


[L'Empire des Cerisiers V1.1] Add Magic tab and minor updates

Added : magic tab
Added : simple roll at the top of the sheet
Updated : changing an injury level now update the others accordingly
Fixed : skill aspect name not updating when source aspect is changed

Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
[x] Is this a bug fix?
[x] Does this add functional enhancements (new features or extending existing features) ?
[x] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

The pull request title include the sheet name and version.
There is a fix on a non blocking bug.
A new feature has been added (new tab). It does change the layout a little.
No attributes were changed or removed in the process.


Rendering child behind parent

Hi,
This is not an issue, but I need to know if there is a "trick" to render children sprites behind a parent sprite ? (without using a container)
Thank you :)

No, there is no trick to drawing a container's children behind the parent AFAIK.


rviz crash on start OSX 10.11.3

$ roscore &
... logging to /Users/Jackson/.ros/log/7d310e85-cdc1-11e5-b7bf-a0999b17aec5/roslaunch-lapfeet.local-27744.log
Checking log directory for disk usage. This may take awhile.
Press Ctrl-C to interrupt

started roslaunch server http://lapfeet.local:59340/
ros_comm version 1.11.16

SUMMARY
========

PARAMETERS
 * /rosdistro: indigo
 * /rosversion: 1.11.16

NODES

auto-starting new master
process[master]: started with pid [27751]
ROS_MASTER_URI=http://lapfeet.local:11311/

setting /run_id to 7d310e85-cdc1-11e5-b7bf-a0999b17aec5
process[rosout-1]: started with pid [27754]
started core service [/rosout]
rviz
[ INFO] [1454866747.087930000]: rviz version 1.11.10
[ INFO] [1454866747.087977000]: compiled against OGRE version 1.7.4 (Cthugha)
libpng warning: iCCP: known incorrect sRGB profile
libpng warning: iCCP: known incorrect sRGB profile
[ INFO] [1454866747.309860000]: Stereo is NOT SUPPORTED
[ INFO] [1454866747.309961000]: OpenGl version: 2.1 (GLSL 1.2).
libpng warning: iCCP: known incorrect sRGB profile
[ WARN] [1454866747.538880000]: OGRE EXCEPTION(7:InternalErrorException): Cannot find serializer implementation for current version [MeshSerializer_v1.8] in MeshSerializer::importMesh at /tmp/ogre20151005-4615-w3w2k8/ogre_src_v1-7-4/OgreMain/src/OgreMeshSerializer.cpp (line 118)
libc++abi.dylib: terminating with uncaught exception of type Ogre::InternalErrorException: OGRE EXCEPTION(7:InternalErrorException): Cannot find serializer implementation for current version [MeshSerializer_v1.8] in MeshSerializer::importMesh at /tmp/ogre20151005-4615-w3w2k8/ogre_src_v1-7-4/OgreMain/src/OgreMeshSerializer.cpp (line 118)
Abort trap: 6

You're using the indigo version of rviz which requires Ogre 1.8 or 1.9. You're using 1.7.4. You'll need to use the Hydro version of rviz or update to use a newer version of Ogre (>= 1.8). See:

http://answers.ros.org/question/186635/rviz-crashes-on-indigo-osx-due-to-serializer-implementation/


Create DNS SRV record DiscoveryClient

https://stackoverflow.com/questions/10138844/java-dns-lookup-for-srv-records
https://github.com/dnsjava/dnsjava
https://search.maven.org/search?q=g:dnsjava AND a:dnsjava&core=gav
https://javadoc.io/doc/dnsjava/dnsjava/2.1.9

Interested in tackling this


when I  set enableHermes: true, bundleInDebug: true  in build.gridle， an error occurred

this is my setting
project.ext.react = [
        entryFile        : "haul.config.js",
        cliPath<EMAIL_ADDRESS>        extraPackagerArgs: ["--minify false"],
        nodeExecutableAndArgs: ["node", "--max_old_space_size=8192"],
        bundleInDebug: true,
	enableHermes: true,
]

when i run react-native run-android,  an error occurred:
Execution failed for task ':app:bundleDebugJsAndAssets'.
> Process 'command 'node'' finished with non-zero exit value 1

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 16s

error Failed to install the app. Make sure you have the Android development environment set up: https://reactnative.dev/docs/environment-setup. Run CLI with --verbose flag for more details.
Error: Command failed: ./gradlew app:installDebug -PreactNativeDevServerPort=8081
haul.js bundle

Hermes is taking up a lot of memory. Are you sure you have some free disk space? Also try: cd android && ./gradlew clean

Hermes is taking up a lot of memory. Are you sure you have some free disk space? Also try: cd android && ./gradlew clean

yeah, I also try  ./gradlew clean command ,but the error is still there

You can also try to increase your gradle heap size, if that's the case about the memory sortage - to 5g, in build.gradle configuration


Arab and Asia-Pacific regional pages on Wiki

When I clicked CC Arab world(http://arabic.creativecommons.org/) and CC Asia-Pacific(http://creativecommons.asia/) on https://wiki.creativecommons.org/wiki/Affiliates, I was taken to a Lexus car repair website.
Could you fix the links to the original webpages or at least to CC blog for now?
Thanks!

I checked all the other sites I could and found no others with an unusual redirect. I fixed those two.  Leaving the ticket open in case Matt sees anything else that's strange.

I'm working with Rob to get rid of the new spam pages.  Yay!

It seems like an old IP address for an old website got reused by a car company.


Fix postinstall npm script on Windows

When installing, the build step complains that
'babel' is not recognized as an internal or external command, operable program or batch file.
I installed babel as a dev dependency in my project and then history was able to install correctly.
Could the build step be changed to run babel from the node_modules instead?
(I'm on Windows btw)

Looks like https://github.com/rackt/history/commit/f4623ba906cea29ac0198dac59b4590c6680aba4 introduced a postinstall script. That in turn invokes a Babel based build. This change was needed by #52. The problem is that if you rely on history through git, it won't contain a correct build (generated by prepublish).
I suppose ideally it shouldn't trigger postinstall if you are treating history as a regular dependency. If it's possible to detect how it was installed (from git or in a normal way), then you can skip Babel build altogether (it will point at lib in this case as it should).

Are you installing from a git repo, @bryaneaton13?

@mjackson I'm using npm to install it. npm i history -D
It works when I already have babel installed locally or globally, but on a fresh project install (no babel installed anywhere) with history as a dependency, I get this error.

@mjackson The problem is that postinstall, and hence Babel, gets triggered in a regular install as well for some reason. I can see it has a guard in place
node -e \"require('fs').stat('lib', function (e, s) { process.exit(e || !s.isDirectory() ? 1 : 0) })\"

I suppose the question is why that guard might fail in this case. Why else would it try to trigger Babel unless this check fails?

I get an error on that line also before I get the babel error message.

@bryaneaton13 Can you provide more info? Which version of Node are you using? Do you get the same error when running the node -e ... script standalone?

node -v  v0.10.21
npm -v  1.3.11
If I copy and paste that command without  || npm run build, it completes.

@bryaneaton13 Are you running on Windows?

@taion Yep

I think that's the issue - I believe you can't do cmd1 || cmd2 on Windows.

I guess that means cmd1 && cmd2 won't work on Windows either, which means our npm test script is probably broken as well. Nice debugging, @taion.
Now that we know what the problem is, anyone want to make a PR?

cmd1 && cmd2 actually does work on Windows. It's just the || that doesn't work. ¯_(ツ)_/¯

ok, @bryaneaton13. I think if we use && instead of || your problem should go away. Can you please try installing from the fix-postinstall-windows branch?
npm install rackt/history#fix-postinstall-windows

I think that might break installing when the built files are present, though, since you'd be exiting with a non-zero status code. Per https://docs.npmjs.com/misc/scripts#best-practices:

Don't exit with a non-zero error code unless you really mean it. Except for uninstall scripts, this will cause the npm action to fail, and potentially be rolled back.

Bah, good point. Any ideas? Last time I wrote code for CMD.exe was at least
a decade ago.
On Thu, Oct 8, 2015 at 5:53 PM Jimmy Jia<EMAIL_ADDRESS>wrote:

I think that might break installing when the built files are present,
though, since you'd be exiting with a non-zero status code. Per
https://docs.npmjs.com/misc/scripts#best-practices:
Don't exit with a non-zero error code unless you really mean it. Except
for uninstall scripts, this will cause the npm action to fail, and
potentially be rolled back.
—
Reply to this email directly or view it on GitHub
https://github.com/rackt/history/issues/73#issuecomment-146727736.

I'm not super familiar with child_process, because I don't script in Node, but this seems to work:
-    "postinstall": "node -e \"require('fs').stat('lib', function (e, s) { process.exit(e || !s.isDirectory() ? 1 : 0) })\" || npm run build"
+    "postinstall": "node -e \"require('fs').stat('lib', function (e, s) { if(e || !s.isDirectory()) require('child_process').spawn('npm', ['run', 'build']) })\""

Ya, I think you're right. Let's try that.


removing lru failure to read xlsx, fix bug when reading multiple files with wide/long format

This is a quick fix for #26.
@gidden: Happy to update that if you can enlighten me on why this lru_cache fixture might be confused about lists...

The issues wasn't the file name in the arguments, but calling the IamDataFrame() with a list of regions as filter.
Suggest to remove lru_cache for reading IamDataFrame data from xlsx and ixmp.


Logging Middleware

Most version of observe seem to consume to body and creates errors in writing length, these two combinations seem to avoid this.
Not Sure How Valid These Tests are if at all. Recreating the failures of consuming the body is hard. However I found it a couple times in unexpected places.

Update: Have had no time to get back around to this. Basically no full tests but I feel this needs to take a backseat to Multipart. I'd welcome either merging and making better from here, or I'd accept a PR against this branch, or it can hold until I finish my highest priority with Multipart.
Would love ideas.

I'm okay merging it if we know that a body can still be read using it.

How is 80%? Because I'm definitely not positive, but I'm fairly confident. Would love for someone else to check me on it.

I don't know, but I'll try to PR it this weekend with another test and then we'll all know.

Build failure is because master was in bad state when this ran.


ruby safe import in production code

I do not understand this safe import. Does this allow "if bond.testing" ? Why do we need the DUMMY_BOND name, can't we just assign to bond directly?
begin
require 'bond'
rescue LoadError
module BondTargetable
DUMMY_BOND = Class.new { def method_missing(meth, *args); end }.new
def self.included(base); base.extend(BondTargetable); end
def bond; DUMMY_BOND; end
end
end

It doesn't allow for 'if bond.testing' at the moment; good point. I created
DUMMY_BOND so that it isn't creating and instantiating a new class on each
call to Bond, since this is supposed to be for production code, but it's
definitely not ideal. I was hoping to discuss some of the implementation
details with you when we met these week because there are some oddities
that I was hoping to get your opinion on.
Erik
On Wednesday, October 21, 2015, George Necula<EMAIL_ADDRESS>wrote:

I do not understand this safe import. Does this allow "if bond.testing" ?
Why do we need the DUMMY_BOND name, can't we just assign to bond directly?
begin
require 'bond'
rescue LoadError
module BondTargetable
DUMMY_BOND = Class.new { def method_missing(meth, *args); end }.new
def self.included(base); base.extend(BondTargetable); end
def bond; DUMMY_BOND; end
end
end
—
Reply to this email directly or view it on GitHub
https://github.com/necula01/bond/issues/16.


Fixes #166. Use distroless as base image.

Update Dockerfile to use distroless as base image. Brings down the image size from 95.5MB to 59.7MB.
Fixes #166

Thank you for this PR, @siddhant94 👍

We can merge this until that @siddhant94 run the test pass at the localhost

We can merge this until that @siddhant94 run the test pass at the localhost

I tried setting up meshery on master branch but I am unable to make it connect to minikube cluster. Cluster is up and running and kubectl is also set. But meshery shows this as error
/home/sid/.minikube/profiles/minikube/client.crt: no such file or directory (along with various other files in same dir path).
But all these file paths are correct and host system has these.
I have even tried to put this config for meshery by running kubectl config view --minify --flatten > /tmp/meshery/kubeconfig.yaml, so that meshery finds it when I do docker-compose up

@siddhant94 have you tried (this is from a thread in channel meshery, what's your slack handle?):

Marcin Kycia  3:43 PM
after that You will need also upload via GUI (setting icon) kubeconfig exported via kubectl config view --minify --flatten > config_minikube.yaml . Last but not least, if You would like to install istio via meshery (or use other adapeters) You will also need to connect other containers to minikube network. Just execute the following command:
for i in `docker container ls --no-trunc -f name='meshery_' --format "table {{.Names}}" | grep -v NAMES`; do docker network connect minikube ${i}; done;

Also, what about the permissions of the crt files?
(Btw I'm using microk8s instead of minikube.)

for i in docker container ls --no-trunc -f name='meshery_' --format "table {{.Names}}" | grep -v NAMES; do docker network connect minikube ${i}; done;

@mgfeller  This seems to have solved connecting to minikube. I am trying to run meshery as is, then will do the same with adapter's new image, however now

When I install linkerd through GUI, it fails with error Error downloading linkerd binary ..... (Attached screenshot)

While looking at logs before this linkerd binary error I have these
app=linkerd-adaptor ts=2020-12-23T07:16:58.720764801Z caller=level.go:63 level=error code=1001 severity=1 short-description=None long-description="Error creating adapter instance.Error validating kubeconfig.open /home/sid/.minikube/profiles/minikube/client.crt: no such file or directory" probable-cause=None

But these files are present on my host(below snippet), I don't know why are those not accessible to linkerd-adapter from docker (I don't know if some volume flag I am missing)?
➜ cat /home/sid/.minikube/profiles/minikube/client.crt
-----BEGIN CERTIFICATE-----

@siddhant94 after success here, we'll want to take this smaller image to the rest of the adapters. Is this something you can take on?

@siddhant94 I tried to reproduce it on my PC, but it works. I used the default .meshery/meshery.yaml file.
I updated meshery from master, and built mesheryctl in mesheryctl using make make. No special reason for that, just to make sure I have the latest. (Installing the usual way failed a few times in the past, that's when I started building it directly.)

@siddhant94 have we invited you to join the Layer5 community, yet? Please jump in - http://slack.layer5.io - the community is warm. Ping me when you do. You're on-deck in today's Meshery Dev call - https://docs.google.com/document/d/1Dg4Uz9rGTuGHmYZwIt47z1j5x3LmnpxD1M1SgypAkQ0/edit#

@siddhant94 I tried to reproduce it on my PC, but it works. I used the default .meshery/meshery.yaml file.
I updated meshery from master, and built mesheryctl in mesheryctl using make make. No special reason for that, just to make sure I have the latest. (Installing the usual way failed a few times in the past, that's when I started building it directly.)

@mgfeller Thanks for all the help, got it working and was able to test it. So as part of testing I installed linkerd, some sample apps and then uninstalled linkerd. Tested that behaviour was same with  image: layer5/meshery-linkerd:stable-latest.
But I couldn't run SMI conformance test, on both the above mentioned image and with this new smaller(this PR).
For this looking at the logs, meshkit throws ErrInstallSmi, but I could not deduce anything further from logs, will have to look at meshkit's code.

@siddhant94 after success here, we'll want to take this smaller image to the rest of the adapters. Is this something you can take on?

hey @leecalcote, I am little caught up for some days, next month end of first week I can take this up, if it's still unassigned.

@siddhant94 have we invited you to join the Layer5 community, yet? Please jump in - http://slack.layer5.io - the community is warm. Ping me when you do. You're on-deck in today's Meshery Dev call - https://docs.google.com/document/d/1Dg4Uz9rGTuGHmYZwIt47z1j5x3LmnpxD1M1SgypAkQ0/edit#

Apologies for late reply Lee, thanks for the invite. Joining soon, will definitely ping you 👍🏼

But I couldn't run SMI conformance test, on both the above mentioned image and with this new smaller(this PR).
For this looking at the logs, meshkit throws ErrInstallSmi, but I could not deduce anything further from logs, will have to look at meshkit's code.

@siddhant94 I experienced the same a couple of weeks back, didn't manage to find out why, @kumarabd and @utkarsh-pro ?

But I couldn't run SMI conformance test, on both the above mentioned image and with this new smaller(this PR).
For this looking at the logs, meshkit throws ErrInstallSmi, but I could not deduce anything further from logs, will have to look at meshkit's code.

@siddhant94 I experienced the same a couple of weeks back, didn't manage to find out why, @kumarabd and @utkarsh-pro ?

@mgfeller @siddhant94, there is a bug in the previous implementation of the SMIConformance function. That implementation will eventually be dropped as a another implementation of the same was added with this PR.  I hope this will fix the issue.
@siddhant94, would you mind confirming if your minikube issue was similar to https://github.com/layer5io/meshery/issues/2134? If it was then expect a fix soon :slightly_smiling_face:

@siddhant94 did you test with the nonroot image as well (as discussed in #166)? Did it work?

But I couldn't run SMI conformance test, on both the above mentioned image and with this new smaller(this PR).
For this looking at the logs, meshkit throws ErrInstallSmi, but I could not deduce anything further from logs, will have to look at meshkit's code.

@siddhant94 I experienced the same a couple of weeks back, didn't manage to find out why, @kumarabd and @utkarsh-pro ?

@mgfeller @siddhant94, there is a bug in the previous implementation of the SMIConformance function. That implementation will eventually be dropped as a another implementation of the same was added with this PR. I hope this will fix the issue.
@siddhant94, would you mind confirming if your minikube issue was similar to layer5io/meshery#2134? If it was then expect a fix soon

Awesome @utkarsh-pro 👍🏼  And yes, I faced similar issue.

@siddhant94 did you test with the nonroot image as well (as discussed in #166)? Did it work?

Thanks for the reminder @mgfeller, this actually somehow slipped out of my mind. I have added nonroot tag to distroless image and tested, and everything seems to work, just the SMI part does not. Let me know if there's anything we need to do.

@siddhant94 Do not forget sign off, https://github.com/layer5io/meshery-linkerd/pull/170/checks?check_run_id=1607098379

n other adapters, the whole source directory is added, here, files and directories are copied individually (at top level). Is this best practice? Any specific reasons?

hey @mgfeller ,
So based on the discussion in issue thread, I followed the Istio adapter's approach. The argument for this kind of approach is that we build our binary from only those files that are needed. Second, is that FROM, COPY, RUN, CMD each instruction creates a docker layer (reference). This way we can take advantage of docker build cache for faster builds.
PS: Let me know if should elaborate on anything & I guess this would have been the thought behind istio adapter Dockerfile also.

Also, building is done slightly different, modules are downloaded separately here

Same as 2nd point before, to take advantage of docker build cache i.e. this layer would only change if our dependencies gets updated.

Here, the binary is copied to root directory, in some other adapters, it is copied to $HOME/bin

For this I went with Istio adapter Dockerfile approach, but I agree we can change the WORKDIR other than root(/) before copying binary.

Also, adapters use $HOME/.meshery, some create that directory if it is missing, in some, expected directories are created in Dockerfile. Also, adapters download tools and config files and put them somewhere. I think it would be useful to discuss this a bit and handle this in a uniform way.

For those adapters that expect directories beforehand, we can add that to Dockerfile.
Happy to discuss. I'll take a look at other adapters Dockerfile to see if we are unable to apply this approach to other ones.
Let me know any suggestions/improvements in this Dockerfile.

@siddhant94 thanks for your detailed feedback. For adapters using meshery-adapter-library and Viper, Viper writes the configuration to a directory specified in its config, e.g. $HOME/.meshery,  adapters also write the kubeconfig file to disk. Placement of these and other files should work both when running an adapter as a container as well as a process. $HOME/.meshery is already used, so I suggest to add it to the Dockerfile to make it explicit. Also, rather use this than TMP. For containers, the app binary could go to $HOME.
What do you think?

$HOME/.meshery

@mgfeller Apologies for delay in response. So what path are you thinking for storing the binaries for docker image? Just the $HOME  path?
And I am fine with explicit declaration of required dirs(in this case, $HOME/.meshery) in Dockerfile itself.

@mgfeller Apologies for delay in response.

no worries

So what path are you thinking for storing the binaries for docker image? Just the $HOME path(currently it is /, so we need to change this)?

I just think it is practical to collect everything in one place ($HOME), and that the root of this is WORKDIR. I think $HOME is a natural place, as it is also used in code (as we discussed). Also, file permissions and ownership will be uniform. Whether the binary is there or in a $HOME/bin subdirectory, or  $HOME/.meshery/bin is not so important.
I just think it is useful that it is uniform across all adapters and other apps, and in quite predictable places.

hey @mgfeller, updated the PR. And I tested it again, except SMI everything works. Let me know if I missed something or it needs correction.

hey @mgfeller, updated the PR. And I tested it again, except SMI everything works. Let me know if I missed something or it needs correction.

hey @mgfeller, updated the PR. And I tested it again, except SMI everything works. Let me know if I missed something or it needs correction.

LGTM

hey @mgfeller, updated the PR. And I tested it again, except SMI everything works. Let me know if I missed something or it needs correction.

LGTM

Thank you, guys.

Thank you, guys.


Investigate using EBS snapshots to load Elasticsearch data

We've generally advocated for using S3 snapshots to load Pelias data into Elasticsearch. S3 is a nice, reliable data storage service, has great support in Elasticsearch, and is pretty darn cheap.
However, it takes a long time to load the ~400GB of data a full planet Elasticsearch cluster needs.
One approach that has sometimes been considered is to avoid using Elasticsearch's built in clustering functionality, and have each Elasticsearch instance have a complete, identical, separate copy of all data.
This has some potential disadvantages, but has some very notable advantages:

Prevents catastrophic performance loss from shard imbalance
Easy to scale up/down without hurting overall cluster performance during shard rebalancing

Another advantage is that it might be possible to considerably speed up launching of new Elasticsearch instances by using EBS snapshots to store data. EBS volumes can be created quickly from an EBS snapshot.
There are some questions to answer here, but it's worth investigating:

What is the performance of a freshly created EBS volume? My understanding is that while the EBS volume is available quickly, the EBS volume will be copying data from S3 (Where EBS snapshots are stored) silently in the background, and until that's complete, performance is quite a bit worse.
How quickly can Elasticsearch start up from a fresh snapshot? My testing thus far suggests it's only a few seconds, which is pretty great!
Does Kubernetes support creating a Persistent volume from an EBS snapshot? It looks like draft support is at least partly underway

As part of some exploratory work, we looked into snapshots with a complete copy of Pelias (all docker images and data for a full planet build) to run as an AMI. It works, and is easy to use, but there are some downsides:
EBS snapshots are slow to create
The full planet Pelias AMI ends up being 600GB. Just saving the snapshot takes about 3 hours. So iteration cycles are slow during development.
EBS snapshots are expensive to store
Although they are stored on S3, EBS snapshots cost 5 cents per GB-month, or over double the standard S3 storage rate of 2.3 cents per GB-month. So every snapshot costs $13 per month just to store. This could add up quickly.
EBS snapshots have painfully slow startup performance characteristics
Although EBS volumes can be created from EBS snapshots effectively instantly, under the hood, the EBS volume itself is lazy-loaded from S3.
This means that the first access to each bit of data on the EBS snapshot will effectively perform at HTTP request speeds, rather than network disk speeds that can be expected from EBS. In practice this means performance is terrible when first starting up Elasticsearch from a snapshot, and most requests fail the first time.
A mitigation is listed in the AWS EBS documentation (use fio to force-load all data from S3 to EBS, with a command like sudo fio --filename=/dev/nvme0n1p1 --rw=read --bs=128k --iodepth=32 --ioengine=libaio --direct=1 --name=volume-initialize).
However, this still takes an hour or two, during which time performance will be horrible. An Elasticsearch instance that is quick to start up but has bad performance for 2 hours is probably worse than an Elasticsearch instance that starts up in 20-40 minutes but has predictable performance from the start
Next steps
After investigating EBS performance, I believe the biggest bottleneck in Pelias Elasticsearch snapshot loading is actually the 250 MB/sec throughput limit that is enforced on all EBS volumes.
During snapshot loads, this limits the speed of snapshot loading to 250MB/sec * number of Elasticsearch instances. And during replica creation, its 125MB/sec * number of instances (half), because one instance is reading, and another is writing. This explains why loading replicas takes almost exactly twice as long as loading from the Elasticsearch snapshot stored on S3.
So going forward, I believe things to look into to improve Pelias snapshot loading speed from S3 are:

Look into using AWS instances with local NVMe storage, such as c5d, m5d, etc. These instances allow for much more than 250MB/sec throughput on the local disks. EC2 instance network speed and S3 speeds should allow for much quicker loading. This will probably at least slightly improve general Elasticsearch performance as well
Look into using multiple EBS volumes in a RAID configuration. This would only make sense if there is a significant downside to using instances with NVMe storage, but creating multiple EBS volumes with RAID is a relatively simple, tried and true, method for getting around the 250MB/sec throughput limit.


Update git repo links to point to the new Ruby for Good location

I think I've caught them all. Please let me know if I've missed any

Resolves #628


testOperationRedo_smartRoutingDisabled - DRAFT

Windows test FAILed.

Linux test PASSed.

Windows test FAILed.

Linux test FAILed.

Windows test FAILed.

Linux test PASSed.

Windows test PASSed.

Windows test FAILed.

Linux test PASSed.

Linux test PASSed.

Windows test FAILed.

Windows test FAILed.

Windows test FAILed.

Windows test FAILed.

Windows test FAILed.

Linux test PASSed.

Linux test PASSed.

Windows test FAILed.

Windows test FAILed.

Linux test PASSed.

Linux test PASSed.

Linux test PASSed.

Linux test PASSed.

Linux test PASSed.


CF encoding should preserve vlen dtype for empty arrays

[x] Provides a workaround for #7328
[x] Tests added

The idea is that it enables a workaround for #7328, where the dtype metadata can be set to a vlen string so that it doesn't get changed to float.
The line that is changed in the example in #7328 is:
ds = xr.Dataset({"a": np.array([], dtype=coding.strings.create_vlen_dtype(str))})

Thanks @tomwhite for the PR. I've only quickly checked the approach, which looks reasonable. But those changes have implications on several locations of the backend code, which we would have to sort out.
Considering this example:
import numpy as np
import xarray as xr
print(f"creating dataset with empty string array")
print("-----------------------------------------")
dtype = xr.coding.strings.create_vlen_dtype(str)
ds = xr.Dataset({"a": np.array([], dtype=dtype)})
print(f"dtype: {ds['a'].dtype}")
print(f"metadata: {ds['a'].dtype.metadata}")
ds.to_netcdf("a.nc", engine="netcdf4")

print("\nncdump")
print("-------")
!ncdump a.nc

engines = ["netcdf4", "h5netcdf"]
for engine in engines:
    with xr.open_dataset("a.nc", engine=engine) as ds:
        print(f"\nloading with {engine}")
        print("-------------------")
        print(f"dtype: {ds['a'].dtype}")
        print(f"metadata: {ds['a'].dtype.metadata}")

creating dataset with empty string array
-----------------------------------------
dtype: object
metadata: {'element_type': <class 'str'>}

ncdump
-------
netcdf a {
dimensions:
	a = UNLIMITED ; // (0 currently)
variables:
	string a(a) ;
data:
}

loading with netcdf4
-------------------
dtype: object
metadata: None

loading with h5netcdf
-------------------
dtype: object
metadata: {'vlen': <class 'str'>}

Engine netcdf4 does not roundtrip here, losing the dtype metadata information. There is special casing for h5netcdf backend, though.
The source is actually located in open_store_variable of netcdf4 backend, when the underlying data is converted to Variable (which does some object dtype twiddling).
Unfortunately I do not have an immediate solution here.

Thanks for taking a look @kmuehlbauer and for the useful example code. I hadn't considered the netcdf cases, so thanks for pointing those out.

Engine netcdf4 does not roundtrip here, losing the dtype metadata information. There is special casing for h5netcdf backend, though.

Could netcdf4 do the same special-casing as h5netcdf?

@tomwhite Special casing on netcdf4 backend should be possible, too.
But it might need fixing at zarr backend, too:
ds = xr.Dataset({"a": np.array([], dtype=xr.coding.strings.create_vlen_dtype(str))})
print(f"dtype: {ds['a'].dtype}")
print(f"metadata: {ds['a'].dtype.metadata}")
ds.to_zarr("a.zarr")
print("\n### Loading ###")
with xr.open_dataset("a.zarr", engine="zarr") as ds:
    print(f"dtype: {ds['a'].dtype}")
    print(f"metadata: {ds['a'].dtype.metadata}")

dtype: object
metadata: {'element_type': <class 'str'>}

### Loading ###
dtype: object
metadata: None

Could you verify the above example. I'm relatively new to zarr :grimacing:

@tomwhite I've put a commit with changes to zarr/netcdf4-backends which should preserve the dtype metadata here: https://github.com/kmuehlbauer/xarray/tree/preserve-vlen-string-dtype.
I'm not really sure if that is the right location, but as it was already present that location at netcdf4-backend I think it will do.

@tomwhite I've added tests to check the backend code for vlen string dtype metadadata. Also had to add specific check for the h5py vlen string metadata. I think we've covered everything for the proposed change to allow empty vlen strings dtype metadata.
I'm looking at the mypy error and do not have the slightest clue what and where to change. Any help appreciated.

@kmuehlbauer thanks for adding tests! I'm not sure what the mypy error is either, I'm afraid...

xarray/tests/test_coding_strings.py:36: error: No overload variant of "dtype" matches argument types "str", "Dict[str, Type[str]]"  [call-overload]

cc @Illviljan @headtr1ck It's only on py3.9 so we could just ignore...

This seems to be a numpy issue, mypy thinks that you cannot call np.dtype like you do.
No idea why it is only in python 3.9 though...
Might be worth an issue over at numpy with the example from the test.
For now we can simply ignore this error.

Might be worth an issue over at numpy with the example from the test.

https://github.com/numpy/numpy/issues/23886

Might be worth an issue over at numpy with the example from the test.

numpy/numpy#23886

The issue is already resolved over at numpy which is really great! It was also marked as backport. @headtr1ck How are these issues resolved currently or how do we track removing the ignore?

If you want you can leave a comment
But the mypy CI should fail on unused ignores, so we will notice :)

But the mypy CI should fail on unused ignores, so we will notice :)

I've added an ignore for now.

Thanks @tomwhite and all others for getting this in.


Support for importing external types

Is there a way of registering a component that resolves to importing a type from some specified path?
We have some complex types from an external system that get synced frequently. These types are trusted, but we can't create Zod schemas for them, so I was hoping we could import them in the generated schema.

@RamonBalthazar in the dawn of this library we were rewriting a project in our company so that we can automatically generate documentation instead of manually writing yamls. So what I can suggest to you is to figure out a way to create the schema for them and use a tool like https://www.npmjs.com/package/openapi-merge or something similar to build our document and then combine it with whatever you have. I think that would suite your needs best. We do not plan to implement this. Good luck :rocket:


Make the input background white

All other components have a white background, so I made regular inputs consistent. It's visible only when the background color of the custom edit action is different than white.

Codecov Report

Merging #390 into v2.2 will decrease coverage by 1.98%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             v2.2     #390      +/-   ##
==========================================
- Coverage   58.13%   56.15%   -1.99%
==========================================
  Files         247      247
  Lines        4063     4064       +1
  Branches      627      626       -1
==========================================
- Hits         2362     2282      -80
- Misses       1562     1656      +94
+ Partials      139      126      -13

Impacted Files
Coverage Δ

.../frontend/components/design-system/atoms/input.tsx
47.36% <0.00%> (-52.64%)
:arrow_down:

...esign-system/molecules/form-group/form-message.tsx
57.14% <0.00%> (-42.86%)
:arrow_down:

src/frontend/login-template.tsx
55.17% <0.00%> (-41.38%)
:arrow_down:

src/frontend/layout-template.tsx
48.27% <0.00%> (-41.38%)
:arrow_down:

...components/design-system/molecules/message-box.tsx
60.52% <0.00%> (-28.95%)
:arrow_down:

...nents/design-system/atoms/illustrations/planet.tsx
42.85% <0.00%> (-28.58%)
:arrow_down:

...ts/design-system/atoms/illustrations/astronaut.tsx
42.85% <0.00%> (-28.58%)
:arrow_down:

.../design-system/atoms/illustrations/flag-in-cog.tsx
42.85% <0.00%> (-28.58%)
:arrow_down:

src/frontend/store/pages-to-store.ts
75.00% <0.00%> (-25.00%)
:arrow_down:

... and 12 more

Continue to review full report at Codecov.

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


Initial 0.0.1-alpha release

See Changelog for details

Admittedly, I did not check every line in detail, but I'm glad with where this is going. Good job on added Ord, hopefully division will work out with future versions of the stable compiler.


Button

Dear All,
How can I add button to my GearVR framework application (If Possible at all)?
Thanks,

i think you might need to be a little more specific.
one suggestion could be to look at the eye-picking sample and then listen to taps from the touchpad for 'clicking' the 3d object you're pointing to.

gvrf_controls_sample has some facilitators related to button, interaction and things like that, take a look, specially in the ControlSceneObject.
https://github.com/Samsung/GearVRf/blob/805bedebb3eb957539cb649c2afee426615a57dd/GVRf/Sample/gvrf_controls_sample/src/org/gearvrf/controls/focus/ControlSceneObject.java

@mrgzadeh did the two samples mentioned help you? Do you need further help? If not, can we consider this issue closed?

I assume it is all good. @mrgzadeh please open another issue if necessary.


TypoScript incorrectly recognized as TypeScript

It looks like my TypoScript files (.ts) are being incorrectly recognized as TypeScript.
http://en.wikipedia.org/wiki/TYPO3#TypoScript
http://docs.typo3.org/typo3cms/TyposcriptReference/

Why was this Closed? More than 3 Years later TypoScript is still missing.

Becaus no one made a PR:

These will be misidentified without TypoScript added to https://github.com/github/linguist/blob/master/lib/linguist/languages.yml. Closing this, but we'll take a look at PR adding the language.

Please see the guidelines to add support for a new language to Linguist. If TypoScript meets the requirement on in-the-wild usage, don't hesitate to open a pull request.

I don't understand what in need to do on step 2

You need to add a grammar for syntax highlighting if you know one, with the command indicated.

but what is this grammar? Where can i get one, or how can i create one?

For grammars, you can use TextMate bundles or Sublime Text/Atom packages. Please see the respective editor's documentation for more information.


Add support for package a installer

Description
Since the build app has a lot files and the user maybe create a shortcut on the desktop manual, that's not quit convenience and is also not release or share to anyone else.
Details

add a -p, --package option to create a installer.
run nativefier -p http://<your_web_app_url> to create a platform related installer.

ps: I tried to package the build app using electron-installer-windows or windows-installer, but all end to the The Product/@Version attribute problem, see https://github.com/jiahaog/nativefier/issues/674 for more details.

@liudonghua123 thanks for the feedback but I don't think that's Nativefier's job. Closing.


tools: Fix message-screenshot.js failing to take screenshot.

To take screenshots, we log in as Desdemonia who doesn't seem
to have permissions to the streams where we capture screenshots.
This commit fixes it by logging in as Iago.
Also added a waitForNavigation until DOMContentLoaded to
avoid flakes that could occur because of us going to another URL
before Iago gets logged in.

Testing Plan:
https://circleci.com/gh/chdinesh1089/zulip/tree/message-screenshot
GIFs or Screenshots:

Thanks for the quick fix here!  Merged.


Unable to sign in to ChatGPT

I'm running Supermium 122 on Windows XP, and for some reason, it doesn't seem to show the ChatGPT sign in page. How come??

It's works.
Are talking about the blank page when opening a site? It should works normally after wait for around a 1 minute.

I'm running Supermium 122 on Windows 11 and can confirm the same issue. Didn't have this problem for the previous versions.
Once you click on "Log in":

... you just get taken to a white page, which doesn't go away even after 30 minutes.

Windows 10 test VM is working, as is my XP x86 test VM, but XP x64 test VM is not.
The console reads:
Failed to load module script: Expected a JavaScript module script but the server responded with a MIME type of "text/html". Strict MIME type checking is enforced for module scripts per HTML spec.
Looks like a longstanding issue for OpenAI:
https://community.openai.com/t/blank-page-at-the-login-of-the-chatgpt/383878/21
And the solution specified today works on that "bad" VM, which is to change auth.openai.com to auth0.openai.com when encountering the blank login page.


Nearest Neighbors

Thanks for this, very handy!
I was wondering whether it is possible to export/print, for each residue in a chain, the site (number) of the residue that was selected for the nearest neighbor to the virtual center in the 3Di state assignment? If this can already be done, can you let me know how? if not, is it possible to introduce such a feature?
Many thanks

Hi there,
Actually, no worries, I have done a little mod and exported the relevant array.
Best wishes
David

Hi David,
I have added an encoder class to do that, you can use the PartnerIndexEncoder to get the neighbor indices.


Cannot make according to the doc

Environment:

eKuiper version (e.g. 1.3.0):
Hardware configuration (e.g. lscpu):
OS (e.g. cat /etc/os-release):
Others:

What happened and what you expected to happen:
make command should be successful.
How to reproduce it (as minimally and precisely as possible):
Refer to doc - https://github.com/lf-edge/ekuiper#build-from-source
Clone code, and type make command.
make
make: *** No targets specified and no makefile found.  Stop.

make pkg
make: *** No rule to make target `pkg'.  Stop.

Anything else we need to know?:

Invalid issue, close it.


[Fix-5699][UI] Fix create user error in user information introduced by #5700

Purpose of the pull request
[Fix-5699][UI] Fix create user error in user information introduced by #5700
This closes #5699
Brief change log

Fix create user error in user information introduced by #5700

Verify this pull request
This change added tests and can be verified as follows:

Manually verified the change by testing locally.

@kyoty

Codecov Report

Merging #5735 (3805348) into dev (2ba569a) will increase coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                dev    #5735   +/-   ##
=========================================
  Coverage     45.35%   45.35%
+ Complexity     3687     3686    -1
=========================================
  Files           607      607
  Lines         24857    24857
  Branches       2825     2825
=========================================
+ Hits          11273    11274    +1
+ Misses        12501    12499    -2
- Partials       1083     1084    +1

Impacted Files
Coverage Δ

...er/master/dispatch/host/assign/RandomSelector.java
77.77% <0.00%> (-5.56%)
:arrow_down:

...dolphinscheduler/remote/future/ResponseFuture.java
81.35% <0.00%> (-1.70%)
:arrow_down:

...olphinscheduler/plugin/alert/email/MailSender.java
74.30% <0.00%> (+1.38%)
:arrow_up:

...lphinscheduler/rpc/common/AbstractRpcCallBack.java
100.00% <0.00%> (+100.00%)
:arrow_up:

Continue to review full report at Codecov.

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


hmac_sha256: Presto throws while Velox doesn't

Description
Query: select hmac_sha256(c0, c1) from (values (cast('12345' as varbinary), cast('' as varbinary))) t(c0, c1);
Presto-0.289: Throws an exception "Empty key".
Velox: Returns non-empty results.
Error Reproduction
No response
Relevant logs
No response

The same issue applies to hmac_sha1 and hmac_sha512.
select hmac_sha1(c0, c1) from (values (cast('12345' as varbinary), cast('' as varbinary))) t(c0, c1);
select hmac_sha512(c0, c1) from (values (cast('12345' as varbinary), cast('' as varbinary))) t(c0, c1);


fix: attempt to export generic types differently

@jwallet - taken from https://stackoverflow.com/a/73795494/12071652. LMK if this solves the same root cause as #20 for you.

@jwallet : LMK if this solves the typing issues you ran into (esp. with the various Typescript versions you tested)! If it works, I'll merge it in and release so that it can solve it for everyone.


The maven remote repository does not have the latest version of the package for the project

Hi, Is there any plan to package the latest version of maven dependencies to upload to remote maven central repository.
This will be handy for me to add dependency packages to the project, thanks again!

Sorry for not finding the latest mirrors because of the use of third party libraries in the maven configuration.
I changed the configuration of the maven mirror repository and I can already download the latest dependency packages properly.


Palera1n : Unable to perform softboot: e0004051

Device
iPad Pro 12.9
Version
Ios 16.7.1Hellotrie d
Passcode?

[ ] Passcode is enabled.

Start going into detail..
Hello, I tried to jailbreaking iPad 12.9 iOS 16.7.1 using the following commands palera1n -c -f and palera1n -f,  but get the following  : Unable to perform softboot: e0004051.
appreciate any support.
Familiarity with palera1n

[X] I am familiar with palera1n and how to use it!

Familiarity with CLI

[X] I am familiar with the command-line!

I tried again and it worked


Create one or more tutorials

Create one or more tutorials on resistome data analysis using the latest R/Bioconductor methods and release them via the Github repository (vignettes/ subfolder).
Note: the current vignettes are Rmd but we should do them with Quarto (qmd).

Start by creating a short vignette that reads in data and generates some summary and visualization.

Initiate exploratory analysis workflow, and ensure that it will appear on the project homepage articles. We can then discuss how to develop it further. Could be reading the data, providing PCoA visualization and some other useful statistics.
You can also check if you could add initial summary on linking with NDARO?

Should we consider opening another issue for replicating visualization from the study?

Yes, thanks


Getting started section for multiplexed samples

Closes #74.
⚠ Stacked on #83 ⚠
Here I added a section to the Getting started with an ScPCA dataset that includes special considerations for multiplexed samples. I was unsure about the level of detail that we wanted, but based on the two points described in #74, I was sure to include a description of what a multiplex sample is and information on how to filter to the sample of interest.
I also included a statement about how the initial analysis that is described in the beginning of the getting started section for single sample libraries - filtering, normalization, dimensionality reduction - can be performed as usual. Then I included some details about demultiplexing and identifying which cells belong to which samples, explaining that those results are already summarized and providing a link to the SCE file contents page. I then have a chunk of code showing an example of separating by sample into a new object for downstream analysis. Is there anything from this that is missing or not detailed enough?
The last little bit I included was a few external resources for people to use based on the types of demultiplexing that we include. In this PR I'm also adding in the link to this section in the FAQ: What is a multiplexed sample.
Again, I wasn't 100 percent sure on detail and structure here so if there was something else that others had in mind, I am definitely open to it.

Thanks for the review @jashapiro. I incorporated your comments and then specified the actual column names where the results are stored per your comment. Let me know if there is anything else that should be addressed.


Adding methods to delete and search a key

Adding methods to delete and search a key
Overriding create keys and create translations methods to accept only required parameters

Codecov Report

Merging #7 into master will increase coverage by 2.15%.
The diff coverage is 86.66%.

@@             Coverage Diff              @@
##             master       #7      +/-   ##
============================================
+ Coverage     41.61%   43.76%   +2.15%
- Complexity       40       44       +4
============================================
  Files             7        7
  Lines           298      313      +15
  Branches         19       21       +2
============================================
+ Hits            124      137      +13
- Misses          171      172       +1
- Partials          3        4       +1

Impacted Files
Coverage Δ
Complexity Δ

...tlin/com/mytaxi/apis/phraseapi/client/model/Key.kt
47.5% <0%> (-1.22%)
3 <0> (ø)

...ytaxi/apis/phraseapi/client/PhraseApiClientImpl.kt
56.32% <92.85%> (+3.55%)
32 <4> (+4)
:arrow_up:

Continue to review full report at Codecov.

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


Resolve RDS connection issues, again

This PR is a collection of changes that I made will attempting to resolve RDS connection issues.
The main RDS connection resolution is to limit the number of connections that are being created by our database engine object. This is mostly achieved by passing specific parameters to the sqlalchemy create_engine function.
Various logging updates are also included to aid in splunk debugging.
Asana Task: https://app.asana.com/0/1203185253920158/1203344379200929

This all looks good to me. Plus its been working most of the day, so thats good too.


Occassional 500 while developing functions

I can't quite track down a repro but wanted to get this created so I could track it. I've seen this twice in the last hour -- switching tabs (I think?) will result in a long 'box-folding-thingie' and then i'll end up with an error dialog with a bunch of HTML. The useful text says `Error 500
The web app has experienced an unexpected service error. We apologize for the inconvenience. Try to reload the page or visit it again soon.`
I had my chrome debugger opened last time and I see that a PUT to Queue_To_Document (my function name) resulted in the 500. I also see a few calls to api/vfs/data/functions/secrets/Queue_To_Document.json are return 404s. Those requests say {"Message":"'D:\\home\\data\\functions\\secrets\\Queue_To_Document.json' not found."} I'm not sure if those are related...

Anything interesting in D:\home\LogFiles\Application\Functions\Host? Look for most recent file.

Nothing jumps out at me. The only errors I see are some standard binding errors where I had my parameter name wrong. I don't have the timestamp of my original error so I can't correlate it to anything in there, but if it happens again I'll capture that.

Need a repro to move this forward.

Closing since there is no repro.


kitti gt interpolated for training?

Hi,
Do you finetune on kitti using the sparse GT as it is and interpolate predictions only for visualization or are the depth maps interpolated for training itself too (seemingly from the Resize transform method)?

I am wondering the same thing.


Promise pipe redux

The original pipe implementation made a few assumptions:

it should be used outside of arenas => allocation was expensive
pipes would be fairly deep

With our expected implementation of the filter stack, pipes will max out at one to three stages, and so optimizations enabled by Filter seem to be actual pessimizations for our usual case.
Additionally we'll be operating inside of arenas and consequently memory allocation can be trivially cheap - and in doing so we can enable a far simpler implementation.
Builds on #28313

@yashykt

Also general comment - I think it would be useful to explain the pipe implementation usual case you mention in the PR description in the code itself so we know the assumptions in-line


Adding Grades?

I don't see an obvious area to add my grade to a particular class.  Did I miss it somewhere?

Grades are not entered by students. Instead, I scrape them from the following url after every semester:
http://www.irp.gatech.edu/reports/grade_distributionsmry.php

I would consider adding this information on the grades page, as I spent a couple of minutes frantically clicking through the tool trying to find the input field :)


fix: add missing allowBatchedQueries option to MercuriusGatewayService type

Do you want to merge it then?
I don't have permission to do that

Only those with write access to this repository can merge pull requests.


[Feature Request] Whisper/yell if an account is flagged

Now that this anticheat measure is a thing, warnings about flagged accounts would be appreciated, if it's possible with the API.

Atleast log it into the account details and give an option to rotate blinded accounts out.

Should be doing the same thing as "Banned" and "Captcha"

Don't know what's the response from the API, and I'm just a beginner on Python, but shouldn't something like this in the worker.py do the job:
In def get_player:
            if get_player.warning: # assuming the response is 'warning'
                raise ex.FlaggedAccountException

And in def visit:
    except ex.FlaggedAccountException:
            self.error_code = 'BANNED'
            self.log.warning('{} is shadowbanned', self.username) # 'shadowban' seems to be the popular term
            await sleep(1, loop=LOOP)
            await self.remove_account()

Or if there's no need for a different kind of warning, just use the same exception as is used with the normal ban response:
            if get_player.banned or get_player.warning:
                raise ex.BannedAccountException

When I pulled the lastest, some of my bots are reporting [W0: Bad Request]
Scan.log:
[2017-05-22 22:04:13][ WARNING][worker-0] (bot0) received code 3 and is likely banned. Removing until next run.
Which I think means they are flagged with that new system, so it IS sort of finding this out.

I haven't seen those kind of errors yet. But I launched the scan with that get_player.warn addition as in my example, and judging by the exceptions, the 'warn' response seems to be the right one. If that's the case, it seems most of my accounts are flagged.

The latest version of pogeo (or whatever) seems to understand this, as it logs out some of my accounts that  are flagged.
Does monocle have a timed rotation for accounts? Can I say after "X hours of being active, log out and pick a different account from the pool" ?


add iptables --flush to reset role

installation sets up mayny iptables rules, this commits flushes rules when running the reset role/playbook

I signed it

It would be nice to squash 2 commits into one

ci check this


Update the server platform's detect.py to match x11's

This adds support for LTO, UBSAN/ASAN/LSAN, debugging symbols options and optimize=size in the server platform.

Thanks!


Make script as a lib

Hi,
Sorry for the delay, here is the PR about make the script as a simple lib to make it easier to import
I also use a dedicated logger in the script to that logs are catch by higher python loggers.

@Seb-Solon - there's some conflicts. also, in the download instructions in the README it has an inconsistency in the file name. would you be ok calling it /usr/local/bin/delete_docker_registry_image.py in the repo but giving instructions in the README to download it to /usr/local/bin/delete_docker_registry_image? that would be my slight preference, so people don't have to change as much, and since it is likely to be used as a straight executable if it's shoved into /usr/local/bin.

Done. Sorry for conflicts! :)

super, thanks!


ListView has been removed from React Native

The ListView component has been removed from the React Native and hence the project is giving an error during build. Attached below is the screenshot of the error.

**

EDIT: The issue has been fixed in version v0.8.0. Please update the dependencies to resolve the issue

**

Hi, @prabhjodhOYO .
Did you find any break-through ?

@KiranUppunda The issue has been fixed in version v0.8.0. Please update the dependencies to resolve the issue.

@prabhjodhOYO what needs to be updated to v0.8.0?

@prabhjodhOYO  please tell me which dependencies to i need to update and the issue is fixed in version v0.8.0 of what i could not get it?

@prabhjodhOYO please tell me which dependencies to i need to update and the issue is fixed in version v0.8.0 of what i could not get it?

@DheereshSinghKarki Upgrade react-native-keyboard-aware-scroll-view to latest version

but i have not installed this package and neither did i used listview in any of my files

@prabhjodhOYO Sir i have not used this package or listview in my code

I have already replaced this library, but every time I see a warning after npm install. Help only removing this in react-native-implementation.js:
Object.defineProperty(module.exports, 'ListView', {
    configurable: true,
    get() {
      invariant(
        false,
        'ListView has been removed from React Native. ' +
          'See https://fb.me/nolistview for more information or use ' +
          '`deprecated-react-native-listview`.',
      );
    },
  });

@prabhjodhOYO your solution solved my issues

@prabhjodhOYO please tell me which dependencies to i need to update and the issue is fixed in version v0.8.0 of what i could not get it?

@DheereshSinghKarki Upgrade react-native-keyboard-aware-scroll-view to latest version

but i have not installed this package and neither did i used listview in any of my files

Remove all the references to native-base. Do a quick search for the same in node_modules. That seems to be the work-around.


feat: Install only required ansible collections

What problem does this PR solve?:
The single largest component (approximately 42%, 331MB out of 791MB) in the KIB image is the ansible collections install.
/ # du -hmd4 | sort -nr | head -n20
791	.
729	./usr
434	./usr/lib
427	./usr/lib/python3.9
384	/usr/lib/python3.9/site-packages/
331	/usr/lib/python3.9/site-packages/ansible_collections

I believe we only need to use two collections (community.general and ansible.posix), which need  about 20MB:
/usr/lib/python3.9/site-packages/ansible_collections # du -hmd2 | egrep "posix|general"
19	./community/general
1	./ansible/posix

Which issue(s) does this PR fix?:

https://jira.d2iq.com/browse/D2IQ-NUMBER

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

Here is that size reduction
➜  konvoy-image-builder git:(dlipovetsky/remove-ansible-collections) docker images | grep mesosphere/konvoy-image-builder
mesosphere/konvoy-image-builder                    v2.0.0-22-g688aeb74a0d4         05a64cb316bd   26 minutes ago   586MB
mesosphere/konvoy-image-builder                    v2.0.0-22-g688aeb74a0d4-amd64   05a64cb316bd   26 minutes ago   586MB
mesosphere/konvoy-image-builder                    v2.0.0-18-gc02ecff5e944         75b5f955ada5   7 days ago       850MB
mesosphere/konvoy-image-builder                    v2.0.0-18-gc02ecff5e944-amd64   75b5f955ada5   7 days ago       850MB

I had made an incorrect change to the derivation of the containerd artifact filename. I've reverted that change, and instead updated the comments, to reduce the change of a similar misunderstanding in the future.

reason for failures in vSphere tests: SRE introduced new datacenter in vCenter for some migration. So looks like the vCenter API does not fall back to the default datacenter and needs all govc commands to specify datacenter explicitly.
I have a fix.

A couple of failures.
Unexpected:
    amazon-ebs.kib_image: failed: [default] (item=nfs-utils) => {"ansible_loop_var": "item", "attempts": 5, "changed": false, "failures": [], "item": "nfs-utils", "msg": "Depsolve Error occurred: \n Problem: package nfs-utils-1:2.5.4-15.el9.x86_64 requires libnfsidmap, but none of the providers can be installed\n  - package nfs-utils-1:2.5.4-15.el9.x86_64 requires libnfsidmap.so.1()(64bit), but none of the providers can be installed\n  - package sssd-common-2.7.3-4.el9_1.3.x86_64 requires (sssd-nfs-idmap = 2.7.3-4.el9_1.3 if libnfsidmap), but none of the providers can be installed\n  - conflicting requests\n  - problem with installed package sssd-common-2.7.3-4.el9_1.3.x86_64", "rc": 1, "results": []}

-- https://github.com/mesosphere/konvoy-image-builder/actions/runs/4255540366/jobs/7403290967#step:9:2847
This looks like a flake:
    amazon-ebs.kib_image: TASK [packages : enable kubelet] ***********************************************
    amazon-ebs.kib_image: fatal: [default]: FAILED! => {"changed": false, "msg": "Unable to enable service kubelet: Failed to execute operation: Cannot send after transport endpoint shutdown\n"}

-- https://github.com/mesosphere/konvoy-image-builder/actions/runs/4255540366/jobs/7403284575#step:9:2479

Depsolve Error occurred:
 Problem: package nfs-utils-1:2.5.4-15.el9.x86_64 requires libnfsidmap, but none of the providers can be installed
  - package nfs-utils-1:2.5.4-15.el9.x86_64 requires libnfsidmap.so.1()(64bit), but none of the providers can be installed
  - package sssd-common-2.7.3-4.el9_1.3.x86_64 requires (sssd-nfs-idmap = 2.7.3-4.el9_1.3 if libnfsidmap), but none of the providers can be installed
  - conflicting requests
  - problem with installed package sssd-common-2.7.3-4.el9_1.3.x86_64

The first failure happened again.
Some issue installing nfs-utils on rocky 9.1 offline-fips. This seems odd, because both rocky 9.1 offline, and rocky 9.1 fips tests pass:
Depsolve Error occurred:
 Problem: package nfs-utils-1:2.5.4-15.el9.x86_64 requires libnfsidmap, but none of the providers can be installed
  - package nfs-utils-1:2.5.4-15.el9.x86_64 requires libnfsidmap.so.1()(64bit), but none of the providers can be installed
  - package sssd-common-2.7.3-4.el9_1.3.x86_64 requires (sssd-nfs-idmap = 2.7.3-4.el9_1.3 if libnfsidmap), but none of the providers can be installed
  - conflicting requests
  - problem with installed package sssd-common-2.7.3-4.el9_1.3.x86_64

Rebased on main after #693 merged.


Error when compiling

I have a waveshare ESP32 S3 8x8 matrix but I get a few errors with this.
Compiling .pioenvs/waveshare/src/esphome/core/entity_base.cpp.o
src/esphome/components/qmi8658/qmi8658.cpp: In member function 'virtual void esphome::qmi8658::QMI8658Component::setup()':
src/esphome/components/qmi8658/qmi8658.cpp:35:13: error: no matching function for call to 'SensorQMI8658::configAccelerometer(SensorQMI8658::AccelRange&, SensorQMI8658::AccelODR&, SensorQMI8658::LpfMode&, bool)'
         true);
             ^
In file included from src/esphome/components/qmi8658/qmi8658.h:8,
                 from src/esphome/components/qmi8658/qmi8658.cpp:1:
.piolibdeps/waveshare/SensorLib/src/SensorQMI8658.hpp:361:9: note: candidate: 'int SensorQMI8658::configAccelerometer(SensorQMI8658::AccelRange, SensorQMI8658::AccelODR, SensorQMI8658::LpfMode)'
     int configAccelerometer(AccelRange range, AccelODR odr, LpfMode lpfOdr = LPF_MODE_0)
         ^~~~~~~~~~~~~~~~~~~
.piolibdeps/waveshare/SensorLib/src/SensorQMI8658.hpp:361:9: note:   candidate expects 3 arguments, 4 provided
src/esphome/components/qmi8658/qmi8658.cpp:40:13: error: no matching function for call to 'SensorQMI8658::configGyroscope(SensorQMI8658::GyroRange&, SensorQMI8658::GyroODR&, SensorQMI8658::LpfMode&, bool)'
         true);
             ^
In file included from src/esphome/components/qmi8658/qmi8658.h:8,
                 from src/esphome/components/qmi8658/qmi8658.cpp:1:
.piolibdeps/waveshare/SensorLib/src/SensorQMI8658.hpp:413:9: note: candidate: 'int SensorQMI8658::configGyroscope(SensorQMI8658::GyroRange, SensorQMI8658::GyroODR, SensorQMI8658::LpfMode)'
     int configGyroscope(GyroRange range, GyroODR odr, LpfMode lpfOdr = LPF_MODE_0)
         ^~~~~~~~~~~~~~~
.piolibdeps/waveshare/SensorLib/src/SensorQMI8658.hpp:413:9: note:   candidate expects 3 arguments, 4 provided
src/esphome/components/qmi8658/qmi8658.cpp:45:42: error: 'IntPin1' is not a member of 'SensorQMI8658'
         qmi8658.enableINT(SensorQMI8658::IntPin1);
                                          ^~~~~~~
src/esphome/components/qmi8658/qmi8658.cpp:52:42: error: 'IntPin2' is not a member of 'SensorQMI8658'
         qmi8658.enableINT(SensorQMI8658::IntPin2);
                                          ^~~~~~~
*** [.pioenvs/waveshare/src/esphome/components/qmi8658/qmi8658.cpp.o] Error 1
========================= [FAILED] Took 14.16 seconds =========================

The config example says
acceleration_odr: 125hz

but that is not valid, using any of the valid values gives me the above error.

Look like I maybe have not tested that setting :(
sensor:
  - platform: qmi8658
    address: 0x6B
    # interrupt_pin_1: GPIO10
    interrupt_pin_2: GPIO13
    acceleration_range: "4G"
    # acceleration_odr: 100hz
    acceleration_x:
      name: "QMI8658 Acceleration X"
    ...

I'll se if I have time to try it out myself.

Actually it seem that SensorLib has that this includes have made some changes, may have to adapt to those
https://github.com/lewisxhe/SensorLib/compare/v0.2.1...v0.2.2
Or you pin-point to the previous version for now
  libraries:
    - "Wire"
    - "SPI"
    - "SensorLib#v0.2.1"

Actually, when I compile mine with the non-versioned SensorLib it works, and I can't even include v0.2.1 (v0.2.0 is my latest) so I might run on older ESPHome version. May be that you have to revert even to v0.2.0 to make it work.
Can you check the beginning of your compile log what version av SensorLib that is pulled and used.

I get this when I compile:
Dependency Graph
|-- AsyncTCP-esphome @ 2.1.4
|-- Wire @ 2.0.0
|-- SPI @ 2.0.0
|-- SensorLib @ 0.2.2

But using 0.2.1 works fine.
Thanks!


Add ability to create multiple filter levels for subset definition (previously called phases)

We should go up to 7 levels.


manifest generator not

When going through the Progressive Web AMPs Codelab  https://codelabs.developers.google.com/codelabs/amp-in-pwa/index.html?index=#4
I encountered problems generating a manifest.json file using the App Manifest Generator @ https://app-manifest.firebaseapp.com/
After filling in the manifest values and uploading the icon as instructed. I downloaded the zip file and extracted the manifest.json file which I then added to the root of my work folder.
But when I inspected the manifest.json file in my code editor this was all I saw.
{
"icons": null,
"splash_pages": null
}
There were also no image icons generated, Just a manifest.json file.
Those were the only values in the manifest.json generated by the Generator.  Could this be an issue with the App Manifest Generator?

i got the same problem!!!, still waiting for the help...


Work out what to do when ispyb is down

If ispyb goes down we should at least:

Alert that we're waiting on it
Queue up any writes to it
Maybe pause before next collection?

Queueing up writes seems like something we should do together with running the ISPyB callback in a separate process, probably a separate ticket to alerting that it's down?


Providing specific configuration for react-with-typescript

What version of this package are you using?
v14.0.0
What problem do you want to solve?
Avoid having to override eslint-config-standard-react with custom rules when using TypeScript.
What do you think is the correct solution to this problem?
Would you consider creating a eslint-config-standard-react-with-typescript the same way you already provide eslint-config-standard-with-typescript? For instance react/prop-types makes great sense when not using TypeScript, but in our case the rule forces us to define our props twice (once as a TS interface, once using prop-types). It would be nice if a react-with-typescript config could disable this rule by default.
Also when using React, it makes sense (at least for us) to configure @typescript-eslint/strict-boolean-expressions with ignoreRhs: true, as we frequently use syntax such as {showFeature && } . When using eslint-config-standard-with-typescript strict-boolean-expressions are configured to not allow the pattern above, so overwriting with eslint-config-standard-react-with-typescript would be a nice way to avoid having do so manually.
There is probably more things where react+ts specific rules could make sense :)
Are you willing to submit a pull request to implement this change?
Yes, but not sure how/where to provide one, as I think a new package would make sense for this configuration.

Thank you for your suggestion, @felskov.
The intention is clear. A single shareable config to extend from for the case of React plus TypeScript.
Question: not using prop types does seem to make sense in a TypeScript end-project. But what about a React component that is published somehow and could be used in JS or in TS? So that seems like two distinct configs. Is there a TypeScript-based runtime prop validation library that operates in a manner similar to https://github.com/pelotom/runtypes? @pelotom, @patomation.
Regarding the strict-boolean-expressions rule relaxing, that could be discussed outside the context of React, because it seems that it could be widely used in a variety of contexts. So if you care about that, could you please open an issue?

Regarding the strict-boolean-expressions rule relaxing, that could be discussed outside the context of React, because it seems that it could be widely used in a variety of contexts. So if you care about that, could you please open an issue?

A discussion on this would be nice. When I first applied the config to some larger react code bases I was quite annoyed with this rule. But I have to say that once I've gotten used to it I think that I quite like it ☺

Question: not using prop types does seem to make sense in a TypeScript end-project. But what about a React component that is published somehow and could be used in JS or in TS? So that seems like two distinct configs. Is there a TypeScript-based runtime prop validation library that operates in a manner similar to https://github.com/pelotom/runtypes?

I can chime in on regarding PropTypes. While I'm not satisfied with this solution there is a plugin called: (babel-plugin-typescript-to-proptypes)[https://github.com/milesj/babel-plugin-typescript-to-proptypes]
The thing I don't care for is that I have to go through babel to trans-pile TypeScript. So right now I use a typescript plugin within babel. The result is generated prop types added at build time.
With Eslint I'm still having to manually disable the react/prop-types rule.
An example of this can be round here: (react-npm-package-starter)[https://github.com/patomation/react-npm-package-starter]
I hope this helps, at least, shed some light into one way of doing things.

Speaking of prop-types btw, are those still recommended by React? I know that react-native is removing them 🤔

I would be interested in knowing if they plan to keep it. 2 years ago @gaearon mentioned that they don't have any plans to remove them. But he also said they don't promote them because flow and typescript "do a better job." I would imagine with the way updating software goes prop-types might get remove simply because the code base changed enough for it to not be worth implementing or maintaining? This is just conjecture, but I would be happy with not including them in the future. What do you think?

I would personally be okay with completely removing the react/prop-types rule ☺

Hello from two years later. Whatever the perceived problem is and whatever the solution may be, it's unlikely to be a change in this project, as this project shouldn't be influenced by React or any other unrelated library.
Please feel free to continue discussion regarding where this discussion belongs and then when you figure that out, please take the discussion there.


Include name of not-found prison in error message

This will assist in tracking down the reason behind 500 errors caused by PrisonNotFound exceptions.

@threedaymonk, @SteveMarshall  - my bad on this one, I should have included it when I created the model.
However the reason for the 500's is because of an empty visit object in the session(redirects on session timeouts is the cause of this) so don't be alarmed when you just see Can't find prison  in the log file/kibana.


Nothing happens after install

Recently I tried this plugin but it did nothing, it didn't load at all. I went back to server configuration.

Recently I tried this plugin but it did nothing, it didn't load at all. I went back to server configuration.

Could you provide the way how you installed coc + nvim version + :CocInfo, please?
The last time I checked it was working correctly. I don't code in zig in a long time tho.

I'm on Arch Linux, currently using nvim nigthly:
## versions

vim version: NVIM v0.7.0-dev+1275-g00effff56
node version: v17.8.0
coc.nvim version: 0.0.80-5830e03d 2022-03-27 04:58:12 +0800
coc.nvim directory: ~/.local/share/nvim/site/pack/packer/opt/coc.nvim
term: tmux
platform: linux

zig was installed from Arch repos, and zls-bin from AUR. I tried to set zig.path but it did nothing too. Using bare language server config for zls works without problems.

Same happened to me. I have introduced the fix in #3
There were two main issues: config parameter had a type and default export was used. This resulted in plugin been not activated.


layer_whitelist_reset() makes the whitelist block all layers

on version 1.5.1 no layers are loading after using layer_whitelist_reset()
instead of reverting to default "all layers load", it makes the whitelist empty, in a way, that no layer are on the whitelist, so no layers load unless layer_whitelist_add() is used

Whitelisting works as intended, Modlich's issue is coming from elsewhere. Closing this.


Restyling some items

Before:

After:

Looks fine to me

Ok, but then also add the "Lock Element Box" option on bottom to the other Options, so that its still available.

It's still there, left from your health actually.

I thought it was placed near the elements to remove confusion as to why
elements were disabled. Could easily just move it to settings though
man. 15. jun. 2015 kl. 10.05 skrev K4r0qtuYNE5G4qgZ <
<EMAIL_ADDRESS>
Ok, but then also add the "Lock Element Box" option on bottom to the other
Options, so that its still available.
—
Reply to this email directly or view it on GitHub
https://github.com/SteamDatabase/steamSummerMinigame/pull/132#issuecomment-111972772
.

Ok, not seen it. Yeah, its just perfect there, so the empty Place is used well.


UDP downstream api

Hi,
Would like to implement a udp proxy server (dns proxy) with stream and upstream modules.
Both send/receive messages are not longer than a single datagram.
I would also like to intercept response from upstream server (That is 'subrequest' in http module)
Would like to code/port missing parts to existing modules.
Any recommendations to where should I start ?
From what I have seen upstream-lua does not support udp and I'm not sure it's usable in the stream/server context and when considering porting then lua-nginx-module
 does not support udp cosockets just yet
thanks

@alonbg I think we can just extend the existing downstream cosocket API of ngx_stream_lua_module to support datagrams. ngx_stream_lua_module does support upstream datagram cosockets though. You can have a look at that for the downstream part (ideally, we should resuse the implementation if possible).

looked at the code and bit confused. Would need some explicit genuineness, a starting push or can help in any other way ( so I guess "we" is good :-). What ever works.
Also when we say 'upstream' that is using the proxy_pass ? And would proxy_bind be in effect ?

@alonbg When I say upstream, I mean anything talking to the backend. When I say downstream, I mean anything talking to the client side. I'm not referring to nginx's upstream facility in this context because I'm talking about cosockets.
Upstream cosocket API:
https://github.com/openresty/lua-nginx-module#ngxsockettcp
Downstream cosocket API:
https://github.com/openresty/lua-nginx-module#ngxreqsocket

@agentzh I understand now. Thank you.
So as for extending existing downstream to support datagram. Code at most part seems agnostic to transport type as this is handled by nginx. Otherwise I do not see where the extensions should go.
As for the upstream I'd like to extend with capabilities similar to nginx 1.11 proxy_bind $remote_addr transparent. see ngx_event_connect_set_transparent. I guess that would require support for a socket operation bind and a working setoption.

@alonbg I guess ngx.req.socket can just return a datagram-typed cosocket if the downstream is datagram-typed.
Regarding bind, yes, we can have a separate cosocket method. Just ensure it is compatible with LuaSocket's API wherever possible ;)

@alonbg Thanks for looking into this!

@agentzh downstream - I assume you mean ngx.req.socket should already be supporting datagram.
I'll check it out. Maybe a good way to start is to add udp tests ? What do you think ?
One location though (not sure) that might require modifications.
upstream - I will look into bind and setoptions (that if you are not already .. ;)

@alonbg No, ngx.req.socket only supports stream-typed socket operations atm. The object methods should be different for datagrams.

@alonbg how do you implement  a udp proxy server finally?

I am interested in basic udp server functionality as well, and progress/timeframe on when this might arrive?

Hopefully in the next few months :)

Patches and volunteers are always welcome :)

@agentzh
some directives,such as set,access_by_lua_block,are being developped,aren't they?
if I set the proxy_pass value in the access phase,I can dispatch my stream to my will, is that right?
Thanks

@daxiong380380 You'll have to rely on the balancer_by_lua* feature proposed in #7 for dynamic dispatching on ngx_stream_proxy_module. The access_by_lua* thing won't help much here since the nginx variables are not supported in nginx's stream subsystem (thus no proxy_pass $target usage).

@agentzh
What do you suggest:

ngx.udp_req.socket (req would be nil)
ngx.req.udp_socket (req.socket would be nil)
or
ngx.req.socket where the returned object can be either udp socket of tcp socket ?

@alonbg I think I like ngx.req.udp_socketthe most :)
BTW, I'm also going to build a simple DNS name server atop this module for our openresty.org site :) Thank you for working on this!

forked here
nginx.conf with both tcp and udp test servers
testing udp: not working ( just yet ... )
client: socat -T10 -t10 -d -d -d - udp:<IP_ADDRESS>:8070
echo server: socat -d -d -d  udp-l:8071,reuseaddr,fork,crlf system:"cat"
testing tcp: working just fine
client: socat -d -d -d - tcp:<IP_ADDRESS>:5354
echo server: socat -d -d -d  tcp-l:5355,reuseaddr,fork,crlf system:"cat"
@agentzh could you please have a quick look
thanks

@alonbg Will you create a pull request on GitHub? This way is easier for us to review your changes. Many thanks!

done

Hello, so far today in November 2018, can we implement UDP server with nginx-lua-stream module
Kind regards

It is already in the master.

@agentzh You mentioned this is in master, but I can't find anything that makes it seem like support exists to read UDP packets coming into OpenResty.
Can you share a link or connection to a PR?

Ah. It doesn't seem documented. https://openresty-reference.readthedocs.io/en/latest/Lua_Nginx_API/#ngxreqsocket says that ngx.req.socket returns tcpsock, but evidently it also returns a socket capable of inspecting UDP packets.
However, seems that :peek() is nil for UDP request sockets, which makes it useless to do any sort of filtering in preread_by_lua_block. Any suggestions?


TreeForeignKey widget

When I use django-mptt I use TreeForeignKey which changes the widget.
In django-filter it still uses the traditional widget. Shouldn't it use the widget specified for the model field if such is specified?

Hi
In django-mptt you specify a foreign key as parent = TreeForeignKey(to='self'). The field TreeForeignKey inherits from the normal field ForeignKey but with a custom widget.
My question was if django-filter shouldn't check the field's widget instead of just assuming that the widget should be the normal widget Select instead of the widget specified in the model fields.
I hope you get what I mean. I know I can set the widget but I think a filter like this:
class MyFilter(django_filters.FilterSet):
    class Meta:
        model = MyModel
        fields = [
            'group',
        ]

should find the correct widget for field group automatically.

OK. I understand. But I'm going to say you need to do this yourself.
DF defines a default mapping between Django provided model fields and DF provided filter classes. Beyond that it's out of scope.
Define a Filter class with the widget you need and declare it on your filter — That's one line. If you're doing it a lot you may want to declare a base FilterSet class for your project declaring filter_overrides with a mptt.TreeForeignKey key.


Minor Enhancement: Left justification and alignment of comments field.

Purely and aesthetic thing: it would be a little easier to read the comments if they were left justified in the default indicator view.

Comments in the alternate table view might be easier to read if they were aligned with the rest of the fields (the text shoots way left after the first line - or maybe I just write too many words haha).

Should be fixed with the latest update. Thanks for the suggestion, I definitely agree it looks better left-aligned.


Add patch for unclosed comment uninitialized access issue

See https://bugzilla.gnome.org/show_bug.cgi?id=746048 for more information about this security issue.

@rafaelfranca Thank you for the patch! We will merge it and release nokogiri-<IP_ADDRESS>.


MySQL index missing

After a few thousand emails are stored in the database the application becomes sluggish, up to the point it just times out.
Creating this index resolves the issue:
CREATE INDEX idx_project_id_message_id ON email(project_id, message_id) USING HASH;

Hello!
Thanks for you suggestion! I'll add it with closest updates

Hello!
Added a new migration with index on email table.
Run ./bin/console doctrine:migrations:migrate -n to apply migration


can't compile from source

after my fail with the pip version of pyav, i tried to get the bleeding edge from git, but i cannot compile it:
Could not find avformat with ctypes; looked in:
libavformat.so.54
but it's installed
$ dpkg --list | grep avformat
ii  libavformat-dev                             6:9.18-0ubuntu<IP_ADDRESS>                    amd64        Development files for libavformat
ii  libavformat54:amd64                         6:9.18-0ubuntu<IP_ADDRESS>                    amd64        Libav file format library
can you give me a hint how to fix this?
ps: i had to modify setup.py a little to even get this message, otherwise it crashed. line 70 should say: for root in extension_extra.get('library_dirs',[]):

Do you mean this line?
Why was it failing?

the error was in line 56 (don't have a clue why i wrote 70):
for root in extension_extra.get('library_dirs'):
the default return value is None, which isn't iterable. and apparently library_dirs does not exist with my setup.
btw, i've found https://github.com/mikeboers/PyAV/issues/28, which is similar but was closed last year

Similar issue with trying to build from source or use pip to upgrade from 0.2.2 to 0.2.3
Collecting av
Downloading av-0.2.3.tar.gz (66kB)
100% |################################| 69kB 2.6MB/s
Complete output from command python setup.py egg_info:
Traceback (most recent call last):
File "", line 20, in
File "/tmp/pip-build-NxRWz8/av/setup.py", line 184, in
if check_for_func(libs, func):
File "/tmp/pip-build-NxRWz8/av/setup.py", line 70, in check_for_func
lib_path = find_library(lib_name)
File "/tmp/pip-build-NxRWz8/av/setup.py", line 56, in find_library
for root in extension_extra.get('library_dirs'):
TypeError: 'NoneType' object is not iterable

I made the tiny fix for that, and then added python setup.py doctor to dump out the discovered config information. Can you post the results of that?

Got it.
~/src/PyAV ±master> sudo apt-get install libavformat-dev                                                                                2 ↵
Reading package lists... Done
Building dependency tree
Reading state information... Done
libavformat-dev is already the newest version.
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.

~/src/PyAV ±master » python setup.py doctor
Could not find avformat with ctypes; looked in:
        libavformat.so.54
Could not find avutil with ctypes; looked in:
        libavutil.so.52
Could not find avcodec with ctypes; looked in:
        libavcodec.so.54
Could not find ['avformat', 'avutil', 'avcodec'] with ctypes.util.find_library
Some libraries can not be found for inspection; aborting!

Heh. Yeah it would abort there, wouldn't it,.. :stuck_out_tongue:

Good news! My brand new Ubuntu 14.04.2 virtual machine is doing exactly the same thing! This should make it a bit easier...

TL;DR: Try again.
It seems like this version combination of Libav/Ubuntu/pkg-config results in pkg-config not returning the library path (perhaps since it is in the default system path?).
So, now we dump the linker script from ld and parse SEARCH_DIR from that.

Do let me know if this works. I'll bundle up a new patch revision for the PyPI, and work on cleaning up the stuff I added in haste later on...

I'm going to assume this works for you; I've released it as 0.2.4.

My apologies. It does. I just built 0.2.4 this morning. Thanks!


Add tests in Docker image for runtime testing

Create Docker image containing Pester tests and the built module for runtime testing. For now this needs to be run manually until integration tests using Docker can be hooked up.

Hi @heaths, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Heath Stewart). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

:white_check_mark: Build vssetup.powershell 1.0.22-rc+g1745c6c678 completed (commit https://github.com/Microsoft/vssetup.powershell/commit/1745c6c678 by @heaths)


keep one minio bucket

Signed-off-by: hang.jiang<EMAIL_ADDRESS>
What type of PR is this?

/kind feature
What this PR does / why we need it:

keep one minio bucket

Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

/lgtm


Support for Protobuf Editions

Protobuf has switched from version to continual evolution of the syntax via "editions", see https://protobuf.dev/editions/overview. The current version of the plugin assumes proto2 and does not recognize the edition keyword, e.g.

JetBrains ships their own protobuf support these days. This plugin has been unsupported for a number of years.

Oops, filed this in the wrong place
On Sun, Jul 28, 2024 at 1:28 PM Jeremy Volkman @.***>
wrote:

JetBrains ships their own protobuf support these days. This plugin has
been unsupported for a number of years.
—
Reply to this email directly, view it on GitHub
https://github.com/ksprojects/protobuf-jetbrains-plugin/issues/166#issuecomment-2254588272,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADVXGNECIW7U4V2YCERSJK3ZOUS2NAVCNFSM6AAAAABLS66BQ2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENJUGU4DQMRXGI
.
You are receiving this because you authored the thread.Message ID:
<ksprojects/protobuf-jetbrains-plugin/issues/166/2254588272
@.***>


More metrics

added global binary ASSD
added precision and recall, those are not printed to the terminal by default

/format