Feature/multiple classes

Rebased version of #348.

Thanks for this, @WickyNilliams ❤

No problems! Really appreciate the contribution

@hacknug if you'd like to continue to help out around here, I'm more than happy to give you contributor permissions. No pressure to be super active of course. It is up to you, no worries if not :)

@WickyNilliams don't worry, I've been quite busy for the past weeks and haven't had much time for OSS. I'll open issues and PRs first always to discuss so no need to be able to push directly 👍

I just thought it might be a nice gesture as a thank you, and recognition of the time and effort you've put in. There'd be zero obligations - I wouldn't have any expectation of you being active or doing anything really. But if you felt the need, then you could do so freely. Or close issues etc. I would prevent merge to master anyway so PRs would be necessary still.
But like I say, no worries if not, just wanted to reiterate to be clear on expectations :)

Okay. Let's do this then haha

You should have now received an invite!
I've setup the repo so that reviews on PRs are required. I understand that this prevents pushes to master.
I don't think there's a great deal of granularity for access rights if a repo is owned by a user (versus an org), so you can pretty much do anything. Use the power wisely :)


Upgrade to Dataverse 4.15

Upstream released v4.15. New images needed :smile:
Looks like we need a new Solr image, too, as the schema changed. Release notes should mention reindexing via job. Refer to upstream release docs about username issues.

Notes:

No need to manually replace jhove.conf.  Being part of the dvinstall.zip, the new file is in place on next pod/container start with an updated image.


Export the ViewModel and Component from modlets

This makes it so that both the ViewModel and Component get exported from
modlets. The Component gets exported both as a constructor-case name
like PmoRestaurantList and as the default export. The ViewModel is
exported as ViewModel.

Coverage remained the same at 97.541% when pulling f4f3ce92d0f25024dbf51d52c36a2f0a0dc4fa21 on exp into 75b840595ccd7a67a931e0bfc958159e89508a84 on major.


Possible scrollbar_bg fix

When yellow is selected in the demo, there does not appear to be a yellow-tinted background in the scroll bar. If this is intended, please excuse me. But I thought I'd offer a quick fix in case you wanted it to be like the other colors

No problem, but it was intended to be that way!
Thank you!

My mistake then! Good job on the app!


error:create_execution_context

AttributeError: 'NoneType' object has no attribute 'create_execution_context'

Sorry for late response. This could be due to that you failed to create an engine and thus 'None' was returned

AttributeError: 'NoneType' object has no attribute 'create_execution_context'

Sorry for late response. This could be due to that you failed to create an engine and thus 'None' was returned.

Please check info printed on the screen and see what errors happened when building an engine.
Delete the saved engine file for that it could be a broken engine

I think that I fix this issue. I fork @RizhaoCai 's project. My version is here: https://github.com/zwqnju/PyTorch_ONNX_TensorRT/blob/master/pytorch_onnx_trt.ipynb
I just modifed a few lines.

我认为我已解决此问题。我分叉@RizhaoCai的项目。我的版本在这里：https : //github.com/zwqnju/PyTorch_ONNX_TensorRT/blob/master/pytorch_onnx_trt.ipynb
我刚刚修改了几行。

hello  请问您cudnn的版本是什么？

我认为我已解决此问题。我分叉@RizhaoCai的项目。我的版本在这里：https : //github.com/zwqnju/PyTorch_ONNX_TensorRT/blob/master/pytorch_onnx_trt.ipynb
我刚刚修改了几行。

hello 请问您cudnn的版本是什么？

7.6.4
But I don't think that it matters.

我认为我已解决此问题。我分叉@RizhaoCai的项目。我的版本在这里：https : //github.com/zwqnju/PyTorch_ONNX_TensorRT/blob/master/pytorch_onnx_trt.ipynb
我刚刚修改了几行。

hello 请问您cudnn的版本是什么？

7.6.4
But I don't think that it matters.

tensorrt版本必须是7吗？

我认为我已解决此问题。我分叉@RizhaoCai的项目。我的版本在这里：https : //github.com/zwqnju/PyTorch_ONNX_TensorRT/blob/master/pytorch_onnx_trt.ipynb
我刚刚修改了几行。

hello 请问您cudnn的版本是什么？

7.6.4
But I don't think that it matters.

tensorrt版本必须是7吗？

I used version 7.

Hey @zwqnju  and @longnanxi
Thanks for your discussion.
You should check your created engine. If errors happen when building an engine, a None object will be returned.
It has nothing to do with TensorRT version.

I think that I fix this issue. I fork @RizhaoCai 's project. My version is here: https://github.com/zwqnju/PyTorch_ONNX_TensorRT/blob/master/pytorch_onnx_trt.ipynb
I just modifed a few lines.

Hi,You just comment the module of int8,but the problem with int8 is not solved!

I think that I fix this issue. I fork @RizhaoCai 's project. My version is here: https://github.com/zwqnju/PyTorch_ONNX_TensorRT/blob/master/pytorch_onnx_trt.ipynb
I just modifed a few lines.

Hi,You just comment the module of int8,but the problem with int8 is not solved!

Hello huxianer,
What do you mean? This issue has nothing to do with int8.
Can you provide more information?

Connected to pydev debugger (build 181.5540.34)
Loading ONNX file from path ./models/onnx/model.onnx...
Beginning ONNX file parsing
WARNING: Your ONNX model has been generated with INT64 weights, while TensorRT does not natively support INT64. Attempting to cast down to INT32.
Successfully casted down to INT32.
Completed parsing of ONNX file
Building an engine from file ./models/onnx/model.onnx; this may take a while...
[TensorRT] ERROR: Network must have at least one output
Failed to create the engine
onnx.py", line 440, in onnx2rt
context = engine.create_execution_context()

AttributeError: 'NoneType' object has no attribute 'create_execution_context'
Process finished with exit code 1


[Feature request] Allow directly launching multiple independent instances (particularly for multiple desktops)

Is your feature request related to a problem? Please describe.
I often use multiple desktops, and if you click notepads it will switch desktop to the active one.
Describe the solution you'd like
If launching the exe, create a new instance (rather than switching to the current one)
Describe alternatives you've considered
It's possible to create a new (shadow) instance by creating a new tab and dragging it off the window.

It is already possible to create a new (shadow) instance by creating a "new" tab and dragging it off the window.

I'm aware of that, but when running multiple desktops, running the exe will just bring you back to the currently opened instance instead of launching a new one

Unfortunately, the underline app framework that I am using (UWP) cannot detect which monitor the app window is currently located. This is one of the limitations of UWP.

Ah ok that's a shame.
Is it possible to have a new window open whenever the exe is launched? It could even be an option in the settings in case people prefer new tabs.

https://github.com/JasonStein/Notepads/commit/bb0770f875248a0712883db345c4e66560dcd949

You should be able to see it in v<IP_ADDRESS>

That's fantastic! Thanks a million
Now ready to use this as my daily driver :)

available


chore: Clear Up Info Messages for at_widget packages using  flutter 3.23 beta channel

- What I did
closes #859
- How I did it
Run flutter analyze to see info message.
Make the changes to clear the info message
- How to verify it
Github static analyser should run successfully.
- Description for the changelog

Clear up info messages

@cpswan One of the check failed. This because onPopInvokedWithResult is the replacement method on the flutter 3.23 beta  for onPopInvoked but onPopInvokedWithResult is not available on the stable channel. I'll have to revert to onPopInvoked but this will be the only info message on the beta channel.

OK @CurtlyCritchlow
Can we put an annotation in place to supress that info message, and an issue to remind us to make the change once it lands in stable.
Breaking changes > Generic types in PopScope for reference.

@cpswan I'm about to inbox you about a strategy for publishing the new versions of our packages. I didn't update the dependencies as yet because in some cases it cause conflicts due to other at_widgets packages that are dependencies using older versions and not the newer versions.


Allow vacuum block to be configured with SQLD_BLOCK_VACUUM env var

This would allow libsql-server users to run vacuum locally.

@athoscouto do you plan to rebase this after sarna's comments?

@athoscouto I am closing this because there has been no activity on the pull request for six months and it has tons of conflicts. Please open a new one if you still want to do this.


Number of subjects in "Summary" section can be inaccurate

Using: 343c446 HEAD@{0}: clone: from https://github.com/INCF/bids-validator
Basically, if there are any files in the derivatives folder that contain the characteristic BIDS participant id token, they get picked up and added in with the subject count in the Summary.
Example: Start with this BIDS structure:
CHANGES
README
dataset_description.json
sub-01
sub-02
sub-03
sub-04
sub-05
sub-06
sub-07
sub-08
sub-09
sub-10
sub-12
sub-13
sub-14
task-reversalweatherprediction_bold.json
task-weatherprediction_bold.json

which produces the summary:
        Summary:                Available Tasks:                   Available Modalities:
        147 Files, 1.2GB        Reversal weather prediction        inplaneT2
        13 - Subjects           Weather prediction                 T1w
        1 - Session                                                bold

Then create derivatives, add a file derived_data_for_sub-01.txt
CHANGES
README
dataset_description.json
derivatives
   |- derivatives/derived_data_for_sub-01.txt
sub-01
sub-02
sub-03
sub-04
sub-05
sub-06
sub-07
sub-08
sub-09
sub-10
sub-12
sub-13
sub-14
task-reversalweatherprediction_bold.json
task-weatherprediction_bold.json

which yields the erroneous Summary:
        Summary:                Available Tasks:                   Available Modalities:
        148 Files, 1.2GB        Reversal weather prediction        inplaneT2
        14 - Subjects           Weather prediction                 T1w
        1 - Session                                                bold

Thanks for catching this. I actually started a branch last night to ignore /sourcedata that should also fix this. So far we have been calling any file in /sourcedata, /code or /derivatives a valid BIDS file path but have still been passing them through the rest of the checks. So any data in those could currently throw any of the issues.


Remove Old Elements

Find and Remove old component code that is no longer in use.
Part of this will also be simplifying the folder structure of our modules. Ideally we want to move to something like this:
modules/<module>/
  actions
  reducers
  helpers
  selectors

or modules/<module>/ (a good example of the new structure is modals, for the most part)
  containers
  someComponent.tsx (components just live in the baseline folder, ideally these are fed by containers)
  common.tsx (this includes common elements specific to this code section, see modals as an example)
  <module>.styles.less (styling for all files in this folder)

If there is a situation where it makes sense for a module to have actions/reducers/etc and also containers and components then consider a component folder to keep things a little more organized, but do not make specific folders for each component. an example of what we want to avoid would be something like this:
// we don't want this
modules/<module>/
  components/
    someComponent/
      someComponent.tsx
      someComponent.styles.less
 etc...

To many folders and it doesn't really help organize any better than just having a top level folder name on there, we end up repeating the same text over and over in our imports for no real reason. Don't need to be that verbose.

@JohnDanz Make specific tasks for this if they come up.


start downloading from a certain page

Hi.
I was wondering is there a way to download pictures from a certain page and onwards?

