Export to PDF Error - Truncates Diagram

When I export to PDF my diagram is cut off and partially moved to a new page making it hard to use.

Hi,
Which version are you using? Can be seen under Help in the editor.
Do you see the same behaviour in other browsers and Incognito mode?
Does File > Print > Print to PDF/Save as PDF help? or File > Export as > Advanced > PDF?
If not, attach the file if possible so we could test it.
Regards,

Version: diagrams.net 15.1.3
Here is the example draw.io file and pdf.
As a side note when exporting multiple pages it would be nice if it would include the page reference on the export.
Thanks,
John
From: mararad @.***
Sent: Monday, December 13, 2021 6:20 AM
To: jgraph/drawio @.>
Cc: John Haushalter @.>; Author @.***>
Subject: [EXTERNAL] Re: [jgraph/drawio] Export to PDF Error - Truncates Diagram (Issue #2485)
Hi,
Which version are you using? Can be seen under Help in the editor.
Do you see the same behaviour in other browsers and Incognito mode?
Does File > Print > Print to PDF/Save as PDF help? or File > Export as > Advanced > PDF?
If not, attach the file if possible so we could test it.
Regards,
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/jgraph/drawio/issues/2485#issuecomment-992423715, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AW34BCREBXZBKEBVJPFN5SDUQXQHNANCNFSM5JZOOJOA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Hi John,
My suggestion would be to update diagrams.net version to the latest first (just do hard reload) and export again.
If export fails, try the following workarounds:
File > Print > Print to PDF/Save as PDF and File > Export as > Advanced > PDF
If this fails, please include an examples (it seems they are not attached to this issue)
"As a side note when exporting multiple pages it would be nice if it would include the page reference on the export." - placeholders might be a good solution in this case: https://www.diagrams.net/doc/faq/predefined-placeholders
Also check https://github.com/jgraph/drawio/issues/2134

Truncate Example 22.log
Truncate Example 22.drawio.pdf
I've uploaded the .drawio file (had to change the extension to .log to allow me to upload and the resulting pdf export file.  I did this with the most recent version 15.8.7

So, you're using the desktop version? Any other details we might need?

Yes using the desktop version.  I can’t think of any other details.  I am using the File->Export->PDF option.  It just seems when the diagram goes further to the right than normal portrait the export does not adjust as necessary to accommodate the complete diagram and bumps to the next page.
From: David Benson @.>
Sent: Wednesday, December 15, 2021 10:25 AM
To: jgraph/drawio @.>
Cc: John Haushalter @.>; Author @.>
Subject: [EXTERNAL] Re: [jgraph/drawio] Export to PDF Error - Truncates Diagram (Issue #2485)
So, you're using the desktop version? Any other details we might need?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/jgraph/drawio/issues/2485#issuecomment-994951641, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AW34BCWLDEUCS6G5XOW26LDURC6MZANCNFSM5JZOOJOA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


⚠ Turnos 24x7 has degraded performance

In 77bc3cf, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 1409 ms

Resolved: Turnos 24x7 performance has improved in dfd3534.


export 'default' (reexported as 'default') was not found in 'ember-tracked-local-storage/utils/tracked-in-local-storage'

Build with pnpm v9 and see the following error:
WARNING in ../rewritten-packages/ember-tracked-local-storage.a84501d5/node_modules/ember-tracked-local-storage/_app_/utils/tracked-in-local-storage.js 1:0-142
export 'default' (reexported as 'default') was not found in 'ember-tracked-local-storage/utils/tracked-in-local-storage' (possible exports: localStorageGet, localStorageSet, trackedInLocalStorage)

Not sure if the default really needs to be reexported here, but the most conservative is to add a default export to addon/utils/tracked-in-local-storage.js, I guess.
See https://github.com/Leadfeeder/ember-tracked-local-storage/pull/38 and https://github.com/Leadfeeder/ember-tracked-local-storage/pull/36, and pick one

@leadfeeder / @willjthorp this simple missing export gives a warning in webpack. Worse, it breaks the build completely with vite:
No matching export in "node_modules/.embroider/rewritten-packages/ember-tracked-local-storage.91fa62fd/node_modules/ember-tracked-local-storage/utils/tracked-in-local-storage.js" for import "default"

    node_modules/.embroider/rewritten-packages/ember-tracked-local-storage.91fa62fd/node_modules/ember-tracked-local-storage/_app_/utils/tracked-in-local-storage.js:1:9:
      1 │ export { default, localStorageGet, localStorageSet, trackedInLocalStorage } from 'ember-tracked-local-storage/utils/tracked-in-local-storage';
        ╵          ~~~~~~~
 ^

Error: Build failed with 1 error:
node_modules/.embroider/rewritten-packages/ember-tracked-local-storage.91fa62fd/node_modules/ember-tracked-local-storage/_app_/utils/tracked-in-local-storage.js:1:9: ERROR: No matching export in "node_modules/.embroider/rewritten-packages/ember-tracked-local-storage.91fa62fd/node_modules/ember-tracked-local-storage/utils/tracked-in-local-storage.js" for import "default"
}

See https://github.com/Leadfeeder/ember-tracked-local-storage/pull/38 and https://github.com/Leadfeeder/ember-tracked-local-storage/pull/36, for proposed fixes.
An alternative could be to hand the repo over to https://github.com/adopted-ember-addons for adoption? that would also help with the proposition over here: https://github.com/Leadfeeder/ember-tracked-local-storage/issues/27
Thanks,

@Darren-LF or @jbandura perhaps? thanks,

@johanrd Sorry for the delay getting back to you on this! I've merged the PR with the fix so hopefully that solves your issue for now. I'll discuss with the rest of my team about the idea to move to adopted ember addons + merging with the ember-local-storage-decorator package.


Error when trying to reassign a bud

Description:
Error when trying to reassign a bud
Way to reproduce:
No response
(Optional) Link to data to reproduce error:
No response
Terminal output:
Traceback (most recent call last):
  File "C:\Users\SchmollerLab\.virtualenvs\acdc\lib\site-packages\pandas\core\indexes\base.py", line 3652, in get_loc
    return self._engine.get_loc(casted_key)
  File "pandas\_libs\index.pyx", line 147, in pandas._libs.index.IndexEngine.get_loc
  File "pandas\_libs\index.pyx", line 176, in pandas._libs.index.IndexEngine.get_loc
  File "pandas\_libs\hashtable_class_helper.pxi", line 2606, in pandas._libs.hashtable.Int64HashTable.get_item
  File "pandas\_libs\hashtable_class_helper.pxi", line 2630, in pandas._libs.hashtable.Int64HashTable.get_item
KeyError: 57

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "C:\Users\SchmollerLab\Cell_ACDC\cellacdc\__init__.py", line 243, in inner_function
    result = func(self, *args)
  File "C:\Users\SchmollerLab\Cell_ACDC\cellacdc\gui.py", line 6457, in gui_mouseReleaseEventImg1
    self.assignBudMoth()
  File "C:\Users\SchmollerLab\Cell_ACDC\cellacdc\gui.py", line 8777, in assignBudMoth
    eligible = self.checkMothEligibility(budID, new_mothID)
  File "C:\Users\SchmollerLab\Cell_ACDC\cellacdc\gui.py", line 8627, in checkMothEligibility
    is_still_bud = cca_df_i.at[budID, 'relationship'] == 'bud'
  File "C:\Users\SchmollerLab\.virtualenvs\acdc\lib\site-packages\pandas\core\indexing.py", line 2412, in __getitem__
    return super().__getitem__(key)
  File "C:\Users\SchmollerLab\.virtualenvs\acdc\lib\site-packages\pandas\core\indexing.py", line 2364, in __getitem__
    return self.obj._get_value(*key, takeable=self._takeable)
  File "C:\Users\SchmollerLab\.virtualenvs\acdc\lib\site-packages\pandas\core\frame.py", line 3886, in _get_value
    row = self.index.get_loc(index)
  File "C:\Users\SchmollerLab\.virtualenvs\acdc\lib\site-packages\pandas\core\indexes\base.py", line 3654, in get_loc
    raise KeyError(key) from err
KeyError: 57

Version information:
No response

Error happens when trying to re-assign a bud that will disappear in of the future already visited frames.
The function gui.checkMothEligibility should break the for loop checking future frames if the bud does not exist anymore.
Proposed fix:
    def checkMothEligibility(self, budID, new_mothID):
        """
        Check that the new mother is in G1 for the entire life of the bud
        and that the G1 duration is > than 1 frame
        """
        posData = self.data[self.pos_i]
        eligible = True

        G1_duration = 0
        # Check future frames
        for i in range(posData.frame_i, posData.SizeT):
            cca_df_i = self.get_cca_df(frame_i=i, return_df=True)

            if cca_df_i is None:
                # ith frame was not visited yet
                break

            if budID not in cca_df_i.index:
                # Bud disappeared
                break


Drupal's Aegir Project

Not to be confused with https://www.aegirproject.org/

Thanks for flagging that! Hopefully people won't be too confused as that aegir seems to be a hosting platform while this aegir is a build tool :)


https://groups.google.com/forum/#!topic/adt-dev/kdP2iNgcQFM

https://groups.google.com/forum/#!topic/adt-dev/kdP2iNgcQFM

Expected Behavior

Current Behavior

Context

Steps to Reproduce (for bugs)

Your Environment

Build scan URL:

Please follow our issue template.


Add go tests for ECA

Description
Added go test for eca (plus a little related cleanup)
Motivation and Context
There was very limited test coverage within the membersvcs package for the eca component
Fixes #1590
How Has This Been Tested?
This is actually adding test cases.  Current tests in the package still work as well
Checklist:

[X] I have added a Signed-off-by
[X] Either no new documentation is required by this change, OR I added new documentation
[X] Either no new tests are required by this change, OR I added new tests
[X] I have run goimports, go vet, and golint. I have cleaned up all valid errors and warnings in code I have added or modified. These tools may generate false positives.  The goal is clean, consistent, and readable code.

Signed-off-by: Gari Singh<EMAIL_ADDRESS>

@JonathanLevi  - can you take a look when you get a chance?

@mastersingh24 why is coverage.out in? Is it a file that should be in for all packages?

@adecaro: Good point. FYI, this file will be removed auto-magically once the (approved) 1761 PR is merged.
@mastersingh24: Many, many thanks for this. Will do.

Ah right, thanks @JonathanLevi :)

