Distributed.jl - possibility to use other Serialization libraries?

Hi,
since recently Pluto can run Distributed, however it does not work out of the box because the standard serialization is known not to work in Pluto, but Distributed relies on it (I couldn't find a way to change this)
The PlutoSerialization is maintained in this repo https://github.com/MichaelHatherly/PlutoSerialization.jl
It would be great to be able to use this Serialization instead of the default Serialization when using Distributed inside a Pluto notebook.

Looking into which reasoning was taken for the global random number generator it seems that the preferred route is to make the ClusterSerializer pluggable. https://github.com/JuliaLang/julia/issues/23199#issuecomment-858486131
These are the two places where the ClusterSerializer is constructed

https://github.com/JuliaLang/Distributed.jl/blob/fdf56f429cd44da6d5d08cb5208bbf41d5b3d0a5/src/cluster.jl#L122
https://github.com/JuliaLang/Distributed.jl/blob/fdf56f429cd44da6d5d08cb5208bbf41d5b3d0a5/src/process_messages.jl#L159

So I guess we would need:

a AbstractClusterSerializer type for clarity
and make the above two places (and respective wrappers around it) somehow being able to pass an AbstractClusterSerializer object (similar to how rand and other methods pass a RNG object)

It seems to me like it could also be an option to make an AbstractClusterSerializer

@MichaelHatherly you build PlutoSerialization.jl, probably you have some valuable insights what could be potential solutions/problems

Malt shouldn't be relevant here since this issue is relating to running code under Distributed directly (IIRC Pluto only runs its own internal code under Malt)

I would really love to see Distributed be fully supported inside Pluto and would like to try out things myself. (e.g. creating the pluggable ClusterSerializer interface thought about above)
@jpsamaroo do you know whether and how Distributed.jl could already be replace by a fork or the like?


MAISTRA-1218 Rebase bookinfo changes

Do we have/need a process that actually triggers the build and pushes these images?

@dgn We don't. It would be nice to have if we want to provide images for each patch version. If we're ok with one set of images per minor version, we can just do it manually.


nixFlakes has been renamed to nixVersions.stable

Thanks for bringing this to our attention @lukewilliamboswell :)
I've checked the latest version of the wiki for nix flakes and it appears this step is no longer necessary.
So I'm going to close this PR and open a new one where I remove the step.


There is no documentation on how to create dependent parsers

Building parsers for inputTasks are powerful and incredibly easy to use.
But there is a use case, that isn't covered in the provided documentation:
I would like to create a parser like this:
token(Space ~> language) ~ token(Space ~> number).+

where language can be one of theses 3 values:

English
French
Spanish

and number can be one of theses 3 values:

one, two, three for English
un, deux, trois for French
uno, dos, tres for Spanish.

Our parser can be easily written as:
token(Space ~> StringBasic.examples(FixedSetExamples("English", "French", "Spanish")) ~
token(Space ~> number).+

but I don't know how to write number, because it is dependent of the value of language.
Example inputs:

English one one
Spanish dos
French trois deux

I think this must be possible, because i.e. the arguments of a command or an input task, depend on the command type.  I've studied the source code of SBT, but it is hard to understand.
More info:

Related documentation.
Related stackoverflow question

The use of the flatMap function is not documented enough.

Would you like to send us a pull request to sbt/website?

When I have time, I will document what I've learned lately.


work on adapter cut-release and nightly-release update

resolves #76

@emmyoop will we want to readd the ref to main here?

Should we consider namespacing these files since this is a shared repo and these workflows pertain only to dbt-core and dbt-<adapter>?

@mikealfare this is a shared repo for the core group.  Its purpose is not currently to be used across the org.


Add engine.NewWithClient func

closes #8725
Signed-off-by: Paul Brousseau<EMAIL_ADDRESS>What this PR does / why we need it:
Special notes for your reviewer:
If applicable:

[X] this PR contains documentation
[ ] this PR contains unit tests
[ ] this PR has been tested for backwards compatibility

hi @object88. Generally speaking we ask contributors to provide tests along with their PRs to ensure things are tested.
Have you played around with this PR yet? What do you think about the design? As it stands, right now you have to instantiate the Engine before setting the Strict flag:
e := engine.NewWithClient(client)
e.Strict = true

Perhaps there's a better way to improve the design here. What do you think?

I'm wondering maybe instead of a NewWithClient, perhaps a new function called func (e *Engine) UseConfig(config *rest.Config) might be slightly better here.
e := Engine{Strict: true, LintMode: true}
e.UseConfig(config)

Happy to hear your thoughts on this one.

Personally, I love using Option funcs pattern, and I see that's in use in the getter package.  The engine package doesn't have as many properties to contend with, but I find that it's a good way to ensure that one can expand later.  And it allows properties to be kept private (and concurrency-safe, which UseConfig may not, on an assumption).
It wasn't terribly obvious to be what unit tests would be meaningful for the initial PR.  But rewriting this with a New(opts ...OptionFunc) to set config, strict, etc., properties would certainly call for it.

Sorry for the delays in this; setting to draft while I get to reimplementing.


LLVM doesn't like JALR's immediate hexadecimal notation

It should probably be changed to regular decimal notation.

Nvm, seems to be an issue with the compiler installed in the HiFive board. Compiles fine on other CLANG compilers (tested versions 9 through 12). If this issue ever appears again, set the environment variable MICROPROBEASMHEXFMT to none, which disables the generation of all hexadecimal numbers.


Reach feature parity with petgraph for 0.2 release

Tracking issue of list of features that needs to be implemented in order to reach feature parity with petgraph.
Algorithms

[ ] Find Negative Cycle
[ ] Feedback Arc Set
[ ] K Shortest Path
[ ] All Simple Paths
[ ] Dominators
[ ] Transitive Reduction
[ ] Condensation
[ ] Kosaraju SCC
[ ] Bipartite
[ ] Isomorphism
[ ] Matching (Maximal and Perfect)

Storage

[ ] Adjacency List
[ ] Adjacency Map
[ ] Adjacency Matrix
[ ] CSR

View

[x] Reverse View
[x] Complement View

IO

[ ] Graphviz Dot Format

Hey! Commenting here because Id love to see some work done on this project. In particular I'm looking for the All Simple Paths algo. The Graph ecosystem in rust is in a pretty sad state at the moment, and I really love the layout and philosophy behind this project.

Hi @ewoolsey :)
That's really great to hear. Thanks for your warm words.
To not repeat myself, I'll leave the link to my message here as context explaining the current design of the project, and why it's been some time since my last commit.


Split elfldr "standard library" from LibUtils to new library

