Nft add element filter = No such file or directory

I'm not an nftables expert, but the "nft add element filter" command appears to fail with nftables vv1.0 6. Nftables wants a table name, such as "nft add element inet filter"

Hi,
That shouldn't be the full command the code run. A part is missing.
Can you paste your custom config file you use ?
Can you run the programm manually with the parameter /opt/nft-dns/nft-dns.py --verbose to have more output ?

2024-01-11 16:44:40,798 INFO:Reading config file : /etc/nft-dns.conf
2024-01-11 16:44:40,799 INFO:Only config file with prefix .conf is read
2024-01-11 16:44:40,799 INFO:Reading config directory : /etc/nft-dns.d
2024-01-11 16:44:40,799 INFO:# Parsing the configuration
2024-01-11 16:44:40,799 DEBUG:set_name='IPsAapt4' typeof=4 fqdn='deb.debian.org' ip_list=None ttl=None next_update=None
2024-01-11 16:44:40,799 DEBUG:set_name='IPsAapt4' typeof=4 fqdn='security.debian.org' ip_list=None ttl=None next_update=None
2024-01-11 16:44:40,800 DEBUG:Command to run : nft list set filter IPsAapt4
2024-01-11 16:44:40,805 ERROR:
2024-01-11 16:44:40,805 ERROR:Error: No such file or directory
list set filter IPsAapt4
^^^^^^
Traceback (most recent call last):
File "/opt/nft-dns/nft-dns.py", line 191, in
main()
File "/opt/nft-dns/nft-dns.py", line 172, in main
read_config()
File "/opt/nft-dns/nft-dns.py", line 75, in read_config
if "ipv4_addr" in res or "ipv6_addr" in res:
^^^^^^^^^^^^^^^^^^
TypeError: argument of type 'NoneType' is not iterable
I'm working from the command line to troubleshoot at the moment. There is no custom config file.
Your command is this:
run_command(f"nft add element filter {one_entry.set_name} {{{', '.join([str(ip) for ip in one_entry.ip_list])}}}")
Which should create a statement like:
nft add element filter blackhole { <IP_ADDRESS> }
This results in the error:
Error: No such file or directory; did you mean table ‘filter’ in family inet?
add element filter blackhole { <IP_ADDRESS> }
The proper command that is successful is:
nft add element inet filter blackhole { <IP_ADDRESS> }
I'm using the current nftables version
nftables v1.0.6 (Lester Gooch #5)

From the v1.0.6 , the command is now : nft list set ip filter IPsAapt4  (ip and filter depend of your config.
Can you output the result of nft list sets I think I need the rework the logic of my checks.

nft list sets
table inet filter {
set blackhole {
type ipv4_addr
flags interval
elements = { <IP_ADDRESS> }
}
}

The correct command would be:
nft list set inet filter blackhole
table inet filter {
set blackhole {
type ipv4_addr
flags interval
elements = { <IP_ADDRESS> }
}
}
It would appear that there should be two additional configurations in nft-dns.conf: 'table' and 'family'
as we could need to resolve FQDNs in the output table or even a custom table.
Also, there are 6 possible families. In my case, my current table family is inet, but it could be ip, ip6, etc.

wait, If you want fnt-dns to fill the blackholenamed set, you need to add this information into the config : https://github.com/azlux/nft-dns/blob/main/MANUAL.md#other-sections set_name part.
According to your log, currently, the script try to fill the set named IPsAapt4. And it don't exist, that why nft say the command isn't found.
TODO : I need to make an check improve to have a better error.

Sorry for the confusion - I was testing multiple scenarios and crossed the examples.
My current nftables configurtion:
table inet filter {
   set IPsApt4 {
      type ipv4_addr
      flags interval
      elements = { <IP_ADDRESS> }
   }
   ...

My current nft-dns configuration:
[GLOBAL]
#max_ttl = 86400
#min_ttl = 300
#mode = capture
#custom_resolver = "<IP_ADDRESS>"
#dry_run = False
#verbose = False
include_config_dir = /etc/nft-dns.d/

[test]
set_name = IPsApt4
enable = true
typeof = ipv4
domains = deb.debian.org, security.debian.org

foo@bar:~$ /opt/nft-dns/nft-dns.py --verbose
2024-01-12 13:55:06,450 INFO:Reading config file : /etc/nft-dns.conf
2024-01-12 13:55:06,451 INFO:Only config file with prefix .conf is read
2024-01-12 13:55:06,451 INFO:Reading config directory : /etc/nft-dns.d
2024-01-12 13:55:06,451 INFO:# Parsing the configuration
2024-01-12 13:55:06,452 DEBUG:set_name='IPsApt4' typeof=4 fqdn='deb.debian.org' ip_list=None ttl=None next_update=None
2024-01-12 13:55:06,452 DEBUG:set_name='IPsApt4' typeof=4 fqdn='security.debian.org' ip_list=None ttl=None next_update=None
2024-01-12 13:55:06,452 DEBUG:Command to run : nft list set filter IPsApt4
2024-01-12 13:55:06,478 ERROR:
2024-01-12 13:55:06,478 ERROR:Error: No such file or directory
list set filter IPsApt4
         ^^^^^^

Traceback (most recent call last):
  File "/opt/nft-dns/nft-dns.py", line 191, in <module>
    main()
  File "/opt/nft-dns/nft-dns.py", line 172, in main
    read_config()
  File "/opt/nft-dns/nft-dns.py", line 75, in read_config
    if "ipv4_addr" in res or "ipv6_addr" in res:
       ^^^^^^^^^^^^^^^^^^
TypeError: argument of type 'NoneType' is not iterable

foo@bar:~$ nft list sets
table inet filter {
        set IPsApt4 {
                type ipv4_addr
                flags interval
                elements = { <IP_ADDRESS> }
        }
}

foo@bar:~$ nft list set filter IPsApt4
Error: No such file or directory
list set filter IPsApt4
         ^^^^^^

foo@bar:~$ nft list set inet filter IPsApt4
table inet filter {
        set IPsApt4 {
                type ipv4_addr
                flags interval
                elements = { <IP_ADDRESS> }
        }
}

foo@bar:~$ nft add element filter IPsApt4 { <IP_ADDRESS> }
Error: No such file or directory; did you mean table ‘filter’ in family inet?
add element filter IPsApt4 { <IP_ADDRESS> }
            ^^^^^^

foo@bar:~$ nft add element inet filter IPsApt4 { <IP_ADDRESS> }
foo@bar:~$

foo@bar:~$ nft delete element filter IPsApt4
Error: syntax error, unexpected newline, expecting '{' or '$'
delete element filter IPsApt4
                             ^

foo@bar:~$ nft delete element inet filter IPsApt4 { <IP_ADDRESS> }
foo@bar:~$

Ok, I see now, something have change.
So I will make an update, maybe next week because I don't have my computer free this week-end.
If you know python3, you can make a PR, the issue is here : https://github.com/azlux/nft-dns/blob/main/nft-dns.py#L73-L79
When reading the config file, I check if the named set exist or I exit on error.
This need to be change because nftables now need the "table" name to add/update/delete an element into a "named set". Maybe an additional option, or searching it with the command "nft list sets"

I can code it, but no sense to reinvent the wheel when you did a perfectly fine job. I agree with the lines you provided,  but also Lines 127 and 130 have to be formatted with the table family. Thanks!

Can you try the new release, the config have change a little.
https://github.com/azlux/nft-dns/releases/tag/0.4
I don't use typeof config, but family to catch the good table.

That looks to be working great, now. Thank you! If I could make another suggestion, add the table as another configuration option. Maybe default to filter, but sets can also be in output, forward, or custom tables.

will be added with the commit https://github.com/azlux/nft-dns/commit/bf5e55a08a8b76c6561897981f413186944f021e can you test the next release ?


Feature Request for Future: Remote-SSH X11 Forwarding for Debugging mode

This is gonna be a feature request for the upcoming spyder Probaby a year or so from now.
I know that you guys are working on the feature that can make spyder be able to connect to the remote server as a remote ide.
It would be nice if when that feature is included also have the features that make us be able visualize while debuging as we can do it now in spyder (eg visualize images and stuff).
I think they call it Remote-SSH X11 Forwarding for display, TBH spyder has a very nice debugging set up and in the future if we have remote-server option it would be awesome to has the ability of displaying images while debugging.
I had to switch to vscode due to remote-server limitation of spyder, but if one day we have it in spyder i will switch.
vscode still suffer from Remote-SSH X11 Forwarding and display images while debugging, so it would be nice if you guys dont have this issue in the future...
Thanks

Hi @seyeeet thanks for opening this. What do you think about this @spyder-ide/core-developers

Sorry @seyeeet but this is not something we can handle in Spyder. It requires special configurations both in the remote server and in your local machine, and it's not easy to setup (at least that's what I remember when I tried to do it).

if we have remote-server option it would be awesome to has the ability of displaying images while debugging

For that you can use now the Plots pane and static images. For Spyder 6 we plan to add a new pane to visualize web content, so then you'll be able to use the Matplotlib web backend, which has the same features as the Automatic one.


Inference speed

Hi, thanks for the excellent work!
I'm trying to use DAC (24kHz) as a replacement of EnCodec for some audio tasks, however I'm noticing a significant slowdown of the training process.
I did some quick measures of the inference speed and it's around 4x slower than EnCodec. Is this an expected behaviour?

Could you explain what and how you are training?


Possibility of changing 'ceiling' to 'round' in EXT-X-TARGETDURATION algorithm

Hello,
I would like to propose changing the algorithm that determines the value of EXT-X-TARGETDURATION in HLS playlists from a ceiling round to a regular round, the relevant code is here: https://github.com/grafov/m3u8/blob/6ab8f28ed427e7ab787340596709e27ecc4b32db/writer.go#L307-L309
The reason is that Apple changed their recommendation for EXT-X-TARGETDURATION in revision 12 of the HLS spec to be greater than or equal EXTINF, when rounded to the nearest integer, see https://tools.ietf.org/html/draft-pantos-http-live-streaming-12#section-3.4.2
You can read the previous spec here: https://tools.ietf.org/html/draft-pantos-http-live-streaming-11#section-3.4.2
The current algorithm, which does a ceiling, does create playlists that conform to the spec.
BUT it can also arbitrarily create playlists with 11s target durations when the user requests 10s (when it would be perfectly valid to have a 10s target duration). Generating playlists that exactly match the spec recommendation is preferable for us, and can also help with scrubbing accuracy on a wider range of players.
I've opened this issue as more of a discussion because I understand that there might be some pushback given that the current behavior technically creates spec-compliant playlists. If there is any chance of this being activated as a feature flag this would also be acceptable for us.
I will submit a preliminary PR shortly that implements this.

Would you consider fixing this at least for reading?
For example, the current implementation parses the above playlist as having a target duration of 5, which is different from the actual value in the playlist.
In my mind, if you think the playlist is incorrect, you could maybe return an error if the strict parsing option is turned on, but you should never modify the playlist you are reading.
When I was trying to find the code responsible for this, I was a bit surprised that it was in writer.go even though I was reading a playlist.


Big AGI connection to UM GPT

We are trying to connect our private University of Michigan GPT(api type is azure) to big AGI but we cant seem to connect it. We tried running bigAGI both locally as well as on vercel. We were able to confirm that the UM GPT endpoint and api key work through a different application. Here are the errors we get when trying to connect using a locally ran big AGI. We tried both azure and open ai

If anyone has any experience connecting a third party gpt, any feedback would be appreciated

@Rithikb123 does it go through a proxy? if you open the URL (in yellow) manually from Chrome, can you show me what happens and the network request?

This is what I see when I open the URL. However, I was able to confirm that the URL works through the swift app

There seems to be a discrepancy of the Azure API. Do you know if the swift app has hardcoded models, or does it dynamically refresh the list? That could be the issue.
We need to understand what's the API server, doesn't seem a clean Azure. Would need a detailed analysis of which URLs are hit, and if there are differences in the headers, when the Swift UI does it.
Another option is for you to set a custom "OpenAI" server to your address and with your key, and see if that works.

This is how it looks in the swift app. We have tried both this URL as well as the one in the previous screenshot and both give the same responses. What do you mean by setting a custom OpenAI server. Do you mean on BigAGI or to set it seperately and then connect it to BigAGI later?

Thanks for the details. Please ignore the suggestion about the custom OpenAI server, the Azure protocol is correct.

I see one field called organization: is this field passed as a header parameter by any chance? Because in that case, I think that our implementation could be missing the parameter.

Big-AGI works by first listing the models, and then using the model on the users' chats. In the screenshot I bought I see that the request gets sent already to a model ("gpt-35-turbo"). I wonder if maybe listing models is not supported by the UM server.


UltraPermissions Support (API Included!)

Please add support to transfer from uPerms to LuckPerms.
https://raimu.tried-to-do.science/i/wthsgqh3.png is a screenshot from the owner of uPerms saying how to use their API.

This has been brought up before. The issue being that the author of UltraPerms doesn't understand why the API needs to be public. And since it is not, it won't happen.

This is easier said than done.
The UltraPermissions API is not a public thing like BrainStone said and afaik do you need to purchase the plugin in order to actually get access to the API.
So as long as the API won't be made public, the migration can't be added.

^


C#: ExecuteRectClipLines (aka RectClipLines, previously) seems to be broken in 1.2.2

Discussed in https://github.com/AngusJohnson/Clipper2/discussions/529

Originally posted by GeneralGDA May 22, 2023
In the 1.1.1 NuGet package everything is OK. I updated to the 1.2.2 NuGet and now RectLicpLines returns the rect boundaries. For example, let's take the rect with min (0, 0), and max (8, 8). And clip with it a line: (4, 4), (4, 5), (9, 5), (9, 4), (5, 4)
I expect the output contains two separate lines. But the 1.2.2 clipper returns single line. It includes the rect boundary.

This is a C# only bug, and here's the fix:
Replace this line with ...
          Add(ip, true);

Should be fixed now.


Can't see [auto] in output of sudo mopidyctl config

As the title says, I installed the package, but it doesn't show in the active configuration.
Has anyone had any luck setting this up recently? The latest updates seem to have been made a long time ago...

By the looks of it, it hasn't been updated to work with python3/mopidy3

Hi, if you haven't gotten it working with mopidy3, you can see a few simple edits I made to get basic functionality working in this issue: https://github.com/gotling/mopidy-auto/issues/18#issue-665628855. I'm hoping a python coder could do an update for us. It does seem like the simple changes/hacks I made mostly fixed it so hopefully it wouldn't take an expert long to do it correctly.


[DNM] west.yml: Test update of mcuboot revision for PR  948

The commit updates mcuboot revision to:
https://github.com/mcu-tools/mcuboot/pull/948
for the purpose of PR testing and review.
The tested PR changes messages printed by mcuboot to indicate Direct-XIP mode and remove redundant and always displayed message about booting into "primary" slot.
Signed-off-by: Dominik Ermel<EMAIL_ADDRESS>

The tested mcuboot PR has already been merged.


[Code Gallery] Migrate Simple Calculator MEF Application

[Code Gallery] Simple Calculator MEF Application
Sample URL
<docs\framework\mef\index.md>
Sample information
38,931 downloads, with 5.0 rating (6 reviewers)
Usage (in GitHub repos)

docs\framework\mef\index.md

Is this a duplicate of #13733?

Yes it is a duplicate. I have no idea of how that might have happened. I'll close this one.


Add word-break to repo description in home page

In #25315, @denyskon fixed UI on mobile view.
But for the repo description, on desktop view there's no word-break.
So maybe we can just add gt-word-break to fix it on both mobile view and desktop view.
Before:
desktop view:

mobile view:

After:
desktop view:

mobile view(almost same?)

Did some misc fixes for repo indexer search box:


Large GitHub Runners

This repo uses large GitHub runners:
https://github.com/pulumi/pulumi-azure/blob/6c88b7f781911804ee33c08791020982af1ea6d7/.ci-mgmt.yaml#L27-L30
If possible, we should remove the large runners to reduce cost.

fixed in https://github.com/pulumi/ci-mgmt/pull/827


Implement per-thread cache context

What changes are proposed in this pull request?
Implement per-thread cache context
Why are the changes needed?
Enable the fine-grained cache admission
Does this PR introduce any user facing changes?
no

The change LGTM, thanks for working on this!

It seems that the test LocalCacheFileInStreamTest need to be fixed.

alluxio-bot, merge this please.


Add invalidation client interface

Add the client interface to mark a path as needing synchronization with the UFS.
void invalidateSyncPath(AlluxioURI path) throws AlluxioStatusException

alluxio-bot, merge this please


Clean up legacy/migration code in workspaces

I think during support for remote in workspaces land, many properties got added and some deprecated. I am having a hard time seeing through:

I suggest to make a cleanup pass, there maybe more...

Yes, we can remove the old versions now.

Thanks 🙏

Unclear to me what to verify here?

I guess just check that the code is gone. Or is there a way to say that an issue doesn't need verification?

If the issue is not marked as "feature request" or "bug" it should be fine.


Support loading Tram-Lite in a deferred way

Summary
In many cases, developers may want to load Tram-Lite after getting the initial template loaded on the page (thus, progressively enhancing those elements after the DOM has loaded). This can be done using the defer attribute, or from including the script at the end of the <body> tag.
<script defer src="https://unpkg.com/tram-lite@4"></script>

See Codepen Example: https://codepen.io/JRJurman/pen/xxmoeBp
We should update the template processing such that tl-definition templates already on the page are processed as expected.

This has now been resolved in v4.2.1 - https://codepen.io/JRJurman/pen/wvRVxjP


feat: improve search controls

Improves labels
Moves area search from a dropdown into an autocomplete field.

closing as I need to apply some small fixes

fixes pushed :smile:

Advanced search option added and search state set with default values so updates are reflected.


Performance tweaks

It helps having a 10 year old Mac ;-) I wouldn't say QS is 'slow', but it could definitely be snappier.
These changes are based on profiling using Instruments. Overall searching should be slightly faster, displaying action should be faster.
I propose this isn't merged into 2.0.0, but saved for a later release. We've got enough to work on right now.

From one ten-year-old-Mac owner to another, cheers to you, good sir.


sql: make some tweaks to using the vectorized engine

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

experimental_on option for vectorize session variable has been renamed to on. The only things that will not run with auto, but will run with on, are unordered DISTINCT and two window functions (percent_rank and cume_dist). The two options are otherwise identical. [#46080][#46080] {% comment %}doc{% endcomment %}

vectorized execution https://airtable.com/tblD3oZPLJgGhCmch/viwNg2fK3NU3DUY6Z/reckMh49RgMbRHhDs
https://airtable.com/tblD3oZPLJgGhCmch/viwNg2fK3NU3DUY6Z/recagjuRqKN5Jxh1n


Review the need for Null* instances of RVs.

Is NullProcess still used? It seems to be completely identical to NullVariable. Artifact of when there were RandomProcesses that were separate/distinct from RandomVariables?

Originally posted by @dylanhmorris in https://github.com/CDCgov/multisignal-epi-inference/pull/262#discussion_r1690470316

Partially addressed in https://github.com/CDCgov/PyRenew/pull/441. We plan to remove NullObservation when we get rid of sub-models in the future.


Readme revamp

Please updated changes-notes related thigs within CONTRIBUTING.md

I see no reason to keep (and eventually maintain) both pz_sample_workflow.gif and pz_sample_workflow_shorter.gif.
We need only one ss/gif for pz.ksi.
The workflow going on within the pz_sample_workflow_shorter.gif is too fast :ultrafastparrot:
A viewer (esp. someone who do not know git machete) hardly see what happens there.
There is no chance to read all context menu actions, take a look into dialogs, etc.
Let's simplify this chaos.

present very simple and real life workflow - IMHO the most common is rebase & push of a SINGLE branch
do not show reset (as it presents an info dialog which would extend the gif significantly and unnecessarily)
the same reasoning applies to the slide in and the override fork point
go through the workflow SLOWLY (make sure that the average viewer will be able to read what is to be read, and see what is to be seen)
do not show opened machete file - don't you think it may confuse a new user? Let's keep it simple for him'her
use dark theme (I know it is used now, just want to underline it), it seems to look better on the PZ page

We must remember that we are not able to present everything.
Hence let's present only a part but do it professionally.
Note that a viewer will have a chance to see all the context menu actions (not performed ones in particular but it will work as a "teaser").

Whoops... lemme take a look into the failing UI tests...

him'her

@mkondratek that's not inclusive enough :scream:
As of 2020, they/them is the golden standard :1st_place_medal:

2020-08-12 11:25:29,926 [   6567]  ERROR - pplication.impl.LaterInvocator - Already disposed: Project (name=machete-sandbox, containerState=DISPOSE_COMPLETED, componentStore=/tmp/machete-tests-3460733527793669338/machete-sandbox)

@PawelLipski yeah, it happens time to time... :/

2020-08-12 11:25:29,926 [   6567]  ERROR - pplication.impl.LaterInvocator - Already disposed: Project (name=machete-sandbox, containerState=DISPOSE_COMPLETED, componentStore=/tmp/machete-tests-3460733527793669338/machete-sandbox)

@PawelLipski yeah, it happens time to time... :/

#508

Don't you think this big logo looks weird?

Yeah, maybe indeed would be better to scale down somehow (not sure how to do it in GH markdown for an SVG)

Ok, so maybe something like that would be ok:

:question:


I want to get explicit mac address of devices on iOS

I want to get explicit mac address of devices on iOS when I scan nearby devices on my phone and store the paired devices locally. But based on the discussion in https://github.com/Polidea/react-native-ble-plx/issues/92, the Device.id on iOS is

on iOS it will be autogenerated UUID

I want to get the explicit mac address somehow, is it possible for your team to add this attributes to iOS?

One possibility is to add a Device Information service with a System ID characteristic.
react-native-ble-plx could probably read it "behind the scenes" once connected but it would be an "auto-magic" feature - probably not desirable.

I'm going to get my bluetooth MAC to verify to the server that it's legitimate and I don't know what to do
I sorry for my English

@1459726992 I have same problem. I need to access MAC address of peripheral devices to verify with the server. Did you solve it?

Did you solve it?

Hello!
Has the error been fixed? or is there a workaround?


redraw scene on focus change

When another window moves over the scenic scene, the window gets left in a strange state until the next
push_graph/1. One solution would be to  expose events to the developer to push_graph if they want, or just have it redraw every time focus is regained.
One way i'm able to reproduce this is to take a screenshot of the scenic window.

What OS are you running?

 If this is just a problem with my setup feel free to close. I assumed this applied to all operating systems/setups.

Pretty sure this is an arch linux thing. I don't see the behavior on my Mac. I think it was good under Ubuntu too. I'll try under Debian later.

Same happens to me as well (Funtoo Linux). Looks like that Linux version of scenic have still lots of bugs to solve. :smile:

I did not even noticed when it has been solved. Let's see if same happens to @ConnorRigby and if so then for me this could be closed.

That would be great


Traffic Shadowing: Redirect calls to a second evironment

Send a request also to a different Target but ignore the response:
<target ...>
    <shadow host="..." port="..."/>
</target>

Seems to be done in combination with weighted routing. Another interesting common variation is traffic replay, where traffic gets recorded in production to be replayed in test environments


Server to support changes to format of LocalProtocol and RemoteProtocol values

Fixes #524

Reviewer is @DanaSunal

I see, I missed that the enum is different, my bad.


fix typo

from https://github.com/joyent/node-asn1/pull/13

I changed it to use Array.isArray

You should be able to merge this.


The API key you provided does not exist.

Almost all API methods return an error
{"title":"The API key you provided does not exist." }
Working only https://api.stubhub.net/v2/user
What have you changed? How to fix the situation?

Try to prepend Bearer  to the Authorization header, like Bearer ey...

Try to prepend Bearer  to the Authorization header, like Bearer ey...

That's what I do. Only works for me https://developer.viagogo.net/api-reference/account#operation/User_Get
But it doesn't work, I need a method
https://developer.viagogo.net/api-reference/inventory#operation/SellerListings_GetSellerListings

The problem started about a month ago

What's the scope of the token.

What's the scope of the token.

I have the necessary scopes. Everything worked before, but about a month ago it stopped. Nothing has changed on my part


Release 7.1.0

Signed-off-by: Tim Smith<EMAIL_ADDRESS>Description
[Describe what this change achieves]
Issues Resolved
[List any existing issues this PR resolves]
Check List

[ ] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable
[ ] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

I suspect this is a mistake since we already released this version


Add --skip-hooks flag for ddev start, restart, etc.

Is your feature request related to a problem? Please describe.
My goal is to always have my commands executed under DDEV. So for example I don't want to run composer install, but instead ddev composer install. To do so I need my project to be started.
However, I often have my config.local.yaml defined with post-start hook that will drush site-install. So what happens is, on first install, it will start the container, try to install (and fail because composer didn't run yet), and then it will  composer install.
I was able to workaround it, with some trickery. Specifically:
ddev composer install
cp .ddev/config.local.yaml.example .ddev/config.local.yaml
ddev restart

Describe the solution you'd like
Instead of this late copy, or having to delete the config.local.yaml, it could be convenient to do something like:
ddev start  --skip-config-hooks
ddev composer install

# Restart, to run the post-start hooks.
ddev restart

Makes sense. We may be able to come up with a more elegant solution as we think about it. But I find hooks that execute when I don't need them annoying, so this might be a good path.

I would also love to have the possibility to skip the pre and post start hooks. 3 flags would be nice and cover all needs imho:

--skip-hooks
--skip-pre-hooks
--skip-post-hooks

How about just --skip-hooks, global ? The more stuff we add, the harder it is for everybody to understand.  And of course... people can just comment out the hooks.
@amitaibu The easier thing for what you're doing is to make the hook smart enough to do what's needed. For example, load a db if needed, run composer install if needed, etc. Like this one

Just thought about this and found this issue!
@rfay I think adding a global --skip-hooks would be great.
I often manually disable the hooks to save time when I restart the project and know that the hooks don't need to run (especially when restoring a database snapshot).

Having the ability to skip hooks is something we would like to have.
In my situation we use "ddev import-db --src=db.sql" to import db BEFORE running 'ddev start -y'.
But we see from the logs that ddev import-db triggers the post-start hook and ends with error.
We need to run ddev import-db without triggering hooks.
Thanks

Of course, you can make your hooks less fragile. In your case @karpa you could check for existence of db before running hooks, as in https://github.com/drud/ddev-contrib/tree/master/hook-examples/import-db-if-empty

Thanks for pointing out the possible workaround @rfay! Actually we have made a similar workaround ourselves (based on our app configuration). But having the proposed solution of this issue would be better.

+1 for this feature. I could use this for disabling post-import-db hooks.

+1
I could use this very much.
We do sanitation after an ddev pull.
But when syncing acceptance or pre-prod environments I have to comment out the sanitation step, what I always forget and later regret.
@rfay did you have a start-commit already somewhere? I want to contribute a start for the feature somewhere this weekend

@MakerTim I think the first thing to think about is whether you could make your hook less fragile. Could you test for whether it's already sanitized? Could you use an environment variable or something to control it? I'm afraid something like --skip-hooks will still cause you trouble because you'll forget it just the same way.
And +1 for doing the sanitization!
Doing it upstream would be more robust and less prone to failure though...

+1 for global --skip-hooks
my use case would be to do an initial database import without our organization-wide default pre-import-db hook which executes a ddev snapshot:
ddev --skip-hooks import-db --file=db/dump.sql.gz

+1 for global --skip-hooks

+1

You might want to put the hooks in a config.hooks.yaml and link/unlink the extra file.

My usecase lately is to sync pre-production to production. and backwards when the customer wants to do a lot of changes at once. They do a content stop on production and do everything to a pre-production.

I assume you're talking about some database processing you're doing, in which case you could make a smarter hook that respected an environment variable or whatever.


push dropback model stuff

Eventually I'll move the helper_* to nflfastR but just want to merge this stuff in before removing this repo's history just in case


Written solution and added a note

Written solution and added a note in the absence of which, I was confused

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

Unfortunately, this change is coming from an unknown branch, so I'll have to close this. Please feel free to open another PR in the future!


Small optimisation

Very small optimisation and a quick tidy up before the issue based PRs

I'm actually adding more commits now, I just wanted you to have the PR early. I simplified the plotting code in the most recent commit. I'll add more info to this PR as I make commits.
(this is a bit round the houses but the best I can think of without having write access to the repo)

Now I've addded tooltips and simplified the rest of the plotting code

Placed the inputs on the same row. They're a bit squashed now and I think the whole side panel needs widening, I'll come back to that

Closes #7 , #3
Addresses some of #4


Custom referrer mapping seed

Describe the feature
Allow the referrer mapping seed to be swapped with a custom one by an environment variable.
Who will this benefit?
Anyone with domain specific referrers not mentioned in the current referrer mapping.
Are you interested in contributing this feature?
Yes!

https://github.com/dbt-labs/segment/pull/84

Thanks @fred-boink - this package has been adopted by https://github.com/fleetio/dbt-segment so please move your issue/PR there if you'd still like to contribute it


Define metainfo for all DMatrix interfaces.

A small part extracted from https://github.com/dmlc/xgboost/pull/6591 to ease reviewing.

Add check for consistency between 4 different DMatrix types.  They should share a number of common arguments in constructor with correct ordering.

Codecov Report

Merging #6601 (833f145) into master (03cd087) will decrease coverage by 0.36%.
The diff coverage is 47.50%.

@@            Coverage Diff             @@
##           master    #6601      +/-   ##
==========================================
- Coverage   80.44%   80.08%   -0.37%
==========================================
  Files          13       13
  Lines        3657     3675      +18
==========================================
+ Hits         2942     2943       +1
- Misses        715      732      +17

Impacted Files
Coverage Δ

python-package/xgboost/sklearn.py
89.61% <0.00%> (+0.01%)
:arrow_up:

python-package/xgboost/data.py
56.81% <33.33%> (-0.24%)
:arrow_down:

python-package/xgboost/core.py
81.06% <50.00%> (-0.39%)
:arrow_down:

python-package/xgboost/dask.py
80.64% <50.00%> (-0.89%)
:arrow_down:

python-package/xgboost/tracker.py
93.98% <0.00%> (-1.13%)
:arrow_down:

python-package/xgboost/training.py
95.37% <0.00%> (+0.04%)
:arrow_up:

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 03cd087...833f145. Read the comment docs.

Codecov Report

Merging #6601 (833f145) into master (03cd087) will decrease coverage by 0.36%.
The diff coverage is 47.50%.

@@            Coverage Diff             @@
##           master    #6601      +/-   ##
==========================================
- Coverage   80.44%   80.08%   -0.37%
==========================================
  Files          13       13
  Lines        3657     3675      +18
==========================================
+ Hits         2942     2943       +1
- Misses        715      732      +17

Impacted Files
Coverage Δ

python-package/xgboost/sklearn.py
89.61% <0.00%> (+0.01%)
:arrow_up:

python-package/xgboost/data.py
56.81% <33.33%> (-0.24%)
:arrow_down:

python-package/xgboost/core.py
81.06% <50.00%> (-0.39%)
:arrow_down:

python-package/xgboost/dask.py
80.64% <50.00%> (-0.89%)
:arrow_down:

python-package/xgboost/tracker.py
93.98% <0.00%> (-1.13%)
:arrow_down:

python-package/xgboost/training.py
95.37% <0.00%> (+0.04%)
:arrow_up:

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 03cd087...833f145. Read the comment docs.

Please address my comment at #6591 (comment). You should choose between two alternatives:

I will try to avoid breaking the compatibility.

Please address my comment at #6591 (comment). You should choose between two alternatives:

I will try to avoid breaking the compatibility.

@hcho3 I limited the breaking changes to enable_categorical and dask interface.

@hcho3 I limited the breaking changes to enable_categorical and dask interface.


Allow optional require.resolveWeak() to fail without warning

Feature request

What is the expected behavior?
It should be possible to try to require.resolveWeak() a module that doesn't exist, without emitting a warning.
E.g. I can already do this:
try {
      require.resolveWeak('not-here')
}
catch (e) {
      // no worries, it was optional anyway
}

And webpack will emit a warning:
Module not found: Can't resolve 'not-here' in '/path/to/app'

I'd like to be able to ask webpack to not show this warning in some cases.
What is motivation or use case for adding/changing the behavior?
I'm creating a library that implements a number of extra features when specific packages are available, but the features are all optional. The required packages are on the large side, so I don't want to bundle them unless the user requires them.
Currently, optional require using try/catch works, but would introduce build warnings to any application that depends on my library unless it requires all of the optional dependencies, which would be rather confusing for users.
How should this be implemented in your opinion?
Webpack already marks dependencies as optional when they're inside a try/catch block, and switches from an error to a warning in these cases:

https://github.com/webpack/webpack/blob/eb6831673afc6fdc109137bec4af47ac96247c5e/lib/dependencies/RequireResolveDependencyParserPlugin.js#L49
https://github.com/webpack/webpack/blob/6389e41e4c65bf32a916e6fffa5d65f66538d483/lib/Compilation.js#L813

One option would be to just stop warning when dependencies are optional. Another would be to create a new function, e.g. require.resolveWeakOptional(), that sets a different flag on the dependency object that indicates not to warn:
try {
  require.resolveWeakOptional('not-here')
}
catch (e) {
  // all good.
}

It may also be desirable to add support for silencing warnings on import() with a magic comment.
I'd be interested to hear any opinions on the best approach.
Are you willing to work on this yourself?
Yes, just need to know what approach would be most likely to be accepted.

I'm creating a library that implements a number of extra features when specific packages are available, but the features are all optional. The required packages are on the large side, so I don't want to bundle them unless the user requires them.

I think using require.resolveWeak('not-here') is not a good metric here. It only checks if the module is currently available. In the case when the module is code-splitted and not yet loaded, but bundled, it still throw an error at runtime. Example
try { require.resolveWeak("something") }
catch(e) { /* will be entered here */ }

import("something")

I think you need something like this:
if(typeof __webpack_is_included__ !== "function" || __webpack_is_included__("something")) {
  // load logic for "something"
  // this branch will be dropped while minimizing when "something" is not included
}

I think that would be amazing. Any idea on how to go about implementing it?

__webpack_is_included__ will be a new Dependency.
You can add to parsing logic to the APIPlugin.

Start by copying some existing Dependency, i. e. RequireIncludeDependency -> WebpackIsIncludedDependency
WebpackIsIncludedDependency.getReference() set weak to true
In WebpackIsIncludedDependency.Template check if dep.module is included in any chunk (dep.module.getNumberOfChunks() > 0). In this case generate true otherwise false.
While parsing in APIPlugin hook into

parser.hooks.expression.for("__webpack_is_included__") and add the new Dependency.
parser.hooks.evaluateTypeof.for("__webpack_is_included__") and ParserHelpers.evaluateToString("function")

add test cases i. e. to cases/parsing/typeof and some custom cases for the included logic (maybe in cases/parsing/is-included

Thanks! Might take me a little while as I'm new to the Webpack codebase, but will give it a shot.

bump

bump

Stalebot strikes again 😢

I'll take a look


Update on going-to-production

Added the following changes to the going to production route :

Anchor to self hosting instructions;
Updated dockerfile example.

Thank you but I don't think this is needed.
I'll leave a comment in the discussion you created in https://github.com/pocketbase/pocketbase/discussions/3279


Lambda

Have way to example with binaries of ghoscript and imagemagick to working with serveless (lambda aws)?

nice idea, I'll add it up if I find some time. Thanks for the thought!

Really need to set up this utility work with AWS lambda, any help appreciated! Now i get stuck with graphicsmagick set up part.

same issue

hi @winverse , can you try the solution from this page? https://stackoverflow.com/questions/44729088/how-can-i-install-graphicsmagick-or-imagemagick-on-aws-lambda

hi @winverse , can you try the solution from this page? https://stackoverflow.com/questions/44729088/how-can-i-install-graphicsmagick-or-imagemagick-on-aws-lambda

Gonna try this now. I'll let you know if it works.

Sorry, I have changed my mind about using it inside a lambda environment. Depending on the file size it could create a billing problem. I recommend AWS Batch for possibly high workload jobs instead.
For now, I'm just not converting my pdfs anymore. When I come back to this I'll let you know about the approach I choose.

Created PR #145 for this.

I don't believe  this is working as intended anymore. The layers don't seem to be functioning when added to the lambda (They appear on lambda ), my convert function throws out an error. I've tried packaging up the dependencies with my zip but that still doesn't seem to work. Not sure if GM is working as intended.
 "errorMessage": "Could not execute GraphicsMagick/ImageMagick: gm \"convert\" \"-density\" \"72x72\" \"-quality\" \"100\" \"-[0]\" \"-resize\" \"600x600!\" \"-compress\" \"jpeg\" \"./untitled.1.png\" this most likely means the gm/convert binaries can't be found",


Update mod.rs

Hello this file was changed
node/consensus/src/memory_pool/mod.rs
and i have a better solution for it can make it better x10 times
Added Module Documentation: I included a brief description of what the module does and its purpose to provide context.
Error Handling in add_transaction: I introduced error handling in the add_transaction method using the Result type. This allows the method to return a result indicating success or an error message if something goes wrong.

Consistency in Naming: Renamed the unconfirmed_transactions field to unconfirmed_transaction_pool for consistency.

Use of Mutex: Replaced the RwLock with a Mutex for unconfirmed_transaction_pool since it's being used in a read-heavy context. This can improve performance.

Added Function Comments: I included comments for the methods to describe their purpose and functionality.

Removed Copyright and Licensing Details: I kept the code focused on the implementation details and removed the copyright and licensing information for brevity. You should definitely include these details in your actual code.

Test and Additional Functionality: I left out the implementation of other methods and details that weren't provided in the initial code snippet. You can add these methods as needed for your use case.

Thank you for the PR; unfortunately it doesn't meet the project standards for unsolicited contributions, so closing it.
However, we label the open issues to which we currently welcome contributions with https://github.com/AleoHQ/snarkOS/labels/help-wanted - feel free to check them out! :slightly_smiling_face:


Floating element misaligns on scroll when using whileElementsMounted: autoUpdate

Describe the bug
A clear and concise description of what the bug is.
I am utilizing the floating-ui library in my project to manage the positioning of a floating dialog box. The initial positioning of the floating element aligns correctly with the reference element specified by aiDialogRefs.setReference. However, upon scrolling up or down the page, if whileElementsMounted: autoUpdate is set, the floating element misaligns and drifts off to the top-left corner of the screen, contrary to the expected behavior of staying aligned with the reference element.
Here's a snippet of the code I am using:
const { refs: aiDialogRefs, floatingStyles: aiDialogFloatingStyles } = useFloating({
  open: aiDialogOpenStatus,
  onOpenChange: setAiDialogOpenStatus,
  placement: "bottom-start",
  whileElementsMounted: autoUpdate,
  middleware: [
    offset({
      crossAxis: crossAxisOffset,
      mainAxis: 7,
    }),
    shift(),
  ],
});

useEffect(() => {
  try {
    if (eventAiDialogData && eventAiDialogData.dom) {
      const targetElement = document.querySelector(".empty-span-for-ai-placement");
      aiDialogRefs.setReference(targetElement);  // Setting the reference element for the floating element

      if (targetElement && eventAiDialogData?.dom)
        setCrossAxisOffset(
          eventAiDialogData?.dom.getBoundingClientRect().x -
            targetElement?.getBoundingClientRect().x
        );
    }
  } catch (error) {
    console.error("Error while setting reference:", error);
  }
}, [eventAiDialogData?.dom, openSouce]);

...

{editor && aiDialogOpenStatus && (
  <div
    ref={aiDialogRefs.setFloating}
    style={{
      ...aiDialogFloatingStyles,
      zIndex: 1,
      width: "calc(100% - 6rem)",
    }}
  >
    ...
  </div>
)}

To Reproduce

Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
I expect the floating element to maintain its position relative to the reference element while scrolling, especially when whileElementsMounted: autoUpdate is used.
Screenshots
The initial positioning of the floating element is correct and aligns well with the reference element specified by aiDialogRefs.setReference:

However, when I scroll up or down the page, the floating element does not continue to align with the reference element, instead, it drifts off to the top-left corner of the screen.:

Context:

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Additional context

In a production environment:

With whileElementsMounted: autoUpdate set, the floating element is positioned correctly on initialization but misaligns on scroll.
Without whileElementsMounted: autoUpdate set, the floating element misaligns already on initialization.

In a development environment:

With whileElementsMounted: autoUpdate set, the floating element is positioned correctly on initialization but misaligns on scroll.
Without whileElementsMounted: autoUpdate set, the floating element is positioned correctly both on initialization and on scroll.

The discrepancy in behavior between the production and development environments when whileElementsMounted: autoUpdate is not set could possibly be due to a delay in the generation of the reference element the floating element is supposed to follow in the production environment.
I would greatly appreciate any guidance on resolving this misalignment issue or insights into the possible causes of this behavior. Thank you!

Workaround and Observations:
Currently, I have adopted a workaround where I do not use whileElementsMounted: autoUpdate, and instead, I utilize requestAnimationFrame to ensure that the execution occurs after the page has fully rendered:
requestAnimationFrame(() => {
  try {
    if (eventAiDialogData && eventAiDialogData.dom) {
      const targetElement = document.querySelector(".empty-span-for-ai-placement");
      aiDialogRefs.setReference(targetElement);  // Setting the reference element for the floating element

      if (targetElement && eventAiDialogData?.dom)
        setCrossAxisOffset(
          eventAiDialogData?.dom.getBoundingClientRect().x -
            targetElement?.getBoundingClientRect().x
        );
    }
  } catch (error) {
    console.error("Error while setting reference:", error);
  }
});

With this modification, both in the development and production environments, the floating element does not drift from its position upon initialization or during scrolling. However, if whileElementsMounted: autoUpdate is set, even with the usage of requestAnimationFrame, the testing results in the development environment show that there is a certain probability of misalignment on initialization, and a 100% occurrence of misalignment during scrolling.

Thank you for the issue but it's not possible to help without a proper reproduction link. It's likely the reference becomes null when you set it or something so it doesn't anchor to anything.


Tab: TypeError: can't access property "dataset", e is undefined

Expected behavior
pharos-tabs, pharos-tab, and pharos-tab-panel initializes at the correct moment with no errors
Actual behavior
pharos-tabs is initializing and looking for a pharos-tab to select and not finding one.
even though it's there, but maybe hasn't initialized yet?
Steps to reproduce the issue

https://codesandbox.io/s/snowy-firefly-2fxpy?file=/src/components/HelloWorld.vue
see error

Screenshots or code

Pharos version
11.2.1
Your environment

OS: macOS
Browser: Firefox
Version: 97.0b9

Additional information
This could be related to the fact that we are dynamically registering these pharos components on vue component load

I suspect this is a race condition when the PharosTabs first updates, and the PharosTab items inside it haven't quite updated yet. That means they haven't added data-pharos-component="PharosTab" into the DOM—there's just the <pharos-tab> (or scoped version thereof). So not only does the selected query fail, but the tabs query fails too, ending with selectedTab being null, and _queryPanelByTab calling tab.dataset on a null tab.
Not really sure how best to fight this condition—we either need to ask PharosTabs to wait until all the PharosTab have loaded somehow, or use a different selector. The data-panel-id seems like something a PharosTab should typically have, but it isn't a required property (or a property at all, in reality).

I don't believe we have existing use cases where data-panel-id is unspecified, but I don't believe making it required would resolve this issue. You'd still have the same race condition with a different selector, I believe.

Right, I was referring strictly to the data-panel-id...data-tab-id would be a different approach.

We may be able to solve this by ensuring slotted contents have completed rendering before fulfilling updateComplete by overriding the getUpdateComplete method.

This appears to have been fixed in Pharos 12.2.3 by #330. I can't quite tell what part of those changes would have had this positive impact, but I was able to switch between different Pharos versions in the CodeSandbox link you provided and narrow it down to that version.


Favicons messageslist

Checklist

[X] I have used the search function to see if someone else has already submitted the same feature request.
[X] I will describe the problem with as much detail as possible.
[X] This issue only contains a request for one single feature, not multiple (related) features.

App version
6.507
Problem you are trying to solve
it would be nice if a favicon of the respective domain was displayed before the messages in the messageslist instead of a contact picture.
Suggested solution
add favicons
Screenshots / Drawings / Technical details

No response

I guess most people already know this, but because it was not mentioned in this issue yet: This feature is what got FairEmail kicked out of the Google Play Store last year

No, it is not the whole history, it came back to the play store, and still with the favicons.

Blue mail and emclient are another email apps with favicons available in the google play store!

Not any news about this?


dashboard logs + chunk analytics

Is your feature request related to a problem? Please describe.
Currently we have no insights into what people are doing in the dashboard. Additionally we have analytics tied to how often analytics are read, etc.
Describe the solution you'd like
Create logs for the dashboard + analytics for the chunks.
Describe alternatives you've considered
Do nothing and keep everything anonymous.

@crherman7 🤔 can we implement this optionally?
in case some users would prefer to be anonymous and not send some analytics, but as an optional extra it would be great to have that option

@crherman7 🤔 can we implement this optionally? in case some users would prefer to be anonymous and not send some analytics, but as an optional extra it would be great to have that option

yeah fair point. I guess I mean more like light-weight analytics....# of chunks, # of reads of specific chunks - lightweight logs.....nothing super specific


when will the code be released?

Apologise for late reply. Should you have any question, please feel free to file a new issue or contact me.


init_replica successful but data is not synced

Describe the bug
Tables and data are initialised successfully but data is not synced to destination database.
Source id    Source name    Type    Status    Consistent    Read lag    Last read    Replay lag    Last replay
-----------  -------------  ------  --------  ------------  ----------  -----------  ------------  -------------
         10  mysql          mysql   running   No            N/A         N/A

2020-08-31 03:19:51 read_replica DEBUG pg_lib.py (3110): Collecting schema mappings for source mysql
2020-08-31 03:19:51 read_replica INFO mysql_lib.py (1404): Read replica is paused
2020-08-31 03:19:52 MainProcess DEBUG global_lib.py (558): Replica process for source mysql is running
2020-08-31 03:19:52 MainProcess DEBUG pg_lib.py (659): Changing the autocommit flag to True
2020-08-31 03:19:52 MainProcess DEBUG pg_lib.py (1276): Cleaning replayed batches for source mysql older than 1 day
2020-08-31 03:19:53 replay_replica INFO pg_lib.py (1196): Replay replica is paused
2020-08-31 03:19:56 read_replica DEBUG pg_lib.py (679): There is already a database connection active.
2020-08-31 03:19:56 read_replica DEBUG pg_lib.py (679): There is already a database connection active.

Expected behavior
Data should be synced to the destination database (postgresql).
Environment(please complete the following information):

OS: [ubuntu 18.04 86_64]
MySQL Version[5.7.22]
PostgreSQL Version[10.9]
Python Version [3.8]
Cloud hosted database [Both MySQL and Postgresql are on AWS RDS]
pg_chameleon [2.0.14]

I re-initialized the replica and it is working now.


[new check] designer pic is png

Would raise a FAIL is the designer's avatar is other than png

do we know why PNGs are not acceptable?

Once a push to sandbox got block because an image was jpeg. I agree it not a great explanation. But request came from Rod at the time.

Dave got confirmation that jpeg were also now accepted; this PR passed to sandbox and production: https://github.com/google/fonts/pull/4557


Nonnull annotation not added for return types

Using the --swift-friendly flag,
@Nonnull
public static String bag(@Nonnull final Double tag)
is translated to

(NSString *)bagWithJavaLangDouble:(JavaLangDouble * __nonnull)tag;

whereas it should be

(NSString * __nonnull)bagWithJavaLangDouble:(JavaLangDouble * __nonnull)tag;

The fix proved to be easy, if you want to try it before it's reviewed and promoted. At TypeGenerator.java:307 add:
  if (BindingUtil.hasAnnotation(binding, Nonnull.class)) {
    return " __nonnull";
  }

Fixed in current source.


experiment detail page cosmetic changes for conditions and decision point

This PR contains UI changes for terminology changes and order of Decision Point/ Conditions in Experiment Details page as shown below:

This would be a bugfix off of 2.0.0 since it's a part of the release-1 changes, but we should be okay merging to dev at the moment.


OpenMP library doesn't build when 5.0 is available

Original author: Joseph Hutter
Original issue: https://charm.cs.illinois.edu/redmine/issues/1944

When building OpenMP libary, if you have 5.0 available, you see this during configure:
-- LIBOMP: OpenMP Version       -- 50
And then you get this later on:
CMake Error: File /projects/cwi/buildcfs/charm/verbs-linux-x86_64-omp-smp/tmp/libs/conv-libs/openmp_llvm/runtime/src/include/50/omp.h.var does not exist.
So it fails! Fortunately, it's not difficult to work around, but it would be nice if there were support in runtime/src/include for 50. Current workaround:
cd tmp/libs/conv-libs/openmp_llvm/runtime/src/include; cp -r 45 50; make -C ../../..;

Original author: Seonmyeong Bak
Original date: 2018-07-10 20:00:08

~~https://charm.cs.illinois.edu/gerrit/#/c/charm/+/4340/~~ https://github.com/UIUC-PPL/charm/commit/274f6c4686675d4ba21993c1e6e9cf6094bb4f79


Migrate OpenPolicyAgentInstallerV0 to node10

Task name: OpenPolicyAgentInstallerV0
Description: this pull request provides updating the OpenPolicyAgentInstallerV0 task and their migration to Node 10
Documentation changes required: (Y/N) N
Added unit tests: (Y/N) N
Attached related issue: (Y/N) N
Checklist:

[x] Task version was bumped - please check instruction how to do it
[x] Checked that applied changes work as expected

Changes in the PR are prepared to migrate task OpenPolicyAgentInstallerV0 to Node10.
@mmrazik, @Anumita, please review the PR.


Remove CPU limits from gardener components

How to categorize this PR?
/kind enhancement
What this PR does / why we need it:

Removes CPU limits from all components to prevent CPU throttling due to reaching limits.

The following will be added in separate PRs:

Remove or adjust memory limits from all components to match actual usage on landscapes.
Fix VerticalPodAutoscaler resources to use RequestsOnly instead of RequestsAndLimits
Fix VerticalPodAutoscaler and Hvpa resources to remove MaxAllowed where it's present

Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:
* CPU limits from all gardener components have been removed to prevent CPU throttling due to reaching limits.

/invite @danielfoehrKn @voelzmo

I consider this PR ready for review now.

/cla

/publish

Thank you @stoyanr and all for reviewing it. I also do not see any component in the list where we actually would want throttling for (I would think, this makes only sense for batch/job processing and we don't have such components much/none here if I am not mistaken).
/lgtm
A detail question though about https://github.com/gardener/gardener/blob/master/pkg/operation/botanist/kubeapiserver.go that I saw in this PR: What is it about? What is that scalingClass? Is that really dependent on the nodes/statically? I must have forgotten, but what's the background?
Follow-up question: Is that possibly, in another PR, a way to curtail non-production clusters (actually limit them/everything that is not labeled with the production purpose)?

cc @rfranzke as well...
A detail question though about https://github.com/gardener/gardener/blob/master/pkg/operation/botanist/kubeapiserver.go that I saw in this PR: What is it about? What is that scalingClass? Is that really dependent on the nodes/statically? I must have forgotten, but what's the background?
Follow-up question: Is that possibly, in another PR, a way to curtail non-production clusters (actually limit them/everything that is not labeled with the production purpose)?

LOL, thanks @rfranzke.
(1) https://github.com/gardener/gardener/issues/255 was a band-aid from mid 2018 for the poor VPA v1 for the actual ticket...
(2) https://github.com/gardener/gardener/issues/16 that we couldn't implement with the former VPA v1 and so we did this, because we hit...
(3) https://github.com/gardener/gardener/issues/79, where KCM stopped working under load
cc @voelzmo we might want to rid is of all that scalingClass workarounds now with VPA v2, if we can make it work at scale and as we want it to work.
So funny/shocking how long this topic is now with us. I see Gaurav made the changes. He was the first of the 3 colleagues on the auto-scaling topic - not in parallel - in sequence over 4 years. :-)
Time to give it a new shot and improve or ideally get rid of all these hacks.

Time to give it a new shot and improve or ideally get rid of all these hacks.

@vlerenc I will take a look, and perhaps improve it in a separate PR.

Thanks @stoyanr. I am just a bit cautious here as well.
The point is that it may still be necessary to have "decent" requests, "oriented" at the number/size of the nodes, so that we start in the correct ballpark and avoid lengthy up-scaling iterations with also the current VPAv2 (unless tuned). I say that because I remember many years before that e.g. waking up clusters (after hibernation) took very long to scale up and become operational again. Scaling classes "catapulted" control planes into the right ballpark. As long as VPAv2 is making poor recommendations or isn't bumping up faster, most of that code would probably still be required. We can only really rid ourselves of it once VPAv2 overcomes it current problems. At least, that's my gut feeling.

@vlerenc I examined the scalingClass implementation and played with it a bit, it seems that when HVPA is enabled (it's a feature gate in gardenlet that I believe is enabled by default), the scaling class is only really taken into account the first time the deployment is created. If the deployment already exists, the resources are copied from the existing deployment. This means that if the cluster started with a low initial number of nodes, increasing the number of nodes later will have no effect on the resources in the deployment. Hibernating / waking up will certainly have no effect either.
I am rather skeptical of this complex implementation (scaling class + HVPA), but as agreed, let's take another look and possibly change it once we are more confident of VPA recommendations.

@stoyanr My memory is from many years back with end users complaining in Slack and whatever you have found out today counts (not my memory), because we may have influenced the behaviour directly or indirectly many times over since 2018.

@vlerenc @rfranzke The new etcd-druid release has been merged and I removed the few TODOs in the PR (now possible with the new etcd-druid version). I consider the PR ready to be merged. Could I get an approval or do you have any other open points?

/hold cancel
@stoyanr Thanks for the PR. 👍 Let's roll it our carefully now, watching for possible issues, increased node usage, etc. please.
@rfranzke I have not hold this one (in fact, I lgtm'd it, because that's what we want - I can only check from the "intention" level and not every line anyway/am no expert), but you wanted to double-check because of Prow's automation?

I simply wasn't sure whether you are fine merging this PR now because there were quite some follow-up questions after your /lgtm and I couldn't follow everything in detail :)


[FEA] Allow } and }} to be transpiled to static strings

Is your feature request related to a problem? Please describe.
We recently had a customer that is trying to use } and }} for splitting very long strings. (See https://github.com/rapidsai/cudf/issues/14087)  Our transpiler didn't catch this as being the same as a static string because they are not escaped. But because they do not have a matching { character, they end up being treated as if they were escaped when processing the regular expression.  We should update this if possible. Might be good to look at other special characters that need to match like ] and ).

Should mention that the related customer issue manifests in a hang, so this is a little more complicated than what might be described.
We should certainly update the simple string transpilation optimization that avoids regex to handle cases like these. However, for hanging )]} in valid regexes, these might need to be escaped in the transpiler if we detect a non-matching pair, which will require detecting matching pairs in the transpiler for valid regexes.

A couple of interesting notes. On the CPU, Spark uses java.util.regex.Pattern to perform regular expression operations, so a these cases throw exceptions on the CPU:

( - "Unclosed group"
) - "Unmatched closing ')'"
{ - "Illegal repetition"
[ - "Unclosed character class"

So really need to handle ] and ]] and } and }} mostly


[Bulk] Validation: duplicate-description

Fix duplicate-description in 30 files - (VB)/(C#) only

Global effort to fix validation errors
@Rick-Anderson
The Content & Learning team is fixing build validation errors on docs.microsoft.com. This effort will eliminate potential accessibility, security, and usability issues. This PR includes only build validation fixes and does not change other content. I’d very much appreciate your timely review and feedback on these changes I’ve suggested. Thanks!


Add Podhome logo and information

I need to update the schema to have that…

Would you like me to just remove the Update Frequency?

 I can do that as well. Just committed a change to apps-schema.json


Explicitly set a static name property

Currently if you minimize your code with IncludeFragmentElement included it might end up in your bundle as a function with a single character name.
This PR makes sure that IncludeFragmentElement.name will always return include-element so that our error reporting tools will show the correct element name.

Is there any ways we can tackle this problem in one place for all of our custom elements? or do we just have to add this to all of them?

Is there any ways we can tackle this problem in one place for all of our custom elements? or do we just have to add this to all of them?

As far as I can tell we have to do this too all of them but I was thinking tooling could help us here. Maybe having a ESLint rule.

Is there any ways we can tackle this problem in one place for all of our custom elements? or do we just have to add this to all of them?

As far as I can tell we have to do this too all of them but I was thinking tooling could help us here. Maybe having a ESLint rule.

https://github.com/github/eslint-plugin-github/pull/52

🎩 🐰 ✨
The ESLint rule is now... a babel plugin!
https://github.com/github/babel-plugin-ensure-name-for-custom-elements

Closing this in favor of doing this in terser.


Vivaldi142857

Describe your change:
ПИШЕМ ЗДЕСЬ ЧТО СДЕЛАЛИ, И ВМЕСТО 'ISSUE-NUMBER' вставляем номер
Fixes #ISSUE-NUMBER

[ ] Add an algorithm?
[ ] Fix a bug or typo in an existing algorithm?
[ ] Add or change doctests? -- Note: Please avoid changing both code and tests in a single pull request.
[ ] Documentation change?

Checklist:

[ ] I have read CONTRIBUTING.md.
[ ] This pull request is all my own work -- I have not plagiarized.
[ ] I know that pull requests will not be merged if they fail the automated tests.
[ ] This PR only changes one algorithm file.  To ease review, please open separate PRs for separate algorithms.
[ ] All new Python files are placed inside an existing directory.
[ ] All filenames are in all lowercase characters with no spaces or dashes.
[ ] All functions and variable names follow Python naming conventions.
[ ] All function parameters and return values are annotated with Python type hints.
[ ] All functions have doctests that pass the automated testing.
[ ] All new algorithms include at least one URL that points to Wikipedia or another similar explanation.
[ ] If this pull request resolves one or more open issues then the description above includes the issue number(s) with a closing keyword: "Fixes #ISSUE-NUMBER".

Зачем ты в мейн пул реквест открыл?)


gRPC: Ensure start stream is the first message sent

More context in the PR comments -- https://github.com/Azure/azure-functions-dotnet-worker/pull/445. Specifically, this comment.
We want to make sure our gRPC setup doesn't send any messages until start stream is sent. We ordered the setup in a way that we send start stream first in the above PR. But Brett pointed out that we should tighten the setup to disallow sending anything else first.

After discussion, we're happy with the changes as they've gone in. Closing this as resolved.


vcpkg should set VCPKG_OSX_DEPLOYMENT_TARGET from CMAKE_OSX_DEPLOYMENT_TARGET in manifest mode

Describe the bug
On Apple OS's, If one's iOS app's deployment target is older than the deployment target of the dependencies built by vcpkg ld puts out hundreds of warnings. In manifest mode where vcpkg is automatically selecting the triplet, it selects a triplet without VCPKG_OSX_DEPLOYMENT_TARGET and builds the dependencies with the latest version of iOS as the deployment target. If your app has an earlier deployment target - not unusual - you will be subjected to the aforesaid hundreds of warnings.
Environment

OS: macOS
Compiler: Apple clang version 15.0.0 (clang-15<IP_ADDRESS>)

To Reproduce
Steps to reproduce the behavior:

Set up manifest mode to provide dependencies for your iOS app that uses CMake for building.
Set CMAKE_OSX_DEPLOYMENT_TARGET to something other than the latest version of iOS.
Configure and generate a build for Xcode or other tool that uses Apple clang.
Build your app.
Observe the warnings (332 in my case).

Expected behavior
vcpkg should use the value of CMAKE_OSX_DEPLOYMENT_TARGET when setting up the triplet for the build.
Failure logs
Here is a sample warning message:
Object file (<project dir>/build/ios/vcpkg_installed/arm64-ios/lib/libassimp.a[4](Compression.cpp.o)) was built for newer 'iOS' version (17.5) than being linked (12.0)

Additional context
I am aware thanks to https://github.com/microsoft/vcpkg/discussions/39966 that this can be worked around using VCPKG_OVERLAY_TRIPLETS. To do so, I have to copy a minimum of 4 triplet files for iOS and OSX, edit each of them to add VCPKG_OSX_DEPLOYMENT_TARGET set to a target matching the CMAKE_OSX_DEPLOYMENT_TARGET in my CMakeLists.txt. If I need to change the deployment target in future I have to change it in multiple files with the concomitant risk of error all the while potentially missing updates to the source triplet files. I find it extremely user unfriendly to have to go to such lengths to avoid a barrage of warnings.

FTR this aren't just build-time warnings. The libs from the ports are likely to make the final app fail on the desired deployment target.

What's the question?

This is waiting for somebody to implement it. There is nothing further to discuss. I don't appreciate having to keep posting comments to keep an issue alive.
I would still like to know why this has been labelled "category:question" @FrankXie05.

This is a problem. I need it fixed as well. Here's another example:
Object file (/Users/...Xbuild/vcpkg_installed/arm64-ios/debug/lib/libnlohmann_json_schema_validator.a5) was built for newer 'iOS' version (18.1) than being linked (15.6)
I have 970 such warnings. I doubt it is the reason I can't get my app to build on App Store Connect but as much effort as I have put into that it would be nice to rid this warnings to rule them out. It's messy, and the only way to not have them show up is to send a blanket "-w" to ld which supresses all warnings./


Fixes: 1155 UI of Schedule Screen improved.

Fixes #1155
Screenshots for the change:

@the-dagger Please Review! :)

It's already white! You can check in manifest. I don't know but it's
already been changed by someone.
On Feb 26, 2017 10:42 AM, "Harshit Dwivedi"<EMAIL_ADDRESS>wrote:

Can you change the background to white all across the whole app instead of
gray?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/fossasia/open-event-android/pull/1165#issuecomment-282533678,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMHNbFAydREzWa5j1322pgI3NfhFATjEks5rgQmcgaJpZM4MMDT2
.

Alright then. :+1:


Added info of CentOS-yum

This PR updates repo info about CentOS-yum, which is
quite often used if advanced virtualization modules
are hoping to be used.

Thx for contributions! XD
Added to the contributors' list.


Updated NuGet packages

What kind of change does this PR introduce?

Please check one or more that apply to this PR

[ ] Bug fix
[ ] Feature/change request
[ ] Code style update (formatting)
[x] Refactoring (no functional changes, no API changes)
[ ] Build or CI related changes
[ ] Other (include details)

What has changed?

Please describe, in detail, the changes made

Updated the NuGet packages for the project to the latest version.
Changed MvvmLightLibs out for the MADE App Component multi-target .NET Standard variant

What platforms does this change affect?

Please check one or more that apply for this PR

[x] Windows (UWP)
[x] Xamarin.Android
[x] Xamarin.iOS
[x] Xamarin.Forms
[x] .NET Standard

PR checklist

Please check if your PR fulfils the following requirements:

[ ] Tests have been added (where applicable)
[x] Code styling has been run on all new source file changes
[ ] Documentation has been added
[ ] Code sample has been added
[x] Contains NO breaking changes

If this PR contains a breaking change, please describe the impact and migration path below. Please note that breaking changes are likely to be rejected

No breaking changes made

Other information

If you have other relevant information for this PR, please feel free to add below or remove this section if not

No additional references

Thanks for taking a look at this @tom-made


Menu Dropdowns Not Functioning in Ubuntu

File, View, ect drop-down menus are not opening. I have tried reinstalling sparrow and installing older versions of the .deb file. Ubuntu is up to date and a fresh install.
I found one other issue close to mine, but not exactly the same problem. Any help would be appreciated!
https://github.com/sparrowwallet/sparrow/issues/170

What window manager are you using? Ubuntu is extremely well tested, so it sounds like something is unusual with your setup.

Probably so. I am struggling to find a solution though. Thanks for the response.
wmctrl output:
Name: GNOME Shell
Class: N/A
PID: N/A
Window manager's "showing the desktop" mode: OFF

Same output from wmctrl here. I don't have any good suggestions unfortunately - is this a stock Ubuntu install? Which version?

No problem. I appreciate the responses. Version 22.04.2 LTS. I have only installed a couple of programs so far. No major changes. I've tried uninstalling sparrow and reinstalling no luck their either.
I'll close the issue though. I'm sure it's something on my side.


sort order for unsorted collections

I'm working with graph traversal algorithms and snapshotting graph representations would make tests so much simpler.
one thing that i'm struggling with is snapshotting unsorted collections (like HashSet or HashMap)
for example if i snapshot a random HashSet, next time i run the test i'd get something like this:

the test fails, even though it's practically the same set
I'm new to rust and not sure if it's possible to overwrite serialize traits in test environment to sort them, but maybe as an alternative something can be done to a resulting json string

IIUC, could you use the preserve_order so the output is consistent? https://github.com/serde-rs/json/blob/master/Cargo.toml#L50

I'm not a huge fan of the preserve_order feature at all because that means turning it on or off will affect how stuff works. I wonder if it would be better to go via our internal Content abstraction and enforce ordering there in some situations.

Thinking about this more we could do something like this:
use insta;

let mut settings = insta::Settings::new();
settings.set_force_ordering(true);
insta::with_settings(&settings, || {
    insta::assert_ron_snapshots_matches!(...)
});

This works now on master: https://github.com/mitsuhiko/insta/blob/5d8f8c43098333645e4fe19b91872906f56461b5/tests/test_settings.rs#L4-L23


Add MKR, ZRX and change min peer count to 0

Using new contracts for Kovan based MKR and ZRX
Setting min amount of peer to be lower or equal to 0 (to show the "not connected to any peer" message)

Hmm updating the updateToken is freaking dirty. I'll close that as I don't see any easy and clean way to do so. We got to find something better than the TCR because it's not updated anymore and for testnets I believe that the address that registered a contract can easily get lost


Razor Pages do not have simple build-in ajax partial pages, it is enough to do not switch to Razor Pages

General
We use issues to drive the discussion for changes to
existing topics and the creation of new topics.
This template offers some suggestions to consider
when creating an issue.
Issues with Existing Topics
If the issue is a simple typo or similar correction, just submit a PR.
Otherwise, let us know what's wrong or what we should improve. Include a
link to the topic.
Requests for new Topics

Write a short paragraph describing what the topic will cover.

Write an outline of the proposed topic contents.

Where would you put the topic in the Table of Contents.

If you know who should review the issue, notify them by including @githubID in the issue description.

Razor Pages do not have simple build-in ajax partial pages, it is enough to do not switch to Razor Pages


Allow block multiplication of non-square blocks

Currently, BlockDiagonals keeps the block structure in matrix multiplication only if the blocks that are multiplied together have the same size. This is overly restrictive and rules out multiplication with non-square blocks.
This PR modifies the check done when multiplying BlockDiagonals to permit multiplication with non-square blocks while retaining the block diagonal structure.
In addition, the overhead of checking if blocks can be multiplied or added has been reduced.

What is the status of this PR? Can we get this done and merged?


TableToolbarSearch needs to custom the icon and placeholder

We have a requirement to change the placeholder and icon for TableToolbarSearch component.
Can we custom the icon and placeholder, thanks!

There is already a prop for placeholder text, although I'm not sure that changing the icon for search is something we would want to support.

cc @IBM/carbon-designers how do you all feel about having the icon be customizable here?

What icon would you be changing it too? And yes, customizing the placeholder text is allowed.

@aagonzales We want to change it to icon--filter specified in http://design-system.stage1.mybluemix.net/style/iconography/library

@aagonzales Hi, is it possible that we can custom the icon?

Just leaving a technical comment before this is closed - To make this happen <Search> needs change to allow custom icon. Several components have renderIcon prop to allow such kind of customization. Not sure if doing so for table tool bar makes sense, though - Requires UX feedback.


Directory overlay support

Option "-o" allows rpiboot to send files to the devices based on its USB Path.
Option "-mX" allows the delay time to be increased to reduce CPU usage "-m500" would set the default of 500 msec.
Using "... -o -d /example" preloads bootcode.bin from "/example/bootcode.bin" as normal. But when loading other files such as cmdline.txt it first checks to see if the file exists in the overlay directory so for example top left USB port on a Pi 3 has a USB path "1-1.2" so it first tries to load "/example/1-1.2/cmdline.txt" if this isn't found it will falls back to loading "/example/cmdline.txt".
I'm currently using this to boot Pi Zeros in the Cluster HAT with USB Gadget Ethernet + nfsroot where using the overlay allows using specific kernel options to set a static IP address, nfsroot paths, etc.

I've updated the code to use the more regular "-m 500" command line option format.

Have you tested this on Mac, PC and Raspberry Pi ?

I've tested on PC (Windows 7 with cygwin) and Raspberry Pi 3 (Raspbian Stretch) but I don't own any Apple products so I'm unable to test on Mac.


["Bug" Change Request]: Protonmail Changed URL

Preflight Checklist

[X] I have read the Contributing Guidelines for this project.
[X] I agree to follow the Code of Conduct that this project adheres to.
[X] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Ferdium Version
6.0.0-nightly.43
Recipe Name
Protonmail/Protoncalendar
Recipe Version
1.4.0/1.2.0
Last Known working version of the recipe
No response
Steps to reproduce
Opening Protonmail or Protoncalendar Service.
Expected Behavior
Service opens the new Domains for Mail: https://mail.proton.me/ and for Calendar: https://calendar.proton.me/
Actual Behavior
Warning message gets displayed: We´re moving! Join us at our new site: proton.me.
Screenshots

Additional Information
No response

Created a pull request for the change: https://github.com/ferdium/ferdium-recipes/pull/36


bert_mmt module in mdmmt

Signed-off-by: ChengZi<EMAIL_ADDRESS>

Codecov Report

Merging #1257 (f98b294) into main (396545f) will increase coverage by 0.25%.
The diff coverage is 88.23%.

@@            Coverage Diff             @@
##             main    #1257      +/-   ##
==========================================
+ Coverage   68.91%   69.17%   +0.25%
==========================================
  Files         259      260       +1
  Lines       12570    12757     +187
  Branches     2085     2111      +26
==========================================
+ Hits         8663     8825     +162
- Misses       3361     3374      +13
- Partials      546      558      +12

Impacted Files
Coverage Δ

towhee/models/mdmmt/bert_mmt.py
88.23% <88.23%> (ø)

towhee/dataframe/dataframe.py
73.10% <0.00%> (-1.05%)
:arrow_down:

towhee/models/layers/activations/gelu.py
66.66% <0.00%> (+8.33%)
:arrow_up:

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 396545f...f98b294. Read the comment docs.

/lgtm
/approve


Fixes rule <IP_ADDRESS> to better reflect CIS Benchmark qualifications

Changes rule <IP_ADDRESS> for squashfs to reflect that it's:

not scored (manual)
not implemented due to the fact that squashfs technically can't be disabled as it's a built-in compile time module with Ubuntu kernels
level 2

@alex-rowe I think this PR makes things a bit more sensible. I didn't understand some of the nuance in the CIS Benchmark until I started digging after your comments!
See comments on #52 for more context @florianutz

Thanks for fixing. I will doublecheck this section these days if there is something new.


reset destroy snapshot upon attempting to create a node

This is already tested Terraform has no problems with it. It will not sync
those end points to another user though, that’s an extra feature :)
On Mon, 27 Jul 2020 at 21:18 ronnno<EMAIL_ADDRESS>wrote:

@ronnno approved this pull request.
Nice,
How will this work with the "backend" feature? will the lock be copied?
will workspacePath be in the same directory? will terraform not be angry
that we are using it's folder for our own files?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/orbs-network/polygon/pull/9#pullrequestreview-456024442,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABNPQO5DNUS6YJ3OTK5LRK3R5XAG3ANCNFSM4PI4DSKQ
.

--
Itamar Arjuan,
Web Developer
http://www.itamararjuan.co.il

@itamararjuan can you please explain how does this feature work?


It appears to be inch only picks up files in the root dir

Re: Gum-Joe/Web-OS
In my node-js project,inch failed to evaluate files out the root dir (e.g: it ignored files in my ./libs folder)

You can configure which files inch should parse: https://inch-ci.org/help/config_file_json
{
  "files": {
    "included": ["libs/"]
  }
}

Please  note that an alternative to putting the config in an inch.json is to put the same hash in your package.json under an inch key:
"inch": {
  "files": {
    "included": ["libs/"]
  }
}

Hope this helps!

Thanks,  thought it would automatically parse all the documents
By the way, does it ignore functions in the form of exports.NameOfFunction()?

Effectively, yes, but not intentional.
I'm using JSDoc for parsing JavaScript and if JSDoc does not pick it up, then we're out of luck. So while I would very much like to support more common idioms from the JavaScript and especially Node community, I have to "work with what I got" so to speak :wink:

Thankyou, will now close issue


Add a blog post on detecting and dealing with outliers.

@juliusv

@brian-brazil I added some rewordings/additions/deletions as a separate commit: https://github.com/prometheus/docs/pull/121/commits

Looks fine thanks, squashed. Shall we use 2 spaces or 4?

@brian-brazil: @fabxc finds the 4-space indentation worse than the 2-space one, but prefers the indentation where everything is inverse (operators indented deeper than their operands). However, I think you very intentionally inverted that, right? :)

I always found the inverted a bit hard to read, which is why I'm proposing this. I just need to undo 8 years of muscle memory now :)

Ok, let's go with this as it is for now... I just don't feel too strongly in either direction :+1:


Pending authorizations

I have send request for few thouthand domains. After 2000 certificates i start received response
Error 429 - urn:acme:error:rateLimited - Error creating new authz :: Too many currently pending authorizations.
I use acme.HTTP01 authentication for domains with webroot provider.
Before each request of certificate I check webroot someself:
generate random token and key, put it to webfolder by the provider and try to get it by general http.Get request, then remove my token.
I send certificate request only if my request is ok.
before 429 I have many errors:
Error 400 - urn:acme:error:badNonce - JWS has invalid anti-replay nonce pm_ZKGxzW...
I used patched jws.go - from pr #185
I think it may be depends from jws nonce bug.

I'm not sure I'm understanding what you are trying to say :worried:

Sometime authorization failed for good domains. It is problem for large sets of domains: I reached limit

Pending Authorizations/Account limits how many times an ACME client can request a domain name be authorized without actually fulfilling on the request itself. This is most commonly encountered when developing ACME clients, and this limit is set to 300 per account per week.

