text:
Partially fill out row when invalid data is encountered

This is a rebase and minor update of https://github.com/databricks/spark-xml/pull/294 and should resolve https://github.com/databricks/spark-xml/issues/235 , per @Woffendm

Codecov Report

Merging #358 into master will not change coverage.
The diff coverage is 0%.

@@          Coverage Diff          @@
##           master   #358   +/-   ##
=====================================
  Coverage       0%     0%
=====================================
  Files          14     14
  Lines         742    744    +2
  Branches       60     63    +3
=====================================
- Misses        742    744    +2

Impacted Files
Coverage Δ

...scala/com/databricks/spark/xml/util/TypeCast.scala
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

@HyukjinKwon I noticed one thing about this just now -- in this case it doesn't add this record as a corrupted record. It also didn't before. Should it? I might be able to fix that quickly in a follow up... save any exceptions thrown while converting the object but continue, and throw the first one that occurred, to make it a failedRecord in parse?

Yea, it's okay. If that's not difficult, yes. we should fill the corrupt record.

See https://github.com/databricks/spark-xml/pull/368 -- disregard my comment about corrupt rows; we wouldn't set those in permissive mode.

authors: HyukjinKwon codecov-io srowen


text:
Changing buckets when not first page is selected

Version: 3.0.16
We have two buckets: Bucket A, Bucket B. Bucket A has 5 pages. Bucket B has 2 pages. When we have selected Bukcket A on e.g. 5 page and change active bucket to Bucket B, then we have white screen on the app.

fixed since 3.0.17

source: gharchive/issue

repo: databucket/databucket-server

license: MIT

license_type: permissive

license_source: github-api


text:
[nodejs] update highlighted point in line result

pass date in line config
update the highlighted point in the nodejs line result to be the specified date
skip returning a line chart result if there is a specified date, but no data point for that date

thanks for the review!

source: gharchive/pull-request

repo: datacommonsorg/website


text:
Get attempt for consumed msg

By adding attempt field to Message struct make sense to split Message to MessageIncoming which keeps mechanisms how to handle msg and MessageOutgoing which works as payload definition.

LGTM, the Message interface was a relict of the past.

repo: dataddo/pgq


text:
Correction for raw affiliation Professeur émérite de philosophie et d’histoire des sciences à l’université de Nantes et chercheur au Centre atlantique de philosophie, il a récemment publié Ce que peuvent les sciences. Une enquête (éd. Matériologiques, 2020)

Correction needed for raw affiliation Professeur émérite de philosophie et d’histoire des sciences à l’université de Nantes et chercheur au Centre atlantique de philosophie, il a récemment publié Ce que peuvent les sciences. Une enquête (éd. Matériologiques, 2020)
raw_affiliation_name: Professeur émérite de philosophie et d’histoire des sciences à l’université de Nantes et chercheur au Centre atlantique de philosophie, il a récemment publié Ce que peuvent les sciences. Une enquête (éd. Matériologiques, 2020)
new_rors: 059ynac98
previous_rors: 01bvfdq42
works_examples: W4200535083
contact: 0d586ced8a658bf14beee1e5fdd617b6:823c6f6bacc696c148ee6e654ce38e @ univ-nantes.fr

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

repo: dataesr/openalex-affiliations


