text:
68 69 Tidy up docstrings in GTFS module

Description

Adds type hinting and standardises Returns and Raises sections of all modules in GTFS module.
Fixes #68
Fixes #69
Motivation and Context

Standardise func and method signatures for sphinx docs build.
Type of change

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

How Has This Been Tested?

Test configuration details:

OS: macos
Python version: 3.9.13
Java version: NA
Python management system: pip

Advice for reviewer

In completing this work, I have noticed that the module needs to be standardised with changes in utils.defence.py, in places ensuring that utility type defences are used and removing some examples of duplicated type checks. I will handle this in a separate issue #157
Checklist:

[X] My code follows the intended structure of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Additional comments

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

Comparison is base (0be811d) 96.53% compared to head (87f4e54) 96.53%.

Additional details and impacted files
@@           Coverage Diff           @@
##              dev     #158   +/-   ##
=======================================
  Coverage   96.53%   96.53%
=======================================
  Files          11       11
  Lines         953      953
=======================================
  Hits          920      920
  Misses         33       33

Flag
Coverage Δ

unittests
96.53% <100.00%> (ø)

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

Files
Coverage Δ

src/transport_performance/gtfs/gtfs_utils.py
100.00% <ø> (ø)

.../transport_performance/gtfs/report/report_utils.py
100.00% <ø> (ø)

src/transport_performance/gtfs/routes.py
93.33% <100.00%> (ø)

src/transport_performance/gtfs/validation.py
96.69% <100.00%> (ø)

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

Hey @CBROWN-ONS , just double checking this one:

On the docstring for _construst_extended_schema_table() (starting at line 16), there are no 'optional' and 'Defaults to' included for the parameters. If we are being super picky we could also update type hinting to specify the return type (list, list).

I don't have any optional or default params for this func. Do you see the same func signature as me:
def _construct_extended_schema_table(
    some_soup: BeautifulSoup, cd_list: list, desc_list: list
) -> tuple:

Hey @CBROWN-ONS , just double checking this one:

On the docstring for _construst_extended_schema_table() (starting at line 16), there are no 'optional' and 'Defaults to' included for the parameters. If we are being super picky we could also update type hinting to specify the return type (list, list).

I don't have any optional or default params for this func. Do you see the same func signature as me:
def _construct_extended_schema_table(
    some_soup: BeautifulSoup, cd_list: list, desc_list: list
) -> tuple:

Other than this, I believe I've implemented everything bar the internal utility _get_response_text , which is so small it documents itself (just needed it to avoid complexity hook).

Great work clearing all of this up so quickly @r-leyshon . I've reviewed your changes and can confirm that all feedback has been acted upon.
In regards to the above comment. This was a mistake on my end sorry. You are correct in stating that there are not any optional/'default to' params in this function.

authors: CBROWN-ONS codecov-commenter r-leyshon


text:
Bug: Can't see links

Not sure when this happens, but I can't display my links. I can go to the links page, it says that I have five links but none is shown.

Use Chrome, works there

From my testing, this seem to be related to firefox.
Tested on plenty of operating systems and for several users.
In all cases, links did not show firefox, but did in chrome
I assume it should work on everything chromium-based, but haven't tested anything that is not chrome

source: gharchive/issue

authors: Herkarl elmaxe niklasvatn

repo: datasektionen/pico

license: MIT

license_type: permissive

license_source: github-api


text:
Multiple installs in namespace mode fail

Currently if we try to install the operator in 2 different namespaces using helm, the second install fails with
Error: rendered manifests contain a resource that already exists. Unable to continue with install: CustomResourceDefinition "cassandradatacenters.cassandra.datastax.com" in namespace "" exists and cannot be imported into the current release: invalid ownership metadata; label validation error: missing key "app.kubernetes.io/managed-by": must be set to "Helm"; annotation validation error: missing key "meta.helm.sh/release-name": must be set to "cass-operator-1606499002"; annotation validation error: missing key "meta.helm.sh/release-namespace": must be set to "cass-operator-1606498846"

The reason is that CRDs are cluster-wide and can't be installed twice. But your helm chart install it no matter what.
helm 3 supports now a folder called crds, see https://helm.sh/docs/chart_best_practices/custom_resource_definitions, and would skip that step if they already exist. This is what should be used instead.
Here are the steps I followed:
$ kubectl create namespace casskop-install1
namespace/casskop-install1 created
$ kubectl create namespace casskop-install2
namespace/casskop-install2 created
$ helm install --generate-name --set clusterWideInstall=false -n casskop-install1 datastax/cass-operator

NAME: cass-operator-1606500664
LAST DEPLOYED: Fri Nov 27 13:11:07 2020
NAMESPACE: casskop-install1
STATUS: deployed
REVISION: 1
TEST SUITE: None
$ helm install --generate-name --set clusterWideInstall=false -n casskop-install2 datastax/cass-operator

Error: rendered manifests contain a resource that already exists. Unable to continue with install: CustomResourceDefinition
"cassandradatacenters.cassandra.datastax.com" in namespace "" exists and cannot be imported into the current release:
invalid ownership metadata; annotation validation error: key "meta.helm.sh/release-name" must equal
"cass-operator-1606500712": current value is "cass-operator-1606500664"; annotation validation error: key
"meta.helm.sh/release-namespace" must equal "casskop-install2": current value is "casskop-install1"

We just stumbled upon the same problem. Would it be fine commenting out the CRD file and installing the helm chart?

Yeah, if your cluster already has the CRD, you can remove it from additional installs.

Migrated to https://github.com/k8ssandra/cass-operator/issues/128

authors: cscetbon jdonenine jimdickinson slayerjain

repo: datastax/cass-operator


