No module named awscli_plugin_endpoint

I've followed these directions to a t and I continue to get this error:
No module named awscli_plugin_endpoint
I'm running MacOS Mojave (up to date) and using the stock /usr/bin/python for aws (up to date).    No luck, no matter what I do.  Even if I manually add this info, it fails.
I do have MacPorts present, but I've made sure python is using /usr/bin/python.

Same here.
MBP-2015:.aws const$ aws --version
aws-cli/1.16.144 Python/2.7.10 Darwin/18.5.0 botocore/1.12.134

Same here.  I'm using the master branch.  I also did the pip install with the --user flag since I'm on shared hosting.  Might that be the reason?  I see the plugin is installed here:
./.local/lib/python2.6/site-packages/awscli_plugin_endpoint

Any advice?

OK, I figured it out.  Instead of doing pip install awscli-plugin-endpoint (which was using /usr/local/pip) I needed to do aws/bin/pip install awscli-plugin-endpoint.  That used the locally installed pip. I'm guessing that the others on this thread can find this solution useful as well.

Thanks @wbtdc
BTW, how do you install awscli?

I did virtualenv -p python2.7 py27That installs a virtual environment in the py27 subfolder of the current directory. Then I did source py27/bin/activate That switched me to the new py virtual env. Then I didpip install --index-url=https://pypi.python.org/simple/ awscliThat did it. I'm going from memory since I'm on my phone now, but I think that was it. I'm still getting SSL warnings that are caused by too old a version of python (this got me 2.7.8). Apparently I need 2.7.9 to get rid of the warnings and I haven't figured that part out yet. If anyone else can help, I'd appreciate any advice on that part. You can also do python3 but that only gets you to 3.2.3 and apparently your need 3.3 or greater. For some reason it's really hard to make the jump to those newer versions for some reason. I'm not a python guy, so this is all from googling.Good luck! DaveOn May 10, 2019 11:15 AM, Wenbing Li<EMAIL_ADDRESS>wrote:Thanks @wbtdc
BTW, how do you install awscli?
—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or mute the thread.

Has anyone dealt with this issue on windows Python3 ? I just upgraded from 3.6 to 3.8 and my endpoint plugin isn't working anymore, but I haven't really understood the fix mentioned by @wbtdc .
I use aws-shell ontop of awscli:
aws> configure
Traceback (most recent call last):
  File "C:\Program Files\Amazon\AWSCLI\bin\aws-script.py", line 7, in <module>
    sys.exit(main())
  File "C:\Program Files\Amazon\AWSCLI\runtime\lib\site-packages\awscli\clidriver.py", line 58, in main
    driver = create_clidriver()
  File "C:\Program Files\Amazon\AWSCLI\runtime\lib\site-packages\awscli\clidriver.py", line 68, in create_clidriver
    event_hooks=session.get_component('event_emitter'))
  File "C:\Program Files\Amazon\AWSCLI\runtime\lib\site-packages\awscli\plugin.py", line 44, in load_plugins
    modules = _import_plugins(plugin_mapping)
  File "C:\Program Files\Amazon\AWSCLI\runtime\lib\site-packages\awscli\plugin.py", line 58, in _import_plugins
    plugins.append(__import__(path))
ModuleNotFoundError: No module named 'awscli_plugin_endpoint'

Maybe I can just copy in the plugin to the sitepackages ? Any thoughts ?


Parser should also recognize MathML-Presentation annotation

When reviewing the Arvix results from Citeplag, it is noticeable that many formulas simply can not be read. At the moment only the following constellation is read:
<semantics>
  ... pmml
  <annotation-xml encoding="MathML-Content" ...>
    ... cmml
  </annotation>
</semantics>

VMEXT should also acknowledge the following constellation:
<semantics>
  ... cmml
  <annotation-xml encoding="MathML-Presentation" ...>
    ... pmml
  </annotation>
</semantics>

arg why do we reinvent the same functionality over and again.

please fix the tests

any news here?

The Problem was rendering and we didn't want to change this in mathoid. As a workaround, we could implement a mechanism that swapped the cmml and pmml in the printout, or directly control the output of pmml and cmml.
Would that be something for MathMLTools - CMMLInfo object?

I guess it's a problem deep in MathJax. Fixing that would require a lot of effort. We should only send the pmml portion to mathoid, this will also reduce the network usage so I think this approach would be superior.

@physikerwelt thanks for the changes


[Network] az network vnet create: add --private-endpoin-vnet-policies parameter

Related command

Description

https://msazure.visualstudio.com/One/_workitems/edit/28536618
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

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

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Network


No leaf 0 under cpuid function 11 on Ice Lake

When running pcm on a KVM-based vm, I get:
 Processor Counter Monitor  (2022-03-28 15:50:50 +0200 ID=78f12b99)

Detected a hypervisor/virtualization technology. Some metrics might not be available due to configuration or availability of virtual hardware features.

=====  Processor information  =====
Linux arch_perfmon flag  : yes
Hybrid processor         : no
IBRS and IBPB supported  : yes
STIBP supported          : yes
Spec arch caps supported : yes
Max CPUID level          : 27
ERROR: Major problem? No leaf 0 under cpuid function 11.
Access to Processor Counter Monitor has denied (no MSR or PCI CFG space access).

I used host passthrough for cpu, and I enabled and verified nested virtualization is enabled for this vm, running SLES 15 SP3.
lscpu gives:
CPU op-mode(s):                  32-bit, 64-bit
Byte Order:                      Little Endian
Address sizes:                   46 bits physical, 48 bits virtual
CPU(s):                          64
On-line CPU(s) list:             0-63
Thread(s) per core:              2
Core(s) per socket:              32
Socket(s):                       1
NUMA node(s):                    1
Vendor ID:                       GenuineIntel
CPU family:                      6
Model:                           106
Model name:                      Intel(R) Xeon(R) Platinum 8352Y CPU @ 2.20GHz
Stepping:                        6
CPU MHz:                         2199.998
BogoMIPS:                        4399.99
Virtualization:                  VT-x
Hypervisor vendor:               KVM
Virtualization type:             full
L1d cache:                       1 MiB
L1i cache:                       1 MiB
L2 cache:                        128 MiB
NUMA node0 CPU(s):               0-63
Vulnerability Itlb multihit:     Not affected
Vulnerability L1tf:              Not affected
Vulnerability Mds:               Not affected
Vulnerability Meltdown:          Not affected
Vulnerability Spec store bypass: Mitigation; Speculative Store Bypass disabled via prctl and seccomp
Vulnerability Spectre v1:        Mitigation; usercopy/swapgs barriers and __user pointer sanitization
Vulnerability Spectre v2:        Mitigation; Enhanced IBRS, IBPB conditional, RSB filling
Vulnerability Srbds:             Not affected
Vulnerability Tsx async abort:   Mitigation; Clear CPU buffers; SMT Host state unknown
Flags:                           fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ss ht syscall nx pdpe1gb rdtscp lm constant_tsc arch_perfmon rep_good nop
                                 l cpuid tsc_known_freq pni pclmulqdq vmx ssse3 fma cx16 pdcm pcid sse4_1 sse4_2 x2apic movbe popcnt tsc_deadline_timer aes xsave avx f16c rdrand hypervisor lahf_lm abm 3dn
                                 owprefetch cpuid_fault invpcid_single ssbd ibrs ibpb stibp ibrs_enhanced tpr_shadow vnmi flexpriority ept vpid ept_ad fsgsbase tsc_adjust bmi1 hle avx2 smep bmi2 erms invp
                                 cid rtm avx512f avx512dq rdseed adx smap avx512ifma clflushopt clwb avx512cd sha_ni avx512bw avx512vl xsaveopt xsavec xgetbv1 xsaves wbnoinvd avx512vbmi umip pku ospke avx
                                 512_vbmi2 gfni vaes vpclmulqdq avx512_vnni avx512_bitalg avx512_vpopcntdq la57 rdpid fsrm md_clear arch_capabilities

I've also attached cpuid_tool data.
Is there a way to get pcm working in this vm
cpuid_tool output.txt
?

thanks for sending the detailed issue report

Updating to later QEMU ensured that xtopology cpuid feature was also exposed, now PCM is working.

thanks for the update. Good to know that the QEMU update resolved the issue


add red lang

If there’s a checkbox you can’t complete for any reason, that's okay, just explain in detail why you weren’t able to do so.
After making all changes to the cask:

[X] brew cask audit --download {{cask_file}} is error-free.
[X] brew cask style --fix {{cask_file}} reports no offenses.
[X] The commit message includes the cask’s name and version.

Additionally, if adding a new cask:

[X] Named the cask according to the token reference.
[X] brew cask install {{cask_file}} worked successfully.
[X] brew cask uninstall {{cask_file}} worked successfully.
[X] Checked there are no open pull requests for the same cask.
[X] Checked that the cask was not already refused in closed issues.

This seems to be a CLI-only open-source tool. Please submit it first to Homebrew and only if they don’t accept it, submit here (linking back to the pull request where it was refused).
This policy is documented in faq/rejected_casks:

The app is both open-source and CLI-only (i.e. it only uses the binary artifact). In that case, and in the spirit of deduplication, submit it first to Homebrew. If it is rejected there, you may then try again in Homebrew-Cask (link us to the issue on Homebrew so we can see their reasoning for rejection).

Homebrew did not accept it because it is a binary and they only accept source based install. That's why we sent the pull request here.

