text:
Removed KubeService creation

The creation of Kubernetes Services that front EDPM nodes ports was intended to be used primarily to configure Prometheus scraping endpoints. To take in account EDPM deployments with node counts in potentially in the thousands, EndpointSlice Kubernetes object was used, which allows the definition of 100 endoints and, when it is full, it just create a new EndpointSlice, as many as are needed.
Sadly, the cluster-observability-operator that we will be using to provide and configure Prometheus does not yet support reading endpoints from EndpointSlices, only Endpoint, which is limited to 1000 endpoints and then it just stops scaling. After a conversation with the cluster-observability-operator developers, we end up ditching the usage of Services and instead make the telemetry-operator read directly the endpoints from multiple EDPM configuration sources using the same code used here and configure an ScrapeConfig with all of them.
This renders the Kubernetes Service useless for Telemetry purposes and, as we were the only known use case for this feature, I am then submitting this PR to remove it.

I didn't hear any objections to removing this, so it's safe to proceed. I did fix the merge conflict by pushing to your private branch. When CI passes, it's good to go.

/test glance-operator-build-deploy-kuttl

/test dataplane-operator-build-deploy-kuttl

recheck

authors: danpawlik fao89 jlarriba slagle


text:
Use inclusive language for branch naming

In line with Red Hat's commitment to using inclusive language in our products. This change adds the necessary modifications to switch the default branch to use "main".

Branch renaming can be done as described here:
https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/managing-branches-in-your-repository/renaming-a-branch

Project is inclusive! But do we have to do this.... seems like a lot of disruption for not much gain to me.
Does RH intent to pepper this same thing across all of OpenShift too?

We have publicly stated that we plan to do this:
https://access.redhat.com/documentation/en-us/red_hat_openstack_platform/17.0/html/release_notes/making-open-source-more-inclusive
But from a practicality perspective, we have some of these operators using a main branch (newer ones) and some using a master branch(older ones from CNOSP). So for consistency sake, we need to change some of them anyway. It might as well be the ones using master? Then we're in-line with a "making open-source more inclusive" statement and we have nice consistent branch naming.
Happy to leave it up to the general consensus though. But that's my thoughts on the topic.

Most important thing to me is if we do it, we do it everywhere. And agree also that doing it sooner is better than later. But it will break some CI, scripts, and cause general developer disruptions. A slight nit is it isn't the GIT default. But folks can correct that with a local config setting I guess.

source: gharchive/pull-request

authors: bshephar dprince

license_type: permissive

license_source: github-api


text:
Migrate to OVNController from ovn-operator

OVNController CRD from ovn-operator replaces the old OVS CRD from ovs-operator.

@booxter can you fix merge conflicts and also club these commits to single commit to have clean history.
Also noticed an issue as ovncontroller pods logs not collected in [1], but as per [2] it's running, for this need to fix the script
[1] https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/origin-ci-test/pr-logs/pull/openstack-k8s-operators_openstack-operator/315/pull-ci-openstack-k8s-operators-openstack-operator-main-openstack-operator-build-deploy/1660747381668319232/artifacts/openstack-operator-build-deploy/openstack-k8s-operators-gather/artifacts/pods/
[2] https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/origin-ci-test/pr-logs/pull/openstack-k8s-operators_openstack-operator/315/pull-ci-openstack-k8s-operators-openstack-operator-main-openstack-operator-build-deploy/1660747381668319232/artifacts/openstack-operator-build-deploy/openstack-k8s-operators-gather/artifacts/get_all.log

@karelyatin rebased and squashed commits. As for log collection, I will take a look later.

/test

/test all

OVN specific changes looks good, but PR contains other already merged changes(8 commits) would be good to make it clean

authors: booxter karelyatin


text:
renamed subdirectory from "j2ee" to "adapter"

ディレクトリ名に含まれていた "j2ee" は、"adapter" にした方がよいというレビュー指摘の反映

マージします。

authors: naokiiiii yagiaoskywalker

repo: openstandia/keycloak-dockerfiles


text:
List of provinces and municipalities should be updated

Describe the bug
Summary
Currently results for Open Besluitvorming from the province limburg, as well as stadsdelen (amsterdam, rotterdam) and a handful of thers are not imported correctly. This happens because the location is not mapped to internal locations, which have not been updated since 2020.
URL / Environment
https://bron.live/
Steps to reproduce

Expected behavior
Results from Pprovincie Limburg should be in there, as well as Provincie Limburg. Voorne aan Zee should be a location too (although it is not in obv)
Screenshots

Additional context

List of muniipaltiesupdated, still having trouble with getting the results to load in. Stadsdelen are not loaded with a broad query (probably since the number of items is greater than 10.000), but are when the query is limited to a single id.

I am not sure if this is 100% related, but the results between ORI and Bron seem to differ for Limburg Province.
Screenshot

@breyten Had a chance to look at the latest comment?

source: gharchive/issue

authors: breyten vanderburgt

repo: openstate/jodal

license: MIT


text:
Adding a preset field "parking=*" (amenity=parking_entrance)

Adding a preset / field of parking=* (for main tag amenity=parking_entrance), see wiki:
https://wiki.openstreetmap.org/wiki/Tag:amenity%3Dparking_entrance#Additional_tags

@sun-geo @quincylvania I want to do this. What I understood is that I'll have to add the parking.json field to the fields key array in data/presets/presets/amenity/parking_entrance.json .
I am new to open source. I need help to understand the work to be done for this issue.

Done in #7909 by @brokemyspoke.

authors: nafees87n quincylvania sun-geo

repo: openstreetmap/iD

license: ISC


text:
Replace diacritics when doing fuzzy searches

tldr: it means that strings like "fussball" will fuzzy match strings like "fußball"
more details:
In collection.js search results are returned in the following order:

match name
match terms
match tag value
similar name
similar terms

