[internals] fix our base MessageBusError, which should inherit from Exception instead of BaseException

Also bumps the package version number

Pull Request Test Coverage Report for Build 47

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 95.775%

Totals

Change from base Build 44:
0.0%

Covered Lines:
136

Relevant Lines:
142

💛  - Coveralls


SAMZA-2343: Remove original split deployment code

Going to add new code for new split deployment strategy soon, so removing the original unused split deployment code.

@sborya can you please take a look?

Looks good to me, thanks for the cleanup. Let's get a review from @sborya as well since he has more context.

He is busy with oncall this week. If you think it is important to wait for a review from him, then I can wait.

@cameronlee314 If Boris is OK with the overall change (removing the code + docs) then that's fine, we can commit.

Thanks for confirming @srinipunuru.
Boris also confirmed to me that he had no high-level concerns about merging this.


MAINT: Adjust the codebase to the new np.array's copy keyword meaning

Hi @rgommers,
Here's a PR that addresses changes planned for NumPy in https://github.com/numpy/numpy/pull/25168 (new copy keyword for np.asarray and np.array).
I applied changes in a backward-compatible way, so that no NumPy version check & branching is needed.

The mypy failure looks related.

@lucascolley It looks that for MultivariateNormalQMC constructor mypy wasn't happy about np.atleast_1d returning a NumPy array or a list of NumPy arrays. For that method I reversed the order of np.atleast_1d and np.array calls (so the last call is np.array and (surely for mypy) NumPy array is assigned to mean/cov).

I assume you grepped the code base and had a look at all copy=False usages and judged which ones were relevant?

@rgommers Yes - I checked all copy=False occurrences and updated these that were passed to np.array. I searched also for np.array(..., copy=...).

@rgommers Thanks! Once https://github.com/scikit-learn/scikit-learn/pull/28323 and https://github.com/pandas-dev/pandas/pull/57172 are merged then we shouldn't break any nightlies when merging copy keyword PR.

Nice work - and good sign that all the diffs are short.


add and configure information

input company and employees and upload clients into system

php composer-setup.php --version=1.0.0-alpha8
--preview, --snapshot, --1 (EOL), --2, --2.2 (LTS)

php composer.phar install

{composer json in root bottom line
"name": "your-vendor-name/package-name",
"description": "A short description of what your package does",
"require": {
"php": ">=8.2",
"another-vendor/package": "1.*"
}
}


Add supported providers annotation

@samtstern While doing some debugging, I made a typo and passed in a hardcoded string to the IdpBuilder#setProvider method that ended in a runtime exception. 😢 I think we can be fancier with our supported providers so this PR add our own SupportedProvider annotation. 😎:smile:
Here's an example error:

PS: sorry for the yucky commit history, I built it off another PR.

@samtstern do you want to take a look at this one before 2.0? I think it's a nice improvement to help devs know what they should be passing in for the provider.

@SUPERCILEX yep I'll add this one to 2.0 milestone

@samtstern cool!

Forgot about this one, gonna merge it!

Awesome possum! Gonna go merge stuff now 😃


Documentation: basic C++ howto and data structures description

Is there any user level documentation out there on the underlying data structure and supported operations? Or the C++ codebase is the only "doc" at the moment?  Particularly, what data types are supported and how they are stored, and what kind of random access read/write operations are supported (extracting columns, appending columns, extract or write an indexed range of columns etc).
Would be nice have a basic "hello world" example of how to operate on metadata and read and write all supported data types from C++.  And maybe a high level meta discussion of what problems the format solves and how it concretely relates to "arrow" project would be very much appreciated.
Thank you.

After spending a couple of hours looking into the code base I am partially answering my own question.
There is a basic description of the storage types here. That is helpful, but there is no provision there for how lists and string vectors should be stored. Both are aparently stored as ad hoc var length arrays.
The core C++ is a low level interface and no C++ front end is available. So If I want to use feather in my C++ app I would need to re-implement a good part of the logic from R's front end.
As to the modification and access types there is not much to document. AFAICS the core api doesn't allow for in-file modification. One can read arbitrary column in but there is no provision for reading subranges. One can append columns to an open stream, but there is no way to re-write an existent column in a file.

Both are aparently stored as ad hoc var length arrays.

There are a number of different ways to store variable-length arrays, the one we are using is definitely not ad hoc:

See https://github.com/wesm/feather/blob/master/doc/FORMAT.md#variable-length-arrays
and https://github.com/apache/arrow/blob/master/format/Layout.md#list-type

I would welcome some improvements like you are describing to the core libfeather library -- I only did enough C++ engineering to support the initial Python and R use cases (write a table, read a table).

The C++ code has moved to the Apache Arrow codebase https://github.com/apache/arrow/tree/master/cpp/src/arrow/ipc, so we can address more work on the C++ code there


Export class to library but hide from documentation

Is it possible to use both hide and export directions such that my cell is exported to the libraries but is not shown in the documentation?
My particular case: I am creating a class through multiple cells, e.g.:
Cell 1:
#| export
class foo():
    def __init__():
        ''' Description of the class, which is shown in the docs.'''

Cell 2 (I want this one to be exported but hidden in the doc):
#| hide (?)
#| export (?)

class foo(foo):
    def func():
        ''' Description of the function, which I want to later show in the doc with show_doc(foo.func). '''

Combining both directives currently does not work, and in the library that cell shows again the class definition. I want to hide it and use later show_doc to show only the foo.func function I created. Is this possible? If not, is there a preferable way of defining classes over cells in nbdev?

Would exporti work for you?


How to run the chatbot using flask?

i am not able to run the application using the flask ? can you provide a steps to run it ?

Hey @G-Slient, I am closing this issue as there has been 25 days of inactivity. If you are still unable to get it working with click==6.7 please let me know.
Regards

hey @AbrahamSanders  I tried it using click==6.7,  I'm still getting same error.

X:\New folder\Github repos\seq2seq-chatbot\seq2seq-chatbot\models>flask serve_chat "models\cornell_movie_dialog\trained_model_v2\best_weights_training.ckpt" -p 8080
Usage: flask [OPTIONS] COMMAND [ARGS]...
Error: No such command "serve_chat".


deployment target 11.0 does not work

Environment
CPU M1 Pro
macOs 14.3
Flutter 3.19.1 • channel stable • https://github.com/flutter/flutter.git
Framework • revision abb292a07e (7 days ago) • 2024-02-20 14:35:05 -0800
Engine • revision 04817c99c9
Tools • Dart 3.3.0 • DevTools 2.31.1
Issue
As described in getting start section, I changed the minimum deployment target to 11.0. But Is still got "increase your application's deployment target to at least 10.14.6" error. By the way, I can find "MACOSX_DEPLOYMENT_TARGET = 11.0;" in project/macos/Runner.xcodeproj/project.pbxproj file, which means I successfully changed the deployment target.

Sorry, I forgot to change version in Podfile, my bad.


Extend removeRefCommand to support call hierarchy items

This PR is following this issue on VS Code repo. In short, it was asked to also enable the "remove" button for call hierarchy trees to help users cancel out any items they've reviewed.
The problem with adding this feature was that the implementation of the call hierarchy tree, would resolve outgoing/incoming calls on demand (i.e., when the user clicked to expand a node) and then the children data were not taken hold of in the models.
To fulfill the request, a children property was added to the CallItem model, and move, remove and change events were added to the CallsModel class. The original removeRefCommand of the extension was updated accordingly.
:warning: To avoid duplicating private method ReferencesModel._del, it's been moved to the outer context and referenced from withinCallsModel, too. Also, to be consistent, ReferencesModel._tail was moved with the former.

@jrieken Is this PR going to be merged for this release?

Sorry this got lost... Will look into it

Thank you!


tImageDDS needs auto-gamma-compression like tImageKTX.

Converted from project view. Task is already complete.


Fixes for uploading proxy logs

Two fixes:

Add condition: succeededOrFailed() to the copy/delete steps so they run if there are earlier failures in the pipeline.
Add more discrimination to the artifact name to prevent potential collisions.

/azp run azure-sdk-tools - sync - eng-common

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow


vsix extensions that utilise webviews are blocked by CSP

Similar to https://github.com/cdr/code-server/issues/1530
vscode-webview.net is blocked by the current CSP, could it be considered to be adding to the allowlist?
Refused to load the stylesheet 'https://vscode-remote+localhost.vscode-resource.vscode-webview.net:1337/<$HOME>/.local/share/code-server/extensions/mhutchie.git-graph-1.30.0/media/out.min.css'
because it violates the following Content Security Policy directive: "style-src https://*.vscode-webview.net 'unsafe-inline'".
Note that 'style-src-elem' was not explicitly set, so 'style-src' is used as a fallback.