I’ve made several fixes to the cask. Merging after Travis passes.

Thanks for making simpler the cask!

I still think this belongs as a formula, and HB hasn’t yet refused this. As such, this is on hold.

Thanks for your support!

By the way, do you mean that you are waiting for the final decision of HB? It is clear in their answer that they do not want to accept the PR except if you manage to convince them.

I’d say they’re waiting for your reply. Otherwise the PR would be closed.

Ok! They expect an installation from source but it is just impossible. I told them that you thought this PR as a formula better than a cask.

I still think this belongs as a formula, and HB hasn’t yet refused this.

I agree. For what it's worth we're unlikely to "refuse" it as much as we need it to be built from source. I think it could be (someone is building this from source at some stage, it's not just appearing like mana from heaven...) so it shouldn't go into Homebrew Cask. Up to you, though.

Do you know that red does not rely on C but on Rebol which is not an open source project. The goal of red among other goals is to become an open source version of Rebol. As I already told you, one day it would be feasible when would depend only on red itself but it not the case now.

@rcqls That's fine. We could have the formula download Rebol and build it from source.

Sorry! You do not remember my previous comments. With only Rebol it is not possible. You need Rebol SDK  and this is currently no more available on the Rebol Website you only can download Rebol but not Rebol SDK even if we accept to pay it. As you understand, the author of red has a license for that and provide red as a binary. Soon I hope, red could compile its source.  Do you understand me?

@vitorgalvao What is your point? I am pretty sure it is impossible to compile red from source because it depends on product that you cannot distribute for free. However, even if this is possible, I still not understand why you could not share it as a cask above all when it is clearly easier to share it using binary. Why do you share R as a cask ? You can install it from source. Certainly because it is easier to share it as a binary using the wonderful job made by the R team. The author of red creates a language that is very easy to share because like lua it is very lightweight. His intention is to share it in the Homebrew world because he thinks HB is a very good tool and also HB users can be interested in red.
It was not my intention to do these PR. I did it because the author is really busy and to send the PR you need a MacOSX. At the beginning, I told him that HB was not the right tool since it is so easy to install red as an executable. However, HB allows easy installation completing download and installing it at the right place. HB maybe can also help by promoting good tools...

I am pretty sure it is impossible to compile red from source because it depends on product that you cannot distribute for free.

Ok; it doesn't sounds particularly open-source to me if you need paid software to produce a binary for it. Like, I can view the source and submit changes but I can't really do anything with it.

Why do you share R as a cask ?

It provides a GUI.

@MikeMcQuaid I am so sorry to be definitely not clear and I apologize for that.  However, I did not know it was so difficult to share a so simple tools wit HB or HB Cask. That's why I was not so clear. So to clarify:

Red as an open source project want to replace rebol since rebol is not an open source project. In some sense, it was the origin of the project: become an open source version of Rebol2, like R was a replacement of S/Splus (not open source project).
As you can guess, Red shares a lot of syntax with its ancestor Rebol that's why the compiler was first written in Rebol (no need to reinvent the wheel). When Red is ready, the compiler will be written directly in Red and will be a self-contained open source project with a beautiful formula in HB. That's the goal of the author of Red. The difference with R is that is both an interpreted and compiled language.
As an open project, with the Rebol free binary (not the paid Rebol SDK not anymore provided), you can compile  and cross-compile any source Red file to create binary executable.
However, it is much  easier to compile Red source file with the red binary (red-061) that the author want to share here with Homebrew PR and Homebrew Cask. And like I mentioned before, he asks me to do it because it has a deep respect of what you did even if for job consideration he has to work mostly on Windows. The ONLY point is that from now, the ONLY way to provide this software is to use the paid Rebol SDK. The license of the Rebol SDK, allows to create this executable (red-061 and soon red-062) that you can distribute for free (like Rebol) and it is a good news. It then simplifies a lot the Red world since users do not have to  download Rebol to create executables from Red source files. To use Red as a REPL, you can use red-061.  By the way, Rebol was only an interpreted language. Fortunately, since Red team works hard they can provide REPL based on GUI stuff but on windows at first. MacGUI is coming soon. These REPL based on GUI can be compiled with Rebol2. But do not confuse, you still can't compile Red source with these REPL based on GUI (in the Rebol2/View spirit). You really need red-061 now.

Now, @MikeMcQuaid , let me know if you still think that Red is not an open source project.
By the way, I am only interested in open-source project (R, ruby, julia, crystal, nodejs,...). I was really interested longtime ago with Rebol but I clearly was disappointed that this language (like mathematica for example) was not an open source project and I just gave up with a lot of regret because it was really impressive. Now, thanks to Red's author I can appreciate this (powerful) language since its primarily goal was to be an open source replacement of Rebol (actually Rebol2) from the beginning. Of course there is another open source projects with the same goal (like Rebol3) but Red is very active and its roadmap is really impressive.
About the R GUI,  you mean that you can't compile it from source. So how do you create the GUI? Is it depending on some paid software and you can't create a Formula? Do you mean that R is not an open source project. Even RStudio can be compiled from source but it is for sure time consuming to recompile it.
Many thanks for taking time and paying attention on the Red open source project.

Why do you share R as a cask ? You can install it from source. Certainly because it is easier to share it as a binary using the wonderful job made by the R team.

About the R GUI, you mean that you can't compile it from source. So how do you create the GUI? Is it depending on some paid software and you can't create a Formula? Do you mean that R is not an open source project. Even RStudio can be compiled from source but it is for sure time consuming to recompile it.

There’re some assumptions there and a misunderstanding of the places of both HB and HBC, and how we work together. Let’s not go there, as it is mostly irrelevant to the issue at hand. You can check https://github.com/caskroom/homebrew-cask/issues/15603 if you so wish for most of the context and evolution of these policies.

Many thanks for taking time and paying attention on the Red open source project.

I will also add that I thank you for your respect in this whole conversation. Many users in your position would’ve (unfortunately) already thrown a tantrum and started calling us names and insulting our methods, when we work this way for various reasons that evolved as both projects did.
@MikeMcQuaid I’m fine with whatever you prefer, here. We can either:

Refuse this as a cask and work on the formula starting now.
Accept it now as a cask, refuse it as a formula, and when red has matured to be able to compile itself, remove the cask and make it a formula.

As I already told you before, thanks @vitorgalvao for your supports. I really appreciate the hard work from HB and now from HB Caskroom. Thanks for the link, I 'll read it carefully.  BTW, I can't see the point to fight in the context of open-source projects where people spend a lot of time for sharing a lot of tools and most of the time for free.

Refuse this as a cask and work on the formula starting now.
Accept it now as a cask, refuse it as a formula, and when red has matured to be able to compile itself, remove the cask and make it a formula.

Either seems fine to me, I don't have a strong preference. I personally would probably pass on Red for both until it can self-bootstrap but I don't have strong feelings.

Thanks @MikeMcQuaid too! I am waiting your final decision and inform then Red team.

I;m leaning towards putting it in HBC only for now. After it becomes self-compilable, we can remove it from HBC, add it to HB, and add it to migrations.json to facilitate a smooth transition.

@vitorgalvao, @adidalal  After reading the link about deduplication, I fully understand your point now. Since brew search r-app (for example) redirects to the cask, I think that brew search red would redirect too and this is a good news.

Seems reasonable to me too then 👍

@rcqls how can I get started using Red/view and the rest of the GUI tools?

@tekknolagi if you want to get started using Red/View you have to wait for the 0.6.3 version of red coming within a couple of months. The last release 0.6.2 contains only the Windows GUI.  There is also a MacOS branch in the red repository with Red/View. Very soon, it will be integrated in the master branch. If you want more accurate information go to the red website or visit Gitter red rooms which are very good  to ask question to the Red team and many advanced users.


fix: side tabs resize

Fixes side tab resizing
Changes proposed in this pull request:

Bind missing action changeWidthDocs so that the tabs can resize

This issue was originally reported in https://github.com/apollographql/apollo-server/issues/2529 and was fixed in https://github.com/apollographql/graphql-playground/pull/19.
The regression looks like it was introduced in https://github.com/prisma/graphql-playground/pull/1001.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Is there any progress on this one.  Could we please try to get this merged?

@spencersteers do you want to update this for the failing typescript check?
or does someone else want to re-create this PR and give him credit in the commit with a Co-Authored-By in the footer?
otherwise, I can update this PR myself tomorrow

@acao Types are hopefully fixed..
I haven't used the playground in about 6 months but I noticed that only the schema tab is resizable. I tracked down where I think the problem is: https://github.com/spencersteers/graphql-playground/blob/fix-side-tabs-resize/packages/graphql-playground-react/src/components/Playground/DocExplorer/GraphDocs.tsx#L122 - seems to be using it's own logic for width.
If I still used the playground I would feel more comfortable with fixing it but It has been a long time and I won't be using playground anytime soon. Best of luck.

can this be merged ?

@seansnyder we will get to this once there is a working fix, or I have time to work on fixing this issue. this PR no longer fixes the issue with resizing the docs sidebar, according to the latest netlify deploy above

I don't have time to work on this bug unfortunately. Does someone want to update this PR or provide a new PR that fixes the issue?


Website Information Bug

Do you want to request a feature or report a bug?
Report a bug
What is the current behavior?
Text gets covered on squished screen size
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem.
Open a chrome browser and make the window height about 1/2 to 2/3 of the screens height

What is the expected behavior?
The text below "Getting Data Off Blockchains Is Hard" will be covered up

@nenadjaja Could you look into this?

Yep, I'll check it out


select multiple close on ipad

if you open a multiple select on an ipad, you can select multiple options, but it is not possible to close it unless you open another select or set focus on an input field

Hi,
I don't have issue on my iPad, can you itemize your environnement ?

I just went to http://materializecss.com and tried the different selects on ipad. The "normal" ones I can close by tapping on a disabled option, but the multiple select stays open.
Also if you just tap outside the selects, the selects stay open.
This is on an ipad retina with ios 9.1
On android, they close if you tap outside the selects.

Yes, I've reproduced this on my iPad, I'm thinking we need to add some event listener for tap or look for some way to support touch device defocuses.

This issue is still happening, it works fine on Android but iOS will not tap off.

The tap out is only reacting to click events which doesn't seem to be fired on iPad/iPhone.
This seems to work if you add a touchstart event handler or "redirect" touchstart to click (that is, fire a click event on touchstart events).

I have the same issue on iOS. Do you have any update about it ?

There's no proper fix but you can add the class 'browser default' to the input to get it working.
This is explained about 1/3 of the way down this page: http://materializecss.com/forms.html

Can anyone check if this issue is still present inv0.99.0?

Yes, this issue is still present in v0.99.0.

Hey guys this code should fix the problem with multiple select and closing it on iOS devices.
 $(document).on('touchstart', function (e) {
        let target = $(e.target);
        if (!target.is("span")) { //checking if you are tapping on items in multiple select or not
            $(document).trigger('click'); //if you are tapping outside multiple select close it
            $(':focus').blur(); //focus out for multiple select so you can choose another or the same (in case you have more multiple selects on page)
        } else {
            //nothing or fire another event
        };
    });

This works for me.

Fixed in v1-dev


Fix building of geowave_pyspark package on jenkins.

Coverage decreased (-2.9%) to 45.959% when pulling 4f97b408f0504bdb38c9f6d49222dfa0108098a8 on JWileczek:centos-fix into df44a0ebd4dad06b5f6a09aa31c81176b5804fef on locationtech:master.


Revert "Update DLP snippet as canonical for new sample format. (#1232)"

This reverts commit 6bb16d1d5566fc2d2d19e727f63bd3b5e5f53718.
This commit broke our API samples. It is fine to update to a new template, but please do so for the entire set of functionality, and create a new PR with the old samples removed.

@googlebot I fixed it.

Argh - Java 8 is broken by this.

I'm inclined to accept #1795 once it's fixed than this one -- @kurtisvg ping'd me re: this and didn't think that the problem was the new format.

I'm going to close this as we think the main problem has been resolved.


PR #18 breaks requests with joins

First of all thanks for this fork! It fixes a number of crucial issues
#18 fixes some mysql issues, but introduces an issue for requests with manyToOne and manyToMany joins. Because of that we need to stick with 5.1.1 version
Can it be reverted? Or maybe some conditional logic can be added to decide wether skip or offset should be used?
cc: @Darkein

reverted in 5.1.9

so sad, it's my turn to be stuck with 5.1.8 :(


[LLM Content] Add support for webcam video input

We need to support uploading akin to https://ai.google.dev/gemini-api/docs/prompting_with_media?lang=node#upload-image-file


Minor Bug : Can't use folders with same initial characters as the base path.

I have this folder structure in my app:
wwwroot/app/apps/vue-app-one
And kept getting this error:
<script type="module" vite-src="~/apps/vue-app-one/main.ts" asp-append-version="true"></script>

Vite.AspNetCore.TagHelpers.ViteTagHelper ❌ 's/vue-app-one/main.ts' was not found in Vite manifest file

I can change that but I was getting this error. It seems that since the subfolder has the same first three characters as the base path 'app' they get removed. Workarounds seem to be including the base path in the vite-src for those paths or changing the base path or subfolder so they don't overlap.

Hi, if your files will be in a subfolder, you need to configure the base path.
Check the wiki for more information.


[T5A1][F10-B2] Tan Shao Yun

Yay

Ready for review

Reviewed during tutorial.

@shaocloud :  Accept the PR since it is reviewed at tutorial. Proceed to close the PR.


How to deploy in production without impacting performance ?

Hello,
I would like to add the package in production on Heroku, and I'm afraid that it will impact the performance of the application, what is the best way to run it in production without impacting the performance?
Thanks

It shouldn't necessarily impact the performance of your application. As long as you have enough memory in your dyno, your application performance should stay same.
Just add another line in your procfile like this
release: node build/ace migration:run --force
web: node build/server.js
+ worker: node build/ace queue:listen

Hey @Glenn-o! 👋🏻
As @brkn said, there is nothing specific that should impact performance.
Did you have any issues?


Doc generation breaks (RTD)

typer-cli which we use to generate the CLI doc is currently incompatible with typer>=0.9.0 : see https://github.com/tiangolo/typer-cli/issues/50

(cc @yganodot)


Dev Research: Document Management/Storage

Describe the task
Analysing and researching document management services.
Acceptance Criteria

[ ] Create List of options
[ ] Analyse Viability, Pricing, etc...

Additional context
Check and see what solutions have been implemented in gov previously.

Hey team! Please add your planning poker estimate with Zenhub @mariaye-aot @midhun-aot @mishraomp @nikhila-aot @RyanBirtch-aot @sakthi-aot


fix: MultiCompiler support

Fixes #73

path.resolve()?

path.resolve() ?

For compiler.context you mean ? If can you post what the status quo is there atm please? I wonder why it isn't absolute and if this has a reason

@michael-ciniawsky yes, path.resolve & process.cwd() both seem to be used for this in webpack core: https://github.com/webpack/webpack/search?utf8=✓&q="new+RequestShortener("&type=


The ApplicationListeners (e.g. DelayHandler) aren't registered in the ApplicationContext

The DSL components, e.g. DelayHandler:
@Bean
public IntegrationFlow delayFlow() {
	return flow ->
	flow.delay(GROUP_ID)
	.channel(c -> c.queue("results"));
}

Are registered as beans in the ApplicationContext from the IntegrationFlowBeanPostProcessor, but AbstractApplicationContext.registerListeners() is definitely called before any BPP.
We should call this.applicationContext.addApplicationListener(applicationListener); manually from the IntegrationFlowBeanPostProcessor to reach the normal ApplicationListener behavior.

The external discussion: http://stackoverflow.com/questions/35802308/onapplicationevent-is-never-invoked-on-delayhandler


feat(terraform): integrate modular terraform

experimental-setup.sh will deploy resources to the ops, staging, and prod projects and optionally run the configure_auth script.
Deploying triggers is disabled until we integrate them properly into the emblem-app module.

We're going to be aggressive in merging this and working to fix up anything that creates a challenge in follow-up.
Per discussion, Cloud Build triggers are partially implemented a two ways, and neither of them are quite what we want. Correcting that will be in a follow-up. I will file issues later clarifying those actions, which includes rehoming ownership of triggers to app environments.


Mi 8 not work!

Hi, @liuchuancong. This is an OS-level problem with MI UI. Please check this project's readme notes for information on the same.
I'm currently working on an alternative method to fix this issue. Hopefully, if it works out, I'll drop you a message here.


Meta issue for new drag drop functionality

There seems to be many issues with regards to development of new drag'n drop functionality.
So to coordinate those efforts I was thinking that we could create a issue to gather and discuss all of them, it seems that making all of them happen needs a set of new drag'n drop parameters and functionality:

Drag to something outside the tree: #370, #371, #373
Expand node on hovering drop over: #372
Dragging without moving the node (read-only): #375
Ctrl-dragging, i.e. copying: #375

These things seems to be most of the things remaining for this component to be able to mimic all drag'n drop behaviours available in a desktop operation.

Closing this issue due to inactivity. See #815 for more information.
Please create a new issue is this still occurs with the current version of the component.


Move Epsagon to Tracing

Pre-submission checklist:
Please check each of these after submitting your pull request:

[x] Are you only including a repo_url if your project is 100% open source? If so, you need to pick the single best GitHub repository for your project, not a GitHub organization.
[x] Is your project closed source or, if it is open source, does your project have at least 300 GitHub stars?
[x] Have you picked the single best (existing) category for your project?
[x] Does it follow the other guidelines from the new entries section?
[x] Have you included a URL for your SVG or added it to hosted_logos and referenced it there?
[x] Does your logo clearly state the name of the project/product and follow the other logo guidelines?
[x] Does your project/product name match the text on the logo?
[x] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
[x] ~15 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

Deploy preview for landscape ready!
Built with commit 95ac1d519dc713fee4bab7dcce4d05f9fa38192a
https://deploy-preview-1888--landscape.netlify.app

LGTM
@caniszczyk

Hi @taylorwaggoner - why did you move Epsagon from monitoring to tracing? It's not the right category..

@ranrib Chinmay Gaikwad asked me to move Epsagon to the Tracing category on November 4. If this is incorrect, we can move it back to Monitoring, please let me know.

Hi @taylorwaggoner sorry for that we had a small confusion here. We are using tracing, alongside with metrics and logs, but providing an overall application monitoring solution, so I think it best to move us back to monitoring. I created a PR so I will appreciate if you can approve it please.
https://github.com/cncf/landscape/pull/1942

Hi @taylorwaggoner sorry for that we had a small confusion here. We are using tracing, alongside with metrics and logs, but providing an overall application monitoring solution, so I think it best to move us back to monitoring. I created a PR so I will appreciate if you can approve it please.
https://github.com/cncf/landscape/pull/1942


Fix/benchmark breakage

Current coverage is 97.13% (diff: 100%)

Merging #49 into master will not change coverage

@@             master        #49   diff @@
==========================================
  Files            12         12
  Lines           663        663
  Methods         583        583
  Messages          0          0
  Branches         80         80
==========================================
  Hits            644        644
  Misses           19         19
  Partials          0          0

Powered by Codecov. Last update 20b5292...92b1184


Openshift installation

Installation of sharp on openshift fails because user does not have permission to access the dev dir.

<EMAIL_ADDRESS>install /var/lib/openshift/58a620a67628e1625b00001f/app-root/runtime/repo/node_modules/sharp
node-gyp rebuild
gyp WARN EACCES user "58a620a67628e1625b00001f" does not have permission to access the dev dir "/var/lib/openshift/58a620a67628e1625b00001f/.node-gyp/7.5.0"
gyp WARN EACCES attempting to reinstall using temporary dev dir "/tmp/.node-gyp"
gyp: /tmp/.node-gyp/7.5.0/common.gypi not found (cwd: /var/lib/openshift/58a620a67628e1625b00001f/app-root/runtime/repo/node_modules/sharp) while reading includes of binding.gyp while trying to load binding.gyp
gyp ERR! configure error
gyp ERR! stack Error: gyp failed with exit code: 1
gyp ERR! stack     at ChildProcess.onCpExit (/var/lib/openshift/58a620a67628e1625b00001f/app-root/data/.nodejs/lib/node_modules/npm/node_modules/node-gyp/lib/configure.js:308:16)
gyp ERR! stack     at emitTwo (events.js:106:13)
gyp ERR! stack     at ChildProcess.emit (events.js:192:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:215:12)
gyp ERR! System Linux 2.6.32-642.13.1.el6.x86_64
gyp ERR! command "/var/lib/openshift/58a620a67628e1625b00001f/app-root/data/.nodejs/bin/node" "/var/lib/openshift/58a620a67628e1625b00001f/app-root/data/.nodejs/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /var/lib/openshift/58a620a67628e1625b00001f/app-root/runtime/repo/node_modules/sharp
gyp ERR! node -v v7.5.0
gyp ERR! node-gyp -v v3.5.0
gyp ERR! not ok
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/ursa):
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>install: node-gyp rebuild
npm WARN optional SKIPPING OPTIONAL DEPENDENCY: Exit status 1
npm ERR! Linux 2.6.32-642.13.1.el6.x86_64
npm ERR! argv "/var/lib/openshift/58a620a67628e1625b00001f/app-root/data/.nodejs/bin/node" "/var/lib/openshift/58a620a67628e1625b00001f/app-root/data/.nodejs/bin/npm" "install" "sharp"
npm ERR! node v7.5.0
npm ERR! npm  v4.2.0
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: node-gyp rebuild
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.

