Set screen blank time when switching between artist and title

Issue 1
Hi Dhrone, as I posted on the Audiophonics forum, I think it would be good to be able to set a user defined period to blank the screen between the display of the artist and title information (I expect 0.5 or 1 second or so).  I expect it would work best if the scrolling stopped for a short period before the screen went  blank.  Perhaps that could be a user defined setting to go with the length of the screen blank.
Issue 2
I have adjusted your animation speed from .08 to .24.  Could this be a user defined option at the front of the script.
Issue 3
Is it possible for the script to have some display options other than artist and title.  Ones I have thought about are file type (flac, aac, mp3 etc) together with the bitrate and cpu temperature.
Thanks again for your work on this script.

Could you clarify issue 2?  Are you looking for this to be a configuration option that is outside the source code?  In other words, are you asking that I create a config file that I read at start-up?

No I think how you have set up the configuration at the top of the code is fine. It is just to have an option similar to scroll delay which you can set without going down into the code and editing it directly.
Sent from my iPad

On 8 Sep 2016, at 11:28 PM, dhrone<EMAIL_ADDRESS>wrote:
Could you clarify issue 2? Are you looking for this to be a configuration option that is outside the source code? In other words, are you asking that I create a config file that I read at start-up?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I am almost done with a complete rewrite of the code that determines what to send to the display.  It uses a pages metaphor to allow what gets displayed to be much more easily modified.  This includes the ability to add a blank page in-between the transition from artist to title.  Testing and re-integration into the main branch of the code are next but good progress has been made.

Dhrone, thanks for your efforts.  I appreciate it.
On Mon, Sep 12, 2016 at 8:44 PM, dhrone<EMAIL_ADDRESS>wrote:

I am almost done with a complete rewrite of the code that determines what
to send to the display. It uses a pages metaphor to allow what gets
displayed to be much more easily modified. This includes the ability to add
a blank page in-between the transition from artist to title. Testing and
re-integration into the main branch of the code are next but good progress
has been made.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dhrone/Raspdac-Display/issues/11#issuecomment-246311554,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVDMNFqXTeHqcaMT0jVEjipsdizpxHfdks5qpSz-gaJpZM4J3wog
.

Progress has been made.  You may want to take a look at the current version in the working branch.  Not well tested but should be close to what you are looking for.

I will give it a go but probably won't be before the weekend due to work.

On 19 Sep 2016, at 9:13 AM, dhrone<EMAIL_ADDRESS>wrote:
Progress has been made. You may want to take a look at the current version in the working branch. Not well tested but should be close to what you are looking for.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Got it working.  Thanks for your efforts.
Only difficulty was figuring out I had to run sudo pip install pylms
On Mon, Sep 19, 2016 at 10:57 PM, Ben Caldwell<EMAIL_ADDRESS>wrote:

I will give it a go but probably won't be before the weekend due to work.
On 19 Sep 2016, at 9:13 AM, dhrone<EMAIL_ADDRESS>wrote:
Progress has been made. You may want to take a look at the current version
in the working branch. Not well tested but should be close to what you are
looking for.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dhrone/Raspdac-Display/issues/11#issuecomment-247888455,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVDMNHeuNB4zOwInV2CfwxjqWqcqrDwgks5qrcWsgaJpZM4J3wog
.

As a slight complication.  When listening to CDs they have a Artist, Album
and Title.  When listening to webradio there is no Album so it comes up
blank.  I tried taking the labels off (eg "Album:") but that isn't a
perfect solution as it is better to have the labels.
Not sure if the script can tell if there is no variable information in
which case it will skip the display?
On Sun, Sep 25, 2016 at 10:02 PM, Ben Caldwell<EMAIL_ADDRESS>wrote:

Got it working.  Thanks for your efforts.
Only difficulty was figuring out I had to run sudo pip install pylms
On Mon, Sep 19, 2016 at 10:57 PM, Ben Caldwell<EMAIL_ADDRESS>wrote:

I will give it a go but probably won't be before the weekend due to work.
On 19 Sep 2016, at 9:13 AM, dhrone<EMAIL_ADDRESS>wrote:
Progress has been made. You may want to take a look at the current
version in the working branch. Not well tested but should be close to what
you are looking for.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dhrone/Raspdac-Display/issues/11#issuecomment-247888455,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVDMNHeuNB4zOwInV2CfwxjqWqcqrDwgks5qrcWsgaJpZM4J3wog
.

Actually, I just added that logic.  You can now set a page to hide when empty.  Pull the latest code that I literally just pushed.

Thanks, I will give it a go and report back.  Another issue was that when
displaying the bitrate it changes quickly from one number to another.  I
assume that is because of variable bitrate.  However, for display purposes,
I think just showing the current rate and sticking to it would be better
than changing a couple of times every second.
On Sun, Sep 25, 2016 at 11:28 PM, dhrone<EMAIL_ADDRESS>wrote:

Actually, I just added that logic. You can now set a page to hide when
empty. Pull the latest code that I literally just pushed.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dhrone/Raspdac-Display/issues/11#issuecomment-249421883,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVDMNK-7L8559frIZrCFCy3mO5o_hffxks5qtnbmgaJpZM4J3wog
.

I had to do something similar for the temperature and disk space variables.   I'm currently only updating those values once every 20 seconds.  Would a similar approach work for the bitrate problem?  What read frequency would you recommend?

How about 20 seconds or when the track changes whichever comes first.

On 26 Sep 2016, at 11:34 PM, dhrone<EMAIL_ADDRESS>wrote:
I had to do something similar for the temperature and disk space variables. I'm currently only updating those values once every 20 seconds. Would a similar approach work for the bitrate problem? What read frequency would you recommend?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Ok.  I've made the coding changes.  It will now only read the bitrate value every 20 seconds unless a page change occurs in which case it immediately refreshes the value. I am not in a position to test the change right now but will load it later today when I get off work.

When I try the latest script it starts then closes down and says goodbye.
Is that the issue you are resolving?
On Tue, Sep 27, 2016 at 12:57 AM, dhrone<EMAIL_ADDRESS>wrote:

Ok. I've made the coding changes. It will now only read the bitrate value
every 20 seconds unless a page change occurs in which case it immediately
refreshes the value. I am not in a position to test the change right now
but will load it later today when I get off work.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dhrone/Raspdac-Display/issues/11#issuecomment-249594866,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVDMNOiVEXff3KH3DjaVwT6o1g_hmma-ks5qt91PgaJpZM4J3wog
.

The current version should work but perhaps I missed something.   Can I ask you to try a few things things?  First, make sure you are running the version that was committed on Sept 27th.  Second, run the software using the command line.  From the directory that contains the RaspdacDisplay.py file run...
sudo python RaspdacDisplay.py
When the code exits it should show the error message that is causing it to exit.  Send that error message to me.  Finally, you can revert to the last version that was published on Sept 25th.  This is the one with the commit message "Fixed change in alert change logic related to suppression".   FYI, when running the code from the command line, you may want to disable the OLED service.  You can do this by running the following commend.
sudo systemctl stop oled
You can re-enable the service with...
sudo systemctl start oled

I tried the test script with no modifications and the latest one says
starting then goodbye and shuts down. I think there may be an error in it.
On Saturday, 1 October 2016, dhrone<EMAIL_ADDRESS>wrote:

The current version should work but perhaps I missed something. Can I ask
you to try a few things things? First, make sure you are running the
version that was committed on Sept 27th. Second, run the software using the
command line. From the directory that contains the RaspdacDisplay.py file
run...
sudo python RaspdacDisplay.py
When the code exits it should show the error message that is causing it to
exit. Send that error message to me. Finally, you can revert to the last
version that was published on Sept 25th. This is the one with the commit
message "Fixed change in alert change logic related to suppression". FYI,
when running the code from the command line, you may want to disable the
OLED service. You can do this by running the following commend.
sudo systemctl stop oled
You can re-enable the service with...
sudo systemctl start oled
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dhrone/Raspdac-Display/issues/11#issuecomment-250773516,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVDMNOPE0Jou8rv7AcUqPGsTj_snW9zEks5qvSlrgaJpZM4J3wog
.

I got this message.  Hope it helps:
pi@moode:~ $ sudo python RaspDacDisplay.py
Traceback (most recent call last):
File "RaspDacDisplay.py", line 841, in
prev_state = rd.status()
File "RaspDacDisplay.py", line 685, in status
self.avail = int(self.avail)
ValueError: invalid literal for int() with base 10: '208688  86'
On Sat, Oct 1, 2016 at 10:30 AM, Ben Caldwell<EMAIL_ADDRESS>wrote:

I tried the test script with no modifications and the latest one says
starting then goodbye and shuts down. I think there may be an error in it.
On Saturday, 1 October 2016, dhrone<EMAIL_ADDRESS>wrote:

The current version should work but perhaps I missed something. Can I ask
you to try a few things things? First, make sure you are running the
version that was committed on Sept 27th. Second, run the software using the
command line. From the directory that contains the RaspdacDisplay.py file
run...
sudo python RaspdacDisplay.py
When the code exits it should show the error message that is causing it
to exit. Send that error message to me. Finally, you can revert to the last
version that was published on Sept 25th. This is the one with the commit
message "Fixed change in alert change logic related to suppression". FYI,
when running the code from the command line, you may want to disable the
OLED service. You can do this by running the following commend.
sudo systemctl stop oled
You can re-enable the service with...
sudo systemctl start oled
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dhrone/Raspdac-Display/issues/11#issuecomment-250773516,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVDMNOPE0Jou8rv7AcUqPGsTj_snW9zEks5qvSlrgaJpZM4J3wog
.

Yes, very useful.  Try the current version and see if it fixes the exception.  Can you let me know what distribution including version that you are using?  FYI, I'm still working the bitrate problem.  I'll likely get a fix posted for that later today or tomorrow.  Also chasing some bugs in the playlist position logic.

Latest version posted hopefully resolves the bitrate and playlist position bugs.  Code also writes any unhandled exceptions to the log file now.  If you're code suddenly exists, you should check the end of your log file to determine what caused the exit.

Thanks, this one is working.
A couple of things that look like bugs:

I can't get type (eg AAC / Flac etc) to display at all;
the bottom line doesn't fit on the screen if there is a few items in the
playlist.

On Sun, Oct 2, 2016 at 7:11 AM, dhrone<EMAIL_ADDRESS>wrote:

Latest version posted hopefully resolves the bitrate and playlist position
bugs. Code also writes any unhandled exceptions to the log file now. If
you're code suddenly exists, you should check the end of your log file to
determine what caused the exit.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dhrone/Raspdac-Display/issues/11#issuecomment-250937884,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVDMNILQDbrkhJqxFMuBqyJdGe3PtfcBks5qvsx7gaJpZM4J3wog
.

