Errore Plasm in BOOL

with this value:
hpc = MKPOLS([PointNd(p) for p in eachrow(pointlist)],[Cell(tet) for tet in eachrow(tets)])
generated by file  Plasm/test/random-tetgen.jl
Both the following expressions go in error, probably by the same cause:
VIEWCOMPLEX(BOOL(UNION(hpc)), show=["CF"])
OUTERS(BOOL(UNION(hpc)))

with log:
#args=54
ERROR: cannot find internal point
Stacktrace:
  [1] error(s::String)
    @ Base ./error.jl:35
  [2] find_internal_point(lar::Lar; max_attempts::Int64)
    @ Plasm ~/github.com/scrgiorgio/Plasm.jl/src/lar/boolean.jl:369
  [3] find_internal_point
    @ ~/github.com/scrgiorgio/Plasm.jl/src/lar/boolean.jl:289 [inlined]
  [4] #931
    @ ./none:0 [inlined]
  [5] iterate
    @ ./generator.jl:47 [inlined]
  [6] collect_to!
    @ ./array.jl:892 [inlined]
  [7] collect_to_with_first!
    @ ./array.jl:870 [inlined]
  [8] collect(itr::Base.Generator{Vector{Lar}, Plasm.var"#931#933"})
    @ Base ./array.jl:844
  [9] BOOL(arrangement::Lar; input_args::Vector{Lar}, bool_op::Plasm.var"#my_bool_op#941"{Hpc, Vector{Hpc}}, debug_mode::Bool)
    @ Plasm ~/github.com/scrgiorgio/Plasm.jl/src/lar/boolean.jl:511
 [10] BOOL(hpc::Hpc; debug_mode::Bool)
    @ Plasm ~/github.com/scrgiorgio/Plasm.jl/src/lar/boolean.jl:679
 [11] BOOL(hpc::Hpc)
    @ Plasm ~/github.com/scrgiorgio/Plasm.jl/src/lar/boolean.jl:610
 [12] top-level scope
    @ REPL[38]:1

@apaoluzzi do not call OUTERS to the BOOL operation.
The output of BOOL does not contain any OUTER atom

Closing since it is solved


How to connect Chatbot to other Database?

Hi @alfredfrancis ,
Thks for awesome project. I have practiced with your API integrating example, but can you suggest me how to connect data from my database that can interactive with your chatbot?
Ex:
When you ask
"How many dogs in cage?"
Then the chatbot will get data from my database (PostgreSQL, MySQL, etc.), get value from column number_dog and will reply :
"It's %number_dog_value% dogs in cage"
Thank a lot.

You can connect to different databases via REST API


rubygems publishing by tagging?

I've started using tags instead of rake release to push out versions of gems. I've manually pushed mn-requirements to rubygems, since neither 0.0.3 nor 0.1.0 was pushed (the former 5 days ago). Could I confirm that tagging is set up to publish mn-requirements ?

For starters, you need to run these commands to grant publication privileges.
gem owner mn-requirements -a ribose
gem owner mn-requirements -a metanorma

Because we use general workflow for many repositories, some of them require access to GH rubygems.
@ronaldtse could you please confirm that this repo has access to both:

secrets.METANORMA_CI_RUBYGEMS_API_KEY
secrets.METANORMA_CI_PAT_TOKEN

*both are org-level secrets as far as I remember

@CAMOBAP yes they are both available here, it is only with private repos we have to manually set. The issue is that @opoudjis needs to add the CI user as gen owner.

Done.

Confirm done.

@ronaldtse @opoudjis  should we rerun latest failed release workflow?

Btw, Github Actions is going to fail for all of mn-requirements, metanorma-standoc and isodoc, because of circular dependencies: they all need to be upgraded together.

@opoudjis https://github.com/metanorma/metanorma-standoc/tree/main/.github/workflows doesn't have automatic release workflow as many other gems, I tried to add release workflow to only independent components
Anyway let me know if there is make sense to add/remove automatic release workflow for some repos

@CAMOBAP we want automatic release for all gems in the metanorma, relaton orgs. Thanks!

Ok
cc @andrew2net to be on the same page. AFAIK you are probably working on relaton projects more than others

Btw, Github Actions is going to fail for all of mn-requirements, metanorma-standoc and isodoc, because of circular dependencies: they all need to be upgraded together.

This "circle" comes from development deps, to me it looks like order should be:

@ronaldtse could you please confirm that RELATON_CI_RUBYGEMS_API_KEY or similar is available for relaton org?

https://github.com/metanorma/isodoc-i18n/issues/4 Attention @CAMOBAP

CI configuration updated for all rmetanorma and relaton epos

@CAMOBAP I've tried to make tag and push it to relaton-bipm repo but nothing happened.
error parsing called workflow "relaton/support/.github/workflows/release.yml@master": failed to fetch workflow: workflow was not found.

https://github.com/relaton/relaton-bipm/actions/runs/3028659644

@andrew2net my bad forgot to push relaton/support could you please try one more time

@CAMOBAP I've already released the relaton-bipm using rake release command. Will try next time.

@CAMOBAP what is correct workflow to release gem?
I make a commit, then I push the commit and wait until all GHA tests completed, then create a tag and push it.
Then I push tag the GHA tests run again. I think we don't need it.

Then I push tag the GHA tests run again. I think we don't need it.

Agree, we have workaround for this on metanorma gems, I will implement the same approach for relaton gems

@andrew2net done


[Test] Substitute embedded null test with a custom offset test.

I noticed that we don't receive any diagnostics until we invoke Lexer's 'lex' function.
As a result, the current test 'DiagnoseEmbeddedNulOffset' doesn't actually test anything.
Moreover, I don't see any correlation between embedded null and the initial offset of the Lexer. It seems more focused on whether the Lexer's offset works correctly. To address this, I replaced the existing test with one that specifically targets the offset functionality.

Hey, thank you for taking the time to review this @ahoppen. However, I have a little confusion regarding your feedback. Would you mind if I ask some questions? I've left some comments on the code.


Keep window always on top

I use defos on a day to day basis to change resolution on the fly and test against the most common mobile ones and it is great!
It would be also great to have an option to always keep the window on top of the others. I use hot reload a lot and that would help not switch windows constantly.

Should be easy to add
https://stackoverflow.com/a/31170225

BTW if you need this sooner you can download DefOS and modify the set window position function to have the flag enabled by default and then use that version while you are developing.


ResolutionImpossible when constraints seem compatible

I have a project which uses pip-compile to generate requirements.txt and requirements-test.txt. I simply bump some package version in pyproject.toml and run these commands:
    pip-compile --generate-hashes --build-isolation --output-file=requirements.txt pyproject.toml
    pip-compile --generate-hashes --build-isolation --extra=dev --constraint=requirements.txt --output-file=requirements-test.txt pyproject.toml

The requirements-test.txt file is used for tests. All is run via tox.
When I try to bump responses package to a new version (0.23.1 -> 0.25.3), I get the following error:
pip-compile: commands[1]> pip-compile --generate-hashes --build-isolation --extra=dev --constraint=requirements.txt --output-file=requirements-test.txt pyproject.toml
  ERROR: Cannot install foo (pyproject.toml) because these package versions have conflicting dependencies.
Discarding requests==2.28.1 (from -r requirements-test.txt (line 297)) to proceed the resolution
  ERROR: Cannot install foo (pyproject.toml) because these package versions have conflicting dependencies.
Traceback (most recent call last):
  File "/home/dabljues/projects/foo/.tox/pip-compile/lib/python3.10/site-packages/pip/_vendor/resolvelib/resolvers.py", line 316, in _backjump
    name, candidate = broken_state.mapping.popitem()
