Utilize array length index for = <array>

Summary of the changes / Why this improves CrateDB
For arrays from tables created on or after 5.9 can utilize _array_length_ index. For = [], _array_length_ = 0 should be enough, but for non empty arrays(i.e. = [1]), terms query and generic function query are still needed.
Checklist

[x] Added an entry in the latest docs/appendices/release-notes/<x.y.0>.rst for user facing changes
[x] Updated documentation & sql_features table for user facing changes
[x] Touched code is covered by tests
[x] CLA is signed
[x] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in the latest docs/appendices/release-notes/<x.y.0>.rst
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

Thanks for pointing out, I assumed that utilizing the array length index was always better for performance. I ran the benchmarks are the results are as follows:
# Results (server side duration in ms)
V1: 5.9.0-58b054dc47159ca389384d1a20ada3bc94efc8ee
V2: 5.9.0-4124f4da5428009196ff09257dbb3c0784394ad7

Q: select count(*) from t_arrays_biased_empty where xs = []
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |       85.133 ±  363.339 |     44.963 |     46.384 |     47.271 |   5019.434 |
|   V2    |        8.697 ±   71.904 |      2.484 |      3.367 |      3.843 |   1020.350 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               - 162.92%                           - 172.93%
There is a 99.63% probability that the observed difference is not random, and the best estimate of that difference is 162.92%
The test has statistical significance

Q: select count(*) from t_arrays_biased_5_elements where xs = []
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |       34.612 ±  233.043 |     13.774 |     14.379 |     14.766 |   3293.747 |
|   V2    |        2.552 ±    2.528 |      1.776 |      2.272 |      2.466 |     37.490 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               - 172.53%                           - 145.42%
There is a 94.76% probability that the observed difference is not random, and the best estimate of that difference is 172.53%
The test has no statistical significance

Q: select count(*) from t_arrays_biased_empty where xs = [1,1]
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |        1.406 ±    1.344 |      0.959 |      1.256 |      1.401 |     19.689 |
|   V2    |        2.991 ±    1.633 |      2.084 |      2.787 |      3.034 |     24.816 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               +  72.08%                           +  75.77%
There is a 100.00% probability that the observed difference is not random, and the best estimate of that difference is 72.08%
The test has statistical significance

Q: select count(*) from t_arrays_biased_5_elements where xs = [1,1]
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |        1.081 ±    0.219 |      0.793 |      1.029 |      1.176 |      2.113 |
|   V2    |        2.299 ±    0.269 |      1.821 |      2.259 |      2.395 |      3.706 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               +  72.03%                           +  74.85%
There is a 100.00% probability that the observed difference is not random, and the best estimate of that difference is 72.03%
The test has statistical significance

Q: select count(*) from t_arrays_biased_empty where xs = [1,1,1,1,1]
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |        1.120 ±    0.280 |      0.783 |      1.062 |      1.194 |      3.393 |
|   V2    |        2.966 ±    1.381 |      2.327 |      2.813 |      2.947 |     20.987 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               +  90.38%                           +  90.36%
There is a 100.00% probability that the observed difference is not random, and the best estimate of that difference is 90.38%
The test has statistical significance

Q: select count(*) from t_arrays_biased_5_elements where xs = [1,1,1,1,1]
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |        0.901 ±    0.191 |      0.664 |      0.842 |      0.948 |      1.676 |
|   V2    |        2.546 ±    0.375 |      2.119 |      2.492 |      2.656 |      5.966 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               +  95.45%                           +  99.01%
There is a 100.00% probability that the observed difference is not random, and the best estimate of that difference is 95.45%
The test has statistical significance

Notice that select count(*) from t_arrays_biased_5_elements where xs = [1,1], the array length index must pre-filter all rows since non of them should be length=2 but the performance actually degraded. I have done a few followup experiments but could not observe any improvements.
I will keep the changes for =[] but revert non empty array case.

retest this please
unrelated failure, https://jenkins.crate.io/job/CrateDB/job/crate_on_pr/16423/testReport/junit/io.crate.integrationtests/TransportSQLActionTest/test_types_with_storage_can_be_inserted_and_queried/

Thanks for pointing out, I assumed that utilizing the array length index was always better for performance. I ran the benchmarks and the results are as follows

You could also try replacing the  NumTermsPerDocQuery with the array-length query instead of adding both for the pre-filtering

You could also try replacing the  NumTermsPerDocQuery with the array-length query instead of adding both for the pre-filtering

NumTermsPerDocQuery is only used for arrays before 5.9 where array length indexes are not available, could you double check?

You could also try replacing the  NumTermsPerDocQuery with the array-length query instead of adding both for the pre-filtering

NumTermsPerDocQuery is only used for arrays before 5.9 when array length indexes are not available, could you please double check?


Update to documentation for June 2024 the 5th release

Overview/Summary
Replace this with a brief description of what this Pull Request fixes, changes, etc.
This PR fixes/adds/changes/removes

Updated deployment method pages with the latest changes on the param file
Added the documentation on how to update from release 2024-04-12
Added documentation about the BYO UAMI feature
Added Hybrid VM alerts to the policy initiatives page
Update remediation page

Breaking Changes

NONE

As part of this Pull Request I have

[X] Read the Contribution Guide and ensured this PR is compliant with the guide
[X] Checked for duplicate Pull Requests
[X] Associated it with relevant GitHub Issues or ADO Work Items (Internal Only)
[X] Ensured my code/branch is up-to-date with the latest changes in the main branch
[ ] Ensured PR tests are passing
[X] Updated relevant and associated documentation (e.g. Contribution Guide, Docs etc.)

@Brunoga-MS  Looking at your PR and the table for the Hybrid Initiative and my only complaint is that the name doesn't follow the same format as the other tables. The policy name should use underscores between the words. as shown in the image below.
This should also be fixed in the Notifications Inititaive. ;)
cc: @arjenhuitema

@Brunoga-MS Ignore the Notifications Initiative policy update in our documentation, I forgot that I was going to fix that! :P

@bzabber we are now showing the policy display name. it does not contain underscores.


Creating controller model file(s) during installation

Pretty self-explanotry tiny hack that allows controller side models be generated during install. This is to help folks onboard new hardware more easily.

Hey @eriknordmark -- please take a look -- would be nice to at least kick off the tests


Tooling overhaul

I've commented in the code lines.
The commits are pretty clean, so you could also read the PR commit-by-commit, if you like.

More regarding the use of es2015 modules.
I've found out that while the module definition is released, the module loader definition is not yet finalized.
Might there be changes to the module loader in a way that will break whatever rollup is doing, like tree shaking?

In the #Node.js IRC I was advised by two individuals to not use es2015 modules before the module loader is final.

@eventualbuddha please review.

Apologies for letting this languish. I don't have a lot of time for this project but decided to do a bit of spring cleaning today. I've switched from gobble+esperanto to rollup for most of my projects and am doing the same for this one in #34.


Compare: Could you add Accept header: application/vnd.geo+json

If you use the Accept header with the media type application/vnd.geo+json geojson would be served by default and the format=geojson would be not necessary.
Falk

Done


Fix Footer Crown Copyright link

