Recover CMake ARCH_INDEPENDENT usage from dbb2423248

Usage of CMake ARCH_INDEPENDENT was introduced in https://github.com/nemtrif/utfcpp/commit/dbb2423248dd5b97acd4125a29bdfbf6b36cd7b1 before v3.2.4 release, but got lost with v4.0.x release. This commit recovers this functionality in v4.0.x series.

OK, with a reminder that CMake install is not a supported way of deploying the library.


Red Hat: Better readability fixes V2

Red Hat - Better readability, few enhancements

[V2] New file
Fixes "Status" wording to "Fixed" where issues were actually fixed
Added Dates for every product
Minor text placement reorg
Added extra clarification to some product affectedness

Extra notes
Thanks CISA team for curating this content 💪
💭 Motivation and context
There were some lacking information and content / statements were not uniform across all products.
🧪 Testing
N/A
📷 Screenshots (if appropriate)
N/A
✅ Pre-approval checklist

[X] This PR has an informative and human-readable title.
[X] Changes are limited to a single goal - eschew scope creep!
[X] I have read the CONTRIBUTING document.
[X] These code changes follow cisagov code standards.
[X] All new and existing tests pass.

✅ Pre-merge checklist

[ ] Revert dependencies to default branches.
[ ] Finalize version.

✅ Post-merge checklist

[ ] Add a tag or create a release.

Supersedes: pull/298


"The remote certificate is invalid according to the validation procedure"

Hi,
Whilst trying to execute your script I get the following:
Connect to Office 365 IMAP Service.
Ran into an exception while negotating SSL connection. Exiting.
Exception calling "AuthenticateAsClient" with "4" argument(s): "The remote certificate is invalid according to the validation procedure."
Would be grateful to receive any pointers as to how to resolve.
Thanks

i have not been able to repro this issue. i have successfully authenticated with an App registration using certificates.


CAMEL-19675: added test case for aggregation strategy with attachment

Test case for attachment being removed from the Exchange after AggregationStrategy

This one is gonna fail without my fixup patch on #10888. We are good to merge.


Add Natural Date capability

Natural Date allows for user to input any date that they want (i.e. Mon, Monday, Wed, Weds, Wednesday). Time still needs to be in the same format as the usual dates.
Noted that the addo does not allow for multiple orders by the same person.
Will add in test cases later.

Codecov Report

Merging #141 (2429a1c) into master (775ca2b) will decrease coverage by 0.91%.
The diff coverage is 44.70%.

@@             Coverage Diff              @@
##             master     #141      +/-   ##
============================================
- Coverage     65.44%   64.52%   -0.92%
- Complexity      537      542       +5
============================================
  Files           101      103       +2
  Lines          1910     1993      +83
  Branches        215      237      +22
============================================
+ Hits           1250     1286      +36
- Misses          591      623      +32
- Partials         69       84      +15

Impacted Files
Coverage Δ

...va/seedu/address/model/order/DeliveryDateTime.java
71.42% <ø> (ø)

...a/seedu/address/model/order/NaturalDateParser.java
40.00% <40.00%> (ø)

...in/java/seedu/address/model/order/DateChecker.java
43.47% <43.47%> (ø)

...in/java/seedu/address/logic/parser/ParserUtil.java
85.93% <100.00%> (+0.45%)
:arrow_up:

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 775ca2b...2429a1c. Read the comment docs.


Conflicting licenses

package.json reports MIT, but the LICENSE file reports GPL.

fixed in master, released to npm as version 0.6.2


Bugfix/fix-load-atoneJS-ios8

Github Issue URL/Tags (required)
What's this PR do? Please give a technical summary (required)

Fix loading JS in ios 8

Which acceptance criteria does this PR do? (optional)
Have you written a test code? if not, please provide an evidence instead (Screenshot, log ...) (optional)
Screenshots (if appropriate) (optional)
Any references (stackoverflow …) (optional)

@at-suho
Please check this PR.


Updated navbar of linkedIn clone

-added new elements in the navbar such as my network, notifications, and jobs.
-also included icons for every element.

Please make the requet agin after making  a git pull .

Please make the requet agin after making  a git pull .

Just wanted to confirm, do you I need to use the following command in order to make a git pull?
git pull REMOTE-NAME BRANCH-NAME

Just wanted to confirm, do you I need to use the following command in order to make a git pull?
git pull REMOTE-NAME BRANCH-NAME


Use existing type if provided

Instead of forcing a type of 'text', use existing type if available.

I see that's to account for password types. Won't work.


Snake_GoGoGo

Snake_GoGoGo!

Thanks for your contribution! I will deal with this request in July.

Thanks for your contribution! I will deal with this request in July.

Chinese say Chinese,土狗放洋屁

Thanks for your contribution! I will deal with this request in July.

我相信你有能力把这个简单的文件改成c语言形式，大不了学习一下在c模仿实现容器模板


Services do not operate correctly when docker net host option is specified

The addition of --net in #617 is awesome, I need it, but I may have found a bug - I don't have bandwidth to test this one out at the moment though...
The implementation of services looks to rely on Docker Links
https://github.com/drone/drone/blob/0a6227930dce3e356667506bc950530a9d847c61/shared/build/build.go#L346
Links and --net=host are not supported https://github.com/docker/docker/pull/7066
A possible workaround might be to run service containers in --net=host but you might run in to port conflict issues if you are, for example, using postgres as the backend for your Drone install and attempting to run a postgres service.