text:
[#1546] fix(lakehouse-iceberg): fix create Iceberg table failed if comment is null

What changes were proposed in this pull request?
remove  reserveProperties logic from CreateTableRequest since reserved properties is handled by property system
Why are the changes needed?

when build IcebergTable,  comment equals to null is checked when comment added to properties .

      if (null != comment) {
        icebergTable.properties.putIfAbsent(ICEBERG_COMMENT_FIELD_NAME, comment);
      }

when build CreateTableRequest, the previous implement remove all reserved properties from property map including comment,  and then add comment to map, the bug happens, not checking the null, but this overall  code is redundant

Fix: #1546
Does this PR introduce any user-facing change?
no
How was this patch tested?
add UT

@Clearvive can you please help to review?

source: gharchive/pull-request

repo: datastrato/gravitino


text:
Implementation Replacement

Replaced compile with implementation as it is deprecated in 2018.

Thanks!

repo: datatheorem/TrustKit-Android

license: mit

license_source: bigquery


text:
docs: Update configuration docs with line-numbers keyword

A PR updating the docs for https://github.com/datavzrd/datavzrd/pull/648.

This should've been closed as we solved this differently. I forgot about this PR existing sorry.


text:
能否将Markdown版/网页版打包成压缩包发布一下

GitHub下载整个仓库几百兆遥遥无期，使用SVN和DownGit都无法顺利下载指定文件夹，无法下载完整教程进行学习

或者把所有Markdown里的相对链接改为github图片地址，但是github访问速度不稳定，体验可能还不如挂代理下载压缩包，本地进行学习。网页版教程可能要开启Github的前端生成器才能正确渲染，不然点进去直接显示HTML源码

测试了一下Github的网页发布功能：
简单展示Github如何显示自定义网页
开启 GitHub Pages

新建仓库
左边菜单栏，Setting --> Pages
在 Branch处，将下拉框的 None 设置为 Master，保存

直接访问想要展示的网页文件路径
格式为 https:// [用户名] .github.io/ [仓库名] / [文件路径]
比如这个奇丑的界面 https://claus-fj.github.io/html-demo/css3-text.html

看到你的问题，我帮你想到一个比较好的方式，只需要换一下思路。可以尝试看一下gitee，中国的码云，他可以clone任何的Github网址，到时候在gitee你就可以下载，并且以较快的速度下载，使用github可能会很慢，但是gitee是很快的，你可以考虑一下！ @Claus-fj

authors: Claus-fj Dreaming-future

repo: datawhalechina/free-excel


text:
New modules that may work better

Hi mates, finally I had time to test other modules for Echo and I've to say I'm glad I did that. I hope the author won't argue me, but I suggest you to test:

node-red-contrib-virtual-smart-home
node-red-contrib-alexa-remote2v2

node-red-contrib-virtual-smart-home:  It works by internet.. and we need to consider that Alexa too.. without it can't work to recognize your voice so it's quite useless to have local devices. It it's extremely more compatible than this local module, no more problems about recognize duplicated names, have in list double devices or remove all of them manually from the app if you delete echo, have unworking devices if the list is too long, etc. All these issues are on the side of Echo firmware, not because of the author here/bugs, simply it seems new firmwares of Echo improved the skills compatibility and reduced it with local ones. This module connect to a skill which will simulate vrtual devices, and not just the light bulb.. many more! Another big plus.
node-red-contrib-alexa-remote2v2: This is about "do with Alexa extra things", like make her say something you want (you typed in Node Red), debug what she's dong, add to some words multiple functions (for example create scenarios), etc. Try that too! Interesting. Don't forget it's a bit buggy, but it's something worth to use.
Hope this will be useful for all

You saved me dude! Virtual smart home is awesome!

Thanks for posting this!  I was having trouble with some device that had worked fine for a long time just not working anymore.  I removed it from Alexa and couldn't get it to rediscover anymore. I gave that virtual-smart-home a try and it picked it up instantly, no discovery needed even.

repo: datech/node-red-contrib-amazon-echo


text:
Ubuntu install doesn't work

On my Ubuntu 18.04.1 I've done the following:

Run docker
$pip install datmo
$ git clone https://github.com/shabazpatel/quick-start.git
$cd quick-start
When trying to do '$datmo init' I got the message that datmo command cannot be found. Something is wrong with the installation I guess. Could you help me with this problem?

Can you share the error you get during the installation of datmo? I have tried it on Ubuntu 18.04 and it seems to work

The problem is that I had no errors during the installation, everything seemed to be ok until trying to execute 'datmo init'. I guess that it might be something with my environment and many Python versions installed, my mate has tried to install it and had no problems on the same ubuntu. Datmo is available as a package to import by Python but not as terminal command.

Usually any issue with the terminal command is due to a potentially environment issue. That is, it doesn't know the location of the datmo command and where to find the executable. If you still have issues, share the error you get and can try to help.

authors: asampat3090 kamprz shabazpatel

repo: datmo/datmo


text:
Rewrite of flexible type converter logic.

Rewrote the flexible type converter logic to be much more readible and
flexible.  This is the next stage of the flexible type improvements.
The logic used a different metaprogramming method documented in
flexible_type_converter.hpp.  This gives much better control over
error messages and better flexibility in adding additional types.
Also discovered and fixed some minor bugs with the gl_string classes.

jenkins pls

Looks about right, but you want to add fast paths for direct matches to flexible_type types. i.e. if the passed type is exactly a flex_dict for instance, you want it to execute a single copy and no go through the longer path.
There is a template function: is_valid_flex_type::value (see flexible_type_base_types.hpp) for a compile time check if T matches one of the flexible_type base types.

Currently, it appears this is causing issues with the variant type stuff in some cases, so I'm investigating that when I have time.

authors: haijieg hoytak ylow

repo: dato-code/SFrame


text:
[epic] Flowershow CLI v1

Creating a command line interface for flowershow.
Questions

[ ] Use typescript for this as a first tryout? ✅2022-07-01 yes

Acceptance

[ ] Have a chosen a framework (shortlist: commander, oclif, arg - but this seems a bit too basic?)
[ ] npx flowershow or simple flowershow (if i have globally installed flowershow) command works and prints out basic help message (if no args etc)

Tasks

[x] Create packages/flowershow
[x] #134
[ ] CLI v0.2 add a preview / show command
[ ] CLI v0.3 add a build command

Notes
What do i want to do from command line or otherwise
Simplest thing: get into a directory and do flowershow dev/show and i get a running local preview
mkdir test
touch test/README.md
cd test
flowershow show

...

http://localhost:3000/ # browser automatically opened

Other functions
flowershow init
flowershow publish

@airtonix good point and done https://www.npmjs.com/package/flowershow

Add v0.1, v0.2, v0.3 parts of CLI

Can you please clarify if the CLI is deprecated or not? It is very confusing

repo: datopian/flowershow


text:
[site][m] - improvements to lighthouse score

The lighthouse score is now 97, the biggest improvement was in copying the sendinblue styles locally and bundling then with the rest of the CSS(This allow minifying and removing unused styles)

LGTM.

authors: demenech luccasmmg

repo: datopian/portaljs


text:
Performance of Akita under heavy load

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report
[X] Performance issue
[ ] Feature request
[ ] Documentation issue or request
[ ] Support request
[ ] Other... Please describe:

Current behavior
I'm considering to use Akita in a project but during the research reading I came across the following article: https://medium.com/@vpranskunas/deep-comparison-of-state-management-solutions-in-angular-562985d4474e
It states that Akita has some performance issues compared to e.g. NGRX.
StackBlitz with performance tests is here: https://stackblitz.com/edit/angular-state-management-solutions-performance-comparison
Expected behavior
I was hoping for a similar performance compared to other state management libriaies.
Is it caused by the test suite written incorrectly and the results can be improved or indeed it's Akita's issue?

You can read my comments in the article.

I know the article was written by an author of one of these libs under comparison, meaning that the contents shall be taken cautiously, but focusing on the StackBlitz only - I wonder whether I'm going to eventually face some performance issues when using Akita, because, at least for this specific type of test, NGRX doesn't seem to underperforming.
I realize that such a test doesn't necessarily fit into real world example, but still.

I have no interest in convincing you. Use whatever you want.
First, we are a big-data company, and we don't have any performance issues.
Second, the test uses EntityStore.update/add() methods, which supports many update variations that you'll not find in any other library. You can always update the entity directly using the underline setState method.

authors: NetanelBasal mateubo

repo: datorama/akita


text:
disable .line-loading animation on linux

this is heavy on perf. closes #413

I'll gladly take this and file a PR, is anyone currently working on this / is it already completed?

I'll gladly take this and file a PR, is anyone currently working on this / is it already completed?

you're more than welcome to! :) And no, no one is working on this currently.

I dug into this further and was able to replicate the issue on both datproject.github.io/design and within dat-desktop and link it directly to the .line-loading animation. However, I don't believe this is caused by something in the app specifically since I was able to replicate the same issue on other websites & apps that used the same or similar animations.
I'm looking into potential fixes, a bit hung up with what the best way to go about detecting the environment under which we'd want to disable these animations. Any ideas?

for now we can simply disable the animation in the desktop app where we can check the os using require('os').platform() and change the css definition accordingly

Is this still an issue?

authors: juliangruber okdistribute ryanwarsaw

repo: datproject/dat-desktop


text:
Getting the parse tree for an expression

Using DDMathParser, is it possible to extract the parse tree (or a similar structure) for a given expression? If so, what class should I look at?

Yes, absolutely. That is the whole purpose of the Expression class. Please check out the Wiki for specific usage examples.

repo: davedelong/DDMathParser


text:
Exporting lua path

So, I know that you've already bumped into the issue with exporting the LUA path aka:
export LUA_PATH="$BASSETDIR/src/?.lua;$LUA_PATH"
I'm still trying to figure out why it refuses to export it if stuck in the startup script.
But this is the last last stopping point in making an executable jupyter

After reviewing the README, I don't think I fully finished my thought when I was writing it. That export statement works well for me. Are you saying that it's not working, specifically inside jupyter notebooks?

So the export works if you spin up the Docker container and then manually export the variables, but the image won't work if you try to export the paths in the Dockerfile itself ie:
ENV LUA_PATH='$BASSETDIR/src/?.lua;$LUA_PATH'
The image still doesn't find lua. I think it might have something to do with length? or with the ? ? not sure...


text:
No way to configure an adapter that is not hardcoded into the plugin?

I was trying to help a user get set up with Factorio Modding Tool Kit's Debug Adapter (via a global install of the npm package), but after we got it installed we got stumped because it seems there is no way to configure this plugin to use an adapter that it does not itself install?
I'm normally a VSCode user, so maybe I'm missing something by my unfamiliarity with sublime, but surely loading other adapters is the whole point of a DAP client?

AdapterConfiguration is how things are setup and registered and ideally all exist in this package so other users can benefit from them its not a public api and I see little benefit and lots of downsides to splitting things up into a bunch of helper packages like LSP if thats what you are suggesting.
Most adapters here just download the vscode extension since they usually include the adapter as well as snippets and json schema support but thats not strictly required

In FMTK's case, the vscode extension does more workspace-aware config tweaking to the command to run, so the definition form the extension is not quite sufficient by itself. For other editors i've helped users with, they could just point it an an arbitrary program to run as the DAP adapter, so we could fill in the appropriate command line options there. If you aren't loading the whole vscode extension for the in-editor factorio version picker, then you'll have to provide somewhere for users to customize the adapter's command line.

If I wanted to install a DAP for Playdate, I would have to clone this repo and write an adapter for it, right? That's kind of the sticking point with this issue, if I understand it. SublimeDebugger cannot be used as a dependency for other debug adapter packages.

Most adapters here just download the vscode extension since they usually include the adapter as well as snippets and json schema support but thats not strictly required

Sure, but the rest of the VSCode extension is a bunch of TypeScript or whatever that is just ignored. The adapter needs to at least attach to the server through stdio or TCP. It seems like a starting point for a compromise would be providing a "generic TCP adapter" and a "generic stdio adapter" that can be used with minimal configuration. At least until someone wants to dive in and develop a more feature-rich adapter in a fork.

The protocol doesn't specify how to start to start or communicate with the adapter which makes generic tcp/stdio configurations not even remotely powerful enough.
Its like 10 lines of code to configure an adapter outside of this package for testing.
Plugin
|-- .python-version -> 3.8
|-- plugin.py -> ...

from Debugger.modules import dap, core
from Debugger.modules.adapters_registry import AdaptersRegistry

class Adapter(dap.AdapterConfiguration):
	type = 'adapter-type'

	async def start(self, log: core.Logger, configuration: dap.ConfigurationExpanded):
		return await dap.SocketTransport.connect_with_retry(log, 'localhost', 1234)

AdaptersRegistry.register(Adapter())

Sure, but the rest of the VSCode extension is a bunch of TypeScript or whatever that is just ignored

You're talking about an irrelevant amount of space here and the alternative is me building and packaging most adapters which isn't going to happen

You're talking about an irrelevant amount of space here and the alternative is me building and packaging most adapters which isn't going to happen

It is irrelevant to space, but it's functionality that is necessary for VSCode. Functionality which you have to reproduce in this plugin regardless. So, you've already done the work of building and packaging most adapters. Not sure I see the point you are making. But that point I'm making is that I have to add those 10 lines if I want to add a new adapter and there is no alternative.

Those packages include a lot more than just debugging support and they have snippets and schemas for LSP and have compiled the servers (sometimes for multiple platforms) so no the AdapterConfiguration implementations are not even remotely close to the work that would be required from me to setup tooling to build and package all the adapters.
When its 10 lines of code I do not see the need to add an alternative that won't work for most dap servers anyway

Are you talking about packaging servers? I don't think it's relevant, but I agree that duplicating that work would be silly.
In the case of Playdate, I do not want the plugin to install the Playdate SDK (which ships the simulator that implements the server). It feels like we are not talking about the same thing. All I want is a way to connect to some server that is already running somewhere else, and I do not need the plugin to manage that server. The server could be running on a separate machine, for instance.
But the crux is that I have to add a new adapter to this plugin every time I want to support a new debugger. And then I need to create a PR to upstream them all, and hope that the PRs are accepted and not considered "trivial" because it's just 10 lines of code. There is a real issue, here.

I just told you how to add a adapter configuration outside of this package if thats what you wanted to do.
You are acting like connecting to an already running server on specific port is the usual case but its not. The docs you link to don't even tell you how to use their adapter and don't even mention that it is running on a specific port. No one is expecting users to manually use the adapters and thats not how VSCode or this package is designed to work.
There are multiple very simple adapter configurations supported in this project such as the ruby adapter which expects the user to have installed rdbg. The only reason it downloads the vscode package is so there are snippets and an LSP-json schema for debugger_configurations which are automatically extracted from the package.json file.
If you add support for an adapter your should probably make a pull request even if its "trivial" so other users don't need to figure out how to configure the adapter thats the entire point of this package.

authors: daveleroy justarandomgeek parasyte

repo: daveleroy/SublimeDebugger


text:
🛑 Status is down

In 039202a, Status (https://stats.uptimerobot.com/rYO3jsvYJ) was down:

HTTP code: 502
Response time: 511 ms

Resolved: Status is back up in 82ad0a6.

repo: davelevine/upptime


text:
Update refined to 0.9.27

Updates eu.timepit:refined from 0.9.20 to 0.9.27.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "eu.timepit", artifactId = "refined" } ]