@adecaro - I left it in since it was already there
@JonathanLevi  - thanks for checking it out.  still some things which could be better but wanted to get this in since it has about 80% coverage now

reverify jenkins

Ignoring Jenkins' BDD/Behave tests failures for now, as suggested.
LGTM. Thanks again, J

@mastersingh24 looks like you need to rebase before this can be merged. I agree with @JonathanLevi that there is room for improvement (e.g. validating that what was expected to happen, actually happened, as opposed to just checking for an unexpected error).

damn - something went awry.  trying to do too many things at the same time

reverify jenkins

hopefully this passes and the PR is accepted soon.  there is a lot of activity in membersvcs and would like to get these tests checked in so we have a baseline for further changes

I don't think this PR broke the build but let me know

restarted Travis build...

(Good morning) @mastersingh24 Can you please merge down from master?

@JonathanLevi  - okay.  will do.  I certainly hope nothing breaks

looks like we got past make unit-test  woo hoo

@ghaskins @binhn  - could one of you quickly look at this to get it merged?  there are a bunch of things which will come in under membersvcs so would be good to get some base tests checked in

LGTM. Thanks!


Allow rdf parser to be configurable

Fixes #2760

The rdf parser for validating data can be configured to be more or less strict
The rdf parser for fetching/indexing remains lenient to be able to return a resource no matter the strictness defined for validation (and also because there is no need to validate them again)

Yep, checking and strict were the solution.
However your test only handles the case expanded -> Graph, as far as I can see.
Does this also throw when converting compacted -> expanded ?

Nope, nothing is thrown at the expanded/compacted level

Ok. So then as far as I can tell this is not going to solve the root problem, which is: people will still be able to create resources with those "wrong" IRIs (for example iris with a space at the beginning).
We are just going to get another kind of warning/error.

The problem is solved in my PR, we transform the expanded into a graph in the resources state machine (I only had to add this for resources not validated by a schema, we already did it otherwise)
We get an InvalidJsonLdFormat rejection if the iri is wrong


struct array initialization fails

currently transpiles to: var a []interface {} = []interface{}{nil, nil}
void test_structarr()
{
    struct s {
        int i;
        char c;
    };

    struct s a[] = {{1, 'a'}, {2, 'b'}};
    is_eq(a[0].i, 1);
    is_eq(a[0].c, 'a');
    is_eq(a[1].i, 2);
    is_eq(a[1].c, 'b');
}

I am working on this one and will soon have a fix.

@elliotchance The code is merged into master. Closing.


added estacio univesity domain email for students

I added this domain 'alunos.estacio.br' to students emails

@Jodaciel Pull request merged. Thank you!


fix badges card beyond height limit

Pull Request Template
badges card beyond height limit
Briefly describe the changes and the related issue.
Related Issue
Link to the issue here: #(issue)
Type of Change

[V] Bug fix (fix)
[ ] New feature (feat)

Checklist

[V] Code follows project guidelines
[ ] Self-reviewed code

Additional Notes
Any additional information.
Reviewers
xue,d5

Previous view


Using CUDA version 10.1

I'm having some compatibility issues when using pytorch-1.1.0 and cuda 9. What should I include in the Dockerfile in order to use 10.1?
Thanks.

Hi @ksanjeevan, thank you for using SageMaker! Our GPU image uses pytorch 1.1 with cuda 10.1 runtime, as shown here: https://github.com/aws/sagemaker-pytorch-container/blob/master/docker/1.1.0/base/Dockerfile.gpu#L1

Yeah I found that the other day! Yet when I print torch.version.cuda on my entry_point.py file I get:
algo-1-0lcg9_1  | /usr/bin/python -m entry_point --gpu 1
algo-1-0lcg9_1  |
algo-1-0lcg9_1  |
algo-1-0lcg9_1  | CUDA version: 9.0.176
algo-1-0lcg9_1  | Epoch: 0
algo-1-0lcg9_1  | 2019-07-02 20:47:56,107 sagemaker-containers ERROR    ExecuteUserScriptError:
algo-1-0lcg9_1  | Command "/usr/bin/python -m entry_point --gpu 1"
algo-1-0lcg9_1  | THCudaCheck FAIL file=/pytorch/aten/src/THC/THCGeneral.cpp line=383 error=11 : invalid argument

Any thoughts on what could be causing this? Perhaps I have some package not up to date? This is causing an error like the one displayed above when using new torchvision models.

Hi @ksanjeevan, it seems like there's no support for CUDA 10.1 from PyTorch, and to avoid breaking changes, our image does pip install torch==1.1.0 instead of building from source.
I found an article that might be useful for you to install CUDA 10 for PyTorch: https://medium.com/@zhanwenchen/install-cuda-10-1-and-cudnn-7-5-0-for-pytorch-on-ubuntu-18-04-lts-9b6124c44cc

Hmmm ok, just to confirm @ChuyangDeng: despite https://github.com/aws/sagemaker-pytorch-container/blob/master/docker/1.1.0/base/Dockerfile.gpu#L1 the Pytorch 1.1.0 image will install cuda 9 right? And for now cuda 10 won't be an option so my best bet is to create a custom image that installs cuda 10.1?
Thanks!

Ok so I figured out that the pytorch 1.1.0 docker image does in fact use CUDA 10.1, but the pytorch install is CUDA 9. By simply including a pip install of cu100/torch in the Dockerfile all works as expected.


No torrents found - qBittorrent

Environment
Docker
Version
v0.10.0
Describe the problem
I can't seem to find the logs to know why it's not working.
I checked the documentation if there are configurations for the Torrent Module but I also can't find it.
I just get the following status in the module box:
No torrents found
Additional info
No response
Please tick the boxes

[X] You've read the docs
[X] You've checked for duplicate issues
[X] You've tried to debug yourself

Did you enable "Hide completed torrents " by any chance ?