text:
Correction for raw affiliation Courriel :; LADEC (Laboratoire d'anthropologie des enjeux contemporains) FRE 2002 – Université Lumière-Lyon2 – Campus Porte des Alpes, 5 avenue Pierre Mendès France – 69676 Bron

Correction needed for raw affiliation Courriel :; LADEC (Laboratoire d'anthropologie des enjeux contemporains) FRE 2002 – Université Lumière-Lyon2 – Campus Porte des Alpes, 5 avenue Pierre Mendès France – 69676 Bron
raw_affiliation_name: Courriel :; LADEC (Laboratoire d'anthropologie des enjeux contemporains) FRE 2002 – Université Lumière-Lyon2 – Campus Porte des Alpes, 5 avenue Pierre Mendès France – 69676 Bron
new_rors: 03rth4p18;000q6dn27
previous_rors: 03rth4p18
works_examples: W2802160589
contact: e6a2b71bad7dd44312a5e4eb48c8cce8:94f3a35be686a6e1c54df078ac54 @ ens-lyon.fr

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


text:
Correction for raw affiliation [COMPSYS, LIP, ENS de Lyon, UMR 5668 CNRS-ENS de Lyon-UCB Lyon-Inria, Lyon, France]

Correction needed for raw affiliation [COMPSYS, LIP, ENS de Lyon, UMR 5668 CNRS-ENS de Lyon-UCB Lyon-Inria, Lyon, France]
raw_affiliation_name: [COMPSYS, LIP, ENS de Lyon, UMR 5668 CNRS-ENS de Lyon-UCB Lyon-Inria, Lyon, France]
new_rors: 029brtt94;04zmssz18;02feahw73;04msnz457
previous_rors: 029brtt94;04zmssz18;02feahw73
works_examples: W2119166113
contact: 39c82b381f0827b50fa3a818b3699bb5:579aa8643f98a018204854ba10ff @ ens-lyon.fr

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


text:
generate phone numbers using Google library "libphonenumber"

This library already contains phone number patterns for all countries. It's reasonable to reuse them instead of re-inventing the wheel.
Now PhoneNumberTest is stable, and doesn't need all these "max allowed errors count" hacks.
P.S. Maybe it's a good idea not to include this PR into the nearest release.
It's quite a big change, maybe postpone it to a next major release?

Sorry, I'd like to veto this PR. I'm against bringing in new production libraries into Datafaker. It took me a lot of time to get most of them out, and only the Regex one is a special case for this. If you really want this, we could discuss making an SPI for this or so, that when you add a dependency to datafaker-phone-numbers it would use this library instead?

I'm against bringing in new production libraries into Datafaker.

Interesting take. :)
And what's the reason of not including USEFUL libraries?
Do you really want just to do the same job again?
NB! Look at PhoneNumberTest:
DataFaker almost always generates an invalid phone number.
For some countries, in 100% cases, and for most countries - in 77% cases.
@erikpragt-connectid
Is there any reason in the world why generating invalid data is better than adding a dependency?

Despite the fact I understand the rationale of usage libphonenumber I would agree that it's better not to add it.
Datafaker provides a number of different generators and adding a library just to improve one of them could be overkill in future. For instance if we add a lib for phone numbers why not for addresses and etc... Then finally it will tend to be a super fat... At the same time not every user uses phone number generator at least once...
From the side as it was mentioned such lib could improve quality of generated data.
Thus I would suggest to think towards kind of plugin api (SPI or whatever). We already have custom provider support we allows to create a provider with better quality. May be we need to think about more fine-grained api allowing to use more accurate custom code for some providers (currently just brainstorming)

@snuyanzin @erikpragt-connectid Ok, would you mind creating a new repository like https://github.com/datafaker-net/datafaker-phones? Who will be responsible for releasing new versions then? Someone will need to always remember to release datafaker-phones.jar every time when they release datafaker.jar It's easy to forget. Are you sure it's convenient?
P.S. I sincerely don't understand why you want to avoid dependencies.
There is absolutely no reasons for this.
Sorry, it's not normal. All projects have dependencies.
Look around, the world is global. Countries use each other's products. Projects use each other as dependencies. Etc.

Ok, would you mind creating a new repository like https://github.com/datafaker-net/datafaker-phones? Who will be responsible for releasing new versions then? Someone will need to always remember to release datafaker-phones.jar every time when they release datafaker.jar It's easy to forget. Are you sure it's convenient?

should we have a new repo? I guess having another module depending on a library would be ok.
Since then there could be a core module without extra dependencies and some other with. At the same time since everything in one repo then all could be released together and there is no issue with forgetting to release something

Ok, you suggest to make a multi-module project from datafaker? Yes, sure, why not. I can modify the PR accordingly.

Well, do you really need the library at runtime if all you need is the formats? As in, if we would use the lib phone number at  build  time to extract the formats, would that be an option?

libohonenumber's data is available as an xml file. I guess we could create a GitHub workflow to pull it and update our yaml(s) from time to time.
Though that seems like a lot of extra work to for some reason avoid including a 350k dependency. (I know you're worried about bloat but I suspect we could come up with some reasonable criteria/rationale to be applied in this and future cases.)

Javafaker had a ton of dependencies. Most of the dependencies had bugs or CVEs, which caused Javafaker to have CVEs too. Besides that, dependencies can have dependencies, and those dependencies can conflict with other dependencies.
Maybe "no extra dependencies" is  too strict, but I wouldn't know where to draw the line. Just keep in mind we have around 500.000 downloads per month, so that's the impact we can have on people.

Maybe "no extra dependencies" is too strict, but I wouldn't know where to draw the line. Just keep in mind we have around 500.000 downloads per month, so that's the impact we can have on people.

That's a good stat to know, and definitely helps contextualize things.
As for criteria, based just on what's been mentioned so far.

Minimal size (<1MB?)

libphonenumber is 350k. (On it's own, not compressed the library's metadata.xml is ~950k)

No dependency tree (single level, the library itself)

libphonenumber has zero dependencies according to maven: https://mvnrepository.com/artifact/com.googlecode.libphonenumber/libphonenumber/8.13.38)

Fills a need we can't do "simply".

Validating the formats for global phone numbers isn't a trivial task. We proven this with tests etc already.

Thanks for that writeup, appreciate it! Given that it doesn't introduce a whole tree of dependencies (see the merits of dependency free libraries? 😉) and the size is small, and all of the above, plus that we live in a democracy, and "code wins", I'm happy to change my mind on this. Maybe I'll maakt a small note in the contributing section (be mindful when introducing new libraries, or something like that).
What do you think @snuyanzin ?

Small question though : is the result still repeatable? I think I saw something in the PR about having to generate a phone number a couple of times to create a valid one?

is the result still repeatable

Good question.
But yes, the result is highly repeatable (all tests pass).
But yes, inside we have a loop a'la "up to 100 retries".
The reason is that regex is not enough to generate a 100% valid phone number.
We generate a random phone using regex, but then need to validate it using "libphonenumber" because some generated numbers are still not valid because of area codes etc.

@bodiam @snuyanzin You decide. I can prepare another PR which adds "libphonenumber" generation as a separate module, no problems. It may be even good to show others how they could add other modules as well.
The only drawback is that such PR would move all existing sources to a subfolder, something like "core" or "engine" or just "datafaker". It's because Maven multi-module architecture requires the root project have type "pom", and submodules can have type "jar".

I think the current way of doing it is fine, no need to overcomplicate things. I'm happy if you want to merge it.

authors: asolntsev bodiam erikpragt-connectid kingthorin snuyanzin

repo: datafaker-net/datafaker


text:
Add automated test covering the README example

You can just call rake to execute tests. I was doing this manually within OFN and thought it would be much more useful here.

@lecoqlibre I updated this branch for the newest version and added CI for tests. Do you want to merge this?

@mkllnk Thank you! Yes I will merge but the code on this repo is generated from an UML model. I have to merge this PR is the generator. I must take the time to do it ASAP.

authors: lecoqlibre mkllnk

repo: datafoodconsortium/connector-ruby


text:
Facets - distinguish "more" from regular items

Now, regular items seem identical to "more" controls. We need to distinguish them somehow... Bold? Color?

@jakubklimek Italic?

Yes.
Could we also maybe do a doubleline or something like that?  Now it seems slike just another row in the table...

authors: jakubklimek skodapetr

repo: datagov-cz/katalog


text:
Creating Dataframe from csv loses qnum precision

Here is an example%
 let a = DG.DataFrame.fromCsv(Id,Col1,Col2,Col3,Col4,Col5,Col6,
AAO239,0.001415,,,,0.00502,,
BHO269,<0.0009000,,,,,,
CSX801,<0.0009000,,,,,,
DQI713,,,0.002862,,,,
GHN729,,0.0008489,0.009932,2.84E-04,,>100.00,
KCF907,,0.001228,0.0048,,,34.43,
,,,,,,,
`);
grok.shell.addTableView(a);
a.toCsv();
`
After "grok.shell.addTableView" we could see that the "Col1" is shown as "0.00".
But if checked directly by:
a.columns.byName('Col1').get(0) > "0.001415"
After ".toCsv()" the result lose the qnum column (Col1) precision, and all the values look like "0.00"

This issue has been mirrored in Jira: https://reddata.atlassian.net/browse/GROK-15586

1.18.7

authors: LesiaPavlenko denyzhirkov dnillovna

repo: datagrok-ai/public


text:
fix(ingest/clickhouse): pin version to solve column reflection regression

See https://github.com/xzkostyan/clickhouse-sqlalchemy/issues/272
Example failure: https://github.com/datahub-project/datahub/actions/runs/6688931190/job/18171700309?pr=9123
Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

I wonder if we could get away with a != instead of a <=... This is safer though

@asikowitz I'm not 100% sure they'll fix it in the next release, which is why I did a < instead of !=

repo: datahub-project/datahub


text:
Datetime bug with dealing with undefined values

This is bug I found with the IBL tables. It has to do with the rewrite so I didn't push a change log.
Depends #102 (Sorry was working on this before I found this)

Can we cram this in the Beta.1? @guzman-raphael

I patch it temporarly for now, but yeah we really need to consildate on undefined for current value for tableAttribute

Update from dev now

I was hopping to shove it in beta cause it was broken due to the overall to allow copy over to work the other day.

repo: datajoint/datajoint-labbook


text:
Read-only instance

This PR is intended to fulfill #195.
This PR is based on #238. To get the relevant commit for this PR, merge the master branch into the associated branch after #238 is merged into the master branch.

@yarikoptic A read-only instance of Datalad-Registry is now on Falkor.

Per @yarikoptic's recommendation, it may be a good idea to respond 405 Method Not Allowed instead of 418 I'm a teapot in the event of a user posting to the declare_dataset_url endpoint.

repo: datalad/datalad-registry


text:
[FEATURE] DlSmartSearch - omit specific operators from the suggestion window

new prop called Omit-suggestions
is an array of values of the operators and any string
If the value exists in omit-suggestions then we will not display it in the suggestion menu

@fadiDL say we omit "OR" and then someone just types it without suggestion. What do we expect would happen?

authors: RotemDl fadiDL

repo: dataloop-ai/components


text:
Malformed query repeats one field multiple times

We have at table with the following 30 fields:

This fails and datanymizer exits because an internal untransformed query for the dump is repeating the event_type field 3 times in the COPY:

None ret from qw.copy_out(untransformed_query=
"COPY "metrics"."deployment"("deployment_id", "hypervisor_key", "specification_id", "vmware_deployment_target_id",
     "vcenter_url", "vcenter_cluster", "vcenter_folder", "clone_source_id", "event_type", "event_type", "event_type",
     "org_name", "org_id", "mto_node_id", "mto_node_path", "mto_node_org_path", "academy_event_id", "package_id",
     "package_estimated_duration_in_seconds", "course_plan_id", "deployer_user_id", "user_full_name", "user_email",
     "warden_event_id", "created_at", "last_updated", "project_id", "project_org_id", "project_org_name",
     "project_mto_node_id", "project_mto_node_path", "project_mto_node_org_path") TO STDOUT
"),

table=metrics.deployment,
count=0,
error=Error {
    kind: Db,
    cause: Some(DbError {
        severity: "ERROR",
        parsed_severity: Some(Error),
        code: SqlState(E42701),
        message: "column \"event_type\" specified more than once",
        detail: None, hint: None, position: None, where_: None, schema: None, table: None, column: None,
        datatype: None, constraint: None, file: Some("copy.c"), line: Some(5155),
        routine: Some("CopyGetAttnums")
    })
}

Working now to understand why and fix it.

The problem here was there can be multiple rows in pg_catalog.pg_type field typname that join with information_schema.columns field udt_name.  In my case there was three, because they were defined in different namespaces (schemas) in the same db.
Solution is to also join on pg_catalog.pg_namespace to ensure we have the right one.  After this change to schema_inspector.rs, it returns just the 30 fields and dumps properly.
Commented out the old query and added the new one:

// const TABLE_COLUMNS_QUERY: &str = "SELECT cc.column_name, cc.ordinal_position, cc.data_type, pt.oid
//                                    FROM information_schema.columns as cc
//                                    JOIN pg_catalog.pg_type as pt
//                                    ON cc.udt_name = pt.typname
//                                    WHERE cc.table_schema = $1 and cc.table_name = $2
//                                    ORDER BY cc.ordinal_position ASC";
const TABLE_COLUMNS_QUERY: &str = "SELECT cc.column_name, cc.ordinal_position, cc.data_type, pt.oid
FROM information_schema.columns as cc
JOIN pg_catalog.pg_namespace as pn
ON cc.udt_schema = pn.nspname
JOIN pg_catalog.pg_type as pt
ON cc.udt_name = pt.typname AND pn.oid = pt.typnamespace
WHERE cc.table_schema = $1 and cc.table_name = $2
ORDER BY cc.ordinal_position ASC";

PR will be coming with this fix.

repo: datanymizer/datanymizer


text:
81-BE/import-webapp

Import data from webapp.

We don't migrate this table because the customer doesn't request it.

repo: dataquest-dev/DSpace


text:
kubectl plugin

Another ticket to be casskop compatible by adding a plugin similar to casskop plugin. The goal is to ease any operation to run on the cluster using a kubectl plugin with a simple set of commands. That plugin could also be used to simply create clusters by asking some questions and based on the answers building automatically the corresponding objects that cass-operator watches:
# Clean nodes in dc1
$ kubectl cass-operator clean dc1

# Trigger a rolling restart of nodes in rack rack1 and rack2
$ kubectl cass-operator restart --crd my-crd --rack rack1 rack2
...

Migrated over to https://github.com/k8ssandra/k8ssandra/issues/937

authors: cscetbon jdonenine

repo: datastax/cass-operator


text:
Don't mark host down while one connection is active

Refactor pool to allow partially connected pools and prevent nodes to be marked as down until all connections (from all connection pools) have been closed.

@landim scassandra integration for tests is looking good. As the implementation of the new host pool involves a lot of changes, do you mind integrating scassandra in a different branch?

Squashed commits.

There is a failure in the last appveyor build that surfaced an issue in the implementation of the logger (CSHARP-171).
I'm going to fix it on this branch.

After several builds, there is another failure: https://ci.appveyor.com/project/DataStax/csharp-driver/build/CSHARP-274-708/job/qyeyrdfdc6dk8xrh
It looks like a driver issue, I'm looking into it.

The issue described here is now fixed.
I've set the reconnection test to be repeated 20 times per build on f3a4069 and it passes (executed total of 400 times on Jenkins and 120 on appveyor).
I've restore to only execute it few times per build.
I've also added several test fixes. I will merge it once the last build finishes.

Hi @jorgebay
It seems that you merged the implementations of two properties on Host : IsConsiderablyUp and IsUp now return the same value (which is IsUp).
I think it could be nice to mark the IsConsiderablyUp property as obsolete - and remove it from good for the next breaking changes release.
I checked in the java implem and I can't find the same considerably thing, only a  getIsUp method

You are right about deprecating IsConsiderablyUp, it was previously used to determine if reconnection attempts can be made in case it was down.
I would soft-deprecate it, I'm very conservative about using [Obsolete] in a minor version: for users with "Warning as error" setting it will mean a breaking change.
We can now include a paragraph in the xmldoc, saying something like: "This property is going to be removed in future versions, use IsUp instead" and in the next major version, we can mark it as obsolete.
Would you like to send a pull request for it?

Yes I will push a PR later today.
And you're right about the obsolete attribute, I'll just add a comment and leave the breaking change for a future version.

authors: ablanchet jorgebay

repo: datastax/csharp-driver

license_source: bigquery


text:
[WIP] Fixes #181 and adds support for OAuth2 / Okta.

This PR Fixes #181 and provides additional examples for using TLS with S4K and OAuth2.
Also, it's a WIP until I finish doing some testing.
Remaining items:

Resolve auth failures from Prometheus. This requires adding an OAuth2 section to the PodMonitors provided in this PR once it's merged into master.
(Optional) Resolve broker restart that occurs when attemptAuthenticationProviderToken: "true". See note: https://github.com/datastax/pulsar-openid-connect-plugin#using-authenticationprovidertoken
Investigate/resolve auth issues with S4K

Superseded by https://github.com/datastax/pulsar-helm-chart/pull/278

repo: datastax/pulsar-helm-chart


text:
Update Spark version to 2.4.0 and make sure it is supported

This is a fix for https://datastax-oss.atlassian.net/browse/SPARKC-553.

Thanks for the fix! I'll try to quick off the tests

test this please

Jenkins can't hear me, i'll run them manually

test this please

Excellent tests are running

Build against Scala 2.10 finished with failure

Build against Scala 2.11 finished with failure

@nikolovivan looks like there is one more issue in the com.datastax.spark.connector.repl.CassandraRDDReplSpec
Another class not found. Fix up that test and I think we are good to go

Hey, I forgot that I had troubles last time running the integration tests and I had to run them from a clean Docker container. Just did it and I think I had to bump the Scala version to the same number they've used in Spark to fix the error.
I don't have the time to run the tests fully at the moment as I have to run, but feel free to trigger them again. If there are issues, I'll spend time properly addressing them over the weekend.

this this please

Hey, I forgot that I had troubles last time running the integration tests and I had to run them from a clean Docker container. Just did it and I think I had to bump the Scala version to the same number they've used in Spark to fix the error.
I don't have the time to run the tests fully at the moment as I have to run, but feel free to trigger them again. If there are issues, I'll spend time properly addressing them over the weekend.

No problem i'll trigger everything again

test this please

for some reason I have to say it twice ... or maybe since I was looking at the configuration it felt bad and decided to work the second time

Build against Scala 2.11 finished with success

Build against Scala 2.10 finished with failure

Looks like it's passing for 2.11, which is the only supported version since 2.3, so I guess it's good now.
Let me know if you would prefer me to see which could be the lowest possible version between 2.11.8 and 2.11.12 that would make the build pass?

I just checked and we could go as low as 2.11.9 if we wanted to. Might be best to keep it consistent with what version Spark 2.4.0 uses, though (2.11.12). Let me know what you prefer.

Does this also add support for scala 2.12?

No, it doesn't. Adding support for 2.12 might be trivial, but might also involve changing lots of other library versions and code.

2.10 Shouldn't pass on master (I just forgot to turn off triggering it) so This is good to go

Do you have an idea when you'll be merging this one and releasing a new version?

No, it doesn't. Adding support for 2.12 might be trivial, but might also involve changing lots of other library versions and code.

Yes.
Based on this, looks like scala 2.12 is going to be the default scala version going forward.

I've easily built the project against scala 2.12 but I had to disable the scoverage plugin.
Seems that enabling it requires upgrading sbt.

@RussellSpitzer do you have any idea when you're planning on merging this and releasing a new version. Let me know if I can help in any way.

any update here?

Just finished getting caught up at work, I'll do a release tonight.

authors: RussellSpitzer ds-jenkins-builds igreenfield nikolovivan purijatin simpadjo

repo: datastax/spark-cassandra-connector


text:
海豚调度器配合hdfs高可用

Search before asking

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

Description
按照官方推荐的部署步骤安装之后，都使用默认配置，DS没办法做到开箱即用
因为hdfs是高可用的，所以我需要将hdfs-site.xml、core-site.xml，复制到DS的conf目录下，但是DS有4个server，具体复制到哪个生效就不得而知了，我都复制了进去。
所以希望以后可以更智能一点
谢谢！！！
Usage Scenario
No response
Related issues
No response
Are you willing to submit a PR?

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

Code of Conduct

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

Thank you for your suggestion，it will be added in version 1.1.0.

这功能是不是还没加上？

authors: a19920714liou datasophon liushuaihang

repo: datavane/datasophon


text:
Qwen2-vl-2B Lora 微调 SwanLab 可视化记录版 @林泽毅

麻烦问下这个代码库中的数据集格式和qwen2-vl官网提供的数据集格式好像有点不一样，数据集格式看起来和qwen-vl的格式一样。另外想请问这个微调代码是参考的哪个和官网的有点不一样。最后想问一下这个lora微调后看起来并没有合并到原模型中，希望解答感谢

你好，感谢你对代码的反馈：

代码是自己写的
在微调时，Qwen-VL格式的数据仍然可以适用于Qwen2-VL，这一块也可以看看https://github.com/datawhalechina/self-llm/blob/master/models/Qwen2-VL/06-Qwen2-VL-2B-Instruct%20Lora%20%E5%BE%AE%E8%B0%83%E6%A1%88%E4%BE%8B%20-%20LaTexOCR.md，这一块的代码做了更多的精简和改进，微调效果也work
推理实际已经加载了微调后的lora模型，可以看一下这两行，逻辑是先加载原预训练模型，然后再挂载lora训练后的模型。

model = Qwen2VLForConditionalGeneration.from_pretrained(
    "./Qwen/Qwen2-VL-2B-Instruct", torch_dtype="auto", device_map="auto"
)
model = PeftModel.from_pretrained(model, model_id="./output/Qwen2-VL-2B/checkpoint-62", config=config)

repo: datawhalechina/self-llm


text:
Update Python Client Library to match our standardized API (adding methods as needed)

Pattern is in https://github.com/datopian/tech.datopian.com/issues/23
For full details on each method see https://github.com/datopian/ckan-client-js/issues/4 and associated sublinks.
Acceptance

[x] Client class with documented constructor #18

Plumbing

[x] client.action(...) for this just use existing library https://github.com/ckan/ckanapi #18

Porcelain
NB: you can just implement on Dataset for metadataOnly=True for now.
Dataset

[x] create(datasetMetadata, metadataOnly=false): iterate through the resources pushing the data for each one then push the dataset metadata #18
[x] push(datasetMetadata, metadataOnly): ditto but for existing dataset #18
[x] retrieve(datasetIdOrName) #18

Resource

[x] pushBlob #18
[ ] createResource(resourceObject, datasetIdOrName, metadataOnly)
[ ] pushResource(resourceObject, metadataOnly)

Also provide example of creating a dataset from a single file ... ACTUALLY THIS SHOULD NOT HAPPEN. Always explicitly create the dataset.
If you just want to push / create metadata you can use the action API
use the mapper https://github.com/frictionlessdata/frictionless-ckan-mapper
import xxx as f11s2ckan_mapper
datasetFritionless = xxx
ckanDataset = f11s2ckan_mapper.dataset(datasetFriictionles)
client.action('package_create', ckan_metadata)

There are only two methods not implemented in master ATM: create_resource and push_resource — should we create a new issue for them and close this issue, or is it better to leave this issue open that way?

It was done before #18, and, after #18, fixed in e63d759eae8518749a9e2c2c8eb05a5ed814c335.

There are only two methods not implemented in master ATM: create_resource and push_resource — should we create a new issue for them and close this issue, or is it better to leave this issue open that way?

@cuducos any updates on this? ^^
This is a blocker in NHS atm, cause the client needs a reproducible code example (python script) for:

create a dataset and add a resource (CSV) using the API.
update/add a new resource to an existing dataset using the API.

cc/ @rufuspollock

authors: cuducos mbeilin rufuspollock

repo: datopian/ckan-client-py


text:
fix tests on windows CI

I do not know why, but appveyor has some failing CLI tests on windows but the same tests pass when I run them on windows XP in virtualbox (in both git-shell and CMD.exe)
appveyor uses windows server 2008

it'd be nice if we had windows support for beta

We're going to postpone guaranteeing windows compatibility until 1.0

Hey, could maybe be a diff in the MSVC versions. Can you check on your VM and set the version on Appveyor? It seems you can set it somehow.
It could be it, since it's msbuild that's failing for libutp in the package utp-native.

@mafintosh did you need windows support for the contract you were working on? curious what's left

woot! all green: https://ci.appveyor.com/project/joehand/dat/build/1.0.45

authors: joehand karissa maxogden onbjerg

repo: datproject/dat


text:
move to datrs org

and make friends on https://github.com/datrs :tada:

done!

repo: datrs/flat-tree-rs


text:
reauth giving error "Something unexpected happened.  Please try again."

Trying to reauth to a service this morning I got errors about "Something unexpected happened.  Please try again."  I tried again many times, then at some point something unexpected stopped happening and it started working.

This happens when the oauth/authorize endpoint is called with a state parameter that does not match the (possibly) nil state parameter in the session.  Go to https://oauth./session to view.
I've gotten reports that this happens when loading multiple tabs to the same proxy instance while logged out, triggering multiple simultaneous authentications (since they'll stomp on each other's cookie data).  I'll try to think of a way to handle multiple simultaneous logins...

authors: davars drasch

repo: davars/sohop

license: isc


text:
Segment Violation

Hi Dave,
I´m testing drachtio-server under siprec enviroment with opensips. After few minutes running the drachtio-server crash with Segment Violation error. ¿How can I help you to resolve?
018-11-07 09:59:11.571880 tport.c:2324 tport_set_secondary_timer() tport(0x12a5f00): reset timer
2018-11-07 09:59:11.572008 timerG: Adding entry to go off in 4000ms
2018-11-07 09:59:11.572124 timerG: Adding entry to the tail of the queue: length 4
2018-11-07 09:59:11.785494 timerH: running timer function
2018-11-07 09:59:11.785587 timerH: Setting timer for 6149ms after processing expired timers, length: 3
2018-11-07 09:59:11.785625 SipDialogController::endRetransmitFinalResponse - never received ACK for final response to incoming INVITE; irq:0x12c3fe0 source address was 192.168.88.54
2018-11-07 09:59:11.785659 timerG: removing entry, prior to removal length: 4
2018-11-07 09:59:11.785698 SipDialogController::clearIIPFinal:  clearing leg 0x12bd930
2018-11-07 09:59:11.785738 nta.c:5832 incoming_free() nta: incoming_free(0x12c3fe0)
Violación de segmento

can you tell me a bit more:

what version are you running (drachtio -v)
are running in a container or did you do a build?  If a build, which branch did you build
is it easy to recreate this? what is the type of call you are doing?

finally, I would like to see the entire log for those 2-3 minutes you mention.  can you post it as a gist and put a link here?

Version 0.7.1-rc3
Branch develop
I build the solution following your amazing instructions.
Yes, it´s easy. Just make calls to an opensips with siprec using sipp. Reponse to siprec invites with nodejs application, then while working, end the opensips service and the error appear in few seconds.
Thanks in advance, I love your solution. One question, I have write a c++ library to connect to drachtio-server because I didn´t find any in your projects. Have you thought about developing it?
log.txt

I am thinking this might have been fixed in this commit:
b255207df clear IIP when sofia fires timerH on uas leg after retransmissions of final response are exhausted
which is in v0.7.4-rc1 and later (but build either v0.7-4-rc2, or the head of the 'develop' branch)

Thanks I´ll try it

as for your c++ question, I am not understanding -- do you mean that you do not want to write your application in Node.js, and instead want to have a c++ application connect and control the server?

any feedback as to whether you were able to recreate this crash using the latest build?

authors: davehorton falvarezoliva

repo: davehorton/drachtio-server

license: mit


text:
Allow user-defined pull request number

Hi,
We are using this Github Action to report the test coverage in our pipeline. However, the actions are not always triggered by pull_request, but push instead. As a result, in the context there is no pull request number:
let pullRequestNumber = github.context.payload.pull_request?.number;

Therefore a user-defined pull request number as an input would be helpful (suppose the prNumber is given by the user):
let pullRequestNumber = github.context.payload.pull_request?.number || prNumber;

There are other test coverage GitHub Actions which allow this, for example https://github.com/ArtiomTr/jest-coverage-report-action/tree/main.
I could also submit a pull request if needed.

Hey @xli12,
thank you for opening up the issue!
Definitely something I would like to support, and I would welcome a PR for this very much! 👍

Hi @davelosert ,
Thank you for your quick reply! A pull request #369 has been submitted. Please let me know if you have further questions.

@xli12: I just merged the PR and this feature got released in version 2.4.0.
Feel free to chime in here and let me know if it works as expected, then we can close the issue.
Thank you again for contributing! ❤

@davelosert Thank you for all the assistance with this feature. Just tested using the latest version v2.4.0 in our repository, it works really well 👍

@xli12 : Thank you so much for the feedback - glad to see it works and already provides value to you! ❤
I will close this issue then!

@xli12: Coming back here - I've just released v2.6.0 of this action with a feature with this action that allows you to set the pr-number: "auto", which will automatically try to find a pull-request associated with the latest head-sha of the commit on a push event. Maybe this is helpful for you, too?
I guess you have some intermediate action right now that would do the same that you potentially could remove then. Let me know if it's useful and works for you! :)


text:
Radio.Yandex support

Few days ago Yandex launched radio-service. It work based on Music.Yandex so I guess that feature easy to add

#668 is here

repo: david-sabata/web-scrobbler


text:
Merge RadioTunes and JazzRadio into one connector

There is no reason to keep multiple connectors for identical websites.

Makes sense :+1:

I found no information about owners of these websites.

authors: alexesprit inverse


text:
Spanish version

Hi!!! My name es Al and I am a Spanish Emergency Doctor. I think your app is really interesting. Thanks a lot for it!! If you consider it is worthy, I would love to translate it to Spanish. Best Regards!!

Hi, that is a great idea, we can definitely do that. Do you know how Git works? I created a branch called 'Spanish-Version' and invited you. You could update the language.js to include the spanish text and I will do the remaining coding. Remember to include yourself in the about section as translator for the spanish version.
If you are not familiar with Git, Github oder coding in general let me know and I will walk you through it. It's quite easy.
Which system of axis types do you use in Spain? Only three to four types (normal, left axis deviation, right axis deviation, northwest-typ / extreme axis deviation) comparable to the english system?
Have a nice weekend. :)

Dear David,
Thanks for your response
I already updated the Spanish language versión. I hope I did it correctly. In Spain we have the 4 types of axis model:
Normal = Normal
Left = Eje IZQUIERDO
Right = Eje DERECHO
NorthWest = EXTREMO
Best regards,
Al
Enviado desde Correohttps://go.microsoft.com/fwlink/?LinkId=550986 para Windows
De: David @.>
Enviado: viernes, 29 de abril de 2022 18:54
Para: @.>
CC: @.>; @.>
Asunto: Re: [david-shrk/ecgaxistrainer] Spanish version (Issue #1)
Hi, that is a great idea, we can definitely do that. Do you know how Git works? I created a branch called 'Spanish-Version' and invited you. You could update the language.js to include the spanish text and I will do the remaining coding. Remember to include yourself in the about section as translator for the spanish version.
If you are not familiar with Git, Github oder coding in general let me know and I will walk you through it. It's quite easy.
Which system of axis types do you use in Spain? Only three to four types (normal, left axis deviation, right axis deviation, northwest-typ / extreme axis deviation) comparable to the english system?
Have a nice weekend. :)
—
Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdavid-shrk%2Fecgaxistrainer%2Fissues%2F1%23issuecomment-1113524796&data=05|01||25169010733f4034d7c408da2a00f98c|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637868480928699782|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=bj8yBGUtn%2BTvUBcVTdn8mOl96NFVZfeeD3g9jIJCqjI%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FACAECKOBJ3GU24YWQNFDIPDVHQH5TANCNFSM5UWLJQEA&data=05|01||25169010733f4034d7c408da2a00f98c|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637868480928699782|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=iZk7%2BiQwBOgj%2F3SBnk6u2msHVAmZtN3quamjUaAeNLc%3D&reserved=0.
You are receiving this because you authored the thread.Message ID: @.***>

Hi, that looks great! :)
I will include the necessary code to switch to spanish as soon as possible.
I would really like to give you credit as the translator in the about-section. Do you use twitter? I could put a link to your twitter-account in there if you want to.
Best regards,
David

Hi Al,
I added the spanish version and released it. Thank you very much for the translation. I had to inlcude very minor changes because some words/phrases were too long. And I included a (probably poorly translated ;) ) line to at least give you credit with your first name. If you have a twitter-account or something similiar I will happily link to it in the about section.
Best regards,
David

Dear David:

Thanks a lot  for your effort. Its great to have it in Spanish¡¡
I will replace this: (in fact, this sentene is incorrect in Spanih): "espero que pueda resultar útil para entender a calcular e interpretar el eje cardíaco"  with  "espero que pueda ser útil para aprender a interpretar y calcular el eje cardíaco".
My name is Álvaro Pemartin and my Twitte account is @PemartinA

Best regards,
Al

De: David Schaack @.>
Enviado: sábado, 7 de mayo de 2022 11:24
Para: david-shrk/ecgaxistrainer @.>
Cc: Larrey @.>; Author @.>
Asunto: Re: [david-shrk/ecgaxistrainer] Spanish version (Issue #1)
Hi Al,
I added the spanish version and released it. Thank you very much for the translation. I had to inlcude very minor changes because some words/phrases were too long. And I included a (probably poorly translated ;) ) line to at least give you credit with your first name. If you have a twitter-account or something similiar I will happily link to it in the about section.
Best regards,
David
—
Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdavid-shrk%2Fecgaxistrainer%2Fissues%2F1%23issuecomment-1120191598&data=05|01||ae0a50ace9684bd766d808da301c1d66|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637875194581650687|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=ipxIF6E%2FWWI1wvM5gCwjEK2ontGh8HmlqcVMzsTyMWs%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FACAECKLUQLL5SBSFOUIZJB3VIZHF3ANCNFSM5UWLJQEA&data=05|01||ae0a50ace9684bd766d808da301c1d66|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637875194581650687|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=A4XMiekAQYQIrOCEPkh10X0JjrbczHWRuzsnVBv7g3g%3D&reserved=0.
You are receiving this because you authored the thread.Message ID: @.***>

authors: Larrey david-shrk

repo: david-shrk/ecgaxistrainer


text:
Use correct socket path for linux

Fixes #331
I explained in ticket the reason to set the SSH_AUTH_SOCK environment variable

Hi @alinalexandru,
I believe it won't work correctly in case if your user id on the host machine not equal to 1000.
Here are more details about that: https://github.com/davidalger/warden/pull/307#issuecomment-781921198 https://github.com/davidalger/warden/pull/307#issuecomment-783863500

You are right with the user id.
The issue with user with ID 1000 still exists. This week I had to change the user ID to 1000 so that I could fix the composer/cache permissions.
Don't know if there could be done a user id check in entry point.
For us at the moment, this hack is working, as we all have User ID 1000.

@alinalexandru maybe we can introduce a new variable that will be set similar to this?
https://github.com/davidalger/warden/blob/5d1843bca3f6ed0ebc8615d651db4bc57765bb91/commands/env.cmd#L46-L49

That it's a good ideea, it might solve this issue for most of the linux users.

Yes, I will work on that today.

Any updates on when this can be release?
Great job @alinalexandru for the debug. Thank you very much, i am having this issue very often now and couldn't figure out what was happening.

@davidalger could you review this PR?

LG2M as well. Thanks for the efforts at guiding this one @ihor-sviziev. Going to merge it, and once I have a chance to verify on my Linux box over here and update the CL will tag a release.

authors: alinalexandru davidalger ihor-sviziev springerin

repo: davidalger/warden


text:
just curious: why no  ?

Hi David,
your component is pretty cool.
I'm new to react and wondering, why you don't use the native input element?
For a better styling?

@christianhaller Yes styling is difficult with native input elements. Even for styling selectbox we have restrictions. We generally bind it with a  tag and hide selectbox and style label tag with content. Similarly in this library div's are being targeted for the same.

thanks for your answer @prasanna1211

repo: davidchin/react-input-range


text:
Add php-scrypt as a example project

Hey There 👋
We at Appwrite recently created an extension using this library to fix an issue we had with the standard C-based php-scrypt by rewriting it in rust. Thanks to this library being so easy to use, we could get a working version in less than a day!
We would love it if you could add it as an example project in this repository to give more examples of how to use this library.
Thank you for reviewing this pull request,
Bradley Schofield

@ptondereau Thank you for reviewing our example, I was wondering where we should place our stubs within the repo once we have generated them? Is there a standard for this or should we do add it to our build steps?

@ptondereau Thank you for reviewing our example, I was wondering where we should place our stubs within the repo once we have generated them? Is there a standard for this? or should we add it to our build steps?

IMHO, it's up to you: I would go to exposing the stubs with a dedicated Packagist entry. In the ext's repository, add all stubs to a composer.json with a name in a manner that your user just need to composer install vendor/ext-stubs

authors: PineappleIOnic ptondereau


text:
[Feature] Added support for custom Formspree endpoints

Added a setting to allow optional usage of custom Formspree endpoints, rather than using the email address itself as endpoint, which exposes it to the public as part of the POST request.
This also makes it possible to have multiple forms delivering to different addresses.

This is a super nice addition, thank you @ManuelFte. The Formspree team will be pleased, the email method is kinda legacy 😅

Sorry for the delay @ManuelFte. This is looking good! There appears to be a typo on line 18, site:email should be site.email.
Also wondering if we can omit the JavaScript if the email isn't being used, as it's only there to prevent spambots getting peoples' emails 🤔
What if the unless around the <script> block checked for endpoint too? And then make the action value:
{% if include.endpoint %}https://formspree.io/{{ include.endpoint }}{% else %}{{ thanks_url }}{% endif %}

Doing it this way removes the need for the assign at the top, and only applies the JavaScript when they're using it with their email. Does that make sense?

@ManuelFte hey there! I know it's been a while since we talked about this addition. I really appreciate you taking the time to contribute, but I think this update needs to be approached in a more simplified manner.
The old way of achieving these contact forms was with a single email, so it made sense to place it into the config file and used throughout the site. However now most form services provide an endpoint for you to add to a particular form on your site. In that case the most of the form code in the theme can be removed and instead the form component could just accept and endpoint key directly in it.
I'm going to keep this issue open until I've come up with a solution. However if you beat me to the punch I'll defined merge it. Thanks again!

Sorry @ManuelFte but I'm going to have to close this PR since it's been open for over 2 years and I'm getting additional requests to fix this. Regardless I appreciate your contributions!

authors: ManuelFte daviddarnes

repo: daviddarnes/alembic


text:
Bug onLoadMore InstagramItem layout changes

When I want to load more items and scroll down the layout changes, look at image above, as a result of this the sticky headers won't work at this moment. Sometimes i get this when i load more images in the feed and sometimes when i refresh the sticky headers don't work anymore.
It looks like there is a layout behind the feed that hangs there while you scroll in the feed. It's not that the instagram item changes but the background behind the items changes with one item that sticks there and gives a look like the image above. You see this one item in the background only at the comments area.

@yilzer, it happened to me only once, I can say that it is (very) rare, but I can't find a way to reproduce it systematically and I don't find the single cause. I used emulator and real phone as well, but nothing.
So, I don't know where to look into, however it is related to the technique used for the sticky headers.

@yilzer, actually this issue should be resolved with the solution provided in #92.

authors: davideas yilzer

repo: davideas/FlexibleAdapter


text:
Getting Started doc: consider the lack of a self-signed certificate for SSL on Linux

Valid for Linux based development
Those following the step by step on the Getting Started guide, on Linux, will notice an error at frontend web site url hitting time. This is due to the fact that Startup.cs, created by dotnet templates, enforces an HTTPS redirection (for Production time, even HSTS is enforced).
As Linux doesn't get an auto self-signed certificate as Windows and MacOS do, it crashes.
Options:

Comment these enforcements right after dotnet new
Haven't tested @andrewlock's proposal yet:
https://andrewlock.net/creating-and-trusting-a-self-signed-certificate-on-linux-for-use-in-kestrel-and-asp-net-core/

Perhaps this is worth mentioning in documentation?

Ah, it is!

All progress will be moving to https://github.com/dotnet/tye

authors: davidfowl luisgizirian

repo: davidfowl/Micronetes


text:
Zoomed view of the 2D board can hide the spot of interest

A minor thing in the ease-of-use department...:
Scenario: Playing Formic Functions with default settings (2500x1000 arena shown in 2D) in a browser window not quite as wide as the arena is (Safari 11.0.3 on macOS).
When the mouse pointer is in the right half of the arena/board, the zoomed view is displayed flush left with the left edge of the board, and it is less than half as wide as the board:  Works fine.
When the mouse pointer is in the left half of the board, the zoomed view is displayed, not flush with the right edge, but closer to the middle of the board.  When the point of interest
(and thus the mouse pointer) is itself near the middle of the board in the left-right direction, the zoomed view will pop up so as to cover it, and it pops down again at the slightest mouse jiggle. Moreover, it seems not to be possible to pin the zoomed view to such a point of interest, I guess because the click requesting to do so ends up in the zoomed view instead of in the board which is momentarily hidden under it.

Looks like the issue here is that it decides which side to show the zoomed view based on whether the mouse is left or right of the centre of the board, rather than the centre of the window.
Even with that fix, once the window gets small enough it will still be an issue.
As for being able to click-to-pin, I think your analysis is correct. Unfortunately it can't be marked with pointer-events: none since the zoomed view has tooltips on the ants. I'll have a think on what can be done there.

Methinks a sticky view would actually make it worse - the same issue could then occur when the zoomed view is on the left.
Alternatively, how about keeping the zoomed view in one of two fixed absolute positions, deciding (as now) which one to use based on where the mouse is WRT to the centre of the board, and moving the righthand position of the view to align with the right edge of the board, regardless of the window width?
When I've scrolled to show the left board edge, and have the mouse on the left, the zoomed view would then pop up on the right partially hidden under the window edge. But that's OK and consistent: it would mirror what happens on the left when I've scrolled the other way.
The important thing (to me) is that the zoomed view should never cover that spot on the board where the pointer is. I need to see enough context around the pointer when I'm trying to find the best focal point to zoom into. And when the pop-up doesn't obscure the pointer location, click-to-pin will also work.
(There'll still be a problem when the zoomed view is square and the arena is less oblong -- width less than twice the height... But this is no concern for me. I'm not trying to play Formic Square :) .)

