Tensorflow lite build error for Aarch64

Click to expand!
Issue Type
Build/Install
Have you reproduced the bug with TF nightly?
Yes
Source
source
Tensorflow Version
Tensorflow lite master branch
Custom Code
No
OS Platform and Distribution
Ubuntu 18.04.6 LTS
Mobile device
Nvidia Jetson nano
Python version
3.8
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
Trying to build tensorflow lite wheels using the following command
`
PYTHON=python3 tensorflow/lite/tools/pip_package/build_pip_package_with_cmake.sh native
`

While trying to build, getting compilation errors of the likes:

In file included from /home/piyush/tensorflow_src/tensorflow/lite/tools/pip_package/gen/tflite_pip/python3/cmake_build/xnnpack/src/amalgam/neonfp16arith.c:8::
/usr/lib/gcc/aarch64-linux-gnu/7/include/arm_neon.h:30805:1: error: inlining failed in call to always_inline ‘vaddq_f16’: target specific option mismatch
 vaddq_f16 (float16x8_t __a, float16x8_t __b)
 ^~~~~~~~~
/home/piyush/tensorflow_src/tensorflow/lite/tools/pip_package/gen/tflite_pip/python3/cmake_build/xnnpack/src/amalgam/neonfp16arith.c:277:27: note: called from here
         const float16x8_t vsum2345 = vaddq_f16(vsum23, vsum45);

### Standalone code to reproduce the issue

```shell
Build using above command on Aarch64. I am using Nvidia Jetson Nano with Ubuntu 18.04.6 LTS

Relevant log output
In file included from /home/piyush/tensorflow_src/tensorflow/lite/tools/pip_package/gen/tflite_pip/python3/cmake_build/xnnpack/src/amalgam/neonfp16arith.c:8::
/usr/lib/gcc/aarch64-linux-gnu/7/include/arm_neon.h:30805:1: error: inlining failed in call to always_inline ‘vaddq_f16’: target specific option mismatch
 vaddq_f16 (float16x8_t __a, float16x8_t __b)
 ^~~~~~~~~
/home/piyush/tensorflow_src/tensorflow/lite/tools/pip_package/gen/tflite_pip/python3/cmake_build/xnnpack/src/amalgam/neonfp16arith.c:120:27: note: called from here
         const float16x8_t vsum = vaddq_f16(vsum2345, vsum01678);

Hello @tilakrayal @pjpratik any ideas? Thanks in advance.

Hi @pjindal91 Thanks for reporting the issue.
Sorry for the delayed response.
You can try to build with target architecture PYTHON=python3 tensorflow/lite/tools/pip_package/build_pip_package_with_cmake.sh aarch64 in your case.
I was able to build for aarch64 using tool chain as given in documentation successfully on Ubuntu 20.04. Please find the gist here.
Thanks.

Hi @pjpratik thanks for the response. I think the issue might be specific to platform and the ubuntu/glibc versions i am using. Would it be possible for you to try to natively built on aarch64 with Ubuntu 18.04.6 LTS and glibc=2.27?

FYI @pjpratik in case it helps i was able to build v2.4.0 successfully. So not sure if v2.5.0 is pulling some buggy lib or the above setup i have(Ubuntu 18.04.6 LTS and glibc=2.27) is not compatible with it.

Hi @pjindal91 sorry for the delayed response
I have tried to built natively on aarch64 with Ubuntu 18.04.6 LTS and glibc=2.27 with v2.5 and master branch as well and the attempt was successful. Please find the screenshots below
For v2.5

For master

Thanks.


toHaveSubscriptions not working

after update from 0.4.1 to 0.5.0 my test broke on toHaveSubscriptions:
Expected $[1] to be a kind of SubscriptionLog, but was SubscriptionLog({ subscribedFrame: 30, unsubscribedFrame: 50 }).
    at <Jasmine>
    at compare (http://localhost:9876/_karma_webpack_/node_modules/jasmine-marbles/index.js:57:1)
    at <Jasmine>

Tests are for ngrx effects. Versions:
"jasmine-marbles": "0.5.0",
"rxjs": "6.5.2"
"@ngrx/effects": "8.2.0",

I have no idea what's wrong. 0.4.1 works fine when rolled back

The problem here is that SubscriptionLog is being imported in two different ways:

node_modules/rxjs/internal/testing/SubscriptionLog.js
node_modules/rxjs/_esm2015/internal/testing/SubscriptionLog.js

Jasmine is upset that the types are different. Unfortunately, this leaves me with no way to run jasmine-marbles, as setup doesn't work right when rolled back to 0.4.1.

Reported as angular-cli#16941.
As a workaround, add this to tsconfig.json:
{
  "compileOnSave": false,
  "compilerOptions": {
    ...
    "paths": {
      "rxjs/*": [
        "node_modules/rxjs/_esm2015/*"
      ],
    },
    ...
  },
}

Angular-CLI reports that this is a side effect of jasmine-marbles deep importing into RxJS. It should not be using internal paths.

+1

Closed via https://github.com/synapse-wireless-labs/jasmine-marbles/pull/64


Add support to run all 'test.html' files in a dir recursively.

The only problem with this is that it is slightly slow because it has to spawn new PhantomJS pages for each file. It would be better to just be able to reuse the current PhantomJS instance. I don't know if you can do that or how to do it. If someone could help point me in the right direction for that it would be great.

Closing as this is a bad idea since we can't aggregate the tests results in a meaningful way. It's much better to have one test runner and use the grep option (-g). Otherwise like I mentioned above, this is just find . -name "*.test.html" -exec mocha-phantomjs {} \; which you can do yourself if you wish.


Fix leakage of ppd

See https://github.com/apple/cups/pull/5738

Thank you for the review and the merge, Mike!


isPlaying() behaviour across backends

Have I got it right that isPlaying() returns true when paused on desktop, but false on the other backends?
I discovered this when I was testing pause functionality and couldn't get the pause status out of the desktop video player.

Hmm yeah, that shouldn't be. I think the easiest way is to change it to playing && !paused

Changed in f3567a00809da697f0a320cba95e1bc295a51d12


O1 FTSO

Please do not modify existing entries.

Fixed in https://github.com/TowoLabs/ftso-signal-providers/commit/da932b0854ad6bdd0be6964a40c6e183b9759d56


Request to add Point-SAM.

Thank you for your great collection!
I am the author of the paper "Point-SAM: Promptable 3D Segmentation Model for Point Clouds
". Our code can be found at: https://github.com/zyc00/Point-SAM and our web page can be found at https://point-sam.github.io/.
It would be appreciated if you could add our paper and code link, thank you very much.

@zyc00 Thank you for your attention to my work. I have added.


How to use Angular 4?

I am using Orchard branch 1.10.x. I need to use Angular 4 for a new project. The new version of Angular has not a simple JavaScript file. I should install, test, build and bundle it using npm, webpack, ... and a modular system. Also it uses TypeScript. But Orchard uses Assets.json file with Gulp.
How should i setup my module or theme to use the new version of Angular?

Use whatever tool you want instead of Assets.json and Gulp. In the end your javascript files should just go in a folder of the module so that you can reference them from the views.

I use Aurelia in some projects and typically have an "Aurelia" - folder in my project, which is completely self-contained. Then I write a ResourceManifest.cs that points to the exported bundle.
Small tip: Since angular 4 uses TypeScript I highly recommend to check out the TypeWriter Extension for VisualStudio, so you can easily share ViewModels and Interfaces with your angular app.

A special thanks to all you guys.
@sebastienros Do you confirm that creating a package.json or node_modules folder under a module or theme folder cannot cause a problem?
Is there any recommended way specially about Angular 4 and Orchard?
Also as you know Orchard.Layouts and some other modules uses Angular 1.x. Is there any plan to upgrade them for using new version of Angular in the future? (Because loading two different versions of Angular will be a problem.)

I'll jump in here again real quickly, I hope you don't mind ;)
We have jspm and node folders in many different modules and never had any real problems whatsoever.

@Xceno Thank you for your help.


Filtered responses that are text only don't include author property

This is a Bug Report
Description

Steps to reproduce

Add new filter with web:{your-account-email} scope, as well as Closed, Application, and Error selected
Ask davis (web UI) what happened at 1pm? or another specific time that results in a text only response
Notice that the author property isn't used since attachments aren't used

Desired behavior
Always use cards for responses that can be filtered so that the author property can be set to filtered

Note: It also appears that audible responses don't include anything about a filter being used, which could be problematic if the user doesn't remember or know a filter has been applied. I realize Alexa isn't supported yet for filtering, so this is more of a ToDo.

Also, shouldn't we not allow users to ask about the future, like above?

@cwoolfRuxit Can you give me an example of a future query?


add option for synchronous requests to client lib

This would add an option to the client lib to switch the XHR to synchronous behaviour.
I just needed this for a client that switched from some other xhr lib to the superagent client lib but had one weird feature where synchronous was definitely needed.
I do understand that synchronous ajax requests are usually a total no-go and you probably might not want to have this in superagent at all.
If you, against all odds, consider to merge this one I'd be happy to provide more changes to round this up (docs, tests).
Might be related to #34, but the discussion there is about node, I believe.

Still needed when doing a request inside onunload and related events. Please re-consider adding it :-)

+1

+1

+1

This will not be added.
Synchronous XHR is an awful mistake in the DOM API. We shouldn't be getting in the way of its deprecation and eventual removal.

Same reason with @SanderSpies which is onunload event. While Navigator.sendBeacon can only send POST and can not set custom header is spec. I know it will be deprecate in the future, but this is an option variable in XMLHttpRequest spec. I think library should not limit the usage, developer have to know what they are doing.

I know sendBeacon is bad, but I still don't think it's a reason to allow sync XHR. No, no no. If you want to use a deprecated API, then use it directly.

Our use case is we're trying to get off MeteorJS onto a new api. Alot of meteor runs sync due to keeping the DB connection open in the node side. Our api calls need to run sync at the node level similarly.
I know it's might be an edge case,  but it's what we're facing.

AFAIK there is no synchronous networking available in regular node whatsoever. It's not possible to make a synchronous HTTP request in node. Maybe meteor is doing it via fibers but superagent only supports official node and browsers.

@focusaurus I didn't realize this about node, likely due to my confusion about the yield and async syntax. You're exactly right Meteor does this through fibers and their own implementation to do async promises Promise.await(returnsPromise()), for which Superagent worked beautifully.


Integration Test - Release - 2018/05/29 11:31:08

Auto create for integration test. AnotherAuthorUpdatedArticleFlow

@PRMergerTest3 : Thanks for your contribution! The author, @ v-caxian, has been notified to review your proposed change.


BUG: at least one entry from trial should be used in exponential crossover

According to 10.1023/A:1008202821328 it seems like at least one point should be used from the trial vector during exponential crossover.
This is backed up J Heuristics .
This is obviously a reasonably long standing issue, but since the exponential crossovers aren't the default strategy hopefully the impact wouldn't be large.

Could you point to the relevant sections of the paper? If it is clear that at least one entry should be forced to be used, it should be pretty easy to merge. Since I'm not sure what it says in the paper, should we check to see whether at least one element is already chosen before forcing the zeroth element to be chosen?
One way to enforce this would be to set the minimum element of crossovers to be zero after initially generating the array.

From the second paper it's fig 4, sect 2.3.2.
The first paper is the original outlining of the method. It's less clear as the terminology isn't developed yet, but it's the pseudocode at the top of Pg7. The first paper is findable on Researchgate.
The exponential crossover approach says start filling from a random point in the trial array (fill_point is chosen randomly). Swap at least the first item (trial[fill_point]). Then continue to fill, in modulo, until the randomly chosen numbers (contained in crossover) is greater than cross_over_probability. crossovers is used to hold whether the randomly chosen number is less than cross_over_probability, i.e. chosen or not. Setting crossovers[0] = True is saying that we definitely take at least one item.

Setting crossovers[0] = True is saying that we definitely take at least one item.

Yes, but it will take the zeroth item even if another item is already taken. Is that desirable, or do we only want to intervene only if no items are already taken?

Never mind. I looked at the code and the algorithm more closely.

Was just composing a reply when you edited.

Thanks. Sorry for the trouble.
If the algorithm was otherwise correct, it looks to me like this ensures that the first $c_{i, k, n} = v_{i, k, n}$ is performed.

That's correct. At least one is filled. The probabilities of filling the entries is:
p(0) = 1
p(1) = 1 * (U[0, 1) < CR)
p(2) = 1 * (U[0, 1) < CR) * (U[0, 1) < CR)
etc
where each of the random samples is independent.

THere might be a smart way of vectorising the loop, but it's probably clearer the way its written

Now I'm just looking for minor issues with the number of iterations. I think there's one, but it's possible I'm reading incorrectly.
Since this is a do while loop, doesn't this guarantee that a $c_{i, k, n} = v_{i, k, n}$ is performed inside the do while even after it is performed for the first time before the do while? (Not in Fortran, apparently. I guess a Fortran do while works like while in other languages?)
In the Python code, the maximum number of trial[fill_point] = bprime[fill_point] that can occur is self.parameter_count. This is because i starts as 0, increments within the loop, and stops if not i < self.parameter_count. So if self.parameter_count = 5$, trial[fill_point] = bprime[fill_point]`  can happen with $i = [0, 1, 2, 3, 4]$.
In the algorithm, however, it looks like the maximum number of times $c_{i, k, n} = v_{i, k, n}$ is performed is $N-1$. It is guaranteed to happen once before the loop, which we can think of as $L = 0$ (even though $L$ isn't defined yet). Then $L = 1$, and $L$ increments inside the loop, stopping when not $L < N - 1$. So if $N = 5$, it happens for $L=0$ (effectively), and it can happen for $L=1$, $L=2$, and $L=3$, but it can't happen for $L = N - 1 = 4$, right?

I don't think the Fortran pseudocode is correct. All the parameters should be replaceable with entries from bprime.

OK. In any case, that is not modified by this PR, so there's no obligation to change it here.
This PR appears to fix a bug, so let's merge.

This is a one-line change, and the conclusion appears to be that it is a bug fix, so I've labelled for tentative backporting for 1.10.1. If you know of a reason no to backport just remove the label and/or let me know when I open the next backport PR for 1.10.1.

Yes, should be back ported.


Qpdf throwing the range conversion error

I am trying to encrypt the pdf file. But it throws the range error.
"integer out of range converting<PHONE_NUMBER> from a 8-byte signed type to a 4-byte signed type"
Please clarify why it appears and how to fix.
@jberkenbilt
TIA

This is most likely due to a malformed input PDF file, but to be able to help you, I would need the input file and to know what version of qpdf you are using.

We are using the version qpdf version 9.1.1.
The PDF is in a proper format when opened. But I may not be able to share it here, as it is company asset.
Thanks

You might try qpdf 10.0.3 (just out) and see if it does better. I feel like I fixed some problem with out-of-spec encryption dictionary parameters. There are lots and lots of files that are not compliant with the PDF spec but that open fine in most readers. qpdf is very good at handling these most of the time, but there are always special cases that it doesn't (yet) handle. I try to enhance qpdf to cover these when I learn about them.
If you are not able to share the PDF, you can try running qpdf --json file.pdf and send that. This won't include any content (page text or images), though it will include things like text strings from bookmarks, if any, and some metadata. Look over the output and send it if you can. Most of the time, people are able to share this even when they can't share the file. Usually I can diagnose most problems by just looking at the json.
Alternatively, you can try, as a workaround, first running qpdf --decrypt in.pdf out.pdf and then encrypting the result of that. Sometimes, when qpdf just rewrites a file, it will fix a number of problems of this type.

Hi @jberkenbilt
We have tried your commands, they throw the same error even when upgrading to higher version.
Even qpdf --json file.pdf also throws the same error.
But we observed other pdfs are working fine.
I cannot attach the pdf file here, and cannot give the url as well.
integer out of range converting<PHONE_NUMBER> from a 8-byte signed type to a 4-byte signed type
The same error is happening for the document encryption.

Thanks, I am studying the problem, but in the meantime, I was able to manually recover the file and am attaching the repaired file:
LOA-workplace-pension_v6-fixed.pdf
I'll update this ticket when I have a proper fix and/or a usable workaround.

Thanks for the reply.
But this thing goes manually in our application. The template comes from docusign.
What needs to be done in that case?
Is there any problem with docusign template?

I know what the problem is. It's an easy fix in qpdf, but I'm not sure there's a good workaround without building a new version of qpdf. Technically, the issue is that the PDF file contains an out of range value in a place where the value is actually unused. Unfortunately, qpdf is range-checking the value even though it doesn't really have to. The secondary problem is that the range error is not triggering file recovery because of the type of exception that gets thrown in the code. I'm going to try to fix both problems.
Where do you get qpdf from? Do you build your own? What platform? Are you able to use a patched version of qpdf? It's going to be a very simple fix, but this error is happening so early in qpdf's reading of the file that I don't think there is going to be a suitable workaround with 10.0.3 or earlier. I'd rather not release 10.0.4 so soon after 10.0.3 just for this bug if I can avoid it, especially since this problem has actually been in qpdf since 9.0.0 when I started doing the range checks. But let me know.

We install the qpdf - apt-get install qpdf
Platform - Linux
We can use the patched version of qpdf.
Please let me know, if there is any place in the PDF which is causing the problem? So that we can avoid that field.
Thanks

The problem is that one of the xref streams has /W[1 4 4]. The third 4 indicates use of 4 bytes for an offset within the object stream. When the entry type is 0, meaning "deleted object", the value of the third field is all 1s<PHONE_NUMBER>), which is causing the range error. When writing out a deleted object in the xref stream, the software should write all 0s, not a -1 in the final position. It is the combined fact that the third element of /W in the xref stream is >= 4 and that the resulting value is greater than 2^31 that is causing the problem. I'm not sure whether this is within your control to change.

