generate k8s certs bug

Dear author,
Maybe there is a bug about generation k8s certificates in role/kubernetes/secrets/tasks/check_certs.yml.
The variable gen_certs equals true always since v2.1.2, so it always creates new certs. In face, it is necessary to create new certs only when certs not exists or changed.
It is maybe your slip. We should modify bracket ']' to comma ',', and add some comma ',' in proper position.
There are other bugs about generation certs in master branch, not in some old versions. For example, incomplete certificate verification, .... These bugs maybe have an impact on the final result, but this is another problem. If you need it, I can help to modify it.
Sorry submit several time.
Thanks.

I have no kubespray-ci run authority. How can I get authority?

Duplicate of https://github.com/kubernetes-incubator/kubespray/pull/2885 ?

Dear himpich,
#2885 maybe not resolve the problem, it only change the ']', but gen_certs is still true. We should add ',' in four position to solve the problem.

#2885 also adds the , in the for-loops at the needed places.

Sorry, but maybe #2885 added positions were wrong, there should be no comma ',' in the end of for loop。

there should be no comma ',' for the last item in the for-loop

Hi @elementyang
the problem is that the last element in the first for loop will need a , if their are elements in the second loop.
In your case the last element in the first loop will never get a , so it will only generate a proper array if there are no hosts in the second loop.
That is the reason i putt the loops at the beginning and always add a , in the end. We always have the default Certs following.

Dear qeqar,
I think the last item in the first-loop don't need comma ',', If you add ',' in the last item, get_certs is still true. I think all of the last item in the first-loop don't need comma ','. I have debuged locally. Maybe you changed the original structure.

Ahh i missed the , after the first {% endfor %} in that case you always need elements in both loops.
Which normally should be the case.
Both cases should work. i hope it's easier to add more loops in my case and you have some less code to understand. (like missing the , after the {% endfor %})

Dear qeqar,
I am a newer, and I have no ci-auth. If I want to pull request again, do I need someone to join me in the white list. How can I do? Because I have another new module that needs to be submitted.
Thanks!

Hi Just wait for the Reviewer.

Dear qeqar,
If I need send a email to kubespray project in advance, because my new module is little big.
Does kubespray project have mailting list?
I pull request or issue, or other way?
Thanks!

issue or pull request should be ok, mailing list?read the docs?

Dear qeqar,
Sorry, I'm puzzled. I don't find mailing list. Do I need pull request directly or send some email in advance, because my new module is little big. I have no ci-auth yet.
Sorry to bother you.
Thanks!

If there is no list or slack or anything make a pull request. there you can show your code! everything else will come from the maintainer...

Dear qeqar,
Thank you very much!

ci check this

Hi Mattymo,
Thank you very much. I check that ci has succeeded, but pipeline state still failed. My submit is duplicate with #2885, so my submit is refused?

Hi mattymo
thanks for agreement with my request. It is my first time contributing to society and hope we have future discussion and collaboration on kubernetes.


Excuse me, can the RTSP push stream not be pushed to the public network? I can use WiFi to push successfully, but I cannot switch to the SIM card network

Sorry, my push stream address is wrong, I need to change it to the address of the public network

This seems solved. Closing issue


consideration for node

Seeing how this feels over [String: String]
Benefits:
let id = try request.params.extract("id") as Int // any node convertible
// or older style
guard let id = request.parameters["id"].int else { throw or something }

After internal discussion, we decided that parameters doesn't make as much sense as part of engine and will be moved to either routing or Vapor.


The Sentinel triggers summary undervalues the role of alert-based triggers

[Enter feedback here]
Alert-based triggers are just as important as incident-based triggers. The summary suggests that incident-based trigger use is the best practice. I have seen no evidence to support this claim. There are many places where alert-based triggers are equally important. It seems confusing to recommend one over the other. Also, this summary does not highlight how alert based triggers are better when alert grouping is used and when there is a need to process alerts. I recommend rewording. Happy to discuss if needed.

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

ID: 24314eb4-ce82-5076-b34e-3276626f1805
Version Independent ID: 1a91091e-8121-1404-8f19-c45134e4adf1
Content: Use triggers and actions in Microsoft Sentinel playbooks
Content Source: articles/sentinel/playbook-triggers-actions.md
Service: microsoft-sentinel
Sub-service: microsoft-sentinel
GitHub Login: @yelevin
Microsoft Alias: yelevin

@AndrewBlumhardt Thank you for your feedback . We will investigate and update the thread.

@AndrewBlumhardt Thank you for raising this concern related to incident trigger vs alert trigger, do you have any use cases to justify this change, Please send me an email to<EMAIL_ADDRESS>with Sub - Attn: Givary.


Add rustler precompiled

This adds a workflow to precompile the nifs using rustler_precompiled.
Publishing to hex is removed from the pipeline, as the checksum generation step is needed, see: https://hexdocs.pm/rustler_precompiled/precompilation_guide.html#content and especially:
https://hexdocs.pm/rustler_precompiled/precompilation_guide.html#the-release-flow
When running locally and wanting to force the build, set LINGUA_BUILD=1, e.g. LINGUA_BUILD=1 mix test or LINGUA_BUILD=1 mix compile.

Any news about this PR?

The nif.ex doens't found the precompiled binaries.
== Compilation error in file lib/lingua/nif.ex ==
** (RuntimeError) Error while downloading precompiled NIF: couldn't fetch NIF from https://github.com/joshrotenberg/lingua_ex/releases/download/v0.2.0/lingua_nif-v0.2.0-nif-2.16-x86_64-pc-windows-msvc.dll.tar.gz: {:ok, {{~c"HTTP/1.1", 404, ~c"Not Found"}, [{~c"cache-control", ~c"no-cache"}, {~c"date", ~c"Thu, 21 Mar 2024 08:46:17 GMT"}, {~c"server", ~c"GitHub.com"}, {~c"vary", ~c"X-PJAX, X-PJAX-Container, Turbo-Visit, Turbo-Frame, Accept-Encoding, Accept, X-Requested-With"}, {~c"content-length", ~c"9"}, {~c"content-type", ~c"text/plain; charset=utf-8"}, {~c"strict-transport-security", ~c"max-age=31536000; includeSubdomains; preload"}, {~c"x-frame-options", ~c"deny"}, {~c"x-content-type-options", ~c"nosniff"}, {~c"x-xss-protection", ~c"0"}, {~c"referrer-policy", ~c"no-referrer-when-downgrade"}, {~c"content-security-policy", ~c"default-src 'none'; base-uri 'self'; connect-src 'self'; form-action 'self'; img-src 'self' data:; script-src 'self'; style-src 'unsafe-inline'"}, {~c"x-github-request-id", ~c"D7B9:10B68D:309F1CF:317070F:65FBF3D9"}], "Not Found"}}

@minidfx if you want to use the precompiled NIFs from the PR, you need to use
{:lingua, github: "SteffenDE/lingua_ex", branch: "development"},

Pull Request Test Coverage Report for Build 6c4482568c7f847d572aa856eaae4ee72a3886db-PR-11
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 86.364%

Totals

Change from base Build 64f45da57828c262150abbb9742a218cbb4bb462:
0.0%

Covered Lines:
19

Relevant Lines:
22

💛  - Coveralls

Pull Request Test Coverage Report for Build 6c4482568c7f847d572aa856eaae4ee72a3886db-PR-11
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 86.364%

Totals

Change from base Build 64f45da57828c262150abbb9742a218cbb4bb462:
0.0%

Covered Lines:
19

Relevant Lines:
22

💛  - Coveralls


Sklearn Dependency update to 1.4 instead of 1.3

In various places the following function from sklearn root_mean_square_error is imported and used (e.g: here). The function was added with version 1.4 hence the pyproject.toml should be updated from 1.3 to 1.4 to reflect this dependency.
Would've liked to make a PR myself, but too much effort to understand how to get pixy pre-commit to run without documentation (to my understanding already work in progress :))

Great catch - thanks for pointing this out! :)

but too much effort to understand how to get pixy pre-commit to run without documentation

Did you see this?
https://metalearners.readthedocs.io/en/latest/development.html#

Did you see this?
https://metalearners.readthedocs.io/en/latest/development.html#

Maybe we could add a link in the README for this so it's easier to find it?

Did you see this?
https://metalearners.readthedocs.io/en/latest/development.html#

Nope, didn't see that 😅 . I think having a link in the readme or even the whole guide might be easier.


Enable to parse the threads property in profiles.yml

Is your feature request related to a problem? Please describe.
We can configure dbt configurations with environment variables as DBT_DEBUG. Meanwhile, dbt doesn't support such features to vary properties in profiles.yml.
We implement macros to vary properties profiles.yml based on environments. When I tried to use the profiles.yml including the threads property with a macro reference, I got the subsequent error message.
Error message
$ piperider run  --select "tag:piperider"
────────────────────────────────────────────────────────────────── Validating ──────────────────────────────────────────────────────────────────
everything is OK.
Error: '>' not supported between instances of 'LiteralScalarString' and 'int'

Example of the threads property with a macro
The custom environment variable called DBT_THREADS enables us to vary the number of threads in a dbt execution. Though I also tried to use the int Jinja2 function, I got the same error.
https://docs.getdbt.com/reference/dbt-jinja-functions/env_var
jaffle_shop:
  target: dev
  outputs:
    dev:
      type: bigquery
      method: oauth
      project: "your-project"
      dataset: "jaffle_shop"
      priority: "{{ env_var('DBT_BIGQUERY_PRIORITY', 'interactive') }}"
      location: "asia-northeast1"
      threads: |
        {{ env_var("DBT_THREADS", 3) | as_number }}

