Create new stores/tables to existing DB

In my use case, I need to add new tables based on user interaction.  Basically whenever the user selects a certain stock, the client gets data from server, creates a new store (store name same as stock ticker) and then saves the data into that store using bulkPut method.
It works great the first time I am creating the database and the first stock store.  But my attempts to dynamically add stores to the DB is throwing errors
Code so far:
          const storeName = `${ticker.symbol}_${exchange}_${resolution.type}`;
           const db = new Dexie(`App`);
           const tableNames = db._storeNames;
           tableNames.push(storeName);
           var stores = {};
           tableNames.forEach(function (each) {
               stores[each] = "date"
           });
           db.version(1).stores(stores);
           const table = db[storeName];
           table.bulkPut(ohlc).then(function(lastKey) {
              console.log("Successfull loaded",lastKey)
           }).catch(Dexie.BulkError, function (e) {
               console.error ("Some raindrops did not succeed");
           });

Please let me know how do I dynamically add stores to an existing DB

There is a similar question at stackoverflow you could have a look at https://stackoverflow.com/questions/39000780/errors-with-indexeddb-versions-and-dexie-js?rq=1

That said, as IndexedDB is architected around static schemas (all DB connections has to close before mutating the schema), I would advice you to rethink the architecture to use static schema instead if possible. In principle, there's no reason to create new tables for new data as you could store the stock identifier in a property and index that together with your other indexes.
Let's say you're indexing price and have the stock per table.
var dynamicSchema = {};
dynamicSchema.stockX = "id,time,price";
db.version(x).stores(dynamicSchema)

...you could replace that with:
db.version(x).stores({
   stockEvents: 'id,[stock+time],[stock+price]'
});

And to query all prices within a given time range for a certain stock:
db.stockEvents.where('[stock+time]')
  .between(
    [theStock, fromDate],
    [theStock, toDate])
  .toArray();

And to query all times a certain stock has gone over $25:
db.stockEvents.where('[stock+price]')
  .between(
    [theStock, minPrice],
    [theStock, Infinity])
  .toArray();

Much of this is covered in this page

Great.  One last query.  Should I use bulkPut or just put to store the price data against a given stock within the table.  Say the stock is AAPL and the price data (which is an array) that I retrieved from server is in the variable OHLC.  How do I store this array data against AAPL in your suggested schema
OHLC format:
[
[date1,open1,high1,low1,close1,volume1],
[date2,open2,high2,low2,close2,volume2],
[date3,open3,high3,low3,close3,volume3],
[date4,open4,high4,low4,close4,volume4],
]

Use bulkPut()
const data = [
[date1,open1,high1,low1,close1,volume1],
[date2,open2,high2,low2,close2,volume2],
[date3,open3,high3,low3,close3,volume3],
[date4,open4,high4,low4,close4,volume4],
];

await db.tableName.bulkPut(data.map(a => ({
  date: a[0],
  open: a[1],
  high: a[2],
  low: a[3],
  close: a[4],
  volume: a[5]
})));

Per your schema
db.version(x).stores({
stockEvents: 'id,[stock+time],[stock+price]'
});
Will I require the stock name (AAPL) to be part of the array or somewhere which makes it possible to make those queries you suggested

Yes, so just add it into the bulkPut() snippet I suggested.

Based on your data sample and my bulkPut() suggestion, depending on what queries you are going to make, you might want to index '[stock+date]'
db.version(X).stores({
  stockData: `
    ++id,
    [stock+date]`
});

Then you can query a certain stock's data between two given dates. I suppose that's the use case. If you need to make more advanced queries, just add additional indexes.

When I ran the below query, I got the below error
Declaration: // Adding id in the schema was throwing an error on bulkPut
const db = new Dexie("GoCharting");
db.version(1).stores({
    stockEvents: '[stock+date]'
});

Query:
db.stockEvents.where('[stock+time]').below([theStock, startTime]).reverse().limit(limit).toArray().then((data)=> {
					//do Something
				})

SchemaError: KeyPath [stock+time] on object store stockEvents is not indexed

this would be useful for timeseries table naming design, if you are planning to name your tables with  yyyy-mm-dd for large sets of data 50gb / year


Date picker issue on mobile safari

Try to book a holiday on safari

The bootstrap modals are limited in iOS:
http://getbootstrap.com/getting-started/#support-fixed-position-keyboards

Found a fix as described here: https://github.com/twbs/bootstrap/issues/15852#issuecomment-75311227


Weathermap is not getting updated with latest traffic

I am using CACTI 1.2.18 version with 1 main poller and 3 remote pollers. Weathermap plugin is installed (0.98a). Weathermap gets updated everyday when the cron job runs. We are now facing issues with a particular weathermap as it is not getting updated after the cron job runs. Some lags show incorrect traffic. Also certain devices in application is not seen in weathermap.
We have checked the files getting updated when the cron job runs. Everyday the file gets updated with current date. Multiple analysis of script for automation (cron job) is done. But couldn't find the root cause.
Could you please suggest a solution for this.

Moving this to plugin_weathermap

Upgrade to the 1.x version.  But read the readme carefully.


Refactor analysis-runner into separate table

Stop mixing analysis-runner records:

Create new table with meta fields as first class fields
Create entry for graphql
Stop projects from being completely deleted (to avoid deleting analysis-runner records)
Fix grid with new changes

Not many visual changes

Codecov Report
Attention: Patch coverage is 85.84906% with 30 lines in your changes are missing coverage. Please review.

Project coverage is 76.63%. Comparing base (8b7b69e) to head (5bd7d07).

Files
Patch %
Lines

api/routes/analysis_runner.py
52.17%
11 Missing :warning:

api/graphql/schema.py
75.67%
9 Missing :warning:

db/python/utils.py
0.00%
3 Missing :warning:

test/data/generate_data.py
25.00%
3 Missing :warning:

db/python/tables/project.py
0.00%
2 Missing :warning:

db/python/layers/analysis_runner.py
94.44%
1 Missing :warning:

models/models/analysis_runner.py
98.03%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##              dev     #699      +/-   ##
==========================================
+ Coverage   76.47%   76.63%   +0.15%
==========================================
  Files         143      148       +5
  Lines       11515    11704     +189
==========================================
+ Hits         8806     8969     +163
- Misses       2709     2735      +26

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


add getHitbox, collidesWithBlockAtPos

yes

entity.getHitbox()-> returns [pos, pos]
entity.collidesWith(pos, block)-> simulates and returns if entity will be 'pushed' = 'collides' by that block at certain position.
i.e. Item colliding with honey block?


Multiple Product Images

Is this on the agenda?  Would be great to have this and thumbnail versions of each image as well.
Also wondering why the nifty-attachments gem was used instead of using something like paperclip?

I'm closing this Issue for the moment as this functionality will be part of attachments. Please see & comment on this PR #169 for more info

@m-codebastards I'm unable to add cards to this Trello board.

I've created trello board to put, discuss and vote all ideas. There is also column 'In progress' to see who is working on what. What do you think? https://trello.com/b/6ctllnYz/shoppe-core-development

+1 for carrierwave and multiple images per item

has the multiple upload functionality been added now?


Warning when used with surface plots

The data cursor seems to function correctly when used with a surface plot, but the first click to place the cursor results in the warning

/usr/lib/python3/dist-packages/matplotlib/collections.py:360: UserWarning: Collection picker None could not be converted to float
% self._picker)

Minimal example demonstrating the problem below:
import matplotlib.pyplot as plt
from mpl_toolkits.mplot3d import Axes3D
import mpldatacursor as mpldc

x = np.linspace(-1, 1, 10)[np.newaxis]
y = x.T
z = np.sqrt(x**2 + y**2)

fig = plt.figure()
ax = plt.axes(projection='3d')
xg, yg = np.meshgrid(x, y)
ax.plot_surface(xg, yg, z, rstride=1, cstride=1)
mpldc.datacursor()
plt.show()

That is a harmless warning that has (iirc) been fixed upstream.


Incompatibility with black

Black reformats pyi files.
The format is not compatible with flak8-pyi.
Is there a way to establish a friendship between these very useful tools?
I have no configuration except max-line-length = 88.
As a workaround, I've disabled the plugin for yarl project but love to re-enable it back.

What is the exact incompatibility?

yarl/__init__.pyi:9:1: E302 expected 2 blank lines, found 1
yarl/__init__.pyi:32:5: E301 expected 1 blank line, found 0
yarl/__init__.pyi:35:5: E301 expected 1 blank line, found 0

Looks like black removes blank lines: https://github.com/aio-libs/yarl/blob/master/yarl/init.pyi

