Correction for raw affiliation ENS de Lyon, CNRS UMR 5239, Laboratory of Biology and Modelling of the Cell

Correction needed for raw affiliation ENS de Lyon, CNRS UMR 5239, Laboratory of Biology and Modelling of the Cell
raw_affiliation_name: ENS de Lyon, CNRS UMR 5239, Laboratory of Biology and Modelling of the Cell
new_rors: 029brtt94;02feahw73;04zmssz18;01bj4fd12
previous_rors: 029brtt94;02feahw73;04zmssz18
works_examples: W4308707013
contact: eac75d48ea2d82fcd5ff458ea46796d4:255a0cdbd63c6fbc7eb08e6e9fa1 @ ens-lyon.fr

This issue was accepted and ingested by the OpenAlex team on 2024-12-21. The new affiliations should be visible within the next 7 days.


Plan#show fails on customized template

customization_of is being treated as a relationship when it isn't so
customization_of.org fails in _plan_details.html.erb

fixed by https://github.com/DigitalCurationCentre/roadmap/commit/e73a0fa47db69b243731c2d6a94ace918c58e836

Not sure if there's anything @stephaniesimms or I can test on this one so good to close?

Not sure how to test either so closing


H

Hi @hapymaan, closing this issue since it seems like it was opened by mistake. We require a bit more information then this ;)


[Feature] Support show create external database

What type of PR is this:

[ ] BugFix
[x] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes:

Fixes #
Problem Summary(Required):

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto backported to target branch

[ ] 3.0
[ ] 2.5
[ ] 2.4
[ ] 2.3

[FE PR Coverage Check]
:disappointed: fail : 8 / 14 (57.14%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/qe/ShowExecutor.java
8
14
57.14%
[959, 960, 961, 962, 963, 964]

[FE PR Coverage Check]
:disappointed: fail : 8 / 14 (57.14%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/qe/ShowExecutor.java
8
14
57.14%
[959, 960, 961, 962, 963, 964]

[FE PR Coverage Check]
:heart_eyes: pass : 8 / 10 (80.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/qe/ShowExecutor.java
8
10
80.00%
[957, 964]


Ensuring a PR cannot be assigned to the creator

I checked in repositories with not too many people, the probability to assign the PR to the creator is too high. Adding a way to exclude the author by using the allowSelfAssign implementation and adding a new condition in the if flow remove the author from the asignees list.
On the other hand, fixing the payload when requesting the POST action to assign reviewers. A false positive was being raised in the GH result but after testing with postman it worked.
The tests were adapted too.

Good contrib, thanks once more @agomezmoron


Echo Noodles 340.php is awarding a bonus of 1 but no bonus is visible

Echo Noodles 340.php is awarding a bonus of 1 but no bonus is visible

There is just a render bug with all 1 bonuses. There's an option issue for it.


completion/dotnet: new completion

Description
This sets up dotnet to complete itself according to vendor documentation.
This closes #1983.
Motivation and Context
How Has This Been Tested?
I've tested this, and alsö opened an issue upstream to improve their example: dotnet/docs#27828.
Types of changes

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the code style of this project.
[x] If my change requires a change to the documentation, I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] If I have added a new file, I also added it to clean_files.txt and formatted it using lint_clean_files.sh.
[ ] I have added tests to cover my changes, and all the new and existing tests pass.

Looks slightly different (and likely better) than the referenced script  ...
Maybe @EmilySeville7cfg can give it a try and sign off on it?

https://github.com/dotnet/docs/issues/27828

Good job ;)


Not able to sort by nullity

Neither with msno.nullity_sort nor with msno.bar, I'm not able to sort the columns by nullity.
In both functions I try with sort='ascending'.  It doesn't throw me errors, but the plot doesn't change.

Nullity sort is working, actually; but I think you're misinterpreting what it's supposed to do.
msno.nullity_sort resorts data based on the number of null observations in the column. In other words, it is a row-oriented nullity sort. The ascending sort does this in the forward direction:
print(
    msno.nullity_sort(pd.DataFrame({'A': [1, 2, 3, 4], 'B': [np.nan, 2, 3, 4]}),
    sort='ascending')
)

   A    B
0  1  NaN
1  2  2.0
2  3  3.0
3  4  4.0

descending sort does this in the reverse direction:
print(msno.nullity_sort(
    pd.DataFrame({'A': [1, 2, 3, 4], 'B': [np.nan, 2, 3, 4]}), sort='descending')
)

   A    B
3  4  4.0
2  3  3.0
1  2  2.0
0  1  NaN

While sorting the data has an important and highly visible effect on msno.matrix, it does nothing when applied to data passed to msno.bar. The only value that matters in msno.bar is the per-column nullity count, which is unaffected by the data sort order (or lack thereof).
However there is another form of sorting that makes sense in the msno.bar column, which is sorting on the per-column nullity (which I think is what you are asking for). It wouldn't be too hard to implement this, and it would certainly be better than the current sort parameter on msno.bar, which, well, doesn't do anything.

Addressed in #87.

yes, you were right. I misinterpreted what it supposed to do. Thanks for #87 update!


Navbar store specific, fixing BE routes for billboards, categories and products and fixing frontend data fetching and bug in unauthenticated user fetching products

Pull Request: Dedicated Store Navigation and Backend Fixes
Description

Created a dedicated navigation bar for every store to improve usability and provide a store-specific experience.
Fixed backend routes for creating, updating, and deleting billboards, categories, and products to ensure proper functioning.
Refactored frontend and data fetching logic to align with the updated backend routes.
Fixed server action routes responsible for fetching data for users.
Thoroughly tested all changes to ensure everything works as expected.

Changes

Store-Specific Navigation Bar:

Added a new StoreMainNav component for each store.

Backend Route Fixes:

Corrected the logic in routes for handling billboards, categories, and products.
Ensured data consistency across the CRUD operations.

Frontend Data Fetching:

Updated components to fetch the latest data based on the store context.
Fixed any issues related to missing or incorrect data rendering.

Server Action Routes:

Reviewed and fixed issues with server-side functions fetching data for users.
Validated session handling to ensure smooth user interaction.

Testing

Verified the correctness of backend routes through manual testing of CRUD operations.
Tested navigation across different stores and ensured the correct display of data.
Checked for regressions or unexpected behavior in server-side data fetching.

Checklist

[x] Store-specific navigation bar implemented.
[x] Backend routes fixed for billboards, categories, and products.
[x] Frontend data fetching aligned with new routes.
[x] Server action routes verified and corrected.
[x] All changes tested and working as expected.

Related Issues
(Mention any related issues or tasks, if applicable)

Reviewers: Please review the code and provide feedback if needed.

Related Issues

Closes #1034

Screenshots / videos (if applicable)

@ShivanshPlays resolve your branch that I can review and merge accordingly.

@mdazfar2 please check


Widget to display content

An easy way to add generic widgets

@zzuga this looks the same as the LeeanPreview widget to me...

If only description was a textarea in LeeanPreview, I could use it.
Shall I change that instead of adding a new one? do you think its safe?

Yes, I think we could make it a textarea without breaking existing widgets

Then its much simpler, thanks!


jsconfig.json

Hi @robgietema,
I noticed you added recently a jsconfig.json file in the package:
https://github.com/plone/plone-react/blob/master/jsconfig.json
What are you using it for? Is it used in your build?
I am asking because it is an equivalent of our tsconfig.json in TypeScript which allows us to set custom paths to dependencies and we are currently building a mr.developer-like tool (https://github.com/collective/ng-mr-developer/tree/eric-changes) to improve the developer experience when working with several checkouts in a project.
I would be glad to adapt ng-mr-developer so it can be used in Plone React projects too.

@ebrehault that would be awesome! We have a config.json in our large Angular 2 project that is used at build time to pass variables to Webpack to apply custom themes and other stuff. We also have a config.json that our app loads at run-time for customizations that we need at run-time. @sneridagh can tell more about this.
In any case, a JS mr.developer is definitely what we need. And we should try to keep Angular and React close so we can use tools together.
Just one small recommendation, when we make this a generic tool: can we please get rid of the "ng"? :)
We could also move this to the Plone organization. I think this is essential infrastructure for our future Plone versions.

I was already considering renaming ts-mr-developer (because it works in any TypeScript-based project), but if it even works with ES6-based projects, then we need something else. Maybe js-mr-developer.

A mr-developer would be awesome! As far as I know the jsconfig.json is used only for ide's not in the build system.

@ebrehault That sounds awesome!!
+1 to standardize it.
jsconfig.json is used by vscode:
https://code.visualstudio.com/docs/languages/jsconfig
If not set, it complains about JS syntax etc... It has the same purpose as the tsconfig.json file (but not used by the build/tooling).

@sneridagh, our mr-developer https://www.npmjs.com/package/mr-developer is working quite nicely now.
I am wondering if the best way to use it in a React project could be to set the resolve.alias property https://webpack.js.org/configuration/resolve/#resolve-alias in webpack.config.dev.js.
But I am not super confortable with the idea of changing dynamically a JS file.
Maybe webpack.config.dev.js could compute its alias value by reading the tsconfig.json's compilerOptions.paths attribute? What do you think?

@sneridagh so let's say you want to use a checkout of draft-js-buttons, you have to create a mr.developer.json like this:
{
    "draft-js-buttons": {
        "path": "draft-js-buttons/src",
        "url": "https://github.com/draft-js-plugins/draft-js-plugins.git"
    }
}

Then you do:
$ mrdeveloper --config=jsconfig.json

And to override the draft-js-buttons package node_modules path, I think you can do something like:
const pathsConfig = require('./jsconfig').compilerOptions.paths;
const alias = {};
Object.keys(pathsConfig).forEach(package => {
  alias[package] = pathsConfig[package][0];
});

I tried it, but I do not know where it should be done in your webpack jungle :)

Implemented


Add DESTDIR support to Makefile

Please support the use of the DESTDIR variable in your Makefile.

I did not know that was a convention. Does the existing PREFIX variable meet your needs?
The current default is PREFIX = /usr/local.
Happy for a PR that makes it place nicer

No, DESTDIR is different from PREFIX.


can't able to ssh

yeah... seem it broke in the last upgrade. I need to investigate what happened :S

should be fixed. Can you try again and close if it works? thanks!

awesome thanks.
Thank you very much,


Create pep8_compliance.yml

Adds an action to automatically run autopep8 and submit the formatted results as a merge request.

I couldn't quite get this to work.

I couldn't quite get this to work.


bitstream generation for custom block