The value of the offset/generation bytes when the xref entry type is 0 is not used, so there's no reason for qpdf to be stumbling over this. I have just added code to ignore the last field if the entry type is 0.

So do we need to install the patched version now? can you please give me the version number to check.
Thanks

I haven't generated it yet. I am about to push to CI. It will be about 30 minutes before I have a final version. Then I need to figure out how to get it to you.

How big of a problem is it for you if this is not installable via apt-get? Which distribution are you on? Are you using a binary or linking C++ with the library?

If you are able to build qpdf yourself, you can download 10.0.3 and apply the patch from the commit that is referenced above. Or maybe you have the ability to change how the files are written so the issue doesn't get triggered. Using 4 bytes for the third field in an xref entry is strange and probably never necessary in any PDF file.

I am unable to build qpdf or apply the changes. If I could install via apt-get, it would be more helpful for me.
Thanks.

Ordinarily I would not release a new version of qpdf to fix a single issue unless that issue was a regression from the most recent previous release. I just released 10.0.3 on October 31, so I think it's a little soon to release a new version. If I haven't had a chance to work on new features in the next few weeks, I could release 10.0.4 with this fix and anything else that comes up, but barring some catastrophic issue, I would not want to do a release for a few weeks at least. It's disruptive to packagers and some users to get updates too frequently.
In the near term, I can easily supply a patched source distribution or a patched AppImage built from CI. The AppImage would enable you to run qpdf as an executable.

@jberkenbilt Just chiming in to say we are having the same issue with DocuSign docs. Glad to see a patch is coming in the near future.

@msmahon Thanks for letting me know. Knowing that this is affecting multiple people may accelerate my release of a new version. I guess I'll try to get a 10.0.4 out soon that has this fix. I am a consumer of DocuSign documents myself, though I guess I've never run qpdf on them. :-)

I am preparing 10.0.4 now and will have it out in the next hour or so. That will include the fix to this problem.

@jberkenbilt : Can you please confirm, the current latest version has the fix to this problem?

@santoshturamari Yes, the fix for this is in 10.0.4, which is the current latest version.

This fix worked for me too, thank you for your hard work :)

Thanks for dealing with this - had this issue with multiple readers for multiple Docusign PDFs. Now qpdf --linearize creates a document that other PDF readers can read. Kudos.

@donal-s Thanks -- I appreciate the feedback. It's good to know this is helping people.


Passing env= to session.run seems to be broken

With
@nox.session
def file_issue(session):
    session.interpreter = 'python3.6'
    session.run(
        'bash', '-c', 'echo $SOME_ENV',
        env={'SOME_ENV': 'Hello'})

produces
$ nox -s file_issue
nox > Running session file_issue
nox > Re-using existing virtualenv at .../.nox/file_issue.
nox > chdir ...
nox > bash -c echo $SOME_ENV

nox > Session file_issue successful. :)

but if I manually set the variable it "works"
$ SOME_ENV=now-i-am nox -s file_issue
nox > Running session file_issue
nox > Re-using existing virtualenv at .../.nox/file_issue.
nox > chdir ...
nox > bash -c echo $SOME_ENV
now-i-am
nox > Session file_issue successful. :)

I can confirm that the usage of subprocess.Popen isn't the issue. With
import os
import subprocess

def main():
    print('SOME_ENV=' + os.environ.get('SOME_ENV', ''))
    cmd = ['bash', '-c', 'echo SOME_ENV=$SOME_ENV']
    proc = subprocess.Popen(cmd, env={'SOME_ENV': 'HELLO'})
    assert proc.wait() == 0

if __name__ == '__main__':
    main()

the following "works":
$ SOME_ENV=HI python2.7 f.py
SOME_ENV=HI
SOME_ENV=HELLO
$ SOME_ENV=BYE python3.5 f.py
SOME_ENV=BYE
SOME_ENV=HELLO
$ python3.6 f.py
SOME_ENV=
SOME_ENV=HELLO

After sprinkling in some pdb.set_trace() the issue is that Command.run() receives env_override:
        env = self.venv.env.copy()
        env.update(self.config.env)

        for command in self.config._commands:
            if isinstance(command, Command):
                command(
                    path_override=self.venv.bin, env_override=env)
            ...