Try the latest commit to fix the first issue.  MPD doesn't really provide type data but it does provide the sample rate, bits / sample, and channel count.  I've used this to fill the type field.  FYI, SPOP (the Spotify daemon) does not provide any type information that I've been able to find.
The second is really a preference.  The display is only 16 characters wide.  If the current song position is greater than 9 and the tracklist length is greater than 9, at least 6 characters are going to be used to display track position and track length.
'10/12 '
Song time and duration takes a total of 11 characters
'00:03/4:43'
So, we are at least one character over what's needed.  Options include:

Display everything and live with the line getting cropped (current setting)
Display everything and scroll
Remove at least one of the variables being displayed

All of that is configurable by modifying the PAGES_Play variable.  I've added a new file called "Pages Examples.py" that provides two different PAGES_Play examples.  The first is a version that shows the playlist position at the bottom of each page but truncates when the line is too long.  The second doesn't show the playlist data on each page but has a dedicated page that shows the playlist information periodically.

How about another option, instead of showing the current time in a track and then how long the track is, just do a countdown of the time in the track. So if it was 5 minutes long it would start at 5:00 and go down to 0:00.  That would save a lot of characters on the bottom part of the screen.
Otherwise, I think it is really good.
Sent from my iPad

On 3 Oct 2016, at 1:10 AM, dhrone<EMAIL_ADDRESS>wrote:
Try the latest commit to fix the first issue. MPD doesn't really provide type data but it does provide the sample rate, bits / sample, and channel count. I've used this to fill the type field. FYI, SPOP (the Spotify daemon) does not provide any type information that I've been able to find.
The second is really a preference. The display is only 16 characters wide. If the current song position is greater than 9 and the tracklist length is greater than 9, at least 6 characters are going to be used to display track position and track length.
'10/12 '
Song time and duration takes a total of 11 characters
'00:03/4:43'
So, we are at least one character over what's needed. Options include:
Display everything and live with the line getting cropped (current setting)
Display everything and scroll
Remove at least one of the variables being displayed
All of that is configurable by modifying the PAGES_Play variable. I've added a new file called "Pages Examples.py" that provides two different PAGES_Play examples. The first is a version that shows the playlist position at the bottom of each page but truncates when the line is too long. The second doesn't show the playlist data on each page but has a dedicated page that shows the playlist information periodically.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

If you look to implement a track count down option to save space on the
bottom line, it may have to default to count up if playing web radio as it
doesn't have a fixed track length.
On Monday, 3 October 2016, dhrone<EMAIL_ADDRESS>wrote:

Try the latest commit to fix the first issue. MPD doesn't really provide
type data but it does provide the sample rate, bits / sample, and channel
count. I've used this to fill the type field. FYI, SPOP (the Spotify
daemon) does not provide any type information that I've been able to find.
The second is really a preference. The display is only 16 characters wide.
If the current song position is greater than 9 and the tracklist length is
greater than 9, at least 6 characters are going to be used to display track
position and track length.
'10/12 '
Song time and duration takes a total of 11 characters
'00:03/4:43'
So, we are at least one character over what's needed. Options include:

Display everything and live with the line getting cropped (current
setting)
Display everything and scroll
Remove at least one of the variables being displayed

All of that is configurable by modifying the PAGES_Play variable. I've
added a new file called "Pages Examples.py" that provides two different
PAGES_Play examples. The first is a version that shows the playlist
position at the bottom of each page but truncates when the line is too
long. The second doesn't show the playlist data on each page but has a
dedicated page that shows the playlist information periodically.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dhrone/Raspdac-Display/issues/11#issuecomment-250976086,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVDMNJvKQUy5Eo5miZ2hulWbOXZzWdqyks5qv8l3gaJpZM4J3wog
.

Just made the change but can not test until this evening.  You now have a new variable named "remaining" that contains a formatted version of the time remaining in the current song (e.g. ##:##).  If playing content with no duration (e.g. streaming content), it switches to a count-up timer.  To use it, replace the content of all of the bottom lines to reference it instead of position.  Example:
'''
{
'name':"bottom",
'variables': [ "playlist_display", "position" ],
'format':"{0} {1}",
'justification':"left",
'scroll':False
}
'''
Should be changed to
'''
{
'name':"bottom",
'variables': [ "playlist_display", "remaining" ],
'format':"{0} {1}",
'justification':"left",
'scroll':False
}
'''
This needs to be done for every page within PAGES_Playing.

Hi Dhrone, I like the latest changes.
It works great with streaming.  However, when playing AAC files I get the
following error.
2016-10-08 22:21:10,692:INFO:Raspdac display shutting down
2016-10-08 22:21:13,698:ERROR:Uncaught exception
Traceback (most recent call last):
File "/home/pi/RaspDacDisplay.py", line 968, in
cstatus = rd.status()
File "/home/pi/RaspDacDisplay.py", line 710, in status
status = self.status_mpd()
File "/home/pi/RaspDacDisplay.py", line 541, in status_mpd
remaining = time.strftime("%M:%S", time.gmtime(int(duration - current)))
TypeError: unsupported operand type(s) for -: 'unicode' and 'unicode'
I hope this is helpful and you are able to fix it.
On Mon, Oct 3, 2016 at 8:30 PM, dhrone<EMAIL_ADDRESS>wrote:

Just made the change but can not test until this evening. You now have a
new variable named "remaining" that contains a formatted version of the
time remaining in the current song (e.g. ##:##). If playing content with no
duration (e.g. streaming content), it switches to a count-up timer. To use
it, replace the content of all of the bottom lines to reference it instead
of position. Example:
'''
{
'name':"bottom",
'variables': [ "playlist_display", "position" ],
'format':"{0} {1}",
'justification':"left",
'scroll':False
}
'''
Should be changed to
'''
{
'name':"bottom",
'variables': [ "playlist_display", "remaining" ],
'format':"{0} {1}",
'justification':"left",
'scroll':False
}
'''
This needs to be done for every page within PAGES_Playing.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dhrone/Raspdac-Display/issues/11#issuecomment-251074739,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVDMNOs5TIRNdUSXTpGWE10hrYFhT4VYks5qwNk6gaJpZM4J3wog
.

Made a quick fix that I am not in a position to test.  If you get a chance pull the current version within the working branch and try again.

That seemed to fix it.  It is working fine now.
On Sun, Oct 9, 2016 at 6:27 AM, dhrone<EMAIL_ADDRESS>wrote:

Made a quick fix that I am not in a position to test. If you get a chance
pull the current version within the working branch and try again.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dhrone/Raspdac-Display/issues/11#issuecomment-252446540,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVDMNCM8Q18gX-9POXLAuAR8tvK4m2hIks5qx_yggaJpZM4J3wog
.

Ok.  I think the set of issues that was raised at the top of this thread have been successfully resolved so I am closing this issue.  Please open up new issues as needed.  Thanks!


How to prepare abc_complete.fasta?

Hi,
Abc_complete.fasta was created by the method in paper. But i can't find scripts to do this.
Can you guide me to do this?
Others, abc_38_both_pm_update.uniq and abc_v14.uniq, how can we get these files?
Very thanks.

hey @lijanan ! I know just as much as you about the manuscript, unfortunately the work is not very reproducible.  The most I can do is direct you to https://software.broadinstitute.org/cancer/cga/polysolver. If you can reach the authors by some manner via a help forum or contact, please come back and post what you learn here. Sorry I can't be of more help!

@vsoch, very thanks quickly reply. I try to contact authors.


Can FlutterEngineGroup cache a FlutterEngine in advance?

Use FlutterEngine, I can cache an engine in advance so that the Flutter interface opens faster when I use it.
But using FlutterEngineGroup cannot cache one in advance 。
why？if can。how to do。

Hi @Dboy233 Can you be more specific on the issue? Are you implementing multiple Flutters?

@huycozy
Business:
app UI1 button click->open Flutter UI1 and click button -> open appUI2 and button click -> open Flutter UI2
When using FlutterEngineGroup. Only when opening the first FlutterUI, the Flutter is delayed opening due to the creation of the first Flutter Engine within the Group. Open the Flutter page again because sharing resources creates a new engine very quickly.
So I want to make a FlutterEngine when I initialize FlutterEngineGroup, like I've cached a global engine.

It seems this issue only happens in debug mode, it's faster in release mode, please see this.
Also, this is a known issue at https://github.com/flutter/flutter/issues/86231 (is filed from https://github.com/flutter/samples/issues/779) but can not reproduce it. It would be great if you could leave there your case with a completed and minimal reproducible code sample so that others can take a look at it.
I will close this issue in favor of https://github.com/flutter/flutter/issues/86231, so please follow up on it for further updates. Thank you!


(Read-only) typeclass support for case class with private constructors

Many typeclass only require reading fields from their input and do not need to construct any values (e.g. Json Encoders, Show, Eq). Previously, Magnolia couldn't derive instances where the case class constructor isn't accessible. This is due to needing to implement methods in CaseClass that construct values.
This patch introduces a way for users to signal that they do not need to construct any values for their type class. To opt in, user change the parameter type of the combine method they have to implement from CaseClass to ReadOnlyCaseClass.
+ def combine[T](ctx: ReadOnlyCaseClass[Eq, T]): Eq[T] = new Eq[T] {
- def combine[T](ctx: CaseClass[Eq, T]): Eq[T] = new Eq[T] {
  ...
}

It seems like it has the happy side-effect of reducing the compile time of read-only type class derivation by roughly 30% on GraalVM CE 20.0.0. (from my totally scientific method of running bench/clean ; bench/compile many many times)
I have added a bench module in a separate commit you can cherry-pick and compare yourself :)
[ ] Remove fixmes
[ ] Better name for ReadOnlyParam?

I'm super impressed with this, @jatcwang! Thank you so much! The 30% performance improvement is particularly exciting. Hopefully it will translate to a couple of seconds saved for compiling Fury...
Do you want to make the tiny changes I mentioned, and then I'll merge it. :)

Wonderful. Thank you so much! :D

I've published version 0.14.0. Not sure if it's available yet, but it's not listed in the Maven Central Search directory...
https://travis-ci.org/github/propensive/magnolia?utm_medium=notification&utm_source=github_status

@jatcwang @joroKr21 I've just tried using this and it seems I am losing the annotations on parameters (only on parameters, seems okay on the case classes themselves). I looked at it in debug and it seems that ReadOnlyParam#annotations returns an array of array of annotations (instead of an array of annotations).

Yes, seems like a bug. Fix incoming.

Sorry! Definitely should've added more tests. (I don't remember touching annList at all though..!)

Thanks for the quick fix! All my tests are green with the new version 😄


Use owner name as site title for User and Organization sites.

Using the repo name as the site title makes sense for Projects, which can choose their own repo names. But User and Organization sites must use repo names like {owner}.github.io which:

Does not look good as a browser tab title, bookmark, or header for a page that is supposed to be about a person or organization.
Conveys no useful information, as it is unlikely that anyone would clone or fork such a repo.
Looks confusing for users or organizations that use custom domains (CNAMEs), and have no other content indicating their site is hosted by GitHub.

The display name of the user or organization is a better choice. This is an optional property, so fall back to (login) name if display name is not present.

Alternatives considered:

Overriding the site title in _config.yml. This may not be considered as engineering best practice. Configuration files should specify machine behavior, not serve as containers for user-visible content.


Documentation - Add recent third party tutorials

Quick update to add a few recent tutorials that have appeared online.

Thanks @zerolab I'll merge in.


High volume messages emitted from the consumer affect clients

In some circumstances I've seen the consumer client emit 'message' events such that there is not ability for the client service to handle other asynchronous events - thoughts?
Looks like it's somewhere in:
setImmediate(function () { this.handleReceivedData(socket);}.bind(this));

I was running into the same issue myself.
What I ended up implementing is a batch consumer, wrapping the High Level Consumer, which in a nutshell will fetch a batch of messages, and keep calling pause()/resume() when the batch starts/finishes. It also commits the offset at the end of the batch processing. In concept is similar to the KafkaSource implementation in Apache Flume .
If this sounds interesting, I could polish it a bit and create a PR.

@ismriv Yes, I'm curious!

@ismriv Yes, please.
Also how to reproduce this issue, could you file a test example?

ismriv  - I have done the same thing:  Set the max fetch size to be 1X or greater multiplier.  So If a message is 128 bytes.  I set the max fetch size to 128 bytes and  then pause and resume on consumption.  This ensures that I can process one and only one message at a time.  This is the only way to get serialized message consumption with node.js UNLESS your consumer.on('message') does not have callbacks in its call chain or directly in its body.  When in order consumption is desired from the consumer this is the only way.  Expanding out from 128 bytes to 256 bytes implies 2 messages at a time.  Thus fetch size and pause/resume are the keys to controlling consumption parallelism in Kafka node.


loss

I don't know how much your loss has reached in the end? My ID loss is always 11, and my loss is always 17 and 18. @dingwoai

this issue may help #4


Fix always removing current ProfileNode

We weren't checking if ProfileNode::ret returned true. In other
words, we'd move back up the tree early for recursive calls.
This resolves issue #4.

@homu r+

:pushpin: Commit ead90fc has been approved by cmr

:hourglass: Testing commit ead90fc with merge d7f03b8...

:broken_heart: Test failed - status


dotenv not working for sqlx database ... CLI commands

Currently dotenv isn't working for sqlx database ... commands. This is a regression caused by commit 0e51272. (That commit was created as a response to #1391.)
For example, testing on the master branch that includes 0e51272 :
% sqlx database drop
error: The following required arguments were not provided:
    --database-url <database-url>

USAGE:
    sqlx database drop [FLAGS] --database-url <database-url>

For more information try --help

Testing on a similar branch that excludes the 0e51272 commit:
% sqlx database drop
Drop database at postgres://foo_dev@localhost/foo_dev? (y/n): y

The .env file for both is:
DATABASE_URL=postgres://foo_dev@localhost/foo_dev

Please see the PR over at #1432.

Closed by #1432


Yandex turbo pages support

This is not an issue, but more of a question:)
Yandex has recently added support for so-called "Turbo" pages:
https://yandex.com/support/webmaster/turbo/
The purpose of these pages is the same as of Google AMP, but the implementation is quite different.
Basically, it is an extension to the RSS 2.0 protocol (unfortunately, it is not backward compatible):
https://yandex.com/support/webmaster/turbo/feed.html
The question is: do you mind if we create a pull request adding Turbo RSS feed support to Ghost?

Hey @ameshkov 👋 I appreciate that this is something you'd like to make available however we won't be accepting PRs for any specific feed formats. We are currently working on ways to make it possible for blog owners to modify the default RSS feed output and add any other feeds they want via config+theme files, once this lands you'll be able to create a Yandex specific feed for your site.


Yandex map in modern interface

Hi! how I can use yandexmap in modern interface?
Thanks in advance!

It's not supported. The problem is that Yandex maps are not using a standard projection, so we can't use them currently.

You can vote for the feature here as it's still under consideration:
Vote

Closing for now as it's not possible to support. We can revisit it later if projections become available.


⚠ Bofrost.at has degraded performance

In 3e2e59b, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1583 ms

Resolved: Bofrost.at performance has improved in beab98e.


Meaning of adjustedtime

Can plz anybody tell me what is the real meaning o the field "adjustedtime".
Which unit does this field have?
Which time reference?
further more it is really confusing, why is there 6 (X,YZ) cooridnates?
I am working on PCAP-File of an Velodyne VLP16 with an attached GPS-Synchro., after exporting i get following CSV:
"Points_m_XYZ:0","Points_m_XYZ:1","Points_m_XYZ:2","X","Y","Z","intensity","laser_id","azimuth","distance_m","adjustedtime","timestamp","vertical_angle"
0.02177112363278866,5.669942855834961,-1.519267797470093,0.02177112446171241,5.669942802707069,-1.519267794751797,6.0,0.0,22.0,5.87,71194029025.0,2794029025.0,-15.0
0.05673674494028091,14.13373279571533,0.2467072159051895,0.05673674590476176,14.13373314063019,0.2467072173974397,4.0,1.0,23.0,14.136,71194029027.0,2794029027.0,1.0
0.02680676616728306,6.399606227874756,-1.477478504180908,0.02680676556020489,6.399606441468833,-1.477478524930505,6.0,2.0,24.0,6.568000000000001,71194029030.0,2794029030.0,-13.0
0.1157446429133415,27.6318359375,1.448135852813721,0.115744644326603,27.6318368112884,1.448135909242256,4.0,3.0,24.0,27.67,71194029032.0,2794029032.0,3.0

Thanks

Hey @ArashJavan . Where you able to figure out the answer? Please do let know if you did.

@rageobi not really, so I decided to write my own Veloparser.

Ah, right. I have a collected dataset in the veloview format.  And was looking to understand the headers. Cheers, thanks anyway


Mobile menus 2.0.

A new and improved menu system for mobile, intended to provide a more useful API and to iron out some of the inconsistencies with desktop:

Menus and MenuButtons no longer requires a model.
Menus and MenuButtons can be provided menu items as props.
MenuButton uses popperjs and blueprint-style position system, to position the menu relative to the button.
Removed AppMenuModel in favor of passing extraItems to AppMenuButton via AppBar. This is consistent with Desktop.

See issue: https://github.com/xh/hoist-react/issues/2230
See Toolbox PR: https://github.com/xh/toolbox/pull/457
Hoist P/R Checklist
Pull request authors: Review and check off the below. Items that do not apply can also be
checked off to indicate they have been considered. If unclear if a step is relevant, please leave
unchecked and note in comments.

[ ] Caught up with develop branch as of last change.
[ ] Added CHANGELOG entry, or determined not required.
[ ] Reviewed for breaking changes, added breaking-change label + CHANGELOG if so.
[ ] Updated doc comments / prop-types, or determined not required.
[ ] Reviewed and tested on Mobile, or determined not required.
[ ] Created Toolbox branch / PR, or determined not required.

If your change is still a WIP, please use the "Create draft pull request" option in the split
button below to indicate it is not ready yet for a final review.

Pull request reviewers: when merging this P/R, please consider using a squash commit to
collapse multiple intermediate commits into a single commit representing the overall feature
change. This helps keep the commit log clean and easy to scan across releases. PRs containing a
single commit should be rebased when possible.

Nice -- looking forward to reviewing!


No tests? It’s broken

Added initial set of test cases

Merged your tests into the main repo. Thanks for writing them.


can not set the color and font of python in vs using PTVS!

while in vc++, i can set the code color and font size, there is no way to set the python code color! i look for it in the " tools--enviroment--fonts and colors " and " tools--python tools" of vs2013, and find nothing to set!

Hi @genliu777, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

This is an issue, not a pull request - please file it as such.


Ensure Bevy's rendering byte usage is little-endian

Objective

Fixes (partially) #15701.

Solution

Use little-endian bytes over native-endian bytes where applicable.

Testing

Ran CI.

Open Questions

Should we config-gate these for big-endian targets? It looks like there are very few targets that use big-endian.

IMO no cfg gating for now. If users on big-endian platforms ever appear, we can get them to tell us what's broken.

@clarfonthey can I get your review here?

Edited the PR description to ensure the linked issue stays open :)


Warm start ADMM

Summary
Introduced a warm start ADMM where a relaxed (continuous) problem is solved first and the solution is used to initialize the state of ADMM.
Details and comments
Added a parameter warm_start to ADMMParameters to enable this feature. When it set to True the state of x0, u, z is initialized from the relaxed problem solution.

@manoelmarques could you please check the logs:
 ERROR: Cannot uninstall 'certifi'. It is a distutils installed project and thus we cannot accurately determine which files belong to it which would lead to only a partial uninstall.
##[error]Process completed with exit code 1.

Can you also add a reno release note since this a user facing change that should be included in the main release notes.

Can you also add a reno release note since this a user facing change that should be included in the main release notes.

yeah, forgot about reno.

Can you also add a reno release note since this a user facing change that should be included in the main release notes.

Updated.


Refactor gremlin-driver to make SimpleSocketServer accessible to all GLV's

TINKERPOP-2819
Refactors gremlin-driver by extracting any classes shared by both driver and server (serializers etc.) into a new gremlin-util module.
Note: In this PR the classes are moved to a new module but still remain as part of the tinkerpop.gremlin.driver package. This is to prevent any breaking changes in 3.5.x. I will follow up with a PR for 3.7.x which moves these classes to a tinkerpop.gremlin.util package and updates any imports of these classes within tinkerpop.
Extracts SimpleSocketServer and it's initializers from gremlin-driver into a new module gremlin-tools/gremlin-socket-server.
Dockerizes the socket server and decouples it from the java driver.

LGTM


Upper bound on source log scan progress

can be configured with eventuate.log.replication.remote-scan-size
closes #236

LGTM


io.gatling.http.client.WebSocketListener supports onPong() but should implement onPing

Websocket server side use to sends "ping" to websocket clients and expect a pong as a reply to check that connection is still alive.
However, current WebSocketListener abstract class and its scala implementation only offers onPong.
We should implement both onPing and onPong on listener and implement onPing to send a Pong.

Why? Gatling automatically replies to ping frames with pong ones: https://github.com/gatling/gatling/blob/master/gatling-http-client/src/main/java/io/gatling/http/client/impl/WebSocketHandler.java#L139
Looks like the expected behavior to me.

Ah, there’s probably an issue. It probably should be a writeAndFlush and not just a write there.