cool, we can test it out and see. I think we must have accepted a competing pull request (I'll verify), because you can specify the net parameter in your yaml:
docker:
  net: host

@bradrydzewski sorry I might not have been clear in my original message...
I've managed to confirm this one, given the following drone.yml
---
image: library/golang:1.3
docker:
  net: host
services:
  - mysql
script:
  - env
  - echo "Hello World"
  - ip a
  - sleep 30

@dave-tucker can you give a hint, why you need a net: host for drone? We managed to get up and running Docker 1.5 with IPv6-only setup without using it.

@floatdrop we need our services and the container where the tests are run to share the same network namespace. net: host works, although using docker's `--net=container:<container_name_or_id> would be much safer, but at the time of writing that functionality didn't exist in drone.

@dave-tucker I'm working on a patch that implement --net=container:{container}. Hopefully will have something to demo next week.

Without drone supporting vagrant directly, where VM's are needed (involving building VM images, testing software making use of kernel modules, alternative OS's, etc) use of --net=host seems like it would be required for vagrant inside of a container image to be able to talk to VM's launched via libvirt.
Something like https://hub.docker.com/r/twiest/docker-vagrant-libvirt/
Just evaluating drone here as a replacement, and to be able to limit the use of VM's to just the project being tested rather than needing to also bring up additional services it needs as VM's along side it would be very useful.


Support .select() with wildcards

Currently ojai-testing does not support to specify a query with .select("*"). Is there any easy way to support this?

@raman-nbg in OJAI, when you build a Query with no select, that will select all the fields for each retrieved document.

I want to pass "*" as a pure select. We are building a REST service, where you can filter resources using ?select=prop1,prop2. If the select parameter is missing, we just want to pass * to the select. This is easier than building a different query, with different method calls.

@raman-nbg In reality, this can be done within the library quite easy, however, it will then not be OJAI compatible which is the whole idea of the library itself.

Mmh I thought ojai MapR-db does support this... Let me have a look :/ sorry
Nicolas A Perez<EMAIL_ADDRESS>schrieb am Sa., 9. Mai 2020,
14:51:

@raman-nbg https://github.com/raman-nbg In reality, this can be done
within the library quite easy, however, it will then not be OJAI compatible
which is the whole idea of the library itself.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/anicolaspp/ojai-testing/issues/7#issuecomment-626171567,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEYWC32KJJZVAPMGY7QAOPDRQVGWDANCNFSM4M3S3AKQ
.

Maybe MapR-DB doesn't, but that is not the only place this library is used and certainly, it is allowed in OJAI.

@raman-nbg have you tried passing emptyString "" instead of *??? that might be what you want, but you need to test it


Port to Python3

It's not just that, it's also for the moment nice to tell pypi/pip that this is a py2-only package, so that pip fails gracefully on a py3 system instead of thinking it can install and then git-pre-commit-hook dying.


exit vm from within js

If I execute a script loaded from file within the VM, (using RunScript / RunString), can the script terminate the VM?

just use vm.Interrupt in your own Go function...


Feature request - pick with readonly array

Have the Object pick function accept readonly string array of property names.
Then it can work together with nestjs's PickType helper function which can use the same array, but requires it to be readonly.
Now, the below example doesn't work
import { PickType } from '@nestjs/mapped-types'
import { pick } from 'radash'

const FORM_PICKS = ['id', 'labelNl'] as const

export class FormDto extends PickType(ElabFormEntity, FORM_PICKS) {
  constructor(entity: ElabFormEntity) {
    super()
    Object.assign(this, pick(entity, FORM_PICKS))
  }
}

Fails with
Argument of type 'readonly ["id", "labelNl"]' is not assignable to parameter of type '(keyof ElabFormEntity)[]'.
  The type 'readonly ["id", "labelNl"]' is 'readonly' and cannot be assigned to the mutable type '(keyof ElabFormEntity)[]'.ts(2345)

I'm having a similar issue, is there a workaround for this?

Not that I know of

Hello @n-rowe. Over at the Radashi fork, we've fixed this issue. You can try it today by installing radashi@beta. See you over there!
https://github.com/radashi-org/radashi


fix(Firestore): SnapshotTrait getSnapshot method

Fixed #5806
Changes

Decoded the API format timestamp received in batchGetDocument() method to Google\Protobuf\Timestamp using Serialize:decodeMessage().
Added system and unit test for this new change.

@vishwarajanand

LGTM, but pls make sure the feature is available in PROD before merging to main.

Talked internally and found that reading within the GC window using readTime optional parameter is already a GA feature. So fixing this right now won't be a problem.


Remove redundant string resources

Removed redundant English assets.
Updated text files in English assets since they caused issues in TTS implementation.


tBTC website captures users looking to join stakedrop

A flow leading people searching for "tbtc stakedrop" back to the Keep site/info on stakedrop.
Must be done in a brand neutral way/tbtc voice
Potentially mentioned in the announcement section (& takes you away for the info)

