textShaper.cpp may cause overflow, Visual Studio raise error for it。

file: core\deps\alfons\src\alfons\textShaper.cpp
            m_shapes[id] = Shape(_face.id(), codepoint, offset, advance, flags);

The id may not exist in m_shapes,
may condition are: when id == 68
but the size of m_shapes are also 68.
        } else {
            addedGlyphs = true;
            m_glyphAdded[id] = 1;

            uint8_t breakmode = m_linebreaks[clusterId];

             uint8_t flags = 1 |           // cluster start
                //((breakmode == LINEBREAK_MUSTBREAK) ? 2 : 0) | // must break
                ((breakmode == 1) ? 4 : 0) | // can break
                ((breakmode == 2) ? 8 : 0) | // no break
                (_face.isSpace(codepoint) ? 16 : 0);

            m_shapes[id] = Shape(_face.id(), codepoint, offset, advance, flags);
        }

Thanks, I'll look into it.
On first sight it looks correct - m_shapes is large enough for the 'original text'. id is the 'cluster index' that correspondents to the index of the 'original text' (minus the _run.start offset)
From harfbuzz docs for hb_buffer_get_glyph_infos /  hb_glyph_info_t
 * @cluster: the index of the character in the original text that corresponds
 *           to this #hb_glyph_info_t, or whatever the client passes to
 *           hb_buffer_add(). More than one #hb_glyph_info_t can have the same
 *           @cluster value, if they resulted from the same character (e.g. one
 *           to many glyph substitution), and when more than one character gets
 *           merged in the same glyph (e.g. many to one glyph substitution) the
 *           #hb_glyph_info_t will have the smallest cluster value of them.
 *           By default some characters are merged into the same cluster
 *           (e.g. combining marks have the same cluster as their bases)
 *           even if they are separate glyphs, hb_buffer_set_cluster_level()
 *           allow selecting more fine-grained cluster handling.

@lygstate  could you paste the full error or warning?

 	msvcp140d.dll!69592f06()	Unknown
 	[Frames below may be incorrect and/or missing, no symbols loaded for msvcp140d.dll]
 	tangram.exe!std::vector<alfons::Shape,std::allocator<alfons::Shape> >::operator[](unsigned int _Pos) Line 1234	C++
>	tangram.exe!alfons::TextShaper::processRun(const alfons::FontFace & _face, const alfons::TextRun & _run, alfons::FontFace::Metrics & _lineMetrics) Line 356	C++
 	tangram.exe!alfons::TextShaper::shape(std::shared_ptr<alfons::Font> & _font, const alfons::TextLine & _line, const std::vector<alfons::TextRun,std::allocator<alfons::TextRun> > & _range, alfons::LineLayout & _layout) Line 418	C++
 	tangram.exe!alfons::TextShaper::shapeICU(std::shared_ptr<alfons::Font> & _font, const icu_52::UnicodeString & _text, const hb_language_impl_t * _langHint, hb_direction_t _direction) Line 483	C++
 	tangram.exe!Tangram::FontContext::layoutText(Tangram::TextStyle::Parameters & _params, const icu_52::UnicodeString & _text, std::vector<Tangram::GlyphQuad,std::allocator<Tangram::GlyphQuad> > & _quads, std::bitset<64> & _refs, glm::tvec2<float,0> & _size, std::array<Tangram::Range,3> & _textRanges) Line 134	C++
 	tangram.exe!Tangram::TextStyleBuilder::prepareLabel(Tangram::TextStyle::Parameters & _params, Tangram::Label::Type _type) Line 734	C++
 	tangram.exe!Tangram::TextStyleBuilder::addFeature(const Tangram::Feature & _feat, const Tangram::DrawRule & _rule) Line 183	C++
 	tangram.exe!Tangram::TileBuilder::applyStyling(const Tangram::Feature & _feature, const Tangram::SceneLayer & _layer) Line 87	C++
 	tangram.exe!Tangram::TileBuilder::build(Tangram::TileID _tileID, const Tangram::TileData & _tileData, const Tangram::TileSource & _source) Line 125	C++
 	tangram.exe!Tangram::TileTask::process(Tangram::TileBuilder & _tileBuilder) Line 23	C++
 	tangram.exe!Tangram::TileWorker::run(Tangram::TileWorker::Worker * instance) Line 94	C++
 	tangram.exe!std::_Invoker_pmf_pointer::_Call<void (__thiscall Tangram::TileWorker::*)(Tangram::TileWorker::Worker *),Tangram::TileWorker *,Tangram::TileWorker::Worker *>(void(Tangram::TileWorker::*)(Tangram::TileWorker::Worker *) _Pmf, Tangram::TileWorker * && _Arg1, Tangram::TileWorker::Worker * && <_Args2_0>) Line 1340	C++
 	tangram.exe!std::invoke<void (__thiscall Tangram::TileWorker::*)(Tangram::TileWorker::Worker *),Tangram::TileWorker *,Tangram::TileWorker::Worker *>(void(Tangram::TileWorker::*)(Tangram::TileWorker::Worker *) && _Obj, Tangram::TileWorker * && <_Args_0>, Tangram::TileWorker::Worker * && <_Args_1>) Line 1443	C++
 	tangram.exe!std::_LaunchPad<std::unique_ptr<std::tuple<void (__thiscall Tangram::TileWorker::*)(Tangram::TileWorker::Worker *),Tangram::TileWorker *,Tangram::TileWorker::Worker *>,std::default_delete<std::tuple<void (__thiscall Tangram::TileWorker::*)(Tangram::TileWorker::Worker *),Tangram::TileWorker *,Tangram::TileWorker::Worker *> > > >::_Execute<0,1,2>(std::tuple<void (__thiscall Tangram::TileWorker::*)(Tangram::TileWorker::Worker *),Tangram::TileWorker *,Tangram::TileWorker::Worker *> & _Tup, std::integer_sequence<unsigned int,0,1,2> __formal) Line 240	C++
 	tangram.exe!std::_LaunchPad<std::unique_ptr<std::tuple<void (__thiscall Tangram::TileWorker::*)(Tangram::TileWorker::Worker *),Tangram::TileWorker *,Tangram::TileWorker::Worker *>,std::default_delete<std::tuple<void (__thiscall Tangram::TileWorker::*)(Tangram::TileWorker::Worker *),Tangram::TileWorker *,Tangram::TileWorker::Worker *> > > >::_Run(std::_LaunchPad<std::unique_ptr<std::tuple<void (__thiscall Tangram::TileWorker::*)(Tangram::TileWorker::Worker *),Tangram::TileWorker *,Tangram::TileWorker::Worker *>,std::default_delete<std::tuple<void (__thiscall Tangram::TileWorker::*)(Tangram::TileWorker::Worker *),Tangram::TileWorker *,Tangram::TileWorker::Worker *> > > > * _Ln) Line 247	C++
 	tangram.exe!std::_LaunchPad<std::unique_ptr<std::tuple<void (__thiscall Tangram::TileWorker::*)(Tangram::TileWorker::Worker *),Tangram::TileWorker *,Tangram::TileWorker::Worker *>,std::default_delete<std::tuple<void (__thiscall Tangram::TileWorker::*)(Tangram::TileWorker::Worker *),Tangram::TileWorker *,Tangram::TileWorker::Worker *> > > >::_Go() Line 232	C++
 	tangram.exe!std::_Pad::_Call_func(void * _Data) Line 209	C++
 	ucrtbased.dll!invoke_thread_procedure(unsigned int(__stdcall*)(void *) procedure, void * const context) Line 92	C++
 	ucrtbased.dll!thread_start<unsigned int (__stdcall*)(void *)>(void * const parameter) Line 115	C++
 	kernel32.dll!@BaseThreadInitThunk@12()	Unknown
 	ntdll.dll!__RtlUserThreadStart()	Unknown
 	ntdll.dll!__RtlUserThreadStart@8()	Unknown

This is the call stack.

I think the issue is that the size of m_shapes is not always set to its actual capacity. However the use of m_shapes is always within the capacity, so technically there was no out-of-bounds access. Still nice that MSVC can detect this issue. https://github.com/tangrams/tangram-es/pull/1358


traefik: 2.6.3 -> 2.7.1

Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Do we really need codename at link time?

@aaronjheng, @marsam, this PR refactor disables inclusion of the dashboard UI.  Was that intentional?
Also relevant: https://github.com/NixOS/nixpkgs/issues/41378

You mean webui exists before in nixpkgs?

Yes, webUI was available for at least the last few nixos releases, but no longer works after this PR.  You can verify by entering a shell with a few different nixpkgs release versions of traefik and testing locally.
For example:
nix shell github:NixOS/nixpkgs/nixos-21.11#traefik      # traefik 2.5.1 webUI = yes
nix shell github:NixOS/nixpkgs/nixos-22.05#traefik      # traefik 2.6.3 webUI = yes
nix shell github:NixOS/nixpkgs/nixos-unstable#traefik   # traefik 2.7.1 webUI = no

# After entering a shell with a traefik version from above, run locally to check for webUI with:
sudo traefik --api.dashboard=true --api.insecure=true

# Test in a browser for the UI:
http://localhost:8080

@johnalotoski https://github.com/NixOS/nixpkgs/pull/179563 will fix this issue. Thanks!

Thank you!


version number in master branch doesn't match latest release

The master branch, if built manually, ends up being detected as version 2.1.3, rather than the (correct based on git tag and commit id) 2.1.5

2.1.5 was allready given on the master branch.
2.1.5 Labels and assembly version should now be correct with:
https://github.com/TheCheatsrichter/Gw2_Launchbuddy/commit/8ee9cc69fbb6fadfa1bde15374556a3cc8ee633f


I have a question

I ask you a question for a doubt that I have, why do you use the fields posted_at or registered_at instead of created_at?

Hey !
I use posted_at to be able to publish post in the futur.
And by experience I don't like to rely on created_at and updated_at fields because it's handle by laravel itself and I do not have control on this fields.
If you export your data and import it again with a script, this fields will be override.


WIP - clone modal getting prepopulated with form values but unable to…

Resolves #333 Cloning of Reader Entities

Coverage decreased (-0.2%) to 46.503% when pulling 36144febdd53514253d3eeb41b019166c69e81c6 on DAME-Aug-Sept-sprint-333-clone-reader into 650a3ed9d072b65fa5b698972162147a42d3dbf4 on DAME-Aug-Sept-sprint-staging.


⚠ BCR Bot has degraded performance