Hello,
Here is an example : let's consider a websocket endpoint offered by a
microservice in the cloud (let's say a chat). Each connected client opens a
websocket session that usually take some resources on server side. Some
real life experiences showed us that these clients sometimes disconnect
while server side state still "thinks" that its client is connected and
therefore it doesn't free some resources.
To circumvent this issue, we use to implement a ping long initiated from
the server side. Microservice sends a ping from time to time and expects
its Client to listen to "onPing" and reply a Pong to server.
This is in place and now I'm using Gatling to simulate some clients.
However these client do never answer to server pings. Server side then
disconnect them all. While checking into the source code I think that it's
because Gatling web socket listener only listens to onPong messages as of
today.
By listening to onPing and replying Pong, it'll solve the issue.
I may try to do a pull request on that. However I'm not used to sbt(yet).
Do you think it's something relevant to fix or add to Gatling. For us it
would really be great
Le ven. 19 avr. 2019 à 18:51, Stephane Landelle<EMAIL_ADDRESS>a écrit :

Why? Gatling automatically replies to ping frames with pong ones:
https://github.com/gatling/gatling/blob/master/gatling-http-client/src/main/java/io/gatling/http/client/impl/WebSocketHandler.java#L139
Looks like the expected behavior to me.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gatling/gatling/issues/3715#issuecomment-484954647,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE6EOIH2VOPERRHLMNKWDSLPRHZ7NANCNFSM4HHE2E3Q
.

Please properly read my previous answers.

Oups you're right on that. Pong should just be flushed.
I'll have to rewrite the issue title to be more meaningful
Le ven. 19 avr. 2019 à 19:14, Stephane Landelle<EMAIL_ADDRESS>a écrit :

Please properly read my previous answers.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gatling/gatling/issues/3715#issuecomment-484960479,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE6EOIGWETWBL47WHNGGPGDPRH4WLANCNFSM4HHE2E3Q
.

Hello,
Just to let you know that i fixed gatling code in my local workstation and tested the change (replace write by writeAndFlush). Now, my gatling simulation works perfectly and my clients don't get disconnected. So I confirm that you spotted the root cause and that we need to writeAndFlush pong so that it is taken into account.


controlPointStretch option not published to npm

I see in the current source code and demo that controlPointStretch is an option, but it seems like this has not been published to npm or updated in the README documentation. Would love to see that if possible!

Hi @ChrisShank, you can now use it with the new version v0.2.0
By the way, I saw your tweets about bringing arrows to web components. And I'm looking forward to seeing that we have an easier way to use interactive visual elements on the web.

Amazing, thanks so much! Very excited about the arrow stuff too, thanks for making my life a little easier


3 1 2

Reminder to check all libraries are called correctly for functions used (e.g. DT for datatable)

Code review
Documentation

[ ] README file for the indicator - the name of the file must be edited to be README.md so it renders as default at the bottom of the folder's page on the online repository. Currently you have to open the file to see the contents.
[x] Example input and output that run successfully locally.
[x] Section at the bottom of README - "Code edits that may be needed" - I've found solution for that one, see review comments below

Functionality

[x] Installation: Installation or build of the code succeeds.
[x] Guidelines: New code conforms to the project contribution
guidelines.
[x] Example data: Example output generated using the example input matches the given example output.
[x] csv formatting: The csv output is correct, given general guidance for csvs (e.g. columns are in
a logical order, values within columns are in the correct order, sentence case is used, correct terms, observation status column is correctly filled in).
[x] Generally it runs fine with the test and 2021 (most recent) data, but things break when the input format varies slightly as it did in 2020. So some fixes are necessary in case they change the formatting again next year (see review comments below)

Final approval (post-review)
The author has responded to my review and made changes to my satisfaction.

[ ] I recommend merging this request.

Estimated time spent reviewing: #3h

Review comments

as the old data is not recalculated (spreadsheet contains just 1 year of data), I downloaded the 2020 source to check plots. The variation in column names and rows caused issues, some of which you outlines in the README. There is a function in SDGupdater package that can help with standardising the column names.
first change source data headings to lower case, to avoid issues with any variation of that in the future, e.g.
names(source_data) <- tolower(names(source_data))
then change the names of the columns with the rename_column function in SDGupdater package (see explanation in the common code dictionary or by typing ?rename_column in R after you have loaded the package). This will essentially future-proof any slight changes in column names and also if there are any notes attached to the name (as is the case in the 2020 data). You'll have to do it separately for each column name (I think). For example:

source_data <- rename_column(source_data, primary = "place of birth", alternate = "birth", new_name = "place of birth")
source_data <- rename_column(source_data, primary = "all ages", new_name = "mothers all ages")
source_data <- rename_column(source_data, primary = "under 20", alternate = "under", new_name = "mothers under 20")
source_data <- rename_column(source_data, primary = "20 to 24", alternate = "24", new_name = "mothers 20 to 24")

etc....
Note that I haven't put alternative names for all in the function arguments, just because it seemed like for those there is unlikely to be anything much different. If the name is as expected, the function won't do any changes. I didn't complete the renaming for all columns, but hopefully this is enough to demonstrate. It should make sense once you read the function documentation and run the examples, but if not let me know and we can run through it together!

then at line 10 you should change "Place of birth" so it's "place of birth" as we have un-capitalised all headings in the step before.

this sorts the column name issue but currently things can still break at line 19 (pivot_wider) when the source data has empty cells in "Number of previous live born children" - this creates duplicated values with NA entries. I think the best place to correct this would be in the main_data variable before the long_format transformation (so after your current line 13).
Perhaps identify if there are empty fields and if yes, then populate the empty fields for that column by carrying over the cell name until you hit a different name, then carry over again etc. Hope that makes sense! I can't think immediately of the code solution, so only outlining the idea here. Once this is sorted, it should run smoothly and be more future-proofed (I tested it on the 2020 spreadsheet by manually correcting all these fields before reading the input, and it works fine! So the correction needs to be done from within update_3-1-2.R).

code style improvements - just to say that sometimes the comments run too far to the right of the script page, so not very readable. As a rule of thumb everything in the script should be placed to the user doesn't need to scroll horizontally to see the code. So maybe use more line breaks.

Please let me know if any of this is unclear, and well done! :)

@ANikolova22 addressed comments, new program works for 2019 and 2020 data as well

Great work, @alicampbell2! Works fine with the two inputs (2020 and 2021), I didn't get any errors and QA file looks fine :) I'll approve for merging. It looks like there will be a conflict with the main branch. That's because Michael and I changed something in the update_indicator_main script, but it will not affect anyone's code. Resolving should be straight forward
I'll see if I can quickly resolve this as part of this merge to main and then maybe give heads up to the rest of the team in case they get the same.
Thanks!


ImportError on pandas import

Hi All,
When I try running any cgat script that requires pandas I get the following error:
RuntimeError: module compiled against API version 0xb but this version of numpy is 0xa
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/mbp17idy/.conda/envs/sharc/lib/python2.7/site-packages/pkg_resources/__init__.py", line 570, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "/home/mbp17idy/.conda/envs/sharc/lib/python2.7/site-packages/pkg_resources/__init__.py", line 2687, in load_entry_point
    return ep.load()
  File "/home/mbp17idy/.conda/envs/sharc/lib/python2.7/site-packages/pkg_resources/__init__.py", line 2341, in load
    return self.resolve()
  File "/home/mbp17idy/.conda/envs/sharc/lib/python2.7/site-packages/pkg_resources/__init__.py", line 2347, in resolve
    module = __import__(self.module_name, fromlist=['__name__'], level=0)
  File "/home/mbp17idy/devel/CGATReport-0.3/CGATReport/build.py", line 40, in <module>
    from CGATReport import report_directive, clean, Utils
  File "/home/mbp17idy/devel/CGATReport-0.3/CGATReport/report_directive.py", line 21, in <module>
    from CGATReport import Config, Dispatcher, Utils, Cache, Component
  File "/home/mbp17idy/devel/CGATReport-0.3/CGATReport/Config.py", line 24, in <module>
    exec(compile(open("conf.py").read(), "conf.py", 'exec'))
  File "conf.py", line 18, in <module>
    import CGATPipelines.Pipeline as P
  File "/home/mbp17idy/devel/CGATPipelines/CGATPipelines/Pipeline/__init__.py", line 220, in <module>
    from CGATPipelines.Pipeline.Database import *
  File "/home/mbp17idy/devel/CGATPipelines/CGATPipelines/Pipeline/Database.py", line 33, in <module>
    from CGAT import Database as Database
  File "/home/mbp17idy/devel/cgat/CGAT/Database.py", line 14, in <module>
    from pandas import DataFrame
  File "/home/mbp17idy/.conda/envs/sharc/lib/python2.7/site-packages/pandas-0.20.3-py2.7-linux-x86_64.egg/pandas/__init__.py", line 35, in <module>
    "the C extensions first.".format(module))
ImportError: C extension: umpy.core.multiarray failed to import not built. If you want to import pandas from the source directory, you may need to run 'python setup.py build_ext --inplace --force' to build the C extensions first.

If I import pandas from an interactive python interpretor, it works fine. If I do the following, I get the error:
>>> __requires__ = 'CGATReport'
>>> from pkg_resources import load_entry_point
>>> load_entry_point('CGATReport', 'console_scripts', 'cgatreport-build')()

(I know this is a CGATReport Error, but it is also coming through, for eg bam2geneprofile)
Any ideas appreciated!

Hi Ian,
After sourcing your environment, could you please send the output of:
which python

python -c "import pandas"

Also, it might be a problem with the LD_LIBRARY_PATH, so please have a look at:
env | grep LD_LIBRARY_PATH

Best regards,
Sebastian

Hi Sebastian,
I think we've sorted it out.
Turns out that we had two versions of both pandas and numpy installed in the same site-packages directory. One came from the conda build. The other, I think, came from trying to install CGATReport from the github to get an up-to-date fix for something. Doing this caused it to redownload and install many of its dependencies from source, including different versions of numpy and pandas, leaving two installed versions. The naitive python import function was picking up one version, but pkg_resources was picking up the other.
We solved this by doing
conda uninstall --force pandas
conda uninstall --force numpy
pip uninstall pandas
pip uninstall numpy
rm <path_to_site_libs>/pandas* -r
rm <path_to_site-libs>/numpy* -r

conda install pandas

Thanks!


[docs] Fixing indentation in multiple exposed ports example YAML.

The Problem/Issue/Bug:
The code sample at https://ddev.readthedocs.io/en/latest/users/extend/custom-compose-files/#docker-composeyaml-examples has an indentation problem in the labels section.

How this PR Solves The Problem:
Reformats the whole section to use 2-spaces like the other examples on this page.

Interestingly enough, that can be an array as well, in which case it could either have or not have indentation. But we've always used the object syntax.

Thanks @rfay!