Hello,
It's indeed possible, but the solution depends on what you mean by "onward":

If you mean "from page N to max page", you can simply set the "Page" value in your batch download to the first page you want to download
If you mean "from page N to page 1", you have to put the "Page" value to 1, and the "Images limit" value to "N * images per page". For example, if you want to download page 5 to 1 (so 5 pages) on a website with 20 images per page, you have to set "Page" to 1, "Images per page" to 20 (should be automatic), and "Images limit" to 100 (the number of images on 5 pages)


Laravel 6

Is this package support laravel 6.18? its not working on laravel version 6

This package is not bound to any framework. The only problem might be because of guzzle dependency. You might need to update guzzle package on your project or use v1 which uses older version of guzzle.


Stop build if build-only step fails

To reduce resource consumption in the CI the running steps should be short-circuited if the build-only phase fails.
@wkloucek ?
It usually happens that we create a PR and then we need to provide the changelog. By having this we have a very quick feedback and we avoid keeping a CI job that will never make it to green.

@dpakach can you get someone to have a look?

Or just do it as a separate step and make all others depending on it. It takes only 7 seconds and has no dependencies.
Though one might work on a PR first and see if its green and add the change log before marking the PR as ready. This workflow would then definitely break.


Weird thing I noticed with FPS

On Firefox 62 FPS seem to be stuck at 10. Even when I drag this never changes.
On Chromium it varies wildly and gives me about 60.
By the way, I really like the code. Clean and no unnecessary libraries.

Thanks a lot, happy to see you like the code! Drowning in libraries really make me sad nowadays.
About FPS, its really weird. Which OS do you encounter this behavior? I tried in macOS with a couple of browsers, and their FPS rates similar around 60. I'll check this in windows.
BTW; i've disabled drawing FPS when dragging in progress, because its caused miscalculations.

The OS is debian linux.

Another debian / firefox (95) user here.
On a large screen 2560x1440, I don't have any issues. Performance recorder shows ~45 fps.
There are two possible reasons for a low FPS I believe.
First reason might be that, simply FPS counter embedded to the library might be buggy. FPS counting is an extra overhead and in some cases, like doing it on JS, it might not be as accurate as it should.
Second reason might be that your browser might not be getting the full GPU support that it needs. This may be result of misbehaving GPU drivers or Firefox being too old (if you are still using 62 after 3 years 😅 ).
My primary suggestion would be to check for the performance recorder's stats. Try opening the performance tab recording your actions for a small time. Here is how it looks for me.


ISPN-1704 - IllegalStateException in surviving nodes during node crash in cluster

https://issues.jboss.org/browse/ISPN-1704
The method StateTransferLockImpl.waitForStateTransferToEnd() didn't have any way of signalling
that it failed to re-acquire the state transfer lock.
I've added a new exception, StateTransferLockReacquisitionException, but we'll have to revisit this for 5.2.

@danberindei I had just a doubt on one of the commits, as mentioned above ^
If you're sure about that, let me know I'll merge it.

@danberindei right, thanks. BTW I do highly prefer the visitor as well over ugly getClass and instanceof

merged, thanks


Can I configure from the start to do not cache the images at all?

Check List
Thanks for considering to open an issue. Before you submit your issue, please confirm these boxes are checked.

[✓] I have read the wiki page and cheat sheet, but there is no information I need.
[✓] I have searched in existing issues, but did not find a same one.
[✓] I want to report a problem instead of asking a question. It'd better to use kingfisher tag in Stack Overflow to ask a question.

Issue Description
Can I configure from the start to do not cache the images at all?
I can see the possibility while setting the image to add the option with timer but I'm wondaring if I can do not cache it at all.
What
do not chach an image
Reproduce
[The steps to reproduce this issue. What is the url you were trying to load, where did you put your code, etc.]
Other Comment
[Add anything else here]

Kingfisher now does not provide a direct way to force skipping cache (at last it is a library for image downloading and caching). But I guess you can use the code below to "disable" it:
let cache = KingfisherManager.shared.cache
cache.diskStorage.config.expiration = .expired
cache.memoryStorage.config.expiration = .expired

Or even simpler, just use the ImageDownloader to download the image instead of using the image view extensions or cache:
ImageDownloader.default.downloadImage(with: url) { result in
    switch result {
    case .success(let value):
        imageView.image = value.image
    case .failure(let error):
        print(error)
    }
}


Editing AABB in the inspector causes odd behaviors and freezes

Godot 3.2.2
Godot 3.2.3 rc6
Windows 10 64 bits
When trying to edit the fields of an AABB property in the inspector using the Tab key to cycle through its fields, there is a point where the editor no longer switches to the next fields, or freezes completely.
When that happens, the console is flooded with errors like this:
Failed method: LineEdit:_update_callback target ID: 19800
TOTAL BYTES: 1048560
NULL count: 0
CALL : 43687
CALL show_modal: 1
CALL grab_focus: 1
CALL select_all: 1
ERROR: Message queue out of memory. Try increasing 'memory/limits/message_queue/max_size_kb' in project settings.
   At: core/message_queue.cpp:55

Reproduction project:
AABBInspector.zip

Open main.tscn and select the Spatial node
Click in the x field of the AABB and hit Tab until you get  to d or Godot freezes.
If you got to d, deselect the field, and try 2) again. Godot will eventually freeze.

I suppose there's an infinite loop of call_deferred() going on?

I came across this problem, but I was changing margin values of MarginContainer.
Godot version 3.2.4.rc.mono (from fe4f3ff).

 Try increasing 'memory/limits/message_queue/max_size_kb' in project settings.
 Try increasing 'memory/limits/message_queue/max_size_kb' in project settings.
 Try increasing 'memory/limits/message_queue/max_size_kb' in project settings.
 Try increasing 'memory/limits/message_queue/max_size_kb' in project settings.
At: core\message_queue.cpp:55

Same error can occur in many places in the inspector. I just got it trying to type numeric values in the Transform matrix of a CanvasLayer.


Crash on RTCMediaStream dealloc

Hi, @zevarito ,
I use a UINavigationController as my app root controller, and push a controller A(same as MultiConferenceViewController in your example project)  when click a button on previous controller.  It is ok to present controller A.
The viewDidLoad of controller A

(void) viewDidLoad {
localStream_ = [[ECStream alloc] initLocalStream];
if ([localStream_ hasVideo]) {
RTCVideoTrack *videoTrack = localStream_.mediaStream.videoTracks[0];
[videoTrack addRenderer:self.localView];
}
}

And then we do nothing and click the back button instantly, app crashed on dealloc of controller A. And I found that there is a EXC_BAD_ACCESS  in [RTCMediaStream .cxx_destruct] method.

Any suggestions ？
Thanks

@gluckMu Hi,
Can you update the screenshot link? Image is not displayed, would be helpful to see the full stack of the crash.
It looks like some object got cleaned up early, perhaps you need to look at your retention/release cycles in property definitions and blocks.
Please don't rely on the example app because it is just a quick demo not an app, the purpose of that example is to show some API concepts and how delegates are called, not more.

@zevarito , thanks for your reply. I have solved this problem. I modified the dealloc of ECStream class and the crash doesn't appear again.
- (void)dealloc {   _mediaStream = nil; }
I tested and found that the RTCPeerConnectionFactory object cannot be released before RTCMediaStream. Otherwise, a crash would happen.
I have read the source code of WebRTC, but I am not familiar with C++, so I haven't found more details.