labels: library-update, semver-patch

Superseded by #515.

repo: davenverse/cormorant


text:
Update sbt-tpolecat to 0.1.18

Updates io.github.davidgregory084:sbt-tpolecat from 0.1.16 to 0.1.18.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.github.davidgregory084", artifactId = "sbt-tpolecat" } ]

labels: sbt-plugin-update, semver-patch

Superseded by #173.

repo: davenverse/keysemaphore


text:
YT playlist not scrobbling

https://www.youtube.com/watch?v=p9BoEuwuzIk&list=PLgAQes15lM1NSkDXTnvA6zeubQ94b3CTi

Works for me on Chrome 63.

category problems

authors: Nordblum OwnageIsMagic

repo: david-sabata/web-scrobbler


text:
Transparently loading/saving the notebook files on the host machine

As far as I could tell the notebook files are held on the "target" machine, the same one where the kernel code is actually run. Wouldn't it be useful to keep the notebooks on the "host" machine (where JupyterLab is running) so they can be backed up more easily?
The use case I imagine right now is having short lived targets in a spot cloud VM, in k8s or inside an Android/iOS mobile app.

As far as I could tell the notebook files are held on the "target" machine, the same one where the kernel code is actually run.

It shouldn't be the case, because the contents service is running on the "host" machine.

authors: davidbrochart jpc


text:
It seems ComicVine changed their website today.
Can no long get ComicTagger to work. Yesterday it was fine, but not now.
I'm on Win 10, Comic tagger .15-beta
Message on CLI is:
<urlopen error [Errno 1] _ssl.c:504: error:14077410:SSL routines:SSL23_GET_SERVER_HELLO:sslv3 alert handshake failure>

Probably due to the new forced SSL implementation?
https://comicvine.gamespot.com/forums/api-developers-2334/

Correct - since ComicTagger is using Python 2.7 and not Python 3+, the libraries used are defaulting to SSLv3 (which is blocked by ComicVine). However, I have found a way to get them to use TLS 1.0 instead.
I can provide further information and submit a pull request...

afuturefoe,
I am having the same issue.  Can you provide further information on how to use TLS 1.0

Maybe it's the same as #87, looking into it.

Same as #87, yes.
I was able to get it to work with the following:
import ssl
context = ssl.SSLContext(ssl.PROTOCOL_TLSv1)
resp = urllib2.urlopen( url, context=context )
return resp.read()

Thanks for the suggestion, it seems trivial to solve. Actually my problem is how to repackage the application for the various platforms, it seem I'll need lot of dependencies and the latest version of XCode refuses to install on El Capitan. This will require a little more effort..

I don't know if this will be any help, and I can't test it now, since I don't have a mac anymore, but if you need only the XCode Command Line tools, I seem to remember that I was able to get them installed from the terminal using only brew (https://brew.sh/).  Also, it may be that some of the dependencies that you needed to compile are now available  as pre-compiled packages through brew standard or third-party respositories (called Taps).

sudo port self update
sudo port install py27-pyqt4
sudo port install py27-configparser
sudo port install py27-beautifulsoup4
sudo port install py27-pil
python2.7 comictagger.py

Close this to move discussion on #87. I released a MacOS dmg with the fix, I hope. Let me know how it works. Many thanks @afuturefoe for your hints!

repo: davide-romanini/comictagger


text:
Drop support for mypy_extensions.TypedDict

Why:

This kind of TypedDict was introduced in the era of Python 3.7, which is no longer supported by trycast.