This looks like an Openshift permissions problem - see also https://github.com/nodejs/node-gyp/issues/959
Sometimes npm install --unsafe-perm "fixes" these sort of things.

Just tried that with no luck ...

@jchapelle Were you able to make any progress?

Hello,
Thanks for the contact.
I actually couldn't make any progress.
The application is deployed in openshift but I still got a list of errors
during each deployment
On Mon, Mar 27, 2017 at 9:38 PM, Lovell Fuller<EMAIL_ADDRESS>wrote:

@jchapelle https://github.com/jchapelle Were you able to make any
progress?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/lovell/sharp/issues/721#issuecomment-289561978, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AF4GXasL_IwdyMiQ6-EgwTpcU_48epRgks5rqBAngaJpZM4MJ4S1
.

--
Jean Chapelle

@jchapelle Thanks for the update. Let's leave this open in case others with more experience of Openshift can help.

Great thanks
On Mon, Mar 27, 2017 at 11:37 PM, Lovell Fuller<EMAIL_ADDRESS>wrote:

@jchapelle https://github.com/jchapelle Thanks for the update. Let's
leave this open in case others with more experience of Openshift can help.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/lovell/sharp/issues/721#issuecomment-289593987, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AF4GXbwx5FT2FzCEuNITp3JItuZzWU3Dks5rqCw2gaJpZM4MJ4S1
.