I also get the same crash if not set _localVideoTrack = nil in dealloc (https://github.com/bigbangvn/AppRTCMobile-iOS). And still don't understand why. It seems there's a problem with the scoped_ptr when incorporate with Objective-C code


Allow aliases to rely on other aliases

Hi,
when I want to define aliases dependent on aliases. Those are not resolved:
aliases: {
  BASE-LINE-HEIGHT: "1.5";
  BASE-SPACING: "{!BASE-LINE-HEIGHT} * 1em";
  SMALL-SPACING: "{!BASE-SPACING} / 2";
}

resolves to:
$small-spacing: {!BASE-LINE-HEIGHT} * 1em / 2;

Thank you, @faboweb. This is a known limitation, but feel free to submit a pull request!

@faboweb would love a review of my current PR. Does that meet your needs?

This feature was just added in v5.0.0-beta.3!


[Docs] Add "Open Service Mesh" to navigation menu in Meshery docs

Current State:
Open Service Mesh (OSM) is a service mesh supported by Meshery. A new page documenting the Meshery adapter for OSM has been created - https://meshery.layer5.io/docs/service-meshes/adapters/osm
Desired State:
An entry for this service mesh should be added to the navigation menu.

Contributor Resources

Meshery documentation site
Meshery documentation source
Instructions for contributing to documentation

Hi, can i take this?

Sure you can go ahead with it @badta5te

Can you tell me what the problem is?
Trying to execute
make site

And I get an error:
ERROR: YOUR SITE COULD NOT BE BUILT:
                    ------------------------------------
                    Invalid YAML front matter in /Users/pavelmikhaylov/Documents/code/meshery/docs/pages/functionality/smi-conformance-capability.md

@KarenEfereyan FYI
@JubayerJoy do you mind having a look?

@badta5te, hi I am trying to reproduce this error.  I'll get back to you soon.
Meanwhile, you can make changes and open a PR. After each commit, wait a bit then netlify bot will deploy a version with changes you've made.

@badta5te this bug is causing your issue - https://github.com/layer5io/meshery/issues/1421


Scaling performance as sequences exceeds core count

From #84:

Just as a general heads up - the problem with run into with AICI in production is the case where there are more sequences in batch (and thus parallel controller processes) than cores. This is because I spin for a while on futexes (to minimize latency), and this kills performance when we're out of cores. This would need to be fixed somehow. The latency minimization was mostly there when we still had post/pre_process(); for mid_process() it shouldn't matter that much.

Originally posted by @mmoskal in https://github.com/microsoft/aici/issues/84#issuecomment-2342293184

I wonder if the streaming protocol of WASI helps here - instead of using a futex using IPC with efficient reading and writing to shared circular buffers?

I guess regardless how wasm code communicates with host, the host has to organize the threads/processes for each sequence. In AICI this is done with separate processes which means you can kill the process and limit execution time without overhead (if you put a time limit in wasmtime it seems to have significant overhead; I don't remember exactly but I recall 30% or so in the inner bias computation loop).
The processes can also fork, though how needed that is for AICI functionality is another question.
Threads would be potentially easier to synchronize (though one has to be careful with things like rayon - they seem to have issues when you have more than 50 cores or so).

I did some cursory research into the libraries available for cross-process IPC in Rust, and, well, essentially all of them add some significant degree of complexity to the implementation or aren't as latency optimized (e.g.: using eventfd with the ring buffer to signal).
What considerations went into using aicirt as a Rust binary as opposed to a library? I can guess one of those would be integration with vLLM.
I ask because I'm wondering if might make more sense to export a library interface to it to allow the host to decide how to handle concurrency, and e.g.: to oxidize it as a Python library for integration with vLLM?

Indeed, exposing aicirt as a library might be better from python standpoint. However, there is still internal concurrency within aicirt - that is running multiple sequence controllers in parallel. I think you don't want to expose that to Python, as it would have performance implications. Either way, that can be handled as processes (as done now) or processes.


Attribute That Returns Array Fills Model When Not Expected

Laravel Version: v19.17.0
Database Driver & Version: Mysql

Description:
The Steps column is JSON / Array. In the Attribute, if the Setter returns an array the keys are used as fields on the model. It's expected it should fill the column with the array data. If you take off the ->toArray() it saves it fine, but this seems like a bug.
protected function steps(): Attribute
{
    return Attribute::make(
        get: function ($value) {
                $value = Collection::wrap(json_decode($value));
            return Step::query()->whereIn('id', $value)->get();
        },
        set: fn ($value) => $value->pluck('id')->unique()->toArray(),
    );
}

$steps = Step::where(...)->get();
$history = History::first();

$history->steps->push(... $steps);

return $history;

App\Models\History{
  steps: "[]",
  0: "<id>",
  1: "<id2>",
  2: "<id3>",
}

Actually this is documented behavior:

Sometimes your mutator may need to set multiple attributes on the underlying model. To do so, you may return an array from the set closure.

https://laravel.com/docs/9.x/eloquent-mutators#mutating-multiple-attributes
So as you returning an array Eloquent assumes you want to set multiple columns, and as the returned array's are numeric this fails.
This works without the ->toArray() call, as you are returning a Collection  instance instead.
If you want, or need in another mutator, to return an array, you can return an array with the syntax `['column => $value]', which in your case would look something like this:
public function steps(): Attribute
{
    return Attribute::make(
        get: function ($value) {
            $modelKeys = Arr::wrap(json_decode($value ?? '[]'));
            return Step::query()->findMany($modelKeys);
        },
        set: fn ($collection) => [
            'steps' => $collection->pluck('id')->unique()->all(),
        ],
    );
}

Yes, I understand you'd want to set multiple attributes, as I do that in other cases. But I think the difference with this is the return value has numeric keys and array_is_list. Seems weird you'd have model fields as 0,1,2,3,...

I don´t actually see this as an issue, as it is now, but if you think this behavior should be changed, you can try to propose a fix.
The method you will need to change is this one:
https://github.com/laravel/framework/blob/2bc180295ebf8c4848a6b9c4ae8fa0d00ec75217/src/Illuminate/Database/Eloquent/Concerns/HasAttributes.php#L1658-L1668
This method checks if the value is already an array, assuming the developer already sent in the [$key => $value] format, and if it is not and array it converts the attribute to the [$key => $value] format.


Cannot find module 'fs-extra' error

I installed the tool and created a project with tool. When I tried to run it with npm run start, it showed error as: Error: Cannot find module 'fs-extra' but I already have the module installed.

How do I fix the error?

npm run build:css
lcpkg export --filter runtime --arch x64 --mode debug app
lcui build
lcui run

The npm install command will install the fs-extra module. Have you run it in the my-project directory?

Please try to use the following command instead of npm run start:
npm run build:css
lcpkg export --filter runtime --arch x64 --mode debug app
lcui build
lcui run

Error:

npm install
npm run start

npm install
npm run start

Error:

node-gyp uses Python 2. please install Python 2 and try again.

Or try to specify the binary download address to skip building node-sass.
npm install --sass-binary-site=http://npm.taobao.org/mirrors/node-sass

Errors are fixed now. I close the issue.


Fix critical error when space in bin path

The bin-version-check module delegates version extraction to the bin-version module: https://github.com/sindresorhus/bin-version-check/blob/master/index.js#L16
The bin-version module extracts the bin version with childProcess.exec(bin + ' --version', ...): https://github.com/sindresorhus/bin-version/blob/master/index.js#L6
childProcess.exec expects its first argument to be "The command to run, with space-separated arguments": https://nodejs.org/api/child_process.html#child_process_child_process_exec_command_options_callback
Consequently, before my fix, if a space is present in the path to the bin, everything after the first space is interpreted as arguments. And the bin correspond to the first part of the path (before the first space), which is NOT ok.

@marob Can you submit this PR to https://github.com/sindresorhus/bin-version instead? I think it's more proper way to fix this problem.
cc: @sindresorhus

Here is the corresponding PR: https://github.com/sindresorhus/bin-version/pull/6
I'm not convinced it should be fixed in bin-version, but I'll let you and @sindresorhus decide.

@marob Thanks.

Yeah, should be fixed in bin-version. Never thought of the use-case that someone would supply an absolute path.

OK, I've updated my PR on bin-version to try not to break any module while fixing the issue.
Meanwhile, I'm keeping this PR (on bin-wrapper) if fixing the issue in bin-version proves to be impossible.

This was fixed in bin-version.


tmux module disable nested remote sessions

I am running tmux locally or on a server via ssh.
I connect from one server to another and I don't want tmux to autostart when connected from another tmux session.
Is there a way to disabling the autostart in this situation?

Looking at the code for loading tmux, there is no straight forward way to fix this issue.
Not an elegant solution but I would probably use an environment variable (LC_*, as most ssh servers accept such env vars) in tmux.conf and check for it on the server before starting tmux server.
e.g., On client side, add the following line to your tmux configuration
$ vim ~/.tmux.conf

setenv -g LC_TMUX 1

And on server side, check for [ -z "$LC_TMUX" ].

Thanks for your input.
If you connect from a tmux session through ssh this environment variable will not be present on the remote host. To disallow nested sessions locally the $TMUX variable is checked already.
How would you expose/forward this variable to the remote host?

Uh, I missed out on the last detail.
You need to add SendEnv in your ssh/config file, on per host or for all.
For sending this env var to all hosts, edit ~/.ssh/config,
$ vim ~/.ssh/config

Host *
  SendEnv LC_TMUX

I see. Thank you.
I would then do that with $TMUX, because this variable is already checked on tmux start..

If you wish to use "$TMUX", then you would have to allow it on all your ssh servers.
$ vim /etc/ssh/sshd_config
...
AcceptEnv LANG LC_* TMUX
...

good to know, thank you.
the module/tmux/init.sh could instead be extended to check for $LC_TMUX


[BUG] Keen UI logo missing

Describe the bug
Keen UI on the Vue Libraries is missing a logo: https://vue.framework.dev/categories/libraries

We can use the logo from https://madewithvuejs.com/keen-ui

Hi @dustinsgoodman,
Please assign this issue to me,
cheers


blockbench-electron: 4.5.2 -> 4.8.1

Description of changes
Releases
Electron was updated to version 25.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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/)
23.11 Release Notes (or backporting 23.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.

currently burned out on nix! hope you can find a reviewer :purple_heart:

:+1: it's all good, take your time!

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2589

Result of nixpkgs-review pr 242557 run on x86_64-linux 1

1 package built:

blockbench-electron

thank you!


Investigate long GPU semaphore wait time

The screenshot above shows an nsys trace of a task during NDS-H Q21 execution. As seen, this task was blocked on the GPU semaphore for most of its execution time. The NVTX Push/Pop Range Summary shows that this pattern is found not only for this particular task, but for all tasks in the same time range in this node. Here is the settings I used for my testing.

8 workers in the cluster with 1 executor each.
spark.rapids.sql.concurrentGpuTasks was set to 4.
spark.executor.cores was set to 16.

With these settings, there were up to 16 tasks trying to acquire the semaphore at the same time. Given that the semaphore size was set to 4, the wait time in the above screenshot seems suspiciously long. It seems worth looking into this and see if we can avoid or work around it.

It might be a little long, but it is not an order of magnitude off. The hash join gather map range a little below the Acquire GPU range in the table is 2.1% of the time (96.5 s), while Acquire GPU is 11.8% (530.5 s) We have a ratio of 1 task on the GPU to 4 tasks total, So I would expect the wait time for the GPU to be about 3x to 4x the computation time on the GPU. That gives us 133 to 177 seconds of computation on the GPU. That means the join gather accounts for 50% to 70% (back of the envelope calculation) of what we expect. I agree that looking into it would be good, but I don't see any really large red flags in the screenshot.

@revans2 thanks for the input! Your analysis sounds reasonable. Though, there is one particular thing that is not described above is that, per nsys trace, it seems like we are holding the semaphore longer than we need while iterating inputs. I did not mention it originally as I need to gather more information about it. Perhaps this is not a big deal as you described, but I would like to double check it. I will post my finds here. Thanks!


Yearly repeating events raise an error

Consider this fragment:
RRULE:FREQ=YEARLY;INTERVAL=1;UNTIL=20190328
DTEND;VALUE=DATE:20170330
DTSTART;VALUE=DATE:20170329

When icalevents loads this event, it will normalize the date into a datetime. But when dateutil.rrule is called subsequently, it will object:
RRULE UNTIL values must be specified in UTC when DTSTART is timezone-aware

Unfortunately, there are several places where start and end times are compared and the possibility of a mixture of date and datetime objects isn't handled.

I got this too and fixed it by adding few lines after event.all_day and before   try: event.location in the file icalparser.py:
event.all_day = type(component.get('dtstart').dt) is date

`
if component.get('rrule'):

    event.recurring = True
    if( component.get('rrule').get('until') ):
        iIndexListUntil = 0
        for x in component.get('rrule').get('until'):
            currentUntil = component.get('rrule').get('until')[iIndexListUntil]
            if not hasattr(currentUntil, 'tzinfo'):
                component.get('rrule').get('until')[iIndexListUntil] = normalize( currentUntil, tz=tz )
            iIndexListUntil + 1

`
try:
event.location = str(component.get('location'))
except UnicodeEncodeError as e:
event.location = str(component.get('location').encode('utf-8'))
So it adds timezone informations if until does not have one

Weekly would occur the same also
DTSTART;VALUE=DATE:20190706
DTEND;VALUE=DATE:20190713
RRULE:FREQ=WEEKLY;INTERVAL=5;BYDAY=SA

I haven't looked deeply yet but just let me take a memo.

This still seems to be an issue even on 0.1.25:
With this rule string:
FREQ=WEEKLY;UNTIL=20200627;BYDAY=MO,WE;WKST=SU

I ended up using:
        def fix_rule(rule):
            if 'UNTIL' in rule:
                rule['UNTIL'] = [normalize(r, tz=tz) for r in rule['UNTIL']]
            return rule

        # If dtstart is a datetime, make sure it's in a timezone.
        rdtstart = component['dtstart'].dt
        if type(rdtstart) is datetime:
            rdtstart = normalize(rdtstart, tz=tz)
            rrules = [fix_rule(x) for x in rrules]```


"NightVisionBrightness" setting works only when player is outside the facility

I have set "NightVisionBrightness" to 10, and this setting only take affect when player is outside the facility, after player enter the facility, the body cam will show the actual vision.

Is the issue that night vision is too bright in the interior, and do you have CullFactory? If so, it's likely a conflict with CullFactory that's causing your issue, since its method for hiding lights conflicts with OpenBodyCams currently. I have a fix for this in OpenBodyCams locally, and I'll include it in the next release.

Yes I have CullFactory installed, since there is not much mods that can increase the framerate without make game looks like potato.
After uninstall CullFactory it works now, thanks for letting me know!
Screenshot:

Should be fixed in 2.0.0!


Release/v0.2.8: Upgrade go v1.23 & goreleaser

Goreleaser is upgraded including dependent gh actions


Update dependencies

This PR has been merged into v-1.7.x-alpha. Thanks for your contribution!

This PR has been merged into v-1.7.x-alpha. Thanks for your contribution!


__callStatic via trait not absorbed

when a class or enum (in this specific case, an enum) adds __callStatic, all various calls are no longer flagged as invalid calls. this is good.
however, when this class or enum adds __callStatic via a trait, it does not learn this.
this code works as expected and none of my calls to PostContentType::HTML() are underlined as undefined.
enum PostContentType:
string {

	static public function
	__CallStatic(string $Name, array $Argv):
	string {

		return constant("static::$Name")->value;
	}

	case HTML = 'text/html';
	case Text = 'text/plain';

}

this code however, all calls to PostContentType::HTML() are underlined as undefined.
enum PostContentType:
string {

	use Toaster\Packages\EnumCallableValue;

	case HTML = 'text/html';
	case Text = 'text/plain';

}

disregard, i noticed that "indexing" was on in the statusbar... so i let it sit for a while longer. it never went away ever. reloaded the window. it reindexed for like 4 seconds and that was the end of it.


Any plans for GPU.js support?

Hey got a question are there any plans for supporting GPU.js in Babylon.js?
Just curious. :)

HeadClot

Hello not for now.
The math operation we have are pretty short and we are not sure the overhead of creating a GPU shader for them will worth it

Alright thanks for the insight. :)


