[E2E Test 24.08] Service Release Process - Step "Technical Integration" not visible/disappears

Current Behavior

For the "App Release Process", accessing a step "Technical Integration" is possible and successful.
For the "Service Release Process", a step "Technical Integration" also exists in the menu overview (with "Lorem Ipsum" boilerplate text).

When beginning to enter the necessary information for a new Service, this step is also shown initially when on step 1 "Offer Card" in the breadcrumb navigation as a step 4 "technical Integration" (minor typo with capitalization)

But when adding the information in the "Offer Card" and clicking "Save & Submit", this step disappears and then is no longer shown in the breadcrumb navigation menu. After step 3 "Contract & Consent" the next shown step in navigation menu is now "Verify & Submit".

Thus it is not possible to reach a menu to enter details on "Technical Integration" in the Service Release Process.
Expected Behavior
According to E2E test case TEST-1260 / CXTM-29 Service Release: Technical Integration such a step should exist:
Portal Service Release Process: Step Contract Consent is finished and next Step Technical Integration can be selected and filled out.
Thus this may be either a bug in the navigation menu if a step to submit information on "Technical Integration" is required. Or such a "Technical Integration" step is not relevant for a Service offering, and all remnants of this should be removed from the menus + navigation?
Steps To Reproduce
Tested in deployed version for release 24.08 (2.1.0-RC2)
As described above in current behavior.

@evegufy @ds-hzimmer in the code the number of steps is variable. The fourth step is only shown if the service type ID  DATASPACE_SERVICE is present. So I guess it wasn't in the testing scenario.
https://github.com/eclipse-tractusx/portal-frontend/blob/release/v2.1.0-RC3/src/components/pages/ServiceReleaseProcess/components/index.tsx#L96-L100


src/leiningen/new/re_frame/src/cljs/routes.cljs: Should use :import for goog.history.EventType instead of :require

It has been recommended to use :import for goog.history.EventType instead of :require: http://clojurescriptmadeeasy.com/blog/when-do-i-use-require-vs-import.html
See-Also: https://github.com/clj-commons/secretary/issues/98

Fixed by @wegi in #94.


Engine API: define versioning and set initial versions

Defines a versioning mechanism for methods and structures and sets initial versions.

cc @lightclient, @protolambda

there are a number of ways to handle this and they all have different trade-offs...
in the interest of not bike-shedding, i think versioning the JSON-RPC method name itself is fine.
unless someone else can chime in and express i strong preference otherwise, i say we merge this!

Yeah, I guess this is reasonable. It's not beautiful, but I guess it's what we agreed.


Next-day day  notes opened with heatmap for next day not saved when Logseq closed before that day

When I used heatmap to create next day  note and closed Logseq before that day started the data was lost
The note doesn't exist in the git history

@danieltomasz do you have a customized date format for journals?

Yes, I use YYYY-MM-DD for Obsidian consistency,
the Logseq stlil create notes in format YYYY_MM_DD
My settings in edn file
 :page-name-order "file"
:journal/file-name-format "yyyy-MM-dd"
 :journal/page-title-format "yyyy-MM-dd"

When I tried now create note for next day and close Logseq the note was present there
(Although heatmap is telling me that theres no block there)
I suspect that in my case Logseq might just overwrite the notes in the graph when changing date (and there were no files on disk)
(to fully reproduce that the midnight and actually changing date would be needed)

@danieltomasz the new version should now adapt the user-defined :journal/page-title-format. I think whatever the value of :journal/file-name-format should not affect this plugin after all.


Do not register DebugFunction for functions optimized away by inlining.

This fixes https://github.com/KhronosGroup/SPIRV-Tools/issues/3743
The debug info manager was attempting to register functions that had been optimized away by inlining. They all have a function id of DebugInfoNone which caused a problem with the DebugFunction map which keys on the function id, resulting in conflict.
The solution is to not register functions which map to DebugInfoNone.

@jaebaek Please review

Thank you for working on it!


[Snyk] Upgrade: @angular/cdk, @angular/material

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade multiple dependencies.
👯 The following dependencies are linked and will therefore be updated together.

:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

Name
Versions
Released on

@angular/cdkfrom 13.3.3 to 13.3.4
1 version ahead of your current version
21 days agoon 2022-04-20

@angular/materialfrom 13.3.3 to 13.3.4
1 version ahead of your current version
21 days agoon 2022-04-20

Release notes

Package name: @angular/cdk

13.3.4 - 2022-04-20
13.3.4 "adamantium-albatross" (2022-04-20)
cdk

Commit
Description