Extension issue was found against: https://github.com/mhutchie/vscode-git-graph/issues/535
Reproduction:

run code-server locally via any port
open code-server in Chrome / Firefox
install git-graph extension
open a folder with a git project open (so some history is shown)
open git graph
see that the view is broken. (caused by CSP blocking out.min.css)

Code Server Info:
code-server: v3.11.1
VS Code: v1.57.1
Commit: c680aae973d83583e4a73dc0c422f44021f0140e
Date: 2021-08-06T18:33:37Z
Browser: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/94.0.4606.31 Safari/537.36

Yeah, I think when we landed https://github.com/cdr/code-server/pull/3895 it caused this issue.
Thanks for opening and providing repro steps! We'll take a look.

Without pulling locally, i'd hazard a guess that the initial resource check it causing this issue:
https://github.com/cdr/code-server/blob/67b23aaa1da46da4e2a10e3a6bd5e7127d79d284/lib/vscode/src/vs/workbench/api/common/shared/webview.ts#L46
as almost all resources will - at some point, reference http / https which I suspect is causing the auorities not to be leaded because of the return early.

Thanks for digging into that! That sounds likely, but I'll have to investigate to double-check. Hoping to get to this soon 🙏

Fixed by https://github.com/cdr/code-server/pull/4131


Inference with pretrain model

Good morning, thank you for sharing this repository and for your magnificent work. I would like to know, if it is possible to use one of your model pretrained, for example in Kitti3D, to do inference about a new point cloud. In my case i would like to do a 3D semantic segmentation on an acquired pointcloud. Thank you very much!

Hello, thanks for your interest in this project !
You can definitely use the pretrained models on your own data. To this end, you will need to create your dataset. I refer you to the instructions for creating your own dataset.
Besides, if the dataset you are using is open, we would gladly welcome pull requests from integrating new datasets in the project :wink:

Thank you very much for your kind answer, I will work on that. Unfortunately, it is not an open dataset.🤣

Too bad, integrating a new large-scale dataset would be great ! We will try to add more open datasets to this project in the future. Good luck with yours in the meantime ! :blush:

I am not extensively familiar with 3D Lane Detection, but from what I gather, this task seems heavily guided by radiometric  information. Said otherwise, if you wanted to tackle this task using a point cloud based method like SPT, you would need to be sure your points are properly colorized: no 3D-2D misalignments, well-calibrated colors, etc.
Again, I do not hold much expertise for this specific task, but I would assume the 3D geometric information is not as important for lane segmentation as it is for 3D semantic segmentation datasets such as S3DIS and KITTI-360. If so, the geometric information carried by points would not be so useful for lane segmentation and you might be better-off segmenting lanes directly in your images and projecting the result on your 3D points, if you have access to the camera model and pose.
By the way, if you happen to be interested in methods for jointly extracting features from point clouds and multi-view images, you can check our CVPR'22 work: https://github.com/drprojects/DeepViewAgg.
Good luck !

Thank you very much for your advice, it has been extremely useful, I will definitely check the repository that you have shared.


Install applink.c

Install applink.c among the public header files, like in older OpenSSL versions.
This is only done on the platforms where it makes any sense (Windows and mingw).

The issue was uncoverred while discussing in PR #1311

@jamercee, this might interest you.

+1

That was a quick review, I'm impressed.  I'm going to wait on Travis and Appveyor before merging, though...

Yes, definitely wait.  And get feedback from the OP.

I've tested the patch on a clean install -- works perfectly.

Cool.  Merged.


Logging verbosity

Running pyrobox as a systemd service, I would like to know what users are doing. So I redirect the pyrobox output to a log file.
From my point of view that is too verbose.
An example entry (depersonalized):
INFO:
================================== 10digit-ID ==================================
10digit-ID|=>	 request	: GET
10digit-ID|=>	 url     	: /dir/subdir/
10digit-ID|=>	 query   	: {}
10digit-ID|=>	 fragment	:
10digit-ID|=>	 full url 	: /dir/subdir/
++++++++++++++++++++++++++++++++++ 10digit-ID ++++++++++++++++++++++++++++++++++
INFO:
# 10digit-ID by [URL] at [Date time]|=> "GET /dir/subdir/ HTTP/1.1" 200 -

INFO:
---------------------------------- 10digit-ID ----------------------------------
################################################################################

That's 14 lines of information for one entry. The only information that I really need is line 10 starting with # 10digit-ID by.
This line contains the same information (with small exceptions) as all other lines together.
So I propose that you add a command line option --no-verbose that delivers the information of the  aforementioned line 10 only.
Thanks in advance.

Thanks for the heads u. Since It uses standard logging library, so i thought there was already a CLI based log handler.
I'll implement it asap
Ref: https://stackoverflow.com/questions/57192387/how-to-set-logging-level-from-command-line

However the =, + and - has special meanings
= : A request has been received. The request basic data as below

: The request handling is working, the log is below

: The request has been finalized and responded and closing the connection

Ok, thanks for this information. From my point of view, the line mentioned by me is still sufficient.
That is very similar to what I get e.g. from the python http.server or the uploadserver which is also based on the http.server.

Could you kindly rewrite which 10 lines?
Also a word of info,
INFO:
WARNING:
Etc are signs and part of logging, so i don't think I'll remove it. I'll keep checking different opinions

Instead of
INFO:
================================== 10digit-ID ==================================
10digit-ID|=>	 request	: GET
10digit-ID|=>	 url     	: /dir/subdir/
10digit-ID|=>	 query   	: {}
10digit-ID|=>	 fragment	:
10digit-ID|=>	 full url 	: /dir/subdir/
++++++++++++++++++++++++++++++++++ 10digit-ID ++++++++++++++++++++++++++++++++++
INFO:
# 10digit-ID by [URL] at [Date time]|=> "GET /dir/subdir/ HTTP/1.1" 200 -

INFO:
---------------------------------- 10digit-ID ----------------------------------
################################################################################

I would like to get
INFO: # 10digit-ID by [URL] at [Date time]|=> "GET /dir/subdir/ HTTP/1.1" 200 -

only.

I think that can be done... I'll look into it

