[Snyk] Security upgrade setuptools from 68.0.0 to 70.0.0

This PR was automatically created by Snyk using the credentials of a real user.![snyk-top-banner](https://github.com/andygongea/OWASP-Benchmark/assets/818805/c518c423-16fe-447e-b67f-ad5a49b5d123)
Snyk has created this PR to fix 1 vulnerabilities in the pip dependencies of this project.
Snyk changed the following file(s):

cmd/earlystopping/medianstop/v1beta1/requirements.txt

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.
Some vulnerabilities couldn't be fully fixed and so Snyk will still find them when the project is tested again. This may be because the vulnerability existed within more than one direct dependency, but not all of the affected dependencies could be upgraded.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Improper Control of Generation of Code ('Code Injection')
[//]: # 'snyk:metadata:{"customTemplate":{"variablesUsed":[],"fieldsUsed":[]},"dependencies":[{"name":"setuptools","from":"68.0.0","to":"70.0.0"}],"env":"prod","issuesToFix":[{"exploit_maturity":"Proof of Concept","id":"SNYK-PYTHON-SETUPTOOLS-7448482","priority_score":768,"priority_score_factors":[{"type":"exploit","label":"Proof of Concept","score":107},{"type":"freshness","label":true,"score":71},{"type":"fixability","label":true,"score":214},{"type":"cvssScore","label":"7.5","score":375},{"type":"scoreVersion","label":"v1","score":1}],"severity":"high","title":"Improper Control of Generation of Code ('Code Injection')"},{"exploit_maturity":"Proof of Concept","id":"SNYK-PYTHON-SETUPTOOLS-7448482","priority_score":768,"priority_score_factors":[{"type":"exploit","label":"Proof of Concept","score":107},{"type":"freshness","label":true,"score":71},{"type":"fixability","label":true,"score":214},{"type":"cvssScore","label":"7.5","score":375},{"type":"scoreVersion","label":"v1","score":1}],"severity":"high","title":"Improper Control of Generation of Code ('Code Injection')"}],"prId":"906f64fe-1085-4a16-8bac-f8c78095916a","prPublicId":"906f64fe-1085-4a16-8bac-f8c78095916a","packageManager":"pip","priorityScoreList":[768],"projectPublicId":"07cb5b9a-b8e5-4966-b765-6f2a4f6dfc57","projectUrl":"https://app.snyk.io/org/ryanhammerdick/project/07cb5b9a-b8e5-4966-b765-6f2a4f6dfc57?utm_source=github&utm_medium=referral&page=fix-pr","prType":"fix","templateFieldSources":{"branchName":"default","commitMessage":"default","description":"default","title":"default"},"templateVariants":["updated-fix-title","priorityScore"],"type":"auto","upgrade":[],"vulns":["SNYK-PYTHON-SETUPTOOLS-7448482"],"patch":[],"isBreakingChange":false,"remediationStrategy":"vuln"}'

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 72.998%

Totals

Change from base Build<PHONE_NUMBER>:
0.1%

Covered Lines:
3117

Relevant Lines:
4270

💛  - Coveralls


ModuleNotFoundError: No module named 'translator'

Hi,
when running mopidy, it keeps giving me an error, that the module translator is not found. Can someone maybe help?
I also looked into the install pip3 and the file translator.py is there.
I am trying to run mopidy on a raspberry pi with raspian.
Thank you in advance.

See #15


update requirements file

In requirements.txt, updated the opendp-smartnoise package version from <IP_ADDRESS> to <IP_ADDRESS>
tested the analysis, attack, and reconstruction notebooks with the updated package

Glad to have checks for notebooks!


pi_check in qasm and truncate floats

Summary
fixes #4272
Does what the title says.

Details and comments

can you add a test that the qasm produced by this is parsable.

good to go after it passes

Hold on doing sqrt(2) as well

@nonhermitian  that was just as a suggestion in my approval. It could be another pr as there might be a few irrational numbers we want to use (don't do them all but quantum does use a few)

Ok, I hold off for another day.


The Winckelmann-Reimer Gambit

In the French Winawer, lichess calls 4.a3 the "Winckelmann-Riemer Gambit". Not only was the guy's name Reimer, the gambit Thomas Winckelmann developed actually begins with the continuation  4.a3 Bxc3+ 5.bxc3 dxe4 6.f3! At least that's what his two books on the gambit are about, and it's what you'll find in books on the French Defense / Winawer.
I'm hoping this can be fixed, I'm not sure how to do a 'pull request', apologies.

Thanks for reporting! Fixed.
See https://docs.github.com/en/repositories/working-with-files/managing-files/editing-files if you want to learn how to make edits and pull requests yourself.


How to generate the random gaussian noise reported in the paper

Hello, can you point us the code snippet you used for generating the additive Gaussian noise? We used the following python code:
noise = np.random.normal(0, 5, pose2d.shape)
But when we directly add this noise to pose2d, the 3D pose estimation error will increase dramatically. Can you show us how you generated the noise?
Thanks

Did you add it before normalization?
On Mon, Sep 18, 2017 at 9:59 PM CHUNYUWANG<EMAIL_ADDRESS>wrote:

Hello, can you point us the code snippet you used for generating the
additive Gaussian noise? We used the following python code:
noise = np.random.normal(0, 5, pose2d.shape)
But when we directly add this noise to pose2d, the 3D pose estimation
error will increase dramatically. Can you show us how you generated the
noise?
Thanks
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/una-dinosauria/3d-pose-baseline/issues/25, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADj5zCye5tgW6q-otLs0SX_JEHZu-zoqks5sj0pBgaJpZM4Pb25f
.

--
Julieta

Yes, we add it before normalization. By the way, in the training stage, do you add same amount of noises to the training data?

in the training stage, do you add same amount of noises to the training data?

Uhm... I think so! Sorry, it's been a long time since we ran those experiments, but I'd expect that in normalization the mean will remain about the same, but stdev will be larger. Adding noise to the training set should take care of that.


A005132  is returning a 500 error

get_oeis_name_and_values/A005132
is currently returning a 500 on the production server

It is happening because of the unicode accent character in the name of the Recamán Sequence, but it doesn't happen when I run the server on my machine, so I am not yet clear how to fix it. Will keep investigating, but if anyone has any ideas please let me know.

OK, the exception occurs in views.py, line 443:
            seq.name = search_response['results'][0]['name']

The right hand side is a string with non-ascii unicode characters in it, and the left side is a field of the object-relational model for the database entry for A005132. So my guess is that somehow on the production machine, the database is set up to only take ASCII strings, whereas we most decidedly are not limited in such a way, and that the database on my machine is OK with unicode strings.
I will investigate that possiblity in more depth tomorrow.


Feature Request: Copying physics polygons

Not sure if it really fits this project since it's not about bitmasks, but it's still on tilesets and definitely speeds up the workflow.
Would it be possible to add a way to template physics layers on tiles as well? There's no real way to copy paste those either.

These could be copy pasted from the .tres files, but that's a bit tedious.
Another workaround is to just copy the same tileset and change the texture, but if you're gonna do that, you might as well just not use this plugin since it copies the bitmasks, too. Just thought this might be a nice QoL feature to go along with the bitmask stuff, and I could try helping add it if you want (don't know GDScript that well yet, though).

Absolutely agree with needing this feature! I actually got halfway through setting up a copy-paste interface for all kinds of tiledata (including collisions), but while the interface could be shared, it ended up requiring so much refactoring of the code base that I thought it might be better as a separate plugin. It’s on my list to take a look at again


Check, Please v0.1

Check, Please product brief: https://docs.google.com/document/d/1s6oWvbgccVm53-vgl83I0iyBB9jbEKayqeMEWjQUISc/edit?usp=sharing

[x] #2
[ ] #3
[ ] #4
[ ] #5
[ ] #6

Pausing implementation work pending feedback on product PRFAQ.


Uncaught RangeError when attempting to type in "Add Server" dialog

I confirm (by marking "x" in the [ ] below: [x]):

[X] This is not a troubleshooting question. Troubleshooting questions go here: http://www.mattermost.org/troubleshoot/.
[X] This doesn't reproduce on web browsers (such as in Chrome). If it does, issue reports go to the Mattermost Server repository.
[X] I have read contributing guidelines.

Summary
Uncaught RangeError when attempting to type in "Add Server" dialog
Steps to reproduce
Mac v10.12.6; Mattermost pre-release server; Mattermost Desktop App v4.1.0-RC1

Click the "+" icon
Try to type in the "name" or "url" field

Expected behavior
No issues or errors while typing.
Observed behavior
App freezes with attached errors in the developer console.

Duplicate of https://github.com/mattermost/desktop/issues/780


Storage: local user PermissionScope add enums for service

ARM API Information (Control Plane)
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Azure 1st Party Service can try out the Shift Left experience to initiate API design review from ADO code repo.  If you are interested, may request engineering support by filling in with the form https://aka.ms/ShiftLeftSupportForm.
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here.

Contribution checklist (MS Employees Only):

[ ] I commit to follow the Breaking Change Policy of "no breaking changes"
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[ ] Check this box if any of the following apply to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki.

[ ] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.

Ping @JasonYang-MSFT

@magodo
Please get the approve for the potential breaking change.
You can contact the PR assignee for more details.
@ArthurMa1978
Do you have any comments for the potential SDK breaking change from the swagger change in the PR?

Ping @JasonYang-MSFT

Ping @jianyexi

/azp run

@magodo , can you fix ModelValidation check errors and PrettierCheck errors? In addition, please follow above breaking change review actions to get the approval.

@magodo any chance we could get this one reopened/fixed up so that we can get this merged/usable in the provider?

@raych1 I've updated the PR to pass the Prettier CI. Regarding the breaking change, I saw the service team has approved it: https://github.com/Azure/azure-rest-api-specs/pull/21845#issuecomment-1418421342. Meanwhile, AFAIK the breaking change review is managed by the internal review board, should I email Jefferey, or?

Hi @magodo, one or multiple breaking change(s) is detected in your PR. Please check out the breaking change(s), and provide business justification in the PR comment and @ PR assignee why you must have these change(s), and how external customer impact can be mitigated. Please ensure to follow breaking change policy to request breaking change review and approval before proceeding swagger PR review. Action: To initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki. If you want to know the production traffic statistic, please see ARM Traffic statistic. If you think it is false positive breaking change, please provide the reasons in the PR comment, report to Swagger Tooling Team via https://aka.ms/swaggerfeedback. Note: To avoid breaking change, you can refer to Shift Left Solution for detecting breaking change in early phase at your service code repository.

@magodo , please follow above action to get breaking change review approval.

@mikekistler @JeffreyRichter , can you review the breaking change?

@magodo , let me know if this PR is ready to merge.

@raych1 I believe so as far as the service team confirms the API behaviors and adhere to not apply breaking changes..

this js sdk breaking is caused by this pr

@magodo have the service team replied yet?

@MaximeKjaer Would you please take a look at this PR, whether this aligns with the SRP API?

Kindly ping @MaximeKjaer

Kindly ping @MaximeKjaer

ping @MaximeKjaer

Apologies for the delay on this. This looks fine to me, but I'm not the owner of this swagger definition. I'll forward this thread to the relevant SRP owners for a review

ping @MaximeKjaer

ping @MaximeKjaer

@rkmanda any chance you could take a look/ping the Service Team?

@magodo I have removed the Approved-BreakingChange label because it is obsolete. Your PR will have to be re-approved. Please follow the instructions at https://aka.ms/brch. If you provide the reason why your PR was approved in the first place, this will speed up the process and help the Breaking Change Review Board apply appropriate new label. Apologies for the inconvenience!


added getDailyWeatherForecast()

I needed the daily forecast for the next three days, which was not possible with getWeatherForecast(). Added getDailyWeatherForecast() to get always faily forecast, even if $days is smaller than six.

Hey @cklm, thank you for your contribution!
I will look at it tomorrow when I get some spare time.
In the meantime, don't worry too much about the failing tests.

Looks good to me. Thank you for your contribution @cklm! :100:


Ubuntu build failures

OpenGothic CI: https://ci.appveyor.com/project/Try/opengothic/builds/45436817/job/vljogctckhdu2sh0
https://github.com/lmichaelis/phoenix/blob/529137d516615d1a2e62fe380e5690f8cf078a7c/source/world/bsp_tree.cc#L60
[ 41%] Building CXX object lib/phoenix/CMakeFiles/phoenix.dir/source/world/bsp_tree.cc.o
/home/appveyor/projects/opengothic/lib/phoenix/source/world/bsp_tree.cc:60:3: error: extra ‘;’ [-Werror=pedantic]
   60 |  };
      |   ^
cc1plus: all warnings being treated as errors
make[3]: ***

I really gotta set up a pipeline with this compiler version, huh?

from Appveyor logs:
-- The C compiler identification is GNU 9.4.0
-- The CXX compiler identification is GNU 9.4.0

Unfortunately not yet fixed:
https://ci.appveyor.com/project/Try/opengothic/builds/45461180/job/x35as8pfmmvbo2og
In file included from /home/appveyor/projects/opengothic/lib/phoenix/source/ext/daedalus_classes.cc:3:
/home/appveyor/projects/opengothic/lib/phoenix/include/phoenix/ext/daedalus_classes.hh:135:16: error: extra ‘;’ [-Werror=pedantic]
  135 |  FLAG(npc_flag);
      |                ^
/home/appveyor/projects/opengothic/lib/phoenix/include/phoenix/ext/daedalus_classes.hh:262:18: error: extra ‘;’ [-Werror=pedantic]
  262 |  FLAG(item_flags);
      |                  ^
/home/appveyor/projects/opengothic/lib/phoenix/include/phoenix/ext/daedalus_classes.hh:1147:20: error: extra ‘;’ [-Werror=pedantic]
 1147 |  FLAG(c_menu_flags);
      |                    ^
/home/appveyor/projects/opengothic/lib/phoenix/include/phoenix/ext/daedalus_classes.hh:1200:25: error: extra ‘;’ [-Werror=pedantic]
 1200 |  FLAG(c_menu_item_flags);
      |                         ^
cc1plus: all warnings being treated as errors
make[3]: *** [lib/phoenix/CMakeFile

Okay, I've finally set up GCC 9 in my pipeline starting at<PHONE_NUMBER>dadf5bbd681340d6a4dff9b60d8cd1. No build errors for me.

Yep, fixed.
Thanks!


Update entrypoint.sh - Add --host <IP_ADDRESS> to Uvicorn execution

Fixes deployment error:
"The TargetPort 8000 is bound to an inaccessible loopback IP address, bind to a non-loopback IP address instead."

Thank you! I lost that in the port from gunicorn to uvicorn-only.


You Sync, You Win

If we are copying to a FileOutputStream, call getFD().sync().

Addressed in v0.2.0.


Feature request: how much memory video will take

Can you make the feature to see, how much memory will take video after cropping?
For example: I take a big video for 6 hours, I know, it will take about 12 Gb of memory, then i will cut this video from start on 0 and end on 1 hour, how much exactly memory will it take?

Not possible, see #244


After upgrade to 2.0.0, SASS build is not working

I'm having the same problem. After updating Ionic CLI to v2, can't seem to build my sass (scss) files.

Same here.
Gulp Sass triggered manually does the trick.
Something to do with the "watch" part ?

Solution in #1250 worked for me

@AaronMorse thanks for that. Worked for me...

This works for me but I don't get the live reload on ionic emulate ios just have to add the watch task in gulpfile. Added this pull request - https://github.com/driftyco/ionic-cli/pull/1299

Can confirm this. It works with<EMAIL_ADDRESS>This should be working without a workaround.

What happen with this? How is possible to launch a main version like this?

Thanks for opening the issue. It seems that issue has been resolved. I am going to close this issue but please feel free to reopen another issue if this occurs again.

Is still happening, I had to manually add this line to my gulpfile.js
gulp.task('serve:before', ['default','watch']);


Unable to delete the archive after extracting using extractor.ExtractToDirectory

I am trying to replace the usage of System.IO.Compression.ZipFile.ExtractToDirectory() with the Microsoft.CST.RecursiveExtractor nuget package
I am using this code to extract a zip file, then trying to delete it, but the zip file is still in use.
  var extractor = new Extractor();   var extractorOptions = new ExtractorOptions();   extractor.ExtractToDirectory(outputDirectory, sourceArchiveFileName, extractorOptions);
It seems that Extractor does not implement IDisposable and there is no way to close the archive. Am I missing anything necessary to close the archive after extracting it?
Attached is an example console app reproducing the issue
NetFrameworkConsoleApp.zip

Try using the public ExtractionStatusCode ExtractToDirectory(string outputDirectory, string filename, Stream stream, ExtractorOptions? opts = null, bool printNames = false) API instead so you can control the lifecycle of the Stream. If you use the public ExtractionStatusCode ExtractToDirectory(string outputDirectory, string filename, ExtractorOptions? opts = null, bool printNames = false) method under the covers it will call FileEntry.FromFileName(filename) which creates a Stream owned by the FileEntry object. The Stream is then disposed by the FileEntry finalizer but it is non-deterministic when the finalizer runs.
I modified your example slightly to use the alternate API and it seems to resolve the issue:
        private static void ExtractToDirectory(string sourceArchiveFileName, string outputDirectory)
        {
            var extractor = new Extractor();
            var extractorOptions = new ExtractorOptions();
            var fileStream = File.OpenRead(sourceArchiveFileName);
            extractor.ExtractToDirectory(outputDirectory, sourceArchiveFileName, fileStream, extractorOptions);
            fileStream.Dispose();
        }

It would probably be better to switch the filename API to instead open the stream and then dispose it after extraction rather than relying on the FileEntry finalizer.

Thank you, using the FileStream overload is a good workaround for now, and I see you opened a PR that should fix the original method in the future. Thanks for the quick help!


webexteamssdk.exceptions.ApiError: [404] Not Found - Unable to get message. [Tracking ID: ROUTER_62713947-AFC2-01BB-1903-0AFE1B641903]

From last, i am unable to use the webexteamssdk. Can someone help me with this
10:16:44 DEBUG    urllib3.connectionpool    https://webexapis.com:443 "GET /v1/messages/a8d66170-cbbb-11ec-9700-c51bbd345fr2 HTTP/1.1" 404 141
10:16:44 ERROR    asyncio                   Future exception was never retrieved
future: <Future finished exception=<ApiError [404] Not Found>>
Traceback (most recent call last):
File "/opt/rh/rh-python36/root/usr/lib64/python3.6/concurrent/futures/thread.py", line 56, in run
result = self.fn(*self.args, **self.kwargs)
File "/home/fayaz/python_project/python_project_venv/errbot-root/plugins/err-backend-cisco-webex-teams/CiscoWebexTeams.py", line 440, in process_websocket
spark_message = self.webex_teams_api.messages.get(activity['id'])
File "/home/fayaz/python_project/python_project_venv/lib64/python3.6/site-packages/webexteamssdk/api/messages.py", line 321, in get
json_data = self._session.get(API_ENDPOINT + '/' + messageId)
File "/home/fayaz/python_project/python_project_venv/lib64/python3.6/site-packages/webexteamssdk/restsession.py", line 380, in get
response = self.request("GET", url, erc, params=params, **kwargs)
File "/home/fayaz/python_project/python_project_venv/lib64/python3.6/site-packages/webexteamssdk/restsession.py", line 345, in request
check_response_code(response, erc)
File "/home/fayaz/python_project/python_project_venv/lib64/python3.6/site-packages/webexteamssdk/utils.py", line 222, in check_response_code
raise ApiError(response)
webexteamssdk.exceptions.ApiError: [404] Not Found - Unable to get message. [Tracking ID: ROUTER_62713947-AFC2-01BB-1903-0AFE1B641903]

Hi @fayazsk25
I am also facing the same issue while communicating with the bot using WebSocket.  I got the fix, where converting the message-id from UUID to base64 format
I referred to the below link and applied the solution. please check this out
https://github.com/fbradyirl/webex_bot/blob/7df3778fc114f842f4ce6f28e04197a79e2b4347/webex_bot/websockets/webex_websocket_client.py#L78

Closing issue; webexteamssdk, today, does not support websockets.


Feature/fix issue 318

Fixes #318

Coverage remained the same at 100.0% when pulling 47f43b07de96458473d31c6bf59c3071f8952b34 on feature/fix-issue-318 into e6e85e4e4332fba6d01273b3719c1c11abc644e0 on develop.

Coverage remained the same at 100.0% when pulling 47f43b07de96458473d31c6bf59c3071f8952b34 on feature/fix-issue-318 into e6e85e4e4332fba6d01273b3719c1c11abc644e0 on develop.


Finetuning support

Hi,
Is there any way to train a KWS model, in case we want to train a new wake word model, without having much data?
Or can I finetune an existing model on the new wake word, which is trained on a good amount of data? If yes, how?
Thanks

sorry, currently, here is no such method in our framework. We are investigation on the problem, and will try to figure out it in the future.


Development assets

Xcode 11 provides a new feature called development assets. This is most notably useful for SwiftUI preview generation, and SRG Data Provider should thus provide a dedicated package with all kinds of data we have (SRGMedia, SRGShow, etc.).
We need some packaging script which will download the metadata. I do not recommend committing the development asset package so that users can generate fresh recent data when they need.

Some ideas:

Create several versions of objects lists (reasonable amount of objects, 1 object, no object, a few objects, a lot of objects).
Find a way to associate images in the asset catalog with the objects they are related to.

It is better to start small and then expand. We need to be able to retrieve arrays of the following objects for display:

SRGChannel (TV / radio)
SRGEpisodeComposition
SRGMedia (TV / radio + on-demand / live)
SRGMediaComposition (TV, radio + on-demand / live)
SRGModule
SRGProgramComposition
SRGShow
SRGSong
SRGTopic

We should simply dump JSONs from associated requests, store them in development assets and parse them from there.
We should then use this for a while and see if this suffices. There is still one question, though: How should we deal with images?

No need for development assets. It suffices to perform requests synchronously for preview purposes. SwiftUI executes preview code for 5 seconds, which is enough to get the data to display. Something like:
struct MediaCell_Previews: PreviewProvider {
    static var sampleMedias: [SRGMedia] = {
        let dataProvider = SRGDataProvider(serviceURL: SRGIntegrationLayerProductionServiceURL())
        var sampleMedias: [SRGMedia] = []

        let group = DispatchGroup()
        group.enter()
        dataProvider.tvLatestMedias(for: .SRF) { (medias, _, _, _, _) in
            sampleMedias = medias ?? []
            group.leave()
        }.withOptions(.backgroundCompletionEnabled).resume()
        group.wait()
        return sampleMedias
    }()

    static var previews: some View {
        List(sampleMedias, id: \.uid) { media in
            MediaCell(media: media)
        }
    }
}

Here it is important to dispatch the work to a background thread to avoid a deadlock. There is maybe a way to reduce the associated boilerplate, but this approach provides full flexibility over the kind of sample data we want to use (we can even generate several previews for different BUs if we want, or for radio / TV content).
For this reason, I think we can close this issue. We'll reopen a new one if we have an idea about how we could eliminate this boilerplate.

One final remark: I tried development assets, which also can be associated with framework targets. Only problem is that they are discarded when archiving the framework and thus would have been supplied separately if we had used them. But this approach would have failed anyway when we perform the move to SPM.

Another approach, app-level: Use protocol-oriented programming to define the data interface to views, and mock the data with development assets.


added support for SparkSession, Catalog and Dataset

Testing the changes in AppVeyor and Travis

Current coverage is 80.48% (diff: 38.77%)

Merging #555 into master will decrease coverage by 1.40%

@@             master       #555   diff @@
==========================================
  Files            67         71     +4
  Lines          7447       7678   +231
  Methods        1171       1248    +77
  Messages          0          0
  Branches        953        971    +18
==========================================
+ Hits           6099       6180    +81
- Misses          996       1143   +147
- Partials        352        355     +3

Powered by Codecov. Last update 09d8a2f...ebf01d1

LGTM.   But quite a lot of warnings in C# compilation , such as Sql\Catalog.cs, is it better to suppress or solve them ?
By the way, you still modified the file that "unintentional changes" :)

Thanks for the review, @qualiu .
All the compile-time warnings were on missing XML comments on public methods. I was just trying to get the results of running unit tests and samples in AppVeyor and Travis and also collect CodeCov numbers. With my latest commit, codecov numbers are pretty good. So I will add code comments/documentation before I merge my changes.

@skaarthik Sounds good!  LGTM


The serde crate's derive feature interacts poorly with 2018-style macro imports

Say I have a crate that derives Serialize for one type, and has to manually implement it for another. I believe the "standard" way of doing this would be
use serde_derive::Serialize;
use serde::{Serialize, Serializer};

#[derive(Serialize)]
struct Foo(u32);

struct Bar(u32);

impl Serialize for Bar {
    fn serialize<S>(&self, s: S) -> Result<S::Ok, S::Error>
    where
        S: Serializer,
    {
        self.0.to_string().serialize(s)
    }
}

This works fine in isolation, but if this crate is used alongside some other crate that activates serde's derive feature, it will stop compiling:
error[E0252]: the name `Serialize` is defined multiple times
 --> src/lib.rs:2:13
  |
1 | use serde_derive::Serialize;
  |     ----------------------- previous import of the macro `Serialize` here
2 | use serde::{Serialize, Serializer};
  |             ^^^^^^^^^ `Serialize` reimported here
  |
  = note: `Serialize` must be defined only once in the macro namespace of this module
help: you can use `as` to change the binding name of the import
  |
2 | use serde::{Serialize as OtherSerialize, Serializer};
  |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^

error: aborting due to previous error

For more information about this error, try `rustc --explain E0252`.

The crate can be made to compile in both scenarios by importing Serialize and Serializer from serde::ser rather than serde, but it seems like an easy thing to forget, and you're not going to notice until it breaks someone downstream that's trying to use your crate.
What do you think the right way of avoiding this is? Should the recommended approach change from depending on serde_derive to using the derive feature directly?

I am on board with your proposed solution -- let's update all documentation and examples to recommend depending on the derive macros re-exported by serde rather than using serde_derive.

Instead of deprecating serde_derive I would prefer if multiple imports of the same item did not conflict with each other. In this case serde::Serialize (the macro) and serde_derive::Serialize resolve to the same item, after re-exports are considered. It should be fine for the compiler to accept having both of those imports because together they only import a single macro -- thus invocations of that macro can be resolved without an ambiguity. Will see if I can find an existing issue for this.

To be fair, use serde::Serialize; meaning:

Serialize trait when derive feature is disabled;
Serialize trait and Serialize macro when derive feature is enabled;

is quite surprising without knowing the context, both because a single use can import 2 items at once and because a use can add an item of a different type to the import list depending on a #[cfg] --- with non-macros the compiler would complain. But macros were always special in one way or another. so shrug.

I updated all example code to use serde's derive feature rather than depending directly on serde_derive.


Discussion: question 77 need clarification

Question: https://github.com/lydiahallie/javascript-questions/blob/master/README.md#77-is-this-a-pure-function
I could imagine a situation where the function won't be pure.
let x = {
  [Symbol.toPrimitive]: () => {
    console.log('side effect');
    return 1
  }
}

let y = {
  [Symbol.toPrimitive]: () => {
    console.log('another side effect');
    return 2
  }
}

function sum(a, b) {
  return a + b
}

sum(x, y);

In this case, sum function will return 3, but all console.log are executed.
Obviously, this is a corner case, but....

To best of my knowledge, a pure function is something which will return same result for same inputs, more kind of mathematical formula, !

That's true, but it is not enough.
Another mandatory property is absence of side effects.
https://en.m.wikipedia.org/wiki/Pure_function
console.log is a side effect

@RidgeA  Yes, i agree. the above code now become not a pure function ! Thanks for the explaination.

hhh~ I just wanted to create an issue.


Check GPU functionality makes impossible to pass device numbers in tf virtual config

_check_gpu_existence() makes impossible to pass GPU device numbers starting from 1 to tf.ConfigProto.gpu_options.visible_device_list flag. It is a known bug in tensorflow that is promised to be fixed in 1.9 version. See #16753 for example. The issue is caused by  device_list = device_lib.list_local_devices() call.
Steps to reproduce:

On a machine with several GPU devices and at least two devices visible create a tf session with options:

self.sess = tf.Session(config=tf.ConfigProto(
            allow_soft_placement=True,
            gpu_options=tf.GPUOptions(
                allow_growth=True,
                visible_device_list='1'
            )))

Running a session leads to an error TfGpuId: 1 Existing mapped CUDA GPU id: 1 CUDA GPU id being tried to map to: 0

Current workaround: comment from deeppavlov.core.common.check_gpu import GPU_AVAILABLE import in squad.py

It is possible to use CUDA_VISIBLE_DEVICES environment variable instead of setting visible_device_list in python code.

Is it possible to use just a simple test return tf.test.is_gpu_available() in _check_gpu_existence()?

Unfortunately, tf.test.is_gpu_available() doesn't solve the topic issue.

@seliverstov Please, provide comments when closing issues.


[Test PR- do not merge] Update README.md

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please ask for approval from aakankshaduggal by writing /assign @aakankshaduggal in a comment. For more information see:The Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please ask for approval from aakankshaduggal by writing /assign @aakankshaduggal in a comment. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


undefined method `each' for nil:NilClass

The application throws an exception when trying the example command:
$ ./bin/spotify-export.rb spec/support/multiple-tracks.txt

/Users/owenray/spotify-export/lib/spotify-track.rb:49:in `format_artists': undefined method `each' for nil:NilClass (NoMethodError)
	from /Users/owenray/spotify-export/lib/spotify-track.rb:82:in `get_track_attributes'
	from /Users/owenray/spotify-export/lib/spotify-track.rb:32:in `attributes'
	from /Users/owenray/spotify-export/lib/spotify-track.rb:22:in `name'
	from bin/spotify-export.rb:23:in `block in <main>'
	from bin/spotify-export.rb:20:in `each'
	from bin/spotify-export.rb:20:in `each_with_index'
	from bin/spotify-export.rb:20:in `<main>'

Running on macOS Sierra 10.12.5
ruby 2.4.0p0 (2016-12-24 revision 57164) [x86_64-darwin16]

just noticed this gets resolved by https://github.com/jlund/spotify-export/pull/14

I have the same issue here, but In my case #14 does not solve the problem, it causes infinite recursion. The problem seems that artists parameter is null, not a typo.

I'm getting it here too. git reset'ed (9/17/2017). Anyone found a solution for this? Thanks

The problem is that the API is returning a 401 because no token is passed with the request. As stated by the API used in this gem, https://developer.spotify.com/web-api/get-track/ you will need to implement an Authorization (https://developer.spotify.com/web-api/authorization-guide/) workflow to generate a user token that will be passed in the requests to Spotify.

How I made it work :

Create a spotify app at https://developer.spotify.com/my-applications/#!/applications/create

Use the Client Credential workflow to get an api token from Spotify https://developer.spotify.com/web-api/authorization-guide/#client-credentials-flow

For this part, you will need to Base64 encode the string client_id:client_secret of your Spotify app. I used this https://www.base64encode.org/.
Also, beware to pass the grant_type params in the body and not as an URL-encoded param otherwise you will get a 400.

You get the access_token in the response. I added two lines to the code (last ones) :

lib/spotify-track.rb
  def get_track_attributes
    if local
      local_array = uri.split(':')

      # The array should be length 6
      # ["spotify", "local", "artist", "album", "song title", "duration"]
      name   = URI.decode(local_array[4].gsub('+', ' '))
      album  = URI.decode(local_array[3].gsub('+', ' '))
      artist = URI.decode(local_array[2].gsub('+', ' '))
    else
      track_id = uri[/track(:|\/)(.+)/, 2]
      target  = URI.parse("https://api.spotify.com/v1/tracks/#{ track_id }")
      http    = Net::HTTP.new(target.host, target.port)
      http.use_ssl = true
      request = Net::HTTP::Get.new(target.request_uri)
      access_token = "Zwejkrhqwlkrq..."
      request['Authorization'] = "Bearer #{access_token}"

If I have some time, I'll do a PR to allow a user to pass the access token directly in the args when launching the script (the access token generation will be done manually as I explained above). It is not optimal as the token will stay in your bash history but for now it'll do and I'm sure we'll come up with a better solution.

Hi. Still trying to make this work. Does the token look something like this after base 64 encode? (I messes with it so it's wrong):
  access_token = "YTU5YTc3AmVmADk4NDI2M2E2MGU2ZTU4NGFiYWYyM2Y="

Thanks.

Looks like it, yes. Did you try it out ?

Ok wait didn't read well. It's not the token that should be base64 encoded, it is the string client_id:client_secret.

Still throwing me the same exception, even with adding the access token (which I obtained with curl passing the base64 string) to spotify-track.rb. When encoding the client id and secret to base64, does the string need to follow the exact 'client_id:client_secret" format, with the colon in the middle?
Thanks.

It works, cheers!

@glfaulkner Yes it does.

Hello all. I've been trying to use spotify-export again, but haven't been
able to log in to the developer site. Wanted to know if you guys could:
https://developer.spotify.com/
https://developer.spotify.com/web-api/get-track/
I try to log in, when clicking on dashboard shows the same as before, as if
I wasn't logged in.
Thanks,
-- Fred
On Sat, Oct 13, 2018 at 11:10 AM Bachir<EMAIL_ADDRESS>wrote:

@glfaulkner https://github.com/glfaulkner Yes it does.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jlund/spotify-export/issues/17#issuecomment-429545186,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABLxdlRlJvF9utS2w5F1btQKn_Be9OP3ks5ukfQ5gaJpZM4ONXwC
.

--
-- Fred

@snakenerd works for me, I'm logging in with username & password if that helps.

Complementing the awesome instructions above, for those who are as new to javascript as me, you need to run the following script (adapted from the example from Spotify that's not directly runnable 😂):
var request = require('request'); // "Request" library

var client_id = 'YOUR CLIENT ID';
var client_secret = 'YOUR CLIENT SCRET';

var authOptions = {
  url: 'https://accounts.spotify.com/api/token',
  headers: {
    'Authorization': 'Basic ' + (new Buffer(client_id + ':' + client_secret).toString('base64'))
  },
  form: {
    grant_type: 'client_credentials'
  },
  json: true
};

request.post(authOptions, function(error, response, body) {
  if (!error && response.statusCode === 200) {
    var token = body.access_token;
    console.log(token);
  }
});

So the steps are:

Save the snippet above into a file called fetch_token.js. Fill in the client_id and client_secret. You don't have to encode them to base64 since Buffer does it for you.
You can run it with, for example, nodejs: node fetch_token.js
It will return your with a token, you don't have to encode or decode it any further, just copy it.
change te lib/spotify-track.rb as suggested above, fill in your copied token in the line access_token = "...".
Run ./bin/spotify-export.rb your_playlist.text


Interop test timeout: tests.cloud_to_cloud:node:c++_server:client_streaming

https://grpc-testing.appspot.com/job/gRPC_interop_master/3036/testReport/junit/(root)/tests/cloud_to_cloud_node_c___server_client_streaming/

Looks like this has been fixed sometime during the fixit.


Clean up Reaper Boy

Need to clean up the Reaper Boy demo game:

[x] Removed unused sprites and get artwork down to 2 pages of sprite memory
[x] Add comments to code
[x] Remove any files that are not critical to run the game

This has been cleaned up.


chapter 5 user login page 50

chapter 5 user login page 50  , not write
from werkzeug import check_password_hash , generate_password_hash in example book :)
i know is the very small issue but  I see remember you

i'm sorry the ask already answer :) so i closed  the issue


Allow tree-shaking of frontend plugins

Adding rollup's preserveModules to enable tree-shaking in webpack.
rollup is used to bundle libraries/plugins and webpack to build final app.
This results in non-optimal final app, as webpack is not able to do proper tree shaking.
reference: https://www.whatcouldpossiblygowrongblog.com/posts/tree-shaking-a-right-way
Results are not visible in packages/app (I believe because it has access to all .ts files), but clearly visible when building demo app.

And can be even greater on custom apps.

Not sure whether this should be patch, or minor change. On one hand it changes a lot, on the other I tested it, and it works fine. Also from user's perspective there is no action needed, and everything is ocmpatible.
Hey, I just made a Pull Request!

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

This now fails on packages that have relative monorepo-imports. But new packages should be introduced for those?

Fixing now with preserveModulesRoot, but I do not know whether we can assume src to always be there

It fails on e2e. For some reason, preserveModulesRoot does not work when building e2e with moduleResolutions overridden.

The exports do not change. subpath imports do not work, as there are no .d.ts files there.
And yes, it mostly moves some code from eagerly, to async loading.
I also tried going with exposing apirefs and interfaces on different path, but this is a big braking change. https://github.com/backstage/backstage/pull/22968
It's also posible to introduce new package type for apirefs and types.
I haven't tried it, but it also seems possible, that changing webpack to rollup in app build could help.
It seems this is mostly limitation for packages built with rollup and bundled finally by webpack https://github.com/vitejs/vite/issues/5174
It's also worth noting, that no package internals are exposed. There is still as many .d.ts files as there are entrypoints. So it's not possible now to subpath import something.
So it is not posible to rewrite e.g. import { useEntity } from '@backstage/plugin-catalog-react'; to import { useEntity } from '@backstage/plugin-catalog-react/hooks/useEntity';
I am also not sure I understand the downside (besides the modulesExportRoot). Can you provide more explicit example?

I also believe that when providing custom versions of certain pages, some stuff is not tree-shaken.
E.g. PodExecTerminal imports xterm.css https://github.com/backstage/backstage/blob/master/plugins/kubernetes-react/src/components/PodExecTerminal/PodExecTerminal.tsx#L16
It lands as pure import in kubernetes-react

And root of kubernetes-react lands in kubernetes. I do not see a way for webpack/roolup to tree-shake it.
I was able also to remove @azure/identity and it's dependencies from final package by providing my own impl of ScmIntegrationsApi, which did not end successfully without preserveModules.

It's also worth noting, that in current state rollup can decide to remove whole async loading, that should be provided by createRoutableExtension.
https://www.npmjs.com/package/@backstage/plugin-kubernetes?activeTab=code

plugin-kubernetes gets compiled to a single, completely sync file.
compared to output after preserveModules:
const EntityKubernetesContent = kubernetesPlugin.provide(
  createRoutableExtension({
    name: "EntityKubernetesContent",
    component: () => import('./Router.esm.js').then((m) => m.Router),
    mountPoint: rootCatalogKubernetesRouteRef
  })
);

@Rugvip bumping :)
I do not see any downsides in this approach. Typescript interface stays the same. Final app build is a little bit longer. Internal structure looks a bit different, but it's an implementation detail that does not impact how you use an app.
If preserveModulesRoot is the problem, I should be able to figure out a way to calculate it from scriptEntryPoints. And if they do not have common one, just go "the old way" and put a warning to user. or create multiple configs.

It's not stale... @Rugvip any more thoughts?
Can you give a more exact example of downsides to this approach?

It's not stale... @Rugvip any more thoughts?
Can you give a more exact example of downsides to this approach?

Still open for more discussion.


Add BFF Blazor template

By submitting an Issue to this repository, you agree to the terms within the Auth0 Code of Conduct.
Describe the problem you'd like to have solved
It would be great to have a template for the BFF pattern for hosted blazor wasm applications. The current Blazor wasm template, if I'm not mistaken, required the creation of 2 applications in Auth0; an app and an api resource. A BFF template would allow dev to delegate authentication operations to the backend and require less setup within auth0.
Alternatives and current work-arounds
There are samples that show this pattern on GitHub; example

That's a great suggestion @cecilphillip! I'll take it into account for the next releases. Thanks!

Please provide a BFF for the new. NET 9 Blazor Hybrid Maui and Web template against an ASP. NET Core API.


Uses out of date Azure library for keyvault

There's a new library built on top of Azure.Identity/Azure.Security.KeyVaults.Secrets: Azure.Extensions.AspNetCore.Configuration.Secrets.  There are docs here for that:  https://docs.microsoft.com/en-us/dotnet/api/overview/azure/extensions.aspnetcore.configuration.secrets-readme

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

ID: 2cc0b77a-df21-7007-5cec-4399d98c6dc6
Version Independent ID: efe5ae5e-ff64-f1c7-6920-4512fc8f36d1
Content: Azure Key Vault Configuration Provider in ASP.NET Core
Content Source: aspnetcore/security/key-vault-configuration.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

@twsouthwick can you outline the changes that need to be made? Any chance you could provide the first draft PR and I could clean it up?


fatal error: concurrent map read and map write

call "Watch" method for a DigitalPin while another DigitalPin get an interrupt will cause "fatal error: concurrent map read and map write" error , beacause one write map (interruptablePins) and anthoer read this map.
Log:
`fatal error: concurrent map read and map write
goroutine 18 [running]:
runtime.throw(0x233cf7, 0x21)
Go/src/runtime/panic.go:566 +0x78 fp=0x10618364 sp=0x10618358
runtime.mapaccess2_fast32(0x203100, 0x106c9880, 0xa, 0x40, 0xffffffff)
Go/src/runtime/hashmap_fast.go:66 +0x5c fp=0x10618374 sp=0x10618364
github.com/kidoman/embd/host/generic.initEpollListener.func1(0x106e59b0)
/src/github.com/kidoman/embd/host/generic/interrupt.go:65 +0x1b0 fp=0x106187d4 sp=0x10618374
runtime.goexit()
Go/src/runtime/asm_arm.s:998 +0x4 fp=0x106187d4 sp=0x106187d4
created by github.com/kidoman/embd/host/generic.initEpollListener
src/github.com/kidoman/embd/host/generic/interrupt.go:70 +0x194`
can you add an mutex here, thx.

why did you close this issue?


ZwClose call after ZwDeleteKey

"Once the driver has finished its manipulations, it must call ZwClose to close the handle—unless it has already called ZwDeleteKey to delete the key. (Once a key is deleted, all the open handles to it become invalid, so the driver must not close the handle in this case.)"
But also there is "After a call to ZwDeleteKey invalidates the key handles, you must call ZwClose to close the key handles." from
https://docs.microsoft.com/en-us/windows-hardware/drivers/ddi/wdm/nf-wdm-zwdeletekey
So, which one is correct?

Сведения о документе
⚠ Не вносите правки в этот раздел. Это необходимо для связывания страницы сайта docs.microsoft.com с вопросом на GitHub.

ID: c3f638cc-3575-9470-931e-0e14a3e432ab
Version Independent ID: 494be0f1-eea1-1a8b-8451-46539facee90
Content: Using a Handle to a Registry-Key Object - Windows drivers
Content Source: windows-driver-docs-pr/kernel/using-a-handle-to-a-registry-key-object.md
Product: windows-hardware
Technology: windows-devices
GitHub Login: @tedhudek
Microsoft Alias: tedhudek

Hi @drmkeeper, thanks for the feedback and apologies for the delay. The first statement you cite was incorrect. I just pushed an update that will appear on the live site within a few minutes. See 53d8bd47b for details.

The new text reads:
Once the driver has finished its manipulations, it must call ZwClose to close the handle—even if it has already called ZwDeleteKey to delete the key. (Once a key is deleted, all the open handles to it become invalid, however the driver must still close the handle.)


There is no error logging!

I removed error logging because it was causing big performance issues.  Maybe. But I should reimplement it in a better way, either way.

I removed the old error logging code in preparation for a new one: fc00027a3bebc812aa0b4962839650a2b03c11e2

Closing because of upcoming rewrite


Lunarvim lazygit dont show the git Icon

Description
I use lazygit with lunarvim. I'm not sure how lunarvim handles lazygit, if through the plugin or something else, seems like it's calling the lazygit executable with toggleterm which is another plugin.

The expected behaviour is to have the same icon as git filetypes:

Additional information
No response

Implemented in #64, thanks!


[request] Implement dsl_sink_encode_dimensions_get/set services (for File, Record, RTSP, WebRTC SInks)

Need to be able to scale the output of all Encode-Sinks; File, Smart-Record, RTSP, and WebRTC.  This missing functionality was brought up by "trdzung" on discord.
Important Note: the Smart-Record sink has its own get/set dimension services as provided by the NVIDIA smart record-bin.  The new services set/get the dimensions of the video-converter which is linked upstream of the encoder.
New services
/**
 * @brief Gets the dimensions, width and height, in use by the Encode Sink's
 * input video-converter.
 * @param[in] name name of the Encode Sink to query
 * @param[out] width current width setting in pixels. 0 = no scaling (default).
 * @param[out] height current height setting in pixels. 0 = no scaling (default).
 * @return DSL_RESULT_SUCCESS on success, DSL_RESULT_TILER_RESULT
 */
DslReturnType dsl_sink_encode_dimensions_get(const wchar_t* name,
    uint* width, uint* height);

/**
 * @brief Sets the dimensions, width and height, for the Encode Sink's input
 * video converter to use. Set to zero for no-scaling.
 * @param[in] name name of the Record Sink to update
 * @param[in] width new width setting to use in pixels
 * @param[in] height new height setting to use in pixels
 * @return DSL_RESULT_SUCCESS on success, DSL_RESULT_SINK_RESULT
 */
DslReturnType dsl_sink_encode_dimensions_set(const wchar_t* name,
    uint width, uint height);

Complete and merged into v0.26.alpha

Thanks for your support. I will try it as soon as possible.


Suggestion: On the record page, rethink the edit button for pure/binary association tables in the related-records section

Related to #1862: Currently, if you have Table_A and Table_B linked by a pure/binary association table, then Table_A's record page will have a section that is:

labelled Table_B
with rows from Table_B
and view buttons that take you to Table_B rows
and edit buttons that take you to the association table
and unlink buttons that remove the association

I think the fact that the edit button goes to a different table than everything else in that section is confusing, and my suggestion is to either get rid of it or make it open a recordedit window for the appropriate row in Table_B (similar to what would happen if Table_B had a direct foreign key to Table_A).
Admittedly, this means more work for the user in some cases. Swapping out a record on the far end (i.e., switch one Table_B record for another on the Table_A page) would take one or two more mouse clicks -- instead of [click edit button, select record, click submit, (maybe) return to the Table_A record page], the process would be click edit button, select record, click submit, click unlink button, confirm].
Changing the near-end (Table A) record or changing both ends would be more complicated. But I'm not sure anyone outside our group would think to do that -- you need to have a mental model that acknowledges the association table in order to even think about clicking on the edit button for those cases.

@ljpearlman We changed the behavior of the edit button. It should now send users to the same row that the view button does. Can you please verify the change?


Support for aggregate and join operation in KafkaStreamsTracing

I would like to trace aggregate and join operations in the KStream DSL more explicitely but can not manage to wrap them into a transformer.
It would be great to have support for these operations similar to the existing map, etc.
Is that feasible at all? If so, I would be happy to contribute if someone could point me towards a potential solution.

@boxsterman thanks for the interest in contributing!
It might be a bit more challenging than simple transformers, but at the end joiners are materialized as processors, and we could be able to replicate and wrap joining functionality, similar to map operation.
org.apache.kafka.streams.kstream.internals.KStreamKTableJoinProcessor could be a good place to get started.
If you are up to experimenting with this, I'd be happy to help.

@jeqo I've already started digging into that direction and will definitely give it a try.

hi guys, is there any progress with this issue? I would like to trace logs from org.apache.kafka.streams.kstream.ValueJoiner when joining so this would be very helpfull :)

@matus753 want to give a PR a try? It has been over a year so I don't think anyone is working on it.

The challenge here is how to propagate tracing context information in KStream joins and aggregates. These operations are backed by a state store and this state store does store the message, but does not store any metadata (e.g. custom message headers) associated with it. That’s why the tracing context is lost if a state store is involved.
I’ve looked into this and it would require a change to the KStream client lib in order to include the metadata into the stored message. I’ve some ideas on how this could be done, but didn’t manage the time to kick off an official KIP.
Unfortunately this issue significantly limits the whole Kamon Kafka Stream instrumentation since almost all of the streams in my world do use joins and aggregates.
If some one is interested in picking this up, I would be more than happy to share my experiences and of course also help implementing it.

On 14 Oct 2020, at 12:56, Adrian Cole<EMAIL_ADDRESS>wrote:
@matus753 https://github.com/matus753 want to give a PR a try? It has been over a year so I don't think anyone is working on it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/openzipkin/brave/issues/889#issuecomment-708326333, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABS2GL4BI2ZSPILTVYYG6VDSKV7U7ANCNFSM4HDAJOOA.

@boxsterman so you mean it is currently like old kafka where some people used enveloping etc. I wonder if any phases can be known to execute in the same JVM? If so, we aren't limited to what's persisted to disk... Is it possible for you to raise an example project (maybe using zipkin's docker image for kafka) showing this problem? I think that could help all get on the same page even someone who might eventually need to KIP it

Adrian makes a good point. I'd also like to validate this limitation with an existing example. Joins (at least stream-table) actually propagate headers, here is a small example: https://github.com/jeqo/tracing-kafka-apps. Feel free to extend it to show case your issues.
@boxsterman probably you've already saw it, but there is an open issue related to this: https://issues.apache.org/jira/browse/KAFKA-7718
I also created https://cwiki.apache.org/confluence/display/KAFKA/KIP-634%3A+Complementary+support+for+headers+in+Kafka+Streams+DSL, that is still on early discussion. I plan to work on it this quarter.
Would love to get your feedback there.


V2 - CMake

Experiment with CMake and see if it is better than current makefile.

CMake is now included and {fmt} is used as a sub module as well. It is indeed good to use but it can be a bit hard to learn at first.


Support Nginx sidecar as an optional service

Nginx implements reverse proxy and SSL accelerator. It enables us to access the application with HTTPS.
This would be helpful when developers need secure connection like OAuth development.
FYI: https://www.nginx.com/resources/glossary/sidecar/

Feel free to fork Sail and adjust it to your needs! 👍


refactor dropdown layout

This PR following changes:

Refactor dropdown layout
The dropdown icon in the wallet details page appears to be text so far

The width of the dropdown should be equal to the width of the displayed options. I don't think we need to increase width?

The width of the dropdown should be equal to the width of the displayed options

i think you have a point probably should use width of the largest item? but what happens when the largest item is too long, obviously to truncate right?, so what's too long in this case?

The width of the dropdown should be equal to the width of the displayed options

i think you have a point probably should use width of the largest item? but what happens when the largest item is too long, obviously to truncate right?, so what's too long in this case?

the width of dropdown have define max width, so we don't need do it, if dropdown and dropdown menu have different width so ui not good for me

@bugjt justindo has a good point
let's just use a sane max width and truncate anything longer than that

This means we will have the dropdown and the dropdown menu the same size and both will be the width of the largest item. Is that correct?

We cannot rename names longer than 20 characters and when the name is too long it will be truncated with "..."

so the truncation is where i was getting to, the screenshot by the left works well. seem it truncates after 10 characters

So we will cut after 10 characters or 20 characters (default maxDropdownItemTextLen = 20)?

If the width of the dropdown is equal to the length of the largest element, it will look like this:

This is okay

It's ready for review.

i want to move the dropdown and balance to the same line on mobile.
For the sake of the mobile implementation, i'' say we revert the dropdown to a dynamic width

i want to move the dropdown and balance to the same line on mobile.
For the sake of the mobile implementation, i'' say we revert the dropdown to a dynamic width

I think we should hide "watch only" label on watch only wallet

I think we should hide "Watch-Only" label on watch only wallet

I agree

It's ready for review.


Undo supported, UI changes

Added undo support for the majority of actions, including modifying the animator.
Changed UI to allow for (in my humble opinion) slightly easier readability and faster editability.
Hopefully it's helpful <3

Looks really nice! Appreciate the UI changes, I'm not exactly a UX designer so I am definitely open for any changes to make it much easier to use. I'll test it out a bit and see if there are any issues and I'll merge it!


Linux scripts

It would be very easy to make a mistake while doing this, so please have a look.
I've removed all references to Subversion, and made one CloneRepo function that is called in several places.

Coverage increased (+0.07%) to 86.656% when pulling a15c225b916506d8de34ad4738b38d6a7f57628b on linux-scripts into affea8712684d6a367a0c32003cb303ae3aeec89 on master.

I've tested local builds and on the build server, but it's not trivial testing every corner of this. I think the system tests are still directed at Gitolite.


LINK-1366 | Allow only select email as notification type in enrolment

Description
Allow only select email as notification type in enrolment
Closes
LINK-1366
Screenshot

Codecov Report

Merging #196 (31a23be) into develop (1e2602e) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #196      +/-   ##
===========================================
- Coverage    99.88%   99.85%   -0.04%
===========================================
  Files          506      506
  Lines         9452     9452
  Branches      2073     2073
===========================================
- Hits          9441     9438       -3
- Misses          11       14       +3

Impacted Files
Coverage Δ

...olment/enrolmentFormFields/EnrolmentFormFields.tsx
100.00% <ø> (ø)

src/domain/enrolment/utils.ts
100.00% <ø> (ø)

... and 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Missing dependency err after fresh clone

Shows up for both ios and android js files:

Take a peek later. As the README states linting rules are intended to be
sane, which Airbnb is not. What happens when you clone fresh and install?
You didn't provide much context otherwise to help.
On Sat, Feb 27, 2016 at 6:25 PM Chris Vandenberg<EMAIL_ADDRESS>wrote:

Shows up for both ios and android js files:
[image: screen shot 2016-02-27 at 5 23 05 pm]
https://cloud.githubusercontent.com/assets/12105346/13376338/f978ec60-dd76-11e5-8427-88a403019d34.png
—
Reply to this email directly or view it on GitHub
https://github.com/jhabdas/react-native-webpack-starter-kit/issues/56.

--
Josh Habdas
tel: +1<PHONE_NUMBER>
web: http://habd.as


[BUG] Settings modal do not work motion reduce mode

Temp. fix: https://github.com/IgorKowalczyk/igorkowalczyk.github.io/commit/65409d7354cf823b577ea891ce98caea75a6ea16

I leave the code as it is, if I manage to fix it I will reopen this PR.
Final code: https://github.com/IgorKowalczyk/igorkowalczyk.github.io/commit/65409d7354cf823b577ea891ce98caea75a6ea16

I leave the code as it is, if I manage to fix it I will reopen this PR.
Final code: https://github.com/IgorKowalczyk/igorkowalczyk.github.io/commit/65409d7354cf823b577ea891ce98caea75a6ea16


Improvement for helm chart's output message.

I'm installing csi-driver chart with below commands:
helm upgrade juicefs-csi-driver  juicefs-csi-driver/juicefs-csi-driver --install -f  juicefs_values.yaml

and the content of juicefs_values.yaml below:
- name: juicefs
  enabled: true
  reclaimPolicy: Retain
  backend:
    name: juicefs
    metaurl<EMAIL_ADDRESS>    storage: s3
    accessKey: minio
    secretKey: minio123
    bucket: http://minio-service.default.svc:9000/milvus-test

I'm using juicefs as the name parameter. and I've got the output for test if juicefs is working correctly like below:

...

apiVersion: v1
kind: PersistentVolumeClaim
metadata:
  name: juicefs-pvc
  namespace: default
spec:
  accessModes:
  - ReadWriteMany
  resources:
    requests:
      storage: 10Pi
  storageClassName: juicefs-sc

...

the example using juicefs-sc as the storageClassName, but indeed, we create it with the name with juicefs.

fixed by https://github.com/juicedata/juicefs-csi-driver/pull/165


[Request - Product] Elastic beats

Full and short name of product
Elastic beats is also known as beats.
Does this product have LTS versions? What are the intervals between each LTS version?
No.
What is the website for the product and for its version information?

Landing page: https://www.elastic.co/beats/
Support matrix: https://www.elastic.co/support/matrix

Additional context
Elastic beats are in fact composed of multiple softwares:

Filebeat
Metricbeat
Packetbeat
Winlogbeat
Auditbeat
Heartbeat
Functionbeat

Elastic beats are part of the Elastic Stack, and share the same release cadence.

Note: there is already a page on endoflife.date: https://endoflife.date/filebeat.

See also: #1928.

So we'll merge the filebeat page into a single Elastic Beats page?

Yep. This is similar to #1928, but I think it is best to have only one page for beats. What do you think ?

+1 on single page for beats. Haven't checked the versioning scheme for various Beats products, but as long as they're matching - we should have a single page.

Yep they are matching.

Nope I was wrong, there have a consolidated changelog (see https://www.elastic.co/guide/en/beats/libbeat/8.5/release-notes-8.5.3.html).


Can't install NuGet Package in Portable Class Library

When installing the NuGet in my PCL app, I get this error:

Attempting to gather dependency information for package 'Xamarinos.AdMob.FormsPlugin.1.0.8' with respect to project 'LifeBalance', targeting '.NETPortable,Version=v4.5,Profile=Profile259'
Attempting to resolve dependencies for package 'Xamarinos.AdMob.FormsPlugin.1.0.8' with DependencyBehavior 'Lowest'
Resolving actions to install package 'Xamarinos.AdMob.FormsPlugin.1.0.8'
Resolved actions to install package 'Xamarinos.AdMob.FormsPlugin.1.0.8'
For adding package 'Xamarinos.AdMob.FormsPlugin.1.0.8' to project 'LifeBalance' that targets 'portable45-net45+win8+wp8+wpa81'.
Install failed. Rolling back...
Package 'Xamarinos.AdMob.FormsPlugin.1.0.8 : Xamarin.Forms [<IP_ADDRESS>05, )' does not exist in project 'LifeBalance'
Could not install package 'Xamarinos.AdMob.FormsPlugin 1.0.8'. You are trying to install this package into a project that targets '.NETPortable,Version=v4.5,Profile=Profile259', but the package does not contain any assembly references or content files that are compatible with that framework. For more information, contact the package author.

I can however install the NuGet package in the iOS solution, just not the PCL shared library.

I've the same problem, how can i resolve it?
Thanks
Alex

I couldn't clone and add the projects due to specific version of Forms
being required (outdated in mine, and in the wrong format), perhaps that is
the issue. But I'm not familiar with NuGet.
For a quick fix, copy these .cs files to your PCL project:
https://github.com/LosXamarinos/Xamarinos.AdMob.FormsPlugin/tree/master/PlugIn/Xamarinos.AdMob.Forms/Xamarinos.AdMob.Forms.FormsPlugin.Abstractions
Copy the one shared .cs file to each platform (you could create a shared
project if you wish which will make it easier to edit the file):
https://github.com/LosXamarinos/Xamarinos.AdMob.FormsPlugin/blob/master/PlugIn/Xamarinos.AdMob.Forms.Shared/CrossAdmobManager.cs
Copy each specific platform .cs files to the correct platform project:
Here it is for iOS:
https://github.com/LosXamarinos/Xamarinos.AdMob.FormsPlugin/tree/master/PlugIn/Xamarinos.AdMob.Forms/Xamarinos.AdMob.Forms.FormsPlugin.iOS
Then you should be good.
On Mon, Mar 6, 2017 at 10:56 AM, realex82<EMAIL_ADDRESS>wrote:

I've the same problem, how can i resolve it?
Thanks
Alex
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/LosXamarinos/Xamarinos.AdMob.FormsPlugin/issues/8#issuecomment-284364991,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABM8GZsyA4j4Gu8E8W4f3yRpIGmxFd9Rks5ri-Z0gaJpZM4MIzYx
.

ok, thanks, i'll try this, but is too arcane as resolution. no add from developer  Alejandro  ?

I fixed it by doing the following:

Forked the Repository
Cloned it on my computer
Opened the Xamarinos.AdMob.Forms.Abstractions.csproj on Notepad
Edited the TargetFrameworkProfile tag from Profile78 to Profile111

I also commited it and asked the owner for a pull requst. Let's hope he accepts it, if not, you can just compile the project using this modifications or by cloning my fork https://github.com/HarkDev/Xamarinos.AdMob.FormsPlugin in which I also added an extra option to add test devices to the Banner renderer.

I have change me project to Profile 111. It's still not install

Too bad the NuGet package has not been updated.  It's very nice, easy to use plugin.  I built a test app that worked fine.  When I wanted to add it to my project though it wouldn't install because of the compatibility issue.  I tried building the updated version but there are many issues to solve upon install.  Just installing the .cs files as mentioned above introduced problems too.  I will try some other things or maybe another plugin.  BTW, one additional step was left out though VS will alert you to it in that the activity must  be added in the Android manifest like this:
<activity android:name="com.google.android.gms.ads.AdActivity"           android:theme="@android:style/Theme.Translucent"        android:configChanges="keyboard|keyboardHidden|orientation|screenLayout|uiMode|screenSize|smallestScreenSize"/>

@prankard  shared project is only referenced on android project. why should we copy it to both projects? Interstitial ads implementation looks to me not correct. anybody could achieve it=

I just implemented this package and works fine for interstiital https://github.com/ioliver85/Xamarin.Admob


fix(system_monitor): fix parameter threshold of CPU Usage monitoring

Signed-off-by: v-nakayama7440-esol<EMAIL_ADDRESS>Description
Change CPU usage threshold and counter threshold values because there is delay to generate a warning/error after the fix https://github.com/autowarefoundation/autoware.universe/pull/557. There is regression.

Before the fix:

≧96% x 1 time -> Warning, ≧96% x consecutive 2 times -> Error
≧100% x 1 time -> Error

After the fix:

≧96% x consecutive 2 times -> Warning
≧100% x consecutive 2 times -> Error

I believe 100% of CPU usage at a moment shall be allowed. Or rather continuous CPU high load will possibly cause hazardous conditions.

So the change is made as follows:

≧96% x 1 time -> Warning, ≧96% x consecutive 2 times -> Error

Related links
https://github.com/autowarefoundation/autoware.universe/pull/557
Tests performed

Simulate CPU high load by using stress-ng command.
stress-ng -k -c 12 -l 96

Verify cpu_monitor generates a warning at 1 time of ≧96%.

Verify cpu_monitor generates an error  at consecutive 2 times of ≧96%.
ros2 topic echo /diagnostics | grep -e "cpu_monitor: CPU Usage" -A 7

Notes for reviewers

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

Verified.
ros2 topic echo /diagnostics | grep -e "cpu_monitor: CPU Usage" -B 1 -e "cpu_monitor: CPU Usage" -A 7

stress-ng -k -c 12 -l 96


Update CodePush example app

Updates the codepush example app by regenerating the project with RN 0.56.
Note: the release instructions in the README currently seem to assume that only iOS should be built, I wasn't too sure on whether we need to support both, or whether we could get rid of this script entirely in favour of linking to the docs. I've left as-is for now.

@martin308 sorry about the long wait in replying to this. It'd be good to get your opinion on whether you think the npm release step is still helpful, which is why I haven't updated any of the steps to use AppCenter, or documented any required ENVARs yet. My current thinking was that we could change the README to add something like the following steps instead:

Deploy a new code bundle (linking to AppCenter instructions, adding a code sample, making a note about codeBundleId)

Upload source maps to Bugsnag (linking to our docs, adding a code sample)

The disadvantage is that this approach would be a bit more manual than npm release. However, it would mean our instructions work on Windows + Linux, and we wouldn't necessarily have to document setting various ENVARs. What are your thoughts?


Transform own darknet weights for this model

I have received a yolov3 model or the weights of a yolov3 model that was trained with darknet from AlexeyAB. If I now perform the transformation of the weights I get the following error message:
The Yolov3 network was trained with only one channel. Since this change is also supported in this model, this transformation should probably not be an impossible obstacle for the transformation.
 The structure of the yolov3 network was not modified so that it is exactly the same as the ordinal.

i have the same problem on a custom tiny yolo v3. the model seems to parse fine then i get the same 'failed to read all data' error as OP. Please advise

Hi,yolo.weights is COCO's weights file, so when you run convert.py, the parameter (num_classes) should be 80, not the number of categories in your own dataset, otherwise you will get "AssertionError: failed to read all data"

Hi, I've been reading around in the issues here -- I see threads about custom models, but none seem to resolve converting a custom model witth (in my case) more than 80 classes ? Is it possible to do this or no? The model is all trained. I started with thge stock yolov3 weights and ran them on my own (very distinct) dataset-- and it works well. I can even convert it to a .pb file with other repos. But this one offers such clean output tensors and I was hoping to use it. Is it possible ? Or no ? Thanks for any help --

Hi, I've been reading around in the issues here -- I see threads about custom models, but none seem to resolve converting a custom model witth (in my case) more than 80 classes ? Is it possible to do this or no? The model is all trained. I started with thge stock yolov3 weights and ran them on my own (very distinct) dataset-- and it works well. I can even convert it to a .pb file with other repos. But this one offers such clean outputs and I was hoping to use it. Is it possible ? Or no ? Thanks for any help --


docs: Create pull_request_template.md

This is the template for PRs, for us and for the outside world (to, e.g., add models and tasks). MTEB has a dataset and task checklist. For reference, below is the template they use.

Checklist

[ ] Run tests locally to make sure nothing is broken using make test.
[ ] Run the formatter to format the code using make lint.

Adding datasets checklist

Reason for dataset addition: ...

[ ] I have run the following models on the task (adding the results to the pr). These can be run using the mteb -m {model_name} -t {task_name} command.

[ ] sentence-transformers/paraphrase-multilingual-MiniLM-L12-v2
[ ] intfloat/multilingual-e5-small

[ ] I have checked that the performance is neither trivial (both models gain close to perfect scores) nor random (both models gain close to random scores).
[ ] If the dataset is too big (e.g. >2048 examples), considering using self.stratified_subsampling() under dataset_transform()
[ ] I have filled out the metadata object in the dataset file (find documentation on it here).
[ ] Run tests locally to make sure nothing is broken using make test.
[ ] Run the formatter to format the code using make lint.

Adding a model checklist

[ ] I have filled out the ModelMeta object to the extent possible
[ ] I have ensured that my model can be loaded using

[ ] mteb.get_model(model_name, revision_id) and
[ ] mteb.get_model_meta(model_name, revision_id)

[ ] I have tested the implementation works on a representative set of tasks.

anything from the DGEB checklist you wanna add?
On Mon, Jul 15, 2024 at 9:03 AM scopello @.***> wrote:

@.**** approved this pull request.
Looks good, thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/TattaBio/DGEB/pull/13#pullrequestreview-2177658515,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAWWZK2CLMPW4JBBSTWFLZTZMPCA7AVCNFSM6AAAAABKXZ3TYWVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDCNZXGY2TQNJRGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

Two thoughts:

This is totally a nit, but do we care about all of these distinctions? On my end, all I care about is breaking change/non breaking change + motivation for the PR.

- [ ] Bugfix
- [ ] Feature (e.g., including new dataset or model)
- [ ] Code style update (formatting, local variables)
- [ ] Refactoring (no functional changes, no api changes)
- [ ] Build related changes
- [ ] CI related changes
- [ ] Documentation content changes
- [ ] Other... Please describe:

Perhaps contradictorily, I'd like to have a section for the reviewer to go through:
[ ] CI passes
[ ] docs updated
[ ] manual spotcheck

Perhaps off topic, but should we have a separate template for the actual results submission? Or could we fold that into this? I can create an issue and take this on if we think it's a good idea.

On 1, I think my thought is that I would like to appropriately tag the commit, and these correspond 1:1 with the conventional commit standard. I'm open to changing them or making them more aligned with this project, though. Thoughts?
Imo, CI passes: that's automated and required to merge.
docs updated: added
manual spotcheck: not sure if this adds anything?
On 2, I think we could fold into this. It seems like it can get annoying to have multiple PR templates. I'll merge this, and then we can iterate on it if that works!

works for me!
On Mon, Jul 15, 2024, at 12:58 PM, Joshua Kravitz wrote:

Two thoughts:

This is totally a nit, but do we care about all of these distinctions? On my end, all I care about is breaking change/non breaking change + motivation for the PR.
`- [ ] Bugfix

[ ] Feature (e.g., including new dataset or model)
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:
`
Perhaps contradictorily, I'd like to have a section for the reviewer to go through:

[ ] CI passes [ ] docs updated [ ] manual spotcheck

Perhaps off topic, but should we have a separate template for the actual results submission? Or could we fold that into this? I can create an issue and take this on if we think it's a good idea.
On 1, I think my thought is that I would like to appropriately tag the commit, and these correspond 1:1 with the conventional commit standard https://www.conventionalcommits.org/en/v1.0.0/. I'm open to changing them or making them more aligned with this project, though. Thoughts?

Imo, CI passes: that's automated and required to merge.
docs updated: added
manual spotcheck: not sure if this adds anything?
On 2, I think we could fold into this. It seems like it can get annoying to have multiple PR templates. I'll merge this, and then we can iterate on it if that works!
—
Reply to this email directly, view it on GitHub https://github.com/TattaBio/DGEB/pull/13#issuecomment-2229276944, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGA4JHE4NBZ7GJHUUWMMLJ3ZMQSXJAVCNFSM6AAAAABKXZ3TYWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMRZGI3TMOJUGQ.
You are receiving this because your review was requested.Message ID: @.***>


Doc 4047

Corda 4 tutorials content duplicated on the menu bar

@amiegrace12 - do we have the equivalent .md files in the 4.9 folder to the ones deleted from 4.8 folder ?
Have you validated 1 to 1 mapping ?


Nadel.kt namespaced support for namespaced field

!!!!HEAVY WIP!!!!!
will update soon to make it less hacky, just chucking it in in case anyone is wondering where we are at with this

ok, should be reviewable now


Use union type in UseStrictWalker#handleBlock argument type signature

Once tslint supports 1.4 syntax, we can safely use the new syntax in its source code.

In particular,
private handleBlock(node: ts.Declaration, block: ts.Block) {

should become
private handleBlock(node: ts.Declaration, block: ts.Block | ts.ModuleBlock) {


[DON'T MERGE YET] [Fix] Catching up upstream CPFL/Velodyne

Status
DEVELOPMENT
Description
I fixed remapping VLP-32C  topic.
https://github.com/CPFL/velodyne/pull/6
@amc-nu catched up upstream of original repo.
https://github.com/CPFL/velodyne/pull/7
Related PRs
https://github.com/CPFL/Autoware/pull/1668

@amc-nu @aohsato are there any specific changes from the upstream velodyne that are tied to Autoware? or is it possible to use upstream directly?

@kfunaoka @amc-nu @aohsato the differences between the updated CPFL/velodyne fork that is being used in Autoware and the upstream version of the drivers is limited to a bunch of launch and yaml files and an if statement in driver.cc. See images
It is the right time to remove the code from the Autoware repo and use the upstream version directly. The change to driver.cc should be pushed upstream and the launch and yaml files can be kept in the Autoware repo if they are Autoware-dependent.

@sgermanserrano @amc-nu By #1965 #1966, it's better to remove this subtree so I close this PR. Thanks!


Eve history crash: RangeError: The value of "value" is out of range. It must be >= 32768 and <32676. Received 35940

This plugin has been working wonderfully. The Eve history charts are super valuable. They still work great for Electricity Delivered and Gas, but my Electricity suddenly stopped working yesterday!

Then I was able to screenshot this, the actual error: 😨

RangeError: The value of "value" is out of range. It must be >= 32768 and <32676. Received 35940

Happy to provide you with a dump of anything you need. Happy to investigate myself if you can give pointers 😊

P.S.: I just posted a very detailed response to your questions on the PR — which you merged while I was working on it, so I'm now worried you won't see it 😅 https://github.com/ebaauw/homebridge-p1/pull/69#issuecomment-1425648028

The value should have been treated as an unsigned int, so it can handle values up to 65535, see https://github.com/ebaauw/homebridge-lib/commit/4a1ddf006b876fe129ca69fd175d6b1311bce8c4.
I am a bit worried about the history value of 35940.  This would indicate that you consumed 3596 Wh or 3.5940 kWh since the previous history entry (presumably 10 minutes ago).  Was the P1 cable disconnected, causing consumption to be added over multiple 10-min periods?

As written at https://github.com/ebaauw/homebridge-p1/pull/69#issuecomment-1427000431:
git cloned https://github.com/ebaauw/homebridge-p1/commit/e07fc82b69da3b0f48562ee47d4296ec5d177397 and npm linked it.
That still resulted in this:
[2/13/2023, 12:21:41 PM] [P1] Electricity History: History Entries: error: RangeError: The value of "value" is out of range. It must be >= -32768 and <= 32767. Received 35940
    at new NodeError (node:internal/errors:399:5)
    at checkInt (node:internal/buffer:74:11)
    at writeU_Int16LE (node:internal/buffer:724:3)
    at Buffer.writeInt16LE (node:internal/buffer:867:10)
    at ConsumptionValue.writeEntry (/Users/admin/Desktop/homebridge-p1/node_modules/homebridge-lib/lib/ServiceDelegate/History.js:79:47)
    at History.entryToBuffer (/Users/admin/Desktop/homebridge-p1/node_modules/homebridge-lib/lib/ServiceDelegate/History.js:602:19)
    at History._onGetEntries (/Users/admin/Desktop/homebridge-p1/node_modules/homebridge-lib/lib/ServiceDelegate/History.js:687:24)
    at CharacteristicDelegate._onGet (/Users/admin/Desktop/homebridge-p1/node_modules/homebridge-lib/lib/CharacteristicDelegate.js:353:26)
    at Characteristic.emit (node:events:513:28)
    at /usr/local/lib/node_modules/homebridge/node_modules/hap-nodejs/src/lib/Characteristic.ts:2463:14
[2/13/2023, 12:21:42 PM] [P1] Electricity History: History Entries: error: RangeError: The value of "value" is out of range. It must be >= -32768 and <= 32767. Received 35940
    at new NodeError (node:internal/errors:399:5)
    at checkInt (node:internal/buffer:74:11)
    at writeU_Int16LE (node:internal/buffer:724:3)
    at Buffer.writeInt16LE (node:internal/buffer:867:10)
    at ConsumptionValue.writeEntry (/Users/admin/Desktop/homebridge-p1/node_modules/homebridge-lib/lib/ServiceDelegate/History.js:79:47)
    at History.entryToBuffer (/Users/admin/Desktop/homebridge-p1/node_modules/homebridge-lib/lib/ServiceDelegate/History.js:602:19)
    at History._onGetEntries (/Users/admin/Desktop/homebridge-p1/node_modules/homebridge-lib/lib/ServiceDelegate/History.js:687:24)
    at CharacteristicDelegate._onGet (/Users/admin/Desktop/homebridge-p1/node_modules/homebridge-lib/lib/CharacteristicDelegate.js:353:26)
    at Characteristic.emit (node:events:513:28)
    at /usr/local/lib/node_modules/homebridge/node_modules/hap-nodejs/src/lib/Characteristic.ts:2463:14

Updating to 1.3.3-0 resolved this and allowed the Eve history to finish loading:

Yeah, I think (hope), I finally tamed the history for Total Consumption.  Also, I’m now simply ignoring the invalid value (after issuing a warning message), and continue with the next entry.

Published v1.3.3.

This has been reliably ever since updating 1.3.3-0! 🥳  Closing 😊


Fix type stability for indexing fields of Model/GraphInfo

Making a note here to look into the type stability of indexing fields of GraphPPL.Model.
Indexing using m[VarName] is not type stable and is having downstream effects on the field getters (e.g. get_node_value).

The issue is caused by the type constructors not being concrete enough. Will fix this with a small pr. Thanks to @phipsgabler for the help!

fixed by #65


txn: split out process module

Signed-off-by: Neil Shen<EMAIL_ADDRESS>
What have you changed? (mandatory)
This PR split a new module process from scheduler. It moves functions and methods that can be called in workers.
What are the type of the changes? (mandatory)

Improvement (non-breaking change which is an improvement to an existing feature)

How has this PR been tested? (mandatory)
Unit tests.
Does this PR affect documentation (docs/docs-cn) update? (mandatory)
No.
Does this PR affect tidb-ansible update? (mandatory)
No.

PTAL, thanks!
/run-all-tests


Created  a Contributor's Guideline

Have you read the Contributing Guidelines on Pull Requests?
(Write your answer here.)
Description
(I have Created a Contributor's Guideline as Per assigned work a few days ago.)
Here I have used images and screenshots for better understanding. This is going to help new geeks who are new to Open Source.
I have also used Grammarly.
Add Screenshots wherever possible.
Add the basic information about creating Issues, Pull Requests, and Branching.
Mention tutorials related to Git and Github to help them better understand.
Checklist

[X] I've read the contribution guidelines.
[X] I've checked the issue list before deciding what to submit.
[X] I've edited the README.md and link to my code.

Related Issues or Pull Requests
(#542 )

@siddharth25pandey
I think this is an important assignment as it was labeled. Because it is very useful for new geeks so, I would like you to review this.

@siddharth25pandey
@iamrajiv
I have made all the necessary changes. Please do the review.

@siddharth25pandey
please review this.

@siddharth25pandey
@HarshCasper
guys, please review this. as soon as possible.


Add package metadata generation task

Note: the TODOs are because this introduces some code duplication with protoc_tasks. This duplication will be removed in a follow-up PR that switches GrpcMetadataGen to use the new package metadata config yaml.
Corresponds to googleapis/toolkit#870

Jumped the gun on this -- need to update tests first.

Current coverage is 74.68% (diff: 93.54%)

Merging #130 into master will increase coverage by 0.40%

@@             master       #130   diff @@
==========================================
  Files            36         37     +1
  Lines          1326       1355    +29
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            985       1012    +27
- Misses          341        343     +2
  Partials          0          0

Powered by Codecov. Last update 43dde0a...2d2cf89


AngularFireDatabase does not retrieve data from firebase

I've been looking for a solution for this issue for a solid two weeks, and I'm yet to be able to find a solution.
At this point, I have tried all documentation available online. Please help. Link below:
https://stackoverflow.com/q/51126463/10018517

Getting same error. Also getting warnings in the project like below.
npm ERR! peer dep missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! peer dep missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! peer dep missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! peer dep missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! peer dep missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! peer dep missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>Is it the reason behind AngularFireDatabase not working.

Following this thread, mine also does not retrieve data anymore from firebase. It was working a week ago, now it's all effed up..

Please use the issue template so we can properly help you. Closing this one, but feel free to open a new one that follows the template.


make umbilicus function more readable

Hi, RICHI
When I try to understand umbilicus in grid_to_pointcloud.py, it confused me for a while because of x, y, z notation naming.
For example, the comment Create interpolation functions for x and y based on z and the next line Define new y values for interpolation, the "z" mentioned in the former is actually the "y" mentioned in the latter. So I changed the naming and comments slightly to make the whole function easier to read. And I didn't change the output behavior of the function, it's consistent with the previous result.
Just want to start from some small fixes and learn its logic along the way. Feel free to close this PR if you think the original expression is more appropriate, thanks!

Seems grid_empty use the same coordinate as well y, x, z -> x, y, z.


DoSOCS Install

When running the install script for DoSOCS I receive a message saying "Can not find glib-2.0 headers"

If you run across this error all you have to do is sudo apt-get install libglib2.0-dev to install glib-2.0

Thank you for the feedback and help with this error!
On Sun, Feb 14, 2016 at 1:06 PM, Ayy-ay-oos<EMAIL_ADDRESS>wrote:

If you run across this error all you have to do is sudo apt-get install
libglib2.0-dev to install glib-2.0
—
Reply to this email directly or view it on GitHub
https://github.com/kla587/Internet-System-Dev/issues/6#issuecomment-183954188
.


remove my goddamn deadname from the repo

if you want to go back into this, fine.
just please, remove my deadname from everything on this

btw, pip has the thing under an mpl, so use that

btw, pip has the thing under an mpl, so use that

ok, done. Anyway please do not go angry like this.

ok, done. Anyway please do not go angry like this.


Request: a5100 completely disable jpeg noise reduction

I was wondering if there is any way to completely disable noise reduction for jpegs?
even with NR set to low, from iso 400 onwards i can clearly see the NR pattern, and i don't really want to shoot RAW just for that. to clarify I'm not talking about the long exposure NR.
Has this maybe been done before?

I'm looking for a similar solution. The A6000 has the option to turn 'High ISO NR' off. If we could somehow modify the A5100 firmware, as the two cameras are very similar, that could be a solution. However I don't have any experience with this. If somebody has, please let us know :)

I'm looking for a similar solution. The A6000 has the option to turn 'High ISO NR' off. If we could somehow modify the A5100 firmware, as the two cameras are very similar, that could be a solution. However I don't have any experience with this. If somebody has, please let us know :)

@MetroidFusion Do you have access to the A6000?
Do a dump of the 'backup.bin' with and without the option enabled to see if 'we' can located the key that is used to hold this parameter. It may be that we can set the same key of the other cameras...


sum, min, max, mean, median, mode

i no longer believe these numeric-specific functions make sense


⚠ 366.ru has degraded performance

In 0dedf78, 366.ru (https://366.ru) experienced degraded performance:

HTTP code: 200
Response time: 6067 ms

Resolved: 366.ru performance has improved in 224b997.