It is disabled. I also have an ongoing download :(

It is disabled. I also have an ongoing download :(
The qbittorrent service shows green though:

Weird. Can you show me the logs of the network request for the download module (in developer console)
should happen every second

Same issue here.

Request URL: http://<IP_ADDRESS>:7575/api/modules/torrents
Request Method: POST
Status Code: 401 Unauthorized
Remote Address: <IP_ADDRESS>:7575
Referrer Policy: strict-origin-when-cross-origin

I also don't see torrents. I add 3 modules: Deluge, Deluge-VPN and Qbittorrent-VPN and have no torrent found. I don't have any option to disable downloaded torrents.

Regards.

Sorry, I still couldn't replicate your issue.
Could you contract me on discord and give me some kind of access to your homarr instance so that I can debug it ?

I have same issue.
How can i help to solve?

Same issue here as well.

Can you try if this issue also occurs on the dev image?
WARNING: dev is an experimental branch and a backup is highly recommended. We do not recommend it for daily use, as it might be unstable

@ajnart I think I found the general issue, and it's the same in QBT, Sonarr or Radarr, and I will make myself available for you to test this. Essentially: DNS name translation. If I have a service configured with an IP address, everything will work fine. As soon as I switch to name translation, it breaks down.
I have the same behavior as in https://github.com/ajnart/homarr/issues/538

Thanks for pining me! Ajnart is currently quite busy and I will respond instead for now.
It would be awesome if all people in this thread and also #538 could affirm this.
How do I test if I am being affected of this issue?

Copy the hostname from your service URL
Paste it to "On click URL"
Replace your service URL with the internal IPv4 URL

An example:

I honestly don't know any reason, why this might be the case though.
Under the hood, we use the API wrappers of https://github.com/scttcper.
These use https://github.com/sindresorhus/got for sending requests to the clients.
We also use https://github.com/axios/axios for some other modules.
It would be really interesting to see, if there is anything going wrong with name resolution.
If a timeout happens, this could also explain #515

Out of nowhere mine just started populating without me changing anything. I browsed to Homarr on my mobile device and noticed it was working, I then came back to my computer (using Edge browser) and the graph refreshed and is now showing my downloads. Very strange...

It would be really interesting to see, if there is anything going wrong with name resolution. If a timeout happens, this could also explain #515
EDIT: I just remembered, that #551 was a thing. Perhaps we're causing a massive overflow of DNS requests, until a firewall, router or security device blocks the requests? I sadly don't have the time for this, but perhaps somebody could sniff around with Wireshark and check if this is the case? Currently very busy for 0.11 - sorry

Not sure if it's a timeout thing, TBH. I can say that looking at the preview message that comes back when using name resolution, it doesn't strike me as a timeout thing:
{message: "getaddrinfo ENOTFOUND sonarr.rabbit", name: "Error",…}
But like you, I don't know how those wrappers work behind the scenes. I do have a Pi Hole serving my network, so at least I wouldn't discard it. There is another point though. If the system keeps on requesting name translation, without a caching mechanism... It's a baaaad idea. That much is clear to me. Whether those wrappers have something like that or not... No clue.
 So one thing is for sure: name resolution is the issue. I know that it's not a critical thing, but name resolution is a big boon for networks. In my case, I'm using specific IP addresses for certain services, so having name resolution is a life saver for me. Of course, using the IP is a workaround I can deal with, but if at some point I decide to change the IPs behind those names (it has happened before), I'll have to reconfigure Homarr. Not breaking, but annoying. Still, on my side, I can confirm that this works for me.

@manuel-rw just to maybe get some more proof of the name resolution as part of several issues (whether it's flooding, network blocking, or even something not being properly translated), let me point out more issues that have the same base. Do note, I'm not pressuring or anything, just want to centralize some information for later reference:
https://github.com/ajnart/homarr/issues/496
In https://github.com/ajnart/homarr/issues/496#issuecomment-1354707159 the user mentions that as soon as it has name resolution with Nginx, it starts failing.
https://github.com/ajnart/homarr/issues/480
Already a duplicate of this one
https://github.com/ajnart/homarr/issues/538
As I mentioned there, I've had the same issue as seen here.
And then, the two potential issues you mentioned. Not sure I have the time to go sniff the network, but if I have that time, I'll post something here.

@manuel-rw just to maybe get some more proof of the name resolution as part of several issues (whether it's flooding, network blocking, or even something not being properly translated), let me point out more issues that have the same base. Do note, I'm not pressuring or anything, just want to centralize some information for later reference:
#496 In #496 (comment) the user mentions that as soon as it has name resolution with Nginx, it starts failing.
#480 Already a duplicate of this one
#538 As I mentioned there, I've had the same issue as seen here.
And then, the two potential issues you mentioned. Not sure I have the time to go sniff the network, but if I have that time, I'll post something here.

Thank you very much for your investigation

Not sure if it's a timeout thing, TBH.

You wouldn't see in the browser if your DNS resolution would fail.

{message: "getaddrinfo ENOTFOUND sonarr.rabbit", name: "Error",…}

This just indicates that the host was not found or connection was rejected. I really suspect that we do not cache properly which might explain the unusual and unstable behaviour.

Not breaking, but annoying

I disagree. This issue seems to be quite breaking for some people, especially since multiple components of Homarr seem to be affected.

let me point out more issues that have the same base

Amazing, thank you!
We probably won't fix this issue in 0.11 though.
We're making the last polish before releasing, but I can focus more on this issue here afterwards.
Anyhow, thank you everyone here for your engagement and effort - I'm sure we'll figure it out :)

I dug around on Google and it seems like axios does not have a cache at all?
https://github.com/axios/axios/issues/1878
You can use an intercepter though to add such behaviour:
https://www.npmjs.com/package/axios-cached-dns-resolve

One thing that confuses me is, that got seems to have a caching system in place:
https://github.com/sindresorhus/got/blob/e24b89669931b36530219b9f49965d07da25a7e6/package.json#L50
I've had it happen multiple times, that my PiHole notified me, that there were too many requests being sent - not sure if this is being caused by Homarr though.

One thing that would actually interest me is, if how many of you are accessing Homarr with a domain.
Since we know that axios does not cache, requests like these could be a serious problem:
https://github.com/ajnart/homarr/blob/ba6a3df2b599f88a3a9883b28f0c7e6bc293c74c/src/modules/torrents/TorrentsModule.tsx#L63-L87

One thing that confuses me is, that got seems to have a caching system in place: https://github.com/sindresorhus/got/blob/e24b89669931b36530219b9f49965d07da25a7e6/package.json#L50
I've had it happen multiple times, that my PiHole notified me, that there were too many requests being sent - not sure if this is being caused by Homarr though.

I mean how hard is it to switch to something that has caching ? Also @manuel-rw we need singleton for Api-wrapped stuff, this might enable caching. Or we can try to swap to sockets.
My biggest dream would be to have SQLite database that contains all possible intégration data and then we bind components to data, with like an observer

Ok, now is the part where I have to say a "mea culpa" on part of this issue. I have been able to solve it, and this may be an issue for other users as well. I'm very sorry for making you guys sweat over this, but I'm hoping this can be useful in the future for similar failures
I am using Unraid as the server where this is being used. I've been fighting issues with networking on Unraid, when using 3 network interfaces. A long lost solution I had, at a point, was to use the google servers as the DNS configuration of those interfaces. And that was the root cause of the Name Resolution part.
Once I changed that, downed all interfaces but one, and restarted Unraid, proper local Name Resolution started working, as I was now pointing correctly to my Pi-Hole. Even the scenario where I'm doing a more complex routing with reverse proxying (Pi-hole -> Nginx with HTTPS -> Vaultwarden). All working now.
That will not, however, solve the polling part you have been discussing.

One thing that would actually interest me is, if how many of you are accessing Homarr with a domain. Since we know that axios does not cache, requests like these could be a serious problem:

Well, I can say I am using a domain. My PiHole has several name translations, one of the is homarr.rabbit. Never seen if there are issues with flooding on the Pi though.

I've been fighting issues with networking on Unraid, when using 3 network interfaces. A long lost solution I had, at a point, was to use the google servers as the DNS configuration of those interfaces. And that was the root cause of the Name Resolution part.

Interesting... So perhaps Homarr is not the issue, but maybe Docker and it's DNS resolution?
Can somebody else confirm this?
I sadly don't have the time as I try to keep up with all bug reports an issues of 0.11

Sorry @manuel-rw for never answering this. The issue was a mix of Unraid's Docker and the DNS I had setup, indeed. The best I can offer, is that ever since I've fixed a few things (like setting some options to allow the docker networks to better communicate with the outside), I've stopped having most issues.
I've gone as far as to now being able to have a PiHole, that redirects certain names to Nginx Proxy Manager, and have it return the IP and port, which essentially lets me have names that give me access to the full service. And yes, they are configured with those names in Homarr. Not sure this helps this specific case though.
If anyone else is having issues with that, I can definitely try to help, but I'm not constantly active here. @kingp0dd maybe if you add more info on your network setup, I could try to help.

Is this still an issue?

No, this has been solved. You can colose this @Meierschlumpf


[or-tools] adds or-tools as new port

Adds Google or-tools lib as a port.

highs solver is useable as an optional feature
coin-or solver couldn't be added because of https://github.com/microsoft/vcpkg/issues/39342. Once this is issue is resolved coin-or can also be added as an optional feature

[X] Changes comply with the maintainer guide.
[X] The name of the port matches an existing name for this component on https://repology.org/ if possible, and/or is strongly associated with that component on search engines.
[X] Optional dependencies are resolved in exactly one way. For example, if the component is built with CMake, all find_package calls are REQUIRED, are satisfied by vcpkg.json's declared dependencies, or disabled with CMAKE_DISABLE_FIND_PACKAGE_Xxx.
[X] The versioning scheme in vcpkg.json matches what upstream says.
[X] The license declaration in vcpkg.json matches what upstream says.
[X] The installed as the "copyright" file matches what upstream says.
[X] The source code of the component installed comes from an authoritative source.
[x] The generated "usage text" is accurate. See adding-usage for context.
[X] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[X] Only one version is in the new port's versions file.
[X] Only one version is added to each modified port's versions file.

Are there library components here or is this entirely providing a tool? We generally don't put tooling in our registry unless there is some kind of interdependency with the exact tool and some other library component, because we believe binary distribution systems like nuget or apt or dnf are a better fit for those situations. For example, protoc must match the libprotobuf you get exactly for the resulting generated bits to be link compatible, so we will distribute that.
I'm marking this PR as a draft because several builds are failing, please either fix the supports expression to describe what the port actually supports and mark 'ready for review' afterwards.

This is a library for solving optimization problems. The executables build also (the tools) are not really needed. Your right. I'll try to disable the build of them.
I'll also have a look at the supported platforms

@Mengna-Li I disabled the build of the math_opt targets. Now only the really needed ortools target is build.

Unfortunately, there still seems to be a problem.
To use BUILD_MATH_OPT=OFF I also had to set USE_GUROBI=OFFwhich makes sense since gurobi is a proprietary third-party solver.
However, with these settings I have linker errors on linux now. Looks like a bug in the cmake. I'll need to investigate this further.

On b135bb7: Is this expected to pick system libraries if available?
(Reusing the cached artifact will not install system libraries...)

@Mengna-Li It seems USE_GUROBI=OFF isn't supported (See: https://github.com/google/or-tools/blob/8edc858e5cbe8902801d846899dc0de9be748b2c/CMakeLists.txt#L256C1-L256C73)
Therefore BUILD_MATH_OPT=OFF is also not supported and need to be build.
However, I added a using file describing that only the ortools::ortools CMake Target should be used

@dg0yt No, system libs shouldn't be used. Did I make a mistake in the port file so they are picked?

It seems USE_GUROBI=OFF isn't supported

Why shouldn't it be supported to turn it off? That CMake line doesn't enforce it, but adds preqrequisites.
This is one of the "system libs":

@dg0yt I'm not sure why the flag exists if it can't be used. That was confusing me too.
If you compile the lib with USE_GUROBI=OFF you get linker errors when trying to use the lib.
Fromor-tools CMakeListst.txt:
## GUROBI
# Since it is dynamicaly loaded upon use, OFF is currently not supported.
CMAKE_DEPENDENT_OPTION(USE_GUROBI "Use the Gurobi solver" ON "BUILD_CXX" OFF)
message(STATUS "Gurobi support: ${USE_GUROBI}")

In my opinion these options shouldn't exist if they can't be used.
However, because the gurobi lib would be "dynamicaly loaded upon use" there is no hard dependency on gurobi even when USE_GUROBI=OFF is not set

If you compile the lib with USE_GUROBI=OFF you get linker errors when trying to use the lib.

Upstream bug IMO.

However, because the gurobi lib would be "dynamicaly loaded upon use" there is no hard dependency on gurobi even when USE_GUROBI=OFF is not set

I see...

Note: I will be converting your PR to draft status. When you're ready, please revert to "ready for review". That way, I can be aware that you've responded since you can't modify the tags.


move grpc embedded snippets to projects files

Fixes #28555

Internal previews

📄 File
🔗 Preview link

aspnetcore/grpc/services.md
Create gRPC services and methods

@timdeschryver  thanks again, great work, much appreciated.

@timdeschryver your changes will be live in 15 minutes if you want to take a quick check. The internal build looked good to me so I'm sure it's fine.


SSL?

There doesn't seem to be a feature or any documentation for SSL connections. Is that by design or does SSL just work if you need to use it?

I don't think Avocado needs to be involved in that, since this applies at a lower level, to an entire MongoDB connection, and it can be directly set in the client, e.g. Client::with_uri("mongodb://localhost:27017?ssl=true"). I'd assume if you use any database and collection coming from such a client (e.g. by requesting one wrapped in an Avocado collection) that it would just use SSL.


User settings wiped

VSCode Version: Code 1.12.1 (f6868fce3eeb16663840eb82123369dec6077a9b, 2017-05-04T21:40:39.245Z)
OS Version: Linux x64 4.4.0-77-generic
Extensions: the listing exceeds the lower minimum of browsers' URL characters limit

I cant reproduce the bug so this is probably unhelpful, but here's how it happened:

Pressed ctrl + to zoom & got a message VS needs to restart to apply settings.
Clicked cancel (was on a roll).
VS reset to default everything on clicking cancel.
Closed VS & reopened but the settings where still the default.
Checked .config settings json and it was certainly wiped.

Cheers :)

I have made a new settings file and tried to reproduce this but can't even get the "need to restart to apply settings" when I am zooming. I tried hitting all the keys that my fat fingers may have slipped to and nothing.
Anyway I am sure there are better things to think & spend time on than this. I will let you know if this happens again.
Thanks for awesome editor btw!

I think @jrieken saw this too, at least the dialog (unsure if settings got lost)?

Correct, I and @alexandrudima have seen this when zooming in to take a closer look at the cursor. I do have the native window style configure to be native ("window.titleBarStyle": "native") but I don't think that hand an impact. I was also afraid about having lost my changed because until restart the default theme was used.

I saw it on @jrieken 's machine. All he did was to do ctrl+= (i.e. zoom in) many times to zoom in multiple times.

Duplicates #25427

@motebotic Very sad to know that settings are wiped out. I already have an issue #25427 opened for this and collecting data about this. Pain point is that, it was not reproducible and I am try to figure out what could cause this. Thanks for reporting it.

@bpasero I think I had "window.titleBarStyle": "native" as I was getting some strange flickering that was driving me nuts some time back & read about that as a cure. Since I don't have my old setting though I cant say for certain. I had turned telemetry.enableCrashReporter off.. sorry that's on now ;) although it didn't crash so it wouldn't have helped.

@sandy081 it looks to me that what happened to @jrieken is that briefly all settings got lost and thus the dialog got triggered (this would also explain why @jrieken saw the default theme being used). So this seems to be the cause, but still does not explain why settings go empty randomly...

@bpasero @sandy081 what could cause vsc to think that a setting had changed in both mine & @jrieken case. I had one window open, split in two- with two JS files on the left side & one on the right side. I was working in file on right & had an invalid JS file holding some glyph's on the left in a hidden tab. I clicked the valid JS file on the left side, hit ctrl tab to change to the icon file & started zooming in when the dialog popped up. The zoom was slow to respond when I hit it the first time & being inpatient & on a roll I hit it multiple times again before the dialog happened. When I cancelled the dialog & seen the default theme appear, side bar etc I hit ctrl, to check the settings.json & seen it was empty all but the window zoom level. I closed the settings files & restarted VS code hoping my settings would come back, they did not. I tried to reboot & still empty, googled, checked the settings.json in .config/.... & that file only contained the window zoom level.
 C=could it be something to do with trying to write to the settings.json as something else is going on & returns that the file doesn't exist? When I changed focus from the right to left my file would have automatically saved & I was moving quick. I have no idea how this works just throwing things out there.
So yeah hope that was of some help, or at least made .. sense ((:


Highlight on Yank broken when using Mini.jump2d

Contributing guidelines

[X] I have read CONTRIBUTING.md
[X] I have read CODE_OF_CONDUCT.md
[X] I have updated 'mini.nvim' to latest version

Module(s)
mini.jump2d
Description
Mini.jump2d is interfering with custom yank on highlight autocommand
Neovim version
v0.8.0-dev+198-g030417d80
Steps to reproduce
Take the following minimial neovim config
vim.call('plug#begin', plugged_path)
Plug('echasnovski/mini.nvim')
vim.call('plug#end')

vim.api.nvim_create_autocmd('TextYankPost', {
  pattern = '*',
  callback = function(args)
    vim.highlight.on_yank()
  end,
})

vim.keymap.set('n', 'Y', 'y$', { desc = 'Yank till the end of the line' })

local enable_jump2d = 0

if enable_jump2d == 1 then
require('mini.jump2d').setup({
  labels = 'etovxqpdygfblzhckisuran',
})
end

Open a file and press y1<cr> and observe, that the yanked lines are highlighted. Now set enable_jump2d=1 and do the same again and observe, that the mini.jump2d labels are popping up.
Expected behavior
Yank is properly highlighted with enabled mini.jump2d
Actual behavior
Instead of highlighting the yanked lines, mini.jump2d labels are popping up

Thanks for making an issue!
The reason this is happening is that with y1<CR> you are using <CR> in Operator-Pending mode. By default it is mapped to MiniJump2d.start() in that mode also, but you seem to expect its default motion ("[count] lines downward, on the first non-blank character |linewise|").
There are several options here:

Use y2_ instead of y1<CR>.
Use different mapping to start jumping. For example,   require('mini.jump2d').setup({ mappings = { start_jumping = '<M-m>' } }).

Yes, thanks for the explanation.


cleaning up notebooks/Snapping_Demonstration.ipynb

updating snapping_notebook

Coverage seems have gone down by ~35% following this commit/PR. Not sure the reason. Figure out what's happening.


Arduino: Link arduino-builder binary

Thank you for the contribution. It was merged directly as a7d6f631071d56424eac7c3931ba259f6ebecb9f to keep commit history cleaner. Your contribution is still credited to you.


Refactor ITMS configuration

This PR aims to allow an easy configuration of the ITMS uploader parameters, having the default value that fixes previous known errors.

Thanks for the PR @fwigorcferreira, we'll review it asap! 😊

Hi @fwigorcferreira ,
thanks for the pr, it looks great to me!


Make the link in wordpress.md point to the right engine doc

Proposed changes
Change to the correct link. Although there's a redirect, it's better to link to to the right place.

Thanks, @axilleas :)


Minimum samples? for using a custom image set (360)

Hi,
My objects are captured from an upper semi-sphere in relatively equally spaced elevations/azimuths and I have ~600 images per object. I'm interested in training on <100 images.
Will this produce quality renderings on novel (test) views?
What techniques, applicable to this codebase, are there for improving rendering quality (PSNR) with a low number of input samples?
Thanks!

The main results in the paper use the NeRF-synthetic dataset which has 100 training views and produces good results on new/test views. We also include an ablation study with 25 training views, where the trick for retaining reasonably good test results is to increase the weight on the TV regularization.


Dropbox SSO Settings

When entering the Basic SAML Configuration we need to provide
Identifier (Entity ID): Dropbox
Reply URL: https://www.dropbox.com/saml_login
Sign on URL: https://www.dropbox.com/sso/
The Dropbox Sign SSO ID can be found in the Dropbox site at Dropbox > Admin console > Settings > Single sign-on > SSO sign-in URL.

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

ID: 6884bf55-8c3b-aa39-43e3-c0b44dcd6efb
Version Independent ID: 9f606de5-73dd-8985-cf06-0bc9877c0352
Content: Tutorial: Azure Active Directory integration with Dropbox Business
Content Source: articles/active-directory/saas-apps/dropboxforbusiness-tutorial.md
Service: active-directory
Sub-service: saas-app-tutorial
GitHub Login: @jeevansd
Microsoft Alias: jeedes

@merill
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.


The syntax color and highlighting for react file is not the same as documented

Yeah, people wanted some changes since I released- mainly having a different color for components than html tags. I haven't updated the github screenshot yet. Will do soon.

Hmmm, that purple for the import should be different though. Let me take a look and get back to you.

This looks very different from what I'm seeing. Can you please

Let me know what version you're on
PR the demo folder with this file

I'm not able to reproduce. Thanks!

I’m using the latest version of vscode and the js syntax color in my react project looks same like in that screenshot

What version of this theme are you on? It should be 0.5.0 and you may need to restart vscode. If you're still seeing this, please PR the demo folder so I can try to duplicate :)

Already solved in night owl version 5.0 😅.  Thank you for helping me fix this issue @sdras.

My pleasure! Thanks


Core Dumped issues

Hello everyone,
Ubuntu and Linux total noob here.
I installed the neural-style on a virtual machine of ubuntu 14 - 64 bit (VirtualBox, i5 and 4go of ram). Everything went fine so far.
Unfortunatelly, when I test the code 'th neural_style.lua -gpu -1 -print_iter -1', I get this error :
[libprotobuf WARNING google/protobuf/io/coded_stream.cc:505] Reading dangerously large protocol message.  If the message turns out to be larger than<PHONE_NUMBER> bytes, parsing will be halted for security reasons.  To increase the limit (or to disable these warnings), see CodedInputStream::SetTotalBytesLimit() in google/protobuf/io/coded_stream.h.
[libprotobuf WARNING google/protobuf/io/coded_stream.cc:78] The total number of bytes read was 574671192
Successfully loaded models/VGG_ILSVRC_19_layers.caffemodel
conv1_1: 64 3 3 3
conv1_2: 64 64 3 3
conv2_1: 128 64 3 3
conv2_2: 128 128 3 3
conv3_1: 256 128 3 3
conv3_2: 256 256 3 3
conv3_3: 256 256 3 3
conv3_4: 256 256 3 3
conv4_1: 512 256 3 3
conv4_2: 512 512 3 3
conv4_3: 512 512 3 3
conv4_4: 512 512 3 3
conv5_1: 512 512 3 3
conv5_2: 512 512 3 3
conv5_3: 512 512 3 3
conv5_4: 512 512 3 3
fc6: 1 1 25088 4096
Erreur de segmentation (core dumped)

I tried with -image_size 150 -optimizer adam, same issue. I tried with my own files and the exact same error occured. Monitor show a 70% memory usage before it crashed.
Anyone had the same problem ?
Cheers,
Tien

Try running it not in Virtualbox, or with more ram. (my guess)


function math running max added

function math running max added and covered by tests

Thank you @anastansa for the PR!
There are some conflicts after merging your previous PR.
Please rebase this branch from actual dev and resove the conflicts.

Thank you @Smoren!
I've resolved all conflicts, so this branch is ready to be merged:)

Thank you @anastansa!


Fix: SfTabs content on dektop

Related issue
no
Scope of work
fix content showing in the same line with title
Screenshots of visual changes

Checklist

[x] No commented blocks of code left
[x] Run tests and docs
[x] Self code-reviewed

If applicable:

[ ] I followed composition rules for my component
[ ] I tested the component in most common device sizes (can be tested in Storybook from viewport addon in top menu)
[ ] I accept Individual Contributor License Agreement

@AdamPawlinski awesome, but this task needed more changes https://github.com/DivanteLtd/storefront-ui/commit/8e9d9da8fec1d8178478f5d1047a14411d2b3ae1


Fix #1298 by updating max_leaves to max_leaf_nodes

Why are these changes needed?
Fix #1298
Related issue number
Closes #1298
Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/FLAML/. See https://microsoft.github.io/FLAML/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

Hi @hannahbus , could you help to verify if this PR fix your issue #1298 ? Thanks.

Can you remove me from these emails?
On Mon, Jul 22, 2024 at 9:56 AM Li Jiang @.***> wrote:

Hi @hannahbus https://github.com/hannahbus , could you help to verify
if this PR fix your issue #1298
https://github.com/microsoft/FLAML/issues/1298 ? Thanks.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/FLAML/pull/1320#issuecomment-2243028525,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADA4OR5D3CONGV2CLW5M5LDZNUFRBAVCNFSM6AAAAABLILXJDKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBTGAZDQNJSGU
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

@thinkall yes, and many thanks!

Can you remove me from these emails?
…
On Mon, Jul 22, 2024 at 9:56 AM Li Jiang @.> wrote: Hi @hannahbus https://github.com/hannahbus , could you help to verify if this PR fix your issue #1298 <#1298> ? Thanks. — Reply to this email directly, view it on GitHub <#1320 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADA4OR5D3CONGV2CLW5M5LDZNUFRBAVCNFSM6AAAAABLILXJDKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBTGAZDQNJSGU . You are receiving this because you are subscribed to this thread.Message ID: @.>

Hi, @varun-suryan , I can't add or remove you from the emails. You're receiving this because you are subscribed to the thread. You can click the link in the email to unsubscribe.

@thinkall yes, and many thanks!

Hi @hannahbus , it turns out that updating this config will make RandomForestEstimator not be compatible with other many estimators in FLAML. The best_config here is used for FLAML, it will be converted to sklearn RandomForestClassifier parameters with config2params. I think the best way to address your issue is by applying below code:
from flaml.automl.model import RandomForestEstimator

RandomForestEstimator(task=<your task type, such as "classification">).config2params(automl.best_config_per_estimator['rf'])

I'll close this PR, let me know if you've other suggestions. Thank you!


updates footer design and adds additional links

@xuelink I updated the footer design, as well as added the 3 new links. I tested it on 4 major browsers. Safari, Firefox, Chrome and Edge so far. If you see anything out of the ordinary let me know.

We also add new section for Token seperately instead of listing under Company, because day by day we will increase these links about token.
Token

Website: https://token.langx.io/
Twitter: @langx_token
Whitepaper: Gitbook Draft

@xuelink Yeah I can try and add more links and see how it goes!

@xuelink I added more columns and links, let me know what you think. Not sure what you meant by the comparisons though

@xuelink No problem!


Update to the latest conventions & workflows

Reviewer checklist

[ ] Read the contributing guide
[ ] PR should be motivated, i.e. what does it fix, why, and if relevant, how
[ ] Ensure relevant issues are linked (description should include text like "Fixes #")
[ ] Ensure any appropriate documentation has been added or amended

Coverage: 98.238%. Remained the same when pulling eb61bfda2da502ed00ee2519444de87049df9015 on conventions into 5a6c6250011e1ec7535fd24dde91f6f9d1358a58 on main.


Can't add String to text message

Hi!
I was just using your library and I found it very useful! I was just wondering if was possibile to also send some String in the text message in the email.
For example:
 message.html.content = "
Temperature is " + String(temp, 1) + "°C
Humidity is " + String(humidity) + "%
Pressure is " + String(pressurehpa, 1) + "hPa
Wind Speed is " + String(WindSpeed, 1) + "km/h from " + DIRECTIONWIND + "
Rain Today " + String(mmPioggia, 1) + "mm
Rain Intensity is " + String(rainrate, 1) + "mm/h
Current UV is " + String(UVindex) + "

Daily Weather Observations

Highest " + String(maxTemp, 1) + "°C
Lowest " + String(minTemp, 1) + "°C
Gust " + String(Gust, 1) + "km/h
Maximum Intensity of Rain " + String(rainrateMax, 1) + "mm/h

To stop this notification go to Services/EmailAlert/EmailNotification and type 'OFF'";
At the moment it gives an error if I try to do that. The Strings are variable.
And also, is it possible to have the possibility to be a variable also the String of the email of the recipient?
For example.
String email =<EMAIL_ADDRESS>
 message.addRecipient("Admin", email);

Much appreciate for some feedback!
Thank you!

Use c_str() function from String to get the const char array of the string like this.


New term Vertebral Artery Dissection

Closes #7593
Following in new term

Definition with 1 PMID
2 PMID dbxref
Child term of cervical artery dissection

Closing this branch as we concluded VAD is not a disease, and an external factor, #7593


NextJS + Turso + Drizzle issue

Referenced here from another user: https://github.com/tursodatabase/libsql/issues/1276
I get the same error and using libsql client. I tried downgrading multiple times of libsql and it was always getting the same error.
I am using nextjs Version 14.0.3 and have also tried 14.0.2

Hello, I got the same error (on a monorepo setup), here's how I fixed it https://github.com/tursodatabase/libsql/issues/1276#issuecomment-2065694434

Hard to undestand why a "minor" update caused this breaking change

A solution from #184:
import {} from "@libsql/client/web"


Updating & refreshing user metadata causes Error "Invalid Refresh Token: Already Used"

Bug report

[x] I confirm this is a bug with Supabase, not with my own application.
[x] I confirm I have searched the Docs, GitHub Discussions, and Discord.

Describe the bug
I'm trying to use the Supabase user metadata as a means of storing user metadata (like preferences eg. is_subscribed_to_newsletter), since it seems way more intuitive than having a clone of it in Postgres, and updating those... This is my code:
await supabaseClient.auth.updateUser({
            data: {
                ...metadata,
                ...updatedMetadata
            }
        })
        const { error } = await supabaseClient.auth.refreshSession()

Basically, the first time I update any user metadata, it works perfectly, but the second time and all times after that I get the error "Invalid Refresh Token: Already Used"
As seen here and here, other people are also facing this issue.
To Reproduce

Create a Next app with an authenticated user
call updateUser, then refreshSession
Try to updateUser, then refreshSession again, and you will get error "Invalid Refresh Token: Already Used"

Expected behavior
I expect the refreshSession to also refetch a new refresh-token, while refetching updated account information for the useUser hook.
Being able to use the user_metadata field for small personal user preferences is super good for DX. I always cringed at the idea of creating some Postgres triggers (that last I checked weren't even production ready) only to create a matching public.users table that then contained the users data, when we could just use the user_metadata itself.
System information

OS: macOS
Browser (if applies) Chrome
Version of supabase-js: 2.26.0
Version of Node.js: v18.15.0

Friendly bump :)

Friendly bump :)

This is tracked in https://github.com/supabase/gotrue-js/issues/755


X-Auth-Key vs Bearer access

Terraform version
0.12.29
Affected resource(s)
Cloudflare provider 2.11.0
Description
Consider following config:
provider "cloudflare" {
  version    = "~> 2.0"
  email      = "***"
  api_key    = "***"
}

I want to authorize the plugin with email and api_key. When running locally, everything works correctly and I can see within the trace logging following:
[DEBUG] plugin.terraform-provider-cloudflare_v2.11.0: ---[ REQUEST ]---------------------------------------
[DEBUG] plugin.terraform-provider-cloudflare_v2.11.0: GET /client/v4/zones/*** HTTP/1.1
[DEBUG] plugin.terraform-provider-cloudflare_v2.11.0: Host: api.cloudflare.com
[DEBUG] plugin.terraform-provider-cloudflare_v2.11.0: User-Agent: HashiCorp Terraform/0.12.29 (+https://www.terraform.io) Terraform Plugin SDK/1.15.0 terraform-provider-cloudflare/2.11.0
[DEBUG] plugin.terraform-provider-cloudflare_v2.11.0: X-Auth-Email: ***
[DEBUG] plugin.terraform-provider-cloudflare_v2.11.0: X-Auth-Key: ***

However, when I run the same setup in CI, for some reasons, the requests look different as Bearer is used instead of X-Auth-Key:
[DEBUG] plugin.terraform-provider-cloudflare_v2.11.0: ---[ REQUEST ]---------------------------------------
[DEBUG] plugin.terraform-provider-cloudflare_v2.11.0: GET /client/v4/zones/*** HTTP/1.1
[DEBUG] plugin.terraform-provider-cloudflare_v2.11.0: Host: api.cloudflare.com
[DEBUG] plugin.terraform-provider-cloudflare_v2.11.0: User-Agent: HashiCorp Terraform/0.12.29 (+https://www.terraform.io) Terraform Plugin SDK/1.15.0 terraform-provider-cloudflare/2.11.0
[DEBUG] plugin.terraform-provider-cloudflare_v2.11.0: Authorization: Bearer ***

Notice the same version of both Terraform and Provider. However some of the requests with the Bearer are successful, but some of them don't – which is strange – it seems like it uses the X-Auth-Key to create the Bearer, but it does not assign enough rights to it.

Where does the Bearer come from (as I don't use apiToken configuration)?
Why does provider use the Bearer instead of X-Auth-Key?

Expected behavior
In both settings I would expect it to use the X-Auth-Key.
Actual behavior
Usage of Bearer rises from nowhere.
Steps to reproduce
This is probably impossible to reproduce, I am not sure what causes this. I hope someone will know how does this work and help me debug this more.

The API token is triggered by setting the CLOUDFLARE_API_TOKEN environment variable (or api_token if you set it) and it takes precedent over the the API key configuration.
You will either need to unset the variable or migrate to using API tokens which provides a finer grained permission model.

Thanks. Maybe it would be great to log this as this is a bit unexpected as I was unaware of the fact we have this env var set.


KYLIN-3888 TableNotDisabledException when running "Convert Lookup Table to HFile"

See: https://issues.apache.org/jira/browse/KYLIN-3888

Can one of the admins verify this patch?

Can one of the admins verify this patch?


Subgraph should not have duplicate nonce

Describe the bug
To Reproduce
Steps to reproduce the behavior:
If you see:
https://api.thegraph.com/subgraphs/name/connext/nxtp-amarok-runtime-staging-kovan/graphql?query=query+MyQuery+{
++originTransfers(orderBy%3A+nonce%2C+orderDirection%3A+desc%2C+first%3A+100)+{
++++transferId
++++nonce
++}
}
We have duplicate nonce:
{
  "data": {
    "originTransfers": [
      {
        "transferId": "0xbd27fe3822e37289bc66ba4364d8acbd2c8ebd26ec71f1669ad224078c8d3e2a",
        "nonce": "11"
      },
      {
        "transferId": "0x287c3d119ce4fb22505cee3060cbb7ce798ade585c33ee4c9c4ddeae05f7bd59",
        "nonce": "10"
      },
      {
        "transferId": "0xa4069f583e87f8d02bd37f3ee8c1041b8dbe09e9c926d9c7d23f0c3264a14ce8",
        "nonce": "9"
      },
      {
        "transferId": "0x62c7bc36e79072ffcaf9e14983e7e381da4273d20de93eb4d03a19f35a19f282",
        "nonce": "8"
      },
      {
        "transferId": "0x2765dcd6e3705802276883404aef299bc9690998a59ac1839acc65cc5919ffd5",
        "nonce": "7"
      },
      {
        "transferId": "0xe2e0b4f9de412480e3a72bd0eadb7caaac23195b5970640debbed42cb9c2448e",
        "nonce": "6"
      },
      {
        "transferId": "0x6312990bc4b577485b26291a7d01efb604add585a0901fe5f98315d67df5ea53",
        "nonce": "6"
      },
      {
        "transferId": "0xed388c88b22abec6878150d642d18eae7d52c1831bca529f5563c5b3ac0f127b",
        "nonce": "5"
      },
      {
        "transferId": "0xc4c90e680bbb1e47588a637a40e6dbaf66d4d5f6904aba1f24e5418bc68e853c",
        "nonce": "5"
      },
      {
        "transferId": "0xea6b3f1eca8196fedbdc9d63d61563376d5e80289a0b182635f6fbb8581a9337",
        "nonce": "4"
      },
      {
        "transferId": "0xba242677fe0255a9e7ea4529c6cd2bfcd77764882a44595639fc78494194dbb5",
        "nonce": "4"
      },
      {
        "transferId": "0x9841e1033d9724aac371382fddc58237ed98e6f7ef2773effddebdfd621ea631",
        "nonce": "3"
      },
      {
        "transferId": "0x0650b6df949c6bf7ebd03a9dda3e605d54e44271d11c1f9dcaa4cfee502433c6",
        "nonce": "3"
      },
      {
        "transferId": "0xe0f0c95366dd428980fcfc2e7064b02f617e962ae7d00840a1aa4da258ff1cf4",
        "nonce": "2"
      },
      {
        "transferId": "0x4616d1b0600b8cae912934788926757121739b814220f01fc2a7f3c47da38ba5",
        "nonce": "2"
      },
      {
        "transferId": "0xfd14ec5b10530fdc69dbdf9eb78db7358de3b172df1a65ab4e59225135359cc7",
        "nonce": "1"
      },
      {
        "transferId": "0x7add85afebb2f2baa1018320f24d086e8497886f487f3ab14bf9b9a352eed0a8",
        "nonce": "1"
      },
      {
        "transferId": "0xae74eef9d4934f9eeb0ca63b4a111e081d38faedcd0effeb8ed402d1ac69716c",
        "nonce": "0"
      },
      {
        "transferId": "0xa5d662de4201a0d3b98088f3f633846fd3887e23387a2e873ece03fdec539b7a",
        "nonce": "0"
      }
    ]
  }
}

Additional context
This was found only in Kovan staging subraph:
https://api.thegraph.com/subgraphs/name/connext/nxtp-amarok-runtime-staging-kovan

It seems like the subgraph is reading a history of stale XCalled events. Since we are using the diamond proxy pattern, the address is maintained across deployments, so the event history would also be maintained (by provider network). It's unclear to me why the storage nonce would be reset back to 0 on a subsequent deployment. Perhaps core architecture/proxy got redeployed, but I would think that would mean the address would change.
To fix, we should change the "startBlock" in packages/deployments/subgraph/config/amarok-runtime-staging.json to match the deployment block of the contract on kovan: 0xD9C3bc89e8B14a8B01481f0748e24d3d7fa18DB2.
This does not seem to be a bug with the underlying contract code or the subgraph mapping.

The block number of the contract create seems to match what's currently set as the startBlock.
It appears that DiamondInit, one of the facet contracts, was redeployed at block 31982545:
https://kovan.etherscan.io/tx/0x02bb21a47bcc7117315887f721fcb6d0cd1f9e4958926f511c39c28379d58104
Which is > 31682492, the current "startBlock"
And this contract's init method resets the nonce to 0. The block number for this contract's deployment should be what we set the "startBlock" to in packages/deployments/subgraph/config/amarok-runtime-staging.json under kovan


feat: Allow re-use of existing materials

As of now, the materials for particles are always created from scratch (using <ParticlesMaterial>, which uses three-custom-shader-material to patch our shader code into newly created instances of Three's built-in materials.)
Ideally, we'd be able to patch existing materials. This is important for eg. using materials imported from a GLTF file.
This probably requires a PR to three-custom-shader-material in order for it to accept existing instances of materials.

This now works, thanks to three-CSM 3.4.0.


Add license file (MIT)

Things added/changed:

Add license file (LICENSE.md, MIT).

thanks again :smile:


replace deprecated angle brackets

See page 17 of the spec linked from the unicode-math-class repo. The angle bracket character was changed. Currently, using angle.l 1/(1/2) angle.r results in it not correctly resizing the brackets, whereas ⟨ 1/(1/2) ⟩ does. This fixes that.

To quote the linked document: yes, the other angle brackets are deprecated. Maybe it's better to change.

That having said, with the current font it gives identical result anyway?
//#show math.equation: set text(font: "Latin Modern Math")
#show math.equation: set text(font: "DejaVu Math TeX Gyre")

#table(columns: (auto, auto),
[Not resized], [$angle.l 1/(1/2) angle.r$],
[resized angle.l .. angle.r], [$a lr(angle.l 1/(1/2) angle.r) b$],
[resized using deprecated brackets], [$a lr(〈 1/(1/2) 〉) b$],
[resized using Unicode characters], [$a ⟨ 1/(1/2) ⟩ b$],
[not resized], [$\⟨ 1/(1/2) \⟩$],
[resized using \\⟨...\\⟩ and lr()], [$lr(\⟨ 1/(1/2) \⟩)$],
[using deprecated brackets], [$〈 1/(1/2) 〉$],
)


Reference script bounds and cost formula

Description
This closes issue #537.

Checklist

[x] Commit sequence broadly makes sense and commits have useful messages
[x] Code is formatted according to CONTRIBUTING.md
[x] Self-reviewed the diff

The new code (and the Everything module) type-checks.  I'm not sure why the CI is failing.  The error message is not very helpful.

Ah, I found this in one of the log files.

MAlonzo/Code/Ledger/Foreign/LedgerTypes.hs:166:39: error: [GHC-76037]
    Not in scope: type constructor or class ‘ℕ’
    |
166 |   , ppMaxRefScriptSizePerTx        :: ℕ

I'll fix that today.


maxSockets per host?

I've written this to accomplish the goal, but is there a way to do it out of the box with request?
I'd tried using pool:{ maxSockets:1 }, but it appeared to be maximum sockets regardless of host. I want to blast out requests, but never more than one on each host.

I don't want to modify any global settings such as
require('http').globalAgent.maxSockets = 1;

require("request").httpModule.globalAgent = new http.Agent({ maxSockets: 1 });

Or maybe?:
var myAgent = new http.Agent({ maxSockets: 1 })
request({ agent: myAgent })

Nevermind. :smile: Heheh.

Did you find a solution to this?  Not something I've dealt with yet, but would be good to know.

It turned out that my tests were faulty and request's pool:{maxSockets:1} works just fine in a per-host basis.


Fix/pressure parser error

resolves #16

A more meaningful exception (e.g. a TypeError with a descriptive message) would be better than an AssertError.

I'd prefer an assert statement for the following reasons:

a TypeError is fine when you want to inform the dev that they're misusing a function you wrote; that's an "expected" error in a sense.  This is producing an error for a fundamental assumption on which the code is based, and If that assumption is wrong for any reason, it means the code is bad. There is no circumstance under which this should ever be raised, and a user (including other Python dev's) should never see this -> making it a TypeError isn't adding any value in particular.
Since this should never actually get triggered, it'd be fine to remove the assert all together for performance purposes for production code; executing code with the interpreter's optimization flag does just that for assert statements. Alternatively I could explicitly raise an error if __debug__ == True I guess, it's just more verbose.

However I think it's fine to, for clarity, add an error message to the assert explaining the failure.

A more meaningful exception (e.g. a TypeError with a descriptive message) would be better than an AssertError.

I'd prefer an assert statement for the following reasons:
1. a `TypeError` is fine when you want to inform the dev that they're misusing a function you wrote; that's an "expected" error in a sense.  _This_ is producing an error for a fundamental assumption on which the code is based, and If that assumption is wrong _for any reason_, it means the code is _bad_. There is **no** circumstance under which this should **ever** be raised, and a user (including other Python dev's) should **never** see this -> making it a `TypeError` isn't adding any value in particular.

2. Since this should never actually get triggered, it'd be fine to remove the assert all together for performance purposes for production code; executing code with [the interpreter's optimization flag](https://docs.python.org/3/using/cmdline.html#cmdoption-o) does just that for assert statements. Alternatively I could explicitly raise an error `if __debug__ == True` I guess, it's just more verbose.

However I think it's fine to, for clarity, add an error message to the assert explaining the failure.

So, this is essentially checking if the Python and Numpy implementations match up, right?  If we want to check something for the sake of the whole module, which this would be, would it make more sense to move this out of a class definition?  If this is a legitimate concern, should we also be checking platform native data sizes?  These are allowed to change from one platform to another, and so that actually could fail.

I misinterpreted what the assert was doing; I thought it was validating the channel's formatting string. That said, it seems like this sort of sanity check should be in the unit tests.

should we also be checking platform native data sizes? These are allowed to change from one platform to another, and so that actually could fail.

@CFlaniganMide actually there should be no platform where this fails: the left-side of the equality check is from the struct package, which uses standard sizes when an endian-ness is specified that should be consistent across platforms; the right side is numpy dtypes specified using the type-&-size style typestrings (e.g., 'i4', 'f2', etc.), and obviously those should always be the same size regardless of platform. So together the whole thing should be solid.

it seems like this sort of sanity check should be in the unit tests.

@StokesMIDE not my preference but it's addressed in 4aaa724b97e5e242413bb0fa40ef5dddd31a4c53


Keep QL images versioning the same

@wistefan we might to want to have quantumleap-pg-init:edge tagged and fixed as well...

Done: #37

Merged


ScalametaParser: implement ammonite script parser

Adds a new MultiSource tree, and a new Input.Ammonite wrapper which is used as an indication that multi-source tokenization is needed.
Fixes #2155.

@dos65 thank you. wanted to get confirmation that this approach will work with metals (the original question). cc @tgodzik
i have obviously already checked that it works with the formatter.

@kitbellew it should work well. the only we need is to check places where create Input and turn it into ammonite one


Create or Update API endpoint not working when updating an API

Summary
I was trying to use Kong endpoint to create/update a given API, but I am not able to update it after its creation. I have also checked issue #2151.
Don't know if this is the only Kong endpoint to 'create or update' that is not working.
Steps To Reproduce

Add a new API

$ curl -i -H "Content-Type: application/json" -XPUT --url http://localhost:8001/apis -d '{"name":"test","upstream_url":"http://localhost","hosts":"test"}'
HTTP/1.1 201 Created
Date: Fri, 15 Dec 2017 14:28:37 GMT
Content-Type: application/json; charset=utf-8
Transfer-Encoding: chunked
Connection: keep-alive
Access-Control-Allow-Origin: *
Server: kong/0.11.2

{"created_at":1513348117852,"strip_uri":true,"id":"57b4d8cb-d70b-456f-b256-b2310fffa035","hosts":["test"],"name":"test","http_if_terminated":false,"https_only":false,"retries":5,"upstream_url":"http:\/\/localhost","upstream_send_timeout":60000,"upstream_read_timeout":60000,"upstream_connect_timeout":60000,"preserve_host":false}

Change at least one parameter from previous added API

$ curl -i -H "Content-Type: application/json" -XPUT --url http://localhost:8001/apis -d '{"name":"test","upstream_url":"http://localhost","hosts":"test2"}'
HTTP/1.1 409 Conflict
Date: Fri, 15 Dec 2017 14:28:39 GMT
Content-Type: application/json; charset=utf-8
Transfer-Encoding: chunked
Connection: keep-alive
Access-Control-Allow-Origin: *
Server: kong/0.11.2

{"name":"already exists with value 'test'"}

Additional Details & Logs

Kong version: 0.11.2
Operating System: docker

tried PATCH ?

@Tieske yes, but my idea is to use PUT since it supposedly works for both API creation and update and I am creating a script to automatically adds APIs to Kong, so using PUT will avoid separate code for creation and update APIs.

PUT is somewhat limited/not fully RESTy. It's a known issue, will not be fixed shortly. Longer term probably.

PUT is somewhat limited/not fully RESTy.

Ref: https://tools.ietf.org/html/rfc2616#section-9.6

my idea is to use PUT since it supposedly works for both API creation and update and I am creating a script to automatically adds APIs to Kong, so using PUT will avoid separate code for creation and update APIs.

@hugomcfonseca There is a premise that the name must be the primary key. If we followed it to its logical conclusion, we don't have a chance to change the API's name again.

@Tieske
It is recommended to disuse the interface of the following form

/apis/{name}
/consumers/{username}
/apis/{name}/plugins/
/certificates/{sni}
...

the name or username is primary key, but it is not primary key when requests method is PUT, this will confuse people.

@hugomcfonseca It seems to me that your original request has been answered (use PATCH instead for now, as a workaround). May I close this issue?

yes @kikito.
many thanks to all.

Can I suggest the documentation is updated?
The https://getkong.org/docs/0.11.x/admin-api/#update-or-create-api clearly states that you can update using PUT. I am getting 409 too on PUT, it's behaving as if it's POST.

@paulseddon the http methods (and more things) will (gradually, with deprecations) change in future versions of Kong. For now I will close the issue.


Improve test runners bin packing

This configuration allows to run 6 test runners/pods on one 8-node k8s node

/gcbrun


Substat Optimizer - No debug version

From my understanding there's currently no way of disabling the debug log run when calling the sim, this makes run times take longer; originally this isn't a big issue per se when simulating normally, but it could take a bigger impact on the subs stat optimizer.

@srliao I'm more interested in which aspects of this sim could be disabled during a substat optimization run. In theory I think we could turn off some of the logging operations, though this may impractical/impossible.
Turning off the debug log run is along that same vein, but small potatoes in the grand scheme of things.

Note to self, I'm like 65% sure that we queue up all of our sim runners as goroutines and distribute substats after all goroutines have returned. I don't think there's any reason to wait on each run of the sim. I think currently the substat optimizer is psudeo-synchronous processing 10-20 runs of gcsim.

Next idea, can we do small sim_iter runs during optimization passes to guide the larger sim_iter runs?

Staeld — Today at 9:06 PM
If I understood correctly, Raiden has a special case because she scales with ER, right? Shouldn't Mona also be considered or her ER scaling is not big enough to be a problem?

no longer relevant


[5.7] Use container for Fire app callbacks.

We can simply extend the ability of booting and booted methods to accept Class@method notation to be consistent with what the terminating callbacks can accept.

I think this would be better targetted at master.

Is it backward incompatible ? or it is related to versioning strategy ?!
@GrahamCampbell

Well, you extended the method to do more than it did before so all other contract implementation will no longer be valid as they won't support Class@method notation so this is a breaking change and should target master.

@X-Coder264 Nice, Thanks for feedback.


Adds Third DevTools for Beginners Document

What's changed, or what was fixed?
See title. In draft stage as of 8/20.
Fixes: N/A
Target Live Date: 2018-09-07

[ ] This has been reviewed and approved by @kaycebasques
[x] I have run npm test locally and all tests pass.
[x] I have added the appropriate type-something label.
[ ] I've staged the site and manually verified that my content displays correctly.

CC: @petele

Whoops!
There were 4 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/tools/chrome-devtools/beginners/js.md#L152 - Common typo found: 'Javascript' Should it be 'JavaScript'?
src/content/en/tools/chrome-devtools/beginners/js.md#L208 - Common typo found: 'Javascript' Should it be 'JavaScript'?
src/content/en/tools/chrome-devtools/beginners/js.md#L333 - Common typo found: 'webpage' Should it be 'web page'? See: https://developers.google.com/style/word-list#web_page
src/content/en/tools/chrome-devtools/beginners/js.md#L424 - Line must be at most 100 characters


Move "clustered wallaroo" to appendix of book

As part of a review of "core concepts" while working on Go
documentation, I realized that the current "clustered wallaroo"
documentation is very "how to with machida" oriented and doesn't make
sense for a general "language agnostic" core concepts.
This content is really more of a "running Wallaroo" sort of content.
There is no "running Wallaroo" yet, so for now, I'm moving this to the
appendix.

@SeanTAllen this has conflicts

@JONBRWN updated.


1 DAI for unlock

I will work on this

I will work on this

There's some more complication in this, in the sense that we want to figure out if 1 DAI is value or it's 1 DAI, at a basic level this is implemented as ether in the contract, so changing it to 1 ERC20 token is not valuable, so pick it up only if you want to work on the aspect of a router that works on a fixed value.

I think that for the sake of simplicity it should just be 0.001 ether or 1 DAI token.
And if needed, later down the road we can add support for any ERC 20 or a router.

@QEDK Can you sync with @ericstlouis to do a pair programming session to try to get this done for dogfooding ?

@ericstlouis The contract part of this is trivial, let's talk more about the frontend aspect of it?

@Dhaiwat10 Interested to work on this ?

@Cali93 yeah I can take care of this. Had one question: by 'suppot for ERC20 tokens', do you mean that now the fees will be transferred in the form of ERC20 tokens?

@Cali93 yeah I can take care of this. Had one question: by 'suppot for ERC20 tokens', do you mean that now the fees will be transferred in the form of ERC20 tokens?

Yes.

Sorry I couldn't finish this one guys...very busy these pass few weeks

@ericstlouis dont worry I've got you covered :))


Firmware./trustcache does not exist or is a directory

so i tried using this on ubuntu for my iphone 8 plus, and it just refuses to work

update ubuntu, it’s bc i couldnt figure out how to statically compile PlistBuddy


Bump ratis version to 2.5.1

@FLOW4215 The difference CPU usage is already verified by Ozone Community, see https://github.com/apache/ratis/pull/872, with a 100x drop in gRPC log appender.


Create API documentation

Nothing is documented anywhere, could be useful to document a basic API for the bot and Plugins

Something to remember for function overloading that should be documented in the API:
If a function uses the parser type ALL (meaning it grabs all arguments as a single string argument for the command function), it can't be overloaded. This is because there would be no way to distinguish the functions apart.