fixed it (locally) soon will be updated (massive update (ui+backend+how server works) coming. please wait a bit


Package makes Mono.Cecil impossible to use

I have a project that references NUnit.Engine, so it now picks up testcentric.engine.metadata as a transitive dependency.
The same project also already uses Mono.Cecil.
Updating from an older version of NUnit.Engine to a newer one results in the following build errors:

CS0433: The type 'AssemblyDefinition' exists in both 'Mono.Cecil, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=0738eb9f132ed756' and 'testcentric.engine.metadata, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=6fe0a02d2036aa1d'

CS0433: The type 'ModuleAttributes' exists in both 'Mono.Cecil, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=0738eb9f132ed756' and 'testcentric.engine.metadata, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=6fe0a02d2036aa1d'

CS0433: The type 'TargetArchitecture' exists in both 'Mono.Cecil, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=0738eb9f132ed756' and 'testcentric.engine.metadata, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=6fe0a02d2036aa1d'

The README says that this is based on Mono.Cecil, but can you at least rename the APIs to not conflict?

@yaakov-h
The NUnit test engine isn't intended to be referenced by other projects, but to be accessed entirely through the nunit api assembly. That design was intended to isolate your code from situations like this.
That said, I realize that situations can arise where you might be forced to use an actual reference to the engine itself. It will help if you can explain the nature of that situation in your use case.
Although this is a TestCentric project, it will probably be simpler to discuss how the issue arises in using the NUnit engine. You should know, however, that I left the NUnit project after the latest engine release. Consequently, you should also file an issue with the engine project (https://github.com/nunit/nunit-console) since nothing I do here will automatically fix your NUnit problem. In addition, they may want to fix this in some other way.
I will try to resolve this issue in any case, since the TestCentric engine also makes use of TestCentric.Metadata and the same problem could easily arise. As I said, some kind of description of how you use the engine and where the conflicts arise will help me to do that.

We have a distributed test system that essentially needs two core functions:

Build a list of test to run on the build server
Run a specific test, or group of tests, one one of our many test agents.

In our older codebases we use NUnit directly, in newer ones we've switched to using VSTest, which can then invoke NUnit via the NUnit3TestAdapter package (or any other framework/engine tbh) and nicely isolates us from any test-engine-specific shenanigans like this.
From memory we started off by using nunit.engine.api but had to switch to nunit.engine due to crashes and bugs. I also seem to remember that nunit.engine.api doesn't actually include an NUnit engine, and tries to go and find one somewhere else.
With the way that the NUnit NuGet packages are structured at the moment, referencing nunit.engine brings in all of the relevant assemblies, and there doesn't seem to be a package structured such that I reference nunit.engine.api as an assembly reference and get all of the rest as content/files. That is probably the root cause of this, since the testcentric assembly is included in that directory, it isn't a transitive NuGet dependency.
Looking over the code, we have a custom ITestRunner to handle an Exception on disposal (https://github.com/nunit/nunit/issues/1656), though that seems to have been resolved so it may no longer be an issue.
We also build and initialise our own TestEngine - I don't remember why - and comments point to an InvalidOperationException being thrown from System.Diagnostics during application shutdown, so we have a workaround for that by nulling out EventListener.s_EventSourceShutdownRegistered and disposing of all event sources ourselves.
So that's our current situation in a nutshell.
It may be worth going over to the NUnit team and talking with them about restructuring the NUnit.Engine packages, but yes, other consumers including yourself could land up tripping over the same issue.
(FWIW I managed to work around this issue with an extern alias and a custom build target... not ideal, but it works.)

@rprouse
I'm thinking of changing all the Mono.Cecil namespaces to TestCentric.Metadata. Thoughts?

If you mean here then yes, sounds like a good idea. We might want to do the same in the engine.

Well, the NUnit engine actually uses the TestCentric.Metadata package, currently at version 1.7.1. Changing the namespace is breaking, so I'd probably call it version 2. Then you could update by changing some package references and using statements, which sounds easier than renaming methods.
BTW, I originally kept the Mono.Cecil namespace as a sort of homage to where the code came from. That was obviously a bit silly. :-)

@CharliePoole that's what browser user-agents are for 😄

@yaakov-h
This is fixed in the new 2.0.0 release available on GitHub and on nuget.org.
@rprouse @osirisTerje @jnm2 @mikkelbu @SeanKileen @stevenaw
Letting you guys know about this as issue #7 arose with an NUnit user.
I found this an easy fix for TestCentric, simply using a global replace to change Mono.Cecil to TestCentric.Metadata. Watch out for comments, which may be intentionally referring to Mono.Cecil.


Why do edges need an ID

I just started using sigma.js and I was wondering why edges also need to have a unique id..
Can I assign a random id to each (like their index number)?
Or does the id have to be consistent with their position or the nodes they connect?

The ID is only used as an identifier, so it has to be unique but you can choose anything you like for it, for instance a counter that you increase for each edge, or Math.random if you use enough digits.
The id is used to uniquely identify the edge when using methods such as dropEdge(id). If you don't need to address an added edge after it's creation you can discard the id.

In future versions, we'll use a system able to generate ids for you so you don't have to handle this problem. But, indeed, for the time being, the data model needs a unique identifier provided by the user.


Run EC predictions on new molecules

Hello!
Thanks for a great repository.
I wonder if there is any script available to run EC-substrate predictions on a given molecule. Also, I'd like to know if pre-trained models are available.
Many thanks!
M

I will close the issue for now (till the paper published) and feel free to reopen it.


Wait for email in 'register-and-log-in.test' failing

I've got a repository that is being build by Travis-CI. Everything is working, besides the standard test that is supplied by the current cookiecutter release. It fails, while it waits for the email after the signup. The email can be found inside the log, but hitch seems to not find it?
The log of my latest build can be found here: https://travis-ci.org/mimischi/django-clock/builds/92557925
The mail in the log starts here: https://travis-ci.org/mimischi/django-clock#L4547
Exception raised by hitch starts here: https://travis-ci.org/mimischi/django-clock#L4578
I don't really know why it is failing... or is it a problem with Travis-CI?

It looks like you've got your DJANGO_EMAIL_BACKEND set to log emails to the console rather than actually sending an email to the mock SMTP server.

EMAIL_BACKEND = 'django.core.mail.backends.console.EmailBackend' in local.py should be
EMAIL_BACKEND = 'django.core.mail.backends.smtp.EmailBackend'

Well it basically says that in your documentation, that one can set it to the console. See here: http://cookiecutter-django.readthedocs.org/en/latest/developing-locally.html (STRG+F for "Alternatively simply output").
Maybe put a warning that this will break hitchtest with Travis-CI? My basic confusion here was, that it sounds like Hitch is waiting for a line inside the log and Travis-CI is putting that mail in the log. Therefore I didn't make the connection with this setting!

On a related note: I would gladly use MailHog with cookiecutter, but I can't seem to find configuration line where 'grunt serve' will launch it? Am I supposed to launch it by hand everytime I'm developing locally or am I missing something?

Maybe put a warning that this will break hitchtest with Travis-CI? My basic confusion here was, that it sounds like Hitch is waiting for a line inside the log and Travis-CI is putting that mail in the log. Therefore I didn't make the connection with this setting!

Yes, the docs should probably have a note that it expects hitch to send an actual email to localhost:1025. I'll create a pull request to try and clarify it.

Am I supposed to launch it by hand everytime I'm developing locally

Yes. Since MailHog is running its own server the process has to be launched by hand.

Thanks! So it's working with the default SmtpBackend setting and MailHog. Just noticed though, that the file 'all.settings' sets an environment variabe for exactly this setting. Does it get ignored then?

It should get picked up if your django settings are configured to read the DJANGO_EMAIL_BACKEND environment variable.

This explains everything. Thank you very much!


setfulset version update apps/v1

update version beta to apps/v1

mongodb container is not maintained any more in this org. closing.


204 should be considered as a success

When REST server response HTTP 204 No Content, you throw error "Unexpected end of JSON input". But, it is not error, it is success.

Hi thanks for reporting this.
Could you provide a code sample or at least a sketch to help me reproduce the problem? If so,  i can probably get a fix out this week.  Otherwise it may be a while, since I'm not sure what the exact problem is based in your description.
Thanks! And thanks again for reporting the issue!

Ok so i took a quick look at what mighr cause this error. I may ve confused about where this error is arising, so i eagerly await sample code.
If it's the error I'm guessing it is,  the problem is that the app defaults to a json format in the api client. The default api client is configurable to allow changing that value.
The README suggests using this line to configure the api client when you kick off your saga
const client = new ApiClient({ basePath: 'https://example.com/api/v1' })
I suggest you try changing that to thisconst client = new ApiClient({ basePath: 'https://example.com/api/v1' })

Ugh too hard to type this on my phone I'll edit the above comment on a laptop

Also note that you can use the fetchconfig param in your action creators to achieve the same thing on one endpoint instead of on all of them. You probably only want the text format for the endpoints returning 204.
The other alternative is copy pasting ApiClient.js into your codebase and customizing it. I'd hot go that route and create useful improvements,  please make a pull request! Id love to see how your use case works.

I prepared pull request with fix of this bug: #47

Django REST Framework is a good platform to build sample CRUD backend app. https://tests4geeks.com/django-rest-framework-tutorial/

You can see my progress at https://github.com/devvmh/demo-redux-crud-store


Missing assets from gem package

Hi,
I have just upgraded from 0.5.5 to 0.6 and included the refile-s3 gem and refile-mini-magick gems, however, the app/assets/javascripts/refile.js file is missing and my app is complaining as we use that for S3 direct file uploads.
I can see it in the repository though
Thanks
Gary

You can use the gem directly from GitHub until this is fixed.
gem "refile", require: "refile/rails", github: "refile/refile"

Found the same thing, caused  by https://github.com/refile/refile/commit/f457be8e5306971148145e07e6eec228a3569579

Damnit! Pushed 0.6.1 which fixes this.


Upgrade to Fastify v4

Is your feature request related to a problem? Please describe.
https://medium.com/@fastifyjs/fastify-v4-ga-59f2103b5f0e is out
Describe the solution you'd like
N/A
Describe alternatives you've considered
N/A
Additional context
N/A

Thanks!

This is fixed in v4. You can read the migration guide for more information.


Unwanted login in finish step when login -t is used

The help says: -t. --test      Log into the guest after each executed test in the execute phase.
I have plan with one test, run tmt run login -t however there is an additional login in the finish step.

Hi @lukaszachy, would you please help to paste the fmf file and the CLI, I'd like to help to look into it, thanks!

E.g.
$ cat  reproducer.fmf
/plan:
  execute:
    how: tmt
  provision:
    how: virtual
  discover:
    how: fmf

/test:
  test: echo

And when you do tmt run login -t you see that [root@default-0 tree]# is twice, the later shouldn't be there IMO.
/var/tmp/tmt/run-103

/reproducer/plan
    discover
        how: fmf
        directory: /tmp/r_142617_Vj5
        summary: 1 test selected
    provision
        how: virtual
        user: root
        image: fedora
        memory: 2048 MB
        disk: 40 GB
        arch: x86_64
        progress: booting...
        distro: Fedora Linux 37 (Cloud Edition)
        summary: 1 guest provisioned
    prepare
        summary: 0 preparations applied
    execute
        how: tmt
        progress: /reproducer/test [1/1]        login: Starting interactive shell
[root@default-0 tests]#
exit
Shared connection to <IP_ADDRESS> closed.
        login: Interactive shell finished
        summary: 1 test executed
    report
        how: display
        summary: 1 test passed
    finish
        login: Starting interactive shell
[root@default-0 tree]#
exit
Shared connection to <IP_ADDRESS> closed.
        login: Interactive shell finished
        guest: stopped
        guest: removed
        summary: 1 task completed

total: 1 test passed

There are two entrances that invoke Login._login(), one is Login.go(), the other is Login.after_test().
# https://github.com/teemtee/tmt/blob/main/tmt/steps/__init__.py#L1171
  1171	class Login(Action):
  1172	    """ Log into the guest """
  ....<snip>...
  1249	    def go(self) -> None:
  1250	        """ Login to the guest(s) """
  1251
  1252	        if self._enabled_by_results(self.parent.plan.execute.results()):
  1253	            self._login()
# ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  ....<snip>...
  1295	    def after_test(
  1296	            self,
  1297	            result: 'tmt.base.Result',
  1298	            cwd: Optional[Path] = None,
  1299	            env: Optional[tmt.utils.EnvironmentType] = None) -> None:
  1300	        """ Check and login after test execution """
  1301	        if self._enabled_by_results([result]):
  1302	            self._login(cwd, env)
# ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

To fix it,

if option -t is used(e.g. tmt run login -t), should cancel to call self._login in Login.go() in Finish phase;
and if  option -t is not used (e.g. tmt run login), should support to call self._login in Login.go() in Finish phase.


feat: add basic normalization visualizer

Basic way I've been using this:
from therapy.analysis import draw_graph
from therapy.query import QueryHandler
q = QueryHandler()
draw_graph(q, "citalopram")

Nodes are weighted by: https://en.wikipedia.org/wiki/Betweenness_centrality
close #296

needs scipy added to lockfile. Without, visualization fails due to ModuleNotFoundError on draw_graph()


Zip Code Additions

Add the following Zip Codes to /metadata/tables/zip-code-data.csv
1,Alabama,AL,35046,Chilton,Clanton
1,Alabama,AL,36568,Mobile,Saint Elmo
1,Alabama,AL,36685,Mobile,Mobile
1,Alabama,AL,36688,Mobile,Mobile
1,Alabama,AL,36691,Mobile,Mobile
22,Louisiana,LA,70765,Iberville,Plaquemine
22,Louisiana,LA,70302,Lafourche,Thibodaux
22,Louisiana,LA,70835,East Baton Rouge,Baton rouge
22,Louisiana,LA,70707,Ascension,Gonzales
28,Mississippi,MS,39555,Jackson,Hurley

8/23/21 - Created GitHub issue #1939 and reportstream branch named rhood/Reddy Zip Code Updates.  I'll work with Joel to create the Pull Request and move to Staging and then Production.

This was completed.  There is a new pull request for addresses in the ReportStream GitHub.


add support to build unittest cpp on AIX (IBM)

Adding this simple flag allows to build unittest cpp on the IBM unix AIX via the IBM Opensource Toolbox.

@Flow86 thanks! I know it has been sitting for a long time.


Pmaster

@aurambaj While testing I noticed that stats (string and word counts) are incorrect with Android repos. Have you seen problems with stats?

I changed the stats computation to take in account the plural forms depending on the language... it should be working. Did you notice issue with some languages or overall?

I had existing Android repo with plurals that are fully translated. It only had one and other in the source resource file. When I generate localized files with the latest server, it adds more plural forms in the generated localized files depending on the plural rules of each language. However, those newly generated ones (one, two, etc) do not show up on the workbench. The existing translations show up without plural tags in the workbench. When the plural string gets updated, then they appear in workbench as  expected with plural tags. However, the existing approved translations become Needs-Translation.  Is this all expected?

Attached screen shot of stats not working:


fix album year display

Year display was missing from the popup.
Changed the existing anneeEditionMusique to year

looks like I broke a test...

Hi @geoffrey-guilbon! Thanks for your contribution 🙂
All is good: one test is failing because last.fm environment variables are not injected in pull requests, and it relies on them to perform an assertion.
I'll do something to test the correct behaviour.


StatItem.isAllowable() always lets 1 more request pass for a period

[X] I have searched the issues of this repository and believe that this is not a duplicate.
[X] I have checked the FAQ of this repository and believe that this is not a duplicate.

Environment

Dubbo version: 2.7.1
Operating System version: MacOS Mojave
Java version: 8

Steps to reproduce this issue
@Test
public void testAccuracy() throws Exception {
	statItem = new StatItem("test", 5, 60_000L);
	assertEquals(true, statItem.isAllowable());
	assertEquals(true, statItem.isAllowable());
	assertEquals(true, statItem.isAllowable());
	assertEquals(true, statItem.isAllowable());
	assertEquals(true, statItem.isAllowable());
	//Must block the 6th item
	assertEquals(false, statItem.isAllowable());
}

Expected Result
The last assertEquals should  pass. Because according to the JavaDoc of TPSLimiter
 if tps for a method m1 is 5 for a minute then if 6th call is made within the span of 1 minute then 6th      should not be allowed isAllowable will return false.
Actual Result
The last assertEquals failed, because statItem.isAllowable() lets 1 more request pass for a period.

close #4140 via #4141


Place the new message indicator at the top of the screen

Summary
This PR fixes the issue where new messages appeared below the keyboard when the new message indicator is present. We now place the new message indicator as far up in the list as possible but always visible.
Other fixes in this PR:

Scroll pop in certain cases with open graph metadata where the image url does not match the images in the metadata
Post visibility was set to 0 at times causing a refresh of the post list.

@enahum This is ok for 1.21 if non-risky

I'm still experiencing the new messages under keyboard issue with 1.21.1 (Build 210)

@enahum Was there a Jira ticket for this and was this tested by QA? Some people still see this issue:

https://github.com/mattermost/mattermost-mobile/pull/2943#issuecomment-516984005
https://github.com/mattermost/mattermost-mobile/issues/2940#issuecomment-517461095

@amyblais https://github.com/mattermost/mattermost-mobile/pull/3074


fix: issue #14

Apache 2.4 needs a bit different .htaccess
This fix is untested on Apache 2.2

Probably works fine on Apache 2.2, but it would be better if someone can test it.

Will submit proper fix.

Little advice: You can always overwrite your PR by adding more commits or force-pushing alternative ones to the branch of the PR.

Well, not so easy when doing stuff via Smartphone and Github web. ;)


FLINK-6391

FLINK-6391
Improve change-scala-version.sh to correctly deal with references to flink-gelly-examples.

Any ideas why we can't just reference scala.binary.version in the Maven configurations rather than specifying 2.10 and modifying this in the change script? I don't see why change-scala-version.sh is any more complicated than change-version.sh.

@greghogan that might be a good idea but I don't know much about it.  I'm merely focused on fixing the build break.

Fixed by 03afbca5d047988e7fd8aff5d8b83ddee19570c9


create small application for profiling

Use https://github.com/evanw/esbuild

Will be done as part of https://github.com/zakodium/react-plot/issues/289


update gix to the latest version

Please note that I have removed the max-performance-safe feature
which pushes the choice to the consumer of (any) library that uses
gix. It's also causing less C to be compiled by default.
Checklist

[x] I have read the Contributor Guide
[x] I have read and agree to the Code of Conduct
[x] I have added a description of my changes and why I'd like them included in the section below

Description of Changes
An upgrade to the latest gix, which includes less dependencies now and compiles a little faster
because of it. It also avoids making decisions about performance (and the inclusion of C-crates)
by removing the max-performance-safe feature.
The motivation for this update is to get a new tame-index release for consumption in this PR
so that it can also use the latest gix version without duplication.
Related Issues
None

Oh, so the clippy lints are real - I thought this can't be as I have simplified and boxed errors rather than making them more complicated. I typically address these by ignoring the clippy lint, but maybe in this case, it's better to box? Please advise :).
Finally, you might be interested in a new line of conversation that will hopefully lead to a complete revamp of error handling as a mix between anyhow and thiserror: therror :D. Let's see if it happens, I surely hope so.