Hello,
im now working on the bitstream generation for the new custom block, for example blackbox.
According to your paper TGA+19, i generate a physical mode and an operating mode, both have the module adder(because there's only one module: adder in the blackbox). Afterwards i combine them in the openfpga-arc.
But the issue is that the bitstream isn't changed. It still only has "clb" block and "swich" block, what means the bitstream for blackbox isn't generated.
can you please give me some advice about that?
k4_N4_.zip

您好，
实在不好意思，大佬，可不可以用中文说一下。
是这样的，我们学校自己有块板子，并且想把板子移植到你的框架上来。但是我们学校板子上有很多新的逻辑器件逻辑功能，所以想先通过您的框架生成一个一个带有新的逻辑功能的新的模块，以此先尝试一下，查看这是否是可行的。我看了您的例子
，k6_frac_N10_tileable_adder_chain_mem16K_40nm.xml 这边您创建了一个新的bram模块但
是我发现在bitstream那一块并没有为bram那一块生成，而且在routing那一块我也只看到了clb那一部分，并没有找到bram那一部分。所以我想请问一下，bitstream可不可以为除了clb以外的其他板块生成，并且有没有可能也为新的模块布线？
tangxifan<EMAIL_ADDRESS>于2020年9月4日周五 上午1:10写道：

Hello,
In your VPR architecture file, you defined the pb_type blackbox at the
same level as clb, it means the blackbox will be an physical tile but not
an operating mode of clb. In your blackbox, it is 100% hard logic. There
is no LUT and programmable routing. As a result, you see no changes in the
bitstream.
I would suggest you to look into our multi-mode examples at
k6_frac_N10_tileable_adder_chain_40nm
https://github.com/LNIS-Projects/OpenFPGA/blob/master/openfpga_flow/vpr_arch/k6_frac_N10_tileable_adder_chain_40nm.xml
In this directory, you can find around 10 examples of multi-mode CLBs,
BRAMs and DSPs.
Also in our regression tests, you can easy run these examples
https://github.com/LNIS-Projects/OpenFPGA/blob/master/openfpga_flow/tasks/fpga_verilog/hard_adder/config/task.conf
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/LNIS-Projects/OpenFPGA/issues/82#issuecomment-686809365,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AP6XREZS67DXJNLJYCMSIRDSEAPADANCNFSM4QWLXTBQ
.

你好，<EMAIL_ADDRESS>


Details page toolbars

Added initial export feature
Added utils
Refactored how title prop is localized
Added translations for details table toolbar
Added data toolbar
Added detailsReportItem to support group by params
Added DetailsHeaderToolbar
Renamed DetailsToolbar
Updated details page
Refactored overview page loading component

Codecov Report

Merging #142 (a6d9b86) into main (623be36) will increase coverage by 0.22%.
The diff coverage is 9.11%.

Additional details and impacted files

@@           Coverage Diff            @@
##            main    #142      +/-   ##
========================================
+ Coverage   6.50%   6.73%   +0.22%
========================================
  Files         36      41       +5
  Lines       1153    1471     +318
  Branches     178     232      +54
========================================
+ Hits          75      99      +24
- Misses      1078    1372     +294

Impacted Files
Coverage Δ

src/routes/components/data-toolbar/DataToolbar.tsx
0.00% <0.00%> (ø)

src/routes/components/export/exportModal.tsx
0.00% <0.00%> (ø)

src/routes/components/export/exportSubmit.tsx
0.00% <0.00%> (ø)

src/routes/components/perspective/Perspective.tsx
0.00% <0.00%> (ø)

src/routes/details/Details.tsx
0.00% <0.00%> (ø)

src/routes/details/DetailsHeader.tsx
0.00% <0.00%> (ø)

src/routes/details/DetailsHeaderToolbar.tsx
0.00% <0.00%> (ø)

src/routes/details/TableToolbar.tsx
0.00% <0.00%> (ø)

src/routes/overview/Overview.tsx
0.00% <0.00%> (ø)

...tual-spend-breakdown/ActualSpendBreakdownChart.tsx
0.00% <0.00%> (ø)

... and 4 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 623be36...a6d9b86. Read the comment docs.


Enable map_location in load_from_checkpoint when loading a GPU checkpoint on a CPU-only machine

Commit: Changed savings so that legacy pl code does not have issues mapping from cuda ->cpu
What does this PR do?
Fixes #17974
As discussed in Issue 17974 changing the loading option by checking form map_lacation variable
Checking the map_location in _load_form_checkpoint
This hides another layer of complexity from the user when saving and loading models.
No new dependecies

Before submitting

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

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

Reviewer checklist

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

@awaelchli Sorry for taking so long to create the PR.
I hope these minimal invasive changes to the code are working well. If you have any comments on how to contribute, I would love to hear them.
Thank you for your time and great work.

@marcimarc1 I realized that my initial suggestion https://github.com/Lightning-AI/lightning/issues/17974#issuecomment-1621528350 wasn't going to work. The logic in the PR would work when dealing with a checkpoint from a GPU, but it would cause other checkpoints to load on the CPU as well (e.g. XLA, MPS). So I allowed myself to rework the logic a bit and also wrote a test.


Cherry-pick Implementation Changes for azure-search-documents Patch Release

Nice you want to put static logger changes in the patch. I should have thought of this too! Also, I noticed you created your own branch. Any particular reason why you did not  use the mass generated one? Was the name confusing?

API changes have been detected in com.azure:azure-search-documents. You can review API changes here

API changes have been detected in com.azure:azure-search-documents. You can review API changes here

This always confuses me. Do you know why we diff against the latest available version vs latest available GA version.


Travel method buttons components

Please review the component for travelMethodButtons in  components/TravelMethodButtons/TravelMethodButtons.jsx
Trello link :  https://trello.com/c/z50v3GrA/297-dev-travel-method-selection-page
New commit 7

Import the 'modeOfTransport' and 'transportIcons' directly travelMethodButtons component.

There no reason to call it as a props .

I will remove transport Icon as props. It doesn’t parent component doesn’t
need this. I will use import  them inside the button component .
On Wed, 9 Mar 2022 at 9:55 am, ymloh @.***> wrote:

@.**** commented on this pull request.
In components/TravelMethodButtons/TravelMethodButtons.jsx
https://github.com/CodeforAustralia/council-emissions-calculator/pull/68#discussion_r822143716
:

transportIcon,

modesOfTransport,

handleTransportMode,

transportMode,

@hqtan https://github.com/hqtan handleTransportMode - I forget how data
flows in React 😅 Are we able to extract selected transport modes from
outside the component without a passing callback to it? If not then maybe
either passing the handler in or at least a callback that an internal
handler can call could be useful
—
Reply to this email directly, view it on GitHub
https://github.com/CodeforAustralia/council-emissions-calculator/pull/68#discussion_r822143716,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AP724Q4RFYXQF5N6CYMUPDTU67LG5ANCNFSM5QFAZZEA
.
You are receiving this because you were mentioned.Message ID:
<CodeforAustralia/council-emissions-calculator/pull/68/review/903681350@
github.com>

Hi @Amutha37 ,
I'm going to close this PR, in favour of #70 .
I've compared both PRs, and came to the conclusion that all the significant changes in this PR are in #70 .
Please let me know if I've erred in my judgement and many apologies in advance. We can always re-open this PR.

yes you can actually. By doing this the oder of PR review is not necessary
.
On Thu, 10 Mar 2022 at 11:03 pm, hqtan @.***> wrote:

Hi @Amutha37 https://github.com/Amutha37 ,
I'm going to close this PR, in favour of #70
https://github.com/CodeforAustralia/council-emissions-calculator/pull/70
.
I've compared both PRs, and came to the conclusion that all the
significant changes in this PR are in #70
https://github.com/CodeforAustralia/council-emissions-calculator/pull/70
.
Please let me know if I've erred in my judgement and many apologies in
advance. We can always re-open this PR.
—
Reply to this email directly, view it on GitHub
https://github.com/CodeforAustralia/council-emissions-calculator/pull/68#issuecomment-1063982775,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKV7DNEIIN355DR2VENZ43LU7HQIPANCNFSM5QFAZZEA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***
com>


Does not work inside ShadowRoot

The method isElementExist uses document.contains to check if the target element exists.
However, if Zoomist is run inside ShadowRoot, the target element is not inside document. Thus, the check fails and raises immediately an error.
It would be nice, if the check could be adjusted to allow usage inside ShadowRoot.
Thank you and keep up the good work! :-)

@marvinscharle
Thanks for your suggestion! I'll fix it when I have some free time.


Add configuration for completion/signature popups

Note: this PR is into #640.
https://user-images.githubusercontent.com/17034772/140941617-09cb903a-8052-4915-8488-2bf481f3fb06.mp4
@josevalim any preference on the defaults? Now that we insert parentheses I think showing completion list makes sense and it should give a better experience when starting with Livebook/Elixir. As for signature help, it may be too much, but I'm not sure :)

For me, the biggest issue is not showing the signature, but showing the signature with docs. If we could customize it so it only shows the signature on parens and showing the docs only on mouse over or on cmd+space, then I think we would be good. Is that an option?

As discussed offline, I excluded the docs and enabled signature popup by default:
https://user-images.githubusercontent.com/17034772/140950427-773aeca3-c6f6-4c6a-9f09-c93c80c1099f.mp4

We are not sending the docs from the server either, right?
Also, if the user changes the settings, should we show a small notification saying current sessions have to be refreshed for it to apply?

Also, let’s break the settings in two: System Settings and User Settings. They can be different headings on the same page. Alternatively we can move this configuration to the user avatar. Which one do you prefer?

We are not sending the docs from the server either, right?

I think it's fine to keep it in the runtime intellisense protocol, but I strip docs from the response here, so we don't send any.

Also, if the user changes the settings, should we show a small notification saying current sessions have to be refreshed for it to apply?

If the user navigates with the same window it will work as expected and it doesn't change often, so I think it's fine. If we tell them to refresh they may end up refreshing the settings page, which is not necessary.

Also, let’s break the settings in two: System Settings and User Settings. They can be different headings on the same page. Alternatively we can move this configuration to the user avatar. Which one do you prefer?

User avatar is different, I'd avoid packing it with other config options, the setting page seems more appropriate. By dividing the sections do you mean we should have a hierarchy like h1 Settings h2 System settings h3 File systems?

H1 system settings, h1 user settings. Since one of them only affects the current user.

Something like this?

Yes, except I would have the test to say this: "The configuration in this section changes only your Livebook experience and is saved to your browser."

Sounds better!


Autolinking broken on jekyll 3.0.0.pre.beta8

First reported at https://github.com/UniversalDependencies/docs/issues/211 .
When _site is generated with jekyll 3.0.0.pre.beta8, lib/local/collections.js ends up with an empty listing of collection entries:
var listing = {
    "": [
    ],
    "": [
    ],
};

This may have to do with the new incremental build feature in jekyll 3.0.