--
Jean Chapelle

Possibly related sources of help:
http://stackoverflow.com/a/26150783/4625124
https://github.com/nodejs/node-gyp/issues/21

I'm going to close this as there have been no further reports of what appears to be a permissions problem specific to Openshift and/or node-gyp.


Wrong hash for obs-studio-small 21.1.2

https://github.com/lukesampson/scoop-extras/commit/7e46e9ef0b04fd5c35be40192d9ec2ac8fa45f46
Checking hash of OBS-Studio-21.1.2-Small.zip... Hash check failed for 'https://github.com/obsproject/obs-studio/releases/download/21.1.2/OBS-Studio-21.1.2-Small.zip'.
Expected:
    c18921d7b19deb14247a86dfa4db8fcb7323fdc98c69e3c9d4e379d2ab3e859f
Actual:
    5c2b106d4af24aee91c3541743328aea3ecca0df0ae8fb4d94f65877cdd371ae

Fixed


Cryo Not Freezing People

Description

A gas analyzer shows freezing temps in the cryochamber but players are unaffected by it.

What? Cryo exists only for safety down crews temperature without cold damage. Only with a cold body you can use cryoxadone and other chems, which requires cold for some result.

Do you mean that the player’s temperature isn’t being lowered properly, or they aren’t taking damage from the cold? If it’s the latter, that’s intentional behavior.

iirc they should still take cold damage, but the cryoxadone outheals it

no like the pod hooked up to a freezer does not cool the subject enough to use cryox

Do you mean that the player’s temperature isn’t being lowered properly, or they aren’t taking damage from the cold? If it’s the latter, that’s intentional behavior.

Player temperature is not being lowered properly, so the former. We put in a naked moth for a few minutes and they never dropped below 0C.

womm?
getting cold + high pressure since its not being pumped just the canister itself

that strange... could it be because of the higher pressure it was able to cool you?

are you sure the subject did not wear any temperature resist clothing like hardsuit or space suit? because putting them in cyro won't cool them down

I jumped in naked, I’m not the only med personnel that knows cryo just doesn’t work right now

this issue is fake
73K air in the pod, urist very slowly freezes

gave urist some razorium so he is hurt

after giving cryo urist starts to heal

cryo pods still work (but they dont take any damage from cold???) its just its very slow and probably needs heat transfer speed buffed

Ok, well clearly it needs a fix, thanks for testing it

iirc they should still take cold damage, but the cryoxadone outheals it

things inside the pod gets their coldDamageThreshold set to 10 kelvin, which is why leporazine is given to people when they get out of  the pod

Fixed by #27148
See also #22958


Upgrade test suite to new glTF Validator

There were some breaking changes to the list of things reported by the validator.
All roundtrip models have new validation reports.  The 03_skinned_cylinder model was re-exported from Blender to pick up on skinning bug fixes (it was failing validation prior to roundtrip).
Also update Mocha & ESLint.

See #875 instead.


UserConverter NPE when User.authorizations() returns null

Version
4.3.7
Context
UserImpl has a default constructor which does not initialize its authorizations field. The UserConverter class expects User.authorizations() to return a non-null value. If an instance of UserImpl is constructed using the default constructor and then later on serialized (since it implements ClusterSerializable) it will result in a NPE when it delegates serialization to UserConverter because UserConverter does not perform a null check.
I encountered this with the Pac4jUser class from org.pac4j:vertx-pac4j which extends UserImpl and implements the default constructor which leaves the authorizations field null. The VertxProfileManager from org.pac4j:vertx-pac4j uses the Pac4jUser default constructor.
Seems like UserConverter should check if User.authorization() returns null before attempting to serialize it.
Do you have a reproducer?
UserConverter.encode(new UserImpl());
Steps to reproduce

Run UserConverter.encode(new UserImpl()); in a unit test

Extra

I'll make a PR for this.

@pmlopes I have a PR ready to go but do not have access to push my branch to this repo. I signed the ECA and my Eclipse username is jenglert4r8. Can I get access to contribute?

See PR https://github.com/eclipse-vertx/vertx-auth/pull/639

@pmlopes is it possible to get this in a 4.x release?

Fixed in #653


Return user-defined errors in create variable CA

Purpose
Fixes #42127

The same error occurs for the change return type CA as well: https://github.com/ballerina-platform/ballerina-lang/issues/42292. We may use the same methods for this CA and perhaps complete it in a different PR.

Let say the below code is in a separate module.
public type Error1 distinct error;
type Error2 distinct error;
public type NonPublicError int|Error2;

public function getValueOrError4() returns NonPublicError|Error1 => 1;

And we have another ballerina file like below,
import package_test.mod1;

public function main() {
    <cursor>mod1:getValueOrError4();
}

When we apply the code action it gives an error. This should be negative test case where we shouldn't suggest the code action.

Let say the below code is in a separate module.
public type Error1 distinct error;
type Error2 distinct error;
type NonPublicError int|Error2;

public function getValueOrError4() returns NonPublicError|Error1 => 1;

And we have another ballerina file like below,
import package_test.mod1;