Enum + Set is broken when used via an Isolate

$ dart --version
Dart VM version: 2.1.1-dev.0.1.flutter-2cb346bd0c (Tue Jan 8 15:52:54 2019 +0000) on "macos_x64"

import 'dart:isolate';

class Content {
  Content(partOfDay) : this.partOfDay = Set()..add(partOfDay);
  final Set<PartOfDay> partOfDay;
}

enum PartOfDay {
  morning,
  afternoon,
  evening
}

foo() async {
  final r = ReceivePort();
  await Isolate.spawn(bar, r.sendPort);
  final Content content = await r.first;

  Set<PartOfDay> pod = Set()..add(PartOfDay.afternoon);
  print('A: ${content.partOfDay}');
  print('B: $pod');
  print('C: ${pod.intersection(content.partOfDay)}');
  print('D: ${content.partOfDay.intersection(pod)}');
}

bar(SendPort sendport) {
  sendport.send(Content(PartOfDay.afternoon));
}

main() async {
  foo();
}

Output:
A: {PartOfDay.afternoon}
B: {PartOfDay.afternoon}
C: {}
D: {PartOfDay.afternoon}

Expected output:
A: {PartOfDay.afternoon}
B: {PartOfDay.afternoon}
C: {PartOfDay.afternoon}
D: {PartOfDay.afternoon}

This looks similar in scope to https://github.com/dart-lang/sdk/issues/23244
I originally encountered this problem in a Flutter project, and reported it as a Flutter bug: https://github.com/flutter/flutter/issues/26013

This is going to be non-trivial to fix. Looks like maps are special cased in the VM as with a native RawLinkedHashMap, and RawLinkedHashMap::WriteTo copies the key and values one by one (which takes care of the rehashing).
Sets look like they're entirely implemented in Dart, so they get the default bitwise copy. So I think fixing this will require a native RawLinkedHashSet class (plus a ton of plumbing).

As an alternative, perhaps a warning or error could be given when sending an Enum across isolates?
The problem I had was not that I needed enums to work, but that it took significant effort to find out that they weren't working in the case of Sets.


mps2_an385: use default stack sizes

Unclear why this change was made, should never have been done.
Signed-off-by: Andrew Boie<EMAIL_ADDRESS>

on a second thought @andrewboie i wonder if these defines were here due to COVERAGE
(in case COVERAGE, itself does not assign new values to the stack sizes...)

On x86 we have a variant target qemu_x86_coverage which has all the extra stack sizes captured in its defconfig.
This is better because it gets built with CI with coverage turned on, catching coverage issues on x86 during sanitycheck runs. mps2_an385 doesn't enable coverage so if there are problems they are not tested. I think these are "dead" configuration values for mps2_an385.
IIRC mps2_an385 is one of the default targets and one of the few to support emulation of memory protection, we need to know when any default stack sizes are too small.


Fix bug where target configs are used by host scripts when cross compiling.

This fixes a bug where target configs accidentally get picked up for host scripts when cross compiling from hosts that have the same architecture as their targets.
This is not fully backwards compatible, stability guidelines allow breaking changes that fix unreliable behavior however, per:

Changes to resolve possibly unsafe or unreliable behavior.

Note that this change will break some workflows, but will also fix workflows that are currently broken due to the current unreliable behavior.
This is essentially the bug fix part of #9322 separated out from the host table feature addition.

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @alexcrichton (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

As I think the discussion clarifies on #9322 this I don't think is a standalone piece of that change because it's still a breaking change for users relying on the previous behavior. I'm gonna close this because I don't think this being a separate patch will be part of the final transition plan, but work can of course continue on the original PR.


paper-plugin Support

When trying to use bStats in papers new classloader. It fails to start because it fails its relocation check despite that as far as I can tell it should work like this.
Example implementation
public void classloader(@NotNull PluginClasspathBuilder classpathBuilder) {
        MavenLibraryResolver resolver = new MavenLibraryResolver();
        // Paper repo used for bStats
        resolver.addRepository(new RemoteRepository.Builder("paper", "default", "https://repo.papermc.io/repository/maven-public/").build());
        // Add dependencies
        resolver.addDependency(new Dependency(new DefaultArtifact("org.bstats:bstats-bukkit:3.0.2"), null));
        // Add dependencies to classpath
        classpathBuilder.addLibrary(resolver);
}

results in this error
[15:27:44 ERROR]: Error occurred while enabling SlashSlime v1.0.0 (Is it up to date?)
java.lang.IllegalStateException: bStats Metrics class has not been relocated correctly!
        at org.bstats.MetricsBase.checkRelocation(MetricsBase.java:251) ~[?:?]
        at org.bstats.MetricsBase.<init>(MetricsBase.java:121) ~[?:?]
        at org.bstats.bukkit.Metrics.<init>(Metrics.java:74) ~[?:?]
        at net.bonn2.slashslime.SlashSlime.onEnable(SlashSlime.java:39) ~[SlashSlime-1.0.0.jar:?]
        at org.bukkit.plugin.java.JavaPlugin.setEnabled(JavaPlugin.java:279) ~[paper-api-1.19.4-R0.1-SNAPSHOT.jar:?]
        at io.papermc.paper.plugin.manager.PaperPluginInstanceManager.enablePlugin(PaperPluginInstanceManager.java:189) ~[paper-1.19.4.jar:git-Paper-519]
        at io.papermc.paper.plugin.manager.PaperPluginManagerImpl.enablePlugin(PaperPluginManagerImpl.java:104) ~[paper-1.19.4.jar:git-Paper-519]
        at org.bukkit.plugin.SimplePluginManager.enablePlugin(SimplePluginManager.java:507) ~[paper-api-1.19.4-R0.1-SNAPSHOT.jar:?]
        at org.bukkit.craftbukkit.v1_19_R3.CraftServer.enablePlugin(CraftServer.java:555) ~[paper-1.19.4.jar:git-Paper-519]
        at org.bukkit.craftbukkit.v1_19_R3.CraftServer.enablePlugins(CraftServer.java:466) ~[paper-1.19.4.jar:git-Paper-519]
        at net.minecraft.server.MinecraftServer.loadWorld0(MinecraftServer.java:638) ~[paper-1.19.4.jar:git-Paper-519]
        at net.minecraft.server.MinecraftServer.loadLevel(MinecraftServer.java:437) ~[paper-1.19.4.jar:git-Paper-519]
        at net.minecraft.server.dedicated.DedicatedServer.initServer(DedicatedServer.java:308) ~[paper-1.19.4.jar:git-Paper-519]
        at net.minecraft.server.MinecraftServer.runServer(MinecraftServer.java:1104) ~[paper-1.19.4.jar:git-Paper-519]
        at net.minecraft.server.MinecraftServer.lambda$spin$0(MinecraftServer.java:320) ~[paper-1.19.4.jar:git-Paper-519]
        at java.lang.Thread.run(Thread.java:1589) ~[?:?]

Apparently, the developer doesn't allow us to use the library this way, so we are all stuck with the shading process if we wish to use bStats ¯\(ツ)/¯

By allowing unrelocated uses of bStats Metrics, you open up the possibility of plugins inadvertently using other plugins' versions of the library on older Spigot/Paper versions, potentially including modified versions that force enable metrics collection for all plugins (this has happened before).
As explained in #106, the bStats library is small and simple enough that there's no meaningful benefit to downloading it at runtime, and so there's no reason not to shade and relocate the library in.


adding description and ranges for the dora dashboard

⚠ Pre Checklist

Please complete ALL items in this checklist, and remove before submitting

[x] I have read through the Contributing Documentation.
[x] I have added relevant tests.
[x] I have added relevant documentation.
[x] I will add labels to the PR, such as pr-type/bug-fix, pr-type/feature-development, etc.

Summary
What does this PR do?
Adding a panel description about dora ranges to explain the expected score
Does this close any open issues?
n/a
Screenshots
Include any relevant screenshots here.

Other Information
Any other information that is important to this PR.

@yoramshai  I am afraid you have to solve the CI errors, by updating the commit message.
You can follow this guide: https://devlake.apache.org/zh/community/MakingContributions/development-workflow#style-guides

@yoramshai It takes effect in version v1.0.0-beta7.


Fix the missing deps issue from the bash script.

Hello @svengreb, I apologize for the late response, but I have made the requested changes.


Custom filter not working in example at react-table.js.org

What version of React-Table are you using?
Repo example default from:
https://react-table.js.org/#/story/custom-filtering
What bug are you experiencing, or what feature are you proposing?
Learning about this react-table. So i went to the example site and tried out the custom-filtering.
https://react-table.js.org/#/story/custom-filtering
Use https://codesandbox.io/s/X6npLXPRW (by clicking the "Fork" button) to reproduce the issue.
https://react-table.js.org/#/story/custom-filtering
What are the steps to reproduce the issue?

Go to https://react-table.js.org/#/story/custom-filtering
Try to filter by first name or last name, enter one letter at a time and see it does not react as expected.

If you inspect the code, it is indeed working. It is custom because the firstName column is filtering if the name begins and ends with the letter. The second column filters using the match-sorter library, which is a fuzzy finder library.

Here is a visual recording of what happens. See how typing d filters, and thien typing i clears the screen even though diamond is a valid result.

...i see


docs: clarify pinned dependencies with Conda

Rewrote the installation page a bit so that it's clearer how to pin all dependencies with Conda. Preview here.

@Leongrim, please have a look here, perhaps there are other things that can be clarified.

I don`t think you have to add anything more.

Alrighty


Website Dead link

Package version
No response
Flutter version
No response
Steps to reproduce
Click on custom component at the bottom of the page
https://www.fluttermix.com/docs/widgets/introduction#styleimage
https://www.fluttermix.com/docs/tutorials/custom-component => 404
Expected results
page display
Actual results
404
Code sample
No need for code

Thank you for your contribution ❤
I've just fixed it. Could you try again?

Resolved!


Exchange name with element_identifier for msp_sr_bowtie_parser

This is necessary so that we can recover the element name from dataset
collection elements.

well done 👍


refactor(transformer/class-properties): shorten code

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#7913  👈 (View in Graphite)
#7912
#7911
#7910
main

How to use the Graphite Merge Queue
Add either label to this PR to merge it via the merge queue:

0-merge - adds this PR to the back of the merge queue
hotfix - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.


Issue #3900 fixed

BUGS:
issue:- #3900

@vishal-webkul
please fix this also- https://prnt.sc/ucv9i0

@bhanu-webkul
fixed above mention problem please check

done


[SPARK-41832][CONNECT][PYTHON] Fix DataFrame.unionByName, add allow_missing_columns

What changes were proposed in this pull request?
Fix Implementation of DataFrame.unionByName, it wasn't assigning by_name correctly and allow_missing_columns wasn't implemented
Why are the changes needed?
Fixing the implementation and adding allow_missing_columns params
Does this PR introduce any user-facing change?
Yes
How was this patch tested?
Enabling Doctests and added unit tests

I honestly have gotten why we missed allowMissingColumns in the first version of the implementation, but it seems to make sense to have this in the proto.

Cc @zhengruifeng @amaliujia @HyukjinKwon

Cc  @amaliujia @HyukjinKwon @zhengruifeng can you review this PR ? I think its a starightforward change.

merged into master, thank you @techaddict


Release Google.Cloud.Datastream.V1 version 1.0.0-beta01

Changes in this release:
Initial release.

Created release for Google.Cloud.Datastream.V1-1.0.0-beta01


Error: Uncaught (in promise): No Directive annotation found on Dropdown

Hi,
can you please upgrade ng2-bootstraß to angular2 beta.10 or 11?
thx

discussing already on issue #313


fetch a default local image when an avatar has not been set

The current default avatar is set to fetch an image from gravatar. Please do not do this as many do not wish to share their browsing activity with tracking systems. Please include a default avatar image in the gogs release and fetch it from the local server.

Thanks your feedback!
But I don't understand the problem you describle very well as I suppose you didn't read/understand docs deeply.
You can just disable the Gravatar and there is a local default avatar in the Gogs.

My suggestion is that the default behavior with respect to avatars should be "DISABLE_GRAVATAR true". People should have to opt-in to tracking systems.

+1 for gravatar being disabled by default and turned on in config by any admin that wants to use it.
If possible it could be even a per user option instead of a side wide option imposed by admin.

Well... I agree, how about add this config option in the /install page, so user can disable/enable it at the beginning?

Hi, I've pushed code to develop branch that has Disable Gravatar checkbox with default checked.

Closed as implemented. :tada:


Test automation execution is not running in the order

Hi,
I created new test codeunit with 10 test functions
lets say the order of test functions are
Example:
1.Run
2.createsalesorder
3.createpurchaseorder
4.createpurchaseinvoice
5.Createpurchasecreditmemo
but the order when i run from the test tool its coming in different order
like
1.Run
2.createpurchaseinvoice
3.createpurchaseorder
4.createsalesorder
5.createsalesinvoice
The question why i am asking is the order has dependency to it . so i have to follow the same order as i mentioned as above .Please suggest do i need to do anything else or i missed anything ?

@srinivasbanny technically, you should not have dependencies between the different test functions, meaning that each test function should ideally prepare the full context it needs to execute successfully. However, if you really need a specific order, I would suggest that you name the functions alphabetically, because that's how they are executed, at least from what I have seen. So, you then could possibly call your functions:
1_createsalesorder
2_createpurchaseorder
3_createpurchaseinvoice
4_createpurchasecreditmeno
and then they should run in the order that you need them.
But to make it work right, for instance, if you would need a sales order created for the "createpurchaseorder" to run properly, you should create a sales order in the initial setup for the "createpurchaseorder" function. Or you have different functions that test the individual process (create purchase order, create sales order, etc.) and then one test function that tests the process and calls the functions all together in the right order.

Hi @srinivasbanny,
The order of the tests is today defined by the procedure Id, which you can't control. That is a bug and we will fix that.
Thanks for reporting the issue.

@thpeder  thanks for the update. for the time being how do we define the functions in orders .can you suggest what can be done ?

@srinivasbanny
The Id of the procedure is a hash value from its name, return type, and parameters, so you could get lucky by changing the name until you hit the expected order.
A better option is probably to divide your tests into multiple codeunits.

@thpeder the problem is i have a dependency to some of the test functions so i cant create them individually

@srinivasbanny A possible solution would be for you to declare the current methods as local, and create Test Functions that call the local methods as required. Ex.
function CreatePurchaseOrderTest()
begin
createpurchaseorder;
end;

function CreatePurchaseInvoiceTest()
begin
createpurchaseorder;
createpurchaseinvoice;
end;

That way you can be sure that the code is executed in the proper order, and you can also handle errors a bit better (if createpurchaseorder, there's no need to execute createpurchaseinvoice, as that would throw an error that might be misleading).


Roll engine a9a0b89b1c82..d9dfad3b890c (4 commits)

https://github.com/flutter/engine/compare/a9a0b89b1c82...d9dfad3b890c
git log a9a0b89b1c82aa657708eece3e690bf39cf0ead5..d9dfad3b890c392ada33e3b9a95816c0411c1183 --no-merges --oneline
d9dfad3b8 Roll src/third_party/skia 8676929c0a13..893403fb3896 (6 commits) (flutter/engine#9421)
91e31dca5 Made the license check ignore the .vscode directory. (flutter/engine#9398)
24c3f513f Roll src/third_party/skia a1d26a807798..8676929c0a13 (9 commits) (flutter/engine#9416)
e33f83edb Roll src/third_party/skia e3beb99f39f0..a1d26a807798 (1 commits) (flutter/engine#9411)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (amirha@google.com), and stop
the roller if necessary.

Trybots failed. These were the failed builds: https://cirrus-ci.com/task/5958680571281408

Commit queue failed; closing this roll.


[Task]: Simple pipeline logging sample/directions

What needs to happen?
Migration: https://issues.apache.org/jira/browse/HOP-4467
It would be ideal to have a simple pipeline logging setup example / or setup help guide to better understand how to make use of this for logging/reflection.
Issue Priority
Priority: 3
Issue Component
Component: Documentation

the pipeline logging transform is undocumented.
the documentationUrl property in src/main/java/org/apache/hop/reflection/pipeline/transform/PipelineLoggingMeta.java points to /logging/logging-reflection.html. This is the documentation for the Pipeline Logging metadata type, not the Pipeline Logging transform.


Game crash after open killproof.me addon

Hello! After July 18 i always catch game crash when open killproof.me addon tab in game. Help pls, how i can solve this problem?

I heard about that error on discord as well, the easiest way to fix it, is to switch to dx11. Since i am not able to reproduce that error i am not able to fix it.

I heard about that error on discord as well, the easiest way to fix it, is to switch to dx11. Since i am not able to reproduce that error i am not able to fix it.

Thank you! Change to DX11 solve problem for me.

Had this crash too, and while switching to dx11 does solve it, I don't know if this would be a great option for me due to old hardware.
I have found a log file for the crash in dx9 mode.
arcdps_lastcrash.log

yes, i produce a crash within the gw2 renderer when using dx9. It is a lot of effort to fix it and since dx9 is deprecated, i will not fix it :(
Just use dx11 in the future,


SHOW MEASUREMENTS syntax inconsistency with documentation

As per https://docs.influxdata.com/influxdb/v1.7/query_language/schema_exploration/#show-measurements
SHOW MEASUREMENTS syntax is
SHOW MEASUREMENTS [ON <database_name>] [WITH MEASUREMENT <regular_expression>] [WHERE <tag_key>  ['<tag_value>' | <regular_expression>]] [LIMIT_clause] [OFFSET_clause]
When I submit
SHOW+MEASUREMENTS+WITH+MEASUREMENT+/c/
I get error
{
"error": "error parsing query: found /, expected =, =~ at line 1, char 36"
}
InfluxDB version 1.7.9
It appears that now not only regular expressions are supported but also exact match, wich probably return 1 or 0 results, so somewhat useless.
If documentation is not updted, it shoud show
< operator> ['<tag_value>' | <regular_expression>]
instead of just
<regular_expression>
Thank you very much
Igor Toropov

@Igor-Toropov thanks for opening. We will get the docs updated.

Thanks for pointing this out @Igor-Toropov. This has been updated and will be deployed some time today.

Thank you very much for quick turnaround, Russ and Scott.
Just a minor suggestion:
May measurement-name be replaced with measurement_name for consistency?
Thank you very much.

@Igor-Toropov done


Docker "Cons"

When wouldn't you want to use containers? What becomes more difficult?

Before you go over the container cliff
Dev vs Prod

Big items that are configurable yet complicated:

Logging

Some other things that come to mind:

Security
Networking (port conflicts, IP allocation)
Docker versioning
Performance nitpicking (kernel, /proc settings)

Omg, I just realized there's a whole conference for the Cons of Docker: Dockercon.

Omg, I just realized there's a whole conference for the Cons of Docker: Dockercon.
Get that man a :beer:! da duh tish


Flow not initializing on Windows 10 after update to fall creators update

Hello!
When I'm trying to launch flow, I see only this:
yarn run v1.2.1
$ flow
Launching Flow server for C:\Users\...
Spawned flow server (pid=3272)
Logs will go to C:\Users\....
flow is still initializing; this can take some time. [processing]

I tried with flow 0.57.3 and 0.56 — the same result, nothing. No errors, no warnings.
I reinstalled node_modules couple times, turned off Windows Defender — not helped too.

This is an issue with the Fall Creators Update and is reported here.

The issue from your link is with linux subsystem in windows.
I don't think this is a case because I'm not using WSL.

Well, I installed WSL and launched flow from there. It helped.
But this is a terrible way to solve the problem. I must keep the whole OS for launching only one flow.

Closing since @Log1x mentions this is an issue with windows https://github.com/Microsoft/WSL/issues/2448 which seems to have been resolved. Lmk if that's not the case though.


BigNumbers are de|encoded as strings

Adapt requirements by https://github.com/raiden-network/raiden/pull/5390
With this, we could remove the lossless-json dependency and go directly with JSON utilities, as all numbers should be encoded and decoded to/from strings, but to stay in the safe side if any field changes in the future or was left behind, better safe than sorry and still use lossless library to do it.

I tested by hand and changes here are compatible with RC2, with direct and mediated transfers.

Good job @andrevmatos


Fix strikethouth on iOS 10.3

Fix Strikethrough error on iOS 10.3.

Codecov Report

Merging #54 into master will decrease coverage by 0.38%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #54      +/-   ##
==========================================
- Coverage    53.4%   53.02%   -0.39%
==========================================
  Files           8        8
  Lines         279      281       +2
==========================================
  Hits          149      149
- Misses        130      132       +2

Impacted Files
Coverage Δ

Attributed/Attributes.swift
15.9% <0%> (-0.37%)
:arrow_down:

Continue to review full report at Codecov.

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

@PhanVietTrung  Thanks for this! Looks good!


What is the purpose of exposing the list of users?

Description
The users app provides a view to list all the users.
https://github.com/pydanny/cookiecutter-django/blob/28160d06ac15635a3596db53bf53493bc252f141/{{cookiecutter.project_slug}}/{{cookiecutter.project_slug}}/users/urls.py#L12
Probably I am missing something obvious, but is this necessary? Why would a user be able to view all the registered users of the web service? At first I thought this would be only accessible in local environment, but I couldn't find a setting which prevents it from showing up the production environment.

I'd say, our users is pretty much a placeholder app: it's not intended to be rolled out to production as-is, and I'm sure no one would seriously consider doing so. For what it's worth, you're right, it's no good practice. Maybe you could send a PR fixing this?

@demestav we can just remove that route.

For what it's worth I always thought this was a bit weird too, so good suggestion to put make it local only (or even better just remove it).

@yunti In https://github.com/pydanny/cookiecutter-django/pull/1761 (still pending review) the users app is only exposed when DEBUG is True so the developer is free to explore it, and at the same time, prevent an accidental exposure of the users in production.

After seeing the final result to get the views locally debug-only, I now believe the list view should be deleted. I've opened #2062 to do that. I didn't touch the user detail on purpose...
Thoughts on this?


fix empty error messages

Shouldn't it be merged to staging first?

There is no staging for this package :/ but the other PR on the integration side will be to staging first.


Constant stack usage

The current proposal desugars
"hello"
  |> doubleSay
  |> capitalize
  |> exclaim;

to
exclaim(capitalize(doubleSay("hello")));

If I want to reuse this pipeline I can write
const sayItAgainLouder = s => s |> doubleSay |> capitalize |> exclaim;

Now I get reuse, but I'm allocating three stack frames. This isn't a big deal unless I want to compose sayItAgainLouder with some other functions.
A simple alternative desugaring
[doubleSay, capitalize, exclaim].reduce((value, f) => f(value), "hello");

This decreases stack usage, but I imagine at the cost of speed.
It would be useful if this proposal required the compiler to maintain constant stack usage through either some sort of stream fusion or stack frame reuse.

I don't think that's within the intended uses: if you want to compose pipelines you'll be using compose rather than an infix operator. Ditto for long chains using the stack: if your pipeline is ridiculously long then it should probably be built up using compose in pieces rather than one long chain that fills more than your text editor can display.

if you want to compose pipelines you'll be using compose rather than an infix operator.

I might agree with you @jasmith79 if there was a builtin compose that guaranteed constant stack usage.
I'm all for a built-in though. See https://github.com/fantasyland/ECMAScript-proposals/issues/1

Ditto for long chains using lots of stack space: if your pipeline is ridiculously long then it should probably be built up using compose in pieces rather than one long chain that fills more than your text editor can display.

I haven't suggested this usage. I did mention composing functions with other composed functions.

If I understand correctly,
exclaim(capitalize(doubleSay("hello")))

works like
let _tmp = "hello"
_tmp = doubleSay(_tmp)
_tmp = capitalize(_tmp)
_tmp = exclaim(_tmp)
_tmp

Therefore there is no nested stack frame?

@dtinth you're right. For some reason I was thinking the behavior was the same as how some pipe functions are implemented resulting in something like
(x => exclaim((x => capitalize((x => doubleSay(x))(x)))(x)))('hello')

which it does not.


feat: update semantic release version

Remove issues with peer deps

LGTM! 👍

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


⚠ ClamAV has degraded performance

In 136fab2, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 53 ms

Resolved: ClamAV performance has improved in 628961b after 5 minutes.


Accepting sgRNAs with length != 20

Currently, CERES assumes that sgRNAs have 20 nts before the PAM in the function guideAlignments. However, some libraries use 19 nt guides. Ideally this could be solved by taking the alignment end rather than the beginning for positive strand alignments.  Simple hack that worked for me instead: add an argument guide_length to the guideAlignments call signature. guide_length can be determined in map_guide_to_locus by checking the number of characters in the first entry in guides (assumes all guides are the same length).

Thanks Josh!


users function not working on Julia v1.7.1

There seems to be something broken with the users function with PkgDeps v0.6.2 and Julia v1.7.1. I can reproduce the issues with my local Julia installation,, as well as with the official  julia:1.7.1 docker image. In contrast they do not occur with the julia:1.6.0 docker image.
julia> using PkgDeps

julia> users("Flux")
ERROR: ArgumentError: Collection is empty, must contain exactly 1 element
Stacktrace:
 [1] only
   @ ./iterators.jl:1354 [inlined]
 [2] _get_pkg_uuid(pkg_name::String, registry_name::String; depots::Vector{String})
   @ PkgDeps ~/.julia/packages/PkgDeps/afXc8/src/utilities.jl:101
 [3] _get_pkg_uuid
   @ ~/.julia/packages/PkgDeps/afXc8/src/utilities.jl:101 [inlined]
 [4] #users#18
   @ ~/.julia/packages/PkgDeps/afXc8/src/PkgDeps.jl:112 [inlined]
 [5] users (repeats 2 times)
   @ ~/.julia/packages/PkgDeps/afXc8/src/PkgDeps.jl:112 [inlined]
 [6] top-level scope
   @ REPL[7]:1

julia> users(Base.UUID("587475ba-b771-5e3f-ad9e-33799f191a9c"))  # UUID of Flux.jl
String[]

Tests fail as well:

(@v1.7) pkg> test PkgDeps
     Testing PkgDeps
      Status `/tmp/jl_Wwxymf/Project.toml`
  [34da2185] Compat v3.41.0
  [839e9fc8] PkgDeps v0.6.2
  [44cfe95a] Pkg `@stdlib/Pkg`
  [3fa0cd96] REPL `@stdlib/REPL`
  [fa267f1f] TOML `@stdlib/TOML`
  [8dfed614] Test `@stdlib/Test`
  [cf7118a7] UUIDs `@stdlib/UUIDs`
      Status `/tmp/jl_Wwxymf/Manifest.toml`
  [34da2185] Compat v3.41.0
  [839e9fc8] PkgDeps v0.6.2
  [0dad84c5] ArgTools `@stdlib/ArgTools`
  [56f22d72] Artifacts `@stdlib/Artifacts`
  [2a0f44e3] Base64 `@stdlib/Base64`
  [ade2ca70] Dates `@stdlib/Dates`
  [8bb1440f] DelimitedFiles `@stdlib/DelimitedFiles`
  [8ba89e20] Distributed `@stdlib/Distributed`
  [f43a241f] Downloads `@stdlib/Downloads`
  [b77e0a4c] InteractiveUtils `@stdlib/InteractiveUtils`
  [b27032c2] LibCURL `@stdlib/LibCURL`
  [76f85450] LibGit2 `@stdlib/LibGit2`
  [8f399da3] Libdl `@stdlib/Libdl`
  [37e2e46d] LinearAlgebra `@stdlib/LinearAlgebra`
  [56ddb016] Logging `@stdlib/Logging`
  [d6f4376e] Markdown `@stdlib/Markdown`
  [a63ad114] Mmap `@stdlib/Mmap`
  [ca575930] NetworkOptions `@stdlib/NetworkOptions`
  [44cfe95a] Pkg `@stdlib/Pkg`
  [de0858da] Printf `@stdlib/Printf`
  [3fa0cd96] REPL `@stdlib/REPL`
  [9a3f8284] Random `@stdlib/Random`
  [ea8e919c] SHA `@stdlib/SHA`
  [9e88b42a] Serialization `@stdlib/Serialization`
  [1a1011a3] SharedArrays `@stdlib/SharedArrays`
  [6462fe0b] Sockets `@stdlib/Sockets`
  [2f01184e] SparseArrays `@stdlib/SparseArrays`
  [10745b16] Statistics `@stdlib/Statistics`
  [fa267f1f] TOML `@stdlib/TOML`
  [a4e569a6] Tar `@stdlib/Tar`
  [8dfed614] Test `@stdlib/Test`
  [cf7118a7] UUIDs `@stdlib/UUIDs`
  [4ec0a83e] Unicode `@stdlib/Unicode`
  [e66e0078] CompilerSupportLibraries_jll `@stdlib/CompilerSupportLibraries_jll`
  [deac9b47] LibCURL_jll `@stdlib/LibCURL_jll`
  [29816b5a] LibSSH2_jll `@stdlib/LibSSH2_jll`
  [c8ffd9c3] MbedTLS_jll `@stdlib/MbedTLS_jll`
  [14a3606d] MozillaCACerts_jll `@stdlib/MozillaCACerts_jll`
  [4536629a] OpenBLAS_jll `@stdlib/OpenBLAS_jll`
  [83775a58] Zlib_jll `@stdlib/Zlib_jll`
  [8e850b90] libblastrampoline_jll `@stdlib/libblastrampoline_jll`
  [8e850ede] nghttp2_jll `@stdlib/nghttp2_jll`
  [3f19e933] p7zip_jll `@stdlib/p7zip_jll`
     Testing Running tests...
exception: Test Failed at /root/.julia/packages/PkgDeps/afXc8/test/runtests.jl:44
  Expression: PkgDeps._get_pkg_uuid("PkgDepsFakePackage", "General")
    Expected: PackageNotInRegistry
      Thrown: ArgumentError
Stacktrace:
 [1] macro expansion
   @ ~/.julia/packages/PkgDeps/afXc8/test/runtests.jl:44 [inlined]
 [2] macro expansion
   @ /usr/local/julia/share/julia/stdlib/v1.7/Test/src/Test.jl:1283 [inlined]
 [3] macro expansion
   @ ~/.julia/packages/PkgDeps/afXc8/test/runtests.jl:44 [inlined]
 [4] macro expansion
   @ /usr/local/julia/share/julia/stdlib/v1.7/Test/src/Test.jl:1283 [inlined]
 [5] macro expansion
   @ ~/.julia/packages/PkgDeps/afXc8/test/runtests.jl:28 [inlined]
 [6] macro expansion
   @ /usr/local/julia/share/julia/stdlib/v1.7/Test/src/Test.jl:1283 [inlined]
 [7] top-level scope
   @ ~/.julia/packages/PkgDeps/afXc8/test/runtests.jl:14
Test Summary:                   | Pass  Fail  Total
internal functions              |   16     1     17
  _get_pkg_name                 |    2            2
  _get_pkg_uuid                 |    5     1      6
    name to uuid                |    2            2
    Same name in two registries |    2            2
    exception                   |    1     1      2
  _get_latest_version           |    1            1
  _find_alternative_packages    |    1            1
  `_find_latest_pkg_entry`      |    7            7
ERROR: LoadError: Some tests did not pass: 16 passed, 1 failed, 0 errored, 0 broken.
in expression starting at /root/.julia/packages/PkgDeps/afXc8/test/runtests.jl:13
ERROR: Package PkgDeps errored during testing

From Julia v1.7.0 NEWS.md:

Registries downloaded from the Pkg Server (not git) are no longer uncompressed into files but instead read directly from the compressed tarball into memory. This improves performance on filesystems which do not handle a large number of files well. To turn this feature off, set the environment variable JULIA_PKG_UNPACK_REGISTRY=true.

It seems as if the registries directory for the General repository does not contain a General subdirectory anymore, but rather a compressed General.tar.gz file.  Therefore the reachable_registries function does not find the General registry for Julia >= v1.7.0, while it does on older Julia versions.

as a workaround, one could clone General themselves, or mess with the env variables to get Pkg to do it. I think the long term solution is to help get https://github.com/GunnarFarneback/RegistryInstances.jl going as a stable interface in front of Pkg internals and then rely on that.
(FYI, PackageAnalyzer.jl and similar packages also face this same issue)


Can we get the TOC updated to reflect that this is Nvidia specific content now that AMD specific content exists.

Can we update the title/string of the doc to be NVIDAO specific

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 3c5bc749-3f68-e230-37ff-f74ce72efb59
Version Independent ID: 2a382803-e085-82e2-4139-be5c8b9e1a0d
Content: Azure N-series GPU driver setup for Windows - Azure Windows Virtual Machines
Content Source: articles/virtual-machines/windows/n-series-driver-setup.md
Service: virtual-machines-windows
GitHub Login: @cynthn
Microsoft Alias: cynthn

Just to confirm, are you referring to the title on this left side pane?

As shared offline by the original poster,

Yes. The TOC needs a quick edit.
The article name displayed to the customer should also have NVIDIA in the string like it is for AMD
https://docs.microsoft.com/en-us/azure/virtual-machines/windows/n-series-driver-setup
https://docs.microsoft.com/en-us/azure/virtual-machines/windows/n-series-amd-driver-setup

#please-close https://github.com/MicrosoftDocs/azure-docs-pr/pull/100770


Mgmt mcumgr remove zephyr_smp_trim_front  and mgmt_streamer_trim_front

The PR consists of two commits:

replacing usage of zephyr_smp_trim_front with net_buf_pull as additional logic is not longer needed;
removing mgmt_streamer_trim_front and callback pointer mgmt_streamer_cfg.trim_front;

@de-nordic please rebase


longestIncreasingSubsequence.cpp and levenshteinDistance.cpp added

Addressed issues #7 and #9 (which were assigned to me)
@wataridori please have a look :)

@sneaky-potato Thank you very much. It looks good to me 👍
However, to unify the folder structure among all Problems, could you please rename the solution files to just solution.cpp.
It means inside the Longest Increasing Subsequence Folder, we will have problem.md and solution.cpp (as well as solution.py, solution.rb ....) files

Ah aight, will do


M

Checklist

[ ] npm test passes
[ ] tests and/or benchmarks are included
[ ] documentation is changed or added
[ ] commit message follows commit guidelines

Affected core subsystem(s)

Description of change

Codecov Report

Merging #3 (693df60) into master (773df78) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master        #3   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           34        34
  Lines          959       958    -1
=========================================
- Hits           959       958    -1

Impacted Files
Coverage Δ

app/extend/request.js
100.00% <ø> (ø)

lib/egg.js
100.00% <ø> (ø)

app/middleware/meta.js
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


This needs an example of how to use this tool with TT files in a sql project

it doesnt seem to load something correctly. Perhaps an example of the correct parameters would be helpful
error : A processor named 'SqlModelDirectiveProcessor' could not be found for the directive named 'SqlModelDirective'. The transformation will not be run.  The following Exception was thrown:
System.IO.FileNotFoundException: Failed to load directive processor SqlModelDirectiveProcessor.
   at Microsoft.VisualStudio.TextTemplating.CommandLine.CommandLineHost.ResolveDirectiveProcessor(String processorName)
   at Microsoft.VisualStudio.TextTemplating.Engine.ProcessCustomDirectives(ITextTemplatingEngineHost host, TemplateProcessingSession session, ITelemetryScope scope, IEnumerable`1 directivesToBeProcessed)

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: e8177d9b-f8d7-419e-d81d-ce25a44b084f
Version Independent ID: 43a69770-1eab-06e1-868c-3d8f1c32716d
Content: Generating Files with the TextTransform Utility - Visual Studio
Content Source: docs/modeling/generating-files-with-the-texttransform-utility.md
Product: visual-studio-windows
Technology: vs-ide-modeling
GitHub Login: @jillre
Microsoft Alias: jillfra

@StingyJack Thank you for the feedback, assigning this to the author for review.
Please see if installing the latest Microsoft.Net.Compilers package as described in this post resolves the issue.

@msebolt Thanks for the suggestion, but this is for the TextTransform.exe standalone exe.

@gewarren - Do you have any insight into this issue or have a contact we could add?

I think the last contact I had for modeling was @mgoertz-msft .

AFAIK, this is only supported inside of Visual Studio, i.e. devenv.exe.

That's a shame. Anywhere that's documented?

Not at this time, @StingyJack.  We appreciate the suggestion and I have added it to our backlog.  It will require investigation into what needs to happen or if it’s even possible. Thank you for being a valued customer.

I am closing this issue now.  You are welcome to @ mention me for any followup.  We hope to hear from you again.


⚠ Bofrost.be has degraded performance

In 5c08c27, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 3028 ms

Resolved: Bofrost.be performance has improved in e6f8214.


x/pkgsite: Broken links for custom go-source tags (regression)

What did you do?
Navigate to https://pkg.go.dev/blitiri.com.ar/go/spf (or any blitiri.com.ar package).
Click on any type or function name, or any of the source files.
What did you expect to see?
I expected to be taken to the corresponding source page, e.g. https://blitiri.com.ar/git/r/spf/b/master/t/f=spf.go.html#line-49
What did you see instead?
The links are broken because {dir} is not being replaced correctly.
Context and additional information
https://blitiri.com.ar is my personal page, where I host my git and go repositories.
The git web viewer is git-arr, and has been running fine for years.
I create the https://blitiri.com.ar/go/ remote import path by hand for all my Go packages/modules. They use the meta tags as follows:
<meta name="go-import"
  content="blitiri.com.ar/go/log git https://blitiri.com.ar/repos/log" />
