Meri/wait for all shards

Codecov Report

Merging #5693 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##             master   #5693      +/-   ##
===========================================
+ Coverage     82.99%     83%   +<.01%
- Complexity    11802   11806       +4
===========================================
  Files          1394    1394
  Lines         45323   45347      +24
  Branches       5605    5605
===========================================
+ Hits          37618   37640      +22
- Misses         5102    5111       +9
+ Partials       2603    2596       -7

Impacted Files
Coverage Δ
Complexity Δ

...java/io/crate/analyze/TablePropertiesAnalyzer.java
92.18% <100%> (+0.2%)
21 <0> (ø)
:arrow_down:

...eference/information/TablesSettingsExpression.java
100% <100%> (ø)
2 <0> (ø)
:arrow_down:

...io/crate/metadata/settings/CrateTableSettings.java
96.15% <100%> (+0.32%)
1 <0> (ø)
:arrow_down:

...tadata/information/InformationTablesTableInfo.java
98.96% <100%> (+0.04%)
3 <0> (ø)
:arrow_down:

...ate/metadata/information/InformationTableInfo.java
98.78% <100%> (+0.06%)
4 <0> (ø)
:arrow_down:

...main/java/io/crate/analyze/TableParameterInfo.java
100% <100%> (ø)
6 <0> (ø)
:arrow_down:

...nassigned/UnassignedShardsExpressionFactories.java
82.22% <0%> (-11.12%)
10% <0%> (ø)

...ration/reference/doc/lucene/IpColumnReference.java
77.27% <0%> (-9.1%)
4% <0%> (-1%)

...rate/planner/consumer/GlobalAggregateConsumer.java
85.71% <0%> (-8.04%)
15% <0%> (-1%)

...va/io/crate/protocols/postgres/types/JsonType.java
76.66% <0%> (-6.67%)
8% <0%> (-1%)

... and 32 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 2f867b1...7a5bf48. Read the comment docs.


Add more information about adding/removing members to modals

Ticket Link
1324
List of squashed commits

Add more information about adding/removing members to modals

Test Checklist

[x] Unit Tests Passed:
[x] Jasmine Tests Passed:
[x] General Visual Inspection:

I'm not sure if I agree with adding the sync destination message to the bottom because there will be too much information in the add and multi-add modals once my PR is merged. However, I am okay with the compromise of shortening the message. Also, it would be nice to wrap the message in a yellow alert box just like our confirmation modal for sync destinations:


DatePicker overlapview, serious bug

On any size of screen, the day name is always overlap each others. Please fix it :)

This is an android version issue.
I'll have to treat API level < 18 seperately

Tks for your reply.  How can I fix it without pulling your source code. On api > 18, the name of day only displays as the first letter.
And by the way, there's onething I would like to improve is font size on large and x-large screen. The button Ok and Cancel look quite thin and small. Here is screenshot

I love your project so much. It's beautiful.

You can only fix this in the source code.
The single character day labels are the intended design
The dialog on tablets follows the material design specs, but I'll review it.


Created a Static Landing (Home) Page

Developed a static landing page which welcomes users, provides basic information about the platform, and prompts them to begin now by registering.
Changes Made:

Implemented all relevant frontend code for the landing page via home.html, home.css and home.js.
Edited the base (navigation bar) to now redirect to the new home page upon pressing home.
Modified app.py to include new home route, and ensure index redirects to home. This way, users will be met with the landing page directly after loading up the website.
Added a button at the bottom of the home page redirecting users to the registration.

Future Changes:

Anyone with new ideas can add on/modify as needed.

Primary Reviewer: @clarawong20

I looked over the home page and it looks really good! I added some comments and I'm just being picky with the visuals. For the "Welcome to our gamified learning platform" title, I think having a darker colour for the font looks better visually than a lighter font with black lining it, so I suggested a navy blue colour (but you can change it to whatever darker colour you think looks good). For the 'get started by clicking below' part, I just suggested removing the underline so a) it looks nicer and b) it's not misleading because I initially thought it was a link and tried clicking the text first instead of the image (which is really cool by the way). Other than that, great work!

Sounds good, I've made the changes you recommended.


Duplication

In periods of inactivity, a chat that has a custom facebook messenger thread to a discord channel linking seems to flood duplicate messages periodically.

Me too  but be the the wifi white the bot

Could you change the log level to silly and send me the logs when this happens?

Yes I will set it now

Sent log to your email thanks

@Thashiznets Sorry, didn't get your email yet...

Ok it says sent but I forwarded it to you again<EMAIL_ADDRESS>

I'm having a similar issue. When a message is sent to the bot through messenger it will flood discord with duplicate messages.

can do

...and?

Looks for the most part solved. I think I saw maybe one duplicate message only lately but seems really a non issue now.

Alright, thanks.


Add heartbeat to SSE

This PR adds heartbeats to stream.go, which means the stream will write an empty event every 10 seconds.
Its purpose is to keep connections alive in the case where there is no more data to send, but the client is still connected. The current behavior is that the ELB terminates idle connections after 60seconds, resulting in a gateway_timeout. Addresses some of the problems mentioned in #645

Changed the base to horizon-0.15.0 branch.

Yeah sure I'll close this one and address both in the other PR


Fix warning in MapdlBase class

it says you cannot import it... but most our reference call that class .. but they are actually using MapdlGrpc.

I need to clarify that.


WIP improve project infrastructure

Work in progress.

yarn dev for development build
yarn dev-server to start webpack dev-server
yarn build for production build
yarn storybook to start Storybook server

Using webpack 4 and Babel 6. Trying to keep webpack config clean and minimal.
Not using Babel 7 since the dependant projects (like Storybook) haven't adapted yet to the new @babel/foo scheme in terms of peer dependencies. Once we upgrade to Babel 7, we'll put all Babel related config to babel.config.js right next to webpack config.
Trying to keep root directory clean, all build/tool configuration lives in config directory.
Not using whatwg-fetch since all our target browsers already support this API (see targetBrowsers in webpack config).
Using A.x or possibly A.B.x semver ranges instead of ^A.B.C to make version assumptions more apparent. (This convention was taken from openshift/console project.)
There are currently some un-avoidable unmet peer dependency warnings. All of them come from patternfly-react and all of them are irrelevant for us, i.e. pulled in through optional dependencies.
Supported browsers are the same as in openshift/console project: latest version of Chrome, Firefox, Edge and Safari. You can also see our current target browsers when running yarn dev.
Using babel-polyfill instead of "ponyfills" which are better suited for libraries. We own the whole HTML page, we don't care if the polyfill mutates the globals.
Supported Node.js versions declared in package.json: it's either v8 (current active LTS) or v10 (upcoming LTS).

cc @ohadlevy @jelkosz @rawagner @mareklibra

Hmm, all files appear as newly added, I think I messed up something. Next time, I'll rebase on top of recent master.

@vojtechszocs webpack config looks good. But please rebase on top of the master so we can see the actual changes.

@vojtechszocs I believe that based on #22 this PR can be closed?

@vojtechszocs , #22 landed, can we close this one? We can still use bits from here for follow-ups.

Closing in favor of #26.


feat: add unit tests for helpers and also fixes for existing tests

Description

Add fixes for announcementHero and Modal component
Add tests for helpers

Related issue(s)
fixes #2046

/rtm


dirPaginate - Default itemsPerPage value not working when include pagination-id

Initially the pagination works wonderfully on my page. By default, it lists 10 items and also accepts input from user. No problem there. Then, since I have multiple paginations on my page, I decided to implement the pagination-id. That's when the default value does not work anymore.
Code written in Jade
input(class='form-control', type='number', ng-model='pageSize')
dir-pagination-controls(pagination-id='sonydadc')
tr(class='items', dir-paginate='object in objects) | itemsPerPage:pageSize || 10', pagination-id='sonydadc')

Note, that the way I define the default value is by setting itemsPerPage to be pageSize || 10. Without the pagination-id implementation, it works fine. Meaning that on page load, it displays only 10 items unless user specifies otherwise.
However, when implementing the above code instead(that is adding the pagination-id), it will display the entire list on page load. It will however successfully display limited items when the limit is specified.
Thus, I can assume that by adding the pagination-id somehow disrupts my default value. Pretty sure this is not a bug but most likely I overlooked something in my code. Hope someone can look into this.
Sorry if anything is confusing or unclear. Also, do let me know if I need to provide any additional information. Thanks !!

I'll close this issue since I've found a workaround for said problem. Thanks !!


Mobile push notifications don't work when bot is running

Summary
Mobile push notifications don't work when bot is running
Reproduction Steps
Run the bot
Have another account message you so you would get notifications
No notification ever comes
Code
No response
Expected Results
Get notification
Actual Results
No notification ever comes
System Information
not relevant
Checklist

[X] I have searched the open issues for duplicates.
[X] I have shared the entire traceback.
[X] I am using a user token (and it isn't visible in the code).

Additional Information
No response

This isn't related to discord.py-self, in fact, not related to any library at all. Try logging into that account yourself on a device without a selfbot, and receiving a message from another account, you won't get a mobile notification. That's how Discord functions.

This isn't related to discord.py-self, in fact, not related to any library at all. Try logging into that account yourself on a device without a selfbot, and receiving a message from another account, you won't get a mobile notification. That's how Discord functions.

This isn't related to discord.py-self, in fact, not related to any library at all. Try logging into that account yourself on a device without a selfbot, and receiving a message from another account, you won't get a mobile notification. That's how Discord functions.

In Discord notification settings there is "push notification inactive timeout". If the library could somehow trigger the timeout, notifications would work normally.

This isn't related to discord.py-self, in fact, not related to any library at all. Try logging into that account yourself on a device without a selfbot, and receiving a message from another account, you won't get a mobile notification. That's how Discord functions.

In Discord notification settings there is "push notification inactive timeout". If the library could somehow trigger the timeout, notifications would work normally.

You can try changing the presence with afk set to True.

You can try changing the presence with afk set to True.


Typescript compatible?

Hello, do you plan to add types declaration?

We currently have only flow types (e.g. https://github.com/diegohaz/schm/blob/master/packages/schm/src/index.js.flow).
I'd be glad to accept a PR adding TypeScript declarations.

I will take a look for that ;)


Remove phpstan/phpdoc-parser from require-dev

Hi,
I'd like to know if you have a specific reason to require phpstan/phpdoc-parser in your composer.json? It shouldn't be necessary unless you're parsing PHPDocs in your library, which seems out of scope :) Thank you.

I honestly don't know why I have phpstan/phpdoc-parser required. 😆

I included it in this commit, but I don't recall why. https://github.com/ramsey/uuid/commit/0d7b8c2b7a99a5bf69e96b84ead75c74290a5fb2

I stumbled upon this because I was inspecting the list of dependent packages on this one so that I can figure out how serious should I be about backwards compatibility. And this one caught my eye because I use it in my projects but I didn't see any reason why it would depend on the phpdoc-parser :)
It's not necessary for running PHPStan, so if you don't find any other reason, feel free to remove it :) Thanks.

Reopening to remind me to remove this (or for anyone else to submit a PR to remove it).

Hi - I submitted a PR :)