public function main() {
    <cursor>mod1:getValueOrError4();
}

When we apply the code action it gives an error. This should be negative test case where we shouldn't suggest the code action.

Moved this to a separate issue #42340 because this PR is addressing issues related to "create variable with check" code action.


update netty-handler for CVE

Codecov Report

Merging #1335 (9ee217d) into master (455515e) will decrease coverage by 0.09%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1335      +/-   ##
==========================================
- Coverage   82.61%   82.51%   -0.10%
==========================================
  Files         129      129
  Lines        4044     4044
  Branches      129      129
==========================================
- Hits         3341     3337       -4
- Misses        703      707       +4

Impacted Files
Coverage Δ

...om/ing/bakery/components/InteractionRegistry.scala
53.33% <0.00%> (-3.34%)
:arrow_down:

...e/akka/actor/process_index/ProcessIndexProto.scala
98.11% <0.00%> (-1.42%)
:arrow_down:

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


test: remove LOG and LOGF variadic macros

Initial patch by @simar7, thanks!

R=@bnoordhuis
This supesedes #185, hopefully I addressed all issues pointed out there.

Sweet, thanks @saghul

Rubber-stamp LGTM.  Can I suggest you check the CI before merging?

I did, but then added a couple of more fixes. I'll run it again before
merging.
Thanks!
On Apr 10, 2015 5:43 PM, "Ben Noordhuis"<EMAIL_ADDRESS>wrote:

Rubber-stamp LGTM. Can I suggest you check the CI before merging?
—
Reply to this email directly or view it on GitHub
https://github.com/libuv/libuv/pull/313#issuecomment-91597212.

CI run: https://jenkins-iojs.nodesource.com/view/libuv/job/libuv+any-pr+multi/83/


storage rewards question

I spent some bzz when I purchased the postage stamp. where did the bzz end up. How the node storage rewards are issued and how they are claimed.

I read the PostageStamp contract code and there is no bzz transferable function

So for questions, you can join our discord


Cake on .Net core

According to the documentation, Cake can run on .Net core. However, when running the build.sh package for linux, it requires mono.
How can I run a cake script without relying on mono on Linux?

@trondhindenes You will need to install the Cake.CoreClr package instead of the Cake. We currently do not have any bootstrappers for Cake.CoreClr that doesn't use NuGet to fetch packages so you will need to download the packages by some other means.
I think @devlead have written scripts for this.

Here's an example bootstrapper for running only on .NET Core
https://github.com/devlead/BitbucketPipelinesShield/blob/master/build.sh

Thanks!
I'm getting this: The specified framework 'Microsoft.NETCore.App', version '1.0.4' was not found, I guess it's caused by outdated references in the Cake.CoreCLR. Nuget.org doesnt list any dependencies for the package, so it's hard to see if the newest versions are updated either. It would be great if you could publish more metadata to nuget.

Which version of the .NET CLI do you have installed? Package has no dependencies as everything but the .NET CLI is in the package.

The script which the bootstrap installer downloads is not compatible with my OS (Ubuntu 17)  it seems, so I commented it out - hoping that I could use one of the installed .net core versions already on my computer instead - which didn't work. But no worries, I can install some older CoreFX runtimes/SDKs tomorrow and try again then. Will report back. I guess the script could be a made to check if a framework download is needed before it goes ahead and downloads it, but no biggie. I'll report back once I've looked more at this in the morning.

The example supplied script downloads 0.19.3,  latest version of Cake is 0.21.1 so you could try getting latest version if you haven't already.
Which version of the .NET CLI & SDK do you have installed? Cake currently compiled with .NET CLI 1.1.0 and SDK 1.0.4.

The specified framework 'Microsoft.NETCore.App', version '1.0.4' was not found.
  - Check application dependencies and target a framework version installed at:
      /usr/share/dotnet/shared/Microsoft.NETCore.App
  - The following versions are installed:
      1.1.2
      1.1.1

and
~/Documents/projects/cakecorefx  dotnet --version
1.0.4

I guess I'm mising the 1.1.0 .Net cli. Version numbers in .Net confuse me.

All I get so far after hours of struggling is "Failed to initialize CoreCLR, HRESULT: 0x80131500"
I'm on a Docker image with Debian Stretch (microsoft/aspnetcore-build:1.0-2.0`)
dotnet --version is working (version 2.0.0)
mono --version also (Mono JIT compiler version 4.6.2 (Debian <IP_ADDRESS>+dfsg-1)
I'm using a modified build.sh script from @devlead (removed core cli install since it's in the docker image already).
I guess there is no other way than to compile Cake with CLI and SDK 2.0?

I'm looking into this, to see what it takes to run Cake on .NET Core 2.0

