Additional Cipher Suites

My bitbucket server uses TLS.  When attempting to get the sonar user from bitbucket, my sonar-runner encountered a handshake_failure.  I did a bit of research and determined that netty, used by StashClient via the AsyncHttpClient, did not include whatever cipher suite my bitbucket server wants to use.
I was able to overcome the problem by changing StashClient.createHttpClient() to pass all the cipher suites supported by the JVM to the DefaultAsyncHttpClientConfig.Builder.
`  AsyncHttpClient createHttpClient(String sonarQubeVersion){
try {
    SSLContext context = SSLContext.getDefault();
    SSLSocketFactory sf = context.getSocketFactory();
    String[] cipherSuites = sf.getSupportedCipherSuites();
    return new DefaultAsyncHttpClient(
            new DefaultAsyncHttpClientConfig.Builder().setEnabledCipherSuites(cipherSuites).setUserAgent(getUserAgent(sonarQubeVersion)).setHandshakeTimeout(30000).build()
            );
} catch (NoSuchAlgorithmException e) {
    throw new IllegalStateException(e);
}

}`
Please provide an option to supply additional cipher suites.

Can you try whether the default HTTPUrlConnection can connect to the bitbucket server?

Can you also try to set the standard https.cipherSuites system property.

I didn't try HTTPUrlConnection, but I did set the https.cipherSuites property.  That didn't help, at least in my environment.

Can you try: org.asynchttpclient.enabledCipherSuites?

Setting org.asynchttpclient.enabledCipherSuites to the netty-excluded cipher suite (TLS_ECDHE_RSA_WITH_3DES_EDE_CBC_SHA) works, at least in StashClientTest.testGetUser().
Is there any chance something could be done in the plugin so that users don't need to worry about setting the command line argument?

This cipher was probably disabled for a reason.
This seems to be a platform (configuration) and not a plugin issue.


CombiAdapter support

I faced some issue using Windows Combi drivers.
Windows version:

Linux:

usbcombi_win_ver.zip
combi_linux_ver.zip
As I do not have JohnC CombiAdapter, could you please Share USB logs ?
Install Wireshark (mark USBCAP to install),
Run Wireshark and start capture on interface where Combi is connected. (Best if only one device on bus)
My Combi impl is based on USB logs log.zip, (combi firmware1.3c burned to Mbed LPC) + analysis of decompiled  firmware.
I'm 99% sure it's should be written as packet, not stream with unknown data (probably EP address)

getinfo_combi.zip

your logs looks totally different then the ones I get with a JohnC combi.

dump_t8.zip

PC sent as packet of data,
Combi sending as fifo stream, I think, my solution can be reverted.

also with ifl 32


Unit testing

Under the TODOs you mention a unit test suite. Are you looking for unit tests beyond just assert statements? So using something like Python unittest module? Currently, there are only a few unit tests so before I add more I wanted to know which direction you'd like to go.

Оооол

Вот

Well I would recommend the built in Python unit test module.
I would also recommend using tox to try different Python versions and other things, like PEP8 compliance.
And last but not least setting some integration testing.
I can help with all these topics if you give me a go.

You certainly have a go on that one, and I agree with @AndreMiras on the route.

Great here is the first pull request that brings tox and Travis all together https://github.com/corpetty/py-etherscan-api/pull/29
This is the first of a long serie, in next ones I would like to:

drop pytest requirement and rely on core unittest module (you can still use the pytest command for testing locally if you like it)
integrate linter checks and fix lint errors
increase test coverage (maybe later)

you can drop pytest completely.  unittest works just fine for all.  I have no preference.

point 1 is fixed in https://github.com/corpetty/py-etherscan-api/issues/32 @corpetty can you merge so I tackle point 2


Refactor layer generation into a separate file

This PR separates the instantiation and ordering of layers into a separate file to improve modularity and separation of concerns.  This will simplify the workflow for anyone wishing to generate a standalone style.json.

buildLayers() returns layers containing labels that are blank until Label.localizeLayers() is called. To set a more reasonable initial default of the local-language name(s), replace the empty string in the following lines:
https://github.com/ZeLonewolf/openstreetmap-americana/blob/27a07671a318c114f1dfca7191febbb9543d3789/src/constants/label.js#L125 https://github.com/ZeLonewolf/openstreetmap-americana/blob/27a07671a318c114f1dfca7191febbb9543d3789/src/constants/label.js#L207
with ["get", "name"].

buildLayers() returns layers containing labels that are blank until Label.localizeLayers() is called. To set a more reasonable initial default of the local-language name(s), replace the empty string in the following lines:

This fix seems to break our unit tests.  Perhaps it would make more sense to localize the static file to English (and accordingly, we can generate "style-en.json")?

This fix seems to break our unit tests. Perhaps it would make more sense to localize the static file to English (and accordingly, we can generate "style-en.json")?

The script could take language codes as an option. Either way, we should update the unit tests whenever there’s new functionality.

Converting to draft until the legends PR is resolved

I'm going to start over on this project in order to work this in smaller, more manageable chunks.


[WiP] Use a Makefile include to avoid recalculating versions on every run.

Description
I think this is the trick that we need to avoid recalculating the versions every time.
Todos