The old link returns a 404.
Old: https://www.nationalarchives.gov.uk/information-management/re-using-public-sector-information/copyright-and-re-use/crown-copyright/
New: https://www.nationalarchives.gov.uk/information-management/re-using-public-sector-information/uk-government-licensing-framework/crown-copyright/
It looks like this has been correct in govuk_template in Static (https://github.com/alphagov/static/blob/main/app/views/layouts/govuk_template.html.erb#L90) for a few years but isn't correct in govuk_publishing_components.

Closing as Kelv's put a dupe up with the changelog.


receive value is huge(impossible) in har result sometimes

I notice sometimes when I try to generate the har with browsermob-proxy + phantomjs. Some of the receive values in the result are huge. See example,
{
"pageref": "http://yahoo.com",
"startedDateTime": "2015-10-12T15:38:44.374Z",
"request": {
"method": "GET",
"url": "https://s.yimg.com:443/nn/fp/rsz/101115/images/smush/bokoharam_120x48_1444589770.jpeg.jpg",
"httpVersion": "HTTP/1.1",
"cookies": [],
"headers": [],
"queryString": [],
"headersSize": 338,
"bodySize": 0,
"comment": ""
},
"response": {
"status": 200,
"statusText": "OK",
"httpVersion": "HTTP/1.1",
"cookies": [],
"headers": [],
"content": {"size": 0, "mimeType": "", "comment": ""},
"redirectURL": "",
"headersSize": 643,
"bodySize": 4422,
"comment": ""
},
"cache": {},
"timings": {
"comment": "",
"send": 0,
"wait": 67,
"receive":<PHONE_NUMBER>,
"ssl": -1,
"blocked": -1,
"dns": -1,
"connect": -1
},
"serverIPAddress": "<IP_ADDRESS>",
"comment": "",
"time":<PHONE_NUMBER>
}
It is<PHONE_NUMBER>ms, about 20 days!! I am not sure where the bug is from, it could be phantomjs or browsermob proxy? Have you seen this before?

How does browsermob proxy get the receive value? calculated or gets from phantoms?

@chemicwepn: I suggest using beta-3 or later, in which this issue is fixed. I'll close, but please reopen if you continue to have issues.


Building object followed by error

Looks like the full value of $(CC) becomes 'C:\Program' maybe because of space:
 [pre-make.r3] DONE
 [SISKIN] Environment: ${CL} is {C:\Program Files (x86)\Microsoft Visual Studio\2019\BuildTools\VC\Tools\MSVC\14.27.29110\bin\HostX86\x86\cl.exe}
 [SISKIN] Preparing objects dir...
 [SISKIN] Preparing output file name...
 [SISKIN] Environment: ${_} is "make\tmp\cl-x86\host-core-windows-x86"
 [SISKIN] Environment: ${CC} is {C:\Program Files (x86)\Microsoft Visual Studio\2019\BuildTools\VC\Tools\MSVC\14.27.29110\bin\HostX86\x86\cl.exe}
 [SISKIN] Environment: ${CCPP} is {C:\Program Files (x86)\Microsoft Visual Studio\2019\BuildTools\VC\Tools\MSVC\14.27.29110\bin\HostX86\x86\cl.exe}

Compiling 11 files:
 [   9% ] Building object: $_\C\Users\utilisateur\Rebol3\src\os\host-main.c.o
'C:\Program' n'est pas reconnu en tant que commande interne
ou externe, un programme exécutable ou un fichier de commandes.

Good point... I'm personally avoiding installation into these Windows' system folders and use mapped drives (to have shorter paths), so I was not hit by it yet.
The question is, if it should not be fixed in Rebol directly? Like instead of:
>> to-local-file %"/c/a b"
== "c:\a b"

We could have {"c:\a b"}?

It should be fixed in the next release.


chore: rsync not working under windows

Feature Description
There is a prebuild step in the build command which generates the documentation. Unfortunately this command does not work under windows, because rsync does not exist there.
Therefore, this should be adapted so that developers who use a Windows PC can also execute the command
Use Case
No response

Tip: Don't use windows.

Unfortunately, we can't influence that and in order to make Anglify accessible and welcome for everyone, we have to adapt that. This will also allow a lot more people to collaborate on Anglify

Joking, joking.
I have great experience with using https://github.com/sindresorhus/cpy-cli for this.


Minor refactor

These changes are mainly just stylistic but seem more readable to me.
URL validator is changing to match https://github.com/linuxfoundation/cii-best-practices-badge/blob/master/app/models/project.rb#L95

Ummm.. let's talk about this.
I'm a little nervous about monkey-patching, especially something so fundamental as the string type.  Should I be?

Monkey patching is just a very common practice in the Rails/Ruby world. See http://www.justinweiss.com/articles/3-ways-to-monkey-patch-without-making-a-mess/
But in this case, I'm using Refinements, which means that the monkey patches only apply to code files where they are explicitly included. So, I don't really see any possibility for side effects.
Incidentally (and unfortunately), I have found a way to use Refinements in models, views, and helpers, but not in controllers. See http://stackoverflow.com/a/37061045/1935918
So, I was unfortunately prevented from refactoring https://github.com/linuxfoundation/cii-best-practices-badge/blob/master/app/views/projects/_status_chooser.html.erb#L3-L4.
However, I think the passing? method is perhaps the core business logic of the whole app, and I find the reasoning behind it to be clearer with this refactor. Since it's a ultimately a style issue, I'd appreciate if you would give me this one.
Separately, I just added tests.

Yes, I know it's common in the Ruby/Rails world.    I generally avoid it; it's easier to reason about a class when all of its definitions are in one place.  There's also the risk of namespace separation. because as things scale, there's the risk of multiple subsystems monkey patching the "same" method to different values, causing total chaos. Refinements definitely reduce the risk of the latter case, though.
So, okay, signed off.  It's not how I would do it, but does put the "check for met" in one place (in case we change that definition).


Getting polygon annotations into CSV and through pipelines

Being able to support passing polygon annotations into the pipelines by placing them inside of the CSV.
The input should be similar to the way attributes are added into the CSV.

We now have updated VIAME Documentation that specify the format for the Polygon annotations in the CSV.  As of right now they only support a single Polygon.
https://viame.readthedocs.io/en/latest/section_links/detection_file_conversions.html
We also have a new GeoJSON format for polygons and other GeoJSON like data.  I think combined in this issue would be the support for the new Head/Tails GeoJSON data format as well as the Polygon geoJSON format.

@BryonLewis why did #305 not close this?

I don't know, but I'm guessing we can close this.  It looks like I removed the link for some reason.


needs_login functions need @wraps

This has been added: https://github.com/3ptscience/steno3dpy/blob/v0.3.0/steno3d/client.py#L126


how to find the relative or absolute position of a control

Hello,
I am using pywinauto to get the controls on a window , how to get the absolute position (coordinates of a particular button relative to the L,T,B,R position of the window) ..thanks

Is it possible to get x, y coordinates of an element? for example I need the center point of an element.

@vasily-v-ryabov hi, for backend win32 + ListView20WndClass, _listview_item.rectangle() returns relative rect to listview ?

It's not clear from the docs for LVM_GETITEMRECT window message: https://docs.microsoft.com/en-us/windows/win32/controls/lvm-getitemrect We don't convert this rectangle additionally. Please try and let us know if it's relative to main window or to list view or not.

It's not clear from the docs for LVM_GETITEMRECT window message: https://docs.microsoft.com/en-us/windows/win32/controls/lvm-getitemrect We don't convert this rectangle additionally. Please try and let us know if it's relative to main window or to list view or not.

relative to listview according to my test  :)

@laoyur thanks! It looks like a reason to improve consistency. Marked as enhancement.


Avoid hitting proto

This avoids hitting __proto__ for hardened environments where __proto__ is disabled.
See discussion in https://github.com/nodejs/node/issues/31951 and https://github.com/nodejs/node/pull/32279.
To check that this works, run with NODE_OPTIONS=--disable-proto=throw node -e 'require("./graceful-fs.js")' before and after this change.
There is no reason to set .read() proto when Object.setPrototypeOf is not available, as no special handling of util.promisify exists on those versions (even if util.promisify is shipped separately somehow), so no __proto__ fallback there.
This change works all the way down to 0.8 afaik — Object.getPrototypeOf is available there, and Object.setPrototypeOf is guarded. Have not checked on versions below 0.8.6.
I believe this could be a semver-minor or a semver-patch.

CI failure seems unrelated, I think.

CI failure seems unrelated, I think.


Simplified component state

As a developer, there should be minimal duplicate data in the state. For recipes, we currently have components, dependencies, and packages which all contain similar data. This will be simplified to just components and packages. We then filter our selected components and dependencies from the lists of packages and components.

LGTM.
All tests are passed in my local env.
The Travis test will be re-runed when #202 got merged.

@jkozol, This PR has code conflict with master after #197 got merged. Could you please fix the conflict? Thanks.

@jkozol This is awesome! I'd like to spend more time reviewing, but have a few questions for you...

When I look at the redux dev tools, I see that blueprint packages are included in the list of blueprint dependencies. Can you point out where in the code these packages get filtered out when displaying the dependencies?

One of the updates I'd like to make in the UI is to show the circle around components that are listed in the blueprint, and alter the circle styling based on whether the component is selected or a dependency. And do this wherever these components are listed, including the list of available components (e.g. a dependency that is listed in the available components list would have inRecipe: true and user_selected: false). I'm not expecting these UI changes as part of this PR, but I just want to check that these updates support that update (i.e. conditionally add a class based on user_selected: false).

Example of circle around a selected component:

Example of circle around a dependency:

I remember you mentioned something related to this, but currently, when I click a blueprint component list item to expand and view details, I see a list of dependencies. But if I click the component name to view the full details panel, I see no dependencies. Is it possible to have dependencies listed in both places? Are there issues or concerns with doing this? When you mentioned this, I thought you were describing the reverse (i.e. no dependencies listed in the list item, but still listed in the component details panel)

Example of expanded list item (Dependencies count is 22)

Example of details panel (Dependencies count is 0)

I get different results when I click a selected component in the list of Available Components on the left versus the list of Selected Components on the right.

Example of Component Details when clicking name in list on left (this is correct—this is how the panel should display for a selected component):

Example of Component Details when clicking name in list on right (this is incorrect—this is how the panel should display for a component that has not been added to the blueprint yet):

When I opened a blueprint that had pending changes from a previous session, things did not work at all. I was able to create a new blueprint, and things worked fine though. I'm assuming that's more of an issue with old blueprints with bad contents rather than an issue with the workspace, but just thought I'd mention it. I'll continue reviewing this along with the save workflow and let you know if I notice anything.

Codecov Report

Merging #203 into master will decrease coverage by 0.69%.
The diff coverage is 55.83%.

@@            Coverage Diff            @@
##           master     #203     +/-   ##
=========================================
- Coverage   43.33%   42.63%   -0.7%
=========================================
  Files          70       70
  Lines        2340     2350     +10
  Branches      319      320      +1
=========================================
- Hits         1014     1002     -12
- Misses       1113     1127     +14
- Partials      213      221      +8

Impacted Files
Coverage Δ

components/ListView/BlueprintContents.js
44.44% <ø> (ø)
:arrow_up:

core/reducers/blueprintPage.js
50% <ø> (ø)
:arrow_up:

core/store.js
75% <ø> (ø)
:arrow_up:

data/NotificationsApi.js
89.65% <ø> (ø)
:arrow_up:

components/ListView/ComponentDetailsView.js
1.21% <0%> (ø)
:arrow_up:

components/ListView/ComponentInputs.js
50% <100%> (ø)
:arrow_up:

core/selectors.js
94.64% <100%> (-1.52%)
:arrow_down:

core/actions/blueprints.js
100% <100%> (ø)
:arrow_up:

core/actions/blueprintPage.js
100% <100%> (ø)
:arrow_up:

core/sagas/modals.js
62.5% <100%> (+2.5%)
:arrow_up:

... and 14 more

Continue to review full report at Codecov.

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

@jgiardino, do you have any comment on this PR? Thanks.

@jgiardino

One of the updates I'd like to make in the UI is to show the circle around components that are listed in the blueprint, and alter the circle styling based on whether the component is selected or a dependency. And do this wherever these components are listed, including the list of available components (e.g. a dependency that is listed in the available components list would have inRecipe: true and user_selected: false). I'm not expecting these UI changes as part of this PR, but I just want to check that these updates support that update (i.e. conditionally add a class based on user_selected: false).

I have added these classes to the the input components when we load them through the inputs saga.

I remember you mentioned something related to this, but currently, when I click a blueprint component list item to expand and view details, I see a list of dependencies. But if I click the component name to view the full details panel, I see no dependencies. Is it possible to have dependencies listed in both places? Are there issues or concerns with doing this? When you mentioned this, I thought you were describing the reverse (i.e. no dependencies listed in the list item, but still listed in the component details panel)

The dependencies should load now regardless of if you select the component from the left or right panel.

I get different results when I click a selected component in the list of Available Components on the left versus the list of Selected Components on the right.

I believe this is fixed. I updated the blueprints to make sure they had the inBlueprint and/or user_selected classes.

When I opened a blueprint that had pending changes from a previous session, things did not work at all. I was able to create a new blueprint, and things worked fine though. I'm assuming that's more of an issue with old blueprints with bad contents rather than an issue with the workspace, but just thought I'd mention it. I'll continue reviewing this along with the save workflow and let you know if I notice anything.

Are you still experiencing this issue?

