error log -  Get consumer records has error, msg is Error reading field 'metadata': String length -16913 cannot be negative

[2018-03-02 17:15:09] KafkaOffsetGetter.Thread-58 - ERROR - Get consumer records has error, msg is Error reading field 'metadata': String length -16913 cannot be negative
cat logs/error.log| grep "ERROR - Get consumer records has error, msg is Error reading field" | wc -l
1095600

@saup007
Maybe your kafka version is old. What is your current version of Kafka ?

@smartloli kafka cluster <IP_ADDRESS>


npm persmission issue while installing on local machine.

solution of issue#1..fix npm permissions, before doing npm install.

Codecov Report

Merging #1093 into development will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           development    #1093   +/-   ##
============================================
  Coverage        62.42%   62.42%
============================================
  Files                8        8
  Lines             1344     1344
  Branches           210      210
============================================
  Hits               839      839
  Misses             505      505

Continue to review full report at Codecov.

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


Bindgen Refactor

See https://github.com/dylibso/xtp/issues/887
The Go PR maybe be the closest analogue https://github.com/dylibso/xtp-go-bindgen/pull/22

Also tracked by https://github.com/dylibso/xtp/issues/890


mspider already released v0.10.0

Release notes: https://gitlab.daocloud.cn/ndx/mspider/-/blob/release/0.10/changes/v0.10/v0.10.0-minorReleaseNotes.md
Waiting for creating release notes in docs.

tip: a good habit not to share private links


Service 'rails' failed to build

When trying to start dev/docker-compose.yml, I get the following error:
 The command '/bin/sh -c wget --quiet -O /opt/jdk-8u172-ea-bin-b03-linux-x64-18_jan_2018.tar.gz --no-check-certificate --no-cookies --header 'Cookie: oraclelicense=accept-securebackup-cookie' https://download.java.net/java/jdk8u172/archive/b03/binaries/jdk-8u172-ea-bin-b03-linux-x64-18_jan_2018.tar.gz &&     tar xzf /opt/jdk-8u172-ea-bin-b03-linux-x64-18_jan_2018.tar.gz -C /opt &&     rm /opt/jdk-8u172-ea-bin-b03-linux-x64-18_jan_2018.tar.gz &&     update-alternatives --install /usr/bin/java java /opt/jdk1.8.0_172/bin/java 1000' returned a non-zero code: 8
Looks like the same or similar issue encountered in: https://github.com/asciinema/asciinema-server/issues/260

oracle changed the url, you have to use a different url.

