Error comparing licenses in production test instance

The following error occurs when running check license:
Traceback (most recent call last): File "/spdx/src/app/views.py", line 1001,
in check_license matchingId,matchingType = utils.check_spdx_license(licensetext) File "/spdx/src/app/utils.py", line 489,
in check_spdx_license build_spdx_licenses() File "/spdx/src/spdx-license-matcher/spdx_license_matcher/build_licenses.py", line 44,
in build_spdx_licenses normalizeText = normalize(licenseText) NameError: global name 'normalize' is not defined

This occurs in the production test instance which can be accessed at http://<IP_ADDRESS>/app/check_license/

The issue is caused by this PR : https://github.com/spdx/spdx-license-matcher/pull/1/files
In spdx_license_matcher/build_licenses.py , imports have been removed which are used later in the file.
cc @Ugtan @goneall

@Ugtan Do you want to submit a PR adding the imports?
@rtgdk Thanks for identifying the error.

Sure @goneall.

I merged the PR - it solved the initial problem.
However, I ran into a different error when the license text did not exactly match:
Traceback (most recent call last): File "/spdx/src/app/views.py", line 1001, in check_license matchingId,matchingType = utils.check_spdx_license(licensetext)
File "/spdx/src/app/utils.py", line 505, in check_spdx_license listedLicense = getListedLicense(licenseID)
File "/spdx/src/spdx-license-matcher/spdx_license_matcher/utils.py", line 88, in getListedLicense listed_license = licenseinfofactoryclass.getListedLicenseById(licenseId)
File "/usr/local/lib/python2.7/site-packages/jpype/_jclass.py", line 78, in _getClassFor def _getClassFor(javaClass):
File "/usr/local/lib/python2.7/site-packages/gunicorn/workers/base.py", line 202, in handle_abort sys.exit(1) SystemExit: 1

@goneall The getListedLicense function uses spdx tools to get a spdx listed license from the license ID.
Gunicorn has aborted the process for some reason. I found this on the internet. I guess changing the timeout value might help. Refer to the link to add the value of timeout in supervisor_api.conf file. Let me know if it works.

@Ugtan Thanks for the research.  I think I know what's happening.  The first time you call the license match in Java, it loads all of the licenses from the internet which can take some time - likely causing a timeout.  I'll find a way to pre-load the licenses and see if that works.

@goneall If that's the case then let me know if you find a way to preload the license or else, we have have to find another way to build the licenses in the database.

@Ugtan I just updated the branch with a fix when there is a 100% match - it now works 👍
Unfortunately, if there isn't a perfect match, I'm getting a timeout.  Before my fix, I didn't get a timeout if there wasn't a perfect match, so I'm not sure what has changed.  I suspect it may be a configuration problem, but I'm not sure.
I'll do a bit more research tomorrow.

Just an update - I tried this on my development docker instance and it takes 40 seconds (!)  Not sure what is is computing, but it does eventually return.
It looks like we have a performance issue with near matches.  If the match is perfect, the performance is acceptable.  If the match is way off, the performance is good.
@Ugtan if you could do some investigating, that would be great.
I also noted a related issue: https://github.com/spdx/spdx-online-tools/issues/210

The error was resolved with PR #212 which increases the timeout.
Added a new issue to track the performance issue #213

It looks like we have a performance issue with near matches. If the match is perfect, the performance is acceptable. If the match is way off, the performance is good.

@Ugtan if you could do some investigating, that would be great.

The performance issue could be because we are bound to use a bridge to run java code in python. But, I will try to look for a workaround that will help in the computation speed.


config: simplify tests by using testutil.NotOk

Simplify package config tests by using testutil.NotOk.
When a "table driven test" fails due to testuitl.NotOk, there is no way to know which entry in the table is actually failing, because, as opposed to testutil.Ok, testutil.NotOk does not receive any information from the test (its input is nil when the test fails). To fix this I have added a formatted message to testuitl.NotOk.
Also, I have guaranteed all config.LoadFile errors include the filename so people can know what file is not loading correctly.

Thanks!


Possible plugin: Lensing

Hi!
I've enjoyed working with OSD for specific application that requires the use of a "lens". The tooling was deliberately abstracted for other use cases as a plugin - if the OSD team thought it might be useful for others in the community. It can be implemented pretty quickly:
this.viewer.lensing = Lensing.construct(OpenSeadragon, this.viewer, viewer_config, dataLoad);

Feel free to check the demo, the Github repo and the npm package.
Perhaps this is not the best approach for asking - if so, I apologize.  However, I was checking in while this is still in development to determine if it worthwhile to continue working it as a generic plugin.
Tks!

That's really slick! Seems like it would be a great plug-in, so yes, if you want to do publish it as such, that would be awesome. Let me know if you need any pointers; it should be fairly straightforward.
BTW, while I found it to be smooth on Chrome, it flickers a lot on Firefox.

Thanks! Sorry for any bugginess - alot more quality control work is needed O_O. We'll keep building to be for general use and pass in domain-specific data analysis filters separately, and then will reach out as we get closer to production ready. Many thanks for all the work you do on this awesome tool!

Sounds good. Looks like a very promising project :-)


Update int8 reference bitpack code

What do these changes do?
I noticed that on Cortex-M the generic int8 bitpacking code compiles to a ldr+str for every OR'd bit instead of first ORing 32 bits and then storing the result.
How Has This Been Tested?
Manually verified the generated assembly.

I just realized that there was another easy tweak that also removed a few cycles per element on Cortex-M, so I also pushed that on this branch.


More cypress tests included

Managed to translate the old tests for the cluster's tooltip into to the new UI/UX.
Still have to figure out how the UI/UX of the beadplot's tooltip and write tests for them.

closed the pull-request because 6 of the 12 new tests were failing. not sure why. but all failures were timeout errors. The UI elements were not detected. The tests passed on my local machine so I'll have to look more into this.


Automated releases (CI/CD)

Is your feature request related to a problem? Please describe.
Releases currently require me to manually deploy them.
Describe the solution you'd like
I want to setup CircleCI with semantic-release to handle this automatically.
Describe alternatives you've considered
We could use other CI tools, but I have done most professional projects with CircleCI, so my preference lays there.
Additional context
It would require a standard for composing commit messages.

FYI I'm using CirrusCI in other MRs for unit test and coverage (#202, #203)


Add cred file handling for connect

@sfc-gh-jsummer I ran pre-commit over this to clean a few things up, this looks good! I think it'll be helpful.


Package for Meteor

Any chance on getting this packaged for meteor?  There is an existing package that seems to be falsely claiming to be an official package.  https://atmospherejs.com/swiper/swiper

Yes, it is not official. Will take a look to package it for meteor. As i understood i just need to create package.js file for Atmoshpere?

A package.js for atmosphere, then meteor publish --create when in the directory to actually publish the package.

Ok, will do it

I created a PR for this https://github.com/nolimits4web/Swiper/pull/1473.

Published


prepdocs.py doesn't support spaces in file path

Please provide us with the following information:

This issue is for a: (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

Download the solution in a directory whose path contains spaces

Any log messages given by the failure

Running "prepdocs.py" usage: prepdocs.py [-h] [--category CATEGORY] [--skipblobs] [--storageaccount STORAGEACCOUNT] [--container CONTAINER] [--storagekey STORAGEKEY]                    [--tenantid TENANTID] [--searchservice SEARCHSERVICE] [--index INDEX] [--searchkey SEARCHKEY] [--remove] [--removeall] [--localpdfparser]                    [--formrecognizerservice FORMRECOGNIZERSERVICE] [--formrecognizerkey FORMRECOGNIZERKEY] [--verbose]                    files prepdocs.py: error: unrecognized arguments: - Microsoft\Documents\OpenAI\azure-search-demo/data/*

Expected/desired behavior

prepdocs.py escapes any spaces in the file path

OS and Version?

Windows 11

azd version?

1.0.1

Versions

Mention any other details that might be useful

Thanks! We'll be in touch soon.

I'd say spaces in paths is bad form.. but if you want to support it submit a PR

When working Microsoft OneDrive, the path automatically includes a space. It is {my name} - Microsoft. What workaround do you suggest @EMCP?

to clarify, are you loading docs from a one-drive account.. and many of the directories it creates or uses have spaces?

I'm cloning the repo to my local computer. The OneDrive directory on my computer is called '{my name} - Microsoft', which has a space in its path.

I'm cloning the repo to my local computer. The OneDrive directory on my computer is called '{my name} - Microsoft', which has a space in its path.

if it were me, I dunno if this is possible since I don't know OneDrive or your particular computer.. but I'd just copy the cloned repo out of OneDrive, as well as any data you want to work with .. if possible.
I am assuming you cannot do this though because of the size of the local hard drive and your datasets?

This workaround worked for me. Thank you @EMCP. However, others will run into this issue in the future without knowing the cause. To enhance UX, I would suggest fixing this issue or adding a hint in the documentation.

I agree with socullen - it would be great to have a note in the readme to ensure the solution is downloaded to a file path without spaces, and/or a more apparent error message - the one I shared above is not highlighted as an error message in the console.

I do not work @ MSFT , but I would guess they'd welcome a PR stating what bad state you found and the workaround..?  or even patching the behavior away entirely.. I do not have a windows machine or onedrive

The relevant line of code is https://github.com/Azure-Samples/azure-search-openai-demo/blob/main/scripts/prepdocs.ps1#L39C79-L39C82 - where it uses $cwd in the call to prepdocs
I'd love a PR to patch that up. (Am not much of a Powershell writer myself since I don't have a Windows machine, but I assume it could be put in quotes or escaped)

Actually, there was already a PR:
https://github.com/Azure-Samples/azure-search-openai-demo/pull/71
It is now merged, so this should no longer be an issue.


Move Service Bus Pubsub/Binding to common auth

Description
Both the pubsub and input/output binding for Azure Service Bus were
connecting via a connection string. This is still supported but will
now fallback to using AAD from the common auth library. This is also
the recommended auth pattern going forward.
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Partial: #1103
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[x] Created/updated tests
[x] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: https://github.com/dapr/docs/issues/1867

FYI, @halspang and I successfully verified this works with managed identity now
apiVersion: dapr.io/v1alpha1
kind: Component
metadata:
  name: azure-servicebus
spec:
  type: pubsub.azure.servicebus
  version: v1
  metadata:
  - name: namespaceName
    value: "dapr2-conf-test-servicebus"
  - name: consumerID
    value: "helloworld"

We ran the conformance test in an environment with managed identity where the identity was given "Azure Service Bus Data Owner" role for the particular Service Bus Namespace

FYI there's a new "track 2" SDK for Azure Service Bus: https://github.com/Azure/azure-sdk-for-go/tree/main/sdk/messaging/azservicebus
Eventually the component will need to be updated to that (and pending merge of #1290 for auth support)


Timeline.addTweenAt

This is unnecessary for the vast majority of tweens, but could be helpful for the rare case where people are trying to add custom content into exported MovieClip elements at runtime.
See:
http://community.createjs.com/discussions/tweenjs/563-timelineaddtween-problems

The .tweens property is now public, which partly resolves this, though it's still not ideal since we want to ensure tweens are removed from any previous Timeline before being added. I'm going to leave this open for now, and we'll see if it comes up again.


[ci] Sival jobs timing out after running only a few tests

Description
Recently we have seen several silicon validation CI jobs get timed out at 45 minutes after only executing ~12 tests. Bazel's timeout after each test is 60 seconds as far as I'm aware. In the example I'm looking at only 10 tests have failed so at most that's a 10 minute runtime if they all timed out.
This needs investigating.
Here is an example Azure run:
https://dev.azure.com/lowrisc/opentitan/_build/results?buildId=148506&view=logs&j=5011a995-22d7-5073-e1ce-26fc382a8885&t=875997a6-7e2b-594d-05e6-ce6a085f3383
And a copy of the logs for posterity (Azure cleans up after 14 days):
https://gist.githubusercontent.com/jwnrt/93653a0e57a58c103f2e333514459995/raw/597e5de4c4693c36491d9ed15503ec2e0464c661/gistfile1.txt

I've bumped the timeout on this job up by 30 minutes in #22557 since successful jobs are taking ~40m and it might be legitimate to increase it. I'm still skeptical though given only ~12 tests even run before the timeout.
We've also started seeing the other sival job timeout at 60 minutes similarly after only executing 18 minutes.

Note that in the linked gist, basically every test has fallen over and timed out. The FPGA OT's UART is outputting nothing, and it looks like only JTAG somehow works.


components/init.py: Expose component functions for direct import

Description:
This PR updates the components/__init__.py to expose main component functions (create_chart, create_grid, create_markdown) for direct imports. This provides a more user-friendly import experience and aligns with the README's example usage.
Changes:

Updated components/__init__.py to import create_chart, create_grid, and create_markdown for direct accessibility.

Motivation:
Users can now perform the following import:
from mesa_interactive.components import create_chart, create_grid, create_markdown

instead of diving into sub-modules, like this:
from mesa_interactive.components.charts import create_chart
from mesa_interactive.components.grid import create_grid
from mesa_interactive.components.markdown import  create_markdown

Closes #1.

Thanks, that makes more sense


Consider committing grammar.json

While keeping parser.c out of version control is reasonable (and in fact recommended by upstream), the generated grammar.json is much smaller and has meaningful diffs. The benefit of keeping this in the repo is that downstream users (like nvim-treesitter) do not require node for building and can just do tree-sitter g grammar.json && tree-sitter build -o parser.so (which is what nvim-treesitter is transitioning to for 1.0).
In addition, consider making proper releases including the generated artifacts (which is what nvim-treesitter will require for "stable" parsers). Upstream is in the process of adding reusable workflows and tree-sitter support to make this easier.

This is a good idea. I tried to keep a separate branch for artifacts but it's just kind of a hassle. I'll look into what's need for these proper releases, too.

https://github.com/tree-sitter/workflows

@clason wanted to thank you for the pointers to workflows, I followed it and have CI set up, including a fuzzer, and the fuzzer caught memory leaks that I've since fixed. You made my code better!


HeatmapWithTime is visualized in the back of overlay image

HeatMapWithTime is visualized in the back of the overlay image, even after including  in my html.
According to https://github.com/python-visualization/folium/issues/1201, including this  in my html should bring the Heatmap to the front. However, that did not work.

Hi, and thanks for taking the time to open a bug report in Leaflet.
However, in this repository we only handle bugs in "vanilla" Leaflet. This means that we do not handle bugs which are specific to frameworks such as:

React
Ractive
AngularJS
Bootstrap
Leaflet for R
Joomla
Wordpress
Ionic
Cordova
Vue
Electron
Polymer
TypeScript
D3
Folium
NativeScript
...etc

Please understand that we only have the time and energy to test Leaflet in plain web browsers.
Please try to either reproduce the bug without using any frameworks, or submit a bug to the appropriate repo.

If anybody else has problem with HeatMapWithTime drawing data behind ImageOverlay. Instead of using above mentioned line of HTML code which tries to push heatmap layer foward.
You can instead push ImageOverlay back with code  (Index -1 sets image back of the stack).


fix: Make operations correctly model service errors

Issue #

Addresses the Issue #1080
Description of changes

Adds code generation for a helper method makeService that handles service errors to AWS protocols.
Corresponding PR in smithy-swift: https://github.com/smithy-lang/smithy-swift/pull/581
Scope

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

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

/test

/test

/test

/test

/test

/test


[Tab] All pane contents should have a top border (React)

### Steps
Use react implementation of tab component pane shorthand example with renderActiveOnly={false}.
Use codepen link below to see issue.
### Expected Result
Every pane content should have top border.
### Actual Result
Notice the first tab's pane content gets borders. However, any other tab's pane content is missing its top border. Looking at the CSS it looks like border-top: none; is added to every tab other than the first.

### Version
2.3.1
### Testcase
https://codepen.io/celik88/pen/deGaVN

@hammy2899 I believe the issue is when using it in conjunction with the "attached" class / prop. If we don't declare it, the first tab content gets the top border and others don't. If we include it, "attached:true" in the menu prop then we don't see the issue. I removed the attached class from the example you have provided and can see a similarity. However, in your example when I removed attached class none of the tabs have the top border. Thus, either the first tab having the top border is the bug on the react implementation or all other tabs not having the top border is the bug. Not sure what the expected result should be, but I think we can agree that there is an inconsistency.

Also, in your example try adding the class "secondary" to the top div and see the top border disappear of all contents. Looks weird

This issue is more glaring in the case of the vertical variation which can be seen here: https://codesandbox.io/embed/semantic-ui-react-example-4b0us


Add black step to pre-commit

Add the managed black pre-commit step to pre-commit.
It is unclear to me if the new pre-commit GH action step plays well with black. The correct way to lint for black in CI is black --check .

The pre-commit step in github action looks good, it is just running black . and if it would modify a file it returns a nonzero exit code.


[IMPROVE] Error message display

Currently Keysign errors (swap or send) get displayed in a confusing and unpleasant way for the user.
Example:

Please please display each error in our error standard and simplyfy the error that the user understands.
Example above:

All known Keysign errors (swap or send) get displayed in a simple and understandable way for the user.


social login should also optionally get the refresh token and pass th…

…is token the adapter.parse_token, so that SocialToken instance also caches the refresh_token and refresh is possible for "offline" mode (e.g. tested with google api offline mode)

Coverage decreased (-0.2%) to 96.2% when pulling c47b641c628648038e31fe183245e965f7e9179f on fpghost:master into 479a40d2ccd69d69f7a05d716f30239dd3ef2343 on Tivix:master.

@fpghost thanks for your changes, it gave me a clue what I had to do to fix that on my side.
However, it looks like the misunderstanding lies in misusing of parse_token which can extract refresh_token and other things on its own - https://github.com/pennersr/django-allauth/blob/0.39.1/allauth/socialaccount/providers/oauth2/views.py#L55. So, my fix was to change one line only and everything worked out:
- social_token = adapter.parse_token({'access_token': access_token})
+ social_token = adapter.parse_token(token)

Note, my aim was to implement a Google connect view like these - https://django-rest-auth.readthedocs.io/en/latest/installation.html#additional-social-connect-views, so the fix might not work in other cases.

@potar Ah I see! Great, I'm glad it helped anyhow

@potar your fix is absolutely the right one. The parse_token method will extract both the refresh token and the expires_in time (if there is one). I was knocking my head out over why the I couldn't see that data in the database for a while until I did some debugging. Wish I had seen this pull request and your comment earlier.

Would love this to be merged sooner rather than later. If you wish to use refresh tokens for whatever reason, we're currently forced to fork or subclass the relevant serializer and copy a lot of the validate method logic.

@fpghost any chance of revising your PR with @potar's simple approach in mind and maybe adding a test or two? I suspect the project maintainers will expect some minimal testing.

@fpghost @potar I needed storage of refresh tokens too - here is my implementation, along with a couple of tests, that is a half-way point between both your approaches.
https://github.com/Tivix/django-rest-auth/pull/565
Cheers!

Hi,
as a user of rest-auth, thanks for the contribution!
This repo is not maintained anymore, so the development moved to dj-rest-auth. (reference: https://github.com/Tivix/django-rest-auth/issues/568)
It may be best, if you move this PR there. (and upgrade to using dj_rest_auth)
new repo link: https://github.com/jazzband/dj-rest-auth (I'm not the upkeeper of that repo, it just makes sense for me to help you merge your PR)
Many Thanks,
Barney


Implement Main page and Login page

#5 #6
Implement Main page and Login page

There is a problem with npm run build.
This deal with next Pㄲ. #11


Fix miscellaneous typos in docs and source

Fix miscellaneous typos in docs and source

d2i_PKC8PrivateKey -> d2i_PKCS8PrivateKey
bechmark -> benchmark
ciperhsuite -> ciphersuite
EncyptedPreMasterSecret -> EncryptedPreMasterSecret

Update: Originally it was my intention to backport the fixes as possible. However, since none of the recent typo fixes was backported, I won't bother to do it either.

@alex, if you find some more simple typos within the next days you can just post the misspelled word here on this thread. I have a script to automate finding and fixing them. Collecting the typos will save some work, in particular if they are to be merged in the stable branches. Of course you can just as well continue creating your own pull requests if you prefer.

Sure. I've also got a script that finds typos, but it also has a ton of false positives, so I usually only go through 1 file a day :-)

Rebased without changes to kick the CI bots.

ping

Merged, thanks!


Remove bootstrap and development widgets

Describe your changes
Remove the TableWidget and FormWidget, which seem to have been left over from early development.  As a side effect, this removes Bootstrap, which had been messing up the display of a number of parts of Jupyterlab.
Issue number
Closes #101, closes #102
Checklist before requesting a review

[X] Performed a self-review of my code
[X] Formatted my code with pkgmt format
[ ] Added tests (when necessary).
[ ] Added docstring documentation and update the changelog (when needed)

hi @rschroll, thanks for contributing to this? is this ready for review?

If you want to review, I'm happy to hear your thoughts.  But the connector widget is a bit ugly looking right now.  I thought I'd play with the CSS a bit to see if there are early wins.  But if you prefer to get this in quickly as a fix and then clean up the widget in another PR, that's reasonable too!

@rschroll oh, you're right! I completely forgot that our connector widget depends on boostrap. I'd appreciate it if you can add some CSS so it looks ok. I guess JupyterLab already provides some CSS that would make this easier? but up to you!

closing due to inactivity


fix: tls install script not running on first boot

Changes

[!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

#765
#764  👈
#763
main

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

Join @MasterPtato and the rest of your teammates on  Graphite


New package: FiloSottile.age version 1.1.1

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

I'm guessing that the package name age might be triggering the Policy 2.7 check.

@wingetbot waivers Add Policy-Test-2.7


Add support for decoding the JSON API media type (see https://jsonapi.org/format/).

Requires https://github.com/vapor/core/pull/202.

Ah, nice. Hadn't seen this one yet. HTTPMediaType is now a part of this package in Vapor 4. Core is deprecated in Vapor 4. See my related comment here: vapor/core#202 (review)

Thanks! Added the jsonAPI declaration to this PR.

Hey @MrMage, you just merged a pull request, have a coin!
You now have 20 coins.


Column mappings failed

Based on this docs: https://github.com/aaberg/sql2o/wiki/Column-mappings

Failed mapping when SELECT

Old query: SELECT created_date createdDate, user_id userId, phone FROM mob_user WHERE userId = :userIdParam
Exception: Database error: ORA-00904: "USERID": invalid identifier
But its success when query like this:
New query: SELECT created_date createdDate, user_id userId, phone FROM mob_user WHERE user_id = :userIdParam

Failed mapping when UPDATE

Old query: UPDATE mob_user SET cuid = :cuidParam WHERE user_id = :userIdParam
Exception: - (but field not updated)
But its success when query like this:
New query: UPDATE mob_user SET cuid = :cuidParam WHERE phone = :phoneParam
Can you help me what's wrong??? I see something is wrong with the column using underscore. I'm using sql2o version 1.6.
MAVEN

org.sql2o
sql2o
1.6.0

@LinggaAskaEdo could you extend your context , how your table looks like, how do you execute queries above, what database do you use


Dev Tools Error: Already in transaction

I see this error frequently in the dev tools:
https://github.com/microsoft/vscode/blob/e87da4b1388ee441663fdfa6441d445551aaa625/src/vs/base/browser/ui/list/rowCache.ts#L66

Maybe because of smooth scrolling?
I guess this error is in error telemetry to get the non-minified stack trace.

@hediet Any idea what tree/list this is for?

Unfortunately not. You don't see in your day to day usage of VS Code?

No I just checked the logs for a few long open VS Code window and don't see this error

I see "smooth" in the call stack, so I guess enabling smooth scrolling might cause this.


Knowledge Transfer

I implemented a knowledge transfer strategy as was discussed in #325 .
Currently the code can work with all acquisition functions and surrogates, and should work with new ones if added. I tried to make the changes to the code minimal.
I have a toy problem to test the code in tmp/example.py.. There, 5 optimization tasks which share some structure are solved. Every such task corresponds to parameter tuning of SVM on regression learning task. Some features are available to the optimizer that provide extra information about instance of optimization problem.
Here are the results, when following strategy outlined in #325 :

In the subplot title, "Task number" means that modified knowledge transfer optimizer was only provided with the index of similar task being solved. "Task features and number" means that optimizer was provided with both index of similar task and some features describing the task.
Clearly, initially the solver which utilizes prior knowledge experiences much less of a cold start problem. However, with more iterations the solver which starts from scratch (no prior knowledge) achieves better objective values.
My current hypothesis is that solver with more data (prior knowledge) becomes overconfident about objective values in unexplored regions, which prevents the solver from getting to the good values (quickly).
One strategy that I intend to test  to circumvent this is to take the best of two worlds - start with solver with prior knowledge, and after some number of iterations to switch to solver with no prior knowledge. I expect this to yield the minimum of two curves.
This might be a bug in my code of course, but it seems like it is not. I will also test this on instances of optimization with larger number of parameters to tune.
Consider this PR a more of an "experiment in progress" to keep you updated about this. As I will have more data I will post it here.

It seems that I have added changes to the older version of skopt. Will take a look on how to move this to the newer one.

@betatim sorry, I know you asked for the minimal pull request - but it seems that overall integrating this does not take much change, most of the code in the commit is actually code of the example problem. Plus it is always a possibility that there is something completely different that is better than the current approach, so I decided to close the loop.

Hmmm I will close this one for now, and as I will put the updates into newest version of skopt and as I will fix the convergence, I will create a new PR. I apologize if I am spamming you with this :)


[UESDK] Get access to FVector2D

Hello,
I'm working on a proof of concept to let enduser be able to adjust widgets.
For this i need to call functions Widget::SetRenderTranslation and Widget::SetRenderScale which expects an FVector2D as argument.
Can this be added to UESDK?

Due to UE using doubles in 5.X for some structs (including FVector2D) and floats in 4.X, it would have the same problem as FVector.
UE4SS has runtime support for all supported engine versions but C++ doesn't support different versions of structs with the same name at runtime so we have to choose whether our structs use doubles or floats at compile time which means that support for UE functions that take those as a param is a bit shaky.
Basically, we implement the largest struct we can (so for FVector that's 3xSize of double), and we convert floats to and from doubles when constructed and used.
One problem is that if you pass this FVector directly to a UE function, it would only work in 5.X since 4.X expects floats.
Another problem, which isn't solved as easily as the above problem by just having a wrapper and converting the struct at runtime to a version with a different name that uses floats, is the fact that we can't do this easily for containers.
We are unable to support containers (like TMap, TArray, or even TPair) with these structs at all unless the game just happens to be using a compatible engine version which obviously will never be guaranteed and we don't have a way right not to choose the engine version at compile time.
Since this is a fairly simple struct, I would actually recommend that you implement it yourself since we can't satisfy everyone without having some way of selecting which version of the struct you want at compile time.

Thanks for the awnser, and i'm very sorry, added this feature request to the wrong repository..... ;-(((
consider this request as never asked...
Best regards


Installation reported "The server name or address could not be resolved"

Version
Microsoft Windows [Version 10.0.22621.963]
WSL Version

[ ] WSL 2
[ ] WSL 1

Kernel Version
NULL
Distro Version
NULL
Other Software
NULL
Repro Steps

Expected Behavior
It should be installed automatically.
Actual Behavior

Diagnostic Logs
No response

It's solved by enabling system proxy in Clash for Windows, I don't know why...

Closing since the issue was solved

Clash can be downloaded from here: https://www.clashforwindows.net/clash-for-windows-download/

thank you @ktrNozomi @vighnesh153 , such a weird issue lol. any theories as to why this works?

Both of you saved my 1 or 2 days in life : @ktrNozomi @vighnesh153 - (worked for windows 11 pro 22H2)

Thank you so much @ktrNozomi and @vighnesh153. I wish I could hug you both right now.

I configure my DNS to <IP_ADDRESS> and <IP_ADDRESS> (you may need ipconfig /flushdns), then hostname can be resolved. My ISP is ChinaTelecom.

wsl is not getting installed after using clash for windows too

@ktrNozomi @vighnesh153 Thanks a lot! I couldn't find a solution anywhere before this.


excluding host overrides port

In config, if you don't enter a "host", it should default to localhost. However, you should still be able to supply other attributes like a port
In current implementation, if no host is provided, it overrides host AND port to localhost:4848
Current functionality:
RxQ.connectEngine({port: 9076}) -> "localhost:4848"

Preferred functionality:
RxQ.connectEngine({port:9076}) -> "localhost:9076"

fixed in 1.0.0


Import error when using example (python 3.3.0 - win 10)

Occurs when trying to import native_wifi from wifiutils. This is done by using the example provided.
Traceback (most recent call last):
  File "C:/Users/goose/PycharmProjects/wifiSwap/main.py", line 1, in <module>
    import pywifi
  File "C:\Python33\lib\site-packages\pywifi\__init__.py", line 3, in <module>
    from .pywifi import *
  File "C:\Python33\lib\site-packages\pywifi\pywifi.py", line 5, in <module>
    from .iface import Interface
  File "C:\Python33\lib\site-packages\pywifi\iface.py", line 7, in <module>
    from .win import wifiutils
  File "C:\Python33\lib\site-packages\pywifi\win\wifiutils.py", line 6, in <module>
    from . import native_wifi as wifiapi
ImportError: cannot import name native_wifi

Not sure if there is an issue with the package or my current setup (using latest version of pycharm). Doesn't seem to have a circular import.

Now pywifi moves to v1.0.0, please try it.

I tested under 3.5.2 env and it work fine. But testing under python3.3.5 env. I found the it failed:
`Python 3.3.5 (v3.3.5:62cf4e77f785, Mar  9 2014, 10:35:05) [MSC v.1600 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.

import pywifi
Traceback (most recent call last):
File "", line 1, in
File "C:\Python33\lib\site-packages\pywifi_init_.py", line 14, in
from .wifi import PyWiFi
File "C:\Python33\lib\site-packages\pywifi\wifi.py", line 15, in
from .iface import Interface
File "C:\Python33\lib\site-packages\pywifi\iface.py", line 11, in
from .win import wifiutil
File "C:\Python33\lib\site-packages\pywifi\win\wifiutil.py", line 10, in
from enum import Enum
ImportError: No module named 'enum'`

Maybe you can try it under python3.4+

Maybe install enum? (It is in pypi)

I'll try this later on today, Might be a good idea to show the interpreter requirement in the readme.

Yes, you are right. I found the 'from enum import Enum' can be removed because now pywifi doesn't need it anymore. After that, pywif can work properly in py3.3 env.
I will update both of the readme and code recently.

@thepizzagui  After changing the test env to py3.3.0, I saw the same issue as you reported. It seems there are some changes between 3.3.0 and 3.3.5. I think this issue will not be fixed recently because I want to replace the d-bus communication between pywifi and wpa_supplicant with sockets.

Upgrading seems to do the trick, looks like we'll never know!

import pywifi error!
my python env is 3.6.3

import pywifi
Traceback (most recent call last):
File "", line 1, in
File "/usr/local/lib/python3.6/site-packages/pywifi/init.py", line 15, in
from .wifi import PyWiFi
File "/usr/local/lib/python3.6/site-packages/pywifi/wifi.py", line 15, in
from .iface import Interface
File "/usr/local/lib/python3.6/site-packages/pywifi/iface.py", line 15, in
raise NotImplementedError
NotImplementedError

this version don't support pywifi?

What operating system?

mac OS

mac OS  10.13.1

If you look at the readme you will see that Mac is not currently supported.

On Dec 5, 2017, at 8:46 AM, edgedroper<EMAIL_ADDRESS>wrote:
mac OS 10.13.1
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

OK,Thank you wgaylord , i will use it at linux!

Your welcome

On Dec 5, 2017, at 8:55 AM, edgedroper<EMAIL_ADDRESS>wrote:
OK,Thank you wgaylord , i will use it at linux!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.


400 - Bad Request on array-some and array-reduce-right

Searching org.mvnpm:array-some or org.mvnpm:array-reduce-right under https://mvnpm.org/ returns 400 - Bad Request.
Same result using a simple maven project with a clean .m2/repository, settings.xml having https://repo.mvnpm.org/maven2 and pom.xml:
<?xml version="1.0" encoding="UTF-8"?>
<project>
    <modelVersion>4.0.0</modelVersion>

    <groupId>org.example</groupId>
    <artifactId>mvnpm</artifactId>
    <version>1.0-SNAPSHOT</version>

    <dependencies>
        <dependency>
            <groupId>org.mvnpm</groupId>
            <artifactId>mermaid</artifactId>
            <version>10.4.0</version>
            <scope>runtime</scope>
        </dependency>
    </dependencies>
</project>

.\mvnw clean package returns:
[INFO] Scanning for projects...
[INFO]
[INFO] -------------------------< org.example:mvnpm >--------------------------
[INFO] Building mvnpm 1.0-SNAPSHOT
[INFO]   from pom.xml
[INFO] --------------------------------[ jar ]---------------------------------
Downloading from mvnpm.org: https://repo.mvnpm.org/maven2/org/mvnpm/array-some/maven-metadata.xml
[WARNING] Could not transfer metadata org.mvnpm:array-some/maven-metadata.xml from/to mvnpm.org (https://repo.mvnpm.org/maven2): status code: 400, reason phrase: Bad Request (400)
Downloading from mvnpm.org: https://repo.mvnpm.org/maven2/org/mvnpm/array-reduce-right/maven-metadata.xml
[WARNING] Could not transfer metadata org.mvnpm:array-reduce-right/maven-metadata.xml from/to mvnpm.org (https://repo.mvnpm.org/maven2): status code: 400, reason phrase: Bad Request (400)
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  7.495 s
[INFO] Finished at: 2023-10-16T22:32:12+02:00
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal on project mvnpm: Could not resolve dependencies for project org.example:mvnpm:jar:1.0-SNAPSHOT: Failed to collect dependencies at o
rg.mvnpm:mermaid:jar:10.4.0 -> org.mvnpm:mdast-util-from-markdown:jar:1.3.0 -> org.mvnpm:micromark:jar:3.0.0 -> org.mvnpm:micromark-core-commonmark:jar:1.0.0 -> or
g.mvnpm:micromark-factory-label:jar:1.0.1 -> org.mvnpm:power-assert:jar:1.0.0 -> org.mvnpm:empower:jar:1.0.0 -> org.mvnpm:array-some:jar:[1.0.0,2): No versions available for org.mvnpm:array-some:jar:[1.0.0,2) within specified range -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/DependencyResolutionException

Having java -version
java version "17.0.5" 2022-10-18 LTS
Java(TM) SE Runtime Environment (build 17.0.5+9-LTS-191)
Java HotSpot(TM) 64-Bit Server VM (build 17.0.5+9-LTS-191, mixed mode, sharing

And .\mvnw -version
Apache Maven 3.9.3<PHONE_NUMBER>9f1ead511c958d89bd2f672198ae9f)
Maven home: XXXXXX\.m2\wrapper\dists\apache-maven-3.9.3-bin\326f10f4\apache-maven-3.9.3
Java version: 17.0.5, vendor: Oracle Corporation, runtime: C:\Program Files\Java\jdk-17.0.5
Default locale: fr_FR, platform encoding: Cp1252
OS name: "windows 11", version: "10.0", arch: "amd64", family: "windows

Blocks the use of Mermaid versions 10.4.0 and 10.5.0.

@dcotfr Thanks for logging this issue. It's now fixed. Please try again. There was an issue with fetching the versions and json marshaling, that has now been fixed


Create an intro

Add an intro about yourself: who are you, what is your strength, why are you here.

I create a new block intro with my info


infinite recursion crash with a subclass of dict

The code below reproduces a RecursionError: maximum recursion depth exceeded in __instancecheck__ crash.
import jsonpickle

class Container(dict):
    def __init__(self, *args, **kwargs):
        super().__init__(*args, **kwargs)
        self.__dict__ = self

def main():
    c = Container(a=17)
    jsonpickle.encode(c)

if __name__ == '__main__':
    main()

Since pickler.py already seems to have code to handle subclasses of dict, I'm assuming that the self.__dict__ = self in the code above is the culprit causing the issue.
Adding and obj != obj.__dict__ as shown below to pickler.py seems to fix the problem.  Is this something you'd consider fixing, or accepting a PR to fix?
"""
File: pickler.py
773:         # Sub-classes of dict
774:         if hasattr(obj, 'dict') and self.unpicklable and obj != obj.__dict__:
775:             dict_data = {}
776:             self._flatten_dict_obj(obj.dict, dict_data)
777:             data['dict'] = dict_data
"""

Yes, I'd be happy to merge a PR for this (or fix it myself!) I'll leave it up to you as to whether you want to submit the PR and have it merged, to show in your GitHub activity, or whether you want me to submit it (since this is a super quick PR).

I'm somewhat old and somewhat lazy!  If you can do it, great, cuz I'm not
super fluent with the whole PR process and fear making a mess of things!
Do you think my suggested fix is correct?
Jeff Robbins, CEO
LiveData, Inc. https://www.livedata.com/
(617) 571-5739 mobile
LinkedIn  https://www.linkedin.com/in/jeffsrobbins
Twitter https://twitter.com/jeffrobbinsceo
2 minute video https://youtu.be/l9CpI7XR8YU/?utm_source=s
On Mon, May 22, 2023 at 7:54 PM Theelx @.***> wrote:

Yes, I'd be happy to merge a PR for this (or fix it myself!) I'll leave it
up to you as to whether you want to submit the PR and have it merged, to
show in your GitHub activity, or whether you want me to submit it (since
this is a super quick PR).
—
Reply to this email directly, view it on GitHub
https://github.com/jsonpickle/jsonpickle/issues/443#issuecomment-1558204887,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AS43FGGTTPDV2VJYNYDJQQDXHP4DHANCNFSM6AAAAAAYI35FVQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Yep, your solution is correct!


Confilicting content in the list of services supported over the Microsoft peering

Azure DevOps Services Load Testing is listed both among the supported and the not supported services. According to this blog post (https://blogs.msdn.microsoft.com/devops/2018/10/23/expressroute-for-azure-devops/), it should be supported, along with the other DevOps services.

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

ID: 3cdb32c0-628d-0be2-f514-477d46dfa1b8
Version Independent ID: fa096059-2d16-30a2-1e8b-8b9671afcaa4
Content: Azure ExpressRoute FAQ
Content Source: articles/expressroute/expressroute-faqs.md
Service: expressroute
GitHub Login: @cherylmc
Microsoft Alias: cherylmc

@fguerri Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

@fguerri - checking with the PMs now.

Thanks Cheryl. It seems that the most up-to-date content about ER support for Azue devops is in this blog post: https://blogs.msdn.microsoft.com/devops/2018/10/23/expressroute-for-azure-devops/
Thanks,
Federico
From: Cheryl McGuire<EMAIL_ADDRESS>Sent: Monday, October 29, 2018 5:18 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Federico Guerrini<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Confilicting content in the list of services supported over the Microsoft peering (#17672)
@fguerrihttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ffguerri&data=02|01|Federico.Guerrini%40microsoft.com|f40675ab9c1e4490ec6308d63dba0ee5|72f988bf86f141af91ab2d7cd011db47|1|0|636764266659760600&sdata=70i7pIzpAVMlFld3P9aknAZVJT9wylWFlSiTv%2BH6oFU%3D&reserved=0 - checking with the PMs now.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F17672%23issuecomment-433972386&data=02|01|Federico.Guerrini%40microsoft.com|f40675ab9c1e4490ec6308d63dba0ee5|72f988bf86f141af91ab2d7cd011db47|1|0|636764266659770617&sdata=Sl%2BatHMuDLHS8T18Pb4FU6Inz5cRuVbHWIAasnE9aX8%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FASaoM53BmeI3_OZ-gVVda3mLwUuJKeTLks5upyongaJpZM4X7uwl&data=02|01|Federico.Guerrini%40microsoft.com|f40675ab9c1e4490ec6308d63dba0ee5|72f988bf86f141af91ab2d7cd011db47|1|0|636764266659770617&sdata=e7eAIWLYg87U7Z9sbwo2RYdLWs5FQn381vnoDJatVfQ%3D&reserved=0.

Fixed. Thanks, @fguerri! #please-close

@fguerri We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Nodemailer #330

Have you read the Contributing Guidelines ?
Yes
Description
This is nodemailer by which you can mail a form having message to anyone, when i was beginner , i suffered alot to learn to this .So, i am sharing this to open source.
Fixes #330

Checklist

[ y] I've read the contribution guidelines.
[y ] I've checked the issue list
[ y] I've edited the README.md and link to my code.

No issues present in this work.
screenshots of the output

@mayank-del Please check the issue and files changed.They are for ping pong game

Very sorry for that, I will fix it today
On Wed, 9 Mar 2022, 20:15 photon149, @.***> wrote:

@mayank-del https://github.com/mayank-del Please check the issue and
files changed.They are for ping pong game
—
Reply to this email directly, view it on GitHub
https://github.com/Kushal997-das/Project-Guidance/pull/333#issuecomment-1062994737,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATTNVGRFNYYZQ5WMMHIWCSDU7C2PZANCNFSM5QBSAXFA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

@photon149  Please check  it once, I have sent nodemailer now.

sir see I added Screenshots  of output in description at the top and added Readme.md in the recent commit as well

@mayank-del Add the output SS in your README.md file as well.

@photon149 , please check it , I have copied the screenshot to readme.md as well

Great . Approved from my end.
@Kushal997-das @prakhar3062 have a look.

@prakhar3062 sir, i think nodemailer is at correct position , i have to change directory of pingpong game, can u suggest any different directory

@Kushal997-das  ,Sir i fixed everything , now please accept the pull request

@Kushal997-das  sir you told to to reduce  screenshot size only , i did it,. now, please merge it.

@Kushal997-das  , please check , screenshot size reduced

@Kushal997-das  sir , kindly check it and inform me, please

@Kushal997-das sir , kindly check it and inform me, please

@mayank-del reduce the image size fromreadme.md file don't you understand ?

@Kushal997-das , I changed the size of img in both readme.md  as well as above in comment, please check and merge .

now check @photon149  sir, i decrease dimension to 50%

@photon149 , @Kushal997-das  please it

@photon149 Have a look !

Looks fine to me.
@Kushal997-das It's good to merge .

@photon149  yes yes ,it is good to merge


Precompiling plotting crashes when headless workers are available

I have julia running on my local machine and I use Distributed to connect to a headless remote server. When I try to plot something (locally), this fails because it can't install GLMakie on the headless server.
I neither can nor want to install X11 and graphics drivers on the server but also I don't need GLMakie running on the worker instances, because I don't need to plot anything there.
Example:
using GLMakie
using Distributed

addprocs(<...>) # This should connect to the headless server

scatter(randn(10), randn(10))

This fails with the error message
Errors encountered while load FileIO.File{FileIO.DataFormat{:PNG}, String}("/home/Uroc327/.julia/packages/Makie/lgPZh/assets/icons/icon-128.png").
All errors:
===========================================
On worker 2:
ArgumentError: Package PNGFiles [f57f5aa1-a3ce-4bc8-8ab9-96f992907883] is required but does not seem to be installed:
 - Run `Pkg.instantiate()` to install all recorded dependencies.

Stacktrace:
 [1] _require
   @ ./loading.jl:1012
 [2] require
   @ ./loading.jl:936
 [3] #1
   @ /buildworker/worker/package_linux64/build/usr/share/julia/stdlib/v1.6/Distributed/src/Distributed.jl:79
 [4] #103
   @ /buildworker/worker/package_linux64/build/usr/share/julia/stdlib/v1.6/Distributed/src/process_messages.jl:274
 [5] run_work_thunk
   @ /buildworker/worker/package_linux64/build/usr/share/julia/stdlib/v1.6/Distributed/src/process_messages.jl:63
 [6] run_work_thunk
   @ /buildworker/worker/package_linux64/build/usr/share/julia/stdlib/v1.6/Distributed/src/process_messages.jl:72
 [7] #96
   @ ./task.jl:411
===========================================
ArgumentError: Package ImageMagick [6218d12a-5da1-5696-b52f-db25d2ecc6d1] is required but does not seem to be installed:
 - Run `Pkg.instantiate()` to install all recorded dependencies.

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

Fatal error:
Error showing value of type Makie.FigureAxisPlot:
ERROR: On worker 2:
ArgumentError: Package PNGFiles [f57f5aa1-a3ce-4bc8-8ab9-96f992907883] is required but does not seem to be installed:
 - Run `Pkg.instantiate()` to install all recorded dependencies.

I'm not 100% sure how to solve this with Distributed.jl, but it's definitely not a Makie issue so I'm going to close this!

What is it an issue of, then?
] instantiate was, of course, executed previously to that. So was ] precompile.

Well, somehow the project that the process uses doesn't contain PNGFiles. I don't know Distributed well enough to speculate on the cause for this, but one thing to know is, that FileIO requires PNGFiles indirectly. So you may need to add PNGFiles explicitly somewhere.
If that doesn't give you an idea about how to fix the issue, I'd suggest asking on discourse or slack.

I see, thanks.


CLI functionality

Treat depth-anything-tensorrt like a command line interface with multiple different flags and arguments. This also allows for the processing of entire folders of both videos and images.
Refer to readme to find changes.

win32 & tensorrt 10 support added

Hi, I got the following two errors when running your fork:
1). main.cpp(99,19): error C2512: 'DepthAnything': no appropriate default constructor available
2). main.cpp(141,21): error C2039: 'init': is not a member of 'DepthAnything'

Sorry, small oversight. Should be fixed now! @spacewalk01

thank you for your contribution!


chore: cleanup coveralls/simplecov setup

The coveralls_reborn mentions that the gem isn't needed for Actions.
Followed setup from https://github.com/coverallsapp/github-action/issues/29#issuecomment-704366557

This doesn't fix the JRuby failures, but it get's rid of some unrelated errors like https://github.com/omniauth/omniauth/actions/runs/5125273149/jobs/9218237165?pr=1119#step:4:210 when it tries to load some of that stuff at the end of the runs


Feature request: display data labels on line/area charts for extremums only

If you have lot of data (say 1000+ points) its good to see labels only for maximums and/or minimums.
See attached picture (not the best realization, just to illustrate the idea).

Also it would be nice to have more "intelligent" data labels which will be placed not just by text align and offset options but positioned depending on other labels and graphs on the same chart to avoid overlaying each over.

@Dimanoid Good suggestions.
Regarding overlapping data labels, I have already tried not to overlap dataLabels when there is a lot of data as shown below.

Regarding intelligent dataLabels, I can provide an option in dataLabels config to only show max/min values and hide all. But, I think - annotations (point annotations to be specific) would be better suited in these scenarios. An example of this
chart.render().then(() => {
    const lowest = chart.getLowestValueInSeries(0)
    const highest = chart.getHighestValueInSeries(0)

    chart.addPointAnnotation({
        x: new Date(chart.w.globals.seriesX[0][chart.w.globals.series[0].indexOf(lowest)]).getTime(),
        y: lowest,
        label: {
            text: 'Lowest: ' + lowest,
            offsetY: 2
        },
    })

    chart.addPointAnnotation({
        x: new Date(chart.w.globals.seriesX[0][chart.w.globals.series[0].indexOf(highest)]).getTime(),
        y: highest,
        label: {
            text: 'Highest: ' + highest,
            offsetY: 2
        },
    })
});

About maximum/minimum - not exactly what I mean. Not the global maximum or minimum, but local ones. May be I found a better illustration.

As for overlapping of data labels, please look at this example https://stackblitz.com/edit/angular6-apexcharts-14dvla

Thank you for better illustration, I got your point.
And yes, overlapping looks bad in multi-series chart with lots of data-points. Need to find a more intelligent way to manage data-labels for such scenarios.

addPointAnnotation doesn't seem to work for me. More specifically, chart.w.globals.seriesX is an empty array for some reason. I do see seriesXvalues though.
https://hotinfluencer.com/user/7b87c59b71ca5a2c2e0124d9663c1e6914

Can you please create a codepen demo in a new issue? Your provided link doesn't contain any chart.

hmm super weird. The chart is there but I will try to recreate a sample on codepen. Thanks for the swift reply.

@Dimanoid Good suggestions.
Regarding overlapping data labels, I have already tried not to overlap dataLabels when there is a lot of data as shown below.
Regarding intelligent dataLabels, I can provide an option in dataLabels config to only show max/min values and hide all. But, I think - annotations (point annotations to be specific) would be better suited in these scenarios. An example of this
chart.render().then(() => {
    const lowest = chart.getLowestValueInSeries(0)
    const highest = chart.getHighestValueInSeries(0)

    chart.addPointAnnotation({
        x: new Date(chart.w.globals.seriesX[0][chart.w.globals.series[0].indexOf(lowest)]).getTime(),
        y: lowest,
        label: {
            text: 'Lowest: ' + lowest,
            offsetY: 2
        },
    })

    chart.addPointAnnotation({
        x: new Date(chart.w.globals.seriesX[0][chart.w.globals.series[0].indexOf(highest)]).getTime(),
        y: highest,
        label: {
            text: 'Highest: ' + highest,
            offsetY: 2
        },
    })
});

@junedchhipa can you give an example how to achieve what you showed in your first screenshot?


Database query API should return one snapshot per document id

Method private getForAlias

moved to jira https://aerobase.atlassian.net/browse/ARB-6


need clean up measurement opaque data provisioning

Current code requires provision measurement .
    size_t opaque_measurement_rsp_size;
    uint8_t *opaque_measurement_rsp;

We had better allow user to provide an input based upon need.

Related issue : https://github.com/DMTF/libspdm/issues/1393

fixed in https://github.com/DMTF/libspdm/pull/1991


Add support to run init scripts during container initialization

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
linuxserver.io images support running custom init scripts located in /custom-cont-init.d/ folder. These scripts run as root and it is very useful to perform any runtime initialization during container startup. For example, I use it to modify container route table to route traffic via another container (VPN).
Currently, I don't see a feature like this in recyclarr, so the only workaround is to override ENTRYPOINT in custom Dockerfile, do the init in my own entrypoint, then delegate back to the original entrypoint via shell script. However, recyclarr expects to run as user 1000, so this is not a straightforward change. Additionally, after I change the ENTRYPOINT, I now have to check every single recyclarr image update and verify whether the original entrypoint has been changed in the update or not, and update my startup shell script accordingly to invoke the correct original entrypoint.

The official recyclarr image is rootless by design. 1000:1000 is the default UID:GID but can be changed with the user: property in compose.
I will not be adding this functionality because it requires significant overhaul of the way the image is set up and requires root which is no longer recommended for docker containers for security reasons. There's nothing about Recyclarr itself that requires root privileges. Thus, making the container itself requires root is unnecessarily prohibitive, especially in environments where containers are running on nodes that should or may not have root privileges.
I recommend you work on your own linuxserver/recyclarr image if you want this functionality. Or, if possible, find an approach that allows you to properly network traffic in the way you want without requiring modification to application containers.


Improve README for new users

These changes provide improvements to "onboarding" for new
users who might not be familiar with Go or don't have
any zeek logs sitting around.  This is based on some simple
user testing observations of a new user trying out zq.

I also added a small version of conn.log from the zq-sample-data repo to the top-level directory in the repo so people can run the zq examples out of the box.


MasterDetailRelation with ID of another table

Hi!
I have a structure of tables for Customers, People and People's Addresses. I've already mapped in Serenity, but I need register in a one single form, as below:

Everything is working, but the problem is in the addresses, because [MasterDetailRelation("IDDetail")] writes the Customers IDRow (table master), and I wanted it to bind to the People IDRow.
Could someone guide me how to do this in the Serenity?

Maybe MasterKeyField of MasterDetailRelation is what you need.

Thanks, @hannesb !
I upgraded my version of Serenity to <IP_ADDRESS> and used the MasterKeyField attribute, created by you at issue #3272, and was resolved.


#315 missing parsers msgsubmitproposal

Add missing parsers for MsgSubmitProposal for CSV output formats of koinly, cointracker and taxbit.
Solves #315

reopening, why does this not happen on accointing?
{"level":"error","error":"no parser for message type '/cosmos.gov.v1beta1.MsgDeposit'","time":"2023-03-31T19:20:21Z","message":"Error processing taxable transaction."}              │
{"level":"error","time":"2023-03-31T19:20:21Z","message":"Error processing taxable transaction."}                                                                                {"level":"error","time":"2023-03-31T19:20:21Z","message":"Error getting rows for addresses: [osmo1083svrca4t350mphfv9x45wq9asrs60cq5yv9n]"}


virito-blk event handler

write event handler to connect fds and memory regions to virito-blk

done since a9baad7661bf2a8045afdedc98a375e985595ec6


added a python program file

python program that can find total number of path between 2 coordinates if a person can only move rightwards and downwards

Kindly please merge this pull request in you code


add preferredPeakBitrateProp to video component

@tnaughts fyi, this marks the first feature add to a newly forked react-native-video library hosted on our side


onCanvasClick call a component

Hi,
I would like to ask. It is written that onCanvasClick I can open a window to create a new event. Is it possible somehow
onCanvasClick open a  new component to create an event?
Thank you!

@Ilaiwi

@mnob2019 yes you can do that differently.
you need to use onCanvasClick(groupId, time, e) of course. On click you would open something like Modal from material ui and ask the user via a form to fill some info and create a new item by adding it to the items passed to Timeline

@llaiwi thank you for the response,I will look through that form:) new to React and doing quite a big front-end project. You have an amazing library here, than  you for the work!

youre welcome. It would be awesome if you could share a codesanbox if you got things working here.
let me know if you have any other questions but I will close this for now

I will as soon as something works!:)) Thank you!

@Ilaiwi  not sure how I can reopen the issue to continue, but I am trying to open a component form the Timeline and it returns me the following error:
https://codesandbox.io/s/wandering-sound-fkg3r
The function works I tried it, rendering a separate component, it also works inside, the Timeline  with console.log and alert, but when I pass the Form component it gets undefined. The logic is pretty the same as in material UI. I was just wondering if it can be something in the library predefined that I cannot understand?  Or I am doing something completely wrong?

@mnob2019  there are couple of issues with the code iteself not the library
fixed them in this fork
https://codesandbox.io/s/brave-wescoff-i56o9

@Ilaiwi  Oi, that is amazing! thank you!! Ye....I usually assume myself being stupid, actually))))
where can I read more about this ? <></> Its like an empty component?
In the view it looks a bit behind the bars, imprisoned))
https://codesandbox.io/s/jolly-leftpad-mdhnc


SST fails because it attempts to delete log groups in an environment that does not allow deleting logs.

Hello,
It seems SST ion fails to function in my environment as it attempts to delete log groups on startup.
Is this behavior configurable? If not, will this ever be on the roadmap?
I have already attempted to set the retention policy in sst.config.ts to retain and retain-all with no success.
This is a hard requirement for my team.
I believe log preservation is likely a requirement for other people working in corporate environments and therefore think it would be a valuable addition to the project if it is not yet there.
With some guidance, I could likely look into submitting a pull request.
Thank you,
Pascal V.
Here is a snippet of logs with the relevant error message:
DataLakeSubscriberUbxscrNotification aws:s3:BucketNotification
|  Error       MyWebServer sst:aws:Function → MyWebServerLogGroup aws:cloudwatch:LogGroup
|  Logs: DeleteLogGroup, https response error StatusCode: 400, RequestID: 2818f445-2820-4650-89
a6c45d5fa341, api error AccessDeniedException: User: arn:aws:sts::006288227511:assumed-
<EMAIL_ADDRESS>is not
authorized to perform: logs:DeleteLogGroup on resource: arn:aws:logs:ca-central-
1:006288227511:log-group:/aws/lambda/app-ion-pv-MyWebServerFunction:log-stream: with an explici
deny in a service control policy
|  Error       AppServer sst:aws:Function → AppServerLogGroup aws:cloudwatch:LogGroup
|  Logs: DeleteLogGroup, https response error StatusCode: 400, RequestID: 13b91ced-da7c-45ce-8f
16a8f2b9172e, api error AccessDeniedException: User: arn:aws:sts::006288227511:assumed-
<EMAIL_ADDRESS>is not
authorized to perform: logs:DeleteLogGroup on resource: arn:aws:logs:ca-central-
1:006288227511:log-group:/aws/lambda/app-ion-pv-AppServerFunction:log-stream: with an explicit
in a service control policy

×  Failed
   MyWebServer sst:aws:Function → MyWebServerLogGroup aws:cloudwatch:LogGroup
   Logs: DeleteLogGroup, https response error StatusCode: 400, RequestID: 2818f445-2820-4650-89
a6c45d5fa341, api error AccessDeniedException: User: arn:aws:sts::006288227511:assumed-
<EMAIL_ADDRESS>is not
authorized to perform: logs:DeleteLogGroup on resource: arn:aws:logs:ca-central-
1:006288227511:log-group:/aws/lambda/app-ion-pv-MyWebServerFunction:log-stream: with an explicit
deny in a service control policy
   AppServer sst:aws:Function → AppServerLogGroup aws:cloudwatch:LogGroup
   Logs: DeleteLogGroup, https response error StatusCode: 400, RequestID: 13b91ced-da7c-45ce-8f
16a8f2b9172e, api error AccessDeniedException: User: arn:aws:sts::006288227511:assumed-
<EMAIL_ADDRESS>is not
authorized to perform: logs:DeleteLogGroup on resource: arn:aws:logs:ca-central-
1:006288227511:log-group:/aws/lambda/app-ion-pv-AppServerFunction:log-stream: with an explicit
in a service control policy

Interesting. Can you give me more context? What's in your sst.config.ts? Did you get these errors when you were trying to remove or change some Functions?

Yes,
I currently have a project running correctly on sst v2 in the same environment. I am trying to migrate it to sst v3.
I have created a project based on the default SvelteKit demo as a sanity check. I had gotten it running once, but it no longer works now that it attempts to delete logs when I restart it since that initial success. I will use that config as reference to reduce the variables for debugging.
I get these errors when trying to change the config. I believe I would likely get them when trying to remove it too, but I have not tested that.
Here is the sst.config:
/// <reference path="./.sst/platform/config.d.ts" />

export default $config({
  app(input) {
    return {
      name: "my-app-pascal",
      removal: input?.stage === "production" ? "retain" : "remove",
      home: "aws",
    };
  },
  async run() {
    const infra = await import("./infra");
    const OIDC_CLIENT_SECRET = new sst.Secret("OIDCCLIENTSECRET");
    new sst.aws.SvelteKit("MyWeb",
      {
      path: "packages/web-app/",
      link: [infra.userTable, infra.dataLake],
      environment: {
			VITE_OIDC_CLIENT_ID: process.env.OIDC_CLIENT_ID,
			VITE_COGNITO_USERPOOL_ID: process.env.COGNITO_USERPOOL_ID,
			VITE_OIDC_CUSTOM_DOMAIN : process.env.OIDC_CUSTOM_DOMAIN,
			VITE_GEOCORE_API_DOMAIN : process.env.GEOCORE_API_DOMAIN,
			VITE_FEATURE_SIGN_IN: "false"
    }
  }
    );
  },
});

That is great, thank you :)

@PVautour to set all log groups in your app to retain on delete, add this to the top of run():
async run() {
  $transform(aws.cloudwatch.LogGroup, (args, opts) => {
    opts.retainOnDelete = true;
  });

  ...
}

You can read more about $transform here - https://sst.dev/docs/components#transform-1
Feel free to reopen if this doesn't work for you.


Add metric for known clients.

Signed-off-by: Ben Kochie<EMAIL_ADDRESS>

@brian-brazil Slightly hacky, but it seemed the least complicated way to do this. I was thinking about doing this via Const, but it felt like a lot of overhead to make it that way.

Tests are failing, looks like a vendoring issue.

Fixed, also bumped the client_golang up to 1.0.0.


nf-test with custom plugin gets killed halfway when asserting results

I'm using a custom plugin with nf-test right now, to capture md5sum from complex data (Nifti-gz, it require a dedicated reader to get consistent md5 from the data inside the file).
I was wondeing why my tests tend to get killed halfway. Note I am working in a container and docker enforces resource limits, but I hadn't configured anything that could potentially go over those. Then I discovered the infamous line 54 in the nf-test executable, where there is a -Xmx10g, and I don't have that much RAM. I changed the value to a lower one - 6g in my case - and it works !
Now it's quite a pain to go and change that line everytime the executable changes. Would it be possible to add an option to set that value ? Or default to a standard java way of setting it like the JAVA_TOOL_OPTIONS or something ?

Thanks. This issue has been fixed in PR #264 and will be available in the next update.
btw: are you interested in listing your nf-test plugin (https://github.com/AlexVCaron/nft-nifti) on the plugins page at https://plugins.nf-test.com/?


[Bug]: The game will start even if we set rows,columns and player to 1

Give a brief about the bug ✍
The game will start even if the the values of rows,columns and player is set to 1.
What is the expected behavior? 🤔
min player should be 2
min box size should be 5*5
Provide step by step information reproduce the bug 📄
set rows,columns and player to 1

Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

pls assign me these issue under GSSOC'24

please assign me this issue @Durgesh4993

i would like to resolve this issue

@yash21sriv @Pratik2401 @Yashc885 Please raise any other issue, because someone is already working on this issue.
If you have any doubt you can directly send message on our discord (https://discord.gg/rZb46cCMmK), we have created seprate chennals for all projects...
Thank you :)


Added functionality to return promotion via status

Ability to fetch promotions using status
95% coverage
10.0 Pylint score

Codecov Report

Merging #52 (12ce12e) into main (718a895) will increase coverage by 0.06%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #52      +/-   ##
==========================================
+ Coverage   96.04%   96.10%   +0.06%
==========================================
  Files           7        7
  Lines         253      257       +4
==========================================
+ Hits          243      247       +4
  Misses         10       10

Impacted Files
Coverage Δ

service/models.py
100.00% <100.00%> (ø)

service/routes.py
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Update to 5.2.0 - stripe branch

Rebased off stripe/confidant:master branch to keep all the changes on the top of the list.

@elewis-stripe it would be nice to merge this PR first and address the comments in the follow-up PR.


docs linkcheck skip

🚀 Pull Request
Description
This PR skips a URL that is causing a redirects limit traceback for the documentation linkcheck CI task

Consult Iris pull request check list

Go @lbdreyer go! :rocket:


feat(agent): Execute jobs returned by the server even if the telemetry update request failed server-side

Require: nuvla/api-server#731

Done but differently.
Now we have heartbeat operation which also return the list of jobs.
heartbeat operation doesn't require any input data so no spec error can happen.


Array throws an error when declared on multiple lines

Consider the following twig template:
{% for foo in ['foo','bar'] %}
    {{ foo }}
{% endfor %}

Render perfectly.
Now this one:
{% for foo in [
    'foo',
    'bar
'] %}
    {{ foo }}
{% endfor %}

Throws TypeError: Cannot read property 'length' of undefined.
This second template is perfectly valid (https://twigfiddle.com/y6kt19).

@dave-irvine, yes, definitely. Thanks.


Fix tests configuration

Various minor fixes

Add pytest to requirements-test.txt file

Fix tests_cli : if grip package is not installed, use python grip/__main__.py to run cli tests

Regenerate HTML content for tests

Thanks for the PR, @Methacrylon!
Happy to get the pytest dependency in there (and credit you for it). If you remove the conditional (and the output files since master now contains the latest from GitHub), I'm happy to merge this in.

I went ahead and pulled in your pytest change. Closing this, but still crediting you for fixing the requirements-test.txt file. Thanks again 😃


[Feature Request] Allow "jsconfig.json" within ".vscode" directory

The .vscode directory contains config for the IDE.
The jsconfig.json file must be within the project root, which is already crowded and messy.
Please allow us to place the file within the .vscode directory so that all config is in the same place.

Isn't it typescript feature that looks jsconfig.json in the root?

For a particular project I'm working on, it's just plain JS (not TS) and it looks in the root. It's a mess in there. The config should be in .vscode, to keep it manageable.

It's still typescript language server that parses jsconfig.json/tsconfig.json.

Closing this as out of scope.
These files are not VS Code specific. Other editors can also jsconfig and tsconfig to improve their intellisense, and tsconfig.json files in particular are used by command line tooling


Stabilize execution of test.sh on dev Linux environments: certifier_tests_wrap.cc:178:11: fatal error: Python.h: No such file or directory

The newly added test.sh script is not working correctly on engineers' Linux dev env.
There are some hard-coded dependencies in various build scripts that need to be corrected.
There are at least a couple of diff issues to resolve under this issue:

Dir-path for PY_INCLUDE = -I /usr/include/python3.10/ needs to be fixed in all make files.
Support passing down LOCAL_LIB=/usr/local/lib64 from test.sh to sub-scripts / makefiles.

Here are some examples of failures:
483 + make -f certifier_tests.mak --always-make -j8 sharedlib LOCAL_LIB=/usr/local/lib64
484 protoc --cpp_out=. --proto_path ../certifier_service/certprotos ../certifier_service/certprotos/certif    ier.proto
485
486 compiling support.cc
[...]

550 compiling x509_tests.cc
551 g++ -I ../include<EMAIL_ADDRESS>-I ./sev-snp -I ./gramine -g -std=c++17 -D X64     -Wall -Wno-unused-variable -Wno-deprecated-declarations -O3 -fpic -o ./x509_tests.o -c x509_tests.cc
552 compiling certifier_tests_wrap.cc
553 g++ -I ../include<EMAIL_ADDRESS>-I ./sev-snp -I ./gramine -g -std=c++17 -D X64     -Wall -Wno-unused-variable -Wno-deprecated-declarations -O3 -fpic -I /usr/include/python3.10/ -o ./ce    rtifier_tests_wrap.o -c certifier_tests_wrap.cc
554 certifier_tests_wrap.cc:178:11: fatal error: Python.h: No such file or directory
555   178 | # include <Python.h>
556       |           ^~~~~~~~~~
557 compilation terminated.
558 make: *** [certifier_tests.mak:195: certifier_tests_wrap.o] Error 1
559 make: *** Waiting for unfinished jobs....
560 ++ cleanup
561 ++ set +x
562
563 test.sh: **** Error **** Failed command, make -f certifier_tests.mak --always-make -j${NumMakeThreads}     sharedlib LOCAL_LIB=/usr/local/lib64, at line 130 ./CI/scripts/test.sh while executing function test-core-certifier-programs

I tried running test.sh on my Linux dev machine and I hit the following issues:

compilation failures due to hardcoded Python include path (submitted PR #216)
missing dependencies like pytest and protobuf
incompatible openssl options; I have openssl 1.1.1 and it doesn't support noenc; this one is minor, not sure if we should care for openssl v1.x at all
waiting 120sec. for sockets to be closed; we should be able to fix this with SO_REUSEADDR?
test script deleting non-git files; this is not ok -- it's common practice to keep all kind of files in the project dir like virtual environments, IDE configurations, etc. and deleting those is quite unexpected and frustrating
using sudo during tests; I understand we need this to emulate SEV but I won't give root permissions to a test script running on my machine

I guess the best way to go here is to provide a docker container with all of the dependencies that can be used for running tests. The Dockerfile can be used as a reference for what packages and configurations are needed if you want to run this on your physical machine.

The Docker container is a good approach. Let's discuss this next week to put this work item in its priority queue.

@rgerganov -- I investigated one of the points you raised above:
waiting 120sec. for sockets to be closed; we should be able to fix this with SO_REUSEADDR?
The sleep is coming in the pytest-case test_certifier_framework.py in-between channel.init_server_ssl() and cfm.server_dispatch().
In server_dispatch() -> open_server_socket(), we are using this re-use-address semantic, as seen below:
2283       // Reuse addresses and ports
2284 #define REUSE_SOCKETS_AND_PORTS
2285 #ifdef REUSE_SOCKETS_AND_PORTS
2286     int reuse = 1;
2287     if (setsockopt(sfd,
2288                    SOL_SOCKET,
2289                    SO_REUSEADDR,      <<<<<
2290                    (const char *)&reuse,
2291                    sizeof(reuse))
2292         < 0) {
2293       fprintf(stderr, "Can't reuse socket %s\n", __func__);
2294       return false;
2295     }

Empirically, testing evidence shows that even though this flag has been used, on dev-machine, if you re-run the test-case repeatedly, the socket is not being closed and re-used fast enough. Hence, this sleep has been added.
This test file test_certifier_framework.py has several test cases each of which establishes a SSL connect, does some work and then disconnects. So, when run back-to-back in CI, I need a mechanism to ensure that the socket re-use property kicks-in "fast enough".
I know this is irritating, but I don't think this is functionally broken or use of this sleep is masking some broken functionality.
I can investigate further, but debugging this is a bit difficult on CI machines.

@yelvmw -- Here's the trace evidence showing why some of the Python tests in test_certifier_framework.py are hanging when run on the SGX machine.
I think it's because in the shared library, the code for ENABLE_SEV=1 is automatically enabled for all hardware platforms. (I had raised this 'issue' in our previous visits to certifier.mak. At that time, it was felt that it's better to have all code #include'ed by default for all TEEs.)
That I suspect is causing an issue as seen in the execution flow trace instrumentation:
test_certifier_framework.py::test_run_app_as_a_client_init_client_ssl_with_trust_manager
*** cc_trust_manager_get_certified() ...

init_policy_key() ... Starting ...
PublicKeyFromCert() Starting ...
PublicKeyFromCert() ... Starting ...
GetX509FromCert() succeeded.
X509_get_pubkey() succeeded.
X509_NAME_get_text_by_NID() succeeded, len=15.
X509_NAME_get_text_by_NID() succeeded.
PublicKeyFromCert():425 succeeded.
PublicKeyFromCert():432 succeeded.
PublicKeyFromCert():435 Executing ...
get_tcb_version_from_vcek():270 Executing ...
vcek_ext_byte_value():230 Executing ...

Basically, the Python method invokes init_policy_key(), which goes through till:
get_tcb_version_from_vcek() -> vcek_ext_byte_value().
I instrumented the last function, and it's hanging on L232 below:
 230   printf("%s():%d Executing ...\n", __func__, __LINE__);
 231   // Use OID for both lname and sname so OBJ_create does not fail
 232   nid = OBJ_create(oid, oid, oid);
 233   printf("%s():%d Executing ...\n", __func__, __LINE__);

OBJ_create() seems to be a SEV_SNP interface of some sort. I can't locate its sources.

To validate this theory that running SEV_SNP-specific code in this simulated-enclave, I conditionalized the definition of ENABLE_SEV in certifier.mak and re-built the shared lib:
  4 ifndef NO_ENABLE_SEV
  5     ENABLE_SEV=1
  6 endif

This allows the tests to run cleanly without running into this hanging issue.

It's surprising to see OBJ_create hanging. It's an OpenSSL function. I checked the bugs section of the manual and only see the caveat of concurrent invocation. Is this the case?

@yelvmw - About the concurrent use of OBJ_create(), the server & client code are being executed concurrently, although there isn't any sort of race condition.
However in the affected test case(s) [e.g., test_run_app_as_a_client_init_client_ssl_with_trust_manager()], we just start the client-app portion on its own, talking to the Certifier Service. So, no concurrent invocation of said SSL interface should come into play.

@rgerganov : I'm back to cleaning up some items you logged here: "test script deleting non-git files; this is not ok -"C
Firstly, I do need rm_non_git_files or some hook like that, to clean-up cruft between run-to-run of this run_example.sh.
Otherwise, I've found that some workflow steps for some sample-apps may interfere causing instability. I just need a way to "cleanup the entire dev env" at the start of each run of this script.
Second, the command does git ls from certifier-root dir. Why would a user's config / env-files be living inside the certifier-framework-for-confidential-computing/ sub-dir? Wouldn't we expect these to usually live under $HOME?
Are you worried about IDE config files for this Certifier Framework to be living inside this projects's root sub-dir?
What if we exclude dot-files; i.e. filter-out \.* files from the rm? Would that suffice to address your concern?
Alternate Solution: Let me know if you think this is good enough:
I took a look at the files that do need to be cleaned-up by this command. Seems like if we can filter out the output from git ls-files to only delete the files in app_data*/ and provisioning sub-dirs, and pick-up build output files (e.g. *.o, *.exe, *wrap* etc.) that may be sufficient.
This kind of filtering is prone to being incomplete ... but it may be the only reliable way to do this clean-up w/o impacting the user's config / env-files.
Let me know your thoughts on this approach ... and whether we are really to be worried-about config / env-files in the certifier root-dir.


Promise based api

redux-persist just switched to requiring promise based storage in v5.0.0-beta.13
Unfortunately it is going to be a pain to get everyone updated. Technically the storage can support both (like AsyncStorage and localforage) but since this module is specific to redux-persist may was well just go 100% promises and make the peer dep be redux-persist@5

Great! I'm currently very busy, but I always accept PRs. It should not be too hard. Otherwise, I'll have a look this weekend.

cool, since the underlying implementation here is sync, im thinking this can be as easy as adding some return Promise.resolve(value and return Promise.reject(err) calls. I suppose the open question is, should the lib support both promise and callback (thus backwards compat with older redux-persist versions) or should it move forward promise only and enforce via peer deps?
Ill probably take a longer look this weekend.

Whats missing for publishing a new version of the npm?


Feature/token based authentication

resolved #23

Token based authentication.
Ban & unban user.

@bellaabdelouahab
@Khalid1G
please review the changes. I have pushed my RSA keys so no one have to generate keys.

okay, I can implement ban user functionality other ways where just four lines of code will be enough.
That approach will be easy.Should i close this pr😁?
@Khalid1G
@bellaabdelouahab

I propose implementing a standardized requirement for all contributors to provide a detailed description, diagrams, or any other relevant documentation pertaining to the features they intend to add, prior to commencing the coding process. This approach will greatly facilitate our understanding and subsequent implementation of the proposed features.
@muttaqin1

I propose implementing a standardized requirement for all contributors to provide a detailed description, diagrams, or any other relevant documentation pertaining to the features they intend to add, prior to commencing the coding process. This approach will greatly facilitate our understanding and subsequent implementation of the proposed features.
@muttaqin1

yeah sure.

I propose implementing a standardized requirement for all contributors to provide a detailed description, diagrams, or any other relevant documentation pertaining to the features they intend to add, prior to commencing the coding process. This approach will greatly facilitate our understanding and subsequent implementation of the proposed features.
@muttaqin1

No, I don't Agree Again. Placing so many barriers in the way of contributors' information would just slow down their efforts. If the topic has already been discussed enough in issue , you can ask for an explanation before the PR is made if necessary.

@muttaqin1
@Khalid1G

Actually this approach for authentication is very common in production.you can find many videos in YouTube.
Here is a repo for reference: https://github.com/janishar/nodejs-backend-architecture-typescript
But i believe this approach is very tuff to understand for others.This was very confusing for me when is was implementing this in my project🤕.so it was my bad decision to implement this headache 😁
we may need this in future but for simplicity its better to avoid this kind of confusing implementation now.
And i am sorry for my unprofessional behavior .I should have explained this clearly.But i am still learning .This is my first time working in a team.
btw,The auth Controller looks scary 😁

after rereading your pr, i found that you store the private key too generated token on the database, that's a bad idea, it is like putting you password on the database, what you can do is search who we store the generated jwt for each login request, this allows us to get the logs of a user, what do you think.
@muttaqin1

the whole process is:
generate access token & refresh token with private key.private key should be private.
we can decode the token using the public key.And we can keep this public key public if we need.
now can we track the logged in user ?
we generate 2 random string called access token key & refresh token key.
now we put this access & refresh token key in the access & refresh token .
and also save this access & refresh token key in the database.
so this creates a connection with access & refresh token to database.
this access & refresh token keys are not any kind of secret key.
@bellaabdelouahab

after rereading your pr, i found that you store the private key too generated token on the database, that's a bad idea, it is like putting you password on the database, what you can do is search who we store the generated jwt for each login request, this allows us to get the logs of a user, what do you think. @muttaqin1

yeah this approach is perfect

if sounds good, i will create a issue to discuses it more.
@muttaqin1

the whole process is: generate access token & refresh token with private key.private key should be private. we can decode the token using the public key.And we can keep this public key public if we need.
now can we track the logged in user ?
we generate 2 random string called access token key & refresh token key.
now we put this access & refresh token key in the access & refresh token . and also save this access & refresh token key in the database. so this creates a connection with access & refresh token to database.
this access & refresh token keys are not any kind of secret key. @bellaabdelouahab

You should understand how JWT work:
JSON Web Tokens (JWTs) are a popular method for securely transmitting information between parties as a compact, URL-safe string. They consist of three parts: a header, a payload, and a signature. The header and payload are Base64Url-encoded JSON objects, while the signature is used to verify the integrity of the token.
JWTs are often used for authentication and authorization purposes. When a user logs in to an application, the server generates a JWT containing relevant user information (payload) and signs it using a private key. The server then sends the JWT to the client, who stores it, typically in local storage or a cookie.
Whenever the client makes subsequent requests to the server, it includes the JWT in the request headers. The server can then verify the JWT's authenticity by recalculating the signature using the public key associated with the private key used to sign the token. If the signature matches and the token hasn't expired, the server can trust the information contained within the token and process the request accordingly.

Regarding the storage of private keys, it is crucial to understand that private keys should be kept confidential and secure. Storing the private key in a database is generally not recommended because it introduces potential vulnerabilities. As you rightly mentioned, if someone gains unauthorized access to the database, they could retrieve the private key and generate valid tokens for any user.

To maintain the security of private keys, best practices suggest storing them in a separate secure location, such as a dedicated key management system or hardware security module (HSM). These systems are designed to protect sensitive cryptographic material and restrict access to authorized personnel only.
Instead of storing the private key directly in the database, the server should keep it in a secure environment and use it to sign JWTs on the fly during the authentication process. By following this approach, even if an attacker gains access to the database, they won't be able to generate valid tokens without the private key.

if sounds good, i will create a issue to discuses it more.
@muttaqin1

sure go ahead.


Adding a Password Strength Calculator

Description
It will suggest whether a password is strong weak medium based on runtime of user input with different emojis use too describing strength to weakness
Screenshots
No response
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

Assign it to me for GSSoc'24 sir

Kindly assign this issue to me in GSSoC'24 sir.

I like to work on this issue. Kindly assign it to me in GSSOC'24

Pls open project related relevant issues @cxde-Sujal


Grayscale image segmentation

Great job, can I input a grayscale map into the SAM model instead of a three-channel color image？

The following works for now:
image = np.expand_dims(image, axis=2)
image = np.repeat(image, 3, axis=2)

With this, I assume there is no benefit in performance to running grayscale?

我猜@dvdrplus的意思是说加一个灰度图的模块

I guess @ dvdrplus means adding a grayscale module
似乎SAM的数据集里面没包括医学影像
It seems that the SAM dataset does not include medical imaging
但是,我试了一下,对于医学影像SAM做的也不错👍🏻
But I gave it a try and did a good job with medical imaging SAM 👍🏻
所以, 有必要做个这样的模块专门处理像xRay, CT, MRI等这样的灰度图
So, it is necessary to create a module specifically for processing grayscale images such as xRay, CT, MRI, etc


optimize live view playback

Live view working well it seems for v3.5.13.

Live view is not working for me. Here are the logs (with Debug enabled).
[11/1/2020, 19:11:39] [Blink] handleStreamRequest
[11/1/2020, 19:11:39] [Blink] {
sessionID: '2a5c4fda-ab61-48df-ab67-3370fe18fb55',
type: 'start',
video: {
profile: 2,
level: 2,
packetizationMode: 0,
cvoId: undefined,
width: 1280,
height: 720,
fps: 30,
pt: 99,
ssrc:<PHONE_NUMBER>,
max_bit_rate: 299,
rtcp_interval: 0.5,
mtu: 1378
},
audio: {
codec: 'OPUS',
channel: 1,
bit_rate: 0,
sample_rate: 24,
packet_time: 20,
pt: 110,
ssrc:<PHONE_NUMBER>,
max_bit_rate: 24,
rtcp_interval: 5,
comfort_pt: 13,
comfortNoiseEnabled: false
}
}
[11/1/2020, 19:11:39] [Blink] FFMPEG: received first frame
client connected from <IP_ADDRESS>:43046
conencting to: <IP_ADDRESS>
(node:18872) [DEP0123] DeprecationWarning: Setting the TLS ServerName to an IP address is not permitted by RFC 6066. This will be ignored in a future version.
Error: self signed certificate
at TLSSocket.onConnectSecure (_tls_wrap.js:1501:34)
at TLSSocket.emit (events.js:315:20)
at TLSSocket._finishInit (_tls_wrap.js:936:8)
at TLSWrap.ssl.onhandshakedone (_tls_wrap.js:710:12) {
code: 'DEPTH_ZERO_SELF_SIGNED_CERT'
}
[11/1/2020, 19:11:39] [Blink] [Video] ffmpeg exited with code: 1 and signal: null (error)

It looks like you have a proxy or something in the middle? It is getting a
self signed certificate. Also looks like the sni servername is being
tampered with by the api responses.
On Sun, Nov 1, 2020 at 7:12 PM rcoletti116<EMAIL_ADDRESS>wrote:

Live view is not working for me. Here are the logs (with Debug enabled).
[11/1/2020, 19:11:39] [Blink] handleStreamRequest
[11/1/2020, 19:11:39] [Blink] {
sessionID: '2a5c4fda-ab61-48df-ab67-3370fe18fb55',
type: 'start',
video: {
profile: 2,
level: 2,
packetizationMode: 0,
cvoId: undefined,
width: 1280,
height: 720,
fps: 30,
pt: 99,
ssrc:<PHONE_NUMBER>,
max_bit_rate: 299,
rtcp_interval: 0.5,
mtu: 1378
},
audio: {
codec: 'OPUS',
channel: 1,
bit_rate: 0,
sample_rate: 24,
packet_time: 20,
pt: 110,
ssrc:<PHONE_NUMBER>,
max_bit_rate: 24,
rtcp_interval: 5,
comfort_pt: 13,
comfortNoiseEnabled: false
}
}
[11/1/2020, 19:11:39] [Blink] FFMPEG: received first frame
client connected from <IP_ADDRESS>:43046
conencting to: <IP_ADDRESS>
(node:18872) [DEP0123] DeprecationWarning: Setting the TLS ServerName to
an IP address is not permitted by RFC 6066. This will be ignored in a
future version.
Error: self signed certificate
at TLSSocket.onConnectSecure (_tls_wrap.js:1501:34)
at TLSSocket.emit (events.js:315:20)
at TLSSocket._finishInit (_tls_wrap.js:936:8)
at TLSWrap.ssl.onhandshakedone (_tls_wrap.js:710:12) {
code: 'DEPTH_ZERO_SELF_SIGNED_CERT'
}
[11/1/2020, 19:11:39] [Blink] [Video] ffmpeg exited with code: 1 and
signal: null (error)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/colinbendell/homebridge-blink-for-home/issues/11#issuecomment-720175540,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMMERK4V66IYJDJAWN6ONDSNX2QTANCNFSM4TGXYGHQ
.

Super strange. No proxy here. It’s all just on my network with a standard blink set up.

So, I think this has to do with the Official Raspberry Pi image. It has Nginx and a self-signed cert. I didn't realize before. Now, I don't have the SSL in use but may need to figure out how to disable it.

Based on only a few minutes of testing, I'm unable to disable the use of the proxy on the official pi image. You can disable it, but it turns itself right back on. I'm assuming your instance has no cert at all. Is that right?

Got it, just pulled the logs and sent to you on Discord.

I have the exact problem. I have a Blink Mini in use.

Live view doesnt seem to be working for me with the XT2 cameras.  I get this when trying to bring up live view (logs)
/2020, 9:54:10 PM] [Blink] [Video] Failed to start video stream: spawn ffmpeg ENOENT
listening on <IP_ADDRESS>:34275
[11/4/2020, 9:54:15 PM] [Blink] [Video] Failed to start video stream: spawn ffmpeg ENOENT
listening on <IP_ADDRESS>:36433
[11/4/2020, 9:54:22 PM] [Blink] [Video] Failed to start video stream: spawn ffmpeg ENOENT
[11/4/2020, 9:54:50 PM] [Blink] Blink Dewberry Lane Armed (Target) for Blink Dewberry Lane is: 2
[11/4/2020, 9:54:52 PM] [Blink] Blink Dewberry Lane Armed (Target) for Blink Dewberry Lane is: 3
[11/4/2020, 9:55:07 PM] [Blink] Enabled for Blink Patio Right is: true
[11/4/2020, 9:55:07 PM] [Blink] Enabled for Blink Patio Right is: true
listening on <IP_ADDRESS>:45509
[11/4/2020, 9:55:14 PM] [Blink] [Video] Failed to start video stream: spawn ffmpeg ENOENT
[11/4/2020, 9:55:22 PM] [Blink] [Video] Failed to start video stream: spawn ffmpeg ENOENT
[11/4/2020, 9:55:23 PM] [Blink] Temperature for Blink Driveway is: 25
[11/4/2020, 9:55:29 PM] [Blink] [Video] Failed to start video stream: spawn ffmpeg ENOENT
[11/4/2020, 9:55:34 PM] [Blink] [Video] Failed to start video stream: spawn ffmpeg ENOENT
listening on <IP_ADDRESS>:36023
[11/4/2020, 9:56:42 PM] [Blink] [Video] Failed to start video stream: spawn ffmpeg ENOENT
[11/4/2020, 9:56:59 PM] [Blink] [Video] Failed to start video stream: spawn ffmpeg ENOENT
[11/4/2020, 9:59:06 PM] [Blink] Temperature for Blink Patio Right is: 27.8

Getting the exact same issues as @swadhwani51 on XT2's

Same issue here with stock Raspberry Pi 4 Homebrige image and new Blink Mini cam.
Extract from Homebridge protocol when trying to open live view in Home App:

listening on <IP_ADDRESS>:46555
client connected from <IP_ADDRESS>:45358
conencting to: <IP_ADDRESS>
Error: self signed certificate
at TLSSocket.onConnectSecure (_tls_wrap.js:1501:34)
at TLSSocket.emit (events.js:315:20)
at TLSSocket._finishInit (_tls_wrap.js:936:8)
at TLSWrap.ssl.onhandshakedone (_tls_wrap.js:710:12) {
code: 'DEPTH_ZERO_SELF_SIGNED_CERT'
}
[11/5/2020, 20:10:09] [Blink] [Video] ffmpeg exited with code: 1 and signal: null (error)

Ok. I just ordered the newer camera hardware to experiment. I suspect that the protocol is different and I just need to debug it

Same issue here!

Ok. I just ordered the newer camera hardware to experiment. I suspect that the protocol is different and I just need to debug it

Will you answer here when you have the Live view resolved?
Thanks Colinbendell, you are amazing! :)


docs(guides): change localhost to <IP_ADDRESS> to avoid error

Description
Using localhost on linux in the SemaphoreEthers constructor causes an error.
Changing it to <IP_ADDRESS> would make it work everywhere.
Related Issue
https://github.com/semaphore-protocol/semaphore/issues/469

Hey @agazso! Thank you so much!