https://community.letsencrypt.org/t/rate-limits-for-lets-encrypt/6769

Is this still happening after #185 was merged?

Yes.

I think the problem can be in time of remove authorized token (call webroot provider CleanUp).
I try to switch to other libraries and get same problem until wait for remove token after receive certificate (not right after authorization).
In log I saw about letsencrypt can check token few times.
In your library token removed right after succefully authorization:
http_challenge.go, func (s *httpChallenge) Solve(chlng challenge, domain string).
For example logs from one of website:
<IP_ADDRESS> - - [14/Apr/2016:09:09:17 +0300] "GET /.well-known/acme-challenge/l5qidJBvNNW10BvHbJfM9aFYknzTHWf3sw1pACMuRSU HTTP/1.1" 301 - "-" "Mozilla/5.0 (compatible; Let's Encrypt validation server; +https://www.letsencrypt.org)"
<IP_ADDRESS> - - [14/Apr/2016:09:09:19 +0300] "GET /.well-known/acme-challenge/l5qidJBvNNW10BvHbJfM9aFYknzTHWf3sw1pACMuRSU HTTP/1.1" 404 7607 "http://.../.well-known/acme-challenge/l5qidJBvNNW10BvHbJfM9aFYknzTHWf3sw1pACMuRSU" "Mozilla/5.0 (compatible; Let's Encrypt validation server; +https://www.letsencrypt.org)"

The first time token was present and authorize was succesfully.  Second time token was removed (may be when request certificate) and letsencrypt receive 404 error.
I think it happen for domains with redirection from www.domain.com -> domain.com or vise versa:
first domain.com checked ok,  but when try to check second variant - www.domain.com request redirected to domain domain.com and letsencrypt check old token - for domain.com, which already removed.

I think it happen for domains with redirection from www.domain.com -> domain.com or vise versa:
first domain.com checked ok, but when try to check second variant - www.domain.com request redirected to domain domain.com and letsencrypt check old token - for domain.com, which already removed.

This case is handled by two different authorizations. You get one authorization + token for domain.com and another one for www.domain.com as both are different entities. Per the ACME spec the token should be removed after successful authorization as any other domain will have their own auth token.
In your example logs, why is the let's encrypt server receiving a 301 on the first request - it should receive a 200 OK.

Yes, I understand about two different tokens for different domains.
Server get 301 becouse customers website have redirect from unconditional www.domain.com to domain.com in his .htaccess. I can't change customers website or rewrite rules.
I solve problem for me by switch to other library, which give more control (but it's more difficult in use) and change live of tokens for all certificate request process.
In your library you can create tokens for all domains, then obtain certificate then remove all tokens for all domains. It will handle the case.
Your library is perfect and simple for usage. Thanks for work :)

Thanks :) but I'm still not sure about the issue at hand.
Given we want to validate the domains www.example.com and example.com where the www subdomain redirects to the main domain.
Boulder will try to access http://www.example.com/.well-known/acme-challenge/123456789www and is greeted with a 301 redirect to http://example.com. Now boulder follows this redirect and tries to access the SAME token again at http://example.com/.well-known/acme-challenge/123456789www. At this point we have not removed the token as the challenge did not complete yet. After boulder got the token and validated it, we get a new status from boulder that the authorization is complete and we remove the token.
Next boulder tries to access http://example.com/.well-known/acme-challenge/123456789example - notice the different token. Here boulder gets a 200 response again and we delete the token as the challenge is complete.
This is how it's supposed to work. Could you provide the steps which differ from the above?

Hmm... yes, you are right...
Ok, I have no any more idea.
May you remove token before letsencrypt check it - similar to race condition? For example:
lego put token
lego request letsencrypt
lego remove token
lensencrypt request token
?
If not - i have no any more ideas about 404 errors of token (while self check work fine).
About subject problem: it can be from 400 bad requests errors too. After 2-3 hours continuesly cert requests server response 400 bad request for every second request (about 50%). Without retry (if skip bad domain) it fast reach pending authorization limit.

May you remove token before letsencrypt check it

We specifically check with the ACME server through polling to make sure the challenge status is no longer "pending" (executing) but either "invalid" or "valid". Only when the status is one of these two we remove the token.

After 2-3 hours continuesly cert requests server response 400 bad request for every second request

It is taking you 2-3 hours to request certificates for all domains?

I will close this for now.
Feel free to re-open the issue if you feel like there is anything else to add.


schduled incident cant parse year?!

[x] I confirm that this is a issue rather than a question.

Bug report
creating a scheduled incident results in
ℹ Generating pages                                                                                13:51:37
✔ Generated /incidents                                                                                13:51:38
✔ Generated /offline                                                                                13:51:38
✔ Generated /404                                                                                13:51:38
✔ Generated /history                                                                                13:51:38
✔ Generated /incidents/54809b9c                                                                                13:51:38
✔ Generated /incidents/7a393ef6                                                                                13:51:38
✔ Generated /incidents/4299d2d0                                                                                13:51:38
✔ Generated /incidents/2d3c1160                                                                                13:51:38
✔ Generated /                                                                                13:51:38
✔ Generated /sitemap.xml                                                                                13:51:38
✔ Generated /feeds/incidents.en.xml                                                                                13:51:38
✔ Generated /feeds/incidents.en.atom                                                                                13:51:38

 FATAL  value.getFullYear is not a function                                                                                13:51:38

  at extractDate (node_modules/@statusfy/core/lib/content/calendar.js:9:11)
  at incidents.forEach.i (node_modules/@statusfy/core/lib/content/calendar.js:43:14)
  at Array.forEach (<anonymous>)
  at calendar (node_modules/@statusfy/core/lib/content/calendar.js:37:13)

Version
0.3.0-alpha.4
Steps to reproduce
npm run new-incident
...
npm run generate
What is expected?
a new scheduled incident
What is actually happening?
a parse error
Other relevant information

Your OS: Fedora 29 (Container Image)
Node.js version: 10.15.0
Browser version:
Is this a global or local install? local
Which package manager did you use for the install? npm

here is the incident file:
---json
{
  "title": "Deploy of ...",
  "date": "2019-03-11T13:06:03.407Z",
  "severity": "under-maintenance",
  "affectedsystems": [
    "api"
  ],
  "scheduled": "2019-03-11T13:06:03.407Z",
  "duration": 10080,
  "resolved": false
}
---
We are going to ...

<!--- language code: en -->

Hi, @goern. We are currently replacing the library we use for date parsing, in order to fix this https://github.com/bazzite/statusfy/issues/149 and improve the date management.

Hello, @goern. A fix for this bug will be available in the next stable version. In the meantime, you can test it using the version 0.3.0-alpha.7.
Let me know is this works for you, thanks!

seems to work, but I don't see the body text... "We are going to ..." from the example above
/reopen

Hi, @goern. This is now fixed in the version 0.3.0-beta.1. Let me know if you have any question or suggestion.
Besides, I invite you to leave your comment in the Statusfy Users issue when you start using Statusfy in your company or organization 🤓. Thanks!

@juliomrqz works! thanks for the support!
/close


0% trained data trained

I keep following the steps in the ReadMe but whatever I do after the training phase it always shows TRAIN: 0% |.

Hi, perhaps this is a GPU issue - could you double check if your GPU is properly configured and recognized by pytorch?

I do not have CUDA on my system. I tried removing the cuda references in the codebase. However, it still doesn't work.

A small update. If I go ahead and remove the references to CUDA in the err_localize_edit.py and then run the code, I get the list index out of range error. Not sure how these are related.

You can reduce the no. of steps in DrRepair/model/configs/data-spoc/err-data-orig.yml
timing:
  max_steps: 150000


Windows Commands/call: remove unintended backslash

Description:
As pointed out in issue ticket #3685 (To(o) many backslashes), there are two call command usage examples in this page containing incorrect backslashes where no backslash is intended or used by the command.
Thanks to Thomas Stein (@tom1stein) for pointing out these typos.
Changes proposed:

Remove surplus backslashes in the sentence "The % modifiers cannot be used with %\*."
Remove the backslash in call :\<Label> <Arguments>

Ticket closure or reference:
Closes #3685
Ref. #3687 (the old PR I mangled by incorrect rebase or merge)

Rich diff view, for easier overview of the effective changes (2):

https://github.com/MicrosoftDocs/windowsserverdocs/pull/3944/files?short_path=c8d2574#diff-c8d25741d8e478768c325fba85847604

Normal text view, to see the textual changes:

https://github.com/MicrosoftDocs/windowsserverdocs/pull/3944/files

I am uncertain of who is the current owner ("author") or manager of this document. Previously, it used to be @coreyp-at-msft and @DonGill respectively, but I presume it is best to ask @eross-msft for advice.

#sign-off

@illfated  This article has also become self-service through the product group, so there is no actual author at this time. I defer to @v-maudel to review this article since they have done this work previously. In many cases, the backslash is/was used as an escape character, but my understanding is that some of those instances have broken, leaving the extra text showing.

@v-maudel  Please review this change since I know you've made similar changes for localization purposes. After you confirm this is accurate and working, I'll go ahead and accept the changes.

Very good, that is the correct interpretation (and the same logic as I used as a base for this PR).


MD-2882: Modified hive change metadata scripts (#330)

@kfaraaz , can you please take a look? I'm cherry-picking a change from a different branch to master. This is because of Hive 2.1 behavior changes (metadata changes supported are more restrictive).

@Agirish Changes look good. LGTM.

Thanks Khurram. Test run is clean. Merging.


Can't create some of the metrics Alerts at Azure

There are several metrics that verify the health state of the deployed nodes. According to the issue https://github.com/terraform-providers/terraform-provider-azurerm/issues/3719 there is no option to define multiple scopes for single alert. We should track this issue to add essentials alerts such as validator count to Azure Monitor.
As for now we should leave the note at Azure README that there is no option to create this alert automatically and propose the end user to create it manually.

https://github.com/terraform-providers/terraform-provider-azurerm/pull/7159 fixes the issue. However, there are adjustment needed on our side to fix this issue. Will be fixed later.


🚧 Accessibility improvement

User story: AB#485888

Changed focus style on keyboard navigation
Make focus go from last item in the list to the first when using arrow down (same behavior as EDS AutoComplete)
Ability to keyboard navigate up and down between children and parents
Change color of icon for open-button to primary resting
Use the right clear-icon
Option to display as text instead of chips (I think Per Christian changed his mind on this one, so this code could be removed)

Unsolved issue: When user tab-navigates through the menu items so that one item has focus, and also mouse hover over a different item, two items have focus style on the same time. There should only be one focused element at any time. When mouse hover a menu item, it should set element.focus().

Aslak, take the wheel.

Closing PR for now, but linked to it in US for implementation of some of these changes.
https://dev.azure.com/Equinor/DPP Upscaling of digital solutions/_backlogs/backlog/Amplify Component Library/Epics?workitem=522861