I have added these classes to the the input components when we load them through the inputs saga.

Awesome!

The dependencies should load now regardless of if you select the component from the left or right panel.

This works as expected, except the selected component is also included in the list of dependencies. Can you remove that one from the list? For example, in the image below, basesystem is the selected component but it also displays in the list of it's dependencies.
I also have some questions for you about the list of dependencies for an individual component as shown in the example below:

Are you able to know when a dependency of one component is also a selected component? For example, in the image below, bash is a dependency of basesystem but it's also a selected component. If this is not there already and not easy to add for this PR, we can capture it as a future enhancement.
I eventually need to update the list of dependencies shown below so that the user click a dependency and open the Component Details view. I can't find where the list of dependencies shown below is in the state. I'm assuming clicking a dependency in this list would work the same as clicking a dependency in the Dependencies tab, but just want to confirm that's the case and is possible to add later.

I believe this is fixed. I updated the blueprints to make sure they had the inBlueprint and/or user_selected classes.

Yes, this is awesome.
This is more of a rhetorical question, and not something we need to address now, but I'm curious what your thoughts are on the structure of inputs.selectedInput.  inputs.selectedInput.status is kind of a duplicate of inBlueprint and user_selected, but this information is only available on the first component. Ideally, if I click on a dependency of a component, it would be nice to be able to indicate if that dependency is also user_selected.

Are you still experiencing this issue?

No
Finally, I should have mentioned this before, but it would be nice to standardize on property names. It looks like the preference is camelCase. If it's easy enough to do a find and replace, could you update ui_type and user_selected to be camelCase.

@jgiardino

This works as expected, except the selected component is also included in the list of dependencies. Can you remove that one from the list? For example, in the image below, basesystem is the selected component but it also displays in the list of it's dependencies.

This should be fixed now.

Are you able to know when a dependency of one component is also a selected component? For example, in the image below, bash is a dependency of basesystem but it's also a selected component. If this is not there already and not easy to add for this PR, we can capture it as a future enhancement.

This should be possible but may be better captured in a future enhancement.

I eventually need to update the list of dependencies shown below so that the user click a dependency and open the Component Details view. I can't find where the list of dependencies shown below is in the state.

These dependencies are actually fetched from the api on an as needed basis per the component being viewed.

could you update ui_type and user_selected to be camelCase.

This should be fixed now.

@jgiardino, do you have any comment on this PR? Thanks!

Thanks for the updates! This looks good to me!!


Bulk import notifications

There is a sample code for bulk import monitors from UI to YAML but nothing similar for notifications. It is possible to bulk import all notifications by querying Monte Carlo's GraphQL API, unlike Monitors which can be fetched easily by using an object of the Query() class. Can we add that code sample to this repo? If yes, I can take up this issue and submit a PR.

@melissa-hale @lgavish @bheltzel @glencwillis - Raised a PR for this as well, please review!


Graphs Visualizations Showing Gaps After Update from 4.0.2 to 4.1.2

Describe the bug
After updating both my cloud and on prem zabbix plugin from 4.0.2 to 4.1.2, a one minute random gap is now appearing on the graphs.  Inspecting the data shows random data missing.
Expected behavior
Pre update, all graphs were rendering all data points.
Screenshots

Network data
Object
request:Object
url:"api/tsdb/query"
method:"POST"
data:Object
queries:Array[1]
0:Object
refId:"A"
format:"time_series"
datasourceId:7
rawSql:"SELECT CAST(itemid AS CHAR) AS metric, MIN(clock) AS time_sec, AVG(value) AS value FROM history_uint WHERE itemid IN (183598) AND clock ><PHONE_NUMBER> AND clock <<PHONE_NUMBER> GROUP BY (clock-1613663901) DIV 1, metric ORDER BY time_sec ASC"
maxDataPoints:10000
hideFromInspector:false
response:Object
results:Object
A:Object
refId:"A"
meta:Object
executedQueryString:"SELECT CAST(itemid AS CHAR) AS metric, MIN(clock) AS time_sec, AVG(value) AS value FROM history_uint WHERE itemid IN (183598) AND clock ><PHONE_NUMBER> AND clock <<PHONE_NUMBER> GROUP BY (clock-1613663901) DIV 1, metric ORDER BY time_sec ASC"
rowCount:0
series:Array[1]
0:Object
name:"183598"
points:Array[30]
0:Array[3141696,1613663910000]
1:Array[8338872,1613663940000]
2:Array[2046104,1613663970000]
3:Array[2539488,1613664001000]
4:Array[3455688,1613664030000]
5:Array[3115808,1613664060000]
6:Array[4677304,1613664090000]
7:Array[3283224,1613664120000]
8:Array[3623080,1613664150000]
9:Array[3527784,1613664181000]
10:Array[2732392,1613664210000]
11:Array[10731920,1613664240000]
12:Array[3688488,1613664270000]
13:Array[4051048,1613664300000]
14:Array[3925448,1613664330000]
15:Array[7162408,1613664361000]
16:Array[4936280,1613664390000]
17:Array[6617008,1613664421000]
18:Array[11618008,1613664450000]
19:Array[3599640,1613664480000]
20:Array[3490800,1613664510000]
21:Array[6085904,1613664541000]
22:Array[3649960,1613664570000]
23:Array[3254600,1613664601000]
24:Array[5698512,1613664630000]
25:Array[3037088,1613664660000]
26:Array[3365144,1613664690000]
27:Array[2300376,1613664720000]
28:Array[3317320,1613664750000]
29:Array[4180984,1613664780000]
tables:null
dataframes:null
Software versions

Grafana
Zabbix
Grafana-Zabbix Plugin

7.4.1-7.4.2
5.0.8
4.1.2

Same problem when upgrade plugin to 4.1.2, downgrade to 4.0.1 is workarround

+1

+1

+1

Same problem here, and the change also generated the same log errors as those described in  #1157

This behaviour now enabled by default in order to keep compatibility with changes in Grafana stacking graphs. Howewer, you can disable it in query options or globally in data source config by switching Disable data alignment option:

Hello @alexanderzobnin . I have just met the issue and turning on "Disable data alignment" helped. But I'm totally confused trying to get how the only few affected graphs in our grafana differ from plenty of others (all from zabbix). Can you provide a little bit more details on what graphs it may happen and why and what problems with stacking graphs can we get if we turn on this option globally?


Support sparse_segment_sum ops on GPU.

SparseSegmentSum and SparseSegmentSumWithNumSegments are two important opeartions in sparse training and inference. Currently, there are no GPU impl in Tensorflow for them.
This PR provide GPU version of SparseSegmentSum and SparseSegmentSumWithNumSegments

LGTM


explore the dataset

Hey thank you for sharing the code and dataset! I am wondering if there is a plan to build an online demo for the dataset (e.g., COCO Explorer), so we can explore image sets easily.

hi @zhaoyanpeng, thanks for your interest! We are in the process of creating an explorer, but that may not happen in April likely due to the semester's busyness.

Hi, I am sorry I haven't posted an update since Siva responded. There has been a demo since May!
https://huggingface.co/spaces/BennoKrojer/imagecode-demo


select: inconsistent invalid styling for required attribute and reactive form validator

Describe the bug
Native select menus configured with Validators.required turn red immediately after clicking them. When configured with the required attribute, the select does not turn red until blurred. Seems like the bug is on the select configured with the reactive form since selects did not always turn red immediately.
To Reproduce
Steps to reproduce the behavior:

Go to the Native Select - Leading Icon select example

Click the select menu. Color will be the primary focused color and label will be still be floated (should it still be floated at this point? possibly a separate bug?)

Click once outside without selecting an option (should still be the primary color)

Click again and it should finally turn red/invalid

Go to the Select with FormControl

Click the select menu. Color will immediately turn red

Click once outside and label will be floated and select still red

Click again and label will no longer be floated, but color is still red/invalid

Expected behavior
Validity styling should be consistent between select configured with reactive form validity and select configured with required attribute. The correct behavior would be that of the select configured with the required attribute
Screenshots
Select with required attribute after clicking on it and clicking outside of it once (remains primary color):

Select with reactive form control required immediately after clicking on it (turns red):

What Angular MDC version are you using?
0.43.7
What OS are you using?:
MacOS Mojave
What browser(s) is this bug affecting?:
Chrome

@mrp67 Thanks for reporting this!
Adding this to v0.44.1 milestone.

PR incoming


[BUG] the runtime master pod is not restricted to the node where hostpath is located

What is your environment(Kubernetes version, Fluid version, etc.)
Descirbe the bug
When I try to accelerate hostpath located on one of my kubernetes cluster nodes, which is a new feature introduced in fluid v0.3.0 ,I have set the correct nodeAffinity  in dataset.yaml to let fluid know which node my hostpath is on, but it seems that fluid did not use that nodeAffinity for runtime master pod, So fluid is likely to create master pod on an unexpedted node, and consequently the pvc created by fluid is empty.
What you expect to happen:
How to reproduce it
Additional Information

@iluoeli I've fixed this issue, please review it. Thanks.

@iluoeli I've fixed this issue, please review it. Thanks.

/lgtm. thanks @cheyang

fixed by #130


It does not let me enter the following link:hhttps://pancakeswap.finance/farmsttps://pancakeswap.finance/farms

It does not let me access to pancakeswap the following link:hhttps://pancakeswap.finance/farmsttps://pancakeswap.finance/farms

https://github.com/polkadot-js/phishing/pull/4122


Long array having id 62 instead of 12?