But definitely interested to see what you can do for errors in future revisions, it is a bit of pain point with gix at the moment unless one goes the lazy anyhow route.


Invalid JSON-LD context

The document says

The VPP is based on Json-Ld message types. The CS Json-Ld context is:
https://w3id.org/tractusx-trust/v0.8

but the URL does not resolve to a valid JSON-LD @context document. should probably resolve to something like in this file https://github.com/eclipse-tractusx/identity-trust/blob/main/specifications/M1/context.json in which case, top level domain is probably not w3id.org..?

Thanks for your input. A few notes and clarifications:

The Json-Ld context is valid. The issue is that the context is not published at the designated URL. This is planned and will be done when we have enabled publication of the specifications as static resources (see below).
The file should not resolve to an untagged GitHub file resource as that would be bad practice, the content type will be set by GitHub incorrectly, and clients will run into throttling imposed by the latter.
We have already planned to publish the context to a static location that uses the proper content type.
The context URL will use w3id.org and redirect to the static location. This approach is a common and well-documented industry practice.
Publishing the context is not critical. Implementations should load contexts from a trusted cache and never resolve them over the Internet for performance and security reasons.

I'm going to close this issue out and open and open an issue to track the Json-Ld context publication.


High CPU usage on startup

VS Code version: 1.8.0
OSX: Sierra 10.12.1
I restarted VS Code just now, and on a non-PHP project, I see this in the process list:

It's currently using 95% CPU without any indication of doing anything, and this is on a project that doesn't contain any PHP code.

Dup of #193 ?
Look at the output panel, it is probably indexing.
Also, the LS is only activated when you open a PHP file, so this happening in a non-PHP project is very unlikely

If that's the case, it was probably hanging from when I was working on a PHP-project.
I restarted VS Code and killed the process. Will report back if it happens again.
It might be related to xdebug too, since I have that enabled.

XDebug has a major impact on performance. If you notice a hang-up please open a new issue, thanks!


Bump rack_strip_client_ip from 0.0.1 to 0.0.2

Bumps rack_strip_client_ip from 0.0.1 to 0.0.2.

Changelog
Commits

(Generated using Dependabot.)

thanks @greysteil but we'll do this as part of the Rails 5 upgrade

No worries. Bets of luck!


OLS-624: Rest of integration tests for attachments

Description
Rest of integration tests for attachments
Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library
[ ] Bump-up library or tool used for development (does not change the final image)
[x] Integration tests

Related Tickets & Documents

Related Issue #OLS-624
Closes #

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

Project coverage is 95.40%. Comparing base (10be3e2) to head (3a1143b).
Report is 6 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1080   +/-   ##
=======================================
  Coverage   95.40%   95.40%
=======================================
  Files          55       55
  Lines        2352     2352
=======================================
  Hits         2244     2244
  Misses        108      108

see 2 files with indirect coverage changes

/approve

/override "Red Hat Konflux / lightspeed-service-on-pull-request"


[GOBBLIN-1943] Use AWS version 1.12.261 to fix a security vulnerability in the previ…

…ous version
Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[ ] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-1943

Description

[ ] Here are some details about my PR, including screenshots (if applicable):

The current AWS SDK version has a security vulnerability:
aws-java-sdk-s3 is vulnerable to path traversal. The vulnerability exists due to the insufficient guard logic used for the download directory in the leavesRoot function of TransferManager.java, allowing an attacker to access files from the S3 bucket that is one level up in the file system by evading the validation logic by adding a UNIX double-dot to the bucket key when the directory name prefix matches the destination directory.
Tests

[ ] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[ ] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

We're currently processing your upload.  This comment will be updated when the results are available.


LEGO: Pull request from lego/hb_04604851-bac4-4681-9f74-73de611d6e48_20230304164235493 to main

LEGO: Pull request from lego/hb_04604851-bac4-4681-9f74-73de611d6e48_20230304164235493 to main with localized lcls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
3 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.005%) to 41.95%

Files with Coverage Reduction
New Missed Lines
%

extensions/sql-database-projects/src/common/httpClient.ts
1
66.18%

extensions/sql-database-projects/src/dialogs/publishDatabaseDialog.ts
2
52.88%

Totals

Change from base Build<PHONE_NUMBER>:
-0.005%

Covered Lines:
28800

Relevant Lines:
63743

💛  - Coveralls


Method TOldRttiMarshal.ToClass do not populates object's properties, just public fields.

Hi Fabrício, I downloaded the last version and installed it in Delphi 2010. I notice that properties of my objects weren't populated, so I had to changed some piece of code to it works. You can see changes below.
In my undertanding about object oriented programming, classes didn't should expose fields directly. So, If you agree with if you can incorporate my code in your component.
function TOldRttiMarshal.ToClass(AObject: TObject): ISuperObject;
var
  i: Integer;
  vTypeInfo: PTypeInfo;
  vTypeData: PTypeData;
  vPropList: PPropList;
  vPropInfo: PPropInfo;
  value: ISuperObject;
  vObjProp: TObject;
  vAdapter: IJsonListAdapter;
begin
  if AObject = nil then
  begin
    Result := SuperObject.SO();
    Exit;
  end;

  if (AObject is TList) then
  begin
    Result := ToList(TList(AObject));
  end
  else if Supports(AObject, IJsonListAdapter, vAdapter) then
  begin
    Result := ToList(TList(vAdapter.UnWrapList));
  end
  else
  begin
    vTypeInfo := AObject.ClassInfo;

    if vTypeInfo = nil then
    begin
      raise ENoSerializableClass.Create(AObject.ClassType);
    end;

    vTypeData := GetTypeData(vTypeInfo);

    Result := SO();

    New(vPropList);
    try
      GetPropList(vTypeInfo, tkProperties, vPropList);

      for i := 0 to vTypeData^.PropCount-1 do
      begin
        vPropInfo := vPropList^[i];

        value := nil;

        case vPropInfo^.PropType^.Kind of
          tkMethod: ;
          tkSet, tkInteger, tkEnumeration: value := ToInteger(AObject, vPropInfo);
          tkInt64: value := ToInt64(AObject, vPropInfo);
          tkFloat: value := ToFloat(AObject, vPropInfo);
          tkChar, tkWChar: value := ToChar(AObject, vPropInfo);
          tkString, tkLString,
          {$IFDEF UNICODE}
          tkUString,
          {$ENDIF}
          tkWString: value := ToJsonString(AObject, vPropInfo);
          tkClass: begin
                      value := nil;
                      vObjProp := GetObjectProp(AObject, vPropInfo);
                      if Assigned(vObjProp) then
                      begin
                        if vObjProp is TList then
                          value := ToList(TList(vObjProp))
                        else
                          value := ToClass(vObjProp);
                      end;
                   end;
        end;

        if Assigned(value) then
        begin
          Result.O[String(vPropInfo^.Name)] := value;
        end;
      end;
    finally
      Dispose(vPropList);
    end;
  end;
end;

Hi @fabiolindemberg
I got your point, but this is by design.
The first marshaller was SuperObject, who uses fields for serialization. Thats why TOldRttiMarshal also use it. Besides that, Delphi TJSONMarshal are also based on object fields.
We can't just change this to use properties instead, because we need to maintain compatibility. If you want to give a try, you should change all supported Marshalls, and make this option configurable, making serialization using fields the default option.
Anyway, these objects in general are not domain objects, they are Data Transfer Objects(DTO), created with purpose for transferring data over the wire. So, for me there is no problem exposing the fields.
IMHO, i think the current solution is good enought, and the effort to change (and mantain) the Marshalls  not worth it.
Feel free to send a pull request.


Fix prod navbar conditional colors not working

For some reason, in prod, the pathname has a trailing slash, see: https://github.com/withastro/astro/issues/4252

Hi @zzzsyyy, thanks for pointing this out! Being new to Astro, I didn't even know this config option existed; maybe you should point this out in the original issue to guide people like me.


Add mock to test requirements.

Fixes error AttributeError: 'module' object has no attribute 'test_configargparse'
Credits to @hlieberman for finding the cause.
Fixes #125.

Thanks @macfreek and @hlieberman


Added pick game page

Description
Created pick game page.
Fixes #126
How to test?
npm run dev
Checklist

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] I have merged the latest version of the source branch (typically develop) into this branch before pushing my changes
[ ] This PR is ready to be merged and not breaking any other functionality

that is not done manually.. it comes automatically from the database..


this plugin doesn't support DiffEditor?

my code is

but it throws

the editor is instance of IStandaloneCodeEditor here, is there a way i can use this plugin in IStandaloneDiffEditor's ModifiedEditor ?

