Login on Android hangs with "Redirecting back to the application"

I'm having a problem signing into Android. After displaying the authentication dialog and clicking the Authorize button, the "Redirecting you back to the application" screen is displayed. The circular indicator spins forever and never redirects my back to my application.
My same code is working good on iOS.
This seems similar to issue #69 , however the Developer account has been created for a number of months, and also I have tried both login and loginV2 functions on both versions 4.0.1 and 4.1.0-dev, with the same results.
Both OAuth 1 and OAuth 2 have been enabled.
The redirects on in Twitter are set to leofinance:// and leomobile://.
In my app/src/main/AndroidManifest.xml, I have:
<meta-data android:name="flutter_deeplinking_enabled" android:value="true" />
<intent-filter android:autoVerify="true">
   <action android:name="android.intent.action.VIEW" />
   <category android:name="android.intent.category.DEFAULT" />
   <category android:name="android.intent.category.BROWSABLE" />
   <data android:scheme="http" android:host="leofinance.io" />
   <data android:scheme="https" />
   <data android:scheme="leofinance" />
</intent-filter>

shawn@MacBook-Pro ~ % flutter doctor -v
[✓] Flutter (Channel stable, 2.8.1, on macOS 12.1 21C52 darwin-arm, locale en-US)
• Flutter version 2.8.1 at /Users/shawn/Development/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 77d935af4d (4 days ago), 2021-12-16 08:37:33 -0800
• Engine revision 890a5fca2e
• Dart version 2.15.1
[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
• Android SDK at /Users/shawn/Library/Android/sdk
• Platform android-31, build-tools 31.0.0
• Java binary at: /Users/shawn/Downloads/Android Studio.app/Contents/jre/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 13.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• CocoaPods version 1.11.2
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2020.3)
• Android Studio at /Users/shawn/Downloads/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
[✓] VS Code (version 1.62.3)
• VS Code at /Users/shawn/Downloads/Visual Studio Code.app/Contents
• Flutter extension version 3.29.0
[✓] Connected device (3 available)
• SM A025F (mobile)       • R9PR604VDCF                              • android-arm    • Android 11 (API 30)
• Shawn’s iPhone (mobile) • 2e950c689065bc8c3604deb71d053816d35793ba • ios            • iOS 14.8.1 18H107
• Chrome (web)            • chrome                                   • web-javascript • Google Chrome 96.0.4664.110
• No issues found!

@shawnlauzon  Please try to upate your maniefest file like this

@shawnlauzon  can you help me how can I call loginV2 function because I am not able to call that function

Hi @zoylee-dev I already had all of that, except that the scheme I used was leofinance instead of twittersdk (but I forgot to include the meta-data field; I'll add that now). I tried changing it, but it fails, presumably because I still have set leofinance:// as the redirect URL in the Twitter developers site. I'm fairly certain that changing it there is not going to resolve the issue.
As for calling loginV2, you need to first upgrade to the 4.1.0-dev release, and then replace the call to twitterLogin.login() to twitterLogin.loginV2().

I found the issue: I just needed to relaunch the Twitter app which was launched, and then the redirect worked fine :)

@shawnlauzon is it worked for you? i solved the return part but its returning null. wow the documentation here is so bad

@firstep-official yes it worked for me. I haven't looked at it since, so anything I know is in this issue. Hope it works out for you!


[DO NOT MERGE] - [BEAM-6698] increase maxHeapSize to prevent OutOfMemoryError (direct …

…memory)
Direct memory size is directly correlated to java heap space. As we significantly reduced maxHeapSize with upgrade to gradle 5 we increase here again to prevent test failures

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---
---
---
---
---

Java

Python

---

---
---
---

See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.

 So 4g seems to be sufficient.
I d like to get this in as to restore green state. Still looking into why we need so much direct memory on test runner side.
R: @mxm ptal

Ah, yes. Thanks for that pointer. Will look into that.

@mxm Could you help merging? I forgot to change title when asking for review.

Sure! Thanks for the fix.

Thx for merging!


Xiaomi HUB - mode ZHA or zigbee2mqtt