<meta name="go-source"
  content="blitiri.com.ar/go/log https://godoc.org/blitiri.com.ar/go/log https://blitiri.com.ar/git/r/log/b/master/t/{dir} https://blitiri.com.ar/git/r/log/b/master/t/{dir}/f={file}.html#line-{line}" />

These go-import meta tags have been working fine for years, and worked well in godoc.org and (IIRC) also in pkg.go.dev at least at some point.
I think what's happening is that {dir} is not being expanded correctly, and instead {file} contains the full path.
As an example, if you navigate to pgd's view of blitiri.com.ar/go/chasquid/internal/safeio, then click on any of the links, you can see they're broken because they are like this:
https://blitiri.com.ar/git/r/chasquid/b/master/t/%7bdir%7d/f=internal/safeio/safeio.go.html#line-26

Note how {file} is being expanded to internal/safeio/safeio.go.html, and {dir} being expanded to {dir} (sic).
Instead, {dir} should be expanding to internal/safeio/, and file to safeio.go as per the spec and historical behaviour.
This is probably related to issue #40477 in some way.
Please let me know if you need any additional information, thanks!

This does look related. @jba should we close in favor of or merge this with #40477.

Technically you are correct: that your links don't work with master is a bug. On the other hand, linking to master when the version is v1.2.3 is not a great user experience.
Why don't I make the master links work now, and when you have upgraded your software to handle tags and commits, let me know and I will generalize.

SGTM, thanks!
Please let me know when the fix is in and I can check the links by hand to check it works.
I'll let you know once git-arr supports views of tags (and maybe arbitrary commits, although it being a static generator that might not be practical).
Thanks again!

@albertito This should be live.

Thanks! I can see that it works for my particular site now, thanks, I appreciate the workaround :)
Has the regression for the general case been fixed though? What I mentioned above (https://github.com/golang/go/issues/44607#issue-816319006 and https://github.com/golang/go/issues/44607#issuecomment-795317951) affects all sites relying on the go-source spec, and IIUC you committed to continue to support.
Thanks!

Sorry I didn't respond to your last comment, but I think I answered your question at https://github.com/golang/go/issues/44607#issuecomment-795684007.


[RFR] Update to React 15

updates components from using deprecated createClass() method to ES6 classes
updates React to 15.6.1
updates example to new syntax
bumps version to 1.1.0

ping @livmadsen @artibella @sigmarstern

ping @edenspiekermann/devs?

ping @dsteinel ? 😺

@artibella @livmadsen @dsteinel does someone want to merge?

Hey @dsteinel @artibella @livmadsen someone needs to run npm publish with the espi npm account, or nobody will get the update 🎉
NPM still sees the old version: https://www.npmjs.com/package/faster-react-tabs


Sanitize input before rendering to users

Goal

[x] Sanitize input before rendering to users using dangerouslySetInnerHTML (src/components/project-details/readme-card.tsx)
[x] Run prettier on the whole project

Why
We add one more layer of security so if our database is compromised. Our users are not at stake. This is Defense in Depth principle (if one layer fails, another steps up immediately to thwart an attack)
How to test

Update src/mocks/data/project-readme-react.html.js like this

const projectReadmeReact = `

+

...

Run

npm run start:mock

Navigate to any project, for example: http://localhost:3000/projects/freecodecamp. You should see the javascript is executed if we do not sanitize input.

Screenshots

@michaelrambeau FYI that we also have alternatives to dompurify as @slorber's suggestion:

purify on the server
using server components
future: WIP HTML sanitizer API

We already did 1. We can try 2 when we migrate to React 18. And we hope that we can use 3 in the next few years.
For now, to add another layer of protection, I think dompurify should be a good solution.


Component card details on Catalog

Is your feature request related to a problem? Please describe.
As an Airy user, i need to view all available component details on Catalog to learn more about it.
Describe the solution you'd like
We need to design card details view for components to present each component details.
CARD DETAILS VARIABLES from DynamoDB:

Component logo.
Component Name.
Component Description.
Available for : "Open-source" / "Enterprise" / "Cloud"
Price : Free
Categories : "Communication Source" / "API" / "UI" / "Machine learning & Conversational AI" / "Analytics" / "Storage" / "Customer Service" / "Marketing".
Docs reference.
Back button to go to Catalog main view.
Install Button/Uninstall button.

Buttons flow:

Install button on Catalog view > Instant install > Pop-up to confirm Installation and action to either configure component or exist pop-up.
Install button on Card details view > Instant install > Pop-up to confirm Installation and action to either configure component or exist pop-up.
Open button on Catalog view > Navigates to Component configuration > ""/airy.co/control-center/connectors/connected/#componentName".
Uninstall button on Card details view > Pop-up to confirm uninstalling this component or exit.

DESIGN URL:
add /components.list info on the attached design.

FIGMA LINK:
https://www.figma.com/file/CRt7x2PjccilajBaPO4HVW/Control-Center-0.6_3?node-id=260%3A300


[FLINK-2107] Add hash-based strategies for left and right outer joins.

This PR adds hash-based execution strategies for left and right outer joins, that have the outer side as the probe side of a hash table.

I'd like to merge this later today, unless somebody speaks up.