No, This plugin is not intended to be used in Diff Editor.
As I'm using some internal functions of monaco editor, I want the instance to be exactly ICodeEditor.
If you feel there is a proper usecase for this plugin in Diff Editor, feel free to create a pull request incorporating that functionality

@lynnxyli Hello, how did you solve it in the end?


Dynamic load runnable tasks runing by ThreadPoolTaskScheduler. No lock configuration for task.

Read the documentation
Read the documentation, especially the troubleshooting section.
Update to the latest version.
Describe the bug

Which version do you use: 4.33.0
Which Lock Provider: Jedis
ShedLock configuration: That's what i'm looking for
ShedLock logs:

SpringLockConfigurationExtractor:getLockConfiguration:67 - Unknown task type xxxx
DefaultLockManager.executeWithLock:49 - No lock configuration for xxx. Executing without lock.

Expected behavior
I'm using a ThreadPoolTaskScheduler to load task information from MySQL. Creating task beans which implementing Runnable. Then call threadPoolTaskScheduler.scheduleAtFixedRate(task, initialDelay, interval) to start all tasks
My expectation is to run multiple application nodes at same time and use Shedlock to avoid duplicate tasks running from every node.
After reading doc I thought TaskSchedulere Proxy mode may be suitable for my use case? I added @EnabledSchedulerLock(interceptMode = PROXY_SCHEDULER, ...), and set JedisConnectionFactory.
But when I check debug log, Shedlock shows Unknown task xxx and No lock configuration for xxx. I read source code it shows DefaultLockManager's getLockConfiguration only deal with ScheduledMethodRunnable?
In my case, how should I made my runnable to be proxied as a ScheduledMethodRunnable? Or is there any mistake I make it wrong?
Actual behavior
Multiple appplication nodes run fixed rate scheduler tasks with Shedlock.
Thanks

Hi, it seems you are trying to implement your own scheduler. Please use an existing one, I can recommend db-scheduler.
If you insist on implemnting your own, I would recommend on using locking without framework approach.


No App_Config folder appear when installing on .Net Core Class Library

I've got a .Net Core Class Library project targetring .Net 452 so that I can use the assemblies under Sitecore 8.2:
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <TargetFramework>net452</TargetFramework>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="Unicorn" Version="4.0.2" />
  </ItemGroup>

</Project>

The Unicorn assemblies appear under the bin folder but unfortunately the App_Config folder is missing Though, it appears if I install the package on a regular .Net Framework 452 project.
Is this a Unicorn nuget package issue?

It's a limitation of Nuget 3 which is required for .net core. It no longer
supports solution files being deployed from packages.
The workaround is to get the config out of the GitHub repo or from the
Nuget package manually.
On Wed, Sep 27, 2017, 3:23 AM sergeyshushlyapin<EMAIL_ADDRESS>wrote:

I've got a .Net Core Class Library project targetring .Net 452 so that I
can use the assemblies under Sitecore 8.2:

net452

The Unicorn assemblies appear under the bin folder but unfortunately the
App_Config folder is missing Though, it appears if I install the package
on a regular .Net Framework 452 project.
Is this a Unicorn nuget package issue?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/kamsar/Unicorn/issues/247, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGU_SbBIsY3ghH784hWkyhz1vQE5qfVks5smiIogaJpZM4Pli2c
.

Yep... That is what I did to solve that. Thanks!


Create developer guide

I am trying to get the hello-world example running on my machine. I notice that there are a number of undeclared dependencies. So far I have created the portable container image but to do so I had to install:
jq
pigz
podman - which does not have packages for some distro and architectures. I modified the code to use docker instead. Hope that will be OK down the line. If so you might consider that in your code.
and of course puccini
The scripts also expect directory /Depot/Temporary to be present. I checked this on a distro which did have podman and confirmed it is not created by that package.
There is also a permissions issue, at least with my docker workaround, as the image file is saved by root at mod 611 and thus cannot be rewritten by scripts/save-portable-container-image running as another user.

Thank you. Definitely there needs to be a developer's guide for Turandot. I will leave this issue open for now.
By the way, Podman exactly avoids these Docker permission issues, as it does not require root.

OK thanks
A further dependency is reposure.

Here is a link to the guide.

I'll just point out that I create a MUCH simpler quickstart guide that does not require registries (and it goes together with a simpler Hello World example).
The more advanced quickstart guide (and example) is still there.


setcap cap_net_raw,cap_net_admin=eip for tcpdump in lxc ?

Required information

Distribution: Ubuntu
Distribution version: xenial
The output of "lxc info" or if that fails:

$ uname -a
Linux dpjenkins 4.4.0-38-generic #57-Ubuntu SMP Tue Sep 6 15:42:33 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
$ lxc info|egrep '(version|storage)'
apiversion: "1.0"
  driverversion: 2.0.4
  kernelversion: 4.4.0-38-generic
  serverversion: 2.0.4
  storage: dir
  storageversion: ""

Issue description
I want to set above capability inside lxc.
goal is to be able to setup cuckoo sandbox inside lxc.
As seen above, I tried to removed to all cap drop without success.
setting container to privileged worked but is too much.
Is it possible? or not like httpd/setfcap on centos?
Thanks
Steps to reproduce
root@default-ubuntu-1604-1476629798:~# setcap cap_net_raw,cap_net_admin=eip /usr/sbin/tcpdump
Failed to set capabilities on file `/usr/sbin/tcpdump' (Invalid argument)
The value of the capability argument is not permitted for a file. Or the file is not a regular (non-symlink) file

It's currently not possible because of the lack of kernel support for unprivileged filesystem capabilities.
You could make your container privileged at which point setcap should work just fine, but this also makes the whole container much more dangerous to the system, so you may prefer sticking with setuid inside an unprivileged container for the time being.
@hallyn is working on a kernel patchset which will add this feature to the kernel, at which point setcap will just start working the way you'd expect.
Closing this issue as there's unfortunately nothing we can do about it in LXD itself.

no problem. thanks for the detailed information @stgraber !

Hi @sgraber and @candlerb  , unfortunately I have exactly the same issue with a privileged lxc container.
setcap cap_fowner,cap_mknod,cap_sys_chroot=ep /usr/bin/loolforkit
Failed to set capabilities on file `/usr/bin/loolforkit' (Operation not supported)
The value of the capability argument is not permitted for a file. Or the file is not a regular (non-symlink) file

I have checked that extended attributes are enabled:
cat /proc/fs/ext4/sda6/options | grep xattr
user_xattr

Is it OK to add this to the existing issue or should I open a new one?

You did check with ls -l /usr/bin/loolforkit that it is not a symlink?
But in any case, it's a different issue.  Firstly, you said it's an lxc container (not an lxd container).  And secondly, you said it occurs in a privileged container - this ticket is about setcap in unprivileged containers.

Thank you, I have opened a new ticket with the additional information:
lxc privileged container: setcap Failed to set capabilities on file #3172

Wrong link, should be to https://github.com/lxc/lxc/issues/3172

thank you, sorry my mistake


Pronouns field on NPC sheet doesn't save

Bug Description:
I noticed that the pronouns field on the NPC sheet wasn't saving when I edited it, so I exported the data for the actor and searched the JSON-- it looks like NPCs don't have a key/value pair for this field at all. It would be nice to have!
Steps to Reproduce:

Create NPC actor
Fill out pronouns field
Close sheet, or edit sheet settings
Reopen sheet
Field is empty