Someone can help me?
Model: lumi.gateway.mgl03
I can't tick the checkbox "[DANGER] Mode ZHA or zigbee2mqtt'
RESTART Gateway / Raspberry - done
Reinstall integration Xiaomi Gateway 3 - done
<Logger: aiohttp.server
Source: custom_components/xiaomi_gateway3/config_flow.py:236
Integration: Xiaomi Gateway 3 (documentation, issues)
First occurred: 10:31:40 (2 occurrences)
Last logged: 12:01:38
Error handling request
Traceback (most recent call last):
File "/usr/local/lib/python3.10/site-packages/aiohttp/web_protocol.py", line 435, in _handle_request
resp = await request_handler(request)
File "/usr/local/lib/python3.10/site-packages/aiohttp/web_app.py", line 504, in _handle
resp = await handler(request)
File "/usr/local/lib/python3.10/site-packages/aiohttp/web_middlewares.py", line 117, in impl
return await handler(request)
File "/usr/src/homeassistant/homeassistant/components/http/security_filter.py", line 60, in security_filter_middleware
return await handler(request)
File "/usr/src/homeassistant/homeassistant/components/http/forwarded.py", line 100, in forwarded_middleware
return await handler(request)
File "/usr/src/homeassistant/homeassistant/components/http/request_context.py", line 28, in request_context_middleware
return await handler(request)
File "/usr/src/homeassistant/homeassistant/components/http/ban.py", line 81, in ban_middleware
return await handler(request)
File "/usr/src/homeassistant/homeassistant/components/http/auth.py", line 236, in auth_middleware
return await handler(request)
File "/usr/src/homeassistant/homeassistant/components/http/view.py", line 136, in handle
result = await result
File "/usr/src/homeassistant/homeassistant/components/config/config_entries.py", line 241, in post
return await super().post(request, flow_id)
File "/usr/src/homeassistant/homeassistant/components/http/data_validator.py", line 73, in wrapper
result = await method(view, request, data, *args, **kwargs)
File "/usr/src/homeassistant/homeassistant/helpers/data_entry_flow.py", line 110, in post
result = await self._flow_mgr.async_configure(flow_id, data)
File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 281, in async_configure
result = await self._async_handle_step(
File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 368, in _async_handle_step
result: FlowResult = await getattr(flow, method)(user_input)
File "/config/custom_components/xiaomi_gateway3/config_flow.py", line 236, in async_step_user
raise Exception("Can't update zigbee firmware")
Exception: Can't update zigbee firmware

Check this #862


Add Isolation level in query logs

Hi! Since queries can be executed using different transaction isolations levels, I've found that it's going to be useful to see Isolation level in logs:
Name:northwind, Connection:2, Isolation:SERIALIZABLE, Time:3, Success:True
Type:Statement, Batch:False, QuerySize:1, BatchSize:0
Query:["SELECT * FROM products LIMIT 1"]

Name:northwind, Connection:3, Isolation:REPEATABLE_READ, Time:37, Success:True
Type:Statement, Batch:False, QuerySize:1, BatchSize:0
Query:["SELECT * FROM customers LIMIT 1"]

I've created a commit that adds isolation level to logging message. However, if you find, that Isolation information is too verbose for being included by default in the log message, I can improve ProxyDataSourceBuilder  like this:
   ProxyDataSourceBuilder.create(datasource)
              .logQueryBySlf4j(SLF4JLogLevel.INFO, "proxy ds - stats")
              .logIsolationLevel(true)
              .build();

Please leave your feedback, if you find this useful or if it needs to be improved.
Many thanks, Alexei Brinza.

Hi @sleepo581
Thanks for the suggestion and PR, it's a good idea.
Since isolation level is only set when user has explicitly specified it, I think it makes more sense to display it as optional.
Can you make the isolation level logging optional as suggested in the comment? Probably just logIsolationLevel() method on builder without boolean argument, since this method already indicates to include the isolation level in the log.
Also, please add a javadoc saying that this value is only available when the isolation level is explicitly defined. When a user doesn't specify the isolation level(using default level), the value becomes just empty.
Thanks!!

Hi @ttddyy!
I've added writeIsolation property to ProxyDataSourceBuilder. Now it's false by default, but can be configured.

Also, please add a javadoc saying that this value is only available when the isolation level is explicitly defined. When a user doesn't specify the isolation level(using default level), the value becomes just empty.

Even though isolation level isn't explicitly defined, it's retrieved from original JDBC connection and stored in ConnectionInfo (when connection is created through ProxyDatasource).
After that, if user changes isolation level, it's intercepted by ConnectionProxyLogic and stored in ConnectionInfo.
An example:
try (var connection = dataSource.getConnection();
     var statement = connection.createStatement();
     var resultSet = statement.executeQuery("SELECT * FROM customers ORDER BY customer_id LIMIT 5")) {
}

Name:northwind, Connection:2, Isolation:READ_COMMITTED, Time:1, Success:True
Type:Statement, Batch:False, QuerySize:1, BatchSize:0
Query:["SELECT * FROM customers ORDER BY customer_id LIMIT 5"]
Params:[]

Thanks @sleepo581 I just merged the PR.
I'll cut a new release but bear with me to take some time. Since bintray has sunset, I need to update my release script.


Please, clarify description for Isolate.kill(): default value for parameter priority

The description for method Isolate.kill() reads:

void kill({
int priority: BEFORE_NEXT_EVENT
})
Requests the isolate to shut down.
The isolate is requested to terminate itself. The priority argument specifies when this must happen.
The priority must be one of IMMEDIATE or BEFORE_NEXT_EVENT. The shutdown is performed at different times depending on the priority:

Since parameter priority is optional, it may be omitted. The method's behavior is not clear in this case. Please, clarify.

The parameter has a documented default value of Isolate.BEFORE_NEXT_EVENT, so if the parameter is omitted, it will use that value.
We usually don't repeat that information in the documentation - but perhaps we should.


feat(FX-3739): Size Filter Metric

The type of this PR is: Feature

This PR resolves FX-3739
Description

WIP

PR Checklist (tick all before merging)

[ ] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[ ] I have tested my changes on iOS and Android.
[ ] I have added tests/stories for my changes, or my changes don't require testing/stories, or I have included a link to a separate Jira ticket covering the tests.
[ ] I have added a feature flag, or my changes don't require a feature flag. (How do I add one?)
[ ] I have documented any follow-up work that this PR will require, or it does not require any.
[ ] I have added an app state migration, or my changes do not require one. (What are migrations?)
[ ] I have added a changelog entry below or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

It will also be necessary to update the logic here (now the user will always be shown the values in in regardless of his chosen measurement system)

previously selected metric value is automatically overwritten (even if filters have been applied) if the metric has been updated globally. As a result, the custom values in the size filter and saved search pills may be displayed incorrectly
Not quite sure if this is a problem and more like an edge-case, but I decided to describe it
https://user-images.githubusercontent.com/3513494/160455940-0cc252e5-9c43-4529-ad50-bafbc2edd4d1.mp4

Saved search pills is displayed in the unit that the user specified globally, and not in the size filter (if we need to do this as part of a separate task, then just ignore this message)
https://user-images.githubusercontent.com/3513494/160457137-76c2e604-bc43-4027-8a00-48d7538cc788.mp4

We discussed this with @dblandin and decided that it would be better for the user to have an experience like this:
scenario
- Collector's global metric preference is "cm"
- Collector visits artist screen > artworks grid
- Collector opens filter menu
- Collector opens size filter
- Collector sees metric toggle set to "cm"
- Collector filters by inches 50 min / 100 max width
- Collector clicks "Show results"
- Collector reopens filter menu > size filter
- Size filter still reflects options in inches 50 min / 100 max width
For this reason I did the changes in this commit.
The next commit just adds initial state to tests.


how can I change the default TCP service proxy_timeout ?

when I use the tcp-configmap with nginx-ingress,  I found the TCP service proxy_timeout was been set to 60s.    How can I change the default proxy_timeout to 600s ?

@loutian right now there is no option to do that. You can use a custom template and change the timeout manually.

@aledbf  Thanks bro!  I had solved it by use a custom template.   and I suggest that add the timeout option into the configmap.

@loutian care to share how you did it exactly? maybe via a gist?

@Dexyon   pls see nginx-ingress-controller-tmpl

Closing. Fixed in #1075


Private Catalog Use OAuth Token To Download Releases

Question
I am trying to get clarity that the OAuth option for private catalogs is used on the urls served from a index.yaml file.
index.yaml Example:
apiVersion: v1
entries:
  example-app:
  - apiVersion: v2
    appVersion: 1.16.1
    created: "2020-11-12T00:12:57.53060062-06:00"
    description: A Helm chart for Kubernetes
    digest: 573df9104c1b43802b9745221cc885fd9c572396ad13f077faf5456d3049928d
    name: example-app
    type: application
    urls:
    - https://github.com/Example/helm-charts/releases/download/example-app-0.1.0/example-app-0.1.0.tgz
    version: 0.1.0

I am using github pages to serve the index.yaml, and this works fine for rancher when it is a public catalog. When it is a private catalog I can add the catalog and it shows under the catalog apps correctly but when I go to launch it cannot download the urls to the chart packages.
If the url above https://github.com/Example/helm-charts/releases/download/example-app-0.1.0/example-app-0.1.0.tgz was behind authentication, is the desired behavior to use the OAuth token in the request?
Result:
I am not seeing this behavior and am getting a 404 from github (normal response when you try and access a restricted repo on github) when rancher is trying to pull the chart release down. The file is there and the chart works when I set the repository to public. So I know the url and the chart are not the issue.
Ideally it would be great if we could add support for this. But mainly I just want to know if perhaps I am doing something wrong or if it is indeed the intended behavior.
Environment information

Rancher Version: rancher/rancher v2.5.2

I used this documentation when setting up the private repo:
https://rancher.com/docs/rancher/v2.x/en/helm-charts/legacy-catalogs/catalog-config/#using-an-oauth-token

I just ran into the same issue. There seems to be no way to set the credentials from the UI however i found a workaround for now.
This is unsafe and should not be used in production environments.
You can change your catalog URL and use personal access tokens to get access to the repo by using this schema:
<EMAIL_ADDRESS>
Hope this helps as a workaround.


Issue from website page Create a horizontal list

From URL: https://flutter.io/cookbook/lists/horizontal-list.md
The "Complete example" does not have the horizontal scrolling inside. It is probably copied from another article.

The url you've provided gives back a 404.

Thanks for the report! This is a dupe of https://github.com/flutter/flutter/issues/14914 and fixed by https://github.com/flutter/website/pull/822. I'm bugging some peeps for more reviews!

Maintainers: This is now fixed on the website. Please close!


deploy fails if name in package.json contains a scope

deploy fails if name in package.json contains a scope (starting with @)
Expected Behaviour
deploy should succeed
Actual Behaviour
deploy fails
Reproduce Scenario (including but not limited to)
deploy a package with a scope
Steps to Reproduce
deploy a package with a scope
Platform and Version
current (master)

yes scoped packages are not supported right now, this is documented in the generated app README https://github.com/adobe/generator-aio-app/blob/master/generators/app/templates/README.md#packagejson

thanks for raising the issue @tmathern!
Feel free to add next ones in @adobe/aio-cli-plugin-app, so we keep them centralized


How to use this image to expose volumes to other containers?

Hi Justin, I'm trying to tail the docker logs from a Node app.
Of course the problem seems to access the files itself, and your project looks a promising step toward a solution.
Is there a way to have your project expose /var/lib/docker as a volume that could be inherited by other containers?

Why can't you just do docker run -v /var/lib/docker:/var/libdocker ... to share this?

Hej, I'm not sure what do you mean. Here is a pseudo-coded docker-compose that express what I aim for:
services:

  nsenter1:
    image: justincormack/nsenter1
    ...

  myservice:
     image: foo
     volumes_from: nsenter
     volumes:
       - /var/run/docker.sock:/var/run/docker.sock

"myservice" should then be able to access host's docker info and logs even on MacOS.
Right now I've build a small utility that tries to tail the log files (on linux) and falls back on child process spawning a docker logs -f that give or takes works, but I read around is less efficient compared to tailing the files.
Marco

@justincormack I have the same question.
When I run docker run -it --rm --privileged --pid=host -v /var/lib/docker:/var/lib/docker justincormack/nsenter1 or any other host:guest combo, the folders don't link.
If the folders exist on both the host and the guest, the data isn't mirrored
Any ideas?

I'm wondering if anybody has managed to extract data from host image.
I've obviously tried to mount a volume, but as the previous commenter, it doesn't work.
docker run -it --rm --privileged --pid=host -v /var/lib/docker:/var/lib/docker justincormack/nsenter1
I then tried to just docker cp stuff out of there, but again that doesn't work.
Being relatively new to the more techie side of Docker, I don't understand why.
Interestingly, doing docker cp nsenter-container:/etc . copies only a handful of files, not the whole /etc folder.
Does anybody reading this know more?

Hi I made another repo that will dynamically add mounts to containers from any other namespace, including root and another container https://github.com/justincormack/addmount - its not the most polished UI, but it does work.


Authenticated after refresh

How is the reducer keeping it's state, after refreshing the page? I checked through out the code and I'm not seeing how it's being saved.

You need to distinguish between a full page refresh and a client-side "refresh".
When you do a full page refresh - like when you enter the site from a new tab or hit F5 then the redux state is recreated. It takes the state from the server side (for example, the user), and starts to manage the state from that point. So if you have reducers running at this point (Fetching data or whatever) then the state will be updated accordingly.
When you do a client-side "refresh", its not a real refresh but rather a transition. This happens with the help of react-router and the "supporting modules" (All possible word combinations of react, router and redux 😄 ). So basically when you go to some other page, you dont go to the server and ask for the whole HTML from the start, you stay in the browser so the state is still there.

How is it getting the state from the server side? I'm using passport, redux, react and everytime i do a full page refresh, the states are gone and i have to log back in.

This is happening in middleware.js. As you can see, a new store is created with a initialState with the user. As you can see, it put the authenticated prop based on req.isAuthenticated() which is something that passport exposes when a logged in user session is created.
Once the store is created it is being used to run all the server side reducers. They change the store and once they finish running, the final HTML that will be sent to the user is built. As part of that, we save the state or redux store in INITIAL_STATE.
Once you reach to the client side, the client.jsx is triggered where we take the __INITIAL_STATE and use it to create the client side store - which contains the authenticated prop

Oh this is pretty cool. Does this mean this is server side rendering?

Exactly. The server side basically prepares the initial HTML that the client should render.
This means that once the client gets this HTML, it only has to render it, it should not run a lot of code to get to the same final html. This should reduce the time until the user actually sees the page.
Of course it will not prevent stuff that happen on the client side only - like stuff that happen in componentDidMount (which is run only on the client side)
Please bear in mind that although this is a cool feature and usually allows performance, it does make things more difficult when developing and creating new features because you have to be aware when where and how stuff happen. For example, there is an attempt to update the react-router module to its latest version and it did not prove to be trivial as to just update the package, change a couple of lines and move on.

We're trying to clean up the project and close issues.
In case you still have an issue with this, please comment here or open a new issue.


Conda env not activated

I am a bit confused by the way environment.yml works.
I tried to install R and SageMath from conda-forge, with
name: sage-r-environment
channels:
  - conda-forge
dependencies:
  - python=2.7
  - sage
  - r-essentials

but when I launch the Jupyter server, neither R nor SageMath notebooks are available. SageMath also does not start from a terminal (it complains about some env variables not being set).
It seems that the reason is that the conda environment is not activated prior to launching the Jupyter server. The fix for me was to add a start file:
#!/bin/bash

source /srv/conda/bin/activate kernel
exec "$@"

Also, the fact that the environment is named kernel instead of sage-r-environment is surprising.
Demo repo is here: https://github.com/defeo/sage-r
Is this a bug, or am I doing something wrong?

hmmm that is strange - you shouldn't need to do any source stuff in a start file. E.g. here's the binder-examples conda repo:
https://github.com/binder-examples/conda/blob/master/environment.yml
Did you resolve this issue in the end, or still having problems with it? It feels like a bug to me...

Well, my resolution, for the moment, is to source activate. I had the same problem (and same resolution) in a related deployment based upon docker-stacks: https://github.com/defeo/jupyterhub-docker/blob/master/jupyterlab/Dockerfile
Maybe somenthing's wrong with the sage conda package. @saraedum, what do you think?

I think it's because the environment specified is Python 2.7.
repo2docker doesn't officially support Python 2.7 (though, you can get it to work like you did). By default, it starts the notebook server in a Python 3.6 environment.
See the "Which versions of Python are supported?" docs, specifically this part:

Additionally, if Python 2.7 is specified, a separate environment for the kernel will be installed with Python 2. The notebook server will run in the default Python 3.6 environment.

Here is a minimal example of a environment.yml specifying Python 2: https://mybinder.org/v2/gh/betatim/python2_binder/master
This works as advertised no? If you don't want to use conda you can specify the Python version via runtime.txt.

If you add sage to the environment you also get a Python 2 and Python 3 notebook choice: https://mybinder.org/v2/gh/betatim/python2_binder/with-sage
Should installing the sage package result in an additional kernel being available?

Yes, an additional SageMath kernel should be there. Contrast with https://mybinder.org/v2/gh/defeo/sage-r/master

I think registering new kernels is the job of the conda package so we should report it there. I think other kernels manage to set themselves up correctly so my guess would be that this is something to do with what conda does/doesn't do when you run source activate and how the sage package configures/not configures itself.

Another issue is that in postBuild, the conda env is not activated. I'm not sure if that belongs here. If not, I'll open another issue

@isuruf that sounds like something for a new issue. I think it would surprise the user that the environment with their dependencies isn't activated so we should document it and/or fix it.

@minrk did a bunch of work recently around activating conda environments at startup. Not sure if that applies to postBuild though. You can set your first line to #!/bin/bash -l to run a login Shell, which should activate the environment for you


Set valid open state after success connection

Bug summary:
When Link is rendered in a component that is a Route element and Link was connected successfuly unless 1 time, link iframe is not destroyed after going to another Route.
Preconditions:

Simple App with BrowserRouter
2 page:

first with Plaid Link (route /) and navigation link to second page
second should be blank, used only to emulate Route change (route /test)

Steps to reproduce:

Use Link and connect to any Bank
Enter valid credentials, start auth
Should connect successfully & press Continue button to close the Link

Expected:
After changing route from / to /test (using navigation link on first page), Plaid Link iframe should be destroyed
Actual:
After changing route from / to /test (using navigation link on first page), Plaid Link iframe is not destroyed and affects other pages elements with artefacts
As I investigated, factory open state is not set to false after Link is successfully connected and closed via Connect button. So, when the Link was used at least once, the open state is true and iframe is hidden. As a result, plaid exit method skip calling its callback argument as state.open is still true.

hey @SnapeEye we recently just fixed an issue with exit() not working correctly (#242), and a few weeks ago fixed an issue related to how the iframe cleans up after itself on destroy (internal, not in this repo). Can you update to v3.3.1 and let me know if this is still an issue for you?

Hi, @skylarmb
Updated to 3.3.1, tested it - the issue is still here.
Used the Link, changed the Route - iframe is still in DOM. Moved backward - got 2 iframes in DOM.

@skylarmb
Here is the proof. IDK what was changed for current version, but open state is still not being updated after the Link is closed.


Fixes \th interpolation

For some mystical byond comprehension reason byond will convert 4.52 to 4. should fix that without creating any extra bugs.


Public Comment: Update text of the selected peak period or the selected Time of Day when user selects option from Drop Down Menu.

Gil is working on the display fix.


-x hip Should have same C++ std as -x c++

Does this repo change the default of -std=c++<...> for clang++?
I am trying to transition to the clang++ + hip::device target logic (#2275) with ECP AMReX & ECP WarpX.
The compiler identifies as Clang 12 but it builds by default in C++11.
My targets set
target_compile_features(${tgt} PUBLIC cxx_std_14)

and only with this clang++ in hip 4.1.0 I see no flags getting added to the command line.
Note that Clang 6+ already default to C++14: https://gist.github.com/ax3l/53db9fa8a4f4c21ecc5c4100c0d93c94
But -xhip does not it seems:
rocm-4.1.0/llvm/bin/clang++ -dM -E -xhip  /dev/null | grep -F __cplusplus
#define __cplusplus 201103L
#define __cplusplus 201103L

Vs.:
rocm-4.1.0/llvm/bin/clang++ -dM -E -x c++  /dev/null | grep -F __cplusplus
#define __cplusplus 201402L

My Workaround
export CXXFLAGS="-std=c++14"

before all CMake builds, because the target compile feature cxx_std_14 is currently ignored for -xhip objects.
Alternative work-around: CXX_STANDARD always adds -std=c++XX, even if the compiler default fulfills it
set_property(TARGET mytarget PROPERTY CXX_STANDARD 14)

The reason for that is that cxx_std_14 asks for feature support (at least a minimal version) while CXX_STANDARD is an exact request for a version.

Does this repo change the default of -std=c++<...> for clang++?

This repo doesn't change the C++ version. That is set by the llvm compiler. Using -x hip sets the version to C++11 in llvm because that is the minimum version needed to use hip header files, but this can be overwritten with passing a different version like -std+c++14.
At the time, I believe hip was created, llvm used C++98 by default, which is why we bumped it to c++11. When llvm switched to C++14 by default, it seems they didn't update it for all languages.

and only with this clang++ in hip 4.1.0 I see no flags getting added to the command line.

That sounds like a bug in CMake. The cxx_std_14 should add at least the -std=c++14 flags(or later).

Hi @pfultz2,
thanks for the background.

At the time, I believe hip was created, llvm used C++98 by default, which is why we bumped it to c++11. When llvm switched to C++14 by default, it seems they didn't update it for all languages.

Makes sense, I think we should now update this to C++14.

That sounds like a bug in CMake. The cxx_std_14 should add at least the -std=c++14 flags(or later).

Depends, CMake does he following logic for all languages it supports: If the default compiler flags are fulfilling the requested compile feature, it will not add (verbose) extra flags. Since your CMake logic exposes itself as CXX targets (HIP is not yet a in CMake), it will queryclang++ -x cxxand see it supports C++14. CMake does not know that your configs inject an-x hip` and that this is a different language.
One can of course build work-arounds in CMake, but the least confusing for all downstream would just be to keep the defaults for -x hip and -x cxx the same, imho.

Maybe @yxsamliu can provide feedback about upgrading llvm.

I think it is reasonable to take c++14 as default language standard for HIP. I will try making that change.

I think it is reasonable to take c++14 as default language standard for HIP. I will try making that change.

I've already done it for hipify-clang. So, the minimum GNU C/C++ supported version now is 6.1 (was 5.4).

Thanks! Just to clarify: With this issue, I am not suggesting to drop C++11 support.
(That said, our projects are C++14 and newer anyway.)
I just would like to keep the -std=c++XX default for the hip and c++ compiler frontend in sync.

understood

Upstream PR in https://reviews.llvm.org/D103221

cc: @kmaehashi @takagi

This is now fixed, thanks a lot!

https://github.com/llvm/llvm-project/commit/f7e87dd6ff0c1c4a41a6b654d2460557ae97ab8d


Move from php-http/client-implementation to psr/http-client-implementation

As per https://github.com/php-http/httplug

New client implementations and consumers should use the PSR-18 interfaces directly. In the long term, we expect PSR-18 to completely replace the need for HTTPlug.

For projects using guzzlehttp/guzzle in version 7 or higher this removes the need to also require php-http/guzzle7-adapter.

Hey @Kingdutch changes for this have just been added to the master branch. We'll try this with a couple of different HTTP clients and if nothing comes up, release this later this week. I'll keep you posted.

This has been release as 1.28.0


Added support to work behind corporate proxy and configuration for low and high coverage values

Support to work behind corporate proxy.
Support to configure low and high values from coverage.

Merge new features.

Looks good I would like this to be merged to master. If @pamepeixinho is still maintaining this could you please take a look at this PR?


110_shutdown_test occasionally fails in CI

recent failure:
>>> Test ./110_shutdown_test.sh finished after 59.0 secs with error: exit status 1
Check scope exits cleanly within 10 seconds
9e780882ce9958163aafd85b3b53c72e4cb3af56d84852727f82cf16ab3e2854
Weave Scope is reachable at the following URL(s):
  * http://<IP_ADDRESS>:4040/
test #1 "docker_on host1-5353-0.us-central1-a.scope-integration-tests logs weavescope 2>&1 | grep 'app exiting'" failed:
	program terminated with code 1 instead of 0

Fixed via https://github.com/weaveworks/scope/pull/1888


Video restarts when exiting app

Not ideal when listening to albums on YouTube and switching apps

sounds like you need to manage how you use the app better. maybe put it in the background right away and it wouldnt be such a bother. why would you wait till your half way through an album to put it in the background anyway... just do that to begin with

Will this issue be resolved??

I agree with jacklefrost.
When I have finished watching a video (the video has stopped playback) and I exit to the homescreen the video begins to play from the start again. This is not how the app should function. It should realize that the video has completed playback and not begin playing again on exit.

Finally got around to setting up xposed on 5.1 for this extension, and yeah, any time you exit Youtube while a video is playing, the extension will kick in and play the video in the background, but it starts it from the beginning.
Locking the screen while Youtube is in the foreground does not restart the video, so I've been using it that way, but it would be nice to see this addressed. Re-entering the Youtube app picks up where the background player left off, but when minimizing Youtube, the video will start from the beginning.

The way to fix it is downgrade YouTube to 10.21.xx

The background playback of OGYouTube 1.0 (based on 10.31.55) still works without restarting the video. I turned off its own background playback and installed this module since the notifications are better, but I went back to the OGYouTube version because the restarts were annoying.


jquery 3.0

I'm getting:
carousel.js:16 Uncaught TypeError: this.clip.size is not a function
in my event log.

Which version are you using? The current 0.3.x or the older 0.2.x?

Jquery 3.0 beta 1. The problem is size() function which is deprecated. I solve problem by myself. size() > length (length is supported by jquery 1.0 >)
I rewrite the code. (https://jsfiddle.net/Lwx1h8aq/) - here is, minified version. (working with 3.0.x)
I write to you, to fix this problem for all other users which using this pretty carousel :)

aww.. about the carousel version, i don't know, maybe 0.2.x..

Sorry for the delay. I've released 0.2.10 which should fix this.

Guess this can be closed then ?

yep :)

@val4o0o0 Then feel free to close when your issue has been resolved :)


wrong tabs closing

There was a bug reported a  couple months ago and then corrected, where if there were multiple tabs, you'd close a record and assume that the record you were looking at (the highlighted tab) would close, but one of the other tabs would close instead.  So if you aren't careful you can close an unsaved record (since they are back to saying changes are unsaved again).
I'm in production, using Firefox on a Windows machine.  I tried to close aw ork record but one of the administrative  metadata records closed instead.

Sorry, it was actually the instance record that closed when i closed the work.  I'm using the monograph unnested templates.

And in attempting to close the work it, it closed all other tabs one by one until it's the only tab left.

Hi Greta, is this with using the "Close" button or using the little "x" on the tab?

Using the close button.  It's related to the "save and close" thing when it tells you you have unsaved changes.  THen you go ahead and click close and it closes the wrong one.  I haven't tried the x.
thanks
greta

From: michelleif<EMAIL_ADDRESS>Sent: Monday, May 11, 2020 11:22 AM
To: LD4P/sinopia_editor<EMAIL_ADDRESS>Cc: Greta de Groat<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [LD4P/sinopia_editor] wrong tabs closing (#2183)
Hi Greta, is this with using the "Close" button or using the little "x" on the tab?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/LD4P/sinopia_editor/issues/2183#issuecomment-626872584, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKRYGHZRHINWKHSX45EWON3RRA67JANCNFSM4M4QS6UQ.

I used the close button, which i would assume would work on the
highlighted tab.  I see that the x is available only on the
non-highlighted tabs.
thanks
greta
On 5/11/2020 11:22 AM, michelleif wrote:

Hi Greta, is this with using the "Close" button or using the little
"x" on the tab?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/LD4P/sinopia_editor/issues/2183#issuecomment-626872584,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKRYGHZRHINWKHSX45EWON3RRA67JANCNFSM4M4QS6UQ.

I'm trying to reproduce this but can't. Is it still happening after yesterday's new release? if so could you add a couple screenshots here?

Interesting, i opened some and then closed, and i didn't have the problem.  Then i tried making a change and saving, which got me the "you have unsaved changes."  Then when i tried to close the one that i had saved, which was the first tab, instead it closed the last tab.

to clarify, you made changes in Monograph Instance Un-Nested, and then saved, and then still got the "you have unsaved changes" message when trying to close the tab you had just saved? What version of Sinopia is your screen showing?

Yes, that was the one i made a small change in and saved and got the message.  Then i tried to close but the administrative metadata closed instead.  Sinopia version is v.2.0.21

Hi @gdegroat, thanks for the additional information and screen-shots. I believe I have replicated the problem locally and I have started work on a fix.

This bug seems to be back.

open two resource templates. In this example on stage I opened _PCC BF2 Work (Monograph) and _PCC BF2 Instance (Monograph) and the active tab is Work

add some data to the Work, and then attempt to close without saving

when you click Close in the warning screen, it closes the Instance tab instead of the active tab

see also #2057


🐛 Fixed some more error messages not fitting on narrow screens

Realised that I missed the login and reset password screens when I fixed this the first time...

Thank you!


docs: mention tighter fromEvent types

Description:
This PR mentions, in the changes doc, that prior to separating the fromEvent signatures by target type, it was possible to pass options to APIs that didn't support them. (The options are ignored in the fromEvent implementation for targets that don't support them.) Attempting to do this in v7 will effect a TypeScript compilation error.
Related issue (if exists): #6301

Closed in favour of #6313.


Add support for Generator<void>

Generator<void> is useful for creating a simple synchronous coroutine that can be paused and resumed with no other special functionality. As of now Generator<void> in a compile error.
Example usecase: a test driver function for testing a program that uses some sort of an asynchronous messaging mechanism
#include <QCoroGenerator>

struct Connection;
Connection* get_connection();
auto conn = get_connection();

struct Message;
Message currentMessage;

QCoro::Generator<void> test_driver()
{
    co_yield;
    check_message_ok(currentMessage);
    conn->sendMessage(...);
    co_yield;
    check_message_ok(currentMessage);
    conn->sendMessage(...);

    //
    // ...
    //

    co_return;
}

struct App;
App* create_app(Connection*);

int main(int argc, char* argv[])
{
    auto driver = test_driver();
    auto it = driver.begin();

    auto app = create_app(conn);

    connect(conn, &Connection::messageReceived, [&] (const auto& msg) {
        currentMessage = msg;
        ++it;
        if (it == driver.end()) {
            app->exit(0);
        }
    });

    app->exec();
}

While I get the idea, unfortunatelly this is not possible (at least not the way you outlined it). As per the standard, co_yield must be followed by an expression, just co_yield; is not a valid statement.
See Godbolt and the C++ standard.
For your usecase, you'll probably have to do with some empty dummy type.


stochastic sampling function is terribly wrong

Testing stochastic sampling.pdf

Testing stochastic sampling.pdf
I think its OK


gateway url has to have a closing slash

Description
If the gateway url does not contain a closing slash, the upload request will fail.
Example
const bzz = new BzzAPI({ url:https://swarm-gateways.net` });
bzz.upload('some text');
Fails with network error
const bzz = new BzzAPI({ url: https://swarm-gateways.net/ });
bzz.upload('some text');
`
Upload completes.
Expected behavior
The documentation should be specific about it (at the moment not even the examples contain the slash), or both cases should be handled.

What package are you using please?
Both @erebos/api-bzz-browser and @erebos/api-bzz-node should behave as you describe already.

I'm using @erebos/api-bzz-browser

What browser are you using please?
It seems its behavior with the URL interface is different from node, Chromium and Firefox that I tested.

This was caused by my modifications, closing the issue.


Mention the prior art: SimpleSchema, CollectionHooks, Collection Behaviors

Many people in the Meteor community are familiar with @aldeed's SimpleSchema and @matb33's CollectionHooks. @Sewdn's collection-behaviors is another prior art that also seems to have been used as source of inspiration.
After reading the documentation, Astronomy seems to reinvent all these three packages and use neither. A lot of work has gone into both SimpleSchema, CollectionHooks and Collection Behaviors on one hand, and Astronomy on the other. There are probably good reasons for not reusing the  existing packages, and why Astronomy decided to build its own schema, hooks and behaviors. Users would love to know these.
Thanks!

First of all, the first version of the package was created summer 2013 and it was about the same time the aldeed's package was created. That time no one known about simple schema. My package was not inspired by simple schema.
If I'm reinventing the wheel I would like to ask why I receive many comments that my package is far more better than simple schema? If you want I can post many of them here. But after using astronomy and simple schema, people can just choose their favourite tool.
Moreover the concept of MVC, behaviors, hooks is not new and wan't invented by aldeed. My package was inspired by PHP Doctrine library and when you look at my code you will see many similarities and you won't see these similarities between astronomy and simple schema.

Thanks, this is informative. I didn't know when Astronomy was created. Might be worth adding this little bit of history to the README.

After using astronomy and simple schema, people can just choose their favourite tool.

I'm a new user, and haven't used Astronomy. The majority of users will be in this category, and some of them will have used SimpleSchema. These users will want to know how Astronomy (which will hopefully become popular) is better or easier than Simple Schema (which is very popular).
A comparison will help.
For example, Flow Router explains how it's different from the much more popular Iron Router.

The first public version was named verin-model and you can find it here https://github.com/jagi/verin-model. The first commit was made Oct 26, 2013 however it was already working library, so I've started creating it few month before. It can be dated on mid 2013 :). Originally it was named zeitgeist-model because it was a model layer for Zeitgeist website and I decided to share it with a community. At the beginning it was simple package but with the time the original idea had been improved to be what it is now :).
But yes, I think writing a little bit about history of the package can be good idea. I will definitely do it :). Thanks for this suggestion.

The majority of users will be in this category, and some of them will have used SimpleSchema, These users will want to know how Astronomy /* ... */ is better or easier than Simple Schema

Yes probably you are right, but you know I don't like to say things like: "Hey guys, here is my package it's much better than SimpleSchema. Just leave your old tool because mine is better" :). I'm glad that people like it and I received many comments that it's actually better than SimpleSchema. Many people coming from other languages (especially PHP) are glad that they can use similar approach when creating model. Maybe I will try pointing differences without saying that my work is better.

which will hopefully become popular with a bit of marketing

Yes I need to make it more popular. People just need to know that it exists. I have an idea how to achieve that. I'm creating video tutorial about Astronomy. When I'm finished I will publish it on Twitter (with proper tags) and send it to the Meteor team. I hope that they will post info about it on their FB and Twitter profiles. But I don't want to promote to widely unfinished product. But I'm almost ready. The 1.0.0 version is closer and closer.
I will take your suggestions into account :). Thanks!

I like competition.

jagi :+1:

Ok I've added two sections History and Why use Astronomy?
I hope that I did not exaggerated :)
I'm closing this issue for now

:+1: !
BTW, out of curiosity, how/where were the testimonials received? I hope Atmosphere implements comments soon because that's exactly where these testimonials should go..

Great idea with the comments on the Atmosphere!
Right now comments mostly comes from GitHub issues and from Crater.io https://crater.io/posts/MT9dPLYiydEWmqYi5/comment/tCQH3LGahRQDhCkNs

Here's another one for you:
"If this package were around when I created SimpleSchema, I would have used it instead of creating SimpleSchema."
It looks pretty slick. People only like SimpleSchema because it was the first decent schema package with good test coverage for Meteor. (And because they want to use AutoForm.) They'll eventually move on.
Speaking of AutoForm, the AF schema logic could be abstracted pretty easily allowing this or other types of schemas to be used instead of SimpleSchema/Collection2. It's on the "someday" list.

:100: for @aldeed for that testimonial.

@aldeed I appreciate it :)
@rclai said that he will work on the package allowing transformation of SimpleSchema to Astronomy schema and vice versa.
This Friday I'm going on two weeks vacation but after that I will focus on Astronomy Forms package.

Thanks for all your work!
BTW another "prior art" package I hadn't mentioned: @dburles collection-helpers

I haven't gotten the chance to work on that transformation package yet. I was going to wait until Astronomy was more ready before I do that. Thanks again for making it easier to see the validation definitions.

@rclai sounds good. 1.0.0 is very close.

Very similar project: @emmerge:graviton by @mhwheeler and @fractallian.

Ok thanks I will add it to the list


Serve dynamics-nav images from MCR instead of Docker Hub

However, newer Microsoft images and tags are available directly from the Microsoft Container Registry (MCR) at mcr.microsoft.com. Search and discoverability of the images are available through Docker Hub, however docker pull, run and build statements should reference mcr.microsoft.com. For example, pulling the windows-servercore image:
docker pull mcr.microsoft.com/windows/servercore
source
The README for microsoft/dynamics-nav still references the Docker Hub instead of MCR by using docker pull microsoft/dynamics-nav:2017.

I will update the readme.
NAV images are available on mcr.microsoft.com/dynamicsnav:

Thanks for your quick response, @freddydk.


listener job and listener job update mongodb is not being properly ingested

After submitting to wordcount extractor, the mongodb “listener_jobs” collection  (summary for logs) has an incomplete entry ingested.
{
    "_id" : ObjectId("63f503adb427ca74dfbf9439"),
    "listener_id" : "ncsa.wordcount",
    "resource_ref" : {
        "collection" : "file",
        "resource_id" : ObjectId("63efe87d6d2f826653fa65fe"),
        "version" : NumberInt(1)
    },
    "creator" : {
        "id" : ObjectId("63a32586cdf079dcd6077a1a"),
        "email"<EMAIL_ADDRESS>        "first_name" : "Chen",
        "last_name" : "Wang"
    },
    "parameters" : {

    },
    "created" : ISODate("2023-02-21T17:47:25.291+0000"),
    "started" : null,
    "updated" : null,
    "finished" : null,
    "duration" : null,
    "latest_message" : null,
    "status" : "CREATED"
}

And there is no entries for “listener_jobs_update”

assigning this to myself, clowder2 needs to catch [DONE] message from pyclowder (and others) to properly update the timestamp fields.


querySelectorAll doesn't support namespaced attributes

Selectors targetting namespaced attributes (for instance, [*|href]) raise a SyntaxError.
Basic info:

Node.js version: v8.7.0
jsdom version: v11.3.0

Minimal reproduction case
const { JSDOM } = require("jsdom");

const dom = new JSDOM('<!doctype html><svg xlink:href="foo"/>');

dom.window.document.querySelectorAll('[*|href]');

throws:
    SyntaxError: The string "[*|href]", is not a valid CSS selector

although the selector is valid.
How does similar code behave in browsers?
It selects the svg element. See http://jsbin.com/relisatula/edit?html,console
See also #1587 (although I think this one is slightly different, as it not only affects XML documents but also HTML documents).

@dperini interesting nwmatcher feature request

@domenic / @rdeltour ,
the new "nwsapi" has support for this, the namespace grammar is supported in the validator and parser for both attributes and tag names, I could not test the results but at least no error should be thrown. The addition was necessary to pass the complete w3c test suite.
Please see here: https://github.com/dperini/nwsapi
I have not talked/publicized much the new "nwsapi", but it is ready, so I take the chance here ...
"nwsapi" is the new Selectors Level 4 API emulation library, the continuation of my "nwmatcher" project. It targets newer browsers and headless environments, it has support for most of the new additions in the Selectors Level 4 specification, namely:

L4 web-form validation related pseudo-classes
L4 dynamic and user action pseudo-classes
namespace grammar and elements resolver
throwing real DOMException SyntaxError
native closest method on elements
:scope reference pseudo-class
:matches(s1, s2) pseudo-class
:not(s1, s2) negation pseudo-class
install/uninstall methods to replace QSA
modular design allows future L4 extension
only 24Kb minified, 8Kb minified & compressed

I am especially happy about the unbelievable speed improvements, have a peek yourself here:
http://javascript.nwbox.com/nwsapi/test/speed/
the biggest improvements in speed have been implemented on the most weak areas of "nwmatcher" and other libraries, for example on comma-separated groups, the :not() pseudo-class and on tree-structural pseudo-classes (:root, nth child-indexed and typed child-indexed).
The comparison in the above speed test is against the latest version of "nwmatcher" (1.4.3) and the native browsers QSA, there is no game in comparing with other CSS Selectors library, for most selectors "nwsapi" already outruns native QSA implementations and I can still see there is space for improvements.
It would be nice to have your feedback after running the above speed benchmark, I used Sizzle own Selectors Test Suite as a start but rewrote parts of it. I need to do more work on that.
I also included the latest w3c/web-platform-tests in the GitHub release in the test directory.
Everything passes in the new and old testing I have, a few corrections and it will be ready for npm.

Ow wow, that's so exciting!!! This will make people really happy, and be great for jsdom. I'll definitely try to find some time to play around with it and give feedback.

@dperini Those speed improvements are incredible - not to mention all the features you've added! Fantastic work!

@domenic @Zirro @rdeltour thank you for sharing the excitement and for the support ...
I hope you can test "nwsapi" in "jsdom" environments and report about problems/inconsistencies.
Notice that the version on GitHub is 5/6 commits behind my local WIP that you are testing at:
http://javascript.nwbox.com/nwsapi/test/speed/
The reason is that I am still improving the nth/of-type and comma-separated resolvers to squeeze out every drop of speed  enhancements. So the on-line speed test is what will be the real "nwsapi" in a few more days as soon as I commit the rest of the WIP.
I am very happy about the speed enhancements, the 2x/4x figures I predicted were met and probably also under-estimated in some area, however be assured there is no trick, no caching of results and no use of native QSA.
The gains comes from a really good RTL strategy (inherited from "nwmatcher") which allows me to avoid having to sort, reorder elements and check for uniqueness, they are in fact already document ordered and unique since the beginning of the elements collection phase using the bare-bone gEBTN dom API or an efficient recursive RTL walk where gEBTN is not available, also not having to support detrimental IE browsers helps with speed and simplicity. Traversal API helps too.
Hope it works out of the box for jsdom, i did some cursory tests but will wait for feedback in case some fixes and tweaks were necessary. At the moment "nwsapi" passes all the tests at 100% including w3c latest Selectors Suite, throws the same errors/exceptions like the real QSA, it is easily auto installed as a full emulation and replacement, support extra callback on results and ...

Looking forward to that update, right now I have a lot of unit tests that I cannot run because of all the :scope query selectors. Any ETA?

@rraziel the :scope pseudo-class is quite new and I could do just minor testing.
Do you think you could use the current "nwsapi" as is to run your tests ?
Even running your unit tests in a normal browser would be enough.
It could be of great help and also speed up the release process.
If :scope is working I would try to port it to "nwmatcher".

@dperini I can try to help, although I'm not quite sure how I would go about testing it directly (I'm using jest, which eventually uses jsdom which uses nwmatcher, from what I understand).
Or you'd just like me to write tests for it in the nwsapi project?

@rraziel, if you have the time, writing tests for "nwsapi" would also help, as you can see I only have a couple of test for the ':scope' pseudo-class and they do not cover enough cases.
Maybe @domenic or @Zirro can help us here by providing a copy/fork of "jsdom" using "nwsapi" instead of "nwmatcher". That would help in transitioning from "nwmatcher" to "nwsapi".
If necessary I can already publish "nwsapi" on "npm".

This is still not working, although this time I think the problem is more likely to be in the jsdom layer, not the nwsapi layer. The selector no longer throws, but it doesn't match anything.
Test:
<!DOCTYPE html>
<meta charset="utf-8">
<title>querySelectorAll must work with namespace attribute selectors on SVG</title>

<svg id="thesvg" xlink:href="foo"></svg>

@domenic, @zirro,
in this case the reported issue is not due to jsdom but to nwsapi.
I just pushed a commit as an attempt to fix this in my nwsapi repository.
Care to try it out ?

@dperini Nice! It fixes this issue, though it created some new failures in the WPT dom/nodes ParentNode-querySelector-All-xht.xht with the selector [*|TiTlE] that appear to be related to case-sensitivity. [*|TiTlE] should not match

.

@zirro, yes you are correct ...
I forgot to make it case-insensitive !
Will fix this asap, thank you for testing it.

@dperini I tried the test again with the latest<EMAIL_ADDRESS>but it still fails. Any ideas?

Can you test with the latest code in my "nwsapi" repository.
The latest commits should have solved this specific issue.
I am ready to publish that on npm as 2.1.4 if you confirm.

Pushed new release 2.1.4 of 'nwsapi' on NPM.
Should fix this issue.

The test in https://github.com/jsdom/jsdom/issues/2028#issuecomment-473572499 still fails. I have posted it in a branch at https://github.com/jsdom/jsdom/tree/test-2028 if that makes testing easier.

@domenic
this issue should be resolved by commit: https://github.com/dperini/nwsapi/commit/f6509bcd434f7438ec1599430c579289928d6a77
It is ready for publishing but due to other changes I believe it deserve 2.2.0 version.
Waiting for your review and agreement before publishing on npm.

@dperini confirmed those commits fix the issue (and don't cause any new regressions). When you have time, a new release would be great!

@domenic  A new release is on the way, it may be next weekend I have the time for it.
Thank you for reviewing the two commits above.

@domenic @dperini there is still an issue with this. I have a case were there are backslashes in the attribute name. When I replace them with normal slashes it finds the node. Otherwise it fails.
I tried this in a non-namespaced context and it worked out fine

Please file an issue following the issue template. Comments on closed issues are not tracked.


🐛(front) force to display thumbnail while the video is not played

Purpose
We have a regression on the thumbnail display feature. Before we forced
the frame code to 0 on the first seeked event because it happened before
the initialize event. We changed how the initialize event is caught and it
can happen now before the first seeked event. To fix this we have to
check in the initialized event itself if we have to reset the
currentTime property.
Proposal

[x] Check in the initialize event if currentTime property must be reset

@lunika Not sure about the explanation in PR description. From what you explain, I understand that the initialize and and seek event are both called, and only the order in which they are called has changed.
Why does the order with which they are called matter? As long as we reset he time to 0 at some point, the thumbnail should still display?

@lunika Not sure about the explanation in PR description. From what you explain, I understand that the initialize and and seek event are both called, and only the order in which they are called has changed.
Why does the order with which they are called matter? As long as we reset he time to 0 at some point, the thumbnail should still display?

The way we determine is the player is initialize has changed. Before we were only listening the canplay event and this event always occurs after the first seeked event. Since 1fa49ae most of time the initialize is done before the first seeked event so we have to handle this case in the initialize method itself.

@lunika so if I understand now you do it both at initialize and at seek.

@lunika so if I understand now you do it both at initialize and at seek.

I check in both events and I reset only one time, the earlier.


Getting "Validation failed in API: certs failed to find cert that matched private key" for certain projects

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):
we were not able to reproduce the problem - nor to solve it
Result:
Randomly (after weeks of working properly) one of our project generates an Error when accessing it: Validation failed in API: certs failed to find cert that matched private key. We were unable to fix the problem and had to delete and recreate the project to get rid of the error.
Some weeks later another Project now has the same problem.
Other details that may be helpful:
We tried to investigate the problem and found out that:

Manually rotating keys didn't help
Other projects within the same cluster are NOT affected
Accessing the workloads within the affected project using kubectl works flawlessly - all namespaces, workloads and so on are accessible - workloads are still working properly. It seems to be a problem solely related to rancher's own "projects" concept.
This error occurrs when accessing the project's workloads via the Rancher GUI or the API for one of the projects "namespaced*" services - for example:

https:/rancher-host/v3/projects/c-<id>:p-<id>/namespacedbasicauths
{ "baseType": "error", "code": "InvalidBodyContent", "fieldName": "certs", "message": "failed to find cert that matched private key", "status": 422, "type": "error" }

The Problem is server related since every user in every browser (even in incognito mode) now has the same problem.

Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): v.2.3.1 (but occurred in 2.9.x already)
Installation option (single install/HA): single install

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): Custom - created by Rancher

Machine type (cloud/VM/metal) and specifications (CPU/memory): Cloud / several nodes affected

Kubernetes version (use kubectl version): v1.15.4 (occurred in earlier versions as well)

Docker version (use docker version): several docker versions affected

I might add: We don't care so much why this happens every few weeks (while it shouldn't, obviously) - we just need a way to fix it, other than to delete and recreate the whole project. We're at a point where this could be a showstopper for using Rancher as our K8S-Maintenance tool although we really love it.

We were finally able to fix the problem:
We assumed that this error had something to do with the inter-node-communication between the k8s nodes and/or rancher. This now seemed not to be the case:
We found additional log messages on local worker nodes like
Nov 13 12:09:50 my-hostname 7b73c950acf8[2161]: I1113 11:09:50.964852       9 store.go:387] secret my-namespace-name/my-service-name.local-tls was updated and it is used in ingress annotations. Parsing... Nov 13 12:09:50 my-hostname 7b73c950acf8[2161]: W1113 11:09:50.966579       9 backend_ssl.go:48] Error obtaining X.509 certificate: unexpected error creating pem file: tls: private key does not match public key
which pointed to a certain workload within our failing project. Using kubectl we first removed the ingress and then we also removed the secrets/certificates previously generated by cert-manager (acme/letsencrypt). We assume that something broke parts of the certificate retrieved by cert-manager (we are using jetstack-cert-manager-controller:v0.9.1 at the moment).
I hope this helps anyone who came across the same problem.


Auto import: dependencies listed in package.json should be prioritized

Background
When I'm importing a dependency, I never want to import an implicit dependency (one that is not listed in my local package.json). See how that works.
Motivation
Quoting TSLint and ESLint documentation, respectively:

Disallows importing transient dependencies and modules installed above your package’s root directory.

This helps prevent accidentally depending on a module that is present in node_modules as a result of being installed further down your dependency tree, but is not listed as an explicit dependency of your project.

Existing solutions
Both TSLint and ESlint offer rules and plugins to prevent importing implicit dependencies, but instead of importing one and immediately deleting it, it would make sense to exclude implicit dependencies from the list of suggestions so they don't get imported in the first place.

https://palantir.github.io/tslint/rules/no-implicit-dependencies/
https://github.com/lennym/eslint-plugin-implicit-dependencies

Proposed solution
Add a configuration option.
TSLint and ESLint allow their rules to be configurable in a way that allows for importing from:

devDependencies
optionalDependencies
peerDependencies

So the configuration could look like that:
{
  "typescript.preferences.importFrom": {
    "dependencies": true,
    "peerDependencies": true,
    "devDependencies": false,
    "optionalDependencies": false,
  }
}

However, it should be also possible to allow importing from aliased paths. If a project uses a mapping from ./src/ to 'src', then 'src' must not be treated as an implicit dependency.
In that case, it might be easier to reverse the situation and disallow directories as opposed to whitelisting them.
{
  "typescript.preferences.excludeFromAutoImport": {
    "dependencies": false,
    "peerDependencies": false,
    "devDependencies": true,
    "optionalDependencies": true,
  }
}

Potentially related: https://github.com/microsoft/TypeScript/issues/31199

It will be partially fixed with: https://github.com/microsoft/TypeScript/pull/32517 which will remove anything that is not added in package.json (there won't be any transitive dependencies in suggestions).
However more control after that will be great!

This feature is powered by the TypeScript project. Please file a feature request upstream


[Bug] Image cleanup job fails to delete the images

Current Behavior
https://github.com/cryostatio/cryostat-operator/blob/ac9929e51df515886d8c3018e8c8af574dd94da9/.github/workflows/test-ci-reusable.yml#L19
The clean up job fails to clean images:
finding latest tagged 0 packages matching regex pr-643-ac9929e51df515886d8c3018e8c8af574dd94da9. Also finding latest untagged null packages...
🤓 ignoring missing package error
🔎 found pkgVer pr-643-ac9929e51df515886d8c3018e8c8af574dd94da9...
🔎 tag pr-643-ac9929e51df515886d8c3018e8c8af574dd94da9 matches. VersionId: 137998401. Deleting...

...output-omitted...
1 tagged packages to delete. Keeping top 0...
🔎  1 total packages to delete...
🗑 delete 1 packages
  deleting package cryostatio/cryostat-operator-scorecard:137998401
Error: 🚨 Error ignored - HttpError: Package not found.

Anything else?

Hey @andrewazores, this repo has write access to the package/image cryostat-operator, cryostat-operator-bundle and cryostat-operator-scorecard rite?
https://github.com/orgs/cryostatio/packages

I wonder if this could be due to bump in version of action r26d/ghcr-delete-image-action. This repo uses 1.3.0 while cryostat uses 1.2.2.

This action runs show that packages: write permission is enough to delete package:
https://github.com/cryostatio/cryostat/actions/runs/6541061653/job/17762014670

Hey @andrewazores, does this repo has admin access to those packages/images (i.e. cryostat-operator, cryostat-operator-bundle and cryostat-operator-scorecard)? I think that should solve the issue.
https://github.com/cryostatio/cryostat/pull/1635#issuecomment-1700327190

Ahh right. Looks like it needs admin access to delete package as in the screenshot there. I guess this can be closed after switching settings?

Sounds good! Seemed to work here:
https://github.com/cryostatio/cryostat-operator/actions/runs/6551987436/job/17794613108


Task execution target file can no longer be outside of root directory when using PowerShell3 task execution handler

Environment
azure-pipelines-task-lib version: VstsTaskSdk.psm1 version 0.11.0
Issue Description
The target command does not allow the task execution target file to be outside of the task's root directory when using the PowerShell3 task execution handler.
Expected behaviour
We expect the task execution target file to get invoked.
Actual behaviour
An error message saying the task execution target file cannot be found is thrown.
Steps to reproduce

Create a task that uses the PowerShell3 task execution handler.
In the task.json's target command, specify a path to the task execution target file that is in a different directory.
e.g.

  "execution": {
    "PowerShell3": {
      "target": "$(currentDirectory)/Code/Wait.ps1",
      "workingDirectory": "$(currentDirectory)"
    }
  }

Logs
More details including file structure and output can be found in this StackOverflow question. I thought this was maybe intended behaviour and proposed updates to the documentation in #582 but it was determined this is a bug in the software.

Reading through the answer on the Stack Overflow post, perhaps the issue is not that the task execution target file needs to be in the root directory, but that the SDK ps_modules directory needs to be in the same directory as the task execution target file (Wait.ps1 in the example given above). If that's the case, then perhaps the documentation could just be updated to say that. Currently the docs say:

The ps_modules folder should be in the root of the task folder.

I think that's right. Technically, its not impossible for them to be at different levels, but I agree that its simplest for them to be at the same level, and that's what we should be recommending. Right now, if you follow the recommended path this will actually work because we do recommend putting the script at the root (mostly I think because its easier to reason about things) - https://github.com/microsoft/azure-pipelines-task-lib/blob/master/powershell/Docs/Consuming.md#taskjson-modifications - but I think it would be good to change the wording a little bit in case people don't follow that for whatever reason. I've added #602 to clarify

Awesome, thanks @damccorm . For myself, I was migrating my extension from a regular PowerShell execution handler to a PowerShell3 execution handler, and originally I had all of my "business logic" code separated from the "extension infrastructure" code by placing it in it's own directory. That's what led me to falling into this scenario. Thanks for updating the docs!


Added kops-utils-cp image

Issue #, if available:
Description of changes:
Attempt to fix this error:
Error: cannot determine hash for "https://eks-d-postsubmit-artifacts.s3.amazonaws.com/kops/1.27.0/images/kops-utils-cp-amd64.tar.gz" (have you specified a valid file location?)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve

/lgtm


Fix the bug of marketplace name must be unique

Signed-off-by: feihujiang<EMAIL_ADDRESS>Here checks that names must be unique, but when transactions apply, the store don't bind name to the transaction identifier,  this is a bug. This PR fixes this bug.

Hi @feihujiang when starting a validator with this pull request I get this error:
File "/project/sawtooth-core/extensions/mktplace/mktplace/transactions/market_place.py", line 514, in apply
self.Update.apply(store)
File "/project/sawtooth-core/extensions/mktplace/mktplace/transactions/participant_update.py", line 116, in apply
super(Register, self).apply(store)
File "/project/sawtooth-core/extensions/mktplace/mktplace/transactions/market_place_object_update.py", line 158, in apply
name = self._get_absolute_name(store)
File "/project/sawtooth-core/extensions/mktplace/mktplace/transactions/participant_update.py", line 110, in _get_absolute_name
name = "{0}{1}".format(store.i2n(self.CreatorID), self.Name)
AttributeError: 'Register' object has no attribute 'CreatorID'
Can you show the initial behavior that this PR is fixing?

@boydjohnson Sorry, I made a mistake, fixed and updated this PR. Please review again, thanks.

The bug occurs when building a transaction block which has more than a transaction with the same name. The validator invoking is_valid() could't check the error. This PR fixes this bug.


Weight/Priorities for groups

My use case is a pretty complex one. We have +30 player groups +10 staff groups +30 servers where most require specific permissions.
Therefor I'm having difficulty migrating from zPerms, as I can't define which group should overwrite another. Inheritance is already in use for the player and staff groups, but not working out for mixed groups.
E.g. a user can be part of multiple staff groups:
Group: player07
Inherit: player06
Inherit. player05
..
Group: Sr. Mod
Inherit: Mod
Inherit: Jr.Mod
Inherit: default
Group: specific
negates permissions based surroundings (user state (e.g. spectating, pve), server, world, region..)
player07 and Sr. Mod can coexist, they usually don't need to negate any permissions of the other group and inheritance tree. But the specific groups need to be able to overwrite all those. Ofc. I could create a group like specific_Sr.Mod_player07 which is the specific group inheriting Sr.Mod and player07, but that would require one group for every possible/appearing combination, which makes it impossible to manage as a human.
Therefor it would be great to have those group weights stored in database, on the actual group entity. Kinda like the prefix/suffix priorities.

Thank you :)


Merge bigchaindb-common repo into the bigchaindb repo

TODO:

[x] merge the code inside a subfolder of bigchaindb bigchaindb/common -- see PR https://github.com/bigchaindb/bigchaindb/pull/731 -- see PR https://github.com/bigchaindb/bigchaindb/pull/732
[x] merge the tests inside a subfolder of tests tests/common
[x] fix all the tests and problems with the merge
[x] move all issues from bigchaindb-common into bigchaindb
[x] move the docs to ?

Assigned the label common to all imported issues so its easier for us to track what was imported from bigchaindb-common

I don't think we need to bring the docs from the bigchaindb-common repo. AFAIK they were just auto-generated by cookiecutter. I did add a ← Back to All BigchainDB Docs link, but I think that's the only change I made. You can see the rendered bigchaindb-common docs at:
https://bigchaindb-common.readthedocs.io/en/latest/
The intro page (imported from the README) has a short list of features that Tim probably wrote but we don't need that either.

@sbellem I believe this is finished now?

@sohkai indeed

Some notes on how this merge was done, especially regarding:

merge the code inside a subfolder of bigchaindb bigchaindb/common
merge the tests inside a subfolder of tests tests/common

We'll assume the following directory structure:
$ ls
bigchaindb           common-implementation     common-tests

Cloning the repos to yield the above:
$ git clone git@github.com:bigchaindb/bigchaindb
$ git clone git@github.com:bigchaindb/bigchaindb-common common-implementation
$ git clone git@github.com:bigchaindb/bigchaindb-common common-tests

Have three shells, one for each repo:
shell 1: bigchaindb
$ cd bigchaindb/

shell 2: common-implementation (bigchaindb-common)
$ cd common-implementation/

shell 3: common-tests (bigchaindb-common)
$ cd common-tests/

shell 2: common-implementation (bigchaindb-common)
$ git filter-branch -f --subdirectory-filter bigchaindb_common/ -- --all
$ ls
crypto.py  exceptions.py  __init__.py  transaction.py  util.py
$ mkdir -p bigchaindb/common
$ git mv *.py bigchaindb/common/
$ git commit -am 'Extract common lib implementation'

shell 1: bigchaindb
$ git remote add common-implementation ../common-implementation/
$ git fetch common-implementation
remote: Counting objects: 6, done.
remote: Compressing objects: 100% (3/3), done.
remote: Total 6 (delta 1), reused 1 (delta 1)
Unpacking objects: 100% (6/6), done.
 Kudos


How to set and get the rotation of each joint

Hi there,
I would like to use your library for my inverse kinematics of Ergonomic project ,however the instructions are not very clear for a beginner.Mainly lie in the input and output of rotations.The questions can be listed as follows:
1、Before ik.solver.solve(solver)，should we set the rotations of root->rotation and target->rotation in global space ?
At the same time,should we set the rotation of child->rotation in local space?
2、After ik.solver.solve(solver)，I want to get the rotation of each joint in order to realize the inverse kinematics,but I don't know whether the rotation that I get is a global rotation or a local rotation?
3、After ik.solver.solve(solver)，whether the joint's rotation that I get is a delta rotation?
What's more,I have tried several forms of input and output of rotations, but the result is always unsatisfactory.Look forward to hearing from you.

Hey!

Yes, you should write the rotations to each node->rotation field before calling ik.solver.solve(). These rotations are all in local space. root->rotation usually is always an identity rotation is most game engines, but if you do want to set this you can. For the root node, local space is the same thing as global space. target->rotation is set in global space.

After calling ik.solver.solve() you get the rotations back from each node->rotation field in local space.

You do not get delta rotations, you get the "solved" rotations. That is, each node will have an absolute orientation that represents the solution. If you want to get delta rotations you can subtract the solved rotations from the initial rotations that are stored in each node->initial_rotation field.

Please let me know if this helped.
This library has undergone a lot of changes and I must admit I am no longer certain if V1.1 fully works as intended. If you are unable to get satisfactory results then there is an older API V1.0 which definitely does work, as this is the version used in the Urho3D game engine. However, it is very different.
Did you find the wiki page? Most of the information should be here: https://github.com/TheComet/ik/wiki/Tutorial-1.1

Thanks for your response!
According to your guidance,I updated my own program,But I encountered  a problem ,namely, after calling ik.solver.solve(),the positions I get back from each node->position can work well;however, I use the rotations which I get back from each node->rotation to set the joint angle ,but I can not get the desired posture.Now I suspect I didn't use the solved rotations correctly or other reasons.Pardon my asking,whether you have relative sample program?

Can you check if you have set the IK_ENABLE_JOINT_ROTATIONS flag? Without it, FABRIK does not calculate rotations.
solver->features |= IK_ENABLE_JOINT_ROTATIONS;

Right now there are no samples that directly use this library unfortunately. There are only Urho3D samples that use the library indirectly. You can find those here: https://github.com/urho3d/Urho3D/tree/master/Source/Samples/45_InverseKinematics

I have set the "solver->flag |= IK_ENABLE_JOINT_ROTATIONS;".
I plan to use the library(API V1.0) and try to dig in the Urho3D's sourse code.
Thanks again.If there is any progress, I will communicate with you in time.

Alright. The biggest difference in V1.0 is all of the node positions and rotations are in global space, and the function names are global rather than ik.group.func.
Hopefully it won't be long until I can release V2.0.

     Hey,I have downloaded and compiled the code of V1.0,**but I still encountered the issue that the solved rotations can't work**.I think there may be some sets or functions that I didn't use correctly.

    So,I paste the code written by myself, please help me to checkif there is an error. The following code is an implementation of a wrapper function.The input of this function is the initial data of each node and target, and the output is the calculated quaternion of node rotation and the corresponding positions.

//=========================Function Code==============================
void FabrikIKSlover::Slove(const vector& nodePosition,const vector& nodeRotation,
const FVector& targetPosition,const FQuat& targetPosition,
vector& newPosition_global,vector&	outRotation_global)
//nodePosition、nodeRotation、targetPosition and targetPosition are the initial data of the tree;
//newPosition_global and outRotation_global are the solved data
{
int nodeSize=nodePosition.size();

mysolver = ik_solver_create(SOLVER_FABRIK);;
mysolver->max_iterations = 30;
mysolver->tolerance = 1e-3f;

/* We want to calculate rotations as well as positions */
mysolver->flags |= SOLVER_CALCULATE_FINAL_ROTATIONS;
mysolver->flags |= SOLVER_CALCULATE_CONSTRAINT_ROTATIONS;
mysolver->flags |= SOLVER_CONSTRAINT_SPACE_GLOBAL;
mysolver->flags |= SOLVER_CALCULATE_TARGET_ROTATIONS;

root = NULL;
ik_node_t* parentNode=NULL;
for (int i=0;i<nodeSize;i++)
{
	ik_node_t* crtNode = ik_node_create(i);
	if(parentNode!=NULL)
		ik_node_add_child(parentNode,crtNode);
	else
		root=crtNode;
	crtNode->initial_position.v.x=0;
	crtNode->initial_position.v.y=0;
	crtNode->initial_position.v.z=10;
	nodes.push_back(crtNode);
	parentNode=crtNode;
}
eff = ik_effector_create();
ik_node_attach_effector(parentNode,eff);
eff->chain_length = nodeSize - 1; /* how many parent segments should be affected */

eff->target_position.v.x = 0;
eff->target_position.v.y = 0;
eff->target_position.v.z = 0;
//eff->flags |= EFFECTOR_WEIGHT_NLERP;
eff->weight = 1;
eff->rotation_weight = 1;

      // Set the original positions and rotations of nodes
for (int i=0;i<nodes.size();i++)
{
	ik_node_t*& crtNode =nodes[i];
	const FVector&	crtPosition=nodePosition[i];
	const FQuat&	crtRotation=nodeRotation[i];

	crtNode->initial_position.v.x=crtPosition.X;
	crtNode->initial_position.v.y=crtPosition.Y;
	crtNode->initial_position.v.z=crtPosition.Z;

	crtNode->initial_rotation.q.x=crtRotation.X;
	crtNode->initial_rotation.q.y=crtRotation.Y;
	crtNode->initial_rotation.q.z=crtRotation.Z;
	crtNode->initial_rotation.q.w=crtRotation.W;

	crtNode=crtNode;
}
     // Set the positions and rotations of target
eff->target_position.v.x = targetPosition.X;
eff->target_position.v.y = targetPosition.Y;
eff->target_position.v.z = targetPosition.Z;

eff->target_rotation.q.x = targetRotation.X;
eff->target_rotation.q.y = targetRotation.Y;
eff->target_rotation.q.z = targetRotation.Z;
eff->target_rotation.q.w = targetRotation.W;

/* Assign our tree to the solver, rebuild data and calculate solution */
ik_solver_set_tree(mysolver, root);
ik_solver_rebuild_data(mysolver);
//ik_solver_update_translations(solver);
ik_solver_solve(mysolver);
ik_solver_iterate_tree(mysolver);
   // Get the solved global datas ;
for(int i=0;i<nodes.size();i++)
{
            // Get the solved global positions of every node;
	newPosition_global.push_back(FVector( nodes[i]->position.v.x,nodes[i]->position.v.y,nodes[i]->position.v.z));
            // Get the solved global rotations of every node;
	FQuat quat( nodes[i]->rotation.q.x,nodes[i]->rotation.q.y,nodes[i]->rotation.q.z,nodes[i]->rotation.q.w);
	quat.Normalize();
	outRotation_global.push_back(quat);
}

}

Hey! I was going to test the code you posted a week or so ago just now but I think you deleted it.
Does this mean you got it to work? :)

Thanks!
I have corrected the code that I posted somedays ago and it can work,so I delete the code.
At the same time,I plan to study how to add constraints to joints,because I find that it(the virtual human) can go wrong all of a sudden without the constraints.


Not deployable as OSGI bundle

biweekly is in itself nicely deployable as an OSGI bundle. But starting from 0.6.0 it uses vinnie, and if I try to deploy vinnie as a bundle I get "Bundle-SymbolicName header missing, cannot install bundle" from Felix. It would be nice if vinnie was deployable, too - otherwise we are stuck with biweekly 0.5.0. Thank you so much!

Ok, I updated vinnie to use OSGi and published a new version Maven Central (2.0.2).  I also published a new biweekly build to Maven Central (0.6.2) which uses the new vinnie build.  Let me know how it works out.


question: can this be used with caching turned on?

It looks like the only way to selected an element is by ID.  But if I am caching the views in my Ionic app, won't this cause an issue?

It shouldn't cause issues, unless perhaps if your using multiple elements with same id

I will sometimes end up with multiple elements with the same ID.  I should be able to workaround that by concatenating something unique onto the end of each element ID.

If I understood correctly, that should solve it - as long as the selector is unique, there should be no problem

cool thx!


Ooops, do i need to jump lines to have markdown rendering here?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Anderson-Darling Test Metric

Feature that you want to add.
According to this, Anderson-Darling Test provided better estimate of goodness of fit. We would want to add this statistical metric.
Describe the solution you'd like

Add ADTestColumnMetricStrategy in strategy
Add ADTestMetric in statistical/metric

Files which are needed to be modified or added

strategy
statistical/metric

Additional context

Ref: Kolmogorov-Smirnov Test
Ref: Anderson-Darling Test

Resolved by @harsh020 in PR #11


host specific externModules

Is your feature request related to a problem? Please describe.
As briefly described in https://github.com/nrdxp/nixflk/issues/44#issuecomment-756378163 there is a problem with externModules if they are supposed to be host specific (like for nixos-hardware).
Describe the solution you'd like
A similarly simple solution to the externModules but host specific would be great.
Additional context
I created this issue because https://github.com/nrdxp/nixflk/issues/44#issuecomment-756378163 is closed but the problem is not fully solved yet.

Perhaps we could treat nixos-hardware as a special case and wire up a path via specialArgs.

Perhaps we could treat nixos-hardware as a special case and wire up a path via specialArgs.

indeed this seems to work, do you want a PR to add nixos-hardware to the template?
If so I would be happy to create one!

indeed this seems to work, do you want a PR to add nixos-hardware to the template?
If so I would be happy to create one!

That would be great

That would be great

I'll close this as nixos-hardware is now supported and users have a documentation for similar flakes, see https://github.com/nrdxp/nixflk/pull/95.

I'll close this as nixos-hardware is now supported and users have a documentation for similar flakes, see https://github.com/nrdxp/nixflk/pull/95.


Incorrect type (object instead of object?) in [NotNull] example

In the example use of [NotNull], the value parameter should be declared as object? rather than object.

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

ID: 9ae1afa1-7c5f-862b-08f9-23cf5928953e
Version Independent ID: 9ae2922b-5108-ed50-768b-30f08cc3fc30
Content: C# Reserved attributes: Nullable static analysis
Content Source: docs/csharp/language-reference/attributes/nullable-analysis.md
Product: dotnet-csharp
Technology: csharp-language-reference
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Thanks for noting this @ihorwill
I opened #24554 to address this.

I closed #24554 in favor of #23549, which also fixes this.


Fix handling of CSV as pages in Notion

Description
Fixes: https://github.com/dust-tt/tasks/issues/1290

Fixes the rendering of the section when inserting a database as page in Notion
Uses the non sanitized header when doing so to prevent replacing accents or other special characters in the document format.

Will test locally before merging:

[ ] use the small db I created to reproduce and sync it locally
[ ] check the content of the document locally

Risk
Low after local testing
Deploy Plan

deploy connectors

tested locally:
Testytest
Tags,Sétup,Name
Bar,Fo,Férie
Bar,Yù,A
Foo,Hey,A

looking good

The csv may be invalid (headers might have commas, etc...) but since it's the document format I think we're good

Excellent, the customer who reported the bug was asking about progress on that one :)

@spolu thanks for fixing. However, some existing tables are broken and won't be fixed until they change, right ? Do you see a way to identify and re-import theses specifically ?


⚠ St. Lawrence Global Observatory CKAN Sitemap has degraded performance

In 40c662a, St. Lawrence Global Observatory CKAN Sitemap (https://catalogue.ogsl.ca/sitemap/sitemap-1.xml) experienced degraded performance:

HTTP code: 200
Response time: 13003 ms

Resolved: St. Lawrence Global Observatory CKAN Sitemap performance has improved in 6216599 after 16 minutes.


Batch - It will return an error when use submit batch operation on table in Azurite

Error Description:
In Azurite, It will return an error when use submit batch operation on table.
The error is shown as following:

To Reproduce:
The demo code is shown as following:
import { TableClient } from "@azure/data-tables";

interface Entity {
  partitionKey: string;
  rowKey: string;
  name: string;
  price: number;
  quantity: number;
}

async function main() {
  const tableName = "batch";
  const client = TableClient.fromConnectionString("<Azurite-https-connection-string>", tableName);
  const partitionKey = "somPartition";
  const entities: Entity[] = [
    {
      partitionKey,
      rowKey: "A1",
      name: "Marker Set",
      price: 5.0,
      quantity: 21
    },
    {
      partitionKey,
      rowKey: "A2",
      name: "Pen Set",
      price: 2.0,
      quantity: 6
    },
    {
      partitionKey,
      rowKey: "A3",
      name: "Pencil",
      price: 1.5,
      quantity: 100
    }
  ];
  const batch = await client.createBatch(partitionKey);
  await batch.createEntities(entities);
  const response = await batch.submitBatch();
  console.log(response.subResponses);
}

main();

Error Track:
In Azurite, the response returned in Azurite doesn't have contentType field. When Azure Sdk handles the response, it will return error.
In Azurite, the response returned in Azurite doesn't have contentType field.
https://github.com/Azure/Azurite/blob/table/src/table/handlers/TableHandler.ts#L662

When Azure Sdk handles the response returned in Azurite, it will return error without the contentType field.
https://github.com/Azure/azure-sdk-for-js/blob/46a342ee298ae4b7a213827699bf4f289ed1d9fc/sdk/core/core-http/src/policies/deserializationPolicy.ts#L346

Expected Behavior:
In Azurite, it can submit batch on table successfully.
@jongio for notification.

Thanks for reporting this! Batch operation is under implementation by @edwin-huber.

Thanks for reporting this! Batch operation is under implementation by @edwin-huber.

Close this in favor of #493

azsdk-azurite


Previous button not working

Hi, thanks for this fine library. It's been performing well so far but I encountered a strange problem recently. I created a wizard with four steps, and everything works fine when i try to use previous and next buttons from step1 and step2. When i get to step3 it allows me go back to step2 but I cannot seem to access step1 any longer.
Here is the code for the wizard steps

        <role-step wizardStep optionalStep (stepEnter)="scrollToTop($event)"
              navigationSymbol="1"
              (selectedRole)="getSelectedRole($event)">
            <ng-template wizardStepTitle>

                        {{'REGISTER.STEP_HEADING.ROLE.TITLE' | translate}}

                        {{'REGISTER.STEP_HEADING.ROLE.DESCRIPTION' | translate}}

            </ng-template>
        </role-step>
        <step1 wizardStep optionalStep (stepEnter)="scrollToTop($event)"
              navigationSymbol="2" [wiz]="wizard">
            <ng-template wizardStepTitle>

                        {{'REGISTER.STEP_HEADING.PERSONAL_DETAILS.TITLE' | translate}}

                        {{'REGISTER.STEP_HEADING.PERSONAL_DETAILS.DESCRIPTION' | translate}}

            </ng-template>
        </step1>
        <step2 wizardStep optionalStep (stepEnter)="scrollToTop($event)"
               navigationSymbol="3"
               [showRole]="selectedRole">
            <ng-template wizardStepTitle>

                        {{'REGISTER.STEP_HEADING.PROFESSIONAL_DETAILS.TITLE' | translate}}

                        {{'REGISTER.STEP_HEADING.PROFESSIONAL_DETAILS.DESCRIPTION' | translate}}

            </ng-template>
        </step2>
        <step3 wizardStep navigationSymbol="4"
                (stepEnter)="scrollToTop($event)"
                [user]="selectedRole"
                (userDetails)="getThisUser($event)">
            <ng-template wizardStepTitle>

                        {{'REGISTER.STEP_HEADING.PAYMENT_CONFIRMATION.TITLE' | translate}}

                        {{'REGISTER.STEP_HEADING.PAYMENT_CONFIRMATION.DESCRIPTION' | translate}}

            </ng-template>
        </step3>

    </wizard>

Thanks in advance

The part of the code you're added to your issue looks good to me.
To better help you I would need a more complete code sample.
For example your sample is missing the beginning <wizard ...> tag.
In addition I don't know how you navigate between the steps.
Do you use the nextStep and previousStep directives, or do you use the goToStep directive, or do you use the navigation bar, or do you use some custom code?

If this question is still relevant/open, feel free to open a new issue.


Let user change triggering character

I'm enjoying your work with Obsidian for emojis.
I tried to add a feature with which user can choose triggering character.
That feature made few hardcoded variables of EmojiSuggester to be initialized in constructor. I think it's acceptable modifications for adding feature that dynamically determines the character to trigger EmojiSuggester.
But the problem is  emojiPostProcessor module.
I thought you separated this module for advantage in testing and debugging because Obsidian API doesn't officially supporting debugging tools. But to add the feature I thought, I think there is no way to avoid importing Obsidian dependent class in emojiPostProcessor module.
So, my question is

Is emojiPostProcessor module separed for debugging?
Is this project open for contribution?
If you are interested in adding feature to letting user to choose a charater to trigger suggestion, should emojiPostProcessor module be integrated?

I don't quite understand the first question, but the emoji post processor module isn't really meant to be configurable - it just turns the shortcode form of emoji, like, :heart: into it's emoji equivalent: ❤. I might reevaluate this later if given convincing arguments but for now, i think we should keep the configuration in the other parts of the plugin - the postprocessor is supposed to be a just works thing.
The project is open to contribution and I welcome any pull requests!
I think letting the user choose a trigger character is a good feature, but it shouldn't affect the emojiPostProcessor as I mentioned previously - there's no standard saying that \heart\ should be considered an emoji. however, it makes much more sense with the immediate replace mode. it's more like a command, and I would make sense for the trigger character to be configurable.
As for EmojiPostProcessors separation, it's just so that different parts of the plugin that do different things are in separate files. I find it makes the code easier to maintain.

Don't mind the first question.
It's good to know that this project is open for contribution!
I thought about this, and there is reasonable motivation to add the feature in expense of compromising EmojiPostProcessors separation. I agree that it should be separated from Obsidian.
Thanks.


sqlite3.IntegrityError: UNIQUE constraint failed: EXPERIMENTS.ID

Hi, I have a new problem in training:

Found that it generated such a folder:

My path setting is like this:

I can't understand what this problem is. Thank you for your help.

Is it a split version problem？

Is it a version issue with the splite package

You have to set a new workname for every experiment. You get this error when you use a previously used workname for another experiment.

I presume your issue is resolved. Closing this.

Is it a version issue with the splite package

How did you solve this problem? Thank you very much


does not generate events

Is the plugin not available yet?

Thank you
I reactivated the plugin and it worked

📅 🛫
Is there a feature that can handle this emoji?
This emoji is used by the tasks plugin
https://github.com/obsidian-tasks-group/obsidian-tasks

Oh, this is very interesting. I had not seen this plugin before. I've had a quick look at the Obsidian Tasks Dates docs and I will be able to support Due Date, Scheduled Date, and Start Date easily.
However, things like this might be complicated to convert into an iCal RRULE:
- [ ] Send Kate a birthday card - with a scheduled date 🔁 every January on the 4th ⏳ 2023-01-04

I'll add this to the todo list.

For the example that you provided:
- [ ] banana 🛫 2023-10-30 📅 2023-10-31

What would you expect to see in your calendar?
I assume two calendar events like this:

Wouldn't it be nice to see it as one event?

Sure thing. That is why I asked. I will implement this when I can.

Hi @pppaapp
Support for Obsidian Tasks in emoji format has been done. Please use the latest release (https://github.com/andrewbrereton/obsidian-to-ical-plugin/releases/tag/1.3.0).
I'm closing this issue it has been resolved. If there are further problems, please create a new issue. Thanks.