Running jekyll with  --full-rebuild does not help. Also, iteration over collections seems to be similarly broken for index.md, where
{% for i in site.collections %}{% assign c = i[1] %}
* Collection: {{ c.label }}
  {% for d in c.docs %}
  * [{{ d.title }}]({{ d.url | remove_first:'/' }}): {{ d.shortdef }}
  {% endfor %}
{% endfor %}

gives

Collection:

Collection:

i.e. collections with neither labels nor documents.

changing the above loop from
{% for i in site.collections %}{% assign c = i[1] %}
...
{% endfor %}

to
{% for c in site.collections %}
...
{% endfor %}

appears to fix the issue. So, it seems site.collections is now a list instead of a hash.
Possibly related: https://github.com/jekyll/jekyll/issues/2346


Revised fontconfig pinning.

This revises the intermediate fontconfig pinning as discussed in https://github.com/conda-forge/sensormapgeo-feedstock/pull/13.

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


How does the index system work?

Could you expand your description of the index system in the readme or wiki?
Looks like you store all indexes in the_index bucket and then each key in the index has a gobencoded slice of []byte values that are the actual references back to whatever objects are indexed.
I don't quite understand the lookup code other than it supports gt, ge, & eq operations.

I think I'm going to write up a more detailed blog post on all of this later, but here's the gist:
Lets say you have a Type called KeyColor and data like this:

Key
Color

1
Blue

2
Red

3
Blue

4
Blue

5
Red

If you add an index on Color, Bolthold will create a bucket called _index:KeyColor:Color per this.
The contents of that index will look like this:

Key
Value

Blue
[1, 3, 4]

Red
[2, 5]