That sounds reasonable. Easier to do than my proposed fix too!


text:
Throttle karma requests

We need to prevent high-frequency botwars of attrition

Someone went nuts in the channel tonight; getting Donald.Trump to -666 (LOL).
This throttle comes not a moment too soon.
Please clarify, if I'm reading the code correctly it only throttles per user?  That would be my desired operation.  I.e. A single user cannot do multiple consecutive karma for a given string, but others can so if one person gets a flood of praise they don't lose out.
Right?

Answered my own question by re-reading the code and testing; yes it has the desired function of throttling per user.
Suggest we close this and revisit with a new issue if we need to adjust the throttle.

Actually there is a bug.
Repro:
Try foo++ then bar++.
Expected result:  Both update.
Actual result: bar gets ignored since the user previously incremented on foo within timelimit.

That was intentional for me, didn't want someone spamming random crap into the karma db either. I'd suggest we close this one, as technically we've done it, and open another if you want to change the way it works

Ack - happy to go with this.  Thanks

authors: alsm davidjpeacock

repo: davidjpeacock/shelbot


text:
sprites for each of Sam's body parts

Some basic sprites for each section would be good so we can figure out an art pipeline.

basic texturing being worked on in branch
https://github.com/davidjtferguson/silly-sam/tree/feature/sam-texturing

