Passthrough Endpoint for Weather API challenge

Currently, campers have to find their own weather APIs and leave their API keys in their solutions.
We need to create an endpoint where campers can pass params and we'll go fetch the data for them using our unexposed API key, then return that data to them.
This will also make the challenge much easier, as they can focus on working with the data and presenting it, rather than having to figure out the API themselves.
Challenge Zipline: Show the Local Weather has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/47.0.2526.106 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.

The OpenWeather API isn't difficult to obtain. I'm not sure if you cover how to set environment variables in Heroku, but it seems like a good topic as people advance in building apps (if there were any chance of moving the hosting from Codepen to Heroku). I haven't looked at the actual limits, but I'd worry about maxing out calls to a shared API as more and more campers complete the challenge.

Is there any way to move from Codepen to just using Github project pages?

Open Weather App say that they will remove API limits for open source projects, in case that's useful / new information for you guys.
I've provided a screenshot of exactly what they say, and a link to where I found the information.

http://openweathermap.org/faq#error401

That's great, but that doesn't negate the fact that there information is
just downright erroneous.  :(
On 03/08/2016 09:25 PM, Dawson DePretis wrote:

Open Weather App say that they will remove API limits for open source
projects, in case that's useful / new information for you guys.
I've provided a screenshot of exactly what they say, and a link to
where I found the information.
image
https://cloud.githubusercontent.com/assets/9157052/13623281/4c5f1a22-e55a-11e5-988d-e32147f6986e.png
http://openweathermap.org/faq#error401
—
Reply to this email directly or view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/5965#issuecomment-194074620.

Yeah that's absolutely true. I was just hoping that knowing that a special non-restricted API key is possible to acquire might make a system where campers would go through a FCC server to get open weather map data more doable.

I used Weather Underground, and I'd recommend it to anyone.  :)
On 03/08/2016 10:44 PM, Dawson DePretis wrote:

Yeah that's absolutely true. I was just hoping that knowing that a
special non-restricted API key is possible to acquire might make a
system where campers would go through a FCC server to get open weather
map data more doable.
—
Reply to this email directly or view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/5965#issuecomment-194100476.

Does Weather Underground require an API key to use?

It does, but so does almost every other API. That said it's free for upto 500 calls per day, which for the most part is fine. It's unlikely an FCC project would need more than that.
Sent from my T-Mobile 4G LTE Device
-------- Original message --------
From: Dawson DePretis<EMAIL_ADDRESS>Date: 3/9/2016  5:51 PM  (GMT-05:00)
To: FreeCodeCamp/FreeCodeCamp<EMAIL_ADDRESS>Cc: Krista Prenesti<EMAIL_ADDRESS>Subject: Re: [FreeCodeCamp] Passthrough Endpoint for Weather API challenge
(#5965)
Does Weather Underground require an API key to use?
—
Reply to this email directly or view it on GitHub.

That's very true, but having a publicly posted API key (in codepen) means that people could potentially find that API key and use it for their own purposes and max out that 500 calls per day or get the API key banned because it's abused. As a result, FCC is trying to find a way to allow campers to not have to expose an API key.

Easy.  Use github to host the projects instead of codepen. Code should
be pushed to github sooner rather than later.  I had learned github
myself before coming to FCC, but I don't understand why they wait so
long to teach it.  With any github repo (for a static site), you need
only create a branch called gh-pages and push the code. That way you can
show off your site and the api key is not as easily seen.  Besides,
Codepen is a nightmare to work with anyway.
On 03/09/2016 06:31 PM, Dawson DePretis wrote:

That's very true, but having a publicly posted API key (in codepen)
means that people could potentially find that API key and use it for
their own purposes and max out that 500 calls per day or get the API
key banned because it's abused. As a result, FCC is trying to find a
way to allow campers to not have to expose an API key.
—
Reply to this email directly or view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/5965#issuecomment-194564434.

Still a public API key, even if it's slightly harder to find.
On Wed, Mar 9, 2016 at 3:39 PM Krista Prenesti<EMAIL_ADDRESS>wrote:

Easy. Use github to host the projects instead of codepen. Code should
be pushed to github sooner rather than later. I had learned github
myself before coming to FCC, but I don't understand why they wait so
long to teach it. With any github repo (for a static site), you need
only create a branch called gh-pages and push the code. That way you can
show off your site and the api key is not as easily seen. Besides,
Codepen is a nightmare to work with anyway.
On 03/09/2016 06:31 PM, Dawson DePretis wrote:

That's very true, but having a publicly posted API key (in codepen)
means that people could potentially find that API key and use it for
their own purposes and max out that 500 calls per day or get the API
key banned because it's abused. As a result, FCC is trying to find a
way to allow campers to not have to expose an API key.
—
Reply to this email directly or view it on GitHub
<
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/5965#issuecomment-194564434
.

—
Reply to this email directly or view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/5965#issuecomment-194568502
.

Is this still needed or was it handled elsewhere?
I've done a few basic node/express backend projects and think that I could handle this.
I think I understand the requirements.
Campers should be able to make a get request including latitude/longitude in the request url , and get back a response containing the daily forecast in json.
Is OpenWeather API data that bad? It seemed to provide the basic info I needed.
If a request/day limited API provides much better data then OpenWeather I could setup a  check on the server. I would log out # of requests in a 24 hour period. If it turns out we are going over the limit for Weather Underground I could create additional api keys that are cycled through once we start getting request failures.
Any ideas on dealing with the geolocation issue in the same app? Chrome recently (~6 weeks maybe) removed support for HTML5 Geolocation on non secure origins, i.e free codepen. Some people are using API's for handling that also.

I am working on this.

Sorry we no longer need this.
Happy Coding

Hey this is Megan. Im going to lose my phone plan tomorrow. Do u know any
wi fi passcodes or any way i can access internet on LG phone?
On Jun 8, 2016 6:38 AM, "Andrew Schmidthuber"<EMAIL_ADDRESS>wrote:

Is this still needed or was it handled elsewhere?
I've done a few basic node/express backend projects and think that I could
handle this.
I think I understand the requirements.
Campers should be able to make a get request including latitude/longitude
in the request url , and get back a response containing the daily forecast
in json.
Is OpenWeather API data that bad? It seemed to provide the basic info I
needed.
If a request/day limited API provides much better data then OpenWeather I
could setup a check on the server. I would log out # of requests in a 24
hour period. If it turns out we are going over the limit for Weather
Underground I could create additional api keys that are cycled through once
we start getting request failures.
Any ideas on dealing with the geolocation issue in the same app? Chrome
recently (~6 weeks maybe) removed support for HTML5 Geolocation on non
secure origins, i.e free codepen. Some people are using API's for handling
that also.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/5965#issuecomment-224590574,
or mute the thread
https://github.com/notifications/unsubscribe/ARoqChjsZX2Fgrf223BeG9WuuZXwILGtks5qJsXfgaJpZM4HA4Ak
.


Flutter 2 with null safity

Hi, Bro I used this for a long time it's very helpful and time-saving.
but the problem is here dart release null safety and model changed in null safety.
please upgrade the dart JSON to a model with null safety.
thanks in advance.

i ve same problem with format @required so manualy i fix it to 'required' only

I faced same issue. Please update dart JSON part. Many thanks.

可以看看我提交的PR，现在等待官方合并。#1904


Cannot xrepo remove package. package(pkg) not found.

Xmake Version
2.7.1+HEAD.0162040
Operating System Version and Architecture
macOS Monterey 12.5.1
Describe Bug

Expected Behavior
Successfully removed the selected package.
Project Configuration
No response
Additional Information and Error Logs
No response

BTW, when I xrepo install fmt, there is something wrong too.

BTW, when I xrepo install fmt, there is something wrong too.

try xrepo remove --all fmt

BTW, when I xrepo install fmt, there is something wrong too.

just network issue

try xrepo remove --all fmt

Solved. Thanks for the time.

BTW, when I xrepo install fmt, there is something wrong too.

just network issue

Closed.


Refactor Controller to use standard webapi conventions

Method Names like, Post, get, etc...

definetly not going to that. I dont wanna end up with tons of methods, i will never use!


msi package not found

I haven't found any msi here:
http://repos.sensuapp.org/
Is the link mentioned here wrong?
http://sensuapp.org/docs/0.16/windows

I've encountered this problem too. When I use my desktop browser (IE, Firefox, Chrome) vist this link, the page printed as a plain XML content. But when I vist it in my Virtualbox Ubuntu Guest (using firefox), it works fine. Still don't know why !!!
By the way, I'm in China, and I wondered if this issue has something to do with region?

Because the index file is on S3, you need to specify the full path http://repos.sensuapp.org/index.html


Fix for Incorrect File Extension in Generated Report Links

For #377, contributed by @MarkLee131.

/gcbrun exp -n dg-report

Regression caused by the model, not this PR:
https://llm-exp.oss-fuzz.com/Result-reports/ofg-pr/2024-06-24-378-dg-report-comparison/index.html


overflow and scrollbars

When the component is loaded, it pushes the scrolling back down. We should add an option that allow the component to have a overflow:scrolling.
Tasks

[ ] The user does not set any parameters (e.g. pdf_viewer(file)) -> we unwrap the PDF (no scroll bars, every page one after the other)
[x] The user does set height = 700 (absolute number) -> then we show the scrollbars and limit the window to the height
[ ] The user does set height = 80% (relative number) -> same as previous, show the scrollbars and limit the window to the height

The user does set height = 80% (relative number) -> same as previous, show the scrollbars and limit the window to the height

The argument of Streamlit.setFrameHeight() supports only numbers, not str like 80%, so to support this feature, we need to calculate from the user's window size, which is not sure it can make it.

The user does not set any parameters (e.g. pdf_viewer(file)) -> we unwrap the PDF (no scroll bars, every page one after the other)

I can make it. it would take 1 - 4 hours.

The argument of Streamlit.setFrameHeight() supports only numbers, not str like 80%, so to support this feature, we need to calculate from the user's window size, which is not sure it can make it.

Let's drop it then, I can validate the parameter in the python part.

I can make it. it would take 1 - 4 hours.

Isn't it already the current behaviour?

I can make it. it would take 1 - 4 hours.

Isn't it already the current behaviour?

Yeah, that's true. now, it's handling both cases
https://github.com/lfoppiano/streamlit-pdf-viewer/commit/9196b8509bf2f285062af0eff71799037d3d7be4

@t29mato could you please summarise what exactly should be tested? The point number 2 is it implemented?
Is this implemented in the main or as a PR?

summarise what exactly should be tested?

Test for Unset height (Default Behavior):

When height is not set (i.e., None), the PDF should display in its entirety without any overflow.

Test for Set height (Numeric Value):

If a numeric value is assigned to height, the PDF should display with the specified height in pixels, and overflow should be enabled.

Test for Invalid height Input:

The function should throw an error if height is given a value other than a number or None.

The point number 2 is it implemented?

Is point number 2 The user does set height = 700 (absolute number) ? Then, yes, I've implemented it.

Is this implemented in the main or as a PR?

Sorry, I don't understand this.


Nodes initially "offline" and not catching up until a block is gossiped?

The description is a little bit vague.
Node would be offline in two situations:

when it was just created;
when it hadn't received any message for 8 slots.

For 1, when the node was just created, it should ask its peers for their best tip and use that to trigger catchup. Only in situations that it fails to collect peers' best tip, then it needs to do wait for network transitions to trigger catchup manually.
For 2, it probably means there's something wrong with the network?

The description is a little bit vague.
Node would be offline in two situations:

when it was just created;
when it hadn't received any message for 8 slots.

For 1, when the node was just created, it should ask its peers for their best tip and use that to trigger catchup. Only in situations that it fails to collect peers' best tip, then it needs to do wait for network transitions to trigger catchup manually.
For 2, it probably means there's something wrong with the network?

After looking into #7294, it seems like the online status could be broken.

After looking into #7294, it seems like the online status could be broken.


chore(Styleguide): Add prop descriptions Gamut A-F GM-42

Changes

Adds prop descriptions for docgen typescript for prop tables on components A-F.
Moves card to atoms

Further work

Button fitText prop no longer is mapped to a style and is no op.  (New ticket: GM-52)
ContentContainer, FlexGrid, and others do not have the standardized as prop for specifying a custom element.  (New ticket: GM-53)

PR Checklist

[ ] Related to Abstract designs:
[x] Related to JIRA ticket: GM-42
[x] I have run this code to verify it works
[ ] This PR includes unit tests for the code change

🚀 Styleguide deploy preview ready!https://5f0cecb9e45bb4411642129b--codecademy-styleguide.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@1.7.40-alpha.0ffecc.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-labs@1.7.41-alpha.dc585d.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5f0dc5369d3f07ee03e037fa--codecademy-styleguide.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@1.7.41-alpha.23b189.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5f0dd74b0b196379ec6a4369--codecademy-styleguide.netlify.appDeploy Logs

This is fantastic, thanks!
I left a few nits early on that are repeated later in props, but up to you if you want to apply them. Just some copy editing 📝

Thank you!    Found some other places where I was doing the screensizes thing.  Great suggestions :D

📬Published Alpha Packages:@codecademy/gamut-labs@1.7.41-alpha.807f4a.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-labs@1.7.41-alpha.48511e.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5f0e0cb0fd40063d87ef0c2b--codecademy-styleguide.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://5f0e0ef13f36956f07a8cc38--codecademy-styleguide.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@1.7.41-alpha.9037d8.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-labs@1.7.41-alpha.4917cf.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5f0e21931e3ece02875cf0c3--codecademy-styleguide.netlify.appDeploy Logs


Dependencies on PyPI

Hello! It seems that some changes in dependencies list on PyPI are needed. sklearn is now deprecated, scikit-learn should be used instead.

× python setup.py egg_info did not run successfully.
│ exit code: 1
╰─> [18 lines of output]
The 'sklearn' PyPI package is deprecated, use 'scikit-learn'
rather than 'sklearn' for pip commands.

#6 This should help code wise. With the PR you can install the package via
pip install git+https://github.com/David-Woroniuk/chowtest#egg=chowtest

Hello, I received the following error after "pip install git+..."
ERROR: chowtest from git+https://github.com/David-Woroniuk/chowtest#egg=chowtest does not appear to be a Python project: neither 'setup.py' nor 'pyproject.toml' found.


Add tutorial for load the data from other formats

We can add a tutorial in the document about loading data from numpy, scipy, matlab, networkx and other common data formats. Some of the data loaders can be found in https://pytorch-geometric.readthedocs.io/en/latest/modules/utils.html

PyGOD use torch_geometric.data.Data to handle input data. You only need to convert your data to PyG format, and PyG has already provided many useful resources about it:

Tutorial: Create your own datasets
Tutorial: Loading graphs from CSV
Plenty of utility functions under torch_geomertic.utils: to_scipy_sparse_matrix, from_scipy_sparse_matrix, to_networkx, from_networkx, etc. (Link)


Getting Error: "IndexError: index 0 is out of bounds for axis 0 with size 0"

Describe the bug
While executing ica.plot_properties(raw, picks=ecg_indices), the compiler return an error
IndexError: index 0 is out of bounds for axis 0 with size 0
The full error trail:
    ica.plot_properties(raw, picks=ecg_indices)
  File "lib\site-packages\mne\preprocessing\ica.py", line 1746, in plot_properties
    figsize=figsize, show=show, reject=reject)
  File "\lib\site-packages\mne\viz\ica.py", line 427, in plot_ica_properties
    psds, freqs = psd_multitaper(epochs_src, picks=picks, **psd_args)
  File "<decorator-gen-61>", line 21, in psd_multitaper
  File "\lib\site-packages\mne\time_frequency\psd.py", line 334, in psd_multitaper
    n_jobs=n_jobs, verbose=verbose)
  File "<decorator-gen-48>", line 21, in psd_array_multitaper
  File "\lib\site-packages\mne\time_frequency\multitaper.py", line 422, in psd_array_multitaper
    n_times, sfreq, bandwidth, low_bias, adaptive)
  File "<decorator-gen-47>", line 21, in _compute_mt_params
  File "\lib\site-packages\mne\time_frequency\multitaper.py", line 350, in _compute_mt_params
    interp_from=interp_from)
  File "lib\site-packages\mne\time_frequency\multitaper.py", line 95, in dpss_windows
    dpss = _get_dpss()(N, half_nbw, Kmax)
  File "\lib\site-packages\scipy\signal\windows\windows.py", line 1938, in dpss
    if w[w * w > thresh][0] < 0:
