Dynamically discover lucene version for use in build

The test :solr:validateConfigFileSanity checks that <luceneMatchVersion> is correct in various solrconfig files. Now that lucene version is different from solr version (e.g. on  main branch right now), the build needs to know what lucene version we have.
I tried using Lucene's Version.LATEST.toString(), but the build found a 8_10_0 version in my env.
So now I try to just pull it from versions.props, and that works.
However, it may noe be compatible with the local lucene version override feature in lucene-dev-repo-composite.gradle??

No, I have not overridden lucene version locally. But I think the gradle build may pick up another lucene-jar from .m2 than Solr itself, not sure which classpath the .gradle files use for themselves...

So now I try to just pull it from versions.props, and that works.

In short: no. If you plan to extract Lucene version/ dependency version from within the build file then you should be using getVersion("artifact coordinates") - this will work. But this method can be used at build execution time only so sometimes you have to resort to using providers. An example is in icu.gradle in Lucene:
  // Resolve version lazily (can't resolve at configuration time).
  def icu4jVersionProvider = project.provider { getVersion('com.ibm.icu', 'icu4j') }

Note: It will get a bit more complicated if you need this property at evaluation time.

I tried using Lucene's Version.LATEST.toString(), but the build found a 8_10_0 version in my env.

I'm not sure what code you used here but if you used this exact snippet from within the build then you accessed gradle's internal references somehow. I'm surprised it even compiled (means gradle has Lucene inside somewhere?).

See updated PR for a provider approach, unfortunately it also fails
1: Task failed with an exception.
-----------
* Where:
Script '/Users/janhoy/git/solr/gradle/testing/randomization.gradle' line: 39
* What went wrong:
A problem occurred evaluating script.
> A problem occurred configuring project ':solr:server'.
   > Not allowed to call gradle-consistent-versions's getVersion("org.apache.lucene", "lucene-core", configurations.unifiedClasspath) at configuration time
* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.
==============================================================================

Naaah, that can't work as the provider is resolved a evaluation time.

We can live with hardcoded '9.0.0' for now, but should find a solution to not have theTest LuceneMatchVersion test fail on you if you have a custom lucene version in dev.

Right - that's why I said:

Note: It will get a bit more complicated if you need this property at evaluation time.

The problem here is that in general you won't know the version of a dependency until the configuration is "resolved" and gradle folks try hard to defer this resolution until execution time so that only those configurations that are really needed/ used are resolved. Palantir's version consistency plugin also comes into play. It is complicated.

Looking at the code I think the top-level property from build.gradle should be removed and its initialization moved entirely to lucene-dev-repo-composite.gradle - then you'd know which version is actually in use (there are hacks that read versions.props there); the only place that uses this top-level property is in randomization.gradle; this should follow lucene-dev-repo-composite.gradle in the build file inclusion order and then it'd have access to a properly evaluated Lucene version (as would solr.config-file-sanity.gradle).

Another option would be to resolve the property when the test starts (e.g., using a "before" closure) and it is then added to the command line?

Maybe add an JVMArgumentProvider to the test task. I can check if this works.

Hi, I got it running. Will push a commit. You still may need to adopt it with the local Lucene checkout and the version number. Also the verison number may need to be made compatible to Lucene Match Version, so parsing won't fail. Maybe use a regex to extract correct parts (so it only returns "9.0").

OK, see my changes. Tests are starting up correctly and it prints the lifecycle message (we should remove it). Also i checked the local "ps -ef" and have seen the command line args passed correctly.
I "normalized" the LuceneMatchVersion to be only formatted "x.y", as this is what Version class expects. Or am I wrong?
Also there are some properties passed to tests which are no longer needed. I think the tests.LUCENE_VERSION one is only used by a single Lucene Core test and may be obsolete. And I am not sure about tests.bwcdir!

For some reason "solr:validateConfigFileSanity" fails on the PR. Thats unrelated I think.

I think for the solr:validateConfigFileSanity test you can do the same: Just resolve the version at runtime of task. As this is a handwritten task we can easily use a provider.
Maybe make a more generic provider available in the same way like you did before and let the JVMAgrumentProvider just call it.
So solr:validateConfigFileSanity may be fixed by that. SHould I look into that or do you want to check this?

Ah I see, solr:validateConfigFileSanity uses the test option. I'll fix.

OK, I fixed both.
There is still one open question: Normally Lucene Version should only be provided in config files as "x.y", not "x.y.z". So I'd better go and use LuceneMatchVersion as "9.0" (so strip last part in the regex) and fix all config files.
Otherwise it will get a desaster on updates. You would need to touch config files on every bugfix release.

What you wrote is fine, Uwe, but it won't work with a custom Lucene location - that's why I suggested moving the entire logic you currently have in that provider to lucene-dev-repo-composite.gradle which does this (and more) to extract the locally-overridden lucene version number. I'm sorry but I can't help with this now.

OK, I expected something like this. I thought that the versions are patched by the repo-composite, so it just "replaces" the dependency. But when replacing it changes its name.
I will first to setup something like this side-by-side repos in my local computer with env args (I did this a few weeks ago) to do testing and make it work with a relationship to lucene 9.x branch..

this should follow lucene-dev-repo-composite.gradle in the build file inclusion order and then it'd have access to a properly evaluated Lucene version (as would solr.config-file-sanity.gradle).

This move in order should not be needed, as the provider is only used in another provider, so the evaluation is lazy-lazy :-)

This looks really good. I'm not going to test this locally myself, but I can change the luceneMatchVersion inserted into solrconfig.xml to Major.Minor. Wehenver you guys says it's ready I can then merge and backport.
(I have hacked a cherrypick.sh script that I want to test out, streamlining backporting of trivial, non-conflicting changes) 😁

This looks really good. I'm not going to test this locally myself, but I can change the luceneMatchVersion inserted into solrconfig.xml to Major.Minor. Wehenver you guys says it's ready I can then merge and backport.

All fine. About the 9.0.0 vs. 9.0 I am not fully sure. In the past we changed this on every version, so the current state with full version is fine, although it is not needed for config file sanity (IMHO there we should only use x.y versions). When executing tests it does not matter which version we pass, we can use full version, too. It obly must be in the digit-only format, so we have to remove any appendixes.
My proposal would be: read the version as done currently and sanitize to x.y.z. Pass this to tests, but the config files shipped with release should use the x.y version to make it easier for users to understand that only this is needed for "correct config files". This check should be adapted in the config file checker. But maybe that's a separate issue, isn't it?

(I have hacked a cherrypick.sh script that I want to test out, streamlining backporting of trivial, non-conflicting changes) 😁

I use TortoiseGit for cherrypicking. Thats grandious: Open the log window on 9.x/9.0 checkout, switch branch in log window to show main, select commit, right click, cherry-pick (also works with multiple).

I use TortoiseGit for cherrypicking. Thats grandious: Open the log window on 9.x/9.0 checkout, switch branch in log window to show main, select commit, right click, cherry-pick (also works with multiple).

I use IntelliJ and it is also very simple - git -> log -> find commit -> right-click -> cherry-pick.
But my script does a bit more, it takes branch and one or more hashes, does the cherry-picking, if not clean, open mergetool, else runs gradlew check -x test. So you could run one command to backport 2-3 commits to branch_9x and branch_9_0 in one go. Intended for the more trivial cases only of course...

I committed the X.Y version change, and an update to addVersion.py tool a bit, so it does the same. Now need to fix the configfile-sanity to expect X.Y too

Thanks, I can do the rest including the check script. I think I will rename the provider to luceneBaseVersionProvider.