donzeoo total first pass just to see how bad they look

authors: davidjtferguson rcsweeney

repo: davidjtferguson/silly-sam


text:
Fails to connect to myob

class MyobSessionsController < ApplicationController
def new
redirect_to myob_client.get_access_code_url
end
def create
  @token         = myob_client.get_access_token(params[:code])
  @company_files = myob_client.company_file.all
  # then show the user a view where they can log in to their company file
end

def myob_client
  @api_client = Myob::Api::Client.new({
    :consumer => {
      :key    => "u6785eurqcjv6wawgby7b3ag",
      :secret => "VmCkQxrMJwefmf5XYwKD5Zv7",
    },
  })
end

end
routes
resources :myob_sessions
match "myob" => "myob_sessions#new", as: :myob_sessions_new, via: [:get, :post]
localhost:3000/myob
goes to myob login screen
then returns this screen
Your authorised applications
Allow access to your MYOB Account?
We have encountered some issue with the request, please try again later or contact your app support
invalid_request

My registered apps
https://my.myob.com.au/Bd/pages/DevAppList.aspx
Application: mml
Key 	Secret 	Redirect Uri 	Usage (last 24 hr) 	Quota Limit 	Throttle Limit 	Action
u6785eurqcjv6wawgby7b3ag 	VmCkQxrMJwefmf5XYwKD5Zv7 	http://desktop 	2 	5000 per day 	2 per second 	Edit
What am i missing here?