@Gluzman why can't I add labels to this issue, do I not have permissions in the tbtc-website repo?
Also, I can't figure out what "adding as a collaborator" means practically, I only see options to assign.

@creckhow correct, collaborator is just vernacular for our own process. The actual action in Github is going to be tagging with @ in the description. And then I can assign myself if I will end up adding a lot to this issue.
As for labels—In GH, labels are specific to each repo. So the repo in which your issue lives, also has a label set from which you can choose existing labels and create new ones. The sprint board links many repos together but from my experience, it appears that labels cannot be shared between repos even if they all contribute issue to the same project board.

@Shadowfiend I don't have permissions to assign or add labels to tbtc-website issues, Amanda and Kris should probably be able to do this too

Should be good for the whole growth group, sorry for the delay.

Cool @Gluzman I think this is your issue, please assign yourself if you agree. There is a similar one for Liz in the keep-website repo. https://github.com/keep-network/website/issues/191
We may want to repurpose the 'join the crowdcast' banner for a CTA that brings people to the stakedrop page, the gitbook, etc... or we could do away with the banner and place this somewhere else on the site.
cc @mhluongo for thoughts, personally I'm not sure if keeping a banner up throughout the entire stakedrop period feels on-brand, but we do want people googling "tBTC stakedrop" and landing on the tBTC site to find their way to the right info...
Also one for Amanda to weigh in on (once she accepts the invite)

The way tBTC talks about Keep's stakedrop: "Be a tBTC signer and earn. Stake ETH and help bring Bitcoin to DeFi." And then "learn more" (going to stakedrop page - probably?)

@creckhow is the announcement strip on tBTC fulfilling this at the moment? or do we still need a copy change?


glb schema update

Unit test:
:go test -v
=== RUN   TestDnsSvcsV1
Running Suite: DnsSvcsV1 Suite
==============================
Random Seed:<PHONE_NUMBER>
Will run 156 of 156 specs

Integration Test:
go test -tags integration -run TestDnsLoadBalancerOperation -v
=== RUN   TestDnsLoadBalancerOperation
--- PASS: TestDnsLoadBalancerOperation (21.02s)
PASS
ok  	github.com/IBM/dns-svcs-go-sdk/dnssvcsv1	23.540s


Make id optional

Json response without an id is now possible. Simply return false on the getId method of a Resource's Serializer and no id will be returned.
Documents without an id are conform the JSON API v1.0 specification. For example, see: http://jsonapi.org/format/#crud-creating

I'm curious why you have used a false return value from the getId methods of both the serializer and the resource. Why haven't you simply used a null value or '' (empty string) for example, since they are both falsy values.

My curiosity is mostly sparked from the rule that both id and type Resource properties MUST be strings. Therefore it would make more sense to me to return an empty string from the getId methods.
If you only use strings to as return value, you can replace the line #166 with
return $this->serializer->getId($this->data); making it cleaner.

Hi Cedric,
Thanks for your feedback! I've changed the return value on the getId() method as you suggested.

 When I get home today I will check on why the tests are failing. That will be around 19:00 (CEST). Maybe someone with admin access can check on the StyleCI errors?

Well I dug a little bit deeper into the issue, and here is my contribution. This pull request tries to tackle the following exception of the JSON API spec v1.0.

The id member is not required when the resource object originates at the client and represents a new resource to be created on the server.

This says that when trying to create a new resource instance, the client can create a resource that omits the id property. Mainly because it is not available and it doesn't make sense to use { "id": null }. I think actually don't think this library was designed to be used as client. Mainly because in the README the word responses is used (JSON-API responses in PHP.), implicating this library should be used as server. So I think this feature should be discussed first.
Besides the point if this should be implemented or not, this pull request needs a little more work. It needs some proper testing and it contains changes that has implications on other files and tests. E.g. the serializer getId method should be able to return null for this feature. For the full list of things that should be done when contributing, see the contributing guidelines.
TL;DR;

Try to start a discussion in a new issue. We can openly discus there if this feature should be implemented and/or how.
Try to adhere the contributing guidelines by adding tests for next pull requests.

I completely understand your opinion. I will openly ask whether this library should allow omitting the id property and thus can be used as a client.

Howdy, interesting suggestion.
However, I mostly agree with @byCedric that this is not needed for the current feature set of the library. Can you explain what exactly the use case is? Are you using this to build a JSON-API client or what is going on?

Closing for now in favour of discussion in #101.


Stop using the reserved 'main' entry for CSS

Uses 'css' field instead.
Fix for #122.

See https://docs.npmjs.com/files/package.json#main for reference. main can only be a JavaScript file.

It looks like you are right :) Thanks man..

css or maybe put it in files instead.. where we could include all the source files?

files is not necessary, since NPM includes all files except for node_modules, or .gitignore. You don't actually need any field, but I saw some CSS packages using css (either bootstrap or awesome-font). It's not standardized though.

then let's just remove it completely.. :)

jsut remove this line.. if you are still around :)


Roll Flutter Engine from 496c11601c72 to c29bf8fcf002 (1 revision)