Diacritical marks are only replaced when calculating the "similar" search results string distance, so this is a fallback strategy from strict string matches.
(closes #3159)
I also

[X] Expanded the tests to make sure results are returned in the right order
[x] Added a few tests to test the diacritical mark replacement on fuzzy matches

cc @1ec5

I tried this branch out with a few preset search terms that have been inconvenient in the past. The change produces a slight improvement over master. I haven’t found any real problems yet.
As a further test, I took this branch and removed the manually diacritic-folded Vietnamese terms using the regular expression (, [-a-z ,]+)(?=") on the presets object in vi.json. (I had to perform the find and replace manually because a handful of terms, like “TV”, “boong ke”, and “ga ra”, are supposed to lack diacritics.) It’s a wash, with significant improvements in cases where the search term mostly matches the diacritic-folded preferred term, but regressions in cases where the search term only partly matches the diacritic-folded preferred term or exactly matches a diacritic-folded secondary term. It may be possible to get better results by fiddling with the orders of terms in individual presets. I’ve pushed my changes to the diacritics-vi-folding-removed branch in my fork, but I understand that I’d have to apply the changes in Transifex for them to stick.

preferred
master
this branch
my branchsans manually folded terms

⬆ sân chơi⬇ công viên⬆ sân cỏ⬆ sân bóng đá

⬆ trường học⬆ nhà trường⬇ sân bay⬇ ga sân bay⬆ tường

⬇ trung tâm bơi lội⬇ tiệm dụng cụ bơi lội

⬆ nhà ở⬇ tòa nhà dân cư⬇ đất dân cư⬆ nhà thờ⬆ nha sĩ

@1ec5 Thanks for digging into this!

It’s a wash, with significant improvements in cases where the search term mostly matches the diacritic-folded preferred term, but regressions in cases where the search term only partly matches the diacritic-folded preferred term or exactly matches a diacritic-folded secondary term. It may be possible to get better results by fiddling with the orders of terms in individual presets.

Yes, this change definitely won't solve all problems, but it should be possible to work around the more common issues by using the "preset terms".
collection.js search() contains the code which prioritizes how matches happen.
Exact matching on the leading part of name and leading part of terms are stuff you can control by adjusting the strings in Transifex.  Exact matching of the leading tag value might cause more problems in other languages.  Maybe we should disable that unless the locale is en?  In your examples above, I don't think it's affecting the results.
I think the diacritic replacement has a more pronounced effect on 'ß' -> 'ss', because it normalizes the Levenshtein distance between the search strings.  For example before this change, "grass" looks 3 chars different from "glaß", and after this change, they only differ by 1 character.

it should be possible to work around the more common issues by using the "preset terms".

To clarify, the Vietnamese localization is already (ab)using the preset terms to include the main preset name, any synonyms, the main name diacritic-folded, and the synonyms diacritic-folded, in that order. I believe that's why this change has little effect.
Removing the diacritic-folded terms results in some results getting a lot better and some getting a lot worse, which in my opinion shows that the language-agnostic diacritic folding may be weighted too high for Vietnamese. (It would ideally count less toward the edit distance than base letter changes, whereas for other languages it should count more or the same.) To the extent that the workaround works, it's because we've specified a lot of synonyms in the Vietnamese presets.
So I'll probably keep the workaround in the Vietnamese localization (despite the bloat) and hold out for a more sophisticated solution in the future.


text:
Ignore invalid units in field input

If the user enters something into the unit box of the Speed Limit or Max Height field that isn’t one of the two recognized units, revert the unit to what it was before without changing the measurement system.
Fixes #9110.

Some builds are failing due to #9101, which is unrelated to these changes.


text:
chore: upgrade monaco version 0.45.0

Types

[x] 🎉 New Features

Background or solution

copilot:walkthrough

Changelog

copilot:summary

相关变动见：https://github.com/opensumi/core/issues/3213

repo: opensumi/core


text:
GCST003044 has been removed from GWAS Catalog

...due to a curation error, so is not found in our study table. However, I have summary statistics for it so it exists in our credible set and colocalisation table.
@mkarmona Will this break the API?

@edm1 having the study in the study table is a prerequisite for the api when a study object is required

List of all studies that have summary statistics but are not showing up in the study table:
GCST006363
GCST005068
GCST006348
GCST006358
GCST005072
GCST005063
GCST006352
GCST005070
GCST003497
NEALE2_6160_1
GCST006357
GCST005061
GCST006345
GCST006359
GCST005058
GCST006347
GCST006355
GCST005073
NEALE2_670_1
GCST006344
GCST006354
GCST005074
GCST005062
GCST006349
GCST005059
GCST005069
GCST005060
GCST005067
GCST006327
NEALE2_6142_7
GCST005065
GCST003044
GCST006362
GCST006350
GCST006938
GCST006346
GCST005071
GCST005066
GCST006351
GCST005353
GCST005064
GCST006328

Fixed by https://github.com/opentargets/genetics-v2d-data/commit/99fb8fdadaff0bf87c956837b179c482d790f9d6

repo: opentargets/genetics


text:
docs: minify plugin removed to prevent clash in local development

html minify had a clash with other dependencies that prevented generating documentation locally.
Remote (github) docs were still working because only doc-specfic dependencies are installed.

Codecov Report

Merging #284 (9c1b49e) into main (a4a44da) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #284   +/-   ##
=======================================
  Coverage   86.21%   86.21%
=======================================
  Files          85       85
  Lines        1952     1952
=======================================
  Hits         1683     1683
  Misses        269      269


text:
PIS transformation extraction: so step

Description
The so step downloads the sequence ontology.
Transformations PIS was doing
PIS was downloading the OWL version and using Apache Jena to convert it to JSONL. There is a JSON version, so we can use that one to avoid Jena and transform it into JSONL directly.
The query PIS used to convert Jena's output into JSONL is:
'.["@graph"][] | select (.["@type"] == "owl:Class" and .id != null)|.subClassOf=([.subClassOf]|flatten|del(.[]|nulls))|.hasExactSynonym=([.hasExactSynonym]|flatten|del(.[]|nulls))|.hasDbXref=([.hasDbXref]|flatten|del(.[]|nulls))|.inSubset=([.inSubset]|flatten|del(.[]|nulls))|.hasAlternativeId=([.hasAlternativeId]|flatten|del(.[]|nulls))|@json'

But we must adapt it to the so.json version, as it is not the same as Jena's output.
Tasks

[ ] Figure out where to put this transformation
[ ] Implement transformation
[ ] Test output is equal to the one legacy PIS generates

Json files are exposed in a github repo

The Json file needs to be mapped the structure is different to the owl file. I'll be using the extractor to extract 3 different inputs and create a new step in the ETL to process the inputs and produce the SO in the correct format.

The extractor changes are done and the mappings are being processed

After discussing with @d0choa the mapping has been simplified and the output now only contains id and label which are the only fields being exposed by the API

repo: opentargets/issues


text:
Revert "Update newly encrypted ak and sk"

Reverts opentelekomcloud-scs/zuul-config#14

recheck

repo: opentelekomcloud-scs/zuul-config


text:
Metagraph async

Async migration for the following modules:

metagraph.py
commands/metagraph.py
metagraph in subtensor

@ibraheem-opentensor hey I didn't get a chance to review this, but let's update this:
difficulty = await subtensor.difficulty(cli.config.netuid)
subnet_emission = bittensor.Balance.from_tao(
    await subtensor.get_emission_value_by_subnet(cli.config.netuid)
)
total_issuance = bittensor.Balance.from_rao(
    (await subtensor.total_issuance()).rao
)

to
difficulty, subnet_emission_, total_issuance_ = await asyncio.gather(
    subtensor.difficulty(cli.config.netuid),
    subtensor.get_emission_value_by_subnet(cli.config.netuid),
    subtensor.total_issuance()
)
subnet_emission = bittensor.Balance.from_tao(subnet_emission_)
total_issuance = bittensor.Balance.from_rao(total_issuance_).rao

We want to take full advantage of asyncio with big push.

authors: ibraheem-opentensor thewhaleking

repo: opentensor/bittensor


text:
[tests] unexpected test_advertising_proxy failing for openthread#10318

Describe the bug
Unexpected test_advertising_proxy failing for https://github.com/openthread/openthread/pull/10318
Console/log output
https://github.com/openthread/openthread/actions/runs/9779759835/job/27023842229?pr=10318
FAIL: test (__main__.SingleHostAndService)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/runner/work/openthread/openthread/tests/scripts/thread-cert/border_router/test_advertising_proxy.py", line 179, in test
    self.check_host_and_service(server, client, '2001::2', 'my-service')
  File "/home/runner/work/openthread/openthread/tests/scripts/thread-cert/border_router/test_advertising_proxy.py", line 301, in check_host_and_service
    self.assertEqual(client_service['state'], 'Registered')
AssertionError: 'Refreshing' != 'Registered'
- Refreshing
+ Registered

Another failing https://github.com/openthread/openthread/actions/runs/9791676882/job/27035979267?pr=10318#step:8:6748

Thanks, @librasungirl.
I noticed the same CI failures and submitted #10474 yesterday (before you filed this issue and submitted #10479). I guess you probably didn't see it. 🙂

authors: abtink librasungirl

repo: openthread/openthread


text:
Make disabled 'B' bit of SecurityPolicy take effect

To resolve #1228 and also make disabled 'R' to take effect.

Codecov Report

Merging #1438 into master will increase coverage by 0.33%.
The diff coverage is 55.26%.

@@            Coverage Diff             @@
##           master    #1438      +/-   ##
==========================================
+ Coverage   70.52%   70.86%   +0.33%
==========================================
  Files         142      143       +1
  Lines       19328    19348      +20
  Branches     2366     2365       -1
==========================================
+ Hits        13632    13711      +79
+ Misses       4804     4736      -68
- Partials      892      901       +9

Impacted Files
Coverage Δ

src/core/api/link_api.cpp
27.2% <0%> (-0.62%)
:x:

src/core/mac/mac_frame.hpp
62.79% <55.55%> (-1.32%)
:x:

src/core/mac/mac.cpp
79.63% <78.57%> (+0.52%)
:white_check_mark:

src/core/thread/mle.cpp
80.44% <83.33%> (-0.05%)
:x:

src/core/thread/lowpan.hpp
86.84% <0%> (-13.16%)
:x:

src/core/net/udp6.cpp
82% <0%> (-2%)
:x:

src/core/thread/lowpan.cpp
92.29% <0%> (-1.44%)
:x:

src/core/thread/network_data.cpp
87.54% <0%> (-1.14%)
:x:

src/core/thread/mle_router.cpp
79.59% <0%> (-0.43%)
:x:

examples/platforms/posix/radio.c
78.15% <0%> (-0.43%)
:x:

... and 16 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 1eb61a4...320ad1a. Read the comment docs.

updated to only fix #1228, will submit a separated PR for 'R' bit

authors: codecov-io librasungirl


text:
[openthread] update to 96fe3dd59

This commit updates OT to 96fe3dd59 to include https://github.com/openthread/openthread/pull/6268

The raspbian-check passed by chance :+1: . At least we can merge this PR now. @jwhui

repo: openthread/ot-br-posix

license_source: bigquery


text:
Which ESP32 should i get

Which ESP32 would be the best for using thread protocol through a radio co-processor design.

Which ESP32 would be the best for using thread protocol through a radio co-processor design.

You may find our ESP Thread Border Router guide helpful.

Closing stale issue.


text:
Provider: seal-io/courier

Provider Repository
seal-io/terraform-provider-courier
DCO

[X] I sign this project's DCO

does the "new" provider mean new to the world not new to the Opentofu registry?

cc @cam72cam

Hi @thxCode when we first set up the OpenTofu registry we wanted to make sure that thr existing providers continue working so it's easy for people to switch. To that end, we used the GitHub API to find and index all providers and that's likely how we picked up this one. Going forward, new providers will need to be added to the OpenTofu registry using this process though.

Hi @thxCode when we first set up the OpenTofu registry we wanted to make sure that thr existing providers continue working so it's easy for people to switch. To that end, we used the GitHub API to find and index all providers and that's likely how we picked up this one. Going forward, new providers will need to be added to the OpenTofu registry using this process though.

gotta, should i close this?

If you can, please test if the provider works with OpenTofu correctly and then feel free to close the issue.

authors: janosdebugs thxCode

repo: opentofu/registry


text:
okhttp client instrumentation

Hello,
I have done okhttp client 1 instrumentation. I would like to move it to contrib org if there is interest for it.

@pavolloffay sounds good to me – where is the instrumentation at the moment? What sounds good re a repo name?

It's not pushed anywhere. I can create a repo if you would like to have a look.
Repo name:
java-okhttp ?

Created: https://github.com/opentracing-contrib/java-okhttp/invitations

Closing, it is released and referenced from /meta.

authors: bensigelman pavolloffay

repo: opentracing-contrib/meta


text:
Multiple jax-rs instrumentations

There are multiple jax-rs instrumentation:

https://github.com/opentracing-contrib/java-jaxrs
https://github.com/uber/jaeger-client-java/tree/master/jaeger-jaxrs2 @yurishkuro
https://github.com/opentracing-contrib/java-dropwizard/blob/master/dropwizard-opentracing/src/main/java/io/opentracing/contrib/dropwizard/ServerRequestTracingFilter.java @bhs

https://github.com/opentracing-contrib/java-jaxrs ropadmap:

use with combination with https://github.com/opentracing-contrib/java-web-servlet-filter to capture errors like no URL mapping, unauthenticated requests, exceptions in filters...
in-process propagation, as intermediate solution maybe use spanmanager , once https://github.com/opentracing/specification/issues/23 is solved use what standard says.

Would it make sense for you to reuse java-jaxrs in 2. and 3.?

Would it make sense for you to reuse java-jaxrs in 2. and 3.?

Yes, but I would wait for things with span manager / global tracer to settle.

authors: pavolloffay yurishkuro


text:
Add release instructions to update change log

Added instructions to RELEASE.md that describe required changes for CHANGELOG.md during release process.

Coverage remained the same at 73.368% when pulling aff97d58a5eda07fc9747e7c7a0646085edded8d on MikeGoldsmith:master into 3233f4982174b3243457e2092c17354f17a3a95e on opentracing:master.

@yurishkuro I've moved the change log stage to after alerting there is a release happening and before creating a tag commit.

authors: MikeGoldsmith coveralls

repo: opentracing/opentracing-java


text:
🛑 [CN] OpenUPM Website Package Detail is down

In 0e4a266, [CN] OpenUPM Website Package Detail (https://openupm.cn/packages/com.littlebigfun.addressable-importer/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: [CN] OpenUPM Website Package Detail is back up in 7b6f4fe after 7 minutes.

repo: openupm/upptime-openupmcn


text:
🛑 [CN] OpenUPM Website Docs is down

In 64b78a2, [CN] OpenUPM Website Docs (https://openupm.cn/docs/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: [CN] OpenUPM Website Docs is back up in 92308c9 after 8 minutes.


text:
Is it possible to make jpostal available on maven repository?

Currently jpostal released as source code, it's hard to use it in any project. Is it possible to make it available on central maven repository?

Fine to publish it, though that may not make it much easier to install/use because of e.g. #3. As far as I can tell, most Java package managers need special coaxing to handle JNI libraries and java.classpath correctly, so it can't simply be included in a pom like a normal Java library and expected to work. If you use Gradle, this approach seems useful: https://github.com/cjstehno/coffeaelectronica/wiki/Going-Native-with-Gradle.
Generally I'm not a huge Java/JVM language user. For this binding, I'll maintain the JNI/C parts of the repo, but am very much relying on the community to help out with packaging, etc. Happy to merge pull requests if you want to give it a shot.

I agree, maybe add JNI library (not libpostal.so) into jar file itself? so reduce one dependency?

That should work.
Just before jpostal was released I was reading about packaging the native lib as part of a jar and came across this blog post, which might be useful (Github repo: https://github.com/adamheinrich/native-utils).
Also possibly of interest are the Gradle docs on publishing custom artifacts: https://docs.gradle.org/current/userguide/publishing_maven.html#sec:publishing_custom_artifacts_to_maven

I've been uploading to our internal third-party maven repo.
It's a simple one-liner in the main directory after the gradle build:
mvn deploy:deploy-file -DgroupId=mapzen -DartifactId=jpostal -Dversion=1.0 -Dpackaging=jar -Dfile=./build/libs/jpostal.jar -DrepositoryId=<repoID> -Durl=<repoUrl>
You can can specify whatever groupId, artifactId, and version you want but I think mapzen & jpostal make the most sense for the first 2.

Since there hasn't been any activity here, and I was too bothered distributing the library alongside our application, I took up the chore of publishing it on Maven Central. You can find the package there:
https://central.sonatype.com/artifact/com.matchory.packages/jpostal
Since I don't have control of the mapzen.com domain, I took the liberty to rename the namespace to com.matchory.packages; the code is unmodified otherwise, including the license document. I also added open venues as the original developer and pointed the SCM links to this repository. However, if any of the maintainers are unhappy with the publication, reach out to me via email (my first name, "moritz", below our company domain).
This is the final grade file required for publishing, adapted from the Dockerfile:
import org.gradle.internal.jvm.Jvm

plugins {
    id 'c'
    id 'java'
    id 'java-library'
    id 'application'
    id 'signing'
    id 'maven-publish'
}

group = 'com.matchory.packages'
version = '1.0.0'

repositories {
    mavenCentral()
}

java {
    withJavadocJar()
    withSourcesJar()
}

sourceSets.main.java.srcDirs = ["src/main/java"]
sourceSets.test.java.srcDirs = ["src/test/java"]

model {
    platforms {
        x64 {
            architecture "x86_64"
        }
    }

    components {
        jpostal(NativeLibrarySpec) {
            binaries.all {
                def jvmHome = org.gradle.internal.jvm.Jvm.current().javaHome
                if (targetPlatform.operatingSystem.macOsX) {
                    cCompiler.args '-I', "${jvmHome}/include"
                    cCompiler.args '-I', "${jvmHome}/include/darwin"
                    cCompiler.args '-mmacosx-version-min=10.9'
                    linker.args '-mmacosx-version-min=10.9'
                    linker.args '-stdlib=libc++'
                } else if (targetPlatform.operatingSystem.linux) {
                    cCompiler.args '-I', "${jvmHome}/include"
                    cCompiler.args '-I', "${jvmHome}/include/linux"
                    cCompiler.args '-D_FILE_OFFSET_BITS=64'
                } else if (targetPlatform.operatingSystem.windows) {
                    cCompiler.args "-I${jvmHome}/include"
                    cCompiler.args "-I${jvmHome}/include/win32"
                } else if (targetPlatform.operatingSystem.freeBSD) {
                    cCompiler.args '-I', "${jvmHome}/include"
                    cCompiler.args '-I', "${jvmHome}/include/freebsd"
                }
                linker.args '-lpostal'  // Link with libpostal
            }
        }
    }
}

classes.dependsOn 'jpostalSharedLibrary'

application {
    applicationDefaultJvmArgs = ["-Djava.library.path=" + file("${buildDir}/libs/jpostal/shared").absolutePath]
}

dependencies {
    testImplementation 'junit:junit:4.+'
}

tasks.withType(Test) {
    systemProperty "java.library.path", file("${buildDir}/libs/jpostal/shared").absolutePath
}

artifacts {
    archives javadocJar, sourcesJar
}

publishing {
    publications {
        mavenJava(MavenPublication) {

            groupId = 'com.matchory.packages'
            artifactId = 'jpostal'
            version = '1.0.0'

            from components.java

            versionMapping {
                usage('java-api') {
                    fromResolutionOf('runtimeClasspath')
                }
                usage('java-runtime') {
                    fromResolutionResult()
                }
            }

            pom {
                name = 'jpostal'
                description = 'Java/JNI bindings to libpostal for fast international street address parsing/normalization'
                url = 'https://github.com/openvenues/jpostal'
                developers {
                    developer {
                        id = 'openvenues'
                        name = 'Open Venues'
                        url = 'https://github.com/openvenues'
                    }
                }
                licenses {
                    license {
                        name = 'MIT'
                    }
                }
                scm {
                    connection = 'scm:git:git://github.com/openvenues/libpostal.git'
                    developerConnection = 'scm:git:ssh://github.com/openvenues/libpostal.git'
                    url = 'https://github.com/openvenues/libpostal'
                }
            }
        }
    }

    repositories {
        maven {
            name = 'com.matchory.packages.jpostal'
            url = layout.buildDirectory.dir("repo")
        }
    }
}

signing {
    sign configurations.archives
    sign publishing.publications.mavenJava
}

javadoc {
    if(JavaVersion.current().isJava9Compatible()) {
        options.addBooleanOption('html5', true)
    }
}

authors: Noodle05 Radiergummi mkaranta thatdatabaseguy

repo: openvenues/jpostal

license: mit


text:
Make pauses / silence inside singing

Hi,
Thank you for the great work! The results are really amazing!
I trained the model and now I have some questions about pauses inside the sample. Is it possible to synthesize a single sample with the silence inside? Adding SP does not help, because the right syllable is concatenated and the model simply sings (lengthens) the syllable, and does not pause it.

What editor are you using, and if you use *.ds files to synthesize, how did you get/generate them? Adding SPs is expected to work.

I don't use an editor. I create *.ds files programmatically. I looked in samples/ and it looks like I found an error with ph_num.
I connected SP as AP according to 3. Add ph_num attribute from making of variance model. In the samples/ I saw that ph_num for SP is equal to 1.
Thanks for the quick response!

authors: MrDiplodocus yqzhishen

repo: openvpi/DiffSinger


text:
Minor fixes and updates

Add retries to the CoreDNS validation task and for the task to create the Windows infra-container Docker image
Disable realtime protection for testing clusters

Signed-off-by: Ionut Balutoiu ibalutoiu@cloudbasesolutions.com

/LGTM

authors: aserdean ionutbalutoiu

repo: openvswitch/ovn-kubernetes


text:
upload logs to gcs after failure

Hi @gerickson @jaylogue @robszewczyk :
This is the PR to address issue of "not uploading logs after test failure", please help to review.
I have tested it on one of my test branch, which I did return value 2 by intention to make test fail everytime:
https://github.com/openweave/openweave-core/tree/lwip_upload_fail_log
result link for test branch when test failedis:
https://travis-ci.org/openweave/openweave-core/builds/557980883
Thanks
Jennie

I am testing this PR by changing return value to 2, so test will fail and prove it can upload test logs after test fail
I will remove "return 2" after test proved work.

I am testing this PR by changing return value to 2, so test will fail and prove it can upload test logs after test fail
I will remove "return 2" after test proved work.

@jaylogue @robszewczyk, I have confidence for uploading logs after test fail, prove is: build #1409 https://travis-ci.org/openweave/openweave-core/builds/558090243
uploaded logs to gcs:
https://pantheon.corp.google.com/storage/browser/openweave/happy-test-log/?project=openweave-test-project&authuser=0&organizationId=433637338589
next step:
I will remove "return 2" now and we can merge it to master.

I have removed "return 2" from ./travis/script.sh, and changed the gcs key to a new encrypted key.
The PR is ready for merge now.
https://github.com/openweave/openweave-core/pull/305

repo: openweave/openweave-core


text:
Can we use Vite + Vue3 + Typescript + Antd Vue together?

Should this be an RFC?

[X] This is not a substantial change

Description
I try to use Vite + Vue3 + Typescript + Antd Vue in webf, but it do not work. It repeat load css forever.
Alternatives and Workarounds
No response

We didn't provide ES6 Module support from now on, So Vite can't be used from now on.
Vue3 now works well except <transition></transition> component which depends on window.getComputedStyle API, which is not support in the latest version of WebF.
The JavaScript Engine that WebF use is QuickJS, and it support ES2022 standard. It's free to use TypeScript as well.
The Antd could not be works well from now on, because we didn't provide much more CSS features to drive antd.

authors: andycall xiaocode

repo: openwebf/webf


text:
Fix concurrent user activation throttler

ConsulClient failed silently and with the wrong exception
A failure in one call (not even related to the other calls) tore down the entire update logic

Note to self: Try to optimize the numbers of action invocations by stopping as soon as they get throttled.

@perryibm please take a look at the test specifically

3 suggestions:

Move the throttle window 1.minute to a constant and group it with the 60.   The time window of 1 minute and 60 firings are whisk limit properties and these can be adjusted as needed if we start changing the way we throttle.
The parameter maximumInvoke of 120 is really a testing parameter and perhaps shoud be restated as 2*maximumFirings to show that we decide to go at twice the throttle rate to trigger it.
Once something is throttled, it would be great if you could stop the Future's that have not been fired from occurring.  One way would be through a condition variable but I leave it to you to do it or not.  It would be good to know how long this test takes on average to complete.  In a perfect world, it would be 1 minute.  When one gets unlucky, it would be 2 minutes if we coded things well.  Since you've gone to the trouble if optimizing, can you indicate what you usually get?

Thanks @perry, i'll have a look at the constants.
To 3: Already did that, check the last commit. It stops as soon as a request is throttled.

Can we lift the rate limits out of the throttle and parameterize this test accordingly --- If we change the limits upward, will this test catch the lack of throttling?

@perryibm in the tests take somewhere between 4 to 5 minutes.

Can we move these bits from LoadBalancerToKafka to the RateThorttle?
                val userCount = activationThrottle.countForNamespace(subject)
                val userLimit = activationThrottle.limitForNamespace(subject)
                info(this, s"(DoS) current activation count for '$subject': $userCount (limit=$userLimit)")
                if (userCount > userLimit) {
                    info(this, s"(DoS) '$subject' maxed concurrent invocations")
                    Future.successful(throttleError)

@rabbah moved the code into the entitlement package, factored in the "long vs int" comment. Squashed to what i think are the 3 orthogonal bits in this PR.

PG1 219 is good.

Merged as https://github.com/openwhisk/openwhisk/compare/e5cf3c1806b8...2895a9626c84.

authors: markusthoemmes perryibm rabbah

repo: openwhisk/openwhisk


text:
Loadbalancer tweaks

Bumping active ack to 60 seconds (match blocking invoke timeout); with tighter invoker health check, active ack is short circuited when invoker health changes.

pg2/1235 🔵


text:
Luci Bug on Trunk /usr/lib/lua/luci/model/network.lua:1005

Update today cause ath9k drop issues after 12hours:
System:
OpenWrt SNAPSHOT r6349-9046e92 / LuCI Master (git-18.060.68662-3e17668)
TP-Link 3600
Log:
Fri Mar  2 14:19:50 2018 daemon.err uhttpd[2409]: The called action terminated with an exception:
Fri Mar  2 14:19:50 2018 daemon.err uhttpd[2409]: /usr/lib/lua/luci/model/network.lua:1005: bad argument #2 to '?' (no value)
Fri Mar  2 14:19:50 2018 daemon.err uhttpd[2409]: stack traceback:
Fri Mar  2 14:19:50 2018 daemon.err uhttpd[2409]:       [C]: in function 'assert'
Fri Mar  2 14:19:50 2018 daemon.err uhttpd[2409]:       /usr/lib/lua/luci/dispatcher.lua:487: in function 'dispatch'
Fri Mar  2 14:19:50 2018 daemon.err uhttpd[2409]:       /usr/lib/lua/luci/dispatcher.lua:121: in function </usr/lib/lua/luci/dispatcher.lua:120>
Error seen on NETWORK-> INTERFACES  Echo "Collecting ..."
Howto fix?
Thanks

Please provide the following SSH command output:
ubus call network.interface dump

Nevermind, fix pushed.

Fri Mar  2 14:53:53 2018 daemon.err uhttpd[2548]: The called action terminated with an exception:
Fri Mar  2 14:53:53 2018 daemon.err uhttpd[2548]: /usr/lib/lua/luci/model/network.lua:1005: bad argument #2 to '?' (no value)
Fri Mar  2 14:53:53 2018 daemon.err uhttpd[2548]: stack traceback:
Fri Mar  2 14:53:53 2018 daemon.err uhttpd[2548]:       [C]: in function 'assert'
Fri Mar  2 14:53:53 2018 daemon.err uhttpd[2548]:       /usr/lib/lua/luci/dispatcher.lua:487: in function 'dispatch'
Fri Mar  2 14:53:53 2018 daemon.err uhttpd[2548]:       /usr/lib/lua/luci/dispatcher.lua:121: in function </usr/lib/lua/luci/dispatcher.lua:120>
burned latested sysupgrade .. same
OpenWrt SNAPSHOT r6350-175b262328 / LuCI Master (git-18.060.68662-3e17668)
my mistake?

authors: jow- linuxonlinehelp

repo: openwrt/luci


text:
[Feature request]: luci-base: Allow sorting of DynamicList

Allow sorting of DynamicList.
A simple concept map:

I whipped something together. See no https://github.com/openwrt/luci/pull/7415 drag button though, use the actual item to drag.
Please test!

@Ramon00 Thank you for your attention to this request, I will test it and get back to you in the PR.


text:
routes.js  The translation is not displaying

How can we otherwise help?
I can see that Simplified Chinese and Polish are provided, but the translation is not displaying. This is after a fresh compilation and installation.
#: modules/luci-mod-network/htdocs/luci-static/resources/view/network/routes.js:96
#: modules/luci-mod-network/htdocs/luci-static/resources/view/network/routes.js:173
msgid ""
"A numeric table index, or symbol alias declared in %s. Special aliases local "
"(255), main (254) and default (253) are also valid"
msgstr "数值表索引，在 %s "
"中声明的符号别名。特殊别名local（255）、main（254）和default（253）是有效的"
#: modules/luci-mod-network/htdocs/luci-static/resources/view/network/routes.js:96
#: modules/luci-mod-network/htdocs/luci-static/resources/view/network/routes.js:173
msgid ""
"A numeric table index, or symbol alias declared in %s. Special aliases local "
"(255), main (254) and default (253) are also valid"
msgstr ""
"Indeks tablicy numerycznej lub alias symbolu zadeklarowany w %s. Specjalne "
"aliasy local (255), main (254) i default (253) są również prawidłowe"

Using OpenWrt 24.10 for the build.

It may be a coding error in
https://github.com/openwrt/luci/blob/09e8d5108ed678db5b940b1eb51dff2b9b6af189/modules/luci-mod-network/htdocs/luci-static/resources/view/network/routes.js#L96
The translate function "_" has parentheses that cover also the contents of .format ...
The closing parentheses might need to be before .format
I am not sure if that is right, or should that just be split into three parts, two translated strings and untranslated code in between.

Yeah, looks like a recent error from @systemcrash in a9e1685
I will change the place of the erroneous parentheses:
_('A numeric table index, or symbol alias declared in %s. Special aliases local (255), main (254) and default (253) are also valid'.format('/etc/iproute2/rt_tables'))
-->
_('A numeric table index, or symbol alias declared in %s. Special aliases local (255), main (254) and default (253) are also valid').format('/etc/iproute2/rt_tables')


text:
CMake configure time has increased a lot

By my count we are at about 15s for the configure step on my machine. This has taken it from mildly annoying to full on annoying.
I don't relish the idea of optimizing this but might be worth a look at some point.

I've noticed this too - I suspect maybe CUDA or clang? (could try disabling either and timing)

I think it happened around the clang side but wasn't paying a ton of attention.

We might want to measure and optimize both clean configure and reconfigure time.
CI logs with timestamps can help with clean configure, e.g. https://github.com/openxla/iree/actions/runs/4388479188/jobs/7685023493#step:4:206
21:13:17 start -> 21:13:59 end (42 seconds)

Setting this as P2 'needs scheduling' - please bump up in priority if needed.

I still see CMake configure take 40-60 seconds on my Linux and Windows machines sometimes :/ would be nice to get that down somehow.

authors: ScottTodd allieculp benvanik stellaraccident

repo: openxla/iree


text:
Fix runtime pointer direct cast

Add intptr_t cast in between uint64_t and the void* or flatbuffer pointer types.
To void the int-to-pointer-cast and pointer-to int-cast warnings when build with 32-bit system
iree/runtime/src/iree/hal/allocator_heap.c:237:13: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
  237 |       ptr = (void*)external_buffer->handle.device_allocation.ptr;
      |             ^

iree/runtime/src/iree/vm/bytecode/module.c:528:7: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
  528 |       (iree_vm_FunctionSourceMapDef_table_t)data[0];
      |       ^

iree/vm/bytecode/module.c:590:34: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
  590 |   out_source_location->data[0] = (uint64_t)source_map_def;
      |                                  ^

Thanks for the suggestion. PR updated.


text:
Disallow unranked dynamism in ops that take output shape as an operand

This includes:

dynamic_broadcast_in_dim
dynamic_iota
dynamic_reshape

#1991

Double checked bytecode compatibility; no issues detected.

repo: openxla/stablehlo


text:
Add interpreter for DotGeneralOp

closes #336

Merging with #307


text:
Adds the remote client address to server spans for gRPC

This address the feature request for #185

one test tidy if you don't mind, then let's merge this! thx
also highly appreciate if you can take a stab at formalizing the "CA" thing as you mentioned (in a different PR)

Done and done.  Will track "CA" under #205

nicely done!

authors: adriancole ewhauser

repo: openzipkin/brave


text:
Add missing component declaration in shared library

closes #294

Missing signed commit for this PR. Will recreate with appropriate commit signing.

repo: openziti/ziti-console


text:
Enable the Operator API by default

Description of the change:
Enables the Operator API by default in the v1 group.

/hold cancel
/retest

/retest

/lgtm

authors: dinhxuanvu ecordell njhale


text:
alm-examples json contains trailing commas

The alm-examples section in the CSV is generated from the yaml in config/samples and overwritten when the bundle is rebuilt. This is great. But the generated alm-examples section doesn't pass Red Hat's certification system for marketplace operators.
I've been told by Red Hat support (case 02870927) that my json fails because:
"The JSON in the alm-examples section contains four errors.  I was able to find these by taking the entire text from line 6 (the opening square bracket) to line 212 (the close square bracket) and using a JSON formatter/validator. Using the line with [ as line 1, you have extra trailing commas on lines 11, 15, 18, and 93."
You can see this json in github as well as the sample yaml that was used to generate it.
Currently I am thinking I will have to create a hacky script to remove the trailing commas in order to work around this.

Actually sorry I think it's something I'm doing that is adding these.

repo: operator-framework/operator-sdk


text:
Can't add a new api ->  'operator-sdk add api' can only be run for Go operators.

Bug Report
The following command
operator-sdk add api --api-version=component.k8s.io/v1alpha1 --kind=Component

executed within my project : https://github.com/snowdrop/component-operator/tree/operator-0.1.1
generates this error
2018/11/09 10:34:27 'operator-sdk add api' can only be run for Go operators.

Version used
operator-sdk version v0.1.1+git

@cmoulliard Hello, what kind of an operator were you building? Ansible or go?

go operator

This command also generates the same error
operator-sdk generate k8s
2018/11/09 10:57:21 'operator-sdk generate k8s' can only be run for Go operators.

when executed at the root of my project

@cmoulliard The error is because operator-sdk is expecting the main.go file to be at cmd/manager/main.go path.

Many thanks. Can you update please then the migration guide as I don't think that this is documented ?

@cmoulliard There is a mention of it, but feel free to maybe point out how we can be more clear in that so we can update it: https://github.com/operator-framework/operator-sdk/blob/master/doc/migration/v0.1.0-migration-guide.md#copy-changes-from-maingo Thanks!

I propose to do 2 things:

Add another section within the Guide or Migration Guide to mention that it mandatory that the main.go file is created under the path cmd/manager due to the fact the operator-sdk is scanning this path to determine if the project is a Go Operator vs something else
Update the message reported by the operator-sdk client to explain why it thinks that the project is not a Go Operator and what should be the rules validated

authors: LiliC cmoulliard


text:
Use https://github.com/einride/aip-go

this might allow us to maintain less code...

first PR #434

secodn PR #435

maybe also https://github.com/einride/aip-go/blob/master/pagination/pagetoken.go
	// Use pagination.PageToken for offset-based page tokens.
	pageToken, err := pagination.ParsePageToken(request)
	if err != nil {
		return nil, status.Errorf(codes.InvalidArgument, "invalid page token")
	}
        ...
	// Set the next page token.
	if result.HasNextPage {
		response.NextPageToken = pageToken.Next(request).String()
	}
	// Respond.
	return response, nil

maybe also https://github.com/einride/aip-go/blob/master/pagination/pagetoken.go
	// Use pagination.PageToken for offset-based page tokens.
	pageToken, err := pagination.ParsePageToken(request)
	if err != nil {
		return nil, status.Errorf(codes.InvalidArgument, "invalid page token")
	}
        ...
	// Set the next page token.
	if result.HasNextPage {
		response.NextPageToken = pageToken.Next(request).String()
	}
	// Respond.
	return response, nil

repo: opiproject/opi-spdk-bridge


text:
replace remaining legacy theme instances

git grep "/themes" | grep -v "ui/themes" | grep -v "usr.local.opnsense.www.themes"

Currently being used 53 times.
Currently being used 21 times.
Currently being used 17 times.
Currently being used 12 times.

The theme is already done. Remaining work are image to icon replacements.

@AdSchellevis @jschellevis we should wrap this up, last 12 instances are not my specialty though :)

repo: opnsense/core


text:
add some futures in unbound dns

hello  ,,
its seems GUI doesn't have an option for DNS over TLS yet, which will be a great add  !
also options like  Aggressive NSEC
https://tools.ietf.org/html/rfc8198
Bit 0x20 Support
https://tools.ietf.org/html/draft-vixie-dnsext-dns0x20-00

There is a DNSCrypt plugin that can be used for DNSoverTLS. In that case you can either use that one as DNS server or you can pass Unbound towards DNSoverTLS like mentioned on this page: https://forum.opnsense.org/index.php?topic=10670.0

1 - using two DNS servers is quite annoying :/ , i believe it better to relay on only one , while UNBOUND is supporting TLS, beside other privacy/security features.
2 - i was able to activate DNS over TLS by filling TLS server list , but it will not work unless remove servers list form general configuration (system itself) . i think it's going to be better if unbound able to grab system dns server list, and start DoT if option selected , Thoughts?

@AdSchellevis
is any thing is not clear about adding option for  Aggressive NSEC and , Bit 0x20 Support ?
1 -  Aggressive Use of DNSSEC-Validated Cache:
This increases performance, decreases latency, decreases
resource utilization on both authoritative and recursive servers, and
increases privacy.
2 -  Use of Bit 0x20 in DNS Labels to Improve Transaction Identity .
Thoughts?

best read the contribution guidelines about issues and feature requests (https://github.com/opnsense/core/blob/master/CONTRIBUTING.md)

authors: AdSchellevis Bramzor amrahmades


text:
Unbound-plus feature request: redirect denied URLs to local block page

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:
[X] I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
[X] I have searched the existing issues and I'm convinced that mine is new.
[X] When the request is meant for an existing plugin, I've added its name to the title.
Is your feature request related to a problem? Please describe.
When using a DNSBL, the result of a blocked site is a "ERR_NAME_NOT_RESOLVED" page. Although this is a perfectly functioning block, the error message is not very clear to a user
Describe the solution you'd like
I would like to see a block page, which indicates that the URL has been blocked by an administrator due to an enforced policy instead of showing the NOT_RESOLVED error.
Additional context
Some random internet screenshots that present a rough idea:

And where is this site hosted? There is more behind this vendor feature. No idea how to do this without running a website on OPNsense itself

Maybe I am oversimplifying things, but isn't Lighttpd already serving Captive portal and GUI pages? Is it unsafe if it serves a ' dead end'  webpage also?

The problem is that the current script sets the IP address, this would need to be changed to a templated value chosen from UI. Also when this would be implemented in any way then there'll be the next wish to make the html page customizable.
This would also mean templated files in /usr/local/www .. I don't think any of core member will accept this.

I understand your concerns and the implications, let's abandon the request as it has more implications than initially thought of.

authors: Northguy mimugmail

repo: opnsense/plugins


text:
Feature Request -- Safe Mode (Auto Rollback Changes if connection lost or not re-established within X time)

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

I realize I am duplicating a post, however, I assumed when I created a reply it would re-open the closed issue. That appears not to be the case and I simply hoped that opening a new issue would generate more feedback than a closed one. Should the original issue be able to be re-opened I am happy to close this.
Is your feature request related to a problem? Please describe.
I have multiple sites that am having to make several changes that could break access (Network address changes, Firewall changes, Routing, Etc.)  ........ I'm scared. HAHA!
The remote sites have 0.75% technical experience. Yes, I might be able to find someone to power cycle a router. However, I've also worked as an MSP and had the resident "IT person" unplug the network cable to "reboot" which they thought worked because "the lights went off and came back on" (Link Lights) #facepalm
Needless to say, took me a bit to figure out that's what they were doing.
Describe alternatives you considered
To my knowledge OPNSense doesn't have any reasonable alternative to this particular problem. If I am wrong, my apologies for wasting time and please help direct me in a correct path.
Additional context
Originally posted by @tcsi-github in https://github.com/opnsense/core/issues/3042#issuecomment-1119297907
I also created a forum post regarding this but no reply as of yet.
https://forum.opnsense.org/index.php?topic=28238.0
@banym had a good template as well and I include his here for comparison. Personally, I feel like the simpler thing would be to take a snapshot as soon as the breaking change option is enabled. I can see the use in giving the option for a timer, however, I would think setting a "middle-road" 120 seconds would be a default.
Describe the solution you'd like
It would be nice to lock the firewall in a "major change" mode where only one session is able to do changes until the major change mode is exited. This mode should be able to define a working configuration from the backup config history or the current configuration when this specific mode is activated. It should be possible to set a timer for change commitment. Now the administrator can for example make significant changes to routing or rules that possible could lock him out of the firewall. If he does not approve that his change was successful and works as intended the firewall roles back to the defined configuration. This way the administrator can log in an "try" again or rethink his change
Describe the solution you like
I was thinking about my MikroTik days and remember that if I happened to make a incorrect change that those changes would be reverted if I failed to connect back or didn't apply them within X time. Sometimes it was a pain, but it saved my butt so so many times. Kept me from having to make calls to talk someone through a reboot and just gave me a bit of room to breathe.
I would submit something the below as a rough draft for an option.
The ability might stay disabled by default and only enabled prior to the change. This would allow the option to stay out of the way and only be used when explicitly needed.
@fichtner -- I believe this to be a worthwhile addition to the OS and would be a step in the direction of more enterprise use-cases. My development skills are limited, however, I would commit to time testing and anything within my abilities to help this become a actual feature.
I would welcome any thoughts or feedback on the suggestion.
Example Change --->  Admin has to change a WAN IP

Scenario - Successful Change

Admin logs in and enables "Breaking Changes Mode" (Or whatever better name)

The working config is immediately snapshot.
Notification alert is placed at the top bar
No other logins are allowed but the user
Working config is marked as the config to be used for restoration
Perhaps the connection is marked for monitor?
Timer Starts (120 second?)
Notification Alert shows at top (See above)

Admin makes changes just as they normally would

Complete with "Apply Changes", etc.

Tests changes and is happy with results

Connection either not broken or reconnected within timer

Admin disables "Breaking Changes Mode"

Previous snapshot config is then removed to prevent use on reboot.
System returns to normal operation
Notification removed from top bar

Now lets try a screw-up....

Scenario - Failed Change

Admin logs in and enables "Breaking Changes Mode" as before

Same as above happens to enable feature

Admin makes changes just as they normally would

Complete with "Apply Changes", etc.

Finds they have made a mistake and are unable to connect back to the GUI

Connection timer expires, or some other defined trigger

"Breaking Changes Mode" begins

Previous Snapshot is automatically restored
Router is Rebooted (if needed)
Other actions such as logging the issue, etc...

Admin wait for completion of restore
- Perhaps an email or other notification could be sent notifying all is well again.
Admin is able to start again, thankful they have not created a bigger problem requiring an on-site visit (For me, an EXPENSIVE multiple day trip)

Thank you sir!

I may look in to implementing this for OPNsense routers on ZFS filesystems, but no guarantees. Will update if I decide to take this on

yeah I figured ZFS snapshots would be the way to go. I've heard that UFS supports snapshots as well, but don't know much about that

Anyone experienced in writing plugins for opnsense? Could try and create something together for zfs snapshots and rollbacks.

I think the first step would be a plugin that you can interact with in the UI to manually snapshot and restore snapshots. Once we've got that, hooking it in to system events should be a little less... painful? I have no experience with OPNsense plugins but will look around and see if I can get some knowledge to start building a base for that or something

see https://github.com/opnsense/plugins/tree/master/devel/grid_example and https://github.com/opnsense/plugins/tree/master/devel/helloworld. I'm poking around those rn

Yep, that sounds like a good way to start.

and here are some more docs. I'll see if I can create a bit of a template interface for this, give me a bit https://docs.opnsense.org/development/examples/helloworld.html

If the backend is in python - does it have an API for zfs or do we need to call zfs shell commands?

I suspect we'll have to use zfs shell commands, even if there are python libraries for interacting with zfs. see here for configd which we would probably use to do this: https://docs.opnsense.org/development/backend/configd.html.
It would be nice to use a pip package, unsure on what we are allowed to do with that

am going to try to develop here an interface for the manual component of this: https://github.com/CorvetteCole/opnsense-plugins-snapshot
would be nice to have an experienced plugin dev help create the interface though. Will update this issue with info once I get this package sort of set up

here it is on my branch. Calling it opn-snapshot for now but none of this is set in stone. I'll try to work out a front-end when I have time. I also know how we will implement automatic rollbacks although I think it will only be able to recover if the system boots successfully, unsure if we will be able to save from failed upgrades.
I've added you as a collaborator on the repository so you are welcome to work on what you find interesting as well. We need to figure out some sort of framework for how we will call for zfs snapshot restore and creation, as well as ways to query what snapshots are available etc. Obviously some thought to be done
https://github.com/CorvetteCole/opnsense-plugins-snapshot/tree/master/sysutils/opn-snapshot

I also know how we will implement automatic rollbacks although I think it will only be able to recover if the system boots successfully, unsure if we will be able to save from failed upgrades.

That's why I raised the question about adding this to the boot menu as well. If opnsense does not come up properly one could choose another dataset as root from the boot menu.

We need to figure out some sort of framework for how we will call for zfs snapshot restore and creation, as well as ways to query what snapshots are available etc. Obviously some thought to be done

We might want to talk to the opnsense-core guys to change the zfs hierarchy as depicted in my comment earlier today. Would make finding the right datasets easier.

You're totally right about it being a good idea to discuss this with them. I think this kind of feature does belong in core, especially if we want deep integration with the system. I do think a plugin offers a nice testbed for at least a good portion of what we want to implement

As long we snapshot the right datasets and subscribe to the right signals we might keep this completely in a plugin - only the prerequisites should be set properly, and that's what datasets (aka folders) opnsense uses and how they are named. This might change anytime in future so it would be easier to have them in a common branch (that's zroot/prod in my example). Otherwise we would have to  maintain the list of folders somehow manually, which I don't like ...
Of course one could break the entire setup if they did a complete manual zfs installation but there are ways to fix this as well - later.

Let's see what opnsense-core respond to that idea...

For now I expect we'll just wait for a pull-request (which we can review). The ticket has moved to plugins for good reasons, if there are specific questions  you can always try to ping, although accompanied with code it's often easier to point people in a direction.
As mentioned earlier, we might add some feature in our business edition in the future, which doesn't mean the community shouldn't try to come up with it's own plugin at all, just that our time is limited and we need to choose carefully where to spend it on. Plugins are the responsibility of the author, which is also one of the reasons pull-requests on core are less likely to be accepted for this. If core functionality is missing to support a specific plugin or at some point it turns out some parts would better be served from core, we can always discuss.

Sounds very good to me. For the time being I'd like to understand the idea behind the various folders set up and mounted from the zfs pool. Are there any informations about that to point me into the right direction? Or is that "just" FreeBSD behavior? Also I'd like to understand the rationale behind the zfs structure, i. e.
zroot
zroot/ROOT
zroot/ROOT/default
zroot/tmp
zroot/usr
zroot/usr/home
zroot/usr/ports
zroot/usr/src
zroot/var
zroot/var/audit
zroot/var/crash
zroot/var/log
zroot/var/mail
zroot/var/tmp

Could that be changed to something different or is it needed to support something else?

default FreeBSD behaviour sounds about right, good place to start reading is likely https://docs.freebsd.org/en/books/handbook/zfs/  , I believe the source can be found here : https://github.com/freebsd/freebsd-src/blob/a320e9dd51e6e58715a1390f2e00790a94017dfb/usr.sbin/bsdinstall/scripts/zfsboot#L140-L175

default FreeBSD behaviour sounds about right, good place to start reading is likely https://docs.freebsd.org/en/books/handbook/zfs/ , I believe the source can be found here : https://github.com/freebsd/freebsd-src/blob/a320e9dd51e6e58715a1390f2e00790a94017dfb/usr.sbin/bsdinstall/scripts/zfsboot#L140-L175

Checked the docs. I assume you use the FreeBSD install script without modification and wouldn't like to modify it? Am I right?
From what I've seen snapshotting and running an opnsense update only the root filesystem (zroot/ROOT/default) got changed. Is it save to assume that snapshotting and rolling back just this filesystem would be enough to return to a previous state?

Checked the docs. I assume you use the FreeBSD install script without modification and wouldn't like to modify it? Am I right?

yes? https://github.com/opnsense/src/blob/22.1/usr.sbin/bsdinstall/scripts/zfsboot#L141-L170

From what I've seen snapshotting and running an opnsense update only the root filesystem (zroot/ROOT/default) got changed. Is it save to assume that snapshotting and rolling back just this filesystem would be enough to return to a previous state?

You have to figure that out on your own, with specific questions I meant OPNsense specifics, can't really help you with the zfs part.

You have to figure that out on your own, with specific questions I meant OPNsense specifics, can't really help you with the zfs part.

The zfs part I can figure out myself - the question was if an update of opnsense only touches files in / but not /tmp, /usr and /var?

I quickly checked some update packages from https://pkg.opnsense.org/FreeBSD:13:amd64/22.1/sets/. From what I can see, the update changes files in / (obviously) and some other folders being part of the zroot/ROOT/default dataset. And there are files being modified in /tmp, /usr and /var mounted from other zfs datasets.
Would it be a huge trouble to change the zfs hierarchy from 'zroot/' to something like zroot/prod/?

I've added you as a collaborator on the repository so you are welcome to work on what you find interesting as well. We need to figure out some sort of framework for how we will call for zfs snapshot restore and creation, as well as ways to query what snapshots are available etc. Obviously some thought to be done
https://github.com/CorvetteCole/opnsense-plugins-snapshot/tree/master/sysutils/opn-snapshot

Could you please set up some issues/discussion/etc. lists there so we can have our thinking over there?

unstale!

Anyone else interested in that topic? I can help with zfs but would need somebody to work on the plugin part.

I can't work on this anymore, I do not have the time unfortunately. I am very sorry!

what about having a minimal OpnSense in a separate partition?
OR a second instance of a webui running with default confs

what about having a minimal OpnSense in a separate partition? OR a second instance of a webui running with default confs

That's what we can get with a zfs snapshot to boot from.

@OPNsense-bot reopen

+1

I'm interested to work on a plugin. However, I'll probably go for an extremely simplified approach in the initial version. Something like the reload in 5 feature found in some routers/switches. However, that's not going to be released "soon"... so if someone wants to start early on a new plugin, just do it and submit a PR. We'll see whoever is the first to release it. :D

I'm interested to work on a plugin. However, I'll probably go for an extremely simplified approach in the initial version. Something like the reload in 5 feature found in some routers/switches. However, that's not going to be released "soon"... so if someone wants to start early on a new plugin, just do it and submit a PR. We'll see whoever is the first to release it. :D

I am not familiar with plugin development for opnsense but can try to help with the zfs part although I decided to keep my own full virtualized environment setup rather than switching to a separate HW with a zfs filesystem for opnsense.

The development version of a new Auto Recovery plugin is now available for testing. See https://github.com/opnsense/plugins/pull/3321 for installation instructions. Please report success and/or bugs in https://github.com/opnsense/plugins/pull/3321.

My proposal for a new plugin was rejected. Unfortunately, I cannot put more time into this.
If someone wants to step in and make the required code modifications, the code is still available:
https://github.com/opnsense/plugins/pull/3321
https://github.com/fraenki/plugins/tree/auto_recovery
I'll unassign myself from this issue. If someone wants to take ownership, please let me know. Otherwise the issue may be automatically closed due to timeout.

authors: AdSchellevis CorvetteCole allanlaal fraenki jeremyrnelson spi43984 tcsi-github


text:
acme-client: add support for 1984Hosting API

This PR adds support for dns_1984Hosting API.

Merged, thanks!

authors: fraenki phedoreanu


text:
Setup a way to measure macro performance

@propensive created a library call (probably)[https://github.com/propensive/probably] that can measure how long it takes to generate macros.
It woould be useful to measure the performance of GenLens and compare it to the new Focus macro https://github.com/optics-dev/Monocle/issues/954.
See some discussion on Twitter: https://twitter.com/propensive/status/1329745251679035394

Encouraged by this, I've just published Probably 0.8.0 which has simple support for running compiletime tests. You'll need to add a dependency on com.propensive:probably-cli:0.8.0 and then,
import probably._

Then, in your macro definition, wrap an expression you want to check with,
ctx.test("test name") {
  <expression>
}.check { _ => true }

Currently the predicate on the check is required. It operates on the expression, and just returns a boolean. If you're more interested in timings (or counts) than properties of the expression's result, then just return true.
That should be all that's necessary to have a compiletime report printed at the end of compilation.

picking this up

Give me a shout if you need any help, @asjad02!

@propensive
couldn't get the tabular output:: sbt compile, is there any additional info needed in build.sbt (PR)

done, code is in this branch: https://github.com/optics-dev/Monocle/tree/benchmark-macro
Thanks @asjad02  and @propensive

authors: asjad02 julien-truffaut propensive

repo: optics-dev/Monocle


text:
chore: fix version error in CHANGELOG

Summary

fix version error in CHANGELOG

Coverage remained the same at 98.138% when pulling 38c35eaba70fc03f0756ba29380b5eb5b43ecfed on jaeopt-patch-1 into eef1206e36b1372e8f1d0ab174439b309c7a8d82 on master.

authors: coveralls jaeopt

repo: optimizely/objective-c-sdk


text:
Allow uglify mangling

Now you can mangle "official-addthis-angularjs.min.js" file and reduce its size without problem. (5.25Ko instead of 8.79Ko)

The (manual) build provided in this PR to not mangle yet.
You probably have to remove "mangle: false," from your "gulpfile.js" and launch a new build.
Sorry I have not launched tests and build, I can only say that it's working on my website.

Thank @jpierront. Can we get this pull request merged??

authors: jpierront selected-pixel-jameson

repo: oracle/addthis-angularjs


text:
This plugin makes Android give an error

After having installed this plugin my app started working on iOS (loading mp3 files) but stopped working for android giving an error:
I/chromium: [INFO:CONSOLE(535)] "Fetch API cannot load file:///android_asset/www/statics/sound/piano/C6.mp3. URL scheme "file" is not supported.", source: file:///android_asset/www/js/vendor.js (535)

@martijnmichel This plugin is targeted to be used only on iOS platform. It is not supposed to be installed on an android app.

How can I install it on iOS only?


text:
WCS: An extra slash on DB_CONNECTSTRING format

README.md says like that and I confirmed it works:

This is the Database connection string:
DB_CONNECTSTRING=<hostname/containername>:1521/ORCLPDB1.localdomain

Though wcsitesadminserver.env.listhas following format with an extra /:
https://github.com/oracle/docker-images/blob/24d81205939961f330028f1b6f78d675d53851d4/OracleWebCenterSites/dockerfiles/wcsitesadminserver.env.list#L11

@ashishchalak this looks like something you should fix.

This is fixed. Can we please close this issue.

Thank you.

authors: Djelibeybi Makopo ashishchalak

repo: oracle/docker-images


text:
Rename config auth.key_passphrase to auth.passphrase

auth.key_passphrase does not follow our naming conventions.
See: https://github.com/oracle/oci-cloud-controller-manager/issues/142

Also update documentation where relevant e.g. README.md

authors: owainlewis prydie

repo: oracle/oci-volume-provisioner


text:
JMSServer/PersistentStore is not discovered.

With Online Discover domain, JMSServer/PersistentStore is not discovered.
####<Mar 8, 2021 3:48:47 PM> <INFO> <JmsResourcesDiscoverer> <get_jms_servers> <WLSDPLY-06471> <Adding JMS Server SOAJMSServer_auto_1>
####<Mar 8, 2021 3:48:48 PM> <INFO> <Discoverer> <_populate_model_parameters> <WLSDPLY-06154> <Attribute PersistentStore for model folder at location /JMSServer is not in the lsa map and is not defined with get to retrieve the attribute value>

As the PersistentStore is not discovered, it is defulted to FileStore instead of JDBCStore.
Adding manually PersistentStore in the model, did not set the PersistentStore in the created domain still defaults to FileStore.

Fixed by #850

authors: brahankv ddsharpe

repo: oracle/weblogic-deploy-tooling


text:
add readiness check after ingress is created

In the current test framework, after the ingress is created, we just check the ingress exists but not check the ingress can actually route the app to the server pod.
Add the readiness check for ingress by calling WebLogic ready app and wait for 200 response code.

looks good, please update the PR with Jenkins results.

authors: vanajamukkara xiancao

repo: oracle/weblogic-kubernetes-operator


text:
how  to clear the input

i want to use you plugin but how to clear the text ,i try set the input.value = "" , but that doesn't work

InputField.setText('')

thank you very much

authors: AleBles HaiboLee

repo: orange-games/phaser-input


text:
No activity detected in Iframes

Doesn't work when used with a page that has a iframe. (I.E. CKEditor etc).

This is correct and is not a bug. Javascript is (to my knowledge) unable to catch any events happening in an iframe (keyboard or mouse). The best it can do is detect mouse enter on a frame and mouse leave but that still doesn't let you know what's happening when the user is over the iframe (is he active or not).
If you are however able to create a pull request that adds this ability I'd be happy to merge it.

repo: orangehill/bootstrap-session-timeout


text:
feat: dangling node detection for graph.Memory

Part of #472
This PR:

Adds doc for fields of graph.Memory
Make graph.Memory.Remove() return dangling successors

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

Comparison is base (79a08b4) 75.23% compared to head (407d871) 75.31%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #652      +/-   ##
==========================================
+ Coverage   75.23%   75.31%   +0.07%
==========================================
  Files          59       59
  Lines        5480     5485       +5
==========================================
+ Hits         4123     4131       +8
+ Misses       1001      999       -2
+ Partials      356      355       -1

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

close this PR as its content is contained in #653

repo: oras-project/oras-go


text:
oradb-manage-db failed

Hi
I try to install and create Oracle database 18c and it failed with this below error.
I just download ansible-oracle-master.zip and ansible-oracle-module.zip (extracted to ansible-oracle-master/library) and place Oracle 18c software to /tmp and then run the single-instance-db-on-fs.yml (already configured hostname).
My platform is Oracle Linux 7.5
TASK [oradb-manage-db : manage-db | create/manage database] ********************
fatal: [anstest]: FAILED! => {"changed": false, "cmd": "/u01/app/oracle/12.2.0.1/db1-base/bin/sqlplus -V", "msg": "[Errno 2] No such file or directory", "rc": 2}
Why it would create by using 12.2 version? How can I fix this?
Thanks

Hi - I know what the problem is. I’ll fix this asap

Thank you

Should be fixed with 98cc3175d541109d5d4cd84a253efafe2f24bd80

Hi,
I tried running again and the issue has been solved. Thank for that.
but I got new issue below.
TASK [oradb-manage-db : manage-db | create/manage database] ********************
fatal: [anstest]: FAILED! => {"changed": false, "msg": "Could not connect to database - ORA-12541: TNS:no listener, connect descriptor: (DESCRIPTION=(ADDRESS=(PROTOCOL=TCP)(HOST=ansible-srs)(PORT=1521))(CONNECT_DATA=(SERVICE_NAME=orcl)))"}
I checked my /etc/hosts and saw
127.0.0.1   localhost localhost.localdomain localhost4 localhost4.localdomain4
::1         localhost localhost.localdomain localhost6 localhost6.localdomain6
192.168.122.1 anstest anstest
My ifconfig:
**enp0s3: flags=4163<UP,BROADCAST,RUNNING,MULTICAST>  mtu 1500
inet 192.168.1.101  netmask 255.255.255.0  broadcast 192.168.1.255
inet6 fe80::a04e:5ebd:6ed0:dce5  prefixlen 64  scopeid 0x20
ether 08:00:27:77:4a:e9  txqueuelen 1000  (Ethernet)
RX packets 6635  bytes 5277409 (5.0 MiB)
RX errors 0  dropped 0  overruns 0  frame 0
TX packets 1743  bytes 144959 (141.5 KiB)
TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0
lo: flags=73<UP,LOOPBACK,RUNNING>  mtu 65536
inet 127.0.0.1  netmask 255.0.0.0
inet6 ::1  prefixlen 128  scopeid 0x10
loop  txqueuelen 0  (Local Loopback)
RX packets 1719499  bytes 13731524274 (12.7 GiB)
RX errors 0  dropped 0  overruns 0  frame 0
TX packets 1719499  bytes 13731524274 (12.7 GiB)
TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0
virbr0: flags=4099<UP,BROADCAST,MULTICAST>  mtu 1500
inet 192.168.122.1  netmask 255.255.255.0  broadcast 192.168.122.255
ether 52:54:00:ae:d8:4f  txqueuelen 0  (Ethernet)
RX packets 0  bytes 0 (0.0 B)
RX errors 0  dropped 0  overruns 0  frame 0
TX packets 0  bytes 0 (0.0 B)
TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0**
I configured the /etc/hosts as below before running the playbook I swear.
192.168.1.101 anstest anstest
Why do my /etc/hosts have configured to 192.168.122.1 after running the playbook? Is it relate to the issue of failed above?
Thank you

I think that this is related to a missing listener configuration.
2 solutions:

start a listerner and restart the playbook
configura a listener in oracle_databases and other variables for the listener

I always configure dedicated listener in my setups, because it will create a lot of issues when we try to find solutions for default configurations without a dedicated listenerdefinition.
Example for my 12.2 configuration: (oracle_databases includes only the mandatory variables for a listener definition!)
oracle_databases:
      - home: 12201-latest
        oracle_db_name: testdb
        listener_name: LISTENER
        listener_port: 1521

oracle_listeners_config:
   LISTENER:
        home: 12201-latest
        address:
          - host: "{{ansible_hostname}}"
            protocol: TCP
            port: 1521

listener_installed:
      - home: 12201-latest
        listener_name: LISTENER
        state: present

How can I do that?  Sorry I am beginner in Ansible.

Hi
I have tried to uninstall and re-install Oracle Linux 7.5 with Ansible again. I removed the virbr0 interface first before running Ansible playbook for installing and creating database and it worked!!!.
Just a question, how to delete database? I tried delete-db.yml and it resulted this below error:
**[ansible@anstest ansible-oracle-master]$ ansible-playbook delete-db.yml
PLAY [Delete Database(s)] *******************************************************
TASK [Gathering Facts] **********************************************************
ok: [anstest]
ERROR! 'configure_cluster' is undefined**
Thanks

Usemanage-db.ymlinstead. I'll remove delete-db.yml, it's an old version.
The state of the database in question needs to be set to absent (state: absent)
e.g

oracle_databases:
   - home: 12102-base
     oracle_db_name: db121
     .......
     state: absent  <--- THIS

Hi
I tried to re-install again and will use your suggestion for deleting database. But I got an error below:
TASK [orahost : Add host to /etc/hosts if needed] ******************************
fatal: [anstest]: FAILED! => {"msg": "The task includes an option with an undefined variable. The error was: list object has no element 1\n\nThe error appears to have been in '/home/ansible/ansible-oracle-master/roles/orahost/tasks/main.yml': line 75, column 5, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n\n  - name: Add host to /etc/hosts if needed\n    ^ here\n"}
My /etc/hosts
127.0.0.1   localhost localhost.localdomain localhost4 localhost4.localdomain4
::1         localhost localhost.localdomain localhost6 localhost6.localdomain6
192.168.1.104   anstest    anstest
Please advice.

Hi
I ignored orahost : Add host to /etc/hosts by commenting the task in main.yml in orahost role. Then I can create database successfully.
Finally I can delete database by follow your suggestion.
Thank you so much

authors: Rendanic dddd45 oravirt

repo: oravirt/ansible-oracle


text:
Browser ipfs-http-client to go-ipfs localhost node... getting status 400 bad request

browser webapp
"orbit-db": "^0.24.2",
"ipfs": "^0.47.0", also tested 0.46.0 result is same.
go-ipfs
0.6.0
Error:
http://127.0.0.1:5001/api/v0/pubsub/sub?arg=%2Forbitdb%2FzdpuB2sR2Q1J1fU2LpKkUp7NC1zfP4jj9RQHVQbeaVSowtPAy%2Fhistory
Failed to load resource: the server responded with a status of 400 (Bad Request)

already did the following
    ipfs config --json API.HTTPHeaders.Access-Control-Allow-Origin  '["http://127.0.0.1:3000", "http://localhost:3000", "http://localhost:5001", "http://127.0.0.1:5001", "https://webui.ipfs.io", "http://webui.ipfs.io.ipns.localhost:8080"]'
    ipfs config --json API.HTTPHeaders.Access-Control-Allow-Methods '["GET", "PUT", "POST"]'

and started ipfs with
 ipfs daemon --enable-pubsub-experiment

my web started from localhost:3000
node = await IpfsHttpClient('/ip4/127.0.0.1/tcp/5001')
orbitDB = await OrbitDB.createInstance(node)

this line is good, and i can get orbitDB.identity.id
however it received forbid error when trying to create the docs store
history = await orbitDB.docs("history", {
  accessController: {
    write: ['*'],
}})

"Access-Control-Allow-Origin": ["*"]

the only way to get it working
and only chromium based browser working... safari would fail

@thienpow which version you have of secp256k1 please check in node_modules/secp256k1/package.json

3.8.0

Hi @thienpow please try downgrading go-ipfs to parallel with https://github.com/orbitdb/orbit-db-test-utils/blob/e5becbf6ba414e91a465fb91c672930990191831/package.json#L24

No further feedback. Closing.

authors: geolffreym haydenyoung tabcat thienpow

repo: orbitdb/orbitdb


text:
Find / Query split

This PR addresses a couple areas related to the find requestable method:
Extract query from find
OC sources should now support query as a separate method from find. This simplifies the logic required to determine whether a request is a query or a request for a specific individual record or records. It also allows for separate requestable hooks for find vs. query (e.g. rescueFind vs rescueQuery).
Remove support for options in find requests from memory and json-api sources
Options, such as include, were not tested in these sources and the implementations need re-evaluation.
For now, these sources will throw exceptions if you pass options into find, query, findLink, or findLinked.
Of course, options can still be passed into custom sources.
Introduce sparse option for Cache
The concept of a "sparse" cache is introduced to indicate that a cache is an incomplete representation of data. Most caches will be sparse, except perhaps a local storage source which is treated as the definitive source of data for a browser-only app (i.e. not connected to other definitive sources).
Sparse caches will not automatically initialize top-level collections. As find request for a collection that isn't initialized will throw a RecordNotFoundException.

Options, such as include, were not tested in these sources and the implementations need re-evaluation.

This seems reasonable, to implement solve properly will require the introduction of the collection meta-data that we've spoken about.

Looks good to me! :+1:

Thanks for the review @opsb and @gnarf! I've made the suggested changes and will merge.

authors: dgeb gnarf opsb


text:
Broken links in docs

There are several broken links in the docs.
E.g. the webpack bin links on http://orbitjs.com/v0.15/guide/data-stores.html
Happy to send a PR to fix, but not sure where they should link.

@larowlan Thanks for pointing this out. I'll recreate the examples on codesandbox.io.

Thanks, happy to help if I can, let me know
On Tue, May 1, 2018, 12:02 AM Dan Gebhardt notifications@github.com wrote:

@larowlan https://github.com/larowlan Thanks for pointing this out.
I'll recreate the examples on codesandbox.io.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/orbitjs/orbit/issues/499#issuecomment-385407098, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAh49pE3web291v0v2QpoeqkH-_OXxcUks5ttxl0gaJpZM4TXkqu
.

The code examples in the guides have been recreated on codesandbox.io (see https://github.com/orbitjs/orbitjs-site/pull/11). Thanks for everyone's patience.

Thanks Dan
On Tue, Aug 7, 2018, 4:10 AM Dan Gebhardt notifications@github.com wrote:

Closed #499 https://github.com/orbitjs/orbit/issues/499.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/orbitjs/orbit/issues/499#event-1773535853, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAh49khBqsKHeBLqfz7HVcbgodc1KUqLks5uOIapgaJpZM4TXkqu
.

authors: dgeb larowlan

repo: orbitjs/orbit


text:
Remove spatie/laravel-ray dependency

Hello 👋🏽
Is it possible to remove the spatie/laravel-ray dependency or maybe move it to dev dependencies? (I see it was moved from dev to regular dependencies some time ago, so I guess it has a reason)
The context is that Larastan has a dev dependency to orchestra/testbench which requires orchestra/workbench which requires spatie/laravel-ray which requires rector/rector which requires phpstan/phpstan version 1 thus PHPStan 2 is not allowed to be installed.
There are multiple workaround/solutions to this of course, but none of them are easy 😅 This was the easiest one that I found, so here I am asking. If not, it's totally understandable. I will try to find other ways. orchestra/testbench is a dev dependency of Larastan, so I have some freedom to do hacky stuff because it'll not be shipped to the user.

@canvural For development packages it may be best to require orchestra/testbench-core instead of orchestra/testbench as this has minimal require for this very reason. You just need to add laravel/framework to require-dev along with other missing deps from testbench

I'll try that, thanks!

Yeah, that's a much better idea! Thanks for the tip.

authors: canvural crynobone

repo: orchestral/workbench


text:
66 unit tests failed in OC Catalyst project in the dev branch

Need to understand the reason for the recent change in the GlobalExceptionHandler and which benefits it brings.
Fix tests or revert/reimplement the commit d466808536f48f1e81ca92c3f168d9b9c8a19fa0 [d466808].

I believe the mentioned change should be reverted which will make the dev branch green aagain. We don't need this change until we got a request from the community why we need this (and I believe the implementation should be improved due to not expected exception type).
This change makes clients to implement more complicated solutions on their side and forces to add additional try-catch construction. The throw shows not the relevant exception in the first place, for example, if we have a request with a wrong address or any other we got the: Error while copying content to a stream exception.
Clients can utilize the GetBaseException method on the mentioned exception variable, cast to the CatalystBaseException type, and only after that retrieve the relevant information.
Prototype code of how to fix the unit-test or what should have clients on their side:
try
{
    await TestFramework.Client.Request("demo/notfound").GetAsync();
}
catch (Exception e)
{
    CatalystBaseException baseException = e.GetBaseException() as CatalystBaseException;
    baseException.ShouldHaveFirstApiError("NotFound", 404, "Not found.");
}

If we revert the throw from the handler, the code style will be as before, much simpler:
var result = await TestFramework.Client.Request("demo/notfound").GetAsync();
await result.ShouldHaveFirstApiError("NotFound", 404, "Not found.");


text:
Сhanging adapter in dialog

Can I change the adapter without closing the dialog?

I changed the adapter as follows:
`final GridHolder holder = new GridHolder(3);
    final DialogPlus dialog = DialogPlus.newDialog(getActivity())
            .setHeader(R.layout.attachment)
            .setContentHolder(holder)
            .setGravity(Gravity.BOTTOM)
            .setExpanded(true, heightDisplay / 3 * 2)
            .create();

    dialog.show();
    holder.setAdapter(adapter);`

repo: orhanobut/dialogplus


text:
Cannot install git-cliff from crates-io due to missing config folder

Describe the bug
Cannot install git-cliff from crates-io due to missing config folder.
Hunch: If the build process requires the config folder, it must be specified to be included to via a cargo manifest key https://doc.rust-lang.org/cargo/reference/manifest.html#the-exclude-and-include-fields (note that you need to specify everything to include then, the globs need to be added explicitly).
To Reproduce
Steps to reproduce the behavior:

`cargo install -f git-cliff

Expected behavior
Build and install the binary.
Screenshots/Logs
   Compiling git-cliff-core v0.2.3
error: proc-macro derive panicked
  --> /home/bernhard/.cargo/registry/src/github.com-1ecc6299db9ec823/git-cliff-core-0.2.3/src/embed.rs:16:17
   |
16 | #[derive(Debug, RustEmbed)]
   |                 ^^^^^^^^^
   |
   = help: message: #[derive(RustEmbed)] folder '/home/bernhard/.cargo/registry/src/github.com-1ecc6299db9ec823/git-cliff-core-0.2.3/../config/' does not exist. cwd: '/home/bernhard/.cargo/registry/src/github.com-1ecc6299db9ec823/git-cliff-core-0.2.3'

error[E0599]: no function or associated item named `get` found for struct `EmbeddedConfig` in the current scope
  --> /home/bernhard/.cargo/registry/src/github.com-1ecc6299db9ec823/git-cliff-core-0.2.3/src/embed.rs:23:15
   |
18 | pub struct EmbeddedConfig;
   | -------------------------- function or associated item `get` not found for this
...
23 |         match Self::get(crate::DEFAULT_CONFIG) {
   |                     ^^^ function or associated item not found in `EmbeddedConfig`
   |
   = help: items from traits can only be used if the trait is implemented and in scope
   = note: the following traits define an item `get`, perhaps you need to implement one of them:
           candidate #1: `SliceIndex`
           candidate #2: `bitvec::slice::api::BitSliceIndex`
           candidate #3: `lexical_core::util::sequence::SliceLike`
           candidate #4: `RustEmbed`

error: aborting due to 2 previous errors

System (please complete the following information):

OS Information: [e.g. Arch GNU/Linux x86_64 5.4.8-arch1-1] Fedora 34
Project Version: [e.g. 0.1.0] 0.2.3

I was able to reproduce this issue. It should be fixed now. (a9b286cf023148da0800c2a0408d87571c239847)
Thank you for reporting!

authors: drahnr orhun

repo: orhun/git-cliff