@jonparrott https://github.com/jonparrott/nox/commit/9116b5a0009fa0f3c20e2a11e3b0481b479499ce appears not to have been a free lunch.
(venv1) $ pip show nox-automation
Name: nox-automation
Version: 0.15.0
...
Location: ${HOME}/.pyenv/versions/3.6.1/envs/venv1/lib/python3.6/site-packages
...
(venv1) $ nox -s docs
nox > Running session docs
...
nox > Session docs successful. :)

vs.
(venv2) $ pip show nox-automation
Name: nox-automation
Version: 0.16.0
...
Location: ${HOME}/.pyenv/versions/3.6.1/envs/venv2/lib/python3.6/site-packages
...
(venv2) $ nox -s docs
nox > Running session docs
...
Exception occurred:
  File "${HOME}/.pyenv/versions/3.6.1/envs/venv2/lib/python3.6/site-packages/pkg_resources/__init__.py", line 854, in resolve
    raise DistributionNotFound(req, requirers)
pkg_resources.DistributionNotFound: The 'bezier' distribution was not found and is required by the application
...
nox > Session docs failed. :(

I also confirmed (by pip install /path/to/nox) that #30 fixes this regression.


V2Ray uses wrong IP after DDNS update （DDNS更新后V2Ray没有更新IP地址）

What version of V2Ray are you using (If you deploy different version on server and client, please explicitly point out)?

V2Ray 4.10.0  windows-x64 version

What's your scenario of using V2Ray? E.g., Watching YouTube videos in Chrome via Socks/VMess proxy.

mainly for Google service (Google search, YouTube etc)
主要用来使用Google的服务

What did you see? (Please describe in detail, such as timeout, fake TLS certificate etc)

After DDNS updating its IP address, V2Ray still uses old IP address
在动态域名更新IP地址后，V2Ray仍然使用旧的IP地址

What's your expectation?

I want V2Ray keep to use the domain name to get the IP Address all the time, rather than use the domain name to get IP Address only once and stick to it.
我希望V2Ray可以保持使用域名来获取IP地址，而不是仅仅使用一次动态域名来获取IP地址，然后一直使用那个IP地址

Please attach your configuration file (Mask IP addresses before submit this issue).

Server configuration:
服务端配置
 {
    "log":{
        "access":"/var/log/v2ray/access.log",
        "error":"/var/log/v2ray/error.log",
        "loglevel":"warning"
    },
    "inbounds":[
        {
            "port":*****,
            "protocol":"vmess",
            "settings":{
                "clients":[
                    {
                        "id":"uuid",
                        "level":1,
                        "alterId":100
                    },
                    {
                        "id":"uuid",
                        "level":1,
                        "alterId":100
                    },
                    {
                        "id":"uuid",
                        "level":1,
                        "alterId":100
                    },
                    {
                        "id":"uuid",
                        "level":1,
                        "alterId":100
                    },
                    {
                        "id":"uuid",
                        "level":1,
                        "alterId":100
                    }
                ]
            }
        },
        {
            "port":*****,
            "protocol":"shadowsocks",
            "settings":{
                "method":"aes-256-gcm",
                "password":"**********"
            }
        }
    ],
    "outbounds":[
        {
            "protocol":"freedom",
            "settings":{

            }
        }
    ],
    "routing":{
        "rules":[
            {
                "type":"field",
                "ip":[
                    "geoip:private"
                ],
                "outboundTag":"blocked"
            }
        ]
    }
}

Client configuration:
客户端配置
    {
   "log":{
      "loglevel":"warning",
      "access":"access.log",
      "error":"error.log"
   },
   "inbounds":[
      {
         "port":1080,
         "listen":"<IP_ADDRESS>",
         "tag":"socks-inbound",
         "protocol":"socks",
         "settings":{
            "auth":"noauth",
            "udp":false,
            "ip":"<IP_ADDRESS>"
         }
      },
      {
         "port":4321,
         "listen":"<IP_ADDRESS>",
         "protocol":"http",
         "settings":{
            "timeout":1
         }
      }
   ],
   "outbounds":[
      {
         "protocol":"vmess",
         "settings":{
            "vnext":[
               {
                  "address":"dynamic domain 动态域名",
                  "port":*****,
                  "users":[
                     {
                        "id":"uuid"
                     }
                  ]
               }
            ]
         }
      }
   ],
   "routing":{
      "domainStrategy":"IPOnDemand",
      "rules":[
         {
            "type":"field",
            "ip":[
               "geoip:private"
            ],
            "outboundTag":"blocked"
         }
      ]
   }
}

Please attach error logs, especially the bottom lines if the file is large. Error log file is usually at /var/log/v2ray/error.log on Linux.

Server error log:
服务端错误日志
There is no error in the log
日志无错误
Client error log:
客户端错误日志
2019/02/26 08:32:58 [Warning] v2ray.com/core: V2Ray 4.10.0 started
2019/02/26 08:33:38 [Warning] [212057009] v2ray.com/core/app/proxyman/outbound: failed to process outbound traffic > v2ray.com/core/proxy/vmess/outbound: failed to find an available destination > v2ray.com/core/common/retry: [dial tcp 旧的IP地址: connectex: A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond. dial tcp 旧的IP地址: operation was canceled dial tcp: operation was canceled] > v2ray.com/core/common/retry: all retry attempts failed
2019/02/26 08:33:43 [Warning] [1379026083] v2ray.com/core/app/proxyman/outbound: failed to process outbound traffic > v2ray.com/core/proxy/vmess/outbound: failed to find an available destination > v2ray.com/core/common/retry: [dial tcp 旧的IP地址: connectex: A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond. dial tcp 旧的IP地址: operation was canceled dial tcp: operation was canceled] > v2ray.com/core/common/retry: all retry attempts failed
2019/02/26 08:33:57 [Warning] [1061853628] v2ray.com/core/app/proxyman/outbound: failed to process outbound traffic > v2ray.com/core/proxy/vmess/outbound: failed to find an available destination > v2ray.com/core/common/retry: [dial tcp 旧的IP地址: connectex: A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond. dial tcp 旧的IP地址: operation was canceled dial tcp: operation was canceled] > v2ray.com/core/common/retry: all retry attempts failed
2019/02/26 08:33:59 [Warning] [3358482760] v2ray.com/core/app/proxyman/outbound: failed to process outbound traffic > v2ray.com/core/proxy/vmess/outbound: failed to find an available destination > v2ray.com/core/common/retry: [dial tcp 旧的IP地址: connectex: A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond. dial tcp 旧的IP地址: operation was canceled dial tcp: operation was canceled] > v2ray.com/core/common/retry: all retry attempts failed
2019/02/26 08:34:00 [Warning] [1319245132] v2ray.com/core/app/proxyman/outbound: failed to process outbound traffic > v2ray.com/core/proxy/vmess/outbound: failed to find an available destination > v2ray.com/core/common/retry: [dial tcp 旧的IP地址: connectex: A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond. dial tcp 旧的IP地址: operation was canceled dial tcp: operation was canceled] > v2ray.com/core/common/retry: all retry attempts failed

？？？我个人拿ddns域名作为服务器地址一切正常
可以尝试先检查一下客户端dns解析记录有没有成功更新（可能ddns ttl设置的太大还没有刷新）

？？？我个人拿ddns域名作为服务器地址一切正常
可以尝试先检查一下客户端dns解析记录有没有成功更新（可能ddns ttl设置的太大还没有刷新）

请问你在使用V2Ray的过程中试过更新DDNS的IP地址吗？
我的DDNS是一天一更，昨天开的V2Ray到了今天就用不了，DDNS已经更新成功了，我检查过DNS解析记录，显示的是新的IP地址，但是V2Ray还在用旧的IP地址

v2ray可能不会连接一次就解析一次吧，这样可能会增加DNS解析的时间
建议使用DDNS脚本执行DDNS更新后重启v2ray服务。
举例，使用cloudxns的ddns脚本：
https://github.com/kkkgo/CloudXNS-DDNS-with-BashShell/blob/master/cloudxns_ddns.sh
在48行后加入：

service v2ray restart

检查了几天之后，我觉得应该是校园网ISP(中国移动)的问题。
使用电信或联通网络时，V2Ray是能够更新IP地址的，打扰了。


when php artisan migrate throws error

so, it happened like this:

I created few migrations
ran php artisan migrate
php artisan migrate:rollback -> threw an error after deleting 2 tables.

so now, php artisan migrate believes that my 2 tables were not deleted according to migrations table.
I see manually changing entries in database as only option.
so, I delete other tables from recent migration and proceed to delete migration log from migration table. as it happens to be ---I am using postgres with pgAdmin (on windows),--- its gui does not allow editing migration table because it lacks a primary key. so I have to use sql. I am like screw it, I will delete the database and raise an issue on github.
My issue is if error occurs while php artisan migrateing, then restoring db to a valid state manually is a big pain. what is the recommended way to handle it?
PS: I am using laravel 5.1

The only option available is carefully creating the proper down method for every migration taking into account everything that may break your schema.

the error came while it was running down method, but it is also possible during up method as well.
The best thing would be, in my opininon, to run migrations in a transaction and roll back in case of exceptions. I don't think mysql supports it. but for other dbs, this is an option. only I don't how to do this with laravel migrations?

Please ask questions on the forums.


Uninitialized variable from outer scope can be used inside function

TypeScript Version:  2.2.0-dev.20170123
Code
function f() {
    x.match(/\d/);
}

let x: string;

f();

Expected behavior:
Error Variable 'x' is used before being assigned.
Actual behavior:
No errors.
Config:
{
    "compilerOptions": {
        "module": "commonjs",
        "moduleResolution": "node",
        "target": "es2015",
        "noImplicitAny": true,
        "noImplicitThis": true,
        "strictNullChecks": true,
        "outDir": "build"
    },
    "files": [
        "test.ts"
    ]
}

The compiler does not do flow analysis at call sites. so the assumption here is that you declared x as string, so it should be string.
The complete solution requires running the control flow analysis with in lining the function at every call site, which is not that feasible. you can see more related discussion in https://github.com/Microsoft/TypeScript/issues/9998

Related to #4244

@mhegazy That's OK, but why does let x: string; even compile with strict null checks on? Shouldn't it requite x to be string | undefined to  x be able to uninitialised?

that would be too restrictive. consider..
let x: string;
if (condition) {
   var foo = ..
   x = foo();
}
else {
    var bar = ..
    x = bar();
}

x.indexOf("...") //OK.

What's wrong with writing let x: string = '';?
For me it basically breaks whole 'if something is not | null (or | undefine) then it's definitely a valid object` guarantee.


CrystalFormat has not triggered for some split window

Hello, thank you for the plugin!
Steps to reproduce the issue:

Create two files, say,

a.cr:
def foo
  "foo"
end

b.cr:
def bar
  "bar"
end

Open the files with split windows:
[n]vim -o a.cr b.cr

Make some changes to file a.cr (e.g. change lines indentation) and save the buffer: :w.

Expected result: the CrystalFormat should be triggered and lines should be reindented.
Actual result: No format changes happened.
But for the file b.cr all works just fine.
I've tried Nvim 0.2.2 with this minimal config:
" initialize
call plug#begin('~/.config/nvim/plugged')

Plug 'rhysd/vim-crystal'
let g:crystal_auto_format = 1

" finalize
call plug#end()

Yes, auto formatting is only for the current buffer in order to avoid formatting an unexpected file. So the file in another window will not be formatted.

@sudo-nice Thank you for reporting this. I fixed this at d1acf59. Please update to the latest.

Thank you!


Add mute for chatty rooms

Add a UI element so users can temporarily mute notifications on a per-room basis

#219


after live sass compiler, sftp don't upload compiled file

Do you read the FAQ?
Yes.
Describe the bug:
Live sass compiler with https://github.com/ritwickdey/vscode-live-sass-compiler make style.css.min file but this file do not auto upload
in sftp.json (accesses are hide, only one file do not upload after save):
{
"name": "name",
"host": "host",
"protocol": "ftp",
"port": 21,
"username": "username",
"password": "password",
"remotePath": "/remotePath/",
"uploadOnSave": true,
"watcher": {
"files": "/style.min.css",
"autoUpload": true
}
}
What i need to do^ to make auto upload generation file style.min.css?

I've the same issue in a Gulp context while generating many files. Transfer hangs after few files of many. Developer is not active the project is dead

?

maybe we can fix this problem, or can find alternative plugin?

Fix this problem by using Gulp + vinyl-ftp

did not solve the problem? Without Gulp


[ACTION NEEDED] Fix flaky integration tests at distribution level

What is the bug?
It was observed in 2.13.0 and previous other releases that alerting manually signed off on the release for failing integration tests. See https://github.com/opensearch-project/opensearch-build/issues/4433#issuecomment-2026436873
The flakiness of the test runs take a lot of time from the release team to collect go/no-go decision and significantly lower the confidence in the release bundles.
How can one reproduce the bug?
Steps to reproduce the behavior:

Run integration testing for altering and see the failures.
Issues can be reproduced using the steps declared in AUTOCUT issues for failed integration testing

What is the expected behavior?
Tests should be consistently passing.
Do you have any additional context?
Please note that this is a hard blocker for 2.14.0 release as per the discussion here

@praveensameneni @eirsep Can you please provide your inputs?

Adding 2.14.0 release manager @rishabh6788

As mentioned in https://github.com/opensearch-project/alerting/issues/1517#issuecomment-2096928443, test SecureMonitorRestApiIT.test create monitor with an user without index read role appears to be flaky.


Implement the free/busy component

Implement the free/busy component.
The Period class was never really exercised before, so this fixes some issues. The Period class is converted into a pydantic model and now takes a dict as input rather than a single string and also gets upgrades to support property parameters.
Issue #62

Codecov Report

Merging #77 (7c72738) into main (bdbc374) will decrease coverage by 0.61%.
The diff coverage is 85.98%.

:exclamation: Current head 7c72738 differs from pull request most recent head aee4afc. Consider uploading reports for the commit aee4afc to get more accurate results

@@            Coverage Diff             @@
##             main      #77      +/-   ##
==========================================
- Coverage   96.45%   95.83%   -0.62%
==========================================
  Files          14       15       +1
  Lines        1184     1274      +90
==========================================
+ Hits         1142     1221      +79
- Misses         42       53      +11

Impacted Files
Coverage Δ

ical/todo.py
98.33% <ø> (ø)

ical/freebusy.py
80.88% <80.88%> (ø)

ical/types.py
96.60% <94.59%> (+0.52%)
:arrow_up:

ical/calendar.py
100.00% <100.00%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Issues in documentation

Hello,
I'm sending you a several suggestions about the documentation, just to be clear and to prevent some possible misinterpretations:

PBKDF2 should be more specified

Please add which pseudorandom function is used in the scheme (e.g. HMAC-SHA256, SHA1, etc...)
Please specify what's your notation for that function. Now the implementer don't know what's the salt, password, etc...

Mysterious GET_BYTES function

...is completely unclear. I didn't get it. Is it a conversion from already pure ASCII to "just to be sure UTF-8" or some conversion from BASE32 to binary data? Or some kind of type abstraction, like "now we're casting a string into a blob of binary data"?
GET_BYTES(ACTIVATION_ID_SHORT + "-" + ACTIVATION_OTP, "UTF-8") ... i guess that in this case it is converting B32 into bytes. So then, the function will have to deal with separator, which is not a valid character for B32 encoding. That's creepy. I understand that in some activation step we'll show that to the user and thus the visual separator is very appreciated, but this fact should not affect a way, how the information is transmitted over the network or via the QR code..

Usage of AES

The block mode and padding scheme is not specified.
Instead of simple "AES" & "AES^inverse", you should use AES_encrypt / AES_decrypt, just to be more clear
The same as PBKDF2, it's unclear what's your notation (e.g. what's password, data, iv...)

Usage of ECDSA

Similar to AES. It's better to use ECDSA_sign / ECDSA_verify naming instead of ECDSA / ECDSA^inverse
There's SHA256withECDSA signature used, but that's mentioned in the code examples only. Should be also in the pseudo-algorithm part of the documentation

BASE64

BASE64Encode doesn't have "encoding" parameter, like UTF8. I think it's not necessary to highlight, that result is a string :)

Well, hope that's all. I think it will be enough to add some introduction notes, which will describe what's going on when, for example, PBKDF2 is used in the documentation. Something like "PBKDF2(salt, password, iterations) uses HMAC_SHA256 as pseudorandom function.", "AES_Encrypt(data, iv, key) means AES in OFB mode with PKCS7 padding", etc...

Thank you for detailed comments, Juraj!
I should probably:

Start the document with some basic function definition.
Use Java-like syntax for pseudo-code to make some things more clear

I guess that ("hello" + "world").getBytes("UTF-8") is more readable than GET_BYTES("hello" + "world", "UTF-8")
This also causes the issue with "Base64 encoding" syntax - does not make sense unless you know that I mean Base64.encode("hello".getBytes("UTF-8")) by this cryptic syntax.

The whole use base64 is a bit chaotic there - in code, it is apparent that base64 plays role only in networking (there is no base64 reference in the powerauth-java library). I should drop base64 from the cryptography description and re-introduce it with RESTful APIs.

For now, please have a look at used cryptography in "powerauth-java" project to see what is going on while I fix the documentation.

I have committed an improved version:

added definitions in the beginning
pseudo-code is more Java-like
I cleared Base64 from the crypto description

Please could you have another pass-through and send updated feedback, before we can close the issue?

There is one more hint: we should probably split the "monolith document" into multiple documents to improve information discoverability.

Document is broken down into multiple smaller chapters, hopefully this will do.

Thank you. That's what I call an improvement :+1: I'm closing this issue.


floorp: fix build by using python311

Description of changes
fix build by using python311
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Probably it could be reverted, when newer floorp would use FF 128+ codebase, which ok w/o this patch

I think #326101 also fixes this one.

Already fixed by https://github.com/NixOS/nixpkgs/pull/326101.


Documentation examples

Add a sphinx-gallery with examples of:

[ ] Reading trx files.
[ ] Writing trx files.
[ ] Reading/writing groups of streamlines.
[ ] Reading/writing data per vertex / per streamline.

it would be great! thanks for the reminder


API client not respecting start and end dates

There is a bug in how we handle filtering rows out of the CSV row response when start and end dates are set.

Notice the 2030 start and end dates and the response containing events from 2023.

good catch, thanks!


Use nikic/PHP-Parser for php parsing

Probably better to share some common code then to maintain our own.

or https://github.com/Roave/BetterReflection


⚠ GitLab Server has degraded performance

In 6331606, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 13807 ms

Resolved: GitLab Server performance has improved in f59af3b.


Feat/navigation

Context
Link to Notion ticket
Additional information
This PR implements the website navigation.
Animations for navigation links are not part of this PR and will be done separately.
The background color is transparent so it will match the general website styles.
The social icons component is replaced by a placeholder for now. Waiting for the real component
Side changes
Add a bundler to the eleventy config. Now we can use any npm packages in our scripts.
Testing (tester)
The reviewer should check on this branch:

The code

Check the desktop and mobile version of navigation. They should match the design except for the background color.
Scrolling should be locked in mobile view when navigation is open.

The site in CloudCannon

Before merge (PR owner)

[ ] Delete the test site in CloudCannon
[ ] For "generic" components only: remove - content_blocks from structures in the YAML file (this was added for testing).

Changed top level class to be header not head
Changed header to only take full vh when nav is open
Added comments in data file
Added option to change the font of a text logo
Changed variable name to be more applicable in context


GetIt - Paths to Svg and Source not automatically available to project

I was surprised that I had to explicitly include the paths to Svg and Source in my projects for the GetIt installation of SVGIconImageList.  That doesn't happen for f.x. EurekaLog and TMS - and we compile those sources there in our builds as well.
Ref. a comment in another issue about directories - I am slightly concerned that the source is split into even more directories, as it makes it cumbersome to maintain paths for projects.  What about using unit names for grouping instead?
F.x.
SVGIcon.Factory.Common;
SVGIcon.Factory.Cairo;
SVGIcon.Factory.D2D;

I like your suggestion. Normally that would be my choice as well, so only one path has to be included. But the cairo headers for example are copied from another project. We would have to rename them everytime we pull in a new version and change the references inside the files as well, which is probably not going to happen very often.
The same would probably also apply to the Pas SVG factory.
If carlo likes the suggestion, we can go this way, but then we have to agree on a naming convention first.
The files inside the cairo folder could all receive a "Cairo." prefix.

About GetIt, probably the other components adds the folder to global search path of Delphi.
I will ask Embarcadero to do it for these components too, it is not something I can do ...

Now, in the last version published to GetIt It's ok.


[FLOC-4094] Adding version checking in CommonOptions

The flocker version, when passed to CommonOptions (when passed as a command like argument to the acceptance test runner or the setup cluster script, --flocker-version ), did not validate it to start with, meaning that, it would deploy as many nodes as requested (instances), and later on, when trying to actually install flocker, would fail, and delete the nodes.
By validating the version within the options parser, we will fail sooner, and avoid creating cloud instances that we won't use at the end.

Thanks @mpgomez
I verified that the validation works:
$ ./admin/run-acceptance-tests --flocker-version=x.y.z --distribution=centos-7 --provider=rackspace
run-acceptance-tests: Flocker vesion 'x.y.z' is not a valid version

But the tests need some more work.
Please resubmit when you've checked the tests and got green builds.

Implemented review comments, and waiting for a green build.
Thanks Richard for the review!

Thanks @mpgomez
This looks much better. I made some suggestions. Answer or address those and then merge when green.


Added new T Data to IEvent and removed GetData to simplify the end usage and make it backward compatible

Yeah, you can't do this without breaking the co/contra variance on IEvent<T>. It's not as simple as just reverting back to the property unless we want to redo the functionality from https://github.com/JasperFx/marten/issues/1659

@oskardudycz I feel dumb now. Sorry for the friction this caused.


Move TempDataPropertyProvider into filter

Fixes #5979.

:up::date:. Should be more unified now.

:up::date: @pranavkm let me know what you think.


Add 1.62.0 support

How to add 1.62.0 support

Have made changes to the scripts and .jam files to try to get 1.61.0 built using NDK r13b. Got to the point where the libraries start to build but unfortunately thread failed due to not finding process.h.

I've pushed support for Boost 1.65.1 to the experimental branch clang-ndk-r16, you will need NDK r16-beta2 to build it.


GPL-327 - As a developer I want to research the use of GraphQL to replace JSON API

User story
As a developer I want to research the use of GraphQL to replace JSON API. Using Graph QL could be a much better fit for our applications. It is supposed to be a Good fit for complex systems and microservices, allows Fetching data with a single API call, allows tailoring requests to our needs and has validation and type checking out of the box.
Who the primary contacts are for this story
Stephen Inglis
Acceptance criteria
To be considered successful the solution must allow:
1 - a prototype which will demonstrate how it will be used
2 - documentation on how to implement it
3 - documentation on the advantages and disadvantages
Additional context
This is a story which will affect the UI and the service. We also need to investigate any libraries or tools whcih can be used for Rails and Vue.

Research for back end done. Conclusion is that using GraphQL is going to work for our needs.

TODO: Research for front end using GraphQL.


Can't get it to run on Windows 10

I click on the executable and a window opens for a brief instant and then closes. Have tried downloading previous versions, it's the same. I made sure to install UsbDk. I'm assuming this tool has a GUI as per the comment that the original software "is very ugly and inconvenient". Am I wrong? Should I be running it in CMD? Please advise.

It runs in command line. No GUI.
Open command window and type
ch57x-keyboard-tool.exe<enter>
That will show you help.


Setup Prometheus /metrics endpoint for kepler model server for weights

Setup a rough Prometheus /metrics endpoint for exporting weights. In the future, would like to consider using a lightweight sql database to improve runtime.

I changed the path back. This created a bug for os.path.join so I replaced that. I also added the desired fixes to the model and it works  better. I will likely reduce the epochs because I noticed that after a reduction in rmse, the rmse began to increase again.


GO:0004809 tRNA (guanine-N2-)-methyltransferase activity and GO:0016429  tRNA (adenine-N1-)-methyltransferase activity map to transferred ECs

GO:0004809 tRNA (guanine-N2-)-methyltransferase activity, maps to EC:<IP_ADDRESS>, transferred to

[ ] Transferred entry: <IP_ADDRESS> & RHEA:43124
[ ] Transferred entry: <IP_ADDRESS> & RHEA:43128
[ ] Transferred entry: <IP_ADDRESS> & RHEA:43124
[ ] Transferred entry: <IP_ADDRESS> & RHEA:43140

GO:0016429  tRNA (adenine-N1-)-methyltransferase activity: maps to EC:<IP_ADDRESS>, transferred to

[ ] Transferred entry: <IP_ADDRESS> & RHEA:43144
[ ] Transferred entry: <IP_ADDRESS> & RHEA:43148
[x] Transferred entry: <IP_ADDRESS> & RHEA:41740 >>>  tRNA (adenine(57)-N1-/adenine(58)-N1-)-methyltransferase activity
[ ] Transferred entry: <IP_ADDRESS> & RHEA:43152

Action needed:

[x] create the 8 new GO terms
[ ] obsolete GO:0004809 and GO:0016429

GO:0043827 tRNA (adenine(57)-N1-)/(adenine(58)-N1-)-methyltransferase activity EC xref has a typo.

[Term]
id: GO:0160101
name: tRNA (guanine(10)-N(2))-dimethyltransferase activity
namespace: molecular_function
def: "Catalysis of the reaction: guanosine(10) in tRNA + 2 S-adenosyl-L-methionine = 2 H(+) + N(2)-dimethylguanosine(10) in tRNA + 2 S-adenosyl-L-homocysteine." [EC:<IP_ADDRESS>]
synonym: "N(2),N(2)-dimethylguanosine tRNA methyltransferase" EXACT []
xref: EC:<IP_ADDRESS>
xref: RHEA:43124
is_a: GO:0008170 ! N-methyltransferase activity
is_a: GO:0016423 ! tRNA (guanosine) methyltransferase activity
property_value: term_tracker_item "https://github.com/geneontology/go-ontology/issues/26104" xsd:string
created_by: rynl
creation_date: 2023-09-14T00:15:09Z
[Term]
id: GO:0160102
name: tRNA (guanine(10)-N(2))-methyltransferase activity
namespace: molecular_function
def: "Catalysis of the reaction: guanosine(10) in tRNA + S-adenosyl-L-methionine = H(+) + N(2)-methylguanosine(10) in tRNA + S-adenosyl-L-homocysteine." [EC:<IP_ADDRESS>]
xref: EC:<IP_ADDRESS>
xref: RHEA:43128
is_a: GO:0008170 ! N-methyltransferase activity
is_a: GO:0016423 ! tRNA (guanosine) methyltransferase activity
property_value: term_tracker_item "https://github.com/geneontology/go-ontology/issues/26104" xsd:string
created_by: rynl
creation_date: 2023-09-14T00:20:53Z
[Term]
id: GO:0160103
name: tRNA (guanine(26)-N(2)/guanine(27)-N(2))-dimethyltransferase activity
namespace: molecular_function
def: "Catalysis of the reaction: guanosine(26)/guanosine(27) in tRNA + 4 S-adenosyl-L-methionine = 4 H(+) + N(2)-dimethylguanosine(26)/N(2)-dimethylguanosine(27) in tRNA + 4 S-adenosyl-L-homocysteine." [EC:<IP_ADDRESS>]
xref: EC:<IP_ADDRESS>
xref: RHEA:43136
is_a: GO:0008170 ! N-methyltransferase activity
is_a: GO:0016423 ! tRNA (guanosine) methyltransferase activity
property_value: term_tracker_item "https://github.com/geneontology/go-ontology/issues/26104" xsd:string
created_by: rynl
creation_date: 2023-09-14T00:24:30Z
[Term]
id: GO:0160104
name: tRNA (guanine(26)-N(2))-dimethyltransferase
namespace: molecular_function
def: "Catalysis of the reaction:guanosine(26) in tRNA + 2 S-adenosyl-L-methionine = 2 H(+) + N(2)-dimethylguanosine(26) in tRNA + 2 S-adenosyl-L-homocysteine." [EC:<IP_ADDRESS>]
synonym: "tRNA (m(2)2G26)dimethyltransferase" EXACT []
xref: EC:<IP_ADDRESS>
xref: RHEA:43140
is_a: GO:0008170 ! N-methyltransferase activity
is_a: GO:0016423 ! tRNA (guanosine) methyltransferase activity
property_value: term_tracker_item "https://github.com/geneontology/go-ontology/issues/26104" xsd:string
created_by: rynl
creation_date: 2023-09-14T00:28:33Z
[Term]
id: GO:0160105
name: tRNA (adenine(22)-N(1))-methyltransferase  activity
namespace: molecular_function
def: "Catalysis of the reaction: adenosine(22) in tRNA + S-adenosyl-L-methionine = H(+) + N(1)-methyladenosine(22) in tRNA + S-adenosyl-L-homocysteine." [EC:<IP_ADDRESS>]
xref: EC:<IP_ADDRESS>
xref: RHEA:43144
is_a: GO:0016426 ! tRNA (adenine) methyltransferase activity
property_value: term_tracker_item "https://github.com/geneontology/go-ontology/issues/26104" xsd:string
created_by: rynl
creation_date: 2023-09-14T00:42:44Z
[Term]
id: GO:0160106
name: tRNA (adenine(9)-N(1))-methyltransferase activity
namespace: molecular_function
def: "Catalysis of the reaction: adenosine(9) in tRNA + S-adenosyl-L-methionine = H(+) + N(1)-methyladenosine(9) in tRNA + S-adenosyl-L-homocysteine." [EC:<IP_ADDRESS>]
synonym: "tRNA m(1)A(9)-methyltransferase" EXACT []
synonym: "tRNA(m(1)G(9)/m(1)A(9))-methyltransferase" EXACT []
xref: EC:<IP_ADDRESS>
xref: RHEA:43148
is_a: GO:0016426 ! tRNA (adenine) methyltransferase activity
property_value: term_tracker_item "https://github.com/geneontology/go-ontology/issues/26104" xsd:string
created_by: rynl
creation_date: 2023-09-14T00:53:53Z
[Term]
id: GO:0160107
name: tRNA (adenine(58)-N(1))-methyltransferase activity
namespace: molecular_function
def: "Catalysis of the reaction: adenosine(58) in tRNA + S-adenosyl-L-methionine = H(+) + N(1)-methyladenosine(58) in tRNA + S-adenosyl-L-homocysteine." [EC:<IP_ADDRESS>]
synonym: "tRNA (m(1)A(58)) methyltransferase" EXACT []
synonym: "tRNA m(1)A(58) methyltransferase" EXACT []
xref: EC:<IP_ADDRESS>
xref: RHEA:43152
is_a: GO:0016426 ! tRNA (adenine) methyltransferase activity
property_value: term_tracker_item "https://github.com/geneontology/go-ontology/issues/26104" xsd:string
created_by: rynl
creation_date: 2023-09-14T00:57:53Z


feat(dashboard_rbac): single dashboard api

SUMMARY
enforce access for a single dashboard on api.py endpoint
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

Merging #12976 (c592a27) into master (137cf5d) will decrease coverage by 1.25%.
The diff coverage is 93.93%.

@@            Coverage Diff             @@
##           master   #12976      +/-   ##
==========================================
- Coverage   68.20%   66.94%   -1.26%
==========================================
  Files        1025      489     -536
  Lines       48765    28703   -20062
  Branches     5188        0    -5188
==========================================
- Hits        33259    19215   -14044
+ Misses      15366     9488    -5878
+ Partials      140        0     -140

Flag
Coverage Δ

cypress
?

javascript
?

python
66.94% <93.93%> (-0.68%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

superset/views/core.py
74.90% <85.71%> (-0.52%)
:arrow_down:

superset/utils/decorators.py
94.44% <86.95%> (-5.56%)
:arrow_down:

superset/dashboards/api.py
87.44% <100.00%> (+0.81%)
:arrow_up:

superset/dashboards/commands/exceptions.py
100.00% <100.00%> (ø)

superset/models/dashboard.py
75.62% <100.00%> (-0.13%)
:arrow_down:

superset/db_engines/hive.py
0.00% <0.00%> (-85.72%)
:arrow_down:

superset/databases/commands/create.py
32.65% <0.00%> (-59.19%)
:arrow_down:

superset/sql_validators/postgres.py
50.00% <0.00%> (-50.00%)
:arrow_down:

superset/views/database/mixins.py
59.64% <0.00%> (-22.81%)
:arrow_down:

superset/db_engine_specs/hive.py
73.84% <0.00%> (-16.93%)
:arrow_down:

... and 565 more

Continue to review full report at Codecov.

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

@dpgaspar could you review ?

LGTM but, I prefer not add/merge this. I'm assuming you need HTTP 403 for some reason, but your previous work was already secure. I can argue that a response with HTTP 404 is actually more secure then HTTP 403 on this case, if an attacker brute scans the API with id's he can infer what id's actually exist and may be subject to some other form of attack. By using HTTP 404, what you see is what you get, and no additional information is given besides what the user has access to.
Also code wise, the current new API on GET item, is based on this premise.
Thoughts?

Ok then
Sounds legit


⚠ Bofrost.be has degraded performance

In 86a5b01, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1375 ms

Resolved: Bofrost.be performance has improved in 58bbefd.


[Snyk] Fix for 10 vulnerabilities

Snyk has created this PR to fix one or more vulnerable packages in the `npm` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

package.json

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

586/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 5.3
Regular Expression Denial of Service (ReDoS) SNYK-JS-BROWSERSLIST-1090194
Yes
Proof of Concept

526/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 4.1
Arbitrary Code Injection SNYK-JS-EJS-1049328
Yes
Proof of Concept

586/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 5.3
Regular Expression Denial of Service (ReDoS) SNYK-JS-GLOBPARENT-1016905
Yes
Proof of Concept

601/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 5.6
Prototype Pollution SNYK-JS-IMMER-1540542
Yes
Proof of Concept

611/1000  Why? Recently disclosed, Has a fix available, CVSS 6.5
Information Exposure SNYK-JS-NODEFETCH-2342118
Yes
No Known Exploit

586/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 5.3
Open Redirect SNYK-JS-NODEFORGE-2330875
Yes
Proof of Concept

529/1000  Why? Has a fix available, CVSS 6.3
Prototype Pollution SNYK-JS-NODEFORGE-2331908
Yes
No Known Exploit

586/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 5.3
Regular Expression Denial of Service (ReDoS) SNYK-JS-PROMPTS-1729737
Yes
Proof of Concept

646/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 6.5
Prototype Pollution SNYK-JS-RFC6902-1053318
Yes
Proof of Concept

619/1000  Why? Has a fix available, CVSS 8.1
Remote Code Execution (RCE) SNYK-JS-SHELLQUOTE-1766506
Yes
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.

Commit messages

Package name: react-scripts
    The new version differs by 69 commits.

221e511 Publish
6a3315b Update CONTRIBUTING.md
5614c87 Add support for Tailwind (#11717)
657739f chore(test): make all tests install with `npm ci` (#11723)
20edab4 fix(webpackDevServer): disable overlay for warnings (#11413)
69321b0 Remove cached lockfile (#11706)
3afbbc0 Update all dependencies (#11624)
f5467d5 feat(eslint-config-react-app): support ESLint 8.x (#11375)
e8319da [WIP] Fix integration test teardown / cleanup and missing yarn installation (#11686)
c7627ce Update webpack and dev server (#11646)
f85b064 The default port used by `serve` has changed (#11619)
544befe Update package.json (#11597)
9d0369b Fix ESLint Babel preset resolution (#11547)
d7b23c8 test(create-react-app): assert for exit code (#10973)
1465357 Prepare 5.0.0 alpha release
3880ba6 Remove dependency pinning (#11474)
8b9fbee Update CODEOWNERS
cacf590 Bump template dependency version (#11415)
5cedfe4 Bump browserslist from 4.14.2 to 4.16.5 (#11476)
50ea5ad allow CORS on webpack-dev-server (#11325)
63bba07 Upgrade jest and related packages from 26.6.0 to 27.1.0 (#11338)
960b21e Bump immer from 8.0.4 to 9.0.6 (#11364)
134cd3c Resolve dependency issues in v5 alpha (#11294)
b45ae3c Update CONTRIBUTING.md

See the full diff

Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Closing this PR as it contains a breaking change. Will fix manually.


Doxbin is detecting SA 1.5.9

HI, doxbin.org is still detecting SA.

I didn’t get that message. Are you sure it’s not actually because of an extension you have installed? Doxbin just spun for me, but no error.

On Sep 8, 2021, at 6:06 PM, ctaity @.***> wrote:

HI, doxbin.org is still detecting SA.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

that screen is from SA BROWSER: SA_SHOW_BROWSER =true

Can you send me a session db? Maybe I can find something in it

On Sep 8, 2021, at 6:28 PM, ctaity @.***> wrote:

that screen is from SA BROWSER: SA_SHOW_BROWSER =true
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

What does this site do in regular chrome? I can’t get it to do anything in any browser I’ve tried besides bounce the logo

On Sep 8, 2021, at 7:28 PM, Blake Byrnes @.***> wrote:

Can you send me a session db? Maybe I can find something in it

On Sep 8, 2021, at 6:28 PM, ctaity @.***> wrote:

that screen is from SA BROWSER: SA_SHOW_BROWSER =true
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

on a regular browser, pass the checks and go to the site:

and CloudFlare is still detecting SA:
https://www.loom.com/share/b4e64a724d1e4ca098d46a824c4842a4

I need more to go on than just urls. Please either provide session databases (private is fine) or way more details. I’m not sure I follow how a 522 error on voat.co is related to doxbin.

On Sep 8, 2021, at 9:02 PM, ctaity @.***> wrote:

and CloudFlare is still detecting SA:
https://www.loom.com/share/b4e64a724d1e4ca098d46a824c4842a4
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

no voat.co was an error, please excuse me. Tomorrow i will send the session database. Thanks for all

@blakebyrnes  session db
Uploading Archive.zip…

Yep, thanks! I poked through, but I don't see anything obvious. It looks like their bot blocker is seeing something it doesn't like, but it's not showing up in the logs in any way I can see. Do you know if any other tools are making it through on this site?

No, I am tried with selenium, puppeteer with stealth plugin, and normal requests but not lucky, and with SecreatAgent :(

It's because they have no idea how to build a proper flask application

Tell kt i said i hope they like prison


Add MacroExpansionNoteMessage

Per https://forums.swift.org/t/why-are-swiftsyntaxs-diagnosticmessage-notemessage-etc-protocols/68205/4

Thanks for the PR @stephencelis. Looking at the PR, I realized that the message types should be defined in the SwiftSyntaxMacros instead of SwiftSyntaxMacroExpansion. I’m moving them in https://github.com/apple/swift-syntax/pull/2338 and would like to introduce MacroExpansionNoteMessage into the SwiftSyntaxMacros module once my PR is merged.

I just merged https://github.com/apple/swift-syntax/pull/2338. Could you update this PR to place MacroExpansionNoteMessage in the SwiftSyntaxMacros module?

@ahoppen Done! Not sure why it says I requested Doug's review...some automatic thing?

Not sure why it says I requested Doug's review...some automatic thing?

Yeah, marked as codeowner of SwiftSyntaxMacros (https://github.com/apple/swift-syntax/blob/main/.github/CODEOWNERS)

@swift-ci please test

@swift-ci please test Windows platform

@swift-ci Please test


Make bpf() wrapper ABI compatible with Linux

Describe the feature you'd like supported
This is an alternative proposal to https://github.com/microsoft/ebpf-for-windows/issues/3700. Based on feedback by my colleague Timo I had a look at invoking the bpf() syscall wrapper via LoadLibrary (this was mentioned by @shankarseal during the last triage). It turns out this works, and would make it easier to port cilium/ebpf.
The problem is that at efW is currently API compatible with bpf(), but does not follow the Linux ABI. This means we can't use our existing syscall wrappers to call into ebpfabi.dll. As I mentioned earlier we can't make use of API compatibility since that forces use of CGO.
Proposed solution
Guarantee that the exposed bpf() function follows the Linux ABI. Figure out a way to reuse bpf_attr, etc. from upstream as much as possible.
Additional context
Here is a non-exhaustive list of things that would need to be fixed:

Accept both larger and smaller bpf_attr from the caller of bpf()
Fix values of enum bpf_cmd_id to correspond to Linux equivalents
Make union bpf_attr actually match Linux ABI (flags on linux are 32 bit, etc.)

We'd also need some new API (these don't have to go via bpf()):

A way to load a native image and enumerate maps and programs without being tied into bpf_object lifecyle.
Add ebpf_close_fd again so that we have an easy way to call into the right UCRT _close function.

Linux headers can be used as long as it has BSD clause or syscall clause.

@Alan-Jowett would it be possible to use BTF from Linux to generate/test/create ABI compat? You should have enough information there to understand structure layout for example.

Also discussed during triage: we can run ebpf-go unit tests during efW CI as integration tests.

Adding myself as an assignee to track the BTF related suggestion. I agree this makes sense and we should do something like this.
Ideally we would be able to just grab the BTF data from the eBPF for Windows build and from the Linux kernel symbols and then diff the BTF types.
Unfortunately, this won't work as MSVC doesn't emit BTF data.
What I think we can do is:

Write a trivial C file that pulls in and references the various structs being passed the BPF syscall.
Compile this C file as BPF
Extract the BTF data for the types passed to this syscall

It should then be possible to validate alignment etc.

Another problem is that Windows errno.h defines constants for errno which have the same names as on Linux but do not always have the same value. See https://learn.microsoft.com/en-us/cpp/c-runtime-library/errno-constants?view=msvc-170
For example, ENOBUFS is 119 on Windows, while on Linux it is 105.

BPF_OBJ_NAME_LEN is 64 but Linux expects 16.

struct bpf_map_info and struct bpf_prog_info don't follow Linux ABI by having fields in the wrong order, and by defining Windows specific fields.

The original goal was to be source compatible rather than binary compatible with Linux, so that is not surprising.


Add more detail for usage

Add more detail to the usage section. Include sample Maven commands to invoke the plugin.
eg. mvn deploy com.avojak.mojo:aws-p2-maven-plugin:1.0:deploy -Daws-p2.bucket=p2.example.com to explicitly invoke or mvn deploy to automatically use configuration in pom.xml.

Also need to include a note that if the bucket has versioning enabled, not all versions are deleted during an overwrite. When a new site is uploaded, only current content is deleted, and delete markers are not touched.


Fix systematics logic

This implements my approach to the logic for drawing the systematics band, which was the following for each bin:

largest_val = max(bin contents for all systematics)
smallest_val = min(bin contents for all systematics)
Systematics central (not plotted):  (largest_val + smallest_val)/2.0
Systematics "uncertainty":   largest_val - smallest_val)/2.0

In other words, the pink band would just go from smallest_val to largest_val.  Note that the min and max run over "up" systematics, "down" systematics, and also the central value.  This is because shifting a systematic (e.g. the JEC) up/down may increase the entries in some bins and decrease it in others, and indeed it's actually possible that both up and down will move the contents in the same direction, so that the central value is largest_val or smallest_val.
The previous version had (min+max)/2 and abs(max-min)/2, but min and max were taken from the deviations of the systematics from the central value.  Thus the pink band range covered something proportional to the systematic uncertainty, but didn't run from below the central value to above it, as we saw on the plots on Friday.
Finally, I added an argument called actuallyDrawSystErrorBand to make it clearer when reading the code that this function is replacing the statistical uncertainties with systematic ones (and allow shifts between them).  In the future we should have a stat method, a syst method, a combined method, and options/choices for which to draw.
Let me know if this all makes sense!

Thanks a lot Seth for this usefull add.
Yes indeed this makes sens. I had in mid to add such modules for taking care of the erros. This will be added in the next update, on top of your changes.


Station > guard_mode Value is not accepted

When changing the value manually, this is not confirmed by the iobroker. The status of the station remains unchanged.

@Catwatch Try version 0.6.1 released a few seconds ago.

@Catwatch Try version 0.6.1 released a few seconds ago.

Works fine- THX!


Cross-domain issue with 'window.top.forceNativeAudio'

I've just run into a breaking error with v1.0.2 and I think it's related to the new force native audio code. I get the following error when window.top.forceNativeAudio is checked:

Uncaught DOMException: Blocked a frame with origin "http://springroll-tc.pbskids.org" from accessing a cross-origin frame.

I think it's breaking on this line, but this source shows it on "window.top.springroll", not directly on "window.top" like I'm seeing in my console: https://github.com/SpringRoll/SpringRoll/blob/master/dist/modules/sound.js#L647

@probityrules This is a known issue, I have a patch for this coming shortly.

@probityrules PR #143 Fixes this. Release is pending. You may point bower at master if need be.

This is a SpringRoll 1 issue, closing


Analysis Module: First Time Series Analysis Implemented

User story

As a Software developer
I want to analyse the data
So that I can retrieve valuable information of it.

Acceptance criteria

Criterion 1: The data structure for time series based analysis is defined.
Criterion 2: First tests have been made in regards of which valuble information could be retrieved with Time series analysis.
Criterion 3: Example scenarios for outlier we can detect are defined.
Criterion 4: Possible pitfalls and blockers to implement the example scenario are documented. (Could be in this ticket)

Note: This ticket depends of the outcome of #35

Preliminary:

Darts was chosen as a library for doing the actual analysis
it is compatible with our current build structure and poetry setup
on first glance it seems straightforward to use and a good fit for our use case

Pitfalls:

data requires extensive preprocessing, e.g.:

filtering out non updates, null valued entries etc
filtering the results into different backup types
backup table does not have a variable that holds the frequency of backups. this can be set manually or inferred using Darts, but the accuracy is not guaranteed

time series analysis always requires a time axis with regular frequencies, variations will lead to less precise results
requires iterative testing and adjustment to get useful results. as mappings are not always clear (e.g. times in performance table, times of backups in results table)

Possibilities: (more detailed in work folder files)

matching the performance table to backups that were run at specific times, then finding outliers of disk performance per size
finding backups that take longer in relation to their size


Passing of kwargs to inference network sample method

First, thanks so much for your work on Bayesflow - it's great.
On the dev branch, it's currently not possible to pass kwargs to the inference network sample method via the approximator. This would be useful, since some inference networks have important parameters set here (e.g., the FlowMatching network can have n_steps passed to sample which can be very useful). Any kwargs passed when approximator.sample() is called are currently passed only to the adapter.
This could be added by allowing separate kwargs for the adapter and inference network in approximator.sample() and passing these to the relevant methods, e.g.:
    def sample(
        self,
        *,
        num_samples: int,
        conditions: dict[str, np.ndarray],
        adapter_kwargs: dict[str, Any] = None,
        sample_kwargs: dict[str, Any] = None,
    ) -> dict[str, np.ndarray]:

Or it may be better to instead restrict the use of additional arguments in the _forward / _inverse methods of the inference network and instead require these to be set when the class is instantiated, so that there's no need to pass kwargs when sampling?
Thanks!

Thank you for bringing this up. We have had a PR in progress that addresses this issue (#248), but it went under the radar. Can you check it out and see if this would fix your problems?

Thanks for bringing this up again and testing the changes, they are now merged into dev.


Rename SyncRepository to SyncAccountRepository

Task/Issue URL: https://app.asana.com/0/488551667048375/1204280905195084/f
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

Current dependencies on/for this PR:

develop

PR #3328   👈

This comment was auto-generated by Graphite.


Added Focus

I see that you have changed the default behavior of greedy_probs.py: output_attentions=False => output_attentions=True, maybe we should avoid changing the defaults.

Please add this method to the table "Overview of methods" in the readme file.

@alfekka Lets try to finalize this method already =)


fix 105 by adding a condition to check for null and unit test

fix for #105
This field is optional (Cardinality 0..1 in FHIR 4.0 reference and according to the HL7 reference as well), thus it should convert into a JSON file.```

Done


Templates and configuration for release 5.0.0

Basically just adding the templates and adding configuration parameters as they existed in 4.8.1.
Some things you might want to check with regard to if they're done to your liking:

graph parameter are given in attributes/graph.rb
had to add template templates/default/cassandra-jvm.options.erb, which was introduced in 5.0.0
added configuration options listen_interface and rpc_interface, which will be used instead of listen_address and rpc_address respectively.

Let me know if I can help with making this more consumable for you in any way.

looks good on the surface, I am pulling it down now to check it out. Also not sure why the travis job failed but I will look at that too.

fixed the travis job; the rest of this looks good to me.


Add support for other valid extensions

Currently it seems the upload (to XML Editor at least) only accepts files with an *.xml extension.
According to §1.6.9 of the SPDX Specification 2.1 it

[s]hould be easy to recognize in a file system without opening the file.  A suggested naming convention is to use *.spdx (for tag-value format) and *-spdx.rdf for RDF format.

So while *.spdx for tag-value and *-spdx.rdf are not a required naming, they areconventions that the SPDX Online Tools should at least support.

@silverhook The XML editor is designed to support the license list XML (https://github.com/spdx/license-list-XML), not the SPDX documents.  To make this clear, we should rename "XML Editor" to "License XML Editor"

@goneall, ah, that explains a lot :) My bad for missing that detail. I think the renaming would be a good solution to the actual underlying problem.
I will rephrase this issue accordingly.


2.8

@louissegal thanks for the PR!
Do you think it would be viable to add a CI workflow to check the healthchecks repo for a new version every day and auto update the three files?

Probably, but it’s one of those things though where the time spent building a CI flow would take longer than manually updating the version tags - not to mention that in some cases with each new build it requires new dependencies. I will have a think about it 😎

On 11 Apr 2023, at 11:22, Alexander Trost @.***> wrote:
—

@louissegal Well.. it would be a cool thing to build though ;-) Let's stick with the manual way for now till it might make sense to automate it using a new/ improved CI workflow.