An easier solution to this might be to create a "self-contained" deployment for Cake.CoreClr (https://docs.microsoft.com/en-us/dotnet/core/deploying/#self-contained-deployments-scd)
I've spent all morning trying to get Cake.CoreClr running in an environment where only the 2.0 SDK is installed (installing 1.0.4 locally for example) but just gave up. A self contained install might solve this.

@elevate-andrewlock To do an OS specific publish and you get a cake binary that's framework independent you'll do something like
dotnet restore .\Cake\Cake.csproj -r ubuntu.16.04-x64
dotnet build .\Cake\Cake.csproj -r ubuntu.16.04-x64 -f netcoreapp1.0
dotnet publish .\Cake\Cake.csproj -r ubuntu.16.04-x64 -f netcoreapp1.0

which will give you an src/Cake/bin/Debug/netcoreapp1.0/ubuntu.16.04-x64/publish folder and there you'll find an native Cake binary with no need for "dotnet" core and should work and can be tested with
./Cake --version
One of the issues with this is that the result will be OS specific and potentially we would need to publish/package a version for each os, which might be where we end up int the feature, looking at the RID docs there would be a few to support (this is subject to change though)
Windows RIDs

Windows 7 / Windows Server 2008 R2

win7-x64
win7-x86

Windows 8 / Windows Server 2012

win8-x64
win8-x86
win8-arm

Windows 8.1 / Windows Server 2012 R2

win81-x64
win81-x86
win81-arm

Windows 10 / Windows Server 2016

win10-x64
win10-x86
win10-arm
win10-arm64

Linux RIDs

Red Hat Enterprise Linux

rhel.7-x64

Ubuntu

ubuntu.14.04-x64
ubuntu.14.10-x64
ubuntu.15.04-x64
ubuntu.15.10-x64
ubuntu.16.04-x64
ubuntu.16.10-x64

CentOS

centos.7-x64

Debian

debian.8-x64

Fedora

fedora.23-x64
fedora.24-x64

OpenSUSE

opensuse.13.2-x64
opensuse.42.1-x64

Oracle Linux

ol.7-x64
ol.7.0-x64
ol.7.1-x64
ol.7.2-x64

Currently supported Ubuntu derivatives

linuxmint.17-x64
linuxmint.17.1-x64
linuxmint.17.2-x64
linuxmint.17.3-x64
linuxmint.18-x64

OS X RIDs

osx.10.10-x64
osx.10.11-x64
osx.10.12-x64

@devlead I believe it's also possible to publish all RID's into a single folder, so you end up with a sub-folder for each RID, containing platform specific files. You can also publish the earliest RID for a platform e.g. win7-x64 and omit later versions e.g. win10-x64 as the earlier version has all the files needed for all versions (Tested this with Windows but not Mac or Linux). In theory you end up one sub-folder for mac and Windows and then one for each linux distro. Not sure what the final file size would be.

I don't think we necessarily have to do that. The main problem now is that there is no .NET Core App 2.0 compatible version of Cake released. This should be a separate package besides the existing CoreClr one I think, or else we have to create some hierarchy in the Cake.CoreClr package.
While working on this, I had to update xUnit to the latest version (or else xUnit won't run on a machine with only .NET Core 2.0 installed), but I found a bug with xUnit: https://github.com/xunit/xunit/issues/1443

it's a shame one can't dotnet publish from a nuget package because Cake.dll could essentially be an .NET Standard 1.6 assembly and a netcore2.0 app "should" be able to reference that.

Yeah, it is all a bit dicey at the moment.

PowerShell bootstrapper:
https://gist.github.com/luigiberrettini/19a124d24af74039ae87065adb007e2c

As of now Cake Tool can be installed as global tool (see https://www.nuget.org/packages/Cake.Tool/) and the tool can be run via dotnet-cake. The tool will execute your cake script.

Yep, I've already used it and it is working like a charm!

Does the cake tool use .NET Framework or .NET Core to execute .cake scripts?

ATM checking nuget packages we provide

Cake.Tool is a .NET global tool, targeting/executing on .NET Core 2.1
Cake.CoreCLR is a .NET Core console targeting/executing on .NET Core 2.0
Cake is a .NET Framework console targeting/executing on .NET Framework 4.6.1

I think this can be closed. Available runners and supported plattforms are documented at https://cakebuild.net/docs/running-builds/runners/#supported-platforms.


:robot: triggering CI on branch 'release-next' after synching from upstream/main

/test unit

/test unit

/test unit

OCF Webhook is merging this PR


Remove dependency on custom core

Use the direct message method to send the text and just display if wanted.

done in 4.0


Updated system schema for Foundry 10.

All the errors preventing the module from loading in Foundry 10 are repaired, but some optional data (e.g. author contact information) is still missing.  It also looks like the "esmodules" attribute should be changed to "scripts", but it does load without warnings as it is.
The version number has been incremented, but I suggest reviewing and updating the schema prior to release of a new version.

This is running into keyerrors when updating document attributes (luck, ability scores, hp, etc). This is likely due to the changes to how documents work in v10.

OK, after playing around with this, I have decided that it's a mess.  =]  The data model is a bit wonky, and I think it's mostly due to the GetData method override.  I might be able to straighten it out, but I'm thinking that data migration is probably way more trouble than it is worth.  Do you have any idea how many people used this and have a bunch of data in the existing version?
The good news is that most of what I'm seeing are warnings that the deprecated .data fields are being read, but they still function.  I don't have enough data in my test world to see the calculation errors yet.

On the topic of data migration... I dont think there would be more than 2 groups using WOIN left, if that. So breaking things is probably fine and I can keep an old version of the system available for v9 users.

The maximum luck value isn't editable for some reason.

The maximum luck pool is based on the LUC attribute, and if you update the LUC attribute it should update fine (I didn't change this.)
Note that it only checks whether the remaining luck pool is valid if an attribute is updated.  You might consider moving that code to the prepareData method where other similar validations are done.

Deleting a language from a character causes a undefined key error.

This was because it tried to delete id rather than _id.  I'm not sure if this is something that changed in V10 or if it was broken already, but it seems to work now.


Can this run under win11? Is it possible to make compatibility adjustments for Win11 in the future?

I configured the environment and successfully lauched in powershell , but the web interface cannot be opened under win11. Is it possible to make compatibility adjustments for Win11 in the future?

What are your steps to launch the web interface?

We tested this project on the Windows platform using a 3080-8G GPU (with --sam_model_type vit_b). Feel free to report any issues you encounter while running it on Windows.

What are your steps to launch the web interface?

Please refer to the Get Started section.

Note that the terminal or powershell window will give you http://<IP_ADDRESS>:6080/ as the address to open the web ui. Under Windows I had to change this to http://<IP_ADDRESS>:6080/ for the web browser to connect and display the interface.

Note that the terminal or powershell window will give you http://<IP_ADDRESS>:6080/ as the address to open the web ui. Under Windows I had to change this to http://<IP_ADDRESS>:6080/ for the web browser to connect and display the interface.

Yes, <IP_ADDRESS>:6080 is intended for a GUI-less server. We will include this updated address information in the README.


Mail gem could break valid DKIM signatures

If I parse a message with Mail.read and output it as string valid DKIM signatures could be broken.
A gist to reproduce: https://gist.github.com/fabn/2dbf04ba442ba0b2a91b
Output on my system
Mail did a good job for Github
opendkim-testmsg: dkim_eom(): Bad signature
pid 22132 exit 70
Mail broke signature for Google

Nevermind I just found that I can access message raw_source so this is definitely a non issue.


Requiring header fields for creating certain packets

In my admittedly not terribly thorough look through the code, it seems that we have a mechanism to require certain fields in the body of the packet, per packet type, but there doesn't appear to exist a way to require particular header fields.
It would be useful to be able to require the originating_site_name in the header for notify_person_ids (and/or have it default to the remote_site_name), as this packet type originates at the remote site, and is not a reply to any other packet.

It'd be pretty easy to require certain header fields on packet creation and/or validation, but I'd need more info on what is required when in what cases. Separately, having default values in certain cases is doable but might complicate things a bit.
For example, can we assume for all packets that originating_site_name should be remote_site_name unless otherwise specified? Or would the 'rule' be, for certain specific packets, we can assume that; otherwise we must require it?

This was in reference to the issue described at https://jira.xsede.org/browse/A3M-482
Namely, notify_person_ids is one of the few packets that a remote site can POST to start a new transaction--it isn't in reply to another packet, and thus the API (which knows as little as possible about the specifics of the packets it processes) needs to know what the originating_site_name is.
As of the resolution of that JIRA issue, the API rejects packets that are posted not-in-reply-to another packet, if they don't include an originating_site_name. It also enforces that the originating_site_name must match either "TGCDB" or the remote_site_name.
It would still be nice if the amieclient would do this too, but the API gives a reasonable error message now if not, so it's probably not a big deal.

End result of the discussion is that adding a validation check for notify_person_ids that will check if originating_site_id is present and equal to either TGCDB or remote_site_name will be added to the next release, but it's nothing that needs to be pushed out on its own.


OSSM-1297: Use a central bazel cache

Once this merges:

https://github.com/maistra/envoy/pull/153
https://github.com/maistra/proxy/pull/149


nuget: allow aarch64-linux and darwin builds

Description of changes
Previously, the package allowed arm builds, but after moving I accidentally removed it.
This PR brings it back, since I tested it to work on arm.
Anyone with a Mac, if they can confirm that it works there, then we can extend the meta.package further.

Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

It does work on darwin.

It does work on darwin.

Ok, I updated the platforms attribute


tflint: 0.31.0 -> 0.32.0

Motivation for this change
https://github.com/terraform-linters/tflint/releases/tag/v0.32.0
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[x] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 137404 run on x86_64-linux 1

1 package built:

tflint


Using svelte for project(s) showcase.

https://github.com/sveltejs/svelte shall be used at projects.xithrius.cloud to show popular GitHub repositories that I'm a part of.

Began tracking in #130


Allow the user to select the number of processes from the parameters

@zshaheen we can close this issue, right?

Wow, yes

Actually no. This is for the distributed computing stuff. Which no one really uses. So it's okay to close it.


Improve logs

Give better config options in terms of formatting, pretty-printing, log-file support, etc.

Formatting still confined to one option, but am complacent with this for now; will support log files then closing the issue.

I'll add logs files later. Not a big concern of mine atm.


FIND-13346: Github artifacts v3 action deprecated

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Add optional argument support for encode method

https://github.com/tarkhov/guzzle-xml/pull/13

https://github.com/tarkhov/guzzle-xml/pull/15

https://github.com/tarkhov/guzzle-xml/pull/15


Zerossl or Google's Acme Server should be supported providers

Just a suggestion, I always have issues with letsencrypt, and that seems to be the case here as well. It'd be great to have additional support for either https://zerossl.com, https://cloud.google.com/blog/products/identity-security/automate-public-certificate-lifecycle-management-via--acme-client-api, or both. Additional options are are a Good Thing imho! To clarify, the installation is stuck on "https://gitlab.example.com/help is not yet ready" and the reason is because the certificate is failing to be issued.

this is no longer needed with 1.8 moving gitlab to kubernetes, it'll now take advantage of cert-manager and the letsencrypt certificate provider. closing issue.


Help about reuse includes

I can't figure out how to fix this: suppose I have a Customer with his contact details. I can create a Customer specification that includes contact details. But when the Customer appears in a LINQ query of an Order, if I wanted to get the Order with the Customer's details (including its contact details) how can I reuse the Customer specification without having to redo the inclusions in the Order? contact details? Maybe I didn't understand something...
Regards
Alessandro

If you're using the Aggregate design pattern you probably will have separate aggregates for Order and Customer, and the two will only be linked by an ID (e.g. Order would have a CustomerId property of type int or Guid not of type Customer). Then if you needed to get details from customer in addition to order you would typically need to load the Customer aggregate(s) individually for each ID. Note that aggregates are all about maintaining invariants and business rules which 99% of the time apply only when you're mutating the state of the system.
If you are just querying data and you need a list of results (not an Aggregate) then a lot of the time you don't want to use Specifications. You're better of to use a separate abstraction (or no abstraction at all) that allows you to just fetch the data you need in the shape you want. For instance you could use the dbContext directly or Dapper plus some custom SQL or a stored procedure, and your result wouldn't be Orders or Customers but would instead by a DTO like OrderSearchResult which would have just exactly the data you needed in it.
Does that help?

Closing the issue due to inactivity. Feel free to re-open the issue if you have further questions.


Automatically detect whether a v1 query could have run on the v2 query engine

With the v2 / multi-stage query engine becoming more robust and feature complete (https://github.com/apache/pinot/pulls?q=is%3Apr+label%3Amulti-stage+is%3Aclosed), existing users might want to potentially migrate their existing v1 query workload to the multi-stage query engine.
However, there are some differences between the two query engines in terms of supported syntax (some of which is documented in https://docs.pinot.apache.org/reference/troubleshooting/troubleshoot-multi-stage-query-engine).
In case a user wants to migrate from the v1 query engine to the v2 query engine, it would be useful to be able to automatically detect whether the existing query workload would work as is without modification. This patch adds a counter metric (optionally enabled via a broker configuration that defaults to false for performance reasons) that is incremented each time a v1 query is run that fails during compilation with the v2 query engine. If this counter remains 0 during regular query workload execution, it signals that users can potentially migrate their query workload to the multi-stage query engine.
Note that there could be situations where a query compiles successfully but fails during execution. There could also be cases where a query runs successfully but there are differences in the result between the v1 and the v2 query engine - in terms of result types, result set size etc.
In order to address these above concerns, we plan to also add a tool / API in the near future that actually executes a given query against both the query engines and compares the results. This is much more heavyweight and shouldn't be done for every query during regular execution and is intended more for ad-hoc use cases.

Metric name and broker config are up for debate (naming is hard) 😄

Approved. Waiting to fix the conflicts, which should be pretty simple given they are in the CommonConstants.


Move extheader into different package

Resolves #676

We discussed this again with @vgonkivs, and it turns out that this change is pretty invasive, which we are not satisfied with. Instead, we found a better solution:

move services/header/... to header/
keep services/header/service.go that depends on header/
This way, we keep all the services still defined under services but with actual components/data strictures being defined into their own pkg in the root. While resolving the dep cycle, its is also improves coherence in out architecture. This is also part of https://github.com/celestiaorg/celestia-node/issues/322 effort.


Run tests with current release of google-cloud-python instead of head

Currently we use the head version, because we use some things that were added after the current 0.20 release.
Relevant code in system_tests/Dockerfile and tests/google-cloud-python/Dockerfile
# Checkout the latest release.
RUN git checkout $(git describe --tags --abbrev=0)

Fixed


🎉 5.0.2

ℹ About this release

Version: 5.0.2
Type: patch
Last Release: 5.0.1 (8/17/2023, 7:07:40 PM) [?]
Commits to merge: 104 [?]

↩ Reverts

Revert "ci: Run tests with node.js v18, v20 and v21" (405853b)
Revert "ci: Downgrade is-semantic-release till it's fixed" (91c2ab5)

📦 Dependencies

Update @types/node-fetch from ^2.6.4 to ^2.6.9
Update puppeteer from ^20.9.0 to ^21.5.2

📦 Development Dependencies

Update @qiwi/semantic-release-gh-pages-plugin from ^5.2.6 to ^5.2.10
Update @sebbo2002/semantic-release-docker from ^4.0.0 to ^4.0.2
Update @semantic-release/npm from ^10.0.4 to ^11.0.1
Update @types/mocha from ^10.0.1 to ^10.0.4
Update @types/node from ^20.4.2 to ^20.8.9
Update @typescript-eslint/eslint-plugin from ^6.2.1 to ^6.10.0
Update @typescript-eslint/parser from ^6.2.1 to ^6.10.0
Update eslint from ^8.45.0 to ^8.53.0
Update eslint-plugin-jsonc from ^2.9.0 to ^2.10.0
Update typedoc from ^0.24.8 to ^0.25.3
Update typescript from ^5.1.3 to ^5.2.2

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

npm package (@latest dist-tag)
GitHub release
Docker container (sebbo2002/iubh-campus-sync:5.0.2)
v5.0.2

Your semantic-release bot :package::rocket:


Code is overlapping buttons on examples page

Actual Behaviour
Code is overlapping edit, reset, copy buttons on Examples page.
Also in the edit mode, code is overlapping the run, reset and copy buttons.

Expected Behaviour
Code should not overlap buttons in both cases.

Steps to reproduce it

Go to https://p5js.org/reference/#/p5/ambientMaterial
Move to example codes.

Screenshots of the issue

Would you like to work on the issue?
YES

@Rahulm2310 You can go ahead with a fix, possibly just break the comments into shorter lines. Thanks.

@limzykenneth I thought of adding the code or the editor below the buttons

I think moving it down would be a bit distracting in terms of alignment as the canavs and the code block would be out of alignment while the buttons on the far side would be in alignment.

@limzykenneth comments are part of code. We can't make comments shorter without making the complete code block shorter.

What I mean by making comment line short is that instead of
// ambientLight is both red and blue (magenta),
// so object only reflects it's red and blue components

We make it something like:
// ambientLight is both red and
// blue (magenta), so object only
// reflects it's red and blue components

Ohh OK, I will try to implement it. Thanks

But this way if the code is long, it may break it too

The code is not on the first line and doesn't overlap the buttons, in edit mode the user can scroll so the overlap is less of an issue there I think.

@limzykenneth Could you please explain, how are we going to make comments lines shorter. What if user enters a very long comment ? How should we handle it ?

User inputed comments is not something I'm particularly concerned about because it is not something that we can control. A user can do a lot more than just inputing a long line of comments and it isn't reasonable for us to take all of the possibilities into account.
In this particular case for ambientMaterial it is the case that the code comments that are in the documentation is causing problems and it is something we can control, so we can shorted the line length of at least the first line of comments so that it flows to the next line and an additinal line if necessary. It is not meant to address the case where a user enters a very long line of text.

Okay, so i need to shorten the length of first line of comments in all those examples in which it is overlapping the buttons

@limzykenneth I can't locate the file. Could you suggest me where to start looking

You'll need to edit the inline reference directly here

@limzykenneth how should I handle these?

Closed in #908


Ball state & behavior

Add a BallReducer to save the current position of the ball on the field, add the respective methods to be able to add a ball and move a ball on the field.

This actually needs some thinking - unlike the robot, the ball should probably not receive a target position, but rather a direction and velocity?

The result of the discussion is that the ball also receives a target position, so the simulator logic that simulates a shoot and walk against the ball needs to calculate the target position and the tick() function then works similarly to the robot position update.


[timeline] click target for chart labels

chart labels can link to the browser page for sv + place
e.g. in the screenshot below, California --> https://datacommons.org/browser/geoId/06?statVar=Annual_Consumption_Coal_ElectricPower

when it's a single place, those labels show the stat vars. In those cases, do we still want the labels to link to browser page for sv + place? I feel like it might be a little confusing if both the chips and those labels have the same text but link to different places?

good point - for a single place, the legend should also be the place label for consistency. wdyt?

I think that could be confusing for single place + multiple stat vars because the legend would just have that same place repeated multiple times? Maybe the legend could be "{statvar} - {place}"

i like that - maybe {statVar} ({place})
single place, n statvar --> link to stat var x place page, and label is {statVar} ({place})
multi place, multi statvar --> link to place page, label stays as {place}

thinking about it, I think it makes more sense to do:
single statvar -> link to stat var x place page & label is {stat var} ({place})
multi statvar -> link to place page, label stays as {place}

I feel click target to the graph browser is more helpful than the place page. If I want to know more about those stat vars of a place, the place page does not tell much (and have no way to find where they are..)
I wonder if we should have a simple page that shows a list of "place stat var" page links and the click target goes there. This would be much more helpful for debugging etc.
Another option is to allow the graph browser expand for multiple stat vars, which is doable given the recent changes. That will also help for debugging and seeing all the different charts for a given stat var (which is the most popular use of it).
When there is a single place, showing {stat var} {place} is a bit surprising as the place just feel redundant there. I still vote for just showing the stat var label.
If there is concern on chip and label leading to different places. Maybe we should have a way to click from the stat var hierarchy to the stat var explorer when it is available. Since there is no chip in other tools anyway, and the function to explore the stat var information should belong to the hierarchy rather than the chart region?

I agree that graph browser will be more helpful than the place page.
I think the browser can already be expanded for multiple stat vars. Do you mean allowing the graph browser page to render with those stat vars already expanded when the user goes to a specific url?
I think chip and label having the same text but leading to different places would be confusing for the user. But we could think about how to click from the stat var hierarchy instead of the chip. Or maybe have a separate click target for going to the stat var explorer.

Consider the case of a single place, but many stat vars (example). As you scroll, the context of the place is easily lost, and each chart is no longer standalone. The stat var legend labels also become redundant with the chips.
I think linking to the graph browser page with the stat vars expanded sounds good. That sounds sufficient though, and hoping we don't need to make it more complicated if we can avoid it.

Okay, using graph browser for all click targets sounds good. I am okay with the label name change, we can see how it looks like for more cases.

ok, so:
single statvar -> link to stat var x place page & label is {stat var} ({place})
multi statvar -> link to place page with all stat vars expanded, label stays as {place}

yea that sounds good!


Virtualization is not enabled error on macos13 M1 runner

Description
Hi,
With the new macos-13-xlarge runner. We are getting Virtualization is not enabled in this hardware error.
Previously, in macos-13 this was working fine
Platforms affected

[ ] Azure DevOps
[ ] GitHub Actions - Standard Runners
[ ] GitHub Actions - Larger Runners

Runner images affected

[ ] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] macOS 11
[ ] macOS 12
[X] macOS 13
[ ] Windows Server 2019
[ ] Windows Server 2022

Image version and build link
https://github.com/lima-vm/lima/actions/runs/6404371464/job/17384805822?pr=1879
Is it regression?
No
Expected behavior
Virtualization machine running fine in macOS M1 runner
Actual behavior
Virtualization machine throwing error in macOS M1 runner
Repro steps
Run lima in macos-13-xlarge runner

Hello @balajiv113. We will take a look.

According to this article : "Due to a limitation of Apple's Virtualization Framework, which our hypervisor uses, nested-virtualization is not supported by arm64 runners."
So you can proceed with the macos-13  or macos-13-large runners.
I will close this issue. If you have other questions feel free to reach us.


fix(vscode): hover message, formatting and highlighting

Fixes:

"Scheme not found" error on my machine
Hover message wasn't rendered as Markdown
Prettier used to parse frontmatters as Markdown instead of YAML
Highlight all inside code block attributes
Add trailing comma to generated schemas so that ESLint won't complain (should the generated files be added to .gitignore?)

Add trailing comma to generated schemas so that ESLint won't complain (should the generated files be added to .gitignore?)

I am up to it.
Feel free to merge and do a release.

Sorry I meant "trailing new line"🥲


Your component is wholely featured in our blog post and it's awesome

Hi Juicy,
I am Binh from Vaadin. Just want to let you know that we dedicate a whole blog post for your component, A powerful web component that lets you edit JSON in browser.
Hope that brightens up your day and keep rocking these components!
Binh

Hi Binh!
That's great! Thank you so much 😃


New feature: could you add option to set default headers?

angularSails supports setting of default headers, could angular-sails also support that?
Check http://balderdashy.github.io/angularSails/#/api/ngsails.$sailsSocket, under Setting Headers.
By the way, awesome library, has nice documentation and everything, I like it much more than angularSails.
Thanks!

We are trying to update everything to work with the new sails.io.js abilities while still working for older sails.io.js.  Headers is one of the things that is involved with this.  Its in the works along with full test suites for more code confidence as its a lot of changes.

See; https://github.com/janpantel/angular-sails/blob/master/src/service/angular-sails.js#L67
You could add a request interceptor which adds your headers.

That will not work because angular-sails uses sails.io.js.  sails.io.js will just ignore the headers when it makes the actual call. See: https://github.com/balderdashy/sails.io.js/blob/master/sails.io.js#L725

Is token based Authentication currently possible with angular-sails.., it needs the authorization header?

headers (and thus setting the auth header) is on the roadmap for 2.0 which is not 100% ready, but close.

Oh rlly? No header support?
Looked into my sails project: Ive a /login and /login/authSocket route with support to authenticate the Socket. So i dont have to send credentials every request.

Any update on the progress on this?
As I also would like to be able to send a token in the header.

+1

+1

I was going to mention beta release of 2.0 and say that it has default headers... but it has been a while since I made it and looking at it now I see.... // TODO: default headers :(
But while I'm at it... RFC for what the default default headers should be (if any).  Angular's $http has some defaults mostly around Content-Type... not sure if that is relevant with $sails or not.

So... there is now 2.0 beta 2 release.  It has default headers abilities similar to how angular's $http works, just use $sailsProvider instead of $httpProvider (i.e. $sailsProvider.defaults.headers.get = {'my-header': 'some value'}).
Let me know if you have any issues with 2.0 beta.

I have just tested this with the following from an AuthService:
$sails.defaults.headers.common = {'Authorization' : 'Bearer ' + token};
$sails.defaults.headers.post = {'Authorization' : 'Bearer ' + token};
Checking the socket frames in chrome devtools I can see that the headers appear to be empty.

Is it possible to automatically append the http headers to the socket, which means we just have to replace the $http with $sails?

@snowcxt not sure what you mean.  Something like "use the headers set in $httpProvider.default.headers when making $sails calls?  So it would be something like
finalHeadersSent <- $http.default.headers.common <- $http.default.headers[method] <- $sails.default.headers.common <- $sails.default.headers[method] <- request.config.headers
Each one extending the other to add or override the values.
That could be useful when you want to use both $http and $sails together so you don't have to duplicate setting the headers on $httpProvider and $sailsProvider

Right. And it's nice to have the "headers" parameter in the $sails function as what $http does.

It is.  You can pass headers in the config and they will be merged into the defaults. Current it is set up to do something like:
finalHeadersSent <- $sails.default.headers.common <- $sails.default.headers[method] <- request.config.headers
as you can see here and really here

@AdamJessop It turns out that if I don't actually update the dist files, then no one gets them :(  lol.. yeah, I forgot to run gulp to create the dist files that you use.  So 2.0.0-beta-2 dist files are actually 1.x... No fear, 2.0.0-beta.3 has the legit dist files now and you should be able to see the headers working! Give it a try.
@snowcxt as far as your request.  I would probably put that ability behind a flag, and if I were to do that and have users/devs set the flag, I might as well just recommend doing $sailsProvider.default.headers = $httpProvider.default.headers as it would be the same affect.

Has this feature been implemented yet ? Doesn't look like it has, but I may be wrong ...

Any update on this?


Question: Best approach to implementing a timestamp with nanosecond precision and timezone awareness

We have a situation where we'd like to pass a Ruby Time instance to Rust and back, and we'd like to preserve nanosecond precision and awareness of the timezone (UTC in our case).  It seems that Magnus's support for timestamps does not include these two things.
I started looking into implementing magnus::TryConvert for a newtype that wraps chrono::DateTime<Utc>, but I see that magnus::error::protect and magnus::Value::as_rb_value are private, suggesting I'm approaching this problem in the wrong way.
What is the recommended way to add a type like this to a crate using Magnus?

The public versions of protect  and as_rb_value are in the rb_sys module (as magnus::rb_sys::protect and magnus::rb_sys::AsRawValue::as_raw) and require enabling the rb-sys feature. They're hidden behind a feature because you should only need to use protect when interfacing with Ruby C api  directly with rb-sys, and converting to/from raw VALUE lets you break all sorts of rules.
I'm not sure why I didn't implement nanosecond precision, the APIs are all there. I'll give it a go and see if I encounter any problems.
I'd welcome a PR for conversion to/from chrono::DateTime, with that and the dependency on Chrono behind a feature, like RString::to_bytes needs the bytes feature enabled to bring in the dependency on the bytes crate.

Many thanks for the helpful answer.  We're consulting on the best approach to use here.  The PR is a tempting idea.  Currently we have a Ruby wrapper around our own Rust struct using magnus::wrap(...), and that is working at a basic level but is a little unergonomic on the Ruby side.  There's a good chance we'll come back to this at some point and maybe try the TryConvert implementation.

I implemented nanosecond precision, you can give it a try by setting your Magnus dependency like:
magnus = { git = "https://github.com/matsadler/magnus.git", rev = "4109aec" }


loading SimpleForm: Assertion failed

OpenUI5 version:

=1.68

Browser/version (+device/version):
Chrome
Any other tested browsers/devices(OK/FAIL):
Firefox - fail
URL (minimal example if possible):
https://openui5.hana.ondemand.com/1.68.0/#/entity/sap.ui.layout.form.SimpleForm/sample/sap.ui.layout.sample.SimpleForm354
Hi Guys,
from version 1.68 on loading of a SimpleForm prints out an error:
assert-dbg.js:34 Assertion failed: Method 'class' must be called with exactly one class name
Best
user1545

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


Icon Cache Script Issue

Environment
Docker
Version
0.13.3
Describe the problem
Getting an "sed: -e expression #1, char 1: unknown command:.'" error when running the icon cache script. It downloads the icons to the cache but it doesn't make any changes to the default.json file.
Here is the script that I'm using.
`
pathToHomarr="/var/lib/docker/volumes/homarr_data/_data"
pathToHomarrIcons="/var/lib/docker/volumes/homarr_icons/_data"
configFile="${pathToHomarr}/default.json" iconCacheDir="${pathToHomarrIcons}/cache"; mkdir -p $iconCacheDir;
cat $configFile | grep cdn.jsdelivr.net | cut -d'"' -f 4 | while read -r line; do echo "Processing $(basename $line)"; curl -o $iconCacheDir/$(basename $line) $line; done
sed -i ".date +%F.bak" "s%https://cdn.jsdelivr.net/gh/walkxcode/dashboard-icons/png%/icons/cache%g" $configFile
echo "Done! Relaunch Homarr to update the icons."
`
Logs
No response
Context
No response
Please tick the boxes

[X] I confirm that I attached the proper logs
[X] I've read the docs
[X] I've checked for duplicate issues
[X] I've tried to debug myself

Resolved the issue with the script.

Great ! Can you explain how you did it ? @McTubbs

Absolutely!
To fix the first issue I had with getting sed: -e expression 1, char 1: unknown command: .' error, I had to remove the space between -i and ".data. Once I made that change I no longer got a error when running the script.
When I checked the default.json file to see if it was updated it wasn't. So to fix that issue I added @master to the URL string so it could find the string and replace it with the local path. I kept both sed lines since I'm not sure what the URL is going to be when creating a new app(s). I'm still testing out the dashboard and just haven't gotten around to that just yet.
This program is running in a docker container using Portainer Stacks.
New Script:
`
pathToHomarr="/var/lib/docker/volumes/homarr_data/_data"
pathToHomarrIcons="/var/lib/docker/volumes/homarr_icons/_data"
configFile="${pathToHomarr}/default.json"; iconCacheDir="${pathToHomarrIcons}/cache"; mkdir -p $iconCacheDir;
cat $configFile | grep cdn.jsdelivr.net | cut -d'"' -f 4 | while read -r line; do echo "Processing $(basename $line)"; curl -o $iconCacheDir/$(basename $line) $line; done
sed -i".date +%F.bak" "s%https://cdn.jsdelivr.net/gh/walkxcode/dashboard-icons/png%/icons/cache%g" "$configFile"
sed -i".date +%F.bak" "s%https://cdn.jsdelivr.net/gh/walkxcode/dashboard-icons@master/png%/icons/cache%g" "$configFile"
echo "Done! Relaunch Homarr to update the icons."
**Old Script:**
pathToHomarr="/var/lib/docker/volumes/homarr_data/_data"
pathToHomarrIcons="/var/lib/docker/volumes/homarr_icons/_data"
configFile="${pathToHomarr}/default.json"; iconCacheDir="${pathToHomarrIcons}/cache"; mkdir -p $iconCacheDir;
cat $configFile | grep cdn.jsdelivr.net | cut -d'"' -f 4 | while read -r line; do echo "Processing $(basename $line)"; curl -o $iconCacheDir/$(basename $line) $line; done
sed -i".date +%F.bak" "s%https://cdn.jsdelivr.net/gh/walkxcode/dashboard-icons/png%/icons/cache%g" "$configFile"
echo "Done! Relaunch Homarr to update the icons."
`
@ajnart