using pip install -u when downloading packages

I was originally going to do this but then I didnt because using -u will mess up if the user is using a virtual_env
Originally posted by @filip-halt in https://github.com/towhee-io/towhee/pull/1269#discussion_r878576884

When installing packages with pip, we should consider --user option if the user is using a system python, but we should install the package without --user option if the user is using a virtual env.
We need to discuss the detail behavior of package install. @GuoRentong @junjiejiangjjj @fzliu @filip-halt

When installing packages with pip, we should consider --user option if the user is using a system python, but we should install the package without --user option if the user is using a virtual env.
We need to discuss the detail behavior of package install. @GuoRentong @junjiejiangjjj @fzliu @filip-halt

we can provide such control flexibility via environment variables

When installing packages with pip, we should consider --user option if the user is using a system python, but we should install the package without --user option if the user is using a virtual env.
We need to discuss the detail behavior of package install. @GuoRentong @junjiejiangjjj @fzliu @filip-halt

we can provide such control flexibility via environment variables

does the user have to restart ipython kernel when he is using jupyter notebook to set the environment variables?


一天，甚至几小时就被封端口

请问大家有什么解决办法了吗？谢谢

Thanks, but there is no method to input a port range in android device.

https://github.com/segfault-bilibili/ShadowsocksGostPlugin  maybe you can try this gost plugin on Android.
gost support kcp  and is safe for now.
server version: https://github.com/ginuerzh/gost

https://github.com/segfault-bilibili/ShadowsocksGostPlugin maybe you can try this gost plugin on Android. gost support kcp and is safe for now. server version: https://github.com/ginuerzh/gost

Many thanks. I have downloaded the plugin and sucessfully installed. But I don't know how config it at Android and server.
take my need as example:

SS is working with server 333.333.333.333 port 33333
KCPTUN is working at port 33334 and accelerating port 33333
If I installed gost at server, should I stop the KCPTUN? If I want to use the kcptun config for gost, the server side config looks like__gost -L ss+kcp://AES-256-GCM:PASSWORD@33333?c=/path/to/conf/file__ right?
What the android plugin config looks like? same ? gost -L ss+kcp://AES-256-GCM:PASSWORD@33333_
what to input in SS for remote part/password/encrypt method? leave password blank and encrypt none?

双方已经开了1000个端口....
可惜出门SS只能设置一个端口

https://github.com/segfault-bilibili/ShadowsocksGostPlugin maybe you can try this gost plugin on Android. gost support kcp and is safe for now. server version: https://github.com/ginuerzh/gost

from now on，gost‘s kcp have been blocked out

https://github.com/segfault-bilibili/ShadowsocksGostPlugin maybe you can try this gost plugin on Android. gost support kcp and is safe for now. server version: https://github.com/ginuerzh/gost

from now on，gost‘s kcp have been blocked out

I don't think gost's kcp is blocked.  If you think so, use this mod for kcptun.
https://github.com/woodlyer/kcptunMod

现在，kcptun 无法使用。 总是被封锁。

本发明已经部分部署在边境监控审查系统平台中，具有对大量数据的分析能力，且检全率和准确率均在90％以上。
一种KCP协议加密流量检测方法.pdf

本发明已经部分部署在边境监控审查系统平台中，具有对大量数据的分析能力，且检全率和准确率均在90％以上。
一种KCP协议加密流量检测方法.pdf

他们方法不太对，加密数据是无法按照字节推断的。一定是从流量特征入手，加上AI大数据模型。

我想也没啥流量特征吧？流量比较大？

流量特征与流量大小没什么关系。


Wrong newline handling on simple terminals (compiler output is stuck at the end of previous line)

This issue applies only to fallback console output mode, used for unrecognized terminal models. Rich mode with interactive animated progress bars does not suffer from this issue.
Expected behavior: Gradle should emit newlines at the end of line whenever it prints to console.
Observed behavior: newlines are emitted at the beginning of each logged line, which causes output from external sources to stick onto the same line:

:app:generateDebugSources
:compiler:compileJavawarning: [options] bootstrap class path not set in conjunction with -source 1.7

:compiler:processResources NO-SOURCE
:compiler:classes
:compiler:jar
:app:javaPreCompileDebug
:app:compileDebugJavaWithJavacNote:
Note: APT plugin says: Started new scan at 2018-02-09
Note: APT Linker says: Finished scan after 1929ms
Note: APT Linker says: Empty working set!

Note the bolded text, — it clearly should not be there.
Context
This issue is reproducible in IntelliJ Idea terminal, Travis CI logs and probably dozen of other places. It obfuscates Gradle output, making it harder to read, and ultimately harder to debug.
Steps to Reproduce (for bugs)
Try to compile any java project, that produces warnings.
Your Environment
Gradle 4.5.1
Groovy 2.4.12
Arch Linux

I ran into this, too. For now, this works around it for me:
compileJava {
    doFirst {
        logger.lifecycle('')
    }
}

And there's probably a way to apply it to all compile tasks.

@chrislong This is a nice attempt, but it does not really solve the underlying problem.
I suspect, that the reason, why this odd behavior was introduced is desire to guarantee a line break before each Gradle's line, while avoiding excessive line breaks. Putting a line break at the end of line can lead to ugly output when output from multiple parallel sources interleaves.
Here is my take: replace entire stdout of Gradle process with pipe and have a Thread process the other side of pipe to strip excessive line breaks. This way Gradle should be able emit line breaks on both sides of line without worrying about wasting screen space.


Chatterino Firefox Extension - Size/scale

If your PC's Display Settings has the scale set to anything other than 100%, the browser chat doesn't take up the whole space, not accounting for the scale setting.

Fixed


add reference to SNOBOL4

What does this PR do?
Add Resource(s) | Remove Resource(s) | Add info | Improve Repo
For resources
Description
Why is this valuable (or not)
How do we know it's really free?
For book lists, is it a book?
Checklist:

[ ] Not a duplicate
[ ] Included author(s) if appropriate
[ ] Lists are in alphabetical order
[ ] Needed indications added (PDF, access notes, under construction)

Yes, I once wrote a SNOBOL program!
But no, this doesn't seem to be a book, it's a web page, and half of the links are dead.
It's a melting SNOBOL page. :disappointed: : snowman:

I wasn't sure of the right format (and should have researched) -- but there are PDF versions aplenty at that page, with fine young vintages like 1987.
The one which stood out for me was near the top at ftp://ftp.snobol4.com/spitman.pdf - which was last updated in September, 2000 and allegedly runs on WindowsNT.
The book listed at http://www.math.bas.bg/bantchev/place/snobol/gpp-2ed.pdf is from 2005, and despite it's dated format, contains code which still works on the versions of snobol4 which can be installed on a Mac OSX 10.15.4 using brew install snobol4
Now, I'm not going to go on record saying that SNOBOL4 is my language of choice... but to lose this language in the dust of time when other languages made the cut on this list such as APL, which has a much harder learning curve, seems a sin.  (For the record, the entry for APL also links to a web page that seems to be cut of the same cloth as the SNOBOL4 one).
Many thanks for your review of my PR, Eric, and if you don't want to include it, I will peacefully withdraw.  :-)

Would love to see some of the still extant resource added to a new SNOBOL4 section. For some of the older ones, the versions at internet archive may provide more stable (less melty!) links.
APL!


Add a Projects section to the manual #17

How is this for a humble beginning?

LGTM
Thanks again @g-gundam

Closes #17


ASP.NET Core support

Hello,
Is there any plan to support ASP.NET Core? I am asking this for hosting QBit.Ninja inside a docker container  which could be run on Linux: https://github.com/MetacoSA/QBitNinja/issues/16 .
Thank you for your help

hey, I am not against if you can do it and submit a PR. It should be doable, but I won't do it myself.
I have been working on a lightweight block explorer which work on ASP.NET Core recently: https://github.com/dgarage/NBxplorer
Features:

Can track HDPubKey
Index only the transactions related to those HDPubKey
Do not need to sync the whole chain
Soon, SignalR support for transaction/block notification

Disadvantage:

You can't query an arbitrary transaction and arbitrary address.

Depending on your use case, it is worth considering.
Nicolas,

I would be happy if I can contribute a useful code. I think a document explaining internals of indexer and QbitNinja would also be a great help to whoever wants to contribute.
Thank you for your enormous contributions to .Net Bitcoin and also providing timely support and updates.

You can take a look at this article:
https://www.codeproject.com/Articles/819567/NBitcoin-Indexer-A-scalable-and-fault-tolerant-blo
The only difference is that NBitcoin.Indexer now use P2P connection to a trusted node instead of block directory. QBitNinja is only a API which query those tables.

Basically there is two part: The indexer, and the Web API part.
You should start the API part imho, this is the simplest. The big question is whether the azure storage library is good enough in .NET Core.
Nicolas,

Thanks for update.
About the path for implementing .Net Core support; I think the proper approach would be to first port the NBitcoin.Indexer to .Net standard, so in future, we have a common code base between .net core and legacy .net which could be managed easily; By using
https://marketplace.visualstudio.com/items?itemName=ConnieYau.NETPortabilityAnalyzer it seems indexer has little problem in this path.
About internal documentation, if it is appropriate, it is probably also helpful to have more details about things not much covered in the provided link, for example the details of tables: balances, callbacks, chain, chaincache, crudtable, transactions, wallets in Azure.