I was reading the code and I saw the LongArrayTag is mapped with ID 62 instead of 12 as specified in the NBT specification. (see https://wiki.vg/NBT)
https://github.com/Darkhax/OpenNBT/blob/b1fe46640bf8c9f301d06428b8773f62f829fbfb/src/main/java/net/darkhax/opennbt/tags/TagRegistry.java#L43

The tag IDs 60+ in this library are not part of the official NBT spec. It looks like long arrays are a new feature added by Mojang.

So you can update the id to support the new tag from newer nbt files, and maybe add something to remap 62 to 12 for existing files created with this lib


How can I get the performance across a suite of popular benchmarks like code/Commonsense Reasoning

Thanks for such a great job.
I have a question.
How to compare the performance of the models before and after quantization across a suite of popular benchmarks like code/Commonsense Reasoning Reasoning listed on Table 3 of Llama2 paper.
Thank you in advance.

@taoxunqiang  to confirm my understanding firstly, "the performance" here means the accuracy rather than the latency or throughput, correct?  If so, a DIY approach is to refer the existing summarize.py to add the related accuracy measurement logic.
June

@juney-nvidia Thank you, your understanding is correct
Thanks for the quick reply too


⚠ Bofrost.de has degraded performance

In fb6ce19, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1873 ms

Resolved: Bofrost.de performance has improved in c3cb1fb after 12 minutes.


Add 2 bitsong fantokens

Description
Adding CLAY from chain BitSong
Adding 404DR from chain BitSong
Checklist

Adding Assets

If adding a new asset, please ensure the following:

[ ] Asset is registered to the Cosmos Chain Registry.
[ ] Add asset to bottom of zone_assets.json.

[ ] chain_name and base_denom are provided and use values exactly as defined at the Chain Registry.
[ ] path is provided, and the IBC channel referenced is registered at the Chain Registry (unless native to Osmosis).
[ ] osmosis_verified is set to false
[ ] listing_date_time_utc is specified and accurate
[ ] Optional: transfer_methods, peg_mechanism, override_properties, canonical, categories, where necessary (see README for details).

Adding Chains

If adding a new chain, please ensure the following:

[ ] Chain is registered to the Cosmos Chain Registry.

Chain's registration must have staking defined, with at least one staking_token denom specified.
Chain's registration must have fees defined; at least one fee token has low, average, and high gas prices defined.

[ ] IBC Connection between chain and Osmosis is registered.
[ ] Add chain to bottom of zone_chains.json

[ ] rpc and rest does not have any CORS blocking of the Osmosis domain, and RPC node has have WSS enabled.
[ ] explorer_tx_url correctly directs to the transaction when the hash is inserted into the URL.

Upgrading Asset to Verified

If upgrading an Asset to Verified, please see the requirements specified at LISTING, and ensure the following:

[ ] Asset is defined thoroughly at the Cosmos Chain Registry.

[ ] A meaningful description and extended_description.
[ ] Associated socials, including website and twitter.
[ ] Logo Image has a square Aspect Ratio and < 250 KB file size.

[ ] This pool contains contains at least $1k USD-worth of liquidity of the asset (Provide Pool ID): ______

'Verified' Status Validation Checklist (to be completed by Osmosis Zone maintainers):

[ ] Verify appearance and metadata
[ ] Accurate Price
[ ] Trading and routing functionality
[ ] Withdraw and Deposit

[ ] Deposit Transaction URL (if in-app)

This is not the method assets are added (see docs). They should only go into the zone_assets file. Also please see the PR checklist.

Hi @JeremyParish69, I remembered another procedure. I've made the changes and updated the task list, could you please review to ensure I've done everything correctly?


How to open the downloaded trace.

"For further analysis, you can click Download Trace to download the trace in to Visual Studio."
Is it really possible to open the downloaded etl file with Visual Studio? I could open in PerfView but I could not open in Visual Studio.

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

ID: 90bab7a8-40fe-0736-c86c-a34ac821c006
Version Independent ID: 6b872c6f-a8ca-f548-24d9-28d82e350b69
Content: Diagnose performance issues using Azure Application Insights
Content Source: articles/azure-monitor/learn/tutorial-performance.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @mrbullwinkle
Microsoft Alias: mbullwin

Hi @levi106
Thank you for your question. I will take a look and get back to you shortly.

I believe you are correct that PerfView is required. I've submitted a pull request to correct the documentation.


[$500]  IOU - Replace receipt option is available for requestee in combined IOU report

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.59-0
Reproducible in staging?: y
Reproducible in production?: n
Issue reported by: Applause - Internal Team
Action Performed:

Go to staging.new.expensify.com
[User A] Go to chat with User B that has no unsettled IOU.
[User A] Request money from User B and attach a receipt.
[User B] Go to combined IOU report with User A.
[User B] Click on the receipt.
[User B] Click 3-dot menu.

Expected Result:
There will be no Replace receipt option for User B (requestee).
Actual Result:
Replace receipt option is available for User B (requestee).
This issue only happens in combined IOU report.
Workaround:
n/a
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/93399543/a007796a-23c6-4792-b540-9087803e5546
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01e259e1fb396c9678
Upwork Job ID:<PHONE_NUMBER>648724480
Last Price Increase: 2024-04-04

We think that this bug might be related to #wave-collect - Release 1

Proposal
Please re-state the problem that we are trying to solve in this issue.
Replace receipt option is available for User B (requestee).
This issue only happens in combined IOU report.
What is the root cause of that problem?
When we open the receipt in combine report, the reportID in URL is the expense report so canEditReceipt check is wrong here because the parentReportAction now is the report preview action.
https://github.com/Expensify/App/blob/879378ff68dc026c1b4ba6af30b70051b99c6d35/src/components/ReportActionItem/ReportActionItemImage.tsx#L98-L101
https://github.com/Expensify/App/blob/879378ff68dc026c1b4ba6af30b70051b99c6d35/src/pages/TransactionReceiptPage.tsx#L36
What changes do you think we should make in order to solve the problem?
We should get the transactionReportID and pass it as reportID param of getRoute function here. So all check in TransactionReceiptPage will work well even if we open the receipt in combine report.
https://github.com/Expensify/App/blob/879378ff68dc026c1b4ba6af30b70051b99c6d35/src/components/ReportActionItem/ReportActionItemImage.tsx#L98-L101
To do that we should store transactionThreadReport or transactionThreadReportID in ShowContextMenuContext.

Add transactionThreadReport in ShowContextMenuContextProps

type ShowContextMenuContextProps = {
    ...
    transactionThreadReport: OnyxEntry<Report>;
};

const ShowContextMenuContext = createContext<ShowContextMenuContextProps>({
    ...
    transactionThreadReport: null
});

https://github.com/Expensify/App/blob/879378ff68dc026c1b4ba6af30b70051b99c6d35/src/components/ShowContextMenuContext.ts#L12

Store transactionThreadReport into contextValue here.

https://github.com/Expensify/App/blob/879378ff68dc026c1b4ba6af30b70051b99c6d35/src/pages/home/report/ReportActionItem.tsx#L364

Here, get transactionThreadReport and pass transactionThreadReport.reportID to getRoute function

{({report, transactionThreadReport}) => (
    <PressableWithoutFocus
        style={[styles.w100, styles.h100, styles.noOutline as ViewStyle]}
        onPress={() => Navigation.navigate(ROUTES.TRANSACTION_RECEIPT.getRoute(transactionThreadReport?.reportID ?? report?.reportID ?? '', transaction?.transactionID ?? ''))}
        accessibilityLabel={translate('accessibilityHints.viewAttachment')}
        accessibilityRole={CONST.ROLE.BUTTON}
    >
        <ReceiptImage {...propsObj} />
    </PressableWithoutFocus>
)}

https://github.com/Expensify/App/blob/879378ff68dc026c1b4ba6af30b70051b99c6d35/src/components/ReportActionItem/ReportActionItemImage.tsx#L98-L101
What alternative solutions did you explore? (Optional)
NA

I don't think this is worth blocking deploy on this as we're marking other bugs related to the combined IOU report view as non-blockers.

@jasperhuangg The PR is here

@jasperhuangg Wanted to check if C+ is needed here? Seems I was removed and deployment blocker is also removed?

@nkdengineer - I've sent an offer via Upwork. Please accept and I can complete the payment process. Thanks!

Closing, I was able to complete the payment to @nkdengineer. @allroundexperts please submit a request in Chat. Thanks!

$500 approved for @allroundexperts


4.2 - Path3D Control Nodes - Cannot create bezier handles in top down ortho view.

Tested versions
works correctly in v4.1.3.stable.official [f06b6836a] as it doesnt have the Tilt Handles in editor.
issue occurs in v4.2.stable.official [46dc27791] as tilt handles were introduced.
System information
Godot v4.2.stable - Windows 10.0.22621 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3050 Laptop GPU (NVIDIA; <IP_ADDRESS>92) - AMD Ryzen 5 4600H with Radeon Graphics (12 Threads)
Issue description
with the path3d node and looking in top down orthgraphic view cannot use control point button to drag and create bezier handles.
only the tilt handle is selectable due to it being higher up and making the control point unclickable to drag.
Steps to reproduce
all i done was in 4.2 was create a new 3d scene.
create a path3d node.
into top down ortho view.
created a few points with the  button.
then, using the select control points button

to try and create bezier handles.
this only indicates that the tilt handle has been selected and cannot create beziers.
my only workaround was to look from bottom up ortho, create the bezier handles and then return to top down ortho view to adjust placements.
possible solution would maybe have another button that hides or only controls the tilt handles

Minimal reproduction project (MRP)
N/A

cc @xiongyaohua

I hope this can be fixed soon, as I'd like to work a lot with paths.
I suggest to have another button that enables/disables working with tilt handles, enabling working on bezier handles and tilt handles separately.

This issue is broader than just top down ortho view, really. Any time the camera is far enough away from path points that the tilt handles overlap with the points, it starts to become correspondingly difficult to create bezier handles instead of mistakenly tilting the path.

Could anyone besides xiongyaohua fix this? I just wonder if it is waiting on one person's inbox. 😐 This is fairly high-priority for Path3D users, IMHO.

I could probably take a look at it, but I'd go this route and avoid complications:

I hope this can be fixed soon, as I'd like to work a lot with paths. I suggest to have another button that enables/disables working with tilt handles, enabling working on bezier handles and tilt handles separately.

@Wolfe2x7 would that satisfy you?
This would eventually need a new icon for the button, not sure how to approach that (I can make one that follows the already existent style?).
Should we open a proposal?
I'm not sure if this issue could be already solved by https://github.com/godotengine/godot/pull/80753 and its proposal https://github.com/godotengine/godot-proposals/issues/7462

I could probably take a look at it, but I'd go this route and avoid complications:
A button like you described sounds fine to me. 👍

@Wolfe2x7 I opened a proposal https://github.com/godotengine/godot-proposals/issues/8890

Yeah, I have the same issue.


Removing semantic-release.

Not being used.

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


Bulk COPY returns rows inserted

Fixes #30 . Simply adds int return type to PostgreSQLCopyHelper.SaveAll().
This PostgreSQLCopyHelper change depends on upstream npgsql/npgsql#2115 being merged and deployed, so that npgsql supports the Postgresql COPY return rows.

Yes, I think I could be release a Pre Release to NuGet, once the feature is merged to Npgsql. I still remember my 4.0 Upgrade disaster. 🤗

This feature will be added by Pull Request #33.
@chrisatx Thanks for all the work you have put into it!


Improve sparse adagrad update

Description

The weight decay term in Adagrad is incorrect. See https://github.com/apache/incubator-mxnet/issues/9363 for more details.
The sparse update is not fast enough for the language model I'm training. Moving the logic to cpp backend so that we don't suffer from blocking calls such as RowSparseNDArray.indices and RowSparseNDArray.data.

@ZiyueHuang @piiswrong @sxjscience @cjolivier01 @szha
Commit author is mistaken. Nvm.
 I finished coding on this PR)
