Fatal Error - App Crash Log

Receiving fatal error when accessing just the fragment that the AdvancedWebView should be on... I have AdvancedWebView.java source in it's own file. Not using the main java onCall, just another fragment to display the AdvancedWebView.  Any ideas? The Line 26 it mentions is the rootView = inflater.inflate.... etc etc. line.
Here's the error:
: FATAL EXCEPTION: main
10-18 16:43:36.821 27247-27247/com.test.app E/AndroidRuntime: Process: com.test.app, PID: 27247
10-18 16:43:36.821 27247-27247/com.test.app E/AndroidRuntime: android.view.InflateException: Binary XML file line #6: Error inflating class im.delight.android.webview.AdvancedWebView
10-18 16:43:36.821 27247-27247/com.test.app E/AndroidRuntime:     at android.view.LayoutInflater.createViewFromTag(LayoutInflater.java:764)
10-18 16:43:36.821 27247-27247/com.test.app E/AndroidRuntime:     at android.view.LayoutInflater.rInflate(LayoutInflater.java:813)
10-18 16:43:36.821 27247-27247/com.test.app E/AndroidRuntime:     at android.view.LayoutInflater.inflate(LayoutInflater.java:511)
10-18 16:43:36.821 27247-27247/com.test.app E/AndroidRuntime:     at android.view.LayoutInflater.inflate(LayoutInflater.java:415)
10-18 16:43:36.821 27247-27247/com.test.app E/AndroidRuntime:     at com.test.app.report2_Fragment.onCreateView(report2_Fragment.java:26)

Caused by: java.lang.ClassNotFoundException: Didn't find class "im.delight.android.webview.AdvancedWebView" on path: DexPathList[[zip file "/data/app/com.test.app-2/base.apk"],nativeLibraryDirectories=[/vendor/lib64, /system/lib64]]
10-18 16:43:36.821 27247-27247/com.test.app E/AndroidRuntime:     at dalvik.system.BaseDexClassLoader.findClass(BaseDexClassLoader.java:56)
10-18 16:43:36.821 27247-27247/com.test.app E/AndroidRuntime:     at java.lang.ClassLoader.loadClass(ClassLoader.java:511)
10-18 16:43:36.821 27247-27247/com.test.app E/AndroidRuntime:     at java.lang.ClassLoader.loadClass(ClassLoader.java:469)
10-18 16:43:36.821 27247-27247/com.test.app E/AndroidRuntime:     at android.view.LayoutInflater.createView(LayoutInflater.java:578)
10-18 16:43:36.821 27247-27247/com.test.app E/AndroidRuntime:     at android.view.LayoutInflater.createViewFromTag(LayoutInflater.java:750)
10-18 16:43:36.821 27247-27247/com.test.app E/AndroidRuntime:     at android.view.LayoutInflater.rInflate(LayoutInflater.java:813)
10-18 16:43:36.821 27247-27247/com.test.app E/AndroidRuntime:     at android.view.LayoutInflater.inflate(LayoutInflater.java:511)
10-18 16:43:36.821 27247-27247/com.test.app E/AndroidRuntime:     at android.view.LayoutInflater.inflate(LayoutInflater.java:415)
10-18 16:43:36.821 27247-27247/com.test.app E/AndroidRuntime:     at com.test.app.report2_Fragment.onCreateView(report2_Fragment.java:26)
10-18 16:43:36.821 27247-27247/com.test.app E/AndroidRuntime:     at android.support.v4.app.Fragment.performCreateView(Fragment.java:1962)

Here's the fragment:
package com.test.app;

import android.annotation.SuppressLint;
import android.content.Intent;
import android.graphics.Bitmap;
import android.os.Bundle;
import android.support.annotation.Nullable;
import android.support.v4.app.Fragment;
import android.view.LayoutInflater;
import android.view.View;
import android.view.ViewGroup;
import android.webkit.WebView;

public class report2_Fragment extends Fragment implements AdvancedWebView.Listener {

    private AdvancedWebView mWebView;

    public report2_Fragment() { }
    View rootView;
    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) {
        rootView = inflater.inflate(R.layout.report2, container, false);

        mWebView = (AdvancedWebView) rootView.findViewById(R.id.webview);
        mWebView.setListener(getActivity(), this);
        mWebView.loadUrl("http://www.example.org/");

        // ...

        return rootView;
    }

    @SuppressLint("NewApi")
    @Override
    public void onResume() {
        super.onResume();
        mWebView.onResume();
        // ...
    }

    @SuppressLint("NewApi")
    @Override
    public void onPause() {
        mWebView.onPause();
        // ...
        super.onPause();
    }

    @Override
    public void onDestroy() {
        mWebView.onDestroy();
        // ...
        super.onDestroy();
    }

    @Override
    public void onActivityResult(int requestCode, int resultCode, Intent intent) {
        super.onActivityResult(requestCode, resultCode, intent);
        mWebView.onActivityResult(requestCode, resultCode, intent);
        // ...
    }

    @Override
    public void onPageStarted(String url, Bitmap favicon) { }

    @Override
    public void onPageFinished(String url) { }

    @Override
    public void onPageError(int errorCode, String description, String failingUrl) { }

    @Override
    public void onDownloadRequested(String url, String userAgent, String contentDisposition, String mimetype, long contentLength) { }

    @Override
    public void onExternalPageRequest(String url) { }

}

Also here's my report2.xml... no changes from your code:
<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
    android:layout_width="match_parent" android:layout_height="match_parent"
    android:id="@+id/container">

    <im.delight.android.webview.AdvancedWebView
        android:id="@+id/webview"
        android:layout_width="match_parent"
        android:layout_height="match_parent" />

</RelativeLayout>

Thanks for your question!
The error message says:
Didn't find class "im.delight.android.webview.AdvancedWebView"
So it seems you have the AdvancedWebView class in your com.test.app package as well.
Please try either including our JAR instead of copying the source file or creating a package named im.delight.android.webview in your own source code and moving the class there.
Can you try this?

So when I include the .jar in my libs directory, and I can see the project structure already includes all *.jar's in lib as dependencies, I immediately get an error in the fragment at "implements AdvancedWebView.Listener".  It's like it doesn't know about that .jar file to see the classes/methods. Sorry, I'm a bit new to this Android thing, I don't mean to waste your time.

Disregard, it was my mistake. For anyone else new like myself, and this is already documented I just wasn't getting it....

Save the .jar file (I placed it in my projects lib directory)
Right click on your app, select New > Module
On the pop up screen, select "Import .JAR/.AAR Package". Click Next.
Select the .JAR file for AdvancedWebView. Click Finish.
Complete.

Then the native classes and methods will be recognized in Android Studio in your other java files within the project.
Thanks for the assistance!

One last question then... this solution still requires additional code added by myself in order to successfully upload files?  I see the upload prompt works, but after selecting an image nothing further occurs. Is this permissions related, or due to additional code being required on top of the base classes provided?

No problem!
Thanks for documenting this (regarding Android Studio) for everyone who may have the same problem.
Turning to the file uploads: If you followed the guide in the README, everything should work automatically. Please check if you included all method calls and callbacks.

From the permissions, you only need internet access, as documented
Check if you have the mWebView.setListener(...) line
Check if you have the onActivityResult(...) method and call, paying special attention to the issues that come with Fragment instances here (see README and other issues)
You can check if your device is supported (by Android, not by this library) via AdvancedWebView.isFileUploadAvailable()
If the upload prompt closes, what happens? What are you expecting? The file should be available to the website then, either for async upload immediately (via AJAX) or via normal upload after posting the form

Does that help?

I'll take a look and you can close this out if you'd like as it's likely my fault.  The prompt opens allowing choice of an image. After selecting one, it closes however the websites do not act as though they received a file. I cannot afterwards open the upload button again at that point, but completing the submission doesn't send an image. I've tried a few websites and had the same results. I'll look into the fragment issues.
I have the code mentioned:
mWebView.setListener(getActivity(), this);

public void onActivityResult(int requestCode, int resultCode, Intent intent) {
        super.onActivityResult(requestCode, resultCode, intent);
        mWebView.onActivityResult(requestCode, resultCode, intent);
        // ...
    }

phone is a GalaxyS6, I think it should be good.

Oh, regarding that additional code to pass data from the Fragment code, I get an error about mFragment cannot be resolved. Should this already be defined? Or can it not use it because this isn't the main and is a fragment instead?

This is really what you need to do and why it's not working for you, yet:
https://github.com/delight-im/Android-AdvancedWebView/tree/a876e0fc2aa8a6912aea68ceaa7afe962802db07#with-fragments-from-the-support-library-androidsupportv4appfragment
So you need to override onActivityResult(...) both in the Fragment and in the FragmentActivity, as shown in those docs.
Well, it seems you tried all that already. So the only thing you need to fix is that you cannot leave out the if (mFragment != null) { ... } part.
It's expected that mFragment is not defined. This is a variable (class member) that you have to define in your own code (in your FragmentActivity).
It must point to the Fragment that contains the AdvancedWebView, i.e. the instance of report2_Fragment, which is usually the current or active Fragment.
You have these instances in your Activity already. You need them because otherwise you cannot show them. So just track which is the currently shown Fragment instance (put it into mFragment) and you're done.
You can also get the current Fragment like this, perhaps:
http://developer.android.com/training/basics/fragments/communicating.html#Deliver
Otherwise, maybe you'll want to try an Activity without a Fragment or use the standard Fragment classes, not the ones from the support package. If you use the standard ones (your app will work on API levels 11+ only), you can refer to the other part of the setup guide, which is easier.


Shipyard release for 0.8.0-rc0

Signed-off-by: Mike Kolesnik<EMAIL_ADDRESS>

The E2e globalnet job is failing in Lighthouse tests -
[Fail] [discovery] Test Service Discovery Across Clusters when there are active pods for a service in only one cluster [It] should not resolve the service on the cluster without active pods
/go/src/github.com/submariner-io/submariner-operator/vendor/github.com/submariner-io/shipyard/test/e2e/framework/framework.go:416


Return values from render functions

Discussed in https://github.com/blacksmithgu/obsidian-dataview/discussions/630

Originally posted by omarkmu November  6, 2021
I was going to create a feature request for this, but I thought it would be best to ask in case I missed something. Is there already a way to get the created container element from functions like dv.table, dv.list, and dv.el?
Whenever I've wanted to manipulate the elements created by dataview, they've been simple elements created with dv.el so I've just resorted to using dv.container.createEl as a workaround. Recently, though, I've wanted to extend the lists and tables generated by dataview. As far as I know, this can't be done without copying from the source code or trying to get the created element from the container, which is less than ideal.
Is there something I've missed? If not, would it be possible to implement this feature? I don't imagine the return values of those functions can be modified without breaking changes (I believe return values are rendered in certain cases), but implementing additional helper functions which do return seems possible. Thanks in advance for any help.

This is possible via the tryQuery function (example). I hope that this was your goal - if not, sorry for closing :)

@omarkmu


Github pull request merge commits not appearing in the changelog