Here is what I got, which works
Index: gradle/validation/solr.config-file-sanity.gradle
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/gradle/validation/solr.config-file-sanity.gradle b/gradle/validation/solr.config-file-sanity.gradle
--- a/gradle/validation/solr.config-file-sanity.gradle	(revision 636ffd7daea7245d1b31932d549372f31b52cca5)
+++ b/gradle/validation/solr.config-file-sanity.gradle	(revision be6503578dbd8e49ac658464c3add8e28ec944c0)
@@ -21,6 +21,8 @@
   task validateConfigFileSanity() {
     doFirst {
       def matchVersion = rootProject.luceneMatchVersionProvider.get()
+      def (major, minor) = matchVersion.tokenize('.')
+      def matchVersionMajorMinor = "${major}.${minor}"

       // Verify solrconfig.xml files declare proper luceneMatchVersion.
       [
@@ -33,8 +35,8 @@

         configFiles.each { file ->
           def content = file.getText("UTF-8")
-          if (!content.contains("<luceneMatchVersion>${matchVersion}<")) {
-            throw new GradleException("Configset does not refer to the correct luceneMatchVersion (${matchVersion}): ${file.absolutePath}")
+          if (!content.contains("<luceneMatchVersion>${matchVersionMajorMinor}<")) {
+            throw new GradleException("Configset does not refer to the correct luceneMatchVersion (${matchVersionMajorMinor}): ${file.absolutePath}")
           }
         }
       }

Oh thats also a nice one, let me change it

..still fighting lucene-dev-repo-composite.gradle: this is the craziest thing I have ever seen!

Status update:

-Plucene.dev.version=SOME_SNAPSHOT_FROM_LOCAL_M2_REPO works out of box
-Plucene.dev.path=../lucene (my local setup is in progress). The strange thing is that the included build is not resolved, looks like I need to add so dependency. Have to dig...

Hi @dweiss I am stuck how to get the version from the included build. It looks like the included build is completely sandboxed and you only have access to dependencyResultion rules on settings phase and when running at config phase of main build the gradle.includedBuild('lucene') is also just a shim hiding all important facts. You can only lookup tasks, its name, and projectDir. The only solution I have is to do regex on build.gradle to get Lucene version.

Perhaps we can pragmatically document that when using -Plucene.dev.path= you also need to supply -Plucene.version= explicitly, and make that an override that always "wins"?

This is the only thing that worked, although it is a hackydihickhack hell:
// replace Lucene version by dynamic one:
rootProject.ext.luceneBaseVersionProvider = rootProject.provider {
  (new File(gradle.includedBuild('lucene').projectDir, 'build.gradle').getText("UTF-8") =~ /baseVersion\s*=\s*'(\d+\.\d+\.\d+)'/)[0][1]
}

I updated it with a heavy hack alarm. Basically I grep through the build.gradle file, match on the line with baseVersion and then execute the statement.
I also renamed the provider, but basically it was not complicated.
I tested 2 versions:
A lucene/ checkout of branch_9_0 (all others are not compatible) next to my solr checkout:
solr$ gradlew :solr:validateConfigFileSanity -Plucene.dev.path=../lucene
Local Lucene development repository will be used substituting 9.0.0: C:\Users\Uwe Schindler\Projects\lucene\lucene

> Configure project :
Local Lucene development repository will override luceneBaseVersion with: 9.0.1

BUILD SUCCESSFUL in 3s
3 actionable tasks: 1 executed, 2 up-to-date

And I also deployed the 9.0.1 artifacts to local Maven:
lucene$ gradlew mavenToLocal
[...]
BUILD SUCCESSFUL in 2m 17s
248 actionable tasks: 177 executed, 71 up-to-date

solr$ gradlew :solr:solrj:test -Plucene.dev.version=9.0.1-SNAPSHOT
...works and ps -ef shows correct version...

This fails (I have a local 8.11.0 in my maven repo), as expected because the config files are wrong:
solr$ gradlew :solr:validateConfigFileSanity -Plucene.dev.version=8.11.0

> Configure project :
Lucene version forced by -Plucene.dev.version=8.11.0

> Task :solr:validateConfigFileSanity FAILED

FAILURE: Build failed with an exception.

* Where:
Script 'C:\Users\Uwe Schindler\Projects\lucene\solr\gradle\validation\solr.config-file-sanity.gradle' line: 38

* What went wrong:
Execution failed for task ':solr:validateConfigFileSanity'.
> Configset does not refer to the correct luceneMatchVersion (8.11): C:\Users\Uwe Schindler\Projects\lucene\solr\solr\server\solr\configsets\sample_techproducts_configs\conf\solrconfig.xml

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 6s
3 actionable tasks: 1 executed, 2 up-to-date

that development build file is indeed very, very difficult because it is included in gradle at build phase - it took a great deal of time to get that working and it's indeed two different environments (as it was noted in the comments).

that development build file is indeed very, very difficult because it is included in gradle at build phase - it took a great deal of time to get that working and it's indeed two different environments (as it was noted in the comments).

All fine!
What I was wondering about: According to documentation the dependency substitution should be completely automatically. But it looks like we need that artifact name to project name mapping. Is this because Lucene/Solr's gradle.build files do not explicitly name "group" and "name" (those coordinates are only generated on-the-fly in the maven publishing)?

Is this because Lucene/Solr's gradle.build files do not explicitly name "group" and "name" (those coordinates are only generated on-the-fly in the maven publishing)?

Maven coordinates only exist in maven. Gradle has module paths - build inclusion is kind of hacking these two worlds together. What's in that file is a result of trial-and-error and my understanding of how these two worlds come together, I'm by no means trying to say I understand everything going on under the hood of Gradle...

Thanks for merging!


⚠ Secret Website 3 has degraded performance

In 150f7aa, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 2189 ms

Resolved: Secret Website 3 performance has improved in a2aff26.


Individually include the files via Visual Studio UI

Fixes #1519

Remove the wildcard ItemGroup from the project file
Explicitly include the various files using the VS UI designed for this purpose

we have to find a different way that doesn't include manual edits of the solution for every hex file etc

I know you did it on purpose but it's wrong. I lost so much time today because it was automatically copying files when I was trying to debug problems in a specific file without the others showing up.
Back when I worked on msbuild we never, ever, advocated customers use wildcards like this because of the future grief it will cause. The short term pain of having to use the VS UI is well worth it compared to the future anguish we will have.


Vue treat every element attribute named like 'v-[something]' as directive

Version
2.5.17
Reproduction link
http://jsfiddle.net/wf48v9de/3/
Steps to reproduce
Run the fiddle and look at console.
What is expected?
If there is not registered directive named 'v-fake', Vue should ignore this attribute of an element.
What is actually happening?
Vue treat every 'v-[something]' attribute as directive, regardless if it is registered globally or locally, or not at all. It is trying to compute the value of 'v-' beginning attribute, and in effect throws TypeError, as it doesn't find property named like string passed to attribute.

if it is intentional, there should be information about such behavior in Vue docs. However in my opinion it should ignore such attributes.

IMHO, I think the docs are clear enough. The v-* lookup has a good behavior. It makes debugging easier, if you want to add a property v-something and not want Vue to treat it as a directive, you should be using data attributes
So instead of <div v-fake/> it should be <div data-v-fake/>

Maybe it would make more sense to resolve the directive and show an error (like when there is no value)

@posva I guess that could work.


vue.common.js:433 TypeError: Cannot read property 'name' of undefined

vue.common.js:433 TypeError: Cannot read property 'name' of undefined
at a.render (App.vue?db21:41)
at a.Rt.t._render (vue.common.js:2952)
at a. (vue.common.js:2189)
at Ui.get (vue.common.js:1652)
at Ui.run (vue.common.js:1721)
at z (vue.common.js:1539)
at Array. (vue.common.js:471)
at t (vue.common.js:420)
o @ vue.common.js:433

Please follow the guidelines on how to report an issue. In particular, provide an example on www.jsfiddle.net (or a similar service) that reproduces the problem. If nessessary, create a repository for us to clone with a minimal reproduction. repositories of actual projects will generally not be accepted .
Thank you.

Please reopen with information that conforms to the issue reporting guidelines.


Extract Short JDK Name

Problem Description
Currently, we don't extract the JDK name while downloading a JDK in getBinaryOpenjdk() in get.sh as shown below.
https://github.com/AdoptOpenJDK/openjdk-tests/blob/4865e8d913fb4f4a43307e6d8cfd400946ca0dc9/get.sh#L138-L188
This one line info is pretty handy for displaying benchmark results since it doesn't take space for storing and can be easily used for querying data from the database for a build, without requiring to search multiple times for a specific build.
This one line product info is also expected by the Test Result Summary Service for parsing and storing benchmark results for different builds as shown below.
https://github.com/AdoptOpenJDK/openjdk-test-tools/blob/858993efb7e454786dba984856ed454e3290bd26/TestResultSummaryService/parsers/BenchmarkParser.js#L8
While we do print the java version, it doesn't have the summarized info of the java stream, bit, platform, java implementation and date in one line. In fact, java version could be missing some info such as the platform info in the HotSpot java version as shown below.
Example for OpenJ9 Java Version
16:01:38  curl -OLJks --retry 5 --retry-delay 300  https://api.adoptopenjdk.net/v2/binary/nightly/openjdk8?openjdk_impl=openj9&os=linux&arch=x64&release=latest&type=jdk&heap_size=normal
16:01:49  unzip file: OpenJDK8U-jdk_x64_linux_openj9_2019-05-21-11-05.tar.gz ...
16:01:51  Run /home/jenkins/workspace/Grinder/openjdkbinary/j2sdk-image/bin/java -version
16:01:52  openjdk version "1.8.0_222"
16:01:52  OpenJDK Runtime Environment (build 1.8.0_222-201905180358-b02)
16:01:52  Eclipse OpenJ9 VM (build master-72a537c81, JRE 1.8.0 Linux amd64-64-Bit Compressed References 20190518_311 (JIT enabled, AOT enabled)
16:01:52  OpenJ9   - 72a537c81
16:01:52  OMR      - 3172276c
16:01:52  JCL      - 7b46afb743 based on jdk8u222-b02)

Example for HotSpot Java Version
13:56:37  curl -OLJks --retry 5 --retry-delay 300  https://api.adoptopenjdk.net/v2/binary/nightly/openjdk8?openjdk_impl=hotspot&os=linux&arch=x64&release=latest&type=jdk&heap_size=normal
13:56:47  unzip file: OpenJDK8U-jdk_x64_linux_hotspot_2019-05-21-11-05.tar.gz ...
13:56:48  Run /home/jenkins/workspace/Piyush-Grinder/openjdkbinary/j2sdk-image/bin/java -version
13:56:48  openjdk version "1.8.0_212"
13:56:48  OpenJDK Runtime Environment (AdoptOpenJDK)(build 1.8.0_212-201905180333-b03)
13:56:48  OpenJDK 64-Bit Server VM (AdoptOpenJDK)(build 25.212-b03, mixed mode)

Proposed Changes
We should extract the name from the downloaded binary and export it so that it's available for use in benchmark scripts and other places if needed.
For example, if we download OpenJDK8U-jdk_x64_linux_openj9_2019-05-21-11-05.tar.gz, then we should extract OpenJDK8U-jdk_x64_linux_openj9_2019-05-21-11-05.
Similarly, OpenJDK8U-jdk_x64_linux_hotspot_2019-05-21-11-05.tar.gz should give OpenJDK8U-jdk_x64_linux_hotspot_2019-05-21-11-05.

I think it is not a good idea to rely on the file name to determine SDK information such as java stream, platform, java implementation and date, etc. A filename may not have accurate information and it can be modified easily. Different sites are highly likely to have a different naming convention.
For example, the file generated from Openj9 is called OpenJ9-JDK11-ppc64le_linux-20190524-022309.tar.gz

Yeah I do agree that the names could be different unless we're always pulling our builds from one place which might be consistently be sticking to one naming convention.
Anyway, I guess we can decide our own convention and use that to generate this short JDK name instead of parsing that info from the archive file name.

Closing this issue since we've added support to dynamically get the JDK version and date on TRSS, which can be used in various tools such as Tabular View and Perf Compare.


[CI] SVE-512 tests timeout

Some tests such as unit.algo.algorithm.min_element_generic_one_pass.exe and unit.algo.algorithm.swap_ranges_generic.exe regularely timeout under SVE-512.

I am dropping this old idea back here for discussion.
Instead of compiling tests for cardinal 1 2 4 ... N 2N , what about comiling only for 1 N/2 N N2. It prevents the ballooning with large register ISA.
Currently on AVX512 we test :

(1 2 4 8 16) x 3 for 64 bits types
(1 2 4 8 16 32 ) x 3 for 32 bits types
(1 2 4 8 16 32 64) x 2 for 16 bits types
(1 2 4 8 16 32 64 128 ) x 2 for 8 bits types
For a total of 63 instances of tests

With the other one, it'll be 4 tests per types, for 10 types total, for a grand total of 40. SO a reduction of 40% of the nulber of tests.

Those are algo tests, they try reduced number of types already.

2.I think it's mostly about trying to validate too many things. Those are two_range generic tests. We should look at those and see if we can loop less

Should we then investigate using fuzz8ng firbthe algo test ?

Fuzzing would be amazing. I could spend less time designing horrific contraptions to catch bugs. But you need like somewhere to run those. I doubt oss fuzz will take eve

We can ask


#9748: [WIP] Migrate untilize ops to ttnn

Ticket
#9748
Problem description
Migrating untilize ops family to ttnn.
What's changed
All C++ code has been migrated.
Checklist

[ ] Post commit CI passes

This PR is a WIP!


chore(deps): update sigstore-rs to 0.8.0 and oci-distribution to 0.10.0

The second round of updates that allow image-rs to move to released versions of the crates

thanks to @arronwy for tips


OpenShift demo write-up

For v3 IKS now supports OpenShift clusters - indeed I have one deployed for testing.
I think v4 support comes in the fall

I seem to have lost track of who volunteered for which of these, but thinking this one was yours?

yes

See #1203 for some cloud related blogs.
Instead for core egeria I will add a blog/content around using the open source 'microk8s' project which a number of people including myself have used very successfully.


sum and avg are not in the functions list

Hey,
The functions list currently does not contain the sum and avg funcs, I think there should be a note about them

This is because they are aggregation operators, not functions. They are thus listed with the operators.

It makes more sense to ask questions like this on the prometheus-users mailing list rather than in a GitHub issue. On the mailing list, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.


Website crashed

Environment
OS: Windows 11 Version 23H2 (Build 22631.3737)
UA: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36 Edg/<IP_ADDRESS>
Reproduction

https://unhead.unjs.io/
https://unhead.unjs.io/usage/composables/use-head-safe etc.

Describe the bug
The first one responses:
<!DOCTYPE html>
<html>
    <head>
        <meta http-equiv="refresh" content="0; url=https://unhead.unjs.io">
    </head>
</html>

This causes the page to constantly refresh.
The second one shows an 500 error:

This Serverless Function has crashed.

Additional context
No response
Logs
No response

Thanks for the issue, seems to be fixed now.

Oops, It crashes again - nuxt CI log
This time it's

Thanks! Fixed this after like 30 attempts 😥


Enforce a message size limit

Forces all messages to be at most 512 bytes, to stop cmd help all from the irc_commands DoSing the server (making it quit with the message RecvQ exceeded).
This is done in the core IRC mod so it limits everything rather than just irc_commands.
Credit to @bigfoot547 for finding this vulnerability.

Yes, but, can we please maybe get some logging in the console as well if players want to cause floods this way? Silently dropping output may also cause unintentional side effects, so we should consider replying back with an error message and throttling the user for at least a few seconds, too.

What about adding ... to the end of long messages?
PRIVMSG #channel :A really long testing messag...

That would need a change somewhere else, there may be rouge irc.say()-s in other mods that need fixing too.


Update PageStreaming by introducing PageAccessor.

Current coverage is 51.24%

Merging #66 into master will decrease coverage by -0.06% as of cc41538

@@            master     #66   diff @@
======================================
  Files           41      41
  Stmts         1873    1887    +14
  Branches       284     285     +1
  Methods          0       0
======================================
+ Hit            961     967     +6
+ Partial         56      55     -1
- Missed         856     865     +9

Review entire Coverage Diff as of cc41538

Powered by Codecov. Updated on successful CI builds.

@aozarov Please take a look.
Note: This PR only adds PageAccessor. PageStreaming renaming will be addressed in a separate PR.

Looks nice. Few comments.

PTAL

PTAL

PTAL

Looks good to me. Few nits but I think it is fine regardless.

LGTM.

Do you plan to build and push a new version and to generate PubSub to use it?

Yes


infer_schema! will no longer treat views as tables

When using PostgreSQL infer_schema! treats views as tables. This will produce errors like:
error: table geography_columns has 0 primary keys, only one is currently supported
On this pull request i just added AND table_type NOT LIKE 'VIEW' to the SQL query that fetches all the table names.

Wow, that was quick! It might be nice to make this configurable in the future but if this breaks your current setup I think this is good as-is.
Can you mention "Fix #373" somewhere in the commit message? This will automatically close that issue when this is merged.
It would be great if you could also add a test for this. I think there are already some teste for infer_schema!, so you could probably just add a view to the db schema.

@killercup Thank you for the fast reply. I just changed the commit message and added "Fix #373" to it.
About the tests, i will try to do that later today or tomorrow. I am quite new to rust and need to figure how to write that test (from what i saw so far, it shouldn't be hard, but i still need the time to figure it out).

@killercup As making it optional, i don't really think that is a good idea because views don't actually have primary keys. In my opinion, views should be dealt in some other way.

I'm fine with this until we add better support for nonstandard primary keys. Can you add a test for this and a changelog entry?

@sgrif I just added a changelog entry and modified the SQL query so that instead of excluding views from the query, it only fetches 'BASE TABLES'.
As diesel doesn't have any kind of views support i don't really see the point of adding another test. This will just enforce propper behaviour when fetching tables (views aren't really tables).
Let me know if you need me to change something else.

Well one argument for adding a test: I have no clue if we need to make an equivalent change for SQLite or not.

Seems like SQLite also supports views and also fetches them in the same way, alongside with normal tables: Stack Overflow. I don't use SQLite but next week i can do a new pull request with the SQLite fix.
As it currently is, diesel crashes if we have a view (views are just queries, not tables and as such have no primary keys) so the current tests would already be what we need to test this situation. If you still want me to add something to the tests just for the sake of it, i will just add the SQL to create a view and check if it wasn't fetched (If it was, it would crash diesel anyway when using infer_schema!).

Yes, I would prefer if a minimal test case could be added for this as
you've described. I'll write the patch to fix this for SQLite as well.
On Wed, Jul 13, 2016 at 6:43 AM António Cascalheira <
<EMAIL_ADDRESS>wrote:

Seems like SQLite also supports views and also fetches them in the same
way, alongside with normal tables: Stack Overflow
http://stackoverflow.com/questions/82875/how-to-list-the-tables-in-an-sqlite-database-file-that-was-opened-with-attach.
I don't use SQLite but next week i can do a new pull request with the
SQLite fix.
As it currently is, diesel crashes if we have a view (views are just
queries, not tables and as such have no primary keys) so the current tests
would already be what we need to test this situation. If you still want me
to add something to the tests just for the sake of it, i will just add the
SQL to create a view and check if it wasn't fetched (If it was, it would
crash diesel anyway when using infer_schema!).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/diesel-rs/diesel/pull/374#issuecomment-232320544, or mute
the thread
https://github.com/notifications/unsubscribe/ABdWK4Jd9lURFxZeS8MOfG-aOhgCHgSRks5qVMFTgaJpZM4JE-eA
.

I think this already works for SQLite because of this line.
I'm going to rebase this and write a small test. Or, actually, I'm just going to add views to the diesel_test migrations, as there don't seem to be any tests for infer_schema itself.

I think just a test for load_table_names that creates a view should be sufficient.

@cascalheira I rebased your branch on master and added some tests. #434 includes your commits (with you as author, of course), so I'll close this PR. Feel free to comment over there :)

@killercup Sorry for the late answer i've been quite busy lately :)
Thank you for including that as it will make possible for me to use the master branch on my projects.


Feature/update resource in central database

Changes to resource syncing should be targeting master, not 3.x, since resource syncing got significantly changed in v4
Provide a detailed description for a PR like this


feature support set and use statement syntax only

This PR:

add a new Command type: UnsupportedCommand to handle the statement only support parse but could not execute.
support syntax about set and use
add keyword VAULT to follow legacy planner

TODO

support all statment syntax in Nereids

run buildall

run buildall

run buildall

run buildall

run p0


[run] Fix ansible-runner artifacts cleanup

Depends-On: https://github.com/ansible/ansible-navigator/pull/288
Fixes https://github.com/ansible/ansible-navigator/issues/255
Fixes #92

Pass rotate_artifacts value set to 1 to runner API
which will ensure at a given time only latest runner artifact
folder exist
Update default runner artifact storage to path
~/.ansible-navigator/ansible-runner in order to
avoid future path collision as ansible-runner will
store the artifact by default in ~/.ansible-runner.
Update tmux_session logic to combine the command executed value
shown within the pane as longer commands can be split into
mulitple lines when capturing the pane
Add integration test cases from run action

multiple instances of ansible-navigator will remove each others artifact directories with rotate=1,
Let do this. /tmp/ansible-navigator/uuid/
where uuid is generated when the navigator starts
and we can delete the directory before we exit, we cna store the uuid in the config subsystem internals

recheck

Recheck

recheck

recheck


Feature Request: Implement Blackjack Splitting

Enhancement Request:
In regular Blackjack games, players have the option to split their hand when dealt two cards of the same rank, allowing them to double their bet and play two separate hands.
Mayb you could add the ability for players to split their hand when dealt matching cards (e.g., two 7s, two 10s, two Aces). This feature allows doubling the bet and playing two separate hands.
Best Greetings,
AztecCodes

Sorry for the delay, this feature will come soon

@doofy-dev Cool that you look into that! Thank you for contributing open source applications for the community!


add test for enterprise -> enterprises

Currently enterprises as plural, is converted to singular as enterpris instead of enterprise.
Let's fix it!

I added a test and a fix. However if there is a better way to solve this issue (with one of the RegEx rules) then I am happy to change the code.

Coverage remained the same at 100.0% when pulling ca0de660b3e4e0abedfb649d78fcfa2e66a5fd34 on RobinMalfait:feature/add-enterprises-enterprise into 86f29d630f071f0e382cb516b2ee70c9129e5605 on blakeembrey:master.

Nice, thank you!


peer+htlcswitch: improve logging

Changes so that a grep on the log files with just a channel point will yield all link ~and peer~ activity for that channel.

Took off the second commit - that needs more thought.

This needs a lot of updates in the itest error whitelist.

Needs a rebase to trigger the correct set of CI checks and also an entry to the release notes.


Use chocolatey to install chocolatey

Can we instead, just add "include chocolatey" to manifests/windows.pp ?
Then we could just say require chocolatey in https://github.com/puppetlabs/pltraining-classroom/blob/54bb5aa4267fbc6054769ed46c65377e045f62e1/manifests/agent/git.pp#L22
This would mean we don't need the classroom::windows::chocolatey class anymore.
Finally the dependencies for the module should be updated.


[Bug] error occured when gpinitsystem on redhat8.9 or openEuler 22.03 (LTS-SP4) or Anolis OS 8.6,after rpm/Compile install successfully.

Cloudberry Database version
cloudberrydb-1.5.4
What happened
After rpm/Compile install successfully,  when gpinitsystem, error 'Error: unable to import module: No module named 'pg''happened,then after 'pip3 install pg -i https://pypi.tuna.tsinghua.edu.cn/simple/', another error 'File "/usr/local/lib/python3.9/site-packages/pg/init.py", line 3
async SyntaxError: invalid syntax' happened when used python3.9,error'AttributeError: 'NoneType' object has no attribute 'glGetError''happend when used python3.6,but now actually, database is open ,but cannot stop or start ;  please see below pictures  like my describe .

What you think should happen instead
No response
How to reproduce
I suggest reduce dependencies.   avoid ''No module named 'pg'error.
Operating System
redhat8.9 or openEuler 22.03 (LTS-SP4) or Anolis OS 8.6
Anything else
No response
Are you willing to submit PR?

[X] Yes, I am willing to submit a PR!

Code of Conduct

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

Hi @fsmiledream the 1.5.4.rpm is a little outdated. Please try the latest 1.6.0 version to see if it works. Our 1.6.0 version has a robust RPM package and has been tested many times and solved the dependencies problem.
If 1.6.0 has the same issue, we can continue to look into the cause.
cc @Zhangbaowen-Hashdata @edespino

Duplicate of #620


Convert pandas solution 626 to PyArrow

import pandas as pd

def exchange_seats(seat: pd.DataFrame) -> pd.DataFrame:
    combined = seat.assign(exchange_id=seat.id % 2 + seat.id)
    joined = combined.merge(combined, how="inner", on="exchange_id")
    swapped = joined.loc[joined["id_x"] != joined["id_y"], ["id_x", "student_y"]].rename(
        columns={"id_x": "id", "student_y": "student"}
    )
    if len(seat.index) != len(swapped.index):
        return pd.concat([swapped, seat.tail(1)])
    return swapped

Closed by https://github.com/tablexpr/pyarrow-exercises/commit/f7ef5dd29eadf4ade75e0f645841afd18607dcbd


New: Adding overwriteFile rule.

Description
Adds overwrite file to delivery rule for cloud based delivery systems where previous data exports can be reasonably overwritten.
Proposed Changes

Adds overwriteFile to scheduling rules.

License
I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

Yes we will update the poll mechanism in another PR. For export it's less necessary. But want to make sure we have good full integration test before we switch over to that mechanism.


Section 4 Wards slight adjustments

Ward highlights now disappear as you pick each ward. Fixes https://github.com/ModDota/dota-tutorial/issues/288
Ward pickup check intervals reduced from 1 second to 0.2 to be more responsive to when you take the wards.
Added 6 ward cliff locations, to a total of 10, in a way that shows them in a series of closest cliff positions to each other. To compensate for the added dead time, camera speed was increased from 2000 to 2500, and the wait time between each cliff was reduced from 1 second to 0.25 seconds. Tweaked it a few times and I feel the result's good, but we can adjust it further if necessary.

Added 6 ward cliff locations, to a total of 10, in a way that shows them in a series of closest cliff positions to each other. To compensate for the added dead time, camera speed was increased from 2000 to 2500, and the wait time between each cliff was reduced from 1 second to 0.25 seconds. Tweaked it a few times and I feel the result's good, but we can adjust it further if necessary.

We had this in initially but we thought it was way too much

Not a problem to remove them. Can you try testing that once and see if it's still too much after those changes?
Alternatively, we can show just two cliffs, one at Dire and one at Radiant, closest to the player.


Ensure readiness for comment updates

Hopefully fixes #130201849

The tests passed locally. They're timing out here. This good to go @colinta?


Loading forever

Hey, it looks like your site stopped working a few days ago. Any plans to fix it? :)
Thanks.

Hi, seems like the issue is back again. Could you have a look at it?
Thanks in advance :)

Maybe open a new issue for it...


Remove e2e waits

Summary
Fixes #9040

Occurred changes and/or fixed issues

Removed E2E waits and added timeout for requests instead
No functional changes

Areas which could experience regressions

No UI changes

We have crossed also other places where we are actually waiting for something to make pass the test, should we tackle down them too?
https://github.com/rancher/dashboard/commit/5b1d393b5983d82c7ad4d2a8f83a86ed2b4d793d


can't configure different Thresholds directions (Ok > Warn > Critical) for different mapping rules

Hello,
I'm trying to configure different thresholds directions (Ok > Warn > Critical) for different mapping rules. But seems like they apply globally for all available mappings.
Here what I have:
-an existing mapping for Elasticsearch

an existing mapping for another system component:

And when I tried to add the third component (Kafka) with the opposite threshold direction (Critical > Warn > Ok), the first two changed their color to red:

is it a bug, or am I doing something wrong?

Hi @arnitolog
Thx for this feedback.
Stroke and fill works only for drawio shapes, for pictures use border.
If in xml, you see base64 or an url, it's a picture otherwise it's a shape
See that :
https://github.com/algenty/flowcharting-repository/blob/master/images/colormode_ani.png?raw=true

@algenty what I'm trying to do is just have different directions for thresholds. I click on Invert  button for one of the rules. And seems like it affects all existing rules

ok, it's maybe a bug, when you clone rules, can you try to create rule without clone options ?

yeah, the new one works for me thanks a lot. This is really cool plugin. Thank you very much

you're welcome, i'll try this bug this week to fix it

Ok, can reproduce bug, i'll fix it

Can you try 0.7.0 snapshot in https://algenty.github.io/flowcharting-repository/ARCHIVES.html

looks good to me. Thank you


netclient rewrites /etc/hosts unnecessary when internal ordering of hosts changes

Problem:
netclient rewrites /etc/hosts unnecessary when internal ordering of hosts changes
Expected:
rewrite file only when meaningful change occurs
Impact:
possible race condition risk / change monitoring / manual editing of hosts file
Suggested solution:

sort the host list
do not modify the file if no meaningful change

I have also seen this. v0.17.1.  I have just recently setup and started trying out netmaker coming from zerotier, and wasn't sure if this was typical behavior. I probably would have never known except GlassWire pops up an alert every time netclient writes the new hosts file.

For me seems completely random and every 5-15min, though sometimes it may not happen for an hour or two.

https://github.com/gravitl/netclient/pull/211 updates how /etc/hosts entries are created, modified and deleted.


Visaul studio 17

[Enter feedback here]
How to specify icon in the windows control desc > Programs > Program unload in Visiual studio 17 for visual basic
Thanks

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

ID: f36735fb-5d5f-72c0-e71d-cff66dd1e909
Version Independent ID: 8f3d80cc-7f07-1f8e-130a-cc28051418db
Content: Application page of VB project properties - Visual Studio
Content Source: docs/ide/reference/application-page-project-designer-visual-basic.md
Product: visual-studio-windows
Technology: vs-ide-general
GitHub Login: @TerryGLee
Microsoft Alias: tglee

@cesesa -- thank you for your feedback; we appreciate it. To make sure we have all the information needed to triage your issue, please provide more context around your general task.  Meanwhile, consider these resources:

Visual Studio on Q&A
https://social.msdn.microsoft.com/Forums/vstudio
https://www.stackoverflow.com
https://www.reddit.com/r/VisualStudio/


Don't hard-code pointer size.

Fixes #29

@kylef @neonichu Just had an idea that's worth trying: wrapping T in an Optional (even if it's an optional already) then use flatMap to reduce levels of optionals to 1:
public struct Attribute<T : AnyObject> {
    func expressionForValue(value:T) -> NSExpression {
        // Cleaner implementation?
        let optValue = Optional<T>(value).flatMap{$0}

        if let value = optValue {
            return NSExpression(forConstantValue: value)
        }

        return NSExpression(forConstantValue: NSNull())
    }
}

Would this solve your issue?

@AliSoftware doesn't seem to work, either:
.../QueryKit/Attribute.swift:42:20: Cannot find an initializer for type 'NSExpression' that accepts an argument list of type '(forConstantValue: T)'

in the if let case.

Yeah did more tests and unfortunately we're down to the same old problems again.
Did a lot of tests with some func optionalize<U>(v: U) -> U? { return U?(v) } + func optionalize<U>(v: U?) -> U? { return v} override too.
Those works outside of the struct Attribute<T> and the proper override is used in that case…
But inside Attribute<String?> then we got T=String? which is seen by the compiler as an un-dissociable unit T…
The problem is that the compiler's type-resolver backtracking algorithm will always try to resolve everything against T as a whole, not against its decomposed version String? seen itself as a composed type. If it were about to solve it against the decomposed type, it would have allowed it to match U=String then match optionalize<String>(String?) -> String? and solve our problem, but it can't match U=String and only U=T=String? as T is not decomposed at type-resolution time in that context :sad:

I wonder if this is worth a radar? :satellite:


unnecessary polling?

When a user is logged in to proof.fredhutch.org and on the PROOF Server tab, the shiny app polls the GET /cromwell-server endpoint of the PROOF API every 2 seconds.
Usually the API can handle this. Occasionally there are problems and I have some ideas about fixing them on the API side.
But I am wondering if this polling is necessary. I could be wrong, but I think calling this endpoint is only really necessary when

starting a PROOF Cromwell server (to make sure one is not up already); after starting, this is the only time we may want to poll (and we'd stop when the server comes up)
before running any operation that requires a cromwell server be up.
Before stopping a cromwell server

If I am not mistaken about this, then it seems wrong to continue polling every 2 seconds.
What do people think? If there is agreement maybe we can change this.

If polling is unavoidable, maybe we can look into having a long running connection, such as with websockets, or(probably a better fit for a REST API) Server-Sent Events (SSE).

See also #75

In #75 I changed polling to only happen when the user is actively on the PROOF Server tab, so that's some improvement.

we just send something from the server to the client when there is an update.

I don't know if Shiny can do this. Maybe it can. I've tried longer times between polling requests, but the issue is that creates a what I think is too long of a lag where the user assumes nothing is happening or something didn't work.

but I think calling this endpoint is only really necessary when

In your list of 3 scenarios that's not quite right. The polling is only done to get metadata from the PROOF API on the server status. Polling doesn't have to be done before or after starting or stopping a server. In theory we could remove the metadata from this page altogether and remove the polling code, but I don't think we want to remove the information on the user's server.

If I'm understanding the discussion above, this feels like a v2.0 discussion since it only applies to a relatively small use case and functions relatively well as is. Plus, moving the app out of Shiny in v2.0 might alleviate this issue entirely. But I might be underestimating the occasional API problems that @dtenenba mentioned. Mind if I close this issue for now and we can reopen it if those API problems start becoming more prevalent?

OK with me to close this. I did a bunch of testing and was not able to definitively prove or disprove that this was a problem or related to the problems we saw in the stress test.
We may want to reopen if further evidence surfaces either way.


Should this be put on the App Store?

Extensions for Xcode are still allowed with Apps fromthe AppStoreandcan be enabled in the System Settings.
Would this notbe a viable route for XVim?
P.S. For reference see Swiftify

If possible, I think it's a great idea.

See https://github.com/XVimProject/XVim/issues/964 on why this is not possible.


fix json install for debians

Fix #12398 which caused a problem on bionic debian creation. It works on focal and jammy.
For now, no direct linking with nlohmann_json -- I went back to just directly pointing to the include dir.
Tracked on [LRS-958]

Release flow started:
https://rsjenkins.iil.intel.com/view/LRS/job/LRS_release_flow_pipeline/5860/


JsonDeserializer unable to deserialize Java8 objects

Hello!
I'm not sure if this is a bug, but from what I can tell by reading the JsonDeserializer class, there is no way to add modules (for e.g. JSR310) to the ObjectMapper so that it can deserialize Java 8 classes such as LocalDateTime.
Reference: https://blog.oio.de/2015/06/13/add-support-for-java-8-date-time-api-to-jackson-serialized-rest-web-services/

There is a second constructor, where you can inject a customized ObjectMapper ...
public JsonDeserializer(Class<T> targetType, ObjectMapper objectMapper) { ... }

Ah of course. I can't believe I missed that. Thanks!


:construction_worker: chore: Add test coverage to gha

Description

add test coverage to config package
add to github actions
don't bother adding to packages that are just exporting stuff from bundler like all the bundler plugins

Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

Current dependencies on/for this PR:

main

PR #311   👈

This comment was auto-generated by Graphite.


List item not highlighted when clicked

The highlight of a child or parent is not always updated after selection .. appears to randomly work.

Hi Peter,
Thanks for the report.
Could you let me know the following details about where you are seeing the problem:

What mac is being used e.g. mine is a rather old school MBP, Intel i7
OS and version number, e.g. macOS Ventura 13.3
Xcode version used to build.
Anything else  that might be relevant.

Kind regards
Jon

Thanks - I like the component, but it would be good to be able to select and optionally set a highlight color to set it apart from the default color theme.
Macbook M1, Ventura 13.0.1 (22A400), XC 14.2

Thanks Peter,
I'll have a look this weekend.
Jon


Make available via Bower

Having people install SocialCount by downloading a zipped folder is not desirable when package managers exist to keep people's dependencies up to date. This PR does everything required to make the package available on Bower except for actually registering the package.
Once this PR is merged you can register the package as socialcount by following the instructions here but it's as simple as:
bower register socialcount https://github.com/filamentgroup/SocialCount.git

Thanks!

My pleasure!


pypy3 isn't a classifier for some reason

Coverage remained the same at 95.031% when pulling ae941a836ca7daf3a517179ade5395b4411ba890 on implement_options into 0f3b02bab08613a64cc322916e0a0c0a1ed3be08 on master.

Coverage remained the same at 95.031% when pulling ae941a836ca7daf3a517179ade5395b4411ba890 on implement_options into 0f3b02bab08613a64cc322916e0a0c0a1ed3be08 on master.


foundry.toml optimizer=false

are we ok with optimizer=false?
I think the external/BigNumbers.sol is the blocker on changing this to true:
with optimizer=true i see the message when i run forge build:
mikeseiler@Mikes-MacBook-Air filecoin-solidity % forge build
[⠢] Compiling...
[⠰] Compiling 52 files with 0.8.17
[⠔] Solc 0.8.17 finished in 80.03ms
Error:
Compiler run failed
error[6553]: SyntaxError: The msize instruction cannot be used when the Yul optimizer is activated because it can change its semantics. Either disable the Yul optimizer or do not use the instruction.
   --> contracts/v0.8/external/BigNumbers.sol:903:9:
    |
903 |         assembly {
    |         ^ (Relevant source part starts here and spans across multiple lines).

error[6553]: SyntaxError: The msize instruction cannot be used when the Yul optimizer is activated because it can change its semantics. Either disable the Yul optimizer or do not use the instruction.
   --> contracts/v0.8/external/BigNumbers.sol:964:9:
    |
964 |         assembly {
    |         ^ (Relevant source part starts here and spans across multiple lines).

error[6553]: SyntaxError: The msize instruction cannot be used when the Yul optimizer is activated because it can change its semantics. Either disable the Yul optimizer or do not use the instruction.
    --> contracts/v0.8/external/BigNumbers.sol:1059:9:
     |
1059 |         assembly {
     |         ^ (Relevant source part starts here and spans across multiple lines).

:link: zboto Link

BigNumbers is not a library that we actually need on the API. It is being used internally for mocks contracts only. However, any operation devs want to make over BigInt will require to use this library, so I guess devs will import it at the end. :/

By forcing optimizer=false we are changing the defaults. It is fine as long as we have a very good reason to do it. Let's explore if it's absolutely needed. Otherwise there's a chance of it failing in for random tools (like foundry in this case) in the future.

I don't have a solid answer here @maciejwitowski. BugNumbers is likely to be used by devs to handle bignum operations like mul or div. There is no other lib out there for that :/

But we are using it only in tests, right? Isn't there any workaround not to use it at all so we don't enforce this optimisation flag?
I'm not sure what other devs have to do with it 😕 Do you mean that the optimisations won't be applied anyway when they use BigInts in their contracts?

So, I'm a solidity noob, but...
Shouldn't:
https://github.com/zondax/solidity-bignumber/blob/39dbb9e2dd186a549c9ad7eb1b4b204dc6705245/src/BigNumbers.sol#L880
Be let freemem = mload(0x40)?

Hey @emmanuelm41 @maciejwitowski would love to figure out a solution to this... Our codebase can't rely safely on the filecoin-solidity library because some of the API calls we make to the precompile require BigInt, and to make use of it, we then end up using BigNumber. Any library that relies on this code then can't be optimized, so we have to use more dangerous workaround


Rotated map jumps around when panning

The MapController sample allows rotation, but panning a rotated map causes the map to jump. This may be due to the fling animation.
See https://github.com/johnpryan/flutter_map/pull/359 for details related to rotation

I can still reproduce this on master, are you sure it's fixed?

I'm not sure, I'll reopen this.

Woops, I already opened #387. If it's the same cause we can close it and continue here.

So this is the behavior I recorded on a Pixel 3: https://youtu.be/wEW-7ZECZrw
As you can see, only when the map is rotated, the panning animation isn't smooth and the map jumps down.

Any update on this?

Not sure I can see this in master, but it would be good to be fixed, is it because the CI tests failed ? (not that familiar with it yet!)

I also have this problem, setting the rotation to 180 and 0 have no problem though

Yes, it seems to be worst for rotation values of 90 and 270 degrees. We tried it too.

Hello, PR #719 fixes this problem.


Added PDF to DOCX Converter

In addition to this, I've added a seperate README.md file with this PR inside convert_pdf_to_docx directory explaining about the requisite libraries and an example usage of this tool.
Related issue: #55

Hi @harshjais369,
your code has some issues but I appreciate your efforts,
I will open new issues to fix the code, you can try it if you can.
Thank you for your contribution.
Happy coding!


[BEAM-10985] Fix flakiness with precommit flakes when publishing test report

The previously declared output directory was completely empty. The test reports are stored in the newly declared output directory.
This will ensure that the task outputs are properly restored and that the tests are re-run when the test reports are missing.
Revert "Revert "Fix up-to-date checking mechanism on Tox Task.""
This reverts commit 36f9c1243e96f9d7f114275be427901dc7a7903b.
The coverage reports are places in the directory declared as an output:

One downside is that this potentially means that this directory is both an input and an output.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @kennknowles R: @udim R: @ibzib

Isn't this "potential downside" a major problem? How do up to date checks work then?

If all the files are in place then the up-to-date checks will work. However, gradle will recognize the overlapping outputs and not pull things from the build cache in that case.

The coverage reports are places in the directory declared as an output

What does that mean? Usually when I think coverage reports, I think of unit test coverage. Is that what you mean here?


maximumLineLength should optionally allow string literals to break the rule

Sometimes one puts a huge data string into a JavaScript file. I'm working with a legacy code base and while I'd like to enforce concise lines of code, I would also like to passthrough massive existing string literals.

See https://medium.com/@markelog/jscs-end-of-the-line-bc9bf0b3fdb2 and
http://eslint.org/blog/2016/04/welcoming-jscs-to-eslint
We are not adding new features at this time

I see. On the readme.md it says that you have merged with ESLint, but I interpreted this to mean that ESLint had merged into you.  Perhaps a note on the readme and on the front page of http://jscs.info/ that this repo is deprecated would help others to redirect themselves appropriately.


Jira integration: cannot select project from modal view

OS version: Windows 10
Browser version: Chrome 79
Extension version: Version 79.0.3945.79 (Official Build) (64-bit)

Relevant integration (if any): Jira Cloud (*.atlassian.net)
🐛 Describe the bug
the Project dropdown is no longer clickable when you click "Start Timer". Also the "Start Timer" button appears twice.
Expected behaviour
There should only be 1 "Start Timer" button and the "Project" dropdown should be clickable.
Steps to reproduce

Enable the Jira integration
Go to a board for a sprint
Open a Jira story modal view
Click "Start Timer" on the Jira story
Attempt to set the Project. It won't work.

Other details or context
It does work if you go to the full page for the Jira story, but not the modal view.

@mrjones2014 thank you for the bug report!
We are working on a fix for the project dropdown at the moment. Could you confirm if both the project and tag dropdowns don't work for you, or just the project one?
Re the timer appearing twice, are you comfortable saving or taking a screenshot of the relevant DOM portion and sending it into our support team?

@lise-toggl confirmed, the issue is for both dropdowns.
Here's some screenshots of the DOM with sensitive information censored.

If it helps, the URL I am using if of the format myorgname.atlassian.net

Thanks @mrjones2014! We're trying some ideas for the dropdown issue and hopefully will find a solution soon.
For the double button issue, would you be able to show me any ancestor DOM elements with the .toggl class? Thanks :)

@lise-toggl I think it may be on Jira stories that are part of an Epic, because it shows 2 Jira numbers (the Epic number / the Story number)

@mrjones2014 awesome - it was indeed the epic and we've been able to reproduce it now. Thanks for the screenshots, I should be able to fix it soon :)

@lise-toggl any idea when this version will be published to the chrome web store?

Hey @mrjones2014, we'll be publishing to the web stores shortly. It sometimes takes a little while to be rolled out to all users, but you should find yourself on the new version some time today 🤞


Documentation and Code Updates

This PR updates AFI to build with the latest Alamofire code (for now), as well as updating the README to reflect the new API names and adding a Migration Guide for version 3.

Made it through...looks great @jshier! Just a few minor tweaks and formatting updates.

@cnoon Comments addressed.


TensorFlow build failure

Operating system
Windows x64
Compiler
Visual Studio 2022
Steps to reproduce the behavior
vcpkg install tensorflow

Failure logs
Error building
-- Configuring TensorFlow (dbg)
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:112 (message):
Command failed: C:/vcpkg/downloads/tools/msys2/70baefe917a26e68/mingw64/bin/python3.exe C:/vcpkg/buildtrees/tensorflow/x64-windows-dbg/configure.py --workspace C:/vcpkg/buildtrees/tensorflow/x64-windows-dbg
Working Directory: C:/vcpkg/buildtrees/tensorflow/x64-windows-dbg
Error code: 1
See logs for more information:
C:\vcpkg\buildtrees\tensorflow\config-x64-windows-dbg-out.log
Call Stack (most recent call first):
installed/x64-windows/share/tensorflow-common/tensorflow-common.cmake:187 (vcpkg_execute_required_process)
ports/tensorflow/portfile.cmake:5 (include)
scripts/ports.cmake:175 (include)
error: building tensorflow:x64-windows failed with: BUILD_FAILED
Additional context
Contents of C:\vcpkg\buildtrees\tensorflow\config-x64-windows-dbg-out.log
Cannot find bazel. Please install bazel/bazelisk.
There is no port for bazel.

I rename bazel to bazel.exe in the .installed\x64-windows\tools directory and get this error in the debug file
You have bazel 5.2.0 installed.
Would you like to override eigen strong inline for some C++ compilation to reduce the compilation time? [Y/n]: Eigen strong inline overridden.

Preconfigured Bazel build configs. You can use any of the below by adding "--config=<>" to your build command. See .bazelrc for more details.
	--config=mkl         	# Build with MKL support.
	--config=mkl_aarch64 	# Build with oneDNN and Compute Library for the Arm Architecture (ACL).
	--config=monolithic  	# Config for mostly static monolithic build.
	--config=numa        	# Build with NUMA support.
	--config=dynamic_kernels	# (Experimental) Build kernels into separate shared objects.
	--config=v1          	# Build with TensorFlow 1 API instead of TF 2 API.
Preconfigured Bazel build configs to DISABLE default on features:
	--config=nogcp       	# Disable GCP support.
	--config=nonccl      	# Disable NVIDIA NCCL support.

Is set the environment variable for bazel (BAZEL_VC) to "C:\Program Files\Microsoft Visual Studio\2022\Community\VC"
and now get
Starting local Bazel server and connecting to it...
ERROR: Expected value after --jobs
/usr/bin/bash: line 1: 17: command not found
/usr/bin/bash: line 1: --compilation_mode=dbg: command not found

The tensorflow port is far from working for windows.
Related: https://github.com/microsoft/vcpkg/pull/31704

This fix was not added to TensorFlow.
https://github.com/microsoft/vcpkg/pull/31704
I am getting farther in the build

Starting local Bazel server and connecting to it...
INFO: Options provided by the client:
  Inherited 'common' options: --isatty=0 --terminal_columns=80
INFO: Reading rc options for 'build' from c:\vcpkg\buildtrees\tensorflow\x64-windows-dbg\.bazelrc:
  Inherited 'common' options: --experimental_repo_remote_exec
INFO: Options provided by the client:
  'build' options: --python_path=C:/vcpkg/downloads/tools/msys2/70baefe917a26e68/mingw64/bin/python.exe
INFO: Reading rc options for 'build' from c:\vcpkg\buildtrees\tensorflow\x64-windows-dbg\.bazelrc:
  'build' options: --define framework_shared_object=true --define=use_fast_cpp_protos=true --define=allow_oversize_protos=true --spawn_strategy=standalone -c opt --announce_rc --define=grpc_no_ares=true --noincompatible_remove_legacy_whole_archive --enable_platform_specific_config --define=with_xla_support=true --config=short_logs --config=v2 --define=no_aws_support=true --define=no_hdfs_support=true --experimental_cc_shared_library --experimental_link_static_libraries_once=false
INFO: Reading rc options for 'build' from c:\vcpkg\buildtrees\tensorflow\x64-windows-dbg\.tf_configure.bazelrc:
  'build' options: --action_env PYTHON_BIN_PATH=C:/vcpkg/downloads/tools/msys2/70baefe917a26e68/mingw64/bin/python3.exe --action_env PYTHON_LIB_PATH=C:/vcpkg/downloads/tools/msys2/70baefe917a26e68/mingw64/lib/python3.11/site-packages --python_path=C:/vcpkg/downloads/tools/msys2/70baefe917a26e68/mingw64/bin/python3.exe --define=with_xla_support=false --copt=/d2ReducedOptimizeHugeFunctions --host_copt=/d2ReducedOptimizeHugeFunctions --define=override_eigen_strong_inline=true
INFO: Reading rc options for 'build' from c:\vcpkg\buildtrees\tensorflow\x64-windows-dbg\.bazelrc:
  'build' options: --deleted_packages=tensorflow/compiler/mlir/tfrt,tensorflow/compiler/mlir/tfrt/benchmarks,tensorflow/compiler/mlir/tfrt/jit/python_binding,tensorflow/compiler/mlir/tfrt/jit/transforms,tensorflow/compiler/mlir/tfrt/python_tests,tensorflow/compiler/mlir/tfrt/tests,tensorflow/compiler/mlir/tfrt/tests/ir,tensorflow/compiler/mlir/tfrt/tests/analysis,tensorflow/compiler/mlir/tfrt/tests/jit,tensorflow/compiler/mlir/tfrt/tests/lhlo_to_tfrt,tensorflow/compiler/mlir/tfrt/tests/lhlo_to_jitrt,tensorflow/compiler/mlir/tfrt/tests/tf_to_corert,tensorflow/compiler/mlir/tfrt/tests/tf_to_tfrt_data,tensorflow/compiler/mlir/tfrt/tests/saved_model,tensorflow/compiler/mlir/tfrt/transforms/lhlo_gpu_to_tfrt_gpu,tensorflow/core/runtime_fallback,tensorflow/core/runtime_fallback/conversion,tensorflow/core/runtime_fallback/kernel,tensorflow/core/runtime_fallback/opdefs,tensorflow/core/runtime_fallback/runtime,tensorflow/core/runtime_fallback/util,tensorflow/core/tfrt/common,tensorflow/core/tfrt/eager,tensorflow/core/tfrt/eager/backends/cpu,tensorflow/core/tfrt/eager/backends/gpu,tensorflow/core/tfrt/eager/core_runtime,tensorflow/core/tfrt/eager/cpp_tests/core_runtime,tensorflow/core/tfrt/gpu,tensorflow/core/tfrt/run_handler_thread_pool,tensorflow/core/tfrt/runtime,tensorflow/core/tfrt/saved_model,tensorflow/core/tfrt/graph_executor,tensorflow/core/tfrt/saved_model/tests,tensorflow/core/tfrt/tpu,tensorflow/core/tfrt/utils
INFO: Found applicable config definition build:short_logs in file c:\vcpkg\buildtrees\tensorflow\x64-windows-dbg\.bazelrc: --output_filter=DONT_MATCH_ANYTHING
INFO: Found applicable config definition build:v2 in file c:\vcpkg\buildtrees\tensorflow\x64-windows-dbg\.bazelrc: --define=tf_api_version=2 --action_env=TF2_BEHAVIOR=1
INFO: Found applicable config definition build:windows in file c:\vcpkg\buildtrees\tensorflow\x64-windows-dbg\.bazelrc: --copt=/W0 --host_copt=/W0 --copt=/Zc:__cplusplus --host_copt=/Zc:__cplusplus --copt=/D_USE_MATH_DEFINES --host_copt=/D_USE_MATH_DEFINES --features=compiler_param_file --copt=/d2ReducedOptimizeHugeFunctions --host_copt=/d2ReducedOptimizeHugeFunctions --cxxopt=/std:c++17 --host_cxxopt=/std:c++17 --config=monolithic --copt=-DWIN32_LEAN_AND_MEAN --host_copt=-DWIN32_LEAN_AND_MEAN --copt=-DNOGDI --host_copt=-DNOGDI --copt=/experimental:preprocessor --host_copt=/experimental:preprocessor --linkopt=/DEBUG --host_linkopt=/DEBUG --linkopt=/OPT:REF --host_linkopt=/OPT:REF --linkopt=/OPT:ICF --host_linkopt=/OPT:ICF --verbose_failures --features=compiler_param_file --distinct_host_configuration=false
INFO: Found applicable config definition build:monolithic in file c:\vcpkg\buildtrees\tensorflow\x64-windows-dbg\.bazelrc: --define framework_shared_object=false --experimental_link_static_libraries_once=false
WARNING: Usage: bazel build <options> <targets>.
Invoke `bazel help build` for full description of usage and options.
Your request is correct, but requested an empty set of targets. Nothing will be built.
Loading:
Loading: 0 packages loaded
Analyzing: 0 targets (0 packages loaded)
Analyzing: 0 targets (0 packages loaded, 0 targets configured)
INFO: Analyzed 0 targets (0 packages loaded, 0 targets configured).
INFO: Found 0 targets...
[0 / 1] [Prepa] BazelWorkspaceStatusAction stable-status.txt
INFO: Elapsed time: 2.788s, Critical Path: 0.03s
INFO: 1 process: 1 internal.
INFO: Build completed successfully, 1 total action
INFO: Build completed successfully, 1 total action
/usr/bin/bash: line 1: --compilation_mode=dbg: command not found


./apisnoop.sh --download-apiusage should respect data-gen/sources.yaml

Given an checkout of apisnoop
And ./data-gen/sources.yaml referencing specific gcs buckets
And there are no other files in ./data-gen/processed/
When I run ./apisnoop.sh --download-apiusage
Then ./data-gen/processed/ will only include apiusage for jobs defined in ./data-gen/sources.yaml

Currently we are using gsutil to recursively download the entire GCS prefix:
download_apiusage() {
  mkdir -p "$APISNOOP_DEST"
  gsutil -m cp -R -n "$APISNOOP_GCS_PREFIX" "$APISNOOP_DEST"
}

The function should loop through ./data-gen/sources.yaml only downloading defined processed job output.


Documentation error under StreamingClient.delete_rules

There's an error in the documentation under StreamingClient.delete_rules section.

"dry_run (bool) – Set to true to test a the syntax of your rule without submitting it. This is useful if you want to check the syntax of a rule before removing one or more of your existing rules."

Refer to link

Thanks! This was actually a typo copied and pasted from the Twitter API documentation on https://developer.twitter.com/en/docs/twitter-api/tweets/filtered-stream/api-reference/post-tweets-search-stream-rules.


load 3dtiles the scene appear blink

when load 3dtiles the scene appear blink .
Is the computer setting problem and how to config?
The specific situation see the red box in screenshot.
any help thank you!
offical url

Cesium Version 1.64.

This is a known issue, see https://github.com/AnalyticalGraphicsInc/cesium/issues/7189 for a discussion of the cause and a workaround.

This is a known issue, see #7189 for a discussion of the cause and a workaround.

thank you, my problem solved.


DEV-15887: Include title in epub manifest if no subtitle

This epubTitle function should now work in all permutations of readingLine and subtitle being defined (assuming title is defined, which if it isn't causes problems elsewhere)

@mphstudios changes applied + tested


Sync api

Codecov Report
Attention: Patch coverage is 47.61905% with 22 lines in your changes missing coverage. Please review.

Project coverage is 48.54%. Comparing base (369c9fa) to head (066fe42).
Report is 6 commits behind head on main.

Files
Patch %
Lines

src/desktop_notifier/main_sync.py
45.00%
22 Missing :warning:

:exclamation:  There is a different number of reports uploaded between BASE (369c9fa) and HEAD (066fe42). Click for more details.
HEAD has 10 uploads less than BASE
| Flag | BASE (369c9fa) | HEAD (066fe42) |
|------|------|------|
|pytest|12|2|

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #145       +/-   ##
===========================================
- Coverage   75.03%   48.54%   -26.49%
===========================================
  Files           8        9        +1
  Lines         833      861       +28
===========================================
- Hits          625      418      -207
- Misses        208      443      +235

Flag
Coverage Δ

pytest
48.54% <47.61%> (-26.49%)
:arrow_down:

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

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


Can't set primary key property to existing value

I have two model which are related.
// LMRRealmBook.h
@interface LMRRealmBook : RLMObject
@property NSInteger bookId;
@property NSString *name;
@property RLMArray<LMRRealmChapter *><LMRRealmChapter> *chapters;
@end
RLM_ARRAY_TYPE(LMRBookDetail)

// LMRRealmBook.m
@implementation LMRRealmBook

#pragma mark - Realm
+ (NSArray *)indexedProperties {
    return @[@"name"];
}
+ (NSString *)primaryKey {
    return @"bookId";
}

@end

// LMRRealmChapter.h
@interface LMRRealmChapter : RLMObject
@property NSInteger chapterId;
@property NSInteger bookId;
@property NSString *title;
@end

// LMRRealmChapter.m
@implementation LMRRealmChapter
#pragma mark - Realm
+ (NSArray *)indexedProperties {
    return @[@"title"];
}

+ (NSString *)primaryKey {
    return @"chapterId";
}

#pragma mark - Setters & Getters
- (NSArray *)owners {
    return [self linkingObjectsOfClass:@"LMRRealmBook" forProperty:@"chapters"];
}
@end

I wonder how to insert object to LMRRealmBook if i have a array of LMRRealmChapter?
Anyone know how to do that?
Thanks very munch!

With Realm, you don't need foreign keys to have relationships between objects.
// Creating a book with the same primary key as a previously saved book
LMRRealmBook *book = [[LMRRealmBook alloc] init];
book.title = @"Lord of the Rings I";
book.bookId = @1;

LMRRealmChapter *chapter1 = [[LMRRealmBook alloc] init];
chapter1.title = @"A Long-expected Party";
chapter1.chapterId = @1;

LMRRealmChapter *chapter1 = [[LMRRealmBook alloc] init];
chapter2.title = @"The Shadow of the Past";
chapter2.chapterId = @2;

[book.chapters addObject:chapter1];
[book.chapters addObject:chapter2];

// Updating book with id = 1
// This will automatically update all related chapters and create new ones.
[realm beginWriteTransaction];
[Book createOrUpdateInRealm:realm withValue:cheeseBook];
[realm commitWriteTransaction];

Given that you have already some independent books and chapters in your Realm, you can connect them by:
// Given that there are already disconnected objects LMRRealmBook with bookId = 1
// and objects of LMRRealmChapter with the bookId = 1.
[realm beginWriteTransaction];
LMRRealmBook *book = [LMRRealmBook objectForPrimaryKey:@1];
NSArray *chapters = [LMRRealmChapter objectsWhere:@"bookId = %@", @1];
[book.chapters addObjects:chapters];
[realm commitWriteTransaction];

Perhaps you might be also looking for +[RLMObject createOrUpdateInDefaultRealmWithValue:]. You can use that e.g. like seen below to map an object structure retrieved from JSON to a Realm object:
[LMRRealmBook createOrUpdateInDefaultRealmWithValue:@{
     @"bookId": @1,
     @"name": @"Lord of the Rings I",
     @"chapters": @[
         @{ @"chapterId": @1, @name: @"A Long-expected Party"  },
         @{ @"chapterId": @2, @name: @"The Shadow of the Past" }
         /* … */
     ]
}];

I hope this helps. Please let me know what you were looking for and how might be able to improve our docs around that.

I see. Thanks you very much!


modmenu requires fabric-resource-loader

this bug prevents the game from loading I even got fabric-resource-loader and it still crashes

nvm I guess I needed this mod called fabric api


Extract schools data from Airtable

Extract based on Workstream == CLE

Update on conversation with Chris:
We are looking to pull Event type == CLE - Creative Learning session.
However:
a) there are some items currently labelled as this which are incorrect. Chris is addressing this internally
b) There is one item which should be included which show as "CLE - Creative Learning session,Public Event - Performance" in excel extract. This is likely because the entry has two tags. There may be future events which have multiple tags, one of which is "CLE - Creative Learning session" which should be included in the schools data

Hey @gilesdring, L23 got in touch today about this and points raised on https://github.com/open-innovations/leeds-2023/issues/28
L23 will be adding data to Airtable as proposed in the email e.g. there will be one entry per event with multiple schools tagged. They are aware that this means we will not be able to visualise pupils engaged by ward - and I will make an amendment to our definition of done to match this.

Thanks for this @gilesdring. Reflections:

After I said "Event type == CLE - Creative Learning session" in the comment above, you corrected me to point out that it needed to include CLE etc, not ==, when I highlighted the fact that there maybe multiple tags on activities (given that it seemed csv downloads merged these tags.
We have a lot of 'test school activity' which L23 will need to remove once we're happy we've got the right extract
Looking through the data it's interesting to note that there are a bunch of activities which I would consider not to be schools level activities e.g. row 2, Leeds Beckett Freshers Fair, rows 21 and 30 which appear to be teachers meetings.
I'll take a look through the data extract you previously shared which pulled master events out of AirTable and compare and contrast to check what we're pulling out is what we expect to see based on previous conversations.

I will share a copy of the extract with Chris once we're happy with this to check with him

Pipelines now more or less working (MVP style)
Data is processed to this form: https://github.com/open-innovations/leeds-2023/blob/main/data/metrics/schools/schools_events.csv
Then prepared for presentation in here
https://github.com/open-innovations/leeds-2023/tree/main/docs/metrics/schools/_data
summary.json - top level stats
engagements_by_week = time series of engagements with particpants / schools, including cumulative. Note this is not equivalent to events - if there is one event, but 5 schools turned up to it, we'd end up with a count of 5 here.
schools_engagement_counts = number of engagements with each schools

@gilesdring - As mentioned before there is one item that is on the Master Events list, that isn't being pulled through into the extract so far.
This is Feel the Music: Feel the Music (AirTable ID 401).
We will have a blocker on this item at present, as it is a future event (July 2023). However it is also our only event with two tags in the system at present.
I don't know if you want to test the extraction process with the 'past events only' filter removed to test if we are extracting events with two tags - or if you want to pick this up in the future.

Email sent to L23 re: this data

@gilesdring i have feedback from L23 on the email I sent them. Would be good to chat about this today if possible

Will need to pick up in next sprint. Kicking back into Ready column

Sounds good, contact at L23 is off today anyway, so we won't get any further response from them until later in week.

Email sent and as work is progressing on #28, I think we can consider this closed now.


elf: parse library from symbol versioning table

This change parses the library name for a dynamic symbol from the ELF symbol versioning table and removes a // TODO

Thanks for the reviews.


intptrcast: Never allocate two objects directly adjecent

When two objects directly follow each other in memory, what is the
provenance of an integer cast to a pointer that points directly between
them?  For a zero-size region, it could point into the end of the first
object, or the start of the second.
We can avoid answering this difficult question by simply never
allocating two objects directly beside each other.  This fixes some of
the false positives from #1866.

Thanks for the MR!
I am not very happy about this work-around, since it is just that -- it doesn't actually solve the problem, it just makes it less likely to occur in practice. But that is probably still better than the status quo. I opened https://github.com/rust-lang/unsafe-code-guidelines/issues/313 to track the fundamental problem that causes this.

I am not very happy about this work-around, since it is just that -- it doesn't actually solve the problem, it just makes it less likely to occur in practice.

Agreed, although depending on what “the problem“ is, I expect it's either very difficult or impossible to solve :).  I'm not sure there's a consistent semantics at all for a language with both pointer provenance and pointer<->integer casts.  Though I haven't been following the C provenance working group stuff.

I'm not sure there's a consistent semantics at all for a language with both pointer provenance and pointer<->integer casts.

I sure hope there is, because otherwise I have to either remove int-to-ptr casts from Rust or live with the fact that Rust is internally inconsistent and fundamentally impossible to get right. ;)  But if there is a counterexample that demonstrates that no such semantics can exist, that would sure be very interesting.

@bors r+
Thanks again for the fix. :)

:pushpin: Commit 6a98c64c8b684ad12ae1a11fc7ad55a176b45bfe has been approved by RalfJung

:hourglass: Testing commit 6a98c64c8b684ad12ae1a11fc7ad55a176b45bfe with merge 81e59e6b92cf1729aabbbbf09b81a81a03775d64...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing 81e59e6b92cf1729aabbbbf09b81a81a03775d64 to master...


socket.io.js(3.x,4.x) can't interact with servesocketio server(netty-socketio)

I start netty-socketio server, try to use socket.io.js(3.x,4.x) to connect the server. The connection is established between client and server, but amazing things has happened as follows.

server received two connection events.
server can't receive message from client, and client can't receive message from server.

But when I downgrade to version under version 3.0(e.g. 2.4.0), all things works as excepted.

fixed in https://github.com/mrniko/netty-socketio/pull/897


Need a clarification

Hi,
I am trying to understand how netopeer can help me in bringing up mplane. As I am new starting with basic doubts:
Objective:
Establish TLS netconf session with client->Initiate Callhome->On callhome reply wait for request
So to achieve this, am I supposed to netconf API for TLS session and and use sysrepo api,the callback will invoke to read the config data.
OR in netopeer we have apis which can be achieved without calling sysrespo apis explicitely.
Regards,
Biswajit

netopeer2 is just a NETCONF interface for sysrepo, you will not interact with it locally at all. I would suggest you start by reading about sysrepo to learn about its features. As for a NETCONF connection, I do not understand what you want to do. You can either connect normally or using Call Home so I am going to assume you want the latter. For that you will have to configure netopeer2 to initiate Call Home connections, find out in README how. You can use sysrepocfg to import/merge configuration files into sysrepo.


i2c Display with mcp23008 wrong pins

Hi there,
I have a i2c backpack from Adafruit that seems to have a slightly different pinout to what is written in your code as i can't get it for the life of me to display any characters but i can't find the pin definitions within the code. attached is a circuit schematic that has the pinout of the backpack.
kind regards
Tiaan Giliomee

Hmm.  pydPiper does support i2c displays.  Do you have i2c enabled correctly?  What address does your display show up as?  (e.g. what does i2cdetect report)?
Take a look at https://learn.adafruit.com/adafruits-raspberry-pi-lesson-4-gpio-setup/configuring-i2c

As I ran the script for pydPiper the sreens backlight started flickering at regular intervals with just a series of blocks on the screens, this led me to believe that my i2c board might be malfunctioning. I then proceeded to use a quick test script from the Adafruit libraries that is supposedly written for the backpackfor which I got the same results, backlight still flickers. After editing the pins that correspond to the above attached pcb diagram I had success with the messages being displayed clear as daylight which lead me to belive that there is possibly a fault with the pin mapping for this specific backpack within the code.
this is the backpack in question.
https://www.adafruit.com/product/292

I'm not sure what pins you are referring to.  When using an i2c device interfaced with a Pi, you will always be connected to pins 3 and 5.  Can you share the code you used for the test that worked?

This is the code that has worked, the i2c from the mcp23008 is connected to those pins, its the pins on the mcp23008 to the display that need to be mapped.
import Adafruit_CharLCD as LCD
import Adafruit_GPIO.MCP230xx as MCP
GPIO = MCP.MCP23008(address=0x27, busnum=1)
Define PCF pins connected to the LCD.
lcd_rs        = 1
lcd_en        = 2
d4,d5,d6,d7   = 3,4,5,6
lcd_backlight     = 7
cols,lines    = 16,2
Instantiate LCD Display
lcd = LCD.Adafruit_CharLCD(lcd_rs, lcd_en, d4, d5, d6, d7,
cols, lines, lcd_backlight, gpio=GPIO)
lcd.set_backlight(1.0)
lcd.clear()
lcd.message('  Raspberry Pi\nI2C LCD MCP23008')

If it helps at all.
This library works perfectly by using the simple initialize for the mcp23008.
https://github.com/dbrgn/RPLCD/tree/master/RPLCD

The problem is that the MCP23008 uses different pin assignments than a standard I2C hd44780 display.  Using the code you provided as an example, I have created a new driver specific to the MCP23008 backpack on an HD44780 LCD.  Unfortunately, I do not have an MCP23008 so I am not in a position to test the new driver.  Pull the changes from the repo and try to run the driver standalone.
python hd44780_mcp23008.py -r 16 -c 80

Just updated libraries, etc. ran the code and still getting a flashing backlight with blocks on the screen.
the frequency seems to shift as it runs through the code so the mapping isn't quite right

Not sure I can take it much further as I lack the hardware to test the solution.

I'm quite new to python as i mainly code in Java, would it help if we scheduled a time to try various solutions?

The code for the driver is reasonably well contained so feel free to experiment.  It can be run separate from the rest of the pydPiper code for testing.


Update kuttl to v0.15.0 in the scorecard-test-kuttl image

Description of the change:
Update kuttl to v0.15.0 in the scorecard-test-kuttl image
Motivation for the change:
Fixes cleanup of created resources in kuttl (created resources of a kuttl test will be deleted after the test is run).
This change is especially relevant when using a single namespace for testing (default with this image).
See https://github.com/kudobuilder/kuttl/releases/tag/v0.14.0
Checklist
If the pull request includes user-facing changes, extra documentation is required:

[X] Add a new changelog fragment in changelog/fragments (see changelog/fragments/00-template.yaml)
[ ] Add or update relevant sections of the docs website in website/content/en/docs

@oceanc80 @everettraven could we move this PR forward? It's blocking us in https://github.com/open-telemetry/opentelemetry-operator/issues/1721

@oceanc80 @everettraven  could we please merge the PR?

Closing and re-opening to pick up latest CI changes/fixes

@oceanc80 @everettraven when can we expect this to be released?

@pavolloffay this should be included in the v1.29.0 release which is currently slated for May 31st

Great, thanks for merging!

@everettraven do you know when the container image at https://quay.io/repository/operator-framework/scorecard-test-kuttl?tab=tags&tag=latest gets tagged with a new version?

@andreasgerstmayr I would have assumed with the 1.29.0 release it would have happened but that doesn't seem to be the case ... Would you mind opening a new issue regarding this so we can track it and get some eyes on it?

@everettraven sure, I've created https://github.com/operator-framework/operator-sdk/issues/6456. Thanks!


Add function Events::clear(&mut self)

Add function Events::clear(&mut self) and expose to other layers of the module or user-app

In kqueue.rs the Selector::select() method does not perform any clearing. So I am not sure the implementation Events::clear() is correct there!?


Fix Incorrect time calculation for dates before the Unix epoch

AMQP timestamp is a signed value, but decoded from buffer as unsigned.
The previous code took the remainder of the unsigned value before
converting to signed which is incorrect for negative timestamps.
Fixes  #173

Thanks!


Practice questions + many fixes

Closes #170
Closes #160

many things that aren't in the issues

Merging this as practice basics are done (not well tested) and this is getting way out of hand


Semantic Segmentation on ADE20K

Thank you for your outstanding work, have you tested the Semantic Segmentation on ADE20K in the code? I have encountered many problems (such as mmcv version issues and model init problems), I want to confirm that you can use this code to test normally on ADE20k？

Hi,
Sorry for the mistakes. This is due to the compatibility issue. We run the repo with mmsegmentation==0.12.0 and you can install by running the following command:
git clone -b v0.12.0 https://github.com/open-mmlab/mmsegmentation
cd mmsegmentation
pip3 install -v -e .
cd ..

or simply:
pip3 install mmsegmentation==0.12.0

We have updated the configuration in INSTALL.md.

Sorry for getting back to you so late. Thanks for your kind reply!

Sorry to bother you again, I ran into some problems during testing, when test in training process, there are some problems in the eval_hooks as follow:

when just test by test.py, another problem arises:

I tried different versions of mmcv and mmseg but none of them worked out, hope you can provide some help

While evaluating ade20k_dense_linear, I get the following error.
KeyError: "EncoderDecoder: 'VisionTransformer is not in the backbone registry'"

Sorry to bother you again, I ran into some problems during testing, when test in training process, there are some problems in the eval_hooks as follow:  when just test by test.py, another problem arises:  I tried different versions of mmcv and mmseg but none of them worked out, hope you can provide some help

Hi @Trent-tangtao,
I've been recently using mmseg and encountered the first issue you mentioned NotImplementedError: Do not know how to handle these types to promote: {'FloatTensor', 'DoubleTensor'} is due to type mismatch can not be handled automatically when using apex. There are two solutions in my case. The first is not using apex when eval, i.e., commenting the apex optimizer and using the default optimizer in the config. The second is to change
    total_area_intersect = torch.zeros((num_classes, ), dtype=torch.float64)
    total_area_union = torch.zeros((num_classes, ), dtype=torch.float64)
    total_area_pred_label = torch.zeros((num_classes, ), dtype=torch.float64)
    total_area_label = torch.zeros((num_classes, ), dtype=torch.float64)

into
    total_area_intersect = torch.zeros((num_classes, ), dtype=torch.float)
    total_area_union = torch.zeros((num_classes, ), dtype=torch.float)
    total_area_pred_label = torch.zeros((num_classes, ), dtype=torch.float)
    total_area_label = torch.zeros((num_classes, ), dtype=torch.float)

here in mmseg lib. Hope that solves your issue!


FR translations

Init FR translation with Googletranslate : https://support.google.com/docs/answer/3093331

Thanks for contribution!


Colormap range is reset to default if a nan array is shown.

When scrolling through layers of 3-D data, if a layer of nans is shown, then the colormap range setting is lost and resets to the default.

Thanks!


Adding testing for BuildManifestListOrSingleManifestStep

This PR adds testing for BuildManifestListOrSingleManifestStep

This PR will closed without merging to master : We have made a design decision to merge createManifestListTest into createManifestList then in the end createManifestList will be merged into master


UsingAliasDirectivesMustBeOrderedAlphabeticallyByAliasName - Orderer by Current Culture

Why UsingAliasDirectivesMustBeOrderedAlphabeticallyByAliasName rule doesn't follow Invariant Culture ?
In Slovak Culture we have special letter 'CH', which goes after H. (a b c d e f g h ch i j ...)
For example:
Namespaces Core.Build, Core.Chain and Core.Dependency are ordered in English as
Core.Build
Core.Chain
Core.Dependency

but in Slovak
Core.Build
Core.Dependency
Core.Chain

as our code is English, but Windows locales is Slovak we have inconsistency.

Duplicate of #87.


Point to the contributors in the README

Currently the code reads as:
See also the list of [contributors](https://github.com/your/project/contributors) who participated in this project.
This should be fixed to actually point to the contributors in this project.

Hi @arvind0598,
I would like to fix this one.

Fixed in #14


Rework QG docu

This PR contains a few suggestions.

Many thanks for the feedback! I accepted most of the changes.


[Update Extension] Google Tasks

Description

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

@elliotdes do you mind checking this?

Just managed to get around to this. Looks good to me! Happy to merge @pernielsentikaer


Async implementation of RegisterDoneEvent

What does this Pull Request accomplish?
Replace the synchronous, thread-blocking implementation of RegisterDoneEvent with an async implementation.
This is implemented as a "hybrid" grpc server, which starts with a synchronous server and opts out specific methods using WithAsyncMethod_.
Why should this Pull Request be merged?
Allows grpc-device to implement callbacks without blocking a grpc thread for each call.
What testing has been done?
Ran and passed all tests. Some extra testing with python client.

Overall I think this seems like a reasonable direction, although I'll admit I don't follow 100% all of the code. But I don't see any red flags in particular.


update linux.rs

Optimization: If ASLR is already turned off, the disable_aslr() function will no longer be called.
This can expand the usage scenarios of tarpaulin. For example, after the root permission administrator shuts down the server ASLR, ordinary users can still use tarpaulin normally even if they cannot obtain root permission~

Cool code looks fine from a brief skim. I'm abroad right now so I'm look properly on Monday when I'm back home

thank you,tarpaulin has been a great help to me~  (^o^)/

Hello~ May I ask if it is convenient for you to publish the latest tarpaulin code on http://cartes.io recently? I want to try out the new features XD @xd009642

Sure I'll do a release this weekend or Monday

And release is done, only a day later than planned :sweat_smile:


Email receipt showing too many nomination lines

From Marguerite Smith:
I get six entries in the email record (specifically for Short Story & Semiprozine), not five.

I would need to see that email in order to address this.

Emailed that email to you at your Glasgow account.

I would appreciate it if someone can check if this is still occurring, after a ballot has been updated.