If you can port. This does not matter, just a problem of referencing another package, changing the csproj format, and run again.

Reopened as I think it should be done

Some progress: this project is now using the new csproj system


Issue Title: "Enhance 'Contact Us' Page"

@Kritika30032002
Issue Description: I would like to enhance the "Contact Us" page by improving the layout, adding a more user-friendly contact form, updating the design, and making it more interactive. This will make it easier for users to reach out and improve overall user experience.
Please assign this issue to me under the labels: level2, gssoc-ext, and hacktoberfest-accepted.

I would to like to work on this project. Please assign it to me.

Please assign this task to me.


1.19 PLS

I need LightAPI for 1.19 ....
It's going to be 1 month since you said "soon" I can't work without the pl :/

And I've been waiting for 12 days :/
While all the pl who need light api have gone to 1.19 :/

Patience... it will arrive eventually.

Please 1.19 API need ItemsAdder

dev isnt active, someone needs to fork it and finish this job

first time looking plugin source so a little help would be nice

someone knows whats going on

first time looking plugin source so a little help would be nice  someone knows whats going on update: the plugin it self is broken??? there files missing

LightAPI is not working with 1.19
i doubt that it ever will work, the dev is inactive.

first time looking plugin source so a little help would be nice  someone knows whats going on update: the plugin it self is broken??? there files missing

LightAPI is not working with 1.19 i doubt that it ever will work, the dev is inactive.

i know im asking the community, i was trying to fix it but plugin knowledge is 0

+1 please update 1.19

The package is in, so yall can stop complaining now.
If you want I'll fork it and add actions so you can use it now.

Thanks Eric :)

The package is in, so yall can stop complaining now. If you want I'll fork it and add actions so you can use it now.

That would be great!
I just looked at your fork, and it seems like the build failed :(
Could you please fix it so that we are able to download it from your fork?

https://www.spigotmc.org/resources/lightapi.4510/update?update=465301

Issue closed: LightAPI is no longer being maintained. Use LightBlock instead


Added AirPressure Python interface

Signed-off-by: ahcorde<EMAIL_ADDRESS>🎉 New feature
This PR is part of this mete ticket https://github.com/ignitionrobotics/sdformat/issues/931
Require

https://github.com/ignitionrobotics/sdformat/pull/969

Summary
Added AirPressure Python interface
Test it
from sdformat import AirPressure

airPressure = AirPressure()

Checklist

[x] Signed all commits for DCO
[x] Added tests
[ ] Added example and/or tutorial
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[x] codecheck passed (See contributing)
[x] All tests passed (See test coverage)
[x] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

Codecov Report

Merging #971 (d00a861) into main (a3560f3) will not change coverage.
The diff coverage is n/a.

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

@@           Coverage Diff           @@
##             main     #971   +/-   ##
=======================================
  Coverage   66.66%   66.66%
=======================================
  Files           2        2
  Lines          27       27
=======================================
  Hits           18       18
  Misses          9        9

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 a3560f3...bec7dc7. Read the comment docs.


FEATURE: allow resemble.compare even if dimensions are not the same

The documentation says:
Note: seeVisualDiff and seeVisualDiffElement work only when the dimensions of the
screenshot as well as the base image are same so as to avoid unexpected results.

which is found in the code https://github.com/codeceptjs/codeceptjs-resemblehelper/blob/3d9974df5db59206bfd2f322a5fb926ee652071d/index.js#L79-L83
However the underlying resemble.js works just fine in calculating the diff percentage and tolerance and taking that into account.
I propose to make this check optional and thus introduce an option/parameter to skip that check
Background:
We use visual regression testing for webpages and as we allow some tolerance for minor layout changes. But this could also make a full page screenshot some few pixel longer.
This should be able to fail the test (as is the state now), but in fact only with 0 tolerance. However if you have a tolerance defined, tests should be able to run fine nevertheless if the change is within the tolerance.

Thanks a lot, I think it should fine to add this check as an optional parameter, which is defaulted to true.
@jpmschuler could you please have a PR for this request, if you have time. It would be great help. I will make a new release.

Thank you, fixed with https://github.com/codeceptjs/codeceptjs-resemblehelper/releases/tag/v1.9.5


Thickness consistency

The model prints a warning if the inconsistency between the summed layer thicknesses and the depth of the domain is above a certain threshold. This pull request adds the functionality for the user to specify the threshold. The default value is set at 1%.
This was discussed in #14 .

I assume you meant for me to review this.  Looks good.

Yeah. I figured it would be better branch discipline if I put these changes as pull requests rather than simply pushing them into the master branch. Plus, this way you might catch anything silly I've done.


Add bitbucket auto approve pull request feature

I'm using bitbucket and pronto to auto check style violations in pull request, but there are no way to know if a pull request is checked or not. So I added an approve option to let us know which pull request is checked.
Just set auto_approve: true

And here is the result

@mmozuras yes, it is a signal to let us know that pronto has been run.
The check will run again and take off approval if there are new messages.

@starchow 👍


Lightmap example.

I used basic_light example by Chris Camacho (@codifies) as a base. Not sure if he should be mentioned in the contributors , though I'm not sure how much of his code there still is. Also copied some textures since there was no suitable ones in the shader resources.

@nullstare Thank you very much! It's a very nice example! I'm just reviewing the TABS to be 4 spaces but I can do it! :)


128903155: Unit test failure: Service instance test failure

Thank you for finding the issues. I will rework what you have found to align better with the current model. We are trying to avoid "done". I am not convinced about the require check which I will also verify.


Add CBMC Stub Library and queue proof infrastructure

CBMC Stub Library
Description
While working on the proofs for linear containers,
I figured out that there are a couple of stubs and function models I have to reuse in different harnesses. Therefore, I started a CBMC Stub library for keeping a single model for each of these stubs.
This library is still work in progress but I appreciate some early feedback on my assumptions.
Checklist:

[ ] I have tested my changes. No regression in existing tests.
[ ] My code is Linted.

Dropped pvPortMalloc and vPortFree from this PR in favor or  #774.

@dcgaws @dan4thewin this PR contains a bunch of utility functions that other proofs call into, to initialize various data structures. I've had a look and the assumptions seem sane to me, but I'd appreciate it if you would confirm that---thank you.