It is reasonable (in @davidfstr 's opinion) to encourage other users to migrate to a version of TypedDict from the standard library (Python 3.9+) or from typing_extensions (Python 3.8).

Extra code is required to support this very old kind of TypedDict.

Timing:

Suggest actually dropping mypy_extensions.TypedDict support at the same time as dropping support for Python 3.8, because then the trycast(strict=...) parameter (which involves both Python 3.8's TypedDict and mypy_extensions.TypedDict, and nothing else) can be cleanly removed at the same time

mypy_extensions.TypedDict was deprecated in Aug 2023, according to https://github.com/python/mypy_extensions/issues/46

repo: davidfstr/trycast


text:
Generic load BDOS from CP/M filesystem

Hi,
You mentioned a generic BDOS loader in my last pull request's comment section. This would indeed be really helpful in case the size of BDOS changes and avoid having to adjust the bootloader and disk image generation, at least for the Atari. Depending on the amount of assumptions we allow, this could be done in a few lines of code.

assume BDOS.SYS is first file written to the CP/M fs
it is always the first directory entry
it is stored in consecutive sectors after the directory
we need to adjust for reserved sectors (hardcoded, derived from dph)
retrieve first directory sector
extract rc counter
load that amount of sectors into  __USERTPA_START__
relocate
run

Were you planning on doing this yourself? If not, I could take a stab at it.
Regards,
Ivo

Proof of concept:
https://github.com/ivop/cpm65/tree/bdos-loader

Proof of concept. Created a pull request here: #39

Ah. I just did it in #40!
It'll actually load any one-extent file with arbitrary block numbers (so the file can be fragmented). There are currently two versions, for large disks and small disks, but I'm not sure it's worth it --- it ended up being only 245 (discardable) bytes. I was able to drop one of the reserved tracks on the Apple II with this, which is great because it opens the door for data disks which don't have the BDOS and CCP on them (although they'll still need the reserved track).

Sorry, I couldn't resist trying it out myself ;)
My loader is 71 bytes, but it has more restrictions.

The intention is that the loader gets discarded after use, so it doesn't matter how big it is. It looks like the Atari port doesn't discard anything, so it's not appropriate there, so I have no objection to using yours.
(Although I think the Atari port could be made to discard its init code --- in the boot sector you get to specify the load address, right? So you could do the intial load somewhere high, then copy the resident bits down to their final address, before loading the BDOS. When you run it, all the init code gets overwritten. The PET ports do this.)

I have converted it to your new loader anyway. I like it to be consistent with the other ports. Right now I only discard part of the init code by overwriting it with the directory buffer. I'll look into loading it somewhere around $3000 and copy just the bios down to $0500. Should still work on a 16kB 400, 800 or 600XL then.

(closing for cleanup)

authors: davidgiven ivop


text:
Error state should be reset at onChange

Hi, I have a Control with an async validator, something very classic directly used from doc.
          <Errors
            className="errors"
            model=".email"
            messages = {{
              available  : val => `${val} is already used. Please choose another address`
            }}
          />
          Email
          <Control.text
            type="email"
            model=".email"
            validators={{
              required: val => val.length,
              isEmail
            }}
            asyncValidators={{
              available: (val, done) => actions.checkEmailAvailability(val).then(done)
            }}
          />

Normal behaviour is to check onBlur so I put my already used email address, blur and the error show. Everything fine.
Problem comes when I try to change my email input. I focus, then change the email address ... but the error does not disapear. Even worst, the message is updated with the new value let me thinking that everything I type is not valid.
I just discovered this so I don't have found a workaround, do you have an idea to set async validators to true on change ?

I just published 1.5.1 - can you use that and see if the issue still exists?

Just tried it right now and I still have the problem.
Thanks for your help. Easiest workaround right now is to trigger validation also on change but it is not what I would prefer for production :s

A stopgap for now (while I look into this issue) is to do something like this:
dispatch(actions.resetValidity('your.model', ['available']))

That will reset the validity only for the specified keys.

Works perfectly. Not very elegant but efficient, thanks for your solution.
If people has the same problem, there is my code without unecessary information.
Feel free to close the issue if you consider it as solved or left it open if you consider this need to be changed/fixed.
@connect()
export default class SignInForm extends Component {
  constructor(props){
    super(props)
    this.resetError = this.resetError.bind(this);
  }

  resetError(event){
    const field = event.target.name
    this.props.dispatch(form_actions.resetValidity(field), ['available'])
  }

  render() {
    return (
      <Form model="signUp">
          Username
          <Control.text
            type="text"
            model=".username"
            asyncValidators={{
              available: (val, done) => actions.checkUsernameAvailability(val).then(res => done(res))
            }}
            onChange = {this.resetError}
          />
          <Errors  className="errors"  model=".username"  messages = {{ available  : val => `${val} is already used. Please choose another username`}}  wrapper="span"/>
      </Form>
    )
  }
}

I'll keep this open as I'd like to add an actions.resetAsyncValidity action.

On second thought, would it make sense for async validity to automatically be reset when a field changes?

@davidkpiano
WRT:

On second thought, would it make sense for async validity to automatically be reset when a field changes?

I think so. I'm having the same problem as the original author. I think it would make sense for the validity to be reset on change.

authors: davidkpiano fidiego l1br3

repo: davidkpiano/react-redux-form


text:
Missing Keymap support

Hi there, great project you've built here.
Was wondering if there'll be support for custom keymaps anytime soon?, (as available on CodeMirror).
Relevant discussion I found: https://discuss.codemirror.net/t/keymap-for-bold-text-in-lang-markdown/3150
I can probably do this manually right now by listening for keyup events. but i think it'll be much nicer if we had this as part of our configuration.

@FemiBlack Any updates on this?
Apparently I can't listen for keyup either because CodeMirror intercepts it, so I'm at a bit of a loss...

authors: EvanZhouDev FemiBlack

repo: davidmyersdev/ink-mde


text:
Opening the window in full screen fills the rest of the window size with black

When I open the application in full screen the rest of the screen gets filled with black as it appears in the screenshot.

Thanks! Yes, this is a known .NET MAUI issue to be implemented along with moving between different OS scalings.

repo: davidortinau/WeatherTwentyOne


text:
Setup wizard, can not enter serial number containing letters.

Description
I am trying to configure the integration with the wizard, but my inverter serial number contains a letter which is not accepted. I also noted the wizard does not accept leading zeroes is the serial number.
Reproduction Steps
Try to enter text string with letters and numbers for inverter serial number.
Log
There is no debug log
Version
24.11.02

Hi @RadioactiveMan56,
That is correct, loggers with alphanumeric serial numbers are an older version of the hardware which is not (and won't be) supported by pysolarmanv5.

Thanks !
On Thu, 28 Nov 2024 at 18:29, David Rapan @.***> wrote:

Hi @RadioactiveMan56 https://github.com/RadioactiveMan56,
That is correct, loggers with alphanumeric serial numbers are an older
version of the hardware which is not (and won't be) supported by
pysolarmanv5.
—
Reply to this email directly, view it on GitHub
https://github.com/davidrapan/ha-solarman/issues/269#issuecomment-2506556721,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A64VE5LNEY3JSBQUBIQ2I4L2C5HHDAVCNFSM6AAAAABSVSUDJWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMBWGU2TMNZSGE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hey @RadioactiveMan56, can you tell me which device you have exactly?

Hi David,
It is an S2-WL-ST.
Using this integration now with some success:
https://github.com/wills106/homeassistant-solax-modbus
Eric
On Thu, 19 Dec 2024 at 03:40, David Rapan @.***> wrote:

Hey @RadioactiveMan56 https://github.com/RadioactiveMan56, can you tell
me which device you have exactly?

—
Reply to this email directly, view it on my GitHub
https://github.com/davidrapan/ha-solarman/issues/269#issuecomment-2552655742,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A64VE5MYXZ7IFCITPE2GK7L2GIW2LAVCNFSM6AAAAABSVSUDJWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJSGY2TKNZUGI
.

You are receiving this because you were mentioned.Message ID:
@.***>

So this integration should work too if you force passthrough = True mode. 😉 But you have to change it directly in the api.py file.

Thanks, I will have a look at it in the new year.
On Thu, 19 Dec 2024 at 13:05, David Rapan @.***> wrote:

So this integration should work too if you force passthrough = True mode.
😉 But you have to change it directly in the api.py file.
—
Reply to this email directly, view it on GitHub
https://github.com/davidrapan/ha-solarman/issues/269#issuecomment-2553586173,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A64VE5OZU7PE4UPDDDQDE7T2GKY67AVCNFSM6AAAAABSVSUDJWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJTGU4DMMJXGM
.
You are receiving this because you were mentioned.Message ID:
@.***>

repo: davidrapan/ha-solarman


text:
Build process using webpack

Set up a webpack build process with:

ES6 transpilation (yay! arrowfunctions!)
TypeScript
Browserification of NPM libraries
Index.ejs
Angular 2 + Router
Chunk configuration
Component CSS support

First part (TypeScript + HMR + Browserification): https://github.com/davidstellini/code-pulit/pull/8

Closed with:
#8
#9
#10
#11

repo: davidstellini/code-pulit


text:
namespace not working

Hi! I'm migrating to this from postcss-bem-linter and running into an issue with namespacing. My config looks like this:
{
  "extends": "stylelint-config-suitcss",
  "plugins": [
    "stylelint-selector-bem-pattern"
  ],
  "rules": {
    "selector-bem-pattern": {
      "preset": "suit",
      "presetOptions": {
        "namespace": "qz"
      }
    },
    "indentation": "tab",
    "number-leading-zero": null,
    "no-missing-eof-newline": null
  }
}

when I run the command, I get
Invalid value "[object Object]" for option "presetOptions" of rule "selector-bem-pattern"

are my presetOptions properly formatted? Any idea what could be amiss?

thank you sir! amazing work as always.

Thanks for finding and reporting the bug. I guess the stylelint options validator function was not ready for deeply nested objects.

Released in 0.2.1

authors: VinSpee davidtheclark


text:
IndexError in DEBUG logging of Proposed optimization window

Describe the bug
Upon upgrade to 0.6.6 EMHASS stopped working in Home Assistant due to the following error.
2024-02-22 15:37:00,247 - web_server - ERROR - Exception on /action/dayahead-optim [POST]
Traceback (most recent call last):
  File "/usr/local/lib/python3.11/dist-packages/flask/app.py", line 1463, in wsgi_app
    response = self.full_dispatch_request()
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/dist-packages/flask/app.py", line 872, in full_dispatch_request
    rv = self.handle_user_exception(e)
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/dist-packages/flask/app.py", line 870, in full_dispatch_request
    rv = self.dispatch_request()
         ^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/dist-packages/flask/app.py", line 855, in dispatch_request
    return self.ensure_sync(self.view_functions[rule.endpoint])(**view_args)  # type: ignore[no-any-return]
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/dist-packages/emhass/web_server.py", line 66, in action_call
    opt_res = dayahead_forecast_optim(input_data_dict, app.logger)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/dist-packages/emhass/command_line.py", line 234, in dayahead_forecast_optim
    opt_res_dayahead = input_data_dict['opt'].perform_dayahead_forecast_optim(
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/dist-packages/emhass/optimization.py", line 572, in perform_dayahead_forecast_optim
    self.opt_res = self.perform_optimization(df_input_data, P_PV.values.ravel(),
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/dist-packages/emhass/optimization.py", line 288, in perform_optimization
    self.logger.debug("Deferrable load {}: Proposed optimization window: {} --> {}".format(k, def_start_timestep[k], def_end_timestep[k]))
                                                                                ~~~~~~~~~~~~~~~~~~^^^
IndexError: list index out of range

To Reproduce

Do not configure def_start_timestep or def_end_timestep in YAML,
Send a dayahead-optim request without providing either def_start_timestep or def_end_timestep in the payload.

Expected behavior
Configurations that were previously working should continue to work, especially for optional parameters. Alternatively, if these parameters are required then they should be validated and an appropriate error message issued.
EMHASS installation type

Add-on

Additional context
Add any other context about the problem here.

Do not configure def_start_timestep or def_end_timestep in YAML

Just configure these. They just should have the same length as the number of deferrable loads.
This was an unexpected breaking change, so that's why no warning was issued. But the solution is easy and the error messages were improved after this.
Closing.

Yes, I configured locally once I figured out what was the cause. Just looking to avoid this for others who upgrade, glad it's fixed in later versions. 👍

authors: davidusb-geek luuuis

repo: davidusb-geek/emhass


text:
Shell class for "Proffer" could not be found

I have a problem with bin/cake proffer
Exception: Shell class for "Proffer" could not be found. in [/Users/vnt/Sites/test/vendor/cakephp/cakephp/src/Console/ShellDispatcher.php, line 324]
How make to resolve?

Then you'll need to load the plugin in your config/bootstrap.php file.
Plugin::load('Proffer');


text:
Seeking new primary maintainer

Hi all,
I'm sure it's clear by the time since the last release as well as the number of open issues that have gone un-responded to that, due to life being the chaotic and ever-changing thing that it is, I haven't had much time to work on or maintain Obsidian Full Calendar for a while now.
It's clear from the bug reports and feature requests that this is a popular plugin, and I do want to see it continue to be a part of Obsidian's vibrant plugin ecosystem.
To that end, I'm looking for a primary maintainer to take over from me. If you're interested, please send me an email -- you can find it in my GitHub profile or by clicking the little envelope on my homepage.
Thanks to all of you who have used and continue to use Obsidian Full Calendar, and I'm looking forward to see how this plugin grows going forward!
All the best,
-davish

Has there been any updates on this? I just found this plug-in and would really like it to continue.

Hi! I have initial interest in helping with development, commenting here as a reminder to myself. Primary maintainership is a lot of commitment though, so interested to see if we can find others interested. Are you looking to bow out completely, or still ready to maintain and read PRs?

@davish Perhaps this repository can be moved over to obsidian-community? There are several features that I've been wanting to add, but I do not have the time to commit to being the sole primary maintainer. If the repository is moved to obsidian-community, then the load is shared. In that case, I'll be able to help by triaging issues and closing issues that are out of scope or closing duplicates and making the dev environment more reproducable (i encountered a few issues when forking and building this plugin for myself). That way, things are a lot more clear and easy for anybody else that wants to jump into things. I love this plugin and I wouldn't want to see it unmaintained.
cc @joethei @phibr0 @konhi @chrisgrieser

Yes, that can be done and is one of the reasons "obsidian community" exists.

Hi @chrisgrieser and @hyperupcall,
Thanks for posting here! I think moving Full Calendar to obsidian-community sounds like a great way to ensure the sustainability of the plugin. What would the process look like to make this happen? Also happy to move this conversation to email -- you can find my address in my bio.

Hi @davish I've invited you to the obsidian-community org, once you have accepted it, go to the repository settings, scroll down and click the "Transfer ownership" button and select the org.
After after that is done I will update the entry in the community plugins list.

I'm interested in maintaining the project. @davish lmk how you want to take this. if it's transfered to the obsidian-community org I can contribute there as well.

bump.
I am interested in updates regarding the continuation of this project

Hi @chrisgrieser @joethei, apologies for the delay, but I just tried to accept the obsidian-community invite, and it unfortunately expired. Would it be possible for one of you to re-send the invite so I can transfer ownership? Thanks so much!

Hi @davish sorry for the delay, invited you again.

The repository has been moved to @obsidian-community! I'll close this issue as completed.

authors: Dragonslayersakata chrisgrieser davish diorcula hyperupcall joethei julesvirallinen linqian1400

repo: davish/obsidian-full-calendar


text:
Fix crash when libX11 is missing locale support

This fixes a crash I was running into trying the dnn_face_recognition_ex with XQuartz on a Apple Silicon Mac.
It seems that the prebuilt libX11.6.dylib bottle from Homebrew is missing full locale support, which causes the XwcTextListToTextProperty call to return -2 (XLocaleNotSupported), which caused this code to crash when calling XFree as the XTextProperty value was not initialised.
Window titles seemed generally low value here, so just avoiding the crash seemed like the easiest path forwards.

Nice, thanks :)

authors: asherkin davisking

repo: davisking/dlib


text:
🛑 Line of Succession is down

In 5b0a953, Line of Succession (https://lineofsuccession.co.uk/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Line of Succession is back up in bf4db7e after 1 hour, 32 minutes.

repo: davorg/uptime


text:
extensions fail to install during the default workspace start process

Describe the bug
Certain extensions fail to install during the default workspace start process.
To Reproduce
Steps to reproduce the behavior:

Start the default workspace.
Observe the installation process for extensions.
Note the extensions that fail to install.

Expected behavior
All specified extensions should install successfully, ensuring a complete and functional development environment.
Screenshots

Desktop (please complete the following information):

OS: macOS
daytona: 0.34.0

Additional context
The following extensions failed to install:

ms-dotnettools.csharp
xdebug.php-pack
ms-python.vscode-pylance

The error message suggests that the full extension ID, including the publisher, should be used. This issue is reproducible, as it occurred consistently across multiple workspace setups. The problem appears to be related to incorrect extension names in the default workspace configuration.

@nkkko does this happen on every workspace? Can't seem to reproduce

Bug: Certain Extensions Fail to Install During Default Workspace Start Process
Description:
Some extensions fail to install during the default workspace startup, preventing a fully functional development environment.
Steps to Reproduce:
1.	Start the default workspace.
2.	Observe the installation process for extensions.
3.	Note which extensions fail to install.

Expected Behavior:
All specified extensions should install successfully to ensure a complete development setup.
Desktop:
•	OS: macOS
•	Daytona Version: 0.34.0

Failed Extensions:
•	ms-dotnettools.csharp
•	xdebug.php-pack
•	ms-python.vscode-pylance

Additional Context:
The error message suggests that the full extension ID, including the publisher, is required. This issue has been consistently reproducible across multiple workspace setups and seems to stem from incorrect extension names in the default workspace configuration.

@Tobi-Babalola123 can you clarify the meaning of your comment since it seems to be a copy-paste of the description?

Closing as stale. @nkkko reopen if this happens again.

authors: Tobi-Babalola123 Tpuljak nkkko

repo: daytonaio/daytona


text:
ADD: update for _run

In mongodb I was able to do the following:
db.users.update( {}, { $rename: { 'permissions': 'accesses' } }, { upsert: false, multi: true } )
For the migration I had to add the update implementation to the mongo-handling. Now I can do:

  db._run('update', 'users', {
      query: {},
      update: { $rename: { 'permissions': 'accesses' } },
      options: { upsert: false, multi: true }
    },
    callback()
  );

In this current state I can't merge this, first of all b/c this is not exposed as an api and also b/c of the following:
You should never use _ functions. Those are in all node projects considered to be private. You have no guarantees that the behavior is not being broken. Thus using this opens you to a serious risk, b/c of the missing guarantees. If you use an underscore prefixed function, be prepared that your software is already broken on the next update.

Also please specify, your use case:
Is it

data definition/migration

or is it

data manipulation/transformation

I'm absolutely aware of the meaning of Underscore-Functions. And because I don't wanna mess with any API I don't understand I added it to a private function. It solved my problem, it's a minor addition and it wont break any existing code since it's a private function. And for people who really wanna deal with fire and use private API we have at least an option.
I have basically a similar problem as described here: https://github.com/db-migrate/node-db-migrate/issues/342 and I took the solution you suggested there. I would understand it as a data manipulation/transformation because of some changes in my data structure.

Ok, good, so you mean that as an addition to the internals of the mongodb driver.
From that perspective I'm fine with your additions and would just change the title to avoid anyone being confused this being an official function.

:blush: perfect thank you!

authors: signalwerk wzrdtales

repo: db-migrate/mongodb


text:
Cursor jumps to end of file when query was changed and executed using CTRL+ENTER

System information:

Windows 10
Version 21.3.5.202202200327

Connection specification:

Database name and version: Postgres 13

Describe the problem you're observing:
I have many queries I execute often in a script file for testing purposes. When I change a query (e.g. new condition) and execute it using CTRL+ENTER, the cursor jumps away from the current query and goes to end of the file. This does not happen when using the mouse and clicking the button.
This is very annoying, since I need to scroll back all the time. Happens since updatting to Version 21.3.5.202202200327
Steps to reproduce, if exist:
Include any warning/errors/backtraces from the logs

thanks for the report
closed as a duplicate of #15562

authors: rhuesken uslss

repo: dbeaver/dbeaver


text:
Regression: Dump Database Feature Causes Performance Issues and Wrong Dump File Options in Version 24.2.0

Description
I recently upgraded from DBeaver version 24.1.5 to 24.2.0 and encountered several issues with the dump database feature that were not present in the previous version.
Issues:

Result Dump Printed in Dump Process Tab: In version 24.2.0, the content of the result dump file is printed in the dump process tab. This change introduces a few problems:

I can no longer disable this feature, which was possible in previous versions.
When dumping a large database, the dump file can become enormous, causing the pop-up to lag and become unresponsive at times.

Incorrect Dump File Options: After dumping a database, the generated dump file contains incorrect options. Specifically, the file includes a LOCK TABLE command, even though I did not select this option in the Export configuration tab. This renders the dump file unusable for restoring on another machine, resulting in failed restores.

Steps to Reproduce:

Upgrade DBeaver from version 24.1.5 to 24.2.0.
Use the "Dump Database" feature to export a database.
Observe that the dump result is printed in the dump process tab, with no option to disable it.
Attempt to restore the generated dump file on another machine, and note the incorrect options in the dump file (e.g., LOCK TABLE).

Expected Behavior:

There should be an option to disable printing the dump file content in the dump process tab.
The dump file should not include incorrect options such as LOCK TABLE when the user has not selected them in the Export configuration.

Actual Behavior:

The dump process tab prints the content of the dump file, causing performance issues when dealing with large databases.
The generated dump file contains incorrect options that prevent successful restoration on other machines.

Workaround:
I downgraded to version 24.1.5, where the dump feature works as expected.
Environment:

DBeaver Version: 24.2.0 (issue present)
DBeaver Version: 24.1.5 (working)
Database Type: MariaDB
Operating System: Window 10

Please investigate this regression as it severely impacts the usability of the dump and restore process in the latest version.
DBeaver Version
Community Edition 24.2.0
Operating System
Window 10
Database and driver
Database: 10.3.39-MariaDB-0ubuntu0.20.04.2
Driver: MariaDB JDBC driver - org.mariadb.jdbc:mariadb-java-client:RELEASE - 3.3.2
Steps to reproduce
No response
Additional context
No response

Thank you for the bug report.
I'm closing it as it's a duplicate of #35421


text:
add support run stores procedure for ProfileBD/FIS

Feature request/Bug report
adding of support of performing store procedures for ProfileBD/FES
wiki FIS_(company)
wiki GT.M is a high-throughput key-value database/MUMPS
(support JDBC driver)
at the moment
CALL NAME_PROCEDURE(Param1, Param2, 'ParamX');

result

java.sql.SQLException: Not supported for anything other than Aggregate Message Service Class.
at sanchez.jdbc.driver.ScCallableStatement.execute(Unknown Source)
at org.jkiss.dbeaver.model.impl.jdbc.exec.JDBCPreparedStatementImpl.execute(JDBCPreparedStatementImpl.java:259)
at org.jkiss.dbeaver.model.impl.jdbc.exec.JDBCPreparedStatementImpl.executeStatement(JDBCPreparedStatementImpl.java:203)
at org.jkiss.dbeaver.model.impl.jdbc.exec.JDBCCallableStatementImpl.executeStatement(JDBCCallableStatementImpl.java:229)
at org.jkiss.dbeaver.runtime.sql.SQLQueryJob.executeStatement(SQLQueryJob.java:445)
at org.jkiss.dbeaver.runtime.sql.SQLQueryJob.lambda$0(SQLQueryJob.java:389)
at org.jkiss.dbeaver.model.DBUtils.tryExecuteRecover(DBUtils.java:1539)
at org.jkiss.dbeaver.runtime.sql.SQLQueryJob.executeSingleQuery(SQLQueryJob.java:387)
at org.jkiss.dbeaver.runtime.sql.SQLQueryJob.extractData(SQLQueryJob.java:818)
at org.jkiss.dbeaver.ui.editors.sql.SQLEditor$QueryResultsContainer.readData(SQLEditor.java:2276)
at org.jkiss.dbeaver.ui.controls.resultset.ResultSetJobDataRead.lambda$0(ResultSetJobDataRead.java:93)
at org.jkiss.dbeaver.model.DBUtils.tryExecuteRecover(DBUtils.java:1539)
at org.jkiss.dbeaver.ui.controls.resultset.ResultSetJobDataRead.run(ResultSetJobDataRead.java:91)
at org.jkiss.dbeaver.model.runtime.AbstractJob.run(AbstractJob.java:95)
at org.eclipse.core.internal.jobs.Worker.run(Worker.java:56)

work example java
        connection = dataSource.getConnection();
        callableStatement = connection.prepareCall( "{call NAME_PROCEDURE (Param1,?,?)}" );
        callableStatement.setInt(1, 1);  // Param 2
        callableStatement.setString(2, xml); // Param 3
        callableStatement.registerOutParameter(1, java.sql.Types.VARCHAR);
        resultSet = callableStatement.executeQuery();

        while (resultSet.next()) {
                outputStr.append(resultSet.getString(1));
            }

base
connection.prepareCall( "{call NAME_PROCEDURE (Param1,?,?)}" );
DBeaver 5.1.2/Arch Linux/Windows 10

This seems to be driver-specific issue.
Could you try to execute it as
{ CALL NAME_PROCEDURE(Param1, Param2, 'ParamX'); }

?

Perhaps it will be useful:
documentation driver and driver
+ The YottaDB GitHub project (fork GT.M), uses this driver
dbeaver 5.1.4 (30 july git )
{call mrpc(param, param, param)};

Error:
Not supported for anything other than Aggregate Message Service Class.

I need access to a live instance of this db server to fix this.
Or somebody's assistance. Please write me on serge@jkiss.org

authors: Bsplesk serge-rider


text:
Import task editing: btn "Next" doesn't work on step "settings" of Wizard

System information:

Operating system (distribution) and version Windows 10
DBeaver version 6.3.0 EE

Connection specification:

Database name and version Mysql

Describe the problem you're observing:
Btn "Next" doesn't work on step "settings" of Wizard, if an import task was opened for editing and Target table was changed.
Steps to reproduce, if exist:
Precondition:
An import task is created.
Steps:
Select "Edit" in the task context menu>
change the table in "Import into">press "Next" btns in Wizard steps.
Actual result: btn "Next" doesn't work on step "settings" of Wizard.

Are there any logs?

Please see:
null
org.jkiss.dbeaver.model
Error
Sat Dec 07 09:20:37 MSK 2019
Event loop exception

dbeaver-debug.log
.log

Still can't reproduce. From the log I see that task config become corrupted somehow but I can't realize how.
Could you make a video from the very beginning of task edit? Perhaps you did some other changes.

I have the same issue when i backup a database,  there are a NullpointerException :
!ENTRY org.jkiss.dbeaver.model 4 0 2019-12-23 15:18:04.891
!MESSAGE Event loop exception
!SUBENTRY 1 org.jkiss.dbeaver.model 4 0 2019-12-23 15:18:04.891
!MESSAGE
!STACK 0
java.lang.NullPointerException
at org.jkiss.dbeaver.tasks.ui.wizard.TaskWizardExecutor.runTask(TaskWizardExecutor.java:48)
at org.jkiss.dbeaver.tasks.ui.wizard.TaskProcessorUI.executeTask(TaskProcessorUI.java:67)
at org.jkiss.dbeaver.tasks.ui.wizard.TaskConfigurationWizard.runTask(TaskConfigurationWizard.java:186)
at org.jkiss.dbeaver.tasks.ui.wizard.TaskConfigurationWizard.performFinish(TaskConfigurationWizard.java:166)
at org.jkiss.dbeaver.tasks.ui.nativetool.AbstractToolWizard.performFinish(AbstractToolWizard.java:267)
at org.jkiss.dbeaver.tasks.ui.nativetool.AbstractImportExportWizard.performFinish(AbstractImportExportWizard.java:99)
at org.jkiss.dbeaver.ext.postgresql.tools.PostgreBackupWizard.performFinish(PostgreBackupWizard.java:159)
at org.eclipse.jface.wizard.WizardDialog.finishPressed(WizardDialog.java:832)
at org.eclipse.jface.wizard.WizardDialog.buttonPressed(WizardDialog.java:472)
at org.jkiss.dbeaver.tasks.ui.wizard.TaskConfigurationWizardDialog.buttonPressed(TaskConfigurationWizardDialog.java:122)
at org.jkiss.dbeaver.tasks.ui.nativetool.ToolWizardDialog.buttonPressed(ToolWizardDialog.java:83)
at org.eclipse.jface.dialogs.Dialog.lambda$0(Dialog.java:619)
at org.eclipse.swt.events.SelectionListener$1.widgetSelected(SelectionListener.java:84)
at org.eclipse.swt.widgets.TypedListener.handleEvent(TypedListener.java:252)
at org.eclipse.swt.widgets.EventTable.sendEvent(EventTable.java:89)
at org.eclipse.swt.widgets.Display.sendEvent(Display.java:4175)
at org.eclipse.swt.widgets.Widget.sendEvent(Widget.java:1057)
at org.eclipse.swt.widgets.Display.runDeferredEvents(Display.java:3988)
at org.eclipse.swt.widgets.Display.readAndDispatch(Display.java:3587)
at org.eclipse.jface.window.Window.runEventLoop(Window.java:823)
at org.eclipse.jface.window.Window.open(Window.java:799)
at org.jkiss.dbeaver.ext.postgresql.tools.PostgreToolBackup.execute(PostgreToolBackup.java:39)
at org.jkiss.dbeaver.tasks.ui.ExecuteToolHandler.executeTool(ExecuteToolHandler.java:59)
at org.jkiss.dbeaver.tasks.ui.ExecuteToolHandler.run(ExecuteToolHandler.java:51)
at org.jkiss.dbeaver.ui.ActionUtils$1.run(ActionUtils.java:363)
at org.eclipse.jface.action.Action.runWithEvent(Action.java:474)
at org.eclipse.jface.action.ActionContributionItem.handleWidgetSelection(ActionContributionItem.java:579)
at org.eclipse.jface.action.ActionContributionItem.lambda$4(ActionContributionItem.java:413)
at org.eclipse.swt.widgets.EventTable.sendEvent(EventTable.java:89)
at org.eclipse.swt.widgets.Display.sendEvent(Display.java:4175)
at org.eclipse.swt.widgets.Widget.sendEvent(Widget.java:1057)
at org.eclipse.swt.widgets.Display.runDeferredEvents(Display.java:3988)
at org.eclipse.swt.widgets.Display.readAndDispatch(Display.java:3587)
at org.eclipse.e4.ui.internal.workbench.swt.PartRenderingEngine$5.run(PartRenderingEngine.java:1160)
at org.eclipse.core.databinding.observable.Realm.runWithDefault(Realm.java:338)
at org.eclipse.e4.ui.internal.workbench.swt.PartRenderingEngine.run(PartRenderingEngine.java:1049)
at org.eclipse.e4.ui.internal.workbench.E4Workbench.createAndRunUI(E4Workbench.java:155)
at org.eclipse.ui.internal.Workbench.lambda$3(Workbench.java:660)
at org.eclipse.core.databinding.observable.Realm.runWithDefault(Realm.java:338)
at org.eclipse.ui.internal.Workbench.createAndRunWorkbench(Workbench.java:559)
at org.eclipse.ui.PlatformUI.createAndRunWorkbench(PlatformUI.java:150)
at org.jkiss.dbeaver.core.application.DBeaverApplication.start(DBeaverApplication.java:229)
at org.eclipse.equinox.internal.app.EclipseAppHandle.run(EclipseAppHandle.java:203)
at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.runApplication(EclipseAppLauncher.java:137)
at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.start(EclipseAppLauncher.java:107)
at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:401)
at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:255)
at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
at java.base/java.lang.reflect.Method.invoke(Unknown Source)
at org.eclipse.equinox.launcher.Main.invokeFramework(Main.java:657)
at org.eclipse.equinox.launcher.Main.basicRun(Main.java:594)
at org.eclipse.equinox.launcher.Main.run(Main.java:1465)

@ghanmikarim this seems to be dup of #7462

Still can reproduce with little changes: btns "Next" and "Start" are not active in "Preview Data Import" of the wizard

Steps are the same:
Precondition:
An import task is created.
Steps:
Select "Edit" in the task context menu>
change the table in "Import into">go through the Wizard.

I cannot reproduce anymore

authors: ghanmikarim kseniiaguzeeva serge-rider


text:
automatically update objects tree after DDL command

System information:

Operating system WINDOWS 10 64Bit
DBeaver version 6.3.0 EE

Connection specification:

Driver name by default

Describe the problem you're observing:
tree does not automatically update after DDL command
Steps to reproduce, if exist:
Open Dbeaver with database tree in left part of interfase
Open SQL editor for this database.
Execute command CREATE TABLE or CREATE VIEW etc
We don’t see changes, we don’t see created tables or other objects in objects tree until we update the tree manually..

Thank you for the bug report.

This is not a bug but a new feature request.
Moving to 7+

version 7.0.1 i do not see the change.

version 7.2.0 i do not see the change.

version 7.2.3 problem not resolved

Will try to implement the feature after 7.3 release.

authors: detkliff kseniiaguzeeva serge-rider


text:
Snowflake - Backslash escape of single quote breaks syntax highlighting

System information:

Windows 10
DBeaver 7.0.1.202003211804 Community Edition
Git support, Darkest Dark Theme

Connection specification:

Snowflake 4.10.2
Snowflake 3.12.3
No tunnels or proxies

Describe the problem you're observing:
Even though Snowflake supports backslash escaped single quotes in string literals, these break the syntax highlighting in DBeaver.
This is a similar issue to these ones, but for a different connection type:
#6606, #2716, #2969
Steps to reproduce, if exist:

if anyone stumbles on this: while the default might not be correct, one can easily fix this for themselves by setting the Advanced parameters > String escape character to backslash in the driver settings.


text:
implement automatic retry

Addresses https://github.com/dbekaert/RAiDER/issues/219 by automatically retrying 'get' request to UNR

@kam3545 Can you try if this resolved your issue?

As a disclaimer, there is no guarantee that this will fix anything.
In my experience these issue usually arise due to a problem in the server - client connection.
For example, my home internet connections isn’t the greatest, and I usually can’t download anything large without an interruption.
The additional retries I’ve implemented is merely an additional level of redundancy.

On Dec 2, 2020, at 7:55 PM, dbekaert notifications@github.com wrote:
@kam3545 https://github.com/kam3545 Can you try if this resolved your issue?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/dbekaert/RAiDER/pull/220#issuecomment-737649148, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADWNVR347RABYQOE45G5OJTSS4D4XANCNFSM4UJX6H2A.

I tested these changes, still downloading of all GNSS stations are stopped for few stations.

recommend you to rerun a few times and use an ethernet connection

repo: dbekaert/RAiDER


text:
YOLACT and cuda 10.1

Hi, thanks for you'r amazing work!
When i tray to evaluate i get the following error:
ImportError: libcudart.so.10.0: cannot open shared object file: No such file or director
libcudart.so.10.0 is realy missing becuse libcudart.so.10.1 is installed.
Can i run yolact with cuda 10.1?
thx

That sounds like a CUDA / Pytorch installation problem, not a YOLACT problem. Maybe this thread would be helpful?
You probably have to install a different version of Pytorch or install CUDA 10.0 instead.

Thank!!!
I forgot install torchvision :-\

Thanks!

authors: dbolya sdimantsd

repo: dbolya/yolact


text:
[Bug] SQL compilation error when running --empty flag on on model that utilizes dbt_utils.union_relations() macro

Is this a new bug in dbt-core?

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

Current Behavior
When running the simple model below with the --empty flag I get the following error:
-- models/unioned_model_a.sql
with unioned as (
    {{ dbt_utils.union_relations(
        relations=[ref('source_y__model_a'), ref('source_z__model_a')]
    ) }}
)

select * from unioned

Database Error in model unioned_model_a (models/unioned_model_a.sql)
  001003 (42000): SQL compilation error:
  syntax error line 1 at position 15 unexpected '('.

Expected Behavior
The --empty model should be built successfully like what happens with an explicit union that does not leverage dbt_utils.union_relations()
-- models/unioned_model_a.sql
with unioned as (
    select *
    from {{ ref('source_y__model_a') }}
    union all
    select *
    from {{ ref('source_z__model_b') }}
)

select * from unioned

Steps To Reproduce

Setup a dbt project with Snowflake
Create the model described above that uses dbt_utils.union_relations()
Run dbt run --select unioned_model_a --empty

Relevant log output
23:22:18  Completed with 1 error and 0 warnings:
23:22:18
23:22:18    Database Error in model silver_truck_visits (models/unioned_model_a.sql)
  001003 (42000): SQL compilation error:
  syntax error line 1 at position 15 unexpected '('.
23:22:18
23:22:18  Done. PASS=0 WARN=0 ERROR=1 SKIP=0 TOTAL=1

Environment
- OS: macOS 14.7
- Python: 3.10.11
- dbt: dbt-core=1.8.6, dbt-snowflake=1.8.3

Which database adapter are you using with dbt?
snowflake
Additional Context
A very similar bug was brought up in #10673 and solved by tacking .render() onto the end of the relation reference in the pre-hook causing trouble.
If I try and translate that solution to the dbt_utils.union_relations() macro here I get a failure because the union_relations macro is expecting a relation:
-- models/unioned_model_a.sql
with unioned as (
    {{ dbt_utils.union_relations(
        relations=[ref('source_y__model_a').render(), ref('source_z__model_a').render()]
    ) }}
)

Compilation Error in model unioned_model_a (models/unioned_model_a.sql)
  Macro union_relations expected a Relation but received the value: my_db.my_schema.source_y__model_a

  > in macro _is_relation (macros/jinja_helpers/_is_relation.sql)
  > called by macro default__union_relations (macros/sql/union.sql)
  > called by macro union_relations (macros/sql/union.sql)
  > called by model unioned_model_a (models/unioned_model_a.sql)

Even if the above solution had worked, I would still feel like needing to tack on the .render() after every relation would be more of a workaround than a solution for this example.

Thanks for reporting this @datadisciple !
I was able to reproduce what you described using both dbt-snowflake and dbt-databricks. But it doesn't affect dbt-postgres, dbt-redshift, or dbt-bigquery. See "reprex" below for details.
Digging deeper
Looking at the logs in logs/dbt.log, here's the SQL that triggers the error on Snowflake:
describe table (select * from analytics_dev.dbt_dbeatty.my_model where false limit 0)

Here's the SQL that triggers the error on Databricks:
    describe table (select * from `hive_metastore`.`dbt_dbeatty`.`my_model` where false limit 0) _dbt_limit_subq_my_model

Looking at the source code for dbt_utils.union_relations more closely, it looks like this is the line that is ultimately triggering the error in dbt-snowflake and dbt-databricks when it reaches here and here, respectively.
It looks to me like this is solved in dbt-snowflake in https://github.com/dbt-labs/dbt-snowflake/pull/1100 (which hasn't been released yet), but would still need a similar code update in dbt-databricks here:
    describe table {{ relation.render()|lower }}

Workarounds in the meantime
The following overrides worked for me until these are released in dbt-snowflake and dbt-databricks:
macros/overrides/snowflake/get_columns_in_relation.sql
{% macro snowflake__get_columns_in_relation(relation) -%}
  {%- set sql -%}
    describe table {{ relation.render() }}
  {%- endset -%}
  {%- set result = run_query(sql) -%}

  {% set maximum = 10000 %}
  {% if (result | length) >= maximum %}
    {% set msg %}
      Too many columns in relation {{ relation.render() }}! dbt can only get
      information about relations with fewer than {{ maximum }} columns.
    {% endset %}
    {% do exceptions.raise_compiler_error(msg) %}
  {% endif %}

  {% set columns = [] %}
  {% for row in result %}
    {% do columns.append(api.Column.from_description(row['name'], row['type'])) %}
  {% endfor %}
  {% do return(columns) %}
{% endmacro %}

macros/overrides/databricks/get_columns_in_relation.sql
{% macro get_columns_comments(relation) -%}
  {% call statement('get_columns_comments', fetch_result=True) -%}
    describe table {{ relation.render()|lower }}
  {% endcall %}

  {% do return(load_result('get_columns_comments').table) %}
{% endmacro %}

Reprex
Create these files:
models/my_model.sql
select 1 as id

models/my_model_unioned.sql
with unioned as (
    {{ dbt_utils.union_relations(
        relations=[ref("my_model")]
    ) }}
)

select * from unioned

Run these commands:
dbt run -s +my_model_unioned --empty

Summary
Closing in favor of https://github.com/dbt-labs/dbt-snowflake/issues/1033.

super insightful, thanks @dbeatty10!

repo: dbt-labs/dbt-core


text:
[CT-801] [Feature] The model argument to generic tests should always be a relation

Is this your first time opening an issue?

[X] I have read the expectations for open source contributors

Describe the Feature
Me:

Hi Jeremy just want to run something by you - it would seem like the get_where_subquery macro can return either a relation object or a str object, is that correct? A project I'm working with has overridden that macro to add some filtering for generic tests, but it's then causing an issue where the model argument to generic tests is no longer a relation type but is just a string. Would it be better if the get_where_subquery macro was designed to just alter the string representation of the relation, rather than replacing the relation with a string object altogether?

Jeremy:

You're totally right, that is exactly what happens [...] This feels like a really good idea! Would just require a modification to BaseRelation.render, I think. Trying to think of how it might go wrong, but it's passing most mental checks...

Me:

It's causing an issue with https://github.com/elementary-data/dbt-data-reliability for example, which attempts a {%- set model_relation = dbt.load_relation(model) %} on the model argument to the test. That model argument becomes a string type depending on what get_where_subquery does.

This feature encompasses a couple of things which I'd love to see:

The response from get_where_subquery is not passed directly as the model argument to generic tests, but is set as the string representation of the relation object which is passed to generic tests (non breaking change unless someone relies on the argument being a string type not a relation, but I highly doubt this considering that the argument can be either a relation or a string type today)
An attribute on the relation object which always represents its location in the database. This would be unaffected by the result of get_where_subquery.

Describe alternatives you've considered
No response
Who will this benefit?
Anyone who writes custom generic tests that would like to either ignore the response of the get_where_subquery macro, or rely on access to the relation object (which is what the Elementary package does).
Are you interested in contributing this feature?
No response
Anything else?
No response

Adding this to the top of the get_where_subquery works as a temporary fix:
  {% if model | selectattr("package_name", "equalto", "elementary") %}
  {% do return(model) %}
  {% endif %}

@NiallRees Thanks for opening!
I think the most desirable outcome here manages to accomplish two outcomes:

The {{ model }} argument to generic tests is always a Relation, regardless of the config set — so that authors of generic tests can comfortably pass it into macros like get_columns_in_relation
Every generic test can support the where config, without needing to make any changes to the generic test definition — because, if where config is specified

I'm honestly not sure if there's a way to make that possible, but that's what feels desirable to me!
Maybe that looks like a new Relation method/property to accept a subquery/filter, which then updates the output of Relation.render(), while still preserving the Relation object itself — while still giving users a way to render the relation's database location, without the subquery / filter. (In the worst case, they'd still have direct access to all attributes and methods of the Relation, including database.schema.identifier.)
I don't think this is something we're going to be able to prioritize on our end—but I think the code for it is relatively self-contained, the test cases are highly straightforward, and the biggest risk is the potential blast radius of mucking with BaseRelation (and object used everywhere). Maybe a new subclass, just for tests...? Probably a bad idea :)


text:
[CT-1334] [Feature] Add entire dbt command under single to invocation_args_dict

Is this your first time submitting a feature request?

[X] I have read the expectations for open source contributors
[X] I have searched the existing issues, and I could not find an existing issue for this feature
[X] I am requesting a straightforward extension of existing dbt functionality, rather than a Big Idea better suited to a discussion

Describe the feature
I previously implemented adding invocation args dict to the ProviderContext Class
This returns a JSON object that looks like the following:
{'write_json': True, 'use_colors': True, 'printer_width': 80, 'version_check': True, 'partial_parse': True, 'static_parser': True, 'profiles_dir': '/Users/jerco/.dbt', 'send_anonymous_usage_stats': False, 'event_buffer_size': 100000, 'quiet': False, 'no_print': False, 'parse_only': False, 'which': 'compile', 'rpc_method': 'compile', 'indirect_selection': 'eager', 'select' : ['+jercos_awesome_model'}
I would like to extend this to add a single key with the full dbt command as the value:
{'dbt_invocation_command': 'dbt run --select jercos_awesome_model'}
rather than having the flags passed to dbt available under different keys as they currently are:
{'select': ['+jercos_awesome_model'], 'exclude': ['jareds_terrible_model}
Describe alternatives you've considered

Creating a macro that joins all the elements back together again
Rewriting the implemention in dbt-core to do this joining when invocation_args_dict is called

I'm a little reluctant to do this joining back together as whenever a new flag is added / removed I think it would require a code update and release
Who will this benefit?

Users of the dbt_artifacts package
Anyone that wants to track the full invocation argument for whatever purposes they wish

Are you interested in contributing this feature?
Yes
Anything else?
No response

@jared-rimmer Cool idea! I can see the benefit of: "Run exactly this CLI command, and you're guaranteed to be using exactly the same config as this invocation."
I could be mistaken, but I don't think we have an elegant way to convert a set of resolved flags/configs back into a set of CLI flags — e.g. to figure out that 'use_colors': False should mean --no-use-colors.
Good news: We are working on a new CLI, powered by click, with more programmatic definition of each flag/parameter. I see two options here:

Add a to_cli_params method of the Flags class, so that we can serialize (?) a Flags object as a CLI string
Allow the dbt-core CLI to accept a dictionary of flags (!), so that there's no need to perform the dict-to-string conversion: dbt --flag-dict {'write_json': True, ...}

@iknox-fa If you happen to have a moment, I'd be curious to hear your thoughts on which of those you prefer (or a third option I haven't thought of)

jercos_awesome_model

:heart:

I could be mistaken, but I don't think we have an elegant way to convert a set of resolved flags/configs back into a set of CLI flags — e.g. to figure out that 'use_colors': False should mean --no-use-colors.

You're correct, we don't have a way to do this and given the complexity of the various factors that might modify the flags, it's probably not a good idea to try to create a system to do so, but luckily we don't have to.
Either of your solutions are easy enough to implement (in fact, I don't see a reason not to do both).

If we want to track the actual string provided on the cli we should simply add it to flags from sys.argv[1:]. a single line change for the click side of things and probably not much more than that for the legacy flags module.
Adding a --flag-dict flag would also be fairly easy, although more complex than option 1.  The main issue would be that the parameters provided would have to be injected into a click context which was then forwarded to the necessary subcommand.

Mind reopening this one @jtcohen6? Got a PR here: https://github.com/dbt-labs/dbt-core/pull/7939

authors: NiallRees iknox-fa jared-rimmer jtcohen6


text:
make column name to be fully qualified to prevent BQ confusion

resolves #173
This is a:

[ ] documentation update
[x] bug fix with no breaking changes
[ ] new functionality
[ ] a breaking change

All pull requests from community contributors should target the main branch (default).
Description & motivation
For better support for an edge case in Bigquery, which gets confused by casting a column with the same name as its table name.
Checklist

[x] This code is associated with an Issue that has been triaged and accepted for development.
[x] I have verified that these changes work locally on the following warehouses (Note: it's okay if you do not have access to all warehouses, this helps us understand what has been covered)

[x] BigQuery
[ ] Postgres
[ ] Redshift
[ ] Snowflake

[x] I followed guidelines to ensure that my changes will work on "non-core" adapters by:

[x] dispatching any new macro(s) so non-core adapters can also use them (e.g. the star() source)
[x] using the limit_zero() macro in place of the literal string: limit 0
[x] using dbt.type_* macros instead of explicit datatypes (e.g. dbt.type_timestamp() instead of TIMESTAMP

[x] I have updated the README.md (if applicable)
[x] I have added tests & descriptions to my models (and macros if applicable)
[x] I have added an entry to CHANGELOG.md

I am not so sure why the CI is failing and giving irrelevant error(s).

repo: dbt-labs/dbt-utils


text:
Align Snowflake docs content

Contributions

[X] I have read the contribution docs, and understand what's expected of me.

Link to the page on docs.getdbt.com requiring updates
In the dbt docs (here) for the 'Overview of dbt-Snowflake' base configs for client_session_keep_alive and the Snowflake docs (here) for 'CLIENT_SESSION_KEEP_ALIVE' seem to offer differing descriptions.
What part(s) of the page would you like to see updated?
We should get these docs aligned. Thank you @amychen1776 for looking into this!
Additional information
No response

Closed with this PR https://github.com/dbt-labs/docs.getdbt.com/pull/1705


text:
Start an About References landing page

Contributions

[ ] I am a PM or subject matter expert at dbt who is responsible for this feature.

Where does this content belong?
docs.getdbt.com/references
Link to source material
docs.getdbt.com/references

This is awesome @matthewshaver - Leona asked me to provide any feedback. So far i have:

Def agree on a landing page and I personally think this landing page should have cards with a quick sentence of what each card (or section) represents (similar to our dbt features or dbt deploy jobs pages) to give the users a simple overview of what this section is and use cards to have them link to the relevant sections.
I also think a 'related docs' section may be appropriate. Eg. guides, quickstarts, dbt cloud section, dbt core section.

hey @matthewshaver , i've created a landing page for this already so I'll add myself as the assignee if that's cool! cc @runleonarun


text:
Improve enterprise permissions table based on user feedback

Contributions

[X] I have read the contribution docs, and understand what's expected of me.

Link to the page on docs.getdbt.com requiring updates
@nehahystad raised the enterprise permissions table doc has scope of improvement based on user feedback saying the table is difficult to parse.
The writer should work with Neha to further improve the table for users so it's accurate, clear, and easy to follow.
@b-per had some additional feedback on this:

Ideally, we would see that a Developer can “run the IDE”,  and “see environments”, and “review/extract/download job logs” rather than it can W:Develop, W:Jobs etc (more in the thread)
Our RBAC page doesn’t list who can and can’t set external notifications but I am quite sure that this is only Account Admin. source for add'l context

What part(s) of the page would you like to see updated?
https://docs.getdbt.com/docs/cloud/manage-access/enterprise-permissions
Additional information
No response

related issues: https://github.com/dbt-labs/docs.getdbt.com/issues/2795 and https://github.com/dbt-labs/docs.getdbt.com/issues/2796

additional question related to owner role from @kbrock91 : https://dbt-labs.slack.com/archives/C0186FAQTMF/p1721223051873219

Additional feedback is clarifying the difference between data platform connections and data platform configurations in the docs longer term:

Matt to address post coalesce


text:
dbt-core incremental model with --full-refresh failing on Azure Synapse after initial creation

After successfully creating and running an incremental table model using dbt-core and the dbt-synapse adapter, if I attempt to recreate the table by using the --full-refresh command line option, I get an error:
[42S01] [Microsoft][ODBC Driver 17 for SQL Server][SQL Server]There is already an object named 'real table name here' in the database. (2714) (SQLExecDirectW)"
It appears that the expected "drop table" script component is not executing?

Hi @geoff-muldoon This issue repo is for our docs.getdbt.com site. Is there a page there you believe is leading you astray?

authors: geoff-muldoon matthewshaver


text:
combine-state-result-selector-description

Description & motivation

@bethanyhipple-dbtlabs and I discovered that the description of the command should be OR not AND because the set operator used is a   and not a ,

Thank you @jeremyholtzman and @bethanyhipple-dbtlabs for catching and correcting this.

authors: jeremyholtzman matthewshaver


text:
Update data-tests.md

Hi.
This modification proposal adds for the "accepted_values" test, a feature that it shares with the Unique test, not present in the text. Regards.
IL.
What are you changing in this pull request and why?

Checklist

[x] Review the Content style guide so my content adheres to these guidelines.
[ ] For docs versioning, review how to version a whole page and version a block of content.
[ ] Add a checklist item for anything that needs to happen before this PR is merged, such as "needs technical review" or "change base branch."

Adding or removing pages (delete if not applicable):

[ ] Add/remove page in website/sidebars.js
[ ] Provide a unique filename for new pages
[ ] Add an entry for deleted pages in website/vercel.json
[ ] Run link testing locally with npm run build to update the links that point to deleted pages

Created as issue to be addressed https://github.com/dbt-labs/docs.getdbt.com/issues/5892

authors: ialdg matthewshaver


text:
Pedida de opinión en el uso de BETO

Hola! Escribo para pedir una opinión respecto del uso de BETO.
Se las adelanto, hace sentido un fine tuninng en este contexto?
El contexto
Estoy construyendo un sistema recomendador de propuestas ciudadanas. El asunto es que tengo una base de datos de unas 700 propuestas escritas por ciudadanos durante las elecciones del 2017, estás propuestas tienen un título y una descripción, las cuales vectorizo usando BETO y construyo un sistema para recuperar las propuestas más cercanas.
Saludos y muchas gracias por esto.

Hola! En mi opinión, casi siempre es necesario un fine-tuning porque tomás de base el modelo pre-entrenado de BETO y lo ajustás a tu necesidad.
Recordá que BETO es un modelo de red neuronal multipropósito.

Muchas gracias por tu respuesta.

repo: dccuchile/beto


text:
Error: Cannot find module 'rimraf'

Hello, i have an issue with installing node-red-contrib-zeromq . I tried to install it from palette and i show me that :
"

zeromq@5.2.8 install /usr/lib/node_modules/node-red/node_modules/node-red-contrib-zeromq/node_modules/zeromq
node-gyp-build || npm run build:libzmq

gyp ERR! UNCAUGHT EXCEPTION
gyp ERR! stack Error: Cannot find module 'rimraf'
gyp ERR! stack     at Function.Module._resolveFilename (module.js:489:15)
gyp ERR! stack     at Function.Module._load (module.js:439:25)
gyp ERR! stack     at Module.require (module.js:517:17)
gyp ERR! stack     at require (internal/module.js:11:18)
gyp ERR! stack     at Object. (/usr/lib/node_modules/node-red/node_modules/node-gyp/lib/clean.js:10:10)
gyp ERR! stack     at Module._compile (module.js:573:30)
gyp ERR! stack     at Object.Module._extensions..js (module.js:584:10)
gyp ERR! stack     at Module.load (module.js:507:32)
gyp ERR! stack     at tryModuleLoad (module.js:470:12)
gyp ERR! stack     at Function.Module._load (module.js:462:3)
gyp ERR! System Linux 4.14.9-klk
gyp ERR! command "/usr/bin/node" "/usr/lib/node_modules/node-red/node_m... "
thnx for answering :)

any clues like which version of nodejs, npm, Node-RED , platform etc ?

I m implementing this api in a  gateway using : nodejs (version v8.4.0) , nodered (verion v0.18.1), npm (version 5.3.0) in an os keros (kind of linux distibution " Linux 4.14.9-klk arm LE")

node 8 went out of support a while back. I guess you could try manually installing the rimraf package to your ~/.node-red directory and then re-installing.

Thanks for the support :)

authors: aaffany dceejay

repo: dceejay/node-red-contrib-zeromq