[Bug]: Outputting an array via code node produces a render error for the node (and I'm pretty sure tanks the performance)

What happened?
I have this code node that takes arr1 like ['a', 'b'] and arr2 like [['1', '2', '3', '4', '5'], ['7','8','9','10','11']] and produces arr3 like [[a,1][a,2][a,3][a,4][a,5],[b,7],[b,8],[b,9],[b,10],[b,11]].
The code node works, but it doesn't render the output (shows a rendering error).
And I'm pretty sure is the cause of a rendering-related performance drop for that graph (can barely click & drag the graph).
The code for the code node:
for(let i = 0; i < inputs.arr1.value.length; i++) {
    for(let j = 0; j < inputs.arr2.value[i].length; j++) {
        arr3.push([inputs.arr1.value[i], inputs.arr2.value[i][j]])
    }
}
return {
    arr3: {
        type: 'string[]',
        value: arr3
    }
};

What was the expected functionality?
Expected no rendering error or performance impact.
Describe your environment
Win11. Browser mode.
Relevant log output
No response
Relevant screenshots
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

It's expecting you to output a string[], so it's trying to render a list of strings, but it's actually a string[][] so that's why the rendering error is happening

omg :/ 🤦 ; that is true


[Task] When adding tags in tag management, random colors are given by default.

Description
标签量比较大的话选颜色是一个头疼的问题，可以默认随机一个颜色

Task List
No response

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑🤝🧑👫🧑🏿🤝🧑🏻👩🏾🤝👨🏿👬🏿

Description
If the number of labels is relatively large, choosing a color is a headache. You can default to a random color.

Task List
No response

I would like to help

I would like to help

welcome. this has assigned to you.
suggest edit code here.  https://github.com/dromara/hertzbeat/blob/master/web-app/src/app/routes/setting/tags/tags.component.ts#L149
addthis.tag.color = randomxxx when add a new tag

Hi @tomsun28 please review this PR: #1412

Hi @tomsun28 please review this PR: #1412

ok👍👍


Issue with Auto-IP and Multicast/socket connection

Hello together.
I am using the STM32H747-DISCO board with Zephyr v3.1 .
I configured a socket to receive Multicasts send via UDP by a client software running on a PC and I send back UDP packages to the client software as an answer to the received packages.
The socket is configured in the following way:
int32_t init()
{
    int32_t ret;
    struct sockaddr_in addr4;

    addr4.sin_family = AF_INET;
    addr4.sin_port = htons(MULTICAST_LISTEN_PORT);
    addr4.sin_addr.s_addr = INADDR_ANY;

    ret = configSocket((struct sockaddr *)&addr4, sizeof(addr4));
    ...
}

static int32_t configSocket(struct sockaddr *bindAddr, socklen_t bindAddrLen)
{
    struct zsock_addrinfo hints;
    struct zsock_addrinfo* pRetAddrInfo = NULL;
    const struct in_addr* multiAddr = NULL;
    struct net_if_mcast_addr* mcAddr = NULL;

    /* Create socket */
    udpData.sock = zsock_socket(bindAddr->sa_family, SOCK_DGRAM, IPPROTO_UDP);
    ...

    /* Bind socket */
    ret = zsock_bind(udpData.sock, bindAddr, bindAddrLen);

    /* Resolve network address */
    memset(&hints, 0, sizeof(struct zsock_addrinfo));
    hints.ai_family = AF_INET;
    hints.ai_socktype = SOCK_DGRAM;
    hints.ai_flags = AI_PASSIVE;
    hints.ai_protocol = IPPROTO_UDP;

    ret = zsock_getaddrinfo(MULTICAST_LISTEN_ADDRESS_STR, MULTICAST_LISTEN_PORT_STR,
                             &hints, &pRetAddrInfo);
    ...

    /* Get IPv4 multicast address */
    multiAddr = &net_sin(pRetAddrInfo->ai_addr)->sin_addr;
    ....

    /* Get network interface */
    struct net_if* ptrNetIf = net_if_get_first_by_type(&NET_L2_GET_NAME(ETHERNET));
    ....

    /* Add IPv4 multicast address to network interface */
    mcAddr = net_if_ipv4_maddr_add(ptrNetIf, multiAddr);
    ....
    net_if_ipv4_maddr_join(mcAddr);
    ...
}

The socket is polled within a thread to receive the UDP packages and send back answers to the received packages via:
void thread()
{
    received = zsock_recvfrom(udpData.sock, udpData.recvBuffer,
			sizeof(udpData.recvBuffer), 0,
			&clientAddr, &clientAddrLen);
    ...
    ret = zsock_sendto(udpData.sock, udpData.sendBuffer, sizeof(struct KistlerDeviceCfg), 0,
		(struct sockaddr *)&serverAddr, serverAddrLen);
    ....
}

If a static IP is configured, zsock_recvfrom receives the Multicast and zsock_sendto sends an answer to the client software.
If the IP settings are changed during running my application from static to DHCP and a DHCP is available zsock_recvfrom receives the Multicast and zsock_sendto sends an answer to the client software.
If the IP settings are changed during running the application from static to DHCP and a DHCP is not available, Auto-IP is used and Zephyr sets an IP automatically. In that case and zsock_recvfrom receives the Multicast but zsock_sendto does not send an answer to the client software and returns an error, -EINVAL.
When debugging the issue I found out that in the function net_if_ipv4_select_src_addr in line 3215 of zephyr/subsys/net/ip/net_if.c the variable src is not set and stays NULL.
In case of Auto-IP the function call to net_ipv4_is_ll_addr(dst) in the if statement in line 3223 returns true as the Auto-IP address is a local-link address. Therefore the else statement in line 3254 is executed. Within that statement src is not set. That's why the if statement in line 3258 is executed. Within this statement net_if_ipv4_get_global_addr returns NULL and  net_ipv4_unspecified_address returns NULL. That's why at the end zsock_sendto fails.
I do not understand why src is not set to a valid value in case of switching from static IP to Auto-IP. Why is src not set and zsock_sendto fails at the end?
In case of static IP the function call to net_ipv4_is_ll_addr(dst) in the if statement in line 3223 returns false and the if statement is executed. In that case line 3227 with the function call net_if_ipv4_get_best_match is executed and src is set to the static IP address.

I do not understand why src is not set to a valid value in case of switching from static IP to Auto-IP

It's not clear to me either. The logic behind address selection in case dst address is a LL address is pretty simple - use LL address as a src. The only reasonable explanation to me is that there is no LL address on an interface - are you 100% sure this isn't the case? Can you check for example with shell net iface command if the LL address is configured on the interface before trying to send a packet? Note, that IPv4 autoconf address is configured after a total delay of ~10 seconds.
I can see however another potential issue with IPv4 autoconf - it does not seem to set the subnet mask on the interface when the address is configured (/16 or autoconf address). So if the statically configured address has used a larger subnet mask, it can cause problems for example with ARP, which will not recognize the LL peer address as a part of the local subnet, and will try to request GW address instead. This could lead to errnoneous behaviour or the following error in logs in case GW address is not set:

 net_arp: Gateway not set for iface 0x24004bd8

Have you seen those?
The subnet issue should likely be fixed in the IPv4 autoconf module itself, i. e. when autoconf address is set, the subnet mask on the interface should also be updated, @jukkar do you agree? I see the limitation though, that it's not possible to have IPv4 addresses configured with different sub masks on a single interface, not sure if that's a big problem though.

@bonetor can you please see if #53666 fixes this issue?

@rlubos : I did see net_arp: Gateway not set for iface 0x24004bd8. After I applied #53666 I did not see it again.
Interface 0x24002558 (Ethernet) [1]
===================================
Link addr : 00:03:05:00:00:01
MTU       : 1500
Flags     : NO_AUTO_START,IPv4
Ethernet capabilities supported:
        10 Mbits
        100 Mbits
IPv4 unicast addresses (max 2):
        <IP_ADDRESS> autoconf preferred infinite
IPv4 multicast addresses (max 1):
        <IP_ADDRESS>
IPv4 gateway : <IP_ADDRESS>
IPv4 netmask : <IP_ADDRESS>
DHCPv4 lease time : 0
DHCPv4 renew time : 0
DHCPv4 server     : <IP_ADDRESS>
DHCPv4 requested  : <IP_ADDRESS>
DHCPv4 state      : selecting
DHCPv4 attempts   : 3

@rlubos , @carlescufi : #53666 does not fix the issue. src is still not set to a valid value.

@bonetor Just to clarify, what is the destination address in this scenario?

The serverAddr  in  zsock_sendto is set to
const struct in_addr xMcast = { .s4_addr = {239, 253, 253, 239}};
struct sockaddr_in serverAddr = { .sin_family = AF_INET,
                                .sin_addr.s_addr = htonl(xMcast.s_addr) ,
                                .sin_port = htons(MULTICAST_LISTEN_PORT)};

The client software running on a PC sending and receiving Multicasts via UDP is in a network with IP starting with 10.x.x.x.

Ok, then I've got confused by your previous statement, which suggested that the destination address is also IPv4 link-local:

In case of Auto-IP the function call to net_ipv4_is_ll_addr(dst) in the if statement in line 3223 returns true

But it seems that you want to send a packet to a non-link-local destination address, having only a link-local one, and this is simply not supported by Zephyr. In such case, the link-local source addresses are explicitly ignored:
https://github.com/zephyrproject-rtos/zephyr/blob/main/subsys/net/ip/net_if.c#L3129
I think this approach is justifiable in simple implementations, given that LL addresses are not routable and should only be considered for link-local communication. RFC3927 however does mention a possibility of sending a packet to a host that does not have a LL address:

A host with an IPv4 Link-Local address may send to a destination
which does not have an IPv4 Link-Local address.
@jukkar What's your opinion, shall we introduce such a possibility? I think it should be possible to introduce a final fall-back, in case there's really no other address, to try to choose LL address here (only if autoconf is enabled):
https://github.com/zephyrproject-rtos/zephyr/blob/main/subsys/net/ip/net_if.c#L3268

It'd still be needed however to register a valid gateway address (peer address in such case), otherwise ARP module will get confused, as from its perspective the IP packet is within different subnet, so it'll try to send it via gateway.

@jukkar Any opinion on the above?

A host with an IPv4 Link-Local address may send to a destination
which does not have an IPv4 Link-Local address.

Typically auto link local addresses are meant to point-to-point connection to a host with similar auto ip address, and have limited functionality (no routing etc). I would say that what we have implemented now is ok and fine, but I have no big objections if someone implements it and proposes patch, the review will then tell whether it should be accepted or not.

@rlubos In regard to your comment

Ok, then I've got confused by your previous statement, which suggested that the destination address is also IPv4 link-local:

The statement

In case of Auto-IP the function call to net_ipv4_is_ll_addr(dst) in the if statement in line 3223 returns true as the Auto-IP address is a local-link address.

actually says, that the destination address is a link-local address. But in the mentioned case we want to send the package from the device with a link-local address to the destination address which is the Multicast IP address.
const struct in_addr xMcast = { .s4_addr = {239, 253, 253, 239}};  // destination address
struct sockaddr_in serverAddr = { .sin_family = AF_INET,
                                .sin_addr.s_addr = htonl(xMcast.s_addr) ,
                                .sin_port = htons(MULTICAST_LISTEN_PORT)};

net_ipv4_is_ll_addr returns true, if the address is a link-local address, beginning with 169.254.
static inline bool net_ipv4_is_ll_addr(const struct in_addr *addr)
{
    return (ntohl(UNALIGNED_GET(&addr->s_addr)) & 0xA9FE0000) == 0xA9FE0000;
}

But why does the function returns true if zsock_sendto with serverAddr.sin_addr.s_addr = htonl(<IP_ADDRESS>) is called?

But why does the function returns true if zsock_sendto with serverAddr.sin_addr.s_addr = htonl(<IP_ADDRESS>) is called?

I've now noticed that net_ipv4_is_ll_addr() is wrong, it should use a proper subnet mask (0xFFFF0000) when filtering the first two bytes of the address - otherwise, it can lead to unexpected results, for instance with multicast address, if only specific bits match.
Anyway, fixing this doesn't fix the issue, as LL source would still be ignored for multicast destinations. Please give this branch a try, where I've implemented a fallback to LL address in case no better match is found:
https://github.com/rlubos/zephyr/commits/net/use-ll-addr-as-last-resort
I've tested it briefly with the autoconf sample, and I was able to sucessfully send packets to mutlicast address or non-LL peers on the same local network. If you can confirm that this helps in your case as well, I can send a PR out of it for further review.

I did a first test and with the changes on your branch it works also on my side. I will so some further tests with my setup and provide you more feedback.

@bonetor Did you have a chance to do some more tests?

Sorry for the late feedback. The described issue is solved with the implementations on https://github.com/rlubos/zephyr/commits/net/use-ll-addr-as-last-resort.
From my side the issue can be closed.

We still need to open a PR :) I'll send one.