When I create a pull request and perform a Squash and Merge, the merge commit for the PR is not showing up in the changelog after running auto-changelog.
I've been trying to troubleshoot why that is but can't find any reason for it.  Only regular commits on master are being added to the changelog.
I assume there's something unique about the PR merge commit that means it gets ignored. GitHub appends a PR number at the end of the commit message and adds some additional 'co-author' details.
Anyone else experienced this issue and know how to fix it?

I'm not using the standard handlebars template, so I guess it's something to do with that. There is no {{#if merges}} section, I guess it could be to do with that.
# Changelog

All notable changes to this project will be documented in this file.

{{#each releases}}
{{#if href}}
  ## [{{title}}]({{href}})
  {{#if tag}}
    > {{isoDate}}
  {{/if}}
{{else}}
  ## {{title}}
  {{#if tag}}
    > {{isoDate}}
  {{/if}}
{{/if}}
{{#if summary}}
  {{summary}}
{{/if}}

{{#commit-list
  commits
  heading='### :rocket: Features'
  message='(feat)|(Feat)|(feature)|(Feature)'
  excludes='(fix)|(Fix)'}}
* {{subject}} ([`{{shorthash}}`]({{href}})) : {{author}}
{{/commit-list}}

{{#commit-list
  commits
  heading='### :wrench: Fixes'
  message='(fix)|(Fix)'
  excludes='(feat)|(Feat)|(feature)|(Feature)'}}
* {{subject}} ([`{{shorthash}}`]({{href}})) : {{author}}
{{/commit-list}}

{{#commit-list
  commits
  heading='### :bulb: Other'
  message=''
  exclude='(fix)|(Fix)|(feat)|(Feat)|(feature)|(Feature)'}}
* {{subject}} ([`{{shorthash}}`]({{href}})) : {{author}}
{{/commit-list}}

{{/each}}

OK merges are now being included in the changelog after adding the section for the keepchangelog.hbs template in this repo.
{{#if merges}}
  {{#each merges}}
    - {{#if commit.breaking}}**Breaking change:** {{/if}}{{message}} {{#if href}}[`#{{id}}`]({{href}}){{/if}}
  {{/each}}
{{/if}}

There are in a separate section and not listed along the regular 'Features' and 'Fixes' commit-list though.

OK so after running:
auto-changelog --template json --output changelog-data

It's clear that merges and commits are separate objects and the merge.commit is not part of the commits list.
This is a pain when using Trunk based development like we are as generally speaking there will be no direct commits on master and only merges.
I would like to have one list of ALL commits on Master and not distinguish between merge commits or regular commits. We are a using a Squash merge strategy for all PRs and want to categorize every commit under either Fixes or Features headings which this determines the next semver tag. It would be good if auto-changelog had an option such as --include-merge-commits to optionally add all the merge commits into the overall commits list.
In the meantime, seems I'll just have to accept having a Merges list and a separate Commits list.

I use my approach
Handlebars.registerHelper('commit-parser', function (merges, commits, options) {
        const commitsFromMerges = merges.map((merge) => merge.commit);
        const result = commits.concat(commitsFromMerges);

        return options.fn(result);
    });

# Changelog

All notable changes to this project will be documented in this file. See
[standard-version](https://github.com/conventional-changelog/standard-version) for commit guidelines.

{{#each releases}}
    {{#if tag}}
     ##{{#unless major}}#{{/unless}} [{{#replaceTitle}}{{title}}{{/replaceTitle}}]({{href}}) ({{isoDate}})
    {{/if}}

  {{#if summary}}
    {{summary}}
  {{/if}}

  {{#commit-parser merges commits}}

      {{#commit-list this heading='### ⚠ BREAKING CHANGES' message='[bB]reaking [cC]hange:|[bB]reaking:' }}
        - {{#replaceCommit}}{{subject}}{{/replaceCommit}} [({{shorthash}})]({{href}})
      {{/commit-list}}

      {{#commit-list this heading='### 🚀 Features' message='^[fF]eat:|[fF]eat\(|^[rR]efactor:|^[rR]efactor\(' exclude='[bB]reaking [cC]hange:|[bB]reaking:'}}
        - {{#replaceCommit}}{{subject}}{{/replaceCommit}} [({{shorthash}})]({{href}})
      {{/commit-list}}

      {{#commit-list this heading='### 🐞 Bug Fixes' message='^[fF]ix:|^[fF]ix\(|^[pP]erf:|^[pP]erf\(' exclude='[bB]reaking [cC]hange:|[bB]reaking:'}}
        - {{#replaceCommit}}{{subject}}{{/replaceCommit}} [({{shorthash}})]({{href}})
      {{/commit-list}}

  {{/commit-parser}}

{{/each}}

same here


Added strace prints for system calls.

Added strace prints for system calls in RETURN macro. Added checks in open and openat to check for attemtp to access /proc/self/mem.

Looks good


integration with Angular 2

Hello,
I am looking for table row editing function in Angular 2. Then I found your repository. It looks very nice! But now we all trying to use angular 2, how can I use your component in Angular 2?
Regards,
Katrina

I haven't used this with Angular 2 yet.  Maybe you can figure it out and post your results here.

hi vitaliy. any update on this port to angular2/4? thank you!
(thinking i should just use the jquery version of the plugin in my angular 4 project.)

Pull requests are welcome :)


Plans to release 1.12?

Are there any plans to publish version 1.12 on Nuget in the foreseeable future?

See #6384 for discussion of CC future. We're looking for volunteers to contribute and will hopefully be drawing up some kind of roadmap for future releases shortly.


Python:Words Score

I am willing to contribute my code for the problem mentioned . Please assign this issue under hacktoberfest 2022 tag.
problem statement- https://www.hackerrank.com/challenges/words-score

I am writing to contribute to this project.
Assign this project to me!!!

@AyushJain001 Sorry, I cannot assign multiple issues to the same contributor at a time. Once you complete that issue, you can claim this issue if it no one is assigned for this problem.

@roshan71 Sorry, this issue was raised by someone else. As per the norms, you can raise your own issue or claim an issue raised by the owner of the project. You may work on this issue if @AyushJain001 don't have any problem.

Since I have completed my first issue #58 . I request to assign this to me under hacktoberfest 2022 tag


Display values in vector/quaternion math function errors

This can help track down the source of the error more easily.

Related to https://github.com/godotengine/godot-proposals/discussions/8817.

Preview
func _ready():
	Quaternion(1, 2, 3, 4).inverse()
	Vector3.FORWARD.reflect(Vector3(1, 2, 3))

Results in:
ERROR: The quaternion (1, 2, 3, 4) must be normalized.
   at: inverse (./core/math/quaternion.cpp:91)
ERROR: The normal Vector3 (1, 2, 3) must be normalized.
   at: reflect (./core/math/vector3.h:526)

Needs to include variant.h for vformat, but otherwise to avoid adding more dependencies it could also use plain old concatenation.

Doesn't work unfortunately due to circular include, would need to just do concatenation

Also can't use String in Vector3.

Thanks!


j.data.serializers.get seems incomplete?  Serializer class is commented out?

SerializerFactory.get uses a class that has been commented out?  raising this bugreport as a placeholder ONLY for FUTURE investigation (NOT NOW).

it also looks like the initialisation of e.g. the blowfish class could never receive a key parameter as shown in the code (now removed).   it looks like this code has basically never been properly tested (and there do not appear to be any unit tests for it).


docs: 添加英文版 README.md

Motivation
(Write your motivation here.)
Have you read the Contributing Guidelines on pull requests?
(Write your answer here.)
Test Plan
(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work. Bonus points for screenshots and videos!)
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/JDFED/drip-form/tree/dev/website, and link to your PR here.)

我们需要测试下all-contributors bot在英文 readme中是否会自动修改


grpc-netty-shaded issue with initializing gcloud logging

Upgrading dependencies of our tomcat-runtime app from grpc-core-1.7 to grpc-core-1.11.0, and grpc-netty-1.7.0 to grpc-netty-shaded-1.110, we get this exception:
Caused by: java.util.ServiceConfigurationError: io.grpc.ManagedChannelProvider: Provider io.grpc.netty.NettyChannelProvider not a subtype

netty-shaded appears to have a io.grpc.netty.shaded.io.grpc.netty.NettyChannelProvider.

Created this instead.


Added Mediamodifier to Mockup tools.

Checklist

[x] I made something good today 💐.
[x ] I put links in the correct format: [Tool](link) — description.
[x ] My description starts with the lower-case letter and ends with the full stop.
[x ] I put the tool in the alphabetical order.

Optional

[ ] I added the appropriate labels: free, open-source or mac.
[x ] I put the tool only to one category.
[x] I followed Contribution Guidelines.

@jaankoppe thanks a lot!


How can I set the content-type as application/json for DELETE method in $resource

I'm trying to set the content-type as application/json for $resrouce delete action. The only reason that I need to enforce the content-type to application/json is that IE10, and IE11 detects the content-type for DELETE request as plain/text, but all other browsers and older versions of IE detects it as application/json.  I have a limitation from back end, which always check for content-type as Json.
Here is how i'm trying to enforce it:
remove: {
method: 'DELETE',
headers: { 'Content-Type': 'application/json; charset=utf-8'},
transformRequest: function(data, headerGetters) {
headerGetters()['Content-Type'] = 'application/json';
return angular.toJson(data);
},
update: {
method: 'PUT'
}
But this still doesn't work and couldn't set the content-type as json.

This is not Angular-specific, i.e. you can replicate it with "raw" XMLHttpRequest.
Closing as it is not actionable on the Angular side.

@gkalpak, I'm a little confused how this is not angular specific when this is not an issue with angular v1.3.15.
I'm in a similar situation as @mdibay1367 where I don't have control over the API I'm hitting, and it is expecting "Content-Type" : "application/json". Any other browser besides IE10/IE11 is not sending a "Content-Type" header at all, but IE10/IE11 insists on attaching a "Content-Type": "text/plain".
Again, this issue does not exist with v1.3.15... are we sure this is an issue that should be closed?

@aghreed , could you, please, provide a live demo (e.g. using CodePen, Plnkr etc) of this working on 1.3.15 ?
I'd be happy to re-open this and investigate further if it turns out to be Angular-specific.

Using Angular v1.4.3. IE11 bug. Not occurring on IE9 or any other browser. Did not check IE10.
For what it's worth, I'm having this exact same problem with $http delete. There is a known bug where certain IE versions will treat the "delete" in $http.delete as a JavaScript keyword instead of a method. The workaround is to use $http['delete'].
Both of these methods, however, have the exact same issue that mdibay described, as well as $http({method: 'DELETE', url: ...}).
I tried explicitly adding 'Content-Type': 'application/json' both on the $httpProvider default headers and in the above object-style $http({headers: {'Content-Type': 'application/json'}}). Also tried lowercased 'content-type' and all permutations of syntax, as suggested by a Stack Overflow post.

Was able to fix my related problem by setting the message body to null, which removes the Content-Type header in IE11
$http({
method: 'DELETE',
data: null,
url: ...
})

For me it was enough to prepare following request
$http.delete(url, {    data: {name: 'name to be deleted},     headers: {'Content-type': 'application/json'} })


Adding physics doesn't work

I want to add BouncingScrollPhysic() but physic is not implemented in code

I see, physics is exposed but not actually implemented, good catch.

available in 1.1.4


[FEAT] Support TLS Vault servers

Is your feature request related to a problem? Please describe.
Hi from HashiCorp!  Testing this GitHub action against a Vault server protected by SSL.  The action cannot verify the server certificate because there's no way of configure a CA path and/or tls-skip-verify.
Error from build: unable to verify the first certificate
Describe the solution you'd like
Additional parameters should be added to support TLS:

vaultCA: Path to certificate for the client to verify the server's certificate
tlsSkipVerify: Optionally disable verification of the server (not recommended but useful for test)
clientKey: Path to a client key used for mTLS
clientCert: Path to a client certificate used for mTLS

Describe alternatives you've considered
TLS really needs to be supported, so I don't think there's other alternatives.

Did you find a solution or are you planning to re-open this separately post-transition? I'm pretty sure this is indeed not supported still.

I'm adding support for it now.  I think transition will be one of the last steps so I'll likely open a PR here to add the feature.

Sounds good! Let me know if you have any questions. Only possible change I'd make is supporting base64'd version of each Path to argument so people can use certs direct from secrets rather than having to write them out first.

@RichiCoder1 We're ready to transfer the repo when you are. Let us know when you want to make it happen.


Add format.sh as pre-commit hooks

i think this is better than running format.sh

Hi @yunfeng-scale thanks for submitting the PR! Honestly I'm not familiar with pre-commit. Could you elaborate a bit more on this change?

https://pre-commit.com/ runs a set of checks every time git commit ran using git hooks. by adding pre commit hooks you can ensure formatting and lint for every commit. by default pre-commit only runs on changed files. i find it more convenient than running format.sh from time to time.
after installing pre-commit every time runs git commit looks like
(vllm) ubuntu@ip-10-202-98-80:~/vllm$ git commit -am "test"
yapf.....................................................................Passed
pylint...................................................................Failed
- hook id: pylint
- exit code: 4

************* Module vllm.model_executor.models.llama
vllm/model_executor/models/llama.py:368:16: W0612: Unused variable 'a' (unused-variable)

-----------------------------------
Your code has been rated at 9.95/10

pre-commit run can be used for manually checking. Maybe we could call pre-commit run from format.sh and make format.sh run as before.
BTW, I often forget to run format.sh before submitting, so I have to commit with --amend once again. Hooks is better than running format.sh manually before committing.

You can set pre-commit to use local script. Since vLLM already have format.sh and lint.sh, pre-commit can just run a system hook, like this
repos:
  - repo: local
    hooks:
      - id: format_lint
        name: vLLM format and lint script
        language: system
        always_run: true
        pass_filenames: false
        entry: ./format.sh

You can set pre-commit to use local script. Since vLLM already have format.sh and lint.sh, pre-commit can just run a system hook, like this
repos:
  - repo: local
    hooks:
      - id: format_lint
        name: vLLM format and lint script
        language: system
        always_run: true
        pass_filenames: false
        entry: ./format.sh

The workflow for dev would be
pip install -e . -r requirements.txt -r requirements-dev.txt

pre-commit install

<edit>

git add . && git commit # pre-commit will run the pre-commit hook this step

thanks, i'm taking your suggestion.

@zhuohan123 @WoosukKwon i think people do prefer precommits rather than running format.sh manually. let me know how you want to proceed, either just running format.sh in precommit, or solely depend on comprehensive precommit hooks

@yunfeng-scale , @zhuohan123 , @WoosukKwon - My cent for your reference. I pursued pre-commit hooks because I noticed that format.sh does not cover C++ code; however, vLLM contains C++ code, right?  If we simply let .pre-commit-config.yaml invoke format.sh, we will need to add C++-stuff in format.sh, together with git diff-related code, which seems pretty cumbersome.

Tbh a format.sh would suffice. I think setting up pre-commit.ci would be nicer, such that it will run the format script on CI. So it is not intrusive, users don't have to install pre-commit locally.

Do we think that this will ever be merged or shall it be closed as stale?

Close as stale for now.


[PR1] DEV-30 MVP-2 Backend

This PR creates the main file that orchestrates the backend for the MVP-2 and fixes some conflicts between modules

this PR is a certified programmer moment :)

Will separate the front from the back in future commits, as of now the working frontend will stay in cmd/MVP-2/static so that it's easier to distribute


support integer enum

Currently,  Enum type is created as string enum regardless of enum type.
This PR add supporting integer type.

Thank you for the PR, please check the failing test. Thanks.

@fazpu this fail depends on #68 . It will get success when #68 will merge.

Thanks. Please, run npm run lint to check problems.

@jnwltr sorry my bad code. I've fixed my code, please re-review it :)

The build is failing...

this build failing is happened by this issue.

@soundTricker, please, rebase and we can merge. Thx

@jnwltr Rebase & Pushed :)


fix breaking api change

Summary
Removes the breaking api change for the native redirect flow for v4.4.2. Not planning on merging this PR but we will use this branch. This version contains the fix for https://nvd.nist.gov/vuln/detail/CVE-2018-1000211 . It was easier to fix the api change and update the gem then to implement the fix ourselves.

close. Made a PR to Doorkeeper https://github.com/doorkeeper-gem/doorkeeper/issues/1143.


Running on Mono : System.MissingMethodException: Method 'Array.Empty' not found.

Hi,
I have a script that I want to run under Mono, but I have a different behavior between Windows and Mono.
Javascript is equivalent to navigators' :
function draw()
{
}

window.requestAnimationFrame(draw);

On Microsoft Framework on Windows, works fine.
On Mono :
System.MissingMethodException: Method 'Array.Empty' not found.

It happens in Binder.cs, in method public object Call(ScriptEngine engine, object thisObject, params object[] arguments) when trying to execute return binderDelegate(engine, thisObject, arguments);
Running on Mono : 4.2.3 (Stable <IP_ADDRESS>/832de4b)
Environment.Version : 4.0.30319.17020

Partial stacktrace :
System.MissingMethodException: Method 'Array.Empty' not found.
  at (wrapper dynamic-method) Jurassic.Compiler.JSBinder:binder_for_RpiMono.JsObjects.WindowInstance.RequestAnimationFrame (Jurassic.ScriptEngine,object,object[])
  at Jurassic.Compiler.Binder.Call (Jurassic.ScriptEngine engine, System.Object thisObject, System.Object[] arguments) <0x6d65ec50 + 0x00043> in <filename unknown>:0
  at Jurassic.Library.ClrFunction.CallLateBound (System.Object thisObject, System.Object[] arguments) <0x6d65e8c0 + 0x00133> in <filename unknown>:0
  at Jurassic.Library.FunctionInstance.CallWithStackTrace (System.String path, System.String function, Int32 line, System.Object thisObject, System.Object[] argumentValues) <0x6d657a60 + 0x00067> in <filename unknown>:0
  at JavaScriptClass375.anonymous (Jurassic.ScriptEngine scriptEngine, Jurassic.Compiler.Scope scope, System.Object thisValue, Jurassic.Library.FunctionInstance , System.Object[] ) <0x6cbcda60 + 0x03b1b> in <filename unknown>:0
  at (wrapper delegate-invoke) <Module>:invoke_object_ScriptEngine_Scope_object_FunctionInstance_object[] (Jurassic.ScriptEngine,Jurassic.Compiler.Scope,object,Jurassic.Library.FunctionInstance,object[])
  at Jurassic.Library.UserDefinedFunction.CallLateBound (System.Object thisObject, System.Object[] argumentValues) <0x6d657c10 + 0x00057> in <filename unknown>:0
  at Jurassic.Library.FunctionInstance.CallWithStackTrace (System.String path, System.String function, Int32 line, System.Object thisObject, System.Object[] argumentValues) <0x6d657a60 + 0x00067> in <filename unknown>:0
  at JavaScriptClass370.anonymous (Jurassic.ScriptEngine scriptEngine, Jurassic.Compiler.Scope scope, System.Object thisValue, Jurassic.Library.FunctionInstance , System.Object[] ) <0x6cb06000 + 0x1c72f> in <filename unknown>:0
  at (wrapper delegate-invoke) <Module>:invoke_object_ScriptEngine_Scope_object_FunctionInstance_object[] (Jurassic.ScriptEngine,Jurassic.Compiler.Scope,object,Jurassic.Library.FunctionInstance,object[])
  at Jurassic.Library.UserDefinedFunction.CallLateBound (System.Object thisObject, System.Object[] argumentValues) <0x6d657c10 + 0x00057> in <filename unknown>:0
  at Jurassic.Library.FunctionInstance.CallWithStackTrace (System.String path, System.String function, Int32 line, System.Object thisObject, System.Object[] argumentValues) <0x6d657a60 + 0x00067> in <filename unknown>:0
  at JavaScriptClass369.global_sampleCube_js (Jurassic.ScriptEngine scriptEngine, Jurassic.Compiler.Scope scope, System.Object thisValue) <0x6cbca670 + 0x0015b> in <filename unknown>:0
  at Jurassic.Compiler.GlobalMethodGenerator.Execute () <0x6d656848 + 0x00083> in <filename unknown>:0
  at Jurassic.CompiledScript.Execute () <0x6d6567f8 + 0x0001f> in <filename unknown>:0
  at Jurassic.ScriptEngine.Execute (Jurassic.ScriptSource source) <0x6e431228 + 0x0006f> in <filename unknown>:0

Reproduced with Mono for Windows (4.2.3).

Found a potential cause : may be related to dotnet 4.6. Seen this issue on another site.
My assemblies were compiled with VS2015 for version 4..6.1. I will try to downgrade.

Downgraded my assemblies to framework 4.5.2 : works fine on Mono for Windows.

Thanks, that may help others who are trying to get Jurassic to work on Mono.


[ContainerRegistry] support passing service version via client options

This PR adds an optional serviceVersion proeprty to ContainerRegistryClientOptions. It allows overriding the default service api version of "2021-07-01"

The change itself seems fine, but does this client actually support multiple service versions? It feels odd to add this capability without a second version to test against,

Good point! This is the first version the service supports.  I will wait for the next version then add the support.

The KV approach is fine too because the service will report an error for unsupported versions.

I would avoid throwing for invalid values, if the service can throw meaningful errors instead.

I thought that could save the customer a network request. But yes in general we shouldn't validate service parameters.


pfTableView: error when items array is empty

Current behavior
When the itemsAvailable configuration option is set to true and the table is populated with an empty array, an error happens in the Javascript code.
Expected behavior
An empty array is a valid source of data for the Table View and no error should happen in that case.
Steps to reproduce

Open the table/tableview/examples/table-view-basic.js file and change the itemService.getItems() function to return an empty array
Open the pfTableView - Basic page
Open the browser DevTools console
Click on the Items Available checkbox

Logs
angular.js:14199 TypeError: Cannot read property 'innerText' of undefined
    at getVisibleRows (angular-patternfly.js:13679)
    at setSelectAllCheckbox (angular-patternfly.js:13660)
    at angular-patternfly.js:13648
    at angular.js:19942
    at completeOutstandingRequest (angular.js:6045)
    at angular.js:6324

    function getVisibleRows () {
      ...

      for (i = 0; i < anNodes.length; ++i) {
        rowData = anNodes[i].cells;
        if (rowData !== null) {
          visibleRows.push(_.trim(rowData[ctrl.selectionMatchPropColNum].innerText)); // ERROR HERE
        }
      }

      ...
    }

@akieling, I followed these steps but I do not get an error message in the console, and the table view renders with a nice empty message:

The latest code from master.
On Tue, Oct 31, 2017 at 3:01 PM, David Taylor<EMAIL_ADDRESS>wrote:

What angular-patternfly version are you using?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/patternfly/angular-patternfly/issues/667#issuecomment-340830921,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB0lT9matBRWOznc7p5KsFKTDP9ICvJ3ks5sx1J-gaJpZM4QJvoY
.

--
Alexandre KielingSr. Software Engineer, JBoss FuseRed Hat

@akieling Are you still seeing this error?

@akieling, is this still an issue for you?  going once, going twice...:-)

It doesn't break the UI but I can reproduce the error in both Chrome and Firefox.

@akieling, so you don't see the "No data available in table" message?

@dtaylor113 I can see the "No data available in table" message. The UI is fine. The only issue is the error logged to the browser console.


[BUG] Wazuh server crashes after creating a per document monitor

I tried to follow the following documentation: Detecting unauthorized access to sensitive servers using Per document monitor.
Every time, after a minute more or less, I get the error in the screenshot and my Wazuh server crashes

To Reproduce
Steps to reproduce the behavior:

Go to Alerting -> Monitoring
Click on new Monitor
Follow the documentation
See error

Expected behavior
Everything but a server crash
Host/Environment (please complete the following information):

OS: Ubuntu
Wazuh version: 4.9.2

I was unable to replicate the issue. I try to replicate the same steps but I work everything to me.

That's unfortunate. However, I have been stuck with this bug for weeks. Is there any way to resolve this?
Thanks for your help

Please provide any available logs, including those from wazuh-indexer, wazuh-manager, wazuh-dashboard, and wazuh-agent.

Hello, I've tried to change the OS and install Wazuh again on a new VM. Now I am able to create the per document monitor. However, when I try to send the test message or to see the trigger preview I get this error:
Failed to run the trigger
[alerting_exception] routing is required for [.opendistro-alerting-alerts]/[error-alert--592bf3b6-4144-4cba-80bd-e8f6a85a57e2]

If a routing error is displayed, it's possible that the channel is not configured correctly. Could you request the applied configuration, ensuring that API keys and other sensitive information are obfuscated?

Hello Guido, thanks for your reply.
It's not totally clear to me what you mean for configuration. Are you referring to the ossec.config file? Could you please be more precise?
Sorry for my confusion

Please provide the channel configuration and any other settings as depicted in the issue description's image.

I use Microsoft Teams. I generated a hook url with IncomingWebHook.
I also noticed this warning in the monitor. Is it possible to edit those setting in the visual editor or I need to switch to the other and what are the best settings to avoid query size problems?

To eliminate the possibility of other issues, you could replicate this error using a different channel, such as Discord or Slack.

I used Slack and it worked fine, thanks. Focusing on the main problem, I was monitoring the /var/log/wazuh-indexer/wazuh-indexer-cluster.log while I started the per document monitor and I found this
"cancelling task [2196] due to high resource consumption [heap usage exceeded [3.4gb >= 47.1mb]".
However I put 8G for the heap usage, is there a way to solve this?

I’m also encountering the same issue. I noticed that you can use a date math index resolution in your index pattern. I want Wazuh to work only with the ‘Today’ alerts index, but it is not accepting the default date math pattern: wazuh-alerts-*-{now/d}

You can apply a custom timestamp filter to filter alerts by time, displaying only those that occurred today.

I used Slack and the routing problem was resolved but not the monitor performance problems.
Focusing on the main problem, I was monitoring the /var/log/wazuh-indexer/wazuh-indexer-cluster.log while I started the per document monitor and I found this
"cancelling task [2196] due to high resource consumption [heap usage exceeded [3.4gb >= 47.1mb]". However I put 8G for the heap usage.
"[circuit_breaking_exception] [parent] Data too large, data for [<http_request>] would be [16122992016/15gb], which is larger than the limit of [15986170265/14.8gb], real usage: [16122989152/15gb], new bytes reserved: [2864/2.7kb], usages [request=0/0b, fielddata=2014/1.9kb, in_flight_requests=28688/28kb], with { bytes_wanted=16122992016 & bytes_limit=15986170265 & durability="TRANSIENT" }"

Is your wazuh-indexer configured in this manner?
-Xms4g
-Xmx4g

Where the total heap space:
-Xms4g - initial size is set to 4Gb of RAM.
-Xmx4g - maximum size is to 4Gb of RAM.

What I meant was that if I create a per-document monitor and use “wazuh-alerts-*” as the data source, the server crashes after a minute. However, if I use a single index, such as “wazuh-alerts-4.x-2024.11.19,” everything works fine. I assume the issue lies in the amount of data Wazuh needs to process. For example, if I want to monitor unsuccessful logins and create a per-document monitor for that, I could use only the latest alert index, as it should theoretically be sufficient for such a task. But how can I configure it to use only “today’s” alert index?

You could use an index alias that allows you to set up the monitor pointing to the alias, and then some cronjob that changes the index the alias points to each day. You can try this.

We are closing due to inactivity.


Move Swift Algorithm Club to Data Structures / Algorithms category

Move Swift Algorithm Club to Data Structures / Algorithms category
Data Structures / Algorithms is the best fit category for this Swift Algorithm Club.

          1 Warning

:warning:
Found 3 link issues, a project collaborator will take care of these, thanks :)

Link issues by awesome_bot

Line
Status
Link

8
522
https://img.shields.io/badge/PRs-welcome-brightgreen.svg

521
301
https://github.com/bwide/BWCollectionView redirects tohttps://github.com/bwide/CollectionNode

2764
301
http://www.coreint.org/ redirects tohttp://coreint.org/

  Generated by :no_entry_sign: Danger

Thank you!


ci(changelog): generate changelog on releases

refs #889

I did
git checkout v0.9.0-rc13
make --eval=GORELEASER_TAG=v0.9.0-rc13 release-dry-run

Afterwards the file .cache/changelog is generated
v0.9.0-rc12

## [Unreleased]

### Ci
- **changelog:** generate changelog on releases
- **codeowners:** remove Ropes add hydrogen18 ([#987](https://github.com/ovrclk/akash/issues/987))

## [v0.9.0-rc13] - 2020-12-04
### Feat
- **provider-gateway:** hostname for exposed ports

### Fix
- **audit:** validate genesis for audited attr ([#977](https://github.com/ovrclk/akash/issues/977))
- **ingress hosts:** stable hostnames
- **provider:** do not return error if ErrServerClosed
- **provider:** Make the provider error codes a bit more helpful. Show long form errors if they come back from the server

### Tests
- **integration:** fix test

## [v0.9.0-rc12] - 2020-12-01
### Chore
- comment unused code to stop linter complains ([#958](https://github.com/ovrclk/akash/issues/958))

### Cleanup
- **docs:** move docs to ovrclk/docs repo

### Fix
- **cmd:** hack env commands to be available to client ([#957](https://github.com/ovrclk/akash/issues/957))
- **network-policies:** configurable, default off

[Unreleased]: https://github.com/ovrclk/akash/compare/v0.9.0-rc13...HEAD
[v0.9.0-rc13]: https://github.com/ovrclk/akash/compare/v0.9.0-rc12...v0.9.0-rc13
[v0.9.0-rc12]: https://github.com/ovrclk/akash/compare/v0.9.0-rc11...v0.9.0-rc12

It's showing v0.9.0-rc12 on the first line of the file, I don't know why.
My only other comment was it would be nice if the line ### Feat could say ### Features and ### Fix could say ### Fixes

Yeah, v0.9.0-rc12 is nasty, I'll check
Fix -> Fixes and Feat -> Features - good point


MPI_Session is an MPI 4.0 handle

It appears that MPI_Session is used in general/communication.hpp as a class name. MPI 4.0 added MPI_Session as a standard MPI handle type. Spack E4S builds with MPICH versions >= 4.0 fail because of the ambiguity.
https://github.com/mfem/mfem/blob/168b19fc4904dc90e217bef9b5c3f595c4865da2/general/communication.hpp#L90-L104
ex1p.cpp(66): error: "MPI_Session" is ambiguous
1 error detected in the compilation of "ex1p.cpp".
makefile:78: recipe for target 'ex1p' failed
make: *** [ex1p] Error 1
make: Leaving directory '/tmp/root/spack-stage/spack-stage-mfem-4.3.0-eh3uxmai2x52ienouug4aiftaydipfd6/spack-src/examples'

Hi @raffenet,
Do you know if this failure is specific to new mfem-4.4?
I am asking because MPI_Session has been in MFEM for quite a few years and while we did some refactoring in mfem-4.4, we maintained it for backward compatability.
Thanks,
Tzanio

This error is probably from MFEM version < 4.40. In 4.4.0, MPI_Session was replaced by Mpi::Init() in most examples and miniapps.
Also, one can probably just continue to use MPI_Session by adding the namespace qualifier,mfem::MPI_Session, unless, MPI_Session can be a macro in MPI v4?

By the way, @raffenet, I'm close to completing the addition of MFEM v4.4.0 to Spack: https://github.com/spack/spack/pull/29658 which may help resolve the issue if E4S uses the default MFEM version from Spack.

This error is probably from MFEM version < 4.4.0. In 4.4.0, MPI_Session was replaced by Mpi::Init() in most examples and miniapps.
Also, one can probably just continue to use MPI_Session by adding the namespace qualifier,mfem::MPI_Session, unless, MPI_Session can be a macro in MPI v4?

MPI_Session is just an opaque type. In MPICH it is typedef'd to int in mpi.h.

Addressed in #3040.


Switch to pytest and add optional Python 3.8 test

This PR switches the unit testing to use pytest for Travis CI instead of the deprecated setup.py test.  It also adds an optional test for Python 3.8.

Coverage remained the same at 84.599% when pulling 74e15b673b07c9e0ff9c9fe5eddd042506b867ba on murrayrm:setup-test-deprecated into 874d52eca13f1644eddad52be3e74a5c79883cfd on python-control:master.

Coverage remained the same at 84.599% when pulling 74e15b673b07c9e0ff9c9fe5eddd042506b867ba on murrayrm:setup-test-deprecated into 874d52eca13f1644eddad52be3e74a5c79883cfd on python-control:master.

Coverage remained the same at 84.599% when pulling 74e15b673b07c9e0ff9c9fe5eddd042506b867ba on murrayrm:setup-test-deprecated into 874d52eca13f1644eddad52be3e74a5c79883cfd on python-control:master.

For some reason the allow_failures tests are not being run.  This was working when they were originally generated (several months back).  Did something change in Travis CI?
Without the allow_failures functionality, the Python 3.8 test does not get run at all.

Rebased changes on top of master to get some of the fixes from #366 the eased precision on some of the tests.


Fix older VS compatibility issues and PDB files generation issue.

This fixes several windows builds related issue:

the issue #1411 (with visual studio 2013 and earlier);
another compilation issue with VS2012 (strtoll not defined)
the PDB files related warnings when building ninja/ninja_test by providing a unique name per library/binary for the PDB file.

Looks like you merge the changes? Can you do
git rebase origin/master
git push ikifof issue1411_MSVC12_broken --force

?
(I'm assuming that origin is https://github.com/ninja-build/ninja.git and ikifof is your fork)

Did I do something wrong? I hope not... Git has some cool features... but way too uselessly complex... :(
I just ran the 2 commands you gave. I think I answered all your review comments but somehow the CR summary still shows an open request... Let me know if I need to do something.
Note that the MinGW build has some bad warnings and fails the disk_interface_test that was disabled for msvc only. It probably should be disabled on windows.

What's your output of git remote -v?

the disk_interface_test that was disabled for msvc only. It probably should be disabled on windows.

Will do that tomorrow.

origin  https://github.com/ikifof/ninja.git (fetch)
origin  https://github.com/ikifof/ninja.git (push)
upstream        https://github.com/ninja-build/ninja.git (fetch)
upstream        https://github.com/ninja-build/ninja.git (push)

Ah! Run these then please:
git rebase upstream/master
git push origin issue1411_MSVC12_broken --force

Let me know if I need to do anything else. I can create a new clean pull request if I made things complicated by messing up this one.

No it's okay, I squashed the commits using GitHub. Thank you!

Thanks!


Copy edit notebooks, round 1

These notebooks are ready for copy editing:

Notebook whose name starts "00"
All notebooks whose name starts with "01"

Matt, could you please add @lglattly to this repo and assign her to this issue. Thanks!

@kelle @lglattly -- assigned!


SonarQube issue : Complete the task associated to this TODO comment (Group 5)

Title: Complete the task associated to this TODO comment
Description:
Found an incomplete TODO comment in the codebase during SonarQube review. The absence of a completed task associated with the comment suggests unfinished work or missed functionality, impacting code quality and maintainability.

Consequences:

Compromised Code Quality
Maintenance Challenges
Documentation Deficiency
Accumulation of Technical Debt

Solution:

Identify the task linked to the TODO comment.
Complete the task as required.
Update the comment with task status and details.
Submit changes for review and integration.

Positive Impact upon Resolution:

Enhanced Code Quality
Reduced Technical Debt

During code review, identified an incomplete TODO comment that lacked associated tasks. Since there were no pending actions or tasks related to the comment, it was removed from the codebase to maintain clarity and avoid confusion.


Implement step solver featores to the main Solver component

This implements both, the optional Shannon entropy switch and step solver feature.
The Step solver is embedded into the original Solver component. If Observation count limit is specified, the Solver turns into a step solver.
~TODO: Currently the Solver returns non-deterministic results as if they were deterministic.~

Thanks for tagging me here. I'll review this thoroughly after we are done with https://github.com/subdgtl/WFC/pull/17


Make github pages version nicer?

It would be nice if the github pages version of the site had a sidebar and such niceties. Maybe a nicer theme.
EDIT: See https://github.com/mmistakes/minimal-mistakes/issues/476

Keep in mind that we've already imported the default.html in _layouts for the slate theme, so switching themes must be done cautiously.

Stupid question, what's the URL for the github pages version?

https://bluddy.github.io/ocamlverse or just https://git.io/overse

We might want to take some inspiration from https://reasonml.github.io/. We don't need anything near as fancy, but taking elements of it would be nice.

This Jekyll theme seems to have sidebar support, plus it looks pretty great: https://github.com/mmistakes/minimal-mistakes/issues/476

Linking this page so we have it for reference: https://help.github.com/articles/adding-a-jekyll-theme-to-your-github-pages-site/

I looked into other themes last night. I looked at minimal-mistakes, lanyon, and documentation-theme. Of these, lanyon is my favorite so far, but I can easily be convinced to choose documentation-theme if we want to modify it a little. Minimal-mistakes so far seems heavily biased towards blog posts. It has a ton of features, but I couldn't get it to do what I wanted. Had I been able to tame it, it may have become my favorite. So no conclusions so far.
I'm reading up on the jekyll docs. I'm currently missing some pieces about how this stuff works.
Also, I'll be offline until Monday night, so if anyone else wants to dig in, feel free to do so.

@bluddy I'll start by saying thanks for starting this initiative!
Sidebars and search are some features that would indeed be very nice to have. I like the layout of the reasonml/bucklescript websites. They seem to use Docusaurus for the website. Could that work for the type of content in ocamlverse?

Github pages uses jekyll as its default generation engine, and it seems like docusaurus is now a new alternative from facebook using node.js. It takes a little more work to integrate it with github pages, so for now I think it makes sense to try stick with jekyll (though I could be convinced otherwise). For Reason folks, node.js is a natural fit. Also, I think docusaurus makes more assumptions about site layout and features that don't necessarily correspond to ocamlverse's design.
Both these platforms are doing really simple things, so I think we can ultimately manage with either one, and the closer integration with github pages makes this a little easier for us. At least that's my point of view right now.

Sounds fair. I agree that the ease of integration with github pages does make Jekyll a more attractive option to start with

After investigating some themes and learning about jekyll, I actually think it'll be easier to just edit our own css to make it nicer. @pmetzger let me know if you're interested.
I recommend setting up jekyll locally following these instructions so you can try stuff out.
The easiest way to modify css is to follow the instructions here. I'm personally working on getting a sidebar going.

Jekyll's primary goal seems to be to support creation of blogs. I'm not sure how much effort /maintenance it will take to mold it to support a proper documentation/wiki style layout while keeping the structure easy to understand. Is native support by github the only reason to stick with it? There might be other tools that better support the style of layout that suits ocamlverse better.
Personally I like the style provided by Sphinx when it comes to reading technical content, or a documentation/wiki style heavy content.

Sphinx uses restructuredText, which while not bad, is going to be a new syntax for contributors to learn. On the other hand, everyone knows markdown.
All of these tools ultimately do very simple things, and I'm not sure we need to switch from Jekyll right now. At the end of the day, it's just generating static pages. The convenience of having it built into github pages is quite significant.

Sphinx now supports markdown
I agree that there is nothing that complicated in all of these tools, but It might just be more difficult to switch layouts once we have a substantially more amount of content.
That being I was merely stating my preference for a documentation layout, which is highly subjective 😄 At the end of the day the important part if working towards a good resource for people interested in OCaml and i'll be happy to support any effort towards that.

Sphinx now supports markdown
Oh, interesting.

Well, listen, I'm open to anything that keeps our light process while improving the results. I really like that I can navigate OCamlverse equally well from the github view, starting with the readme. I really like that we can be concerned with content first. If you think you can come up with a process that's light, that automatically updates the site, and that looks better and has a sidebar + search or something like that, you can fork and create that in a branch and I'll check it out. I can't promise we'll switch to it, but if it has these properties, I may be tempted to do so.
Of course you're right about the subjective aspects -- I can't guarantee the other admins won't critique the new look, but presumably that can be modified on any platform.

I think I'm pretty happy with the look of the site right now (aside from access from mobile, which has its own issue). We can improve on it if need be, but for I'm good with where it's at. Closing.

Any progress with sidebar support in GitHub pages?

(Side note, I recently did a scan of site generators and see some potential in (vercel | netlify) + (hugo | 11ty), if at some point a jekyll alternative is being considered. I agree with trying to use github pages by default)


Add support for S3 StorageClass (Glacier, Reduced Redundancy, Infrequent Access)

This PR adds support for S3's storage class...

Could you open this against elasticsearch repo? It won't be merged in this repo.
Also, if you want to udpate AWS SDK, please do it in another PR (against elasticsearch repo again).
Thanks!

@dadoonet erm, sure... Reason I didn't do it from the start is cos the main es repo requires Java 1.8 to build... Is there a way around that? Otherwise, I'll get that set up in a few days

Well. You can build it against 2.x branch.

Fixed with https://github.com/elastic/elasticsearch/pull/13656


Question: When is this plugin considered "ready"?

The CoreDNS /ready endpoint will return HTTP 200, when all plugins are loaded See this.
Can this endpoint used to check if all ingress hostnames are loaded into the index?
I want to use the /ready endpoint in a readinessProbe to add this CoreDNS instance to the service endpoints only when the entries are ready. Startup time could be a few seconds and I want to avoid any downtime of the CoreDNS.

Yep, this should be fairly easy to add. The ready plugin should already be baked into the binary and all that we're missing is a file similar to this one https://github.com/coredns/coredns/blob/5b9b079dabc7f71463cea3f0c6a92f338935039d/plugin/kubernetes/ready.go
Do you want to do a PR @mstoetzer ?


ConfigurationChangeEvent not fired with resource scoped properties on multiroot workspace

You can use https://github.com/jlelong/config-event-vscode to reproduce the issue. This is a very basic extension build on https://github.com/microsoft/vscode-extension-samples/tree/main/configuration-sample. It only prints the value of the property dummySetting, which has a resource scope, whenever an ConfigurationChangeEvent is fired.
Consider the multi-root workspace.
event.code-workspace

A/
  .vscode/settings.json
  A.txt

B/
  B.txt

with A/.vscode/setting.json containing
{
    "conf.dummySetting": false
}

Note that the default value is true. Run the extension and toggle between A.txt and B.txt and note that nothing is printed in the console whereas the value of conf.dummySetting changes.
Version: 1.64.2
Commit: f80445acd5a3dadef24aa209168452a3d97cc326
Date: 2022-02-09T22:00:58.347Z
Electron: 13.5.2
Chromium: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Darwin x64 21.3.0

I think it is not a bug. When toggling between A.txt and B.txt, the value of conf.dummySetting for each workspace does not change. Only the active editor does change.

Yes you are right. I somehow misinterpreted how WorkspaceConfiguration.get was obtained. It is clear that the WorkspaceConfiguration's of workspaces  A and B do not change.
I am closing the issue


Direction keys are not marked 'enhanced' in VSCode terminal

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.62.2
OS Version: Windows 10 21H1

I can use the code snippet from https://github.com/microsoft/vscode/issues/73480 to illustrate the issue:
//
// build me in a Visual Studio Command Prompot with: cl vskeybug.cpp
//
#include <windows.h>
#include <stdio.h>

#define INPUT_BUFFER_LENGTH 128
INPUT_RECORD input_buffer[INPUT_BUFFER_LENGTH];

int main(int argc, char* argv[])
{

    HANDLE stdin_handle = GetStdHandle(STD_INPUT_HANDLE);
    if(stdin_handle == INVALID_HANDLE_VALUE)
    {
        printf("Error: GetStdHandle failed, (error=%d)\n", GetLastError());
        return 1;
    }

    // Save old input mode
    DWORD old_mode;
    if(!GetConsoleMode(stdin_handle, &old_mode))
    {
        printf("Error: GetConsoleMode failed (error=%d)\n", GetLastError());
        return 1;
    }
    DWORD new_mode = old_mode;

    new_mode &= ~(
          ENABLE_ECHO_INPUT      // disable echo
        | ENABLE_LINE_INPUT      // disable line input, we want characters immediately
    );
    printf("Current console mode 0x%x, setting to 0x%x\n", old_mode, new_mode);

    if(!SetConsoleMode(stdin_handle, new_mode))
    {
        printf("Error: SetConsoleMode failed (error=%d)\n", GetLastError());
        return 1;
    }

    printf("Type on the keyboard (ESC to exit)...\n");
    fflush(stdout);

    BOOL continue_loop = true;
    while(continue_loop)
    {
        DWORD input_count;
        if(!ReadConsoleInput(stdin_handle, input_buffer, INPUT_BUFFER_LENGTH, &input_count))
        {
            printf("Error: ReadConsoleInput failed (error=%d)\n", GetLastError());
            break;
        }

        printf("Got %d input events...\n", input_count);
        for(DWORD i = 0; i < input_count; i++)
        {
            switch(input_buffer[i].EventType)
            {
                case KEY_EVENT:
                    {
                        WORD code = input_buffer[i].Event.KeyEvent.wVirtualKeyCode;
                        printf("  key%s %d (state=0x%x)\n",
                            input_buffer[i].Event.KeyEvent.bKeyDown ? "down" : "up  ",
                            code,
                            input_buffer[i].Event.KeyEvent.dwControlKeyState);

                        if(code == VK_ESCAPE)
                            continue_loop = false;
                    }
                    break;
                default:
                    printf("  OtherEvent(%d)\n", input_buffer[i].EventType);
                    break;
            }
        }
    }

    return 0;
}

Here the state from input_buffer[i].Event.KeyEvent.dwControlKeyState indicates the extension mode. According to https://docs.microsoft.com/en-us/windows/console/key-event-record-str#remarks , the direction keys should be marked 'enhanced'. However, this is not true in the Visual Studio Code terminal. Let's see the output from a VSCode terminal and other terminals:
VS Code

Windows Terminal

pwsh.exe (ConHost)

Notice the difference of the state=... output. Why is VS Code so different from the other terminals?
Steps to Reproduce:

Compile the given code snippet
Run the code from a VS Code terminal and ConHost/whatever, see the difference

I used the library https://github.com/wmcbrine/PDCurses and found it not able to distinguish the direction keys in the VS Code terminal due to this different behavior. Is that a bug or a feature by design?

OK, I reproduced this in https://github.com/sedwards2009/extraterm which uses ConPTY as well. It seems to be a issue with ConPTY and not VS Code itself.


Zmai-90 with CBU module (the earlier version has WB3S)

Flash dump from a fellow user.

 Zmai-90 with CBU module.zip

This post has been tagged as new-dump-needed and has had not response.  This issue is being closed as stale.  To re-open this issue, please provide a new dump following the guildines in the device dumps section of the Readme.

i am trying to extract dump now but i get only this:
root@adrian-huawei-d16:/home/adrian# bk7231tools read_flash -d /dev/ttyUSB0 -s 0 -l 0x200000 dump_zmai90.bin
Traceback (most recent call last):
File "/usr/local/lib/python3.10/dist-packages/bk7231tools/main.py", line 468, in cli
with closing(connect_device(args.device, args.baudrate, args.timeout, args.debug)) as device:
File "/usr/local/lib/python3.10/dist-packages/bk7231tools/main.py", line 270, in connect_device
s.connect()
File "/usr/local/lib/python3.10/dist-packages/bk7231tools/serial/init.py", line 39, in connect
raise TimeoutError("Timed out attempting to link with chip")
TimeoutError: Timed out attempting to link with chip

try adding --timeout 9999, executing the command, then briefly shorting the CEN pin to reboot the device.  You may need to use an external power source (not mains) if your ttl is being reset by that process.

Same problem connected to 220v i dont know if is ok pin 15 and 16 for tx1 and rx1, on datasheet this write and CEN is pin 18

You should absolutely not be attempting to read flash when powered by mains.  If you did, you may have killed the module completely.

is working :D but wtill can't read, i will give up now maybe someone will succesfully extract from CBU ver 2.3.3


Got not "enough values to unpack" error when pass additional headers

Trying to create webhook with additional headers, but got the error below.
Traceback (most recent call last):
  File "/opt/netbox/venv/lib/python3.9/site-packages/rq/worker.py", line 1061, in perform_job
    rv = job.perform()
  File "/opt/netbox/venv/lib/python3.9/site-packages/rq/job.py", line 821, in perform
    self._result = self._execute()
  File "/opt/netbox/venv/lib/python3.9/site-packages/rq/job.py", line 844, in _execute
    result = self.func(*self.args, **self.kwargs)
  File "/opt/netbox/netbox/extras/webhooks_worker.py", line 58, in process_webhook
    raise e
  File "/opt/netbox/netbox/extras/webhooks_worker.py", line 55, in process_webhook
    headers.update(webhook.render_headers(context))
  File "/opt/netbox/netbox/extras/models/models.py", line 166, in render_headers
    header, value = line.split(':', 1)
ValueError: not enough values to unpack (expected 2, got 1)

The header I added was simple, just an authorization header
Authorization:
eyJ0eXAiOiJKV1QiLCJhbGciOiJSUzI1NiJ9.eyJleHAiOjE3NDcwMDgwMDAsInN1YiI6Im5ldGJveC13ZWJob29rIiwibmJmIjoxNjQwODU2NTg2LCJtesttest_keysYsImF1ZCI6WyJrYWZrYS1yZXN0LXByb3h5Il0sImlzcyI6InNlY3JldC9kYXRhLXN0b3JhZ2Uva2Fma2EtcmVzdC1wcm94eS9kZXYvc2VydmljZS1hdXRoIn0.GAEcbzR5Hk7SKg2ZGcE37Ti85VzxxTJkqBzHv7abNjyWYZ2UfI9MZLG--GE2AYSP6ZxOGSylU69gfXiH-jlzrlXylgHt9ZfQzQPqTGXzyy3orinr3tpepKvkKTA_uPRK1vAoyux3Uit0Z617_QCatnvpSDcm5EKCbIa8StvxcLRu6D_zK8uzTBHyIDnnwVwvBDGc6Cd40Cfg7JGTest_keys_testEDG5YTR5xBknSiEXnmXEKWtGnLmyFOoKjifmy1ui78pc4testkeystestV9psBdKGLKEkzmmDsTL6SxMBgNY_nKCAWbZl4SQ

Not sure why the error happend.
The netbox version: v3.1.5

How exactly is this related to pynetbox package?

It doesn't appear to be. Also, your NetBox installation is a bit dated; please try upgrading to the current stable release and you may find that it resolves your issue.

Closing this issue as it appears to be a configuration error.


tidy #1 of maven build instructions

however will likely move soon to default being to skip deb, rpm, and go-client, unless you opt in

LGTM


Support for multiple Embeds

https://discord.com/developers/docs/resources/channel#create-message-jsonform-params
In v9, multiple embeds are allowed.
so, we need to implement for it.

Was merged by https://github.com/bwmarrin/discordgo/commit/842ddb357aa177c7ef7ea7728cd3dac04e345831.


Failed to connect to MySQL 8 database

Context
When using a clean MySQL container in version 8, the following error occurs at first launch:
 Error happened during creating seafile admin.

And in seafile.log:
../common/seaf-db.c(709): Failed to connect to MySQL: Plugin caching_sha2_password could not be loaded: /usr/lib/x86_64-linux-gnu/mariadb19/plugin/caching_sha2_password.so: cannot open shared object file: No such file or directory

Fix attempts
Tried to use default-libmysqlclient-dev instead of libmariadb-dev, with no visible improvement.
Workaround
A more detailed procedure here. In a nutshell:
Connect to db container:
$ docker exec -it mysql_container_name /bin/bash

Update authentication plugin:
$ mysql -u root -p
mysql>  ALTER USER 'seafile' IDENTIFIED WITH mysql_native_password BY 'secret';

Come back on the host and set the credentials in interactive mode:
$ docker exec -it seafile_container_name seafile-server-latest/seahub.sh restart

Then everything should work fine.

The roadmap seems to have been updated, maybe a fix already exists:

Will be automatically fixed when rebasing from debian:bullseye or maybe buster-backport.
Could maybe done quite quick but I'll wait for official debian 11 support:


Actions: Job with an "if: always()" are skipped

Description
Jobs with an if: always() statement that depends on a skipped one are also skipped.
on:
  push:
    branches:
      - master

jobs:
  job1:
    name: Skipped job
    if: false
    steps:
      - run: echo "I am skipped"

  job2:
    name: Mustn't be skipped
    needs: job1
    if: always()
    steps:
      - run: echo "I should have been displayed :'("

This makes intermediate optional job impossible
Gitea Version
1.21.0+rc2
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
ubuntu
How are you running Gitea?
container from docker hub on linux
Database
SQLite

Does Github support this?

@KN4CK3R Yes, exemples :
https://stackoverflow.com/questions/66343052/github-actions-or-operator-for-needs-clause
https://stackoverflow.com/questions/76750973/how-to-execute-a-job-that-needs-a-job-that-was-skipped-in-github-actions

According to GitHub's documentation, always() should cause the step to always execute

Causes the step to always execute, and returns true, even when canceled.


Better intensity conversion in PTXPointReader.

Currently the PTX reader transforms the intensity using sqrt to brighten middle tones. It is a bit too bright. The logarithm approximates better the human perception.

Wouldn't the viewer be a better place to do intensity mapping? Leaving the scanned values as they are but rendering them differently.

Really you're right. It was so in the PTX reader implementarion I reused from a previous project of ours. I'll remove the transposition.

If you want to add the log mapping to the renderer, this is probably the best place:
https://github.com/potree/potree/blob/develop/src/materials/PointCloudMaterial.js#L741
If you intent to create a PR request, I'd need an implementation that makes logarithmic intensity optional with a define like used here: https://github.com/potree/potree/blob/develop/src/materials/PointCloudMaterial.js#L859
But you can also just do your modification, create the PR and I'll just use it as a reference to implement it myself.

By the way, I just noticed that this is wrong:
(intensity - intensityMin) / intensityMax
it should be
(intensity - intensityMin) / (intensityMax - intensityMin);
Just in case you get weird intentensity behaviour.


Fix assertion with 100-continue --- 7.1.x

#1923
Discussed the #1777 issue with @oknet. We could reuse HttpSM::state_watch_for_client_abort as 100-continue handler.
please review!

@zwoop Please review!

[approve ci]


Security group update causes: "from_port (80) and to_port (80) must both be 0 to use the 'ALL' "-1" protocol"

I have a security group defined as follows:
let lbSecurityGroup = new aws.ec2.SecurityGroup("fib-lb-sg", {
    namePrefix: "fib-lb-sg",
    ingress: [{
        protocol: "tcp",
        fromPort: 80,
        toPort: 80,
        cidrBlocks: ["<IP_ADDRESS>/0"]
    }],
    egress: [{
        protocol: "-1",
        fromPort: 0,
        toPort: 0,
        cidrBlocks: ["<IP_ADDRESS>/0"]
    }],
    vpcId: network.vpcId
});

Now I run pulumi up and then update the security group to look like this:
let lbSecurityGroup = new aws.ec2.SecurityGroup("fib-lb-sg", {
    namePrefix: "fib-lb-sg",
    ingress: [{
        protocol: "-1",
        fromPort: 80,
        toPort: 80,
        cidrBlocks: ["<IP_ADDRESS>/0"]
    }],
    egress: [{
        protocol: "-1",
        fromPort: 0,
        toPort: 0,
        cidrBlocks: ["<IP_ADDRESS>/0"]
    }],
    vpcId: network.vpcId
});

Run pulumi up. This will fail due to invalid configuration. Now we fix the configuration:
let lbSecurityGroup = new aws.ec2.SecurityGroup("fib-lb-sg", {
    namePrefix: "fib-lb-sg",
    ingress: [{
        protocol: "-1",
        fromPort: 0,
        toPort: 0,
        cidrBlocks: ["<IP_ADDRESS>/0"]
    }],
    egress: [{
        protocol: "-1",
        fromPort: 0,
        toPort: 0,
        cidrBlocks: ["<IP_ADDRESS>/0"]
    }],
    vpcId: network.vpcId
});

Run pulumi up. This will result in this error message:
updating urn:pulumi:fibonacci-dev::fibonacci::aws:ec2/securityGroup:SecurityGroup::fib-lb-sg: from_port (80) and to_port (80) must both be 0 to use the 'ALL' "-1" protocol!

Taken from a Slack thread here.

This is fixed by #500 - once that is merged, I'll get a new release out. Thanks for the detailed report allowing verification of the fix, @kennyjwilli!


Common git hooks

Is this the right place to hold and manage git hooks for all repos?
Eg for spotless?

Added (use ./setup.sh in either this repo, or in each dir for an existing project)


Request for direct message playback support

I love the playback support, however, it has one flaw -- it does not playback direct messages; if it did it would be very nice.

This should be working now.


colmap

Library name
colmap
Library description
No response
Source repository URL
bfgh
Project homepage (if different from the source repository)
No response
Anything else that is useful to know when adding (such as optional features the library may have that should be included)
F20231124 10:07:41.138367  4620 bundle_adjustment.cc:302] Check failed: solver_options.IsValid(&solver_error) Can't use SPARSE_SCHUR with Solver::Options::sparse_linear_algebra_library_type = NO_SPARSE.
: 'datasets/lego_ds2\images'

There is a colmap port in vcpkg, is it what you expected?
https://github.com/microsoft/vcpkg/tree/master/ports/colmap

There is a colmap port in vcpkg, is it what you expected? https://github.com/microsoft/vcpkg/tree/master/ports/colmap
colmap is ok.    But my issue ：
Sometimes the use is completed smoothly, sometimes it is not. Based on the data, it may be possible that the internal calculation results do not meet the conditions and an error will be reported

So what is the new port you want to add? Is there a specific download path?


CAS-87

In this PR I added code so that the strategies each implement a FetchBalance function
FetchBalance(db *shared.Database, b *models.Balance, sc *shared.SnapshotClient) (*models.Balance, error)
Before we were doing this logic in the controller. Now I have updated the code to call the same models function on struct Balance GetBalanceByAddressAndBlockHeight, I also moved the generic error and if balance.ID == "" block into the FetchBalance. This is beneficial as it reduces the length of the controller function createVoteForProposal and abstracts some logic away.
However, currently, each strategy is implementing the exact same logic for Fetch Balance, and I can't imagine a case where they would be different. If they're not going to be unique for each strategy, we could move the logic from lines 14 to 35 into a reusable function inside /shared
Additionally in other controller functions such as getVotesForProposal we are calling the models method on struct Vote GetVotesForProposal which already joins the balances to the votes in one sql query, I thought it best not to alter this.

Looks good!  I think the repeated code is fine for now.


[DEPRECATE] Service Bus - Deprecate AutoComplete and AutoLockRenew in Low Level ReceiverClient

The implicit AutoLockRenewal and AutoComplete won't work as expected when it is combined with a reactor pipeline with multiple stages.
This Gist describes the problem statement, deprecation options, agreement messaging crew, and architects reached.

@michaelqi793 - In the upcoming releases, we will be removing the auto-lock-renewal feature from the (low level) Async client. Please see this work item for why the SDK cannot reliably implement the auto-lock renewal. The work item also has a reference to more general recommendation prefetch and message lock duration.

Draft PR is out for discussion and review

The work is split into two to ease the work 33707 and 33710. Closing this.


[nrf noup] west: multi-image: Change variable used to find app dir

APPLICATION_SOURCE_DIR is not in the cache due to multi-image
requiring cache variables to be image-prefixed and due to a desire to
not image-prefix this variable.
But west expects it to be so. To resolve this, have west use a
different variable that points to the same thing.
Signed-off-by: Sebastian Bøe<EMAIL_ADDRESS>

see https://github.com/NordicPlayground/fw-nrfconnect-nrf/pulls

I'm OK with this as an nrf temphack

Do you want me to change the tag? The patch will be removed when multi-image is migrated.


Video Support

Is there a way to make a carousel of video views?

Use ExoPlayer and try to incorporate MediaSliderView library. Working perfectly for me.


crossbeam-utils v0.6.6 fails to compile with mir-opt-level>1

Hey, thanks for an awesome library - we use it in tantivy
This is less of an issue and more of an FYI ticket that you might find interesting, if you are ever poking at MIR optimisations. If you don't think this is relevant - feel free to close it, so it doesn't get in the way of more important tickets.
rustc nightly provides a -Zmir-opt-level flag that allows to set the level of optimisation that runs on MIR-level during compilation.
The default value is 1 and building tantivy with the following command succeeds.
cd tantivy/
tantivy$ RUSTFLAGS=-Zmir-opt-level=1 cargo +nightly build
    Finished dev [unoptimized + debuginfo] target(s) in 4m 04s

However running either
tantivy$ RUSTFLAGS=-Zmir-opt-level=2 cargo +nightly build
tantivy$ RUSTFLAGS=-Zmir-opt-level=3 cargo +nightly build

Fails with the following error
tantivy$ RUSTFLAGS=-Zmir-opt-level=3 cargo +nightly build
   Compiling regex v1.3.1
   Compiling rustc_version v0.2.3
   Compiling backtrace-sys v0.1.28
   Compiling rand_pcg v0.1.2
   Compiling byteorder v1.3.1
   Compiling crossbeam-utils v0.6.6
   Compiling rand_chacha v0.1.1
   Compiling rand_xorshift v0.1.1
   Compiling backtrace v0.3.15
   Compiling rand_isaac v0.1.1
   Compiling num-traits v0.2.6
Cannot invoke an intrinsic other than donothing, patchpoint, statepoint, coro_resume or coro_destroy
  invoke void @llvm.x86.sse2.pause()
          to label %100 unwind label %107, !dbg !418
Cannot invoke an intrinsic other than donothing, patchpoint, statepoint, coro_resume or coro_destroy
  invoke void @llvm.x86.sse2.pause()
          to label %100 unwind label %107, !dbg !418
in function _ZN4spin4once13Once$LT$T$GT$9call_once17he92fc40c3d396554E
LLVM ERROR: Broken function found, compilation aborted!
error: Could not compile `crossbeam-utils`.

Please find my versions of rustc and OS below
$ cargo +nightly -V
cargo 1.36.0-nightly (c4fcfb725 2019-05-15)
$ lsb_release -a
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 18.04.3 LTS
Release:	18.04
Codename:	bionic
$ git show HEAD -s
commit 7e08e0047bf842d2ce0f9cd7791451bffb85e2f6

In the current master (crossbeam-utils 0.7), the error message changed to:
$ uname -a
Linux jeehoonkang-home 5.3.12-1-MANJARO #1 SMP PREEMPT Thu Nov 21 10:55:53 UTC 2019 x86_64 GNU/Linux

$ git rev-parse HEAD
424f926673d652f306f266d4309ed8717e931144

$ RUSTFLAGS=-Zmir-opt-level=2 cargo +nightly build                                                                                ±[master]
   Compiling crossbeam-utils v0.7.0 (/home/jeehoonkang/Works/crossbeam/crossbeam-utils)
error: internal compiler error: src/librustc_mir/monomorphize/collector.rs:1240: alloc id without corresponding allocation: 136

thread 'rustc' panicked at 'Box<Any>', src/librustc_errors/lib.rs:892:9
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace.

note: the compiler unexpectedly panicked. this is a bug.

note: we would appreciate a bug report: https://github.com/rust-lang/rust/blob/master/CONTRIBUTING.md#bug-reports

note: rustc 1.41.0-nightly (a44774c3a 2019-11-25) running on x86_64-unknown-linux-gnu

note: compiler flags: -Z mir-opt-level=2 -C debuginfo=2 -C incremental --crate-type lib

note: some of the compiler flags provided by cargo are hidden

error: aborting due to previous error

error: could not compile `crossbeam-utils`.

To learn more, run the command again with --verbose.

At this stage, I believe it's not an issue of crossbeam-utils but that of miri.

This seems to be fixed on the current master (https://github.com/crossbeam-rs/crossbeam/commit/2f1ed42da69f93d54d4a648c1fee6c54fb739aec).
$ uname -srvmpio
Darwin 19.5.0 Darwin Kernel Version 19.5.0: Tue May 26 20:41:44 PDT 2020; root:xnu-6153.121.2~2/RELEASE_X86_64 x86_64 i386 MacBookPro15,1 Darwin
$ git rev-parse HEAD
2f1ed42da69f93d54d4a648c1fee6c54fb739aec
$ cargo +nightly -V
cargo 1.45.0-nightly (40ebd5220 2020-06-01)
$ RUSTFLAGS=-Zmir-opt-level=2 cargo +nightly build
   Compiling autocfg v1.0.0
   Compiling lazy_static v1.4.0
   Compiling cfg-if v0.1.10
   Compiling crossbeam-utils v0.7.2 (/Users/taiki/rust/local/crossbeam/crossbeam-utils)
    Finished dev [unoptimized + debuginfo] target(s) in 1.70s
$ RUSTFLAGS=-Zmir-opt-level=3 cargo +nightly build
   Compiling autocfg v1.0.0
   Compiling lazy_static v1.4.0
   Compiling cfg-if v0.1.10
   Compiling crossbeam-utils v0.7.2 (/Users/taiki/rust/local/crossbeam/crossbeam-utils)
    Finished dev [unoptimized + debuginfo] target(s) in 1.58s


Representing CHEMINF in the Bioregistry

The Bioregistry is a registry of prefixes and their associated metadata (it accomplishes the same goals as Identifiers.org, but it's completely open source). It additionally has a slew of manual, automated, and semi-automated tools for mapping to external registries like Identifiers.org, N2T, etc. that list additional metadata about prefixes.
CHEMINF curates information about identifiers as children of IAO_0000577 and is therefore of interest. In https://github.com/biopragmatics/bioregistry/pull/255, I've implemented automatic alignment and curation sheet generation for CHEMINF in the Bioregistry. I was able to match several of the entries here, but the following 7 remain:

cheminf_id
name
description
Issue/comment

000234
PubChem conformer
None
This seems like something that's only locally useful given a CID and there isn't an endpoint that resolves them

000303
GenBank Protein Identifier
None
Could this be the same as NCBIProtein?

000447
European Community number
A seven-digit identifier for chemical substances for regulatory purposes within the European Union.
https://github.com/biopragmatics/bioregistry/issues/257

000467
validated chemical database
None

000565
National Service Center number
Identifier used by the Cancer Chemotherapy National Service Center.
https://github.com/biopragmatics/bioregistry/issues/258

000566
RTECS
Identifier used by the RTECS database (http://www.cdc.gov/niosh/rtecs/).
Seems like RTECS is a proprietary database. Not sure why it's in the ontology, this isn't generally useful

Would anyone be able to help give a bit more information about what some of these are and why they're in CHEMINF? E.g., I am familiar with PubChem, but don't really know what a PubChem conformer idea is. Same for the rest (except RTECS, which I haven't looked into via that link yet)
If you want to request a new prefix in the Bioregistry, you can do so with this GitHub issue template: https://github.com/biopragmatics/bioregistry/issues/new?assignees=cthoyt&labels=New%2CPrefix&template=new-prefix.yml&title=Add+prefix+[X]

@egonw can you comment on this?

Intuitively, I would say, most of these are not worth the effort right now. Perhaps except for the last one?
About why, I can look up late who added it (git blame) and figure out the context, but it's not high on my last, I'm afraid.


Ayodola added his name

Can you please update your code by pulling the latest code from the original repo?

git pull upstream main

git push origin main


Generate a no_std binary for test crate

When using trybuild with a target that doesn't support the standard
library, the previous main.rs would not compile. This new one is less
demanding, and works in such a scenario.

Travis fails with an error that I didn't catch locally. I'm currently investigating.

I don't think this approach can work. It just happened to work when I tested it in my project, I'm guessing because everything there is set up for cross-compilation to a microcontroller target. It won't work when running cargo test in a regular project.
The compiler error could be fixed by setting panic = "abort" in Cargo.toml, but I don't think that's the right thing to do.
I'm closing this pull request and will open an issue to propose another solution to fix my problem.


Support for Azure?

Hi!
I am using this in our project and it is a great product. Just now we got a customer using the Azure platform and I'm not sure if the LiteDB works for Azure blob storage sinca you cannot use physical paths in Azure?
If not, do you think there would be a lot of work to add Azure support?
Thanks for a great job Mauricio!
/Kristoffer

not yet

Any news?
Have you guys been using Azure websites with LiteDB? Can it work somehow, or the only solution is VM?

https://docs.microsoft.com/en-us/azure/storage/files/storage-dotnet-how-to-use-files

Has anyone implemented this in 2021? 😅

Hi.
Would like to ask if there were any development regarding supporting LiteDb over TCP.
I have an azure implementation and planned to use LiteDb but between paying for a VM and changing the DB I would Change the DB but would love to use LiteDb if possible.
Thanks.


Fix typos in Complex Arithmetic katas

Fix a LaTeX error in Cartesian to Polar Conversion solution.
Fix a mistake in Polar Coordinates.

Hope that helps! 😊

@microsoft-github-policy-service agree


Add command to generate .sdkmanrc file

To raise a new Pull Request, the following prerequisites need to be met. Please tick before proceeding:

[x] a conversation was held in the appropriate SDKMAN Slack channel.
[x] a Github Issue was opened for this feature / bug.
[x] test coverage was added (Cucumber or Spock as appropriate).

This addresses Issue #730

Hi @marc0der,
this is the current WIP of the sdk env init command.
Some open questions:

Which Java version should I add as an example? Is there some easy way to get the currently used Java version, and fall back on some default?
If sdk env is called with some "subcommand" other than init what would be the expected outcome? Should an error be displayed? Should just sdk env be called?

Apologies, I was reviewing this commit by commit so not all the comments make sense by the end.

@marc0der Thanks for the valuable suggestions, I suck at error messages / warnings ;).
I've also simplified things a bit, the lines are now normalised (stripped by whitespace and comments) before performing the format check, which will help the user in case he entered something invalid.
I've also fallen back to using = for separating candidate and version, as some devs may already use the beta version (and it actually doesn't make a difference what delimiter is used).

Sounds sensible yes. Haven't looked into it, but the build on your branch seems to be failing. Once we have a green build I'd be happy to merge.

Hi @marc0der,
I've fixed the outstanding issues, added some more tests, fixed some broken tests and changed how the .sdkmanrc file is created (it inserts the currently used java version or falls back to some fixed java version).
There are still some untested code branches and minor things which can be improved but IMHO nothing which prevents a release.

It seems I can use version=$(__sdkman_secure_curl "${SDKMAN_CANDIDATES_API}/candidates/default/${candidate}") to determine the fallback Java version?

Some very minor naming bits  in the tests, but almost ready to merge! Also, I hope you don't mind me squashing your commits.

Hi @marc0der,
I've added one more test and performed the suggested fixes.


Add annotations to multiple gene orthologs

Let's make it easy to spread new annotation data to relevant gene orthologs. Somewhere in each annotation submission form, ask the user to check boxes for other orthologs to copy the annotation to.  Should be unchecked by default, to avoid inadvertent copying. Maybe also on the final 'review annotation submission' view, make it clear to the user that they will be writing the annotation to additional strains.

@djow2019 I was just thinking about this feature/issue. It isn't urgent, but sure would be nice to have before I proceed with annotating a bunch of genes. If easy to implement, then 'urgent'. 😁

@khybiske Which forms? Just GO annotations? HP?

I would say every one except mutants. Gene name too (the one you created the other day).

This is a good example, since I/you changed the gene name for CTL0260, but that should be pushed to CT_005 as well. I meant to change it for that ortholog but forgot to.

Ok. I can make it work for all of them except the operon form. That might have to be done individually...

No problem. Operon is the least important, since we don't have operon data for 99% of the genes.

@khybiske Multiple ortholog option now live for GO annotations only

HP Interactions with orthologs now done

@khybiske All done! You can now select which genes to apply the annotations to for the gene name form, GO forms, and HP form. I reset CTL0260 back to hypothetical protein so you can confirm that the feature works.


docs: Add user guide for Advanced search feature

Automated message from Dropbox CLA bot
@synicalsyntax, it looks like you've already signed the Dropbox CLA. Thanks!

Ready to go, thank you @synicalsyntax for the great work. @timabbott please merge when you are able. Thanks!

This is awesome, merged, thanks @synicalsyntax!!!

(I added a quick link to this from the end of the basic search article).


Master to Live

OPS Build status updates of commit 4ce43f8:
:clock10: Pending: waiting for processors (19 builds ahead of you)
:warning: OPS Build is busy, click here to report any potential performance downgrade issues, or you could view OPS Portal for trend of build queue length.

OPS Build status updates of commit 4ce43f8:
:clock10: Preparing: average preparing time is 1 min(s) 12 sec(s)

OPS Build status updates of commit 4ce43f8:
:clock10: Incremental building: average incremental building time is 12 min(s) 22 sec(s)

OPS Build status updates of commit 4ce43f8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/advanced-threat-protection/Set-AtpPolicyForO365.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/antispam-antimalware/New-HostedContentFilterPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/antispam-antimalware/Set-HostedContentFilterPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/client-access-servers/Get-ClientAccessServer.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/client-access-servers/Set-ClientAccessServer.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/client-access-servers/Set-ClientAccessService.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/client-access/Get-CASMailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/client-access/Test-PopConnectivity.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/database-availability-groups/Set-DatabaseAvailabilityGroup.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/database-availability-groups/Update-MailboxDatabaseCopy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/devices/Get-ActiveSyncDevice.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/devices/Get-MobileDevice.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/encryption-and-certificates/Set-OMEConfiguration.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/encryption-and-certificates/Set-SmimeConfig.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/federation-and-hybrid/Disable-RemoteMailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/federation-and-hybrid/Enable-RemoteMailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/federation-and-hybrid/Get-RemoteMailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/federation-and-hybrid/New-RemoteMailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/mailbox-databases-and-servers/Set-MailboxDatabase.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/mailboxes/Connect-Mailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/mailboxes/Disable-Mailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/mailboxes/Enable-Mailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/mailboxes/Get-Mailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/mailboxes/Get-MailboxFolderPermission.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/mailboxes/Get-MailboxFolderStatistics.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/mailboxes/Get-UserPhoto.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/mailboxes/New-App.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/mailboxes/New-Mailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/mailboxes/Remove-Mailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/mailboxes/Remove-UserPhoto.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/mailboxes/Set-Mailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/mailboxes/Set-UserPhoto.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/mailboxes/Undo-SoftDeletedMailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/move-and-migration/New-MigrationBatch.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/move-and-migration/New-MigrationEndpoint.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/move-and-migration/New-MoveRequest.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/move-and-migration/Set-MigrationConfig.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/organization/Set-OrganizationConfig.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/policy-and-compliance-audit/Search-UnifiedAuditLog.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/server-health-and-performance/Set-ServerComponentState.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/server-health-and-performance/Set-ThrottlingPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/unified-messaging/Set-UMService.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/users-and-groups/Get-MailUser.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/users-and-groups/Get-Recipient.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/users-and-groups/Get-UnifiedGroup.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/users-and-groups/Get-User.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/users-and-groups/Set-MailContact.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/users-and-groups/Set-User.md
:white_check_mark:Succeeded
View (exchange-ps)

sharepoint/sharepoint-ps/sharepoint-online/Add-SPOTenantCdnOrigin.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-online/Export-SPOQueryLogs.md
:white_check_mark:Succeeded
View (sharepoint-ps)

This comment lists only the first 50 files in the pull request.
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-1.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 2358

Questions ?
Please refer to the Backport tool documentation


error on $ inside quoted_argument

message(STATUS "$")

normal_command [225, 0] - [225, 19]
  identifier [225, 0] - [225, 7]
  argument [225, 8] - [225, 14]
    unquoted_argument [225, 8] - [225, 14]
  argument [225, 15] - [225, 18]
    quoted_argument [225, 15] - [225, 18]
      ERROR [225, 16] - [225, 17]

this bug requires a substaintial rewrite of the grammar, so I'm not sure when will i have time to fix this bug.

this bug requires a substaintial rewrite of the grammar, so I'm not sure when will i have time to fix this bug.

This bug is fixed now