Expected behavior:
The pronouns field persists after closing the sheet.
(Thanks for all your hard work on this system! I'm excited to be running my first game 😁)

Added field for NPC pronouns, so they can get saved.
Fixed in next release.


Migrate testing functionality from nose to pytest

Migrate testing functionality from nose to pytest. See pysal/libpysal#241.
Examples of pytest usage:

tobler
giddy
region
geopandas

To Do:

[ ] determine how to convert all current nosetests into pytests


I can make a PR if need be. If I have a group, I see it as having a series of children. That is,
CheckoutRoutes = FlowRouter.group({
  prefix: '/checkout/:orderId',
  subscriptions: function (params) {
    this.register('order', Meteor.subscribe('order', params.orderId));
  }
});

CheckoutRoutes.route('/shipping', {
  step: 0
  // ...
});

CheckoutRoutes.route('/payment', {
  step: 1
  // ...
});

The way I see this is that it has a heirarchical structure like this
/  (FlowRouter)
└── /checkout/:orderId  (CheckoutRoutes)
    ├── /shipping
    └── /payment

I want to get the 'children' of the CheckoutRoutes. I am not sure if this is currently possible without utilizing private methods.
FlowRouter._routes.filter(route => route.group === CheckoutRoutes);

+1

Current solution would be to use FlowRouter.onRouteRegister to construct representing tree yourself, as I am doing now to generate navigation dynamically.
So I would also vote for API call which return JSON representation of whole route tree. Than any1 can crawl it as he wants and get information.

To provide example what i was talking about, my dynamically generated navigation menu.
Fill the ROUTE_LIST automatically.
LocalState.set("ROUTE_LIST",[]);

  FlowRouter.onRouteRegister( (route) => {
       LocalState.set("ROUTE_LIST", [ ...LocalState.get("ROUTE_LIST"), route]);
    }
  )

  FlowRouter.route('/', {
    name: 'posts.list',
    action() {
      mount(MainLayoutCtx, {
        content: () => (<PostList />)
      });
    },
    navigation: {
      label: "Home",
      position: 0
    }
  });

And than I can iterate/filter/sort how I need.
const routeList = _.chain(LocalState.get("ROUTE_LIST"))
    .filter( route => (!! route.options.navigation))
    .sortBy( route => route.options.navigation.position)
    .value();
  console.log("route list: ", routeList);

I did not tested it, but I would expect your route names to be available as route.group.name or route.group.parent.name .
If not I would try if the group itself does not have similar hook function of adding new route to it.
Did not checked sources.
Good luck

I hope this is well answered.
Thanks @ShockiTV


Add TensorFlowLite 2.18.0 to Maven Central/CocoaPods Specs.

Can we please publish the latest (last?) version of TensorFlowLite for Android and iOS before we all move on to LiteRT?
Thanks!

Hi, @v-hogood
I apologize for the delayed response, I checked and we published last tensorflow-lite version is 2.16.1 but I need to check with relevant internal team the possibility of publishing the latest TensorFlow Lite version (2.18.0) to Maven Central (Android) and CocoaPods (iOS) before transitioning entirely to LiteRT.
Meanwhile please check this official documentation of Migrate to LiteRT from TensorFlow Lite
To migrate an Android application using Tensorflow Lite, replace the dependency from org.tensorflow:tensorflow-lite to com.google.ai.edge.litert. The LiteRT Maven repository includes the following packages:
com.google.ai.edge.litert:litert
com.google.ai.edge.litert:litert-gpu
com.google.ai.edge.litert:litert-metadata
com.google.ai.edge.litert:litert-support
You can make this change in your build.gradle dependencies:
dependencies {
  ...
  implementation `com.google.ai.edge.litert:litert:1.0.1`
}

Thank you for your cooperation and patience.


Director interface

Add Origin metrics interface v1
Add Director metrics interface v1
Add Graph Overlay and Context for control
Move time context to root layout
Update the Nginx Proxy to auto reload credentials
Add test framework and tests
Add Boxplot graphs
Add usage Bar graphs

@h2zh I made the changes we discussed. Do you mind approving this PR when you get a chance?


Add-combined-datasets-to-other-data-sources

Functionality to add datasets from the category "combined" to the other data source categories.
It's not a chunky PR, just a lot of config changes as part of it:)
The idea here is that combined datasets can be attributed to the category they fit most in (e.g. altimetry or gravimetry). Therefore the config files now accept the keywords "include_combined_annual_datasets" and "include_combined_trend_datasets" (both optional), where combined datasets can be listed to be added to that category as an exception.
Part of the PR is also the now updated config files with the combined datasets assigned to different categories to be discussed in the workshop
This change has prompted several smaller changes such as:

restrict min-max of datasets for nicer plotting
quick handle of gaps when extending annual series
handle cumulative plotting in the case where reference timeseries for vertical adjustment has gaps

Thanks for review Alex, I think I have implemented all the changes now:)


Remove ElasticSearch port variable and update references in documenta…

…tion.
With this change, the ElasticSearch URL should be passed through the variable elasticsearch.url and prometheus.es_url.
 Assuming so, can we figure out a time to flip it so you can get a happy green check mark?

This one is depending on https://github.com/cloud-bulldozer/benchmark-wrapper/pull/219 (which passed tests after changing the ES url in the jenkins job)
e2e-benchmark scripts also will need this PR https://github.com/cloud-bulldozer/e2e-benchmarking/pull/61

!!!  Couldn't read commit file !!!


Two separate "one-child-links" show up in the contextual menu.

Please see attached image.

I also have this issue.

Me, too.

Seems to be caused by the new board hub experience.

Is there a plan to address this duplication or any workaround to eliminate one of them?


Add host helper

Would be nice to have a host helper (specifically needed for sonarqube).

Done :+1:


What is the main difference between this project and michuanhaohao's reid-strong-baseline? (except the new ideas in paper)

I tried this project under same dataset and config file, but this project's mAP and R1 is about 1% lower than the origin baseline. Will it be affected by the ignite lib ?  Thanks !

This repo uses the origin baseline as a starting point, focusing on finding the strong baseline of vehicle reid and the baseline of the AICITY challenge.


Float64() returns ok=false -- true expected

In the following example, Float64() returns ok=false whereas I would have expected true here.
	b := &decimal.Big{}
	b.SetString("6190.000000000000")
	_, ok := b.Float64()
	if !ok {
		log.Fatalf("6190 should fit in a float just fine")
	}

Hmmm, yeah. While it does return the correct float result (6190.000...), the 'false' is definitely wrong. I suppose if the decimal, x, is an integer and x <= math.MaxInt64 we can return it directly as a float64.
The case that was hit in your example was meant for something like 6190.12345678912398 which can't be exactly represented as a float (you'd get 6190.12345678912425 instead).


Make empty area under categories scroll the markets - similar to the empty area under betslip

.. as it looks like it should scroll

@JohnDanz This is working fine at larger sizes but at smaller sizes the categories aren't remaining sticky:

The issue you see below might be related to this (clicking into a category cuts off categories)

perfect


feat: Add gen_arg test with arbitrary block

Ref https://github.com/kkrt-labs/keth/issues/182
Draft PR to check if this is in the intended direction.
Will correct the asserts to make sure the test passes

hey @varun-doshi it's actually the intended direction thanks

Ready for review


Well Edit - Cannot change a value to unspecified

Currently the user can change a field in well edit to an allowed value, but blanking out the field (to change it from its value to empty) has no effect. For example, if a well record has the sample unspecified, once a sample has been specified, it cannot be changed back to unspecified.
This behavior was selected originally because some fields load as empty if multiple wells are selected and there are multiple values across the selected wells for that field. If a value is not specified in that field, then the form submits with blank. From the perspective of the form submit, this case is indistinguishable from the case where the user deliberated submitted with a blank value.
In summary, there are two cases that need to be handled:

Multiple wells are selected and for a given field those wells have different values, so the field loads as blank and when submitted will be blank in that field. Desired behavior: do nothing (currently works).
Regardless of the number of wells selected or their current values in a field, the user deliberately blanks out a field and intends for all records to be blank in that field. Desired behavior: change all selected records in that field (currently doesn't happen because of behavior implemented in first bullet).

Unsure of how to handle this conflict in behavior.

Consider a form element that tracks whether a field was blanked. Use that (rather than value of the field itself) to signal blanking of the field in the selected Well Condition records.


issue: Faster  Whisper Model download program flashing

OS: Win11 PRO X64
Buzz Version:  0.8.1
issue: Faster Whisper Model download program flashing

I downloaded the model of Whisper.cpp and used it to recognize subtitles, but at the beginning, the CMD window showed a flash of a few tenths of a second. After a while, the Buzz display was completed, but I clicked on the subtitles to see that they were blank.

Have you found a solution?

Have you found a solution?

NO. Did you encounter the same problem?

Have you found a solution?

NO. Did you encounter the same problem?

Yes, I ended up doing everything based on the WhisperDesktop CLI with my gpu it was much faster and better.

Yeah, I'm not sure what is causing the issue here tbh. I just tested on a new Windows 11 VM and both Whisper.cpp and Faster Whisper work fine. I'll update the build to add better logging for the errors, but looks like WhisperDesktop might be a better solution atm.

Maybe you've already downloaded the model, so there is no error for you.
In model_loader.py
    if model.model_type == ModelType.FASTER_WHISPER:
        try:
            return download_faster_whisper_model(size=model.whisper_model_size.value, local_files_only=True)
        except (ValueError, FileNotFoundError):
            return None

def download_faster_whisper_model(size: str, local_files_only=False, tqdm_class: Optional[tqdm] = None):
    if size not in faster_whisper.utils._MODELS:
        raise ValueError(
            "Invalid model size '%s', expected one of: %s" % (
                size, ", ".join(faster_whisper.utils._MODELS))
        )

    repo_id = "guillaumekln/faster-whisper-%s" % size

    allow_patterns = [
        "config.json",
        "model.bin",
        "tokenizer.json",
        "vocabulary.txt",
    ]

    return huggingface_hub.snapshot_download(repo_id, allow_patterns=allow_patterns, local_files_only=local_files_only,
                                             tqdm_class=tqdm_class)

You call download_faster_whisper_model  with local_files_only=True, so huggingface_hub.snapshot_download would run with local_files_only=True. I didn't download the model previously, when I run it I got this,
>>> huggingface_hub.snapshot_download(repo_id, allow_patterns=allow_patterns, local_files_only=True, tqdm_class=None)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/chaomai/.local/opt/miniconda3/envs/common_env/lib/python3.10/site-packages/huggingface_hub/utils/_validators.py", line 120, in _inner_fn
    return fn(*args, **kwargs)
  File "/home/chaomai/.local/opt/miniconda3/envs/common_env/lib/python3.10/site-packages/huggingface_hub/_snapshot_download.py", line 169, in snapshot_download
    with open(ref_path) as f:
FileNotFoundError: [Errno 2] No such file or directory: '/home/chaomai/.cache/huggingface/hub/models--guillaumekln--faster-whisper-large-v2/refs/main'

If some unexpected issue happens during model download Like network disconnect or something. Part of model files will be downloaded, but not all. This will cause errors. To fix this you need to delete model cache folder and re-download it.
Also in latest development versions there are improvements to model download, so try the latest development version. To get it log into GitHub and download the latest development version from some successful action run https://github.com/chidiwilliams/buzz/actions?query=branch%3Amain
Please re-open issue or create new one if this is still a problem on latest development version.


The last release was a long time ago

Any plans for cutting a new release? The last one was a long time ago and there have been many commits since...

install with homebrew, you'll get the newest version


ISSUE#11 feat: add printColorList() and global colorList

Prints a global colorList by iterating over the list.
Assuming that colorList contains hexcode of all the colors detected.
Issue #11

@SilverDragonOfR  Please make a pull request again and make the necessary changes accordingly.

Can you specify which changes?

@SilverDragonOfR Please add your code and then make a pull request again from the updated main branch.

ok will do


Fixed the syntax warnings that arise when executing the script.

This fix also seems to improve pattern matching performance.

Or better yet, use a raw string literal.
match_siteinfo = re.findall(r'window\.siteInfo\s*=\s*({[^}]+})', main_page)


Update concepts-network.md

Contact me at v-dibr if you need more info. The pods can't have public IP addresses so this wording is misleading.

@dcbrown16 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


silent/uninteractive mode?

is there any type of non interactive mode for doing updates?  either by API or CLI?

I think it's now possible by #280

Yes, pass the -y flag 🙌


java lib updates

Latest:

spark 2.3
immutables 2.1.15
slf4j 1.7.20
logback 1.1.7
hikari 2.4.5
jackson 2.7.0

Marking as small change (prob wishful thinking)


Cursor position is off

Platform: OSX

This is a direct clone of master.

Thanks for the report; should be fixed.


Template-ify aws cert

Rather than have the AWS saml cert embedded in the j2 file, I've split it out to a local yml file so it's just pulled in when needed.
This was approved in #2834 by @rhdedgar

[test]


Convert onnx to tensorflow fails

Issue Type
Support
OS
Ubuntu
OS architecture
x86_64
Programming Language
Python
Framework
ONNX
Description
Sorry to bother. I intend to convert an onnx model to tensorflow and then to tflite. The story goes like this: The int8 quantization has very awful accuracy for lite-model_movenet_singlepose_thunder_3.tflite. So I plan to inverse quantize lite-model_movenet_singlepose_thunder_tflite_int8_4.tflite, a quantization-aware model, to float32 version, and then check the quantization accuracy for int8. You know, I want to do some fine tune based on the thunder model, not just simply integrate the model to applications. One possible flow is: tflite INT8 -> onnx float32 -> tensorflow -> tflite. I can now succeed in converting tflite to onnx but fail in onnx to tensorflow:
<1> convert tflite to onnx:
python3.6 -m tf2onnx.convert --opset 13--tflite lite-model_movenet_singlepose_thunder_tflite_int8_4.tflite --output model.onnx --inputs-as-nchw serving_default_input:0
The opset must be set to 13 since per-axis quantization for some layers must be supported. Above command can be executed normally on my side.
<2> convert onnx to tf
from onnx_tf.backend import prepare
import onnx
import os
import sys
TF_PATH = "./my_tf_model.pb"
ONNX_PATH = "./simplified.onnx"
onnx_model = onnx.load(ONNX_PATH)
tf_rep = prepare(onnx_model)
tf_rep.export_graph(TF_PATH)
where "simplified.onnx" is the model simplified by onnxsim using above model.onnx as the input.
The following error is then reported:
onnx_tf_prefix_truediv_1;truediv_1/y1_prequant' is not a valid scope name
I cannot google any similar information. Any suggestions? Thanks.
Relevant Log Output
/usr/local/lib/python3.6/dist-packages/onnx_tf/backend_tf_module.py:98 __call__  *
        output_ops = self.backend._onnx_node_to_tensorflow_op(onnx_node,
    /usr/local/lib/python3.6/dist-packages/onnx_tf/backend.py:328 _onnx_node_to_tensorflow_op  *
        return handler.handle(node, tensor_dict=tensor_dict, strict=strict)
    /usr/local/lib/python3.6/dist-packages/onnx_tf/handlers/handler.py:59 handle  *
        return ver_handle(node, **kwargs)
    /usr/local/lib/python3.6/dist-packages/onnx_tf/handlers/backend/mul.py:27 version_13  *
        return [cls.make_tensor_from_onnx_node(node, **kwargs)]
    /usr/local/lib/python3.6/dist-packages/onnx_tf/handlers/backend_handler.py:157 make_tensor_from_onnx_node  *
        return cls._run_tf_func(tf_func, inputs, attrs)
    /usr/local/lib/python3.6/dist-packages/onnx_tf/handlers/backend_handler.py:237 _run_tf_func  *
        return tf_func(**kwargs)
    /usr/local/lib/python3.6/dist-packages/tensorflow/python/util/dispatch.py:206 wrapper  **
        return target(*args, **kwargs)
    /usr/local/lib/python3.6/dist-packages/tensorflow/python/ops/math_ops.py:530 multiply
        return gen_math_ops.mul(x, y, name)
    /usr/local/lib/python3.6/dist-packages/tensorflow/python/ops/gen_math_ops.py:6250 mul
        "Mul", x=x, y=y, name=name)
    /usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/op_def_library.py:377 _apply_op_helper
        with g.as_default(), ops.name_scope(name) as scope:
    /usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/ops.py:6542 __enter__
        return self._name_scope.__enter__()
    /usr/lib/python3.6/contextlib.py:81 __enter__
        return next(self.gen)
    /usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/ops.py:4275 name_scope
        raise ValueError("'%s' is not a valid scope name" % name)

    ValueError: 'onnx_tf_prefix_truediv_1;truediv_1/y1_prequant' is not a valid scope name

URL or source code for simple inference testing code
from onnx_tf.backend import prepare
import onnx
import os
import sys
TF_PATH = "./my_tf_model.pb"
ONNX_PATH = "./simplified.onnx"
onnx_model = onnx.load(ONNX_PATH)
tf_rep = prepare(onnx_model)
tf_rep.export_graph(TF_PATH)

Did you read the issue I just suggested? Have you not read it? Have you tried it? Only a partial response is attached. I think you should try to work on it yourself.
lite-model_movenet_singlepose_thunder_tflite_int8_4_float32.onnx.zip


Unable to convert HBD w/ Peakd

When attempting to convert HBD on Peakd it just sits there spinning and waiting.

Fixed on previous release


A social media scraper and analyzer

Pull Request Template
script name - A social media scraper and analyzer
Brief about script
This system is been design in a way that it reads data (text or image) from WhatsApp (currently) and analyzes it whether it has any toxicity in the content which might affect in the group.
If it does have any toxicity then the user who posted the content will be intimated by the bot to delete the message.
Issue no.(must) -
Self Check(Tick After Making pull Request)

[ ] This issue was assigned to me.
[x] One Change in one Pull Request
[x] My file is in proper folder (Name of folder should be in lowercase with no space in between)
[x] I am following clean code and Documentation and my code is well linted with flake8.
[x] I have added README.md and requirements.txt with my script

If issue was not assigned to you Please don't make a PR. It will marked as invalid.

Can anyone please check and evaluate the idea...
Thanks.

@pawangeek Can you please help and evaluate?