Events fired on fill_in

I tried switching from capybara-webkit to poltergeist and stumbled over some differences, especially which events get fired.
When I execute fill_in on a textarea, CW fires:

focus
keydown/keyup for every letter inserted
change
blur

Poltergeist fires:

change

This doesn't really simulate real user interaction and resulted in some broken tests in my suite.

I am also experiencing this problem.

I'm also bumping into it ; sadly, this is a feature that would make sense to test what happens when emptying a search field for example.


[Label] Basic label has same colored background since 2.3.2

Steps
https://semantic-ui.com/elements/label.html -> Basic
Expected Result
up to version 2.3.1
Actual Result
colored background, if color was defined
Version
2.3.2
Testcase
https://semantic-ui.com/elements/label.html -> Basic

Caused by https://github.com/Semantic-Org/Semantic-UI/pull/6220
background-color: @basicBackground !important; should most likely be background: @basicBackground !important;

same issue

I'm also experiencing this issue on a few deployments. Seems .ui.green.label's background-color is overriding .ui.basic.label's background color.
It's visible on the current SemanticUI documentation site (see basic section):
https://semantic-ui.com/elements/label.html

same issue. I installed semantic ui 2.3.2 via npm in my angular app

@basicBackground equals to none @white; becoming a background-color invalid property value, caused by none.
Since specific attribute have precedence over combined, a possible solution would be adding a new variable for specific background attribute @basicBackgroundColor: @white; in label.variables and change
.ui.basic.[color].label {   background-color: @basicBackground !important;   ... }
for
.ui.basic.[color].label {   background-color: @basicBackgroundColor !important;   ... }
in label.less.

Same issue, tried to open a pr to fix this, access denied.

Same issue. thanks

I'll fix in patch

Same problem for v 2.3.3

Are there any updates on this?
Really like to use the last release without manual hacks.

@fridzema Fixed in FUI https://github.com/hammy2899/Fomantic-UI

This is on your demo site guys.  This is embarassing get it fixed:

This is still an issue in 2.3.3.

Important text in basic labels with a color cannot be read!

IMO this is high priority.

As noted by hammy2899, it fixed in https://github.com/hammy2899/Fomantic-UI .
For those interested in a patch, I found the commit reference:
https://github.com/fomantic/Fomantic-UI/commit/2bbd88060e30c3791e1b2c182b8071758761595c
It's part of Version 2.4.0, released June 19, 2018:
https://github.com/fomantic/Fomantic-UI/pull/32

Is there a Formantic-React?

"No but you can use fomantic with semantic react you just can use any of the new features which require implementation."
I'm not quite sure what you mean here. If you can use fomantic with this library https://github.com/Semantic-Org/Semantic-UI-React (assuming you mean using just fomantic css), how? and where do I find the info on how to? (I can't find anything on the repo linked).
To be clear, I'm using https://github.com/Semantic-Org/Semantic-UI-React and have been required to come to this library to discuss this issue because the issue can't be fixed in the react version, as the css is determined here. https://github.com/Semantic-Org/Semantic-UI-React/issues/2995.

I mean any bugs which are fixed in fomantic like this label one will be fixed in if you use fomantic with semantic-ui-react however if you want to use any new features which require new classes you will need to add them manually with className="some new class names" or if a new component is released like the calendar or range slider (coming in 2.5) the react library will not have support for these since they only implement features from the main semantic repo.
I have also just made a CSS only repo for easy integrations like react. https://github.com/fomantic/Fomantic-UI-CSS
I hope this clears things up if not please make an issue on the fomantic repo.

Any plans to make a new release of semantic-ui-css that includes this fix?

It would be great ! There has been no release since July 9... the problem is quite urgent, this type of label is often used for error messages on forms for example.
Can a Semantic-UI staff member tell us more about the date of the next release ?
Thanks ;)

@J2BB There is no exact date when this will be fixed, this will be fixed when @jlukic merges the PR and creates a release.
@ThiefMaster The CSS repo will be updated once this repo gets a new release.

Any workaround for web app? I use this semantic via the CDN.

@awansoekamto, I added the following code in my main css file:
// WORKAROUND - Fixing basic labels text colors in semantic-ui v2.3.1
.ui.basic.label {
  background: none #FFFFFF !important;
}

I think it worth to mention that this stylesheet is loaded by last in my app.

Thank you so much!

I've fixed this in next.

I've fixed this in next.
Thanks!
Let us know when this fix is pushed to CDNs.

I just released 2.4.0 with new goodies, is live on NPM

Awesome! Thanks team.

Perfect. Everything works!!! Thank you.

Coming from Semantic UI React and Next.js. Easiest way to use this library for my case is via CDN, can we get 2.4.0 added to the CDN?


Fix: create records using for_each instead of count

what

Modify default and ipv6 aws_route53_record resources to use for_each instead of count.

why

Prevent destroying and recreating DNS records when removing elements from aliases list.

/test all

Hope you don't mind I take this opportunity to update the contributors list in README.md (via README.yaml) 😄

/test all

Note that this is a minor release because it will cause destruction and recreation of resources due to the address change.

Perfect. thank you @korenyoni 👍


Js my sql prisma no auth template

i don't want pull request but i only want to remove the button says make pull request that is why am closing


Fix issue loading prior segmentation from csv column.

I ran into a couple issues trying to run Baysor on some Xenium using the segmentation from 10x's software as prior segmentation in Baysor, which I've fixed here.
First was assigning a number to a String field, and second was load_prior_segmentation! not returning what it was assumed to return.

Thank you for the fix @dcjones , I'll look into that!

Thank you for the contribution!


[6.0.0-beta.4]: warning appeared in UglifyJS when using lazy-loading

Versions
Angular CLI: 6.0.0-beta.4
Node: 9.7.1
OS: darwin x64
Angular: 5.2.7
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router

@angular/cli: 6.0.0-beta.4
@angular-devkit/build-optimizer: 0.4.3
@angular-devkit/core: 0.4.3
@angular-devkit/schematics: 0.4.3
@ngtools/json-schema: 1.2.0
@ngtools/webpack: 6.0.0-beta.4
@schematics/angular: 0.4.3
@schematics/package-update: 0.4.3
typescript: 2.5.3
webpack: 4.0.1

Repro steps
I struggled to extract the root cause as the message is not very clear, but I managed to find a use case to reproduce.
See repo https://github.com/cexbrayat/cli-6-lazy-moment-issue
Basically, it's a CLI app generated with 6.0.0-beta.4, using a lazy-loaded module (see first commit https://github.com/cexbrayat/cli-6-lazy-moment-issue/commit/100522b8341bcfdbc81f1ab91be0975588f8890a) with just one component.
At this step, ng build --prod succeeds without warning.
Another commit then adds moment.js to the lazy component (see https://github.com/cexbrayat/cli-6-lazy-moment-issue/commit/7d8d24692ee86812c2cb44de3775051d501be42d), and then ng build --prod produces the following warning:
ate: 2018-03-06T11:05:56.425Z
Hash: 9466e74cd993e3a86842
Time: 44384ms
chunk {0} 0.442690b173e99102856d.js () 334 kB  [rendered]
chunk {1} runtime.288d03f85133bfd7d654.js, runtime.9cf9739d90bc49f8571c.css (runtime) 1.81 kB [entry] [rendered]
chunk {2} styles.ac89bfdd6de82636b768.css (styles) 0 bytes [initial] [rendered]
chunk {3} polyfills.e8209731003adc2b9f02.js (polyfills) 59.1 kB [initial] [rendered]
chunk {4} main.800d45556c80d4a11074.js, main.9cf9739d90bc49f8571c.css (main) 242 kB [initial] [rendered]

WARNING in Unexpected '}' at 262:0.

WARNING in Invalid character(s) 'return Object.keys(map);
;
webpackContext.resolve = webpackContextResolve;
module.exports = webpackContext;
webpackContext.id = "RnhZ";' at 264:1. Ignoring.

WARNING in Invalid property name '"./af"' at 2:1. Ignoring.

WARNING in Invalid selector ';

function webpackContext(req)' at 246:1. Ignoring.

WARNING in Invalid selector 'var id = webpackContextResolve(req);
	var module = __webpack_require__(id);
	return module;

function webpackContextResolve(req)' at 250:1. Ignoring.

WARNING in Invalid selector 'var id = map[req];
	if(!(id + 1)) { // check for number or string
		var e = new Error('Cannot find module "' + req + '".');
		e.code = 'MODULE_NOT_FOUND';
		throw e;

	return id;
}
webpackContext.keys = function webpackContextKeys()' at 255:1. Ignoring.

WARNING in Unexpected '}' at 262:0.

WARNING in Invalid character(s) 'return Object.keys(map);
;
webpackContext.resolve = webpackContextResolve;
module.exports = webpackContext;
webpackContext.id = "RnhZ";' at 264:1. Ignoring.

WARNING in Invalid property name '"./af"' at 2:1. Ignoring.

WARNING in Invalid selector ';

function webpackContext(req)' at 246:1. Ignoring.

WARNING in Invalid selector 'var id = webpackContextResolve(req);
	var module = __webpack_require__(id);
	return module;

function webpackContextResolve(req)' at 250:1. Ignoring.

WARNING in Invalid selector 'var id = map[req];
	if(!(id + 1)) { // check for number or string
		var e = new Error('Cannot find module "' + req + '".');
		e.code = 'MODULE_NOT_FOUND';
		throw e;

	return id;
}
webpackContext.keys = function webpackContextKeys()' at 255:1. Ignoring.

Mention any other details that might be useful (optional)
Note that the warning is not present in 6.0.0-beta.3 or 1.7.2.
It's also not present if moment is not used in a lazy loaded module.

This is no longer the case in 6.0.0-beta.6 and above


[enhancement] How to secure APIs?

Figure out a way to secure APIs without screwing up the usability

Separating public and private APIs
Using Django's permission class for APIs
Dynamically adding/removing serailizer fields

@jitendravarma thank you for comments, I was also thinking about JWT


{rolling} shared-queues-vendor: Refresh integration patch

This extends https://github.com/sgstreet/meta-ros/commit/b8532f08bfdb7d94d29bdd53cd1b75e31236ea46 to ROS2 Rolling

I think this may have already been covered by e1bd3d6b85fba932c59617ea56fc0c8a2ec8264b and 976b0603a1ce83cb27d542bd87f1a97380a4ada7


Airqualibrate pdt1   similar devices

<Feature/Chore/Hotfix/Topic/Bug Title>
WHAT DOES THIS PR DO?
Creates 2 APIs

For calibrating data from the user using the model trained with AirQo data.
For training and calibrating data using a model trained with collocation data from the user. (If the user has their own data from the collocation of a low-cost and a reference monitor)

WHAT JIRA STORY/TASK IS RELATED TO THIS PR?
https://airqoteam.atlassian.net/jira/software/projects/DP/boards/44?assignee=5d91eed256bb950dd6489270&selectedIssue=DP-172
WHAT IS THE LINK TO THE PR BRANCH
https://github.com/airqo-platform/AirQo-api/tree/Airqualibrate-pdt1---similar-devices/
HOW DO I TEST OUT THIS PR?
cd AirQo-api/src/calibrate
python -m venv [DESTINATION_FOLDER]
[DESTINATION_FOLDER]/Scripts/activate
pip install -r requirements.txt
python jobs\regression.py (one time)
flask run
NB: Make sure you add a GCP service account key under jobs/      (available on request)
flask run
SAMPLE REQUEST BODY:
See images below for sample request bodies
NB:

accepted values for the KEY "pollutant"  are PM2.5 and PM10 (this will be a radio button on the frontend)
for KEY "file", please upload a CSV containing data for each of the keys specified except "file" and "pollutant"
for the rest of the entries except where KEY is "file" or "pollutant", the KEY is the column name in the uploaded CSV corresponding to the fields given by VALUE.  i.e. datetime, sensor1 pm2.5, sensor2 pm2.5, sensor1 pm10, sensor1 pm10, temperature, humidity and collocated reference monitor PM values.
SAMPLE CSV FILE AVAILABLE ON REQUEST

API 1

API 2

EXPECTED RESULTS:
API 1: data in CSV format with calibrated data (PM2.5 and PM10).
API 2:data in CSV format with calibrated data (PM2.5 or PM10) depending on the choice of reference pollutant selected.
WHICH ENDPOINTS SHOULD BE READY FOR TESTING?:
[POST] http://<IP_ADDRESS>:4001/api/v1/calibrate_tool
[POST] http://<IP_ADDRESS>:4001/api/v1/train_calibrate_tool
IS THERE ANY JENKINS CONSOLE LINK FOR CODE COVERAGE AND BUILD INFO?
NA
ARE THERE ANY RELATED PRs?
NA

Hi @Priscilla-AL , any way we can have some of this sample input as text? So that the reviewer's can just copy paste? Thanks.
Also, I have had to lower my pandas version to resolve some installation errors.

Thanks, Martin for the review, I have updated the instructions

itsdangerous==2.0.1

Hi Noah, Thanks for the review.

I have added the package to the requirements file
The documentation is on my to-do
There is actually no index. It defaults to numbered rows when no index is set.


Documentation for running-tests installing packages using production flags

Describe the bug
running-tests.md is showing the installation commands as npm install --save xxxx and yarn add xxxx.
Did you try recovering your dependencies?
Not applicable as this is documentation
Which terms did you search for in User Guide?
tests
Environment
Chrome Version 75.0.3770.100 (Official Build) (64-bit)
Steps to reproduce

Visit shallow-rendering

Expected behavior
Should the installation commands read npm install --save-dev xxxx and yarn add --dev xxxx since there are dev dependencies instead of production dependencies
Actual behavior
The commands are missing the dev flags as shown

Reproducible demo
Official documentation
I can put in a PR if that is the required change.

This is done intentionally. We install everything as regular dependencies. See #6180 (comment) for more information.


Export data error for queries Using variable

System information:

Operating system (distribution) and version : Windows 10 Pro
DBeaver version : 6.3.5
Additional extensions

Connection specification:
Using Postgresql 9.6.
Describe the problem you're observing:
While doing csv export for queries having variables, it is not working. it is saying syntax error.  Simple execute works and getting the data.
Query used is:
@set var_sid = 4
select * from <Table_Name> where <column_name> =:var_sid;
Steps to reproduce, if exist:
Select the Query and right click on it > Execute > Export from Query > CSV export.
and I got this error
Include any warning/errors/backtraces from the logs

thanks for the bug report.
as a workaround you could select set and select statements and then execute Export from, query

System information:
* Operating system (distribution) and version : Windows 10 Pro

* DBeaver version : 6.3.5

* Additional extensions

Connection specification:
Using Postgresql 9.6.
Describe the problem you're observing:
While doing csv export for queries having variables, it is not working. it is saying syntax error. Simple execute works and getting the data.
Query used is:
@set var_sid = 4
select * from <Table_Name> where <column_name> =:var_sid;
Steps to reproduce, if exist:
Select the Query and right click on it > Execute > Export from Query > CSV export.
and I got this error
Include any warning/errors/backtraces from the logs

Hello @uslss ,
Thanks for the response. But, by selecting set and select both statements, I'm getting the same error.
Regards,
Amit Kumar

Can't reproduce this anymore.
Please check version 7.0 - there were several fixes of data export task.


add tenant "RAMESSESII2"

For implementation of Network Isolation, I need to be able to find out the Network Plugin used by AppStudio Staging cluster.

@sbose78 please take a look.

Do you really need something this broad?

Hi @sbose78, if it looks good to you, please merge it or I can quickly add more changes if you need them. 😸

Have you logged into appstudio in the past, if yes, you should add yourself to the appstudio namespace as part of this.

Hey @sbose78, no I haven't logged into appstudio before and would need access to at least one of the namespace where I can do the query for which I an trying to create an admin role for myself in the "sabhardw" ns. Thanks


Explore using service loaders for composite json (de) serializers

I don't think service loaders are going to work for Android runtime (they can't be packaged in the app because of duplicates). I'm going to explore an annotation processor route.