The more I've written the more I've kind of realized this would be a good change...
One problem though, I don't really know where to put it :face_with_spiral_eyes:
If I do this I think it's probably going to end up in modeco80/Suzi or something (or i can put it in here, i guess.)
- Suzi.Core (core library bits, needed to use Suzi. All Suzi components need this)
   include/suzi/core/Allocator.h
   include/suzi/core/Expected.h
   include/suzi/core/FioFile.h (idk, this one seems like it can stay in elfldr. maybe provide a io stream type and provide a FioFileStream in Utils or something? I don't know the best way to handle this)
   include/suzi/core/Fnv1a.h (or is the hash algorithm a internal detail? if it is, then that's no problem)
   include/suzi/core/Hash.h
   include/suzi/core/Optional.h
   include/suzi/core/TypeTraits.h
- Suzi.Containers (containers and stuff)
   include/suzi/containers/Array.h
   include/suzi/containers/CharTraits.h
   include/suzi/containers/String.h
   include/suzi/containers/StringView.h (probably going to split stringview out into its own header, yes.)
   include/suzi/containers/Vector.h
- ... ? (other Suzi partitions/modules can be added as needed)

Alt idea and probably what I'm gonna end up doing:
I think splitting out into a "runtime" library would be good, and also re-implementing libc that we pay for
That way ERL/ELF use the same runtime and everything...
I might also implement imports once ERLs are fully working to attempt deduplication

Currently being tracked in new-runtime.

partially done, I merged the branch for now but I'm gonna probably get libc fully working before I nuke it

Done.


Creating python environment for oggm-3dviz

I tried to adapt my oggm environment to work with oggm-3d-viz, but after adding all required dependencies, I still get an error of:
ImportError: cannot import name 'vtk' from 'trame.widgets'
So I tried to create a separate environment using the yml here.
mamba env create -f environment.yml
But then get this error at the end:
Installing pip dependencies: | Ran pip subprocess with arguments:
['/home/bpelto/miniconda3/envs/oggm-3dviz/bin/python', '-m', 'pip', 'install', '-U', '-r', '/home/bpelto/oggm-3dviz/condaenv.hd8f66lr.requirements.txt', '--exists-action=b']
Pip subprocess output:
Pip subprocess error:
ERROR: Could not find a version that satisfies the requirement xpystack (from versions: none)
ERROR: No matching distribution found for xpystack
failed
CondaEnvException: Pip failed
Any tips? Thanks!

ERROR: No matching distribution found for xpystack

Not sure what causes this error but with #12 this package won't be needed anymore.

Yes, for me the creation of the environment worked with the updates from https://github.com/OGGM/oggm-3dviz/pull/12

Thanks @benbovy & @pat-schmitt , I started fresh with the updates from #12 and this worked for me too.
Then when testing the getting_started.ipynb, I can't get the viz.show() to work (running in VScode on WSL: Ubuntu).
Perhaps I need to install xsfb?
error:
/home/bpelto/miniconda3/envs/oggm-3dviz/lib/python3.12/site-packages/pyvista/plotting/plotter.py:151: UserWarning:
This system does not appear to be running an xserver.
PyVista will likely segfault when rendering.
Try starting a virtual frame buffer with xvfb, or using
``pyvista.start_xvfb()`
....the Kernel crashed while executing ....
My install of oggm-3dviz is:
mamba create env -f environment.yml
mamba install pip
pip install -e .  (install with pip from oggm-3dviz cloned repo)
mamba install -c conda-forge ipywidgets             (just in case noting your link in the notebook about having ipywidgets correctly installed -- but was already satisfied)
Thanks for any insights!

This seems to be related to pyvista and/or its rendering backend (trame) rather than an issue with oggm-3dviz. I don't know if pyvista/trame supports VSCode. I don't use VSCode but I know that it has its own implementation of the jupyter widget protocol, which might causes some issues as well. The notebooks in the tutorial should work fine in Jupyterlab or Jupyter Notebook front-ends.

Thanks @benbovy, I tried running in jupyter lab but got the same error, then found this info on Pyvista on Running on WSL: https://docs.pyvista.org/version/stable/getting-started/installation.html#running-on-wsl
Which did the trick for me!


Notify internal movio repository of updates

So that we can run internal deployments when the public code changes.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (8950960) 69.49% compared to head (2950177) 69.49%.

:exclamation: Current head 2950177 differs from pull request most recent head 7fd8588. Consider uploading reports for the commit 7fd8588 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #248   +/-   ##
=======================================
  Coverage   69.49%   69.49%
=======================================
  Files          28       28
  Lines        4213     4213
=======================================
  Hits         2928     2928
  Misses       1133     1133
  Partials      152      152

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Issues/PRs on this repo are largely ignored

I was super excited to hear about this project. And I was even more excited to hear that the website encourages open participation through github. However, the reality seems that most issues go by without a comment from the maintainers, and PRs don't get attention. Is this just a temporary situation (it's only been a month or so), or is this what we can expect from this project going forward? People will be hesitant to put much time and effort to the discussions here when we are not sure if our work can actually contribute to the standard.

Hi Michael,
Apologies for the perceived lack of interest. We have been discussing and
are excited by your participation.
We are gathering the PRs for review at a convening in mid-May. I'm right
now working to gather them all up and get them ready for review by the
project partners. We are also hiring someone who will have a more active
role within the GitHub repo.
If you want to chat directly, I'm @taliesan on Twitter.
Thanks!
G.
On Apr 21, 2017 10:38 AM, "Michael Henretty"<EMAIL_ADDRESS>wrote:
I was super excited to hear about this project. And I was even more excited
to hear that the website encourages open participation through github.
However, the reality seems that most issues go by without a comment from
the maintainers, and PRs don't get attention. Is this just a temporary
situation (it's only been a month or so), or is this what we can expect
from this project going forward? People will be hesitant to put much time
and effort to the discussions here when we are not sure if our work can
actually contribute to the standard.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/TheDigitalStandard/TheDigitalStandard/issues/28, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEdrvWdec-bcnMEwU90v9odNvUmSZYI6ks5ryL9zgaJpZM4NEY1S
.

Totally makes sense. I'm very happy to hear that stuff is brewing behind the scenes, and I will be patient while you plan. Great project 👍

I need to say that I recognize what Michael states concerning the amount of comments or attention. But I also second the idea and excitement behind this project.
In my case it are my first steps on Github, I'm still very unfamiliiar with how it works and how the interaction between participants goes. So that makes me hesitant to move forward. Also it will take some time to learn.
This project has been usefull already for me, although that did not translate back into the community yet. I took the digital standard out, exported it in Excel (very bad, not open source, but I speak the language).
Then I searched for relevant aspects to put a smart/IoT washing machine to a test. So there we are.
It would be nice to have some means to easily 'filter' the digital standard for use on specific productgroups. Also a 'normal', and a 'light' testprogram can come out of it. Thats something for the future I think. First the testaspects themselves should be worked out further.

Hi,
We are currently hiring the manager who will oversee this repo. We are also
meeting in mid-May to review the PRs; we'll be back with questions and
suggestions leading up to that meeting (so shortly).
Right now, the team at CR is working to put it's first set of tests in
place  I hope to have a regular cadence established by mid-Spring.
Thanks,
G.
On May 2, 2017 8:24 AM, "includeyv"<EMAIL_ADDRESS>wrote:

I need to say that I recognize what Michael states concerning the amount
of comments or attention. But I also second the idea and excitement behind
this project.
In my case it are my first steps on Github, I'm still very unfamiliiar
with how it works and how the interaction between participants goes. So
that makes me hesitant to move forward. Also it will take some time to
learn.
This project has been usefull already for me, although that did not
translate back into the community yet. I took the digital standard out,
exported it in Excel (very bad, not open source, but I speak the language).
Then I searched for relevant aspects to put a smart/IoT washing machine to
a test. So there we are.
It would be nice to have some means to easily 'filter' the digital
standard for use on specific productgroups. Also a 'normal', and a 'light'
testprogram can come out of it. Thats something for the future I think.
First the testaspects themselves should be worked out further.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/TheDigitalStandard/TheDigitalStandard/issues/28#issuecomment-298620356,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEdrveNHRr3T6wILHnPh76s8ySPOjDuOks5r1yB-gaJpZM4NEY1S
.


chore: add readme for demo istio

Please add a requirements section with the required tools needed.

Please explain what the demo does.


Update super navigation styles to avoid govuk-layout clashes

What

Adds more specific selectors to override the generic a:link styles coming from govuk-layout.
Swaps rem for px as the base font size for the core_layout template is different to the base font size in the gem_layout template.

Why

Frontend applications that use the core_layout template were seeing blue links and tiny icons in the header - this fixes those problems so that the different templates can use the super navigation header and have it look consistent.
Visual Changes

Before:

After:

We're good we're good we're good. Just needs a quick changelog update and then 🚀


Provider template

It would be nice to have a template for new providers with some basic tests.
This would reduce the effort required to implementing a new provider.
The template could also include a provider.go some basic example, a README and a LICENCE.

Done at https://github.com/libdns/template


python/error/even_fibonacci_nums.py

Find the errors in the code and correct them

can you assign me this issue?

@reeba212 do assign me this issue as soon as possible


Let the gRPC server listen to a specified port.

The gRPC CLI now has a port option that allows the specification of a custom port. This is necessary to start multiple servers running in parallel. This can then be used by run multiple WandB runs concurrently.

@khituras Thanks for this PR.   I am going to make a couple of tweaks but this is very useful.
The grpc interface isnt fully supported but we are moving in that direction.
If you have time, i'd like to chat about what you are using the grpc interface so we can make sure to support that use case better.

@khituras Thanks for this PR.   I am going to make a couple of tweaks but this is very useful.
The grpc interface isnt fully supported but we are moving in that direction.
If you have time, i'd like to chat about what you are using the grpc interface so we can make sure to support that use case better.

@raubitsj Thanks a lot for the prompt reaction! I'll gladly chat about why I need it, see E-Mail communication.

@raubitsj Thanks a lot for the prompt reaction! I'll gladly chat about why I need it, see E-Mail communication.


Prerender.io service - lots of 504 errors

We're seeing a lot of 504 responses from prerender.io's hosted service to crawlers from our site.

Any ideas?

I can confirm the issue. Lots of 504s in webmaster tools. I tried running a local copy of prerender and its returning 504's on css files. Ive tried with css files on google hosted libraries.

Oddly I've only seen it with Bingbot and Facebook crawlers so far.  Google seems to have no issue.

@hades200082 Please send any support issues for our hosted service to<EMAIL_ADDRESS>It's easier to manage there so we can keep our github for the open source project issues.
@maxweldsouza Can you also send us an email at<EMAIL_ADDRESS>I'm happy to take a look and figure out what might be going on. If you are seeing css files get passed through to Prerender, that means something isn't set up correctly. The Prerender server should only try to load the html pages and should not try to prerender a css file.
Thanks!

I also am seeing the same issue running prerender locally.. Was there a resolution that was a general solution for this or was it really specific to your environment? Any guidance is greatly appreciated. Thanks!

These issues are usually specific to setup. Please email me at<EMAIL_ADDRESS>and I'd be happy to help.

Hi ,
I too getting same 504 error in many urls from google webmaster. Please find the attached file.

Please send us an email at<EMAIL_ADDRESS>for any issues with 504s so we can look into it. Usually it's not an issue with the Prerender server.

Hi,
I am also getting same 504 error with escape_fragment_=. Any resolution yet?
Raja K

We are also getting a lot of 504 errors with the reason being given in console that PhantomJS restarted in the middle of the request. Is there a way to avoid this happening?
Here is my server config:
PRERENDER_SERVER_PORT=4000
REDIS_URL="redis://localhost:4001/0"
PAGE_TTL=604800
RESOURCE_DOWNLOAD_TIMEOUT=10000
PAGE_DONE_CHECK_TIMEOUT=300
WAIT_AFTER_LAST_REQUEST=2000
JS_TIMEOUT=10000
JS_CHECK_TIMEOUT=300
NO_JS_EXECUTION_TIMEOUT=3000
EVALUATE_JAVASCRIPT_CHECK_TIMEOUT=300
PHANTOM_CLUSTER_NUM_WORKERS=16
PHANTOM_WORKER_ITERATIONS=500
PHANTOM_WORKER_SOFT_ITERATIONS=100

Please send us an email at<EMAIL_ADDRESS>for any issues with 504s so we can look into it. Usually it's not an issue with the Prerender server. I'm going to lock this thread but please do get in touch!


vue-cli 2.9.4 cannot resolve template's name (unknown command init)

Version
2.9.3
Reproduction link
https://codesandbox.io/s/vue
Steps to reproduce
Just init a Vue project.
What is expected?
success
What is actually happening?
PS D:\Git> vue init webpack certain-name
Unknown command init.
vue-cli · Failed to download repo vuejs-templates/[object Object]: Response code 404 (Not Found)
=> seems to resolve the word "webpack" to [object Object]
actully the version of vue-cli is 2.9.4

in 2.9.3 it works fine.

But you don't have vesion 2.9.4 right?
https://www.npmjs.com/package/vue-cli

I have the same problem.
when i run command available in vue-cli like init/list
it will show Unknow command init/list(vue init will fail,while vue list sucess)

$ npm install vue-cli -g
C:\Users\root\AppData\Roaming\npm\vue-list -> C:\Users\root\AppData\Roaming\npm\node_modules\vue-cli\bin\vue-list
C:\Users\root\AppData\Roaming\npm\vue -> C:\Users\root\AppData\Roaming\npm\node_modules\vue-cli\bin\vue
C:\Users\root\AppData\Roaming\npm\vue-init -> C:\Users\root\AppData\Roaming\npm\node_modules\vue-cli\bin\vue-init

<EMAIL_ADDRESS>added 252 packages from 208 contributors in 12.444s

$ vue init webpack my-project
Unknown command init.
vue-cli · Failed to download repo vuejs-templates/[object Object]: Response code 404 (Not Found)

$ npm install<EMAIL_ADDRESS>-g
npm WARN deprecated<EMAIL_ADDRESS>CoffeeScript on NPM has moved to "coff                      eescript" (no hyphen)
C:\Users\root\AppData\Roaming\npm\vue -> C:\Users\root\AppData\Roaming\npm\node_modules\vue-cli\bin\vue
C:\Users\root\AppData\Roaming\npm\vue-init -> C:\Users\root\AppData\Roaming\npm\node_modules\vue-cli\bin\vue-init
C:\Users\root\AppData\Roaming\npm\vue-list -> C:\Users\root\AppData\Roaming\npm\node_modules\vue-cli\bin\vue-list

<EMAIL_ADDRESS>updated 1 package in 29.3s

root@DESKTOP-BJ4BL45 MINGW64 /f/test
$ vue init nuxt/starter nuxt
A newer version of vue-cli is available.
latest:    2.9.4
installed: 2.9.3
? Project name (nuxt)

It seems it has been introduced in 2.9.4.

same issue:
Unknown command init.
vue-cli · Failed to download repo vuejs-templates/[object Object]: Response code 404 (Not Found)

seems should use command vue-init like vue-init mpvue/mpvue-quickstart my-project

看不懂大家都在聊啥....

at global variable environmental,we should use 'npm i<EMAIL_ADDRESS>-g'.and then,we can vue init webpack XXX to use vue-cli

same problem here with 2.9.4
using vue-init instead works at the time being

It is a patch update!
@yyx990803, does it should contain breaking changes?

This wasn't intended as a breaking change. It's an issue which wasn't foreseen and should be resolved soon. Sorry for the inconvenience.

$vue init webpack templeVueDemo
Unknown command init.
vue-cli · Failed to download repo vuejs-templates/[object Object]: Response code 404 (Not Found)
same problem here with 2.9.4

@Akryum don't worry, it happens! 🙂

Fixed in 2.9.5

vue2.9.4不识别那个命令 我安装2.9.3就可以了
卸载当前版本 npm uninstall vue-cli -g
安装 cnpm install<EMAIL_ADDRESS>-g

A new Update fixed the issue

@alishamohanty please open a new issue, this is unrelated to this one.


Alarm for Collateralization Ratio "at Risk"

What would you like to be added:
I would like to see an alarm function (loud if possible, using the phones alarm or equivalent) if my collateralization ratio is reaching a certain percentage. Many drops happen at times you do not expect them.
Why is this needed:
Better sleep at night times …

Hi @benameless , thanks for your feedback, the team will consider your feature request for future updates, I’m unable to provide a timeline at the moment as we’re prioritising other issues, however we’ll keep this ticket open for tracking. Thank you for your understanding and patience.


Fix libuvc depends

libuvc is needed to build so it needs to be added to the build dependencies in package.xml. Additionally the uvc config file needs to be processed and used.

Thank you very much!


Secure WebSocket error on Safari -9801: Cipher Suite negotiation failure

Describe the bug
I'm using secure websockets in my project, I have a Linux server and a WebGL client.
Everything works well on all major browsers (Chrome, Firefox, Edge) except on Safari, where the client gets the following error when connecting:
WebSocket network error: OSStatus Error -9801: Cipher Suite negotiation failure
Meanwhile the server generates the exception:
IOException: Remote prematurely closed connection
Expected behavior
No errors when connecting on Safari
Desktop (please complete the following information):

OS: Linux server, Windows/Mac client
Build target: WegGL
Unity version: 2018.4.19f1
Mirror branch: https://github.com/vis2k/Mirror/pull/1967

Closing because there is a new websocket Transport
Repo: https://github.com/MirrorNetworking/SimpleWebTransport
Folder in mirror: https://github.com/vis2k/Mirror/tree/master/Assets/Mirror/Runtime/Transport/SimpleWebTransport


Empty bag Key will conflict with other actions.

There is a conflict with press E to open the door.
Recommended to change to a key with a low reuse rate such as C.
Also, you can add a prompt that shows the use of pack slots like ( 0/15 ).

the key used is the one from the Basegame ( same as the shotgun reload ) in settings Item Tertiary use


Safari port

Merged with https://github.com/algolia/github-awesome-autocomplete/pull/27 to avoid using https://github.com/algolia/github-awesome-autocomplete/pull/25


用docker部署不能用吗

访问<IP_ADDRESS>:7088 日志返回这个提示，也无法正常解码
Multi stream daemon lite[81]: HACKING ATTEMPT: <IP_ADDRESS>:62450 set in host header loopback address.
Multi stream daemon lite[81]: HACKING ATTEMPT: <IP_ADDRESS>:62456 set in host header loopback address.
Multi stream daemon lite[81]: HACKING ATTEMPT: <IP_ADDRESS>:62466 set in host header loopback address.
如果不用127用本地地址 日志返回这个提示，也无法正常解码
Multi stream daemon lite[1846]: /udp/<IP_ADDRESS>:8008@: Created. (fd: 13)
Multi stream daemon lite[1846]: /udp/<IP_ADDRESS>:8008@ - <IP_ADDRESS>:47640: attached, cli_count = 1
Multi stream daemon lite[1846]: /udp/<IP_ADDRESS>:8008@ - <IP_ADDRESS>:47640: deattached, cli_count = 0
Multi stream daemon lite[1846]: /udp/<IP_ADDRESS>:8008@: Destroyed.
不确定是docker问题还是部署的有问题？

Я не понимаю язык на котором вы мне пишите.

Я не понимаю язык на котором вы мне пишите.
my system： macos 15 m1
docker-desktop: 4.36.0 (175267)

When accessing <IP_ADDRESS>:7088, the log returns this message, and it cannot be decoded properly:
Multi stream daemon lite[81]: HACKING ATTEMPT: <IP_ADDRESS>:62450 set in host header loopback address.
Multi stream daemon lite[81]: HACKING ATTEMPT: <IP_ADDRESS>:62456 set in host header loopback address.
Multi stream daemon lite[81]: HACKING ATTEMPT: <IP_ADDRESS>:62466 set in host header loop
back address.
If I don't use 127 and use the local address instead, the log returns this message, and it cannot be decoded properly:
Multi stream daemon lite[1846]: /udp/<IP_ADDRESS>:8008@: Created. (fd: 13)
Multi stream daemon lite[1846]: /udp/<IP_ADDRESS>:8008@ - <IP_ADDRESS>:47640: attached, cli_count = 1
Multi stream daemon lite[1846]: /udp/<IP_ADDRESS>:8008@ - <IP_ADDRESS>640: deattached, cli_count = 0
Multi stream daemon lite[1846]: /udp.3.1.129:8008@: Destroyed.
I'm not sure if this is a Docker issue or a deployment issue.

HACKING ATTEMPT - this is result of work some paranoid security code, probably I should remove it and let OS be responsible for security. :)
Multi stream daemon lite[1846]: /udp/<IP_ADDRESS>:8008@: Created. (fd: 13)
after @ should be valid interface name that will receive multicast. Check your config and replace <ifName>vlan777</ifName> with something real for your environment.
PS: it is a bit strange to see that msd used inside docker: msd is simple tool (like ping) that have no deps.


Consistent capitalization in names and titles

Capitalization in urls, titles and proper nouns should be consistent throughout the docs.
Some guidelines from pull request #44 :

"How To Create a HTTP Request": Capitalized.
"How To Use npm": Lowercased. In my opinion, this should even override title capitalization rules.
/articles/http/clients/how-to-create-a-http-request: Lowercased.

Idea: Replace the client-side css with a server-side javascript function that can recognize special cases like program names and acronyms.

@dresende said:

Why not have a file in the same folder of the article that could have some options like the correct title. If that file does not exist, the current capitalization would be done.

Other than guessing it could be done by the author. The javascript function would to be updated "constantly".

Why not have a file in the same folder of the article that could have some options like the correct title.

Sounds like writing down the title twice to me.
We can do our best to write the titles properly, but I think catching them all by-hand just isn't gonna work long-term. So I think the proper approach is to fix mistakes as we spot them (by-hand), keep our guidelines in mind when committing, and add some protection in the form of either a "fixer", "linter" or both. (Maybe like: WARN capitalized "npm" in /foo/bar/baz/article.md line 23 // automatically changing to "npm" or some such.)

I'm going to close this issue because:

Nobody has complained about this for some time, so there probably aren't too many egregious errors in the titles.
If there are egregious errors in the titles, they should get fixed on a case-by-case basis.

If you see a particular case of capitalization fail, feel free to send us a pull request.


Big mistake at MaxTipAge made in last commit

nMaxTipAge = 60 * 60 * 60;

This is big mistake: 6 * 60 * 60 is per default even to high. (=for BTC blocktime 10 Minutes!)
When the coin uses a blocktime of 300 seconds, that must be reduced to 3 or better 1,5
and NOT rised x10 ;-)
By the way... BTC Timing is used on other Routines, too.

Yes, it's high but temporarily because of the not succeeded fork we have last block more than 6h already. It should be raised now because the chain is stuck. After this it will be definitely lowered. For blocktime 2 minutes it may be 1h or even 30m.

we use actualy 1.5 * 60 * 60 for a coin with 240 minutes blocktime, this is linear to the 10 minutes of BTC. but this MaxTipAge with the same formula is at another place again in the fork! don´t have info actualy by hand, where it is. But in another part of the code again 10 minutes blocktime is expected. When you made a change here to correct to real blocktime, you need to correct the other function, too.


Fix bug where git extension fails in packaged builds

Fixes #4235 where the git extension was failing to activate.
I don't fully understand the root cause and we should investigate it further since it could affect other files in the future. It's some issue with how we process and use web pack on extensions that caused undefinedundefined to get inserted in the middle of the minified file.
The built version of extensions/git/src/main.ts has a line at the end //# sourceMappingURL=main.js.map. We have code in build/lib/extensions.ts and elsewhere that rewrites the sourceMappingURL comment and I think somehow it was getting messed up, possibly due to the block comment interfering with whatever regular expressions we use to find that line. When I ran webpack on the extension directly the file came out correctly so I don't think it's a webpack bug.

Yes it would if we tested the Git extension in the tests. It seems that our edit exposed some bug in the build scripts


Props from getInitialProps lost on page refresh

Bug report
Describe the bug
I use getInitialProps on pages to fetch data and populate the page with it.
If I directly visit the page (I write the address in the search bar of the browser or click a link)
data is fetched properly, page is displayed.
class SomeClass extends Component {
  constructor(props) {
    super(props)
  }

  static async getInitialProps () {

    let res = undefined
      try {
        res = await someRequest()
      } catch (err) {
        console.log("Requests failed", err)
      }
    }

    return {      res    }
  }
(...)
  render () {
    const { res } = this.props;
    return (
      <SomeComponent res={res} />

I also checked the example (https://github.com/zeit/next.js#fetching-data-and-component-lifecycle) but I see no difference.
To Reproduce

Go to '/somepage'
Everything fetched and rendered
Refresh page  '/somepage'
All props are lost ("res" deleted)
no related error is shown

Expected behavior
Props are not lost, page can be refreshed as in the example and/or helpful error message is shown
System information

OS: Ubuntu
Version of Next.js: "next": "^5.1.0",

Additional context
"react": "^16.3.1",
same behavior with isomorphic-unfetch and isomorphic-fetch
"isomorphic-unfetch": "^2.1.1",
I use redux, but I don't store page props there.

The most logical explanation would be that your request fails, because then it would be set to undefined as per your code. This is definitely not a bug in Next.js as we have tests for getInitialProps behavior and it hasn't changed in 1,5 years. If you provide a full reproduction we might be able to help. This is a question so I'm going to close the issue as per the issue template asking questions to be posted on https://spectrum.chat/next-js.

First page load is successful so this does not explain my problem.


About geometric affine module

Thank you for sharing the code.
There is a question about geometric affine module.
The funtion of this module stated in paper is that we can transform the local points to a normal distribution while maintaining
original geometric properties.
But can it really do that??????
The module consists of two main steps:
First, we need to substrct the mean value, which means substract the local center point.
Second, we need to divide the point value by the variance.
Maybe it is a process of normalization. The mean of this point data is indeed 0 and the variance is indeed 1.
But it cannot prove that the local points obey the trend of normal distribution.
非常感谢作者开源代码。
关于几何仿射模块有一个问题。
论文中说，局部点云数据经过该模块处理以后，那么该部分的点云数据服从正太分布。
但是真的可以做到吗？
该模块主要包含两个步骤。
第一，用局部点云的数据减去均值，其实也就是局部点云的中心点。
第二，除以方差。
这应该是一个典型的标准化的过程，此时点云数据的均值确实为0，方差确实为1。
但是此时的点云数据，应该还不是服从正太分布的吧。

According to the code realeased by author, I write a small example as below.
`import matplotlib.pyplot as plt
import numpy as np
def rand_data():
return np.random.uniform(low=0., high=3., size=(200,))
data = []
x1, y1 = [rand_data() for i in range(2)]
data.append(x1.reshape(-1, 1))
data.append(y1.reshape(-1, 1))
data = np.concatenate(data, axis=1)
plt.plot(data[:, 0], data[:, 1], 'o', color='b')
plt.show()
----------------------------------------------------------------------------------------------------------------------
mean = np.mean(data, axis=0)
std = np.std((data-mean).reshape(1, -1), keepdims=True)
data = ((data-mean).reshape(-1, 2))/(std + 1e-5)
plt.plot(data[:, 0], data[:, 1], 'o', color='r')
plt.show()`
The final result does not show that the data follows the normal distribution.
Looking forward to your reply.

Hi @LAB123-tech
thanks for pointing out this. Yes, this is definitely not a normal distribution, it is like a normalization. We will update our arrival paper in case of any misleading. Thanks for your kind reminder.
Best,
Xu

邮件已经收到，我是李阿标。

@LAB123-tech We have just updated this in arXiv, as well as some other small errors (e.g., DGCNN in Table 1, references, etc.). Thanks again for your kind reminder.

邮件已经收到，我是李阿标。


Aggregate COUNT function does not return a value on empty table

Using a pre-built minified 2.2.1 (the most recent version I could find), running a SELECT COUNT(*) operation does not return a value.
The following script creates an empty table, then runs a SELECT COUNT(*) query on that table:
<!DOCTYPE html>
<html lang="en">
<head>
    <title>NanoSQL Empty COUNT test</title>
    <meta charset="utf-8">
    <script src="https://cdn.jsdelivr.net/npm/<EMAIL_ADDRESS></head>
<body>
</body>
<script>
    nSQL().connect({
        id: 'test',
        mode: 'TEMP',
        tables: [
            {
              name : 'users',
              model : {
                'id:uuid' : {'pk': true},
                'forename:string' : {},
                'surname:string' : {},
                'gender:string' : {},
                'age:int' : {},
                'education:string' : {},
                'occupation:string' : {},
                'salary:int' : {}
              },
              indexes: {
                'forename:string' : {},
                'surname:string' : {},
                'age:int' : {},
                'gender:string' : {},
                'education:string' : {},
                'occupation:string' : {},
                'salary:int' : {}
              }
            }
        ]
    }).then(() => {
        nSQL('users')
            .query('select', ['COUNT(*) as count'])
            .exec()
            .then((rows) => { console.log(rows) });
    });
</script>
</html>

The expected result is ({count: 0}) or similar. Instead, a completely empty array is returned.
If this is the intended behaviour, perhaps the documentation could be updated to clarify, since this differs from standard SQL behaviour.

Thanks so much for reporting this!  2.2.2 has just been released and should resolve everything!
I also added a test for this that runs against SQLite to make sure empty results of aggregate functions match against SQLite.

Many thanks, I confirm the problem is resolved in 2.2.2.


P46

For consistency:
In the first test of watch functions the error thrown is all lower case, but all the others start with a capital E: throw "error" vs throw "Error"

Fixing, thanks


Environment variable in unsupported in configuration

telescope.setup {
  extensions = {
    frecency = {
      ignore_patterns = {
        "*.git/*",
        "*/tmp/*,
        "$XDG_DATA_HOME/nvim/*/doc/*.txt"  -- ignore documentations
      "},
      workspaces = {
        ["conf"]    = "~/.config",
        ["data"]    = "~/.local/share",
        ["project"] = "~/projects",
        ["wiki"]    = "~/wiki"
      }
    }
  },
}

Paths like $XDG_DATA_HOME/nvim/* or those stated with ~ can't work.
And also, complex glob is unsupported either. ~/.local/share/nvim/*/doc/*.txt in ignore_patterns do not have effect.

It is the way to use stdpath() or vim.fs.normalize() for dealing with such paths. Plugins should not convert them implicitly, and users should use them in their config, I think.
telescope.setup {
  extensions = {
    frecency = {
      ignore_patterns = {
        "*.git/*",
        "*/tmp/*,
        vim.fn.stdpath "data" .. "/*/doc/*.txt"  -- ignore documentations
      "},
      workspaces = {
        ["conf"]    = vim.fs.normalize "~/.config",
        ["data"]    = vim.fs.normalize "~/.local/share",
        ["project"] = vim.fs.normalize "~/projects",
        ["wiki"]    = vim.fs.normalize "~/wiki"
      }
    }
  },
}

I am not much familar with lua and neovim API. So kind of you. Thanks!


update kube-rbac-proxy params

To Reproduce
Steps to reproduce the behavior:

Deploy kubectl apply -k github.com/confidential-containers/operator/config/default
Check logs kubectl logs cc-operator-controller-manager-68ff8494b7-jdw27 -n confidential-containers-system -c kube-rbac-proxy

Describe the results you expected
No deprecation warnings.
Describe the results you received:
W1110 12:48:25.378333       1 main.go:165]
==== Deprecation Warning ======================

Insecure listen address will be removed.
Using --insecure-listen-address won't be possible!

The ability to run kube-rbac-proxy without TLS certificates will be removed.
Not using --tls-cert-file and --tls-private-key-file won't be possible!

For more information, please go to https://github.com/brancz/kube-rbac-proxy/issues/187

===============================================

I took a closer look and it looks this is a warning regardless of whether the parameter is used or not. We don't use it so I guess this can be ignored for now.


OR/MS Generator Port

About the PR

Ports OR and MS generators. A better system for generators, and more awesome stuff for power management
Media

[ ] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

shit


eliminate random int on schema creation (fallback to topic…

This pull request will resolve issue #145
Instead of randomly set a schema name on every query, just take the name of the topic which. This will fix the following behaviors:

Schema name correspond with topic name.
will prevent schema topic overwrite when user start a sync with a different topic
will prevent error on too much schema versions created

This pull request was already tested on a real HANA database and everything was fine.

thanks!


AttributeError: module 'evtx' has no attribute 'Evtx'

I feel like I must be making some stupid mistake, but I haven't been able to find the solution after a few hours of poking around.

Using Windows 10 and using Python 3.9.1, if I install python-evtx using "python -m pip install python-evtx"; then import the module and try to use it, I consistently get an error which states the attributes do not exist.
On the other hand, IDLE appears to see the modules and auto suggests them.

Is there something I am missing to get this to work correctly?
I appear to be having the same issue on Linux.
I don't believe the issue is in regards to how I am importing python-evtx as I have been unable to use a number of projects that use Python-Evtx; they all seem to break at calling Evtx or View.
Thank you for any help that you can provide.

In short:

More detail:
When I first started to develop this library (back in 2013 :-O) I didn't think enough about the module layout and so we end up with slightly confusing fully-qualified module/class names.
The project is Evtx and so there's a top level namespace Evtx.
There's a primary file that contains top level structures and so there's a mid level namespace Evtx.Evtx.
And within this file is a class named.... you guess it, Evtx, so the class you're trying to reference is Evtx.Evtx.Evtx. I'm sorry 🤦🏼
To debug this yourself, you can use dir(any instance, including modules) to see all the names available under that instance, e.g.

Final note, using the name evtx might kinda sorta work on Windows, which by default uses NTFS that is not case sensitive, but this is not portable and might break in weird places.

Feel free to re-open this issue with any further questions or problems - happy to help!

you could also take a look at how the example scripts do their imports:
https://github.com/williballenthin/python-evtx/blob/master/scripts/evtx_dump.py#L20-L21


Can't install Debugger for Chrome

VSCode Version: Code 1.7.1 (02611b40b24c9df2726ad8b33f5ef5f67ac30b44, 2016-11-03T13:53:14.822Z)
OS Version: Windows_NT ia32 6.1.7601
Extensions:

Extension
Author
Version

Steps to Reproduce:

Install latest VS Code
Clicked on Extensions
Selected "Debugger for Chrome"
Clicked "Install"
The button says "Installing" forever, and no feedback is given on the progress or success of the operation.

I was with connectivity problems. I am keeping the case, since tool should state that instead of "installing" forever and show no feedback to the user.

Code was very likely still installing, due to the connectivity problems.


Example Implementation does not include code for reversed output

The example implemented printed when --verbose true does not include the code for reversed output. The output reflection function had to be manually included.
// Function that reflects its argument
my_crc_basic::value_type my_crc_basic::reflect(value_type  x ) const {

  value_type        reflection = 0;
  value_type const  one = 1;

  for( std::size_t i = 0 ; i < width_ ; ++i, x >>= 1 ) {
    if ( x & one ) {
      reflection |= ( one << (width_ - 1u - i) );
    }
  }

  return reflection;
}

Thank you very much for your feedback. It's true, that part is missing and maybe other ways to use the CRC, too.

Hi Martin! Thanks for accepting my PR. Assign this issue to me. Ill work something out

Thank you very much for your support and offer. I'll transfer the issue to you.

@nitram2342 could you review my PR, please?


[FEATURE] Implement embedding removal methods for Pinecone embedding store

PineconeEmbeddingStore should have remove(String id), removeAll(Collection<String> ids), removeAll(Filter filter) and removeAll() methods implemented, similarly to PgVectorEmbeddingStore.

Can you assign this to me?


Support latest Facebook SDK on master

The is #1024 to master :+1:

  | 1 Warning
------------- | ------------
    :warning: | Needs testing on a Phone if change is non-trivial

  Generated by :no_entry_sign: danger


Moving engine into runtime

Plan to move engine into runtime.

Please keep me in the loop regarding your plans for this, as it affects my work on the Flutter license screen. Thanks.

@Hixie Acknowledged. I plan on focusing on this is around the 25th and will keep you in the loop.


[BUG]If focus have multiple icons, preview do not show any of them.

Describe the bug
As title.
To Reproduce
Open norwegian focus tree preview.
Look at NOR_a_new_norway
Expected behavior
At least one of icons should be visible
Screenshots

Fix suggestion
If icon is block instead of value, look in each icon and take one where there is no trigger block, if all icons have trigger block take first from top.
Desktop (please complete the following information):

OS: Windows 7
VSCode version: 1.70.3
Extension version: v0.7.4

This is a feature from new version of HOI4. I haven't got a chance to implement it. Same for #76.

Fixed in 0.8.0.


Multi-region banners w/ audio + Improvements

I did try my best in finding and editing the highest quality logos and audios as possible, especially to non-English game releases since they are hard to find.

Some banners were already uploaded here by the original poster but I made some minor adjustments such as removing thin white lines that are visible on darker backgrounds, turning blurred/low quality banners to HD while maintaining the integrity from the original logo, etc. The credit work is still named to them and not me as a whole.

This is only part 1. There are more parts to upload but it'll take quite some time to finish.

 Credits to the original owner who first submitted.
" * " No banner sound.
" ^ " Added a new banner sound that wasn't uploaded before.

Just out of curiosity (not trying to call you out or anything, but this is a lot of banners so I want to be sure)
There are a lot of these, you did test them all, yes?

yes i did test them one by one (even the region roms) today before finally uploading them here.

Thank you for your efforts. That's all I ask. Great work.
I'll do a last minute check when I get home and merge them.

hello! i just noticed there's an unwanted folder i uploaded so i have to remove it

YB9P (redirected as Bratz: The Movie)
idk why gamefaqs displayed it as french ver. for Bratz: Forever Diamondz but the original code was AVDF, hence AVD banner will be used normally]

What's up with the assets/YF4/YF4 (1).png and etc? (the ones with what appears to be duplicate file name thing done by Windows)

What's up with the assets/YF4/YF4 (1).png and etc? (the ones with what appears to be duplicate file name thing done by Windows)

oh that. i haven't decided which banners shall be displayed on the 3ds but all of them are working alright. is there anything need to delete some files?

It won't break the app, but I personally would prefer to stick to one. You can always PR again if you think it is not good enough and want to switch to something else.

Sorry about the delay, and thanks for the contribution!


Added Vue Example

Added Vue JS example with Fetch API and Axios.
Added Image for the result preview

@Devdre1909
This looks really awesome 👌
Nice work
I will get to this sometime today and ship it into production as soon  as possible

Hey @Devdre1909
Can you make this a column like in React example , this does not look good on Mobile Devices

On mobile, this cards become smaller and difficult to read.
There is a small issue of Dark Mode

What React example is like

Also try to use Verdana  font and  color:"purple",
Thanks
Just make these updates and we are good to go
Let me know if you have any questions

@lelouchB
Okay.
I will fix all that, but about the font Verdana, it a paid Microsoft font, how do I use it?

Hi @lelouchB
I made the changes as requested.
I also tried to dynamically change the images using themeContext. I am not really sure if the logic can be used in production.
Kindly, check it to let me know if It will work, or I should just use one image.
Thanks.

@Devdre1909
Great
I'll check and let you know
Thanks

Hey @Devdre1909
It's live
https://finalspaceapi.com/docs/vuejs/
I made a few minor changes.
Thanks for this amazing example

@lelouchB
You are welcome.


Can't even download now - trojan

After one of the latest updates, Windows Defender sees WinLaunch as a trojan, as do many other antiviruses. If defender is enabled it will delete it as soon as you download the latest release.

it seems Microsoft Defender is not seeing the latest update as malicious (<IP_ADDRESS>)


Compiler warning when compiled with -Wall

I pulled your code into a project I'm writing now and it works great, thanks for providing this, it was exactly what I was looking for.
Compiling the code with a old gcc 5.4.0 cross compiler with the -Wall flag gives one warning:
embedded-cli/lib/src/embedded_cli.c:7:20: warning: unused variable 'impl' [-Wunused-variable]
EmbeddedCliImpl* impl = (EmbeddedCliImpl*)t->_impl;
^
embedded-cli/lib/src/embedded_cli.c:803:5: note: in expansion of macro 'PREPARE_IMPL'
PREPARE_IMPL(cli)
^
It looks like the PREPARE_IMPL macro does not need to be called in the initInternalBindings() routine.
Thanks!

It looks like you're not using the latest version. In master there is no PREPARE_IMPL on line 803:
SET_FLAG(impl->flags, CLI_FLAG_DIRECT_PRINT);

Currently project already compiles with -Wall and -Werror (look in CMakeLists.txt for a full list of flags, there are plenty) so all issues should be fixed.

You're right, I had cloned the project right before this change was made and I hadn't checked the latest release.  Sorry for the bother and thanks again for providing this.


Migrate/hugo

WIP

the hugo website is now the default version for dev domain. We will test it in the next few days or weeks. Probably 1 week will be sufficient to check that it works okay.


Refactor: SectionList to styled-component

I would like to migrate section-list component to styled-component #532

@shmesa22 thanks! I'm going to close this one – could you comment on #532 in the future when you're ready to claim another?


Rowdies

Rowdies Rowdies Rowdies Rowdies Rowdies Rowdies Rowdies Rowdies Rowdies Rowdies Rowdies Rowdies

u should remove the code too btw cuz tos

Noo too much smart

yeah so that didn't happen


Support redirecting specific route of a specific subdomain

Is your feature request related to a problem? Please describe.
I'm trying to redirect a specific path of a specific subdomain to another url. The Amplify redirect rules only appear to allow redirecting every route of a subdomain or redirecting a specific route of every subdomain.
Say I have 2 subdomains redirect.example.com and noredirect.example.com. I would like

redirect.example.com and redirect.example.com/ to redirect to somemarketingdomain.com
redirect.example.com/all-other-paths, noredirect.example.com, noredirect.example.com/, and noredirect.example.com/all-other-paths to not redirect

I think this is a feature request since I've tried a bunch of things and haven't figured it out. Here are my attempts:

Write redirect rule for the root path:[
    {
        "source": "https://redirect.example.com/",
        "target": "somemarketingdomain.com",
        "status": "302",
        "condition": null
    },
]

Doesn't appear to do anything and doesn't satisfy number 1 or 2 of my requirements above.
Write redirect rule for every domain:[
    {
        "source": "/",
        "target": "somemarketingdomain.com",
        "status": "302",
        "condition": null
    },
]

Satisfies number 1 above but doesn't satisfy number 2 since noredirect.example.com also redirects.
Write redirect rule for the whole subdomain:[
    {
        "source": "https://redirect.example.com",
        "target": "somemarketingdomain.com",
        "status": "302",
        "condition": null
    },
]

Satisfies number 1 above but doesn't satisfy number 2 since redirect.example.com/all-other-paths also redirects.

Describe the solution you'd like
I would like for my attempt number 1 to work where you can redirect specific routes for a specific domain
Additional context
I saw that https://github.com/aws-amplify/amplify-console/issues/233#issuecomment-558835732 suggested creating a new Amplify app for each subdomain, in which case my attempt number 2 would work. But this seems like overkill and a really hacky solution with duplicate config and more room for error.
It seems like requiring one Amplify app per subdomain defeats the purpose of all the domain management features and being able to quickly change which subdomain points at which branch.
Use cases I see for this:

Test redirects on staging before production. I would like to test a redirect on stage.example.com before moving it into production at example.com
Migrating to a new domain. If I wanted to migrate incrementally from domainone.com to domaintwo.com, I could only redirect some routes until we were ready to move them all without having to create a whole separate Amplify app.
The ability to have a shortcode domain served from the same repo. Say I had example.com and ex.app and only wanted ex.app to serve /a/123456 routes, otherwise redirect to the main app.

Hi @OzzieOrca,
Thanks for the reporting this with a detailed explanation :)  Currently we don't support the functionality to skip certain routes under a given subdomain during redirects. We have added this as a feature request to our current backlog.

Thanks for prioritizing it. 😄 Guess I'll need to make multiple apps for now. 😦 I appreciate the answer and the clarification of the current redirect support.

I'm facing the same scenario we have a project landings that contains a ton of miniprojects for marketing, etc and we would want to use subdomains

I have the same scenario described by @luishdez. This would be super helpful.

Same issue here, would be great if this feature can be done.

some scenario is needed,  but would be also perfect to allow some access (per redirect) per IP address. Cause I do not want everybody to watch things which I do not want to share at a specified time

Same issue. Will be helpful to have some updates here.


[as2_state_estimator] Initial origin (earth->map) in ground_truth plugin

Initial origin (earth->map):

Read from an external sensor (mocap, GPS)
Read from config

Done in #430


Correct and simplify core typed array typings

Fixes #45198 #15402 #45199 #38665 and makes fixes for a number of other typed array typings issues a lot easier. May also reduce memory usage somewhat for apps not using the DOM typings.
This also makes a few other assorted fixes, like with the final parameter to methods like .find and .map, to align better with the spec.
This only touches typings. I ran into errors with paths being too long (it's not even all that heavily nested), so I wasn't able to run tests. (I created this using a sparse checkout to work around that, and so I'm also relying on CI here to check my work.)

For reference: some of the changes in this PR seem to overlap with https://github.com/microsoft/TypeScript/pull/47278

Note for reviewers: I did find this causes Node's Buffer#slice method to fail to type-check, but only the type definitions. If you disable lib checking, it'll still work as desired, but I felt this was worth calling out. (I tried doing this by adding an override for the slice as per this PR, so it might not reproduce from outright replacing it.)

To help with PR housekeeping, I'm going to close this PR since it's pretty old now.


Bug Fix: PML

Currently the PML implementation is unfinished and buggy. Default EMpy boundary conditions are being used. Only Lumerical API has a working PML boundary condition at the moment. Possibly, this could be addressed in conjunction with the modified mode solver.

Maybe we want to change how boundary conditions are dealt with in EMEPy completely. Currently it's a very ugly flag that gets passed in as a boolean.

Looking forward to your fix for PML, will it be possible to support mode solving for Bend when PML is available?

Hi @694856790,
I have mostly implemented PML using the tidy3d modesolver which can be found on the tidy3d branch. Eventually this will be merged with main. Their mode solver also allows for bend radii when solving. In the future, we will likely provide eme support for bends as well in emepy using this mode solver routine. Thanks!

Hi, thank you for your reply! I have been studying your masterpiece recently. I understand that the accuracy of the EME algorithm results is related to all the modes solved (such as EME of Lumerical MODE), but the modes solved by FDE in EMEpy are quite different from those of Lumerical and COMSOL in high-order modes (after comparison, the results of Lumerical and COMSOL are quite consistent), I would like to ask you if EMEpy can modify some parameters to make it consistent with the simulation results of commercial software. Is there a problem with the metal boundary conditions? Thanks!

Hi @694856790,
I have mostly implemented PML using the tidy3d modesolver which can be found on the tidy3d branch. Eventually this will be merged with main. Their mode solver also allows for bend radii when solving. In the future, we will likely provide eme support for bends as well in emepy using this mode solver routine. Thanks!

Hi @694856790,
Thank you for your feedback. I have a few suggestions and questions to help debug the differences you experience.
First, Lumerical has a fancy sub pixel smoothing method that creates more accurate mode solves at a given resolution. We also have our own sub pixel smoothing but it only works if you provide the polygon (waveguide width and thickness if you are just calling a mode solver). If you are providing a raw permittivity profile, then it will not use sub pixel smoothing and will be less accurate unless you have a very fine resolution.
The quickest way to test the differences is to extract the permittivity profile provided by Lumerical and feed it into our mode solver. At the moment, EMEPy only uses a simply profile where the permittivity is the same in every vectorized direction, but electromagnetic python does allow for a permittivity tensor. My first guess is that because Lumerical is using a tensor where the sub pixel smoothing is dependent on the directionality of epsilon, those small differences on the edges of the waveguide are noticeable for higher order modes. So if feeding in one of the directions from Lumerical to EMEPy does not benchmark results successfully, we will need to allow for the full tensor to be fed in.
If the issue is not related to sub pixel smoothing and the tensor profile, I have some other ideas.
Since EMEPy is using Electromagnetic python and tidy3d wrappers to perform mode-solving, the accuracy will be entirely dependent on those parameters we pass to their solvers (which I think is what you were saying). With the higher order modes, it can be tricky sometimes. Lumerical seems to have in place an efficient algorithm for excluding the spurious modes that are solved for in their backend, and organizes the physical modes by TE and TM as well.
Using electromagnetic python, all modes (including spurious) that are solved for are returned simply in the order of their effective index. We have in place a simple way to exclude spurious modes using an estimated threshold for the ratio of power in the core vs the cladding, which can be adjusted. The polarization, however, as I mentioned is just provided in the order of their effective index.
So the first thing to check is to ensure that the same mode order and polarization is being compared between various solvers. If you're sure that the same polarization and mode index is compared between our wrappers and Lumerical, there are a few other ideas that come to mind. Are you using PML with Lumerical and COMSOL? If so, the tidy3d wrapper we now have is better to use as it allows for PML boundaries.
However, if you're using a standard silicon core around 1.5µm surrounded by a full oxide cladding, with enough cladding the results seem to be quite similar with and without PML since the fields are fairly confined (near fundamental mode orders at least). I haven't explored wavelengths outside of the 1.4-1.6 µm region with these mode solvers and generally don't go beyond 10 orders of modes, so if you're doing that then the spurious threshold might need to be adjusted as well as the mesh density. Possibly some other parameters as well.
In addition, are you using the same mesh grid for EMEPy as with Lumerical? I know Lumerical does some sneaky tricks sometimes to estimate finer grids than are provided so sometimes a denser grid is needed in EMEPy. But the only major tricks for Lumerical MODE I'm aware of are 1) sub pixel smoothing and 2) some sort of analytical-computational hybrid approach to estimating transmissions across a wavelength band.
Otherwise if none of these ideas seem to fix the discrepancies than we might need to take a deeper dive into the boundary conditions used by electromagnetic python ( which I assume is the wrapper you are using).
Let me know if you want some clarification on any of these points or if you have any disagreements and I would be happy to discuss further.
Thanks, Ian

Hi, thank you for taking the time to reply me, I have tested some of your suggestions before. I have some resonance with some of your ideas. Lumerical's sub-pixel smoothing does exist. When I observed the index distribution on the monitor, I found that the index at the edge of the object was between the refractive index of the cladding and the core layer. Lumerical's sub-pixel smoothing operation may be performed using the “Sigmoid” function. Smoothing, not quite the same as EMEpy (I found that EMEpy's sub-pixel smoothing results in a smaller Neff than Lumerical), I guess the sub-pixel smoothing method is the source of the smaller difference in the fundamental modes. But for higher-order modes, I don't think sub-pixel smoothing is the main reason, because I found in testing that changes in the thickness of the boundary layer (the n_layer parameter) cause significant changes in the Neff for higher-order modes (you can test it), but I didn't find the most correct parameters. In addition, during the test comparison between EMEpy and Lumerical, I used the same boundary conditions (Metal or PEC? I may not have described it very professionally) and the same mesh structure, and I also tested rectangular silicon waveguides at a wavelength of 1.55 microns, the refractive index of the core layer is 3.47, the thickness is 0.22 microns, and the width is 0.40 microns; the refractive index of the cladding layer is 1.444, and the thickness and width are both 3 microns. Although I don't know what kind of tricks Lumerical used, it's really amazing that they can achieve the same results as those calculated by a FEM algorithm like Comsol. I hope that our discussions can improve the accuracy and reliability of numerical calculation of EMEpy. Finally, I noticed that you mentioned Tidy3d for mode solving. As far as I know, the project is closed source. Excuse me, how did you do it?

@hammy4815 Hi, thank you for taking the time to reply me, I have tested some of your suggestions before. I have some resonance with some of your ideas. Lumerical's sub-pixel smoothing does exist. When I observed the index distribution on the monitor, I found that the index at the edge of the object was between the refractive index of the cladding and the core layer. Lumerical's sub-pixel smoothing operation may be performed using the “Sigmoid” function. Smoothing, not quite the same as EMEpy (I found that EMEpy's sub-pixel smoothing results in a smaller Neff than Lumerical), I guess the sub-pixel smoothing method is the source of the smaller difference in the fundamental modes. But for higher-order modes, I don't think sub-pixel smoothing is the main reason, because I found in testing that changes in the thickness of the boundary layer (the n_layer parameter) cause significant changes in the Neff for higher-order modes (you can test it), but I didn't find the most correct parameters. In addition, during the test comparison between EMEpy and Lumerical, I used the same boundary conditions (Metal or PEC? I may not have described it very professionally) and the same mesh structure, and I also tested rectangular silicon waveguides at a wavelength of 1.55 microns, the refractive index of the core layer is 3.47, the thickness is 0.22 microns, and the width is 0.40 microns; the refractive index of the cladding layer is 1.444, and the thickness and width are both 3 microns. Although I don't know what kind of tricks Lumerical used, it's really amazing that they can achieve the same results as those calculated by a FEM algorithm like Comsol. I hope that our discussions can improve the accuracy and reliability of numerical calculation of EMEpy. Finally, I noticed that you mentioned Tidy3d for mode solving. As far as I know, the project is closed source. Excuse me, how did you do it?

@694856790 ,
Thanks for the reply. I'll first address your question regarding our tidy3d wrapper - Indeed you are correct that the tidy3d FDTD engine is closed source and requires a pay-per simulation routine. However, they have open sourced their mode-solver. Here is there example
This allowed me to build a wrapper for their mode-solver engine which can be found on the tidy3d branch of emepy
As for the sub pixel, that is useful to know they are using a sigmoid function for their smoothing technique, and I agree it makes sense we should see small differences in the fundamental mode as a result.

But for higher-order modes, I don't think sub-pixel smoothing is the main reason, because I found in testing that changes in the thickness of the boundary layer (the n_layer parameter) cause significant changes in the Neff for higher-order modes

Did you turn off the sub pixel smoothing for both EMEPy and Lumerical and compare for the higher order modes? I am confused by your statement here - it sounds like you are chaining the number of PML layers?
Thanks again,
Ian


Animate Sun Position

Should be possible using built-in nodes.

prerequisite for #16

Requirements:

Grab sun settings of current view
SunAndShadowSettings.SunAndShadowType needs to be SunAndShadowType.OneDayStudy or SunAndShadowType.MultiDayStudy
Intervals should just be set in the Revit UI
SunAndShadowSettings.SharesSettings needs to be set to true
Needs to pick a separate view for the animation
Animation phases can be set by calling SunAndShadowSettings.ActiveFrame = n where n needs to be in 1 .. SunAndShadowSettings.NumberOfFrames

This should open the door for a number of things:

Solar studies
Solar radiation studies
Illumination studies

These could be combined with:

Linear parameter changes
Reactive parameter changes (based on sun position or analysis results for active frame)
Camera movement

This turned out to be a lot easier than expected... b973a247309731c990f27dcd7bac077b08fba2a6


Added extra test projects for missing DB/test project combinations.  …

…Added first negative test for verifying incorrect total.
Also fixes #87 by rewriting how AssertRowDoesNotExist works out if the row exists.

Closing in favour of a PR into a version branch


Add support for non-ephemeral leases in the semaphore recipie

This is useful when the task you are tring to limit concurrency (such as a deployment) when a failure should retain a lock.

Please re-open to rebase and update as needed.


fix ble bug and compatible with the use cases from the documentation

use cases from the documentation:
# Run script on the first device we find called Pybricks hub
pybricksdev run ble "Pybricks Hub" demo/shortdemo.py

# Run script on device with address 90:84:2B:4A:2B:75 (doesn't work on Mac)
pybricksdev run ble 90:84:2B:4A:2B:75 demo/shortdemo.py

work on Mac!

This should be fixed in v1.0.0-alpha.8.


DynDNS pushes temporary private IP address to dynamic DNS provider

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[X] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[ ] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

-> I'm not quite sure about the second checkbox. There are at least two "related" but somehow different issues. I will differentiate this issue to these two below:

https://github.com/opnsense/core/issues/2368
https://github.com/opnsense/core/issues/3422

Describe the bug
From time to time, OPNsense, i.e. the dynamic DNS plugin, publishes a private IP address. In my case, this IP address is always <IP_ADDRESS> which I do not use and which is outside of all local subnets configured on my OPNsenes box.

In contrast to https://github.com/opnsense/core/issues/2368, I do not have any other router or NOT before the OPNsense system. In my case, the OPNsense box is directly connected to a VDSL "modem" and uses PPPoE to authenticate.
In contrast to https://github.com/opnsense/core/issues/3422, the bug occurs even if the DSL connection did not break.

To Reproduce
Steps to reproduce the behavior:

Setup dynamic DNS (in my case with https://no-ip.com).
Configure OPNsense to push new IP addresses to https://no-ip.com
Wait a longer period of time.

(I'm sorry, but I'm unable to give further information on how to reproduce the bug :(.)
Expected behavior
Never push any other IP than the public IP to the dynamic DNS provider.
Describe alternatives you considered
I've configured another dynamic DNS provider (self-hosted, uses only curl). This one did not receive the wrong IP but I'm not quite sure if that's just a coincidence.
Screenshots
If applicable, add screenshots to help explain your problem.
Relevant log files

Additional context
Dynamic DNS plugin configuration:

Environment
Software version used and hardware type if relevant, e.g.:

OPNsense 21.7.7-amd64
I'm sure that this issue did also affect multiple earlier versions, but I'm unable to tell on which version it started.
(Hardware should not be relevant for this issue.)

If you need any other information, please let me know!

CC @HackHerz (also affected IIRC)

Never push any other IP than the public IP to the dynamic DNS provider.

I think there is misunderstanding of what the "dummy" update is trying to achieve here.
But anyway the real issue seems to be the 15 seconds timeout on the update. Maybe you want to look into why that happens because I cannot.
Cheers,
Franco

@fichtner Thank you for your fast response!

I think there is misunderstanding of what the "dummy" update is trying to achieve here.

I'm quite sure the misunderstanding is on my side. Can you please clarify it for me?

But anyway the real issue seems to be the 15 seconds timeout on the update. Maybe you want to look into why that happens because I cannot.

Do I understand the timeline correctly?

IP set to <IP_ADDRESS> temporarily.
Propagate IP to https://no-ip.com
Curl timeout (What is curl used for in this step? Get the public IP from an external site? Propagate the IP to https://no-ip.com ?)

Alternative is to just prevent DynDNS from ever using anything in the private IP range. As I've volenteered to take this under the TR umbrella I can do and issue a PR.

@marjohn56 Yes, but I do not use <IP_ADDRESS>/24 as a private IP range on my OPNsense firewall. I'm glad you volunteered to issue a PR :).

I assume it's the WAN interface that you want to set the DDNS address for. It's likely that the WAN interface is picking up the <IP_ADDRESS> address from the modem. This sometimes happens if the ISP connection resets and the modem gives out that address before it gets the propper address from the ISP. You can also get around this issue by entering the address of the modem in the 'Reject Leases from' field in the DHCP client config settings on the WAN interface page. The only issue there is that you need to know the address the modem is using when it hands out the lease, and it won't be <IP_ADDRESS>!
Whatever, I'll add a bit of putty that will force DynDNS to skip if the address is in the RFC 1918 range.

Thank you for the explanation, that makes sense to me.
And once again thank you for updating the DynDNS plugin to skip private addresses!

Rather odd that it happens, however I've pushed a routine in that checks the address being used for the update is not in the RFC1918 ranges.  You can try the patch in the PR above to see if that fixes it for you.

Sorry for not following up earlier. Basically this is being set for the provider:
https://github.com/opnsense/plugins/blob/6a7babcf20398b1fa845deff35edcda498384925/dns/dyndns/src/etc/inc/plugins.inc.d/dyndns/phpDynDNS.inc#L371-L374
Either we disable dummy updates if the provider doesn't lock you out for not having your address changing or all of this is looking at it from the wrong end.

@fichtner Do I understand you correctly: The problem with the IP issued by the modem is not the problem in my case?
Maybe the dummy updates could be made "opt-out" in the DynDNS settings? With this change, normal behavior would be fine (at least for most people). People with the problem discussed here could then simply remove the flag.
Another idea that comes to my mind: If an IP update fails (like my second one, after the dummy one) the DynDNS mechanism could re-try several times. This should resolve my problem, too.

Yes, understood. Fortunaly I've never used a ddns provider that requires me to update once a month to keep it free; I can see thier logic though, OK, I'll bin that PR and re-think this. I'll create my own account at no-ip and see what gives. Might not be able to get back to this until next week though.

@marjohn56 AFAIK, the dummy update is not necessary for no-ip. I've other (consumer) routers that have the no-ip DynDNS functionality built-in and they do not necessarily update the IP every month.

I think perhaps the best way forward, so as not to break systems where it obviously does work, or we'd have heard about it, is to add a flag in the GUI to enable/disable dummy updates.

This would be a good solution, at least in my opinion.
Additionally, I would welcome a parameter in the advanced tab to configure:

Number of curl retries (if, e.g., the connection is temporarily broken)
Time delta for periodic re-tries if curl fails
etc.

I've set up a free no-ip account and  I cannot make it fail, works every time. Logs show me updating using the dummy, then 10 seconds later it updates with the correct address. I'll keep trying to make it fail though.

I've set up a free no-ip account and I cannot make it fail, works every time. Logs show me updating using the dummy, then 10 seconds later it updates with the correct address. I'll keep trying to make it fail though.

Maybe @fichtner is correct and the problem with my instance was that the second update (with the "real" IP) did not come through. If that is correct, the procedure can be the following:

Dummy IP update to no-ip succeeds.
curl (update with "real" IP) fails, e.g., because of a broken VDSL connection.
OPNsense/the DynDNS plugin logs an error and the dummy IP was the last IP submitted to no-ip.

Possible solutions:

Option to disable the dummy IP update completely.
Implement a re-try mechanism in case a curl IP update fails.


Move index functionality to rattler and create python bindings

Moves the functionality for indexing a local conda channel from rattler-build to a new crate in rattler and create python bindings so that you can index dependencies via py-rattler as faster alternative to using something like conda index. This could happen after solving and downloading dependencies from the python bindings, for example.
This would replace the logic in rattler-build and thus I will create a follow up PR to remove this logic and instead use this new crate in rattler-build.

@wolfv @baszalmstra Hey! This is my first attempt at doing what we discussed of moving the index functionality to rattler. There are some higher level design decisions that happy to talk about further like I created a new crate for the index logic, and the interface I created for the python function may not be what you expected, so let me know what you think. I didn't change any of the indexing logic.
I tried running most of the CI tasks manually but still might need to fix some things, could I get approved for running CI?

@BenjaminLowry to get rid of the Unverified badge, maybe you want to use the same email address in your git config that you are using on Github? :)
Thanks for doing this, by the way, I am excited to get rid of this in rattler-build!

@wolfv Yeah absolutely! I'm excited to start using it too. Fixed the verification

@baszalmstra Thanks, addressed!

Thanks! If you can remove the vendored feature from openssl then Ill merge!

Ah that is already the case! athanks!


Command "workbench.action.exitZenMode" would turn on ZenMode when not in ZenMode

Issue Type: Bug

Add a shortcut like

  {
      "key": "alt+e alt+w",
      "command": "workbench.action.exitZenMode",
  },

ensure not in ZenMode
fire the shortcut "alt+e alt+w"
the ZenMode would be turned on

i.e. exitZenMode behaves just like toggleZenMode ...

Does this issue occur when all extensions are disabled?: Yes
VS Code version: Code 1.63.2 (899d46d82c4c95423fb7e10e68eba52050e30ba3, 2021-12-15T09:40:02.816Z)
OS version: Windows_NT x64 10.0.19043
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-8265U CPU @ 1.60GHz (8 x 1800)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
7.80GB (0.73GB free)

Process Argv
--crash-reporter-id aef19635-1b79-4f09-9c0d-1ff29d49308d

Screen Reader
no

VM
50%

Extensions (17)

Extension
Author (truncated)
Version

dendron
den
0.76.0

dendron-paste-image
den
1.1.0

githistory
don
0.6.19

todo-tree
Gru
0.0.214

vscode-wordcount-cjk
hol
1.3.1

vscode-settings-cycler
hoo
1.0.1

path-autocomplete
ion
1.18.0

python
ms-
2021.12.1559732655

wordcount
ms-
0.1.0

markdown-checkbox
PKi
1.7.2

vscode-yaml
red
1.2.2

multi-command
ryu
1.5.1

cjk-word-handler
Sha
0.1.1

pdf
tom
1.2.0

markdown-all-in-one
yzh
3.4.0

material-theme
zhu
3.13.10

writing-extension-pack-for-binly
N/A
0.0.1

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythontb:30283811
pythonptprofiler:30281270
vsdfh931cf:30280410
vshan820:30294714
vstes263cf:30335440
vscoreces:30384385
pythondataviewer:30285071
vscod805:30301674
pythonvspyt200:30340761
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
pythonvs932:30410667
vscop804:30404766
vscop940:30404999
vsrem710cf:30416617
py55gd98cf:30419166
vscexrecpromp3t1:30407762

usage scenario:
Since the config setting zenMode.hideTabs (after toggled) would not be effective until re-enter the ZenMode (if already in ZenMode), and I only need hideTabs==true for few scenario, the command exitZenMode would help for command macros.
workaround:
define multiple shortcuts depending on the when clause "inZenMode" and "!inZenMode", which may be very troublesome when combined with another when clauses ... (especially without paranthesis... https://github.com/microsoft/vscode/issues/91473

Hello! Can I take a look at this issue? Thanks!

@chenjefferson sure! let me know if you need a pointer. Searching the codebase for 'exitZenMode' should get you started.

Thanks! Searching for exitZenMode did the trick. Fix for the issue above, although no worries if it's in your backlog!
@Binly42 In the interim, you could use "when": "inZenMode" for the keyboard shortcut.
{
      "key": "alt+e alt+w",
      "command": "workbench.action.exitZenMode",
      "when": "inZenMode",
},

@JacksonKearl , Can I please have a look at this if this has not been fixed yet?
@chenjefferson , hope it's fine with you, please let me know.


fix path name

https://github.com/dantrim/parquet-writer/blob/93338e25ebbd143f8092a270b395f13557eb585a/src/cpp/parquet_writer.cpp#L409
should be <parent>.<child> not <parent>/<child>

addressed in #23


Issue - 2580 - Etag log level changed for already committed response

https://github.com/Adobe-Consulting-Services/acs-aem-commons/issues/2580 changing log level from error to warn

Pull Request Test Coverage Report for Build 6437

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 57.366%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

bundle/src/main/java/com/adobe/acs/commons/etag/impl/EtagMessageDigestServletFilter.java
0
1
0.0%

Totals

Change from base Build 6434:
0.0%

Covered Lines:
17246

Relevant Lines:
30063

💛  - Coveralls

LGTM, but please also add a changelog entry as outlined in https://github.com/Adobe-Consulting-Services/acs-aem-commons/blob/master/CONTRIBUTING.md

LGTM, but please also add a changelog entry as outlined in https://github.com/Adobe-Consulting-Services/acs-aem-commons/blob/master/CONTRIBUTING.md

Added changelog entry.


mgmt redis, update to 2022-06

Description
Manual code on tests in https://github.com/Azure/azure-sdk-for-java/commit/eb91c93656fe2743b68d2ca4f4259fb45a5e7d36

aof-backup-enabled moved from additionalProperties to root of RedisConfiguration.
Added linked server removal after test.

live tests pass.
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.

Did you run "RedisConfigurationTests.generateConfigurationUtils" and update the code in ConfigurationUtils?

Yeah, it passed.
I checked again and find out that aof-backup-enabled is a newly added property of RedisConfiguraion. I'll add a test case of aof-backup-enabled and add it to toMap in ConfigurationUtils.

Yeah, it passed.
I checked again and find out that aof-backup-enabled is a newly added property of RedisConfiguraion. I'll add a test case of aof-backup-enabled and add it to toMap in ConfigurationUtils.

I mean this is not actually a test. It outputs the code of ConfigurationUtils.java. You should be able to just paste and replace the output to ConfigurationUtils.java

I mean this is not actually a test. It outputs the code of ConfigurationUtils.java. You should be able to just paste the output and replace part of ConfigurationUtils.java <-- this replacement is still manual

OK, got it... Thanks!


Incorrect verbage?

https://github.com/permitio/docs/blob/11d18ffcdb0729b6dff2124e7d4fe1a071633208/docs/how-to/ownership.mdx?plain=1#L110
Is this correct? Shouldn't it read:
As `folder` is set as parent of `file`, all permissions set on a `folder` will automatically propagate to `files` within those folders as well.

I think the first reference to folder and file are incorrectly flipped.

You are very clearly correct.
Would you like to open this as a quick PR for the open-source contribution creds? (Otherwise I'd fix it myself)


Rereference MIssing how outgoing communication works.

please consider adding references on how outgoing networking works in kubernetes / AKS.

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

ID: 9615c579-305d-2482-7eaf-2d120c75e8d5
Version Independent ID: 064f4a10-bd74-f554-9550-781dd3e838cb
Content: Concepts - Networking in Azure Kubernetes Services (AKS) - Azure Kubernetes Service
Content Source: articles/aks/concepts-network.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

@porrascarlos802018
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.

@mlearned
Can you please check and add your comments on this doc update request as applicable.

Thank you @porrascarlos802018 for the feedback!
We have added an item to our backlog and will prioritize accordingly.


Ansible like manipulating data

While utilizing Maestro for our UI tests, I've observed the need for enhanced data and string manipulation capabilities similar to those provided by Ansible. Having a more robust set of tools for these operations would substantially augment Maestro's functionality and ease many testing scenarios.

map: This Ansible filter can extract and transform items from a list. In a testing scenario, it could be useful to transform test data on-the-fly.

combine: This filter allows merging of dictionaries into one. This could be handy when combining multiple sets of test data into a single set for certain tests.

regex_replace: Allows users to replace strings or patterns using regular expressions. When dealing with dynamic strings or generating unique identifiers for tests, this becomes invaluable.

default: Provides a default value when a specific variable is undefined or null. This can make tests more robust by ensuring there's always a fallback value.

search: Useful for searching a pattern within a string. It could aid in verification tests where you need to ensure certain patterns or data points are present in your output.

json_query: This filter allows complex data querying of JSON structures based on specific conditions, helping in parsing and verifying complex test outputs.

flatten: Converts lists of lists into a single list. It can be essential when dealing with hierarchical or nested test data

Uniq: Ability to remove duplicate values from lists or sets, ensuring each entry is unique.

Count: Count occurrences of a particular item or characteristic within a set of data.

Hello @danieltaub, thank you for this detailed feature request.
To me, this sounds like something that'd be better off implemented in JS, rather than YAML. Wdyt?


Changing service network and pod

After installing a cluster, I noticed that the pod's network was not changed as configured in the inventory, setting the default network to <IP_ADDRESS>/14.
After adding the two lines below, the following installation correctly configured the pod's network.
We add the lines below:
"cluster_networks": [{"cidr": cluster_network_cidr, "host_prefix": (cluster_network_host_prefix | int)}],
"service_networks": [{"cidr": service_network_cidr}],
Changed file:
"roles/create_cluster/tasks/main.yml"

Hey Douglas sorry for the delay. Thanks for the PR. This functionality has already been added in PR #168 in a different part of the code.


Make the Highlights tab the default one

After the implementation and testing of all sections in the Highlights tab
Navigate the user to the Highlights tab on the asset link

Closed in https://github.com/digma-ai/digma-intellij-plugin/issues/2177


[Error message] add file permissions in message

Hi
I had an error with file permissions (exactly this one https://github.com/actions/deploy-pages/issues/303#issuecomment-1915616098)
I think we can add the file permissions message to users
Artifact could not be deployed. Please ensure the content does not contain any hard links, symlinks and total size is less than 10GB.

to
Artifact could not be deployed. Please ensure the content does not contain any hard links, symlinks, have correct file permissions and total size is less than 10GB.

Related to:

https://github.com/actions/deploy-pages/issues/303
https://github.com/actions/upload-pages-artifact
file permissions explained in https://github.com/actions/upload-pages-artifact/blob/main/README.md

Pages deployments won't fail based on file permissions anymore.


merge master and follow to golang-lint 1.40

keep v9 branch and master branch in sync

@monkey92t you should have perms to merge PRs, right? If so - feel free to do so.

No, I don’t have any permissions

@monkey92t strange - I've sent you an invite again. Please check your email.

OK, thanks!


Future of the library liquid and vdom

Hello,
First I thank you for your work.
My question is have you planned improvements to these two libraries ?

Hi,
I am not sure what to do right now with this libraries. I've found a way to make them work way much faster, but it will require complete rewrite of vdom and liquid. I also don't like some parts of the API, like final myComponent = vdom.componentFactory(MyComponent);, but I am out of ideas how to implement it in a better way without any serious perf cost.
I also found that dart2js generated code is really slow in Firefox, even with '--trust-type-annotations', '--trust-primitives' flags, and dart:html abstraction has a ridiculous perf cost for things like assigning classes to dom elements.
The problem is that I don't use this library in any projects, so its future depends on how much free time I have to work on it. And I really don't want to be a maintainer of project like this, I would be way much happier if I could just write code for project like this.

Thank you for your response and your work
I understand your point of view.
Liquid is for me a really good and simple framework for dart.

I like tagtree, but I fear that tagtree API will be too complicated for newcomers.


(chore) cleanup dependencies

Signed-off-by: Jerome Simeon<EMAIL_ADDRESS>

Coverage remained the same at 95.312% when pulling 271634f4164224d2659fdf33517a42dbda6461dc on jeromesimeon:js-fix-typo-in-packagejson into dfbbf56f3792bd1f6da2c2bedced14e643bde7e2 on accordproject:master.


k-space data undersampled or (synthesized) unsampled?

Hi @ad12 , thanks for such a great contribution to our community.
I have two minor questions about the k-space data:

the y direction is under-sampled, while the paper suggests that 'unsampled k-space data was subsequently synthesized using Autocalibrating Reconstruction for Cartesian imaging (ARC)'
according to the appendix, the size of the readout direction is 416 and padded to 512. However, the k-space data does not have zero elements near the edge.
Not sure if these questions are just features of the Orchestra toolbox. Thanks in advance!

Another little one: the appendix does not show the FOV and resolution in the z direction.

Hi @guanhuaw, thanks for your interest in the dataset and for these questions! Adding some details below

The ky direction is fully-sampled in the elliptical region (see image below). The scan was acquired with 2x1 parallel imaging with elliptical sampling. ARC was used to synthesize the k-space data in the elliptical region.
The kspace has been hybridized such that the matrix structure is (x, ky, kz). To use the 3D kspace, where all dimensions correspond to the kspace domain, undo the hybridization by taking the fft along the x dimension.
Thanks for this note - we will add the FOV and resolution in the z-direction in the updated manuscript

Thanks for the reply! The cvs header file shows that the slice thickness is 1.6mm and slice spacing is 0.8mm. The z-axis resolution, accordingly, is 2.4mm?

The nominal acquisition resolution along the z direction is 1.6mm. This direction is then zero-padded to 2x the matrix size (following ZIP2 protocol), so the effective z-resolution of the zero-padded data (and DICOM images) is 0.8mm.

Thanks again for the further help!


getting error while bower install

This is the error
EMALFORMED Failed to read /Users/deepak/Workspace/nite-foodie/.bower-tmp/angularjs-autocomplete-45395-cpx7kn/bower.json

Thanks for reporting this. I will make a fix soon

updated with 0.4.5

Thanks @allenhwkim


Implement ternary operator fixes in VB and add IIF -> If analyzer/fix

Translates ternary code fixes to VB and adds a new code fix to translate the old IIF to If for VB.

@jwooley Some tests are failing, can you check?

I'll recheck this evening. Was passing locally.

I tried updating the project to the current master, but am now getting an error in the XUnit runner with the xUnit RC 2 update in that build. I ran out of time to debug further tonight. I'll try to look at it again tomorrow. I did see that there's already a xUnit RC3 update in nuget as well, so I wonder if it would be an issue to try bumping that up on more rev.

@jwooley Please update to RC1 using current HEAD.

Build failing on C# tests which were not impacted by this pull request.
6C:\projects\code-cracker\packages\xunit.runners.2.0.0-beta5-build2785\tools\xunit.console.x86.exe "C:\projects\code-cracker\test\CSharp\CodeCracker.Test\bin\Debug\CodeCracker.Test.CSharp.dll" -appveyor

@jwooley I rebuilt it, and it ok as you can see on the appveyor status.


https in sitemap

How do I force sitemap to be generated with https, not http?

Sorry, I just realized it's configured in hexo itself


Booking Order

Move Cancel Confirm to View page
Edit cancel booking for canceled booking
Error when ctreate booking order deliveryDate more than 45 days from now

Coverage increased (+0.002%) to 77.65% when pulling 7503a3467711cdd0e8c4638a0dbd7b8b63d91ff3 on kuswandanu:dev into ce2bac492949af89ebd63bfea8ab9eb0b9be67d7 on danliris:dev.


Update to slash commands

Also make compatible with discord 2.0 https://gist.github.com/BobDotCom/81bfea226e330675e7f1e282b24b371e

https://pycord-squid-branch.readthedocs.io/en/latest/migrating_to_v2.html


Inconsequential questions about mounting

The rclone example command copyparty shows sets vendor=owncloud... any particular reason why? I don't think copyparty has anything to do with owncloud? They do have a WebDAV vendor which feels more accurate.
It also mentions that "rclone is much slower" than webdaw/davfs2 - is this true? I would have though rclone would be way faster, using a newer protocol and all? At least one random dude out there benchmarked it to be faster than davfs2, but idk if that's true for copyparty specifically?

the main reason for vendor=owncloud is so it enables useOCMtime in rclone, so files will retain their lastmodified timestamps after uploading, although canChunk might have been a cool feature to enable as well, to improve cloudflare support... would require some work on the server, so I'll make a note of that :>
as for the remark on the services page saying "you can use rclone instead, which is much slower", this is probably still true when  you're transferring many small files, but rclone performance improved A LOT when it became possible to specify pacer_min_sleep=0.01ms in the rclone configs, so this may need to be benchmarked again, thx for the heads-up!

Thank you!

well, we can scrap the davfs2 recommendation -- rclone beats it out of the water, especially when there's a bit of latency. This is assuming you're on rclone v1.63 or later and using the pacer_min_sleep=0.01ms rclone option; if that's not the case, then you're back in the timeframe from when the docs were written and davfs2 was still the best choice heh
I'll update the docs when I have some more actual numbers, but it's looking like rclone being up to about twice as fast :>