[#3149] Allow queries with a local handler to shortcut to the local segment

Resolves #3149

I have added a simple bypass at the moment only for one method: query. I am not sure if this approach is ok.
I will be grateful for any guidance

@rsobies, I see you keep working on this PR, for which many thanks! :pray:
But, my reason for commenting is to let you know that you should feel free to nudge me directly once you hit a point where you want me/us to review your changes again.

@smcvb @abuijze please, take a look on my latest changes

I am on vacation, i wil be back in a week

Although a bummer for us, it's good for you you're on vacation, of course.
I'll go ahead and merge the PR, fixing the build issue locally.

@smcvb can you tell me what was the root of this problem?

@smcvb can you tell me what was the root of this problem?

Axon Framework is still build in JDK8, which isn't aware of the List.of(...) operation you used in the test you added. So, in commit 30bec628b5ef44dba08420dbe279698c9796fd7f, I adjusted that behavior.


Document how to create nodes or port them from node-red

I would be willing to contribute with additional nodes, but right now there is no documentation about how to create new nodes or port them from node-red.
Can you give a few guidelines?
Thank you so much for your work!

I noticed the repo "https://github.com/iceddev/pagenodes-extras" to create additional nodes. Might be helpful.

@mlopezr , sorry been a bit bogged down.   @rsrini7 is on the right track.  We also added https://github.com/iceddev/pagenodes-nodes  for the included nodes.  I'll try and get some docs on all this up soon.

That would really helpful. I also request you to change the title of the README.txt, It says nodejs, babel and new inputs, which is quite misleading.
thanks.


Bump cosmos to latest version

High-level description
What features does this change enable? What bugs does this change fix?
Corresponding DC/OS tickets (required)

DCOS-54077 cosmos: fetching enterprise cli fails with 400 response
DCOS-44111 Cosmos behaviour changed from 1.11 to 1.12 when running describe

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Change log from the last version integrated (this should be a link to commits for easy verification and review): 04b70bf9d2...9afd051e46
[x] Included a test which will fail if code is reverted but test is not. If there is no test please explain. No such tests are included.
[x] Test Results: link to CI job test results for component
[x] Code Coverage (if available): link to code coverage report

@mesosphere-mergebot label ready for review

@mesosphere-mergebot request-review @alenkacz @kaiwalyajoshi @kvish

@mesosphere-mergebot changelog-not-required no new features were added - only minor bug fixes.


Disable npm audit.

Pull Request
🤨 Rationale

#1317

👩💻 Implementation
Just commenting out npm audit part of pipeline.
🧪 Testing
N/A
✅ Checklist

[ ] I have updated the project documentation to reflect my changes or determined no changes are needed.

@atmgrifter00 jesse beat ya to it


Provide JDK 18

Description:
For Oracle OpenJDK 18 has been released. https://openjdk.java.net/projects/jdk/18/
For AdoptOpenJDK, I think it is related to the Temurin and so to this ticket https://github.com/adoptium/adoptium/issues/130
For Zulu, seems it is released https://docs.azul.com/core/zulu-openjdk/release-notes/18-ga
For Liberica, seems available https://bell-sw.com/pages/downloads/#/java-18-current
For Microsoft, build not available currently on https://docs.microsoft.com/en-us/java/openjdk/download but might never be available as itis nto a LTS. based on https://docs.microsoft.com/en-us/java/openjdk/faq

We plan to release OpenJDK binaries for versions where there is mutual LTS consensus across OpenJDK contributors and vendors.

Justification:
JDK 18 is the latest release of Java.
Are you willing to submit a PR?
maybe for AdoptOpenJDK/Temurin when available if it is a low effort

Hello @apupier. Thank you for your proposal. The action uses API to grab the latest versions of distributions. I can confirm that the Java 18 for Ubuntu can be installed for Liberica, Zulu and Temurin. Unfortunately, for Microsoft the action does not have an API to grab all available versions.

Would be awesome, my builds currently fail, as I switched a project already to Java 18.

Hello @JohannesLichtenberger. What distribution and os are you using?

Ubuntu / Temurin, but actually I don't care which distribution :)
The workflow is this:
https://github.com/sirixdb/sirix/blob/master/.github/workflows/gradle.yml

I think it's not an issue with setup-java itself, because the version setup step worked correctly.

I am having an issue were it is recognising java adopt 18.

I am having an issue were it is recognising java adopt 18.
Could not find satisfied version for SemVer '18'.
This is what I have
 - name: Set up JDK 18
        uses<EMAIL_ADDRESS>        with:
          java-version: '18'
          distribution: 'adopt'

I had the same problem and switching over to temurin did the trick:
        uses: actions/setup-java@v3
        with:
          distribution: 'temurin'
          java-version: '18'

I am having an issue were it is recognising java adopt 18.
Could not find satisfied version for SemVer '18'.
This is what I have
 - name: Set up JDK 18
        uses<EMAIL_ADDRESS>        with:
          java-version: '18'
          distribution: 'adopt'

I had the same problem and switching over to temurin did the trick:
        uses: actions/setup-java@v3
        with:
          distribution: 'temurin'
          java-version: '18'

Thank you very much.

Hello everyone. Java 18 is available for AdoptOpenJDK (Temurin), Zulu and Liberica. The setup-java action does not provide Oracle. We have a separate issue with add it https://github.com/actions/setup-java/issues/69. For Microsoft builds are not available for now.
For now I'm going to close the issue.  If you have any concerns feel free to ping us.


Remove version: OSGeo.GeoServer version 2.24.2

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://sourceforge.net/projects/geoserver/files/GeoServer/2.24.2/GeoServer-2.24.2-winsetup.exe/download
OK
(Automated message - build 923)

URL: https://sourceforge.net/projects/geoserver/files/GeoServer/2.24.2/GeoServer-2.24.2-winsetup.exe/download
OK
(Automated message - build 923)

Close with reason: Package still available;


Using type alias causes error where directly using the type doesn't

It is surprising (at least to me) that in the following pkl definition:
typealias Step = GetStep | PutStep

class Job {
  steps: Listing<Step>?
}

class GetStep {
  prop1: Step?
}

class PutStep {
  prop1: Step?
}

An error is thrown for prop1 on the PutStep saying:
–– Pkl Error ––
Type alias definitions must not be cyclic.

8 | prop1: Step?
           ^^^^
at foo#GetStep (file:///Users/jonathan.harden/test/pkl/foo.pkl, line 8)

1 | typealias Step = GetStep | PutStep
                     ^^^^^^^
at foo#Step (file:///Users/jonathan.harden/test/pkl/foo.pkl, line 1)

4 | steps: Listing<Step>?
                   ^^^^
at foo#Job (file:///Users/jonathan.harden/test/pkl/foo.pkl, line 4)

106 | text = renderer.renderDocument(value)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
at pkl.base#Module.output.text (https://github.com/apple/pkl/blob/0.25.2/stdlib/base.pkl#L106)

However if instead of the typealias, I use the value of the typealias directly it is fine:
class Job {
  steps: Listing<GetStep | PutStep>?
}

class GetStep {
  prop1: (GetStep | PutStep)?
}

class PutStep {
  prop1: (GetStep | PutStep)?
}

It seems I should be able to use a typealias anywhere I would use a type and get the same result.
I can't see in the documentation for type alias where this is mentioned.
Clearly the workaround is to not use a type alias and instead just redefine the complex type everywhere which is ok for now, but it's a surprising limitation.
Versions:
$ pkl --version
Pkl 0.25.2 (macOS 14.1, native)

We are discussing the possibility to create recursive type aliases. But, indeed, this is not currently supported.


RTCStatsReport should inherit or extend from Map

⚙ Compilation target
ESNEXT
⚙ Library
No idea why this item is mandatory
Missing / Incorrect Definition
RTCStatsReport is defined as follows:
declare var RTCSessionDescription: {
    prototype: RTCSessionDescription;
    new(descriptionInitDict: RTCSessionDescriptionInit): RTCSessionDescription;
};

/** [MDN Reference](https://developer.mozilla.org/docs/Web/API/RTCStatsReport) */
interface RTCStatsReport {
    forEach(callbackfn: (value: any, key: string, parent: RTCStatsReport) => void, thisArg?: any): void;
}

As per documentation, RTCStatsReport is a Map-like object so it should include methods such as values(), entries() and members such as size. However it only exposes forEach().
Sample Code
const pc = new RTCPeerConnection();

pc.createDataChannel('foo', { negotiated: true, id: 123, ordered: true });

const stats = await pc.getStats();

console.log('--- num stats:', stats.size);

for (const [id, stat] of stats) {
	console.log('--- stat:', stat);
	console.log('--- stats.get(id):', stats.get(id));
}

Produces:
--- num stats: 2
--- stat: {id: 'D37', timestamp:<PHONE_NUMBER>612.802, type: 'data-channel', bytesReceived: 0, bytesSent: 0, …}
--- stats.get(id): {id: 'D37', timestamp:<PHONE_NUMBER>612.802, type: 'data-channel', bytesReceived: 0, bytesSent: 0, …}
--- stat: {id: 'P', timestamp:<PHONE_NUMBER>612.802, type: 'peer-connection', dataChannelsClosed: 0, dataChannelsOpened: 0}
--- stats.get(id): {id: 'P', timestamp:<PHONE_NUMBER>612.802, type: 'peer-connection', dataChannelsClosed: 0, dataChannelsOpened: 0}

Documentation Link
https://developer.mozilla.org/en-US/docs/Web/API/RTCStatsReport

The reason we ask for sample code is so that we can inspect a value of this type at runtime and verify the things you're saying about it.

Thanks. I've added sample code.


how to take each frame from VideoPlayerController to display that frames in timeline ?

Hi @yashanghan22
This issue doesn't seem to describe a bug or a feature request. Please see https://flutter.dev/community for resources and asking questions like this,
you may also get some help if you post it on Stack Overflow and if you need help with your code, please see https://www.reddit.com/r/flutterhelp/
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you

yes this is not a bug but i want frames from videoplaycontroller in my video editor timeline so that i am asking about that it is possible or not?

yes this is not a bug but i want frames from videoplaycontroller in my video editor timeline so that i am asking about that it is possible or not?

yes this is not a bug but i want frames from videoplaycontroller in my video editor timeline so that i am asking about that it is possible or not?

It may be possible but this is out of scope for the video player currently

ok no problem i just wanted to know about that posibility. so if this can be possible in future please make it it's very need to make video editing apps in flutter.

if you want to make package related video editing app we can collaborate and work on this.


Add Japanese documents for Block Kit in Modals

Summary
In addition to #261 , this pull request adds Japanese documents. I would like to have @girliemac as a reviewer to verify the quality and consistencies with other docs. 🙏
Requirements (place an x in each [ ])

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.

LGTM ❤


[Bug]: Address format is always the default format ignoring Localization > Countries > United States > Format

What happened?
Addresses formatted using Countries_model::addressFormat() always use default format : {address_1}\n{address_2}\n{city} {postcode}\n{state}\n{country}
Because of:
app/system/libraries/Country.php
Countries_model::addressFormat()
$format = $this->getDefaultFormat();
$address = $this->evalAddress($address);
if (!empty($address['format']))
    $format = $address['format'];

Countries_model::evalAddress() will strip the format value by rebuilding the address based on the Countries_model::$requiredAddressKeys array which does not include 'format', so $format will always be equal to Countries_model::getDefaultFormat()
Could 'format' be safely added to Countries_model::$requiredAddressKeys[] without breaking things elsewhere?
What did you expect to happen?
Address to be formatted with the custom value entered in Localization > Countries > United States > Format
{address_1}\n{city}, {state} {postcode}\n{country}
Version
3.x
What browser are you seeing the problem on?
No response
Relevant log output
No response

Yea, should be safe to add the format key

This issue will be closed and archived in 3 days, as there has been no activity in the last 60 days. If this issue is still relevant or you would like to see it actioned, please respond and we will re-open this issue. If this issue is critical to your business, consider the Dedicated Support Service where a Service Level Agreement is offered.


[Proposal] Automatically swap inputs when possible

Feature/enhancement description:

Example - node with inputs 1 and 2 occupied by two connections.  When user pulls out the input 1 wire and connects it into input 2, the connection from input 2 is automatically rewired to replace the one removed from input 1.
(I find that this is often the desired result, but even if I'm wrong this can be limited to only happen while holding down a modifier key )

Hmm the only case I can think of where this would be desirable is the Blend node (to swap between foreground/background channels). But yes, it could be implemented with a modifier.


Onboarding checklist: Workshop evaluation form

The last item in debriefing is
"One week after the workshop, the local coordinator will send out a workshop evaluation form to all participants. A summary of all feedback is provided after each term in the form of a presentation or a written report by the coordinator. Ask the local coordinator for the feedback for your workshop at any time."
Is it a UiO evaluation form or Carpentries post-workshop survey?
If it is Carpentries post-workshop survey, a link to the survey results will be shared by Carpentries team to the instructor directly, so probably it should be revised.

It is - or better: it was - a UiO-form (Nettskjema). We have not used it for the workshops in spring, it was supposed to be revised by the Board or a special task force, but it never happened. I think this should be put as a task for the Board: review the Nettskjema.
I don't want to send out too many surveys to participants, it's annoying if you have to fill out a lot and chances are high that only a few will do it.

Why one week after the workshop? Wouldn't be better if the form was sent max 3 days after the workshop? Then the event is still fresh in the participants memory. What do you think?

The reason is that the survey asks for how/if people have had the chance to use what they have learned in their day-to-day work. If we ask this too early, the answer is likely to be no. A week is still a fairly short time to implement something newly learned, but given that people will have forgotten about the other aspects of the workshop, one week seemed to be a good compromise.
However, the entire post-workshop survey is under revision, so we can discuss how to handle this in the future.
#33

We should take this up again for discussion with the Board and the library. In 2020, we’ve been using the Carpentries post-workshop survey bit not the Nettskjema one to not create feedback-fatigue. The response rate for the surveys is generally low. Is there anything we - the UiO- Carpentry community- wants/needs to know in regards to learner feedback that the Carpentries survey doesn’t cover?

Closed Issue #33 - Once Onboarding checklist is updated, this Issue should be closed, too.

Merge with #96


Not an issue. Just a thank you

I started playing around with the Raspberry Pi by getting a Pi model B in 2012. Hadn't touched it for years, and decided to get a 3 B+. I found the MegaPi Case on Amazon, reasonably priced and promising proper shutdown via scripts and functioning buttons. All of this was true, and the scripts were rather easy to get working. At first I was confused, because hitting the reset switch was actually resetting the Pi, and not ES. But I realised that crcerror's rewrite was not part of Master, so I got that from his repo and it was all good. Everything works great, and I really appreciate everyone's contributions and effort to get everything working smoothly and easy to setup.

nice


Conversion of Diffusion model inside Colab generates a non valid .tflite file

1. System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Colab connected to VM of GCP (high RAM is needed)
TensorFlow installation (pip package or built from source): Version 2.11.0
TensorFlow library (version, if pip package or github SHA, if built from source):

2. Code
Option B: Paste your code here or provide a link to a custom end-to-end colab
Link for colab notebook to open:
https://colab.research.google.com/drive/1I7m4sGWi6U1ODbG0Fln42GU45CEf7Qsk?authuser=1#scrollTo=uGjOTEFoZrDq
Link for colab notebook to download:
https://colab.research.google.com/drive/1I7m4sGWi6U1ODbG0Fln42GU45CEf7Qsk?usp=share_link

Include code to invoke the TFLite Converter Python API and the errors.

converter = tf.lite.TFLiteConverter.from_keras_model(diffusion_model)
converter.optimizations = [tf.lite.Optimize.DEFAULT]
converter.target_spec.supported_types = [tf.float16]
converter.target_spec.supported_ops = [
  tf.lite.OpsSet.TFLITE_BUILTINS,
  tf.lite.OpsSet.SELECT_TF_OPS # enable TensorFlow ops.
]
tflite_model = converter.convert()

# Save the model.
with open('diffusion_model.tflite', 'wb') as f:
  f.write(tflite_model)

Provide links to your TensorFlow model and (optionally) TensorFlow Lite Model.
https://huggingface.co/fchollet/stable-diffusion/blob/main/kcv_diffusion_model.h5
https://drive.google.com/file/d/1coWb7CWBYrU7pDFX3SWpmmWUqus68R7w/view?usp=share_link

3. Failure after conversion
If the conversion is successful, but the generated model is wrong, then state what is wrong:

Model cannot be opened by netron.app and android application. The error is:
"Not valid .tflite flatbuffer file"

5. (optional) Any other info / logs
When the default quantization is used
converter = tf.lite.TFLiteConverter.from_keras_model(diffusion_model)
converter.optimizations = [tf.lite.Optimize.DEFAULT]
converter.target_spec.supported_ops = [
  tf.lite.OpsSet.TFLITE_BUILTINS,
  tf.lite.OpsSet.SELECT_TF_OPS # enable TensorFlow ops.
]
tflite_model = converter.convert()

# Save the model.
with open('diffusion_model4.tflite', 'wb') as f:
  f.write(tflite_model)

the procedure exits with:
" WARNING:absl:Found untraced functions such as conv2d_84_layer_call_fn, conv2d_84_layer_call_and_return_conditional_losses, _jit_compiled_convolution_op, group_normalization_63_layer_call_fn, group_normalization_63_layer_call_and_return_conditional_losses while saving (showing 5 of 1320). These functions will not be directly callable after loading.
INFO:tensorflow:Assets written to: /tmp/tmptbzu7bs5/assets
INFO:tensorflow:Assets written to: /tmp/tmptbzu7bs5/assets
WARNING:absl:Buffer deduplication procedure will be skipped when flatbuffer library is not properly loaded
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
[<ipython-input-9-c1e1de4224af>](https://localhost:8080/#) in <module>()
      6   tf.lite.OpsSet.SELECT_TF_OPS # enable TensorFlow ops.
      7 ]
----> 8 tflite_model = converter.convert()
      9
     10 # Save the model.

16 frames
[/usr/local/lib/python3.7/dist-packages/tensorflow/python/framework/importer.py](https://localhost:8080/#) in _import_graph_def_internal(graph_def, input_map, return_elements, validate_colocation_constraints, name, producer_op_list)
    495   # _ProcessNewOps.
    496   with graph._mutation_lock():  # pylint: disable=protected-access
--> 497     with c_api_util.tf_buffer(graph_def.SerializeToString()) as serialized:
    498       try:
    499         with graph._c_graph.get() as c_graph:  # pylint: disable=protected-access

ValueError: Message tensorflow.GraphDef exceeds maximum protobuf size of 2GB:<PHONE_NUMBER> "

but per documentation this is supposed to create a 4x times smaller file??
ie ~850MB?
https://www.tensorflow.org/lite/performance/post_training_quantization#optimization_methods

Converting Keras Stable Diffusion models to tflite models should just work for fp16 or dynamic range quantization if recent TensorFlow master branch is used. For older TensorFlow versions, you probably need

set input batch size, and
remove tf.lite.OpsSet.SELECT_TF_OPS

I was able to convert

fp32 models to fp32 tflite models by splitting diffusion_models into two chunks
fp32 models to fp16 tflite models,
fp32 models to dynamic range int8, which is what you tried to do
full-integer quantized

I also managed to converted fp32 tflite models to performance text-to-image.
See my comments for a keras cv [1] and scripts I used [2].
[1] https://github.com/keras-team/keras-cv/issues/1033
[2] https://github.com/freedomtan/keras_cv_stable_diffusion_to_tflite

Hi @farmaker47 !
Could you give us an update with after testing @freedomtan 's work around on going with fp16 lite model or full integer quantization.
Thank you!


While(Active()) - Say Output

Create what you think the structure of the output will be
def evolve():
passs
while(Evolve.active):
pass

EasyGA Class (including active() method)
import random

class EasyGA():

    #If user provides all parameters (that I can think of right now anyway)
    #opt is an object that includes information on users selection method, mutation rate, fitness type, etc.
    def __init__(self, chromosome_count, chromosome_length, gene_lower_bound, gene_upper_bound, opt):
        self.population_set = [] #Array of type Population

        #converting integer-input into array
        if type(gene_lower_bound) == int:
            gene_lower_bound_temp = []
            for i in range(chromosome_length):
                gene_lower_bound_temp.append(gene_lower_bound)

            gene_lower_bound = gene_lower_bound_temp

        #converting integer-input into array
        if type(gene_upper_bound) == int:
            gene_upper_bound_temp = []
            for i in range(chromosome_length):
                gene_upper_bound_temp.append(gene_upper_bound)

            gene_upper_bound = gene_upper_bound_temp

        self.gene_lower_bound = gene_lower_bound
        self.gene_upper_bound = gene_upper_bound

        initial_population = Population()
        initial_population.generate_first_chromosomes(chromosome_count, chromosome_length, gene_lower_bound, gene_upper_bound)
        population_set.append(initial_population)

    def active(self):
        #Rather than having a standard .run() function that itself decides when to stop running...
        #... There will be no 'running' state for EasyGA, just a status flag (the return value of this function)...
        #... that determines if the algorithm should continue doing things based on typical termination factors (maximum generation or target value)
        if current_generation < generation_limit and fitness < target_fitness:
            return True
        else:
            return False

class EasyGA:

    ga.update_fitness = True

    def fitness_impl(self, chromosome):
        """Returns the fitness of a chromosome"""
        pass

    def evolve(self):
        """Updates the ga to the next generation. If update_fitness is set then all fitness values are updated. Otherwise only fitness values set to None (i.e. uninitialized fitness values) are updated."""
        for chromosome in ga.population.get_all_chromosomes():
            if self.update_fitness or chromosome.get_fitness() is None:
                chromosome.set_fitness(self.fitness_impl(chromosome))

    def active(self):
        """Returns if the ga should terminate or not"""
        return current_generation < generation_limit and fitness < target_fitness


Add At The Speed of Light x Face Off by Dimrain47

Song Name
At The Speed of Light x Face Off
Artist Name
Dimrain47
Source
Youtube
Youtube Link
Rs6WXfzbgAI
Direct File Link
No response
Song ID
467339
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 3979721

accept

accept


webapps: Updating default node version used during creates

Fixes: #7749, #7689
This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[ ] I adhere to the Command Guidelines.

Need to update history will update with the next iteration


add network commands tests

add tests for functionality implemented in https://github.com/cloudwm/cloudcli-server/pull/7 and https://github.com/cloudwm/cloudcli/pull/35

merged


Synacormedia: Added video parameters to documentaion.

Synacormedia: Added video parameters to documentaion.

Just following up on this, the linked PR was merged & released in yesterday's release, so this documentation PR should be merged and make available too.


Unhandled execption in script

Traceback (most recent call last):
File "main.py", line 6, in
File "", line 1178, in _find_and_load
File "", line 1149, in _find_and_load_unlocked
File "", line 690, in _load_unlocked
File "PyInstaller\loader\pyimod02_importers.py", line 499, in exec_module
File "maingui.py", line 22, in
System.IO.FileNotFoundException: Unable to find assembly 'C:\WINDOWS\system32\data\System.Management.Automation.dll'.
в Python.Runtime.CLRModule.AddReference(String name)

try this and tell me what it shows in the terminal console
https://we.tl/t-4gxmeCiWqq


Update github.com/coreos/etcd v3.3.10+incompatible as it is an vulnerable dependency

Summary

Steps To Reproduce

Expected Behavior

Screenshots

Sentry.io Event

Environment

SDK

sentry-go version:
Go version:
Using Go Modules? [yes/no]

Sentry

Using hosted Sentry in sentry.io? [yes/no]
Using your own Sentry installation? Version:
Anything particular to your environment that could be related to this issue?

Additional context

Our package doesnt depend directly on this package, thus it can be updated by the consumer.


[GiteaBridge] Add Tags context

Following feedback in #2069, here is a new PR, with a more limited scope.
Add support for tags context.

This makes more sense in the gogsbridge. Can you redo the functionality for the gogsbridge and add changes to the gitea bridge if neccessary? I dont know if gogs-tags work different to gitea-tags. It's hard to find any public instances with actual data and releases.

Hello Bockiii,
If you follow the link in the OP, you'll see that this feature was implemented in Gitea, independently of Gogs.
You can compare both implementation of tags here :

https://try.gogs.io/gileri/test/releases
https://try.gitea.io/fnetx/testing-gitea-releases/tags

I don't think there is enough in common between the two to warrant a solution that works for both, implemented in GogsBridge.

We at least need to remove all the duplicate code. Right now you basically copied the whole gogs bridge into the gitea bridge.

I'm not a big fan of inheritance. But if we follow bockiis advice and do it properly, it should be done like this:
diff --git a/bridges/GiteaBridge.php b/bridges/GiteaBridge.php
index 3324787..25165aa 100644
--- a/bridges/GiteaBridge.php
+++ b/bridges/GiteaBridge.php
@@ -24,4 +24,17 @@ class GiteaBridge extends GogsBridge {
 			);
 		}
 	}
+
+	protected function collectTags($html)
+	{
+		$tags = $html->find('table#tags-table > tbody > tr')
+		or returnServerError('Unable to find tags');
+
+		foreach($tags as $tag) {
+			$this->items[] = array(
+				'uri' => $tag->find('a', 0)->href,
+				'title' => 'Tag ' . $tag->find('.release-tag-name > a', 0)->plaintext,
+			);
+		}
+	}
 }