When Bolthold runs a query, it loops through all the records using an iterator and checks if the current record matches the query.
for k, v := iter.Next(); k != nil; k, v = iter.Next() {

In a non indexed query, the iterator simply returns all of the keys in the store, and BoltHold needs to loop through ALL the records to see if any of them match the query's criteria.  So in the dataset above, it'll need to loop 5 times without an index.
In an indexed query, the iterator instead returns the values in the index (which is a list of keys) for each index that matches the index criteria.  So if the query criteria was .Where("Color").Eq("Red").Index("Color") BoldHold only needs to loop through and test records 2 and 5.
The piece of code you linked to above (https://github.com/timshannon/bolthold/blob/master/index.go#L295) is simply a short cut, so we don't need to cycle through all of the keys.  If we know the starting range of the key, we can simply seek directly to a starting key value.  For example; if you have a set of records with keys that range from 1 to 100, and your query is where the key is >= 87, you don't need to check the first 86 records, you can skip right to 87.


Issues on ui5 bootstrap

OpenUI5 version:
1.80.0
Questions:

Why ui5 would download the library-preload.js, library.css and library-parameters.json?
Can it be avoided? Coz this will require too much to download and also cause the latency of the app performance?
Can I customize the size of a ui5 module from  the server?

For example, if I used 10 controls in sap.m, I just want these 10 to get downloaded instead the whole sap.m
I tried the data-sap-ui-preload="async", but it would download all controls used one by one. This would cause a number of requests that would finally impact the performance.
Thanks a lot
Victor Ye

Hello @Victor716,
Thank you for sharing this finding. I've created an internal incident<PHONE_NUMBER>. The status of the issue will be updated here in GitHub.
Best Regards,
Florian

Glad to see your response. Looking forward to your good news.

Hey Victor, Answering your questions:

Why ui5 would download the library-preload.js, library.css and library-parameters.json?

A: For every UI5 library we provide a "library-preload.js" bundle containing (almost) all  modules of that library. The idea is that in a shared environment with multiple UI5 applications these bundles can be used by all of them and therefore highly benefit from a browsers caching mechanism.

Can it be avoided? Coz this will require too much to download and also cause the latency of the app performance?

A: If you have a standalone deployment scenario you can use the self-contained build of the UI5 Tooling as I mentioned in your other issue (I know your comment here predates that): https://github.com/SAP/ui5-tooling/issues/268#issuecomment-717113840

Can I customize the size of a ui5 module from the server?

What do you mean exactly?

For example, if I used 10 controls in sap.m, I just want these 10 to get downloaded instead the whole sap.m
I tried the data-sap-ui-preload="async", but it would download all controls used one by one. This would cause a number of requests that would finally impact the performance.

I think your application's manifest.json is missing a library dependency to sap.m. Therefore the library-preload.js is not loaded and all controls are requested individually. Please have a look at the "Use manifest.json Instead of the Bootstrap to Define Dependencies" chapter of our performance guide: https://ui5.sap.com/#/topic/408b40efed3c416681e1bd8cdd8910d4


viz_3d from hloc.utils does not work in demo.ipynb and returns the black region.

Thank you for your marvelous work!
However,
viz_3d from hloc.utils does not work in demo.ipynb and returns the black region.
The screenshots:
https://github.com/BenoitKAO/for_testing_1/blob/main/螢幕快照 2023-01-07 15-21-17.png
https://github.com/BenoitKAO/for_testing_1/blob/main/螢幕快照 2023-01-07 15-23-05.png
Any help is appreciated.

Same issue. Particularly on .plot_reconstruction.

Check out this issue: https://github.com/cvg/Hierarchical-Localization/issues/215


[FEAT]: Implement Backend API for Squeeze Page - Backend

Description:
Develop the API endpoint for the Squeeze Page.
Purpose:
Provides a backend service that allows us collect the user email, store in the database and send a template to the user.
Requirements:

[x] Develop a RESTful API endpoint to handle POST requests.
[x] Validate and verify incoming user data.
[x] Store user information securely in the database.
[x] Return appropriate response.
[x] Handle errors and return appropriate status codes.
[x] Send template to a specified email address using an email service (e.g., SendGrid, SMTP).

Acceptance Criteria:

The page should be accessible at /api/v1/squeeze.
The endpoint should accept HTTP POST requests.

Squeeze Form Submission:

Endpoint: POST /api/v1/squeeze
Request Body:

{
  "email"<EMAIL_ADDRESS>  "first_name": "John",
  "last_name": "Doe",
  "phone": "08098761234",
  "location": "Lagos, Nigeria",
  "job_title": "Software Engineer",
  "company": "X-Corp",
  "interests": ["Web Development", "Cloud Computing"],
  "referral_source": "LinkedIn"
}

Validation: Ensure all fields are required and the email is in the correct format.
Response (Success):

{
  "message": "Your request has been received. You will get a template shortly."
}

Response (Error):

{
  "message": "Failed to submit your request",
  "status_code": 400
}

Tasks:

[x] Create an endpoint (POST /api/v1/squeeze) to handle contact form submissions.
[x] Validate user input on the server side.
[x] Securely store user information in the database.
[x] Implement error handling and return appropriate status codes.
[x] Write unit tests for contact form submission and data validation.
[x] Write integration tests for confirmation of requests.

Testing:

Write unit tests for backend validation and data handling.
Write integration tests for the end-to-end contact form submission process.
Perform security testing to ensure data protection and compliance.
Test various scenarios for form submissions (e.g., successful, invalid data, server errors).

Expected Outcome:
A secure and efficient API endpoint that can receive user email, store it in the database, and confirm the user's request.
The API endpoint allows users to submit a request via the "Squeeze" form, and ensures the template is sent to the appropriate email address.
Labels
feature

First of all, I can't find your issue. Link your issue to this PR

First of all, I can't find your issue. Link your issue to this PR

I do not have permissions to link an issue. I contacted the team lead but I don't think he's available. @NdubuisiJr

https://github.com/hngprojects/hng_boilerplate_java_web/issues/9
Corresponding Issue

LGTM


Fix build fail with serde feature

And added check to CI.
Also, fix build fail with --no-default-features (https://github.com/tokio-rs/bytes/pull/293/commits/bbf7785c722266d566a01d7fb8a099cb44e3bef5)

Thanks


Filter for comments

As a FOI analyst
I want a filter ability for comments
so that I can filter out automated comments, etc

Assumptions & Scope
What are the assumptions for this story?
What is in and not in scope?

Dependencies? What is the impact of this dependency? (If so, link dependency in the ticket, make it visible in a team´s backlog)
Validation Rules? (If yes, list here)
Design
@xxx - please link the Design here
Definition of Ready

[ ] Is there a well articulated User Story?
[ ] Is there Acceptance Criteria that covers all scenarios (happy/sad paths)?
[ ] If there is a user interface, is there a design?
[ ] Does the user story need user research/validation?
[ ] Does this User Story needs stakeholder approval?
[ ] Design / Solution accepted by Product Owner
[ ] Is this user story small enough to be completed in a Sprint? Should it be split?
[ ] Are the dependencies known/ understood? (technical, business, regulatory/policy)
[ ] Has the story been estimated?

Definition of Done

[ ] Passes developer unit tests
[ ] Passes peer code review
[ ] If there's a user interface, passes UX assurance
[ ] Passes QA of Acceptance Criteria with verification in Dev and Test
[ ] Confirm Test cases built and succeeding
[ ] No regression test failures
[ ] Test coverage acceptable by Product Owner
[ ] Ticket ready to be merged to master or story branch
[ ] Developer to list Config changes/ Update documents and designs
[ ] Can be demoed in Sprint Review
[ ] Tagged as part of a Release
[ ] Feature flagged if required
[ ] Change Management activities done?

Suggested by Patrick Craib during review of design prototype at Sept 14/21 sprint demo

Duplicate


Should limit the length of overview table names

The name of CARTO tables is limited to a maximum of 63 characters,
according to PG's limit NAMEDATALEN).
But when a overview table name is generated the name is extended and can overpass this limit.
This currently has the effect of having overview creation fail, since here the base table name will be truncated.
When generating overview table names we should limit them to NAMEDATALEN-1 characters (64) while at the same time taking care of avoiding duplicate names.
This affects also the handling of overviews in the import API here

Not happening


insufficient funds error message not showing

go to "add liquidity" (v1 pool, i've tested bnt/usdt)
enter the amount in the token row (usdt in this case).
notice that even if you do not have sufficient balance on token or bnt, there is no error message.

cannot reproduce this. @ashachaf are you still experiencing this? @Velua are you able to reproduce this?

@pingustar Yeah looks like it's happening on the second text field https://streamable.com/h5h07o

fixed with commit https://github.com/bancorprotocol/webapp/commit/abaefa3986993ceb471df285b93a777729fa3e6d and on staging for review.


fix: Take scale into account

Description

Take scale into account when calculating offsets
Memoize the work of offsets - don't calculate it each time.
Tests - everyone loves tests.

In scaled destination tile size, this would render the pyramids in different locations:

Example pre-fix wrongness:

Checklist

[x] The title of my PR starts with a [Conventional Commit] prefix (fix:, feat:, docs: etc).
[x] I have read the [Contributor Guide] and followed the process outlined for submitting PRs.
[x] I have updated/added tests for ALL new/updated/fixed functionality.
[x] I have updated/added relevant documentation in docs and added dartdoc comments with ///.
[x] I have updated/added relevant examples in examples.

Breaking Change

[ ] Yes, this is a breaking change.
[x] No, this is not a breaking change.

Related Issues
Fixes #1904

@kurtome @ufrshubham could you review this PR too? :)

LGTM, although can you double check that _applyParallaxOffset looks right, it takes camera.zoom into account so I'm not sure if additional change are needed there. Maybe not since it's all relative anyways...

LGTM, although can you double check that _applyParallaxOffset looks right, it takes camera.zoom into account so I'm not sure if additional change are needed there. Maybe not since it's all relative anyways...

Sure thing. Probably needs a test as well.

LGTM, although can you double check that _applyParallaxOffset looks right, it takes camera.zoom into account so I'm not sure if additional change are needed there. Maybe not since it's all relative anyways...

Sure thing. Probably needs a test as well.

So it doesn't look like it uses the destination tile size into account.. and I'm not sure it should. Yes, destination tile scale might seem like a cheap "scale" or "zoom", but it could just be sizing different maps to be the same for mixing. I don't know what the original intent was behind having it passed in versus reading it from the map.... and now that Tiled Component is positionable - it's also scalable.
That mixed with my unfamiliarity with parallax and I'm not sure.

LGTM, although can you double check that _applyParallaxOffset looks right, it takes camera.zoom into account so I'm not sure if additional change are needed there. Maybe not since it's all relative anyways...

Sure thing. Probably needs a test as well.

So it doesn't look like it uses the destination tile size into account.. and I'm not sure it should. Yes, destination tile scale might seem like a cheap "scale" or "zoom", but it could just be sizing different maps to be the same for mixing. I don't know what the original intent was behind having it passed in versus reading it from the map.... and now that Tiled Component is positionable - it's also scalable.
That mixed with my unfamiliarity with parallax and I'm not sure.

My hunch is that it will work fine as is because parallax is really a function of solely the current Viewport/Camera relative to the 0,0 position of the map. And 0.5 parallax is going to do the same thing regardless of the tile size (I think)

parallax is really a function of solely the current Viewport/Camera relative to the 0,0 position of the map

This is correct. Tile size won't matter for parallax. I'll try to play around with the changes just to be sure. But if I don't post anything within the next few days, consider a LGTM from my side.

These changes LGTM 🔥


RTD Revision

Description
Implement documentation feedback recommendations from #46
Issues:
closes #46

Codecov Report
Base: 97.77% // Head: 97.77% // No change to project coverage :thumbsup:

Coverage data is based on head (5315cb8) compared to base (4b8f247).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #47   +/-   ##
=======================================
  Coverage   97.77%   97.77%
=======================================
  Files           8        8
  Lines         225      225
  Branches       30       30
=======================================
  Hits          220      220
  Misses          4        4
  Partials        1        1

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

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


observer repsonse

I have found a small issue if client tries to observe any resource, irrespective of response, the client starts to listen to the resource.

Please elaborate :)
A screenshot of the message kog might help.
Also make sure it is not happening because of "display rejected" behavior or similar debug functionality.

When the response from server as 4.03, still the client starts to observe
i am not sure, its UI glitch or a real bug

The problem is that your server includes the Observe option in the response :)


ui: rationalize GC Bytes Age on the range status page

PR: https://github.com/cockroachdb/cockroach/pull/38283
From release notes:

Corrected the units displayed for the MVCC Intent Age and the MVCC GC Bytes Age fields in the Range Status page. [#38283][#38283] {% comment %}doc{% endcomment %}

This version is no longer supported. Closing.


Examples with Fractions

Hi
I'm trying to work with custom fraction but I found it a little bit confusing and took me a couple of days to figure out everything.
I think a deeper chapter on Fractions would be helpful for the end user, maybe a Thorntail-Example would also help to kick-start the project.
It would be also helpful to see the interaction with other fractions (e.g. configuring and accessing the data sources, keystores, realms or similar).
I still didn't figure out yet how to interact with the database that I've set with my Fraction via the DataSourceFraction, as long as the drivers are only available at a later stage.

This is about Thorntail v2 and hence belongs to JIRA: https://issues.jboss.org/projects/THORN


[$500] Workspace - Connect Bank account options appear when Bank account is added

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.58-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause - Internal Team
Action Performed:
Precondition:
Fully verified Bank account should be added on Workspace

Login with<EMAIL_ADDRESS>Profile> Workspace> Open the workspace from precondition
Go to Bank account and verify that Bank account is added
Go to Travel
Go back to Bank account> Dismiss the modal

Expected Result:
On travel page there should be Issue and manage cards, Book travel with Concierge and Learn more option
Actual Result:
Connect Bank account options appear when dismiss the Bank account modal
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/10db570e-8b51-46b9-b6e4-e07e117a85c6
View all open jobs on GitHub
Upwork Automation - Do Not Edit

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

Another same issue/root cause as https://github.com/Expensify/App/issues/34919

We're evaluating the root cause of this issue in https://github.com/Expensify/App/issues/36376#issuecomment-2032673029  since the root cause is the same as https://github.com/Expensify/App/issues/36376

@Christinadobrzyn so do you think we should put this issue on hold for that one?

yes, let's hold for 36376 to see if we can fix these together. I'll add a Hold to this GH.

Moving to weekly while this is on hold.

Still on hold.

Ah cool. Let me ask for a re-test: https://expensify.slack.com/archives/C9YU7BX5M/p1714762425554409

Looks like it's confirmed no longer happening. Closing out!


Add lambda ZipFile property

@phobologic I've added the logic to the validate method (and rewrote it a little bit).
If you like this as a single commit I can squash it too.

Thanks!


Storage Accoung Self Recovery requires permission

The self-Recovery requires user to have permission "Microsoft.Storage/storageAccounts/write" on the scope of which they want to restore, otherwise, the user would see an error "The storage account "****" could not be recovered. If the issue persists, we may not be able to recover this storage account." on the portal.
Can you add this part somewhere in this document?

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

ID: 19bf80a6-331d-dbc1-1a4d-067eff42e159
Version Independent ID: 06c51db4-0789-f196-aed3-9753a2e0af57
Content: Recover a deleted storage account - Azure Storage
Content Source: articles/storage/common/storage-account-recover.md
Service: storage
Sub-service: common
GitHub Login: @tamram
Microsoft Alias: tamram

@uonie , thank you for your feedback. We'll review this and get back to you shortly!

@uonie - I have made the change. It should go live later today.
#please-close


Correction for raw affiliation Montpellier Interdisciplinary Center of Sustainable Agri-Food Systems (UMR MoISA), French Agricultural Research Center for International Development (CIRAD)M, Montpellier, France

Correction needed for raw affiliation Montpellier Interdisciplinary Center of Sustainable Agri-Food Systems (UMR MoISA), French Agricultural Research Center for International Development (CIRAD)M, Montpellier, France
raw_affiliation_name: Montpellier Interdisciplinary Center of Sustainable Agri-Food Systems (UMR MoISA), French Agricultural Research Center for International Development (CIRAD)M, Montpellier, France
new_rors: 05kpkpg04;011xg1225
previous_rors: 05kpkpg04
works_examples: W4390411341
contact: 3cae0f85d1bc89e47b12ce16a1ebe12e:4b298ef9db437986 @ supagro.fr

This issue was accepted and ingested by the OpenAlex team on 2024-12-19. The new affiliations should be visible within the next 7 days.


Using the validation set

Since the validation set section is currently commented out, how do you think it is best used for this case of neural network?
Thanks

I used the ImageNet validation set but I'm not sure if it's best.


PPA Support

Thank you.
I see that there was an appveyor failure, but master is clean. I'm on mobile, so I haven't checked yet properly... Did appveyor became flaky and unreliable for us?

I think it was just because i merged before the tests passed.
FYI I uploaded the PPA to: https://launchpad.net/~kennethreitz/+archive/ubuntu/pipenv

Unfortunately, to make this work, I had to change the author name to "Kenneth Reitz" in setup.py. Going to work to make that dynamic so when uploading it gets changed to your name.

This is because of GPG keys.


Fix __resolveType process InternalNumber as Object

Fix a bug in JDK 8:
s1=0B;s1+=1;s1+=1;s1  will get wrong result 11
while process the first s1+=1(BigDecimal(0) + Integer(1)), BigDecimal(0).add(InternalNumber(1)) -> InternalNumber(1)
and then process the second s1+=1(InternalNumber(1) + Integer(1)), Object + Integer, process as string append, get result 11.
add InternalNumber class in typeCodes map could fix this problem.

Makes sense!
I think that the pull request should include a unit test. You could take your example above directly:
      String str = "s1=0B;s1+=1;s1+=1;s1";       Serializable expr =  MVEL.compileExpression(str);       Object result = MVEL.executeExpression(expr, new HashMap<String, Object>());       assertEquals(new BigDecimal(2), result);

I hava added the test case~


Some question about dataset in the paper

Thanks for the perfect work.
I'm confused by some dataset settings in the experiment from the paper.
Traffic-Train Dataset:
It was mentioned in the paper that 4160 frames were used for testing in your experiment, which was not aligned to the source dataset from http://vision.eecs.yorku.ca/research/anomalous-behaviour-data/. Could you help to provide the testing setting of your experiment on this dataset, i.e.  how to get the testing set of your experiment from the source testing set provided by Andrei Zaharescu and Richard P. Wildes?
Subway Dataset:
After a survey on this dataset, I found that the annotations were redefined. And I found the redefined annotations from http://vision.cs.utexas.edu/projects/abnormalactivity/cvpr2009-annotation.htm. Is this the same definition for your experiments?
Thanks!

Hi @dannyyangya,
Our experiment in Traffic-Train was different from the dataset's source because we found some difficulties when following its description. That is why we re-implemented some other methods and performed the experiments in another way for comparison. For the Traffic-Train, first 800 frames were used for training and 13840th-18000th frames were for evaluation. You may take a look here.
For the subway dataset, we used the annotation in the link you mentioned (maybe with a little bit modification in training part, I am not sure). You may check the annotation here. That repository is for another improvement of this work, but it should have a similar configuration.
Hope this helps!


New translations and fix

[ ] Add Chinese translations

[ ] Fix language switcher style in Edge

Before:

After:

@picturepan2 is there a reason why we need the extra space between the icon and language string?

@Patrick1904 Fixed. It is aligned to the layout now.

@picturepan2 I noticed this didn't work as intended.


Typescript: Make MithrilVirtualElement and MithrilRoutes non-generic

Two proposed commits, which would make MithrilVirtualElement and MithrilRoutes non-generic.
These changes are suggested because of issues arising from TypeScript's limited ability to infer types.  This simple example on the Typescript playground illustrates the particular problem.
To illustrate in terms of Mithril, here is an example of a typical call to m.route():
m.route(document.body, "/a", {
    "/a": ComponentA,
    "/b": ComponentB,
})

In this example, assume both ComponentA and ComponentB implement MithrilComponent<T extends MithrilController>, with each component having a different concrete controller
type (ControllerA and ControllerB respectively).
Unless ControllerA's type is a subset of ControllerB's type (or
vice-versa), TypeScript will be unable to infer the type of the MithrilRoutes<T>
generated by m.route() - this is the same thing that happens in the simple playground example, where neither Bar nor Baz is a proper subset of the other. To get the call to m.route() to compile, a third type which is a subset of both ControllerA and ControllerB would need to be explicity provided:
// Both ControllerA and ControllerB implement MithrilController
m.route<MithrilController>(document.body, "/a", {
  "/a": ComponentA,
  "/b": ComponentB,
})

A similar situation is needed for plain m():
function view(ctrl: ParentController): MithrilVirtualElement<MithrilController> {
   return m<MithrilController>("div", [
     m.component(ComponentA),
     m.component(ComponentB)
   ])
}

There are a number of factors which motivate a change:

In the case where either m() or m.route() does not compile due to type-inference failure,
the resulting syntax error from Typescript is very unhelpful because both m() and m.route()
have a large number of overloads. Leaving this as is will likely result in confusion for
downstream users.
Relatively little utility is provided by defining either MithrilRoutes<T> or MithrilVirtualElement<T> over a type more specific than MithrilController. At most, it could be used as a form of type assertion to ensure that there is a more specific common interface for the components of a specific application.
Even given the possibility of using these generic collections as a type assertion, more common use cases would see TypeScript/Mithril developers specifying m.route<MithrilController> or m<MithrilController> all over the place just to get the code to compile; this provides no additional type-safety over a non-generic interface and increases boilerplate in an unpredictable fashion.

Instead, these commits adjust both of these container types to be non-generic; instead, they always contain collections of MithrilComponent<MithrilController>, which will match all valid MithrilComponents.

I like this much better. It doesn't remove the type safety enforced in the controllers, which was my main concern. I will admit I did make an oversight when I did most of this, in that TypeScript doesn't infer super types.
I have an added trivial fix for m() that's easy to make, but I can wait for this to go through first.

I actually learned a good bit about TypeScript's type inference system from our discussions; this issue was more subtle than I thought.
Just a quick note: I believe this has a merge conflict with PR #806, due to #806 containing a new reference to MithrilVirtualElement<T>.  The PRs can be merged in either order, but the second one merged will need a slight modification (to convert that new reference to MithrilVirtualElement).

I would have that be accepted first, and then this fixed off of that. You won't have merge conflicts AFAIK, but it won't compile otherwise.

I have rebased this commit with #806 to remove the (undetectable) merge commit.  This now compiles correctly and should be ready to merge.

I like this. @lhorie You okay with this?

@lhorie ping!

@impinball @lhorie can you guys take a look at this? It'd be good to be able to use these typings.

@tamird LGTM
@lhorie Ping


Move ttnn::embedding to match new pybind structure and replace C++ ttlib embeddings usage with it

Ticket
#9753
Problem description
Embeddings doesn't match new ttnn pybind format
What's changed
Moved embeddings into its own embedding/ folder independent from the data_movement folder as it was originally in its own namespace
TODO: need to add the EmbeddingsType enum to ttnn as rn it's only in the TTLib side, preventing us from using that API to fully match the TTLib API. Once that's done then we can completely replace the TTLib API calls with the TTNN API calls and delete the TT_eager implementation
Checklist

[ ] Post commit CI passes
[ ] Model regression CI testing passes (if applicable)
[ ] New/Existing tests provide coverage for changes

Approving, but where are the unit tests? Can we get these moved over and setup properly?

TTNN tests already exist, though there's a couple cases that are missing. I'll add them to the ttnn side as well.


The runs.using key in action.yml must be one of: [docker node12], got composite

When running the job locally using https://github.com/nektos/act I get the following error:
[asciidoctor-ghpages/build] ⭐  Run<EMAIL_ADDRESS>[asciidoctor-ghpages/build]   ☁  git clone 'https://github.com/manoelcampos/asciidoctor-ghpages-action' # ref=v1.3.0
[asciidoctor-ghpages/build]   ❌  Failure -<EMAIL_ADDRESS>Error: The runs.using key in action.yml must be one of: [docker node12], got composite

Would be good to have support for act since it would allow for easy local before-commit testing.

Yeah. I'm new to this GitHub Actions thing. I suppose I have to change the action to use a docker container instead.

Could you please try the v2.0.0 version to check if it works for you?
The code is available at the v2 branch.
Everything seems to work fine, but the process hangs at the last line of the entrypoint.sh,
when I try to push the changes to remote.

After changing the action to a docker type, it works perfectly on GitHub servers, but not locally using the nektos/act.
I think the issue is with the checkout process.

In the end, I figured out what was wrong.
If you run the action inside GitHub's servers, the container authenticates using an ssh key.
But if you are running the action locally, you need to authenticate via https using a GitHub Personal Access Token.
Check PR #6 for details.


Add support for postincrement and postdecrement

I would not recommend anyone to use them, but for the sake of completeness...

Is there a genuinely good reason for provding postfix operators? I can't think of one offhand.

I can't either, but as already written, this is more for the sake of completeness. As a user, i would expect the fixed point data type to give me the same possibilities as the integrated data types. Whether this can be used in a useful way or not should, in my opinion, also be the decision of the user himself. I would simply give him the possibility to do so.

Trying to provide capabilities similar to the integral/arithmetic types is a fair point.
Thinking back, I think the only reason I provided the prefix versions in the first place was because the floating point types support them.
Chances are they won't get used anyway since it seems they don't get used with the floating point types very often.


Add author placeholder

Resolves #29.

[x] Add author placeholder
[x] Update README.md
[x] Add test in generate-release-notes.test.js

@BelKed you should not open a PR from your main (default) branch for a few reasons see this
You said one yourself yesterday

@fregante, please, could you merge this pull request now, so I can open another?

Originally posted by @BelKed in https://github.com/fregante/release-with-changelog/issues/43#issuecomment-803647711

@yakov116, thanks for pointing on this problem 😄

@fregante, sorry for long inactivity on this PR.
Anyway, please, could you review?

@fregante, sorry for long inactivity on this PR.
Anyway, please, could you review?

CI is not passing

Thank you! The big issue is that the token is now required whether one wants the author or not. Also it's silencing a lint rule which should not be silenced.
While it's interesting I'm not sure it's worth the complexity. Definitely the token cannot be required when the authors aren't included.

I think I'm gonna avoid this for now, sorry.


Add optional default_extra param to formatter initializer.

This is used to specify extra parameters that are valid for all log messages formatted with the formatter. It is usefull to avoid repeting extra={...} in every log call.
Closes #11

@matteo-zanoni thank you for your pr. I will test it today, also I will update with tests and documentation before merge.

@jteppinette happy to help with any of those tasks... Just looking for some guidance I guess

@jteppinette I have updated the documentation and added simple tests for the new behaviour. Hoping this can be helpful

Hi @jteppinette have you been able to review this PR? Can I help with anything to get this merged and released?


Rename elite classes to PIGS...

As George Orwell opined, some animals are more equal than others.
As these classes are more equal, they should be considered Python Intrinsic Global Structures... PIGS? 😄

So should we propose the python committee that they should rename the __builtins__ to __PIGS__ ?

So should we propose the python committee that they should rename the __builtins__ to __PIGS__ ?

Seems to be a good idea. Ask for a new PEP
:-D

Seems to be a good idea. Ask for a new PEP :-D
PIG Extension Proposal


iOS truncates text instead of wrapping

Can't figure out how to get text to wrap to a 2nd line. Any ideas?
Hoping to use this on an internal project here at Xamarin.

Any news on this ticket ?
What's the actual iOS development status ?
I have NullReferenceExceptions when I try to use a custom MessageStyleSheet class...
Thanks

+1

I have found that when building an iOS project with supported architectures set to just "arm7" this problem doesn't exist. But release builds require "arm7 + arm64" for the app store and this seems to cause text to truncate. (Tested on a iPhone 6 Plus)

When targeting x64 the methods that use NSString.DrawString need to be converted to the non-deprecated extension methods and then the truncation disappears.
to fix change
Title.DrawString(titleRectangle, TitleFont, UILineBreakMode.TailTruncation, UITextAlignment.Left);
to
Title.DrawString(titleRectangle, new UIStringAttributes { Font = TitleFont, ForegroundColor = TitleColor });
and
Description.DrawString (descriptionRectangle, DescriptionFont, UILineBreakMode.TailTruncation, UITextAlignment.Left);
to
Description.DrawString(descriptionRectangle, new UIStringAttributes { Font = DescriptionFont, ForegroundColor = DescriptionColor });
Perhaps this has to do with Unified API. See
http://developer.xamarin.com/guides/cross-platform/macios/updating-tips/
Objective-C category methods are now generated as extension methods in the Unified API. For example, code that previously used UIView.DrawString would now reference NSString.DrawString in the Unified API.

+1

I'm beginning to look more into this issue, @EgorBo you tweeted me some information to use to help reproduce but I have been unable even after switching the build type and trying on an iPhone 6s +/iPhone 6+ any other steps/info someone can provide to help reproduce?

@zawadzkip - try building exclusively in ARM64 to see if that replicates it.

This was fixed in a diff app that used extremely similar code as yours....
https://github.com/aritchie/userdialogs/issues/55
On Thu, Jan 21, 2016 at 3:16 PM, Adam<EMAIL_ADDRESS>wrote:

@zawadzkip https://github.com/zawadzkip - try building exclusively in
ARM64 to see if that replicates it.
—
Reply to this email directly or view it on GitHub
https://github.com/EgorBo/Toasts.Forms.Plugin/issues/28#issuecomment-173730902
.

Fixed as of 2.0.0-pre2


QuandlRequestFailedException when use Tablesaw

http://localhost:8888/notebooks/doc/groovy/Tablesaw.ipynb

This generally works. I'm guessing that Quandl was having a temporary outage. Can you try again?

Also of note, Quandl4j 2.0.0 doesn't give great error messages. I've fixed that in master, but am waiting for the next release to be cut: https://github.com/jimmoores/quandl4j/commit/c4067f6f079a01b38058f8f394df71e05d1b65c9


'React/RCTView.h' file not found

I manually added the RNCWKWebView.xcodeproj to the XCode Libraries and now I've run into this problem: ../node_modules/react-native-webview/ios/RNCWKWebView.h:8:9: 'React/RCTView.h' file not found
react-native: 0.61.5
react-native-webview: 8.0.1
Has there been a fix? Please help.

manually? Why did you have to do that? Using autolink would work


Validation fails with "No JSON object could be decoded" when trying to load conifg

I'm not sure why exactly this is happening, but I'm getting No JSON object could be decoded when validation jobs are run.
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/rq/worker.py", line 588, in perform_job
    rv = job.perform()
  File "/usr/lib/python2.7/site-packages/rq/job.py", line 498, in perform
    self._result = self.func(*self.args, **self.kwargs)
  File "/srv/app/src/ckanext-validation/ckanext/validation/jobs.py", line 47, in run_validation_job
    options = json.loads(options)
  File "/usr/lib/python2.7/json/__init__.py", line 339, in loads
    return _default_decoder.decode(s)
  File "/usr/lib/python2.7/json/decoder.py", line 364, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/lib/python2.7/json/decoder.py", line 382, in raw_decode
    raise ValueError("No JSON object could be decoded")
ValueError: No JSON object could be decoded

CKAN version: 2.7.5
Validation version 0.0.7
CKANEXT__VALIDATION__DEFAULT_VALIDATION_OPTIONS='{"row_limit": 100000}'
And this is not happening for all instances that have same exact configurations.

Few things I've tried to debug this:

When I'm logging options and type(options) on line 47 (from above), I'm getting '{"row_limit": 100000}' and <type 'str'>

json.loads({"row_limit": 100000}') should be wokring and is working, but:

According to the above I assumed that options equals to '{"row_limit": 100000}' and it's string, but it does not seem so, as eval(options) and type(eval(options)) are giving {"row_limit": 100000} and <type 'str'> while it shoul be <type 'dict'>

however if I hardcode json.loads(eval(options)) the error  disspars and validation is succeded .

We are getting options from config like this https://github.com/frictionlessdata/ckanext-validation/blob/master/ckanext/validation/jobs.py#L40-L41

options = t.config.get(u'ckanext.validation.default_validation_options')

If I hardcode options='{"row_limit": 100000}' there and repeat above, now I'm getting <type 'dict'> as it should be. and error dissaperas in this case as well (note I'm just logging eval(options) here - not asigning to any variables)

Possible solution
I'm not sure what is exactly returned by t.config.get() that looks like string and is the type of str, but json.loads() is complaining about it anyway... Maybe it's core bug but as a quick solution (that will not harm anybody or anything) would be to check if eval still returns the str and if so pass to json.loads() that string.  Something similar to
if options:
        if isinstance(eval(options), str):
            options=eval(options)
        options = json.loads(options)

I'll push the PR shortly for review

INVALID the env variable was set to '{"row_limit": 100000}' instead og {"row_limit": 100000}


doc: Save to docker-compose.yml

Observed issue was:
docker-compose --version
docker-compose version 1.21.0, build unknown

ERROR:
        Can't find a suitable configuration file in this directory or any
        parent. Are you in the right directory?

        Supported filenames: docker-compose.yml, docker-compose.yaml

Change-Id: Ia750efa6da96509e2e095ae8f76f094b2b01bde6
Forwarded: https://github.com/edgexfoundry/edgex-go/pull/2261
Signed-off-by: Philippe Coval<EMAIL_ADDRESS>

Codecov Report

Merging #2262 into fuji will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             fuji    #2262   +/-   ##
=======================================
  Coverage   44.06%   44.06%
=======================================
  Files         186      186
  Lines       11682    11682
=======================================
  Hits         5148     5148
  Misses       6235     6235
  Partials      299      299

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 de0e9b0...a4f2d01. Read the comment docs.


feat(install_panels): add NTP servers panel

related https://github.com/harvester/harvester/issues/935

Please don't use linking keywords in the PR description/commit/comments. We want some verification before closing the issue.

I wonder if we should still allow users to finish the installation if the NTP server can't be reached.
If the cluster is in an air-gapped environment, the NTP server can't be reached.

Probably, we can just valid whether the domain is FQDN. Also, we can allow users to input empty string.

The NTP server can be an IP address (e.g. A local server).
How about displaying a warning dialog to the user if the entered server can't be reached, but still allow the user to continue?

Probably, we can just validate whether the domain is a FQDN. Also, we can allow users to input empty string.

Hi @bk201 @gitlawr , I updated it. Please help me review again. Thank you 😄 .

Looks good! But there is a minor issue:

Enter an NTP server that can't be reached
It prompts an error and I can press enter to conintue.
Press ESC to go back
If I press enter again, the NTP server is not validated again.

I will reset c.config.OS.NTPServers when users press ESC. Thank you.

I will reset c.config.OS.NTPServers when users press ESC. Thank you.

I meant going back from Proxy page to NTP page. The validation is not performed.

If I press enter again, the NTP server is not validated again.

Seems like the problem still exists if I go back from the proxy page.

@bk201 @gitlawr  Users already press Enter to ignore unreachable NTP servers. Do we need to validate it again after users go back from the proxy panel?

The user can change his mind and use a different NTP server, it would be good if validation is done again.

The user can change his mind and use a different NTP server, it would be good if validation is done again.

Yes, I think we already do it now. Like following steps:

Users input error.ntp and press Enter twice to ignore error.
Users press Esc to go back NTP servers panel.
Users change error.ntp to another.error.ntp and press Enter. We check the input again.
Users press Enter again to go to the proxy panel and press Esc to go back to NTP servers panel.
Users change another.error.ntp to error.ntp. We still check the input again here.

So we revalidate when users change the input, but we go through if users have already ignored the error.

Looks like it doesn't validate the server provided in IP form.

Looks like it doesn't validate the server provided in IP form.

Thanks for indicating the error. I send real NTP request now, not only test server connection. Thank you 😄


Touch ui limit parsys fires ajax calls when hovering over components in authoring UI

Problem
When an author hovers on a component or parsys, ajax calls are fired from javascript to the author instance.
These ajax calls are fetching json information from within /etc/designs/.. with a selector '2'
F.e. http://localhost:4502/etc/designs/..../jcr:content/general/banner/contents.2.json
All these ajax calls return a 404 and so they serve no purpose but polluting our client/server logs and contributes to a high load on our author instance.

Our temporary fix ( it will probably break the feature, but we are not using this )
Remove following lines prevents the ajax calls from beeing triggered.
/apps/acs-commons/touchui-widgets/source/touchui-limit-parsys.js:82 -> 96

/apps/acs-commons/touchui-widgets/source/touchui-configure-parsys-placeholder.js:130

Issue described on Adobe forums https://forums.adobe.com/thread/2325086
We are using AEM 6.1 with acs-commons 2.10.0.

This is a duplicate of #989. That issue is labeled as help wanted. Feel free to supply a pull request.


[Tables] Fix issue handling empty nextRowKey

Packages impacted by this PR
@azure/data-tables
Issues associated with this PR
#20913
Describe the problem that is addressed by this PR
When querying entities from a table, the service may respond with a nextPartitionKey and no or empty nextRowKey. Currently, the library assumes that nextRowKey will always be present. When an empty nextRowKey is sent to the service, we get a 400 error.
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Do not try to use nextRowKey if the service didn't send it or sent it empty
Are there test cases added in this PR? (If not, why?)
Yes added Unit Tests
Checklists

[ ] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[ ] Added a changelog (if necessary)

/azp run js - data-tables - tests


Fix typo

We can't afford to further lose clean README. 😁

Haha, thank you :) Sometimes I'm a little rushed 😅


reportTelegram - updated documentation

added documentation for reportTelegram reporting module

Closing this PR, we can continue using the old one.


Attribute and enchant factories should have a more obvious option to delete entries

This feature already exists (though it is hard to find). If you right click on the bar to the left of the enchantment or attribute entry, it will open a menu that allows you to remove it.

Oh well that's why left clicking all over the place didn't help

I would like to make this feature more obvious, but I'm not sure how to do that.
Maybe I could add the triple dot icon (⋮) to the bar?

I've added the triple dot icon to the bar in the latest update (v1.10.0). Additionally, you can now left click the blue part of the bar to open the menu as well. (Left clicking on the black sections still moves the entry up and down.)


fix(tanstack-start): Fix bug for getAuth

Description

Checklist

[ ] npm test runs as expected.
[ ] npm run build runs as expected.
[ ] (If applicable) JSDoc comments have been added or updated for any package exports
[ ] (If applicable) Documentation has been updated

Type of change

[ ] 🐛 Bug fix
[ ] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

!snapshot

Hey @octoper - the snapshot version command generated the following package versions:

Package
Version

@clerk/astro
1.2.1-snapshot.vf6cf34f

@clerk/backend
1.8.1-snapshot.vf6cf34f

@clerk/chrome-extension
1.2.4-snapshot.vf6cf34f

@clerk/clerk-js
5.16.0-snapshot.vf6cf34f

@clerk/elements
0.14.1-snapshot.vf6cf34f

@clerk/clerk-expo
2.2.2-snapshot.vf6cf34f

@clerk/express
0.0.31-snapshot.vf6cf34f

@clerk/fastify
1.0.33-snapshot.vf6cf34f

@clerk/localizations
2.6.2-snapshot.vf6cf34f

@clerk/nextjs
5.3.4-snapshot.vf6cf34f

@clerk/clerk-react
5.4.3-snapshot.vf6cf34f

@clerk/remix
4.2.17-snapshot.vf6cf34f

@clerk/clerk-sdk-node
5.0.30-snapshot.vf6cf34f

@clerk/shared
2.5.3-snapshot.vf6cf34f

@clerk/tanstack-start
0.3.0-snapshot.vf6cf34f

@clerk/testing
1.2.13-snapshot.vf6cf34f

@clerk/themes
2.1.22-snapshot.vf6cf34f

@clerk/types
4.15.0-snapshot.vf6cf34f

Tip: Use the snippet copy button below to quickly install the required packages.
@clerk/astro
npm i<EMAIL_ADDRESS>--save-exact

@clerk/backend
npm i<EMAIL_ADDRESS>--save-exact

@clerk/chrome-extension
npm i<EMAIL_ADDRESS>--save-exact

@clerk/clerk-js
npm i<EMAIL_ADDRESS>--save-exact

@clerk/elements
npm i<EMAIL_ADDRESS>--save-exact

@clerk/clerk-expo
npm i<EMAIL_ADDRESS>--save-exact

@clerk/express
npm i<EMAIL_ADDRESS>--save-exact

@clerk/fastify
npm i<EMAIL_ADDRESS>--save-exact

@clerk/localizations
npm i<EMAIL_ADDRESS>--save-exact

@clerk/nextjs
npm i<EMAIL_ADDRESS>--save-exact

@clerk/clerk-react
npm i<EMAIL_ADDRESS>--save-exact

@clerk/remix
npm i<EMAIL_ADDRESS>--save-exact

@clerk/clerk-sdk-node
npm i<EMAIL_ADDRESS>--save-exact

@clerk/shared
npm i<EMAIL_ADDRESS>--save-exact

@clerk/tanstack-start
npm i<EMAIL_ADDRESS>--save-exact

@clerk/testing
npm i<EMAIL_ADDRESS>--save-exact

@clerk/themes
npm i<EMAIL_ADDRESS>--save-exact

@clerk/types
npm i<EMAIL_ADDRESS>--save-exact


Missionaries & Cannibals game

Missionaries & Cannibals game. Added the instructions and added in the new UI

ok done @AnantAgarwaL11


Using SeedData can generate code that violates code analysis rules SA1118 and CA1814

When using EntityTypeBuilder.SeedData, EF Core migrations can generate code that violates code analysis rules SA1118 and CA1814.
A workaround is to globally suppress these rules, with Target = "~M:<project>.Migrations.InitialCreate.Up(Microsoft.EntityFrameworkCore.Migrations.MigrationBuilder)".
Steps to reproduce

Create an entity named MyEntity:

   public int Id { get; set; }
   public string Name { get; set; }

In OnModelCreating, seed the table as follows:

            modelBuilder.Entity<MyEntity>().SeedData(
                new { Id = 1, Name = "Test 1" },
                new { Id = 2, Name = "Test 2" };

Drop and recreate the database via migrations:

dotnet ef database drop -f
dotnet ef migrations remove
dotnet ef migrations add InitialCreate
dotnet ef database update

Enable code analysis by adding project references for Microsoft.CodeAnalysis.FxCopAnalyzers and StyleCop.Analyzers, and create a ruleset file.

Build the project.

Code analysis reports:
Migrations\20180302122836_InitialCreate.cs(118,17): warning SA1118: The parameter spans multiple lines
Migrations\20180302122836_InitialCreate.cs(118,25): warning CA1814: Up uses a multidimensional array of object[*,*]. Replace it with a jagged array if possible.

The offending lines in<PHONE_NUMBER>2836_InitialCreate.cs are:
            values: new object[,]
            {
                { 1, "Test 1" },
                { 2, "Test 2" },
            });

In case of use, here is an extract from my csproj:
  <ItemGroup>
    <PackageReference Include="Microsoft.EntityFrameworkCore.Design" Version="2.1.0-preview1-final" />
    <PackageReference Include="Microsoft.EntityFrameworkCore.SqlServer" Version="2.1.0-preview1-final" />
    <PackageReference Include="Microsoft.Extensions.Configuration.FileExtensions" Version="2.0.0" />
    <PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="2.0.0" />
  </ItemGroup>

  <ItemGroup>
    <DotNetCliToolReference Include="Microsoft.EntityFrameworkCore.Tools.DotNet" Version="2.1.0-preview1-final" />
  </ItemGroup>

Further technical details
EF Core version: 2.1.0-preview1-final
Database Provider: Microsoft.EntityFrameworkCore.SqlServer
Operating system: Windows 10 Professional
IDE: Visual Studio 2017 15.5.4

For SA1118, the code generation has been changed slightly so that this pattern is only used in more limited cases, however we have no further plans to change where it is still being used.
For CA1814, this is by-design. It is intentionally not a jagged array.

That's fine but how about adding suppression pragmas to the generated code?

@edwiles Because adding suppressions makes the code less readable for everyone, even those people (which is a lot) who don't use these tools. The discussion in #10203 may be relevant to this point,


always validate tableacl config before using it.

table names or prefixes should be sorted.
acl entries should not have two overlaped entries (overlap means a concrete table name will
match both entries).
'%' should be presented only as the last character of a name, meaning this is a prefix.

LGTM


jest not working with react 17

I have updated react to 17 and configured babel to new automatic runtime. Jest stopped working.
Whats the issue here?

Code:
  "scripts": {
    "test": "node --experimental-vm-modules node_modules/jest/bin/jest.js",
  },

  "jest": {
    "roots": [
      "<rootDir>/src"
    ],
    "moduleNameMapper": {
      "\\.(css|scss|sass)$": "identity-obj-proxy"
    },
    "setupFilesAfterEnv": ['./test.js'],
    "transform": {
      "\\.[jt]sx?$": "babel-jest"
    },
    "testEnvironment": "jest-environment-node",
    "modulePaths": [],
  }

import Enzyme from "enzyme";
import Adapter from "enzyme-adapter-react-16";

Enzyme.configure({ adapter: new Adapter() });

{
    "presets": [
      [
        "@babel/preset-react",
        {
          "runtime": "automatic"
        }
      ]
    ]
  }

import { mount } from 'enzyme';

it("renders", () => {
  const wrapper = mount(<div />);
});

Im still not getting why Enzym is required to test react components :(

The react preset injects import while you use CJS. Adding preset-env or some such should work. If it doesn't, can you put together a repository that works without "runtime": "automatic", but fails with it?

Stale, please open up a new issue if it's still a problem


Planning scene namespace

There might be a bug related to the moveit namespacing. It appears objects are not published correctly to the /planning_scene topic.
This is likely related to #146.

Relevant to #152

closed in #155


2024-01-01-preview

This checklist is used to make sure that common guidelines for a pull request are followed.
Related command

General Guidelines

[x] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[x] Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)
[x] My extension version conforms to the Extension version schema

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Thank you for your contribution! We will review the pull request and get back to you soon.

tests are passing locally, not sure why it doesnt pass in the pipeline

Please fix CI issues

⚠ Suggestions
Module: nginx

Update version to 2.0.0b1 in setup.py

Notes

Stable/preview tag is inherited from last release. If needed, please add stable/preview label to modify it.
Major/minor/patch/pre increment of version number is calculated by pull request code changes automatically. If needed, please add major/minor/patch/pre label to adjust it.
For more info about extension versioning, please refer to Extension version schema

@jamesfan1 there is a breaking change in this pr which deleted a parameter. If it's unintentional, please fix it, if it is intented, that's ok.
Another problem is the version number, the last released version is 1.0.0, since there is a breaking change, so the next release version would be 2.0.0. And if you plan to release a preview version, it should be 2.0.0b1.   1.0.0b1 is smaller than 1.0.0 and we should not release an older version than current one.
For version details, please refer to our doc here

@AllyW

I am unsure what is wrong for
Azure.azure-cli-extensions Integration test
it is passing locally, and generated a recording file
however, on ci it's failing with
ERROR    cli.azure.cli.core.azclierror:azlogging.py:212 Can't overwrite existing cassette ('/mnt/vss/_work/1/s/src/nginx/azext_nginx/tests/latest/recordings/test_deployment_cert_config.yaml') in your current record mode ('once').
on this line, but this line runs fine locally
public_ip = self.cmd('network public-ip create --resource-group {rg} --name {public_ip_name} --version IPv4 --sku Standard --zone 2').get_output_in_json()
have you seen such error?

added the tag, please merge when possible. Thank you all for the help!!
@jsntcy @AllyW


Dynamic Device Groups with Extension Attributes

As per Microsoft documentation DEVICE dynamic groups can use only specific attributes listed on the link below :
https://docs.microsoft.com/en-us/azure/active-directory/enterprise-users/groups-dynamic-membership#rules-for-devices
I would like to create device groups based on Extension Attributes. It seems the functionality is working fine for users.
Now if i query a device by using graph i get the following:
"extensionAttributes": {
"extensionAttribute1": null,
"extensionAttribute2": null,
"extensionAttribute3": null,
"extensionAttribute4": null,
"extensionAttribute5": null,
"extensionAttribute6": null,
"extensionAttribute7": null,
"extensionAttribute8": null,
"extensionAttribute9": null,
"extensionAttribute10": "MyCustomProperty001",     <-This is my test property/attribute
"extensionAttribute11": null,
"extensionAttribute12": null,
"extensionAttribute13": null,
"extensionAttribute14": null,
"extensionAttribute15": null
Is this omitted by design ? Please explain as i might be missing something fundamental here.
Is there any way to create any custom attribute and later use it on a device?

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

ID: 5f7a4070-1be8-5f1f-0092-68fb6b9e2305
Version Independent ID: cd34312f-b507-affe-555e-1e0c63e7d494
Content: Rules for dynamically populated groups membership - Azure AD
Content Source: articles/active-directory/enterprise-users/groups-dynamic-membership.md
Service: active-directory
Sub-service: enterprise-users
GitHub Login: @curtand
Microsoft Alias: curtand

If that is the case can we also check that Custom attributes synced from on-prem can be used as well?

@mmtechconsulting
Thanks for your feedback! We will investigate and update as appropriate.

@mmtechconsulting
Thank you for your time and patience throughout this issue.
Since this issue isn't directly related to improving our docs, and to gain a better understanding of your issue, I'd recommend working closer with our support team via an Azure support request. Or you can leverage our Q&A forum by posting your issue there so our community, and MVPs can further assist you in troubleshooting this issue or finding potential workarounds.

@JamesTran-MSFT
I like to open this topic again, as I think it is fundamental in order to create a dynamic device group based on extension. So far you can only use extension attributes in Conditional Access policies. We need to have it also for the creation of dynamic groups. Can you please let us know if this is coming anywhere soon?
Thanks
Marco

@schiffne
Thank you for following up on this!
Unfortunately, we don't provide product release updates through our GitHub channels. For this, I'd recommend referring to our Azure Updates or Microsoft 365 Roadmap pages for this info.
When it comes to the ability to create a dynamic device group based on extensions, since this feature isn't available yet. I'd recommend leveraging our User Voice forum and creating a feature request so our engineering team can look into implementing this. I've also created an internal feature request, so our engineering team is aware of this as well.


[BUG]

Describe the bug.
In the navigation bar the three line graphics-symbol are not align perfectly.
when the scroller shows up it's almost getting disappear.
Expected behavior
It is expected that the menu symbol should be align with some gap so that while the scroller appear user can clearly see the menu symbol.
Screenshots

How to Reproduce
Open the page (i.e https://www.asyncapi.com/en )
Use the chrome pesticide extension to figure out alignment of the card
🥦 Browser
Google Chrome
👀 Have you checked for similar open issues?

[X] I checked and didn't find similar issue

🏢 Have you read the Contributing Guidelines?

[X] I have read the Contributing Guidelines

Are you willing to work on this issue ?
Yes I am willing to submit a PR!

@souvikjs01 Could you mention at which size its having this issue.

@souvikjs01 Could you mention at which size it's having this issue?

I don't know the exact size but you can find it of mobile width size.


Web crawler

Still need to add user input to "generate" the URL for requested location.

Added user input to 'generate' URL via locations.txt that will be created from user input in the WeatherNode app.


Project Name easily confused with NVidia Triton

Hi,
just want to put this out there: Why was the name chosen to be Triton? It is easily confused with Nvidia's Triton Inference Server. Since both projects are within the Machine Learning and CUDA space this is especially confusing.

I agree with the comment above. Actually, when I read the announcement of this library, I double-checked twice if it was some work done in collaboration with NVIDIA. Maybe considering finding an alternative name to this project might help.

Definitely agree that changing the name is crucial. Triton from NVIDIA is quite a huge and old project and well-known in the community. Project related to DL, CUDA, GPUs, etc. with the same name causes confusion.

Triton from NVIDIA is well-known in the community. Projects related to DL, CUDA, GPUs, etc. with the same name could cause confusion.

Hi,
This is an important topic and I think that you deserve a detailed and transparent answer, so I'll give you a factual timeline of how things unfolded over the last 5 years or so.
In 2016 I started working on a system called ISAAC, which ended up being published at the SC17 conference (https://dl.acm.org/doi/10.1145/3126908.3126939). At GTC 2017, Nvidia unveiled the "Isaac" simulation environment, so I decided to use the name Triton for a big rewrite of ISAAC that I had in mind. In 2019 I published Triton in the proceedings of the MAPL19 workshop. Keep in mind that at the time there was no other DL system at that time, as NVIDIA's software was called "TensorRT inference server" -- you can see it in the version log or by running git reset --hard v1.9.0 && grep -ir "triton" . on their codebase.
Anyway, a year later Nvidia rebranded "TensorRT inference server" as "Triton inference server" and I chose not to rename my project. This decision was because (1) the MAPL19 paper was already cited, and the only thing that tied my two helpful PhD advisors to the project; (2) Triton was already established on PyPI (pip install Triton) and I couldn't find any other available name that I liked.
Just earlier this year, Jensen Huang talked advertised the Triton inference server, and the community had a month-long discussion on whether to change the name. This was basically a 50/50 split, with about half being in favor of keeping the name, and half wanting to change the name to "jade". But jade was not available on PyPI, so I opened a PEP request and got no quick response. This is when we decided to stick with Triton. Note that just a couple of weeks ago I finally got ownership of the jade name, but it was too late: the blog post was just around the corner, and our internal usage of the Triton name had grown substantially.
I hope this answers all your question. I think that this is an unfortunate coincidence. Nvidia probably didn't know about the Triton language when they rebranded their TensorRT server in 2020, and when they advertised it at GTC2021 the Triton language still had a small but active community -- plus a 2019 paper outlining the details of the original system.


[Modify]Show map coordinate in facility feature #1408

Coverage decreased (-0.003%) to 83.378% when pulling 0a068f3e7b34bc345e5e8e5bb56ed98243dbd92a on ShinjiTanimoto:modify_issue_facility_feature into 9d4aaf206e4875207c44ccfabd30e6282693b6da on shirasagi:master.


Enable additional warnings

Enable -Wsign-conversion, and on compilers that support it, -Wshorten-64-to-32.
Fix build errors.
Fixes #2127.

@jboeuf - PTAL for security bits

Why not use the more general -Wconversion?

That's a fair point... updating the PR.

-Wconversion is now enabled.

That'll need a merge. But otherwise looks fine to me. Still waiting for a green thumb from @jboeuf.

Merged.

Alright, I'll merge now so it's not too horrible to maintain. If @jboeuf has a problem with it, he'll speak up later :)

Sorry guys. Lots of stuff going on. I'll have a look now and issue a PR if
I don't agree with some changes.
On Fri, Sep 11, 2015 at 1:06 PM, Nicolas Noble<EMAIL_ADDRESS>wrote:

Merged #3322 https://github.com/grpc/grpc/pull/3322.
—
Reply to this email directly or view it on GitHub
https://github.com/grpc/grpc/pull/3322#event-407465729.

LGTM. Thanks much for the fixes, particularly in ssl.
On Fri, Sep 11, 2015 at 1:58 PM, Julien Boeuf<EMAIL_ADDRESS>wrote:

Sorry guys. Lots of stuff going on. I'll have a look now and issue a PR if
I don't agree with some changes.
On Fri, Sep 11, 2015 at 1:06 PM, Nicolas Noble<EMAIL_ADDRESS>wrote:

Merged #3322 https://github.com/grpc/grpc/pull/3322.
—
Reply to this email directly or view it on GitHub
https://github.com/grpc/grpc/pull/3322#event-407465729.

Heads up: on windows, this has made headers and implementation disagree on parameter types. Fix is here: #3396.


Allow setting scale factor for images created from memory

Current all native images created from memory are set to 1x scale factor, we should add a way to allow setting scale factor:
var image = nativeImage.createEmpty();
image.addRepresentation({ scaleFactor: 2.0, dataURL: '...' });
image.addRepresentation({ scaleFactor: 1.0, png: new Buffer('...') });

This would be really helpful. We currently do this as a workaround, which is definitely not ideal:
 const dataUrl = "..."
 const pngData = NativeImage.createFromDataUrl(dataUrl).toPng();
 const out2x = NativeImage.createFromBuffer(pngData, 2);


Rename to skript-regions

Support all the region plugins or some from Skript

That's gonna be hard to maintain and problematic afair we had struggles with supporting multiple region plugins in one addon, it's probably preferred to have split addons for different region plugins, easier to maintain.

Looking back at this, I do not think it is in our best interests to support multiple region plugins at this time. Additionally, WorldGuard is likely the most widely used. The syntax will differ with each plugin, so I think it's best to take a unique approach for any addons.


Distant access with dns ?

Hi,
I know we have to use AppleTV or iPad to get distant access.
But isn't it possible to access the rasp via a dynamic DNS, and set this dynDNS in config.json to access it when not at home ?
Problem may be to access it at home over dynDNS then but who knows.

HomeKit outside access is connect to iCloud account! Not homebridge!

You need appletv or compatible iPad in the house to work as HomeKit hub

The logic is to have a device always on local network, which can synchronise with iCloud so those outside can get data from iCloud, right?
No way to hack it so the raspberry with homebridge is connected to iCloud and play this synch part ?

Probably Apple never going to allow anyone get access to HomeKit hub

Simply use a VPN - Most routers include a vpn server these days :)

@KiboOst my guess is that you are trying to control your devices while not at home. As the previous comments briefly explain, remote access for your Home App is handled via iCloud. That said, it is perfectly valid to install something like https://www.npmjs.com/package/homebridge-config-ui and use that UI instead of the Home App to control your Homebridge when you are not at home.

wow, wasn't aware of this ! Will give it a test ;-)

Actually, I thought that a HomeKit device can only be bound to one peer, so either an iCloud-Account (including Personal devices and iCloud-Accounts of ‚allowed‘ People (and their devices!)) or homebridge-ui can be used with one homebridge instance.
Isn‘t this still the case?

Simply use a VPN - Most routers include a vpn server these days :)

You would need a bridging VPN or Bonjour won't work remotely.  Without Bonjour, your HomeKit app will still not find homebridge (or any other accessory), even though technically it can reach it.  Unfortunately, my router (a Fritz!Box) use a routing VPN.


Sleep, what's it good for?

Can you explain the purpose of the sleep flag on a state?

It's in the Machine class documentation:
https://github.com/tinkerspy/Automaton/wiki/Automaton-Machine-class#uint8_t-sleep-int8_t-v-

A machine in a sleeping state does not execute its event loop and does not call its action() handler, it does, however, process incoming messages.

Thank you.  By "incoming messages", do you mean just custom methods?  Does a machine put to sleep delay any state timers.  Would they not occur if the machine were asleep when it expired (or maybe it occurs immediately on wake up) or it expires in the future as if the machine were never asleep?


Can't find load row count button

Since the 5.0 update I can't find the load row count button.
It used to be in the low-right corner in the results window.
Is it removed (for some reason) or just moved ?

Sometimes it hide and appear again after window resized

Thank You!

Nevertheless I treat it as a bug. You should have left the issue open

Could you post steps to reproduce?


godot 4.3 dev5 on android is crashing when i try to start it

Tested versions

reproducible in godot 4.3 dev5 on android

System information
Android 14 One Ui 6.0
Issue description
I try to open the godot 4.3 dev5 that I installed in my phone and it just crashes and dont open
Steps to reproduce
the process i do is: i go to the godot org site and go to see the newest dev, i go install the apk file then i go and i install the apk on my phone, after this when i try to open it just crashes
Minimal reproduction project (MRP)
its not an in editor bug

If you're able to compile from source, feel free to give PR #89703 a try to see if it fixes it. It only changes java code so it should be possible to test the fix in your android project without recompiling the native libraries.

This is happening to me on the release for Godot 4.3. I changed nothing in my project besides updating it. Then on export to android it crashes on launch.

@ZachBeauchamp Please open a new issue with a minimal reproduction project attached.


build error with HAVE_AVX512BW

Hi,
I got the following error with clang-13 because it detected HAVE_AVX512BW
c++ -Wall -Wunused -Wextra -O2 -I. -I../include -mavx512bw -DHAVE_AVX512BW -c matcher_avx2.cpp
matcher_avx2.cpp:39:4: error: matcher_avx2.cpp must be compiled with -mavx2 or /arch:avx2.
#  error matcher_avx2.cpp must be compiled with -mavx2 or /arch:avx2.
   ^
1 error generated.

do you think if the workaround is good?
diff --git a/RE-flex-3.3.6/lib/matcher_avx2.cpp b/RE-flex-3.3.6/lib/matcher_avx2.cpp
index d2b4bef642..f560af6ad9 100644
--- a/RE-flex-3.3.6/lib/matcher_avx2.cpp
+++ b/RE-flex-3.3.6/lib/matcher_avx2.cpp
@@ -34,8 +34,8 @@
 @copyright (c) BSD-3 License - see LICENSE.txt
 */

-#if defined(HAVE_AVX2) || defined(HAVE_AVX512BW)
-# if !defined(__AVX2__) || defined(__AVX512BW__)
+#if defined(HAVE_AVX2)
+# if !defined(__AVX2__)
 #  error matcher_avx2.cpp must be compiled with -mavx2 or /arch:avx2.
 # endif
 #endif

Looks good. Thanks for the feedback!
I should check with an AVX512 capable machine more often. I may have overlooked this problem after an update. Sorry for that.


Cant walk on home planet

As soon as you land on home starter planet you move your mech you get error and game breaks. tested with seed

should be fixed now - reopen if needed