Describe the solution you'd like
I appriciate if piperider could parse the use case with profiles.yml.
Describe alternatives you've considered
N/A
Additional context
N/A

Thanks for your request.
sc-32473 for internal tracking

Fixed in v0.38.0


Fix issue in returning masked? status of a unit when using systemd_unit resource

Signed-off-by: Neha Pansare<EMAIL_ADDRESS>Description
(Finishing off https://github.com/chef/chef/pull/12227)
When a process is masked using systemd, its UnitFileState becomes bad.
The method masked? on resource systemd_unit is updated to consider this since it would result in resource not being converged when run multiple times.
Related Issue
https://github.com/chef/chef/issues/11735
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Chore (non-breaking change that does not add functionality or fix an issue)

Checklist:

[ ] I have read the CONTRIBUTING document.
[ ] I have run the pre-merge tests locally and they pass.
[ ] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.
[ ] All commits have been signed-off for the Developer Certificate of Origin.

@neha-p6  do we still need this?


Exercise 5 level 5, incompatible type

Hi, ive followed the Optional  method, however, how do i force it to change to a Maybe T, so that i do not get the incompatible type error? Appreciate it alot !

You can try using the methods you defined earlier to help you convert it.
Specifically I decided to use orElse since it returns the thing if it is not null and then I convert it back into a Maybe using the of method.

I did the same thing as what @OngMinXian said and it worked perfectly!

Thank you guys for your advice!

I got similar issue and I solved it by casting the supplier.get() to a Maybe<? extends T> first


KeyError: "The name 'deepspeech/logits:0' refers to a Tensor which does not exist.

While excute --- Step1: extract images from vids ---
Traceback (most recent call last):
File "data_util/deepspeech_features/extract_ds_features.py", line 128, in
main()
File "data_util/deepspeech_features/extract_ds_features.py", line 120, in main
extract_features(
File "data_util/deepspeech_features/extract_ds_features.py", line 80, in extract_features
conv_audios_to_deepspeech(
File "/data/chenxianhao/AD-NeRF/data_util/deepspeech_features/deepspeech_features.py", line 41, in conv_audios_to_deepspeech
graph, logits_ph, input_node_ph, input_lengths_ph = prepare_deepspeech_net(
File "/data/chenxianhao/AD-NeRF/data_util/deepspeech_features/deepspeech_features.py", line 105, in prepare_deepspeech_net
logits_ph = graph.get_tensor_by_name("deepspeech/logits:0")
File "/opt/anaconda3/lib/python3.8/site-packages/tensorflow/python/framework/ops.py", line 3931, in get_tensor_by_name
return self.as_graph_element(name, allow_tensor=True, allow_operation=False)
File "/opt/anaconda3/lib/python3.8/site-packages/tensorflow/python/framework/ops.py", line 3755, in as_graph_element
return self._as_graph_element_locked(obj, allow_tensor, allow_operation)
File "/opt/anaconda3/lib/python3.8/site-packages/tensorflow/python/framework/ops.py", line 3795, in _as_graph_element_locked
raise KeyError("The name %s refers to a Tensor which does not "
KeyError: "The name 'deepspeech/logits:0' refers to a Tensor which does not exist. The operation, 'deepspeech/logits', does not exist in the graph."

Hey, What exactly did u do to resolve the issue?.

Have you solved this problem?

I wonder have you solved this problem too.

Downgrading tensorflow-gpu to 1.15.0 helped me solve this problem.

Look for "logits:0" instead of "deepspeech/logits:0" will do the trick. This can be found after inspecting the name of all the nodes in the graph.

tensorflow-gpu to 1.15.0 is right

mark


Create new pallet for rewarding system

The rewarding system should allocate tokens to all tickets owners. The process should run offchain and periodically.

Hey team! Please add your planning poker estimate with Zenhub @aang114 @sinkingsugar


Fix typo in doc

That's not a typo, it's correct as it stands.

This is not a typo, it is correct English.


mayavi problem

i follow mayavi_install.sh  to install mayavi.
However, when i run test_drawline.py, it shows：
File "test_drawline.py", line 2, in
from mayavi.mlab import *
File "/usr/local/lib/python2.7/dist-packages/mayavi/mlab.py", line 16, in
from mayavi.tools.camera import view, roll, yaw, pitch, move
File "/usr/local/lib/python2.7/dist-packages/mayavi/tools/camera.py", line 24, in
from .engine_manager import get_engine
File "/usr/local/lib/python2.7/dist-packages/mayavi/tools/engine_manager.py", line 12, in
from mayavi.preferences.api import preference_manager
File "/usr/local/lib/python2.7/dist-packages/mayavi/preferences/api.py", line 4, in
from .preference_manager import preference_manager
File "/usr/local/lib/python2.7/dist-packages/mayavi/preferences/preference_manager.py", line 29, in
from traitsui.api import View, Group, Item
File "/usr/local/lib/python2.7/dist-packages/traitsui/api.py", line 36, in
from .editors.api import ArrayEditor
File "/usr/local/lib/python2.7/dist-packages/traitsui/editors/init.py", line 23, in
from .api import ArrayEditor
File "/usr/local/lib/python2.7/dist-packages/traitsui/editors/api.py", line 24, in
from .code_editor import CodeEditor
File "/usr/local/lib/python2.7/dist-packages/traitsui/editors/code_editor.py", line 37, in
class ToolkitEditorFactory(EditorFactory):
File "/usr/local/lib/python2.7/dist-packages/traitsui/editors/code_editor.py", line 49, in ToolkitEditorFactory
mark_color = Color(0xECE9D8)
File "/usr/local/lib/python2.7/dist-packages/traits/traits.py", line 522, in call
return self.maker_function( *args, **metadata )
File "/usr/local/lib/python2.7/dist-packages/traits/traits.py", line 1236, in Color
return ColorTrait( *args, **metadata )
File "/usr/local/lib/python2.7/dist-packages/traitsui/toolkit_traits.py", line 8, in ColorTrait
return toolkit().color_trait(*args, **traits)
File "/usr/local/lib/python2.7/dist-packages/traitsui/toolkit.py", line 109, in toolkit
_toolkit = find_toolkit('traitsui.toolkits')
File "/usr/local/lib/python2.7/dist-packages/pyface/base_toolkit.py", line 272, in find_toolkit
toolkit = plugin.load()
File "/home/yuan/.local/lib/python2.7/site-packages/pkg_resources/init.py", line 2322, in load
return self.resolve()
File "/home/yuan/.local/lib/python2.7/site-packages/pkg_resources/init.py", line 2328, in resolve
module = import(self.module_name, fromlist=['name'], level=0)
File "/usr/local/lib/python2.7/dist-packages/traitsui/qt4/init.py", line 25, in
import toolkit
File "/usr/local/lib/python2.7/dist-packages/traitsui/qt4/toolkit.py", line 32, in
from .constants import screen_dx, screen_dy
File "/usr/local/lib/python2.7/dist-packages/traitsui/qt4/constants.py", line 27, in
_palette = QtGui.QApplication.palette()
TypeError: palette() takes exactly one argument (0 given)
Any ideas please？

i have found the problem.
Moreover, i find another problem:
"mlab.show()" should be added before "return l", otherwise, nothing shows in the viewer.

Hi, @pyni , how can you fix this issue?

At first I have same problem with you, but when I finished installing PyQt and PySide,  there is another problem appeared. My operating environment is python2.7,ubuntu16.04,tensorflow1.13.1.related info as follows.
Traceback (most recent call last):
File "test_drawline.py", line 2, in
from mayavi.mlab import *
File "/usr/local/lib/python2.7/dist-packages/mayavi/mlab.py", line 16, in
from mayavi.tools.camera import view, roll, yaw, pitch, move
File "/usr/local/lib/python2.7/dist-packages/mayavi/tools/camera.py", line 24, in
from .engine_manager import get_engine
File "/usr/local/lib/python2.7/dist-packages/mayavi/tools/engine_manager.py", line 13, in
from mayavi.core.registry import registry
File "/usr/local/lib/python2.7/dist-packages/mayavi/core/registry.py", line 16, in
from mayavi.core.metadata import Metadata, import_symbol
File "/usr/local/lib/python2.7/dist-packages/mayavi/core/metadata.py", line 11, in
from mayavi.core.pipeline_info import PipelineInfo
File "/usr/local/lib/python2.7/dist-packages/mayavi/core/pipeline_info.py", line 12, in
from .utils import get_tvtk_dataset_name
File "/usr/local/lib/python2.7/dist-packages/mayavi/core/utils.py", line 3, in
from vtk.numpy_interface import dataset_adapter as dsa
ImportError: No module named numpy_interface

At first I have same problem with you, but when I finished installing PyQt and PySide, there is another problem appeared. My operating environment is python2.7,ubuntu16.04,tensorflow1.13.1.related info as follows.
Traceback (most recent call last):
File "test_drawline.py", line 2, in
from mayavi.mlab import *
File "/usr/local/lib/python2.7/dist-packages/mayavi/mlab.py", line 16, in
from mayavi.tools.camera import view, roll, yaw, pitch, move
File "/usr/local/lib/python2.7/dist-packages/mayavi/tools/camera.py", line 24, in
from .engine_manager import get_engine
File "/usr/local/lib/python2.7/dist-packages/mayavi/tools/engine_manager.py", line 13, in
from mayavi.core.registry import registry
File "/usr/local/lib/python2.7/dist-packages/mayavi/core/registry.py", line 16, in
from mayavi.core.metadata import Metadata, import_symbol
File "/usr/local/lib/python2.7/dist-packages/mayavi/core/metadata.py", line 11, in
from mayavi.core.pipeline_info import PipelineInfo
File "/usr/local/lib/python2.7/dist-packages/mayavi/core/pipeline_info.py", line 12, in
from .utils import get_tvtk_dataset_name
File "/usr/local/lib/python2.7/dist-packages/mayavi/core/utils.py", line 3, in
from vtk.numpy_interface import dataset_adapter as dsa
ImportError: No module named numpy_interface
Thx for any of your advice helping me solve this problem.^ ^

I got the same problem because I installed the vtk by "apt-get install python-vtk"
As discussed

VTK used to offer vtkpython binaries on their download page. However, they've dropped this since VTK-8.x.x

So my solution is to re-install the vtk by "pip install vtk"

我好像是装了个PySide就不报这个错了（不记得是PySide还是PyQt）

所以不需要单独安装mayavi这个了吗兄弟


Accelerating message processing

In the current implementation, message processing is performed in a single thread. I want to speed it up by making it multithreaded without using a crate such as tokio.

#4 Close


Update code-blast.js

You will get an error if cm is not initialized.
cm.off() is not a function

I'm doing a hidden feature, it will be trigger by some specified words.
when I initialize CodeMirror, BlastCode will initialize together. At this time,  blastCode option will get a null value, so if you enter destroy function, cm.off() is not a function.


add get request support

add get request support

Sorry, just saw it. I have add field skip for get request, and add test for it. Please forgive me for not being good at English.


Resolve: Create migration for column position change if a field position is changed in spec #58

Partially fixes #58.
This pull request does not cover the case: lot of new columns are added, lot of columns are deleted and lot of columns are moved at different positions.
Because current state of library first compute all columns to create, drop and alter and then builds migrations (e.g. compiled languages).
In order to fix above issue, we should consider columns one by one and build its migration (e.g. interpreted languages). This will require lot of changes in current library and lot of changes in existing tests.

Thank you!

FYI: I moved tests for this issue to a separate file: tests/unit/Issue58FixTest.php


Persist attributes defined on code blocks

When using https://www.npmjs.com/package/markdown-it-attrs we are able to define attributes to be appended to elements in our markdown, e.g:
paragraph {data-toggle=modal}

would render:

paragraph

It does also support injection of attributes on the fenced code blocks, but this Eleventy Plugin does not consider the attrs defined. This PR adds that in such that the introduced parent
 takes ownership of the attributes.
For example, if, in my markdown, I include:
```python
nums = [x for x in range(10)]
```

```python {data=hello}
nums = [x for x in range(10)]
```

Then the following elements are rendered to my DOM:

This does also include a change from https://github.com/mamezou-tech/eleventy-plugin-code-clipboard/pull/12 - happy to revert that in favour of this attrs approach if that's preferred.

Thanks for such a quick turnaround @kudoh


Fix a potential security leak with ssh private keys and ignore empty keys

This pull request fix a potential security leak in workflow logs when the private key is not a GitHub deployment key.
Instead of writing the entire private key in logs, the zero based position of the key in the provided list is now outputted.
At the same time, I tweaked the flow to ignore empty keys before verifying if it's a GitHub deployment key!

ssh-add -L outputs public key parameters.
Can you confirm that in fact no private key data is written to logs?

Oh! You're right, I totally missed it. It's the public key that was written to logs, not the private key! 🤦🏻
I confirm that in fact no private key is written to logs.
I will close this pull request since no modifications are required!


Morning session

Hello, Abdulaziz good job!
You should implement the edit feature and improve your styles :D
keep hard work!

Thanks @jessicafarias, for your comments. I have fixed all the issues.


Added fonts 'Roboto' and 'Lato'

To use/apply the fonts into any html page, we just need to add a <link> in ....
Example:

    <link rel="stylesheet" href="/css/fonts.css">

PS: It looks like I changed a lot in the files, but its my auto-formatter(prettier). The content are the same. I just added a <link> to add the fonts.

To use/apply the fonts into any html page, we just need to add a <link> in ....
Example:

    <link rel="stylesheet" href="/css/fonts.css">

Question.  Is the first slash in front of css needed?  Is it a different location from css/fonts.css?  Or does it really refer to the same place?  I'm actually not sure.

To use/apply the fonts into any html page, we just need to add a <link> in ....
Example:

    <link rel="stylesheet" href="/css/fonts.css">

Question. Is the first slash in front of css needed? Is it a different location from css/fonts.css? Or does it really refer to the same place? I'm actually not sure.

I added the slash because now the html file is inside the pages folder. Without the slash it will look for a relative path. With the slash it will look in our root directory.

Perfect.  Thanks!


Can the maxWidth for the GoogleLogin button be made configurable?

Hello @MomenSherif i'm passing a width of 480px but the width of the button is 400 because of the set max width, and there's no way to override that. Could that be worked on? Thanks
<GoogleLogin useOneTap={false} size="large" onSuccess={responseMessage} onError={errorMessage} width={480}/>

yes same, please add also an attribute for maxWidth


consistency analysis

In order to verify roughly that we are not changing the way we annotate over time, we have agreed to try to see if having a small script analysing all the input file can be helpful.
The idea is the following, the script collect all the annotations and inform when the same token has been annotated with different classes. For example if President of the US has been annotated as PERSON and as PERSON-TYPE.
The idea is not to check the context, but purely the tuple (token, class)so we know that we might see many false positive.
the script has to be something script inputdirectory and should print out something like:
President of the US:
PERSON: preview of a chunk of sentence with the annotation PERSON (including the XML)
PERSON_TYPE: preview of a chunk of sentence with the annotation PERSON_TYPE (including the XML)

I'm currently doing it in Python 2.7. The way of how i'm doing it is just parse each xml files of the inputdirectory to catch all ENAMEX tag and put in a dictionary the token (annotation) as the key and classes as the value.
And i will just show all keys with multiple values.
Tell me if there is a misunderstanding or if there is a better way for doing this.
I'm finishing it probably tomorrow.

Looks good for me, you can use a Set as value of the map so you only add unique values


Update bundle (policy-controller)

Should this pull in a release version to avoid the verify-bundle failing randomly when we update anything in a different repo.
@maleck13 fyi

Yes agree. I have created a release-0.2 branch now and will update the kustomize to point at that


Facebook authorization without React-Native-WebView

is there a way to authorize app without WebView? for example if user has facebook app in own device, we just redirect to facebook app and then authorize?

Hi! It's default SDK behavior

ios https://github.com/facebook/facebook-ios-sdk/blob/master/FBSDKLoginKit/FBSDKLoginKit/FBSDKLoginManager.h#L78
android https://developers.facebook.com/docs/reference/android/current/class/LoginBehavior/

Hello, Yes, I see, but it still open WebView instead native FB-app, use platform IOS > 10

I published the new version! Try it, maybe SDK was outdated

yes, I see , webView is opened faster then before(new FB SDK), but it still open WebView :((

Is there any logs in xcode console?

and check you plist file
<key>LSApplicationQueriesSchemes</key>
<array>
		<string>fbapi</string>
		<string>fbauth2</string>
</array>

Both keys must exist

@Mactub07 Does it work?

I tested, It didn't help

mistake in console when close webview "Warning: Attempt to present <UIAlertController: 0x101273ea0> on <FBSDKContainerViewController: 0x1012778d0> whose view is not in the window hierarchy!" may be this can help

The error is just about presenting RN Alert component

Try to add fb sceme
<key>LSApplicationQueriesSchemes</key>
    <array>
        <string>fb</string>
    </array>

will try the same %) http://stackoverflow.com/questions/32566734/native-facebook-app-does-not-open-with-facebook-login-in-ios-9

Hm... I found this https://developers.facebook.com/blog/post/2015/10/29/Facebook-Login-iOS9/
So, it's default behavior at all (for >=iOS9)

btw , fb also don't help


django will cascade delete for us

The custom delete logic is no longer necessary as Django does the cascade delete for us now.  It's both more efficient and should prevent a problematic edge case we have seen.  We have seen cases where a resource is being deleted, but the same resource is then viewed.  Part of the view logic updates the view count on the resource object, so the resource object can be deleted by the normal delete logic but then restored by the view.  Django makes this process atomic, avoiding the issue altogether.

Pull Request Checklist:

[x] Positive Test Case Written by Dev

[x] Automated Testing

[x] Sufficient User and Developer Documentation

[x] Passing Jenkins Build

[x] Peer Code review and approval

Positive Test Case

Delete a resource with aggregations.  Ensure the aggregation and metadata objects are deleted when the resource is deleted.

METRIC
VALUE

NEW VULNERABILITIES
0

NEW SECURITY HOTSPOTS
0

NEW CODE SMELLS
0

NEW BUGS
0

https://sonarqube.cuahsi-workstation.com:9000/dashboard?id=hydroshare-4182


Added pylint as a pre-commit hook.

Removed separate github actions for running pylint.
Changes to be committed:
deleted:    .github/workflows/pylint.yml
modified:   .pre-commit-config.yaml

Pylint was raising line-too-long on setup.py. I had the following approaches

Exclude setup.py from pylint check totally using either pylintrc or exclude parameter of pylint pre-commit hook. This would have been the original behavior as well.
Disable the specific pylint check locally for that file
Refactor setup.py to get rid of that pylint issue.
Add the regex pattern to match only the .py files under nbqa and tests folder using files attribute in the pylint hook configuration.

I went with 2, because I thought its good to have other pylint checks run on setup.py. If original behaviour is desired I can exclude setup.py from pylint.

Legend, thanks @girip11 !

@all-contributors please add @girip11 for infra
@all-contributors please add @s-weigand for review

I've got to dash now, I'll figure out how to give all due credit later, but at least you both show up now

https://github.com/nbQA-dev/nbQA/pull/207#issuecomment-690953995 didn't work out, lets see if it works when you add a blank line in between.
@all-contributors please add @girip11 for infra
@all-contributors please add @s-weigand for review

Ok looks like you need to add people with one comment at a time.
@all-contributors please add @girip11 for infra

@all-contributors please add @girip11 for infra

@all-contributors please add @s-weigand for review

That much for NLP 😝

@all-contributors please add @s-weigand for review

Cool, thanks for figuring this out!
I was looking at https://allcontributors.org/docs/en/repository-maintainers and it says

You should encourage contributors to add themselves to the list as much as possible

You've both made contributions of different kinds, which I'm really grateful to you both for - so in keeping with the suggestion above, do you want to get the all-contributors bot to add anything else you've done (e.g. bug, code, doc, etc.)? Girish here's the key if it's not something you're familiar with https://allcontributors.org/docs/en/emoji-key

You could even add custom tags 😉
And I guess I'm fine ^^

@all-contributors please add @s-weigand for doc, ideas

@all-contributors please add @girip11 for bug, review

@all-contributors please add @girip11 for bug, review

@all-contributors please add @girip11 for bug, review


TypeScript support

I am interested in having type-safe resources support like Vue I18n. Is it possible to achieve this?
import { createI18n } from 'vue-i18n'
import enUS from './locales/en-US.json'

// Type-define 'en-US' as the master schema for the resource
type MessageSchema = typeof enUS

const i18n = createI18n<[MessageSchema], 'en-US' | 'ja-JP'>({
  locale: 'en-US',
  messages: {
    'en-US': enUS
  }
})

https://vue-i18n.intlify.dev/guide/advanced/typescript.html

I would love to review a PR with that capabilities.

@daniser do you think this is solved?
Thanks

I will leave this here if someone wants to contribute. Thanks @daniser 💪

@xiCO2k ,
Am I correct to understand that this plugin does not support TS for the moment?
Many thanks.

@mathiasselleslach, it should support, but probably missing some typing.

Thanks @xiCO2k .
Using a Laravel/Vue(3)/Inertia setup myself and cant' get it to work.
I am in a Typescript setup.
App.ts (as per documentation)
            .use(i18nVue, {
                resolve: async (lang: string) => {
                    const langs: any = import.meta.glob("../../lang/*.json");
                    return await langs[`../../lang/${lang}.json`]();
                }
            })

vite.config.js

export default defineConfig({
    plugins: [
        laravel({
            input: 'resources/js/app.ts',
            refresh: true,
        }),
        vue({
            template: {
                transformAssetUrls: {
                    base: null,
                    includeAbsolute: false,
                },
            },
        }),
        i18n(),
    ],
});

Am i missing something here?
When rendering
$t('test')

It just defaults to the key in the en.json file (yes, locale is set to en ;-) that i have checked)


Views on Golang as a primary development language

Reply from Tanay :
I think I have answered it multiple times. Great language. Future proof. If you have to learn one language for backend. Go for it. (pun intended)


uxn: init at 0.0.0+unstable=2021-08-27

Motivation for this change
Things done

Built on platform(s)

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

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

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

[ ] Fits CONTRIBUTING.md.

Oh, man, a whitespace?


Roll Engine from 32e3e844cd3a to 34578ab4e680 (1 revision)

https://github.com/flutter/engine/compare/32e3e844cd3a...34578ab4e680
2021-07-16<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from Cijd5dDSL... to FwgLM4h6t... (flutter/engine#27477)
Also rolling transitive DEPS:
fuchsia/sdk/core/mac-amd64 from Cijd5dDSLL-3 to FwgLM4h6tJNU
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


core[patch]: Fix JsonOutputParser returns empty json when text ends with "".

Issue: Close #20204
@baskaryan, @efriis, @eyurtsev, @hwchase17.

@eyurtsev This is a bug, please take a look, thanks.

@eyurtsev PTAL

Restore the original test, and then create a new test based on the streaming version and make sure that the JSON that's being streamed doesn't get mutated with additional \

closing due to inactivity


Audio devices

Is it possible to select one of the audio devices (webcam/microphone/no)?

I find this at webrtc


Mixpanel incremental workflow

Hi @kamikaseda,
embulk-input-mixpanel will have 2 more options to let user use custom column as incremental column, that column will be use as condition for mixpanel API query. This feature can also be used with workflow too. The flow would be:
td_run> run a saved query to calculate the latest ingestion timestamp, expose query latest resutls
--> Run Mixpanel td_load with YAML configuration file interpolated with that latest ingestion timestamp

I'm wondering if it's ok to merge this example workflow to this repository or not. If It ok then I will update README file. Or we should make another folder for this one to not confuse user.
Changes:
Add query to calculate max ingest timestamp.
Add incremental_column and latest_fetched_time to mixpanel configuration
Thanks

@taikhuu It's okay to merge! Could you merge this after upgrading embulk-input-mixpanel upgrade? I saw embulk-input-mixpanel repository, but it seems it's not upgraded yet.

I will merge after release embulk-input-mixpanel, also should update the README too

@taikhuu This is a reminder. Is this under developing?

@taikhuu This is reminder again!


notify event behavior

I have a confusion regarding notify. I have seen a related issue but I think this one is slightly different.
So, for promises if it is already resolved then I can use it anywhere in my code later & get the value immediately. And if is not already resolved then I will get the value once it is resolved.
But, If I attach a handler after calling the notify then I don't get the current progress of the promise if I use it later in my code.
Is there any way around? Or I didn't understand the concept. I have tried with jQuery Promise and it's working fine. I'm aware that jQuery Promises are very different under the hood. But, Still just to compare I have put it below.
Q
function get() {
    var deferred = Q.defer();
    deferred.notify('hello');
    setTimeout(function () {
        deferred.resolve('world!');
    }, 2000);
    return deferred.promise;
}

function doTheThing(data) {
    alert(data)
}

get().then(doTheThing, doTheThing, doTheThing);

jQuery Promise
function get() {
    var deferred = $.Deferred();
    deferred.notify('hello');
    setTimeout(function () {
        deferred.resolve('world!');
    }, 2000);
    return deferred.promise();
}

function doTheThing(data) {
    alert(data)
}

get().then(doTheThing, doTheThing, doTheThing);

One possible ugly workaround would be to add setTimeout.
function get() {
    var deferred = Q.defer();
    setTimeout(function() {
        deferred.notify('hello');
    });
    setTimeout(function () {
        deferred.resolve('world!');
    }, 2000);
    return deferred.promise;
}

function doTheThing(data) {
    alert(data)
}

get().then(doTheThing, doTheThing, doTheThing);


Add numbers to repeated sections/list items

It would be good to have repeated list items show numbers, so we can easily reference them without adding names to each file. Example of this in practice is the multiple image carousels within the product pages just using "Slide":

+1 - I think that every list item in the editor (including partials) should be numbered. @Zoramite can this be done within selective edit? And then CSS can be used to hide the number if desired.
For example:
Partials:
1. spacer
2. hero
3. spacer
4. 5050-simple

We can also update the {Item 1} (the curly braces look a bit funky) to 1. Untitled item and then we'll be consistent across the board.


Update to openai v2.0.0

https://github.com/openai/openai-openapi/releases/tag/2.0.0

Released in v0.8.0


[Event Request] codeunit 5763 "Whse.-Post Shipment" - OnInitSourceDocumentHeaderOnBeforeServiceHeaderInit

We would like to add a new event OnInitSourceDocumentHeaderOnBeforeServiceHeaderInit in the following trigger. Same way as it has been done in codeunit 5760 "Whse.-Post Receipt"
local procedure InitSourceDocumentHeader()
...
DATABASE::"Service Line":
begin
//A108F0000.Event18.03.ns
IsHandled := false;
OnInitSourceDocumentHeaderOnBeforeServiceHeaderInit(ServiceHeader, WhseShptHeader, WhseShptLine, ValidatePostingDate, ModifyHeader, IsHandled);
if not IsHandled then
//A108F0000.Event18.03.ne
if (ServiceHeader."Posting Date" = 0D) or (ServiceHeader."Posting Date" <> WhseShptHeader."Posting Date") then begin
ReleaseServiceDocument.Reopen(ServiceHeader);
ServiceHeader.SetHideValidationDialog(true);
ServiceHeader.Validate("Posting Date", WhseShptHeader."Posting Date");
ReleaseServiceDocument.Run(ServiceHeader);
ServiceHeader.Modify();
end;
Event Signature:
[IntegrationEvent(false, false)]
local procedure OnInitSourceDocumentHeaderOnBeforeServiceHeaderInit(var ServiceHeader: Record "Service Header"; var WarehouseShipmentHeader: Record "Warehouse Shipment Header"; var WarehouseShipmentLine: Record "Warehouse Shipment Line"; var ModifyHeader: Boolean; var ValidatePostingDate: Boolean; var IsHandled: Boolean);
begin
    //A108F0000.Event18.03
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Some words missing, and typos

Under 'Resources and Settings' the test says "Windows Forms projects also include Windows Form project specific files, such as Properties/Settings.settings and Properties/Resources.resx. These files may need to be migrated they are declared in your original project."
There are three problems here:

The forward slashes should be backward slashes. My search for these in the old projects file failed for obvious reasons.

There is a word missing after "migrated" - this is not good grammar and ambiguous. Should it be "if" or "as" or some other word?

"such as" is not much help. My fairly small VB project has quite a number of references to "My Project" - which of these need migrating, and which don't?

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

ID: 98778ad1-6c20-6310-e4bc-9d9851a2270a
Version Independent ID: d1ef1f43-00e2-b46f-4f38-6274a34f07f6
Content: Migrate a Windows Forms app to .NET 5 - Windows Forms .NET
Content Source: dotnet-desktop-guide/net/winforms/migration/index.md
Product: dotnet-desktop
Technology: dotnet-winforms
GitHub Login: @adegeo
Microsoft Alias: adegeo

Thank you for opening this issue. I'll get this fixed up.


NeighborSampler / DataFlow question

❓ Questions & Help
Hi, I am a bit confused about the definition of n_id in NeighborSampler's output blocks. From what I can tell, it should be the nodes referred to by edge_index. However, I verified by hand that if I take that as the definition, the nodes are not connected.
Basically, I have a large sparse graph which I then sample (single layer around each node) with NeighborSampler using:
sampler = NeighborSampler(Data(
    X,
    edge_index=edge_index,
    y=y),
    size=1.0, num_hops=1, batch_size=32, shuffle=True)

Then, when I take a sample like this:
for subgraph in trainloader():
    break

I observe that none of the nodes in subgraph.n_id are in subgraph.blocks[0].n_id:
np.isin(
    subgraph.n_id.numpy(),
    subgraph.blocks[0].n_id.numpy())
> array([False, False, False, False, False, False, False, False, False,
       False, False, False, False, False, False, False, False, False,
       False, False, False, False, False, False, False, False, False,
       False, False, False, False, False])

So, my question is: what is the difference between n_id in subgraph and blocks[0].n_id? They are both very large sets of numbers so I assume they are indexes for nodes in my original graph, but curiously neither overlaps.
Another way of solving this: is there a way to convert this subgraph (a dataflow) into a new Data object? I have been trying to do this manually but I'm not sure which n_id correspond to what edge.
Thank you!
Best,
Miles

Let me see if I understand it:
subgraph.n_id is the receiving nodes.
subgraph.blocks[0].n_id is the sending nodes.
subgraph.blocks[0].edge_index[0] indexes the sending nodes.
subgraph.blocks[0].edge_index[1] indexes the receiving nodes.
is this correct?
Thanks!
Miles

That is correct! However, I agree that the NeighborSampler interface was quite unintuitive. I completely revamped the interface in master. It is now much cleaner and way faster than before, even allowing for num_workers attributes to allow generation of subgraphs in parallel. Please have a look here and the examples and tell me if there are any questions.

Fantastic, thanks! I will install the latest version. Thanks for all your work, this library is awesome!!


open Feliz.Plotly overwrites built-in "box" function

Description
The "box" function exposed by Feliz.Plotly clobbers the built-in F# box function in F#. One needs it in order to box dependencies when using Feliz when creating stateful components. Opening this library makes the code much easier to read..Perhaps there is an easy resolution?
Steps to reproduce

Open Feliz.Plotly
Try something like React.useEffect(loadData >> Async.StartImmediate, [| (box state.startDate) |])

Expected behavior
Opening a library should probably overwrite any core language functions.

I managed to find the namespace and using the fully qualified name for box it works i.e. Microsoft.FSharp.Core.Operators.box but needless to say this doesn't feel less hacky.

Hi @mfrawley,
Yeah I think I agree, I'll get the API modified to rename box to box'. In the meantime you can also do state.startDate :> obj instead of boxing.

As a more elegant temporary fix, you can override box back right after opening Feliz.Plotly.
open Feliz.Plotly
// Feliz.Plotly shadows `box`: https://github.com/Shmew/Feliz.Plotly/issues/25
let box = Microsoft.FSharp.Core.Operators.box


Fix Bicep in VS build

Fix Bicep in VS build
Fixes #10722

Verified the generated assembly does not have a runtime dependence on Microsoft.CodeAnalysis.CSharp.CodeStyle


[BLOCKING] Unable to fetch the Bearer token

Running the following while connected to the bastion host shows error: no token is currently in use for this session
Steps to reproduce
ssh<EMAIL_ADDRESS>oc whoami --show-token
error: no token is currently in use for this session

We are blocked with further testing of the lab until this issue is resolved.
cc: @newgoliath @rhjcd

sudo -i; su ec2-user


Add a send all move event option

This allows sending all move events to help with hooking into libraries like react, by allowing them to process all of the events and run their own page specific functions on the event stream.

All these checks are necessary for correct work, can not ignore them.

It doesn't remove any checks? It adds a parameter (default false) that when flipped sends all move events instead of forcefully limiting the type of events that get sent.


Unhandled runtime error when choosing https://inrupt.net

Ran the server and attempted to access my pod via https://inrupt.net and received the bellow error
Unhandled Runtime Error Error: URL not found for given property
This is not the case when using https://solidcommunity.net pods

Hi,
To help us triage the issue you are having, could you provide some additional information:
What WebIDs were you trying to use on h

ttps://inrupt.net and https://solidcommunity.net?
When you say "Ran the server", what server are you referring to?
What are you trying to accomplish?

Thanks,
Kevin
Inrupt Support

Also reported (with one :+1:) here: https://github.com/inrupt/solid-ui-react-example-project/issues/22

We are deprecating this library, so unfortunately the open issues will not be addressed.


[TASK] Add github actions to test HERON

Issue Description
Is your feature request related to a problem? Please describe.
It would be useful if github actions were used to test HERON.
Describe the solution you'd like
Add github actions to run tests on windows and linux.
Describe alternatives you've considered
Continue to use civet.

For Change Control Board: Issue Review
This review should occur before any development is performed as a response to this issue.

[ ] 1. Is it tagged with a type: defect or task?
[ ] 2. Is it tagged with a priority: critical, normal or minor?
[ ] 3. If it will impact requirements or requirements tests, is it tagged with requirements?
[ ] 4. If it is a defect, can it cause wrong results for users? If so an email needs to be sent to the users.
[ ] 5. Is a rationale provided? (Such as explaining why the improvement is needed or why current code is wrong.)

For Change Control Board: Issue Closure
This review should occur when the issue is imminently going to be closed.

[x] 1. If the issue is a defect, is the defect fixed?
[x] 2. If the issue is a defect, is the defect tested for in the regression test system?  (If not explain why not.)
[x] 3. If the issue can impact users, has an email to the users group been written (the email should specify if the defect impacts stable or master)?
[x] 4. If the issue is a defect, does it impact the latest release branch? If yes, is there any issue tagged with release (create if needed)?
[x] 5. If the issue is being closed without a pull request, has an explanation of why it is being closed been provided?

Cleared to close via #231.


Fix stiff joints with runtime rigging

Despite using the seemingly best suitable Two Bone IK constraint, there is the following issue: the elbow joint does not bend in the pointing rig, leading the entire arm that is pointing at the target to look stiff.

I tried fixing this by changing the agent-related GameObjects' hierarchy and applying a different constraint (Chain IK). Besides, I attempted adding an optional Hint GameObject to the Two Bone IK constraint. Finally, I tested the constraint without the ThirdPersonObject components to make sure there are no animator conflicts. Unfortunately, none of this has helped so far.

Next up, I could try creating a different rig for the agent.


Updates in the eQTL Catalogue

Hi Brian,
I wanted to let you know about few changes that we have made to eQTL Catalogue that might also influence how catalogueR works:

In the release 3 of eQTL Catalogue we had and issue where AC and AN fields were accidentally swapped in the summary statistics files. Since the AN field is commonly used to infer sample size then this might have some influence on the colocalisation performance. This should not lead to false positives, but might have reduced the power to detect colocaliations in some instances.
The column order of the tabix-indexed files has now changed and unfortunately it is no longer consistent between the uniformly processed files and imported GTEx_v8 files. It is still consistent within all of the uniformly processed files. Sorry about that! Going forward, I think it is probably safer to also fetch the first row to make sure that the column names are correct.
We recently discovered the seqminer R package and have now switched to that in our colocalisation tutorial:
http://htmlpreview.github.io/?https://github.com/kauralasoo/eQTL-Catalogue-resources/blob/master/tutorials/tabix_use_case.html
We haven't done a lot of testing yet, but it seems to be more stable that using Rsamtools and does not require systems requests to tabix command-line tool. It is also a lot cleaner.

Best,
Kaur

Hi Kaur,
Thanks for much for all these updates! Apologies for the super long delay, I just realized that GitHub wasn't sending me notifications for this repo.

Good to know, thanks!
I actually made some updates recently that address this using the strategy you outlined.
I hadn't heard of this one! I'll have to give it a try. Might be worth rewriting the tabix portions of catalogueR to include this.

Thanks again,
Brian


[util] Cap Prince of Persia (2008) max fps

Game can get stuck when loading at framerates above 1000

Does 1000 really work? Shouldn't we go for something lower to make sure?

From my testing to far yes. But i don't mind going lower if you think. The clothes physics behave a bit twitchy at those high framerates anyway

360 fps is the state of the hard for competitive esports shooters, I think it's fine going with 240 or 360 fps for a casual single player action adventure.


Toggle functions not working

I am trying to create a toggle function for the taskbar, but can't seem to get it to work. Even using the example listed in the lua migration guide doesn't seem to work for me.
Here is the relevant parts of my init.lua
nog.config.display_app_bar = true
nog.config.remove_task_bar = true

nog.nbind("alt+control+i", function ()
    nog.config.remove_task_bar = not nog.config.remove_task_bar
end)

nog.nbind("alt+t", function ()
    nog.config.display_app_bar = not nog.config.display_app_bar
end)

In the log file it seems Nog is receiving the keybinding, but still nothing happens.
[2021-06-21 12:50:38.525798 -04:00] DEBUG [twm\src\main.rs:1085] Received keybinding Keybinding(ALT+CONTROL+I, 72, Work, 3073, None)
[2021-06-21 12:50:39.352874 -04:00] DEBUG [twm\src\main.rs:1085] Received keybinding Keybinding(ALT+T, 73, Normal, 1084, None)

This might be a problem with the hot reloading.

I did try to completely restart Nog a few times and still couldn't get it to work.
But I will say the hot reloading does seem to also be less stable. I can't really pin point what is wrong, but it feels like it is not always working especially compared to the nogscript hot reloading. Maybe it could be useful to display a popup notification to indicate that the hot reload has successfully taken place?

Some additional details. When I was in work mode and try to toggle the task bar on, nothing happens. When I exited work mode I noticed that my task bar was missing. I entered work mode again and toggle the task bar, and after exiting work mode the task bar was back. This same thing happens with the app bar toggle. It seems that it is not reflecting the toggles changes in work mode.

I noticed this too and started looking at it a little bit
I made a branch here that's like the initial work.
I noticed that..
A) the toggle/increment/decrement functions weren't hooked up
B) the hot reloading function is never called
In that branch I linked above I added the toggle function in and replaced the ReloadConfig event handler to instead call the hot reload just to test out that it works. I was making it toggle my app_bar like this
nog.nbind("alt+i", function()
    nog.toggle_config("display_app_bar")
end)

It works but only if I'm on a workspace that is actively managing windows. I'm not sure why that is yet...
@TimUntersberger  I added some comments on that commit I linked above

@ramirezmike the thing is the config object is actually a proxy that should automatically tell us when the config has been changed and then call the hot reload function. Having an extra toggle_config shouldn't be needed.

haha. hot reloading is working, but when we switched to lua I forgot to add the edge cases. I'll open a PR, so we can track this easily.

@ramirezmike the thing is the config object is actually a proxy that should automatically tell us when the config has been changed and then call the hot reload function. Having an extra toggle_config shouldn't be needed.

Oh, right, so, when I first looked at it I thought "the config object behaves like it's read only".
Like, after looking at the documentation I thought you could make a lua function that changes a value using the config table but when that didn't work I started looking at hooking up those functions. Are you saying that config table shouldn't behave like it's read-only?

It isn't readonly actually haha. The thing is the proxy is working, but we currently don't handle transitions.
Basically you are actually changing the config value, but we dont remove/add the appbar for example.
#277 will fix this.

cool cool. I'll delete the branch I made.


merge dev/extend_model_list into central dev branch

updates to acquisition functions and some house keeping details handled, incl some updates to CI/CD pipeline stuff
all tests passing

all tests have passed, no outstanding questions

closes #49

closes #94


list of items - A Road to MVP (minimum viable product)

This issue enumerates the work we need to do before releasing MVP for native Docker VMDK Data Volume support. Please extend the list if you see something missing.

[x] Discuss and review/finalize plans

[x] Finalize use cases/ clean up wiki / publish functional spec with use cases (depends on other items in this section)
[ ] * authentication / config: How do we identity trusted VM ? What is the config format (a list of "enable" glob expressions for .vmx config or VM uuid (think about vmotion ) ? Where do we keep the info ? Do we force port_id < 1024 ?
[ ] * policy  and quotas. Do we introduce VMDK grouping for this purposes? Format? Where to keep data?
[x] * packaging - ESX side (VIB, live install/uninstall, VUM updates in cluster, watchdog, jumpstart)
[x] * packaging - docker side (daemon config, minimal container with caps to run the plugin)
[x] * release timing and documentation  -- what do we need? Also,  release process - legals? docs? goto market? anything else ?
[x] * CI and test. (note: if needed, We can do scaffolding around cron/nimbus, and then go to drone.io)
[ ] * VC story: how to get a list of all VMDKs used (or explicitly requested) by docker VMs? Size? Check for stale VMDKs (i.e. created but not used in ages) ?  Etc...

[x] Clean up and review/fix of TODO in the code

[x] * Design and Use proper logs (ESX and linux), drop prints
[x] Add timing metrics for ops to logs for t-shooting
[x] * Handle time outs in calls to FindChild() and friends – it’s normal if we need to reconnect to hostd as session may time out.
[ ] * Discuss and Add support needed for K8S: "no formatting" option (or explicit formatting type request), location.
[x] * Add support for VSAN/Vvol. Currently mkfs uses -flat.vmdk. Need to check disk.backing and choose the backend appropriately (note: not needed when there is no FS formatting)
[x]  * Add proper PVSCSI controller/disks create() automation. Currently we rely on pre-configured VM, this is unacceptable and already mentioned in "installation issues".
[ ]  Make VMDK requests non-blocking: split Go/C API into SendRequest and GetReply (blocking), have ESX code spawn threads tor this and have Go use goroutines.
[x] Drop exec(“mount”) from Go - use syscall.mount().  Drop exec("vmkfstools") from python, use VirtualDiskManager
[x] * Add internal volume tracking per Docker spec - e.g. no need to attach stuff already attached by another container. Detach also happens on refcount only.
[x] * Other TODOs - signal handling, track volumes UUIDs for more reliable delete (currently full file name is the ID of the disk to delete), clean up sizes (drop fixed size) in C/vSocket code.

[x] * Design and implement build ,CI & testing set

[ x] Separate components (see "installation issues" Issue docker-vmdk-plugin#1). Needs to have

Go module (vmware/docker-vmdk-plugin/vmdkops) - uses 'go get' and 'go build' (in container)
ESX build (vmware/docker-vmdk-plugin/vmdkops-esxsrv) - uses vibauthor container
Docker plugin (vmware/docker-vmdk-plugin/plugin) - uses 'go' in container
container used for build only

[ x] Build: Make all container-driven, per above.
[x] DeployToESX for CI: assuming packaging (above) is done and ESX is deployed, govmomi or pyvmomi code to deployVIB, deploy docker VM (a few flavors) with plugin, run basic tests
[x] CI: GitHub, CI and test infra - deploy ESX (numbus? drone.io?) , run deployToEsx
[x] * unit tests. (_test.go) – includes a vmdkops replacemend module to do local "vmdk" (falloc , format, loopback mount) - that will test the rest of the logic.
[ x] use Photon  for build and Plugin container (low priority) - already supported
[ ] * Busy work: Support Python3 and keep Python2 (since newer ESX support only Python 3).

[ ] * Docker PR - Missing functionality (independent from MVP)

[ ] * 'docker volume inspect' is missing metadata passed with -o. So if we pass size or policy via -o, there is nothing about this in inspect
[ ] * 'docker volume ls' does not allow to enumerated driver-specific volumes which were not "created" In the current docker daemon. So there is no way to see what's already there for VMDKs.
[x ] Docker 1.10 seems to be addressing these already, see  "https://github.com/docker/docker/pull/16534 -  Move responsibility of ls/inspect to volume driver #16534 "   -(comment: not true. Still missing a way to pass our metadata)
[x] * It also adds support for anon. volumes (https://github.com/docker/docker/pull/19190) , we need to  support / test it

(note: * before the text indicates that there is a separate PR opened for the item

Liping will go through the list first, and then we should call a meeting to finalize the list.


Video treatment on the Asset Details page

One of our business teams is planning to upload a bunch of videos into AEM, and would like to make them available via their Asset Share for people to view.
When I upload a video into our AMS AEM 6.3 stage instance and view it on the Asset Share item details page the video cannot be played. Is there anything that needs to be configured within the Asset Share to allow a video to be played from the item details page, or is this something that's on the roadmap for a future release?
As an FYI, we're currently using Asset Share Commons version 1.1.2.

I just tested uploading a video to our AMS AEM 6.3 stage instance and the thumbnail shows up on the Asset Share search page and the item details page, but there's still no way to play the video from within the Asset Share.
With this noted, it looks like my local AEM 6.3 environment is missing something that keeps the thumbnail from being shown on the Asset Share search and item details page, so the only outstanding item is determining if it would be possible to configure the video to be played from the item details page.

As of now Asset Share Commons doesn't have any video preview/play functionality. I think it would be a new Video Renditions component that could play a preview of the video based on some pre-configured types. Marking this as an enhancement as others have asked about it.

@artdirk are you envisioning something like how the ootb Assets UI does it?

Danny, thanks for the quick responses!
If it could be implemented in both the search results card (as in your example) and the item details page that would be ideal, but if it could just be implemented in the item details page that would probably meet the business requirements.
Out of curiosity, do you have any ideas regarding a rough timeframe as to when this functionality might be added? This could be a show-stopper for this one business team unless we can communicate out any known plans...

@arpithaar this looks great!
Curious - what happens if a non-video asset is used on an Asset Details page with this component?
Also, which video file (rendition) will actually be played? Always the Original?

Hi @davidjgonzalez ,

If the video component is used for a non video asset, I tried to display the message like below. Should the message be authorable?

Yes, the original rendition is played . I guess its a good idea to make it authorable to select the rendition?

@arpithaar i think a authorable "not a video asset" is a good idea; keeps it simple but somewhat extensible. I wonder if the default styles should try to make it a grey rectangle to keep the "flow" of the page? thats just CSS of course..
It might make sense to have the rendition selectable - i think we do that for the Image component... im only thinking if the original is massive... this is really a "preview" of video; so if they have a smaller/lower bitrate version that might be better for this usecase - they can always download the full (big) original if they need that one.. wdyt?

Yeah. It makes sense to make the rendition selectable. Shall take care of that. Thanks @davidjgonzalez

@artdirk
Waiting to hear your thoughts.

Hi @arpithaar,
Thanks so much for providing this solution! Let me run it by my team and our business partners who will be using the video functionality the most and report back.
Art

@artdirk FYI - this has been merged as is - enhancements can of course be contributed for later versions.


Bug on clustering_DPA and its pure python version

Subject of the issue
This issue appears mainly when using small maxk values. When looking for the nearest element with a higher g, it may happen that all the elements in the self.dist_indices vector are still not assigned. This is a consequence of the last step while finding the centers, when we remove centers from list if they are neighbors of higher density points.
I think that I know how to solve the issue, so I will do it when I have time.
Your environment

OS: [e.g. iOS]
Python version: [e.g. 3.7.2]
Package Version [e.g. 0.1.2]
Anything else you consider helpful.

Steps to reproduce
Tell us how to reproduce this issue.
Expected behaviour
Tell us what should happen
Actual behaviour
Tell us what happens instead

To reproduce the bug:
download the data at:
https://figshare.com/ndownloader/articles/20317350/versions/1
run:
d = data.Data(distances=(dist, index), maxk = 50, verbose = False)
d.compute_id_2NN(fraction = 0.95)
d.compute_density_PAk()
d.compute_clustering_ADP(Z =1.65, halo =False)

I found the same bug on the data that I posted in order to reproduce it.
The data are distances and dist indices in (numpy matrices) of an hidden layer representation of ImegeNet.

@diegodoimo, could you add the line loading the data?

I'm acually working on it. I'll post a possibile solution asap. Before I added the lines to load it.

@diegodoimo, there are two possible solutions: One, that it would be the coherent with the paper description, is that when the nearest element with higher g is not in the neighbor list, compute the distances to all the elements with higher g and then take the minimum (but for this you would need the coordinates, not only the neighbor distances). The other option would be to ignore the restriction that you cannot have a center that is a neighbor of a point with higher g (this is a patch that would generate some inconsistencies but would allow to use only distances and indexes as input).

I posted an attempt of  solution at:
https://github.com/sissa-data-science/DADApy/pull/71
I don't think we should use the coordinates otherwise, the entire class would not work if one gives as input just the distances.
I assigned a point that does not have a neighbor already assigned to a cluster, to the closest center c* of higher density that has been assigned.

Was this issue solved via #71 ?

Yes


How to use the sm2_do_sign function.

I added the sm2_do_sign function in my test code,when I compiled the code with gcc, I got something as followd:
gcc -g -o sm2_sign_test sm2_sign_test.c -L/usr/local/lib64 -lssl -lcrypto
/tmp/ccUWXXJc.o: In function main': /usr/local/src/test_doc/sm2_test.c:108: undefined reference to sm2_do_sign'
collect2: error: ld returned 1 exit status

That is an internal function and you should not use that directly for standalone applications.
Use the EVP interface for SM2 instead. Here is a reference: https://github.com/openssl/openssl/blob/f81b043ad856d8b9af5239a4978f8bd4b965dab9/doc/man7/SM2.pod

I have done with this, thanks a lot!


Add installation and import information

Adds the information needed to find the maven packages and to import the adaptivecards object model and renderer

OPS Build status updates of commit a1f40d0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

AdaptiveCards/sdk/rendering-cards/android/getting-started.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Adding integration module for parent POM product ues

merged along with https://github.com/wso2/product-ds/pull/35


make%GRPC_STATUS% support snake case

Signed-off-by: Xuyang Tao<EMAIL_ADDRESS>Commit Message: make command operator %GRPC_STATUS% to %GRPC_STATUS(X)% to support snake case. The optional param X can be  CAMEL_STRING, SNAKE_STRING and NUMBER. If it is not set, it will be CAMEL_STRING.
Additional Description:
Risk Level: None
Testing: unit test added
Docs Changes: done
Release Notes:updated
Platform Specific Features:
[Optional Fixes #Issue]https://github.com/envoyproxy/envoy/issues/22307

@shuoyang2016 @qiwzhang @nareddyt

/docs

@TAOXUY thanks for this - a few docs nits inline
wrt to the X parameter - i think we should treat as a literal - it was confusing me reading through at first - its a kinda borderline case

Thanks, all done.

@TAOXUY Please do not force push, because it messes up code reviews.

/retest

/retest

Hi @wbpcode PTAL

LGTM overall. Thanks for your contribution. smile And some minor comment are added.
/wait

Thanks, all comments resolved!

Hi @phlax @cpakulski @wbpcode, could you please help submit this PR?


Demo: update dropdowns to using "fluid" styles

Labels were added to the dropdown on the component demo in https://github.com/carbon-design-system/carbon-website/pull/1626 but the default style causes type misalignment. The dropdowns should be updated to use the new fluid input style.

Replaced with storybook demo https://github.com/carbon-design-system/carbon-website/pull/3716


Preload css

Hi dev Grav, i have a question, is possible add rel="preload" in css?
It would be great to be able to use this tag to optimize performance

Not in the 1.5. In 1.6rc you should definitely be able to pass any attribute to the asset, including rel. Assets manager had been rewritten quite a bit.
I have to test it, though, I know rel=“stylesheet” was forced in 1.5 but I’m pretty sure that’s not the case in 1.6 anymore.

I do plan on documenting this for 1.6


Pin arrow to the left

Use Version

Headless UI: v1.7.7
Headless UI Float: v0.10.1
Framework: Vue 3.2.0

Describe the question
Thank you for this awesome library! It made integration super easy, I really appreciate it.
Is there a way to pin the arrow to the left? Right now the parent element is changing size and so the arrow is changing size, and I'd prefer to keep it pinned on the left. See the video below.
Thanks again!
Screenshots
https://user-images.githubusercontent.com/881931/213461634-465f12a4-e9b5-4b12-9700-9149df195758.mp4

Hi @aarondfrancis, can you try to see if using !important left 0 on arrow can solve it?
<FloatArrow class="absolute !left-0 ..." />

It's not the best solution, but I'll be think of a better on while.

Brilliant solution! Thank you 😄


HOCS-4383 - Move Case Deadline Calculations to Casework

Move date calculations to casework
Create endpoint to get deadlines

Just a few bits and bobs from me - overall great work and definitely a move in the right direction!

This is a great start, I'll raise what I feel the next steps are in the ticket, but lgtm.


Use default orthography if cookie is invalid

Instead of returning a server error page.

Codecov Report

Merging #942 (86cf7f7) into main (00530c3) will decrease coverage by 0.05%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##             main     #942      +/-   ##
==========================================
- Coverage   78.79%   78.74%   -0.06%
==========================================
  Files         137      137
  Lines        4692     4699       +7
  Branches      610      611       +1
==========================================
+ Hits         3697     3700       +3
- Misses        872      875       +3
- Partials      123      124       +1

Impacted Files
Coverage Δ

...tionary/morphodict/templatetags/morphodict_orth.py
100.00% <ø> (ø)

src/CreeDictionary/morphodict/orthography.py
87.87% <50.00%> (-12.13%)
:arrow_down:

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 00530c3...86cf7f7. Read the comment docs.


Added Docker aliases and functions

Separated functions from aliases from PR revans/bash-it#262 and added a couple additional ones.

Looks good - thanks!


vs code : Configured debug type 'coreclr' is not supported.

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce

Install .net6
open vs code
attach breakpoint on any line
click in run option
After that click on start debugg

Expected behavior
Debug must hit the line .

Actual behavior
C#
v1.25.0
Microsoft
18,003,040
(403)
C# for Visual Studio Code (powered by OmniSharp).
Set Color Theme
Disable
Uninstall
This extension is enabled globally.

Details
Feature Contributions
Changelog
Runtime Status
Categories
DebuggersProgramming LanguagesLintersSnippets
Extension Resources
Marketplace
Repository
License
Microsoft
More Info
Released on
26/02/2016, 21:50:58
Last updated
24/05/2022, 22:52:25
Identifier
ms-dotnettools.csharp
PROBLEMS
OUTPUT
DEBUG CONSOLE
TERMINAL

-------------------------------------------------------------------
You may only use the Microsoft .NET Core Debugger (vsdbg) with
Visual Studio Code, Visual Studio or Visual Studio for Mac software
to help you develop and test your applications.
-------------------------------------------------------------------
Unable to attach to CoreCLR. Unknown Error: 0x80131c3c

Error details
C#
v1.25.0
Microsoft
18,003,040
(403)
C# for Visual Studio Code (powered by OmniSharp).
Set Color Theme
Disable
Uninstall
This extension is enabled globally.

Details
Feature Contributions
Changelog
Runtime Status
Categories
DebuggersProgramming LanguagesLintersSnippets
Extension Resources
Marketplace
Repository
License
Microsoft
More Info
Released on
26/02/2016, 21:50:58
Last updated
24/05/2022, 22:52:25
Identifier
ms-dotnettools.csharp
PROBLEMS
OUTPUT
DEBUG CONSOLE
TERMINAL

-------------------------------------------------------------------
You may only use the Microsoft .NET Core Debugger (vsdbg) with
Visual Studio Code, Visual Studio or Visual Studio for Mac software
to help you develop and test your applications.
-------------------------------------------------------------------
Unable to attach to CoreCLR. Unknown Error: 0x80131c3c

Environment data
vs code in linux

Visuals
You may only use the Microsoft .NET Core Debugger (vsdbg) with
Visual Studio Code, Visual Studio or Visual Studio for Mac software
to help you develop and test your applications.
Unable to attach to CoreCLR. Unknown Error: 0x80131c3c

Seems like you found the right repo to file this (OmniSharp/omnisharp-vscode#5402) so I'm gonna go ahead and close this.


propertyIsEnumerable for...in Symbol

Request type

[ ] Please close this issue, I accidentally submitted it without adding any details
[ ] New documentation
[x] Correction or update

Details
propertyIsEnumerable

Every object has a propertyIsEnumerable method. This method can determine whether the specified property in an object can be enumerated by a for...in loop, with the exception of properties inherited through the prototype chain. If the object does not have the specified property, this method returns false.

Symbol

Symbols are not enumerable in for...in iterations. In addition, Object.getOwnPropertyNames() will not return symbol object properties, however, you can use Object.getOwnPropertySymbols() to get these.

so this code how explain
let a = {};
let b = Symbol('key');
a[b] = 'b';

console.log(a.propertyIsEnumerable(b)); // true

for (let i in a) {
  console.log(i) // there not `Symbol`
}

propertyIsEnumerable should say ...an object can be enumerated by a for...in loop, but not symbol

@chrisdavidmills Hi sir.
I can work on this to start my contribution journey with MDN. Can this be assigned to me? :rocket:

@vkWeb absolutely! Thanks so much.

@chrisdavidmills @Linbubin I have updated the Description of https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/propertyIsEnumerable.
@chrisdavidmills should we put a code example of this or the description alone will do?

@vkWeb your added note looks good, thanks! I've given it a small tweak, and put it in a note for more visiblity, but I think this is done now.
Congrats on your first contribution.


[Bug]: Duplicated channel sweeps

I closed a channel locally, and my reserve was already sweeped to my on-chain wallet, but the sweep is still shown as pending. (ps: channel was opened via the Bitfinex and Blocktank integration).

I later opened and closed another channel, and it happened again (fortunately, the 4900 sats sweeped from the previous channel is no longer appearing as "pending"):

Sweep needs 6 confirmations to become spendable when coop-closing. Currently improving this.


Configuration Interception

One additional feature that Tamaya provides is the possibility to intercept the evaluation of configuration. The main mechanism here is the PropertyFilter, which can apply changes to the raw String value, before applying any converter logic. This mechanism has shown to be very useful for security based filtering and implementing placeholder/resolver logic. The question is if that should be a vendor specific feature, or it should be part of the official API.

In MicroProfile Config we had this initially. It's basically the DeltaSpike ConfigFilter.
In MP we intentionally removed this after some feedback from Viktor Klang https://github.com/eclipse/microprofile-config/pull/1

I cant find the exact discussion on filters. must look at it on a pc ...
Am 31.10.2017 14:25 schrieb "Mark Struberg"<EMAIL_ADDRESS>In MicroProfile Config we had this initially. It's basically the DeltaSpike
ConfigFilter.
In MP we intentionally removed this after some feedback from Viktor Klang
eclipse/microprofile-config#1
https://github.com/eclipse/microprofile-config/pull/1
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/eclipse/ConfigJSR/issues/11#issuecomment-340761021,
or mute
the thread
https://github.com/notifications/unsubscribe-auth/AC5sczFP6mF5EcMlU8pCXeAvJMpINa4Dks5sxx_FgaJpZM4QMhGQ
.

I cannot recall the ConfigFilter discussion.

I vividly remember that we had the discussion. It was between Ondro, Viktor Klang and me mostly.
You can still find the contribution notice in our code:
https://github.com/eclipse/ConfigJSR/blob/master/api/src/main/java/javax/config/ConfigProvider.java#L27

Is this used for filtering out some properties? Can this be achieved by the implementation of ConfigSource? Basically read the properties and filter some out based on some criteria.

Let's close this and then reopen it at a later release if needed.


Multiple Sway syntax highlighting files

It seems there are several Sway syntax highlighting files. It may be best to condense this into a single file if possible to avoid inconsistencies.
There are at least 2 that I know of in the scripts directory, and one in the sway-vscode-plugin repo that is used by the LSP here which is necessary for syntax highlighting to work in VS code.

Snytax is my favourite Pokemon.

Snytax is my favourite Pokemon.

oops 😅


change font size bug

Challenge Change the Font Size of an Element has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.2; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/50.0.2661.102 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
 The code you have written is incorrect. Or is that the bug you are pointing at?

you dont need p {} tag inside you .red-text class just remove p and { ,}  you will be good. ;)


Broken syntax of the output CSV files for find_inactive_members.rb

The CSV output format of the script api/ruby/find-inactive-members/find_inactive_members.rb seems to be broken.
Output files:
inactive_users.csv
unrecognized_authors.csv
Observed Syntax:
<EMAIL_ADDRESS>Expected Syntax:
<EMAIL_ADDRESS>

I think I can get to the bottom of this, and will try to send a Pull Request to fix it.

@jonico thanks for merging the PR! I am closing this issue.


Docking panel in VR is emerged in NonVR mode

Docking panel in VR is emerged in NonVR mode.

fixed.  https://github.com/barrykui/VRmol/commit/8c4e6ee135803e06124a403d9418bb91951ee8a8


Main Thread Checker: UI API called on a background thread

I'm seeing the following issue when calling Sift.sharedInstance() during didFinishLaunchingWithOptions:
Main Thread Checker: UI API called on a background thread: -[UIApplication applicationState]
PID: 22781, TID: 4612768, Thread name: com.apple.CoreMotion.MotionThread, Queue name: com.apple.root.default-qos.overcommit, QoS: 0
Backtrace:
4   libobjc.A.dylib                     0x00000001818676f4 <redacted> + 56
5   CoreMotion                          0x0000000187fe1d9c CoreMotion + 294300
6   CoreMotion                          0x0000000187fe22cc CoreMotion + 295628
7   CoreMotion                          0x0000000187fe21dc CoreMotion + 295388
8   CoreMotion                          0x000000018801001c CoreMotion + 483356
9   CoreMotion                          0x0000000188010060 CoreMotion + 483424
10  CoreFoundation                      0x00000001825f627c <redacted> + 28
11  CoreFoundation                      0x00000001825f5b64 <redacted> + 276
12  CoreFoundation                      0x00000001825f0940 <redacted> + 972
13  CoreFoundation                      0x00000001825f0254 CFRunLoopRunSpecific + 452
14  CoreFoundation                      0x00000001825f0f88 CFRunLoopRun + 84
15  CoreMotion                          0x000000018800f9f4 CoreMotion + 481780
16  libsystem_pthread.dylib             0x000000018226e908 <redacted> + 132
17  libsystem_pthread.dylib             0x000000018226e864 _pthread_start + 48
18  libsystem_pthread.dylib             0x0000000182276dcc thread_start + 4
2019-07-16 17:29:15.213216+0100 CurveFeature1[22781:4612768] [reports] Main Thread Checker: UI API called on a background thread: -[UIApplication applicationState]
PID: 22781, TID: 4612768, Thread name: com.apple.CoreMotion.MotionThread, Queue name: com.apple.root.default-qos.overcommit, QoS: 0
Backtrace:
4   libobjc.A.dylib                     0x00000001818676f4 <redacted> + 56
5   CoreMotion                          0x0000000187fe1d9c CoreMotion + 294300
6   CoreMotion                          0x0000000187fe22cc CoreMotion + 295628
7   CoreMotion                          0x0000000187fe21dc CoreMotion + 295388
8   CoreMotion                          0x000000018801001c CoreMotion + 483356
9   CoreMotion                          0x0000000188010060 CoreMotion + 483424
10  CoreFoundation                      0x00000001825f627c <redacted> + 28
11  CoreFoundation                      0x00000001825f5b64 <redacted> + 276
12  CoreFoundation                      0x00000001825f0940 <redacted> + 972
13  CoreFoundation                      0x00000001825f0254 CFRunLoopRunSpecific + 452
14  CoreFoundation                      0x00000001825f0f88 CFRunLoopRun + 84
15  CoreMotion                          0x000000018800f9f4 CoreMotion + 481780
16  libsystem_pthread.dylib             0x000000018226e908 <redacted> + 132
17  libsystem_pthread.dylib             0x000000018226e864 _pthread_start + 48
18  libsystem_pthread.dylib             0x0000000182276dcc thread_start + 4

The issue occurs here:
+ (instancetype)sharedInstance {
    static Sift *instance;
    static dispatch_once_t once;
    dispatch_once(&once, ^{
        instance = [[Sift alloc] initWithRootDirPath:[SFCacheDirPath() stringByAppendingPathComponent:SFRootDirName]]; // <--- triggered here
    });
    return instance;
}

So far this has only been observed on an iPhone XR device (possibly other X type devices too). I couldn't reproduce on iPhone 6S, 7, and 8 devices.
Using Sift v1.0.1

Hi,
Thanks for reaching out. I'm attempting to reproduce with the HelloSift application included in this repo, the I'm so far unable to see the error. Could you please give me a little more information, specifically:

iOS version
XCode version

Can you also attempt to run the HelloSift app and let me know if you see this error there as well?
Thanks.

Hi @jkbach
Thanks for getting back to me so quickly. I ran the HelloSift app and saw the same warning in the console.
Sorry I forgot to add these details:

It only shows up on a physical iPhone XR device and not on the simulator. Interestingly my colleague tested on an iPhone XS device and could not see this warning.
iOS: 12.3.1 (for the iPhone XR)
XCode: 10.2.1 (10E1001)

Let me know if you need anything else.
Thanks again!

Hi, is there any update on this issue? I am also experiencing the same issue with iPhone XS (real device, not simulator), iOS 12.4.1, Xcode 10.3.

May be it is this issue --> https://openradar.appspot.com/46210367

@SL62 @umerasif is that issue still valid?

@SL62 do you know if this is still an issue?

Closing as outdated one