https://github.com/flutter/engine/compare/496c11601c72...c29bf8fcf002
2022-07-15<EMAIL_ADDRESS>[Windows] Fix GDI resource leaks in the software fallback path (flutter/engine#34574)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

@keyonghan The autosubmit bot has let this one sit for ~20 hours. I am going to land it manually.


Some text input changes

Hi,
I'm not sure if you are even interested in changes, but when I was trying out your library I noticed that backspace wasn't quite working.
I tried to copy your code patterns to not be intrusive, even if some of the code looks a bit repetitive with my additions.

the macos fix is kind of fragile since it relies on the if statement of backspace being before the delete. Hopefully that's okay since the current method will not be able to differentiate between the two anyway.
Maybe it could just use KeyCode::Back and KeyCode::Delete? I don't know what works on your machine I'll leave that to you :)

well this has been an adventure. I've never dropped a commit before, I think it worked out though. Thanks for being patient with me

@rmsthebest My apologises, I was wrong about timer. It wasn't required. One of condition to not exit system early is non-empty keyboard events stream. bevy emits pressed events while you hold key pressed based on os preferences. I rethought how input should be handled and came to solution in 6ce07af. Without your PR, this would not have happened soon. Thank you!


Finish the migration from ImportedTypeName over to TranslatedType

TranslatedType captures more information about how the type is represented in Swift, Java, and the intermediate stages in between. Migrate all remaining functionality from ImportedTypeName to TranslatedType.
Start resolving translated types through a centralized "lookup" mechanism rather than assuming that the types named in Swift are always going to have Java counterparts. For now, this means we won't try to translate functions that reference types that we don't know will be imported into Java. In the future, it means we'll be able to reason about these types to (e.g.) pass arguments and return values of those types correctly across the Swift/Java barrier.

@ktoso  I'm going to go ahead and merge this, because I have a follow-up PR that benefits from being separate.


Correction for raw affiliation Department of Chemical Engineering, École Polytecyhnique de Montréal, Université de Montréal Campus, 2900, boul. Édouard-Montpetit, 2500, Chemin de Polytechnique, Montreal, QC, H3T 1J4, Canada

Correction needed for raw affiliation Department of Chemical Engineering, École Polytecyhnique de Montréal, Université de Montréal Campus, 2900, boul. Édouard-Montpetit, 2500, Chemin de Polytechnique, Montreal, QC, H3T 1J4, Canada
raw_affiliation_name: Department of Chemical Engineering, École Polytecyhnique de Montréal, Université de Montréal Campus, 2900, boul. Édouard-Montpetit, 2500, Chemin de Polytechnique, Montreal, QC, H3T 1J4, Canada
new_rors: 05f8d4e86
previous_rors: 0161xgx34;05f8d4e86
works_examples: W2464659683
contact: b03097925488d20f63c93d3606753edf:f7ffd9ed1e578025b647ad @ polymtl.ca

This issue was accepted and ingested by the OpenAlex team on 2024-11-01. The new affiliations should be visible within the next 7 days.


Cleanup fix

Do not fail for missing settings.

Codecov Report
Base: 61.44% // Head: 37.70% // Decreases project coverage by -23.73% :warning:

Coverage data is based on head (6f1b057) compared to base (57f25a9).
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1765       +/-   ##
===========================================
- Coverage   61.44%   37.70%   -23.74%
===========================================
  Files         178      178
  Lines       13628    13628
===========================================
- Hits         8374     5139     -3235
- Misses       4194     7722     +3528
+ Partials     1060      767      -293

Flag
Coverage Δ

acceptance-api
?

acceptance-apps
36.10% <ø> (+0.18%)
:arrow_up:

acceptance-cli
?

unittests
8.54% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

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


Fix for long file names

This fixes "error: filename too long" error when creating screenshots or .json names longer than the OS (in my case, Xubuntu 14) can handle.
Not sure if it worth it to merge or not.

Cool! Thanks!!!


Missing license

Hi @adamw !
It seems this project is lacking a license file.
Without one, it's "all rights reserved", hence not open-source.

Indeed, the build specifies Apache2, but the file was missing - added. Thanks for letting me know :)

Thanks :)


Android: attach thumbnail to /dispatch

Even if resource has already been fully uploaded!
Useful when target has poor internet.
Drawback: dispatch request ~8k instead of ~1k

This would also help #309 Inappropriate photo filter, because a thumbnail (~256px wide) is perfect to quickly check for inappropriateness.

In some cases, the extra 7KB would be sent twice: with /scanned and with /dispatch.
It's the price to pay, to save the workflow from yet another headache.


Changes on the gdrive are not consistently reflected when looking at the mountpoint

System
Debian Jessie VPS, google-drive-ocamlfuse installed using the how-to in the wiki.
google-drive-ocamlfuse was installed using a new user (airsonic). This user owns /home/airsonic/.gdfuse as well as the mount point, /home/airsonic/googledrive/gdrivemus1.
I have set metadata_cache_time=10 in the /home/airsonic/.gdfuse/gdrivemus1/config.
Issue
When I add or remove some folders or files using the Google Drive interface, they may or may not (far more often not) show up in ls -la /home/airsonic/googledrive/gdrivemus1, even hours later (so it is not just a refresh time issue).
Some folders and their content did show up initially for my tests, but then when I wanted to continue adding some, I noticed the changes were not appearing in the mountpoint. I thought maybe there was an error with the authentication or mount, or just that I needed to manually refresh the drive to trigger the changes, so I tried the following:
fusermount -u /home/airsonic/googledrive/gdrivemus1
google-drive-ocamlfuse -label gdrivemus1 /home/airsonic/googledrive/gdrivemus1