19 scene editor

modeling map grid, rotation of the selected image, modify the size of the map grid

changed map Grid from 2d tuple list to 2d tuple array and fixed stylistic


QRCode namespace doesn't exist in .NET 8

Type of issue
[X ] Bug
[ ] Question (e.g. about handling/usage)
[ ] Request for new feature/improvement
Expected Behavior
Should be able to create new QRCode Object by passing in QRCodeData.
Current Behavior
QRCode namespace doesn't exist.
Steps to Reproduce (for bugs)
Upgrade to .NET Core 8
Your Environment
Visual Studio 2022 latest version with .NET Core 8.
Code sample that works in .NET Core 5
 QRCodeGenerator qrGenerator = new QRCodeGenerator();
 QRCodeData qrCodeData = qrGenerator.CreateQrCode(PublicProfileURL, QRCodeGenerator.ECCLevel.Q);
 QRCode qrCode = new QRCode(qrCodeData);
 Bitmap qrCodeImage = qrCode.GetGraphic(20);

Note, the above code does not work in either .NET Core 6 OR 7.

Hi @ScottDurkin ,
I am having the same issue since a long time, for me as a workaround it worked downgrading the package to 1.4.1

Yeah, for me too. I believe the problem was created with the 1.4.3 changes.
They'll need to add net7 and net8 as target frameworks as well.
@codebude, would you like me to create a PR to address this issue?