So #11 did actually solve the problem. I've attached a patch with a failing test case.
json.patch.zip
I think this is an important use case, I plan to figure out a way to support it.

I also have some WIP changes for another gradle plugin here https://github.com/JayNewstrom/json/tree/composite-factories-gradle

Glide uses some annotation processor hacks to do this. https://github.com/bumptech/glide/pull/1742

This is complete and working!


StateNotifierProvider always refreshs when invalidated

Describe the bug
Once a StateNotifierProvider was created (e.g. via ref.watch(provider), it cannot be invalidated with ref.invalidate(provider), even though the widget that called ref.watch was disposed long ago and nothing else is listening.
I further analyzed the problem and figured out that the StateNotifierProvider is the only Provider calling ref.watch in their create() in base.dart:
class StateNotifierProvider<Notifier extends StateNotifier<State>, State>
...
  @override
  State create(ProviderElementBase<State> ref) {
    final notifier = ref.watch(this.notifier);

    void listener(State newState) {
      ref.setState(newState);
    }

    final removeListener = notifier.addListener(listener);
    ref.onDispose(removeListener);

    return ref.requireState;
  }
...

This call to ref.watch adds the _NotifierProvider, inialized during StateNotifierProvider's inialization, to the ProviderElement's _depenents once the StateNotifierProvider was created.
Later upon ref.invalidate, the StateNotifierProvider is refreshed since hasListeners is true in scheduler.dart:
void _performRefresh() {
  /// No need to traverse entries from top to bottom, because refreshing a
  /// child will automatically refresh its parent when it will try to read it
  for (var i = 0; i < _stateToRefresh.length; i++) {
    final element = _stateToRefresh[i];
    if (element.hasListeners) element.flush();
  }
}

To Reproduce
I created a minimal app which reproduces the bug. Just note the reaccuring state notifier initialized when clicking on Invalidate Providers.
Expected behavior
Expected behavior is that the StateNotifierProvider will not be refreshed as no listener (in the client code) is attached. Therefore, the ref.watch called in the create method must be disposed while the StateNotifierProvider is disposed.
I already created an issue (#1704) which did not describe the issue that clearly. Therefore, I closed that one to describe the bug in this issue in more detail.

I'm pretty sure that what you're describing was fixed some long ago on master.
The dev release is quite out of date. I'd suggest trying the same thing with a git dependency on master to see if it still happens.

Yess indeed it is working successfully on the master branch.
I am wondering why exactly is the dev release so far behind?

A nice:) Thanks for your help!

I have the same problem and I am using the latest version of flutter_riverpod (2.5.1)


Update readme to reflect #1763

When checking changes in the 0.0.218 release I noticed that auto fixing PT004 and PT005 was disabled but this change was not reflected in README. So I create this small PR to do this.

These docs are actually auto-generated! Do you mind changing the structs in src/violations.rs? (MissingFixtureNameUnderscore and IncorrectFixtureNameUnderscore should be changed from AlwaysAutofixableViolation to Violation, and the autofix_title method should be removed.)

All good, I got it :)


Refactor: render function

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


error compiling ddynamic reconfigure.

Im getting this error with the last two versions of ros wrapper and SDK. Im using Aeeon UP with 4.15 kernel,and SDK with realsense driver correctly installed. I have openVINO installed too.