KeyError: 'dictionary is empty'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/dabljues/projects/foo/.tox/pip-compile/lib/python3.10/site-packages/pip/_internal/resolution/resolvelib/resolver.py", line 95, in resolve
    result = self._result = resolver.resolve(
  File "/home/dabljues/projects/foo/.tox/pip-compile/lib/python3.10/site-packages/pip/_vendor/resolvelib/resolvers.py", line 546, in resolve
    state = resolution.resolve(requirements, max_rounds=max_rounds)
  File "/home/dabljues/projects/foo/.tox/pip-compile/lib/python3.10/site-packages/pip/_vendor/resolvelib/resolvers.py", line 434, in resolve
    success = self._backjump(causes)
  File "/home/dabljues/projects/foo/.tox/pip-compile/lib/python3.10/site-packages/pip/_vendor/resolvelib/resolvers.py", line 318, in _backjump
    raise ResolutionImpossible(causes)
pip._vendor.resolvelib.resolvers.ResolutionImpossible: [RequirementInformation(requirement=SpecifierRequirement('requests<3.0.0,>=2.24.0'), parent=LinkCandidate('<some_package_from_internal_artifactory> (from <internal_artifactory>) (requires-python:>=3.10)')), RequirementInformation(requirement=SpecifierRequirement('requests<3.0,>=2.30.0'), parent=LinkCandidate('https://files.pythonhosted.org/packages/12/24/93293d0be0db9da1ed8dfc5e6af700fdd40e8f10a928704dd179db9f03c1/responses-0.25.3-py3-none-any.whl (from https://pypi.org/simple/responses/) (requires-python:>=3.8)'))]

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

Traceback (most recent call last):
  File "/home/dabljues/projects/foo/.tox/pip-compile/bin/pip-compile", line 8, in <module>
    sys.exit(cli())
  File "/home/dabljues/projects/foo/.tox/pip-compile/lib/python3.10/site-packages/click/core.py", line 1157, in __call__
    return self.main(*args, **kwargs)
  File "/home/dabljues/projects/foo/.tox/pip-compile/lib/python3.10/site-packages/click/core.py", line 1078, in main
    rv = self.invoke(ctx)
  File "/home/dabljues/projects/foo/.tox/pip-compile/lib/python3.10/site-packages/click/core.py", line 1434, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/dabljues/projects/foo/.tox/pip-compile/lib/python3.10/site-packages/click/core.py", line 783, in invoke
    return __callback(*args, **kwargs)
  File "/home/dabljues/projects/foo/.tox/pip-compile/lib/python3.10/site-packages/click/decorators.py", line 33, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/home/dabljues/projects/foo/.tox/pip-compile/lib/python3.10/site-packages/piptools/scripts/compile.py", line 470, in cli
    results = resolver.resolve(max_rounds=max_rounds)
  File "/home/dabljues/projects/foo/.tox/pip-compile/lib/python3.10/site-packages/piptools/resolver.py", line 604, in resolve
    is_resolved = self._do_resolve(
  File "/home/dabljues/projects/foo/.tox/pip-compile/lib/python3.10/site-packages/piptools/resolver.py", line 636, in _do_resolve
    resolver.resolve(
  File "/home/dabljues/projects/foo/.tox/pip-compile/lib/python3.10/site-packages/pip/_internal/resolution/resolvelib/resolver.py", line 104, in resolve
    raise error from e
pip._internal.exceptions.DistributionNotFound: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/topics/dependency-resolution/#dealing-with-dependency-conflicts

The responses library is in dev requirements of the package:
[project.optional-dependencies]
dev = [
    "responses == 0.25.3",
]

so it's obviously the 2nd command (with requirements-test.txt) that fails.
So, the "internal package" (redacted, sorry, let's call it bar) requires requests, and the responses package also requires them.
So, looking at the ResolutionImpossible message:
[RequirementInformation(requirement=SpecifierRequirement('requests<3.0.0,>=2.24.0'), parent=LinkCandidate('<some_package_from_internal_artifactory> (from <internal_artifactory>) (requires-python:>=3.10)')), RequirementInformation(requirement=SpecifierRequirement('requests<3.0,>=2.30.0'), parent=LinkCandidate('https://files.pythonhosted.org/packages/12/24/93293d0be0db9da1ed8dfc5e6af700fdd40e8f10a928704dd179db9f03c1/responses-0.25.3-py3-none-any.whl (from https://pypi.org/simple/responses/) (requires-python:>=3.8)'))]

well - it looks like those constraints are not conflicting. The constraints are >=, I'm using Python 3.10, serves both cases. These two packages in the exact versions mentioned here can be installed in a virtual environment, no problem.
requests are at version 2.28.1 in both requirements files. I know I can bump it and then it would work. However, in this case especially, requests is not even a direct dependency. So I would have to pin its version here in this project, just because some 3rd party library uses it. I also know I can use --upgrade, but it bumps all the dependencies, which is not desired - for example it can attempt to bump some library which dependencies could potentially collide with some other one's. I would expect that the first pip-compile would bump the requests in requirements.txt. If it was a direct dependency (like requests >= 2.30 directly in my project's pyproject.toml) - pip-compile would bump it. But since it's indirect - it doesn't?
Is it a bug? Or am I doing something wrong?
Environment Versions

OS Type: Linux
Python version: 3.10
pip version: 23.3.1
pip-tools version: 7.3.0

I guess the best option I have is to remove requests==2.28.1 from requirements.txt and run pip-compile again.


tsp, cadl-ranch Type/Property/Optional

dashboard https://azure.github.io/cadl-ranch/

Add for "Type/Property/Optional".
already had some tests in typespec-tests\src\test\java\com\type\property\optional

click the case on dashboard for details (expected input, expected output that need assert, etc.)
do not call "WithResponse" API, unless no alternative.

relate to pr#2438


Updated the follow workitem sample to include unfollow

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot


Enable consistent type import/export rules

Since TS 4.5, you can do import { type A, B } from 'abc'; to combine type and non-type imports in a single import statement. Also, type imports are preferable because they are more explicit and also allow the transpiler to more easily drop the import since it knows it is just used as a type
We should be consistent about type imports. There are several related linter rules we can enable. I think we should prefer the inline type keyword so we can combine type and non-type imports from the same packages/files into 1 import statement. Right now we have many where we have duplicate imports: 1 for types and 1 for non-types.
Rules to add

import/consistent-type-specifier-style with prefer-inline: true option (should be default, but doesn't hurt to specify it)
Modify import/no-duplicates to include prefer-inline: true
@typescript-eslint/consistent-type-imports with { prefer: 'type-imports', fixStyle: 'inline-type-imports' }
@typescript-eslint/consistent-type-exports with fixMisedExportsWithInlineTypeSpecifier set to true. This rule probably won't change much.

I don't think any of these will conflict with each other. The import rules should handle the style of the imports while the typescript-eslint rules should use type info to enforce using a type only import when possible.
// Error. Prefer inline type import
import type { A } from 'a';

// Error. Duplicate imports
import { type BType } from 'b';
import { B } from 'b';

// Correct
import type A from 'a';
import { type BType, B } from 'b';
import { type C } from 'c';

Thinking about this a bit more - should we be consistent? Perhaps in some cases we do want to import a bunch of types, and having it all inline is more code, e.g.:
// What we could have now
import type { A, B, C } from 'a';

// Is this too verbose?
import { type A, type B, type C } from 'a';

Maybe just the typescript-eslint rules then which enforce using a type import when possible?


Add bold (700) weight for Roboto Condensed

Description
Vocabulary uses Roboto Condensed weight 700. It's not added at present but it should be. The goal is to add the weight and all associated documentation.
Additional context
Weight 400 was added in #5.

I would like to take this up @dhruvkb
Thank yoi

For the updation of the weights I suppose changes need to be made in the src/stories/roboto_condensed.stories.mdx
Please advise @dhruvkb

@dhruvkb can you please help

@kushaggarwal have a look as to how it was added for Source Sans Pro. If you need any further help let me know

Okay @Tanuj22
Thanks

It seems that the bold file woff,woff2 for roboto condensed isnt available in the current repository
So, @dhruvkb can you please advice as if should I add the same in the repository

@dhruvkb can you please help

I'm really sorry I hadn't checked in. Can you find the WOFF or WOFF2 files for Roboto Condensed from their officail repositories?

Hello @dhruvkb I have linked a pull request for Roboto Bold Addition
I noticed that in the current Source-Sans Pro there seems to be Semi Bold too which hasnt been added
So, shall I make a pull request for that too in roboto as well as in accidez common
Please help

@kushaggarwal I'm pretty sure Source Sans Pro has a semibold version with weight 600. Is your branch up to date? Here is a list of all fonts.

Hello @dhruvkb
I have updated the current repository of mine and updated the PR too

@kushaggarwal I'm pretty sure Source Sans Pro has a semibold version with weight 600. Is your branch up to date? Here is a list of all fonts.

I think I wasnt able to make the point clear I wanted to ask that should I add the semi bold font for the roboto-condesed font story as it is not present till now
@dhruvkb Please help

No @kushaggarwal we only need Roboto Condensed in 400 and 700 versions.

@dhruvkb can you please help as if the addtion of bold or semi bold needs to be done for accidez common font too
Thanks


Python 3.2 coverage has a syntax error

which again causes Travis CI tests to fail (#176)
https://travis-ci.org/geopy/geopy/jobs/111224816
The root cause seems to be that the latest version of coverage (4.1b2) has a syntax error
https://github.com/nedbat/coveragepy/blob/master/coverage/annotate.py#L85
introduced by adding utf8 support here
https://github.com/nedbat/coveragepy/commit/fdfdb2432dcf9e544e7c6a70421e74e5c56adf45
The standard action seems to be to use coverage 3.7.1 on Python 3.2 or drop Python 3.2 testing (see https://github.com/menegazzo/travispy/issues/20)

Per Ned Batchelder, the maintainer of coverage, v4.x is not supported on Python 3.2
https://bitbucket.org/ned/coveragepy/issues/407/coverage-failing-on-python-325-using


WIP: C-ICAP plugin

Hi,
first draft for an icap plugin to connect to new clamav plugin.
Two things are open where I could need some help (before reviewing):

Can I use two different models withing tabs? I a bit confused now. I created "general" for global icap stuff and "antivirus" to enable virus scanner. I put it in a separate tab but I'm not sure how to do the JS magic with var data_get_map. The values from general get loaded but not from antivirus. I know the JS call is wrong, but I have no idea how to make two of them

Second is to build a function, IF os-clamav installed then display contents of tab antivirus, IF NOT display a warning it has to be installed. I have some ideas with @fabianfrz hint of clamav, but I want first problem to be solved.

Any ideas? :)

Forget the first ... create a new ApiController and that's it :|

@fichtner You told me HAProxy has a function to check for acme installation but I can't find it in the code. Do you know where this was?

@fraenki where was your acme/haproxy detection ?
it is better to only list first level/top dependencies. Since we have a clamav plugin this may look like kind of circular...

@fichtner würde mal sagen hier: https://github.com/opnsense/plugins/blob/master/security/acme-client/src/opnsense/mvc/app/controllers/OPNsense/AcmeClient/Api/SettingsController.php#L128

@fichtner nice that you read all this early stage comment of mine.

It's basically the whole config, but we shouldn't use a lot of settings from plugin to plugin, im almost all cases an enable is enough to check for if the plugin detection itself is not enough.

On 12. Aug 2017, at 09:12, Michael<EMAIL_ADDRESS>wrote:
@mimugmail commented on this pull request.
In www/c-icap/src/opnsense/mvc/app/models/OPNsense/CICAP/General.xml:

   <maxrequestsperchild type="IntegerField">

       <default>0</default>

<Required>Y</Required>

   <listenaddress type="TextField">

       <default><IP_ADDRESS></default>

       <Required>Y</Required>

   </listenaddress>

   <serveradmin type="TextField">

<EMAIL_ADDRESS>

       <Required>Y</Required>

   </serveradmin>

   <servername type="TextField">

       <default>YourServerName</default>

       <Required>Y</Required>

   </servername>

Oh cool, didn't know this. Is there a list of this? Grepping for system also shows system.authserver for SSO plugin.
But is it also possible to use values such as if helpers.exists('OPNsense.quagga.bgp.enabled') ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

IPv6 can be disabled in the filter as a global block, but it won't block local IPv6 traffic as that prevents squid from starting because it uses an IPv6 health check. It's a good idea to do a default of ::1 here unless the user wants to access this remotely.

On 12. Aug 2017, at 08:59, Fabian Franz<EMAIL_ADDRESS>wrote:
@fabianfrz commented on this pull request.
In www/c-icap/src/opnsense/mvc/app/models/OPNsense/CICAP/General.xml:

   </minsparethreads>

   <maxsparethreads type="IntegerField">

       <default>20</default>

<Required>Y</Required>

   <threadsperchild type="IntegerField">

       <default>10</default>

<Required>Y</Required>

   <maxrequestsperchild type="IntegerField">

       <default>0</default>

<Required>Y</Required>

   <listenaddress type="TextField">

       <default><IP_ADDRESS></default>

       <Required>Y</Required>

I have tried it now - only a single address is supported.
So I would prefer a default of ::1 (localhost).
IPv4 should not be the default for anything since 20 years of IPv6 and there should not be any issues with IPv4 only networks as it will probably not leave the machine. Maybe this is just my preference to avoid IPv4 whenever it is possible.
Anyway you should use this field: https://github.com/opnsense/core/blob/2dcb24c5f855059a547a93fd229300679b9f944b/src/opnsense/mvc/app/models/OPNsense/Base/FieldTypes/NetworkField.php
@fichtner Is it even possible to disable IPv6? I think it's not.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

What Fabian meant was you just need configdRun('firmware plugin clamav') from the controller. It's already defined in the core so you can use it without a definition.

On 12. Aug 2017, at 07:18, Michael<EMAIL_ADDRESS>wrote:
@mimugmail commented on this pull request.
In www/c-icap/src/opnsense/service/conf/actions.d/actions_cicap.conf:

+command:/usr/local/opnsense/scripts/OPNsense/CICAP/setup.sh;/usr/local/etc/rc.d/c-icap restart
+parameters:
+type:script
+message:reconfigure c-icap

+[status]
+command:/usr/local/etc/rc.d/c-icap status;exit 0
+parameters:
+type:script_output
+message:request c-icap status
+
+[checkclamav]
+command:/usr/local/sbin/configctl firmware plugin clamav
+parameters:
+type:script
+message:check if ClamAV is installed
@fabianfrz You mean with parameters % and call via parameters via Controller?
@fichtner I asked Ad for an easy way, but I have no idea to build a hide function via API :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@fichtner Is it also ok to "just" display a warning when clamav plugin is missing?
@fabianfrz Is the code style ok?
https://github.com/mimugmail/plugins/blob/c-icap/www/c-icap/src/opnsense/mvc/app/views/OPNsense/CICAP/general.volt#L70-L80

I think I'm done with all requested changes.
If the Warning implemented today is ok for you this can be merged from my side.

@mimugmail I think it would be better if you would use a bootstrap alert (https://getbootstrap.com/docs/4.0/components/alerts/) and show it if there is a problem $('#idofelement').show() in the callback. You just have to set the style to display:none;

@fabianfrz You mean like this: https://github.com/opnsense/plugins/blob/master/security/clamav/src/opnsense/mvc/app/views/OPNsense/ClamAV/general.volt#L30-L33

yes, exactly this. the advantage is, that you don't have to click it away so it's better from the view of usability.

Ok please check:
https://github.com/opnsense/plugins/pull/226/commits/8ca3881d5ae460a787fafd8e8c3bb37180929900
I'll remove WIP since the end is near :)

\o/

Renamed the .inc to get displayed in Services.
Remark to @fichtner change +x to setup.sh for c-icap and clamav plugin after merge.
Thanks!

@mimugmail @fichtner should this be merged?

For me it's ok to merge
Am 16.08.2017 10:11 nachm. schrieb "Fabian Franz"<EMAIL_ADDRESS>
:

@mimugmail https://github.com/mimugmail @fichtner
https://github.com/fichtner should this be merged?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/opnsense/plugins/pull/226#issuecomment-322884804, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ATfeoT3QuduxGKVzsB8KhVpv19_HK7T6ks5sY0zcgaJpZM4O0cs9
.

@mimugmail I did the chmod for you in: https://github.com/opnsense/plugins/commit/3c51bbd2be96089dc1f6a7f34a34536f9fc46449

@fichtner I merged this branch as your review was already green and no relevant changes since but just a question: does this plugin have to provide a log viewer?

@fabianfrz Thanks, errors at reboot are gone.

Please let assignee merge, or ask for assignment, I was just waiting for your review to go green ;)

On 17. Aug 2017, at 07:04, Michael<EMAIL_ADDRESS>wrote:
@fabianfrz Thanks, errors at reboot are gone.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@fichtner ok, will not merge your PR next time (it looked like it has been forgotten to me as there was no hint what was missing).


【𝕏𝚆𝙰𝚃𝙲𝙷:⊱ᐯIᗪEO𝚂】Sona Dey Viral Exclusive ᴠɪᴅᴇᴏ ʟᴇᴀᴋᴇᴅ 𝙻𝚒𝚗𝚔 On Social Media 𝕏𝘛𝘸𝘪𝘵𝘵𝘦𝘳 【ＸＸＸ KHJ87B】 #23

24 sec ago Equatorial Guinea ʟᴇᴀᴋᴇᴅ ᴠɪᴅᴇᴏ Viral On Social Media.【𝕏𝚆𝙰𝚃𝙲𝙷:⊱ᐯIᗪEO𝚂】Equatorial Guinea Viral Exclusive ᴠɪᴅᴇᴏ ʟᴇᴀᴋᴇᴅ 𝙻𝚒𝚗𝚔 On Social Media 𝕏𝘛𝘸𝘪𝘵𝘵𝘦𝘳 【ＸＸＸ HKJ7B8】
𝑾𝒂𝒕𝒄𝒉 🟢 ➤ ➤ ➤ 🌐 𝖢𝗅𝗂𝖼𝗄 𝖧𝖾𝗋𝖾 𝖳𝗈 𝗅𝗂𝗇𝗄 (𝖥𝗎𝗅 𝗅 𝖵𝗂𝗋𝖺𝗅 𝖵𝗂𝖽𝖾𝗈 𝖫𝗂𝗇𝗄)
🔴 ➤► 𝖣𝖮𝖶𝖭𝖫𝖮𝖠𝖣👉👉 (𝖥𝗎𝗅𝗅 𝖵𝗂𝗋𝖺𝗅 𝖵𝗂𝖽𝖾𝗈 𝖫𝗂𝗇 𝗄) 𝖶𝖺𝗍𝖼𝗁
🟢 ➤ ➤ ➤ 🌐 𝖢𝗅𝗂𝖼𝗄 𝖧𝖾𝗋𝖾 𝖳𝗈 𝗅𝗂𝗇𝗄 (𝖥𝗎𝗅𝗅 𝖵𝗂𝗋𝖺 𝗅 𝖵𝗂𝖽𝖾𝗈 𝖫𝗂𝗇𝗄)
🔴 ➤► 𝖣𝖮𝖶𝖭𝖫𝖮𝖠𝖣👉👉 (𝖥𝗎𝗅𝗅 𝖵𝗂𝗋𝖺𝗅 𝖵𝗂𝖽𝖾𝗈 𝖫𝗂𝗇 𝗄)
Equatorial Guinea's latest video has taken the internet by storm, sparking widespread discussions across various social media platforms. The rising digital influencer, renowned for her captivating content—especially her recent Spiderman-themed video—has seen a significant boost in popularity, with fans eagerly anticipating the full release. Audiences are flocking to Instagram, X (formerly Twitter), and Discord to engage with her newest creation.
As excitement continues to grow, fans are actively sharing their thoughts online, highlighting how vital social media is in helping creators like Equatorial Guinea gain recognition. With her unique style and creative approach, she has made a notable impact on the digital scene, leaving her followers eager to see what she’ll do next.


Question about Delayed Operators

Hi,
Sorry to disturb you again. I might misunderstand the principle behind the delayed operators. In the following program, I tried to simulate a system with two qubits (2-level cavity FockBasis(1)) connected by a waveguide, where the waveguide should have a finite length, causing the traveling time between the qubits to be $\Delta$.
For simplicity, let's assume that the couplings are all constant. With the only initial excitation in qubit 1 (Q1), I wrote the following simulation code:
using QuantumOptics
using WaveguideQED
using LinearAlgebra
using Plots

times = 0:0.1:10
dt = times[2] - times[1]

bw = WaveguideBasis(1, times)
bc = [FockBasis(1), FockBasis(1)] # 2 qubits
Iw = identityoperator(bw)
Ic = identityoperator.(bc)

Δ = 5
wd = create(bw)
wdΔ = create(bw; delay=Δ/dt)
w = destroy(bw)
wΔ = destroy(bw; delay=Δ/dt)
ad = create.(bc)
a = destroy.(bc)

γ = 5
H = im*√(γ/2/dt)*(ad[1]⊗Ic[2]⊗w - a[1]⊗Ic[2]⊗wd) +
    im*√(γ/2/dt)*(Ic[1]⊗ad[2]⊗w - Ic[1]⊗a[2]⊗wd) +
    im*√(γ/2/dt)*(ad[1]⊗Ic[2]⊗wΔ - a[1]⊗Ic[2]⊗wdΔ) +
    im*√(γ/2/dt)*(Ic[1]⊗ad[2]⊗wΔ - Ic[1]⊗a[2]⊗wdΔ)

ncc = (ad[1]*a[1])⊗(ad[2]*a[2])⊗Iw
nc1 = (ad[1]*a[1])⊗Ic[2]⊗Iw
nc2 = Ic[1]⊗(ad[2]*a[2])⊗Iw
nw = Ic[1]⊗Ic[2]⊗(wd*w)
function expval(time, psi)
  (expect(ncc, psi), expect(nc1, psi), expect(nc2, psi), expect_waveguide(nw, psi))
end

ψ = fockstate(bc[1], 1) ⊗ fockstate(bc[2], 0) ⊗ zerophoton(bw)
ψ, ecc, ec1, ec2, ew = waveguide_evolution(times, ψ, H, fout=expval)
plot!(times, abs.(ec1); label="Qubit 1", xlabel="Time", ylabel="Expectation Value")
plot!(times, abs.(ec2); label="Qubit 2")
plot!(times, abs.(ew); label="Waveguide")
savefig("./plot.pdf")

This program yields the following plot:

which is confusing because qubit 2 (Q2) rises at t=0. The photon released by Q1 should travel for time $\Delta$ before getting to Q2. I suspect that the Hamiltonian I wrote is not correct. Would you offer some clues about simulating such a system (2 qubits connected by a waveguide with some length)?
Thank you very much!

Hi,
Very exciting that you are testing out delayed operators! I realize that perhaps the documentation is misleading in the way one should think about the delayed operators. It is kind of counterintuitive, but when you have a delayed operator, it acts in the future, meaning, let's say, 5 timesteps AHEAD. Thus, in your example, you actually want to start the initial state in qubit 2 since this acts in the future (more forward on the conveyor belt), and qubit 1 will then 5 timesteps later see this spot on the conveyor belt. This is not really intuitive and not obvious from the documentation, but I'm currently thinking about how it could be illustrated better.
Another comment is the Hamiltonian. You have too many terms much? The way you write it, both qubits talk with the waveguide at both times. Instead, you should only have qubit one talk with the waveguide at a "not delayed time" and qubit two at the delayed time. Fixing this, I have:
using QuantumOptics
using WaveguideQED
using LinearAlgebra
using Plots

times = 0:0.1:10
dt = times[2] - times[1]

bw = WaveguideBasis(1, times)
bc = [FockBasis(1), FockBasis(1)] # 2 qubits
Iw = identityoperator(bw)
Ic = identityoperator.(bc)

Δ = 5
wd = create(bw)
wdΔ = create(bw; delay=Δ/dt)
w = destroy(bw)
wΔ = destroy(bw; delay=Δ/dt)
ad = create.(bc)
a = destroy.(bc)

γ = 5
H = im*√(γ/2/dt)*(ad[1]⊗Ic[2]⊗w - a[1]⊗Ic[2]⊗wd) +
    im*√(γ/2/dt)*(Ic[1]⊗ad[2]⊗wΔ - Ic[1]⊗a[2]⊗wdΔ)
ncc = (ad[1]*a[1])⊗(ad[2]*a[2])⊗Iw
nc1 = (ad[1]*a[1])⊗Ic[2]⊗Iw
nc2 = Ic[1]⊗(ad[2]*a[2])⊗Iw
nw = Ic[1]⊗Ic[2]⊗(wd*w)
function expval(time, psi)
  (expect(ncc, psi), expect(nc1, psi), expect(nc2, psi), expect_waveguide(nw, psi))
end

ψ = fockstate(bc[1], 0) ⊗ fockstate(bc[2], 1) ⊗ zerophoton(bw)
ψ, ecc, ec1, ec2, ew = waveguide_evolution(times, ψ, H, fout=expval)

Plots.CURRENT_PLOT.nullableplot = nothing
plot!(times, abs.(ec1); label="Qubit 1", xlabel="Time", ylabel="Expectation Value")
plot!(times, abs.(ec2); label="Qubit 2")
plot!(times, abs.(ew); label="Waveguide")

savefig("./plot.png")

which gives me:

Obviously, the above configuration of the operators makes it so that after qubit 1 reemits into the waveguide qubit 2 will not pick up the photon again. For this to happen, one should add another term with an operator that is two times delayed: wd2Δ = create(bw; delay=2 * Δ/dt). But this quickly grows impractical when you then want to consider a third bounce. Instead, it should be possible to code the waveguide to loop around on itself. Would this be interesting?

Thank you for your reply! That makes perfect sense. Yes, I am very interested in all kinds of looping/connecting systems. One further question on the system mentioned above is how to simulate the system with initial excitation in both qubits? Namely, to simulate a symmetric two-photon system (Q1 - W - Q2).
We are particularly interested in this because our ultimate goal is to simulate the full HOM dip setup, i.e. a system like
Q1 - W1 - BS - W2 - Q2
where Q is qubit, W is waveguide, BS is beamsplitter. Ideally, the waveguides should have a finite length. I find it particularly tricky with the delay operators to simulate two photons in one system. Is there any clever method to build a system like this?
Thank you very much!

Alright, super cool! I will look into looping. I almost have it done now, but I will do some testing and see if I can make the time ordering and so on more intuitive.
For your question about two initially excited qubits, I would simply just have a waveguide with a maximum of two photons, e.g., WaveguideBasis(2,times). I just tested this, and it should work. Please let me know if problems arise.
For the final configuration Q1 - W1 - BS - W2 - Q2, I'm guessing that you have a detector after the beamsplitter? So that Q1 and Q2 are inputs on one side of the BS and then you detect? In that case, have you tried something like a waveguide basis with two waveguides and then Q1 coupled to W1 and Q2 to W2, and then via delay operators, you place the beamsplitter Hamiltonian $H_bs = V(w_1^\dagger w_2 + w_1 w_2^\dagger)$  a certain distance from the two qubits? I don't think this is an easy exercise, and I would suggest you draw a conveyor belt and try to visualize how the system evolves as the conveyor belt moves forward. I think in this instance, your solution is simply having H_bs being delayed by $\Delta \tau$ where $\Delta \tau$ is the time it takes the light to travel from Q1 and Q2 to the BS.

Regarding the simple system (Q1 - W - Q2) with two photons, how to write the Hamiltonian? The question is that if we write a symmetric Hamiltonian regarding both qubits to release their photons at t=0, we might write
H = im*√(γ/2/dt)*(ad[1]⊗Ic[2]⊗w - a[1]⊗Ic[2]⊗wd) +
    im*√(γ/2/dt)*(Ic[1]⊗ad[2]⊗w - Ic[1]⊗a[2]⊗wd) +
    im*√(γ/2/dt)*(ad[1]⊗Ic[2]⊗wΔ - a[1]⊗Ic[2]⊗wdΔ) +
    im*√(γ/2/dt)*(Ic[1]⊗ad[2]⊗wΔ - Ic[1]⊗a[2]⊗wdΔ)

then the released photon from Q1 might interact with Q2 immediately without a delay?
Regarding the beamsplitter settings, our physical experiment would capture the photons after the beamsplitter. The resulting qubit states serve as the "detection". I understand that this scenario is more complicated, and maybe I should first understand the simple system discussed above.
Thank you very much!

Ahh, now I see the confusion. The Hamiltonian should not change, but instead, the waveguides should loop. I will try to address this case of two photons in an update sometime next week, where I introduce the looping (which is necessary). Hopefully, this will help.
As for the beamsplitter experiment, I think this could already be implemented now using a waveguide basis with two waveguides, by having the BS Hamiltonian be delayed and the two qubits interacting with W1 and W2, respectively.

I have just released an update 0.2.6, where I have implemented the above mentioned looping. I suggest you take a look at the example I provide in the documentation: https://qojulia.github.io/WaveguideQED.jl/dev/time_delay/#emitters
Please let me know if you have any questions.

That's great! The QWQ model works completely fine. Thank you very much!
Regarding the Q1 - W1 - BS - W2 - Q2 model, there might be a glitch(?). The following code will raise error when solving the waveguide evolution:
dt = 0.1
times = 0:dt:10+dt
times_sim = 0:dt:20
bw = WaveguideBasis(2, 2, times)
bc = [FockBasis(2), FockBasis(2)]

Δ1 = 5
Δ2 = 5
Iw, Ic = identityoperator(bw), identityoperator.(bc)
wd = [create(bw, 1), create(bw, 2)]
wdΔ = [create(bw, 1; delay=Δ1/dt+1), create(bw, 2; delay=Δ2/dt+1)]
w = [destroy(bw, 1), destroy(bw, 2)]
wΔ = [destroy(bw, 1; delay=Δ1/dt+1), destroy(bw, 2; delay=Δ2/dt+1)]
ad = create.(bc)
a = destroy.(bc)

n = 1/2
γ = 5
H = im*√(γ/2/dt)*(ad[1]⊗Ic[2]⊗wΔ[1] - a[1]⊗Ic[2]⊗wdΔ[1]) +
    im*√(γ/2/dt)*(Ic[1]⊗ad[2]⊗wΔ[2] - Ic[1]⊗a[2]⊗wdΔ[2]) +
    asin(√n)/dt*(Ic[1]⊗Ic[2]⊗(wd[1]*w[2]) + Ic[1]⊗Ic[2]⊗(wd[2]*w[1]))

ncc = (ad[1]*a[1])⊗(ad[2]*a[2])⊗Iw
nc1 = (ad[1]*a[1])⊗Ic[2]⊗Iw
nc2 = Ic[1]⊗(ad[2]*a[2])⊗Iw
nw1 = Ic[1]⊗Ic[2]⊗(wd[1]*w[1])
nw2 = Ic[1]⊗Ic[2]⊗(wd[2]*w[2])
function expval(time, psi)
  expect(ncc, psi), expect(nc1, psi), expect(nc2, psi), expect_waveguide(nw1, psi), expect_waveguide(nw2, psi)
end

ψ = fockstate(bc[1], 1) ⊗ fockstate(bc[2], 0) ⊗ zerophoton(bw)
ψ, ecc, ec1, ec2, ew1, ew2 = waveguide_evolution(times_sim, ψ, H, fout=expval)

This raises error:
ERROR: LoadError: BoundsError: attempt to access 21115-element view(::UnsafeArrays.UnsafeArray{ComplexF64, 1}, 1:9:190027) with eltype ComplexF64 at index [21116]

This error only appears when times_sim exceeds the range of times, so I believe that it is related to waveguide loop. I am not sure where the problem is.
Thank you very much!

Hi,
Yes, I'm sorry, I didn't include looping in waveguide interaction operators. I have now fixed this in the update 0.2.7, and the above should work. Below, I also plot the evolution of the above system, where I included the coincidence counts in the two waveguides. It is clear that after the beamsplitter, there are no coincidence counts; thus, there is a full HOM dip. Note that calculating the coincidence counts is quite expensive to compute, and thus, calculating it at each step makes the code take quite a while to run. In reality, just computing the coincidence counts in the end is fine, and this should not be too expensive.
using WaveguideQED
using QuantumOptics

dt = 0.1
times = 0:dt:10+dt
times_sim = 0:dt:10
bw = WaveguideBasis(2, 2, times)
bc = [FockBasis(2), FockBasis(2)]

Δ1 = 2
Δ2 = 2
Iw, Ic = identityoperator(bw), identityoperator.(bc)
wd = [create(bw, 1), create(bw, 2)]
wdΔ = [create(bw, 1; delay=Δ1/dt+1), create(bw, 2; delay=Δ2/dt+1)]
w = [destroy(bw, 1), destroy(bw, 2)]
wΔ = [destroy(bw, 1; delay=Δ1/dt+1), destroy(bw, 2; delay=Δ2/dt+1)]
ad = create.(bc)
a = destroy.(bc)

n = 1/2
γ = 5
H = im*√(γ/2/dt)*(ad[1]⊗Ic[2]⊗wΔ[1] - a[1]⊗Ic[2]⊗wdΔ[1]) +
    im*√(γ/2/dt)*(Ic[1]⊗ad[2]⊗wΔ[2] - Ic[1]⊗a[2]⊗wdΔ[2]) +
    pi/4/dt*(Ic[1]⊗Ic[2]⊗(wd[1]*w[2]) + Ic[1]⊗Ic[2]⊗(wd[2]*w[1]))

ψ = fockstate(bc[1], 1) ⊗ fockstate(bc[2], 1) ⊗ zerophoton(bw)

using LinearAlgebra
psi_c = copy(ψ)
function expect_coincidence(n1, n2, psi, times)
  expval = 0
  for i in eachindex(times)
    set_waveguidetimeindex!(n1, i)
    for j in eachindex(times)
      set_waveguidetimeindex!(n2, j)
      QuantumOpticsBase.mul!(psi_c, n1*n2, psi)
      expval += dot(psi_c.data, psi.data)
    end
  end
  return expval
end

ncc = (ad[1]*a[1])⊗(ad[2]*a[2])⊗Iw
nc1 = (ad[1]*a[1])⊗Ic[2]⊗Iw
nc2 = Ic[1]⊗(ad[2]*a[2])⊗Iw
nw1 = Ic[1]⊗Ic[2]⊗(wd[1]*w[1])
nw2 = Ic[1]⊗Ic[2]⊗(wd[2]*w[2])
function expval(time, psi)
  expect(ncc, psi), expect(nc1, psi), expect(nc2, psi), expect_waveguide(nw1, psi), expect_waveguide(nw2, psi),expect_coincidence(nw1,nw2,psi,times)
end
#ψ, ecc, ec1, ec2, ew1, ew2,coincedence = waveguide_evolution(times_sim, ψ, H, fout=expval)
psi = waveguide_evolution(times_sim, ψ, H, fout=1)

out = zeros(ComplexF64,6, length(psi))
for (i,p) in enumerate(psi)
  out[1,i] = expect(ncc, p)
  out[2,i] = expect(nc1, p)
  out[3,i] = expect(nc2, p)
  out[4,i] = expect_waveguide(nw1, p)
  out[5,i] = expect_waveguide(nw2, p)
  out[6,i] = expect_coincidence(nw1,nw2,p,times)
end

using PyPlot
pygui(true)
fig,ax = subplots(1,2,figsize=(9,4))
ax[1].plot(times, out[2,:], label="Q1")
ax[1].plot(times, out[3,:], label="Q2")

ax[2].plot(times, out[4,:], label="W1")
ax[2].plot(times, out[5,:], label="W2")
ax[2].plot(times, out[6,:], label="Coincidence")
ax[2].legend()

Giving the plot:

Note that it might be possible to just simulate the two qubits emitting their photons in one simulation and then take the output state of the first simulation and apply the beamsplitter operation in a subsequent simulation.


[freeglut] static linking is not working

Machine: Windows 10 Creators Update, fully up to date, vcpkg at the latest commit, VS 15.3.2 (very latest release)
I installed freeglut (both dynamic and static) and in fact if I do a .\vcpkg.exe list I have
...
freeglut[core]:x86-windows     3.0.0-1          Open source implementation of GLUT with source and binary backwards ...
freeglut[core]:x86-windows-... 3.0.0-1          Open source implementation of GLUT with source and binary backwards ...
...

Using CMake, if I ask for
find_package (GLUT REQUIRED)

using the static x86 triplet
cmake .. -G "Visual Studio 15" "-DCMAKE_TOOLCHAIN_FILE=$env:WORKSPACE\vcpkg\scripts\buildsystems\vcpkg.cmake" "-DVCPKG_TARGET_TRIPLET=x86-windows-static"

it fails saying that it cannot find the GLUT_glut_LIBRARY, while it works ok for the dynamic linking.
I think that the official CMake FindGLUT.cmake module is looking for libraries just called freeglut also for static (there are variants, on StackOverflow for example, which look for both names adding freeglut_static, but I'd prefer to use the official module of course). If you think that renaming is a good option, I can provide a PR with the proper fix in place for the static library.

closed with #2154


build(deps): bump github.com/spf13/afero from 1.4.1 to 1.5.1

Signed-off-by: Robert-André Mauchin<EMAIL_ADDRESS>

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

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


Fix Travis OSX builds

A change in Homebrew is breaking OSX builds.
This fix handles the problems we are now having
installing GMP. Specifically, we are hitting errors
were it can't be linked into /usr/local when
being installed as a dependency for various
llvm version. This ends up failing the llvm installs
and from that, the build.
This fixes by "manually" installing gmp and then
forcing the link overwrite.

I merged this early before the windows version passed because I knew that would pass.


406 Not Acceptable when requesting access token

The request from readme seems to not work anymore. It returns 406 Not Acceptable.
However you can change the user-agent to iOS 13 instead of 11 and it will work again:
curl -X "POST" "https://home.nest.com/session" \
     -H 'User-Agent: iPhone iPhone OS 13.0 Dropcam/5.14.0 com.nestlabs.jasper.release Darwin' \
     -H 'Content-Type: application/x-www-form-urlencoded; charset=utf-8' \
     --data-urlencode "email=YOUR_NEST_EMAIL" \
     --data-urlencode "password=YOUR_PASSWORD"

Please update the readme file.

Changer user agent by something else like Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/41.0. 2272.118 Safari/537.36.

Thanks, that worked for me.

I was able to just go to https://home.nest.com/session in a browser and get my access token. This assumes that you're currently logged into https://home.nest.com/.
Do note:
I have 2FA enabled.
I did have Nest-Cam working prior to my access_token expiring when I noticed that my Nest Cam's weren't refreshing in HomeKit today.

^ Same here. I went to the /session page and got the token, which is massive. I DO have 2FA enabled.

curl wasn't working for me via the README or by using another user agent.
I was able to just go to https://home.nest.com/session in a browser and get my access token. This assumes that you're currently logged into https://home.nest.com/.
Do note:
I have 2FA enabled.
I did have Nest-Cam working prior to my access_token expiring when I noticed that my Nest Cam's weren't refreshing in HomeKit today.

This needs to be in the README


add createConnectHook factory.

What is the new or updated feature that you are suggesting?
React-Redux has provided createSelectorHook and createStoreHook, and what I want is createConnectHook.
Why should this feature be included?
When I provide my own ReactReduxContext, I have to inject to connect every time I use it:
export default connect(_, _, _, { context: MyOwnReactReduxContext })(App);

Maybe createConnectHook can help me
export const myConnect = createConnectHook({ context: MyOwnReactReduxContext, store: MyStore });

And I can import it from every files:
import { myConnect } from './myConnect';
export default myConnect(mapState)(App);

What docs changes are needed to explain this?
No.

Sorry, but we have no plans to add any further features around connect. It's there and is supported as-is, but we're not adding anything more to it.


Windows support?

I'm not quite fond with ubuntu and/or lua, does it work with windows? If yes, how?

https://github.com/nagadomi/waifu2x/issues/2

@WL-Amigo released waifu2x-converter-cpp.
waifu2x-converter-cpp supports windows and without GPU.
https://github.com/WL-Amigo/waifu2x-converter-cpp
https://github.com/WL-Amigo/waifu2x-converter-cpp/releases


Keep the check for rule AWS021 consistent with the error message

The error message for AWS021 says -
"You should not use outdated/insecure TLS versions for
encryption. You should be using TLS v1.2+."
The rule is now consistent with this version and will not
complain if you afix a version higher than what tfsec prescribes.
https://docs.aws.amazon.com/AmazonCloudFront/latest/DeveloperGuide/secure-connections-supported-viewer-protocols-ciphers.html#secure-connections-supported-ciphers

I don't think we'll need this as the next iteration is likely to be a TLS 1.3 policy rather than 1.2 2022 etc.


Build failed: Unrecognized VM option 'UseCGroupMemoryLimitForHeap'

Orb version: 5.6.0
What happened
The build failed with the following error (in "Downloading Gradle Dependencies"):
Picked up _JAVA_OPTIONS: -Xmx2g -XX:+UnlockExperimentalVMOptions -XX:+UseCGroupMemoryLimitForHeap
Unrecognized VM option 'UseCGroupMemoryLimitForHeap'
Error: Could not create the Java Virtual Machine.
Error: A fatal exception has occurred. Program will exit.

Exited with code exit status 1

I noticed this orb uses a non-pinned image of react-native-community/react-native, and that image was updated last week to use JDK 11 which completely removes the UseCGroupMemoryLimitForHeap option.
I'm not that familiar with JVM options so I just changed config.yml to pass java_options without that flag and the build succeeded. Moving forward I found this article which might help decide with the defaults should be.

I noticed this orb uses a non-pinned image of react-native-community/react-native

I believe that's an issue and should be fixed.

and that image was updated last week to use JDK 11 which completely removes the UseCGroupMemoryLimitForHeap option.

True. If running on JDK11 the UseCGroupMemoryLimitForHeap option should be removed.
I think this orb should be update to:

Release a version (say 5.7.0) which relies on a pinned version of the docker container that runs on JDK8 (anything on 4.x).
Release another version (say 6.0.0) which relies on a pinned version of the docker container that runs on JDK11 (anything on 5.x).

In the meantime, we're dead in the water because of this line :-)
https://github.com/react-native-community/react-native-circleci-orb/blob/1b12f2a14a3bc7a5dba2ea7f340508b9d6b4cb2f/src/executors/linux_android.yml?#L5
I liked your suggestion @cortinico and I lined up two PRs #121 and #122 to implement it. Until they are done our CI is broken and we will have trouble validating releases, for example:
https://github.com/react-native-share/react-native-share/pull/1124/checks?check_run_id=4251523347

closed via #122. Please open a new issue if the problem persists :)

v6.0.1 of this orb is working correctly - comment with links to proof https://github.com/react-native-community/react-native-circleci-orb/pull/123#issuecomment-974095135


Multiple tests fail for lego_test.py, on Python 2

On Python 2.7, 24 of the 95 tests in lego_test.py fail.  This includes:

Infinite recursion for test_binary_3, test_base_N, test_everythingbut, test_isinstance_bug, and others (maybe related to issue #32 ?)
Assertion failures for test_conc_common, test_pattern_commonconc, test_pattern_commonconc_suffix, test_conc_dock, and others
Exception raised for test_mult_doc and others

On Python 3.5, all the tests pass.  I'm using greenery version 3.0, as installed by Pip.

Same response as for #32: I'm sorry, but unfortunately Python 2.x is not supported. I will rectify the README to signal this now.

README updated.


Add a common switcher for alternatives in the documentation

First check

[x] I added a very descriptive title to this issue.
[x] I used the GitHub search to find a similar issue and didn't find it.
[x] I searched the FastAPI documentation, with the integrated search.
[x] I already searched in Google "How to X in FastAPI" and didn't find any information.
[x] I already read and followed all the tutorial in the docs and didn't find an answer.
[x] I already checked if it is not related to FastAPI but to Pydantic.
[x] I already checked if it is not related to FastAPI but to Swagger UI.
[x] I already checked if it is not related to FastAPI but to ReDoc.
[x] After submitting this, I commit to one of:

Read open issues with questions until I find 2 issues where I can help someone and add a comment to help there.
I already hit the "watch" button in this repository to receive notifications and I commit to help at least 2 people that ask questions in the future.
Implement a Pull Request for a confirmed bug.

Description
In the documentation, there are pages which suggest two alternatives for a problem. Like here, it suggests either Uvicorn or Hypervisor to deploy the code. However, in the multiple instances of the examples, the switching as to be done manually for each one of them.
Ideally, could it be shifted to something where switching in one example switches all the snippets to that option. Like this maybe where switching between Kotlin and Java for one switches it for all
Environment
Not relevant

I did not understand the problem here, in the example you sent you are also switching the snippet manually. Can you elaborate this?

I believe he means that if you switch to 'hypercorn' all the other optional code samples should also switch.
Aka I shouldn't be able to do this:

Yes, that is what I meant. Thank you @ArcLightSlavik


Sort tests by cpu estimate

Built on #12419

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences


refinementList: limit should be 1000 by default

Current refinementList limit is always 100 which will lead to incomprehension, we should show all the facet values by default since the limit is already in the API server: 1000
So basically if no limit is passed there's nothing to do we just use the index settings. If a limit is passed then we limit it programmatically after receiving the data.

Hmmm, I would rather go for 10 :) It doesn't make sense to display too much values. Also, what could be cool would be to set the maxValuesPerFacet query parameter to max(all limit options).

It doesn't make sense to display too much values.

agree but I prefer the user to see that there's a lot of values and he then has to limit it if he wants to. Rather than having himself thinking there's a bug somewhere because he cannot see all the values.

Yes OK, that makes sense.

Hi,
I guess I'm experiencing an issue with this, take a look at the screenshots please:

As you can see I have the list of country_iso as my facet values, and there are around 200 countries, so I set this limit, but all I can see both in the dashboard plus on the UI side (with Vue InstantSearch lib) is 100 values.
Did I miss something?


Add pixels for autofill

Task/Issue URL: https://app.asana.com/0/0/1203013158998989/f
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

Current dependencies on/for this PR:

develop

PR #2317   👈

This comment was auto-generated by Graphite.


Drag handle

Is your feature request related to a problem? Please describe

I am implementing a list of draggable items and for responsive purposes, I would like to have just an Icon that is triggering the drag, but the drag & drop visual animation to remain on the draggable item.

In responsive design, in a list, you must be able to scroll freely without triggering drag &drop, the most common UX solution is the "drag handle" on a specific area inside the draggable item

Additional context

This sounds reasonable, I'd be happy to accept a PR that adds this functionality. You could copy the implementation from here


Add support for TIME and BYTES query parameters

BigQuery has both TIME and BYTES data types, but the current query parameter implementation does not yet support them.

Supporting BYTES should be fairly easy. We support bytes on other libraries and we can check for an IO-ish object. So users can provide a File or StringIO object containing the bytes they want written.
Supporting TIME is not going to be as easy. Ruby doesn't have an object that only represents time. So how will users format an object to indicate that the value should be a TIME type? What about using a Time object that is less than 24 hours from the epoch? Would that work?

Another option is to use Ruby's DateTime for BigQuery's TIMESTAMP, Ruby's Time for BigQuery's TIME, and Ruby's Date for BigQuery's DATE.

@quartzmo Do you have an opinion on this? Thoughts?

@blowmage What about BigQuery's DATETIME type? Also Ruby's Time, correct?
I like this solution. Isn't this how Rails maps these types?

Ugh, I'm blind. I was going off a different set of types and didn't see those until I started looking for documentation. I agree it would be best to keep Ruby's DateTime for BigQuery's DATETIME, and Ruby's Time for BigQuery's TIMESTAMP. But then we are back to the original question of how do we represent BigQuery's TIME?

Supporting TIME is looking more and more difficult. I'm starting to think we will need to add a new object to represent it. Perhaps a Google::Cloud::Bigquery::Time class or struct, and a Project#time helper to make creating a TIME easier.
require "google/cloud/bigquery"

bigquery = Google::Cloud::Bigquery.new

data = bigquery.query "SELECT * FROM tbl WHERE time_of_day > @time},
                      params: { time: bigquery.time(Time.now) }

Thoughts?

Rails uses Ruby's Time for TIME because some databases can store TIME values larger than 24 hours, so it makes sorta sense to use Ruby's Time for that. But Ruby's Time is really a TIMESTAMP, so I think it best we allow it to represent that and create something new for TIME.

We might also need a second pass on the documentation to make sure users understand how values passed in for query parameters will be converted to BigQuery data types.

@blowmage I also thought of a new object, and this seems OK to me given the situation.


Add Elasticsearch Log4Shell Mitigation

This patch adds the mitigation for CVE-2021-44228 to the default
Elasticsearch configuration in the docker-compose files.

Pushed this also to the 10.6 and 9.10 branches.

Makes sense. Didn't think about that yesterday. Thanks


CUJ Testing for Beta v2 - Creator can Add a Page Attachment

Feature Description
In preparation for the Beta 2 release we would like to have validation and confirmation of the CUJ functionality detailed in
https://github.com/google/web-stories-wp/issues/1626

[ ] Add page attachment
[ ]  Delete page attachment

Not in scope for Beta 2 release


Render Issues in Bootstrap Modal

Hello,
I need to render two bootstrap-selects in bootstrap modal, but can't get it working properly.
Here's jsFiddle with the code that in my environment actually works on the first load/button Add click (cannot reproduce correct load in fiddle), but fails on the second Add button click. Fiddle shows the issue right away, on the first click.
The second fiddle shows one of tested approaches that fails at the first attempt in my environment.
There're two reproducible issues in total with modal render I've encountered:

Dropdown-toggle part is always rendered. This is the best scenario, as still possible to select options and submit data.
Or options are not selectable, as if refresh wasn't called.

Would appreciate any help from your side.

Use a class other than selectpicker so it's not initiated via the data-api.
<button data-action="open-modal-link-items" class="btn btn-success" type="button">
        Add
    </button>

        Sale Item

      <select data-live-search="true" name="LinkedItems[mbo_id]" class="select form-control show-menu-arrow show-tick" id="linkeditems-mbo_id" tabindex="-98">
          <option value="">Nothing Selected</option>
          <option value="1064">3 Pack Boxing Hand Wraps</option>
          <option value="509056">500ml Pop</option>
          <option value="509057">500ml water</option>
      </select>

      Infusionsoft Product
      <select name="LinkedItems[inf_id]" class="select form-control show-menu-arrow show-tick" id="linkeditems-inf_id" tabindex="-98">
        <option value="">Nothing Selected</option>
        <option value="18">$1 Trial</option>
        <option value="53">1C-TEST</option>
        <option value="107">5ML21</option>
        <option value="123">ACME Bird Seed</option>
        <option value="25">ACME Rocket Fuel</option>
    </select>

    function refreshBootstrapSelect() {
        $('div.bootbox-body .select').selectpicker();
    }
    $('body').on('click', '[data-action="open-modal-link-items"]', function (e) {
        e.preventDefault();
        var bootboxModal = bootbox.dialog({
            message: $("#link-items-select").html(),
            title: 'Link Items',
            buttons: {
                "Submit": {
                    className: "btn-default",
                    callback: function () {
                        var $modalElem = $('div.modal-dialog'), $dataElem, sendData, url;
                        return false;
                    }
                }
            },
            show: false
        });
        bootboxModal.on('shown.bs.modal', function () {
            refreshBootstrapSelect();
        });
        bootboxModal.modal("show");
    });

Or put the html directly into message instead of using .html().

Thank you for the response.
I had to make both updates to make it work on the second modal open (the second Add button click).
Working example: https://jsfiddle.net/yugene/zh0v1k04/

thank you very much

this issue does'n work for me，and then I choose Select2，there is a good solution. solution


Fix "null with the vertical line" issue, globally

Example URL-
https://www.moleculardevices.com/en/assets/tutorials-videos/dd/img/fast-and-simple-3d-spheroid-analysis-using-the-automated-high-content-imaging
https://www.moleculardevices.com/en/assets/tutorials-videos/dd/img/automation-high-content-imaging-for-organoid-research

@MeghnaTewari - have you found this issue only on tutorials-videos pages or other sections as well?

@duynguyen  Till now I have only seen this issue on tutorial videos resource pages where the banner image is coming. I am checking other pages as well and I will update if I see it on other pages as well.

Sure @duynguyen Sending you the list.

I have fixed the 2 pages you reported originally.
The fix is straightforward, you can also do it yourself in Sharepoint:

remove the table column that contains the null
preview the document, make sure the updated page looks good
publish the page

Thanks, @duynguyen Will do.
Another issue that I have noticed is that on these pages the title font size is very large. Should be 36 px and in blue color.
How can we fix this globally? There are around 35 pages.
https://www.moleculardevices.com/en/assets/tutorials-videos/dd/img/fast-and-simple-3d-spheroid-analysis-using-the-automated-high-content-imaging

@MeghnaTewari so all tutorial videos pages should have smaller and blue title? or only a subset of them?

@duynguyen  No, not all of them. This styling is only implemented for video resources for landing pages.
Like this one-
https://www.moleculardevices.com/en/assets/tutorials-videos/dd/img/fast-and-simple-3d-spheroid-analysis-using-the-automated-high-content-imaging
To distinguish such resources, they have a wave and a banner image, so on those pages, the body title should be 36 px and blue. Also the label "ON-DEMAND WEBINAR" should be blue.
For video resource pages that are created for tagging (having the default blue banner), no changes on those.
Example - https://www.moleculardevices.com/en/assets/tutorials-videos/bpd/colony-picking-automation

@MeghnaTewari is it all clear to you in this issue (null column), and you can update the content yourself? closing it here, please lmk if you have more questions.

Yes, @duynguyen. We have fixed this issue on all the pages. Thanks.


Couple of bugs

I have seen one thing broken: Marketing samples.
And also when clicking any "View HTML" button it add some extra characters at the end of the textarea like ">, sometimes </header> "> (without an opening tag <header>), etc.
Cheers!

thanks @hhkaos  - this is fixed in #672 and deployed. Turns out I was doing those sample blocks very, very wrong.

hahaha, great, but it is not fixed in the gh-pages branch, right?

I mean, the carousel and the image switcher:

Oh! Those aren't broken per se – rather they just don't really work yet. Another team is bringing those to completion before we move them in for real.

hahah, okok 👍 , thx


build/docs: Updated ttypescript version

Tested with a fresh environment on node 18.17.1. As was mentioned in #5764 , I also added version numbers to the recommended global packages in CONTRIBUTING (the same ones as in package.json), and version numbers to the README for nodejs and npm (current LTS).
Apologies if I used the wrong label for this PR.

Also also, thanks for testing this and the update! Glad this was an easy fix.

Of course!
I removed the version numbers from contributing, and changed the readme to not reference a specific version but rather LTS as a whole. Hopefully that's a bit more reasonable.

Thank you!!


I will not be accepting New Feature PRs until New Design is complete

rip

f

oof

F

F

When will the new design be completed? Can we help?

@benawad When will the new design be completed? Can we help?

Ben was saying that by this week there should be a sub-domain up serving the new design.
So ig it will fully be released in 1-2 weeks

Can we contribute?

Contribute to the new design?

Guys will the new design be in a new branch or the main branch ?

@KittyX29 the new design is already there just not being used. Look at kibbeh

Is it possible to run kibbeh locally?

Is there public access to the figma files for the new design?

Yes! Ask @overlisted, I don't have them on me right now.
On Tue, Mar 23, 2021 at 11:32 AM Raz Teve @.***> wrote:

Is there public access to the figma files for the new design?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/benawad/dogehouse/issues/1462#issuecomment-805048572,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQ6MKIJBQC3MAVPJAHZKRGTTFC7CDANCNFSM4ZR6HM4A
.

--
Samar Mohan
Two things are infinite: the universe and human stupidity; and I'm not
sure about the universe.                             **     ~Albert
Einstein

Is there any update regarding the new design? or any kind of E.T.A? I'm just really pumped for the new design :)

@FrostyTheDumDum ETA 1-2 weeks

https://www.youtube.com/watch?v=I2j4w4xYXtM


No error lens for remote files (when running as UI extension)

run this extension with the UI (remote.extensionKind-property)
connect to a remote extension host, e.g docker, and run the css language service in there
cause a squiggle to appear
:bug: no error lenses

The reason are these scheme-checks https://github.com/usernamehw/vscode-error-lens/blob/e9a57bc6f4407a483e4eda0764ea851c9e548692/src/extension.ts#L129 which aren't needed and artificially limit the functionality of this cool extension. That fact that an editor shows a document should be proof enough that the user is interested in error lens and a hardcoded list of schemes to filter is bad.

I'll remove them in the next version then.

Thanks!

@jrieken Wait, is this the only line that has to be changed?
https://github.com/usernamehw/vscode-error-lens/blob/e9a57bc6f4407a483e4eda0764ea851c9e548692/src/extension.ts#L129
Or package.json needs to have "extensionKind": "ui",?

Yeah, only those checks need to change. You shouldn't fiddle with the extensionKind, that was just me trying out things and wondering how I broke error-lens...

k then. Hope this fixes the issue. YOLO, published 2.7.0.

works!


In student_list_partial code that generates the status should be in a tag

In student_list_partial.html there is code in the tbody for loop that generates HTML for the student status. All of this would be better if it moved into the backend (tag) rather than the logic in the template. We could also use a method tag to calculate a numeric average.

I'm implementing this differently in #110


Trouble building portable Linux build of PhantomJS 2.0.0

Built from source on a CentOS 7.0 box and the resulting binary breaks on a RHEL 5.9 box with error
libicudata.so.50: cannot open shared object file: No such file or directory
Is there any way to build the resulting binary in way that is compatible across machines?

This is a known issue. We're working on a fix. See here #12948


Create base class for Task Manager

The Task Manager should become specific for RP. This will let us expand to other RTS

Closing in favor of https://github.com/radical-cybertools/radical.entk/issues/211


Batch RTL runs to ensure proper draw order

Consecutive RTL GlyphRuns are drawn from the last to the first.
References
#538, #7149 , all those issues asking for RTL closed as dupes.
PR Checklist

[ ] Closes #xxx
[x] CLA signed. If not, go over here and sign the CLA
[X] Manual test.
[X] I've discussed this with core contributors already. See description below.

Detailed Description of the Pull Request / Additional comments
As @miniksa suggested in a comment on #7149 -- handle the thingy on the render side.
If we have GlyphRuns abcdEFGh, where EFG are RTL, we draw them now in order abcdGFEh.
This has ransom-noting, because I didn't touch the font scaling at all. This should fix the majority of RTL issues, except it doesn't fix issues with colors, because those get split in the TextBuffer phase in the renderer I think, so they show up separately by the GlyphRun phase.
Validation Steps Performed

except it doesn't fix issues with colors, because those get split in the TextBuffer phase in the renderer I think, so they show up separately by the GlyphRun phase.

That's completely true. And I've got a mental backlog item to make it so every line in the DxRenderer is collected up completely (including all the colors) into a single CustomTextLayout prior to drawing the whole line at once. If I prioritized that, it would fix it, I believe, in combination with this PR.

Updated title and reduced description body to read more easily in git log.

The prior behavior is just capricious. Adding would be a new flag (I think in effect, split every Hebrew char into its own GlyphRun). But this isn't breaking any sort of prior behavior that worked.

Or that flag could just zero out bidiLevel

@miniksa I made a small change to the commit message (from last to first) since I realized that my earlier one was too confusing.

@DHowett for this one specifically, batching the RTL runs together is behavior that the default IDWriteTextLayout would have been doing for us if I didn't roll the CustomTextLayout. Having them be drawn as a unit so the origins are offset is correct. We wouldn't flag this specific PR on and off.
We would do, as @schorrm says, something like zeroing/ignoring all bidiLevel flags for a guest-controlled RTL scenario.

Okay. I don't want to ship 1.3 without a flag that gives the users an escape hatch (config flag) to force LTR when they're using a guest application that tries to handle RTL on its own (by assuming LTR.) It doesn't have to be in this PR, but if it regresses vim -H without the ability to opt into a behavior that works with vim -H by the time we ship, we'll revert it and put it back on the table for 1.4. :smile:

Does this replace #7149, require it or aurmgne tit?

So I would say it actually doesn't regress vim -H, barring an incredibly bizarre font fallback accident to make it work.
Re #7149: in terms of fixing RTL this replaces it. However, 7149 may be worthwhile to adapt into a random note fix on its own.

We didn't support ConHost behavior.  Sometimes.

If you can tell me how I can get a flag from the JSON, I'm happy to add this behavior, but I think it makes sense to treat this separately.

Okay, take a look at the last. That's not on me, that's a runtime error in the CI I think.

You can see it passed x86 which was the one it was failing, and then it failed x64 because of memory starvation while compiling something unrelated.

That's nice. It recovered the memory issues. Do I have to do the spellcheck thingy?

I’d like that :)

I did the spelling thingy.

(I'm consciously leaving this for @DHowett to hit the merge button, since he was the last one with concerns about it)

Thanks @schorrm!


why sentPacketHistory cached skipped packet numbers

sentPacketHistory cached skipped packet numbers when packet number skipping.
But it looks like haven't used them.
All sentPacketHistory.Iterate has just ignored skipped packet numbers.
Am I missing something?

Did you see https://github.com/lucas-clemente/quic-go/blob/e3c1a677087dd88db476fdc77e8e1a5c2db44a46/internal/ackhandler/sent_packet_handler.go#L314-L317

Without cache skipped packet numbers, func detectAndRemoveAckedPackets can also check if received an ACK for skipped packet number?

I don't understand your question.

Sorry for my poor English, and thank you for your time.
For instance, sent packet numbers are 1, 3, 5.
There are 2 ways to implement sentPacketHistory:

only cache [1, 3, 5]
cache [1, 2, 3, 4, 5], each item with a skipped flag, and item 2 and 4's flag is true

Both ways can detect the situation of receiving an ACK for skipped packet numbers:

not found in the container
found but skipped flag is true

The 1st way is much simpler.

The problem with the 1st way is the following:

We send packets [ 1, 2, 3, 4 ].
We receive an ACK for packet 2, and delete packet 2 from the history. Now we have [ 1, 3, 4 ].
Now we recieve an ACK for packet 1, 2 and 3.

Maybe it's better to don't to remove the skipped number in sentPacketHistory::Remove.
But it's ok since we don't need to handle receiving two ACK for the same skipped number.

Well, we need to remove them eventually to keep the state bounded. We just rely on QUIC loss recovery for this: Once a skipped packet number is declared "lost", we can just remove it.


Does not react to @media prefers-color-scheme

Bug already reported?

[X] I confirm that I have checked if the bug already has been reported

For which framework/library you are reporting the bug
Chlorophyll
Component name
all?
Description
Components should use @media prefers-color-scheme
Steps To Reproduce
Open a page that uses chlorophyll and set darkmode for your browser.
Current Behaviour
No change
Expected Behaviour
Expect component to use dark theme

@mackelito You need to add the class auto-mode somewhere up-tree in the document. Here's an example: https://sebgroup.github.io/green/latest/chlorophyll/?path=/docs/use-cases-dark-mode-use-preferred-color-scheme--page
In a nut-shell:

Add auto-mode to abide by prefers-color-scheme
Add dark-mode to force dark mode
Not setting either will default to light mode

That said, though, the current dark-mode implementation in Chlorophyll is spotty at best. We're working on a rewrite of the css variables/tokens setup that will eventually make it more consistent and easier to maintain.


Tidy headers

Copyright notice per file is not required per-se, will have my lawdude look at it, will merge for now can go over it afterwards.


Allow users to create roles when creating users

I often create a new role for each user to have granular control over user permissions. This PR makes this easier by adding an option to create a new role on the fly while creating a new user:

Thanks @timoschwarzer for providing this and apologies for my late feedback.
While I can definitely see the use-case/benefits for some people at the moment, this is not a feature that I'd specifically want to add. Realistically per-user roles are a workaround for a lack of user-level item permissions which is something I'd still like to attempt later on in the roadmap. Additionally, encouraging users to potentially create one-off roles where not explicitly needed could lead to performance considerations, especially with large user-counts. Therefore I'm going to close this off, Sorry.

Thanks @timoschwarzer for providing this and apologies for my late feedback.
While I can definitely see the use-case/benefits for some people at the moment, this is not a feature that I'd specifically want to add. Realistically per-user roles are a workaround for a lack of user-level item permissions which is something I'd still like to attempt later on in the roadmap. Additionally, encouraging users to potentially create one-off roles where not explicitly needed could lead to performance considerations, especially with large user-counts. Therefore I'm going to close this off, Sorry.


Add support for multicast?

I'd like to use this for an installation where the networking can change. It'd be nice to publish to a multicast address so the IPs are the same regardless of network setup. Thanks for publishing this, it works like a charm otherwise!

I'm sorry, I currently don't plan to support multicast because I don't have much time to work on this. I hope you can contribute about it!


Offline Access

From an email I received from Andre:
I have a question regarding Passport-Youtubev3 strategy. It seems to be working however every single time, Google is prompting me to authorize with ‘Have offline access’. The strategy doesn’t seem to allow any additional parameters, however in your code, I found hardcoded following piece of code:
https://github.com/yanatan16/passport-youtube-v3/blob/master/lib/passport-youtube-v3/strategy.js#L103
Strategy.prototype.authorizationParams = function(options) {
  //  test, should be passed through options
  return {
    access_type : 'offline',
    approval_prompt: 'force'
  }
}

Should this piece of code be there? Would you have any other ideas?

Yes these authorization parameters cause Google to try to authenticate the offline access ability. My gut feeling is that you'll want offline access, so you can use the access tokens later.
https://developers.google.com/identity/protocols/OAuth2WebServer#offline
The best fix is probably to actually use the options object passed into that function with a default to the current parameters (for backwards compatibility)

We could just copy the code from https://github.com/jaredhanson/passport-google-oauth/blob/master/lib/passport-google-oauth/oauth2.js#L130

Thank you Jon

Ok I merged the req and tagged the new version at 1.0.0. Figured it should be 1.0 anyway. You'll have to update your package.json via npm install<EMAIL_ADDRESS>--save

Great! Thanks Jon

Hi Jon,
I must be missing something, I still have the same problem. It seems that whatever options I set, strategy defaults to 'force' however works fine if I change the hardcoded defaults in your code. Here is how I configure the strategy. What I am doing wrong?
passport.use(new YoutubeV3Strategy({
    clientID: "xxxxxxxxxxxxxxxxxxxxxxx",
    clientSecret: "xxxxxxxxxxxxxxxxxxxxx",
    callbackURL: config.SERVER_LOCATION + "/auth/youtube/return",
    approval_prompt: 'auto',
    access_type: 'online',
    scope: ['https://www.googleapis.com/auth/youtube.readonly', 'https://www.googleapis.com/auth/plus.profile.emails.read']

},
  function (accessToken, refreshToken, profile, done) {...

Try
passport.use(new YoutubeV3Strategy({
    clientID: "xxxxxxxxxxxxxxxxxxxxxxx",
    clientSecret: "xxxxxxxxxxxxxxxxxxxxx",
    callbackURL: config.SERVER_LOCATION + "/auth/youtube/return",
    authorizationParams: {
        approval_prompt: 'auto',
        access_type: 'online'
    },
    scope: ['https://www.googleapis.com/auth/youtube.readonly', 'https://www.googleapis.com/auth/plus.profile.emails.read']

},
  function (accessToken, refreshToken, profile, done) {...

@yanatan16 I have tried a config (see below) nearly identical to the one you proposed, and approval_prompt is still being set to force. If I set a breakpoint at the authorizationParams method in the Strategy source, options has no own properties (not even the other ones like clientID).
{
  "clientID": "xxxxxxxxxxxxxxxxxxxxxx",
  "clientSecret": "xxxxxxxxxxxxxxxxxxxxxxx",
  "callbackURL": "/path/to/url",
  "profileURL": "https://www.googleapis.com/oauth2/v3/userinfo",
  "authorizationParams": {
    "access_type": "offline",
    "approval_prompt": "auto"
  },
  "scope": ["https://www.googleapis.com/auth/youtube", "email", "profile"]
}

@bbatliner You are correct. The options that authorizationParams gets isn't the same as the options passed into new YoutubeOAuth.... So I've fixed this in 0b01605


updated constraint-declaration-validation.xml to contain Default grou…

Continuation of https://hibernate.atlassian.net/browse/BVAL-493
I'm not sure if anything more should be updated here?

Going to close this in favor of #234. Thanks again for the suggestion, @lpandzic!


utf case conversion

Why I'm doing:
Currently case conversion only works for asci symbols.
Other unicode symbols are left as is.
What I'm doing:
I'm using https://www.alphabet.se/download/UtfConv.c as a library which converts utf8 to utf32, converts case and converts back to utf8.
Fixes #issue
fixes uf case converion https://github.com/StarRocks/starrocks/issues/45064
What type of PR is this:

[ x] BugFix
[ ] Feature
[ x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[x] Yes, this PR will result in a change in behavior.
[ ] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.3
[ ] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

Can be backported.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Evgeniy Shishkin seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

As for the code license here the author of UtfConv.c states it as "To me it is a give away, absolutely free"
https://stackoverflow.com/questions/36897781/how-to-uppercase-lowercase-utf-8-characters-in-c/69555744#comment124413290_69555744

it will change the function behaviour. we prefer add a new function

@stdpain current behavior for non asci symbols is a bug
for all asci symbols behavior doesn't change

cc @satanson


Modules?

Continuing discussion from https://github.com/haumea-lang/haumea/issues/19

I think what we need is a mechanism to link against external code, so that way libraries will be quite easy to implement...
The problem is that these libraries could probably only be run under specific compilers - for example, if somebody made an ncurses library for Haumea, it is likely it would not work with the JavaScript compiler.
In order to do this, of course, there would need to be a few things:

Headers for Haumea code
A standard format for libraries to be linked

I'm thinking what would be best for the far future is a C library that provided methods for building Haumea library code.
However, for right now, it would be possible to just have Haumea headers correspond directly with C headers, and have their implemented methods correspond directly with C methods.
Of course, this would still require access to Haumea datatypes (particularly lists) in C.


Pass password using stdin

Hi there,
This PR implements passing a password through stdin, which increases security. In the current implementation passwords are passed as arguments, and are thus visible in /proc/PID/cmdline. For mounting, the problem might be small, as the cmdline is exposed only shortly (even then, its exposed in a world readable file...). For creating a volume, this would be a big issue, as the cmdline is exposed for the entire duration of creation. In that sense, the PR is the first step to securely creating volumes as well. See the example below for more info.
Let me know if you want me to merge to a different branch instead of develop!
Current implementation
//main.go

package main

import (
	"github.com/mark-hartmann/vera"
)

func main() {
	DismountAll()
	password := Param{Name: "password", Value: "1234"}
	_, err := Mount("/data/veracrypt/test.vc", 2, password)
	if err != nil {
		fmt.Println(err.Error())
		os.Exit(1)
	}

}

I print the info in /proc/PID/cmdline, a world-readable file (!). In the below example I implemented the print functionality in ExecCommand, but I could also have used external shell scripts or tools.
//cmd.go

func ExecCommand(args ...Param) (stdout *bytes.Buffer, err error) {
	cmd, stdout, stderr := newCommand(args...)
	err = cmd.Start()
	if err != nil {
		return stdout, err
	}

	o, err := getCmdline(cmd.Process.Pid)
	if err != nil {
		return stdout, err
	}

	fmt.Printf("I am PID %d and I am the following command:\n %s \n", cmd.Process.Pid, o)
	err = cmd.Wait()
	if err != nil {
		return stdout, parseError(stderr.String())
	}
	return stdout, nil
}

func getCmdline(pid int) (string, error) {
	dat, err := os.ReadFile(fmt.Sprintf("/proc/%s/cmdline", strconv.Itoa(pid)))

	if err != nil {
		return "", err
	}

	return string(dat), nil
}

Console output:
Console output to show the password is clearly visible.

I am PID 238119 and I am the following command:
veracrypt -t --non-interactive --dismount
I am PID 238139 and I am the following command:
veracrypt -t --non-interactive --password=1234 --slot=2 /data/veracrypt/test.vc

New implementation
Piping the password to stdin
Again, we add some debugging to ExecCommand()
//cmd.go

func ExecCommand(args ...Param) (stdout *bytes.Buffer, err error) {
	cmd, stdout, stderr := newCommand(args...)
	err = cmd.Start()
	if err != nil {
		return stdout, err
	}

	o, err := getCmdline(cmd.Process.Pid)
	if err != nil {
		return stdout, err
	}

	fmt.Printf("I am PID %d and I am the following command:\n %s \n", cmd.Process.Pid, o)
	err = cmd.Wait()
	if err != nil {
		return stdout, parseError(stderr.String())
	}
	return stdout, nil
}

func getCmdline(pid int) (string, error) {
	dat, err := os.ReadFile(fmt.Sprintf("/proc/%s/cmdline", strconv.Itoa(pid)))

	if err != nil {
		return "", err
	}

	return string(dat), nil
}

Add a password parameter to Mount(), as Mount() needs a password anyway, so the additional benefit is that we enforce this now.
Pass the password string to ExecCommandWithStdin (I am open to another suggestion on how to pass to stdin, but I thought this was the most elegant and it resembles how Veracrypt would do it on the CLI as well)

//mount.go

// Mount mounts the volume in the given slot. Depending on the volume, the appropriate parameters must be set.
// password is passed through stdin
func Mount(volume string, slot uint8, password string, opts ...Param) (MountProperties, error) {
	// check if the slot number is supported
	if err := slotValid(slot); err != nil {
		return MountProperties{}, err
	}

	// append path and mount as Param
	opts = append(opts, Param{Value: volume}, Param{Name: "slot", Value: strconv.Itoa(int(slot))}, Param{Name: "stdin", IsFlag: true})
	_, err := ExecCommandWithStdin(password, opts...)
	if err != nil {
		return MountProperties{}, err
	}

	return PropertiesSlot(slot)
}

//cmd.go
func ExecCommandWithStdin(stdin string, args ...Param) (stdout *bytes.Buffer, err error) {
	cmd, stdout, stderr := newCommand(args...)
	cmd.Stdin = strings.NewReader(stdin)

	err = cmd.Start()
	if err != nil {
		return stdout, err
	}

	o, err := getCmdline(cmd.Process.Pid)
	if err != nil {
		return stdout, err
	}

	fmt.Printf("I am PID %d and I am the following command:\n %s \n", cmd.Process.Pid, o)
	err = cmd.Wait()
	if err != nil {
		return stdout, parseError(stderr.String())
	}
	return stdout, nil
}

Console output

I am PID 246836 and I am the following command:
veracrypt -t --non-interactive --dismount
I am PID 246856 and I am the following command:
veracrypt -t --non-interactive --slot=2 --stdin /data/veracrypt/test.vc
I am PID 246913 and I am the following command:
veracrypt -t --non-interactive --list --slot=2
2: /data/veracrypt/test.vc /dev/mapper/veracrypt2 /media/veracrypt2

Excellent work, both with the above elaboration and the actual implementation. I actually had not yet thought about this quite serious security issue, thank you for covering this topic so thoroughly. The tests and test coverage look good and the password parameter of the Mount function definitely makes sense, develop as branch is fine.
I will merge your PR and create a new release with these adjustments later today, as I think this fix is important enough to be merged into master asap.

Much appeeciated! And thanks for your quick reply


Assets folder not generated

Hi! I'm trying to generate a file but my assets folder is empty.
This is the error:
Creating Element { name: Frame 1, type: FRAME }
Element with the name: `Frame 1` cannot be parsed. Would be displayed as Black Rectangle

Here is the link to my figma design: https://www.figma.com/file/8rZwC9OAJbugW7sG48ygvJ/Fitness-dashboard-Community?node-id=0%3A1

The type of UI you have created seems to have been copied from a different source. And it wont work as Tkinter Designer Functions on Naming of the elements.
Please read the instructions carefully and then create a simple UI to understand it and then start to build up on it.

Closing the issue because you did not read the instructions. Feel free to reopen if you face any problems after you have read the instructions.


Show active users

Description
SHOW ACTIVE USERS INFO is a command that displays information about users currently logged into Memgraph and session information about them.
Pull request type
Please check what kind of PR this is:

[X] Fix or improvement of an existing page
[ ] New documentation page, release related

Related PRs and issues
PR this doc page is related to:
https://github.com/memgraph/memgraph/pull/2183
Checklist:

[X] Check all content with Grammarly
[X] Perform a self-review of my code
[X] Make corresponding changes to the rest of the documentation (consult with the DX team)
[X] The build passes locally
[X] My changes generate no new warnings or errors
[X] Add a corresponding label
[X] If release-related, add a product and version label
[X] If release-related, add release note on product PR

Addressed @kgolubic


[ci skip] [cf admin skip] NO_CI adding bot automerge

Hi! This is the friendly automated conda-forge-webservice.
I've added bot automerge as instructed in #3.
Merge this PR to enable bot automerging.
Fixes #3

thanks for merging @jourdain, FYI this bot will make it so that we can be hands off this feedstock during wslink releases

yes, that is fine with me


Pull mirror doc is not accurate anymore(?)

Description
The doc at https://docs.gitea.com/usage/repo-mirror#pulling-from-a-remote-repository / https://github.com/go-gitea/gitea/blob/release/v1.21/docs/content/usage/repo-mirror.en-us.md explains you can click a "box This repository will be a mirror.". Such a box does not exist (anymore?).
Screenshots

Actually in gitea
Here for GitHub this is the interface, I see no such box:

URL: https://codeberg.org/repo/migrate?service_type=2&org=&mirror=
Gitea Version
I dunno
Can you reproduce the bug on the Gitea demo site?
No
Operating System
No response
Browser Version
Firefox 121

Oh found the solution: Mirroring is disabled for my instance :( https://docs.codeberg.org/getting-started/faq/#why-can't-i-mirror-repositories-from-other-code-hosting-websites%3F'
Maybe you can add a note in the doc that – if not available – this feature has/may/can be disabled by an administrator. But apart from that I don't see much you can do here, unfortunately.


webkit module not available in pip

python-webkit(debian package) is not maintained, or pythonwebkitgtk(pip package) is not maintained.

I try to modify python-webkit to gir1.2-webkit-3.0 and python-gi.
And I have modified one version, but it is not applied with the current code.


Feat: side (left/right) modal

What:
Add a side (left/right) modal card.

Why:
We need a side modal for this design.

How:
Add a new component SideCard which renders when it receives the side props ("left" | "right") and slides from the relative side.

Media:

See more in this video on loom
Checklist:

[ ] Ready to be merged

I kind of like the solution, however I am not sure this right search is a modal tho, maybe we need to create a side bar component instead. What do you think @rafael-sepeda Vini?

@agonzalezUX agree. It works as a sidebar. And then we need something generic: a sidebar container-component, that will allow to inject different content: menu, dialogs, etc.

maybe we can treat it as an off-canvas component
https://getbootstrap.com/docs/5.0/components/offcanvas/


Create basic basic layout.

What this PR does.
Creates a basic application layout using the material ui framework.
It also establishes a routing system using React Router.

Changes Unknown when pulling 5c9302ffda7d8a52c0ee34a591597e23ebff6a42 on feature/basic-layout into ** on develop**.


Make colour red

Can we make the style look like https://sydney-informatics-hub.github.io/geopython/
Maybe just swap out the css file? Or is there some setting I don't know.
https://github.com/Sydney-Informatics-Hub/geopython/blob/main/_quarto.yml

This repo gives you some options for changing themes. I thought we switched to blue last year?

This repo gives you some options for changing themes. I thought we switched to blue last year?

Ah right, that is why I tagged you. Easy. Blue is good too. Thankkkks

I think it's okay to pick the colour you like the most!

This option
format:
  html:
    theme: simplex
    css: [lesson.css, bootstrap-icons.css]
    toc: true
    code-overflow: wrap
    highlight-style: github

works for the ochres.
I'll add the blue option too.

Red! Ta


FR: exit from an active session by 'q'

Right now one must press Ctrl+C which is somewhat rude.
Would be good to adhere to common behaviour and support exiting from program by 'q'

q - exit with patterns removal
Q - exit without patterns removal

Need Enter press.
Otherwise q won't be sent to a program.
Raw input from console is hardly possible - there is no portable way to configure console from out of Java without 3rd-party dependencies (e.g. jCurses)

Fixed - console is configured from under .sh


Plotting counts showing horizontal

Encountered issue as per snippet below.
This happened in R 3.6.3, tried updating to R 4.0.0 and 4.0.1, but getting same issue. Happy to supply additional information if required to resolve.
cross_plot(data = heart_disease, input = "oldpeak", target = "has_heart_disease")

Thanks for reporting it!
Can you execute the sessionInfo() and copy paste the output?
On Sun, Jun 7, 2020 at 2:24 PM carstensal<EMAIL_ADDRESS>wrote:

Encountered issue as per snippet below.
This happened in R 3.6.3, tried updating to R 4.0.0 and 4.0.1, but getting
same issue. Happy to supply additional information if required to resolve.
cross_plot(data = heart_disease, input = "oldpeak", target =
"has_heart_disease")
[image: image]
https://user-images.githubusercontent.com/9154013/83975222-11f4da00-a8f3-11ea-950b-60ce0cf375d1.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pablo14/funModeling/issues/15, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACHPQVLYHVHY7U4H2UQFWWDRVPELBANCNFSM4NXOLI5Q
.

No problem, let me know if can provide anything else or test something.
When running on a new session (rm(list = ls()))
`> sessionInfo()
R version 4.0.1 (2020-06-06)
Platform: x86_64-w64-mingw32/x64 (64-bit)
Running under: Windows 10 x64 (build 18363)
Matrix products: default
Random number generation:
RNG:     Mersenne-Twister
Normal:  Inversion
Sample:  Rounding
locale:
[1] LC_COLLATE=English_South Africa.1252  LC_CTYPE=English_South Africa.1252    LC_MONETARY=English_South Africa.1252
[4] LC_NUMERIC=C                          LC_TIME=English_South Africa.1252
attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base
loaded via a namespace (and not attached):
[1] compiler_4.0.1 tools_4.0.1  `
After loading funModelling:
`

library(funModeling)
Loading required package: Hmisc
Loading required package: lattice
Loading required package: survival
Loading required package: Formula
Loading required package: ggplot2

Attaching package: ‘Hmisc’
The following objects are masked from ‘package:base’:
format.pval, units

funModeling v.1.9.3 :)
Examples and tutorials at livebook.datascienceheroes.com
/ Now in Spanish: librovivodecienciadedatos.ai

cross_plot(data = heart_disease, input = "oldpeak", target = "has_heart_disease")
Plotting transformed variable 'oldpeak' with 'equal_freq', (too many values). Disable with 'auto_binning=FALSE'
sessionInfo()
R version 4.0.1 (2020-06-06)
Platform: x86_64-w64-mingw32/x64 (64-bit)
Running under: Windows 10 x64 (build 18363)

Matrix products: default
Random number generation:
RNG:     Mersenne-Twister
Normal:  Inversion
Sample:  Rounding
locale:
[1] LC_COLLATE=English_South Africa.1252  LC_CTYPE=English_South Africa.1252    LC_MONETARY=English_South Africa.1252
[4] LC_NUMERIC=C                          LC_TIME=English_South Africa.1252
attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base
other attached packages:
[1] funModeling_1.9.3 Hmisc_4.4-0       ggplot2_3.3.1     Formula_1.2-3     survival_3.1-12   lattice_0.20-41
loaded via a namespace (and not attached):
[1] tidyselect_1.1.0    xfun_0.14           reshape2_1.4.4      pander_0.6.3        purrr_0.3.4         splines_4.0.1
[7] colorspace_1.4-1    vctrs_0.3.0         generics_0.0.2      htmltools_0.4.0     base64enc_0.1-3     rlang_0.4.6
[13] pillar_1.4.4        foreign_0.8-80      glue_1.4.1          withr_2.2.0         entropy_1.2.1       RColorBrewer_1.1-2
[19] plyr_1.8.6          jpeg_0.1-8.1        lifecycle_0.2.0     stringr_1.4.0       munsell_0.5.0       gtable_0.3.0
[25] moments_0.14        htmlwidgets_1.5.1   labeling_0.3        latticeExtra_0.6-29 knitr_1.28          fansi_0.4.1
[31] htmlTable_1.13.3    Rcpp_<IP_ADDRESS>        acepack_1.4.1       ROCR_1.0-11         scales_1.1.1        backports_1.1.7
[37] checkmate_2.0.0     farver_2.0.3        gridExtra_2.3       png_0.1-7           digest_0.6.25       stringi_1.4.6
[43] dplyr_1.0.0         grid_4.0.1          cli_2.0.2           tools_4.0.1         magrittr_1.5        lazyeval_0.2.2
[49] tibble_3.0.1        cluster_2.1.0       crayon_1.3.4        pkgconfig_2.0.3     ellipsis_0.3.1      Matrix_1.2-18
[55] data.table_1.12.8   assertthat_0.2.1    rstudioapi_0.11     R6_2.4.1            rpart_4.1-15        nnet_7.3-14
[61] compiler_4.0.1     `

Thanks!
It's fixed now :D A bad compatibility with the latest ggplot. Please update funModeling (the latest version is 1.9.4). I'm closing the ticket, reopen if the problem persists. Have a nice day.


internal: publish pre-release

@rest-hooks/core: 3.1.0-beta.2 => 3.1.0-beta.3
@rest-hooks/endpoint: 2.1.0 => 2.2.0-beta.0
@rest-hooks/legacy: 4.1.0-beta.1 => 4.1.0-beta.2
@rest-hooks/normalizr: 8.1.0 => 8.2.0-beta.0
rest-hooks: 6.2.0-beta.2 => 6.2.0-beta.3
@rest-hooks/test: 7.1.3 => 7.2.0-beta.0

Codecov Report

Merging #1643 (ddfd895) into master (7a99fae) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1643   +/-   ##
=======================================
  Coverage   98.71%   98.71%
=======================================
  Files         102      102
  Lines        1865     1865
  Branches      370      370
=======================================
  Hits         1841     1841
  Misses         10       10
  Partials       14       14

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7a99fae...ddfd895. Read the comment docs.


[CMake] CMAKE_BUILD_TYPE(s) cleaning

This PR fixes https://github.com/medInria/medInria-public/issues/439.

CMAKE_BUILD_TYPE_medInria was never used, only medInria _BUILD_TYPE in medInria.cmake
CMAKE_BUILD_TYPE_externals_projects was too long

:m:

Too dangerous to merge for 3.1 so I postpone it to the next one

@Florent2305 CMAKE_BUILD_TYPE_medInria was already not used in 2019, i did a PR about it. We could look at it during 4.0

I rebased this PR. This is still needed because 2 variables are set for medInria

CMAKE_BUILD_TYPE_externals -> CMAKE_BUILD_TYPE_ExtProjs for a better reading, as seen with @Florent2305


Filter is unexpectedly reset when column used in histogram is changed, if PC plot is also present

1.5.1
Steps to reproduce:

Open a table, e.g. SPGI
Add Histogram and PC plot viewers
Open filters panel and enable some filtering
Change column selected in histogram

Expected results: changing histogram configuration should not affect currently set filters.
Actual results: filtering is un-applied when histogram column is changed:

Issue can be reproduced for all kind of filters, including structure filters.

Good catch! Fixed in Datagrok v1.6.2