Hi @ScottDurkin ,
I am having the same issue since a long time, for me as a workaround it worked downgrading the package to 1.4.1

Thank you for your response! I'll take a look!

@cezar-pimentel Thanks for your offer. I would love to see such PR. But currently I'm going through all the older PRs and see if I can merge them. I fear if you start your PR now, the master will change a couple of times, while you're working on the PR. Maybe it's better to wait until the majority of the older PRs is merged.

@cezar-pimentel Thanks for your offer. I would love to see such PR. But currently I'm going through all the older PRs and see if I can merge them. I fear if you start your PR now, the master will change a couple of times, while you're working on the PR. Maybe it's better to wait until the majority of the older PRs is merged.

No worries, we can wait. There's no need to rush.
Thank you my friend! 👊💪

Not sure if this is related to the same problem but the latest version (1.4.3) when running:
var qrCodeImage = qrCode.GetGraphic(20, bArray, wArray, image);
raises:
Argument type 'System.Drawing.Image' is not assignable to parameter type 'bool'
From quick read of the doc it should accept bitmap? This is on .NET 8
I tried downgrading but that didn't work either.

@cezar-pimentel @ScottDurkin FYI, your issues are related to the fact that portions of QRCoder have been removed that rely on System.Drawing.Common when targeting .NET 6+ on Linux, as System.Drawing.Common is not supported in that scenario (and indeed would just throw NotSupportedException).
You have a couple options for 1.4.3:

Target net6.0-windows or newer (not just net6.0)
Use one of the other classes such as PngByteQRCode that does not rely on System.Drawing.Common

In addition, there are a few PRs that already exist that attempt to address to this problem:

#407
#460
#462

Another solution would be to take a dependency on System.Drawing.Common for all platforms (which would include net8.0), and simply mark relevant methods with [SupportedOSPlatform].  This would reduce the confusion for developers on .NET 6+ who are targeting Windows; they would not have to target net8.0-windows specifically.  As a side effect, developers for Linux would have an extra dependency that is unnecessary.
cc: @codebude

@cezar-pimentel @ScottDurkin FYI, your issues are related to the fact that portions of QRCoder have been removed that rely on System.Drawing.Common when targeting .NET 6+ on Linux, as System.Drawing.Common is not supported in that scenario (and indeed would just throw NotSupportedException).
You have a couple options for 1.4.3:

Target net6.0-windows or newer (not just net6.0)
Use one of the other classes such as PngByteQRCode that does not rely on System.Drawing.Common

In addition, there are a few PRs that already exist that attempt to address to this problem:

Adds Support for ImageSharp and SkiaSharp using two independent projects for maximum compatibility #407
Add ImageSharp renderers for .net 6.0, 5.0 and standart 2.0 #460
Use SkiaSharp to support SvgQRCode in .NET6 #462