Scanning dependencies of target roscpp_generate_messages_lisp
[  8%] Built target roscpp_generate_messages_lisp
Scanning dependencies of target roscpp_generate_messages_py
[  8%] Built target roscpp_generate_messages_py
Scanning dependencies of target _realsense2_camera_generate_messages_check_deps_Extrinsics
[  8%] Built target _realsense2_camera_generate_messages_check_deps_Extrinsics
Scanning dependencies of target _realsense2_camera_generate_messages_check_deps_IMUInfo
[  8%] Built target _realsense2_camera_generate_messages_check_deps_IMUInfo
Scanning dependencies of target sensor_msgs_generate_messages_cpp
[  8%] Built target sensor_msgs_generate_messages_cpp
[ 11%] Building CXX object realsense-2.2.0/ddynamic_reconfigure/CMakeFiles/ddynamic_reconfigure.dir/src/param/dd_double_param.cpp.o
Traceback (most recent call last):
File "/opt/ros/kinetic/share/genjava/cmake/../../../lib/genjava/genjava_gradle_project.py", line 14, in
genjava.main(sys.argv)
File "/opt/ros/kinetic/lib/python2.7/dist-packages/genjava/genjava_main.py", line 82, in main
gradle_project.create(args.package, args.output_dir)
File "/opt/ros/kinetic/lib/python2.7/dist-packages/genjava/gradle_project.py", line 152, in create
raise IOError("could not find %s among message packages. Does the package have a <build_depend> on message_generation in its package.xml?" % msg_pkg_name)
IOError: could not find ddynamic_reconfigure among message packages. Does the package have a
<build_depend> on message_generation in its package.xml?
realsense-2.2.0/ddynamic_reconfigure/CMakeFiles/ddynamic_reconfigure_generate_messages_java_gradle.dir/build.make:63: recipe for target 'realsense-2.2.0/ddynamic_reconfigure/java/ddynamic_reconfigure/build.gradle' failed
make[2]: *** [realsense-2.2.0/ddynamic_reconfigure/java/ddynamic_reconfigure/build.gradle] Error 1
CMakeFiles/Makefile2:341: recipe for target 'realsense-2.2.0/ddynamic_reconfigure/CMakeFiles/ddynamic_reconfigure_generate_messages_java_gradle.dir/all' failed
make[1]: *** [realsense-2.2.0/ddynamic_reconfigure/CMakeFiles/ddynamic_reconfigure_generate_messages_java_gradle.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
[ 13%] Building CXX object realsense-2.2.0/ddynamic_reconfigure/CMakeFiles/ddynamic_reconfigure.dir/src/param/dd_bool_param.cpp.o
[ 16%] Building CXX object realsense-2.2.0/ddynamic_reconfigure/CMakeFiles/ddynamic_reconfigure.dir/src/param/dd_string_param.cpp.o
[ 19%] Building CXX object realsense-2.2.0/ddynamic_reconfigure/CMakeFiles/ddynamic_reconfigure.dir/src/param/dd_enum_param.cpp.o
[ 22%] Building CXX object realsense-2.2.0/ddynamic_reconfigure/CMakeFiles/ddynamic_reconfigure.dir/src/dd_param.cpp.o
[ 25%] Linking CXX shared library /home/ihexa/realsense2_ws/devel/lib/libddynamic_reconfigure.so
[ 25%] Built target ddynamic_reconfigure
Makefile:127: recipe for target 'all' failed
make: *** [all] Error 2
Invoking "make -j4 -l4" failed

Any updates on this? I've got the same issue on my laptop (Ubuntu 16.04).

Maybe https://github.com/intel-ros/realsense/pull/624 fixes this?

@stwirth that worked. Thankyou


hpcups crash in libImageProcessor

Issue description
Printing with a HP Color LaserJet printer fails due to a crash in the hpcups filter
when using the driver "hpcups 3.19.1"
With the "CUPS+Gutenprint v5.2.14" driver printing works.
# dmesg
hpcups[2415]: segfault at 17a3000 ip 00007f261fb9902f sp 00007fff2cf18930 error 6 in libImageProcessor-x86_64.so[7f261fb96000+6e000]

# journalctl -u cups
cupsd[1261]: Started filter /nix/store/vm4qjbiy907l5xzpvjb8l04yljai53gq-cups-progs/lib/cups/filter/bannertopdf (PID 13999)
cupsd[1261]: Started filter /nix/store/vm4qjbiy907l5xzpvjb8l04yljai53gq-cups-progs/lib/cups/filter/pdftopdf (PID 14000)
cupsd[1261]: Started filter /nix/store/vm4qjbiy907l5xzpvjb8l04yljai53gq-cups-progs/lib/cups/filter/gstoraster (PID 14001)
cupsd[1261]: Started filter /nix/store/vm4qjbiy907l5xzpvjb8l04yljai53gq-cups-progs/lib/cups/filter/hpcups (PID 14002)
cupsd[1261]: Started backend /nix/store/vm4qjbiy907l5xzpvjb8l04yljai53gq-cups-progs/lib/cups/backend/hp (PID 14003)
hpcups[14002]: prnt/hpcups/HPCupsFilter.cpp 647: DEBUG: imageProcessorStartPage failed result = 2

Steps to reproduce
Try to print anything, a test page would do
Technical details
Please run nix-shell -p nix-info --run "nix-info -m" and paste the
results.
#58946  nix-shell -p nix-info --run "nix-info -m"
these paths will be fetched (0.05 MiB download, 0.28 MiB unpacked):
  /nix/store/9z1b1ldrhxh7brsi6fxnyjm7gk8kiynv-bash-interactive-4.4-p23-dev
copying path '/nix/store/9z1b1ldrhxh7brsi6fxnyjm7gk8kiynv-bash-interactive-4.4-p23-dev' from 'https://cache.nixos.org'...
error: restoring parent mount namespace: Invalid argument

oops.. let's try again:
# nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 4.19.32, NixOS, 19.03beta171931.3a4ffdd38b5 (Koi)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.2`
 - channels(root): `"nixos-19.03beta171931.3a4ffdd38b5"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

I'm having the same problem, and made some observations:

hplip 3.18.5 (used in release-18.09) works, even if compiled with pkgs from release-19.03
Here's a bug report with a proposed patch: https://bugs.launchpad.net/hplip/+bug/1787289
...as is probably here: https://bugs.archlinux.org/task/59548#comment172098

I didn't come around to test any of those patches thoug as I have only very limited access to my HP printer.


tileSources type image not working in IE 8

Example image tile source
Definition:
OpenSeadragon({
    ...
    tileSources: {
        type: 'image',
        url:  '/example-images/grand-canyon-landscape-overlooking.jpg'
    }
    ...
});

Showing only black page in IE 8 Windows XP machine.

Yes, this tile source is not supported on IE8 because it only works with the canvas API.
Maybe we should make that clear in the doc.

@avandecreme  The thing is it works in all other cases.
I face this problem only in example with type: 'image'
The other cases works fine.
I make quick fix for this case, maby i should make pull request?

If you have a fix, yes please share :)
This wasn't supported because when we develop new features we do no take the time to support IE8, but I believe @iangilman is still accepting patches for IE8.

Ok, i create pull request. That's my first time so i hope i did it right. Tell me if did something wrong.

Fixed by #1041


Ablation experiment

Description
There seems to be no ablation experiments on feature adapters.
(Optional:) Please add any files, screenshots, or other information here.
No response
(Required) What is this issue most closely related to? Select one.
Choose One
Internal issue ID
ae42b0bb-72df-4650-9d0d-48dd0306eea4
Paper URL
https://arxiv.org/html/2404.14837v1?_immersive_translate_auto_translate=1
Browser
Chrome/<IP_ADDRESS>
Device Type
Desktop

Please contact the authors directly for questions on document content.
This repository only tracks the quality of the experimental HTML format. Thank you!


Update to Quarkus 3.7

I wasn't able to test it as tests require a token but I thought it might help anyway.

I was able to set up CI with a test Discord server.

I'm going to merge this as it's a step in the right direction.


Support scientific visualization in graphics created on the frontend

Sunand's application team wants to produce graphics on the frontend from polyfaces with aux data and have the meshes be deformed and/or recolored using AnalysisStyle.
Cleaned up and promoted AnalysisStyle APIs.
Ported native code for building AuxChannelTable from PolyfaceAuxData.
TODO: Add a keyin to display-test-app to serve as the basis for a showcase sample.
TODO: Remove analysis-importer.
Incidental changes:
ThematicDisplay.toJSON() omits properties that match the defaults, instead of producing undefined (which becomes null when serialized).
Clean up bang-happy ThematicDisplayEditor code.

This looks overall good to me - particularly the thematic changes.

Thanks but it's not ready for review yet.

This looks overall good to me - particularly the thematic changes.

Thanks but it's not ready for review yet.

@pmconne - Missed the 'draft' status.  Let me know when and I'll take another look.

Running into following test build error:
==[ core-full-stack-tests ]======================================[ 65 of 88 ]==
src/frontend/standalone/ViewportChangedEvents.test.ts(236,131): error TS2345: Argument of type '{ inputName: string; }' is not assignable to parameter of type 'AnalysisStyleProps'.
Object literal may only specify known properties, and 'inputName' does not exist in type 'AnalysisStyleProps'.
npm ERR! code ELIFECYCLE
npm ERR! errno 2
npm ERR! core-full-stack-tests@ build:tests: tsc 1>&2 && npm run webpack:frontend
npm ERR! Exit status 2
npm ERR!
npm ERR! Failed at the core-full-stack-tests@ build:tests script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!  C:\Users\Mark.Schlosser\AppData\Roaming\npm-cache_logs\2021-06-21T18_56_12_547Z-debug.log
Returned error code: 2
"core-full-stack-tests" failed to build.


Add fuzzer

This is an updated version of https://github.com/pion/sdp/pull/12 by @maxhawkins.

This now also includes a test and fix for #13, which I rediscovered while running this fuzzer.

I've added my name to the readme, although I'm not happy about it. I'll raise that on slack.
I've also added some performance optimizations. There's plenty more low hanging fruit after this, but it is a start.

The build failure is because it claims that "marshalling" is the wrong spelling.
https://stackoverflow.com/questions/4434590/marshall-or-marshal-marshalling-or-marshaling
The wikipedia entry contains both spellings: https://en.wikipedia.org/wiki/Marshalling_(computer_science)
I will change it, but under protest.

Travis failed this PR, saying:

it failed 'Separate subject from body with a blank line'

But I don't see that any of my commit messages match this description. (And, unhelpfully, it doesn't appear to tell me which commit message is the problem.) I extracted the test it is doing  (awk 'NR == 2 {print $1;}' | wc -c != 1 and ran it on each commit in turn, and they all passed.
Any hints?
I've now spent more time trying to fix lint complaints (spelling, commit messages, where to write my name) than actually working on the code.

OK, the CI finally passes on this PR.

@josharian amazing work :fire:
So my only question is how can we get this into CI? I see there are some services that run fuzz testing against master non-stop, should we sign up for something like that?


Compiled Export Function

AmCharts5 supports multi-content exports, if we have time and it is desired by the health departments, we could add an additional export function to the ESID menu that allows a compiled export of all of the information on the ESID site into a neat and organized pdf.
https://www.amcharts.com/docs/v5/tutorials/creating-multi-content-pdf-export/
waiting on #93

What are these information exactly that can be exported?
Could that be also relevant for the health reporting on infectional diseases of the LHAs? (one LHA asked if that reporting could be included in the statistics part of ESID; @JonasGilg you have a template of a health reporting of one LHA)
Might be also interesting for the communication of scenarios to decision makers

What are these information exactly that can be exported?

So in general (and according to the documentation) the Information we can export are the information that is generally available to the website at the point when creating the snapshot.
This includes:

Data from the API that has been called to generate the visualizations
Snapshots of the AmChart graphs
Additional Text and Tables (e.g. to recreate the Scenario cards)

Could that be also relevant for the health reporting on infectional diseases of the LHAs? (one LHA asked if that reporting could be included in the statistics part of ESID; @JonasGilg you have a template of a health reporting of one LHA)

It seems the export function allows for arbitrary elements to be added to the exported document.
so as long as the LHA reporting is not too complex, we could provide an export function based on a template report (as long as there are no complext text segments that would need to be generated without a specific template).

Might be also interesting for the communication of scenarios to decision makers

If the API can provide additional explanations of components or similar stuff. It could be quite easy to generate a document to give an overview over the different scenarios.

Thanks! The export function is something we could discuss in the focus groups with the LHAs. We already noted the item.

This topic came up in the focus group today. The LHAs asked if it's possible to have a description of the simulations in the export to know exactly what simulation they used.

Could that be also relevant for the health reporting on infectional diseases of the LHAs? (one LHA asked if that reporting could be included in the statistics part of ESID; @JonasGilg you have a template of a health reporting of one LHA)

What does that mean, statistical part of ESID? Could you elaborate, probably a small example.

We thought that the statistics function should be programmed anyway. For further details, we need to have specific questions, which we can then discuss with the LHAs.


ReplayMod replay replaces modded blocks with other blocks.

Mod Name
ReplayMod
Mod Homepage
https://modrinth.com/mod/replaymod
Minecraft version
1.20.1 (LTS)
Describe the bug
Modded blocks get replaced with modded blocks from other mods. possibly fabric mods. but honestly no idea
Steps to reproduce

Download replay mod with connector
download a mod like another furniture or farmers delight.
record in a single player server with some of the modded blocks placed down.

Then you should see that for example a chair has become a lantern.
Logs
not needed
Additional context
No response

Probably a duplicate of #833
Also, use ReforgedPlay instead of ReplayMod

I am trying to use the replay voice chat compat mod. And I cannot find a
version that works.

Message ID: @.***>


fix read_result always  false

read_result always  false when read file success

@ZKvAH what's your nick on mbed? CAn you sign http://developer.mbed.org/contributor_agreement/ please

@0xc0170 my nick on mbed is ZKVAH
Excuse my bad English!


Construction 'if ... else' with 'try' throws an "Unexpected Else" error

V1
Process, Exist, notepad.exe
if !ErrorLevel
try Run notepad.exe
else
MsgBox Process Notepad.exe already exists!
Return
V2
ErrorLevel := ProcessExist("notepad.exe")
if !ErrorLevel
try 	Run("notepad.exe")
else
MsgBox("Process Notepad.exe already exists!")
Return
The variant without "TRY" (just Run(..) ) works correctly, but it is necessary in my working script.
If you manually add parentheses, this also works.
V2
ErrorLevel := ProcessExist("notepad.exe")
if !ErrorLevel {
try Run("notepad.exe")
}
else
MsgBox("Process Notepad.exe already exists!")
Return

Indeed a strange error, thanks for reporting.
Strange that V2 displays now an error.

Its because the else is paired to the try, not paired to the if
see here:
https://www.autohotkey.com/boards/viewtopic.php?f=37&t=114516
solution, use braces

It also seems that you could shorten the code to remove the ErrorLevel variable, though the converter script doesn't seem to do such refactoring.
You should be able to use:
if ProcessExist("notepad.exe")
try Run("notepad.exe")
or
if ProcessExist("notepad.exe") {
try Run("notepad.exe")
}


Is this project still maintained?

It does not seem so. Some part of the core are quite old and the upgrade of dependencies requires a major refactor.

If you're asking if it is actively being updated with the latest versions, the answer is no. However if there was something important that stopped working, then it will most likely be fixed. Also PRs are welcome

Gotcha. I was looking into the code - the trouble is that the graphiql-explorer is old, this package uses the last version, but the owner of that project has stopped any maintenance. Incidentally, the graphiql client has now the same feature, fully maintained, so was wondering whether it makes sense to swap the dependency in a quest to have a more manageable software

That makes sense to me. I haven't looked into the new graphiql explorer yet.


Return instruction missing in HLIL graph view

When a function returns void, sometimes the return instruction is omitted. This is fine in linear form, where it is obvious where the function ends. However, in graph view, it can make the control flow confusing. Also, analysis scripts might want to be able to rely on there being return instructions at the end of functions.
Return instructions should always be emitted, with a flag that the return instruction can be visually omitted at the end of linear display if it doesn't matter (last instruction in a void returning function)

Fixed in build 2100


IOS: APNS device token not set before retrieving FCM Token for Sender ID '<>'. Notifications to this FCM Token will not be delivered over APNS.Be sure to re-retrieve the FCM token once the APNS device token is set.

Hey, thanks for the awesome package.
Works like a charm in android, but i am getting this issue on IOS.

APNS device token not set before retrieving FCM Token for Sender ID '458258671137'. Notifications to this FCM Token will not be delivered over APNS.Be sure to re-retrieve the FCM token once the APNS device token is set.

when trying to retrieve APNS token through
FirebaseMessaging.instance.getAPNSToken().then((String token) {         print(token);       });
it returns null.
i have followed all steps shown in https://firebase.flutter.dev/docs/messaging/apple-integration.
what exactly am i missing, can you please look into this matter.
Thanks,

I recommen you to go on firebase_messaging github’s plugin to search a solution to your problem.

Hi! Sorry for taking so long to respond to your issue on GitHub. I was focused on releasing the new 0.7.0 version of awesome_notifications and awesome_notifications_fcm and i didn't had time enough to do both. But now I can answer all your questions.
So now i'm asking you to recreate this topic using the new issue template. There's a lot of missing informations that i need to understand your problem, and all the instructions are already in the new issue template. Also, remember to check beforehand if your issue was posted by another user.
So I will automatically close all previous issues so far.
Sorry for the inconvenience and i will be waiting for your new issue request.
Thank you so much for your support!


Support for javadoc response tag hint

In some situations it is not possible/hard to infer Response via byte code analysis, for example in exception mappers and RequestFilters.
In this situations we could leverage javadoc tags, for example:
/**
 *  @response 401 Not authorized (class level tag applies to all resource methods)
 *  @response 403 Not authenticated
 */
@Path("/cars")
@Produces("application/json;charset=utf-8")
public class CarEndpoint {

    @Inject
    CarService carService;

 /**
     * Deletes a car based on its ID
     *
     * @response 400 business logic exception
     * @response 404 This response will be ignored because it is already added via byte code analysis
     * @param user name of the user to log in
     * @param id car ID
     */
    @DELETE
    @Path("/{id:[0-9][0-9]*}")
    @RestSecured //activates a container request filter (a.k.a interceptor) which can return a FORBIDDEN or UNAUTHORIZED  response
    public Response deleteById(@HeaderParam("user") String user, @PathParam("id") Integer id) {
        Car entity = carService.findById(id);
        if (entity == null) {
            return Response.status(Status.NOT_FOUND).build();
        }
        carService.remove(entity); //here a ExceptionMapper can return a BAD_REQUEST response
        return Response.noContent().build();
    }

This will produce the following swagger.json:
 "delete":{
                "description":"Deletes a car based on its ID",
                "consumes":[
                ],
                "produces":[
                    "application/json;charset=utf-8"
                ],
                "parameters":[
                    {
                        "type":"integer",
                        "name":"id",
                        "in":"path",
                        "required":true,
                        "description":"car ID"
                    },
                    {
                        "type":"string",
                        "name":"user",
                        "in":"header",
                        "required":true,
                        "description":"name of the user to log in"
                    }
                ],
                "responses":{
                    "204":{
                        "description":"No Content",
                        "headers":{
                        }
                    },
                    "400":{
                        "description":"business logic exception",
                        "headers":{
                        }
                    },
                    "401":{
                        "description":"Not authorized",
                        "headers":{
                        }
                    },
                    "403":{
                        "description":"Not authenticated",
                        "headers":{
                        }
                    },
                    "404":{
                        "description":"Not Found",
                        "headers":{
                        }
                    }
                },

Merged and closed in 29c0669


new WebView API

@filipesabella . I really like to use live-p5 you developed, but now vscode-p5canvas has adopted the new WebView API, live-p5 can not run, Is there any updating plan in the near future? thank you!

I'm looking forward to the new version.Thank you again!

Done!


[chore] Fix GH yaml check warning

Merge requirement checklist

[x] CONTRIBUTING.md guidelines followed.
[x] Change log entry added, according to the guidelines in When to add a changelog entry.

If your PR does not need a change log, start the PR title with [chore]

[ ] schema-next.yaml updated with changes to existing conventions.

the warning is still there


Is this TFRA package of current version  compatible  with TF of GPU version?

I do not figure out how to specify the tensorflow as the GPU version, is it supported or not?

Hi @huifengguo, TFRA still not compatible with TF GPU version ,but The next release will support GPU version, coming soon!


Date is fixed at November 11, 2021

The current date is not being honored and seems to be fixed at November 11, 2021. See https://codesandbox.io/s/react-live-clock-date-bug-xwd5w as an example.

I think you just copied and pasted from the documentation.
There was a confusing part that was mentioned in #129.
November is the 11th month, there is no bug there.
Try
"dddd, MMMM Do, YYYY HH:mm:ss"

instead of
"dddd, MMMM Mo, YYYY HH:mm:ss"


Allowing to show time in time slider when range is less than 72h

Ref #357
@javisantana
@iriberri

This would make me happy :-)

:us:


Inconsistent XML detection

My repository has a GUI partially driven by a GLADE XML file.  Sometimes this is detected as XML, but just as often it seems the file is ignored and only the Python code is detected by linguist.

Sometimes this is detected as XML, but just as often it seems the file is ignored and only the Python code is detected by linguist.

Since very recently, Linguist counts markup languages like XML in the statistics. Could the inconsistencies happen because some repositories haven't been updated recently? (Language statistics update when you push to the repository.)

The repository in question was last pushed five days ago. Also, it had been reporting the XML in the statistics, but it ceased to do so with the latest push.

It's probably normal. There were some big changes recently on the way Linguist handles XML in language statistics. See #2097 (v4.0.0) and #2139 (v4.4.2) for more information.

As @pchaigno mentions, #2139 intentionally classified XML as data, which means it won't show up in the language stats bar any more. We appreciate any feedback if this is not ideal.


SSST: Style with warning cannot be pulled

Pull test case
https://github.com/abapGit-tests/SSST branch: test_tab.
Which is a smartstyle with this warning.

Then an unrelated message appears and the status is not deserialized

and one can no longer use system navigation buttons

Reason is that some list output is generated which blocks. It appears after e.g. starting debugging.

CI didn't catch this one https://ci.abapgit.org/ since it ran in batch. Oh well.

I missed that you had a branch. Yes, CI only pulls main. Can we merge your test branch?

Done.


Removes all other node_modules folders

My Ember app is in a "mono-repo", so the top folder is the git repo, and has a couple Ember apps and an Express app. I ran this update in one of the Ember apps, and all of the other node_modules folders are gone. In the other Ember apps and in the top level folder. As well as in an engine inside of one of the Ember apps.

This might be related to #143, where we are trying to track it down.

Going to close because I'm pretty sure #143 encompasses this.


Misleading nodepool name in example command

Most parameter values in the az CLI commands start with "my", indicating that the value is only a placeholder.
One exception of this pattern is in section https://docs.microsoft.com/en-us/azure/aks/use-system-pools#show-details-for-your-node-pool where the given command queries the details of the nodepool. The nodepool's name is set by "-n systempool" in the command. The missing "my" prefix makes it look like the given parameter value is the actual name but it is not.
I suggest to use either the default name "minion" or add the prefix, i.e. "mysystempool" to be more clear on this.

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

ID: 225b99ee-4c4d-54c2-cafa-ddcd286438da
Version Independent ID: ca9410a8-8ddb-5528-a0ac-0963458f190e
Content: Use system node pools in Azure Kubernetes Service (AKS) - Azure Kubernetes Service
Content Source: articles/aks/use-system-pools.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

@ingomaas we'll review your feedback and get back to you shortly. Thanks.

Thanks for bringing this to our attention. I have created a PR with the update, changes will be live shortly.
Thanks for improving our documentation! Just so you know, it's also possible to make these changes yourself, if you are interested. Doing so will improve our documentation, and it will add you as a contributor to the document.
In case you're interested, these are the steps to do so:

Navigate to the page of interest
Select the "Edit" button at the top right of the page
Select the "Pencil" icon on the right
Modify the text within the text box in markdown format - more info: https://github.com/adam-p/markdown-here/wiki/Markdown-Cheatsheet
Name your file change and provide a description, if necessary
Select "Propose file change"
Select "Create pull request" on the 'Comparing changes' page
Select "Create pull request" on the 'Open a pull request' page
You can find a GIF that demonstrates the end-to-end process for submitting changes here: https://aka.ms/editsqldocs.


CSS Crash with Buefy UI

Hallo,
this is nice library, but i found css crash with Buefy UI 😭

can anyone helpme 😔

I think indeed there might be a some style collisions but I'm also wondering if you've linked up the image preview CSS? If you didn't, can you do so and then post a new screenshot? A public test case would also be useful.

thx u, in this case i using Buefy v0.6.6
this is my css list

i'm just clear my browser cache, and now work but how to fill this space?

I think something like this should fix it:
.filepond--list
.filepond--item,
.filepond--file-wrapper {
  margin:0 !important;
  padding:0 !important;
}

this is no effect
.filepond--list
.filepond--item,
.filepond--file-wrapper {
  margin:0 !important;
  padding:0 !important;
}

i just added this code for remove unordered lists from Buefy UI
.content ul {
    list-style: none;
}

and finally i set the margin like this, in my case this look little bit better 😅

Okay, well, if you have a public test case I can look into it further. I would advise to inspect the elements (list, item and file-wrapper) and see if any margins or paddings are overruled, you can then specifically revert those.


Upgrade voluptuous-serialize to 2.0.0

Description:
Need wait home-assistant/home-assistant-polymer#1529 merged first
Change log: https://github.com/balloob/voluptuous-serialize/releases/tag/2.0.0
It changes data structure feed to ha-form, need change frontend's components/ha-form.js.
Particularly,
vol.Schema({'s': vol.In(['a','b','c'])}) will be serialized to
{
  "name": "s",
  "type": "select",
  "options": [
    ["a", "a"],
    ["b", "b"],
    ["c", "c"]
  ]
}

hue and deconz component's config flow used vol.In validation.
Related issue (if applicable): fixes #
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>
Example entry for configuration.yaml (if applicable):

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass

home-assistant/home-assistant-polymer#1529 has been merged, but didn't included in latest frontend, need wait another frontend version bump to merge this PR.


Issues cell collapsing on iPhone landscape

Attaching iPhone xs max screenshot on landscape. Cell labels are collapsed and can't read properly.

Bug Report Dump (Auto-generated)

Version 1.23.0<PHONE_NUMBER>)
Device: iPhone11,6 (iOS 12.0.1)
TestFlight: true

Yicks
Sent with GitHawk


Update zio, zio-test, zio-test-sbt to 2.0.19

About this PR
📦 Updates

dev.zio:zio
dev.zio:zio-test
dev.zio:zio-test-sbt

from 2.0.6 to 2.0.19
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "dev.zio" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #123.


Issue526/gpl 691 document describe workflow creation

Closes #526
Adds yard documentation for pipeline creation, including auto-generated summary pages of presenters, creators and state-changers.
I'd post a link to the yard docs on https://www.rubydoc.info/github/sanger/limber/ but its ignoring my attempts to tell it to build docs for a particular hash. So you'll need to check the docs locally by following the instructions in the Readme.
Edit: Live online link https://www.rubydoc.info/github/jamesglover/limber/doc_ifo_friendly

Ahh, there we go: https://www.rubydoc.info/github/jamesglover/limber/doc_ifo_friendly It didn't the / in the branch name

Rubydoc.info seems to be messing up the 'View class documentation' links, but they are working locally. I don't think its very happy, as I'm also getting 503s.


[noelware-log] Make default writer configurable?

https://github.com/Noelware/core-rs/blob/7001ac845e3bf805ee15401addb9dd39c06b4d96/crates/log/src/writers/impl.rs#L137-L147

This will allow to configure the Stream since it uses Stdout to detect if colours can be emitted, allow span metadata to be togglable.

Added in Noelware/core-rs@80673a1d


Build failure: cross compilation onnxruntime

Steps To Reproduce
nix build nixpkgs#pkgsCross.aarch64-multiplatform.onnxruntime -v -L on x86_64-linux
Build log
onnxruntime-aarch64-unknown-linux-gnu> CMake Warning at CMakeLists.txt:1559 (message):
onnxruntime-aarch64-unknown-linux-gnu>   MPI and NCCL disabled on Win build.
onnxruntime-aarch64-unknown-linux-gnu>
onnxruntime-aarch64-unknown-linux-gnu> -- Looking for clock_gettime in rt
onnxruntime-aarch64-unknown-linux-gnu> -- Looking for clock_gettime in rt - found
onnxruntime-aarch64-unknown-linux-gnu> -- Python Build is enabled
onnxruntime-aarch64-unknown-linux-gnu> -- Found pybind11: /nix/store/6awycaxs0a0q53yxlnwbq90vn4n3ggfr-python3.11-pybind11-2.12.0-aarch64-unknown-linux-gnu/include (found version "")
onnxruntime-aarch64-unknown-linux-gnu> -- Configuring done (7.7s)
onnxruntime-aarch64-unknown-linux-gnu> CMake Error at onnxruntime_unittests.cmake:61 (target_link_libraries):
onnxruntime-aarch64-unknown-linux-gnu>   Target "onnxruntime_test_all" links to:
onnxruntime-aarch64-unknown-linux-gnu>     GTest::gtest
onnxruntime-aarch64-unknown-linux-gnu>   but the target was not found.  Possible reasons include:
onnxruntime-aarch64-unknown-linux-gnu>     * There is a typo in the target name.
onnxruntime-aarch64-unknown-linux-gnu>     * A find_package call is missing for an IMPORTED target.
onnxruntime-aarch64-unknown-linux-gnu>     * An ALIAS target is missing.
onnxruntime-aarch64-unknown-linux-gnu> Call Stack (most recent call first):
onnxruntime-aarch64-unknown-linux-gnu>   onnxruntime_unittests.cmake:820 (AddTest)
onnxruntime-aarch64-unknown-linux-gnu>   CMakeLists.txt:1693 (include)
onnxruntime-aarch64-unknown-linux-gnu>
onnxruntime-aarch64-unknown-linux-gnu> CMake Error at onnxruntime_unittests.cmake:57 (target_link_libraries):
onnxruntime-aarch64-unknown-linux-gnu>   Target "onnxruntime_shared_lib_test" links to:
onnxruntime-aarch64-unknown-linux-gnu>     GTest::gtest
onnxruntime-aarch64-unknown-linux-gnu>   but the target was not found.  Possible reasons include:
onnxruntime-aarch64-unknown-linux-gnu>     * There is a typo in the target name.
onnxruntime-aarch64-unknown-linux-gnu>     * A find_package call is missing for an IMPORTED target.
onnxruntime-aarch64-unknown-linux-gnu>     * An ALIAS target is missing.
onnxruntime-aarch64-unknown-linux-gnu> Call Stack (most recent call first):
onnxruntime-aarch64-unknown-linux-gnu>   onnxruntime_unittests.cmake:1258 (AddTest)
onnxruntime-aarch64-unknown-linux-gnu>   CMakeLists.txt:1693 (include)
onnxruntime-aarch64-unknown-linux-gnu>
onnxruntime-aarch64-unknown-linux-gnu> CMake Error at onnxruntime_unittests.cmake:57 (target_link_libraries):
onnxruntime-aarch64-unknown-linux-gnu>   Target "onnxruntime_global_thread_pools_test" links to:
onnxruntime-aarch64-unknown-linux-gnu>     GTest::gtest
onnxruntime-aarch64-unknown-linux-gnu>   but the target was not found.  Possible reasons include:
onnxruntime-aarch64-unknown-linux-gnu>     * There is a typo in the target name.
onnxruntime-aarch64-unknown-linux-gnu>     * A find_package call is missing for an IMPORTED target.
onnxruntime-aarch64-unknown-linux-gnu>     * An ALIAS target is missing.
onnxruntime-aarch64-unknown-linux-gnu> Call Stack (most recent call first):
onnxruntime-aarch64-unknown-linux-gnu>   onnxruntime_unittests.cmake:1289 (AddTest)
onnxruntime-aarch64-unknown-linux-gnu>   CMakeLists.txt:1693 (include)
onnxruntime-aarch64-unknown-linux-gnu>
onnxruntime-aarch64-unknown-linux-gnu> CMake Error at onnxruntime_unittests.cmake:1366 (target_link_libraries):
onnxruntime-aarch64-unknown-linux-gnu>   Target "onnxruntime_mlas_test" links to:
onnxruntime-aarch64-unknown-linux-gnu>     GTest::gtest
onnxruntime-aarch64-unknown-linux-gnu>   but the target was not found.  Possible reasons include:
onnxruntime-aarch64-unknown-linux-gnu>     * There is a typo in the target name.
onnxruntime-aarch64-unknown-linux-gnu>     * A find_package call is missing for an IMPORTED target.
onnxruntime-aarch64-unknown-linux-gnu>     * An ALIAS target is missing.
onnxruntime-aarch64-unknown-linux-gnu> Call Stack (most recent call first):
onnxruntime-aarch64-unknown-linux-gnu>   CMakeLists.txt:1693 (include)
onnxruntime-aarch64-unknown-linux-gnu>
onnxruntime-aarch64-unknown-linux-gnu> CMake Error at onnxruntime_unittests.cmake:57 (target_link_libraries):
onnxruntime-aarch64-unknown-linux-gnu>   Target "onnxruntime_customopregistration_test" links to:
onnxruntime-aarch64-unknown-linux-gnu>     GTest::gtest
onnxruntime-aarch64-unknown-linux-gnu>   but the target was not found.  Possible reasons include:
onnxruntime-aarch64-unknown-linux-gnu>     * There is a typo in the target name.
onnxruntime-aarch64-unknown-linux-gnu>     * A find_package call is missing for an IMPORTED target.
onnxruntime-aarch64-unknown-linux-gnu>     * An ALIAS target is missing.
onnxruntime-aarch64-unknown-linux-gnu> Call Stack (most recent call first):
onnxruntime-aarch64-unknown-linux-gnu>   onnxruntime_unittests.cmake:1549 (AddTest)
onnxruntime-aarch64-unknown-linux-gnu>   CMakeLists.txt:1693 (include)
onnxruntime-aarch64-unknown-linux-gnu>
onnxruntime-aarch64-unknown-linux-gnu> CMake Error at onnxruntime_unittests.cmake:57 (target_link_libraries):
onnxruntime-aarch64-unknown-linux-gnu>   Target "onnxruntime_logging_apis_test" links to:
onnxruntime-aarch64-unknown-linux-gnu>     GTest::gtest
onnxruntime-aarch64-unknown-linux-gnu>   but the target was not found.  Possible reasons include:
onnxruntime-aarch64-unknown-linux-gnu>     * There is a typo in the target name.
onnxruntime-aarch64-unknown-linux-gnu>     * A find_package call is missing for an IMPORTED target.
onnxruntime-aarch64-unknown-linux-gnu>     * An ALIAS target is missing.
onnxruntime-aarch64-unknown-linux-gnu> Call Stack (most recent call first):
onnxruntime-aarch64-unknown-linux-gnu>   onnxruntime_unittests.cmake:1638 (AddTest)
onnxruntime-aarch64-unknown-linux-gnu>   CMakeLists.txt:1693 (include)
onnxruntime-aarch64-unknown-linux-gnu>
onnxruntime-aarch64-unknown-linux-gnu> -- Generating done (0.2s)
onnxruntime-aarch64-unknown-linux-gnu> CMake Warning:
onnxruntime-aarch64-unknown-linux-gnu>   Manually-specified variables were not used by the project:
onnxruntime-aarch64-unknown-linux-gnu>     CMAKE_EXPORT_NO_PACKAGE_REGISTRY
onnxruntime-aarch64-unknown-linux-gnu>     PYBIND11_PYTHONLIBS_OVERWRITE
onnxruntime-aarch64-unknown-linux-gnu>     PYTHON_INCLUDE_DIR
onnxruntime-aarch64-unknown-linux-gnu>     PYTHON_SITE_PACKAGES
onnxruntime-aarch64-unknown-linux-gnu>
onnxruntime-aarch64-unknown-linux-gnu> CMake Generate step failed.  Build files cannot be regenerated correctly.

Additional context
Add any other context about the problem here.
Notify maintainers

@jonringer @puffnfresh @ck3d @cbourjau
Additionally CC recently committer and reviewer(sorry for bother): @autrimpo @SomeoneSerge
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
 - system: `"x86_64-linux"`
 - host os: `Linux 6.8.6-zen1, NixOS, 24.05 (Uakari), 24.05.20240419.5c24cf2`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.2`
 - channels(root): `"nixos, nixpkgs"`
 - nixpkgs: `/home/cryolitia/.nix-defexpr/channels/nixpkgs`

Add a :+1: reaction to issues you find important.

We already disable tests at cmake level with onnxruntime_BUILD_UNIT_TESTS for CUDA builds. Changing it from doCheck to false cross continues with the cross-compile for me on the current master (80368c5). It fails at the end:
onnxruntime-aarch64-unknown-linux-gnu> [100%] Linking CXX shared library libonnxruntime.so
onnxruntime-aarch64-unknown-linux-gnu> /build/source/onnxruntime/python/onnxruntime_pybind_ortvalue.cc:11:10: fatal error: numpy/arrayobject.h: No such file or directory
onnxruntime-aarch64-unknown-linux-gnu>    11 | #include <numpy/arrayobject.h>
onnxruntime-aarch64-unknown-linux-gnu>       |          ^~~~~~~~~~~~~~~~~~~~~
onnxruntime-aarch64-unknown-linux-gnu> compilation terminated.
onnxruntime-aarch64-unknown-linux-gnu> make[2]: *** [CMakeFiles/onnxruntime_pybind11_state.dir/build.make:132: CMakeFiles/onnxruntime_pybind11_state.dir/build/source/onnxruntime/python/onnxruntime_pybind_ortvalue.cc.o] Error 1onnxruntime-aarch64-unknown-linux-gnu> make[2]: *** Waiting for unfinished jobs....
onnxruntime-aarch64-unknown-linux-gnu> /build/source/onnxruntime/python/onnxruntime_pybind_mlvalue.cc:11:10: fatal error: numpy/arrayobject.h: No such file or directory
onnxruntime-aarch64-unknown-linux-gnu>    11 | #include <numpy/arrayobject.h>
onnxruntime-aarch64-unknown-linux-gnu>       |          ^~~~~~~~~~~~~~~~~~~~~
onnxruntime-aarch64-unknown-linux-gnu> compilation terminated.
onnxruntime-aarch64-unknown-linux-gnu> make[2]: *** [CMakeFiles/onnxruntime_pybind11_state.dir/build.make:104: CMakeFiles/onnxruntime_pybind11_state.dir/build/source/onnxruntime/python/onnxruntime_pybind_mlvalue.cc.o] Error 1
onnxruntime-aarch64-unknown-linux-gnu> /build/source/onnxruntime/python/onnxruntime_pybind_iobinding.cc:11:10: fatal error: numpy/arrayobject.h: No such file or directory
onnxruntime-aarch64-unknown-linux-gnu>    11 | #include <numpy/arrayobject.h>
onnxruntime-aarch64-unknown-linux-gnu>       |          ^~~~~~~~~~~~~~~~~~~~~
onnxruntime-aarch64-unknown-linux-gnu> compilation terminated.
onnxruntime-aarch64-unknown-linux-gnu> /build/source/onnxruntime/python/onnxruntime_pybind_state.cc:10:10: fatal error: numpy/arrayobject.h: No such file or directory
onnxruntime-aarch64-unknown-linux-gnu>    10 | #include <numpy/arrayobject.h>
onnxruntime-aarch64-unknown-linux-gnu>       |          ^~~~~~~~~~~~~~~~~~~~~
onnxruntime-aarch64-unknown-linux-gnu> compilation terminated.
onnxruntime-aarch64-unknown-linux-gnu> make[2]: *** [CMakeFiles/onnxruntime_pybind11_state.dir/build.make:90: CMakeFiles/onnxruntime_pybind11_state.dir/build/source/onnxruntime/python/onnxruntime_pybind_iobinding.cc.o] Error 1onnxruntime-aarch64-unknown-linux-gnu> make[2]: *** [CMakeFiles/onnxruntime_pybind11_state.dir/build.make:188: CMakeFiles/onnxruntime_pybind11_state.dir/build/source/onnxruntime/python/onnxruntime_pybind_state.cc.o] Error 1
onnxruntime-aarch64-unknown-linux-gnu> /build/source/onnxruntime/python/onnxruntime_pybind_sparse_tensor.cc:11:10: fatal error: numpy/arrayobject.h: No such file or directory
onnxruntime-aarch64-unknown-linux-gnu>    11 | #include <numpy/arrayobject.h>
onnxruntime-aarch64-unknown-linux-gnu>       |          ^~~~~~~~~~~~~~~~~~~~~
onnxruntime-aarch64-unknown-linux-gnu> compilation terminated.
onnxruntime-aarch64-unknown-linux-gnu> make[2]: *** [CMakeFiles/onnxruntime_pybind11_state.dir/build.make:174: CMakeFiles/onnxruntime_pybind11_state.dir/build/source/onnxruntime/python/onnxruntime_pybind_sparse_tensor.cc.o] Error 1
onnxruntime-aarch64-unknown-linux-gnu> /nix/store/hmjkyxmnr2nc6mbq3r8cwwg552j1vqaf-aarch64-unknown-linux-gnu-binutils-2.41/bin/aarch64-unknown-linux-gnu-ld: /nix/store/pl0jivzg5w84j8pbkpg9dmc3gg4rz7rd-protobuf-21.12/lib/libprotobuf-lite.so.<IP_ADDRESS>: error adding symbols: file in wrong format
onnxruntime-aarch64-unknown-linux-gnu> collect2: error: ld returned 1 exit status
onnxruntime-aarch64-unknown-linux-gnu> make[2]: *** [CMakeFiles/onnxruntime.dir/build.make:168: libonnxruntime.so.1.16.3] Error 1
onnxruntime-aarch64-unknown-linux-gnu> make[1]: *** [CMakeFiles/Makefile2:1277: CMakeFiles/onnxruntime.dir/all] Error 2
onnxruntime-aarch64-unknown-linux-gnu> make[1]: *** Waiting for unfinished jobs....
onnxruntime-aarch64-unknown-linux-gnu> make[1]: *** [CMakeFiles/Makefile2:1365: CMakeFiles/onnxruntime_pybind11_state.dir/all] Error 2
onnxruntime-aarch64-unknown-linux-gnu> make: *** [Makefile:166: all] Error 2
error: builder for '/nix/store/91h4j7fz7m8qknsfh841hrc9g2m3vcg7-onnxruntime-aarch64-unknown-linux-gnu-1.16.3.drv' failed with exit code 2;
       last 10 log lines:
       >       |          ^~~~~~~~~~~~~~~~~~~~~
       > compilation terminated.
       > make[2]: *** [CMakeFiles/onnxruntime_pybind11_state.dir/build.make:174: CMakeFiles/onnxruntime_pybind11_state.dir/build/source/onnxruntime/python/onnxruntime_pybind_sparse_tensor.cc.o] Error 1
       > /nix/store/hmjkyxmnr2nc6mbq3r8cwwg552j1vqaf-aarch64-unknown-linux-gnu-binutils-2.41/bin/aarch64-unknown-linux-gnu-ld: /nix/store/pl0jivzg5w84j8pbkpg9dmc3gg4rz7rd-protobuf-21.12/lib/libprotobuf-lite.so.<IP_ADDRESS>: error adding symbols: file in wrong format
       > collect2: error: ld returned 1 exit status
       > make[2]: *** [CMakeFiles/onnxruntime.dir/build.make:168: libonnxruntime.so.1.16.3] Error 1
       > make[1]: *** [CMakeFiles/Makefile2:1277: CMakeFiles/onnxruntime.dir/all] Error 2
       > make[1]: *** Waiting for unfinished jobs....
       > make[1]: *** [CMakeFiles/Makefile2:1365: CMakeFiles/onnxruntime_pybind11_state.dir/all] Error 2
       > make: *** [Makefile:166: all] Error 2
       For full logs, run 'nix log /nix/store/91h4j7fz7m8qknsfh841hrc9g2m3vcg7-onnxruntime-aarch64-unknown-linux-gnu-1.16.3.drv'.

I'll try playing around with the inputs a bit.

Just a quick update on this - i tried moving numpy and protobuf around to no avail, and their current location is probably where they should be according to the docs.

How about explicitly adding host's -I${numpy}/${python.sitePackages}/numpy/core/include/ (et cetera) to NIX_CFLAGS_COMPILE

Sorry, I was banned for 6 weeks, unable to respond.
This is a bit tricky, as python packages have two feet: one in the depsBuildHost and the other in the depsHostHost world. Not sure how python packages should look when both compiling against them and using them for c-style linking

One initial thought, would be to add an additional dev output, which then should get picked up by the default build to add them as a NIX_CLFAGS_COMPILE, and then the headers should at least be available. Linking of pre-built artifacts is bit trickery (e.g. libnumpy.so if it exists).


ModuleNotFoundError: No module named 'yacs'

I follow the manual the  https://github.com/lzccccc/SMOKE

but this code error occur $ python tools/plain_train_net.py --eval-only --config-file "configs/smoke_gn_vector.yaml"

Hey, I am getting the same issue, how did you solve it?

I can solve this  problem by using anaconda  environment
2021년 2월 20일 (토) 오전 3:49, Saurabh<EMAIL_ADDRESS>작성:

Hey, I am getting the same issue, how did you solve it?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/lzccccc/SMOKE/issues/25#issuecomment-782270084, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/APVIQTOELYUAYSTUE6464KLS72XFHANCNFSM4PBY6OAQ
.

solved it using "pip install yacs"


Mark newly added permission

Currently the only way to see which permission was added is by remembering which the last one was and see which are below it. It would be nice to indicate this a bit better, e.g. via a different colored background or some icon?

Yep, agreed.
Maybe we could take some inspiration from the way GitHub handles things.

Yeah, that would probably look nice too.


Please release and add a version tag

We want to load arcgis_hybrid.json in plotly.js source code
https://github.com/plotly/plotly.js/blob/34cade7e861435b4fbbb3005de72646008dc2754/src/plots/map/constants.js#L13
but to have stable results I prefer to load a version tag instead of a file that may possibly change in future.
Please let me know if you could possibly add a tag on your main branch?
If not we may use last commit hash.
Thank you @go2garret
cc: @birkskyum @emilykl @ndrezn @gvwilson

Greetings, I have released a stable version of the package here:
https://github.com/go2garret/maps/blob/v1.0.0/src/assets/json/arcgis_hybrid.json
Cheers :)