I know what issue you're facing and unfortunately it's a #wontfix.
Black follows the intended .pyi style described in PEP 484 and subsequently elaborated in typeshed docs. Flake8's E301 and E302 (and quite a few others) are incompatible with this style.
Flake8-pyi introduces new behavior to Flake8 but due to how Flake8 is built, Flake8-pyi cannot disable or change behavior of other warning codes :(

Should I suppress E301 and E302 in setup.cfg?

If you're using Black and have CI or pre-commit to help you if you forget, then there's little point leaving E and W warnings on.

Got you. Thanks!


Address issue with more modern builds of libpq

cc: @h-vetinari @akrherz
Checklist

[ ] Used a static YAML file for the patch if possible (instructions).
[ ] Only wrote code directly into generate_patch_json.py if absolutely necessary.
[ ] Ran pre-commit run -a and ensured all files pass the linting checks.
[ ] Ran python show_diff.py and posted the output as part of the PR.
[ ] Modifications won't affect packages built in the future.

The packages might as well be called broken:
osx-arm64::libpq-15.4-h18bfe24_4.conda
osx-arm64::libpq-16.1-hd435d45_1.conda
osx-arm64::libpq-16.1-hd435d45_2.conda
osx-arm64::libpq-15.4-hd435d45_3.conda
osx-arm64::libpq-16.1-h9aca644_4.conda
-    "openssl >=3.2.0,<4.0a0"
+    "openssl >=3.2.0,<3.2.0a0"

means they are uninstallable.

As mentionned in https://github.com/conda-forge/postgresql-feedstock/issues/191
it seems like only the new version 16 builds are affected, and they might just be outright broken given the constraints.


remove comma two support

commatwo_master is the place for the remaining comma two work: https://github.com/commaai/openpilot/commits/commatwo_master
The jenkins pipeline there will push to release2-staging instead of master-ci.

Can remove:

https://github.com/commaai/openpilot/blob/master/common/file_helpers.py#L84


Poor UI

The user interface is not good.

Can't really close this issue yet because it's involved with UI, so I'll have to test it out on real users before then.


How to combine/split vectors use rvv intrinsics?

Hi,
I want to use rvv intrinsics to combine/split vectors.
Combining operation is like the vcombine instruction in NEON, to contact two short vectors.
And splitting operation is something like vget_low and vget_high instructions in NEON, to get halves of the vector.
But I didn't find intrinsics to combine/split vectors in the reference manual.
I think maybe I can use shift operation and some arithmetic operations to realize these two functions, but it is too complicated. Is there an easy way to do this? Or should we add such a function to intrinsics?
Sincerely,
Yin

I asked this question at riscv-v-spec and as Krste mentioned, I can combine two vectors as bellow:
vsetvli  t0, 8, e8,m1
vle8.v  v1 src1
vle8.v  v2 src2
vsetvli  t0, 16, e8,m1
vslideup.vi  v1, v2, 64

But now I want to use rvv intrinsics in C++ to implement this function.
And the intrinsic of vslideup instruction is defined as bellow:
vint8m1_t vslideup_vx_i8m1 (vint8m1_t src, size_t offset);
I can not pass the dest vector (v1) to vslideup, so they are not combined. How can I use intrinsics to implement this assembly code? Should the dest vector parameter be added in the vslideup intrinsic?
Sincerely,
Yin

Hi Yin,
We can use mask vslideup with all-ones mask, like
vslideup_vx_i8m1_m (vmset_m_b8() , v1, vint8m1_t v2, 64);

Ideally compiler can do optimization to generate vslideup.vi without mask.
Thanks.

Hi,
There may be some problems with my previous description. In fact, what I want to do is
for example to combine two vint8mf2_t to one vint8m1_t and to split one vint8m1_t to two vint8mf2_t. It looks just like is a register allocation problem and do not need vector instructions. Are there intrinsics working for this?

Hi,
There may be some problems with my previous description. In fact, what I want to do is
for example to combine two vint8mf2_t to one vint8m1_t and to split one vint8m1_t to two vint8mf2_t. It looks just like is a register allocation problem and do not need vector instructions. Are there intrinsics working for this?

Should the intrinsic for vslide{up,down} take the destination as an argument, since its previous contents are partially preserved by the operation?  This is easy to express in assembly, but the intrinsics abstract this.

@joy2myself  Why not to combine two vint8m1_t to one vint8m1_t like Krste's example?

@zakk0610 Hi Zakk. In fact, the input I met is two vint8mf2_t but not vint8m1_t. If I use vslide{up,down} to combine, I should firstly append zero to these two vint8mf2_t to convert them into vint8m1_t. This can also be regarded as a kind of combination (combine vint8mf2_t and a full zero vint8mf2_t).
And also, split is hard to implement with vslide{up,down}, right?
In my opinion, this kind of combination is just a register alloction problem. We just need to put them together and rewrite the vtype register. We do not need to use vector instructions.
I think combination and split are very useful functions to make conversions between multiple short vectors and one long vector. For example two __mf2 and one __m1 or two __m1 and one __m2. And these functions are independent of vector assembly instructions. Should we add these intrinsics? Is there any other way to implement it？

For fractional LMUL, it's not just register allocation issue, because fractional LMUL still occupy one register, if you want to combine two vector values with fractional LMUL, you need to slide up one of the value and then combine with another one.
For LMUL >= 1 case, it's just register allocation issue only if vl == vlmax, consider vlmax = 4 and vl = 3 (for LMUL=1), v0 (m1) = 1, 2, 3, 4 and v1 (m1) = 5, 6, 7, 8, combine two register you will get v0 (m2) = 1, 2, 3, 4, 5, 6, 7, 8, not 1, 2, 3, 5, 6,7.

Hi Kito, thanks a lot for the guidance.
Then how could I implement combination and split? For example from two vint8mf2_t to one vint8m1_t? Do I need to store them first and then use C/C++ for combination? vslide{up,down} works with the same LMUL vector type. And I don't find intrinsics to convert vint8mf2_t to vint8m1_t.

There are discussions converting to different LMUL under the same SEW in #37.

We have lmul_trunc, lmul_ext, vget and vset in the intrinsics to match your use.


Bump Fabric8, use the BOM and dependabot

Cleanup PR, to use the new Fabric8 BOM to get the aligned versions of sundrio and lombok.
I noticed that fabric8 was not getting updated and added an entry to dependabot.

Checks failures unrelated. Merging, thanks!


YNotebook error with y-py v0.6.0

Description
Trying to instantiate and set a YNotebook fails with jupyter_ydoc v0.3.2. Downgrading y-py from v0.6.0 to v0.5.5 fixes the issue.
Reproduce

pip install jupyter_ydoc==0.3.2
Execute the following Python code:

from jupyter_ydoc import YNotebook

nb = {"cells": []}

ynb = YNotebook()
ynb.set(nb)

See error:

Err(PyErr { type: <class 'y_py.MultipleIntegrationError'>, value: MultipleIntegrationError("Cannot integrate a nested Ypy object because is already integrated into a YDoc: {'id': '1d9dbf0a-34ac-46a4-bef9-101fb83d4cea', 'cell_type': 'code', 'execution_count': 1, 'metadata': YMap({}), 'outputs': YArray([]), 'source': YText(1 + 1)}"), traceback: None })

Expected behavior
No error.

I created a minimal reproducible example in https://github.com/y-crdt/ypy/issues/126.

@davidbrochart should we close this following https://github.com/jupyter-server/jupyter_ydoc/pull/147? or keep it open to carry actions when it is fixed upstream?

Let's close it for now.


[Enterprise InfluxDB] Add fine grained authorization documentation

Add documentation for FGA.
Fixed by https://github.com/influxdata/docs.influxdata.com/pull/1173

Done


How to change default cache policy?

I am using Picasso for image loading. Picasso doesn't have a disk cache. OkHttp maintains an HTTP cache that is controlled by HTTP cache headers. I want to set an expiry for disk cached images so added cache-control headers in HTTP response cache-control: public, max-age=7200 but it is not respecting cache headers.

As you can see in the above image we have set max age for 2 hours but still, it is fetching from disk
Is there anything we are missing?

Thanks for your question. This issue tracker is only for bug reports with test cases and feature requests. Please ask usage questions on Stack Overflow.
https://stackoverflow.com/questions/tagged/okhttp


Add 'first' instead of '1st' for referees + change heading size

Context
Fixes some discrepancy between app and design.
Referee headings were coming up as '1st reference', '2nd reference' this PR changes this so they read 'First referee', 'Second referee' etc.
More details in the trello card.
Also changes the <h2> tags to govuk mediums rather than large.
Changes proposed in this pull request
Before:

After

Link to Trello card
https://trello.com/c/tU8HFZco/1732-fix-referee-sizing-and-content
Things to check

[x] This code doesn't rely on migrations in the same Pull Request
[x] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[x] API release notes have been updated if necessary
[x] New environment variables have been added to the Azure config

Thank you 🙏


Summary plot for a multi-input network?

Experiencing "AttributeError: 'list' object has no attribute 'shape'":
summarize the effects of all the features
shap.summary_plot(shap_values, [X11, X22])

Can you please help me with this?

Your features parameter is a list with only two values, whereas shap_values, I assume, has a lot more. Either pass as many values as you have in shap_values, or select the two respective values from sha_values and pass them as arguments.
For more details see doc strings of the method, with more focus on (# samples x # features):
Signature:
shap.summary_plot(
    shap_values,
    features=None,
    feature_names=None,
    max_display=None,
    plot_type=None,
    color=None,
    axis_color='#333333',
    title=None,
    alpha=1,
    show=True,
    sort=True,
    color_bar=True,
    plot_size='auto',
    layered_violin_max_num_bins=20,
    class_names=None,
    class_inds=None,
    color_bar_label='Feature value',
    auto_size_plot=None,
)
Docstring:
Create a SHAP summary plot, colored by feature values when they are provided.

Parameters
----------
shap_values : numpy.array
    For single output explanations this is a matrix of SHAP values (# samples x # features).
    For multi-output explanations this is a list of such matrices of SHAP values.

features : numpy.array or pandas.DataFrame or list
    Matrix of feature values *(# samples x # features)* or a feature_names list as shorthand


Constantly recurring problem

Describe the bug
There always seems to be a problem and you're going around in circles trying to solve it. First comes:
Let's fix the missing getCurrentUser export in the supabase service. Here's the corrected version:
Fix Supabase Service
src/services/supabase.ts
This update:
Adds the missing getCurrentUser function
Properly exports all necessary functions
Includes proper error handling
Maintains type safety with TypeScript interfaces
The application should now work without the "getCurrentUser" error.
and then, after you fix that one, comes:
email rate limit exceeded
Then you do it again:
Let's fix the email rate limit issue by modifying the Supabase authentication settings and updating our registration flow. Here's what we'll do:
Fix Email Authentication
src/services/supabase.ts
This update:
Removes email verification requirement
Checks for existing email before registration
Auto signs in the user after successful registration
Maintains all other functionality
You should now be able to register without email verification issues. The same email can be used for multiple children since we're not requiring email verification.
And then we come back to:
Let's fix the missing getCurrentUser export in the supabase service. Here's the updated version:
Fix Supabase Service
src/services/supabase.ts
This update adds the missing getCurrentUser function and ensures all necessary exports are available. The application should now work without the error.
The whole thing goes on like this and we don't make any progress while my tokens keep getting fewer and fewer.
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-wwkmwe
Steps to reproduce
Navigate to the registration page of the application.
Enter the following details:
Name: Damon Michen-Gruber
Email<EMAIL_ADDRESS>Password: 12345678
Grade: 6th Grade
Click on Register.
Multiple issues occur:
Error 1: "Password is known to be weak and easy to guess, please choose a different one."
Error 2: "Error sending confirmation email."
Error 3: "SyntaxError: The requested module '/src/services/supabase.ts?t=1732064979639' does not provide an export named 'resetPassword'."
Error 4: "over_email_send_rate_limit."
Expected behavior
Users should be able to successfully register.
Email verification should be disabled, allowing children or parents to reuse the same email address for multiple accounts.
Passwords should only require a minimum of 6 characters without additional complexity checks.
Users should be automatically logged in after registration.
Screen Recording / Screenshot

Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 131
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 112774793,
  "usedJSHeapSize": 107401417,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = samsunga31967
Chat ID = 5fa46868c7c9

Additional context
Additional Context
Problem Summary:
Fixes applied to the authentication system repeatedly cause a loop of issues.
Resolving one issue (e.g., missing exports) causes others (e.g., email confirmation errors) to resurface.
Changes like adding resetPassword or getCurrentUser exports do not persist or conflict with other fixes.
Priorities:
Disable email verification to simplify the registration process and support families registering multiple children under the same email.
Simplify password requirements to ensure children can easily create accounts (minimum 6 characters).
Resolve API export issues for functions like resetPassword and getCurrentUser so they work consistently.
Avoid email rate limits by allowing multiple accounts with the same email and either increasing the rate limit or bypassing verification entirely.
Ethical Considerations:
The platform targets children and families.
Children should not face technical barriers during registration.
Parents should be able to register multiple children using a single email address.

I would be very happy about a quick processing, solution and refund of tokens (also because of the many hours I lost) in order to solve the recurring problem - which I was unfortunately unable to solve.
Best regards,
Claudia Michen-Gruber

Hi @samsunga31967,
Sorry to hear about this! At times, there are programming challenges that are too complex for the AI to solve and manual intervention is required. Unfortunately, we don't have the resources to troubleshoot individual project code. You can read more about next steps here: #2310.
Refund requests are handled through<EMAIL_ADDRESS>

The whole thing goes on like this and we don't make any progress while my tokens keep getting fewer and fewer.


Unfolding stops at 0

I have seen a case where the unfolding/deliasing stops at zero velocity.  The velocity is large enough that is wraps around and goes past the 0m/s mark. but the unfolding does not catch it.

isn't this a pyart issue?
for reference the desiasing is done either with:
pyart.correct.dealias_region_based(radar, gatefilter)
pyart.correct.dealias_unwrap_phase(radar, gatefilter)
depending on which plugin you are using
if you are talking about the manual unfold, such a case can not be corrected with that simple of an algorithm, I recommend using the pyart methods before.

No, This is manual unfolding/deliasing so not the automatic dealiasing of pyart.  Thinking about this, I wonder that it might not be possible to cross the 0 line since that would make it harder to distinguish between those points that do not need to be dealiased and those points that do.  (If you are really careful selecting the boundary you can do it.)
Here are two plots.  One before the dealiasing and one after.  The dealiasing happens in the upper right quadrant of the plot.
Before dealiasing

And the after dealiasing

You can see that the discontinuity remains but it was shifted to the 0m/s contour.

I'm not really sure how we can solve this, since as you said the zero line is what is being used to separate what need dealing and what does not. We could allow the user to change this threshold value, but I'm not sure about that. We could also add a component to do a linear transformation in a selected region + gatefilter, that would probably be much more difficult to work with, but gives you full control.
Let me know what you think and we may add as a to do, and it will hopefully be addressed at some time. Also if you want to do it your self I would be glad to give directions.

by the way @pfhein I'm planning to be presenting ARTView in the AMS's 38th Conference, on august. If you have any thoughts I would be glad to hear, send me an e-mail.


Plotting WRF output file example gives "AttributeError"

Description
I tested the example of generating a surface plot from WRF output file on the web page (https://pratiman-91.github.io/2020/07/29/WRF-Surface-plot-using-Python.html).
Steps to reproduce
A "Minimal, Complete and Verifiable Example" will make it much easier for maintainers to help you.
import wrf
from netCDF4 import Dataset
import proplot as plot
import cartopy.crs as crs

# Download the dataset from https://www.unidata.ucar.edu/software/netcdf/examples/wrfout_v2_Lambert.nc
# Load the dataset
ncfile = Dataset('wrfout_v2_Lambert.nc')

# Get the variable
slp = wrf.getvar(ncfile, "slp")

# Get the latitude and longitude points
lats, lons = wrf.latlon_coords(slp)

# Get the cartopy mapping object
cart_proj = wrf.get_cartopy(slp)

#Plotting
fig, axs = plot.subplots(proj=cart_proj)

# Define extents
lat = [lats.min(), lats.max()]
lon = [lons.min(), lons.max()]

#format the plot
axs.format(
    lonlim=lon, latlim=lat,
    labels=True, innerborders=True
)

#Plot using contourf
m = axs.contourf(wrf.to_np(lons), wrf.to_np(lats), wrf.to_np(slp),
                      transform=crs.PlateCarree(), cmap='roma_r')

#Adding colorbar with label
cbar = fig.colorbar(m, loc='b', label='Sea Level Pressure (hPa)')

#Saving the figure
fig.savefig('slp.png')

Expected behavior: [What you expected to happen]
https://pratiman-91.github.io/uploads/2020/07/29/Fig1.png
Actual behavior:
When the script is executed, the following error occurs.
————————————————————————
Traceback (most recent call last):
File "test.py", line 33, in
m = axs.contourf(wrf.to_np(lons), wrf.to_np(lats), wrf.to_np(slp),
File "/opt/anaconda3/lib/python3.8/site-packages/proplot/axes/plot.py", line 1889, in _redirect_or_standardize
return func(self, *args, **kwargs)  # call unbound method
File "/opt/anaconda3/lib/python3.8/site-packages/proplot/axes/plot.py", line 4320, in contourf
kw = self._parse_cmap(x, y, z, contour_plot=True, **kw)
File "/opt/anaconda3/lib/python3.8/site-packages/proplot/internals/warnings.py", line 96, in _deprecate_kwargs
return func_orig(*args, **kwargs)
File "/opt/anaconda3/lib/python3.8/site-packages/proplot/axes/plot.py", line 2924, in _parse_cmap
norm, cmap, kwargs = self._parse_discrete(levels, norm, cmap, **kwargs)
File "/opt/anaconda3/lib/python3.8/site-packages/proplot/axes/plot.py", line 2777, in _parse_discrete
norm = pcolors.DiscreteNorm(
File "/opt/anaconda3/lib/python3.8/site-packages/proplot/internals/warnings.py", line 96, in _deprecate_kwargs
return func_orig(*args, **kwargs)
File "/opt/anaconda3/lib/python3.8/site-packages/proplot/colors.py", line 2417, in init
self.vmin = vmin
File "/opt/anaconda3/lib/python3.8/site-packages/matplotlib/colors.py", line 1148, in vmin
if value != self._vmin:
AttributeError: 'DiscreteNorm' object has no attribute '_vmin'
Proplot version
matplotlib version is 3.5.1
proplot version is 0.9.5

This is duplicated with #302. Please run your example with the newest version (not released yet): pip install git+https://github.com/lukelbd/proplot.git

Note the newest proplot release 0.9.7 enforces a 0.9.5-compatible matplotlib version when you install, and fixes some numpy dependency issues (#309).


SCIM2 Test case to update a provisioned users profile

Purpose
This test case updates a provisioned users profile attribute "first name" and the "last name" and assert if the first name is what we changed
Tested environment
1.ubuntu 18.04
2.java version "1.8.0_161"
3.Identity server wum updated pack wso2is-5.7.0+1543336788716.full with jdbc user store with the UsernameJavaRegEx changes to [a-zA-Z0-9._-|//]{3,30}$
Automation tests
The test attempts to update a provisioned users profile. First a user is provisioned and then the first name attribute is added with a value. Next with the updated users method we update the provisioned users profile firstName

closing this PR as the new PR is https://github.com/wso2/product-is/pull/4154


using watchify with babelify for react transpiling

Hello,
With browserify I am using the command:
browserify -t [ babelify --presets [ react ] ] app.js -o bundle.js
I installed watchify globally, and replaced browserify with watchify, but nothing happens when I run the command. My terminal just hangs. Am I missing something?

watchify doesn't write what it's doing to stdout by default. It's doing its job. If you want to see what it's doing, you can pass in the -v flag.

@substack okay thanks. I'll give it a shot with the flag.


Update style.css for better mobile experience

This edit hides the dropdown menu while being closed on mobile view

Thank you for this!


Update WMR Plugin version to 1.0.21

Overview
1.0.21 updated the remoting plugins to 2.1.3, which fixes some stability bugs that some devs were hitting. A selection of changes between the previous version and this one:

Starting with version 2.1.3 HolographicSpace.CameraAdded is synchronized with pose data coming from the Holographic Remoting Player and users can expect that when a camera is added there is also a valid HolographicCameraPose available for that camera on the the next frame.
Added support for HolographicCameraRenderingParameters.CommitDirect3D11DepthBuffer.

/azp run

lgtm


Introduce autotools

This PR introduces autotools for detecting whether or not some
functions exist.  The benefit of using autotools is that we don't need
to write our own feature detection C files, and that we'll use the same
HAVE_* macros that Ruby uses.  That means we can copy our C files in
to Ruby and piggyback off Ruby's configure scripts when Ruby is built.

Thank you Aaron!

We need to figure out how to get the proper HAVE_ (HAVE_STRNCASECMP, HAVE_STRNSTR, etc) defined for JRuby & TruffleRuby.
That already didn't work before for TruffleRuby since we don't use the Makefile from the repo but let mx+ninja do everything, so it was just using the fallbacks.

This has the side effect to create .o files under src/. That's rather inconvenient to import the project, but I can workaround. Maybe there is an easy fix to avoid mixing sources and build results?

Ah my bad, that's unrelated to this PR as @kddnewton told me.


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

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

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.001%) to 41.578%

Totals

Change from base Build<PHONE_NUMBER>:
0.001%

Covered Lines:
30778

Relevant Lines:
69331

💛  - Coveralls


HDFS-16121. Iterative snapshot diff report can generate duplicate records for creates, deletes and Renames.

Please refer https://issues.apache.org/jira/browse/HDFS-16121 for details

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
12m 10s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
30m 38s

trunk passed

+1 :green_heart:
compile
1m 24s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
compile
1m 18s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
checkstyle
1m  3s

trunk passed

+1 :green_heart:
mvnsite
1m 27s

trunk passed

+1 :green_heart:
javadoc
0m 58s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
1m 28s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
spotbugs
3m 10s

trunk passed

+1 :green_heart:
shadedclient
16m 24s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 14s

the patch passed

+1 :green_heart:
compile
1m 16s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

-1 :x:
javac
1m 16s
/results-compile-javac-hadoop-hdfs-project_hadoop-hdfs-jdkUbuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04.txt
hadoop-hdfs-project_hadoop-hdfs-jdkUbuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 generated 1 new + 471 unchanged - 0 fixed = 472 total (was 471)

+1 :green_heart:
compile
1m 15s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

-1 :x:
javac
1m 15s
/results-compile-javac-hadoop-hdfs-project_hadoop-hdfs-jdkPrivateBuild-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10.txt
hadoop-hdfs-project_hadoop-hdfs-jdkPrivateBuild-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10 with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10 generated 1 new + 455 unchanged - 0 fixed = 456 total (was 455)

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
1m  2s

the patch passed

+1 :green_heart:
mvnsite
1m 29s

the patch passed

+1 :green_heart:
javadoc
0m 55s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
1m 28s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
spotbugs
3m 40s

the patch passed

+1 :green_heart:
shadedclient
18m 19s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
230m  6s

hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 46s

The patch does not generate ASF License warnings.

328m 55s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3188/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/3188

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell

uname
Linux 0dd86ec84eb5 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / a36d747eee8b00cd838232e2d8879ab8c60e4a0b

Default Java
Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3188/1/testReport/

Max. process+thread count
3237 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3188/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 33s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
31m  2s

trunk passed

+1 :green_heart:
compile
1m 25s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
compile
1m 15s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
checkstyle
1m  3s

trunk passed

+1 :green_heart:
mvnsite
1m 25s

trunk passed

+1 :green_heart:
javadoc
0m 56s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
1m 29s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
spotbugs
3m  9s

trunk passed

+1 :green_heart:
shadedclient
16m 14s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 15s

the patch passed

+1 :green_heart:
compile
1m 15s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javac
1m 15s

the patch passed

+1 :green_heart:
compile
1m 11s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
javac
1m 11s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 54s

the patch passed

+1 :green_heart:
mvnsite
1m 14s

the patch passed

+1 :green_heart:
javadoc
0m 47s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
1m 17s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
spotbugs
3m 12s

the patch passed

+1 :green_heart:
shadedclient
16m  1s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
227m  0s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 48s

The patch does not generate ASF License warnings.

311m 19s

Reason
Tests

Failed junit tests
hadoop.hdfs.server.balancer.TestBalancerWithHANameNodes

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3188/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/3188

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell

uname
Linux d4e1962d1e0d 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 0224c6bcbfb9f89858b9bc7ce11c25a4e206c021

Default Java
Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3188/2/testReport/

Max. process+thread count
3016 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3188/2/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.


[PT] Update Mnesia translation

Related to #711

I wasn't sure about what to use for "Match" in "Match and select" section. I ended up using "combinação", but I also thought about using "equivalência" or "correspondência" (as seen in the glossary).
@pragmaticivan, any thoughts here?

@ruan-brandao @capaci, guys, can I ask for your thoughts here too?

@doomspork, are we good to go with this one?

We are good to go @anderkonzen 👍


Error: The incoming JSON object does not contain a client_email field

I get this error when I'm using a OAuth client ID JSON file as GOOGLE_APPLICATION_CREDENTIALS.
Here's the content of this JSON file.
{
    "installed": {
        "client_id": "",
        "project_id": "",
        "auth_uri": "https://accounts.google.com/o/oauth2/auth",
        "token_uri": "https://accounts.google.com/o/oauth2/token",
        "auth_provider_x509_cert_url": "https://www.googleapis.com/oauth2/v1/certs",
        "client_secret": "",
        "redirect_uris": ["urn:ietf:wg:oauth:2.0:oob", "http://localhost"]
    }
}

Here's the code where I get this error.
var GoogleAuth = require("google-auth-library");
new GoogleAuth().getApplicationDefault(function(err, auth) {
  console.log(err);
});

I am pretty sure we require client_email. Did you download this JSON file from one of our systems?

I'm having the same problem and I have a JSON file with the exact same format (no client_email field). I got my JSON file from the Google API developer console to work with the Gmail API. My file is also an OAuth 2.0 Client ID. I tried making a service account credential instead and that file did have a client_email field, but I think I need to use OAuth2 not a service account.

@daywaves You should use the service account JSON for OAuth2 authentication. Can you describe what you are trying to do, and where it is failing?

If you want to use the credentials of the user to perform actions on behalf of the user, you want to use OAuth2.  In that case - you wouldn't use application default credentials, rather a standard OAuth2 workflow.
We have some improved docs to cover this stuff sitting over in the next branch.   I'm going to close this out for now - feel free to keep asking questions!

This doesn't seem accurate. The Ruby gem for authenticating (particularly with Google Cloud Storage) does not require a client_email in order to operate.

So, I haven't dug really deep into the Ruby library, but the functionality seems to be located here: https://github.com/GoogleCloudPlatform/google-cloud-ruby/blob/master/google-cloud-core/lib/google/cloud/credentials.rb#L49
I confirmed locally that loading a bucket and listing its files doesn't require a client_email, while the node client generates an error. It seems the Ruby client allows it to not be present, but also allows it to be passed in discretely.

A similar problem arose. Thank you.

Hello, I hate to message on old threads, but I just encountered this issue as well. Is there a solution somewhere I am not seeing?

Im followin the google calendar API docs in a NodeTS project and getting the same issue. This is my credentials.json. Is there something that i dontt notice in the docs or this JSON?
{   "web":{     "client_id":"client_id.com",     "project_id":"project-homol",     "auth_uri":"https://accounts.google.com/o/oauth2/auth",     "token_uri":"https://oauth2.googleapis.com/token",     "auth_provider_x509_cert_url":"https://www.googleapis.com/oauth2/v1/certs",     "client_secret":"client_secret"   } }

I am facing the same issue, do we have solution to this one ?
I am using Workload identity federation and package is googleauth-1.6.0. The json file is {
"type": "external_account",
"audience": "",
"subject_token_type": "",
"token_url": "",
"credential_source": {
"file": "",
"format": {
"type": "text"
}
},
"service_account_impersonation_url": ""
}


remove tests

Sorry about that - trying to replicate an issue I'm experiencing and selected the wrong repo as target.


Rework pagination : add support for first button, slots

Add support for first button, button slots
Still a work in progress

Hey Jc , Did you want to merge my branch into yours and move in the direction however you choose?
Also I am having issue with running vuepress. All I have to do is run npx vuepress dev in project directory?

I'm not sure the merge will be easy because I did some major rework.
The reason for the rework was that there were a number of type script errors and bugs in the code and the v-model wasn't working.
Also the unref of props and refs when they are used makes for a lot of unnecessary code in my opinion.
I surely want to take your code for the parts that are still missing tough.
To get the docs site to work locally, follow following procedure

install yarn globally if you don't have yarn already. It works with npm as well, but you have to be careful not to create an extra package.json.lock file.
build the library : yarn run build  (or npm run build)
run the docs site : yarn run dev:docs  (or npm run dev:docs)

Yeah my final steps are to get rid of the the redundant refs. I am sure some of the decision trees are unnecessary at this point as well. That would have been my final contribution final to tests but go ahead and rework what you see fit. Ive been looking into to writing a plugin in and moving some existing Docs components over from boostrap vue as to make the reference easier to build

@aceofwings : All functionality of your PR should now be included in this PR : slots, alignments,
I've implemented the rest of the functionality as well : pills, size, aria-control
Still to be done :

complete the documentation
aria-... attributes (Check all are done )

If you've reviewed this PR and made sure I didn't miss anything, you can remove your PR and we can move forward from this one.


Move pino-pretty as a dev dependency

pino-pretty seems to be only useful as part of the benchmarking code, so moving to dev dependencies instead.

Coverage remained the same at 98.976% when pulling b99e37c47f541b806c42ec0549fa41c1521a6c47 on jonathansamines:feature/remove-pino-pretty into 204c96e0a248c899147243b7a22414eb02edf943 on pinojs:master.


Add canonical addresses for CELO, MANTLE and X_LAYER

Adds missing canonical addresses for

CELO
MANTLE
X_LAYER

Pull Request Test Coverage Report for Build<PHONE_NUMBER>3
Details

0 of 0   changed or added relevant lines in 0 files are covered.
618 unchanged lines in 20 files lost coverage.
Overall coverage decreased (-6.5%) to 87.008%

Files with Coverage Reduction
New Missed Lines
%

safe_eth/eth/utils.py
3
86.46%

safe_eth/eth/account_abstraction/bundler_client.py
3
92.5%

safe_eth/eth/tests/utils.py
6
59.52%

safe_eth/safe/tests/test_proxy_factory/test_proxy_factory.py
7
88.61%

safe_eth/eth/oracles/superfluid.py
10
54.55%

safe_eth/eth/multicall.py
10
84.96%

safe_eth/eth/oracles/kyber.py
11
59.18%

safe_eth/eth/tests/clients/test_etherscan_client.py
11
63.64%

safe_eth/eth/tests/oracles/test_kyber.py
11
57.69%

safe_eth/eth/tests/oracles/test_superfluid.py
13
43.48%

Totals

Change from base Build<PHONE_NUMBER>4:
-6.5%

Covered Lines:
8177

Relevant Lines:
9398

💛  - Coveralls

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

recheck

@dasanra LGTM, thanks!


When a video is playing, the tab of the video become purple.

Everything is explained in the title. Here's a picture of what it look.

Yes. That's not an issue but a feature. Here's the relevant code: https://github.com/manilarome/blurredfox/blob/6976b5460f47bd28b4dc53bd093012780e6bfed3/parts/tabbar.css#L200-L203
So yeah, closing this now!


Scenario improvements

Content: minor changes to Yield Curve comparer and importer, Scenario validation for any type of importer

Change value of Scenarios.EnableScenario

https://github.com/Systemorph/IFRS17CalculationEngine/pull/279#issuecomment-1520843635
done


Bug at level 26?

Hi there, amazing work here! But i think i found a bug:

Kind regards,

Alright... looking at your code i realize you want me to do grid-template-rows: 50px 0 0 0 1fr, but I never knew this was possible.
Anyway, i still think my solution may be considered valid. And a little explanation about the zeroes here, would be quite interessting. ;)
As i said before, very nice job here.

You didn't solve the challenge correctly;

4 * 10px = 40px

but the challenge requires

50px

of space above the carrots. I solved it with
grid-template-rows: repeat(4, calc(50px / 4));

which worked just fine.

my mistake! :) you're totally right... thanks for pointing it out!

@fspin Can you explain grid-template-rows: 50px 0 0 0 1fr for me please?

@Dean-Coakley As you can see, this is not the right answer. But I can explain, of course, what I thought I was doing:
The exercise ask "to water all but the top 50 pixels of your garden". But you also see in the code that the water will only fill the fifth row: grid-row: 5 / 6; this translates to grid-row: grid-column-start / grid-column-end > more on MDN.
So, I have to water the fifth row, and i also have to let 50px without water.
In my solution the first row == 50px, the second, third and fourth row are 0px (or just 0) which means the rows exist, but they are 0 pixel height, and lastly, the fifth will take all the available space to fill with water, so == 1fr.
Or, in a more visual way:

Row 1 = 50px
Row 2 = 0
Row 3 = 0
Row 4 = 0
Row 5 = 1fr

But, as you also can see, this is a little hacky. What i wanted to do was what @jgierer12 pointed out. And, I assume you understand what he said.
I hope this clarify my hacky-solution. :)

I did the same as @fspin and wondered why it did not work: after all, it complied with the problem statement (all but top 50px were watered) and it was never explained that zero-tall rows were valid in general or allowed in the challenges.
Ideally the player should be able to figure out the solution without searching in GitHub issues.


MASS test suite is failing on Py 3.5 with a seg fault!

Original report by Joseph Fowler (Bitbucket: joe_fowler, ).

Young, I have no idea why, but the test suite of MASS (branch:develop) has been failing lately (see Semaphore) with a segmentation fault. That happens only on Py 3.5. On 2.7, the tests pass.
Can you reproduce that on your own machine? Can you help figure out what in the world this means?
@oneilg: have you ever seen such a thing?

Original comment by Joseph Fowler (Bitbucket: joe_fowler, ).

Wow. That is strange. Do you think we should just close this issue, or maybe leave it open and see if we can make it resolve just by being patient?
I'll try to re-run just once and see if the problem arises again.

Original comment by Young Il Joe (Bitbucket: jojo978, GitHub: jojo978).

I couldn't reproduce segmentation fault on my home machine. But tests are also not reproducible when I sshed to Semaphore and manually test. Sometime it seg-faulted. Sometimes it gaves different faults (assertAlmostEqual).

Original comment by Joseph Fowler (Bitbucket: joe_fowler, ).

Young, thank you for checking on this. It is very strange to me that we're getting seg-faults on the testing machines, but I suppose it's reassuring that we are not getting them on our own machines.  I'm putting this on hold. We can see what happens.

Original comment by Joseph Fowler (Bitbucket: joe_fowler, ).

Removing milestone: v0.6 (automated comment)


Please provide autoCorrect for TrailingWhitespace

Expected Behavior of the rule
The TrailingWhitespace rule is complaining but cannot fix whitespaces automatically. It would be really great to have it though, cause it's quite boring to fix them manually.
PS: same idea for PsiWhiteSpace but unfortunately this is ktlint.

Ideally you should be able to use the NoTrailingSpaces rule that is a wrapper for the ktlint homonymous rule. That will allow you to enable the autoCorrect.

If you want the autoCorrect feature you should use ktlint's NoTrailingWhitespace rule, which does exactly what you want.
Detekt does not support autoCorrect at the moment. Because that's were ktlint shines, we decided against it in the early RC versions, though that's not set in stone.


Fix errors where OCI hooks directory does not exist

Hopefully fixes https://bugzilla.redhat.com/show_bug.cgi?id=1657263

Tested it locally and it works.
/lgtm

Since (presumably) Dan found the bug in Fedora, might be nice to rebase so the F29 cirrus test can chew on it.

gaa! I was 12-seconds too slow :(


Enable backport of unmerged PRs

This PR enabled the backport of https://github.com/grpc/grpc/pull/25493. It adds two features:

The ability to backport unmerged PRs (including those with multiple commits)
The ability to run a script after cherry-picking the desired commits, including scripts that generate files.

It also fixes a bug where the script previously created PRs on the grpc/grpc repo instead of on the user's fork.

@drfloob Thanks for the review! Even more useful than usual, since there are no tests for this script.


Fix path for barclamp_install.rb

When installing a barclamp not from rpm, but from a file location, a non-
correct path is used resulting in not installed roles etc. This fixes
the path
Note: This will upload all data bags and roles present in the path. Worth fixing this too?

Hmm, sounds like a code smell hardcoding absolute paths in the middle of code like that.  There should be a better way.  Can you clarify why a non-correct path is used in the scenario you describe?  I don't understand the problem yet.

I don't think this is correct: "when not installing from rpm" more or less means "install from a git clone", and that path does exist.
Or am I missing something?

I cloned a git repo and manually copied it in into the /opt/dell/ tree. Then I ran /opt/dell/bin/barclamp_install.rb  /opt/dell/barclamps/neutron. The path I pass is bc_path in the barclamp_mgmt_lib.rb. The script appends the other names to the path and expects to find the roles e.g. in  /opt/dell/barclamps/neutron/chef/roles and the data bags in /opt/dell/barclamps/neutron/chef/data_bags. See lines 431 to 433 in barclamp_mgmt_lib.rb

Is this wrong usage of the script? I don't get it to work when not pointing it to /opt/dell/barclamps/<barclamp_name>
But  I agree that there are prettier ways to fix this.

The script appends the other names to the path and expects to find the roles e.g. in  /opt/dell/barclamps/neutron/chef/roles and the data bags in /opt/dell/barclamps/neutron/chef/data_bags.

That's correct. So why don't you have these files if you cloned a git repo and copied it to /opt/dell/barclamps? Oh, I read you copied it to /opt/dell?

Yes, I copied it to /opt/dell.
Also, this works as long as there are no changes to data bags or roles, which might be why it always worked for me that way.

So yeah, you're supposed to simply copy the whole git clone to /opt/dell/barclamps.

I see. I could have probably discovered that myself too. Anyway, closing this now.


Error in opening browser platform's build package file, on mac

Short description of the problem:
Error in opening browser platform's build package file, on mac
What behavior are you expecting?
unzip the file without error.
Steps to reproduce:

Ionic build browser
double click on file "package.zip"

Ionic Version: 2.x
Browser & Operating System:  Mac OS
Run ionic info from terminal/cmd prompt:
Your system information:

Cordova CLI: 6.0.0
Gulp version:  CLI version 3.9.0
Gulp local:   Local version 3.9.1
Ionic Version: 2.0.0-beta.1
Ionic CLI Version: 2.0.0-beta.17
Ionic App Lib Version: 2.0.0-beta.8
ios-deploy version: 1.8.3
ios-sim version: 5.0.4
OS: Mac OS X El Capitan
Node Version: v4.2.4
Xcode version: Xcode 7.2.1 Build version 7C1002

This issue was moved to driftyco/ionic-platform-issues#42


An error will be thrown during the rendering when set 'Time Remap' property for layer

Describe the bug
I added a audio layer, I hope that the audio can auto-cycle, so I set the 'Time Remap' for the layer's property, however, it doesn't work, it will throw an error during rendering, the part json is following:
       {
            "type": "data",
            "layerName": "BGM",
            "property": "Time Remap",
            "expression": "loopIn(); loopOut();"
        }

it works when I enabled the 'timeRemapEnabled' for layer by changes the output encoded .jsx script, the snippet code is:
(function() {
    nexrender.selectLayersByName(null, 'BGM', function(layer) {
        layer.timeRemapEnabled = true; // it's the point

        var parts = ["Time Remap"];
        var value = { "expression": 'loopIn(); loopOut();' }
        nexrender.changeValueForKeypath(layer, parts, value);

        return true;
    });
})();

I use nexrender1.8.4 version on my Mac pro.
Please fix this issue,  thanks!

You can try creating a custom jsx script asset, that with this exact snippet (without wrapping function) https://github.com/inlife/nexrender#script-items
// your-custom-script.jsx
nexrender.selectLayersByName(null, 'BGM', function(layer) {
    layer.timeRemapEnabled = true; // it's the point

    var parts = ["Time Remap"];
    var value = { "expression": 'loopIn(); loopOut();' }
    nexrender.changeValueForKeypath(layer, parts, value);

    return true;
});

And that script would be embedded directly into the main script.

Wow, your response was just in time!  Thanks, I will have a try


[Clock sync] Only last few dash chunks being played by shaka player

Have you read the FAQ and checked for duplicate open issues?: Yes
What version of Shaka Player are you using?: v2.5.0-beta2
Can you reproduce the issue with our latest release version?: Yes
Can you reproduce the issue with the latest code from master?: Yes
Are you using the demo app or your own custom app?: comparing the result of both
If custom app, can you reproduce the issue using our demo app?: No
What browser and OS are you using?: Chrome, mac OS
What are the manifest and license server URIs?:

What did you do?
Generated dash chunks and live manifest. The duration is 10 hr long and about 3601 chunks
What did you expect to happen?
Based on the current time offset from the time of generation, I expect the shaka player to calculate which chunk to play. So if it is 2 hrs from time of generation, I expect shaka player to play that chunk.
What actually happened?
The shaka player that I installed using instructions here plays only the last 3-4 chunks. I can see in the network tab it fetches 3598, 3599, 3600, 3601.m4a.
In the demo app i.e. https://shaka-player-demo.appspot.com/demo/#build=uncompiled it works fine as expected i.e. the chunks is played based on what the time offset is from the time of generation.
And the same issue repros with dash player as well what happens with the shaka player installed on my mac and being served by node js.
https://shaka-player-demo.appspot.com/demo/#asset=https://storage.googleapis.com/wvtemp/kqyang/514_user/master.mpd;lang=en-US;build=uncompiled
The above URL was tried by the shaka packager team to help me. And it worked. The dash data that I provided for this can be found here: https://drive.google.com/drive/folders/1h8hpazrUNANvJo0ozE8aLSHAK7IT5zMK?usp=sharing
The folder has master.mpd and the chunks to be played.
Can you please help me understand why is this behaviour happing on shaka player installed on my mac. Same happens with dash player but the files seem to be fine as the demo app can play them as expected.

We can't reproduce a live stream issue if you have a static snapshot of the live content.  But I can take a guess at the problem.
If you get correct chunks played in the hosted demo, but not your own application, you may be missing some configuration.  In particular, the clock sync configuration.
Clock sync is critical to live streaming.  You can't trust your users to have the correct time set, and sometimes, even your own clock as a developer is wrong.  Clock sync provides a way to correct the client clock to sync with the server clock.
There are two main ways to accomplish clock sync: in the manifest, or through configuration.
In the manifest, you can use the <UTCTiming> element to indicate to the player how to do clock sync.  See the DASH spec or the configuration documentation of your packager for details on how to set this up.
Without support from the packager to put <UTCTiming> in the manifest, you can set the manifest.dash.clockSyncUri field to any URL hosted on a server with the same clock as your content.  For example, if your packager has an accurate clock, this could be any URL on any server with an accurate clock (and CORS enabled so you can read the Date header).  Here's how our demo app sets this value:
player.configure(
    'manifest.dash.clockSyncUri',
    'https://shaka-player-demo.appspot.com/time.txt');

Does this help?

Clock sync is critical to live streaming. You can't trust your users to have the correct time set, and sometimes, even your own clock as a developer is wrong. Clock sync provides a way to correct the client clock to sync with the server clock.

I have generated the dash files using shaka packager on cloud machine, copied the files to my local machine and have hosted the server on the same local machine. So the manifest needs to sync the client clock with my local machine(acting as the server) or the cloud machine (where the dash files were generated)?

In the manifest, you can use the <UTCTiming> element to indicate to the player how to do clock sync. See the DASH spec or the configuration documentation of your packager for details on how to set this up.

I am using shaka packager. Here is the command I used
'in=/media/test/data/inputAudio.mp4,stream=audio,init_segment=/media/dashWithUTCTiming/init.m4a,segment_template=/media/dashWithUTCTiming/$Number$.m4a' —segment_duration 10 --time_shift_buffer_depth=36000 --mpd_output /media/dashWithUTCTiming/master.mpd --utc_timing  urn:mpeg:dash:utc:http-iso:2014=https://d2pqwv4o8zcpge.cloudfront.net/iso-timestamp.
The value of utc_timing points to the current UTC time always. I thought this will work as the cloud machine I used to generate dash chunks has clock as per UTC
Attaching the generated mpd in txt format.
mpd.txt
After running the above command and using the generated mpd (attached above), I still face the same issue of only the last few chunks being played.

Without support from the packager to put <UTCTiming> in the manifest, you can set the manifest.dash.clockSyncUri field to any URL hosted on a server with the same clock as your content. For example, if your packager has an accurate clock, this could be any URL on any server with an accurate clock (and CORS enabled so you can read the Date header). Here's how our demo app sets this value:
player.configure(
    'manifest.dash.clockSyncUri',
    'https://shaka-player-demo.appspot.com/time.txt');

Haven't tried this yet. In which file do I need to do this?
Few additional questions:

If there is a mis match in the client and server clock, I still expect some chunk to fetched based on the wrong calculation, but in my case the browser is always fetching the last few chunks. Can you please explain this as well? After packaging using utc_timing, the issue still persisted, hence I also updated the utc_timing tag in the mpd with this: UTCTiming schemeIdUri="urn:mpeg:dash:utc:direct:2014" value="2018-12-14T10:13:08Z"/> , where the time is the UTC time at which I tried it. This was about an hour after generating the files and the file is 10 hr long, so if some minutes of mismatch is fine, then shouldn't this have worked?

Just curious why is this flag known as utc_timing? Is it something to do with utc timezone? From the definition it just seems like setting the clock of client exactly as that of server.

You should put that configuration after creating the player, but before loading.
So, as you said you were following the tutorial, you could end up with something like...
function initPlayer() {
  // Create a Player instance.
  var video = document.getElementById('video');
  var player = new shaka.Player(video);

  // Configure to add a clock sync uri.
  player.configure(
      'manifest.dash.clockSyncUri',
      'https://shaka-player-demo.appspot.com/time.txt');

  // Attach player to the window to make it easy to access in the JS console.
  window.player = player;

  // Listen for error events.
  player.addEventListener('error', onErrorEvent);

  // Try to load a manifest.
  // This is an asynchronous process.
  player.load(manifestUri).then(function() {
    // This runs if the asynchronous load is successful.
    console.log('The video has now been loaded!');
  }).catch(onError);  // onError is executed if the asynchronous load fails.
}

@theodab @joeyparrish

In the manifest, you can use the <UTCTiming> element to indicate to the player how to do clock sync. See the DASH spec or the configuration documentation of your packager for details on how to set this up.

I tried this : <UTCTiming schemeIdUri="urn:mpeg:dash:utc:http-head:2014" value="http://<IP_ADDRESS>:8080"/>
I am serving the dash chunks through localhost so this should have worked as it will sync with server time. But this didn't work

Without support from the packager to put <UTCTiming> in the manifest, you can set the manifest.dash.clockSyncUri field to any URL hosted on a server with the same clock as your content. For example, if your packager has an accurate clock, this could be any URL on any server with an accurate clock (and CORS enabled so you can read the Date header). Here's how our demo app sets this value:
player.configure(
    'manifest.dash.clockSyncUri',
    'https://shaka-player-demo.appspot.com/time.txt');

This is what I tried in myapp.js and this also didn't work.
player.configure(     'manifest.dash.clockSyncUri',     'http://<IP_ADDRESS>:8080');
What else could I be missing. What else informartion can I provide you that will enable you to help with the root cause?

We added a feature to ignore the availabilityStartTime and use the segment list in the manifest to determine the live edge.  This allows us to play content that has bad clock sync issues (which is unfortunately common).  This means that if your manifest specifies all 10 hours of content, we'll play at the end, even if the availabilityStartTime and the current time says to only play at 2 hours in.
Really, your packager should only generate content in real time.  I think FFmpeg has an input flag of -r to limit output to real time if you have all the input at once.
We currently don't have a way to disable this behavior, so we should add a flag.  You can verify my guess by using the v2.4.x version locally instead of master.

We added a feature to ignore the availabilityStartTime and use the segment list in the manifest to determine the live edge.

Was this added in v2.5.x ?

This means that if your manifest specifies all 10 hours of content, we'll play at the end, even if the availabilityStartTime and the current time says to only play at 2 hours in.
Really, your packager should only generate content in real time. I think FFmpeg has an input flag of -r to limit output to real time if you have all the input at once.

I am using shakapackager to generate dash output. I am a little confused here. Is it not possible to generate a 10 hr long dash output with a live manifest which should from the availabilityStartTime be able to serve dash chunks till 10 hrs? How else must the player fetch the chunks reading the mpd by not making use of availabilityStartTime ?

We currently don't have a way to disable this behavior, so we should add a flag. You can verify my guess by using the v2.4.x version locally instead of master.

Thanks a lot for pointing this out. v2.4.5 version of shaka player worked for me with and without adding UTCTiming flag.

How can I make this work with the latest version of shaka player?
If the new master version ignores availabilityStartTime, can't  PublishTime be used to calculate the right chunk? How do availabilityStartTime and PublishTime differ in their use?
Is there a plan to fix this issue in future? This feature is critical to us and hence we need a stable option.

Thanks!!

I am using shakapackager to generate dash output. I am a little confused here. Is it not possible to generate a 10 hr long dash output with a live manifest which should from the availabilityStartTime be able to serve dash chunks till 10 hrs? How else must the player fetch the chunks reading the mpd by not making use of availabilityStartTime ?

Let's say you have a manifest that has 3 hours of media in it that are from 2-5 hours into the live stream.  The manifest specifies these segments for download by the client.  Let's also say the availabilityStartTime is 3 hours ago.  This means the "live edge" is at 3 hours, or is playing at 1 hour into the segments in the manifest.  So there are 2 hours of segments left in the manifest.  But with our new feature we ignore it and start playing at 5 hours into the live stream, at the end of the available segments.
|---------------------------------------------------------->
|                      |-------------- Segments -----------|
|---------------------------------------------------------->
                                   | Live edge
0          1           2           3           4        5
                     Time (hours)

Most live content is being generated in real time.  The manifest gets updated every few seconds, adding new segments for us to play.  So we should start playing at the last segment in the first manifest so we have the shortest delay between generation and playback.
But if you are generating a live stream from VOD that you have in advance, then your manifest may have many more segments than should be played.  In this case, we should use the availabilityStartTime to determine where to start playing.

How can I make this work with the latest version of shaka player?

You can't.  I'm working on adding a flag to disable our new live fixing so your content will work (delays due to the holidays).

If the new master version ignores availabilityStartTime, can't PublishTime be used to calculate the right chunk? How do availabilityStartTime and PublishTime differ in their use?

We don't know if we can trust publishTime and some manifests don't have it.  It is supposed to be when the manifest is generated, but that may not be "now".  For example, some live manifests don't need updating, but the "live edge" is still moving forward.
The current "live edge" is equal to now - availabilityStartTime.  But this only works if the client's clock is the same as the server's.  publishTime is just the time the manifest is generated.  For many live manifests this will be the live edge since the manifest is updated every few seconds; but some manifests don't need updating and so it won't be correct.

Is there a plan to fix this issue in future? This feature is critical to us and hence we need a stable option.

Yep, I'll add it soon and it will appear before the v2.5 release.


Upgrade dependencies after 0.12

Another pin in docs
Pins due to Flask-Classful still needed, adding another.

TODO:

[ ]  Timely-beliefs doesn't like the numpy upgrade. I wonder if the [forecast] option is applied during pip-compile.
[ ]  Flask-Security 5 & Flask-Login 0.6 need to be looked at. Updating Flask-Login as first step, it already leads to tests failing due to 401 status in API endpoints.

@Flix6x do you have an idea why we get a numpy version installed by pip-compile, which leads to the following error during python setup.py develop:

error: numpy 1.23.5 is installed but numpy<1.23,>=1.21.0; python_version > "3.7" and extra == "forecast" is required by {'timely-beliefs'}

It seems pip-compile is not installing timely-beliefs with respect to the [forecasting] option.
requirements/app.txt installs timely-beliefs[forecast]==1.18.0.
By the way, timely-beliefs claims that the numpy limitation is due to this: "# required by sktime==0.13.1"
In this PR, we have moved to sktime==0.15.0. I believe that sktime is now requiring "numpy>=1.21.0,<1.25". And also "scipy<2.0.0".
So either we can figure out why pip-compile and python setup.py develop don't agree somehow, or we could simply upgrade timely-beliefs to relax the numpy and scipy requirements.

Does this help? https://github.com/SeitaBV/timely-beliefs/pull/130

I guess, if it is released

What about now? https://pypi.org/project/timely-beliefs/1.19.0/

Thanks, I believe that moves us one step ahead, next errors appearing now :)

I found out that forecasting jobs are silently failing due to the upgrade to rq==1.12 (it works until 1.11.1).
Maybe there is some info in https://github.com/rq/rq/releases or https://github.com/rq/rq/compare/v1.11.1...v1.12.0


Fix pubkey_to_address function: add native ripemd-160

Added python native ripemd-160 algorithm implemented by user @sipa. Fixed pubkey_to_address function that wasn't working anymore. generate_wallets (and probably more) is broken without it
Solution found in this Bitcoin issue.

Is this going to be merged in? The OpenSSL issue is currently blocking us from integrating Cosmos: https://github.com/aleph-im/aleph-sdk-python/pull/20

@hukkin I would like to connect with you on how to push this issue forward

Ah, @waddafunk one more thing I noticed while going through the @sipa implementation of the hashing algorithm: It is not constant time, therefore it must be only used for testing purposes (and not by default):
https://github.com/bitcoin/bitcoin/pull/23716#discussion_r765396525

But then again, not being constant time is not that bad, as it is only being used to calculate the address from the public key.


Refactor Docker engine as a preparation for Podman support

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area driver-kmod

/area driver-ebpf

/area libscap

/area libsinsp

/area tests

/area proposals

What this PR does / why we need it:
The current Docker engine has the Linux and Windows implementations entangled and the class boundaries do not fall at lines useful for cleaning this up.
This PR effectively moves all the Docker code around so that we end up with a clean split into:

low-level platform specific code (talking to the Docker daemon)
platform-agnostic code (issuing API requests and handling responses)
high-level platform specific code (detecting containers in the first place)

The split enables us to cleanly support Podman as a container engine (its API is very similar to the Docker one, although there are differences). With this split, Podman is able to reuse almost all of the Docker code (except detection) and still keep relatively clean code.
Which issue(s) this PR fixes:

Special notes for your reviewer:
This is a giant no-op. The only actual code change this PR introduces is passing the Docker socket within each background lookup request (as Podman may use multiple sockets when starting containers as different users).
~I'll submit the actual Podman support as a separate PR targeting this branch. IMHO the optimal flow would be:~
~- review both PRs~
~- merge the Podman one into this one~
~- merge this PR~
Just realized I cannot submit a PR on top of this one that you'll actually see. I'll submit the actual Podman PR targeting falcosecurity/libs/master separately. To review the Podman-specific changes, it would probably be easiest to go to https://github.com/draios/agent-libs/compare/podman-upstream-prepare...draios:podman-upstream?expand=1
Does this PR introduce a user-facing change?:

NONE

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

ccc0952 Rename docker_async_instruction to docker_lookup_request
62568a0 Fix comparison of docker_lookup_requests
3baa083 Pass container_id as a const ref to docker_lookup_request ctor
6f57fd9 Move docker_lookup_request to a separate file
0a03a92 Pass docker socket to the async engine separately for each request
b0f2f04 Move platform-specific Docker-API code to a separate class
3d4cf05 Remove docker_connection::build_request
b9bc493 Move docker_async_source to its own source file
f5bef37 Pass a complete request to parse_docker_async
db549eb Move most of docker::resolve to a new method
ec4700b Move common docker bits to a base class
57d87fc Split Linux/Windows docker implementations
585c886 Split docker_async_source::parse_docker into helper methods

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: gnosek
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [gnosek]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@gnosek: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Rerun command

build-libs-with-chisels
8656df4cb098d5f91cfd40fedac0b273b2488dd6
link
/test build-libs-with-chisels

Full PR test history. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@gnosek: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Rerun command

build-libs-with-chisels
8656df4cb098d5f91cfd40fedac0b273b2488dd6
link
/test build-libs-with-chisels

build-libs
8656df4cb098d5f91cfd40fedac0b273b2488dd6
link
/test build-libs

Full PR test history. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@gnosek: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Rerun command

build-libs-with-chisels
8656df4cb098d5f91cfd40fedac0b273b2488dd6
link
/test build-libs-with-chisels

build-libs
8656df4cb098d5f91cfd40fedac0b273b2488dd6
link
/test build-libs

build-libs-bundled-deps
8656df4cb098d5f91cfd40fedac0b273b2488dd6
link
/test build-libs-bundled-deps

Full PR test history. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

LGTM label has been added.  Git tree hash: cbde54936c89b7970662893df8fe69067c91bd9d

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: gnosek, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [gnosek,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


2 Apollo on the same Simualtion

I'm running 2 autonomous vehicles on the same simulation to test a cooperative autonomous driving scenario. But I can't understand if those vehicles can see each other or not and stop if one vehicle is in front of the other

Self-driving cars obtain surrounding environment information through perception modules. That is to say, the perception module will detect vehicles and pedestrians and provide feedback to the self-driving car to avoid them.
Back to your question, you can parse the autonomous car's perception messages(topic: /apollo/perception/obstacles) to see if another car is detected.
go further
I think you may need to identify it automatically. You can use the simulator to obtain the position of the self-driving vehicle, and then compare it with the perceived results. For example, use IOU to determine whether it is the same car.

https://github.com/Software-Aurora-Lab/DoppelTest This project allows multiple Apollo instances to be simulated in SimControl. The underlying mechanism passes one Apollo's localization as perception to other instances, so all the instances can see each other.

https://github.com/Software-Aurora-Lab/DoppelTest This project allows multiple Apollo instances to be simulated in SimControl. The underlying mechanism passes one Apollo's localization as perception to other instances, so all the instances can see each other.

So are there no other NPCs? All vehicles are Apollo self-driving cars?

https://github.com/Software-Aurora-Lab/DoppelTest This project allows multiple Apollo instances to be simulated in SimControl. The underlying mechanism passes one Apollo's localization as perception to other instances, so all the instances can see each other.

So are there no other NPCs? All vehicles are Apollo self-driving cars?

Not exactly. In this implementation, all cars are Apollos, pedestrian walk around crosswalks (didn’t have a better way to simulate realistic pedestrian, no bicycle, with valid traffic lights (conflicting ones won’t be green at the same time).

What I want to express is that there will be conflicts between groundtruth and perception modules. It is best for them to choose 1 from 2.

I see. Yes, in the project linked above, perception module is replaced by ground truth perception messages generated by the project.


Add Jupyter Releaser for NPM and PyPI binary releases

The Jupyter ecosystem has a workflow for publishing to both PyPI and NPM at the same time. We'd like to make an alpha release and it is a natural time to do this.
That said, this isn't critical for an alpha release, and is more like "nice to have", so:

Time box this to 1 hour. If it will take more than 1 hour, just make a release and use this issue to track a long-term goal to use Jupyter publishing infrastructure.

Closed by #2257 and subsequent PRs

@agoose77 I did share these with you, let me know if that worked?


Add Dockerfile for haproxy + lua-acme

Add a simple Dockerfile with haproxy + haproxy-lua-acme

Thanks for the pull request, I'll test it shortly. Meanwhile, we've re-licensed the code to Apache v2.0 (from GLP v2.0), id you don't mind, we'd pull this in under Apache as well.


portico: Use 1 commit instead of 1 commits on the Team page.

Currently, 1 commits is shown for users with one commit to a project on the Team page. This PR makes it so that 1 commit is displayed instead of the incorrect 1 commits.
GIFs or Screenshots:
Before (1 commits):

After (fixed 1 commit):

I can't think of other places in the UI where we display numbers along with units, where the numbers might be singular or plural. If there are, probably someone will notice and report.  Thanks for fixing @skunkmb!


fix(blob/service): fix handling of the padding shares

Fixes the issue with the missing blob. Initially, there was an incorrect assumption that only one padding share is possible between two blobs. So, the Blob service was skipping only one share instead of multiple.

Fixed parsing logic allowing to skip multiple padding shares;
Added a test case that retrieves the EDS and finds the correct number of blobs;

Codecov Report
Attention: Patch coverage is 81.81818% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 44.72%. Comparing base (2469e7a) to head (8b3a48d).
Report is 72 commits behind head on main.

Files
Patch %
Lines

blob/parser.go
81.81%
1 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3404      +/-   ##
==========================================
- Coverage   44.83%   44.72%   -0.12%
==========================================
  Files         265      272       +7
  Lines       14620    15281     +661
==========================================
+ Hits         6555     6834     +279
- Misses       7313     7662     +349
- Partials      752      785      +33

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

@renaynay, I propose to make a new patch for this fix, instead of 0.14.0 as its blocking Astria and in general a critical bug.


single threaded applications fail when asserts are enabled

When asserts are enabled, the single thread hello world is stuck and does not print anything.

close this and track in #8361


@Tool class not invoked in 0.9.1

Hi I have the following example https://github.com/lordofthejars/quarkus-langchain-examples/tree/main/email-a-poem
It's the email in the poem example, but I am using DevServices to send the email.
If I use version 0.5.0, everything works as expected, the poem is generated, and the email tool is invoked. But with just a "simple" change updating to 0.9.1, now the email tool is not invoked even though ChatGPT is not returning that it cannot send an email.
I have not debugged it, so I don't know if the problem is in Quarkus, in Langchain4J, or in the extension.
It seems like @Tool was not registered. The good news is that it is easily reproducible; take my example and update the version to 0.9.1.
(I also tried the email the poem example from the SNAPSHOT current version, and I couldn't see the log lines of sending the email so I assumed it is a bug on the version).

I could not reproduce the problem with https://github.com/quarkiverse/quarkus-langchain4j/tree/0.9.1/samples/email-a-poem

No idea :)

Is this still a problem with 0.10.1?

I am going to try

@geoand I am able to reproduce the issue @lordofthejars reported, even in the latest 0.13.1 version.
I used the sample from https://github.com/quarkiverse/quarkus-langchain4j/tree/main/samples/email-a-poem
with quarkus.langchain4j.openai.api-key=demo

I tried it again and I still could not reproduce the problem

After talking to @kdubois we figured out that the problem is the use of the  demo key. When this is used, a fake OpenAI REST API is hit which apparently does not work with tools.

FYI looks like there's an open issue in the LangChain4j project for this : https://github.com/langchain4j/langchain4j/issues/927


Feature Request: Erase()/Destroy() function to remove everything writer has written to console

hi team,
This feature can be helpful if someone wants to remove/clean the output from screen.
We already have Stop() function but that does not clean the output.
FYI, we are using this library in our open source product YugabyteDB Voyager
Having this above functionality can be really helpful for us.
Thanks

Since we are have Flush() which everytime refreshes the output. So i assumed the code for this requested kind of functionality might be already there internally in the code


Unable to use use transport methods

On 1.8.9-3.1.0-BETA-227 I am unable to use saddles,boats, or minecarts. One entered i am kicked out once movement starts.
Tested it with (EssentialCmds-7.2g worldedit-spongevanilla-mc1.8.9-6.1.1-dist) & and without mods and it is same result.

Same issue as https://github.com/SpongePowered/SpongeCommon/issues/468

Closing because duplicate.


Disable 79 tests in run_py3_core.sh in order to establish a CI baseline

Disabling unit tests in rocm/run_py3_core.sh to establish a CI baseline.
out of 459 tests: 459 tests pass

@whchung - with TF1.8 all checks have passed!
Executed 458 out of 458 tests: 458 tests pass.
Finished: SUCCESS

If you merge this PR, we can kick off the test cycle for existing TF1.8 PRs.
(almost finished with the same for TF1.3)

A check mark! Great!


generate k8s gw apis

reference: https://github.com/solo-io/gloo-mesh-enterprise/issues/18646

closing in favor of #49


Retain complete event stream

We should not retain the event stream, since it the number of events are going to be huge and it will make replication slow(not the problem we want to solve right now).


Command for connection string is incorrect

The command for setting the connetion string in Windows should be
set AZURE_COMMUNICATION_CONNECTION_STRING "<yourConnectionString>"
not
setx AZURE_COMMUNICATION_STRING "<yourConnectionString>"

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

ID: 176e3f9d-0f8e-d58e-b425-01c233fcd3fc
Version Independent ID: 694c721b-22fe-41b3-dd67-e6ca8f044441
Content: Quickstart - How to send an email using Azure Communication Service - An Azure Communication Services Quickstart
Content Source: articles/communication-services/quickstarts/email/send-email.md
Service: azure-communication-services
GitHub Login: @bashan-git
Microsoft Alias: bashan

@bjorn-jarisch
Thanks for your feedback! We will review and update this doc accordingly.

@bjorn-jarisch thanks again for the feedback! I have submitted a PR to update the doc, the change is merged internally and will go-live in the public docs shortly.


4.x: Ability to Inject MockBeans in Helidon #7694

Description
https://github.com/helidon-io/helidon/issues/7694
Documentation
None

Setting it as a draft for now. I have a couple of CDI questions:
Question 1:
Currently @MockTest requires also @Inject:
    @Inject
    @MockBean
    private Service service;

Is there any way to not require @Inject?.
Question 2:
It does not work if the bean already exists in CDI. For example, this will not work because the service will be defined twice:
@HelidonTest
@AddBean(Service.class)
public class TestMockBean {

    @Inject
    @MockBean
    private Service service;

I didn't find how to replace the existing bean defined by the mocked one.

I didn't find how to replace the existing bean defined by the mocked one.
I'm not a CDI expert, but can you set the mocked bean as an alternative ? That's how the current solution (#8339) works.

I didn't find how to replace the existing bean defined by the mocked one.

I'm not a CDI expert, but can you set the mocked bean as an alternative ? That's how the current solution (#8339) works.

I think that requires a producer, and I cannot use a producer because you need to define the type. I will try it further.

For TestNG it seems we cannot use the @Inject in constructor, so the @MockBean applies only in fields.
For example, this simple test:
@HelidonTest
class TestMockBeanField {

    private final WebTarget target;

    @Inject
    public TestMockBeanField(WebTarget target) {
        this.target = target;
    }

    @Test
    void test() {
    }
}

It fails with:
org.testng.TestNGException:
An error occurred while instantiating class io.helidon.microprofile.tests.testing.testng.TestMockBeanField. Check to make sure it can be instantiated/accessed.
	at org.testng.internal.objects.SimpleObjectDispenser.createInstance(SimpleObjectDispenser.java:113)
	at org.testng.internal.objects.SimpleObjectDispenser.dispense(SimpleObjectDispenser.java:40)
	at org.testng.internal.objects.GuiceBasedObjectDispenser.dispense(GuiceBasedObjectDispenser.java:28)
	at org.testng.internal.ClassImpl.getDefaultInstance(ClassImpl.java:106)
	at org.testng.internal.ClassImpl.getInstances(ClassImpl.java:136)
	at org.testng.TestClass.getInstances(TestClass.java:129)
	at org.testng.TestClass.initTestClassesAndInstances(TestClass.java:109)
	at org.testng.TestClass.init(TestClass.java:101)
	at org.testng.TestClass.<init>(TestClass.java:66)
	at org.testng.TestRunner.initMethods(TestRunner.java:483)
	at org.testng.TestRunner.init(TestRunner.java:356)
	at org.testng.TestRunner.init(TestRunner.java:309)
	at org.testng.TestRunner.<init>(TestRunner.java:184)
	at org.testng.SuiteRunner$DefaultTestRunnerFactory.newTestRunner(SuiteRunner.java:652)
	at org.testng.SuiteRunner.init(SuiteRunner.java:224)
	at org.testng.SuiteRunner.<init>(SuiteRunner.java:116)
	at org.testng.TestNG.createSuiteRunner(TestNG.java:1375)
	at org.testng.TestNG.createSuiteRunners(TestNG.java:1349)
	at org.testng.TestNG.runSuitesLocally(TestNG.java:1191)
	at org.testng.TestNG.runSuites(TestNG.java:1114)
	at org.testng.TestNG.run(TestNG.java:1082)
	at org.apache.maven.surefire.testng.TestNGExecutor.run(TestNGExecutor.java:155)
	at org.apache.maven.surefire.testng.TestNGDirectoryTestSuite.executeSingleClass(TestNGDirectoryTestSuite.java:102)
	at org.apache.maven.surefire.testng.TestNGDirectoryTestSuite.execute(TestNGDirectoryTestSuite.java:91)
	at org.apache.maven.surefire.testng.TestNGProvider.invoke(TestNGProvider.java:137)
	at org.apache.maven.surefire.booter.ForkedBooter.runSuitesInProcess(ForkedBooter.java:385)
	at org.apache.maven.surefire.booter.ForkedBooter.execute(ForkedBooter.java:162)
	at org.apache.maven.surefire.booter.ForkedBooter.run(ForkedBooter.java:507)
	at org.apache.maven.surefire.booter.ForkedBooter.main(ForkedBooter.java:495)

Looks good, please update the PR description to fill-in the documentation section:
* describe the feature (what it solves)

* describe the API (annotation + new GAV)

* add sample snippet

Let's also file an issue to create the actual docs for this feature.

Do we re-use this issue?: https://github.com/helidon-io/helidon/issues/8339

Do we re-use this issue?: #8339

Let's file a separate issue as #8339 has more to it than strictly document this new feature.
If we want to proceed with #8338 we have to figure out where it fits in the docs.

I didn't work in any documentation issue before, is there any guide?

No, however  adding one more document is relatively easy as there is a lot of precedence.

The bullet items I provided was guidance not a template. It should read like a short documentation of the feature not like a form !

This is the new issue:
https://github.com/helidon-io/helidon/issues/8772
Are there more changes required in this PR or can it be approved/merged. I guess the documentation changes will be submitted in the new issue I created.
I still don't have any idea where/how should I document this. I will check it with @tvallin because he is in the same time zone than me.


Fr translation

Took time, still have more trouble that I though with forks and pull requests xD

I'm grateful for your contribution—it will be in the next release :)


Trying to Install on Hubitat

I have 2 mss620 smart plugs, one is Hardware/Firmware 3.x and the other is 6.x. I have the the 3.x working great with the info from the install page but I'm struggling with where to put the userkey, etc for v6. Thanks for your help!

Hi @mattries sorry for lack of response - please re-open if you still need some assistance

Nope! I don't have it working yet, still not sure where to put the userkey and all that for my v6 plug.


Code in commented lines #2

What is the issue?
There are commented lines of code which does not give a detailed description and are not relevant to the code.
How is this issue relevant?
In general, comments are used to increase readability and understandability but in this case, there are lines of code that have been commented which decreases the readability of the code which is exactly the opposite of what comments should be used for.
How can we resolve this issue?
To resolve this issue, we have removed the unused code and if necessary it can be retrieved from the control history

Again, only half the comment is removed. The code comment is gone, but the explanation and a part of the commented code is left there.


SVG export button

applies CSS styles inline
exports SVG chart node
injects A tag to download
based on SVG-crowbar

Fixes #555

Current export function modifies the SVG in the main DOM which leads to sometimes disappearing borders of nodes. Would be good to copy the SVG subtree and inject the inline styles to the copy.

LGTM (based on the demo & as soon as the tests pass).

Nice, LGTM


Please reopen with steps if this still exists.


Error "Invalid expression term 'int'" in "ServiceConfig.cs	39

When using Visual studio 2015 it will error during compile for this line.
 if (Int32.TryParse(ConfigurationManager.AppSettings["API_AUTH_MODE"], out int mode))
You can break it into two lines
            int mode = 0;
            if (Int32.TryParse(ConfigurationManager.AppSettings["API_AUTH_MODE"], out mode))

You can close this issue. Just wanted to write it, incase someone else ran into it.

@munib00 looks like you included this change in your PR thanks for that


does it support multi-gpu or multicores cpu.

This depends on what you mean by multi-GPU. You can use multiple GPU executors in parallel by specifying different device IDs, but true multi-GPU as well as multi-node support for distributed matrices is currently still under development.
The OpenMP executor allows you to fully utilize multicore CPUs, but it is not highly tuned for performance yet.

Thanks. I have noticed that only the iterative solver is provided in the solver factory. I want to know that does it supports a dense direct solver.

Ginkgo is primarily a sparse numerical linear algebra library, so we do not support dense direct solvers right now. That may change in the future, but for now, the answer is unfortunately no.