Another solution would be to take a dependency on System.Drawing.Common for all platforms (which would include net8.0), and simply mark relevant methods with [SupportedOSPlatform]. This would reduce the confusion for developers on .NET 6+ who are targeting Windows; they would not have to target net8.0-windows specifically. As a side effect, developers for Linux would have an extra dependency that is unnecessary.
cc: @codebude

I like the Supported Platform OS , but I would recommend the method signature stays consistent between them.  Right now the number of parameters for each renderer is not the same, and a feature in one (like an overlay Icon), is not in the other.

Having the method signature identical for each renderer may not be feasible, as the parameter types will be different per renderer.  For example, a renderer based on System.Drawing.Common will use the Bitmap class while one based on SkiaSharp would use SKBitmap.  The SVG renderer returns a string and the PDF renderer returns a byte[].  Some renderers allow different encodings (e.g. PNG or JPEG), and some could allow multiple .NET return types (such as MemoryStream or byte[]).  Besides that, the renderers will not likely all support the same feature set.  While you could argue that the overlay icon for ASCII could be implemented as a string which overwrites the center of the QR code, the chance that someone really wants this functionality in an ASCII output is minimal.

Has the problem been solved?

I'm trying to use Base64QRCode in .NET 8 and having the same issue, how could that be resolved?