Closing this issue as it looks like you're able to connect (either via the myob-api gem, or the omniauth-myob gem). If you run into another problem, please open another issue – thanks!

authors: bruced davidlumley

repo: davidlumley/myob-api


text:
Enqueued time of a sent message cannot have a value

https://github.com/davidrevoledo/ServiceBus.LogginPlugin/blob/def057b3647e67d9edd9cbf6d72e750876b363cf/src/ServiceBus.LogginPlugin/Infraestructure/MessageExtensions.cs#L53

will be removed here https://github.com/davidrevoledo/ServiceBus.LogginPlugin/issues/5

authors: SeanFeldman davidrevoledo


text:
Dynamically edit toolbar

I was to change the colours in the toolbar dynamically, I built a little component with inputs where you can add your own hex's but the colours in the toolbar are just the ones i set on load.

This issue is stale because it has been open for 60 days with no activity.

authors: davidroyer samjonesigd


text:
How to use this in vue ssr?

Uncaught TypeError: QRCode is not a constructor,when I user it in vue ssr.
import it in vue mounted.

Strict mode is not supported, let alone ssr

authors: DrawAChicken JMFxialin

repo: davidshimjs/qrcodejs


text:
Change focus after focus-trap unmounts requires setTimeout

Im using focus-trap in a modal component. When the modal is closed I need to set the focus on a specific element (using a ref). I have this working in a setTimeout but it doesn't work without. Is this expected or is this a code smell?
Here is my code. The modal is closed and focus is set based on a URL change provided by React Router.
class MyComponent extends React.Component {
  constructor(props) {
    super(props);
    this.buttonRef = React.createRef();
  }

  componentDidUpdate(prevProps) {
    if (this.props.url === '' && prevProps.url = "old-url") {
      console.log('target element: ', this.buttonRef.current.firstChild)

      // This doenst work if not in a setTimeout
      // this.buttonRef.current.firstChild.focus();
      setTimeout(() => {
        this.buttonRef.current.firstChild.focus();
      }, 1);
    }
  }

What if you set the focus-trap option returnFocusOnDeactivate: false? See https://github.com/davidtheclark/focus-trap#focustrap--createfocustrapelement-createoptions

returnFocusOnDeactivate is already set to false. returnFocusOnDeactivate works great except sometimes the button that triggered the modal no longer exists by the time the modal is closed, in which case I need to focus on a different button.
Also modals are controlled by the url hash so they're kinda 'global' rather than being within a component in the normal React way. This is why there's some weirdness in componentDidUpdate. To be honest this has turned out to be really problematic and will need refactoring so no worries if there's no easy solution. Turns out breaking React's component modal was a bad call, who knew? ;-)

authors: davidtheclark jameschetwood

repo: davidtheclark/focus-trap-react


text:
Improved graph-tool installation instructions

To install graph-tool via conda the actual commands are:
conda create --name gt -c conda-forge graph-tool
conda activate gt

The command currently given in the instruction will not work.
It might be good to inform users that they can install (without compilation) using homebrew in MacOS, and also in Ubuntu/Debian. The installation instructions are here: https://git.skewed.de/count0/graph-tool/-/wikis/installation-instructions

I recommend the following instructions.  FWIW, I tested these on my machine (a Mac) and successfully ran conftest.py.

schist has the following dependencies:

numpy
scipy
anndata
pandas
scanpy
graph-tool

Conda users can create an environment with all of them using the following commands:
conda create -n schist -c conda-forge -c bioconda numpy scipy anndata pandas graph-tool scanpy
conda activate schist

Non-conda users should obtain graph-tool via either apt-get or homebrew (see the graph-tool installation instructions), and then use pip for the remaining dependencies.
Since schist itself is not yet available on PyPI or anaconda.org, you must clone the git repo and install it from source.  Once you've installed the above dependencies, run the following commands:
git clone https://github.com/dawe/schist.git
cd schist
pip install .

Thanks for for the comments, I will update installation instructions.
I was thinking to provide a conda recipe for schist, but I never did that, give me time to study.

authors: count0 dawe stuarteberg

repo: dawe/schist


text:
Feature request: Highlight lines

It would be cool to be able to select lines to highlight similar to how you can do it in the github viewer.
The interaction would probably be similar. Click to the left of a line to highlight it, and if we're holding the Shift key down we can click another line to select a range of lines.
Codemirror supports this via the Active Line option, so it should be feasible.
I would also be down to work on a PR for this. Should be fun.

Unless I am mistaken, you can already highlight the lines of the Carbon editor, and those highlights even remain in the image after saving as well  . . . Is this what you mean? 🙂

Oh haha. I guess it kinda does. It's a bit finicky if you want to highlight the entire line, and it looks like it the highlight color renders funky once the editor is not in focus so it makes it a bit hard to read (for the record I haven't tested if using the Active Line option in code mirror is more readable). This also doesn't really let me select more than one selected range.

I think if using the Active Line option makes it look better this feature would be worth it. I'll experiment and post and update here to see if it's better.

@jalvarado91 please do! Thanks 🙂

After looking into this more, you are already able to highlight the lines, and then click away and the highlighting remains. You can also click save and the highlights will be in the PNG. Closing . . .

repo: dawnlabs/carbon


text:
Installation instructions for newbies

It'd be nice to add some installation instructions for newbies. Maybe it's common knowledge among Clojurists that to install a lein script, you need to add it to ~/.m2 but for those that don't, some research is required.

@martimatix
Could you give a more concrete example, please?  I'm not understanding what you mean by "installation instructions".
Do you mean something like this:
https://purelyfunctional.tv/guide/how-to-install-clojure/

Suppose you're brand new to the world of Clojure and Clojurescript. You've read about the new re-frame design on reddit. You start reading about re-frame and you're sold. "I want to get started," you think to yourself. You go to the Start Coding page and see that there's a template to scaffold your project (this repo).
You're on a Mac and have run brew install clojure but don't really know much about Clojure or Leiningen.
You click on the link to this repo and start reading the readme. You read that you need to run lein new re-frame <project-name> but how do you install it? That is to say, if I run git clone https://github.com/day8/re-frame-template.git in any old directory, Leiningen won't be able to find it.
For the sake of Clojurescript newbies, I was suggesting that something like this be added:

clone this repo
copy src/leiningen/new to ~/.m2/repository
(by the way, is there an easier way?)

That was a long winded way of describing the situation that I found myself in. I ended up having to search for installing Leiningen templates and found this page.
Going through this experience prompted me to open this issue.

Never mind, turns out that you don't need to clone this repo at all. 🤦♂
lein new re-frame <project-name> is all you need.

I'm glad you worked it out.
But you got me thinking, and I added this to the README https://github.com/day8/re-frame-template/blob/master/README.md#before-you-start.


text:
Feature: limitate number of call performed for one job

A job can be cancelled after a precise number of calls.
All existings tox tests are ok. No new test, except one doctest in schedule/init.py
# print text every seconds at most 5 times :
schedule.every().seconds.do(lambda : print('hello')).at_most(5)

Currently, the method identificator Job.at_most() is not so readable. Ideas ?
Can Job.is_cancelled() method be used for detect other cases of cancellation ?

Coverage decreased (-1.52%) to 98.48% when pulling ded076b69b59e45cb0c58433ceed24e326bbcf2e on Aluriak:master into 4282b887aafed6c6be14ecc78e4886d86a27333a on dbader:master.

Coverage decreased (-1.52%) to 98.48% when pulling ded076b69b59e45cb0c58433ceed24e326bbcf2e on Aluriak:master into 4282b887aafed6c6be14ecc78e4886d86a27333a on dbader:master.

I appreciate the PR but I think this would be better handled by the job function itself (keep a counter and then CancelJob when necessary). That way we can keep the base functionality lean.
Thank you – I'm closing this so we can keep the tracker clean of loose ends. Feel free to follow up (or make a separate issue) with more questions/discussion 😃

@Aluriak you are genius. Thank you very much man, I love you, I've been struggling with this over my lovely weekend. You just made my day, you're my bro now.
@dbader please see #142 regarding your job counter suggestion – it is not working.

@heisen273 no problemo ; reuse this patch at will :)

authors: Aluriak coveralls dbader heisen273

repo: dbader/schedule


text:
Upgrade MSSQL Tools Service