I have just (4 weeks ago) updated this URL (https://github.com/asciinema/asciinema-server/commit/a016895188dee7715af1875a05604cc98b2150ca) and they broke it again... 😞

It’s ok, I understand that’s kind of out or your hands haha
The annoying part is that it was pointed at the archive! Archives should be immutable... You’d think someone would have asked “before we change this url, how many monthly active downloads do we have?”
Anyways, I wonder if there is a way to build with the right jdk pre downloaded.

On Apr 21, 2018, at 1:10 PM, Marcin Kulik<EMAIL_ADDRESS>wrote:
I have just (4 weeks ago) updated this URL (a016895) and they broke it again... 😞
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Yeah... Anyway, I'm happy to accept PR with fixed URL.
I made this Dockerfile.dev for development purposes, mainly because there's many dependencies at the moment due to migration from Rails to Phoenix (Rails and Elixir/Phoenix deps are needed). The setup will be much simpler when we get rid of Rails completely. Frankly, I am now developing on bare metal, it's just too much hassle with Docker for dev.

Service rails is no longer part of docker-compose so I'm gonna close it.


Added the C++11 changes back in. Replaces #327

Ok so these are the C++11 changes as discussed I think. I've included all the changes you've made up this point. Is this better for you?

I've kept the 'doc' directory with the markdown in it, so hopefully we'll be able to integrate that later on.

@OniDaito there was a few issues though, see my additional commits.
In case the build is failing (including SOCI_CXX_C11=OFF), feel free to submit additional PRs to correct things. I will try to add C++11 job to Travis CI, but AFAIS setting cutting-edge GCC there is PITA and clang seems to be a troublemaker with Oracle.


microk8s.enable metrics-server fails due to 'no matches for kind "APIService" in version "apiregistration.k8s.io/v1beta1"'

In a freshly installed instance microk8s.enable metrics-server fails due to error: unable to recognize "/home/richter/snap/microk8s/2284/tmp/temp.metrics-server.yaml": no matches for kind "APIService" in version "apiregistration.k8s.io/v1beta1". The complete output is
$ microk8s.enable metrics-server
Enabling Metrics-Server
clusterrole.rbac.authorization.k8s.io/system:aggregated-metrics-reader created
clusterrolebinding.rbac.authorization.k8s.io/metrics-server:system:auth-delegator created
rolebinding.rbac.authorization.k8s.io/metrics-server-auth-reader created
serviceaccount/metrics-server created
deployment.apps/metrics-server created
service/metrics-server created
clusterrole.rbac.authorization.k8s.io/system:metrics-server created
clusterrolebinding.rbac.authorization.k8s.io/system:metrics-server created
clusterrolebinding.rbac.authorization.k8s.io/microk8s-admin created
error: unable to recognize "/home/richter/snap/microk8s/2284/tmp/temp.metrics-server.yaml": no matches for kind "APIService" in version "apiregistration.k8s.io/v1beta1"

microk8s.disable metrics-server fails due to the same error:
$ microk8s.disable metrics-server
Disabling Metrics-Server
Error from server (NotFound): configmaps "metrics-server-config" not found
Error from server (NotFound): deployments.apps "metrics-server-v0.2.1" not found
clusterrole.rbac.authorization.k8s.io "system:aggregated-metrics-reader" deleted
clusterrolebinding.rbac.authorization.k8s.io "metrics-server:system:auth-delegator" deleted
rolebinding.rbac.authorization.k8s.io "metrics-server-auth-reader" deleted
serviceaccount "metrics-server" deleted
deployment.apps "metrics-server" deleted
service "metrics-server" deleted
clusterrole.rbac.authorization.k8s.io "system:metrics-server" deleted
clusterrolebinding.rbac.authorization.k8s.io "system:metrics-server" deleted
clusterrolebinding.rbac.authorization.k8s.io "microk8s-admin" deleted
error: unable to recognize "/home/richter/snap/microk8s/2284/tmp/temp.metrics-server.yaml": no matches for kind "APIService" in version "apiregistration.k8s.io/v1beta1"

The reproduction occurred on a fresh installation after snap remove --purge microk8s in both of two attempts.
experienced with v1.22.0-alpha.3 (2284) on Ubuntu 21.04, but not with v1.21.1 (2262), so clearly limited to the bleeding edge version
inspection-report-20210623_164715.tar.gz

Thank you for reporting this @krichter722 . We will have to update the metrics server for the next releases.


create list of all part revisions required

Discussed in https://github.com/krachtr/FtcRobotController22-23/discussions/8

Originally posted by krachtr September  6, 2023
What revisions are required to the onshape parts?

this it the list i have
Think about removing the encoder pod and put the encoder on the wheel
Encoder pod block
Encoder pod gap width bigger
Modor mount screw holes bigger
Encoder on the outside of the pod and fill in the pod so the Encoder pod has motor support
Spring hooks bigger
Open encoder back for the baring
Fix design so you can get shaft out easier
Motor set screw hole bigger


[C#] Consider replacing LINQ expression when accessing columns by name

Describe the enhancement requested
We've recently been doing some profiling of code that uses the C# Arrow library, and which frequently accesses columns by name in a tight loop - and found the following LINQ expression makes a significant contribution to execution time.
https://github.com/apache/arrow/blob/2df7b23af717f505c16fedbef6e4223ac163ac5a/csharp/src/Apache.Arrow/Schema.cs#L81-L86
Obviously there are workarounds for this, such as creating our own cached mapping of column names to ordinals - but it would be nice if the built-in method was a little more efficient. In addition to the use of linq, we're also iterating through _fieldsList twice (once to find a match, and then another to find the index of the match).
We've also noticed that usage of StringComparer.CurrentCulture causes an allocation every time this method is called (assuming a value for comparer is not provided). I'm assuming the choice to use a culture-aware comparison by default is intentional?
Component(s)
C#

The use of StringComparer.CurrentCulture is definitely a head-scratcher. It apparently dates back to the very first checkin of the file. It's especially surprising because the very same checkin used StringComparer.OrdinalIgnoreCase to build the dictionary. While I ordinarily argue against making breaking changes, I think it might be justified this time.
I would expect the allocation to be a result of the closure that needs to be created to pass to First.

Issue resolved by pull request 44576
https://github.com/apache/arrow/pull/44576


add metrics tracking lucene near real-time refresh delay

Since the realtime Lucene index doesn't immediately make results available to searchers, this can cause confusion for users who see data, then search for it only to get no results.
This adds two gauges:
LUCENE_INDEX_DELAY_MS
LUCENE_INDEX_DELAY_DOCS

These are per partition, and contain the value of the greatest delay across all columns. e.g., if you have two text indexes, on colA, and colB, if colA delay = 10s and colB delay = 20s, the gauge will emit 20s. This is pre-aggregated to avoid high cardinalities which is problematic for many metric systems.
Tested in an internal cluster.
sample delay:

suggested tags: enhancement

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

Project coverage is 0.00%. Comparing base (59551e4) to head (bf9a824).

Files
Patch %
Lines

...ertedindex/RealtimeLuceneIndexingDelayTracker.java
0.00%
65 Missing :warning:

...l/invertedindex/RealtimeLuceneRefreshListener.java
0.00%
34 Missing :warning:

...me/impl/invertedindex/RealtimeLuceneTextIndex.java
0.00%
11 Missing :warning:

...a/org/apache/pinot/common/metrics/ServerGauge.java
0.00%
2 Missing :warning:

...ment/creator/impl/text/LuceneTextIndexCreator.java
0.00%
1 Missing :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##             master   #13307       +/-   ##
=============================================
- Coverage     61.75%    0.00%   -61.76%
=============================================
  Files          2436     2463       +27
  Lines        133233   135754     +2521
  Branches      20636    20989      +353
=============================================
- Hits          82274        0    -82274
- Misses        44911   135754    +90843
+ Partials       6048        0     -6048

Flag
Coverage Δ

custom-integration1
?

integration
0.00% <0.00%> (-0.01%)
:arrow_down:

integration1
?

integration2
0.00% <0.00%> (ø)

java-11
?

java-21
0.00% <0.00%> (-61.63%)
:arrow_down:

skip-bytebuffers-false
0.00% <0.00%> (-61.75%)
:arrow_down:

skip-bytebuffers-true
?

temurin
0.00% <0.00%> (-61.76%)
:arrow_down:

unittests
?

unittests1
?

unittests2
?

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.

cc @swaminathanmanish


The building steps assume the thrift program is available by default

Following the steps to build the project, it failed in my environment due to lacking of thrift tool:
Downloaded: https://repo.maven.apache.org/maven2/com/google/collections/google-collections/0.8/google-collections-0.8.jar (473 KB at 157.1 KB/sec)
[ERROR] thrift failed output:
[ERROR] thrift failed error: 'thrift' is not recognized as an internal or external command,
operable program or batch file.
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] carbondata ......................................... SUCCESS [  5.717 s]
[INFO] carbon-common ...................................... SUCCESS [  7.693 s]
[INFO] carbon-format ...................................... FAILURE [ 20.535 s]
[INFO] carbon-core ........................................ SKIPPED
[INFO] carbon-processing .................................. SKIPPED
[INFO] carbon-hadoop ...................................... SKIPPED
[INFO] carbon-spark ....................................... SKIPPED
[INFO] carbon-assembly .................................... SKIPPED
[INFO] carbon-examples .................................... SKIPPED
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 34.313 s
[INFO] Finished at: 2016-05-15T16:07:14-07:00
[INFO] Final Memory: 19M/169M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.thrift.tools:maven-thrift-plugin:0.1.11:compile (generate-thrift-java) on project carbon-format: thrift did not ex
leanly. Review output for more information. -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException
[ERROR]
[ERROR] After correcting the problems, you can resume the build with the command
[ERROR]   mvn  -rf :carbon-format

I think we should add the building info like other open source projects are providing. For example  hadoop provides building details in
BUILDING.txt.

Yeah, sound good to have the txt file and then mention the doc file in the home page I guess.


Remove: Unity.Unity.6000.MacServer version 6000.0.14f1

Microsoft Reviewers: Open in CodeFlow

URL: https://download.unity3d.com/download_unity/e12ab2d6a089/TargetSupportInstaller/UnitySetup-Mac-Server-Support-for-Editor-6000.0.14f1.exe
OK
(Automated message - build 923)

https://github.com/microsoft/winget-pkgs/pull/186121#issuecomment-2445759087

[Policy] Needs-Manual-Merge


[Snyk] Fix for 2 vulnerabilities

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `pip` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

apps/sphinx-doc/utils/requirements.txt

Vulnerabilities that will be fixed
By pinning:

Severity
Issue
Upgrade
Breaking Change
Exploit Maturity

Regular Expression Denial of Service (ReDoS) SNYK-PYTHON-PYGMENTS-5750273
pygments:  2.11.2 -> 2.15.0
No
No Known Exploit

Information Exposure SNYK-PYTHON-REQUESTS-5595532
requests:  2.27.1 -> 2.31.0
No
No Known Exploit

Some vulnerabilities couldn't be fully fixed and so Snyk will still find them when the project is tested again. This may be because the vulnerability existed within more than one direct dependency, but not all of the affected dependencies could be upgraded.
Check the changes in this PR to ensure they won't cause issues with your project.

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

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Regular Expression Denial of Service (ReDoS)

Fixed by most updated


chore(package): use shallowequal instead of fbjs and use alias in docs

Picked from #2925.

I also used shallowequal instead of fbjs, reasons are same as there facebook/react#13069.

Codecov Report

Merging #2940 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2940   +/-   ##
=======================================
  Coverage   99.78%   99.78%
=======================================
  Files         161      161
  Lines        2760     2760
=======================================
  Hits         2754     2754
  Misses          6        6

Continue to review full report at Codecov.

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

Released in<EMAIL_ADDRESS>


[DO NOT MERGE YET] matrix: pull explicit constructors

Describe problem solved by the proposed pull request
Pulling in explicit array constructors https://github.com/PX4/Matrix/pull/99 to reduce possibility for errors.
Test data / coverage
SITL flying.
Additional context
Only merge after https://github.com/PX4/Matrix/pull/99 and updating to the matrix library hash on the master branch.

Nice one. I was also wondering if making most of Matrix explicit (if we can get away with it) might help catch other implicit conversions. Some of them are relatively expensive.

Updates to matrix master since https://github.com/PX4/Matrix/pull/99 was merged.


Set proper value to -n

Too small -n makes no response, while too large value consumes memory.
Set proper value to -n and wake warning for too small value.

I tried with several -n values. In all cases but one, nothing is output to console. In only one try with -n =1 0  I could had something transcribed. First result is in a wrong language; second time it was the right transcription. I was not able to replicate it, though, i.e. I normally do not get any transcription.
[2022-09-23 20:29:26,345] transcriber._deal_timestamp DEBUG -> Length of consecutive: 0
0.00->2.00	無限に
[2022-09-23 20:29:26,347] transcriber._deal_timestamp DEBUG -> Length of buffer: 0
[2022-09-23 20:29:26,347] transcriber.transcribe DEBUG -> Last rest_start=None
[2022-09-23 20:29:26,349] cli.transcribe_from_mic DEBUG -> Segment: 1
[2022-09-23 20:29:26,353] transcriber.transcribe DEBUG -> seek=0, timestamp=2.0, rest_start=None
[2022-09-23 20:29:32,840] transcriber.transcribe DEBUG -> Result: temperature=0.00, no_speech_prob=0.24, avg_logprob=-0.80
[2022-09-23 20:29:32,840] transcriber._deal_timestamp DEBUG -> Length of consecutive: 0
2.00->4.00	 It is okay.
[2022-09-23 20:29:32,840] transcriber._deal_timestamp DEBUG -> Length of buffer: 0
[2022-09-23 20:29:32,840] transcriber.transcribe DEBUG -> Last rest_start=None
[2022-09-23 20:29:32,843] cli.transcribe_from_mic DEBUG -> Segment: 2
[2022-09-23 20:29:32,846] transcriber.transcribe DEBUG -> seek=0, timestamp=4.0, rest_start=None

@fantinuoli Did you set proper value to  --language?
if the language is English, you need to set --language en like this.
poetry run whisper_streaming --language en --model base -n 20

I added the instruction about that in README. (e9e286d)

I also found a bug about --lanauge!
I fixed it at 9cd80ab.

pad_or_trim return torch.Size([1, 80, 3000]).
While speaking, padding is not expected.
When -n 160,   torch.Size([1, 80, 3000]).
So, 160 or larger is expected.


JSON format error

Hi
I got the following error: '$..[?(@.status<3)]' is not a valid JSONPath expression. (Parse error at 1:4 near token ? (?))
When im trying to filter with JSON
I've testet the format at https://jsonpath.com/
As you can see here, its fine by the evaluator, but when i put the string in changedetector, then i got the error?

@djsmix assume i cannot read  your mind, and I am not infront of your computer sitting next to you, what version are you running?

@djsmix a 'thank you' or a 'please'... you know, we're all human here too

maybe learn how to be more helpful and kind
why dont you paste the kind of thing you are trying to match, paste an example of the JSON that actually works

why so mad? i ask you kindly what you need, i mean i provide what i think you need.
The json data or?

im so sorry if you got hurt. that was not my meaning.
i just wanted some help :/
[
{
"id": "DKCLEE114991",
"status": 3,
"price": null,
"free": false,
"link": null
},
{
"id": "DKCLEE114992",
"status": 2,
"price": null,
"free": false,
"link": null
},
{
"id": "DKCLEE115081",
"status": 3,
"price": null,
"free": false,
"link": null
},
{
"id": "DKCLEE115082",
"status": 3,
"price": null,
"free": false,
"link": null
},
{
"id": "DKCLEE115131",
"status": 3,
"price": null,
"free": false,
"link": null
},
{
"id": "DKCLEE115132",
"status": 3,
"price": null,
"free": false,
"link": null
}
]

try 'json:$[?(@.status==3)]'  does it work?

i replaced == with < and that worked.
i appreciate your help, thank you and sorry.


Added placeholder guidance to style guide

Fixes https://cockroachlabs.atlassian.net/browse/DOC-1920.

This change is

@rmloveland NP! TFTR!


Added the Hyperlink-Induced Topic Search (HITS) algorithm in GraphX

@rfurman Thanks for implementing HITS! Some high-level comments:

Please check the Spark code style guide:
https://cwiki.apache.org/confluence/display/SPARK/Spark+Code+Style+Guide
Some code doesn't follow the guide closely, e.g., spacing and parentheses.

There are some unnecessary caching and materialization code. Could you run some experiments on a medium dataset and see which RDDs should be cached?

Should we support warm start?

Please also find some minor comments inline.

Thanks @mengxr, I have fixed up the style (except maybe for long method declarations) and added in warm start functionality with three options: ColdStart (default), WarmStartFromHubData, WarmStartFromAuthData


fix: segment config schema visibility typo

Resolves: https://github.com/janus-idp/backstage-plugins/issues/959

cc @invincibleJai


[Feature] delete不支持where条件中in使用子查询

Search before asking

[X] I had searched in the issues and found no similar issues.

Description
delete不支持where条件中in使用子查询，如下sql：
delete
from ipo_appr_detail_echart
where id in (select id
             from (SELECT t.id
                   FROM ipo_appr_detail_echart t
                            left JOIN (SELECT iad.appr_date as apprDate
                                       FROM ipo_appr_detail iad
                                       WHERE iad.statistic_type IS NOT NULL
                                         AND iad.statistic_type != '18'
                                         AND iad.statistic_type != ''
                                         AND iad.case_type = '1'
                                         AND (
                                                   date_format(iad.appr_date, '%w') = '4'
                                               OR iad.appr_date = date_sub(curdate(), interval 1 day))
                                         AND iad.appr_date <= date_sub(curdate(), interval 1 day)
                                         and iad.appr_date >= '2017-01-01'
                                       GROUP BY iad.appr_date) t1 on t.appr_date = t1.apprDate
                   WHERE t1.apprDate is null) temp);

希望能支持这种sql写法
Use case
No response
Related issues
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

delete语句的where条件必须是一个常量值，不能是计算出来的值，例如：
delete from db.table where date >= date_sub(CURRENT_DATE(), interval 7 day);
这样的语句也不行，报错：
SQL 错误 [1105] [HY000]: errCode = 2, detailMessage = Right expr of binary predicate should be value。
建议尽快支持这类语法，实时数仓中这类语法场景相当多。


Unable to specify allowed components for Segmentation Item component

Not sure how relevant this defect is if we are planning to get rid of the Segmentation Item component.
But, essentially Segmentation Item does not have a design dialog and does not allow us to specify which components can be added inside the Segmentation Item component.

Issue was due to an error while installing the package. It's resolved now.


Literature search

The CMS API and the search functionality should expose the literature information from the Mendeley tracking.

[ ] Create content type for the Literature in Drupal
[ ] Modify Mendeley crawl to populate the Drupal content
[ ] User interface supports faceted search

Just to note, further: the paper is in our Mendeley library (GBIF_discussed, GBIF_author, but not GBIF_used). I know it will be a while before we have the literature in as actual drupal objects, but I would hope that we could link to it here as a bit of recommended/related reading.

[ ] Mendeley tags have to be imported/merged with core Drupal taxonomy, in order to support faceted search.


🔧 Fix: Excessive Line Spacing in Markdown-rendered User Messages

Summary
This PR addresses an issue where excessive line spacing was added　when rendering user messages in Markdown format, resulting in an unintended appearance.
Before:

After fix:
User message rendered in markdown

User message rendered as plain text

Details
Currently, Markdown-rendered user messages have a whitespace-pre-wrap class applied to their wrapper div. This class introduces extra line spacing in markdown text.
This fix removes the whitespace-pre-wrap class from the wrapper div for markdown-rendered user messages, ensuring the line spacing is consistent and appropriate.
Please let me know if this solution isn't suitable.
Change Type

[x] Bug fix (non-breaking change which fixes an issue)

Testing
I have manually verified the following:

Markdown-rendered user messages display with correct line spacing
Markdown formatting functions properly
Non-markdown user messages maintain appropriate line breaks

Checklist

[x] My code adheres to this project's style guidelines
[x] I have performed a self-review of my own code
[x] My changes do not introduce new warnings
[x] Local unit tests pass with my changes

Yes, I can confirm that this is a real problem right now.


insert explicit casts to calm compiler warnings for -Wsign-conversion

During compilation of the latest source, I received several warnings for -WSign-conversion. Here is a patch inserting explicit casts to calm them.

Sigh...  This is all because the newest GCC defaults to unsigned enums... Will consider for 2.3.x.

I see. How about add "-Wno-sign-conversion" to CFLAGS as quick workaround?

@mpsuzuki The problem with -Wno-sign-conversion is that it also eliminates valid issues, like between size_t and ssize_t, that we want to see reported.

Merged mostly as-is...
[master 8ef4f507d] Eliminate new GCC compiler warnings (Issue #5591)


minor graphical issue when extract assets.

When extracting assets with modlunky, there seems to be a problem with the sprites being slightly deformed.
Translucent parts(especially glowing effect) on a extracted image turns black.
This can be clearly seen at Arrow of light, jellyfish, orb, etc.

Interesting. Is this observed in the png itself or after loading extracted assets back into the game? Would be good to isolate if this is from the DDS to png conversion or if it's happen on Playlunky when being converted back for loading into the game.

Yes, and this seems to be appeared both in DDS to png conversion and Playlunky.

Thoughts from Discord:

DDS files in the game are premultiplied alpha, but extraction does not un-premultiply when making PNGs (premultiplied PNG is not even a thing but whatever)
Multiplying/Dividing the alpha is lossy and we can't really get an original PNG back that could be both loaded in PL and look right in an editor
Playlunky is premultiplying all images from mods, cause they are likely PNG and not premultiplied
Playlunky is also premultiplying the cached vanilla files it extracted when stitching entity sheets or sprite maps, which is bad and any unmodified parts get premultiplied twice in the end
Using any mods that custom sprite map to items.png will break all the other items because of 4
There might even be a feedback loop so files in the cache get premultiplied again and again

Solutions:

Modlunky could predivide alphas when extracting, but it's not going to look the same when premultiplying again

We could just say the extracted PNG files are premultiplied and anyone who uses very transparent parts as base for their mods is stupid

Playlunky definitely shouldn't divide or premultiply cached (vanilla) files ever, only premultiply user-provided files once before caching or stitching

I extracted the divided textures with this modification but nothing looks quite right either way...
def dds_to_png(data):
    """Takes a .DDS `Image` and returns .png data."""
    img = Image.open(io.BytesIO(data))
    px = img.load()
    width, height = img.size
    for i in range(width):
        for j in range(height):
            if px[i, j][3] != 0:
                R = int(255.0 * px[i, j][0] / px[i, j][3] + 0.5)
                G = int(255.0 * px[i, j][1] / px[i, j][3] + 0.5)
                B = int(255.0 * px[i, j][2] / px[i, j][3] + 0.5)
                a = px[i, j][3]
                px[i, j] = (R, G, B, a)
    new_data = io.BytesIO()
    img.save(new_data, format="PNG")
    return new_data.getvalue()


ACE 7.0.0

* recipe/meta.yaml:

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

Cool! Do you think that ACE 6 will continue to have releases? In that case, we may want to create a v6 branch out of the current master (before this PR), and add:
bot:
  abi_migration_branches:
    - "v6"

to conda-forge.yaml file, to ensure that the v6 branch still receives migration see for example  https://github.com/conda-forge/libignition-transport4-feedstock/pull/23/files .

Cool! Do you think that ACE 6 will continue to have releases? In that case, we may want to create a v6 branch out of the current master (before this PR), and add:
bot:
  abi_migration_branches:
    - "v6"

to conda-forge.yaml file, to ensure that the v6 branch still receives migration see for example  https://github.com/conda-forge/libignition-transport4-feedstock/pull/23/files .

Yes, it could be that ACE v6 has new releases, can you make a PR for the proposed change, pretty busy today with getting ACE 7 out

Yes, it could be that ACE v6 has new releases, can you make a PR for the proposed change, pretty busy today with getting ACE 7 out

Yes, it could be that ACE v6 has new releases, can you make a PR for the proposed change, pretty busy today with getting ACE 7 out

Done. I created the v6 branch and added the necessary config in https://github.com/jwillemsen/ace-feedstock/commit/1c3fadf8a30f6ab2af5255e712139609772b594c .

Yes, it could be that ACE v6 has new releases, can you make a PR for the proposed change, pretty busy today with getting ACE 7 out

Done. I created the v6 branch and added the necessary config in https://github.com/jwillemsen/ace-feedstock/commit/1c3fadf8a30f6ab2af5255e712139609772b594c .


"Cannot read property 'android' of undefined"

Calling js method onCreateView failed
TypeError: "Cannot read property 'android' of undefined"
File: "/data/data/org.nativescript.AppName.files.app.tns_modules/nativescript-google-maps-sdk/map-view.js". line: 32, colomn: 39

Make sure you have the Google Play services SDK install, and try reinstalling...
rm -rf node_modules platforms
tns install

Thanks. It halped me =)

@dapriett  how to install the  Google Play services SDK for nativescript??


feat : Prevstats section added

Not responsive at all @anmol5936 . Please complete by tonight

Also the length of individual stats should be according to design

Change the website title to HackNITR 5.0 @anmol5936

Fix this overflow @anmol5936

The cards should be left aligned and should move in one direction like it is in the design. Follow the design properly and keep animation exactly the same @anmol5936
https://www.figma.com/proto/xwjkgivbXD5BKUwJwT4JIy/HN-5.0-website-real?type=design&node-id=255-1275&t=jpQdYxTCNbIGDKZc-0&scaling=min-zoom&page-id=55%3A710&starting-point-node-id=255%3A1275

Horizontally center the component.


Update PostgreSQL JDBC driver to 42.2.18

Hello @anghelutar and @tgeens,
The Alfresco images are still including an old version of the PostgreSQL JDBC driver (v9.4.1212 released on 2016-11-02; latest v9.4 version).
I believe the Alfresco image, should at least include as minimal version the PostgreSQL JDBC driver version mentioned in the Supported Platform for Alfresco Content Services documentation.
Minimal PostgreSQL JDB driver versions:

Alfresco 6.2: 42.2.1, 42.2.6, 42.2.14
Alfresco 6.1: 42.2.1, 42.2.6
Alfresco 6.0: 42.2.1
Alfresco 5.2: 9.4.1211, 42.2.12
Alfresco 5.1: 9.4.1201
Alfresco 5.0: 9.3.1102
Alfresco 4.2: 9.0.802

I took the liberty to upgrade the PostgreSQL JDBC driver for all Alfresco versions, since the current version (42.2.18) supports PostgreSQL 8.2 or newer and Java 6 or newer (see documentation).
Kind regards,
Koen

this PR is obsolete now we switched to tomcat-embedded. We have updated the postgresql jdbc driver there.


Adaptive line scroll

Hi, first of all I'd like to thank you for creating and sharing this app 😊 This  "feature" in sierra has been driving me crazy for the last two months.
I was wondering if you could add an option to only scroll one line when the scroll wheel ticks once, but scroll multiple lines if the scroll wheel ticks multiple times in for example a 0.2 second time window.
I'd like this feature because it would help when viewing a pdf with Quick Look. If the scroll is set to three lines then on a scroll wheel tick the document would scroll down 3 pages, when in reality I only want to scroll down one page. But if I use the release where you set the scroll to 1 line, then thats a little bit inconvenient when scrolling in any other app on a daily basis.
Thank you for your help in advance!
Regards,
Kmikiy

10.12.2 fixes scrolling behavior for me, but this did the trick for me in 10.12.1:
diff --git a/DiscreteScroll/main.m b/DiscreteScroll/main.m
index bddf9a4..36c8fe8 100644
--- a/DiscreteScroll/main.m
+++ b/DiscreteScroll/main.m
@@ -7,9 +7,12 @@ CGEventRef cgEventCallback(CGEventTapProxy proxy, CGEventType type,
                            CGEventRef event, void *refcon)
 {
     if (!CGEventGetIntegerValueField(event, kCGScrollWheelEventIsContinuous)) {
-        int64_t delta = CGEventGetIntegerValueField(event, kCGScrollWheelEventPointDeltaAxis1);
-
-        CGEventSetIntegerValueField(event, kCGScrollWheelEventDeltaAxis1, SIGN(delta) * LINES);
+        if (CGEventGetIntegerValueField(event, kCGScrollWheelEventDeltaAxis1) == 0) {
+            int64_t delta = CGEventGetIntegerValueField(event, kCGScrollWheelEventPointDeltaAxis1);
+            if (delta != 0) {
+                CGEventSetIntegerValueField(event, kCGScrollWheelEventDeltaAxis1, SIGN(delta) * LINES);
+            }
+        }
     }

     return event;

Thanks @thomasst 10.12.2 solved it for me too :)


Scaling Canvas to Image Resolution

currently it is necessary to specify the canvas bounds up front: Canvas::fromIntBounds(0, 100, 0, 200), but this causes issues if we want the canvas to be, for example, the exact size of the image that is rendered upon it.
in the slideshow branch I added the $imageShape->resolution() accessor, and did Canvas::fromDimensions($imageShape->resolution()) but this no longer works since splitting the painters from the shapes. It may be I can just revert the Image shape change so that the Imagick object is held in the shape rather than in the renderer.

maybe this is beyond the scope of the library


Training code

I’m wondering if you have any plans to release the code you used to train the model? Thanks

Thanks for the interest! Yes, there is! Coming soon.

Great, thanks!


Integration 2 step verification trouble

After installing with HACS and attempting integrate it shows a list of devises/ama to send verification code. I tuo sms or push and the message never comes to finish the integration. What am I doing wrong?

Same issue - I pass the captcha, get a list of OTP methods, I select email, no emails are received.
Logging in via Chrome using incognito mode immediately sends a OTP (which doesn't work with alexa_media_player).

Thanks.  This is a duplicate of #568 which lists workarounds.  We are still trying to get logs and other info to actually solve the problem, so please provide them there if you don't do the workaround.

Than

Thanks. This is a duplicate of #568 which lists workarounds. We are still trying to get logs and other info to actually solve the problem, so please provide them there if you don't do the workaround.

Thank you


commons-codec dependency managed to "runtime" is a problem

When a downstream project imports the RDF4J management, it also gets the dependency management for commons-codec bound as a "runtime" dependency from org.eclipse.rdf4j:rdf4j:pom:3.5.1
			<dependency>
				<groupId>commons-codec</groupId>
				<artifactId>commons-codec</artifactId>
				<version>1.13</version>
				<scope>runtime</scope>
			</dependency>

This implicitly turns all the dependencies that the downstream project may have on commons-codec from compile scope (default) to the runtime scope and breaks compilation.
In my experience, it is generally a good idea not to specify scopes in the dependency management section at all and use it only in the dependencies section.

Why would a downstream project ever import our root pom directly?

It's easy enough to fix btw, I'll put up a PR in a sec. But I'd appreciate it if you could give me some background on when this is a problem.

Well, yes - I was as surprised as you are. I didn't import the root POM - just the BOM. But still Maven picks up the managed dependency from the root POM. Here is an example:
<project xmlns="http://maven.apache.org/POM/4.0.0"
  xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
  xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 https://maven.apache.org/xsd/maven-4.0.0.xsd">
  <modelVersion>4.0.0</modelVersion>
  <groupId>test</groupId>
  <artifactId>rdf4j-dependeny-import-test</artifactId>
  <version>0.0.1-SNAPSHOT</version>
  <dependencyManagement>
    <dependencies>
      <dependency>
        <groupId>org.eclipse.rdf4j</groupId>
        <artifactId>rdf4j-bom</artifactId>
        <version>3.5.1</version>
        <type>pom</type>
        <scope>import</scope>
      </dependency>
    </dependencies>
  </dependencyManagement>
  <dependencies>
    <dependency>
      <groupId>commons-codec</groupId>
      <artifactId>commons-codec</artifactId>
    </dependency>
  </dependencies>
</project>

Result:

I guess one could also take the perspective: the actual issue is that importing the BOM does not only import RDF4J versions but also versions of third-party libraries.

I guess one could also take the perspective: the actual issue is that importing the BOM does not only import RDF4J versions but also versions of third-party libraries.

But that is a feature, not a bug :) You can of course always override this if you want. The idea is that you don't have to figure out for yourself what versions of third party libraries are compatible with rdf4j.

True - but then we're back at: we want to have the managed versions, but managing the scope does more harm than good.

Yeah fair enough - though there too you can of course override locally, but I'll grant you that it is less obvious. PR is up.

By the way you mentioned commons-codec here specifically, but presumably the principle applies for any dependencies managed in the root pom.

Well yeah probably so. But unless they override the scope they do not interfere with the rest of my project setup. In JAR module POMs, I declare compile dependencies plain (i.e. without scope or version) fully relying on dependency management. That works fine and largely as expected. But if the imported dependency management starts changing the default scope of dependencies from compile to something else (e.g. to runtime), then the compilation breaks because the classes of that dependencies can no longer be found during the compile stage.


custom function wasn't supported via HTTP

EdgeDB Version: V2.9
EdgeDB CLI Version:
OS Version: WSL in widows 10

Steps to Reproduce:

run the query select SellingOrder {computedPurchaseDateTime } via repl will return the correct result.
run the query select SellingOrder {computedPurchaseDateTime } via http by insomnia will result null.

Schema:
type SellingOrder {
		property purchaseDateTime -> datetime;
		property computedPurchaseDateTime := get_fdow(.purchaseDateTime);
	}

	function get_fdow(dt: datetime) -> str # get the first day of specified week
        	using (
            	with pur_date := cal::to_local_datetime(dt,"America/Los_Angeles"),
            	weeks := datetime_get(pur_date, "isodow"),
            	first_date_of_week := pur_date - <duration>(<str>((weeks-1)*24*60*60*1000) ++ " milliseconds"),
            	week := str_split(<str>first_date_of_week, "T")[0],
            	select week
        );

Confirmed this doesn't seem to be related to the computed property. Even just calling the function directly in a query (like select get_fdow(datetime_current());) doesn't work over HTTP even though it works fine in the REPL. That returns just an object with a data key containing an empty array.


Remove ability for Post component to load from URI (close #1302)

The <Post> component had old logic for passing in a uri to load from as an alternative to using the supplied view. This was a pretty poorly thought-out piece of code (by me) to begin with, but I made it worse when I modified it in #1243. Since it's not actually used now, it's good to remove.
I didn't create a repro to confirm this caused #1302 and I can't quite find the logical pathway for for non-termination. I assume the issue arises if initView is ever undefined or if uri ever changes.

Wonder if this will help #1304. I just tested this using my test accounts and it appeared to do so 🤔

Wonder if this will help https://github.com/bluesky-social/social-app/issues/1304. I just tested this using my test accounts and it appeared to do so 🤔

It shouldnt. The problem in #1304 is that we go to the notifications tab but we dont automatically refresh the view.


How to pass attribute to a controller converter?

As of here: http://silex.sensiolabs.org/doc/master/usage.html#route-variable-converters
I want to define the converter as service to use Doctrine ORM.
If I use:
$app['converter.user'] = function ($user, Request $request) use ($em){
     print($request->attributes->get('user'));
     return new UserConverter($em);
};

$app->get('/user/{user}', function (Silex\Application $app, Request $request, $user) {
      return $app->json($user);
})
->convert('user', 'converter.user:convert');

The 'user' attribute is not available in service and I get the error:
Fatal error: Call to a member function get() on null
But if instead of a service I use a variable like below then the attribute 'user' works fine! what wrong I did above? I need to pass the value of 'user' attribute to pass to convert() method of UserConverter() class, how can I do so from the service above?
$user = function ($user, Request $request) use ($em){
        print($request->attributes->get('user'));
        return new UserConverter($em);
};
$app->get('/user/{user}', function (Silex\Application $app, Request $request, $user) {
      return $app->json($user);
})
->convert('user', $user);

Is here any support or is this repository still maintained??

Hi @nimasdj, this place is not for support, Github issues are for reporting bug in the software. Someone may answer you here but for support you should try stackoverflow instead
for this question, you could do it like that:
$app['em'] = $em;
$app['converter.user'] = function (Silex\Application $app) {
     return new UserConverter($app['em']);
};
$app->get('/user/{user}', function (Silex\Application $app, $user) {
      return $app->json($user);
})
->convert('user', 'converter.user:convert');

Thanks @mathroc, sorry for confusion. $em was working fine with my code, I meant to ask how to use
$request->attributes->get('user') inside my UserConverter as it has no value there and how can I pass its value from controller to the class?
So should I use:
$app['user'] = $request->attributes->get('user');
then pass it to the class constructor? Or is there any better way?

you should not use $request inside your converter, in fact, you don't need it at all in your exemple.
$app['em'] = $em;
$app['converter.user'] = function (Silex\Application $app) { // this function returns a converter
     return new UserConverter($app['em']);
};
$app->get('/user/{user}', function (Silex\Application $app, User $user) {
      return $app->json($user);
})
->convert('user', 'converter.user:convert');

silex will get the converter.user service and call the convert method on it with the value of {user} as its first argument, the value it returns will then be passed as $user to the controller

@mathroc Do you mean the value of {user} attribute is available as $user inside my convert() mthod? What about request (_POST) values? They will be available with the same posted variable name?

@mathroc Thanks for the advice. It is working fine now, but what if this is a post method? how to pass posted data to convert method()?
$app->get('/user/{user}', function....` this way {user} attribute is the first argument of method but if we have $app->post('/user/add', function...``` with some posted data how to pass them to convert method()?

you might be able to do it with a converter but I wouldn't recommend it, do it inside the controller instead

@mathroc Thanks for the advice. so something like
$vars  = $request->request->all();
inside controller then pass $vars to method the same way like passing $em?
Or even this way is not recommended and better to avoid service and do it with simple function like I described in my first post?

Closing as indeed, stackoverflow should be used for support.


Can I use a USB to serial cable to send traffic/wx data to a GRT Avionics HxR EFIS display. The display has serial ports that can be assigned to "ADSB."  I have the latest Stratux kit on a Pi 3.  WIFI works fine with my Foreflight app, but looking to get info on EFIS display.  Thanks, Mike.

Stratux version:
Stratux config: single/dual SDR, GPS yes/no (type?), AHRS yes/no, battery, battery cable.
EFB app and version (e.g., WingX Pro7 8.6.2). EFB platform (e.g., iOS 9.2). EFB hardware (e.g., iPad Mini 2).
Description of your issue. If possible, enable "Replay Logs", reproduce the problem, and provide a copy of the logs in http://<IP_ADDRESS>/logs/stratux/ and http://<IP_ADDRESS>/logs/stratux.log.

@cowboy737 - http://www.grtavionics.com/StratuxSupplement.pdf


Vite automatically tries to disable the next available port configuration.

Describe the bug

The first bug
First of all, I started a V3 project with the port number of 3000 using parcel2. Then, I configured the port number to be 3000 according to the vite2 document, and I also configured strictport to be true. However, after I started the vite project, the port was 3000, and it was not detected that the 3000 port number was occupied by another project.
The second bug
In addition, let me turn on the<EMAIL_ADDRESS>And then I started again<EMAIL_ADDRESS>Project. There is also a problem that no port reuse is detected.
Reproduction

Corresponding to the first bug
package.json
 "dependencies": {
    "vue": "^3.0.5"
  },
  "devDependencies": {
    "@vitejs/plugin-vue": "^1.2.2",
    "@vue/compiler-sfc": "^3.0.5",
    "vite": "^2.3.0"
  }

vite.config.js
import { defineConfig } from 'vite';
import vue from '@vitejs/plugin-vue';

// https://vitejs.dev/config/
export default defineConfig({
	plugins: [vue()],
	server: {
		port: 3000,
		strictPort: true,
	},
});

I put the picture on my website, you can click the link below.
Bug map
Corresponding to the second bug
I put the picture on my website, you can click the link below.
Bug map

Before submitting the issue, please make sure you do the following

[ ] Read the Contributing Guidelines.
[x] Read the docs.
[ ] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[ ] Provide a description in this issue that describes the bug.
[ ] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/vue-next instead.
[ ] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.

@yyx990803

@yyx990803 Please take a look at this problem。

@yyx990803 Please take a look at this problem.

Looks like parcel is listening to all address (as --host ::). Vite 2.3 switched the default to listen only to <IP_ADDRESS>, so there isn't an error when listening to it in the same port.
If you use --host or --host :: in vite, you will get the same behavior as with 2.2 and it will fail as you expect. You could also set parcel to listen to --host <IP_ADDRESS> and also get a hard error.
I don't know if there is something that should be fixed in Vite regarding this. Other tools like sirv-cli also work in the same way, and will not recognize that 3000 is being used in this case.

related to the port-reuse issue of node.
@maomincoding you can specified server.host to '::' as a temporay workaround.
Since 1e604d5b60900098f201f90394445fea55642e74, httpServer  will listen to a specified default port caused this issue.

@patak-js Thank you.

@maomincoding For the future, please do not ping Evan directly.
We have an active team around Vite that is working / triaging issues and PRs every day and we decide what is needed to reach Evan so he can manage higher order stuff in the whole Vue ecosystem.

@Shinigami92 Sorry, it won't happen again.

vite: ^2.3.7  has the same error
// vite.config.ts

when i set server.host to ::


fix/node_modules cleaning script

I need to add --glob flag.
But after all, error occur because of the lack of node_modules when rimraf is removed by itself xD.
Rimraf suggest to install it globally, i have done postinstall script, which install it automatically with 'pnpm i' :)

@Jonghakseo Maybe add 'clean:node_modules' to 'clean' script ?

@Jonghakseo Give me response to the comment above and after this we can merge it 😄

#620


[#noissue] cleanup test

change deprecated class
use RanomUtils class instead of Random class
remove some Thread.sleep

Codecov Report

Merging #3120 into master will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #3120      +/-   ##
=========================================
- Coverage   38.85%   38.8%   -0.05%
=========================================
  Files        2003    2003
  Lines       66087   66087
  Branches     9224    9224
=========================================
- Hits        25677   25648      -29
- Misses      37824   37856      +32
+ Partials     2586    2583       -3

Impacted Files
Coverage Δ

.../java/com/navercorp/pinpoint/rpc/cluster/Role.java
0% <0%> (-42.86%)
:arrow_down:

...npoint/rpc/server/SimpleServerMessageListener.java
80% <0%> (-20%)
:arrow_down:

...rp/pinpoint/rpc/control/ControlMessageDecoder.java
64.28% <0%> (-14.29%)
:arrow_down:

.../com/navercorp/pinpoint/rpc/packet/PongPacket.java
50% <0%> (-14.29%)
:arrow_down:

...om/navercorp/pinpoint/rpc/codec/PacketDecoder.java
68.67% <0%> (-7.23%)
:arrow_down:

.../pinpoint/rpc/client/PinpointClientHandshaker.java
61.11% <0%> (-6.25%)
:arrow_down:

.../profiler/sender/StandbySpanStreamDataStorage.java
72.88% <0%> (-3.39%)
:arrow_down:

...ofiler/sender/StandbySpanStreamDataSendWorker.java
80.59% <0%> (-2.99%)
:arrow_down:

...orp/pinpoint/rpc/server/DefaultPinpointServer.java
76.13% <0%> (+0.75%)
:arrow_up:

...m/navercorp/pinpoint/test/OrderedSpanRecorder.java
74.03% <0%> (+1.92%)
:arrow_up:

... and 1 more

Continue to review full report at Codecov.

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


Pass NULL-terminated buffer to pthread_chdir_np

Works for me, though shouldn’t be necessary in C++11.  data() and c_str() are aliases.

Ah, but I suppose it is possible our StringRef here may actually point to a non-std::string object.

Yes, StringRef isn’t guaranteed to be null terminated.


LMB+RMB in Spinbox during mouse capture refuses to release capture

Godot version:

3.0.2, master  (At least 3.0.3 rc1)
OS/device including version:

Windows 10
Issue description:

When using a Spinbox in the runtime, during a dragging event which activates mouse capture (invisible cursor etc), if one accidentally presses the right mouse button during the drag, the cursor will be stuck in a captured and invisible state while the window is in focus.
This likely affects other controls which have similar behavior; though I haven't been able to reproduce it reliably with ColorPicker, this can wreck havoc in the editor when altering an int or float value with constrained bounds in the property inspector, requiring a restart of the editor.

Duplicate of https://github.com/godotengine/godot/issues/17891


 I had quarkus in there because I was obviously fiddling and trying around.
Btw @radcortez it would be worth mentioning the .** pattern in the docs, I found it in the sources.


feat: Add dark_mode option in settings

Just learned that dark_mode exists, would like to have an easier settings option for users.

My only minor concern about this is that if people already have imgui.dark_mode = true elsewhere in a different settings file, this will potentially clash suddenly and I guess possibly switch them out of dark mode depending on precedence. For now it's kind of an unavoidable problem, so just have to accept it I think. That said, I will wait to see if anyone else says anything.

We could leave it commented out so that it doesn't override any configs

Oh right, ya that works. Looks like that's the case for a bunch of the settings anyway.

Changed to uncomment if format


Could not clone the remix tutorial template

Reproduction
I'm new to Remix 😄 !
I started learning the tutorial and when I wanted to clone the template each attempt I faced this issue :

System Info
System:
    OS: Linux 6.8 Ubuntu 22.04.5 LTS 22.04.5 LTS (Jammy Jellyfish)
    CPU: (16) x64 13th Gen Intel(R) Core(TM) i7-13620H
    Memory: 7.70 GB / 15.35 GB
    Container: Yes
    Shell: 5.1.16 - /bin/bash
  Binaries:
    Node: 22.1.0 - ~/.nvm/versions/node/v22.1.0/bin/node
    npm: 10.7.0 - ~/.nvm/versions/node/v22.1.0/bin/npm
    pnpm: 9.3.0 - ~/.nvm/versions/node/v22.1.0/bin/pnpm
  Browsers:
    Chrome: 129.0.6668.58

Used Package Manager
npm
Expected Behavior
Setup the project with the default tutorial template by Remix.
Actual Behavior
▲  Oh no! There was a problem fetching the file from GitHub. The request responded with a 404 status. Please try again later.

Hey @alamenai, I can't replicate this, I wonder if GitHub was having a bad day at the time. Sorry you were having a bad experience either way
At this point, I do recommend you try out the React Router v7 tutorial, since that's the next version of Remix context

Thank you @brookslybrand , yes when I tried it next day, it works.


json_fwd.hpp no longer standalone

Description
json_fwd.hpp now has a line:
#include <nlohmann/detail/abi_macros.hpp>
which means this file is no longer standalone. This makes it no longer possible to have a simple 2-file installation, with just json_fwd.hpp and the large single json.hpp.
This seems to be related to #3590
Reproduction steps
json_fwd.hpp now has a new external file dependency.
Expected vs. actual results
Expected:
Be able to have json_fwd.hpp standalone with no dependencies.
Actual:
json_fwd.hpp has an include file, so it can't be used by itself anymore.
Minimal code example
any code that wants to only do:
#include "json_fwd.hpp"

Error messages
can't find include file 'nlohmann/detail/abi_macros.hpp'

Compiler and operating system
any
Library version
3.11.1
Validation

[ ] The bug also occurs if the latest version from the develop branch is used.
[ ] I can successfully compile and run the unit tests.

Just to understand your use-case better: do you have parts of your code that use json_fwd.hpp, but have no access to the other library headers?

As far as I can tell (see #700), it was never the goal of this file to make it a "standalone" file but just to make it a smaller header that handles the forward declaration of the class, for files that just want to be able to pass the objects around, but not to operate on them.
I assume that you are copying json_fwd.hpp from the include/json directory to the single_include directory.  Is that accurate?  Is it a significant burden to copy the abi_macros.hpp file as well?  Is it the nlohmann/detail directory that's the issue?  It might be possible to embed the contents of the header into json_fwd.hpp and include that instead everywhere that abi_macros.hpp is currently included.
@falbrechtskirchinger Thoughts?

I'm not sure where I read it, but I was under the impression that json_fwd.hpp was in fact meant to be standalone. So, big derp on my part.
Maybe we should just amalgamate json_fwd.hpp? I.e., add a step to make amalgamate and just output it to single_include/nlohmann/json_fwd.hpp. Then we need to remember to add that file to the release scripts (to replace the version number).

Just to understand your use-case better: do you have parts of your code that use json_fwd.hpp, but have no access to the other library headers?

Yes, my workflow is to download the package .tar.gz file, and copy the json_fwd.hpp and single_include/json.hpp files into my source tree. Now that becomes more difficult with this extra dependency.
So perhaps one way to reframe the original description would be something more like a request for an amalgamated single_include/json_fwd.hpp. This can be in addition to the newly update json_fwd.hpp that has the extra include.
I think this is along the lines of what @falbrechtskirchinger commented on to be a possibility. This would be sufficient and great. Thanks!


Default Solidity Contract

On SE-1 we have a basic set purpose contract as the default solidity contract.
Let's discuss what would be good to have as the default contract on SE-2.
I feel that we could have:

More types: address (owner?), uint (counter?), array / struct?
Payable & non payable function
Withdraw function?
Extending some openzeppelin?
..

I think having this out of the box, make the /debug page richer. It will also be a key part of #81
P.S: Right now, on the SE-2 contract we have a few pointless stuff. (the intent was to the display all the posible types / cases on the debug page to do the UI/UX correctly)

I think that maybe we can have 2 example contracts. One really simple, like the Purpose contract and another one with more features, to be able to have a more complete example page based on it. What do you think?

The reason to have two contracts is to keep the first contract simple to new users.

Here are some of my contract ideas:

v2 of Purpose contract :

The idea is - In the beginning, you have your(owner) purpose set and once the contract is deployed people can change your purpose but there will be a minimum required amount to change the purpose.
Only the owner can withdraw the funds
Some data structures : mapping(address => purpose), event(address, purpose, amount sent), Counter(noOfTime purpose changed)
don't know if it makes sense 😅

FundMe / By me a Coffee contract :

fund me example here
By me a Coffee example here
these are just some ideas and we can add/remove stuff accordingly

YourNFT contract from se-typescript :

an example is here

A simpler and more robust version of the challenge-1-staking contract :

It will be simple but we can have some more data structures like structs and all.

I agree with Damu that it shouldn't be a complex contract and must be intuitive and even a beginner should be able to grasp it in one go.
Having 2 contracts is a nice idea but maybe we can have 1 contract which is beginner-friendly and also makes /debug and Example UI richer because at the beginning it was a little bit frustrating to me in se-typescript(which had/has 2 contracts) and you need to remove deployment and UI of another contract(this is subjective to me) other than that 2 contracts is also a good option

Thanks you for this @damianmarti & @technophile-04 <3

I agree with Damu that it shouldn't be a complex contract and must be intuitive and even a beginner should be able to grasp it in one go.

+1

Having 2 contracts is a nice idea but maybe we can have 1 contract which is beginner-friendly and also makes /debug and Example UI richer because at the beginning it was a little bit frustrating to me in se-typescript(which had/has 2 contracts) and you need to remove deployment and UI of another contract(this is subjective to me) other than that 2 contracts is also a good option where we have the freedom of keeping 2nd contract more robust and a bit complex showing all the abilities of se-2

+1
So maybe we can try to find the sweet spot with 1 contract?? Keeping it simple, but having some mappings, payable, etc. I like the idea of a Purpose contract v2.
I'll say, let's try and see how it feels.

Awesome discussions here and spending time finding the right level of complexity!
My vote is leaning toward a simple, single contract with just a few of the basics.
Hopefully this repo will also be forked a lot and the forks will have more complex contracts.
Def want the two contract thing to be a very simple next step too.

Just merged a draft of a Purpose contract v2. Let's play with it and see how it feels.
Thanks all!!


Is it possible to use Template event maps with the typeahead events?

I'd love to do it this way as it feels more organized but I can't find any documentation on this approach. Thanks!

@matthewforr this feature would be useful. Now it does not work, but it should be easy to implement. Please feel free to make a PR for this.


Section Stacking and Type checks for hard.lp

I finished the hard constraints in section stacking and type checks.
Somehow I couldn't figure out why two tests are failing. Let me know what you think. Thank you!

Great. Please make sure the tests pass.

Please update after https://github.com/cmudig/draco2/pull/194 is merged

I converted this pull request to a draft. Please mark as ready when it's ready.

This pull request has too many constraints to review, I am separating this into multiple ones.


create a custom resource samba_share

The resource will be responsible for:

[ ] Creating the directory for the share
[ ] Appending the configuration to an accumulator for /etc/samba/smb.conf

Am I correct in understanding that is still to be implemented?
This is currently a bit of a pain in the neck for me, as I am using Vagrant with chef-solo, and apparently adding support for data-bags (I had none in my cookbook so far) has some consequences... for the moment I get an error 'I cannot read /etc/chef/client.pem, which you told me to use to sign requests!'

Correct, this is not implemented. I wanted to open the issue in case someone has cycles to work on this, as I don't.

As for your issue, I recommend using chef-client with local mode rather than chef-solo - particularly for cookbooks like this one that rely on data bags.


I have 12.2 Beta, but the unc0ver not recognizing it and show "Unsupported" message!

Describe the bug
I have 12.2 Beta, but the unc0ver not recognizing it and show "Unsupported" message!
The Beta build Number "16E5227a" - iOS 12.2 Beta - iPhone 9,4 - Model: D111AP
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Screenshots
If applicable, add screenshots to help explain your problem.
Device (please complete the following information):

iOS Version:
iOS Device:
unc0ver Version:

Place an "x" between the brackets if true:

[x] this is a bug others will be able to reproduce
[x] this issue is present with all tweaks uninstalled(except for default packages) or disabled
[ ] this issue is present after a rootfs restore
[x] this issue is present on the latest version of unc0ver

Logs
If applicable, add logs or error messages here.

Also can reproduce

You'll have to wait for pwn, GitHub source isn't updated, so I can't submit a pull request to fix this :/


auroracompute: Add Barcelona (Spain) region to driver

This PR only adds a new region to the Aurora Compute driver, it does not change functionality of it.
Barcelona is the soon to open new region of Aurora Compute and we would like to have it supported in libcloud prior to launch which is ~2 weeks away

Merged, thanks.


readPWMFreqAndValue() new kernel uses new names

for Kernel 4.1 (4.1.25-ti-r62 #1 SMP Tue May 24 21:23:52 UTC 2016 armv7l GNU/Linux)
modify lines 25 & 26 of hw_mainline.js:
//var period = fs.readFileSync(pwmPrefix[pin.pwm.name]+'/period_ns');
var period = fs.readFileSync(pwmPrefix[pin.pwm.name]+'/period');
//var duty = fs.readFileSync(pwmPrefix[pin.pwm.name]+'/duty_ns');
var duty = fs.readFileSync(pwmPrefix[pin.pwm.name]+'/duty_cycle');

this fixes the empty PWM freq/duty object from getPinMode() on PWM enabled pins.

We are likely going to skip kernel 4.1 and go straight to supporting 4.4. See the 0.5.x branch.

Looking at the Readme.md in https://github.com/beagleboard/linux, it looks like 4.1 has fallen off the list, with 4.4, 4.9 & 4.14 remaining. Given the effort required to support so many different versions, I propose to close this PR.


Mediakit 1.1.2: Cannot build on MacOS - ld: framework not found Mpv

I tried updating media_kit to the newest version 1.1.2 and am now hitting this error while building on MacOS on M1 pro.
Connected devices:
macOS (desktop) • macos • darwin-arm64 • macOS 13.4.1 22F770820d darwin-arm64

Checking for wireless devices...

[1]: macOS (macos)
Please choose one (or "q" to quit): 1
Launching lib/main.dart on macOS in release mode...
2023-08-06 23:10:54.816 xcodebuild[2805:6788141] DVTCoreDeviceEnabledState: DVTCoreDeviceEnabledState_Disabled set via user default (DVTEnableCoreDevice=disabled)
--- xcodebuild: WARNING: Using the first of multiple matching destinations:
{ platform:macOS, arch:arm64, id:00006000-000265022223401E }
{ platform:macOS, arch:x86_64, id:00006000-000265022223401E }
ld: warning: directory not found for option '-F/Users/dario/dev/DaKanji/DaKanjiApplication/macos/Pods/../Flutter/ephemeral/.symlinks/plugins/media_kit_libs_macos_video/macos/Frameworks/.symlinks/mpv/macos'
ld: framework not found Mpv
clang: error: linker command failed with exit code 1 (use -v to see invocation)
ld: warning: directory not found for option '-F/Users/dario/dev/DaKanji/DaKanjiApplication/macos/Pods/../Flutter/ephemeral/.symlinks/plugins/media_kit_libs_macos_video/macos/Frameworks/.symlinks/mpv/macos'
ld: framework not found Mpv
clang: error: linker command failed with exit code 1 (use -v to see invocation)
warning: Run script build phase 'Run Script' will be run during every build because it does not specify any outputs. To address this warning, either add output dependencies to the script phase, or configure it to run in every build by unchecking "Based on dependency analysis" in the script phase. (in target 'Flutter Assemble' from project 'Runner')
warning: Run script build phase 'Run Script' will be run during every build because it does not specify any outputs. To address this warning, either add output dependencies to the script phase, or configure it to run in every build by unchecking "Based on dependency analysis" in the script phase. (in target 'Runner' from project 'Runner')
** BUILD FAILED **

Building macOS application...
Exception: Build process failed
#0      throwToolExit (package:flutter_tools/src/base/common.dart:10:3)
#1      buildMacOS (package:flutter_tools/src/macos/build_macos.dart:141:5)
<asynchronous suspension>
#2      MacOSDevice.buildForDevice (package:flutter_tools/src/macos/macos_device.dart:72:5)
<asynchronous suspension>
#3      DesktopDevice.startApp (package:flutter_tools/src/desktop_device.dart:119:7)
<asynchronous suspension>
#4      FlutterDevice.runCold (package:flutter_tools/src/resident_runner.dart:524:33)
<asynchronous suspension>
#5      ColdRunner.run (package:flutter_tools/src/run_cold.dart:57:28)
<asynchronous suspension>
#6      RunCommand.runCommand (package:flutter_tools/src/commands/run.dart:750:27)
<asynchronous suspension>
#7      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1336:27)
<asynchronous suspension>
#8      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#9      CommandRunner.runCommand (package:args/command_runner.dart:212:13)
<asynchronous suspension>
#10     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:296:9)
<asynchronous suspension>
#11     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#12     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:244:5)
<asynchronous suspension>
#13     run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:91:9)
<asynchronous suspension>
#14     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#15     main (package:flutter_tools/executable.dart:91:3)
<asynchronous suspension>

1.1.0 throws this:
macOS (desktop) • macos • darwin-arm64 • macOS 13.4.1 22F770820d darwin-arm64

No wireless devices were found.

[1]: macOS (macos)
Please choose one (or "q" to quit): 1
Launching lib/main.dart on macOS in debug mode...
Running pod install...                                           2,907ms
2023-08-06 23:02:49.034 xcodebuild[175:6773776] DVTCoreDeviceEnabledState: DVTCoreDeviceEnabledState_Disabled set via user default (DVTEnableCoreDevice=disabled)
--- xcodebuild: WARNING: Using the first of multiple matching destinations:
{ platform:macOS, arch:arm64, id:00006000-000265022223401E }
{ platform:macOS, arch:x86_64, id:00006000-000265022223401E }
ld: warning: directory not found for option '-F/Users/dario/dev/DaKanji/DaKanjiApplication/macos/Pods/../Flutter/ephemeral/.symlinks/plugins/media_kit_libs_macos_video/macos/Frameworks/.symlinks/mpv/macos'
ld: framework not found Mpv
clang: error: linker command failed with exit code 1 (use -v to see invocation)
warning: Run script build phase 'Run Script' will be run during every build because it does not specify any outputs. To address this warning, either add output dependencies to the script phase, or configure it to run in every build by unchecking "Based on dependency analysis" in the script phase. (in target 'Runner' from project 'Runner')
warning: Run script build phase 'Run Script' will be run during every build because it does not specify any outputs. To address this warning, either add output dependencies to the script phase, or configure it to run in every build by unchecking "Based on dependency analysis" in the script phase. (in target 'Flutter Assemble' from project 'Runner')
** BUILD FAILED **

Building macOS application...
Exception: Build process failed

I am using the default dependencies from the README
  #media_kit: android, iOS, macos, windows, linux
  media_kit: ^1.1.2                              # Primary package.

  media_kit_video: ^1.1.2                        # For video rendering.

  media_kit_native_event_loop: ^1.0.7            # Support for higher number of concurrent instances & better performance.

  media_kit_libs_android_video: ^1.3.0           # Android package for video native libraries.
  media_kit_libs_ios_video: ^1.1.0               # iOS package for video native libraries.
  media_kit_libs_macos_video: ^1.1.0             # macOS package for video native libraries.
  media_kit_libs_windows_video: ^1.0.5           # Windows package for video native libraries.
  media_kit_libs_linux: ^1.1.0                   # GNU/Linux dependency package.

I tried
pod install
pod update
flutter clean

and all of them seem not to help.
Sorry if I am missing something obvious.

I can reproduce.

Flutter changes file encoding when a package is created…

I'm seeing this too. Is there a work around?

The patch is deployed, so sorry for the annoyance.
I don't know whether Flutter or Windows changes the encoding of files before publication, to watch…
dependencies:
  media_kit: ^1.1.2
  media_kit_video: ^1.1.2
  media_kit_native_event_loop: ^1.0.7
  media_kit_libs_android_video: ^1.3.0
  media_kit_libs_ios_video: ^1.1.1
  media_kit_libs_macos_video: ^1.1.1
  media_kit_libs_windows_video: ^1.0.5
  media_kit_libs_linux: ^1.1.0


Proxy support

Older (ver 1.0) has proxy support -
define("RECAPTCHA_HTTP_PROXY", "proxy.foo.com");
define("RECAPTCHA_USE_PROXY", true);
do you plan to have this in ver 2.0?

I have solved this by creating my own RequestMethod for Drupal 6/7/8 modules and it now uses Drupal internal http requests and no longer the limiting library RequestMethod's.

I am not using Drupal. Recaptcha v. 1.0 has proxy support now, what was the point to remove it in v2.0?

I definitely need proxy support as well - tried implementing something using curl with the new API, but I couldn't get it working.

it's an old issue but it could still be used today...
I use recaptcha behind a proxy too. I used curl request method with a small modification in src/ReCaptcha/RequestMethod/CurlPost.php:79
        $options = array(
            CURLOPT_POST => true,
            CURLOPT_POSTFIELDS => $params->toQueryString(),
            CURLOPT_HTTPHEADER => array(
                'Content-Type: application/x-www-form-urlencoded'
            ),
            CURLINFO_HEADER_OUT => false,
            CURLOPT_HEADER => false,
            CURLOPT_RETURNTRANSFER => true,
            CURLOPT_SSL_VERIFYPEER => true,
            CURLOPT_HTTPPROXYTUNNEL => true,
            CURLOPT_PROXY => 'proxy.domain.tld:3128'
        );

Closing super old issues. Please re-raise if still relevant.


wstunnel 9.6.0 not working on arm64

Describe the bug
Binary doesn't run.  I'm assuming that's the not the right tarball or I'm missing some dependency that isn't super obvious?
To Reproduce
Steps to reproduce the behavior:
See steps down below.
Expected behavior
Binary should run.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Debian ARM64
Version: Wstunnel 9.6.0

Additional context
Downloaded from tarball:
debian@debian:~$ wget https://github.com/erebe/wstunnel/releases/download/v9.6.0/wstunnel_9.6.0_linux_arm64.tar.gz
debian@debian:~$ tar xf wstunnel_9.6.0_linux_arm64.tar.gz
debian@debian:~$ sudo mv wstunnel /usr/local/bin/wstunnel
debian@debian:~$ sudo chmod +x /usr/local/bin/wstunnel
debian@debian:~$ ldd /usr/local/bin/wstunnel
/usr/local/bin/wstunnel: error while loading shared libraries: /lib/aarch64-linux-gnu/libc.so: invalid ELF header
debian@debian:~$ uname -a
Linux debian 6.1.0-21-arm64 #1 SMP Debian 6.1.90-1 (2024-05-03) aarch64 GNU/Linux
debian@debian:~$ /usr/local/bin/wstunnel
-bash: /usr/local/bin/wstunnel: cannot execute: required file not found
debian@debian:~$ ls -l /usr/local/bin/wstunnel
-rwxr-xr-x 1 debian debian 8483760 May 25 05:02 /usr/local/bin/wstunnel
debian@debian:~$ file /usr/local/bin/wstunnel
/usr/local/bin/wstunnel: ELF 64-bit LSB pie executable, ARM aarch64, version 1 (SYSV), dynamically linked, interpreter /system/bin/linker64, stripped

ha indeed, I messed up the packing while adding the android build.
Going to make a new release

Oh good, i'm not imagining things... Fantastic news... I appreciate you!

Should be fixed in the latest release https://github.com/erebe/wstunnel/releases/tag/v9.6.1
Sorry about that

Confirmed working... thank you!


Highlight code snippet

I have been using apiary for HTTP Rest endpoints and it is really great!
Now I wanted to add the websocket functionalities, but could not find an easy way. Seems like it is not supported yet. So, I copy-pasted code snippet into the documentation and it looks like a regular text. Is there a way to format the code snippet and highlight the syntax. It is written in Python, if that is relevant.
Thank you!

I am sorry, but I don't think we support syntax highlighting in the plain markdown of an API's documentation. Can you send an email to<EMAIL_ADDRESS>about this?
Thanks.

+1

duplicate of #315?


Add other related rOpenSci packages to the README?

Would you mind adding (or receiving a PR) adding a "related packages" section to the README (and therefore pkgdown website)?
It could mention

tesseract by @jeroen for more direct control of the OCR process.

pdftools for extracting metadata and text from PDF files (therefore more specific to PDF, and without a Java dependency)

tabulizer by @leeper and @tpaskhalis, Bindings for Tabula PDF Table Extractor Library, to extract tables, therefore not text, from PDF files.

epubr by @leonawicz (about to be onboarded) which is more specific than rtika and therefore gives an user-friendlier output when parsing epub files.

I think it'd help people choose a package suitable to their needs. rtika is a powerful package for many formats, which I hope you don't think I'm forgetting. 😺
Thanks in advance!

Okay, I revised it!

Thank you!


Step problem with chart.js

Hi Guys,
I have discovered a problem that I, as I am new to chart.js, can not solve. I want a graph that also shows data that are not in the label. At the moment:
The problem is that I have a month on dates(1st to 31st), I want only 5 labels(not 31 labels on the x-axis). Chart.js scans my data and submit only for the first 5 data listings dots in the graph but I also want to have dots between the labels.
Here it something to illustrate you my problem:
http://imgur.com/a/VJBvj

@Lauchix you should use the time scale. It will do what you are looking for. http://www.chartjs.org/docs/#scales-time-scale

@etimberg Thanks for your help. I will look at it :+1:

@etimberg  I don't know exactly how I should use it. Can you give me a hint or something else ?
I know that I have to put it in options: but after this I have no clue

@Lauchix there are some samples in https://github.com/chartjs/Chart.js/tree/master/samples/timeScale
The key part to using the timescale is to do
options: {
  scales: {
    xAxes: [{
      type: 'time' // uses time axis
    }]
  }
}

Closing as resolved.


Revert "workflows: add concurrency group to check-by-name workflow"

This reverts commit 75600cde6bb25b0e04d2a10d0fb9aaf975e1da81.
Description of changes
@infinisil I don't think it's working as expected.

https://github.com/NixOS/nixpkgs/pull/306427
https://github.com/NixOS/nixpkgs/pull/306418

Add a :+1: reaction to pull requests you find important.

Hmm maybe it does run after all queued checks complete. Still looking for an example of it being canceled with no other pending checks.

Huh weird, yeah let's revert for now, thanks for double-checking this!

Ahh:

Canceling since a higher priority waiting request for 'check-by-name-refs/heads/master' exists

I guess we should use ${{ github.event.pull_request.number }} instead.

Ahh:

Canceling since a higher priority waiting request for 'check-by-name-refs/heads/master' exists

I guess we should use ${{ github.event.pull_request.number }} instead.

Ah that makes sense!

Btw I restarted all the cancelled builds manually (were only a couple), I think it should all be resolved now


Resource identity incorrect

Currently, the encoders use IdType.getValueAsString to populate the ID element within the underlying dataset. This results in a "qualified identifier" being used within the id element, and it causes confusing behaviour when performing query operations directly on the id element. It also causes resource references to fail when the target has technical version metadata, as HAPI adds the version suffix on to the generated qualified identifier.
Two changes need to be made:

Modify the encoders such that only the "ID part" is populated into the id element.
Change resource reference resolution to convert relative literal references into raw identifiers before comparison.

The following things are NOT supported (along with issue references for the work required to support them, where this is planned):

References to individual technical versions of a resource (#216)
Logical references (via identifier) (#217)
References to contained resources
Absolute literal references

Resolved by #138.

Resolved by #138.


Add Tipue Search

Successor to #60. Rebased and updated.

So, I have three outstanding issues with this. It may be that we just accept them for now, but I'm not 100% happy.

Themes must place Tipue search in theme_dir/js/tipuesearch/ which isn't that bad, but then theme_dir/js/tipuesearch/tipuesearch_content.js must contain exactly what we use for our themes, not what Tipue provide (they ship dummy data, we need to turn it into a Jinja template).
Themes that use a dedicated search page must provide a search.html too, much like the 404.html. This is okay I think, but it adds another "global" filename that has specific meaning.

The first bothers me most and it will confuse people. However, I'm leaning towards just accepting that, risking some issue and improving incrementally. Thoughts @tomchristie?

I see we also need a rebase, I'll do that shortly.

Could we bundle the required tipue files as part of MkDocs? Remove the theme requirement apart from search.html?

@tomchristie Good idea, I'll look into providing tipue search for people. I guess we'd output bundled files into a directory like "$site_dir/_mkdocs" to avoid collisions?

Bundling it would also make it easier to update to a new version of Tipue search if we wanted too.

I think 'mkdocs' would be fine. Not a wild fan of the underscore and if someone happens to have a file named 'mkdocs/tipuewhatever.js' then just let it override the default. Obv we'd only want to include those files if search is enabled.

@tomchristie Updated with that new approach, would you have time for a review? Ping me on IRC sometime if it is easier to talk through, I know it is a big change.

We're looking forward to this feature for a long time. I think @d0ugal has done a lot and we can improve it step by step :+1:

Some small bits of initial review in there.

Ping me if you want to discuss properly.

I'd like to test this, which branch of which repo do I have to clone?

@jniggemann Go to my fork and checkout the search branch. However, you don't even need to do that. I install it with pip install https://github.com/d0ugal/mkdocs/archive/search.zip into a virtualenv. With that approach, I use it here http://d0ugal.github.io/review_analysis/ (in a otherwise very work in progress project).

Tested the search on our manual that contains de, en and fr together, no problems using the mkdocs theme. There's a self-made issue with multiple languages at the same time, so stemming and stop words might not work or need some more love.
Thank you all for your work!

Tipue search uses synchronous ajax, which is the biggest no-no in web dev. It not only sets up synchronous ajax for itself, but for all of jQuery on the page, which is even worse. You should probably use an alternative to tipuesearch.
Next, based on my experience from docs.docker.com it seems like a very large search file is downloaded when the page loads, up to a few megabytes. This is completely unnecessary! Load this data asynchronously based on what the user has started typing, not all up front!

This may well be true, but IMHO it's still better than not having any search at all.
@mariusGundersen Do you know other solutions that would be preferrable?

There is some confusion here. @mariusGundersen has been looking at how Docker uses Tipue search, which they do in a forked MkDocs at the moment. This implementation uses a different method for loading the search index, and thus we don't have this issue.
Next, based on my experience from docs.docker.com it seems like a very large search
file is downloaded when the page loads, up to a few megabytes. This is completely
unnecessary! Load this data asynchronously based on what the user has started typing,
not all up front!

We don't however, take this part into account.
@mariusGundersen Do you have a recommendation as an alternative to Tipue?

There's lots of autocomplete plugins to jQuery (which seems to be the base of the front-end for this project), for example in jQuery Ui. It requires a clever backend, one that can find suggestions from the entered text on the fly. That means you can't send the entire cache to the front-end when the page loads, you'll have to do several small loads ads the user types in the search field. This is the way it is usually done.

Thanks, but autocomplete isn't what we are looking for and we want a fully client side solution.
Starting to wonder if we should look at a full text JavaScript library with more traction and support. For example: http://lunrjs.com

Why do you want to use only client-side for search? Wouldn't the search result page need to be generated server-side? Having it all be client-side might work for simple sites, but as the content grows it will become unfeasible to downloaded the entire index for every user.

Because it is extremely easy to use and deploy. It works on static hosts like github pages. Basically, we generate a static website and we are not changing that at this point. Almost everyone using MkDocs will be making a smallish website. The special case is people like Docker but they don't use a released MkDocs version anyway.
I agree it may be worth adding something to help people use server side search. However, a custom theme and using the JSON we generate for Tipue search would get you pretty close to doing that. I'm fairly sure we shouldn't do anything out of the box for this.

@mariusGundersen the Docker documenation is a totally static web site - and the expectation is that the index is cached, so should be slow only initially.
At one point, we also loaded the index was async, and as it was extremely rare that a user would search for something within seconds of loading, that gave good results.
Admittedly, I haven't looked at mkdocs in almost 10 months, so I need to catch up a bit first.

@d0ugal does lunr have more traction? I have no idea, I picked tipue about 12 months ago, as it was one of the few that worked well enough - though I'm saddened that they've still not converted to putting the index into structs, despite the interesting perf stats someone made them.

@SvenDowideit It is hard to measure :) I'm basing that on development activity and github stats. There are also some things that concern me about Tipue, like they don't tag releases in git so it's actually quite hard to see the canonical source of a version. There also only appear to be 10 commits in the github history of Tipue, so I'm not quite sure how it is developed. Maybe I'm looking in the wrong place.

I pip installed the search branch, and it worked well out of the box - I hit one issue, which was that a page with ^ in the text appeared oddly in the search results, which I think is due to https://github.com/tomchristie/mkdocs/pull/222/files#diff-7c8abd762e852c02611bf315b510c46bR303 - Tipue joins some text, a URL and the title together with ^, then splits on ^, which breaks if ^ is in the text. I worked around it (by saying 'O(N**2) algorithm' not 'O(N^2) algorithm' - and entity encoding would probably work too.
Generally this is a good improvement and good quality - I think it'd be nice to see a 0.12.0 release with this in soon.

Report of this not working with: https://github.com/dirkk/basex-rtd/tree/search - needs verification/testing.

@d0ugal: What exactly is the issue? Can you give details, a trace, whatever?

@jniggemann From a quick chat on IRC:
<dubiosos> How can I install the search functionality provided by d0ugal (see https://github.com/tomchristie/mkdocs/pull/222). I did pip install git+https://github.com/d0ugal/mkdocs@search and triggered a new build (mkdocs build --clean), but there is no search.html created
<d0ugal> dubiosos: what theme?
<d0ugal> ReadTheDocs will have a search.html, mkdocs does it in a modal and wont
<d0ugal> None of the other themes support it yet
<dubiosos> I do use the rtd theme
<dubiosos> I did include include_search: true in my mkdocs.xml file, but I am not sure if it is needed though
<d0ugal> dubiosos: it is
<d0ugal> dubiosos: I'm not sure what is going wrong
<dubiosos> I uploaded my current working branch at https://github.com/dirkk/basex-rtd/tree/search
<dubiosos> If you could take a look, I would be very grateful!

Is it ready to merge? It's a large patch and we're all looking foreword to use it. I hope we can merge it and improve it from time to time. @d0ugal :+1:

Reminder to self - figure out integrating this with the Django REST framework theme.

I was planning on doing that, first I want to settle my doubts with Tipue.

To help move this along I plan to rebase and split the change up. I want to land the index generation first and then implement it in themes - disconnecting the index code from the client side search implementation.

This seemed to be on the brink of being merged in early December but now seems to have relapsed.
Is there any ETA for it being merged or has it been replaced by a Lunr.js approach?

@samuelcolvin I've been quietly working on a large refactor of this implementation and I've been getting close. So far I'm happier with the results but I still have work to do - I don't think it is quite fully functional yet.
See #457 for the new work being done here. I'll close this PR for now, I didn't want to rebase on this one as there is some interesting discussion etc. here.

hello. what is the currently status for searching? is it working? I've just upgraded to version 0.12.2 and it is not working using readthedocs theme.

@gustavosoares Search will be released with 0.13.0 track progress at #502


Add provider lease-shell command

This adds a way to get a lease shell into an existing container that is already running. There are basically two ways to do this

Proxy everything through the provider and do auth there
Issue temporary authorizations via kubernetes RBAC so a customer can directly connect to their containers via kubectl or similar commands.

I chose route 1. The other option is to use ephemeral containers that share cgroups & namespaces with the exiting containers. If we want to do this in the future we can add that later without any real rework. The main advantage to doing that is the customer's container doesn't need to actually have a shell stored within it.
The way I implemented this was by reverse engineering the implementation of kubectl exec one step at a time, then splitting our implementation between the provider & the CLI. The provider proxies everything over a single websocket at this time. I've tested every scenario I can come up with and tried to make the error messages as helpful as possible to the end user. The biggest stumbling block is when users deploy containers which either don't exist (image can't be pulled) or the container crashes immediately on startup. In that case, they won't be able to get a shell into them since they never actually run. This isn't actually a step backwards however, it just indicates we need better visibility into deployment health or status.
The biggest drawback I think is the amount of code I had to rewrite. It's large enough that most of the implementation got split into its own files. I added some simple unit tests around utilities I created for use from within the code. Actually testing the implementation is quite difficult and would require a huge number of additional mocks to be generated at this time. I added another integration test that is larger identical to the others. It tests that it can remotely invoke a command, then tests the error cases that the client can commonly run into.
There are basically 4 components that live as part of this now

The provider  CLI command, which also does terminal management. Since this command runs an interactive terminal, it's important that the state be saved & restored in that case. I'm using the kubernetes code for this directly
The CLI "client", which handles dialing a websocket and de-multiplexing everything associated with this. All data comes back over the websocket as a binary message and has an identifier prepended to it. The client is responsible for directing this data to stdout, stderr, etc.
The provider router, which is responsible for accepting the websocket request. This has to the opposite of the client, and multiplex all the data onto the websocket. It also tries to validate all the parameters before reaching out to kubernetes.
The kubernetes client code, which actually talks to kubernetes. I don't really like the way the golang API for this works. It's very awkward and I had to instantiate many objects by hand in order to get it to work. You also have to use two clients to talk to kubernetes, one to build the request and another SPDY client to send it to kubernetes. If you build the request improperly, you get a silent failure. It was however faster to go through the trial and error process to figure this out rather than implement my own client library for talking to kubernetes.

The kubernetes client in part 4 is quite frustrating with respect to errors. Essentially all errors come back from kubernetes as untyped when something fails. It falls back to string matching in order to identify errors, but avoids ever just returning errors verbatim. There is no way to safely do this, since we don't know what information kubernetes could return.
Overall I think this is a pretty food first attempt. None of this requires blockchain updates, so we can iterate on it as fast as we would like to.

@troian since im using a websocket do I need to setup the pong handler goroutine after the websocket is established?

yeah, better to have keepalive messages going through to maintain tcp session alive

I think I have addressed all the comments at this point in time

Comments have been addressed

This pull request has been mentioned on Akash Support. There might be relevant details there:
https://forum.akash.network/t/sharing-my-first-deploy-experience-feedback-welcome/605/6


SCIM identities support and documentation

Medplum stores identities as resources and plans to support SCIM through FHIR Mapping language.
Related: #1211

https://github.com/medplum/medplum/pull/1581
https://github.com/medplum/medplum/pull/1585
https://github.com/medplum/medplum/pull/1588
https://github.com/medplum/medplum/pull/1709


Add Java 16 to minecraft docker images

Feature Request
Feature Information:
Since Minecraft 1.17 will be released, Java 16 or newer will be the only versions supported by Paper and forks.
I confirm:

[x] that I have searched for an existing feature request matching the description.

This has been completed, and also added to the GetAMP script.


Loss become nan after several epoches

Thank you for the amazing open-source project! I recently applied this code to train a model on ucf101, but the loss becomes nan after several epochs. Does anyone have similar problems? Are there any suggestions? Thanks!

After the Open-Sora update, --enable_flashattn occurs NaN loss on my side. @oahzxl

thats weird. have you used bf16? we have not met such problem in training.

and what global batch size are you using

Hi.
I tried 1. disable modulate kernel 2. use bf16 3. grad clip; and all possible other things.
And only I disabled the flashattn, loss was not NaN.
I used a global batch of around 8(512x512) and 32(256x256).

My cuda is 12.4
I think it could be the problem.

close due to inactive


PropType issue in TraceTypeSelector

Heads-up @VeraZab @aulneau
Warning: Failed prop type: Invalid prop `actions` of type `function` supplied to `Item`, expected `array`.
    in Item (created by TraceTypeSelector)
    in TraceTypeSelector (created by LocalizedTraceTypeSelector)
    in LocalizedTraceTypeSelector (created by ModalProvider)
    in ModalProvider (created by PlotlyEditor)
    in div (created by PlotlyEditor)
    in PlotlyEditor (created by App)
    in div (created by App)
    in div (created by App)
    in App (created by HotExportedApp)
    in AppContainer (created by HotExportedApp)
    in HotExportedApp

I'll tack this fix on with my icons pr later today

thanks @aulneau !


Break raw transaction in sandbox

Do we need a test?

Do we need a test?

It would be a good idea :)

Really we should re-add as many as possible of the removed sandbox'ed tests from pool_run.exs and pool_transactions.exs.

Yes, we can add them to the existing integration_test/sql/test_transaction.exs.

Tests uncovered 2 more bugs

How severe are those bugs? Should we get v0.13.1 out?

The first bug means that if a Ecto.Adapters.Pool.run fun raises an exception, the worker is checked in twice. Unfortunately poolboy doesn't use references to track messages and so treates the second checkin as a valid checkin. Ecto.Adapters.SQL.Sandbox uses references to track messages and crashed which led to the discovery of the bug.
I have not verified but I think the following race condition can occur:

Run fun raises and fuse breaks
Worker closes connection and starts a new one
The first checkin occurs
A client checks out the worker with checkout to do a run
The second checkin occurs
A clients checks out the worker with open_transaction to do a transaction
Transaction begins and the query from run/1 is run inside it.

If so it is severe :(.
The second bug is sandbox only where break does not break in raw mode. Ironically the first bug may just crash sandbox. However it can lead to unpredictable behaviour which I think is severe enough on its own.


Improved the description of loadable and address-only types

Added a missing requirement that loadable types must be loaded and stored in terms of their parts. Added some examples of loadable and address-only types that are not expressible in Swift today but help illustrate the concepts.

@swift-ci Please test

@swift-ci Please smoke test

⛵