[ ] Improve the write-out of the versions file (echos with nested $(shell...) seems a bit long-winded)
[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

Ugh, fails on semaphore due to several targets re-execing the makefile in go-build with a different versions file.

Does only removing the specific versions.*.mk associated with this invocation of make address the read-only filesystem error, e.g. rm -f versions.$(RELEASE_STREAM).mk.
Also it looks like the $info() statements are no longer reporting the version. I had a fix which @neiljerram and I were discussing which moved the version reporting from an $info which I think is evaluated in the first pass to the second pass. We might need to do something similar to get version reporting working if we use this include versions*.mk approach. See https://github.com/bcreane/calico/blob/dd54f28df6bfcbc926e848ed491555af6d9586df/calico_node/Makefile#L667

Not sure anyone should be reviewing this until I fix it up to run on semaphore...

Deleting all the versions files is intended.  It's a bit of a hack to make sure that we don't accumulate them after a few runs at different versions.  I can't just use a single filename because then make won't pick up when we switch branches.

@fasaxc is this still a WIP? What does this buy us? Faster rebuilds?

@caseydavenport it speeds up the build a lot if you don't have the yaml command on your path but it's currently broken for the targets that semaphore uses due to the way they call back into the makefile.

(It also illustrates a general technique for managing complex or awkward to express dependencies in make.  You can import a makefile and then remake that imported file, triggering a recalculation of the deps.)

To be honest, this PR scares me a bit because it feels super easy to build the wrong thing and not realize it, because the versions for a given release stream change fairly regularly. If we're careful about cleaning them up then I suppose it's OK, but still scary.
I've not personally had a problem with build speed in this repo, but I suppose we can continue with it if we can address the semaphore build stuff and we're confident we won't end up with releases with the wrong content in them... I'm still kinda hesitant...

@caseydavenport The intent was to rebuild the new versions makefile whenever the versions.yaml changes so that shouldn't be an issue.  I'm going to close this for now; the extra complexity in the targets semaphore uses makes it awkward to fix up.


Join forces with "jobcacher" plugin?

There is another caching plugin at https://github.com/jenkinsci/jobcacher-plugin which was created some years ago and got a new maintainer. It currently performs caching differently than this plugin here but for example also supports Freestyle jobs and storage on the Jenkins controller.
Maybe both plugins and functionality could be merged.
What do you think @j3t and @repolevedavaj?

I think the features in this plugin are more flexible:

cache keys (key and restoreKeys) and hashFiles() instead of a single file (cacheValidityDecidingFile) and a branch  (defaultBranch) as cache fallback
an LRU cache with periodic cleanup instead of deleting the cache completely once it reaches a certain size

@darxriggs thanks for initiating this discussion. I am definitely interested into this. @j3t is there a reason why you created your own plugin instead of using the jobcacher plugin?

If I remember correctly, the jobcacher plugin was not maintained actively and it was not solving our problems (no MinIO support, no fallback cache, no support for maven multi Module projects). It is not intuitive, complicated and less flexible.

I see your points. We had some similar issues, that's the reason why I adopted the plugin.
Are you interested in merging the plugins and join the Jenkins community? If yes, I can try to make a proposal how the merged plugin could look like.

In general I think it's a good idea, but I'm also a little bit sceptical how this could look like. Lets give it try. Maybe we should sharpen the expectations/requirements first.
How would the following example look like with the jobcacher plugin after the migration?
node {
    git (url: 'https://github.com/spring-projects/spring-petclinic', branch: 'main')
    cache (path: "$HOME/.m2/repository", key: "petclinic-${hashFiles('**/pom.xml')}", restoreKeys: ['petclinic-', 'maven-']) {
        sh './mvnw package'
    }
}

I would expect only one of the following outcomes:

cache is restored by key
cache is restored by one of the restoreKeys and is stored by key (assumption: inner step was successful)
cache gets stored by key (assumption: inner step was successful)

The test coverage should be at least on the same level as it is now.

From the usage perspective the new cache implementation could look like this:
node {
    git (url: 'https://github.com/spring-projects/spring-petclinic', branch: 'main')
    cache(maxCacheSize: 250, caches: [[$class: 'FooCache', path: "$HOME/.m2/repository", key: "petclinic-${hashFiles('**/pom.xml')}", restoreKeys: ['petclinic-', 'maven-']]]) {
        sh './mvnw package'
    }
}

So the already existing ArbitraryFileCache could be kept for backward compatibility.
It has to be decided, how the defaultBranch option should be treated. It is optional, so not a very big problem for backward compatibility. But it might better be converted to an ArbitraryFileCache option and not an overall option. If kept as is, in case only FooCache is used as cache, a validation could be implemented to tell the user that the defaultBranch option is not supported for this cache. This is just a rough sketch. The final API should be further improved.
With the possibility to have different caches, #9 (specific maven cache) could also be implemented.

When the assumption is that an API similar to the Github Action is the way to go anyway, then I would argue that maxCacheSize and caches should be optional as well. The plugin could then decide, depending on the provided configuration, which implementaion is used and it could also throw an error if the configuration is not valid. This way, we are still backward compatible and we also have a clean and user friendly API.

When the assumption is that an API similar to the Github Action is the way to go anyway, then I would argue that maxCacheSize and caches should be optional as well. The plugin could then decide, depending on the provided configuration, which implementaion is used and it could also throw an error if the configuration is not valid. This way, we are still backward compatible and we also have a clean and user friendly API.

Yes, I prefer this way too

I'd like to discuss the maximum cache size. We should decide if it should be configurable on the global Jenkins scope or per job.
It should take into consideration that Jenkins is used in small setups as well as big setups where individual persons might not have the permissions to configure all options.
I think there are two roles involved. An administrator that provides the cache and a developer that uses the cache. There is only a single cache storage available per Jenkins instance which is used for every job. The administrator has to provide the storage space (which might be "infinite" for certain cloud providers).
Therefore the administrator should be in control of the maximum available cache storage which means it has to be configured on the Jenkins scope. If it would be configurable per job, the administrator had no control of how much space would be used for the complete Jenkins instance and it could run out of space.
What's you opinion on this?

Another question: From the discussion so far my impression is that we want to go for the jobcacher plugin and not the jenkins-pipeline-cache plugin and migrate some functionality / API over there. For me this looks like the way to go because this plugin is already released, has some users and supports more job types. The downside is that we have to deal with the current API.
Is this what you have in mind too?

Regarding the cache size: I agree that an administrator has some interest to control the space caching requires, but I think there is a reason to configure it on a job level too, because if you do not restrict the size and you are increasing the cache size every build, the time needed to cache/restore will increase too.

I'd like to discuss the maximum cache size. We should decide if it should be configurable on the global Jenkins scope or per job.

We have to introduce a global configuration parameter for the jenkins-pipeline-cache anyway, so I guess this parameter could also be used by the existing job types to implement a global limit (which is defined by the admin). The limit on the job level, must also be respected (backward-compatibility) but it could be optional.

From the discussion so far my impression is that we want to go for the jobcacher plugin and not the jenkins-pipeline-cache plugin and migrate some functionality / API over there. For me this looks like the way to go because this plugin is already released, has some users and supports more job types. The downside is that we have to deal with the current API.

In my opinion we have to be backward-compatible so that both user groups are happy, which is the case when we make maxCacheSize and caches optional and let the plugin decide. What is you opinion about my my proposal (see https://github.com/j3t/jenkins-pipeline-cache-plugin/issues/10#issuecomment-1171609106)?

And regarding the repo to use in the future, I would go with the jobcacher too.

Fully agree

@j3t Yes, we can try go this way, but we have to see whether this can be done in a clean way. If not, we could introduce a completely new step/wrapper (e.g. arbitraryFileCache(...)) and reuse some parts of this in the old implementation. This would allow us to have a clear cut between the old and the new world and I think if we would like to introduce the Maven cache, we would like to use something like mavenCache(...) anyway.


Feature/new step it fails

This step addresses the requirement defined in #105

Pull Request Test Coverage Report for Build 351

7 of 7 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+2.2%) to 76.561%

Totals

Change from base Build 348:
2.2%

Covered Lines:
527

Relevant Lines:
664

💛  - Coveralls


Fix MaskingContainer not working in the nvidia graphic card.

It's the root cause of https://github.com/karaoke-dev/karaoke/issues/1318

The normal one should be like this:

But seems some graphic card cannot see the red box.
But masking still works.


CLOUDP-225849: add openapi-cli to repo

Ticket: CLOUDP-225849
Description:
Add openapi-cli to the repo

Splitting into multiple PRs


treesitter jsonc certificate error

Did you check docs and existing issues?

[X] I have read all the LazyVim docs
[X] I have searched the existing issues of LazyVim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
0.9.2
Operating system/version
Windows 10
Describe the bug
Everytime I open LazyVim I get this error:
Cloning into 'tree-sitter-jsonc'...
fatal: unable to access 'https://gitlab.com/WhyNotHugo/tree-sitter-jsonc.git/': SSL certificate problem: unable to get local issuer certificate

Several days in a row as we are hoping a temporary issue in GitLab but status seems to be ok. So, we guess if there's a workaround, maybe installing by hand.
Steps To Reproduce

install Lazyvim
Open Lazyvim
get error

Expected Behavior
no error is shown.
Repro
no changes to init.lua

Sounds like a problem with git really rather than LazyVim. Check out this thread.

could it be installed manually as seems that I can donwload the repository from the website?

No idea. Don't really use Windows myself. You should try the solutions on that thread, it also has a link to another one, that has other suggestions. For example try git config --global http.sslbackend schannel and see what happens.

this did the trick. Lazyvim gets unresponsive for almost 8 seconds and then is usable. Not cool but fine.


Limit Network Routing by Tenant by CZF and Country

As we take on new tenants we often negotiate to protect our service by limiting them to on net (CZF) or in country traffic so as not to allow them to impact our peering or connections.
We would like to be able to setup a tenant and specify whether they are limited to delivery by CZF or CZF plus country codes.  We would also like to tie in how to handle overflow traffic into these settings.
Tenant should not be able to override these settings.

Already done.


Remove @typescript-eslint/eslint-recommended

Based on v3.0.0 section at https://github.com/typescript-eslint/typescript-eslint/releases/tag/v3.0.0

Reopen due to

The issue only happens to client folder. Same code won't have issue for server folder.
Might related with create-react-app own ESLint.


bug(autocomplete): up arrow/down arrow causing a crash

Bug, feature request, or proposal:
There is an issue currently that occurs ONLY if the Angular application is in development mode.
If it is running in production mode this does not occur.
Error: Expression has changed after it was checked. Previous value: 'md-option-2'. Current value: 'null'.
at ExpressionChangedAfterItHasBeenCheckedError.ZoneAwareError (http://localhost:4201/vendor/zone.js/dist/zone.js:958:33)
at ExpressionChangedAfterItHasBeenCheckedError.BaseError [as constructor] (http://localhost:4201/vendor/@angular/core/bundles/core.umd.js:1174:20)
at new ExpressionChangedAfterItHasBeenCheckedError (http://localhost:4201/vendor/@angular/core/bundles/core.umd.js:6535:20)
at checkBinding (http://localhost:4201/vendor/@angular/core/bundles/core.umd.js:6710:23)
at Wrapper_MdAutocompleteTrigger.checkHost (/MdAutocompleteModule/MdAutocompleteTrigger/wrapper.ngfactory.js:51:7)
at View_AutocompleteDemo1.detectChangesInternal (/DemoAppModule/AutocompleteDemo/component.ngfactory.js:118:35)
at View_AutocompleteDemo1.AppView.detectChanges (http://localhost:4201/vendor/@angular/core/bundles/core.umd.js:12584:18)
at View_AutocompleteDemo1.DebugAppView.detectChanges (http://localhost:4201/vendor/@angular/core/bundles/core.umd.js:12731:48)
at ViewContainer.detectChangesInNestedViews (http://localhost:4201/vendor/@angular/core/bundles/core.umd.js:12842:41)
at CompiledTemplate.proxyViewClass.View_AutocompleteDemo0.detectChangesInternal (/DemoAppModule/AutocompleteDemo/component.ngfactory.js:950:15)
at CompiledTemplate.proxyViewClass.AppView.detectChanges (http://localhost:4201/vendor/@angular/core/bundles/core.umd.js:12584:18)
at CompiledTemplate.proxyViewClass.DebugAppView.detectChanges (http://localhost:4201/vendor/@angular/core/bundles/core.umd.js:12731:48)
at CompiledTemplate.proxyViewClass.AppView.internalDetectChanges (http://localhost:4201/vendor/@angular/core/bundles/core.umd.js:12569:22)
at CompiledTemplate.proxyViewClass.View_AutocompleteDemo_Host0.detectChangesInternal (/DemoAppModule/AutocompleteDemo/host.ngfactory.js:29:19)
at CompiledTemplate.proxyViewClass.AppView.detectChanges (http://localhost:4201/vendor/@angular/core/bundles/core.umd.js:12584:18)
What is the expected behavior?
When clicking into the autocomplete the panel opens up, when navigating via the up/down arrow keys, then when typing in it should not crash and panel should be filtered properly.
What is the current behavior?
Current behavior is when doing what I mentioned above the app will crash with the "Expression has changed after it has been checked". Like I mentioned this does not occur if the application is in production mode. EX: the https://material.angular.io/ site works just fine but when cloning material2 and running the demo-app this error occurs.
What are the steps to reproduce?
Providing a Plunker (or similar) is the best way to get the team to see your issue.
Plunker template: https://goo.gl/DlHd6U
PLUNKER:
http://plnkr.co/edit/hxbnHiZ7Ll8kp5rN4bhu?p=preview
To reproduce click the drop down, press down arrow on keyboard, Hit any letter such as "A" and crash will occur.
What is the use-case or motivation for changing an existing behavior?
To me this is a decently important issue as in development mode, so anyone that is not running in production the app can crash in this scenario which is pretty annoying, and after some research the Expression after value checked I have read shouldn't go unfixed. But maybe this isn't the case will see what you all think :)
Which versions of Angular, Material, OS, browsers are affected?
Tested on chrome.
Angular is whatever the plunker is.
Material: Using Angular Material (SHA 4ee2aef) from 3/6/2017, 7:02 PM
Is there anything else we should know?
I have dug into the root of this problem a little bit and have determined the root cause of the code is this:
https://github.com/angular/material2/blob/master/src/lib/autocomplete/autocomplete-trigger.ts#L57
It seems like with this commented out it all works, but guessing that line is needed for screen readers/accessibility. I have been unable to figure out a fix for this though, which is why I am just going to put up the issue and hopefully you guys will know.

Note to self: to repro, must hit down arrow then type. If you type then hit down arrow, nothing errors.


Respond case insensitively to A and SOA requests

Respond case-insensitively to A and SOA requests. Add corresponding tests.
This fixes the autocert feature with Let's Encrypt, because Let's Encrypt does a lookup for the A record with a deliberately mangled case.

Coverage remained the same at 93.676% when pulling 145dcee4a68a42cb90a72f110d43f4a23c9b6a75 on cure:respond-case-insensitively into 4f5fad0d321dae805b253c4aaaa092876e17b0fc on joohoi:master.

This is a regression that happened in #144
Thanks for fixing it and adding tests to prevent another regression in the future!


Ishucr7/radh

Explanation

Checklist

[ ] The PR title starts with "Fix #bugnum: ", followed by a short, clear summary of the changes.
[ ] The PR explanation includes the words "Fixes #bugnum: ...".
[ ] The linter/Karma presubmit checks have passed.

These should run automatically, but if not, you can manually trigger them locally using python scripts/pre_commit_linter.py and bash scripts/run_frontend_tests.sh.

[ ] The PR is made from a branch that's not called "develop".
[ ] The PR follows the style guide.
[ ] The PR is assigned to an appropriate reviewer.

If you're a new contributor, please ask on Gitter for someone to assign a reviewer.
If you're not sure who the appropriate reviewer is, please assign to the issue's "owner" -- see the "talk-to" label on the issue.

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@8dce68e). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop       #2   +/-   ##
==========================================
  Coverage           ?   46.28%
==========================================
  Files              ?      500
  Lines              ?    29257
  Branches           ?     4452
==========================================
  Hits               ?    13543
  Misses             ?    15714
  Partials           ?        0

Impacted Files
Coverage Δ

...domain/statistics/ExplorationIssueObjectFactory.js
91.66% <ø> (ø)

...main/question/EditableQuestionBackendApiService.js
74.41% <ø> (ø)

...tion/ContentIdsToAudioTranslationsObjectFactory.js
100% <ø> (ø)

...ad/components/OutcomeDestinationEditorDirective.js
1.85% <ø> (ø)

...ad/components/VersionDiffVisualizationDirective.js
0.78% <ø> (ø)

...tes/dev/head/components/QuestionCreationService.js
6.66% <ø> (ø)

...ts/top_navigation_bar/TopNavigationBarDirective.js
0.86% <ø> (ø)

...ev/head/domain/skill/MisconceptionObjectFactory.js
100% <ø> (ø)

...domain/suggestion/SuggestionThreadObjectFactory.js
96.77% <ø> (ø)

.../dev/head/components/SummaryListHeaderDirective.js
16.66% <ø> (ø)

... and 46 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8dce68e...45d8e8d. Read the comment docs.


Question, trying to connect server in application.

Hi!
I just installed Tindroid application (tindroid-arm64-v8a-debug.apk) and also installed server (
tinode-mysql.linux-amd64.tar.gz) at http://<IP_ADDRESS>:6060.
but when i set my server in application, it says that :"login failed: failed to connect to server after 3000ms"
I would appropriate if you can help.
Best Regards .

Go to login screen, select Settings (three dots in the upper right corner), take a screenshot, post it here.
Thanks.

it seems is my port setting issues, Thnx for replying.


Update resources template to include NSMI v2 codes for each Topic_Name for Alaska

Update resources template to include NSMI v2 codes for each Topic_Name for Alaska

ON HOLD - Requirements Spike

Is this for tagging only?


Script is missing that Find My is turned on

I ran the script on a device where I had Find My turned on for testing purposes, but it seems it's not picking this up.
The response is empty and says that the activation lock is disabled. This device is on Sonoma 14.3.1.
Any idea why it would be return this as such?
Thank you.

It checks the Find My status of the machines by checking the value in /Users/$currentUser/Library/Preferences/MobileMeAccounts.plist.  Either there was a delay in Apple writing that value to the plist or something else is up... I tested the script on a 14.3.1 machine and am getting the correct output.

Did it eventually correct itself, or is it still reporting that FindMy is not enabled even though it is?

hi [bvanpeski], there is a guide to use the code for newbies ?

hi [bvanpeski], there is a guide to use the code for newbies ?

Deploy it via your MDM and modify the User Input section of the script if you want to modify any of the default parameters. If you need help deploying scripts via your MDM solution, you can reach out to their support teams. There's also a great community of admins over at the MacAdmins Slack that are always happy to assist and answer questions as well.


perf: Use Cow in get_format_string in FFI_ArrowSchema

Which issue does this PR close?

N/A
Rationale for this change

We're using a lot of FFI in Comet and are looking for any performance wins we can get.
ffi_arrow_schema_try_from
                        time:   [497.34 ns 499.01 ns 500.56 ns]
                        change: [-11.996% -11.725% -11.490%] (p = 0.00 < 0.05)
                        Performance has improved.

What changes are included in this PR?

Use Cow in get_format_string to reduce some string allocations.
Are there any user-facing changes?

No

It fails msrv check.

Thanks @alamb and @viirya for the reviews


Make package compatible with AoT Compiler

This PR aims to make this package compatible with Angular 2 AoT compiler, without producing errors during this process.
The Module is now it's own file within the src/module directory.
The ng2-uploader.ts file is now an index of exports.

hello,Celso Bem dos Santos
can you help me to solve a ng2-uploader error.    when i webpack app , i got a browser_adapter.js:84 EXCEPTION: Error: Uncaught (in promise): No Directive annotation found on NgFileSelect   problem .   if you can help me  i  will very thanks.
thank you .
张艳娟
北京智城伟业科技有限公司
北京市海淀区上地十街辉煌国际大厦5号楼1706室
邮编：100085
电话：010 69946148
手机：15201546326
网站：www.icst.com.cn
From: Celso Bem dos Santos
Date: 2016-11-24 23:22
To: jkuri/ng2-uploader
Subject: [jkuri/ng2-uploader] Make package compatible with AoT Compiler (#136)
This PR aims to make this package compatible with Angular 2 AoT compiler, without producing errors during this process.
The Module is now it's own file within the src/module directory.
The ng2-uploader.ts file is now an index of exports.
You can view, comment on, or merge this pull request online at:
https://github.com/jkuri/ng2-uploader/pull/136
Commit Summary
Make package compatible with AoT compilation
1.5.4
File Changes
M .gitignore (7)
M ng2-uploader.ts (23)
M package.json (25)
M scripts/clean.sh (1)
A src/module/ng2-uploader.module.ts (20)
M tsconfig.json (34)
Patch Links:
https://github.com/jkuri/ng2-uploader/pull/136.patch
https://github.com/jkuri/ng2-uploader/pull/136.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Hello @isct ,
I'm sorry but I'm not familiar with webpack so I don't think I can help you. But check that you have the latest versions of everything installed and if that still presents errors.

thank you
张艳娟
北京智城伟业科技有限公司
北京市海淀区上地十街辉煌国际大厦5号楼1706室
邮编：100085
电话：010 69946148
手机：15201546326
网站：www.icst.com.cn
From: Celso Bem dos Santos
Date: 2016-11-28 23:04
To: jkuri/ng2-uploader
CC: isct; Mention
Subject: Re: [jkuri/ng2-uploader] Make package compatible with AoT Compiler (#136)
Hello @isct ,
I'm sorry but I'm not familiar with webpack so I don't think I can help you. But check that you have the latest versions of everything installed and if that still presents errors.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@jkuri can somebody take care of this PR?

@CelsoSantos can you rebase please?

Hello @jkuri
I ended up creating a new PR since I had some local issues with this one, so I re-forked and applied the modifications.
The new PR is https://github.com/jkuri/ng2-uploader/pull/148
Best Regards

Yet another PR. This time, #149


[SPARK-41742][SPARK-41745][CONNECT] Reenable doc tests and add missing column alias to count()

What changes were proposed in this pull request?
Cleanup previously disabled test cases. In addition, add a column alias to the "count()" method without arguments for compatibility with PySpark.
Why are the changes needed?
Compatibility
Does this PR introduce any user-facing change?
No
How was this patch tested?
Reenabling doc tests.

Sure, I'll revise. Though I'd say that the "contribution" is rather checking that the previously disabled tests actually pass.

Merged to master.


Headers.delitem signature incompatible with ImmutableHeadersMixin.delitem

Mypy recently introduced stricter checks for function signature compatibility (https://github.com/python/mypy/pull/2521). Since werkzeug is included in the typeshed library with typing stubs, it found an issue with the signatures of __delitem__() in datastructure.py. Consider:
class Headers(Mapping):
  def __delitem__(self, key, _index_operation=True): ...

vs.
class ImmutableHeadersMixin(object):
  def __delitem__(self, key): ...

class EnvironHeaders(ImmutableHeadersMixin, Headers): ...

Basically, it's invalid to replace __delitem__() accepting an optional kwarg with an implementation that doesn't.

@ambv Am I correct that changing the signature of ImmutableHeadersMixin.__delitem__ to accept arbitrary args would work?

Yes, that would work. __delitem__() in the immutable mixin is supposed to be a no-op so it's reasonable for it to accept (and ignore) arbitrary arguments.
To rephrase the problem for future reference:

When you're using Headers directly, you can pass _index_operation=True to __delitem__(). Remember, the "user" of this functionality might be the class itself.
EnvironHeaders is a subclass of Headers so it should accept _index_operation=True, too, but it doesn't because ImmutableHeadersMixin comes in first in the MRO.

It sounds to me that the correct fix would be to rename Headers.__delitem__ and add a new __delitem__ implementation that just calls the renamed method with default arguments.

Going with the simple solution by just adding **kwargs to ImmutableHeadersMixin.__delitem__.


bug(chat): url encoding not displaying properly

STR:

go to chat
send this

[GitHub](https://www.github.com) is a code hosting platform for version control and collaboration.

on the chat after sending is displaying a > after the url

macOS, m1

STR:

go to chat
send this

[GitHub](https://www.github.com) is a code hosting platform for version control and collaboration.

on the chat after sending is displaying a > after the url

macOS, m1

irrc i talked to matt about this and we wanted to not support this as this can make people "hide" urls.

i only meant this that extra > after the url


create btcapeconics

file transmission:
inscription.json
meta.json

i really don´t understand what the issue is. The files are containing exactly the description. Meta file was eben build with the editor that is proposed


Add advanced Samba options to all.yml for Windows Server environments

What this PR does / why we need it:
REF: https://github.com/davestephens/ansible-nas/pull/184
Though these values already default to true from https://github.com/bertvv/ansible-role-samba this makes it easier to set them to false for users with PDCs.
Which issue (if any) this PR fixes:
Fixes # n/a
Any other useful info:
I have Windows servers too!

I'm not keen on having a bunch of options 99% of people will never change in the config - especially above the shares section - which 99% of people will change :-) Also, the link provided is to the root of the Samba docs, there is a ton of stuff there that isn't configurable through Ansible-NAS.
I think it would be better to link to the bertvv.samba Ansible role Github page, along with a note of something like "Further Samba configuration is possible through Ansible-NAS, see  for more info". This would probably be best placed at the bottom of the Samba section too.
It may not be immediately obvious (especially now it has grown so big), but group_vars/all.yml is roughly ordered in the likelihood that people will need to change stuff...

So I should move it towards the bottom of all.yml?  No problem, I can do that. Personally I think best to keep all related items grouped together. Let me know if you change your mind.
Will change link.
Prepare for my "Return rage: Rage against the NAS"

No, towards the bottom of the samba section. But only a link and a note,
not all the options - because most won't use them (as you say).
On Tue, 21 Apr 2020, 05:51 bcurran3<EMAIL_ADDRESS>wrote:

So I should move it towards the bottom of all.yml? No problem, I can do
that. Personally I think best to keep all related items grouped together.
Let me know if you change your mind.
Will change link.
Prepare for my "Return rage: Rage against the NAS"
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/davestephens/ansible-nas/pull/252#issuecomment-616950537,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFDGMHXYY3YRWOM4W74BMTRNUQ4JANCNFSM4MJGSH7A
.

Changes made. I'm hoping they are amicable to all.

With the move to roles and nas.yml, users can add these settings straight into their config if they want to, there's no need for placeholders in all.yml any more.


Reject lakeFS output committer

Not possible to attain reasonable performance on lakeFS KV.

Docs change, we don't need tests to pass where we're going...


Adding some statistic evaluation to check how bad our luck is

Codecov Report

Merging #11 (deb18ef) into master (ff709f2) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master       #11   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            6         6
  Lines          108       118   +10
=========================================
+ Hits           108       118   +10

Impacted Files
Coverage Δ

wuerfelbecher/commands.py
100.00% <100.00%> (ø)

wuerfelbecher/statistics.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ff709f2...deb18ef. Read the comment docs.


Remove hack in fields with multiselect

to_dict() will always return a dict with its value being a dictionary, so we unwrap the dicts of length 1 which would be for non-multiselect fields.

Coverage increased (+0.02%) to 81.878% when pulling fd62e627fa25863cb80b03324ce3619df14290df on georgeke:not-grouped-by-2 into 89f9efd3a3de9316d5163ccc68d74d8802b90406 on airbnb:master.


Minor compile fix for VS Warning C4819

Some Unicode characters in ANSI source file will cause C4819 on  a system with a codepage that cannot represent all characters in the file. like Chinese (CP936).

Thanks for the fix!
PS: you should set your code editor to 'tabs as spaces' :)

Thank you :)


NTI fails to detect typing error

The following incorrect program emits no warning with --new_type_inf --warning_level VERBOSE
(compiler version v20160911):
/** @constructor */ function A() {}
/** @constructor */ function B() {}

/**
 * @param {!Array<A>} as
 * @return {A}
 */
function compiles(as) {
    let a = new A();
    let b = new B();
    as.filter((it) => it !== b);
    return b;
}

The function contains two errors: comparing it (of type A) to b (of type B), and returning b where an A is expected. OTI correctly reports both errors (with --warning_level VERBOSE):
src/main.js:11: WARNING - condition always evaluates to true
left : (A|null)
right: B
    as.filter((it) => it !== b);
                      ^

src/main.js:12: WARNING - inconsistent return type
found   : B
required: (A|null)
    return b;
           ^

The problem seems to be the invalid filter callback. If I fix that error, NTI correctly reports the second error. That is:
/** @constructor */ function A() {}
/** @constructor */ function B() {}

/**
 * @param {!Array<A>} as
 * @return {A}
 */
function doesNotCompile(as) {
    let a = new A();
    let b = new B();
    as.filter((it) => it !== a); // replaced b with a
    return b;
}

correctly warns when compiled with --new_type_inf --warning_level VERBOSE:
src/main.js:12: WARNING - Returned type does not match declared return type.
Expected : A|null
Found    : B

    return b;
    ^

Note to self: compare to the following code, that doesn't use generics and finds the correct warnings. Maybe the issue here is generics related?
https://closure-compiler-debugger.appspot.com/#input0%3D%252F**%2520%2540constructor%2520*%252F%2520function%2520A()%2520%257B%257D%250A%252F**%2520%2540constructor%2520*%252F%2520function%2520B()%2520%257B%257D%250A%250A%252F**%250A%2520*%2520%2540return%2520%257B!A%257D%250A%2520*%252F%250Afunction%2520doesNotCompile()%2520%257B%250A%2520%2520var%2520a%2520%253D%2520new%2520A()%253B%250A%2520%2520var%2520b%2520%253D%2520new%2520B()%253B%250A%2520%2520ext(function%2520g()%2520%257B%2520takesA(b)%253B%2520%257D)%253B%250A%2520%2520return%2520b%253B%250A%257D%250A%250Afunction%2520takesA(%252F**%2520!A%2520*%252F%2520x)%2520%257B%257D%26input1%26conformanceConfig%26externs%3Dvar%2520ext%253B%26refasterjs-template%26includeDefaultExterns%3D1%26CHECK_SYMBOLS%3D1%26CHECK_TYPES%3D1%26CHECK_TYPES_NEW_INFERENCE%3D1%26CLOSURE_PASS%3D1%26LANG_IN_IS_ES6%3D1%26MISSING_PROPERTIES%3D1%26PRESERVE_TYPE_ANNOTATIONS%3D1%26PRETTY_PRINT%3D1%26TRANSPILE%3D1


When will the latest versions be pushed to MS Store?

Okay. So the settings not persisting issue is due to the version number. I have 2.0.6 but apparently it is fixed in 2.0.7 and 2.0.8. However, MS Store only provides versions up to 2.0.6. When should I expect MS Store to automatically update to 2.0.8?
Additionally, I would like to share that I do not have the folder
C:\Users\<user name>\AppData\Local\Percentage.App\
in my computer under that path (I only have one User so there is no mixup).
Thanks. Great app. However, I do not enjoy it popping up on startup every day.

@scrutiny6239 MS Store release can be delayed depending on how fast MS can finish the certification process of each version. I submitted 2.0.8 to the store the same time I released it on GitHub, however, 2.0.8 is still under certification at the time of writing.
If you don't have C:\Users\<user name>\AppData\Local\Percentage.App\ I suspect that either your local security settings or security app like antivirus is blocking the app from being able to access the location to create the user settings file.
This app has always been using the exact same way of saving and loading the user settings since its first release years ago. Were you having the same issue before with version 1.x?

Closing this issue because the actual problem is duplicated with #4.


Update dependencies

Update php-cs-fixer to ^2.0 to allow running on php > 7.2

Do composer update to update phpseclib to resolve security alert

Update README (vendor.phar approach no longer used)

Update version number in main plugin file

[X] Version number has been bumped

@brent-dxw Would it be possible to split the various types of changes here (e.g. updating PHP CS fixer, running the fixer,  updating other dependencies etc.) out into separate commits? That would make it easier to see what's happening where.

@RobjS that's done...

Thanks @brent-dxw, that's miles better!
I feel like the PR template for this repo is actually a bit misleading - we probably don't want to be bumping the version number each time. Instead, we should add a changelog (as per https://github.com/dxw/tech-team-rfcs/blob/main/rfc-019-use-changelogs-to-track-changes.md), then do PRs for version bumps.
We should also migrate the CI to GitHub actions rather than Travis.
Up to you whether you want to tackle some of that in this PR - if not, I'd be happy to approve it without the version bump, and we could deal with those other issues in separate PRs.

Thanks @RobjS I reckon let's get this merged to sort out the security issue


Unicode support issues

Issue Report Checklist

[ ] Searched the issues page for similar reports
[ ] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[ ] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[ ] Could not reproduce inside jupyter qtconsole (if console-related)
[ ] Tried basic troubleshooting (if a bug/error)

[ ] Restarted Spyder
[ ] Reset preferences with spyder --reset
[ ] Reinstalled the latest version of Anaconda
[ ] Tried the other applicable steps from the Troubleshooting Guide

[ ] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
What steps reproduce the problem?

Paste in the editor

𨭎 = 10
𨭎𨭎𨭎𨭎 = 100000000
a = "𨭎"

What is the expected output? What do you see instead?
Paste Traceback/Error Below (if applicable)

PASTE TRACEBACK HERE

Versions

Spyder version:
Python version:
Qt version:
PyQt version:
Operating System name/version:

Dependencies

PASTE DEPENDENCIES HERE

Hi @impact27!
On my end I do see the chinese character,

I'm not sure if this is a Windows/Linux problem.


[Bug]: lib has issues with fast refresh on expo app

https://github.com/wcastand/bug-true-bottom--androidsheet
after looking into, it looks like it only ignore the props after a fast refresh.
if i fully reload the app, it seems to take into account the values.
Seems to be an issue on both ios and android where changing values/props makes the app behave weirdly or ignore the new values and i have to fully reload the app.
using default expo starter with prebuild to add the lib.

Hey @wcastand.. what prop are you trying to modify on-the-fly specifically? I don't see issue on the provided example.


Ability to get path of queried nodes

Hi,
I have a use-case where I would need to get the path (in some format) and value of queried nodes, this doesn't seem possible at the moment. How much work do you think this would require? I might take a stab at this eventually. 😄

https://github.com/hiltontj/serde_json_path/pull/77

Hey @FruitieX - thanks for opening the issue. I see the PR and may take a look at that but think it would be good to discuss here a bit first.
This concept is documented in the IETF standard, see Normalized Paths. There is a C# IETF JsonPath implementation that has them built into the query evaluation mechanism, and which you can see in their demo environment.
Unfortunately, serde_json_path, as you have discovered, does not cover this portion of the standard. I have some ideas though, which is why I want to discuss first, and which I will try to get down here when I get a minute or two. Ultimately, my time is a bit tight to implement this these days, so if it is something that you could take the lead on then that would be awesome!

I would start by defining a type to represent a normalized path, e.g.,
struct NormalizedPath<'a>(Vec<PathElement<'a>>);

enum PathElement<'a> {
    String(&'a str), // for object keys
    Index(usize), // for list indices
}

This representation should allow you to construct it as you recurse down the JSON structure using borrowed/copied data from the JSON value being queried, vs. having to clone everything into an owned string. That is, 'a here is the lifetime of the JSON value being queried - string keys from that JSON value should have the same lifetime. Array indices are usize which is copy, so no need to worry about lifetimes for those.

Thanks for the quick response, and yeah, that sounds more reasonable than my current implementation. I will try to adapt it into using borrowed data instead.

@FruitieX - I may have some time to implement this. I am curious, based on your description and your PR, you are looking for both the path and the node to be returned, e.g.,
impl JsonPath {
    fn query_path<'a>(&self, value: &'a Value) -> Vec<(NormalizedPath<'a>, &'a Value)> {
        /* ... */
    }
}

Is it important that both the node, i.e., the &'a Value is also returned, or would you just need the path. I am trying to decide if it is better to do the above, or just return the paths, e.g.,
impl JsonPath {
    fn query_path<'a>(&self, value: &'a Value) -> Vec<NormalizedPath<'a>> {
        /* ... */
    }
}

It would be helpful to understand your use-case a bit more and why you need both, if that is the case.

My use-case is a bit unusual, but basically I have a JsonPath with wildcards and I need to know which keys matched the wildcards for each matched value. While less convenient, returning just the paths would also work, as I could then use the path to build a JSON pointer and query the underlying value with serde_json::value::Value::pointer()
The long(er) version is:
I'm writing a program that controls my home automation devices. I'm allowing users (=myself 😃) to configure the behavior of their devices using a simple scripting language called evalexpr. It doesn't have JSON support, but instead I can use assignments like:
devices.hue.office.state.power = true;
devices.hue.office.state.color.r = 255;
devices.hue.office.state.color.g = 128;
devices.hue.office.state.color.b = 0;

devices.lifx.bedroom.state.power = true;
devices.lifx.bedroom.state.brightness = 0.5;

I then build a serde_json::Value from the resulting variable context, and start querying it with serde_json_path. For example to get the state struct of each device I can use the following query:
$.devices.*.*.state

In addition to the state, I also need to know the vendor ID and device ID from the path, so that I know which devices to send the state updates to.
There are probably better ways of doing this, but this way was pretty convenient to implement since my state structs already implemented Serialize/Deserialize due to them being directly used in a REST API / web frontend.

Thank you for the write-up @FruitieX!
To be clear, my understanding is that you use the JSONPath to query for the state objects, but then you require the paths themselves to extract the vendor and device IDs, e.g., lifx/bedroom and hue/office, that are associated with the given state objects provided by the query.

Given that Normalized Paths are a part of the JSONPath spec, and this crate is meant to support that spec,  I feel obliged to incorporate the feature. Since this could affect the API and underlying query execution in a substantial way, I want to determine if there is significant overhead to having the underlying query logic produce Vec<(NormalizedPath<'a>, &'a Value)> vs. what it is currently doing, i.e., Vec<&'a Value>. So, I can't necessarily promise quick delivery, but I have already started putting something together in #78.

Sounds good, thanks! Also no hurry from my side, I'm using my inefficient
fork for now 😁
On Fri, 26 Jan 2024, 15.28 Trevor Hilton, @.***> wrote:

Thank you for the write-up @FruitieX https://github.com/FruitieX!
To be clear, my understanding is that you use the JSONPath to query for
the state objects, but then you require the paths themselves to extract the
vendor and device IDs, e.g., lifx/bedroom and hue/office, that are
associated with the given state objects provided by the query.
Given that Normalized Paths are a part of the JSONPath spec, and this
crate is meant to support that spec, I feel obliged to incorporate the
feature. Since this could affect the API and underlying query execution in
a substantial way, I want to determine if there is significant overhead to
having the underlying query logic produce Vec<(NormalizedPath<'a>, &'a
Value)> vs. what it is currently doing, i.e., Vec<&'a Value>. So, I can't
necessarily promise quick delivery, but I have already started putting
something together in #78
https://github.com/hiltontj/serde_json_path/pull/78.
—
Reply to this email directly, view it on GitHub
https://github.com/hiltontj/serde_json_path/issues/76#issuecomment-1912067817,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA4AYLTWUPQBAYDVVTFWNTLYQOVOFAVCNFSM6AAAAABB6FH3LOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJSGA3DOOBRG4
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hey @FruitieX - I have made some solid headway in #78. With it, you would be able to do something like:
struct Device {
    vendor_id: String,
    device_id: String,
    state: Value,
}

let config = json!({ /* JSON of home devices configuration */});
let path = JsonPath::parse("$..state")?; // use `..` operator to get all nested `state` nodes
let devices: Vec<Device> = path
    .query_located(&config) // use new `query_located` method
    .iter() // iterate over `LocatedNode`s
    .map(|q| { // map them into the `Device` type (or whatever)
        let loc = q.location(); // get the location, i.e., full normalized path to `state` node
        // extract elements of interest from path:
        let vendor_id = loc.get(1).to_string();
        let device_id = loc.get(2).to_string();
        // the state object itself is the node that was queried for:
        let state = q.node().to_owned();
        Device { vendor_id, device_id, state }
    })
    .collect();

Excellent, this should cover all my needs, thanks!

@FruitieX - just released v0.6.5. Thank you again for raising the issue and for the helpful discussion! 🍻


unable to close dropdown

Dropdown menu is always open and i cant toggle it except when i visit my login page.
Here is my website: https://ns2.skooog.se/Special:UserLogin

Issue is fixed. I forgot i had a Mediawiki:common.css page that had a few settings that destroyed my navbar.


chore: fix typo

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

Project coverage is 100.00%. Comparing base (186950a) to head (e85b6c3).

Additional details and impacted files
@@            Coverage Diff            @@
##            master      #276   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           42        42
  Lines         3596      3596
=========================================
  Hits          3596      3596

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


How to connect two neural networks in tensorflow using graphs?

Im stetting up a system capable of recognizing faces using python and tensorflow.
I want to recognize two things. "Person" for unknown people and "Name of the person" with known people (Previously train the tag in the model).
For that reason I need to implement 2 neural networks.
The first one is going to recognize faces among all the objects (example chair, desk, ..) and the second one is going to classify that frame with the face and check for the label taking in consideration all the known faces for the model. (previously trained)
so, in order to solved something like this I knows that I need two neural networks and the output of the first one is going to be the input of the second. But I just dont known how can I connect this two neural networks using the graph files. (Created in the training process)
Right now I can train my model, get the graph file and use that in the recognition process. But I cant mixed the two neural networks or how to add the output of the first model into the input of the second model
with detection_graph.as_default():
    with tf.Session(graph=detection_graph) as sess:
        while True:
            # Read frame from camera
            ret, image_np = cap.read()
            # Expand dimensions since the model expects images to have shape: [1, None, None, 3]
            image_np_expanded = np.expand_dims(image_np, axis=0)
            # Extract image tensor
            image_tensor = detection_graph.get_tensor_by_name('image_tensor:0')
            # Extract detection boxes
            boxes = detection_graph.get_tensor_by_name('detection_boxes:0')
            # Extract detection scores
            scores = detection_graph.get_tensor_by_name('detection_scores:0')
            # Extract detection classes
            classes = detection_graph.get_tensor_by_name('detection_classes:0')
            # Extract number of detectionsd
            num_detections = detection_graph.get_tensor_by_name(
                'num_detections:0')
            # Actual detection.
            (boxes, scores, classes, num_detections) = sess.run(
                [boxes, scores, classes, num_detections],
                feed_dict={image_tensor: image_np_expanded})
            # Visualization of the results of a detection.
            vis_util.visualize_boxes_and_labels_on_image_array(
                image_np,
                np.squeeze(boxes),
                np.squeeze(classes).astype(np.int32),
                np.squeeze(scores),
                category_index,
                use_normalized_coordinates=True,
                line_thickness=8)

            # Display output
            cv2.imshow('object detection', cv2.resize(image_np, (800, 600)))

            if cv2.waitKey(25) & 0xFF == ord('q'):
                cv2.destroyAllWindows()
                break

detection_graph is the model (reference to the graph file in memory)
The previous code works fine running one model (graph file) But I want to integrate a second neural network. (add the output of the first in the input of the second one). Any idea how can I do something like this? I dont see anything like this in the documentation

@JulianaCP Have a look on image captioning using tensorflow. This may help you. Thanks!

@JulianaCP This is not Build/Installation or Bug/Performance issue. Please post this kind of support questions at Stackoverflow. There is a big community to support and learn from your questions. GitHub is mainly for addressing bugs in installation and performance. Thanks!


Honor interdocument xrefs in reduced document

An AsciiDoc feature that is often used in documents that make use of includes is the interdocument xref. If the xref points to a document which is included into the parent document, Asciidoctor will automatically convert it to an internal xref. When the reduced document is written to a file and converted by a separate process, the information that activates this feature is lost. The reduced document should keep track of the list of docnames of the includes that have been reduced (stored in the Document#catalog[:includes] table), then import them into the document catalog when converting the reduced document.

This could work adding the following metadata line to the end of the reduced document:
//# includes=chapter-a,chapter-b

Here's an extension that can be used both when reducing the document and converting it that implements this idea:
Asciidoctor::Extensions.register do
  tree_processor do
    process do |doc|
      if doc.extensions.groups[:reducer]
        includes = doc.catalog[:includes].select {|_, v| v }.keys
        doc.source_lines.concat ['', %(//# includes=#{includes.join ','})] unless includes.empty?
        nil
      elsif (last_line = doc.source_lines[-1]) && (last_line.start_with? '//# includes=')
        includes = ((last_line.slice 13, last_line.length).split ',').map {|it| [it, true] }.to_h
        doc.catalog[:includes].update includes
        nil
      end
    end
  end
end

Here's how it's used:
asciidoctor-reducer -r ./ext-include-mapper.rb -o doc-reduced.adoc doc.adoc
asciidoctor -r ./ext-include-mapper.rb doc-reduced.adoc

For now, I'm thinking about bundling this extension and marking it as experimental. That allows people who need it to use it, but gives us time to think about the best way to approach direct integration.


Deadlinks/404s in articles

Article issue
Article - Designing for Scale contains links to other articles that no longer work.
Please assign, ill review.

Good catch, and thanks for taking on the fix.


Docs error:  Names RemoteEvent as 'Simulation' when it needs to be 'Simulate'

https://1axen.github.io/Secure-Cast/Getting Started/Simulation/

Resolved in cd5b6aa


New Error

Been working fine for a year or so but now Getting this error message: Cannot parse response to an image.
iOS 14.4.1

Maybe something is wrong with the downloaded homebridge logo that is saved in your iCloud. What if you manually delete it and execute the script again?


Error state should not be cleared with required flag errors just because a value was added

There may be a larger reason for this to have an error state. Since we don't control when the error flag is added we probably shouldn't assume that simply defining a value removes the error.

This UX is consistent with how auro-input works, https://auro.alaskaair.com/components/auro/input
When an error is triggered, a user can come back and interact with that UI again and the error is removed. It is after this interaction that the state of the data will be again evaluated by either the client with code created by the consumer or with a page return post submission.


Add parameter validation for the DHM module

Summary: Based on #1643, this PR implements parameter validation for the DHM module.

@mpg As mentioned in the bignum PR, I don't think we should make parameter validation too fine-grained, as it is unavoidably only an approximation to the preconditions. I'd therefore prefer to stick to the primitive non-NULL check. What do you think?

@AndrzejKurek @mpg Please re-review.

@hanno-arm - Can you please rebase? Thanks

Rebased on top of #1643.

PR has been reviewed, approved, passes CI, but needs rebasing!
PR #1643 has now been merged, so removing the 'needs: preceding PR' label.

@sbutcher-arm I don't think this branch needs rebasing, it has no conflicts and was based on a recent enough version to pass CI.

@mpg - Agreed! I just saw duplicate commits from #1643, and assumed it needed it.
Jenkins CI passes in all but the BSD timing test, reviewed and approved, so labelling as 'ready to merge'.


Configure NGINX Ingress Controller to expose TCP and UDP services.

#1149
Adds the ingress controller arguments and ConfigMaps necessary to support ingress of TCP and UDP. Individual services can then be proxied by patching the Ingress Controller Service (Daemonset in MicroK8s) and the relevant ConfigMap.
Configuration details at:
https://kubernetes.github.io/ingress-nginx/user-guide/exposing-tcp-udp-services/
and further examples at:
https://minikube.sigs.k8s.io/docs/tutorials/nginx_tcp_udp_ingress/
Documentation specific to MicroK8s
The configuration in MicroK8s is slightly different from the above links. This is shown in the following example for exposing Redis with the Ingress Controller.
The Redis Service in namespace "pi"
$ k -n pi get service
    NAME    TYPE        CLUSTER-IP       EXTERNAL-IP   PORT(S)    AGE
    redis   ClusterIP   <IP_ADDRESS>   <none>        6379/TCP   10h

Expose the ports in the Ingress Controller
# redis-patch-ingress-daemonset.yaml
---
apiVersion: apps/v1
kind: DaemonSet
metadata:
  name: nginx-ingress-microk8s-controller
spec:
  selector:
    matchLabels:
      name: nginx-ingress-microk8s
  template:
    metadata:
      labels:
        name: nginx-ingress-microk8s
    spec:
      containers:
      - name: nginx-ingress-microk8s
        ports:
        - containerPort: 6379
          hostPort: 6379
          protocol: TCP

Apply with the command:
$ k -n ingress patch daemonset nginx-ingress-microk8s-controller --patch "$(cat redis-patch-ingress-daemonset.yaml)"
Configure the Redis service in the Ingress TCP ConfigMap
#  redis-patch-ingress-tcp-configmap.yaml
---
apiVersion: v1
kind: ConfigMap
metadata:
  name: nginx-ingress-tcp-microk8s-conf
data:
  6379: "pi/redis:6379"

Apply with the command:
$ k -n ingress patch configmap nginx-ingress-tcp-microk8s-conf --patch "$(cat redis-patch-ingress-tcp-configmap.yaml)"

Please let me know if you need documentation written, and where to put it.

Thank you for this PR @trulede
With this PR we are essentially adding placeholders that users need to edit to expose their services. Indeed we will need to pair this PR with documentation because further user specific steps are needed.
Would you be able to create a page along the lines of [1] and link it to the ingress addon in [2]. The documentation pages are in [3]. @evilnick may be able to guide you should you run into any trouble.
[1] https://microk8s.io/docs/addon-dns
[2] https://microk8s.io/docs/addons#list
[3] https://github.com/canonical-web-and-design/microk8s.io/tree/master/docs

OK, I created that documentation and opened a PR.


Canot from dataloader.bigquery_pypi import LLMDataset in file pl_data.py in line 11

Hello, I am stuck at Step 2: running training.
Could you please let me know if the function dataloader.bigquery_pypi import LLMDataset is from a third-party library of dataloader or a specific implementation within this project? My local version of dataloader is 2.0, and it cannot be imported. Moreover, there is no dataloader file within the project, which makes it impossible to import and run pl_data. Am I missing the implementation of LLMDataset?
Expecting a response from the author. Thx

Hi,
Thank you for raising the issue. We missed a file when preparing the code. I will submit a pr to fix that. For now, you can create a file dataloader/bigquery_pypi.py and put the following content in it:
from torch.utils.data import Dataset

class LLMDataset(Dataset):
    def __init__(self, data):
        super(LLMDataset, self).__init__()
        self.data = data

    def __len__(self):
        return len(self.data)

    def __getitem__(self, ind):
        # indexing the chunked data directly
        source_tokens = torch.tensor(self.data[ind]['token_ids'])
        return {"input_ids": source_tokens}


Extend sema.commons.store with read-only graph access

Implement a read-only or cloned access to the graph and expose all triples.

Links to #19

Unclear what is concretely implied here?
What use case will this cover?


Branch update user guide to include mail functionality

Closes #106

Codecov Report

Merging #107 (bc84fea) into master (84988b2) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #107   +/-   ##
=========================================
  Coverage     66.62%   66.62%
  Complexity      570      570
=========================================
  Files           102      102
  Lines          1801     1801
  Branches        204      204
=========================================
  Hits           1200     1200
  Misses          534      534
  Partials         67       67

Impacted Files
Coverage Δ

...a/seedu/address/logic/commands/MailAllCommand.java
0.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 84988b2...bc84fea. Read the comment docs.

LGTM from my side:) The second reviewer can go ahead and merge!

Thanks for fixing them!


Add a way to install zipkin through homebrew on macos

Feature
I would like to install zipkin using homebrew using brew install zipkin
Rationale
It would be helpful for other macos user to have a way to experiment quickly and easily with zipkin while keeping it up to date with brew
Example Scenario
I got a mac, I don't have docker, I want to try zipkin out, I would like to have a way to test it without compiling.
Prior Art

Links to prior art
More links

sounds very sensible

for now you can try the quick start https://zipkin.io/pages/quickstart.html
$ curl -sSL https://zipkin.io/quickstart.sh | bash -s
$ java -jar zipkin.jar

One question for the home-brew thing, is if this should be the full zipkin build (includes kafka etc) or slim (smaller and a bit quicker to startup, but no messaging support)

Kafka is already available in homebrew, maybe the other components in the full zipkin build are also present.
I guess you could have a jar that contains only zipkin code and either include a dependency on kafka explicitly (and the other dependencies) or having something without kafka support included but that could find it opportunistically if present.

oh zipkin's support of kafka is optional. I'm leaning towards the full build

someone who wants to help with this can copy something like this https://github.com/Homebrew/homebrew-core/blob/4ef4ac3f2708daa8be276fcea3fdaade8ddded3a/Formula/d/denominator.rb into a new pull request
except use https://repo1.maven.org/maven2/io/zipkin/zipkin-server/2.26.0/zipkin-server-2.26.0-exec.jar as the jar
@reta wanna give this a try? I've done several homebrew recipes and the community is nice.

@reta wanna give this a try? I've done several homebrew recipes and the community is nice.

@codefromthecrypt really want to help here, but I don't have MacOS box available to me (I have Windows and Linux boxes) :(

ah yeah indeed macos is important for testing. I'll help with this!

as promised https://github.com/Homebrew/homebrew-core/pull/159482 please 👍 cc @openzipkin/devops-tooling

https://formulae.brew.sh/formula/zipkin is shipped. only thing left is to update the README here and the website quick start. Volunteer anyone?

https://github.com/openzipkin/zipkin/pull/3692 for the readme
https://github.com/openzipkin/openzipkin.github.io/pull/178 for the website


Better error handling techniques on the render

The method of handling the render stderr data streams automatically infers critical errors. This might not always be true. As is with  #13, #87, #90, #79, #96. With the current method, terminalizer fails to render from little things such as GTK CSS theme warnings, warnings from failure to find the appmenu-gtk-modules as it did with mine.
Solution: Await the process to complete execution. If a non-zero return code is gotten, reject the render process.
Opening a PR to address this.

Fixed in #97.
@faressoft, please review.


Transition for new ListView Items

I am trying to add a transition for every ListView item that gets added. My idea is to incorporate the transition code before I call notifyDataSetChanged(). Here is my code:
 mUpdateComments = new ValueEventListener() {
        @Override
        public void onDataChange(DataSnapshot dataSnapshot) {
            mNumberOfCommentsAtPoll = (int) dataSnapshot.getChildrenCount();
            for (DataSnapshot x : dataSnapshot.child(COMMENTS_LABEL).getChildren()) {
                Log.v("DATA_SNAPSHOT", x.toString());
                if (mCommentIDArrayList.contains(x.getKey())){
                    Log.v("Comment_Already_Added", x.getKey());
                } else {
                    Log.v("Child_Added_Called", "Child Added Called");
                    mCommentIDArrayList.add(x.getKey());
                    String commentID = (String) dataSnapshot.child(COMMENTS_LABEL).child(x.getKey()).child("COMMENT").getValue();
                    Log.v("New_Comment", "The new comment is " + commentID);
                    String userID = (String) dataSnapshot.child(COMMENTS_LABEL).child(x.getKey()).child("USER_ID").getValue();
                    Log.v("New_User_ID", "The new userID is " + userID);
                    mCommentArrayList.add(new Comments(mUserAvatar, userID, commentID));
                    Collections.reverse(mCommentArrayList);
                    mCommentAdapter.notifyDataSetChanged();
                }
            }
        }

        @Override
        public void onCancelled(FirebaseError firebaseError) {

        }
    };

And its not working? Where your TransitionManager.beginDelayedTransition call?
Actually I can recommend you to use RecyclerView with build-in item animators for that task.

Thanks, much appreciated.
On Tue, May 24, 2016 at 10:32 AM, Andrey Kulikov<EMAIL_ADDRESS>wrote:

And its not working? Where your TransitionManager.beginDelayedTransition
call?
Actually I can recommend you to use RecyclerView with build-in item
animators for that task.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/andkulikov/Transitions-Everywhere/issues/34#issuecomment-221309828

--
Troy Chuinard, CPA


Typescript died unexpectedly

We have written the needed data into your clipboard because it was too large to send. Please paste.
Issue Type: Bug
Was editing files. Nothing out of the ordinary.
VS Code version: Code 1.57.1 (507ce72a4466fbb27b715c3722558bb15afa9f48, 2021-06-17T13:28:07.755Z)
OS version: Windows_NT x64 10.0.19042
Restricted Mode: No
Remote OS version: Linux x64 5.4.72-microsoft-standard-WSL2
Remote OS version: Linux x64 5.4.72-microsoft-standard-WSL2

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz (12 x 2208)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.88GB (10.97GB free)

Process Argv
--folder-uri=vscode-remote://wsl+Ubuntu-20.04/usr/src/dojo/web/admin --remote=wsl+Ubuntu-20.04 --crash-reporter-id 5c622570-0879-4027-b07e-abcbe09199c7

Screen Reader
no

VM
0%

Item
Value

Remote
WSL: Ubuntu-20.04

OS
Linux x64 5.4.72-microsoft-standard-WSL2

CPUs
Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz (12 x 2208)

Memory (System)
24.96GB (15.85GB free)

VM
0%

Item
Value

Remote
WSL: Ubuntu-20.04

OS
Linux x64 5.4.72-microsoft-standard-WSL2

CPUs
Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz (12 x 2208)

Memory (System)
24.96GB (15.85GB free)

VM
0%

Extensions (14)

Extension
Author (truncated)
Version

expressjs
ama
2.0.0

bracket-pair-colorizer
Coe
1.0.61

es7-react-js-snippets
dsz
3.1.1

auto-rename-tag
for
0.1.7

remote-containers
ms-
0.187.0

remote-wsl
ms-
0.56.5

highlight-matching-tag
vin
0.10.1

JavaScriptSnippets
xab
1.8.0

npm-intellisense
chr
1.4.0

path-intellisense
chr
2.4.0

vscode-eslint
dba
2.1.23

githistory
don
0.6.17

prettier-vscode
esb
8.0.1

vscode-typescript-next
ms-
4.4.20210715

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
pythonvspyt602:30300191
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyt639:30300192
pythontb:30283811
vspre833:30321513
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
pythondataviewer:30285071
vscus158:30321503
pythonvsuse255:30335481
vscod805:30301674
pythonvspyt200cf:30331938
vscextlang:30333561
binariesv615:30325510
bridge0708:30335490

/needsMoreInfo

This bug report was created by your issue reporting template. If it needs more info, maybe the reporting template needs to get fixed. It is your code that generated the bug report! Please be specific of what info is missing, and I can tack it on.

See the issue reporting guidelines. This ins't actionable with without steps to reproduce or more context about what causes this issue


⚠ Bofrost.es has degraded performance

In 82e1bc5, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1555 ms

Resolved: Bofrost.es performance has improved in 7280f08 after 6 minutes.


v1.0.0

Hey, can you wait a few minutes merging this PR/updating versions? :D I want to open very minor PR myself, so it could be included in 1.0.0 as well

@njuro For sure 👌

@corygibbons thanks, it was just 1 line in the end -> #16

@corygibbons may I ask, why did you decide to remove size property? I am now trying to style it using css with maxWidth and maxHeight, but it doesn't quite work.

@njuro The sizing already wasn't precise visually because there was built in horizontal spacing on both sides of the icon. So setting size={48} was really giving you an icon that was around 42px wide, and you'd have to factor in the that space to get precise alignment between FileIcon's and other elements.
size wasn't necessary in the use cases I was testing but I'll look into it further and potentially bring it back in some capacity, might be better naming it maxWidth as that's really what it's doing.
In the meantime including something like svg { max-width: 100%; } in your app should hopefully solve the height problem for you, allowing the icon to scale within a box always staying within it's width and height.

but I think it should be highlighted in the readme (that icon will be always 100% size of parent container)

I had it in the release notes for breaking changes but makes sense to include in the readme as well, I'll ad it 👍


Filter client config on service labels

This change relates to AEROGEAR-2419.
Because bindings use secrets by default, our APB's have followed this format and standardised on the ServiceConfig object based on this proposal
Going forward, all mobile APB secrets created via a binding will have the label clientId applied to them
Changes proposed in this pull request

Search for related secrets in the get clientconfig command, filtering on mobileClientId label
Remove redundant code used to convert ConfigMaps to specific ServiceConfig objects since the linked proposal invalidates these formats

@mikenairn I am not a big fan of the table out put here. I think we should just always output json. I don't really see any use for it

I am not a big fan of the table out put here. I think we should just always output json. I don't really see any use for it

@maleck13 If we change this command to just return json regardless of the specified output type, would that make it different from every other command? Would it not seem a bit inconsistent, or do we do this already for other commands?

Most commands return table out put also. I would take usefulness over consistency here. Is there value in the table view? if so lets keep it otherwise remove

Most commands return table out put also. I would take usefulness over consistency here. Is there value in the table view? if so lets keep it otherwise remove

In two minds about this actually, because someone familiar with using the k8 or oc client wouldn't expect such behaviour.
Say for example someone runs oc get po xyz they get a subset of information. To get more detailed info they would run the describe command or else as above with -o json
Personally I think this is a better approach but do understand in this instance, the information supplied by the command would be almost useless

@mikenairn @PhilipGough looking good. Can we get some verification steps in the PR and once verified we can merge

👀  sorry took so long to get to this. Verifying now


Simplify setting up a test environment

As we just wrote in the documentation, right now we have to install the following packages manually:
Django
Wagtail
psycopg2
pytest
pytest-cov
pytest-django
coverage
factory-boy

It would be convenient to either let the Makefile take care of this or another best practice.

Running tests via tox will take care of this: https://github.com/LUKKIEN/wagtailtrans/blob/master/tox.ini
Perhaps it's good to have a tests_require in your setup.py, that way you can just install pip install -e .[test] other dependencies (for the sandbox) can for example be done via make sandbox

Yeah tox takes care of this but this is not the case when running make qt which depends on those additional packages. So I'm good with having something like tests_require in addition to tox.

Merged with 0754f17


Should we use https://github.com/naholyr/github-todos?

Could be useful, given way we are currently writing todos and issues.

Voted against this after discussing because it would conflict with ESLint.


[suggestion] INSTALL.BAT Cannot find lua (or anything else)

C:\luarocks>install

C:\luarocks>rem=rem --[[
The system cannot find the path specified.

After this patch, it works correctly.
--Andrew

I just downloaded luarocks-2.2.0-win32.zip (on a Windows 8.1 VM with VC2013) and got the same error as @andrewstarks in his first message. Is our Windows installer broken (or perhaps the win32 zip in the server is the incorrect one?...)

Aaand the zip file in http://keplerproject.github.io/luarocks/releases was really broken. Apparently I messed up during the move to the new site. Things should be fixed now, but I'm off to actually test it.