It didn't fix the issue, nor did google-drive-ocamlfuse -cc.
I also tried touch /home/airsonic/googledrive/gdrivemus1/test, and test appeared immediately on Google Drive UI meaning the computer has no issue writing on it. I could see test showing up in the terminal as well, and could delete it from there (which did remove it from the Google Drive UI).
It seems it's not an issue to write to the Google Drive, but reading from it is inconsistent and unreliable. Is that a known issue? Is there a fix?

Six folders on Google Drive, five in the mount point. These images show only recent folders, but this is with a new installation of google-drive-ocamlfuse from scratch. I had the exact same issue yesterday and the files/folder were still invisible to the server after hours, or after remounting.

Note than I am having trouble getting Airsonic to see my files in the mount point, but I am not sure whether it is 100% related to the above issue. The issue I have with Airsonic is some files that do show up in terminal are not scanned in the Airsonic library. Those that do not show up in terminal, of course, are not scanned either.
Now could it be that files that don't show up in terminal and files that do but are not seen by Airsonic are caused by the same thing? No idea, but I created an issue on both Github projects so we can see how interrelated they are.

Two hours later, the folder Public broadcasting service was still not listed in the mount point. I did the following again:
touch /home/airsonic/googledrive/gdrivemus1/toto.txt
rm /home/airsonic/googledrive/gdrivemus1/toto.txt

And the folder suddenly appeared. It is important to remember that the same manipulation two hours ago had no effect though, despite the fact that the folder was successfully added to Google Drive 30 minutes earlier.

Should be fixed in the latest versions.


fix: made fixed to env tests and readme

Ticket Link

Related Links

Description

Made appropriate changes to test for env files
Steps to Reproduce / Test

Request

Response

Codecov Report

Merging #116 (e5b7a8e) into master (5873b8a) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #116      +/-   ##
==========================================
+ Coverage   84.22%   84.23%   +0.01%
==========================================
  Files          28       28
  Lines        1103     1104       +1
==========================================
+ Hits          929      930       +1
  Misses        123      123
  Partials       51       51

Impacted Files
Coverage Δ

internal/config/env.go
95.31% <100.00%> (+0.07%)
:arrow_up:

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


Remove Linux Cgo requirement

Did you consider using https://github.com/BurntSushi/xgb which is Cgo free/native binding to XCB? I am not sure though, but it should be possible to rewrite with XCB instead of Xlib so Linux implementation can be Cgo free.