Currently, mssql-cli relies on 2.1 builds of the Sql Tools (which, in turn, require libssql1.0. This causes issues for users using recent releases of libssl (especially those on Ubuntu 19.04).
This PR updates the packages to more recent releases.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: CrshOverride sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for the PR @CrshOverride, but unfortunately we didn't have the bandwidth to test and merge this. However we did updated the SqlToolsService with the 0.16.0 release.

authors: CrshOverride msftclas pensivebrian

repo: dbcli/mssql-cli


text:
ClickHouse : download driver fail

System information:

Operating system : 5.8.0-41-generic #46~20.04.1-Ubuntu
DBeaver version : Version 7.3.4.202101311042
Additional extensions

Describe the problem you're observing:
When I setup a connection to clickhouse-server ( docker )
docker run --rm -e CLICKHOUSE_DB=my_database -e CLICKHOUSE_USER=username -e CLICKHOUSE_PASSWORD=password -p 9001:9000/tcp yandex/clickhouse-server

Dbeaver ask to download driver and fail :
Driver file download failed.
Do you want to retry?

Reason:
Can't open 'https://repo1.maven.org/maven2/ru/yandex/clickhouse/clickhouse-jdbc/${revision}/clickhouse-jdbc-${revision}.jar': Not Found

Thank you all :+1:

If I change the version of the driver to 0.2.4 it work
but 0.2.5 do not work

Same problem for me today.

thanks for the report

It is a bug in Maven artifact version resolution. That's quite an unusual artifact definition.

Fixed

verified

thanks for the report

Thanks, helped a lot.

authors: gansiorml raphaelauv serge-rider uslss zverev-iv

repo: dbeaver/dbeaver


text:
Unable to set result fetch size in 22.2.5

Description
Unable to set result fetch size in 22.2.5

Before it was like:

Maybe there is a setting somewhere but I can't find it - anyway this possibility to set fetch size on the fly is very important and handy to me.
DBeaver Version
CE 22.2.5
Operating System
win 10
Steps to reproduce
Just select on table with more than 200 rows

Hello @DamijanD
What database do you use?

This was tested with oracle (jdbc)

You can use the "Configure" button to go to the preferences.

Yes I can -  but I don't know if this is by design or it is a bug that this option is now removed from status bar. Anyway in my humble opinion this was very useful to have it here where you need it and (if this is by design) I would say that this change is step back

Also for me this change is a step back, i usually change the fetch size many times at once executing query.
I ve to return to the 22.2.3

Agree.
I'm used to changing the number of fetch with one click. Now need to go somewhere else to do this.
Also, I think it's UI bug because of this

please bring back the 'fetch' buttons!

Hello everyone. You can vote with any emoji on the main topic to help us understand - that this feature was essential for you.

I use the FetchAll hundreds of times a day.   I found it nested in the Navigate menu (doesn't seem like navigation..shouldn't it be data? or even 'sql editor')  and that told me the hot-key, but it still seems like a button worth having.  They could just have it be tied to the 'count rows' button that's there.   Clicking it again would fetch the rows it just counted.  ;)

please bring back the 'fetch' buttons!

The disappearance of fetch all is incredibly annoying. definitely going to revert to the previous version until this is addressed.

Thank you for bringing back the button fetch all rows!

authors: D-750 DamijanD Gabriel-Voyager LonwoLonwo Olga-B apdma marc-hawke qilinsanshao


text:
table column name autocomplete does not work after updating to latest version 22.3.2

Description
autocomplete table column names does not work at all after updating version to 22.3.2 latest version
DBeaver Version
22.3.2
Operating System
Windows 11 Home
Database and driver
PostgreSQL [12.11
PostgreSQL 12.11 on x86_64-pc-linux-gnu, compiled by gcc (GCC) 7.3.1 20180712 (Red Hat 7.3.1-12), 64-bit]
PostgreSQL JDBC Driver [42.5.0]
Steps to reproduce
start typing a select query on the script and it normally shows the suggestions for table column names, but after updating to version 22.3.2 it does not show the suggestions

Additional context
No response

Hello @buddhiprab
Can't reproduce it for now.

Is the "broadcast" table from the default SQL Editor schema/catalog?

Ok, I see. It doesn't work if SQL Editor has another schema/catalog in the context.

#10955

Global search enabled.

authors: LonwoLonwo buddhiprab


text:
Snowflake $$ delimiter not parsing correctly

Description
see issue #17700
DBeaver Version
Community Version 23.0.5.202305211610
Operating System
Windows 10 64 bit
Database and driver
Snowflake jdbc 3.13.6
Steps to reproduce
Using this sql:
EXECUTE IMMEDIATE
$$
DECLARE
P_ACCOUNT VARCHAR DEFAULT 'X';
P_TIN VARCHAR DEFAULT 'Y';
P_NAME VARCHAR DEFAULT 'Z';
P_CLAIM VARCHAR DEFAULT 'Z';
P_DOS DATE DEFAULT '9/7/2022';
p_ALLOWED number default 290;
CLEANFNAME VARCHAR;
CLEANLNAME VARCHAR;
CLEANCLAIM VARCHAR;
SOUNDEXFNAME VARCHAR;
SOUNDEXLNAME VARCHAR;
V_BU VARCHAR;
res RESULTSET;
BEGIN
V_BU := 'DX';
CLEANFNAME := CAST(UPPER(REGEXP_REPLACE(SPLIT_PART(:P_NAME, ' ', 1), '[^[:alpha:]]', '')) AS VARCHAR(50));
CLEANLNAME := CAST(UPPER(REGEXP_REPLACE(SPLIT_PART(:P_NAME, ' ', 2), '[^[:alpha:]]', '')) AS VARCHAR(50));
CLEANCLAIM := CAST(REGEXP_REPLACE(:P_CLAIM, '[^[:alnum:]]', '') AS VARCHAR(50));
SOUNDEXFNAME := SOUNDEX(UPPER(REGEXP_REPLACE(SPLIT_PART(:P_NAME, ' ', 1), '[^[:alpha:]]', '')));
SOUNDEXLNAME := SOUNDEX(UPPER(REGEXP_REPLACE(SPLIT_PART(:P_NAME, ' ', 2), '[^[:alpha:]]', '')));

CREATE OR REPLACE
TEMP TABLE results ( BU varchar,
INV_NUM varchar,
CLEANLNAME varchar,
CLEANFNAME varchar,
SOUNDEXF varchar,
CLEANCLAIM varchar,
DOS date,
BILLED_AMT number,
PAID_AMT number,
ADJ_AMT number,
BALANCE number ,
MATCH_TYPE varchar,
PF_MATCH number);
if (V_BU = 'PT') then
insert into results
SELECT
BU,
INVOICEHEADER_ID INV_NUM,
CLEANLNAME,
CLEANFNAME,
SOUNDEXF,
CLEANCLAIM,
DOS ,
SUM(AMTALIGNBILL) BILLED_AMT,
SUM(AMTPAYMENT) PAID_AMT,
SUM(AMTADJUSTMENT)ADJ_AMT,
SUM(BALANCE) BALANCE,
MATCH_TYPE, PF_MATCH
FROM
(
SELECT
IH.INVOICEHEADER_ID,
ID.INVOICEDETAIL_ID,
UPPER(REGEXP_REPLACE(R.PATIENTNAMELAST, '[^[:alpha:]]', '')) CLEANLNAME,
UPPER(REGEXP_REPLACE(R.PATIENTNAMEFIRST, '[^[:alpha:]]', '')) CLEANFNAME,
SOUNDEX(REGEXP_REPLACE(R.PATIENTNAMEFIRST, '[^[:alpha:]]', '')) SOUNDEXF,
REGEXP_REPLACE(IFNULL(CF.CLAIMNUMBER, R.CLAIMNUMBER ) , '[^[:alnum:]]') CLEANCLAIM,
CAST(ID.SERVICEDATETIME AS DATE) AS DOS,
ID.CPTCODE,
ID.AMTALIGNBILL,
ID.AMTPAYMENT,
ID.AMTADJUSTMENT,
ID.AMTALIGNBILL - ( ID.AMTPAYMENT + ID.AMTADJUSTMENT + ID.AMTREFUND + ID.AMTCHARGEBACK ) BALANCE ,
CASE
WHEN RD.INVOICEDETAIL_ID IS NULL THEN 'PT'
ELSE 'PT-R'
END BU,
'ACCOUNT' MATCH_TYPE,0 PF_MATCH
FROM
PROD_ENT_WAREHOUSE.ALIGN_DBO.INVOICEHEADER IH
JOIN PROD_ENT_WAREHOUSE.ALIGN_DBO.INVOICEDETAIL ID
ON
IH.INVOICEHEADER_ID = ID.INVOICEHEADER_ID
JOIN PROD_ENT_WAREHOUSE.ALIGN_DBO.REFERRAL R
ON
IH.REFERRAL_ID = R.REFERRAL_ID
LEFT JOIN PROD_ENT_WAREHOUSE.CLAIMFEED_DBO.CLAIMFEED_ENTERPRISE CF
ON
R.ALIGNCLAIMFEEDID = CF.ALIGNCLAIMFEEDID
AND R.CLAIMFEEDCOMPANYID = CF.ALIGNPAYORID
LEFT JOIN PROD_ENT_WAREHOUSE.REVENUECYCLE_DBO.RETROCLAIMDTL_HISTORY RD
ON
RD.INVOICEDETAIL_ID = ID.INVOICEDETAIL_ID
AND RD.ALIGNLINESTATUS = 'A'
WHERE
IH.INVOICEHEADER_ID =try_to_number( :P_ACCOUNT)
AND DOS = :P_DOS
union
SELECT
IH.INVOICEHEADER_ID,
INVOICEDETAIL_ID,
UPPER(REGEXP_REPLACE(RRM.PATIENTNAMELAST, '[^[:alpha:]]', '')) CLEANLNAME,
UPPER(REGEXP_REPLACE(RRM.PATIENTNAMEFIRST, '[^[:alpha:]]', '')) CLEANFNAME,
SOUNDEX(REGEXP_REPLACE(RRM.PATIENTNAMEFIRST, '[^[:alpha:]]', '')) SOUNDEXF,
REGEXP_REPLACE(IFNULL(CF.CLAIMNUMBER, R.CLAIMNUMBER), '[^[:alnum:]]') CLEANCLAIM,
CAST(ID.SERVICEDATETIME AS DATE) AS DOS,
ID.CPTCODE,
ID.AMTALIGNBILL,
ID.AMTPAYMENT,
ID.AMTADJUSTMENT,
ID.AMTALIGNBILL - ( ID.AMTPAYMENT + ID.AMTADJUSTMENT + ID.AMTREFUND + ID.AMTCHARGEBACK ) BALANCE,
'PT' BU,
'NAME' MATCH_TYPE,0 PF_MATCH
FROM
PROD_ENT_WAREHOUSE.ALIGN_DBO.INVOICEHEADER IH
JOIN PROD_ENT_WAREHOUSE.ALIGN_DBO.REFERRAL RRM ON IH.REFERRAL_ID = RRM.REFERRAL_ID
JOIN PROD_ENT_WAREHOUSE.ALIGN_DBO.INVOICEDETAIL ID ON IH.INVOICEHEADER_ID = ID.INVOICEHEADER_ID
JOIN PROD_ENT_WAREHOUSE.ALIGN_DBO.REFERRAL R ON IH.REFERRAL_ID = R.REFERRAL_ID
LEFT JOIN PROD_ENT_WAREHOUSE.CLAIMFEED_DBO.CLAIMFEED_ENTERPRISE CF ON R.ALIGNCLAIMFEEDID = CF.ALIGNCLAIMFEEDID AND R.CLAIMFEEDCOMPANYID = CF.ALIGNPAYORID
WHERE
CLEANLNAME = :CLEANLNAME
AND CLEANFNAME = :CLEANFNAME
AND ID.SERVICEDATETIME = :P_DOS
UNION
SELECT
id.INVOICEHEADER_ID,
ID.INVOICEDETAIL_ID,
UPPER(REGEXP_REPLACE(RH.PATIENTLNAME, '[^[:alpha:]]', '')) CLEANLNAME,
UPPER(REGEXP_REPLACE(RH.PATIENTFNAME, '[^[:alpha:]]', '')) CLEANFNAME,
SOUNDEX(REGEXP_REPLACE(RH.PATIENTFNAME, '[^[:alpha:]]', '')) SOUNDEXF,
REGEXP_REPLACE(RH.CLAIMNUMBER, '[^[:alnum:]]') CLEANCLAIM,
CAST(ID.SERVICEDATETIME AS DATE) AS PT_DOS,
ID.CPTCODE,
ID.AMTALIGNBILL,
ID.AMTPAYMENT,
ID.AMTADJUSTMENT,
ID.AMTALIGNBILL - ( ID.AMTPAYMENT + ID.AMTADJUSTMENT + ID.AMTREFUND + ID.AMTCHARGEBACK ) BALANCE,
'PT-R' BU,
'NAME' MATCH_TYPE,0 PF_MATCH
FROM
PROD_ENT_WAREHOUSE.REVENUECYCLE_DBO.RETROCLAIMHDR_HISTORY RH
JOIN  PROD_ENT_WAREHOUSE.REVENUECYCLE_DBO.RETROCLAIMDTL_HISTORY RD ON RD.RETROCLAIMHISTORYID = RH.RETROCLAIMHISTORYID AND RD.ALIGNLINESTATUS = 'A'
JOIN PROD_ENT_WAREHOUSE.ALIGN_DBO.INVOICEDETAIL ID ON RD.INVOICEDETAIL_ID = ID.INVOICEDETAIL_ID
--JOIN PROD_ENT_WAREHOUSE.ALIGN_DBO.INVOICEHEADER IH ON IH.INVOICEHEADER_ID = ID.INVOICEHEADER_ID
WHERE
CLEANLNAME = :CLEANLNAME
AND CLEANFNAME = :CLEANFNAME
AND PT_DOS = :P_DOS
)
GROUP BY
INVOICEHEADER_ID,
CLEANLNAME,
CLEANFNAME,
SOUNDEXF,
CLEANCLAIM,
DOS,
BU,
MATCH_TYPE ;
END IF;
END;
$$;
excution throws error:
If I remove the semicolon at line 156 (make "END IF;" = just "END IF" It will compile
If I switch $$ quotes from code block to Sting literal, it will compile.
Selecting all text and trying throws error.
So Snowflake has no problem with the code, DBeaver is not parsing correctly again
Additional context
No response

dbeaver-debug.log
Log files

Thank you for bug report.


text:
[CUBRID] Extend navigator to display 'db_server' information

Extend CUBRID plugin to display 'db_server' information on the navigator
The db_server object in the CUBRID catalog is connection information for a remote database server supported by CUBRID 11.2 or later (database link)

Hello @longhaseng52
Please provide a link to a db_server info if you have it.

Hello @LonwoLonwo, here is the link of db_server info.
https://www.cubrid.org/manual/en/11.3/sql/catalog.html#id57


text:
Redis Hash or Set values throws "Unsupported key type: none"

DBeaver fails to show Redis HASH and SET values.
When I connect to Redis v5.0 Server via DBeaver EE (Trial), It connects and lists all available databases and keys successfully. However, HASH and SET data cannot be opened.
Does it not support HASH and SET (if not when) or is a bug? Redis + Elasticsearch will be my buying reasons.
Query execution failed

Reason:
Unsupported key type: none

Target: Redis v5/v4
OS: Debian 9 x64
DBeaver: 5.2.2 EE

DBeaver supports both Hash and Set values (as they are the most standard key types). However we didn't yet tested this with Redis 5. But it works fine with standalone Redis 4.x.
Do you use standalone Redis or some kind of service (like AWS Elastic Cache)?

I'm using standalone Redis.

Just tested on Redis 5 and hash/set keys seems to be working ok.
Could you provide a bit more details?
Some steps to reproduce would be helpful.
Also, if possible, attach error/debug logs (https://github.com/dbeaver/dbeaver/wiki/Log-files)

I have the similar issue.
Redis server v=4.0.9
DBeaver EE 5.3.3
Exception Stack Trace:
org.jkiss.dbeaver.model.exec.DBCException: Unsupported key type: none
	at com.dbeaver.ee.redis.exec.RedisKeyStatement.executeStatement(RedisKeyStatement.java:68)
	at com.dbeaver.ee.redis.model.RedisKey.readData(RedisKey.java:247)
	at org.jkiss.dbeaver.ui.controls.resultset.ResultSetJobDataRead.lambda$0(ResultSetJobDataRead.java:103)
	at org.jkiss.dbeaver.model.DBUtils.tryExecuteRecover(DBUtils.java:1633)
	at org.jkiss.dbeaver.ui.controls.resultset.ResultSetJobDataRead.run(ResultSetJobDataRead.java:101)
	at org.jkiss.dbeaver.model.runtime.AbstractJob.run(AbstractJob.java:101)
	at org.eclipse.core.internal.jobs.Worker.run(Worker.java:63)

Session Data:
java.version=1.8.0_152
java.vendor=Oracle Corporation
BootLoader constants: OS=macosx, ARCH=x86_64, WS=cocoa, NL=pl_PL
Framework arguments:  -keyring /Users/user/.eclipse_keyring
Command-line arguments:  -os macosx -ws cocoa -arch x86_64 -keyring /Users/user/.eclipse_keyring

I hope it will be helpful

This bug was fixed in 6.2. Please verify.

Fixed for "folder" keys (they are not real keys but fake ones for hierarchy organization)

verified

authors: dariuszjastrzebski gencer serge-rider uslss


text:
Empty Entity column in exec plan for CTE in Postgres

This is reopen of #2308.
@uslss, you can explain the following simple query to reproduce
with
p as (select 1),
f as (select 2),
g as (select 3)
select *
from p, f,g

Thanks for the bug report

verified

authors: offstate serge-rider uslss


text:
[BUG] Postrges Table DDL ignoring UNLOGGED status

Creating a table (or altering a table to) UNLOGGED is not specified in table DDL.

Thanks for the bug report

verified

authors: serge-rider uslss wiwwo


text:
DBeaver updates vs linux repository

DBeaver updates are too often to download all time these updates directly. Please, consider creating repositories for ubuntu and other distros, so it will be available to update in one click

Probably this is what you are looknig for: https://github.com/dbeaver/dbeaver/issues/1667

thanks a lot

authors: icegood tchibatron


text:
#23013 hide local clients for RisingWave

No local client on the connection page. And in the navigator tree. Please check databases, schemas, tables.
No Restore, Backup, Backup All, Execute script

verified

authors: LonwoLonwo uslss


text:
Update HRRR weather model archive location to new NOAA AWS API

The HRRR weather model has (finally!) started to be archived by NOAA. Here's a link to the original University of Utah repository with the explanation of the new data systems. NOAA is storing the GRIB files on AWS, while Brian Baylock's group at UU is storing them as ZARR files. However, according to their description of this dataset it sounds like they won't be storing all of the available data, so probably we simply need to switch to the NOAA AWS archive. I think file I/O should be pretty similar to what it is now.
NOAA has a readme on accessing the data on S3 on Github: https://github.com/awslabs/open-data-docs/tree/main/docs/noaa/noaa-hrrr

addressed in PR #379

authors: dbekaert jlmaurer

repo: dbekaert/RAiDER


text:
Build Expressions from Anonymous Types

There are instances where I need to make projections in Entity Framework. However, projections are anonymous types. For example:
Context.Set<Person>().Select(e=> new { e.Id, e.Code, e.FirstName, e.LastName }).Where(<pass_expression_here>);
Is it possible to build expressions when the source is an anonymous type?

Hi @whizkidwwe1217,
Thanks for using the project. 😄
That's a very intriguing question. 🤔 I'll have a look and keep you posted on what I find.
Cheers.

repo: dbelmont/ExpressionBuilder


text:
SonarCloud - Analizar proyecto Neurodegenerative-Disease-Detection

Se analizará el proyecto Neurodegenerative-Disease-Detection con SonarCloud.

Análisis en SonarCloud
Se añadirá un nuevo proyecto desde la opción "Analyze new project". Después se escogerá el proyecto y se dará a "Set Up".

Se activará la opción de análisis automático en "Administration"> "Analysis Method":

Se añadirá la extensión de python notebook para que lo reconozca como python desde Administration > General Settings > Languages > Python.

Resultado


text:
Error in masks = masks.reshape(-1, height, width)

Hi, I previously trained a dataset where the images were not all the same size, they all ranged from 950 to 1000 in both height and width and I had no problem. Now I am using images with larger sizes (between 1000 and 1200 both in height and width) and it is showing me a problem with the reshaping of the masks. I've already spent quite a bit of time on labeling so it would be helpful to know if anyone knows if there is any way to fix it without having to change the dataset. The error is as follows:
Resuming training, loading weights/yolact_im700_54_800000.pth...
Ignoring "Error(s) in loading state_dict for Yolact:
size mismatch for prediction_layers.0.conf_layer.weight: copying a param with shape torch.Size([243, 256, 3, 3]) from checkpoint, the shape in current model is torch.Size([33, 256, 3, 3]).
size mismatch for prediction_layers.0.conf_layer.bias: copying a param with shape torch.Size([243]) from checkpoint, the shape in current model is torch.Size([33]).
size mismatch for semantic_seg_conv.weight: copying a param with shape torch.Size([80, 256, 1, 1]) from checkpoint, the shape in current model is torch.Size([10, 256, 1, 1]).
size mismatch for semantic_seg_conv.bias: copying a param with shape torch.Size([80]) from checkpoint, the shape in current model is torch.Size([10])."
Begin training!
Traceback (most recent call last):
File "train.py", line 504, in
train()
File "train.py", line 270, in train
for datum in data_loader:
File "/usr/local/lib/python3.7/dist-packages/torch/utils/data/dataloader.py", line 435, in next
data = self._next_data()
File "/usr/local/lib/python3.7/dist-packages/torch/utils/data/dataloader.py", line 475, in _next_data
data = self._dataset_fetcher.fetch(index)  # may raise StopIteration
File "/usr/local/lib/python3.7/dist-packages/torch/utils/data/_utils/fetch.py", line 44, in fetch
data = [self.dataset[idx] for idx in possibly_batched_index]
File "/usr/local/lib/python3.7/dist-packages/torch/utils/data/_utils/fetch.py", line 44, in
data = [self.dataset[idx] for idx in possibly_batched_index]
File "/root/yolact/data/coco.py", line 94, in getitem
im, gt, masks, h, w, num_crowds = self.pull_item(index)
File "/root/yolact/data/coco.py", line 150, in pull_item
masks = masks.reshape(-1, height, width)
ValueError: cannot reshape array of size 9138360 into shape (841,866)

Hi, I previously trained a dataset where the images were not all the same size, they all ranged from 950 to 1000 in both height and width and I had no problem. Now I am using images with larger sizes (between 1000 and 1200 both in height and width) and it is showing me a problem with the reshaping of the masks. I've already spent quite a bit of time on labeling so it would be helpful to know if anyone knows if there is any way to fix it without having to change the dataset. The error is as follows:
Resuming training, loading weights/yolact_im700_54_800000.pth... Ignoring "Error(s) in loading state_dict for Yolact: size mismatch for prediction_layers.0.conf_layer.weight: copying a param with shape torch.Size([243, 256, 3, 3]) from checkpoint, the shape in current model is torch.Size([33, 256, 3, 3]). size mismatch for prediction_layers.0.conf_layer.bias: copying a param with shape torch.Size([243]) from checkpoint, the shape in current model is torch.Size([33]). size mismatch for semantic_seg_conv.weight: copying a param with shape torch.Size([80, 256, 1, 1]) from checkpoint, the shape in current model is torch.Size([10, 256, 1, 1]). size mismatch for semantic_seg_conv.bias: copying a param with shape torch.Size([80]) from checkpoint, the shape in current model is torch.Size([10])." Begin training!
Traceback (most recent call last): File "train.py", line 504, in train() File "train.py", line 270, in train for datum in data_loader: File "/usr/local/lib/python3.7/dist-packages/torch/utils/data/dataloader.py", line 435, in next data = self._next_data() File "/usr/local/lib/python3.7/dist-packages/torch/utils/data/dataloader.py", line 475, in _next_data data = self._dataset_fetcher.fetch(index) # may raise StopIteration File "/usr/local/lib/python3.7/dist-packages/torch/utils/data/_utils/fetch.py", line 44, in fetch data = [self.dataset[idx] for idx in possibly_batched_index] File "/usr/local/lib/python3.7/dist-packages/torch/utils/data/_utils/fetch.py", line 44, in data = [self.dataset[idx] for idx in possibly_batched_index] File "/root/yolact/data/coco.py", line 94, in getitem im, gt, masks, h, w, num_crowds = self.pull_item(index) File "/root/yolact/data/coco.py", line 150, in pull_item masks = masks.reshape(-1, height, width) ValueError: cannot reshape array of size 9138360 into shape (841,866)

I have same poroblem. did you solve it?

@MHasanlou1  I found out that the problem came from the COCO JSON files with the annotations. I made the skins using VGG Image Annotation and noticed that the COCO JSON format you export the annotations in is not correct. The image_id saves them as text and they must be integers, in addition, the points of the masks must be saved as a list of lists. You can check your JSON files to verify that they are in the correct format (I leave a picture of how the annotations are with the segmentation).

I have the same problem but my JSON files are in the correct format

repo: dbolya/yolact


text:
Multiple occurence of lightbox on one page

I am using the 'showGalleryFromArray'  approach,  I am using an image instead of words as the button and it works great, but I need to have multiple clickable "buttons" on one page, each with it's own array of images.  The problem is, when you click on the first one, both slide shows open - both the one you clicked on and the other one on the page.
Is there a way to make each 'showGalleryFromArray' unique so that it will only open when it is clicked. I tried using role="button1", and role="button2" but that didn't work
http://thefreshflowermarketevents.com/sandbox/wedding-gallery.html

Perfect.  Thank you so much!
Vicki S Jones
Willow Circle Web Services
vicki@willowcirclewebservices.com

On Jul 9, 2018, at 4:09 AM, Damir Brekalo notifications@github.com wrote:
On site provided you are binding click handler twice to same selector - you should alter your selectors (notice added gallery1 and gallery2 classes)
$('.showGalleryFromArray.gallery1').on('click', function() {
$.SimpleLightbox.open({
items: [...]
});
});
$('.showGalleryFromArray.gallery2').on('click', function() {
$.SimpleLightbox.open({
items: [...]
});
});
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/dbrekalo/simpleLightbox/issues/7#issuecomment-403429272, or mute the thread https://github.com/notifications/unsubscribe-auth/AmawUQ8gbUe2c3TwJOY5OeLjh4VDFNOSks5uEyvbgaJpZM4Uo9eH.

repo: dbrekalo/simpleLightbox


text:
[CT-788] Support disabling metrics + exposures

Pretty simple:
# models/specific_file.yml

metrics:
  - name: disabled_metric
    config:
      enabled: false

exposures:
  - name: disabled_exposure
    config:
      enabled: false

# dbt_project.yml

metrics:
  +enabled: false

exposures:
  +enabled: false

Expectations:

Moved to disabled section of manifest.json
Needs to work with partial parsing
If a disabled metric is referenced via metric(), it should raise an error at the end of parsing (just as ref() or source() do)

Grouping these together because I believe it would require touching the same code paths!

Need to make sure we loop over the right things anywhere we're looking at enabled nodes to ensure metrics/exposures are included.

repo: dbt-labs/dbt-core


text:
[ADAP-947] [Bug] Dynamic table copy grant is not working

Is this a new bug in dbt-snowflake?

[X] I believe this is a new bug in dbt-snowflake
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
with copy grant set in dbt_project.yml, when running dbt build --select some_dynamic_table_model --full-refresh, the grants that was given manually have been removed.
Expected Behavior
with copy grant set in dbt_project.yml, the existing grants on the dynamic table should be remain as it is.
Steps To Reproduce

create snowflake dynamic table model and run it
`{{
config(
materialized='dynamic_table',
snowflake_warehouse = target.warehouse,
target_lag = '5 minutes',
)
}}

select * from {{ref('some_dynamic_table_model')}} 2. manually grant to snoflake dynamic tablegrant select on some_database.some_schema.some_dynamic_table_model to role some_role;3. set copy grant config# dbt_project.yml
models:
+copy_grants: true4.dbt build --select some_dynamic_table_model --full-refresh5. check snowflake dynamic table grantsshow grants on table some_database.some_schema.some_dynamic_table_model;`
Relevant log output
No response
Environment
- OS:Ventura 13.6
- Python:3.11.5
- dbt-core:1.6.5
- dbt-snowflake:0.21.3

Additional Context
No response

@zztan-ipg thanks for opening the issue! My first thought was "oh shoot, we totally missed that!", until I did some digging.
I'm going to confirm directly with some Dynamic Table folks at Snowflake, but my understanding based on the docs is that the COPY_GRANTS parameter is not supported for a CREATE DYNAMIC TABLE statement, where it is supported for other table types (e.g. CREATE TABLE)

Hey @dataders just wondering if you got the reply from Snowflake dynamic table folks yet?

thanks for the bump @zztan-ipg! I reached out to @sortalongo and team and the response was

COPY GRANTS is on the release train, but its delayed and going through a behavior change process. Opt-in period should start mid Nov

my advice would be to reach out to your Snowflake reps and tell them you're interested!

Noted with thanks!

Relabeling this as a feature request since dbt is not behaving incorrectly. We will need to confirm that it works once its available within Snowflake

thanks for the bump @zztan-ipg! I reached out to @sortalongo and team and the response was

COPY GRANTS is on the release train, but its delayed and going through a behavior change process. Opt-in period should start mid Nov

update that is looks as if COPY GRANTS is now at least publicly available, and maybe GA?

authors: dataders dbeatty10 zztan-ipg

repo: dbt-labs/dbt-snowflake


text:
when columns removed for an incremental model

What are you changing in this pull request and why?
If we remove a column from an incremental model and on_schema_change: ignore then execute dbt run, it results in an error. However, based on the documentation, it seems that dbt run should not fail in this scenario. I find this confusing.
##Link of the page
https://docs.getdbt.com/docs/build/incremental-models#default-behavior

thanks for opening this PR, @RatandeepSharma ! appreciate your contribution to the dbt Labs docs!
@joellabes : i think it's ok to approve and merge this change but want to double-check with you first. do you agree? or have any suggestions? thanks!

Hey @RatandeepSharma wanted to check in to see if you can make those changes my teammate suggested?

Hey @mirnawong1 - I've implemented the changes as recommended.

authors: RatandeepSharma mirnawong1 nghi-ly


text:
실행해도 무반응인데 혹시 무슨 문제인지 아시나요?

내용 그대로 실행해도 반응이 없어요.....
히토미 카피 쓰다가 오류발생 경로 : 메타데이터 다운로더  이런식으로 나와서 코로모 다운로더 쓰려고 하는데 더블클릭해도 아무 반응이 없네요 혹시 해결방법 아시나요?
그리고 프로그램 잘 쓰고 있습니다 항상 감사합니다

새로운 폴더를 만들고, 그 폴더에 https://github.com/dc-koromo/koromo-copy/releases/download/1.7.6985.22934/Koromo.Copy.exe 이 파일만 놔둔 채로 실행해주세요
정상적으로 실행된다면 히토미 카피의 다운로드 기록 파일인 log.json도 새로운 폴더에 넣으면 됩니다
히토미 카피하고 Koromo Copy의 데이터들은 log.json을 제외하고는 호환이 안되요

덕분에 해결했습니다 항상 잘 쓰고있어요 감사합니다
https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail
Virus-free.
www.avast.com
https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail
<#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>
2019년 2월 18일 (월) 오후 5:26, koromo notifications@github.com님이 작성:

새로운 폴더를 만들고, 그 폴더에
https://github.com/dc-koromo/koromo-copy/releases/download/1.7.6985.22934/Koromo.Copy.exe
이 파일만 놔둔 채로 실행해주세요
정상적으로 실행된다면 히토미 카피의 다운로드 기록 파일인 log.json도 새로운 폴더에 넣으면 됩니다
히토미 카피하고 Koromo Copy의 데이터들은 log.json을 제외하고는 호환이 안되요
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dc-koromo/koromo-copy/issues/5#issuecomment-464633435,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AthbiodA8rejJKPFByIm324uAwo4fpkeks5vOmPMgaJpZM4bAJtv
.

repo: dc-koromo/koromo-copy


text:
style selection window issues

When I installed this the window to select a citations style was initially blank. I hit "esc" and it went away, then reopened. This time I could pick a style, but there was no button to close the window. Nor did clicking outside the box make it go away. I hit "esc." again but it kept coming back. Finally got it to work by restarting the app a few times... Also don't see a way to invoke the window again if I want to change the style later on?
(Love this by the way - I wish it was a built-in feature of Zotero!)

Thank you.
My guess is that this has something to do with resolution and screen size. Including an image that shows where the style chooser is - bottom left of the Preview tab.

Hi,
I'm having this issue too - MacBook Pro 13" 2021 model - any thoughts as I could really do with the preview pane!
Jim

Screenshot would be helpful.

Screenshot below:

Please check for updates and see if the new version does the trick. Hard
for me to test.
On Fri, Aug 13, 2021 at 5:30 PM jimwillshire @.***>
wrote:

Screenshot below:
[image: Screenshot 2021-08-13 at 22 29 05]
https://user-images.githubusercontent.com/88906679/129420272-1627b80b-c391-4158-b4a8-a92ff836850f.png
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dcartertod/zotero-plugins/issues/6#issuecomment-898729963,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGTHNBHZGPSZ4AWNFJD3DTLT4WFG5ANCNFSM47AS6CWQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

No updates available currently...

What version do you have? I hadn't worked out auto-updating in earlier
versions, so perhaps download this one:
https://github.com/dcartertod/zotero-plugins/raw/main/ZoteroPreview.xpi
Thanks
On Sat, Aug 14, 2021 at 3:36 PM jimwillshire @.***>
wrote:

No updates available currently...
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/dcartertod/zotero-plugins/issues/6#issuecomment-898949815,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGTHNBG7B4733WEGMVDYKP3T43AVFANCNFSM47AS6CWQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

Morning, I've just removed the old version I had, and have installed 2.0 (from your link) and unfortunately still the same behaviour.
Is there supposed to be a button to confirm the selection?

I appreciate the update. I don't have access to OSX so this is difficult
for me to troubleshoot. I do have another idea I can try.
On Mon, Aug 16, 2021 at 4:51 AM jimwillshire @.***>
wrote:

Morning, I've just removed the old version I had, and have installed 2.0
(from your link) and unfortunately still the same behaviour.
Is there supposed to be a button to confirm the selection?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/dcartertod/zotero-plugins/issues/6#issuecomment-899337848,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGTHNBAZ74O3V3LKBT5GRBDT5DGPXANCNFSM47AS6CWQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

This is still a problem with the latest version downloaded from the website.

Hi @timjore - I've just updated and unfortunately it is still an issue

Is there a similar tool or function in Zotero or another add-on that pops-up a window like this (that does not have this problem)? I can review their code to see what they're doing.

Okay...just did a release (I think--github is still a bit of a mystery to me) that might help. I just can't test without a Mac though.

Hi, thanks for the update - no change I'm afraid

New version for Zotero 7 uses the application settings box.

authors: dcartertod jimwillshire kerim timjore

repo: dcartertod/zotero-plugins


text:
slow binning

Since the switch to pd.IntervalIndex instead of passing around the combination of expected_groups and isbin , binning is slow for large arrays.
I opened a pandas issue
The other solution would be to switch to our own flox.Bins structure and use np.searchsorted everywhere (#80)

Fixed by #120 but it would still be nice if Pandas fixed it

repo: dcherian/flox