IndexError: index 0 is out of bounds for axis 0 with size 0

Steps to reproduce
filename = r'S17_3.mff' # can be downloaded via the link: https://drive.google.com/drive/folders/14R_2A-ceqOFvRkDZH6Skm4CqF5B-K7Co?usp=sharing
raw = mne.io.read_raw_egi(filename, preload=True)
montage = mne.channels.make_standard_montage('GSN-HydroCel-129')
montage.ch_names[-1] = 'E129'
raw.set_montage(montage, match_case=False)
raw.filter(1., 40.)
raw.crop(0, 595)  # crop huge artifact at the end
filt_raw = raw.copy()

ica = ICA(n_components=30, random_state=97)
ica.fit(filt_raw)
ica.exclude = []

# find which ICs match the ECG pattern
ecg_indices, ecg_scores = ica.find_bads_ecg(raw)
ica.exclude = ecg_indices

# barplot of ICA component "ECG match" scores
ica.plot_scores(ecg_scores)

# plot diagnostics
ica.plot_properties(raw, picks=ecg_indices)

# plot ICs applied to raw data, with ECG matches highlighted
ica.plot_sources(raw)

Additional information
System:
Windows 10
anaconda3
mne-0.21.dev0
Python 3.7
Link to download S17_3.mff

I can't replicate on Linux, it's possible that this is a windows issue. Before I boot a VM to look into it, when you hit the error can you try:
>>> import pdb; pdb.pm()
(Pdb) p (NW, Kmax, sym, norm, M, thresh, np.abs(m).max())
...

and let us know the output? It can help with debugging

Thanks for the response @larsoner .  Im using PyCharm and I am not familiar with pdb for troubleshooting. May I know other approach to capture specific output that you have in mind?

Can you run Python from a standard terminal instead? Copy-pasting your code (with an import mne at the top) into some file then do:
python -i whatever.py

and you should end up dropped to a prompt where you can import pdb; pdb.pm(). But really I'm surprised Pycharm doesn't give you a way to interact with the Python terminal once a script completes. It's standard Python behavior if you pass python -i or use "interactive mode".
It's also likely that pycharm has some built-in debugging capability where you can have it drop to a prompt if it hits an error.

Thanks for the clear instruction @larsoner . Surprisingly, the plot was rendered without any hiccup when executing the commands using "interactive mode". However, there is a catch whereby the figure was plotted properly only during the first execution (i.e.,  execute only the ica.plot_properties(raw, picks=ecg_indices)). But, after I close the pop figure and execute the line ica.plot_properties(raw, picks=ecg_indices) , the same error  "IndexError: index 0 is out of bounds for axis 0 with size 0" was shot.
The same error also was given if executing the line ica.plot_properties(raw, picks=ecg_indices) while the first figure is still open.

Hopefully this was fixed by #8103, let's reopen if not