It is an interesting option (maybe this fork is more up to date https://github.com/jezek/xgb), and also would be wonderful if we could remove cgo completely. Nevertheless, I think we will consider writing everything ourselves without introducing third dependencies outside golang.org.


EntityPersistentDataReader/Writer - Add helper methods

Add helper methods to EntityPersistentDataReader and EntityPersistentDataWriter

GetKeyArray
GetValueArray
GetEnumerator
RemoveAndDispose (Writer only)

What is the current behaviour?
Consumers of EPD readers/writers are not able to iterate over all of the persistent data. This is an unusual usage bust still required in some situations.
Data in an EPD implements IDisposable but it's easy for the developer to forget to dispose the elements when removing them.
What is the new behaviour?
The pass through methods to the backing collection are now exposed.

GetKeyArray
GetValueArray
GetEnumerator

Add RemoveAndDispose to the writer to help make it easier to correctly remove EPD elements. I'm open to making this the default behaviour of Remove and instead having a RemoveWithoutDispose as a special case method that can be called. Thoughts?
What issues does this resolve?

None

What PRs does this depend on?

None

Does this introduce a breaking change?

[ ] Yes
[x] No

Sounds good. I just figured that because you're dealing with persistent data...which is something special. There's almost no case where you would want to remove and not dispose.


Allow the dependency constraints associated with a configuration to be queried

In general, it should be possible to query the data structures associated with the model. The dependency constraints associated with a configuration are part of the model and should be queryable, in a similar way that the dependency declarations for a configuration can be queried.
Expected Behavior
There is a way to query dependency constraints associated with a configuration that is consistent with the way that dependency declarations are queried (which can be different to whatever that happens to be now).
Current Behavior
Dependency constraints are write-only.

Actually, dependency constraints are part of the dependency set at the moment (because they also extend Dependency) and can be accessed through that. This will change with #4184. So these two issues are best addressed together.

Added as part of #4255:
https://github.com/gradle/gradle/pull/4255/files#diff-123dd87518f4d90f42e82145375e901a


Added test case for info.go

Added test case for info.go:
TestCreate - tests the creation of Info
TestDeleteAndSave - tests deleting and saving Info
TestCreateDuplicate - tests creating duplicate Info
TestDeleteDuplicate - tests deleting duplicate Info

great job now add some for the domain storage

Please rename them to TestInfoSaveDelete etc...
Then add same tests for domains


Profile Sqitch to Identify Performance Bottlenecks

Sqitch is slow. And not just because it talks to the database. I suspect it's slow mainly because of Moose constraints. But we should profile it to be sure.
Best approach is probably to run the test suite with Devel::NYTProf. And if Moose is in fact the problem, maybe see about the feasibility of switching to Moo.

Gonna call this one done. DateTime is an independent fix, and the switch to Moo was a huge improvement. We can revisit in the future, but no need to keep tracking it for 1.0.


Error batch downloading a teachable course

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2019.03.18. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2019.03.18

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--username', 'PRIVATE', '--password', 'PRIVATE', '--verbose', '--sleep-interval', '60', 'https://lee-jae-bok-badminton-school.teachable.com/courses/452504']
[debug] Encodings: locale cp1252, fs mbcs, out cp850, pref cp1252
[debug] youtube-dl version 2019.03.18
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.17134
[debug] exe versions: none
[debug] Proxy map: {}
[generic] 452504: Requesting header
[redirect] Following redirect to https://lee-jae-bok-badminton-school.teachable.com/p/mastering-clear-badminton-hitting-skills
[generic] mastering-clear-badminton-hitting-skills: Requesting header
WARNING: Falling back on generic information extractor.
[generic] mastering-clear-badminton-hitting-skills: Downloading webpage
[generic] mastering-clear-badminton-hitting-skills: Extracting information
[TeachableCourse] Downloading lee-jae-bok-badminton-school.teachable.com login page
[TeachableCourse] Logging in to lee-jae-bok-badminton-school.teachable.com
[TeachableCourse] mastering-clear-badminton-hitting-skills: Downloading webpage
[download] Downloading playlist: Course 2. How to teach forehand clear - the most important hitting skill in badminton
[TeachableCourse] playlist Course 2. How to teach forehand clear - the most important hitting skill in badminton: Collected 16 video ids (downloading 16 of them)
[download] Downloading video 1 of 16
[Teachable] Downloading lee-jae-bok-badminton-school.teachable.com login page
[Teachable] Logging in to lee-jae-bok-badminton-school.teachable.com
ERROR: Unable to download webpage: HTTP Error 404: Not Found (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpcs52imf5\build\youtube_dl\extractor\common.py", line 626, in _request_webpage
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpcs52imf5\build\youtube_dl\YoutubeDL.py", line 2225, in urlopen
  File "C:\Python\Python34\lib\urllib\request.py", line 470, in open
  File "C:\Python\Python34\lib\urllib\request.py", line 580, in http_response
  File "C:\Python\Python34\lib\urllib\request.py", line 508, in error
  File "C:\Python\Python34\lib\urllib\request.py", line 442, in _call_chain
  File "C:\Python\Python34\lib\urllib\request.py", line 588, in http_error_default
...
<end of log>

Description of your issue, suggested solution and other information
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.

For any work on this you must provide account credentials.

Same here :(


iOS: deprecation warnings

Building the iOS app results in a number of deprecation warnings, for UIAlertView, UIPopoverController, openURL:, CFReadStreamCreateForStreamedHTTPRequest, and others.
It'd be a good idea to update these to modern APIs sometime.  I'd be happy to help with that if you wish.

Yep, I'm on these in the ios8 branch. Gonna require removing ASI and moving to AFNetworking. We already have AFNetworking, but some requests are still on the old ASI.


Added Golang Gin Web Framework to free-programming-books-langs.md

What does this PR do?
Add resource(s)
For resources
Description
Added Golang Gin Web Framework
How do we know it's really free?
you can check the link
For book lists, is it a book? For course lists, is it a course? etc.
It is a book
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Thanks, and welcome to free-programming-books!

Thanks, and welcome to free-programming-books!

Thank you


[Bug]: List databases crash when databases count large than 8192

Is there an existing issue for the same bug?

[X] I have checked the existing issues.

Branch name
main
Commit ID
7147a847db41ebdb01714d6b8f10bed5b3cc73bb
Other environment information
No response
Actual behavior
Infinity crash since the vector size is more than 8192, exceed the capacity.
Expected behavior
No response
Steps to reproduce
1. Create too many databases, more than 8192. (run case: test_create_drop_show_1M_databases, cancel the case running before it finished)
2. List databases.
3. Crash since the vector size is more than 8192, exceed the capacity.

Additional information
No response

Hello, I'm a newbie in the community and I'd like to try this issue, I understand that it's a crash caused by overrunning the capacity when creating a large database, but I don't have the slightest clue as to how to solve the problem, I wonder if you could give some advice?

Firstly, I am very pleased that you are interested in this project and issue.
The transfer of data between Infinity operators is done through structure DataBlock. And DataBlock is composed of multiple columns, with each column being a data structure called ColumnVector. For the ListDatabase operation, it is to put the names of all databases in the current database into ColumnVector, and then form a DataBlock to return.
The current issue is that each ColumnVector has 8192 rows of data, which means each DataBlock has 8192 rows. If the number of databases obtained by ListDatabase operation exceeds 8192 rows, the ColumnVector cannot hold them, which will trigger a crash.
In this case, we can use multiple DataBlocks, that is, multiple ColumnVectors, to contain the results of the ListDatabase operation. That is to say, as soon as we reach row count 8192, we will create another ColumnVector/DataBlock to hold more data.
The location of the ListDatabase code is in: PhysicalShow. cpp: L341, which is the PhysicalShow:: ExecuteShowDatabases method.
In terms of implementation, we may need to introduce an IsFull check in the class of ColumnVector or DataBlock to determine that the current ColumnVector/DataBlock is full. This makes it convenient for us to create a new ColumnVector/DataBlock in PhysicalShow:: ExecuteShowDatabases to load the new data.
The above is the relevant information about this bug, hoping to be helpful to you.

Thank you very much for your reply, but I think, I should try to reproduce the problem first, I successfully created 8193 databases by using the following codes.
HOST="localhost"
PORT="5432"
USER="your_username"
DATABASE="postgres"
for i in {1..8193}
do
    DB_NAME="database_$i"
    echo "Creating database: $DB_NAME"
    PGPASSWORD=your_password psql -h "$HOST" -p "$PORT" -U "$USER" -d "$DATABASE" -c "CREATE DATABASE \"$DB_NAME\";"
done

echo "Done creating databases."

But there are some problems when enumerating the databases, I use \l as well as SELECT datname FROM pg_database; both are unsuccessful, can I trouble you to give some suggestions?

Although the SQL interface of infinity uses the PG protocol, it has not been fully implemented and can only be accessed through the command line. And infinity is not a database system implemented based on PG. So, there is no catalog like pg_database.
Currently, it is recommended to use the Python SDK API and REST API to access infinity.
Meanwhile, you can also refer to [test database cases](https://github.com/infiniflow/infinity/blob/6a4227affc665fba9bd012306669099d821880fa/python/test/test_database.py#L163C9 -L163C43)

Thanks for the reply. I will try this problem next week cause I have a test on Saturday. :)

Although the SQL interface of infinity uses the PG protocol, it has not been fully implemented and can only be accessed through the command line. And infinity is not a database system implemented based on PG. So, there is no catalog like pg_database.
Currently, it is recommended to use the Python SDK API and REST API to access infinity.
Meanwhile, you can also refer to test database cases

Thanks for the reply. I will try this problem next week cause I have a test on Saturday. :)

Although the SQL interface of infinity uses the PG protocol, it has not been fully implemented and can only be accessed through the command line. And infinity is not a database system implemented based on PG. So, there is no catalog like pg_database.
Currently, it is recommended to use the Python SDK API and REST API to access infinity.
Meanwhile, you can also refer to test database cases

Good luck.

@colommar still working on it? Otherwise I will take it.

@colommar still working on it? Otherwise I will take it.

Nope, I was quite busy these days...


clusteringend

how to implement clusteringend clusteringbegin events. it are not working

I don't know. Please check on StackOverflow instead.
Also,  6.0.0 is released on npm beta tag now. We also have a new demo page. Feel free to try it:
https://tomchentw.github.io/react-google-maps/


Migration from/to physical tables and columns

Support for data migration from/to dedicated tables and physical columns.

Scenario:
I want to quickly create a set of modules to hold my data. I do not touch the storage settings, and all magic happens inside the compose_record table.  Models evolve, and data grows through time.
There might be some performance issues.
So, I decide to move the entire module out of the compose_record table into a new one. This isn't very pleasant task but a matter of a single insert-into-from-select command. Same for moving data back in.
Moving data from JSON-encoded blob into columns is trickier but still doable. Moving it back -- a bit harder.
This should be achievable via CLI (first) or HTTP REST API (later).
Unexisting tables and columns should be created or removed implicitly when data is moved.

@skamensky interested in implementing this?
At least core functionality + CLI for now.
How I see this is that you specify a target module and set of instructions for the reconfiguration of one or more columns and migration of the data in those columns.

@darh ,
I personally wouldn't need this since I'd only ever use physical tables/columns and don't have legacy migrations to have to worry about. I am interested in contributing to Corteza where needed, even if it's not for something I use. However, currently I don't have the bandwidth.
If you'd ever have to deprecate support for physical columns/tables due to lack of resources, I'd jump in to prioritize of course.

Ok, Looks like a slight misunderstanding here.
First - there are no plans to deprecate this. This is something we want to support.
Your recent patch will only make it more usable.
I've updated the issue description with a scenario. Hope it will make more sense then.

@darh ,
I think I understood, though the added detail does help me understand the implementation strategy.
Still, as I don't have a vested interest in this (I have no JSON value columns - everything is physical), I'm not going to have time to work on it for now.


build istio and proxy RPMs is broken

Bug description
tried to build istio and sidecar rpms for RedHat VM mesh expansion, but failed with a lot of errors because the build scripts are not up-to-date.
Steps to reproduce the bug
Steps to build the RPM package:

Clone Istio repo (https://github.com/istio/istio.git) into dir /root/git/src/istio and cd into this dir
Run cmd make VERSION=1.4.0 rpm/builder-image to build docker builder image: istio-rpm-builder
Run cmd make VERSION=1.4.0 rpm/proxy to build, but had some git issue during build
Try below command to debug the issue:
docker run -it \
-v ${GO_TOP}:${GO_TOP} \
-w /builder \
-e USER=${USER} \
-e ISTIO_ENVOY_VERSION=${ISTIO_ENVOY_VERSION} \
-e ISTIO_GO=${ISTIO_GO} \
-e ISTIO_OUT=${ISTIO_OUT} \
-e PACKAGE_VERSION=${PACKAGE_VERSION} \
-e USER_ID=$(shell id -u) \
-e GROUP_ID=$(shell id -g) \
istio-rpm-builder \
bash
In docker constainer bash, run script in tools/packaging/rpm/build-proxy-rpm.sh, found the issue was because CentOS's Git version is too low, so upgrade Git to latest version
Run tools/packaging/rpm/build-proxy-rpm.sh again, got C++ compilation errors, logs are attached.

Environment where bug was observed (cloud vendor, OS, etc)
RedHat 7

Fixed by #19119

duplicate of https://github.com/istio/istio/issues/16787


bypass greeter

Hi all ,
I'm using a single session with saved username and password on xrdp.ini .
It works but every time I login I have to hi the ok button.
I don't need auth as I'm running a chrooted read only app that have his auth.
Is there a way to directly open session without seeing the greeter ?
Thank's in advance,
marco.

Hi @neroita
No there isn't. This isn't a use-case we support.
Another way to solve your problem would be to edit the PAM file for xrdp-sesman and allow any password for the user. That would mean it wouldn't matter what password was stored in the RDP file.


Number of Routes significantly impacts performance

Current behaviour
I currently have 5 routes (tabs), and the initial tab makes an API (using redux-api-middleware) call on componentDidMount. Without the TabView I have a benchmark of 300 ms from componentDidMount. With the 5 routes, the timing takes 1.5s and with 9 routes its about 6 seconds.
It seems that the more routes I have, the worse the JS thread gets locked up and thus takes longer for the full redux-api-middleware lifecycle to complete. The degraded performance happens regardless of what is on the routes since I'm lazy loading and ensured they all return null in the renderScenes function.
Expected behaviour
Hoping for better performance, similar to that of react-native-scrollable-tab-view.
What have you tried
I've tried all of the performance optimization techniques in the readme.
I benchmarked with and without the TabView as well as benchmarking different number of routes.
Is anyone else having issues like this? Any suggestions or ideas on how to improve the performance?
Your Environment

react-native: 0.59.4
react-native-tab-view: 2.2.0
react-native-reanimated: 1.0.1
react-native-gesture-handler: 1.1.0

Please provide repro

I experienced this same problem because by default this library loads all of the tabs' contents, the easiest solution for me was to use a skeleton page when a route is not loaded and render the actual page when the index changes to it, this of course requires this open pr to work
This looks something like this:
renderRoute = ({ route }) => {
  const { index } = this.state;

  if (index !== route.index) {
    return (
      <SkeletonPage />
    );
  }

  const RouteComponent = route.component;
  if (RouteComponent) return <RouteComponent {...this.props} />
}

The biggest problem with loading all of the routes is the quantity of active state listeners increases by quite a bit, and some actions that cause major changes on one page get slowed down on other pages as a result.

I experienced this same problem because by default this library loads all of the tabs' contents, the easiest solution for me was to use a skeleton page when a route is not loaded and render the actual page when the index changes to it, this of course requires this open pr to work

There's a lazy prop which renders screens as you need.
There's also removeClippedSubviews which can help with memory usage on Android

The biggest problem with loading all of the routes is the quantity of active state listeners increases by quite a bit

Which listeners do you refer to? How many routes do you have? Number of listeners shouldn't be significant. If it's a problem, you could also remove the listeners from inactive pages.

and some actions that cause major changes on one page get slowed down on other pages as a result.

By slow down, do you mean the animation is slowed down? What gets slowed down? Can you provide a repro?

This could be #720 (see most recent comments)

@satya164
Can't use removeClippedSubview because of the iOS bug Don't enable this on iOS where this is buggy and views don't re-appear.

Which listeners do you refer to?
State listeners, redux or mobx or something else, those components will continue to be updated simply because they're "visible"

By slow down, do you mean the animation is slowed down?
No, the animations are fine, the slowdown is a whole app slowdown as a result of rendering and updating all of the views at all times, this is not a problem is you don't have a state system like redux. An alternative approach would be stop listening when views are offscreen, either way it requires knowing the current route as soon as possible.

It's possible that lazy handles this, if I have time next week I'll run some tests

the slowdown is a whole app slowdown as a result of rendering and updating all of the views at all times

What exactly slows down? How do you notice this showdown?
Also if few redux listeners are slowing down your app, you probably need to work on optimizing that. Otherwise you're going to run into issues one way or the other, for example inactive screens in stack navigator.

I am also having some perf issues.
I have lazy loading working - I can see it only fetches/renders the tab after I scroll to it.
However the more tabs I load, the slower the rest of my app gets, I think because it just holds onto them and never lets go. Load 1 => ok, Load 3 => a little slower, Load 6 => grinds to a halt.

@booboothefool yes the react-native-scrollable-tab-view has a significantly better performance. I've actually switched back to that library for the time being, but would prefer to use this library since it's actively maintained.
I ran some perf tests a couple weeks ago, but didn't find any conclusive data as to what the issue is. A large portion of native thread resources were spent on the gesture handling conditional nodes, but i still haven't figured out if/why that would slow down the tab views significantly.

Closing in favour of #720 as it seems similar and there is no repro provided in the issue. Please track the issue there.