diff --git a/bridges/GogsBridge.php b/bridges/GogsBridge.php
index a2adc1f..0167dab 100644
--- a/bridges/GogsBridge.php
+++ b/bridges/GogsBridge.php
@@ -53,6 +53,7 @@ class GogsBridge extends BridgeAbstract {
 			),
 		),
 		'Releases' => array(),
+		'Tags' => array(),
 	);

 	private $title = '';
@@ -98,6 +99,7 @@ class GogsBridge extends BridgeAbstract {
 			case 'Issues':
 			case 'Releases': return $this->title . ' ' . $this->queriedContext;
 			case 'Single issue': return $this->title . ' Issue ' . $this->getInput('issue');
+			case 'Tags': return 'Tags for ' . $this->title;
 			default: return parent::getName();
 		}
 	}
@@ -127,6 +129,9 @@ class GogsBridge extends BridgeAbstract {
 			case 'Releases': {
 				$this->collectReleasesData($html);
 			} break;
+			case 'Tags':
+				$this->collectTags($html);
+				break;
 		}

 	}
@@ -202,4 +207,9 @@ class GogsBridge extends BridgeAbstract {
 			);
 		}
 	}
+
+	protected function collectTags($html)
+	{
+		// Not implemented for gogs
+	}
 }

We at least need to remove all the duplicate code