In 01545c1, BCR Bot (https://bcr.alternativasinteligentes.com/BCRApi/web/chatbot) experienced degraded performance:

HTTP code: 200
Response time: 953 ms

Resolved: BCR Bot performance has improved in 073967c.


Prompt template is not adapted to the dataset

Adaptive Prompt Template
Issues:

Currently, the provided templates in prompts are relatively hard-coded for all datasets.
The templates were designed for not very good models. If we use Llama3+, it might be better to at least ask the model to generate more queries.
Would be good if the templates were adapted to the specific dataset, e.g. for webqsp, the template could be "What is the answer to the question: {question}?".

can't we just fit a prompt with dspy instead of trying to hard-code new ones?

Issue is, it might not be very compute-efficient. We can try to generate one prompt per dataset. I need to say that I am thinking along the lines of using a 70B+ model for generation. If we only use <=8B models, this concern is not as real at all.

Partially resolved using DSPy and DynamicPrompt in #5 and #6.

Partially resolved using DSPy and DynamicPrompt in https://github.com/danilotpnta/IR2-project/pull/5 and https://github.com/danilotpnta/IR2-project/pull/6.


bug: Formatting table with bracket string key results in invalid lua code.

Describe the bug
Formatting a table with double square brackets string keys collapses the space between the surrounding brackets and the string's brackets, resulting in invalid lua code.
To Reproduce
Format the following code:
local foo = {
    [ [[bar]] ] = 123,
}

Expected behavior
Formatting should not result in invalid code
Screenshots

Environment:

OS: Windows
Is WSL remote? No
Client: VSCode

Provide logs
file_c%3A_Users_Fire_source_lua_lls.log

The same happens when just indexing
foo[ [[bar]] ]

In the meanwhile, is there a way to ignore formatting for single a node? Like --codestyle-ignore or ---@codestyle ignore-next-line ?

format bug report to EmmyLuaCodeStyle

this bug has been fix,wait for next release


PHP tags inside HTML code not highlighted if Doctype tag is present

When Emmet is enabled, the  PHP tags code inside HTML are not highlighted properly if the file has the  tag or similar included. I've tried using different themes, with the same result.
To be able to see the problem, after including DOCTYPE you have to save, close, and reopen the file, same if you take it away.

Emmet has nothing to do with code highlighting.

You're right, I've been having many problems with Atom and some packages, so I got in the "disable, restart, check, enable..." loop and got this wrong, sorry.

here:
https://github.com/atom/language-php/issues/48


MongoDB Scaler: ScaledObject give error when we specify port field value without database name.

Report
When i have tried MongoDB scalar with the following ScaledObject yaml:
apiVersion: keda.sh/v1alpha1
kind: ScaledObject
metadata:
  name: mongodb-scaledobject
spec:
  scaleTargetRef:
    apiVersion: apps/v1
    kind: Deployment
    name: nginx-deployment
    envSourceContainerName: nginx
  pollingInterval: 30
  cooldownPeriod:  60
  minReplicaCount: 0
  maxReplicaCount: 100
  advanced:
    restoreToOriginalReplicaCount: true
  triggers:
  - type: mongodb
    metadata:
     host: mongodb-svc.default.svc.cluster.local
     port: "27017"
     username: test_user
     dbName: "test"
     collection: "test_collection"
     query: '{"region":"eu-1","state":"running","plan":"planA"}'
     queryValue: "1"
     metricName: "global-metric"
    authenticationRef:
     name: mongodb-trigger

The ScaledObject is created with False status:
$ kubectl get so
NAME                   SCALETARGETKIND      SCALETARGETNAME    MIN   MAX   TRIGGERS   AUTHENTICATION    READY   ACTIVE    AGE
mongodb-scaledobject   apps/v1.Deployment   nginx-deployment   0     100   mongodb    mongodb-trigger   False   Unknown   7s

And SO show following Warning:
failed to ping mongoDB, because of connection() error occured during connection handshake: auth error: sasl conversation error: unable to authenticate using mechanism "SCRAM-SHA-1": (AuthenticationFailed) Authentication failed.
But when i have changed the port: "27017" to port: "27017/test" where test is the database it is working fine. scaling up and scaling down is working. Here is changed ScaledObject yaml:
apiVersion: keda.sh/v1alpha1
kind: ScaledObject
metadata:
  name: mongodb-scaledobject
spec:
  scaleTargetRef:
    apiVersion: apps/v1
    kind: Deployment
    name: nginx-deployment
    envSourceContainerName: nginx
  pollingInterval: 30
  cooldownPeriod:  60
  minReplicaCount: 0
  maxReplicaCount: 100
  advanced:
    restoreToOriginalReplicaCount: true
  triggers:
  - type: mongodb
    metadata:
     host: mongodb-svc.default.svc.cluster.local
     port: "27017/test"
     username: test_user
     dbName: "test"
     collection: "test_collection"
     query: '{"region":"eu-1","state":"running","plan":"planA"}'
     queryValue: "1"
     metricName: "global-metric"
    authenticationRef:
     name: mongodb-trigger

Here is the Secrets, TriggerAuthentication and Deployment yaml which i have used:
apiVersion: v1
kind: Secret
metadata:
  name: mongodb-secret
type: Opaque
data:
  connect: dGVzdF9wYXNzd29yZA==

apiVersion: keda.sh/v1alpha1
kind: TriggerAuthentication
metadata:
  name: mongodb-trigger
spec:
  secretTargetRef:
    - parameter: password
      name: mongodb-secret
      key: connect

apiVersion: apps/v1
kind: Deployment
metadata:
  name: nginx-deployment
  labels:
    app: nginx
spec:
  replicas: 1
  selector:
    matchLabels:
      app: nginx
  template:
    metadata:
      labels:
        app: nginx
    spec:
      containers:
      - name: nginx
        image: nginx:1.14.2
        ports:
        - containerPort: 80

Expected Behavior
The ScaledObject should work for port:"27017" and take database name from dbName field.
Actual Behavior
The scaledObject work when we give database name with port value port:"27017/test"  in case when we use host,port method (detailed information method)
Steps to Reproduce the Problem

Set-up minikube v1.18.1
Deploy mongodb.
Create target deployment with above yaml and apply it.
Create secrets and TA with above yaml and apply it.
Create SO with above yaml's and apply it.
Check SO events by kubectl describe so
Check the logs of keda-operator pod by kubectl logs  -n keda

Logs from KEDA operator
{"reconcilerGroup": "keda.sh", "reconcilerKind": "ScaledObject", "controller": "scaledobject", "name": "mongodb-scaledobject", "namespace": "default", "error": "error getting scaler for trigger #0: failed to ping mongoDB, because of connection() error occured during connection handshake

KEDA Version
2.2.0
Kubernetes Version
1.20
Platform
No response
Scaler Details
MongoDB
Anything else?
No response

I think we should add here

Hi @tomkerkhove @zroubalik
Could you please take a look!

@Shubham82 good catch, you are right. I think that the whole scaler (the input parameters) would really use refactoring on both code and docs side.
Would be great if you can open PRs and we can discuss the changes there, thanks!

Thanks @zroubalik  for the response.
Will soon create a PR for it.

Fixed it here #1787


Generic analysis

Add generic analysis folder, i.e. analysis not specific to any version.
Can be run on any data set. Data-set specific information (SF and recenergy
vectors) and extracted from em analysis. MC legend passed within parser
to macro. Data folder passed as well to macro. Created starting from
v2.6 analysis. Analysis to select channels not changed at the moment (keeping v2.6 ones).

Accepting PR #25.


Minify main.js

Releases of QuickAdd use an unminified version of main.js. This causes the plugin to take longer to load than necessary. By adding a minify step to the build script, you should be able to significantly speed up the app load time, particularly on mobile.

Thanks for the suggestion, @kepano. I've just pushed a release with a minified main.js. This reduced size by ~9%.


forge fmt moved the comma in front of comment out parameters

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (f7e3677 2023-01-23T00:04:47.538258979Z)
What command(s) is the bug in?
forge fmt
Operating System
Linux
Describe the bug
forge fmt changed a style which I can't explain. please see the changes everywhere my code has now:
before running forge fmt
function approve(address /owner/, uint256 /amount/ )
after running 'forge fmt'
function approve(address, /owner/ uint256 /amount/ )
Notice the comma is moved and it's not what we get used to.

Ah just saw this before I created https://github.com/foundry-rs/foundry/issues/4451, will close in favor of that one since the code did not format properly here


ecdsatool: fix build

Description of changes
Part of #309482.
kaniini/ecdsatool is a long unmaintained package, that has failed to build since November 2023.
I have patched some missing headers and old openssl invocations, and the tool seems to work now (at least as far as the README commands from upstream go).
I am not keen to maintain this however, and we should consider removing it outright - it is an ancient piece of openssl-reliant code that deals with cryptography, it is only a matter of time before it breaks again.
If there is no feedback supporting its continued inclusion in nixpkgs, I will update this PR to remove it instead.
Test run

$ ./result/bin/ecdsatool keygen /tmp/test.key
AzM0YzHDf9Y1dZaxPfAJHyJ7IWurlADtnyBCbQEvgEiA
$ cat /tmp/test.key
-----BEGIN EC PRIVATE KEY-----
MFcCAQEEIHZlrPOEqHsKWH76rZEVy1UAcfzeJkEUoGdIFx5GNRGjoAoGCCqGSM49
AwEHoSQDIgADMzRjMcN/1jV1lrE98AkfInsha6uUAO2fIEJtAS+ASIA=
-----END EC PRIVATE KEY-----
$ ./result/bin/ecdsatool pubkey /tmp/test.key
AzM0YzHDf9Y1dZaxPfAJHyJ7IWurlADtnyBCbQEvgEiA
$ ./result/bin/ecdsatool keyinfo /tmp/test.key
Information on /tmp/test.key:
    Private-Key: (256 bit)
    priv:
        76:65:ac:f3:84:a8:7b:0a:58:7e:fa:ad:91:15:cb:
        55:00:71:fc:de:26:41:14:a0:67:48:17:1e:46:35:
        11:a3
    pub:
        03:33:34:63:31:c3:7f:d6:35:75:96:b1:3d:f0:09:
        1f:22:7b:21:6b:ab:94:00:ed:9f:20:42:6d:01:2f:
        80:48:80
    ASN1 OID: prime256v1
    NIST CURVE: P-256
$ ./result/bin/ecdsatool sign /tmp/test.key 'QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUE='
MEQCIHlB6er4whQ6dS9XmyB0mjLxUnK5JzaAsu0SrSij1qBFAiAyie7QOoDhbiiE3Z25R6aI83cdWnSqSFCTzgj1/SA9xw==

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

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

cc @aakropotkin for your thoughts on its continued usefulness as the original contributor

Yeah I have no problem with removing it or updating.
LGTM.


New env var DD_HASHCODE_FIELDS_PER_SCANNER

Description
Simply a new env var that enables users to hardcode their hashcode fields another way, which will be possible for Kubernetes deployments too.
It copies from DD_CELERY_BROKER_TRANSPORT_OPTIONS, because it's loading a JSON String from the env, into a dict, and replaces the settings.dist.py values with the ones in the dict
Test results
Not much testing to add I think, it works well, no exception handling needed because we don't want the startup to work if the value isn't valid.
Documentation
A lot of env vars are kinda undocumented, I think I'll push another commit to mention it.

Might be a bug... the variable HASHCODE_FIELDS_PER_SCANNER doesn't seem like it's modified


Error: llama runner process has terminated

Cannot seem to run any model - I tried 3B and 7B models and no luck. They all fail immediately - I wonder if I am missing a dependency.
ollama run orca-mini --verbose

Error: llama runner process has terminated

Does not appear to be a RAM issue, this VM has 64GB. The process fails immediately.

From journalctl -u ollama :

Dec 19 10:46:26 wt-chatbot-v1 ollama[639]: 2023/12/19 10:46:26 llama.go:403: skipping accelerated runner because num_gpu=0
Dec 19 10:46:26 wt-chatbot-v1 ollama[639]: 2023/12/19 10:46:26 llama.go:436: starting llama runner
Dec 19 10:46:26 wt-chatbot-v1 ollama[639]: 2023/12/19 10:46:26 llama.go:494: waiting for llama runner to start responding
Dec 19 10:46:26 wt-chatbot-v1 ollama[639]: 2023/12/19 10:46:26 llama.go:451: signal: illegal instruction
Dec 19 10:46:26 wt-chatbot-v1 ollama[639]: 2023/12/19 10:46:26 llama.go:459: error starting llama runner: llama runner process has terminated
Dec 19 10:46:26 wt-chatbot-v1 ollama[639]: 2023/12/19 10:46:26 llama.go:525: llama runner stopped successfully

The ollama serve service daemon seems to be running

● ollama.service - Ollama Service
Loaded: loaded (/etc/systemd/system/ollama.service; enabled; preset: enabled)
Active: active (running) since Tue 2023-12-19 11:17:39 UTC; 3s ago
Main PID: 901 (ollama)
Tasks: 7 (limit: 77155)
Memory: 8.9M
CPU: 6ms
CGroup: /system.slice/ollama.service
└─901 /usr/local/bin/ollama serve
Dec 19 11:17:39 wt-chatbot-v1 systemd[1]: Started ollama.service - Ollama Service.
Dec 19 11:17:39 wt-chatbot-v1 ollama[901]: 2023/12/19 11:17:39 images.go:737: total blobs: 16
Dec 19 11:17:39 wt-chatbot-v1 ollama[901]: 2023/12/19 11:17:39 images.go:744: total unused blobs removed: 0
Dec 19 11:17:39 wt-chatbot-v1 ollama[901]: 2023/12/19 11:17:39 routes.go:871: Listening on <IP_ADDRESS>:11434 (version 0.1.16)
Dec 19 11:17:39 wt-chatbot-v1 ollama[901]: 2023/12/19 11:17:39 routes.go:891: warning: gpu support may not be enabled, check that you have installed GPU drivers: nvidia-smi command failed

Does anyone have any ideas?
Environment
ollama version is 0.1.16
Installed via curl https://ollama.ai/install.sh | sh
Running in a Proxmox VM, with 12 threads and 64GB ram.
No LSB modules are available.
Distributor ID:	Debian
Description:	Debian GNU/Linux 12 (bookworm)
Release:	12
Codename:	bookworm

Facing the same error

I stopped the daemon, and tried the official docker container. Same result.

I set the VM's CPU 'type' to 'host' in Proxmox. From this issue it seems to be related to missing CPU instructions such as AVX.
https://github.com/jmorganca/ollama/issues/1503


Weird install behavior wants files in /include/internal/*.hpp

The main dynamic.hpp wants to include internal files as #include <internal/whatever.hpp>. To satisfy this when installing DYNAMIC into a prefix, we have to dump DYNAMIC's internal directory straight into the root of the prefix (or tell downstream projects to use -I<prefix>/dynamic). Either solution causes trouble if any other e.g. internal/bwt.hpp exists in the project's include paths.
DYNAMIC ought to reference its internal includes via a path that has the library's name in it, to prevent filename conflicts and people scratching their heads at what package is responsible for a directory like /usr/local/include/internal.

Hi Adam, you're right: the directory structure was a bit messy. It should be fixed now.


Features that we would like to have

Hi!
As you have noticed I have started to contribute to this project. I am doing that because I am POCing a service mesh solution for Egnyte that is using this project.
To make it work, apart from #51 and #59, I would also need a few more features, so I would like to ask how do they align with your plans - perhaps you are already working on some of them or on the contrary: you will not even accept PRs with them. Let's have a chat about it. :)
So here it goes, in no specific order:

Customizable HAproxy logs format
Add custom config fragment to the upstream services frontends - mostly to capture some request headers for enriching the logs
Add custom config fragment to the upstream services backends - for various things, like rspirep directives for example,
Add a completely custom additional frontend(s) - where we would some custom logic above, f.e. splitting the traffic based on the paths between the upstream backends
Listening on the UNIX sockets - as an alternative to above approach we could put the custom loginc into a separate HAproxy instance in front of the sidecar. To minimize the performance penalty we would like it to connect to the sidecar over these sockets.
Custom load balancing algorithms? - maybe it doesn't make sense but we are looking for a way to make the load balancing as fair (even) as possible with the distributed load balancers..

So what do you think?

After discussion with the team we got to the conclusion that our goals may not align with yours and because the pace of your project does not suit our POC needs we will work for now on our fork.
We have highly customized HAproxy configs with a lot of logic in them so we plan to implement a different approach than you. We want to make the HAproxy sidecar use configuration from a template file, with frontends and backends already in it (without the servers) and then fill it in and keep up to date dynamically with service mesh-specific stuff, like userlist controller and backend spoe_back and the servers list.
However if this works out we would strongly prefer to make this an alternative to your HAproxy configuration method and merge our fork back with your project.
Please ping us back with your thoughts. We could also arrange a call to sync and cooperate better. :)


python310Packages.gnureadline: 8.0.0 -> 8.1.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.gnureadline is: "The standard Python readline extension statically linked against the GNU readline library"
meta.homepage for python310Packages.gnureadline is: "https://github.com/ludwigschwardt/python-gnureadline"
meta.changelog for python310Packages.gnureadline is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 8.1.2 with grep in /nix/store/1pvpwc4h6bas74gx4l47l4ayswlfghnw-python3.10-gnureadline-8.1.2
found 8.1.2 in filename of file in /nix/store/1pvpwc4h6bas74gx4l47l4ayswlfghnw-python3.10-gnureadline-8.1.2

Rebuild report (if merged into master) (click to expand)

28 total rebuild path(s)

15 package rebuild(s)

14 x86_64-linux rebuild(s)
14 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
certbot
certbot-full
mavproxy
python310Packages.certbot
python310Packages.certbot-dns-cloudflare
python310Packages.certbot-dns-google
python310Packages.certbot-dns-rfc2136
python310Packages.certbot-dns-route53
python310Packages.gnureadline
python39Packages.certbot
python39Packages.certbot-dns-cloudflare
python39Packages.certbot-dns-google
python39Packages.certbot-dns-rfc2136
python39Packages.certbot-dns-route53
python39Packages.gnureadline

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/1pvpwc4h6bas74gx4l47l4ayswlfghnw-python3.10-gnureadline-8.1.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.gnureadline https://github.com/r-ryantm/nixpkgs/archive/e404aadb3b5a7a1cee1a22feae4bec1b24e41eb0.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/1pvpwc4h6bas74gx4l47l4ayswlfghnw-python3.10-gnureadline-8.1.2
ls -la /nix/store/1pvpwc4h6bas74gx4l47l4ayswlfghnw-python3.10-gnureadline-8.1.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

13 packages built:

certbot (python39Packages.certbot)
certbot-full
python310Packages.certbot
python310Packages.certbot-dns-cloudflare
python310Packages.certbot-dns-google
python310Packages.certbot-dns-rfc2136
python310Packages.certbot-dns-route53
python310Packages.gnureadline
python39Packages.certbot-dns-cloudflare
python39Packages.certbot-dns-google
python39Packages.certbot-dns-rfc2136
python39Packages.certbot-dns-route53
python39Packages.gnureadline

Maintainer pings

Result of nixpkgs-review pr 177650 run on aarch64-darwin 1

13 packages marked as broken and skipped:

certbot
certbot-full
mavproxy
python310Packages.certbot
python310Packages.certbot-dns-cloudflare
python310Packages.certbot-dns-google
python310Packages.certbot-dns-rfc2136
python310Packages.certbot-dns-route53
python39Packages.certbot
python39Packages.certbot-dns-cloudflare
python39Packages.certbot-dns-google
python39Packages.certbot-dns-rfc2136
python39Packages.certbot-dns-route53

2 packages built:

python310Packages.gnureadline
python39Packages.gnureadline


#704 Fix  bug with deserializing model

Issue #, if available:
#704
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Fixing as suggested by @sxjscience .

Really appreciate your contribution! Is it possible to also add a test case in the file: https://github.com/awslabs/autogluon/blob/master/text/tests/unittests/test_nlp_text_prediction.py?

Job PR-708-1 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-708/1/index.html

Really appreciate your contribution! Is it possible to also add a test case in the file: https://github.com/awslabs/autogluon/blob/master/text/tests/unittests/test_nlp_text_prediction.py?

I added a small test

Job PR-708-3 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-708/3/index.html

Job PR-708-4 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-708/4/index.html


How to know when the dragging is finished?

Currently I am using
mDragSelectionProcessor = new DragSelectionProcessor(new DragSelectionProcessor.ISelectionHandler() {
           @Override
           public HashSet<Integer> getSelection() {
               return mAdapter.getSelection();
           }

           @Override
           public boolean isSelected(int index) {
               return mAdapter.getSelection().contains(index);
           }

           @Override
           public void updateSelection(int start, int end, boolean isSelected, boolean calledFromOnStart) {
               mAdapter.selectRange(start, end, isSelected);
           }
       }).withMode(mMode);

I also tried
mDragSelectionProcessor  = new DragSelectTouchListener.OnAdvancedDragSelectListener()
        {
            @Override
            public void onSelectChange(int start, int end, boolean isSelected) {
                // update your selection
                // range is inclusive start/end positions
                mAdapter.selectRange(start, end, isSelected);
            }

            @Override
            public void onSelectionStarted(int start) {
                // drag selection was started at index start
                Log.e("michaelflisar",start+" Start");
            }

            @Override
            public void onSelectionFinished(int end) {
                Log.e("michaelflisar",end+" End");
                // drag selection was finished at index start
            }
        };

but on using OnAdvancedDragSelectListener() the first cell is not selecting in the sample app you provided

The DragSelectionProcessor is handling all the logic, the listener is just listening for events. I made the four modes for the processor so that all cases of desired logics is implemented.
If you want to use the listeners directly you need to manually implement the logic (tracking first item, handling dragging over an item or region and also dragging back). This logic is all done by the DragSelectionProcessor

Actually I just want to track when the user released the recyclerview
So i can update my alert window

Then try using this: https://github.com/MFlisar/DragSelectRecyclerView/blob/master/library/src/main/java/com/michaelflisar/dragselectrecyclerview/DragSelectionProcessor.java#L66

 .withMode(mMode).withStartFinishedListener(new DragSelectionProcessor.ISelectionStartFinishedListener() {
                    @Override
                    public void onSelectionStarted(int i, boolean b) {

                    }

                    @Override
                    public void onSelectionFinished(int i) {

                    }
                });

Above code helped me to track dragging start and finished state

Glad I could help


Button to swap secondary & tertiary colour

Is your feature request related to a problem? Please describe.
I'm making mock-ups and hesitating which colour to use as 'secondary' and which as 'tertiary' colour. Switching the Secondary & Tertiary colours (in order to see them in the example components) is a bit of a copy-and-paste pain.
Describe the solution you'd like
A simple button (e.g. with down/up arrows) to swap/switch the colours.
Describe alternatives you've considered
None.
Additional context
None.

Thank you for the feedback @keunes
Would you want to do the same between primary? If so, would you still expect primary to act as the source color for the rest of the color scheme?

added to feature requests log, closing issue


Clarify consent and configuration frames

Refs: https://openscience.atlassian.net/browse/LEI-
Companion to:
Purpose
Make it easier for parents to collect video we'll be able to include in the study; reduce "noreading" consent videos and dark/off-center videos
Summary of changes
Change layout of consent frame and add clear numbers
Require reloading on exp-video-config frame and add troubleshooting tips
Add exp-video-config-quality frame with checklist to collect good video
Testing notes
Can use 5813bd253de08a007fc59507 on staging to test. Expected sequence at start of a study would actually be

"sequence": [
"video-config",
"video-consent",
"video-quality",
"exit-survey"
]

Also: this adds a new frame, and doesn't change the data expected by the server.
I'm fairly certain it could be added to an existing experiment on the staging server to confirm this works as intended, and then edited into the production piloting study as well if that would make your life easier.

That would be great! I can try adding it to an existing experiment on staging.


vertical-stack-in-card: no columns in view possible

When I add several vertical-stack-in-card to a view, they are all sorted into one single column. If I use the "classic" vertical-stack-card, the stacks are arranged in columns, just like single cards.

Fixed, good job!


need instructions on what needs editing to make the website work for the new repo

as it is, after following the instructinos,
the title of index.html is still ashlar, and github links to the original repo,
not the new repo...
can we add instructions to say what needs changing to make the website work?

seems the main thing I needed to change is
include/before_body.html

See the new instructions in the readme:
https://github.com/jhsiao999/ashlar


Dashboard

User Story
As a Product Owner,
I want to know how Follow The Learners is performing on various measures,
So that I can make sure it is functioning the way it should be and how I might be able to optimize it in the future.

This will be rolled into FTL Admin.


Error with backup script

Using Lokl.  Been great so far.
Just tried to backup sites for the first time and encountered an error.
To produce the error: I started Lokl, inputted m for manage, Lokl detected my sites and I entered the number of the chosen site,  then entered t for backup.
Any idea as to how I can get the backup script to run?
https://www.loom.com/share/267a70dc8b4146469df5d66c555917af

Hi @tjc4 - thanks for trying it out - cool to hear of some usage!
Hmm, I haven't tried under MinGW before. I believe it's been used in an earlier version (without backup functionality) using Git bash or WSL.
It's trying to store the backup file that it generates inside the Lokl instance (Docker container) into your host OS (Windows in this case)'s /tmp/ directory.
From a quick search, there's a chance there isn't this /tmp dir within MinGW. You could try to manually add it with something like:
mkdir /tmp
And then re-run the Lokl backup. I don't have a Windows handy to test that right now, but please let me know how that goes.

Looking at your video again, it seems to fail to find the generated backup file within the container itself, as if it weren't generated...
What we can do to debug, is to run the parts of the Lokl/Go script manually within the container, let me dig out the bits.
If you start by using the SSH option within Lokl to connect to the container, then...

This is the code within the Lokl/Go script which starts the backup:
https://github.com/lokl-dev/go/blob/687527c331c9335f090547f17adaceba8ea48761/go.sh#L329
And this is the script that it runs within the container itself to generate the backup, which seems to be failing in our case:
https://github.com/lokl-dev/lokl/blob/master/scripts/backup_site.sh
If you're able to step through those and see where it fails, that would help, but I understand this isn't the easiest process!

@leonstafford thanks for the quick reply!
I tried creating a tmp folder but was told the folder already existed.

I tried the take_site_backup at the "choose action to performn" screen.  I couldn't tell if anything happened or it worked.  How would I check?

As far as stepping through things to see where it fails, I'm happy to try to help but would need a bit of hand holding.
Let me know if there's anything else you'd like me to try.  Thanks!


Add more Private Send minimum protocol version checking

Check nVersion before Relay out PrivateSend Messages
Check nVersion after create new connection to Masternode for Private Send processing.

utACK


[FLINK-32019][Connector/Kafka]  EARLIEST offset strategy for partitions discoveried later based on FLIP-288

What is the purpose of the change
As described in [FLIP-288](https://cwiki.apache.org/confluence/display/FLINK/FLIP-288%3A+Enable+Dynamic+Partition+Discovery+by+Default+in+Kafka+Source), the strategy used for new partitions is the same as the initial offset strategy, which is not reasonable.
According to the semantics, if the startup strategy is latest, the consumed data should include all data from the moment of startup, which also includes all messages from new created partitions. However, the latest strategy currently maybe used for new partitions, leading to the loss of some data (thinking a new partition is created and might be discovered by Kafka source several minutes later, and the message produced into the partition within the gap might be dropped if we use for example "latest" as the initial offset strategy).if the data from all new partitions is not read, it does not meet the user's expectations.
Other ploblems see final Section: User specifies OffsetsInitializer for new partition .
Therefore,  it’s better to provide an EARLIEST strategy for later discovered partitions.
Brief change log

Expand KafkaSourceEnumState with TopicPartitionWithAssignStatus to distinguish between initial partitions and newly discovered partitions. TopicPartitionWithAssignStatus is also better for future expansion, as new statuses can be added without changing the state results.
Add a newDiscoveryOffsetsInitializer(EARLIEST) to get offsets for newly discovered partitions.
Modify kafkaSourceEnumStateSerializer to handle the expanded KafkaSourceEnumState.

Verifying this change

Test the backward compatibility of state when deserializing in KafkaSourceEnumStateSerializerTest.
Expand KafkaEnumeratorTest#testSnapshotState method to test snapshot state in more scenarios:

Before first discovery, so the state should be empty
First partition discovery after start, but no assignments to readers
Assign partials partitions to readers
Assign all partitions to readers

Expand KafkaEnumeratorTest#testDiscoverPartitionsPeriodically method to test whether new partitions use EARLIEST offset while initial partitions use specified offset strategy.

Now that @RamanVerma is busy, could anyone else help me? CC, @PatrickRen

@PatrickRen CC


Releases/Versioning mechanism

This action will automate the tagging and releases when the master branch is updated
Note you will need to expose a github personal access token in the project secrets, this cannot be seem by people but the publish will require the action. @anthdm
add a secret named: GITHUB_PERSONAL_ACCESS_TOKEN and give it the access token < the token should not need very many permissions
Changes:

new github action
goreleaser config < just tells the releaser that this is a library and there is no main package essentially

@rekram1-node  Thanks!

Merging this soon boys! Thanks


Allow fmt arg to printf to be an array of i8 in non-constant space

If SPV_EXT_relaxed_printf_string_address_space is enabled llvm-spirv allows the format argument to be in Generic space.  Also allow format argument to point to an array of i8 instead of only an i8.

Hi @alan-baker.  Could you please review these changes?  Thanks.


mutable variables should not be the default parameter

Describe the bug
The problem arises from the code below:
@BACKBONES.register_module()
class BEiT(nn.Module):
""" Vision Transformer with support for patch or hybrid CNN input stage
"""
def init(self, img_size=224, patch_size=16, in_chans=3, num_classes=80, embed_dim=768, depth=12,
num_heads=12, mlp_ratio=4., qkv_bias=False, qk_scale=None, drop_rate=0., attn_drop_rate=0.,
drop_path_rate=0., hybrid_backbone=None, norm_layer=None, init_values=None, use_checkpoint=False,
use_abs_pos_emb=True, use_rel_pos_bias=False, use_shared_rel_pos_bias=False,
out_indices=[3, 5, 7, 11]):
super().init()
Expected behavior
mutable variables should not be the default parameter

if somewhere the out_indices list (a mutable object) is modified (e.g. out_indices.append(13) ), in subsequent calls to the function the default value for out_indices will be e.g. [3, 5, 7, 11, 13] instead of [3, 5, 7, 11] as originally intended.
relevant snippet from the python docs:

Default parameter values are evaluated from left to right when the function definition is executed. This means that the expression is evaluated once, when the function is defined, and that the same “pre-computed” value is used for each call. This is especially important to understand when a default parameter value is a mutable object, such as a list or a dictionary: if the function modifies the object (e.g. by appending an item to a list), the default parameter value is in effect modified. This is generally not what was intended.

https://docs.python.org/3/reference/compound_stmts.html#function-definitions


Error: Illegal Parameter specification! with Tesseract4Alpha

After upgrade to Tesseract-4-Alpha, I found this error making the OCR from my JAVA code:
ITesseract instance = new Tesseract();  instance.setDatapath("/usr/share/tessdata/");  instance.setLanguage("spa");  (...) result = instance.doOCR(imageFile);

Environment

Tesseract Version: tesseract 4.00.00alpha
Leptonica Version: leptonica-1.74.4
Platform: CentOS 6.7
Server: Wildfly 10.1

Current Behavior:
Error: Illegal Parameter specification!
"Fatal error encountered!" == NULL:Error:Assert failed:in file globaloc.cpp, line 75
A fatal error has been detected by the Java Runtime Environment:
SIGSEGV (0xb) at pc=0x00007ff1b3098549, pid=25091, tid=0x00007ff29d7d7700
JRE version: OpenJDK Runtime Environment (8.0_121-b13) (build 1.8.0_121-b13)
Java VM: OpenJDK 64-Bit Server VM (25.121-b13 mixed mode linux-amd64 compressed oops)
Problematic frame:
C  [libtesseract.so+0x26f549]  ERRCODE::error(char const*, TessErrorLogCode, char const*, ...) const+0x129
Failed to write core dump. Core dumps have been disabled. To enable core dumping, try "ulimit -c unlimited" before starting Java again
An error report file with more information is saved as:
/opt/wildfly/wildfly-10.1.0.Final/hs_err_pid25091.log
If you would like to submit a bug report, please visit:
http://bugreport.java.com/bugreport/crash.jsp
The crash happened outside the Java Virtual Machine in native code.
See problematic frame for where to report the bug.
*** JBossAS process (25091) received ABRT signal ***
Suggested Fix:
Any idea?

Please use the latest source from master branch of github and inform whether you still get the error.

I'm using the lastest source yet. I have the same Error (*) in two different OS.
(*) Error: Illegal Parameter specification!
"Fatal error encountered!" == NULL:Error:Assert failed:in file globaloc.cpp, line 75

what version of c++ are you using?

two different OS

which ones? I have been able to build on ubuntu 14.04. Travis and appveyor builds are building ok.

Also, are you able to run tesseract from command line ?
tesseract -v
also try to OCR the sample image from testing folder.

I'm using g++ 7.1.1 in Arch and 4.8.2 in CentOS 6.7.
I launch tesseract from Java. No problems with 3.05 version but I get the error previously commented with 4Alpha version.
tesseract 4.00.00alpha
 leptonica-1.74.4
 libgif 5.1.4 : libjpeg 8d (libjpeg-turbo 1.5.1) : libpng 1.6.29 : libtiff 4.0.8 : zlib 1.2.11 : libwebp 0.6.0

what about
tesseract --list_langs
Are you able to OCR an image from command line?
Do you have 4.00.00alpha version of traineddata files? It is possible that you have the 3.04 version of traineddata, hence 3.05 works fine.
Download 4.0 traineddata to a different folder and refer to that

No problems detecting langs with tesseract --list_langs (eng, spa and osd trainned files for LSTM based 4.00.00alpha version).
About command line recognition, I have done fine an example from testing folder properly.
Perhaps, some Java code has changed from this 4Alpha version?

@nguyenq Quan, Any suggestions regarding running tesseract 4.0 from java?

@nachobit Please see Quan's Java JNA wrapper for Tesseract OCR API at https://github.com/nguyenq/tess4j

The problem with 3.05.01 version is that I get different resutls from both OS using same Leptonica and Tesseract ver.
Example:
0000 0340 º71º ZL (in CentOS) and 0000 0340 0710 ZL (in Arch).
For that reason I'd like to improve the 4Alpha but it's impossible for the error commented some lines back.

If you have an issue with a wrapper to Tesseract's C/C++ API, please report the issue to the developers of that software.

0000 0340 º71º ZL (in CentOS) and 0000 0340 0710 ZL (in Arch).

Ray said, many years ago, that you can get different results with different compilers.

Perhaps, some Java code has changed from this 4Alpha version?

Yes.
https://github.com/nguyenq/tess4j/commits/master

Updated to the lastest libs from Tess4J-3.4.0-src I get same error when launch the OCR from Java code.
From 3.05.01 version, is there any solution to solve the fail recognizing "zeros" ( º instead of 0)?

3.4.0 does not include the 4.00 changes.
https://github.com/nguyenq/tess4j/commits/tess4j-3.4.0

From 3.05.01 version, is there any solution to solve the fail recognizing "zeros" ( º instead of 0)?

You can try to compile with a newer version of gcc. I can't promise that this 'solution' will help you with this issue.

Ok, that's the problem? Tess4J-3.4.0 (Java) is not supported by 4.00Alpha release? Then I will try compilling with a newer version of GCC.

Ok, that's the problem? Tess4J-3.4.0 (Java) is not supported by 4.00Alpha release?

I assume that's the source of the problem. To be sure, ask the developer.
https://sourceforge.net/p/tess4j/discussion/1202294/
https://github.com/nguyenq/tess4j/issues

https://github.com/nguyenq/tess4j/commit/74c850980af1268c100f8798ed475e6d69fe34b8
+Version 4.0.0 beta (8 June 2017)
+- Upgrade to Tesseract 4.0.0 alpha (8c29e68)
+- Update Lept4J to 1.5.0 (Leptonica 1.74.2)
ShreeDevi

भजन - कीर्तन - आरती @ http://bhajans.ramparivar.com
On Thu, Jun 29, 2017 at 3:40 PM, Nacho Romero<EMAIL_ADDRESS>wrote:

Updated to the lastest libs from Tess4J-3.4.0-src I get same error when
launch the OCR from Java code.
From 3.05.01 version, is there any solution to solve the fail recognizing
"zeros" ( º instead of 0)?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tesseract-ocr/tesseract/issues/1010#issuecomment-311923010,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE2_oxFIpUnAMuY0-FEdUVI_o0dR3pxUks5sI3f8gaJpZM4OH3Lp
.

tess4j's master branch is for Tesseract 4.0alpha and includes the latest Tesseract 4.0alpha Windows binary up to revision 8c29e68. All of its unit tests passed on Windows 10. We have not tested on Linux OS yet.

We do not support 3rd party sw including tesseract wrapper. Please reproduce error with c++

Hi .. it seems you just have to add environment variable LC_NUMERIC="C" ... and it works. :)

I dug into Tesseract's code and found that the string "Illegal Parameter specification" only exists in one place, namely in the file classify/clusttool.cpp. After some debugging I realised that the function ReadParamDesc() calls sscanf() at line 82 (for git commit hash 2b854e3749d62012787dd4160fc30e86603cc540), which is locale dependent. It fails since the numeric input (two floating point values) are written with dots (example: 1.23), but using a different locale other than en_US for LC_NUMERIC may cause sscanf() to expect other characters, like commas (1,23).
With other words, the error is in tesseract, assuming a locale. It should rather be set explicitly.

https://github.com/tesseract-ocr/tesseract/wiki/FAQ#error-illegal-min-or-max-specification
https://web.archive.org/web/20150510151209/http://code.google.com/p/tesseract-ocr/issues/detail?id=228
https://web.archive.org/web/20150509203443/http://code.google.com/p/tesseract-ocr/issues/detail?id=250
https://stackoverflow.com/questions/13919817/sscanf-and-locales-how-does-one-really-parse-things-like-3-14

i am facing the same issue.Please could u share the file which has to be changed, so that i can jus go replace the specific file and continue creating the traineddata.

@nizzeberra : i see the same files as you say, but dont know where to place the code. Please could you share that file.

@stweil Is it possible to address this for final 4.0.0?

Setting LC_NUMERIC in the Tesseract code would perhaps solve the problem, but is not a good solution for people who use the Tesseract library. They don't expect that Tesseract changes LC_NUMERIC, and perhaps they need a different value.
I wonder whether the sscanf handling of %f does really depend on the locale settings. It does not on my Debian GNU Linux system, nor could I find a hint in the MSDN documentation on sscanf.
@nizzeberra, which systems / C libraries show that strange behaviour? Do you have links to documentation?

Here is the man page, and it's pretty clear about the locale:
http://man7.org/linux/man-pages/man3/scanf.3.html

Linux Mint uses Debian packages, so the result should not be much different. The man page only says that LC_NUMERIC can be used to allow separators for multiples of thousand.
Here is the test scenario which I used (maybe you can try it on Linux Mint):
$ cat sscanf-test.cpp
#include <stdio.h>

int main(int argc, char *argv[])
{
  for (int arg = 1; arg < argc; arg++) {
    float f = 0.0f;
    sscanf(argv[arg], "%f", &f);
    printf("f[%d] = %f\n", arg, f);
  }
  return 0;
}
$ g++ -std=c++11 -Wall -Wextra sscanf-test.cpp -o sscanf-test
$ ./sscanf-test 3.14
f[1] = 3.140000
$ ./sscanf-test 3,14
f[1] = 3.000000
$ LC_NUMERIC=de_DE.UTF-8 ./sscanf-test 3,14
f[1] = 3.000000

https://en.cppreference.com/w/cpp/locale/setlocale
Here 3.14 -> 3,14

That's interesting. So C/C++ programs don't use the locale which was set in the environment, but start running with the "C" locale. That is exactly what I observed in my test. Only if I set LC_NUMERIC inside of my test program, I get a different behaviour.
That implies that we have no problem for the tesseract executable or the training programs which are provided by Tesseract. Nor will external software have a problem as long as it does not set LC_NUMERIC.
Maybe Java uses the environment settings to set LC_NUMERIC internally. That would explain the reported problem.
In addition to the problem with sscanf, more code is possibly affected by "wrong" locale settings, for example these lines:
classify/ocrfeatures.cpp:    fprintf (File, "%f  %f\n",
wordrec/params_model.cpp:    if (fprintf(fp, "%s %f\n", kParamsTrainingFeatureTypeName[i], weights[i])

@Shreeshrii, I don't think that all that code can be found and rewritten for 4.0.0. It would be possible t o report a warning when the Tesseract initialisation code detects an unsupported locale setting.

@stweil Thanks for the investigation. Yes, please make possible changes to point the users in the right direction.
Related issue  reg: locales.
https://github.com/tesseract-ocr/tesseract/issues/1250#issuecomment-377347837

sorry to ask about this in detail, i have no expertise in this.
i am trying to use this command... cntraining eng.TimesNewRoman.exp0.tr how do i make this command work. can use please tell me where exactly do i need to make changes to set the locale differently? Please do help me out.
Regards

Hi , Tesseract is working fine in main method(JAVA), but when i try to run in web application i am facing below error
**#
A fatal error has been detected by the Java Runtime Environment:

SIGSEGV (0xb) at pc=0x00007f96274dbac7, pid=4516, tid=0x00007f9699212700

JRE version: OpenJDK Runtime Environment (8.0_171-b11) (build 1.8.0_171-8u171-b11-0ubuntu<IP_ADDRESS>-b11)
Java VM: OpenJDK 64-Bit Server VM (25.171-b11 mixed mode linux-amd64 compressed oops)
Problematic frame:
C  [libtesseract.so.3.0.4+0x9dac7]  tesseract::Tesseract::recog_all_words(PAGE_RES*, ETEXT_DESC*, TBOX const*, char const*, int)+0x5e7

Failed to write core dump. Core dumps have been disabled. To enable core dumping, try "ulimit -c unlimited" before starting Java again

An error report file with more information is saved as:
/home/ahextech/suresh/softwares/eclipse/hs_err_pid4516.log

If you would like to submit a bug report, please visit:
http://bugreport.java.com/bugreport/crash.jsp
The crash happened outside the Java Virtual Machine in native code.
See problematic frame for where to report the bug.
#**
My sample code is:
Tesseract tesseract = new Tesseract();
TesseractOCRConfig config = new TesseractOCRConfig(); 				config.setTessdataPath("/home/ahextech/suresh/ehubWorkspace/BITBUCKETCODE/");
config.setLanguage("Eng");
String text = tesseract.doOCR(imageProcessed);
My current Os : Ubuntu 16.04
Java JDK version 1.8
My Tomcat version : 8
Tessercat Version :3.4.4
Help me if u can
Thanks a Ton..!!!


Variables SPL_BINARY, UBOOT_INITIAL_ENV and UBOOT_SUFFIX are unknown

Hi,
we're writing a custom recipe for U-Boot and use both the uboot-config.bbclass as well as u-boot.inc from Poky.
We set the SPL_BINARY as well as the UBOOT_SUFFIX variable in our recipe. They're both used in uboot-config.bbclass and documented but we get an OELint error that those variable are unknown. Should they be added to the known variables?
We're also setting the UBOOT_INITIAL_ENV variable which is used in u-boot.inc but we also get an OELint error that this variable is unknown. Should it be added to the known variables, too? Not sure about this here, because it's not documented.
Thanks for investigating.

The latest release of oelint-adv names both variables for kirkstone release and up.
If you are using an older release you will need to add
--constantmods=+my-variables.json  to the CLI and have a json file my-variables.json  with the following content available
{
  "variables": {
  "known": ["SPL_BINARY", "UBOOT_SUFFIX", "UBOOT_INITIAL_ENV"]
}
}

And btw it's not an error, but barely an info message that's being reported ;-)


chore: stop publishing @prisma/migrate package

It's bundled in prisma CLI and not needed as an independent package.

We should probably deprecate the package on Npm as well to make clear for the old releases and versions that they are not to be used.

@janpio yes though it was "never" used/mentioned I think.

That will not hinder people to try :D


Query Builder method limit() not allow 0 values

CI: 3.1.9
dbdriver: sqlite3
$this->db->where ( 'ag', $kat );
$this->db->limit($limit, $offset);
$this->db->order_by('sort', 'ASC');
$query = $this->db->get("artikel");

In CI Version 3.1.9
$this->db->limit(0, 0);
limit does not return results if the first parameter is 0.
With values other than 0 a result is returned.
In CI Version 3.1.8
$this->db->limit(0, 0);
it worked.

Quite the opposite; limit(0) was ignored before 3.1.9 ... See #5471. Whatever you were trying to do with that, it was pointless.

My mistake. I thought 0 = false = no limit and that worked in version 3.1.8.
Two bugs that have mutually fixed each other ;-)


lef.tab.h: No such file or directory.

When running make, a "No such file or directory" error is reported for lef.tab.h.
https://github.com/abk-openroad/RePlAce/blob/73cb0d39557ef30df3316b3a9f456622857c71c0/module/lef/5.8-p027/lef/lefrSettings.cpp#L34
A possible mitigation to this issue is to ignore the error.

Solved in commit: https://github.com/abk-openroad/RePlAce/commit/c77d24f07820ae109c59206a7cb998f8b99f1a94
closed


#31 Convert module to PDK

Travis CI checks seems to be failing - check https://travis-ci.org/github/saltedsignal/puppet-certmonger/pull_requests

Travis CI checks seems to be failing - check https://travis-ci.org/github/saltedsignal/puppet-certmonger/pull_requests

@raildo any chance you could take a look at that? ^^

Sure, I'll take a look at this soon.

Checking the job result, I believe that all the issues look to be related to ruby lint issues. Do we care about ruby lint on this project? If so, I suggest that we can create a parallel issue to have all these lint problems fixed and unblock the CI result.

@raildo thanks for looking into this. Converting the module to PDK isn't urgent, so lets not merge until the PR is ready with all checks (including ruby lint checks) passing. No need for a a parallel issue.

@raildo thanks for looking into this. Converting the module to PDK isn't urgent, so lets not merge until the PR is ready with all checks (including ruby lint checks) passing. No need for a a parallel issue.


Document index.query.max_nested_depth fix #6346

Description
Document index.query.max_nested_depth setting.
Issues Resolved
Closes #6346
Checklist

[x] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Thank you, @mkhludnev! A small change and this PR will be ready to merge.
Appreciate!

Is this for 2.12?
depends on https://github.com/opensearch-project/OpenSearch/pull/11670

Should "20" and "1" be in code font? Otherwise, LGTM.

Fully agree. But neither is numbers around follow this convention.

@kolchfa-aws this is going to land in 2.13.0

Still little bit puzzled.

here's backport PR https://github.com/opensearch-project/OpenSearch/pull/12286 with MAX_VAL toward 2.13 and in this PR we are going to document it. It's clear. Got it.
where we are going to document main 3.x change with default 20 https://github.com/opensearch-project/OpenSearch/pull/11670 ?

Done with applying suggestion for 2.13. Still not clear how we remember to change documentation for default for 3.x corresponding to opensearch-project/OpenSearch#11670?

Thanks, @mkhludnev ! You can create a new documentation issue that describes the changes for 3.0 and points to the change in functionality, or go straight to a PR. We will label it as '3.0'.

Great @hdhalter. After this is merged I spin 3.0 PR.

@mkhludnev Is this PR ready to merge for 2.13?


[ENHANCEMENT] Better internal batching for images inference

Is your feature request related to a problem? Please describe.
Currently batching is effectively performed over text based fields (due to the internal splitting creating batches) but for images this not the case. This means there is probably up to a 2x improvement for throughput that could be gained by better batching of images for inference.
Describe the solution you'd like
Batching of images for inference.
Describe alternatives you've considered
Batching at the model level via an inference server. However the performance of earlier tests for this was that although batching at the model worked reasonably well, the overall throughput was worse.

@pandu-k  Thinking a bit more lazy calls to inference might be the best way to go without changing much code then running inference in batches for the respective image and text.


Missing kafka.producer and kafka.consumer metrics

What changed, and why it matters
Migrated https://help.aiven.io/en/articles/5088895-missing-kafka-datadog-metrics
Fixes #1158

I made a fairly major edit to remove some of the content about what we are not doing, just left in the bits that we are doing, and where to go to get the rest. @ftisiot check it still seems OK to you and then I'll approve if you're happy

Looks very ok!


Warning: An update to AuthProvider inside a test was not wrapped in act(...).

We have a component that wraps the AuthProvider with the settings and the callback functions, and when writing tests for the component we started to see these errors in the console:

It happens even when writing a simple test like this:
  it("tests", () => {
    const settings: AuthProviderNoUserManagerProps = {
      authority: "authority",
      client_id: "client_id",
      redirect_uri: "callback",
    };

    render(<AuthProvider {...settings}></AuthProvider>);
  });

Cannot find a way to drop that message, does not work by wrapping - unnecessarily - the render in act(), or cannot follow any help I found since it (https://kentcdodds.com/blog/fix-the-not-wrapped-in-act-warning)
React: v18.2.0
Jest: v27.5.1

I have the exact same problem right now, would really like a solution to this as well.

I managed to resolve the issue by creating a promise and awaiting it with an act
  it("tests", async () => {
    const promise = Promise.resolve();
    const settings: AuthProviderNoUserManagerProps = {
      authority: "authority",
      client_id: "client_id",
      redirect_uri: "callback",
    };

    render(<AuthProvider {...settings}></AuthProvider>);

    await act(async () => {
      await promise;
    });
  });


BibTeX batch load - gracefully handle DOI (and other) parse errors

Failed to parse BibTeX on value "doi" (NAME) ["@", #<BibTeX::Entry >

@typophyllum I think you raised this error playing with the batch loader?  If so can you email me the bibtex text file?

No need to be sorry- the opposite, finding errors means we are sorry. ;)
On Mon, Nov 5, 2018 at 11:28 AM Holger Braun<EMAIL_ADDRESS>wrote:

Could be, I was experimenting with "New source from BibTeX". One was a
quite large text, with keywords and other stuff, but don't remember from
where I copied it. And I also tried to enter simply a DOI. Sorry.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/SpeciesFileGroup/taxonworks/issues/627#issuecomment-435961367,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABYSAmvmI2OEU8DcfcbSTcglGig1Df0ks5usHUwgaJpZM4YOydB
.


CSV BOM stripping is now handled internally by league/csv

league/csv no longer exposes stripBom so we can't use it.
The stripping of BOMs is now handled internally in the library.

Acknowledging there are still some failures, but they’re related to PostgreSQL and PHPUnit 7 + PHP 7.4. This clears up a large number of other failures, so merging


Add script for computing SHA256 of tars of our kubectl plugin

Signed-off-by: Haoxiang Zhou<EMAIL_ADDRESS>
What this PR does / why we need it:
This script make it easier to manually update the our plugin manifest in krew-index. It downloads the .tar files for different platforms and outputs the SHA256.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

NONE

/lgtm
/approve
/kind cleanup


[Backport 1.3] Update jackson to 2.13.4 (#557)

Update jackson to 2.13.4

Signed-off-by: Siddhant Deshmukh<EMAIL_ADDRESS>
Get jackson version from core

Signed-off-by: Siddhant Deshmukh<EMAIL_ADDRESS>Signed-off-by: Siddhant Deshmukh<EMAIL_ADDRESS>(cherry picked from commit fe8bf44efc2bb154c03c62590bd96a358ed34b35)
Signed-off-by: Siddhant Deshmukh<EMAIL_ADDRESS>Issue #557, if available:
Description of changes:
CheckList:

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #563 (2ca1e24) into 1.3 (b72eeaa) will increase coverage by 0.05%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                1.3     #563      +/-   ##
============================================
+ Coverage     76.72%   76.77%   +0.05%
  Complexity     2072     2072
============================================
  Files           254      254
  Lines         11770    11770
  Branches       1830     1830
============================================
+ Hits           9030     9036       +6
+ Misses         1743     1740       -3
+ Partials        997      994       -3

Impacted Files
Coverage Δ

...anagement/indexstatemanagement/model/Transition.kt
63.01% <0.00%> (-2.74%)
:arrow_down:

.../opensearch/indexmanagement/rollup/model/Rollup.kt
86.04% <0.00%> (ø)

...arch/indexmanagement/rollup/RollupSearchService.kt
61.11% <0.00%> (ø)

...earch/indexmanagement/transform/model/Transform.kt
85.89% <0.00%> (+0.42%)
:arrow_up:

...nt/indexstatemanagement/ManagedIndexCoordinator.kt
68.29% <0.00%> (+0.57%)
:arrow_up:

...management/indexstatemanagement/MetadataService.kt
65.67% <0.00%> (+2.23%)
:arrow_up:

.../rollup/action/start/TransportStartRollupAction.kt
75.29% <0.00%> (+2.35%)
:arrow_up:

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


Pipeline stalls after genomicsdbimport

Description of the bug
When running a multi family run, the pipeline stalls after genomicsdbimport until all families have been run through genomicsdbimport
Command used and terminal output
No response
Relevant files
No response
System information
dev version on the 4th of January 2023

Doesn't seem to be stalling but just skipping after genomicsdbimport. Only had this happen once on the HPC...

Possible fixed in #42

I was able to recreate the issue using the stub run on the HPC :partying_face:

Issue found: It was a meta id parsing error in the BED_SCATTER_GROOVY subwf. It couldn't handle the . present in the family names. Should be fixed in #44


Update font

Before:

After:

This is likely a temporary change. I like the Serif font for the headings but I think Sans-Serif is better for the body


In parsing methods, check for default(EncodingData) and default(TextFormat)

PrimitiveParser's main overloads - those which take EncodingData and TextFormat objects as parameters - are currently failing when given the default values for those objects. This change introduces better behavior. If default(EncodingData) is passed, the parser will default to InvariantUtf8 parsing. If default(TextFormat) is passed, the parser will not interpret the format as having precision, as was the case previously.
This is a fix for Issue #1106.

Looks good.


[SPARK-37205][FOLLOWUP] Should call non-static setTokensConf method

What changes were proposed in this pull request?

This fixes a bug in the original SPARK-37205 PR, where we treat the method setTokensConf as a static method, but it should be non-static instead.
Why are the changes needed?

The method setTokensConf is non-static so the current code will fail:
06/29/2022 - 17:28:16  - Exception in thread "main" java.lang.IllegalArgumentException: object is not an instance of declaring class
06/29/2022 - 17:28:16  - at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
06/29/2022 - 17:28:16  - at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
06/29/2022 - 17:28:16  - at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
06/29/2022 - 17:28:16  - at java.base/java.lang.reflect.Method.invoke(Method.java:566)

Does this PR introduce any user-facing change?

No
How was this patch tested?

Manually tested this change internally and it now works.

cc @dongjoon-hyun @viirya for review

Yes, I'll commit it to both 3.3 and master.

Merged to master/3.3, thanks!

Sorry for being late, @sunchao . BTW, please consider like a release manager.

Unfortunately, this PR is completely lost in term of JIRA information and mislead the users.
3.3.0 is already shipped and this PR is only fix the issue at 3.3.1, isn't it?
In short, we should not use FOLLOWUP PR after release because of the fixed version information should be different.

@dongjoon-hyun you are right, I forgot the fixed version for this is different now. Should I revert the commit, create a new JIRA and link this PR to it?

You can follow the best practice next time. That will be enough. There exist multiple instances like this.


Theme breaks tab background

After installing theme through hpm and hitting Cmd+Shft+R (Full reload), everything seems fine. However, after quitting Hyper, and firing it up again, tabs are transparent.

HyperTerm keeps changing a lot with every update and it just pushed and update that broke the theme.
I'll fix it with an update asap. ✌

Just pushed an update that should fix every issue. Let me know if I've missed something. 👋

@henrikdahl I just install it and I have the same error. Last version of Hyper, and this plugins used :
  plugins: [
    "hyperterm-1password",
    "hyperterm-tab-icons",
    "hyperterm-alternatescroll",
    "hyperlinks",
    "hyper-statusline",
    "hyperterm-base16-ocean-saturated"
  ],

Thank you for your work !

@Florian95 Try commenting out the theme and launch Hyper, then uncomment it and launch Hyper again. Does it solve it?
Otherwise, what's your Hyper version and OS?

It's good only in the same window where I uncomment the theme. If I launch a new one, nothing...
MacOS 10.12.1
Hyper 1.0.1 (<IP_ADDRESS>9)

@henrikdahl Any news about this bug ? How can I help ?
Thank you.

@Florian95 sorry, didn't know it was still an issue.
What's your plugin list? ✌

  plugins: [
    "hyperterm-1password",
    "hyperterm-tab-icons",
    "hyperterm-alternatescroll",
    "hyperlinks",
    "hyperterm-base16-ocean-saturated"
  ],

@Florian95
Name was changed to hyper-oceans16 to match the new plugin naming convention of Hyper. Additionally the tabs were separated into its own plugin for the ability to use them with any theme https://github.com/henrikdahl/hyper-tabs-enhanced
Would you mind trying these to see if it solves it for you? ✌


Increase padding and font size for tile checkbox

Related: https://github.com/18F/identity-idp/pull/5279#pullrequestreview-727676760
Slack discussion: https://gsa-tts.slack.com/archives/C01JVKYE4KD/p1628869289001500
Implements padding and font size for tile variants of checkbox and radio buttons to align to Figma designs.
Screenshots:

Before
After

Live Preview:

Before: https://design.login.gov/components/form-fields/#tile
After: https://federalist-2f194a10-945e-4413-be01-46ca6dae5358.app.cloud.gov/preview/18f/identity-style-guide/aduth-check-tile-padding/components/form-fields/#tile

Visually, the padding looks a little too big to me? But if it matches the designs that's good enough for me 👍

Yeah, there was some prior discussion about this on Slack. I'll defer to @anniehirshman-gsa or @cdelivuk on the final call, since maybe it makes a difference to see it "live". One thing I'll note is that it might appear a bit excessive in the examples only because there's not much supporting text, whereas in the real world we'll typically have a longer description.
For example:


Unit tests, integration tests and fuzzing

This would be a huge nice to have by now because:

We are reaching the point where accurate emulation matters. Automatically testing edge cases especially for some API drawing functions may be useful, e.g. to optimize code more aggressively while reducing the testing workload.
Test-driven development would make sense in some cases while implementing the API. String rendering, RNG, and some other things could benefit from this approach, in particular.
We could probably do some nice testing against PICO-8 by injecting a printh of the PICO-8 memory somewhere in update functions. This would allow detecting subtle bugs and differences in the implementation especially if performed automatically.
Fuzzing the API to detect differences in parameter count handling or argument type handling may unravel some inaccuracies in the interface implementation.

This raises a few issues:

CMake does not support cross-compilation for different architectures within the same build directory. Making two build directories is possible, but this would likely need some tinkering with VSC so that the CMake integration isn't dogcrap to use. If external development is ever opened up this has to be tackled. Ideally only one command should be run to compile for both arches and execute tests, at least as an option. Emulating ARMv6-M with qemu or something is an option but would be complicated to set up, would make instrumentation difficult, and is of limited use since recent bugs that were difficult to debug happened both on the Pico and on the desktop builds.
How to even execute tests? We could use CTest from CMake, but how about running the tests themselves? Integration tests might be easier to implement in pure Python, unit tests might be a lot easier to make using Catch or a similar library.

Closing as there is a test harness now


When training on my datasets , the loss =0.00000

I convert my datasets format  from voc to icdar2015. And then, i use icdar2015_to_tfrecord.py to convert them to tfrecord file. I use this file to train pixel_link, but loss always 0.000. What's wrong with it? Thanks.

When i convert voc to icdar2015, i set labels from '###' to anyother text, solved it.


Use the latest ERC-721 draft interface, fixes #745

Fixes #745
🚀 Description
The ERC-721 is near finalization. I have included the latest interfaces into here.
We are now working on interop testing and would love to work with the OpenZeppelin project on this.

[x] 📘 I've reviewed the OpenZeppelin Contributor Guidelines
[ ] ✅ I've added tests where applicable to test my new functionality.
[x] 📖 I've made sure that my contracts are well-documented. HELL YES
[ ] 🎨 I've run the JS/Solidity linters and fixed any issues (npm run lint:all:fix).

Quoting @spalladino from https://gitter.im/ETH-NFT/Lobby to move the discussion here.

For minor discussion on OpenZeppelin/zeppelin-solidity#777, would you rather keep the discussion here, or on the PR itself? For instance, I'd consider changing visibility of external methods to public (as in ERC20), or remove seemingly unnecessary payable modifiers.

I have made these requested changes.
Just wanted to note here that it is okay that OpenZeppelin's interface for ERC-721 is different than the canonical ERC-721 (external -> public, no payable). And it does not mean that the standard is deficient. The standard recognizes and makes allowance for this already.
Specifically, the standard is correct for the long term, future versions of Solidity. And this file in OZ is correct today.
Citing https://github.com/fulldecent/EIPs/blob/patch-2/EIPS/eip-721.md#caveats:

https://github.com/ethereum/solidity/issues/2330

Current versions of Solidity allow using public in an interface and disallow using external in an interface with public in a contract.
Future Solidity requires external in interfaces and allows contracts with public to implement those functions.

https://github.com/ethereum/solidity/issues/3412:

Current versions of Solidity require a payable interface to be implemented by a payable contract function.
Future versions of Solidity will allow a payable interface to be implemented by a default non-payable contract function.

I'm pointing this out to assert that the standard does regulate on the payability and non-payability of functions. And I continue to fight to make the Solidity interface grammar more expressive!

Thanks @fulldecent! I'll compare this interface with the candidate implementation from @facuspagnuolo at https://github.com/OpenZeppelin/zeppelin-solidity/pull/682, rework the implementation as needed, and revert back here if there is any strong mismatch.

Closing in favour of https://github.com/OpenZeppelin/zeppelin-solidity/pull/803. Note that this interface was included in that PR. Again, thanks for your work @fulldecent!


Add my first branch

I have created a new branch, created a file, and made a commit!

Add my first branch


Display the app notification

Hello sir,
I want to load bool value from my Godaddy SQL server in the background, even though the app is closed. If bool is true then I want to display a notification. Please anyone can help me with that

Hi @bhavikvashi1804
This platform is not meant for assistance on personal code. Please see https://flutter.dev/community for resources and asking questions like this, you may also get some help if you post it on Stack Overflow. Closing, as this isn't an issue with Flutter itself,
if you disagree please write in the comments and I will reopen it
Thank you


Implement auto-reconnect on server restart

After a restart of the orientdb server, the pyorient client can no longer communicate with it.
Right now our only resolution in this case is to restart our entire server (which uses pyorient as it's orientdb client) each time the orientdb server has to be restart. That much for availability.
Suggestion: when identifying connection problems, try to reconnect at least once. Only if the reconnection fails, raise a connection error. The price may be one lost message, which is way better then restarting the entire server (at least in our case).
Would love to hear your opinion.

update: eventually we decided to catch connectivity problems which appear on command executions (socket.error & PyOrientConnectionException) and create a whole new pyorient client context. This is not optimal, but works. In my opinion supporting it directly in the pyorient library would be better.

yep should be handled by your code by catching the exception


Code completion for expressions

Do you plan to add support for suggestions for expressions? Is something like that (type safety) even achievable with thymeleaf?
What I mean is if I type "${person.", the IDE would suggest firstName + lastName.

Has there been any progress in implementing this feature？

I think the biggest issue for that might be where it is supposed to get information about the available objects from.


Installation instructions for Debian 9 do not actually work

Specifically, the apt-installed packages list an older version of PHP than is required by composer.json (7.0 vs 7.3|7.4).

Were you able to find a fix for that?

Not in the amount of time I was willing to spend on the issue. I moved on to another product instead.

Which product did you move on to?

https://github.com/dani-garcia/bitwarden_rs


Java support

The following code seems to be only supported by Kotlin:

as the setupWithNavController call in java asks for a second parameter, which must be an AnimBuilder. But AnimBuilder seems not to be available for Java:

Is there any workaround in order to make it work with Java as well?
Thanks a lot in advance!

Found out we can use both Java and Kotlin in the same project :-)

Hi @srserx,
The AnimBuilder class is written in Kotlin, but it is still interoperable with Java so yes you can use the function like this:
StepperNavigationView stepper = findViewById(R.id.stepper);
NavController navController = ActivityKt.findNavController(this, R.id.frame_stepper);
stepper.setupWithNavController(
        navController,
        builder -> {
            builder.setEnter(android.R.anim.fade_in);
            builder.setExit(android.R.anim.fade_out);
            builder.setPopEnter(android.R.anim.fade_in);
            builder.setPopExit(android.R.anim.fade_out);
            return Unit.INSTANCE;
        }
);

I will add a @JvmOverloads annotation to the setupWithNavController function so you can use it with the default AnimBuilder like:
stepper.setupWithNavController(navController);


Cannot reshape tensor of 0 elements into shape [0, -1, 256, 256] because the unspecified dimension size -1 can be any value and is ambiguous

Hi! Both the web form and API calls to Replicate fail when I run this model: https://replicate.com/schananas/grounded_sam
Running prediction: 3341693a-59dd-4e9d-b89f-2ba89f3ab53c...
Traceback (most recent call last):
File "/root/.pyenv/versions/3.10.13/lib/python3.10/site-packages/cog/server/worker.py", line 222, in _predict
for r in result:
File "/root/.pyenv/versions/3.10.13/lib/python3.10/site-packages/torch/autograd/grad_mode.py", line 43, in generator_context
response = gen.send(None)
File "/src/predict.py", line 78, in predict
annotated_picture_mask, neg_annotated_picture_mask, mask, inverted_mask = run_grounding_sam(image,
File "/src/grounded_sam.py", line 100, in run_grounding_sam
neg_segmented_frame_masks = segment(image_source, sam_predictor, boxes=neg_detected_boxes)
File "/src/grounded_sam.py", line 55, in segment
masks, _, _ = sam_model.predict_torch(
File "/root/.pyenv/versions/3.10.13/lib/python3.10/site-packages/torch/autograd/grad_mode.py", line 27, in decorate_context
return func(*args, **kwargs)
File "/src/weights/segment-anything/segment_anything/predictor.py", line 229, in predict_torch
low_res_masks, iou_predictions = self.model.mask_decoder(
File "/root/.pyenv/versions/3.10.13/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1190, in _call_impl
return forward_call(*input, **kwargs)
File "/src/weights/segment-anything/segment_anything/modeling/mask_decoder.py", line 94, in forward
masks, iou_pred = self.predict_masks(
File "/src/weights/segment-anything/segment_anything/modeling/mask_decoder.py", line 144, in predict_masks
masks = (hyper_in @ upscaled_embedding.view(b, c, h * w)).view(b, -1, h, w)
RuntimeError: cannot reshape tensor of 0 elements into shape [0, -1, 256, 256] because the unspecified dimension size -1 can be any value and is ambiguous

Having tested this again, it works only with the default photo on Replicate, any other photo, it fails. I've tried different formats, color profiles, sizes, compression levels, but alway hit this error either via the replicate web form or the Replicate API. I guess it has somthing to do with the source image but I can't figure out what.. Any help much appreciated!

Thanks for reporting, Il check it in following days

Thanks for reporting, Il check it in following days

Ok thanks! 👌🏽

I just tried it, same problem.

Having tested this again, it works only with the default photo on Replicate, any other photo, it fails. I've tried different formats, color profiles, sizes, compression levels, but alway hit this error either via the replicate web form or the Replicate API. I guess it has somthing to do with the source image but I can't figure out what.. Any help much appreciated!

I think i have figured out the problem, you have to define the correct negative prompt(items you don't want masked).

Having tested this again, it works only with the default photo on Replicate, any other photo, it fails. I've tried different formats, color profiles, sizes, compression levels, but alway hit this error either via the replicate web form or the Replicate API. I guess it has somthing to do with the source image but I can't figure out what.. Any help much appreciated!

I think i have figured out the problem, you have to define the correct negative prompt(items you don't want masked).

Hi, I'm not sure because the script has a default negative prompt and running it on Replicate doesn't fail if you remove the negative prompt


Add workaround for avahi not working on loopback interfaces

As of now, avahi doesn't work with loopback interfaces. This results in Clever ROS nodes being unable to communicate with ROS master, even though they are all running locally.
This adds a workaround for this problem, placing a record for <hostname>.local in /etc/hosts. This does not sound like a good solution (a better one would be to make Clever resolvable by its hostname on every network, but so far I couldn't find a way to do that reliably).

So, if you want to rename your drone, you would have to do it in two places, again?

With this P/R there are now five places where you have to change hostname:

/etc/hostname
/etc/hosts (twice with this P/R)
/lib/systemd/system/roscore.env
/home/pi/.bashrc

I've updated the clever_rename script to automate these changes.

I think this problem has to be analysed yet another time. I hope I have some time to it. Otherwise, we will use this solution.

I really hope we'll come up with something else, to be honest. This is more of a last resort.

@bart02, that's related to the problem, you experienced on the Big Challenges.

Yes, I have also changed hostname in this five places. It wasn't problem for us. And it works normally.
But we got another problem. ROS had fallen down, when copter lost wifi connection. Copter was in wifi client mode.

Implemented in #150.


refactory on ffmpeg encoder

Current coverage is 68.45% (diff: 86.48%)

Merging #125 into master will increase coverage by 0.38%

@@             master       #125   diff @@
==========================================
  Files            18         18
  Lines           805        818    +13
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            548        560    +12
- Misses          205        207     +2
+ Partials         52         51     -1

Powered by Codecov. Last update 739870a...2f6f308


Backport PLAT-10436 : Change roleDetails definition to support string ids without Long format constraint

PLAT-10436 : Change roleDetails definition to support string without Long format constraint.
RoleDetails was defined as a string in Long format which prevent us to create roles as defined in Roles Object documentation. This is changed to support Strings without Long format constraint.

ok to test


Remove anyhow

The reasoning is that for a library, having a dedicated Error type makes it more easier to handle error in a programmatic manner.
It is easier to do the migration now than do it once we actually use the library and want to programatically handle some error cases.

Outdated. Good points from @luckysori. I haven't taken the time to decide on a better approach.


Add cq_cache decorator

Add a file based cache decorator function that allow to cache the result of a function creating a cadquery object.
When exploring design ideas from a complex geometry, rerunning the whole script at each change can be cumbersome. This plugin aims at helping on that.
It does it by writing a step file of the object created by decorated the function the first time a set of parameters is given and import this step if the script/the decorated function is run again with the same parameters.
While writing the tests I realized that the wrapper function doesn't necessarily return the same type of object as the decorated function. That's not goot.
I couldn't come up with a way to check the return object type of the decorated function so I'm asking help on this, if someone as any idea.
A solution could be to force the decorated function to return a specific type of object so the wrapper could return the same, but I think that makes the decorator less useful.
I'm happy to receive your feedback on this, thanks !

You probably want to use internal data format of OCCT to have no losses. So cq.Shape.exportBrep, I'll add cq.Shape.importBrep to enable this.

Yes definitely, I searched for this on the cadquery doc but I'm not sure there info on it. I may just have missed it though.

Docmentation link for exportBrep

Thanks I'll add this and I may have a solution for returning the right type of objects each time, will do that

This should be done now, I've switched to BREP files ( @adam-urbanczyk I wrote a importBrep function if you want to add it in cadquery) and I added some logic to be able to return the right type of object when decorating a function that returns a cadquery object.
I think I took care of everything but in the end it got a bit more complicated than expected so if you see any flaws let me know.

Thanks @Jojain this looks great. I think it would be helpful for there to be docstrings in all the cq_cache.py methods for us to reference in the future, and if I run black against the directory it still shows changes in cq_cache.py. I need to get the lint check fixed for PRs on this repo. I'll try to take a look at that tomorrow.

@jmwright @marcus7070  I have added docstrings and tried to use hash instead of str for the identification of passed arguments.
I think the use of hashing for everything would probably better than the use of strings but honestly I don't really understand how it works and I don't want to spend time on trying to figure it out.
If someone more knowledgeable than me on this wants to implement more robust changes I see no problem.
If you are good with the changes we can call it done.
(BTW I don't know what the lint test is about, I tried to run black and it still failing, is there a way to make it work ?)

@Jojain Thanks for all the work you've put into this. The black check here on GitHub always fails for PRs because of a missing feature in GitHub. One thing I have on my list is to run black manually in CI and see if I can get that to work instead of the current Action from the Marketplace.

@marcus7070 I'm ready to merge this. Let me know if you are too.

I just had some questions but I'm good with the changes ! Thanks @marcus7070  you found a nice way to remove the use of eval. that's great !

We might also want to wait https://github.com/CadQuery/cadquery/pull/735 to be resolved and use the importBrep implementation from cadquery ? (Altough it should be identical so we can merge it as it is)

We might also want to wait CadQuery/cadquery#735 to be resolved and use the importBrep implementation from cadquery ? (Altough it should be identical so we can merge it as it is)

I would say merge now and fix that later.

I see that @Jojain gave a thumbs-up on merging, so I'm going to go ahead and do that now. Thanks for the contribution @Jojain !


Automate android deployment

This is mostly complete. However a mistake with the signing keys means this cannot be fully tested for 2 days while waiting for Google to reset the certificate.


Deploy htsget-rs

This is basically a copy of https://github.com/umccr/infrastructure/blob/master/cdk/apps/htsget/htsget/goserver.py. It changes the deploy to use a HTTP proxy integration rather than a REST proxy integration, with added CORS, domain names, and JWT authorization.
Still to do

Clean up the configuration a bit as it is not as user friendly as I'd like.
Link this to the https://github.com/umccr/infrastructure/tree/master/cdk/apps/htsget docs.
Need to allow arbitrary URLs in the RegexResolver so that sources from anywhere can be used, similar to the htsget-refserver sources. See #134
Should the lambda function be run from a VPC in some way? Is there any benefit to this? E.g. https://docs.aws.amazon.com/lambda/latest/dg/configuration-vpc-endpoints.html

Good idea on reusing htsget-refserver's CDK as a template for our own, but I guess there's testing to be done after this is deployed? I'd go ahead and try to deploy this, but I doubt it's going to work as-is, hence the Draft status?
Let me know if me or @victorskl can help you out with any deployment hiccup and/or how AWS constructs work in the context of cognito/apigwv2/vpc/etc...?

I agree with keeping the docs as generic as possible. Should the config files also be generic then? Currently the config_*.toml files contain UMCCR specific config.

I agree with keeping the docs as generic as possible. Should the config files also be generic then? Currently the config_*.toml files contain UMCCR specific config.

I'd keep the docs generic and the configs specific because that way we help ourselves in ease of deployment and we also help third parties with some concrete examples that they can edit away easily? See comment in https://github.com/umccr/htsget-rs/pull/133#discussion_r1088410147

That sounds good. The other alternative could be to put the config somewhere on AWS and fetch it dynamically, leaving a generic config for this repo.