[x] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[x] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
[ ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[x] Feature1, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

Is this ready to go in?

Hi everyone,
Considering the wd term is implemented inconsistently in many other optimizers (see https://github.com/apache/incubator-mxnet/issues/9881) , I removed the changes for wd in this PR so that the scope of changes is limited. Let's discuss how to fix the behavior of wd systematically in a separate issue/PR.


How to add font icon in labels or series  part

Default chartis have number or text as label but i want to add font awesome(star as rating) instead of labels or series.

I think you should be able to do this in CSS, right?


display version info when running in electron

I find it frustrating that i dont' know if the elecron app is updating or not. Add some info about online version availability in electron.

not online -> please connect to internet to update
online, updated -> you're up to date
online, outdated, an update is avilable

@abchatra this is a simple hint like Chrome about the current version.

I don't understand the change. How do you know the update is available? Just based on the live version? We may not have released the latest one yet. It should say update available only if there a latest electron version available.

My assumption (which may be wrong) is that targetConfigAsync() pulls the live version of targetconfig.json; then we can compare the version in electronManifest.latest to the current version. Since the electron layer is already doing that work, it could be added to the onStatusUpdate event too.

I think you are right and it looks good. Ideally we can link to download page to update too (assuming automatic update is failing for some reason)

Is there a default url scheme for the download page? I can add a link.

I guess we don't have latest URL probably we should add to the backend api
We have this in the download page. It is annoying to edit it every time.
            window.open("https://makecode.com/api/release/microbit/v1.2.13/win64");
            tickEvent("offlineapp.download", { "target": "microbit", "platform": "win64" });

But you could make the url from the electron.manifest version
https://makecode.com/api/release/microbit/[[electronManifest.latest]]/win64

Just add JavaScript that downloads the target config and generate the url.

From: Abhijith Chatra<EMAIL_ADDRESS>Sent: Friday, April 19, 2019 4:22:30 PM
To: Microsoft/pxt
Cc: Peli de Halleux; Author
Subject: Re: [Microsoft/pxt] display version info when running in electron (#5490)
I guess we don't have latest URL probably we should add to the backend api
We have this in the download page. It is annoying to edit it every time.
        window.open("https://makecode.com/api/release/microbit/v1.2.13/win64");
        tickEvent("offlineapp.download", { "target": "microbit", "platform": "win64" });

But you could make the url from the electron.manifest version
https://makecode.com/api/release/microbit/[[electronManifest.latest]]/win64https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fmakecode.com%2Fapi%2Frelease%2Fmicrobit%2F%255B%255BelectronManifest.latest%255D%255D%2Fwin64&data=02|01|jhalleux%40microsoft.com|dff06722174b4926131a08d6c51de516|72f988bf86f141af91ab2d7cd011db47|1|0|636913129566229113&sdata=Nv4sr6gTJ%2BVCEfciqKVDmGh2teVfrkDVd7rgEF3klHs%3D&reserved=0
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fpxt%2Fpull%2F5490%23issuecomment-485036520&data=02|01|jhalleux%40microsoft.com|dff06722174b4926131a08d6c51de516|72f988bf86f141af91ab2d7cd011db47|1|0|636913129566239126&sdata=Uco%2Fvj8Po3bfNXBJxb%2BG3jh8MLP1N%2FDMJ%2BIoHwwHvZI%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAA73QKLAYLCUJNUGRQRZS7LPRJH3NANCNFSM4HHFNUMA&data=02|01|jhalleux%40microsoft.com|dff06722174b4926131a08d6c51de516|72f988bf86f141af91ab2d7cd011db47|1|0|636913129566239126&sdata=6vlj%2FRP2LdcWH%2BncLHS7aou16e%2BQUwxtnHE5iFNjnj0%3D&reserved=0.


Detect missing make setup call, or make it automatic

I'd be interested in this. Mind if I give it a go? Just added hellogopher to my project :)

Absolutely :) the simplest and least invasive it is, the most likely it will get turbo-merged.

Thanks!
I've made it complain if you haven't run it yet, thought perhaps the best solution was to add a .GOPATH/.setup dependency like you have with .GOPATH/.ok


Adding +3 lightning bow increases Animate Weapon tooltip.

Here elemental animate weapon build. https://pastebin.com/G7xJyqLw
Auras socketed into Victario's Charity. Try to remove/add bow from main hand and check tooltip. AW gem setup socketed into "None".

You haven't set up an item set for Animate Weapon to use, so it's using the default item set with the +3 bow.
You need to create a new item set in the Items tab (click "Manage" at the top, above the item slots), equip a weapon of some sort (you can use the "Craft Item..." option to create a white weapon), then select the new item set from the dropdown in the sidebar.
The result should be something like this.


WIP 306 pjg climproduction devel

still trying to sort out permissions probs... maybe now working???

yahoo! this one sounds good!

@gleckler1 this is looking a fair bit cleaner.. Although you have changed the mode on one file:
src/python/pcmdi/scripts/pcmdi_compute_climatologies.py
But this is far better than 138 files..

@doutriaux1 @durack1 finally, a little progress...


test_autograd.py very slow (hangs?) on single-cpu-socket system.  Meaning of mx.cpu(1) in that case?

Description
I'm seeing the command:
pytest --verbose -s tests/python/train/test_autograd.py::test_autograd

timeout after 20 minutes on a system with 1 cpu socket, where the test runs in under a minute on a 2 cpu system.  The test is a cpu test that references mx.cpu(1).
What is the meaning of mx.cpu(1), in particular on a single-socket system?  Should this throw an exception?

Turns out the 'slow' system with one socketed cpu was an AMD arch, and the 2-cpu system had Intel.  Exploring further, a 1-cpu Intel system is fast, and a two-cpu AMD arch system is slow.  In other words, the slowness appears tied to the AMD architecture.  Perhaps a misconfiguration of openmp for AMD arch?

All CPUs will be used for compute and the device index doesn't have special meaning with respect to compute, threads, or locality.

I developed a work-around for myself for the tests/python/train/test_autograd.py test:

If the cpu type is AMD, set OMP_NUM_THREADS=min(32, nproc)
If the cpu type is AMD and there's only one cpu socket, omit the portion of the test that references mx.cpu(1)

These rules get the test to pass in a reasonable time on the machines I have access to.  I'm reluctant to make a PR of these changes, since clearly our upstream CI does not exercise the failure mode.  Also, this does not represent a proper tuning of openmp on the AMD arch.  Perhaps someone with a greater stake in training on AMD cpus can pick up this issue and run with it?


Have a 'div' operator

I keep writing (a divmod b)[0] in all kinds of situations, which is silly. We should have a div operator. Integer division.
For consistency, I guess this would mean also renaming % to mod, but... neither Perl 5/6, Python or JavaScript agrees with that.
We could go with Python's // spelling, but we're already using that for Perl 6's defined-or, and I don't think we should change that.

Ping #438 and @claes-magnus: so I think you were right. :wink: But we're going to call the operator div and not /. Just so people don't get any ideas.


Refactor metal tc wmma kernel rendering

Can't use cleaner simdgroup_load instruction as inputs are in thread memory. The method requires source to be either in device memory or threadgroup memory. Same for simdgroup_store instruction. (so stuck with mat_a.thread_elements()[0] = a[0];)
I know right now low LOC are a priority but I couldn't make it more readable without those extra lines. (It could be shortened with shorter variables names but I think conceptually is a good refactor)
Generated wmma kernel
float2 __WMMA_8_8_8_half_float(half2 a, half2 b, float2 c) {
  simdgroup_half8x8 mat_a, mat_b; simdgroup_float8x8 mat_c;
  mat_a.thread_elements()[0] = a[0];
  mat_a.thread_elements()[1] = a[1];
  mat_b.thread_elements()[0] = b[0];
  mat_b.thread_elements()[1] = b[1];
  mat_c.thread_elements()[0] = c[0];
  mat_c.thread_elements()[1] = c[1];
  simdgroup_multiply_accumulate(mat_c, mat_a, mat_b, mat_c);
  return float2(mat_c.thread_elements()[0], mat_c.thread_elements()[1]);
}

Forced to use os.linesep instead of \n as CI throws SyntaxError: f-string expression part cannot include a backslash. Worked perfectly fine with \n in local setup.

LOC is not a priority! Always readability over LOC

can you use walrus if you wanna alias arg idxs?

float2 __WMMA_8_8_8_half_float(half2 a, half2 b, float2 c){
  simdgroup_half8x8 mat_a, mat_b; simdgroup_float8x8 mat_c;
  mat_a.thread_elements()[0] = a[0]; mat_b.thread_elements()[0] = b[0]; mat_c.thread_elements()[0] = c[0];
  mat_a.thread_elements()[1] = a[1]; mat_b.thread_elements()[1] = b[1]; mat_c.thread_elements()[1] = c[1];
  simdgroup_multiply_accumulate(mat_c, mat_a, mat_b, mat_c);
  return float2(mat_c.thread_elements()[0], mat_c.thread_elements()[1]);
}

generated kernel

can you use walrus if you wanna alias arg idxs? or destructure arg later. either way multiple variable assignments in one line is even more unreadable.

thanks for your feedback!


Can't save edited quests

Version: Quests 4.0.1 (Naughty Update)
Problem: Basically, cannot save edited quests, I can create quests but when trying to edit, the plugin doesn't let me press "yes" when saving. Also, this pops up in console when I press it: https://pastebin.com/4AGR2YWH
It affects EVERY quest.

Thanks for the report. Please try https://ci.codemc.io/job/PikaMug/job/Quests/232/

Thanks for the report. Please try https://ci.codemc.io/job/PikaMug/job/Quests/232/

Worked, thank you so much


Update MD file on new branch 2017-12-06_09-40-30

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon1.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


A request for train dataset

Thanks for your wonderful work! I saw your article that you sampled the PointFilter data set, but I downloaded the training data set of you and PointFilter and found that there was no original mesh data. So would you kindly share the mesh data of training?
I would really appreciate it!

Since the dataset of our method comes from pointfilter, as far as I know, there is no mesh in the training set. If you need mesh for training, I recommend you use the dataset of score-denoise .


Touble in install tha package

Hi, thanks for developing such wonderful tools for us. Bot I have trouble in installing the packages. I'm looking forward to your conda install method.
Dan

Hi Dan, I am hoping to have a conda install method soon. In the meantime, I am happy to try and help debug any errors you are having!


Added solution for cf_987A

Problem Link: https://codeforces.com/problemset/problem/987/A

Looking forward to your next contribution. Add some more problems solutions ❤


Cache and reuse ELM once CQL is translated

Describe the issue
The CQL engine that's embedded into HAPI currently always translates CQL from source code into ELM prior to execution. This means that each $evaluate-measure call has several ms of overhead that can be avoided if it were able to cache the ELM (and know when to invalidate it).
In order to make this happened, the CQL engine needs to:

Prioritize using ELM over CQL if it's available
Cache the ELM object graph when it's either deserialized from a Library resource or generated from CQL
Invalidate the cache if a Library resource changes (hapi exposed the ResourceChangeListener interface to facilitate this).

Environment (please complete the following information):

HAPI FHIR Version 5.3.0

Resolved in #4347


fix certificate expiration issue.

Description:
fix certificate expiration issue.

Testing done:
DONE

Affected area:

Functional Area

New Functionality
[ ]

CI System
[ ]

CLI Tool
[ ]

Certificate Management
[ ]

Control Plane
[ ]

Demo
[ ]

Documentation
[ ]

Egress
[ ]

Ingress
[ ]

Install
[ ]

Networking
[ ]

Observability
[ ]

Performance
[ ]

SMI Policy
[ ]

Security
[ ]

Sidecar Injection
[ ]

Tests
[ ]

Upgrade
[ ]

Other
[X]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? no

Is this a breaking change? no

Has documentation corresponding to this change been updated in the osm-edge-docs repo (if applicable)? no

Codecov Report
Patch coverage: 66.66% and project coverage change: -0.01 :warning:

Comparison is base (197f936) 60.94% compared to head (0e4da36) 60.94%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@               Coverage Diff                @@
##           release-v1.3     #177      +/-   ##
================================================
- Coverage         60.94%   60.94%   -0.01%
================================================
  Files               218      218
  Lines             17105    17108       +3
================================================
+ Hits              10425    10426       +1
- Misses             6627     6629       +2
  Partials             53       53

Flag
Coverage Δ

unittests
60.94% <66.66%> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/k8s/announcement_handlers.go
73.61% <33.33%> (-1.76%)
:arrow_down:

pkg/k8s/client.go
91.96% <100.00%> (ø)

...r/providers/pipy/registry/announcement_handlers.go
79.31% <100.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


External collaborator cannot leave the project

An external collaborator who is also an Owner cannot leave the project
To recreate the issue-

Click on 'Your Projects'
Select a Project where the user is an external collaborator and Owner of the Project

The external collaborator cannot leave the project

This is closely related to #4729.


Data missing - 2021-02-05

Using yf.download for the date 2021-02-05 with interval 5M and duration 7D with many tickers leads to many missing rows, tried multiple times.
Tried downloading in chunks of 250, also same.
But downloading individual tickers for that day, but that will take forever. Why does this happen?

If this is still happening, why do you think 'missing rows' is bad? 5m is a short interval, maybe no trades occurred?


refactor(wasmbindings): remove gamm and twap bindings

Closes: #XXX
What is the purpose of the change
Removing bindings gamm and twap. These have been causing additional overhead during refactors while not having production use cases.
In production, everyone should be using osmosis-std instead.
Bank and tokenfactory bindings have use cases outside of Osmosis so these are kept for now.
An announcement about removal is to be made post-merge.
Testing and Verifying
This change is a trivial rework / code cleanup without any test coverage.
Documentation and Release Note

Does this pull request introduce a new feature or user-facing behavior changes? yes
Is a relevant changelog entry added to the Unreleased section in CHANGELOG.md? yes
How is the feature or change documented? not applicable

Planning to merge. No one has gotten back to me about using these on Telegram or Discord

The only concern raised by the integrators is having the following queries whitelisted:
https://github.com/osmosis-labs/osmosis/blob/a5bc6077a60f825ee95cec9be1f442d66a04cbfa/wasmbinding/stargate_whitelist.go#L87-L88
As a result, this should be good to be merged


Should round_trip=True ignore @computed_field ?

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
If I understand the round_trip=True option correctly, I think it should ignore @computed_field ? With the example bellow, using extra='forbid' makes the round trip fails.
Example Code
from typing import List

from pydantic import BaseModel, Json, computed_field, ConfigDict

class Model(BaseModel):
    json_obj: Json[List[int]]

    model_config = ConfigDict(extra='forbid')

    @computed_field
    @property
    def first(self) -> int:
        return self.json_obj[0]

dumped = Model(json_obj="[1, 2]").model_dump_json(round_trip=True)
print(dumped)
# > {"json_obj":"[1,2]","first":1}
print(Model.model_validate_json(dumped))
# pydantic_core._pydantic_core.ValidationError: 1 validation error for Model
# first
#   Extra inputs are not permitted [type=extra_forbidden, input_value=1, input_type=int]

Python, Pydantic & OS Version
pydantic version: 2.1.1
pydantic-core version: 2.4.0

Selected Assignee: @dmontagu

Ye, I guess it should 👍.
My guess is there are a number of other changes we should make to improve round_trip, but I don't know what they are right now.

PR welcome to pydantic-core to skip computed_field if round_trip=True.

Apologies pretty low value contribution but a quick easy temporary "fix" for those affected is:
from pydantic import BaseModel, model_serializer, SerializationInfo

class RoundTripModel(BaseModel):

    @model_serializer(mode="wrap")
    def _pop_extra_fields_round_trip(self, handler, info: SerializationInfo):
        """
        Removes all computed fields to support serialize/deserialize when `round_trip=True`:
        https://github.com/pydantic/pydantic/issues/7158

        Warning: A side effect of defining a `@model_serializer` is that any raised exceptions will be translated to
        `PydanticSerializationError` by `pydantic_core`.
        """
        result = handler(self)

        if info.round_trip:
            if info.by_alias:
                excl = (f.alias for f in self.model_computed_fields.values())
            else:
                excl = self.model_computed_fields

            for key in excl:
                del result[key]

        return result


Live update to log files in editor

Issue: Stale Log File View
Solution: Automatically update view in VS Code of the Log File
More Detail:
It would be really nice to have the IDE automatically update file changes to .log files. For debugging live changes instead of after the file is completely written. There is a hacky way to do it now by clicking off window and clicking back on. But it would be really nice for basically a "live refresh" of files. Thank you.
VS Code Version: 1.40.1

Please open issue for the extension you are using for debugging live changes.

I just want to say that I am not using an extension. I just want the IDE to auto update its view when a file changes that wasn't caused by that IDE. @octref

But this is limited by file system change events. There's no way for VS Code to get "live updates" from your file system. We can only poll for changes, and doing it too often causes slowness.


oz deploy fails to deploy empty contract using upgradeable and minimal types.

Given a simple contract:
pragma solidity ^0.5.0;

contract Foo {
    function hello() public pure returns(string memory) {
        return "hello";
    }
}

in empty project, running oz deploy with either upgradeable or minimal would produce the following error Given network 'dev-1580295554917' is not defined in your networks.js file.

One possible fix is to assign config to a property Object.assign(params, { config });


Rails 7 support?

I am currently using the gem directly from github for rails 7. Any idea (or help needed) to get the rails 7 version out soon?

bumping this one, would also love to see that.

bump

bump

bump

bump

bump

bump

bump

bump

bump

is there anything else besides #1517 ? any help is welcome!

any update?

Until new release will be pushed to RubyGems you can point the gem to GitHub master branch:
gem "devise_token_auth", github: "lynndylanhurley/devise_token_auth"

v1.2.1 was released!


Added ListSeriesEvent and extracted ListSeriesHandler

This is not ready to be merged, I just pushed an example in order to gather feedback.
This is an attempt to make REST handlers classes shorter. CDI events decouples the JAX-RS annotated bean from the actual handling code (and vice versa).
We could have for example:
@Observes @NewData NumericMetricEvent

Then both InfluxHandler and vanilla handler could fire a @NewData NumericMetricEvent
WDYT?

Closing without merging as:

no interest shown
we might want to minimize adherence to CDI


Create criteria menu (closes #5)

[x] Group layout
[x] Ands
[x] Ors
[x] None of (NOT OR)

[x] Account age
[x] Amount
[x] Fee fine type
[x] Location
[x] Patron group
[x] Service point

tests

[x] Added queries
[x] Group layout
[x] Ands
[x] Ors
[x] None of (NOT OR)

[x] Account age
[x] Amount
[x] Fee fine type
[x] Location
[x] Patron group
[x] Service point

still no idea why the debug test mode is faster in GH actions and GH actions alone, but oh well...


feat: introduced base security context

Signed-off-by: James Milligan<EMAIL_ADDRESS>

This PR

adds a base security context to the container definition for flagd:

    container name: flagd
    image: ghcr.io/open-feature/flagd:v0.2.5
        allowPrivilegeEscalation: false
        capabilities: map[drop:[all]]
        privileged: false
        readOnlyRootFilesystem: true
        runAsGroup: 65532
        runAsNonRoot: true
        runAsUser: 65532

Related Issues

https://github.com/open-feature/open-feature-operator/issues/225
Notes

Follow-up Tasks
It is likely going to worthwhile to allow for the specification of a sec comp profile path, however discussion is required as to how this is going to be handled. One option is for the path for the specified seccomp profile could be passed into configuration via helm.

Codecov Report

Merging #245 (5629c36) into main (a35cc7e) will increase coverage by 3.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #245      +/-   ##
==========================================
+ Coverage   53.16%   56.18%   +3.01%
==========================================
  Files           3        3
  Lines         363      388      +25
==========================================
+ Hits          193      218      +25
  Misses        156      156
  Partials       14       14

Impacted Files
Coverage Δ

webhooks/pod_webhook.go
81.08% <100.00%> (+2.40%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

@james-milligan have you deployed and tested this? If we're blocking open on root directories that would mean we can't use /etc for example to mount the config map option ( Remember its still possible to use configmaps and not just the API for K8s )

@james-milligan have you deployed and tested this? If we're blocking open on root directories that would mean we can't use /etc for example to mount the config map option ( Remember its still possible to use configmaps and not just the API for K8s )

Ive tested this using this syncProvider config and have found no issues, its my understanding that this flag only prevents the writing to these root directories by the container, although I may have additional configuration issues that is allowing the code to continue behaving normally:
apiVersion: core.openfeature.dev/v1alpha1
kind: FeatureFlagConfiguration
metadata:
  name: test-config
spec:
  syncProvider: {
    name: filepath
  }
  featureFlagSpec: |
    {
      "flags": {
        "myFlag": {
          "state": "ENABLED",
          "variants": {
            "on": true,
            "off": false
          },
          "defaultVariant": "off"
        }
      }
    }

Using a base config kind cluster and the current flagd release


Change IP/URL of Frigate instance?

I've been searching around but can't find anything on this particular topic.
I'm moving my Frigate instance to another system which will have a different IP address. I'd like to keep Frigate Proxy in place on my Home Assistant system but just change the IP address it connects to, but I can't find any way to do that. I assume it's possible if I uninstall Frigate Proxy but then I will potentially break all of my automations relying on Frigate. Is there a configuration file I can edit that will allow me to simply change the IP/URL?

Once again, I failed to notice the "Configuration" tab under Settings > Add-Ons > Frigate Proxy in Home Assistant. Problem solved.


feat: dutch translation

@mcpsbot What do you think about the folder structure for translations?

@all-contributors please add @jaronz for translation

 Translate Dutch

@all-contributors add @JaronZ for infra


The Developer’s Guide to Azure Ebook Added.

The Developer’s Guide to Azure Ebook Added.
What does this PR do?
Adding New Ebooks.
For resources
Description
The Developer’s Guide to Azure Ebook Added.
Why is this valuable (or not)
Yes. Its a good Ebook for Azure Developers.
How do we know it's really free?
It is from official microsoft.
For book lists, is it a book?
Book.
Checklist:

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

Thanks!

I dont understand this


Added tracing-opentelemetry-kafka module.

Upgrade libs versions.
Fixed use deprecated code.
Corrections for tracing-opentelemetry-grpc module.

@graemerocher I have added documentation for opentelemetry-kafka settings. I think it's time to merge these changes and release them. Because now it is impossible to use tracing normally with kafka due to incorrect library versions

Thanks for the contribution!

@graemerocher thanks! And what about release?


Updated link to dashboards developer guide

Signed-off-by: Amardeepsingh Siglani<EMAIL_ADDRESS>Description
Changes the link to the correct developer guide
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #349 (df07561) into main (6305d7f) will decrease coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #349      +/-   ##
==========================================
- Coverage   52.73%   52.68%   -0.06%
==========================================
  Files         209      207       -2
  Lines        5450     5442       -8
  Branches      763      763
==========================================
- Hits         2874     2867       -7
+ Misses       2574     2573       -1
  Partials        2        2

Impacted Files
Coverage Δ

...gins/alerting-dashboards-plugin/utils/constants.js

...plugins/alerting-dashboards-plugin/babel.config.js

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Handle resolv.h regardless of glibc version

Changes in the interface of resolv.so have historically made it hard to use this crate. I propose a change that simplifies this significantly.
First, the binding to resolv.so has to be generated on the target system. Then we need an adaptor that provides a consistent interface to that binding. That is the only part that has to be pre-prepared for each breaking change in glibc versions (as is the case in glibc 2.34). Lastly, we should use the official interface provided by resolv.h, which is enabled by the aforementioned adaptor.
These changes have made it significantly easier for me to compile this crate against multiple machines. My hope is that it may be useful to others, as well. That is why I am submitting this pull requests. I welcome all constructive criticism and suggestions for improvement.

Brilliant.

Thanks!
I had some issues with the diff against master because I originally based my branch off of branch glibc234. So I made a new branch based off of master and turned that into pull request #8 .


prune deprecated EvalResult

What does this PR do?
another pre-step for #5005
Before submitting

[x] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes? (if necessary)
[x] Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
[x] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is free to review the PR once the tests have passed.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified
[x] Check that target branch and milestone match!

Did you have fun?
Make sure you had fun coding 🙃

failing flake8 and isort are unrelated to this PR and they are fixed in #5647

failing flake8 and isort are unrelated to this PR and they are fixed in #5647


Divs inside of list items

1.
   :::

The JS implementation provides the result I expect (divs inside of list item):
https://djot.net/playground/?text=1.+Hello
+++
+++%3A%3A%3A+hi
+++inside+list%3F
+++%3A%3A%3A

+++%3A%3A%3A+hi
+++inside+list%3F
+++%3A%3A%3A&sourcepos=false
The haskell implementation does not (the second div makes the list end):
pandoc -i foo.dj -t html

Hello

A workaround is removing the empty line between the divs. Paragraphs and divs together in list items, or divs in list items followed by other list items, seem to be problematic also.

Thanks!

The wrapper="1" is also wrong on those divs, but that needs fixing elsewhere.


delegate is not called when the camera does not have permission.

When I use async/await to wrap capture, cameraController(didCancel cameraController: CameraController) delegate is not called when the camera does not have permission or the camera is unavailable .
Is it possible to consider adding this call back, or is there currently any other way to get the cancel callback?
if !UIImagePickerController.isSourceTypeAvailable(.camera) {
    bottomView.isGestureEnable = false
    view.addSubview(bottomView)
    PhotoTools.showConfirm(
        viewController: self,
        title: "相机不可用!".localized,
        message: nil,
        actionTitle: "确定".localized
    ) { [weak self] _ in
        guard let self = self else { return }
        self.dismiss(animated: true)
        //TODO: add delegate call back
    }
    return
}
AssetManager.requestCameraAccess { isGranted in
    if isGranted {
        self.setupCamera()
    }else {
        self.bottomView.isGestureEnable = false
        self.view.addSubview(self.bottomView)
        PhotoTools.showNotCameraAuthorizedAlert(
            viewController: self
        ) {
            self.dismiss(animated: true)
           //TODO: add delegate call back
        }
    }
}

The next version will solve this issue
You can fix the permissions before going to the camera interface and that should fix it


Allow disabling auto extending ActiveRecord::Base

Method names like .search can collide with existing scopes which then results in the following error:
You tried to define a scope named "search" on the model "…", but Active
Record already defined a class method with the same name.

With that setting being set to false it allows one to enable ThinkingSphinx per model instead of globally.

@pat It’s been a long time that we’ve chatted about this in the RubyNZ Slack. Anyway, I’m curious whether you’d be open to this type of feature? For the moment there are no tests as I’m testing the waters. Moreover, not entirely sure how to approach testing other than creating another test app and then testing for the absence of methods on ActiveRecord::Base.

Hi @ur5us :) Very happy to have this feature, though yes, it's a hard thing to test. If you could try it out with a test app that'd be a great place to start - not sure we can do much more than that.

Ended up implementing this in #1265, and is part of 5.6.0 which has just been released :)


Change: Add placeholder and tooltip to clarify IPAM address format

Description
Per Chris's suggestion, be a little clearer about the required format for IPAM addresses.

Looks good; I pushed up a few changes, mostly to fix the error message positioning:

Also changed the color of the tooltip in TextField which broke the StackScript page, so I made an adjustment in ImageSelect to fix that.
Since these are global changes, I'd really love a review from you @Jskobos and @tiffwong.


Call signature of tf.concat has changed in Tensorflow 1.0.0rc0

Because the call signature of tf.concat has changed to the one of the deprecated tf.concat_v2, the creation of a bidirectional RNN layer fails. This happens in either Version 1.2.1 or the latest master versions.
Example:
import keras.layers as kl

test_input = kl.Input(shape=(10, 10), dtype='int32')
test_emb = kl.Embedding(input_dim=100, output_dim=10)(test_input)
test_bgru = kl.Bidirectional(kl.GRU(50, return_sequences=True))(test_emb)

This fails with
TypeError: Expected int32, got <tensorflow.python.ops.variables.Variable object at 0x7f162824e588> of type 'Variable' instead.
since the arguments of value and axis are switched: https://www.tensorflow.org/versions/r1.0/api_docs/python/array_ops/slicing_and_joining#concat
https://www.tensorflow.org/api_docs/python/array_ops/slicing_and_joining#concat

same here.
tributeError                            Traceback (most recent call last)
/usr/local/lib/python3.5/dist-packages/keras/backend/tensorflow_backend.py in concatenate(tensors, axis)
1424         try:
-> 1425             return tf.concat_v2([to_dense(x) for x in tensors], axis)
1426         except AttributeError:
AttributeError: module 'tensorflow' has no attribute 'concat_v2'

I met the similar problem when writing code like below:
“up6 = merge([UpSampling2D(size=(2, 2))(conv5), conv4], mode='concat', concat_axis=1)”
Seems that there are lots of api changes in new version of tensorflow according to:
https://groups.google.com/a/tensorflow.org/forum/#!msg/discuss/OePXmC9kJ7o/SRErOoYCDQAJ

Keras 2 is coming soon, so you should use that. It's developed for TF 1.0.
On 11 March 2017 at 18:41, qjin<EMAIL_ADDRESS>wrote:

I met the similar problem when writing code like below:
“up6 = merge([UpSampling2D(size=(2, 2))(conv5), conv4], mode='concat',
concat_axis=1)”
Seems that there are lots of api changes in new version of tensorflow
according to:
https://groups.google.com/a/tensorflow.org/forum/#!msg/
discuss/OePXmC9kJ7o/SRErOoYCDQAJ
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fchollet/keras/issues/5238#issuecomment-285917235,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AArWb6e4MJPvFPm4thRpzH7A84nK79WYks5rk1tQgaJpZM4Lzo0n
.

Thanks for reply. I had planed to make a pull request. But I found the fix for my problem is already in the newest code.
Looking forward to embrace Keras 2 :)


types2: missing error for invalid new call

types2 doesn't report an error for
package p
func _() {
	_ = new(1 + 1)
}

(go/types appears to work correctly.)

See also disabled test at end of $GOROOT/test/used.go.

This was fixed with https://golang.org/cl/277074 in the dev.typeparams branch.


feat: Use istanbul-lib-processinfo

Coverage increased (+0.4%) to 97.21% when pulling 763bd514a270f9aab2c502d01527c3f7f94f45f4 on coreyfarrell:use-processinfo into 00c3b3440a5b2ffe11b9c19ae4e08ad2f5b70e33 on istanbuljs:master.

Looks good to me!


Migrate to null safety

I've used the migrate tool to migrate the code to null safety. All tests are passing.

The truth function cannot return null. Is it possible to remove the '?' for the bool return value for this function.

Done! All tests passing.

Thanks a lot @fermuch


Scissors - Aida R.

Great work on this project! Great code re-use and helper functions! Nice work with the reset extension and the UI changes. This project is green.


The component does not work in Vaadin 23

The overlay is not closed correctly.
cancel Popup should call the event with bubbles true.
  _cancelPopUp(e) {
    e.target.dispatchEvent(new CustomEvent('_closeColorPickerPopUp', {bubbles: true}));
  }

@appreciated if I make a Pull request, would you have the time to release the component? I will try to bump the version and migrate this component with minimal changes.

I started to migrate the color-picker component to Vaadin 23: https://github.com/jcgueriaud1/color-picker/tree/v23
I have some dependency issues when I'm trying to use it in the color-picker-field.
The dev dependencies have not been updated to Vaadin 23. It's probably easier to get rid of the demo helpers:
"@vaadin/vaadin-component-dev-dependencies": "^3.0.0",
"@vaadin/vaadin-demo-helpers": "^3.1.0",


bug Fix method invoking (#26131)

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

#26131


Fetch resource variable fix

The test added in #261 was using a malformed protobuf as TF functions have to have at least one input. This caused the test to always fail. This PR fixes the python generation function, along with the protobuf file and the test itself. The python file was slightly reformatted to make it a little more PEP 8 friendly.
I also fixed a few small Javadoc issues in the hand written code of tensorflow-core-api, presumably the generated ones will be fixed by the PR that rewrites that.
cc @rnett

Odd, I ran them locally just fine.  Something may have been cached somewhere, but I'm pretty sure I cleaned first.

The TF function code barfed on it because it didn't have any inputs. I'm not sure what it actually wrote out, but the TF native library did not like it. Maybe you've got an older version of Python tf which does something different? I tried it on 2.4.1 and it did not like the protobuf that was there.

Yeah, I've got 2.3.1 in Python, but the Java library is still using 2.4.1.  Odd, but whatever, it works now.


Fix ctr push image can't show right rate

Relate to the https://github.com/containerd/nerdctl/issues/2279,
The ctr has the same issue.
Issue Description
I push the local image to harbor with ctr  , and ctr nerdctl shows the wrong upload rate .

The command line show the rate of upload is 374 B/s, But the actual upload speed is much greater.
Issue Fixed
The issue is because of layer upload process bar is skipped. So the PR is to fix it. And If the layer exists in the registry, the upload size is 0.

After the Fix the rate of upload is 318 KB/s, It's correct.
The PR can also fix the nerctl push

Hi @Iceber
Would you please help to review the PR

Hi @estesp
Would you please help to review it,  Thank you very much.

HI @dmcgowan
Would you please review the PR :-)
Thank you very much

Looks correct to me, @cpuguy83 can still take a look to see if another solution would be better. Merging this one

Thanks @kzys @Iceber  @dmcgowan for the PR review :-)


Where are linux driver ?

Hi all,
I have a Neje Master.
I really appreciate this project; it compiles, it starts but I am looking for Linux driver and can't find it.
Can anyone help me ?

Take a look here: https://wiki.nejetool.com/

Many thanks shantarsecurity, but I am looking for linux driver there, without success.

Linux driver? What driver?

When you plug it in it will register as QinHeng Electronics CH340 serial converter when you type lsusb.
You might see a different serial converter depending on the model of the engraver. Just run lsusb before and after plugging it in to see the device.
If it is CH340 or CH341 run lsmod | grep ch34 and see if anything is listed. If not then you need to install the driver. You can find drivers here: https://learn.sparkfun.com/tutorials/how-to-install-ch340-drivers/all#linux

I have neje master 2 2020 and have this same trouble.
software wrote:
connections sucessfuly but nothing happened, I cand moving head and nothing send to engraver


How long does the cache expire

How long is the cache ?
how to set

Docs says:

A repository can have up to 5GB of caches. Once the 5GB limit is reached, older caches will be evicted based on when the cache was last accessed. Caches that are not accessed within the last week will also be evicted.

Can someone close this?

Docs says:

A repository can have up to 5GB of caches. Once the 5GB limit is reached, older caches will be evicted based on when the cache was last accessed. Caches that are not accessed within the last week will also be evicted.

@cordova2009 Size is increased to 10GB. Rest remains same. Closing this issue, you may reopen if you seek more clarity


Avoid using unserialize when caching NULL default values

is_scalar should just be an inverted is_object check, instead.

Hi @danog, thank you! As this PR comes from an organization I don't have write access on it, could you rebase your branch on master please? 🙂

Thanks @danog!


[Button] Unable to align Button alignment to the middle and to the right

Docs state that you can vertically align your content within columns by using valign attribute, but it doesn't work with buttons. Additionally, I'm not able to align buttons to the right.
Besides the normal flow, the only option to position a button is by using the center tag.

Yeah, the button was somehow forgotten. There's already a PR merged into Inky (https://github.com/zurb/inky/commit/11805b4d90987361f4e009cb26714e8c37d4f5ec), but the 2.2.2 Foundation for Emails release is pending, so you won't get it through an npm update or install.
Not sure what the plans are for the next release, but I hope it'll be soon, as there were many, very useful PRs merged over the past month or so.

The valign attribute goes on the container wrapping the content - like the column. You need 2.1+ to get this.
Working example:
<row>
    <columns small="6" large="3" valign="middle">
      <button href="#" class="align-right">Button</button>
    </columns>

CSS
table.button.align-right {
  display: inline-block;
  float: right;
}

2.2 is set to release 11/10. You can follow releases here: https://github.com/zurb/foundation-emails/wiki/Roadmap
Since https://github.com/zurb/inky/pull/61 is in that resolves the 2nd part of your question, we'll close this here.


[analyzer] Tests for Cppcheck standard handling

Analyzer tests were missing for the cppcheck standard forwarding. The standard translation is also tested in case of a gnu** standard is given.
Also added a small change in the business logic for handling the standard parameter. This should make it more robust against future refactoring.

Updated the title & message.


Solidity support revisited

Fixes #1404  . Revised PR from #1406

@mAAdhaTTah
This language would undoubtedly benefit from #1679. Just in case you weren't convinced already 😉

Any plan to merge this PR? Is the last blocker to migrate from highlight.js to Prism for us.

Since the author never got to make the requested changes, I can make the required changes within the next few days.


Design for target judging system

With help from @prattmic & @jemoster.
Need interface for judging target data (imagery mostly) submitted by teams, and a way to auto-grade the data (e.g. maximum matching).

Design sent for review.


Need ability to override ServerChangeScene function in NetworkManager

Please explain the suggested feature in detail.
This function is already marked public virtual void, however it contains references to two private variables that prevent us from fully overriding this in a base class.

loadingSceneAsync
startPositionIndex

How exactly does this keep you from releasing your game right now?
We would like to override this function to send a custom SceneMessage to the clients.  Since this happens after loadingSceneAsync is created and before startPositionIndex is set, we cannot do this in a derived class.  Additionally if we wanted to implement custom spawning code this would be impossible because startPositionIndex is private.
Can you suggest a possible solution?

Make loadingSceneAsync a protected member.
Make startPositionIndex a protected member.

Additional context
Generally a public virtual function should probably not have references to private member variables.  It makes it problematic to override.

There's some work in this area in progress that will make this better, but generally ServerChangeScene should really only be used when you're actually wanting the server and all clients to do a full scene change, not for additives.  I know the signatures suggest otherwise, but that's the reality.  When some work gets done, to tell a client to load an additive scene will not use these methods, but rather NetworkServer.Send or SendToClient with a SceneMessage you create.

Understood - however I am not using this for additive scenes.  I actually do want everyone (server and all clients) to change scenes at the same time, and I need to pass a custom message as well.  I am glad to hear that this is being worked on.

Understood - however I am not using this for additive scenes. I actually do want everyone (server and all clients) to change scenes at the same time, and I need to pass a custom message as well. I am glad to hear that this is being worked on.

Stop by our Discord sometime and ping me to discuss your needs.

:tada: This issue has been resolved in version 6.5.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[ci] update lockfile

This PR is auto-generated by a nightly GitHub action.  It should automatically be merged if tests pass. If you want to update manually: 1. npm install -g npm-check-updates (first time only) 2. ncu -u 3. npm install

closing in order to try to rebuild


Encryption support for the statement store

This implements simple encryption scheme that uses ed25519 key exchange and AEAD for the statement store. See https://github.com/paritytech/labs/issues/3 and https://github.com/paritytech/substrate/issues/13893
cumulus companion: https://github.com/paritytech/cumulus/pull/2876

@bkchr Changed to use local keystore.
There are some additional changes introduced with cargo +nightly fmt. They seem harmless.

bot merge


Fix ips css

Closes #672

Coverage remained the same at 83.284% when pulling 3c0b820e95c96a704ef17d35862e968405db0e34 on eatonphil:fix-ips-css into 556fed4e64a971fcae9e2bf41536aa8b5a0e3651 on linode:master.

Coverage remained the same at 83.284% when pulling 57291d417ea5e54cd486d70aeb4e84b5333d3b04 on eatonphil:fix-ips-css into 556fed4e64a971fcae9e2bf41536aa8b5a0e3651 on linode:master.


register root command

https://github.com/octomation/go-service/blob/60004a28c6bbddb146b28bcba01c448b90dc0209/cmd/server/main.go#L44-L46

invalid issue, root is already what I want