I like the Supported Platform OS , but I would recommend the method signature stays consistent between them. Right now the number of parameters for each renderer is not the same, and a feature in one (like an overlay Icon), is not in the other.

@codeputer never say never, but I'm pretty sure it can't be implemented that way. The whole philosophy behind the modular renderers is to get the best out of each target format/renderer. This is where the different method signatures come from. If the GetGraphic method of every renderer had the same signature, we would have to remove many great features. Example: Many renderers can be passed an image that is drawn as a logo on the QR code. However, the purely text-based ASCIIQrCode simply cannot draw images and will therefore never accept an image in the GetGraphic method. If we were to harmonize the method signatures, we would have to delete the logo functionality in all other renders. I don't think that makes sense.

Has the problem been solved?

@Seanxwy Yes and no. The comment by Shane32 explains very well why the QRCode renderer is not available in .NET8. (TL;DR; It is based on System.Drawing.Common, which is no longer supported by Microsoft). However, as Shane explained, you can use other renderers or target Windows only (net8.0-windows) as the target platform. In the medium term, there will be a QRCoder 2.0 version, which will then include other renderers that (hopefully) emulate the functionality of the QRCode renderer under net7.0+. But then with an alternative graphics library (such as e.g. SkiaSharp). To be honest, QRCoder 2.0 is certainly not something that will see the light of day in the next few weeks. Unfortunately, I simply don't have enough free time to invest in the QRCoder. :-(
Please also check the compatibility matrix in our wiki, to see which renderer works with which target platform: https://github.com/codebude/QRCoder/wiki/Advanced-usage---QR-Code-renderers#2-overview-of-the-different-renderers

I'm trying to use Base64QRCode in .NET 8 and having the same issue, how could that be resolved?

@nicetomytyuk this will be resolved with PR #495 which will be merged this week. The code changes will be part of QRCoder 1.5.0 that I plan to release soon. (Next couple of days.)

Any plan to make QRCoder supported in .net8?


[ISSU-164]CSI support for OpenShift4.12

Purpose
Issue dell/csi-baremetal#164
Describe your changes
PR checklist

[ ] Add link to the issue
[ ] Choose Project
[ ] Choose PR label
[ ] New unit tests added
[ ] Modified code has meaningful comments
[ ] All TODOs are linked with the issues
[ ] All comments are resolved

Testing
Testing details

Passed https://asd-ecs-jenkins.isus.emc.com/view/Atlantic/job/csi-custom-ci/1494/

Passed https://asd-ecs-jenkins.isus.emc.com/view/Atlantic/job/csi-custom-ci/1494/

https://asd-ecs-jenkins.isus.emc.com/job/csi-operator-custom-build/147/
https://asd-ecs-jenkins.isus.emc.com/view/Atlantic/job/csi-custom-ci/1495/
https://asd-ecs-jenkins.isus.emc.com/job/csi-custom-acceptance/589/


Make changes to the report according to the policy situation

1.Make changes to the report according to the policy situation

Codecov Report

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

@@           Coverage Diff           @@
##             main     #132   +/-   ##
=======================================
  Coverage        ?   43.70%
=======================================
  Files           ?       10
  Lines           ?     1167
  Branches        ?        0
=======================================
  Hits            ?      510
  Misses          ?      624
  Partials        ?       33

Flag
Coverage Δ

unittests
43.70% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

LGTM


Fix #16899: Iterate over all results collected from response

Describe your changes:
Fixes #16899
Minor update to the logic when iterating over the ML model results.

Type of change:

[x] Bug fix
[ ] Improvement
[ ] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation

Checklist:

[x] I have read the CONTRIBUTING document.
[x] My PR title is Fixes <issue-number>: <short explanation>
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

thank you for the fix!


Remove Typescript and add cache for developer

:tada: This PR is included in version 2.1.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Better time display for timeouts

Fixes #598

See my comment on the issue https://github.com/fourtf/chatterino2/issues/598#issuecomment-404117865

Changed it so:
60 seconds -> 1 minute
600 seconds -> 10 minutes
601 seconds -> 10.02 minutes


Fix small issue in README example

While I was checking moto library I found a small typo in the first example. This PR fix it.

Coverage remained the same at 92.24% when pulling 53bbd829ccf138e9fd5e65df8e2ae6bdc3302166 on gardenunez:readme-example into e8c65d3d85ba62e857c7448691568ad4b02f4a46 on spulec:master.

I don't think this is a typo. We are making the string a byte-string. Let me know if I'm reading it wrong though.

The .decode("utf-8") would return a normal string unless im reading it wrong. Think we're good on this one


[1.2.x] Minor test script changes, and make ArtifactStore Locations sticky

Main change here is to make ArtifactStore keep the associated Location object as a transient metadata attached to the store to prevent recreating it, and save on object instances in the CacheProvider Transfer map.

retest this please

retest this please

retest this please


[ADDED] #155 #161 Added csv exports for the comments stats.

For https://github.com/hossain-khan/github-stats/issues/155 https://github.com/hossain-khan/github-stats/issues/161


Change acceptance test tag so old instances will be reaped

@sersut @adamedx this look right?

:+1:


Configure webpack5 as build process instead of vue-cli

Description
Because of problems with vue-cli's implicit webpack configuration, which could not be used with cypress' preprocessing, we had to develop our own webpack configuration to build and process our project.
The webpack configuration is similar to vue-cli's.
Notice:

First build and first lint can take some time, as all caches are cold

WIP:

cypress preprocessor does not yet work with webpack 5. As soon as that works, we can reconfigure cypress and, if necessary, fix any test issues.
https://github.com/cypress-io/cypress/issues/8900
I had to branch off of toasts to test if the issues are resolved, so the # of files changes is higher than expected. This will resolve as soon as toasts is merged

Type of change (remove all that don't apply)

Refactoring

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Test Configuration:

OS: Windows

Browser: Chrome

Frontend: (remove all that don't apply)

[X] Development build
[X] Dockerized production build

Checklist: (remove all that don't apply)

[X] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[X] My changes generate no new linting warnings or console warnings
[ ] I have updated the CHANGELOG.md to include any changes made in this PR (add WIP to the top, if there is none already)

Closed, because this PR is highly dependend on the solution for the linked external issue. We will not adapt the build-process in the scope of the project group. The issue #696 will remain.

Closed, because this PR is highly dependend on the solution for the linked external issue. We will not adapt the build-process in the scope of the project group. The issue #696 will remain.


Change maintainer from @brian-brazil to @csmarchbanks

Yesterday at the dev-summit, the Prometheus team has proposed @csmarchbanks as the new maintainer for this repo after @brian-brazil's retirement from the project.
This change will only be merged after an announcement on the prometheus-developers mailing list and is subject to lazy consensus.
Thank you, @brian-brazil, for the incredible amount of work done in this repository and for the project in general.

@csmarchbanks please verify that I have used your preferred email address for this purpose.

@csmarchbanks please verify that I have used your preferred email address for this purpose.


Turkish translation needed

Turkish translation ca be added as new branch

I decided not to include Turkish and work on other project


Add task to run bash scripts on App Service slots

Question, Bug, or Feature?
Type: Feature
Enter Task Name: There is no "App Service Execute Bash Task". It would be great if there was one.
Environment

Server - Azure Pipelines or TFS on-premises? Azure Devops

Issue Description
We would like to run some Bash scripts in a running App Service slot. There seems to be no task for that. This would be the equivalent of ssh -c.
The only alternative right now seems to be to (mis)use the Azure App Service deploy task, aloing with its "Postdeploy Script" setting. However, that doesn't seem right, and also it is broken for Linux Web App's with WEBSITE_RUN_FROM_PACKAGE=1 setting as outlined in issue #11440

See https://github.com/projectkudu/kudu/issues/3041#issuecomment-559796949 for a workaround using expect.


correct passing of parameter so args show up as args instead of input

incorrect position of a parameter which resulted in the args passed as input instead of as args
fix https://github.com/PowerShell/PowerShell/issues/4923

CI failures waiting on https://github.com/PowerShell/PowerShell/pull/4958


[Discussion] Azure Functions Runtime release 2.0.13033

Discussion issue for the Azure Functions runtime release 2.0.13033: https://github.com/Azure/azure-functions-host/releases/tag/v2.0.13033

Closing as the build number is updated and tracked here: https://github.com/Azure/app-service-announcements/issues/240


[Self-Help] Feature self-help 2023-09-01-preview

This checklist is used to make sure that common guidelines for a pull request are followed.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Self-Help

Hi @arthurwang2021, please use fake values in examples instead of placeholders

Hi @arthurwang2021, please use fake values in examples instead of placeholders

Updated all examples to use fake values here in this repo and aaz repo. @kairu-ms


Android error

Hi, I installed the newest version (4.0.2) using npm and then react-native link and when I launch the application and try to send a view, it crashes with this error:
undefined is not an object (evaluating '_NativeBridges.GoogleAnalyticsBridge.trackEven')
...\src\GoogleAnalyticsTracker.js:66

The GoogleAnalyticsBridge is undefined when I run the app.
This is the analytics code:
import { GoogleAnalyticsTracker } from 'react-native-google-analytics-bridge';
let tracker = new GoogleAnalyticsTracker('UA-XXXXXXXX-X');

tracker.trackEvent('ReactNative', 'test');

Sorry, react-native messed up and failed to add the Google Analytics Bridge to my Application setup. I added it manually and it is working now, thanks :-)


Reduce confusion about hashicorp/go-getter format

Resolves https://github.com/kubeflow/pipelines/issues/3074.
This is a common confusion.
/assign @joeliedtke

This change is

Not sure why @Bobgy 's assign command didn't work.
/assign @joeliedtke

/lgtm

This change can be previewed at:
https://deploy-preview-1724--competent-brattain-de2d6d.netlify.com/docs/pipelines/installation/standalone-deployment/

@joeliedtke @renmin Can you review and approve?

/lgtm
/approve

Thanks @joeliedtke