I still have the same problem with todays 0.22dev version, also on windows 10. If I run it through debugger it works. Somtimes it works one time and not the second
If I run your post-mortem debugging thingy I get this:
 (4.0, 8, True, 2, 2048, 0.00048828125, 2048)```

(note that I changed the np.abs(m) to np.abs(M) hope that was intended)

@behinger do you have NumPy 1.19.4? I wonder if it's this Windows bug:
https://developercommunity2.visualstudio.com/t/fmod-after-an-update-to-windows-2004-is-causing-a/1207405
https://github.com/numpy/numpy/issues/16744
For now the workaround is to install NumPy 1.19.3 on Windows.

nice! yes indeed I had Numpy 1.19.2 installed. With 1.19.3 it works now.
Thanks alot! it might also explain random SVD errors I got :)

Is there a specific Scipy library we should maybe be using too?
I've got a similar error when trying to open multiple figures from my epochs object and have tried installing my Numpy to version 1.19.3 and have MNE 0.21.2 installed. I'm using a Windows machine too.


Rework node status to use watches

A very early draft to see if I am going in the right direction with this. Contains three functions (ready/not ready/unknown) that use a watch to determine the status of the given node. Related to krkn/issues/243.
I Haven't tested this with a proper Krkn test, yet, but I have tested it using a simple python script to ensure that the functions return the correct status.

I was successfully able to run the reboot scenario now on a 3 worker azure cluster, a couple of comments though

@obrienrobert can you please rebase the patch?


Add option to build configuration file

If a configuration file isn't found, ask a user for some basic information (like url, user/pass), then save it to the folder.

Whenever a Thoom::ConfigFileError is thrown, a simple wizard is shown that asks the user for the basic info that is needed: url, username, password.


Add support for Bigtable

Could you please add support for Bigtable?
Support for the Bigtable Admin API can be added to the master branch as it does not require any additional work beyond just calling the gRPC APIs. We can start with the Bigtable Data API on a separate branch and merge to master once it's ready for general consumption; Bigtable protocol requires additional development beyond just calling the gRPC APIs.
Thanks!
/cc: @jgeewax, @sduskis

@jdpedrie -- can you please document how to generate PHP code for a new service from gRPC protos so we can get started?

/cc @michaelbausor

Hi @mbrukman, I can help out with generating PHP libraries for Bigtable, I will follow up directly.

Closed by #731.


Run CI build using Github Actions

This PR adds CI builds using Github Actions.
Right now it runs a build on master on push to master and every day at 9AM UTC.
PRs are also built.
In the beginning we can keep this in parallel to the build on TravisCI until we are confident that Github works as we need it.
Open points:

Github Actions also supports building on Windows, I plan to add this later
I have to look up how to implement the trigger that we currently use for the asciidoctor upstream build. Until we have that we have the scheduled daily build though which should also capture changes, even though it is of course not easy to track down the breaking changes if there were multiple commits to Asciidoctor within a day.

This is probably a way trigger an upstream build from the asciidoctor/asciidoctor repo: https://help.github.com/en/actions/automating-your-workflow-with-github-actions/events-that-trigger-workflows#external-events-repository_dispatch

Super cool!

Thanks! :)
I’ll be traveling next week, I hope I’ll find some time then to add the upstream triggers


Background Color for Dropdowns

The background color of the dropdown menus is white -- rendering the text unreadable. It is also fairly ugly, and it can be fixed. The other themes seem to change the background of the dropdowns fine, so I know it can be done.
Here's a screenie with the Atom Material Theme:

Here's a screenie with the Atom Dark Theme:

Edit:
I took a look at your index.less file... It seems as though you aren't including the dropdowns.less file. Is that the issue?

What version of the theme are you using? This was fixed a few releases back.

I used the version that the install tab uses. 0.5.0?

Yep, that's the latest. I just checked my code and I made the fix specifically for Windows hosts. I just pushed some code to make the fix available to other platforms. Will publish a patched version as soon as I finish setting up a Ubuntu virtual machine to test it.

I'm using arch linux with gnome if it makes any difference.

should be fixed in 0.5.1. Cheers!

Thanks mate!


is there a mistake about the np.real ?

Hi professor ,
in my mentioned link , the line 137 ,the np.real is used to get the real part of the complex ,but in your book ,P498 ,there is "where Im(z) is the imaginary part of a complex number".
is there some wrong with that?
https://github.com/Allen-Tildesley/examples/blob/94be209763167a52233de6b3b9143255da5d13f8/python_examples/ewald_module.py#L137

Thanks for your interest in the Python examples. In this case, the argument of the np.real function will have zero imaginary part, because term is multiplied by its complex conjugate. So, the function is simply discarding this zero imaginary part, and providing a real quantity for the remainder of the calculation. Does this answer your question? If not, please can you clarify your question, because I don't see where the problem lies.

it was  my mistake. the code is used to calculate potential .I got it wrong (i mixed the potential formula with force formula).
and i have another three  questions.

in the potential formula, there is 0.5,and in the force formula,the 0.5 is gone, but I do not get rid of 0.5 after the derivative.
and the second term in the potential formula has 2Π/V ,but from the chapter6,there should be 1/(2V).
and the third term in the potential formula has a plus sign ,but i think it should minus sign.

I am going to close this issue, as it seems that there is no problem with this example. This is not the right place to ask questions about the book. We maintain a separate corrections page for that. Please, however, recheck your algebra first. If you believe that you have found a mistake, please give the full details there. It may take a day or two before I can respond.


Plot discontinuous lines

Question
Hello,
I'm currently using the Chart class to plot a line chart for a time series data. However, I've encountered an issue when trying to plot a line that has gaps (i.e., discontinuous line).
Here's a simplified version of my code:
chart = Chart()
bid1_line = chart.create_line('bid1_line', width=1, color='#FF0000')
data = df[['time', 'bid1_line']].dropna()
bid1_line.set(data)

In my data, there are periods where 'bid1_line' is NaN, and I would like these periods to appear as gaps in the line chart. However, the current implementation seems to connect the points before and after the gap, creating the impression of a continuous line.
I was wondering if there's a way to create a broken line chart using the Chart class? If not, could this feature be considered for a future update?
Thank you for your help.
Code example
No response

Hi, I also encountered the same situation. I checked and found that when I zoom in, I can occupy non continuous lines, but when I zoom out, it shows continuous lines. By the way, have you solved this problem? If there is a good solution, can you share it?

I changed the line style, still doesn't solve the problem.

nope, unfortunately, I couldn't find any solution for this behavior.

I just solved this problem by spliting the series and put them into two lists
def splitUpSeriesWithGaps(supertrend_positions,chart):
Lines_uB = []
Lines_lB = []
Index_upper = 0
Index_lower = 0
df_length = len(supertrend_positions)
for i in range(0,df_length):
    if(np.isnan(supertrend_positions["upperband"][i]) == True and np.isnan(supertrend_positions["upperband"][i-1]) == False):
        Index_lower = i
    if(np.isnan(supertrend_positions["lowerband"][i]) == True and np.isnan(supertrend_positions["lowerband"][i-1]) == False):
        Index_upper = i
    if(np.isnan(supertrend_positions["upperband"][i]) == False and np.isnan(supertrend_positions["upperband"][i-1]) == True and i != 0):
        Lines_lB.append(supertrend_positions[Index_lower:i])
    if(np.isnan(supertrend_positions["lowerband"][i]) == False and np.isnan(supertrend_positions["lowerband"][i-1]) == True and i != 0):
        Lines_uB.append(supertrend_positions[Index_upper:i])
    if(i == df_length - 1):
        if(np.isnan(supertrend_positions["lowerband"][i]) == True and np.isnan(supertrend_positions["lowerband"][i-1]) == True):
            Lines_uB.append(supertrend_positions[Index_upper:i])
        elif(np.isnan(supertrend_positions["upperband"][i]) == True and np.isnan(supertrend_positions["upperband"][i-1]) == True):
            Lines_lB.append(supertrend_positions[Index_lower:i])
for i in range(0,len(Lines_uB)):
    temp = chart.create_line("upperband",color="red",style="solid",price_line=False)
    temp.set(Lines_uB[i])
for j in range(0,len(Lines_lB)):
    temp2 = chart.create_line("lowerband",color="green",price_line=False)
    temp2.set(Lines_lB[j])

It works fine, I tried to modify the tradingview js code, too hard for me to understand those 8000 lines of code.

Fixed in latest version.
Louis


Some records/repos and methods from initial anoncreds-rs integration

While doing some testing for anoncreds-rs integration and checking if the ongoing AnonCreds API fits properly, I've created some draft classes and done some modifications to interfaces that maybe you find useful.
In this PR we have these classes and some code that helps on the discussion about the fields to include in the interfaces. This is the summary:
About Master Secret:

Creation is part of AnonCreds API and delegated to the registered AnonCreds Holder service. AnonCreds package provides the records and repositories to store it, although the service might not expose its value (in which case it is just stored as a reference)
Usually (I'd say 99% of the time) there will be a single Master Secret per wallet. However, it is technically possible to have more than one. Therefore there is some logic there to handle a default one and so on
In createCredentialRequest we need to pass the masterSecretId, so every Credential derived from it is associated to a particular Master Secret. This parameter can be optional, so we can use the default one as we are currently doing in AFJ
When doing a Presentation, as multiple credentials can be retrieved and can be issued using different Master Secrets each, there should be some logic in the Holder service to prevent this

About Private part of Credential Definition
Surely you already knew this, but I realized that it is not possible for the IssuerService to store the private part and Key Correctness Proof in createCredentialDefinition, because it will not know the credentialDefinitionId beforehand. So these private parts are added to the return (I didn't add any typing to those fields but of course it could be) and the storage is handled by the caller (namely AnonCredsApi).
I did not add a CredentialRecord yet, but I'll have probably some fun with it when collecting and adapting them in HolderService's createProof method.

About Private part of Credential Definition
Surely you already knew this, but I realized that it is not possible for the IssuerService to store the private part and Key Correctness Proof in createCredentialDefinition, because it will not know the credentialDefinitionId beforehand. So these private parts are added to the return (I didn't add any typing to those fields but of course it could be) and the storage is handled by the caller (namely AnonCredsApi).

Interesting. Missed this part. That's unfortunate.... Will think about this a bit more


Make miniwidgets order controlled by configuration file

The miniwidgets are currently not ordered in the same way as the sections. They should be.

Closed in https://github.com/opentargets/platform-app/pull/73


Update sites.js

Personal handcrafted website. Blog with wiki-elements. I included both the black and white icon on the landing page (https://www.edwinwenink.xyz/), so that the icon is correctly displayed both in the light and dark theme. I appreciate the effort you are making!

Welcome to the webring!
I love your website :)

Thanks a lot! I'm really fascinated by your wiki project and the design of 100r.co is sublime. You're setting a good example  to live up to 👍


Support for helm tpl function

Hi!
Helm can render a file template with tpl function, helmfile has readFile option, but it reads file "as is". It would be nice to have tpl alternative here.
Maybe you can:

render files with tmpl.RenderTemplateToBuffer with readFile when file has gotmpl extension
introduce a new function to render file instead of just loading it

@astorath I realized that using tpl seems like the easiest way when you want to extract a HUGE helm chart value whose part(s) needing to be templated.
So I've authored #504 😄

So If you got to want writing something like the below, try tpl in your values.yaml.gotmpl:
set:
- name: foo
  # This doesn't work as you might have expected, of course!
  file: hugefile.txt.gotmpl

@mumoshu thanks, sorry, missed your question in january. My usecase is similar: we use repo with lot's of different configurations for different environments.
If I place all these in helm chart, it exceeds 1Mb limit, so I have to store configs outside and pass as values to the chart.


Convenience utility for installing a provisioning profile

Right now there's a bunch of gnarly script that installs a provisioning profile (in scripts/build.sh). This bloats out the build script and makes it a little harder to see what's going on. It'd be great if this could be added to the new build-tools Python script so that it's just a one-liner in the build script.
The code in question is:
# Install the provisioning profile.
file="macos/Bookmarks_Developer_ID_Application.provisionprofile"
uuid=`grep UUID -A1 -a "$file" | grep -io "[-A-F0-9]\{36\}"`
extension="${file##*.}"
PROFILE_DESTINATION=~/"Library/MobileDevice/Provisioning Profiles/$uuid.$extension"
if [ ! -f "$PROFILE_DESTINATION" ] ; then
    echo "Installing provisioning profile '$PROFILE_DESTINATION'..."
    mkdir -p ~/"Library/MobileDevice/Provisioning Profiles/"
    cp "$file" "$PROFILE_DESTINATION"
else
    echo "Provisioning profile installed; skipping"
fi

Done.


003 model training

@Aishwarya120111 The part for model training is committed. Great if you can review and comment before proceeding
cc @Jing-19 @WeilinHan8

I have updated the the sklearn version in environment.yaml to avoid warning for old sklearn version


HTTP2Transport::networkLoop  ""Could not setup downchannel"  question

Hello,
I have the following error when running  AlexaDirectiveSequencerLibraryTest :

Curl_http_done: called premature == 0
Connection #0 to host avs-alexa-na.amazon.com left intact
Could not setup downchannel, retry count: 1

I also see "ALPN, server did not agree to a protocol" in the log.

ALPN, offering h2
ALPN, offering http/1.1
Cipher selection: ALL:!EXPORT:!EXPORT40:!EXPORT56:!aNULL:!LOW:!RC4:@STRENGTH
successfully set certificate verify locations:
CAfile: amazon.crt
CApath: none
SSL re-using session ID
SSL connection using TLSv1.2 / ECDHE-RSA-AES256-GCM-SHA384
ALPN, server did not agree to a protocol

Any idea of what I am doing wrong? Thanks.

Hi baronlin1474,
I'd like to ask what version of libcurl / nghttp2/ openssl you're using.  HTTP/2 is fairly new, and ALPN is the client / server negotiation to upgrade from HTTP/1(.1) to HTTP/2.
Based on the output you're seeing, I expect this is the cause of your problem.
Please refer to our README.md where we outline minimum requirements of dependencies, particularly:

libcurl 7.50.2 or later
nghttp2 1.0 or later
OpenSSL 1.0.2 or later

Please ensure the versions you're using meet the minimum requirements.  Let us know if this advice does not solve the issue, thanks.

The versions I used are in the following: (meet the minimum requirements)
libcurl : Version: 7.50.2
libnghttp2 : Version: 1.22.0
OpenSSL : Version: 1.1.0e
Is there any other reason why I got the error?

Hi baronlin1474,
Great.  The trace you're showing from our SDK is from an earlier version, at least before the 0.5 release.  We've updated many things (including bug fixes within ACL) since that time.  Could you please download the recent version of the SDK and see if the issue persists?
Thanks.

The 3 newer versions (v0.5, v0.6 and v1.0) are tested.    Still not working.
v0.5
[==========] Running 10 tests from 1 test case.
[----------] Global test environment set-up.
[----------] 10 tests from AlexaCommunicationsLibraryTest
[ RUN      ] AlexaCommunicationsLibraryTest.testConnectAndDisconnect
2017-08-22 02:23:28.494 [  1] I ConfigurationNode:initializeSuccess
2017-08-22 02:23:30.426 [  2] 0 HttpPost:doPostSucceeded:code=200
2017-08-22 02:23:30.426 [  2] 0 AuthDelegate:handleLwaResponseSucceeded:expiresInSeconds=3600
2017-08-22 02:23:30.426 [  2] 0 AuthDelegate:onAuthStateChangeCalled:state=1,error=0
2017-08-22 02:23:30.426 [  1] 0 MessageRouter:connectionStatusChanged:reason=ACL_CLIENT_REQUEST,newStatus=PENDING
2017-08-22 02:23:31.707 [  3] I ?:?::Could not setup downchannel, retry count: 1
2017-08-22 02:23:33.510 [  3] I ?:?::Could not setup downchannel, retry count: 2
v0.6
[==========] Running 10 tests from 1 test case.
[----------] Global test environment set-up.
[----------] 10 tests from AlexaCommunicationsLibraryTest
[ RUN      ] AlexaCommunicationsLibraryTest.testConnectAndDisconnect
2017-08-22 02:26:23.882 [  1] I ConfigurationNode:initializeSuccess
2017-08-22 02:26:25.641 [  2] E HTTP2Transport:networkLoopRetryingToConnect:reason=establishConnectionFailed,retryCount=1
2017-08-22 02:26:29.373 [  2] E HTTP2Transport:networkLoopRetryingToConnect:reason=establishConnectionFailed,retryCount=2
v1.0
[==========] Running 10 tests from 1 test case.
[----------] Global test environment set-up.
[----------] 10 tests from AlexaCommunicationsLibraryTest
[ RUN      ] AlexaCommunicationsLibraryTest.testConnectAndDisconnect
2017-08-22 02:27:24.883 [  1] I ConfigurationNode:initializeSuccess
2017-08-22 02:27:26.825 [  2] E HTTP2Transport:networkLoopRetryingToConnect:reason=establishConnectionFailed,retryCount=1
2017-08-22 02:27:28.824 [  2] E HTTP2Transport:networkLoopRetryingToConnect:reason=establishConnectionFailed,retryCount=2

Hi @baronlin1474,
Are you still experiencing the issue?
If so try running the below command and paste the output here:
curl --version
curl -vl https://www.example.com
curl -vl ttps://api.amazon.com/auth/o2/token

Thanks,
Benny

Closing due to inactivity. Please reopen if you're still having the issue.


add cluster check details for table alicloud_cs_kubernetes_cluster

Is your feature request related to a problem? Please describe.
7.4 Ensure Cluster Check triggered at least once per week for Kubernetes Clusters (Automated)
Describe the solution you'd like
Describe alternatives you've considered
Additional context
https://api.alibabacloud.com/?spm=5176.14113079.top-nav.19.c4fa7815F269Iq#/?product=CS&version=2015-12-15&api=DescribeClusterDetail&tab=DEMO&lang=GO
Need to explore for the API.

API is not available for go sdk for the time being.


Reference time series file type

The reference time series file type has been implemented based on the following json file format:
https://drive.google.com/drive/u/0/folders/0Bz8VFV0kd1eXSGMwVEdWbHdRdDQ
In addition to the above google doc documentation on the json file format, the following elements are optional in the json document:

title
abstract
keyWords
symbol

@alvacouch and @hyi Though this PR is in work in progress state, it would be good to get some feedback on the current state of implementation.

@pkdash - I've been trying this out. A couple of issues I've noticed.
The spatial coverage for the resource seems to get set correctly, but the temporal coverage for the resource is displayed incorrectly. I don't know if this is an issue specific to this implementation/issue:

The Abstract that gets created for the file has a URL in it, but it doesn't get converted to a link:

None of the URLs in the Reference Time Series metadata block for the file are links. So, I can't really do much with the information that is given. @danames - once we have the file in the composite resource with all metadata populated (as @pkdash has done), what is the user supposed to be able to do?  Retrieve the data? Open with?
Do we need the Reference Time Series JSON File Content section since the file is actually in the resource and can be opened by the user?

@pkdash - I looked at the resource map and the resourcemetadata.xml document, and I couldn't find any issues with them in the time that I had to test.
So, the biggest remaining question is - once I've got one of these refts JSON files in a composite resource, what am I supposed to be able to do with it? @danames

@horsburgh Thanks for testing. I will try to resolve the issues you have found. If a decision is made not to display the content of the json file, I can fix that too. I have no idea what one should be doing with the json file in the composite resource.  May be a good question for @danames and @martinseul .

@horsburgh I have tried to fix the temporal data display issue, urls should now be clickable and abstract if contains any urls those should also be clickable.

@danames Can you do some testing of this feature at testservices.hydroshare.org? It needs to go in release 1.12.

@hyi Let me know when you are done reviewing this PR.

@pkdash I am done with code review and left a few comments.

@hyi I have now implemented your suggested changes.

@pkdash +1, with one more comment regarding checking fed_resource_file not None just to be safer. I'll defer that to your discretion.


Mr classic tv

@anywherepilot Hallelujah  you can find amazing great content that we make you happy and laugh and laugh
https://youtu.be/m5w01DvIcZ0

Why:
Closes [issue link]

What's being changed:

Check off the following:

[x] I have reviewed my changes in staging (look for the latest deployment event in your pull request's timeline, then click View deployment).
[ ] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[x] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

Hello
The document is complete the degree of the document is complete approved by
the Superiors of the system, if it caused an error it was because he was
not authorized
El lun., 1 de nov. de 2021 12:59 p. m., github-actions[bot] <
@.***> escribió:

A stale label has been added to this pull request because it has been open
7 days with no activity. To keep this PR open, add a comment or push a
commit within 3 days.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/github/docs/pull/11191#issuecomment-956406297, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AT5ETD5DGM3XJUJIENZPOEDUJ3BPZANCNFSM5HEO4L3A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Lovely


Unable to replay from online log

Hello!
First of all, thank you very much for the amazing research work and the open source code. It's really amazing stuff and great learning!
I wanted to file an issue:
When I try to run the online_log/app.py
I am able to upload the ChaiChainConfig.json but when I try to do a replay and upload the log file, the "File Upload" button is unresponsive.
The logs are quite readable and so is the json file, but it would be super cool if I was able to re run the log visualiser.
I'm on Mac OS, tested on Safari, Chrome and Firefox
Thanks in advance!

Thank you for your feedback!

After you run online_log/app.py, there are three pages, on the main page there are two buttons linked to other two pages. Make sure you are using the right page.

on http://<IP_ADDRESS>:8000/ you can see the real-time log information in run.py;
on http://<IP_ADDRESS>:8000/static/chain_visualizer.html you can upload and visualize a ChatChainConfig.json in CompanyConfig/Default/
on http://<IP_ADDRESS>:8000/static/replay.html  you can upload and visualize a log under WareHouse/xxx/

There may be some old logs that can not be replayed, you can try any software with a timestamp in 20230825 or newer, such as WareHouse/DigitalClock_THUNLP_20230825093556
Every time you try to upload and replay a new log, you can refresh the page. We will improve the demo in the future

Please let me know if it solves your problem.

Hello!
First of all, thank you very much for the amazing research work and the open source code. It's really amazing stuff and great learning!
I wanted to file an issue:
When I try to run the online_log/app.py
I am able to upload the ChaiChainConfig.json but when I try to do a replay and upload the log file, the "File Upload" button is unresponsive.
The logs are quite readable and so is the json file, but it would be super cool if I was able to re run the log visualiser.
I'm on Mac OS, tested on Safari, Chrome and Firefox
Thanks in advance!

Hello my friend
i looked your message and was glad.actually i,m amatur at programing but i suggest you using of chatAI for resolving problem.
if you find it forward to me for study please.
thank you indeed


Toast closes immediately

When the page initially loads, if you open multiple toasts consecutively, the second toast and any subsequent ones close immediately.
Here's the original behavior:
https://github.com/wobsoriano/svelte-sonner/assets/13049130/88a9d6a1-356d-4c11-bb19-ba97935c89ac
And after the internal refactor PR:
https://github.com/wobsoriano/svelte-sonner/assets/13049130/d8258455-c6f2-459c-b0ab-4eaf0d027184

Will look into this! I think I have an idea as to why!

This is only happening immediately after the component mounts right? It seems afterwards it works as expected but want to confirm on your end as well!

I just noticed now that they all close at the same time 😆
https://github.com/wobsoriano/svelte-sonner/assets/13049130/339a62e1-6c11-4e66-994f-faa350df985a
Investigating this right now


add filter to dump()

add filter for dump() function

@9585999 this looks good! Can't thank you enough for your work on the debugger, I'll merge soon.

@darrenjacoby No problem :).
I by the way solved a problem https://github.com/soberwp/controller/issues/43 with a conclusion of unnecessary variables in var_dump (), using this filter.
add_filter('sober/controller/dump', function ($data) {
    $data = collect($data)->except([
        'loop', 'query', 'term', '__currentLoopData'
    ])->all();

    dump($data);
});

But the solution is not universal, because variables remain in the Controller Debugger ... This can be seen in the screenshot.

@darrenjacoby How do you look at it to hang a filter on the sanitize() function?

protected function sanitize()
{
    $data = array_diff_key($this->data['__data'], array_flip(['__env', 'app', 'obLevel']));
    $this->data = has_filter('sober/controller/sanitize') ? apply_filters('sober/controller/sanitize', $data) : $data;
}

This gives control over the output data in the debugger:

add_filter('sober/controller/sanitize', function ($data) {
    $data = collect($data)->except([
        'loop', 'query', 'term', '__currentLoopData'
    ])->all();

    return $data;
});

I like that. I think we need to be more explicit with the filter name though, something like sober/controller/debugger/sanitize unless ultimately this could be added to the actual Controller as well. Let me check that out.

@darrenjacoby Of course, without any problems :),
and by the names of the filters I think really sober/controller/debugger/* a more telling name.

@9585999
All PR's look good, only thing I would prefer is to not add color to the Debugger output. I do like the idea of adding formatting though, and wrapping in a class so that the developer can assign their own styles. Similar to how the Sass partial tinymce in Sage works.

@darrenjacoby
 =>
 =>

How do you look at this option?

@9585999
I think we're going to need to use a class name that is 'safe', and wouldn't conflict with other styles. Perhaps something prefixed, -controller-debugger and then append from there. -controller-debugger-count etc.  also isn't considered valid HTML5, so  would be more appropriate for text based classes, otherwise a div for the wrapper. I'm happy to edit after merging your PR.

@darrenjacoby Let me then while I remove these changes? https://github.com/soberwp/controller/commit/6bcc83200c6069edc48c81a9f2b165e05a56c719

@9585999
Yea cool, I'll add in some classes and merge over the weekend! Thanks for contributing so much, it's such a big help.


mapnik.Image.data() is not documented

Returns underlying image buffer to JavaScript, NAN_METHOD(Image::data). Not documented at http://mapnik.org/node-mapnik/documentation/#mapnik-image

@mikemorris you can add the markup like this if you want to add it :)


Is raspberry pi 3 b+ compatible?  Failed to launch.

Tell us about your environment:

Puppeteer version: 1.13.0
Platform / OS version: raspberry pi 3 b+, ubuntu mate
URLs (if applicable):
Node.js version: 8.10.0

What steps will reproduce the problem?
const browser = await puppeteer.launch({
   headless: true,
   args: ['--no-sandbox', '--disable-setuid-sandbox']
});

this gives me error message
Error: Failed to launch chrome!
/home/xxxx/.../node_modules/puppeteer/.local-chromium/liniux-641577/chrome-linux/chrome: Syntax error: word unexpected (expecting ") "

So i tried adding executablePath, for instance (not sure if this is valid though)
const browser = await puppeteer.launch({
   headless: true,
   executablePath: '/home/xxxx/.../node_modules/puppeteer/.local-chromium/liniux-641577/chrome-linux/chrome',
   args: ['--no-sandbox', '--disable-setuid-sandbox']
});

but it gave me
Error: Failed to launch chrome! spawn
/home/xxxx/.../node_modules/puppeteer/.local-chromium/liniux-641577/chrome-linux/chrome ENOENT

and if i do chromium-browser --version it says
Chromium 73.0.3683.75 Built on Ubuntu, running on Ubuntu 18.04

Thank you guys.

solved.

Hello,
How have you solve this ?

Same problem here, How hava you solve this ???

@SeongwoonHong I'm having this issue too. Please tell us how to solve this!

Try using your chromium-browser, rather than Puppeteer's chrome which might not work on the PI's ARM architecture.
const browser = await puppeteer.launch({         executablePath: 'chromium-browser'   });
I had to do this on a PI 4.

For anyone for whom the above mentioned solution doesn't work, make sure that you have chromium installed on your RPI or install it using.
sudo apt-get install chromium-browser --yes


[FEATURE] Option to give legend a title

Expected Behavior
I would like to have a title on the legend that says what type of data the legend represents.
For example, in Radar Chart in the docs (http://www.chartjs.org/docs/#radar-chart), the legend might have the title "Datasets", then instead of "My first dataset" and "My second dataset", the legend labels could be "My first" and "My second".
The label "My first dataset" makes sense for the data the example is presenting. But for other data types, the legend labels are not self-explanatory.
Context
I have a radar chart showing the number of users who have access to certain things by the company to which the user belongs. Upon looking at the chart, people don't immediately understand that the names in the legend are companies. Adding "Company" after each company as is done with datasets in the docs example ends up looking silly.

The legend is implemented as a plugin. Probably the best way to do this would be to create your own legend plugin and use that.

Closing per my answer above since this hasn't been requested enough to place in the core.


ncdu needs a "refresh" key

When using rclone ncdu, any extra output messes up the screen. E.g., one of those “Your browser didn’t send a complete request” messages (or perhaps from some other process running in the same terminal session). It would be nice to have a “refresh” key (perhaps Control+R or L?)

If you want to have a go at this @barkofdelight then first set up a go dev environment - make sure you've either set GOPATH or are happy to use ~/go/ as the go dev environment.
go get -u github.com/ncw/rclone will install the rclone source in ~/go/src/github.com/ncw/rclone.
You'll need to edit cmd/ncdu/ncdu.go.  You'll find the keyboard handling loop easily enough.  You'll then need to force a complete redraw somehow - take a look in the termbox docs.
Add the code, update the docs and you are good to go! (The UI for ncdu doesn't have tests alas)

Ok, all went as expected. And yes, it's just one line of code and a couple of lines of documentation.
Now I suppose I fork in order to commit? Or just commit to a branch?
Thanks.

The usual thing would be to commit to a branch locally, then fork the rclone project in github, add a git remote pointing to that forked repository, push the branch to your fork on github then click the pull request button on the web site.
That was a super quick overview - let me know if you need more help!

Ok, it all seemed to work. Your contributing guidelines are nicely written!

Ok, it all seemed to work. Your contributing guidelines are nicely written!

Thank you!
This will be in the latest beta now - thank you very much for your contribution :-)

The refresh key ^L doesn't work on Windows, correct?


failed to resolve name error occrued

I have add my etcd endpoints into a array and I try to telnet these endpoints, all are successed
moreover, when this code can be accepted
Client client = ClientBuilder.newBuilder().setEndpoints(endPointList).build();
however when my process try to connect etcd , this error shows:
2017-08-10 22:58:24  [ grpc-default-executor-0:1 ] - [ WARN ] com.coreos.jetcd.resolver.SmartNameResolver.doResolve(SmartNameResolver.java:118) Error wile getting list of servers
com.coreos.jetcd.exception.EtcdException
at com.coreos.jetcd.exception.EtcdExceptionFactory.newEtcdException(EtcdExceptionFactory.java:9)
at com.coreos.jetcd.exception.EtcdExceptionFactory.newEtcdException(EtcdExceptionFactory.java:13)
at com.coreos.jetcd.resolver.SmartNameResolver.doResolve(SmartNameResolver.java:111)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
at java.lang.Thread.run(Thread.java:745)
Caused by: java.lang.IllegalStateException: Unable to resolve endpoints [http://<IP_ADDRESS>:2379, http://<IP_ADDRESS>:2379, http://<IP_ADDRESS>:2379]
... 4 more
Aug 10, 2017 10:58:24 PM io.grpc.internal.ManagedChannelImpl$NameResolverListenerImpl onError
WARNING: [io.grpc.internal.ManagedChannelImpl-1] Failed to resolve name. status=Status{code=NOT_FOUND, description=null, cause=null}
it seems the http://<IP_ADDRESS>:2379 http://<IP_ADDRESS>:2379 http://<IP_ADDRESS>:2379 can't be resolve. why?

Looks strange, can you provide the full code ?

@lburgazzoli of course
try {
String path = "/home/kptech/etcd/etcdendpoints";
File file = new File(path);
InputStream inputStream = new FileInputStream(file);
byte bytes[]=new byte[(int)file.length()];
inputStream.read(bytes, 0, bytes.length);
inputStream.close();
String endpoints =  new String(bytes);
String[] endPointList = endpoints.replace("\n","").split(" ");
for(int i = 0; i < endPointList.length; i++) {
endPointList[i] = "http://" + endPointList[i];
System.out.println(String.format("by jony %s",endPointList[i]));
}
Client client = ClientBuilder.newBuilder().setEndpoints(endPointList).build();
Watch watchClient = client.getWatchClient();
return watchClient;
} catch (Exception e) {
e.printStackTrace();
return null;
}
this is the code initialize the client
and in the file
/home/kptech/etcd/etcdendpoints
it write like this:
<IP_ADDRESS>:2379 <IP_ADDRESS>:2379 <IP_ADDRESS>:2379
and when the program run this code
Watcher watcher = watchClient.watch(ByteSequence.fromString(whiteListKey));
this error occurs!

Sorry, can you test with the latest source ? There was a bug with the service loader so the resolver was unable to load resolvers for protocols.

@lburgazzoli OK, let me have a try. thanks for your help,In chinese there is nearly 0 o'clock ,can I give you a feedback tomorrow?

@lburgazzoli sorry, how can I get the latest version? I only find and build the jetcd-0.1.0-SNAPSHOT.jar.

you need to build it from sources

@lburgazzoli
this is my workflow
I download the source code
git clone https://github.com/coreos/jetcd.git
and then I build it
./mvnw install -Dmaven.test.skip=true
and then I can locate the jar in
/root/bootstrap/cluster/kpdev/lib/jetcd-0.1.0-SNAPSHOT.jar
is that right?
I rarely use java, So this question seems stupid , sorry.

@shuijing198799 once you git clone jetcd. you need to update the local repo with the upstream. then you build the jar with the latest master.

@shuijing198799 also your workflow is correct. you just need to make sure that the local repo is updated with the upstream one https://github.com/coreos/jetcd.git.

@fanminshi I feel asleep on sofa . sorry. thanks for your help ,I try to fix it now.

@shuijing198799 is the issue fixed with latest master?

@fanminshi  @lburgazzoli  thx for resolve my problem. I have fixed it.


Adopt Rust coding style

As seen here https://aturon.github.io/
Reason being:

snake_case seems to be more common in general for code bases.
Will help to  have the Plugin API consistent across with the rest of the code
Ease the transition to Rust if it would happen.

Mostly done. Will migrate more as moving more code over to Rust anyway.


Missing CODE_OF_CONDUCT.md

There is a redirection to CODE_OF_CONDUCT.md on https://github.com/acikkaynak/aya-website/blob/main/CONTRIBUTING.md?plain=1#L15 but there is no CODE_OF_CONDUCT.md on the root directory.

You can send Read ./CONTRIBUTING.md message to users who doesn't read the CONTRIBUTING.md in peace. Have a nice day/night.

Thanks for your contribution.
I've created a organization-wide code of conduct for GitHub.

To not to have redundant copies (due to maintainability issues) of our code of conduct, I'll try to reference https://acikyazilimagi.com/aya/policies.
What do you think, if we add a reference to this document in our readme.md or contributing.md, would this be sufficient for satisfying to concerns you raise?

Referrencing the policies repo will be so useful. People should know how to contribute. 👍


UVI Request

--- UVI JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from  to before v4.14.253",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=37e4f57b22cc5ebb3f80cf0f74fdeb487f082367"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "37e4f57b22cc5ebb3f80cf0f74fdeb487f082367",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "isdn: mISDN: Fix sleeping function called from invalid context\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v4.14.253 by commit 37e4f57b22cc5ebb3f80cf0f74fdeb487f082367. For more details please see the references link."
}
--- UVI JSON ---

This issue has been assigned UVI-2021-1001961


Add possible tutorials to the navigation.

Could you also add this. With this it is possible to add links to the tutorials in the navigation by only adjusting one template. I could also create a pull request for the navigation.tmpl.

@kozi do you have an example so I can see how this is used?

Thanks. Sorry this too so long to merge. It's now in the published version 1.1.0.


Crashed when modify class variable in catch-all initialization (default initialization) of instance variable

Something like this triggers a runtime crash.
class A
  @@count : Int32 = 0
  @id : Int32 = (@@count += 1)

  def id
    @id
  end
end

p A.new.id

The program outputs BUG: '@@count = @@count + 1' at eval:3:18 has no type (Exception). It also happens in https://play.crystal-lang.org/#/r/4tsa
The bug also happens for the code @id : Int32 = (@@count = 1) (play crystal), so I guess modification to the class variable in catch-all initialization triggers this bug.

The bug doesn't happen after I warp @@count += 1 in a class method.

Duplicate of #6440


Correct count matrices compatibility and consolidate tests

In tests, all tests use mc.cores = 1 as on my Windows computer I kept getting surprising failures.
Changes:

runPseudotimeDE returns an additional column "notes", which contains error messages (or NA), this helps for debugging.
added tests, especially for Seurat objects
revert the change in 0d57 (enhance Seurat compatibility) that subsets the matrix in runPseudotimeDE before sending to pseudotimeDE: this does not seem necessary for Seurat objects but breaks when using count matrices (since we don't use [,,drop=FALSE]).
in my hands changes to parameters such k when calling runPseudotimeDE are not properly transmitted to pseudotimeDE, even though they should be in the ... argument of bplapply. Solved by adding as explicit arguments of the pseudotimeDE call.

Solve bugs based on AlexWeinreb


Vanilla promises of result

Since I couldn't find a configuration solution that I thought was worth the extra complexity and indirection, I'm removing reason-promise in favor of vanilla promises of a result.

Great. I do strongly agree with this one! It also makes it easy to adopt whatever the ReScript team comes up with in due time.


Bump Go verions to latest patch release

To have infra ready in response to https://nvd.nist.gov/vuln/detail/CVE-2023-44487.
/cc @knative/productivity-leads
/cc @knative/security-wg-leads

/test pull-knative-infra-prow-tests-image-build

/test pull-knative-infra-prow-tests-image-build


Lack of distribution mechanism

I suggest to create a script like SDKMAN! and Nix, something like:
$ curl -O https://etcdevteam.com/emerald/install
$ curl -O https://etcdevteam.com/emerald/install.sig
$ gpg2 --verify ./install.sig
$ sh ./install

https://github.com/ethereumproject/emerald-rs/issues/164#issuecomment-315544678


Lambda fromFunctionArn has bug

Describe the bug
The lambda.fromFunctionArn is calling extractNameFromArn
const functionName = extractNameFromArn(attrs.functionArn);

This function will never work because the the cdk.fn.select steps from 0 not 1
function extractNameFromArn(arn) {
    return cdk_1.Fn.select(6, cdk_1.Fn.split(':', arn));
}

So at minimum this should be return cdk_1.Fn.select(5, cdk_1.Fn.split(':', arn));
But also this doesn't take into consideration arns with resource and/or / in them .
since an arn can have the form:
arn:partition:service:region:account-id:resource
arn:partition:service:region:account-id:resourcetype/resource
arn:partition:service:region:account-id:resourcetype/resource/qualifier
arn:partition:service:region:account-id:resourcetype/resource:qualifier
arn:partition:service:region:account-id:resourcetype:resource
arn:partition:service:region:account-id:resourcetype:resource:qualifier
To Reproduce
Command line used or code snippet
Expected behavior
since an arn can have the form:
arn:partition:service:region:account-id:resource
arn:partition:service:region:account-id:resourcetype/resource
arn:partition:service:region:account-id:resourcetype/resource/qualifier
arn:partition:service:region:account-id:resourcetype/resource:qualifier
arn:partition:service:region:account-id:resourcetype:resource
arn:partition:service:region:account-id:resourcetype:resource:qualifier
You have to look for the "/" and adjust.  Something like this:
function extractNameFromArn(arn) {
    var functionName = cdk_1.Fn.split(':', arn);
    var functionReturn = ""
    var functionLength = functionName.length;
    if (cdk_1.Fn.conditionContains(functionName, "/")){
        functionLength = cdk_1.Fn.split('/', arn).length;
        functionReturn = cdk_1.Fn.select(functionLength - 1, cdk_1.Fn.split('/', arn));
    } else {
        functionReturn = cdk_1.Fn.select(functionLength - 1, cdk_1.Fn.split(':', arn));
    }
    return functionReturn;
}

Version:

Cloud9
Typescript
CDK Version 0.33.0

https://docs.aws.amazon.com/general/latest/gr/aws-arns-and-namespaces.html#arn-syntax-lambda
We know for a fact that a Lambda ARN will look like this:
arn:aws:lambda:region:account-id:function:function-name[:version]
 0   1    2      3       4          5          6           7

So as far as I can tell the code would appear to be correct.
What is the error you're experiencing?

This:
const lambdaSingleFunction = lambda.Function.fromFunctionAttributes(this, "LambdaSingleUserRotation", {
        //functionArn: "arn:aws:lambda:" + props.regionName + ":function:function-name",
        functionArn:"arn:aws:serverlessrepo:us-east-1:297356227824:applications/SecretsManagerRDSMySQLRotationSingleUser",

    })
new cdk.CfnOutput(this, "functionLength", {
    value: lambdaSingleFunction.functionName,
})

On synth give me:
/home/ec2-user/environment/set-acaweb/cdk/node_modules/@aws-cdk/cdk/lib/cfn-output.ts:86
throw new Error(Missing value for CloudFormation output at path "${this.node.path}");
^
Error: Missing value for CloudFormation output at path "set-acaweb-dev-LambdaStack/lambda/lambdaSingleFunction"
at new CfnOutput (/home/ec2-user/environment/set-acaweb/cdk/node_modules/@aws-cdk/cdk/lib/cfn-output.ts:86:13)
at new Lambda (/home/ec2-user/environment/set-acaweb/cdk/lib/lambda-stack.ts:34:1)
at new LambdaStack (/home/ec2-user/environment/set-acaweb/cdk/bin/dev/lambda.ts:14:9)
at Object. (/home/ec2-user/environment/set-acaweb/cdk/bin/dev/lambda.ts:18:1)
at Module._compile (module.js:653:30)
at Object.Module._extensions..js (module.js:664:10)
at Module.load (module.js:566:32)
at tryModuleLoad (module.js:506:12)
at Function.Module._load (module.js:498:3)
at Function.Module.runMain (module.js:694:10)
Subprocess exited with error 1
If I change it to arn:aws:serverlessrepo:us-east-1:297356227824:applications:SecretsManagerRDSMySQLRotationSingleUser
The synth works.  But that isn’t the arn, because of the “/”.
I get the error above if I try to use arn:aws:serverlessrepo:us-east-1:297356227824: SecretsManagerRDSMySQLRotationSingleUser
Which should be a valid arn.
I don’t have a log of the deploy error, but it’s something about again about not being able to parse the arn to find the function name.
The cdk.Fn.select function start the array element count at 0, not 1.  So for a valid arn of arn:aws:serverlessrepo:us-east-1:297356227824: SecretsManagerRDSMySQLRotationSingleUser if you use cdk.Fn.select(6 you get nothing
Arn(0):aws(1):serverlessrepo(2):us-east-1(3):297356227824(4): SecretsManagerRDSMySQLRotationSingleUser(5)
I changed the function locally and it works for all cases:
function extractNameFromArn(arn) {
var functionName = cdk_1.Fn.split(':', arn);
var functionLength = functionName.length;
var functionReturn = cdk_1.Fn.select(functionLength - 1, cdk_1.Fn.split(':', arn));
if (cdk_1.Fn.conditionContains(functionName, "/")){
functionLength = cdk_1.Fn.split('/', arn).length;
functionReturn = cdk_1.Fn.select(functionLength - 1, cdk_1.Fn.split('/', arn));
}
return functionReturn;
}
Carol Brothers
Senior DevOps Engineer
[signature_1887532665]
Email<EMAIL_ADDRESS>Phone:<PHONE_NUMBER>
University of California, Office of the President
300 Lakeside Drive, 12th floor, 12216
Oakland, CA 94612
[Title: LinkedIn - Description: LinkedIn icon]https://www.linkedin.com/in/carolbrothers
From: Rico Huijbers<EMAIL_ADDRESS>Reply-To: awslabs/aws-cdk<EMAIL_ADDRESS>Date: Monday, June 3, 2019 at 5:38 AM
To: awslabs/aws-cdk<EMAIL_ADDRESS>Cc: Carol Brothers<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [awslabs/aws-cdk] Lambda fromFunctionArn has bug (#2696)
https://docs.aws.amazon.com/general/latest/gr/aws-arns-and-namespaces.html#arn-syntax-lambda
We know for a fact that a Lambda ARN will look like this:
arn:aws:lambda:region:account-id:function:function-name[:version]
0   1    2      3       4          5          6           7
So as far as I can tell the code would appear to be correct.
What is the error you're experiencing?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/awslabs/aws-cdk/issues/2696?email_source=notifications&email_token=AK5UG7VFMJZGVUGXOQ4YRDDPYUGDNA5CNFSM4HRNNOY2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWZIK4I#issuecomment-498238833, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AK5UG7VGEFOAVYRFFGRYNGTPYUGDNANCNFSM4HRNNOYQ.

Seems to me that arn:aws:serverlessrepo:us-east-1:297356227824:applications/SecretsManagerRDSMySQLRotationSingleUser is not the ARN of a function but the ARN (application id in this case) of a Serverless Application (see AWS::Serverless::Application).

Oh, you are correct.  Sorry.  The function works fine, when you know what you’re doing 😊
Carol Brothers
Senior DevOps Engineer
[signature_1338018081]
Email<EMAIL_ADDRESS>Phone:<PHONE_NUMBER>
University of California, Office of the President
300 Lakeside Drive, 12th floor, 12216
Oakland, CA 94612
[Title: LinkedIn - Description: LinkedIn icon]https://www.linkedin.com/in/carolbrothers
From: Jonathan Goldwasser<EMAIL_ADDRESS>Reply-To: awslabs/aws-cdk<EMAIL_ADDRESS>Date: Tuesday, June 4, 2019 at 12:30 AM
To: awslabs/aws-cdk<EMAIL_ADDRESS>Cc: Carol Brothers<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [awslabs/aws-cdk] Lambda fromFunctionArn has bug (#2696)
Seems to me that arn:aws:serverlessrepo:us-east-1:297356227824:applications/SecretsManagerRDSMySQLRotationSingleUser is not the ARN of a function but the ARN (application id in this case) of a Serverless Application (see AWS::Serverless::Application).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/awslabs/aws-cdk/issues/2696?email_source=notifications&email_token=AK5UG7VFBCYEFSBBXNZJDPDPYYKXLA5CNFSM4HRNNOY2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODW3VYJA#issuecomment-498555940, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AK5UG7TD7UJ5UYER55UPH33PYYKXLANCNFSM4HRNNOYQ.

Alright - I'm closing this since there is in fact no problem :) Thanks for confirming!


Extended Event logs are not cleaned up when running against a remote server

When running against a remote server, the extended event log file is not cleaned up properly. This is because the Drop method in SqlTraceController assumes that the tool is running on the local SQL Server and just deletes from the local filesystem, leaving the .xel files in place on the remote server.
Short of using xp_cmdshell or similar there's not much that could be done to resolve this other in SQLCover itself, but it is probably worth noting this in documentation to ensure that people are prepared.

Agreed, I need to sort out the documentation for this and a few other things, happy to accept pr's to the README :)

I have added a note in the readme about this :)


Add coverage tag, compat

Codecov Report

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

@@            Coverage Diff            @@
##             master       #5   +/-   ##
=========================================
  Coverage          ?   90.38%
=========================================
  Files             ?       10
  Lines             ?      447
  Branches          ?        0
=========================================
  Hits              ?      404
  Misses            ?       43
  Partials          ?        0

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 7f0dd7e...30db5ac. Read the comment docs.


remove machine id in ubuntu images

Copy of #675, #675 could not be tested due to CI constraints.
Authored by @jklippel
"Currently the image contains an id in /etc/machine-id. This file should contain a unique machine ID which uniquely identifies the host (see machine-id(5)).
Some processes rely on this file to generate other values e.g. mac-addresses on bond interfaces.
If the machine-id is the same accross many hosts
these mac-addresses will under certain circumstances be generated identically. Two hosts having the same mac address leads to L2-networking issues.
This commit empties the machine-id file so that it is regenerated on the next boot (which will be the first boot of the machine the image is used for)."

/test-integration-metal3-dev-tools-centos
/test-integration-metal3-dev-tools-ubuntu

/test-integration-metal3-dev-tools-centos
/test-integration-metal3-dev-tools-ubuntu

/test-integration-metal3-dev-tools-ubuntu


DisplayBPM<->DisplayBMP,  EFIAPI for functions of ScreenshotDriver.c

Hi Murphy,
First of all, thank you for sharing these useful UEFI utilities.
I'd like to raise two issues in your UEFI-Utilities-2019:

The folder name DisplayBPM is wrong which should be DisplayBMP.
The prototypes of some functions in ScreenshotDriver.c are missing "EFIAPI". This causes "conflict type" compilation error.

--
Timothy

Fixed DisplayBPM issue. Thanks for pointing it out.
Sorry but I do not get any "conflict type" errors in my build environment - Linux, x64, clang.


Test ubuntu 16.04

Purpose
Fix failing github action using khan/pull-request-comment-trigger@master
For reviewers

Related Github issues

Changes

Tests

Deployment Considerations
Version Notes
{ major | minor | patch }

NOTE: Include the version notes in the squash commit message

/deploy

Failed to deploy prnet


KAFKA-4476: Kafka Streams gets stuck if metadata is missing

break loop in StreamPartitionAssigner.assign() in case partition metadata is missing
fit state transition issue (follow up to KAFKA-3637: Add method that checks if streams are initialised)
some test improvements

@guozhangwang @enothereska @dguy @miguno

Thanks for the PR @mjsax, can you please merge trunk into your branch? It seems like your branch doesn't include https://github.com/apache/kafka/commit/128d0ff91d84a3a1f5a5237133f9ec01caf18d66 for example.

@ijuma Done.

@mjsax - it is not immediately clear to me what this is fixing? Do you think you can add some unit tests to cover the change? They'll probably help with understanding the issue, too.

@ijuma There is another bug...
@dguy Agreed. I had the same thought but wanted to trigger Jenkins build. That's why I opened the PR already.
This is actually tackling two different problems and I did discover a third one. From a bug tracking point of view it would make sense to have different PRs -- however, as all three issue make Reset tool test fails, it's not possible to get a clean build (and be sure there is not fourth one) if we split into multiple PRs... WDYT?

@mjsax I think one PR is fine if there are unit tests for each of the issues uncovered.

Updated this.
Reactivated ResetToolIntegrationTest. Test might still fail until #2227 and #2233 are merged.

Merged to trunk.


pythonPackages.plover.dev: 4.0.0.dev6 -> 4.0.0.dev8

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build plover.dev

Success on x86_64-linux (full log)
Attempted: plover.dev
Partial log (click to expand)
test/test_rtfcre_dict.py ....                                            [ 82%]
test/test_steno.py ..                                                    [ 83%]
test/test_steno_dictionary.py ........                                   [ 84%]
test/test_stentura.py ...................                                [ 88%]
test/test_translation.py ............................................... [ 98%]
........                                                                 [100%]

========================== 488 passed in 1.99 seconds ==========================
postPatchMkspecs
/nix/store/lrzksdf5221rg3n6z615mzy7isqfczhc-python3.6-plover-4.0.0.dev8

Failure on x86_64-darwin (full log)
Attempted: plover.dev
Partial log (click to expand)
copying path '/nix/store/c4hng4j8z4vsyl60d2b3f1prz1mh0k3n-qtwebengine-5.10.1-bin' from 'https://cache.nixos.org'...
copying path '/nix/store/rmclz730l5gxwkpk1kj5yfq71lyyl5k1-qtwebkit-5.9.1' from 'https://cache.nixos.org'...
copying path '/nix/store/mgkzjx5p72pjspsrq6adr0f4zjf76kfd-qtlocation-5.10.1-dev' from 'https://cache.nixos.org'...
copying path '/nix/store/7mx3gn4vh93r798fcd295djnbmxh69d1-qtwebengine-5.10.1-dev' from 'https://cache.nixos.org'...
copying path '/nix/store/crr43zm553bba5nim8rk46yafhiawvz5-python3.6-PyQt-5.10.1' from 'https://cache.nixos.org'...
copying path '/nix/store/rna16248arhn4jy2kkjr39ccim37zsmm-qtwebkit-5.9.1-dev' from 'https://cache.nixos.org'...
copying path '/nix/store/aqb33q08a7a03jv2yz9gi7s8z0jx2vpl-python3.6-PyQt-5.10.1-dev' from 'https://cache.nixos.org'...
building of '/nix/store/kb078zvkdqg1jca32x9ksgnqrrl76bg3-python3.6-pyserial-3.4.drv' timed out after 1800 seconds
cannot build derivation '/nix/store/npwdd8dnhbzdikk3j0nicvzf4i65hsbr-python3.6-plover-4.0.0.dev8.drv': 1 dependencies couldn't be built
error: build of '/nix/store/npwdd8dnhbzdikk3j0nicvzf4i65hsbr-python3.6-plover-4.0.0.dev8.drv' failed

Failure on aarch64-linux (full log)
Attempted: plover.dev
Partial log (click to expand)
[4961/17529] CC obj/third_party/libvpx/bundled_libvpx/vp8_skin_detection.o
[4962/17529] CXX obj/third_party/icu/bundled_icuuc/ubidi.o
[4963/17529] CC obj/third_party/libvpx/bundled_libvpx/dboolhuff.o
[4964/17529] CXX obj/third_party/icu/bundled_icuuc/propname.o
[4965/17529] CC obj/third_party/libvpx/bundled_libvpx/decodemv.o
[4966/17529] CC obj/third_party/libvpx/bundled_libvpx/denoising.o
building of '/nix/store/8nj7ymifklw7cdrjs99s93l4df59aihs-qtwebengine-5.10.1.drv' timed out after 3600 seconds
cannot build derivation '/nix/store/mbxr6gli572pdfkanj6rhlndvw0xh6wm-python3.6-PyQt-5.10.1.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/dps50zbkh7nz4nkgwxs1k2983hpnn152-python3.6-plover-4.0.0.dev8.drv': 1 dependencies couldn't be built
error: build of '/nix/store/dps50zbkh7nz4nkgwxs1k2983hpnn152-python3.6-plover-4.0.0.dev8.drv' failed


Automatic reconnect creates multiple terminal divs

Describe the bug
When wetty disconnects and automatically reconnects, a second terminal <div> element is created. This results in the scrollbar appearing very odd.
To Reproduce

Create a Docker container running wetty.
Start a wetty session in the browser (I use Firefox, but this happens in Chrome and Chromium as well).
Restart the container. Note that wetty shows the Reconnect button briefly.
Wetty automatically reconnects.

A second <div class="terminal xterm"> is created.
Content from the first session is duplicated and pushed over to the scrollbar area.

Expected behavior

The scrollbar area is not polluted with content from the previous session.
No second div is created.

Screenshots

Desktop:

OS: Windows 10 Professional
Browser: Firefox (latest)
Version: Wetty 2.0.3

Additional Information:

Using the browser's reload button works correctly. No additional <div> is created in this case.

This is still an issue for me as of 2.1.2.

Sorry for the stale notices that plugins a lot more aggressive then i intended.
I''ll have fix in the next release


Kotlin sample app

Motivation
Kotlin sample app showcasing Kotlin coroutines with Kentico Kontent Delivery SDK.
Related to #113
Checklist

[ ] Code follows coding conventions held in this repo
[ ] Automated tests have been added
[ ] Tests are passing
[ ] Docs have been updated (if applicable)
[ ] Temporary settings (e.g. variables used during development and testing) have been reverted to defaults

How to test
If manual testing is required, what are the steps?

After merge:
release beta of the deliverySDK
@makma - check if the app is working
release final


Pick all test-fixes from master to 1.4 branch

https://github.com/mesosphere/marathon/pull/5290

Currently in testing on CI, could be rebased and pushed to release/1.4 if tests passed

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-2241. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.


v8 mekanism stderr spamming message: NO

lines like this:
[02:21:20] [Server thread/INFO] [Mantle-STDERR]: [mekanism.common.tile.TileEntityMetallurgicInfuser]: NO
just keeps spamming it out on console

Fixed!


Clean up dead code in the tiflow.

Is your feature request related to a problem?
There are some dead codes in tiflow repo which makes understanding the codebase more challenging.
So, it is an important things to make the codebase more concise.
Any PR that related to clean dead codes can refer to this issue to pass the merge check.
Describe the feature you'd like
as title.
Describe alternatives you've considered
No response
Teachability, Documentation, Adoption, Migration Strategy
No response

Maybe duplicate with https://github.com/pingcap/tiflow/issues/4954.

close by the reason commented above.


[Bug] Guarding or masking a field on an Object Type hides the entire object and not just the individual field

Versions
graphql 1.11.2
graphql-guard 2.0.0
Example Code
module Types
  class PostType < Types::BaseObject
    field :id, ID, null: false
    field :name, String, null: true
    field :masked_or_guarded_field, String, null: true, mask: -> (ctx) { !ctx[:current_user].is_admin?} # or, guard: -> (obj, args, ctx) { !ctx[:current_user].is_admin?}
  end
end

Expected Response
{
  "id": "1",
  "name": "This is a name",
  "masked_or_guarded_field": null, // or just not there at all
}

Actual Response
null

I've also tried it using this syntax with the same results:
field :masked_or_guarded_field, String, null: true do
  mask: -> (ctx) { !ctx[:current_user].is_admin?} # or, guard: -> (obj, args, ctx) { !ctx[:current_user].is_admin?}
end

Let me know if I can provide any more information!

Hi, just upgraded to 2.0 and have found a couple of issues (see issues) so was curious about this one.
It would be good to include what you have in your GraphQL::Schema subclass.
In my case I have something like:
use GraphQL::Guard.new(
  policy_object: MyPolicy,
  not_authorized: ->(*) { handler(*) }
)

If a user is authorized and I add guard: ->(*) { false } to a field it results the not_authorized handler being called. Without the not_authorized handler it raises a GraphQL::Guard::NotAuthorizedError. So maybe these things (or lack there of) are why you're getting your response?

GQL Ruby adds this stuff (I remove it as it find it useless/inconsistent). Maybe this factors in to why you get the response you do?

That's what I thought at first too, but I added this and it's what I currently have in my GraphQL::Schema:
use GraphQL::Guard.new(
  	not_authorized: ->(type, field) do
      GraphQL::ExecutionError.new("Not authorized to access #{type}.#{field}")
    end
  )

Which I believe should only log an error message, but not nullify the entire object.

@sshaw Just tried removing the prepapre_variables method from my graphql_controller but unfortunately, I'm experiencing the same problem

Okay! I've figured something out and got it "working", but I feel there may still be improvements that can be made to the gem. It's all based around this section of the docs: https://github.com/exAspArk/graphql-guard#error-handling.
# By default it raises an error
not_authorized: ->(type, field) do
  raise GraphQL::Guard::NotAuthorizedError.new("#{type}.#{field}")
end

This behaves as I would expect, it raises an error and the query fails.
# Returns an error in the response
not_authorized: ->(type, field) do
  GraphQL::ExecutionError.new("Not authorized to access #{type}.#{field}")
end

I would expect this to return either 'null' or the error for the individual field, but not the whole object type. Could this possibly be because my actual field was an integer value and it just couldn't cast the error to an integer? I didn't see any error that indicated that being the reason, but I wouldn't rule it out.
not_authorized: ->(type, field) do
end

This works and only nulls out the object level field I want. Interestingly enough returning nil (compared to actually nothing) actually returned 0 (I'm guessing because the field was of an Integer type).
--
Upon further review, after I started this comment, the following technically "works" if I change my field type to String instead of Integer.
not_authorized: ->(type, field) do
  GraphQL::ExecutionError.new("Not authorized to access #{type}.#{field}").to_s
end

But of course, it returns Not authorized to access #{type}.#{field} for the field which I don't believe is an ideal response.
--
Now that I've gone through these few different scenarios, I think the real bug is that if the not_authorized lambda returns a value that can't be cast to the requested field it just nullifies the entire object but logs/raises no error. Ideally, it would raise/log an error, or at least only nullify the field instead of the whole object.
Hopefully, that helps! At the very least I'm now able to use the gem as I intended with the below, but let me know if there's anything else I can assist with!
not_authorized: ->(type, field) do
end

Thanks for the info. I also see with the latest version of the Ruby GraphQL library one cannot return a String from rescue_from any more.