Thanks @go2garret!
This is great for now.
I am wondering if you want to publish a npm package?


KeyError in tf.contrib.graph_editor.graph_replace

When applying graph_replace to graphs containing ops with the _original_op attribute, it can fail with a KeyError. The error occurs in Transformer._copy_ops when trying to copy an op whose _original_op has not yet been copied. The ordering of ops that are copied is not deterministic so this error pops up somewhat randomly.
The _original_op attributes appear to be created by tf.gradients to point back to the op from the forward pass.
Example code snippet (note: you may need to run this multiple times to get a failure):
import tensorflow as tf
graph_replace = tf.contrib.graph_editor.graph_replace
w = tf.Variable(0.0, name="w")
y = tf.multiply(tf.multiply(w, w, name="mul1"), w, name="mul2")
g = tf.gradients(y, w)[0]
g_new = graph_replace(g, {w.value(): g})

Error:
/usr/local/lib/python2.7/dist-packages/tensorflow/contrib/graph_editor/transform.py in transform_op_if_inside_handler(info, op, keep_if_possible)
    122   """
    123   if op in info.sgv.ops:
--> 124     return info.transformed_ops[op]
    125   else:
    126     if keep_if_possible and info.graph is info.graph_:

KeyError: <tf.Operation 'mul1' type=Mul>

I see three possible fixes:

Remove _original_op attributes in the copied graph (I don't see anywhere in the TF codebase where it is used)
Move the creation of the _original_op attribute from the copy_op_handler function to the end of Transformer._copy_ops after all ops have been copied.
Topologically sort the ops being copied so that ops that are _original_op attributes are created before their children.

My implementation of option 2 seems to fix this problem, but I might be missing something about the usage of _original_op.

@purpledog Here's an interesting graph editor issue for you. If you need toposort I wrote a Tarjan implementation in Python I can give you. Also get_walks_intersection_ops looks quadratic.

The similar issue still persist when variable is matix:-
import tensorflow as tf
graph_replace = tf.contrib.graph_editor.graph_replace
w = tf.Variable(tf.zeros([784, 784]),name="w")
y = tf. matmul(tf.matmul(w, w, name="mul1"), w, name="mul2")
g = tf.gradients(y, w)[0]
g_new = graph_replace(g, {w.value(): g})
return info.transformed_ops[op]

KeyError: <tf.Operation 'mul1' type=MatMul>

I'm happy to submit a PR using my implementation of the proposed second option. Please let me know whether that would be helpful as graph_replace was the only way of copying and replacing subgraphs. It would be great to have this working again, especially given that other PRs for similar features have been rejected (#5802).

Hi,
What's the current status of this bug as of r1.4?
Thanks!

@gokul-uf it still exists. This is the same issue as https://github.com/tensorflow/tensorflow/issues/9978 . Current work-around is to modify graph_editor source locally although PR would be welcome

It has been 14 days with no activity and this issue has an assignee.Please update the label and/or status accordingly.

Closing out this issue, as tf.contrib is no longer supported in TensorFlow 2.x.


I can't reproduce this (using pow 0.3.2). Can you upgrade it and try with the latest version?

I doesn't happen in pow 0.3.2, thanks.

:+1: