Receive NSEvent error when inserted into title bar

Hi there,
Using the native NSWindow APIs* I’m inserting a TabsControl instance into the title bar of an NSWindow. Whenever I try to resize the window, I receive either an exception or failed assertion, both involving NSEvent:

2017-05-22 17:58:32.728021-0400 SECLookup[1518:41383] *** Assertion failure in -[NSEvent trackingArea], /Library/Caches/com.apple.xbs/Sources/AppKit/AppKit-1504.83.101/AppKit.subproj/NSEvent.m:3000

You can find a full output log here.
The bug occurs within AppKit, but it does not occur unless I insert the tabs control into the window title bar, so I surmise that this class is interfering with AppKit somehow.
You can find a sample project here, stripped to the bare-bones necessary to let you see the problem. Launch the app, re-size the window, and the crash will occur.
I would appreciate any assistance you can provide.
* - I’m using the WAYWindow subclass to make this simple, but I can confirm that this also causes a crash when I do the insertion manually.

Hi. Thanks for your report. Interesting problem. I've look at your bare project, and one easily see where the crash occurs indeed.
I've always used this small snippet in the function updateTrackingAreas to ensure the state of elements depending on the mouse being inside or not is correct. But here it causes a crash, and inspection of the event in my case reveals only a "System Event".
I am a bit reluctant to remove that snippet without understanding the root cause, even if it suppress the problem...

@onekiloparsec Where is the troublesome code in your framework? I will try commenting it out and seeing if it affects the behavior of my application at all.
I can't identify the root cause, either. I've filed a bug with Apple but haven't heard back from them.

Simple, the issue points in the updateTrackingAreas function of the TabButton class. But the answer is already known: commenting out the code will solve your issue. And you can certainly close the radar at Apple, this isn't a bug on their side.
The problem is : this code is making useful things. I could remove it for sure, but I would rather prefer making it smarter.
The thing I would investigate (no time for now, but you can check): is to forward the event to the mouseExited and mouseEntered functions only for certain type of events and not others.

I’ve got it working @onekiloparsec. It would be nice to have a work around but in the meantime I just commented out the code.

For reference, we're talking about a crash here, right?
https://github.com/onekiloparsec/KPCTabsControl/blob/master/KPCTabsControl/TabButton.swift#L133
@SevenBits can you try what's the absolute minimum you have to comment-out to make this work?
For what it's worth, I have experimented with a hover-aware sidebar component and use similar code:
https://github.com/CleanCocoa/FatSidebar/blob/master/FatSidebar/FatSidebarItem.swift#L419
There, I call super.updateTrackingAreas() first. Maybe the order of instructions is important, too?

@DivineDominion @onekiloparsec If I comment out the self.addTrackingArea(self.trackingArea!) statement in the updateTrackingAreas method, then everything works except for the fullscreen/maximize button. If I need the fullscreen/maximize to work, I need to comment out two additional lines. The fully working code is this:
open override func updateTrackingAreas() {
    if let ta = self.trackingArea {
        self.removeTrackingArea(ta)
    }

    let item: AnyObject? = self.cell?.representedObject as AnyObject?

    let userInfo: [String: AnyObject]? = (item != nil) ? ["item": item!] : nil
    self.trackingArea = NSTrackingArea(rect: self.bounds,
                                       options: [.mouseEnteredAndExited, .activeInActiveApp, .inVisibleRect],
                                       owner: self, userInfo: userInfo)

    //self.addTrackingArea(self.trackingArea!)

    if let w = self.window, let e = NSApp.currentEvent {
        let mouseLocation = w.mouseLocationOutsideOfEventStream
        let convertedMouseLocation = self.convert(mouseLocation, from: nil)

        if NSPointInRect(convertedMouseLocation, self.bounds) {
            //self.mouseEntered(with: e)
        }
        else {
            //self.mouseExited(with: e)
        }
    }

    super.updateTrackingAreas()
}

I do not know what impact this workaround has on the remainder of the library.

Commenting out the line self.addTrackingArea effectively removes the tracking area from doing anything useful. It will just sit in memory.
I keep thinking the investigation must focus on what type of event we receive, and forwarding them only the ones that make sense.

Hi @onekiloparsec, I agree, it is not the ideal solution. However I was looking at the minimum amount of code I needed to comment out to get it work. I will examine different types of events next. Do you have any ideas on which events might be the best places to start looking?

Sorry but you just make "get it work" for you. I have tried a few things this morning, and maybe this type of events is not the issue.
I've tried to get rid of the WAYWindow class to make sure this isn't creating an addition layer of complications. But for adding tabs to a titlebar one must use a NSTitlebarAccessoryViewController.
But when trying to do so, I get an error message warning: could not execute support code to read Objective-C class data in the process. This may reduce the quality of type information available. WTF.
Anyway, I would prefer to understand what's going on before changing something that is otherwise working. I would suggest to get rid of the WAYWindow first in your tests, as I tried myself without success so far.


Functional request: Share and collab playlists

I would like to propose  sharing and collaborative editing functionalities. I saw something in the issues regarding sharing playlists but haven't found how to do it

#187 Duplicate


Players bug out mid train ride and are left floating as the train continues

Expected Behaviour
Train goes from point A to point B with players on board.
Actual Behaviour
Train goes from point A to point B but will randomly bug out and drop off players in the middle of the track.
Reproduction Steps
Steps to reproduce the behaviour:

Get on train (on a server)
At some random point the train may or may not bug out and drop you off mid track.

 I copied the world file into singleplayer and it worked perfectly. Its not caused by any client-side mods either since I was still able to reproduce the bug without them.

Thanks for your report. What mod version are you using? beta-9-unofficial something?

MTR-forge-1.18-3.0.0-beta-8

Please verify if this is fixed in the latest release, thanks!


Adding xtoys.app webhook functionality for interactions with toys.

The webhook is generic enough to use with any webserver accepting JSON, but was created for xtoys.app.
How to test:

Follow Misc/Webhook.md instructions, should work even without a bluetooth connection to the toys.

Pretty cool but I can't really maintain it since I don't use that website and the setup seems to be a bit tricky x3. Ideally you'd somehow turn it into a mod, but that's probably gonna be tricky too..

Is there a way to expose stats updates to mods? Some kind of an external UI mod? I've seen mostly additional .gd files or .tscn files in mods, I'm not sure how extendable hooking would work in Godot.

Its possible to add a few game-related callbacks to the Module class maybe. And mods can add their own modules easily, they are objects that always exist loaded in memory

i had an idea: species: kangaroo


Support MSYS2 64-bit platform for windows

Is your feature request related to a problem? Please describe.
Template and demo project Makefiles broken for 64-bit msys2 / windows environment.
Describe the solution you'd like
Fix Makefiles to ensure ZIP tool dir is correctly set for msys2 64-bit windows env.
Describe alternatives you've considered
Use 32-bit version of msys2 for building projects or manually fix makefile conditional logic to check for msys2 / windows build flavor more appropriately.
Additional context
Makefile expects $(MSYSTEM) to equal "MSYS" for msys2 platform. However, when using the 64-bit version, $(MSYSTEM) is MINGW64. Ideally, a less flakey approach for checking the build env should be used, or the Makefile should be modified at line 31 to check for both cases before overriding ZIP path.
E.g.,
ifneq ($(MSYSTEM), MSYS)
ifneq ($(MSYSTEM), MINGW64)
ZIP = ....
endif
endif
See:
Makefile.fixed.txt

In 64-bit msys2 installations I have access to MSYSTEM does report MSYS and not MINGW64.
However if it is the case under some installations the suggested Makefile modification is worth it.


Fixes due to GitLocalize misinterpreting things

Changes
Fixes due to GitLocalize misinterpreting some LaTeX and HTML statements
In file translations/es/ch-applications/vqe-molecules.ipynb
This PR must be merged before https://github.com/Qiskit/platypus/pull/1254
Problems

GitLocalize is interpreting symbols within LaTeX statements as if they were HTML characters, for example the & symbol was placed as & (HTML equivalent symbol). See images 1 and 2.
GitLocalize added a <br> tag inside a LaTeX statement, which does not exist in the original file. See image 3.

Screenshots
Image1

Image 2
!
Image 3

@0sophy1 Please review these changes.
This PR must be merged before https://github.com/Qiskit/platypus/pull/1254
(this PR will be merged into the gitlocalize-19646 branch)


Post release automated changes for keyvault releases

Post release automated changes for azure-keyvault-admin

Error is unrelated to this changeset and is failing at "Prevent changes to eng/common outside of azure-sdk-tools repo", overriding

/check-enforcer override


High wakeup count on idle

Alacritty 0.4.0 on Wayland/Sway
Using Powertop, I see Alacritty having a steady ~19-20 wakeups/s. This is on a completely idle prompt, with the default (no file) config.
There's no measurable CPU usage, just the high wakeup count. This does however cause battery drain.
Just for reference, XTerm shows 0 wakeups/s in the same setup.
Not very high-priority I suppose :)

Have you looked into what might be causing these wakeups? Waking up to redraw would be much more power hungry than waking up for checking if the config file changed for example (I'd assume it's something like the latter).

@dnkl Are there exactly ~20 wakeups btw? Since our clipboard sleeps for 50ms on avarage then wake up and dispatches pending events. Are you fine with testing the patch that disable our clipboard on Wayland, I can send you one in case you're fine with retesting things?

@kchibisov more like "exatly" 19,4. Close enough I'd say ;)
Yeah, sure, I can test a patch

It should completely disable our clipboard.
diff --git a/alacritty/src/main.rs b/alacritty/src/main.rs
index 64e0b9c..7cf7b51 100644
--- a/alacritty/src/main.rs
+++ b/alacritty/src/main.rs
@@ -146,7 +146,7 @@ fn run(window_event_loop: GlutinEventLoop<Event>, config: Config) -> Result<(),

     // Create new native clipboard
     #[cfg(not(any(target_os = "macos", windows)))]
-    let clipboard = Clipboard::new(display.window.wayland_display());
+    let clipboard = Clipboard::new_nop();
     #[cfg(any(target_os = "macos", windows))]
     let clipboard = Clipboard::new();

That was it. I don't see any wakeups with this patch.

Well, our clipboard lives in its own crate, because upstream windowing library decided not to include clipboard things in it, so we can't block forever and should wake up sometimes to dispatch pending events to not block paste from alacritty, etc, etc.

Alacritty having a steady ~19-20 wakeups/s

Thx for this info, otherwise it will be harder for me to guess.
The relevant code in our clipboard is right here https://github.com/Smithay/smithay-clipboard/blob/e28a54171f8b0df4b0a5240d349540993bd5ac5e/src/threaded.rs#L501 , however I don't think that we can do something about it.

My limited rust knowledge is showing here, but is there no way to select(3)  or poll(3) instead? That'd be the normal way to handle things like this.

@dnkl I don't know Wayland that much, but the only thing we have in clipboard is a wl_display, that's it. So I'll try to take a look on how can we handle and improve things, however it's more QoI then a big issue for now, as far as I can see.

In the C API, you have wl_display_get_fd(). You can then select() on that FD (with POLLIN) When select indicates you have data to read, you can call wl_display_dispatch() to trigger all the Wayland registered listener callbacks.
Not sure how much of this maps to the rust bindings though.

Of course, if you're also listening for events from alacritty, then you'd have to throw that into the select() too, somehow.
I'm guessing there are nice rust wrappers for all this though...

@dnkl The current rust "interface" is implemented in channels, so it's a little bit more work, but thanks for your help. I'll take a look on it when I'll have time for this task (probably during upcoming holidays).

I'm always up for brainstorming :)


docs: update README with additional configuration options

Thank you for adio! It's a really helpful tool to mitigate issues with transitive deps brought on by tools like lerna and yarn workspaces :)
Related Issue
#7
Your solution
Added in that documentation to the README
Please let me know if this language makes sense and if I can change anything.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Missing binary compilation instructions

Description
Missing binary compilation instructions for capa. Looking for "Method 3: Compiling on the command line without pip" in the Installation.md
Steps to Reproduce

Open doc/installation.md
It does not contain binary build instructions without using pip at all.

Expected behavior:
Since the project is able to provide binary builds, then I would expect some build documentation so that users can compile their own binaries.
Actual behavior:
It is not clear how to compile your own capa binary.
Versions
N/A
Additional Information
N/A

take a peek at the CI configuration in the .github directory. basically, it boils down to invoking PyInstaller with the config also found in the .github directory.

I run sudo pyinstaller setup.py in the root directory. It then appears to "build". When I look through the results I can't find the capa binary. I'm sure I'm missing something here.

you shouldn't need to use sudo.
pyinstaller should be invoked against the spec file, not setup.py.
the resulting binary ends up in build/ or dist/ (I don't recall off the top of my head)

Were you able to get it running? The command is: pyinstaller .github/pyinstaller/pyinstaller.spec and the compiled binary will be located in dist/.


Add Support for OpenAPI v3.0

The OpenAPI (fka Swagger) was recently released. It includes several changes and new features which should also be useable in Swagger Combine.
This is only possible after Swagger Parser supports OpenAPI v3.0.

https://github.com/BigstickCarpet/swagger-parser/issues/72

Looks like Swagger Parser finally supports OpenAPI 3.0 which means we can start implementing support for OpenAPI 3.0 as well.

So after reading through the code and not understanding what was going on, I just experimented and this seems to work. Just drop an  "openapi": "3.0.0" in place of the "swagger": "2.0"  as the first line of the config, and everything seemed to work. Haven't tried to validate it or use for real, but looks promising 🤷♂


CC: Delete conversation with bot

The user should be able to delete a conversation with a bot (and other users). The UX should follow the same patterns established by Microsoft Teams. This means only deleting the conversation for the current user and maintaining the conversation for any other users.

@alliscode @matthewbolanos Gina's actively working on this; the deletion part has been implemented and tested but she has some edge cases she needs to take care of https://github.com/orgs/microsoft/projects/852?pane=issue&itemId=30423531

V1 is done - any user that is apart of the chat can delete the chat; it will delete the chat for all users
https://github.com/microsoft/chat-copilot/pull/193


Feature request: audio example for windows and windows-sys

Motivation
Looking at all the examples, there is no use of any audio libraries, such as DirectSound or XAudio2, or any sound at all. it would help me immensely to see an implementation of this.
Drawbacks
Time consuming?
Rationale and alternatives
Impact: Less knowledge on how to use the audio crates.
Additional context
No response

You can take a look at the Direct2D sample. It demonstrates how to use (nano) COM with the windows crate. DirectSound and XAudio2 are based on (nano) COM as well, and I wouldn't expect their API use to be substantially different. If this isn't the case that would make a good point for providing this sample.
Regarding windows-sys, as I understand it, there is virtually no practical COM support, and I'm not aware of any plans to bring this closer to windows. If you need a code sample that illustrates how you'd use COM with windows-sys you can inspect the generated code in the windows crate. That is essentially what you'd be writing.

Yep, sorry I'm not familiar with the audio APIs and thus am not in a position to write such a sample. Provided they're mostly COM-based APIs, Tim's suggestion is a good one. 😊


bgpv2: cleanup service reconciliation logic

Refactors service reconciliation logic to use common code path for full and diff reconciliation: in both cases, the list of services to reconcile and to withdraw is collected, and common methods are then used to compile & reconcile desired route policies and paths.

/test


DDT breaks when resolve()'ing on middlewares

I have a Middleware that is trying to resolve "/__debug__/m/js/toolbar.min.js" (the current request.path) url which is not defined in urls.py, so it throws a Resolver404 exception which inherits from Http404, so "/__debug__/js/toolbar.min.js" (request.path) can't be dispatched.
I have created a project that reproduces the error:
https://github.com/andres-torres-marroquin/django-debug-toolbar-issue-project/tree/master/ddt_issue
The Middleware breaking DDT is: https://github.com/andres-torres-marroquin/django-debug-toolbar-issue-project/blob/master/ddt_issue/main/middleware.py
A possible solution is adding django-debug-toolbar urls manually to the project instead the current automagically way.
Best regards. Thanks for the hard work.

Seems to work fine with the latest ddt. The only change I had to do (in your requirements file) was because the syntax was wrong. Can you try with the latest ddt?
diff --git a/ddt_issue/requirements.txt b/ddt_issue/requirements.txt
index db3204f..a497a10 100644
--- a/ddt_issue/requirements.txt
+++ b/ddt_issue/requirements.txt
@@ -1,2 +1,2 @@
-django=1.3.1
-django-debug-toolbar=0.8.3
+django==1.3.1
+django-debug-toolbar

You are right the requirements.txt is wrong I fixed it, and yeah, it works well with 0.9.4. I'll close the issue.
Thanks for the hard work, again.


MemoryError and rewriting data_generator

Since I don't want to bombard your inbox with emails, I'll make an issue here for the MemoryError I got with the Flickr30k data. Loading everything at once into memory seems to be too cumbersome for my machine. So maybe a generator-based approach is better. Keras now provides a fit_generator function that seems perfect to keep memory requirements as low as possible.
I just wrote a simple generator that automatically randomizes the training and val data, see 6dd0b27215594d6602036588fbfd4d6379bb59e1. If I can get everything to run, this should make the module scale really well.

Hi Emiel,
I haven't looked at data_generator for a while now, but have you tried
using yield_training_batch? It is a generator over "big_batches"  for
keras.fit(), precisely to keep the memory usage within limits. If you're
running into Memory Errors, maybe lowering big_batch_size will help.
Admittedly this is hardcoded for training data, so if the problem is in val
this won't work. Also IIRC it doesn't randomize, certainly not outside the
big_batch.
But +1 for rewriting data_generator, it has turned into a monster.
On 15 March 2016 at 14:21, Emiel van Miltenburg<EMAIL_ADDRESS>wrote:

Since I don't want to bombard your inbox with emails, I'll make an issue
here for the MemoryError I got with the Flickr30k data. Loading everything
at once into memory seems to be too cumbersome for my machine. So maybe a
generator-based approach is better. Keras now provides a fit_generator
function that seems perfect to keep memory requirements as low as possible.
I just wrote a simple generator that automatically randomizes the training
and val data, see 6dd0b27
https://github.com/elliottd/GroundedTranslation/commit/6dd0b27215594d6602036588fbfd4d6379bb59e1.
If I can get everything to run, this should make the module scale really
well.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/elliottd/GroundedTranslation/issues/17

Thanks for the response, but I'm afraid that won't help. For some reason my machine keeps running out of memory after the first epoch, while computing the perplexity of the model. Here is the full error message:
Traceback (most recent call last):
  File "train.py", line 271, in <module>
    model.train_model()
  File "train.py", line 131, in train_model
    shuffle=True)
  File "/home/ubuntu/anaconda2/lib/python2.7/site-packages/keras/models.py", line 490, in fit
    shuffle=shuffle, metrics=metrics)
  File "/home/ubuntu/anaconda2/lib/python2.7/site-packages/keras/models.py", line 231, in _fit
    callbacks.on_epoch_end(epoch, epoch_logs)
  File "/home/ubuntu/anaconda2/lib/python2.7/site-packages/keras/callbacks.py", line 36, in on_epoch_end
    callback.on_epoch_end(epoch, logs)
  File "/data/GroundedTranslation-0.1/Callbacks.py", line 96, in on_epoch_end
    val_pplx = self.calculate_pplx()
  File "/data/GroundedTranslation-0.1/Callbacks.py", line 402, in calculate_pplx
    self.use_sourcelang, self.use_image)
  File "/data/GroundedTranslation-0.1/data_generator.py", line 451, in get_data_by_split
    targets = self.get_target_descriptions(arrays[0])
  File "/data/GroundedTranslation-0.1/data_generator.py", line 735, in get_target_descriptions
    target_array = np.zeros(input_array.shape)
MemoryError

Ok - it looks like it's running out of memory on the val data, which we definitely don't have a solution for (and will require rewriting calculate_pplx too).
Quick for-now fixes: Can you use a smaller vocabulary? smaller dev set?

Good call, I could use --unk for this! Raising the lower bound will get rid of most of the long tail while maintaining the size of the dataset. (Rewriting calculate_pplx is also on the list).

Using a larger value for --unk was a good idea. The smaller the vocabulary, the further I get.
If I understand the model correctly, this is a fundamental problem with 1-hot encoding. As the dataset grows, the vocabulary grows. And as the vocabulary size grows, the size of the matrix the model has to predict grows along with it. As it stands, I have to remove about 5% from the Flickr30k data in order for everything to fit in memory. From the model:
Retained / Original Tokens: 1695886 / 1785683 (94.97 pc)
This percentage will be bigger for the MS COCO dataset, which will mean that we’re throwing out words that aren’t that rare. (In fact, we probably are, with the Flickr30k data. I am currently seeing if the system holds up with unk=30. It would be interesting to log which words get thrown out.). One solution is to make the whole algorithm leaner by having only one sentence in memory at any time. (Plus one for the prediction.) This will raise the upper bound for vocabulary size somewhat, but it remains a hard limit.
Another solution is not to have one-hot encodings, but rather use word vectors from a word2vec model, such as the GoogleNews one. This might also make learning a bit easier for the description model.

Ok, by increase unk just a little more a couple of times, I think I managed to localize the bottlenecks. The first bottleneck is computing the perplexity. But if you increase unk just enough, it's no longer an issue. The bigger problem comes with generating descriptions and computing the BLEU score. This requires more memory, and thus I needed to increase unk even more.
This was a bit surprising to me, as predicting the next word and building a sentence is exactly what the model had been doing for the entire epoch. (Or isn't this what training is all about?) So I suspect there might be a more memory-efficient way to generate sentences. (Perhaps in a trade-off with time-efficiency.)
One thing that worries me is the BLEU score. For the first two epochs (now in the third), the BLEU score has been 0.00. I'd hoped for the score to be at least a little bit better than that. Could this also be because guessing the correct word for a huge vocabulary is extremely difficult? If so, could beam search help here? Or a different encoding of the vocabulary? (E.g. word2vec or something similar.)

Both perplexity and generation happen with val, so you're running into
issues whenever you need to load a val data matrix into memory.
Does Keras now support passing generators for the validation data to
keras.fit()? When we wrote data_generator, it required a numpy array, which
is why there's no generator for val. If that's changed, moving to
generators would be ideal (and maybe it would be worth using generators in
our own Callbacks anyway). Des also said something about keras now
supporting the kind of integer-> embedding mapping you were talking about,
for moving away from one-hot mappings; this would help a lot. I'm not using
keras myself at the moment so I'm not really up to speed on recent
developments.
For your current experiments: can you cut your validation data down to a
very small number (50-100 sentences) and train on --unk 3 or so? I.e. is
the bottleneck for the vocabulary just due to the val array sizes? In this
scenario, you might also see better bleu scores, since you'll have fewer
unks in your output. Is your training cost going down? Val cost/error?
These measures are less brittle than bleu.
On 16 March 2016 at 09:06, Emiel van Miltenburg<EMAIL_ADDRESS>wrote:

Ok, by increase unk just a little more a couple of times, I think I
managed to localize the bottlenecks. The first bottleneck is computing the
perplexity. But if you increase unk just enough, it's no longer an issue.
The bigger problem comes with generating descriptions and computing the
BLEU score. This requires more memory, and thus I needed to increase unk
even more.
This was a bit surprising to me, as predicting the next word and building
a sentence is exactly what the model had been doing for the entire epoch.
(Or isn't this what training is all about?) So I suspect there might be a
more memory-efficient way to generate sentences. (Perhaps in a trade-off
with time-efficiency.)
One thing that worries me is the BLEU score. For the first two epochs (now
in the third), the BLEU score has been 0.00. I'd hoped for the score to be
at least a little bit better than that. Could this also be because guessing
the correct word for a huge vocabulary is extremely difficult? If so,
could beam search help here? Or a different encoding of the vocabulary?
(E.g. word2vec or something similar.)
—
You are receiving this because you commented.

Ah, so adding a couple of del statements (i.e. actively stepping in to manage the memory) might already help a lot in terms of efficiency.

Does Keras now support passing generators for the validation data to keras.fit()?

Yes, or at least there's a new function called fit_generator. It has arguments for both a training data generator as well as a validation data generator. Link to models.py, which has very informative docstrings.

For your current experiments: can you cut your validation data down to a very small number (50-100 sentences) and train on --unk 3 or so? I.e. is the bottleneck for the vocabulary just due to the val array sizes?

Good idea. Will try that later today.

The idea seems to work. I started the training this evening with --small_val and (i) it didn't get a memory error after the first two epochs, and (ii) it returns nonzero BLEU scores:
INFO:Callbacks:Best BLEU: 1 | val pplx 53.43271 bleu 10.13
INFO:Callbacks:Best BLEU: 2 | val pplx 32.22855 bleu 10.27

Now I'm definitely convinced that a generator-based solution is the way to go. Then the one-hot encoding strategy should still work, even with a huge vocabulary. And the size of val will no longer be the limiting factor.

So here is a short summary of the run last night:
INFO:Callbacks:Checkpoint 1 | val pplx: 53.43271 bleu 10.13
INFO:Callbacks:Checkpoint 2 | val pplx: 32.22855 bleu 10.27
INFO:Callbacks:Checkpoint 3 | val pplx: 27.18124 bleu 8.66
INFO:Callbacks:Checkpoint 4 | val pplx: 24.51813 bleu 9.16
INFO:Callbacks:Checkpoint 5 | val pplx: 22.97953 bleu 8.70
INFO:Callbacks:Checkpoint 6 | val pplx: 21.89622 bleu 7.13
INFO:Callbacks:Checkpoint 7 | val pplx: 21.17004 bleu 8.80

It's still going, but because BLEU hasn't improved since the second checkpoint it will not go on for much longer. Anyway, this is more evidence that the model will keep training if we don't process everything all at once.

You've raised some great issues about how the data_generator is currently built. We do need to fix this if we're to ever work with substantially larger datasets.
I've identified a few key places we can reduce the memory footprint without changing the input data types. I'll fix this up sometime in the next week. But a validation / test data generator would be a great addition to the project!

Memory issues have now been addressed my merge commit 3b735585d123151f6b7c4ce8e2605094600d0141


VSCode Crashes when I use install node_modules or clone a repository using the Inbuilt Terminal (The window has crashed (reason: 'crashed', code: '5'))

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

VS Code Version: 1.63.2 (Universal)
OS Version: macOS Big Sur v11.4

Steps to Reproduce:

My VS Code started having weird behavior this entire month of January 2022. It only does it when you have an activity in the internal VS Code terminal.
Whenever I try to install node modules using the vscode internal terminal or fetch a repository from Github using the internal VS Code Terminal, it throws an error.
Sometimes it works if it is a small install for example npm install express, but when I chain multiple installs for example npm i express cors dotenv pg, it crashes.
Error is thrown: The window has crashed (reason: 'crashed', code: '5'), then I am prompted to restart VS CODE

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Corrected on_frame in doc

Couldn't get on_frame event to work, found that the docs show the event as on_frame but it is actually on_texture.

@bliz937 thanks! Don't know how this went unnoticed for so long Oo.


FormRequest Custom Messages with Rule::

Laravel Version: 5.8.21
PHP Version: 7.3.6
Description:
Within the FormRequest messages() function, any item using Laravel's internal Rules will have its message overridden by Laravel's message.
Below is an example...
class RegisterRequest extends FormRequest {

    /**
     * Get the validation rules that apply to the request.
     *
     * @return array
     */
    public function rules() {
        return [
            'club_id' => [
                'required',
                'registration_code' => Rule::exists('clubs', 'id')->where('registration_code', 5))
            ],
        ];
    }

    public function messages() {
        return [
            'club_id.required' => 'He who laughs last didn’t get the joke', // Works
            'club_id.registration_code' => 'Invalid registration code for the specified club', // Fails
        ];
}

In the example above, if the club_id is not passed, the message ''He who laughs last didn’t get the joke' will be populated in the message bag
However, if the registration_code fails the resulting message will be 'The selected club id is invalid.' rather than the expected 'Invalid registration code for the specified club'.

You can't (re)name rules like this.
As with required, you need to add the rule name to the message key:
public function messages() {
    return [
        'club_id.required' => 'He who laughs last didn’t get the joke',
        'club_id.exists' => 'Invalid registration code for the specified club',
    ];          ^^^^^^^
}

Oh ok. Thanks!
It would be nice if the documentation was more clear as to what to do with Rule:: cases.


"ValueError: Witness exceeds the given maximum length (1960>240)."

Hi,,
can anyone help me? plez,,
I want to use FCS and the length of Witness that I have been used is a list with 30 binary elements ( the bit size should be 240) but I got an assertion error about that, said  "ValueError: Witness exceeds the given maximum length (1960>240).", thus , the code forces me to enter the t (error capability- tolerance in the code-) greater than 30 (list length) may reach to 120 ! (which is wrong )
and
What dose polynomial mean in  BCH(polynomial, tolerance ), is it the same of (n) in BCH(n,k,t) as in the papers?

The second part first: You've asked the same question in #1 . Please find my answer there.
To the first part: Sounds like your list of elements is not converted into bytes as you might think. If you do not provide a custom extractor FCS uses bytes(...) on your list. Have a look at the result and I guess you'll understand what the error is ;-)

Thank you so much  for you answer..
but, how can I provide a custom extractor ?
sorry I am new on Python

@lolib11 I have a test that should demonstrate that nicely. Have a look at https://github.com/cburkert/fuzzy-commitment/blob/master/fcs/test_fcs.py#L69

@cburkert thank you , I got it,,,

@cburkert I apologize for the inconvenience and sorry for these questions ..but , same error came again!
I make a custom extractor for a numpy array
def list_extractor(value: np.ndarray) -> BitVector:     value_bytes = np.ndarray.tobytes(value)     return BitVector(hexstring=value_bytes.hex())
and I make the object like...
cs = fcs.FCS[np.ndarray](30, 3,extractor=list_extractor)
this is my array:
`ar=[1. 1. 1. 1. 1. 1. 0. 1. 0. 0. 1. 1. 1. 1. 1. 0. 0. 1. 1. 0. 1. 1. 0. 1.

1.]`
with size =30

the assertion error :
ValueError: Witness exceeds the given maximum length (960>30).
could you tell me how can I solve this problem,

@lolib11 Your error is that you assume that your bit list or rather np.array consists of 1 bit sized elements. That's not the case. There is no 1 bit sized data type in Python or IIRC in numpy.
Since you did not provide the dtype of your np.array, I'm not sure, but it seems to be an 32 bit integer. Hence 32*30=960. So the FCS's error is correct.
If you want a list of 30 bits, you cannot simply put 30 zeros and ones in a list or array.
Consider using the BitVector class directly.

@cburkert thank you for answering...
but,,if I use 960 length; the system reject all the users with multiple trails (different n,k and t)
and when I use this extractor (below) ; the false rejection become lower.. but I don't know, If it is right or wrong!
the extractor:
def list_extractor2(value: list) -> BitVector:
        return (BitVector(bitlist=value))

what about if I use it? Is it right ( by logic)  ? It is work for me,
thank you so much ,,, for you code , and helping me,,

@lolib11 I fear, I don't know enough about your use case to be able to help you here.


Some error messages displayed seem irrelevant

Bug Report

TSLint version: 5.8.0
TypeScript version: 2.6.1
Running TSLint via: CLI / Node.js API / VSCode

TypeScript code being linted
(function () {
    console.log('Hello World!');
})();

Actual behavior
without tslint.json configuration, it shows the error message:
> tslint test.ts

Could not find implementations for the following rules specified in the configuration:
    label-undefined
    no-duplicate-key
    no-trailing-comma
    no-unreachable
    use-strict
Try upgrading TSLint and/or ensuring that you have all necessary custom rules installed.
If TSLint was recently upgraded, you may have old rules configured which need to be cleaned up.

Warning: The 'no-unused-variable' rule requires type information.
Warning: The 'no-use-before-declare' rule requires type information.

ERROR: test.ts[1, 11]: expected call-signature to have a typedef
ERROR: test.ts[2, 17]: ' should be "

Why does it complains about not able to find implementations for the rules label-undefined, no-duplicate-key, no-trailing-comma, no-unreachable, use-strict specified in the configuration? As there are no custom rules defined, no extended rules as well, it should come from TSLint, but I searched through the TSLint package downloaded from npm, apart from they have been mentioned in CHANGELOG.md, nowhere else have been defined/referenced in TSLint source. So where do they come from?
But with below tslint.json configuration, it shows OK:
{
    "defaultSeverity": "error",
    "extends": [
        "tslint:recommended"
    ],
    "jsRules": {},
    "rules": {},
    "rulesDirectory": []
}

Expected behavior
without tslint.json configuration, it shouldn't complain about not able to find implementations for the those rules.

You have an outdated tslint.json in a parent directory or your home directory.

@ajafff Thanks, you r right. somehow in my user folder there is already a \Users\<MyUserName>\tslint.json before I installed TSLint this time. So close it.

Somehow I had something create a tslint.json in my home folder too and cause this problem. Would be nice if tslint had the option to not go fishing in user home directory. Or if the error message indicated where the listed rules came from. Thanks for the tip @ajafff! I was banging my head over this error.


BUG: cannot import name 'make_vec_env' in PPO2 Example

Describe the bug
I'm just starting to run through the examples in stable-baselines and was able to get the initial code examples working in the getting started section of the documentation without issue.  I am currently trying to run the PPO2 code example, but I am running into an issue and receive the following Exception:
cannot import name 'make_vec_env'
Code
The following code is a direct copy from the documentation and is currently failing
import gym

from stable_baselines.common.policies import MlpPolicy
from stable_baselines.common import make_vec_env
from stable_baselines import PPO2

# multiprocess environment
env = make_vec_env('CartPole-v1', n_envs=4)

model = PPO2(MlpPolicy, env, verbose=1)
model.learn(total_timesteps=25000)
model.save("ppo2_cartpole")

del model # remove to demonstrate saving and loading

model = PPO2.load("ppo2_cartpole")

# Enjoy trained agent
obs = env.reset()
while True:
    action, _states = model.predict(obs)
    obs, rewards, dones, info = env.step(action)
    env.render()

Traceback (most recent call last):
  File "c:\program files (x86)\microsoft visual studio\2019\community\common7\ide\extensions\microsoft\python\core\ptvsd_launcher.py", line 119, in <module>
    vspd.debug(filename, port_num, debug_id, debug_options, run_as)
  File "c:\program files (x86)\microsoft visual studio\2019\community\common7\ide\extensions\microsoft\python\core\Packages\ptvsd\debugger.py", line 39, in debug
    run()
  File "c:\program files (x86)\microsoft visual studio\2019\community\common7\ide\extensions\microsoft\python\core\Packages\ptvsd\__main__.py", line 316, in run_file
    runpy.run_path(target, run_name='__main__')
  File "F:\anaconda3\envs\envTensorflow\lib\runpy.py", line 263, in run_path
    pkg_name=pkg_name, script_name=fname)
  File "F:\anaconda3\envs\envTensorflow\lib\runpy.py", line 96, in _run_module_code
    mod_name, mod_spec, pkg_name, script_name)
  File "F:\anaconda3\envs\envTensorflow\lib\runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "C:\Users\Craig\source\repos\Stable_Baseline_testing\PPO2_Example.py", line 5, in <module>
    from stable_baselines.common import make_vec_env
ImportError: cannot import name 'make_vec_env'

System Info

Python 3.6
Tensorflow-GPU 1.14
stable-baseline 2.8.0

Rather using following:
from stable_baselines.common import make_vec_env
env = make_vec_env('CartPole-v1', n_envs=4)
You can use the below code for vectorization:
from stable_baselines.common.vec_env import DummyVecEnv, VecNormalize
env = gym.make('Walker2d-v2')
env = DummyVecEnv([lambda: env])
env = VecNormalize(env, norm_obs=True, norm_reward=False,clip_obs=10.)

stable-baseline 2.8.0

The make_vec_env helper was introduced after this version, and is currently only available on master.
You have three solutions:

Install stable-baselines from source (cf doc)
Wait a bit (the version 2.9.0 will be released on pip soon, before the end of december)
Copy the code snippet from master into your code

@araffin thank you.  I figured this was the case after investigating futher over the weekend.  Thank you for verifying.  I was able to get everything working properly once I copied the code into my own.
Thank again for the response.


Flexible way to register endpoints

Hi!
First of all, thanks for a great project!
Before version of 2.x there was a way, that allow to register views manually without having all bunch of endpoints. In version 2.x, i've notice that we're migrated to ModelViewSet and now I can't realize, how to achieve that.
I tried this workaround, but in my case it doesn't work cause I need only the user registration endpoint.
djoser_router = DefaultRouter()

djoser_router.register('users', djoser_views.UserViewSet)

urlpatterns = [
    path(
        'auth/',
        include([i for i in djoser_router.urls if i.name in ['user-list', 'user-activation']])
    ),
    ...
]

That's a very good point. I'll research and prepare a feature allowing users to register endpoints at will for version 2.1.

@dekoza thanks a lot!

see https://github.com/sunscrapers/djoser/pull/440


Issue with training

Hi! I am trying to train the vtunet but the process is not starting i nthe background with nohup. Any idea why?

Hi! Did you try with sudo?

Hi! yes, I think the directory is hardcoded in one file so I think that might be causing the issue. I'll try running everything from /home/ and let you know how it goes.

solved: change the env variables in etc/environment instead of ~./profile bashrc as stated in the readme.

question now remains how to configure for training? Whenever I run it I think I'm overflowing my GPU.. any ideas?

Hi, can you reduce the batch size and run it again? Modify batch size in this file: https://github.com/himashi92/VT-UNet/blob/main/VTUNet/vtunet/run/default_configuration.py

Hi, can you reduce the batch size and run it again? Modify batch size in this file: https://github.com/himashi92/VT-UNet/blob/main/VTUNet/vtunet/run/default_configuration.py

should be ok now if I reduce to "batch-size=1".
Thanks!

Question, the pre-determiend "Synapse"-parameters in the default_configuration.py ----- Are these defined for the "synapse multi-organ segmentation dataset"??
@himashi92

yes, the code was adapted from nnUNet and nnFormer.  We haven't trained VT-UNet on those datasets yet. But you can try those configurations to train VT-UNet on Synapse Multi Organ dataset.


10704 better odata types

Changes
⚠ May be easier to view individual commits

Improved types:

Authenticated media URLs
Geographic Hash
Cascading select Hash
Select multiple array

Better "warner" info for OptionSets in use, because I needed to find one for testing and this seemed useful!

Thanks Tom! Fixed "placeholder" naming, agree that's much better. Will merge after CI passes.


is_jitted does not properly detect if inside a jit call (tested in version 0.4.12)

Since jax 0.4.12 (could be for other versions>0.4.1) the is_jitted function does not properly detect if inside a jit call.
import jumpy

def f(a):
    if jumpy.core.is_jitted():
        print("JITTED")
    else:
        print("NOT JITTED")
    return a*2

jax.jit(f)(1) --> NOT JITTED

Changing the is_jitted function to the following resolves the problem, but it is a very hacky fix. Also not sure about backward compatibility.

def is_jitted() -> bool:
    """Returns true if currently inside a jax.jit call or jit is disabled."""
    if jp.is_jax_installed is False:
        return False
    elif jax.config.jax_disable_jit:
        return True
    else:
        return len(jax._src.core.thread_local_state.trace_state.trace_stack.stack) > 1
        # return jax.core.cur_sublevel().level > 0

I was planning on updating the whole project to jax 0.4.X as 0.4 removes DeviceArray for Array.
Therefore, I would be good to include in fix in the project when we update


python312Packages.sigstore-rekor-types: 0.0.11 -> 0.0.13

Diff: https://github.com/trailofbits/sigstore-rekor-types/compare/refs/tags/v0.0.11...v0.0.13
Changelog: https://github.com/trailofbits/sigstore-rekor-types/releases/tag/v0.0.13
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] Fits CONTRIBUTING.md.

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

Done in https://github.com/NixOS/nixpkgs/commit/d007946944cbe7ec8e83f32c6afc0b904aa63854.


Upgrade java dependencies

Upgrade the java dependencies if possible.
This one is to track the upgrades that does not require too much code/config modification.
Reporter: Gabor Szadovszky / @gszadovszky
Assignee: Gabor Szadovszky / @gszadovszky
PRs and other links:

GitHub Pull Request #398

Note: This issue was originally created as PARQUET-822. Please see the migration documentation for further details.

Julien Le Dem / @julienledem:
Issue resolved by pull request 398
https://github.com/apache/parquet-mr/pull/398
https://github.com/apache/parquet-mr/commit/6fb60857be1fed21bdacc4ce830bbf99103b6fdd


Web console: compaction dialog update

This PR updates the compaction dialog form to be better. Adds a JSON tab to make it possible to edit the raw JSON
Adds partition UI:

Raw JSON UI

I think I addressed all the feedback, thank you for the review!

I retriggered (Compile=openjdk8, Run=openjdk11) query integration test since ITWikipediaQueryTest.testWikipediaQueriesFromFile failed, which is unrelated to this PR.

Thank you for all the review!


Error while importing tickets to Azure

I have successfully exported tickets from JIRA in .JSON files but getting errors while importing them to Azure project.
Adding log report for your reference:
wi-import-log-230115-104126.txt
Adding Jira export log file:
jira-export-log-230115-104059.txt
config scrum file:
config-scrum (2).zip

What command are you using to invoke the importer?
Can you also attach the .json file for one of the failing exported workitems?
Have you read through the Overview.md and FAQ under docs before getting started with the migrator? The most common issues should be there.

@Bikrammabzone We've just released https://github.com/solidify/jira-azuredevops-migrator/releases/tag/v3.0.85, where we added more logging for exceptions raised by the workitem create endpoint. Using this, we managed to solve a related issue that my colleague ran into.
Try it and see if you can get more details about the error. Most likely it is due to some misconfiguration on the ADO process :)


start working on a blogpost that outlines r4 outcomes

Signed-off-by: Felix Kronlage-Dammers<EMAIL_ADDRESS>

/remind me to merge this PR on 2022-11-02 08:00:00

/remind-me Wednesday 9am Merge this PR

@garloff are you finished adding items to this PR?

/remind-me Wednesday 9am Merge this PR

Please not. I was waiting until @garloff finished adding items to this PR in order to then give it final polishing.

@itrich from my side, good to be published.
@garloff you want to read over the final version? Especially since I've added you as the author.

Looks really good to me now.


jkd8 win64 dragonwell mk-cacerts.sh: line 79: C:Program/bin/keytool: No such file or directory

https://ci.adoptopenjdk.net/view/Failing Builds/job/build-scripts/job/jobs/job/jdk8u/job/jdk8u-windows-x64-dragonwell/3/console
17:12:54  Done (138 CA certs processed, 29 skipped).
17:12:56  Processing certificate with alias: subject=C = BE, O = GlobalSign nv-sa, OU = Root CA, CN = GlobalSign Root CA
17:12:56  ./mk-cacerts.sh: line 79: C:Program/bin/keytool: No such file or directory

The problem is higher up:
18:06:57  Boot jdk directory: C:\Program Files\AdoptOpenJDK\jdk-8.0.275.1-hotspot\:
18:06:57  BOOT JDK: ./build-farm/make-adopt-build-farm.sh: line 118: C:\Program: command not found
18:06:57  JDK IN PATH: openjdk version "1.8.0_275"

So, the problem is somewhere in the build config for Dragonwell. The path to the boot JDK should be something like /cygdrive/c/openjdk/jdk-7, not C:\Program Files\AdoptOpenJDK\jdk-8.0.275.1-hotspot\. It's correctly set in
https://github.com/AdoptOpenJDK/openjdk-build/blob/b3fff8ed4e5a02035787726c745eee08af28c68e/build-farm/platform-specific-configurations/windows.sh#L22
and then turned into JDK_BOOT_DIR in
https://github.com/AdoptOpenJDK/openjdk-build/blob/b3fff8ed4e5a02035787726c745eee08af28c68e/build-farm/platform-specific-configurations/windows.sh#L206
but that may be too late.

Note that /dragonwell-bootstrap/jdk8u272-ga is the correct place to pick up the Dragonwell JDK8 boot JDK from. It will not bootstrap with another JDK7 unlike the other builds. Looks like something may be dereferencing a link somewhere to get to the path with spaces in it which is causing the problem.
FYI @joeyleeeeeee97

Looks to be passing that section for now, although it's tripping over somethign in the installer creation. Almost certainly an easy fix:
J:\jenkins\workspace\build-scripts\release\create_installer_windows>cd wix\SourceDir

J:\jenkins\workspace\build-scripts\release\create_installer_windows\wix\SourceDir>call powershell.exe ./CreateSourceFolder.AdoptOpenJDK.ps1
filename : OpenJDK8U-jdk_x64_windows_dragonwell_2020-12-23-17-07.zip doesn't match regex (?<jvm>hotspot|openj9)

J:\jenkins\workspace\build-scripts\release\create_installer_windows\wix\SourceDir>IF ERRORLEVEL 1 (
ECHO unzip failed
 EXIT /b 2
)
unzip failed
Build step 'Execute Windows batch command' marked build as failure

@sxa Thanks for digging into this. And I guess the error is caused when there is no such Dragonwell bootstrap image on the worker.
In https://ci.adoptopenjdk.net/computer/build-alibaba-win2012r2-x64-1/ the build seems ok.
So maybe after download that for all windows build workers that problem should be resolved.

I believe this is ok now - we haven't had any problems on the other machines after deploying the Dragonwell boot JDK :-)

I believe this is ok now - we haven't had any problems on the other machines after deploying the Dragonwell boot JDK :-)


User Page restrictions

Hi i installed this packages..
if User login as a staff he can only see uers List .. Staff can't do edit update or delete and can't see other pages how can i do that..

That's a good question and sorry I can't explain how you do that...

Have any Reference site

What reference site??

It can be done, even thought this project is better at whole sections, but if you declare each route individually, you can wrap them in separate routeNeedsPermission middleware for each CRUD operation.


Django 1.6 Deprecation Warning: django.utils.simplejson is deprecated

After an upgrade to Django 1.6, I get the following DeprecationWarning from Django when using django-disqus:
2013-11-13 11:20:12,740 WARNING /path/to/site-packages/disqus/__init__.py:5: DeprecationWarning: django.utils.simplejson is deprecated; use json instead.
  from django.utils import simplejson as json

Using the systems Python json library fixes this. Pretty simple fix.

I have just installed django-disqus 0.4.3 from pypi and ran into this problem. Since I am using django 1.7, instead of a deprecation, I am getting an error when I start the development server (simplejson has been removed from django.utils). How is the situation with django 1.7 support in general for this project?

I am in the same situation in django 1.7.

@sterago and @devstorm21: If I am understanding your comments correctly, this closed issue shouldn't be a problem for you if your install of Python has the json library installed. The fix that @derekjamescurtis made corrects the problem with the deprecated Django version of JSON. Look at the file api.py in the source:
try:
    import json
except ImportError:
    from django.utils import simplejson as json

If your install of Python doesn't have the json lib installed, then the fallback is to try and import the Django version. Failing it, you SHOULD get an import error, because you don't have either (1) the Python JSON lib installed, nor (2) the Django version that does have simplejson installed, and the django-disqus library won't work, period.
A simple test would be to open Python on the command line and type import json. What is the result of that test?

@robnewman thanks for your advise.

@robnewman but there is still one more. Please see the init.py

@devstorm21 you are correct! I missed that one. Yes - that is a legitimate bug. I will quickly fork the repo, put in a fix and send a pull request.

Pull request sent: https://github.com/arthurk/django-disqus/pull/62
Or you can use my fork for now until @derekjamescurtis merges it in: https://github.com/robnewman/django-disqus
BTW it looks like another person already fixed this issue and have their own fork due to the lack of merging by the owners: https://github.com/arthurk/django-disqus/issues/43
You might like to try that fork instead - it looks like they caught another instance of simplejson in the template tags!

Hmmm. Looking through the issue list it seems that multiple developers have sent in pull requests solving this same issue over and over again, but some miss all the import fixes. It's pretty messy. For some reason none of them have been merged by @derekjamescurtis.
I just updated my fork to fix the template tags import of simplejson to just json. The fall back is to use simplejson as json. You should be set to use my fork, but it would be nice to see this fixed in the master branch...
Let me know if you have any problems with my fork.


Add installation videos to the docs

Proposed changes:

Add installation videos the documentation

Status (please check what you already did):

[ ] added some tests for the functionality
[x] updated the documentation
[ ] updated the changelog (please check changelog for instructions)
[ ] reformat files using black (please check Readme for instructions)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.JustinaPetr seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Looking for beta testers

All PySM 2 models except HD2017 dust are implemented.
Looking for beta-testers, please look at the docs:
http://pysm3.readthedocs.org/
Try to install and run some models, if you find bugs, please open a dedicated issue.
If you have general feedback, please reply here. In particular I am looking for high level feedback:

is there any feature you need that is missing?
can the interface be simplified?
do you have suggestions for better function/classes names?
can the docs be improved?

Thanks a lot @zonca. I'll upgrade the fgbuster dependency to pysm3 and let you know if I have comments.

Now I also have a conda package: https://anaconda.org/conda-forge/pysm3
Still, for now better to build from source as the package is changing a lot, but for a quick test, the conda package is fine.

Thanks for your effort to update the PySM2 package @zonca !
So except for the improvements for engineering part, what's the main scientific differences from PySM2?

Nothing yet, but next will add all models from https://github.com/simonsobs/so_pysm_models with resolution up to 8192

@aiolasimone, just released PySM 3.2.0,
this is the first release which includes all the models, it would be nice to have more beta-testers,
install instructions at https://pysm3.readthedocs.io/en/latest/#installation


Support matching objects by label in test harness

Currently, using the test harness, it is only possible to write an assert for an object that has a deterministic name. However, many resources have non-deterministic names, but deterministic labels: PlanExecutions, Pods, etc.
We should add a way to match these objects.
A possible syntax:
apiVersion: v1
kind: Pod
metadata:
  labels:
    app: my-app
status:
  status: RUNNING

This would list pods by label and ensure that there is one matching the label with the correct status.

@zen-dog thoughts on this design?

Typically, you would solve it with a regexp on the label e.g. app: r"my-app-.[1-9]*" or smth. along those lines. Not sure how K8 idiomatic this is.

In kubernetes, you typically match a specific label so wouldn’t need a regex - e.g. the label selector from your Deployment or Instance. That said, I like that syntax a lot as a way of expressing regex in the asserts in general.. I’m not sure it’s valid YAML either, but i think we can do something similar.

Actually if we treat all strings as exact match regex, we don’t need any special syntax at all.


”Roșcani”, ”Rădeni” are listed in the drop-down menu

Description:
When user writes ”Roșcani”, ”Rădeni” the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Roșcani"

Expected results
The location appears in the drop-down menu as "Sat Roșcani, IAȘI (Roșcani)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Rădeni"

Expected results
The location appears in the drop-down menu as "Sat Rădeni, IAȘI (Roșcani)".


Unable to see GeoIP options in Grafana 7.4

The panel tab in Grafana does not show the GeoIP provider/token options in Grafana 7.4.  Is this plugin compatible with this version of Grafana?  I was sure to download the latest zip and install from that.

Hi, dodgeboy.
Thanks for your report. Actually, the plugin has not got a chance to test on Grafana 7.4 yet.
I confirmed the problem with my installation and am looking into this issue. Because I am celebrating the Spring Festival with family, the fix might take a long time.
Anyway, at a first glance, it appears to be due to some breaking changes of UI components introduced by Grafana.

Thank you for your response!  I look forward to the fix when you have time.  Enjoy your holiday!

Hello, dodgeboy.
A new beta release is here. Besides fixing this issue, it also comes with a bunch of new features, which take a longer time than expected to implement.
Since the new version has not yet been included in the official plugin repo of Grafana, installation requires manual operation and does not support the Grafana cloud. If everything goes well, I will make a stable release and update it in the plugin repo next week.
Suggestions and feedback would be greatly appreciated.

I am now able to configure GeoIP, and see the visual traceroute information on the map.  Thanks for fixing this!

I noticed that changing the Bogon filtering space option doesn't seem to save.  If i set it to none, it reverts back to "Bogon (private + reserved)" next time I enter the settings menu.

My oversight. It should be fixed by https://github.com/Gowee/traceroute-map-panel/commit/5e98274c45a2dbc38478677fe31db62fa908113d, which will be included in a later release.
Thanks again!


feature request: docker image

as all my apps are running as Docker apps,. would be great if you can create a Docker app

There is currently an image LinuxServer.io is testing which can be used: https://github.com/TheSpad/docker-adguardhome-sync
Hopefully this is something we will get into the main repository soon.

The docker image is building and running on my system.
Just note I created the config file and then just mapped it from disk to container using -v :/config/adguardhome-sync.yaml

I added a docker image that is hosted on quay.io.
See the instructions in the readme.

works great, I like the very small image 3.5mb!!


Remove dependency on Bootstrap

Currently with the combination of angular material and bootstrap css, the UI looks a bit cobbled together. By removing Bootstrap and using angular material for all components, it will make the UI look a bit more uniform.

Merged into develop.


fix(partners): Fix Laravel 11 Compatibility- Implement getAuthPasswordName() Method in OAuthUser Model

Overview
This PR addresses compatibility issues with Laravel 11 for the oauth2-merchant package by implementing the getAuthPasswordName() method in the OAuthUser model.
Problem
In Laravel 11, the Authenticatable contract introduces a new method getAuthPasswordName(). Models implementing this contract must provide an implementation of this method.
Solution
We've added an implementation of getAuthPasswordName() to the OAuthUser model that throws a BadMethodCallException when invoked, since OAuth users do not have a traditional password:
public function getAuthPasswordName()
{
    throw new \BadMethodCallException('Not available for OAuth users');
}

Changes

Implemented getAuthPasswordName() method in OAuthUser model
Added test coverage for the new method implementation

Reference
This change is based on the Laravel framework update: Laravel Framework PR #48665

/merge

https://github.com/SallaApp/oauth2-merchant/pull/35 is been merged ✅

Merge by: @EngRashaAmer


preventing vue's compileTemplate from removing style tag

@jpkleemans any info when this pr will be merged?

Thanks for your PR @DirectorRen-TV! Could you please add a test for this?

I'll try to do it, I need some time to deal with cypress


Lack of New Contributor Documentation

I've been meaning to help out with HTBox for a while now, and I've finally been able to set aside some time to start getting 2WR stood up and kicking the tires to get a feel for the app before I start looking at open issues.
There is a bit of friction getting started. I've been able to get the application running in debug, but that's about it.
I hope this doesn't get taken as "you're doing everything wrong, you should fix this for me!", but I wanted to lay out some of the difficulties I've had getting started. If anyone can help me gather some of this info, I'd be happy to flesh out some of the READMEs myself and submit a few PRs.
Auth0 Integration
In the README and in the root of the Vue project, there is an example of a .env file. While sticking that info into a new .env file makes it so you can log in to the app, there's no good documentation of how the auth works. It might be obvious if you're familiar with Vue and/or Auth0 (of which I am overly-familiar with neither). However, some documentation on what callback URLs need to be added to Auth0 (for example) would be helpful!
A sub-issue here, for me, was my lack of experience with Auth0 specifically. I don't expect a detailed primer on Auth0 (they have an excellent one on their site), but I'd love to see a surface-level description of what the environment variables do and the basic flow from App -> Auth0 -> App.
Application Architecture
There's a lot going on here. Three different applications, essentially, each with their own tech stack and runtime requirements. I know there are some UML/class diagrams/ERDs in the Wiki, but I had trouble parsing through them. Each individual README gives a brief overview of how to get its parts of the app running, but there isn't a good description of how they all fit together. A "Getting Started" doc, in the Wiki or in the root README, would help get potential contributors going a lot faster.
Contributor Guidelines
Besides the (currently empty) style guide over on the wiki, there isn't a lot of guidance for new contributors, such as:

Generally accepted contribution types (new features vs bug fixes vs documentation updates)
Where to find/ask for more info on the app's development (a Slack/Discord/Discourse/forum/whatever). Even a list of Twitter handles would help.

That's what I have for now. Like I mentioned previously, I'd be happy to start submitting documentation PRs, once I've gotten my hands a bit more dirty.

@b4ux1t3
On familiarizing yourself with Auth0, I found these Get Started with Auth0 Video Series helpful.


Improve the pagination

What's the style idea?
Can improve the pagination and home page with containers placed at a right places in a better way
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a GSSoC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

#35 Issue is already open.


classes being readded after sync

only happens sometimes...?

Fixed in recent update

Happened because Database wasn't responding fast enough. Fixed by refreshing database on start.


Update docs

Can we update the docs for this repo on how to configure for an express server? It seems everything it still written for micro :)
PS - Should we rename this library since it's not depending on micro under the hood anymore? 🤔

So long as express doesn't handle async/await we'll have the handler function with a catch: https://github.com/bufferapp/buffer-publish/blob/master/packages/server/index.js#L85-L98
I rewrote the express version here: https://github.com/bufferapp/buffer-rpc
So we should just be able to deprecate this after we swap it out.


No output produced by run_inference.sh

Thank you for producing this convenient and efficient tool!
When I run run_inference.sh, I receive the following output and no error messages:
Loading checkpoint from /home/al979/Khare/PIPPack/model_weights/pippack_model_1_ckpt.pt 	Epoch 286
An empty directory named pippack_output is also created, but no output files are generated within this directory.
The run completes without generating any error messages, but it seems to not be generating the output files.
I did do a recursive regular expression search (grep -r "kuhl") to find any paths that I needed to change to the appropriate ones for my directory and changed these appropriately.
Could you provide some guidance on how to resolve this issue?
Any advice would be greatly appreciated!

Thanks for your interest in PIPPack! I'm happy to help.
My best guess is that the inference.py script isn't finding any PDBs. Line 153
(pdb_files = glob.glob(os.path.join(cfg.inference.pdb_path, '*.pdb'))) should be searching for the PDBs containing the proteins you want to pack, so I'd make sure that inference.pdb_path in run_inference.sh is pointing to the right spot. You could also make sure that the pdb_files, proteins, and batches variables in inference.py aren't empty by adding some print statements to the code.
Hope that helps!
Nick

Thank you so much! I was able to resolve the issue with your guidance!


Development local parachain setup with zombienet

[ ] Fix: Parachain-id command not available on parachain binary, hence node not starting with zombienet.

Solutions:

[ ] try downgrading zombienet to just before polkadot v0.24
[x] Upgrade to polkadot v0.24
[ ] Completely automate current manual setup pre zombienet

The problem is a lack of support for the parachain-id subcommand

Dealing with unexpected epoch change error when blocks aren't produced for an epoch (i.e nodes are down after a while)
https://github.com/paritytech/substrate/issues/4464


Update CLI extensions available doc

Update CLI extensions available doc.
Triggered by Azure/azure-cli-extensions - TRAVIS_BUILD_ID=472728305
https://github.com/Azure/azure-cli-extensions/commit/3aa0f25caf249c3667392a5464f03ea67966919a

OPS Build status updates of commit 70f1c17:
:clock10: Preparing: average preparing time is 57 sec(s)

OPS Build status updates of commit 70f1c17:
:clock10: Incremental building: average incremental building time is 12 min(s) 21 sec(s)


fixes #2567 corrects race condition in tablet metadata verification

The diffs for this are much easier to look at when hiding whitespace diffs.  @milleruntime I think this may fix the problem you saw in #2567, but I am not completely sure yet.  I want to come back and review it myself in a few days.  My goal in this PR was to detect changes for the not just updating of the java map of files but also changes written to the metadata table.  As you identified in the issue, the existing code only detects changes to the java map but it does not detect overlapping writes to the metadata table.

Before merging this thinking next week of adding some trace level logging of the counts and running this to see if things are really working as intended.

@keith-turner wrote:

@ctubbsii any interest in looking at it again before I merge?

No, it was fine before. I skimmed, and didn't see anything substantively different from what I previously reviewed. So, unless there's something specific that you changed that you think might warrant a closer look, I'm fine with merging (the merge conflict will have to be resolved... should be easy... I think it's just indentation conflicts on most lines).


Shiny capability

Filtering by FHIR Version and Vendor supported on the Capability page. Still uses R to parse capability JSON.
Pull requests into this repository require the following checks to be completed by the submitter and reviewers.
Submitter:

[x] Administrative tasks are complete.

This pull request describes why these changes were made.
The JIRA ticket links to this PR.
The JIRA ticket for this PR is: https://oncprojectracking.healthit.gov/support/browse/LANTERN-187

[x] Tests are complete.

Tests are included and test edge cases.
Tests have been run locally and pass.

[x] Code review has been performed.

Code diff has been done and checked to ensure only expected code is being committed.
A linter for the language has been run.

Primary Reviewer:

[ ] Code works.

Code has been run locally and the basic functionality affected by the PR has been checked.
Edge cases have been checked to ensure behavior is as expected.

[ ] Code review has been performed.

Code is maintainable, reusable, efficient, and correct.
Code accomplishes the tasks purpose.
Code follows style guidance appropriate for the language, including passing linter checks.
Code is well commented (which does not mean verbosely commented).

[ ] Tests are complete.

Tests are included and test edge cases.
Tests have been run locally and pass.

Secondary Reviewer:

[ ] Code works.

Code has been run locally and the basic functionality affected by the PR has been checked.
Edge cases have been checked to ensure behavior is as expected.

[ ] Code review has been performed.

Code accomplishes the tasks purpose.

[ ] Tests are complete.

Tests have been run locally and pass.

Merger:
Prior to merging, ensure that all checks have been completed. Delete the branch after the merge
is complete.
Create a new branch <branch name>_gomodupdate. Run make update_mods to update the go.mod
and go.sum files to point to master. Create a PR. Require one sanity check reviewer.

When I went to the shiny dashboard on initial run (so my database was empty), the webpage broke saying it could not connect (it looked like docker was saying something about vendor_id?) I assume this is related to the new filtering stuff; is there a way to make it so that the webpage still loads even if the database is empty?

Yes, it should not fail when the database is empty. I think that should be a separate ticket.

I noticed on page refresh no matter what view you're on it reloads on the main dashboard page. It would be nice if it knew what view you were on for page refresh, but also this feels pretty low priority so if you think this is a good idea we could just put a ticket in the backlog.

Agreed, we can add support for updating the url to match the tab you are on. Good one to add to the backlog.


Container for rust CLI

This is fixing the problem that we do not have a container for the rust cli.
It has some bonus fixes, that we might roll out to other actions components

The CLI is shipped in a FROM scratch container. Small.
There is some magic in there, that does extract the binaries from the container and provides them as GH Action artifacts (including bom. That may not technically be neccessary but is how things "should be done"(tm) if we were to include it in a release.
In the GH Action summary page it will print the location of the temporary docker for testing (easier than scrolling through hundreds lines of code)

In the GH Action summary page it will print the location of the temporary docker for testing (easier than scrolling through hundreds lines of code)

https://github.com/eclipse/kuksa.val/actions/runs/3713953054

Everything should be modified according to your suggestions


Helm chart should always return latest image

Currently when using this from CAPI we are pinning to a specific image tag, and this will require some maintenance to keep up with new releases.
Instead, we would like the helm chart to always return the latest image by default

Sure, I think we can set it up so the Helm chart references the latest tag instead of a specific version. The only thing is that older versions of the Helm chart might not work with the latest Docker image, i.e. the older versions don't have a ClusterRoleBinding in the chart. Is there a best practice for how to deal with that? Or would it be easier for our purposes to maintain a rolling version of the chart?

Could it be possible for CAPI to get always the latest version of the chart?
If this is possible, then it doesn't matter if a specific version of the chart is pinned to an image version or if it uses :latest tag.
What is important from my PoV is that the version pinning does not happen in CAPI because most of the maintainers don't have visibility into the visualizer release cadence

Yes, that should be possible. If you do helm install without a version it will fetch the latest chart from the repo. If I remove the image tag from values.yaml and instead put it in the chart, I believe fetching the latest chart will then include the latest Docker image. The only thing is for CAPI, do we need to manually delete hack/observability/visualizer/charts for it to pull the latest version?

It seems like we are just pinning the version in CAPI now.


Fix IllegalReferenceCountException in ResponseDecoder

Pull Request Checklist

[x] Have you read How to write the perfect pull request?
[x] Have you read through the contributor guidelines?
[x] Have you added tests for any changed functionality?

Purpose
This fixes io.netty.util.IllegalReferenceCountExceptions to be thrown constantly in the ResponseDecoder. I noticed these exceptions when debugging another problem we're seeing regarding "No primary node available" errors.
Background Context
reactivemongo.io.netty.handler.codec.DecoderException: reactivemongo.io.netty.util.IllegalReferenceCountException: refCnt: 0, decrement: 1
 	at reactivemongo.io.netty.handler.codec.MessageToMessageDecoder.channelRead(MessageToMessageDecoder.java:98)
 	at reactivemongo.io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:362)
 	at reactivemongo.io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:348)
 	at reactivemongo.io.netty.channel.AbstractChannelHandlerContext.fireChannelRead(AbstractChannelHandlerContext.java:340)
 	at reactivemongo.io.netty.handler.codec.ByteToMessageDecoder.fireChannelRead(ByteToMessageDecoder.java:323)
 	at reactivemongo.io.netty.handler.codec.ByteToMessageDecoder.channelRead(ByteToMessageDecoder.java:297)
 	at reactivemongo.io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:362)
 	at reactivemongo.io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:348)
 	at reactivemongo.io.netty.channel.AbstractChannelHandlerContext.fireChannelRead(AbstractChannelHandlerContext.java:340)
 	at reactivemongo.io.netty.handler.ssl.SslHandler.unwrap(SslHandler.java:1432)
 Caused by: reactivemongo.io.netty.util.IllegalReferenceCountException: refCnt: 0, decrement: 1
 	at reactivemongo.io.netty.buffer.AbstractReferenceCountedByteBuf.release0(AbstractReferenceCountedByteBuf.java:124)
 	at reactivemongo.io.netty.buffer.AbstractReferenceCountedByteBuf.release(AbstractReferenceCountedByteBuf.java:107)
 	at reactivemongo.io.netty.util.ReferenceCountUtil.release(ReferenceCountUtil.java:88)
 	at reactivemongo.io.netty.handler.codec.MessageToMessageDecoder.channelRead(MessageToMessageDecoder.java:90)
 	at reactivemongo.io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:362)
 	at reactivemongo.io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:348)
 	at reactivemongo.io.netty.channel.AbstractChannelHandlerContext.fireChannelRead(AbstractChannelHandlerContext.java:340)
 	at reactivemongo.io.netty.handler.codec.ByteToMessageDecoder.fireChannelRead(ByteToMessageDecoder.java:323)
 	at reactivemongo.io.netty.handler.codec.ByteToMessageDecoder.channelRead(ByteToMessageDecoder.java:297)
 	at reactivemongo.io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:362)

since the decoder will release the ByteBuf itself

Be aware that you need to call ReferenceCounted.retain() on messages that are
just passed through if they are of type ReferenceCounted. This is needed as
the MessageToMessageDecoder will call ReferenceCounted.release() on decoded
messages.

I do not know (yet) whether this fixes my initial problem, but keeping it running under -Dio.netty.leakDetectionLevel=paranoid neither showed any leaks nor any IllegalReferenceCountException anymore.

How to reproduce the initial issue? Because the tests were already "paranoid" for me and not encountering that.

You just need to set the logging level to trace and connect to mongod on localhost -- this guy will show up instantly.
Unfortunately, it seems not to fix the the issue with "No primary node available" errors we are seeing.

Oh, and maybe the log level should be changed -- an exception in the decoder is rather concerning, not something to let go by undetected...

Please provide a reproducer, as I haven't seen such case locally.

I am on vacation right now. Will look into it next week.

#803 is simple reproducer for the exceptions. I am counting 27.687 errors for the integration test run with CI_CATEGORY=INTEGRATION_TESTS AKKA_VERSION=2.5.13 ITERATEES_VERSION=2.6.1 MONGO_PROFILE=default MONGO_VER=3 (here)[https://travis-ci.org/ReactiveMongo/ReactiveMongo/jobs/498030452].


how could you guys publish the document "Package an app by using Visual Studio (Desktop Bridge)" without checking everything?

This doc is a joke I have to say. So many obvious errors in this doc and there is absolutely no way you can package your app successfully using this method. I am so disappointed with this document, Microsoft. Wasted me hours of time. If you are not ready to publish this document, do not publish it!!!!!!!!!!!!!!!! Check everything before posting it. Anyone in Microsoft should read this doc, try it and see how bad it is. @zastrowm 's reference is better, Thank you! I spent hours to debug this manual and those people who wrote this should go back to school and learn how to become the qualified engineer. I usually do not leave a comment. I am always grateful to those engineers who write documents as detailed and accurate as possible. This doc completely destroys my impression with Microsoft. A wrong document is even worse than nothing. I am an engineer, so I know how hard it is to build something and achieve the goal. And before this, I always think Microsoft is a great company with a great number of highly skilled and responsible engineers. I am so disappointed to read this doc. Although I am still a student, at least I know whenever I plan to publish a product in the industry, I will check everything and make sure there is no obvious error. Because I know somebody will use it and I should help them as much as I can. I just found out the comment section in the following link is disabled? After I post this comment? Seriously? See the link: https://docs.microsoft.com/en-us/windows/uwp/porting/desktop-to-uwp-packaging-dot-net

Hi Yaquinh Thank you for sharing your honest thoughts and opinions. We're sorry for the frustration this has caused you. We've reviewed this doc and have made a number of changes that should clear up all of the confusion. If you have a chance, please do have another look and see if those updates have made a difference.

Thanks, it is fixed.


Error Reading Directory and Corrupted file when Reserializing configuration

[D] Clearing existing items from Serialization File System Data Store (if any)
ERROR: Error reading the C:\Projects\\.Sitecore\App_data\Unicorn\\Templates directory. (System.IO.FileNotFoundException)
at System.IO.FileSystemWatcher.StartRaisingEvents()
at Rainbow.Storage.TreeWatcher..ctor(String rootPath, String extension, Action`2 actionOnChange)
at Rainbow.Storage.SerializationFileSystemTree..ctor(String name, String globalRootItemPath, String databaseName, String physicalRootPath, ISerializationFormatter formatter, Boolean useDataCache)
at Rainbow.Storage.SerializationFileSystemDataStore.CreateTree(TreeRoot root, ISerializationFormatter formatter, Boolean useDataCache)
at System.Linq.Enumerable.WhereSelectArrayIterator`2.MoveNext()
at System.Collections.Generic.List`1..ctor(IEnumerable`1 collection)
at System.Linq.Enumerable.ToList[TSource](IEnumerable`1 source)
at Rainbow.Storage.SerializationFileSystemDataStore.Clear()
at Unicorn.ControlPanel.Pipelines.UnicornControlPanelRequest.ReserializeVerb.Process(IProgressStatus progress, ILogger additionalLogger)

Bug, when trying to serialise any configuration. Unicorn throws this error and when I try to access the file system directory it appears to be corrupt/Denied access to:

have been struggling with this as the only solution seems to be resetting iis, which tells me either SC or Unicorn are conflicting in locking on the file while the other is trying to process /delete it
Running:
Sitecore 8.1 U3.
Unicorn Version 3.3.1

It was an Error in my Unicorn Configuration. I have defined two serialisation configs however the datastore was being defined as a single directory and not as per config which meant Unicorn was tracking files and trying to serialise them while trying to delete them.


crypto: tinycrypt: Add macro for ecc

application may use micro-ecc for some ecc operation, which may be faster than tinycrypt, due to assamble code, but both of them have same function defination, cause build error on those case.
Summary
Impact
Testing

same with https://github.com/apache/nuttx-apps/pull/2003


fix(@schematics/angular): Correct paths for linting libraries

When creating a new library, the paths for lint were false.
Now the correct paths should be set.
CC @filipesilva

Oh, good catch! Thanks for fixing this!


DocPreviewCleanup workflow sometimes does not clean up previews

Right now, we have two documentation previews that were not deleted although the corresponding PR is not open anymore:

[ ] #703 was merged
[ ] #715 was closed but not merged

See also:
https://github.com/trixi-framework/Trixi.jl/tree/d6e8fa39a754d24216232e14671c08d040fd7cf0/previews
Both points needs some investigating to find out what really is going on there.

It looks like DocPreviewCleanup ran successfully for #703; https://github.com/trixi-framework/Trixi.jl/runs/3015705350?check_suite_focus=true. However, the preview is obviously still there...

Time (cleanup action): Thu, 08 Jul 2021 04:59:27 GMT
Time (merge): Thu, 08 Jul 2021 06:59:16 GMT+2

Thus, it looks like the final Documenter build was still running when the PR was merged. Thus, DocPreviewCleanup removed an old version of the preview but Documenter pushed a newer version afterwards. We will probably need some additional logic to prevent a race condition like this - or we need to wait until all Documenter builds are finished before merging a PR (which we should probably do anyways).

Thanks for investigating! Thus, should the rule then be to

either wait for the Documenter build to be finished before a merge, or
if one closes a PR without merge, kill the Documenter action first

?
Would another possible solution be to modify the workflows to do this automatically? That is, on a pull request being closed (or merged), delete all currently running workflows for that PR? Or make the cleanup workflow depend on all (Documenter) workflows of a PR being finished?

That sounds reasonable

Maybe something like this could be used: https://github.com/marketplace/actions/cancel-workflow-action#advanced-canceling-other-workflows

Xref: https://github.community/t/cancel-workflow-s-on-pr-being-closed-merged/189939


docs: initial

I've added Sphinx, sphinx-rtd-theme as a requirements to the poetry.
This is an initial version of the docs. I used a README.md as a baseline and reorganised it in Sphinx docs (not adding a lot of new information). It is not yet integrated with readthedocs, but that can be one of next steps when we will be happy with the docs themselves.
What you need to do to see the docs:

poetry install
cd docs
make html
open _build/html/index.html

You should see the same page from the screenshot.

I have read the CLA Document and I hereby sign the CLA

recheck

:tada: This PR is included in version 5.0.0-develop.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 5.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


node-kstat doesn't currently support node 0.10.x

Looks like it needs to make use of the NODE_MODULE macro as per http://nodejs.org/docs/latest/api/addons.html

Duplicated by #7.


RHOAIENG-2244: Make model location the same for every fetch type

Description
When cloning from git we clone the whole repo and then refer to the model with modelRelativePath. When fetching from s3 we are copying the model directly which means modelRelativePath is not needed. This ticket is to make the location of the model after each fetch step the same so the following tasks do not need to change their behaviour based on the tasked used before in the pipeline.
This pr adds a new task to be used after the git fetch step to ensure the location of the model once we have moved on from fetching is in a common location.
There are other work arounds instead of adding a new task but we decided to do this as a first solution.
How Has This Been Tested?

Run e2e tests as normal

Merge criteria:

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work

/retest

/retest


Replace Hardcoded Pathseperators

Replace the hardcoded separators "\" with File.seperator to support better platform independency.

Hi @mirko-lelansky, you can create a pullRequest if you want.


Update creation form - things I am grateful for

User story
As a user I want to get a better explanation about the steps "Things I am grateful for", get information this step is optional so it's easier for me to fill it out and complete tracking.
Subtasks

[ ] Edit field description

Added helper texts
Removed description


bitcode build fix

Is there a way for us to know when we have lowered the flakiness of our tests? Ideally, we'd remove this once we don't need it anymore, but it feels like it'll be here for a while because we won't know when it's not running.

Not yet, but it doesn't seem like a good trade-off to introduce more code to check this extra code :smile:
I don't think it's a significant cost to have this code still be here when it is no longer needed.


InvalidOperationException: The calling thread cannot access this object because a different thread owns it

Hello Samir,
I've recently updated CodeNav to v8.8.29 and I now often get the following exception in the Output Window while editing or searching in the current document:
System.AggregateException: One or more errors occurred. ---> System.InvalidOperationException: The calling thread cannot access this object because a different thread owns it.
   at System.Windows.Threading.Dispatcher.VerifyAccess()
   at System.Windows.DependencyObject.GetValue(DependencyProperty dp)
   at CodeNav.Helpers.HighlightHelper.<FindItemContainer>d__7.MoveNext() in D:\a\CodeNav\CodeNav\CodeNav.Shared\Helpers\HighlightHelper.cs:line 208
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at CodeNav.Helpers.HighlightHelper.<>c__DisplayClass3_0.<<Highlight>b__0>d.MoveNext() in D:\a\CodeNav\CodeNav\CodeNav.Shared\Helpers\HighlightHelper.cs:line 114
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at CodeNav.Helpers.HighlightHelper.<Highlight>d__3.MoveNext() in D:\a\CodeNav\CodeNav\CodeNav.Shared\Helpers\HighlightHelper.cs:line 131
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at CodeNav.Helpers.HighlightHelper.<HighlightCurrentItem>d__1.MoveNext() in D:\a\CodeNav\CodeNav\CodeNav.Shared\Helpers\HighlightHelper.cs:line 47
   --- End of inner exception stack trace ---
---> (Inner Exception #0) System.InvalidOperationException: The calling thread cannot access this object because a different thread owns it.
   at System.Windows.Threading.Dispatcher.VerifyAccess()
   at System.Windows.DependencyObject.GetValue(DependencyProperty dp)
   at CodeNav.Helpers.HighlightHelper.<FindItemContainer>d__7.MoveNext() in D:\a\CodeNav\CodeNav\CodeNav.Shared\Helpers\HighlightHelper.cs:line 208
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at CodeNav.Helpers.HighlightHelper.<>c__DisplayClass3_0.<<Highlight>b__0>d.MoveNext() in D:\a\CodeNav\CodeNav\CodeNav.Shared\Helpers\HighlightHelper.cs:line 114
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at CodeNav.Helpers.HighlightHelper.<Highlight>d__3.MoveNext() in D:\a\CodeNav\CodeNav\CodeNav.Shared\Helpers\HighlightHelper.cs:line 131
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at CodeNav.Helpers.HighlightHelper.<HighlightCurrentItem>d__1.MoveNext() in D:\a\CodeNav\CodeNav\CodeNav.Shared\Helpers\HighlightHelper.cs:line 47<---

I suspect you're trying to execute some WPF-related operation while not in the main UI Thread.
Note: using Visual Studio 2019 Professional, v16.11.4.
Please fix this. Thanks.

Please try the new release! 🚀

Hello Samir,
Thanks a lot for the bug fix. It works!
Best regards,
Didier


Fix Typo in Switch referring to this.props

It looks like this reference was missed in the conversion to a functional component.

Thanks!

@SirensOfTitan It breaks the CI, can you fix this?

@charpeni Looks like i've broken it during my previous commit. What kind of command i should run locally to prevent that? I've run npm run eslint, but it outputs a lot of errors always.

@aksonov You're right, I added Example folder into eslint scripts but not into CI since it contains a lot of errors. I'll do a pull request to remove Example folder from npm run eslint.


Core || Synthesis Wavelet Transform

Translate of the synthesis wavelet transform from s2let (c) to s2wav (base python).
Difficulty:

High

Background:

The wavelet transform can be performed as either analysis or synthesis, both with corresponding adjoint operators. Here we consider the synthesis transform which should map from wavelet coefficients to a signal on the sphere (thereby "synthesising", or re-creating , the signal).

S2let File to translate:

can be found here ignore the function on this line as it is no longer necessary.

S2Wav File location:

should be put in the main package directory here

Notes:

These functions will require some helper functions from ssht and so3 You should be able to access the necessary functions from the python versions of these packages for the time being. Look here for SSHT and here for so3 functions.

It is important that these transforms support both real and complex fields, however these two implementations are extremely similar and should (in the end) be refactored into a single function. Hence the sub-issues will be as follows:

[ ] Implement real transform
[ ] Implement complex transform
[ ] Refactor real + complex transform into a single function call (i.e. use parameter dict as a switch)


How to print the native trace when catching a promise error?

In react-native. How to print the native trace when catching a promise error and registering it with Bugsnag.notify?

Hi @nes123, would you mind adding a bit more context around exactly what you are trying to achieve? When you say "print" do you mean having it appear as part of the event information you see on the Bugsnag dashboard? Are you not seeing the stacktrace or event appearing?
https://docs.bugsnag.com/platforms/javascript/reporting-handled-errors/#handled-promise-rejections is recommended way to catch and report rejected promises for react native. Is this what you are currently doing?

Sure thank you for the reply.
AdsConsent.requestInfoUpdate({
      // debugGeography: AdsConsentDebugGeography.EEA,
      // testDeviceIdentifiers: ["TEST-DEVICE-HASHED-ID"],
    })
      .then(async (res) => {
        if (
          res.isConsentFormAvailable &&
          res.status === AdsConsentStatus.REQUIRED
        ) {
          AdsConsent.showForm().catch((error) => {
           Bugsnag.notify(error);
          });
        }
      })
      .catch((error) => {
        Bugsnag.notify(error);
      });

and getting the following error log in the bugsnag dashboard:
Error Attempted to request a consent info update but the current Activity was null.
node_modules/react-native/Libraries/BatchedBridge/NativeModules.js:106:50 promiseMethodWrapper
node_modules/react-native-google-mobile-ads/src/AdsConsent.ts:76:11 requestInfoUpdate
~/OpeningPage.js:49:4 anonymous
/tmp/hermes/staging/hermes/cmake/intlRelease/arm64-v8a/lib/InternalBytecode/InternalBytecode.js:53:16 tryCallOne
/tmp/hermes/staging/hermes/cmake/intlRelease/arm64-v8a/lib/InternalBytecode/InternalBytecode.js:139:27 anonymous
(native) apply
node_modules/react-native/Libraries/Core/Timers/JSTimers.js:248:12 anonymous
node_modules/react-native/Libraries/Core/Timers/JSTimers.js:112:14 _callTimer
node_modules/react-native/Libraries/Core/Timers/JSTimers.js:162:14 _callReactNativeMicrotasksPass
node_modules/react-native/Libraries/Core/Timers/JSTimers.js:413:41 callReactNativeMicrotasks
node_modules/react-native/Libraries/BatchedBridge/MessageQueue.js:391:6 __callReactNativeMicrotasks
node_modules/react-native/Libraries/BatchedBridge/MessageQueue.js:133:6 anonymous
node_modules/react-native/Libraries/BatchedBridge/MessageQueue.js:368:10 __guard
node_modules/react-native/Libraries/BatchedBridge/MessageQueue.js:132:4 flushedQueue
node_modules/react-native/Libraries/BatchedBridge/MessageQueue.js:128:11 invokeCallbackAndReturnFlushedQueue
the package owner says I should see logs that are coming from the native part in order to investigate it.

Hey @nes123
Unfortunately theres not a lot we can do here as we suspect there is no native trace available.
It looks like the API you are using produces the error message via:  https://github.com/invertase/react-native-google-mobile-ads/blob/main/android/src/main/java/io/invertase/googlemobileads/ReactNativeGoogleMobileAdsConsentModule.java#L86-L92
Which also appears to swallow the native errors:
https://github.com/invertase/react-native-google-mobile-ads/blob/main/android/src/main/java/io/invertase/googlemobileads/ReactNativeGoogleMobileAdsConsentModule.java#L109
You code ends with https://github.com/facebook/react-native/blob/main/ReactAndroid/src/main/java/com/facebook/react/bridge/Promise.java#L92-L100
which does not include a native Exception at all, and therefore no native stacktrace.
I would suggest going back to the package owner for more clarification.

@johnkiely1 thank you for taking the time to explain it. Very helpful!


Question:after changing parameters.yml

Hi:
I installed crm-application and it works fine.Now,i want to make a change to app/config/parameters.yml,i had changed it,but it don't take effect,how to make it work?Was it been cached?

You can run :
php app/console cache:clear --env prod
php app/console cache:clear --env dev

or delete the app/cache/dev and app/cache/prod folders

@sadortun Got it,thank you!