portal: prevent calling ApplicationRef.detachView on destroyed instance (#24775)

Special Thanks
Andrew Kushnir, Kristiyan Kostadinov, Paul Gschwendtner and Zach Arend

13.3.3 - 2022-04-13
13.3.3 "tweed-table" (2022-04-13)
material

Commit
Description

checkbox: add the boolean property coercion for checked input (#20645)

chips: prevent default behavior on remove button (#24722)

datepicker: avoid rerender when min/maxDate changes to different time on the same day (#24434)

list: wrong order of arguments when calling custom compareWith function (#24743)

select: empty space read out by VoiceOver on Chrome (#24741)

snack-bar: update generic types for openFromComponent (#24634)

tabs: wrong scroll distance if selected tab is removed (#24118)

progress-bar: do not run change detection if there are no animationEnd listeners (#24673)

cdk

Commit
Description

a11y: visually hidden element affecting scrolling (#24620)

Special Thanks
Artur Androsovych, Georgian Stan, Klemen Oslaj, Kristiyan Kostadinov, Michael Doner, Pascal Weyrich, Paul Gschwendtner, RobStrader and Zach Arend

    from @angular/cdk GitHub release notes

Package name: @angular/material

13.3.4 - 2022-04-20
13.3.4 "adamantium-albatross" (2022-04-20)
cdk

Commit
Description

portal: prevent calling ApplicationRef.detachView on destroyed instance (#24775)

Special Thanks
Andrew Kushnir, Kristiyan Kostadinov, Paul Gschwendtner and Zach Arend

13.3.3 - 2022-04-13
13.3.3 "tweed-table" (2022-04-13)
material

Commit
Description

checkbox: add the boolean property coercion for checked input (#20645)

chips: prevent default behavior on remove button (#24722)

datepicker: avoid rerender when min/maxDate changes to different time on the same day (#24434)

list: wrong order of arguments when calling custom compareWith function (#24743)

select: empty space read out by VoiceOver on Chrome (#24741)

snack-bar: update generic types for openFromComponent (#24634)

tabs: wrong scroll distance if selected tab is removed (#24118)

progress-bar: do not run change detection if there are no animationEnd listeners (#24673)

cdk

Commit
Description

a11y: visually hidden element affecting scrolling (#24620)

Special Thanks
Artur Androsovych, Georgian Stan, Klemen Oslaj, Kristiyan Kostadinov, Michael Doner, Pascal Weyrich, Paul Gschwendtner, RobStrader and Zach Arend

    from @angular/material GitHub release notes

Commit messages

Package name: @angular/cdk

2b90924 release: cut the v13.3.4 release (#24807)
0a06523 refactor(cdk/table): expose data source type (#24782)
ab04f21 docs(material/chips): use correct terminology (#24798)
a98f5ad build: disable renovate lock file maintenance (#24794)
eef4757 fix(cdk/portal): prevent calling `ApplicationRef.detachView` on destroyed instance (#24775)
dd4dbd6 build: add Zach (zarend) to datepicker code owners (#24787)
46657d0 build: preparation for primary branch rename in the Angular repos (#24783)

Compare

Package name: @angular/material

2b90924 release: cut the v13.3.4 release (#24807)
0a06523 refactor(cdk/table): expose data source type (#24782)
ab04f21 docs(material/chips): use correct terminology (#24798)
a98f5ad build: disable renovate lock file maintenance (#24794)
eef4757 fix(cdk/portal): prevent calling `ApplicationRef.detachView` on destroyed instance (#24775)
dd4dbd6 build: add Zach (zarend) to datepicker code owners (#24787)
46657d0 build: preparation for primary branch rename in the Angular repos (#24783)

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
59 unchanged lines in 7 files lost coverage.
Overall coverage decreased (-0.5%) to 73.208%

Files with Coverage Reduction
New Missed Lines
%

src/app/datasets/dataset-file-uploader/dataset-file-uploader.component.ts
3
67.44%

src/app/datasets/dataset-lifecycle/dataset-lifecycle.component.ts
4
78.79%

src/app/logbooks/logbooks-dashboard/logbooks-dashboard.component.ts
6
73.47%

src/app/datasets/admin-tab/admin-tab.component.ts
9
30.95%

src/app/datasets/dataset-details-dashboard/dataset-details-dashboard.component.ts
10
33.87%

src/app/datasets/dataset-detail/dataset-detail.component.ts
12
78.53%

src/app/datasets/reduce/reduce.component.ts
15
38.36%

Totals

Change from base Build<PHONE_NUMBER>:
-0.5%

Covered Lines:
3571

Relevant Lines:
4473

💛  - Coveralls


Problems with various string attributes (CON-969)

In general, the declaration of the char[] buffers should use the constant and not a numeric size.

This issue occurs multiple times.

I changed this from 61 to 60 and now esp-matter doesn't abort, but there is still some problem.

I can't get this work when using sizeof(config->node_label) which matches the field limit. I get an "Error 1" from the Android decode library, but the buffer does look ok on the wire. (Maybe Android lib has a bug when the string is not zero terminated and max length?).
If I change this to strlen(config->node_label) I can get it to work.

When you retrieve a max length string saved to flash, do you trim off the zero terminator before giving it to CHIP? If it is not max length you want to keep the zero terminator.

in esp_matter_nvs.cpp

What happens if I am setting the string to NULL to remove it? Does look like it gets removed. Then later in the code I fault trying to print the attribute, but I think that is because the state is messed up.
So instead of 'err = ESP-OK' if needs to check to see if the key is in storage and remove it.

As a test I added this in my S3 code:

I verified that this does not remove the attribute from NVS. Still there, should be gone or have zero length

And internal RAM state is messed, a report now on this attribute will cause GFP in ROM code.
Reboot and attribute is back to what is in the NVS.

hi @jonsmirl ,
We are seeing exactly the same problem in node_label (basic information), where the default string is a null 32 bytes char array

We have create an issue in connectedhomeip (https://github.com/project-chip/connectedhomeip/issues/31793)
They suggest we should pass-in strlen instead of sizeof when creating the attribute.
However in our previous trials, the strlen method will be ok in first read, but we can't write it by chip-tool
We do have a quick workaround (call an esp matter update for that attribute after power up, and everything works fine afterwards)
But this seems a patching fix and just want to know the issue better.
Have you got any further progress on this?

@jonsmirl
Just letting you know, found this fix in esp-matter today  :)
https://github.com/espressif/esp-matter/commit/3c6600df3fd281f3228bd8f677db60f60a7b223c
I have tested a bit and it has no problem reading and writing

@jonsmirl Bounds related problem is fixed in https://github.com/espressif/esp-matter/commit/d481fa9cf1d09001b1bbd9cd59dbb7836bbb13fe

@jonsmirl  nvs_set_blob() won't let us store the nullptr, so if thats the case, erasing that key, please check https://github.com/espressif/esp-matter/commit/8943861e30b6f96c17dfc3a8986f517977dc0451. And with this I think, string related problems are resolved.


Missed semicolon in vagrant.mk

Signed-off-by: Ilya Lobkov<EMAIL_ADDRESS>
Description

Motivation and Context

How Has This Been Tested?

[ ] Covered by existing integration testing
[ ] Added integration testing to cover
[ ] Tested locally
[ ] Have not tested

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

@lobkovilya I've improved this in https://github.com/networkservicemesh/networkservicemesh/pull/1907 so closing your pr.


Introduce PureModelContextCollection type and use it in WorkspaceSDLCLoader

What type of PR is this?
Improvement

What does this PR do / why is it needed ?
As of today, WorkspaceSDLCLoader calls ModelLoader for each dependency. This introduces as many numbers of HTTP requests and doesn't account for dependency override.
Solution

Introduce PureModelContextCollection type
Modify WorkspaceSDLCLoader to call ModelLoader with a PureModelContextCollection
Support fetching PureModelContextCollection using AlloySDLCLoader

Additionally, PureModelContextCollection can be used further to send smaller payloads from clients like Studio.

Which issue(s) this PR fixes:

Fixes #
Other notes for reviewers:
For testing, I rely on change in test setup and manual testing.
Does this PR introduce a user-facing change?
No

@abhishoya-gs
can we not change the model loader and add another type of PureModelContext.
I think @rafaelbey  started something. see : https://github.com/finos/legend-engine/pull/2446

@abhishoya-gs can we not change the model loader and add another type of PureModelContext. I think @rafaelbey started something. see : #2446

Okay sure, seems like a better approach

This PR is stale because it has been open for 30 days with no activity. Please remove stale label or add any comment to keep this open. Otherwise this will be closed in 5 days.

This PR was closed because it has been inactive for 35 days. Please re-open if this PR is still relevant.


refactor(codegen): rename CodeBuffer::print_byte_unchecked method

Pure refactor. *_unchecked is the conventional name for unsafe methods, not *_unsafe.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#6496  👈
#6495
#6494
#6493
#6492
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @overlookmotel and the rest of your teammates on  Graphite

Only a naming change, so merging without review.

Merge activity

Oct 13, 5:38 AM EDT: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.


chore: 🤖 更新 changelog 文件

  🤖 由 [🚀 After the release is successful] action 自动创建.

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

Project coverage is 79.26%. Comparing base (1adc25d) to head (e1e05a0).
Report is 162 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2713      +/-   ##
==========================================
+ Coverage   76.48%   79.26%   +2.77%
==========================================
  Files         240      160      -80
  Lines       12312     9524    -2788
  Branches     2630     2057     -573
==========================================
- Hits         9417     7549    -1868
+ Misses       1377      662     -715
+ Partials     1518     1313     -205

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

:tada: This PR is included in version @antv/s2-v1.55.5 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Starlette integration does not allow to configure inner client options in straightforward way

A double "client" layers when using Starlette integration is confusing and makes tedious to configure underlying 'real' httpx async client.
A sample code below displays a few problems:
oauth = authlib.integrations.starlette_client.OAuth(config=Config(environ=CONFIG))
client: StarletteRemoteApp = oauth.create_client(CLIENT_ID)  # Problem 1

async def httpx_event_hook(request):
   logging.info(f"Request event hook: {request.headers}")

client.client_kwargs['force_include_body'] = True  # Problem 2 and 5

authlib.integrations.httpx_client.utils.HTTPX_CLIENT_KWARGS.append('event_hooks')  # Problem 3
client.client_kwargs['event_hooks'] = {'request': [httpx_event_hook]}

await client.post('https://address', token=access_token, json=json) # Problem 4

Problems:

OAuth.create_client confusingly creates StarletteRemoteApp, not the actual client
There is no explicit way to pass-through any options to the actual client. The actual client is created in the middle layer by protected function _get_oauth_client. The sample code above is the only way I found by analysing the code flow.
Arbitrary client params are passed to httpx only. There is no easy way to hook for httpx events. Moreover, client_kwargs holds mixed args for AsyncClient and _OAuth1Client
The token must be passed in every call. Again, this could have been managed on the client initialization level
The most nasty one: default value for force_include_body (False) causes to silently drop body and send content-length 0 when communicating via json, not form (see rfc5849.client_auth.ClientAuth.prepare, line 166). I could not find any documentation for this parameter.

Describe the solution you'd like
Ad 1 - Should this be renamed to create_client_app()  (older name could be alias with deprecation)
Ad 2, 3 - This should be at least documented. Even better, create_client/create_client_app could accept kwargs which are passed to the inner client. Also, maybe some wrapper around _get_oauth_client could be exposed to the user and documented.
Ad 4 - The tokens could have been optionally provided to create_client/create_client_app to be maintained through the session.
Ad 5 - Should be at least documented and emphasized that default value might result in surprising behavior.

This also seems to apply to the Django client, and I found this bit particularly cumbersome:

The token must be passed in every call. Again, this could have been managed on the client initialization level

Per the docs, I could use OAuth2Session directly instead, but in doing so, I lose the auto-refresh.


Flash error

I get this error when I try to run an empty flash target build.
TypeError: Error #1007: Instantiation attempted on a non-constructor.
at Empty/init()
at kha::Game/loadFinished()
at kha::Starter/onReady()

Can not reproduce. How exactly do you do that? haxelib version or git version? All submodules up to date? Flash player up to date?

Sorry I should have been more descriptive. I posted the error in a hurry.
It seems the problem is as diagnosed similarly from this page: http://stackoverflow.com/questions/11010041/as3-typeerror-error-1007-instantiation-attempted-on-a-non-constructor
I was using the git version of Kha.
And sorry, it was not an empty project. It was the default bunnymark used for benchmarking Kha (by Dmitry Hrppa), from here: https://github.com/dmitryhryppa/Frameworks_test/tree/master/bunnymark/kha
Previously it worked, but somehow now it doesn't. The offending code here is the:
fps = new FPS();

It seems if I rename the class to something other than FPS, then it will work fine. Has the FPS class been reserved in Kha?

There are no reserved words or anything like that in Kha (everything is inside the kha package). It compiles for me as is with the haxelib version and the git version. What's your exact khamake call?

But just realized you're not talking about a compile time error and running Stage3D code with my old laptop graphics drivers isn't working - will investigate further when I'm back at home.

Funny thing, FPS seems to be reserved by Flash itself. According to the debugger FPS is 60.

Oh! Thanks for looking into it, and sorry, I should have noticed. :(


Update DG with add and adddebt use case

Part of #13

Codecov Report
Base: 72.15% // Head: 72.15% // No change to project coverage :thumbsup:

Coverage data is based on head (f91ffe3) compared to base (c655915).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master      #46   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Memory leak

My project make with this library has memory leak
and i fix that below some code
add method at JazzyViewPager.java
public void delObjectForPosition(int position)
{
mObjs.remove(Integer.vauleOf(position));
}
and modify my pageradapter
@Overrid
public void destoryItem(View container, int position, Object obj)
{
jazzPager.removeView(View)obj);
jazzPAger.delObjectForPosition(position);
}

I think its worth noting that this memory issue depends on what your pager adapter is. FragmentStatePagerAdapter only keeps one view at a time, so if I'm not mistaken, you wouldn't run into a memory issue then, as opposed to FragmentPagerAdapters.


Ensure valid polygons and lines

[x] Test winding order for polygons ("exterior ring is orientated in a clockwise direction and the interior rings in a counter-clockwise direction.")
[ ]  Zero sized interior rings ("an area less than 0.5")
[x] Clipped polygons - if clipped still return and valid
[ ] Lines must be at least two points.
[ ] Empty geometry tests.
[ ] Ids are maintained throughout.

Using:
http://www.spatial-data.brisbane.qld.gov.au/datasets/e01461124ae146eaa5e02d75bd823c4c_0?geometry=152.986%2C-27.475%2C153.048%2C-27.461
Bounds:
"{"_sw":{"lng":152.75666065986928,"lat":-27.481156891709432},"_ne":{"lng":153.35822970590516,"lat":-27.35099332289336}}"
Layer configuration:
"qpp_transport_noise_corridors": {
"security": "public",
"table-name": "qpp_transport_noise_corridors",
"format": "source",
"simplify": {
"10": "douglas-peucker"
},
"bounds": [
152.747009018039,
-27.6612689983585,
153.191780420144,
-27.2809313072319
]
},
Before:

After:


Added port-argument to connect

This field is mandatory when 'host' is used.

Thank you very much for catching this!
We actually intended for the port field to be optional by applying a default port of 6397. However, this issue helped uncover a typo in our code that caused the default port to never be applied. I fixed the underlying issue in https://github.com/GUI/lua-resty-auto-ssl/commit/7fcb04aa80dbbbf39f9322cffcd849942e706520.
So I'll go ahead and merge this, since being explicit about the port does workaround the issue when using the redis adapter with the current version (v0.8.2). But once we release v0.8.3 with the default port fix, I'll probably then rollback this README change, since setting the port should no longer be necessary.
Sorry for the trouble, but thanks again for helping uncover the issue!


html: change title of signin page

we havent set a title for the signin page. a block for title is defined in the base template main.html

edit signin.html and set appropriate title

Example

title: Sign In — Cinlist
✨ Feel free to add your own changes !
💬 drop a comment if you like to take up this issue, so i can assign it to you.

Hi, I'm Interested to take up this issue!

Hey, its Updated check pr #43


Auto-Populate owner field if user is authorized via static auth checks

Description of changes
The Auth checks in a create mutation currently do not set the ownerField if the @model type has both static group auth and owner auth rules specified on it and the user is successfully authorized via static auth checks. This is reported by customers as a regression from V1 behavior.
This PR fixes this issue behind a FF populateOwnerFieldForStaticGroupAuth which is set to false for existing projects, true otherwise. FF PR in CLI repo: https://github.com/aws-amplify/amplify-cli/pull/10630
Behavioral changes
When the above FF is true, we automatically set the value of ownerField if the user is authorized via static group checks in a create mutation. This means that if the user is removed from the group, he will still continue to have access to the records he created since he is the owner.

Issue #, if available

https://github.com/aws-amplify/amplify-category-api/issues/37
Description of how you validated changes
Unit tests: Updated screenshots indicate the change
Checklist

[x] PR description included
[x] yarn test passes
[ ] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)
[ ] New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #580 (5bbf8f4) into main (d52e7d9) will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #580      +/-   ##
==========================================
+ Coverage   62.94%   62.98%   +0.03%
==========================================
  Files         278      278
  Lines       18016    18035      +19
  Branches     4330     4334       +4
==========================================
+ Hits        11341    11360      +19
  Misses       6092     6092
  Partials      583      583

Impacted Files
Coverage Δ

...hql-auth-transformer/src/__tests__/test-helpers.ts
100.00% <100.00%> (ø)

...l-auth-transformer/src/graphql-auth-transformer.ts
94.70% <100.00%> (ø)

...-graphql-auth-transformer/src/resolvers/helpers.ts
100.00% <100.00%> (ø)

...-auth-transformer/src/resolvers/mutation.create.ts
95.55% <100.00%> (+0.10%)
:arrow_up:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@AaronZyLee added a check in an e2e test

@phani-srikar I think this needs a rebase. There are commits in this PR that I think are from a previous version of main


frontend: Regarding the color of loading

When I'm using Casdoor, for example, when the login page redirects to the registration page, there is a loading effect in between. Why isn't the theme color I set taking effect? The color of this loading effect is purple, not the theme color I set. Can any handsome guy tell me how to fix this? I've even overridden the style of the spin, but it still doesn't work.

@hsluoyz ok wait a moment

Uploading 录屏2024-01-11 上午9.32.18.mov…
@hsluoyz

https://github.com/casbin/casdoor/assets/23206991/ecd68f52-bb22-4346-b8d8-5aa15af4655c
@hsluoyz

My theme color is green, but the loading icon and spin are purple.


it supports the netcore?

Hi it supports the netcore?

I haven't tried it with .net core yet, if you have a chance to try it out it would be great to get your feedback on it.

I tested and it works with asp.net core v1.0 using .NetFramework not sure about Core one

I tried it on core and it didn't work, I think I have found one line of code that is making it incompatible, on a feature that is not documented

it now supports netcore! and many other .net flavors using the .NetStandard library

Currently in pre-release on nuget


Config.json not saved changes if use very big oneOf function.

Describe The Bug
The config not save its changes if use very big oneOf functions. In my case is 600 objects.
The plugin to test:
https://github.com/grzegorz914/homebridge-denon-tv
If reduce objects then saved correctly.
Thanks.
Logs
No response
Config
No response
Homebridge UI Version
All versions
Homebridge Version
All versions
Node.js Version
20.10
Operating System
macOS
Environment Info
{"label"=>"Using hb-service?"}
Raspberry Pi Model
Other (specify in description)

I fixed this in my plugin config.schema by reduce objects count  in oneOf but the problem still exist in ConfigUi X.

Hi @grzegorz914 is this something that I could reproduce myself, if I could install your plugin could you attach to this ticket an example config that I could use?

Please instal version 4.7.0  because in 4.8.0 contain temporary workaround for this issue. After install it try to create / change some buttons using UI. The UI will not save the changes.

Is this related to the "Accessories" tab and not saving the layout of rooms and accessories?


Question about reliability of accessing parent property inside $parseJson when using insertGraph

I have two models Orders and Line items. Each order can have many line items. I've setup the relation mappings and calling insertGraph on the Order model is working perfectly. This data is coming from an external service, so in the case of Line Items, I'd like to have the table use both our own internal id's and the external service's id's when referencing the parent Order. I noticed that when doing an insertGraph on Order, in the Line Item model's $parseJson, the opt param contains the parent order's external id that I'd like to store on the line item.
It's available inside the opt param's cache key. I'm assuming there isn't a better way to do this at the model's relationMappings level? If not, I just want to make sure I can rely on that data to always be in the cache key of the opt param inside line item's $parseJson?
As always thanks for the help.

The cache is a private implementation detail an can be renamed or removed at any given time. You shouldn't use it. Only documented things should be used.
Unfortunately there is no other way to do this out of the box either.

As always thanks for the fast reply and info @koskimas cheers!


Localization of static resource files such as js css fonts; add produ…

…ction-ready docker-compose.yml
Description
Localization of static resource files such as js css fonts; add production-ready docker-compose.yml.
Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I wrote unit tests for my code (if tests is required for my changes)
[ ] I have made changes in CHANGELOG.md file

Thanks for your pr! But I think that moving assets under the git is not a good solution. What's the reason?

I'm very sorry, but I cannot accept this PR. The reasons are:

Web assets should be loaded directly from the CDN (as of now) or bundled into a single js file using WebPack, for example
Existing docker-compose file is for development already, and I suppose a separate file for this is not needed

But anyway, you can develop your fork! And thanks for your time!

I'm very sorry, but I cannot accept this PR. The reasons are:

Web assets should be loaded directly from the CDN (as of now) or bundled into a single js file using WebPack, for example
Existing docker-compose file is for development already, and I suppose a separate file for this is not needed

But anyway, you can develop your fork! And thanks for your time!

But some users who cannot connect to the Internet may not be able to use this program because they cannot get the CDN resources. Committing CDN resources to a git repository may not be an elegant way to handle this. A model that does not rely on CDNs is obviously more widely used.
I will continue to develop the fork repository for a more suitable solution, such as npm, and I will submit the most suitable version to you at that time.

Could you please make an issue in this repo with your assets building suggestion? I will try to make all required changes shortly

Could you please make an issue in this repo with your assets building suggestion? I will try to make all required changes shortly

The issue is created here


I expect the first Y axis has the correct scale.
What is happening instead?
The first Y axis has the same scale as the second Y axis.
What error message are you getting?
None.

The option chart.brush.autoScaleYaxis doesn't work well with multiple y-axis chart.
Hence, I suggest turning it off.
chart: {
    id: "brush",
    brush: {
      target: "chart",
      enabled: true,
      autoScaleYaxis: false
   }
}


error in step 4

Hi uglyog,
I follow the workshop smoothly until step 4:
it seems that pactVerify task isn't run to check pact here.
Thanks a lot!

gradlew :providers:springboot-provider:pactVerify
...
2017-07-18 18:28:11.699  INFO 26756 --- [           main] .s.b.c.e.j.JettyEmbeddedServletContainer : Jetty started on port(s) 8080 (http/1.1)
2017-07-18 18:28:11.707  INFO 26756 --- [           main] a.c.d.p.s.MainApplication                : Started MainApplication in 9.825 seconds (JVM running for 11.318)
java -jar C:\github\pact-workshop-jvm\providers\springboot-provider\build\libs\springboot-provider.jar is ready.
:providers:springboot-provider:startProvider FAILED
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':providers:springboot-provider:startProvider'.

java.lang.NoSuchFieldException: pid

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED
Total time: 24.886 secs

Haven't tested it on Windows. Looks like the spawn plugin does not work on Windows.
Could you run the workshop in a Linux VM?

thanks!
I'll have a try.

发件人: Ronald Holshausen<EMAIL_ADDRESS>发送时间: 2017年8月3日 14:37
收件人: DiUS/pact-workshop-jvm
抄送: 虞科敏; Author
主题: Re: [DiUS/pact-workshop-jvm] error in step 4 (#3)
Haven't tested it on Windows. Looks like the spawn plugin does not work on Windows.
Could you run the workshop in a Linux VM?
―
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/DiUS/pact-workshop-jvm/issues/3#issuecomment-319881698, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AH4W8EP38nBl3tgV-MIjFqQt2Ofp751Kks5sUWq8gaJpZM4ObJ04.

I'm running OSX (this might be a different problem than described above) and I'm a total gradle & springbook virgin. My difficulty at Step 4 was a fight between gradle and springboot. Fix was to change providers/springboot-provider/build.gradle to say id "au.com.dius.pact" version "3.6.6."
Apparently earlier versions of the plug-in failed to cast a GStringImpl variable (arising from 'Our_Provider') to a righteous Java string that Gradle's NameValidator.asValidName() could digest.
I think the righteous fix to this issue is to carefully track proven-good versions of the tools and/or update to the latest-greatest as soon as they can be verified.

I've updated to the latest Pact-JVM version (4.1.2)


Wallet not syncing. No block source available...

Actual behaviour
No block source available... 0 active connection(s) to Zcoin network
Desired behaviour
Synchronizing with network
Steps to reproduce

backup wallet.dat
Reinstall wallet from scratch

What wallet version please? Are you behind any sort of firewall and how long did you wait?
Could you attach your debug.log?

This happened on v<IP_ADDRESS>. Reindexing didn't work it would stop at 821 866 or 1313 blocks. So I did fresh install of v<IP_ADDRESS> with wallet.dat backup, but now it won't connect to Zcoin network to download blockchain and sync wallet. I'm not behind firewall since I've been using wallet for quite some time now.

If you only tried reindex on <IP_ADDRESS>, please reindex again on <IP_ADDRESS>

@catchingknives I reinstalled wallet from scratch, which means I deleted blockchain from disk

I clicked on "anonymous communication with Tor" and it triggered syncing process. Before installing fresh copy I also did zcoin related registry cleanup if it made any difference at all. When it's done I'll close this, if I get stuck again I'll let you know.


Testing openning issue with script 2024-04-18

Testing scheduled capacity to open issues when tests fails.

It works!


Allow for better customization of Test output

Right now there's 2 flaws with our existing Test result printer:

It relies too heavily on the Symfony Console OutputInterface
It is effectively hardcoded into the CLI application and can't be easily replaced

A new solution should be designed that addresses both of these concerns.

https://github.com/labrador-kennel/async-unit/discussions/49 proposal to resolve this issue


Reliable method to look up icon by name

Since the method to look up an icon by name as discussed in #37 is buggy, it would be nice to have a reliable method for said lookup.

Before adding this feature, I'm wondering why you need it. I've never actually need this feature in my project. The reason I added this method is I want to add some debugging support. Do you want to save the icon's name as some kind of configuration and recreate that icon later? In that case I think I should implement NSCoding on FAKIcon so an icon can be archived.

It's exactly that - I am configuring a list of labels by providing them with a serialized json feed from a server. Since I want the user to be able to pick an icon, I have to send the name of the icon over the wire somehow. It would be awesome having this functionality.

@keyboardsamurai second this, I also encountered the duplicate issue. Enumerating through [FAKFontAwesome allIcons] also do not work.  I have a configuration which holds which icon to user by name.

I also encountered this issue but in my case, I received the font awesome string from the server and had to do the translation to properly display the icon.

+1, I have dozens of items, each has different icon. I put them in a JSON file (and updatable from server). name lookup will help greatly for this scenario.
Thanks!

Another library has name look up: https://github.com/alexdrone/ios-fontawesome
But I want to stick to this library.

@top2tech Do you need to save any other property into the JSON file, like icon color and the size?

I guess the other properties can be used in other ways. If they have size and color, it's easy to actually create the icon properly knowing those. The main issue is the name lookup.
I don't think adding the other properties would be useful with the existing methods.

Yes, the issue is name lookup. I'm currently using the output as an attributed string and configure it separately, so no urgent need for icon color/size.  Not being able to lookup some of these icons is actually creeping up on me since our designer heavily relies on icons from FontAwesome.  I'm currently hardcoding these until it's fixed.

Same here, I have implemented a few of them, is there anything preventing a simple lookupMethod?
I can do it and send a PR if @PrideChung doesn't have the time.

@GabrielCartier I'm busy for baby sitting lately, you would do us a big favor if you can implement this.

@GabrielCartier Looking at FAKFontAwesome.m, I think the simplest way would be to ensure [FAKFontAwesome allIcons] has no duplicates and that it has correct mapping to its matching class method.
The long way would be to change the code scheme to match the Font Awesome web scheme.
i.e.
So instead of @"\uf006", I'd propose @"fa-star-o".

Perhaps we can create a separate lookup method to fetch the appropriate class method to perform selector.
ex:
FAKFontAwesome *icon = [FAKFontAwesome iconWithLookup:@"fa-star-o" size:18.0f];

Yea, that's what I was thinking. I'll write it today if I got time and do a PR


Temporary workaround for Crash in Date.description[SR-2462]

@parkera  : Please find the answers inline
Does it introduce a new warning though? Yes it introduces new  warning .
And where else would we have to do this? I observed the crash in description of NSDate only.Other places am not sure of it .
Do you know if this problem is a regression in Swift 4? No.I have see my test case crashing with Swift 3.0.2 also

@parkera  modified workaround to avoid compiler warning.Please review


add Volatility request

Hey Brian,
I wanted to ask you if it is possible to add the volatility indicator (for crypto screener).
Thanks a lot!

@brian-the-dev
Hello Brian, I have another question.
I managed to add new indicators to the single analysis, like this:
`handler = TA_Handler(
symbol="MANAUSDT",
exchange="binance",
screener="crypto",
interval=Interval.INTERVAL_1_DAY
)
handler.add_indicators(["Volatility.D"])
analysis = handler.get_analysis()
print("MANAUSDT", analysis.indicators["Volatility.D"])`
But I want to add new indicators to the get_multiple_analysis and I don´t know how to do that. Could you help me out?

This is currently not implemented, but there is a workaround:
Just append "Volatility.D" to TradingView.indicators:
from tradingview_ta import *

# Add this line below
TradingView.indicators.append("Volatility.D")

analysis = get_multiple_analysis(screener="crypto", interval=Interval.INTERVAL_1_DAY, symbols=["binance:ethusdt", "binance:manausdt", "binance:btcusdt"])

# print(analysis["BINANCE:BTCUSDT"].indicators["Volatility.D"])
# Output: 6.32796553

Make sure to append only once, at the beginning of the code (preferably below import).

@brian-the-dev
Thanks a lot, it works but unfortunately only on the 1_DAY Interval, even if the volatility value is exactly the same on the 5 mins - 4h Intervals. For the single analysis it works also on the lower Intervals. Strange...

sorry I made a mistake, it doesn´t work on the lower Intervals at all. Thats unfortunate because as I said, the value for the 1 min- 1 Day Interval, no matter what Interval, is the same. And i really like the set the Interval to 5 min and get the Volatility value....

Normally, you don't need to add "|1", "|5", etc as tradingview-ta will automatically append them based on the interval. "change" indicator should work for all intervals. By default, "change" indicator is already included (in percentage) so you don't need to add it. If you want to get the absolute change instead of percentage, add the "change_abs" indicator.
If you're unable to get the change, a workaround is to calculate it manually:

Change (absolute) = close price - open price
Change (percentage) = absolute change / open price * 100


ScalaTest: move pact writing to afterAll

Instead of writing out pacts at the end of run(testName: Option[String], args: Args), it would be more flexible to do it in afterAll(): Unit of the BeforeAndAfterAll trait -- that way users can provide custom logic to fail the tests prior to the pact files being written.
In our case, I want to ensure that a test exists for every message defined. If not, I want the suite to fail and pacts to not be written.

I'm in two minds about this, as it does make the PactForger mixin heavier, and mean that using BeforeAndAfterAll for anything else on top of this becomes a bit clunkier. Maybe the PactForger trait itself could provide a beforeWritePacts method or something like that?

I think it's pretty common in Scalatest to do:
override def afterAll(): Unit = {
  // .. do whatever
  super.afterAll()
}

But yes, having a method like beforeWritePacts would be more explicit. I guess I should have made this a problem statement instead of a solution -- I am really open to any solution for this.

True about it being common in scalatest, I just don't like it 😛
Actually it would be good to have it as a pact4s native method, because them we can include it for the other testing Frameworks. I would like to keep feature parity between them all if possible

@solarmosaic-kflorence I added a beforeWritePacts method to the pact forger traits. For scalatest its got signature () => Either[Throwable, Unit].  If it's a Left then the error gets thrown and the pacts don't get written. Available in v0.0.7.

Looks good, thanks!


Refractor to serve JS locally

Fixes: None
Content served CDN is an area of concern for privacy & performance.
Ensure all the JS code is served locally.

Serve ‘url-search-params.js’ locally.
Serve ‘symbol-es6.min.js’ locally.
Serve “jquery.fancybox.min.js" locally.

I need to look at the lint issues. Do not merge.

Easy way to get the select2 js and css to also be local?

Easy way to get the select2 js and css to also be local?

Are you talking about the django-select2 package? They do have css/js in their static local folder. Would you be able to point me where it's you see them remotely served?

Other than select2 issue I believe this is good to merge in, I can take a stab at select2 js/css in another PR.


Download a new 3d images site

Hello,
you are create a greate program .
Can you upgrade your program, added this site support
http://www.cgbytes.com/gallery.aspx ???? To hard copy images one by
one into a personal collection. Please help.
Maybe i need pay for you for that update?

mirror url https://www.renderotica.com/gallery.aspx

Not gonna happen. Premium paid access for most images.  That said someone likes to post some of the more popular crap from this site on other sites like [this one] (g.e-hentai.org)

i can buy premium there and give you access, and you can create for me, please.

@MaximIvano As cool of you as that would be I have to decline. What little I know about this program is all self taught from existing templates - so a new site would be like me trying to write/read in Chinese. For meow I would suggest something with a more global scope like FoxySpider (addon for Firefox) or messing around with the "DownThemAll" addon. I've had good luck with FoxySpider in the past; unfortunately so did many other people, as a result a lot of the Boorus/imageboards can pretty easily detect and punish you (like BAN your IP and/or login!) so be careful.
Before you try anything extreme or get desperate I suggest you take a gander at 4chans [Collection Management, Hentai Resources, & General Assistance] (http://boards.4chan.org/h/thread/4405833) and start researching there. I've been there before and IMHO it led me here many moons ago :) [Good luck. We're all counting on you.] (https://www.youtube.com/watch?v=SmHeP9Sve48)


KAFKA-3454: add Kafka Streams web docs

Also fixes the indents for tools and streams in build.gradle.

@ymatsuda @gwenshap @miguno for reviews.
Some related changes:

fix indents in the build.gradle file.
remove Consumer configs "offset.auto.reset" from StreamsConfig.

@gwenshap Unit tests passed locally.

LGTM. We can iterate on the contents / formatting later.


Fix reading structs of primitives

The length argument for DirectCopy() is in bytes, not in number of elements,
therefore DirectCopy() must not multiply length with the field size.

Fixed in master. Thanks!


Fix license attributes to SPDX standard

Sorry ‘bout the errant commits. You want me to redo?

Gonna redo this, sorry.

What was wrong with this one? You can push updates to the same pull-request 😕


[WPF] Updated text content rendering to use Text.CurrentValue

On WPF the text content on controls wasn't rendered correctly.
Turned out the Content/Text properties of the native controls were set to a property of type Binding.
Updated it to use the CurrentValue property of the Binding class.
Before:

After:

Awesome thanks!


Accordiong coloring not working as expected

Bug description:
Accordion is not working as expected. I read the known issues but the accordion is not working. Before I was able to set accordions with different colors one after the other one.
Link to minimally-working plunker that reproduces the issue:
https://plnkr.co/edit/sfkTApfUbSi5QoxNOdHP?p=info
Steps to reproduce the issue:
Just run the plunker and you see that the accordions are not build correctly (doesn't have round corners and spacing between them)
Version of Angular, UIBS, and Bootstrap
Angular: 1.6.2
UIBS: 2.5.0
Bootstrap: 3.3.6

ng-class="task[$index].panelColor"


[REQUEST] Support for Proxy Protocol and PoW Defenses in tor-controller

Problem Description:
The inability to track circuit IDs and activate Proof of Work (PoW) defenses in Tor services limits traffic analysis, optimization, and protection against abuse.
Desired Solution:
Implement two settings in the onion/onionha manifest:

"HiddenServiceExportCircuitID haproxy" for enabling circuit ID transmission via HAProxy's Proxy Protocol.
"HiddenServicePoWDefensesEnabled 1" to allow activation of PoW defenses for additional security.

Additional Context:
These features will enhance both the analytical capabilities and security of high-traffic Tor services, contributing to the overall effectiveness and safety of the network.
Entry of news on the Tor blog:
https://blog.torproject.org/introducing-proof-of-work-defense-for-onion-services/
Support of the new PoW Tor feature:
https://gitlab.torproject.org/tpo/onion-services/onion-support/-/wikis/Documentation/PoW-FAQ
PoW parameter on the torrc file:
https://gitlab.torproject.org/tpo/core/tor/-/blob/main/doc/man/tor.1.txt#L3108
Export Circuit parameter on the torrc file:
https://gitlab.torproject.org/tpo/core/tor/-/blob/main/doc/man/tor.1.txt#L3509
Example of hidden service configuration:
https://gitlab.torproject.org/tpo/onion-services/onion-support/-/wikis/Documentation/PoW-FAQ#example-configuration

https://github.com/bugfest/tor-controller/issues/56


[apostrophe-alerts] Implement custom mechanism for displaying alert and confirm messages.

duplicate of #916


Deadlock when ADB got killed

Original report by me.

I started logview with adb configured to binary v39. It started the adb server process apparently.The app has connected to device and started to retrieve logs from it. Then I started command-line adb binary v36 that caused server to restart. The logview app hanged.
jstack output is attached. Log output is quite short though:
2017-09-06 22:36:29,358 [main] INFO  org.bitbucket.mlopatkin.android.logviewer.Main - Android Log Viewer 0.19-SNAPSHOT
2017-09-06 22:36:30,034 [Device List Monitor] WARN  org.bitbucket.mlopatkin.android.liblogcat.ddmlib.AdbDataSource - This kind of log isn't supported by adb source: UNKNOWN
2017-09-06 22:47:38,268 [Device List Monitor] ERROR DDMLIB - DeviceMonitor: Adb connection Error:EOF
2017-09-06 22:47:38,287 [Device List Monitor] ERROR DDMLIB - DeviceMonitor: Connection attempts: 1
2017-09-06 22:47:39,289 [Device List Monitor] ERROR DDMLIB - DeviceMonitor: Connection attempts: 2
2017-09-06 22:47:40,389 [Device List Monitor] WARN  org.bitbucket.mlopatkin.android.liblogcat.ddmlib.AdbDataSource - This kind of log isn't supported by adb source: UNKNOWN

I cannot reproduce the issue anymore and the thread dump got lost after Bitbucket's purge of Mercurial repos.


Handle Envoy 1.26 breaking changes

https://www.envoyproxy.io/docs/envoy/latest/version_history/v1.26/v1.26.0
need to analyze and mitigate any changes

istio/proxy master already tracks envoy's master branch.
but in general you can override it with
--override_repository=envoy=/path/to/envoy
On Wed, Apr 26, 2023 at 8:30 AM Alexandre Pauwels @.***>
wrote:

Is there documentation as to how to build istio/proxyv2 using a different
version of envoy? I'd like to try envoy 1.26 as it contains this commit:
envoyproxy/envoy#26167 https://github.com/envoyproxy/envoy/pull/26167
Which will resolve this issue:
envoyproxy/envoy#25727 https://github.com/envoyproxy/envoy/issues/25727
Which prevents us from using client certs signed using ed25519 in mutual
TLS.
—
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/issues/44422#issuecomment-1523619648, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXLCAN53STSFUASKHQDXDE5P7ANCNFSM6AAAAAAXCHQMPU
.
You are receiving this because you authored the thread.Message ID:
@.***>

So just to clarify:

Clone istio/proxy
Run make build (I'm running it with BUILD_WITH_CONTAINER=1)
Extract resulting envoy binary and use it with the Dockerfile at istio/istio:pilot/docker/Dockerfile.proxyv2
Push resulting proxyv2 image and override it in istio helm charts

Thanks for your help!

Extensions moving/off by default: we should have updated all these
validate_upstream_headers - we should see what changed. Users can set the flag if needed, but we probably want a release note.
x-envoy-auth-failure-mode-allowed added when FailOpen is used on CUSTOM AuthorizationPolicy

Should we scrub that last header for requests outside of the mesh?

I think that is for sending to the ext_authz server which already has plenty of headers and doesn't need to be (and should not be) transparent?


[ci:component:github.com/gardener/aws-lb-readvertiser:0.3.0->0.4.0]

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

Will be closed in favour of #426.


4.0 styled(Component) not returning component

Since upgrading to 4.0 styled(Component) no longer returns a react component, but instead returns an object literal:
import * as React from 'react'
import styled from 'styledComponents'

import { LauncherContainer, WindowContainer } from 'components'
import { Flex } from 'components/_utility'

export interface Props {
  className?: string,
}

const Component = ({ className }: Props) => (
  <div className={className} >
    <Flex column alignItems='flex-end'>
      <WindowContainer />
      <LauncherContainer />
    </Flex>
  </div>
)

export const FrameDisplay = styled(Component)`
  background-color: red;
`

Error:

Dup of https://github.com/styled-components/styled-components/issues/2106. You'll need to hold off on your 4.0 upgrade until react-redux supports React.forwardRef.


Re: Slidebars content disappears when auto close is triggered on tablet/mobile

Hello Adam,
Excellent plugin! It’s exactly what I’ve been looking for. Like Peter referenced in thread [https://github.com/adchsm/Slidebars/issues/169], I also needed an ‘auto-hide’ solution for mobile/tablet devices. The following script you provided in the referenced thread appears to do the trick...,
var yourVariable = new $.slidebars();

function slidebarsStatus() {
  var windowWidth = $( window ).width(),
  breakpoint = 768; // Or whatever your breakpoint is.

  if ( windowWidth > breakpoint && ( ! yourVariable.slidebars.active( 'left' ) || ! yourVariable.slidebars.active( 'right' ) ) ) {
    yourVariable.slidebars.open( 'left' ); // Right or left.
  } else if ( yourVariable.slidebars.active( 'left' ) || yourVariable.slidebars.active( 'right' ) ) {
    yourVariable.slidebars.close();
  }

}
slidebarsStatus();
$( window ).on( 'resize', slidebarsStatus );
...however, I’m experiencing a bit of a glitch that I’m hoping you can help resolve, or at least point me in the right direction. I've provided an example of the glitch via the following link:
http://uwjdesign.com/bench/slidebars/
Here's the problem..., Although the left Slidebar does close automatically when the browser is scaled down (horizontally) below the defined break-point, the Slidebar content disappears intermittently while the site content (“sb-site”) remains in the .open state. Any suggestions you might have would be greatly appreciated. Thanks in advance for your time.
Respectfully,
Ullysee

Hey Ullysee,
So sorry for the extremely late reply, I was aware of this issue, and it should have been fixed with the latest version (2.x).
I'll close this for now, but feel free to open the issue again should you continue to experience any problems.
Thanks,
Adam


What does "tiny" refer to?

What does tiny refer to for implementations under member/tiny? Is this an optimized implementation in terms of space usage or computation cost?

Hi the term tiny refers to implementation optimized in terms of space usage.

Is there a performance-optimized version of the tiny library available?

No that's the only tiny implementation.


IDE Support for WiFi Connections

If your OPENMV is set to connect to your the same WIFI as the computer you're running OpenMV IDE on, you will see connect button (lower-left) allow you to connect over WiFi to your OpenMV Camera. This is not yet full supported, but coming soon according to this thread:
https://forums.openmv.io/viewtopic.php?f=6&t=1854&p=11168#p11168
I spent a few hours trying to figure out why this wasn't working. Hope this helps others.

Yes, it's not functional as of right now.

We got this working now! Coming next release.


Project without cmake nor compile_commands.json

I wonder what would be the "good" behavior for a project without any compile informations.
Now I get usually stuck (with a nil in the function cmake-ide--get-project-key)
I have corrected my own version of cmake-ide to "do nothing", but before create a pull request, I prefere to discuss here what is the good behavior ... I mean how to find there is nothing to do and "disable" cmake-ide when visiting such a file.
It happens for me when I open a code file just to inspect, but this file would not be compiled in a near futur
++ d

This might be a bug - cmake-ide shouldn't do anything if it doesn't find a CMakeLists.txt file. If it does, it's a bug, pure and simple.
However, due to how CMake works, cmake-ide has to keep searching upwards through the filesystem hierarchy to find the topmost CMakeLists.txt file. It's also possible it's acting due to finding one above the file you're visiting.

Ok, let me finalize my tests and I will propose a PR.
++ d

So my investigations turn me crazy.
Could you maybe tell me more how you see the difference between project-dir and cmakelists-dir
Btw, for test test-issue-125 to work, what is expected from get-build-dir without any CMakeLists nor project-dir ?
Thank for your reply, my version is quite functionnal, but clearly need to be tested by other since we do not use cmake-ide the same way.

There shouldn't be any difference between where the top CMakelists.txt is and the project dir. The confusion is due to the code having grown too organically.
For non-cmake projects, get-build-dir should return what it does for cmake projects - by default it builds in a temporary directory, and has to option to be set to one of the user's choosing.
There shouldn't be any manual testing by either of us - every issue should be covered by an automated test.

Regarding the test thing, what I mean is when the test is running, if I understand well, there is no CMakeLists file in the folder structure not project-dir set, but it relies on one of those to get the build-dir (at least in my version, because I see (setq cmake-ide-build-dir build-dir) is reintroduce by commit 69cfce070ee9e23f45736aec7c92c7ce7c86b479
I have no time to get things up (and tested) right now, but I might create a PR soon to show how I try to fix this (and discuss if you are agree with it ;))

The build directory doesn't depend on the project directory. But I guess we'll see better when I see the PR. I really need to add more tests to cmake-ide.


Changing sky_relative default in core.py [issue #23]

The changes made here seem to work for the most part. I also changed the line width types for Poly() and PolyLine() objects from float to AstropyQuantity and gave them their own version of _on_trait_change() as well. A couple of questions:

Is just changing self.sky_relative depending on the unit of the radius OK? The commented _send_msg() doesn't change the unit as it should.

I commented out some print statements that I was using for troubleshooting. Is it OK to leave them for future use, or would you prefer them to be left out of this pull request?

@ojustino - thanks for getting this to work!

yes we should just change depending on the units of the radius, but do you mean that this doesn't work currently if you uncomment _send_msg?

we should ideally remove print statements to keep the 'real' code clean

Nothing breaks, it's just that sky_relative and the units don't change. So if I start in pixels and want to switch to arcsec with _send_msg() enabled, I have to manually change circle.sky_relative. With the current setup, if I switch units, sky_relative will automatically switch as well. I tried with True and False earlier and the same happened; I was just testing different ways of sending the message.

All good now, the problem was the discrepancy between the WWT Web Control name (skyRelative) and its adapted Python name (sky_relative).

@ojustino - looks good! Before I merge, I was wondering whether for line_width we should just use a plain float - for the radius it makes sense to require a unit since there is an ambiguity between pixels and angle units, but for line width since there isn't a choice, maybe we don't need to do anything with units in that case?

We could. I changed line_width because it was the only trait that didn't take an astropy quantity (among those that take numbers of some sort), which I thought didn't fit.

Ok, I'll merge it as-is and we can always revisit later if needed - thanks!


Document Ray class

The Ray class is quite important and needs more documentation.

Moving to v0.3 since we are expecting more changes.

Fixed in commit 2d881a7.


[ci:component:github.com/gardener/etcd-druid:v0.18.1->v0.18.3]

Release Notes:
Bumped up the custom image version to v3.4.13-bootstrap-11

Druid now exposes metrics related to snapshot compaction, on default port 8080. Please expose the desired metrics port via the etcd-druid service to allow metrics to be scraped by a Prometheus instance.

/retest

/test pull-gardener-e2e-kind

/approve
/kind enhancement

/approve
/cc @abdasgupta


Review JSDoc comments for accuracy

What's this issue about?

The JS Documentation of the functions of the code has to be reviewed or filled in many cases. Given the constant refractory we have done, we just focused on creating the functions but not updating the JS Documentation that we add to the top of a function. For instance:
/**
 * Add a label to an issue or pull request in a GitHub repository.
 *
 * @param {InstanceType<typeof GitHub>} octokit - An Octokit instance initialized with a GitHub token.
 * @param {string} owner - Owner of the repository.
 * @param {string} repo - Repository name.
 * @param {number} issueOrPullRequestNumber - Issue or pull request number.
 * @param {string} label - Label to be added.
 * @returns {Promise<void>} A Promise that resolves when the label is added.
 * @throws {Error} - If an error occurs while adding the label.
 */
export const addLabel = async (
  octokit,
  owner,
  repo,
  issueOrPullRequestNumber,
  label
) => {
// code of function
}

In the example above, the JS documentation is correct. However, in many other cases, this needs to be completed or accurate. We mainly do this work using ChatGPT. I would rather pass this task to some other contributor so he can better understand the base code and continue contributing later in the construction of the bot. From my side, I prefer to focus on the release of our beta version, including the development of unit tests.
What do you think @ashwin-pc?

Thanks for the added context @BigSamu. Yep this makes sense :)


TransformError: Unknown plugin "add-module-exports" specified

Quick question: Can I use react-waypoint for React Native project?
When I tried to import it, I got an error as in the title.

@haruelrovix , I've never used React Native, but I ran a few google searches to see if this is a common problem. One recommendation people have would be for us to exclude the .babelrc file from the npm package. It shouldn't be needed for consumers, so that might be fine.
Possibly useful thread here: https://github.com/airbnb/react-native-maps/issues/1206

That sound like a good start, although we'd have to make a few substantial changes to the component in order to remove the dependency on a DOM. Here are just a few places where we assume a DOM of some sort:
https://github.com/brigade/react-waypoint/blob/0ba33d8add28614c321fee97c81f4c94a136ce78/src/isDOMElement.js#L10
https://github.com/brigade/react-waypoint/blob/0ba33d8add28614c321fee97c81f4c94a136ce78/src/resolveScrollableAncestorProp.js#L5
https://github.com/brigade/react-waypoint/blob/0ba33d8add28614c321fee97c81f4c94a136ce78/src/waypoint.jsx#L226
I haven't done any react-native development, and we don't have anyone at my company doing it either. Perhaps @lencioni can reach out to someone to ask if it's a good idea/how to do it?

Webpack 2 is not happy with add-module-exports  and fixing is easy https://gist.github.com/iamakulov/966b91c0fc6363a16ff0650b51fb991b
Can this dependency be removed?

I quickly took a look at your code and I wonder why do you need such dependency? Everything seem to be exported and imported in es6 style.

This is tricky. I agree that it would be nice to get rid of the add-module-exports dependency. But doing so would break a lot of applications (because they would have to require('react-waypoint').default suddenly).
What version of webpack are you using? We're on 2.6 and I'm not seeing this issue.
I wonder if we could use js:next inside our package.json, and support both es6 projects (non-transpiled) and commonjs projects (transpiled with babel, including add-module-exports). Would that be something you'd be willing to explore?

As far as I understand this, webpack finds that .babel in react-waipoint and understands that add-module-exports is required. I'm using webpack 2.5.1 (just did not upgrade to 2.6 yet, they started to release too often.
the thing is that I'm using lot s of different libraries, like redux, redux-saga, react-toolbox, redux-form, store and still none of them required to add add-module-exports.
Not sure if js:next fixes the issue, but if does, it's worth trying

Are you processing node_modules through babel by any chance? babel-plugin-add-module-exports is a dev dependency that we use in the build process. When you have installed react-waypoint, you shouldn't have to use babel to transpile it.
That being said, there might be something we could do to improve this. I noticed that redux has a js:next specified in its package.json, that could be a place to start.

Are you processing node_modules through babel by any chance?

Well, seems like that
  resolve: {
    extensions: ['.js', '.json', '.css'],
    modules: [
      'node_modules',
      'src/main/react', // allows importing modules using absolute path, @see https://goo.gl/luH0Xa
    ]
  },
  //...

        test: /\.js$/,
        loader: 'babel-loader',

I see. For now I think you should exclude react-waypoint from being loaded by babel. Something like
  {
    test: /\.js$/,
    use: {
      loader: babelLoader,
    },
    exclude: [/node_modules\/react-waypoint/],
  },

This should work around your issue.

@trotzig Thanks!
Just tried this, did not help:
ERROR in ./~/react-waypoint/build/waypoint.js
Module build failed: ReferenceError: Unknown plugin "add-module-exports" specified in "[projectpath]\\node_modules\\react-waypoint\\.babelrc" at 1, attempted to resolve relative to "[projectpath]\\node_modules\\react-waypoint"
    at W:\mahjong-website\backend\node_modules\babel-core\lib\transformation\file\options\option-manager.js:180:17

Also tried (since I'm on windows):

        exclude: [
          /node_modules[\\\/]react-waypoint/
        ]

but this did not help as well, I faced this:
ERROR in ./~/consolidated-events/lib/index.js
Module build failed: Error: Couldn't find preset "airbnb" relative to directory "[projectpath]\\node_modules\\consolidated-events"
    at [projectpath]\node_modules\babel-core\lib\transformation\file\options\option-manager.js:293:19
    at Array.map (native)

So I had add that to excude as well:
        exclude: [
          /node_modules[\\\/]react-waypoint/,
          /node_modules[\\\/]consolidated-events/
        ],

and now it works!

Glad you found a workaround!

Yep, but a plug-and-play solution should exists. Maybe it will be a js:next one. Feel free to ask me to check when npm version will be ready.

Alright. I changed the title to better reflect what the goal of this thread is.

We'll likely need two builds for this – @tbranyen and I have been chatting about supporting this in libs at work 🤔

i had same problem yesterday.
i tried

npm i babel-plugin-add-module-exports --save-dev
to fix 'add-module-exports' error
and
npm install --save-dev babel-preset-airbnb
to fix 'airbnb' error
And it works!

Thanks for the update, @Godsenal !

As an aside, I just updated this issue's title to be more accurate. You can use Waypoint as an ES6 module. This issue is specifically about react native support, which is entirely different.

I'm using react-waypoint in a basic app using webpack (not using React Native so not sure if I'm hijacking this thread) and still have this issue.
@jmeas @Godsenal I did the same fix suggested & it worked for me however I don't believe this is the long term solution?

@robhuzzey I see, thanks for reporting. I wonder if things would just work ™ if the .babelrc file wasn't there? Would you mind removing it manually (something like rm node_modules/react-waypoint/.babelrc) and trying to see if that makes things work?
We use babel to transpile the react-waypoint "bundle" (the file targeted in main from package.json). But if folks want to babel things themselves, we can just leave it up to them to configure. We can just exclude .babelrc when publishing the npm module.

@robhuzzey are you transpiling your node_modules with Babel using the Babel loader? If so, I would expect this error, but also, you shouldn’t need to do that. Most libraries, including Waypoint, are already transpiled, so you can blacklist the whole node_modules directory.
My guess is that, if you are currently transpiling your node_modules, then ignoring them will fix this issue, and make your build step much faster.
—
@trotzig , I agree that removing .babelrc from the node module makes sense.

Anyone had any luck with getting it to work with React Native or should I attempt to port it ? :)

@jamesplease did the idea to have two versions ever go anywhere?


passport-google-oauth updated and is now using google+

I've fixed our version to 0.1.5 which works for now but we need to look into how we have to proceed with this.

Not sure how relevant this is, but google is fasing out OpenID, so we might want to look into how long they're going to keep the oauth endpoint around

https://developers.google.com/accounts/docs/OpenID#shutdown-timetable
The above indicates we must have the switch done by 20th April, which makes
this quite urgent.
On 16 March 2015 at 11:54, Simon Gaeremynck<EMAIL_ADDRESS>wrote:

Not sure how relevant this is, but google is fasing out OpenID
https://support.google.com/accounts/answer/6135882?authuser=0, so we
might want to look into how long they're going to keep the oauth endpoint
around
—
Reply to this email directly or view it on GitHub
https://github.com/oaeproject/Hilary/issues/1140#issuecomment-81603726.

We're not using Google's openid though. We're using oauth. Everything should remaining working after April 20th, but we should probably try and migrate sooner

Are we using Oauth2? If so no worries :)
On 16 March 2015 at 13:52, Simon Gaeremynck<EMAIL_ADDRESS>wrote:

We're not using Google's openid though. We're using oauth. Everything
should remaining working after April 20th, but we should probably try and
migrate sooner
https://developers.google.com/+/api/auth-migration#timetable
—
Reply to this email directly or view it on GitHub
https://github.com/oaeproject/Hilary/issues/1140#issuecomment-81678036.

I've attached some code that addresses this. Here are some basic steps to verify this PR:

On master: enable google authentication
On master: create a user account by signing in through google
On master: Create some content with that user
Sign out
Switch to this branch
Run npm install
In the Google developer's console, enable Google+ integration (I've done this for the default key that works for cam.oae.com already)
Restart Hilary
Sign back in with the user
Assert you signed into the correct account

Maybe also try out configured the hostedDomain when bugbashing/verifying

I pushed a commit that addresses #1147

@simong so is authentication using Google+ tied to a user's API key or is there some kind of organizational concept for Google+ behind the scenes where you basically tie an API key to a domain?

Looks good for the most part, just some recommended refactors. Back to @simong for followup before I do some dev testing.

@simong so is authentication using Google+ tied to a user's API key or is there some kind of organizational concept for Google+ behind the scenes where you basically tie an API key to a domain?

IIRC, you need to create a project in Google's developer console. Here you can:

Create a new OAuth client. This client has:

An id
A secret token
Scoped to a domain

Enable API integrations

You need to enable the Google+ API:

Pushed some follow-up. Back to @mrvisser for more review

Coverage increased (+0.05%) to 90.15% when pulling c69e234c1b5086064f4acd327948eb39ed0e11b4 on simong:google-oauth into a4173dba2770964d1cb9152cb0ebe4c685f5e340 on oaeproject:master.

I'll change that 2015. This is good to merge.
FWIW, the "old" client ID and secret seems to continue to work as well, probably until the old system is complete phased out.

Yes, the old client ID and secret shouldn't have to be changed


Allow redefinition of compiled gen function

Currently the name of a Gen function is used as the name of an underlying data type. Since data types cannot be redefined, compiled Gen functions cannot be redefined. This should be possible to solve by having the gen function use a new symbol for the name of the type.

Fixed in


Document how people using Task's volumes field can migrate to Workspaces

We are considering removing volumes from Tasks.  See issue https://github.com/tektoncd/pipeline/issues/2058
If we're going to do this then we should document the path for users who are currently employing the Volumes field to migrate over to Workspaces instead.

We are no longer planning to remove volumes from tasks (#2058) and so this migration doc doesn't need to happen.  Closing.


feat(cli): backstage-cli plugin:export

This introduces a backstage-cli plugin:export option that simply builds one-time what backstage-cli plugin:serve would do. The use case for this is to support Backstage local documentation previews, although this could also later down the line be helpful for live demos of Backstage plugins.
Hey, I just made a Pull Request!

:heavy_check_mark: Checklist

[ ] All tests are passing yarn test
[ ] Screenshots attached (for UI changes)
[ ] Relevant documentation updated
[ ] Prettier run on changed files
[ ] Tests added for new functionality
[ ] Regression tests added for bug fixes

Naming things is one of the hardest things in CS, right? :D


[Github] Configure the repository to require sign-off for web commits

Description
Twice now I've tried to use the Github web UI to make a one-line change to documentation, and both times they failed CI because I didn't add a signoff to the commit.
Repositories that require signoff are supposed to mark this in their repo settings so that the Github web UI automatically adds it. Please do this to reduce churn.

CC: @mundaym

@kupiakos Thanks, sign-off of web UI commits should now be enabled.

Thanks @mundaym !

Thanks @mundaym, I've verified this works with https://github.com/lowRISC/opentitan/pull/22202.


Dictionary to JSON question

Hi Tim,
Thank you so much for the converter!  This has been such a great help!  I have a few question about the converter and can't seem to figure it out, and I'm just wondering if you could help me out.
1.)
I have a dictionary data structure that contains about 70k objects(these objects only contain 2 strings).  Regarding the ConvertToJson function, when I use the dictionary you provided (mac-support), it took about 8 seconds to convert, but when I use the windows scripting dictionary it took 60 seconds.  I was wondering if you could shed some light on this.
2.)
The reason I asked the first question is because in order to create the 70k dictionary structure, I extracted the data from a spreadsheet, and when I created the dictionary structure using the mac-support dictionary , the .add function took significantly longer.  When creating the dictionary structure, the mac-support dictionary took around 30 seconds, whereas the windows scripting dictionary was almost instant.
I'm at a dilemma here because I'm between 2 options of choosing a 40 seconds convert and a 60 seconds convert, of which both are not ideal.  I am in the process of finding a way to maybe use the scripting dictionary for the first half of the program and convert it to the mac-support dictionary for the ConvertToJson part.  However, I do want to avoid this because there will be mac users using the macro I'm writing.  Any help will be much appreciated.
Thank you,
Kevin
Edit:  From what I can see, the mac support dictionary is basically the same as the scripting dictionary with preprocessed platform check.  From what I understand the performance should be about the same, so I'm pretty stumped here.

Hi @kchen12 you're right that for Windows, VBA-Dictionary just calls Scripting.Dictionary underneath, but there is a definite (but generally minor) performance penalty (there are some performance tests in VBA-Dictionary specs). I haven't seen a case where VBA-Dictionary is faster than Scripting.Dictionary, so if you have a specific test case that would be interesting to see.
I looked through VBA-JSON and New Dictionary only occurs in one place so it would be pretty easy for you to move the platform check to that location and avoid using VBA-Dictionary on Windows (to skip the dispatch penalty):
#If Mac Then
  Set json_ParseObject = New Dictionary
#Else
  Set json_ParseObject = CreateObject("Scripting.Dictionary")
#End If

(I have no plans to add this, but you could do it in your performance-critical code and see if it helps)


Nit: repo link in package.json is a link to badge.fury.io

This is a minor issue, but I often use the tooltip over a dependency in package.json file to go to the repos of my dependencies. Almost every other project has a link to their repo in the hover state. However, functionless projects have a link to badge.fury.io images.

vs

Thanks for pointing this out, do you know where this information is configured?

Hard to say. I think it might be coming from the first line of the README file. If you look at the Yarn package page of both packages, you can see the first line is the bumper in the tooltip.
https://yarnpkg.com/package/aws-cdk
https://yarnpkg.com/package/functionless

Or maybe it's the description field of the package config....


Add work configuration filed for cluster manager

Signed-off-by: zhujian<EMAIL_ADDRESS>

/lgtm


Timestamp on digital signatures

Hi,
I have noticed that when I create an update package there is no timestamp showing on the digital signatures tab on the CAB file.  Basically showing timestamp not available.  Is there any way to fix that?

Hi Daskydasky, the latest release of WPP will timestamp newly published updates. If you use a recent release of WPP, it should download the latest release and upgrade itself. Otherwise, you’ll have to manually download the latest version here.


Change the install.md documentation about how to build peer and membersvc image

Description
The install documentation instruct us to build the image using this procedure.
go test github.com/hyperledger/fabric/core/container -run=BuildImage_Peer
go test github.com/hyperledger/fabric/core/container -run=BuildImage_Obcca

This is no longer valid, as these methods have been removed from vm_test.go file. Please change the documentation appropriately.

@muralisrini @ghaskins - With regards to the behave tests:  are the above parameters still correct when using the "go test" function to build the peer and member services images.  I would have imagined that all instances of OBC had been deprecated by now.

Thank you @nickgaski.
@ibmamnt Are there still related documentation corrections/suggestions that should be submitted under this issue?

Looks like the original install documentation has been refactored into several .md files. If I picked up one of them, it correctly described how to build image. So I think this issue can be closed.


change minTraceroutePktSize

ref: https://github.com/openconfig/featureprofiles/pull/687

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
586

Relevant Lines:
1261

💛  - Coveralls

@bstoll I think this PR itself it's ok because I don't see that the test currently specifies the size. The default value would be upto the system/OS implementation. Raising up the bar here seems fine to me. Also reference https://github.com/openconfig/featureprofiles/pull/687
My question is more about the test itself. The test plan talks about different packet size (not TODO), but I don't see it in the code. Also, I don't see gNOI API supports packet size. Shall we completely remove the mentioning of packet size at all from this test? Thoughts?

I agree that in the current version of this RPC spec it doesn't make a lot of sense to look into the packet size.  It would be useful to be able to specify the packet size but that would be a change to the GNOI spec first before we can test for it.
I think we could completely remove the packet size from this version and tie it as a new feature requiring GNOI spec to be updated first.

Thanks, @bstoll. SGTM.
@Hobbydos can you help use this PR to remove all the packet size related code and in the README too? Thanks.
cc: @vcheny @sachendras @vosipchu. We plan to use this PR to remove packet size related tests (README part and code part), because gNOI API currently does not even support it.  See https://github.com/openconfig/featureprofiles/pull/944#issuecomment-1372598410 and https://github.com/openconfig/featureprofiles/pull/944#issuecomment-1374006566 for more context.
It should have 0 impact on passed tests. Ping me if any objection, otherwise I"ll merge the PR upon my own LGTM.

Thanks, @bstoll. SGTM.
@Hobbydos can you help use this PR to remove all the packet size related code and in the README too? Thanks.
cc: @vcheny @sachendras @vosipchu. We plan to use this PR to remove packet size related tests (README part and code part), because gNOI API currently does not even support it. See #944 (comment) and #944 (comment) for more context.
It should have 0 impact on passed tests. Ping me if any objection, otherwise I"ll merge the PR upon my own LGTM.

Removed pktSize associations to test file and readme file, PTAL and let me know if anything needs changing.

@xw-g  @jjing1122 Does this PR look good for merging? Once confirmed, I'll add deviation to this test.

/gcbrun


能否支持if 和表格循环？

能否支持if 和表格循环？

Hi. I guess this is a feature request to support iteration over arrays and conditional rendering (if statement). It would be nice to be able to generate table rows based on some array of structs :)

Hey, sorry for getting back so late. I'm not really monitoring this repo anymore.
I've experimented with conditional rendering as well as insertion of new data.
But implementing this would be very complex since the project's aim is not to project the WordprocessingML spec into a Go-struct. It is meant to work on the raw bytes of the document.
Thus injecting new XML structs is somewhat out-of scope, sorry.
There is one project on github which is aimed at such use-cases: https://github.com/unidoc/unioffice
Unfortunately that project is not free of charge, which is why I've developed this library for my specific use-case.
But if you want to introduce that feature into this lib, I'm happily accepting PRs :)


Give PD the ability to detect offline

If the PD does not receive a message from the CP(any message, including a poll), then it will now indicate that it is offline through the osdp_get_status_mask method.

See issue: #133

I had something else in mind, let me push that change to a branch so you can take a look if it works for you.

See: https://github.com/goToMain/libosdp/pull/135


Fix logger fmt

I fixed logger format.

/kind bug

/assign @wzshiming

/ok-to-test

The logger library has been updated now, so let me close this. Thank you.
/close


How does one determine a fileId/documentId using the python SDK?

Your environment
Built from main, commit 678d60 from November 9th (because the document support committed in October hasn't been released yet)
Connect Server Version: 1.5.0
OS: macOS 11.6.3

Python Version: 3.9.10
What happened?
The API for download_file, consistent with the library's other file APIs, expects a file ID, an item ID, and a vault ID.
Starting from a vault name and an item name, I can determine the vault ID and the item ID, but the output of get_item doesn't include a file ID. It is included in the CLI output of get item, under a different name:
op get item <document-item-id> --vault <vault-id> | jq -r .details.documentAttributes.documentId

The same documentId is shown via the Copy JSON command in the Mac app, and when I feed that identifier to the download_file API, it succeeds. But it isn't useful if I can't programmatically determine the ID, and I am not seeing any way to do that.
What did you expect to happen?
I should be able to use the python library, vault access, and knowledge of a file's name to access the file.
Notes & Logs
Here's the output of get_item:
{'category': 'DOCUMENT',
 'created_at': datetime.datetime(2022, 2, 8, 0, 10, 12, tzinfo=tzutc()),
 'favorite': False,
 'fields': [{'entropy': None,
             'generate': False,
             'id': 'notesPlain',
             'label': 'notesPlain',
             'purpose': 'NOTES',
             'section': None,
             'type': 'STRING',
             'value': None}],
 'id': 'a7qk2fixrbalrifv3heqcxntyi',
 'last_edited_by': 'FRAQH4JNUJG6XPMLJO2GVA72T4',
 'sections': None,
 'tags': None,
 'title': 'redacted-file-title',
 'trashed': False,
 'updated_at': datetime.datetime(2022, 2, 8, 0, 10, 13, tzinfo=tzutc()),
 'urls': None,
 'vault': {'id': 'jm4ixdhaljgodbzehdxjxxwagy'},
 'version': 1}

Thanks for your time.

Hi there!
Looks like the solution to this is not currently documented, so thank you for bringing this issue to our attention. To get data on files (including the file ids) for an item you will need to use the get_files method.
For example the following:
client.get_files(<some_item_id>, <some_vault_id>)

Will return an object of the following format:
{
        'id': 'str',
        'name': 'str',
        'size': 'int',
        'content_path': 'str',
}

I'll update the README to reflect this information. Please let me know if this resolves your issue.

Thank you, yes.


Vue3.0 cannot be used in Firefox. Vue2.0 can

Hey! 👋
I second this.
Latest release of the Vue DevTools does not work with Vue 3.
I am getting the following error in the console:
TypeError: api.on.editInspectorState is not a function

TypeError: api.on.visitComponentTree is not a function

Please use the issue helper. https://new-issue.vuejs.org/?repo=vuejs/vue-devtools

It is because firefox gas devtools v5. Only devtools V6(currently in beta) supports Vue3

Version beta https://github.com/vuejs/devtools/releases


A user can only operate on the data owned by himself.

Add data level access control, A user can only operate on the data owned by himself or can operate on any data based on the role definition

Some thoughts

Separate xxx_view to xxx_view_self and xxx_view_all etc.
Related to #30

See https://github.com/flask-admin/flask-admin/issues/1104 for how to access current user in the form.

 A user can only operate on the data owned by himself


Wrapped missing WebSecurity, Roles and Membership methods to make code more testable.

Only some of the WebSecurity methods were wrapped. I wrapped the rest and all calls to Roles, Membership and controller.CurrentUser to make future testing more easy.
I also did some cleanup/refactoring of the existing controller tests.

I'll make a new pull request.


Added code to get UUID

Returns order UUID from Openfinex to Client. Will close #221

Hey @haerdib , where I can call this function :- https://github.com/Polkadex-Substrate/polkadexTEE-worker/blob/82c15d151e4a11e31f949e387e6802dfbcfbff07/enclave/src/rpc/worker_api_direct.rs#L286?

Hey @haerdib , where I can call this function? :-
https://github.com/Polkadex-Substrate/polkadexTEE-worker/blob/82c15d151e4a11e31f949e387e6802dfbcfbff07/enclave/src/rpc/worker_api_direct.rs#L286

I think it would make sense to place it here: https://github.com/Polkadex-Substrate/polkadexTEE-worker/blob/82c15d151e4a11e31f949e387e6802dfbcfbff07/enclave/src/polkadex_gateway.rs#L339-L340

Update
These two test cases are failing:-

https://github.com/Polkadex-Substrate/polkadexTEE-worker/blob/zk-get-uuid/enclave/src/tests.rs#L142
https://github.com/Polkadex-Substrate/polkadexTEE-worker/blob/zk-get-uuid/enclave/src/tests.rs#L121

Reason
We are making ocall here :- https://github.com/Polkadex-Substrate/polkadexTEE-worker/blob/zk-get-uuid/enclave/src/polkadex_gateway.rs#L344. And as this is not called by given test cases. It is returning None.

I think we have to do something like benchmarking, or is there any other way 🤔  @haerdib

I think there might be an easier solution: Instead of unwrapping here we could just return  sgx_status_t::SGX_ERROR_INVALID_STATE or whatever error you want to return (sgx_status_t or another enum, etc...).
The same should actually also be done here and here. Unwrapping (=panic) on worker/enclave side is not good anyway.

sure I will do that. But to run those tests we need Ok() from those function. 🤕

Isn't an Ok(()) returned anyway here?

Because of this, its not returning ok(). I think we have to use features to ignore this particular line.

@haerdib please look into this https://github.com/Polkadex-Substrate/polkadexTEE-worker/blob/zk-get-uuid/worker/src/enclave/worker_api_direct_server.rs#L245-L273 and https://github.com/Polkadex-Substrate/polkadexTEE-worker/blob/develop/client/src/main.rs#L732-L757. Let me know if have to change anything on enclave side.


future proofing: use ManuallyDrop rather than mem::forget for callback management

Feedback from zulip is questioning how I'm handling the forgetting of the callback type, namely Elichai Turkel notes:

Doesn't this moves the callback type from the stack and thus invalidating the pointer? IMHO it's easiest to use a ManuallyDrop

https://github.com/pnkfelix/cee-scape/blob/1ce76e4aefdabacae29696846d41189d57d688a9/src/lib.rs#L314

See also discussion on https://github.com/rust-lang/unsafe-code-guidelines/issues/188


Can WeChat applets be developed using openlayers？

I need to develop the map function on the WeChat applet, but there was an error when building the npm package of the applet, so I want to get some answers as soon as possible
1.Whether the WeChat applet can use openlayers to develop the map function, and whether there will be compatibility problems
2.If there is a compatibility problem, whether there is a good solution to the compatibility problem, or what I should do to avoid this problem

The error message you pasted above is nothing that has to do with this repository. The package here is ol, not @mapbox/jsonlint-lines-primitives, and ol does not depend on that package.
Please provide more detail to help us figure out what you want.