The code is duplicated for a reason : the projects are drifting away more and more, and some features are not available in Gogs, or Gitea (e.g. the Tags page). We could do more convoluted inheritance between the two bridges, but to what end ? Both bridges are simple enough anyway.

But if we follow bockiis advice and do it properly, it should be done like this

@dvikan Thank you for suggesting an implementation ! However doing it this way makes the Tags context available in the GogsBridge in the web UI, making users think it's implemented. Is using inheritance more important than presenting the correct available contexts to users ? I think we away with it, especially if you're not a fan of it too ;)
Also your comment made me realize that the Bridge did not have a name/title for the Tags context, that's fixed in f758993.

Your comments make sense gilleri. I think I prefer duplication here. So let's do it properly. Duplicate the bridge, drop the inheritance and implement the desired changes.

I agree, thanks for your support. I've rewritten completely GiteaBridge to decouple it from GogsBridge, and will open a new PR, since this one strayed quite far from the original goal.


BUG?: KeyError(s) when running read2tree

Hello,
I am trying to use read2tree, able to install it and run it, the example runs without a hitch and I get the expected output files.
When I try to use my sequences though, I was getting many "Invalid marker group" errors, this was relatively straightforward to take care of, I just renamed the fasta header lines accordingly.
Now I cannot get past an error that reads KeyError: 'U1810' in particular,
I think this definitely has to do with the five letter you use/infer, but don't really know how to make it work properly,
Any ideas?

Hi @almosnow
Thanks for reaching out. Could you please give us more information on how you get the gene markers? would be great if you could share with us the mplog.log file and the command line(s) you use.
One thing is that the fasta record ID of gene markers in both amino acid level (OGXX.fa files) and nucleotide level should match. This is needed when  you concatenate all fna files as  dna_ref.fa and provide read2tree with --dna_reference  dna_ref.fa. Otherwise, read2tree uses RestAPI to download them from OMA web browser assuming that the gene markers are downloaded from the OMA web browser.
Best,
Sina

Hmm, ok I see, I did not set up the gene markers properly I think.
Actually, now that I've read more, what I did was completely wrong.
Here's my scenario, perhaps you can advice on what to do.
We have a set of ~15 sequences (coding sequences from the same gene and the same organism, different samples around the world), with minor variations between them, a phylogeny shows two major groups distinct of each other (but changes between them are small, SNPs and the like).
We have another set of a few hundred SRA libraries and we would like to find out to which of the aforementioned 15 sequences they are most similar to.
Is it ok to use those initial 15 sequences as marker genes and try to fit the reads into them?

For this case, Read2tree can generate a tree in Multiple species mode. However, one gene might not be enough to describe the evolution of organism or  provide enough resolution for distinguishing all samples.
Anyway, you can put the amino acid sequences in a fasta file in the marker_genes folder  and the nucleotide sequences of coding regions (with exact order) in another fasta file, mentioned with --dna_reference  genes.nuc.fa . Note that the gene names should match in both files. Each starts with a five letter code for each strain, like this
>ASTMX02439
>PYGNA12763
>ELEEL42119


Dialog 'button2' response

Hi! Button2 is usually used as an 'exit/cancel' button - for now, we cannot know if it was pressed or not. We only get inputText from the promise.

*This is incorrect, you can access which button was pressed by 'response' argument
1-Left button
0-Right button


onnx-mlir failed with rnn-mnist benchmark

Hi,
With the rnn-mnist benchmark (https://github.com/abrahamchan/LLFI-TF/blob/udit_benchmarks/sample_programs/rnn-mnist/rnn-mnist.py), onnx-mlir failed with the following error message:
$LLFI-TF/sample_programs/rnn-mnist$ onnx-mlir --EmitLLVMIR model.onnx
Loop op doesn't support dynamic dimensions for scan output.
UNREACHABLE executed at /home/uditg/UBC/onnx-mlir/src/Conversion/ONNXToKrnl/ControlFlow/Loop.cpp:254!
Aborted (core dumped)

Reopening this issue as it looks like that ONNX-MLIR started supporting RNNs now (https://github.com/onnx/onnx-mlir/blob/main/test/numerical/TestRNN.cpp).
We should try using the recent version of ONNX-MLIR with LLTFI.


gl-testing: Cleaning imports

I've misconfigured my editor and did some clean-ups which appear to make the tests fail.


Prepared input files for DLK

Input submission checklist

[x] created a directory for each system based on the template directory
[x] named and placed the directory with the following pattern: input_structures/prepared_structures/<set_name>/<system_name>

For each submitted directory:

[x] filled in system preparation details in the PREPARATION_DETAILS.md file
[x] added a PDB file with the protein named protein.pdb
[x] removed any cofactors from the PDB file and placed them in cofactors.sdf
[x] did not re-protonate the residues, but left protonation state of the protein unchanged
[x] kept cystallographic waters and metals in the PDB file
[x] tested the PDB and, if available, cofactors.sdf file using the input validation script
[x] copied over the ligands SDF file to a file named ligands.sdf and removed any duplicate binding modes or protonation states, keeping the more favorable state

Summary of changes

Removed ligands 4neu and 17neu
Protein termini were capped, and NMA renamed to NME
Residues ASN241, PHE270, SER326, and CYS335 contained alternative conformations, the default one was used
The system had no crystallographic waters, metal ions, or other cofactors

Licensing agreement

[x] I agree to release these inputs under the combined MIT and CCBY SA 4.0 licenses of this repository

Thanks, @tlhr this looks great! Just the one comment for the ligands file! If you remove the alternate protonation states, could you then also update the preparation instructions accordingly? Thank you!

Sorry about that, it should be fixed now!

Just adding some minor comments here for completeness:

Downgraded to cudatoolkit=11.4 for cluster compatibility
Ran using Nvidia multi-process-service (MPS), by adding the following lines to the job submission script (our GPUs are in Exclusive Process mode by default, which is not compatible with openmmtools):

export MPS_DIR=$(mktemp -d)
export CUDA_MPS_LOG_DIRECTORY=$MPS_DIR
export CUDA_MPS_PIPE_DIRECTORY=$MPS_DIR
nvidia-cuda-mps-control -d

Just adding some minor comments here for completeness:

Downgraded to cudatoolkit=11.4 for cluster compatibility
Ran using Nvidia multi-process-service (MPS), by adding the following lines to the job submission script (our GPUs are in Exclusive Process mode by default, which is not compatible with openmmtools):

export MPS_DIR=$(mktemp -d)
export CUDA_MPS_LOG_DIRECTORY=$MPS_DIR
export CUDA_MPS_PIPE_DIRECTORY=$MPS_DIR
nvidia-cuda-mps-control -d

O that's really interesting, I didn't know you could leverage MPS this way, thanks so much for sharing @tlhr ! We would really be interested in any performance number you get.

O that's really interesting, I didn't know you could leverage MPS this way, thanks so much for sharing @tlhr ! We would really be interested in any performance number you get.

Sure! simulation_real_time_analysis.yaml is reporting ~300 ns/day on a single V100 (This must be the aggregated number over all replicas, so around 30 ns/day/replica).


Release v1.0.3 staging

The last/latest release tag was/is v1.0.2, dated 26 July 2019, created on the develop branch:
https://github.com/readium/readium-desktop/releases/tag/v1.0.2
There was no structured changelog between the tag v1.0.2 and the HEAD of the master branch at the time (03 May 2018), because of the huge number of commits between the distant release dates, coupled with the fact that Pull Requests were merged without squashing (thereby making it extremely difficult to extract a CHANGELOG). See:
https://github.com/readium/readium-desktop/commits/26f2fdae1becdc70638a39dd4e96f76dff0786df
=>
https://github.com/readium/readium-desktop/compare/26f2fdae1becdc70638a39dd4e96f76dff0786df...v1.0.2
Note that at this time of writing, the HEAD of the master branch is still on 03 May 2018 (we will synchronize master with develop at the v1.0.3 milestone):
https://github.com/readium/readium-desktop/commits/master
Now, the commits between the v1.0.2 tag and the current HEAD of develop branch are annotated using the "conventional commits" notation, so we will be able to extract a comprehensible CHANGELOG:
https://github.com/readium/readium-desktop/compare/v1.0.2...develop
Reminder of the recommended method to create the upcoming v1.0.3 release tag:
# Just to double-check the current status - clean, conflict-free local copy of the `develop` branch:

git checkout develop && git pull
git branch && git status && git --no-pager diff
npm run lint && npm run build

...note that at this stage, the semantic version in package.json, src/package.json and package-lock.json should be bumped to the appropriate value (i.e. 1.0.3 in this case).
# This creates a release tag with a date stamp exactly the same as that of the latest commit (HEAD) at the tip of the `develop` branch:

git tag --list

GIT_COMMITTER_DATE="$(git show develop --format=%aD | head -1)" \
git tag -a "v1.0.3" develop -m "tag v1.0.3" \
&& git push --tags origin develop \
&& git tag --list --format='%(refname)   %(taggerdate)'

This can then be followed by a fast-forward merge into the master branch (i.e. no merge commit):
git checkout master && git pull && git log
git merge develop

Then, it would probably be a good idea to bump-up semantic versioning in the develop branch (by manually updating package.json, src/package.json and package-lock.json), so that further developments and Continuous Integration / Deployment is clearly differentiated from the official latest release, for example 1.0.4-rc.0 (see https://semver.org )
See related issues:
https://github.com/readium/readium-desktop/issues/145
https://github.com/readium/readium-desktop/issues/98

I'm ok with this procedure.

Status Update -

The head of the master branch is at release tag v1.0.3 attached to commit 71f5e94c1d6eb71f553729e818d2153066633323:

https://github.com/readium/readium-desktop/releases/tag/v1.0.3
https://github.com/readium/readium-desktop/commits/71f5e94c1d6eb71f553729e818d2153066633323

The master branch is a copy of the Git commit history of the develop branch, created via git merge with fast-forward (no merge commit). Consequently, commit 71f5e94c1d6eb71f553729e818d2153066633323 also exists in the develop branch, and so does the v1.0.3 release tag.
The head of the develop branch is currently ahead of the master branch by one commit, a version bump to 1.0.4-rc.0 for upcoming development in the codebase "trunk":

https://github.com/readium/readium-desktop/compare/71f5e94c1d6eb71f553729e818d2153066633323...develop

I will generate the v1.0.3 changelog from the following commit diff:

https://github.com/readium/readium-desktop/compare/v1.0.2...v1.0.3

Changelog:
https://github.com/readium/readium-desktop/releases/tag/v1.0.3
(please feel free to edit the summary, or to suggest edits here so I can integrate them into the release blurb)